Error while running fisher

I have no problem in installing. But after that, when I try to run fisher, an error shows:
fish: Could not expand string “' function okay(s) { printf("'"$okay"'%s'"$nc"' %s\n", "OK", s) } function info(s) { printf("%s\n", s) } function error(s) {
printf("'"$error"'%s'"$nc"' %s\n", "!", s) } { sub(/^[ ]+/, "") gsub("``", " ") if (/&[^&]+&/) { n = match($0, /&[^&]+&/) if (n) { sub(/&[^&]+&/, "'"$$log"'" substr($0, RSTART +
1, RLENGTH - 2) "'"$nc"'", $0) } } s[++len] = $0 } END { for (i = 1; i <= len; i++) { if ((i == 1 || i == len) && (s[i] == "")) { continue } if (s[i] == "") { print } else {
'"$log"'(s[i]) } } } '”
/home/m161047/.config/fish/functions/fisher.fish (line 1367):     printf "%s\n" "$message" | command awk '
                                                                                ^
in function “__fisher_log”,
	called on line 21 of file “/home/m161047/.config/fish/functions/fisher.fish”,
	with parameter list “error You need fish &2.2.0& or higher to use fisherman.”

in function “fisher”,
	called on standard input,

command: Unknown option “brew”
/home/m161047/.config/fish/functions/fisher.fish (line 23):             if command -s brew > /dev/null
                                                                        ^
in function “fisher”,
	called on standard input,

       command - command - run a program

command - run a program
   Synopsis
       command COMMANDNAME [OPTIONS...]

   Description
       command forces the shell to execute the program COMMANDNAME and ignore
       any functions or builtins with the same name.

   Example
       command ls causes fish to execute the ls program, even if an ’ls’
       function exists.

fish: Could not expand string “' function okay(s) { printf("'"$okay"'%s'"$nc"' %s\n", "OK", s) } function info(s) { printf("%s\n", s) } function error(s) {
printf("'"$error"'%s'"$nc"' %s\n", "!", s) } { sub(/^[ ]+/, "") gsub("``", " ") if (/&[^&]+&/) { n = match($0, /&[^&]+&/) if (n) { sub(/&[^&]+&/, "'"$$log"'" substr($0, RSTART +
1, RLENGTH - 2) "'"$nc"'", $0) } } s[++len] = $0 } END { for (i = 1; i <= len; i++) { if ((i == 1 || i == len) && (s[i] == "")) { continue } if (s[i] == "") { print } else {
'"$log"'(s[i]) } } } '”
/home/m161047/.config/fish/functions/fisher.fish (line 1367):     printf "%s\n" "$message" | command awk '
                                                                                ^
in function “__fisher_log”,
	called on line 26 of file “/home/m161047/.config/fish/functions/fisher.fish”,
	with parameter list “info
                    Refer to your package manager documentation for
                    instructions on how to upgrade your fish build.
                ”

in function “fisher”,
	called on standard input,

My fish version:
$ fish -v
fish, version 2.1.1

Hi @pwwang! You only need to upgrade to fish 2.3.1.

Hi @bucaran, I upgraded fish to 2.3.1:
$ fish -v
fish, version 2.3.1

but it didn't help.
Here is my OS information:
$ lsb_release -a
LSB Version:	:base-4.0-amd64:base-4.0-noarch:core-4.0-amd64:core-4.0-noarch:graphics-4.0-amd64:graphics-4.0-noarch:printing-4.0-amd64:printing-4.0-noarch
Distributor ID:	CentOS
Description:	CentOS release 6.8 (Final)
Release:	6.8
Codename:	Final

@bucaran My bad, it worked. I should stop all old fishes that are running and rerun the new fish.


OWLS-86816 - Fix to generate domain processing event only when there domain updates

OWLS-86816 - Operator generates domain processing events every time domain presence is rechecked even when there are no domain updates. This happens in cases where cluster's replica count is lower than the max cluster size. In such cases, serversToStop list in the ManagedServersUpStep contains servers that should not be started. Later in processing, the ServerDown step checks if the server pod is null and invokes DeletePod step only when the server pod is not null.
This fix removes the servers that are already stopped (servers with null pod) from serversToStop list and it also checks whether server only needs it's service pre-created. In such cases, the domain processing events are not generated.

Please don't forget to include unit test

Please don't forget to include unit test

Please don't forget to include unit test

@alai8 @doxiao I'm trying to write a unit test for this. However, it seems domain processing event are being generated in two places (once in DomainIntrospectorJobStep and once in ManagedServerUpStep). So even if ManagedServerUpStep doesn't generate domain processing event, unit test still passes. Additionally, I'm seeing that even when same set of events (with same reason etc.) is generated twice, the testSupport.getResources(KubernetesTestSupport.EVENT) still return only one set of events. Not sure if I'm missing something. I'll keep looking into it but wanted to let you know.

Please don't forget to include unit test

@alai8 @doxiao I'm trying to write a unit test for this. However, it seems domain processing event are being generated in two places (once in DomainIntrospectorJobStep and once in ManagedServerUpStep). So even if ManagedServerUpStep doesn't generate domain processing event, unit test still passes. Additionally, I'm seeing that even when same set of events (with same reason etc.) is generated twice, the testSupport.getResources(KubernetesTestSupport.EVENT) still return only one set of events. Not sure if I'm missing something. I'll keep looking into it but wanted to let you know.
@ankedia The operator should not generate multiple Starting event in the same round of domain processing even though you see create event steps are put into the step chain in different parts of the workflow. There is a logic in EventHelper to skip duplicated createEventStep.

Please don't forget to include unit test

@alai8 @doxiao I'm trying to write a unit test for this. However, it seems domain processing event are being generated in two places (once in DomainIntrospectorJobStep and once in ManagedServerUpStep). So even if ManagedServerUpStep doesn't generate domain processing event, unit test still passes. Additionally, I'm seeing that even when same set of events (with same reason etc.) is generated twice, the testSupport.getResources(KubernetesTestSupport.EVENT) still return only one set of events. Not sure if I'm missing something. I'll keep looking into it but wanted to let you know.
@ankedia The operator should not generate multiple Starting event in the same round of domain processing even though you see create event steps are put into the step chain in different parts of the workflow. There is a logic in EventHelper to skip duplicated createEventStep.

Please don't forget to include unit test
@ankedia The operator should not generate multiple Starting event in the same round of domain processing even though you see create event steps are put into the step chain in different parts of the workflow. There is a logic in EventHelper to skip duplicated createEventStep.

Added unit tests in c26deeb.

Please don't forget to include unit test
@ankedia The operator should not generate multiple Starting event in the same round of domain processing even though you see create event steps are put into the step chain in different parts of the workflow. There is a logic in EventHelper to skip duplicated createEventStep.

Added unit tests in c26deeb.


annotate incompletely sequenced/assembled regions and filter blacklisted regions during prediction

Reference Issues/PRs
This pull request mainly focuses on improving annotation or filtering incomplete or blacklisted regions of the genome. We also addressed some issues related to prediction according to internal discussions.
What does this implement/fix? Explain your changes.
New:

Add check for whether a input sequence includes UNK_BASE (e.g "N" for DNA sequence). If true, then output a warning and write True to the "contains_unk" column in an output file.
For vcf and bed, blacklisted region variants/intervals (if specified) are now directed to .NA files and do not get predicted.

Changes / fixes:

Add a header row to the row_labels output file (when hdf5 format output is specified)
Now determine whether the input file is fasta or bed based on the file extension (prevent generating empty fai files on non-fasta input)
Open hdf5 output file with 'w' instead of 'a', so now the output hdf5 file is always truncated to zero length first if the file exists.
Open .NA output file with 'w' instead of 'a', so now the output .NA file is always truncated to zero length first if the file exists.

What testing did you do to verify the changes in this PR?
Ran prediction for a few small fasta, bed, and vcf files with both regions with “N”s and regions without “N”s.
Made predictions for a few large bed and vcf files with and without blacklisted regions specified.

Thanks for looking over this @kathyxchen ! This commit should address these issues and I have rerun the tests.

Yes I have finished. You can merge now.


add post about modelling inheritance

I came up with this idea when starting work on a codegen library.  It's adjacent to something I'm using in that implementation to work out whether a schema is supported.

A few comments...
Do you know about Device Tree used in Linux?
https://www.devicetree.org
https://github.com/devicetree-org/dt-schema
It occurs to me that it could be possible to prevent schemas which are abstract classes from being used directly...
If you used a dynamic reference in the abstract class schema, which referenced a definition of isNotAbstractClass or something, the schema value could be false. Doing so means that the abstract class schema always fails validation... until a derrived schema defines a dynamicAnchor of the same name with a schema value of true or empty object.
It would add another step, but it could help prevent mistakes. But then, simply naming the schema "Abstract..." could be enough to help prevent the same mistakes.
Do you think it's worth mentioning that if you want to have schemas built at build time, you could do that sort of thing using Jsonnet? Details would be out of scope of the article, but people could dig if they wanted to.

It looks like Device Tree are using their own modified version of draft-07, where the meta-schema is defined in 2019-09... so I'm not sure what exactly is going on 😅
https://github.com/devicetree-org/dt-schema/blob/main/dtschema/meta-schemas/base.yaml

I looked at using dynamic references, but couldn't get it all working right.  What you suggest is similar to what I used to describe generic types.
I think leaving them out is simpler anyway.  It might be better to start simple and maybe have a follow-up post for advanced cases.

Do you know about Device Tree used in Linux?

I had not heard of this.  Devices just presented more of a "property" inheritance opportunity, whereas most polymorphism examples use a "function" inheritance, e.g. Animal defines a .MakeSound() function that both Cat and Dog implement differently.  But in JSON Schema, we don't care about functions; we care about properties.

There's one place where I don't think this example correctly reflects how polymorphism works. Your ideal schema for the Peripheral class includes additionalProperties: false. However, polymorphism actually depends on additional properties being allowed. - @jdesrosiers

I included it because I see people put additionalProperties on abstract classes in order to avoid creating an instance of an abstract class, which is something that languages like C# will prevent you from doing.  You can't create just a Peripheral; you can only create a Mouse or a Keyboard.  So the thought is that by adding additionalProperties to the abstract class it prevents someone from satisfying the schema:peripheral constraints and just tacking on whatever else they feel like.
What they inevitably run into is that "inheriting" the schema doesn't overwrite the additionalProperties constraint.
I go on after the example to explain why this approach is wrong, and it becomes my first concession.

I'm going to leave the TS classes.  It may not be the common way to do it in TS, but it still illustrates what's going on, and I'd like to have the snippets be more consumable than just for people who know C#.


Undo accidental comment changes from 0c1c05b

The dangers of non-semantically-aware find-and-replace...

Thanks!


Improve the plugins block parser to handle version() and apply()

Including infix functions syntax
This PR is on top of https://github.com/gradle/gradle/pull/22435

@bot-gradle test AFT

OK, I've already triggered the following builds for you:

AllFunctionalTestsPullRequestFeedback build

@bot-gradle test AFT

OK, I've already triggered the following builds for you:

AllFunctionalTestsPullRequestFeedback build


Add ACCESS_NETWORK_STATE permission

Add ACCESS_NETWORK_STATE permission because of getActiveNetworkInfo. More at https://developer.android.com/reference/android/net/ConnectivityManager.html#getActiveNetworkInfo()

This is already added.


Add sample code on how to update a plot3d widget from a thread

This PR adds an example on how to update a 3D scatter plot from a thread.
There was already such example for 1D/2D plots, this is adding one for 3D data.
This PR also fixes an issue when there is items in the scene but not containing any data (as initially in the sample code).

LGTM


Parse tags in profile description

Is your feature request related to a problem? Please describe.

Tags (hashtags) in profile descriptions (BIOs) are not clickable.

Describe the solution you'd like

Parse tags and highlight them just like links and mentions.
When a tag is clicked, it should behave the same way as when you click a tag in a post (Show menu).

Links and mentions in profile descriptions are already highlighted, so it would be natural to display tags as well.
Describe alternatives you've considered

I've considered opening an issue in atproto. Currently profile descriptions are stored as a single string, but I think it's more secure to store profile description facets on the server.
But links and mentions are currently parsed on the client, so I guess it's best to handle tags similarly.
Additional context
Related:

https://github.com/bluesky-social/atproto/issues/1227
#848
#2968

Likely it's because it's using a different RichText component atm, so even if descriptionFacets were to be added it still wouldn't show the tags, it is being parsed however.

@mary-ext
It might be because displaying tags with the RichText component is disabled by default right now.
I'll see if I can take a crack at it today.


Run PCA memory allocation errors

Hello everyone!
I'm trying to run the fastsparseGRM PCA step. I've tried before with many samples and hadn't any problems.
I create the bed file with the following command:
plink --bfile $experiment_path/seq2bed/chrall --maf 0.05 --mac 5 --make-bed --out $experiment_path/seq2bed/chrall_non_monomorphic
When running a larger cohort of 3100 samples with 7M SNPs, I get this error:

R version 4.2.0 (2022-04-22) -- "Vigorous Calisthenics"
Copyright (C) 2022 The R Foundation for Statistical Computing
Platform: x86_64-pc-linux-gnu (64-bit)

R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.

  Natural language support but running in an English locale

R is a collaborative project with many contributors.
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.

Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.
....

Matrix products: default
BLAS/LAPACK: /data/apps/extern/spack_on/gcc/9.3.0/openblas/0.3.23-htztuye3zvg76xhteew36pn2wqt3dogc/lib/libopenblas_skylakex-r0.3.23.so

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] FastSparseGRM_1.01 optparse_1.7.3

loaded via a namespace (and not attached):
 [1] mitml_0.4-5            Rcpp_1.0.10            mvtnorm_1.2-2
 [4] lattice_0.21-8         tidyr_1.3.0            formula.tools_1.7.1
 [7] Biostrings_2.66.0      glmnet_4.1-8           foreach_1.5.2
[10] utf8_1.2.3             R6_2.5.1               GenomeInfoDb_1.34.9
[13] pan_1.9                backports_1.4.1        jomo_2.7-6
[16] gdsfmt_1.34.1          stats4_4.2.0           pillar_1.9.0
[19] zlibbioc_1.44.0        rlang_1.1.1            minqa_1.2.5
[22] data.table_1.14.8      nloptr_<IP_ADDRESS>00      S4Vectors_0.36.2
[25] rpart_4.1.19           Matrix_1.6-1.1         splines_4.2.0
[28] lme4_1.1-34            GWASExactHW_1.01       igraph_1.5.0
[31] RCurl_1.98-1.12        broom_1.0.5            compiler_4.2.0
[34] pkgconfig_2.0.3        BiocGenerics_0.44.0    shape_1.4.6
[37] mgcv_1.8-42            nnet_7.3-19            tidyselect_1.2.0
[40] tibble_3.2.1           GenomeInfoDbData_1.2.9 IRanges_2.32.0
[43] codetools_0.2-19       fansi_1.0.4            crayon_1.5.2
[46] dplyr_1.1.2            MASS_7.3-60            bitops_1.0-7
[49] grid_4.2.0             xtable_1.8-4           nlme_3.1-162
[52] lifecycle_1.0.3        magrittr_2.0.3         SeqVarTools_1.36.0
[55] estimability_1.4.1     cli_3.6.1              XVector_0.38.0
[58] getopt_1.20.4          SNPRelate_1.32.2       mice_3.16.0
[61] generics_0.1.3         vctrs_0.6.3            boot_1.3-28.1
[64] iterators_1.0.14       tools_4.2.0            GMMAT_1.4.1
[67] CompQuadForm_1.4.3     Biobase_2.58.0         glue_1.6.2
[70] purrr_1.0.1            emmeans_1.8.6          parallel_4.2.0
[73] survival_3.5-5         SeqArray_1.38.0        GenomicRanges_1.50.2
[76] operator.tools_1.6.3   logistf_1.26.0
> runPCA(prefix.in,file.unrels,no_pcs,num_threads,prefix.out,no_iter)
[1] "PCA started at 2024-04-13 12:26:35"
[1] "270 relateds, 2830 unrelateds, and 7380307 variants"
Reading the file: /scratch4/seq2bed/chrall_non_monomorphic.bed
Reading BED file complete
Number of SNPs read: 7380307
Number of samples: 3100
Using 22 threads
Processing samples complete
Allele frequencies calculated
[1] "Reading Completed in 21 seconds"
[1] "1 iterations completed"
[1] "2 iterations completed"
[1] "3 iterations completed"
[1] "4 iterations completed"
[1] "5 iterations completed"
[1] "6 iterations completed"
[1] "7 iterations completed"
[1] "8 iterations completed"
[1] "9 iterations completed"
[1] "10 iterations completed"
[1] "Iterations Completed in 2129 seconds"
Error: cannot create std::vector larger than max_size()
Execution halted

I'm running the script on a slurm cluster with 1.5TB RAM and 48 cores.

Hi,
The max_size() of std::vector is determined by the CPU architecture. RAM memory does not matter here. Are you running the code on a 32-bit CPU or 64-bit CPU? I am guessing 32-bit.
7 million is a lot of variants to be used for PCA. I will suggest reducing the number of variants if the error persists. Again, increasing RAM memory is not going to solve this. This is a limitation of the CPU.
Best,
Rounak

Hi Rounak,
I'm using 64-bit architecture which make me question why max_size() is being an error. As you can see on the R code, Im using Platform: x86_64-pc-linux-gnu (64-bit), that is why I think this might be a bug
I pruned my variants to 2 million in total, which ceased the symptom of memory error

I see. Glad it got resolved. 2 million is already more than enough variants for PCA.


Remove named volume mounts

Could you give a bit more clarification on the thought process of this PR, the motivation? Why would we do this? I mean it looks good, but really I cannot tell which way is the better way.

The reason is that until now I did everything by the book and used named volumes, and for development reasons I also mounted such named volumes to filesystem. The mount though only happens with absolute paths, and such absolute path is only working out-of-the-box within Vagrant.
Ideally to keep everything close to a production system we still should use named volumes, and the non-vagrant users should modify the paths at the end of the docker-composer file. This way, to make it production ready one can only remove the mount in driver_opts and uncomment driver=local line.
This change only removes the use of named volumes, to use relative paths.
What are your thoughts?

I believe this is fixed on dev by now, can you verify @linuxbandit ?


User UUID should use IDIR guid instead of JWT sub

The JWT subject is an id that is specific to the keycloak realm.
Using IDIR guid instead to identify users would:

make an eventual transition to a different keycloak realm, or away from keycloak, easier
allow us to restore prod data in a test environment for QA purposes without having to deal with users with different UUIDs but duplicate emails

@matthieu-foucault @pbastia How will this affect authentication for external (non-government) users? Will we have to continue to use sub for them?

I'm wondering if we could use the BCeID as an alternative, that siteminder supports? I've been recently made aware that anyone can get a basic one instantly https://www.bceid.ca/aboutbceid/ even if they're not a BC resident

Usage of BCeID for CIF applications will need to be discussed. The use case for the basic BCeID is "when accessing a service in a personal capacity", which does not seem to be not the case here. A business BCeID might be needed instead.
For the scope of this issue, the user UUID should probably be coalesced to sub, to ensure that there is always a UUID (will be required when using our test accounts in development)


Mark MsgModuleQuerySafe requests field as non-nullable

Randomly bumped into it, the requests field on the query safe message is required to be populated in the Validation fn. There's no benefit to keeping it as a pointer.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Hi @DimitrisJim, could I take this issue?


Issues with RDNS

Hello!
Not sure if I am doing something wrong or if this is a actual bug but I can't seem to set RDNS with this library.
When I try it for IPv4 I get a stacktrace:
>>> l.ips.ipv4.public[0].rdns = 'my-rdns'
>>> l.ips.ipv4.public[0].save()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/private/tmp/kaka/lib/python2.7/site-packages/linode/objects/base.py", line 144, in save
    data=self._serialize())
  File "/private/tmp/kaka/lib/python2.7/site-packages/linode/linode_client.py", line 430, in put
    return self._api_call(*args, method=requests.put, **kwargs)
  File "/private/tmp/kaka/lib/python2.7/site-packages/linode/linode_client.py", line 375, in _api_call
    endpoint = endpoint.format(**{ k: str(vars(model)[k]) for k in vars(model) if 'id' in k })
KeyError: 'address'
>>> print l.ips.ipv4.public[0].address
my.ip.addr

I also tried sending save() to the linode itself (i.e: l.save()), I don't get a stacktrace but the RDNS isn't updated.
For IPv6 I don't seem to see a updatable property for RDNS or a save function.

Thanks, I'll take a look

This fix has been released in v4.1.3b1


Check for character e to support scientific notation

Adds the capability to support float numbers Scientific Notation such as 3e-5.

Thanks! This should close #12
Fwiw, you might want to consider using boto3's dynamodb.types.TypeSerializer and TypeDeserialzer classes as those are officially maintained by AWS as part of the SDK. That said – at a glance it doesn't look like they support non-integer numbers somehow.


Where to find the benchmarks?

Hi,
With the release notes, some benchmarks are shown (e.g. v24.10).
I would like to know if the code of these benchmarks is available publicly and, if so, where to find them.
Best,
Ruben Laso

Hi Ruben,
all benchmarks are available publicly.
cloverleaf and minibude are miniapps from the University of Bristol and can be found on github.com/uob-hpc/cloverleaf and github.com/uob-hpc/minibude.
There are also publications about those two apps, which you can find using the search engine of your choice.
nbody, FFT, easywave, XSBench` are from HeCBench (https://github.com/zjin-lcf/HeCBench), which however only collects benchmarks from various other sources. The original XSBench repository is https://github.com/ANL-CESAR/XSBench.
toybrot is from here: https://gitlab.com/VileLasagna/toyBrot (raymarched directory). Run with ACPP_PERSISTENT_RUNTIME=1 to prevent unnecessary runtime restarts for every iteration due to the current design of the SYCL version of the benchmark.

Thanks so much for the detailed response!
I'll try to run some of these benchmarks in the next few days.

You're welcome :)


Failed Build React App

When executing my project with the "yarn start" command, after adding any react library, the library reports the following warning and the project does not execute:

The previous solution for this issue is to comment all lines in the month-picker.scss file that contains "@include". Would this problem have a definitive solution?

please try v2.1.1. I just removed all mixins defined in bourbon library


Remove projects from nav and merge content to resources

This PR removes the Projects page from navigation and merges the content of the projects into the Resources page
Fixes #415

I think as many core team members should likely review this PR tagging some folks for review.

Thanks, for approving the changes.


Ganache client hanging, think it may be related to having a large number of log messages

Description
I am having a problem with v 1.2.2 of the Ganache client.  It appears to hang presenting only a blank white screen in the client area of the app and is completely unresponsive to RPC requests.  I think it may be related to log message area having excess content.  If I enable verbose logs it can happen as quickly as within an hour or two of having the client app running.  If I disable verbose logs it takes a day or two or longer.
My only recourse at this point is to close the app.  Fortunately the x mini-button on the app's title bar still works and I can close the app.  I then restart the app.
--

I asked about this on gitter: [x]
Link to my question on gitter:

https://gitter.im/ConsenSys/truffle?at=5bd4aea56e5a401c2ddd362d
Steps to Reproduce
Just leave the client app running.
Your Environment

Version used: 1.2.2
Version of Truffle/Remix/Other tools used: n/a
Operating System and version (include distro if Linux): Ubuntu Linux 18.04

I have the same problem that @roschler is describing on macOS 10.14. It's not new in v 1.2.2 of the Ganache client. Is there a way to either disable logging entirely or clear the logs on an interval?

Thanks for reporting this! I've moved this to https://github.com/trufflesuite/ganache/issues/947 as this issue is specific to the Ganache UI. The current Redux/React structure does cause noticeable delays/hanging for large amounts of RPC calls. While logging is a Redux action, this is probably just one contributor to the delay. I'm going to continue this discussion over on that issue (I think yall were mentioned.


Fix for Coverity analysis issues

Running a Coverity scan on this project yielded two possible issues which I have resolved:

Big parameters passed by value
Uninitialized scalar fields

Also please note,
The signature of one of the methods (TestRegistry::LexSort) has changed, if this method represents a part of Catch's public API, then we should consider incrementing the major version as per semantic versioning.

@Tarnasa I just noticed that this pull request is to the master branch.  It might need to be made to the develop branch.


Tooltip.updateTitleContent() and default options.innerSelector don't work well together

CodePen demo
https://codepen.io/anon/pen/WgRoZp?editors=1111
Steps to reproduce the problem

Notice that all 4 tooltips start out with "default title", as they should
Hover over all 4 li elements
Notice that only 1 tooltip has updated to "updated title"

What is the expected behavior?
All 4 should have updated.
What went wrong?
The code you use in your _updateTitleContent (in index.js) function looks like:
  _updateTitleContent(title) {
    if(typeof this._tooltipNode === 'undefined') {
      if(typeof this.options.title !== 'undefined') {
        this.options.title = title;
      }
      return;
    }
    const titleNode = this._tooltipNode.parentNode.querySelector(this.options.innerSelector);
    this._clearTitleContent(titleNode, this.options.html, this.reference.getAttribute('title') || this.options.title)
    this._addTitleContent(this.reference, title, this.options.html, titleNode);
    this.options.title = title;
    this.popperInstance.update();
  }

Why do you use .parentNode? I think that should be removed. I would think the innerSelector should be scoped so it selects descendants of the root tooltip element. But, by using parentNode, you select from the next parent up instead (in my case, parentNode is the ul element). This causes the innerSelector to find too many matches, and an arbitrary single match is chosen by querySelector(), causing only a single tooltip update.
Also, note that this is only a problem when you have more than 1 tooltip with the same parent, like I do with my ul and 4 li tooltip children.
Any other comments?
This is default config, and it's broken. I don't think telling people to choose a more distinct/selective innerSelector would be a good solution to this. It would work, but pushes tedious bookkeeping onto the client to create unique ids and stick them into the innerSelector.
Off topic - thanks for the great work on this lib!

Good point, the idea was to allow to have arrow elements outside of the tooltip, but I didn't consider this case.
Please feel free to send a PR to get rid of the .parentNode bit.

I've just run into this issue as well! I'd love to see the PR merged.

Good point, the idea was to allow to have arrow elements outside of the tooltip, but I didn't consider this case.
Please feel free to send a PR to get rid of the .parentNode bit.

if container set body,this problem still exists.

@FezVrasta It looks like this has been merged?
If so, could you maybe release a new release on NPM with the fix?

@FezVrasta we would also be interested in an updated release on NPM with this fix.

I already released the fix


no type named 'uint_fast16_t' in namespace 'std' (Ubuntu + Clang)

Building with Clang 18 on Ubuntu 24, I'm getting the following build error:
/include/cpprealm/internal/bridge/realm.hpp:77:35: error: no type named 'uint_fast16_t' in namespace 'std'; did you mean simply 'uint_fast16_t'?
   77 |                 using port_type = std::uint_fast16_t;
      |                                   ^~~~~~~~~~~~~~~~~~
      |                                   uint_fast16_t
/usr/include/stdint.h:62:27: note: 'uint_fast16_t' declared here
   62 | typedef unsigned long int       uint_fast16_t;
      |                                 ^

@adamski Had the same issue.
This patch fixes it it's just a missing: #include <cstdint> in include/cpprealm/internal/bridge/realm.hpp
missing-cstdint.patch
Tested on Ubuntu 24.04 with:
gcc (Ubuntu 12.3.0-17ubuntu1) 12.3.0

Thanks @BestITUserEUW !
@leemaguire any chance of getting this patch merged in?

@adamski I'll try merge it in this week.

@leemaguire There is also another error when building example with gcc. I can try to fix it and submit a pr next few days.


Graphs in toolbar not update if "Start hidden" enabled

Good day
If enable Options >>  General >> "Start hidden" and restart System Informer, then graphs in toolbar will not be updated.
At the same time graphs in "System Information" window update correctly.
System Informer v3.0.7578
Microsoft Windows 10 Enterprise LTSC / 1809 / 64-bit (Build 17763.5696)

Unable to reproduce. Try renaming/resetting the application settings?

Unable to reproduce. Try renaming/resetting the application settings?

Deleting current config (settings.xml).
Start System Informer >> enabling "Start hidden" and 7 graphs on toolbar.
Restarting System Informer and see that graphs not updating.

I also couldn't reproduce the problem on the same Windows version and build, with approximately the same set of software, but different hardware/drivers.
Any ideas how to debug such problem? Is that even possible?


max-len ignoreComments

    /**
     * Determines whether two tokens can safely be placed next to each other without merging into a single token
     * @param {Token|string} leftValue The left token. If this is a string, it will be tokenized and the last token will be used.
     * @param {Token|string} rightValue The right token. If this is a string, it will be tokenized and the first token will be used.
     * @returns {boolean} If the tokens cannot be safely placed next to each other, returns `false`. If the tokens can be placed
     * next to each other, behavior is undefined (although it should return `true` in most cases).
     */

error:
  1261:1  error  Line 1261 exceeds the maximum line length of 128  max-len
  1262:1  error  Line 1262 exceeds the maximum line length of 128  max-len

Thanks for the PR. Actually, I've been thinking it would be better to just disable the max-len rule entirely.

Thanks!


Microsoft.ServiceFabric.Client.Http v4.9.0: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>

Describe the bug
Microsoft.ServiceFabric.Client.Http v4.9.0 is compiled against Newtonsoft.Json v<IP_ADDRESS>; but the nuget package refererences Newtonsoft.Json v<IP_ADDRESS>. So any builds/packages will include 10.0, but will result in a runtime failure like:
Unhandled exception: System.IO.FileNotFoundException: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'. The system cannot find the file specified.

File name: 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'
   at Microsoft.ServiceFabric.Client.Http.ClusterClient.GetClusterHealthAsync(Nullable`1 nodesHealthStateFilter, Nullable`1 applicationsHealthStateFilter, Nullable`1 eventsHealthStateFilter, Nullable`1 excludeHealthStatistics, Nullable`1 includeSystemApplicationHealthStatistics, Nullable`1 serverTimeout, CancellationToken cancellationToken)

To Reproduce
Using asp.net core 7.0, build any sample executable that uses Microsoft.ServiceFabric.Client.Http v4.9.0 and calls a ServiceFabric cluster.  Package the executable and run it.
Expected behavior
Not this
Additional context
Here's the nuspec from the nupkg:
<?xml version="1.0" encoding="utf-8"?>
<package xmlns="http://schemas.microsoft.com/packaging/2013/05/nuspec.xsd">
  <metadata>
    <id>Microsoft.ServiceFabric.Client.Http</id>
    <version>4.9.0</version>
...
    <dependencies>
      <dependency id="Newtonsoft.Json" version="10.0.3" />
    </dependencies>
  </metadata>
</package>

Here's a screenshot from dotpeek:

The workaround is to add a reference to Newtonsoft.Json 13.x for any projects using Microsoft.SErviceFabric.Client.Http.

@johncrim thank you for reporting this. we will get a new version released to fix this

@johncrim  we have released 4.10.1, which should fix the dependency issue.

Verified that the issue is fixed in 4.10.1.


Support Descriptor Interactions

Problem
Currently, it is only possible to read all Characteristic Descriptors upon discovery. However, it is not possible to read/write descriptors within the lifetime of a connected device.
Solution
Provide access to read/write descriptors within the BluetoothPeripheral actor and find a respective translation for the DSL language (e.g., a @Characteristic equivalent?).
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct and Contributing Guidelines

Maybe we could implement Descriptor support as another property wrapper usable in BluetoothService types with a syntax similar to the code snippet below.
public struct DeviceInformationService: BluetoothService, Sendable {
    public static let id: BTUUID = "180A"

    @Characteristic(id: "2A29")
    public var manufacturerName: String?

    @Descriptor(id: "2901", for: \Self.$manufacturerName)
    public var manufacturerNameDescription: String?

    // ...

    public init() {}
}

The property wrapper @Descriptor works similar to the @Characteristic with additionally requiring a KeyPath to the CharacteristicAccessor binding of the characteristic it refers to. This design would however require a runtime check that the KeyPath refers to a characteristic declared on the same Service type (in this case DeviceInformationService) as we can't express this relationship through the Swift type system.
The code snippet below roughly sketches the public interface of the Descriptor property wrapper. Note that we would likely want shorthand initializers for the common descriptor UUIDs (see CBDescriptor).
@propertyWrapper
public struct Descriptor<Value>: Sendable {
    public var wrappedValue: Value? {
        fatalError("Needs implementation")
    }

    public var projectedValue: DescriptorAccessor<Value> {
        fatalError("Needs implementation") // read and write accesses for the descriptor
    }

    public init<S: BluetoothService, V>(wrappedValue: Value? = nil, id: BTUUID, for characteristic: KeyPath<S, CharacteristicAccessor<V>>) {
        fatalError("Needs implementation")
    }
}


fix: OverlayPlane module and Palette Color LUT issues

This fixes OverlayPlane module usage for ArrayBuffer, BulkDataURI, and InlineBinary cases.
Also fixes PaletteColor LUTs for ArrayBuffer (i.e. local drag/drop) case.
These were slightly broken after the recent metadata overhaul #1460.
PR Checklist

[x] Brief description of changes
[x] Links to any relevant issues
[ ] Required status checks are passing
[ ] User cases if changes impact the user's experience
[x] @mention a maintainer to request a review

LGTM


Explore GitHub

branch
wiki
.gitignore
gist
repo management

wiki

Projects with great wikis
Example: Titan wiki
Footer
Sidebar
Images to wikis
Adding and editing wiki pages locally
Adding wiki pages via the online interface

.gitignore
added following two lines in .gitignore

*.p
*.csv

Can't git add/commit .p/.csv files in Terminal anymore.
Allow to upload .p/.csv files directly in github webpage.

repo management

Minimal Structure
Repository Structure and Python

gist
like google drive, share scripts or data in any format. public or private. new repo.

About gist
gist

Great Github Tutorial (by Justin, CUSP)
@kayzhou22 @xinshizheng strongly recommend!


Adding handler for definitions and internal $refs.

This simple change populates the schemaMap with a copy of a definition where ever one is referred to with a $ref.

Hi Omri,
could you add an example to index.html showing the new functionality?
Thanks!

Done.
Also, my fork now has a handler for patternProperties too, as well as greying out for elements marked readOnly. (basic types are greyed out, and array types marked as "readOnly" have the addButton and removeButtons greyed out. Not sure how or whether to do the same with objects and one's.

Hi Omri, excuse the delay, I have been very busy lately... First of all thank you for your contributions!
I am working on it now

Hi Omri, I've just merged your PR.
I did some minor refactoring so you should merge your code to align ourselves.
The readonly implementation (http://brutusin.org/json-forms/#12) seems to be unfinished. Shouldn't the input values be also disabled? I think so.
I open a new issue for it
https://github.com/brutusin/json-forms/issues/26


Remove Runner ID from build information

Runner ID is no longer very useful as it just either indicates Orka or Linux 1/2, so lets remove it.

Hold on a sec

TBH, I think it'd be easier and safer to redo this and just limit the change to the build monitor page.


Fix libaio1 docker dependency and rename Dockerfile-ubuntu to Dockerfile-debian

The current Dockerfile based on openjdk:8 does not build due to an unavailable version of libaio1 specified. In addition, the Dockerfile states it is Ubuntu, when in fact openjdk:8 is now Debian.
I removed the hardcoded 0.3.110-3 libaio1 version, renamed the Dockerfile and updated the scripts and readme.md to indicate what changed.

awesome work...
only thing. it would be nice if you could create a JIRA.. and squash all these commits into one explaining all the changes... associated with the JIRA
ARTEMIS-XXX Docker Improvements

explain your changes here...

However, let me know if you can't do it.. and I will do it myself while merging your changes.

Thank you! I have created a Jira ticket, squashes the commits and created a new PR.

@zekeoptimo you didn't need to create a new PR.. push -f would have done it..
thanks!

@clebertsuconic thanks! Yeah, sorry about that, I wasn't familiar with squashing the commits and such - thanks for the tip, that makes sense.
Question: for the Jira ticket I created, do I close that myself ?
Lastly, I discovered another bug in the Dockerfiles. The ANONYMOUS_LOGIN environment var is never used and the result is that anonymous access is always enabled. I'm going to file another ticket for that and get another PR going once I've tested everything - I also have another simple change that will give more control over how the cluster is created that I'd like to add.


Mastodon is rejecting our signatures for www.jvt.me

@jamietanna congrats and condolences, your site managed to hit a domain-specific bug that we're not seeing with other users. Mastodon is rejecting our HTTP Signatures for AP activities that we send on your behalf, but it's accepting the same activities for other users. Example, BF replying to @Tymscar's Follow (https://twitter.com/Tymscar/status/1589550831036403713) with Accept: https://fed.brid.gy/log?key=https://fosstodon.org/c129199f-c63d-4a13-8f8f-997e5f8c8984&start_time=1667768647
I 2022-11-06 21:04:53.112473+00:00 new Follower for jvt.me https://fosstodon.org/users/Tymscar
I 2022-11-06 21:04:53.237011+00:00 Sending AP request from jvt.me: {
  "@context": "https://www.w3.org/ns/activitystreams",
  "id": "tag:fed.brid.gy:accept/jvt.me/https://fosstodon.org/c129199f-c63d-4a13-8f8f-997e5f8c8984",
  "type": "Accept",
  "actor": "https://fed.brid.gy/www.jvt.me",
  "object": {
    "type": "Follow",
    "actor": "https://fosstodon.org/users/Tymscar",
    "object": "https://fed.brid.gy/www.jvt.me"
  }
}
I 2022-11-06 21:04:53.645525+00:00 requests.post https://fosstodon.org/users/Tymscar/inbox {'gateway': True, 'data': b'{"@context":"https://www.w3.org/ns/activitystreams","id":"tag:fed.brid.gy:accept/jvt.me/https://fosstodon.org/c129199f-c63d-4a13-8f8f-997e5f8c8984","type":"Accept","actor":"https://fed.brid.gy/www.jvt.me","object":{"type":"Follow","actor":"https://fosstodon.org/users/Tymscar","object":"https://fed.brid.gy/www.jvt.me"}}', 'auth': <httpsig.requests_auth.HTTPSignatureAuth object at 0x3e03a24f4f70>, 'headers': {'Content-Type': '...', 'Date': '...', 'Digest': '...', 'Host': '...'}}
I 2022-11-06 21:05:04.589976+00:00 Received 401: Verification failed for<EMAIL_ADDRESS>https://fed.brid.gy/www.jvt.me using rsa-sha256 (RSASSA-PKCS1-v1_5 with SHA-256)

Interestingly, this might be Follow/Accept specific. Mastodon has accepted at least one of our signatures for a www.jvt.me activity: https://fed.brid.gy/log?key=https%3A%2F%2Fwww.jvt.me%2Fmf2%2F2022%2F11%2Fbdk3t%2F+https%3A%2F%2Findieweb.social%2F%40tchambers%2F109243684867780200&start_time=1667671212
I 2022-11-05 17:59:26.777305+00:00 Sending AP request from www.jvt.me: {
  "id": "https://fed.brid.gy/r/https://www.jvt.me/mf2/2022/11/bdk3t/",
  "published": "2022-11-05T14:39:47Z",
  "summary": "Liked a post on indieweb.social by\u00a0Tim Chambers\u00a0 Post detailsWelcome to the 22 new users to this instance TONIGHT, and & for many of you, welcome to the #Mastodon & the #Fediverse. Promise this will be different than Twitter for those of you migrating on the \u2026(https://indieweb.social/@tchambers/109243684867780200)",
  "url": "https://fed.brid.gy/r/https://www.jvt.me/mf2/2022/11/bdk3t/",
  "@context": "https://www.w3.org/ns/activitystreams",
  "type": "Like",
  "object": "https://indieweb.social/users/tchambers/statuses/109243684867780200",
  "..."
}
I 2022-11-05 17:59:27.056943+00:00 requests.post https://indieweb.social/users/tchambers/inbox {'gateway': True, 'data': b'{"id":"https://fed.brid.gy/r/https://www.jvt.me/mf2/2022/11/bdk3t/","published":"2022-11-05T14:39:47Z","summary":"Liked a post on indieweb.social by\\u00a0Tim Chambers\\u00a0 Post detailsWelcome to the 22 new users to this instance TONIGHT, and & for many of you, welcome to the #Mastodon & the #Fediverse. Promise this will be different than Twitter for those of you migrating on the \\u2026(https://indieweb.social/@tchambers/109243684867780200)","url":"https://fed.brid.gy/r/https://www.jvt.me/mf2/2022/11/bdk3t/","actor":{"url":"https://fed.brid.gy/r/https://www.jvt.me","type":"Person","name":"Jamie Tanna","image":[{"url":"https://www.jvt.me/img/profile.png","type":"Image","name":"Jamie Tanna\'s profile image"}],"icon":[{"url":"https://www.jvt.me/img/profile.png","type":"Image","name":"Jamie Tanna\'s profile image"}],"id":"https://fed.brid.gy/www.jvt.me","preferredUsername":"www.jvt.me"},"@context":"https://www.w3.org/ns/activitystreams","type":"Like","object":"https://indieweb.social/users/tchambers/statuses/109243684867780200","cc":["https://www.w3.org/ns/activitystreams#Public","https://indieweb.social/users/tchambers","https://indieweb.social/users/tchambers/followers"]}', 'auth': <httpsig.requests_auth.HTTPSignatureAuth object at 0x3e5655aaa190>, 'headers': {'Content-Type': '...', 'Date': '...', 'Digest': '...', 'Host': '...'}}
I 2022-11-05 17:59:37.310606+00:00 Received 202

Is this related to Bridgy Fed constructing the HTTP signature, or the other Mastodon server?
If it's Bridgy Fed, it may be related to an issue I've seen at $previousJob that may explain why it's just me seeing it seo far.
The issue was where the querystring that is given to AWS Lambda (which I'm using for my redirects) isn't the same as the querystring that's being signed, so the solution we had was to make sure that the signature generation always uses an alphabetically-sorted set of querystring params, rather than in the order they come in.
May not actually be the case, but thought I'd share!

Thanks for the ideas! There's no query string in these inbox URLs, and the JSON body is serialized before it's hashed and passed to the HTTP request. More importantly, Mastodon is accepting our signatures for other Accepts and our signature for your Create activity, examples above.
The tag is possible, but I also suspect this:
https://github.com/snarfed/bridgy-fed/blob/cfb6653d77e0b1c0c507eae18e4a333b4164e6cc/activitypub.py#L76
Ideally, I'd always drop www., for all users, everywhere, but I'd need to think through the UX first.

Would that mean I'd need to ask people to follow<EMAIL_ADDRESS>in that case?

Yes, but there may be valid UX needs for some people who want www vs non-www. No decisions or changes here yet on my part, initial step here is just to get www.jvt.me working as is.

Fixed! Will attach the commit here soon.

Woop, I can see in the indieweb.social UI that I've been able to follow 👏🏽


Port metrics-servlets to an HttpService

This is the display side of #35.  MetricsServlet is the immediate need, and other things in AdminServlet are pretty neat.
I think maybe this can go right into the http4s-server package, since it already depends on metrics-core for the MetricsSupport that is emerging in #35.
If you're working on this, please assign to self (committers) or comment (new contributors).

metrics have been moved into the metrics-server module.

This has existed for a long time.


Add an Option to Redownload Winetricks

This commit fixes #4.
It also places the TUI app in a while loop and moves the quick launch code behind an if GUI check.
Depends on #29.

I need to figure out how to place my comments so the correct lines are previewed with them. Sorry about the mismatches above. They're easier to follow if you open the "Files Changed" tab.
Maybe we need to think through the logic of main a little bit more at a bigger-picture level. The CLI options in some cases preselect choices that would be found later in the TUI or GUI (e.g. SKIP_FONTS), but in other cases they completely override the need for the UI (e.g. --run-winetricks). This is why I added the ACTIONS variable. It served to short-circuit the UIs when this kind of non-interface "action" was requested from the CLI. So my knee-jerk reaction is to say "just put it back", but then that would mess up being able to put the TUI menu into a loop, which is a good idea.
But maybe the if config.DIALOG is None or config.DIALOG == 'tk': block (L197-L201) could be moved up and out of the if config.GUI is False block, along with the "actions" block (L167-L180), then the if config.GUI is True|False parts could be reverted while the curses block could just start with:
``
`if config.DIALOG == 'curses':
while True:
[...]
This was an `elif` statement originally. Would something like that work?

Thanks for the review. I will rebase and then try to fix up what you've shared.

Rebased. Now need to review and fix up your comments.

@n8marti, this should be ready for a new review.

Also, perhaps we can do a voice style to rethink the optargs/logic of main. I will open a discussion for this.

I think this PR is dependent on the outcome of #34, unless the scope is reduced to only add the new --get-winetricks and --run-winetricks options but not yet implement them.

I think this PR is dependent on the outcome of #34, unless the scope is reduced to only add the new --get-winetricks and --run-winetricks options but not yet implement them.

Yes, agreed. I will push the syntax fix, and then we can discuss how we want to split optargs that are for runtime config and optargs that are actionable.

Squashed and pushed these modifications.
I will merge this change, then we can merge the reinstall dependencies code, then we can discuss how best to handle the variable optargs and the action optargs.


Event URI throws server error

http://data.carnegiehall.org/events/58948/about
Throws: Server Error (500)

Re-submitting under https://github.com/CarnegieHall/ch-lod-app


Port of NonBufferedBody from Slim 3.x

Original PR: https://github.com/slimphp/Slim/pull/2504
Linked Issue: https://github.com/slimphp/Slim-Psr7/issues/24

I realized that in Slim 3 I forgot about withAddedHeader() and withoutHeader(), as they are "hidden" in the base Message class (whereas here the three of them are there).
I'll also open a corrective PR there.


writers.makePythonWriter: fix interpreter executable

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

rebuilds due to
+ /nix/store/hj4ad4avnlmma2jcanvhhy3gk5fxyk05-replace-workspace-values.drv:{out}
      • The environments do not match:
          interpreter=''
          -/nix/store/qrygidrcibdg2yk4yiqws3320bb2xfqi-python3-3.10.12-env/bin/python
          +/nix/store/qrygidrcibdg2yk4yiqws3320bb2xfqi-python3-3.10.12-env/bin/python3.10
      ''


adminserver，停止board后，adminserver回到输入uuid页面

In GitLab by @cuiaq123 on Apr 30, 2020, 07:39

In GitLab by @cuiaq123 on Apr 30, 2020, 07:50

建议考虑用户管理放到adminserver里，adminserver的用户不能依赖于board

In GitLab by @JackZxj on Apr 30, 2020, 08:33
assigned to @luotian-github and unassigned @JackZxj

In GitLab by @luotian-github on Apr 30, 2020, 12:34
assigned to @JackZxj and unassigned @luotian-github

In GitLab by @cuiaq123 on May 6, 2020, 03:00
建议考虑用户管理放到adminserver里，adminserver的用户不能依赖于board

In GitLab by @JackZxj on May 6, 2020, 03:16
It's what we designed. You can access our wiki for more info. @cuiaq123
As for user management, @luotian-github should make a decision.

In GitLab by @JackZxj on May 6, 2020, 03:17
assigned to @luotian-github and unassigned @JackZxj

In GitLab by @luotian-github on Jun 24, 2020, 02:23
next release


This seems broken.

No button is visible for me anymore. Was working yesterday. Waterfox Classic v 2020.01

You can try the current beta version (<IP_ADDRESS>).
See issue #86. You can use it by getting it from my fork (method explained). Tampermonkey will handle updates for you.
Or you can copy the code from the comments in that issue.

Did it not use to be an overlay in the lower left of each image with its dimensions?
And there's no longer any option to search for alternative sizes.
Has Google changed something the last few days so this has disappeared or have I gone full senile?

@psonnosp I didn't noticed that. But yes, google has apparently removed the size overlay. But not from every versions of its layout.
From what I can see in the source code, there is a possibility to bring it back. Until google breaks it again...   ;)
I'm testing a stylesheet to load into stylus. When ready I can post it here if you want.
Concerning the search for alternative sizes, this has been remove months ago. There is another extension/userscript fixing this. I've also been participating there.

Yes, please do that, thanks.
The thing with it not being removed from every version has to be the case with alternative sizes to then cause I know I haven't been missing it for months. It's one of the things I use the most so kind of sure I would of get tired of it earlier if that was the case.
But that doesn't matter, where can I find the script that fix it?

The userscript/extension is google-image-sizer. (It's in my list of repositories)
It has been updated just a couple of days ago, and still works.
With versions of google images that still display the sizes, you'll notice a difference. Daniel has reversed the behavior of sizes captions (to match an old GIS behavior?): the size are always displayed, and hidden on mouse over.
As for the userstyle, here is my take. It's in userstyle format. You have to load it in stylus, or an extension which understand it.
How to install it in stylus:

open manager
check the as Usercss checkbox next to Write new style
click Write new style
select all (ctrl+a) the default code, and paste the code below
save

In stylus context menu on google image pages, you'll see a gear symbol next to the stylesheet name. It opens a dialog to configure this stylesheet with the behavior you like best.
If you check on change at the bottom of that dialog, change will be applied immediately.
Don't let the editor opened at the same time unless Live preview is disabled on that page.
If it works for you, I will create a repository to host the file, and to keep users automatically updated.
/* ==UserStyle==
@name           google image search - size back
@namespace      github.com/Procyon-b
@version        0.1.0
@description    ...
@author         Achernar
@preprocessor stylus

@var checkbox active  'Show size' 1
@var checkbox Always  'Always show / Show on mouse over' 1
@var checkbox hover   '  hide on mouse (if always show)' 1
@var checkbox top     'Display at the top' 0
==/UserStyle== */
@-moz-document regexp("^https:\\/\\/www.google\\..*?\\/search\\??tbm=isch.*") {

if active {

  div[data-ow]::before, div[data-ow]::after {
    display: none;
    z-index: 9999;
    padding: 6px;
    color: #70757a;
    font: normal 11px arial,sans-serif;
    background-color: rgba(255, 255, 255, 0.9);
  }

  if top {
    div[data-ow]::before {
      content: attr(data-ow) " × " attr(data-oh);
      position: absolute;
      border-radius: 0 0 6px 0;
    }

  }
  if top ==0 {
    div[data-ow]::after {
      content: attr(data-ow) " × " attr(data-oh);
      position: relative;
      top: -6.3em;
      border-radius: 0 6px 0 0;
   }
  }

  if Always {
    div[data-ow]::after, div[data-ow]::before {
      display: inline-block;
    }
    if hover {
      div[data-ow]:hover::after, div[data-ow]:hover::before {
        display: none;
      }
    }
  }
  if Always == 0 {
    div[data-ow]:hover::after, div[data-ow]:hover::before {
      display: inline-block;
    }
  }

}
/* to prevent the userstyle from disappearing from stylus menu when options are disabled */
.none {
  dummy:1;
}
}

@psonnosp Is it working ?

Sorry I've been a bit of lately.
No idea what stylus is and google just showed me a lot of pens :D So don't know what to do with that.
As of the first script the only thing I found different was that it added more options under Size in search tools and that wasn't really what I was looking for.
I was talking about the Search by Image-button that used to be located together with the View Image button that you see under the image when you click on a search result so it open to the right on the screen.

Stylus is an extension that handles userstyles. Just like tampermonkey handles userscripts.
The format used above is enhanced in a way that allows stylus to modify the stylesheet based on options (defined in the stylesheet) you have selected.

Search by Image-button
You're right. I completely forgot about it.
I don't use it that often.
I'll look into this.

New version available in issue #95
And stylus for FF (56.0.2 if you're still using this) is here (if you ever want to test).

CSS updated.
2 versions available. Simpler, more compatible, non-configurable version below.

stylus version:
/* ==UserStyle==
@name           google image search - size back
@namespace      github.com/Procyon-b
@version        0.2.0
@description    ...
@author         Achernar
@preprocessor stylus

@var checkbox active  'Show size' 1
@var checkbox Always  'Always show / Show on mouse over' 1
@var checkbox hover   '  hide on mouse (if always show)' 1
@var checkbox top     'Display at the top' 0
==/UserStyle== */
@-moz-document regexp("https:\\/\\/www.google\\..*?\\/search\\?.*?tbm=isch.*") {

if active {

  div[data-ow]::before, div[data-ow]::after {
    display: none;
    z-index: 9999;
    padding: 6px;
    color: #70757a;
    font: normal 11px arial,sans-serif;
    background-color: rgba(255, 255, 255, 0.9);
  }

  if top {
    div[data-ow]::before {
      content: attr(data-ow) " × " attr(data-oh);
      position: absolute;
      border-radius: 0 0 6px 0;
    }

  }
  if top ==0 {
    div[data-ow]::after {
      content: attr(data-ow) " × " attr(data-oh);
      position: relative;
      top: -6.3em;
      border-radius: 0 6px 0 0;
    }
    [role="list"] > div[data-ow]::after {
      margin-top: -10em;
      margin-bottom: -10em;
      top: -4.6em;
    }
  }

  if Always {
    div[data-ow]::after, div[data-ow]::before {
      display: inline-block;
    }
    if hover {
      div[data-ow]:hover::after, div[data-ow]:hover::before {
        display: none;
      }
    }
  }
  if Always == 0 {
    div[data-ow]:hover::after, div[data-ow]:hover::before {
      display: inline-block;
    }
  }

}
/* to prevent the userstyle from disappearing from stylus menu when options are disabled */
.none {
  dummy:1;
}
}

simpler CSS version:
@-moz-document regexp("https:\\/\\/www.google\\..*?\\/search\\?.*?tbm=isch.*") {

/* ==UserStyle==
@name           google image search - size back
@namespace      github.com/Procyon-b
@version        0.2.0
@description    ...
@author         Achernar
==/UserStyle== */

div[data-ow]::after {
  display: none;
  z-index: 9999;
  padding: 6px;
  color: #70757a;
  font: normal 11px arial,sans-serif;
  background-color: rgba(255, 255, 255, 0.9);
}

div[data-ow]::after {
  content: attr(data-ow) " × " attr(data-oh);
  position: relative;
  top: -6.3em;
  border-radius: 0 6px 0 0;
}
[role="list"] > div[data-ow]::after {
  margin-top: -10em;
  margin-bottom: -10em;
  top: -4.6em;
}
div[data-ow]::after, div[data-ow]::before {
  display: inline-block;
}
div[data-ow]:hover::after, div[data-ow]:hover::before {
  display: none;
}

}

Thanks you so much for the help with this, much appreciated! 👍

Have you tried the stylesheet with stylus, or only the last version of the script?

Both of them and no issues to report so far.
So once again, thank you.


Bump go 1.19

Codecov Report
Base: 60.08% // Head: 60.01% // Decreases project coverage by -0.07% :warning:

Coverage data is based on head (706ea1f) compared to base (d1005d7).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
- Coverage   60.08%   60.01%   -0.08%
==========================================
  Files          28       28
  Lines        1656     1653       -3
==========================================
- Hits          995      992       -3
  Misses        562      562
  Partials       99       99

Impacted Files
Coverage Δ

internal/bot/server.go
42.13% <100.00%> (ø)

internal/channels/messages/messages.go
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Improved usability by allowing label tap

Thank's for your suggestion but I don't think that the labels are supposed to be clickable, only the floating action buttons.

I would also appreciate this feature!

Actually, nvm I just implemented this lol


Add some more flexible usage

using BB_WWW_PLUGINS instead of BB_WWW so that we can set and install needed plugins.
using BB_CHECKOUT to set the branch or tag to checkout.
There is a little problem on heroku that install and compiling the base or md_base plugin is too slow, considering adding support for pre-compiled pack.
compiling base or md_base on dokku is fast. maybe it is the limit of CPU or storage of heroku caused the slow of building.

Do you know that wheels of the packages are available there:
ftp://ftp.buildbot.net/pub/latest/
I think this is a better option than trying to rebuild everything each time
matabuildbot is building those, you can make a pr to enable autogeneration of base-md:
https://github.com/buildbot/metabbotcfg/blob/master/builders.py

Also, I think an easier option to install buildbot deps is to use the standard requirements.txt.
Here is how I did it for buildbot_travis:
https://github.com/tardyp/buildbot_travis_bin
With this method, it is not necessary to maintain a special buildpack, the application is just a plain python app.

@tardyp I use this buildpack to deploy live demo, so sometimes install from git repo may be necessary.
For common users installing from wheels is a better choice. I will add the support in the future.

yes, of course. You could also give an option to give the wheel as part of the app. (that's what I did for bb travis)


allow access to k8s secrets via app environment variables

I'ld like to be able for my app to be able to access specified k8s secrets. Perhaps this could be done at "deis create" or perhaps with a Procfile extension. Not sure what the best approach would be.

Right now doing confit:set creates a per version secret. What are you trying to achieve?

Just passing in S3, Slack creds/tokens, etc

You could do that using config:set if you want that per app. However you'd be able to see the values doing a config:list so they aren't fully secret but at least use the kubernetes primitivies (secrets)

Actually, that will work for the short term. Thanks!

Actually, that will work for me. Thanks


Error Faraday::ResourceNotFound (NOT_FOUND: The requested resource does not exist):

I am creating an integration to salesforce where I want to push leads to salesforce. Spent over an hour figuring out restforce and I am stuck with the error, for every call I make
Faraday::ResourceNotFound (NOT_FOUND: The requested resource does not exist):
the code i tried is
 client = Restforce.new(:oauth_token => request.env['omniauth.auth']['credentials']['token'], :instance_url => request.env['omniauth.auth']['credentials']['instance_url'])
 client.user_info
 client.query("select count() from leads")

The app is build in rails4
I am doing the authentication with omniauth for oauth
can someone help ?

When you're making a SOQL query, Salesforce uses singular names for the
resources, e.g.
SELECT COUNT() FROM Lead

Let us know how you get on!
On Thu, Feb 16, 2017 at 8:12 PM sojan v jose<EMAIL_ADDRESS>wrote:

I am create an integration to salesforce where i will be pushing leads to
salesforce. Spent over an hour figuring out restforce and I am stuck with
the error, for every call I make
Faraday::ResourceNotFound (NOT_FOUND: The requested resource does not
exist):
the code i tried is
client = Restforce.new(:oauth_token => request.env['omniauth.auth']['credentials']['token'], :instance_url => request.env['omniauth.auth']['credentials']['instance_url'])
client.query("select count() from leads")
The app is build in rails4
I am doing the authentication with omniauth for oauth
can someone help ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ejholmes/restforce/issues/289, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHFpkUK7fu8MsZe136qpXvKXTIJLhvsks5rdK2rgaJpZM4MDfQG
.

@timrogers  thanks for pointing out that tim.
but the code is not reaching that point. i am getting the exception at the line
client.user_info
so still looking for some help. 😅

Aha! Can't see any obvious reason why that wouldn't work. Can you try some other API calls? (If it was an authentication problem, I'd expect to see an error to that effect.)

Can you describe what user_info looks like?  Like @timrogers mentioned, trying different API calls would be a good start.


Error should not trigger when a user enters a decimal as an amount

Frequency: 100%
iOS App version: iOS Testflight build : v1.8..0 (31), Android build v1.8.0
Repro on devices: Vivo 1606 (6.01), Google Pixel XL (8.1), Iphone 6+ (12.4.5)
Testing account:<PHONE_NUMBER>/Backup key
Repro Steps:

Launch the app and Login with test account
Tap on Gold tab.

Tap o Buy button

Tap on Switch to USD link.
Enter dot enter amount (USD) field.
Observe

Current Behavior:“Not enough celo dollars…” error message is shown
Expected Behavior: Error message should not shown
Investigation:

Issue does not repro when user Enter Amount in “Amount (USD)” filed.
User is able to proceed for exchange even after showing error message

Impact: User may get confused as they have balance however facing “Not enough celo dollars…”  error message.
Attachment: ErrormessageGoldTab.mp4

@annakaz Verified with latest build Test Flight build Valora app build  v1.0.0 (15), Android Valora Mainnet build v1.0.0 (16) and found that issue is still reproduce.

“Not enough celo dollars…” error message is shown when user enter decimal value in Amount (USD) field while “Buying/Selling” gold

Verified Devices: iPhone 7+ (13.3.1), Google Pixel Xl (8.1)

We can easily avoid this by not doing any input validation when the user enters a "." but keep the "next" button disabled

Just tested this and it looks to be working now

Hi @gnardini  I have verified this issue using latest Android internal build v1.3.0(1004294316) & Test Flight Build v1.3.0(27). and observe issue is still repro

“Not enough celo dollars…” error message is shown when user enter decimal value while “Buying/Selling” tab “
Repro on devices: Vivo V11 pro (10),iPhone 6+(12.4.5)

Hi @gnardini I have verified this issue using latest Android internal build v1.3.0(1004294317) & Test Flight Build v1.3.0(28). and observe issue is still reproduced.
Repro on devices: Samsung Galaxy A5(7.0) ,iPhone SE (13.5.1)

Hi @gnardini I have verified this issue using latest Android internal build v1.5.1(1004294319) & Test Flight Build v1.5.1(30). and observe issue is still reproduced.
Repro on devices: Samsung Galaxy A5(7.0) ,iPhone SE (13.5.1)

Hi @gnardini I have verified this issue using latest Android internal build v1.5.5(1004294322) & Test Flight Build v1.5.5(34). and observe issue is still reproduced.
Repro on devices: Samsung Galaxy A5(7.0) ,iPhone SE (13.5.1)

Hi @gnardini I have verified this issue using latest Android play store build v1.5.5 & App store Build v1.5.5 and observe issue is still reproduced.
Repro on devices: Samsung Galaxy A5(7.0) ,iPhone SE (13.5.1)

Hi @gnardini I have verified this issue using latest Android internal build v1.6.0(1004294325) & Test Flight Build v1.5.1(36) and observe issue is still reproduced.
Repro on devices: Samsung Galaxy A5(7.0) ,iPhone SE (13.5.1)

Hi @gnardini I have verified this issue using latest Android internal build v1.6.0(1004294326) & Test Flight Build v1.6.0(37) and observed that issue is still reproduced.
Repro on devices: Samsung Galaxy A5(7.0) ,iPhone 7+(14.1)

Hi @gnardini I have verified this issue using Play store build v1.6.0 & App store Build v1.6.0 and observed that issue is still reproduced.
Repro on devices: Samsung Galaxy A5(7.0) ,iPhone SE (14.2)

Hi @gnardini I have verified this issue using Android internal build V1.7.2(1004294329) and observed that issue is still reproduced.
Repro on devices: LG K7i(6.0)

Hi @gnardini I have verified this issue using Android internal build V1.7.2(1004294329) and observed that issue is still reproduced.
Repro on devices: LG K7i(6.0)

Hi @tarikbellamine @jeanregisser I have verified this issue with latest Android internal build V1.8.0 & Test Flight build V1.8.0(40) & observed that issue is still reproduced.
Verified on Devices: Samsung Galaxy A5(7.0), iPhone 6+ (12.4.5)
Issues:

Not enough celo dollars to make exchange Error message is shown when user enter decimal as amount

Hi @tarikbellamine @jeanregisser I have verified this issue with latest Android internal build V1.8.0 & Test Flight build V1.8.0(40) & observed that issue is still reproduced.
Verified on Devices: Samsung Galaxy A5(7.0), iPhone 6+ (12.4.5)
Issues:

Not enough celo dollars to make exchange Error message is shown when user enter decimal as amount

Hi @tarikbellamine @jeanregisser I have verified this issue with latest Android internal build V1.9.1 & Test Flight build V1.9.1(42) and observed that issue is still reproduced.
Verified on Devices: Samsung Galaxy s7(6), iPhone 6+ (12.4.5)
Issues:
Not enough celo dollars to make exchange Error message is shown when user enter decimal as amount

Hi @tarikbellamine @jeanregisser I have verified this issue with latest Android internal build V1.9.1 & Test Flight build V1.9.1(42) and observed that issue is still reproduced.
Verified on Devices: Samsung Galaxy s7(6), iPhone 6+ (12.4.5)
Issues:
Not enough celo dollars to make exchange Error message is shown when user enter decimal as amount

Hi @gnardini have verified this issue with latest Android Internal Build V1.11.0(1004294337) & Test Flight Build V1.11.0(47) & observed that issue is still reproduced.
Verified on Devices: Samsung Galaxy A5(7.0), iPhone SE (13.4.1)


[style]: Styling the back to Top Arrow Button

What's the style idea?
We need to improve  Back to Top Arrow button

Adding Hovering effect
Adding moving arrow animation
Adding Bulb like glowing effect
Creating a pulse effect around the arrow button
Changing color according to the website
Attractiver user interface with the styling color as that of website for a unique look of the website

Add screenshots
Add screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@SurajPratap10 sir, i would to style this arrow button, please assign this to me


Immutable dict in parameters.

After running the same code for quite some time. I start getting this error.
sqlalchemy.exc:StatementError: (sqlalchemy.exc.InvalidRequestError) Can't reconnect until invalid transaction is rolled back [SQL: u'SELECT TOP 1 drive_ungroup.drive_id AS drive_ungroup_drive_id, drive_ungroup.is_split AS drive_ungroup_is_split, drive_ungroup.is_deleted AS drive_ungroup_is_deleted, drive_ungroup.user_id AS drive_ungroup_user_id, drive_ungroup.created_at AS drive_ungroup_created_at, drive_ungroup.updated_at AS drive_ungroup_updated_at \nFROM drive_ungroup \nWHERE drive_ungroup.drive_id IS NULL'] [parameters: [immutabledict({})]]
I am using apscheduler and kubernetes structure. I am running my job on independent pods. But after running for some time I start getting this error. Why are parameters getting converted into immutable dict.?? And this starts affecting my other job as well where I get this empty dict error.
IN (?, ?, ?) ORDER BY subscription.user_id'] [parameters: [{}]].

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.


Using send for private methods?

If you have a method that could be private, but should be tested, for example because it is sending an outgoing command message. Should it be public so that it is easily tested or private and use send in RSpec to test?

I don't think this is something we need to worry about.
The public methods should be using the private methods, so any private methods should be tested implicitly in the tests for the public methods.

I believe the way that most students write code, not all private methods are tested implicitly in a public methods test. This is a "real world" expectation that does not work in these game projects (at least until we revamp the OOP lessons).
My original post did not explain the situation very clearly. Most student projects have a class where there is only one public entry point and the rest of the methods can technically be private. The one public method is usually a big long script, that is hard to test.
For example, in the following file, all the method except #play_game can be private. However, for this project, I have opted to keep every method public that is included in a script method & should be tested.
https://github.com/TheOdinProject/ruby_testing/blob/master/lib/15a_binary_game.rb
I just wanted to confirm that I should be keeping all of these methods public. As opposed to making these methods private and using send to test them.
Students are taught to keep everything private that is not needed to be public, so this creates confusion when it comes to testing these methods that can 'technically' be private, (but really should be public).

ahh I see what you mean, IMO this is a failure the earlier part of the course to teach separating those concerns. The idea we had about teaching the student to write a script which would handle the game loop and all the user interaction so the objects could be left to the game logic would be the approach to solve this in the long term.
I don't think we should be teaching students to use send to test private methods in the short term however as that would just be another bad habit we could get them into.
I think having all those methods public like you have is the way to go with this 👍

Sounds good. Thanks for your input on this issue!


Please invite me to the GitHub Community Organization

Name
Lucy
Discord Username (if applicable)
No response
Additional Context
Please invite me to the EddieHubCommunity Community Organization

You are already part of eddiehub 🚀


trim eth:0x97843608a00e2bbc75ab0C1911387E002565DEDE

it would be cool if our address inputs just trimmed the eth: out
maybe we could start with address vision but it probably needs to go all the way to SE2

Closed in #23


nsb format bug when run with table_test

nsb format with no additional arguments returns error message : "ValueError: tuple.index(x): x not in tuple", works fine when --capitalize is used however.

Thanks, good catch. It seems that when I stopped auto-capitalization of the answer line (to help handle chemical formulae and math equations), it made it so the W, X, Y, or Z in the answer line MUST be capitalized. This should be fixable by changing this line:
https://github.com/rishi-kulkarni/nsb-toolbox/blob/cdbdad467006d86c80465d72ecd1ce86c835ed85/nsb_toolbox/tables.py#L553
to:
choice_num = CHOICES.index(test_choice_match.group(1).upper() + ")")

I'm happy to review and merge a PR if you open one, otherwise I can probably push a fix this weekend. It'd probably be good to add a test that fails on current master due to this behavior and make sure it passes on the PR branch.

This has been fixed and release in v0.5.5. Thanks for the report!


UnicodeEncodeError: 'ascii' codec can't encode character u'\xfa' in position 5: ordinal not in range(128)

$ fbcap ./messages.htm
Discovered chat thread with  [,  ]...Traceback (most recent call last):
  File "/usr/bin/fbcap", line 9, in <module>
    load_entry_point('fbchat-archive-parser==0.7.post12', 'console_scripts', 'fbcap')()
  File "/usr/lib/python2.7/site-packages/fbchat_archive_parser/main.py", line 164, in main
    app.run()
  File "/usr/lib/python2.7/site-packages/clip.py", line 652, in run
    self.invoke(self.parse(tokens))
  File "/usr/lib/python2.7/site-packages/clip.py", line 634, in invoke
    self._main.invoke(parsed)
  File "/usr/lib/python2.7/site-packages/clip.py", line 519, in invoke
    self._callback(**{k: v for k, v in iteritems(parsed) if k not in self._subcommands})
  File "/usr/lib/python2.7/site-packages/fbchat_archive_parser/main.py", line 109, in fbcap
    raise e
UnicodeEncodeError: 'ascii' codec can't encode character u'\xfa' in position 5: ordinal not in range(128)

Thanks for reporting this! I believe I have fixed the issue. Try version 0.7.post15.


Payment failed

Payment fail after user subscribe to the non-free plans. Can this be because the site send the user to an upgrade/change plan? When the user get the free plan, the "payment" goes through but not when the user upgrade the plan. Also when the user buys the non-free plan, the site doesn't subscribe the user to that plan, instead it subscribe the user to the free plan, then the user can upgrade to other plans but the payment doesn't go through on Stripe.

I recopied the repo into a different project and now the payment works fine. Thank you!


HHH-13895 Upgrade the PostgreSQL JDBC driver used for testing to v. 42.2.11

https://hibernate.atlassian.net/browse/HHH-13895
It seems the new PostgreSql driver has changed the behaviour if a commit is attempted after an error occurred, in such a case the org.postgresql.util.PSQLException: The database returned ROLLBACK exception is thrown.
This behaviour causes the failure of tests where we try to check failures, in such a case a call to transaction.setRollbackOnly(); in the try/catch avoids the  to call a commit in the TransactionUtil#doInJPA() method.
For tests using the Session#doWork where we cause an exception on purpose the solution I found is to add the following try/catch block
try {
	conn.setAutoCommit( true );
}
catch (Exception e) {
	conn.rollback();
	conn.setAutoCommit( true );
}

in the DriverManagerConnectionProviderImpl$PooledConnections#add() methos

Right, that's expected as the PostgreSQL team just fixed this bug - which some considered critical, so that's why I updated.
I was expecting to see such errors reported on the CI build for #3296 .. remind me please, so we don't run anything other than H2 on pull request checks?

Did you check if your changes to tests have any impact on the other databases?

at the moment I think (but not 100% sure) for PR we are just running tests against h2

running test against Oracle and after I'll try MySQL

the tests passed with both Oracle and MySQL but what I'm not 100% sure about the fix i proposed is the change I made to the DriverManagerConnectionProviderImpl$PooledConnections#add() method, not sure this is the right approach.

Hi @dreab8 , interestingly the pgsql reverted that breaking fix, there's a new update of the driver available now.

Replaced by #3576 .. the fix should no longer be neccessary.


Is there any plan to drop jQuery in the near future?

I"m using Semantic on Mobile Web. It would be great if Semantic could drop jQuery requirement in future version. Is there any plan for that?

Take a look at this issue:
https://github.com/Semantic-Org/Semantic-UI/issues/1175


 Another reason to avoid Fireeye.

Page not found
Sorry, we couldn’t find the page you were looking for.
There’s no cause for alarm. It’s probably a simple typo, or an out-of-date link. You can:
Check the URL and try again
Use FireEye.com search to find what you need
Navigate to one of our more popular pages:
FireEye Security Solutions
Mandiant Consulting Services
FireEye Blogs
Contact Us
Thank you!


All projects: review and improve ignored tests

We have a lot of ignored Java tests - those marked with @Ignore junit annotation.
We have some additional tests ignored using:
OpValidationSuite.ignoreFailing();
and https://github.com/eclipse/deeplearning4j/blob/master/nd4j/nd4j-backends/nd4j-tests/src/test/java/org/nd4j/imports/TFGraphs/TFGraphTestAllSameDiff.java#L69-L130
We need to clean these up.
Specifically:

[ ] Write a simple script (add to dl4j-dev-tools repo) for counting the number of ignored tests
[ ] Go through and fix ignored tests where possible
[ ] Where not possible to fix, make sure we have a GitHub issue open for it (and link to the issue in code, next to the @Ignore)

ignore-results.tar.gz
 147 total
  65 nd4j/ignored_methods.csv
  30 nd4j/ignored_classes.csv
  22 deeplearning4j/ignored_methods.csv
  12 datavec/ignored_methods.csv
  10 arbiter/ignored_methods.csv
   6 deeplearning4j/ignored_classes.csv

Download and untar for details like below:
 cat deeplearning4j/ignored_classes.csv
IGNORED/ALL IGNORED: PROJECT, PACKAGE NAME, TEST CLASS, TEST METHOD, IGNORED COMMENT, LOCAL PATH TO TEST CLASS
ALL IGNORED: deeplearning4j, org.deeplearning4j.datasets.iterator, TestAsyncIterator, *,, /Users/susaneraly/SKYMIND/deeplearning4j/deeplearning4j/deeplearning4j-core/src/test/java/org/deeplearning4j/datasets/iterator/TestAsyncIterator.java
ALL IGNORED: deeplearning4j, org.deeplearning4j.nn.layers.capsule, CapsNetMNISTTest, *,"AB - ignored due to excessive runtime. Keep for manual debugging when required", /Users/susaneraly/SKYMIND/deeplearning4j/deeplearning4j/deeplearning4j-core/src/test/java/org/deeplearning4j/nn/layers/capsule/CapsNetMNISTTest.java
ALL IGNORED: deeplearning4j, org.deeplearning4j.nn.misc, LargeNetTest, *,, /Users/susaneraly/SKYMIND/deeplearning4j/deeplearning4j/deeplearning4j-core/src/test/java/org/deeplearning4j/nn/misc/LargeNetTest.java
ALL IGNORED: deeplearning4j, org.deeplearning4j.optimizer.listener, TestFailureListener, *,, /Users/susaneraly/SKYMIND/deeplearning4j/deeplearning4j/deeplearning4j-core/src/test/java/org/deeplearning4j/optimizer/listener/TestFailureListener.java
ALL IGNORED: deeplearning4j, org.deeplearning4j, RandomTests, *,, /Users/susaneraly/SKYMIND/deeplearning4j/deeplearning4j/deeplearning4j-core/src/test/java/org/deeplearning4j/RandomTests.java

Instruction on how to run locally here

Updated results including OpValidationSuite.IgnoreFailing
ignore-results.tar.gz
Result summary (Files have headers so line count 1 is really empty):
wc -l ~/Desktop/ignore-results/*/*csv
       1 /Users/susaneraly/Desktop/ignore-results/arbiter/ignore_failing_methods.csv
       1 /Users/susaneraly/Desktop/ignore-results/arbiter/ignored_classes.csv
      10 /Users/susaneraly/Desktop/ignore-results/arbiter/ignored_methods.csv
       1 /Users/susaneraly/Desktop/ignore-results/datavec/ignore_failing_methods.csv
       1 /Users/susaneraly/Desktop/ignore-results/datavec/ignored_classes.csv
      12 /Users/susaneraly/Desktop/ignore-results/datavec/ignored_methods.csv
       1 /Users/susaneraly/Desktop/ignore-results/deeplearning4j/ignore_failing_methods.csv
      26 /Users/susaneraly/Desktop/ignore-results/deeplearning4j/ignored_classes.csv
      85 /Users/susaneraly/Desktop/ignore-results/deeplearning4j/ignored_methods.csv
      37 /Users/susaneraly/Desktop/ignore-results/nd4j/ignore_failing_methods.csv
      30 /Users/susaneraly/Desktop/ignore-results/nd4j/ignored_classes.csv
      61 /Users/susaneraly/Desktop/ignore-results/nd4j/ignored_methods.csv
     266 total


Listening for an event with specific data

Hi,
I am loving this functionality! Thank you very much for making it so easy to have very complex automations :D!
I was trying out a custom automation using my zwave minimote. The tl;dr about it is that it is a multi-button remote. When a button is pressed, it emits an event, zwave.scene_activated, and it also associates some data about which button was pressed in the form of scene_id in an object:
2016-09-07 22:19:11.654286 DEBUG Event type:zwave.scene_activated:
2016-09-07 22:19:11.654432 DEBUG {'object_id': 'aeotec_dsa03202_minimote_4', 'entity_id': 'aeotec_dsa03202_minimote_4', 'scene_id': 7}

I am currently listening for events like this:
self.listen_event(self.cycle_scene, 'zwave.scene_activated', entity_id='aeotec_dsa03202_minimote_4')

I had originally thought that by passing another named parameter, scene_id, I'd be able to automatically filter on that.
self.listen_event(self.cycle_scene, 'zwave.scene_activated', entity_id='aeotec_dsa03202_minimote_4', scene_id=self.scene_id)

However, that did not work and cycle_scene was called for each different button press.
I'm currently getting around this by persisting a configured scene_id in initialize, and in the callback method, checking the incoming data and aborting the action if the numbers do not match:
  def cycle_scene(self, event_name, data, kwargs):
      if 'scene_id' in data and int(data['scene_id']) != int(self.scene_id):
          self.log('done')
          return

      # other code removed

Not bad, but I'm wondering if I've just missed something in the docs about how better to deal with this?
Thanks

Hi there - that's about as good as you can get with current functionality - unlike listen_state(), listen_event() is pretty simple, it responds to the named event and hands the data over.
This is partly because I haven't had a more complex usecase - I use custom events myself, but only in a simple way. However, there is no reason I couldn't expand it to handle the usecase of listening for specific attributes as is possible in listen_state() - to be somewhat consistent with listen_state() I would probably do something like:
self.listen_event(self.cycle_scene, 'zwave.scene_activated', attribute="scene_id", value=self.scene_id)

Does that sound like it would work?
FWIW, I have a minimote too but haven't gone through the hassle of setting it up for HA yet, so maybe I will do so so I can test this feature :)

That sounds perfect! Thank you very much :D

OK, I will add it to the list, but for now you have a working setup :)

Got my minimote working and I have added filtering to the listen state call - it has info in the API file but here are a couple of examples of what I ended up with:
Listen for a minimote event activating scene 3:
self.listen_event(self.generic_event, "zwave.scene_activated", scene_id = 3)
Listen for a minimote event activating scene 3 from a specific minimote:
self.listen_event(self.generic_event, "zwave.scene_activated", entity_id = "minimote_31", scene_id = 3)
I am calling these "event filters" and you can use them to filter for any arguments in any event.
I am just about to push the release with this in - 1.3.3.

Thank you so much for this! (Sorry for the delay - circling back to automation stuff now).
Looking forward to cleaning up the code I have.


Automatically cherry-pick #152 and #153

Automatically cherry-pick #152 and #153.

This change is

CI depends on #158.


sdl2.dll path error

hi, i use VS2015 compile this,  it says "Contrib/SDL/lib/x86/SDL2.dll file not found" when trying to copy it into Bin. Maybe the path in premake5.lua
postbuildcommands {
-- Copy the SDL2 dll to the Bin folder.
'{COPY} "%{wks.location}../../Contrib/SDL/lib/x86/SDL2.dll" "%{cfg.targetdir}"'
}
should be supplemented like
postbuildcommands {
-- Copy the SDL2 dll to the Bin folder.
'{COPY} "%{wks.location}../../../../Contrib/SDL/lib/x86/SDL2.dll" "%{cfg.targetdir}"'
}
, but i have't tested on other VS release versions.
However, this error does not affect the output exe file, it runs fine.

Did you download SDL2 and unzip it so that the path RecastDemo\Contrib\SDL\lib\x86 is valid? You can read more under the build instructions in the readme.

yes, i did. It is weird that "premake5 vs2015"  generate

IF EXIST "Contrib\SDL\lib\x86\SDL2.dll"\ (xcopy /Q /E /Y /I "Contrib\SDL\lib\x86\SDL2.dll" "....\Bin" > nul) ELSE (xcopy /Q /Y /I "Contrib\SDL\lib\x86\SDL2.dll" "....\Bin" > nul)

in RecastDemo.vcxproj file. Maybe this issue is up to premake-core. I compile premake5 by VS2015 in win10 64bit.

That is strange, I get the expected:
IF EXIST "$(SolutionDir)..\..\Contrib\SDL\lib\x86\SDL2.dll"\ (xcopy /Q /E /Y /I "$(SolutionDir)..\..\Contrib\SDL\lib\x86\SDL2.dll" "..\..\Bin" > nul) ELSE (xcopy /Q /Y /I "$(SolutionDir)..\..\Contrib\SDL\lib\x86\SDL2.dll" "..\..\Bin" > nul)
(and equivalent for x64) in both vs2015 and vs2017.
By the way, I notice that you aren't using the newest bits, so maybe try updating (the postbuildcommands in the premake file now takes x64/x86 into account).
Do you get the same behavior with a prebuilt binary from here? Also, does the file path containing the sources contain any strange characters?

Thank you for suggestions! The prebuilt binary premake5 does correct it by generating the complete path argument as yours. Please close this issue. :-)

Thanks for confirming, I will try to take a look at premake and see if it is something that should be reported.


HoodieCommitException: Failed to archive commits Caused by: IOException: Not an Avro data file at org.apache.hudi.table.HoodieTimelineArchiveLog.archive(HoodieTimelineArchiveLog.java:324)

Hi Team,
Getting the below error in hudi job.
HoodieCommitException: Failed to archive commits Caused by: IOException: Not an Avro data file
at org.apache.hudi.table.HoodieTimelineArchiveLog.archive(HoodieTimelineArchiveLog.java:324)
To Reproduce
we are running the hudi job through Azure databricks .
below dependancies we added in our code.
org.apache.spark:spark-avro_2.12:3.1.1
org.apache.hudi:hudi-spark3-bundle_2.12:0.8.0
org.apache.hudi:hudi-utilities-bundle_2.12:0.8.0
Environment Description

Hudi version :0.8.0

Spark version :3.2.1

Storage (HDFS/S3/GCS..) : AZURE blob storage.

you may have to apply 2 patches on this regard
https://github.com/apache/hudi/pull/4016
https://github.com/apache/hudi/pull/5261
let us know how it goes.

those are the fixes for the issue you are facing. you may need to apply those patches to your hudi if you cannot upgrade to 0.11 for now.
@pratyakshsharma : can you help here.

Ack. @nleena123 Were you able to apply the patches? If not, we can probably connect on slack and I can help you there.

@nsivabalan
slack is not allowed in my organisation , can we connect by any other app ?

You can suggest some other app as per your convenience, else google meet also works.

@nsivabalan @pratyakshsharma is there any followup on this issue?

No followups with me. @nsivabalan can you confirm as well?

@nleena123 : any updates here please. I don't think I had any conversations in slack either.

Closing this due to inactivity. Feel free to reopen if you are still facing issues @nleena123


feat(cloudfront): Adding in feature to create cloudfront functions and associate to cloudfront distribution

Coverage Report

Totals

Statements:
96.19% ( 4212 / 4379 )

Methods:
90.85% ( 149 / 164 )

Codecov Report

Merging #15 (b2f79fc) into develop (506ee60) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #15      +/-   ##
===========================================
+ Coverage    96.11%   96.18%   +0.07%
===========================================
  Files           27       27
  Lines         4295     4379      +84
  Branches       288      293       +5
===========================================
+ Hits          4128     4212      +84
- Misses         165      167       +2
+ Partials         2        0       -2

Impacted Files
Coverage Δ

src/lib/construct/siteWithEcsBackend.ts
97.89% <100.00%> (+0.10%)
:arrow_up:

src/lib/construct/staticSite.ts
96.19% <100.00%> (+0.35%)
:arrow_up:

src/lib/manager/cloudFrontManager.ts
100.00% <100.00%> (ø)

src/lib/manager/s3Manager.ts
88.55% <0.00%> (ø)

src/lib/construct/graphQLApiLambda.ts
94.18% <0.00%> (ø)

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 506ee60...b2f79fc. Read the comment docs.

Coverage Report

Totals

Statements:
96.22% ( 4246 / 4413 )

Methods:
90.85% ( 149 / 164 )

Coverage Report

Totals

Statements:
96.23% ( 4264 / 4431 )

Methods:
90.96% ( 151 / 166 )


Secure connection between Orthanc and OHIF viewer without login

Hi guys,
I was just looking to see if anyone has some experience with this as I've tried and failed. I'm currently integrating the OHIF viewer into a web platform I'm building and I need to make sure that:

The users don't have to login to OHIF as they're already logged into my platform
Only those with passwords can access Orthanc

So my plan is to implement authorisation using my nginx reverse proxy that already handles CORS for Orthanc, but I would need to send an auth header in every request OHIF sends to the server, does anyone know where in the code I could do this or what methods I should be looking to change?
Thanks
Evan

By including
requestOptions: {
  auth: 'user:password'
}

in my server config I could retrieve study and instances information. But retrieving info through WADO seems to still be a problem, I think I might have to change the cornerstone code. If anyone has any information on this it would be much appreciated!

@Signo-Alaris There's a couple of options that you can take. We've been integrating the viewer into a larger web platform called Sonador (also based on Orthanc), and took the direction of implementing an authentication and authorization server (see https://code.oak-tree.tech/oak-tree/medical-imaging/sonador).
Orthanc has an "advanced authorization plugin" (https://book.orthanc-server.com/plugins/authorization.html, although this has been deprecated in favor of handling auth inside of a Python plugin), which can be used to secure the server endpoints. When a client asks for a specific resource, it sends a request to the auth server with information about the resource the credentials for the client. The auth server can then determine if the user has permissions and allow/deny the resource.
Because we need Sonador to interoperate in larger cloud/enterprise environments, we decided to implement support for OpenID connect as the central component of our authentication workflow. For that reason, we pass a session based JWT token to OHIF and utilize that as the base of authentication.
There some tweaks we need to make to this structure to allow for more granular permissions at the study/series level, but so far it's been pretty robust. We have a large deployment of the platform and have started using it for building a larger patient matched process for an orthopedics company.
Here are some additional resources which describe the choices we made:

Blog post covering how the authentication system works and configuration for a PWA build of OHIF: https://www.oak-tree.tech/blog/sonador-ohif-config
Sonador Web Application code: https://code.oak-tree.tech/oak-tree/medical-imaging/sonador. Implementation of the authorization backend is under apps/visionaire/auth/views/service.py
Sonador Orthanc Python Plugin: https://code.oak-tree.tech/oak-tree/medical-imaging/orthanc-sonador. Shows how Sonador and Orthanc are integrated and the choice of plugin.
Sonador Development Environment: https://code.oak-tree.tech/oak-tree/medical-imaging/imaging-development-env. Reference environment showing configuration and components. Includes Docker Compose manifests for running the environment locally and reference manifests for deploying to Kubernetes.

Could you please review our comprehensive documentation on Cross-Origin Resource Sharing (CORS) and OHIF? Below are the links provided for your reference.

https://docs.ohif.org/deployment/cors
https://docs.ohif.org/deployment/iframe


Using Revel on Windows

Hello,
I tried out Revel in my workstation at work (powered by Linux) and it worked great but when I wanted to use Revel on windows (my home machine), there are no guidelines on how can I run the revel command or how to map the command to my PATH.
Can you please point me to the right direction?
Thanks.

i had tried but gain following error
C:\Users\dkvon>go get github.com/revel/cmd/revel
C:\Users\dkvon>revel run github.com/revel/samples/chat
'revel' is not recognized as an internal or external command,
operable program or batch file.
I am new in golang suggest me solution

Windows 10 user here.
Instructions appeared a bit confusing for me or just missed a step.
if you install go from msi installer, you have to add this path to PATH enviroment variable.


Allow usage of a persistent cache outside of /tmp/

Ideally the restic cache should persist between backup sessions. Usage of /tmp/ is not ideal for this. I'd like to be able to specify a cache directory (something like ~/.cache/restic, as the cache is only really effective if it persists between sessions.

Update: i checked, it uses the regular path, I was confused by the bit from the documenation talking about /tmp/