Update contrib install instructions

Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] Optimization
[X] Documentation Update

Description
Updated contributing documentation and pre-commit dependencies.
Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[X] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/assign @bparees

@thoraxe Those are inline code comments, not markdown titles.

/ok-to-test
/lgtm
/approve

But "no capitals-case" is probably not what we want here.

since they are just incline script comments, all lower case is fine.    As it was would have been fine too imho, so i hope not to see too much of people making personal preference style edits(we need to focus on more material concerns), but since this PR also updates some steps that were legitimately wrong/out of date, i'm ok with it in this case.
@onmete if you join the openshift github org, we will no longer have to explicilt approve testing for your PRs


fix: ssr.external/noExternal should apply to packageName

Description
Fixes

https://github.com/sveltejs/kit/issues/5549

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

cc @benmccann @dominikg @bluwy
We should check vite-ecosystem-ci and merge this PR, so it is included in Monday's release

Merging as this is an important fix for downstream


Stamp React Native Version Into C++ Code

Summary
The PlatformConstants native module exposes the ability to query the React
Native version used to build native code. This is managed on iOS and Android by
a version bumping script, which replaces module code based on a template.
It is currently difficult to accurately determine this version for out-of-tree C++
platforms (I.e. React Native Windows). The version of upstream react-native we resolve to is ultimately
dependent on the version of react-native chosen a peer dependency, which is not
neccesarily constant given a build of react-native-windows.
We could try to hack around this, and make our native build try to reason about
the resolved pacakge for react-native using a lockfile, but a much cleaner
solution is to embed version into C++ code, similar to what is done for Android
and iOS. This change does that, adding a header with React Native version and
updating the build stamping script to write to it.
Usage sample:
  constants["reactNativeVersion"] = folly::dynamic::object();
  constants["reactNativeVersion"]["major"] = ReactNativeVersion.Major;
  constants["reactNativeVersion"]["minor"] = ReactNativeVersion.Minor;
  constants["reactNativeVersion"]["patch"] = ReactNativeVersion.Patch;

Changelog
[General] [Added] - Stamp React Native Version Into C++ Code
Test Plan
Validated that the bumping script will accurately update the header, can compile under both MSVC and Clang.

@hramos do you know what failed for Facebook Internal CI?

@NickGerleman this patch no longer applies on top of master. Looks like ReactCommon/cxxreact/ReactNativeVersion.h no longer exists.

Oh, my bad, that file is added in this PR. In any case, we failed to apply the patch internally, usually it's either due to a file that moved or because the PR is based off an older revision and the process timed out (can happen if the base commit is over 3 months old). I'm applying the changes manually on top of HEAD and will push them out. Hold tight.


rewriting variable units

Calcpad has unique feature in rewriting variable at calculation. however, some US units not displayed as original units.

as can be seen, the units of 'A_se' variable is displayed as original, but 'f_uta' does not.

Yes, I can see that. Thank you! This is a bug. I have an idea why this is happening. Please, give me 1-2 days and I will fix it. :)

right, Calcpad getting better. none catch the unique feature to rewrites variable and the units.


Language.cs: added Dutch (Nederlands)

I wish to commit a full translation in Dutch (nl_NL or nl_BE), which is Nederlands. This language is mainly used in The Netherlands and Flanders (Belgium). I chose nl_NL as the locale identifier, but both are really the same. I used polite and formal language (e.g. "u" instead of "je", which both means "you", but "je" usually is used informally).
Each line is translated from the English version. some lines have been reworded to comply with our language, but they carry the exact same meaning.
In case of issues, please feel free to contact me in order to look into it.
(If you wish, I can also send a slightly customized theme (cyan text on the top Menu Bar, based on the default theme, so really only a text colour change) but I hope this contribution is useful and helps a bit.
Kind regards

Looks good, thanks!


Add draconic essence drops from story

Stories should drop draconic essence to allow players to unbind dragons without paying other dragons/sunlight stones.
This data is probably in the masterasset somewhere, unsure where

Closed by #1107


update splitlines on string of config

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

Awesome, thanks!

Awes, thank you!


Steam update broke the app

After receiving a steam client update today the application no longer works and just closes after a few seconds when trying to send an invite or when trying to create an invite link.

+1

+1

Can confirm, sadly is broken as of 2022/05/13

+1

+1

+1, help! This is very cool app to play with friends, sadly I cant play since last update :C


Update hugo.yaml, added permalink for v0.33

This PR is related to issue #6889
Since v0.33 is released have added the permalink for the same

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Added a line to hugo.yaml

Verification

i have already added that randomly to the release-0.33 branch; sorry


docs: typo fix Lighting → Lightning

Summary

Fix typo in the README, Lighting CSS → Lightning CSS

Checklist

[ ] Tests updated (or not required).
[x] Documentation updated (or not required).

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


[CON-244] Ability to change session title in history page

Description

Adds ability to change session title under history by clicking edit icon alongside the delete button.
Fixes issue with updating existing session touching createdAt timestamp
Fixes recalculating title when switching between sessions.

Closes #1426
Related #479
Checklist

[x] The base branch of this PR is preview, rather than main

@sestinj Good to go. Let me know if any changes need.

@priyashpatil Finally looked through this, it was great work! Congrats on a great first big PR!

@priyashpatil Finally looked through this, it was great work! Congrats on a great first big PR!

Thanks. More on the way.


Support outputting as Mermaid

version 0.2.0 implementation is incorrect, since it connects all nodes for all ancestors-decendants for each.
Each node should be connected only by its direct parent/child as is done with the plantuml implementation


flops and params

How to obtain the floats and params of this model? The result I obtained using thop is 0.

hello,do you know how to calculate th flops?


Extend supported labels and tasks

Currently ART supports either one-hot encoded or index labels for categorical classification tasks.
We plan to extend ART to accept (ideally) all possible label formats. The following labels and tasks should be supported:

[ ] object detection labels, image segmentation
[ ] images / tensors as labels / autoencoder / VAE
[ ] sequences as labels / seq2seq models
[ ] regression tasks
[ ] ...

I'm closing this epic because it has been replaced with multiple detailed epics which will provide a general solution: #308 #309 #310 #311


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 09:51:29 on 2023.06.10
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 1, errors out: 0
Original problem ID: 28489
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=28489

Problem has been resolved in 3m 0s at 09:54:29 on 2023.06.10
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 28489


[MiniLLM] mismatch between formula and implementation (gradL)_long?

    def _pg_loss(
        self,
        logprobs: TensorType["batch_size", "response_size"],
        old_logprobs: TensorType["batch_size", "response_size"],
        advantages: TensorType["batch_size", "response_size"],
        mask: TensorType["batch_size", "response_size"],
        w: TensorType["batch_size", "response_size"],
    ):
        """PPO objective function.
        References:
        - https://stable-baselines.readthedocs.io/en/master/modules/ppo2.html

        """
        n = mask.sum()

        log_ratio = (logprobs - old_logprobs) * mask
        ratio = torch.exp(log_ratio.float())
        ratio = ratio * w

In https://github.com/microsoft/LMOps/issues/255, it states w=1, is it also 1 here?
In the paper, this part has no mention of w factor

No. $w$ is absorbed into $\rho_t(\theta)$ in the formula.

cool, thanks


mirror registry problem

Bug description
TASK [populate_mirror_registry : Build pruned OLM index] ********************************************************************************************************
fatal: [registry_host]: FAILED! => changed=true
cmd:

/usr/local/bin/opm

index

prune

--from-index

registry.redhat.io/redhat/redhat-operator-index:v4.12

--packages

advanced-cluster-management,cluster-logging,kubevirt-hyperconverged,local-storage-operator,ocs-operator,performance-addon-operator,ptp-operator,sriov-network-operator

--tag

registry.example.lab:5000/olm-index/redhat-operator-index:v4.12
delta: '0:00:37.338661'
end: '2023-07-18 10:31:20.759498'
msg: non-zero return code
rc: 1
start: '2023-07-18 10:30:43.420837'
stderr: |-
time="2023-07-18T10:30:43+08:00" level=warning msg="\x1b[1;33mDEPRECATION NOTICE:\nSqlite-based catalogs and their related subcommands are deprecated. Support for\nthem will be removed in a future release. Please migrate your catalog workflows\nto the new file-based catalog format.\x1b[0m"
time="2023-07-18T10:30:43+08:00" level=info msg="pruning the index" packages="[advanced-cluster-management cluster-logging kubevirt-hyperconverged local-storage-operator ocs-operator performance-addon-operator ptp-operator sriov-network-operator]"
time="2023-07-18T10:30:43+08:00" level=info msg="Pulling previous image registry.redhat.io/redhat/redhat-operator-index:v4.12 to get metadata" packages="[advanced-cluster-management cluster-logging kubevirt-hyperconverged local-storage-operator ocs-operator performance-addon-operator ptp-operator sriov-network-operator]"
time="2023-07-18T10:30:43+08:00" level=info msg="running /bin/podman pull registry.redhat.io/redhat/redhat-operator-index:v4.12" packages="[advanced-cluster-management cluster-logging kubevirt-hyperconverged local-storage-operator ocs-operator performance-addon-operator ptp-operator sriov-network-operator]"
time="2023-07-18T10:31:11+08:00" level=info msg="running /bin/podman pull registry.redhat.io/redhat/redhat-operator-index:v4.12" packages="[advanced-cluster-management cluster-logging kubevirt-hyperconverged local-storage-operator ocs-operator performance-addon-operator ptp-operator sriov-network-operator]"
time="2023-07-18T10:31:20+08:00" level=info msg="Getting label data from previous image" packages="[advanced-cluster-management cluster-logging kubevirt-hyperconverged local-storage-operator ocs-operator performance-addon-operator ptp-operator sriov-network-operator]"
time="2023-07-18T10:31:20+08:00" level=info msg="running podman inspect" packages="[advanced-cluster-management cluster-logging kubevirt-hyperconverged local-storage-operator ocs-operator performance-addon-operator ptp-operator sriov-network-operator]"
Error: opm index prune only supports sqlite-based catalogs. See https://github.com/redhat-openshift-ecosystem/community-operators-prod/issues/793 for instructions on pruning a plaintext files backed catalog.
Usage:
opm index prune [flags]
Flags:
-i, --binary-image opm        container image for on-image opm command
-c, --container-tool string   tool to interact with container images (save, build, etc.). One of: [docker, podman] (default "podman")
-f, --from-index string       index to prune
--generate                if enabled, just creates the dockerfile and saves it to local disk
-h, --help                    help for prune
-d, --out-dockerfile string   if generating the dockerfile, this flag is used to (optionally) specify a dockerfile name
-p, --packages strings        comma separated list of packages to keep
--permissive              allow registry load errors
-t, --tag string              custom tag for container image being built
Global Flags:
--skip-tls-verify   skip TLS certificate verification for container image registries while pulling bundles
--use-http          use plain HTTP for container image registries while pulling bundles
stderr_lines:
stdout: ''
stdout_lines:

OpenShift version
other (provide in the description)
Assisted Installer version
other (provide in the description)
Relevant log output
No response
Inventory file
No response
Required statements

[X] I have removed all sensitive details from the attached logs and inventory files.
[X] I acknowledge that Red Hat does not provide commercial support for the content of this repository.
[X] I acknowledge that any assistance is offered purely on a best-effort basis, as resource permits.

@wufeng2x this should be fixed now can you verify?

@wufeng2x I'm going to assume this is fixed now. Feel free to reopen it if you have this issue in the future.


Missing emum lskCustomEmojiKeys caused tdata loading fail

Telegram add a value named lskCustomEmojiKeys in (https://github.com/telegramdesktop/tdesktop/commit/ddd5021966568a7d999aa6d35cd069a6fbc2aa3b)
so opentele can not load tdata when find this key
it will raise an exception here
https://github.com/thedemons/opentele/blob/ff90c36a867cf7902e80f480a35041c5e7902e4c/src/td/account.py#L214

Hi, thanks for the detailed research of the issue!
Unfortunately, I don't have the time right now to take a look at the problem. Would you mind making a pull request?
The map data isn't really that important so I think just adding lskCustomEmojiKeys to the enum and an empty elif keyType == lskType.lskCustomEmojiKeys: will solve the problem


Hyperlink of  OpenVINO™ pre-trained models returns 404.

On page https://github.com/opencv/dldt/blob/2019/get-started-linux.md ， the hyperlink of  OpenVINO™ pre-trained models doesn't work and returns the error code 404 . I successfully found the model zoo folder, but I think it may be better to update the hyperlink :D.

Fixed, thank you!


In v6, Grid columnSpacing misbehaves with string pixel values like "16px"

Steps to reproduce
Link to live example: (required)
Steps:

View editable example with v6: https://stackblitz.com/edit/react-hp2ukf?file=Demo.tsx

corresponding v5 example that shows expected behavior https://stackblitz.com/edit/react-hp2ukf-xjzryc?file=Demo.tsx

Note that specifying columnSpacing="16px" produces extra space on left.

Current behavior
When specifying columnSpacing on Grid using pixels, there's extra space to the left of the grid items.
Note: Locally in development, but not on stackblitz, I can see margin-left: NaNpx on <div class="MuiGrid-root MuiGrid-container">.
Expected behavior
No extra space.
Context
We recently updated MUI from v5 to v6.1.6. I have just noticed this behavior and believe it is a regression. (Edit: I've added a corresponding stackblitz demo for v5)
I understand that Grid2 is preferred now, however we have not had a chance to update all the Grid components yet.
Your environment

npx @mui/envinfo
Using Chrome. Note: This envinfo is for my local, not Stackblitz
  System:
    OS: macOS 14.3.1
  Binaries:
    Node: 20.17.0 - ~/.nvm/versions/node/v20.17.0/bin/node
    npm: 10.8.2 - ~/.nvm/versions/node/v20.17.0/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: 130.0.6723.92
    Edge: Not Found
    Safari: 17.3.1
  npmPackages:
    @emotion/react:  11.13.3
    @emotion/styled:  11.13.0
    @mui/base:  5.0.0-beta.61
    @mui/core-downloads-tracker:  6.1.6
    @mui/lab:  6.0.0-beta.14
    @mui/material:  6.1.6
    @mui/material-nextjs:  6.1.6
    @mui/private-theming:  6.1.6
    @mui/styled-engine:  6.1.6
    @mui/system:  6.1.6
    @mui/types:  7.2.19
    @mui/utils:  6.1.6
    @types/react:  18.3.11
    react:  18.3.1
    react-dom:  18.3.1
    typescript:  5.6.3

Search keywords: grid, spacing, nan, negative

Thanks for reporting, a PR to fix this issue has been created.


Bug: Idle notifier sends notifications while attacking Pest Wars portals

title.
This also happens even if being attacked while attacking a the Pest Wars portal.

It also sends notifications while picking flax, which is rather silly.

I get notified everytime I minimize my runelite client. Rather annoying..


Add latitude and longitude to application_forms

Context
We want to analyse distances between candidates and the sites they
choose to train.

Changes proposed in this pull request
Add columns to the application_forms table that will
allow us to start geocoding candidate addresses.

Guidance to review

Link to Trello card

https://trello.com/c/6A8t8vVY
Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

Think it may be worth considering what library will be used for geocoding as it may specify the type you require for lat and long.
E.g. Teacher Training API uses the Geokit gem
https://github.com/geokit/geokit-rails#how-to-find-all-stores-within-a-10-mile-radius-of-a-given-latlng
Possibly the rationale for floats may be as described here - https://stackoverflow.com/a/18182565

I was wondering about the data type for these too. Didn't see this earlier but I went with floats full the sites table. Will need to change those if we go with decimals here. https://github.com/DFE-Digital/apply-for-teacher-training/pull/3498

Library:
I opted for the geocoder gem as it's actively maintained and gives us the distance calculations we need for Locations work. Interestingly, Find uses both geocoder and geokit, although for our purposes I think we only need the one. Open to other suggestions if they're a better fit :+1:
Data type:
Good shout, and definitely worth discussing. I just defaulted to decimal because of precision at the expense of storage. Do we understand the rationale for Find adopting floats?

Publish also use floats. Might be worth considering as we're going to pulling lat/long for Provider and Sites from them.

Library:
I opted for the geocoder gem as it's actively maintained and gives us the distance calculations we need for Locations work. Interestingly, Find uses both geocoder and geokit, although for our purposes I think we only need the one. Open to other suggestions if they're a better fit 👍
Data type:
Good shout, and definitely worth discussing. I just defaulted to decimal because of precision at the expense of storage. Do we understand the rationale for Find adopting floats?

I think it may be worth checking if the library requires you to use a certain type (e.g. Geokit gem). I believe Floats are accurate up to 6 d.p and this will give still give you location accuracy to +- 11cm. You then also get the optmisation benefit over decimal

Good to know, and the accuracy argument makes sense to me :+1: . We'll have more than enough precision in practice, especially as the default  column type created by a Rails migration using the float method appears to be double precision (looking at #3498 ), ie - 15 dp.
Will update this to use float.
(sources, for posterity):
https://www.postgresql.org/docs/9.6/datatype-numeric.html
https://en.wikipedia.org/wiki/Decimal_degrees#Precision

I think it may be worth checking if the library requires you to use a certain type

Geocoder supports both floats and decimals.


Release/1.0.0

I think it's time for a 1.0.0.
@nderwin if you could take a poke at the code and do the normal quality nit-pick, that'd be nice. Or, if you'd rather just open issues for things you see, it'll give me something to do when insomnia hits. :-)

A bunch of refactoring to simplify the abstractions and clean up their meanings
Added the ability to resolve expressions which may not be resolved by the normal maven expression resolver, and setup expressions to allow configuring how the GIT_BRANCH or GIT_URL (now gitBranchExpression and gitURLExpression) are resolved.
Breaks the dependency on Jenkins / Hudson semantics for environment variables! It should work with travis-ci now!

I have a PR waiting to go into release/1.0.0 with the nits picked.

@nderwin nit picks merged. Care to comb it out again?

meh. If we ned to, we'll hotfix it and start dog-fooding the build. :-)


Missed php-includepath delcaration in ruleset_xml_schema.xsd

An issue
The actual implementation of \PHPMD\RuleSetFactory allows declaring php-includepath to provide possibility to explore and use rulesets defined in 3rd party libraries. However, http://pmd.sourceforge.net/ruleset_xml_schema.xsd does not declare php-includepath so its usage makes ruleset file declaration invalid.
Additional Information
\PHPMD\RuleSetFactory::filePaths returns a possible locations list of referenced ruleset. The algorithm searches in PHPMD installation folder, then in a current working directory (in most cases that would be projected directory),  and then in all folders from include paths. So, if some project would like to use rulesets provided by 3rd party library, a path to the library should be added to include paths. It is not a requirement of PHP code as autoloading mechanism may not require modification of include paths. Because of this, the best place to declare modification of include paths is a projects ruleset file.
Related issue

phpmd/phpmd#545
magento/magento2#13400

Proposed solution
@addiks in phpmd/phpmd#545 provided an updated XSD file that needs to be deployed to http://pmd.sf.net/ruleset_xml_schema.xsd

http://pmd.sf.net/

This is not PHP Mess Detector's but Java's Mess Detector's (the original) Website.

So it also requires us to extends ruleset_xml_schema.xsd and publish our own xsd file for PHPMD ruleset format.


refactor(linux): scan easier to read

Code to scan wifi on linux is not easier to read and also tested.
It also introduce a new architecture to more easily manage all functions
from all platforms.
Description
Use architecture based on folders and command and parser.
Motivation and Context
It is easier to unitary tests parser and command generator.
Usage examples
No change of interfaces.
How Has This Been Tested?
Unit tests have been added
Types of changes

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

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Devil's Sight

Devil's Sight is gray scale when it should be perfect vision.

You are correct, but the warlock should also be able the distinguish between illuminated areas and darkness in some way. That wouldn't be the case if I made it perfect, unless the light sources are animated/colored. I think it's probably more important being able to tell what is illuminated and what isn't than the ability to see colors. I didn't desaturate it a 100%, so the player should still able to tell what the colors are roughly.
Let me know what you think. I'm open to suggestions.

Hard agree that the warlock should be able to see where darkness is. In v9 I had a toggle macro to disable my bright vision for this reason.  V5E does let you toggle it off from the HUD though, so I don’t get the need for grayscale.

We were toying with colorized vision but it carries over between normal and devilsight so it wouldn't be a perfect solution plus theres no way to pick your color for just devilsight.

Hmm, is it intended that all tokens have Darkvision 0ft if they don't have the darkvision trait? It means all tokens have a HUD button that does nothing 🤔

I have to assume though that a lot of players just select devil's sight for 99% of the time, because they don't have darkvision, or because devil's sight is overall better because of the longer range and magical darkness, or just because it can be annoying to switch all the time. There are also situations where switching doesn't help: the warlock wouldn't be able to tell whether there's light behind magical darkness if devil's sight is full color, because if you switch to darkvision you won't be able to see this area at all. Vision tint is also not an option because vision effects are applied to illuminated just like they are applied to nonilluminated areas, and so the tint would make everything red. You might have noticed that even darkvision and any other desaturated vision can change the look of illuminated areas slightly. If vision was not applied to illuminated areas at all, we would have more options to create a distinct look for devil's sight in darkness, and devil's sight wouldn't affect the brightness of dim light at all.

Hmm, is it intended that all tokens have Darkvision 0ft if they don't have the darkvision trait? It means all tokens have a HUD button that does nothing 🤔

Yes, it's necessary to be able switch back to normal sight for tokens that doesn't have darkvision. I could probably name the button something else in this case, but I wasn't sure what to call it and basically everyone has darkvision anyway and so it didn't seem too important.

Do you mean switch back to normal vision is they have a Detection Mode like Detect Magic going?
Would it be possible to just not show the vision button at all if they only have one mode available? Unless it serves some other purpose…

Disclaimer: I am still on v10, and am only going by what I've seen in the readme and some videos I've seen in discord.
I can only report that my players will definitely, 100% complain if the devil's sight is presented as desaturated grayvision.  Source: they already do whenever the DS settings for their tokens get messed up (for me in v10,  where I occasionally have scenes they were placed in long ago where they just had DV).
I'm not sure I understand what "should also be able the distinguish between illuminated areas and darkness in some way" means.  If you mean they should be able to tell what is illuminated normally and what is darkness that they can see normally because of their devil's sight, I'm not sure I agree, that doesn't seem RAW.  As Lukas says, if they want to turn off their devil's sight, and see in whatever other vision they have, to try to understand that they have that option.
My opinion, since you asked: if for practical reasons you want to somehow make that more evident inside of DS so players don't have to go around switching their vision back and forth, I guess I would be open to that, but not at the expense of making one of the definitely explicit parts of devil's sight not work (using darkvision desaturated coloring instead of normal coloring).
Ideas:

Make the desaturation optional - we can all set it the way we prefer at our tables.
Tone down the desaturation much more significantly to find a better happy medium.
Add the brightness indicators from perfect vision back in (with a keybind presumably).
Draw faint lines where the boundaries change in the vision polygon, or something along those lines.

Though this also makes me wonder if this module has a way to handle most monster "devil's sight", which is annoyingly quite different: e.g "Devil’s Sight. Magical darkness doesn’t impede the imp’s [Darkvision]".
I guess I'm asking if there's any setting or flag that does nothing but grant the ability to pierce magical darkness with a vision sense (e.g. devil's sight for warlocks with that eldritch ability, or devil's sight for monsters with that enhancement to darkvision).

Tweaked the appearance of devil's sight. Make sure you switch to another vision mode and back to get the new look.
NPCs now gain devil's sight equal to their darkvision range if they have the feat.

https://github.com/dev7355608/vision-5e/releases/tag/v1.8.0

Thanks for the speedy update! I was experimenting with the values in devils-sight.mjs and was just about to suggest what you just did... beat me to it! 👏


GitHub like repo home page

Replace #27931
Laptop View:

Mobile View:

TODO: (in other PRs, we can improve it one by one)

add contributors
why: the original PR of the implement of contributors does not contain a usable function to fetch the top contributors
license/readme tab.

The table's width is controlled by this, I need some help about how to adjust it.

Consider expanding the screen width for a better viewing experience.

Need more works for mobile view. desktop view is almost finished.

In mobile view, I want to move search code and descriptions to the top of the page.
Is there any good way to do it?
In my mind, maybe we can have two elements in both side and hide one of them by css if it is mobile view or not.

This issue is still existing. https://github.com/go-gitea/gitea/pull/32213#issuecomment-2398818350

The table will be outside of the container.

Inputting too long contents may broken the UI. 🤔

Maybe we could hide the overflow content and add a tooltip for the release name.

@silverwind
Do you have time to have a look of this issue:
https://github.com/go-gitea/gitea/pull/32213#issuecomment-2398818350
https://github.com/go-gitea/gitea/pull/32213#issuecomment-2425300909
The max width of this table is calculated dynamically. It comes from #30374.

@silverwind
Do you have time to have a look of this issue:
https://github.com/go-gitea/gitea/pull/32213#issuecomment-2398818350
https://github.com/go-gitea/gitea/pull/32213#issuecomment-2425300909
The max width of this table is calculated dynamically. It comes from https://github.com/go-gitea/gitea/pull/30374.

I found a way to fix it. Add fixed class to the table.

All UI bugs have been fixed, ready for review.

license missed.

license missed on tab? Or it has been moved to right sidebar by design?

Yes. It is by design. It will always in the sidebar in the home page.
So that's why I have said this PR is the second step in #32686

releases will always be displayed even if releases have been disabled.

Done in d11bf78fa1ed53f003092224ff71561330d1e662

the layout is broken

the layout is broken

It works in my PC. 🤔

the layout is broken

It works on my PC. 🤔 Are u using Mac?

Windows, resolution: 2560x1440

Still many flaws, will fix

Sidebar is also broken, please fix

Incorrect layout when viewing files

Incorrect padding

Have done my best to fix various problems (by the way, move some code to refactor the legacy functions)

@yp05327 could you take a look at whether my changes are right? If nothing wrong, we can merge it and keep improving in following PRs.

Tooltips looks not good (especially for license, it is designed for the original UI), changed into title.

LGTM
@hiifong
Does it work now?

LGTM
@hiifong Does it work now?

The layout is still broken, This time I am using a Mac system.

LGTM
@hiifong Does it work now?

The layout is still broken, This time I am using a Mac system.

@wxiaoguang Can you fix this?

LGTM
@hiifong Does it work now?

The layout is still broken, This time I am using a Mac system.

@wxiaoguang Can you fix this?

And the spacing between the items is too tight

The white space LGTM, but maybe you are using high DPI display, so the result is different to us.

Fixed in 6412de600917023fa1d0a443bdfe16de6a2d8d2d

Fixed in 6412de6

LGTM

One more question.

One more question.

Does this meet your expectations?

This is by design. Same to GitHub.

Then we could add a "sync fork" button like GitHub:
->  Support "merge upstream branch" (Sync fork) #32741


Create test.md

I am submitting a new organization:

[ ] I am only submitting one organization.
[ ] The organization is not already listed.
[ ] The organization is a pro-life organization.
[ ] The links to the organization are in their simplest form.
[ ] The name of the organization is in lowercase.
[ ] All information regarding the organization is included in their respective fields.

I am fixing a bug or miscellaneous issue
Related Issue

Insert link to issue here.

Description of Changes

Description of your pull request here.

Oh no, it failed, for some reason. Best make some changes!


CI (Continuous Integration) Proposal: Consolidate Documenter and Test GitHub Actions workflows into a single CI workflow

The GitHub Actions workflows generated by the PkgTemplate package now includes a CI.yml workflows file that consolidates test and documentation generation, so it seems reasonable to combine the Documenter and Test workflows into a single CI workflow to follow the current Julia packaging trend.
Thoughts?
If we would like to proceed in this direction, I have a customized version of CI.yml that makes a few improvements to the default CI.yml, such as

generation of release-specific documentation and

improved efficiency by loading of code coverage data to Codecov for only a single configuration of the job matrix.

No objections against that from my side. Feel free to submit a PR.


Add Spell Open Research Grant

It seems that it sadly expired before merge

The deadline to apply is June 1st.


Piemenu minor glitches

A few more polishing / "nice to have" fixes...

Pressing Z outside the 3D viewport prints an error message in show_debug_draw_pie_menu because overlay_control is null.
If you press Z to open piemenu then Z again, the menu disappears but remains active until you click the mouse.

It would be nice to match Blender's UI. I think it has 4 ways to select a pie menu command:

Press Z, then keyboard shortcut
Press Z, move mouse, click
Press Z, move mouse, press Z again
Hold Z, move mouse, release Z

See https://github.com/imjp94/gd-blender-3d-shortcuts/issues/18#issuecomment-1732594920
Handling the error above solves part of #18.


Tests fail 100% on CI

I have OS project: https://github.com/ealeksandrov/EARestrictedScrollView
Some time ago it was working well with TravisCI, but when I pushed recent changes it started to fail every time. I can't reproduce it locally even running the same command:
set -o pipefail && xcodebuild test -workspace Example/EARestrictedScrollView.xcworkspace -scheme EARestrictedScrollView-Example -sdk iphonesimulator ONLY_ACTIVE_ARCH=NO | xcpretty

Succeeded locally:

Failed on TravisCI:
https://travis-ci.org/ealeksandrov/EARestrictedScrollView/builds/95062376
I even added Second Curtain to verify images by myself, here is the result:
https://s3.amazonaws.com/hadle-travis-errors/95062377/index.html
I can't see any difference by myself, but Kaleidoscope show a lot of pixels:

Can you suggest any ideas? Maybe 32 vs 64 bits? But TravisCI should be using the same SDK and simulator...

hmm this is strange, how did you record the images at the first place (what simulator does it default to?), Your Travis CI seems to be iPhone-4s with 8.1. Is it the same when you record locally?

You are right. Looks like it was my mistake: -sdk iphonesimulator9.1 doesn't mean it will use iOS 9.1.
Adding -destination 'platform=iOS Simulator,name=iPhone 6s,OS=9.1' solved this issue for me.
Thanks!


Intraday Trading

Hello everyone
I want to know if there is any method or environment present that can be good for doing intraday trading.
cause In finrl There is only a one-day resolution I want to change to 5 minutes, 15 mint, etc, and train the model.

Just use instead column "Date" column with "day and time". I tried to use "YYYY-MM-DD hh:mm". Everything works fine.
Just replace dates to minutes - logic of program doesn't change.


Question about contribution to this project

Hello, I long time watching this repo and I want to help you with the implementation of libp2p under Kotlin. How can I start, and where do you discuss the development of this implementation?
@raulk

Hey! Thanks for reaching out. We’re gonna be promoting this project to the libp2p org, and will create a Gitter channel along the way. Will update this issue when all of that is done. Then we’ll have a venue to coordinate.

Ok, thank you!


Added Pokemon Ball Finder Game

PR Description 📜
Added a Pokemon game using HTML, CSS, and JavaScript, where players control a character using arrow keys to search and collect Pokeballs, earning points to win the game.
The game will offer an immersive and enjoyable experience to the user.

Fixes #1976

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

screen-capture (4).webm

Thank you soo much for contributing to our repository 💗

Thank you @abhinav-m22 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @abhinav-m22 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @abhinav-m22,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


Single charges

Helllo and thank you for creating this.
I'm wondering does this package support creating single charges for connected accounts?

I've forked the project to add a lot of functionality on.
https://github.com/syn6UK/laravel-cashier-stripe-connect
Supports direct and destination charges.

Please see the documentation for the aforementioned new package
https://updev-1.gitbook.io/cashier-for-connect/

Hi @syn6UK, is it compatible with the latest version of Cashier (v14.2.1)?

Yes it is.

After testing on staging and now on a production projext the New package is released
Webhooks are working fine, obviously given the flexible nature of connect you need to declare and build your own listeners, but i've provided some example patterns including how to support segmented multi tenant platforms.
Simply install with
composer require lanos/laravel-cashier-stripe-connect
https://updev-1.gitbook.io/cashier-for-connect/

@syn6UK great work! Feel free to submit a PR here updating the README.md to recommend the new project :).

Done mate! PR Made. I've left credit on my readme file and also on my gitbook to credit everyone up until this point.


Mali - Unit 3

Module 1 Assignment

Great job creating .py files with your custom functions 👍
Be sure to edit your README (top level of your Forked repository, below the image of the lamb) to link your separate Lambdata repository.
Try and deploy it with PyPi so you can import it in different python environments!
You can see how, with time, you can create a custom set of imports for the different stages (ETL, fine-tuning, etc). Definitely something to return to over the remainder of the course!


Support extraPodLabels in apiServer and Frontend

#32
Creates extraPodLabels so that a user can set specific pod labels in either the apiServer or frontend deployment.

Thanks @initharrington! Could you please also extend values.schema.json accordingly?

Switched to object to fix, added to statefulset, and rebased


operator cert-manager (1.13.1-rc1)

Signed-off-by: Richard Wall<EMAIL_ADDRESS>

/merge possible

/merge possible


Additional Props Request

I would like to request a few additional props.
Mainly the only difficulty I have in implementation is the lack of title and description for items.
Something like this perhaps?
<PaypalExpressBtn client={client} note={'Thanks for your patronage!'} description={'A large mallet'} name={'Loony Tunes Hammer'} currency={'USD'} total={this.state.value} onError={onError} onCancel={onCancel} onSuccess={onSuccess} />

@firefox7025 looking thru the paypal-checkout code that this library uses I can't find a reference to title & description.
any ideas where to look?
https://github.com/paypal/paypal-checkout


Add an option to only use WiFi for upstream

The whole reason for WiFi sharing apps is to share a free WiFi connection (I can use the standard hotspot functionality to share mobile internet). I want to have an option (preferably on by default) to ensure that the app won't use mobile internet if the upstream WiFi connection breaks. I don't want to pay a higher bill if the disconnect happens while downloading a big file.

Hi, since this is a simple proxy application, it is out of scope for it to decide to act based on connection state, sorry.
I recommend looking at other solutions for strict WiFi sharing.

There are not so many solutions for WiFi sharing without root. The solution I've been using so far is "NetShare no-root-tethering". I believe there another closed source app, PdaNet+. This is the third solution, finally an open source app. Neither has an option to limit traffic to WiFi.
My use cases for sharing mobile data and WiFi are different.
Sharing mobile data: I'm driving and sharing my internet connection with other passengers who don't have mobile data. The passengers are told not use download big files. The hotspot functionality is perfect for that, there is no proxy.
Sharing WiFi: I'm working on my laptop in a park with a weak WiFi signal that only my phone can receive more or less reliably if oriented a certain way. I start a large download. The WiFi signal gets too weak even for the phone. The phone switches to mobile data, my phone bill gets more expensive.
Sure, I can disable mobile data. The problem is that the use of mobile data happens accidentally and I don't want to have another setting to change and another thing to worry about.

Yes sadly with modern Android developments there are very few realistic ways to reliably determine whether traffic from a device is from a WiFi or Mobile Data radio.
At the socket level that tetherfi operates on, there is no way that I know of to determine which traffic source is being used.
Let me be clear, the solution certainly may exist, but is outside of my ability to discover and implement. I welcome any contributions for this feature but will not be able to add this feature as things currently stand.

OK, good to know. Thank you for the explanation!


Expose presentCodeRedemptionSheet

Hi @rwrz
Could you please clarify your request?

Sure.
On iOS, there is this method for the users to redeem offer codes. You guys already did the iOS SDK for it, just need to expose it on the Flutter SDK.
Qonversion docs link.

It is a great idea. Thanks for the suggestion!
At the moment we can't estimate the details on when this will be implemented.  So let's keep in touch :raised_hands:

The method presentCodeRedemptionSheet was added to Qonversion release 4.6.0


Testing npm run deploy with local xampp server failed

After reading the docs I would assume that after running npm run deploy I just copy/paste the dist folder into a server and just browse to that url
I copy pasted the dist folder into the htdocs but when I browse to localhost/dist I get the following errors:
ERR_CONNECTION_REFUSED

What could I be doing wrong?

ok, after testing with npm run deploy:prod and browse to localhost/dist it didn't worked, but when I moved the files to the root folder and browse to localhost it works. May be a bug or something.


IOPS throttling of AZCopy

Which version of the AzCopy was used?
10.3.1
Which platform are you using? (ex: Windows, Mac, Linux)
Linux Ubuntu on a Azure VM with standard HDD
What command did you run?
azcopy sync PATH BLOBPATH_WITH_SAS --delete-destination=true --put-md5
What problem was encountered?
The Azure VM was shut down because the osdisk IOPS limit was reached. The azcopy executable is on the osdisk and the data on the datadisk.
Values received from azure support:
We also see the IOPS limit both on the OS and Data disk is been throttled during the mentioned time .

Below are the values for your reference :
OS disk :
Read IOPS: 730 (!!!)
Write IOPS : 82
Data disk :
Read IOPS: 530
Write IOPS : 350

The weird thing is that it is the osdisk that got the VM shut down. Can this be related to the logs (would be weird since it was read IOPS that was the highest). Are there any other read operations that could cause this?
Is there a way we could throttle the IOPS to stay below the limit?
How can we reproduce the problem in the simplest way?
Check IOPS when running azcopy sync.
Have you found a mitigation/solution?
Not yet. We used blobxfer to do the copy.
Thanks a lot for your help and insights

Hi @EvertEt, thanks for reaching out!
Could you please tell us a bit more about the VM that you are using? Which SKU is it? And what kind of disk?
Could you please also elaborate on the data size that you are transferring? How many files were there?
AzCopy does keep track of transfers with a set of job plan files stored under $HOME/.azcopy by default, but 730 IOPS is excessively high, so I'm not sure what caused the high disk usage. Btw was anything else running at the same time? Was AzCopy the only tool that was accessing the OS disk?

The VM was a NC6 with a 60 GiB Standard HDD OSDisk and a 500 GiB Standard HDD datadisk.
The folder copied contains 40,211 blobs; 89,135,360,201 bytes. It contains some large files (Up to 9GB and smaller ones of a few KBs)
I do believe AZCopy was the only process running. As the VM restarted, our job also restarted, so it kept happening as well.

@EvertEt  I asked around in the High Performance Compute team (where VMs like the NC6 are often used). No-one seems to have heard of us ever shutting down any VM due to high IOPS.   What indications do you have that that was the reason why your VM shut down?
BTW, even if the shutdown was caused by something else, we still have the mystery of why the 730 read IOPS were happening.  But it would be good to understand more about why the shutdown is believed to relate to the IOPS.

Thanks for the effort you are putting into this!

What indications do you have that that was the reason why your VM shut down?

Since in our live logs we can see it shuts down during AZCopy transfer.

But it would be good to understand more about why the shutdown is believed to relate to the IOPS.

This is what azure support told me, and we can see the following health event:

We're sorry, your virtual machine is unavailable because of connectivity loss to the remote disk. An unexpected problem is preventing us from automatically recovering your virtual machine.

Where the diagnostics returned the following (like support confirmed):

This unexpected occurrence was caused by an Azure initiated VM shutdown triggered by detection of temporary IO transaction timeouts between the physical host node where your VM was running, and the Azure Storage services where your Virtual Hard Disks (VHDs) reside.

Hmm. Interesting.  Technically, "timeouts" don't sound exactly the same as "we shut it down because of high IOPS", but I guess the assumption is that the high IOPS were the cause of the timeout. Ze and I might have to look into that a bit more.
I've been unable to figure out why AzCopy would have that many read IOPS. How sure are out that no other app was generating read IOPS at the time?
Finally, the problem is reproducible, right?  If so, you can do an experiment if you want: set the environment variables AZCOPY_LOG_LOCATION and AZCOPY_JOB_PLAN_LOCATION to point to a different drive, then run it again, and see if this time that high IOPS are reported on the new drive.
One last question: which version of Ubuntu were you using?

Update: yes, the shutdown from high IOPS situation is documented

VMs might be temporarily shut down when I/O requests are consistently throttled because the volume of I/O operations per second (IOPS) exceeds the I/O limits for the disk. (Standard disk storage is limited to 500 IOPS.)

https://docs.microsoft.com/en-us/azure/virtual-machines/troubleshooting/understand-vm-reboot
This is the first time we've ever seen it affect AzCopy, and there's been a lot of AzCopy usage!  Which makes we wonder, again, if something else caused those IOPS.

99% percent sure. We have a single python script running that calls AZCopy at the end. Only something from the machine or ubuntu itself would be running at the same time.
The experiment sounds like a good plan, but sadly we don't have time to reproduce at the moment since we are quite busy.
Ubuntu version is 18.04-LTS.

This is the first time we've ever seen it affect AzCopy, and there's been a lot of AzCopy usage! Which makes we wonder, again, if something else caused those IOPS.

We have also been using AZCopy for lots of different payloads so it might be very case specific indeed. We assume it is the first time but since the VM just restarts silently we are not actually sure

Thanks for the info.  I'm not exactly sure where we should go from here on this issue. I'll discuss with @zezha-msft tomorrow.

I propose to wait to hear more feedbacks (perhaps keep this issue open and see if anyone else encounters it), and decide on the action to take. It's important to confirm that it wasn't a fluke.


Can't get bgset to work

This has been a really useful script for lazyloading individual background images via unveilhooks, but I can't seem to get the bgset plugin to work at all.  In Chrome it seems to load at least one image, but doesn't switch out when the viewport changes.  It doesn't request the images at all in Firefox or Safari.
I've tried it locally with picturefill, rias, and respimage, but none of them seem to do the trick.
Am I missing something really simple?
Link here:
http://codepen.io/TPDBrendan/pen/azyYKx

I'm very sorry. Will look into this soon.

The plugin was actually working, but had some problems, because the polyfill (respimage or picturefill) can be loaded to late. The fact, that Chrome doesn't change the size, was due to the fact that I was to lazy to also implement this.
I just changed and improved those things and here is a working example of this plugin:
http://jsfiddle.net/trixta/bfqqnosp/embedded/result/
Please let me know whether this helps.

Hey, thanks for taking the time to work on this.  I'm not sure if I'm missing something, but it looks like it works upon load in Firefox, but not on resizing the viewport, and that it doesn't work at all in Chrome.  I don't know if I'm not looking at it correctly, though.
Before finding your plugin, I was setting the sources of different image sizes as data-attributes and then using enquireJS to inline them.  This works just like a CSS media query across all browsers, so I'm going to stick with that.
I was just hoping I could use bgSet because I really like how unveilhooks had worked for non-responsive BG images so far and I was hoping to use one less script and get the lazyload effect with media queries.  I appreciate the help, though!

@TPDBrendan
Thx, for feedback. I think you are looking at this in a wrong way. But please help me to confirm, that it is not a bug.
The data-bgset feature is basically based on the native srcset feature. The srcset feature means, that all images in the list are the same image, but only have different image sizes. If the browser has an image that is large enough it will not switch the source.
You can test this by doing the following steps:

open http://jsfiddle.net/trixta/bfqqnosp/embedded/result/
resize your window to make it less than 800px wide
clear your cache
reload
resize your window to make it wider than 1200px

You should see that it image will be switched than. Can you confirm this.
What you seem to need is something like a mediaset feature, which is working with media queries:
Something like this:
<div class="lazyload" data-bgset="img-1.jpg (max-width: 480px), img-2.jpg (max-width: 1024px), img-3.jpg">
</div>

Is this working for you know? Can you please re-test in general as also in Chrome (Could not reproduce this with Chrome Canary or Chrome stable)

In case you are interested in this, I can work something out. I think a lot of people might want something like this. (As long as object fit isn't supported by modern IE). Just let me know...

I thought I must be missing something - I see what you mean now.  I would certainly use it if you were to work on it and I think you're right that this would be useful for a lot of people.  Yea, the mediaset example you gave above is the format I was expecting.  Here's the way that my current solution works using enquireJS:
// HTML

// JS
enquire.register("screen and (min-width:768px)", {
  // Triggered when a media query matches.
  match : function() {
    $('.billboard-image').each(function(){
        var lrg = $(this).data('largebanner');
        $(this).css('background-image','url(' + lrg + ')');
      });
  },
  unmatch : function() {
    $('.billboard-image').each(function(){
      var sml = $(this).data('smallbanner');
      $(this).css('background-image','url(' + sml + ')');
    });
  }

});

I have just extended the bgset plugin to also support art direction.
Here is the documentation:
https://github.com/aFarkas/lazysizes/tree/gh-pages/plugins/bgset
Here is the demo:
http://jsfiddle.net/trixta/bfqqnosp/embedded/result/
Note: You need to update also lazysizes itself not just the plugin to get this to work.
Please close, if this makes you happy ;-).

Just did some test and I might have found and fixed the problem in your enviroment. The fix is included in the master branch. Note: that you need to update both lazysizes core as also the bgset plugin for this.
Another nice feature, I just added is a extreme lightweight respimg polyfill for lazySizes. Instead of being a full polyfill, I constrained the feature set so that you can actually implement all use cases of responsive images, but with a limited markup syntax, as result it is extreme small and fast.

@TPDBrendan
Could you test the 1.0.0-RC1 version for me and report if your problem still occurs?

Hey!  Sorry for the delay, yesterday was quite busy after the long weekend.  Unfortunately, it still seems to be lagging a bit on my local version.  It's odd, it seems to have the most difficulty when I'm decreasing viewport width, but not as much when increasing it.  The link below is a screencast of the issue in question, I hope that helps a bit.
https://db.tt/u0I4cdXU

Sorry, this doesn't ring a bell. If it is possible you can mail me your testcase or an url, so that I can debug the problem.

If you can mail me or if side is online drop a link would be great. Until then I'm closing this


Download Agda binary if not exists in the path

Agda's CI/CD currently builds binaries for every commit on the master branch and produces the artefact https://github.com/agda/agda/blob/master/.github/workflows/deploy.yml
The extension can exploit the CI/CD to download artefact if agda is not found in the path.

Where can I find binaries for official releases (non-nightly builds)?

We haven't released these binaries officially. Maybe the requested feature could be added as an "experiment feature". Then, once I sort out https://github.com/agda/agda/issues/6384, we could make the feature official?

Sounds good, we can start with the nightly binaries first!


TryUpdateModelAsync fails with argument null exception Parameter name: metadataProvider

I have an API controller with a Post method like this:
[HttpPost]
public async void Post([FromBody]Page page)
{
    if(ModelState.IsValid) {
        using (var session = _documentStore.OpenAsyncSession())
        {
            var p = await session.LoadAsync<Page>(page.Id);

            if (await TryUpdateModelAsync(p))
            {
                await session.StoreAsync(p);
                await session.SaveChangesAsync();
            }
        }
    }
}

I issue the request using Fiddler like this:

The model binds correctly:

But TryUpdateModelAsync fails with the following exception:

System.ArgumentNullException was unhandled by user code
  HResult=-2147467261
  Message=Value cannot be null.
Parameter name: metadataProvider
  ParamName=metadataProvider
  Source=Microsoft.AspNet.Mvc.Core
  StackTrace:
       at Microsoft.AspNet.Mvc.ModelBinding.ModelBindingHelper.TryUpdateModelAsync[TModel](TModel model, String prefix, HttpContext httpContext, ModelStateDictionary modelState, IModelMetadataProvider metadataProvider, IModelBinder modelBinder, IValueProvider valueProvider, IList`1 inputFormatters, IObjectModelValidator objectModelValidator, IModelValidatorProvider validatorProvider)
       at Microsoft.AspNet.Mvc.Controller.TryUpdateModelAsync[TModel](TModel model, String prefix, IValueProvider valueProvider)
       at Microsoft.AspNet.Mvc.Controller.TryUpdateModelAsync[TModel](TModel model, String prefix)
       at Microsoft.AspNet.Mvc.Controller.TryUpdateModelAsync[TModel](TModel model)
       at src.Controllers.PageController.<Post>d__4.MoveNext() in C:\Projects\Sandbox\aspnet5\src\Controllers\PageController.cs:line 44
  InnerException:

Is this an issue or am I doing something wrong?

The problem only seems to appear when I use Fiddler or Javascript to post my data, not if I put the same code on my HomeController and post the form, TryUpdateModelAsync works as expected?

This seems really strange. Is there a repo somewhere we could look at?

@rynowak Sure, here is the controller I use and here is the script I use to post the data. The HomeController works with the exact same code.

Your issue here that the action method is declared as async void - it should be async Task

See: https://github.com/aspnet/Mvc/issues/3901#issuecomment-170610622

@rynowak I'd love to throw in that case but as you said yourself, it is difficult to reliably do so.
Closing this bug as it's effectively a dup of https://github.com/aspnet/Mvc/issues/3901

Changed the return type to Task and it seems to work as it should, thanks @rynowak for your the help!


how do you launch GFPGAN

I got everything installed but don't know how to launch the program/gui on win 11.

Hi
Thanks for using my tool! 🔥
GFPGAN itself is a command-line only tool.
Made to easily integrate into any python project.
But you can just launch a commandline/terminal window and navigate to the folder where you installed gfpgan to:
Then put your photos in the folder inputs/whole_imgs, then finished photos will then be in results folder.
And run the following command:
python inference_gfpgan.py -i inputs/whole_imgs -o results -v 1.3 -s 2

how to solve that?

Can you try pip install cv2 ?

@spaceyspeed @rraj000000
Could you figure it out, or do you need further help?

Can you help me with these errors. The installation was completed successfully. I used the command indicated on this page but was unsuccessful. Thank you.

Hi
could you try this:
pip uninstall numpy
pip install numpy<2

Hi could you try this:
pip uninstall numpy
pip install numpy<2

Thank you for your help, this resolved the error, but another one appeared.

Unfortunately the same error persists.
I selected without CUDA.

Unfortunately the same error persists. I selected without CUDA.

https://github.com/CrazyWolf13/auto_gfpgan-installer?tab=readme-ov-file
take a look at the steps recommended on the warning at the top of the readme, they should solve your problem :)

Are you referring to this change?
%localappdata%\Programs\Python\Python311\Scriptsdegradations.py edit with notepad from torchvision.transforms.functional_tensor import rgb_to_grayscalefrom torchvision.transforms.functional import rgb_to_grayscale
Only Python310 was automatically installed and the degradations.py file does not exist in the Scripts folder.

that is not good then, could you fully remove the gfpgan folder and re-run the script again?
after installing python11 ?
You can install python11 by running winget install Python.Python.3.11

I'll close this ticket, as it's getting a bit messy,
@hesicos if your issue persists after installing python11 and running the script again, after removing the gfpgan folder and uninstalling python, please create a new ticket.


Update README with Screenshots and Progress

Place image (PNG) in /screenshots:

File format: ReportID_X.png
X = the report number that corresponds with the table on README and your updated application.

Update README.md with new counts. See "x requirements of 32 have been implemented". Change this number and update percentage.

Update the README.md report ID table with YES to close off item.

Ferdinand and David to update screenshots based on developed INDEX html page

Readme Updated with screen shot and index html page created


bind Redis to loopback interface to avoid open Redis port on host machine

The Redis port is bound to <IP_ADDRESS> when running docker-compose with the given docker-compose file. From my understanding, the Webrecorder does not seem to need the Redis port to be open to function properly, so this would improve overall security on host machines without additional security measures in place.

Thanks for pointing this out, actually we can probably just remove this port binding altogether


Datasets library not suitable for huge text datasets.

Hi,
I'm trying to use datasets library to load a 187GB dataset of pure text, with the intention of building a Language Model. The problem is that from the 187GB it goes to some TB when processed by Datasets. First of all, I think the pre-tokenizing step (with tokenizer.map()) is not really thought for datasets this big, but for fine-tuning datasets, as this process alone takes so much time, usually in expensive machines (due to the need of tpus - gpus) which is not being used for training. It would possibly be more efficient in such cases to tokenize each batch at training time (receive batch - tokenize batch - train with batch), so that the whole time the machine is up it's being used for training.
Moreover, the pyarrow objects created from a 187 GB datasets are huge, I mean, we always receive OOM, or No Space left on device errors when only 10-12% of the dataset has been processed, and only that part occupies 2.1TB in disk, which is so many times the disk  usage of the pure text (and this doesn't make sense, as tokenized texts should be lighter than pure texts).
Any suggestions??

Hi ! Looks related to #861
You are right: tokenizing a dataset using map takes a lot of space since it can store input_ids but also token_type_ids, attention_mask and special_tokens_mask. Moreover if your tokenization function returns python integers then by default they'll be stored as int64 which can take a lot of space. Padding can also increase the size of the tokenized dataset.
To make things more convenient, we recently added a "lazy map" feature that allows to tokenize each batch at training time as you mentioned. For example you'll be able to do
from transformers import BertTokenizer

tokenizer = BertTokenizer.from_pretrained("bert-base-uncased")

def encode(batch):
    return tokenizer(batch["text"], padding="longest", truncation=True, max_length=512, return_tensors="pt")

dataset.set_transform(encode)
print(dataset.format)
# {'type': 'custom', 'format_kwargs': {'transform': <function __main__.encode(batch)>}, 'columns': ['idx', 'label', 'sentence1', 'sentence2'], 'output_all_columns': False}
print(dataset[:2])
# {'input_ids': tensor([[  101,  2572,  3217, ... 102]]), 'token_type_ids': tensor([[0, 0, 0, ... 0]]), 'attention_mask': tensor([[1, 1, 1, ... 1]])}

In this example the encode transform is applied on-the-fly on the "text" column.
This feature will be available in the next release 2.0 which will happen in a few days.
You can already play with it by installing datasets from source if you want :)
Hope that helps !

How recently was set_transform added? I am actually trying to implement it and getting an error:
AttributeError: 'Dataset' object has no attribute 'set_transform'
I'm on v.1.2.1.

Yes indeed it was added a few days ago. The code is available on master
We'll do a release next week :)
Feel free to install datasets from source to try it out though, I would love to have some feedbacks

@lhoestq I wanted to try datasets 2.0 as am dealing with huge text datasets as well and I would benefit from these new features. Which branch? From master I see now only version 1.3. Thanks!

For information: it's now available in datasets 1.3.0.
The 2.0 is reserved for even cooler features ;)

Hi @alexvaca0 , we have optimized Datasets' disk usage in the latest release v1.5.
Feel free to update your Datasets version
pip install -U datasets

and see if it better suits your needs.


[INFINITY-1074] Authorize marathon to launch tasks as root in strict mode tests

I've also refactored setup_permissions.sh to be organized by permission, and I've changed the comments in that file, which were incorrect.

retest this please

@mgummelt dcos-cassandra-service/test.sh and dcos-kafka-service/test.sh both call setup_permissions.sh. They would have to be updated, too.

@susanxhuynh good catch.  I'll update those.

That shouldn't block this PR, though, since this has to go in first.

I just removed the interface change to setup_permissions.sh

cc @susanxhuynh for approval.


Missing "./CodeMirror.svelte" specifier in "svelte-codemirror-editor" package error on npm run build

Hi there,
I've run into an error with this Svelte library. When I try to build my project, I get an error saying that ./CodeMirror.svelte is missing.
Here's the entire error log:
[commonjs--resolver] Missing "./CodeMirror.svelte" specifier in "svelte-codemirror-editor" package
error during build:
Error: Missing "./CodeMirror.svelte" specifier in "svelte-codemirror-editor" package
    at e (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/dist/node/chunks/dep-V3BH7oO1.js:21914:25)
    at n (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/dist/node/chunks/dep-V3BH7oO1.js:21914:627)
    at o (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/dist/node/chunks/dep-V3BH7oO1.js:21914:1297)
    at resolveExportsOrImports (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/dist/node/chunks/dep-V3BH7oO1.js:29466:20)
    at resolveDeepImport (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/dist/node/chunks/dep-V3BH7oO1.js:29485:31)
    at tryNodeResolve (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/dist/node/chunks/dep-V3BH7oO1.js:29210:20)
    at Object.resolveId (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/dist/node/chunks/dep-V3BH7oO1.js:28978:28)
    at file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/node_modules/rollup/dist/es/shared/node-entry.js:18670:40
    at async PluginDriver.hookFirstAndGetPlugin (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/node_modules/rollup/dist/es/shared/node-entry.js:18570:28)
    at async resolveId (file:///home/ben/Documents/svelte-report-creator-v1/client/node_modules/vite/node_modules/rollup/dist/es/shared/node-entry.js:17239:26)

Let me know if I didn't give enough information for this issue, and what I could do to help solve this issue.
Development setup

Svelte 4.2.8 (not using SvelteKit)
Vite 5.0.8 (I did the exclusion for the optimization. Even tried the method in #27)

Steps to duplicate

Run npm install svelte-codemirror-editor
Run npm run build

Same issue in my case :-(

I can't replicate your issue.
Can you create a replication project on Github?


Adding some RTP files

I made 3 battle animations and 1 midi track. All CC0.

Thank you for your contribution.
The battle animations do not look like simple repaints, so imo they are fine to add this way (the animations are so simple, is hard to  :)
Though have a change request for the arrow:

Can you make it a bit longer in both directions? Compared to the "real" RTP arrow it is quite short.
Take a look at Cell 3 of the original file. Here the chopped off arrow is 5 pixel lower than in the previous frame. I guess this shall simulate some motion and indicate that the arrow in now inside the target. Can you adjust this?

(Also the dimensions of the files are different, but this is something we can easily fix afterwards)

The files must be in a compatible indexed color mode, with the color 0 being the background color.
Fixed the issue, and uploaded it here:
https://github.com/EasyRPG/RTP/pull/34/commits

Thank you! your files have been added in #34 with the corresponding authors file update.

feel free to join us on discord, feedback there is a lot faster https://t.co/0MJZlivP9D


tilegrid is recreated often in ol.source.TileWMS

why is the tile grid recreated all the time in ol.source.TileWMS? Why is the result of ol.tilegrid.getForProjection not cached as this.tileGrid?
https://github.com/openlayers/ol3/blob/master/src/ol/source/tilesource.js#L181:L185

ah my change doesn't make sense since there could be multiple views with a different projection calling this function through the renderer

Although maybe we should consider caching the result in case the projection is the same?

ol.tilegrid.getForProjection caches the result - no need to cache it again elsewhere.

ah indeed, I had missed that
thanks @ahocevar


google scholar like citekey

Zotero version:<IP_ADDRESS>
BBT version:5.6.11
Support log ID:NHSQBSMH-refs-apse
Expected behavior:afanasyev2019towards
Actual behavior:afanasyev2019blockchainbased
It seems the BBT ignore words like "towards, a, for..." when generating citekey

By default, it does, yes. If you don't want that, you can use either Auth instead of auth, or edit the skipWords to remove towards etc.

I want to keep towards, but remove the, an,etc as google scholar BibTex.
For example, the google scholar BibTex citekey of  the paper "Towards Blockchain-based Multi-Agent Robotic Systems: Analysis, Classification and Applications" is afanasyev2019towards.
While, for the paper "The swirlds hashgraph consensus algorithm: Fair, fast, byzantine fault tolerance", it is baird2016swirlds.
If I use [auth:lower][year][veryshorttitle:lower] in BBT, I can get baird2016swirlds, but I can't get afanasyev2019towards. While if I use  [auth:lower][year][Title:lower:select=1,1], I can get afanasyev2019towards, but I can't get baird2016swirlds.

So why not just remove "towards" from the skipwords and use auth?

How to remove "towards" from the skipwords

At the top of the page I linked to you will find instructions on how to get to the hidden preferences, once there, the skipwords is just a string of comma separated words, and you can just remove towards from it.

@luoxiaojie9527: Did you try this: https://github.com/retorquere/zotero-better-bibtex/issues/1705#issuecomment-749546297?

Hahhah, it works after I remove "towards" from the skipwords list under the instruction of you mentioned. Thank you！ @retorquere


Quantification of the impact of resistance model improvements

I'm opening this issue to discuss how we quantify the impact of resistance model improvements and in general how we compare different models. I think the comparison on the level of partial factors is not sufficient and can even be misleading. I suggest to make the comparison on the level of design resistance values obtained from optimized semi-probabilistic formats. For the formulas we consider the design resistances can be considered as proxies of material usage and in turn cost.
For example to have an estimate of what kind of gain we may expect from using MC2010 instead of EN1992, I used the model uncertainty coefficient of variation (cv) of MC2010 (see #6 ) in combination with the rest of the EN1992 model (approximation). Here are the results:

model
cv of theta_R
C_Rck
gamma_C
C_Rck / gamma_C

EN1992
0.2376
0.1471
1.38
0.107

EN 1992 with MC2010 cv
0.2082
0.155
1.32
0.118

Because the two models are the same except C_Rck and gamma_C the gain from a reduced model uncertainty can be easily computed as single value:
0.118 / 0.107 = 1.10
(This is the same as what we would get by only comparing the partial factors if we used the same C_Rck value for both models.)
If the two models differ in more than just two parameters then the comparison could be done as the ratio of design values considering all design scenarios (that would be equivalent to the above shown case (special case)). This would lead to a distribution of ratios, we could use the mean of it to indicate the average gain.
What do you think?

Though I think it's an interesting exercise, I have two reservations:

Are we really interested in quantifying the model improvement? I thought there we do this variation to show the impact of using another resistance model (with different model uncertainty) on the partial factor.
If we assume that the shear database contains experiments that are representative for all design applications we have in mind for the shear formula, would your proposal then lead to a different outcome than comparing partial factors?

Probably, this assumption is not correct, and the measure you propose can correct for a possible misinterpretation that one resistance model is better than the other, while it may only be better fit with the observed data. Is that indeed what you want to indicate?

Are we really interested in quantifying the model improvement? I thought there we do this variation to show the impact of using another resistance model (with different model uncertainty) on the partial factor.

I only called it model improvement because last week talked about using an alternative model that is more accurate, i.e. has a smaller model uncertainty, but what I present above is not tied to improvement. In my opinion it (or something similar) is needed for a fair model comparison. Ultimately, the partial factor is not telling us much if we also change at least another parameter in the formulas, e.g. C_Rck. How would you make the comparisons?

If we assume that the shear database contains experiments that are representative for all design applications we have in mind for the shear formula, would your proposal then lead to a different outcome than comparing partial factors.

I think the question is not related to weather the experimental data is representative or not. I do not correct this as I do not have information to do so. A comparison of only the partial factors is going to be distorted (in the case that I think we advocate, i.e. also calibrate the C_Rck or whatever parameter) by the other parameter that we touch in the formula.

As discussed during a phone call:

The proposed comparison is indeed relevant to give some interpretation to the outcomes;
Since it's not hard to explain, we think it is manageable to add it to the paper and it will not distract the reader from the main message.

In conclusion, the proposed comparison will be added.

The required changes are implemented in #20.


tabs: URL referencing

Given:
<!-- /some/page.md -->
<toggle>
<tab title="Foo">
</tab>
<tab title="Bar">

some text

### sub section

</tab>
</toggle>

Should be able to use:

[x] /some/page?tab=Bar (or /some/page#Bar?) to auto-open the appropriate tab (p1-important, #130)

[x] /some/page/?tab=Bar#sub-section (or /some/page#Bar-sub-section?) to auto-open the tab and scroll to containing section (p2-nice-to-have)

[ ] remember the ?tab=<arg> when navigating to a different page (p2-nice-to-have)

Other tasks:

[x] initial tabs implementation (#113)

[ ] update styling a little (#136, TODO: follow-up)

[x] ~blocking #122~
related #70

@julieg18 yes

remember the ?tab= when navigating to a different page (p2-nice-to-have)

Are we sure we want this behavior? Currently it seems like we don't have many similiar tab names minus github, gitlab, and   bitbucket 🤔

Are we sure we want this

yes, the GH/GL/BB tabs are important and frequent enough to make this desirable on https://cml.dev/doc. Potentially could clear ?tab=X only when navigating to a page that doesn't contain an X tab.


[CDK task] Support for CDK_DEFAULT_REGION & CDK_DEFAULT_ACCOUNT

When deploying or removing a cdk workload, the CDK_DEFAULT_REGION & CDK_DEFAULT_ACCOUNT should contain the target region and account.

released as part of v0.9.5


Improved Docs

Improved the readme.md file and added new information.

Sorry, but the longer sentence essentially has the same meaning as "encyclopedia that anyone can edit."