Unrecognized param

(not sure i'm posting on right repo, since i'm using API blueprint from @danielgtaylor Aglio)
My API allow for parameters prefixed with "@", for instance "@name=".
However, describing such an endpoint result in the following error:
>> Line 134: ignoring unrecognized list item, expected parameter discussion, e.g. '<parameter name> ... lorem ipsum' (warning code 5)

Where line 134 is:
+ Parameters
    + @meta (optional, string, `John`) ... Full text search on `meta` key.  Replace `meta` with the name of the meta you wish to search on.

However, i believe such a parameter is actually valid HTTP. Would it be possible to improve snowcrash to handle this use case ? (and possibly more exotic charsets).
I tried to HTML escape / encodeURIComponent without success.

Migrated to apiaryio/drafter#644


jax.lax.cond evaluates None as True

When using jax.lax.cond, passing None as the first argument causes the true_fun argument to get called. This is contrary to the behavior one would expect for Python if statements.
For example, running the following
import jax

true_fun = lambda x: True
false_fun = lambda x: False

for pred in [True, False, None]:
	print(pred, jax.lax.cond(pred, true_fun, false_fun, None))

gives the following output:
True True
False False
None True

I am using the newest PyPI versions of jax and jaxlib (0.3.14).

I suppose a better behavior here would be to raise an error when None is passed in a place where a JAX array is expected.

That would work. The only thing to keep an eye on is the documentation of jax.lax.cond, which currently says:
cond() has equivalent semantics to this Python implementation:

  def cond(pred, true_fun, false_fun, *operands):
    if pred:
      return true_fun(*operands)
    else:
      return false_fun(*operands)

Since bool(None) is False, some caveat would be helpful to warn users about unexpected errors that wouldn't arise in the Python equivalent.
Regardless, throwing an error is still better than the present situation, since the branch that is selected is currently the opposite of what the documentation implies.


"Insert code" option is inserting extra new line, and not count it on line dislocation

Hi. I'm doing a tour, with requires inserting a mock in a unit test.
And found this bug:
The Insert Code option, in inserting the "new line" character at the end of the code block too:
. . . \n
```
The "line" parameter of the tour thought, just add up the amount of code lines, not the extra \n bit.
So it offsets the tour box from the original line.

Without this new line between code and the closing back-ticks, it doesn't match the code block sintax, so the option is not available

Thanks for reporting this! This is definitely an odd bug, and so I'll take a look at see what's going on here. Stay tuned!


Surface-capabilities queries return out of date information due to caching

Issue
PhysicalDevice::surface_capabilities and PhysicalDevice::surface_capabilities_unchecked return out of date information after they're first called due to the new caching functionality introduced in #2006.
Reproduce
Run an example (e.g. triangle) and resize the window. Swapchain recreation always fails due to SwapchainCreationError::ImageExtentNotSupported because the surface capabilities in Swapchain::validate are out of date after the first call to surface_capabilities_unchecked.
This means caching surface queries is unfortunately not viable as surface properties can change at runtime for a given surface and physical device.

not present in 0.31.1 btw


bugfix: add onPopupClick listener to close when closeQuery matches

Before submitting the PR:

[x] Does your PR reference an issue? If not, please chat to the team on Discord or GitHub before submission.
[x] Did you update and run tests before submission using npm run test?
[x] Does your branch follow our naming convention? If not, please amend the branch name using branch -m new-branch-name
[ ] Did you update documentation related to your new feature or changes?

What does your PR address?
Fixes #1229
Adds an onPopupClick handler to close when an element which matches closeQuery or is a descendant of an element that matches closeQuery, as in onWindowClick.
I've moved the logic for closing based on closeQuery to its own function, closeOnQueryClick since the code is reused by both onWindowClick and onPopupClick.

@Cucumberrbob I've submitted a quick change to this PR. The goal with this Action is to create a handful of event handlers that can be mixed, matched, and reused when possible. The onPopupClick handler was redundant with the existing onWindowClick, so I've dropped it in favor of this.
Let's be careful not to introduce superfluous code whenever possible. That can greatly impact maintainability in the long run. Our goal should be to reuse and extend what already exists first and foremost, and only introduce additions when absolutely required.
With this change it's working as expected so I'll aim to merge now. Thanks!


DASHBUILDE-160: Data source selector

Data source selector Implementation+ tests.
Further details at https://issues.jboss.org/browse/DASHBUILDE-160
@wmedvede Can you review please?

Hello @dgutierr It looks good. Just a quick note, I believe that it's still missing a way for providing a different SQLDataSourceLocator in a pluggable by CDI or SPI fashion. If I'm not wrong current code only permits the SQLDataSourceLocatorCDI. ¿?

@wmedvede you can provide your own CDI implementation, for instance:
https://github.com/dgutierr/dashbuilder/blob/8e0647440d130c155ec73708f74112f4c74b2364/dashbuilder-backend/dashbuilder-dataset-cdi/src/test/java/org/dashbuilder/dataset/SQLDataSourceLocatorMock.java
Notice the usage of the @Specializes annotation, which forces the CDI subsystem to lookup the extended class.
Another way of plugging a different impl is to invoke to the proper SQLDataProvider setter:
https://github.com/dashbuilder/dashbuilder/blob/master/dashbuilder-backend/dashbuilder-dataset-sql/src/main/java/org/dashbuilder/dataprovider/sql/SQLDataSetProvider.java#L158

Hello @dgutierr , good point, it can be extended by the @Specialize strategy. I have now a specialized that works in the kie-wb, etc.
Not sure if I have to approve this PR, but form my side it's a +1.
Thanks.

Merged: 2190d78be2c8e50528b580928281617a939160af


[Networking] Decouple Evaluator from Contributor

Idea
We need to have two machines (hence 2PC) for the process: a garbler (in Tandem's language, Contributor) and a node (that will evolve towards being the VM - in Tandem's language, Evaluator).
The Contributor is the client/who garbles. In our current architecture, it would be the Secure Binder.
Right now, our implementation on cryptomata simulates the execution by creating a local Evaluator and Contributor, which has effectively zero latency and ideal network conditions. We now need to separate both entities so that each lives in a different machine.
Associated with this: on the process of building a circuit, we need to generate it separately, instead of building AND executing. They need to be separate things.
Resources

Tandem is doing something interesting with their tandem_http_server and tandem_http_client.

Forgot to close this. Solved with Evaluator, Garbler and the circuit_macro


Migrate test suite to GoogleTest framework

This PR does the following:

Introduces GoogleTest as the testing framework for RTNeural
Introduces two test targets: rtneural_test_unit and rtneural_test_functional

Unit tests contain a small new example demonstrating unit testing patterns for Activation layers
Functional tests are simply the old test header files, moved to the new target. Some old assertions have been simplified to make use of test assertions built into GoogleMock, and there have been a couple of very minor clarifying refactors in the test code. No new functional tests have been added.

Adds a new README file in the tests folder, explaining how to build and run the tests (or a subset of them!)
Updates test implementations to use #include <RTNeural/RTNeural.h>, to match the intended client code inclusion pattern

Thanks for the PR! The changes look good overall, I just had a couple of notes.

For the RTNeural header, I think using #include <RTNeural/RTNeural.h> in both the tests and client code is the way to go.
I think there's some kind of linker error happening with gmock on MSVC. I think this can be fixed by adding this to our test targets:

set_property(TARGET foo PROPERTY
  MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>")

No problem - thanks for the review!

Regarding the include directories and include style, I've reverted the target property and have updated the tests to use the new style, and
regarding the gmock linker error - oops! Missed that one. I think the solution to this is in the GoogleTest README, so I'll try that first. If it doesn't work, let's try doing it manually as you suggest?

Awesome, thanks for the update! Yeah, trying the CMake option provided by GoogleTest sounds good.
I think there's also a different include path thing happening in some of the examples, but otherwise we should be good to go :)

Nice one, thanks for the excellent review @jatinchowdhury18! I've updated the contributors list now and I'm ready for it to be merged too, assuming the CI passes 😁


Battleship game ends abruptly if given bad input

Description
Entering I9ASDF causes the .battleship game to error and end.
Steps to Reproduce
Start a game, enter a valid target, then add more to the message before sending it. Any text will do.
Expected Behaviour
The bot to respond with a red X like it does for non-matching input (such as AAAA)
Actual Behaviour
The game ends abruptly with An error occurred. Game failed
Possible Solutions/Details
The core issue lies on this line:
https://github.com/python-discord/sir-lancebot/blob/3454a88a43bec5d0959495c99b28ed80e96f12fb/bot/exts/evergreen/battleship.py#L262 and this line https://github.com/python-discord/sir-lancebot/blob/3454a88a43bec5d0959495c99b28ed80e96f12fb/bot/exts/evergreen/battleship.py#L230
Specifically, re.Match.string returns the entire text given to match against, and we're using re.match not re.fullmatch. Therefore, any message that contains a match is returned in entirety, rather than the match itself (which could be a portion of the message).
We should probably use re.fullmatch instead.
Would you like to implement a fix?
Note: For high-priority or critical bugs, fixes may be implemented by staff.

[ ] I'd like to implement the bug fix
[x] Anyone can implement the bug fix

i think  this line self.match = re.fullmatch("([A-J]|[a-j]) ?((10)|[1-9])", message.content.strip())  will solve this issue @bast0006


memory explosion from large cases of collecting graph edges

here is a draft fix
for collectEdges 1393
test <- lapply(iRowRange, function(x) {iRow <- x; unique(as.numeric(eval(accessExpr)))})
ctest <- do.call('c', test)
edgesToTest <- lapply(iRowRange, function(x) rep(x, length(unique(as.numeric(eval(accessExpr))))))
cEdgesToTest <- do.call('c', edgesToTest)

This is now fixed on devel.


WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

Hi all,
After setting up Docker, I ran the following command docker run --gpus all -v /home/user/my_mri_data:/data \                       -v /home/user/my_fastsurfer_analysis:/output \                       -v /home/user/my_fs_license_dir:/fs60 \                       --rm --user XXXX fastsurfer:gpu \                       --fs_license /fs60/.license \                       --t1 /data/subject2/orig.mgz \                       --sid subject2 --sd /output \                       --parallel
from https://github.com/Deep-MI/FastSurfer/blob/master/Docker/README.md
with the file path, user ID, and the license key file path replaced accordingly.
However, the following warning showed up:
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
I searched the exact result online and found that many people using Docker on Apple Silicon chips (M1) were having similar problems. The solution, according to the Docker official documentation https://docs.docker.com/docker-for-mac/apple-silicon/ was to add the following command to our existing command
--platform linux/amd64
I tried adding the command after --parallel but the same error still showed up.

After a bit of testing, I found out that shifting the position of --platform linux/amd64 resolved the issue.
Here's the complete code (replace the file path to conform with your environment):
docker run --gpus all --platform linux/amd64 -v /home/user/my_mri_data:/data \ -v /home/user/my_fastsurfer_analysis:/output \ -v /home/user/my_fs_license_dir:/fs60 \ --rm --user XXXX fastsurfer:gpu \ --fs_license /fs60/.license \ --t1 /data/subject2/orig.mgz \ --sid subject2 --sd /output \ --parallel

After a bit of testing, I found out that shifting the position of --platform linux/amd64 resolved the issue.


add a specific enthalpy route for thermodynamic state

This PR adds an enthalpy route to the thermodynamic state (Dry, PhaseEquil and PhaseNonEquil)
It uses equation (2.15) in the design docs to compute the temperature from given enthalpy.

I applied some fixes. Most importantly,h was not extracted from the ProfileSet. With this fix, the dry and non-equil constructors pass consistency tests, and the equil constructor has reasonably low errors. For example,
        @show maximum(abs.(internal_energy.(param_set, ts_phq) .- e_int))
        @show maximum(abs.(internal_energy.(param_set, ts_phq) .- e_int) ./ e_int * 100)

for temperature_tol = FT(1e-2) yields
# Float32
maximum(abs.(internal_energy.(param_set, ts_phq) .- e_int)) = 2258.25f0
maximum((abs.(internal_energy.(param_set, ts_phq) .- e_int) ./ e_int) * 100) = 0.50207645f0
# Float64
maximum(abs.(internal_energy.(param_set, ts_phq) .- e_int)) = 2258.3451461736113
maximum((abs.(internal_energy.(param_set, ts_phq) .- e_int) ./ e_int) * 100) = 0.5020976536465528

and for temperature_tol = FT(1e-3) yields
# Float32
maximum(abs.(internal_energy.(param_set, ts_phq) .- e_int)) = 2251.4688f0
maximum((abs.(internal_energy.(param_set, ts_phq) .- e_int) ./ e_int) * 100) = 0.50056875f0
# Float64
maximum(abs.(internal_energy.(param_set, ts_phq) .- e_int)) = 2252.6098979914677
maximum((abs.(internal_energy.(param_set, ts_phq) .- e_int) ./ e_int) * 100) = 0.5008225364837869

It's a bit strange is that making the tolerance more strict and/or increasing the precision doesn't seem to significantly reduce this error. However, we do have < 1% error in the internal energy after performing saturation adjustment, so maybe this is fine?

Merging to preserve this so that we can start working on parameters

bors r+


FileResolver doesn't detect .java files

I have added jpa2ddl to a project where I also ahve source classes.
jpa2ddl generates nothing, even if configuration is very similar to the one in example.
After debuggging, I found out that FileResolver#listClassNamesInPackage(...) associates the package name to the source folder of my classes (and not the classes folder). As a consequence, contained files are .java which the pathMatcher object can't recognize as .class. And in turn, this prevent my classes from being recognized and creates an empty schema.
Possible solutions are

Tweak the contect class loader to not use the src/main/java folder but the target/classes one
Change the pathMatcher to match .java and .class files. This one is way easier.

Do you use maven or gradle tool?

I've used maven

By default the plugin runs in the process-classes phase which is executed after compiling all classes. The classpath contains the target as well as the src directory. Could you please provide us with the  https://stackoverflow.com/help/minimal-reproducible-example ?


Fallback question for x.gender assumes that person_type is defined

We should add a hasattr() wrapper or change the question order in the YAML file

That would be good for some extra safety, but IMO the real fix is to make it if you pass in 'person' or 'business' to name_fields, it should set person_type for you automatically. I think I ran into that issue for e-filing at some point (where we do need to know which the individual is).

Fixed in cb34836973ff4875be4aec25c4e9ab0c716d2e92


Add doc for widget examples input

Now that https://github.com/huggingface/huggingface_hub/pull/438 & https://github.com/huggingface/huggingface_hub/pull/448 are merged, submitting doc for how to add widget example inputs & provide titles for them.
Read this file & this file
cc: @julien-c @Pierrci

@stevhliu thanks a lot for the great suggestions! I've applied them: added GIF & wrote about the optional example_title


Fix typo in command in oci-rootfs.sh for podman-based hosts

#74 added the oci-rootfs.sh script which automatically selects podman instead of docker when it is available on the system running the oci-rootfs.sh script. This PR fixes the following command in that script:
https://github.com/flatcar/sysext-bakery/blob/11280db5e80530505857385af088525d3be1675a/oci-rootfs.sh#L50
which assumes the use of docker, even when podman is used for all other commands, and which had resulted in the script quitting with an error when podman is selected instead of docker for https://github.com/flatcar/sysext-bakery/blob/11280db5e80530505857385af088525d3be1675a/oci-rootfs.sh#L53
Testing done
"it works on my machine"
On my computer (which is running the https://github.com/ublue-os/bluefin/pkgs/container/aurora-dx custom Fedora OS image), I attempted to run the following command:
 CMD="apk -U add w3m" ./oci-rootfs.sh alpine:latest /tmp/alpine-w3m

Before I made the change in this PR, this command would fail on my computer with the following output:
Trying to pull docker.io/amd64/alpine:latest...
Getting image source signatures
Copying blob 4abcf2066143 skipped: already exists
Copying config 05455a0888 done   |
Writing manifest to image destination
05455a08881ea9cf0e752bc48e61bbd71a34c029bb13df01e40e3e70e0d007bd
Using temporary container 2090805
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/community/x86_64/APKINDEX.tar.gz
(1/6) Installing libgcc (13.2.1_git20231014-r0)
(2/6) Installing libstdc++ (13.2.1_git20231014-r0)
(3/6) Installing gc (8.2.4-r1)
(4/6) Installing ncurses-terminfo-base (6.4_p20231125-r0)
(5/6) Installing libncursesw (6.4_p20231125-r0)
(6/6) Installing w3m (<IP_ADDRESS>30718-r1)
Executing busybox-1.36.1-r15.trigger
OK: 12 MiB in 21 packages
Error: no container with name or ID "2090805" found: no such container

When I had turned on debugging output (by changing the shebang line from #!/usr/bin/env bash to #!/usr/bin/env -S bash -eux), I saw the following output:
+ set -euo pipefail
+ REGISTRY=docker.io
+ ARCH=amd64
+ CMD='apk -U add w3m'
+ FETCH=1
+ '[' 2 -lt 2 ']'
+ '[' alpine:latest = -h ']'
+ '[' alpine:latest = --help ']'
+ IMAGE=alpine:latest
+ FOLDER=/var/tmp/alpine-w3m
+ SUFFIX=
+ '[' amd64 = x86-64 ']'
+ '[' amd64 = x86_64 ']'
+ '[' amd64 = aarch64 ']'
+ '[' amd64 = arm64 ']'
+ IMAGE=docker.io/amd64/alpine:latest
+ DOCKER=docker
+ command -v podman
+ DOCKER=podman
+ rm -rf /var/tmp/alpine-w3m
+ '[' 1 = 1 ']'
+ podman pull docker.io/amd64/alpine:latest
Trying to pull docker.io/amd64/alpine:latest...
Getting image source signatures
Copying blob 4abcf2066143 skipped: already exists
Copying config 05455a0888 done   |
Writing manifest to image destination
05455a08881ea9cf0e752bc48e61bbd71a34c029bb13df01e40e3e70e0d007bd
+ '[' 'apk -U add w3m' '!=' '' ']'
+ ID=2322803
+ echo 'Using temporary container 2322803'
Using temporary container 2322803
+ trap ''\''podman'\'' rm --force --time 0 '\''2322803'\''' EXIT INT
+ '[' 'apk -U add w3m' '!=' '' ']'
+ docker run --name 2322803 docker.io/amd64/alpine:latest sh -c 'apk -U add w3m'
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/community/x86_64/APKINDEX.tar.gz
(1/6) Installing libgcc (13.2.1_git20231014-r0)
(2/6) Installing libstdc++ (13.2.1_git20231014-r0)
(3/6) Installing gc (8.2.4-r1)
(4/6) Installing ncurses-terminfo-base (6.4_p20231125-r0)
(5/6) Installing libncursesw (6.4_p20231125-r0)
(6/6) Installing w3m (<IP_ADDRESS>30718-r1)
Executing busybox-1.36.1-r15.trigger
OK: 12 MiB in 21 packages
+ rm -f /var/tmp/alpine-w3m.tar
+ podman export 2322803 -o /var/tmp/alpine-w3m.tar
Error: no container with name or ID "2322803" found: no such container
+ podman rm --force --time 0 2322803

After the change in this PR, I no longer have errors on my computer when running the same command:
Trying to pull docker.io/amd64/alpine:latest...
Getting image source signatures
Copying blob 4abcf2066143 skipped: already exists
Copying config 05455a0888 done   |
Writing manifest to image destination
05455a08881ea9cf0e752bc48e61bbd71a34c029bb13df01e40e3e70e0d007bd
Using temporary container 2324638
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/community/x86_64/APKINDEX.tar.gz
(1/6) Installing libgcc (13.2.1_git20231014-r0)
(2/6) Installing libstdc++ (13.2.1_git20231014-r0)
(3/6) Installing gc (8.2.4-r1)
(4/6) Installing ncurses-terminfo-base (6.4_p20231125-r0)
(5/6) Installing libncursesw (6.4_p20231125-r0)
(6/6) Installing w3m (<IP_ADDRESS>30718-r1)
Executing busybox-1.36.1-r15.trigger
OK: 12 MiB in 21 packages
2324638

"it works on GitHub Actions's machines"
After making the fix in this PR, I also attempted to use oci-rootfs.sh on GitHub Actions. The version of Podman or Docker available in the ubuntu-22.04 runner appeared to be too old, resulting in this error in the following output (full CI logs here):
Trying to pull docker.io/amd64/alpine:3.19.1...
Getting image source signatures
Copying blob sha256:4abcf20661432fb2d719aaf90656f55c287f8ca915dc1c92ec14ff61e67fbaf8
Copying blob sha256:4abcf20661432fb2d719aaf90656f55c287f8ca915dc1c92ec14ff61e67fbaf8
Copying config sha256:05455a08881ea9cf0e752bc48e61bbd71a34c029bb13df01e40e3e70e0d007bd
Writing manifest to image destination
Storing signatures
05455a08881ea9cf0e752bc48e61bbd71a34c029bb13df01e40e3e70e0d007bd
Using temporary container 2501
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/v3.19/community/x86_64/APKINDEX.tar.gz
(1/6) Installing libgcc (13.2.1_git20231014-r0)
(2/6) Installing libstdc++ (13.2.1_git20231014-r0)
(3/6) Installing gc (8.2.4-r1)
(4/6) Installing ncurses-terminfo-base (6.4_p20231125-r0)
(5/6) Installing libncursesw (6.4_p20231125-r0)
(6/6) Installing w3m (<IP_ADDRESS>30718-r1)
Executing busybox-1.36.1-r15.trigger
OK: 12 MiB in 21 packages
Error: unknown flag: --time
Error: Process completed with exit code 125.

This error was easily resolved by upgrading to the ubuntu-24.04 runner (refer to the GitHub Actions CI log here, and the associated commit here, and the context for that change here). To conclude, this confirms for me that the script, [even] after the changes made in this PR, can work as expected on GitHub Actions (as long as we use the ubuntu-24.04 runner).

Didn't know that --time 0 is so new. We could make it optional depending on the detected Docker version.


Prefix authority with double slash if present.

The PSR-7 docblock of the UriInterface::__toString method, states that:

If an authority is present, it MUST be prefixed by "//".

This PR tests and fixes that behavior.

Every bug fixed is a sort of BC break as you cannot know if there are things that relay on the old behavior, but for me it should be fixed in order to be PSR7 compliant.

@ajgarlag why you have closed this issue?

@sharifzadesina It was an error,
is open again!

Merged to develop for release with 1.4.0. Thanks, @ajgarlag


[9.0.2] getThumbnailURL() method ignores alternate storage location

Fix to issue #10342 probably also #9867
When using an alternate storage location, especially a remote one like Amazon S3 for instance, the getThumbnailURL() method returns the wrong URL.
The custom storage location is ignored and the website's canonical URL is added at the beginning of the remote file's URL (e.g. https://www.website-canonical-url.com/https://actual-file-remote-url/xxx/xxx/thumbnail_type_handle/file-name.jpg)
This PR fixes it.

Fix to issue https://github.com/concrete5/concrete5/issues/10342 probably also https://github.com/concrete5/concrete5/issues/9867

Yeah, fixes https://github.com/concrete5/concrete5/issues/9867, too 👍


Clarify bug bounty's status

The bug bounty is still running (it looks like it might have ended in Jan. from the current text). We may also want to suggest increases in payout values, with notice that no security issues have been found through the bounty yet.

Closing; we have updated the bug bounty language.


Sync with pcf/cfo Dec 11th

Description

How Has This Been Tested?

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/retest

/lgtm

I think it could be better to merge https://github.com/project-codeflare/codeflare-operator/pull/398 first

@sutaakar I could wait for that PR to merge. The purpose of this particular sync is to bring upstream to red-hat-data-services/CFO to get real Snyk reports (Snyk is currently only performing scans on rhds/cfo repo).
Meaning, after the sync, I might have a PR on upstream to solve the CVEs we have, then re-sync as necessary and have downstream repos free of critical/high CVEs as soon as possible. During those re-syncs, if that PR is merged I could bring it over, what do you think?

Meaning, after the sync, I might have a PR on upstream to solve the CVEs we have, then re-sync as necessary and have downstream repos free of critical/high CVEs as soon as possible. During those re-syncs, if that PR is merged I could bring it over, what do you think?

Oh, ok. I though that this is for a release purposes.
Fine for me to merge it without 398.

@sutaakar would you know why the CI is failing? Logs show null pointer exception but don't understand why

Looks like a bug in a test function checking that Route is ready for requests. Will take a look on that later.
In the meantime lets retrigger the PR check:
/retest

@sutaakar @ChristianZaccaria are you ok to have that PR merged manually?

it is fine for me, tests were passing (except for that one)

@sutaakar We've commented at the same time :) Let's wait for the retest.

Let's then wait on CI test, then let's merge it!

Test failed with same nil pointer error

OK let's merge it manually.

@ChristianZaccaria GitHub doesn't seem to be able to merge it, which is not surprising for such a rebase. We'll have to force push the branch.

@astefanutti before I make a mistake, I suppose this is what we want? :
I'm in this branch (`sync-dec11`)
git push odh-cfo main --force

https://github.com/project-codeflare/codeflare-common/pull/21 should show us reason for the test failure.


Create Hourly Emissions Facility Aggregation Endpoints

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[x] Does this have a dev link been provided in the design?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a CAMPD user I want to aggregate hourly emissions data at the facility level  So that I can conduct analysis
Acceptance Criteria
Given that a user has entered one or more valid values for all the parameters (see parameters below)
When API endpoint is executed
Then the system displays the records that match the criteria (containing data elements below)
Given that a user has entered one or more valid values for all the parameters in an upper case or lower case format
When API endpoint is executed
Then the system displays the records that match the criteria
Given that a user has entered one or more valid values for all the parameters without white space(s) before or after the delimiter
When API endpoint is executed
Then the system displays the records that match the criteria
Given that a user has entered one or more valid values for all the parameters with white space(s) before or after the delimiter
When API endpoint is executed
Then the system displays the records that match the criteria
Given that a user has made a GET request with the media type set to json or csv
When request is executed
Then the system displays the records that match the criteria in the appropriate format (json or csv)
Given that a user has entered one or more invalid values for a parameter
When the API endpoint is executed
Then the system displays the appropriate error message (see error message repo)
Development Notes
The scope of this ticket includes creating the following endpoints:
/emissions-mgmt/apportioned/hourly/facility
/emissions-mgmt/apportioned/hourly/facility/stream

will not have an exclude parameter
ensure that data labels and data descriptions adhere to the Master data dictionary

Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
:custom data download key column
Error message repo

CAMPD Master Data Dictionary
Parameters
Begin Date: Required, single value, ISO standard format, 1995-LATEST REPORTING QUARTER
End Date: Required, single value, ISO standard format, 1995-LATEST REPORTING QUARTER
Facility: Optional, multiple values allowed
State: Optional, multiple values allowed, 2 letter postal abbreviation for the 50 states plus Puerto Rico.
Unit Type: Optional, multiple values allowed
Unit Fuel Type: Optional, multiple values allowed, query will filter on both primary and secondary fuel type
Control technologies: Optional, multiple values allowed
Program: Optional, multiple values allowed,
Operating hour only: true, false
Validations
Begin Date:
Validate that one and only one value exists for the parameter
Validate that the date range is valid: 1995- latest reporting quarter
Validate that the date is in the format: YYYY-MM-DD
Error Message 1: < insert property > should not be null or undefined FV-106
Error Message: Ensure that < insert property > is in the FV-108
Error Message: Update the < insert property > to < insert valid range > FV-104
End Date:
Validate that one and only one value exists for the parameter
Validate that the date range is valid: 1995- latest reporting quarter
Validate that the date is in the format: YYYY-MM-DD
Validate End date is equal to or greater than start date
Error Message 1: < insert property > should not be null or undefined FV-106
Error Message: Ensure that is in the FV-108
Error Message: Error Message: Update the < insert property > to < insert valid range > FV-104
Error Message Enter an < insert property > that is greater than or equal to the < constraint > FV-103
State:
Validate that the value matches a valid 2 letter postal abbreviation for at least 1 of the 50 states, PR, or DC
Error Message: One or more states codes are not valid. Use the two letter postal abbreviation (use TX, not Texas). FV-101
Facility ID (ORIS Code):
Validate that the value is a number
If the value is a number, validate that the format is correct
positive integer (no negative numbers, not 0)
whole numeric value (no letters or special characters)
no more than 6 digits
Error Message: One or more < insert property > are not valid. Refer to the list of available < insert property > for valid values [placeholder for link to endpoint] FV-100
Unit Type:
Validate that the value is a valid unit type.
Error Message: One or more < insert property > are not valid. Refer to the list of available < insert property > for valid values [placeholder for link to endpoint] FV-100
Unit Fuel Type:
Validate that the value is a valid fuel type.
Error Message: One or more < insert property > are not valid. Refer to the list of available < insert property > for valid values [placeholder for link to endpoint] FV-100
Control Technologies:
Validate that the value is a valid control technology.
Error Message: One or more < insert property > are not valid. Refer to the list of available < insert property > for valid values [placeholder for link to endpoint] FV-100
Program:
Validate that the value matches a valid program code
Error Message: One or more < insert property > are not valid. Refer to the list of available < insert property > for valid values [placeholder for link to endpoint] FV-100
Exclude:

Validate that the provided parameter can be removed

Error Message: One or more excluded values cannot be removed
Example. user enters stateCode into the exclude parameter. StateCode is a key value and cannot be removed.

Validate that the provided parameter is valid

Error Message: : One or more excluded values are not valid.
Example. user enters "sjsj" into the exclude parameter.  "sjsj" is not a valid value
Data Elements:
Facility Aggregation
When the system identifies at least one record that matches the criteria, then for each record the system will display ordered by: facilityId > date > hour
stateCode
facilityName
facilityId
date
hour
grossLoad
steamLoad
so2Mass
co2Mass
noxMass
heatInput
Testing
Test Case: TC653
Tester: @JanellC
Definition of Done

[x] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[x] Has the code been unit tested and peer reviewed?
[x] Have the functional tests been executed?
[x] Have all defects been reviewed and dispositioned, resolved, or deferred?
[x] Is the user story ready to be deployed to the test/staging environments?

TC653 passed @tlageman-epa
https://api.epa.gov/easey/dev/emissions-mgmt/apportioned/hourly/facility/?api_key=T8r7OW4f12XtWKLY7CPgKmKu1WoP3TVG0x4eqmlB&beginDate=2020-01-01&endDate=2020-01-01&page=1&perPage=25


Update ipfilter module

I have a couple of use-cases (::1 in the ipfilter "ip" line, multiple ipfilter items in a config) that don't work in current Caddy (0.94 fresh off website), but seem to work if I build Caddy from source with the latest ipfilter.
I suppose I'm not sure how the module system works on the downloader - is there a manual process outside of source the Caddy team has to complete, or would a pull-request I can make to the website repo (the automated build system is probably beyond me at this point) to resolve this?

The download page is manually updated by me for every plugin. :-1: I usually bundle these together when I do a Caddy deploy but that takes half a day currently. So I'm focusing my efforts on finishing the new website which is entirely automated, everything from builds to Caddy deploys to plugin updates directly from their developers.
Unfortunately there's no other way to do it right now. You can compile Caddy from source yourself and include the plugins you want, you just have to add a line to a .go file. See how to import the plugin on this wiki page (we'll probably have a new wiki page just for doing your own custom builds).
So, sorry for the delay. I'm just really focused on getting this new system ready. It's actually all working but the website needs an overhaul to support it and so that's what I'm doing now.

Cool, appreciate the fast response! No expectation of course, but do you have any rough ETA for the new web launch, or need any help a mediocre dev (reasonably good sysad) could pull request in?

I'm doing design work right now. :) I'll put out a call for help, probably testers in the next few weeks.

Cool, thanks again :)
--
Mike OBrien
253.217.7129
On Tue, Jan 10, 2017 at 5:14 PM, Matt Holt<EMAIL_ADDRESS>wrote:

I'm doing design work right now. :) I'll put out a call for help, probably
testers in the next few weeks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mholt/caddy/issues/1341#issuecomment-271749287, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWl90IX3DCHDQN5VjDOFrNsyTDEaNS9ks5rRCz0gaJpZM4LfsCh
.


XMR and BTC QR codes are not vertically aligned in desktop view

Why did you close the issue? Did you push a fix? @LukeSmithxyz


jsx-quotes: Value "double" must be an enum value.

Using this config:
"jsx-quotes": [1, "double"]     // Double quotes in JSX

Was this changed recently? It worked before.

Got it. Should be prefer-double.

Yep, the react/jsx-quotes rule has been deprecated for the ESLint jsx-quotes rule which does not have exactly the same options.


Microsoft Learn - Exam references/filtering

Is your feature request related to a problem? Please describe.
In Microsoft Learn, it's hard to find which learning paths and modules are related to skill required for specific exams/certifications.
Describe the solution you'd like
It would be nice if there was a way you could select/filter on a specific exam and you'd get the related learning paths.
On a side note: Even better if you could cluster them in a "favorites" or "todo" section in your profile.
Describe alternatives you've considered
Search for exam numbers, selecting a role and use other existing filtering options.
Additional context
On the exam description pages, a lot of the time there was a tip "Self-paced training" with a link to a specific MVA (Microsoft Virtual Academy) course. As MVA is being replaced with Microsoft Learn, it'd be nice if learning paths/modules could be linked in a similar way.

@Devvox93 Thank you for this feedback. We have added this to our list of customer suggestions.


Update docker-compose.mdx

Minor updates to fast deployment process due to changes in Docker Compose

Thanks @inerserp if you want to contribute more, then I think the docs (in the docs folder) could benefit from a similar update as well.


Release <IP_ADDRESS>

Changelog
- description: |
    Release <IP_ADDRESS>, featuring cardano-api <IP_ADDRESS>
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - improvement    # QoL changes e.g. refactoring
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context
We want https://github.com/IntersectMBO/cardano-api/pull/400 available in cardano-node, i.e cardano-api <IP_ADDRESS>, so that adding new Conway tests in cardano-node is easier.
Checklist

[X] Commit sequence broadly makes sense and commits have useful messages
NA New tests are added if needed and existing tests are updated. See Running tests for more details
[X] Self-reviewed the diff

CHaP PR: https://github.com/IntersectMBO/cardano-haskell-packages/pull/634


config file?

with the newest version, is there no longer a Bing_search.py config file?
In an older version I was able to set the LOAD_DELAY and SEARCH_DELAY as well as hard set the browser "--exe" to my chromium of choice!

If I had that before, I don't remember haha.
But it is a good idea, and I will look into implementing it. Now that the tool is a pypi package, configuration file will probably have to be set in the user ~ directory.
For reference, the --exe flag and other options could be set in a shell alias or windows shortcut target. But some user may prefer a static config file, which shouldn't be too hard to implement.

This feature will be implemented in the next release (1.3.0)

thanks, the delay was exactly what I needed! plus the added bonus for --exe!!!


account command

Hello, i've add a qrcode to the !account command

I corrected a  error in the readme, and I added things in the embed

I corrected a  error in the readme, and I added things in the embed


Installation Guide: Required Dependencies for Supported Operating Systems and Distributions

The installation guide currently lacks detailed information on the dependencies required for various supported operating systems and distributions. To help users set up the project smoothly, we should add a section in docs/installation.md that outlines the dependencies for each supported OS and distro (e.g., Ubuntu, Debian, macOS, etc.). This will ensure that users have a clear understanding of the prerequisites and can avoid common setup issues.

Are you running into an issue with a specific distro? I believe HELM should work on Ubuntu and MacOS; Debian hasn't been tested but I don't see why it wouldn't work.
There is a known issue that HELM doesn't work on Windows; see #2907.

Oh, sorry it's for Developer setup, so it should be docs/developer_setup.md
For Ubuntu, it's quite trivial such as build-essential and unzip.
Even though they are trivial, some open-source projects include git in their setup dependencies if a distro needs it.
This issue should be a very low priority I think.
On the other hand, do you think that the setup of cuda should be mentioned?

That's a good point. Do you have an example of documentation from another open source project that has done something similar?
Yes, CUDA setup is quite complicated, and it's worth discussing this in the documentation.

Sorry, I should have given some examples. Here are three.

Electron. There are build-essential curl for Ubuntu
Android. There are build-essential zip unzip curl for Ubuntu
OpenWrt.  There are build-essential git unzip wget for Debian/Ubuntu/Mint

Thanks for your time!

Thanks @shakatoday! I will take a look.


luaPackages.plenary-nvim: 2022-10 -> 2023-01

a new luarocks release system has been merged into plenary.nvim to have tagged release. The update is also needed for https://github.com/MrcJkb/haskell-tools.nvim/pull/93
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/209765

3 packages marked as broken and skipped:
lua54Packages.plenary-nvim lua54Packages.rest-nvim lua54Packages.sqlite

13 packages built:
lua51Packages.plenary-nvim lua51Packages.rest-nvim lua51Packages.sqlite luaPackages.plenary-nvim luaPackages.rest-nvim luaPackages.sqlite lua53Packages.plenary-nvim lua53Packages.rest-nvim lua53Packages.sqlite luajitPackages.plenary-nvim luajitPackages.rest-nvim luajitPackages.sqlite vimPlugins.rest-nvim


Chapter titles do not display in TOC

https://github.com/oduwsdl/wsdlthesis/blob/6e814c6b4a2ed50c6c7a4174937f6e39f1d28720/wsdlthesis.cls#L292-L296
The chapter titles do not display in the table of contents unless lines 292-294 and 296 are removed. I do not have any appendices yet, so I am unsure as to the side effects of removing those lines.

This may be related to #29.

That is possibly very much related to the issue seen in #29 as you already pointed out. However, that conditional is in place because ODU thesis style guide has an inconsistent indentation between chapters and appendices.

I cloned the latest repo and determined that I had old versions of wsdlthesis.cls and custommacros.sty with corrections for 4811ccd (mentioned in #24). It looks like I never got the fix from 56323dd (mentioned in #27).
When I replace the files in my repository with the most recent (6e814c6b4a2ed50c6c7a4174937f6e39f1d28720) class and style files, all works as expected.
Sorry for wasting your time.


Set up Kanban board

We are 2 persons.
The link for the project is:
https://github.com/edahigure/space-travelers-hub/projects/1

Hi @edahigure,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights

[ ] A link to the project is provided in the Issue page :100:
[ ] Information about the number of people available :1st_place_medal:

Required Changes ♻
Check the comments under the review.

[ ] It would be better if we assigned each team member their respective cards :rocket: as seen in the screenshot below

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Status: Approved :heavy_check_mark:
Hi @Team
Your project is complete! Great job for making the changes🥇. There is nothing else to say other than... it's time to merge it 🍾🚢 :shipit: . Congratulations!  🎉💯🌟
Good Points 👍

Board information in other
info about members ok

Cheers..Happy coding!..and keep soaring higher! 💻 🍷 🚀
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have 2 more limited reviews per this project. If you think that the code review was not fair, you can request a second opinion using this form.


Remove filtering test cases by test suite ID in TestCase list API

Describe your changes :

TestSuite GET API already has a way to get test cases for the given test suite ID. Removing the redundant filter functionality on TestCase list API.

Type of change :

[x] Improvement

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

@sureshms lets hold off merging this. we may have dependencies

Lets handle this API post release for 0.12.1

@TeddyCr @ayush-shah can we incorporate this for 0.13.1?
lets make sure we follow - up in next release


Failed to reset value in retained_map on erase

There is still an issue in this PR, so don't merge yet

Now should work OK

LGTM, thanks!


feat: 🌟 add sound effects

Description
Add sound effects to improve playability.
Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[X] cargo test run with all tests passing

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] 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
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Codecov Report
Attention: Patch coverage is 10.95890% with 130 lines in your changes missing coverage. Please review.

Project coverage is 52.27%. Comparing base (a968821) to head (72ca91e).

Files
Patch %
Lines

src/main.rs
0.00%
88 Missing :warning:

src/ui/mod.rs
0.00%
22 Missing :warning:

src/ui/audio.rs
0.00%
20 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
- Coverage   56.94%   52.27%   -4.68%
==========================================
  Files           3        4       +1
  Lines         850      947      +97
==========================================
+ Hits          484      495      +11
- Misses        366      452      +86

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add test checking that we do run MIR validation

Fixes https://github.com/rust-lang/miri/issues/2840

@bors r+

:pushpin: Commit 94c8e176649793e50b97de68dbd8c83415e296fe has been approved by RalfJung
It is now in the queue for this repository.

@bors r+

:pushpin: Commit 74a9f06074cea0772cfc699ddf6474759c5ff5e8 has been approved by RalfJung
It is now in the queue for this repository.

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 4373a8edca6de924d0b1c2c4be8416405e682bf7 to master...


Remove bit-packing for rendering

Codecov Report

Merging #750 (122fca6) into main (75392f1) will increase coverage by 0.05%.
The diff coverage is 15.78%.

@@            Coverage Diff             @@
##             main     #750      +/-   ##
==========================================
+ Coverage   70.23%   70.28%   +0.05%
==========================================
  Files         112      112
  Lines        5544     5530      -14
  Branches     1108     1108
==========================================
- Hits         3894     3887       -7
+ Misses        895      888       -7
  Partials      755      755

Impacted Files
Coverage Δ

...anyo/composelife/openglrenderer/GameOfLifeShape.kt
0.00% <0.00%> (ø)

...composelife/ui/cells/OpenGLNonInteractableCells.kt
0.00% <0.00%> (ø)

...m/alexvanyo/composelife/wear/GameOfLifeRenderer.kt
46.20% <0.00%> (+1.14%)
:arrow_up:

...o/composelife/ui/cells/AGSLNonInteractableCells.kt
92.00% <100.00%> (-0.99%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Store Search Pattern in List Context

Store the search pattern in the resulting list's "context" attribute.
This enables later retrieval, which is useful if you want to re-apply the corresponding search highlighting after running :cnewer or :colder.
For example,
func! Cnewer()
  cnewer
  let pattern = getqflist({'context': 0}).context
  if !empty(pattern) | let @/ = pattern | endif
endf

I'm doing something along these lines in a plugin I use to navigate/manipulate the quickfix list.

Thank you! ✨


difficulty difficulties

Describe your bug here.
I've done this a lot now so basically I have more than one difficulty and only 1 difficulty icon will show even if I switch to the other ones. The difficulties are there its just not changing the difficulty icon.plz help
also I use the compiled version.
Command Prompt/Terminal logs (if existing)
none as of now

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

make sure you haven't misspelled anything, this should just work fine.

Describe your bug here.
I've done this a lot now so basically I have more than one difficulty and only 1 difficulty icon will show even if I switch to the other ones. The difficulties are there its just not changing the difficulty icon.plz help also I use the compiled version.
Command Prompt/Terminal logs (if existing)
none as of now

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

I see one problem but I don't think it's related to the help you want. So basically your difficulties are "Easy, Normal, Hard, Insane, Expert" the images should have to expect same wording including case. I say this cause the images are the same but the first letter in each one is lowercase.

Describe your bug here.
I've done this a lot now so basically I have more than one difficulty and only 1 difficulty icon will show even if I switch to the other ones. The difficulties are there its just not changing the difficulty icon.plz help also I use the compiled version.
Command Prompt/Terminal logs (if existing)
none as of now

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

I see one problem but I don't think it's related to the help you want. So basically your difficulties are "Easy, Normal, Hard, Insane, Expert" the images should have to exact same wording including case. I say this cause the images are the same but the first letter in each one is lowercase when in your week.json it's uppercase.

Also, the problem of the week images won't change yeah I get that too but weirdly if it's using the base coded difficulties then it doesn't do that but since you changed the images for the base difficulties the bug will occur.

Describe your bug here.
I've done this a lot now so basically I have more than one difficulty and only 1 difficulty icon will show even if I switch to the other ones. The difficulties are there its just not changing the difficulty icon.plz help also I use the compiled version.
Command Prompt/Terminal logs (if existing)
none as of now

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

I see one problem but I don't think it's related to the help you want. So basically your difficulties are "Easy, Normal, Hard, Insane, Expert" the images should have to exact same wording including case. I say this cause the images are the same but the first letter in each one is lowercase when in your week.json it's uppercase.
Also, the problem of the week images won't change yeah I get that too but weirdly if it's using the base coded difficulties then it doesn't do that but since you changed the images for the base difficulties the bug will occur.

I'm the one who has those diffs names, and you don't actually need case letter on the first.
Mine works perfectly, both when done via source or lua, doesn't matter, so I have no clue why it doesn't work for this guy.

for now ima just remove them


feat: refactor entire codebase

[x] Move all scripts inside index.html to main.js for better type checking
[x] Move main.js file to root project as the main purpose of this file is just for rendering DOM, so it won't polluting the lib directory
[x] Exports Round, Match, MatchNext and Party as actual native js classes instead of just type definition
[x] Prepare exports field in package.json file.

Based on d228e4d now we should easily noticed the difference between completed and upcoming match(es)
https://github.com/user-attachments/assets/5b4359d4-e14d-4652-8675-b45c03eabbee

Update match box layout based-on client' requests.
https://github.com/user-attachments/assets/bc81f810-4c40-4dcf-96f8-7d5c56190a92

Update UX that show only "winning paths", meaning all loser paths will be dimmed. That way the user can easily track the winner and all their origin matches and rounds.

https://github.com/user-attachments/assets/5c3948bc-8415-40cd-927d-715973f63eba

Better accessibility
https://github.com/user-attachments/assets/cc489d31-0fa8-49ff-948c-fca44ba8a176


Archive data upload must switch to rsync

Currently we are using scp to upload the data. It is not reliable for high volume. We need switch to rsync, so it will be able to continue interrupted session. rsync currently used at nightly builds and it works fine.

done


Backport 2.1: Wrong preproccessor condition fix

Fix for issue #696
Change #if defined(MBEDTLS_THREADING_PTHREAD)
to #if defined(MBEDTLS_THREADING_C)

Back ported from https://github.com/ARMmbed/mbedtls/pull/739

Reviewed, approved, tested and merged into mbedtls-2.1.


fisheye lens spectator cam

Would you mind describing the issue more clearly?

the fisheye spectator cam is broken in patch 5 can you try make a fix but it might be impossible


cannot connect to X server

i have this error when i run sudo python app.py
i added the line:    Defaults env_keep="DISPLAY XAUTHORITY" at the end of visudo but i still have this error

That seems something wrong with your pc libs. It seems not related to this project however, if you provide more information, I may be able to help u.

Close due to lack of followup and seems more like a pc setup rather than repo issue.
Please reopen it if necessary.


Permissions

@ibillett
I have re-instated write permissions for this repo (and docker-base)  to the docker team.  You are the maintainer for this team (which I thought should give you write permission but apparently not).  I can not seem to work out how to give only one person on a team write permissions to the repos controlled by that team.

Based on what you have done, I now have the ability to create new branches in the repo, but not commit directly to master because it is 'protected'.
Must be the case that only the creator can commit / merge PRs into master?
I'll do my best to keep the PRs short and sweet 😎

@ibillett protecting the master branch is a "best practice" it prevents, among other things, collaborators accidentally deleting the branch.  Create a new feature branch with your changes and push the branch to this repo and then submit a PR.

@ibillett Has your permissions issue been sorted? Let me know.  We seemed to have lost momentum and I would like get it back :)

@davidrpugh Yes I think so, managed to push changes to the example-auctions branch. I need to sit down and figure out security permissions before submitting a PR.
Apologies for the lack of comms, it has been a crazy busy couple of weeks at work.


ForexService and OptionsService tests

Added tests for ForexServiceImplTest with following coverage:

Commented out  one printStackTrace because I need it to fail in the tests, but not want to show stackTrace in tests.
Also added @Disabled annotation for the ListingStockServiceImplTest until they get fixed.

jel ovo gotovo ili ujutru?

Also added tests for OptionsServiceImpl with following coverage:

Added one getter in service, so I could test it easier. All tests are passing, but some throws exception (which is intended behavior).
Branches coverage isn't > 80% because it wasn't easy to cover all cases, but instruction coverage is > 80%, which I hope is okay.


Fix extern crate value in the readme

The example in the readme doesn't seem to be referencing the correct crate. This fixes it.

Thanks!


Sidebar menu vanish after switching screen size from mobile to desktop

Menu vanishes after resizing the window
here is a loom record of the issue
 Baat Cheet Sidebar bug

@kpbot claim

Fixed in #78 ✅


Syntax suggestion for the change handler of a select

This is just a suggestion for handling the onchange of a select element.
This is how i would do it in html:
<select onchange="onChange(this.value)">

It would make sense then that the following would also work in aurelia:
<select change.delegate="onChange(this.value)">

(in the spirit of keeping things like html)
But as i understand it you would need to use a ref attribute to make this scenario work in aurelia, because the this.value would not work to pass the value of the select as an argument to the function.
Or you could use value.bind to bind to a model.
I was just wondering if also making the above syntax work would make sense ?

there's also <select change.delegate="onChange($event)"> which has the target, etc

i did not think about that... thanks @jdanyow


adds: fzf-lua picker, phase 1

Describe what this PR does / why we need it
This adds the following pickers for fzf-lua:

Issues
Prs
Commits
Changed files
Project cards
Project columns

I picked the ones I use most often. I felt like this was a big enough change to show the pattern I'm going with but small enough to still review appropriately, while also adding some useful functionality.
Does this pull request fix one issue?

NONE (partially https://github.com/pwntester/octo.nvim/issues/206)
Describe how you did it
Sync calls
Once you look at the code it's obvious this is very directly copied from the Telescope picker implementation, with a few changes. I don't have any experience with Telescope but it seems like it can take data structures that contain functions as entries.
fzf-lua does not (as far as I can tell! I'm still learning :)). In general it takes tables of strings, which are the entries and then the previewing is broken out and based on that string entry.
To supplement fzf-lua's data format I used a pattern common to all the pickers I implemented, shown below
local formatted_issues = {}
local titles = {}

for _, issue in ipairs(issues) do
  local entry = entry_maker.gen_from_issue(issue)

  if entry ~= nil then
    formatted_issues[entry.ordinal] = entry
    table.insert(titles, entry.ordinal)
  end
end

This generates two structures: titles ({ Entry.Ordinal }) which is the table of strings that fzf uses to display the entries and formatted_issues ({ [key: Entry.Ordinal]: Entry } which is referenced for the full data structure when passing data back to Octo.nvim.
Async calls
For two pickers (issues and prs) I found that for active repositories (https://github.com/NixOS/nixpkgs 👀) the picker opened after an extremely long wait. To solve this I added a stream_cb to the gh/init.lua file, which gets input from stdout. This allows me to stream the data from the gh calls as it comes in rather than waiting for the data and loading it all at once.
The synchronous pattern above still generally holds, but instead of building out the titles table I called fzf_cb(some_title) to add the entries to fzf. Then finally when the process exits I have fzf_cb() to tell fzf that the loading is done.
Describe how to verify it

Have this branch installed as well as fzf-lua.
Test out following pickers:

Issues
Prs
Commits
Changed files
Project cards
Project columns

Special notes for reviews

Question thread is here: https://github.com/ibhagwan/fzf-lua/issues/759
TODOs

[x] use with-nth to hide ugly lines.

Great work! Thanks for the contribution. I took a look and the UI looks pretty neat.

Alright, that's the last time I'm going to be bitten by stylua issues :) Thanks for approving these runs and I think I consider Phase 1 to be done, so I won't be adding anything else besides suggestions or linting/style fixes

the UI looks pretty neat.

@milogert, if you feel like it, would love to see some screenshots of the final result of the UI including the previewer 🎉

Good idea! Here are screenshots of the six pickers I implemented here. I ended up finding a bug with the messages in the commits picker as a result of this :)
Issues

PRs

Commits

Changed Files

Projects

Project Columns

Looks great @milogert!
I have a suggestion, since you don’t like using the prompt to indicate the current picker, why don’t you add a title to the main window instead? You can supply one through winopts = { title = 'PRs', title_pos = 'center' }.

@ibhagwan that's a good suggestion. That brings up a question for @pwntester: When are the opts.*_title options filed out in the providers? I see opts.preview_title, opts.prompt_title, and opts.results_title in many of the Telescope providers but it's unclear how they are being populated up the chain

Good question, they are not set anywhere so they always default to an empty string. Do you think its worthy adding some configuration options for them?

BTW, @milogert, In your Commits screenshots, I can see that your vertical lines indicating the issue/pr comments are broken for long lines. You can use the new status column feature to fix that (see README)

Good question, they are not set anywhere so they always default to an empty string. Do you think its worthy adding some configuration options for them?

I think it would be cool to have some option for each thing, like prompt, results, and preview and those could accept a boolean or a table. The boolean would just turn that display on or off for all pickers and the table would customize the text for individual pickers and then you could turn them off by setting them to a false or something.
All that being said I would do this outside of adding in fzf-lua pickers, since that's a bigger change and when using Octo I felt like I had enough context to know what I was doing (for instance, when I had run :Octo pr list and it shows a preview window with stuff in it but has no labels I knew what I was looking at)

Thanks, I think this one is good to go. Thanks for the contribution!


Get Conversations: Method Not Allowed

Method Get Conversations returns 405 MethodNotAllowed
{"message":"The requested resource does not support http method 'GET'."}
The expected (documented) behavior is to "Gets a list of conversations the bot has participated in".

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

ID: c36ddb50-9e44-96e3-2df3-174166efa75d
Version Independent ID: 68ac4b64-6616-2216-3519-e0b33d3a53e4
Content: API reference - Bot Service - Bot Service
Content Source: articles/rest-api/bot-framework-rest-connector-api-reference.md
Service: bot-service
GitHub Login: @ivorb
Microsoft Alias: kamrani

@dgancho - Are you still working on this?

I am closing this issue due to inactivity. We are still happy to provide help if you need any, though it may be a good idea to make a new post if that's the case. Please have a look at this to find out the best places to ask for help, and consider asking questions on Stack Overflow.


Split website_sale_search_custom into 2 modules

Split website_sale_search_custom into 2 modules

IOError: File not found: website_sale_search_tags/views.xml

update another module too
IOError: File not found: website_sale_search_custom/views.xml

It's working on runbot


[Frontier] Add Another.Software Validator Token

Thank you! WIll do, don't close PR yet

Can we try to get some liquidity added before listing?

Can we try to get some liquidity added before listing?

Like create first pool and add first liquidity here?

How can we do it before listing? Is there another UIs or should we do through cosmjs?

Ah, yes, clone this repo and add to config locally, then create pool

How can we do it before listing? Is there another UIs or should we do through cosmjs?

You can also use the preview from the vercel deployment link above.

Conflict of changes for merging, will re-open in a day with created pool


fix(breaking-change): update for nodes 18 and 20

Types of changes

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

Checklist:

[X] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.

utils.test.js zip archiver tests are failing because of our fake file system (archiver doesn't like something in node >=18).


fix(aol): fix the aol downloader

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x ] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x ] Searched the bugtracker for similar pull requests
[x ] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:
I am the original author of this code and I am willing to release it under Unlicense
What is the purpose of your pull request?
Bug Fix

Description of your pull request and other information
The aol extractor is failing because the video id for the details API is not right (it likely has changed).
Here is a proposed fix. Unfortunately, to get the video url, we have to download the page and find it from within the page.
The id looks like a hash. I have tried just hashing different pieces of the url (manually) and didn't find a match. I might try to a few extra transformations, but it's likely that this hash is computed from other information. If we did find a way to transform from URL -> hash we would avoid the downloading and xpath selector step which would make this cleaner and require one less request.
I will try to clean this up a little more later but I do have some more higher level questions:

For the page download request, am I running this in the right place, or is there a util function I should be calling?

All aol tests pass without any issue. Clearly describe what are you trying to fix in the first place, provide example URLs and add tests.
Read tutorial and coding conventions.


Quality: Fixed the TypeError in autoformat

fixed #334
After rebasing I was getting another TypeError i.e. TypeError: write() argument must be str, not bytes
Thank your for contributing to Nuitka!
!! Please check that you select the develop branch (details see below link) !!
Before submitting a PR, please review the guidelines:
https://github.com/kayhayen/Nuitka/blob/master/CONTRIBUTING.md
What does this PR do?
Why was it initiated? Any relevant Issues?
PR Checklist

[x] Correct base branch selected? develop for new features and bug fixes too. If it's
part of a hotfix, it will be moved to master during it.
[x] All tests still pass. Check the developer manual about Running the Tests. There
are Travis tests that cover the most important things however, and you are
welcome to rely on those, but they might not cover enough.
[x] Ideally new features or fixed regressions ought to be covered via new tests.
[ ] Ideally new or changed features have documentation updates.

This was during a commit, right? Because I think the code is supposed to be a str, but getFileHashContent produces bytes I think. Just do the if str is not bytes: ...decode("utf8") dance there, because that is wrong. Right now getFileContent gives str, and getHashContent gives bytes, not a good idea.
That said, for non-utf8 code, we are going to have a hard time this way... and maybe, the old_code needs to be bytes, and when we add a mode to getFileContent, allowing us to say rb there, and then make sure we work on bytes all the time.

Yes, it was during a commit. I will try to submit a patch tomorrow morning.

@bksahu You are compensating in your change for the difference bytes from git, unicode from filesystem, when you write the file. But I think that "old_code" is also later compared, and again that issue will be there.
        changed = False
        if old_code != getFileContents(tmp_filename):
            my_print("Updated.")

So I am thinking, instead of converting back and forth, make sure to write the file with wb and teach getFileContents to allow passing the mode, so you can make this and above getFileContents(tmp_filename, "rb").
Always make sure to solve your data problems at the inception. So reading the files as "unicode" has no value in here, as we will have to know the encoding, and limit ourselves, so e.g. no test can have latin1 anymore. To copy, compare, we can work with bytes. Then black, isort, and friends can deal with it when reading from the file.
Do you get me there?

Yes. We want old_code to be in a single format i.e Bytes and not Unicode throughout the whole process.

@kayhayen Thank you for creating git hooks. It will save a lot of time. :relaxed:


Draft: feat: add vuepress docs site

Signed-off-by: Anik Das<EMAIL_ADDRESS>Description

adds VuePress documentation for Vocabulary Components

Issues

nothing loads now, to see the basic structure (sidebar and all), comment out this line of code
currently facing an issue where the components imported into the vuepress instance cannot resolve the @/mixins/* path from the original library path

cc: @MuluhGodson @brylie
Checklist

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

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

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

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

Developer's Certificate of Origin 1.1

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

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

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

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

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

This is looking good so far.

Thanks, I was unaware about this. Would it be okay to have a single final commit mentioning all co-authors, or should I be mentioning those in every commit?
Also I haven't changed the components (ie, DonateButton.vue) apart from adding extension to imports, so we won't be losing its commit history.
The only thing removed are the stories, which are being replaced with the new custom components with the copied code, I believe you mean those?

For now, just add the missing contributors in a subsequent commit. If any other components copy over, just mention the co-authors in the initial commit. There shouldn't be too many contributors to add, and each contributor only needs to be cited at least once for attribution.

Alright, I'll make sure to add them. Thanks for the heads up!

Update!

Added most components
Setup a deployment on Netlify
Added site title, and code to every component page
Back to top official plugin added, just in case we need it later

Things to note

VLocale component is not finished it seems, even in the lib source

Deployment
cc: @brylie @MuluhGodson @shailee-m
Please leave any feedback you might have, thanks!

It looks great!

@brylie I think I can un-draft (?) this for wider review now, there isn't much left to add apart from text content, I feel, which we can gradually add back.


Release 0.0.4

Codecov Report

Merging #41 (38ff0dc) into main (3d3f97a) will decrease coverage by 2.52%.
The diff coverage is 86.20%.

@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
- Coverage   91.07%   88.54%   -2.53%
==========================================
  Files           5        6       +1
  Lines          56       96      +40
==========================================
+ Hits           51       85      +34
- Misses          5        8       +3
- Partials        0        3       +3

Impacted Files
Coverage Δ

io/ioutil/ioutil.go
85.00% <86.20%> (ø)

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 3d3f97a...38ff0dc. Read the comment docs.


fix(@schematics/angular): remove baseUrl from universal tsconfig

The baseUrl is used for non relative module imports, and we should always use the baseUrl set in the workspace tsconfig. When having path mappings set at workspace level and setting a baseUrl in the server tsconfig, this paths will no longer work as paths mapping as relative to the baseUrl.
Fixes #13935

Targeting master as this won't apply cleanly on path


Update windows-defender-application-control.md

#5129

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@Dansimp - PR has been copy edited and is ready for final review. This is based on issue #5129
Thank you.
cc: @kenwith

Hi there! I am a WDAC feature team member and can confirm that these changes are accurate and look good to be merged.


Colors for Cancel and Ok Button and Select Chip when selected

Hey,
I was trying to implement this package into my app and I wanted to change the colors for the buttons when you open the Bottom Sheet and when you select the chips inside the Sheet. I tried looking online and within the actual code but couldn't find anything that worked. If theres a solution for this please let me know. Thank you.
My code for the MultiSelectBottomShieldField looks like the following:
Container(
  width: 335,
  decoration: BoxDecoration(
    color: Color(0xFFF0F0F0),
    border: Border.all(
      color: Color(0xFFE8E8E8),
      width: 2,
    ),
    borderRadius: BorderRadius.circular(8),
  ),
  child: Column(
    children: <Widget>[
      MultiSelectBottomSheetField<Software>(
        initialChildSize: 0.4,
        decoration: BoxDecoration(),
        listType: MultiSelectListType.CHIP,
        initialValue: _selectedSoftware,
        searchable: true,
        items: _items,
        buttonText: Text("Select Pharmacy Software...",
            style: GoogleFonts.inter(
                color: Color(0xFFBDBDBD), fontSize: 16)),
        onConfirm: (values) {
          _selectedSoftware = values;
        },
        chipDisplay: MultiSelectChipDisplay(
          items: _selectedSoftware
              .map((e) => MultiSelectItem(e, e.toString()))
              .toList(),
          chipColor: Color(0xFF5DB075),
          onTap: (value) {
            _selectedSoftware.remove(value);
            return _selectedSoftware;
          },
          textStyle: TextStyle(color: Colors.white),
        ),
      ),
    ],
  ),
),

Never mind! I missed the selectedColor and selectedItemsTextStyle fields.


Add map value source

What type of PR is this?
(REQUIRED)

documentation
feature

What this PR does / why we need it:
(REQUIRED)

In order to make the cli-altsrc more modular in terms on its provider it seems to be better if map value source can be incorporated directly into cli.
Which issue(s) this PR fixes:
(REQUIRED)

Special notes for your reviewer:
(fill-in or delete this section)

Testing
(fill-in or delete this section)

go test -run=TestMapValueSource
Release Notes
(REQUIRED)

New value source for maps has been included. This allows plugin providers for other formats(json, yaml, etc) to utilize base functionality instead of each source do custom nested lookups

Is the the "map value" or the map of value sources?
For debugging and bug reports it might be useful to have a structure and output that shows where option sets and overrides. Like:
$ CLI_PATH=/eee cli --path=/ccc --debug
urfave/cli: --path (default: none, env: /eee, cmdline: /ccc, final: /ccc)

Is it the "map value" or the map of value sources?
For debugging and bug reports it might be useful to have a structure and output that shows where option is sets, and which value is selected finally. Like:
$ CLI_PATH=/eee cli --path=/ccc --debug
urfave/cli: --path (default: none, env: /eee, cmdline: /ccc, final: /ccc)

It is a map-value source i.e a source which can lookup values based on key.


CSV Import feature request

It would be nice to add dates via csv. Our authority has a website where i can download the dates as csv file.

You can copy and paste the comma separated dates to the configuration. I do not plan to implement reading that from a CSV file directly. I am sorry, but the integration is already bloated with a number of custom functionalities that make it quite hard to maintain.


Intellisense doesn't populate for locally-declared enum branches

Observed on OS X with RustyCode 0.19.1
fn metric(s: &'static str) -> ast::Expression {
    ast::Expression::from(ast::MetricKey::from(String::from(s)))
}

fn validator_demo() {
    let cmp = metric("device.http.server.errorRatio").op(ast::Operator::Gte, ast::Expression::Constant(0.1.into()));

    let subtraction = ast::Expression::Binary(ast::BinaryOp::new(
        metric("server-errors"),
        ast::Operator::Sub,
        metric("client-errors")
    ));
}

In the code above, adding a new line inside validator_demo and typing cmp. will provide all the methods declared on ast::Expression; see screenshot

Doing the same with subtraction only provides word suggestions from elsewhere in the document:

Do you know whether this works in a different editor / IDE? I suspect the problem may come from the underlying completion tool (racer). If this is the case, you should open an issue over there.

I don't think I have any other editors that work with racer; do you have a suggestion of one I could test on OS X?

I think you can run racer from the commandline to see what it finds for a given position. Take a look at racer's documentation. If the wanted entries do not show up in the output, head over to the racer repository.


Intrinsics

Hi! Thanks for your work, it is a very interesting dataset for such an important problem!
I wonder, is it possible somehow to get the good approximation of intrinsics of the camera that this dataset was taken with?
It may be crucial for some tasks and not that hard to evaluate using the checkerboard printed pattern.
Best regards.

@ivasiljevic has uploaded an txt file describing the intrinsics.


LogBuilder - Fluent assignment of message-template after properties

Replaces #2963 and resolves #2965, which is also reported here:
https://stackoverflow.com/questions/52819710/nlog-changes-behaviour-depending-on-logger-priority

@304NotModified Please merge  #2986 first. Then Travis should work again

@304NotModified Please merge #2986 first. Then Travis should work again

done and restarted CI builds


Corrects dynatrace plugin README.md

Dynatrace pluguin proxy configuration fix
Provides correction to dynatrace plugin proxy configuration instructions. When attempting setup, the backstage proxy service for dynatrace will return a 404 error due to trying to retrieve a missing parameter in the configuration.
Current documentation

While the readme instructions advise to setup proxy configurations like this:
proxy:
  '/dynatrace':
    target: 'https://example.dynatrace.com/api/v2'
    ...

The parent key "endpoints" is missing and should be added like the following:
proxy:
  endpoints:
    '/dynatrace':
      target: 'https://example.dynatrace.com/api/v2'
      ...

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@ionSurf, any chance you still plan to see this PR get merged? Just need a changeset?
@tudi2d @Rugvip this is a correction that's needed, not sure what we can do to still get this in?

as this PR was closed I took the freedom to pick the commits & create a new one and added a changeset.


Question: Specify single select field with an ID

I can't seem to find any documentation on how to specify which select(s) I'd like this library to interact with.
I have multiple select forms on a page, but don't want bootstrap5-tags to alter all of them (which is what's happening out-of-the-box).
Any guidance on how to accomplish this would be very welcome.
NOTE: I'm using including this with a

Tags.init takes any selector you want. if you have a specific class like .tags-input you can use Tags.init('.tags-input')
or you can use new Tags(el) where el is the select element

closing this, feel free to reopen if any other question

Apologies - I got caught up with a bunch of things at work, but yes, this works well. Much appreciated!


Include built-in UI translation for regional docs

What kind of changes does this PR include?

Changes to Starlight code

Description
This PR makes it possible for regional languages, e.g. pt-BR (Brazilian Portuguese) to use the built-in UI translation for the main language (e.g pt).
I've intentionally made that this would only work for the built-in translations, so you still can create an i18n/pt-BR.json file and override the built-in translation in case you want.
I've made a fairly simple case-insensitive function looking for the expected BCP-47 tag, we could throw an error if it isn't a valid string or make some more complex type checking for it, but I went simple to not overengineer for now.
TO-DO: docs! (let's see if we're going to change the API or something before)
Testing
The feature was tested with a copy branch of https://github.com/withastro/starlight/pull/469/, with the following steps:

Add getting-started.mdx translation for pt-BR, the UI elements should be translated with the built-in UI i18n ✅
Add i18n/pt-BR.json, the UI elements should ignore the built-in UI and use the user-provided file instead ✅

size-limit report 📦

Path
Size

/index.html
13.17 KB (0%)

/_astro/*.js
16.55 KB (0%)

/_astro/*.css
8.55 KB (0%)

Would you be up for writing a couple of tests for the cases you mentioned?

Sure, will take care of making some tests!

Re: docs, I don’t know how much are needed. In general, I don’t think there’s new API here, just smoothing over use of regional specifiers.

Yeah, I feel like probably just a note on the internationalization page would suffice! I am leaving for today but will add that as part of the PR tomorrow!

Note that I haven't added tests to ensure the built-in regional variant gets precedence over the base language one, it seems we don't even have support for them yet, since the translations/index.ts implementation won't allow for dashes to match with the language.
// import statements can't have dashes
// and renaming to something like `zhTw`
// won't match with the user-provided lang.
import zh-TW from './zh-TW.json';

const { parse } = builtinI18nSchema();

export default Object.fromEntries(
	Object.entries({ cs, en, es, de, ja, pt, fr, it, nl, da, tr, ar, nb, zh }).map(([key, dict]) => [
		key,
		parse(dict),
	])
);

Note that I haven't added tests to ensure the built-in regional variant gets precedence over the base language one, it seems we don't even have support for them yet, since the translations/index.ts implementation won't allow for dashes to match with the language.

Oh we could still support it! We just couldn’t rely on the shorthand object notation:
import en from './en.json';
import zhTW from './zh-TW.json';

const { parse } = builtinI18nSchema();

export default Object.fromEntries(
	Object.entries({ en, 'zh-TW': zhTW }).map(([key, dict]) => [
		key,
		parse(dict),
	])
);

Oh we could still support it! We just couldn’t rely on the shorthand object notation:

Oh nice! Tested locally and worked perfectly, we're good to ship then! 🚀


Will you provide a UMD build?

I'm not very familiar with UMD or the advantages. If there is a strong use-case I might be open to it.

UMD build would let me load this library from a script tag. I have a client using a rails asset pipeline that does not include npm, browserify etc. Like many large projects, they choose not to make asset pipeline infrastructure changes at this point of the project.

Thanks for explaining your use-case. If we see multiple other UMD use-cases being requested we may think about supporting it.
If you really want to use this library in your context today then I suggest forking. I do not think we would accept PR for this because of the burden of maintaining it.


Gulp Concat doesn't include parent folder from blob in path

Hello,
I am having a bit of a problem getting my concatenated CSS to be moved into the correct directory. Normally, Gulp will move files from the blobbed (**) source directory to a directory of the same name in the destination directory. For some reason, when I include the concat function, it removes all traces of the blobbed parent directory, and places it directly inside the destination folder.
Is there a way to get the blobbed parent directory to move over with the concatenation? I have included a screenshot and working gulp task as an example below my 'theme-style' task.
gulp.task('theme-style', function () {
  // Function to check if file contains WordPress theme metadata
  var notThemeMetadata = function(file){
    var filenameSplit = String(file["history"]).split('/');
    var filename = filenameSplit[filenameSplit.length-1];
    if(filename != 'theme-metadata.css'){
      return true;
    }
    return false;
  }

  return gulp.src(['src/themes/**/theme-metadata.css', 'src/themes/**/style/*.scss'])
    .pipe(sass().on('error', sass.logError))
    .pipe(autoprefixer({
      browsers: ['last 2 versions']
    }))
    .pipe(gulpif(notThemeMetadata, cssnano()))
    .pipe(concat('style.css')) // This is the troublesome bit
    .pipe(gulp.dest('builds/themes/'));
});

// Example of a task putting the files where it should
gulp.task('theme-js', function(){
  return gulp.src('src/themes/**/js/**/*.js')
      .pipe(babel({
        presets: ['es2015']
      }))
      .pipe(uglify())
      .pipe(gulp.dest('builds/themes/'));
});

You have files coming from multiple theme directories being concatenated into one file. What do you expect the behavior to be? Can you make a more concise example?

I expect the behavior to be putting style.css into whatever the blobbed directory is. I want it set up so that any directory in my themes source folder gets put into the same named build folder.
In this case, I expect style.css into uw-oshkosh-divi.

@musicaljoeker You're combining files from multiple blobbed directories into one file, I don't think your code is working the way you think it is.
You want to combine each theme into one file then put that file in the root of that theme's folder? You'll have to write code that does that. All this plugin does is combine given files into one file.


add threads for handling requests

Coverage decreased (-0.03%) to 84.924% when pulling a7fbac7af6fb80dd8bb94d92e2baa8f8abf3d03e on threads-in-config into 41f4b9ad77bec5bee91152363e592afc47437594 on master.


feat: sbb-map-container intersection observer and scroll up button

Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR Closes #
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)

See Review Guidelines for more information what is checked during review process.
Changes
Changes in this pull request:

Browsers
I tested the build on the following browsers:

[x] Firefox Desktop
[x] Chrome Desktop
[x] Edge Desktop
[x] Safari Desktop
[x] Chrome Mobile
[x] Safari Mobile

Screen readers
I tested the build on the following browsers:

[ ] JAWS Firefox Desktop
[ ] JAWS Chrome Desktop
[ ] NVDA Firefox Desktop
[ ] NVDA Chrome Desktop
[ ] VoiceOver Safari Desktop
[ ] VoiceOver Chrome Desktop
[ ] VoiceOver Safari Mobile
[ ] Android Accessibility Suite Chrome Mobile

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

In order to fix the build you need to create an additional file like it was done for the MutationObserver (see mutation-observer.ts).

Moved the Intersection Observer to global/helpers. :)

Codecov Report

Merging #1702 (8dafa35) into master (ab917b0) will decrease coverage by 2.69%.
The diff coverage is 53.79%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master    #1702      +/-   ##
==========================================
- Coverage   54.85%   52.16%   -2.69%
==========================================
  Files          49      137      +88
  Lines        1659     5024    +3365
  Branches      406     1382     +976
==========================================
+ Hits          910     2621    +1711
- Misses        671     2226    +1555
- Partials       78      177      +99

Impacted Files
Coverage Δ

...ts/sbb-accordion-item/sbb-accordion-item.events.ts
100.00% <ø> (ø)

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

src/components/sbb-checkbox/sbb-checkbox.events.ts
0.00% <ø> (ø)

src/components/sbb-header/sbb-header.tsx
27.63% <ø> (ø)

src/components/sbb-icon/sbb-icon-request.ts
70.00% <ø> (ø)

src/components/sbb-icon/sbb-icon-validate.ts
45.16% <ø> (ø)

src/components/sbb-icon/sbb-icon.tsx
92.30% <ø> (ø)

src/components/sbb-image/sbb-image.helper.tsx
0.00% <ø> (-64.52%)
:arrow_down:

src/components/sbb-image/sbb-image.tsx
0.00% <ø> (-62.97%)
:arrow_down:

...mponents/sbb-journey-header/sbb-journey-header.tsx
85.71% <ø> (-1.25%)
:arrow_down:

... and 23 more

... and 114 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more