Add definition for MetadataCache.getBacklinksForFile

Fixes #58

Thanks, but these definitions are automatically generated from the list of APIs we internally mark as public, and this specific function is not marked for public use. Again, you are free to use them using a variety of typescript tricks in your own codebase.


Laravel 7.0.8 bug in get query get params

Laravel Version: 7.0.8
PHP Version: 7.3.24
Database Driver & Version:

Description:
Hello, how is everything?
I was using the new HTTPCLIENT class in laravel 7, however I already tried in the get function to put query params in several ways, and none of them successfully.
Follows images of prints, the code always send without query params.
Steps To Reproduce:

URL:

Laravel 7 isn't supported anymore => https://laravel.com/docs/7.x/releases#support-policy

Laravel 7 isn't supported anymore => https://laravel.com/docs/7.x/releases#support-policy

Sorry for my ignorance, but do you know where I can get help for this function?


error

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited
setup(self)
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Traceback (most recent call last):
File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in
asyncio.run(load_cogs())
File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run
return loop.run_until_complete(main)
File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete
return future.result()
File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs
await client.load_extension("cogs.startup")
TypeError: object NoneType can't be used in 'await' expression

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This was an issue with the requirements.txt file, I will update the installations steps.

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This error has been fixed, please try to install using the executable that has now been provided and installation instructions for which are provided in the readme.

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This error has been fixed, please try to install using the executable that has now been provided and installation instructions for which are provided in the readme.

Traceback (most recent call last):
File "C:\Users\ADMIN\Downloads\PokeGrinder-master (1)\PokeGrinder-master\main.py", line 20, in
client.run(client.token)
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 742, in run
asyncio.run(runner())
File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run
return loop.run_until_complete(main)
File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete
return future.result()
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 739, in runner
await self.start(*args, **kwargs)
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 712, in start
await self.login(token)
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 583, in login
data = await state.http.static_login(token.strip())
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 660, in static_login
await self.startup()
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 378, in startup
self.user_agent, self.browser_version, self.client_build_number = ua, bv, bn = await utils._get_info(session)
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1246, in _get_info
bn = await _get_build_number(session)
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1255, in _get_build_number
login_page = await login_page_request.text()
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1076, in text
await self.read()
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1032, in read
self._body = await self.content.read()
File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\streams.py", line 344, in read
raise self._exception
aiohttp.client_exceptions.ClientPayloadError: 400, message='Can not decode content-encoding: br'
new error :(

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This error has been fixed, please try to install using the executable that has now been provided and installation instructions for which are provided in the readme.

Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master (1)\PokeGrinder-master\main.py", line 20, in client.run(client.token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 742, in run asyncio.run(runner()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 739, in runner await self.start(*args, **kwargs) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 712, in start await self.login(token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 583, in login data = await state.http.static_login(token.strip()) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 660, in static_login await self.startup() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 378, in startup self.user_agent, self.browser_version, self.client_build_number = ua, bv, bn = await utils._get_info(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1246, in _get_info bn = await _get_build_number(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1255, in _get_build_number login_page = await login_page_request.text() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1076, in text await self.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1032, in read self._body = await self.content.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\streams.py", line 344, in read raise self._exception aiohttp.client_exceptions.ClientPayloadError: 400, message='Can not decode content-encoding: br'
new error :(

https://github.com/MehulKhanna/PokeGrinder/releases/download/Full-Release/PokeGrinderExecutable.zip please try to use the executable given in this file

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This error has been fixed, please try to install using the executable that has now been provided and installation instructions for which are provided in the readme.

Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master (1)\PokeGrinder-master\main.py", line 20, in client.run(client.token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 742, in run asyncio.run(runner()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 739, in runner await self.start(*args, **kwargs) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 712, in start await self.login(token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 583, in login data = await state.http.static_login(token.strip()) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 660, in static_login await self.startup() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 378, in startup self.user_agent, self.browser_version, self.client_build_number = ua, bv, bn = await utils._get_info(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1246, in _get_info bn = await _get_build_number(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1255, in _get_build_number login_page = await login_page_request.text() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1076, in text await self.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1032, in read self._body = await self.content.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\streams.py", line 344, in read raise self._exception aiohttp.client_exceptions.ClientPayloadError: 400, message='Can not decode content-encoding: br'
new error :(

https://github.com/MehulKhanna/PokeGrinder/releases/download/Full-Release/PokeGrinderExecutable.zip please try to use the executable given in this file

This version works fine, but when the captcha appears, there is an error
| Time Elapsed: 0:00:02 | Encounters: 1 | Catches: 1
| Time Elapsed: 0:00:14 | Encounters: 2 | Catches: 2
Task exception was never retrieved
future: <Task finished name='Task-25' coro=<timer() done, defined at C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py:19> exception=TimeoutError()>
Traceback (most recent call last):
File "asyncio\tasks.py", line 456, in wait_for
asyncio.exceptions.CancelledError
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py", line 21, in timer
await command()
File "discord\commands.py", line 615, in call
File "discord\commands.py", line 261, in call
File "discord\commands.py", line 128, in call
File "asyncio\tasks.py", line 458, in wait_for
asyncio.exceptions.TimeoutError

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This error has been fixed, please try to install using the executable that has now been provided and installation instructions for which are provided in the readme.

Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master (1)\PokeGrinder-master\main.py", line 20, in client.run(client.token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 742, in run asyncio.run(runner()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 739, in runner await self.start(*args, **kwargs) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 712, in start await self.login(token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 583, in login data = await state.http.static_login(token.strip()) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 660, in static_login await self.startup() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 378, in startup self.user_agent, self.browser_version, self.client_build_number = ua, bv, bn = await utils._get_info(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1246, in _get_info bn = await _get_build_number(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1255, in _get_build_number login_page = await login_page_request.text() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1076, in text await self.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1032, in read self._body = await self.content.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\streams.py", line 344, in read raise self._exception aiohttp.client_exceptions.ClientPayloadError: 400, message='Can not decode content-encoding: br'
new error :(

https://github.com/MehulKhanna/PokeGrinder/releases/download/Full-Release/PokeGrinderExecutable.zip please try to use the executable given in this file

This version works fine, but when the captcha appears, there is an error | Time Elapsed: 0:00:02 | Encounters: 1 | Catches: 1 | Time Elapsed: 0:00:14 | Encounters: 2 | Catches: 2 Task exception was never retrieved future: <Task finished name='Task-25' coro=<timer() done, defined at C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py:19> exception=TimeoutError()> Traceback (most recent call last): File "asyncio\tasks.py", line 456, in wait_for asyncio.exceptions.CancelledError
The above exception was the direct cause of the following exception:
Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py", line 21, in timer await command() File "discord\commands.py", line 615, in call File "discord\commands.py", line 261, in call File "discord\commands.py", line 128, in call File "asyncio\tasks.py", line 458, in wait_for asyncio.exceptions.TimeoutError

Can you please check and see if this happens with every captcha you encounter?

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This error has been fixed, please try to install using the executable that has now been provided and installation instructions for which are provided in the readme.

Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master (1)\PokeGrinder-master\main.py", line 20, in client.run(client.token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 742, in run asyncio.run(runner()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 739, in runner await self.start(*args, **kwargs) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 712, in start await self.login(token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 583, in login data = await state.http.static_login(token.strip()) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 660, in static_login await self.startup() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 378, in startup self.user_agent, self.browser_version, self.client_build_number = ua, bv, bn = await utils._get_info(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1246, in _get_info bn = await _get_build_number(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1255, in _get_build_number login_page = await login_page_request.text() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1076, in text await self.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1032, in read self._body = await self.content.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\streams.py", line 344, in read raise self._exception aiohttp.client_exceptions.ClientPayloadError: 400, message='Can not decode content-encoding: br'
new error :(

https://github.com/MehulKhanna/PokeGrinder/releases/download/Full-Release/PokeGrinderExecutable.zip please try to use the executable given in this file

This version works fine, but when the captcha appears, there is an error | Time Elapsed: 0:00:02 | Encounters: 1 | Catches: 1 | Time Elapsed: 0:00:14 | Encounters: 2 | Catches: 2 Task exception was never retrieved future: <Task finished name='Task-25' coro=<timer() done, defined at C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py:19> exception=TimeoutError()> Traceback (most recent call last): File "asyncio\tasks.py", line 456, in wait_for asyncio.exceptions.CancelledError
The above exception was the direct cause of the following exception:
Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py", line 21, in timer await command() File "discord\commands.py", line 615, in call File "discord\commands.py", line 261, in call File "discord\commands.py", line 128, in call File "asyncio\tasks.py", line 458, in wait_for asyncio.exceptions.TimeoutError

Can you please check and see if this happens with every captcha you encounter?
yes, when the captcha appears, the activity stops and nothing happens after that

C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\ext\commands\bot.py:618: RuntimeWarning: coroutine 'setup' was never awaited setup(self) RuntimeWarning: Enable tracemalloc to get the object allocation traceback Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 18, in asyncio.run(load_cogs()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\Downloads\PokeGrinder-master\PokeGrinder-master\main.py", line 14, in load_cogs await client.load_extension("cogs.startup") TypeError: object NoneType can't be used in 'await' expression

This error has been fixed, please try to install using the executable that has now been provided and installation instructions for which are provided in the readme.

Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinder-master (1)\PokeGrinder-master\main.py", line 20, in client.run(client.token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 742, in run asyncio.run(runner()) File "C:\Program Files\Python310\lib\asyncio\runners.py", line 44, in run return loop.run_until_complete(main) File "C:\Program Files\Python310\lib\asyncio\base_events.py", line 646, in run_until_complete return future.result() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 739, in runner await self.start(*args, **kwargs) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 712, in start await self.login(token) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\client.py", line 583, in login data = await state.http.static_login(token.strip()) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 660, in static_login await self.startup() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\http.py", line 378, in startup self.user_agent, self.browser_version, self.client_build_number = ua, bv, bn = await utils._get_info(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1246, in _get_info bn = await _get_build_number(session) File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\discord\utils.py", line 1255, in _get_build_number login_page = await login_page_request.text() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1076, in text await self.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\client_reqrep.py", line 1032, in read self._body = await self.content.read() File "C:\Users\ADMIN\AppData\Roaming\Python\Python310\site-packages\aiohttp\streams.py", line 344, in read raise self._exception aiohttp.client_exceptions.ClientPayloadError: 400, message='Can not decode content-encoding: br'
new error :(

https://github.com/MehulKhanna/PokeGrinder/releases/download/Full-Release/PokeGrinderExecutable.zip please try to use the executable given in this file

This version works fine, but when the captcha appears, there is an error | Time Elapsed: 0:00:02 | Encounters: 1 | Catches: 1 | Time Elapsed: 0:00:14 | Encounters: 2 | Catches: 2 Task exception was never retrieved future: <Task finished name='Task-25' coro=<timer() done, defined at C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py:19> exception=TimeoutError()> Traceback (most recent call last): File "asyncio\tasks.py", line 456, in wait_for asyncio.exceptions.CancelledError
The above exception was the direct cause of the following exception:
Traceback (most recent call last): File "C:\Users\ADMIN\Downloads\PokeGrinderExecutable\cogs\hunting.py", line 21, in timer await command() File "discord\commands.py", line 615, in call File "discord\commands.py", line 261, in call File "discord\commands.py", line 128, in call File "asyncio\tasks.py", line 458, in wait_for asyncio.exceptions.TimeoutError

Can you please check and see if this happens with every captcha you encounter?
yes, when the captcha appears, the activity stops and nothing happens after that

I know what could've caused this error but I am unable to reproduce this error myself. Did you make any changes to any files except config.json in the PokeGrinderExecutable folder? Can you please share the image of the captcha message?

Is there a faster way to contact you? telegram or facebook?

Is there a faster way to contact you? telegram or facebook?

https://t.me/PokeGrinder

i can't chat

 i can't chat

Now you can


VLAZ: Watch cepat.click

VLAZ requests the watch of the watch_keyword cepat\.click. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

cepat\.click has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Why that error?

My file in correct folder, why that error?
https://github.com/Hacktoberfest-2018/Hello-world/pull/4573

DontClose.cs is not in C# folder


CODEOWNERS: Include .github directory

First commit renames smoke test workflow file. Second commit adds .github/ to CODEOWNERS.

Coverage decreased (-0.006%) to 37.074% when pulling 3b6f9a6c2686fe4894934351cc3732d1b126975b on pr/pchaigno/github-dir-in-codeowners into 819185568eb5cf653052043c5299d1224460368b on master.


loading custom fonts does not seem to work

Environment

react-native -v:
npm ls react-native-macos<EMAIL_ADDRESS>node -v: v10.16.0
npm -v: 6.14.5
yarn --version: 1.16.0
xcodebuild -version: Xcode 11.5 Build version 11E608c

i have followed every step in this tutorial  link here among many on the internet but it seems react-native-macos does not load fonts
also followed tutorial here : https://github.com/oblador/react-native-vector-icons#osx-via-react-native-desktop

I haven't been able to load fonts either. That includes with expo and rnvi.

@shayneoneill I "solved" it by simply making my app with vanilla RN and setting the macOS as a target platform. As for fonts, I use expo-fonts, which you can see a quick example of in my starter-kit/template here: https://gitlab.com/NoahGray/react-once-template/-/blob/main/template/src/App.js#L120. You can give the template a shot with with npx create-react-once-app myapp as described here: https://www.npmjs.com/package/create-react-once-app


missing useNuxtApp import in components

Environment

Operating System: Windows_NT
Node Version:     v18.16.1
Nuxt Version:     3.6.2
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-h3vzyp?file=nuxt.config.ts,app.vue
Describe the bug
There's some imports missing in components
Additional context
No response
Logs
No response

:tada: This issue has been resolved in version 6.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Wow, you just saved my day. Thank you for this release!


Add "fixed" mesos_master.

What's this PR do?
Adds a fixed version of the mesos_master integration, as a temporary thing until this PR to Datadog is released.
Motivation
We're dropping about 40 metrics that would be nice to have in Mesos. Patching this now lets us proceed with OBS-14.
Notes
I experimented locally and it seems that our /etc/dd-agent/checks.d is higher priority and having this integration present there overrides the one in Datadog's package. I will verify this in QA!
How To Deploy

Merge
Separately create a branch with a new SHA for stripe_datadog_agent::oss_integrations::sha1 and deploy to Mesos hosts, which will happen in another PR.

cc @tummychow @rhwlo @antifuchs

Looks great to me - if you want to make really super sure that our patched version is picked, you could name it stripe_mesos_master.py and rename the config file (:


versioned binaries?

Can you start publishing versioned binaries at https://github.com/cloudbase/cloudbase-init/releases?

Currently, the MSI installers are published in a stable/nightly build format:
Stable
(64bit) https://www.cloudbase.it/downloads/CloudbaseInitSetup_Stable_x64.msi
(32bit) https://www.cloudbase.it/downloads/CloudbaseInitSetup_Stable_x86.msi
Beta
(64bit) https://www.cloudbase.it/downloads/CloudbaseInitSetup_x64.msi
(32bit) https://www.cloudbase.it/downloads/CloudbaseInitSetup_x86.msi
https://github.com/cloudbase/cloudbase-init#downloads

Indeed, those are clearly mentioned on the documentation, but that's not what I was really asking for :-)
From a user standpoint, it would be easier (or normal for a project hosted in github) to have the artifacts in this project release page. The release page is more explicit about the binaries as they will have a dedicated page per release/version and we can easily download any version from a stable / versioned URL.

As this is Python project, you can run the code directly if you have a Python environment.
I understand what you are asking, only that the process of creating the MSI installer is not that straightforward. I suppose the GitHub actions environment can allow such a build in the future, at least for the newer releases.
This is the installer code, in case you ever need to build your own installer: https://github.com/cloudbase/cloudbase-init-installer

Hi @rgl, added them for 0.9.11 and we'll keep doing so moving forward. Thanks for bringing this up!


Kaoss-pad-like UI for tweening between and around prompts.

Sometimes the image you want is between two or more prompts. Or perhaps you just want to go for a wander in feature space without having to deal with language.
This is a feature request for a Kaoss-pad-like interface for exploring feature space. You first generate a collection images using different prompts but the same seed, CFG scale, size, and sampler.
You can drag the dot to move it towards or away from the different prompts, triggering image generations for the point where the dot currently is, taking blend weights from the distance between the different images. Move the dot on top of image A and you get image A; move it halfway between image A and image B and you get a 50:50 blend of prompt A and prompt B.
Advanced:

You can also move the dot beyond the plane/line defined by the tween to explore the latent space on either side and outside of it.
You can zoom in to move about in finer increments through a smaller region of the feature space.
Probably the advanced stuff would require #993.

I'm having trouble imagining how to do this with 4 images... wouldn't that need a 4D visualization?

Depends on how the math is done. actually you might be right, perhaps best to restrict it to 3 - or just accept that it won't be possible to blend between the two points opposite each other on the plane.

Hi all, new to this project and subscribed and this looked interesting. Is it well-defined how to created a weighted image between an arbitrary number of prompts? If so, you can create an N dimensional space based on N+1 unique prompts. Another idea is to consider the base prompt the origin, and all other N prompts define a space using img2img where a 1 gets you the prompt at the end of the dimension and 0 gets you the origin. Order would matter, i.e. the first dimension is resolved first before moving along the second dimension.


Restructure of /Includes/

Hi 👋
this is draft about if and how the code could be rearranged to be more readable and to improve maintainability.
I tried to orient myself with the filenames as well as their includes. How ever it wasn't straightforward.

/Includes/net is supposed to house network related code
/Includes/ftp holds the ftp server, it could also be placed in net/
/Includes/menu if filled with code for drawing and operating menus
/Includes/render contains all files which aim to help drawing stuff on the screen
/Includes/sys should contain code that provides lower functionality and basic setups

This is a draft, it's not working code. It is intended to provide a starting point for a discussion.

Good stuff, thanks for looking into this!
Problem runs (at least) one level deeper though IMO - sources should not reside inside "Includes" but rather a folder named "Sources" or similar.
"Includes" should probably only contain the header files suitable for including in main.cpp, the rest should be "private" and live inside "Sources" along with the *.cpp/*.c files. The concept of splitting into subfolders is sound/good - just take it one step further and I'll be happy as a clam.


No support for basic diacritics

Hello, I'm using your app to create a font, but found it useful only in English, since it won't display basic diacritics.
A quick solution I've found is changing this function:
    private string[] GenerateChars()
    {
        var minChar = CharHelper.ConvertToChar(MinChar);
        var maxChar = CharHelper.ConvertToChar(MaxChar);
        return CharHelper.GetChars(minChar, maxChar);
    }

To use 255 instead of maxChar.
Of course it's sub-optimal since it prints many non-printable characters, and if a glyph is not found in the current font uses the previous selected font.
Would you consider support for non-english glyphs?

Hi @axelei. You can specify the MaxChar value right in the app, there's a text field for that. Or maybe I'm not following you.


Fix DeleteTrace (fd leak and go-routine leak)

Fix DeleteTrace (fd leak and go-routine leak)
Gadgets' DeleteTrace() functions were never called. This can cause fd leaks and go routine leaks.
This patch fixes the issue.
How to use
No changes for users.
Testing done
TODO:

Use gadgets such as dns, network-policy, seccomp or biolatency which have a DeleteTrace function
Delete the Trace custom resource without stopping it
Check the file descriptor list of thegadgettracermanager process (/proc/$pid/fd): the bpf program and maps for the gadget should have been removed
Check the go routine list of thegadgettracermanager process (gadgettracermanager -dump): go routines for gadgets should not be present anymore

Unit tests were added in the local-gadget PR.

Patches squashed and rebased.


Fix newlines consistency, list item trims and overriding list rendering

Hello again,
Following things were fixed, and that could mean this is a major semver change:

Block sections (paragraph, code, blockquote, list, headings) would sometimes have different vertical padding (new lines)
There was a space before list items
Missing distinction between rendering lists and list items, overridable through options

listitem is still just a chalk reset
list (new option) takes the function (body, ordered) signature. it is responsible for attaching points onto list item

I've introduced a weird padding behaviour; there will always be a newline at the start of the output.
I needed to introduce it to make consistent vertical padding AND make sure that nested lists work. I don't really want it there, but couldn't think of a way to get rid of it.
I'll need you to weigh in with this, concerning the newline-at-the-start-of-output and also how these changes may require a major semver change due to how old expected behaviours have changed.

Also #hacktoberfest because I want that free shirt, Mmmmm!

Oh. How have I not seen that hacktoberfest is an actual organized thing before now. I've seen it mentioned several placed, but never looked into it. August/September/October are traditionally very hectic times for me, so probably for the better I didn't get another thing to try to achieve 😛

Dat free shirt tho 😻

Awesome work @loklaan! I tried some of my presentation md files and noticed sublists acting a bit odd:
# test

hello there

1. one
2. two
 - bar
3. three

appears as
# test

hello there

    1. one
    2. two
    3.     * bar
    4. three

on master it renders as:
# test
hello there

    1. one
    2. two
       3. bar
    4. three

Which also seems incorrect. I'd expect it to render just the input (an ordered list with an unordered sublist).

Great catch @noffle! Fixed and added tests.

Back from a conference! I'll remind myself to check this out tonight.

So, looking into this, the prefixed newline for it is due to all sections starting with \n instead of seperating different sections to have newline in the end? Before, things as headers and paragrafs didn't start with a newline. What do we get by doing this, other than convenient coding?
I'm very skeptical of the leading whitespace, I must say.

I'd prefer to have a double newline after the sections, but then nested lists would break. They require a newline before their individual outputs as to avoid rendering into the same line between list nest levels.
Because there isn't (correct me if I am wrong) a way for the blocks to know about the previously rendered block, achieving consistency means accommodating the lowest denominator (nested list support) with a single approach.

OK :clap:
COOL :clap:
DONE :clap:
Leading newline has been removed. I think the PR is ready ready.

Great work! Looking good. 👏 🎉

Since the output looks different, in terms of newlines/whitespace, should this be a major or minor?

Really unsure about this, if the output is a part of the API or not, but I guess there's no reason not to do a major release.

There's also breaking changes in the listitem behaviour.

New version out: 2.0.0


Needinfo regression author when "regressed by" field is set on new bugs

Fixes #1247
Still left to do: don't needinfo if the regressor author already interacted with the bug (by commenting or doing some other action in the bug).

@calixteman can you take a look and tell me if you see some other case where we should avoid needinfoing?

What you said is correct and maybe the same for the manager of the author and the triage owner if no manager.

I've added the skipping of bugs the regressor author already interacted with.
Here's the list of bugs from a dry run:

Bug
Summary

1754850
Intermittent TV image/test/mochitest/test_image_cache_notification.html | second load of over.png should be cached

1754592
With HCM enabled and Firefox's default theme, primary buttons' focus rings are invisible

1754455
Intermittent TVw-fis TinderboxPrint: Per-test run of .../opaque-origin.html: FAILURE

1754138
Intermittent [TVw] Return code: 1 after focus-method-delegatesFocus-nested-browsing-context.html

1754074
[vaapi] Video decode attempts to use incorrect EGL display

1753849
Intermittent browser/components/places/tests/browser/browser_controller_onDrop_query.js | Test timed out -

1753829
Intermittent [TVw] TinderboxPrint: Per-test run of .../first-letter-punctuation-317.xht: FAILURE

1753828
Intermittent [TVw] TinderboxPrint: Per-test run of .../margin-applies-to-006.xht: FAILURE

1753607
Creating a tab with the openerTabId of the discarded tab resulted in an unexpected error

1753525
Hand-off Dock icon bounces in and out frequently during browsing on other macOS machine

1753477
Intermittent TV  browser/base/content/test/siteIdentity/browser_bug906190.js | Uncaught  exception received from previously timed out test - ViewShown listener  on #identity-popup not removed before the end of test

1753472
Intermittent [TVw] ::: Test verification FAIL |  TinderboxPrint: Per-test run of  .../streams/queuing-strategies.any.js: FAILURE

1753458
Intermittent [TV] browser/components/preferences/tests/browser_moreFromMozilla.js | Test timed out -

Interesting to see so many "Intermittent...".
Could it mean that people pay less attention to those kind of issues ?

What about checking if author's manager changed something in the bug ?

Let's first see if we get complaints, it should be a rare enough occurrence anyway


add "group library by: original language" option

Describe your suggested feature
add group library by original laguage option so it can separate between manga, manhwa, and manhua in library
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to the newest version Latest.
[X] I have checked through the app settings for my feature.
[X] I will fill out all of the requested information in this form.

2.11.8


MH-13347, Don't always look for orphaned properties

For every delete query executed by the asset manager and regardless of
what is being deleted, the asset manager will look for any orphaned
asset manager properties (defined as properties for media packages for
which no snapshot exists) by joining the complete snapshot table with
the complete property table which is a quite expensive operation.
This patch will remove this procedure and instead add a simple,
performant cleanup procedure for a single media package before the first
snapshot is being created.
Note that this uses the fast property deletion of pull request #694

This patch will remove this procedure and instead add a simple,
performant cleanup procedure for a single media package before the first
snapshot is being created.

AFAIK, the term "orphaned properties" refers to properties (oc_assets_properties) which refer to a media package that does not exist anymore because no snapshots for this media package exist anymore.
Doesn't seem that "...before the first snapshot is being created" is the right time to handle properties where no snapshots for a given media package exist anymore...
... How are those orphaned properties handled with this patch?

The only time that this might become problematic is when these data affect new recordings which might happen if you delete and then re-ingest recordings. Additionally to fixing the performance issues, this patch will assure that there are no leftover properties for new events which are being added.

@lkiesow But how are the orphan properties handled?
Example:
TakeSnapshotOf(MediaPackageA) -> Creates Version 0
TakeSnapshotOf(MediaPackageA) -> Creates Version 1
TakeSnapshotOf(MediaPackageA) -> Creates Version 2
AddPropertiesTo(MediaPackageA)
DeleteSnapshotVersion(0 and 1) of MediaPackageA
DeleteSnapshotVersion(2) of MediaPackageA (no snapshots anymore)
... but now the properties of Media Package A are still there (hence "orphaned"), aren't they?

Yes, they are still there but that does hurt far less than joining this table every time and you can easily clean up the table anytime you want to.

Are there plans to address @staubesv's comment long term?  I don't like the idea of accumulating cruft.  Also, conflicts now that I've merged #696.

Fixed conflict

deleteOrphanedProperties() is a non-starter for any production system, so the goal for 6.x is to make it perform well and correctly.
Overall, this seems a crazy problem to have, and hopefully we can refactor somehow for a better solution in Opencast 7.
There is also the outstanding issue of not being able to delete an event completely as a non-admin user in 6.x which needs work (but beyond the scope of this PR).
So I support merging this PR as-is because it is part of the fix for critical performance problems (and are also running this change in our 6.x production system).

To give a perspective on performance: On my test machine, I could usually run any select query on oc_assets_properties in under 100ms while having > 8 000 000 properties in the table (for comparison: SWITCH production has ~1.3M). When I tested the clean-up from #718, it took 3 min 12.553 sec to run the query and I found the query used to be faster than the join which was generated by deleteOrphanedProperties().

Merging this because it resolves a blocker issue in 6.x, and there is cleanup code in the 6-to-7 db conversion to remove the orphaned property records.
If there's a cleaner solution, it can be applied to develop (or even a subsequent 6.x PR), but 6.x needs to move forward in a performant state.


Ignore class properties in destructuring-assignment

Fixes #1875
Please let me know if you would like me to add more tests or if anything can be improved

That's one of the points of the rule - if you need to rely on props values, it's supposed to force it to go in the constructor.

Okay I get it!
How would you like the option to be implemented? Not sure that I see a clear way to do it well without a breaking change. I guess we could add another enum option like ignoreClassProperties but I feel that it would maybe be a bit ambiguous?

allowInClassFields seems reasonable to me, or allow: ["class-fields"] (where that's the only allowed value atm)

Okay so I've now added the option, I went for another name because I thought it was more logical than using the "allow" word, but let me know if you want me to change it. I also a bit unsure if I wrote the schema correctly
So you would do this;
["error", "always", { "ignoreClassFields": true }]

@ljharb should this be behind a flag because it doesn't have to do anything with destructuring assignments? I would agree that it would be a great rule to disallow props usage inside class properties but doing it here might be a little bit confusing for people.

@HenriBeck the rule is to force, or forbid, destructuring assignment regardless of location inside a class body. That class fields make destructuring awkward might be an inconvenience to some, hence the need for an option - but personally, I like the side effect that if you're trying to do anything complex in a class field, it forces you to properly put it in a constructor.

@ljharb I agree with you that doing complicated things inside class properties isn't the best idea.
But forbidding it in this rule seems odd because class properties aren't about destructuring and should be able to disallow them whether or not you have this rule configured.

Methods aren't about destructuring either - the rule deals with any references to state, props, or context, no matter where they appear.

Well shit, this does not work when assigning an object with properties containing this.props to a class field.
class Foo extends React.Component {
  state = {
    // this will still throw an error
    bar: this.props.bar
  }
}

Did a quick try to fix this but didn't make any progress, can somebody either try to solve this or point me in the right direction? @ljharb

That code you posted isn't valid JS. You cannot use this.props in a class property definition. You'd have to rewrite this to:
class Foo extends React.Component {
  constructor (props) {
    super(props);

    this.state = {
      // this will still throw an error
      bar: props.bar
    }
  }
}

Yup 7.11.1, the below still warns me about de-structuring props assignment:
const sampleSchemaData = this.props.schemaData;

@anna93 that’s a variable, not a class field.

This still throws one for me using the same setup as @anna93:
class Input extends React.Component {
  id = `${this.props.name}-${generateUUID()}`;
 ...

@zslabs if with the option enabled you still get a warning there, please file a new issue


lost date formatting/handling

dates are coerced to numeric now

[ ] POSIXct
[ ] Date
[ ] test for POSIXct
[ ] test for Date

some_dates <- seq(as.POSIXct("2010-10-01"), as.POSIXct("2011-09-30"), by="month")
gs <- gsplot() %>%
  points(some_dates, 1:12)
gs

weird part is that xlim is correct:
xlim(gs)
$side.1
[1] "2010-10-01 CDT" "2011-09-01 CDT"

is the bug in draw_axis?

Note what draw_custom_grid does, which draw_axis does not do (see #343 ?)
if(is.numeric(lim)){
    at <- axTicks(side)
  } else{
    fun <- getFromNamespace(paste0('axis.',class(lim)[1L]),'graphics')
    at <- fun(side, x = lim, lwd=0, lwd.ticks=0, labels = FALSE)
  }

see #396


Restore spellcheck colors, fix #418 & #548

Spellcheck colors are disabled since v3.3 of peaksea.vim, see: https://www.vim.org/scripts/script.php?script_id=760. This commit restores the colors, thus fixing #418 & #548. See more explanations in https://github.com/amix/vimrc/issues/548#issuecomment-1207165114.

Thanks for the patch, @bryango 🙏


Add open source license

Perhaps add an open source license to your repo?
MIT or BSD would be good choices if you want to have a very permissive license. If it is important to you that your code is not used in proprietary software, you can select a copyleft license, such as GNU GPL

Done!


Align appVersion from chart with release version

Describe the Bug
Currently appVersion is no aligned with the release version (See https://github.com/openzipkin/zipkin/blob/master/chart/zipkin-server/Chart.yaml#L17). We should update that after https://github.com/openzipkin/zipkin/blob/master/.github/workflows/create_release.yml#L40 and then publish helm chart after https://github.com/openzipkin/zipkin/blob/master/.github/workflows/deploy.yml#L71. This will trigger a consistent workflow.
Ping @jonkerj

Yeah, sounds like a great plan to me.

Do you have time to work on this @jonkerj?

Not anytime soon, holidays are coming and working at home with kids (schools closing).


chore: downgrade dependencies versions for flutter 3.0.5

Related to https://github.com/statsig-io/dart-sdk/issues/1

I'll merge this after your other diff so we don't need to bump the version again.

Thanks again


Ensure :ssl is started before Ecto.Migrator is ran

If an Ecto.Repo is configured with ssl: true, we have to make sure the :ssl application is started before Ecto.Migrator is used (eg. when the application is started).

🦀 Requesting reviewers for this pull request:

@thermech (contributor with 0 commits in the last 90 days and 8 commits overall)
@pmartel12 (expert reviewer for the elixir stack)

@simonprev I think it’s safe to start :ssl before migrations are ran even if DATABASE_SSL is not enabled. What do you think?

Sure, the ssl app is included anyway in the app, there is no "risk" in starting it for non-ssl database. 👍


Getting an error 'Attribute "address" of type "M" has an invalid value of "[object Object]"' in version 1.0.1

Summary:
I have one nested field 'address' in my collection which then will be converted to "M" type. But when I tried to Insert document with v1.0.1 it thrown an error 'Attribute "address" of type "M" has an invalid value of "[object Object]"' but when I tested the same with v0.8.7, it worked perfectly.
Please check the below stack trace
Stack Trace
{ ParseError: Attribute "address" of type "M" has an invalid value of "[object Object]"
    at dedynamofy (/Users/indianic/Workspace/serverless-testing-examples-master/node_modules/dynamoose/lib/Attribute.js:498:13)
    at Attribute.parseDynamo (/Users/indianic/Workspace/serverless-testing-examples-master/node_modules/dynamoose/lib/Attribute.js:573:11)
    at Schema.parseDynamo (/Users/indianic/Workspace/serverless-testing-examples-master/node_modules/dynamoose/lib/Schema.js:194:24)
    at NewModel.Model.put (/Users/indianic/Workspace/serverless-testing-examples-master/node_modules/dynamoose/lib/Model.js:322:12)
    at Function.Model.create (/Users/indianic/Workspace/serverless-testing-examples-master/node_modules/dynamoose/lib/Model.js:367:16)
    at Function.NewModel.create (/Users/indianic/Workspace/serverless-testing-examples-master/node_modules/dynamoose/lib/Model.js:168:20)
    at ./functions/asyncConcat.js.module.exports.addUser (/Users/indianic/Workspace/serverless-testing-examples-master/.webpack/service/functions/webpack:/functions/asyncConcat.js:68:1)
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:228:7)
  name: 'ParseError',
  message: 'Attribute "address" of type "M" has an invalid value of "[object Object]"' }

Environment:
Node.js version (node -v):  8.10.0
NPM version: (npm -v): 6.4.1
Dynamoose version: 1.0.1
Type (select 1):

[x] Bug report
[ ] Feature suggestion
[ ] Question
[ ] Other suggestion
[ ] Something not listed here

Other:

[x] I have read through the Dynamoose documentation before posting this issue
[ ] I have searched through the GitHub issues (including closed issues) and pull requests to ensure this issue has not already been raised before
[x] I have searched the internet and Stack Overflow to ensure this issue hasn't been raised or answered before
[x] I have tested the code provided and am confident it doesn't work as intended
[ ] I have filled out all fields above
[x] I am running the latest version of Dynamoose

@sahilkanani1812 I think in order to debug further I'll need to see your schema, model, and the code you are running that is producing that error.
I think it'd also be helpful to see the raw DynamoDB object in your table that is causing the error.
At first glance, it looks like you are trying to get a M type out of DynamoDB, but it is actually stored as a string.
In Dynamoose version 1.0.0 there was a breaking change to set the default for useDocumentTypes and useNativeBooleans to true instead of the previous false. See PR #376 for more information about that. It might be worth trying to set those back to true to override the new default to see if that works for you.
Once you provide that information I can try to help debug further.

I am also running into this bug. The error happens for me when I try to call SomeModel.scan().exec(). I have {saveUnknown: true} in the options provided when creating SomeModel. The error occurs when de-dynmofying the second record. One of my unknown attributes that I am fetching from dynamodb is a Map. When the first record is parsed, the unknown attribute is added to the Schema attributes. I assume that somehow this logic is being done incorrectly, but I haven't investigated further yet.

@dobrynin Can you please provide the information I requested above?
As well can you run export DEBUG=* then reproduce the issue and copy the entire log results in here?

So I figured out what the problem is. We have a deeply nested field dynamoObj.topLevelField.otherField[0].anotherField[0].relevantField. The first object that dynamoose encounters encounters a number as the value of relevantField so it sets that type to be Number in the schema. Unfortunately, our data is not perfectly type consistent for every field across different records. The second object dynamoose encounters has a string value instead of a number. This breaks the parser with the somewhat confusing error ParseError: Attribute "topLevelField" of type "M" has an invalid value of "[object Object]".
At the least it seems like we should raise a more descriptive error of the actual field that errored.
As for whether this is actually a bug: is there a reason why dynamoose wouldn't want to allow for fields to contain values of different types if you don't specify the field in your schema (as when using saveUnknown: true)?

@dobrynin The whole point of Dynamoose is to have strict typing and have a defined schema for your items. Let's pretend you have two DynamoDB objects.
{
    name: "Test"
}

and
{
    name: 50
}

No matter if you have that name property defined in your schema or not, and no matter if you have saveUnknown enabled or disabled, this will cause problems.
Dynamoose is designed to ensure your data follows strict patterns, and if you have completely different types for the same property Dynamoose isn't meant to handle that.
I'm not sure how massive of a change it would be to add that functionality, and I feel like there would have to be a strong argument for why we should add that feature. My suggestion would be to update your documents to have consistent types throughout each item.
Therefor I do not believe this is a bug.

I hope I understood what you said correctly. If I didn't or if you have further questions please feel free to let me know. Hopefully this helps!

@sahilkanani1812 Was the above comment by @dobrynin the same problem you were having or were you having a different issue?

@sahilkanani1812 Was the above comment by @dobrynin the same problem you were having or were you having a different issue?

@fishcharlie I guess I am having different issue. I mean I am getting the error only when I change my dynamoose version from 0.8.7 to 1.0.1. If I set my dynamoose version back to 0.8.7, same code runs without an error.
Sorry I may not be able to provide the whole code but i can provide the portion which is creating an issue. So basically I have my collection of Users and I need to add address of that user. So my schema will have address field which will have following attributes:
address: {
        streetAddress: String,
        city: String,
        state: String,
        country: String,
        zipCode: Number,
    }

and my schema configuration is:
var Users = dynamoose.model('Users', users, {
    create: true, // Create table in DB, if it does not exist,
    update: true, // Update remote indexes if they do not match local index structure
    waitForActive: true, // Wait for table to be created before trying to use it
    waitForActiveTimeout: 180000, // wait 3 minutes for table to activate
    saveUnknown: true,
    useDocumentTypes: true,
    useNativeBooleans: true,
    timestamps: false,
    throughput: {
        read: 5,
        write: 5
    }
});

and the code which is creating an issue is:
let user = await Users.create(userObj);

where userObj contains following object:
let userObj = {
"streetAddress":"Street String",
"city":"city String",
"state":"state String",
"country":"country String",
"zipCode":222222
}

@sahilkanani1812 I think in order to debug further I'll need to see your schema, model, and the code you are running that is producing that error.
I think it'd also be helpful to see the raw DynamoDB object in your table that is causing the error.
At first glance, it looks like you are trying to get a M type out of DynamoDB, but it is actually stored as a string.
In Dynamoose version 1.0.0 there was a breaking change to set the default for useDocumentTypes and useNativeBooleans to true instead of the previous false. See PR #376 for more information about that. It might be worth trying to set those back to true to override the new default to see if that works for you.
Once you provide that information I can try to help debug further.

Should I make useDocumentTypes and useNativeBooleans to false to get it worked across all the dynamoose versions?

Should I make useDocumentTypes and useNativeBooleans to false to get it worked across all the dynamoose versions?

@sahilkanani1812 I would try that, yes. Report back after you try that.

Can it be related to a bug #445 ??

@sahilkanani1812 I don't believe so, but I could be wrong

Closing this due to no recent activity. If this is still a problem or you have questions please feel free to comment again and we can try to help further.


Escaping shell arguments to exec()

First of all, I just want to say thanks for writing shelljs. It cleans up my code a lot.
I need to invoke commands with shell.exec(), however, those commands will include some input from external sources, to be passed as arguments to system executables. Is there a "safe" way to do this? I was hoping for the classic array syntax, like:
shell.exec(["ls", "-l", "/some/path"])

Though this doesn't seem to be implemented. It doesn't appear nodejs itself even has a shell escape function neither. Any plans on this front?

+1

+10000000

@d11wtq we don't currently support an array syntax for exec(), but you can achieve that with ['ls', '-A', '/path/to/dir'].join().
Or is your concern about escaping values (such as how printf "\\" is written as exec('printf "\\\\"') in shelljs)?

Please.  Providing only a string argument here is a recipe for disaster, honestly.  As pointed out, it's basically impossible to "properly" quote any arguments given.  As is, exec is vulnerable to file names containing spaces, special characters, semicolons, the works. Cf http://www.dwheeler.com/secure-programs/Secure-Programs-HOWTO/handle-metacharacters.html and the "motivation" section of (for example) https://www.python.org/dev/peps/pep-0324/.  A security-conscious API should strongly encourage providing arguments as an array by default.

Looks like this might be related to #103.
@cscott The concern is that, under-the-hood, we rely on Node's child_process module (specifically, exec() and execSync(). According to the docs, these only support a string argument. So although we could write something to allow exec() to take an array, I suspect we'd have to .join() the strings anyway.

As pointed out, it's basically impossible to "properly" quote any arguments given.

Could you give an example where it's impossible to quote arguments? I thought it was just a matter of escaping the appropriate characters, i.e. exec('head file\\ with\\ spaces.txt'). The point of exec is to take a string like you'd use for a /bin/bash command.
Maybe it would be better to expose a function for shell-escaping? Then we can write things such as:
// This gives us an alternative syntax
exec('printf "\\\\"'); // this works, but that's lots of backslashes! Very error-prone
exec(shellEscape('printf "\\"')); // this looks like the bash command: `printf "\\"`

// or, this example
ls('src/').forEach(function (fileName) {
  exec('head ' + shellEscape(fileName)); // it's safe to use the output of ls() now
}

child_process.execFile takes separated arguments -- as does the
underlying c function.  The point is to avoid using the shell (bash or
whatever) where it is not necessary, which avoids having to
reverse-engineer the shell's parsing algorithm and avoids any shell-related
exploits.  The point is to provide a mechanism that is "secure by default"
and force folks to jump through a few hoops perhaps if they need to do
something insecure --- not to provide an "insecure by default" interface
and hope that all clients remember to use the appropriate escape functions
(hint: they won't).
On Feb 7, 2016 2:52 AM, "Nate Fischer"<EMAIL_ADDRESS>wrote:

Looks like this might be related to #103
https://github.com/shelljs/shelljs/issues/103.
@cscott https://github.com/cscott The concern is that, under-the-hood,
we rely on Node's child_process module (specifically, exec() and
execSync(). According to the docs
https://nodejs.org/api/child_process.html#child_process_child_process_exec_command_options_callback,
these only support a string argument. So although we could write something
to allow exec() to take an array, I suspect we'd have to .join() the
strings anyway.
As pointed out, it's basically impossible to "properly" quote any
arguments given.
Could you give an example where it's impossible to quote arguments? I
thought it was just a matter of escaping the appropriate characters, i.e. exec('head
file\ with\ spaces.txt'). The point of exec is to take a string like
you'd use for a /bin/bash command.
Maybe it would be better to expose a function for shell-escaping? Then we
can write things such as:
// This gives us an alternative syntaxexec('printf "\\"'); // this works, but that's lots of backslashes! Very error-proneexec(shellEscape('printf "\"')); // this looks like the bash command: printf "\\"
// or, this examplels('src/').forEach(function (fileName) {
exec('head ' + shellEscape(fileName)); // it's safe to use the output of ls() now
}
—
Reply to this email directly or view it on GitHub
https://github.com/shelljs/shelljs/issues/143#issuecomment-180971847.

Just spitballing potential issues with "just the quotes" escaping -- what happens if you follow a Unicode surrogate character with a quote.  Does node/bash treat that as an actual quote, or as a invalid utf character (and replace it with a different utf character)?  Is this the same on all platforms and all shells?
This is just the first example I can think of---I guarantee you that if you are passing untrusted values to the shell via string concatenation and relying on naïve escaping  you will find plenty of similar edge cases and surprises.  This is really security 101.

@cscott from the docs for ShellJS's exec():

Executes the given command synchronously, unless otherwise specified.

Since it says the word "command," I argue that this should, by default, behave like executing a command in the shell (we don't call them "commands" in C and other relevant languages). I interpret exec('git status') as translating to git status, although I suppose another valid interpretation is translating to exec git status. Either way, however, these commands respect special characters (I don't see it as a security flaw, per se, since I think glob characters are a nice feature).
I'm not opposed to secure code. exec() is not the only function that can have unintended effects due to glob characters:
# in an empty directory
$ touch a.txt # creates a normal file
$ touch b.txt # again, works as expected
$ touch *.txt # does not create a file, this just updates time stamps for the other two
$ touch '*.txt' # this creates a file named *.txt
$ for k in *.txt; do echo $k; done # no quotes around $k is insecure
a.txt
b.txt
a.txt b.txt *.txt
$ for k in *.txt; do echo "$k"; done # double quotes allow $k to work sensibly
a.txt
b.txt
*.txt

One thing that's unique about exec is if you try exec('git add ' + filename);, where filename is actually the string test.txt;rm -rf ~. That's a valid filename, and if we allow code to create that file, the user then does an ls() & exec(cmd + filename) loop, they would get unintended side effects.
The point being, we can't just have one solution for exec(), because that doesn't really solve the issue for the other commands. Would you be opposed to exposing an execFile() alternative for exec() that handles the case involving characters like semicolons, redirection, etc. in a non-shell way, more similar to C's exec() system call?

Actually, a better idea would be to add an option to exec called "noGlob". When true, all characters are treated literally (as if the shell argument were wrapped in single quotes, or like an argument passed to C's execvp system call). As well as an explicit shellEscape function that prevents glob expansion on its arguments

I'm not opposed to an execFile alternative.  But I'd be much happier with
an option called glob or shell that enabled the dangerous functionality
for those who actually want/need it, and left most users "safe by default".

@cscott Here's what I'm thinking the interface should look like.
exec(cmd); // this will use a default value for glob (haven't decided if that should be false or true)
exec(cmd, {glob: true}); // '*' expands like a shell glob. Other characters have special meanings
exec(cmd, {glob: false}); // ';', '*', '?', '>' are all taken literally
exec(arg1, arg2, arg3); // executes something like `'arg1' 'arg2' 'arg3'` in the shell. arg2 could have a space, and it will still count as one argument
exec(arg1, arg2, arg3, {glob: true}); // arg2 can have a space, but it will still be one argument. * is a glob character
exec(arg1, arg2, arg3, {glob: false}); // same as above, but * is a literal *. ? is a literal ?. Etc.

The question remains, should this option be named glob, noGlob, shell, etc.? I'm leaning toward glob or shell, depending on what behavior it really provides. Also, we also have to figure out its default value (true or false). Could you provide some example ShellJS scripts that exhibit potentially unintended behavior?
Also, while we're on the subject, it may be worth considering adding a -f option to set() to disable globbing (see https://www.gnu.org/software/bash/manual/html_node/The-Set-Builtin.html). I think we should migrate to supporting globbing by default on all commands, so set('-f') would be a nice way to let people have extra-safe code.

@nfischer: +1 on set('-f'). IMHO, option should be glob, should be true` by default. The rest of your idea looks great.

I've got an idea for an API:
What if we did this (using ES6 string tags):
$mycommand --opt ${unsafeVar}
That gets transpiled to:
$('mycommand --opt ', unsafeVar);
Which is a totally acceptable syntax for <ES6.

@cscott Although I think we may try to add in the execFile-like interface for shelljs proper, here's an extension I wrote that handles most (not all!) of the security concerns raised in this issue:
https://github.com/nfischer/shelljs-exec-proxy

I would like to see @ariporad's suggestion implemented, making multi-line scripts possible as well (I think..).

Ok, so I finally had a moment to stop and think about ShellJS, so here's what I learned:
I misread the docs... I had the ES6 syntax right, but the ES5 syntax would be like this:
shell(['git doThing --foo', '--bar'], unsafeVar, unsafeVar2);

Which is a bit awkward.... that could also lead to the problem that if someone gets it wrong, and does this:
shell(['git doThing --foo', unsafeVar, '--bar', unsafeVar2]);

They actually won't be escaped, leading to bugs (or worse, security holes). It's also tricky to check against that case (only an array passed), because:
shell`git thingWithoutArgs`; // Becomes:
shell(['git thingWithoutArgs']);

So, our options would be:
// Option 1:
shell.tag`unsafe --thing ${unsafeVar}`

// Option 2 (sorry for the pseudocode):
function shell(strings, ...values) {
    if (strings.length === 1 && values.length === 0) {
        shell.normalExec(strings[0]);
    }
    if (Array.isArray(strings) && strings.length > 1 && values.length === 0) {
        child_process.execFile(...strings);
    }
    // Do the actual thing.
}

// Option 3:
shell('command', unsafeVar, '--arg', unsafeThing2);

Thoughts?

Ouch, 2+ year-old, high-priority security vulnerability? No consensus or fix yet?

@mikeatlas I've got something sitting on my computer that'll fix the security issues (it's a new API which would soft-deprecate exec). I've been waiting to push a PR because:

I'm not sure how I feel about the interface (but we can open that up to community discussion).
It still relies on the file system for IPC, which I don't really like. I'm still experimenting to see if there's a way to get IPC off the file system, but we've looked it up in the past and haven't found any sort of solution.

I'll test on Windows and open up a PR tonight or tomorrow, tagging the relevant issues.

I have a mild preference for glob:false by default, since if you need a glob you're likely to know it, whereas you are probably not going to stop to think "what if the user passes a string with a literal wildcard here, where i'm expecting a filename"?
But the important part (from my perspective) is glob or no glob, don't split arguments on spaces and use node's execFile instead of trying to build a string command.

I think the most important is to avoid command injection, which my implementation will fix. It's based off spawn instead of exec file, but the two are mostly interchangeable (I think this separates it further from exec, to alleviate confusion).

Are there any updates on this? This has been open for 3 years now.
Ref: https://snyk.io/vuln/npm:shelljs:20140723

For anyone who gets here looking for something like PHP's escapeshellarg(), nfischer implemented a rather slick way of calling shell commands via proxy and passing unsafe arguments as parameters:
https://github.com/shelljs/shelljs/issues/143#issuecomment-229830399
https://github.com/nfischer/shelljs-exec-proxy
yarn add shelljs-exec-proxy
# or
npm install shelljs-exec-proxy

const shell = require('shelljs-exec-proxy');

const myUrl = "google.com ; echo You have been pwned";

shell.curl(myUrl);
// prints: Could not resolve host: google.com ; echo You have been pwned

shell.exec('curl ' + myUrl);
// prints: You have been pwned

Bumping these back to v0.9, but this should be quick to fix once v0.8 is released.

can someone please update/resolve this issue please

Merging into #495, which is starting to see some traction (see #866).
Design doc: https://shelljs.page.link/cmd-design


Pj/replace/react fe

Added the new React FE. I left the old one in case there are any issues at all and it can be deleted at any point.
I moved a couple of things around in the BE and made a script to watch the BE TS, and start BE and FE.
I moved the app start into server.js as creating the app and running it in the same file can cause test issues. I never work with index.ts/js and now I am thinking having index and server is a bit weird. I should have moved the app in app.ts and left index. Not to bothered but we should talk about this. Does not affect this PR really.

@pablisch Can we pop the dist files in the gitignore please :) all good otherwise

@pablisch Can we pop the dist files in the gitignore please :) all good otherwise

Well that was harder than it needed to be. dist just would not go! Thank goodness for StackOverflow as I dread to think what nonsense chatGPT would have come up with.
I renamed index and server to app and index which makes more sense. There was a dev script which I am not sure what it did as it targeted the TS files, I ran it and it returned an error so I deleted it as it seemed like it might be trying to do what the server script does anyway.
As soon as we are happy that this FE works (it is the same as in Aura but for TS) then we can delete the old CRA FE.


Drawing of sheet entries

Added drawing of sheet entries. Actually all signal flow directions (in, out, bidi, unspecified) and all symbol directions (upper, lower, left, right edge) are supported. Also added more documentation for sheet entries.

Hi Martin,

@VADMIUM commented on this pull request.
Thanks for you work. I have a few questions, the major one being about
an inconsistency with the _FRAC1 units.
In altium.py [1]:

@@ -385,7 +388,7 @@ def main():
parser.add_argument("--renderer", choices={"svg", "tk"},
default="svg",
help=render.init.annotations["Renderer"])
args = parser.parse_args()

renderer = import_module("." + args.renderer, "vector")

renderer = import_module(".svg" , "vector")

Is it okay to drop this change?

OOps, yes. Please drop it, it was for testing purposes

In format.md [2]:

@@ -391,7 +391,10 @@ circ_angle = atan2( RADIUS * sin(angle),
SECONDARYRADIUS * cos(angle) )

|SYMBOLTYPE=Normal: Optional

Sheet entry
-Child of Sheet symbol
+Child of Sheet symbol. In SchDoc files you will
first see a RECORD=15 which defines a sheet symbol.
+That record is directly followed by multiple RECORD=16 entries which
define the sheet entries for the previously defined sheet symbol.
+Actually there is no other way to determine the parent-child
relation.
Isn’t there an OWNERINDEX property that you can use to determine the
parent? From memory if you give all the other records except the first
(header) indexes starting from zero, the OWNERINDEX always points back
to the parent.

No, neither the OWNERINDEX nor the other indices work for determining
the direct parent. First I thought I could use it but after examining
some SchDoc files, I had to drop the idea. At least in Altium 14, 16 and
17 (which I have installed on my work computer) it does not work.

In altium.py [3]:

     ):

       obj.check(name, value)

   obj.get("HARNESSTYPE")

   obj.check("SIDE", None, b"1")

   obj.get_int("INDEXINSHEET")

   obj.get_int("DISTANCEFROMTOP")

   obj.get_int("STYLE")

   obj.get_int("IOTYPE")

   obj.get("NAME")

       obj.get(property)

   if self.last_sheet_symbol:

       with

self.renderer.view(offset=get_location(self.last_sheet_symbol)) as
view:
I wonder if you can get automatic access to the parent sheet symbol as
objects.properties, i.e. you could call
get_location(objects.properties) etc. I think “objects” was meant
to be the parent or owner object. I could probably use a better name
:)

Yes, you could reverse-search the objects list for the last sheet
symbol. BUT in all cases I examined, the sheet symbol was defined
directly prior to the sheet symbol entries. Using the child relation in
the objects works for most types.....but sadly not for sheet entries:
|RECORD=15|INDEXINSHEET=1|OWNERPARTID=-1|LOCATION.X=800|LOCATION.Y=510|XSIZE=200|YSIZE=130|COLOR=128|AREACOLOR=8454016|ISSOLID=T|UNIQUEID=MXHFSEBJ|SYMBOLTYPE=Device
Sheet
|RECORD=16|OWNERINDEX=37|OWNERPARTID=-1|SIDE=2|DISTANCEFROMTOP=10|COLOR=128|AREACOLOR=8454143|TEXTCOLOR=128|TEXTFONTID=1|TEXTSTYLE=Full|NAME=CFGDIR|IOTYPE=2|STYLE=3|ARROWKIND=Block
& Triangle
|RECORD=16|OWNERINDEX=37|INDEXINSHEET=1|OWNERPARTID=-1|SIDE=2|DISTANCEFROMTOP=6|COLOR=128|AREACOLOR=8454143|TEXTCOLOR=128|TEXTFONTID=1|TEXTSTYLE=Full|NAME=OSZ25|IOTYPE=1|STYLE=3|ARROWKIND=Block
& Triangle Ò
|RECORD=16|OWNERINDEX=37|INDEXINSHEET=2|OWNERPARTID=-1|SIDE=2|DISTANCEFROMTOP=8|COLOR=128|AREACOLOR=8454143|TEXTCOLOR=128|TEXTFONTID=1|TEXTSTYLE=Full|NAME=SPI|HARNESSTYPE=SPI|STYLE=3|ARROWKIND=Block
& Triangle
In this example you can see a RECORD=15 which defines a sheet symbol
with INDEXINSHEET=1. Directly afterwards some sheet entries are being
defined, all with OWNERINDEX=37.

In altium.py [4]:

@@ -259,6 +259,7 @@ def get_int_frac(obj, property):
'''Return full value of a field with separate integer and
fraction'''
value = obj.get_int(property)
value += obj.get_int(property + "_FRAC") / FRAC_DENOM

value += obj.get_int(property + "_FRAC1") / FRAC_DENOM #sometimes
FRAC1 is used instead of FRAC.

Your code below has get_int_frac(obj, "DISTANCEFROMTOP")*10. Suppose
there is DISTANCEFROMTOP=10 and DISTANCEFROMTOP_FRAC1=500000 (500
thousand). According to format.md, this should give a final result of
105 in the usual units. But according to my reading of the code, you
are returning value = 10 + 500,000/100,000 = 15, and the caller will
multiply that by ten to get 150. Something’s not right.

You could be right, this is strange. I will check it. Can you please
reject my pull request, I will clarify this and send another request.

In altium.py [5]:

     ):

       obj.check(name, value)

   obj.get("HARNESSTYPE")

   obj.check("SIDE", None, b"1")

   obj.get_int("INDEXINSHEET")

   obj.get_int("DISTANCEFROMTOP")

   obj.get_int("STYLE")

   obj.get_int("IOTYPE")

   obj.get("NAME")

       obj.get(property)

   if self.last_sheet_symbol:

It looks like last_sheet_symbol is only set inside
“handle_sheet_symbol”, so I think this would give an exception
rather than be false, right?

You are right. If you have a valid SchDoc file handle_sheet_symbol will
always be called prior to the sheet entry drawing routine. I will rework
this part.
Maybe this results from the fact that I am learning python for three
days now. Normally I am working with other languages. ;-)

About "objects.properties" versus "last_sheet_symbol", I was assuming that you could rely on OWNERINDEX working. So if the index isn't working, ignore that suggestion.
But I am skeptical. The indexes that OWNERINDEX refers to are implicit. I think INDEXINSHEET is irrelevant for the parent-child relationship. (Perhaps it is related to deciding which objects are drawn over the top of other objects; I dunno.)
You may be able to use the dump.py program to print out the indexes. If the indexes are working to my understanding of the file format, you should see something like
$ python dump.py schematic.SchDoc
Header: |HEADER=b'Protel for Windows - Schematic Capture Binary File Version 5.0'|[. . .]
0: [. . .]|RECORD=b'31'|[. . .]
. . .
37: [. . .]|RECORD=15|INDEXINSHEET=1|[. . .]
38: [. . .]|RECORD=16|OWNERINDEX=37|[. . .]
39: [. . .]|RECORD=16|OWNERINDEX=37|[. . .]
40: [. . .]|RECORD=16|OWNERINDEX=37|[. . .]

Closing this as requested, but I am happy for a new pull request or reopening this one, whichever you prefer.

Martin,
Thanks for your explanation. I thought, there was a relation between
OWNERINDEX and INDEXINSHEET but if you just count the records as your
dump.py does, the OWNERINDEX makes sense again.
I'll change my code to use that relation.
Carsten
Am 2017-02-24 01:20, schrieb Martin Panter:

About "objects.properties" versus "last_sheet_symbol", I was assuming
that you could rely on OWNERINDEX working. So if the index isn't
working, ignore that suggestion.
But I am skeptical. The indexes that OWNERINDEX refers to are
implicit. I think INDEXINSHEET is irrelevant for the parent-child
relationship. (Perhaps it is related to deciding which objects are
drawn over the top of other objects; I dunno.)
You may be able to use the dump.py program to print out the indexes.
If the indexes are working to my understanding of the file format, you
should see something like
$ python dump.py schematic.SchDoc
Header: |HEADER=b'Protel for Windows - Schematic Capture Binary File
Version 5.0'|[. . .]
0: [. . .]|RECORD=b'31'|[. . .]
. . .
37: [. . .]|RECORD=15|INDEXINSHEET=1|[. . .]
38: [. . .]|RECORD=16|OWNERINDEX=37|[. . .]
39: [. . .]|RECORD=16|OWNERINDEX=37|[. . .]
40: [. . .]|RECORD=16|OWNERINDEX=37|[. . .]
Closing this as requested, but I am happy for a new pull request or
reopening this one, whichever you prefer.
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or mute the
thread [2].

Links:
[1]
https://github.com/vadmium/python-altium/pull/5#issuecomment-282165098
[2]
https://github.com/notifications/unsubscribe-auth/AI4gSHVqpedxWpbx9gkOTsAaGTR_0-HKks5rfiJMgaJpZM4MJrTZ


Refactor sets to have idempotent merges

One attribute of CRDTs is that they have idempotent merge functions. Right now, in the cast set, reaction set, and verification set, we throw an error in cases where we should no-op (i.e. when a message already exists or has already been overwritten).
TODO for this issue:

[x] Refactor add and remove methods (or equivalent methods) to no-op when valid messages cause no side effects
[ ] [Bonus] Update our return logic to provide result statuses that differentiate between no-ops and successful updates

Implemented https://github.com/farcasterxyz/hub/pull/82


[Docs] Improve Getting Started Content

We need to come up with a better getting started guide content wise in the documentation.

This has been resolved in release 1.5.2


--augmentation parameter overwritten with default value

Rasa Open Source version
3.0.0rc1
Rasa SDK version
No response
Rasa X version
No response
Python version
3.8
What operating system are you using?
OSX
What happened?
The --augmentation parameter is not making it into the TrainingDataGenerator component but instead seems to be overwritten with the default value.
Here in model_training.py it is correctly passed to train:

Here in TrainingTrackerProvider.provide it has been overwritten with the default value of 50:

Command / Request
rasa train --augmentation 200

Relevant log output
No response

@joejuzl easy fix but needs to be done

Woah, so quick! Was just typing a message about this 😅

Should note I didn't try this out for any different parameters, it was just a coincidence I found it. Wanted to write a test to demonstrate but couldn't quite wrap my head around how to do it so figured screenshots would be quicker


Inconsistent help for test

Dart SDK version: 2.13.4 (stable) (Wed Jun 23 13:08:41 2021 +0200) on "macos_x64"
dart help test

===>
Run tests for a project.

Usage: dart test [arguments]

dart test --help

===> Detailed help on the arguments.
A random selection of other commands provides the same help both ways.

This was fixed in https://github.com/dart-lang/test/pull/1554


Set Up Integration Suite Trial

Tutorial URL: https://developers.sap.com/tutorials/cp-starter-isuite-onboard-subscribe.html
Step 5: Assign roles for the capabilities
The below roles are not available if only the Design, Develop and operate integration Scenarios  capability is selected.

Labels:
Landscape: trail1, trail2, trail3
Label: SCPTest-2010A
Scenario Label: SCPTest-cloudin
Priority: Low
Best Regards,
Priyanka

Hi Priyanka,
You are right, this tutorial has been created to showcase only the Integration Suite Scenarion.
It's ab expected behavior.
Regards,
Vikram Kulkarni

Hi Vikram,
If this tutorial has been created to showcase only the Integration Suite Scenario, Please remove these roles as it may cause confusion for the customer.
Best Regards,
Priyanka

Hi Priyanka,
Our Idea is to inform the availability of all roles. However, I added the screenshot with the role to be selected.
Hence we are closing this issue.


bug(CdkOverlay): CdkOverlay incorrectly calculates viewport bounds

Reproduction
https://stackblitz.com/edit/components-issue-yh1yjm?file=src/app/example-component.ts
Steps to reproduce:

Spawn a flexible position based overlay.
Make overlay contents height larger than viewport height such that overflow occurs.
Make viewport height a floating point number with decimal portion between .0 to .5.
The overlay will now overflow out of the viewport bounds.
Make the viewport height a floating point number between .5 to next whole number, or a whole number.
The overlay will now correctly stay within the viewport bounds.

Notes:
If you just try slowly resizing the window height while the overlay is open, you will see it bounce in and out of the viewport as the window height cycles between bad and good numbers.
Problem is that viewport height is a rounded number, and overlay height is a floating point. Therefore when viewport rounds down, the following check fails.
https://github.com/angular/components/blob/e436479a7bef69b87e8954616a9e29a265a1a2b1/src/cdk/overlay/position/flexible-connected-position-strategy.ts#L632
To compare the two numbers, the rounding needs to be consistent.
Expected Behavior
Flexible dimension overlay should always remain within viewport bounds.
Actual Behavior
Flexible dimension overlay does not remain within viewport bounds when viewport height rounds down.
Environment

Angular: 10.2.3
CDK/Material: 10.2.7
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

I can imagine how this would happen if one of the numbers was rounded and the other one wasn't, although I'm having a hard time reproducing it with the demo, because I get rounded numbers for both of them. Does it happen for you in the demo, or is it inside an app under specific circumstances?

It appears to be OS dependant. I've been unable to reproduce it on macOS (chrome), but able to reproduce on windows/linux (chrome).

I tested it out against a few different browsers on Windows (Chrome, Firefox, Edge, IE).

I think getBoundingClientRect can return floating point numbers when zooming or scaling.
Both machines I tested this on were high resolution displays with scaling applied. Haven’t verified yet but it sounds likely to be the cause.

I managed to reproduce it by zooming in one level.


[SDA-7847] Add OIDC attributes to tf provider ocm

Related issue: https://issues.redhat.com/browse/SDA-7847
What
Add attributes for BYO OIDC support through tf providder for ocm
Why
Allow clients to use BYO OIDC

@nirarg please review it

@gdbranco please run make generate and insert the files under resources only

@gdbranco please add tests for the new logic
Other than that, looks good


[BUG] error in syscall server will terminate the application

Describe the bug
When the map request more memory than shared memory support, or the uprobe failed, it should not terminate the eBPF application.

Fixed by #321


Floating names on map

@ItsiW what's the idea here? show the name of restaurants on the map automatically so that people don't have to hover over (desktop) or press down on (mobile) the circles to see the name?

Yes that was the idea, but I've already made the changes. They start appearing at a certain zoom level so things aren't crowded


doc: fix unaddressable rfc5234(ABNF) link

It seems that rfc5234 has been moved into datatracker. The origin one is now 404 .
@nicolasnoble

The current link still works for me.  It forwards to the new one.


Unwrap exception in the client

We have pre-defined exceptions and throw them in the client depending on the response the client gets.
For example, if the client gets a shutting down exception message from the server, the client throws ShuttingDownException.
However, the error message from the server isn't unwrapped so CentralDogmaException is thrown instead:
com.linecorp.centraldogma.common.CentralDogmaException: unexpected response: [:status=500, ...], {"exception":"java.util.concurrent.CompletionException","message":"com.linecorp.centraldogma.common.ShuttingDownException"}
 at com.linecorp.centraldogma.client.armeria.ArmeriaCentralDogma.handleErrorResponse(ArmeriaCentralDogma.java:1135)
 at com.linecorp.centraldogma.client.armeria.ArmeriaCentralDogma.normalizeRevision(ArmeriaCentralDogma.java:396)
 at java.base/java.util.concurrent.CompletableFuture$UniApply.tryFire(CompletableFuture.java:642)
 at java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:506)
 at java.base/java.util.concurrent.CompletableFuture.complete(CompletableFuture.java:2073)
 at com.linecorp.armeria.common.util.UnmodifiableFuture.doComplete(UnmodifiableFuture.java:152)
 at com.linecorp.armeria.common.util.UnmodifiableFuture.lambda$wrap$0(UnmodifiableFuture.java:103)

We should unwrap the message and throw the corresponding exception.
https://github.com/line/centraldogma/blob/master/client/java-armeria/src/main/java/com/linecorp/centraldogma/client/armeria/ArmeriaCentralDogma.java#L1116-L1135

https://github.com/line/centraldogma/pull/737 (Peel an exception before it is converted to an HttpResponse in HttpApiUtil.) will solve this issue?

Thanks, @imasahiro for noticing this. 😄
I think so. cc @ikhoon


Addon system - ability to load an instanced scene as a custom type or have a custom type with children nodes that are exposed to editor

Either I am doing something wrong or it seems not possible to do.
The goal is to create a custom type game object addon that consists of a node that contains a number of children nodes that are all exposed to the editor
I know I can make children nodes via the runtime, but that happens only while the game is running. I want to have these children nodes be created and available in the editor when the custom type is added to the scene
Is there a way to do this in godot? If so I can't seem to find any examples or documentation on how to do it

Related to #3895

@blurymind workaround is to set the owner of the child nodes to be the edited scene, then they will appear in the editor.
I would use normal scenes for this tbh, not sure what is the point of having a custom way of doing what a scene already does in the first place. It's like defining new nodes that always have children, there can be a lot of ways to break this as well.

I came up with a hacky setget solution :)
I guess I want to save a few steps of creating the child nodes for the user
extends KinematicBody2D
tool

export (bool) var enable = false setget initialize
func initialize(value):
	if get_tree().is_editor_hint():
		if value == true:
			if get_child_count() == 0: # initialize helper nodes
				var root =  get_tree().get_edited_scene_root()

				var SpriteNode = Sprite.new()
				add_child(SpriteNode)
				SpriteNode.set_owner(root)

		else: print("behavior disabled")
	enable = value

This will add a tick box to enable the behavior. If the user hasn't added any custom children to it, my code will do that for him/her. So the option is there to be custom, but you can also have my addon do the work

Beware, as I said it can be broken in multiple ways, what if the user adds unrelated children to the nodes you add dynamically? What if the user decides to delete these nodes?

@Zylann  yes I know =) thats why I exposed all the resource variables in the addon's master script like this:
export var animationNodePath = NodePath()
onready var animationNode = get_node(animationNodePath)
export var idleAnimation = "idle"
export var idleWaitAnimation = "idleWait"
export var walkAnimation = "walk"
export var jumpUpAnimation = "jumpUp"
export var jumpDownAnimation = "jumpDown"

If the user deletes these nodes, they will also disappear from the exposed variables. The user will then get a message that the nodes are missing and need to be set.
If the user adds new children, it wont affect it, unless they rename the current nodes - at which point they will have to reconnect the nodes via the exposed nodepaths in the editor.
A nice side effect to this init method is that the user can go custom, but can also go automagic. Because the addon is not enabled when added to the scene, the user can take the manual root before enabling it. If they enable it before adding any children,the script will do some of the work and create the nodes.
If they want to reset the behavior, they can delete all of it's children and re-enable it

Closing due to lack of support. If you still have a need for this, please open a proposal on the Godot Proposals repository.


Expose version in package

Permit such as
import charset_normalizer

print(charset_normalizer.__version__)

Codecov Report

Merging #18 into master will decrease coverage by 0.19%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master      #18     +/-   ##
=========================================
- Coverage   83.07%   82.88%   -0.2%
=========================================
  Files          12       13      +1
  Lines         851      853      +2
=========================================
  Hits          707      707
- Misses        144      146      +2

Impacted Files
Coverage Δ

charset_normalizer/version.py
0% <0%> (ø)

Continue to review full report at Codecov.

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


feat: include jvm build

Integrates JVM build to generate JAR files.
"Closes #6"

@barbeau Any suggestions on where do I write tests?
Should I create a compose desktop similar to BikeShare and write unit tests with in?


Configure Jenkins Location on Boot

It would be nice if the jenkins location was configured to the route used to expose it so that env vars like JOB_DISPLAY_URL were valid.

So on double check, the value actually appears to be set, but JOB_DISPLAY_URL=http://unconfigured-jenkins-location/job/my/job/url is not properly set. Not sure what's up there, maybe it needs to be set in two places?

hey @sherl0cks
Was previously unaware of the env var
Some quick searching points me to the https://wiki.jenkins.io/display/JENKINS/Display+URL+API+Plugin plugin
Is that one you happen to be using?

Looks like it you got it @gnufied . I was unaware it was a plugin. I'm currently doing a string replace on that env var with the contents of oc get route jenkins in Jenkinsfile, but it would be nice if the env vars were set and did require it.

Opened https://trello.com/c/SIJZ95Cl/1552-integration-with-the-display-url-api-plugin-jenkinsintegration
Closing per process


Attempt direct read on ResourceInputStream::read()

I suspect the read buffer is filled by OpenSSL in another I/O operation and then the stream is never marked readable again, because the readable data is already in PHP's buffer space. An instant read attempt solves this by always ensuring the read buffer is completely empty.
/cc @brstgt @bwoebi @trowski

That also solves #21 I think.

It still hangs with that PR.

Closing for now.


Data out > Batch > Write ODBC / JDBC

This is a how-to guide in a larger section on exporting Batch data. Our current CSV data import / export guides are suitable templates.
@devinrsmith to review

I don't know how to cover this ticket. We don't currently support writing directly to SQL. This ticket should either be pushed back or closed.


If you want to have an actual conversation about the merits of this project, then please offer your arguments. Otherwise, why are you being just as rude and offensive as the people who want CoCs?

lol just lol if u think this project has merits
also there's no code of conduct so I can say whatever I want about this project, including the verifiable fact that its supporters are sad little people, and you can't do anything about it

spam requires no code of conduct. you have opened up multiple issues that exist solely as spam.

also there's no code of conduct so I can say whatever I want

That is literally just not true at all. Github itself has terms of use.

and it would be completely antithetical to this project to adhere to them ;-)

Maybe if you have no concept of precedence