download from Mapbox endpoint pulls .webp files

Hello, I have been working through your diary example on Tanzania but I want to pull from Mapbox instead as that is closer to what I'll ultimately be doing. I ran the command:
rs download https://api.mapbox.com/v4/mapbox.satellite/{z}/{x}/{y}@2x.webp?access_token=[API_KEY] output_folder_name
But I get back .webp files rather than .png files. I am less experienced with Mapbox/OSM and I'm coming from a deep learning background so I'd prefer to work directly with png (or jpeg) files so I can try some different segmentation models outside of what is available in Robosat (and potentially try to build out some features in the process). Can I change my rs download line in some way to get png/jpeg?
Thanks for building such a great product!

Hi @CooperNederhood - check out the "Retrieve Tiles" documentation for details on how to pull different tile formats.
Gonna close here, but please reopen if ^ doesn't answer your question.

Thanks for the link, that was good information to know. However, when I changed my command to:
rs download https://api.mapbox.com/v4/mapbox.satellite/{z}/{x}/{y}@2x.png?access_token=[API_KEY] output_folder_name
I still was pulling .webp files. However, when I added the --ext option and ran the below command, I receive png files. I seem to be getting the output I wanted, but just wanted to point this out
rs download --ext png https://api.mapbox.com/v4/mapbox.satellite/{z}/{x}/{y}@2x.png?access_token=[API_KEY] output_folder_name

The .webp URl results in Mapbox sending a webp image file over the wire.
The --ext extension flag results in us saving the image file in the specified format.
We provide the ext flag so we not tied to tile endpoint formats.

Hello. I am not sure if that changed during last year, but the current version of Mapbox Raster API Documentation says that "Tiles that include mapbox.satellite are always delivered as JPEGs, even if the URL specifies PNG". That can be proven by using gdalinfo example_raster.png that will return:
Driver: JPEG/JPEG JFIF
Files: example_raster.png
Size is 512, 512
Image Structure Metadata:
  COMPRESSION=JPEG
  INTERLEAVE=PIXEL
  SOURCE_COLOR_SPACE=YCbCr
Corner Coordinates:
Upper Left  (    0.0,    0.0)
Lower Left  (    0.0,  512.0)
Upper Right (  512.0,    0.0)
Lower Right (  512.0,  512.0)
Center      (  256.0,  256.0)
Band 1 Block=512x1 Type=Byte, ColorInterp=Red
  Overviews: 256x256, 128x128
  Image Structure Metadata:
    COMPRESSION=JPEG
Band 2 Block=512x1 Type=Byte, ColorInterp=Green
  Overviews: 256x256, 128x128
  Image Structure Metadata:
    COMPRESSION=JPEG
Band 3 Block=512x1 Type=Byte, ColorInterp=Blue
  Overviews: 256x256, 128x128
  Image Structure Metadata:
    COMPRESSION=JPEG

Moreover, as we can see this image is not geo-referenced (any info about CRS). The same is shown by rasterio crs which returns None. Could anyone please confirm that API returns not geo-referenced rasters?

Slippy Map tiles are geo-referenced by means of their tile ids, i.e. z,x,y in /z/x/y.png. Please reads e.g. the OSM wiki docs on that.
Re. the Mapbox API I don't know if they have changed anything but in the past it was possible to download webp image tiles.
Good luck.
On June 7, 2020 8:22:37 AM UTC, radoslawkrolikowski<EMAIL_ADDRESS>wrote:

Hello. I am not sure if that changed during last year, but the current
version of Mapbox Raster API Documentation says that "Tiles that
include mapbox.satellite are always delivered as JPEGs, even if the URL
specifies PNG". That can be proven by using gdalinfo example_raster.png that will return:
Driver: JPEG/JPEG JFIF
Files: example_raster.png
Size is 512, 512
Image Structure Metadata:
 COMPRESSION=JPEG
 INTERLEAVE=PIXEL
 SOURCE_COLOR_SPACE=YCbCr
Corner Coordinates:
Upper Left  (    0.0,    0.0)
Lower Left  (    0.0,  512.0)
Upper Right (  512.0,    0.0)
Lower Right (  512.0,  512.0)
Center      (  256.0,  256.0)
Band 1 Block=512x1 Type=Byte, ColorInterp=Red
 Overviews: 256x256, 128x128
 Image Structure Metadata:
   COMPRESSION=JPEG
Band 2 Block=512x1 Type=Byte, ColorInterp=Green
 Overviews: 256x256, 128x128
 Image Structure Metadata:
   COMPRESSION=JPEG
Band 3 Block=512x1 Type=Byte, ColorInterp=Blue
 Overviews: 256x256, 128x128
 Image Structure Metadata:
   COMPRESSION=JPEG

Moreover, as we can see this image is not geo-referenced (any info
about CRS). The same is shown by rasterio crs which returns None.
Could anyone please confirm that API returns not geo-referenced
rasters?


FEATURE: Add Endpoint to List Users in an Organization

Description
This pull request introduces a new API endpoint api/v1/organizations/:orgId/users to the application. This endpoint allows users to retrieve a list of all users associated with a specific organization.
Details
Endpoint: GET /api/v1/organizations/:orgId/users
Description: Retrieves a list of all users in the specified organization.
Parameters:
orgId (Path Parameter): The unique identifier of the organization for which users are being requested.

Related Issue (https://github.com/hngprojects/hng_boilerplate_python_fastapi_web/issues/78)

Testing
run pytest tests/v1/test_org_users.py to run tests

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@Dev-wonderful
@bolexs
Pls review and merge

@Freeman-kuch
Pls check this
It's for the Auth test

The workflow is failing on your pr, fix that then I will take a look


Question on Functionality: Closing and Opening Portals

👋
I have a question regarding expected functionality.
I have a project folder opened in Atom. I then:

Opened a file
Opened a portal
Folks joined

Then I wanted to close that and get ready for my next session, so I:

Closed the portal (stopped sharing)
Closed that file
Opened another file
Started sharing again

When I did this

Started sharing again

I noticed that the person that joined the first time were still able to access the new file I had opened.

It seems that the URLs were the same for both times:
atom://teletype/portal/f0e1e689-71b8-4448-9287-d80d99653130
And new folks are not able to join.
Maybe the portal was "closed" but not all the way? I'm not sure.
I closed Atom and restarted it and was able to properly open a new portal.
So maybe this is a bug after all.
Atom: 1.22.1
Teletype: 0.12.0
I also received this error in the initial sharing session.

I have a question regarding expected functionality.

I think your expectations are correct. Closing a portal should cease all sharing. 😅 All of your steps were reasonable steps to obtain the desired effects, so thanks for letting us know that you ran into this incorrect behavior.

I also received this error in the initial sharing session.

Gotcha. We're tracking that issue in https://github.com/atom/teletype/issues/360.

And new folks are not able to join.
Maybe the portal was "closed" but not all the way? I'm not sure.
I closed Atom and restarted it and was able to properly open a new portal.

Given that you experienced that error while collaborating in the first portal, I suspect the error likely prevented the portal from being closed, and prevented the new one from opening properly. Because of that I'm going to close this issue. (I'm pretty sure #360 is the real culprit.) If you experience this behavior in the absence of the issue you observed in #360, please let us know, and I'll re-open this issue.


Now setColor will set background color of label.

When calling setColor(), the background color of the label is not updated.

Add a color picker
Call setColor from your code
-> now the color number on qs_label, will update but not the background color of qs_color_label.

Repro on CodePen: http://codepen.io/DonKarlssonSan/pen/ALqpQZ
I'm not sure whether this is the most elegant fix but it seems to work. =)

Thanks! I just fixed that in the "next" branch, which will be merged to master shortly.

Can't get quicksetting.js fixed in the master repository(?)
setColor() doesn't update the label yet...
Thanks for your reply


Feat/add strict option to date format validation

What is this?
I used the date_format validation rule specifying the RFC::3339 method, and there was a non-intuitive behavior. It was previously raised as a bug in a Lumen Issue, but it seems it hasn't been fixed yet, so I addressed it.
Details
About RFC3339 formatting
Whether to make the date_format validation rule strict or not changes the result when, for example, the format is specified as RFC3339.
As a premise, RFC3339 is in the form of 'Y-m-d\TH:i:sP', but in this format, there are two patterns for expressing the date: one uses 'Z' to indicate Coordinated Universal Time (UTC), and the other uses a time offset of '+00:00'.
The conventional implementation
The date_format validation rule checks whether it can parse with the specified format using DateTime::createFromFormat(). However, because there are two ways to represent UTC, there is a pattern where something that should succeed in the inspection fails.
In the conventional implementation of date_format, after parsing with DateTime::createFromFormat(), it verifies whether the reformatted string perfectly matches the input value.
// Parse the input with the specified format.
$date = DateTime::createFromFormat('!'.$format, $value);

// Inspect to ensure it has been parsed correctly and that the reformatted parsing result matches the input.
if ($date && $date->format($format) == $value) {
	return true;
}

Now, let's parse and reformat the following two types of date strings with RFC::3339.
2023-01-01T00:00:00+00:00
$date = DateTime::createFromFormat('!'.DateTime::RFC3339, '2023-01-01T00:00:00+00:00');
echo $date->format(DateTime::RFC3339) . "\n";

// The output is "2023-01-01T00:00:00+00:00"

2023-01-01T00:00:00Z
$date = DateTime::createFromFormat('!'.DateTime::RFC3339, '2023-01-01T00:00:00Z');
echo $date->format(DateTime::RFC3339) . "\n";

// The output is "2023-01-01T00:00:00+00:00"

In this way, in PHP, dates written in +00:00 and those written in Z are both reformatted to +00:00 notation. Therefore, even values that should pass the inspection can fail in the old implementation because they are treated as not complying with the format.
Here, from PHP8.0, in addition to the "P" format, the "p" format has been added.
https://php.watch/versions/8.0/date-utc-p-format
Both of these represent the time difference from GMT, but if you use p, it will return Z instead of +00:00. Now, let's replace the P used in the previous validation with p and perform the same validation.
2023-01-01T00:00:00+00:00
$format = str_replace('P', 'p', DateTime::RFC3339);
$date = DateTime::createFromFormat('!'.$format, '2023-01-01T00:00:00+00:00');
echo $date->format(DateTime::RFC3339) . "\n";

// The output is "2023-01-01T00:00:00Z"

2023-01-01T00:00:00Z
$format = str_replace('P', 'p', DateTime::RFC3339);
$date = DateTime::createFromFormat('!'.$format, '2023-01-01T00:00:00Z');
echo $date->format(DateTime::RFC3339) . "\n";

// The output is "2023-01-01T00:00:00Z"

With this, if you enter in Z notation, it will be reformatted in Z notation. However, even those entered in +00:00 notation will return as Z when using the p format, so the date_format inspection will not succeed after all. I think it's not recommended to branch these processes within validateDateFormat.
What I did this time.
Considering the above, I have divided the strictness of the date_format check into two levels:

Default: Loose check

It simply checks whether the parsing is successful with the specified format.

Option: Strict check

Like the original implementation, it checks whether the parsing is successful with the specified format, and furthermore, whether the reformatted value is equal to the input value.

By making the strict check optional, a normal validation can simply check whether the input complies with the specified format, enabling a safe verification that is not dependent on the language specifications of PHP.
The reason why we have been checking whether the reformatted result matches the input.
DateTime::createFromFormat() often returns quite lax results. Normally, if it cannot parse, DateTime::createFromFormat() returns false, but for example, in the following case, it returns an instance of DateTime instead of false.
$date = DateTime::createFromFormat('Y-m-d H:i:s', '2023-13-14 30:60:70');
var_dump($date);

object(DateTime)#1 (3) {
  ["date"]=>
  string(26) "2024-01-15 07:01:10.000000"
  ["timezone_type"]=>
  int(3)
  ["timezone"]=>
  string(16) "Europe/Amsterdam"
}

Because of such cases, it was necessary to verify that the value reformatted from the parsed DateTime instance matches the input.
However, such unexpected successful parses can be handled using DateTime::getLastErrors().
$date = DateTime::createFromFormat('Y-m-d H:i:s', '2023-13-14 30:60:70');
var_dump(DateTime::getLastErrors());

array(4) {
  ["warning_count"]=>
  int(2)
  ["warnings"]=>
  array(1) {
    [19]=>
    string(27) "The parsed date was invalid"
  }
  ["error_count"]=>
  int(0)
  ["errors"]=>
  array(0) {
  }
}

Even when DateTime::createFromFormat() does not return false, warnings and errors can be captured using DateTime::getLastErrors(). Therefore, in this pull request, I used this method to ensure a minimum level of inspection accuracy even in the case of loose checks.

I'm currently fixing an issue where tests that didn't fail locally are failing on CI.

I understand that the RFC3339 format is a bit tricky.
IMO, I just don't think the new default (loose) behavior is intuitive.
00-01-01 or 1-1-1 passes the 'date_format:Y-m-d' validation.
You know, the default behavior of alpha isn't intuitive.
Also if we want to keep the same behavior as before when we migrate to Laravel ver.11, we need to add the strict option which can be cumbersome.
Instead of adding the strict option, why don't we just add a loose option?
When you want to validate the RFC3339 format, you just need to add the loose option.
Well, that's just my personal opinion.
Btw, thank you for your last PR. It was really nice.😊

@nshiro
Thank you for your comment!
As you say, making the strict check optional could indeed be a valid option. However, my viewpoint is that for the date_format validation rule, whose main purpose is to check if "the input follows the format", in most cases we won't require this strict option. In other words, the current system may be "too strict".
Regarding parsing "00-01-01" or "1-1-1" with Y-m-d, when I tested it, DateTime::createFromFormat() returned false. So, even with a lenient check, I believe it wouldn't pass the validation.

You are correct, there may be cases where work will be needed when transitioning to Laravel 11.x. Taking that into account, I made this pull request so that it would be released not in the current 10.x, but when the major version is updated.
PS: Is your last PR referring to the discussion about getRawQueryLog()? I'm honored by your praise!

@fuwasegu
Thank you for your replay.
As for your screenshot, the order of the arguments is reversed.
You put the !Y-m-d in the first argument.
Well in my cases, after validating the date format, I would store the data in the database.
From what I have found out, as for MySQL these loose formats look ok, but for PostgreSQL (I tested with PostgreSQL 13.5 ) I got an error like below.
foo=# INSERT INTO hoge (start_date, end_date) VALUES ('00-01-01', '1-2-28');
ERROR:  date/time field value out of range: "00-01-01"
LINE 1: INSERT INTO hoge (start_date, end_date) VALUES ('00-01-01', ...
                                                        ^
HINT:  Perhaps you need a different "datestyle" setting.
foo=#
foo=#
foo=# INSERT INTO hoge (start_date, end_date) VALUES ('2000-01-01', '2001-02-28');
INSERT 0 1

So if you are using PostgreSQL, you need the strict option if I understand correctly.
Yes. I meant getRawQueryLog.😅

Oops, I made a very embarrassing mistake😅.
Indeed, you are right, a loose inspection may not be sufficient because not only can it be parsed, but there is no warning!
I will consider making the loose inspection optional!
Thank you for your precise point of view!

And if we make loose inspection an option, we could make it a pull request for 10.x, since it is not a destructive change.

Based on the feedback I received, I have reverted the default to the strict validation as before and made the loose validation an option.

@adideas
Thank you for your code review!

If you need date validation beyond what Laravel currently offers, you can easily write a custom validation rule.

@taylorotwell
RFC3339 is a format natively supported by PHP's DateTime class. However, the fact that it cannot be accurately validated is, in a way, a flaw in the framework itself. Of course, you can address this with custom rules, but I believe that if a rule like "date_format" exists, it should be able to handle the basic formats defined in standards, even if this capability is optional.


Office Plugin - Data Transfer Screen Blank

System information:

Operating system (distribution) and version: Windows 10 1909
DBeaver version 7.1.2
Java version
java version "1.7.0_51"
Java(TM) SE Runtime Environment (build 1.7.0_51-b13)
Java HotSpot(TM) 64-Bit Server VM (build 24.51-b03, mixed mode)
Additional extensions
Darkest Dark
Office Integration

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
With both the Darkest Dark theme installed and the Office plugin.  When selecting Execute -> Export from Query the Data Transfer screen is blank.

Steps to reproduce, if exist:

install Darkest Dark
install Office integration
open new sql query
right click and Execute -> Export From Query

Include any warning/errors/backtraces from the logs

thanks for the bug report
close as duplicate of #8113


Configure the pod security context in the Falco Sidekick Deployment

What type of PR is this?
/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release
Any specific area of the project related to this PR?
/area falcosidekick
What this PR does / why we need it:
A pod security context is useful in EKS when we want to mount the ServiceAccount token and use it to get our AWS credentials. Without the fsGroup, the ServiceAccount token is not accessible in the container.
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign fntlnz
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @JPLachance! It looks like this is your first PR to falcosecurity/charts 🎉

LGTM label has been added.  Git tree hash: 08dc05bb373a843ef4db7b5b08c7f9b190f77ee5

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif,leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


How to uninstall?

Hi,
How do I uninstall CO?

It's simple.
Go to directory where Steam is installed, then to steamapps/sourcemod and delete csco folder.

Thanks!
I appreciate the installer btw. Would be nice to have an uninstall button as well.


fix serialization of rest query key value when object

Query param should be string. The only exception is array of strings. I don't think we need to allow/handle other cases.
@tomeresk , WDYT?


Incorrect hint on local variable declaration

This local variable declaration causes a problem for the analyzer.
  var fns = <Function()>[];

The analyzer says:

info: The return type of ' Function()' cannot be inferred. (inference_failure_on_function_return_type at [experiments] lib/patterns.dart:239)

I checked with @munificent and was assured that the code is correct. It comes from the patterns spec.

The return type of the type cannot be inferred, so should be specified. See the last example at https://github.com/dart-lang/language/blob/master/resources/type-system/strict-inference.md#function-return-types.
inference_failure_on_function_return_type should only be reported when strict-inference is enabled. Can you tell if it is enabled?

I had forgotten I enabled that option. Closing. Thanks @srawlins !


possible cognate review on a few items?

Hi UraLex team. Thank you for a lovely dataset!
I noticed a few forms in different languages that are identical to each other and pertain to the same parameter, but have been put in different cognate classes. Some of them are very short, so probably cases of different forms being reduced down to the same. Some of them may be loans as well, I don't know . "muž*" I think is a loan perhaps, so that might explain that. But "tuman" for "fog" maybe should be looked over once more?
Table of forms for the same parameters and how many cognate classes the form occurs in.
Form   Parameter_ID     n
1 da              103     2
2 i               103     4
3 ja              103     3
4 mam             154     2
5 mužik           145     2
6 mužik           253     2
7 onu             298     2
8 tolʹko          254     2
9 tuman           133     2

Hi,
Thank you for the comment and feedback. Points like this are valuable so we can critically examine and develop our dataset further.
The items listed are indeed all borrowings as you pointed out. The different cognate groups are intentional but definitely worth double-checking especially if they are confusing.
The multiple cognate classes are mostly caused by parallel borrowing which is quite a tangle. As the cognate column represents vertical relationships, more recent borrowings acquired independently need their own sets. Almost all the words are Russian borrowings which are relatively recent and usually noted in our source literature too. The different groupings are intended to also reflect that a word is likely borrowed from the same donor language at a different time point into different recipient languages. Unfortunately, in source literature not all languages are discussed equally and the stratification of some borrowings is often not very detailed. We have attempted to capture this with the chosen donor language labels and the borrowing certainty tags (clear/probable/possible) also included in the datasheet. For some languages, it is very likely that a word comes from the same origin as in its sister languages but in literature sometimes even closely related languages are discussed with different degree of certainty. This way UraLex is tied to and reflects the state of research too. In addition, some languages lack a strong standard and it has been necessary to include some alternate forms which are very similar but are put in different cognate sets. We are hoping to resolve these compromises in the future.
A few comments on the items:
In the case of ID103 ‘and’ –words, parallel borrowing mainly from Russian (‘I’ and ‘da’).
The grouping of the ‘ja’ word derives from the complex contact situation between Finnic, Saamic and (North) Germanic languages. While the Finnic word is a Germanic borrowing, the Saamic languages have often borrowed it from a Finnic source. For some words, it is also possible that the word has been mediated via a Scandinavian language.
ID154, 145, 253, 245, 153, 254, 133 independent borrowings from Russian likely at different times or at least different geographical locations. Meanings 145 and 253 have the same parameter mužik because for some languages it fills the ‘man’ and ‘husband’ slots.
ID298 ‘uncle’ appears twice, the first being the Estonian word for the meaning belonging to a cognate set with its cognates. The second one is an internal borrowing and is given its own set.
For the upcoming Uralex version, we have corrected and improved the separation of parallel borrowings when possible so there will be more classes. I hope this has clarified the situation a bit. We are always happy to hear about new literature and ideas that help to make Uralex more accurate and usable.

Thanks @MervideHeer ! That sounds great. I'm happy to close the issue if that's ok with you.

Sure! We can close the issue.

Sure! We can close the issue.


Add support for mobile app

Could you please make the plugin compatible with the Obsidian mobile app?

I'm not sure which mobile you are referring to. How do you run pandoc on iOS or iPadOS?

I mean the Obsidian Android app.


Tons of Settings Are Missing Including TypeScript & Javascript

Issue Type: Bug

Open my typescript project in Insiders.
Typescript doesn't work at all and no typescript settings exist when searching settings, nor do the ones in my user settings resolve to anything. It just says "Unknown configuration setting." for my javascript and typescript user settings.

If I open in the normal vscode, it works fine and typescript is there.
There are 864 settings in regular vscode and only 268 settings in insiders vscode. 🤔
VS Code version: Code - Insiders 1.24.0-insider<PHONE_NUMBER>4dcbe9aca0e5ca3ba8a53808b122b6, 2018-06-01T13:37:15.690Z)
OS version: Darwin x64 17.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2700)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
4, 4, 4

Memory (System)
16.00GB (0.33GB free)

Process Argv
/Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (62)

Extension
Author (truncated)
Version

ts-deporter
Acr
1.0.2

react-native-react-redux
EQu
1.9.0

output-colorizer
IBM
0.1.2

json-to-ts
Mar
1.5.5

vscode-extension-auto-import
Nuc
1.4.1

vscode-ios-common-files
Ort
1.0.4

vscode-jest
Ort
2.8.0

vscode-react-native-storybooks
Ort
2.5.1

code-settings-sync
Sha
2.9.2

quokka-vscode
Wal
1.0.128

vscode-file-peek
abi
1.0.1

google-search-ext
ade
0.2.1

project-manager
ale
0.25.2

jest-snippets
and
1.2.0

vscode-yarn-script
car
0.0.1

npm-intellisense
chr
1.3.0

path-intellisense
chr
1.4.2

regex
chr
0.2.0

jsrefactor
cms
2.13.0

vscode-eslint
dba
1.4.10

githistory
don
0.4.1

gitlens
eam
8.3.3

tslint
eg2
1.0.30

vscode-npm-script
eg2
0.3.4

vscode-great-icons
emm
2.1.32

prettier-vscode
esb
1.3.1

vscode-jest-runner
fir
0.2.2

flow-for-vscode
flo
0.8.1

auto-close-tag
for
0.5.6

auto-rename-tag
for
0.0.15

vscode-yarn
gam
1.6.0

vscode-npm-dependency
how
1.2.0

path-autocomplete
ion
1.10.0

language-polymer
jon
1.1.0

types-autoinstaller
jvi
2.4.0

vscode-colorize
kam
0.6.19

refactorix
kri
0.3.6

javascript-test-runner
leg
0.2.0

dotenv
mik
1.0.1

react-javascript-to-typescript-transform-vscode
moh
1.10.0

vscode-stylefmt
mrm
2.5.0

debugger-for-chrome
msj
4.5.0

color-highlight
nau
2.3.0

vscode-code-outline
pat
0.2.1

vscode-versionlens
pfl
0.21.1

tsimporter
pmn
1.2.14

quicktype
qui
10.0.24

typescript-hero
rbb
2.3.2

java
red
0.26.0

code-spell-checker
str
1.6.10

move-ts
str
1.11.3

code-navigation
vik
0.2.0

vscode-java-debug
vsc
0.9.0

vscode-java-pack
vsc
0.3.0

vscode-java-test
vsc
0.6.1

vscode-maven
vsc
0.8.0

vscode-appcenter
vsm
0.1.4

vscode-react-native
vsm
0.6.10

gitblame
wad
2.4.2

vscode-todo-highlight
way
0.5.12

JavaScriptSnippets
xab
1.6.0

propertylist
zho
0.0.2

(1 theme extensions excluded)

I was able to solve this by completely wiping out what was inside /Users/justin/Library/Application\ Support/Code\ -\ Insiders.
I zipped everything that was in there before wiping it in case anyone wants it.

Seems most likely that the Typescript was disabled or broken somehow.

Do you see this if you disable extensions? You have some like setting sync that may have caused this


Add environment.yml for conda based installations

Should help with installation issue reported in #43

This looks good. Merging since Django CI / build (3.9) (pull_request)  failure is due to platform not the software.
and continuous-integration/travis-ci/pr failure is due to the just Travis


[Device ID] The share button does not work

Once a Credential is stored on the Device ID App the Share button does nothing

Fixed in https://github.com/jmcanterafonseca-iota/zebra-iota-edge-sdk/pull/112


'UTF8' is not a supported encoding name

Hello, I get an error 'UTF8' is not a supported encoding name  when I call  this method
            TokenResponse response = await client.RequestAuthorizationCodeAsync(                 code,                 urlRedirection);
Have you an idea how can I  solve it ?
Thanks

Need more context.

`public ActionResult Login(string prompt = null, string acrValues = null)
{
string state = Guid.NewGuid().ToString("N");
string nonce = Guid.NewGuid().ToString("N");
SetTempState(new TempState { state = state, nonce = nonce });
        var request = new RequestUrl(ConfigHelper.IdentityServer.AuthorizeEndpoint);

        string url = request.CreateAuthorizeUrl(
            clientId: ConfigHelper.IdentityServer.ClientId,
            responseType: "code",
            scope: ConfigHelper.IdentityServer.Scopes,
            redirectUri: Request.Url.GetLeftPart(UriPartial.Authority) + Url.Action("LoginCallback", "Account"),
            state: state,
            nonce: nonce,
            acrValues: acrValues,
            prompt: string.IsNullOrEmpty(prompt) ? null : prompt);

        return Redirect(url);
    }`

`            public async Task LoginCallback(string code, string state, string error)
{
TempState tempState = await GetTempStateAsync();
        var urlRedirection = Request.Url.GetLeftPart(UriPartial.Authority) + Url.Action("LoginCallback", "Account");
        TokenResponse response = await client.RequestAuthorizationCodeTokenAsync(
            new AuthorizationCodeTokenRequest() {
                Address = ConfigHelper.IdentityServer.TokenEndpoint,
                Code = code,
                ClientId = ConfigHelper.IdentityServer.ClientId,
                ClientSecret = ConfigHelper.IdentityServer.ClientSecret,
                RedirectUri = urlRedirection
            });`

The method .RequestAuthorizationCodeTokenAsync get from the server "Method not allowed"

Sorry this is not helpful. I have never seen this error message before.
Which version of .NET / platform is that?

.NET 4.5.2 & MVC

Then I have no idea. This works fine for me.

I found something, I called my server without https.
Now the RequestAuthorizationCodeTokenAsync method return
Unexpected character encountered while parsing value: B. Path '', line 0, position 0.  :(

all set?


Add single-page button HTML to the API response

This is so that the javascript can just replace the existing button,
rather than trying to do some more fancy DOM manipulation.

To try this out locally, you'll also need #277

Trello card

This looks good to me, but I'm getting {"message":"No bearer token was provided"} when I try and test locally with curl (I have pulled in the cookie middleware from the linked PR).
I wasn't expecting that based on the docs - am I doing something wrong?

Is your cookie (or header) valid?
I think the include GDS::SSO::ControllerMethods in the ApplicationController is intercepting 401 responses and giving that message.  I'm planning a refactoring PR which adds some different superclasses, since it seems that skipping the authorise_sso_user! action isn't enough to fully remove the gds-sso stuff in this case.

It should be, I'm logged in to http://frontend.dev.gov.uk/account/home and getting the session cookie from there.
Your explanation seems reasonable and the tests cover the invalid and unauthenticated states so I'm going to approve as to not block the last few bits of work, and I'll work out what's going on on my local setup separately.


Confused about how to externalize babel helpers in certain circumstances

I'm attempting to create a library of vue components. I'm using the @vue/preset-app which automatically includes the transform-runtime babel plugin. So, I had set runtimeHelpers to true to get the build to run. But I would still like to externalize the babel-helpers so they're not bundled with the library. I have a @babel/runtime dependency for the library. So I'm allowing the consumer app to handle resolving these dependencies in the most bundle-size efficient manner.
However, right now, babel is transforming runtime imports into a relative path to the closest @babel/runtime package it can find in my library node_modules. For example, import _taggedTemplateLiteral from '../../../../../../node_modules/<EMAIL_ADDRESS>How can I get it to just transform this to import _taggedTemplateLiteral from '@babel/runtime'?
My rollup config, in case it's helpful:
import path from 'path';
import babel from 'rollup-plugin-babel';
import nodeResolve from 'rollup-plugin-node-resolve';
import alias from 'rollup-plugin-alias';
import vuePlugin from 'rollup-plugin-vue';
import jsonPlugin from 'rollup-plugin-json';

const pkg = require('./package.json');

const version = pkg.version;

const babelConfig = {
  presets: ['@vue/app'],
  externalHelpers: true,
  runtimeHelpers: true,
  exclude: '!(node_modules/@storybook/addons/src/index.js)node_modules/**'
};

const nodeResolveOptions = {
  module: true,
  jsnext: true,
  preferBuiltins: true,
  extensions: ['.js', '.vue']
};

const aliasOptions = {
  resolve: ['.js', '/index.js'],
  '@': path.resolve(__dirname, 'src')
};

export default {
  input: './src/library.js',
  external: id => id.includes('node_modules') || !(path.isAbsolute(id) || id.startsWith('.') || id.startsWith('@/')),
  treeshake: {
    propertyReadSideEffect: false,
    pureExternalModules: true
  },
  preserveModules: true,
    format: 'esm',
    dir: './dist',
    name: [omitted],
    globals: {
      vue: 'Vue'
    },
    sourcemap: true
  },
  plugins: [babel(babelConfig), alias(aliasOptions), nodeResolve(nodeResolveOptions), vuePlugin(), jsonPlugin()]
};

This is caused by this https://github.com/vuejs/vue-cli/blob/c078633be1127b0dd19b0ccb1b6cb6022288d960/packages/%40vue/babel-preset-app/index.js#L62
You should be able to override this behaviour with something like ['@vue/babel-preset-app', { absoluteRuntime: false }]

Great! I'll give that a try. Appreciate the super timely response.

Well, wanted to update you to let you know that overriding that option did not fix the issue. But since I can see that this isn't an issue with this plugin, but the vue preset, you're fine to close this.

If you want you can share your current setup (in a manner that I could do npm install && npm run build or similar) and I could take a quick look at it.

Ok, I'll see if I can get a minimal repro for you at some point today.

The repro case has not been provided for quite some time, so I assume the issue is stale and as there are no actionable points here for me - I'm going to close it.


Sprite3D Billboard isn't work when used Material Override

If do that sprite will not show image.

I'm override material with standard shader
I just found it must set Invert Faces frag, then image will be appear.

Is this issue still valid in the current master branch?

No answer, and the original bug report had too little details for easy reproduction, so closing. Please comment if you still reproduce this issue so that I can reopen it.


Unable to search structures using the Java client or the web service

When using the Java client there's a problem when calling the function getStructureSearch when calling it like this:
LiteEntityList querySMILESResult = client.getStructureSearch(smiles, StructureType.SMILES, StructureSearchCategory.IDENTITY, 100, 0.90F);
Getting this error with the following Identifier and SMILE:
Identifier{inchi=InChI=1S/C5H12O7P2/c1-5(2)3-4-11-14(9,10)12-13(6,7)8/h3H,4H2,1-2H3,(H,9,10)(H2,6,7,8), inchi_key=CBIDRCWHNCKSTO-UHFFFAOYSA-N, smiles=CC(C)=CCOP(O)(=O)OP(O)(O)=O}
Exception in thread "main" com.sun.xml.ws.fault.ServerSOAPFaultException: Client received SOAP Fault from server: java.lang.NullPointerException Please see the server log to find more detail regarding exact cause of the failure.
at com.sun.xml.ws.fault.SOAP11Fault.getProtocolException(SOAP11Fault.java:163)
at com.sun.xml.ws.fault.SOAPFaultBuilder.createException(SOAPFaultBuilder.java:98)
at com.sun.xml.ws.client.sei.StubHandler.readResponse(StubHandler.java:223)
at com.sun.xml.ws.db.DatabindingImpl.deserializeResponse(DatabindingImpl.java:176)
at com.sun.xml.ws.db.DatabindingImpl.deserializeResponse(DatabindingImpl.java:263)
at com.sun.xml.ws.client.sei.SyncMethodHandler.invoke(SyncMethodHandler.java:89)
at com.sun.xml.ws.client.sei.SyncMethodHandler.invoke(SyncMethodHandler.java:62)
at com.sun.xml.ws.client.sei.SEIStub.invoke(SEIStub.java:131)
at jdk.proxy3/jdk.proxy3.$Proxy37.getStructureSearch(Unknown Source)
at uk.ac.ebi.chebi.webapps.chebiWS.client.ChebiWebServiceClient.getStructureSearch(ChebiWebServiceClient.java:197)
Trying the web service with the getLiteEntity function and inchi key: CBIDRCWHNCKSTO-UHFFFAOYSA-N, we get this:
S:ServerERROR (General): we are having problems with this service... we are trying to rectify the situation.data 0: CBIDRCWHNCKSTO-UHFFFAOYSA-N, data 1: INCHI/INCHI KEY, ERROR (General): we are having problems with this service... we are trying to rectify the situation.
and when trying the function getStructureSearch with the smile: CC(C)=CCOP(O)(=O)OP(O)(O)=O, we get this:
S:Serverjava.lang.NullPointerException
Thank you

The ChEBI developer is currently taking a look at this issue.

Dear Blanca, we had some issues in the database side of things when your issues ocurred. I understand you're not experiencing this issue anymore. Could you please confirm? Many thanks.


Is there any possibility to read environment variable from file at path?

Question Subject

I develop white label tvOS app and can  gym the app. But also I have multiple brands, which has their unique "app ID", "server url" and other parameters. Brands resources are in the separate git project, which connected as submodule to the tvOS project.
------------- Project.xcproject
------------- fastlane/Fastfile
---------------------- Appfile
------------- brands/development
--------------------- pilot
--------------------- demo
My plan was to define parameters in brands/pilot/.env.pilot , brands/development/.env.development, brands/demo/.env.demo files
But .dotenv plugin requires to locate these env files in the root folder of the project
So is there a possibility to load environment variables from path brands/pilot/.env.pilot ?

You can either add symlink to your .env file, or mess with Fastfile like that:
fastlane_require 'dotenv'

before_all do
  Dotenv.overload 'brands/pilot/.env.whatever'
end

Neither of above is the prettiest solution I would say, but 🤷♂

Thank you @Katsz . Elegant and simple.
But there is the issue that .brands/brand/.env.whatever path depends of selected brand. So it can be .brands/development/.env, .brands/pilot/.env or whatever.
So if I run build with $ fastlane build_brand --env demo
then I have to access variables from:
before_all do
  Dotenv.overload 'brands/demo/.env.demo'
end

Is there a method to read passed --env argument?

Fastlane::Actions.lane_context[Fastlane::Actions::SharedValues::ENVIRONMENT] should do.


Do u got Discord?

If u do give me ur user

If u do give me ur user

Join the support server here:
https://discord.gg/4yjy8KgCnq

Join the discord as mentioned above and my tag is I am Ben#6686


#issue80 solved

#80   add footer in read a shlok page
@akshitagupta15june please review it

Can u add a screenshot

Check it.

Thank u so much for merging as this is my first open source contribution to public repository👍🏻


please add tag for analyzer-0.33.0

GitHub doesn't let me tag the version I want to use because it is outside the "recent commits" window,
and I get this error in trying to add the tag myself:
~/dart/sdk/sdk
$ git tag -a analyzer-0.33.0 644342a6e62d738b4c04031851cdb2e565cd0ec7 -m "Analyzer 0.33.0 publish"
~/dart/sdk/sdk
$ git push --tags origin master
Enumerating objects: 1, done.
Counting objects: 100% (1/1), done.
Writing objects: 100% (1/1), 176 bytes | 176.00 KiB/s, done.
Total 1 (delta 0), reused 0 (delta 0)
remote: Processing changes: refs: 1, done
To https://dart.googlesource.com/sdk.git
 ! [remote rejected]       analyzer-0.33.0 -> analyzer-0.33.0 (prohibited by Gerrit: not permitted: create tag)
error: failed to push some refs to 'https://dart.googlesource.com/sdk.git'
~/dart/sdk/sdk
$

Please tag 644342a6e62d738b4c04031851cdb2e565cd0ec7 to match the imminently published version of analyzer as "analyzer-0.33.0".  Thanks!

Done: https://dart.googlesource.com/sdk/+/analyzer-0.33.0 (should be synced to GitHub soon).


HOW CAN I TRAIN MY OWN VIDEO

Is their any way to train our own video on rolo model. and how i create groundtruth_rect.txt , yolo out ,rolo_out_test file for my video please provide step bt step guid on it.

Any update on this?

any update?


Adds rustic-cargo-init

Adds rustic-cargo-init, and refactors both interactive functions to use the same underlying function, since they're almost identical commands.

I'm not sure either, actually. Haven't had any luck reproducing it locally.

It's always worth a try to rerun the tests. I don't know the reason, but sometimes it fails although everything is working.


Skill translation

I've translated fields in the file, and most of them do work - except for skill names. Am I missing something ?

Sorry, what? The skills should all be translated. Did you add custom ones?

Nope. Right inside copied ru.json I edited values for things like "ACROBATICS" and etc. and it had no effect inside sheet in game. Here's my contents:
RUS.txt

If you'd like to maintain the russian translations I'd appreciate it: https://github.com/mlenser/roll20-character-sheets/blob/master/5eShaped/translations/ru.json
You need to put the translations into the sheet editor alongside the html and css.

First of all, thank you for replying and trying to help out.
Yes, I did it. As I said above - a lot of things in sheet got indeed translated:
http://imgur.com/rxb04Tc
But ... not skills in particular O-o

If you've updated a lot of missing translations can you send me a Pull Request?
Skills come from Sheet Workers and perhaps may not update until roll20 integrates the changes (which has to go through me pushing to them)

I've just tried copying whole en.json and editing "ACROBATICS"'s value in it - again, no luck.
I've tried to copy whole thing over to dev servers - again, no difference.
As per pull request - yes, I understand that original ru.json I used as template is pretty outdated and needs some fixes to it, which I will figure out in some time - and then make a proper Pull. Allthough if you would insist - I can do it right now, though I am not very much acquainted with how GitHub works.
If there's anything else you want of me, some tests or logs - feel free to ask.

I think you may have misunderstood what I wrote above. The skills come from sheetworkers which pulls the data from roll20. Until I push my code to roll20 it will not be up to date. There is absolutely nothing you can do manually to make it work until I do so.
If your text file is up to date then I could use that. Up to you.
The earliest I'll push the code is next Tuesday, but I'm not certain that I will. I'm trying to get it ready in time, but we'll see.


Idea: change the desugaring of if let to improve borrow checking

Related: rust-lang/rust#28449
Currently if let desugars to a match with guards. Using the example from the related post:
let mut f = Some(1);
if let Some(a) = f {
    bar();
} else if f.as_mut().is_some() {
    baz();
}

Desugars to
match f {
    Some(a) => {bar();}
    _ if f.as_mut().is_some() {baz();}
    _ => {}
}

However, this leads to f being borrowed in the else if which is truly annoying. Instead, I propose the following desugaring:
    if match f {Some(_) => true, _ => false } {
         let Some(a) = f;
         { bar(); }
    } else if f.as_mut().is_some() {
        baz();
    }

Which (I believe) does not suffer from the same problem.
One unanswered question is: does non-lexical lifetimes solve this?

The proposed desugaring evaluates f twice.

Oh, that's true... But I believe it is easily fixed by just evaluating once beforehand, right?

I'm pretty sure non-lexical lifetime will solve this, since the code this desugars to is essentially the same as the classic example of trying to insert a value into a map if it's not already in the map. Niko discusses that example both on his blog way back in April (http://smallcultfollowing.com/babysteps/blog/2016/04/27/non-lexical-lifetimes-introduction/#problem-case-2-conditional-control-flow) and in his recent RFC for NLL (https://github.com/nikomatsakis/nll-rfc/blob/master/0000-nonlexical-lifetimes.md#problem-case-2-conditional-control-flow).

@Ixrec Ah, thanks. I think NLL definitely subsumes this idea...


Compatibility of Nlog between .Netcore 3.1 and .Net framework4.6.2

Hi
I have class library which is developed in .net core 3.1  where a particular class(named TokenProducer) has a dependency on Ilogger.
This class library (as nuget package) is used in another project which is developed on .net framework 4.6.2. Now i need to inject ilogger dependency to the class TokenProducer mentioned above.
i am able to do it as follows

but how to configure nlog.config section from current project(which is on .Net framwork 4.6.2) to NLogLoggerProvider.?
please suggest on how to resolve this issue

NLogLoggerProvider will automatically attempt to load the NLog.config file if available. Make sure to have configured "Copy if newer".
Notice that the LoggerFactory includes a default-loglevel-filter, that you might want to disable and instead depend on the NLog.config. You can configure the LoggerFactory like this:
_loggerFactory = LoggerFactory.Create(builder => builder.SetMinimumLevel(Microsoft.Extensions.Logging.LogLevel.Trace).AddNLog()));
_logger = _loggerFactory.CreateLogger<TokenProducer>();

Closed due to inactivity


Merged the objects of the icon, with some tweaks

The icon is now cleaner, smaller (in disk size), looks a teeny bit better (notice the the bottom seam between the & and the s), no longer has some weird rendering glitches and is easier to work with.
The untrained eye won't notice a difference with usual icon sizes, so you don't need to worry.

Thank you @DoctorJellyface, I merged your changes!

Thanks!


[Hermes] Add ‘trace’ support

If I understand things correctly, the ‘systrace’ code in RN (specifically for Android) relies on a non-oss FB tool called fbsystrace. It appears that RN for Windows has implemented their own version, so we should look into creating one too. Probably in a manner that interfaces well with Instruments.app.

@mhorowitz @NickGerleman Is my understanding here correct?

That sounds correct. @mganandraj will have more context than I do.

@alloy that's correct. We planned to create a fbsystrace open source library but it never materialized .. We had an fbsystrace implementation for ANdroid platform too in the past which we used extensively for perf analysis couple of years back ..
https://office.visualstudio.com/ISS/_git/sdx-platform?path=%2Fsrc%2Freact-native%2FReactCommon%2Fcxxreact%2Ffbsystrace.cpp&version=GBJSI&_a=contents
https://office.visualstudio.com/ISS/_git/sdx-platform?path=%2Fsrc%2Freact-native%2FReactCommon%2Fcxxreact%2FJSCTracing.cpp&version=GBJSI&_a=contents
Please note that for Android platform, this is needed only for Systrace events from C++ code.


TSAN: fix data race when reading a histogram's sample count

Commit Message: Fix TSAN issue caused from tests when reading histograms.
Additional Description:
This was introduced by #15908. We need to read a histogram's from
the main thread to avoid data races between writes (from the main
thread) and reads (from a worker thread).
Risk Level: Low.
Testing: Unit tests + TSAN should pass.
Docs Changes: NA
Release Notes: NA
Platform Specific Features: NA
Signed-off-by: Raul Gutierrez Segales<EMAIL_ADDRESS>

This should fix the TSAN issue. Happy to move the helper code that does the read from the main thread into
a helper function in test/test_common/utility.h if that's better.
cc: @antoniovicente @jmarantz

/assign @antoniovicente

Ok the ASAN failure seems unrelated:
//test/integration:health_check_integration_test                         FAILED in 2 out of 2 in 61.7s

I'll merge main...

Ok the ASAN failure seems unrelated:
//test/integration:health_check_integration_test                         FAILED in 2 out of 2 in 61.7s

I'll merge main...

Hmm nothing new in main plus it seems possibly transient:
test/integration/health_check_integration_test.cc:276: Failure
Value of: clusters_[cluster_idx].host_fake_connection_->waitForNewStream( *dispatcher_, clusters_[cluster_idx].host_stream_)
  Actual: false (Timed out waiting for new stream.)
Expected: true
Stack trace:
  0x160b9b2e: Envoy::(anonymous namespace)::HttpHealthCheckIntegrationTest_SingleEndpointImmediateHealthcheckFailHttp_Test::TestBody()
  0x208969e8: testing::internal::HandleSehExceptionsInMethodIfSupported<>()
  0x20866c13: testing::internal::HandleExceptionsInMethodIfSupported<>()
  0x20832340: testing::Test::Run()
  0x20833ad5: testing::TestInfo::Run()
... Google Test internal frames ...

[  FAILED  ] IpHttpVersions/

/retest

The remaining coverage failure is unrelated, so we can probably just merge:
  /build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/bazel-out/k8-fastbuild/testlogs/test/exe/win32_scm_test/coverage.dat
//test/integration:health_check_integration_test                         FAILED in 38.4s

/retest

cc: @phlax for visibility on the CI/coverage failure (not sure if it's a global thing)


documentation: But gb18030 is a UTF.

We also can’t handle the non-UTF encodings used for Chinese, Japanese, and Korean, such as shift-jis and gb18030. See issue #34 for why this is so hard.

But GB18030 is a UTF! It handles every code point but surrogates, which is good enough to meet this definition:

A Unicode transformation format (UTF) is an algorithmic mapping from every Unicode code point (except surrogate code points) to a unique byte sequence.

I guess there will be some rewording needed. Perhaps just replace gb18030 with gbk.
PS: members of the GB family may be partially identifiable given the following features:

Most characters found in common GB strings fall inside the GB2312-1980 (EUC-CN) range. EUC G1 bytes don't overlap with ASCII byte range.
GB 18030's 4-byte sequences are highly visible to human in some otherwise GBK soup, as GBK does not make use of 0x30-0x39 for its 2-byte encoding.

I don't think it would be common to call GB18030 a "UTF", despite the definition you give. Even if I changed the example, it wouldn't make GB18030 any easier to fix heuristically.
It's true that the G1 bytes (0x81 through 0xFE) don't overlap with ASCII, but they do overlap with UTF-8 (which is several orders of magnitude more common) and all the old single-byte codepages. Issue #34 is an illustrative example: the user thought he was decoding damaged GB18030 when it was actually damaged UTF-8, and the output was convincing to someone who didn't read Chinese.
As far as I can tell, the only way to get negative evidence for decoding something as GB* is to recognize the result as "that's not Chinese" -- that is, to have a Chinese language model.
In the long term I'd be interested in distinguishing encodings via language models, but in a separate project, not as part of ftfy (that would be scope creep).

I don't think it would be common to call GB18030 a "UTF", despite the definition you give. Even if I changed the example, it wouldn't make GB18030 any easier to fix heuristically.

Hmm. What about rewording to something like "legacy encodings"?


Fix/ donation link fixed

Description of changes

Moved the donate link to a common file url (except for trigroup since this will be changing soon).
import the url now instead of dealing with finding all locations.
Issue Resolved

Fixes #907

The Codeclimate check isn't passing because

Function renderNavContents has 28 lines of code (exceeds 25 allowed). Consider refactoring
in header.js. Is this something we need to fix?

@ksmacleod99 codeclimate is annoying for front-end. I don't know who reactivated it, but we deactivated it before, and now I have the power do so again...
💀 🔫
it's ded


parent study: drop down empty for user

when adding a do, the parent study drop down is empty for a normal user (it works for manager)
see also #22

@sergiocontrino please also tell me what fields of the parent study should appear where parent study is mentioned, fields currently used: displayTitle for DO view page, sdSid as main field in DO edit page dropdown and displayTitle as secondary field (smaller), and displayTitle in DO summary-page


Contact the EPICS core developer team to get some feedback on the extension

During the ICAPECS2021 poster session, a participant adviced me to contact the EPICS core developer team to have some feedback about the extension:

could I use some files in the epics-base to improve the modules ? (record coverage, syntax coverage, ...)
features to add between v3.15 and v7 in vscode-epics?
other points ?
any feedbacks ?
any advice ?

Note that I already get all the EPICS fields (see README.md#3-get-all-the-fields-of-the-epics-database)

Hi @anjohnson @ralphlange, I am taking the liberty of contacting you about vscode-epics. Do you have any comment about this current issue or people I should contact ? Best regards.

We have extended the syntax of the IOC's DB and DBD files several time in recent years, probably the largest change was the introduction of JSON (and now JSON-5) syntax for record info-tags and field values, the addition of JSON link types, and support of Perl's POD markup language in DBD files for writing record reference documentation. Read the Release Notes for the main changes where these were described or at least mentioned.
Supporting the JSON/JSON-5 syntax would be an excellent addition if you haven't already added that — I do have your extension installed and I appreciate it for what I've seen it do, but I don't run vscode very often so I'm not sure if JSON support is included yet. The use of the Perl POD markup isn't something many users will have come across or need, but you can see it used on any recent 3.15 branch in the files src/ioc/db/*.dbd.pod and src/std/rec/*.dbd.pod.
I don't really know what else to suggest at the moment — feel free to make announcements to tech-talk when you release additional features though. Thanks for developing this module!

Andrew


Allows for search history bundles with PATCH requests

Description
Allows for search history bundles with PATCH requests
Related issues
Addresses [issue #].
Testing
Describe how this change was tested.
FHIR Team Checklist

[ ] Update the title of the PR to be succinct and less than 50 characters
[ ] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[ ] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[ ] Tag the PR with Azure API for FHIR if this will release to the managed service
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

It's actually still broken for STU3.
I'm not sure why
#if !Stu3
                    case Bundle.HTTPVerb.PATCH:
                    case Bundle.HTTPVerb.HEAD:
#endif

was added, but in STU3 patch is possible, and if I execute it I would get cryptic error:
 System.NullReferenceException: Object reference not set to an instance of an object.
         at System.Object.GetType()
         at Hl7.Fhir.Utility.EnumUtility.GetLiteral(Enum e)
         at Microsoft.Health.Fhir.Api.Features.Resources.Bundle.BundleSerializer.<>c__DisplayClass3_0.<<Serialize>g__SerializeEntries|2>d.MoveNext() in D:\src\fhir-server\src\Microsoft.Health.Fhir.Shared.Api\Features\Resources\Bundle\BundleSerializer.cs:line 138

and bundle would return only partial information.

Let me correct myself.
https://hl7.org/fhir/STU3/Bundle.html has only following verbs:
GET | POST | PUT | DELETE
It's perfectly fine to do patch in STU3 and we store PATCH in our DB, but we can't map it.
In same time, request.method is required field and we fill it up with null value which leads to exception.

Looks like there will be other problems in STU3 too, the Method/Url will be null too for Patch:
https://github.com/microsoft/fhir-server/blob/84957b0c924902ac9afc1480c4013eaeedee9196/src/Microsoft.Health.Fhir.Shared.Core/Features/Search/BundleFactory.cs#L64-L71

Yep, I'm leaning towards wrapping resource.Request assignment in if (has) to not fill it at all for patch...
Not sure it's a right thing, but it's better than returning half of the bundle.
As another option we can map PATCH to PUT for STU3, but it's a bit meh

I think its less broken if PATCH is returned as PUT. Having the URL missing doesn't seem right.
We could consider removing PATCH support from STU3, but that would be considered breaking now

We could consider removing PATCH support from STU3, but that would be considered breaking now

Let's not -> https://hl7.org/fhir/STU3/http.html#patch  it's a valid operation, it just happened what BundleVerb doesn't have patch verb for STU3.

So just a spec oversight then :)


Issue with k8s.io/docs/tutorials/stateful-application/basic-stateful-set/

Following the document under https://kubernetes.io/docs/tutorials/stateful-application/basic-stateful-set/#updating-statefulsets
Under "Rolling Update" section it is said to:
Patch the web StatefulSet to apply the RollingUpdate update strategy.
The snippet shows that statefulset has been patched.
However, I do NOT see it to be modified to anything else earlier in the documentation. Therefore, updateStrategy should have been of default value, i.e. RollingUpdate
Now, when we change the container image in statefulset, it should update it and not be the previous container image.
Before patching statefulset.
GC02V81JZHTD6E:web vikasbains$ for p in 0 1 2; do kubectl get po web-$p --template '{{range $i, $c := .spec.containers}}{{$c.image}}{{end}}'; echo; done
k8s.gcr.io/nginx-slim:0.8
k8s.gcr.io/nginx-slim:0.8
k8s.gcr.io/nginx-slim:0.8

Updating updateStrategy will not do anything, already set to default value.
GC02V81JZHTD6E:web vikasbains$ kubectl patch statefulset web -p '{"spec":{"updateStrategy":{"type":"RollingUpdate"}}}'
statefulset.apps/web patched (no change)

Updating the image. This will recreate the pods with new image we updated.
GC02V81JZHTD6E:web vikasbains$ kubectl patch statefulset web --type='json' -p='[{"op": "replace", "path": "/spec/template/spec/containers/0/image", "value":"gcr.io/google_containers/nginx-slim:0.8"}]'
statefulset.apps/web patched

Pods image has been updated (and is NOT the previous image).
GC02V81JZHTD6E:web vikasbains$ for p in 0 1 2; do kubectl get po web-$p --template '{{range $i, $c := .spec.containers}}{{$c.image}}{{end}}'; echo; done
gcr.io/google_containers/nginx-slim:0.8
gcr.io/google_containers/nginx-slim:0.8
gcr.io/google_containers/nginx-slim:0.8

Please correct me if I am wrong.

https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.18/#statefulsetupdatestrategy-v1-apps confirms that the default update strategy is now RollingUpdate
I presume that long enough ago there was a different default.
/kind bug
/language en
/priority important-longterm

Please check if this PR is what you wanted.

/remove-lifecycle rotten


Support and Contributions page design

[ ] Design of page
[ ] Ensure content reflects word doc
[ ] Design review
[ ] Pass on to Dev
[ ] Check the links redirect users out to correct Github templates (cc James Nash)

1 Dec - review with content scheduled for later in the day.. hope to pass on to dev to get started


Small typo in the Polymorphic many-to-many guide

Shipping lists are a thing, but this is talking about a Shopping list I think

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Keyed graph

Problem

Every node lookup is hashing the TransactionId which can be expensive since it is done very often

Changes
Breaking Change

Exposed Changes:

Introduce TransactionKey which wraps TransactionId with an index
pop_and_unblock now returns Option<(TransactionKey<Id>, HashSet<usize>)>
pop now returns an Option<(TransactionKey<Id>)>
unblock now takes a TransactionKey<Id> and returns HashSet<usize>
is_blocked takes a &TransactionKey<Id>

Internal Changes:

Graph Nodes:

Stored in indexable Vec, using index from TransactionKey as lookup key. Currently this is append only, so there's no index invalidation.
Store the TransactionId on the node (needed to recreate TransactionKey on retrieval)
edges now stores indexes to other nodes instead of TransactionIds (this is publicly exposed through the top-level prioritization function)

Locks - use index instead of TransactionId for storing most-recent accesses

Making a note to self on initial version of this branch.
It seems inconsistent that we return sets of indices but every other interaction with Graph is with a TransactionKey.
Need to make this more consistent; some thoughts:

User inserts tx using an Id
All interactions after insertion use TransactionKey


Wheel-to-Terrain Contact Interaction at Low Speeds

Description
At low speeds (about 0.1 kph) the Boomer tractor wheels sink into the ground until the bottom of the kingpin hits the ground.  This has only been seen in non-flat terrains.
Steps to Reproduce

Use any sample non-flat terrain provided by Webots
Place the Boomer vehicle (Webots sample tractor) in this world, with the Boomer controller (also provided by Webots)
Driver the tractor at slow speeds (around 0.1 km/h)
As the tractor reaches the interface between 2 terrain tiles, the wheels will start to sink into the ground.

Expected behavior
Vehicle wheels should not sink into the ground, regardless of the terrain geometry.
System

Windows 10
Graphics Card: NVIDIA GeForce RTX 2070, Intel HD Graphics 620

Screenshots or Videos
Attached you will find a couple of short videos and pictures depicting the problem.
https://agjunction-my.sharepoint.com/:v:/p/mvillela/Ead4Z4TufSBDimFK2IWp-1QBQBMu6c6aIAGeGk50m0D2Ow?e=NEHP9K
https://agjunction-my.sharepoint.com/:v:/p/mvillela/ERrNVOxUCYZCoefixsjkuNsB-xHTrgI6GIytDkwspUlrhw?e=oDvXMO

The collision quality depends a lot on the global physics properties, and on the contact properties between the wheels and the ground:

Reducing the WorldInfo.basicTimeStep will improve directly the contacts quality. I would say that 8ms is a good value for such kind of simulation.
Adding a ContactProperty node in WorldInfo.contactProperties, and define a ContactProperty.softCFM between the wheel and the ground will reduce the shrink effect.

Please look at the boomer.wbt world physhics properties here:
https://github.com/omichel/webots/blob/master/projects/vehicles/worlds/boomer.wbt#L2-L15

I confirm the presence of this bug. It's due to ODE.
It seems to be dependent on tile size of the elevation grid. If the tiles are big enough to avoid the vehicle to be several tiles at a time, it seems to work much better. Could it be a workaround for you?

Thank you.  I'll check into our options.
Do the Webots and ODE development teams have a formal coordination agreement to address issues like this, or do I need to contact the ODE group directly?

We already fixed several times issues in ODE.

"Fixed" by added a FAQ entree: https://cyberbotics.com/doc/guide/modeling#my-robot-sinks-into-the-ground-how-to-remove-this-effect


Pod may write wrong disk if two disks that own same serial number exist in same node

Hwameistor use serial number to identify disk, but there may be a special scene which Hwameistor can't handle now.
Pod may write wrong disk if two disks that own same serial number exist in same node.
For example:
step1. Here is a pod using disk provided by Hwameistor:
root@pw1:~/hwameistor# kubectl get pod
NAME                                READY   STATUS    RESTARTS   AGE
nginx-deployment-54c4795d89-h6tzp   1/1     Running   0          30s

And this pod is using /dev/sdc on node pw3
root@pw1:~/hwameistor# kubectl get ldv
NAME                                       NODE   DISK       ALLOCATEDCAP   TYPE   STATUS
pvc-c18f6426-c228-4179-a3fd-d544161c5727   pw3    /dev/sdc<PHONE_NUMBER>0    HDD    Ready

sdc    8:32   0    10G  0 disk

step2. Then shutdown pw3, and plug a new 100G disk which owns same serial number as /dev/sdc. Then start pw3, the following situation may occur:
root@pw1:~/hwameistor# kubectl get ld localdisk-1388bb8f3072919b37c426dd764f5e46 -o wide
NAME                                         NODEMATCH   CAPACITY       PRENODEMATCH   DEVICEPATH   PREDEVICEPATH   OWNER                PHASE   HEALTH   RESERVED   STATE    AGE
localdisk-1388bb8f3072919b37c426dd764f5e46   pw3<PHONE_NUMBER>00   pw3            /dev/sdb     /dev/sdg        local-disk-manager   Bound   Passed              Active   13d
root@pw1:~/hwameistor# kubectl get ldv
NAME                                       NODE   DISK       ALLOCATEDCAP   TYPE   STATUS
pvc-c18f6426-c228-4179-a3fd-d544161c5727   pw3    /dev/sdb<PHONE_NUMBER>0    HDD    Ready

Pod looks like this:
root@pw1:~/hwameistor# kubectl get pod
NAME                                READY   STATUS    RESTARTS   AGE
nginx-deployment-54c4795d89-scvzz   1/1     Running   0          3m
root@pw1:~/hwameistor# kubectl exec nginx-deployment-54c4795d89-vstl9  -i -t  -- /bin/bash
root@nginx-deployment-54c4795d89-scvzz:/# ls -al  /dev/pw-disk
brw-rw---- 1 root disk 8, 16 Sep  4 07:18 /dev/pw-disk

On node pw3, 8:16 is the new 100G disk:
sdb    8:16   0   100G  0 disk

And you can see by-id symbol link like this:
root@pw3:/dev/disk/by-id# ll /dev/disk/by-id/scsi-SQEMU_QEMU_HARDDISK_pw-hot-plug-6
lrwxrwxrwx 1 root root 9 Sep  4 15:18 /dev/disk/by-id/scsi-SQEMU_QEMU_HARDDISK_pw-hot-plug-6 -> ../../sdb

As you can see, pod is using wrong disk.

this should be safe when volume is already mounted, but we should still check this situation before publishing to prevent wrong disk is used.

Yes, that's right.

/assign


libinput: support discrete_scroll_factor for high resolution mouse wheels

I have fancy mouse with a high-resolution wheel (microsoft sculpt ergo). Scroll only advances every 6 ticks (on linux, works fine on windows). I believe the solution requires supporting "high resolution mouse wheels".
See https://gitlab.com/warningnonpotablewater/libinput-config#how-to-use for detail.

Hyprland does already support high resolution mouse wheels, it's the apps themselves that do not.
#728

oh yeah right, forgot. Estebiu is correct.

My mouse mysteriously started scrolling 1-event-per-mechanical tick right after y'all replied, so I don't have motivation to investigate further. This at least demonstrates that adjusting the scroll factor is possible at some level, although which level remains mysterious. Will update with steps taken if/when my setup goes back to 1-event-per-6-ticks (probably next reboot).

I think the factor I'm talking about (discrete scroll factor) is a compatibility mode for apps that are listening for the old-style 120 degree events, while apps can also listen for fine-grained events for which there is a different libinput-level adjustment factor.


Mysql should follow label naming schema

https://github.com/helm/charts/blob/16ab5a288b1a6b0926129a0635db8b54443a83c4/stable/mysql/templates/svc.yaml#L7
https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/#web-application-with-a-database

No one to fix it?


Remove testMatch from package.json

Summary
I discovered that the testMatch in the root package.json meant that a few test suites did not run.
One was simply old, so I deleted it. Then a bunch of extra files in __tests__ tried to run, so I ignored them.
But it still leaves 3 tests which fails.

Test plan
Green CI (eventually)

Ok, the snapshot one was just a require -> import fix.
@mjesun Ideas about the jest-worker one?
The iterator-to-null-test I have no idea about.

On CI the worker test doesn't fail... @pedrottimark mentioned it only fails if run without -i. Is that supposed to happen?
It points to another test failing which I just didn't wait for, though, the one added in #3672 earlier today. @EnoahNetzach Could you take a look?

I won't have time before this weekend.
I'll be glad to help then!

Rebased on master, but a new failure has snuck in, so we're still at 2 failures! The new test from #3458. /cc @felipeochoa

Open PR for the sourcemap error: #4744.

Rebased, and now we're down to the single iterator-to-null-test.js, which I have no idea how to go about fixing

Rebased, and skipping the failing test. Hopefully green ci...

Codecov Report

Merging #4630 into master will increase coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4630      +/-   ##
==========================================
+ Coverage   60.35%   60.74%   +0.39%
==========================================
  Files         198      197       -1
  Lines        6639     6596      -43
  Branches        4        4
==========================================
  Hits         4007     4007
+ Misses       2632     2589      -43

Impacted Files
Coverage Δ

packages/jest-snapshot/src/plugins.js
80% <0%> (-20%)
:arrow_down:

...ages/jest-worker/src/__performance_tests__/test.js

packages/jest-mock/src/index.js
84.83% <0%> (+0.36%)
:arrow_up:

packages/jest-config/src/normalize.js
92.94% <0%> (+2.13%)
:arrow_up:

Continue to review full report at Codecov.

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

This is so good, thanks for doing this.


Execution stage very slow and only using single core

Describe the bug
Hello,
While syncing RETH with the --full sync mode, it only uses a single core and it is very slow.
Should any optimizations could be applied here?
Thanks.
Steps to reproduce
Syncing reth with the --full sync mode
Node logs
Stage committed progress pipeline_stages=5/13 stage=Execution block=2456857 checkpoint=0.3% eta=12days 15h 36m 27s
Status connected_peers=15 stage=Execution checkpoint=0.3% eta=12days 15h 35m 51s

Platform(s)
Linux (x86)
CPU: Xeon E-2286G with 32G memory
Disk: Intel SSD with more than 300MB/s
What version/commit are you on?
No response
What database version are you on?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

execution is inherently sequential, there's no way to make this process multicore without redesigning how execution works.
I'm closing this because this is the expected behaviour, however parallel evm is still being investigated


MailChimp Firebase Integration only adds email and not first name and last name

[READ] Step 1: Are you in the right place?
Issues filed here should be about a feature request for a specific extension in this repository. To file a feature request that affects multiple extensions or the Firebase Extensions platform, please reach out to
Firebase support directly.
[REQUIRED] Step 2: Extension name
This feature request is for extension: auth-mailchimp-sync
What feature would you like to see?
Mailchimp audience list should additionally include first name and last name; I am already collecting that information in my list, can you please provide a way to inject that into mailchimp? I think using mailchimp without using user's first and last name is kind of silly and drops conversion even further!
How would you use it?
When I send an email campaign to my audience, I would refer to them with their first name, improving the quality of the email.

Hello! Looking at the docs, it doesn't seem it's possible to add a name field to a member: https://mailchimp.com/developer/api/marketing/list-members/add-or-update-list-member
Unless, it can be provided to merge_fields.

Thanks for getting back so quickly @Ehesp !
TL;DR; We can use FNAME and LNAME inside merge_fields as documented here.
Interestingly, their API introduction section does document how add first name and last name fields to the creation request, see Add a contact to an audience.
It does seem like they are using merge_fields property in this case with properties FNAME and LNAME. Unfortunately, their Merge Fields API documentation doesn't include all the possible fields, but they do provide some context on it in their merge field cheat sheet.
So their documentation could definitely use some improvement, but I do think this is possible.

@zkhalapyan The main issue here is that only a "display name" is returned, which isn't specifically a full name. We could perhaps add this to the FNAME field if it exists, but it may not be a first name, or may include the last name.

The main issue here is that only a "display name" is returned.

This is from internal firebase auth?
I guess there are three options:

Update firebase auth to provide proper first name, last name, display name, etc. structure to get rid of this limitation
Provide simple instructions on how to extend the extension to add in merge_fields (maybe some tutorial steps & code examples catering towards the top most important fields).
Push display name as a custom merge_field i.e. DNAME or display_name to mailchimp; at least this gives mailchimp users some resemblance of a name they can use to address their audience in the name.

Ideally, we could do #1. This is beneficial from many different aspects. If not, I would pick #2. Probably the worst is #3 since it could end up putting in full names inside display names and just give noisy data to mailchimp users.
WDYT?

Another quick note here... so once a member is created, it is not possible to update the member's name just via the email address. Updating the name would require knowing the member's ID on mailchimp. This is possibly, but annoying to go through all the steps.
In other words, by integrating with Firebase's mailchimp extension, you are making it extremely difficult for developers to later update the contacts with names.

@Ehesp and @zkhalapyan could we just split the display name on spaces, then use the first element as FNAME and the rest as LNAME. Seems like it would work in most cases and IMO is way better than having no name at all.

@sam-jg would work for some simple names and for small startups, but vast majority of international names would start breaking which would probably not be acceptable for medium to large size companies.

@zkhalapyan I would guess that the currently implementation is already not acceptable for medium to large size companies.
Do you have some examples of international names that would cause issues, I don't have a lot of experience in that area sorry.

Checkout Personal names around the world. In some countries, the order is Family-Name Given-Name.

Following +
Any update on this? Seconding the need to have the first name and last name in Mailchimp, as every email we send starts with "Hello FNAME, "

Following +
Any update on this? Seconding the need to have the first name and last name in Mailchimp, as every email we send starts with "Hello FNAME, "

I was hoping to get input from a maintainer before submitting a PR. It should be a pretty small change, so you could probably just submit one anyway @ksiskar.
I don't see any easy fix to the name order issue, but don't think it really matters. To me, some names in the wrong order is better than no names at all.

I was hoping to get input from a maintainer before submitting a PR. It should be a pretty small change, so you could probably just submit one anyway @ksiskar.
I don't see any easy fix to the name order issue, but don't think it really matters. To me, some names in the wrong order is better than no names at all.

Hi there! I would hold back on submitting PRs for the mailchimp extension at the moment, as far as I'm aware, there are potential changes coming so development is on hold in the meanwhile.

Hi there! I would hold back on submitting PRs for the mailchimp extension at the moment, as far as I'm aware, there are potential changes coming so development is on hold in the meanwhile.

I too am having a similar issue here, it doesn't appear FNAME and LNAME are being applied to the merge fields when making the REST call, so I'm hit with errors like:
Error when adding user to Mailchimp audience { Error: Your merge fields were invalid.
at Request._callback (/workspace/node_modules/mailchimp-api-v3/index.js:518:35)
at Request.self.callback (/workspace/node_modules/request/request.js:185:22)
at Request.emit (events.js:198:13)
at Request.EventEmitter.emit (domain.js:466:23)
at Request. (/workspace/node_modules/request/request.js:1154:10)
at Request.emit (events.js:198:13)
at Request.EventEmitter.emit (domain.js:466:23)
at IncomingMessage. (/workspace/node_modules/request/request.js:1076:12)
at Object.onceWrapper (events.js:286:20)
at IncomingMessage.emit (events.js:203:15)
at IncomingMessage.EventEmitter.emit (domain.js:466:23)
at endReadableNT (_stream_readable.js:1145:12)
at process._tickCallback (internal/process/next_tick.js:63:19)
type: 'https://mailchimp.com/developer/marketing/docs/errors/',
title: 'Invalid Resource',
status: 400,
detail: 'Your merge fields were invalid.',
instance: 'bb199068-472d-44bb-9eaa-80a71d61a47f',
errors:
[ { field: 'FNAME', message: 'Please enter a value' },
{ field: 'LNAME', message: 'Please enter a value' } ],
Look forward to seeing an update here :)
Ed

Closing in favour https://github.com/mailchimp/Firebase/issues/30


chore(deps): dedupe

Current dependencies on/for this PR:

main

PR #518   👈

This stack of pull requests is managed by Graphite.


No scroll on mobile android possible, if draggable is false

Hi,
as mentioned in the topic: if draggable== false, I´m not able to scroll down the website on my samsung-android-smartphone touching a GridItem. You can check it also on your demo-site https://jbaysolutions.github.io/vue-grid-layout/examples/06-responsive.html - if you open in on a smartphone. Scolling is only possible if I hit the space between to items.
Many thanks for your support.

Hi, does anyone solve this problem?

Still no fix for this crucial bugs?

Hi, does anyone solve this problem?

add style
<style> .vue-grid-item.no-touch {   touch-action: auto; } </style>


Update sanitise script

Looks good. Let's merge. Going forward we need to find a more maintainable way of doing this.


feat: add coqui.ai for local TTS

Describe the solution you'd like
Since the OpenAI TTS is not local a good local alternative would be coqui.ai:
https://huggingface.co/spaces/coqui/xtts
It can run multiple languages.

Duplicate #126


[Registry Preview] REG_NONE value marked as error

Microsoft PowerToys version
0.69.1
Installation method
GitHub, PowerToys auto-update
Running as admin
No
Area(s) with issue?
Registry Preview
Steps to reproduce
Open a file with a REG_NONE value.
✔ Expected Behavior
The value gets accepted with and without data:

hex(0):
hex(0):0000...

❌ Actual Behavior

Other Software
No response

As far as the parser is concerned, it makes sense: every other Registry type has a value declared, even empty.
Should an easy enough case to handle, but it won't be until the other four PR are approved.  Each of those are atomic, so they can be up in the air at the same time, but one of them deals with value parsing and I want to avoid a collision there.

@randyrants
To avoid misunderstanding: The parser don't know REG_NONE type.

Actually, it would handle it, if there was a binary value.  During testing, I threw hex(15):12,34,56 at it and it parsed it fine, with a "unkonwn" type of "hex(15)" (same as the Editor did on import) but the lack of a value throws the error.

No. What I mean is that you can have a little hex(5|b|...): which means REG_Binary, ... and you can have hex(0): which means REG_NONE. The important thing here is the 0 inside the brackets.

There are actually three bugs here:

An empty "data" value in a hex line, which may or may not already be fixed in the most recent PR.
hex(0) shows up as hex(0), rather than REG_NONE, but that doesn't happen in this instance because of the bug #1.
Empty data values could show up as "(zero-byte binary length)" rather than just blank, to parallel RegEdit.

Got a fix for this in a branch; I'll push the PR when I get some other PR's cleared out of the way.

Pah.  Decided to mash the two touchy branches together.  Bigger CR but it's all in the same area (and using the same ideas.)


Calculate average only when actual connections have been made

So far averages were calculated over all processes even if they haven't
received any connections/requests. This resulted in incorrect value
calculated, especially after resstart/reload, as processes which haven't
received any connections/requests would report value of 0 eg.
process1: requests:0, ctime:0
process2: requests:0, ctime:0
process3: requests:1, ctime:100
process4: requests:0, ctime:0
In such circumstances average ctime should be 100 as only process3
received requests and others have no valid data. Till now value reported
would be 0+0+100+0/4=25 which is incorrect. This change should significantly
improve situation by filtering out processes which have not received any
requests. Keep in mind that for HTTP traffic req_tot instead of lbtot
is a much better option, but unfortunately this would significantly
increase code complexity (more complex condition).

@gringus thanks for the PR and sorry for the late response. Once the only comment I have is address we can merge this.
Do you have graphs with and without the patch to visualize the effect of the change?
Again thanks for the PR

Thanks for addressing the comment. Do you have the graphs I mentioned above?

I do have graphs, but only with a more complex condition and in combination with HAProxy patches to illustrate improvement of both things. In general this particular patch should allow much quicker conversion to the real average value. This is especially visible for low traffic backends.

OK fair enough.


Beta docfix

Fixes of docs

Coverage remained the same at 79.57% when pulling b7114d6dd225d5b1df52ebd5b3357507c67a1263 on beta-docfix into 1029253b7b3b043eaaea4c518a9b649a86d1c3bd on kulshekhar-patch-1.


config_validation.py

Request for date validation
Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hi @varunr047,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

What is the use case for the date() function?

@Danielhiversen validation of  date type  :)

I see that, but where do you need it?

That seems too obvious :smile:
Do you have another PR where you intend to use it?

want to introducing time conditions with date validation

Additions to config_validation will require tests, else it hurts the coverage. It might even be better to combine this with your "time condition" PR, since then it will show that it is used...

I agree with @kellerza, we should only add helper methods when a use case is provided with it.

@ballob added use case

@varunr047 You allowed it in the TIME_CONDITION_SCHEMA, but surely you now have to use date somewhere in the code?

It should be somewhere in homeassistant/helpers/condition.py


common: dont override createDataChannel if it does not exist

fixes #799
cc @lgrahl

Still red, haven't looked into the failures.

its just flakyness I assume. kicking the tests often enough.. ;-)

Test failures unrelated, 3 tries and still red.


decentralist, filamentar

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[d,f]

/trunk merge


Committing generated code

My teammates are a bit reticent to commit what's essentially a generated file from a generated file. I can't explain to them why we shouldn't, so I figured I'd ask the maintainers, why do you commit the Cargo.nix in your repo when it could be generated from Cargo.lock?

Good question, @cryptoquick! The reason why cargo2nix currently requires this is because the CLI tool relies on cargo itself to resolve features and generate the complete Nix build plan for the active Cargo workspace. This was chosen to be fully compatible out of the box with upstream Crates.io crates, without needing to reimplement all of Cargo's behavior ourselves, which changes frequently from version to version and has no formal specification. Cargo workspace resolution, as it is currently implemented in cargo2nix, requires access to the Internet and a writable ~/.cargo directory which can be populated with registry index files. This means that cargo2nix -f cannot feasibly run inside the Nix sandbox, at present, which precludes on-the-fly generation of the Cargo.nix expression.
We do plan on adding support for import-from-derivation (see issue #87), which would enable on-the-fly generation of the Cargo.nix from within Nix itself, but there are some technical hurdles to overcome first. Outside contributions to this effort would be most welcome!
Hope this makes some sense. Thanks for asking!

@ebkalderon Thank you for that explanation! And it does actually look like #87 would resolve the issue that had my teammates confused. I subscribed to that issue, and look forward to the day it lands. Unfortunately, I'm not yet experienced enough at Nix to take a crack at it, but who knows, ask me again in a year!

Sounds great! Feel free to leave feedback or follow along with the other issue as you see fit. Any contribution you or your teammates can provide are welcome!


Can't sign in via GitHub when installed on Homescreen on iOS

I can sign in to the main app. instance on the web, both on my phone and on desktop. However, when I sign in (using the GH provider) after installing to home screen, it just returns me back to a sign in screen.

Unfortunately, I can’t replicate this behavior. I just tried and I was able to login on iOS to GitHub with it added to Home Screen.
Can you try removing the app from your phone, adding it again, and trying again? Or maybe update your operating system if you haven’t already? Or maybe clear your cache?
Also, no one else has ever reported this particular behavior, which is often assigned that it has to do with the device rather than the app :(

ha - removing and re adding seems to have fixed it. Apologies for the time waste :D

Not a problem!


Feature/login

This change is

Reviewed 52 of 52 files at r1.
Review status: all files reviewed at latest revision, 7 unresolved discussions.

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 90 at r1 (raw file):

	@Override
	protected void onNewIntent(Intent intent) {

Comment about how this will be called sorta like onStart when the stack gets popped

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 98 at r1 (raw file):

	}

	// Todo: add a log to adobe analytics here

Resolve TODO

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 116 at r1 (raw file):

					startActivity(new Intent(this, Class.forName(feature.getActivityName())));
				} catch (ClassNotFoundException e) {
					e.printStackTrace();

LOG.e

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 120 at r1 (raw file):

			}), null);
			return;
		}

else instead

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 125 at r1 (raw file):

			startActivity(new Intent(this, Class.forName(feature.getActivityName())));
		} catch (ClassNotFoundException e) {
			e.printStackTrace();

LOG.e

support/src/main/java/edu/byu/support/login/LoginActivity.java, line 93 at r1 (raw file):

	}

	public void handleLogin(EditText usernameEdit, EditText passwordEdit) {

Just pass in the Strings if you can (.getText().toString())

support/src/main/java/edu/byu/support/login/LoginSuccessfulCallback.java, line 27 at r1 (raw file):

			}
		} catch (OperationCanceledException | IOException | AuthenticatorException e) {
			e.printStackTrace();

LOG.e

Comments from Reviewable

Review status: all files reviewed at latest revision, 7 unresolved discussions.

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 90 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Comment about how this will be called sorta like onStart when the stack gets popped

Done.

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 98 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Resolve TODO

Done.

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 116 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
LOG.e

Done.

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 120 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
else instead

Done.

app/src/main/java/edu/byu/suite/mainScreen/FavoritesActivity.java, line 125 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
LOG.e

Done.

support/src/main/java/edu/byu/support/login/LoginActivity.java, line 93 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Just pass in the Strings if you can (.getText().toString())

This is a bit more convenient, as username and password are already fields in the class, which are set and used in this function. I'll still change it if you really want me to though

support/src/main/java/edu/byu/support/login/LoginSuccessfulCallback.java, line 27 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
LOG.e

Done.

Comments from Reviewable

Reviewed 1 of 52 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add Package.swift definition

As a package developer I want to copy/paste the dependency definition from the readme, which is missing here. You could add sth like that to the readme to simplify usage:

If you are developing a Swift package, adding Squid as a dependency is as easy as adding it to the dependencies value of your Package.swift like so:

dependencies: [
    .package(url: "https://github.com/borchero/Squid.git", from: "1.0.0")
]

See my PR #2

Fixed by PR #2


feat(rest): Add more fine-grained options for rate limit control

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

Project coverage is 79.00%. Comparing base (88a78f4) to head (dc1e7fc).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
+ Coverage   78.76%   79.00%   +0.24%
==========================================
  Files          16       16
  Lines         259      262       +3
  Branches       59       60       +1
==========================================
+ Hits          204      207       +3
  Misses         48       48
  Partials        7        7

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


added docker componentry for 1.2

adding docker components and such and creating ENV support within swirl

Approved


Bratseth/apply on restart take 9

@gjoranv fyi.
I remembered wrongly - it's building a ConfigInstance from a ConfigInstance.Builder I moved out of the config model and into ConfigInstance.Builder. I tried to put it in configserver instead now to see if this makes it work. Otherwise I think I'll embark on a major cleanup, but would rather not couple that with this as it is enough here as it is.

👍


Android app crashed on request location permission

React-Native version: 0.51.0
Platform: Android
Device: MI A1
OS version: Oreo 8 and Nougat also
react-native-permissions: 1.0.6
Devtools:  3.0.1
(Android only: 23.0.1)

App is getting crashed in android when it requesting for location permission.
This is code the im running
Permissions.request('location', 'whenInUse').then(async (res) => {       console.log("res",res);       if(res == "denied"){         this.setState({           locationGranted: false         })       }     });
And it throws the error
java.lang.SecurityException: Looks like the app doesn't have the permission to access location. Add the following line to your app's AndroidManifest.xml:
LocationModule.java:236 - com.facebook.react.modules.location.LocationModule.throwLocationPermissionMissing
But i added the location permission in manifest file

This is my Manifest file
 <uses-permission android:name="android.permission.INTERNET" />     <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />     <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />      <uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>     <uses-permission android:name="android.permission.CAMERA" />     <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />     <uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />     <uses-permission android:name="android.permission.VIBRATE" />

@MugeshM Try adding
<uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />

Did you solve this @MugeshM

This happens even if both permissions are in the manifest file (ACCESS_COARSE_LOCATION and ACCESS_FINE_LOCATION).

Any solution on this issue ?

I think we have seen this happen to us, reported via Sentry monitoring.
SecurityExceptioncom.facebook.react.modules.location.LocationModule in throwLocationPermissionMissing
fatalLooks like the app doesn't have the permission to access location. Add the following line to your app's AndroidManifest.xml: <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />

But we really do have it in AndroidManifest.xml, and we do not have ACCESS_COARSE_LOCATION.
    <uses-permission android:name="android.permission.INTERNET" />
    <uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>
    <uses-permission android:name="android.permission.READ_PROFILE" />
    <uses-permission android:name="android.permission.READ_CONTACTS" />
    <uses-permission android:name="android.permission.CAMERA" />
    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />

our code is like
const response = await RNPermissions.request('location');

first this module only look for this permission for location, location: PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION, not the COARSE,
in my code at check everytime if permission doesn't return "authorized" then I'm send it to request.
It works, but I'm using the react native api

fwiw, this sounds possibly related/relevant: https://github.com/facebook/react-native/pull/10291

I encounter same problem,
just change
compileSdkVersion
buildToolsVersion
to
compileSdkVersion 25
buildToolsVersion "25.0.1"
and
implementation "com.android.support:appcompat-v7:25.0.1"

@joshuadiezmo unfortunately that is not a good solution for us (and i suspect for many other devs here) because that pushes the permissions UX back to the old "everything at install time" behaviour, i think?

yeah, its not a good solution, but its working on me.

Any other ideas? This is still an issue and downgrading is not an option in our case.

any solution ?!

Any Solution?? I am facing the same problem

I'm trying to replace with PermissionAndroid but it still crash on production.
RN: 0.55.4
Any help pls?

I am closing this since this issue does not exists anymore with the 2.0.0 release.