Use of package in pydantic/init.py fails if imported from zip

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
If pydantic folder is placed in zip, then when doing for example 'from pydantic import Field' and __init__.py is run to resolve to 'pydantic.fields' this fails because __package__ is not set. As __package__ is discouraged in docs:
https://docs.python.org/3/reference/import.html, it would be nice if __init__.py used '__spec__.parent' instead as this then works in zip files too. At least do:
if not __package__:
   __package__ = __spec__.parent
at some point before using __package__
Example Code
No response
Python, Pydantic & OS Version
Tested in 2.5.3 -> 2.7.0
            pydantic version: 2.7.0
        pydantic-core version: 2.18.1
          pydantic-core build: profile=release pgo=true
                 install path: C:\Python311\Lib\site-packages\pydantic
               python version: 3.11.7 (tags/v3.11.7:fa7a6f2, Dec  4 2023, 19:24:49) [MSC v.1937 64 bit (AMD64)]
                     platform: Windows-10-10.0.22631-SP0
             related packages: typing_extensions-4.11.0
                       commit: unknown

I'm a bit lost, please can you format your issue using markdown so I can read the code.

Hi,
I am unsure of what you want me todo. The issue was really only about following recommendation in the python docs and not use the identifier package in your code.
We run Python embedded and prepackage all modules needed in a zip-file. We can not currently put pydantic in such a zip-file.
Regards,
Johan
Mvh
Johan Andersson

Från: Samuel Colvin @.>
Skickat: Friday, April 12, 2024 2:57:20 PM
Till: pydantic/pydantic @.>
Kopia: Johan Andersson @.>; Author @.>
Ämne: Re: [pydantic/pydantic] Use of package in pydantic/init.py fails if imported from zip (Issue #9221)
I'm a bit lost, please can you format your issue using markdown so I can read the code.
—
Reply to this email directly, view it on GitHubhttps://github.com/pydantic/pydantic/issues/9221#issuecomment-2051712271, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDUU755IPP5S4DR3HFDF53DY47K3BAVCNFSM6AAAAABGDW6DK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJRG4YTEMRXGE.
You are receiving this because you authored the thread.Message ID: @.***>

Thanks @johanandersson-tietoevrycom for reporting this issue.

As package is discouraged in docs: https://docs.python.org/3/reference/import.html,

Agree, it is not recommended to use __package__. Thanks for your suggestion.
Could you please help us to reproduce the problem? We need some steps to reproduce the problem then we can confirm the fix works.

Hi,
I can reproduce the error under Windows with Python3.9. Essentially by putting the pydantic code in a zip-file and use that instead of the ‘pydantic’ folder under site-packages. The real context for me is when we use Python embedded into a C++ executable, and all support libraries we use go into a zip-file.
The following steps exemplifies how to get the error outside the ‘embedded’ context:

Install Python39 under Windows in path ‘c:\Python39’

C:\tmp\testpydantic>c:\Python39\Scripts\pip.exe install pydantic
Collecting pydantic
Downloading pydantic-2.7.1-py3-none-any.whl (409 kB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 409.3/409.3 KB 8.5 MB/s eta 0:00:00
Collecting pydantic-core==2.18.2
Downloading pydantic_core-2.18.2-cp39-none-win_amd64.whl (1.9 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.9/1.9 MB 15.3 MB/s eta 0:00:00
Collecting annotated-types>=0.4.0
Using cached annotated_types-0.6.0-py3-none-any.whl (12 kB)
Collecting typing-extensions>=4.6.1
Using cached typing_extensions-4.11.0-py3-none-any.whl (34 kB)
Installing collected packages: typing-extensions, annotated-types, pydantic-core, pydantic
Successfully installed annotated-types-0.6.0 pydantic-2.7.1 pydantic-core-2.18.2 typing-extensions-4.11.0
WARNING: You are using pip version 22.0.4; however, version 24.0 is available.
You should consider upgrading via the 'C:\Python39\python.exe -m pip install --upgrade pip' command.

Under: C:\Python39\Lib\site-packages, create a ‘pydantic.zip’ from ‘C:\Python39\Lib\site-packages\pydantic’

Rename folder ‘C:\Python39\Lib\site-packages\pydantic’ to ‘C:\Python39\Lib\site-packages\xxpydantic’

Move ‘pydantic.zip’ to ‘C:\tmp\testpydantic’

set PYTHONPATH=c:\tmp\testpydantic\pydantic.zip

C:\tmp\testpydantic>c:\Python39\python.exe
Python 3.9.13 (tags/v3.9.13:6de2ca5, May 17 2022, 16:36:42) [MSC v.1929 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

from pydantic import Field
Traceback (most recent call last):
File "", line 1, in
File "", line 1055, in handle_fromlist
File "c:\tmp\testpydantic\pydantic.zip\pydantic_init.py", line 389, in getattr
File "c:\Python39\lib\importlib_init_.py", line 122, in import_module
raise TypeError(msg.format(name))
TypeError: the 'package' argument is required to perform a relative import for '.fields'
from pydantic.fields import Field

From: Hasan Ramezani @.>
Sent: Thursday, 18 April 2024 18:04
To: pydantic/pydantic @.>
Cc: Johan Andersson @.>; Mention @.>
Subject: Re: [pydantic/pydantic] Use of package in pydantic/init.py fails if imported from zip (Issue #9221)
Thanks @johanandersson-tietoevrycomhttps://github.com/johanandersson-tietoevrycom for reporting this issue.
As package is discouraged in docs: https://docs.python.org/3/reference/import.html,
Agree, it is not recommended to use package. Thanks for your suggestion.
Could you please help us to reproduce the problem? We need some steps to reproduce the problem then we can confirm the fix works.
—
Reply to this email directly, view it on GitHubhttps://github.com/pydantic/pydantic/issues/9221#issuecomment-2064369377, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDUU75YHJG23FILVWWER4QTY57VGRAVCNFSM6AAAAABGDW6DK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRUGM3DSMZXG4.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Thanks @johanandersson-tietoevrycom for helping us to reproduce the problem.
I created https://github.com/pydantic/pydantic/pull/9331 to fix the problem. It would be great if you can confirm the fix.


Explosion docs

Updating the submodule isn't something I necessarily want PRs to do, it actually doesn't really effect anything, but since this needs resolving, I'm going to make a commit to the submodule tonight, and push an update to the branch, just update to that commit and hopefully this can be merged

@jaredlll08 ive force-pushed and removed the conflicting changes to the submodule, should be fine to merge now

@jaredlll08 ive force-pushed and removed the conflicting changes to the submodule, should be fine to merge now

For some reason I didn't get a notif about this, sorry about that

For some reason I didn't get a notif about this, sorry about that


Don't use VSCode ESLint experimental flag anymore

Changes

Don't use ESLint experimental flag anymore (it's now in the official plugin version)
Add the .docker folder to hidden folders
Remove custom Ruby LSP command. This was an attempt to mitigate https://github.com/Shopify/ruby-lsp-rails/issues/382, however it did not work (or rather was too much effort), so we just don't use the few Ruby LSP server-dependent features (for that we would have to run it inside docker containers) and stick with the static code analysis with is sufficient for us.

Missing anything in terms of settings in VSCode? This might be a good time to add respective settings here.


[Bug]: Couldn't use Azure openai with autogenstudio

Describe the bug
Hi experts:
I have also face this problem by using Azure Openai
INFO: <IP_ADDRESS>:49858 - "GET /page-data/build/page-data.json HTTP/1.1" 200 OK
INFO: <IP_ADDRESS>:49856 - "GET /page-data/gallery/page-data.json HTTP/1.1" 200 OK
INFO: <IP_ADDRESS>:49858 - "GET /component---src-pages-gallery-index-tsx-25eaa0ea2f26f1c4361f.js HTTP/1.1" 200 OK
INFO: <IP_ADDRESS>:49856 - "GET /component---src-pages-build-tsx-3bd5608467a276d90ea7.js HTTP/1.1" 200 OK
Traceback (most recent call last):
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\web\app.py", line 68, in add_message
response_message: Message = chatmanager.chat(
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\chatmanager.py", line 22, in chat
flow = AutoGenWorkFlowManager(config=flow_config, history=history, work_dir=scratch_dir)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\workflowmanager.py", line 37, in init
self.receiver = self.load(config.receiver)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\workflowmanager.py", line 158, in load
agent = self.load_agent_config(agent_spec.config, agent_spec.type)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\workflowmanager.py", line 173, in load_agent_config
agent = autogen.AssistantAgent(**agent_config.dict())
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\agentchat\assistant_agent.py", line 61, in init
super().init(
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\agentchat\conversable_agent.py", line 126, in init
self.client = OpenAIWrapper(**self.llm_config)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\oai\client.py", line 108, in init
self._clients: List[OpenAI] = [
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\oai\client.py", line 109, in
self._client(config, openai_config) for config in config_list
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\oai\client.py", line 149, in _client
client = OpenAI(**openai_config)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\openai_client.py", line 93, in init
raise OpenAIError(
openai.OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable
INFO: <IP_ADDRESS>:49856 - "POST /api/messages HTTP/1.1" 200 OK
even I delete all other model in my model list, I still face this problem

How can I sovle this problem ?
Thanks
Steps to reproduce
pip install autogenstudio
use Azure openAI configuration to config
Expected Behavior
No response
Screenshots and logs
INFO: <IP_ADDRESS>:49858 - "GET /page-data/build/page-data.json HTTP/1.1" 200 OK
INFO: <IP_ADDRESS>:49856 - "GET /page-data/gallery/page-data.json HTTP/1.1" 200 OK
INFO: <IP_ADDRESS>:49858 - "GET /component---src-pages-gallery-index-tsx-25eaa0ea2f26f1c4361f.js HTTP/1.1" 200 OK
INFO: <IP_ADDRESS>:49856 - "GET /component---src-pages-build-tsx-3bd5608467a276d90ea7.js HTTP/1.1" 200 OK
Traceback (most recent call last):
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\web\app.py", line 68, in add_message
response_message: Message = chatmanager.chat(
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\chatmanager.py", line 22, in chat
flow = AutoGenWorkFlowManager(config=flow_config, history=history, work_dir=scratch_dir)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\workflowmanager.py", line 37, in init
self.receiver = self.load(config.receiver)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\workflowmanager.py", line 158, in load
agent = self.load_agent_config(agent_spec.config, agent_spec.type)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogenstudio\workflowmanager.py", line 173, in load_agent_config
agent = autogen.AssistantAgent(**agent_config.dict())
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\agentchat\assistant_agent.py", line 61, in init
super().init(
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\agentchat\conversable_agent.py", line 126, in init
self.client = OpenAIWrapper(**self.llm_config)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\oai\client.py", line 108, in init
self._clients: List[OpenAI] = [
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\oai\client.py", line 109, in
self._client(config, openai_config) for config in config_list
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\autogen\oai\client.py", line 149, in _client
client = OpenAI(**openai_config)
File "C:\Users\bozho\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\openai_client.py", line 93, in init
raise OpenAIError(
openai.OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable
Additional Information
No response

only one model

 only one model

For this screen shot, which agent is it?
Can you go to Azure OpenAI studio(the page in your first screenshot), select the gpt-35 model, then click on "Open in Playground". On the playground page, middle panel click on "View code", you should see the settings of the model look like this:

openai.api_type = "azure"
openai.api_base = "https://xxxxxxx.openai.azure.com/"
openai.api_version = "xxxxxx"
openai.api_key = ......

Can you confirm they are the same as what you put in the second screenshot?

Still in "View code", line 14, there's "engine" which should match the name you put in the second screenshot. Can you also confirm that?

Hi @cheng-tan
Really strong issue, I see if I use 2023-07-01-preview API, it works, but when I use 2023-05-15 api version, it couldnot work
2023-07-01-preview is in the code playground

But if I use 2023-05-15 in other project , it works....

 only one model

For this screen shot, which agent is it?
Can you go to Azure OpenAI studio(the page in your first screenshot), select the gpt-35 model, then click on "Open in Playground". On the playground page, middle panel click on "View code", you should see the settings of the model look like this:

openai.api_type = "azure"
openai.api_base = "https://xxxxxxx.openai.azure.com/"
openai.api_version = "xxxxxx"
openai.api_key = ......

Can you confirm they are the same as what you put in the second screenshot?

Still in "View code", line 14, there's "engine" which should match the name you put in the second screenshot. Can you also confirm that?

Do you face this? Try many times, but I still don't know how to solve.
Thank you very much.
my another error is ：
"Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_exceptions.py", line 10, in map_exceptions
yield
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_backends\sync.py", line 206, in connect_tcp
sock = socket.create_connection(
^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\socket.py", line 851, in create_connection
raise exceptions[0]
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\socket.py", line 836, in create_connection
sock.connect(sa)
TimeoutError: [WinError 10060] 。
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_transports\default.py", line 67, in map_httpcore_exceptions
yield
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_transports\default.py", line 231, in handle_request
resp = self._pool.handle_request(req)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_sync\connection_pool.py", line 268, in handle_request
raise exc
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_sync\connection_pool.py", line 251, in handle_request
response = connection.handle_request(request)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_sync\connection.py", line 99, in handle_request
raise exc
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_sync\connection.py", line 76, in handle_request
stream = self._connect(request)
^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_sync\connection.py", line 124, in _connect
stream = self._network_backend.connect_tcp(**kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_backends\sync.py", line 205, in connect_tcp
with map_exceptions(exc_map):
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\contextlib.py", line 158, in exit
self.gen.throw(typ, value, traceback)
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpcore_exceptions.py", line 14, in map_exceptions
raise to_exc(exc) from exc
httpcore.ConnectTimeout: [WinError 10060]。
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\openai_base_client.py", line 885, in _request
response = self._client.send(
^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_client.py", line 915, in send
response = self._send_handling_auth(
^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_client.py", line 943, in _send_handling_auth
response = self._send_handling_redirects(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_client.py", line 980, in _send_handling_redirects
response = self._send_single_request(request)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_client.py", line 1016, in _send_single_request
response = transport.handle_request(request)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_transports\default.py", line 230, in handle_request
with map_httpcore_exceptions():
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\contextlib.py", line 158, in exit
self.gen.throw(typ, value, traceback)
File "C:\ProgramData\Anaconda3\envs\autogen\Lib\site-packages\httpx_transports\default.py", line 84, in map_httpcore_exceptions
raise mapped_exc(message) from exc
httpx.ConnectTimeout: [WinError 10060]。
"

Also facing the same issue here. When I look at the log it seems that the models are not being updated according to the new information I set UI?
INFO:     <IP_ADDRESS>:51138 - "POST /api/messages HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51146 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51146 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51209 - "POST /api/models HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51225 - "POST /api/models HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51257 - "POST /api/models HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51257 - "GET /build HTTP/1.1" 307 Temporary Redirect
INFO:     <IP_ADDRESS>:51257 - "GET /build/ HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51257 - "GET /app-e668c2f8516018ac0a1c.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51262 - "GET /framework-0e118549f33203964972.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /webpack-runtime-5d7173c472e7866cdb8d.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /page-data/app-data.json HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51262 - "GET /page-data/build/page-data.json HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /favicon-32x32.png?v=46131b614287332146ea078703a67d38 HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51265 - "GET /manifest.webmanifest HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /318ce576f236b79fd96f75904c13f6e55c3eee57-bf26f3994740657e0089.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51262 - "GET /component---src-pages-build-tsx-327ca1b88cf4c2d6fc61.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51262 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51262 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51262 - "GET /page-data/gallery/page-data.json HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /page-data/index/page-data.json HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /page-data/index/page-data.json HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /page-data/gallery/page-data.json HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51262 - "GET /component---src-pages-index-tsx-2342fef3effa18a4d864.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51263 - "GET /component---src-pages-gallery-index-tsx-25eaa0ea2f26f1c4361f.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51272 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51272 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51283 - "POST /api/agents HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51283 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51285 - "POST /api/agents HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51285 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51285 - "POST /api/agents HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51285 - "GET /6c05b3ab656a8bfc225a64b3d73a36a92f83c05e-6a87df80060a5da491fc.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51285 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51288 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51285 - "GET /images/svgs/welcome.svg HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:51285 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:51285 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
agent_spec.config  LLMConfig(config_list=[{'model': 'gpt-4-1106-preview'}, {'model': 'gpt-3.5-turbo-16k'}, {'model': 'TheBloke/zephyr-7B-alpha-AWQ', 'base_url': 'http://localhost:8000/v1'}], temperature=0.1, cache_seed=None, timeout=600)

The LLMConfig still shows the default list of models, but I updated it in UI:

remember the deployment name should be same as model. It should be working

@monuminu Do you mean the name in green in the screenshot above should be the same as model name in Azure OpenAI deployment, or? These are the deployments I have:

Why In the log it shows these three model  gpt-4-1106-preview, gpt-3.5-turbo-16k and TheBloke/zephyr-7B-alpha-AWQ while I have different models setup (as shown in previous screenshot). Shouldn't it be gpt-4, gpt-3.5-turbo-16k and dall-e-3?
INFO:     <IP_ADDRESS>:51285 - "GET<EMAIL_ADDRESS>HTTP/1.1" 200 OK
agent_spec.config  LLMConfig(config_list=[{'model': 'gpt-4-1106-preview'}, {'model': 'gpt-3.5-turbo-16k'}, {'model': 'TheBloke/zephyr-7B-alpha-AWQ', 'base_url': 'http://localhost:8000/v1'}], temperature=0.1, cache_seed=None, timeout=600)

 only one model

For this screen shot, which agent is it?
Can you go to Azure OpenAI studio(the page in your first screenshot), select the gpt-35 model, then click on "Open in Playground". On the playground page, middle panel click on "View code", you should see the settings of the model look like this:

openai.api_type = "azure"
openai.api_base = "https://xxxxxxx.openai.azure.com/"
openai.api_version = "xxxxxx"
openai.api_key = ......

Can you confirm they are the same as what you put in the second screenshot?

Still in "View code", line 14, there's "engine" which should match the name you put in the second screenshot. Can you also confirm that?

@lz-chen Can you try deleting the default models and add your new model? Are you able to follow step 2 and 3 in this comment to verify your configs are correct?


koordlet: add a resource usage predictserver to estimate long-term reclaimed resource

Ref #1361
This PR contains:

[x]  collecte metrics and store in the histogram
[x]  checkpoint histogram and restore

Capture Pod/Node utilization data and update it into the model, and provide the ability to query results.

/lgtm

/approve


fix: profiles now open in a new tab

👨💻 Changes proposed

[x] fix: profiles now open in a new tab

🔧 Check List (Check the type of issue it fixes)

[x] Bug Fix
[ ] New Extension
[x] Enhancement
[ ] Documentation

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules
[x] I have done git pull before adding my new changes
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
📷 Screenshots

I think you did it for the wrong line. It needs to be done for the GitHub icon FaGithub also

Oops! I forgot about that! By the way, the profile name is also linked to the GitHub page :P


NETOBSERV-1425: enhance metrics filters

Description

New filters: not_equal, not_match_regex
Breaking change: renamed filter types exact => equal, regex => match_regex
Variable interpolation: e.g. "api.MetricsFilter{Key: "src-ns", Value: "$(dst-ns)"}" to filter for intra-namespace traffic

Dependencies

n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[ ] Will this change affect NetObserv / Network Observability operator? If not, you can ignore the rest of this checklist.
[ ] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[ ] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[x] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

@jotak do you have an Operator PR for these changes ?

/approve


vlc: fix xcb abort with mkDerivation

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc #65399

already fixed on master! :)

guess I verified that in #65518 as well :rofl:


test(modelValidation): Enhance Error Message

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable)

Added path to Model Validation test output for easier troubleshooting when modifying searchable fields. Feel free to modify and merge as you see fit :) I'll be out for a bit, but can make updates and merge it in when I get back. If it needs to be rebased to avoid the merge commits I can do that as well, my fork has been around for a bit, sorry :sweat_smile:

@dexter-mh-lee : mind taking a look?


Reduce installation trouble by adding more dependencies to requirements_dev.txt

Step 4 of the tutorial currently says:

If you have issues building the wheel for cryptography, make sure that the required dependencies are installed. Follow the Cryptography Instruction for your OS

We should make this even easier for new users by adding all needed dependencies to requirements_dev.txt.

Sure, we would love your help with this one.

This should be addressed as of 04d061efaadbf5f223aac50089ce980393a019e3. Thank you for your patience!


Aiven Kafka connector support

Is your feature request related to a problem? Please describe.
Data prepper should be able to support events from Aiven Kafka connector
Describe the solution you'd like
Add Sigv4 support for Aiven Kafka connector enabling the same to post data to Dataprepper pipelines. I would envision the below sequence

Get branch : Sigv4 Changes
Confirm the HTTP format to configure a Data Prepper pipeline
Test against open-source Data Prepper without SigV4; using HTTP source
Test against OSI (same pipeline with SigV4)
Make any changes to the branch so that it works correctly
Create PR in to Aiven

Additional context
References: https://github.com/aiven/opensearch-connector-for-apache-kafka
https://github.com/deepdatta/http-connector-for-apache-kafka/tree/sigv4_support
https://github.com/deepdatta/opensearch-connector-for-apache-kafka-sigv4-configurator

This is not a change for Data Prepper.


Do not call done unless everything is drained, not just 0 active

It seemed to me like the spider was crawling done whenever there was not an active connection.
I changed this to only call done when the queue is completely drained.
This seems to be working more like expected in my instance.

If you check line 78, this branching is only reached if args is falsy, meaning, pending is empty. Adding this check is redundant.
Have you experienced first-hand the done handler being called multiple times?
If you did, please describe the flow that could trigger that given this code.
If not, then let me know and I'll close this.
Thanks

Oh, oops you're right I didnt notice how you are doing that.
I think the issue I might have been having was having allowDuplicates set to true. I am going to run more tests and I will let you know.
Currently, I am having an issue where I am calling spider.queue but I am seeing nothing in the logs about queuing the URL and I know there are far more active than it can handle.


Questions Regarding Preprocessing Implementation

Hi @dnguyengithub ,
I'm trying to implement your proposed solution on another AIS dataset but I'm facing a few problems in the preprocessing steps:

I'm having a hard time understanding this whole sampling process. In the paper you present the problem that we need regularly sampled data to train the model first. However, I am not sure I fully understand your proposed solution (i.e. using  constant velocity model to create regularly time-sampled AIS tracks). Moreover, if I understand your solution correctly, it sounds rather odd to generate regularly sampled data in such an artificial manner. I would really appreciate some explanation on this topic.

I'm a bit concerned about your train-test splitting strategy. In the code I can see that you split your data based on random permutation rather based on continuous time intervals. Doesn't it beat the purpose of learning sequences patterns from the past and check if they are valid in the future?

In the paper you mention that you only chose tracks which the maximum time interval between two successive received AIS messages is 1 hour. However, in your code I see that you split the tracks using 2 hours interval. What is the correct interval value?

In the paper you mention that you created regulary time-sampled AIS tracks at 10 minutes time scale. However, in your code I see that you used a 5 minutes sampling resolution. What is the correct interval value?

Any help with one or all of these questions will be much appreciated.
Thanks in advance.

Hi,

Yes, we used geodetic interpolation to geodesic interpolation to resample the track regularly. (see https://pyproj4.github.io/pyproj/dev/_modules/pyproj/geod.html). As long as the interval between two consecutive AIS messages is small, the error inherited from this interpolation wont be a problem.

Yes, this train-test splitting strategy is wrong. We should use data from one period for the training set then data from the next period for the test set. We tried the correct train/test split and the results are still good. We wrote another paper on this problem, I will update this repository as soon as our new paper is accepted.

The results in the papers were obtained with max_interval = 1 hour. However, for the Brittany dataset, max_interval = 1 hour and max_interval = 2 hours give similar results.

The sampling resolution is 10 mins. (see line #90 in datasets.py)

Hope that this answers your questions.

Thanks for your help, much appreciated.


Hi Artisans, How I can get cart id in products page

Hi Artisans. I wanna build some logic inside of my products page. I wanna change the button when user has been clicked on that product. So my question is. How I know the data information of carted in my products page. For example: I wanna check if the product has the same id with the added cart. I will change the button color.
Just like this.
<button class="btn btn-{{ $product->id == $cart_item->id ? 'success' : 'default' }}

I look a round of it. But I am still can't do this. Anyone can help me. I will really apreciated that :)
Thanks 👍

I think your looking for the Cart::search method. That way you can search for the Product with the given ID and if you get something back (meaning it is in the cart) change the color of the button.

But. This is in the product page. How I can get the rowId from it.

You don't need a rowId. With the search method you can just use the id, or name or whatever. You just want to know if the product with a given id (the one that you're viewing) is in the cart.

Can you give me the example

Example of the search method is in the readme. It'll return a collection. So you can check if it's empty (product not in the cart) or not empty (product in the cart at least once)


Reworked sensitive domain name and restriction logic

Finally a follow-up to pull request #600.

This is a fix for a potential security issue; the old code had edge
cases with unexpected behavior.
LEMUR_RESTRICTED_DOMAINS is no more, instead LEMUR_WHITELISTED_DOMAINS
is a list of allowed domain name patterns. Per discussion in PR #600
Domain restrictions are now checked everywhere: in domain name-like
CN (common name) values and SAN DNSNames, including raw CSR requests.
Common name values that contain a space are exempt, since they cannot
be valid domain names.

Coverage increased (+0.3%) to 60.47% when pulling 72b07143c380e89ce2cb4c5a9f52b73d3af63b6b on intgr:whitelisted-domains into b40c6a1c677288fa78488663d065517633aaddfb on Netflix:master.

Thanks!


New extension: One Signal Notification

Description
Send push Android/IOS notifications via one signal
How to use the extension
Make use of the initialize one signal action

go to https://onesignal.com to create an account.
Follow these steps to generate a Google server API key
https://documentation.onesignal.com/docs/generate-a-google-server-api-key
Checklist

[X] I've followed all of the best practices.
[X] I confirm that this extension can be integrated to this GitHub repository, distributed and MIT licensed.
[X] I am aware that the extension may be updated by anyone, and do not need my explicit consent to do so.

What tier of review do you aim for your extension?
Community (Unreviewed)
Example file
msg.zip
Extension file
OneSignal.zip

OneSignal.zip

Hello,
Thank you for your amazing contribution.
I am trying to integrate the extension but somehow I am not able to register new subscribers.
I followed the guide bellow
OneSignal Documentation
Android: Firebase Credentials
Directions on how to generate a Firebase Cloud Messaging API Credentials for sending Android App Notifications
1/ Activate Firebase Cloud Messaging API (V1)
2/ Generate a Private Key JSON file (Node.js)
3/ Activate the platform Google Android (FCM) in OneSignal (In the part “Select SDK” I tried both Native android and Cordova)
4/ Download and import the extention in Gdevelop (Onesignal GDevelop Extension · GitHub)
5/ Integrate it as follow :
Beginning of the first scene >
Request Notification Permission : TRUE
Initialize Onesignal (My app ID + user must consent TRUE)
I tried the application using mobile manual build and mobile One-click packaging. I tested the .apk in a real phone and in android studio virtual devices (with playstore installed).
I checked, the notification is allowed.
But still, when I launch the application no new subscriber is registered in OneSignal.
Thank you in advance for your precious time.


PowerShell module information

The documentation should provide some information on which PowerShell module contains the ADFS and WebApplicationProxy cmdlets.

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

ID: d10177c6-b241-0a3e-d6ba-3619c094812a
Version Independent ID: 7a42691f-7f28-eee2-0c69-294e24cccdbe
Content: Managing SSL Certificates in AD FS and WAP in Windows Server 2016
Content Source: WindowsServerDocs/identity/ad-fs/operations/manage-ssl-certificates-ad-fs-wap.md
Service: unspecified
GitHub Login: @jenfieldmsft
Microsoft Alias: billmath

Had same issue while trying to debug SSL certs expired.
Get-Command Get-WebApplicationProxySslCertificate | Select -ExpandProperty Source
# WebApplicationProxy
Get-Module WebApplicationProxy | select -ExpandProperty Path
# C:\Windows\system32\WindowsPowerShell\v1.0\Modules\WebApplicationProxy\WebApplicationProxy.psm1

Guessing this module is installed by default when WAP is installed, as it's not on the ADFS box.
ADFS is here:
# And ADFS is here:
Get-Command Get-ADFSProperties | Select -ExpandProperty Source
# ADFS
Get-Module ADFS | select -ExpandProperty Path

I'm struggling with this as well, only on my server there seems to be no WebApplicationProxy module around.
Get-Module WebApplicationProxy | select -ExpandProperty Path returns nothing and there is no "WebApplicationProxy" folder in my Modules folder. I'm not sure when/what/where but I do not believe the module was even installed. So now I'm struggling to find where I can get it from so I can actually run these commands!

Same here. Replacing an expired certificate requires these commands, but they aren't present. Running
Add-WindowsFeature web-application proxy fails with "has a conflict with ADFS". Now I need to call MS Enterprise support to simply replace an expired SSL cert. So lame.

#please-close


Grades from Student Teaching

Notes and Homework
Grade

3.1 & 3.2 Presentation
3.4/4

3.1 & 3.2 Grading
2.5/3

3.3 & 3.4
1/1

3.5 - 3.7
0.9/1

3.8 - 3.10
0.9/1

3.9 - 3.11
0.7/1

3.12 & 3.13
0.8/1

3.14 & 3.15
0.85/1

3.16
0.8/1

3.17 - 3.18
0.93/1

[Vocab](https://sarahliu2006.github.io/Sarah-Liu/vocab/)

I see some effort in your vocabulary.  However, I would really like to see more examples and see this work in an IPYNB file so you would have outputs.   I am really looking for you to build confidence with Teacher on coding and doing technical solutions.   BTW, you did not put link in correctly;  this kind of thing needs to improve in your review tickets.  But, I see progress this week.


Split Metadata Model for Tables

Currently we store TableMetadata as one blob in Postgres.
We should investigate two options:

[ ] Don't store TableMetadata in Postgres at all, but on a Object Store instead
[ ] Split TableMetadata in Postgres into smaller tables, as requests often filter to only include snapshots for a certain ref

Don't store TableMetadata in Postgres at all, but on a Object Store instead

I think supporting this option would be a big boon to adoption, because:

that's how other SQL catalogs work (e.g., JVM's JDBC Catalog, PyIceberg SqlCatalog)
the metadata json file is an integration point for query engines that don't (yet) support the REST API (e.g., BigQuery, but also the admittedly quite crappy duckdb iceberg extension, etc).

I would consider adopting this catalog implementation today in my work if this were supported.

@corleyma thanks for your Feedback!
We are already today writing metadata to the location of the table (as metadata/<uuid>-.gz.metadata.json). We just store it additionally internally, which is our source of truth. This internal storage is what's bugging me mostly currently, because it's a big binary chunk in Postgres. The location of the external one is returned to the client as part of the TableMetadata.
With the integration of other tools you are touching an interesting point:
We are deliberately not writing the toplevel metadata.json in a file called like that, even if it would introduce higher compatibility with other engines. The reason is twofold:

It could lead to inconsistencies on write - only the catalog can guarantee transactions on some storage systems such as S3
We don't want clients to have credentials in the first place

Technically it wouldn't be a problem to enable it optionally. Is this what you are thinking about?
With our model we are following what tabular.io does. They also don't write a toplevel "metadata.json" and if you manually fiddle with metadata/<uuid>-.gz.metadata.json it has absolutely no effect because metadata stored somewhere internally is returned.


Bug - Fetch seems not resilient to failures

The error was caused because the authetication service was not up. It should cause the response to be unsuccessful but the
app should not crash.
Res to Client:

bug found in  #23

The problem was actually beacause I was fetching the auth service in a wrong IP adress, nonetheless the app should not crash.


[SPARK-48033][SQL] Fix RuntimeReplaceable expressions being used in default columns

What changes were proposed in this pull request?

Currently, default columns that have a default of a RuntimeReplaceable expression fails.
This is because the AlterTableCommand constant folds before replacing expressions with the actual implementation. For example:
sql(s"CREATE TABLE t(v VARIANT DEFAULT parse_json('1')) USING PARQUET")
sql("INSERT INTO t VALUES(DEFAULT)")

fails because parse_json is RuntimeReplaceable and is evaluated before the analyzer inserts the correct expression into the plan
To fix this, we run the ReplaceExpressions rule before ConstantFolding
Why are the changes needed?

This allows default columns to use expressions that are RuntimeReplaceable
This is especially important for Variant types because literal variants are difficult to create - parse_json will likely be used the majority of the time.
Does this PR introduce any user-facing change?

no
How was this patch tested?

added UT
Was this patch authored or co-authored using generative AI tooling?

no

cc @cloud-fan FYI

thanks, merging to master!


NOT an issue, but a question :)

Hi,
I read the paper and it very sounds. I have been trying NVIDIA StyleGanV2-ADA for weeks without success. It simply did not converge and generated images were full of unwanted artifacts. My dataset is furniture (living room, bed room etc.) and each dataset has over 60 - 150k images. Yes some has only 10k images but we have not tried them.
Even with 100k of images, StyleGANV2-ADA original version failed us for whatever reasons. We tried R1 all ranges, drop/add augmentation bgcfc etc. Not working. We ran up to 30,000 kimg (35 million images through network), still lots of weird streaks in the images.
We are about to try this repo, and can you advise something along the way:

Is there something special about our dataset that may need special attention parameters? Which parameters are to be taken care of? These images are NOT faces / dog / cat where they cropped / highly focus, our dataset as you may know they are widely different in scenes, color and convergence means minimal artifacts around :)

How long do you think it will take for convergence if at all ? We use 4 GPUs GTX 2080ti in our lab for now hence run batch size of 32 for 256x256 image size.

Would love to keep in touch.
Thanks in advance.
Steve
<EMAIL_ADDRESS>

Your dataset seems large enough so the problem does not sound like a discriminator over fitting issue. Could you share some generated and real images?

Here we go:
REAL:

FAKE

I think the generated images do not look very bad :) So maybe it's because the dataset is quite challenging, and state-of-the-art GAN models are still limited in various aspects like model capacity and training methodology to even though the dataset is large enough.

Getting worsen results...

A clear example of how people overestimate AI capabilities. They trust advertisement/teaser videos and cherry-picked examples in papers too much :-D
Man, these results are incredible for a dataset of just 0.03M images with super complex relations and perspective.


Keep client/server connection alive

Send simple PING/PONG messages. If one side didn't respond, close the connection.
Also clear the the request from the queue if the forward request failed.

Fixed by #16


Support override path to support Azure/GCP Functions

Support override path for the buffer2webpbuffer method
The option should be overridable since node_modules is inaccessible due to symlinks
Instead use the %TEMP% path for Azure Functions where application is allowed to read/write to by default

I have this exact issue but with Google Cloud Functions, can this be merged please?

I have this exact issue but with Google Cloud Functions, can this be merged please?

Just made this identical PR, I'll close mine in favor of this one. This functionality is also needed for AWS Lambda as it has a /tmp/ directory, which is not handled in temp_file.js.

Path issue fixed author


Parsing escaped attributes shouldn't set raw property in resulting node

Example JSX code:

The relevant part of the ESTree:
openingElement: Node {
    type: 'JSXOpeningElement',
    attributes: [
    	Node {
        	type: 'JSXAttribute',
	        name: Node {
    		    type: 'JSXIdentifier',
		        name: 'data-foo'
		    },
        	value: Node {
		        type: 'Literal',
		        value: '"',
        		raw: '"""' // <-- problem is here
        	}
	    }
    ],
    // ...
}

I believe the raw property of the literal is wrong. When transforming such a tree and preserving the literals as-is, some code generators (e.g. astring) will prefer the raw property and emit it as is:
https://github.com/davidbonnet/astring/blob/92d26a05f666fa4f7a3475df67773581c1dff9a0/src/astring.js#L938-L940
This may lead to generated code that looks as follows:
React.createElement(
  "span",
  { "data-foo": '"""' }
)

Naturally this is wrong, because React and other tools will escape values by themselves. Interestingly enough, escodegen appears to ignore the raw property.
The naive solution would be to nuke all raw properties nested within any JSX node, but that would be a little more than necessary:

Here, the escaping rules are regular JS rules.
My proposal would be to remove raw from all literals that are nested directly below an attribute.
The following situation is not affected:
&

... for the sole reason that text nested in elements are not parsed as literals, but rather as the separate JSXText node type.

I don't understand why acorn-jsx is transforming HTML entities at all. That is up for the user agent to do, not the JSX parser.
I have in my JSX </> used within an icon. Parsing this within a bundler and then feeding it into the JSX compiler I'm using (not React in my case) is causing the compiler to think there's a literal </> there, when in fact it's simply text intended for the user agent in particular.
What is the use-case of transforming HTML entities? JSX isn't HTML; the original, un-altered text should be put into the AST nodes, not arbitrarily transformed text.


UDP+PSA

thanks for u code!what about this model?UDP+PSA,do u want to do this

Currently, No. My main focus is to achieve good accuracy with minimal decoding steps and simple model. As far as I know, UDP requires extra post-processing steps to get high accuracy. And PSA is a module that can be integrated in HRNet (this part is simple). Moreover, there is no official pretrained weights since I cannot train myself.

您的仓库的代码实现了哪些新的模型吗？@sithu31296


Fix Emphasis prop

Highlighting with emphasis prop, does not work well with text transformation.

Hi Abraham, thank you so much for all your work on this library.
I'm encountering the same issue using the emphasize prop on transformed text (e.g. searching for "Ja" in "Japan" will cause the suggestions to display "JaPan" instead of "Japan"); this can be reproduced using the codesandbox link in the readme which is using v5.1.5 (although I'm on v4.22.0).
What would be the fix for this issue?
Thanks in advance.


wget: TLS error from peer (alert code 80): 80

These messages appear in the minikube log when attempting 'Deploy as a Sidecar':
get-eds-citadel Connecting to github.com (<IP_ADDRESS>:443)
get-eds-citadel wget: TLS error from peer (alert code 80): 80
get-eds-citadel wget: error getting response: Connection reset by peer

The problem is the old version of wget, because of an older image of busybox (1.28) in deployment.yaml.
Changing this to 1.36.1 (latest as of writing) solved the problem:
image: busybox:1.36.1

Thanks for reporting the issue @Marko-TRG.
The problem was fixed last week in https://github.com/aserto-dev/topaz/pull/357 and released in topaz v0.31.10.

Thanks, Ronen!


[Storage] Upgrade API version to 2022-05-01

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Storage

This change is breaking the Sql VM tests which is blocking the core pipeline - https://dev.azure.com/azure-sdk/public/_build/results?buildId=1791614&view=ms.vss-test-web.build-test-results-tab

@ArthurMa1978, can you update the affected session records or revert this change to unblock the core pipeline? We should also add a dependency in the storage ci.yml so that the Sql tests are triggered for changes to storage files.

@ArthurMa1978, can you update the affected session records or revert this change to unblock the core pipeline? We should also add a dependency in the storage ci.yml so that the Sql tests are triggered for changes to storage files.

@JoshLove-msft, I have merged a PR to fix the broken tests and now the core ci is not blocked: https://dev.azure.com/azure-sdk/public/_build/results?buildId=1792903&view=results

@ArthurMa1978, can you update the affected session records or revert this change to unblock the core pipeline? We should also add a dependency in the storage ci.yml so that the Sql tests are triggered for changes to storage files.

@JoshLove-msft, I have merged a PR to fix the broken tests and now the core ci is not blocked: https://dev.azure.com/azure-sdk/public/_build/results?buildId=1792903&view=results

Thank you!


Freeze at snd_pcm_writei

I am trying to use bluealsa with android phone and my thread is getting freezed at snd_pcm_writei
My phone is connected in hfp-ofono profile
It does works for few frames and then freezes. It starts again if i dial from phone but as soon as i hangup, it freezes again.
Can you help ?

Is it possible to complete blocking snd_pcm_writei call with dummy frames if it is needed?

I get some kind of deadlock at
if ((ret = write(pcm->ba_pcm_fd, head, len)) == -1) {
I am using my Android phone as BT Device.
Any suggestion would be really helpful.
Thanks

I am able to generate same issue when i do it via aplay
aplay -D bluealsa:DEV=A8:3E:0E:68:02:D9,PROFILE=sco /home/pi/one_hour_music.wav
it gets freezed exactly at same spot. I am using
/usr/bin/bluealsa --profile=a2dp-sink --profile=hfp-ofono

Any help ?
I am not facing this problem when i use only PulseAudio but i like bluealsa and want to use it.
Thank you.


lumberjack-bedrock permission error while cloning Bedrock repository

What are the steps to reproduce this issue?

composer global require rareloop/lumberjack-bedrock-installer
lumberjack-bedrock -v new mysite

What happens?
Permission error. lumberjack-bedrock is unable to clone Bedrock repository.
What were you expecting to happen?
For lumberjack-bedrock to create a new Lumberjack site.
Any logs, error output, etc?
Checking out Bedrock
Install failed
Symfony\Component\Process\Exception\ProcessFailedException: The command "git clone --depth=1 'git@github.com:roots/bedrock.git' '/Users/andrew/devilbox/data/www/dev/htdocs/rosefarms/dev/rosefamily' && rm -rf '/Users/andrew/devilbox/data/www/dev/htdocs/rosefarms/dev/rosefamily/.git'" failed.

Exit Code: 128(Invalid exit argument)

Working directory: /Users/andrew/devilbox/data/www/dev/htdocs/rosefarms/dev

Error Output:
================
Cloning into '/Users/andrew/devilbox/data/www/dev/htdocs/rosefarms/dev/rosefamily'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

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

What versions of software are you using?
Operating System: macOS High Sierra 10.13.6
PHP Version: PHP 7.1.16

Thanks for the detailed info. Sometimes you get that error message if you don't have an SSH key added to your Github account. Are you able to rule this out for us please?
https://help.github.com/en/enterprise/2.15/user/articles/adding-a-new-ssh-key-to-your-github-account

Hmm. I did have an SSH key added to my account, but it seems like there was an issue with it. After readding the key the issue seems to be resolved. ¯_(ツ)_/¯
Thanks @adamtomat :)

Awesome to hear you got it resolved @clams4shoes 👍


Move Examples into their respective packages

Was thinking about this over the weekend. Would be easier to maintain, and make more sense to contributors.

Closing this, because alot of the examples depend on multiple pa ckages


Add compare operators for partition filter.

Description
Add compare operators for partition filter, including gt, ge, lt and le.
Related Issue(s)

closes #472

Documentation

Thanks @zijie0


API Database Works

We are planning to complete some works to the API Database Backend.
This is a General Alert as it should not affect the public operations of the API at this time.
Any updates will be posted here.
Thank You. - MPSA

Works complete. Thank You - MPM


[Events.Organizer Rating] The modal window with message ‘Rating sent’ is not shown on the 'Events' page after clicking on the rating mark

Environment: Windows 10 x64, Google Chrome 111.0.5563.147
Reproducible: always
Build found: last commit from https://github.com/ita-social-projects/GreenCityClient/tree/dev
E.g. "commit 147fa44"
Preconditions

The user must be registered in the WEB-site: https://www.greencity.social/#/ubs
The user joined at least one event.
Go to https://www.greencity.social/#/greenCity

Steps to reproduce

Click on the "Events" tab.
Find "Event" which user attended previously.
Click on the "Rate Organizer" button.

Actual result
The modal window with message ‘Rating sent’ is not shown on the 'Events' page after clicking on the rating mark
Expected result
The modal window with message ‘Rating sent’ is displayed on the 'Events' page after clicking on the rating mark
User story and test case links
E.g.: "User story #3268
Test case"

https://github.com/ita-social-projects/GreenCity/assets/130749567/24dbcc0d-e868-4e8e-a2ec-7e96a966e781

It is done already

This bug is fixed. Tnx


Request: Package for f-droid and/or make apk part of the release artifacts on Github

I would like to use this app, but am not a google play user - I strictly use https://f-droid.org/ as an app store for OSS apps. In an ideal world, this app could live there! Is there any plans to do that?
A lower bar to entry, and one I would be willing to use is simply to build and publish an apk (and checksum) in the tags https://github.com/novasamatech/nova-wallet-android/tags . Some helpers for reference (might not be the exact right solution but a good start) :

https://github.com/marketplace/actions/release-apk-publisher
https://github.com/marketplace/actions/generate-checksum

Hey! We currently publish GH release in a separate repository. However we're planning to move releases publication to this repository pretty soon. I will add requirement about check-sum to the migration task, thanks for suggesting it

Nice! Is see in https://github.com/novasamatech/nova-wallet-android/commit/abc810895ed4a8991068e343059c1e21708bcc2e that this was introduced, but I was not going to find that in the README - might this be mentioned in an easy to find place? IMHO this repo should contain the artifacts... But hey, you guys do you :heart:  - very glad to have them!

Thanks for support! That's what we're are aiming to do -  GH releases should be moved to this repo pretty soon


Node.js  Community Committee Meeting 2021-09-16

Time
UTC Thu 16-Sep-2021 16:00 (04:00 PM):

Timezone
Date/Time

US / Pacific
Thu 16-Sep-2021 09:00 (09:00 AM)

US / Mountain
Thu 16-Sep-2021 10:00 (10:00 AM)

US / Central
Thu 16-Sep-2021 11:00 (11:00 AM)

US / Eastern
Thu 16-Sep-2021 12:00 (12:00 PM)

EU / Western
Thu 16-Sep-2021 17:00 (05:00 PM)

EU / Central
Thu 16-Sep-2021 18:00 (06:00 PM)

EU / Eastern
Thu 16-Sep-2021 19:00 (07:00 PM)

Moscow
Thu 16-Sep-2021 19:00 (07:00 PM)

Chennai
Thu 16-Sep-2021 21:30 (09:30 PM)

Hangzhou
Fri 17-Sep-2021 00:00 (12:00 AM)

Tokyo
Fri 17-Sep-2021 01:00 (01:00 AM)

Sydney
Fri 17-Sep-2021 02:00 (02:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Community Committee+Meeting+2021-09-16&iso=20210916T1600
or https://www.wolframalpha.com/input/?i=04PM+UTC%2C+Sep+16%2C+2021+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/12pawqy__RzHY3aMhDg9bJxpq0jWFZ-UYZEAcG8pvlpM/edit

Agenda
Extracted from cc-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
Invited

CommComm Members: @nodejs/community-committee

Observers/Guests
Feel free to follow along on the YouTube live stream, or attend meeting as a guest
by calling in to Zoom, using the links below. If you will be attending as a guest,
please comment on this issue to let us know you'll be joining.
Notes
The agenda comes from issues labelled with cc-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

Link for participants: https://zoom.us/j/95198277339
For those who just want to watch: https://www.youtube.com/channel/UCQPYJluYC_sn_Qz_XE-YbTQ

Invitees
Please use the following emoji reactions in this post to indicate your
availability.

:+1: - Attending
:-1: - Not attending
:confused: - Not sure yet

is there a reason to have this meeting today?

given that I've not seen anyone say "yes" i'll go ahead and say we should cancel this.


Fix spelling mistake in permission check

Description

small fix
User Impact

This is not seem to be an issue on prod. Weird
Demo

Docs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!


Removing focus target code

Removing focus target code
Changes

Modified code to remove focus target code. This was fixed in https://bugs.chromium.org/p/chromium/issues/detail?id=473657

Testing

Run ./frontend.sh.
Visit http://localhost:8000/#main.
Click tab, and verify that the Skip to main content button is visible.
Click the Skip to main content button and verify the #main div element is scrolled into view.

Screenshots
Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer 8, 9, 10, and 11
[ ] Edge
[ ] iOS Safari
[ ] Chrome for Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

The PR title could be a bit more detailed.

At last!!! This mentioned could be removed as well https://github.com/cfpb/cfgov-refresh/blob/master/cfgov/legacy/static/nemo/_/js/v1-nonresponsive-header.js#L7

@anselmbradford, do we still need that. Why can't we use the regular header?


Harbuzz10 and Remove runtime requirement on the xorg-proto headers

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #314.
Closes #316
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

The main issue is that now we've started to move to xorg-xorgproto.
The clobber each other, but we decided that this was "ok".
In either case, we shouldn't need these at runtime.

@conda-forge-admin please rerender
lets combine this with the harbuzz migration (even though I have the build ready already....)

aarch64_log.zip


there is no undo button and i just lost over an hour of work/ a big project

I cannot get my work back after I accidentally used the paint all pixels tool witch is right beside the paint bucket tool

CTRL + Z or CMD + Z is the shortcut to undo

Oh. Thanks

redo doesn't work for me


postgres: sqlx prepare fails if shared_preload_libraries=pg_stat_statements

Bug Description
since sqlx-cli version >=0.7.0 sqlx prepare fails if target database has pg_stat_statements in shared_preload_libraries
bug doesn't exists in =0.6.3
Minimal Reproduction

init project

cargo init sqlx-test
cd sqlx-test

setup db
docker-compose.yml

version: '3.7'

services:
  db:
    image: postgres:15.3
    restart: unless-stopped
    volumes:
      - db_data:/var/lib/postgresql/data
    command: postgres -c shared_preload_libraries=pg_stat_statements
    ports:
      - 5432:5432
    environment:
      POSTGRES_PASSWORD: changeme
      POSTGRES_DB: test
    healthcheck:
      test: [ "CMD-SHELL", "pg_isready -U postgres" ]
      interval: 10s
      timeout: 5s
      retries: 5

volumes:
  db_data: null

Cargo.toml

[package]
name = "sqlx-test"
version = "0.1.0"
edition = "2021"

[dependencies]
sqlx = { version = "0.7.1", features = [
    "macros",
    "postgres",
    "runtime-tokio-rustls",
] }
tokio = { version = "^1", features = ["full"] }

src/main.rs

use sqlx::postgres::PgPoolOptions;

#[tokio::main]
async fn main() -> Result<(), sqlx::Error> {
    let database_url = std::env::var("DATABASE_URL")
        .map_err(|_| "DATABASE_URL env var is missing".to_string()).unwrap();

    let pool = PgPoolOptions::new()
        .max_connections(5)
        .connect(&database_url).await?;

    sqlx::query!("SELECT 1 as foo").fetch_one(&pool).await?;
    Ok(())
}

installl sqlx-cli

cargo install --version 0.7.1 sqlx-cli

run db

docker compose up db

set DATABASE_URL

export DATABASE_URL="postgres://postgres:changeme@localhost:5432/test?sslmode=disable"

run

cargo sqlx prepare

error: error occurred while decoding column 0: expected value at line 12 column 5
  --> src/main.rs:12:5
   |
12 |     sqlx::query!("SELECT 1 as foo").fetch_one(&pool).await?;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: this error originates in the macro `$crate::sqlx_macros::expand_query` which comes from the expansion of the macro `sqlx::query` (in Nightly builds, run with -Z macro-backtrace for more info)

now change this in docker-compose.yml, stop database container and repeat steps 6 and 8

     restart: unless-stopped
     volumes:
       - db_data:/var/lib/postgresql/data
-    command: postgres -c shared_preload_libraries=pg_stat_statements
+    command: postgres -c shared_preload_libraries=
     ports:
       - 5432:5432
     environment:

Info

SQLx version: 0.7.1
SQLx features enabled: [REQUIRED]
Database server and version: Postgres 15.3
Operating system: linux
rustc --version: rustc 1.71.0 (8ede3aae2 2023-07-12)

this is not a big deal really:

one can use SQLX_OFFLINE=true
and in dev env disable pg_stat_statements when running sqlx prepare

that being said I figured that it's worth reporting anyway

It appears that it's somehow changing the output of the EXPLAIN (VERBOSE, FORMAT JSON) <query> statement that the macros use to analyze the query. Can you post the output of that command?

pg_stat_statements enabled
SHOW shared_preload_libraries;

 shared_preload_libraries
--------------------------
 pg_stat_statements
(1 row)

EXPLAIN (VERBOSE, FORMAT JSON) SELECT 1 as foo;

                 QUERY PLAN
---------------------------------------------
 [                                          +
   {                                        +
     "Plan": {                              +
       "Node Type": "Result",               +
       "Parallel Aware": false,             +
       "Async Capable": false,              +
       "Startup Cost": 0.00,                +
       "Total Cost": 0.01,                  +
       "Plan Rows": 1,                      +
       "Plan Width": 4,                     +
       "Output": ["1"]                      +
     },                                     +
     "Query Identifier":<PHONE_NUMBER>456321454+
   }                                        +
 ]
(1 row)

pg_stat_statements disabled
SHOW shared_preload_libraries;

 shared_preload_libraries
--------------------------

(1 row)

EXPLAIN (VERBOSE, FORMAT JSON) SELECT 1 as foo;

           QUERY PLAN
--------------------------------
 [                             +
   {                           +
     "Plan": {                 +
       "Node Type": "Result",  +
       "Parallel Aware": false,+
       "Async Capable": false, +
       "Startup Cost": 0.00,   +
       "Total Cost": 0.01,     +
       "Plan Rows": 1,         +
       "Plan Width": 4,        +
       "Output": ["1"]         +
     }                         +
   }                           +
 ]
(1 row)

Yeah, the parsing doesn't know what to do with thatbQuery Identifier field: https://github.com/launchbadge/sqlx/blob/main/sqlx-postgres/src/connection/describe.rs#L495


Cicada Theme Update for sidebar

This one looks good, ok to pull in?

Yes, there were some issues with the scss files in both tukan and cicada which I only noticed when I checked the commit. These should be fine.
Sent from my Samsung Galaxy smartphone.
null


Burning the release/git hash into the build

This will leave some useful information in the logs
I1117 10:26:16.709600   44258 launch.go:112] &{ALB Ingress Controller 1.0-alpha.7 git-0ddcba1f git://github.com/coreos/alb-ingress-controller}

Coverage remained the same at 21.348% when pulling ca5248524f334eea8dd0c6d7570b114298eb69e1 on builds into 0ddcba1f8274780ff68aaf50e4683023802b5f00 on master.

Works good on my end 👍


From where you get this id const "ZGJBtcsDq"

In the URL
let url = "https://vt.tiktok.com/ZGJBtcsDq/";
there's one id name as ZGJBtcsDq from wher we can get it

I guess there is a share button on tiktok.

How share button is gonna help with this user id

Okay, Got it
But is there any way we can scrap the data from @username

Can you help me with this, I can pay you for that

Okay, Got it
But is there any way we can scrap the data from @username

Maybe we can.

Can you help me with this, I can pay you for that

I'll try :)

Okay, Got it
But is there any way we can scrap the data from @username

Here some results that I can scrape.

That's amazing,
Great work boy
On Mon, 26 Dec 2022, 1:36 am FrierenDv, @.***> wrote:

Okay, Got it
But is there any way we can scrap the data from @username
Here some results that I can scrape.
[image: Result]
https://camo.githubusercontent.com/525989e1c0eeeaa861c3bb4229a00f1944eb534f7894c4ff9e1fc2b6107f4c47/68747470733a2f2f74656c656772612e70682f66696c652f3933363165376337396631303330346634616661372e6a7067
—
Reply to this email directly, view it on GitHub
https://github.com/xct007/tiktok-scraper/issues/2#issuecomment-1364735642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUAROTPW4ZBUEQK65SBJSSLWPCV4NANCNFSM6AAAAAATI4Y6AE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can we have a meeting call,  To discuss this further
Let me know when you are available
On Mon, 26 Dec 2022, 1:37 am Abdullah Afzal, @.***>
wrote:

That's amazing,
Great work boy
On Mon, 26 Dec 2022, 1:36 am FrierenDv, @.***> wrote:

Okay, Got it
But is there any way we can scrap the data from @username
Here some results that I can scrape.
[image: Result]
https://camo.githubusercontent.com/525989e1c0eeeaa861c3bb4229a00f1944eb534f7894c4ff9e1fc2b6107f4c47/68747470733a2f2f74656c656772612e70682f66696c652f3933363165376337396631303330346634616661372e6a7067
—
Reply to this email directly, view it on GitHub
https://github.com/xct007/tiktok-scraper/issues/2#issuecomment-1364735642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUAROTPW4ZBUEQK65SBJSSLWPCV4NANCNFSM6AAAAAATI4Y6AE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Deleting a cluster that provisioned successfully but is in error state doesn't remove management cluster

Rancher Server Setup

Rancher version: 2.6.1-rc10

Information about the Cluster

Cluster Type (Local/Downstream): Downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Digital Ocean

Describe the bug

If a cluster provisioned from Rancher is successful, but goes into an error state for some reason, then deleting it will successfully delete the provisioning cluster object but not the management cluster object.
To Reproduce

Provision a Digital Ocean cluster with one machine pool, put a '.' in the machine pool name (like harvester-v.0.3.0-rc1-creds-pool1)
Wait for provisioning to complete and an error related to namespaces will appear in the UI.
Delete the cluster.

Result
The cluster will be removed from the "Cluster Management" page but not the "Home" page.
Expected Result
The cluster should disappear from both pages.

Still got this problem in 2.6.2. Cluster does not disappear from home page, status Unavailable.
Any workaround how to remove it?

For a workaround, try removing your finalizers on the cluster:
kubectl patch clusters.management.cattle.io ${CLUSTER_ID} -p '{"metadata":{"finalizers":null}}' --type=merge

set CLUSTER_ID to the id of your cluster (c-xxxxxx)


Migrate Topographica's remaining param tests

Things like instantiation of all classes + pickle, etc

what's topographica?!?!?

 Here is some code if you are interested in how it all began.

Topographica's test suite was probably a pretty useful "downstream" test of param (because param was part of topographica).
When we hacked out param we didn't have time to move everything, let alone get all the tests - we weren't even sure exactly how much we would be chopping out to produce the standalone param lib.
It's likely there are still things tested in topographica that are not tested in param, e.g. I just looked really quickly and saw https://github.com/ioam/topographica/blob/master/topo/tests/unit/testFilename.txt ("Filename" does not seem to appear in param's current test suite). Maybe there are other tests scattered around, inline doctests, and so on. Who knows. Whether or not those tests are any good is another question ;)

I remember doing the initial migration of the unit tests: I did everything that looked like it was purely about param but there were certainly param related tests that didn't get moved.


[FEA] Accelerate the data transfer for plan WindowInPandasExec

Is your feature request related to a problem? Please describe.
No, it is an improvment.
Describe the solution you'd like
Read the data from ColumnBatch dierectly and send/receive it to/from Python side.

Hi @firestarman can you give a little more context about this feature?  Is it related to another issue or PR?

This is similar work that was done previously for GpuMapInPandasExec instead of going from CUDF to rows to arrow, can we just go directly to arrow.  The big issue with this is the processing needed on the incoming data. GpuMapInPandasExec just needed to divide the data up into more or less fixed size chunks. For windowing we need to replicate what the window operator does so each chunk sent to pandas is for a single partition window.

Thanks Bobyy for explaining.
Hi @sameerz, I added this to track my task originally, and filed a PR for this issue just now.
Besides, i updated the description.


feat: create new release

PR summary

Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[X] Other (PR to create new release)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

:tada: This PR is included in version 0.28.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix unittests of BatchL2NormSquared

This PR applies changes of #848 to BatchL2NormSquared's unittest.
I also fix the minor coding style error.
Related to #828


python3Packages.hstspreload: 2021.6.28 -> 2021.7.5

Motivation for this change
Update to latest upstream release 2021.7.5
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129541 at 10302eb7 run on aarch64-linux 1

2 packages built successfully:

python38Packages.hstspreload
python39Packages.hstspreload


cart module: refactor public accessors into functions and rename functions

Originating from #272 and discussed in Nov 29 refinement.

Refactor public accessors into functions.
If needed, rename functions to be short and into context
i.e. productService.get() instead of productService.getProduct()
Avoid redundancy.
Additionally we are allowed to not mirror the commerce platform.

There is a follow up ticket for facades to load once in the store (epic/loadonceinstore).

All facades work will be tested with one ticket, #6.

PR: GH-562

Also, please add comments to methods.

Also, please add comments to methods.

To which methods? All methods in the module or just the ones that I added?


Fix page layout

I experimented with various combinations of indentations in an attempt to resolve the issue, but none of them produced the desired result except for the one I'm proposing here. I copied the spacings from the "Red Hat-based" tab that displays correctly on the website.
This will fix https://github.com/kubernetes/website/issues/43431

/retitle Fix broken page layout for "Install and Set Up kubectl on Linux"

Yes, the indentations of shortcodes like note and caution may cause this error.
The preview looks good.
/lgtm


Add var to let smoke tests fail build

SMOKE_FAIL_ON_ERROR = false https://mssqltools.visualstudio.com/CrossPlatBuildScripts/_build/results?buildId=144051&view=logs&j=a5e52b91-c83f-5429-4a68-c246fc63a4f7&t=a5e52b91-c83f-5429-4a68-c246fc63a4f7
SMOKE_FAIL_ON_ERROR = true https://mssqltools.visualstudio.com/CrossPlatBuildScripts/_build/results?buildId=144050&view=logs&j=a5e52b91-c83f-5429-4a68-c246fc63a4f7&t=a5e52b91-c83f-5429-4a68-c246fc63a4f7

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.253%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14802

Relevant Lines:
29124

💛  - Coveralls


Code error

This error occurs when running the code, how to solve it?

You need to set up the directory structure as indicated.

You need to set up the directory structure as indicated.

sorry , i can't understand how to set up the directory structure.Can you tell me more specificly?

You need to have data folder under root dir, then datasets folder under data, etc. etc.


Added first and last aggregate functions

Added missing aggregate functions, first and last

Coverage increased (+0.006%) to 97.47% when pulling 3136b661ab10fc49b439cda152bc674592375398 on add_first_last_functions into 2cdce151431d69a90dff13c6d6f1e07cad0187c5 on master.


Move target_dir to Workspace and fix #2848

Target dir is now an API of Workspace. It is still initialized eagerly, just later.
I also had to errors in the config file when retrieving verbose and color config. Seems fishy, but is probably OK.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+ 886c878e97df4a19511350cab8939370df42492b
Thanks @matklad!

:hourglass: Testing commit 886c878 with merge 43d4259...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing 43d425925323d8736158214d0fb4aabec0748777 to master...


WebSocket HANDSHAKING DISCONNECT loop

I ran into this issue where WebSocket loops on the following sequence after upgrading Django, Channels, and Daphne.
WebSocket HANDSHAKING /chat/test
DEBUG - http_protocol - Upgraded connection daphne.response.YXwYgXwSZe!ZwjzgREEJB to WebSocket daphne.response.YXwYgXwSZe!wuJUpuoblB
DEBUG - ws_protocol - WebSocket closed for daphne.response.YXwYgXwSZe!wuJUpuoblB
DEBUG - worker - Got message on websocket.disconnect (reply daphne.response.YXwYgXwSZe!wuJUpuoblB)
DEBUG - worker - Dispatching message on websocket.disconnect to chat.consumers.ws_disconnect
WebSocket DISCONNECT /chat/test

Warning in Chrome:
WebSocket connection to 'ws://<IP_ADDRESS>:8000/chat/test' failed: WebSocket is closed before the connection is established.

// chat.js
var ws_scheme = window.location.protocol == "https:" ? "wss" : "ws";
var chatsock = new ReconnectingWebSocket(ws_scheme + '://' + window.location.host + "/chat" + window.location.pathname);

Django==1.11
channels==1.1.3
daphne==1.2.0

I added  '<IP_ADDRESS>' to ALLOWED_HOSTS to upgrade Django, and updated routing.py to upgrade Channels.
# routing.py
from channels.routing import route
from consumers import ws_connect, ws_receive, ws_disconnect

channel_routing = [
    route('websocket.connect', ws_connect),
    route('websocket.receive', ws_receive),
    route('websocket.disconnect', ws_disconnect),
]

Everything works fine on previous versions.

Moved this to the Channels repo: https://github.com/django/channels/issues/630


[BD-46] feat: add theme switcher to documentation site

add theme switcher to the docs site, currently two themes are included: openedx theme and edx.org theme
This PR also allows to add new theme quite easily, see README changes
after the merge only one deploy-preview will be required instead of two for each theme

JIRA: PAR-626

Thanks for the pull request, @viktorrusakov! I've created BLENDED-1069 to keep track of it in Jira.   More details are on the BD-46 project page.
When this pull request is ready, tag your edX technical lead.

@viktorrusakov Coming along nicely! The two things I think are worth addressing:

Flash of unstyled content. When flipping themes, it flashes as if no CSS were applied. Is there a moment during rendering no theme is applied? 🤔
Design of the theme selector. Space is a bit tight in the header with the select form control, especially on narrow screen sizes. I'm thinking we might draw inspiration from Material UI by adding a "Settings" icon to the header that opens a right-aligned Sheet containing the theme selector. Material UI also puts in a global language selector there, too which feels like we could also do for Paragon's i18n initiative and even a toggle between left-to-right to right-to-left text direction. Thoughts?

Something along the lines of:

Of course, the "Direction" and "Language" options would come later. I will add a new milestone for adding the ability to change the text direction for future work as well. Both frontend-build and frontend-platform have checks for right-to-left support. I'm not sure if Paragon's documentation site has any support for it currently, so I'll include a discovery ticket as well to figure out if Paragon docs site will even support RTL today.

@adamstankiewicz I do like the idea of adding a Settings component! This should be much better than having everything in the header. I'm not sure though where to put the Settings icon 🤔 , please take a look if I chose a good place!

I'm not sure though where to put the Settings icon 🤔 , please take a look if I chose a good place!

I think where you put it makes sense for now! Let's roll with it.

:tada: This PR is included in version 17.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Closes #570 - Added tree table for config properties (read-only)

Enables viewing of the plain properties in the tree table view. The schema is pulled once the user is logged-in or when the state is rehydrated. The options to show or hide the UI view for the properties is saved in redux, so it becomes a general option.
Open questions and improvements:

should we mark composite elements differently
do we need a note that we only show the plain properties in the tree
I did not implement filtering / searching / etc I think it would be too much for first commit

Looks like following:

Happy reviewing.. I suggest review first commit first, and then the second one, it's completly separate concerns..

This change is

a discussion (no related file):
Capital d in the Ocelot defaults


Add highlighting for JSDoc 3.5.0 @access package

Fixes microsoft/TypeScript-TmLanguage#986.

These grammar files are actually pulled in from the repo you linked to (https://github.com/microsoft/TypeScript-TmLanguage)
Please submit the fix against that repo and it will be picked up by VS Code

An issue in this repo was originally linked to, but then it was moved to microsoft/TypeScript-TmLanguage#986.
I actually opened a PR fixing the issue there at the same time you posted the comment! See microsoft/TypeScript-TmLanguage#987.


Missing newline on cerr before solib-event stops debugger for good

Debugging the following little program on a CentOS 5 target with MIEngine and gdb 7.6.1-47.el5 stops the debugger without any way to continue:
#include <iostream>
#include <dlfcn.h>

int main() {
    std::cerr << "Hello World";
    //std::cerr << std::endl; // fixes the issue

    if(!dlopen("libpthread.so.0", RTLD_LAZY)) {
      std::cout << "error: " << dlerror() << std::endl;
      return 1;
    }
    return 0;
}

The last message in the log reads:
Hello World*stopped,reason="solib-event",thread-id="1",stopped-threads="all",core="11"

The problem appears to be a missing newline on the cerr pipe which hides the gdb message that follows afterwards. Adding the newline fixes the issue.

To fix this, in your launch.json file, you want to enable the external console. @pieandcakes @jacdavis can you help @pborsutzki with this?

Ah I see, you mean the ExternalConsole option mentioned here? I will try that as soon as I get back to work. Thanks for the hint!

@pborsutzki Did that resolve your issue?

I did not figure out how to enable that ExternalConsole from my PipeLaunchOptions-xml configuration file. It was easier to work around the unterminated printing in my target application. The cause for this was colored output which I just disabled.
I'd still be interested in the actual solution using that ExternalConsole, so if you could give me some pointers that would be appreciated.
But beware as I am probably using a crude setup here: I currently cannot cross-platform build my target application for Linux using Visual Studio on Windows, so I am still building my project on the Linux host using make. For debugging I load the CMake generated Visual Studio Solution and start the debugger using the Debug.MIDebugLaunch command with a PipeLaunchOptions-xml file as config. I.e. I have no Cross-Platform Visual Studio Solution I could configure using setting dialogs ...


Cannot find flags.png

Same issue.

@Haapavuo you can try my nuget :) https://github.com/kevinvenclovas/Blazor.Flags


Bug Report

Environment
Windows build number: 18912
Windows Terminal version (if applicable): 0.2.1715.0

Steps to reproduce

Open the terminal in a profile that has acrylic background turn on.
Focus out of the app.
The acrylic background effect disappears.

Expected behavior

The acrylic background effect should stay even when focused out of the terminal app.
Actual behavior

The acrylic background effect disappears when focus is outside of the terminal app.

This is by design for how Acrylic works on windows. It's only allowed to be active on one toplevel window at a time. Sorry.


fix author sorting in discover

Looks like we missed one more updated index name.  This change fixes author sorting.  I also search hs_discover/views.py to ensure there were no more remaining _exact indexes within the view.

METRIC
VALUE

NEW DUPLICATED LINES DENSITY
0.0

NEW BUGS
0

NEW SECURITY HOTSPOTS
0

NEW VULNERABILITIES
0

NEW CODE SMELLS
0

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4213


tuhi-booklet-vuw:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: This is a template to produce course description booklets from JSON (or similar) data, made for VUW but suitable for other institutions.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does. Note the 3 previous packages with the same naming convention
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use. Used dummy content, and the photo is mine.

Thanks :)


Cannot redeclare class CreateApiKeysTable

I just started PHP development with Laravel 5 again, so I hope this isn't some kind of beginners mistake.
I installed api-guard as it is described in the instructions. I used to reset the database and add Test data with artisan migrate:refresh --seed. But after installing api-guard it fails with the message:
[Symfony\Component\Debug\Exception\FatalErrorException]
Cannot redeclare class CreateApiKeysTable

However, using artisan migrate and artisan db:seed seperated works.
Is this an actual issue or just my mistake?

Does this still happen? I can't seem to figure out how that happened. What version are you using anyway?

I'm using the 2.2 and the up-to-data Laravel 5. It does still happen:
Doing
artisan migrate
artisan db:seed
works.
But artisan migrate:refresh outputs these two errors:
[Illuminate\Database\QueryException]
SQLSTATE[42000]: Syntax error or access violation: 1091 Can't DROP 'api_keys_user_i
d_foreign'; check that column/key exists (SQL: alter table `api_keys` drop foreign
key api_keys_user_id_foreign)

[PDOException]
SQLSTATE[42000]: Syntax error or access violation: 1091 Can't DROP 'api_keys_user_i
d_foreign'; check that column/key exists

This can be resolved by removing the "database" path from the classmap section in the composer file. I don't think there is any use for it in there as it should only ever be called directly from the app.
"autoload": {
    "psr-4": {
        "Chrisbjr\\ApiGuard\\": "src"
    }
},

nice catch. thanks @dbonner1987. #60 merged.


Suggestion: Make cubemaps get darker if the lighting is darker

i think this would enhance the graphical quality of cubemaps a lot and would prevent things like this where cubemap is very bright even in shadow

I would need to check up on how other video games do this, cause off the top of my head I am not sure of a way to do this that properly accounts for darkness.

I would need to check up on how other video games do this, cause off the top of my head I am not sure of a way to do this that properly accounts for darkness.

source engine does do it somehow according to vdc but im not exactly sure how, but it seems to be related to lightmapping, so in pathos it could probably be done based on lightmap and then maybe somehow acount for dlights?? im not sure


Error when run docker/Dockerfile

ERROR: failed to solve: process "/bin/sh -c pip install -e ." did not complete successfully: exit code: 1

@Cguanqin, could you please provide more information for this error?

the model variant
hardware platform
command you used

Thanks

Hi @Cguanqin,
Could you please confirm if this issue is resolved for you with the above comment ? Please feel free to close the issue if it is resolved ?
Thank you.


[Fix] In require-optimization corrected test for function declaration

There was a problem with the ad functions and methods in classes, sample code -
import React from "react";
class YourComponent extends React.Component {
  handleClick() {}
  render() {
     return
123
;
  }
}

{
  "rules": {
    "react/require-optimization": 2
  }
}

And this code passed checks, it is wrong

The example in your OP should indeed pass the "require-optimization" check - because it has a lifecycle method. That the method is empty is irrelevant; adding the empty method clearly indicates developer intent to flesh it out later, and as such, they should not be required to make it an SFC.

@ljharb In the code, there is no such logic. A similar operation is due to not a correct execution "FunctionExpression"

In your OP you say "And this code passed checks, it is wrong" - if the code passes checks, what's the bug?

@ljharb Thanks, corrected the wording

OK so the code in your example:
import React from "react";
class YourComponent extends React.Component {
  handleClick() {}
  render() {
     return
123
;
  }
}

has a class method, so it should indeed NOT output a require-optimization error (unless the rule is smart enough to recognize that the method doesn't use this or super, and isn't a lifecycle method - but I don't think that's the case). Are you saying you do get one?

@jkimbo For whatever reason, in your example should not be a mistake? require-optimization should be checked for statelesFunction, some mixins and shouldComponentUpdate. For all other cases, the error should be.
I turned on this rule on my big project, 5000+ react components and has received 0 such errors, although there are a lot of components, without optimization)

Sorry for my earlier confusion - I was confusing this with a "prefer SFCs" rule :-)
Afaik require-optimization should skip any createClass component with PureRenderMixin, and should skip any class component with a shouldComponentUpdate method, and should skip any SFC entirely.
If you have components that don't match these criteria, and require-optimization is not warning on them, then that seems like it might indeed be a bug.

It should also skip classes that extend React.PureComponent.

@lencioni Yes, I agree, but it has already implemented @yannickcr
@ljharb Yes, it is =)

Merged, thanks!


Fix maxLength issue on pasting text

Checking the pasted text length with maxLenght props and if pasted text is greater than the maxLenght props the pastedText is substr with maxLenght.
fixes https://github.com/prakhar1989/react-tags/issues/422

@imtanmoy also can you rebase your branch with latest master and resolve the conflicts

Thanks @ad1992 for helping me out.

@imtanmoy not sure how you rebased but in diff it shows more commits and files than what you  actually committed. those commits have already merged in master . can you fix that

ping @imtanmoy

@imtanmoy  can you do the following steps
git checkout master
git fetch upstream && git merge upstream/master
git checkout maxLength-paste-fix
git rebase -i master
remove other commits and only keep your commits
continue with rebase
force push

Hi @ad1992 , any update?

@imtanmoy any update ?

:tada: This PR is included in version 6.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Installation | pivccu-modules-dkms | No headers found

Describe the issue you are experiencing
I'm trying to install RaspberryMatic on my RPi4 to use it with Home Assistant on the same device.
I did all the installation steps, but installing/usage of pivccu-modules-dkms is not possible due failure message "No headers found".
Afterwards I tried "sudo dpkg-reconfigure pivccu-modules-dkms"
Disabled DKMS service ... Done
Ensure non mixed processor architecure ... Done
Create kernel modules ... FAILED
Check kernel headers ... FAILED
No headers found for current active kernel 6.6.56-v8+
Enable DKMS service ... Done

But from my point of view everything should be correct.
Logs from raspberrypi-kernel and raspberrypi-kernel-headers are posted in log-section.
Describe the behavior you expected
Installation should be suceeded.
Steps to reproduce the issue

Do all installation steps as described in the wiki Installation HomeAssistant -> HomeAssistant Supervised

What is the version this bug report is based on?
Not possible until now
Which base platform are you running?
rpi4 (RaspberryPi4, ARM64/aarch64)
Which HomeMatic/homematicIP radio module are you using?
HM-MOD-RPI-PCB
Anything in the logs that might be useful for us?
apt show raspberrypi-kernel -->

Package: raspberrypi-kernel
Version: 1:1.20230405-1
Priority: optional
Section: kernel
Source: raspberrypi-firmware
Maintainer: Serge Schneider<EMAIL_ADDRESS>Installed-Size: 32,1 MB
Provides: linux-image, wireguard-modules (= 1.0.0)
Breaks: raspberrypi-bootloader (<< 1.20160324-1)
Replaces: raspberrypi-bootloader (<< 1.20160324-1)
Homepage: https://github.com/raspberrypi/firmware
Download-Size: 27,0 MB
APT-Manual-Installed: yes
APT-Sources: http://archive.raspberrypi.com/debian bookworm/main arm64 Packages
Description: Raspberry Pi bootloader
 This package contains the Raspberry Pi Linux kernel.

apt show raspberrypi-kernel-headers -->
Package: raspberrypi-kernel-headers
Version: 1:1.20230405-1
Priority: optional
Section: kernel
Source: raspberrypi-firmware
Maintainer: Serge Schneider<EMAIL_ADDRESS>Installed-Size: 64,8 MB
Provides: linux-headers
Depends: libc6 (>= 2.17)
Homepage: https://github.com/raspberrypi/firmware
Download-Size: 10,2 MB
APT-Manual-Installed: yes
APT-Sources: http://archive.raspberrypi.com/debian bookworm/main arm64 Packages
Description: Header files for the Raspberry Pi Linux kernel
 This package provides the architecture-specific kernel header files for
 the Raspberry Pi Linux kernel, generally used for building out-of-tree
 kernel modules.

### Additional information

_No response_

Hey Dominik,
whilst this might not be of immediate help, I suggest you raise this issue at the https://github.com/alexreinert/piVCCU project as this is at least to origin of your request.
Having said that, it might be required that you have to downgrade your kernel version, as whilst the raspiberry-kernel-header package is installed, the installer cannot properly detect it. However for any of these instruction the pivccu project and Alex over there might be able to give you better advise so that you get hopefully forward.