Allow configuring routing decision when no shard is selected

This an alternative implementation to https://github.com/postgresml/pgcat/pull/567 based on @jmeagher suggestions.
The TL;DR for the change is that we allow QueryRouter to set the active shard to None. This signals to the Pool::get method that we have no shard selected. The get method follows a no_shard_specified_behavior config to know how to route the query.
Original PR description
Ruby-pg library makes a startup query to SET client_encoding to ... if Encoding.default_internal value is set (Code). This query is troublesome because we cannot possibly attach a routing comment to it. PgCat, by default, will route that query to the default shard.
Everything is fine until shard 0 has issues, Clients will all be attempting to send this query to shard0 which increases the connection latency significantly for all clients, even those not interested in shard0
This PR introduces no_shard_specified_behavior that defines the behavior in case we have routing-by-comment enabled but we get a query without a comment. The allowed behaviors are
random: Picks a shard at random
random_healthy: Picks a shard at random favoring shards with the least number of recent connection/checkout errors
shard_<number>: e.g. shard_0, shard_4, etc. picks a specific shard, everytime

In order to achieve this, this PR introduces an error_count on the Address Object that tracks the number of errors since the last checkout and uses that metric to sort shards by error count before making a routing decision.
I didn't want to use address stats to avoid introducing a routing dependency on internal stats (We might do that in the future but I prefer to avoid this for the time being.
I also made changes to the test environment to replace Ruby's TOML reader library, It appears to be abandoned and does not support mixed arrays (which we use in the config toml), and it also does not play nicely with single-quoted regular expressions. I opted for using yj which is a CLI tool that can convert from toml to JSON and back. So I refactor the tests to use that library.

@levkk I have seen the concept of the "default shard" come up in several documentation (including that of Rails) so it is not an outlandish idea to have a default shard and have that be 0.
The reason I went with making this configurable is to avoid breaking this behavior for users. We will probably be using a random routing scheme for unrouted queries and we will experiment with the random_healthy later in the future.

I think configuration here makes sense. I worry that we're adding a lot of obscure configuration settings which somewhat deviates from the original intent of keeping the operation of pgcat as simple as possible, but maybe with all the different Postgres setups out there, that goal was not realistic. Overall, if we keep the default values good, I don't see any issues with adding more config options that may not be immediately useful to the typical user.
I'll read through the PR in a bit, please feel free to merge and use it, if you need it asap.


loss become inf , then Nan

mtwi_2018_train/images/001807_00031.jpg

Train: [1][108/90000]        Time 0.348 (0.361)      Data 0.003 (0.006)      Loss 30.0584 (31.5477)
mtwi_2018_train/images/007947_00013.jpg
Train: [1][109/90000]        Time 0.422 (0.361)      Data 0.003 (0.006)      Loss inf (inf)
mtwi_2018_train/images/002394_00012.jpg
Train: [1][110/90000]        Time 0.332 (0.361)      Data 0.003 (0.006)      Loss nan (nan)

command:
python ./main.py --dataset-root mtwi_2018_train --arch densenet121 --alphabet ./data/alphabet_decode_5990.txt --lr 1e-6 --optimizer rmsprop --gpu-id -1 --workers 1 --not-pretrained --batch-size 1 --keep-ratio --print-freq 1

attach 007947_00013.jpg

I didn't know the alphabet is suitable for this datasets?
Recently, I will download this datasets, and verify this CRNN repo.

And the current Transform used for keep-ratio support all image's width is bigger than height, the attached image is fail for this condition.  Refer issue #11 for more detail.

I have encountered the same problem!

Creating directory if it does not exist:
'./checkpoint/densenet121_rmsprop_lr5.0e-05_wd5.0e-04_bsize64_height32_keep_ratio'
Using model from scratch (random weights) 'densenet121'
Train: [1][1/10]	Time 80.764 (80.764)	Data 0.220 (0.220)	Loss 4.1266 (4.1266)
Train: [1][10/10]	Time 74.218 (75.728)	Data 0.005 (0.025)	Loss 3.2840 (3.6721)
Val: [1][1/2]	Time 17.361 (17.361)	Accu 0.000
Val: [1][2/2]	Time 5.320 (11.340)	Accu 0.000
Train: [2][1/10]	Time 77.894 (77.894)	Data 0.195 (0.195)	Loss 3.2516 (3.2516)
Train: [2][10/10]	Time 79.049 (76.578)	Data 0.005 (0.024)	Loss 2.8698 (3.0496)
Val: [2][1/2]	Time 17.272 (17.272)	Accu 0.000
Val: [2][2/2]	Time 5.313 (11.292)	Accu 0.000
Train: [3][1/10]	Time 75.839 (75.839)	Data 0.210 (0.210)	Loss 2.8418 (2.8418)
Train: [3][10/10]	Time 74.184 (76.081)	Data 0.005 (0.026)	Loss inf (inf)
Val: [3][1/2]	Time 17.391 (17.391)	Accu 0.000
Val: [3][2/2]	Time 5.323 (11.357)	Accu 0.000
Train: [4][1/10]	Time 63.459 (63.459)	Data 0.209 (0.209)	Loss nan (nan)
Train: [4][10/10]	Time 66.387 (65.531)	Data 0.005 (0.026)	Loss nan (nan)
Val: [4][1/2]	Time 17.400 (17.400)	Accu 0.000
Val: [4][2/2]	Time 5.370 (11.385)	Accu 0.000
Train: [5][1/10]	Time 67.657 (67.657)	Data 0.226 (0.226)	Loss nan (nan)
Train: [5][10/10]	Time 65.224 (65.783)	Data 0.005 (0.027)	Loss nan (nan)
Val: [5][1/2]	Time 17.329 (17.329)	Accu 0.000
Val: [5][2/2]	Time 5.386 (11.357)	Accu 0.000
Train: [6][1/10]	Time 67.789 (67.789)	Data 0.215 (0.215)	Loss nan (nan)
Train: [6][10/10]	Time 65.523 (65.778)	Data 0.006 (0.025)	Loss nan (nan)
Val: [6][1/2]	Time 17.458 (17.458)	Accu 0.000
Val: [6][2/2]	Time 5.400 (11.429)	Accu 0.000
Train: [7][1/10]	Time 66.216 (66.216)	Data 0.210 (0.210)	Loss nan (nan)
Train: [7][10/10]	Time 110.504 (78.189)	Data 0.009 (0.026)	Loss nan (nan)
Val: [7][1/2]	Time 29.770 (29.770)	Accu 0.000
Val: [7][2/2]	Time 9.196 (19.483)	Accu 0.000
Train: [8][1/10]	Time 112.352 (112.352)	Data 0.296 (0.296)	Loss nan (nan)

command:
python ./main.py --dataset-root ./data/cropped_images --arch densenet121 --lr 5e-5 --optimizer rmsprop --not-pretrained --keep-ratio --batch-size 64
I train this model using cropped images of bankcard in order to recognize the number in that.

@USTC-Keyanjie I think the arch densenet_cifar is prefer for this datasets.

@USTC-Keyanjie Besides, I have trained a model with only numbers, and parameter keep-ratio performs bad, I think scale the images to fixed size is the best option. BTW, the mean and std is significant for small datasets.

Thank you for your reply so quickly. I'm trying that.
感谢您这么快的回复。我正在尝试。
Best Regards.
顺祝时祺。

@USTC-Keyanjie Once your model is trained, you can generate text recognition using keep-ratio.

I have the same problem that after some epochs the loss just gets nan.
The parameters I train my network with is:
--gpu-id 0 --not-pretrained --arch densenet_cifar --dataset-root /net/dataset/voegtlil/chinese_generated_first_set/split --alphabet ./data/alphabet_decode_5990.txt --lr 5e-5
Example file:

All my images are generated and have the GT in this structure:
00000003.jpg 1160 1460 3068 1686 5574 4533 2955 5616 5784 5828

@lvoegtlin It seems that this dataset is same with YCG09's? For this datasets, architecture densenet121 is prefer.

@zhiqwang it is similar to YCG09. I use a densenet121 but I still run into the nan problem after some epochs

@lvoegtlin Ohh, Can you please provide a more detail description of this datasets. Actually I didn't know last six Chinese characters the example you provide, it is rarely seen.

And I use the last stable version of Pytorch to train this repo, And set criterion = nn.CTCLoss(zero_infinity=True) in this line.

The dataset is just randomly generated based on alphabet_decode_5990.txt. And as I don't speak Chinese I don't know what it generates and it also does not matter :)

BTW I use the last stable version 1.1 of Pytorch to train this repo, and set criterion = nn.CTCLoss(zero_infinity=True) in this line.

Illl try this

@lvoegtlin I check the image you provide, the width of this image is 256, is it the same for all other images?

Yes, all my training images are 32*256.

@lvoegtlin Then the option width=256 is significant.

Or simply comment this line.

替换了mean 和std ,所有图像resize到统一大小，训练几步就出现nan，换了几个backbone也不行，到底是什么原因呢，费解加无所适从 ,求指点  PS: pytorch 1.0.1, python3.6

建议换最新的 pytorch，CTCLoss pytorch 的实现有部分 bug，讨论在 https://github.com/pytorch/pytorch/pull/21244


Add Italian translation

Resolves #399
Checklist

[x] Review Contribution Guidelines.
[x] Make sure all contributed code can be distributed under the terms of the
Apache License 2.0, e.g. the code was written by
you or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Update documentation when applicable.

I just noticed from the twitter screenshot that the quote characters are all displayed. I thought both of them were needed based on the discussion of #401, but maybe one is enough? Should I remove the second ones?

Right. I missed that too. Apparently there is no need to double quote.


每次保存怎么页面都刷新了一次？

Clear and concise description of the problem
每次保存怎么页面都刷新了一次？不是应该存在热更新的吗？
Suggested solution
建议每次保存，自动热更新，而不是页面每次刷新一次，眼睛都花掉了
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

please follow issue template

Please list the plugins used. Let me guess you could try to replace the deprecated plugin @vite/plugin-react-refresh to @vite/plugin-react.

@a524313390 兄弟,  你的问题解决了吗? 我更新到2.7.10也是.


Add member Megan

Overview
Adding one more Latina Dev! 😄
Relevant Issue
https://github.com/Latina-Dev/latina-dev/issues/57

✅ By submitting this PR, I have verified the following

[x] Reviewed the contributing guidelines 🔍
[x] Added my name to the bottom of the list under the Contributors section in the README.md with a link to my personal website or GitHub profile 👥

Otherwise looks good!

Can we remove the change where the analytics component is moved slightly? It's not related and I think prettier would move it back anyways in next diff

Megan's photo looks stretched out on my end. @amandhawb: Can you resize to a 250px square and see if that looks better? The image is rectangular right now and I think that's why it. Thank you!

Can we remove the change where the analytics component is moved slightly? It's not related and I think prettier would move it back anyways in next diff

Megan's photo looks stretched out on my end. @amandhawb: Can you resize to a 250px square and see if that looks better? The image is rectangular right now and I think that's why it. Thank you!

I resized it @EllaCodes4 , not sure if it changed because of the cache... I am still trying to see how to configure it

Can we remove the change where the analytics component is moved slightly? It's not related and I think prettier would move it back anyways in next diff

This was done, thank you! Just made two comments on the image + file name.

image is low res but otherwise looks good!


reserve_network_port not finding free ports

Environment:   Mac 10.12.6, JDK 1.8, maven 3.5
                   <plugin>
                        <groupId>org.codehaus.mojo</groupId>
                        <artifactId>build-helper-maven-plugin</artifactId>
                        <version>3.0.0</version>
                        <executions>
                            <execution>
                                <id>integration-test-ports</id>
                                <phase>pre-integration-test</phase>
                                <goals>
                                    <goal>reserve-network-port</goal>
                                </goals>
                                <configuration>
                                    <minPortNumber>7000</minPortNumber>
                                    <maxPortNumber>8000</maxPortNumber>
                                    <outputFile>${build.directory}/ports.properties</outputFile>
                                    <portNames>
                                        <portName>db.port</portName>
                                        <portName>memcache.port</portName>
                                    </portNames>
                                </configuration>
                            </execution>
                        </executions>
                    </plugin>

```After this plugin runs, we launch a couple of docker containers based on these values.  So far so good.

Typical output:

[INFO] --- build-helper-maven-plugin:3.0.0:reserve-network-port (integration-test-ports)
[INFO] Port assigned7000
[INFO] Reserved port 7000 for db.port
[INFO] Port assigned7001
[INFO] Reserved port 7001 for memcache.port

However, if port 7000 is occupied as shown here:

netstat -tan | grep LISTEN
tcp6       0      0  ::1.7000               .                    LISTEN
tcp4       0      0  *.7000                 .                    LISTEN
tcp6       0      0  ::1.7001               .                    LISTEN
tcp4       0      0  *.7001                 .                    LISTEN
Then when I run the plugin I still get:

[INFO] --- build-helper-maven-plugin:3.0.0:reserve-network-port (integration-test-ports)
[INFO] Port assigned7000
[INFO] Reserved port 7000 for db.port
[INFO] Port assigned7001
[INFO] Reserved port 7001 for memcache.port

And the docker containers fail to launch because of network binding issue.

Workaround to reduce collisions is to randomize the port, but it won't eliminate them altogether.  Thanks.

How do you launch the docker containers? Using docker compose? via command line or via a maven plugin?

Ultimately I launch the docker containers via fabric8.io's docker-maven-plugin, passing in the port numbers per what's been defined here.
Rough example:  (where db.port is assigned by the build-helper-maven-plugin and pulled in by properties-maven-plugin).  I've left out a few properties in the docker-maven-plugin that were specific to a private container.
                    <plugin>
                        <groupId>org.codehaus.mojo</groupId>
                        <artifactId>properties-maven-plugin</artifactId>
                        <executions>
                            <execution>
                                <id>load-port-properties</id>
                                <phase>pre-integration-test</phase>
                                <goals>
                                    <goal>read-project-properties</goal>
                                </goals>
                                <configuration>
                                    <files>
                                        <file>${project.build.directory}/ports.properties</file>
                                    </files>
                                </configuration>
                            </execution>
                        </executions>
                    </plugin>

                      <plugin>
                        <groupId>io.fabric8</groupId>
                        <artifactId>docker-maven-plugin</artifactId>
                        <version>0.21.0</version>
                        <configuration>
                            <logDate>default</logDate>
                            <autoPull>true</autoPull>
                           <image>
                                    <alias>db</alias>
                                    <name>mysql-container-example</name>
                                    <run>
                                        <ports>
                                            <port>${db.port}:3306</port>
                                        </ports>
                                        /.. More run options specific to the customized container we're using..../
                                  </run>
                          </image>
                     </configuration>
                        <executions>
                            <execution>
                                <id>start-docker-db</id>
                                <phase>pre-integration-test</phase>
                                <goals>
                                    <goal>start</goal>
                                </goals>
                            </execution>
                            <execution>
                                <id>stop-docker-db</id>
                                <phase>post-integration-test</phase>
                                <goals>
                                    <goal>stop</goal>
                                </goals>
                            </execution>
                        </executions>
            </plugin>

And yes, my current workaround is to make it a random port so collisions are rare.
Hope this clarifies


editor loses focus when selection suggestion with mouse

open ts file
trigger IntelliSense
select a suggestion with mouse
🐛 the editor doesn't have focus...

https://github.com/Microsoft/vscode/commit/6fa2266435d937f0fab15379dc8c62135647ffc7 is a bad fix

Reverted.


ComboBox Itemsource

When use ComboboxRevealStyle, display member path is not show in comboBox popup.

Thanks for the feedback. I could reproduce it, and fixed it.
It will included in FluentWPF 0.10.0.


Only talk to compact index API

Experimenting with an alternative to #7262.

This does not seem to work and I think I found a proper solution at https://github.com/dependabot/dependabot-core/pull/7269. Closing.


Support: recover already signed users from lost GitHub user credentials

Hi!
Is there a way to recover already signed information from a lost GitHub credential account linked to CLA-assistant?
Ideally, we would like to do a CSV export/import from the old account to a new account.
Thanks for your help!

What do you mean with lost GitHub user credentials? You have linked a repo with a user, whose credentials are lost now and you can not access the cla-assistant with this user anymore, is this the problem? Or have you signed multiple CLAs with the account and you don't want to sign all theese documents again with a new user?

What do you mean with lost GitHub user credentials? You have linked a repo with a user, whose credentials are lost now and you can not access the cla-assistant with this user anymore, is this the problem?

This is the exact problem

Ok then I believe, the new user has become an owner/admin of the repo now. The signatures are not connected to a user, but to the linked repository/organization and gist file. This means, that you can just log into the CLA assistant with the new user (who has admin right on the repository) and you should see all linked repos and signatures there.

Will try this, thanks!

Hi!
Sorry for not closing this issue. I finally regain access through another account on the repo.
Thanks!


Repository-Validation: More specific Error Message

-fixes #2914

thanks, welcome to abapGit as a contributor 🎉


feat!: Allow panic operation to have any input and output wires

Closes #915 .
BREAKING CHANGE: PANIC_OP_ID now instantiated with expected input and output types.

It might be slightly better in terms of future-proofing to have this op take tuples of inputs and outputs, instead of rows. My reasoning is that in the future, when we have row kinded type params, we should be able to define this op without needing compute_signature.
I.e. inputs : Row. outputs : Row. panic : Error, Tuple(inputs) -> Tuple(outputs)
If we define it in terms of tuples now, there's less friction if we rewrite it this way

Good point, though it would make some usages a bit less convenient. I will make a new issue for this.


Where we can place skip: true is unclear.

Describe the bug
The config file mentioned when you set skip: true, a "given config section is not set AT ALL for given project."
I placed skip: true under one of the branches in the branches section because I didn't want to inherit the branch
settings from the group level. This is similar to the inheritance break issue.
e.g.
projects_and_groups:
  some-group/*:
    branches:
      main:
        protected: true
        unprotect_access_level: maintainer
        push_access_level: no access
        merge_access_level: developer
        code_owner_approval_required: true

  some-group/some-project:
    branches:
      main:
        skip: true

I expected skip: true to work in any section or at least similar to the files section.
GitLabForm version
GitLabForm version: 2.10.0
GitLab version
F.e. 14.5.2-ee

As of now „skip: true” works only for the whole sections, so f.e. all branches or all files.
We would like to make this more configurable in gitlabform v3, see (link).

Hmm.. With the inherit: false working at any place in the config it could be used as a replacement for skip: true. Am I right or am I missing something, @ep-linden? 🤔

Yeah, it should make the configs under the main branch empty because it won't inherit anything. So I would say they are similar.

Maybe skip config can be deprecated in future minor release and remove in v3. Would be nice if the tool can print warning if this flag gets used while deprecated.

@amimas : yes, we strive for exactly such way of deprecating syntax - warning first, remove support later.


Complex damage formula roll but do not create a draggable damage box

Using the ammunition/warhead formulae from UT can result in complex damage rolls. 6d×3+18 is one possibility. When foundry encounters such a formula, it will roll but the result will not be draggable to a token to trigger the ADD. See image. I am including the gca and xml files.

complex_damage_formulae.zip

Sorry... that one is just out of our parsing capabilities.    We support:
6d+2x3, where the addition occurs after the dice, but not an addition that occurs after the multiplier.      We would need to add another layer to our damage parsing.    It is possible... but would touch on a lot of methods, so I don't know when we would be able to get to it.

That is fair. I have a workaround and I expect the need/use of such formulas to be rare. Feel free to close the issue, or put it in backlog.


feat: add more delegate choices for TFLite backend

Closes #708

@freedomtan Please check if I chose the correct model for the GPU delegate.

Using this config
  delegate_choice: {
    delegate_name: "GPU"
    accelerator_name: "gpu"
    accelerator_desc: "GPU"
    model_path: "https://github.com/mlcommons/mobile_models/raw/main/v0_7/tflite/mobilenet_edgetpu_224_1.0_float.tflite"
    model_checksum: "0da3fa6adc8bbaaeb0b5647e2d46c8a4"
    batch_size: 2
  }

my Samsung phone crashed with this log
E/tflite  (20060): TfLiteGpuDelegate Invoke: GpuDelegate must run on the same thread where it was initialized.
E/tflite  (20060): Node number 78 (TfLiteGpuDelegateV2) failed to invoke.
E/native  (20060): cpp/backend_tflite/tflite_c.cc:477 Failed to run the inference
F/native  (20060): flutter/cpp/backends/external.h:158 Error while inferencing model
F/libc    (20060): Fatal signal 6 (SIGABRT), code -1 (SI_QUEUE) in tid 20216 (DartWorker), pid 20060 (oid.mlperfbench)

So I removed it for now.

that's because the mobilenet_edgetpu_224_1.0_float.tflite doesn't support dynamic batch size.

These models look fine. However, we need
<uses-library android:name="libOpenCL.so"
           android:required="false"/>

<uses-library android:name="libOpenCL-pixel.so"
           android:required="false"/>
....

otherwise, on Android 12 or later, we got messages like
05-30 14:54:57.407 12551 12581 I native  : cpp/flutter/dart_backend_match.cc:12 checking backend 'libtflitebackend.so' ...
05-30 14:54:57.408 12551 12581 I native  : cpp/backend_tflite/tflite_c.cc:199 TFLite backend matches hardware
05-30 14:55:18.045 12551 12629 I native  : cpp/flutter/dart_run_benchmark.cc:31 in->backend_model_path=/storage/emulated/0/Android/data/org.mlcommons.android.mlperfbench/files/loaded_resources/github.com/mlcommons/mobile_models/raw/main/v0_7/tflite/mobilenet_edgetpu_224_1.0_float.tflite
05-30 14:55:18.045 12551 12629 I native  : cpp/flutter/dart_run_benchmark.cc:32 in->backend_lib_path=libtflitebackend.so
05-30 14:55:18.046 12551 12629 I native  :   framework: "TFLite"
05-30 14:55:18.046 12551 12629 I native  :     model_path: "https://github.com/mlcommons/mobile_models/raw/main/v0_7/tflite/mobilenet_edgetpu_224_1.0_uint8.tflite"
05-30 14:55:18.046 12551 12629 I native  :     model_path: "https://github.com/mlcommons/mobile_models/raw/main/v0_7/tflite/mobilenet_edgetpu_224_1.0_float.tflite"
05-30 14:55:18.049 12551 12629 I tflite  : Created TensorFlow Lite delegate for GPU.
05-30 14:55:18.050 12551 12629 I tflite  : Initialized TensorFlow Lite runtime.
05-30 14:55:18.052 12551 12629 I tflite  : Replacing 77 node(s) with delegate (TfLiteGpuDelegateV2) node, yielding 1 partitions.
05-30 14:55:18.083 12551 12629 E tflite  : Can not open OpenCL library on this device - dlopen failed: library "libOpenCL.so" not found
05-30 14:55:18.083 12551 12629 E tflite  : Falling back to OpenGL
05-30 14:55:18.092 12551 12629 I tflite  : Initialized OpenGL-based API.
05-30 14:55:18.190 12551 12629 I tflite  : Created 1 GPU delegate kernels.
05-30 14:56:18.969 12551 12629 I native  : cpp/flutter/dart_run_benchmark.cc:147 out->backend_name=TFLite

that's because the mobilenet_edgetpu_224_1.0_float.tflite doesn't support dynamic batch size. We can convert a dynamic batch size one from TF pb file.

I will convert the model and add a new delegate choice for it in another PR.


feat(snapshots): write Headers to snapshots during HeaderStage

Writes Header, HeaderTD and HeaderHash to static files during the HeaderStage.
Some necessary changes that might require a more careful look at

genesis initialization takes a ProviderFactory equipped with a snapshot provider
sync_gap() and the header downloader changes.

Tested holesky :heavy_check_mark:
TODO before merge ready - change HeaderStage::tests and usage of tx.get::<tables::Headers

rfr, state tests are failing because they're getting canceled


logged out users aren't redirected from dashboard

If you are logged out of the app (or get logged out if the auth token expires), you can still visit any of the routes (except for /profile) and it will throw a TypeError.
We need to either make all routes a PrivateRoute or render them conditionally based on isAuthenticated.

PrivateRoute already set up, made the public routes private


Update main.inc.php - Issues #40 #41

When updating to version 2.0.3, the version number in main.inc.php was forgotten to be updated.
This change fixes the error message that version 2.0.3 is not compatible with Piwigo.

I confirm. Easy to fix by changing the version number.
Even better fixed with the "auto" version which is automatically replaced when publishing a new version on piwigo.org/ext (see https://github.com/plegall/Piwigo-birthdate/blob/master/main.inc.php for an example)


Add list call

Pull Request Test Coverage Report for Build 14

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.667%

Totals

Change from base Build 11:
0.0%

Covered Lines:
58

Relevant Lines:
60

💛  - Coveralls

Pull Request Test Coverage Report for Build 14

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.667%

Totals

Change from base Build 11:
0.0%

Covered Lines:
58

Relevant Lines:
60

💛  - Coveralls

Pull Request Test Coverage Report for Build 14

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.667%

Totals

Change from base Build 11:
0.0%

Covered Lines:
58

Relevant Lines:
60

💛  - Coveralls

Pull Request Test Coverage Report for Build 14

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.667%

Totals

Change from base Build 11:
0.0%

Covered Lines:
58

Relevant Lines:
60

💛  - Coveralls


Ensure RSP cruft is removed from everywhere...

Does creating the user work / etc.?

Duplicate of #78


[Tab] should escape meta characters before sending to jquery

When tabPath is sent to jQuery in tab.js#L382, it does not escape meta characters such as !"#$%&'()*+,./:;<=>?@[]`{|}~, which can result in uncaught errors.
e.g. kramdown creates footnote fragment links in the format #fn:4, which results in the following error from tab.js+jQuery:
Uncaught Error: Syntax error, unrecognized expression: unsupported pseudo: 4

According the the jQuery docs, meta characters should be escaped via double backslashes (\\): http://api.jquery.com/category/selectors/

jsfiddle to reproduce issue: https://jsfiddle.net/ellemenno/f1tdw9pm/

Added escaping in https://github.com/fomantic/Fomantic-UI
See your adjusted fiddle here https://jsfiddle.net/us495x0f/

@lubber-de —thanks, that was fast!
It looks like FUI also has a supported SASS distro? hmm.. i'll look at this project more closely.


SignalR.Http throws System.MissingMethodException

I tried to setup a new project from scratch using 1.0.0-preview3-25879.
It seems that in order to use app.UseSignalR I have to include the package Microsoft.AspNetCore.SignalR.Http. But when I add this package the following exception is thrown in WebHost.CreateDefaultBuilder during startup:
System.MissingMethodException: 'Method not found: 'Microsoft.AspNetCore.Hosting.IWebHostBuilder Microsoft.AspNetCore.Hosting.IWebHostBuilder.ConfigureLogging(System.Action2<Microsoft.AspNetCore.Hosting.WebHostBuilderContext,!!0>)'.'

You're probably mixing preview1 and preview3 bits, you can't. Update everything else.

Thank you, this was the issue. I upgraded Microsoft.AspNetCore.All to 2.0.0-preview3-25879 and now it works. I was just a little irritated, because in your MSBuild talk you mixed preview1 and preview2...
While I try to get your demo from the MSBuild running, I'd like to ask a follow-up question:
On the client side i do the following:
`
<script>
    var conn = new signalR.HubConnection('http://localhost:31100/livetable'); // separate service

    conn.on('Send', data => {
        // process...
    });

    conn.start()
        .then(() => {
            console.log('Started');
        })
        .catch(error => {
            console.log('Error');
        });`

But I get the error
Uncaught TypeError: Cannot create property 'onDataReceived' on string 'http://localhost:31100/livetable'     at new HubConnection (signalr-client.js:282)     at index.html:17
Has the way of creating clients changed or am I missing something?
Thank you in advance.

I'm sorry, after asking the question I found out, how to use it properly:
var conn = new signalR.HubConnection(new signalR.HttpConnection('http://localhost:31100/livetable'));

👍


fix: remove default_opset annotation

Previous Opset14 was incorrect.

Removing the annotation is no good because intellisense will stop knowing what default_opset is

Removing the annotation is no good because intellisense will stop knowing what default_opset is

Yes, that's what I mentioned earlier! The real answer may be that users don't ever use it directly.

Ahh ok sorry I misread. That makes sense to me. I will abort the pr for now


update 2.60.0 changelog to include a fix in Bigtable

Add the bug fix in BigtableIO to the changelog of 2.60.0.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

thanks, let's add the same item to CHANGES.md

@mutianf could you please take a look at the comments above?

@mutianf could you please take a look at the comments above?

I'll merge once checks pass


Fix date time creation when microtime has no decimals

This PR fixes issue discussed in #30

@estevejm
@bartkamphuis
@shieldo
Partners we have many exceptions from the test and several from the production
http://clip2net.com/s/3CbwbTn
Could you investigate this issue and  fixed this in near time

@estevejm
@bartkamphuis
@shieldo
Hi, we still get exceptions. We are waiting the merge.

@TkachukRuslan I don't have write access, hence I can't merge it.
I think @anyarms is one of the maintainers...

@anyarms
@estevejm
Partners if the issue was fixed please merge it.

I'm no longer with Avalara, so I don't have write access anymore - sorry!
@JanzenMark can you track down the right person to review this PR?

@JanzenMark
Can you please know us the terms when will you merge  it I mean the "merge" not dev merge
Thank you

You should probably just use a fork until this is merged into the main repo.
On 30 Sep 2016 5:44 pm, "TkachukRuslan"<EMAIL_ADDRESS>wrote:

@JanzenMark https://github.com/JanzenMark
Can you please know us the terms when will you merge it I mean the "merge"
not dev merge
Thank you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avadev/AvaTax-Calc-REST-PHP/pull/35#issuecomment-250779020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAF8AEz1KMQW9l6sOczeOgkOchzRekXvks5qvS5MgaJpZM4JzhEC
.

I opened a case with Avalara support to start watching this repo again. We will see if that request gets answered or not.

@skylord123
Thank you. Please let us know when this issue will be merged in master.

@TkachukRuslan The code is now merged in Master branch.

Hi All.
Thank you for a feedback.
I have one more issue which doesn't have fix result from developers end.
Please follow to  https://github.com/avadev/AvaTax-Calc-REST-PHP/issues/31
Let me know when the issue will be fixed ? Waiting а marge in master branch.
Waiting your response.
Ruslan Tkachuk
Project Manager
Net-Craft.com
On Mon, Nov 7, 2016 at 2:09 PM, Rahul Aggarwal<EMAIL_ADDRESS>wrote:

@TkachukRuslan https://github.com/TkachukRuslan The code is now merged
in Master branch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avadev/AvaTax-Calc-REST-PHP/pull/35#issuecomment-258819730,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATYYBhK-vDkk2tC6XW2lhyiORHeBeapeks5q7xTfgaJpZM4JzhEC
.

Hi @TkachukRuslan ,
We are reviewing all the changes mentioned at https://github.com/avadev/AvaTax-Calc-REST-PHP/pull/33
These should be done by 18th Oct.
We'll update here as it is done.
Thanks,
Vijay

Hi Vijay,
Could you please tell us how many time do you need to do with this ?
We would like to finish soon because our clients are waiting the fixes.
Thank you
Ruslan Tkachuk
Project Manager
Net-Craft.com
On Tue, Nov 8, 2016 at 2:50 PM, Vijay Nalawade<EMAIL_ADDRESS>wrote:

Hi @TkachukRuslan https://github.com/TkachukRuslan ,
We are reviewing all the changes mentioned at #33
https://github.com/avadev/AvaTax-Calc-REST-PHP/pull/33
These should be done by 18th Oct.
We'll update here as it is done.
Thanks,
Vijay
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avadev/AvaTax-Calc-REST-PHP/pull/35#issuecomment-259128140,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATYYBlZKdxU1P5bFvIDI1zgJ310MfTQ-ks5q8HACgaJpZM4JzhEC
.

Hi @TkachukRuslan ,
Sorry for the delay. We've taken this task on priority & will merge it by 18th Nov.
Thanks,
Vijay

Hi,
Vijay
Thank you for feedback.
We will be waiting updates from your end till 18th Nov or earlier.
Thank you
Ruslan Tkachuk
Project Manager
Net-Craft.com
On Wed, Nov 9, 2016 at 4:47 PM, Vijay Nalawade<EMAIL_ADDRESS>wrote:

Hi @TkachukRuslan https://github.com/TkachukRuslan ,
Sorry for the delay. We've taken this task on priority & will merge it by
18th Nov.
Thanks,
Vijay
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avadev/AvaTax-Calc-REST-PHP/pull/35#issuecomment-259431324,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATYYBrbXTUbhjT-LRELkT1dnN73v4vmRks5q8d0JgaJpZM4JzhEC
.

Hi @TkachukRuslan,
We've updated pull requests to master branch.
Thanks,
Vijay

Hi All
Thank you for updates.
Ruslan Tkachuk
Project Manager
Net-Craft.com
On Thu, Nov 17, 2016 at 4:38 PM, Vijay Nalawade<EMAIL_ADDRESS>wrote:

Hi @TkachukRuslan https://github.com/TkachukRuslan,
We've updated pull requests to master branch.
Thanks,
Vijay
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/avadev/AvaTax-Calc-REST-PHP/pull/35#issuecomment-261262742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATYYBpnctHgaa8A8wybvlwHExOcrvEB7ks5q_GbOgaJpZM4JzhEC
.


Add GHA workflow

How is this supposed to work? What triggers the build? @egornovivan

What triggers the build?

Any push to these branches
on:
 push:
  branches:
  - 'master'
  - 'develop'
  - 'github-action'

I'll do a full build a little later. RAD Studio somewhat problematic

Look at the settings

all settings are good, i think i found the problem
https://github.com/FunkyFr3sh/cnc-ddraw/blob/master/.github/workflows/build.yml#L9
it only builds if you modify these two files

if you modify these two files

ou, I forgot that I deleted the line from there
  paths:
  - '.github/workflows/build.yml'
  - 'action.yml'
  - '**'

Write it there ** But anyway, later I’ll complete the work flow for the RAD Studio project


compile error in build.sbt

since https://github.com/rickynils/scalacheck/commit/2cdfc5512a48672c6c1439f1aca4271f8c05353e
https://github.com/rickynils/scalacheck/blob/2cdfc5512a48672c6c1439f1aca4271f8c05353e/build.sbt#L81-L84
/home/travis/build/[secure]/scalacheck/build.sbt:83: error: type mismatch;
 found   : String("org.scalacheck")
 required: ?{def %%%(x$1: ? >: String("scalacheck")): ?}
Note that implicit conversions are not applicable because they are ambiguous:
 both method toScalaJSGroupID in trait DependencyBuilders of type (groupID: String)org.scalajs.sbtplugin.impl.ScalaJSGroupID
 and method toPlatformDepsGroupID in object autoImport of type (groupID: String)org.portablescala.sbtplatformdeps.PlatformDepsGroupID
 are possible conversion functions from String("org.scalacheck") to ?{def %%%(x$1: ? >: String("scalacheck")): ?}
      Set("org.scalacheck" %%% "scalacheck" % "1.14.0")
          ^
/home/travis/build/[secure]/scalacheck/build.sbt:83: error: value %%% is not a member of String
      Set("org.scalacheck" %%% "scalacheck" % "1.14.0")
                           ^
[error] sbt.compiler.EvalException: Type error in expression

Fixed in #399


How to use .js instead of .ts?

How to use a JavaScript file .js instead of a TypeScript file .ts? Changing the extension fails the building process.

To anyone struggling with the same problem: I modified Simon Edwards excellent starter template to use Javascript instead of Typescript. It can be found here https://github.com/scanline/nodegui-simple-starter-js/


Firebase PlatformException(channel-error, unable to establish connection on channel., null, null)

Hey,
I integrated firebase core and firebase messaging in my app. It works fine in Android but there is an error in iOS:
PlatformException(channel-error, unable to establish connection on channel., null, null)
After removing packages one by one, i discovered removal of talkjs_flutter solved the issue.
I am using the latest firebase_core: 2.14.0 and firebase_messaging: 14.6.3 with talkjs_flutter: 0.8.1

Did you change your Info.plist file as described here?
https://talkjs.com/docs/Features/Notifications/Mobile_Push_Notifications/Flutter/#using-other-firebase-services-on-ios-optional
You need to do so in order to use Firebase on iOS.

Did you change your Info.plist file as described here?
https://talkjs.com/docs/Features/Notifications/Mobile_Push_Notifications/Flutter/#using-other-firebase-services-on-ios-optional
You need to do so in order to use Firebase on iOS.

Did you change your Info.plist file as described here?
https://talkjs.com/docs/Features/Notifications/Mobile_Push_Notifications/Flutter/#using-other-firebase-services-on-ios-optional
You need to do so in order to use Firebase on iOS.

worked when i added it, thanks


Too many expand/collapse icons in problems

This doesn't look right:

Version: 1.66.0-insider (user setup)
Commit: 6b7453421201c888b81eae5c04ec822a6423608f
Date: 2022-03-11T05:17:19.559Z
Electron: 17.1.1
Chromium: 98.0.4758.109
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000

Agreed that this doesn't look great, seems like the third expander should just be part of the second? I haven't been able to get into a state like that yet:

What @Tyriar seeing is a multiline problem that has related information. @misolori 's screenshot has simple (single line) problem with related information.
Currently related information is a separate entry that can selected so that user can open it. Hence it is rendered as child of the problem. For multiline message problem, only first line is shown and with an expand icon so that user can expand it to see the complete message. All lines are part of the problem and shall be selected when clicking on the problem entry.
I am not sure if we can merge multiline and related information into single twisty.
@misolori Is it possible to use a different icon for showing and hiding a multiline message?


Update CodeMirror CSS URL

Fixes https://github.com/GoogleCloudPlatform/bigquery-geo-viz/issues/74.
It seems that codemirror.net doesn't allow third-party to reference its CSS anymore, breaking the query input box. Load the CSS from JSdelivr instead (arbitrary choice, any CDN should do).
A better fix might be to bundle the CSS at build-time but I'm not familiar enough with Angular to do that.

I've updated the deployment. Could you please sign the CLA so I can merge the change?
https://cla.developers.google.com/about/google-corporate

Thank you so much.

CLA should be signed now @mentin (cla/google check is passing).


Dynamic Selection sort

Fixes #[Add issue number here."First steps for issues #IssueNumber]
Language: [Add here in what language you have implemented this data structure or algorithm.]

Dynamic selection sort

Dynamic Selection sort


Increase default timeout

Should we increase the default timeout? From 10 to 20? I noticed this seems to significantly decrease the number of timed out sites. For example, 20% of sites from the most recent scheduled crawl errored (about 2/3 of that are timeouts, or something like that). (Do we have any historical logs to see what the timeout rate is generally? Would be useful for gauging effectiveness.)
Fewer sites timing out should mean more stability to crawl results which means more meaningful comparisons between runs.

Finished running a 2,000 site crawl with timeout set to 20 seconds. Ran for just over 8.5 hours.
330 sites errored, a 16.5% error rate. Better than 20%, but still high!
Let's look at the error breakdown:

86 "timeout on" timeouts (we're double counting here I think, often both the https and http version of the same domain time out)

The rest are "WebDriverException: Reached error page" Firefox error pages:

117 DNS not found errors
83 netTimeouts (how are these different from "timeout on"?)
79 connection failures
13 misc. SSL errors
8 server resets
3 connection interrupted errors
2 redirect loops
1 unknown protocol

Total Firefox error page count: 306. If 330 sites errored according to log output at the end, this leaves us with only 24 "timeout on" sites, which doesn't make sense given the 86 "timeout on" instances in the log. Not clear yet where the counting went wrong.

Latest scheduled run claims a ~20% error rate again:

Visited 1607 sites and errored on 393.

We should look into #21 more, but in the meantime I think it's harmless to do this too.

Closed with 13c252d050014145b7e95820067922def4e0876b

In the latest scan, after merging #23 and increasing the timeout, there were 273 errors (13%).


Allow example to run on CI and parse URI from args

Closes https://github.com/FerretDB/dance/issues/479.

@AlekSi PTAL


Getting an error in Geonode when adding a new layer from a Geopackage store

Hi All
I am trying to publish a new layer from a Geopackage store which I created using the following code
cat = Catalog("http://localhost:8080/geoserver/rest/")
geoserver_workspace = cat.get_workspace("myworkspace")

ds = cat.create_datastore(store_name, geoserver_workspace)

ds.connection_parameters.update(database='~/Desktop/study001_ins00.gpkg',
                                    dbtype='geopkg')

The above code does create a store in the Geoserver but the path of the database which I am giving is on local hard drive and not on the server.
This creates a problem when I manually try to publish a new layer using the above store
Could not list layers for this store, an error occurred retrieving them: null

Can some one help me in giving the right parameters to connection_parameters.update() function so that the store is correctly created in the geoserver location from the geopackage present in local drive.
I am using the gsconfig Python library as a wrapper to accessing Geoserver REST API
https://github.com/boundlessgeo/gsconfig
When using shp file, the store is created in the Geoserver at file:/workshop/geonode/geoserver/data/data/myworkspace/study001_ins00/
using the create_featurestore() function
Geoserver Version is <IP_ADDRESS>.

Was able to import geopackage using the REST API