Weekly Challenge Rulesets

There have been a number of Weekly Challenges posted to the Demeo discord channel which involve playing the game with some extra challenge added. It could be nice to nice to rules/rulesets to for these challenges where we're able.
To date there have been about 10 challenges announced.
Some of the challenges are out of scope for modding as they are within the normal game rules:

'Play in an all hunter group'
'Play an all assassin group'
'Play an all sorcerer group'
'Kill any rats you see'
'Destroy all lamps'
'Destroy all fungi'
'use as much poision gas as possible'
'set fire to as many tiles as possible'
'speedrun'
'befriend as many enemies as possible'

Some of the challenges would require changes to the game's logic - theoretically possible, but not easy to implement.

'you cannot exit the level until all doees have been opened'
'the next card you use is the last card you picked up'
'instantly use any card you gain from card meter or chests'
'sudden death - complete the entire book without a single player being downed once'
'beat the boss without killing anything except necessary enemise (e.g. keyholder)'
'complete any action within 5 seconds'

The remaining challenges seem quite feasible

No Gold for You
Use the current GoldPickedUpMultiplied rule to set value to zero?

No Chests
Use LevelPropertiesModifiedRule to remove all the chests?

Get ready to put your friendship to the test! Never cast a card on yourself and rely only on help from your teammates.
This could be implemented with a new rule to Set Ability.mayTargetSelf = False for all player abilities.. (maybe for enemy abilities too)

Play with a maximum of 5 cards instead of 10 (9+replenishables)
-Not sure where, but we must be able to set this somewhere.-
This is now catered for by CardLimitModifiedRule #81

Complete Realm of the Rat King without using any torches
This is catered for by CardClassRestrictionRule - this rule allows for the torches to be assigned to a non-player character such as 'mushroom' which will then prevent players from ever recieving this as a card.

I suggest that we close this issue. We've got a good number of rules which would facilitate weekly rulesets, but there is no demand forr them. As 'weekly' rulesets keep on being thought up, this issue is never likely to be marked as complete.


All links (send/receive/rewards) fail to show in clipboard

Description
When I try to create and copy a link on celo.gooddollar.org, the native share menu appears and the copy is available, but the link is missing.
Steps to reproduce
On an iphone,

head to invite rewards
try to copy your invite link
see text but no link

Which network did you use?
celo/fuse
Supporting data

not reproduced at latest versions of iOS

@patpedrosa do we know versions of devices this happens on?
moving it to testing for now...

verified on prod


win32: Use Wide APIs for font handling

Use LOGFONTW instead of LOGFONT, etc.
Now we can use a font even which name cannot be represented in the
current code page. (E.g. Use Japanese fonts on English version of Windows.)

Codecov Report

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

@@            Coverage Diff             @@
##           master    #4000      +/-   ##
==========================================
+ Coverage   79.13%   79.13%   +<.01%
==========================================
  Files         105      105
  Lines      141111   141111
==========================================
+ Hits       111671   111673       +2
+ Misses      29440    29438       -2

Impacted Files
Coverage Δ

src/terminal.c
78.21% <0%> (-0.13%)
:arrow_down:

src/ex_cmds2.c
84.89% <0%> (-0.1%)
:arrow_down:

src/ui.c
50.14% <0%> (-0.08%)
:arrow_down:

src/gui.c
58.1% <0%> (ø)
:arrow_up:

src/if_ruby.c
90.29% <0%> (ø)
:arrow_up:

src/version.c
86.28% <0%> (ø)
:arrow_up:

src/os_unix.c
58.98% <0%> (+0.04%)
:arrow_up:

src/channel.c
83.16% <0%> (+0.07%)
:arrow_up:

src/gui_gtk_x11.c
48.37% <0%> (+0.09%)
:arrow_up:

src/if_xcmdsrv.c
84.02% <0%> (+0.53%)
:arrow_up:

Continue to review full report at Codecov.

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

Note: This fixes the following TODO items since v7.4.202:
https://github.com/vim/vim/blob/6a95c889bf5ac81377bf5bb42a9d5d7e5cd3b1e2/src/gui_w32.c#L3217
https://github.com/vim/vim/blob/6a95c889bf5ac81377bf5bb42a9d5d7e5cd3b1e2/src/os_mswin.c#L3085


Last batch record missing

Relevant system information:

OS: Ubuntu 16.04.5 LTS in Docker
PostgreSQLversion: psql (PostgreSQL) 10.6 (Ubuntu 10.6-1.pgdg16.04+1)
TimescaleDB version: plpgsql | 1.0
Installation method: "apt install"

Describe the bug
Inserting a large csv file results in a table with the last record of each batch missing.
To Reproduce
Steps to reproduce the behavior:

Execute timescaledb-parallel-copy with a large csv file (Personally 90,000,000 records - verified with wc -l command)
In psql, select count(*) from 'table-name'
2a. with default --batch-size, output is 89982000
2b. with --batch-size 4000, output is 89977500
In psql, select * from 'table-name' where 'primray-key-name' = 'id of last record of a batch' ; returns empty

Expected behavior
The number of inserted records should be the same as the number of records in CSV file
Actual behavior
Last record of each batch is not inserted

@ropieur wasn't even aware of the "transfer issue" functionality. Thanks for pointing me to it.

Seems this is due to the HEADER option which, I guess, is applied on each batch, although it should be applied only on the first batch, in order to follow the same semantic as the psql COPY command. My feeling is that the copy options are blindly transferred to the COPY for each batch.

Thanks for the report, will investigate

By the way @ropieur , can you paste the exact command you ran?
We do have an option to skip the header via --skip-header rather than using HEADER in the CSV command. Maybe that's where the problem is?

Hello RobAtticus,
I don't see --skip-header in the help usage. Here follows the list of possible options:
-batch-size int
-columns string
-connection string
-copy-options string
-db-name string
-file string
-log-batches
-reporting-period duration
-schema string
-split string
-table string
-truncate
-verbose
-version
-workers int
And the version is "timescaledb-parallel-copy 0.1.0 (linux amd64)"
This explains that, likely

And indeed I pass --copy-options "CSV HEADER" as argument

Ah I see, I am starting the release process for an update to timescaledb-tools which should contain the new version of this tool which has --skip-header

Hi @ropieur , the updated version should be available in packages now. Let me know if that flag works for you.

Closing because I believe this is resolved.


custom pytorch training out of memory issue

Hi, I have been trying to replicate the great memory performance of using SFTtrainer class but haven't been able to succeed via custom training loop. It looks to me that somehow using the trainer class saved significant amount of memory. However, with same 8bits optimizer, model, and others like data collator, data loader, the custom training loop hits out of memory right away.
My goal is to have customized optimizers thus needed a custom training via pytorch.

@shizheng-rlfresh That's weird - are you using mixed precision training?

The issue is fixed. Thanks for the response.

@shizheng-rlfresh Oh so did u manage to solve the issue?

@danielhanchen i guess, with a minor issue of still using a little bit more memory than blindly using trainer imo. thinking might be state management in optimizer or dataloader used in Trainer class? If you know anything better, please share it. Thanks!
Notebook example is here
Thank you for the great implementation of models/etc, it is very useful!


TypeScript doesn't work

Type: Bug
Open VSCode, and TS server crashes
VS Code version: Code 1.89.1 (Universal) (dc96b837cf6bb4af9cd736aa3af08cf8279f7685, 2024-05-07T05:14:24.611Z)
OS version: Darwin arm64 23.4.0
Modes:

System Info

Item
Value

CPUs
Apple M3 Max (16 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
2, 3, 3

Memory (System)
64.00GB (4.28GB free)

Process Argv
--crash-reporter-id d8e036f5-10a3-4c92-91fb-39f282b4a7db

Screen Reader
no

VM
0%

Extensions (24)

Extension
Author (truncated)
Version

jsonl
Alp
0.0.1

jest-snippets
and
1.9.1

vscode-zipfs
arc
3.0.0

asyncapi-preview
asy
0.6.3

vscode-mac-color-picker
dae
1.1.0

dart-code
Dar
3.90.0

flutter
Dar
3.90.0

vscode-eslint
dba
2.4.4

prettier-vscode
esb
10.4.0

copilot
Git
1.197.0

copilot-chat
Git
0.15.2

gitlab-workflow
Git
4.14.0

go
gol
0.41.4

terraform
has
2.30.1

golang-snippets
hon
1.2.0

gml-support
lia
1.7.4

pnpm-vscode-helper
lin
0.3.0

cpptools
ms-
1.20.5

vscode-pbkit
pbk
0.0.7

platformio-ide
pla
3.3.3

vscode-yaml
red
1.14.0

vscode-icons
vsc
12.8.0

gitblame
wad
11.0.1

clang-format
xav
1.9.0

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pythonmypyd1:30879173
pythoncet0:30885854
2e7ec940:31000449
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
666c4381:31064718
chatpanelc:31048052
dsvsc021:30996838
945dj816:31013170
pythoncenvpt:31062603
a69g1124:31058053
dvdeprecationcf:31061161
pythonprt:31056678
dwnewjupytercf:31046870
fa862199:31048878

@ttarnowski Sharing what worked for me (I use yarn + pnp mode).

It seems that typescript.tsserver.experimental.useVsCodeWatcher was enabled by default somehow which is causing the Language Server to continuously crash, setting it to false worked.

https://github.com/microsoft/vscode/issues/213484

same error here.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Sort out CRDs

Sort out CRDs(both Cluster API and ours) that are required for MachinePool (and provider specific parts of it)

[x] v1alpha3 Cluster API CRDs to apiextensions https://github.com/giantswarm/apiextensions/pull/465
[x] Azure provider extensions for Cluster API types. https://github.com/giantswarm/apiextensions/pull/465
[ ] Ensure CRDs are installed on all installations: https://github.com/giantswarm/opsctl/pull/673

For the record: I solved the CRD version incompatibility problem between AWS and Azure by implementing additional filtering in opsctl.


Set DisableCompression to true to disable unwanted httpclient transfo…

What this PR does / why we need it:
Ports the fix made in #4165 and fixes #2916
If applicable:

[ ] this PR contains documentation
[X] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

I've duplicated the issue where the fix was merged into Helm v2.

Need to perform gofmt -l -w -s . to format code.


Default value/placeholder if nothing is picked

Is there a possibility to add a default value or placeholder if there is nothing picked?

Ah ok i see there is already PR, for this... it would be good to have it though :)


networking: buf: Disallow blocking allocation in syswq

Work items on the syswq should always run-to-completion.
Override the timeout value to always be K_NO_WAIT.
Allocating with K_FOREVER leads to deadlocks if the freeing also happens from the syswq. Non-zero timeouts are also not nice for the other users of the syswq.

@jhedberg fixed, thanks


Made dictionaries for connection string elements case-insensitive

Attempt to resolve #17641.

We also need to write a lot more tests with cases for AllRequired, Optional, AtLeastOne, None, MatchesAll because there are more changes that the tests in this PR do not cover. The tests in this PR only covers checking if the Setting accepts upper and lower case.
Here's the github issue to address this https://github.com/Azure/azure-sdk-for-net/issues/17856

We also need to write a lot more tests with cases for AllRequired, Optional, AtLeastOne, None, MatchesAll because there are more changes that the tests in this PR do not cover. The tests in this PR only covers checking if the Setting accepts upper and lower case.
Here's the github issue to address this https://github.com/Azure/azure-sdk-for-net/issues/17856

@amnguye sounds good. can you please merge this PR then? I don't have permissions. I closed the another one.

@amnguye sounds good. can you please merge this PR then? I don't have permissions. I closed the another one.

Could you also update the changelog to reflect the changes you made. Maybe adding
"- Fixed bug where parsing the connection string for setting values only accepted lowercase values" would be fine

Could you also update the changelog to reflect the changes you made. Maybe adding
"- Fixed bug where parsing the connection string for setting values only accepted lowercase values" would be fine

Sure, updated the changelog.

Sure, updated the changelog.


real-time html accessed on local network shows nothing

I am trying to generate real-time html using the following command:
goaccess site.log -o report.html --log-format=CADDY --real-time-html
The terminal output is:
 [PARSING site.log] {0} @ {0/s}
WebSocket server ready to accept new client connections
But when I try to access locally at 192.168.X.X:7890 the page is blank and the browser indicates 400 Invalid Request. Adding /report.html to the address does not work either.
I can generate a static report.html, and access it locally just fine, but the real-time html shows nothing. Can someone please show me what I am doing wrong? Thanks.

Nevermind. I assumed goaccess was serving the html. For reference, see here


Allow fetching license info from peerDependencies and optionalDependencies

Add support of peerDependencies and optionalDependencies.
Reference: https://github.com/ironSource/license-report/issues/61

@BePo65, could you, please, have a look why the Node.js CI / build (10.x) (pull_request) is failed?
I don't see any suspicious points there.
Thanks.

That is my fault: I updated mocha to version 9 just to fix an issue from the github dependabot and here mocha dropped support for node v10 (as this felt out of LTS in march).
I didn't find the time to clear this with Yaniv and to drop the v10 support from the github actions.
So with your PR everything is fine,don't worry.

That is my fault: I updated mocha to version 9 just to fix an issue from the github dependabot and here mocha dropped support for node v10 (as this felt out of LTS in march).
I didn't find the time to clear this with Yaniv and to drop the v10 support from the github actions.
So with your PR everything is fine,don't worry.

@BePo65, should I still do something to be able to merge this PR?

No, we will do the rest (but please be patient, we are quite busy at the moment).


Fix "no response" error when viewing cameras via apple watch

Apple watch requires specific video resolution and frame rate, and without it the error message "no response" will be shown when the camera is clicked and watched via apple watch. Already paired cameras won't take effect until they are re-paired.

Are you sure all models require that attrs?
The watch will show the stream even if go2rtc gives a completely different resolution?

Hi Alex,
Thanks for the suggestion. I was able to verify that it works on my Apple Watch 7. The specific resolution/fps are also mentioned in HAP-NodeJS and Scrypted. The watch will only show thumbnail and won't show real-time stream if go2rtc gives different resolution.

Thumbnail preview

Streaming

This is because thumbnail can be requested with any resolution

https://github.com/AlexxIT/go2rtc/releases/tag/v1.9.0


Environment speed on different systems

Hi,
we are testing the environment on different systems and it turns out that there are major differences in performance. Bellow I report average episode time when performing random actions:
it takes ~16sec on:

Intel(R) Xeon(R) Silver 4110 CPU @ 2.10GHz + NVIDIA1080 or 2080TI (driver 410.93; cuda10)

it takes ~4sec on:

Intel(R) Xeon(R) CPU E5-1650 v2 @ 3.50GHz + 780TI (driver 410.73; cuda10)
Intel(R) Core(TM) i7-3930K CPU @ 3.20GHz + 1080 (driver 410.73; cuda10)

Any ideas why there is such a huge performance difference?

I'm leaving the Issue open for now. But it looks more and more like it's an issue with our CPUs and config.

Hi @miha-skalic
Can you share a little more information? This may be helpful to other users deciding where to run Obstacle Tower.

In this case we had lower level issues, where the CPUs were running at minimal frequency (800MHz), due to some incompatibility (I don't have proper answer how/why, but fixing frequency in bios helped).
It turns out that the performance of obstacle tower is linearly correlated to CPU Frequency.

Thanks for the information. It is good to know that performance was scaling as expected on your hardware.


Add support for jsconfig.json in language service

This PR adds support for jsconfig.json in the language service. The implementation treats jsconfig.json as a tsconfig.json file with the allowJs compiler option set to true.

@mhegazy @billti any other comments?
In addition, all error messages suggesting adding a new tsconfig.json should be updated with recommending both tsconfig.json and jsconfig.json. Though those can be a separate PR after 1.8 release.

One more change, then :+1:


TimeoutError while fetching Pixiv images

Describe the bug
A clear and concise description of what the bug is.

Other sites work normally (Twitter)
Pixiv images appear in GALLERY
Pixiv images don't appear in STORAGE
Pixiv bookmark function works
Corresponding pngs are created in data/temp but have size 0

To Reproduce
Steps to reproduce the behavior:

Send a pixiv link to bot https://www.pixiv.net/artworks/104233490
See error Error: Timeout, please try again.

Expected behavior
Fetch the images and send them to storages.
Error output / Logs
2023-01-06 15:13:11,215 - nazurin.__init__ - INFO - Loaded 12 sites
2023-01-06 15:13:12,492 - nazurin.__init__ - INFO - Loaded 2 storage(s), using: ['Telegram', 'GoogleDrive']
2023-01-06 15:13:12,498 - nazurin.dispatcher - INFO - Set webhook
2023-01-06 15:13:13,311 - aiogram.executor - INFO - Bot: ナズーリン [@botname]
======== Running on http://<IP_ADDRESS>:80 ========
(Press CTRL+C to quit)
2023-01-07 05:47:37,261 - nazurin.bot - INFO - Collection update: site=pixiv, match=('104233490',)
2023-01-07 05:47:37,373 - aiohttp.access.web_log - INFO - <IP_ADDRESS> [07/Jan/2023:05:47:37 +0000] "POST /<TELEGRAM_BOT_TOKEN> HTTP/1.0" 200 152 "-" "-"
2023-01-07 05:47:37,656 - nazurin.api - INFO - Bookmarked artwork 104233490, privacy = public
2023-01-07 05:47:40,815 - nazurin.image - INFO - Got image size: 2150086
2023-01-07 05:47:48,990 - tenacity.decorators - INFO - TimeoutError() during nazurin.models.file.File.download execution, 1 of 5 attempted.
2023-01-07 05:48:00,993 - tenacity.decorators - INFO - TimeoutError() during nazurin.models.file.File.download execution, 2 of 5 attempted.
2023-01-07 05:48:13,989 - tenacity.decorators - INFO - TimeoutError() during nazurin.models.file.File.download execution, 3 of 5 attempted.
2023-01-07 05:48:28,990 - tenacity.decorators - INFO - TimeoutError() during nazurin.models.file.File.download execution, 4 of 5 attempted.
2023-01-07 05:48:47,991 - tenacity.decorators - INFO - TimeoutError() during nazurin.models.file.File.download execution, 5 of 5 attempted.
Traceback (most recent call last):
  File "/app/nazurin/__main__.py", line 56, in on_error
    raise exception
  File "/usr/local/lib/python3.8/site-packages/aiogram/dispatcher/dispatcher.py", line 1394, in process_response
    response = task.result()
  File "/app/nazurin/dispatcher.py", line 59, in update_collection
    await self.bot.update_collection(urls, message)
  File "/app/nazurin/bot.py", line 131, in update_collection
    await asyncio.gather(save, download)
  File "/app/nazurin/sites/pixiv/models.py", line 21, in download
    await super().download(headers=HEADERS, **kwargs)
  File "/app/nazurin/models/illust.py", line 35, in download
    await asyncio.gather(*tasks)
  File "/usr/local/lib/python3.8/site-packages/tenacity/_asyncio.py", line 48, in call
    do = self.iter(retry_state=retry_state)
  File "/usr/local/lib/python3.8/site-packages/tenacity/__init__.py", line 332, in iter
    raise retry_exc.reraise()
  File "/usr/local/lib/python3.8/site-packages/tenacity/__init__.py", line 168, in reraise
    raise self.last_attempt.result()
  File "/usr/local/lib/python3.8/concurrent/futures/_base.py", line 437, in result
    return self.__get_result()
  File "/usr/local/lib/python3.8/concurrent/futures/_base.py", line 389, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.8/site-packages/tenacity/_asyncio.py", line 51, in call
    result = yield from fn(*args, **kwargs)
  File "/app/nazurin/models/file.py", line 46, in download
    await f.write(await response.read())
  File "/usr/local/lib/python3.8/site-packages/aiohttp/client_reqrep.py", line 1037, in read
    self._body = await self.content.read()
  File "/usr/local/lib/python3.8/site-packages/aiohttp/streams.py", line 375, in read
    block = await self.readany()
  File "/usr/local/lib/python3.8/site-packages/aiohttp/streams.py", line 397, in readany
    await self._wait("readany")
  File "/usr/local/lib/python3.8/site-packages/aiohttp/streams.py", line 304, in _wait
    await waiter
  File "/usr/local/lib/python3.8/site-packages/aiohttp/helpers.py", line 720, in __exit__
    raise asyncio.TimeoutError from None
asyncio.exceptions.TimeoutError

Environment (please complete the following information):

OS: self-built docker image based on d9a2d5c (v2.2.2).
Python Version: 3.8

Additional context
I have tried to refresh Pixiv token by #9, does not work.

Seems working on my side.
What about other Pixiv links? Have you tried /pixiv_download? You may also try to download the image via the pximg.net link in command line (e.g. using wget or curl) on your server to see if it's working.

Have you tried /pixiv_download?

Error: Timeout, please try again.

You may also try to download the image via the pximg.net link in command line (e.g. using wget or curl) on your server to see if it's working.

$ curl -L 'https://i.pximg.net/img-original/img/2023/01/04/19/34/29/104233490_p0.png' \
                               -H 'authority: i.pximg.net' \
                               -H 'accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9' \
                               -H 'accept-language: ja-JP,ja;q=0.9,en-US;q=0.8,en;q=0.7,zh-CN;q=0.6,zh;q=0.5' \
                               -H 'cache-control: no-cache' \
                               -H 'dnt: 1' \
                               -H 'pragma: no-cache' \
                               -H 'referer: https://www.pixiv.net/' \
                               -H 'sec-ch-ua: "Not?A_Brand";v="8", "Chromium";v="108"' \
                               -H 'sec-ch-ua-mobile: ?0' \
                               -H 'sec-ch-ua-platform: "Linux"' \
                               -H 'sec-fetch-dest: document' \
                               -H 'sec-fetch-mode: navigate' \
                               -H 'sec-fetch-site: cross-site' \
                               -H 'sec-fetch-user: ?1' \
                               -H 'upgrade-insecure-requests: 1' \
                               -H 'user-agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36' \
                               --compressed --output 104233490_p0.png
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 2099k  100 2099k    0     0  27797      0  0:01:17  0:01:17 --:--:-- 34576

Seems it is the issue of my network. (Time Total 01:17 for 2099k) I don't know why(

Modified TIMEOUT in env, it works now. Thank you.


dot-prop

1 dot-prop vulnerability found in SaplingMobile/yarn.lock 3 days ago
Remediation
Upgrade dot-prop to version 4.2.1 or later. For example:
<EMAIL_ADDRESS>version "4.2.1"
Always verify the validity and compatibility of suggestions with your codebase.
Details
CVE-2020-8116
high severity
Vulnerable versions: < 4.2.1
Patched version: 4.2.1
Prototype pollution vulnerability in dot-prop npm package before versions 4.2.1 and 5.1.1 allows an attacker to add arbitrary properties to JavaScript language constructs such as objects.

Fixed with https://github.com/PirateNetwork/Skull-Island/commit/05d367b6b8c8640e8dda09d2c864037487ab91b0


Fix radio description spacing change on wrapping

What is the context of this PR?
Fixes: #1180
How to review
Check that when the description wraps on radios, checkboxes and labels the description no longer moves closer to the label itself

 Issue #1180: Vertical space between radio/checkbox label and description is not maintained when description wraps


KeyBinding Problem

Issue Type: Bug
sometimes with toggling recording button it is still stuck in recording mode
VS Code version: Code 1.39.1 (88f15d17dca836346e787762685a40bb5cce75a8, 2019-10-10T23:31:28.683Z)
OS version: Windows_NT x64 10.0.18362

Are you talking about the "recording" mode in the keybindings editor? Can you give exact steps to reproduce this issue?

hi
sorry for my bad english.
yes i'm takling about recording mode in keybindings editor
step:
1.opening keyboard shortcuts
2.pressing record keys button to record a hotkey for example :ctrl + s
3.turning off and again on
4.after step 3 when i turn the recording mode off ,text input section only accept a hot key

@roblourens
This is what happening in my environment.

Start with command $ code --disable-extensions.
Enable screencast mode.
See the below screencast.

Version: 1.39.2
Commit: 6ab598523be7a800d7f3eb4d92d7ab9a66069390
Date: 2019-10-15T15:33:00.827Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0

this is a dupe of #82714 (or vice versa, the other one is still open)


pulumi-bin: 3.134.1 -> 3.135.0

https://github.com/pulumi/pulumi/compare/v3.134.1...v3.135.0
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 345896 run on aarch64-darwin 1

Closing this one in favor of https://github.com/NixOS/nixpkgs/pull/347446


[My Space] Incorrect pop-up message when trying to add a seventh habit

Environment: Windows 10 Pro  (64 bit)
Chrome 128.0.6613.113
Reproducible: always.
Preconditions

User must be logged in.
The user must have 6 habits in progress.
The «My Habits» page must be open.

Steps to reproduce

Click on the button "Add New Habit"
Choose any new habit from the list and click "Add habit"

Actual result
A pop-up appears with the message: " You already have assigned this habit”
Expected result
A pop-up should appear with the message: " You can only have 6 habits in progress”
User story and test case links
E.g.: "User story #1001
Labels to be added
Priority: medium Severity: low Type: UI

When user tries to add a sixth habit pop-up message " You can only have 6 habits in progress” appears and user can have only 5 habits in progress. This message should appears when user attempts to add seventh habit. User should be able to add 6 habits in progress, according to the user story #1001.
https://github.com/user-attachments/assets/49e5cd68-f493-4aad-9e84-5319e56f6013

https://github.com/user-attachments/assets/458fb5ba-27d7-4dc5-b2ca-3b1c969a741b
Doesn't reproduce in my case


Videomega not working

Hi,
Videomega has recently changed it's embedded url. The format has changed to
http://videomega.tv/validatehash.php?hashkey=uniquehashkeyof60numbers
Please check.

Logs -
D:>youtube-dl.exe --verbose http://videomega.tv/validatehash.php?hash
key=099067089084066071100068098077077098068100071066084089067099
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'http://videomega.tv/validatehash.php
?hashkey=099067089084066071100068098077077098068100071066084089067099']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2015.04.28
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[generic] validatehash: Requesting header
WARNING: Could not send HEAD request to http://videomega.tv/validatehash.php?has
hkey=099067089084066071100068098077077098068100071066084089067099: HTTP Error 40
4: Not Found
[generic] validatehash: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPErro
r()); please report this issue on https://yt-dl.org/bug . Make sure you are usin
g the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl
with the --verbose flag and include its complete output.
File "youtube_dl\extractor\common.pyo", line 315, in _request_webpage
File "youtube_dl\YoutubeDL.pyo", line 1693, in urlopen
File "urllib2.pyo", line 410, in open
File "urllib2.pyo", line 523, in http_response
File "urllib2.pyo", line 442, in error
File "urllib2.pyo", line 382, in _call_chain
File "urllib2.pyo", line 629, in http_error_302
File "urllib2.pyo", line 410, in open
File "urllib2.pyo", line 523, in http_response
File "urllib2.pyo", line 448, in error
File "urllib2.pyo", line 382, in _call_chain
File "urllib2.pyo", line 531, in http_error_default

HTTP Error 404: Not Found.

I copied the url from here

it works in the browser..the embedded player, but not working with youtube-dl.

Post the source URL where it is embedded.

http://videomega.tv/?ref=JWCD9PWUR77RUWP9DCWJ

Does not work for me neither in Firefox nor Chrome (direct/proxy).

Should be fixed in the next version.


PANIC: bad argument #1 to 'open' (filename invalid)

I get a "filename invalid" with the current dev branch of nodemcu-firmware (nodemcu/nodemcu-firmware@f5acf996208515eef3ef22c6ed9846953640e8a6) and the current master branch of nodemcu-uploader.
$ python nodemcu-uploader.py --port /dev/ttyUSB0 --verbose --baud 9600 upload --compile --restart alarms.lua
[...]
Transfering alarms.lua as alarms.lua
write: recv()

expect returned: `file.remove("alarms.lc")
> recv()
C> `
sending destination filename "alarms.lua"
write binary: 61:6c:61:72:6d:73:2e:6c:75:61:00
waiting for ack
ack read 50
did not ack destination filename
Compile alarms.lua
write: node.compile("alarms.lua")

expect returned: `ANIC: unprotected error in call to Lua API (stdin:1: bad argument #1 to 'open' (filename invalid))

I formatted the file system after flashing the firmware.
7c7026491a4a730965903d7eb848a47f4e628195 fixes it for me, but I don't know why (no time to do a git bisect on nodemcu-firmware now).  That is why I didn't open a pull request.  Nonetheless, I hope the patch points in the right direction.

I will not change the zero byte marker... It must be firmware related and for the moment I do not have much time to support the dev branch of the firmware. Help is much appreciated.

Same here.
It seems bug in string.gsub() API in NodeMCU firmware...
I tested it with this code:
a = '\000'
print(a == '\000', 'should be "true"')
print(a == '',     'should be "false"')

b = string.gsub(a, '\000', '')
print(b == '\000', 'should be "false"')
print(b == '',     'should be "true"')

When running on my local machine (Ubuntu, Lua5.3), I got a correct result.
$ lua5.3 test.lua
true	should be "true"
false	should be "false"
false	should be "false"
true	should be "true"

But if I run this code on NodeMCU (via nodemcu-uploader terminal command) I got
> a = '\000'
> print(a == '\000', 'should be "true"')
true	should be "true"
> print(a == '',     'should be "false"')
false	should be "false"
>
> b = string.gsub(a, '\000', '')
> print(b == '\000', 'should be "false"')
true	should be "false"
> print(b == '',     'should be "true"')
false	should be "true"

From the test code above, it seems NodeMCU dev version does not handle NULL byte correctly in string.gsub().

I will have to rework the code to make it work with some changes in upstream.
gsub does not replace all strings, nice.

Please test the latest "next" as I just commited some changes that might work.


Improved smart indenting in XML completion.

I've modified the behaviour when pressing enter on XML files. The current system has some problems and may result in files with some erratic indentation, especially with multi line tags.
The new indentation mostly mimics the style of some IDEs like Visual Studio, others like Eclipse-based ones are similar but have some slight differences (if there are multiple tags in the same line or if a tag spawning to multiple lines has different indentation on each one).
I'd say the only key difference with other IDEs is the last Enter shown in the gifs, while the new code always adds a new line, other IDEs may just reindent the following closing tag. I feel adding a new line is better.

The GIF quality is not the best and there are some artifacts, but I think it shows the improvements well. Not all cases are shown.
One thing I don't like of the current indentation system is that it requires several undo steps for getting the original typed code, but that's not a problem of the new code (although maybe avoiding calls to SetLineIndentation when not needed may improve things just a bit), and decided to leave it as is right now.

@Neverbirth I think your formatting system would be a lot smarter if it would account to the fact that closing tags is optional in html5: ie <meta ...>, <img...>, etc. don't need the />. This is an information that you should have access to as "leaf" tags are annotated in the HTML completion support.

I'll try to make some investigation and tests. Strictly speaking that shouldn't be here, as this is generic XML formatting and closing tags is mandatory, Scintilla has some different syntax for HTML , I don't know how FlashDevelop behaves in that way, is that one used instead of the XML syntax? if so it could be checked, but since FD uses an old version I wouldn't be surprised if there are some problems there. I won't be able to look into this today tho.
I've also noticed yesterday at night a slight problem in my code, nothing serious , in most cases it won't even appear, and thought about adding some extra comments for other people as well.


Unable to download credential with infant's UIN.

Describe the bug
Unable to download credential with infant's UIN.
Prerequisite
Infant's UIN from Reg client<PHONE_NUMBER>
To Reproduce

unlock the app
click on Add ID
Enter UIN and click on Generate ID
Enter otp

Current behavior
Displayed error as "Unsupported Authentication Type for child-otp".
Expected behavior
If Otp Authentication is not supported for a child then app should provide other possible authentication types.
Screenshots

Smartphone (please complete the following information):

Device: iPhone7 & iPhone XR
OS: 15.4.1
Inji app version: 0.4.0 (16) in test flight
MOSIP Version: 1.2.1
MOSIP server: qa-121.mosip.net

@Anushree09-N @Raginikrishnamurthy
There is a child auth filter that is configured in https://github.com/mosip/mosip-config/blob/develop-v3/id-authentication-default.properties
# Child Auth Filter configurations
ida.child-auth-filter.factors.denied=otp,bio
ida.child-auth-filter.child.max.age=5

This config controls what auth types are allowed for age groups i.e infant in this case.

Verified in:
iPhone 8
iOS version 16
BLE 5.0
Inji app version: 0.4.1 (25) -> Testflight
MOSIP Version: <IP_ADDRESS>
MOSIP server: qa-1201-b2.mosip.net
Current configured value is below:
https://github.com/mosip/mosip-config/blob/qa-1201-B2/id-authentication-default.properties
Child Auth Filter configurations
ida.child-auth-filter.factors.denied=otp,bio
ida.child-auth-filter.child.max.age=5
Working as expected , below is the test data with results :

Tested results have been discussed with @rakhimosip too.
Commit Id of verified app:


Add Hunzaboy to Contributors List

@hunzaboy, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?

The tutorial was great and the steps were easy.
So this was my first every accepted merge.
Feeling excited :+1:

I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com


NAME_OF_YOUR_PROJECT

write a pseudocode for a program that sets a variable named username as "MyUserName" and another variable named password as "MyPassword" then asks the user to enter the username and password. If the correct username and password are entered the program should display "Logged in" otherwise the program should display either "Incorrect Password" or "Incorrect Username" as appropriate. It should also allow the user to make another attempt at entering the combination. The user should be allowed a maximum of THREE attempts to get the correct password.
How would you describe your project?
RADHMATT WIFI
What problem does this project solve?
security
Who is the target market?
ANSWER HERE
How will this project benefit the network and/or the community?
ANSWER HERE
How is this project centered around thenewboston?
ANSWER HERE
What is the estimated completion date for this project? (rough estimates are acceptable)
ANSWER HERE
Please include a detailed roadmap for your project below. This must include a list of all milestones where payments will be awarded upon completion of each.
ANSWER HERE
What is your Discord username?
DISCORD USERNAME HERE (https://discord.gg/thenewboston)
Additional files and documentation including architectural diagrams, wireframes, research documents, visuals, etc… (optional)

OPTIONAL ATTACHMENT HERE
OPTIONAL ATTACHMENT HERE
OPTIONAL ATTACHMENT HERE

Milestone Details
Please include a detailed breakdown of the first milestone including a list of all quantifiable objectives. Upon
completion of the milestone, each objective will be reviewed by thenewboston team and once all objectives have been
verified as completed, coins will then be rewarded.
Note that all future detailed milestone breakdowns will be written as a separate GitHub issue. Only the first milestone
is included in this project proposal.

First Milestone Description
SHORT DESCRIPTION OF MILESTONE HERE
First Milestone Objectives

#
Objective
How can auditors verify milestone?

1
OBJECTIVE DESCRIPTION
EXPLANATION OF HOW AUDITORS CAN VERIFY

2
OBJECTIVE DESCRIPTION
EXPLANATION OF HOW AUDITORS CAN VERIFY

3
OBJECTIVE DESCRIPTION
EXPLANATION OF HOW AUDITORS CAN VERIFY

How long do you estimate this first milestone will take to complete?
ANSWER HERE
Project Wallet
PROJECT WALLET ACCOUNT NUMBER HERE

@radhmatt
TNB Projects welcome people to come up with ideas that have the potential to add value to thenewboston and TNB will help with funds to build them up.
Learn more about projects by clicking here


Add gce_init.sh startup script and revise instructions

Added a startup script, so the user can launch Nephio Sandbox in a single gcloud command.

cc @electrocucaracha @vjayaramrh @radoslawc
The actual command won't work until the PR merges, because it tries to pull the script from GitHub. If you want to try it, then pull out the PR and run it using gcloud compute instances create --machine-type e2-standard-8 --boot-disk-size 200GB --image-family=ubuntu-2204-lts --image-project=ubuntu-os-cloud --metadata-from-file=startup-script=./gce_init.sh nephio-r1-e2e.

cc @electrocucaracha @vjayaramrh @radoslawc
The actual command won't work until the PR merges, because it tries to pull the script from GitHub. If you want to try it, then pull out the PR and run it using gcloud compute instances create --machine-type e2-standard-8 --boot-disk-size 200GB --image-family=ubuntu-2204-lts --image-project=ubuntu-os-cloud --metadata-from-file=startup-script=./gce_init.sh nephio-r1-e2e.

@johnbelamaric Will review

Reviewed, looks good, could not test out the script though
/lgtm

Closing in favor of #56.


queryAABB does not work

According to box2d documentations this code shoud work but nothing gets logged.
class Game extends Forge2DGame{

  Game(): super(zoom: 1, gravity: Vector2(0, 0));

  @override
  Future<void>? onLoad() async{
    super.onLoad();
    add(Planet(Vector2(100, 50)));
    add(Planet(Vector2(700, 300)));
    add(Player());
    world.queryAABB(MyCallBack(), AABB.withVec2(Vector2(-2000, -2000), Vector2(2000, 2000)));
  }
}

class MyCallBack extends QueryCallback{

  @override
  bool reportFixture(Fixture fixture) {
    print("Called");
    return true;
  }
}

class Player extends BodyComponent{
  @override
  Body createBody() {
    var bodyDef = BodyDef(type: BodyType.dynamic, position: Vector2(320, 220), linearVelocity: Vector2(-10, -10));
    var body = world.createBody(bodyDef);
    var shape = CircleShape();
    shape.radius = 15;
    body.createFixtureFromShape(shape);
    renderBody = true;
    return body;
  }
}

class Planet extends BodyComponent{
  final Vector2 position;
  Planet(this.position);

  @override
  Body createBody() {
    var bodyDef = BodyDef(type: BodyType.static, position: position);
    var body = world.createBody(bodyDef);
    body.createFixtureFromShape(CircleShape()..radius = 50);
    return body;
  }
}

There is clearly overlap between game objects and queried aabb but reportFixture never gets called. Have I missed something?

I think I found the problem
 world.queryAABB(MyCallBack(), AABB.withVec2(Vector2(-2000, -2000), Vector2(2000, 2000)));

It seems like you can't make this call in onLoad method. when I move this part to onMount or any other lifecycle methods, It works.


Vagrant shared folder?

I'm trying to build y app in a Vagrant box with shared folder between the VM and my local computer. However Fresh doesn't seem to rebuild when I edit files locally.
Has anyone experienced something similar?

I think that it's not a problem of fresh.
In vagrant you can synchronise you folder with different mechanisms; I've tested 2, the native virtualbox mechanism and NFS.
I've discarded the native one, it was so slow when I was accessing to the files inside the virtual machine.
NFS works quite well for those reading and writing files, however if the modification of the files are in the host machine, the changes are received in the virtual machine with some delay.
In my case, I've seen how other watchers take almost nothing for the first one or two writes, but after the delay increases up to 20 seconds, so it's not useful as you have to wait 20 seconds each time that you write in a the directory which is watched.
I don't face this problem that much because, I work inside the virtual machine, connected by ssh, therefore I work through the terminal, however I took a look a little bit, long time ago because some colleagues work in the host machine, but I couldn't find anything that allows to force the synchronisation or change the scan rate of NFS.
Another solution that some people use, although I haven't tried, is to use rsync as a synchronisation mechanism, and it may work because rsync with a local ssh to transfer the most of the times a small change in file should be very fast, however I don't like the solution because I have to copy all the projects witch I'm working from the host machine to the virtual machine, increasing significantly the virtual disk space.
If you try rsync, please let us know if works for this cases (watchers) in acceptable conditions.

Works fine with vagrant rsync-auto running.

@evt brilliant, thanks for letting us know.


Scenarios to prefer over cupy

Cupy is quite stable and efficient "numpy for GPU" (which has no restrictions mentioned in readme), chainer over cupy provides necessary audo-diff and primitives for deep learning. There are also other alternatives.
It would be nice to have showcases when jax is expected to be beneficial compared to already existing tools.
Thanks!

Thanks for the question! Clearly that's an important one for users to ask themselves as they consider what libraries to use to do their work.
However, instead of making comparisons to other libraries, we'd rather focus on explaining what JAX can do. At its core, JAX is about transforming functions, including via forward- and reverse-mode autodiff, jit compilation with end-to-end XLA optimization to multiple backends, and vmap for things like batching and per-example gradients. JAX makes all those transformations arbitrarily composable, which is how we build compound transformations like fast Hessian calculations (which might use one level of forward-mode autodiff, one level of reverse-mode autodiff, two levels of vmap, and a level of jit):
from jax import jit, jacfwd, jacrev
def hessian(fun):
  return jit(jacfwd(jacrev(fun)))

So while JAX provides a NumPy API (like many libraries do!) and, because we use jit in library functions, it can execute code on accelerators for individual calls, that's really just the starting point. JAX is about compositional transformations.
Hopefully the README provides more detail on what JAX can do. We'd like to keep improving the README, so if you have questions about it or suggestions on how to make it more clear, we'd love to hear them! Having a "showcase" is a great idea, and we're working on more examples. But we'll leave comparisons to specific libraries up to the users.


does not recursively kill child processes on exit

If a process that winSW wraps executes a child process, when winSW kills the wrapped process, the child process trees are not terminated.
https://github.com/kohsuke/winsw/blob/master/src/Core/WinSWCore/Util/ProcessHelper.cs#L29
essentially calls to (Get-WmiObject Win32_Process | where { $_.ParentProcessID -eq PROCESSID }).Name
You can see that this does not execute recursively.
Proposed fix is to traverse the process tree, killing each visited node during traversal

It does kill processes recursively  iirc, but not all Windows tools have a valid process tree. E.g. cygwin


Topic 8

Topic 8

Looks good Luciano!
Please review my comments and try to test it
Please attach to the pull request the documentation related to point 1 (Using the same SQL-MySql data model, map it to the corresponding database, collections and documents).
I'm not seeing the point 4 in Courses (Add a new field to “courses” collection to identify which courses are finished)

Good job Luciano!!!


[FEATURE] Enforce IMDSv2 by default

Is your feature request related to a problem? Please describe
IMDSv2 enforcement is a security best practice. All AWS provided open-source projects should be secure by default and implement best practices. This project sets http_tokens to optional by default, which means that customers will end up deploying something that does not implement security best practices.  You can see the line here: https://github.com/aws-samples/aws-eks-accelerator-for-terraform/blob/429bd18d13e9c897ecf3b0a987359b7fc7bef000/modules/aws-eks-managed-node-groups/variables.tf#L82
Describe the solution you'd like
Enforce IMDSv2 by default by setting http_tokens to required by default.
Describe alternatives you've considered
Allowing customers to continue running insecurely by not changing this, which is harmful to AWS's brand and trust.
Additional context
https://github.com/SummitRoute/imdsv2_wall_of_shame

Thanks for raising this issue @0xdabbad00 👍🏼   We will push a hot fix for this soon


Always mkdir with parents

On a fresh clone of the repo when the ./data directory has not been created yet, some of the custom datasets may fail because they use mkdir without parents=True.
For example,
python main.py --model gpt2 --device cuda:0 --tasks logiqa

Results in
FileNotFoundError: [Errno 2] No such file or directory: 'data/logiqa'

This PR simply ensures that all mkdir commands (both from os and pathlib) create the necessary parent dirs.

Hey, sorry for taking so long to get around to this - @jon-tow and I have been busy tracking down a bug breaking our CI builds so we can get CI on our PRs, and we think we have it working now - once these tests pass I'll merge this PR


Remove bugs in CalloutStyleRenderer

Remove some bugs in CalloutStyleRenderer where padding isn't calculated correct.
In this case, the map regression test image has changed. It now has attribution (don't know why this wasn't the case before). But also the callouts have changed, and I don't know, why. The callouts are created by random numbers and should always be the same.

In this case, the map regression test image has changed. It now has attribution (don't know why this wasn't the case before). But also the callouts have changed, and I don't know, why. The callouts are created by random numbers and should always be the same.

I had seen this before and assumed it was fixed by the sorting on the ConcurrentBag, but apparently it wasn't. Perhaps there is a ConcurrentBag involved somewhere else?


Correctly support origin_dns in spectrum application

Currently the origin_dns attribute is defined as
type SpectrumApplicationOriginDNS struct {
	Type  string   `json:"type"`
	Names []string `json:"names"`
	TTL   int      `json:"ttl"`
}

However this results in a bad request 400
Proposal
creating a spectrum application in the UI and connecting it to a load balancer results in the following json
{
            "id": "5683dc9a12ba4dc6bceaca011bcafcf5",
            "protocol": "tcp/22",
            "dns": {
                "type": "CNAME",
                "name": "ssh.backofficetechnology.co.uk"
            },
            "origin_dns": {
                "name": "ssh.origin.backofficetechnology.co.uk"
            },
            "origin_port": 22,
            "ip_firewall": true,
            "proxy_protocol": false,
            "tls": "off",
            "created_on": "2018-11-19T16:38:40.797134Z",
            "modified_on": "2018-11-19T16:38:40.797134Z"
        }

This suggests the struct should be defined as follows:
type SpectrumApplicationOriginDNS struct {
	Name  string   `json:"name"`
}

I've test that this works both in postman using the resulting json and in this related PR: https://github.com/terraform-providers/terraform-provider-cloudflare/pull/156
I will open a PR shortly

Thanks! Looks like there are tests missing for this specific field too which explains why CI hasn't picked up on it.
Let me know if you don't get around to the PR today. I'm doing some work on our Spectrum app configuration and happy to address this.

@jacobbednarz i've added a test and PR #253 . I think i've copied the style of the project 😄 . Please let me know if it's not correct

👏 this is great. thanks!


fiatTotal has exponential syntax in JSON

The /balances endpoint returns JSON with values in the exponential format:
{
    "fiatTotal": "6.269762685290865e+43",
    "items": [{
            ...,
            "fiatBalance": "6.269762685290865e+43",
            "fiatConversion": "2.3545769628590505e+38"
    }]
}

E.g. https://safe-client.safe.global/v1/chains/137/safes/0x828d55AaF0fE916879C7dCf0E3278D1E8dbEDDd3/balances/usd?trusted=true
This causes parsing errors in the web app that crash the whole site.

Also, we don't need such huge precision, those values can be rounded to 3 decimals (JP yen requires 3, most currencies just 2).


Server side rendering broken with 2.29.0+

We use the library in a SPA using server side rendering. Due to the implementation of the fix for #97 in js/load-image-orientation.js, the library is broken in this case as the global document object is by default not available on the server.

Thanks for the report @cberg-zalando.
Unfortunately this cannot be lazy loaded, as the check needs to run before any scale() call, which is synchronous.
I'm assuming you're doing isomorphic/universal rendering (including the same JS code on both client/server).
In your use case, would the test code still run on client-side if I add a gating check for the document object?

Yep. It is a React based application and as we are using hydrate for the server side rendering, the created HTML is sent to the client and then the hooks, etc. are created. So I assume it also means that this part will also run in the browser.

I've released<EMAIL_ADDRESS>Please let me know if this resolves the issue.

Yep, its working. Thanks for the quick fix.

Thanks for the confirmation. ☺


Re-purchase of entitlement results in Response Code 3

From what I understand Google no longer saves test purchases, so there is no way to cancel them from the Payment Center. When I try and re-purchase an Entitlement, I get a Response Code 3:  This billing API version is not supported for the type requested. Shouldn't I receive a pop-up stating I already own the item? This occurs regardless of the Google Play account used, as long as the account is linked to the device that I made the test purchase on. If I remove that account everything works again.
I tried using this, but it didn't work:
PurchaseManager mananager = PurchaseSystem.getManager(); if (manager instanceof PurchaseManagerTestSupport) {     ((PurchaseManagerTestSupport) manager).cancelTestPurchases(); }

Are you using gdx-pay-android-googleplay or gdx-pay-android-openiab?

I am using gdx-pay-android-googleplay, the same code as in the wiki with the obvious changes. Currently I work around it by setting the offer as a consumable. I noticed that:  <if(manager instanceof PurchaseManagerTestSupport)> always returns false (according to Android Studio). So their also needs to be a better way to remove test purchases (Google support recommended just consuming it), although I'm not sure why no pop-up appears stating I own the item, it just throws the handlePurchaseError.

What is the output of:
System.out.println("Purchase Manager class:  "+ manager.getClass().getName());

com.badlogic.gdx.pay.android.googleplay.AndroidGooglePlayPurchaseManager

Which version of gdx-pay are you using?
Op 7 jun. 2017 01:51 schreef "Matt"<EMAIL_ADDRESS>
com.badlogic.gdx.pay.android.googleplay.AndroidGooglePlayPurchaseManager
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/libgdx/gdx-pay/issues/149#issuecomment-306646265, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0u3jG1V4peXaDc5-3UJ5zHdqhcSzhAks5sBeX1gaJpZM4NxVSR
.

I am using gdx-pay 0.11.1 on all submodules. Maybe I should also note I am using the latest version of Android Studio 3.0, I'll try testing it out again.

I have the same problem.

libgdx 1.9.6
gdx-pay 0.11.1
Android Studio 2.2

Any news on this? I also get the RESULT_BILLING_UNAVAILABLE = 3 response when I try to purchase an entitlment that was already purchased instead of getting the correct RESULT_ITEM_ALREADY_OWNED = 7 response. I use gdx-pay 0.11.2

Hi, when having problems, please try if your problem is solved by using googlebilling.


[RESTEASY-1721] Eliminate ContextParameterInjector's dependence on se…

…rvlet classes
[RESTEASY-1721] Remove another reference to ServletOutputStream.

 The quarkus test run discovered that I have a reference to ServletOutputStream in ContextParameterInjector, and, in native mode, the servlet classes are unavailable. D'oh. Could you take a look at this fix? ContextParameterInjector loads ContextServletOutputStream only if the servlet classes are available. It's like the MP Config stuff.
-Ron

Hmmm. Works at home ... .

Doesn't work.


Update Nginx AMI for #171

I forgot to add it to the pull request.

Where do you get the AMI reference number from? PR looks good, but I have no idea if it's the right number or not.

:+1:

@TheDoubleK I built the AMI using the packer command in the README.  At the end of the process it gives you an AMI ID that can be used.

@TheDoubleK you can find the AMIs in AWS Console > EC2 > Images > AMIs. Right now all images are created on the development AWS account.

Ah, OK yes should have read the manual...


deploy fails with ROLLBACK_COMPLETE error

You can go  aws CloudFormation to check the specific failure reason

Thanks @pointone-tww. Without more details, I can't help here.


Run test.py encounter TypeErrpr:  'type' object is not subscriptable

I modified esrgan_config.py per https://github.com/Lornatang/ESRGAN-PyTorch#test and I get the following error when running test.py:
  File "test.py", line 21, in <module>
    import imgproc
  File "/home/ESRGAN-PyTorch/imgproc.py", line 513, in <module>
    center: tuple[int, int] = None,
TypeError: 'type' object is not subscriptable

My python version is 3.8.10, Torch 1.13.1

Fixed.


tdTomato

Hi, tdTomato is shown in the graph but data are not shown in the table. Any reason? Or is it shown somewhere else?

It's probably a bug. That said, I am not maintaining this anymore and it is superceded by https://www.fpbase.org/ which is being actively updated and is much more fully featured.


Fail to Authenticate when using with secure solr with kerberos

Hi
i have getting issue when I am using hbase-indexer with secure solr using KERBEROS
i have added keytab file in hbase-indexer-site.xml but its still not authorized to access solr
cloud.ConnectionManager: Watcher org.apache.solr.common.cloud.ConnectionManager@1ee44ef7 name:ZooKeeperConnection Watcher:server1:2181/solr got event WatchedEvent state:SaslAuthenticated type:None path:null path:null type:None
17/02/22 14:26:15 ERROR impl.CloudSolrClient: Request to collection hbaseindex failed due to (401) org.apache.solr.client.solrj.impl.HttpSolrClient$RemoteSolrException: Error from server at http://server1:8983/solr/hbaseindex_shard1_replica1: Expected mime type application/octet-stream but got text/html.

Error 401 Authentication required

HTTP ERROR 401
Problem accessing /solr/hbaseindex_shard1_replica1/update. Reason:
    Authentication required

When using with secure solr5.5 or solr6.3 with kerberos, you should use httpclient-4.4.x,  and add -Djava.security.auth.login.config=${HBASEINDEXER_CONF_DIR}/jaas.conf to your java process. Furthermore, a few code should be added as follows:
HttpClientUtil.setConfigurer(new Krb5HttpClientConfigurer());
This problem is resolved by solr6.5, is a solrj issue.


Join fields lost when using eager

Model.query().select('pets.*')
returns correctly:
[{
    id_pet: 1,
    name: 'One'
},
{
    id_pet: 2,
    name: 'Two'
}]

When using eager like:
related: {
    relation: Model.HasOneRelation,
    join: {
        from: 'pets.id_pet',
        to: 'something.id_pet'
    }
}
Model.query().select('pets.*').eager('related')

This returns wrong, with missing field:
[{
    name: 'One',
    related: [{
        id_pet: 1,
        related_data: 'abc'
    }]
},
{
    name: 'Two',
    related: [{
        id_pet: 2,
        related_data: 'xyz'
    }]
}]

In this example, field id_pet is present in both main table and relation table, and when using eagers it disappears from main table's result.
Note: this is an example, you can try with BelongsToOneRelation or ManyToManyRelation, issue is the same.
Note 2: Just updated objection from 0.6.2 to 0.7.10

I just tested downgrading back to 0.6.2, and then upgrading progressively to 0.7.0, then 0.7.1, then 0.7.2 ...
The issue appears when upgrading to version 0.7.4

Does it work if you leave out the select pets.*? I'm assuming that Model.tableName is "pets" in your example?

Could you chain .debug() method to the failing query and post the generated sql here?

Yes, Model.tableName is "pets".
Yes again, if I remove select pets.* it works. (However I need it to select other fields, pets.* is a simplification).

Debug:
{ method: 'select',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [],
  __knexQueryUid: '6026bdb3-ac37-485b-9920-8da83b888b6b',
  sql: 'select `pets`.*, `pets`.`id_pet` from `pets`' }
{ method: 'select',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings:
   [ 1,
     2 ],
  __knexQueryUid: '75066de3-e48c-4735-b27e-38a5ea11dad3',
  sql: 'select `something`.* from `something` where `something`.`id_pet` in (?, ?)' }

Yeah, both pets.* and a list of columns should work. I was just ruling out a possibility. Seems like a bug that really shouldn't have ended up in a release. I'll try to reproduce and fix this immediately.

I'm not able to reproduce this. I tried that exact query with all different relation types and got all columns back. Could you please write a standalone app that reproduces the problem? There is something in your setup that is triggering this bug that you haven't mentioned here.

For some reason that I cannot figure out the implicit pets.id_pet select appears in the query, even though pets.* should already select the id. For some reason objection thinks the query doesn't select the needed columns. It then adds those columns to the select clause and removes the values afterwards (the feature added in 0.7.4). Is it possible that you have a typo in Model.tableName for example an extra space or something like that?

I agree with your comment: if I debug the same query with 0.7.3 I see
 sql: 'selectpets.* from pets' }
in place of
 sql: 'selectpets.*, pets.id_petfrompets' }
I (nor anybody here) can't see any typo in the code.
However we are prefixing table names with database name (since we are merging tables cross different databases), so it actually looks more like:
 sql: 'selectzoo.pets.*, zoo.pets.id_petfromzoo.pets' }
Note: pets and something are in the same zoo database.

The prefixing is the key here :D. Maybe you could have included that in the original example. Using a dot as a separator is not a good idea. I'm pretty sure many other things with objection and knex will blow up as they interpret anything coming after the first dot as the column name. Could you use for example : as the separator?

Sorry for not including that in original example, I couldn't figure out the problem could be related to that :-) I think I can't change the dot, because of sql (the dot is not inside the table name, it's to separate DB name and table name); actually if I replace it I got this:
  sql: 'select `zoo:pets`.* from `zoo:pets`' }
Error: ER_NO_DB_ERROR: No database selected

The extra id_pet has disappeared, but ...

Oh, sorry! I thought you had just prefixed your table names. I can fix this particular case, but I'm pretty sure you will run into others that I cannot fix because of knex. I have never used multiple databases like this with mysql. Maybe you could use the schema method? That's a knex feature for postgresql schema's which is a similar feature. Objection supports that. Something like this:
Model.query().schema('zoo').select('pets.*').eager('related')

I don't know if that works with mysql, but you could give it a try. Anyway I'll try to fix the issue at hand if I can do it without breaking anything else.

@adiamini 0.7.11 is out with this one fixed. I did some testing and you may not be as screwed with the cross-db dot separated queries as I thought. Everything I tried just worked.

Thank you ! I just upgraded to 0.7.11 and tested it and there are cases when it doesn't work: if I use select('*') it's ok, but if I use select('pets.*') it doesn't (I can see it still adds the extra id_pet field in debug).

If your table is zoo.pets and you select pets.* it won't work, because objection cannot know if there is a pets table in your "main" database your knex is connected to. select('zoo.pets.*') should now work.

Oh, right ! I can confirm that adding the DB name in the select it works. Thank you very much ! :-)

But column names should work without table name! For example select('foo', 'bar') should work now.

Yes of course. ... but I need table name because I'm getting data from joined tables too :-)


[NIFI-10876] Made changes to convert from JUnit 4 to JUnit 5

Summary
NIFI-10876
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 8
[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

@dan-s1 You also have a merge conflict. Do a rebase (ie don't do merge main into your branch).

@dan-s1 After rebasing you will need to use --force with git push to override the remote version with your current local version.

@dan-s1 After rebasing you will need to use --force with git push to override the remote version with your current local version.

Thanks! Done.

Thanks for the updates @dan-s1, it looks like the latest commit build failed on the following unit test:
Error:  org.apache.nifi.authorization.AuthorizerFactoryTest.testUpdateUserWithSameIdentity  Time elapsed: 0.511 s  <<< ERROR!
java.lang.IllegalStateException: User/user group already exists with the identity 'xyz'.
	at org.apache.nifi.authorization.AuthorizerFactory$1$1$1.updateUser(AuthorizerFactory.java:253)
	at org.apache.nifi.authorization.AuthorizerFactoryTest.testUpdateUserWithSameIdentity(AuthorizerFactoryTest.java:216)

Fixed. I forgot to remove those lines after I placed in assertThrows.

@exceptionfactory Look like all tests have passed now


Add documentation with user guide and examples

Docs are clearly needed to make the package useful by others.

This was fixed by #58


Allow dynamic callback_url and client_class class attributes

Callback urls require being absolute, which is pretty cumbersome espacially when dealing with differents environments (dev/staging/production for instance). Being able to provide dynamic callbacks url hence allows for things like this:
class GitHubLogin(SocialLoginView):
    adapter_class = github_views.GitHubOAuth2Adapter
    client_class = OAuth2Client

    # use the same callback url as defined in your GitHub app, this url must be
    # absolute
    def get_callback_url(self):
        return self.request.build_absolute_uri(reverse('github_callback'))

urlpatterns = [
    ...,
    path('auth/github/', GitHubLogin.as_view()),
    path('auth/github/callback/', github_callback, name='github_callback')
]

Which really comes in handy in multiple environments.
I also changed client_class the same way for code homegeneity concerns (even though I did not have to use it in my own projects).
The "basic way" of declaring these will of course still work:
class GitHubLogin(SocialLoginView):
    adapter_class = github_views.GitHubOAuth2Adapter
    client_class = OAuth2Client
    callback_url = 'http://localhost:8000/auth/github/callback/'

Coverage decreased (-0.2%) to 96.186% when pulling d7532b875ed77f9fda94863aa400b3d6d926a64a on michaeldel:master into 95fafe5e0f6716296a1c664c2b870876a6b4e0cc on Tivix:master.

Hi,
as a user of rest-auth, thanks for the contribution!
This repo is not maintained anymore, so the development moved to dj-rest-auth. (reference: https://github.com/Tivix/django-rest-auth/issues/568)
It may be best, if you move this PR there. (and upgrade to using dj_rest_auth)
new repo link: https://github.com/jazzband/dj-rest-auth (I'm not the upkeeper of that repo, it just makes sense for me to help you merge your PR)
Many Thanks,
Barney

@BarnabasSzabolcs thank you for suggesting it, I was not aware of this fork. PR has been submitted on this new fork here: https://github.com/jazzband/dj-rest-auth/pull/80

Thanks Michael!


20MB binary file included in repo now

terrascan version: n/a
Operating System:n/a

Description
It looks like on the 1.0 update (https://github.com/accurics/terrascan/pull/284) that a 20MB binary file slipped it and is now in cmd/terrascan/terrascan It seems like this should be removed.  It's probably too late to clean up history and reduce the repo size, but at least avoiding having it checked out on the mainline branch seems like a good cleanup.
What I Did
MacBook-Pro:terrascan user$ file cmd/terrascan/terrascan
cmd/terrascan/terrascan: Mach-O 64-bit executable x86_64
MacBook-Pro:terrascan user$ ls -l cmd/terrascan/terrascan
-rwxr-xr-x  1 user  staff  20938316 Oct 26 11:08 cmd/terrascan/terrascan

Good catch. Thanks!


Still maintained?

Hi there
This isn't an issue per se, but I would love to know if this is still being maintained and do you plan on keeping it up to date or adding any new features?
Thanks

Hi there, unfortunately I'm lacking time with other projects. I have plans to reboot it from scratch but, time's lacking 😢
It does work though, but you have to edit the caddy config in yml or json in that web ui which is not super super trivial.
Let me know if you need help setting it up 😉

Thank you! I just gave it a go and got it working. It looks good. If you do rebuild it I think it would be cool to be able to add records with a nice form-style UI

Thanks for closing!
Just reopening to:

update deps
update readme to say it works although no dev is being done

Haha fab! Thanks 😊


Update PythonPrograms.tex

Correct two labels.

Update PythonPrograms.tex

@liqihao2000 Thanks for your contribution.

@liqihao2000 Thanks for your contribution.


Enhance conventions

Allow FRA and deposit conventions to be looked up by name.
Rename Convention to TradeConvention and rename Template to TradeTemplate.
Add new Term Deposit conventions.
Tidy up CDS conventions.
Further work is needed on swap and CDS conventions

Merging for later review.


Fix a typo in the documentation

k4a_transformation_depth_image_to_point_cloud() transforms
the depth image into a single image, not 3 planar images
Signed-off-by: v-hkazakov<EMAIL_ADDRESS>
Fixes #1287
Description of the changes:

Fixed a typo in the k4a.h

Before submitting a Pull Request:

[x] I reviewed CONTRIBUTING.md
[x] I built my changes locally
[x] I ran the unit tests
[x] I ran the functional tests with a device
[x] I ran the performance tests with a device

I tested changes on:

[ ] Windows
[x] Linux

/azp run

/azp run

/azp run


Fixes bug #5289: Bars do not appear at correct X axis position when specified in {x, y} format

Fix #5289 by formatting the data if missing some points.

@wmzhong don't forget there are some comments on this PR

@wmzhong a quick reminder that there are some outstanding comments on this PR

@wmzhong a reminder to take another look at this PR when you get a chance

@wmzhong will you be addressing the outstanding comments? If you don't plan on updating this PR then we will close it as inactive

That would be a shame. But when the first review takes three months interest might have shifted 😔

Yeah, can definitely understand that the long review cycles are frustrating. We'd love help reviewing PRs if you're able to go through the pending PRs and help review.
To be clear, if we close any PR due to inactivity, it's always welcome to be re-opened in the future when the feedback is ready to be addressed. The main reason for closing inactive PRs is to help make it clear which PRs need reviews so that we can more efficiently get through them and try to cut down the review cycle

@wmzhong I really appreciate the contribution and I would like to see it make its way in, but it has been quite some time without a response. I'm going to close this PR for now, but please do update it and let us know when you do so that we can reopen it and finish getting it reviewed and merged. Sorry once again for the delay on the original review


Removing matthewscorrcoef & Restructure Docs

Description

removing matthewscorrcoef per https://github.com/fastai/fastai2/issues/439
attempting to resolve docs issue https://github.com/fastai/fastai2/issues/442

removed symlink docs and added empty docs dir.
added the appropriate _data/topnav.yaml file
_config.yaml is repopulated although I imagine you do not want this included in the PR
Trying to reason the changes made in move docs commits, will need some input/insight into how to resolve the above issue (442).

Comments

make test fails due to kernel naming issues (tests running notebooks require python3 environment)
It appears jupyter_client was also missing as a requirement for testing notebooks
02a_data.mixed.ipynb is being removed when running nbdev_update_lib..

I'll close this and fix locally, since we don't want those docs changes.


76 | Minimum Window Substring | C++

Title -
What will change - New Added for this problem will be added in C++.
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Can you assign me this issue?

@jaisaikuntala1 Please assign this to me

@jaisaikuntala1 Please assign this to me

I'll assign it to @jaisaikuntala1 - @Gyan843 I assigned you to #976


Add README.md

This PR adds the following information in README.md:

A concise description of the server.
Describe how to setup the development environment (install node dependencies, docker...).
Links for Github API v4 and GraphQL documentations.

Related to #11.
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Checklist:

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

The build has not passing, because i do not setted setup linter, neither tests.


unable to create mock provider for Jest+React testing Lib

So i am trying to setup up testing for my project and all works well except when i try to create a mock/test provider for our test suite.
(nothing too fancy just Next.js 11 + Jest + RTL)
The main issue is that i want to move away from mocking the return value of useStateMachine in favor of just letting the lib do its job and i test the result of the render.
This is also hindering my ability to test click events since they rely on LSM state changes. (ie hook form controlled by lsm).
So i have working providers for react query, i would assume that they would work the same. i did a test with just creating a React.Context  provider and that works just fine.
So the main error i am getting is that createStore({...mockStore}) comes back with
// works just fine with the query provider etc.
// When i add the provider to wrap other existing providers (ie query client for react-query) it fails

import * as React from 'react';
import { StateMachineProvider, createStore } from 'little-state-machine';
import { initialState } from '../../store/states';
import { QueryClient, QueryClientProvider } from 'react-query';

import { render } from '@testing-library/react';

const createTestQueryClient = () =>
  new QueryClient({
    defaultOptions: {
      queries: {
        retry: false,
      },
    },
  });

const mockStore = () =>
  createStore({
    ...initialState
  });

// HOC for rendering our ui components with the query client along with testing configs
export function renderWithClient(ui) {
  mockStore();
  const testQueryClient = createTestQueryClient();

  const { rerender, ...result } = render(
    <StateMachineProvider>
      <QueryClientProvider client={testQueryClient}>{ui}</QueryClientProvider>
    </StateMachineProvider>
  );
  return {
    ...result,
    rerender: (rerenderUi) =>
      rerender(
        <StateMachineProvider>
          <QueryClientProvider client={testQueryClient}>{rerenderUi}</QueryClientProvider>{' '}
        </StateMachineProvider>
      ),
  };
}

// Query Client Context wrapper for our tests, use this separately when we want to
// use it to render hooks outiside components for testing
export function createWrapper() {
  const testQueryClient = createTestQueryClient();
  mockStore();

  return ({ children }) => (
    <StateMachineProvider>
      <QueryClientProvider client={testQueryClient}>{children}</QueryClientProvider>;
    </StateMachineProvider>
  );
}
}

i get the below error, which basically tells me that little state machine is undefined, and checking console confirms same.
 TypeError: (0 , _littleStateMachine.createStore) is not a function

      27 |
      28 | const mockStore = () =>
    > 29 |   createStore({
         |   ^
      30 |     user,
      31 |     points,
      32 |     transactions,

      at mockStore (__mocks__/utils/index.js:29:3)
      at renderWithClient (__mocks__/utils/index.js:44:3)

This is how i would normally create all my mockProviders (for which all except LSM are 100% working)
Any ideas/thoughts? Has anyone created test suites with LSM provider?

describe('test', () => {
  it('should works', async () => {
    const App = () => {};
    const Root = () => {
      createStore({
        data: {}
      });

      return (
        <StateMachineProvider>
          <App />
          <App />
          <App />
          <App />
        </StateMachineProvider>
      );
    };

    render(<Root />);
  });

could you do something like above?

describe('test', () => {
  it('should works', async () => {
    const App = () => {};
    const Root = () => {
      createStore({
        data: {}
      });

      return (
        <StateMachineProvider>
          <App />
          <App />
          <App />
          <App />
        </StateMachineProvider>
      );
    };

    render(<Root />);
  });

could you do something like above?

I could try to setup a provider on each test, but the idea is to automate the boilerplate portion before each test. Works fine for all others except LSM. There's something i am missing here.

each test probably should have its own provider, what @JamesWong1999 provided is the correct approach.

each test probably should have its own provider, what @JamesWong1999 provided is the correct approach.

So i guess what I am confused about, how does this work for all other providers but not LSM. I can make this work with React.Context since it just accepts a store.
It doesnt even pick up the modules createStore function, so its techinically running on each test startup. (ie creates a new provider for each test case)

That's more of a testing lib question than LSM, LSM is just leverage on top of React context API.

i feel like this should work, it is pretty standard boilerplate for example:
function renderWithProviders(ui, { reduxState, locale = "en" }) {
  const store = createStore(reducer, reduxState || initialState);
  return (
    <ThemeProvider>
      <ReduxProvider store={store}>
        <IntlProvider locale={locale}>{ui}</IntlProvider>
      </ReduxProvider>
    </ThemeProvider>
  );
}

The redux example is not pretty much in line with what I was trying to do
I am going to try to refactor the _app.js providers into one <AllProviders > {children} </AllProviders> component and see if that can circumvent this issue

I finally re-visited the topic in order to resolve this outstanding item. Originally, I had opted for a workaround by completely mocking out LSM and its return values. Although this "works" and is very usefull when you just want to programatically manipulate component ui without any simulated interactions, this still did not resolve the root problem.
for example:
// mockLSM.js
import * as LittleStateMachine from 'little-state-machine';

  jest.mock('little-state-machine', () => ({
    useStateMachine: jest.fn(),
  }));

export const stateMachineSpy = jest.spyOn(LittleStateMachine, 'useStateMachine');

// overload the hook and allow me to return any value or action
export function useSpyStateMachine(state = {}, actions = {}) {
  return stateMachineSpy.mockReturnValue({
    state,
    actions,
  });
}

// then in a test we use it like so:

test('should do something', () => {
    useSpyStateMachine(
      {
        notification: {  // state.notification
          status: 'success',
          message:'it works!'
        },
      },
      {
        updateNotification: jest.fn(),  //actions.updateNotification
      }
    );

     // at this point the mock data is injected by jest and we can test results

    const { container } = render(<Comp />);
    // do some test stuff + assert
  });

This is fine, when doing unit testing or just trying to assert some branching logic that depends on the store's current state value . Unfortunately it was not enough since I could not properly render the app in the same way a user would experience it, which was the original goal (test behaviour based on user interaction with app) .
Again, the goal is provide a seamless, automated way to render components in the same way that the application would spin up; wrapped in LSM provider without having to add repeetitive boilerplate code to every test (we have over 600+).
So the issue boild down to this:
LSM, when rendered by RTL/Jest env=jsdom, returned an empty object on initialization since its running in nodejs first then loaded into jsdom.
Solution:

overload the rtl render method more deets here:
initialize the store in a wrapper that is rendered by the StateMachineProvider to ensure the lib is loaded by the time we init the Context
allow comp to receive props the same way it would be used in the app

The setup:
// test-utils.js

import React from 'react';
import { render as rtlRender } from '@testing-library/react';
import { createStore } from 'little-state-machine';
import Providers from 'components/Providers';

// wrapper should pass down props to
const Wrapper = (props) => {
  const { component: Component } = props;
  createStore({ storeName: { some: 'value' } });

  return <Component {...props} />;
};

// allow providers to receive custom props if necessary
const render = (ui, { languageCode, ...options }) => {
  const AllTheProviders = ({ children }) => <Providers languageCode={languageCode}>{children}</Providers>;

  return rtlRender(ui, { wrapper: AllTheProviders, ...options });
};

// re-export everything
export * from '@testing-library/react';

// override render method
export { render, Wrapper };

Usage:
import { render, Wrapper, screen, act } from 'lib/test-utils';
import MyTestComp from 'components/MyTestComp';

describe('<MyTestComp />', () => {
  test('renders Loader while fetching', async () => {

     // [act is required for handling state updates caused by LSM](https://reactjs.org/docs/test-utils.html#act)
    await act(async () => {
      render(<Wrapper component={MyTestCompnent} handleClick={jest.fn()} />, { languageCode: 'en-ca' });
    });

    const loaderElement = screen.getByText('Loader');
    expect(loaderElement).toBeInTheDocument();
  });
})

In summary, yes all tests should be wrapped by their own providers (which was original intent anyway), but LSM needs to initialize the store within a wrapper component so that the lib can load by the time the test comp gets rendered by RTL.
I hope this helps save some time for others since there's literally 0 examples out there.


[0.9] How to dynamically import elements?

In 0.5.5, I used to dynamically import elements by calling Polymer.import, then inserting that element into the local DOM.
May I ask how can I do that in 0.9?
Thanks so much.

There is a new utility function importHref(href, onload, onerror) to dynamically import one component. See https://www.polymer-project.org/0.9/docs/devguide/utility-functions.html

Yes thank you, I must have missed this. I will test this out. Thanks again.
On 17 May 2015 18:50:54 GMT+08:00, Martin Amm<EMAIL_ADDRESS>wrote:

There is a new utility function importHref(href, onload, onerror) to
dynamically import one component. See
https://www.polymer-project.org/0.9/docs/devguide/utility-functions.html

Reply to this email directly or view it on GitHub:
https://github.com/Polymer/polymer/issues/1554#issuecomment-102782934

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Considering deprecate the sample "blazor-contoso-university"

https://github.com/lohithgn/blazor-contoso-university

@lohithgn Is there any plan to update this sample?


404 ::-webkit-color-swatch link

Request type

Correction or update

Details
The ::-webkit-color-swatch link doesn't exist on this page:
https://developer.mozilla.org/en-US/docs/Web/CSS/::-moz-color-swatch#See_also

This is simply documentation that hasn't been written yet.


GPTQ example README bug

What is the URL, file, or UI containing proposed doc change
https://github.com/vllm-project/llm-compressor/blob/main/examples/quantization_w4a16/README.md
What is the current content or situation in question
from llmcompressor.transformers import oneshot
from llmcompressor.modifiers.quantization import QuantizationModifier

# Configure the quantization algorithm to run.
recipe = GPTQModifier(targets="Linear", scheme="W4A16", ignore=["lm_head"])

What is the proposed change
from llmcompressor.transformers import oneshot
from llmcompressor.modifiers.quantization import GPTQModifier

# Configure the quantization algorithm to run.
recipe = GPTQModifier(targets="Linear", scheme="W4A16", ignore=["lm_head"])

Additional context
GPTQModifier should be imported, not QuantizationModifier.

Thanks - fixed by https://github.com/vllm-project/llm-compressor/pull/139


add Gluon PReLU activation layer

Description
Adds a PReLU layer for Gluon per #8898.
Checklist
Essentials

[ ] Passed code style checking (make lint)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] For user-facing API changes, API doc string has been updated. For new C++ functions in header files, their functionalities and arguments are well-documented.
[x] To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

Could you add unittest and rebase to master? see tests/python/unittests/test_gluon_xxx.py for example

Open question: Since Gluon LeakyReLU is implemented with
 return F.LeakyReLU(x, act_type='leaky', slope=self._alpha, name='fwd')
and since there is a act_type='prelu' in  F.LeakyReLU, why not also implement Gluon PReLU with
 return F.LeakyReLU(x, act_type='prelu', ...etc
What would be more efficient?

Good call @bradcar, I wasn't aware of that parameter. I'll change the implementation to that approach instead, and then add tests and rebase, @piiswrong

working on it in the above PR


added pageTitle - to customise HTML head

added pageFooter - to customise Page Footer
updated reportName referring directly instead from suite, so able to display header in feature-overview.
updated README

Coverage increased (+0.02%) to 98.916% when pulling 1301bfc09e83f8e2804d2ce2633c32bed8d46d53 on muthukumarse:master into 2b4b935a6f1468bf734b5a1f4aa928c99b39c788 on wswebcreation:master.

Hi @muthukumarse
I'm going to merge this PR and make some little changes before I bring a new release. Tnx for your effort.

Sure please. Thanks

New version 1.9.0 has been release. Tnx again for you work

@muthukumarse @wswebcreation Could you please point me a usage example how to create custom name for the header and custom footer. Thanks

@meetravi
See https://github.com/wswebcreation/multiple-cucumber-html-reporter#pagetitle and https://github.com/wswebcreation/multiple-cucumber-html-reporter#pagefooter
or https://github.com/wswebcreation/multiple-cucumber-html-reporter/blob/master/test/test.js#L39 for a code example


Add Mamba2ForSequenceClassficiation

@vasqu I did also change the cache parameter name to cache_params for consistency with Mamba implementation. Please take a look, if ok merge.

Last thing but can we also add the new sequence model to dummy_pt_objects?

@vasqu Thank you so much for the detailed PR review! I resolved most comments, could you take a look?

@Adibvafa Will take a closer relook in a bit and good work!


Add github actions

Add github action workflow running both format and test.

Build has failed @renancaraujo looks like the ./format.sh file can't be found.

Forgot to update the path

Now it goes.

anything preventing this from been merged @renancaraujo ?


Allow empty anchored variable to use

The problem is that when a SecRule contains an negated operator (eg. "!@endsWith .pdf"), and the variable is empty (eg. REQUEST_BASENAME, and the query is a simple "GET /"), then the result of negated evaluate of empty variable will TRUE. Then the rule interpreted as the GET / IS endsWith .pdf, so it will generate a false positive match.
I don't see the reason why didn't allowed the empty variable previously, but this patch fix this bug above, and all other test result keep as PASSED.
See the rule-920200.json regression test file, which inspired the CRS id 920200 and its test.

Merged! Thanks!


fix dependency structure

The dependency list of an application (including parents information) must conform two main rules:

parents must be included in the main set
each dependency in the main set must occur only once.

Currently, the elements which differentiate dependency are : digest, parent, and relativePath.
The existing client is not populating the relativePath but only the path, as a result it may happen that the backend return HTTP 500 in case two dependencies in the main set have same digest and parent null (even if the path differs as we currently use the relativePath to differentiate).
Alternatives to be discussed:

add use of relative path in the client
remove dependency with same digest, parent and relativePath before upload (we loose some details but the upload will succeed)
use the path to discriminate from the backend side (though the path is system-specific, this is usually used just in the scope of a single scan)

We should not use the absolute path, this will make it impossible to relate analyses performed on different machines.
Instead, we should consistently populate the relative path in the analysis clients. Until that is done, we can keep the removal on client side (to avoid constraint violations in the backend) and/or add corresponding checks in the backend.

Regarding the change of JarAnalzer.equals(): The work-around to use path rather than the digest is probably wrong. It is a tempting work-around just because we have the path in the JarAnalyzer, but this information really belongs to the Dependency.

The current PR contains the implementation of option 2 in order to fix the issue up until we do not start using the relativePath. It was tested locally, it would benefit of further tests.


Typescript: How can Voicing mix into a parametric class?

Over in https://github.com/phetsims/ratio-and-proportion/issues/384, I wanted to add voicing to a RectangularRadioButtonGroup, but we don't seem to support this:
class TickMarkViewRadioButtonGroup extends Voicing( RectangularRadioButtonGroup < TickMarkView >, 2 ) {
  ...
}

Notice how Webstorm autoformatted it? The error we get by typescript is:
TS2365: Operator '' cannot be applied to types 'typeof RectangularRadioButtonGroup' and 'typeof TickMarkView'.
If I remove the parametric type:
TS2508: No base constructor has the specified number of type arguments.
@jonathanolson, perhaps we can talk about this as part of our review in #1340.
This is only blocking publication of Ratio and Proportion as far as I know.

Seems problematic, I'll look into it.

@jonathanolson and I talked about it, and were able to get things working with:

type ConstructorOf<C> = { new( ...args: any[] ): C; };

class TickMarkViewRadioButtonGroup extends Voicing<ConstructorOf<RectangularRadioButtonGroup<TickMarkView>>>( RectangularRadioButtonGroup, 2 ) {

We are still working on a way to move that to the mixin files, instead of every usage, but at least this can be non blocking at this point. I'll try it out over in https://github.com/phetsims/sun/issues/699 and https://github.com/phetsims/ratio-and-proportion/issues/384

Lots of things are mixing into the same base type... memoizing sounds helpful there (but would need to figure out how we should memoize with the 2 parameters). Otherwise we're losing the in-memory benefits.
Perhaps we can programmatically determine the number of supertype arguments and autodetect the optionsArgPosition?

The mixed superclass is separated into a separate once-used variable. Is that on purpose?

Whoops wrong issue!

This looks like it will be an issue...

Actually this is type-checking
const GenericMix = memoize( <SuperType extends Constructor>( type: SuperType ) => {
  const result = class extends type {
    _someField: string;

    constructor( ...args: any[] ) {
      super( ...args );

      this._someField = 'Testing';
    }

    get someField(): string { return this._someField; }

    set someField( value: string ) { this._someField = value; }
  };

  return result;
} );

class PropertyMixed<T> extends GenericMix( Property )<T> {
  constructor( value: T, options?: PropertyOptions<T> ) {
    super( value, options );
  }
}

const pMixed1 = new PropertyMixed<number>( 5 );
console.log( pMixed1.value, pMixed1.someField );

const pMixed2 = new PropertyMixed<string>( 'hi' );
console.log( pMixed2.value, pMixed2.someField );

Didn't realize we can just have the mix be... generic, and then specify a type parameter to the mixed type. From https://github.com/microsoft/TypeScript/issues/26542#issuecomment-414355965

The key line is:
class PropertyMixed<T> extends GenericMix( Property )<T> {

@zepumph does this seem to resolve this issue for you?

That seems pretty nice on paper! I will want to try it out in my mixins after I get some bug fixes pushed over in https://github.com/phetsims/scenery/issues/1340


data: add some demo data through cli command

Done: https://github.com/inveniosoftware/invenio-app-ils/blob/master/invenio_app_ils/cli.py#L149


add support for calling restart

Although jupyter_ascending exposes a script to restart the kernel, this was not callable from vim. Add the necessary function and  definition.

Thank you!


[Macos 13 and 14] Go version 1.20.0 will be removed on October 07,2024.

Breaking changes
Removing Go 1.20.0 from Macos 13 and 14 images.
Target date
October 07,2024.
The motivation for the changes
As of the software support policy only 3 latest versions of go are supported i.e., Go 1.21.0,1.22.0,1.23.0
Possible impact
If your builds depend on Go 1.20.0 they can be broken
Platforms affected

[X] Azure DevOps
[X] GitHub Actions

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[X] macOS 13
[X] macOS 13 Arm64
[X] macOS 14
[X] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Mitigation ways
https://github.com/actions/setup-go action for GH actions
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/tool/go-tool?view=azure-devops task for AzDO

Go 1.20 is removed from all macOS13 and macOS14 images.


xcode 7, swift 2, updated to use pods

it compiles and runs with iOS 9 and xcode 7, using swift 2. Updated to use pods

Thank you very much for your PR, you may notice I have been working on Xcode 7 branch for a while and have upgrade every thing, can you please have a look at it and see whether I can invite you work together on the branch. It is ready to release except for the docoments, and there are some features will be added soon.

Not a problem, I simply made it work for me and I thought it could be useful for you. I/You can drop this pull request :)


Feature: Supplementary Function Notes

Description
Feature: Supplementary Function Notes
Suggested solution
Feature: Supplementary Function Notes
Alternative
No response
Additional context
No response

done


add Debian 9.1 template

There has been issue #1106 @mpapis can you briefly look over this?

Thanks, I tried it seems to work for me.

@mpapis seems rather busy, so I'm going to merge it myself.

@FunTimeCoding indeed, but I'm working on getting more time for opensource... someday ;)