Error/API responses return empty body/data content

Sails version: 0.12.13
Node version: 6.11.0
NPM version: 3.10.10
Operating system: Ubuntu 16.04

I have a Sails.js application in a production environment that is also using SSL. The application appears to be working fine, except for when I am expecting responses from the API that my Angular application will act on. After further discovery, I found that my app is returning the correct status codes, but the response body is completely empty. It also looks like the headers are being completely stripped down as well. See screenshot below.

A few more things. I can also verify that my sails.config.keepResponseErrors configuration boolean is set to true within my production.js file. I have also attempted to setup the application in both an Apache and NGINX environments with the exact same issue. I do have a testing environment on a Heroku stack that works just fine.
This is what my Apache config looks like
<VirtualHost _default_:443>
        ServerName XXX
        DocumentRoot /var/www/html
        ServerAdmin XXX

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined

        <Proxy *>
                Order deny,allow
                Allow from all
        </Proxy>

        KeepAlive On
        KeepAliveTimeout 5
        MaxKeepAliveRequests 100

        #SSL Stuff
        SSLEngine On
        SSLProxyEngine On
        SSLCertificateFile XXX.crt
        SSLCertificateKeyFile XXX.key

        ProxyPreserveHost On
        ProxyRequests Off
        ProxyPass / http://<IP_ADDRESS>:8080/
        ProxyPassReverse / http://<IP_ADDRESS>:8080/
</VirtualHost>

Due to these symptoms, I am missing something very drastic on my server configuration, or there is something up with the production settings. Any help would be appreciated. Thanks.

Ok, fixed!

Just in case, here is my NGINX configuration:
server {
    listen 80;

    server_name blah.com;
    rewrite ^/(.*) https://blah.com/$1 permanent;
}
server {
    # SSL configuration
    #
    listen 443 ssl default_server;
    ssl_certificate XXX;
    ssl_certificate_key XXX;
    ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
    ssl_prefer_server_ciphers on;
    ssl_ciphers 'EECDH+AESGCM:EDH+AESGCM:AES256+EECDH:AES256+EDH';

    root /var/www/html;

    # Add index.php to the list if you are using PHP
    index index.html index.htm index.nginx-debian.html;

    server_name blah.com;

    location / {
            proxy_pass http://<IP_ADDRESS>:8080;
            proxy_http_version 1.1;
            proxy_set_header Upgrade $http_upgrade;
            proxy_set_header Connection 'upgrade';
            proxy_set_header Host $host;
            proxy_cache_bypass $http_upgrade;
    }
}

This was solved by commenting out line 38 with the api/responses/badRequest.js file:
  // Only include errors in response if application environment
  // is not set to 'production'.  In production, we shouldn't
  // send back any identifying information about errors.
  if (sails.config.environment === 'production') {
    //data = undefined;
  }


Dokcker - UpdateWebInventory create json files if dosnt exist

Short Description
UpdateWebInventory dosnt seem to create the json files, that it needs to write to, if i create a empty json file with content: [] it registers the inventory, but not player_stats ( i guess it expects the player_stats data to be in the file, so it can append )
Then we cant see lvl/xp on the web gui
Possible solution
if the inventory.json file dosnt exist, create it with default content
How it would help others
For us that use the gui, and started a new bot, we can again see the xp/lvl info

maybe this is more a bug when i think about it.
recreate as bug or what do you guys say

I got the same error https://github.com/PokemonGoF/PokemonGo-Bot/issues/4682#issuecomment-242310674

Looks like a bug. Not sure where this file was originally created but appears not to be doing it now. PR to follow.

PR #4740 submitted

Hi
this dosnt solve the player_stats issue

@MartinTerp It sure doesn't. See PR #4757 for this fix.

@Gobberwart thanks for clearing this up :)


[bug] bad output when using is:issue as a filter

sections.yml
- title: My Pull Requests
  repos:
    - dlvhdr/gh-prs
  filters: is:issue

I think you just need to drop occurrences of "is:issue" filter on filters

I whish I could use this awesome extension with issues as well :/ but yeah, the name implies that's only for PRs, I know.

this was fixed on v2.0.0. thanks.


Crash on adding GPUParticles2D to scene when using Compatibility renderer on macOS

Tested versions

4.3.dev5
4.2.2.stable
4.2.1 stable
4.1.4 stable

System information
Godot v4.2.2.stable - macOS 14.4.1 - GLES3 (Compatibility) - Apple M2 - Apple M2 (8 Threads)
Issue description
While using the Compatibility renderer, if a GPUParticles2D node is added to the scene, the editor crashes.
Steps to reproduce

Create or open a project that is using the Compatibility renderer.
Add GPUParticles2D to the scene
Voilà, your engine crashes.

Minimal reproduction project (MRP)
N/A

Thank you for your report. This same issue has been reported already and is tracked by https://github.com/godotengine/godot/issues/72469
We like to keep bug reports in one place, so I'm going to close this issue in favour of https://github.com/godotengine/godot/issues/72469


feat(inventory): Add secondary smart-equip star

About the PR

Adds a secondary quick-remove marker to the grid inventory to indicate the next-in-line item to be quick-removed.
See #22236
Why / Balance

Provides greater visual indication of the ordering of grid inventory items. This change assists players checking whether items are in the desired order when planning to quick-remove more than one item sequentially.
Technical details

Replaces the marking bool with an enum for each grid piece to indicate a first or second quick-remove marking. During draw, the corresponding texture is selected.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Inventories now show a secondary smart-equip star.

Open for feedback, especially for the icon itself. Here is a mini bronze star replacement.

Open for feedback, especially for the icon itself. Here is a mini bronze star replacement.

Bronze star too dark to see (especially with welder), maybe make the outline brighter

Silver star?

Silver star?

I considered silver but dismissed it because I thought it would be too bright and detract from the primary gold star. However, in practice, it seems very nice.

Matched luminosity. I might keep the edges lighter than the corresponding gold star since the sprite is smaller and the "inside" has fewer pixels to shine with.

maybe make it the same size as the gold star?
i don't really see a reason to make it smaller if it already has a distinct color

maybe make it the same size as the gold star?
i don't really see a reason to make it smaller if it already has a distinct color

I chose a different shape for color vision accessibility.

personally, I think the small silver star was the best looking, and the best at conveying what it meant to the player. The order usually goes Gold > Silver > Bronze anyway.


Scrolling back in input history loses current edit

If you start to compose a message and then navigate backward into the input history by pressing Up and then come back to your current input by pressing Down, it is gone. (Expected behavior: it is preserved.)  Fix: preserve it before beginning history navigation.

CC @dagit


BUG:  install with CUDA support not working (on any OS)

Issue Description
Can anyone confirm that install with CUDA support (and, therefore, GpuTree explainer) is working in shap 0.46.0? I tried on BOTH Windows and Ubuntu 22.04, and in neither case it was a success (for various reasons).
Minimal Reproducible Example
python3 setup.py install --user

Traceback
Ubuntu 22.04:

python3 setup.py install --user
Attempting to build SHAP: with_binary=True, with_cuda=True (Attempt 1)
NVCC ==> /usr/local/cuda-12.5/bin/nvcc
Compiling cuda extension, calling nvcc with arguments:
['/usr/local/cuda-12.5/bin/nvcc', '-allow-unsupported-compiler', 'shap/cext/_cext_gpu.cu', '-lib', '-o', 'build/lib_cext_gpu.a', '-Xcompiler', '-fPIC', '--include-path', '/usr/include/python3.10', '--std', 'c++14', '--expt-extended-lambda', '--expt-relaxed-constexpr', '-gencode=arch=compute_60,code=sm_60', '-gencode=arch=compute_70,code=sm_70', '-gencode=arch=compute_75,code=sm_75', '-gencode=arch=compute_75,code=compute_75', '-gencode=arch=compute_80,code=sm_80']
running install
/usr/lib/python3/dist-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
warnings.warn(
/usr/lib/python3/dist-packages/setuptools/command/easy_install.py:158: EasyInstallDeprecationWarning: easy_install command is deprecated. Use build and pip and other standards-based tools.
warnings.warn(
running bdist_egg
running egg_info
writing UNKNOWN.egg-info/PKG-INFO
writing dependency_links to UNKNOWN.egg-info/dependency_links.txt
writing top-level names to UNKNOWN.egg-info/top_level.txt
reading manifest file 'UNKNOWN.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
no previously-included directories found matching '.github'
no previously-included directories found matching 'appveyor'
no previously-included directories found matching 'data'
no previously-included directories found matching 'docs'
no previously-included directories found matching 'javascript'
no previously-included directories found matching 'notebooks'
warning: no previously-included files found matching '.git-blame-ignore-revs'
warning: no previously-included files found matching '.gitattributes'
warning: no previously-included files found matching '.gitignore'
warning: no previously-included files found matching '.pre-commit-config.yaml'
warning: no previously-included files found matching '.readthedocs.yml'
warning: no previously-included files found matching 'CONTRIBUTING.md'
warning: no previously-included files found matching 'requirements.txt'
adding license file 'LICENSE'
writing manifest file 'UNKNOWN.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_ext
building 'shap._cext_gpu' extension
x86_64-linux-gnu-gcc -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -I/usr/local/lib/python3.10/dist-packages/numpy/core/include -I/usr/include/python3.10 -c shap/cext/_cext_gpu.cc -o build/temp.linux-x86_64-3.10/shap/cext/_cext_gpu.o -fPIC
In file included from shap/cext/_cext_gpu.cc:5:
shap/cext/tree_shap.h: In function ‘void dense_tree_interactions_path_dependent(const TreeEnsemble&, const ExplanationDataset&, tfloat*, tfloat (*)(tfloat, tfloat))’:
shap/cext/tree_shap.h:1321:36: warning: comparison of integer expressions of different signedness: ‘unsigned int’ and ‘int’ [-Wsign-compare]
1321 | for (unsigned l = 0; l < amount_of_unique_features; ++l) {
| ~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
shap/cext/tree_shap.h:1351:36: warning: comparison of integer expressions of different signedness: ‘unsigned int’ and ‘int’ [-Wsign-compare]
1351 | for (unsigned k = 0; k < amount_of_unique_features; ++k) {
| ~~^~~~~~~~~~~~~~~~~~~~~~~~~~~
x86_64-linux-gnu-g++ -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-Bsymbolic-functions -g -fwrapv -O2 -Wl,-Bsymbolic-functions -g -fwrapv -O2 -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 build/temp.linux-x86_64-3.10/shap/cext/_cext_gpu.o -Lbuild -L/usr/local/cuda-12.5/lib64 -l_cext_gpu -lcudart -o build/lib.linux-x86_64-3.10/shap/_cext_gpu.cpython-310-x86_64-linux-gnu.so
creating build/bdist.linux-x86_64/egg
creating build/bdist.linux-x86_64/egg/shap
copying build/lib.linux-x86_64-3.10/shap/_cext.cpython-310-x86_64-linux-gnu.so -> build/bdist.linux-x86_64/egg/shap
copying build/lib.linux-x86_64-3.10/shap/_cext_gpu.cpython-310-x86_64-linux-gnu.so -> build/bdist.linux-x86_64/egg/shap
creating stub loader for shap/_cext.cpython-310-x86_64-linux-gnu.so
creating stub loader for shap/_cext_gpu.cpython-310-x86_64-linux-gnu.so
byte-compiling build/bdist.linux-x86_64/egg/shap/_cext.py to _cext.cpython-310.pyc
byte-compiling build/bdist.linux-x86_64/egg/shap/_cext_gpu.py to _cext_gpu.cpython-310.pyc
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
writing build/bdist.linux-x86_64/egg/EGG-INFO/native_libs.txt
zip_safe flag not set; analyzing archive contents...
shap.pycache._cext.cpython-310: module references file
shap.pycache._cext_gpu.cpython-310: module references file
creating 'dist/UNKNOWN-0.0.0-py3.10-linux-x86_64.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing UNKNOWN-0.0.0-py3.10-linux-x86_64.egg
removing '/home/test/.local/lib/python3.10/site-packages/UNKNOWN-0.0.0-py3.10-linux-x86_64.egg' (and everything under it)
creating /home/test/.local/lib/python3.10/site-packages/UNKNOWN-0.0.0-py3.10-linux-x86_64.egg
Extracting UNKNOWN-0.0.0-py3.10-linux-x86_64.egg to /home/test/.local/lib/python3.10/site-packages
UNKNOWN 0.0.0 is already the active version in easy-install.pth

Installed /home/test/.local/lib/python3.10/site-packages/UNKNOWN-0.0.0-py3.10-linux-x86_64.egg
Processing dependencies for UNKNOWN==0.0.0
Finished processing dependencies for UNKNOWN==0.0.0

Windows:

_Warning: Package 'shap.cext' is absent from the packages configuration.
!!

    ********************************************************************************
    ############################
    # Package would be ignored #
    ############################
    Python recognizes 'shap.cext' as an importable package[^1],
    but it is absent from setuptools' `packages` configuration.

And the install step finishes. But then

explainer = shap.explainers.GPUTree(model, X)
shap_values = explainer(X)
fails with

File ~\AppData\Roaming\Python\Python311\site-packages\shap-0.0.0-py3.11-win-amd64.egg\shap\explainers_gpu_tree.py:14
6 from ._tree import (
7 TreeExplainer,
8 _xgboost_cat_unsupported,
9 feature_perturbation_codes,
10 output_transform_codes,
11 )
13 try:
---> 14 from .. import _cext_gpu # type: ignore
15 except ImportError as e:
16 record_import_error("cext_gpu", "cuda extension was not built during install!", e)

ImportError: DLL load failed while importing _cext_gpu: The specified module could not be found.

Expected Behavior
the package should be correctly compiled, versioned and installed
Bug report checklist

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest release of shap.
[X] I have confirmed this bug exists on the master branch of shap.
[X] I'd be interested in making a PR to fix this bug

Installed Versions
0.46.0

This works for me. Are you executing the python setup.py install --user from within the shap directory? I am using ubuntu 24.04 LTS.


./bootstrap-vcpkg.sh  gives me Exec format error while following Azure Embedded C IoT

Hello,
I am trying to follow the instructions given here for the vcpkg section. When I follow the instruction using vcpkg version tags/2020.06, the ./bootstrap-vcpkg.sh gives me the following error
./bootstrap-vcpkg.sh: 252: /home/uboscubo/EmbeddedDevTutorial/Chap11_Exploring_BBB/vcpkg/scripts/bootstrap.sh: /home/uboscubo/EmbeddedDevTutorial/Chap11_Exploring_BBB/vcpkg/downloads/tools/cmake-3.17.2-linux/cmake-3.17.2-Linux-x86_64/bin/cmake: Exec format error
Is there anything wrong with this specific tag? Azure IoT recommended the using the tag 2020.06 as documented.
Thank you
I tried asking Azure IoT here

I suspect it has to do with something with cmake version 3.17.2 and 2020.06 vcpkg.sh?

Repro:

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


Different wasm file path in Chrome and Firefox

The integration of web-ifc-viewer in my project works fine in Firefox but not completely in Chrome.
Firefox loads the .wasm file from root: my-project.com/web-ifc.wasm
Chrome loads it prepending "js/": my-project.com/js/web-ifc.wasm
Even if I use setWasmPath('my-dir') chrome still prepends "js/":
my-project.com/js/my-dir/web-ifc.wasm (chrome)
my-project.com/my-dir/web-ifc.wasm (firefox)
Finally I could resolve it setting setWasmPath('../'), so all browsers loads wasm file from root

This is a web-ifc bug I guess. SetWasmPath is not working properly.

Yep, but don't worry, I'll solve it there!

After some fiddling, we think that the easiest solution is documenting this in the docs as a caveat, as we are reverse engineering the WASM file in order to be able to change the wasm file, and we don't have a lot of control over it! Anyway, let us know of any other suggestions.


Convert into Org

Instead of doing everything with cowboy commits on the main repo, collect everything under an org and let crates evolve independently.
That way all changes need to be merged through a PR.

I'm calling this a done job now. Repos have been shifted, CI and docs set up, and a redirect repo set up over the old one


chore(lint): lint vue-app templates

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #4689 into dev will not change coverage.
The diff coverage is 50%.

@@           Coverage Diff           @@
##              dev    #4689   +/-   ##
=======================================
  Coverage   90.76%   90.76%
=======================================
  Files          67       67
  Lines        2242     2242
  Branches      549      549
=======================================
  Hits         2035     2035
  Misses        188      188
  Partials       19       19

Impacted Files
Coverage Δ

packages/utils/src/serialize.js
88.88% <50%> (ø)
: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 30fef5d...5a5f91f. Read the comment docs.

@manniL it seems I automatically dismissed your review before I could read what it was about and cant seem to get that review back. Could you please share it again if its still applicable?


Add function to override default configuration

The example for INI file currently contain user-specific path, e.g.,
https://github.com/felixbur/nkululeko/blob/8581a0e8fd0aeadfc53a2d4f49279cec55fbbf2c/tests/exp_emodb_os_mlp.ini#L9
I want to request that the user can override the config without editing the config. The suggested CLI argument is -o or --override . So the command will be,
 python -m nkululeko -i tests/exp_emodb_os_mlp.ini -o "exp_emodb_os_mlp.data.emodb=/home/bagus/emodb"

Benefits:  No need to modify INI file for testing/trying Nkululeko.
Also, should the path variable be consistent across datasets? In that emodb the dataset path is named "emodb" but in other datasets (Android) is named "data".

the path variable must be unique for the databases, because there can be several databases

This suggestion reminds me a bit of a functionality i implemented some time ago (probably doesn't work out-of-the-box any more)
https://github.com/felixbur/nkululeko/blob/main/meta/demos/multiple_exeriments/parse_nkulu.py

For now, it is not a big problem since we can run local test without any change. But it is better to keep this issue open for the future.

@felixbur
There is a discussion in Stackoverflow exactly related to this matter,
https://stackoverflow.com/questions/48538581/argparse-defaults-from-file
But before going further, I have some things in my head now:

User use INI file to define the config (current behavior, simplest).
User use command line arguments to override INI file (raised by this issue)
User use command line arguments without INI file

Having these capabilities will improve the speed of experimenting with Nkululeko. Just use the arrow key from the terminal to recall the last commands (or history/ctrl-z) and edit the command line arguments to experiment with other variables to see their impact.

so, 2 is also already implemented
by using the nkuluflag module. An example is here
http://blog.syntheticspeech.de/2022/03/28/how-to-run-multiple-experiments-in-one-go-with-nkululeko/


Bad CRC-32 while installing using pip

Hello,
I am trying to install opencv-python (<IP_ADDRESS>) on my aarch64 embedded platform. But it is giving me
zipfile.BadZipFile: Bad CRC-32 for file 'cv2/cv2.abi3.so' error.
The GNU/Linux system is Buildroot based and has python version 3.10.4 and pip version 23.2.1
pip3 install opencv-python==<IP_ADDRESS> --no-cache-dir
Log:
Collecting opencv-python==<IP_ADDRESS>
  Obtaining dependency information for opencv-python==<IP_ADDRESS> from https://files.pythonhosted.org/packages/96/7b/9b3a72a6afa0f98035db324966404f2471489068617e2a781ace121f9615/opencv_python-<IP_ADDRESS>-cp37-abi3-manylinux_2_17_aarch64.manylinux2014_aarch64.whl.metadata
  Downloading opencv_python-<IP_ADDRESS>-cp37-abi3-manylinux_2_17_aarch64.manylinux2014_aarch64.whl.metadata (19 kB)
Requirement already satisfied: numpy>=1.21.2 in /usr/lib/python3.10/site-packages (from opencv-python==<IP_ADDRESS>) (1.25.2)
Downloading opencv_python-<IP_ADDRESS>-cp37-abi3-manylinux_2_17_aarch64.manylinux2014_aarch64.whl (41.0 MB)
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 41.0/41.0 MB 11.4 MB/s eta 0:00:00
Installing collected packages: opencv-python
ERROR: Exception:
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/pip/_internal/cli/base_command.py", line 180, in exc_logging_wrapper
    status = run_func(*args)
  File "/usr/lib/python3.10/site-packages/pip/_internal/cli/req_command.py", line 248, in wrapper
    return func(self, options, args)
  File "/usr/lib/python3.10/site-packages/pip/_internal/commands/install.py", line 452, in run
    installed = install_given_reqs(
  File "/usr/lib/python3.10/site-packages/pip/_internal/req/__init__.py", line 72, in install_given_reqs
    requirement.install(
  File "/usr/lib/python3.10/site-packages/pip/_internal/req/req_install.py", line 807, in install
    install_wheel(
  File "/usr/lib/python3.10/site-packages/pip/_internal/operations/install/wheel.py", line 731, in install_wheel
    _install_wheel(
  File "/usr/lib/python3.10/site-packages/pip/_internal/operations/install/wheel.py", line 591, in _install_wheel
    file.save()
  File "/usr/lib/python3.10/site-packages/pip/_internal/operations/install/wheel.py", line 390, in save
    shutil.copyfileobj(f, dest)
  File "/usr/lib/python3.10/shutil.py", line 195, in copyfileobj
  File "/usr/lib/python3.10/zipfile.py", line 925, in read
  File "/usr/lib/python3.10/zipfile.py", line 1015, in _read1
  File "/usr/lib/python3.10/zipfile.py", line 943, in _update_crc
zipfile.BadZipFile: Bad CRC-32 for file 'cv2/cv2.abi3.so'

I have tried below things:

pip cache purge
Also tried installing opencv-contrib-python but same error
pip download opencv-python==<IP_ADDRESS> and pip3 install opencv_python-<IP_ADDRESS>-cp37-abi3-manylinux_2_17_aarch64.many linux2014_aarch64.whl

Embedded board has good network connection and sufficient disk space.
Can anyone please let me know what is the issue here and how to resolve it?
Thanks in advance.

Most probably your wheel file is corrupted. The issue is not visible on my side.


OpenSSL should support a method for providers to self-activate by installing a file

To support the OQS provider of PKCS11 provider in distributions, it would be best if installation of the package that contains the provider module can automatically add the provider to OpenSSL's configuration.
One possible solution for this would be a include.d directive that works like include but instead takes a directory and includes all .cnf files in that directory. This would allow distributions to configure their copy of OpenSSL with a default include directory for configuration files, e.g., /etc/openssl/providers.conf.d, add include.d /etc/openssl/providers.conf.d to the main openssl.cnf.
3rd party providers such as oqsprovider or pkcs11-provider could then install /etc/openssl/providers.conf.d/oqs.cnf or /etc/openssl/providers.conf.d/pkcs11.cnf that load and enable these providers, and automatically remove the configuration when uninstalling the package.

A side comment: currently we have a recursive parsing of config. It means that, to activate a provider, we need a ref to its section from the providers section. I'm not sure if smth like
[provider_sect]
my_prov = my_sect
[my_sect]

in the included file will work. If it doesn't, we should deal with it.

I like this idea, although the issue @beldmit raises is a concern, so we'd need to figure out a solution to that.

This already works, but it does introduce a dependency between the configuration files:
openssl.cnf:
openssl_conf = openssl_init

config_diagnostics = 1

[openssl_init]
providers = provider_sect

[provider_sect]
default = default_sect
[default_sect]
activate = 1

.include legacyprov.cnf

legacyprov.cnf:
[provider_sect]
legacy = legacy_sect

[legacy_sect]
activate = 1

OPENSSL_CONF=openssl.cnf openssl list -providers
Providers:
  default
    name: OpenSSL Default Provider
    version: 3.0.9
    status: active
  legacy
    name: OpenSSL Legacy Provider
    version: 3.0.9
    status: active

The included configuration file needs to know the name of the provider section in the main OpenSSL config file. If that name changes, this stops working.

The included configuration file needs to know the name of the provider section in the main OpenSSL config file. If that name changes, this stops working.

It shouldn't be a big deal for a distribution, right?

@beldmit a nice idea though

The described functionality for include.d should be already implemented as include directive supports taking directory and including all configuration files in that directory. I've implemented test which covers including configuration files with the provider section in #22598.

So do we still have an issue here?

I think no, it just works

Closing this then.


some adjusts for debug convenience

Codecov Report

Merging #492 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #492      +/-   ##
==========================================
+ Coverage   40.97%   41.02%   +0.05%
==========================================
  Files          49       49
  Lines       12572    12572
==========================================
+ Hits         5151     5158       +7
+ Misses       6146     6135      -11
- Partials     1275     1279       +4

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 53a7a7a...5ba8381. Read the comment docs.

👍  LGTM


Missing or unclear how to retrieve available metrics to enable via metricsconfiguration

This article shows how to create a metricsconfiguration, but it doesn't provide a list or a way to look up the list of metrics that can be enabled.  I attempted to use both metricsconfiguration list and metricsconfiguration show prior to a create, but they do not return such a list.  A section that either provides the list of metrics that can be enabled or documents how to look it up would be helpful.  The article also doesn't mention where the metrics will go once enabled.

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

ID: cc144107-6e17-e1e7-216f-50d66c496525
Version Independent ID: 32f9a4bc-d32d-729d-40fd-bdea4f871808
Content: Azure Operator Nexus: How to configure cluster metrics configuration management
Content Source: articles/operator-nexus/howto-cluster-metrics-configuration-management.md
Service: azure-operator-nexus
GitHub Login: @bryan-strassner
Microsoft Alias: bstrassner

@dbradish-microsoft
Please review it.

@pesalaPaven, please assign this to the ms.author that you will find in the metadata at the top of the MD file. Thank you.

@pesalaPaven, please assign this to the ms.author that you will find in the metadata at the top of the MD file. Thank you.

@priyamshet please review it.

@tlovett The details of the available metrics are listed in this link https://learn.microsoft.com/en-us/azure/operator-nexus/list-of-metrics-collected.
The link to this is also mentioned in the first couple lines of the How to document : Azure Operator Nexus: How to configure cluster metrics configuration management
Currently we do not have a way to programmatically list the available metrics before enabling them.


Can't seem to get to mesh level 2 using Automesh

I am using automesh to connect 4 ESP similar to the configuration you have in the picture but I cannot get one of my nodes to be mesh level 2 (they only connect to main AP and do not connect to each other)
Thanks

Try to place them in a row. Probably they all "see" the main AP and it is the shortest link.

I tried this configuration but I still didnt get the to mesh level 2
The distance between the main AP and the first repeater node was 2 meters and the second repeater node was 5 meters from 1st repeater (7m from AP main)

This is too close - as long as the second ESP still sees the main AP with reasonable signal strength, it will choose it.

How far away should I place the second repeater to get the mesh level 2 appear in the mac address? Do you know the max practical range of esp in meters? Thank you for your prompt reply :D

Assume the ESP has 3 nodes connected to it. If I assign a static IP to one of the nodes, will the DHCP adjust accordingly? (meaning if I assign <IP_ADDRESS> to one, the dhcp will assign to the remaining nodes anything but the IP ending in 3)

Try 10m+.
No DHCP will not recognize static IPs. If you want to use static IP, use a high arddess in the /24 network (250-254), these will not be used ba DHCP.

Hello martin
I am building an ESP based project using your firmware. It is a wireless sensor network as you see in the diagram
every AP is an ESP with NAT firmware
I am trying to put static IP on all nodes so I can ping it from remote server, however the dhcp cannot be overwritten. How would you suggest we ping node1 other than using the IP?

Due to NAT you cannot ping from the outside (e.g. "remote server") to let's say "node1". NAT hides internal adresses and ports. It is the same for local PCs behind a "normal" NAT home router. For single nodes and services you might define a port forward, but this is not a solution for many nodes with the same services.
What kind of protocol you want to use. If it is MQTT, each node can easily connect to a broker on "remote server" and then also receive commands via subscribed topics. Also HTTP requests from the nodes to the "remote server" work without problems.


Crash when reloadData() meets the scroll

I have 3 controllers. In each, at viewDidAppear, a request is sent and if there is a success,  calls pagingViewController.reloadData() to reload my custom PagingCells.
But I'm getting a crash when my second view controller appears, start the request and the user  returns quickly to the first view controller.
Crash in EMPageViewController.swift at line 532.
if self.navigationDirection == .reverse { // check if direction changed
    self.didFinishScrolling(to: self.selectedViewController!)
    self.willScroll(from: self.selectedViewController, to: self.afterViewController!)
}

The self.selectedViewController is equal to nil because the reloadData() calls removeAllViewControllers() :/
Is there a solution ? Is there another way to reload only one item from pagingViewController ?

Thanks for the report! I will try to get this fixed as soon as possible.

@rechsteiner I made a PR to solve this issue: https://github.com/rechsteiner/Parchment/pull/230

Released as part of v1.4.1 (#233) now ✌


Check and delete deprecated parts in main branch

I have checked a few things quickly. I decided that it would be best to work on a separate milestone with the lab technicians and Abelardo for refactoring the code after discussing the basics with Colin.
See new milestone Refactoring: Streamline code and reproducibility.


Zoom reset after Dynamic Update

There was a same issue reported earlier and closed due to inactivity. Hence, creating a new one.
I am building real-time charts with dynamic update, but every time dataset is updated, zoom is reset to 1
I have seen one recommendation to cache the zoom settings and reapply them after redrawing. I have tried everything, but just can't seem to get it working.
Request you please post an example

Refer to the example:
https://gist.github.com/andredumas/95f1f22130fb1a3a8181#file-index-html-L233
Start with the example and play around with that particular line:
x.zoomable().domain(<cached display index array>);


Integration test stabilization + update proto and Dapr runtime to 1.12 RC.

Description
Update Dapr runtime to 1.12 RC.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: endgame
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

@artursouza Just one question, as a whole we are waiting for Dapr to be completely ready before invoking any service right?

@artursouza Just one question, as a whole we are waiting for Dapr to be completely ready before invoking any service right?

Yes but with this change, I am also waiting for the app's health endpoint.

Force merging this to unblock the release.


HMAC based webhook security

Signed-off-by: ram.challa<EMAIL_ADDRESS>

Hey @swcurran, This PR solves the issue #2029.

@ramreddychalla94 — I’ll let the devs review this, but it appears your editor has reformatted lines in 426 files, so this is a massive PR.  I think you need to redo it to change only the files you intended to change.


Refactoring the refactor (II): Add artifact creation page

This depends on #1679, so review/merge that one first
This PR adds the page and code needed for adding an artifact to the system. At this point, this page is not integrated with the rest of the system in order to keep the PR small. It will be integrated in the next PR, which I'm almost done.
The tests, however, should pass.
The previous implementation was buggy and it did not allow to upload an artifact with more than a single lane or if it was missing the reverse reads. The user feedback has been also improved by performing the files checks on the fly rather when it clicks the create button.

This is ready for review

Some comments.

Just a few comments.

:+1: @ElDeveloper can you merge?


Edit: on Firefix (Xubuntu, 64bit, 86.0) I get the following errors:
Content Security Policy: The report URI (about:blank) should be an HTTP or HTTPS URI. fd02311.js:1:4556
Content Security Policy: The page’s settings observed the loading of a resource at https://yummyplan.github.io/sw.js (“worker-src”). A CSP report is being sent.

On Firefox Mobile (Android) I can add the app to my screen but starting the app fails with error message: App is not installed

@jankapunkt I suspect this to be some default settings of Firefox in case no CSP is present. https://github.com/yummyplan/yummyplan.github.io/commit/e5ce1031511b3f3969b140a2f459be0cbaa03d77 now explicitly adds a CSP (which would be good practice anyways)
Would you mind trying again?

I can confirm it on Firefox mobile to work now.

@jankapunkt Awrsome! Will close the issue then. If there's any other problems, please feel free to open another issue.


DOC: add source links to api docs

[x] closes #14178
[x] tests not needed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew not needed

cc @jorisvandenbossche  - like you mentioned, this doesn't work for everything (properties, accessors, some generated methods)

Here's an example of how it renders

looks nice.
@jorisvandenbossche

Current coverage is 85.24% (diff: 100%)

Merging #14200 into master will increase coverage by <.01%

@@             master     #14200   diff @@
==========================================
  Files           140        140
  Lines         50563      50563
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          43101      43102     +1
+ Misses         7462       7461     -1
  Partials          0          0

Powered by Codecov. Last update 6c73e76...9184e06

@chris-b1 Thanks a lot! Then it at least works for those where it is possible


Audio visualization does not seem to render correctly 8-bit wave forms

I generated a simple clip using freqPlayer in 8-bit. The sound that plays seems correct, but the wave form visualization seems broken.
http://bridges-cs.herokuapp.com/assignments/4583/esaule
I don't know enough webGL and JS to debug the issue. Maybe @krs-world , @quaiquai , or @AlecGoncharow knows how to debug it?

Here is an encoding of the same clip but in 16 bits http://bridges-cs.herokuapp.com/assignments/4584/esaule
They sound about the same (despite the 8-bit one is definitely a 8 bit sound). But the wave form is quite different. The 16-bit waveform is consistent with exceptation, while the 8-bit one is not.

Assume you are using Java version? Yeah, I dont see the wave visualization at all.
I am still fighting with the C++ version.. the conversion of the audio data to little-endian combined with the sign of teh values is causing  incorrect  values for the audio data..
— krs

K.R.Subramanian                                     Phone:<PHONE_NUMBER>
Assoc. Professor, Computer Science          FAX:<PHONE_NUMBER>
UNC Charlotte, Woodward 435E                Email<EMAIL_ADDRESS>Charlotte, NC 28223-0001                        www.cs.uncc.edu/~krs
If you are not the intended recipient of this transmission or a person responsible for delivering it to the intended recipient, any disclosure, copying, distribution, or other use of any of the information in this transmission is strictly prohibited. If you have received this transmission in error, please notify me immediately. Thank you.

On Jul 15, 2020, at 12:14 PM, Erik Saule<EMAIL_ADDRESS>wrote:
I generated a simple clip using freqPlayer in 8-bit. The sound that plays seems correct, but the wave form visualization seems broken.
http://bridges-cs.herokuapp.com/assignments/4583/esaule http://bridges-cs.herokuapp.com/assignments/4583/esaule
I don't know enough webGL and JS to debug the issue. Maybe @krs-world https://github.com/krs-world , @quaiquai https://github.com/quaiquai , or @AlecGoncharow https://github.com/AlecGoncharow knows how to debug it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/BridgesUNCC/bridgesAPI/issues/56, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLFEOESF4M5M2OWQDVCT2DR3XIXDANCNFSM4O2VOISA.

I am using the C++ API actually.
I am synthesizing the wave form from sine functions, so I don't need
the reader. I pushed tiny changes to AudioClip BTW to throw exception
when values are out of bound.

I just fixed the issue. It is fixed in master, not propagated to heroku yet.
It was a webclient problem.
The audio.js code reconstruct a wav file from the data in bridges. Bridges represent all PCM data signed, but Wav specifies 8-bit as unsigned. So there was a need to convert in webclient javascript.


rendering model shortcomings

Stolen from reddit https://www.reddit.com/r/linux/comments/wwsiaf/comment/ilosxsf/?utm_source=share&utm_medium=web2x&context=3 with phrasing slightly adjusted:
rendering model
How it currently works is that compositors throttle rendering by not sending callbacks to surface that are out of focus. The throttling can indefinitely stall that application's thread until it comes back into view. The problem is that clients have no way of knowing if they hidden or not. What ends up often happening is that certain render loops do clever hacks/workarounds to guess when the surface is hidden or not to avoid this stalling. There was a protocol proposal to fix this, but it got bikeshedded and probably will never be merged. Note that Wayland is literally the only windowing protocol that works like this and everywhere else it's trivial. What Wayland should have done was just send visibility events instead of callbacks and then encourage applications to not draw when the surface is not visible.

This is not relevant to this repository

README says I think so... This is an opinionated answer, if you disagree or if you think something is missing, I would love to hear about it, please open an issue or a pull request.

You are very dismissive stuff you read on social media.

I would prefer a technical argument written in a public space rather than potentially incorrect private conversations.

There is nothing wrong about admitting shortcomings. After all, everything is a tradeoff.

README says I think so... This is an opinionated answer, if you disagree or if you think something is missing, I would love to hear about it, please open an issue or a pull request.

Fair enough, i doubt this is what the repo owner is interested in but i will let him decide if to close the issue.

You are very dismissive stuff you read on social media.

The wayland developers mentioned years ago to be careful about stuff that gets written in places like hacker news and lwn comments, Even thought my interest in wayland is pretty casual, i noticed people writing incorrect things, corrected them only to see them get written again and again , there is even a name for this phenomena (the BS asymmetry principle). I am not saying social media is all bad but you should try to find better and more relaliabe sources.

Where else would you clarify these from your point bogous statements?

Maybe the wayland developers mailing list or the wayland gitlab on freedesktop, The issue tracker of major implementations (wlroots/smithay/KDE) might also be appropriate.
But I think unless you plan to do something about these things you might want to skip it, The time and resources the linux desktop gets is already pretty limited.
Not saying this is not valid criticism but this sounds wrong.


Automatic next/previous pages are wrong when using different locales

The automatic next and previous pages are wrong when using a locale.
All pages only have a next, which is set to the first page in the docs.
Additionally, since the last page in the docs has a next page pointing to the first page, I cannot even remove this by doing next: null, next: "", or next: false.

Thanks for raising! I'll investigate


Investigate EventCode implications

I recently found a bug in myquery that stemmed from a possible misunderstanding of mya event codes.  I discussed with Chris Slominski how to interpret event codes.  Turns out the first and second nibbles of the event code are orthogonal. 0x00, 0x01, ... 0x05 are codes for why there is not data in this update (0x00 for there is data) and 0x00, 0x10, ..., 0x30 are for why there is not data prior to this point (0x00 for there is data prior to this point).
The main point is that any 0x*0 event code will have a valid data value associated with it.  That's different than just 0x00 having a data point to  use.  This will probably impact a variety of methods and structures like streams and downstream dependencies.
myquery issue: https://github.com/JeffersonLab/myquery/issues/11

From the c++ source code, we have essentially two dimensions to a mya event.  One is the reason why there is no data (or if there is data) and the other is the reason why there is previous data (or if there is previous data).
Codes for reasons for data/no data with related explanations.
ET_Data = 0x00;
ET_Disconnect = 0x01;
ET_Off = 0x02;
ET_Shutdown = 0x03;
ET_Unknown = 0x04;
ET_NaN = 0x05;
{
  "Normal channel data point",
  "Network disconnection",
  "Archiving of channel turned off",
  "Archiver shutdown",
  "Unknown unavailability",
  "NaN/infinity encountered"
}

Codes for reasons for no prior data/there is prior data with associated descriptions:
ET_NotFirst = 0x00;
ET_Inception = 0x10;
ET_Offline = 0x20;
ET_Discard = 0x30;
{
  "There are previous data values available",
  "Origin of channel's history",
  "Channel's prior data moved offline",
  "Channel's prior data discarded"
}

Also, there is a general mya "rule" that a table can only start with an update that has data.  A sensible approach probably involves splitting these codes into two orthogonal enums in java and allowing downstream dependencies to check for the  presence of data without having to have a list of which codes or nibbles indicate data presence.
Based on discussions with Chris Slominski, no message is presented for a 0x00 code even though two messages are defined as above.


Chinese translation

About support Chinese
Hello! I want upload Chinese translation but I don't know how to do it.
And localization files in https://github.com/SCICN/zh_translation
Thanks!

@SCICN, thanks a lot for wanting to improve Spyder! Unfortunately it seems you translated our stable branch, i.e. Spyder 2.3.8 or a lower version but we need you to translate our future version, i.e. Spyder 3.0.
But don't worry, this is not much of a problem ;-) To add your translation to our source tree you need to

Follow our Development guide:

https://github.com/spyder-ide/spyder/wiki/Dev%3A-Github-Workflow
to clone our repository in Github and create a git branch on it.

Add you translation under the spyderlib/locale directory.

Update your translation files to use the most recent spyderlib.pot file.

Commit your work to your branch and create a pull request so we can merge it.

If you have any doubts about this process, please let us know about it :-)

Closing because (sadly) there was no response from @SCICN.


Move repo to rustup.rs

(rustup is already taken by rustup.sh).
Holding off on this because last time we moved it broke the appveyor CI and I don't want to deal with it again right now.

:eyes: We're here!


chore(master): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2023-12-28)
Bug Fixes

release-please: allow triggering release_wheel gh action (54f3d38)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/punica-ai/punica/releases/tag/v1.0.1 :sunflower:


Keyboard navigation of clickable link

Description of the new feature/enhancement
Keyboard navigation of clickable link in terminal text. Terminal Roadmap v2.0 is describing about clickable link
| Hyperlinking any links that appear in the text buffer. When clicking on the link, the link will open in your default browser.
However, this involves pointing device such as mouse or trackball. I believe many of terminal users love keyboard and it is important to keep using keyboard without switching to pointing device as much as possble.
Proposed technical implementation details (optional)
We may have a command like "NavigateToPreviousClickableLink" or "NavtigateToNextClickableLink" and "InvokeFocusedClickableLink", then we can reassign key binding those command.
bentaudun commented as https://github.com/microsoft/terminal/issues/204#issuecomment-640016016 but wasn't mentioned in https://github.com/microsoft/terminal/issues/574

I agree this would be super useful. I'd think we could probably leave out InvokeFocusedClickableLink, and just leave that as space or enter, but now as I'm typing this up, maybe people would want to navigate to the previous link, then copy it, then return to the input line and paste it? So maybe another action for that makes sense? Though, we can't have multiple actions bound to the same key, so that wouldn't work with "enter to copy" people...
I'm maybe rambling a bit.
This is a good idea for inclusion once #574/#204 lands.

Copying some ideas from the dup I accidentally made...
Description of the new feature/enhancement
Windows Terminal supports automatic detection of hyperlinks and allows you to open the hyperlink using your mouse (specifically ctrl+clicking the link). I've realized that there's no way to do this using only the keyboard.
Once mark mode lands, the solution is a little better, but still far from perfect and on par. The user would be able to navigate to the text with the hyperlink, create a selection that encompasses it, copy the URL, then manually open it in their preferred web browser.
What if we could make this a nicer experience? Consider this...

enter mark mode
tab (or shift+tab) to navigate to these hyperlinks
when you've navigated to the hyperlink, a selection is created that encompasses the relevant hyperlink.
enter can be used to invoke the hyperlink directly

Some corner cases...

what if enter is bound to an action (like "copy")? --> prefer the action
what if we add more bookmark-like things like "prompt position" or user-configured bookmarks --> might as well navigate to them too? I'm thinking these can be tab stops.

From Mike...

Oh man do I have thoughts in https://github.com/microsoft/terminal/issues/11000.


Implement MAV_CMD_SET_MESSAGE_INTERVAL and MESSAGE_INTERVAL

@LorenzMeier
In handle_message_request_data_stream we convert the received rate (I believe it's meant to be a rate, see below) into an interval (https://github.com/PX4/Firmware/blob/bc5cf50f1a2e3c6113204c48258835d9098b5b0b/src/modules/mavlink/mavlink_receiver.cpp#L1166). But the later invoked method to configure the stream in mavlink_main does it again which leads to huge rates. This might be bug in handle_message_request_data_stream, which I can fix as soon as I know what the message really meant to specify.
In REQUEST_DATA_STREAM, the comment says it specifies an interval, but the parameter name suggests it's a rate.
https://github.com/mavlink/mavlink/blob/378cd337301b3a6b944fedc46b4fcd89e16da42c/message_definitions/v1.0/common.xml#L2095
What is it? Rates seem to make more sense to me, so I assume the comment lies (what a surprise that would be ;)). Or is it rates * 1000 so we can specify slower than 1Hz?

The only usage example I currently have is the ArduCAM OSD which requests rates of 5 or 2.

It needs a completely new API and message spec. What kind of use case do you have / what things would you like to see in the API?

I'm don't want to invent any use-cases here, so I'm trying to keep the OSD or our onboard computers in mind and what we do now:
The component connects via MAVLink. The FCS doesn't have to know about this component, only that it's connected, an initial link would only stream heartbeats. The component then requests FCS capabilities, decides which messages might be supported off that information, then it requests streams of those messages.
So the API would define:

request_stream_of_messages(message_id, rate)

For more link management (already inventing functionality here I don't need yet..):

set_rate_of_message_stream(message_id, rate) // rate == 0 would mean stop

I'm explicitly talking about messages here. The concept of abstract streams that seems to be the idea behind the current API, which can hide multiple different messages for different FCS, doesn't seem to be necessary in that use-case. Thoughts:

would it even be possible to use the streams like that? Technically we compile against one specific dialect anyway, and conceptually different FCS could use the same message IDs for different messages which would make the parsing difficult.
most of the messages used are in common anyway, an effort to bring frequently used messages to the common format is better than having abstract streams

Yes, that was pretty much along the lines of what I was thinking. I just need to implement the API then and more importantly apps like QGC need to learn how to use it.

Ok. It seems my second method is not actually necessary, could all be done with one, assuming it really stays that simple:

configure_stream_of_messages(message_id, rate) // rate == 0 would mean stop

New API is implemented in the new MESSAGE_INTERVAL message and the MAV_CMD_SET_MESSAGE_INTERVAL and MAV_CMD_GET_MESSAGE_INTERVAL. Its not implemented on the PX4 side, but a pull request would be most welcome.

Moving to next release. Will do this once it becomes wanted.

Implemented, closing.

I don't think this is implemented

It definitively is. Closing.


Whitelist @storybook/addons

DefinitelyTyped/DefinitelyTyped#38961 needs this.
CC @sandersn @RyanCavanaugh @orta

It's now in the production branch, so Definitely Typed's CI will get the new whitelist next time it runs.


Replace GCM by FCM

There is some variable and properties using GCM instead of FCM. Since GCM is replaced by FCM we should also replace that variables and properties

Would like to it @danielpassos

@shubhamvashisht Go ahead!

@shubhamvashisht What is the status?


Builder: remove has_xattr flag check for merge

The has_xattr flag is not a user-specified by builder (nydus-image),
the flag value is calculated by checking whether if have any files have
xattr, so for bootstrap merge scenario, we can't assume this value is
same between different bootstraps.
Signed-off-by: Yan Song<EMAIL_ADDRESS>

@luodw Thanks for reporting this issue.

LGTM


Missing information about which VM sizes are premium storage-compatible

The section on Premium storage states:
> To learn more about individual VM types and sizes in Azure for Windows, including which sizes are premium storage-compatible, see Windows VM sizes. To learn more about individual VM types and sizes in Azure for Linux, including which sizes are premium storage-compatible, see Linux VM sizes.
However the linked pages don't seem to mention premium storage at all?

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

ID: 46ddbb6b-a5dd-fdea-6276-4a7f9b1aeff2
Version Independent ID: f4a24ccd-7498-bb91-dcf2-dadf071c58fa
Content: Select a disk type for Azure IaaS Linux VMs - managed disks - Azure Linux Virtual Machines
Content Source: articles/virtual-machines/linux/disks-types.md
Service: virtual-machines-linux
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@crispygoth Thanks for reaching out, this information isn't actually missing. The VM docs don't have a single place where you can see every VM size details. Instead, you have to nav to each individual VM size manually, which you can do from those overviews.
Like so: https://docs.microsoft.com/en-us/azure/virtual-machines/dc-series
Now that you've brought it up, we could add a statement saying that explicitly. I'll add an item in our backlog for that.

@crispygoth We've gone ahead and added a small statement for this. It'll be live later today.
Accordingly, I'm closing this out.
Thanks for reaching out.
#please-close


Prompts don't work in a HTML5 project for me.

Describe the bug
prompt is too long: 214319 tokens > 200000 maximum

Link to the Bolt URL that caused the error
https://bolt.new/~/stackblitz-starters-mgze9u
Steps to reproduce
Import an HTML website project and try to enter any prompt.
https://stackblitz.com/edit/stackblitz-starters-mgze9u?file=package.json
Expected behavior
I should be able to enter a prompt and have Bolt help me edits to my HTML5 project.
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 131522188,
  "usedJSHeapSize": 126911532,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = aleaming
Chat ID = 2850e6006aec

Additional context
No response

Appreciate the feedback! This is a known issue with the AI model’s context window size and is being tracked here: #1322.


Request for collaboration and maintenance regarding the fork

I hope this message finds you well. First and foremost, I want to express my sincere gratitude for your interest in the "Helium" theme and for the time and effort you've invested in creating the fork.
I wanted to reach out to discuss a matter that has been on my mind for a while. I've noticed that your fork, "Helium Enhanced Grid Mod," was created around two years ago and hasn't been regularly updated to incorporate the bug fixes, changes, and improvements that I've made in the original theme during this time. Also I believe this situation has created some confusion among users, as the "Enhanced" text in the fork's name might lead them to believe it's an improved version over the original.
My concern is not just about maintaining the original theme but also ensuring that users have the best experience possible. Having multiple versions of the theme can lead to inconsistencies and missed features that would be beneficial to everyone. For example, in this time multiple fixes and new fields have been added, including games Install Size, games Installation Folder button and many others, and I have as well added support for countless available community extensions. I've also had multiple instances of people having issues with Helium and it has turned out that it was an issue only in this fork.
I would like to propose the idea of consolidating the efforts and encouraging users to use the original "Helium" theme. By doing so, we can ensure that all users are benefiting from the most up-to-date and feature-rich version. I believe this approach can lead to a better user experience and streamline maintenance efforts.
Additionally, I'd like to share that since then I have created an extension to use the layout on Details View in Grid View, which I believe was the original purpose of this fork, which can be seen here: https://playnite.link/addons.html#ThemesDetailsViewToGridViewConverter_ef8a7226-eedc-478d-a506-92ee6c088aa3
I'm open to discussing how we can collaborate on maintaining and improving the "Helium" theme if that's something you'd be interested in. Personally I have no problem if people modify the themes that I work on to better fit the preferences and I'm only reaching out to you to share my thoughts regarding the matter but I will understand and respect your decision.
Thank you for taking the time to consider my proposal, and I look forward to hearing from you.

Hiya,
This message found me in the morning before my coffee :).
I like the simplicity of the theme mod I've done, and I'm using it since its creation all these 2 YEARS AGO? Sheez time flies.
I am aware that Helium and your other themes now can generate GridView with the converter, and new users should be most probably pointed towards this solution.
At the time, I was following the naming of a different 'Enhanced Grid View Mod' theme, but I can see how it can be viewed as 'Helium Enhanced' instead of 'Helium - Enhanced Grid'. To avoid any extra work and confusion this modded theme created, I'll rename the theme to something unrelated and add a note pointing that there's a different, most likely better solution to achieve what this theme was set to achieve.
At the same time, I'm good with what I have here, and since the GridViewConverter was created I don't have anything to 'give back' to the original project.

Thanks, I appreciate the swift response and respect your decision of keeping the fork available separately and I hope me contacting you didn't pose any issues or inconveniences for you.
I see you renamed the entry in the addons database and if you have the chance sometime I think it would also be a good idea to also rename it here in the manifest and release a quick update to also reflect this name change in Playnite itself.

I've updated the manifest and swapped the release package in place so every new download gets the theme with the new name, but you're right, I'll push an update just to get rid of the old name entirely.

I've released a new, renamed, version. Shouldn't be an issue anymore.


[BUG] Forestry is no more

I have checked all the prerequisites below and I'm yet experiencing a problem

[x] Read the README.md
[x] Have the extended version of Hugo installed
[x] Used the exampleSite's config.toml as a reference
[x] If and when requested, I'll link my repo to make it easier to debug.

⚠ NOTE: If the issue isn't easy to reproduce and its author(s) decline to provide reasonable accommodation to debug, the issue will be closed.
Describe the bug
Forestry has been replaced by tina. https://tina.io/forestry/ It would be nice if the config files either could be migrated or new tina ones be added to the template :)

Hi @MTRNord,
Working to avail a Tina config soon. Might take some time as I work on this on my free time.


[🐞] V2 - Exporting a useStore and a component$ causes the useStore to lose its “this”

Which component is affected?
Qwik Runtime
Describe the bug
If a hook returns a useStore and a component$, and the useStore uses function to access “this”, the “this” is undefined.
Reproduction
https://stackblitz.com/edit/github-6f6maw-cm569smw?file=src%2Fhooks%2Ftest.tsx,src%2Froutes%2Findex.tsx
Steps to reproduce
If you click on the “submit” button, you will see an error, since the “this” in the “onSubmit” of the “useStore” is “undefined”.
On the other hand, if in the “useForm”, you remove the export of the “Form” component, the error disappears.
System Info
npmPackages:
    @qwik.dev/core: 2.0.0-alpha.2
    @qwik.dev/router: 2.0.0-alpha.2
    typescript: 5.7.2
    undici: 7.0.0
    vite: 6.0.1

Additional Information
No response

I see you're just calling await onSubmit(), isn't that why this is undefined?

@devcaeg actually, are you looking to have this be the store? You need to do that in this way: https://github.com/QwikDev/qwik/issues/5087#issuecomment-1707185010

@wmertens I'm actually basing this on what the documentation says:
https://qwik.dev/docs/components/state/#methods

I see you're just calling await onSubmit(), isn't that why this is undefined?

You are right, my mistake haha, I should do “await store.submit()”.
The weird thing is that this bug of mine, I have it for many months running in v1. I mean, although I recognize that it's my fault, anyway in v1 it works and in v2 it doesn't haha.

Ok so I'll close, it seems to be accidently working in V1 but doesn't seem proper.


Deprecated repository of 7z file on MacOS

The repository https://github.com/rudix-mac/pkg no longer exists and therefore
https://raw.githubusercontent.com/rudix-mac/pkg/master/p7zip-16.02.pkg returns 404.

Also, https://raw.githubusercontent.com/rudix-mac/packages no longer uses versioning, therefore /packages/{{macOsVersion e.g. 10.11}} no longer exists.
There's only one package https://raw.githubusercontent.com/rudix-mac/packages/master/p7zip-16.02.pkg , but I'm unable to test, whether it works on macOS <10.11.

Thanks, i haven't been keeping up on insuring everything is current.
Yes, It seems rudix-mac have switched things around. I will further look into creating a new build or unless you can do PR.

Well I was thinking about making the new PR to this repo but I can't find any suitable source for macOS 10.6 - 10.9.
Couldn't we simply deprecate the binaries for the older systems and use single source as you do in windows & linux getDataForPlatform() ?
If you agree I can create a PR in a short time.

The PR would be fine, since no CI build system can really test for macOS 10.6 to 10.9 nowadays, and I don't have those systems setup to test on either.


Main Menu sometimes cut off to three items

I built the first draft of my own website: https://janvv.nl/. This is an attempt to migrate my website from papasmurf.nl to the Perplex theme.
There is still a lot of fine-tuning to be done, but I have noticed that the menu items disappear when you reduce the screen width. However, they reappear when you make the screen even smaller.
Is there something that I can do to prevent this behavior?

Hi Jan,
Thanks for your report!
No, there is currently nothing you can do to change the layout of the menu. There is simply not enough space to show all your entries in the 3-column layout version because there is only one column of space reserved for this menu. (One column is around the half of the main text width)
Your menu entries need around 1 and a half major column and this could be working, if I offer another  layout option for larger site menus for this layout size. There is still enough empty space to use, if you don't add more entries.
In the near future I'm not going to find the time to implement this alternative layout for site menus, because I've just started a completely new and demanding full-time job. But this would be a useful enhancement and its on the top of my list.

Thank you for your reply.
You might see pull requests coming in from my side.
I'll happily spend some time enhancing the best Hugo theme I've seen so far.

I have never done this before. I investigated and studied how the grid mechanism works in CSS. After experimenting with it for a day, I succeeded in displaying my entire menu at various screen sizes.
The custom CSS feature is awesome.
My CSS can be found here; https://gitlab.com/janware/papasmurf.website/-/blob/main/assets/css/custom/menu.css.
As for me, this issue can be closed, but maybe you'll want to keep it for later improvement.

Thanks, your solution is an improvement!
As soon as I can find the time, I would like to work on the primary menu and will look into your CSS as an inspiration.


[BLOCKED] Replace cchardet with faust-cchardet

the original package hasn't received updates since 2021, while the latter is still active
cchardet also doesn't play nice with python 3.10 and 3.11

Superseded by https://github.com/inveniosoftware/invenio-previewer/pull/192


feat: [OLS-104] Add token handler utility for context

Description
We need a mechanism to limit tokens based on context window.
The modification covers below

utility to convert text to tokens
get approximate token count
limit the context based on context window.

Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue # OLS-104
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[x] If it is a core feature, I have added thorough tests.

It looks nice so far.
But are you going to integrate it with some existing code?

@tisnik This will be called from doc_summarizer. And multiple changes required in summarizer module (including unit test) so I was thinking of a separate PR for that.. But integration coverage is marginally less than the threshold for this.. I will let you guys decide whether to hold this or not..

separate PR is totally reasonable.  Deliver the utility logic in one PR, wire it up in another.

/lgtm
/approve


freeze: 1.1 -> 1.3

Diff: https://github.com/optiv/Freeze/compare/v1.1...v1.3
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 233006 run on x86_64-linux 1

1 package built:

freeze


Created NRT_PrivlegedRoleAssignedOutsidePIM.yaml

Change(s):

Created NRT_PrivlegedRoleAssignedOutsidePIM.yaml

Reason for Change(s):

New NRT rule addition.

Version Updated:

Initial Version

Testing Completed:

Yes

@aprakash13  - Please let me know for any updates needed to this rule.

@aprakash13  Please review and provide your feedback. thanks!!!

Thank you for the approval & merge @aprakash13 .


Audio prototype

@sjain28 it looks the calendar widget commits are also in this PR. You'll want to close this PR/branch, then open a new branch off master and cherry-pick the correct (audiowidget) commits from this branch into the new branch. Let me know if this doesn't make sense and I can come by.


surrealdb-migrations: init at 0.9.5

Description of changes
just initializing a package to help with surrealdb migrations.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 230980 run on x86_64-linux 1

1 package failed to build:

surrealdb-migrations

Result of nixpkgs-review pr 230980 run on x86_64-linux 1

1 package failed to build:

surrealdb-migrations

I tried building it on the actual linux machine and the build succeeded, I'm not sure why nixpkgs-review failed twice here.
I'm going to go ahead and merge this, since I don't think many people will be interested.


Webhook target configuration mismatch

In current master both README and values.yaml shows that in webhook target you have to use host parameter, but actual chart uses webhook https://github.com/kyverno/policy-reporter/blob/39c354fd3f2c7cb66d844b9ee45c5ccdb065fe35/charts/policy-reporter/templates/_helpers.tpl#L181
Same goes for usetls and certificate there no indication in values or readme.

Thanks, will take a look today

Fixed with the latest RC release


Make Notes[1:5] also return an instance of Notes

Currently if notes is some Notes instance, doing notes[1:5] returns Vector{Note}.

👍


Add a default ClusterBuilder

kpack users can create images with cloudfoundry/cnb:bionic in any namespace

Closing due to possible race condition.


Web page thumbnail not generated for Kibana via proxy URL

The thumbnail icon for Kibana is not generated now that we access Kibana via paths like https://<IP_ADDRESS>/api/v1beta1/proxy/services/kibana-logging/
In the previous setup (using an external load balancer and public IP) the thumbnail was properly served and this made it easier to idetnify the tab in the browser that contains Kibana.

What does the kibana html look like? Are the URLs being rewritten? Was it a relative or absolute URL? Was it pulling the image via https or http?

I'll revert to the previous setup and work the answer to these questions by observing what is going on in my Chrome console. I guess we re-write URLs and we are now using HTTPS (before it was HTTP).

Thanks David. No idea off the top of my head how to fix this. I am sure if I hacked away at it enough I could get to the bottom of it :-)

The move to Kibana 4 makes this work.


filters can survive context loss

I believe that PIXI v4 is LTS, so I backported solution from my fork.
Its not fixed in v5: https://github.com/pixijs/pixi.js/blob/next/src/core/renderers/webgl/systems/FilterSystem.js
It is fixed in my v5 fork: https://github.com/gameofbombs/gobi/blob/master/src/pixi/renderers/systems/ShaderSystem.ts
To test the feature put this code in any of pixi examples that has filters, and see if it survives after context is lost. With my patch there are no errors/warnings in console at all.
function loseContext() {
	var gl = app.renderer.gl;
	var ext = gl;
	if (!ext.loseContext) {
		ext = gl.getExtension('WEBGL_lose_context');
	}
	ext.loseContext();
	setTimeout(function () {
		ext.restoreContext();
	}, 1000);
}

setTimeout(loseContext, 2000);

One more issue that is possible to fix in this commit is that pixi webgl manager types are exported in docs but they're not exported in runtime. I cant just hack "PIXI.FilterManager" in a plugin :( Do you think I should add managers export in this PR?


Enhancement for Analytics Tracking for Email Share functionality

Hi,
Can we add analytics tracking for share functionality in ASC?
Since analytics we use on all websites to track the user-data. I would request if we can add a query parameter at the end of url that gets shared over the email.
Solution:
Can we create a new tab 'Analytics' in E-mail Share modal dialog which is under /content/asset-share-commons/en/light/actions/share.html. Under Analytics tab create a new field 'Query Param'. This value will be picked and appended in assetLinkHTML for every link.
assetLinkHTML is getting generated in EmailShareServiceImpl in getAssetLinkListHtml() line number 168 where url is appended. Here we can add the query param value that comes from dialog.
It would be helpful if this can be achieved and provided as a part of ServicePack.

Hi @skumuda - if we set it at the Dialog level, this will set the same value for every share link -- is that what youre going for? You just want to know what Details page views are from shares vs navigating via search/bookmarks?

@davidjgonzalez Yes, same value should be passed for every share link. This is to track which assets are shared.

Hi @davidjgonzalez Please let me know your thoughts/suggestion on the above implementation.


Warning in HA 2024.6

Describe the bug
2024-06-06 17:55:16.337 WARNING (MainThread) [homeassistant.util.loop] Detected blocking call to open inside the event loop by custom integration 'rte_ecowatt' at custom_components/rte_ecowatt/__init__.py, line 404: return tz.gettz(timezone) (offender: /usr/local/lib/python3.12/site-packages/dateutil/tz/tz.py, line 464: fileobj = open(fileobj, 'rb')), please create a bug report at https://github.com/kamaradclimber/rte-ecowatt/issues
Traceback (most recent call last):
File "<frozen runpy>", line 198, in _run_module_as_main
File "<frozen runpy>", line 88, in _run_code
File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module>
sys.exit(main())
File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main
exit_code = runner.run(runtime_conf)
File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run
return loop.run_until_complete(setup_and_run_hass(runtime_config))
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 672, in run_until_complete
self.run_forever()
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 639, in run_forever
self._run_once()
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1988, in _run_once
handle._run()
File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
self._context.run(self._callback, *self._args)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 742, in async_setup_locked
await self.async_setup(hass, integration=integration)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/rte_ecowatt/__init__.py", line 62, in async_setup_entry
await hass.config_entries.async_forward_entry_setups(
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2032, in async_forward_entry_setups
await asyncio.gather(
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2034, in <genexpr>
create_eager_task(
File "/usr/src/homeassistant/homeassistant/util/async_.py", line 37, in create_eager_task
return Task(coro, loop=loop, name=name, eager_start=True)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2081, in _async_forward_entry_setup
await entry.async_setup(self.hass, integration=integration)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 134, in async_setup_entry
return await component.async_setup_entry(entry)
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 196, in async_setup_entry
return await self._platforms[key].async_setup_entry(config_entry)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 333, in async_setup_entry
return await self._async_setup_platform(async_create_setup_awaitable)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 363, in _async_setup_platform
awaitable = create_eager_task(awaitable, loop=hass.loop)
File "/usr/src/homeassistant/homeassistant/util/async_.py", line 37, in create_eager_task
return Task(coro, loop=loop, name=name, eager_start=True)
File "/config/custom_components/rte_ecowatt/sensor.py", line 82, in async_setup_entry
await rte_coordinator.async_config_entry_first_refresh()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 277, in async_config_entry_first_refresh
await self._async_refresh(
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 411, in _async_refresh
self.async_update_listeners()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 165, in async_update_listeners
update_callback()
File "/config/custom_components/rte_ecowatt/__init__.py", line 414, in _handle_coordinator_update
ecowatt_level = self._find_ecowatt_level()
File "/config/custom_components/rte_ecowatt/__init__.py", line 529, in _find_ecowatt_level
now = datetime.now(self._timezone())
File "/config/custom_components/rte_ecowatt/__init__.py", line 404, in _timezone
return tz.gettz(timezone)

** Versions **
HomeAssistant version: 2024.6
This integration version: 0.6.3

Offending code is:
https://github.com/kamaradclimber/rte-ecowatt/blob/1187c6c5f79152a4791b1f1360e690b978e7e314/custom_components/rte_ecowatt/__init__.py#L402-L404

Yes, it is probably easiest to simply use dt_util.get_default_time_zone():
    def _timezone(self) -> tzinfo:
        return dt_util.get_default_time_zone()

I won't have time to fix this warning in the near future (to be honest, I'm tired of HA developing team constantly deprecating items).
I would however welcome patches to fix this of course.


Changes in UI of  Tracks Session list

Actual Behaviour

The list represent sessions for a particular track thus track's name shouldn't be present in all list items
Expected Behaviour

@the-dagger @mananwason @mariobehling do you think the UI is fine. Should I send a PR for same?

@utkarshnath
What if we are viewing the session by a particular speaker or bookmarked sessions or sessions at one location?
What do you suggest for the following cases

@free4murad I was suggesting to remove tracks name from only TrackSessionActivity. And we should also remove location from each of the item in locationActivity.

@utkarshnath
we are using the same layout in all the above places.
you remove tracks field from one place it will be removed from all places.

@free4murad we can make their Visibility gone when the adapter is used for tracks and location activity

@the-dagger @mananwason @mariobehling can I send a PR for this?

@the-dagger and what do you think about track name in track specific session list, shouldn't it be removed here

@utkarshnath seems like too much overhead imo.
Instead what you can do is display the speaker's name in the card.
Mapping speakers with sessions is still a work in progress.
You can probably help @brainbreaker with it in #1112

@the-dagger what should I do if there are more than 1 speakers

If that's the case then we can display both of the speakers separated by a comma.


Update detection to work with React v0.15.0

Because of this

we can check for both [data-reactid] and [data-reactroot] and if anyone of them is present we can outline the components.

I think we can use that to detect whether react is on the page, but with v0.15.0 there doesn't appear to be any way to identify an individual component from the DOM. So we wont be able to outline components in v0.15.0 :-(

Yup!  But it does outlines the container component  . I tried  , it's good enough until we find something else.


Image loading irritants.

I just started using this library so sorry if this has come up before; i don't really know my way around it well enough to start fixing it but this is a simple request...
In image loading, it really bothers me that when loading a bitmap or jpeg the dimensions are ints, but as a png they're uints; similarly the bitmap has it's own free memory function and the jpeg and png don't.   I should be able to declare a function pointer type and have it be compatible with all the load image functions IMO...
I also think that the image reading functions should take a function pointer for an allocate function, its not good that if i want to manipulate the image a bit before compressing it i need to load it and then copy it into an image instead of just loading it directly into an image.  And having it take a memory allocating function would fix this.
I'm also unsure of why the basisu::image::init function sets a background color if that just gets overwritten anyway.

Please note that BasisU is not an image loading library. The encoder is a command line tool, technically not a library (yet). Once it is a library we will not be exposing any plain image reading functionality as part of the public API (that's a small nightmare). All image loading functionality will remain private, for internal use. I would have rather used something like stb_image, but it's not fuzz safe so we can't use it.
I'm not sure what you mean by an inconsistent interface. It's pretty simple and consistent, but remember this is internal functionality we will never expose to the end user so it doesn't matter:

As for the image::init() method, our goal is deterministic and reliably execution first, performance second. A memset() or equivalent executes at many GB/sec. on even ancient laptops. It's a non-issue. We can afford to clear our bitmaps. :-)


[Bug] List items that should move to the new line do not

Can this issue be consistently reproduced? It appears to be a pure model output problem, rather than a frontend processing problem.


2016-11-26 BETA : Error 80029537 when installing a .vpk

With VitaShell 1.42 (or MolecularShell 1.42) :
Install a .vpk.
0%->100%. Then Error 80029537.

This happens for any vpk?

Yes it was but you fixed it this morning by updating taiHENkaku
Good work !


creating a hasMany relation is not persisting

On ember-cli-mirage 0.2.0.beta8, I'm doing this :
    let activity = server.create('activity', { name: 'Tenis' });

    activity.createCourse({name: 'test'});
    activity.save(); # Just to be sure

    var store = application.__container__.lookup('service:store');
    store.findAll('activity').then(function(activities) {
      expect(activities.objectAt(0).get('courses.length')).to.eq(1);
    });

It fails because courses.length gives 0.
Here is my model :
import { Model, hasMany } from 'ember-cli-mirage';

export default Model.extend({
  courses: hasMany()
});

Is there something wrong?

I may help. This :
    console.log('1', activity._schema.activity.all()[0].course_ids);
    console.log('2', activity._schema.activity.all()[0].courseIds);

    activity.course_ids = [course.id];
    activity.courseIds = [course.id];
    activity.save();

    console.log('3', activity._schema.activity.all()[0].course_ids);
    console.log('4', activity._schema.activity.all()[0].courseIds);

gives
1 []
2 []
3 ["1"]
4 []

I'm using JSON API

After activity.save() on your first line, can you check
server.db.activities;
server.db.courses;

and show me that data?

I have this :
    let activity = server.create('activity', { name: 'Tenis' });
    activity.createCourse({name: 'test'});
    activity.save();

    console.log(JSON.stringify(server.db.activities));
    console.log(JSON.stringify(server.db.courses));

[{"courses":[],"course_ids":[],"name":"Tenis","id":"1"}]
[{"name":"test","activityId":"1","id":"1"}]

Are you loading fixture data? course_ids looks wrong, the ORM uses courseIds but those shouldn't even be in the db anywhere.
Does your activity factory create ids somewhere?

Here is the factory for activity
import Mirage from 'ember-cli-mirage';

export default Mirage.Factory.extend({
  name: 'IT',
  course_ids: [],
  courses: []
});

I have no fixtures. I don't have any data elsewhere.

Remove courses and course_ids from the factory

I did it and I have this result :
[{"name":"Tenis","id":"1"}]
[{"name":"test","activityId":"1","id":"1"}]

It works. I retried with the findAll method and it works now.
Thanks a lot!

 We should probably have relationships override attrs in the event this sort of thing happens again.

Yes, I agree. ember-cli-mirage is really awesome. Please don't stop your work!

I have a similar issue in another case.
I have this code :
    let activity = server.create('activity', { name: 'Tenis' });
    let course = activity.createCourse({name: 'Adult'});
    course.createGroup({name: '12h-13h', nbPlaces: 12, schedule: '12h'});

    var store = application.__container__.lookup('service:store');
    store.findAll('course').then(function(courses) {
      let group = courses.objectAt(0).get('groups').objectAt(0);
      console.log(group.get('name'));
    });

group.get('name') gives undefined. When I add store.findAll('group') after the creation of the group, group.get('name') gives the real name.
I can submit a new issue if you prefer.

Is your Mirage server sideloading the groups with the courses? Are the relationships async or sync?
Btw you can do courses.get('firstObject.groups.firstObject').

I did this
    let activity = server.create('activity', { name: 'Tenis' });
    let course = activity.createCourse({name: 'Adult'});
    course.createGroup({name: '12h-13h', nbPlaces: 12, schedule: '12h'});

    var store = application.__container__.lookup('service:store');
    Ember.run(function() {
      store.findRecord('course', course.id).then(function(course) {
        console.log(JSON.stringify(course.get('firstObject.groups.firstObject')));
      });
    });

The result is undefined. All relations are synchronous. My server is not triggered.
I tried to put the code in Ember.run and it's the same result.

server.create puts records in Mirage's db, not Ember Data's. You need to go through the network to get them into Ember Data's store.


ipython>=5.0.0

ERROR: ipykernel 5.2.1 has requirement ipython>=5.0.0, but you'll have ipython 4.2.0 which is incompatible.

Hi @cero1010,
This has also likely been fixed as part of the work for Issue #313 (https://github.com/mhallsmoore/qstrader/issues/313).
Please let us know if you have any further problems.
Regards,
Mike.


Un-xfail sparse tests for 0.13.0 release

These pass with the latest sparse=0.13.0 release (thanks @hameerabbasi)

Merging to unblock other PRs


Make a good part of the app localizable and add French translations

This PR adds a first round of translation keys and French translations.
Credits go to the Données & Territoires team: everyone chipped in to add translation keys or translate strings here and there.
cc @LouisDelbosc @yohanboniface @csail14 @vviers @ronnix @l-vincent-l @Aurelie-J
We made sure the CI passes, we performed a number of sanity checks, and we did a fair amount of local testing.
However:

We didn't do exhaustive, manual tests. So there's still a slight chance that a few strings got broken along the way.
We haven't translated everything yet. In particular, we haven't really started translating strings with interpolated components (as made possible by #321)

But we figured we should get the ball rolling with this first round, and we'll open additional PRs to keep going (and fix stuff if needed).
More importantly, using t('...') should become the norm whenever new localizable strings are introduced, and adding a bunch of those in the code should be a nudge in that direction 🙂
This PR does five things:

Fix a test that started failing when we added fr.*.json locale files
Replace a bunch of constant strings with t('TranslationKey')
Add the keys to en.client.json
Add fr.*.json files with French translations
Add a crowdin.yml config file

The Crowdin config file allowed us to work together on the translations in a web-based GUI. We created a temporary project linked to our fork of the repo.
Moving forward though, we think the Grist team should own the translation workflow.
Crowdin is one option, you would just have to create a project and use the GitHub integration which would be automatically setup thanks to the config file. It's free for open-source projects.
There are other options to consider, for instance @yohanboniface suggested Transifex or Weblate which are open-source. It's up to you guys :)

Thanks a lot @georgegevoian for doing the tedious work of reviewing this loooong PR in detail 🙏🏻
Hopefully I fixed all the errors you noticed in the last commit 🤞🏻

Side note: did you think about adding linters to harmonize the formatting across files?

Thanks a lot @georgegevoian for doing the tedious work of reviewing this loooong PR in detail 🙏🏻 Hopefully I fixed all the errors you noticed in the last commit 🤞🏻
Side note: did you think about adding linters to harmonize the formatting across files?

We've looked into it briefly. There's some eslint configuration we need to move to the repo first before we can start integrating with a formatter, like Prettier. I think it's a good idea though (and sorry that our formatting is a bit all over the place).

Moving forward though, we think the Grist team should own the translation workflow. Crowdin is one option, you would just have to create a project and use the GitHub integration which would be automatically setup thanks to the config file. It's free for open-source projects. There are other options to consider, for instance @yohanboniface suggested Transifex or Weblate which are open-source. It's up to you guys :)

Crowdin looks easy to set up, and you've already used it, so it seems a good option. For their free open-source plan, though, one of the requirements is: You do not have any commercial products related to the open-source project you are requesting a license for - but Grist Labs does in fact have related commercial products. I see that Weblate has a similar free hosted offer with a different set of listed requirements, all of which we could meet. If I understand correctly, Crowdin incorporates machine translation suggestions - are there features of Crowdin that made the translation work go faster?

We've looked into it briefly. There's some eslint configuration we need to move to the repo first before we can start integrating with a formatter, like Prettier. I think it's a good idea though (and sorry that our formatting is a bit all over the place).

@georgegevoian I now rely only on eslint to handle formatting, thanks to rules like this one.

Crowdin looks easy to set up, and you've already used it, so it seems a good option. For their free open-source plan, though, one of the requirements is: You do not have any commercial products related to the open-source project you are requesting a license for - but Grist Labs does in fact have related commercial products. I see that Weblate has a similar free hosted offer with a different set of listed requirements, all of which we could meet. If I understand correctly, Crowdin incorporates machine translation suggestions - are there features of Crowdin that made the translation work go faster?

@paulfitz I'm guessing you meant to ask if Weblate has features that make the translation work go faster. I have never used it personally. Looking at this page, I see a "Automatic suggestions" button in the screenshot, but they don't advertise it in the text so I'm not sure.
I'll ping @yohanboniface to see if he knows more, otherwise I can give it a try.

Oh OK, then yes! Crowdin's automatic suggestions were quite good and easy to use.

Hey amigos, anything left we can do to have this merged and released ? Can't way to have Grist in French for our users! :)
:partying_face: :rocket: :fr:

@dsagal is this included in the "current latest" Docker image ? (https://registry.hub.docker.com/layers/gristlabs/grist/latest/images/sha256-8d84a71a767059ef8c145572b17b381a775a8c05980c843e1069cc27f67df84b?context=explore)
I've deployed it but the UI is still in English, so before going diving I'd like to be sure it's included :)
Thanks! :)

The latest Docker image contains the material in the latest branch. The main branch is currently not passing tests, which is blocking recent commits from being included in the image. We're working on it.

OK, thanks for the info :)

@yohanboniface this should be in latest now :fr:


Time

hello,
I am working on VS 2015 ASP.Net C # and I can connect to google api (Drive) without problems.
When publishing to IIS shows error when it will connect to google.
var folder = System.Web.HttpContext.Current.Server.MapPath("/App_Data/MyGoogleStorage");

string[] scopes = new string[] { DriveService.Scope.Drive,
                                                 DriveService.Scope.DriveFile,
                                               DriveService.Scope.DriveAppdata};
 var clientId = "xxxxxx";
 var clientSecret = "xxxxx";
 var credential = GoogleWebAuthorizationBroker.AuthorizeAsync(new ClientSecrets
                {
                    ClientId = clientId,
                    ClientSecret = clientSecret

                },
                scopes,
                Environment.UserName,
                CancellationToken.None,
                new FileDataStore(folder)).Result;

Can you please show us the error and or stacktrace

This is my problem
http://stackoverflow.com/questions/36872721/google-calendar-api-net-time-out-exception-on-iis
https://postimg.org/image/stzfyblqr/

Can you give me some info on the IIS server you are deploying to?     Is it set up with all the default settings?    Is it host on a cloud server?  (Azure)

Is a local IIS 7.5 server on Windows 7.
These are the application settings: https://postimg.org/image/6gjupeknx/
These are the IIS configurations: https://postimg.org/image/uki3rn8rz/

Dear, you found a mistake that I could be making?.

I have very little experience with IIS i have looked at it as time has permitted me but I have not gotten very far.     I cant test it until i can actually deploy a site to IIS which i have not even managed to do as of yet.    I haven't worked with ASP in a number of years and the IIS server had been set up for me by our sys admin.   I will need to go though a number of tutorials to get IIS working first i guess.
I can tell you that i dont think your the only one with the problem i have seen a number of issues with this over the last few weeks.    Beyond that i cant tell you much.

Looks like the same as #888.
GoogleWebAuthorizationBroker is not suitable for use in IIS.


getRandomValues throwing error TypeError: Illegal invocation on Cloudflare Workers

Environment
<EMAIL_ADDRESS>node 18
Reproduction
Unfortunately, runnings wrangler@3 from stackblitz it's still not supported. The original issue was found on Nuxt3 but there the error stops the render of the page, in Nitro this seems like just been ignored.
https://stackblitz.com/edit/github-vk94cj-546phx?file=package.json
Steps:

npm run build

npm run simulate

Open debugger
Enable
Clear application storage
Enter in url

Describe the bug
Seems like when we are in Cloudflare worker environment, this kind of reference destructuring:
https://github.com/unjs/unenv/blob/2edb8fde481e3ee83d6b3c76de426a9d3d22da48/src/runtime/node/crypto/node.ts#L12-L22
Makes the invoke "Illegal"
Adding these lines makes work like a charm:
export const webcrypto = new Proxy(
  globalThis.crypto as typeof nodeCrypto.webcrypto,
  {
    get(_, key: keyof typeof globalThis.crypto | "CryptoKey") {
      if (key === "CryptoKey") {
        return globalThis.CryptoKey;
      }

+      if (typeof globalThis.crypto[key] === "function") {
+        // @ts-ignore
+        return globalThis.crypto[key].bind(globalThis.crypto);
+      }

      return globalThis.crypto[key];
    },
  },
);

Additional context
No response
Logs
TypeError: Illegal invocation
    at Object.getRandomValues (49seyck67av.js:1821:85)
    at 49seyck67av.js:1887:15
    at randomBits (49seyck67av.js:1888:5)
    at generateKey (49seyck67av.js:1906:18)
    at 49seyck67av.js:1938:22
    at seal (49seyck67av.js:1940:5)
    at 49seyck67av.js:2807:104
    at updateSession (49seyck67av.js:2809:6)
    at getSession (49seyck67av.js:2801:114)
    at useSession (49seyck67av.js:2765:9) {

Thanks for the nice reproduction and explanations!