Question: Jax-related issues under the linux

What is your question?
Errors related to JAX occur when predicting structures using the "colabfold_batch" command.
(構造予測のコマンド[colabfold_batch]を実行したときにJAX関連のエラーが生じます。)
Computational environment

OS: Linux wa01 4.18.0-240.22.1.el8_3.x86_64
CUDA version if Linux (Show the output of /usr/local/cuda/bin/nvcc --version.)

nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Sun_Feb_14_21:12:58_PST_2021
Cuda compilation tools, release 11.2, V11.2.152
Build cuda_11.2.r11.2/compiler.29618528_0

gcc version

 gcc (GCC) 8.3.1 20191121 (Red Hat 8.3.1-5)
Copyright (C) 2018 Free Software Foundation, Inc.

To Reproduce
Steps to reproduce the behavior:

Do linux version installation(README_ja.md, Did at 2022/11/23)

wget https://raw.githubusercontent.com/YoshitakaMo/localcolabfold/main/install_colabbatch_linux.sh
bash install_colabbatch_linux.sh

Do "colabfold_batch" command

colabfold_batch 6x9z.fasta 6x9z

↓6x9z.fasta file
>6X9Z_1|Chain A|Transmembrane beta-barrels|synthetic construct (32630)
MEQKPGTLMVYVVVGYNTDNTVDVVGGAQYAVSPYLFLDVGYGWNNSSLNFLEVGGGVSYKVSPDLEPYVKAGFEYNTDNTIKPTAGAGALYRVSPNLALMVEYGWNNSSLQKVAIGIAYKVKD

Error happened

2022-11-23 23:33:49,769 generated new fontManager
2022-11-23 23:34:09,559 Found 5 citations for tools or databases
2022-11-23 23:34:14,781 Query 1/1: 6X9Z_1_Chain_A_Transmembrane_beta-barrels_synthetic_construct__32630_ (length 124)
COMPLETE: 100%|██████████████████████████████████████████████████████████████████████████████████████| 150/150 [elapsed: 00:01 remaining: 00:00]
2022-11-23 23:34:15,892 Running model_3
Traceback (most recent call last):
[...]
jax._src.errors.UnexpectedTracerError: An UnexpectedTracerError was raised while inside a Haiku transformed function (see error above).
Hint: are you using a JAX transform or JAX control-flow function (jax.vmap/jax.scan/...) inside a Haiku transform? You might want to use the Haiku version of the transform instead (hk.vmap/hk.scan/...).
See https://dm-haiku.readthedocs.io/en/latest/notebooks/transforms.html on why you can't use JAX transforms inside a Haiku module.
See https://jax.readthedocs.io/en/latest/errors.html#jax.errors.UnexpectedTracerError

↓log.txt
2022-11-23 23:32:56,589 Running colabfold 1.3.0 (b41ed8d6e98bf33a82307557f3e9102bd34eeafe)
2022-11-23 23:33:49,769 generated new fontManager
2022-11-23 23:34:09,559 Found 5 citations for tools or databases
2022-11-23 23:34:14,781 Query 1/1: 6X9Z_1_Chain_A_Transmembrane_beta-barrels_synthetic_construct__32630_ (length 124)
2022-11-23 23:34:15,892 Running model_3

Expected behavior
I will get poly-acid structure.
Related issue?
https://github.com/YoshitakaMo/localcolabfold/issues/119

I updated the dependency packages to be compatible with the recent change of Haiku and JAX. See README of this repository and report the feedback. Thank you in advance.

@YoshitakaMo Thank you for rapid fix !
I did re-install and predict.
bash install_colabbatch_linux.sh

# '/path/to/your/colabfold_batch' should be substituted to your path, e.g. '/home/moriwaki/Desktop/colabfold_batch'
# install GPU-supported jaxlib
COLABFOLDDIR="/path/to/your/colabfold_batch"
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip uninstall "colabfold[alphafold] @ git+https://github.com/sokrypton/ColabFold" -y
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip uninstall alphafold-colabfold -y
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip install "colabfold[alphafold] @ git+https://github.com/sokrypton/ColabFold"
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip install https://storage.googleapis.com/jax-releases/cuda11/jaxlib-0.3.25+cuda11.cudnn82-cp37-cp37m-manylinux2014_x86_64.whl
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip install jax==0.3.25 biopython==1.79
# fix jax.tree_(un)flatten warnings (ad hoc)
sed -i -e "s/jax.tree_flatten/jax.tree_util.tree_flatten/g" ${COLABFOLDDIR}/colabfold-conda/lib/python3.7/site-packages/alphafold/model/mapping.py
sed -i -e "s/jax.tree_unflatten/jax.tree_util.tree_unflatten/g" ${COLABFOLDDIR}/colabfold-conda/lib/python3.7/site-packages/alphafold/model/mapping.py

colabfold_batch 6x9z.fasta 6x9z

Result↓
# '/path/to/your/colabfold_batch' should be substituted to your path, e.g. '/home/moriwaki/Desktop/colabfold_batch'
# install GPU-supported jaxlib
COLABFOLDDIR="/path/to/your/colabfold_batch"
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip uninstall "colabfold[alphafold] @ git+https://github.com/sokrypton/ColabFold" -y
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip uninstall alphafold-colabfold -y
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip install "colabfold[alphafold] @ git+https://github.com/sokrypton/ColabFold"
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip install https://storage.googleapis.com/jax-releases/cuda11/jaxlib-0.3.25+cuda11.cudnn82-cp37-cp37m-manylinux2014_x86_64.whl
${COLABFOLDDIR}/colabfold-conda/bin/python3.7 -m pip install jax==0.3.25 biopython==1.79
# fix jax.tree_(un)flatten warnings (ad hoc)
sed -i -e "s/jax.tree_flatten/jax.tree_util.tree_flatten/g" ${COLABFOLDDIR}/colabfold-conda/lib/python3.7/site-packages/alphafold/model/mapping.py
sed -i -e "s/jax.tree_unflatten/jax.tree_util.tree_unflatten/g" ${COLABFOLDDIR}/colabfold-conda/lib/python3.7/site-packages/alphafold/model/mapping.py

I checked predict-output, so this fix is good for bug.


[Discuss] Including a print operator/standard library function for debugging

For setting up tutorial examples and debugging, it would be helpful to be able to print intermediate values in a Relax program. Relay never had a built-in print function or operator and that made debugging large Relay programs more difficult. Fortunately, with call_packed, it already is possible to call a function in Relax that can print values. I think we should make this easier to do and use it in tutorials and examples because it can help users.
We could have the Python print serve as syntactic sugar in TVMScript for a call to a PackedFunc that prints values. The PackedFunc could be registered as an operator (once we set up the operator registry) or just part of the standard library/prelude if we already have one of those. We might as well get it sooner rather than later.

Hi, @slyubomirsky. Thank you for the interesting idea.
Could you provide some use case scenarios on your mind?
Do you want to print the intermediate runtime values in tensors?

Correct, it would be useful to be able to print values in the middle of a program. I am particularly interested in this for tutorials because it could be used to demonstrate different aspects of the semantics. For example, showing what is returned by a function call or the value after a branch.
The reason I bring up call_packed is because you could use a packed function right now to do this:
@tvm.register_func("print_test")
def print_test(x):
    print(x)

@tvm.script.ir_module
class PrintArg:
    @R.function
    def foo(x: Tensor((), "int32")) -> Tensor((), "int32"):
        y = R.call_packed("print_test", x, type_args=(Tensor))
        return x

It would be nice to have syntactic sugar for it instead of writing it out every time.

@slyubomirsky, I think it's an interesting idea! To achieve this, you can register a relax.print op by using the FCallPacked attribute developed by @psrivas2, the op will get lowered to a call_packed during codegen. You can refer to how relax.unique op is implemented: https://github.com/tlc-pack/relax/pull/135, https://github.com/tlc-pack/relax/blob/relax/src/relax/op/tensor/unary.cc#L41.

I'll do exactly that, thanks for the suggestion.

Now implemented


Update index.md

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
ffd1d7b8ba6984cb4415fac8c2aa9e06f9c8517b

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/63e260c64f2eda00084a1bbb


⚠ Bofrost.de has degraded performance

In 972d326, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1513 ms

Resolved: Bofrost.de performance has improved in 10b4f3a after 7 minutes.


ERR_OSSL_EVP_UNSUPPORTED on npm start

Describe the bug
Starting the studio fails with:
Starting the development server...

Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:67:19)
    at Object.createHash (node:crypto:130:10)
    at module.exports (/studio/node_modules/webpack/lib/util/createHash.js:135:53)
    at NormalModule._initBuildHash (/studio/node_modules/webpack/lib/NormalModule.js:417:16)
    at handleParseError (/studio/node_modules/webpack/lib/NormalModule.js:471:10)
    at /studio/node_modules/webpack/lib/NormalModule.js:503:5
    at /studio/node_modules/webpack/lib/NormalModule.js:358:12
    at /studio/node_modules/loader-runner/lib/LoaderRunner.js:373:3
    at iterateNormalLoaders (/studio/node_modules/loader-runner/lib/LoaderRunner.js:214:10)
    at iterateNormalLoaders (/studio/node_modules/loader-runner/lib/LoaderRunner.js:221:10)
/studio/node_modules/react-scripts/scripts/start.js:19
  throw err;
  ^

Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:67:19)
    at Object.createHash (node:crypto:130:10)
    at module.exports (/studio/node_modules/webpack/lib/util/createHash.js:135:53)
    at NormalModule._initBuildHash (/studio/node_modules/webpack/lib/NormalModule.js:417:16)
    at /studio/node_modules/webpack/lib/NormalModule.js:452:10
    at /studio/node_modules/webpack/lib/NormalModule.js:323:13
    at /studio/node_modules/loader-runner/lib/LoaderRunner.js:367:11
    at /studio/node_modules/loader-runner/lib/LoaderRunner.js:233:18
    at context.callback (/studio/node_modules/loader-runner/lib/LoaderRunner.js:111:13)
    at /studio/node_modules/babel-loader/lib/index.js:59:103 {
  opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

How to Reproduce
Run
npm install
npm start
on Fedora 36 with node v16.14.0.
Expected behavior
I expected the studio to start.

@turbinenreiter Hello! Thanks for the issue. Could you try to run start script by exporting given env?
export NODE_OPTIONS=--openssl-legacy-provider

or try this one https://roytuts.com/how-to-fix-err_ossl_evp_unsupported-in-react-js-application/
Let us know what fix help you and we will try to propagate it in project.

--openssl-legacy-provider did not help.

@turbinenreiter Any update?

Closing due to lack of feedback.

@aeworxet has similar problem.

This error will appear again as more and more users transition to Node.js v17+.
This error is because node.js 17 uses OpenSSL3, which has changed code for
initialization context of md family (including md4), and this is a breaking change.

https://itsmycode.com/error-digital-envelope-routines-unsupported
The approach with
export NODE_OPTIONS=--openssl-legacy-provider

works but I would still prefer to modify package.json with
- "start": "cross-env DISABLE_ESLINT_PLUGIN=true craco start",
+ "start": "cross-env DISABLE_ESLINT_PLUGIN=true craco --openssl-legacy-provider start",

to keep users from frustration and continue uninterruptable shipping of a fully-fledged product.

@aeworxet Could you create PR? We will be glad for it :)

You can also pass env like:
cross-env DISABLE_ESLINT_PLUGIN=true NODE_OPTIONS=--openssl-legacy-provider craco start


items search results are not shown on demo

Referring to paper-search-panel demo
https://www.webcomponents.org/element/Collaborne/paper-search/demo/demo/paper-search-panel.html
I see the items array gets initialized with values ["A", "B", "C"] but I see no effect to items value when searching these items. Are these items supposed to be displayed on the panel? Is the search/filtering really working or is the meaning to implement the actual functionality by one self? it's a little bit unclear from the API doc/demo what to actually expect.
To me it looks like either a bug or simply missing documentation on how items array works and is displayed or not displayed, or I just missed something obvious here?
Any insight much appreciated.

I'm having the same issue/question

Okay, thanks for the prompt reply. That does clarify, thanks. I will make my trials based on this.


iOS Malware Domains

Hi there,
Please can you add the following domains to the malware filter blacklist:
addatamarket.net
backuprabbit.com
businessvideonews.com
cloudsponcer.com
datamarketplace.net
mobilegamerstats.com
snoweeanalytics.com
tagclick-cdn.com
topographyupdates.com
unlimitedteacup.com
virtuallaughing.com
web-trackers.com
growthtransport.com
anstv.net
ans7tv.net
Source:
https://securelist.com/operation-triangulation/109842/

Thank you @Eow2022yia these are added in commit 97b0146f. This will go live in the next release.
Closing.


Join two lines together

Check for existing issues

[X] Completed

Is your feature request related to a problem?
Sometimes you want to join the current line with the next line. Doing this manually is tedious, especially if you join multiple lines together.
Describe the solution you'd like
Add a keybinding, allowing you to join two lines together. Look at the following example (The caret is marked using a vertical bar (|)):
foo|
bar

Pressing Command + Shift + j would transform the above into:
foo |bar

This is inline with how both Sublime Text and Vim does it.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

I would love this feature, I use this all the time in Sublime and it's one of the very few things holding me back from completely switching to Zed

@hovsater Thanks for the report. ctrl-j (or J in vim mode) now works in recent versions of zed!


Add options to set the fft size and to normalize the ifft output; calculate the fft by default; and improve the documentation and the tests

This PR does a few small things:

It improves the fft function documentation
It adds a normalize argument to the fft and dft functions. Normalization is disabled by default but if enabled it will normalize the ifft otuput (i.e. it only applies if inverse is true)
It makes inverse=false the default (i.e. the fft function calculates the fft by default)
It adds an n argument to make it possible to choose the FFT size (it still uses the input length by default).
It improves the tests (which could fail if the folds were negative)

I wonder if there is a more optimum way to implement the "Add n argument to select the FFT size" commit


COMPAT: Support pandas 019

Coverage decreased (-0.02%) to 94.097% when pulling f1ca1bca1fd46a1805e5a7f8b6fa7c7b75895a48 on sinhrks:pandas_019 into 870ef8fe812c4dd53e17ad85347153eeadfbd7d7 on pandas-ml:master.

Coverage decreased (-0.02%) to 94.098% when pulling 59ed703909fbe812d2390a4f608c19ce33e9611f on sinhrks:pandas_019 into 870ef8fe812c4dd53e17ad85347153eeadfbd7d7 on pandas-ml:master.


Error occured

When embedding this Cake plugin, I receive the following error...
========================================
Dependency-Check
An error occurred when executing task 'Dependency-Check'.
Error: One or more errors occurred.
Could not load file or assembly 'System.Reflection.TypeExtensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of
its dependencies. The system cannot find the file specified.

Hi Ronald,
Could you send this informations?
OS version: ?
(.Net or .Net Core or Mono) version: ?

Hi,
first of all, thanks for the swift response.
Using Windows 10 enterprise, version 1607 and .NET 4.7. Also using the latest cake (0.25.0).
If there is any other info I can provide you, just drop me a note. Would be really handy to have the dependencycheck inside cake build.

Hi Ronald,
Could you try again with version 1.1.0? If it throws exception again, please rerun your build with verbose mode and paste logs. (like this -> .\build.ps1 -Verbosity Diagnostic)
I'm using reflection in ArgumentAppender.cs file. So when it call settings.GetType().GetProperties() line, it using System.Reflection.TypeExtensions assembly file and if your system hasn't netstandard1.6 framework it throw System.IO.FileNotFoundException. So I added net4.5 and net4.6 frameworks as my target frameworks. It should works now. :)
Thanks,
Burak

Do you get another error again, or can I close this issue?

Works like a charm! Excellent work, thank you!
We are going to .net core 2.1 in the middle of next week, will it also support this framework?

It's already supported. :)


How is the SSID and Password being written to SPIFFS?

Thank you for this great tool.
I need to add more stored data to be entered on the configuration page.
Where do I add a new code block for another data entry?
Where do I edit the HTML to customize the configuration page?

Thanks for using the library.
I suggest you at least go through all the examples to have the experience of what and how to use the features this ESPAsync_WiFiManager is providing.
Then you will know that to add new configurable data, you don't need to change the library source code or modify any HTML. But just easily use ESPAsync_WMParameter, as in Async_AutoConnectWithFSParameters example to add 5 new configurable parameters.
bool loadFileFSConfigFile(void)
{
   ...
}

bool saveFileFSConfigFile(void)
{
   ...
}

void setup()
{
...
// The extra parameters to be configured (can be either global or just in the setup)
  // After connecting, parameter.getValue() will get you the configured value
  // id/name placeholder/prompt default length
  ESPAsync_WMParameter custom_blynk_server("blynk_server", "blynk_server", blynk_server, BLYNK_SERVER_LEN + 1);
  ESPAsync_WMParameter custom_blynk_port  ("blynk_port",   "blynk_port",   blynk_port,   BLYNK_SERVER_PORT_LEN + 1);
  ESPAsync_WMParameter custom_blynk_token ("blynk_token",  "blynk_token",  blynk_token,  BLYNK_TOKEN_LEN + 1 );

  ESPAsync_WMParameter custom_mqtt_server("mqtt_server", "mqtt_server", mqtt_server, MQTT_SERVER_MAX_LEN + 1);
  ESPAsync_WMParameter custom_mqtt_port  ("mqtt_port",   "mqtt_port",   mqtt_port,   MQTT_SERVER_PORT_LEN + 1);
  ...
}

Certainly, you have to learn and do that yourself, as I can't have time to help you do your research and code.

Negative.
I don't want to connect to Blynk.
I want to store data to SPIFFS/EEPROM.
That's why I asked where to modify the code to customize the HTML and add data storage block for my own settings, not Blynk server login and address data...

Example is just an example, and you must modify according to your use-case.
You certainly have to modify Blynk-related data to what you need. And at least basic knowledge is necessary to use the library.
No more answer until you read and understand what I meant.
Please don't write nonsense here unless you understand what you are doing.

Identify the file and line number of the EEPROM/SPIFFS code blocks that store data in the register (if you didn't steal it from another library and credit yourself). I found the HTML.

Thanks for using the library.
I suggest you at least go through all the examples to have the experience of what and how to use the features this ESPAsync_WiFiManager is providing.
Then you will know that to add new configurable data, you don't need to change the library source code or modify any HTML. But just easily use ESPAsync_WMParameter, as in Async_AutoConnectWithFSParameters example to add 5 new configurable parameters.
bool loadFileFSConfigFile(void)
{
   ...
}

bool saveFileFSConfigFile(void)
{
   ...
}

void setup()
{
...
// The extra parameters to be configured (can be either global or just in the setup)
  // After connecting, parameter.getValue() will get you the configured value
  // id/name placeholder/prompt default length
  ESPAsync_WMParameter custom_blynk_server("blynk_server", "blynk_server", blynk_server, BLYNK_SERVER_LEN + 1);
  ESPAsync_WMParameter custom_blynk_port  ("blynk_port",   "blynk_port",   blynk_port,   BLYNK_SERVER_PORT_LEN + 1);
  ESPAsync_WMParameter custom_blynk_token ("blynk_token",  "blynk_token",  blynk_token,  BLYNK_TOKEN_LEN + 1 );

  ESPAsync_WMParameter custom_mqtt_server("mqtt_server", "mqtt_server", mqtt_server, MQTT_SERVER_MAX_LEN + 1);
  ESPAsync_WMParameter custom_mqtt_port  ("mqtt_port",   "mqtt_port",   mqtt_port,   MQTT_SERVER_PORT_LEN + 1);
  ...
}

Certainly, you have to learn and do that yourself, as I can't have time to help you do your research and code.

^This only tells me that you can add parameters, not how to add them. I assume that you use .addparameter or .newparameter, however, there is a lot of fog in your "explanation". Why is there max length constants? Do I even need the max length constants? Where is the parameter add supposed to take place? Do I need to be inside the configfilewrite function to add parameters?
Forgive me for asking questions. You must be vastly superior to me in every way and have never ever needed anyone's help, ever. Well, I'm not you. I need a little help adding these parameters because your code lacks documentation and sufficient explanatory commenting. Had you written a documentation I would not be bothering you whatsoever. You know about RTFM? Well, WTFM! (w=write)... I thought people in the open source maker community were all about helping others learn. I thought wrong, apparently.

I'm so sorry if you're always thinking the wrong way.
I just hope we're not ungrateful teenagers anymore, always blaming other people problems happening because of our faults.
Many of us have been spending hard time to learn and getting our knowledge, then experiencing by investing to get knowledge once, we'll harvest the good fruit your whole life. That's the ideal target we'll like to have, otherwise, why wasting our time here. So asking you to spend some time to research, to go thru examples, etc. , then from that to have experience, before blaming, is not excessive.
The open source community will help you, by creating many facilities for you to learn, use and save time, not by giving somebody a way to cheat (I mean doing things for you, or giving you some kind of fast food that will harm you in the future, etc.). Especially we're not a code factory.
I've tried my best to WT most complete possible FM, but certainly up to certain level, and many people so far are OK and successfully using the library.
If you think you can rewrite a better manual and contribute back to the community, you're very welcome. Just don't sit and blame everybody around you.
I'll close and delete this thread as it's not helpful and negative.


Roll Plugins from d7125cdb91c0 to 7b9ac6b0c20d (11 revisions)

https://github.com/flutter/plugins/compare/d7125cdb91c0...7b9ac6b0c20d
2021-03-20<EMAIL_ADDRESS>[quick_actions] 2/3 Quick actions federated platform interface (flutter/plugins#3735)
2021-03-19<EMAIL_ADDRESS>[ci] Run more web tests (flutter/plugins#3739)
2021-03-19<EMAIL_ADDRESS>Enable web integration tests in CI (flutter/plugins#3738)
2021-03-19<EMAIL_ADDRESS>Standardize copyright year (flutter/plugins#3737)
2021-03-19<EMAIL_ADDRESS>Moved quickactions to a subfolder (flutter/plugins#3734)
2021-03-19<EMAIL_ADDRESS>[integration_test] Deprecate, and stop using in this repository (flutter/plugins#3723)
2021-03-18<EMAIL_ADDRESS>Standardize Copyrights: Chromium->Flutter (flutter/plugins#2996)
2021-03-18<EMAIL_ADDRESS>Fix cosmetic variations in copyrights and license files (flutter/plugins#3730)
2021-03-18<EMAIL_ADDRESS>Add an AUTHORS file to each plugin (flutter/plugins#3725)
2021-03-18<EMAIL_ADDRESS>[google_sign_in] Adds support to send clientId as a parameter (flutter/plugins#3640)
2021-03-17<EMAIL_ADDRESS>Prep for alignment with Flutter analysis options (flutter/plugins#3703)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Polish tax id validation method

This PR adding new custom method to validate Polish tax identifier number (NIP).
Working example:
https://jsfiddle.net/dycqm91k/1/

Thx


Read config.json file fails several times

Hi there, I'm using your script to sync my local DB and I'm seeing this issue sometimes:
[22-Aug-2015 20:14:01 America/New_York] PHP   1. {main}() /var/sync/sync_sandbox/replicator/Replicator.php:0
[22-Aug-2015 20:14:01 America/New_York] PHP   2. Replicator->__construct() /var/sync/sync_sandbox/replicator/Replicator.php:351
[22-Aug-2015 20:14:01 America/New_York] PHP   3. Replicator->loadConfig() /var/sync/sync_sandbox/replicator/Replicator.php:27
[22-Aug-2015 20:14:01 America/New_York] PHP   4. file_get_contents() /var/sync/sync_sandbox/replicator/Replicator.php:244
[22-Aug-2015 20:14:01 America/New_York] PHP Fatal error:  Uncaught exception 'Exception' with message 'error reading config.json' in /var/sync/sync_sandbox/replicator/Replicator.php:246
Stack trace:
#0 /var/sync/sync_sandbox/replicator/Replicator.php(27): Replicator->loadConfig()
#1 /var/sync/sync_sandbox/replicator/Replicator.php(351): Replicator->__construct()
#2 {main}
  thrown in /var/sync/sync_sandbox/replicator/Replicator.php on line 246

File exists, is readable by the script, giving a try to the script manually one time and it fails, second time it fails, third time and it works, four time and it fails, so, where is the issue here? why it fails sometimes and sometimes not? Can you take a look to this?
I'm running the script every night as a cronjob and it's failing due to this issue.
Thanks in advance

I'm not sure what is happening but I've added some improved error handling so that it is easier to narrow the issue down to a problem reading the config or an issue parsing it.

@tom-gangemi it's working now, thx


Non-standard assembler usage

The Clang/LLVM compiler is stricter than GCC in complying with the RISC-V assembler specification. If I used the latest Clang/LLVM compiler to compile the benchmarks, I get a failure. Extracting an individual test shows this:
riscv32-unknown-elf-clang --target=riscv32-unknown-elf -march=rv32i -c
-o I-ENDIANESS-01.o -I ../../../riscv-test-env -I
../../../riscv-test-env/p -I ../../../riscv-target/ri5cy I-ENDIANESS-01.S

../../../riscv-target/ri5cy/compliance_io.h:187:12: error: operand must
be a bare symbol name
    la t3, 0x10000000; sw a0, (0)(t3);;

This instruction sequence comes from the expansion of one of the standard header macros.
The error is that the RISC-V standard requires the second operand to the la pseudo-instruction to be a symbol, not a constant. It just happens that GCC accepts a constant - arguably a bug in GCC, certainly an unofficial extension.
It seems to me that a compliance test suite should strictly follow the standard!
(this would also seem to indicate that no one is using Clang/LLVM with the compliance tests, which would certainly seem to be a good sanity check).

Historically, the behavior of the GNU assembler has been the de facto spec, though I realize that's a bit hegemonic.
Philosophically, a constant value is a valid address.  My preference in this case is to maximize compatibility: update the spec to clarify that constant-valued addresses are legal addresses for the purposes of la, and encourage the LLVM team to follow suit.

Hi Jeremy,
The erro is in the file: compliance_io.h that is managed and provided by the target. Since the compilation, linking and execution of the test is responsibility of the target, the framework cannot impose any restrictions there. If ris5cy wants the compile the tests with clang and gcc then it should change the respective compliance_io.h.
I did a grep and quick glance indicates that atleast this particular case isn't occurring anywhere else in the tests.

@aswaterman Agreed the spec needs changing, since it says "symbol". Clang is clear what is a symbol and what is a constant. Initial implementations inherently define specifications - that is common in many areas. We just need to clean up the gaps, so the spec becomes the golden reference.
@neelgala It is reasonable for the compliance test suite to require compliance with the spec, but where bugs are found, responsibility for resolution needs to be placed on the target owner. So this issue should be assigned to that person. Do you know who owns the RI5CY target?

I think @bluewww was the author of the PR for adding ri5scy.
I just wanted to say that it is a bug only if RI5CY wants to be Clang compatible. The compliace framework declares a combination of SDK and Device Target as RISC-V compliant.
Also, I think #107 and #106 (post resolution) should probably end up as guidelines within the Test-format spec.
Maybe the RI5CY guys could fix this.

The code in question doesn't actually use the constant directly, it does
this:
#define RVTEST_CUSTOM1 0x10000000
And then uses RVTEST_CUSTOM1 as an argument to a macro.
So, is the fix simple to change it to:
RVTEST_CUSTOM1 equ 0x10000000. ?
On Wed, Apr 22, 2020 at 6:25 AM bluew<EMAIL_ADDRESS>wrote:

I track it for now, but I don't really use clang. The quick fix would
probably be to define a symbol with the correct constant and load that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-compliance/issues/105#issuecomment-617776038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJRPAFAVDLX4ODCXIG3RN3V4ZANCNFSM4MN6OXTA
.

The other fix (which I've mentioned elsewhere) is to change the LA in riscv-target/ri5cy/compliance_io.h to be LI.

Same issue in compiling riscv-tests with
clang -static -mcmodel=medany -nostdlib -nostartfiles -mno-relax --target=riscv64 -mcpu=rocket-rv64 -fuse-ld=lld -mabi=lp64 -march=rv64g -fvisibility=hidden -Ienv/v -Iisa/macros/scalar -I/usr/riscv64-elf/include -L/usr/riscv64-elf/lib env/v/entry.S -DENTROPY=0 env/v/*.c isa/rv64si/sbreak.S

clang is not happy with symbol defined in ld script.
env/v/entry.S:65:10: error: operand must be a bare symbol name
  la sp, (_end + 4096) - ((64 / 8) * 36)

Same issue in compiling riscv-tests with
clang -static -mcmodel=medany -nostdlib -nostartfiles -mno-relax --target=riscv64 -mcpu=rocket-rv64 -fuse-ld=lld -mabi=lp64 -march=rv64g -fvisibility=hidden -Ienv/v -Iisa/macros/scalar -I/usr/riscv64-elf/include -L/usr/riscv64-elf/lib env/v/entry.S -DENTROPY=0 env/v/*.c isa/rv64si/sbreak.S

clang is not happy with symbol defined in ld script.
env/v/entry.S:65:10: error: operand must be a bare symbol name
  la sp, (_end + 4096) - ((64 / 8) * 36)

I have a pending LLVM MC patch to fix this.

https://reviews.llvm.org/D57319 (LLVM 9) has fixed sw a0, (0)(t3)
https://reviews.llvm.org/D92293 will fix  la sp, (_end + 4096) - ((64 / 8) * 36)

Is this something that can be closed now?

No comments, so I'm closing this. Reopen if someone can show it is still failing.

Sorry to reopen this issue.
Now I am trying to compile la ra,_f2802x_usdelay-5, which generates a wrong result.
While when I compile la ra,_f2802x_usdelay+0-5, the result is OK.
LLVM version:13.0.1.
Source code:
_f2802x_usdelay:
	addi  sp,sp,-20
	sw    a1,16(sp)
	csrr  a1,0x7c0
	sw    a1,12(sp)
	csrr  a1,mstatus
	la ra,_f2802x_usdelay+0-5

Disassembly for la ra,_f2802x_usdelay+0-5:
00010000 <_f2802x_usdelay>:
   10000:	1131                	addi	sp,sp,-20
   10002:	c82e                	sw	a1,16(sp)
   10004:	7c0025f3          	csrr	a1,loop
   10008:	c62e                	sw	a1,12(sp)
   1000a:	300025f3          	csrr	a1,mstatus
   1000e:	00000097          	auipc	ra,0x0
   10012:	fed08093          	addi	ra,ra,-19 # fffb <SysPwrCtrlRegs+0x239b>

Disassembly for la ra,_f2802x_usdelay-5:
00010000 <_f2802x_usdelay>:
   10000:	1131                	addi	sp,sp,-20
   10002:	c82e                	sw	a1,16(sp)
   10004:	7c0025f3          	csrr	a1,loop
   10008:	c62e                	sw	a1,12(sp)
   1000a:	300025f3          	csrr	a1,mstatus
   1000e:	00000097          	auipc	ra,0x0
   10012:	ff708093          	addi	ra,ra,-9 # 10005 <_f2802x_usdelay+0x5>

Sounds like you should open the issue on LLVM, not here.

beat me to it. That is pretty strange LLVM behavior.
On Sat, Feb 26, 2022 at 12:21 AM Andrew Waterman @.***>
wrote:

Sounds like you should open the issue on LLVM, not here.
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/105#issuecomment-1051815984,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJWU256JCMBXPAIIHQLU5CEPVANCNFSM4MN6OXTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>

OK, I have posted this issue to llvm maillist.


[MNOE-688] Generate getter and setter for has_one relationship

This PR introduces 2 big changes.

The use of a has_one relationship helper
Stop loading everything in current_user

has_one relationship helper
This PR introduces an helper to easily create elements with relationships.
o = OrgaRelation.new(role: 'Super Admin')
o.relationships.user = User.new(id: user_id)
o.relationships.organization = User.new(id: organization_id)
o.save!

is replaced by
OrgaRelation.create!(user: user, organization: organization, role: 'Super Admin')

make current_user call smaller
In the current version, current_user is preloaded with a lot of dependencies, which are not always used. This can introduce problems of performances, caching and pagination. The objective of this PR is too stop relying on the autoload of theses dependencies and let the controllers manage the loading of what they need.

I will check the performance. The problem is that includes does not really work in v2, as it does not paginates. On top of that the MnoHub calls can be very long to build up the user with all the dependencies. It's definitely more interesting to make more smaller calls, easy to cache and reproduce than complicated calls with multiple includes.


Chromecast 3. disconnects every 15min

Chromecast dies every 15 minutes. I need to stop casting, recast, to fix it.
3 gen.

I've never seen the issue before today (07. okt). I'm watching live.
For the first 45 minutes I had to reconnect 3-4 times.
it disconnects and have to be recasted while the Chromecast is still on
It cuts from video to the BCC Media screen.
I then had to....
"Stop casting"...."start casting""
https://projects.bcc.no/desk/tickets/10533130/messages

how did this go @ høstcamp?

checking with user

The same user didnt have this issue during høstcamp

Couldnt find any cpu/memory issue in the receiver js.
Its probably an overheating issue if theyve had their chromecast running for a long time or something.
I tried with a cooled down chromecast now and its been running for 1.5hours.
for the record ive been testing this on an ultra because it was reproducable there too and couldnt find our 3rd gen

The only workaround I can see is to cap the quality to 720p or lower, but that would be unfortunate for those its working for.
I think for now we should just track how many support tickets are coming in and tell them "it might be overheated", and ask them to "try again after letting it cool and let us know if that helped.".
If many, we can try harder to resolve this.


Process mixin calls to constants.

Fixes #1272.
Process include and extend calls to constants. The following examples will generate the same documentation:
class MyClass
  include MyModule
end

# or

MyClass.include MyModule

[X] I have read the [Contributing Guide][contrib].
[X] The pull request is complete (implemented / written).
[X] Git commits have been cleaned up (squash WIP / revert commits).
[X] I wrote tests and ran bundle exec rake locally (if code is attached to PR).

Coverage decreased (-0.04%) to 93.722% when pulling 81d226099d7615859a1222420b9f155c1e8dbbd7 on castwide:constant-mixins into 890d4ad5f714856a138a5e2aae3b226255e74137 on lsegal:master.


Allow nested mirage modules

When I've tried to put a factory to the nested directory inside of mirage/factories/ directory I've got an error. Is there any reasons for this? With this change seems like everything works.

superseded by #1287


Issue compressing normal maps

Hi there, thanks for the awesome library! We are having some issues compressing normal maps, here's an example:
Input normal map

Compressed normal map

We tried some different parameters but could not manage to make the results significantly better. Then we found the following snippet in the readme:
"For tangent space normal maps, you should separate X into RGB and Y into Alpha, and provide the compressor with 32-bit/pixel input images. Or use the "-separate_rg_to_color_alpha" command line option which does this for you. The internal texture format that Basis Universal uses (ETC1S) doesn't handle tangent space normal maps encoded into RGB well. You need to separate the channels and recover Z in the pixel shader using z=sqrt(1-x^2-y^2)."
For reasons beyond our control this is likely not something we are capable of doing in the near future. But we thought it might still be worthwhile to ask and get confirmation.

Is it possible to improve the quality of the compressed normal map without splitting and shader reconstruction (i.e., are we missing some parameter or using the library the wrong way)?
Or do we have to adapt our pipeline to split and reconstruct (if using basis for normal maps)?

We are thankful for answers, have a pleasant day. :)
Reproducing:
Using latest basisu from git (895ee8ee7e, 2020-01-04), built with latest Visual Studio 2019 (16.4.3), release build.
Compress the original normal map using:
basisu original.png -q 128 -normal_map -comp_level 5 -mipmap -mip_renorm
Then decompress using:
basisu original.basis

@slimbuck Thanks for the input! It's good to get our suspicions confirmed, even if it wasn't the answer we hoped to ear.
I'm closing the issue now as it has been answered.


Improve GasLimit error to hold useful info

We should add extra information for the GasLimit error to also report data such as:

gas spent
gas limit
attempted charge

One of the solutions would be to extend the GasLimit variant and also include those numbers.

Done


Generate app key independent from wallets

Update key generation logic independent from the wallet.
Manage mnemonic outside of wallet for signing and generation.

Verified this issue on keeper app v 1.1.12(303)


The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #599      +/-   ##
==========================================
- Coverage   49.45%   48.79%   -0.66%
==========================================
  Files          73       73
  Lines        3371     3371
==========================================
- Hits         1667     1645      -22
- Misses       1546     1570      +24
+ Partials      158      156       -2

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/addhandler.go
71.13% <0.00%> (-5.16%)
:arrow_down:

cmd/ndm_daemonset/probe/eventhandler.go
40.00% <0.00%> (-3.64%)
:arrow_down:

cmd/ndm_daemonset/probe/udevprobe.go
63.53% <0.00%> (-2.77%)
:arrow_down:

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 e8d7bfd...70c81fd. Read the comment docs.

Thanks @akhilerm ! Yay CI is green.


Skip serializing optional fields.

Add skip_serializing_if attribute to all fields that are optional and can't be null - as of 3.8.0 version of LSP specification.

This should allow us to fix https://github.com/rust-lang-nursery/rls/issues/905.

Awesome!


Remove unused dependencies

@nobodyiam Hi, I am a user of project com.ctrip.framework.apollo:spring-boot-dubbo-provider:1.0-SNAPSHOT. I found that its pom file introduced 40 dependencies. However, among them, 12 libraries (30%) have not been used by your project (the redundant dependencies are listed below). This PR helps com.ctrip.framework.apollo:spring-boot-dubbo-provider:1.0-SNAPSHOT lose weight :) I have tested the revised configuration in my local environment. It is safe to remove the unused libraries.
Best regards
Redundant dependencies----

com.google.code.findbugs:jsr305:jar:3.0.2:compile
org.springframework.boot:spring-boot-starter-logging:jar:2.0.3.RELEASE:compile
com.github.spotbugs:spotbugs-annotations:jar:3.1.9:compile
aopalliance:aopalliance:jar:1.0:compile
ch.qos.logback:logback-core:jar:1.2.3:compile
org.apache.yetus:audience-annotations:jar:0.5.0:compile
ch.qos.logback:logback-classic:jar:1.2.3:compile
org.apache.logging.log4j:log4j-api:jar:2.10.0:compile
javax.annotation:javax.annotation-api:jar:1.3.2:compile
javax.inject:javax.inject:jar:1:compile
org.apache.logging.log4j:log4j-to-slf4j:jar:2.10.0:compile
org.slf4j:jul-to-slf4j:jar:1.7.25:compile

This is only a demo project so I think it's not a big deal to have some unused dependencies.

@nobodyiam Thanks for your feedback. Understand.


backward compatibility of pointwise activations

small fix for #19

i knew this was going to happen when I pushed the change, but cudnn was not being used by many back then, and the refactor HAD to be done as a cleanup, so pushed it.

when in doubt about something breaking, please feel free to ask, I usually know the answer about serialization breakage...especially if I broke it myself.
in nn/cunn i painfully make sure that the serialization is ALWAYS backward compatible, and cudnn will also be kept that way going forward.
however, breakage will occur when we move to R2 bindings again. I will create a tagged branch called R1, that can be used in that case.

thanks Soumith, will keep this in mind

did you mean that breakage will be caused by the same reason that this PR or you had something else in mind?
I've tested CUDNN v2 on ubuntu and os x, seems to work well, so I will add groups option to R2v2 and it lacks in-place pointwise activations, right?

https://github.com/soumith/cudnn.torch/commits/R2RC2
already added groups, and in-place

wow, missed that, cool!
I've tried to run forward in v2 with model saved in with v1, segfaults.

sigh!

working on it, fixed the segfault (it was padW and padH in Pooling), applying this PR


Sleep

Hi,
can you add that an NPC is sleeping?
You could then use it as a corpse.
Thanks in advance!

There already is a function that matches your prescriptiob.
The methode is called 'setSleeping'.


Comments have the same color as some commands

Why does the comment in between """ """  have the same color as  commands in the code? It   should have the same dark gray color as the comments using #
Can we please fix this

@musm sorry for the delayed response, what file type is this?


Update Ledger docs for new extract command

New commands for https://github.com/oasislabs/oasis-core/pull/2610

@david-yan feel free to update an merge these docs


Add global tree to database and display as circular tree

 Issue by stefelisabeth
Wednesday May 20, 2015 at 15:14 GMT
Originally opened as https://github.com/NESCent/dplace/issues/213

Only the global tree should be displayed as circular.
This is currently in progress.

Done in #102


CSUB-923: Provision on-demand self-hosted runners & generate weights from benchmarks

Only the last commit here is relevant!

Successfull test execution for benchmarks against testnet:
https://github.com/gluwa/creditcoin3/actions/runs/7291084292/job/19869614575?pr=135
The auto-generated commit updating weights is (may get overwritten once I change the target branch of this PR):
https://github.com/gluwa/creditcoin3/pull/135/commits/4606f01c3a9f2e44ed25090829e00656eeff95c0
CI jobs reporting PASS against the above auto-generated commit:
https://github.com/gluwa/creditcoin3/actions/runs/7291197492/job/19869649097?pr=135
The auto-generated PR against dev which brings updated weights.rs in sync is (also inspect CI jobs inside that PR):
https://github.com/gluwa/creditcoin3/pull/136


Vimeo player mute button doesn't work on mobile phones any more

I can see the same behaviour on https://plyr.io/#vimeo. On desktop everything works fine, but when i checked on a few different phones "mute" button doesn't work any more (it does not mute the video), even at plyr.io homepage.
Please fix this!

I had the same issue here, but only on IOS.


GITHUB_ACTOR does not support repos not owned by the action user

https://github.com/aormsby/Fork-Sync-With-Upstream-action/blob/c86999d81e8504c61a7eae1bd12ad04eae3449e5/upstream-sync.sh#L63
The GITHUB_ACTOR environment variable doesn't support connecting to an upstream repo owned by someone else. It assumes the action user is also the owner of the remote repo.
Fix:

Replace environment variable with a 'remote_user' input variable
Default input to GITHUB_ACTOR value

Oops, misunderstood my own code. It's been a while. This is not an issue at all. 😅

Oops, misunderstood my own code. It's been a while. This is not an issue at all. 😅


Implement itemsAttributes() for each items in checkboxlist and radiolist widget.

Proposal:
CheckboxList::create('test')
  ->items([1 => 'One', 2 => 'Two',])
  ->itemsAttributes([1 => ['disabled' => true]])
  ->render(),

Currently attributes are applied to all items.

How about:
CheckboxList::create('test')
  ->items([1 => (new Item('One')->disabled(), 2 => 'Two',])
  ->render(),

How about the following?
CheckboxList::create('test')
  ->items([1 => (new Item('One')->disabled(), 2 => 'Two',])
  ->render(),

in widgets you use arrays, to define the items.

Yes. We're going to release 2.0.0 so can change that.

I suggest name specificCheckboxAttributes instead of itemAttributes, because we set attributes it is special for checkboxes.


[benchmark] add max_workspace_size flags for tensorrt/pptrt backend

PR types(PR类型)

Description

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: DefTruth:x: qiuyanjunqiuyanjun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


10/Track enemy moves

Description
This pull request implements the special moves feature for enemies in our combat system. It adds functionality to track enemy action sequences and trigger special moves when specific combinations are completed. This enhancement adds depth to enemy AI and creates more dynamic combat scenarios for players.
Closes #354
How Has This Been Tested?
The following tests were conducted using logger to verify the changes:

[x] Test A: Verified that each enemy type correctly tracks its move sequence
[x] Test B: Confirmed that special moves are triggered when the correct combination is achieved
[x] Test C: Ensured that the specialmoveStore is properly reset after a special move is executed

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Implementation Details

Added UpdateSpecialMoveStore method to track enemy actions.
Implemented CheckSpecialMoveCombination method to verify if a special move combination has been achieved.
Created switch cases for different enemy types (FROG, CHICKEN, MONKEY, BEAR) to handle their specific move combinations.
Added logging statements for better debugging and monitoring.

resolved changes as per the review


"Paren" is missing from ast

I'm writing code generator using this great library. All I needed was to add ast and some other info to return of check.lua / check_or_throw().
return {
 events = events,
 per_line_options = per_line_options,
 line_lengths = utils.map(function(s) return #s end, chstate.source_lines),
 line_endings = line_endings
 ast = ast,
 comments = comments,
 code_lines = code_lines,
 source_lines = chstate.source_lines
}

Problem is if I parse a = (1+2)^3 then there will not be any "Paren" -tags in ast. it is impossible to generate correct source from ast without this info.

Paren nodes are currently created only when they change meaning of the AST, so only around function calls and varargs at the end of expression list. If they were created for all real parens in source then the rest of checking code would have to unwrap them all the time. This can be reworked if it makes the parser easier to use from outside though. Are you trying  to parse some code with the luacheck parser, make some changes in AST, and then generate back code?

Yes, I'm changing ast and then generating fixed Lua code (for example adding needed require's and removing unused variables). I will also change Lua code to json that will be converted to javascript or other languages. I'm also generating automatic tests when module functions are added or parameters are changed.
With parens (1+2)^2 == 9, without parens 1+2^2 == 5 so parens are changing the meaning of code. Also boolean -statements need parens.
Maybe you could add option to include parens?

I think that if you are generating code for other languages then it may be a good idea to add parens automatically during generation instead. I'm not 100% sure about javascript but I wouldn't rely on other languages having exactly same operator precedence. You could simply wrap all operator expressions in parens, there are also ways to add parens only when required by target language precedence and associativity rules.

Language do not matter in this case. But it is impossible to add any parens when there is no information about parens in ast. Easy way is to generate 2 ast structuresin 2 runs, one without parens for easy check code generation and other with parents.

I forgot about this one, but since 0.23.0 parser does preserve parens, they are removed in a different stage now.


Add SQLITE_ENABLE_DBSTAT_VTAB=1 to enable this virtual table

Fixes: #2965
Also verify you have:

[X] Read the contributions page.
[X] Submitted the CLA form, if you are a first time contributor.

Adding this compile option SQLITE_ENABLE_DBSTAT_VTAB=1 to sqlite enables the dbstat virtual table.


Switch position of 'confirm' button with 'cancel' button

I want to merge this change because...
Ref #1340
Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[x] JavaScript code quality checks pass: eslint.

Codecov Report

Merging #1357 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1357      +/-   ##
==========================================
- Coverage   82.07%   82.06%   -0.01%
==========================================
  Files         114      114
  Lines        5495     5492       -3
  Branches      637      637
==========================================
- Hits         4510     4507       -3
  Misses        830      830
  Partials      155      155

Impacted Files
Coverage Δ

saleor/settings.py
87.2% <0%> (-0.3%)
:arrow_down:

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 732970d...1c58644. Read the comment docs.


cigien: Watch theglobalkos(?:@gmail.com)?

cigien requests the watch of the watch_keyword theglobalkos(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

theglobalkos(?:@gmail\.com)? has been seen in 4 true positives, 0 false positives, and 0 NAAs.

theglobalkos(?:@gmail\.com)? has been seen in 4 true positives, 0 false positives, and 0 NAAs.

Rejected by Makyen in Charcoal HQ. Reason: 'Does not resolve resolve that it is already detected.'

Rejected by Makyen in Charcoal HQ. Reason: 'Does not resolve resolve that it is already detected.'


Garbage collect whole target/

Problem
With "cargo script", the target directory is "hidden" from the user, making it easy to leak when you delete your script.
If we move forward with rust-lang/rfcs#3371, a similar situation will happen for regular packages.
If I haven't touched a project in a long while but have run rustup update, there might be nothing of use left in target/, wasting space.
Sometimes I want to cargo clean all projects on my system (see #11305).
Proposed Solution
We should track in the GC data base a list of

Root-manifests (ie the Cargo.toml / cargo script associated with the target directory)
Target directory
(maybe) The path to the Cargo.lock for future potential work like #13137 without having to infer the Cargo.lock (special logic needed for cargo-script, feature requests exist for even weirder situations)

Note that neither of the two fields can serve as a unique / primary key.  If people use CARGO_TARGET_DIR=/tmp/cargo then multiple workspaces may point to the same target dir.  Likewise, people may end up with multiple target dirs for one workspace.
We need to track the Cargo.toml / cargo script because the workspace root is ambiguous when it comes to cargo scripts.
Example entries for CARGO_TARGET_DIR=/tmp/cargo :

id
workspace-manifest
target dir
timestamp

?
/foo/Cargo.toml
/tmp/cargo
?

?
/bar/Cargo.toml
/tmp/cargo
?

?
/baz/script.rs
/tmp/cargo
?

Example entries for rust-analyzer target dir

id
workspace-manifest
target dir
timestamp

?
/foo/Cargo.toml
/foo/target
?

?
/foo/Cargo.toml
/foo/target-ra
?

?
/bar/Cargo.toml
/bar/target
?

?
/bar/Cargo.toml
/bar/target-ra
?

?
/baz/script.rs
~/.cargo/target/...
?

Forms of cleanup

[ ] Delete target/ if unused for X time (this is in the "locally recreatable" category)
[ ] Delete all target/ (I just upgraded Rust, maybe rustup could suggest this)
[ ] Delete leaked target/ (workspace doesn't exist)

However, it might be transient (on a thumb drive).  Should we make this time based?

Notes
No response

@rustbot claim

Three ways for cargo clean script

clean target dir that workspace doesn't used for specific time.
clean target dir that workspace doesn't used anymore.
clean target dir in all workspace.

To clarify, those are use cases for why tracking of whole target/ could be useful.
I'd do a small tweak of wording in case this leads to confusion

clean target dir if it and/or its workspace hasn't been used for specific time
clean target dir for workspace that is no longer present
clean all target dirs

@epage, is this garbage collection specific to target directories under ~/.cargo/target that generated by -Zscript?

Currently, we do the batch-save in PackageSet::get_many, so we likely want this to live in a place before that call
https://github.com/rust-lang/cargo/blob/c9392675917adc2edab269eea27c222b5359c637/src/cargo/core/package.rs#L459
That gets called as part of PackageSet::download_accessible
https://github.com/rust-lang/cargo/blob/c9392675917adc2edab269eea27c222b5359c637/src/cargo/core/package.rs#L540
which gets called as part of resolving:
https://github.com/rust-lang/cargo/blob/c9392675917adc2edab269eea27c222b5359c637/src/cargo/ops/resolve.rs#L221
which gets called as part of create_bcx:
https://github.com/rust-lang/cargo/blob/c9392675917adc2edab269eea27c222b5359c637/src/cargo/ops/cargo_compile/mod.rs#L267
So our options are

Put in resolve (before download_accessible) so every operation gets it recorded
Put it in create_bcx before the resolve so only compiles get it recorded

A code path to model off of is
https://github.com/rust-lang/cargo/blob/c9392675917adc2edab269eea27c222b5359c637/src/cargo/sources/git/source.rs#L147-L157


Hotfix/4.8.1

This PR is ready for review.
Summary
This PR includes the following fixes:
Fix Pressure Tell Tale
Fix potential NPEs in SdlProtocolBase
Update SdlManager state when dispose() is called
Handle origin modules (Rpc Version 1.0 & Protocol Version 1.0)
CLA

[x] I have signed the CLA

Codecov Report

Merging #1063 into master will increase coverage by 0.11%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #1063      +/-   ##
============================================
+ Coverage     45.77%   45.89%   +0.11%
- Complexity     3682     3686       +4
============================================
  Files           433      433
  Lines         20212    20224      +12
  Branches       2177     2180       +3
============================================
+ Hits           9253     9281      +28
+ Misses        10496    10476      -20
- Partials        463      467       +4

Impacted Files
Coverage Δ
Complexity Δ

...java/com/smartdevicelink/proxy/rpc/TireStatus.java
100% <ø> (ø)
17 <0> (ø)
:arrow_down:

...smartdevicelink/proxy/rpc/DisplayCapabilities.java
73.91% <0%> (-7.04%)
20 <0> (ø)

...n/java/com/smartdevicelink/proxy/SdlProxyBase.java
6.46% <0%> (-0.01%)
16 <0> (ø)

...rtdevicelink/proxy/rpc/SoftButtonCapabilities.java
76% <0%> (-14.48%)
10 <0> (ø)

...icelink/managers/screen/BaseSoftButtonManager.java
57.2% <0%> (+2.96%)
41% <0%> (+1%)
:arrow_up:

...tdevicelink/managers/screen/BaseScreenManager.java
70.83% <0%> (+4.16%)
35% <0%> (+1%)
:arrow_up:

...ink/managers/permission/BasePermissionManager.java
73.91% <0%> (+5.07%)
42% <0%> (+1%)
:arrow_up:

...icelink/managers/lockscreen/LockScreenManager.java
32.43% <0%> (+9%)
6% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Update the box layout of the audio track component

Fixes

Related to #2910 by @fcoveram
Description

This PR uses the new VAudioControl component (replacing the old VPlayPause component) in the box layout of the audio track. This layout is used on the All content search page.
This component is updated to match the requirements in #2910 and the mockups
Testing Instructions

Search for something in "All content" view and check that the audio boxes work as expected: you can play the audio by clicking on the play button or pressing spacebar. If you click on the box itself, you should open the single result page. When you hover on the box, you should see the license.
There are two variants of this layout: one is used on the small screens. It doesn't show the waveforms. If you click on the play button on a small box, you should see the global audio player at the bottom of the screen.
The Audio box layout story should also work and match the Figma mockups.
Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I'm going to close this PR to re-open later.


How to get the usb camera stream？

I want to get the usb camera feed to agora_rtc_engineSDK

I get the stream of h264 through callback, but I need the original stream of the camera, can I support getting the original stream of the camera, such as NV12 or YUV420?

I get the stream of h264 through callback, but I need the original stream of the camera, can I support getting the original stream of the camera, such as NV12 or YUV420?

how did you got h264 stream?
did you modified this library?

Hi, can you provide more information on how you get the video stream from the callback? Looking at the source code seems that the method is not implemented. Can you provide a quick example? Thanks! @Nuyoahhjh

I get the stream of h264 through callback, but I need the original stream of the camera, can I support getting the original stream of the camera, such as NV12 or YUV420?

how did you got h264 stream? did you modified this library?

The author provided the callback to get the h 264 stream, and I modified the library to get the RGBA stream

Hi, can you provide more information on how you get the video stream from the callback? Looking at the source code seems that the method is not implemented. Can you provide a quick example? Thanks! @Nuyoahhjh

Here's my example： `Future _initializeUSBCamera() async { usbcameraController = UVCCameraController(); usbcameraController?.msgCallback = (state) { showCustomToast(state); }; usbcameraController?.CameraStreamCallback = (args) { frameRateTool(); if (!isJoined) { return; } print( "size:${args['data'].length} format:${args['format']} width:${args['width']} height:${args['height']}"); if (args['format'] == "RGBA") { _pushVideoFrame( args['data'], args['width'], args['height'], VideoPixelFormat.videoPixelRgba, DateTime.now().millisecondsSinceEpoch); } else if (args['format'] == "NV21") { _pushVideoFrame( args['data'], args['width'], args['height'], VideoPixelFormat.videoPixelNv21, DateTime.now().millisecondsSinceEpoch); } };
//摄像头状态回调
usbcameraController?.cameraStateCallback = (state) async {
  switch (state) {
    case UVCCameraState.opened:
      print("USB相机已打开");
      await Future.delayed(const Duration(seconds: 3));
      usbcameraController?.captureStreamStart();
      // _getCameraSizeList();
      // getCameraList();
      break;
    case UVCCameraState.closed:
      print("USB相机已关闭");
      usbcameraController?.captureStreamStop();
      break;
    default:
      if (state.toString().contains("ERROR")) {
        usbcameraController?.captureStreamStop();
      }
      break;
  }
};
setState(() {
  isCameraOpen.value = false;
  isUsbCameraOpen.value = true;
});
// await Future.delayed(const Duration(seconds: 6));
// _getCameraSizeList();

}` You need to modify the flutter plugin to add the CameraStreamCallback callback

can you provide CameraStreamCallback callback function's code?

I fork a copy and modify it so you can pull it down and use it as a native plugin. @rlatn1234 @Garw00

@Nuyoahhjh do you confirm that you can send the stream to Agora SDK now? It would be very helpful to see an example, we're trying to achieve the same result but the frame payload seems to be corrupted.

@Nuyoahhjh do you confirm that you can send the stream to Agora SDK now? It would be very helpful to see an example, we're trying to achieve the same result but the frame payload seems to be corrupted.

I've got about 15 frames


Add workflow to delete old docker images

Not sure how to test it, I put small number of days to actually have some images to remove for testing purpose

@dkropachev how can we test it? Do you have any idea?

@dkropachev how can we test it? Do you have any idea?

Sure, we need to kick out CICD for https://github.com/scylladb/gocql/pull/263 to publish an image, and then drop DELETE_AFTER_DAYS to -1 and run the script

Sure, we need to kick out CICD for https://github.com/scylladb/gocql/pull/263 to publish an image, which we already did and then drop DELETE_AFTER_DAYS to -1 and run the script

I meant the last part, how to run this script without merging this PR?

Sure, we need to kick out CICD for #263 to publish an image, which we already did and then drop DELETE_AFTER_DAYS to -1 and run the script

I meant the last part, how to run this script without merging this PR?

Run it as part of CICD, temporary add :
on:
  pull_request_target

to the workflow.

Created PR from the repo branch to be able to run the workflow https://github.com/scylladb/gocql/pull/269


How to generate "vqa-resnet-14x14.h5"?

The training process requires file "vqa-resnet-14x14.h5", how it is generated?

It seems that the preprocess-images.py from pytorch-vqa repo is relevant to generation of "vqa-resnet-14x14.h5". I used the "resnet-14x14.h5" generated with the script just mentioned for training, and it works.

Thanks for the nice catch! It seems like I forget to upload the scripts to extract features... I will upload this later but what you did is basically the exact thing that happened.
The feature extraction code is adapted from pytorch-vqa, which I directly used for VQA/VQA2  datasets, and visual genome and visual7w.


add --since 5m --timestamp to the command tip displayed in workload create/apply/update/get

Description of problem
In the output from workload create/apply/get, we recommend users run workload tail to view logs for their workload via a tip at the very end of the command output.
In a troubleshooting scenario, a problem has already occurred, so running the recommended tail command isn't likely to return log messages related to that problem.
We've received feedback from users asking how to view logs that were emitted prior to the moment the workload tail command is invoked. This indicates users aren't readily discovering that this feature is already supported.
And the fact that we're recommending a tail incantation which doesn't improve a user's ability to discover the feature while also not being likely to provide value as-is, is a UX miss.
We've also observed many users tail workload logs and include --timestamp.
Additionally, log message timestamps are considered valuable and we should include the flag in our recco.
Proposed solution
Include --timestamp --since 5m in the example command being recommended.
Example
Update the command tip from:
To see logs:   "tanzu apps workload tail foo"
To:
To see logs:   "tanzu apps workload tail foo --timestamp --since 5m"
Describe alternatives you've considered
recommend use of --since outside of the recommended command:
To see logs:   "tanzu apps workload tail foo" (include "--since" to view previous logs)

@heyjcollins Should we add --timestamp too?

@atmandhol - good suggestion. Done!

Discussed with @heyjcollins and decided to change the help to say 1h.


Struggling to make things work

Hi, there, struggling to make this plugin work.
Tried enabling debug and no log file has been created.
Tested mattermost API and works fine.
Also pointed the plugin as a temporary server and its not being initiated by your plugin.
More than happy to help work out the issue with you!
Thanks in advance

have you tested the communication between both endpoints, ie kanboard and mattermost?
If so, does each endpoint have the correct SSL settings in the URI?
IE if you are using SSL in kanboard then the first part of the link indicating the protocol for the webhook should be set appropriately:
no SSL enabled:   http://
yes SSL enabled:  https://


Intercom js loaded on window.onload instead of scriptElement.onload

Versions

Angular: 5.2.0
Angular CLI: 1.6.8
ng-intercom: 6.0.0-beta.2
Node.JS:
NPM: 5.8.0

Description
IntercomJS is loaded on window.onload event instead of scriptElement.onload, or by a function to init and load the js.
When intercom js is injected before window.load is fired, the module will load and work just fine.
But when intercom is injected after window.load was fired, the intercom JS won't be loaded because window.load already fired, and thus the component won't work.
To fix this issue, we had to use this workaround in our wrapper component to manually load the intercom js.
const intercomJsNotLoaded = !$(`script[src="https://widget.intercom.io/widget/${environment.intercomKey}"]`).length;
if (intercomJsNotLoaded) {
  // Load Intercom JS
  const loadIntercomJS: any = this.intercom.l(this.intercomConfig);
  loadIntercomJS.apply(this.intercom);
}

Possible solution
Is it possible to move the module loading through a function to be called manually or replace the window.onload with scriptElement.onload of the intercom js script injected?

I confirm this is a bug. The workaround work like a charm.

I'd definitely be open to seeing a PR for this issue. This seems like a simple enough fix, but currently the behavior lives inside the provider. I remember, it was formerly outside of the provider, and would execute before Angular's instantiation. I'd be open to seeing a new implementation of a fix for this issue, though. Thanks!

Sure. I'm working on a Fix, I'll send a PR when it's finished. Thanks!

I'm trying to push a new branch fix-injecting-intercom-script where the changes resides but I get this:
ERROR: Permission to CaliStyle/ng-intercom.git denied to tiberiumihai.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

You’ll want to fork the repo and do a pull request. Thanks!
Sent from out and about

On Aug 3, 2018, at 6:45 AM, tiberiumihai<EMAIL_ADDRESS>wrote:
I'm trying to push a new branch fix-injecting-intercom-script where the changes resides but I get this:
ERROR: Permission to CaliStyle/ng-intercom.git denied to tiberiumihai.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@wbhob @scott-wyatt I've added the PR with the fix. Can you take a look and let me know if it requires anything else? It seems that the bug is gone after this fix.

After merging and publishing to 6.0.0beta3, this issue is now fixed. Thanks!
Closing.


Time for a move?

@larskanis, @hsbt, @eregon
Thought I'd ping everyone about the idea of moving this repo to the Ruby org.  It performs package installation across all Actions platforms, and uses the code from ruby/setup-ruby to perform all of setup-ruby's operations.  Cross-platform example:
- name: Set up Ruby & Ragel
  uses: MSP-Greg/setup-ruby-pkgs@v1
  with:
    ruby-version: ${{ matrix.ruby }}
    apt-get: ragel
    brew: ragel
    mingw: ragel

Additionally, it has all the logic needed for Windows package installation.  Now that current Rubies use both the mingw64 and ucrt64 MSYS2 toolsets, the names of the packages are dependent on the toolset used.
If one has simple CI requirements where the OS is part of the matrix, this action simplifies the number of steps needed to install a package, regardless of OS.
Thoughts?

I think it would be fine to move the repo under the ruby org, but it's not my decision.
I trust @MSP-Greg's work on this, and I think it deserves being under the ruby org.
I think it still makes sense to keep it separate from ruby/setup-ruby as it has significant additional logic and many more inputs.

I agreed to transfer this repo to ruby org. If you hope it, Please transfer request to me (@hsbt) at first. After that, I will move it to ruby org, and add you and eregon and others as admin to its repository.


show actual time or clock?

Hi there,
is there an easy way to show the actual time in one of the corners? I miss the clock so I have to switch between some windows to see the system clock.  It would be great to have it somewhere.
//Martin

What you could do is the following:
(setq writeroom-mode-line '(" " global-mode-string))
(setq writeroom-local-effects '(display-time-mode))

That should display the time in the bottom left part of the screen, preceded by a space. For a nicer display, you'd need to customise writeroom-mode-line further, and/or the Customize group display-time.


Add non-human animal genetic diseases

Add OMIA species-specific genetic diseases using NCBIGene ids.
See spreadsheet: https://docs.google.com/spreadsheets/d/1aKuKc2yKy4Z1_yzntsOSi-Z-2V_AMKdmcwOZsVpUwtM/edit#gid=1157876657 'Species specific genetic diseases.'
To do:

[ ] Create patterns
[ ] Add NCBIGene into Mondo. (Checked with Monarch - NCBIGene is fine for this)
[ ] Add the terms from OMIA.

Discussion with @kevinschaper and @sabrinatoro on using NCBIGene for non-human animals in Monarch:
Kevin Schaper
Jan 26, 2024, 8:24 PM
to Sabrina, me
Sorry for not replying earlier.
I'm not 100% sure what it will look like for a gene to disease association to come in from Mondo (via Phenio, and the kgx translation into kg-phenio), but I think I should be able to pick up the association, and leave out the gene node coming in from the ontology in favor of getting it from NCBIGene.
I'm not sure yet about adding genes for 130 species - I might bring up that question in the huddle on Tuesday. It seems like an obvious yes that we should bring in any gene from any species if we have phenotype data for, but I don't know whether that also means that we should include every gene from each species, and then get interactions+expression etc for all of those genes, etc etc.
On Wed, Jan 24, 2024 at 5:09 PM Sabrina Toro<EMAIL_ADDRESS>wrote:
Thank you Katie,
Kevin, please be aware that we are only looking at OMIA for Mondo (ie, we are only adding the OMIA diseases in Mondo).
Monarch (in general) could get a lot of information for OMIA, but I think that it will be much easier to wait for OMIA to update their system to use ontology terms such as Mondo (for disease) and uPheno (or MP) for phenotypes. At this point, I think it would be way too much work for Monarch to ingest their data as they are.
We have been helping them with the transition to using ontology terms, and have been collaborating with them for a while now (on VBO and Mondo). We therefore have a very good interaction with the OMIA team (and they are extremely nice and approachable and will be very excited for Monarch to ingest their data). So whenever you are ready for it, we can facilitate the conversation (whenever you are ready).
On Wed, Jan 24, 2024 at 4:51 PM Katie Mullen<EMAIL_ADDRESS>wrote:
HI Kevin,
Thanks for the info! That's great if we can use NCBIGene since that is what OMIA uses. Of note is that OMIA's database currently covers diseases due to causal mutations in approximately 130 different species. Is it going to be okay to add that number of species from NCBIGene?
For ingesting the OMIA terms in Mondo, unfortunately, OMIA does not separate diseases from phenotypes, so we have manually curated a list of disease terms based on expert knowledge and having cross-species analogous disease terms (in humans) in Mondo. Our hope is after this initial manual ingest, we will be able to set up a synchronization pipeline with OMIA. I think there will still be some manual curation in determining disease vs phenotype, but this should be somewhat easy to resolve.
Sabrina did I miss anything?
Thanks!
Katie
On Wed, Jan 24, 2024 at 5:05 PM Kevin Schaper<EMAIL_ADDRESS>wrote:
Hi Katie!
Our gene ID preference is roughly, HGNC+MOD (ZFIN/MGI/RGD,etc) if available, and then NCBIGene for anything else. We aren't bringing in Felis catus yet, but we can really easily add it to our include list (we have pig/dog/cow/chicken from NCBIGene right now).
The part I'm less sure about is getting gene to disease associations from Mondo, should we try to figure out if we can get all of these associations from OMIA directly as an ingest? I've been holding off on looking at OMIA data because I know they have something in progress that will make it easier, but maybe it's far enough along that we should get started?
Kevin


Update index.md

Edits for clarity. We also don't say please, haha.

:tada: This PR is included in version 0.11.0-next.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Clarify the reason when the program fails because of an IO error

closes #190
This is a small intermediate step towards better error handling. Further discussion will take place in #191.

bors r+


Using healthchecks to determine down upstream hosts

If you are reporting any crash or any potential security issue, do not
open an issue in this repo. Please report the issue via emailing
<EMAIL_ADDRESS>where the issue will be triaged appropriately.
Title: Using healthchecks to determine down upstream hosts
Description:
I am currently researching possibility of using Envoy as a gateway proxy for the environment consisting of multiple services. Each service would have distinct URL prefix and will be served by corresponding pool of service nodes.
I started looking into Envoy's documentation and I also followed some of the scenarios on Try Envoy page: https://www.envoyproxy.io/try/
In particular I was interested in checking how Envoy handles detection of available service nodes using health checks: https://www.envoyproxy.io/try/detecting-service-health-with-healthchecks. I went through this scenario step by step, started Envoy and two test service nodes, was able to send several successful requests through Envoy, then triggered one of service nodes to pretend to be unhealthy, send some more requests and observed how initial request returned with errors and how some time later the unhealthy node was removed from pool of nodes. All worked as described in scenario.
I tried to replay this scenario on my own dev machine. Here is my configuration file:
admin:
    access_log_path: /tmp/admin_access.log
    address:
      socket_address: { address: <IP_ADDRESS>, port_value: 9901 }

static_resources:
    listeners:
    - name: listener_0
      address:
        socket_address: { address: <IP_ADDRESS>, port_value: 8080 }
      filter_chains:
      - filters:
        - name: envoy.http_connection_manager
          config:
            codec_type: auto
            stat_prefix: ingress_http
            route_config:
              name: local_route
              virtual_hosts:
              - name: backend
                domains:
                  - "*"
                routes:
                - match:
                    prefix: "/"
                  route:
                    cluster: targetCluster
            http_filters:
            - name: envoy.router
    clusters:
    - name: targetCluster
      connect_timeout: 0.25s
      type: STRICT_DNS
      dns_lookup_family: V4_ONLY
      lb_policy: ROUND_ROBIN
      hosts: [
        { socket_address: { address: <IP_ADDRESS>, port_value: 80 }},
        { socket_address: { address: <IP_ADDRESS>, port_value: 80 }},
        { socket_address: { address: <IP_ADDRESS>, port_value: 80 }},
        { socket_address: { address: <IP_ADDRESS>, port_value: 80 }},
      ]
      health_checks:
        - timeout: 1s
          interval: 1s
          interval_jitter: 1s
          unhealthy_threshold: 1
          healthy_threshold: 1
          http_health_check:
            path: "/health"

It is slightly modified version of config file from the scenario mentioned above, with 4 service nodes instead of 2 and with enabled Admin interface. I also modified health checks times and set all thresholds to 1 and all time values to 1s.
Next I started Envoy proxy and one test node:
$ docker run -d --name envoy -p 8080:8080 -p 9901:9901 -v /Users/i512296/Samples/Envoy:/etc/envoy -e loglevel=debug envoyproxy/envoy:v1.14-latest
5f1a7c1f765ea87caf6e407ddd1770fe86041320b55460695631e7f742bca347

$ docker run -d katacoda/docker-http-server:healthy
03166c245fcb23fe4cecfec64abaf82a3bb09127ecc8a1a67b7a37a74083c3df

$ docker ps -a
CONTAINER ID        IMAGE                                 COMMAND                  CREATED             STATUS              PORTS                                                       NAMES
03166c245fcb        katacoda/docker-http-server:healthy   "/app"                   2 minutes ago       Up 2 minutes        80/tcp                                                      quirky_gagarin
5f1a7c1f765e        envoyproxy/envoy:v1.14-latest         "/docker-entrypoint.…"   2 minutes ago       Up 2 minutes        <IP_ADDRESS>:8080->8080/tcp, <IP_ADDRESS>:9901->9901/tcp, 10000/tcp   envoy

Then I stared endless loop of requests which produced this output:
$ while true; do curl http://localhost:8080/; echo; sleep 1; done
upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure

A healthy request was processed by host: 03166c245fcb

upstream connect error or disconnect/reset before headers. reset reason: connection failure

A healthy request was processed by host: 03166c245fcb

upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure

A healthy request was processed by host: 03166c245fcb

upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure

A healthy request was processed by host: 03166c245fcb

upstream connect error or disconnect/reset before headers. reset reason: connection failure
upstream connect error or disconnect/reset before headers. reset reason: connection failure
...

It appears to me that Envoy is sending requests to the nodes which aren't available, haven't been started yet. So I used Admin interface to check for cluster's health:
$ curl -s localhost:9901/clusters | grep health
targetCluster::<IP_ADDRESS>:80::health_flags::healthy
targetCluster::<IP_ADDRESS>:80::health_flags::/failed_active_hc
targetCluster::<IP_ADDRESS>:80::health_flags::/failed_active_hc
targetCluster::<IP_ADDRESS>:80::health_flags::/failed_active_hc

It looks like Envoy knows that only one service node is healthy and other 3 failed their health checks but it keeps forwarding requests to those unavailable nodes anyway. I left my request loop running for considerable time and those connection failure errors never disappeared.
When I started another node it still didn't help and I still could see connection failure errors. Situation improved when I started third service node, then connection failure disappeared. However if I terminate those two nodes which I started before and leave only one node running then the situation returns and I can see connection failure errors again.
Did I make some configuration mistakes here or am I missing something?
[optional Relevant Links:]

Any extra documentation required to understand the issue.

@aqua777 have you looked to see if https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/load_balancing/panic_threshold.html?highlight=panic applies here?

@htuch thanks for a hint, let me take a look.

@htuch this worked for me, thanks again for pointing me in right direction.


Fix actionblocker jank

So diseases and forensics have their own snowflake interaction. The best part of this jank is that it gets called during actionblocker, you know, when I can call CanX as much as I want but now internally it's actually doing an interaction which is a hidden side-effect and bad.
Seeking opinions on whether raising InteractHandEvent on the user as well should be done.

The event absolutely shouldn't be raised during the CanInteract check. But AFAIK the events still need to exist, because there isn't a clear substitute. I don't think there's any neat solution.
The AfterInteract and AfterInteractUsing events are badly named, they are not guaranteed to be raised after an interaction, they're just interactions that are attempted after other interactions. The ordering is:

InteractUsing (directed at target)
AfterInteract (directed at the used item)
AfterInteractUsing (directed at target)

If any interaction is handled, the subsequent events don't get raised. AfterInteractUsing exists in case you need to have a low-priority interaction facilitated by the target entity that should be checked after first trying AfterInteract interactions. So just using AfterInteract in place of UserInteractedWithItemEvent is insufficient. Clearly AfterInteract & AfterUserInteract need to be renamed, but I have NFI how to neatly handle priorities. Subscription ordering works well enough if you have a single event, but not for multiple events directed at different entities.
In terms of how forensics and diseases work, I think you will just need a generic informative event that gets raised after a successful interaction occurred. So basically the current events, but instead of being raised in CanInteract they need to be raised whenever an InteractUsing, AfterInteract, AfterUserInteract, InteractHand or ActivateInWorld event gets raised & handled, and whenever some verbs get executed.
For verbs its also quite messy, and probably just needs to be a bool that can be specified per verb which determines whether the events get raised when the verb is executed, The verb category & class should be able to provide defaults. E.g., examine class verbs and debug category verbs shouldn't trigger diseases.  I guess the pulling & drag n drop system should just handle raising their own got-interacted-with events?

Seeking opinions on whether raising InteractHandEvent on the user as well should be done.

It should at least be a separate event with a different name, and should only get raised if the other did not get handled. Or it needs to somehow make it clear that this is an informative event, and not intended to trigger additional interactions.


What does it mean to adopt a drain pop up

This pop up needs to be updated with the most current information.
Can be found here:
https://docs.google.com/document/d/1jDR7CmPc3Mi8NA7SUzjJTIkeebwmx8cQoTIJq857_OU/edit
Please keep the current linked social media icons in this pop up.

Resolved


Fix navigation collapse on IE11

Motivation
Cross browser compatibility sets good example in open source projects. There are few issues happening in Internet Explorer 11 that blocks the usage of the website for its users.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
This PR will mostly be about UI changes that would affect CSS only. Plans for testing would be testing the UI on IE11 and another major browser (like Chrome) to ensure the updates doesn't break the new browsers as well as testing on mobile device.
Related issues
https://github.com/facebook/Docusaurus/issues/585

Before:

After:

The reason:
flex: 1 on IE11 automatically set the flex-basis value to 0 without units on IE11. This leads to the collapse of the element and disallowing it from expanding and taking the size it should. This is a known issue on IE11 and usually countered by using flex: 1 1 auto in order to prevent that behavior

My bad @yangshun
Should I remove the project and fork it again to avoid having the other PR commits on this one as well?

@ahmadalfy From your screenshots, the font looks bad. The screenshot in #585 doesn't look that bad. We want to use the system font if possible. Could you help to look into it and fix it if possible? Ideally we want to emulate GitHub, which also uses system fonts.

@ahmadalfy It's fine, I rebased and it's ok now. You'll have to branch off latest master for your new PRs or rebase on master in future.

Yes I already fixed that and it is coming on the next commit ;)

Please title your PRs according to the specific fix you are making 😄

Great work @ahmadalfy .. much appreciated!

@zenflow most welcomed <3


Allow building this repo with VS 15.x

This is more like an FYI PR. We need this change because our CI environments have updated or will update soon to VS 15.8

cc @BrennanConroy this should unblock SignalR stuff.

I was concerned because our master branch already allows VS 15.8 and is still failing https://github.com/aspnet/Razor/blob/master/korebuild.json#L8

Expanding the range to include 15.8 was one piece of the puzzle. Updating to 15.8 uncovered a new problem. In 15.8, NuGet more correctly handles per-project restore sources. One of the projects in this repo was misconfigured and had no restore sources set. I've resolved this in https://github.com/aspnet/Razor/commit/f703a288ddfdfd90f5d1c0bdd5b82b7b064e6518. This explains why it was passing for you locally -- you already had packages restored into your machine's cache, but CI is a clean machine.


Getting predicted phenotypes from validate()

Hi!
Thank you for your help on crosspred thread, I think this is more lassosum related so I will just put it here.
Is there a way to obtain the predicted phenotypes from the PGS and use them to calculate the specifcity, sensitivity and balanced accuracy for model comparison? I figured the pheno in result.table is the true phenotype we input for the test dataset.
Thank you for the help

No, the model you use to predict the pheno type is beyond the scope of lassosum, so you should fit the model yourself.


Switch to uberon-base for import.

Lots of uberon inferences in the inference diff is okay for now. Should go away if we move to a single mega-import.

Putting on hold; will work on mega-import due to missing inferences.


Update pull_request_template.md once more

Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[x] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[x] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[x] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Label: To cleary categorize this pull request, assign a relevant label to the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

Added a new Screenshots or Videos section to the pull-request template itself, allowing contributors to attach relevant visual aids to demonstrate changes.

Added

The Screenshots or Videos section to the pull-request template.

Changed

Refactored some wording in the PR template.

Deprecated

[List any deprecated functionality or features that have been removed]

Removed

[List any removed features, files, or functionalities]

Fixed

[List any fixes, corrections, or bug fixes]

Security

[List any new or updated security-related changes, including vulnerability fixes]

Breaking Changes

BREAKING CHANGE: [List any breaking changes affecting compatibility or functionality]

Additional Information

This change aims to enhance the communication and understanding of changes within the pull request by providing a dedicated section for visual aids.

Screenshots or Videos

[Attach any relevant screenshots or videos demonstrating the changes]

LGTM!


Cant figure out how to change font size of text in icon

[x] Review the documentation: https://github.com/oblador/react-native-vector-icons
[x] Search for existing issues (including closed ones): https://github.com/oblador/react-native-vector-icons/issues

Environment
Mac OS High Sierra 10.13.6
iPhone X Simulator running iOS 12.0
RNVI version ^4.6.0

Description
Im trying to figure out how to decrease the text size in the icon. Have tried using fontSize, allowFontScaling, textStyle, etc but no luck. Here's an image of what it currently looks like right now, and I would like the i to be smaller.
https://www.dropbox.com/s/bvf4mbg1ozxy1z9/Screenshot 2018-11-08 10.15.50.png?dl=0
Demo
Heres what I have right now:

<Icon
color={GlobalStyles.colors.blue}
underlayColor={GlobalStyles.colors.blue}
name="info-circle"
type="font-awesome"
size={28}
fontSize={4}
//textStyle={{ fontSize: 2 }}
allowFontScaling={true}
containerStyle={styles.rowIcon}
/>

@bear-junior is correct! There is no internal font to change the size of since the glyph is one, single vector.
What you could do is some position: absoluteing and just create your own icon:
<View
  style={{
    alignItems: 'center',
    justifyContent: 'center',
    width: 50,
    height: 50
  }}>
  <View
    style={[
      StyleSheet.absoluteFill,
      { borderRadius: 25, backgroundColor: '#6666cc' }
    ]}
  />
  <Text
    style={{
      color: 'white',
      fontWeight: 'bold',
      fontSize: 30,
      position: 'absolute',
      fontFamily: 'courier'
    }}>
    i
  </Text>
</View>

creates this:

Note that the font-family could create issues across devices (iOS vs Android primarly), only tested this on iOS, but it's a start you can tinker with 🙂
Closing this though as it's impossible with this implementation!

OK, in the readme I saw this, am I reading that wrong?
https://github.com/oblador/react-native-vector-icons#styling

You can change the fontSize property (along with others) yes but since the entire icon (not just the text inside) is considered a character it will change the size of the background as well 🙂

I had the same problem and i solved it using style={{fontSize:30}}  it works perfectly 😊😊
<Icon name='name' style={{fontSize:30}} />


Add CSRF protection

Pull Request Test Coverage Report for Build 132430719

14 of 18 (77.78%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 57.121%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/front-end/server/users_handlers.go
0
4
0.0%

Totals

Change from base Build 131525762:
0.9%

Covered Lines:
369

Relevant Lines:
646

💛  - Coveralls

https://github.com/gira-games/front-end/issues/9


Implement the NamedTupleInterface

I want to use NamedTuples in the workshop but we do not have a NamedTupleInterface. The implementation is pretty straightforward and could be included in Liesel's next release. It needs to be documented and tested.
class NamedTupleInterface:

    def __init__(self, log_prob_fn: Callable[[NamedTuple], float]):
        self._log_prob_fn = log_prob_fn

    def extract_position(
        self, position_keys: Sequence[str], model_state: NamedTuple
    ):
        return {key: getattr(model_state, key) for key in position_keys}

    def log_prob(self, model_state: NamedTuple) -> float:
        return self._log_prob_fn(model_state)

    def update_state(self, position, model_state: NamedTuple):
        new_state = model_state._replace(**position)
        return new_state

Just out of curiosity: Why not use a dataclass or dict?

Dataclasses aren't pytrees and I want to keep the code simple. I could use a dict thats true. I didn't because a namedtuple protects you from misspelling the keys which a dict wouldn't in the dict. I guess it is easy to change in case we want to do that.


Ability to generate final Biconomy independent transaction payload

Ideal flow:
    // In Backend
    let value = ethers.utils.parseEther('0.000000001')
    let tx = {
        to: "0x48C6F6b6828145E051aAf66dFaA3798450176473",
        data: "0x",
        value: value
    }

    let feeQuotes = await smartAccount.prepareRefundTransaction({transaction: tx})
    transaction = await smartAccount.createRefundTransaction({transaction: tx, feeQuote: feeQuotes[0]})

    let web3js_payload = await smartAccount.createWeb3JSPayload(transaction)
    .
    .
    .
    // Send web3js_payload to frontend
    .
    .
    .
    // In Frontend
    // sign and send transaction using web3.js
    let receipt = await web3.eth.sendTransaction(web3js_payload)

This worked:
const signature2 = await eoaSigner._signTypedData({ verifyingContract: smartAccount.address, chainId: (await eoaSigner.provider.getNetwork()).chainId}, EIP712_WALLET_TX_TYPE, transaction)

one payload is generated and signed, you can use sendSignedTransaction() to dispatch.