Hiding year

How can i hide year selection when i use DateRange Limitter?
Ex: I have limited calendar to april 2017 to March 2018. Then it is showing both 2017 and 2018 years. On selecting 2018 nothing happend. So, I wanted to hide year selection

When you select a different year, it should round the selection to the nearest acceptable one. (31st of march most likely in this case)
If it doesn't do that, it's a bug.

How to set that can u let me know.. I dnt want to display the year itself

The year cannot be hidden


Fix for #549 #512 with black screen after Photo changed to Video

It works and video doesn't freeze anymore if you came from Photo. But the problem is that the protocol YPPhotoCapture changed a bit, I don't know how to make things right.

I'm sorry, change in line self.updateMode(with: vc) is not needed, left after my experiment :)

@nickeyzzz This issue has been bothering us for AGES. If this fix works then.. 🥳

It worked perfectly for me, but it changes the public functions, so I want to ask you to take a further look at the fix, may be there can be a more correct way to do that as changing those public functions is not the best way. But at least you got the idea on how to solve the problem.

@nickeyzzz thxs for your time! There are a lot of new changes in master. Can you please test if the bug is already gone?

Closed because no activity.


Problem After calling Shaka Library we are getting extra text track having label "Shaka Player TextTrack" in textTracks which is read-only

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
4.3.5
Please ask your question
After calling player.load in Video html control we are getting textTracks  with one length having label "Shaka Player TextTrack".
is there any way to remove this track?

This track is created automatically for the management of subtitles. There is no way to remove it. If you want there to be a way to eliminate it, please create an improvement issue so that it can be evaluated. Thanks!


Marker settings resizable

Fixes #16

tested this locally and the resizing works!
https://github.com/sillsdev/solid/assets/3444521/1ff4519f-649a-4ec7-a288-fc694ea25529


An error occurred We're sorry, but for some reason we can't open this for you.

my code

using npm

Same issue..

did u find any solution?

I have the same issue, @cyntler can you please look at this?

same here

Есть ответ? Who find solution?


[setup] added option for coverage measuring

use it on travis ci and upload results to codecov.io
fixes #511

@Thrameos To get the coverage it seems mandatory to have the build inside the source directory. How do I convince the setup to place the jar in the correct spot?
The result would look like this: https://codecov.io/gh/jpype-project/jpype/pull/530/tree?path=native%2Fcommon

By build do you mean everything include the CPython module or just the jar portion?  I have just been using hooks inside setupext directory to move files around after it was created.   If I have a tree layout where things need to be and which command they need to appear there I think I can make it happen.
For example the bdist command copies the built pieces back into the source directory (native) at a specific step.

I am going to take a shot at this.  Do you know how the structure should look or doc that have that info?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@257f2b0). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #530   +/-   ##
=========================================
  Coverage          ?   40.67%
=========================================
  Files             ?       95
  Lines             ?     7061
  Branches          ?     2831
=========================================
  Hits              ?     2872
  Misses            ?     2775
  Partials          ?     1414

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 257f2b0...91ca169. Read the comment docs.

I added the copy code command over but we likely need to also add some code to the pytest command so that it will use the jar rather than the thunk code.  You can move it to a different location if you would like.  The pattern is pretty easy.  In the run command of each section use self.distribution to get to the command line options of setup.  So you can push files around for testing or coverage tools as needed.
With regard to thunks, right now it always pulls the thunk code first rather than asking for the external jar.  To use the external jar we must add the external jar to the class path explicitly so that it is found before the bootstrap loader.  I am planning on making an attempt to get the jar file to install with the shared library so that we don't need the thunk code at all.  I am vacation for the next two weeks so I may have a few hours to try that.

I managed to get support for jacoco to work locally, but we need to figure out how to get jacocoagent and jacococli installed to use it in practice.  Then we would have to upload coverage.xml, coverage_py.xml and coverage_java.xml (unless we can merge them).
It doesn't appear that the java source has to be in a particular location, we just need to run the jar with the javaagent and then post process it to xml.

Not sure if this will do it.  I managed to get coverage from all three languages, but I can't really see how to get them all into the coverage report so I am trying to post each of the pieces and see if the web site can handle it.  Also for the record I grossly dislike maven but as far as I can tell it is the only way I can get the jacoco jars.

@marscher Okay I think everything is good to go on this one.  It didn't produce a coverage report (yet), but I am not sure if that is a problem with tokens or other permissions.  Thanks for getting this started.  Once we have working coverage reports I can start addressing it with improved testing.

Do you mind if I merge this one?

On 1/3/20 4:27 PM, Karl Nelson wrote:

Do you mind if I merge this one?

please go ahead


[ZEPPELIN-5824] Checkstyle

What is this PR for?
This PullRequest introduces the new sub-module "build-tools". This allows the complete Zeppelin project to use the packed Checkstyke configuration centrally.
It is not possible to use Checkstyle in the Zeppelin Parent project, otherwise there will be a cyclic dependency to the sub-module.
The build-tools submodule was created on the recommendation of the check-style developers. See https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html
This PR also removes the Scala style configuration. I have found no usage of the configuration. In addition to the Scala style configuration, XML validation was also thrown out. This is done by the IDE or by Maven itself.
What type of PR is it?
Improvement
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5824

How should this be tested?

CI

Questions:

Does the licenses files need to update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

The name of the module comes from the Checkstyle documentation.
I have found other projects which have adopted the folder name. e.g. https://github.com/microsoft/azure-maven-plugins/tree/develop/build-tools
I'm willing to change the name, I just can't think of a better one.


JSON Parse error: Unexpected identifier "internal"

I am using easy state with react native. I get the following error when mutating deeply nested structures:
- node_modules/react-easy-state/dist/cjs.es6.js:50:20 in batch
- node_modules/promise/setimmediate/core.js:37:14 in tryCallOne
- ... 9 more stack frames from framework internals

Although my app does not crash and perfectly re-renders! I am confused about why this is happening without affecting my app's (current) performance. I am using react-easy-state 6.0.5 and react-native 0.55.4. By deeply nested structure, I mean the following object that resides in my store:
[ ...   {        obj1 : {                    string: "string",                    obj2: {                              ...                             string: "string"                            }                  }   } ]
where the ellipses indicate multiples of the value next to it. I am mutating obj2.

Do you get some error message or just the stack that you provided?

i don't know why it didn't c/p, but the exact message above the stack i provided was the title of this issue: "JSON Parse error: Unexpected identifier "internal"

My bad, the error message was the name of the issue 🤦.
I think this is not coming from easy state. Aren't you setting the mutated
data to a result of a JSON.parse call? This is the typical error when
someone tries to parse an already parsed valid js object.
The stack trace worries me though. It should have a first line (just above
the easy state one) that points you to your own function and exact line
where you do the JSON.parse.
Can you please check if any of these are true? Thanks!
On Fri, Nov 23, 2018, 00:30 Noushin Quazi<EMAIL_ADDRESS>wrote:

i don't know why it didn't c/p, but the exact message above the stack i
provided was the title of this issue: "JSON Parse error: Unexpected
identifier "internal"
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/solkimicreb/react-easy-state/issues/68#issuecomment-441133524,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGoj7lyKu8cbwo99kdQx8e16BuHnRuRlks5uxzOGgaJpZM4YqS3I
.

sorry for the delays in my responses-- there is no JSON.parse on my end of the pipeline that leads to this issue. to be clear, this is the process that causes the issue: a user clicks on component A. component A then calls the update function from parent component B. internally the update function calls a state-mutating function from the store.  component A re-renders with the new value, and moments later the json parse issue occurs.
Additonally, component B is wrapped inside view() while component A is not because view() will not accept a pure component.  Hope this clears up the situation!


Create MAINTAINERS.md

I have just copied the MAINTAINERS.md file from
https://github.com/prometheus-community/ansible , because that repository is related.
Due to this relationship, @gardar and @paulfantom have push rights for https://github.com/prometheus . Listing them as maintainers here formalizes those permissions.
@gardar and @paulfantom, once this is merged, I'll also invite you as regular members of the Prometheus GH org (rather than just "outside collaborators", as you are categorized now).
It would be great if all three of you could approve this PR. @SuperQ @gardar @paulfantom

The test deploy CI is broken, ignoring.

@gardar and @paulfantom, once this is merged, I'll also invite you as regular members of the Prometheus GH org (rather than just "outside collaborators", as you are categorized now).

Thanks! Did you send the invite already? Not sure if I missed it somehow

This was part of a cleanup effort to officially track people who already had access to various repos, but are not official "Prometheus Team" members.
You already had write access to the repo. I have promoted you to "Maintainer" level in GitHub permissions.

We also want to clean up the "outside collaborators". I had not invited you yet because this PR wasn't merged yet and just in proposal state. Now that it is merged, I'll invite you both.


Spike an basic API for requesting User input (do not merge)

This spike demonstrates a possible mechanism to ask for (and receive) user input from a task or plugin, in a way that works in the daemon, with both the rich and plain console.
The API is deliberately very basic, and will not support any form of interaction beyond a single request and response. (Sequential requests will be interspersed by other logging).
The implementation pauses logging while waiting for a user response, spooling all events to be replayed after the response has been received. There's no timeout, and a build could quite easily run out of memory while spooling messages waiting for input. (This is probably OK for the build-scan plugin, since it will likely ask for license agreement in a buildFinished hook, when little other logging is going on).
You can see this API in action with the following basic example:
import org.gradle.api.internal.tasks.UserInputHandler

task ask {
    doLast {
        def userInputHandler = project.services.get(UserInputHandler)
        def response = userInputHandler.getUserResponse("Enter your response:")

        println("You entered '${response}'")
    }
}

Need test coverage for:

Plugin apply
Task action
buildFinished

Supersceded by https://github.com/gradle/gradle/pull/3007


doc: Add Nuxt 2 example to README

This PR

Adds Nuxt 2  example to README

I've updated the readme with a slightly modified nuxt 2 instructions (I've tested it with the default nuxt2 starter template, aka. npm init nuxt-app.), since I wasn't able to run your example:

ctx.$cookies is undefined for me. I can't find it in https://nuxtjs.org/docs/internals-glossary/context#the-context. Is $cookie another external plugin? I'm using directly the req and res context objects.

In the component template this.$pocketbase also doesn't seem to work for me. The inject $pocketbase property is available though as part of the asyncData argument.


compress_gsn_kt returns seg fault when input size is not a square

issue as in the subject

This was fixed by the most recent commit.


#789 wrap text table

#789 #84
When a text is wider than its column in the Table report, text will now wrap to a new line.
Includes some fixes for compact table using the density options :

Currently checking :

Report actions
Break word (it only wraps on new word as of now)


test: fix expectation

Redo of https://github.com/envoyproxy/envoy/pull/2875/files.

cc @jeffkaufman

@jmarantz
external/envoy/test/server/http/admin_test.cc:420: Failure
Value of: std::string(content_type.getStringView())
Expected: has substring "blahblah"
  Actual: "text/html; charset=UTF-8"


Roll Flutter Engine from 9779c273aac3 to 734042c153b8 (2 revisions)

https://github.com/flutter/engine/compare/9779c273aac3...734042c153b8
2024-06-15<EMAIL_ADDRESS>Roll Dart SDK from fd8f26dfc971 to f009db7ba764 (1 revision) (flutter/engine#53415)
2024-06-15<EMAIL_ADDRESS>Roll Dart SDK from dca20ab646c5 to fd8f26dfc971 (1 revision) (flutter/engine#53413)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Introduction To Kasini3000

PR Summary

PR Checklist

Do you want this post to be published on a specific date?: Y/N [Specify date]:
[ ] I have read the contributors guide and followed the style and process guidelines
[ ] PR has a meaningful title
[ ] Includes content related an open issue

see Closing issues using keywords.

[ ] This PR is ready to merge and is not Work in Progress

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

problem : how to cla?


Can't make on CentOS 6.8 Error: implicit declaration of function ‘clock_handler’

./configure
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking whether gcc understands -c and -o together... yes
checking whether make supports the include directive... yes (GNU style)
checking dependency style of gcc... gcc3
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for icc in use... no
checking for clang in use... no
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking whether __SUNPRO_C is declared... no
checking for gcc option to accept ISO C99... -std=gnu99
checking sasl/sasl.h usability... yes
checking sasl/sasl.h presence... yes
checking for sasl/sasl.h... yes
checking for gcov... /usr/bin/gcov
checking for main in -lgcov... yes
checking size of void *... 8
checking for library containing clock_gettime... -lrt
checking for library containing socket... none required
checking for library containing gethostbyname... none required
checking for libevent directory... (system)
checking for library containing gethugepagesizes... no
checking for stdbool.h that conforms to C99... yes
checking for _Bool... yes
checking for inttypes.h... (cached) yes
checking for sasl_callback_ft... no
checking for print macros for integers (C99 section 7.8.1)... yes
checking for an ANSI C-conforming const... yes
checking for socklen_t... yes
checking whether byte ordering is bigendian... no
checking for htonll... no
checking for library containing pthread_create... none required
checking for mlockall... yes
checking for getpagesizes... no
checking for sysconf... yes
checking for memcntl... no
checking for clock_gettime... yes
checking for preadv... yes
checking for pread... yes
checking for accept4... yes
checking for getopt_long... yes
checking for alignment... none
checking for GCC atomics... yes
checking for GCC 64bit atomics... yes
checking for setppriv... no
checking for pledge... no
checking for cap_enter... no
checking for xml2rfc... no
checking for xsltproc... no
checking that generated files are newer than configure... done
configure: creating ./config.status
config.status: creating Makefile
config.status: creating doc/Makefile
config.status: creating config.h
config.status: executing depfiles commands
make make  all-recursive make[1]: Entering directory/root/memcached-1.6.8'
Making all in doc
make[2]: Entering directory /root/memcached-1.6.8/doc' make  all-am make[3]: Entering directory /root/memcached-1.6.8/doc'
make[3]: Nothing to be done for all-am'. make[3]: Leaving directory /root/memcached-1.6.8/doc'
make[2]: Leaving directory /root/memcached-1.6.8/doc' make[2]: Entering directory /root/memcached-1.6.8'
gcc -std=gnu99 -DHAVE_CONFIG_H -I.  -DNDEBUG   -g -O2 -pthread -pthread -Wall -Werror -pedantic -Wmissing-prototypes -Wmissing-declarations -Wredundant-decls -fno-strict-aliasing -MT memcached-memcached.o -MD -MP -MF .deps/memcached-memcached.Tpo -c -o memcached-memcached.o test -f 'memcached.c' || echo './'memcached.c
cc1: warnings being treated as errors
memcached.c:99: error: type defaults to ‘int’ in declaration of ‘evutil_socket_t’
memcached.c:99: error: expected ‘;’, ‘,’ or ‘)’ before ‘fd’
memcached.c:163: error: type defaults to ‘int’ in declaration of ‘evutil_socket_t’
memcached.c:163: error: expected ‘;’, ‘,’ or ‘)’ before ‘fd’
memcached.c: In function ‘conn_worker_readd’:
memcached.c:526: error: ‘event_handler’ undeclared (first use in this function)
memcached.c:526: error: (Each undeclared identifier is reported only once
memcached.c:526: error: for each function it appears in.)
memcached.c: In function ‘conn_new’:
memcached.c:707: error: ‘event_handler’ undeclared (first use in this function)
memcached.c: In function ‘update_event’:
memcached.c:2704: error: ‘event_handler’ undeclared (first use in this function)
memcached.c: In function ‘do_accept_new_conns’:
memcached.c:2750: error: implicit declaration of function ‘maxconns_handler’
memcached.c: At top level:
memcached.c:3580: error: type defaults to ‘int’ in declaration of ‘evutil_socket_t’
memcached.c:3580: error: expected ‘;’, ‘,’ or ‘)’ before ‘fd’
memcached.c:4012: error: type defaults to ‘int’ in declaration of ‘evutil_socket_t’
memcached.c:4012: error: expected ‘;’, ‘,’ or ‘)’ before ‘fd’
memcached.c: In function ‘main’:
memcached.c:6286: error: implicit declaration of function ‘clock_handler’
make[2]: *** [memcached-memcached.o] Error 1
make[2]: Leaving directory /root/memcached-1.6.8' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory /root/memcached-1.6.8'
make: *** [all] Error 2
`

what version of libevent is on there? you need at least version 2. the fix for version detection is queued for the next release, sorry

Thanks for your answer!
You are right. After yum remove libevent-devel and yum install libevent2-devel memcached successfully compiled and all tests passed.


gmxselect biobb does not pass docker test

Should check for a workaround:
https://github.com/bioexcel/biobb_gromacs/wiki/Docker-Inout-files-ie:-topology-file-in-gmx-solvate


Turbolinks breaks Bootstrap v4 Alpha Dropdown behavior

Steps to reproduce:

Generate a page with a bootstrap dropdown menu.
Follow a link in said dropdown.
Try to open the dropdown menu again, it refuses to open.

I looked into the code a little, it seems that when the dropdown meny link is followed, at first everything looks fine, but as soon as the dropdown is clicked the second time (before the code that opens the dropdown runs), it acquires the "open" class.* Then the code the opens the dropdown runs, sees the open class, and returns, leaving the dropdown closed.
*I wasn't able to determine how the open class gets added here.
Disabling turbolinks fixes the problem.
If this is a twitter bootstrap issue, please let me know, and I'll gladly file an issue there, but since disabling turbolinks fixed it, my best guess was that it's a turbolinks issue?

Very similar thing happened to me when using materializecss navbar dropdown ... I wanted to give the new turbolinks a chance but I'm not going to debug third party code which gets broken because of them again ...

Long story about fixing third-party JS libraries after enabling turbolinks. For turbo links 3 everything could be fixed by using jquery.turbolinks gem (https://github.com/kossnocorp/jquery.turbolinks), or by manual initialization of libraries inside $(document).on('page:load') event. Sad, but new turbolinks is much more difficult and you can not fix everything by just replacing 'page:load' with 'turbolinks:load'.

@ewollesen @jacek213 @sintro I think that the #213 has a possible answer for this issue.

@serradura , this is nice explanation, and may be that is correct solving of the problem, but, as I can remember, when I was trying to solve this problem, I considered this case (some clearing in html state in different sorts of _before hooks), and had no success. May be I had mistakes in my code, may be the problem was in another place.

Adding data-turbolinks-eval=false to the other <script> tags loaded seems to be a workaround; it worked for me on Bootstrap v4.
https://github.com/turbolinks/turbolinks-classic/issues/539#issuecomment-105162751

I have same problem. I tried with
rails 5.2.0
gem 'bootstrap-sass', '~> 3.3.7'
gem 'sass-rails'
in application.js.coffee
#= require jquery
#= require bootstrap-sprockets
#= require rails-ujs
#= require activestorage
#= require turbolinks
#= require_tree .
in  with slim file
= javascript_include_tag 'application', 'data-turbolinks-track': 'reload', 'data-turbolinks-eval': false
seems bootstrap got multiple times.

I'm unable to reproduce this using the latest version of bootstrap, or with version 3.3.7. If you are still experiencing this problem, you may be able to get some help by posting a question on Stack Overflow (ideally with a live demo), tagged with Turbolinks. Thanks.


statsds-to-prometheus bridge does not extract dimensions from metrics

statsds-to-prometheus bridge does not extract dimensions from envoy metrics
Having dimensioned metrics is easier to view in prometheus.
Example mapping:
apiVersion: v1
data:
  mapping.conf: |
    mappings:
    - match: envoy\.cluster\.(.*)\.upstream_rq_time$
      match_type: regex
      name: "rq_time"
      labels:
        cluster: "$1"
    - match: envoy\.cluster\.(.*)\.upstream_rq_(\d{3}|total)$
      match_type: regex
      name: "rq_count"
      labels:
        cluster: "$1"
        status: "$2"
kind: ConfigMap
metadata:
  name: istio-mixer
  namespace: istio-system

https://github.com/istio/istio/pull/5068


make --remote and --s as default for istioctl version command

make --remote and --s as default for istioctl version command to address the issue in: https://github.com/istio/istio/issues/12703

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: linsun, nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @linsun @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS
pkg/OWNERS
prow/OWNERS

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

@irisdingbj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b678d05842985933c43eadbe23be5f4f7a41741f
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b678d05842985933c43eadbe23be5f4f7a41741f
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
b678d05842985933c43eadbe23be5f4f7a41741f
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn, irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: linsun, nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @linsun @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS
pkg/OWNERS
prow/OWNERS

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

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dwradcliffe, howardjohn, irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: linsun, nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @linsun @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS
pkg/OWNERS
prow/OWNERS

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

@dwradcliffe: changing LGTM is restricted to assignees, and only istio/istio repo collaborators may be assigned issues.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dwradcliffe, esnible, howardjohn, irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: linsun, nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @linsun @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS [esnible]
pkg/OWNERS
prow/OWNERS

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

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

Needs approval from an approver in each of these files:

istioctl/OWNERS [esnible]

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

@irisdingbj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
1b41480130655939a674dc3817ff86195ce4b2e7
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Running cucumber while requesting a html format output causes a scenario to fail

I understand this is not a good test case. Please feel free to close this if this bug report is of no use.
I get the following error...
 [JavaScript Error: "h is null" {file<EMAIL_ADDRESS>line: 8478}]'[JavaScript Error: "h is null" {file<EMAIL_ADDRESS>line: 8478}]' when calling method: [nsICommandProcessor::execute] (Selenium::WebDriver::Error::UnknownError)
... when I run
ephemeral-x.sh bundle exec cucumber --format json -o out/cucumber.json --format html -o out/cucumber.html features/my_lovely.feature
or
ephemeral-x.sh bundle exec cucumber --format html -o out/cucumber.html features/my_lovely.feature
... but not when I run
ephemeral-x.sh bundle exec cucumber --format json -o out/cucumber.json features/my_lovely.feature
or
ephemeral-x.sh bundle exec cucumber features/my_lovely.feature
This makes me think it could be a problem with how cucumber handles the --format option rather than in the selenium webdriver.

Hi,
Is this consistent? The only cause that springs to mind is that the formatters are called synchronously with the steps, so running multiple formatters will have a marginal effect on how long each step takes to run. It could be triggering a race condition in your javascript code maybe?
It would be interesting to understand what that error means, in terms of what has or hasn't happened on your app that you would have expected it to. Have you tried to figure that out?

It's consistent. It only happens on a Jenkins CI server. I haven't deep dived this issue - just done enough to ensure that it the error happens consistently. I was thinking that the extra writing of the HTML format might have been a cause (even though it seems unlikely) but you've nicely described what the problem could be in your description of the potential race condition, thanks.
I will try to get a chance to look into this again soon. But for now, if you want to close this then just do (I can live without the HTML output for a while).

Yeah, these things are a nightmare to debug. One quick hack to test my theory could be to put a well-positioned sleep in your steps to simulate the delaying effect of the formatter.
I don't think there's much we can do right now, so I will close it if that's OK with you. If you get the energy to do any more investigation though I'll be happy to help. You could also hop onto https://gitter.im/cucumber/cucumber for some live support :smile:

Thanks a million @mattwynne. I will update this ticket when I return to the debug nightmare :-)
Much appreciated,
Dec


[WIP] Metrics and structured logging

I think writing your own logger is unnecessary here. Ruby has a Logger class that you could use. It supports messages that are not strings. Take a look at the #formatter attribute. Basically you can set it to a Proc or any object that responds to #call and format the messages the way you like - in this case into a json.
And also I think there is no need to have per object loggers. I would:
require 'logger'

class CSV2Avro
  def self.logger
    @logger ||= Logger.new(STDOUT).tap do |logger|
       logger.formatter = JsonFormatter.new
    end
  end
end

Obviously JsonFormatter does not exist yet.
And then you could use it like this:
CSV2Avro.logger.info(Event.new(...))

And if you build upon the stdlib logger later we can extract this into a gem and use it in other projects. Central and Cartman could definitely use a better logger than what it currently used.

@tmichel How about now? I also moved logging related stuff to their own module and subdirectory. We could also extract to a gem right about now, because I'll need to do pretty much the same thing for Avro2Kafka otherwise. What do you think?

I think we should extract the logging to a gem and make it a little bit more flexible so other applications can use it.

@rgabo ready for review. The CI won't turn green until we release Logr to rubygems.

LGTM.


pull request for e2e test at 12/07/2016 19:15:01

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Since your model didn't use the identity embedding, How your model find the output belongs to search sequence?

Hi, we know the position of search tokens because we concatenate them with the target tokens in the first place. For example, if we have 40 target tokens + 60 search tokens, after transformer, the last 60 search tokens are still search tokens. I am not sure if I understand your question correctly.


Import after use gets removed

With 0.7.4 and --remove --include-star-import --ignore-init --gitignore
In code like this
def bob():
    print(sys.path)

import sys

The import gets incorrectly removed.
You might reasonably ask why I even had code like this, in larger files my work flow is to just drop imports in the nearest bit of module level scope and let unimport and isort clean it up.

Unimport is currently using primary analysis methods, this problem will be solved if scope analysis is added.


split_all/3 eat whitespaces

It is a bit surprised that sys:split_all/3 will eat some whitespaces, e.g.:
:- initialization(main).

main :-
        sys:split_all('#. #\t.\n', '.', L),
        write(L), nl,
        halt.

Output:
% tpl foo.pro
[#,#    ]

As split_string have been added recently, I kind of think that it would be nice to make split_all/3 preserve whitespaces.

Thanks!


Add gripmap v methods

This PR adds three methods to GridMap - get_cell_itemv, get_cell_item_orientationv, and set_cell_itemv - to bring it up to parity with TileMap, which has the equivalent to those methods already.

That's 7,781 commits more than I intended. Apologies...


Unsubscribe offer shows up immediately after renaming a stream when narrowed

This is a bug, triggered by being narrowed to a stream+topic and renaming the stream, and then exiting the streams modal (while still narrowed):

While we're at this, we should clean up that styling for that text (at least the --- should go).

@zulipbot claim

Ideally we should fix this by fixing #4692.


Package request: BalenaEtcher

Project description
Flash OS images to SD cards & USB drives
Metadata

homepage URL: https://balenaetcher.org/
source URL: https://github.com/balena-io/etcher
license: mit, bsd, gpl2+ , ...
platforms: linux Windows, IOS

It's already packaged.

Where? Ive only found
https://search.nixos.org/packages?channel=23.05&show=etcher&from=0&size=50&sort=relevance&type=packages&query=etcher
Which doesnt actually work.

What's the difference?

im currently installing etcher so i will let you know when its done.
Etcher has a EOL dependency so normally you wont be able to install it.

The package needs update. :shrug:


issue with data initialization

Sorry for this repeative post, I'd just clicked submit accidently before I finished writting description.
As we can see in the main.py
def data_initialization(data):
    data.initial_feature_alphabets()
    data.build_alphabet(data.train_dir)
    data.build_alphabet(data.dev_dir)
    data.build_alphabet(data.test_dir)
    data.fix_alphabet()

words in the test and dev file are used to build data.word_alphabet  in data.py
def build_alphabet(self, input_file):
        in_lines = open(input_file,'r').readlines()
        for line in in_lines:
            if len(line) > 2:
                pairs = line.strip().split()
                word = pairs[0]
                if self.number_normalized:
                    word = normalize_word(word)
                label = pairs[-1]
                self.label_alphabet.add(label)
                self.word_alphabet.add(word)

So that the oov problem is hidden as a result of this operation.
If we comment data.build_alphabet(data.dev_dir) and data.build_alphabet(data.test_dir) in main.py, the f1 score will decrease by about 5% in test data.
So is this just a mistake or there are some other considerations concerning this operation?
Thanks

Actually, the oov problem you raised is the out of embeddings (OOE) problem. By adding the dev/test words into alphabet will not affect the final result, but it can make some OOV words have a good pretrained embeddings. But we can also get the same results if we keep all the pretrained Word embeddings in the word alphabet. So this is only a fast implementation which saves the memory usage but does not change the system performance in real application.


Improve Travis CI build Performance

Caching Dependencies and Directories Travis CI can cache content that does not often change, to speed up the build process.
=====================
If there are any inappropriate modifications in this PR, please give me a reply and I will change them.

It seems that Travis CI is not free for us anymore: https://travis-ci.org/
We should migrate to Github Actions and perhaps manually publishing the docs for now.


Dependency guard requires > 1GB of memory for library

Description
Running dependency guard on my project fails because it crashes with an out of memory warning.
PHP Fatal error:  Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 36864 bytes) in /home/travis/build/php-twinfield/twinfield/vendor/nikic/php-parser/lib/PhpParser/ParserFactory.php on line 28

You can see the build output here: https://travis-ci.org/php-twinfield/twinfield/jobs/436207071
Steps to Reproduce

Clone https://github.com/php-twinfield/twinfield
Run dependency guard
Crash!

Expected behavior: [What you expect to happen]
Don't take > 1GB for analyzing a < 50K loc project
Actual behavior: [What actually happens]
Out of memory crash
Reproduces how often: [What percentage of the time does it reproduce?]
100%
Versions
You can get this information from copy and pasting the output of:

As dependency: composer show mediact/dependency-guard | grep version
As root package: composer show --self | grep version

latest versions (no lock files used)

@willemstuursma I cannot reproduce this error at 1024M memory limit in a regression test, but I can get the same result at 512M and will continue with that as limit.


Merge upstream changes (gocql v1.6.0)

Merge upstream changes as of gocql v1.6.0.
Fixes: #143

@sylwiaszunejko Were there any "tricky" merge conflicts or did it go smoothly?

@sylwiaszunejko Were there any "tricky" merge conflicts or did it go smoothly?

nothing tricky, changes in the comments in control.go was the only merge conflict


Downsize loaders to 0.5 vCPU, 1 GB RAM

We've long known that the loaders are probably overprovisioned. This downsizes them by 50%. Partially covers #303.

It’s very possible these could work at an even smaller scale. What we’ve set here is almost certainly fine.


Update Focal Fossa references to Jammy Jellyfish in the README

Just noticed this when I was checking out out the new repository. Thanks for your work!

Oops! Thanks.


Implement SpecLogicEip4844

We need to provide SpecLogicEip4844 which will reference all 4844 specific classes where applicable.
This issue can be closed only when all expected classes were referenced in the create method.

spec logic is complete


beforeCreate is called before checking the unique validation

beforeCreate callback is called before checking the unique validation.
for example:
product.js
attributes: {
  name: {
    type: 'string',
    unique: 'true'
  }
},
beforeCreate: function(values, cb) {
  console.log('beforeCreate');
  return cb();
}

when I run the code beforeCreate is called before checking the unique validation.
I need to add some data to the elasticsearch from beforeCreate callback. What happened to me is that before firing the unique validation error the beforeCreate is called and the data is indexed to elastic. According to the docs beforeCreate is called only after the completing the beforeValidate and afterValidate callbacks
I am using sails-mysql adapter.

The unique validation doesn't do a query before your query is executed to check uniqueness, this would slow everything down and not be transactional anyway. When you define an attribute as unique Waterline instructs the adapter to create a unique index on that attribute. In this case you should move the call to afterCreate or inside your controller method.

@particlebanana For example values.records. The records is not saving in the sails database only to the elastic database. So it will not be available in the afterCreate values.

@jetsonjohn your confusion comes from bad documentation. Technically unique is NOT a validation as it is merely a DB constraint. If you approach the problem with this mindset, @particlebanana suggestion makes the most sense.
However, this issue could be re-opened as it does show a glaring omission in the documentation.


Include reports folder in S3 upload path

Trello: https://trello.com/c/FtV7sVMU/138-admin-download-all-suppliers-for-framework-endpoint-locks-up-an-api-process
In order to serve the S3 file in the admin (via the DM_ASSETS_URL), the router needs to proxy the url to the correct bucket. To do that, it uses a regex to match the path supplied (see https://github.com/alphagov/digitalmarketplace-router/pull/30/files). The other asset buckets include their type in the path structure to assist with this (e.g. /g-cloud-10/communications/filename.csv), and the reports bucket should do the same.
TL;DR - this PR sticks a .../reports/... in the CSV upload path.
Tested against preview already, works as expected.

Thanks for the speedy review 😸


Rename FPRL references with PRL

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PRL-431
Change description
Rename FPRL references with PRL
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

No more needed


Feature request: Shell alias expansion

It would be nice to be able to use aliases in the watch commands :)

I've closed https://github.com/sachaos/viddy/issues/9, since it was a duplicate of this one.

Apparently if we do this:
alias my_alias="viddy " (note the extra space after the command)
It starts working with aliases:
my_alias k get pods -n all (k for kubectl)
¯_(ツ)_/¯

True, but this has the same shortcoming as mentioned here: https://unix.stackexchange.com/a/25329

Unfortunately this method has a limitation that you cannot provide options to watch because the alias has to be the next word, whereas watch needs its arguments before the command to run. If you tried watch -n 10 , bash won't expand the alias

To solve this, should I parse the file which is defining the aliases (e.g. bashrc or zshrc)?
I haven't come up with another good way to do it.

The command could be called via the shell:
shell := os.Getenv("SHELL")
cmd := exec.Command(shell, "-ci", "k get pods")

This could be considered insecure though, so maybe it could be opt-in via flag?

Yep, that would be my suggestion too!
Get the shell and execute the command within that shell.
Should work for the most used shells at least.
* with fish i think the arguments order need to be ic instead of ci.

Thank you! I understood.

@Croissong @davidandradeduarte I've add new options --shell, --shell-options in v0.1.8 .
Now, you can run alias command like following examples.
Please specify your aliases in ~/.profile (the files which are loaded on starting as login shell).
Define you aliases in .profile
alias ll="ls -l"

Run viddy
viddy --shell-options "--login" ll

How do you think this solution?
To run on the shell interactive mode, we should run a command on PTY.
It is not difficult implementation, but it is too complex to just support alias.
And interactive mode will load rc file (e.g. bashrc, zshrc), it is too heavy to call every time. (I'm defining much of functions and using complex prompt).
So I recommend to run a command on login shell with specifying --login option.
This will load profile file only.
The --shell-options is long name option, but I'm planning to implement loading the configuration from a file.

It's good to at least have the option, but I don't think most users will maintain a separate file for aliases just to use in viddy :/
We can get them easily with:
alias | xargs -I {} echo "alias {}" > ~/.profile

But it's hard to maintain.
This will not work for me right now tho, because you're only getting the first string from the alias.
e.g
alias ll=ls -la

only gets ls instead of ls -la.
If I change it to
alias ll="ls -la"

with quotes, it works.
On another note, I've tried the --shell options with:
viddy --shell "/bin/zsh" ll

and doesn't seem to work either. Maybe I'm doing something wrong, haven't looked at the code yet.

@davidandradeduarte

This will not work for me right now tho, because you're only getting the first string from the alias.
e.g
alias ll=ls -la

I think this is invalid alias definition. The quotation is necessary.

On another note, I've tried the --shell options with:

viddy --shell "/bin/zsh" ll
and doesn't seem to work either.

When you want to use alias, you should specify --shell-options "--login" to load the alias definition file.
I think maintaining alias file is easy.
I suggest to create file for the aliases, and load that from .profile and rc file (e.g. .bashrc) like following.
Define alias in .alias
alias ll="ls -l"

*Load that from .profile and rc file (.bashrc) too"
source ~/.alias

@sachaos Thanks for the update!
It works as you described, and I understand the problems with the interactive mode.
For me personally, sourcing my aliases from .profile is fine.
I wasn't aware of the implications of using interactive mode outside of a pty, and to be honest, i still don't fully understand it (and how it can be solved).
As far as i can tell, running the shell in non-interactive mode still provides all the other advantages of spawning a shell (e.g. pipes work), so i am perfectly happy with this solution :+1:

Thank you for trying out.
I couldn't get the output from interactive mode shell.
But, it might be my mistake.
Il've used golang pty package, and that worked.
I'm not using that now, to keep it simple.

I would like to close this issue.
If there are other problems, feel free to open or comment here. :)

My alias file is little complex, so I use this function.
vd() {
	viddy -d -n 1 --shell zsh  "$(which $1 | cut -d' ' -f 4-)"
}

# ex:
alias kgp="kubectl get pod"

After I can run like this
$ vd kgp

caution: but vd can't handle functoin.

In fish this function works for me
function vdy --description 'modern watch'
    viddy -d -n 2 --shell fish $argv[1..-1]
end

Thanks to this suggestion by @lirlia, I found a solution that works for me in zsh.
However, I noticed that this suggestion has the limitation of only being able to handle single word aliases. So, if I wanted to watch a command like the following, it wouldn't work:
vd k get pods

Many people have aliased kubectl to k so the alias should get expanded, but the following words of the command shall be handled as well.
We can do this by setting an alias like the following:
alias vd=(){viddy -d -n 1 --shell zsh  "$(which $1 | cut -d' ' -f 4-)${@:2}";}

${@:2} means that all remaining words of the command still get passed to viddy.

I am not sure if this tool is still being developed, but in case it is - is there a way to add . ~/.zshrc right after the zsh -c?
I assume it should be here: https://github.com/sachaos/viddy/blob/master/snapshot.go#L117-L118.
The idea behind it is to run viddy with zsh -ic ". ~/.zshrc; $ARGS, which will support both aliases, functions and won't fail when using the aforementioned alias/function when running things like watch time

@Shaked This tool is alive.
The suggestion seems interesting.
I think we can not add any argument right after -c now.
In this case we should add that feature as experimental option.
We should add --pre-command option or something to execute arbitary command before the  ARGS.
The PR is welcome.

@sachaos thanks for the quick reply!
I have started playing with it but it seems like my idea doesn't work. Only -ci would and that seems like a problem with viddy.
I don't mind filing a PR but I would have to find a way to make this work first :)


Exception Handling on Git Push for Sensor Data

We should have a procedure to handle the Git exceptions such as "File is larger than GitHub's recommended maximum file size of 50.00 MB" or "File exceeds GitHub's file size limit of 100.00 MB" when pushing the sensor data or logs.

To prevent these kinds of problems from happening, a file/log rotation system may be of help.


Vendor Name
OKDO
Vendor Country
UK
Vendor URL
https://www.okdo.com/
Product Links
https://rpilocator.com/?vendor=okdouk
Additional information
https://github.com/nerves-metal-detector/nerves_metal_detector/blob/master/CONTRIBUTING.md#adding-new-vendors-and-products-from-rpilocator

Not on rpilocator anymore. Needs manual list of products. Doesn't seem to have any raspberry pis in their product list.


Feature request

Is your feature request related to a problem? Please describe.
I am used to use the pc when i am not at home, so i do hotspot with my phone.
In theory the pc and the phone should be in the same network, right?
By the way in the phone i am not able to see the phone and viceversa.
Describe the solution you'd like
If  i do hotspot with the phone to the pc i can send the file ecc..
Describe alternatives you've considered
No others ideas
Additional context
The phone is Android and the pc is Linux (Fedora)

Yes, a hotspot should already work

Yes, a hotspot should already work

I just uninstall both the app and the appImage i now it is working.
sorry my bad


Create Visual Studio Solution and empty projects.

Create solution with these projects:

Library Class Project where SiMS base shall be imported
WPF Project for Hospital Editor
ASP.NET Core Project for Hospital Web Application

 Set up solution and projects


[msal-common] Configuration and Client

This PR is the first for msal-node and does the following:

Move around config and app modules
Rename them to be more generic

This will be followed up with a PublicClientApplication for msal-node and PublicClientSPAApplication for msal-browser working. Currently this branch is compilable only with msal-common.

/lib/.DSStore should be ignored

/lib/.DSStore should be ignored

+1

@DarylThayil  The ignore rules for .gitignore currently are:
MSAL specific ignore files
node_modules/
typings/
lib/
Doesn't this include /lib/.DSStore?

A pattern for .DS_Store files should be added to the root gitignore (doesn't appear to be there), and the ds_store file in this commit should be deleted.

Ah! I was looking at the wrong place!


Fix tuple + subcommand edge cases

Fixed some issues identified by @wookayin in #89!
Specifically:

We previously assumed that field names would always be unique. This was causing problems when we used tyro.conf.arg(name="") to erase the name of multiple fields, which can help collapse nested structures.
- This syntax is kind of just a hack, probably should avoid recommending this and instead stick to tyro.conf.OmitArgPrefixes, tyro.conf.OmitSubcommandPrefixes etc.
Fixed an edge case with nesting union types inside of tuples. Tuples are treated a bit specially because they can either be collapsed into a single argument (tuple[int, int, int]) or expanded into many (tuple[Dataclass1, Dataclass2, Dataclass3]).

We've also:

Expanded the scope of tyro.conf.OmitSubcommandPrefixes[] a little bit. This is technically a breaking change, but it's a niche feature, and after checking GitHub I confirmed that it doesn't break any existing (public) systems.
Added the cases from #89 as tests.
Improved the automatic test generation for Python 3.11 testing (which uses new-style syntax, like A | B instead of Union[A, B]).

Thanks. I will try it out and leave some reviews. Note that PEP-604 requires python 3.10, so need to bump minimum python versions. Or PEP563 (python 3.8+).


[system] Update style function to use vars automatically if available

The PR updates the style function inside the system to pull values automatically from the vars if they exist. If we decide to go in this direction, I will simplify the style function for the sx prop in Joy.

cc @siriwatknp for thoughts

cc @siriwatknp for thoughts

This would simplify a lot on Joy! 👍

@siriwatknp I will  proceed with the changes then, this is not breaking anything for Material UI, and will automatically work if the CSS vars provider is used.

@siriwatknp I will proceed with the changes then, this is not breaking anything for Material UI, and will automatically work if the CSS vars provider is used.

Awesome!


set configuration from the python files

From the examples, setting yLimits work but setting the timeWindow gives an error:
Traceback (most recent call last):
  File "<string>", line 7, in <module>
  File "/home/jcolmenares/development/signal-testing/oscilloscope.py", line 8, in <module>
    addPlot(timeWindow=20, yLimits=[-50, 50])
  File "<string>", line 153, in addPlot
AttributeError: PlotWidget has no attribute named 'setTimeWindow'

I'm just adding this line to the python file:
addPlot(timeWindow=20, yLimits=[-50, 50])
We would also like to be able to set the point size and line style from the python file, so that you don't have to load two files every time. Since this two things are global to all plots the idea is that the user sets them on the first plot and get done with it. If you guide me through the code a little bit I'll be more than happy to do it.

Are you sure you have an up-to-date build on the master branch?  Signal-scope includes some example scripts, and a few of them use the timeWindow attribute with error, it works.  For example, this example command should work:
$ signal-scope examples/example5.py

The Python api can call any Qt function which is marked as a slot.  Simply edit plotwidget.h and move the methods such as setBackgroundColor() and setPointSize() to be listed under the "public slots:" tag, then recompile.  Then you can call plot.setPointSize(4) from the Python console or script.
If you submit a pull request with that change I'd be happy to accept it, please also add a demonstration to one of the example py files.  Thanks!

Done!
The error was on my side.
About loading the configuration from python, this is what I did. Since the curve style and point size are the same for all plots, I made a function on signalScopeSetup.py called  "formatOptions". I also added the time window, so if its the same for all plots you have to write it only once. Plus, added the example about how to use it to the util_example that has three plots. I'll make the pull request so you can go through the code and let me know what you think.


fix managedcluster deploy

/approve
/lgtm

/retest

/retest

/lgtm

/cherry-pick release-2.1

see https://github.com/open-cluster-management/backlog/issues/4135


set brightness?

Is that supported? Thanks!

This function is not binded to js yet.
But if you really need it, feel free to implement this functionnality and do a PR to share it ;)

on it! expect something later tonight

@roykolak did you ever get brightness implemented?

@VictorioBerra I actually just ended up shading the color value. Something like this:
https://gist.github.com/roykolak/9604140af3214dfffd13d5d7950ab524

Thanks! Does that achieve the same result?
On Dec 20, 2017 11:54 AM, "Roy Kolak"<EMAIL_ADDRESS>wrote:

@VictorioBerra https://github.com/victorioberra I actually just ended
up shading the color value. Something like this:
https://gist.github.com/roykolak/9604140af3214dfffd13d5d7950ab524
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zeitungen/node-rpi-rgb-led-matrix/issues/8#issuecomment-353135800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACzG67g5T5cyHG8acTMYPAnKpKNeKeKuks5tCUm-gaJpZM4I6Nwv
.

yep, it does
On Wed, Dec 20, 2017 at 11:57 AM Victorio Berra<EMAIL_ADDRESS>wrote:

Thanks! Does that achieve the same result?
On Dec 20, 2017 11:54 AM, "Roy Kolak"<EMAIL_ADDRESS>wrote:

@VictorioBerra https://github.com/victorioberra I actually just ended
up shading the color value. Something like this:
https://gist.github.com/roykolak/9604140af3214dfffd13d5d7950ab524
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/zeitungen/node-rpi-rgb-led-matrix/issues/8#issuecomment-353135800
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ACzG67g5T5cyHG8acTMYPAnKpKNeKeKuks5tCUm-gaJpZM4I6Nwv
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zeitungen/node-rpi-rgb-led-matrix/issues/8#issuecomment-353136672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAkWoe4U9S-6LBk9kP5Fhy4AaPHIHC1ks5tCUqEgaJpZM4I6Nwv
.


Proposal: Button entity row

Use cases

combine singe action entities into a more compact row
add buttons to trigger events
add buttons to simplify things with many choices (media_player source, light effects)

Examples

      - type: entities
        title: Living Room Lights
        show_header_toggle: false
        entities:
          - type: button
            buttons:
              - icon: mdi:lightbulb-on
                icon_color: yellow
                name: "On"
                service: events.ut_lights_up
              - icon: mdi:lightbulb
                icon_color: grey
                name: "Off"
                entity: scene.lights_out
              - icon: mdi:movie
                icon_color: red
                name: Movie
                entity: scene.movie_lights

          - type: button
            buttons:
              - name: "Power"
                icon: mdi:power
                icon_color: yellow
                entity: switch.tv_power
              - name: "Home"
                service: media_player.select_source
                service_data:
                  source: Home
          - type: button
            buttons:
              - icon: mdi:volume-minus
                entity: switch.tv_volume_down
              - icon: mdi:play-pause
                entity: media_player.roku
              - icon: mdi:volume-plus
                entity: switch.tv_volume_up
          - type: button
            buttons:
              - name: "Netflix"
                service: media_player.select_source
                service_data:
                  entity_id: media_player.roku
                  source: Netflix
              - name: "Hulu"
                service: media_player.select_source
                service_data:
                  entity_id: media_player.roku
                  source: Hulu
          - type: button
            buttons:
              - name: "HBO"
                service: media_player.select_source
                service_data:
                  entity_id: media_player.roku
                  source: HBO GO
              - name: "Prime"
                service: media_player.select_source
                service_data:
                  entity_id: media_player.roku
                  source: Amazon Prime Video

Implementation

I don't think this really fits as a row type in an entities card. I think this would make more sense as a separate card type (hui-button-card?).
I can definitely see the use case in your examples and it could be a simpler alternative for people who create this type of layout using a picture-elements card.

Those buttons would all just be call_service. No defaults for entities (like toggle for light etc)

@balloob I see your point about the unaligned look, but I still think it's very useful to be able to arbitrarily mix buttons with regular entity rows
Only allowing them at the bottom per the material spec would alleviate that, but it feels too restrictive.
Perhaps using an outline or contained button instead would make the buttons more visually distinct and offset that to some degree?

Using space-between would align things to the outer edge of the card better. I could add in some extra checks to account for buttons that are missing an icon to align the text

Some initial CSS to align the outside edges a bit better

It still looks too chaotic. It's also not clear what tapping on it does. Navigate? More info? If it's calling a service, how do we know what the current state is and if the service is needed?
I think that this amount of flexibility better stays as custom UI.

Copy that


Syncing docs across docker-compose and kubernetes for the common comp…

Rationale
While testing deployments with Docker Compose, I updated a few things in its README.md.  The docker/README.md and helm/README.md share some elements. I noticed that each could benefit from syncing with the other.  Let's make them more consistent
Notes
In docker/README.md, the section ## Initial Installation vs. Upgrades isn't fully synced to helm/README.md.  Before I did, I wanted to get input on the suggested changes. And then would sync afterwards.

@findtopher - Copy editing isn't a nit : )
I accepted all of your suggested changes. Thank you!


Do not try to hardlink glslangValidator

Previously, it was possible for the hardlinked glslangValidator to become truncated when the glsl-to-spirv crate was rebuilt. The file would be successfully hardlinked the first time, but on subsequent builds the hardlink attempt would fail becasue the target already exists. This would cause the build script to fall back to a copy, which truncates when source and dest are the same file.
This removes the hardlinking entirely, meaning that we always just copy our built glslangValidator to where we want it. While it would be possible to try to make the hardlinking/copying logic idempotent, the complexity (and potential fragility to similar bugs in the future) doesn't seem worth it to avoid one copy.
This fixes #95

Thanks!


Great example! Perhaps some changes for newcomers..

Hi,
Good to see your example project, leaning a lot reading your code.
Perhaps some small changes at the readme.md instructions:

git clone with the https url to the project

meteor shell only works after starting meteor. Perhaps add the database seeding to a Meteor.startup?

Hope to see more of your example project!

Happy to see this helps out. Those are good recommendations, I'll update the readme.
Thanks!


Heads-Up: Title change and minor edits

Docs Build status updates of commit d773b58:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/mrlearning-base-ch5.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Optimize native animated connection queue handling

Change O(n^2) to O(n)
Summary
Minor follow-up to: https://github.com/facebook/react-native/pull/24177
Changelog
[Internal] [Changed] - Optimize native animated connection queue handling
Test Plan
Run CI

Hmm, why does js_coverage fail with:
Test Suites: 90 passed, 90 total
Tests:       1 skipped, 705 passed, 706 total
Snapshots:   244 passed, 244 total
Time:        78.067s
Ran all test suites.
Done in 78.86s.
/home/circleci/react-native/node_modules/coveralls/bin/coveralls.js:18
throw err;
^
Bad response: 422 {"message":"Couldn't find a repository matching this job.","error":true}
Exited with code 1


Java.lang.NoClassDefFoundError

I don't know if I'm doing anything wrong but I am stuck, I keep on getting this error no matter how I try and fix it

Where do you get that error? please explain.

When i am opening minecraft

I am making a Metal gear Rising game (So Don't Laugh.)

Also I don't know if im doing anything wrong (I have not checked the comment section of the mcreator page)

You need to install the player animator library mod

How though (I am new to coding)

Do I put into the mod file in minecraft


Support signing helm charts

I was wondering if we could use cosign to sign helm oci artifacts.
From the docs and what i breafly have read, it appears this tool was created to specifically sign images (because of the dockerReference and dockerManifestDigest).
I was thinking that it should be viable to make this a more broad tool to sign all kinds of artifact manifests, and not only the docker images.
I just tried to cosign an oci:// helm image and this tool seems to work (not sure if i am missing some details).
Maybe we should think of making manifest signing a thing?

@dioguerra We are already using cosign to sign helm charts, you can see how we are signing our own helm-charts. https://github.com/sigstore/helm-charts/blob/main/.github/workflows/release.yml#L43. You can get more info here https://github.com/sigstore/helm-sigstore.

cc @sabre1041 as well

@hectorj2f we are still leveraging the existing supported method of signing charts (GPG based).
However, given that OCI based charts are just OCI artifacts, you can use cosign directly to target the OCI reference.
For simplicity, a minimal workflow would be the following:

Create a chart

$ helm  create mychart

Package chart

$ helm  package mychart

Push chart to OCI registry

$ helm  push <packaged_chart> oci://<registry+namespace>

Create a cosign private key

$ cosign generate-key-pair

Sign Chart/OCI Artifact

$ cosign sign --key=<key> <registry>/<namespace>/<chart_name>:<chart_version>

Then at install time

Verify Chart/OCI Artifact

$ cosign verify --key=<public_key> <registry>/<namespace>/<chart_name>:<chart_version>

Install Chart

$ helm install <release_name> --version=<version> oci://<registry+namespace>/<chart_name>

@dioguerra Would the aforementioned solution work for you ?

Yes, i belive so. I gave it a try a while back and it worked. Thanks


page.evaluate(() => window.performance) is empty object in 0.13

This code:
const puppeteer = require('puppeteer');

(async () => {
    const browser = await puppeteer.launch();
    const page = await browser.newPage();
    await page.goto('https://google.com');

    const performance = await page.evaluate(() => window.performance);
    console.log(performance);

    await browser.close();
})();

will display performance object in version 0.12 (like here) but in version 0.13 it shows only empty object {}.
page.metrics() and page._client.send('Performance.getMetrics') is not a solution since they are lacking of a valuable metrics like in window.performance.timing.

There were some changes in result serialization before the  0.13. You can try this:
    const performance = JSON.parse(await page.evaluate(
      () => JSON.stringify(window.performance)
    ));

@vsemozhetbyt It solve the problem. Is it mentioned in API somewhere? It should be.

If above solution didn't work, let try with
const performance = JSON.parse(await page.evaluate(
        async () => Promise.resolve(JSON.stringify(window.performance))
    ));


fix(abci/client)!: Remove unused ReqRes method GetCallback

PR checklist

[ ] Tests written/updated
[X] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Fixed in #4040


Configuring plugins versions

To apply a plugin, we can use the plugins DSL https://docs.gradle.org/current/userguide/plugins.html#sec:plugins_block
plugins {
  id 'com.github.jk1.dependency-license-report' version '1.9'
}

The problem is replacing the hard-coded version 1.9 with Versions.gradle_license_report does not work for Groovy files.
Is there a workaround to be able to use buildSrcVersions for managing plugin dependencies as well as Library dependencies?
This is already done for the special case of buildSrcVersions itself (https://github.com/jmfayard/buildSrcVersions/issues/47) , although that only works in .kts files.
plugins {
-  id("de.fayard.buildSrcVersions") version "0.4.2"
+ buildSrcVersions
}

Hello, this is a good question. I don't really get why this works in Kotlin but not in Groovy.
plugins {
  id 'com.github.jk1.dependency-license-report' version Versions.dependency_license_report
}

The generated version is a static string

@sigurdurrafn If you are using Gradle 5.6 or have the possibility to update, I would recommend to try the new Plugins version Management feature
https://docs.gradle.org/5.6/userguide/plugins.html#sec:plugin_management
// settings.gradle
pluginManagement {
  plugins {
      id 'com.github.jk1.dependency-license-report' version Versions.gradle_license_report
    }
}

Yes, moving to 5.6 helps. I can now use Versions.kt for the version.
It would be nice if we were also able to use a const for the library name:
 id Paths.dependency-license-report version Versions.gradle_license_report

but this solves my main concern, thanks!

Good to know!

Yeah!
Just switched to Gradle 5.6.2 in my ever-lasting quest not to repeat myself.  My build.gradle files (I have a multi-project build) now contain id "com.github.voplex95.lesscompiler" and the top-level settings.gradle contains id "com.github.voplex95.lesscompiler" version Versions.com_github_voplex95_lesscompiler_gradle_plugin.
The next big step forward would be to replace the repeated "com.github.voplex95.lesscompiler".  This string is of course already contained in Libs.com_github_voplex95_lesscompiler_gradle_plugin. Perhaps it can be moved into a separate object like the Paths.com_github_voplex95_lesscompiler_gradle_plugin and then used in Libs.kt?
Actually, I have not yet found a use for the Libs constants for Gradle plugins.  Is there any?

Perhaps it can be moved into a separate object like the Paths.com_github_voplex95_lesscompiler_gradle_plugin and then used in Libs.kt?

Yes, a solution similar to this: https://github.com/jmfayard/buildSrcVersions/issues/61

Actually, I have not yet found a use for the Libs constants for Gradle plugins. Is there any?

I don't think so, but they are inclueded in the report.json that I use as a source.

If someone wants to work on this, I think we can copy the solution from here https://github.com/icerockdev/moko-mvvm

Hmm, I took a peek at that and can't see the connection.  Perhaps because I can't read Cyrillic?

They do this:
// settings.gradle.kts
pluginManagement {
    resolutionStrategy.eachPlugin {
        val module = Plugins.map[requested.id.id] ?: return@eachPlugin
        useModule(module)
    }
}

// buildSrc/src/main/kotlin/Plugins.kt

object Plugins {
    const val android =
        "com.android.tools.build:gradle:${Versions.Plugins.android}"
    const val kotlin =
        "org.jetbrains.kotlin:kotlin-gradle-plugin:${Versions.Plugins.kotlin}"
    const val androidExtensions =
        "org.jetbrains.kotlin:kotlin-android-extensions:${Versions.Plugins.androidExtensions}"

    val map: Map<String, String> = mapOf(
        "com.android.application" to Plugins.android,
        "com.android.library" to Plugins.android,
        "org.jetbrains.kotlin.multiplatform" to Plugins.kotlin,
        "kotlin-kapt" to Plugins.kotlin,
        "kotlin-android" to Plugins.kotlin,
        "kotlin-android-extensions" to Plugins.androidExtensions
    )
}

Here are my 2c.
gradle.properties:
com.gradle.plugin-publish.version=0.10.1

settings.gradle.kts:
fun PluginDependenciesSpec.idv(name: String) =
  id(name) version (extra["$name.version"] as String)

pluginManagement {
    plugins {
        idv("com.gradle.plugin-publish")
    }
}

idv stands for id+version
So it looks like a regular plugin declaration. The version is taken from gradle.properties.
Plugin version could be overridden via -P... and so on.

Hmm, I took a peek at that and can't see the connection. Perhaps because I can't read Cyrillic?

@nkiesel ,
The plugin-relevant bits there are
https://github.com/icerockdev/moko-mvvm/blob/2525a284783469981897a84c3569ec239f4049ef/buildSrc/src/main/kotlin/Deps.kt#L1-L8
and
https://github.com/icerockdev/moko-mvvm/blob/2525a284783469981897a84c3569ec239f4049ef/settings.gradle.kts#L17-L20

The gradle.properties is better because it works fine both for the buildSrc mode and the singleFileMode. Also it can be overwritten from the command-line as you note.

buildSrc access from settings.gradle is going to be completely removed in Gradle 6.0, so gradle.properties or some other mechanism is the only method that will continue working in the next release.

The good news is that I tried the propose solution with gradle.properties and it's very nice
https://github.com/jmfayard/buildSrcVersions/commit/76dce4a6e4aa102725eb8ab6b69a6397abaf73bd
The bad news is that I have no idea on how to find which plugins are applied to a project.
project.plugins.onEach { ... } only gives me the class of the plugin. I find no way to find the plugin id and the current plugin version.

ok, I found a way so I have a prototype for this feature now
https://github.com/jmfayard/buildSrcVersions/pull/71/files

buildSrc access from settings.gradle is going to be completely removed in Gradle 6.0, so gradle.properties or some other mechanism is the only method that will continue working in the next release.

Hmm, that seems unfortunate because pluginManagement.plugins in settings.gradle allows to use buildSrc variables as version numbers while the plugins in build.gradle only allows string literals.  I just consolidated my plugins versions in settings.gradle for that reason.  I trust that the Gradle team will come up with a proper solution.

ok, I found a way so I have a prototype for this feature now
https://github.com/jmfayard/buildSrcVersions/pull/71/files

Interesting. Couple of questions:

Seems that will always append the dependencies to gradle.properties but never remove existing dependencies.  Is that really the case?
I loved the fact that I could see available newer plugin versions in Versions.kt and wanted to be able to upgrade a plugin simply by editing that file, similar to normal jar dependencies.  But seems the proposed solution also does not allow this.  Or am I misunderstanding this approach?

@nkiesel

Correct, the rest of the gradle.properties file should/must be kept intact. If that's not the case, feel a ticket.
Adding the information of plugins versions: Yes! But it was 2am so I decided to go to bed :)

It's part of my todo-list "Polishing versionsOnlyMode" before I release 0.5.1
https://github.com/jmfayard/buildSrcVersions/issues/67

@nkiesel @tadfisher @tadfisher @sigurdurrafn
Check out the brand new $ ./gradlew :refreshVersions introduced in version 0.6.0
It generates gradle.properties with versions and available updates and from there you can set the plugins versions!
https://github.com/jmfayard/buildSrcVersions/issues/77


is_pyramid check has an integer overflow with large arrays

🐛 Bug
The pyramid size check uses np.prod which defaults to np.int32 data type and there is integer overflow when calculating size on images with dimension products greater than 2^32.
The function below has the current function in, but commented out.
def is_pyramid(data):
    """If shape of arrays along first axis is strictly decreasing.
    """
    #size = np.array([np.prod(d.shape) for d in data])
    size = np.array([np.prod(d.shape,dtype=np.uint64) for d in data])  #should be 64-bit integer

    if len(size) > 1:
        return np.all(size[:-1] > size[1:])
    else:
        return False

To Reproduce
import napari
import numpy as np

#pyramid of zeros
pyramid = [np.zeros((50000,50000,3),dtype=np.uint8),
 np.zeros((25000,25000,3),dtype=np.uint8),
np.zeros((12500,12500,3),dtype=np.uint8)]

size_default = np.array([np.prod(d.shape) for d in pyramid])

Out[43]: array([-1089934592,<PHONE_NUMBER>,   468750000])

size_uint64 = np.array([np.prod(d.shape,dtype=np.uint64) for d in pyramid])

Out[45]: array([7500000000,<PHONE_NUMBER>,  468750000], dtype=uint64)

with napari.gui_qt():
    # add image pyramid
    napari.view_image(pyramid, is_pyramid=True)

Traceback:
Traceback (most recent call last):

  File "<ipython-input-48-ce4a0e063e17>", line 3, in <module>
    napari.view_image(pyramid, is_pyramid=True)

  File "C:\anaconda3\envs\nap\lib\site-packages\napari\view_layers.py", line 120, in view_image
    path=path,

  File "C:\anaconda3\envs\nap\lib\site-packages\napari\components\viewer_model.py", line 509, in add_image
    visible=visible,

  File "C:\anaconda3\envs\nap\lib\site-packages\napari\layers\image\image.py", line 147, in __init__
    data, pyramid=is_pyramid, rgb=rgb

  File "C:\anaconda3\envs\nap\lib\site-packages\napari\util\misc.py", line 143, in get_pyramid_and_rgb
    init_shape = data.shape

AttributeError: 'list' object has no attribute 'shape'

Pyramidal checks are based on decreasing numbers of points per resolution. From the previous size estimate with 32 bit integers:
np.all(size_default[:-1] > size_default[1:])

Out[49]: False

np.all(size_uint64[:-1] > size_uint64[1:])

Out[51]: True

Expected behavior
Should load a pyramidal image.
Environment
napari, 0.2.5
Qt, 5.12.5
NumPy, 1.17.4
SciPy, 1.3.2
VisPy, 0.6.2
scikit-image, 0.16.2
Platform: Windows-10-10.0.10240-SP0
Python:   3.7.3 | packaged by conda-forge | (default, Jul  1 2019, 22:01:29) [MSC v.1900 64 bit (AMD64)]
Backend:  PyQt5
GL version:  '4.4.0 - Build <IP_ADDRESS>31'
MAX_TEXTURE_SIZE: 16384
Additional context

Oops, good catch @NHPatterson!
We'd happily accept that fix as a PR if you don't mind making it, otherwise I can take care of it.

Ok, I'll submit a pull request.

Thanks!


dwi description seems wrong: T2w instead of T2*w

here it says that "Diffusion-weighted imaging contrast (specialized T2* weighting)."
but given that DWI is based on a spin echo (vs. gradient echo) EPI, I would call it a "specialized T2 weighting" instead.

Pinging @oesteban, who might be able to weigh in on this.
If it is something that needs to be changed, we can do that in the suffixes schema file:
https://github.com/bids-standard/bids-specification/blob/30c61b7eb473cccd2414d3947d38487995720f67/src/schema/objects/suffixes.yaml#L426-L429


ANTLR version mismatch

Task :validateOwnership FAILED
ANTLR Tool version 4.7.2 used for code generation does not match the current runtime version 4.5.3ANTLR Runtime version 4.7.2 used for parser compilation does not match the current runtime version 4.5.3

Caused by: java.lang.NoClassDefFoundError: Could not initialize class org.tomlj.internal.TomlLexer
at org.tomlj.Parser.parse(Parser.java:32)
at org.tomlj.Toml.parse(Toml.java:56)
at org.tomlj.Toml.parse(Toml.java:44)
at se.premex.ValidateOwnershipTask.validationTask(ValidateOwnershipTask.kt:47)

Probably two separate issues - first seems to be merely a warning. Second is that tomlj isn't available on the plugin classpath.

@erikeelde can you test with latest release and see if this is still an issue?


render error with unique values

If all values are the same the output is broken.
An example would be
$linechart = new Linechart();
echo $linechart->addMarkers([1,1,1])->chart();

I know this is an edge case, but I am using the graph to show temperature over time. If the temperature is not changing, the graph is broken and looks like this:
     1┤
  ───

it should render like this:
     1┤───

Somehow the line number calculated in \noximo\PHPColoredAsciiLinechart\Linechart::processLinearGraph (Line 417) is off by one. The result has two rows, even though there is only one line in the graph.
Setting the height is also not helping, it will reproduce the same:
$linechart = new Linechart();
$settings = new Settings();
$settings->setHeight(5);
echo $linechart->setSettings($settings)->addMarkers([1,1,1])->chart();

Thanks for the report, should be fixed now.

It works now. Great.
But with zeros, it will throw an error:
$linechart = new Linechart();
echo $linechart->addMarkers([0,0,0])->chart();

results in
In Linechart.php line 396:
  Warning: Division by zero


Make FakePathlibModule.Path compare equal to pathlib.Path

Sometimes it happens that a test case compares a fake path to a real path - usually when you've created a pathlib.Path in pytest.mark.parametrize, like so:
def my_function():
    return pathlib.Path()

@pytest.mark.parametrize('expected_result', [pathlib.Path()])
def test_my_function(fs, expected_result):
    assert my_function() == expected_result

This fails with AssertionError: assert WindowsPath('.') == WindowsPath('.').
This could of course be avoided by passing in a string and converting it to a path inside the test, but in reality there isn't always such a convenient workaround. Here's a more realistic example, where an object with loads of attributes - some of which are Paths - is loaded from disk and compared to the expected result:
import pytest
from pathlib import Path
from my_module import MyClass

@pytest.mark.parametrize('attrs', [
    {'foo': False,
     'origin': Path(),
     'bar': 17,
     'paths': [Path('/'), Path('somewhere')]},
])
def test_serialization(fs, attrs):
    obj = MyClass(**attrs)
    obj.dump('myobj.dump')

    loaded_obj = MyClass.load_file('myobj.dump')
    assert vars(loaded_obj) == attrs

As I'm sure you can see, having to convert all of those attributes to Paths inside the test is rather annoying. For this reason it would be very convenient if fake Paths would compare equal to real Paths.

Thanks for the report! I see your problem, an I think there is an easy fix (overwriting __eq__) for this as long as the comparison is this way around, e.g. comparing the fake path against a real path. Would that be sufficient for you?

Hmm, I think that would probably be sufficient, yes. I'm not sure if it's a good idea to have an asymmetrical __eq__ though. Might cause some confusing errors. Wouldn't it be better to go all the way and monkeypatch pathlib.Path.__eq__?

Well, the question is where to do this. The actual problem is that the pytest.parametrize decorator is read before any patching occurs - this is one of the instances that are mentioned in the documentation, where patching won't work. I agree that overwriting __eq__ is not the best way to handle this. The corrrect way would be to ensure that the pathlib used in the parameter is patched - I will think about this...

For now, I've worked around the problem by adding this monkey patch to my conftest.py:
# monkeypatch FakePath.__eq__ so that they compare equal to real pathlib.Paths
from pathlib import Path
from pyfakefs.fake_pathlib import FakePath

# Path somehow gets monkeypatched during testing, so in order to have access
# to the original class we'll simply create an instance of it
PATH = object.__new__(Path)

def path_eq(self, other):
    Path = type(PATH)

    if isinstance(other, (Path, FakePath)):
        return str(self) == str(other)

    return super(Path, self).__eq__(other)

Path.__eq__ = path_eq

Yes, this is a valid solution, at least for your case. I have been thinking about adding the same monkeypatching to the pytest plugin, but I'm somewhat reluctant to do this, as generally there could be other non-patched fs calls in a parametrize decorator. I will have another look some time later.

Getting back to this, I now think that the correct way to resolve this would be to reload the sut module during the test (for example using modules_to_reload in the test setup), as this is the only way that ensures that the code that is read at load time is correctly patched -- see the documentation for how to do this with pytest.
Actually, the fixture in the documentation is too complicated, it could look like this:
import pytest
from pyfakefs.fake_filesystem_unittest import Patcher
import sut

@pytest.fixture
def fs_reload_sut():
    with Patcher(modules_to_reload=[sut]) as p:
        yield p.fs

@jmcgeheeiv - instead of closing the issue, I would like to to convert this (and other issues that may be helpful) into a Q/A discussion issue. If you agree, could you enable discussions in the pyfakefs configuration?

Great idea, @mrbean-bremen. Done.


Yusuf gaming the beast

[Enter feedback here]

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

ID: e7031e14-f8ab-1d70-6910-72565048b2c1
Version Independent ID: 23e13d47-a99e-452a-4891-af3147995842
Content: Creating and managing apps - Visual Studio App Center
Content Source: docs/dashboard/creating-and-managing-apps.md
Service: vs-appcenter
Sub-service: app-center-general
GitHub Login: @blparr
Microsoft Alias: blparr

Please make easy update system thanks

Please make easy update to not west time and energy thanks


Do_Abi_* event/callback handlers should wrap invocations with RoReportUnhandledError

Describe the bug
The legacy C# WinRT projection support for UWP apps wrapped all synchronous calls back into C# code with a try/catch, calling RoOriginateLanguageException and then RoReportUnhandledError on any unhandled exceptions, ensuring they were then routed to the App.UnhandledException handler.  This behavior is missing from C#/WinRT projections, causing regressions in Microsoft.Windows.SDK.NET.dll (e.g., Window.Activated) and Microsoft.WinUI.dll (most everything else).  Thus, fixes might be required across several repos.
To Reproduce
For an override callback:

Clone WinUI3_UnhandledExceptionTestAppd
Build, set a BP in OnLaunched and in the App.UnhandledException handler, and debug
When the OnLaunched BP hits, set next statement to throw the exception

For an event:

Clone WinUI3_UnhandledExceptionTestAppd
Build, set a BP in the MainWindow's Activated handler and in the App.UnhandledException handler, and debug
When the Activated handler BP hits, set next statement to throw the exception

Expected behavior
In either case above, callback or event, the App.UnhandledException handler should be invoked as it is in a UWP app.
Workarounds
There are a couple workarounds:

User callbacks can be marked 'async' (whether or not they contain any awaits).  This causes the CLR to wrap invocations in a try/catch, propagating any caught exceptions through the DispatcherQueueSynchronizationContext.
this.Activated += **async** (s, e) => { ... };
User callback code can itself be wrapped in a try/catch to propagate errors:

this.Activated += (s, e) => {
  try
  {
    // ...
  }
  catch (System.Exception e)
  {
    ExceptionHelpers.ReportUnhandledError(e);
  }
}

This issue is better addressed lower in the stack, in Microsoft.UI.Xaml.dll.  Fixing under original issue: https://github.com/microsoft/microsoft-ui-xaml/issues/5221

:tada:Microsoft.WindowsAppSDK.1.2.221209.1 has been released which fixes this issue.
Handy links:

Release Notes
Official Nuget Release


feat: Add survey

Fixes #

Thank you for opening this PR! Please answer our quick survey:

How confident are you in this change? (React with 👍 for high, 👎 for low)
Does this PR require extensive testing? (React with ✅ for yes, ❌ for no)

Your feedback helps us improve our process. Thanks!


CLI doesnt connect to the database via --db-url

Bug report
Describe the bug
I'm running a self-hosted version of supabase. My postgres container exposes its port at <IP_ADDRESS>:5433 but the CLI throws a "connection refused" error when I'm trying to generate types.
To Reproduce
supabase gen types typescript --debug --db-url postgresql://postgres:12345@<IP_ADDRESS>:5433/postgres --schema public > database.types.ts

# ...
Error: connect ECONNREFUSED <IP_ADDRESS>:5433
    at file:///usr/src/app/dist/server/app.js:67:19
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
# ...

To verify the postgres instance is reachable:
docker run --rm -it --network host postgres psql --host <IP_ADDRESS> -p 5433 -U postgre

Password for user postgres:
psql (15.1 (Debian 15.1-1.pgdg110+1), server 14.6 (Debian 14.6-1.pgdg110+1))
Type "help" for help.

postgres=> \dt
             List of relations
 Schema |  Name   | Type  |     Owner
--------+---------+-------+----------------
 public | profile | table | supabase_admin
 public | role    | table | supabase_admin
(2 rows)

postgres=>

Expected behavior
CLI connects to the database and generates types
System information

OS: macOS Ventura 13.1
Version of supabase-js: 1.41.1
Version of Node.js: 18.14.1

I suspect the problem is related to docker networking.

Hi @vacavaca, typegen runs within a docker container so you will need to replace <IP_ADDRESS> with docker.internal.host to reach your host network.

Error: Connection terminated due to connection timeout
    at file:///usr/src/app/dist/server/app.js:67:19
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Tryied "docker.internal.host", "<IP_ADDRESS>"
Here is my docker inspect for the database container, the networks section:
"NetworkSettings": {
            "Bridge": "",
            "SandboxID": "567adfddb37a334ecde4838642015f6e1685c30835c4c0307071adb14cf2fa10",
            "HairpinMode": false,
            "LinkLocalIPv6Address": "",
            "LinkLocalIPv6PrefixLen": 0,
            "Ports": {
                "5432/tcp": [
                    {
                        "HostIp": "<IP_ADDRESS>",
                        "HostPort": "5433"
                    }
                ]
            },
            "SandboxKey": "/var/run/docker/netns/567adfddb37a",
            "SecondaryIPAddresses": null,
            "SecondaryIPv6Addresses": null,
            "EndpointID": "",
            "Gateway": "",
            "GlobalIPv6Address": "",
            "GlobalIPv6PrefixLen": 0,
            "IPAddress": "",
            "IPPrefixLen": 0,
            "IPv6Gateway": "",
            "MacAddress": "",
            "Networks": {
                "local_app": {
                    "IPAMConfig": null,
                    "Links": null,
                    "Aliases": [
                        "supabase-db",
                        "db",
                        "1282b1c41494"
                    ],
                    "NetworkID": "bcc59753c37835bd844e3761dce191d32cee60fde27d865f5c075af4ff57c119",
                    "EndpointID": "eef01dcf3a80231100867b6ab6250b0fa079120f0929374c698f122951b5155c",
                    "Gateway": "<IP_ADDRESS>",
                    "IPAddress": "<IP_ADDRESS>",
                    "IPPrefixLen": 16,
                    "IPv6Gateway": "",
                    "GlobalIPv6Address": "",
                    "GlobalIPv6PrefixLen": 0,
                    "MacAddress": "02:42:ac:15:00:02",
                    "DriverOpts": null
                }
            }
        }

"host.docker.internal" works.
However, it was a bit surprising that a "CLI" application uses some arbitrary bridge network (the default docker bridge) and not the "host" network (lo). For example, when I run psql or redis-cli or some nodejs script, I expect it to reach every network endpoint that I can reach by running curl in the same terminal. But apparently, supabase-cli requires a very specific configuration to reach the database. Maybe it would be better to run all the containers in the "host" networking mode to make the CLI work more conventionally. What do you think?

It's probably sufficient to run the one-off pgmeta container in host network only when --db-url flag is passed to typegen. That would solve for your use case of behaving similarly to psql client.
I'm not sure if there's a strong use case for other containers to be on the host network too.

I agree, using the host networking only with --db-url would be enough.


Remove existing slider?

Hi @quarterto
This is an awesome plugin, thank you!   Just wondering how would you remove an existing slider from elessar bar?  To use your hour example, if I want to remove an slider from it.  How would it works?

See #22. If you set allowDelete: true in the options, you can double-middle-click to remove a range. Alternatively you can implement your own event handler and call rangebar.removeRange(range).


Where is pirated version of Steam.dll?

In README.md it is described that to launch a pirate server, you need to replace 3 files from the folder with the client, but there is no Steam.dll file there. Where can I get it?

I apologize for the long absence.
The answer to the question: download the client of the original game, there will be Steam.dll.


Switch focus to going back to truly open source, rather than a foundation

See the "avoiding war" thread for context.
This is a different approach / request that we can consider. I'm not sure if this is actually better, so we should discuss, and pick whether we like this approach or the one that's live now.

Not only do I appreciate these edits, but I appreciate you calling out that this isn’t to start a war. <3

+1 from Digger - foundation has no value in and of itself, it serves a purpose - to achieve what? to keep terraform with community. So cutting through that link directly makes sense.

I just merged the latest from main into this branch.


autopep8 not working

Bug: Notebook Editor, Interactive Window, Python Editor cells

Steps to cause the bug to occur
1.Use "Format Document" on open file "HelloWorld.py"
Actual & Expected behavior
Expected to reformat with autopep8
Actual: error: Extension 'Python Language Basics' cannot format 'HelloWorld.py'

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Extension version:                               2019.10.44104
VS Code version: Code 1.39.2 (6ab5985, 2019-10-15T15:35:18.241Z)
OS version: Windows_NT x64 10.0.17134
Virtual environment:                          conda | venv | virtualenv | N/A | ...

Developer Tools Console Output

Info Python Extension: 2019-10-24 09:32:35: > ~\AppData\Local\Programs\Python\Python38-32\python.exe -m pylint --disable=all --enable=F,unreachable,duplicate-key,unnecessary-semicolon,global-variable-not-assigned,unused-variable,unused-wildcard-import,binary-op-exception,bad-format-string,anomalous-backslash-in-string,bad-open-mode,E0001,E0011,E0012,E0100,E0101,E0102,E0103,E0104,E0105,E0107,E0108,E0110,E0111,E0112,E0113,E0114,E0115,E0116,E0117,E0118,E0202,E0203,E0211,E0213,E0236,E0237,E0238,E0239,E0240,E0241,E0301,E0302,E0303,E0401,E0402,E0601,E0602,E0603,E0604,E0611,E0632,E0633,E0701,E0702,E0703,E0704,E0710,E0711,E0712,E1003,E1101,E1102,E1111,E1120,E1121,E1123,E1124,E1125,E1126,E1127,E1128,E1129,E1130,E1131,E1132,E1133,E1134,E1135,E1136,E1137,E1138,E1139,E1200,E1201,E1205,E1206,E1300,E1301,E1302,E1303,E1304,E1305,E1306,E1310,E1700,E1701 --msg-template='{line},{column},{category},{symbol}:{msg}' --reports=n --output-format=text c:\Users\tjh36\Documents\PythonProjects\HelloWorld.py

My apologies, I am not a professional developer, just trying to learn Python.  Watched a few videos about setting up VS Code with Python, tried everything and re-installed twice.  No good.

@TimHicks3000 that's fine, we're here to help. However you might also want to file similar (getting started) issues on stack overflow.

I've posted it to StackOverflow here: https://stackoverflow.com/questions/58546617/vs-code-python-ext-autopep8-not-formatting

I fixed it.

De-install Python Extension from Code
De-install Code
De-install Python
Install Code
Install extension
Install Python, don't take quick install, go through custom for all users, and that PATH is updated.
pip install autopep8 and check the scripts folder, that autopep8 is there and the PATH to the scripts folder is right.  Try "autopep8" at a command line to see that it finds it.
Add Python interpreter in Code.

Thank you.