Back to top button takes to top of the page.

Fixes #40
Description
Changed the back to top button to direct to the top of the web page and not the first heading.
Other information
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[x] My commit messages follow best practices.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentttion (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin
Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

@kgodey Please review

Done! @kgodey
Should I squash the commits?

Please check the items of the checklist in the PR (after ensuring that you've verified that all those things are done). Don't squash your commits!

Yes sorry I missed that @kgodey

I did that to check the responsiveness on mobile. I agree xl is better to use. Sorry forgot to change it back in the PR.

Merging this. @sanjana11147 please write more descriptive commit messages in the future, the pull request template links to guidelines.

Okay thanks! @kgodey


Remove unused openapi marker

Signed-off-by: Lan Luo<EMAIL_ADDRESS>

Codecov Report

Merging #3337 (03b2d8f) into main (3f30d84) will decrease coverage by 19.26%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3337       +/-   ##
===========================================
- Coverage   60.85%   41.58%   -19.27%
===========================================
  Files         266      198       -68
  Lines       26520    23964     -2556
===========================================
- Hits        16139     9966     -6173
- Misses       8591    12968     +4377
+ Partials     1790     1030      -760

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.58% <ø> (ø)

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


[WIP] Upgrade all the things, and change many things.

I expect most everything here to be cherrypicked or redone and generally not merged as is.
Some of this is already done in other PRs.
Overview

Require at least node 6.x
Use eslint over jshint
Fix code style issues from a (mostly) default config with eslint:recommended
Include examples/ in the lint
Upgrade node-gyp for (future) N-API support
Upgrade other dependencies for security issues (via npm audit)
Fix binding install path
Rename binding to bluetooth_hci_socket
a build fix for Node 10.x (related to errorConstructor)
Add Node 8.x, 10.x, and 12.x to the build matrices
Rename appveyor.yml to .appveyor.yml
use let/const throughout
replace deprecated Buffer constructor usage

More to come.

Hello, any plans for merging?

@mhammerc this one shouldn't be merged. I just put it all together until anyone is ready to merge even bits of it. After that I will split out the pieces that folks find acceptable.

It now builds on node 12.x.

Please can you  forward this change to:
https://github.com/abandonware/node-bluetooth-hci-socket/
FYI an updated version has been released:
<EMAIL_ADDRESS>


Reduce bounds check in RowIter, add unsafe Rows::row_unchecked

Which issue does this PR close?

Closes #.
Rationale for this change

While benchmarking https://github.com/apache/datafusion/pull/11684, I found the RowIter is taking unexpected long time which doesn't make sense. This pr reduces the # of bounds check.
What changes are included in this PR?

Are there any user-facing changes?

Have we benchmarked this change? In the past I've found removing bounds checks on variable length quantities to not always yield improvements, and sometimes actually regress performance.

Have we benchmarked this change?

Sorry for the late response. I added a benchmark, and it shows 16% performance without bounds check


Replace crazy-max/ghaction-github-labeler in on-manual-label-importer.yml

@eoaksnes status on this? Seems like something that should be high priority?


Can't edit result set in cell

MySQL - Some datatypes, such as decimal, cannot be edited. I can select them, but not modify.
Regards.

I also can not edit cell value:

here I can not set value for parent_id cell. At the same time I can edit other cells

cannot reproduce issue.
ticket closed


Fix dependency of flowRestoreCompleteWait on podNetworkWait

We want flowRestoreCompleteWait to depend on podNetworkWait. However, the implementation was invalid:

the goroutine in charge of invoking flowRestoreCompleteWait.Done() after podNetworkWait.Wait() returns was placed too early in the initialization logic. There was a high probability that podNetworkWait.Wait() would be called before the first call to podNetworkWait.Increment(), making the goroutine useless.
the statement that the dependency is guaranteed by CNIServer was incorrect for Nodes without non-hostNetwork Pods at the time of initialization. For such Nodes, flowRestoreCompleteWait.Done() was called immediately, without a dependency on podNetworkWait.Wait().

/test-all


Use --message=%s for git commit message

Close  #23027
git commit message option only supports 4 formats (well, only ....):

"commit", "-m", msg
"commit", "-m{msg}"  (no space)
"commit", "--message", msg
"commit", "--message={msg}"

The long format with = is the best choice, and it's documented in man git-commit:
-m <msg>, --message=<msg> ...
ps: I would suggest always use long format option for git command, as much as possible.

Codecov Report

Merging #23028 (15d378f) into main (e3cffa7) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main   #23028      +/-   ##
==========================================
- Coverage   47.46%   47.44%   -0.03%
==========================================
  Files        1140     1140
  Lines      150785   150784       -1
==========================================
- Hits        71565    71533      -32
- Misses      70754    70788      +34
+ Partials     8466     8463       -3

Impacted Files
Coverage Δ

modules/git/commit.go
60.24% <100.00%> (ø)

modules/repository/init.go
51.81% <100.00%> (-0.15%)
:arrow_down:

services/pull/merge.go
52.69% <100.00%> (ø)

modules/process/manager_exec.go
86.04% <0.00%> (-6.98%)
:arrow_down:

modules/queue/queue_bytefifo.go
47.29% <0.00%> (-6.86%)
:arrow_down:

models/asymkey/gpg_key_common.go
52.63% <0.00%> (-3.95%)
:arrow_down:

modules/queue/queue_disk.go
60.00% <0.00%> (-3.34%)
:arrow_down:

modules/queue/workerpool.go
52.85% <0.00%> (-1.43%)
:arrow_down:

modules/queue/queue_channel.go
87.35% <0.00%> (-1.15%)
:arrow_down:

modules/indexer/stats/indexer.go
57.44% <0.00%> (+6.38%)
:arrow_up:

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


Support translation into languages other than English

Support translation into languages other than English.

Do we want to add translation as a sub-module? Because, from the Issue you linked, it seems that Whisper can only translate to English.

I'd set this as stale for now as it's totally separate


segfault on linux

./forsp examples/tutorial.fp crashes with a segfault.
gdb gives me the following backtrace:
#0  0x00000000004017e5 in skip_white_and_comments () at forsp.c:214
#1  0x0000000000401b75 in read_list () at forsp.c:242
#2  0x0000000000401b45 in read_list () at forsp.c:249
#3  0x0000000000401b45 in read_list () at forsp.c:249
#4  0x0000000000401b45 in read_list () at forsp.c:249
#5  0x0000000000401b45 in read_list () at forsp.c:249
#6  0x0000000000401b45 in read_list () at forsp.c:249
#7  0x0000000000401b45 in read_list () at forsp.c:249
#8  0x0000000000401b45 in read_list () at forsp.c:249
#9  0x0000000000401b45 in read_list () at forsp.c:249
#10 0x0000000000401b45 in read_list () at forsp.c:249
#11 0x0000000000401b45 in read_list () at forsp.c:249
#12 0x0000000000401b45 in read_list () at forsp.c:249
#13 0x0000000000401b45 in read_list () at forsp.c:249
#14 0x0000000000401b45 in read_list () at forsp.c:249
#15 0x0000000000401b45 in read_list () at forsp.c:249
#16 0x0000000000401b45 in read_list () at forsp.c:249
#17 0x0000000000401b45 in read_list () at forsp.c:249
#18 0x0000000000401b45 in read_list () at forsp.c:249
#19 0x0000000000401b45 in read_list () at forsp.c:249
#20 0x0000000000401b45 in read_list () at forsp.c:249
#21 0x0000000000401b45 in read_list () at forsp.c:249
#22 0x0000000000401b45 in read_list () at forsp.c:249
#23 0x0000000000401b45 in read_list () at forsp.c:249
#24 0x0000000000401b45 in read_list () at forsp.c:249
#25 0x0000000000401b45 in read_list () at forsp.c:249
#26 0x0000000000401b45 in read_list () at forsp.c:249
#27 0x0000000000401b45 in read_list () at forsp.c:249
#28 0x0000000000401b45 in read_list () at forsp.c:249
#29 0x0000000000401b45 in read_list () at forsp.c:249
#30 0x0000000000401b45 in read_list () at forsp.c:249
...
#261683 0x0000000000401b45 in read_list () at forsp.c:249
#261684 0x0000000000401b1c in read () at forsp.c:329
#261685 0x0000000000401167 in main (argc=<optimized out>, argv=<optimized out>) at forsp.c:618

It looks like read_list calls read_list over and over again.

Thanks for reporting. I don't have a Linux machine at the moment, but I'll try to fix it when I get a chance.

Fixed. Clang (Mac) and GCC (Linux) use different function argument evaluation orders. It's unspecified in the standard. I generally code in a way to avoid these dark corners, but I was thinking too "functional" and it slipped my mind.
Fixed now. Enjoy!


feat: Implement previewServer (powered by Vite)

Features

[x] Implement Preview Server
[x] Add react-testing-library example to use with vitest-preview
[x] Run prettier

@all-contributors please add @nvh95 for code


Problems converting http://ww1.microchip.com/downloads/en/DeviceDoc/60001320C.pdf

tabula windows 10 edge and chrome and firefox say "Sorry, your file upload could not be processed. Please double-check that the file you uploaded is a valid PDF file and try again"
Open with IE hangs on "waiting to be processed"
http://ww1.microchip.com/downloads/en/DeviceDoc/60001320C.pdf

Hi @JimNickerson: Looks like this is a bug we're seeing a couple reports of lately; out of the 700+ pages in the PDF, there's likely one or two that Tabula is choking on. My recommendation is to use a tool like pdftk to extract just the pages with the tables you're interested in, then upload those to Tabula.
Stacktrace indicates this is a dupe of #513; closing so discussion of the JPedal issue is at that issue. Feel free to continue the discussion here regarding your workaround.
  at java.util.concurrent.FutureTask.report(java/util/concurrent/FutureTask.java:122)
  at java.util.concurrent.FutureTask.get(java/util/concurrent/FutureTask.java:192)
  at java.lang.reflect.Method.invoke(java/lang/reflect/Method.java:498)
  at RUBY.afterExecute(/home/my_username/code/tabula/lib/tabula_job_executor/executor.rb:42)
  at java.util.concurrent.ThreadPoolExecutor.runWorker(java/util/concurrent/ThreadPoolExecutor.java:1150)
  at java.util.concurrent.ThreadPoolExecutor$Worker.run(java/util/concurrent/ThreadPoolExecutor.java:617)
  at java.lang.Thread.run(java/lang/Thread.java:745)
Caused by: java.lang.ArrayIndexOutOfBoundsException
  at java.lang.System.arraycopy(Native Method)
  at java.lang.reflect.Method.invoke(java/lang/reflect/Method.java:498)
  at RUBY.initialize(/home/my_username/code/tabula/lib/thumbnail_generator.rb:56)
  at RUBY.perform(/home/my_username/code/tabula/lib/tabula_job_executor/jobs/generate_thumbnails.rb:15)
  at RUBY.call(/home/my_username/code/tabula/lib/tabula_job_executor/executor.rb:104)
  at GenerateThumbnailJob_1233439965.call(GenerateThumbnailJob_1233439965.gen:13)
  at java.util.concurrent.FutureTask.run(java/util/concurrent/FutureTask.java:266)
  at java.util.concurrent.ThreadPoolExecutor.runWorker(java/util/concurrent/ThreadPoolExecutor.java:1142)
  ... 2 more
java.util.concurrent.ExecutionException: java.lang.ArrayIndexOutOfBoundsException
  at java.util.concurrent.FutureTask.report(java/util/concurrent/FutureTask.java:122)
  at java.util.concurrent.FutureTask.get(java/util/concurrent/FutureTask.java:192)
  at java.lang.reflect.Method.invoke(java/lang/reflect/Method.java:498)
  at RUBY.afterExecute(/home/my_username/code/tabula/lib/tabula_job_executor/executor.rb:42)
  at java.util.concurrent.ThreadPoolExecutor.runWorker(java/util/concurrent/ThreadPoolExecutor.java:1150)
  at java.util.concurrent.ThreadPoolExecutor$Worker.run(java/util/concurrent/ThreadPoolExecutor.java:617)
  at java.lang.Thread.run(java/lang/Thread.java:745)
Caused by: java.lang.ArrayIndexOutOfBoundsException
  at java.lang.System.arraycopy(Native Method)
  at java.lang.reflect.Method.invoke(java/lang/reflect/Method.java:498)
  at RUBY.initialize(/home/my_username/code/tabula/lib/thumbnail_generator.rb:56)
  at RUBY.perform(/home/my_username/code/tabula/lib/tabula_job_executor/jobs/generate_thumbnails.rb:15)
  at RUBY.call(/home/my_username/code/tabula/lib/tabula_job_executor/executor.rb:104)
  at GenerateThumbnailJob_1233439965.call(GenerateThumbnailJob_1233439965.gen:13)
  at java.util.concurrent.FutureTask.run(java/util/concurrent/FutureTask.java:266)
  at java.util.concurrent.ThreadPoolExecutor.runWorker(java/util/concurrent/ThreadPoolExecutor.java:1142)
  ... 2 more

I am wondering if it is indeed a java exception might it be possible to
catch the exception and skip to the next table ?
Jim
On Mon, Jul 4, 2016 at 8:23 AM, Jeremy B. Merrill<EMAIL_ADDRESS>wrote:

Closed #520 https://github.com/tabulapdf/tabula/issues/520.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabulapdf/tabula/issues/520#event-712157951, or mute
the thread
https://github.com/notifications/unsubscribe/AKtuOaO5d_iBlE-fCEDzDHofuJ2m7Bliks5qSSWIgaJpZM4JDtCV
.

The problem is in generating thumbnails for the GUI; for some reason, JPedal (the PDF drawing library) is failing when trying to open PDFs. So there's really no opportunity to catch the error.
You could certainly try the command-line tool (which doesn't depend on thumbnails, since there's no GUI): https://github.com/tabulapdf/tabula-java/


Image digests pulled by ctr can't be seen through crictl

Description
ctr images pull command doesn't seem to put image digest in canonical format.
image.References of the image pulled by ctr is stored like sha256:XXXXXXXX.
https://github.com/containerd/containerd/blob/98f48d485d25cf7a8b973c3f7477381727ae9f77/pkg/cri/store/image/image.go#L41
However digests can be retrieved only when the reference is in the canonical format through CRI server.
https://github.com/containerd/containerd/blob/98f48d485d25cf7a8b973c3f7477381727ae9f77/pkg/cri/server/helpers.go#L322-L336
https://github.com/containerd/containerd/blob/98f48d485d25cf7a8b973c3f7477381727ae9f77/reference/docker/reference.go#L151-L156
because of these behaviours, a digest of the image pulled by ctr can't be seen from crictl.
Is this a bug in CRI pkg or in ctr images pull?
Steps to reproduce the issue

ctr --namespace k8s.io image pull <image>:<tag>
ctr --namespace k8s.io image list (observe image digests properly displayed)
crictl images --digests (observe  value for each image's digest)

Describe the results you received and expected
# crictl -r unix:///run/containerd/containerd.sock images --digests
IMAGE                       TAG                 DIGEST              IMAGE ID            SIZE
docker.io/library/busybox   1.34.0              <none>              8336f9f1d0946       777kB
docker.io/library/busybox   1.35.0              223ae047b1065       12b6f68a826b2       2.59MB
docker.io/library/busybox   1.36.0              <none>              7cfbbec8963d8       2.6MB

1.34.0 and 1.36.0 are pulled by ctr. (received)
1.35.0 are pulled by crictl.
with ctr images pull, digests are expected to be seen.
What version of containerd are you using?
containerd github.com/containerd/containerd v1.6.12 a05d175400b1145e5e6a735a6710579d181e7fb0
Any other relevant information
related issue: https://github.com/kubernetes-sigs/cri-tools/issues/1082
Show configuration if it is related to CRI plugin.
cat: /etc/containerd/config.toml: No such file or directory

Shared a history here https://github.com/containerd/cri/pull/865.

Merge RepoTags and RepoDigests into a References field in the image store. For containerd, there is only image reference, only CRI has the RepoTag and RepoDigest concepts.

The additional image references are created by CRI-API, instead of ctr command. I'm doubt that it's a bug.
It can be an enhancement.

I found this is because of the difference between pull logic.
When you use crictl, PullImage is called and create three references imageID, repoDigest, and repoTag.
https://github.com/containerd/containerd/blob/f84b2b47b0faca223837fbdb489961d15b023119/pkg/cri/server/image_pull.go#L197-L210
When you use ctr, updateImage is called through a eventtypes.ImageCreate event and create only two references configDesc.Digest.String() and r (image name).
https://github.com/containerd/containerd/blob/f84b2b47b0faca223837fbdb489961d15b023119/pkg/cri/server/image_pull.go#L298-L316
https://github.com/containerd/containerd/blob/f84b2b47b0faca223837fbdb489961d15b023119/pkg/cri/server/events.go#L352-L354
It can be fixed by unifying the process of creating references.
If this plan seems good, I'd like to contribute.


[!] Unable to find a specification for LookinServer

环境：ruby-2.7.0/lib/ruby/gems/2.7.0/gems/cocoapods-core-1.9.1/
报错:
[!] Unable to find a specification for LookinServer
You have either:

out-of-date source repos which you can update with pod repo update or with pod install --repo-update.
mistyped the name or version.
not added the source repo that hosts the Podspec to your Podfile.

pods lock的问题


Add automatic SQS processing using a lambda

The project now has source for a new lambda that, through a mapping, can be invoked every time a message is queued on an SQS queue.  The original profile lambda now writes a time-stamped message to a queue and a new "messages" table has been added as a repository for the queued messages.
When the original lambda is run prior to deploying and mapping the new lambda, the messages table remains empty.  After the new lambda has been deployed and mapped, each invocation of the profile lambda results in an automatic invocation of the SQS lambda when the profile lambda queues the time-stamped message.  These message can then be viewed in the new table.

Hi @eisnerw ,
Thanks again for your effort to make this example better. I ask you two things.
1-) We should create a .gitattributes file and add profile-lambda-csharp/json file to there. Because right now %88 of this project seen as written in javascript.
2-) We need to update the architecture schema since you added an SQS and additional Lambda. You can use Miro for drawing the new architecture and find new AWS icons in here. Alternatively, you can draw just a crude draft, and I can handle the rest.
Again thank you for your contribution.

Hi @eisnerw ,
Thanks again for your effort to make this example better. I ask you two things.
1-) We should create a .gitattributes file and add profile-lambda-csharp/json file to there. Because right now %88 of this project seen as written in javascript.
2-) We need to update the architecture schema since you added an SQS and additional Lambda. You can use Miro for drawing the new architecture and find new AWS icons in here. Alternatively, you can draw just a crude draft, and I can handle the rest.
(Looks like you can use draw.io too, AWS packed their new icons for draw.io. https://app.diagrams.net/?splash=0&libs=aws4)
Again thank you for your contribution.

I've created a new diagram with Miro and added it to the project.  You can't see it within the readme until the PR is accepted.  There may be a problem with its sizing (from my commits, you'll see that I was using GitHub to add and re-add different versions).  I'm afraid that I don't understand the .gitattributes suggestion.

Thanks, @eisnerw much appreciated. I'll handle the .gitattributes issue

See the new .gitattributes file
https://github.com/localstack-dotnet/localstack-serverless-dotnet-demo/blob/master/.gitattributes
Before .gitattributes file, this project was seen as a javascript project on the repository's language stats.

See the new .gitattributes file
https://github.com/localstack-dotnet/localstack-serverless-dotnet-demo/blob/master/.gitattributes
Before .gitattributes file, this project was seen as a javascript project on the repository's language stats. @eisnerw

@Blind-Striker, I can't say that I understand how github categorizes projects, but thanks for updating the .gitattributes file


Update clippy command in Makefile

What was wrong?
The clippy command in the make file was outdated.
How was it fixed?
Updated so that now one can run make lint locally as a simple shortcut to the same checks that run in CI.
To-Do

[ ] OPTIONAL: Update Spec if applicable

[ ] Add entry to the release notes (may forgo for trivial changes)

[ ] Clean up commit history

Codecov Report

Merging #164 (3418515) into master (00121f5) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #164      +/-   ##
==========================================
+ Coverage   81.39%   81.40%   +0.01%
==========================================
  Files          44       44
  Lines        3133     3135       +2
==========================================
+ Hits         2550     2552       +2
  Misses        583      583

Impacted Files
Coverage Δ

compiler/src/abi/elements.rs
74.64% <0.00%> (+0.73%)
: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 00121f5...3418515. Read the comment docs.


chore: update README for web site inclusion

This PR:

updates the README based on latest template for automatic inclusion in http://openfeature.dev/

I've tried to get everything right, but there's 5-6 code-example TODOs.
@beeme1mr feel free to make any corrections you see. Up to you if you think we can include this without the code samples.

We may need to add a fourth status to denote features that don't make sense in a language. I think that may be the case for php. What do you think @tcarrio?

We may need to add a fourth status to denote features that don't make sense in a language. I think that may be the case for php. What do you think @tcarrio?

Potentially dumb question, but what feature(s) wouldn't exist in PHP... shutdown?

Added the missing code examples, rebased on the latest main, and signed off all commits.


Using a phone

is there any way to use toypad on phone (with virtual machine)? Maybe even an application for the phone. Or use from another computer/laptop.

If you have a raspberry pi you can access the webpage on any device including your phone, but I don't think you can plug your phone in and use it as a virtual machine.
I am working on an iOS app that interacts with the pi to give it a more native feeling. I won't be able to upload it to the App Store but I plan to open source the project soon.

Will there be an android version?


[W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '131'. It is not used by any node and should be removed from the model.

Hi Dear i have a issue in runing run_server how can i  solve?this issue?
Downloading: "https://github.com/snakers4/silero-vad/zipball/master" to /root/.cache/torch/hub/master.zip
2023-09-16 19:29:25.999206343 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '131'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999219739 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '136'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999221888 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '139'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999223751 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '140'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999225479 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '134'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999253937 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '628'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999255898 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '623'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999258033 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '629'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999259496 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '620'. It is not used by any node and should be removed from the model.
2023-09-16 19:29:25.999261155 [W:onnxruntime:, graph.cc:3543 CleanUnusedInitializersAndNodeArgs] Removing initializer '625'. It is not used by any node and should be removed from the model.

@abdulhadyabas2 this is not an issue but harmless warnings. If you have an issue it is somewhere else but the Voice Activity detection model.

Closed by #51


typeahead-focus-first should select first if user has partially entered a value

In the getMatchesAsync function, the activeIdx is set to 0 if focusFirst is true and always to -1 if false. I would argue that if user has already entered a partial value and then hits tab, the user would want that value to be selected. Instead, hitting tab without first hitting the down arrow results in no value being selected.
For instance, given a company field with values ['ABC Corp', 'XYZ Corp'] and typeahead-focus-first='false', if a user clicks into the field, both values appear but neither is selected. If the user types in ABC, I think ABC Corp should become the selected value. It seems like this would be more intuitive then forcing the user to hit the down arrow--they could just hit Tab after typing ABC.
Current:
scope.activeIdx = focusFirst ? 0 : -1;
Proposed:
scope.activeIdx = focusFirst ? 0 : inputValue && inputValue.length ? 0 : -1;

@peder, in the future, please adhere to the issue template.  It's there for a reason.  I'm going to leave this open, however, as you've already submitted a PR.  Please note that that PR needs work.

I'll get this and the PR cleaned up.


RUM-2207 Call RUM error mapper even for crashes

What does this PR do?
This changes the RUM error event mapper to call the user supplied mapper even for crashes, which it previously excluded.
The error event mapper still does not support discarding fatal errors, and a warning is sent if the user returns null for a fatal error in the mapper.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1945 (fa8e4cf) into develop (736e81c) will increase coverage by 0.05%.
The diff coverage is 100.00%.

:exclamation: Current head fa8e4cf differs from pull request most recent head 031f2f3. Consider uploading reports for the commit 031f2f3 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1945      +/-   ##
===========================================
+ Coverage    83.19%   83.24%   +0.05%
===========================================
  Files          486      483       -3
  Lines        17560    17414     -146
  Branches      2603     2597       -6
===========================================
- Hits         14609    14496     -113
+ Misses        2249     2210      -39
- Partials       702      708       +6

Files
Coverage Δ

...ndroid/rum/internal/domain/event/RumEventMapper.kt
100.00% <100.00%> (ø)

... and 47 files with indirect coverage changes


Use hyper based client instead of reqwest

Should be merged after or with 0.3.0 release because of the Clone dependency in the examples.


Match packwiz's new mod manifest format

Aka change the name from mod.toml to mod.pw.toml.
The code is untested, but I think it will work

Sorry for noobie question, but how can I checkout it in my vscode env?

https://code.visualstudio.com/Docs/editor/versioncontrol#_git-support

Meh, I'm too lazy to read this. I'll just merge it and we'll see if it works :D


Unexpected Input highlight in stocks example

In the stocks example, clicking the search icon and then clicking on the Input field thickens and highlights the underline in purple to indicate that it is selected. However, after pressing enter and dismissing the keyboard, this highlight remains. Is this intentional, or is this a bug? I would expect it to go back to its original visual state, to indicate that it is no longer selected. Changing the autofocus property of the search bar to false appears to make no difference.

@mpcomplete may know.

This is even more obnoxious lately.  Changing the milestone.

I couldn't repro this, but I might be looking for the wrong thing.
Can we repro this in the gallery or with a smaller test case?
Optimistically closing, but if it's still happening, we can reopen.

I think maybe the issue you're dealing is that submitting an input field doesn't cause it to lose focus. To de-focus an input field, you can call Focus.clear(context). This will cause it to lose the purple underline.
Here's an example that does that: https://gist.github.com/collinjackson/df9b6f9708eaf95017deedd0ad1df7ff


Support for alarms

The processing of alarms as described in https://tools.ietf.org/html/rfc4791#section-8.6 is outside the scope of the caldav library, possibly with the exception of sending emails.  Anyway, the library should support methods for alarm discovery.

will this include setting new alarms?

Setting new alarms is possible through the library today.  It's part of the icalendar data sent to the server, generation of this data is outside the scope of the caldav library.

generation of this data is outside the scope of the caldav library.

I have reconsidered this a bit.
So, to fully support alarms, I think we may need to support ...
Adding alarms
It's always possible to add alarms by editing the raw icalendar, or editing it via the vobject_instance or icalendar_instance properties.  However, recently we've started adding multiple "high level" convenience methods and parameters for editing and generating icalendar code, so we may as well add explicit support for alarms.  So it's needed with:

The possibility to add simple alarms through parameters to the save_*-methods.
Possibility to add simple alarms to an existing Event/Todo-object

Finding alarms

The search-method should be fixed to allow searching for alarms by a time range search
The calendar object should have a method find_upcoming_alarm(after=now, cnt=1) for finding upcoming alarm(s).

Processing alarms
The RFC defines four kind of alarm actions, it's sound, display, email and procedure.  This library is definitively not going to play and sounds or display something for the end user.  This library could theoretically send emails or launch procedures.  However, since this library does not have any kind of event loop or other means of getting the timing correct, I feel that all kind of processing is out of the scope for this library and should be taken care of by the caller.

Adding of alarms (without having to hand-craft the icalendar data) is supported in #221
Searching for events/todos based on alarm time is supported in #221, but not tested
Searching for "upcoming alarms" not supported yet, procrastinating it

If anyone has a great needs for alarms, I will prioritize this and get it merged, testing it towards a calendar server that supports searching for alarms.  It seems like neither radicale nor xandikos supports searching for alarms yet, and those two are my primary test servers.

So I basically wrote some code (but no documentation) for adding alarms easily to new events long long time ago, but got busy with other things.  I will try to prioritize to get it merged in, problem is that I have no idea when I will get time to do any work towards the caldav library next time.  Perhaps tomorrow, perhaps next week, perhaps next month ...


Fix issue with blob publishing

The build is currently failing during publishing to the dotnet-core blob
feed. I think the reason why it's failing is that we're trying to
publish assets for multiple platforms/configurations.
We only care about publishing windows/release.

Merging this. We can't really test it until an official build runs.


Fix passing CORENRN_NMODL_FLAGS to nrnivmodl_makefile

Description
Fixess issue with CORENRN_NMODL_FLAGS not being written in the nrnivmodl_core_makefile
How to test this?
cmake .. -DCORENRN_ENABLE_NMODL=ON -DCORENRN_NMODL_FLAGS="sympy --analytic codegen --force"
make -j8
./bin/nrnivmodl-core mod

Test System

OS: macOS
Compiler: GCC
Version: master
Backend: CPU

Dries has fixed the nodes to new driver. So now it should work with new modules.

Dries has fixed the nodes to new driver. So now it should work with new modules.

Next error is : https://forums.developer.nvidia.com/t/support-for-atomic-in-libstdc-missing/135403

Next error is : https://forums.developer.nvidia.com/t/support-for-atomic-in-libstdc-missing/135403

@iomaganaris : When you build locally, was only tests/unit/alignment/alignment.cpp failing with these errors?

@iomaganaris : When you build locally, was only tests/unit/alignment/alignment.cpp failing with these errors?

@iomaganaris : When you build locally, was only tests/unit/alignment/alignment.cpp failing with these errors?

Yes, I had the same issue with nvhpc locally.

@iomaganaris : When you build locally, was only tests/unit/alignment/alignment.cpp failing with these errors?

Yes, I had the same issue with nvhpc locally.

please retest

please retest

retest this please

retest this please


Does kenx support batch delet?

Hello,
I checked the knex  website and  not found  batch delete function

SQL allows using .where with deletes:
knex('table').delete().whereIn('id', subquerySelectingIdsToDelete);


Bio: Sequence Renderer: different rendering modes

Similarly to Chem, let's provide "Render" panel ("Chem" in Chem) that would appear in context panel when user clicks on a column header. The following options should be editable there:
[ ] "Color code" checkbox
[ ] Monomer width: [short / long]. In short mode, only the first character should be visible, followed by .. if there are more characters
[ ] Reference sequence text box: when defined, all sequences should be rendered in the "diff" mode to the specified sequence
[ ] "Compare with current" check box: when on, all sequences get rendered in the "diff" mode, with the current sequence treated as the reference sequence. This should be on by default.

Can you describe the behavior in diff mode in more detail, are there any examples of this rendering?

I think we already have the "Macromolecule Difference" renderer, the same (or similar) mechanism should be used (similar monomers should be semi-transparent, different ones opaque).
Specifying the reference sequence should set corresponding column's 'referenceSequence` tag. Then, the renderer should check whether this tag exists, and if it does, then the sequence should be rendered in the "difference" mode.

We are not talking about WebLogo, this is all in-grid rendering

Also, note that I've added another feature to the list:
[ ] "Compare with current" check box: when on, all sequences get rendered in the "diff" mode, with the current sequence treated as the reference sequence. This should be on by default.


Fake icons grouping

Hello,
I would like to "group' some icons in my waybar similar to the tray. I've made a screenshot of my waybar to explain. But basically, I would like some icons on right side to be grouped like the icons  group which contains bluetooth.

I guess, I need some CSS tweaking but I have no idea how to do that.
Thanks

this can be achieved by grouping modules
https://github.com/Alexays/Waybar/wiki/Module:-Group

@lirtle
It works thanks. However, how can I decrease the space between the icons. I tried with padding in CSS but it doesn't to make it.

@janemba Yeah, you'll probably have to play around with the styling of the the modules inside the group as well as the styling of the group itself.

Sure thanks.


SegmentStatusChecker should honor no-queryable servers and instance assignment config

SegmentStatusChecker currently has problems in two scenarios:

If user has instance assignment config, a user may make offline segments and consuming segments may have different replica groups (e.g. use more replicas for consuming segments, but less replicas for online segments).
In this situation, when calculate PERCENT_OF_REPLICAS , it's problematic to use global maxISReplicas as denominator and global minEVReplicas as nominator. For example, if user config replica groups for immutable segments as 3, and 5 for mutable segments, PERCENT_OF_REPLICAS will always be 60% even if all replicas are up, which will cause some false negative alerts.

When a sever is configured as queriesDisabled or shutdownInProgress, broker does not send queries to the server. We need technically treat those replicas that on the no-queryable server as "OFFLINE" even if it shows "ONLINE" in helix

Thanks @mqliang . Regarding 2. we have previously run into issues where queriesDisabled is accidentally not cleared but went under the radar and caused availability issues.

We want to

either enhance the existing replicas_up metric
or for better backward compat add an replicas_up v2 to include queriesDisabled and shutdownInProgress


Updated pathtoCustomTestAdapters type to filePath

This field determines how the Build.vNext runtime treats the user entered value when passing it to a PowerShell script.  As a string, it is passed through unchanged.  As a filePath, Build.vNext corrects relative paths to be absolute paths in your solution directory.
Thus, when entering a relative path (as is often common, ex. when the adapters are added as nuget packages, i.e. XUnit), as a String, it operates relative to the location of the PowerShell script! This is nowhere near where your solution files are stored, and so it can't find your custom test adapters.
However, a field with type filePath gets the solution directory prepended to it first (if it's a relative path).  Thus, it is possible to specify relative paths and have them look in the solution directory, rather than deep in the TFS internals.

It should be noted that I made this change on our on premise installation of Team Foundation Server 2015 RC several weeks ago and it 1) solved the relative path issue we were having, and 2) caused no other issues that we can detect.

@Quantumplation can you please sign the contribution agreement here https://cla.microsoft.com/
thanks.

Hi!
I'd love to!  Just to be sure, I checked with my employer, and they've informed me that (at least for now, while we rethink the existing policy), I should indicate the second option:
applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer.```

However, I'm concerned about the lack of granularity here.  is there a separate CLA for each project, or can I change my CLA status after I've signed it, once we revise our policy?  Should I create a separate github account for pull requests that result from work done for my employer and reopen this pull request under that account, and this account for any contributions to Microsoft projects I make outside of work?

Regards,
Pi Lanningham

Hi Pi,
I am not 100% sure how that works.  Let me check with and I will find out.
From: Pi Lanningham<EMAIL_ADDRESS>Sent: Thursday, June 11, 2015 11:05 AM
To: Microsoft/vso-agent-tasks
Cc: Chris Patterson
Subject: Re: [vso-agent-tasks] Updated pathtoCustomTestAdapters type to filePath (#222)
Hi!
I'd love to! Just to be sure, I checked with my employer, and they've informed me that (at least for now, while we rethink the existing policy), I should indicate the second option:
I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or
applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer.
However, I'm concerned about the lack of granularity here. is there a separate CLA for each project, or can I change my CLA status after I've signed it, once we revise our policy? Should I create a separate github account for pull requests that result from work done for my employer and reopen this pull request under that account, and this account for any contributions to Microsoft projects I make outside of work?
Regards,
Pi Lanningham
—
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/vso-agent-tasks/pull/222#issuecomment-111166630.

CLA Can you help here?
From: Pi Lanningham<EMAIL_ADDRESS>Sent: Thursday, June 11, 2015 11:05 AM
To: Microsoft/vso-agent-tasks
Cc: Chris Patterson
Subject: Re: [vso-agent-tasks] Updated pathtoCustomTestAdapters type to filePath (#222)
Hi!
I'd love to! Just to be sure, I checked with my employer, and they've informed me that (at least for now, while we rethink the existing policy), I should indicate the second option:
I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or
applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer.
However, I'm concerned about the lack of granularity here. is there a separate CLA for each project, or can I change my CLA status after I've signed it, once we revise our policy? Should I create a separate github account for pull requests that result from work done for my employer and reopen this pull request under that account, and this account for any contributions to Microsoft projects I make outside of work?
Regards,
Pi Lanningham
—
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/vso-agent-tasks/pull/222#issuecomment-111166630.

There is one and only one CLA that needs to be signed by a contributor for contributing to any of Microsoft sponsored projects.
GitHub users have the option to bind multiple e-mail accounts to GitHub Accounts. When they commit they may indicate a specific e-mail address. So if a user is making a personal contribution, user can use a personal mail bound to a GitHub account. Similarly, if a user is making a contribution on behalf of the user’s employer, user can use a corporate mail bound to a GitHub account.
git commit --author= [see http://git-scm.com/docs/git-commit]

@Quantumplation, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

I signed it with my work email.  I'll amend the commit so that it has that email for the author instead.

Done, though I don't think you can see the difference on github:

Any update on getting this? Anything else I need to do?

No, thanks Pi. We will pull this set of changes in along with our next set of updates.

Pi,
Thanks for taking your time to make this fix.
Unfortunately, we cannot take this in because this could possibly break compat with existing definitions.
The string input allows you to specify paths in double quotes whereas the file path input would not.
This would break anyone who might have an existing definition in which they have adapter paths specified within double quotes.
Thanks again Pi, for bringing this to our notice. We are already in the process of making usability improvements around this, to address concerns such as yours.
Thanks,
Kavipriya.


Infrastructure to help users to submit benchmarks

Speeding up cvxcore is a big priority for us. In order for us to know what works "in practice," it's important for us to have access to a wide range of benchmarks. It's not practical for the cvxpy project maintainers (or a summer intern) to generate those benchmarks themselves. I think we should expand on the SolverTestHelper framework we use in test_conic_solvers.py. Here are three things we could do:

At the very least we can advertise that framework and solicit contributions.
We might make another repo within the cvxpy GitHub organization:  "cvxpy/benchmarks", so we can handle contributions of test cases which involve large amounts of metadata.
We could try to take this even farther and create some automated tools (e.g., a web app on cvxpy.org) where users can submit problem data and we automatically generate code and a PR so the test case meets formatting requirements for our benchmarking infrastructure.

Ping @akshayka, @SteveDiamond, @bstellato, @phschiele, @adishavit.

I 100% agree with 1. and 2. 3. could be great to given additional manpower.

3 is super cool!

@Michael-git96 and I have collected a handful of (still mostly synthetic) benchmarks that were posted in various recent issues. This was done as part of the current initiative of writing a cvxcore prototype in pure Python.
I think it would be great to formalise a framework for such benchmarks and make them easy to extend.

This sounds awesome.
Most of my models have very very complex construction code - not something I can submit.
I am wondering if, at the point where you (finally) create your cp.Problem(), there would be some way to write out (and anonymize) the objective and the constraints in a format that can be submitted as a benchmark.
For the community this would make submissions much easier to make (just one method call) and also safer (no sharing of code). For the CVXPY devs, it would mean no need to run custom example scripts per benchmark.
In addition, it would also be compatible with any submitted benchmark code, since that would eventually create the cp.Problem().


Consider peer timeout when selecting peers

Describe the feature
For this we need to share the timeout between ActiveSession and ActiveSessionHandle. This can be done using an Arc<AtomicU64> (see this commits). Then, when selecting peers, we should prioritize the ones with a lower timeout value (lower latency).
Additional context
ref https://github.com/paradigmxyz/reth/pull/789#pullrequestreview-1244425031

we'll propagate an Arc clone of this via the SessionEstablished event and delegate this to the state so we can use it when selecting peers.

Should we be reusing the Reputation here? And make our get next peers logic reputation-aware?

not really, because the main purpose for Reputation is to decide to which peer to connect to and when to disconnect, once connected there's no real use for it because the assumption here is that all connected peers are good until disconnected.
Instead for this issue we need to find the best connected peers (for requests) and RTT is a good metric here.


Automatic pick up items

Describe the feature you'd like to see added
Automatic pick up of ammo , money and other items when player is next or on top of them.

It has the Qingsheng dll that already does this auto pick

Money makes senese I guess,
but things which take up your finite inventory space will get annoying.


TweenManager.isTweening() should check tween.pendingDelete

This Issue is about

A bug in the API:

Phaser version(s): Phaser CE v2.9.1
What should happen: TweenManager.isTweening() should return false when testing a sprite that has no uncompleted tweens
What happens instead: TweenManager.isTweening() returns true when testing a sprite that has only completed tweens (i.e. with pendingDelete===true)

TweenManager.isTweening() only checks if any tween exists in _tweens, but it does not test whether these tweens are already completed (i.e., pendingDelete===true).
Thus e.g., when myTween.onComplete() is dispatched for a sprite, isTweening() is not useful to check whether any (other) uncompleted tweens still remain on the sprite.

Would the result be correct during the next frame/update? The tween should be removed from _tweens then, right?

I'll add a second argument.

https://codepen.io/samme/pen/xpxbgV

Thanks @samme - that will make my code much simpler in the case where tweens are removed, and this must be detectable within the same update processing.

You actually need another set of parens in this change  - the line should be:
            return (tween.target === object) && (!checkIsRunning || (tween.isRunning && !tween.pendingDelete));


General documentation improvements

Signed-off-by: Nathan McDougall<EMAIL_ADDRESS>

Branch was merged to main instead of develop, so this has been reverted via force-push and I will open another issue to merge into develop instead.


Fix adding performers to entity tree

When running ign gazebo levels.sdf, we see these errors, and the performers are never added to the entity tree:
[GUI] [Err] [EntityTree.cc:128] Failed to find parent entity [6]
[GUI] [Err] [EntityTree.cc:128] Failed to find parent entity [22]

Similar errors can be seen whenever a world with a performer is loaded.
This happens because the performer entities come up before their parent models.
This PR puts such entities in a queue so they're processed later once their parents show up.

Actions CI failure is Diff drive, which is known flaky I believe?

It should be fixed when https://github.com/ignitionrobotics/ign-gazebo/pull/357 is merged forward.


fix(helm): openshift-monitoring resource names

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #7214   👈

PR #7212

This comment was auto-generated by Graphite.


Move unknown URL resolution logic to assembleApp

RedirectDb.getRedirect() now returns the full URL data. This allows the endpoint handler to redirect unknown URLs to the homepage with the unknown URL as a query parameter, so that the user may have the option of defining the URL.

Coverage decreased (-0.4%) to 92.5% when pulling b3710e4a76003b85e33866c0ba0c549c1d683979 on get-redirect into 292900e28a61c35405a8eade429178dd251f39d0 on master.

Coverage dropped due to shorter code. Merging.


This should be the 'Contributor' role not 'Azure Contributor

The role is called 'Contributor' not 'Azure Contributor', so I've adjusted the highlighting

@MattBrowne1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Crosswalk And Document eoAPI Tests

Context
Runtime tests got moved over to this repo and expanded upon it seems. Originally eoapi-k8s copy pasta'd them to make sure things worked across cloud providers. Now given #86 there's a potential third repo that tests might need to get
Goal
This is insanity. So document which tests we have for eoapi-devseed and eoapi-k8s. Make decisions about which ones eoepca+ needs. Then come up with a plan to "reuse" just the tests as much as possible between the projects where they need to be reused
AC

[ ] google doc matrix of tests between eoapi-devseed and eoapi-k8s
[ ] ask around about ideas for sharing
[ ] decide how to share and expand next ACs

the tests were moved to eoapi-devseed because they were testing the custom runtimes, not the classic eoAPI building blocks.
we could design tests for the raster/stac/vector API, but those should and are already well tested individually on their own repo.

we could design tests for the raster/stac/vector API, but those should and are already well tested individually on their own repo.

yeah, we don't need all the tests just s subset of them working for eoapi-k8s and then reused on eoepca+ is how I see this now. Copy pasted for now between eoapi-k8s and eoepcia+. Will break out into a package later.
But I believe these are needed b/c they've been pretty useful to have for testing out SSL, traefik and nginx rewrite capabilities to know when the proxy settings are misaligned 👍


Cleanup & Release Preperation

Signed-off-by: fjogeleit<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 19 (84.21%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.2%) to 92.681%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/config/resolver.go
11
12
91.67%

pkg/target/yandex/yandex.go
5
7
71.43%

Totals

Change from base Build<PHONE_NUMBER>:
2.2%

Covered Lines:
1583

Relevant Lines:
1708

💛  - Coveralls


CPU Usage & Network Traffic Basic not reflecting over Grafana

Node Exporter Running Over An Instance-1, prometheus & Grafana Running on Instance 2.
But Grafana Dashboard not reflecting CPU Usage & Network Traffic Basic info.

Same here, node_exporter running dockerized under linux.
node_cpu_* and friends are present and are visible in prometheus native UI.

This is a DUP to #137
The resolution is adjusting the scrape interval in the data source, corresponding to the Prometheus configuration.

Duplicate of #137


[12.x] Consider after_commit config in SyncQueue

AfterCommit support was added to the SyncQueue in https://github.com/laravel/framework/pull/48860. However, it only works if the job in question implements the ShouldQueueAfterCommit interface or sets the afterCommit property - it does not work if after_commit is set to true in the sync driver's configuration. This makes the behavior inconsistent, which is what this PR intends to solve.
I am adding it to the master branch as I consider it a breaking change.
I spent some time unsuccessfully trying to figure out how to write a test for this. If anyone has an idea, please let me know and I'll add a test.

It's fine to send this to 11.x branch please. Thanks.


Migrate <Gizmo>'s internals to use <HUD>

This should fix click issues that users are experiencing, because with HUD we can simply just raycast from the viewport instead of creating weird invisible DOM elements.

closing for now. We have a new gizmo since #1254 was merged 🎉
happy for the issue to be reopen by OP if needed


Statusline Theme

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

🏷 Feature Type
New Addon
🔖 Feature description
Would it be possible to add the "powerline_dark" theme to the available statusline themes?
✔ Solution
I could use powerline_dark rather then powerline
❓ Alternatives
No response
📝 Additional Context
No response

Could you test #393 for me? The PR adds powerline_dark as an available theme.

Seems pretty good to me Raf!!


Kapt picocliCompiler in okcurl module

According to the picocli-codegen document, the picocliCompiler should be used with kapt.

Test run finished after 408497 ms
[       261 containers found      ]
[         2 containers skipped    ]
[       259 containers started    ]
[         0 containers aborted    ]
[       258 containers successful ]
[         1 containers failed     ]
[      2608 tests found           ]
[       111 tests skipped         ]
[      2497 tests started         ]
[        27 tests aborted         ]
[      2467 tests successful      ]
[         3 tests failed          ]

Error: Process completed with exit code 255.


Use watcher with retries to wait for resources

What this PR does / why we need it:
When waiting for resources use the ListWatchUntil instead of
UntilWithoutRetry so that if the connection drops between tiller and
the API while waiting the operation can still succeed.
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

That was the conclusion I had also come to but I haven't had an opportunity to look into how we can pass a ListWatcher through.
On 29 July 2019 23:09:40 BST, Taylor Thomas<EMAIL_ADDRESS>wrote:

thomastaylor312 commented on this pull request.

@@ -809,7 +809,7 @@ func (c *Client) watchUntilReady(timeout
time.Duration, info *resource.Info) err

ctx, cancel :=
watchtools.ContextWithOptionalTimeout(context.Background(), timeout)
defer cancel()

_, err = watchtools.UntilWithoutRetry(ctx, w, func(e watch.Event)
(bool, error) {

_, err = watchtools.ListWatchUntil(ctx, w, func(e watch.Event) (bool,
error) {

Based on the test failure output, it looks like the watcher we create
with the resource helper isn't the right type to use with this
function. You'll probably need to do something to convert the watch
interface to the cache.ListerWatcher it expects
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/helm/helm/pull/6014#pullrequestreview-268067068


Provision doesn't work on Windows due to symlink issues

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.2.5
VMware Workstation 15 Pro: 15.1.0 build-13591040
Ansible local:
ansible 2.8.5
  config file = None
  configured module search path = [u'/home/vagrant/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /home/vagrant/.local/lib/python2.7/site-packages/ansible
  executable location = /home/vagrant/.local/bin/ansible
  python version = 2.7.15+ (default, Jul  9 2019, 16:51:35) [GCC 7.4.0]

Your OS

Windows (Version	10.0.18362 Build 18362)

Full console output
https://gist.github.com/codebymikey/148ad637125fd2afa94dd2fe275c8875
Summary
The error occurs because the initial composer install creates vendor binaries with symlinks, visible by calling ls -al /tmp/composer-project/vendor/bin/:

So it fails when it tries to copy it into the host.
I believe the composer install needs to set bin-compat: full so that it works on all platforms.
Composer references:
https://github.com/composer/composer/blob/5633a68/src/Composer/Config.php#L282
https://github.com/composer/composer/blob/5633a68/src/Composer/Installer/BinaryInstaller.php#L90

I solved this by enabling symlinks in the Win10 local security policy. I was lazy and added 'Everyone' to the policy for creating symlinks. It seemed reasonable enough but I'm not 100% sure what the security implications of that are in windows. ;-)
This article has more details.
https://community.perforce.com/s/article/3472

Can confirm I am also seeing the symlinks.
https://imgur.com/a/euQhfpa


Cannot compile on new phoenix 1.7.0

== Compilation error in file lib/live_admin/view.ex ==
** (CompileError) lib/live_admin/view.ex:4: module Phoenix.View is not loaded and could not be found
(elixir 1.14.3) expanding macro: Kernel.use/2
lib/live_admin/view.ex:4: LiveAdmin.View (module)

I think adding phoenix_view as a dependency should solve this issue. See https://github.com/tfwright/live_admin/issues/13#issuecomment-1423308893

Hey @dev-rroslan - Phoenix 1.7 is so structurally different in the Web module from <= 1.6.x that several meaningful changes will need to be made in order to achieve full 1.7 compatibility. Specifically, Views have been completely replaced by Components, and routing helpers have been completely reconfigured to use the new sigil verified route syntax. It looks, from the commits, that @tfwright is already working on 1.7 compatibility, but has not issued a new tagged release with those patches.
I'd keep an eye out for the next couple of tagged releases, one of which I assume should add support for Phoenix 1.7. @tfwright please correct me if I'm wrong here.

Hi, thanks for a quick reply. I have a use case and searching for a module
to disable registration but admin or registered user can invite. Thanks
again.
On Thu, 2 Mar 2023 at 10:10, Mike Zazaian @.***> wrote:

Hey @dev-rroslan https://github.com/dev-rroslan - Phoenix 1.7 is so
structurally different in the Web module from <= 1.6.x that several
meaningful changes will need to be made in order to achieve full 1.7
compatibility. Specifically, Views have been completely replaced by
Components, and routing helpers have been completely reconfigured to use
the new sigil verified route syntax. It looks, from the commits, that
@tfwright https://github.com/tfwright is already working on 1.7
compatibility, but has not issued a new tagged release with those patches.
I'd keep an eye out for the next couple of tagged releases, one of which I
assume should add support for Phoenix 1.7. @tfwright
https://github.com/tfwright please correct me if I'm wrong here.
—
Reply to this email directly, view it on GitHub
https://github.com/tfwright/live_admin/issues/28#issuecomment-1451169257,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT7SWTXPH5AZSRKC5Y32QETWZ76TFANCNFSM6AAAAAAVMYHXKI
.
You are receiving this because you were mentioned.Message ID:
@.***>

here's my route but not rendering the default template. I am new to Phoenix
GET     /admin                                 LiveAdmin.Components.Home :home
GET     /admin/:resource_id                    LiveAdmin.Components.Container :list
GET     /admin/:resource_id/new                LiveAdmin.Components.Container :new
GET     /admin/:resource_id/edit/:record_id    LiveAdmin.Components.Container :edit
here's my scope in the router:
scope "/admin", UndiWeb do
live_admin "/", resources: [Undi.User]
end

Hi @dev-rroslan - are you currently upgrading from Phoenix 1.6 to 1.7? Or are you working off of a freshly-generated Phoenix 1.7 application and trying to install live_admin on top of that?

Closing this because there's nothing to do here. @dev-rroslan for general help/advice, please use the elixir forum thread: https://elixirforum.com/t/liveadmin-phoenix-admin-ui-built-on-liveview/46421

@tfwright are you planning to publish a Phoenix 1.7 compatible tagged release? At present it does not appear that installing phoenix_view as a dependency by itself makes live_admin compatible with 1.7.

AFAIK it should be possible to use 1.7 but including phoenix_view and ensuring that route helpers are enabled. But if you have found other issues please open a new bug report, and if you can push a sample app that demonstrates the problem that would be greatly helpful. Thanks!

Understood. I'm still running into some issues on 1.7 even with phoenix_view installed, and cannot load the live-admin index page without error. I'll post the issue I'm experiencing when I have a moment to do so.


White text on white background in Advanved Tile Demo

Minor colouring issue within TileViewDemoAdvanced in Version 4:
Problem
InfoWindow that's displayed when the markers are clicked on has invisible (white on white) text.
Reason

Version 4 hasn't set AppTheme set in Android Manifest (in difference to previous version)
Version 4 has set a white background to the infoWindow (Version 3 has got a grey one)

Solution

Set Background Color back to something dark (TileViewDemoAdvanced.java:getInfoView)
Set Text Color to Something dark (TileViewDemoAdvanced.java:getInfoView)

At first i thought setting AppTheme in the Manifest would solve the issue too, but AppTheme inherits from an Theme that's not compatible with AppCompat. And if that's changed (for example to @style/Theme.AppCompat.NoActionBar  that somehow doesn't solve the issue either (maybe AppCompat Themes don't override colors/set text color to white). I've tested the other two solutions, they do the trick.

Thanks, will fix next time I'm on the project.


[WIP] wctl cleanup

Changes:

Changes logging from key-value to concrete struct with proper unmarshaling

Other minor changes:

Websocket ignores all events below info
Logging methods are now very different (refer to ledger.go changes)
API package is refactored
Events logged into the CLI are now scattered in wavelet and api packages. No more manually updating fields in wctl, cli and elsewhere.

! This PR is untested !

@diamondburned was this branch initially created from himitsu? because it seems like it contains a lot of commits from it.

@diamondburned was this branch initially created from himitsu? because it seems like it contains a lot of commits from it.

Yeah, it didn't make it in time for Himitsu. I'll rebase it to master soon.


Refactor components

Describe your changes
Issue ticket number and link
Closes #xxxx (Replace xxxx with the GitHub issue number)
Checklist before requesting a review

[ ] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[ ] Minor changes
[ ] Bug fixes

[ ] Proposed changes are described in the CHANGELOG.md

[ ] CI Tests succeed and look good!

after this PR is merged, the containers will need to be set to public again


Only set content_model first time. Fix #1676.

Of course, content_model will keep getting set to None for the base classes, which is confusing but not necessarily a problem.

Thanks Ryne


No License given

There is no License.md file, and the Readme also doesn't say anything about the license

Thank you for your suggest! The License file has been committed.


Message/Notification: manage z-index with PopupManager

Please makes sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's Contributing Guide.
[ ] Rebase before creating a PR to keep commit history clear.
[ ] Add some descriptions and refer relative issues for you PR.

#821


Only create a new logger when the relative path is not the current working directory

This prevents creating loggers with only "." as a name
The actual path traversal fix is in multilogger

When you run plan-all and there's an unignored project in the current directory


Increase default adaptiveStream pixelDensity on high-density screens

Mobile devices typically use high density screens (2.5-3+). On these platforms, using default CSS dimensions tends to produce less than ideal streaming quality when using adaptiveStream defaults.

Can we switch to screen pixel density by default? Or does that not work for mobile browser cases?

Can we switch to screen pixel density by default? Or does that not work for mobile browser cases?

I'm trying to avoid a change in behavior on desktop systems. If we defaulted to screen, it will increase the bandwidth consumption on most displays today. On mobile, it'll mean fetching 720p by default for a video tile that takes about 200 css pixels.
Maybe this is a change we make for 2.0? as more efficient codecs are used.


Upgrade to latest Twilio edge library

LGTM locally for the 4 products, joining video rooms, and notifying.


E-mail wrapper and product URL for easier migration to production

This would also close #22 I guess.  :)


Is my local setup correct ?

Hi ! 👋🏻
Basically I want to get as much performance as possible, so I've read few articles saying that the best way is still to have Google Fonts locally. So, as far as I understand, the point is to get it at build time and never fetch it ever again (less HTTP requests, etc...).
So, I tried a configuration but I wished to have a confirmation that my setup looks okay
buildModules: [
  [
    '@nuxtjs/google-fonts',
    {
      families: {
        Mali: {
          wght: [400, 600, 700],
        },
      },
      subsets: ['latin'],
      display: 'swap',
      prefetch: false,
      preconnect: false,
      preload: false,
      download: true,
      base64: false,
    },
  ],
],

Got the fonts downloaded on build time with nuxt generate --modern.

Then, I added this to my main layout file to have it pretty much everywhere.
html {
  font-family: 'Mali', 'Source Sans Pro', -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto,
    'Helvetica Neue', Arial, sans-serif;
}

I checked on Firefox and it seems that I'm downloading it properly from the /dist folder !

Did I miss something or are the options looking good for you ? 😸

Hi @kissu
The settings are correct, that's right, can I help you with something else?

Alright, thank you. Nothing more to ask ! 👍🏻


Design and development of the situation beem weapons and development of ship Eco sestoum fist and control over

This template is here to help you to write your issue ;)
What is the problem?
Try to give as many details about the bug/new feature/enhancement as possible,
but keep it friendly for new comers
How can this be addressed?
New comers might need help on how to approach or build a solution, maybe a
step-by-step on how to address the issue can help
Who could help with this issue?
We know sometimes a good session of pair programming can do wonders, maybe
indicate someone that can be reached to help out and discuss ideas. If you
don't know someone but is up for a pair programming session use this space to
tell us o/
Labels
Feel free to write up labels that you think might apply here. e.g.
[first-timers-only]  [bug]

Reported as #spam.


Added percent bar

Associated Issue: #220
Summary of Changes

Added percent to accompany the progress bar

I'm not sure why I'm failing the tests it looks fine on the preview.


format  command would remove the width and type setting in string format

just past the following code into the fsharp file and run the alt+shift+f command:
let logName = $"""CGGC_{DateTime.Now:yyyy_MM_dd_HH_mm_ss_fff}"""
let msg = $"""{first,10:F}ms-{last,10:F}ms:: GCCGPause_in_{interval}ms -> count:{count, 8:D} sum:{sum, 8:F}ms per:{per,8:P1} av:{av, 8:F}ms min:{min, 8:F}ms max:{max, 8:F}ms {(if attention then "******" else "")}"""

Thanks for submitting! This is because the version of fantomas we have bundled didn't have support for format specifiers like this. The latest preview version does, so we should look into updating.  I'll leave this open, but open another issue in fsautocomplete and link it, because FSAC is what will actually need to change.

thanks for the quick responds

but after I installed the fantmas-fmt extension and associate the alt+shift+f to it, I find it's the same result

that could be the same root cause. I don't know who maintains that extension or what version of fantomas it's using, so it could also very well have the formatting issue, as this was only fixed relatively recently in fantomas.

This has been pushed out in fsautocomplete 0.46.3.  this version can be used from current ionide if you:

dotnet tool install -g fsautocomplete --version 0.46.3
set the "FSharp.fsac.netCoreDllPath" setting to "fsautocomplete", which causes Ionide to invoke dotnet fsautocomplete instead of it's built-in fsautocomplete.

We have improved Fantomas integration since this was reported.


Support for mpq not being the last part of a file

This might not be a common thing, but having multiple file types appended onto a file is perfectly fine with Warcraft3 and the World editor(although saving the map loses all of that). A test case would be getting any .exe, appending an .w3m to it and then appending some other file type like .zip(I used the cat command for that). When the map is opened it works just fine, but trying to get its signature info from StormLib fails.

Could you please send me an example of such map?

testfile.exe.w3x.zip
The MPQEditor is also able to open files like this.


Drive-by flake8 fixes

Just a couple of nitpicks to let flake8 run cleanly on the haas codebase. I didn't address E402 module level import not at top of file since that would have materially altered the code (and can be whitelisted).
@sjagoe Would you be interested in running flake8 or another linter as part of CI?

@jvkersch Yes, I think we do want it in CI.

Thanks @sjagoe. If you don't mind, I'll keep this open until #175 is merged and then I'll add the CI.

I've set up flake8 CI in #179, building on the work done in #175.

Given that this is only two commits, I've cherry-picked them into #179 to allow CI to run without allowed-failures.

Thanks @sjagoe !


[CT-2788] [Bug] When warnings about contract disenforcement are created, ambiguity is introduced by the passive voice

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Sibling to #8028
Another warning that dbt emits is:
Breaking Change to Contract Error in model model_x (model_x.sql)
  While comparing to previous project state, dbt detected a breaking change to an enforced contract.

  The contract's enforcement has been disabled.

When I first read this, I interpreted it as "dbt detected a breaking change to an enforced contract, so dbt has stopped enforcing the contract" (which is the exact opposite of what I would want; that's exactly when dbt needs to step up!). In fact, it is "dbt detected that the user disabled contract enforcement, which means the contract is now broken."
I think this would be less confusing if I happened to see it as part of a list of issues, but my first encounter was this single bullet item.
Expected Behavior
Maybe adding a - at the start to show it's a list would be enough?
Maybe rephrasing it: Enforcement of the contract was disabled in path/to/the/file.yml?
I don't know exactly
Steps To Reproduce

Create a model with a contract which is enforced
Run dbt compile
Disable the contract
Run dbt run -s state:modified --state path/to/artifacts

Relevant log output
No response
Environment
- OS:
- Python:
- dbt:

Which database adapter are you using with dbt?
No response
Additional Context
No response

@joellabes This is an issue after my heart! And my high-school English teacher who penalized me every time I used the passive voice. "'Mistakes were made.' Leave the passive voice to the politicians, and the Times editorial board."

I think this would be less confusing if I happened to see it as part of a list of issues, but my first encounter was this single bullet item.

I think this is exactly what's at play: If you disable a model's contract and make some breaking changes to its content, the idea is to show all of those things in one error message. That feels important because we expect many users to run into this error at CI time, where they don't have access to the same quick iteration loop as dev (although we'll still only be able to report the Breaking Changes error for one contracted model at a time).
$ dbt -q ls -s state:modified --state state/
10:00:32  Encountered an error:
Breaking Change to Contract Error in model my_model (models/my_model.sql)
  While comparing to previous project state, dbt detected a breaking change to an enforced contract.

  The contract's enforcement has been disabled.

  Columns were removed:
   - this_column_will_be_removed

  Columns with data_type changes:
   - id (integer -> text)

  Consider making an additive (non-breaking) change instead, if possible.
  Otherwise, create a new model version: https://docs.getdbt.com/docs/collaborate/govern/model-versions

So, potential changes:

Reword this sentence to be much more explicit: "Previously, this model's configuration included contract: {enforced: true}. It is no longer configured to enforce its contract, and this is a breaking change."
Make the formatting of the error message more like a list:

While comparing to previous project state, dbt detected one or more breaking changes to a model with an enforced contract.
  - Contract enforcement was removed: Previously, this model's configuration included contract: {enforced: true}. It is no longer configured to enforce its contract, and this is a breaking change.
  - Columns were removed:
     - this_column_will_be_removed
  - Columns with data_type changes:
     - id (integer -> text)

Closing in favor of implementation ticket:

https://github.com/dbt-labs/dbt-core/issues/8282


feat(jbang): make export reproducible

Added a build propertues required as specified in https://maven.apache.org/guides/mini/guide-reproducible-builds.html
Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

I wonder if adding a boolean option so users can turn this off of not desired

I wonder if adding a boolean option so users can turn this off of not desired

I can do that but it would be complicating the export subcommand with yet another option. IMO this should be a best practice we enforce since prototype as it neither have any side effect on the final build.

I wonder if adding a boolean option so users can turn this off of not desired

I can do that but it would be complicating the export subcommand with yet another option. IMO this should be a best practice we enforce since prototype as it neither have any side effect on the final build.

Yeah its something that can always be added later, so its good as-is


Man,e,where  is the implementation.....

Working on it. It will release within one or two weeks.

Hahaha there's a hurry-up issue!
2 weeks past I'm looking forward for your implementation as well!

2 weeks have past and I'm looking forward for your implementation as well!

2 weeks have past and I'm looking forward to your implementation as well!

Hi, I have added an implementation in the PR.

2 weeks have past and I'm looking forward to your implementation as well!

2 weeks have past and I'm looking forward to your implementation as well!

Guys, I;ve just uploaded implementations.
Sorry for the delay. I'm still conducting experiments to reproduce the paper's result. For Wide-ResNet-28-10 on CIFAR-100 dataset, I got the similar result.
I need to try to reproduce the paper's result futhermore for other architectures and ImageNet.
Thanks.

I've reproduced results for Wide-ResNet-28x10 for both of cifar-10 and cifar-100.
I believe that there aren't any significant problem in implementation codes.
I started experiments for other architectures on CIFAR-10, CIFAR-100 and ImageNet.
I will update readme as soon as I got reproduced valid result.
Thanks! I'm closing this.

Is there any chance to provide an alternative implementation based on OpenCV?


Fixes the number of traversal events

Can you please make a more descriptive Subject on this PR? The current is very non descriptive.

We know that: poll return a positive number when success; this is the number of structures which have non-zero revents fields (in other words, those descriptors
with events or errors reported). We should go through the number of structures which have non-zero revents fields  instead of the value "ip".

Perfect, thanks!

[approve ci]


Error in download file of load profile

Extreme numbers can be found in the downloaded file of load profiles. Noticed the error in many load profiles, for example (see also attached): ‘Solar PV – Arnhem’ and ‘DHW Demand #1’ (hot water)
Solar_PV_-_Arnhem.default.xlsx
DHW_Demand__1.default.xlsx

Transferred the files from .csv to .xlsx in order to include a graph

Can you clarify in what way the values are extreme or different than you expected?

Hi Anthony,
In the case of the PV, the maximum value is: 9542982,046 although values should be between 0,0 and 1,0
For the hot water, the maximum value is: 1000
This extremes can easily be seen in the excel graphs that I attached.
Kind regards,
Stefan Kop
Van: Anthony Williams<EMAIL_ADDRESS>Verzonden: dinsdag 19 juli 2016 16:56
Aan: quintel/etmoses
CC: Kop, Stefan; Author
Onderwerp: Re: [quintel/etmoses] Error in download file of load profile (#1302)
Can you clarify in what way the values are extreme or different than you expected?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/quintel/etmoses/issues/1302#issuecomment-233658973, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATlolr7oocbeKJpSKY0H9B-vmIzeUB5sks5qXOVjgaJpZM4JPxKq.

@Stefan-87 can you please include a link to your LES?

Sure, but it is regardless of the LES. The error shows up when you download one of the standard profiles available for every LES (see attached screenshots).
The link to the LES I was testing with is: https://moses.energytransitionmodel.com/testing_grounds/521
Best regards,
Stefan
Van: Chael Kruip<EMAIL_ADDRESS>Verzonden: woensdag 20 juli 2016 10:18
Aan: quintel/etmoses
CC: Kop, Stefan; Mention
Onderwerp: Re: [quintel/etmoses] Error in download file of load profile (#1302)
@Stefan-87https://github.com/Stefan-87 can you please include a link to your LES?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/quintel/etmoses/issues/1302#issuecomment-233879797, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATlollxSdjsNE45EcQA03GoGOYO70lIlks5qXdnPgaJpZM4JPxKq.

Sure, but it is regardless of the LES. The error shows up when you download one of the standard profiles

I see. You are downloading the profiles from the profile menu right?

(see attached screenshots).

I can't see your screenshots somehow
I think I know what is happening though. My guess is that you are using Microsoft Excel with comma's as decimal separators (instead of points). When I download the Ameland PV curve I get the following plot:

Here is the Excel:
Solar_PV_-_Arnhem.default.xlsx

Van: Chael Kruip<EMAIL_ADDRESS>Verzonden: woensdag 20 juli 2016 10:18
Aan: quintel/etmoses
CC: Kop, Stefan; Mention
Onderwerp: Re: [quintel/etmoses] Error in download file of load profile (#1302)

@Stefan-87https://github.com/Stefan-87 can you please include a link to your LES?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#1302 (comment)>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATlollxSdjsNE45EcQA03GoGOYO70lIlks5qXdnPgaJpZM4JPxKq.

Please don't reply directly from email but use the Github interface or we get a lot of unwanted clutter in the thread. Thanks!

Yes! That seems to fix it. I changed the settings in Excel and now I get a proper plot.
Thank you ☺
Kind regards,
Stefan
Van: Chael Kruip<EMAIL_ADDRESS>Verzonden: woensdag 20 juli 2016 10:43
Aan: quintel/etmoses
CC: Kop, Stefan; Mention
Onderwerp: Re: [quintel/etmoses] Error in download file of load profile (#1302)
Sure, but it is regardless of the LES. The error shows up when you download one of the standard profiles
I see. You are downloading the profiles from the profile menu right?
(see attached screenshots).
I can't see your screenshots somehow
I think I know what is happening though. My guess is that you are using Microsoft Excel with comma's as decimal separators (instead of points). When I download the Ameland PV curve I get the following plot:
[solar_pv_-_arnhem_default_xlsx]https://cloud.githubusercontent.com/assets/1303760/16980426/269676a8-4e66-11e6-9c04-ff2577616f75.png
Here is the Excel:
Solar_PV_-_Arnhem.default.xlsxhttps://github.com/quintel/etmoses/files/373186/Solar_PV_-_Arnhem.default.xlsx
Van: Chael Kruip [mailto:notifications<EMAIL_ADDRESS>Verzonden: woensdag 20 juli 2016 10:18
Aan: quintel/etmoses
CC: Kop, Stefan; Mention
Onderwerp: Re: [quintel/etmoses] Error in download file of load profile (#1302https://github.com/quintel/etmoses/issues/1302)
@Stefan-87https://github.com/Stefan-87https://github.com/Stefan-87 can you please include a link to your LES?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#1302https://github.com/quintel/etmoses/issues/1302 (comment)>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATlollxSdjsNE45EcQA03GoGOYO70lIlks5qXdnPgaJpZM4JPxKq.
Please don't reply directly from email but use the Github interface or we get a lot of unwanted clutter in the thread. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/quintel/etmoses/issues/1302#issuecomment-233888159, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATlolnnNfSK4Njxn3CVc6hc6JwpM36S4ks5qXd97gaJpZM4JPxKq.

@Stefan-87 Please don't reply directly from email but use the Github interface or we get a lot of unwanted clutter in the thread. Thanks!
Closing this.


Use elastic beanstalk to group environments

Todo
http://docs.aws.amazon.com/elasticbeanstalk/latest/dg/ebcli-compose.html

Document usage (deployment)
Document environments

See https://github.com/dwilkie/twilreapi/issues/14


New version: tailscale.tailscale version 1.22.1

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

/azp run


Add slow flag to test_vision for CI

This PR fixes test so that CI will succeed.
test_vision.py includes downloading large file.
Travis CI cannot run long time job.
https://travis-ci.org/pfnet/chainer/jobs/181078547#L768

LGTM

Thank you!


Tutorial

Hi
is there a steps by step tutorial that shows how to deploy custom policy definitions or initiatives and assignments using Azure DevOps based on the repo structure suggested in this page?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 50f6b34e-7ed0-5ecd-6cf4-192e7393663c
Version Independent ID: cb186c12-22a3-5ac4-667c-b1d07ab5f566
Content: Design Policy as Code workflows - Azure Policy
Content Source: articles/governance/policy/concepts/policy-as-code.md
Service: azure-policy
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

#assign:DCtheGeek

Thanks @geoano for the question.  There isn't a single tutorial that combines the implementation details for Azure Policy on Azure Pipelines. However, Azure Pipelines has some information for working with Azure Policy and you can create PowerShell tasks in Azure Pipelines to use the Azure Policy SDK.  These links should help:

Check Azure Policy compliance with gates in Azure Pipelines
PowerShell tasks on Azure Pipelines paired with Create and assign a policy definition with PowerShell

We may in the future create a tutorial to combine all the steps, so I'll add your ask to my list for prioritization.  Thanks!
@MicrosoftDocs/azure-cxp-triage #please-close

@DCtheGeek Thanks for the super quick response. :+1:


feat(router): add 'replaceRoutes' method

'replaceRoutes' method replaces all routes in router
close #1234

Can someone help me? I don't know why but on my local machine tests are passing while they fail on CircleCI...

@alekbarszczewski Did you look at CircleCI's log? I just glanced at it and there is one case failing.
 ✖ Testing if element <.view> contains text: "Route Y".  - expected "Route Y" but got: "Route X"
    at Object.replace routes (/home/ubuntu/vue-router/test/e2e/specs/replace-routes.js:30:15)
    at _combinedTickCallback (internal/process/next_tick.js:73:7)
    at process._tickCallback (internal/process/next_tick.js:104:9)

@nickforddesign yeah I saw it but I don't get it... When I run tests on my computer all tests pass successfuly. I spent some time to figure out why this is happening without any luck.

https://github.com/vuejs/vue-router/blob/a85c6168e404cea3bd82d5365d7182f1070acbab/test/e2e/runner.js
Line 13: If CircleCI, use phantomjs, otherwise use chrome. So your tests are being executed differently locally than on CircleCI

Please please

Please can this be fixed up and added...?  Need this feature

Thanks for the PR! After going through an RFC, we are going in a different direction. If you are still interested in implementing this, see https://github.com/vuejs/vue-router/issues/3087#issuecomment-626143022


Clarify that 2D cell sizes are defined in pixels

See https://github.com/godotengine/godot/issues/40577#issuecomment-662329021.

Thanks!

Cherry-picked for 3.2.3.


Post 0.56.2 cleanup

This is a partial cherry-pick of #8414 -- without the pin for NumPy. This then also includes the changes originally proposed in #8419.

@stuartarchibald thank you for the review, I have addressed all the comments accordingly.


tf2centerprojectiles 7.2 gamedata update & sm stats cvar

hello again :)
the convar is so usage can be tracked by https://www.sourcemod.net/newstats.php

Thanks for the update :+1:


Fix calling SetWindowShouldClose if _glfwWindow is null

Summary of the PR
Fixes Calling window.Close before window.Run causing an exception on GLFW.
Closes #565

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: roeyskoe sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[stable/nginx-ingress] Metric service name is not truncated to 63 chars

Describe the bug
As per _helpers.tpl in nginx-ingress names are truncate to 63 chars, reasons for this are known. But this seems not to be the case for controller-metrics-service, there is a suffix "-metrics" added. This suffix leads to 71 chars for releases with long name and cause the following issue:
2020-07-10T07:33:14.6835926Z Error: UPGRADE FAILED: failed to create resource: Service "internal-ingress-controller-nginx-ingress-foo-internal-ingress-metrics" is invalid: metadata.name: Invalid value: "internal-ingress-controller-nginx-ingress-foo-internal-ingress-metrics": must be no more than 63 characters
2020-07-10T07:33:14.6896185Z ##[error]Error: UPGRADE FAILED: failed to create resource: Service "internal-ingress-controller-nginx-ingress-foo-internal-ingress-metrics" is invalid: metadata.name: Invalid value: "internal-ingress-controller-nginx-ingress-foo-internal-ingress-metrics": must be no more than 63 characters

There is a legit reason for "-metrics" suffix to prevent naming collisions with other services in  stable/nginx-ingress chart.
Version of Helm and Kubernetes:
Helm v3.2.0
K8s v1.16.10
Which chart:
stable/nginx-ingress
What happened:
Release failed due to chars transgression in resource naming
What you expected to happen:
resource names are truncated to 63 chars
How to reproduce it (as minimally and precisely as possible):
Create a release with long name and enable metrics
Possible fix
We could truncate "nginx-ingress.controller.fullname" to 53 chars, similar issue applys to other resource.

name: {{ template "nginx-ingress.controller.fullname" . }}-metrics
name: {{ template "nginx-ingress.controller.fullname" . }}-admission
name: {{ template "nginx-ingress.controller.fullname" . }}-internal

Create a default fully qualified controller name.
We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).
*/}}
{{- define "nginx-ingress.controller.fullname" -}}
{{- printf "%s-%s" (include "nginx-ingress.fullname" .) .Values.controller.name | trunc 63 | trimSuffix "-" -}}
{{- end -}}

stable/nginx-ingress has been deprecated in favor of the upstream ingress-nginx repo. See https://github.com/helm/charts/pull/22823. Issues if still applicable can be re-opened there. Thanks 😊