Update zh.json

Optimizing Translation Contents
Description

Optimizing Translation Contents
Related Issue :
https://github.com/metatron-app/metatron-discovery/issues/291

How Has This Been Tested?

Need additional checks?
Types of changes

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

Thank you for your contribution! But.. I'm not sure if we can verify it properly. Maybe we should believe in you and then merge. :) Anyway, I'll check with my colleagues as much as we can.

run build
deploy to 1

@kyungtaak I cannot use Chinese on the test1 server. However, i can use Chinese on other test servers. Please check the test1 server.

@qinchaofeng There is no double quotation mark on the 1009 line.
Could you check it again? Thanks in advance for your response.

@minhyun2 sorry for that, I have added the double quotation,Thanks.

run build
deploy to 6

run build
deploy to 6

@qinchaofeng I am sorry to have deleted your comment on my mistake.
Thank you for your contribution.

run build
deploy to 6


API class does not support array parameters

I am trying to use Volto API class to make a request to plone 6 endpoint where the payload is object that contain an array as one of properties. The API class does not seem to support that. Am I missing something?
https://github.com/plone/volto/blob/0f84668688ce1d69030d2769d32a01d7dde28964/src/helpers/Api/Api.js#L60

@FarooqAlaulddin it's not clear what the problem is. Can you add a bit more context? code sample, etc?

@tiberiuichim sure! For example, I passed a similar params object to the API.get method:
 {
   searchTerm: 'power',
   page: 1,
   perPage: 10,
   fields: [
    {name: 'field1', value:'val for field 1'},
    {name: 'field2', value:'val for field 2'}
    {name: 'field3', value:'val for field 3'}
   ]
 }

This problem is the fields will not be treated correctly.
The Query String Prams will look something like this:

Instead of

it seems to be a problem with superagent.query. On top of that, Volto is using an old version of superagent, afaik, the changelog lists a change regarding this https://github.com/ladjs/superagent/releases/tag/v6.1.0
Would it be an option to serialize the query on your own?

Yes! Serialize the queries before sending is the way to go. Thank you.

@FarooqAlaulddin would you please add a new "faq" entry in https://github.com/plone/volto/issues/2810 with this problem and the solution? Thank you!


DependencyChange detection is wrong for runtime graph

Create a project as described in https://github.com/fsprojects/Paket/issues/2233

run paket restore and dependency change detection will go crazy on it. Basically every package will be reported as "not in deps file". I think the issue is that GetTransitiveDependencies is calculating wrong result.
Basically that function looks into all nuspecs and computes union of all deps. But since runtime dependency  resolution we retrieve deps from within the package. So this is no longer correct.
@matthid any chance you could fix that function?

 Yes I will fix this, but probably in a week (or maybe tomorrow).
Basically I think it just needs to ignore deps in the lockfile which are marked with isRuntimeDependency: true

nice, thanks!


Drilldown method to show any menu II

This is a recommit of my previous PR whose history got mixed up
https://github.com/zurb/foundation-sites/pull/10431
@kball as requested by you I've changed the auto focus feature now so it only gets triggered if explicitely setting the param to true
$drilldown.foundation('_showMenu', $menu, true);

Merged! Excellent.


Feature request: Create a Docker image

Pretty self-explanatory. Would be nice to be able to pull a Docker image instead of having to install all of the dependencies locally. Publishing the image to Docker hub would be awesome as well.

Coming in https://github.com/lyft/cartography/pull/32

Closing this as we are not accepting deployment/containerization configuration changes. See #159 for further discussion on this topic.


[NO MERGE] Fixing signed-off-by checker

This PR is just for fixing bug in signed-off-by checker.
DO NOT MERGE

Ah wrong dst branch -_-


e.preventDefault(); breaks scrolling when table is inside scrollable container

If a table is placed within a scrollable container (like in a web app), then this breaks scrolling if the table is 100% height.
When the table is 100% height, then vertical scrolling is the job of the scrollable container and not ag-grid. e.preventDefault(); will prevent the scrolling event from bubbling up to the scrollable container.
Related #781

100% height means the gird will be the height of the containing div.
eg lets say containing div is 100px, and then say each row is 10px high, you will only be showing 10 rows, and the grid will have scrolls for the the other rows, even if grid has 100000 rows.
if you are describign a grid with no vertical scrolls, then you have to set it's height to the height of all the rows, and not 100%???

Okay, I have tried to illustrate it to make it more clear.

So I have made a div that is overflow:auto (the green one). It is important to underline that this is not a div made by ag-grid. It is a div I have made which is part of my design.
Inside this div I have an ag-grid. Let us imagine the table has 10 rows, each row is 50 px high. Now imagine my ag-grid container is height:1000px. This means ag-grid does not create a vertical scrollbar, since the table is smaller than the container.
As shown on the picture a part of the ag-grid is outside the viewport. I therefore scroll, however I expect to scroll to happen by the green box (not ag-grid). However when I scroll the e.preventDefault(); will prevent the event from bubbling up to the green box.
Does it make sense now?

yes understood. sorry for asking explicitly, but i can get different issues mixed up. so to be clear, will the following fix it:
Have option in grid to not preventDefault on mouse wheel events?
This will have unexpected behaviour if the grid has scroll bars (as when you mouse wheel, the grid will scroll and the webpage will scroll) - however we don't care about this, if you set the switch, it's assuming you don't want scrolls in the grid.

Yes, I guess a good option would be to have an option to disable scrolling. This would also solve #801. However, I think it is important that we distinct between horizontal and vertical scrolling. I.e. you should be able to toggle horizontal and vertically scroll separately.
Should we close #801 and merge it into this issue?

What about adding a delay on mouseEnter before applying preventDefault on scroll ?
Or, another 2 cents idea, why do not allow scroll when scroll was already performing before mouseEnter ?
I think this way, all use cases should be covered.

@esbenp trying to look at this now, did you know at what parts of the code exactly we need to 'prevent default' ?

@esbenp i asked to soon, i see you referenced the line of code in the above. will look again. thanks.

i have surrounded the said code with the following:
        // allow the option to pass mouse wheel events ot the browser
        // https://github.com/ceolter/ag-grid/issues/800
        // in the future, this should be tied in with 'forPrint' option, or have an option 'no vertical scrolls'
        if (!this.gridOptionsWrapper.isSuppressPreventDefaultOnMouseWheel()) {
            // if we don't prevent default, then the whole browser will scroll also as well as the grid
            event.preventDefault();
        }

in other words, set the property suppressPreventDefaultOnMouseWheel=true and then the grid will not prevent default on the mouse wheel event. we plan to release this week so you should have this fix soon.


ccd-js-gen: Generate functions for generating web wallet parameters

Purpose
Closes #298.
Changes

Fix bug in JSON generated for schema type ULeb128 and ILeb128.
Generate functions for constructing TypedParameter found in @concordium/wallet-connectors.

Checklist

[x] My code follows the style of this project.
[x] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.
[x] (If necessary) I have updated the CHANGELOG.

Looks good 😄
A side note: I could not run the examples by executing yarn run-example <path>. I don't know what has changed, but I had to do npx tsx <path> instead. Can you successfully run the examples?

I managed to run all of those that are related to ccd-js-gen. Did you make sure to run yarn run-example ./wCCD/generate.ts --endpoint http://node.testnet.concordium.com:20000 as the first thing?


Updated Roadmap Link to point to the finished Roadmap instead of the proposal.

I can't believe GitHub doesn't persist the old links. It was the same page! But thanks for this. Very useful!


Use clang-cl for Windows

I am trying to build the llnode module on Windows, which can only be built with clang-cl. The project previously (ab)used the passthrough of arguments to msbuild to configure the CLToolPath and CLToolExe properties, which has hence been fixed in #1164. (nodejs/llnode#321, nodejs/llnode#377)
I have attempted the following to no avail,

Add in binding.gyp

"msbuild_settings": {
    "ClCompile": {
        "CLToolPath": "<(cl_tool_path)",
        "CLToolExe": "<(cl_tool_exe)",
    },
},

This does put the properties into the vcxproj, but in the ClCompile section. https://stackoverflow.com/a/31362960/12385184

CC and CXX environment variables (https://github.com/nodejs/node-gyp/issues/2813#issuecomment-1462523117)

This does have some effect on the configure phase, but only accepts GCC (not CL) compatible compilers, and it has no effect on the building phase which still uses CL.exe.

This issue might be a case of #1537 if there is no special way to specify the compiler executable on Windows.

Node Version: node v18.20.4 npm 10.7.0
Platform: Microsoft Windows 11 Enterprise 10.0.22631 N/A Build 22631 x64-based PC
Compiler: MSBuild version 17.11.2+c078802d4 for .NET Framework <IP_ADDRESS>701 MSVC 19.41.34120 for x64
Module: https://github.com/nodejs/llnode

Without changes to gyp and without breaking backward compatibility, the only way I can think of to pass arbitrary parameters is to prepend some character illegal in msbuild target names, such as ? or (. It is going to be rather ugly that way though... node-gyp build target1 target2 ?/p:CLToolExe="C:\bin\LLVM\clang-cl.exe" ?/p:CLToolPath="C:\bin\LLVM"
A reasonable compromise might be to pass through all arguments beginning with / to msbuild, on the assumption that target names rarely begins with a slash (although it is probably legal).
@cclauss any comments?

Sorry, I am not a Windows user.


Unable to run godot-rl (stablebaselines3)

I followed the following steps from this tutorial that was listed on the godot_rl github page: https://www.youtube.com/watch?v=f8arMv_rtUU
My computer is running Ubuntu 22.04.4 LTS but I have tried with 24.04 LTS as well and there is no change in result
These are the steps I followed (as stated in the video):
Step 1-3

Step 4 - Download raw file

Step 5 - Place it alongside the rl folder

-----End of Python Setup----
Step 6 - Create project

Now the filesystem looks like this:

Step 7 - Create CharacterBody3D

Step 8 - Add emission

Step 9 - Add floor

Step 10 - Add target(Emission included in this step)

Step 11 - Add walls

Step 12 - Add template script for player

Step 13 - Add Camera3D

Step 14 - Re-arrange target and cube in the scene

Step 15 - Add signals for wall and target to move cube back to original position

Step 16 - Comment out control code

Step 17 - Download the godot_rl_agents_plugin-main.zip from github

Step 18 - Select the script_templates and addons folder

Step 19 - Copy them over

Step 20 - Enable the Plugin

Step 21 - Add and extend script of AIController3D

Add the script_templates script and edit it for purpose of the project

Alter the cube.gd script for AI to control velocity

Step 22 - Finally run the script in the virtual environment through: python3 stable_baselines3_example.py
Step 23 - Add the Sync node in godot
After all this an error appears:

Please check if I've missed a step, I've tried this in both the version of godot used in the tutorial (v4.2.dev1) and the latest stable version, but there is the same issue
Adding ": Node3D" in step 15 does not make a difference to the result
It would be very much appreciated if you could help with this issue
Thanks in advance

Hello, I think it's a version mismatch issue. Try installing the latest version of Godot RL Agents from this repository (main branch).

Thanks for the reply!
This might be a silly question, but are you saying to download the entire main branch(including examples, docs, godot_rl, godot_rl_agents_plugin) or just a specific folder?
If it is a specific folder, it would be really helpful if you could send the link. Just want to be sure I'm choosing the right one.

For the main project, try using pip install https://github.com/edbeeching/godot_rl_agents/archive/refs/heads/main.zip.
For plugin (the files to copy to the Godot project), use the plugin repository to get the latest version: https://github.com/edbeeching/godot_rl_agents_plugin

I used the command in the environment and installed the updated plugin but unfortunately the problem still persists.
I looked a little deeper into the error and it looks like the handshake does complete but then it returns a null value that the handle_message() function cannot manage and the server disconnects.
My logic could be mistaken as there might be a problem with the stable_baselines3_example.py(even though I've downloaded it from the correct branch) but here are some images of the error:

Perhaps you can try an example from the examples repository: https://github.com/edbeeching/godot_rl_agents_examples
You could also try unistalling godot rl agents with pip uninstall before installing (if you didn't try already).

Thanks for the advice, will try and complete one of the examples before coming back to this

I got the example working. Thanks for the help!


Doc/seo

Add topics addressed in Jorge’s Search Engine Optimisation.pptx presentation to the PyAnsys Developer’s group on 4/18/24.
@jorgepiloto @RobPasMue I struggled with where to put this information and decided on the How To > Documenting. Feel free to move and edit. I did add a link from the index file for the RST files section for content writing. I hope to start an outline next week for integrating the Content Writing into the dev guide proper.
Also, ever since the update of Ansys actions to version 6, I've been unable to run Vale locally in any PyAnsys repository. I aways get this error, even after running git clean -f -d -x doc:
(kp_venv) C:\AnsysDev\GitRepos\PyAnsys\dev-guide\doc>vale .
E100 [vocab] Runtime error
'ANSYS' does not exist
This is why I have to wait for the CI/CD to find them and then make so many commits to fix Vale issues!

Also, ever since the update of Ansys actions to version 6, I've been unable to run Vale locally in any PyAnsys repository. I aways get this error, even after running git clean -f -d -x doc:
(kp_venv) C:\AnsysDev\GitRepos\PyAnsys\dev-guide\doc>vale .
E100 [vocab] Runtime error
'ANSYS' does not exist
This is why I have to wait for the CI/CD to find them and then make so many commits to fix Vale issues!

I believe your error might come from the Vale version you are using @PipKat - could you run locally vale --version?

Also, ever since the update of Ansys actions to version 6, I've been unable to run Vale locally in any PyAnsys repository. I aways get this error, even after running git clean -f -d -x doc:
(kp_venv) C:\AnsysDev\GitRepos\PyAnsys\dev-guide\doc>vale .
E100 [vocab] Runtime error
'ANSYS' does not exist
This is why I have to wait for the CI/CD to find them and then make so many commits to fix Vale issues!

I believe your error might come from the Vale version you are using @PipKat - could you run locally vale --version?

I bet you are right. It's really old: 2.29.1. Do I do pip install vale or does a version need to be specified? Thanks!


Change discuss script URL pattern

For some reason when URL for Disqus script starts from 'https://' comments are
not shown. I changed it to start from '//' like instructions
say and now it works fine. Not sure what was the problem, but maybe this change
will help someone else.

How are you hosting your site? Is it served over HTTPS?
Using protocol-relative URLs is an anti-pattern, so it’d be nice if a better solution could be worked out.

I am hosting my site on github. I tried both https and http and neither worked.
However after I reverted changes back to "//", it has stopped working at all. Now neither original minima's template nor mine "patched" version don't work. Despite the fact that latter one was working before.
Also I noticed that absolute_url filter does not work as expected on github:
 var disqus_config = function () {
      this.page.url = '/install-elk-windows.html';
      this.page.identifier = '/install-elk-windows.html';
    };

Seems like this is causing issues. Anyway I am canceling this PR since it doesn't seem to fix the issue.


Unhandled error in Deferred | TypeError: Cannot create a consistent method resolution

Hello there, im having an issue with the autoextract package. I'm not able to get my script to pull up and start working.
My project is a news scraping tool that uses zyte automatic extraction API to scrape articles given some URLs.
It's built following the scrapy project guidelines, i have my so called 'googlenews' spider that performs the task i described before.
Since 08/05/2023, my script stopped working without me making any change to the source code.
Just when the program is starting i get the following error trace:
Unhandled error in Deferred:
2023-08-07 04:00:05 [twisted] CRITICAL: Unhandled error in Deferred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/scrapy/crawler.py", line 245, in crawl
    return self._crawl(crawler, *args, **kwargs)
  File "/usr/local/lib/python3.11/site-packages/scrapy/crawler.py", line 249, in _crawl
    d = crawler.crawl(*args, **kwargs)
  File "/usr/local/lib/python3.11/site-packages/twisted/internet/defer.py", line 1947, in unwindGenerator
    return _cancellableInlineCallbacks(gen)
  File "/usr/local/lib/python3.11/site-packages/twisted/internet/defer.py", line 1857, in _cancellableInlineCallbacks
    _inlineCallbacks(None, gen, status, _copy_context())
--- <exception caught here> ---
  File "/usr/local/lib/python3.11/site-packages/twisted/internet/defer.py", line 1697, in _inlineCallbacks
    result = context.run(gen.send, result)
  File "/usr/local/lib/python3.11/site-packages/scrapy/crawler.py", line 134, in crawl
    self.engine = self._create_engine()
  File "/usr/local/lib/python3.11/site-packages/scrapy/crawler.py", line 148, in _create_engine
    return ExecutionEngine(self, lambda _: self.stop())
  File "/usr/local/lib/python3.11/site-packages/scrapy/core/engine.py", line 98, in __init__
    self.downloader: Downloader = downloader_cls(crawler)
  File "/usr/local/lib/python3.11/site-packages/scrapy/core/downloader/__init__.py", line 97, in __init__
    DownloaderMiddlewareManager.from_crawler(crawler)
  File "/usr/local/lib/python3.11/site-packages/scrapy/middleware.py", line 67, in from_crawler
    return cls.from_settings(crawler.settings, crawler)
  File "/usr/local/lib/python3.11/site-packages/scrapy/middleware.py", line 43, in from_settings
    mwcls = load_object(clspath)
  File "/usr/local/lib/python3.11/site-packages/scrapy/utils/misc.py", line 79, in load_object
    mod = import_module(module)
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import

  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load

  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked

  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked

  File "<frozen importlib._bootstrap_external>", line 940, in exec_module

  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed

  File "/usr/local/lib/python3.11/site-packages/scrapy_autoextract/__init__.py", line 2, in <module>
    from .providers import AutoExtractProvider  # noqa: F401
  File "/usr/local/lib/python3.11/site-packages/scrapy_autoextract/providers.py", line 17, in <module>
    from autoextract_poet.page_inputs import (
  File "/usr/local/lib/python3.11/site-packages/autoextract_poet/__init__.py", line 2, in <module>
    from .pages import *
  File "/usr/local/lib/python3.11/site-packages/autoextract_poet/pages.py", line 31, in <module>
    class AutoExtractItemWebPage(AutoExtractWebPage, ItemPage):
  File "<frozen abc>", line 106, in __new__

builtins.TypeError: Cannot create a consistent method resolution
order (MRO) for bases Injectable, Generic

2023-08-07 04:00:05 [twisted] CRITICAL:
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/twisted/internet/defer.py", line 1697, in _inlineCallbacks
    result = context.run(gen.send, result)
  File "/usr/local/lib/python3.11/site-packages/scrapy/crawler.py", line 134, in crawl
    self.engine = self._create_engine()
  File "/usr/local/lib/python3.11/site-packages/scrapy/crawler.py", line 148, in _create_engine
    return ExecutionEngine(self, lambda _: self.stop())
  File "/usr/local/lib/python3.11/site-packages/scrapy/core/engine.py", line 98, in __init__
    self.downloader: Downloader = downloader_cls(crawler)
  File "/usr/local/lib/python3.11/site-packages/scrapy/core/downloader/__init__.py", line 97, in __init__
    DownloaderMiddlewareManager.from_crawler(crawler)
  File "/usr/local/lib/python3.11/site-packages/scrapy/middleware.py", line 67, in from_crawler
    return cls.from_settings(crawler.settings, crawler)
  File "/usr/local/lib/python3.11/site-packages/scrapy/middleware.py", line 43, in from_settings
    mwcls = load_object(clspath)
  File "/usr/local/lib/python3.11/site-packages/scrapy/utils/misc.py", line 79, in load_object
    mod = import_module(module)
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/local/lib/python3.11/site-packages/scrapy_autoextract/__init__.py", line 2, in <module>
    from .providers import AutoExtractProvider  # noqa: F401
  File "/usr/local/lib/python3.11/site-packages/scrapy_autoextract/providers.py", line 17, in <module>
    from autoextract_poet.page_inputs import (
  File "/usr/local/lib/python3.11/site-packages/autoextract_poet/__init__.py", line 2, in <module>
    from .pages import *
  File "/usr/local/lib/python3.11/site-packages/autoextract_poet/pages.py", line 31, in <module>
    class AutoExtractItemWebPage(AutoExtractWebPage, ItemPage):
  File "<frozen abc>", line 106, in __new__
TypeError: Cannot create a consistent method resolution
order (MRO) for bases Injectable, Generic

Seems to me that the relevant part of the error trace is:
class AutoExtractItemWebPage(AutoExtractWebPage, ItemPage):
  File "<frozen abc>", line 106, in __new__
TypeError: Cannot create a consistent method resolution
order (MRO) for bases Injectable, Generic

Looks like some problem with the autoextract library class inheritance settings. So because of it being an internal issue (at first sight) im a bit stucked here.
My environment:

Linux vps-87cd9c3d 5.4.0-153-generic #170-Ubuntu SMP Fri Jun 16 13:43:31 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Distributor ID: Ubuntu
Description:    Ubuntu 20.04.6 LTS
Release:        20.04
Codename:       focal

Bash version: 5.0.17(1)-release

I use docker compose and this is how mi image is defined:
pyprojects:
    container_name: "pyprojects"
    image: python
    restart: unless-stopped
    tty: true
    environment:
      - HOST=<IP_ADDRESS>
      - PORT=1420
      - DEV=true
    privileged: true
    entrypoint: /pyprojects/entrypoint.sh
    volumes:
      - ../../data/pyprojects/:/pyprojects
    ports:
      - "1420:1420"

Python version:

Python 3.11.4

Installed libs:
scrapy
dataclasses
dataclasses_json
langdetect

# used by autonews
scrapy-autoextract
pymongo
gspread

I can imagine that this is something related with my local host or python environment, i've tried the python:3.10.12 image tag but the problem is still the same.
I hope you can give me some insight about this because i'm probably not the only one facing with this problem, but i can't find any related post (refering to zyte autoextract) browsing google.
I'll be glad to give you any more information you need and discuss the issue if needed.

Some zyte-autoextract dependencies have evolved and became incompatible with zyte-autoextract.
As a workaround, the following dependency freezes might help:
web-poet==0.9.0
autoextract-poet==0.4.0

You mean scrapy-autoextract dependencies that became incompatible with zyte-autoextract?
Should I move from scrapy-autoextract to zyte-autoextract or set that dependency versions in my requirements.txt straight forward?

Should I move from scrapy-autoextract to zyte-autoextract

No, sorry for the confusion.

or set that dependency versions in my requirements.txt straight forward?

This.
In fact, you only need to freeze web-poet (web-poet==0.9.0), I have just checked and, as I suspected, autoextract-poet 0.4.0 is the latest version already.

Thanks Gallaecio, after freezing web-poet all the process began to work again as expected.
With the provided error message i thought it would be more difficult to identify the problem.
Is this something you have alerady seen in the zyte team? Seems a bit tricky to me that you have to configure such freezes in order to make the lib work, are you guys working on that or is there something im missing in the trace that blamed this specific dependency incompatibility?
It will be great to know in order to improve my deep understanding of the library as its part of mi service core.
That being said, thank you so much for the help.

Library freezing is a general best practice when working on a Python project (as opposed to a reusable package).
However, what matters here is that zyte-autoextract should generally not be used anymore for new projects. So existing projects should have frozen dependencies when they installed it, and should not break, and new projects should use Zyte API automatic extraction through scrapy-zyte-api, which is compatible with the latest web-poet.


"Măcăi" is listed in the drop-down menu

Description:
"Măcăi" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Măcăi" in the search bar.

Expected results
The location is listed in the drop-down menu as "Măcăi, Argeș (Cuca)".


[config] simplify UpstreamConfig

Motivation
Simplify existing UpstreamConfig s.t.:

upstream config now only defines the upstream network of a node
validators do not need to explicitly set its validator network as upstream
This PR also updates mempool and state sync uses the actual NetworkId to identify networks, instead of its local peer ID within that network.

Test Plan
Existing/refactored tests

@bors-libra r=davidiw

:pushpin: Commit 1e15cbe has been approved by davidiw

:hourglass: Testing commit 1e15cbe8553450ba67e30f0c3c520a145c348b04 with merge 22cf18c8fada8129a9f946169e50594eba1de925...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: davidiw
Pushing 22cf18c8fada8129a9f946169e50594eba1de925 to master...


#Integrate data received from the API

Description

Integrates API data as opposed to previously loading from the data.json file.
Please include a summary of the changes and the related issue(s) this PR addresses.
Fixed the following issue or added a new feature

Fixes #(93)
Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Testing Checklist

[ ] My changes generate no new warnings

Screenshots / Recordings (if applicable)

Please add any screenshots that demonstrate the changes made.

Looks good, but we'll need to make sure infinite scroll works before merging this to prod.
Currently it only loads the 15 names then stops there.

Looks good, but we'll need to make sure infinite scroll works before merging this to prod.
Currently it only loads the 15 names then stops there.

Is someone available to spawn off this branch and re-enable infinite scroll?

@armstrongsouljah the analytics page, is also currently using the data.json file. Can you please update it to pull the data from the API endpoint? So we hit two birds with one stone?

Is someone available to spawn off this branch and re-enable infinite scroll?

This has been updated.


Log errors from commands which don't return success response

Fixes #70.

Codecov Report
Base: 86.75% // Head: 86.73% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (369a9e2) compared to base (93d5edb).
Patch coverage: 85.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
- Coverage   86.75%   86.73%   -0.02%
==========================================
  Files          33       33
  Lines        1963     1983      +20
  Branches      226      212      -14
==========================================
+ Hits         1703     1720      +17
- Misses        193      195       +2
- Partials       67       68       +1

Impacted Files
Coverage Δ

ansq/tcp/connection.py
66.14% <50.00%> (-0.31%)
:arrow_down:

ansq/tcp/types/response_schemas.py
98.11% <100.00%> (+0.11%)
:arrow_up:

tests/test_connection.py
95.31% <100.00%> (+0.97%)
:arrow_up:

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

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

Thanks @Ivashkaization!


typo: replace comma with and

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #3   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           8      8
  Lines         235    235
=====================================
  Hits          235    235

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 5184bfa...b20bd70. Read the comment docs.


Port medibot + bot spawners from nyano

About the PR
Playtested on Nyano since May. The reagent selection is pretty careful so they don't obsolete doctors but help lessen the load a bit on GOMERs with 5 burn damage or giving a bit of extra time for crit people.
Changelog

:cl:

add: Ported medibots from Nyanotrasen MRP.

oh it's getting the crash again I thought was fixed
I think it was introduced about ~3 weeks ago by https://github.com/space-wizards/RobustToolbox/pull/3006

Requires https://github.com/space-wizards/RobustToolbox/pull/3054 I guess

Engine update fixed the failing tests


Move LegacyBehaviorPolicy references to shim layer [databricks]

Part of https://github.com/NVIDIA/spark-rapids/issues/8121
Spark 3.5.0 moves LegacyBehaviorPolicy out of SQLConf and into a top-level class, so we now need to shim all references to this class.

build

does it make sense to merge before spark350 build is added?

That was my plan. There are many changes like this needed for 3.5.0 support, so I was hoping to introduce some of them incrementally to keep the size of the final PR down to a manageable size.

build

build

build


[bug] backend apps: "ReferenceError: XMLHttpRequest is not defined"

It seems that the SDK assumes a browser environment because I get this error on Node.js.
ReferenceError: XMLHttpRequest is not defined
    at Object.<anonymous> (/Users/remon/projects/sdk-sandbox/node_modules/@polymathnetwork/sdk/dist/index.js:21:140249)

To reproduce the issue, just create a vanilla node.js project, add polymath-sdk, and then run the following js snippet:
const { Polymath } = require('@polymathnetwork/sdk');

I had this issue at some point, and made it so that the SDK distributes two different bundles (one for the browser and one for node)
I tried this just now (beta.46) running the snippet with node test.js and everything worked fine.

I tried too and it worked without problems, could you test it again @remon-nashid?

Works now. Thanks


add sys_windows.go

Minimal (not tested, but allows compilation) windows support.

I'll try to test this on my windows VM tonight

Hey @tgulacsi thanks for the PR. Instead of just allowing windows compilation, I fixed up the unsupported.go file so it allows compilation on all unsupported OSs.

Great!


Panic when out of file descriptors

https://github.com/tiny-http/tiny-http/blob/master/src/util/refined_tcp_stream.rs#L45 can panic when there are no available file descriptors.
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 24, kind: Other, message: "No file descriptors available" }', /source/tiny_http-0.7.0/src/util/refined_tcp_stream.rs:44:54

Rather than panic an error should be returned, so that the calling code can decide on the best course of action.

This will be a breaking change, but we can definitely return a Result here rather than blindly panicking.

Are you having the issue where a tiny-http::Server is panicking because of the code in RefinedTcpStream? As the latter type isn't exported by the library so I assume you aren't using it directly.
It would be useful if you could capture the above error with a full backtrace, but my suspicion is it's coming from the acceptor thread that dispatches to the worker pool.
Handling this situation is interesting, if the acceptor thread has run out of FDs, there's not a lot we can do with it. We need to use try_clone to split the TcpStream into a reader and a writer, and doing so consumes one additional file descriptor. Having the acceptor thread bubble an error up to the caller doesn't feel useful, as the connection we couldn't handle has already been lost.

FWIW, panic is also the results if the limit on threads is hit. As a consequence of the panic, a Mutex is poisoned, and recovery is pretty much impossible, AFAICT.
Feb 08 00:47:23: thread '' panicked at 'failed to spawn thread: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }', /rustc/db9d1b20bba1968c1ec1fc49616d474>
Feb 08 00:47:23: note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Feb 08 00:47:24: thread '' panicked at 'failed to spawn thread: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }', /rustc/db9d1b20bba1968c1ec1fc49616d474>
Feb 08 00:47:29: thread '' panicked at 'called Result::unwrap() on an Err value: PoisonError { .. }', /home/ulrikm/.cargo-docker/registry/src/github.com-1ecc6299db9ec823/tiny_http>
Feb 08 00:47:30: thread '' panicked at 'called Result::unwrap() on an Err value: PoisonError { .. }', /home/ulrikm/.cargo-docker/registry/src/github.com-1ecc6299db9ec823/tiny_http>

Not sure what is the right approach to fix this. For me I typically use tiny_http as an embedded server in applications, where the web-serving is held in a side-thread. I think there should either be some mechanism to automatically recover from these errors, and/or it needs to be very clearly spelled out in the documentation that the application is expected to handle it.

Added tests for the two different scenarios of "Out of file-handles" and "Out of threads" to a draft PR. https://github.com/tiny-http/tiny-http/pull/219

I'm usingtiny-http and rouille as a stand-alone API server. I'm hitting the poison issue on thread spawn failure.  I filed #220 and #221.

@bradfier Sorry, I realized first now I didn't exactly answer the question. The problem with how tiny-http works right now IMO, is that the internal Mutex is poisoned, and never recovers. That means that if the service temporarily hits too many concurrent connections, not only are the "overflowing" connections rejected, but the service is left permanently in a defunctional state. I.E. even if all other connections disconnect, the service will no longer accept new connections either.
This is what I tried to cover with the drafted test-cases. They start a server in a subprocess, arbitrarily confined with narrow resource-limits. The tests then make sure to hit those limits, (assert!(clients.iter().any(Result::is_err));), releases all connections, and tries connecting again.
I've clean up the tests a little bit to better illustrate.


ENH: Add probot-stale GitHub bot configuration

This is the configuration for the probot-stale GitHub App:
https://probot.github.io/apps/stale/
that automatically close stale Issues and Pull Requests that tend to accumulate during a project.

I agree with @jhlegarreta, the automatic comment after a period of inactivity is great, but closing issues and PR's seems like too much. Not strongly opinionated against it though.

I respectfully (but completely) disagree with @jhlegarreta & @phcerdan :).

Keeping the most important tasks front and center is really important.
long lists of stale/inactive issues/tasks consume an inordinate amount of resources (resources spent trying to ignore outdated items).
The items can be re-opened and commented on.  In order to keep an item open for a complete year only requires a single stake owner to add "Still important" comment once every 3 months.  If no stakeholder is willing to write a single word in an issue or pull request every 3 months, then it is highly likely that the effort of filtering out that task/PR during other duties exceeds the likelihood of the task getting completed.

This is my 2 cents, but projects that are good at identifying what is currently relevant and prioritizing those items often perform better.

I agree with Hans that inactive things should be close if no one is even willing to comment infrequently. And we can find a list of items closed due to being stale by filtering by label.

That makes sense @hjmjohnson. Although, there are some issues that go to a different category. They do not require immediate action but maybe deal with structural or more long term changes, some are open, unsolved questions. Commenting on those is usually unnecessary, nothing new to be added, but the issue hasn't disappeared or staled. I see there are flags: pinned, security in the example, that can be used to not close these.
Getting an issue closed just because it has been ignored, or no more time has been put into it... The issue is still there. I guess it depends on the workflow of the maintainers, and what they consider an issue. Is an issue something that has to be taken care of? Or can an issue be a conversation or tracking about some topic?

I also do not like closing issues that are actually outstanding, but I can understand the value of removing clutter and prioritizing. As I understand it, the bot should add a status:Backlog label, so we know why an issue is closed, and we can still filter and search for these issues. Let's try it, and see how it goes. Maybe Marie Kondo will feature our GitHub repository in one of her books someday :jp: :sparkle: :joy:


Exec command timeout on cdc node sometimes

What did you do?
Run cdc redo log system test cases, for each case basically below steps included:

create changefeed with redo log enabled
run workload
run redo apply
remove changefeed

What did you expect to see?
execute command on cdc node should not timeout
What did you see instead?
Execute command on TiCDC seen timeout error seen sometimes:
example 1:  execute echo commend timeout
2023-06-18T21:52:20.768Z        INFO    host/ticdc.go:283       Execute command on ticdc        {"command": "echo -e >/tmp/changefeed.toml \"[cons
istent] \n\"\"level = \\\"eventual\\\"\nstorage = \\\"s3://tmp/test-infra-redolog/redo-apply-cdc-all-node-restart-sync-bee28932-a8f8-4289-bd09-9d1
37926dcad?access-key=minioadmin&secret-access-key=minioadmin&endpoint=http://<IP_ADDRESS>:9000&force-path-style=true\\\"\nmax-log-size = 64 \n\"", "
timeout": "10s", "ticdc peer": 0}
2023-06-18T21:52:20.768Z        INFO    k8s/client.go:132       it should be noted that a long-running command will not be interrupted even the us
e case has ended. For more information, please refer to https://github.com/pingcap/test-infra/discussions/129
[2023/06/18 21:52:30.770 +00:00] [PANIC] [main.go:114] ["run case failed"] [error="Exec command timeout"] [errorVerbose="Exec command timeout\ngit
hub.com/pingcap/test-infra/caselib/pkg/host.init\n\tgithub.com/pingcap/test-infra/caselib/pkg/host/host.go:16\nruntime.doInit\n\truntime/proc.go:6
506\nruntime.doInit\n\truntime/proc.go:6483\nruntime.doInit\n\truntime/proc.go:6483\nruntime.main\n\truntime/proc.go:233\nruntime.goexit\n\truntim
e/asm_amd64.s:1598"] [stack="main.main\n\t./main.go:114\nruntime.main\n\truntime/proc.go:250"]
panic: run case failed

example 2:  execute changefeed remove command timeout
2023-06-18T22:42:45.719Z        INFO    host/ticdc.go:299       Execute command on ticdc        {"command": "/cdc version", "ticdc peer": 0}
2023-06-18T22:42:45.719Z        INFO    k8s/client.go:132       it should be noted that a long-running command will not be interrupted even the us
e case has ended. For more information, please refer to https://github.com/pingcap/test-infra/discussions/129
[2023/06/18 22:43:04.528 +00:00] [INFO] [ticdc.go:199] ["get TiCDC version"] [cdcVersion=v7.2.0-alpha]
2023-06-18T22:43:04.528Z        INFO    host/ticdc.go:283       Execute command on ticdc        {"command": "/cdc  cli  changefeed  remove \"--ser
ver=<IP_ADDRESS>:8301\" \"--changefeed-id=redo-enable-cdc-all-node-restart-sync\"", "timeout": "10s", "ticdc peer": 0}
2023-06-18T22:43:04.529Z        INFO    k8s/client.go:132       it should be noted that a long-running command will not be interrupted even the us
e case has ended. For more information, please refer to https://github.com/pingcap/test-infra/discussions/129
[2023/06/18 22:43:14.532 +00:00] [ERROR] [ticdc.go:72] ["run changefeed operation timeout"] [stack="github.com/pingcap/test-infra/caselib/pkg/host
.(*TiCDCHost).ChangeFeed\n\tgithub.com/pingcap/test-infra/caselib/pkg/host/ticdc.go:72\ngithub.com/pingcap/test-infra/caselib/pkg/steps.(*changeFe
edTask).Execute\n\tgithub.com/pingcap/test-infra/caselib/pkg/steps/cdc.go:80\ngithub.com/pingcap/test-infra/caselib/pkg/steps.(*Serial).Execute\n\
tgithub.com/pingcap/test-infra/caselib/pkg/steps/step.go:31\nmain.main\n\t./main.go:113\nruntime.main\n\truntime/proc.go:250"]
[2023/06/18 22:43:14.532 +00:00] [INFO] [cdc.go:81] ["run changeFeed task finished"] [result=null] [error="Exec command timeout"] [errorVerbose="E
xec command timeout\ngithub.com/pingcap/test-infra/caselib/pkg/host.init\n\tgithub.com/pingcap/test-infra/caselib/pkg/host/host.go:16\nruntime.doI
nit\n\truntime/proc.go:6506\nruntime.doInit\n\truntime/proc.go:6483\nruntime.doInit\n\truntime/proc.go:6483\nruntime.main\n\truntime/proc.go:233\n
runtime.goexit\n\truntime/asm_amd64.s:1598"]

Versions of the cluster
TiCDC version (execute cdc version):
["Welcome to Change Data Capture (CDC)"] [release-version=v7.2.0-alpha] [git-hash=5c368b52c0de831c0b9dd68ff66874d5a37f74ca] [git-branch=heads/refs/tags/v7.2.0-alpha] [utc-build-time="2023-06-16 11:35:10"] [go-version="go version go1.20.5 linux/amd64"] [failpoint-build=false]

clinic: https://clinic.pingcap.com.cn/portal/#/orgs/31/clusters/7246143009251734031?from=1687124126&to=1687130951

/found automation

/severity moderate

also sometime exec timeout seen when creating changefeed:
[2023/07/04 00:41:18.905 +00:00] [INFO] [ticdc.go:201] ["get TiCDC version"] [cdcVersion=v7.3.0-alpha]
2023-07-04T00:41:18.905Z	INFO	1815723	host/ticdc.go:301	Execute command on ticdc	{"command": "/cdc  cli  changefeed  create \"--server=<IP_ADDRESS>:8301\" \"--sink-uri=mysql://root:@downstream.cdc-testbed-tps-1815723-1-93:3306\" \"--changefeed-id=cdc-scale-out-in\" \"--no-confirm\"", "timeout": "10s", "ticdc peer": 0}
2023-07-04T00:41:18.905Z	INFO	k8s/client.go:132	it should be noted that a long-running command will not be interrupted even the use case has ended. For more information, please refer to https://github.com/pingcap/test-infra/discussions/129
[2023/07/04 00:41:28.906 +00:00] [ERROR] [ticdc.go:72] ["run changefeed operation timeout"] [stack="github.com/pingcap/test-infra/caselib/pkg/host.(*TiCDCHost).ChangeFeed\n\tgithub.com/pingcap/test-infra/caselib/pkg/host/ticdc.go:72\ngithub.com/pingcap/test-infra/caselib/pkg/steps.(*changeFeedTask).Execute\n\tgithub.com/pingcap/test-infra/caselib/pkg/steps/cdc.go:80\ngithub.com/pingcap/test-infra/caselib/pkg/steps.withRecover\n\tgithub.com/pingcap/test-infra/caselib/pkg/steps/step.go:24\ngithub.com/pingcap/test-infra/caselib/pkg/steps.(*Serial).Execute\n\tgithub.com/pingcap/test-infra/caselib/pkg/steps/step.go:45\nmain.main\n\ttest-infra/caselib/ticdc/main.go:112\nruntime.main\n\truntime/proc.go:250"]
[2023/07/04 00:41:28.906 +00:00] [INFO] [cdc.go:81] ["run changeFeed task finished"] [result=null] [error="Exec command timeout"] [errorVerbose="Exec command timeout\ngithub.com/pingcap/test-infra/caselib/pkg/host.init\n\tgithub.com/pingcap/test-infra/caselib/pkg/host/host.go:16\nruntime.doInit\n\truntime/proc.go:6506\nruntime.doInit\n\truntime/proc.go:6483\nruntime.doInit\n\truntime/proc.go:6483\nruntime.main\n\truntime/proc.go:233\nruntime.goexit\n\truntime/asm_amd64.s:1598"]
[2023/07/04 00:41:28.906 +00:00] [PANIC] [main.go:113] ["run case failed"] [error="Exec command timeout"] [errorVerbose="Exec command timeout\ngithub.com/pingcap/test-infra/caselib/pkg/host.init\n\tgithub.com/pingcap/test-infra/caselib/pkg/host/host.go:16\nruntime.doInit\n\truntime/proc.go:6506\nruntime.doInit\n\truntime/proc.go:6483\nruntime.doInit\n\truntime/proc.go:6483\nruntime.main\n\truntime/proc.go:233\nruntime.goexit\n\truntime/asm_amd64.s:1598"] [case=cdc_scale_sync] [stack="main.main\n\ttest-infra/caselib/ticdc/main.go:113\nruntime.main\n\truntime/proc.go:250"]


Doesn't find cohttp.lwt even though it's installed :(

Apologies if I'm missing something silly. I'm learning Coq and wanted to play around with this some, but it seems to not be able to find cohttp.lwt even though it appears to be installed.
ricky@t520 99% /tmp/coq-chick-blog/extraction (master)$ opam install --jobs=4 lwt cohttp
[NOTE] Package lwt is already installed (current version is 2.4.5).
[NOTE] Package cohttp is already installed (current version is 0.11.2).

ricky@t520 99% /tmp/coq-chick-blog/extraction (master)$ make
ocamlbuild chickBlog.native -use-ocamlfind -package cohttp.lwt,lwt,lwt.unix,num,str
+ ocamlfind ocamldep -package cohttp.lwt,lwt,lwt.unix,num,str -modules chickBlog.ml > chickBlog.ml.depends
ocamlfind: Package `cohttp.lwt' not found
Command exited with code 2.
Compilation unsuccessful after building 1 target (0 cached) in 00:00:00.
Makefile:2: recipe for target 'build' failed
make: *** [build] Error 10

OK, this is weird. Can you give me the output of:
opam --version
opam list

I also just added a Dockerfile which should run on any Linux machine using Docker:
docker build --tag=chick-blog .
docker run -ti -p 8008:8008 chick-blog

OK, this is weird. Can you give me the output of:
opam --version
opam list

Sure.
ricky@t520 60% /tmp$ opam --version
1.1.1

ricky@t520 53% /tmp$ opam list
Installed packages for 4.01.0:
base-bigarray                 base  Bigarray library distributed with the OCaml compiler
base-threads                  base  Threads library distributed with the OCaml compiler
base-unix                     base  Unix library distributed with the OCaml compiler
camlimages                   4.1.1  Image processing library
camlp4                      4.01.0  Camlp4 is a system for writing extensible parsers for programming languages
camlp5                        6.11  Preprocessor-pretty-printer of OCaml
camlzip                       1.05  Provides easy access to compressed files in ZIP, GZIP and JAR format
camomile                     0.8.5  A comprehensive Unicode library
cohttp                      0.11.2  HTTP library for Lwt, Async and Mirage
conduit                      0.5.0  Network connection library for TCP and SSL
coq                         8.4pl4  Formal proof management system
coq:error-handlers             dev  Simple and robust error handling functions.
coq:function-ninjas            dev  Write readable code using simple functional combinators.
coq:list-plus                  dev  More functions on lists.
coq:list-string                dev  Strings implemented as lists.
coq:moment                     dev  Parse, manipulate and pretty-print dates in Coq.
dypgen                  20120619-1  Self-extensible parsers and lexers for OCaml
fieldslib                109.20.03  Syntax extension to define first class values representing record fields, to get and set record fields, iterate and fold over all fields of a record and create new record values
lwt                          2.4.5  A cooperative threads library for OCaml
ocamlfind                    1.5.1  A library manager for OCaml
omake                <IP_ADDRESS>-0.rc1  Build system designed for scalability and portability
ounit                        2.0.0  Unit testing framework loosely based on HUnit. It is similar to JUnit, and other XUnit testing frameworks
patoline                       0.1  A new typesetting system, programmable in ocaml.
re                           1.2.2  RE is a regular expression library for OCaml
sexplib                  111.17.00  Library for serializing OCaml values to and from S-expressions
sqlite3-ocaml                2.0.6  SQLite3 bindings
stringext                    0.0.1  Extra string functions for OCaml
type_conv                111.13.00  Library for building type-driven syntax extensions
uri                          1.7.1  RFC3986 URI parsing library

Ok, I have not tested the blog with OPAM 1.1.1 but this version seems deprecated (according to the "Your version of OPAM (1.1.1) is not recent enough to read ..." errors I get doing an opam init). This is unfortunate because most Linux distribs propose only the 1.1.1, so you should probably compile and try OPAM 1.2.0.

Sorry for the delay in response. I'll give OPAM 1.2.0 a shot, I didn't realize how out of date I was!


Changed file organization to work with flask, added a new input html

@amanmalali LGTM just a small doubt on what's input.html for?

Merging this so that I can push some of my changes on top of this


some native speaker kazakh improvement

Sorry for not single pull request, refinement has just come for the kazakh. As for the russian, the most relevant analogue for 'й' in actual use in words' context is "i".

Thanks!


WIP: Support for more @format types

See #144.
TL;DR: Added support for more built-in @format types, as well as the ability to define custom string validation formats via the config file.
A nice side-effect of this implementation is that custom format types are actually able to override the output of built-ins. For example, one can override the "email" format type like so:
// ts-to-zod.config.js
{
  // ...
  customJSDocFormats: {
    email<EMAIL_ADDRESS>  }
}

Then as a result, the generated Zod schema for properties with @format email will look like:
{
  /**
   * @format email
   */
  email<EMAIL_ADDRESS>}

What's left:

[X] Built-in support for format types date-time, ip, ipv4, & ipv6.
[X] Support for defining custom format types via the config file.
[ ] Updating the README with documentation for the new features.

Very cool, I can't wait to see this PR merge! Good job 👌

Codecov Report

Merging #145 (01d4fc6) into main (8d3b297) will decrease coverage by 0.40%.
The diff coverage is 94.94%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #145      +/-   ##
==========================================
- Coverage   97.33%   96.94%   -0.40%
==========================================
  Files          14       14
  Lines         676      719      +43
  Branches      275      296      +21
==========================================
+ Hits          658      697      +39
- Misses         18       22       +4

Impacted Files
Coverage Δ

src/core/generateZodSchema.ts
95.16% <66.66%> (-1.04%)
:arrow_down:

src/core/jsDocTags.ts
98.50% <97.72%> (-0.48%)
:arrow_down:

src/core/generate.ts
96.15% <100.00%> (+0.03%)
:arrow_up:

src/utils/resolveDefaultProperties.ts
95.45% <100.00%> (ø)


chore(metadata): add works-with metadata annotations re: rhsso and keycloak, using npm version notation (RHIDP-2680)

What does this PR do?
chore(metadata): add works-with metadata annotations re: rhsso and keycloak, using npm version notation
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
https://issues.redhat.com/browse/RHIDP-2680
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:

[x] The Eclipse Contributor Agreement is valid
[x] Code produced is complete
[x] Code builds without errors
[ ] Tests are covering the bugfix
[x] The repository devfile is up to date and works, if one exists
[x] Sections What issues does this PR fix or reference and How to test this PR completed
[x] Relevant user documentation updated
[x] Relevant contributing documentation updated
[x] CI/CD changes implemented, documented and communicated

Reviewers
Reviewers, please comment how you tested the PR when approving it.

/lgtm


High performance cross product

Fix a few data loading issues for the fraud workload.
Also tested on tpcx-ai workloads (Scale 30, 10GB data file)
scripts/cleanupNode.sh
scripts/startPseudoCluster.py 8 20000
bin/testDecisionForestWithCrossProduct Y 131564161 7 500000 7 32 model-inference/decisionTree/experiments/dataset/tpcxai_fraud_test.csv model-inference/decisionTree/experiments/models/tpcxai_fraud_randomforest_10_8_netsdb RandomForest
bin/testDecisionForestWithCrossProduct N 131564161 7 500000 7 32 model-inference/decisionTree/experiments/dataset/tpcxai_fraud_test.csv model-inference/decisionTree/experiments/models/tpcxai_fraud_randomforest_10_8_netsdb RandomForest 10
Output:
We have finished scanning this set
total count:131564161
positive count:649721
****Model Inference Time Duration: ****59.9618 secs.
Second execution:
****Model Inference Time Duration: ****56.1583 secs.

Can you share the command you ran?

@venkate5hgunda  Could you please quickly verify this pull request? Thank you!


Remove unnecessary SafeMath lib from bonsai/examples/governance

Removes unnecessary SafeMath openzeppelin lib since solidity version is >0.8.0 it has over/underflow protection by default, saving gas.
Forge tests passing.

@nategraf @flaub Any interest in this/approving workflow runs? Minimal changes. Otherwise I will close.

CI currently cannot run on branches outside the risc0 org. We are updating the CI now to allow this, and this PR will be merged when that work is complete.


The license

Why not re-license the plugin with some mainstream open-source license? WTFPL is not considered serious by many people (and distros). I guess it would be a nice idea to use e.g. GPL2 or Apache License.

MIT is probably more apt (if a choice were to be made (i personally find the WTFPL kind of face-palmy, but that's just my own tastes and i don't expect others to share them)) for the spirit intended by WTFPL. it's small and simple to understand and unfettered. GPL and Apache contain a lot of very nuanced legal language with edge cases and honestly who's going to act on any of that withouta big stack of cash to pay a lawyer to know what it actually means in any litigious context (not that i don't think either of those licenses are never appropriate, many projects can make cases for their complexity where it is necessary for their goals, but come on, this is a vim plugin!)
imo WTFPL is fine but it does make me cringe a little :P

I like MIT. Made the change here: 4896868f2308781565118a72f47ebfbe2d20c983


docs(javascript): fix code rendering on GitHub

An extra space is needed for the ~code~ markup to be correctly rendered on GitHub.

I must say those two links pointing to pages you must register an account to view is simply unreasonable for open source projects. Especially the latter (https://doomemacs.org/d/how2commit) which simply redirects to a GitHub Gist (why not just put the link to the gist?)

I must say those two links pointing to pages you must register an account to view is simply unreasonable for open source projects.

They already are: https://gist.github.com/hlissner/4d78e396acb897d9b2d8be07a103a854
And I understand it's unacceptable, but it's simply not finished. The links where published right before I took a break from Doom for health reasons two months ago (I only returned a couple days ago). I'll get around to fixing those links soon, but for the time being, please create an account. If your change is too small to warrant the effort, then wait until the discourse et others are finished.

Anyhow, modifications to our docs is in our do not pr list, so I must turn down this PR.

No hard feelings! The project is awesome, I'm just baffled by the login requirements.


TypeError: {class} object is not iterable

This is a common error that users stumble over, and the error message is not intuitive.  I suggest two ways to help the user intuit what the actual issue is:

Add an explicit type check for iterable
Change the error meassage to something like TypeError: {property} requires one or more values packed in an iterable.  This more clearly indicates to the user that the problem is related to the cardinality of the property, rather than an issue with the child object's class type

See https://github.com/SD2E/opil/issues/160

The attribute name of the property (allowed_time in SD2E/opil#160) is not available when the error message would get generated, only the URI of the property. Here is an example of your requested error message using Component.types:
>>> import sbol3
RDFLib Version: 5.0.0
>>> c = sbol3.Component('http://example.org/sbol3/c1', sbol3.SBO_DNA)
>>> c.types = sbol3.SBO_PROTEIN
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/tmitchel/Projects/sd2/pySBOL3/sbol3/object.py", line 25, in __setattr__
    self.__dict__[name].set(value)
  File "/Users/tmitchel/Projects/sd2/pySBOL3/sbol3/property_base.py", line 160, in set
    raise TypeError(msg)
TypeError: Property http://sbols.org/v3#type requires one or more values packed in an iterable

Is that what you're looking for?

I think that is still preferable to the current error message.  However, it is in fact possible to get the actual Python attribute name (though not necessarily concise):
import sbol3

sbol3.set_homespace('http://nope.nope/')
c = sbol3.Component('c', sbol3.SBO_DNA)

# Get Property object just for demonstration purposes -- from scope of an actual
# Property object you'd want to refer to `self`
self_property = c.__dict__['features']

# Demonstrate how you'd retrieve the property name from within the Property scope
self_name = None
for name, property in self_property.property_owner.__dict__.items():
    if property is self_property:
        self_name = name
print(f'Type Error: Something wrong with values in {self_name} property')

Updated draft error message:
>>> import sbol3
RDFLib Version: 5.0.0
>>> c = sbol3.Component('http://example.org/sbol3/c1', sbol3.SBO_DNA)
>>> c.types = sbol3.SBO_PROTEIN
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/tmitchel/Projects/sd2/pySBOL3/sbol3/object.py", line 25, in __setattr__
    self.__dict__[name].set(value)
  File "/Users/tmitchel/Projects/sd2/pySBOL3/sbol3/property_base.py", line 177, in set
    raise TypeError(msg)
TypeError: types requires one or more values packed in an iterable


Better source map support

Right now sourcemaps are placed by default alongside the lambda code but for people using things like Sentry they will want to upload those seperately (and may not even want it in the bundle uploaded to AWS).
Requests

https://serverless-stack.slack.com/archives/C01JG3B20RY/p1629128753065500?thread_ts=1629077794.053600&cid=C01JG3B20RY

Adrian also mentioned in Slack

I’ve been using AWS Console to compare my old/new resources, and all looks fine, the only thing I noticed is my Lambdas size are 2x now, from 2MB on SLS to 4MB on SST.
I believe might the due the .map file is included in the bundle now, and before with SLS might not be there.
Is there a real need to include .map file? I’m not familiarized how to take advantage of it.

Fixed this with #843


initial black screen on android

On android 5, we get a black screen after the splash screen. maybe delay it until the view is fully rendered ?

I have it too, thanks for letting me know.

I did not have it yet, but maybe you have to add :
cordova plugin add cordova-plugin-whitelist

Closing cannot reproduce anymore


Update README.md

Minor change, removed repeated sentence subject.

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


Update files and packages

Summary of the Pull Request
What is this about?
I have no idea but I've updates NuGet packages and other things on my computer and now I want to update those to the repo.
PR Checklist

[ ] Applies to #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

Sorry, wrong fork.


Any Firefox Version?

Any update? Firefox also uses Web Extensions so @harshjv probably just need to publish it to firefox Add-ons store.


Add capability to hide keyboard

The problem : Feature request
Add capability to hide keyboard after sendkeys or sendvalue
Details
currently after using sendkeys, I have manually hide the keyboard. Doing this for every textbox is cumbersome.
$.sendKeys();
if(((AndroidDriver )(WebDriverRunner.getDriver)).isKeyboardShown()){
 // hideKeyboard call
} else if ios{
//getIosDriver and hide keyboard
}

I think of 2 ways to implement this.

add command called hideKeyboard

$.setValue().hideKeyboard();
$.sendKeys(); $.hidekeyboard(); //since sendkeys is void we cannot chain it

Implicitly add hideKeyboard inside the setvalue method. (Very simple to implement but cannot be used after sendkeys

private void hideKeyboardIfPresent() {
    if (isAndroidDriver() && AppiumDriverRunner.getAndroidDriver()
      .isKeyboardShown()) {
      AppiumDriverRunner.getAndroidDriver()
        .hideKeyboard();
    } else if (isIosDriver() && AppiumDriverRunner.getAndroidDriver()
      .isKeyboardShown()) {
      AppiumDriverRunner.getIosDriver()
        .hideKeyboard();
    }
  }

Tell us about your environment

Selenide Version:  6.10.0
Selenide-Appium Version:  2.3.0
IOS\Android\WEB version:  NA

Code To Reproduce Issue [ Good To Have ]
Not applicable

@BorisOsipov - Could you please suggest which approach we can follow? I can create. a PR for this.

@asolntsev - Any hint on this one. I can create a PR if I get some hints

@amuthansakthivel Isn't it already solved by https://github.com/selenide/selenide-appium/pull/131 ?

Yes, ideally, we should make it chainable.
But at the moment, it seems requires too big refactoring in Selenide.
Another option - probably much better! - is the following:

AppiumSelenideElement should NOT extend SelenideElement.
It will contain only mobile-specific methods.
Drawback: it makes harder combining web- and mobile tests in the same test class.

We can park it for a later time then.
If we can create a method getSelenideAppiumElement in SelenideElement class, then
$(element).getSelenideAppiumElement().callMobileSpecificMethods()
Drawback you mentioned is an antipattern. People should not combine web and mobile in one tests.

@amuthansakthivel  Why it's antipattern? Never heard that.
If I want to test a hybrid mobile app, it seems reasonable to make these steps in a single test:

open a mobile app,
switch to a webview,
login in webview,
and switch back to mobile app.

Drawback you mentioned is an antipattern. People should not combine web and mobile in one tests.

@asolntsev - That makes sense. What about the below option?
If we can create a method getSelenideAppiumElement in SelenideElement class, then
$(element).getSelenideAppiumElement().callMobileSpecificMethods()`

@amuthansakthivel I don't understand how it's possible.
Project selenide-appium depends on selenide, but project selenide knows nothing about selenide-appium.
That's why class SelenideElement cannot have method returning SelenideAppiumElement.
But what is possible - to override ALL methods of SelenideElement (applicable for mobile apps) in SelenideAppiumElement. So that all these methods would return SelenideAppiumElement instead of SelenideElement.

@asolntsev - yes you are correct. My bad. I think we should then override applicable methods

Please close this issue @asolntsev


Webpack entry cannot be found after importing filer

Context
I am trying to add FilerWebpackPlugin to an existing Webpack configuration.
Problem
As soon as I require('filer'), Webpack starts failing because it cannot find the entry file anymore.
Analysis
This problem seems to be caused by a side effect in src/path.js:
/**
 * Patch process to add process.cwd(), always giving the root dir.
 * NOTE: this line needs to happen *before* we require in `path`.
 */
process.cwd = () => '/';

Workaround
The only workaround that I found is to save process.cwd before importing filer, and to restore it afterwards.
const cwd = process.cwd;
const filer = require('filer');
process.cwd = cwd;

With this fix, Webpack is able to find the entry file again.

cc @bcheidemann, who might have thoughts.

Thanks for raising this issue @thomas-jakemeyn.
The problem seems to be that we import the whole of filer when we only need the FilerWebpackPlugin class. In hindsight, this is obviously not the best idea! I will put up a PR to deprecate accessing the plugin through the "normal" filer import and add a specific import path for the plugin. @humphd what do you think about:
const { FilerWebackPlugin } = require('filer/webpack');

@thomas-jakemeyn Until the fix is released, you should be able to import the class directly:
// Direct import
const { FilerWebpackPlugin } = require('filer/src/webpack-plugin');

Let me know if this works for you! :)

Hello @bcheidemann,
I confirm that using a direct import as you suggested works. Thank you for that!
However, don't you think that the real problem is the side-effect on process.cwd?

Hello @bcheidemann,
I confirm that using a direct import as you suggested works. Thank you for that! However, don't you think that the real problem is the side-effect on process.cwd?

@thomas-jakemeyn The process.cwd side effect is, as I understand it, required for filer to work properly in the browser. Maybe @humphd can clarify the exact purpose? Regardless, the Webpack config only needs to load the FilerWebpackPlugin class which does not import any part of the main filer library (nor should it). As such, if imported correctly, there should be no side-effects.
Are you having trouble with the process.cwd side effect other than in the context of the Webpack config (i.e. at runtime)?

The problem that I was personally facing is solved with your suggestion.

re: the process.cwd override, that can probably be removed.  It's used very rarely, and likely could be dealt with through documentation alone.  Unlike fs running in the OS, we have no concept of a process on which the current working directory can be managed.  We could probably move this to the shell instead.  I'd take a PR that did this, but am unlikely to have time to do it any time soon.

Thanks for the clarification @humphd.
@thomas-jakemeyn do you anticipate other issues that wouldn't be solved by PR #793? Happy to rethink our approach if there's reason to do so :)

Since filer is intended to be used in a browser, I guess that it should be fine indeed. Thank you for the clarification and for the quick reaction!

Landed #793, I'll ship v1.4.0 in a min.

https://github.com/filerjs/filer/releases/tag/v1.4.0
https://www.npmjs.com/package/filer is now at v1.4.0

https://github.com/filerjs/filer/releases/tag/v1.4.0
https://www.npmjs.com/package/filer is now at v1.4.0

Thanks!

@humphd I think this issue needs to be reopened. I just checked the latest NPM release (v1.4.0) and the new import is not working. This is because I added a root folder called "webpack" which wasn't released to NPM. Could you advise on what I need to change so that this is included in the release to NPM? This is also the case for the "shims" directory which is the reason for issue #791

Should be fixed by v1.4.1

Should be fixed by v1.4.1

Confirmed, the correct files have been pushed to NPM


[SPARK-18608][ML] Fix double caching

What changes were proposed in this pull request?
df.rdd.getStorageLevel => df.storageLevel
using cmd find . -name '*.scala' | xargs -i bash -c 'egrep -in "\.rdd\.getStorageLevel" {} && echo {}' to make sure all algs involved in this issue are fixed.
Previous discussion in other PRs: https://github.com/apache/spark/pull/19107, https://github.com/apache/spark/pull/17014
How was this patch tested?
existing tests

Test build #81654 has started for PR 19197 at commit b485614.

Test build #81654 has finished for PR 19197 at commit b485614.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81654/
Test PASSed.

ping @jkbradley  @WeichenXu123

LGTM
Merging with master and branch-2.2
Thanks!

having trouble merging...will try again soon


[K32W] Add CI for NXP K32W

[K32W] Integrate NXP Elock in the CI Tool
This addresses https://github.com/project-chip/connectedhomeip/issues/4625
I am not a Docker expert, so please let me know how I can test that the Docker image is running correctly, Basically I just duplicated a Docker file from another vendor and added the required commands for setting up NXP SDK but I didn't know how to test that it works.

you may be able to test the image by building it locally (build.sh should do this) and then running it via docker run -it connectedhomeip/chip-build-k32w

you may be able to test the image by building it locally (build.sh should do this) and then running it via docker run -it connectedhomeip/chip-build-k32w

Ok, followed the steps but I got the below error.
doru@ubuntu:~/Desktop/public_repos/connectedhomeip/integrations/docker/images/chip-build-k32w$ docker image ls
REPOSITORY                        TAG                 IMAGE ID            CREATED             SIZE
connectedhomeip/chip-build-k32w   0.4.21              99682cd3b0f5        3 days ago          3.19GB
connectedhomeip/chip-build-k32w   0.4.19              fccbf0d0c46b        4 weeks ago         5.26GB
connectedhomeip/chip-build        0.4.19              ac8f0043d2d6        4 weeks ago         2.34GB
connectedhomeip/chip-build        0.4.21              ac8f0043d2d6        4 weeks ago         2.34GB
ubuntu                            focal               f63181f19b2f        7 weeks ago         72.9MB
doru@ubuntu:~/Desktop/public_repos/connectedhomeip/integrations/docker/images/chip-build-k32w$ docker run -it connectedhomeip/chip-build-k32w
Unable to find image 'connectedhomeip/chip-build-k32w:latest' locally
docker: Error response from daemon: pull access denied for connectedhomeip/chip-build-k32w, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.
See 'docker run --help'.

@doru91 You don't have permissions to push images to the connectedhomeip project on the docker hub. You can run your local image by specifying a proper tag, i.e. docker run -it --rm connectedhomeip/chip-build-k32w:0.4.21

@doru91 You don't have permissions to push images to the connectedhomeip project on the docker hub. You can run your local image by specifying a proper tag, i.e. docker run -it --rm connectedhomeip/chip-build-k32w:0.4.21

Thanks, @Damian-Nordic . I was able to shell-connect to the Docker container.


Log error wehn events queue if full

Coverage remained the same at 87.854% when pulling 29f2938328cca55a1288181e852db8e5c303d44e on log_error_when_queue_is_full into 0b5123aafa6d799e1e1b891439a063fa0bc01205 on develop.


Build Standalone Binaries for Multiple Architectures

Work in https://github.com/agardnerIT/tracepusher/issues/37 has proven the feasibility of providing standalone binaries for logpusher.
Use PyInstaller to create single file binaries for each of the main OSs:

Windows
Linux AMD64
Linux ARM (includes Darwin / MacOS)

Working on it here: https://github.com/agardnerIT/logpusher/tree/standalone-binaries/dist


GUI Link Broken?

https://kinesiscorporation.github.io/Adv360-Pro-GUI

Is the GUI version link broken?

That link seems to work for me, what issue are you encountering?

The error message was related to Unsecure Connections, which I think might be related by the Chrome SSL certifications.

Which version of chrome and which operating system? I am unable to replicate this issue with several versions of chrome on android and linux

Same here
Chrome 114.0.5735.199 on Windows 11

ERR_SSL_PROTOCOL_ERROR

Version 114.0.5735.199 (Official Build) (64-bit)

I had the same issue. turns out the URL is blocked by my firewall. if you visit the HTTP version, you'd know if it is being intercepted or not.

the issue is not the SSL certificate. for some reason, the firewall doesn't allow access to the URL and tries to intercept the connection. But since it can't/won't impersonate the destination, we end up getting SSL errors.

when I access the url on http so that the interception works, this is what I get

the question is why is it being blacklisted. the solution is changing your network or tunnelling your traffic through a node outside your network, or just disable any firewall in the middle temporarily.

That indicates it's a firewall on your ASUS router that's causing the problem, not anything on the remote side. Have you tried accessing the site using 4g on your phone?

I already sorted it out. I'm just explaining why some people might experience this problem. Asus is outsourcing their blacklist to TrendMicro, https://global.sitesafety.trendmicro.com/result.php and if they have automatically blacklisted the page, the chances are that other firewalls do it too. the firewalls try to intercept your connection, and then the browser complains about the certificate.

Ok, thanks for the explanation. I'm going to close the issue as solved l


Update eval_squad to use API of latest optimum (#17918)

Update eval_squad with latest optimum.
Tested with:

optimum 1.13.1
transformers 4.31.0
onnxruntime-gpu 1.16.0
onnx 1.14.1
datasets 2.14.5
evaluate 0.4.0
torch version 2.2.0.dev20230920+cu121

Example output in A100:
{'exact': 86.66035950804162, 'f1': 92.99622739711005, 'total': 10570, 'HasAns_exact': 86.66035950804162, 'HasAns_f1': 92.99622739711005, 'HasAns_total': 10570, 'best_exact': 86.66035950804162, 'best_exact_thresh': 0.9998456239700317, 'best_f1': 92.9962273971104, 'best_f1_thresh': 0.9998456239700317, 'total_time_in_seconds': 84.74025378189981, 'samples_per_second':<PHONE_NUMBER>8731417, 'latency_in_seconds': 0.008017053337928081, 'provider': 'CUDAExecutionProvider', 'disable_fused_attention': False, 'pretrained_model_name':
'bert-large-uncased-whole-word-masking-finetuned-squad', 'onnx_path': './bert-large-uncased-whole-word-masking-finetuned-squad/optimized_model.onnx', 'batch_size': 1, 'sequence_length': 384, 'use_io_binding': True}
Description

Cherry pick from Microsoft/Onnxruntime for upstream changes to eval_squad.py
Motivation and Context

Without this backport we're broken for migx_onnxrt_bert_distilled_benchmarks due to missing pieces in the script. Testing was originally done upstream and it appears this change hasn't been backported.

This PR has been working upstream and we usually catch this in our remote syncs but it does look like this may have been missed. For the sake of normalization I think it's good to merge this and spot check for other related missed bits.

I have checked it and it seems to work properly now for eval_squad.py.


PhusionPassenger killing ruby process in idle time.

Hi,
Integrating rufus scheduler in Ruby on Rails Application
rufus-scheduler 3.1.10
ruby 1.9.3
Phusion Passenger 5.0.1
Apache 2.4
scheduler = Rufus::Scheduler.new
scheduler.cron '00 16 * * *' do
Model.some_method
end
scheduler.cron '00 17 * * *' do
Model.some_method
end
def self.some_method
end
In development its working fine.
Coming to production its not working
Some of them saying like ruby process killed by PhusionPassenger
I have tried the below configurations like
PassengerSpawnMethod conservative
(or)
PassengerSpawnMethod smart
(or)
RailsAppSpawnerIdleTime 0
PassengerMinInstances 1
Can you please help me out of this problem.

Hello,
have you read https://www.phusionpassenger.com/library/indepth/ruby/spawn_methods/ ?
Closing since it's not a rufus-scheduler issue.

Hi,
I already read this https://www.phusionpassenger.com/library/indepth/ruby/spawn_methods/ ?, I know the issue from phusion passenger. still i thought if any one has got same issue. I'm waiting for a solution.
Once i get a solution. lets close it out.
Thanks in advance.

Why don't you ask to the Phusion Passenger mailing list? It's not an issue caused by rufus-scheduler.

https://groups.google.com/forum/#!forum/phusion-passenger

Passenger + Apache configuration reference: https://www.phusionpassenger.com/library/config/apache/reference/
RailsAppSpawnerIdleTime is no longer available since Passenger 4.x https://www.phusionpassenger.com/library/config/apache/reference/#railsappspawneridletime
They say you should be using smart spawning, you probably have to use the spawn hooks described in https://www.phusionpassenger.com/library/indepth/ruby/spawn_methods/
Read carefully https://www.phusionpassenger.com/library/config/apache/reference/#passengermininstances and
https://www.phusionpassenger.com/library/config/apache/reference/#passengerprestart .
You probably have to combine PassengerMinInstances, PassengerPreStart and a spawn hook to get it to work.
Good luck!


Added public initializer to ResponseParser

Codecov Report

Merging #108 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
+ Coverage   93.66%   93.69%   +0.02%
==========================================
  Files          14       14
  Lines         221      222       +1
==========================================
+ Hits          207      208       +1
  Misses         14       14

Impacted Files
Coverage Δ

Jayme/HTTPResponseParser.swift
100% <100%> (ø)
: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 0bb9477...b25b904. Read the comment docs.


reverse #212 since it does not address the real problem

see remarks by @plietar on #212

It looks like Ctrl-C does not work when the session is invalid (connection reset)


Apps with lots of findings slow down processing

Running against a large number of apps, noticed varying run times and difficulties processing an app with 38,000+ flaws. Is there a way we can filter to focus on a smaller number of flaws, like only those affecting policy?

Updated script to do only policy violating flaws, and increased the default findings page size so that we can make fewer API calls overall.


Workaround for an issue in the quilkin proxy

The proxy makes dns lookups that slows down proxying.
This patch makes dns lookups in the manager instead, sending ips to the proxy instance so it does not have to do lookups.

What type of PR is this?

Uncomment only one  /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:
/kind breaking
/kind bug
/kind cleanup
/kind documentation
/kind feature
/kind hotfix

What this PR does / Why we need it:
Which issue(s) this PR fixes:

Closes #
Special notes for your reviewer:

Unintentional opening of PR, don't think the fix should be there. Sorry :-D


login failed for display 0

When using the account from when I created the VM (the one I use to connect to via SSH) I receive something like:
Sesman connected OK
Login failed for display 0
If I create a new user in the VM:s terminal sudo adduser user2, this new user is allowed to connect to the display.
Why is that?

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

ID: 077ee673-528a-d893-5832-7656b91dbec8
Version Independent ID: 2180e845-1ead-3b47-03be-c26aee00c6f7
Content: Use Remote Desktop to a Linux VM in Azure
Content Source: articles/virtual-machines/linux/use-remote-desktop.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

Why is that?

@cynthn Please add your comments

@karlanka Please make sure you are using the correct username and password

Password and username is correct. Scren dump from the error:

I'm having the same problem, with the new user working and everything. Changed the password on my original user a couple of times just to be dead certain I could type it correctly, same result each time.

same problem - with the correct username and password


Fix event / e. Fixes #7

cc @bsudekum for review & publish

@tmcw looks good, tested in IE, firefox and safari.


Added /license endpoint

Fixes

Fixes #368  by @aldenstpage
Description
Created /license endpoint in the api to list all licenses from rdf file
Screenshots

Checklist

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

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

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

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

Developer's Certificate of Origin 1.1

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

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

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

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

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

Take a look at the build and see what commands are failing. You can also run the tests locally and see which step is failing

@aldenstpage I have made changes so now it passes style test. But i am unable to understand why it is failing main test .The build says that Pipfile.lock  out of date . Can u help me resolve this?

It looks like it's because you added rdflib as a dependency but didn't commit the changes made to Pipfile.lock. If you push that file, that particular issue should be solved. If not, try pipenv lock and recommit.

@aldenstpage I commited Pipfile.lock but now it shows conflicts and the tests wont run.

Try reverting your changes to Pipfile  and Pipfile.lock entirely, running pipenv install rdflib again, and then committing both files to this PR. That seems like the root of the problem with the tests.
As for the merge conflicts, the PR is just slightly out of date with the other changes in the project; you can use the "resolve conflicts" UI to bring it up to date. I would put that off until after the tests have been fixed.

Now I think it is passing the part of test it was failing earlier .But i am not able to find out why the test is timing out now.

This repo has entered maintenance mode so we will not be merging this PR.


Parent directory button

I would like to be able to go to the parent directory with a simple button next to the prev/next buttons.
This isn't a duplicate of #72 and I actually don't think it's very relevant, however I hope that my feature request would be tagged as medium or higher priority.
Thanks

This is definitely something we can try to do for the release.


Timeouts with large number of reportsuites

See this twitter chain
Large numbers of reportsuites can create timeout issues. I suspect this is linked to limits on the Adobe side of request sizes. Events in particular create massive requests even for relatively small report suite sizes.
The solution is to split requests for information on evars, props, and events, perhaps into groups of 10-20? I'll do some math on the maximum possible size per report_suite and go from there.

Let's look at the maximum size of an events response for a single reportsuite. Events responses will be bigger than anything else because of the 1000 custom events, plus 13 default events which Adobe preconfigures. That leaves us with 1013 events per response. The longest IDs are going to be with the preconfigured events. We can go ahead and calculate the typical/max size of the default events by using JSON.stringify(defaultEvents).length. I get a result of 2701, but this could vary a little. To be safe I'll default to 2800 bytes to transmit the default events.
Now all we need to worry about is the size of the remaining 1000 custom events. I don't know if a single production reportsuite in the world has all these configured, but it's worth examining the maximum possible amount.
{
"id":"event1",
"name":"Hostile takeover",
"description":"Fire whenever the Illuminati take over a country (despite not existing for several hundred years).",
"type":"enabled",
"default_metric":false,
"participation":"disabled",
"serialization":"always_record",
"polarity":"positive",
"visibility":"everywhere"
}

Let's break this down into maximum lengths. First, we can figure out the length from the IDs, as they will be static.
var entries = "";
for (var i=1; i<1001; i++){entries = entries + '"id":"event' + i +'",';}
console.log(entries.length);
15893

15893 bytes for IDs.
The other elements are a bit harder to nail down.
"name":limit is 128 chars, so with '"name":"content",' we are looking at 11+128 or 139 bytes max.
"description":256 chars limit. This means 273 bytes max.
"type":"currency_no_subrelations" is the longest at 24 chars. This means a total of 34 bytes max.
"default_metric" is either true or false, so the length is either 22 bytes or 23 bytes.
"participation":" either enabled or disabled, so length is either 26 or 27 bytes.
"serialization": longest option is "record_once_per_unique_id", which results in 44 bytes.
"polarity": positive or negative, but both are the same at 22 bytes.
"visibility":"everywhere" - the API options here aren't well documented. Everywhere is the default and is 25 bytes.

We also get 4 bytes per object for the object initialization {} and also for their newlines.
In total, we get 139 + 273 + 34 + 23 + 27 + 44 + 22 + 25 + 4 = 591 591 bytes maximum per event entry. This doesn't include the ID values we calculated earlier, but we'll add them back in.
OK, so 591 * 1000 = 591000 + 15893 (ids) = 606893 (for custom events).
Then adding in the default events, we get 606893 + 2800 = 609693.
Per reportsuite we have a maximum of 606693 characters (I've been using bytes and characters interchangeably, but this is only true if ASCII characters are exclusively used. This may not be true for international reportsuites, where UTF-8 is used and will use up additional space).
There are a few more characters which are included for each reportsuite.
{
"rsid":"illuminati2014global",
"site_title":"Bavarian Enlightenment",
"events":[all 1000 events go here]
}

I could go through the max size for these too, but I'm tired of this post now. Let's assume 100 chars total for our purposes.
Maximum size per reportsuite is thus 609,703 bytes, or 609 KB. This is massive. With only 2 reportsuites an event API response could be 1.2 megabytes. 100 reportsuites could result in API results of over 60MB!
Typical lengths will be under this because descriptions for non-configured events are much smaller, but 300KB per reportsuite is still typical.

Moving to getSettings, which will allow for getting all the details on a reportsuite at a time. This should fix this issue.


Fix protocol to avoid CORS errors

Description

What was changed in this pull request?

Update the protocol from http to https for fetching OSM tiles.

Why is it necessary?

With the http protocol Chrome is throwing CORS errors
Checklist

~[ ] Unit tests added or updated~
~[ ] Documentation added or updated~
~[ ] Example added or updated~
~[ ] Screenshot for visual changes (e.g. new tracks)~
[x] Updated CHANGELOG.md

Since I branched of develop merging this into hotfix-1.5.8 would add another 28 commits to the hotfix. Essentially all the changes since v1.5.7 that were merged into develop I assume. Any objections against just merging it into develop and pushing it out with v1.6? I don't think this fix is critical. Or is there another way to cleanly merge it into hotfix-1.5.8?


[stable/spinnaker] Add dwardu89 to OWNERS

Is this a request for help?:
I'd like to be added as an Owner. I am currently working at ClearScore and we decided to use spinnaker as our cicd solution and believe in building repeatable solutions.
In order to automate spinnaker, its already hard on its own because of every command, and I want to push towards making it as customisable as possible and contribute it back instead of having our own fork.
@linki and @viglesiasce are currently the owners of this.

@dwardu89 I'm actually not an owner of that chart. Adding @lwander here as well.


Remember state on precognition form

Hello!
I am currently rewriting every useForm() instance to the precognition variant (import { useForm } from 'laravel-precognition-vue-inertia';). I want to "remember" the state of the form after submission, so the form is still "in tact" when going back to the form.
I used to do this with: const form = useForm(`CreateQuote:${usePage().props.user.id}`, {, but this does not work anymore. Does someone have a workaround to still accomplish this or is this not a feature (yet)? Thanks!

Hey @KickSchutte67,
We don't currently support passing a remember key to the underlying Inertia form, but I'll have a chat with @timacdonald about it and see what we can do.

I think this is pretty important, because it feels a little weird to lose functionality when using precognition

We can certainly support this. I supposed Precognition will have to remember some addition things also. Need to dig into the Inertia functionality to see how we can replicate it.

Hey folks, as this isn't specifically an issue - just a feature request we could (should) add, I'm going to close it.
It is on my list to add this feature and would be open to community PRs as well

Was this implemented? I'm using the latest version and the forms don't remember and I can't do something like useRemember(useForm(...)) because it fails. Is there any workaround? Thanks!

It has not been implemented yet.

Thanks! @timacdonald Will it be implemented? Asking because this ticket is marked as closed and I don't see it tracked anywhere else. In case it will not, do you have any workaround we can apply?

I'd love to implement it but we don't have any concrete plans right now.
I'm not sure of any workarounds at the moment.

Was this implemented? I'm using the latest version and the forms don't remember and I can't do something like useRemember(useForm(...)) because it fails. Is there any workaround? Thanks!

Did you find a easy workaround pls ?


.NET 6.0 support

Hi, firstly thanks for the packages, I've been using LocalStorage and SessionStorage for a while now.
Most of our sites are on .NET 6.0 LTS running v2.0.11 of your packages. I've just noticed v7.0.x is now out but only supports .NET 7. Will you be continuing to updat 2.0.x with any fixes?

Hi @daver77 - thanks for your kind words! Are you using WebAssembly or the Server-side bits?
II had intentions of supporting multitargeting, but hit a snag and stopped. I plan on adding that though, and for .NET 6 specifically, that is rather straightforward. Thanks for the idea!
@allcontributors please add @daver77 for ideas

We multi-target now, supporting .NET 7.0 and .NET 8.0. closing this....


add matcher for save firewall

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@lmunro

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.


Visual Studio Code Screen Width

Issue Type: Feature Request
Hello, the feature that I would like to see on the Visual Studio Code  Editor is the availability to make the editor as small as possible when having 3 windows on the screen.
Because I can only make the screen go as small as
6 INCHES in length.
And I want the editor to go as small as 3 INCHES in length like a Youtube video.  So I can have 3 windows on my laptop without having to have windows screen ontop of each other and compromise view.
Thanks.
VS Code version: Code 1.40.1 (8795a9889db74563ddd43eb0a897a2384129a619, 2019-11-13T16:49:35.976Z)
OS version: Windows_NT x64 6.2.9200

/duplicate of #84134


Fixed documentation link

Overview
Documentation links were broken due to missing file extensions
Screenshots (if applicable)
Documentation

[x] Updated Documentation links

Fixed the Overlay one with https://github.com/segmentio/evergreen/pull/1336. Cannot repo the other links.
Thanks for the good catch!


reincorporate project template for changelog updates

reincorporate project template so that changelogs are automatically built in the future for releases

https://github.com/ApeWorX/project-template/pull/6


build(dependencies): pin dependencies and enable dependabot

[x] Did you format your code correctly? NA
[x] Did you write tests for your change? NA
[x] Did you run all the tests?
[x] Relates to the issue? Yes, closes https://github.com/xFAANG/askql/issues/104

Test Suites: 411 passed, 411 total
Tests:       10 todo, 782 passed, 792 total
Snapshots:   0 total
Time:        9.835s
Ran all test suites.

Additional info:

It proved this is possible to enable dependebot via adding yaml config instead of UI settings. Feel free to amend settings to your needs (I set weekly mode, adding  label enhancement and defaults)
I left the PR name as a merge of both commits but I would recommend not to squash and use all commits as they are separated already. Up to you!
Please keep in mind this does not disallow to add non-strict versions.

@kgajowy I opted in for force squash for this project to enforce one PR per feature (commit) and to make it easier for git beginners who sometimes forget to amend. Thanks for this suggestion and feel invited to talk about this further.


add or remove scripts

remove a file is easy, adding one a little bit more complicated, we have to inject it via the content script i guess...


Feature/on delete

@antoineB Thanks for the MR. I have two general remarks:
Would be nice if you would not force push changes that are requested in a review. It makes it easier to review. We can always squash commits before merging. It would also be nice if you could add a small query that is failing or not yet supported and also provide the link to the SQL Dialect documentation. This makes it a lot easier for us.
However, we should definitely think about some contribution rules