test: Upgrade the version for UT in dubbo-config-spring

What is the purpose of the change
Slightly reduce the UT time in dubbo-config-spring module
Brief changelog

Upgrade zookeeper to 3.6.0
Upgrade curator to 5.1.0

Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[x] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #8759 (865e162) into 3.0 (1b100f8) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.0    #8759      +/-   ##
============================================
- Coverage     63.70%   63.64%   -0.07%
+ Complexity      312      310       -2
============================================
  Files          1145     1145
  Lines         48188    48188
  Branches       7255     7233      -22
============================================
- Hits          30699    30668      -31
- Misses        14123    14137      +14
- Partials       3366     3383      +17

Impacted Files
Coverage Δ

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-27.59%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0.00%> (-15.39%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
43.47% <0.00%> (-15.22%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
62.79% <0.00%> (-9.31%)
:arrow_down:

.../remoting/transport/netty4/NettyClientHandler.java
77.58% <0.00%> (-6.90%)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0.00%> (-6.53%)
:arrow_down:

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0.00%> (-5.27%)
:arrow_down:

...nt/metadata/StandardMetadataServiceURLBuilder.java
86.04% <0.00%> (-4.66%)
:arrow_down:

...e/dubbo/remoting/transport/netty/NettyChannel.java
51.13% <0.00%> (-4.55%)
:arrow_down:

... and 34 more

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 1b100f8...865e162. Read the comment docs.


frostbite forest area

NOTE: If you wish to present more than one feature in this suggestion write each one in a comment
What will this feature do?
it will add a new area called the frostbite forest. in the forest there will be frozen fijoas growing on trees. the slimes in this area are called frost slimes and there favorite fruit is the faijoa. the frost slimes ability is it will create snowflakes nearby and when the touch the ground they make a spot of snow. any slime that touches the snow will get frozen for a few seconds. while frozen, slimes dont eat, move, or produce plorts
How it should integrate into the mod? (Optional)
this frost area can be on the new continent and adds a new slime and fruit
Possible names for the feature? (Optional)
the frostbite forest
Additional context
Add any other context or screenshots about the feature suggestion here.

I think it could use a bit more content and a bigger explanation for how it could integrate into the mod

I think it's a cute idea, but maybe not the direction of this mod. You are suggesting one slime area and food. Very much the basic fan slime template. The goal of this mod is to have a complete immersion into the story. There are no vanilla biomes with only one food and slime so why here?

I suggest you look into the new templates added. You can now use a Area Suggestion template to help you out when suggesting a new area. If you still want to suggest this please do so with the new template, otherwise I will not be looking into this suggestion.

ok


USD export from Unity3D not respecting hierarchical transformations (#USDU-217)

I have a few scenes in Unity that I am trying to export using the USD package. That package has various options for export (from the Menu, over the recorder, via API calls), and I have tried them all, but the resulting file (inspected with usdview) – while it has all the graphics pieces inside - doesn’t respect the hierarchical transformations that I have set up.
For more details see this link SO:
https://stackoverflow.com/questions/70246077/usd-export-from-unity3d-not-respecting-hierarchical-transformations
Happy to discuss it here, but I am cross-posting to a few places and SO seemed most central.

I think I have found the cause - a missing "Xform" on the PrimSpec for game object prims. Will probably write a post-processor to fix it if I don't get any response here.

Good catch @MikeWise2718 , thanks for reaching out. I identified the regression and we'll work on a patch.

I am now working on trying to get the animated skinning working. I think I have located a couple of other problems too, but as it doesn't actually work yet, I am not sure.


Separate electrum servers list from code and update it dynamically

Fetch peers once connected to an electrum server, and update the servers list.

Servers removed from code and put in a separate json file: 3ba131ed4bfc5a7cf579f9322c1fa899f6f7d630

Done, 3afeb7b83710e0a65dd4295bd9f63434c3bffdb4.


import the Heiser Lab data

Codecov Report

Merging #401 into master will increase coverage by 0.30%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
+ Coverage   89.26%   89.57%   +0.30%
==========================================
  Files          20       20
  Lines        1314     1314
==========================================
+ Hits         1173     1177       +4
+ Misses        141      137       -4

Flag
Coverage Δ

#unittests
89.57% <ø> (+0.30%)
:arrow_up:

Impacted Files
Coverage Δ

lineage/LineageTree.py
94.11% <0.00%> (+0.98%)
:arrow_up:

lineage/states/stateCommon.py
95.06% <0.00%> (+1.23%)
:arrow_up:

lineage/tests/test_BaumWelch.py
96.00% <0.00%> (+4.00%)
:arrow_up:

Continue to review full report at Codecov.

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

trivial PR, just uploading the data to the repository, moved the old data to a folder named old_data


remove angular 2 dates and info

@Rick-Anderson Angular 2 informations have been removed completely.
I feel, to have some info and progress about ng2 is good for our readers and users.
Even i was thinking to have some info about Aurelia framework even too.
Sooner or later we need to include these pieces.

@spboyer can you respond? Did I remove too much?

This is good for the removal of Angular2 info per the title.
Might to revisit the ng1 content here to provide value though


Question about performance

Hey, guys does anybody tested jitsi's performance? We've deployed jitsi helm chart in kubernetes (on docker-desktop). Then we did a performance test with in about 40 participants, we use one jvb. We noticed 3 things:

Some screens were black;
After adding in about more than 35 participants some screens became hangs(video, audio).
Then we are all together did page refresh to check a how fast we can join again to room, eventually we've got a pause in about 1-1.5 min.

Our resources 2 CPU, 4 RAM, Network 100 Mbps (max).
My question: what we have to fix to avoid these problems?

Hello @dzmitry-zimin!
Here's a couple of things you should consider checking:

Make sure that you have UDP transport set up and working properly (there are no timeout messages in JVB logs);
Check if WebSockets are working (using jitsi-web logs and Development Console in browser) — without those all streams would be limited to sound and lowest quality video;
Check if the clients you use have agreed on the video and audio codec. I've had some issues with Firefox where it either wouldn't receive VP9 video from other users or force everyone to downgrade to VP8. In my experience, we got best results on Jitsi Meet app.

Also, do keep in mind that Jitsi Meet uses Unicast to distribute audio and video streams among users.
  A high-quality video stream takes up to 4 megabits of bandwidth, so it won't take many users in a single room to oversaturate your 100Mbps uplink.


Fix integration test for duplicate name

Due to the drop of things collection after each test run, the duplication test needs to be within another test.

Oh okay yes I guess this makes sense -- the current implementation assumes shared state between tests which is bad. Thanks for fixing @floppyxyz !


Go modules

Adding in go modules - thought this might be nice to have

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           master    #6   +/-   ##
====================================
  Coverage      26%   26%
====================================
  Files          16    16
  Lines         314   314
====================================
  Hits           82    82
  Misses        228   228
  Partials        4     4

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 13abe96...df25da4. Read the comment docs.


pageserver: track total ephemeral layer bytes

Problem
Large quantities of ephemeral layer data can lead to excessive memory consumption (https://github.com/neondatabase/neon/issues/6939).  We currently don't have a way to know how much ephemeral layer data is present on a pageserver.
Before we can add new behaviors to proactively roll layers in response to too much ephemeral data, we must calculate that total.
Related: https://github.com/neondatabase/neon/issues/6916
Summary of changes

Create GlobalResources and GlobalResourceUnits types, where timelines carry a GlobalResourceUnits in their TimelineWriterState.
Periodically update the size in GlobalResourceUnits:

During tick()
During layer roll
During put() if the latest value has drifted more than 10MB since our last update

Expose the value of the global ephemeral layer bytes counter as a prometheus metric.
Extend the lifetime of TimelineWriterState:

Instead of dropping it in TimelineWriter::drop, let it remain.
Drop TimelineWriterState in roll_layer: this drops our guard on the global byte count to reflect the fact that we're freezing the layer.
Ensure the validity of the later in the writer state by clearing the state in the same place we freeze layers, and asserting on the write-ability of the layer in writer()
Add a 'context' parameter to get_open_layer_action so that it can skip the prev_lsn==lsn check when called in tick() -- this is needed because now tick is called with a populated state, where prev_lsn==Some(lsn) is true for an idle timeline.

Extend layer rolling test to use this metric

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Keeping TimelineWriterState alive across TimelineWriter lifetimes had a side effect:

we enforce checkpoint_interval more tightly: it means that after we finish initdb, if the initdb is larger than checkpoint_interval (e.g. in tests that use small layers), we would roll the layer between ingesting initdb, and ingesting the timeline creation WAL data.
there is a code path in flush_frozen_layer that writes out image layers instead of a delta, if it is called immediately after initdb.  It was probably a bug that we weren't hitting this code path before, but when we "fix" this, several tests start failing, because they configured a small layer size but didn't expect all these little image layers.
because this PR's purpose is to get the ephemeral layer size tracking done, not debug the image layer creation during initdb, I've worked around this with a change to get_open_layer_action that skips rolling a layer if called immediately after initdb.  We can remove that logic as/when we want to reinstate the image-layer-after-initdb behavior, at which time we'll need to fix all the tests that have wrong assumptions about layer counts.

Can't help but notice that we are basically trying to match the lifetime of the open layer until freeze with the writer state. Perhaps this can move into the open layer itself later on.

Right, I think that's the realization that's falling out of this -- there isn't really any "writer state", there's just state about the ephemeral layer, which we were tracking in a kind of weird way by building a state object for the duration of a commit() call.
Tracking the ephemeral layers sizes from inside the layer is clearly fine, but I'm not so sure about how the following changes to do background rolls with no writer will stack on that, I'm playing around with it today...

Force-pushed a big simplification of this.
I think we ultimately still want the writer state to live longer than it does today: otherwise we do an incomplete job of checking for layer roll conditions on the first write of each HeatmapWriter, but that can probably wait for a larger refactor that might involve keeping the state somewhere cleaner like in the layer rather than the  Timeline.


[MRG] Fix factor typo

This fixes what looks like an error in the factor naming convention.
Sidenote: this ANOVA implementation gave the same results when tested against R (with help from @drammock)

thanks @wronk


Implement Ternary Conditional Operator

Should be a pretty straightforward implementation of a ternary operator.
Also had a couple miscellaneous things in here that I can split out individually if you'd like:

Removed codegen.cpp from the Makefile since it's deprecated and the #error prevented compilation
Added an additional semantic check to variable declarations to have parity with assignments

Thanks for the new feature. Sorry it took so long for me to review it.


Fix bundle analyzer plugin

Fix yarn analyze command by packing our own version of
webpack-bundle-analyzer integration. It has a few differences in
configuration to the previous integration: it saves the bundle stats by
default and doesn't open an HTTP server. Check output of yarn analyze
command to identify the path to the reports.html file @tiberiuichim

@tiberiuichim good work! This is important!


Support to external webcam WSL

I'm developing opencv app and there is no way to use the external webcam.
Please add the support for the external webcam on WSL.

#412


python310Packages.pdm-pep517: 0.9.4 -> 0.10.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pdm-pep517 is: "Yet another PEP 517 backend."
meta.homepage for python310Packages.pdm-pep517 is: "https://github.com/pdm-project/pdm-pep517"
meta.changelog for python310Packages.pdm-pep517 is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.10.1 with grep in /nix/store/ys029brclinv61n6dgl8i812v60slia4-python3.10-pdm-pep517-0.10.1

found 0.10.1 in filename of file in /nix/store/ys029brclinv61n6dgl8i812v60slia4-python3.10-pdm-pep517-0.10.1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.pdm-pep517
python39Packages.pdm-pep517

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ys029brclinv61n6dgl8i812v60slia4-python3.10-pdm-pep517-0.10.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.pdm-pep517 https://github.com/r-ryantm/nixpkgs/archive/8731e9b02815727e6b93b6e1e9b8ff006d1eea56.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ys029brclinv61n6dgl8i812v60slia4-python3.10-pdm-pep517-0.10.1
ls -la /nix/store/ys029brclinv61n6dgl8i812v60slia4-python3.10-pdm-pep517-0.10.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.pdm-pep517
python39Packages.pdm-pep517

Maintainer pings
cc @cpcloud for testing.

conflicts


Handle reserved / unknown stream types

As stated by the RFC we need to handle reserved / unknown stream types.

https://tools.ietf.org/html/draft-ietf-quic-http-32#section-6.2.3


[feature request] Fix playback to Airplay devices

The current state of the atv integration in HA is that it won't stream audio to Airplay speakers.
An official ATV 4K does work (sometimes) and Homepod is hit and miss.
Maybe we can fix something to send the stream directly to the pyatv library because MA is using a raw PCM stream internally.

just letting you know that on v16, and trying to play a Tunein Radio 4 Nl stream to my HomePod (select the stream in Radio, click Play in the bottom control section), I get:
Logger: homeassistant.core
Source: components/apple_tv/media_player.py:297
First occurred: 08:18:58 (1 occurrences)
Last logged: 08:18:58

Error executing service: <ServiceCall media_player.play_media (c:01G2VEQ90PPFC8QZKSV90KZJVF): entity_id=['media_player.homepod_keuken'], extra=, media_type=music, media_id=http://<IP_ADDRESS>:8095/media_player.homepod_keuken.flac>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/core.py", line 1662, in catch_exceptions
    await coro_or_task
  File "/usr/src/homeassistant/homeassistant/core.py", line 1681, in _execute_service
    await cast(Callable[[ServiceCall], Awaitable[None]], handler.job.target)(
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 204, in handle_service
    await service.entity_service_call(
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 690, in entity_service_call
    future.result()  # pop exception if have
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 964, in async_request_call
    await coro
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 728, in _handle_entity_call
    await result
  File "/usr/src/homeassistant/homeassistant/components/apple_tv/media_player.py", line 297, in async_play_media
    await self.atv.stream.stream_file(media_id)
  File "/usr/local/lib/python3.9/site-packages/pyatv/core/facade.py", line 382, in stream_file
    await self.relay("stream_file")(file, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/pyatv/protocols/raop/__init__.py", line 341, in stream_file
    self.playback_manager.acquire()
  File "/usr/local/lib/python3.9/site-packages/pyatv/protocols/raop/__init__.py", line 141, in acquire
    raise exceptions.InvalidStateError("already streaming to device")
pyatv.exceptions.InvalidStateError: already streaming to device

Even while nothing was playing at all....

ATV streaming implementation is flaky. I think the only way would be if MA is going to send the stream to airplay devices itself.

for MA I guess that would safest. For HA, the ATV is a horror, and should be fixed nonetheless...

Agreed

Moved to discussions, raise your votes!


JS IntelliSense fails to treat exports["default"] the same as exports.default

VSCode Version: 1.35.1（1.35.1）
OS Version: Mac OS Sierra 10.12.5（16F73）

Steps to Reproduce:

Given the repo with package-consumer/ and test-package/ https://github.com/YuCJ/vscode-issue-report
cd test-package and run yarn link or npm link
cd ../package-consumer and run yarn link test-package or npm link test-package to symlink the test-package to package-consumer/node_modules
Try named import and default import when editing package-consumer/index.js. And the IntelliSense works fined with named import but fails with default import. (Expect the properties to show in IntelliSense)

Does this issue occur when all extensions are disabled?: Yes

Confirmed with<EMAIL_ADDRESS>Seems to work fine if you use exports.default = ...; instead of exports['default'] = ...

Thanks for checking, it works fine with exports.default = ... in my environment, too. But the exports['default'] = ... style  is the default output of<EMAIL_ADDRESS>(latest), so I guess this will be a common issue encountered by js developers.
I've not figured out why Babel choose to use the format as exports['default'] (and it's not consistent with their document https://babeljs.io/docs/en/babel-plugin-transform-modules-commonjs ).

I started out by trying to make a fairly small, targeted fix for this, but it’s getting kind of ugly trying to keep it from leaking outside of JS and/or outside of module.exports. I think it might even make sense to allow assignment to element access in more cases, e.g., I don’t see why expando properties on functions couldn’t happen via element access. But, that’s beyond the scope of what we’d want to add for 3.6 at this point. /cc @sandersn for thoughts.

Yeah, I think element access makes sense too. It's just not something that was there when I added the TS feature.

Ok, I'm bumping this to 3.7 as it's fairly feature-y. It's not a bug per se; we don't do special assignment on element access. Adding it is a feature, I guess.


unable to extract playlist

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Dear all, I tried to download a file ('Nawalny') and got "Unable to extract playlist", but if I open it in my Browser (Firefox 122.0.1) I can see the Video; so what's wrong?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vU --cookies-from-browser firefox --download-archive ..\..\archive.txt  -i --geo-bypass -o "%(playlist_index)02d %(title)s.%(ext)s" https://tvthek.orf.at/profile/Nawalny/13894011/Nawalny/14214196
[debug] Command-line config: ['-vU', '--cookies-from-browser', 'firefox', '--download-archive', '..\\..\\archive.txt', '-i', '--geo-bypass', '-o', '%(playlist_index)02d %(title)s.%(ext)s', 'https://tvthek.orf.at/profile/Nawalny/13894011/Nawalny/14214196']
Extracting cookies from firefox
[debug] Extracting cookies from: "C:\Users\khk59\AppData\Roaming\Mozilla\Firefox\Profiles\czhw2lsu.default-release-1623234895724\cookies.sqlite"
Extracted 453 cookies from firefox
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [f10589e34] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg N-106936-gdd99d34d67-20220519 (setts), ffprobe git-2020-06-28-4cfcfb3, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.19.1, brotli-1.1.0, certifi-2023.11.17, mutagen-1.47.0, requests-2.31.0, sqlite3-3.35.5, urllib3-2.1.0, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1798 extractors
[debug] Loading archive file '..\\..\\archive.txt'
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[orf:tvthek] Extracting URL: https://tvthek.orf.at/profile/Nawalny/13894011/Nawalny/14214196
[orf:tvthek] 14214196: Downloading webpage
ERROR: [orf:tvthek] 14214196: Unable to extract playlist; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp\extractor\common.py", line 718, in extract
  File "yt_dlp\extractor\orf.py", line 184, in _real_extract
  File "yt_dlp\extractor\common.py", line 1266, in _search_regex

Probably #2649.
If the same content is available through on.orf.at, try the nightly.

Dear dirkf,

trying the nightly build doesn't work (see below).

yt-dlp -vU --cookies-from-browser firefox --download-archive ....\archive.txt  -i --geo-bypass -o "%(playlist_index)02d %(title)s.%(ext)s" https://tvthek.orf.at/profile/Nawalny/13894011/Nawalny/14214196
[debug] Command-line config: ['-vU', '--cookies-from-browser', 'firefox', '--download-archive', '..\..\archive.txt', '-i', '--geo-bypass', '-o', '%(playlist_index)02d %(title)s.%(ext)s', 'https://tvthek.orf.at/profile/Nawalny/13894011/Nawalny/14214196']
Extracting cookies from firefox
[debug] Extracting cookies from: "C:\Users\khk59\AppData\Roaming\Mozilla\Firefox\Profiles\czhw2lsu.default-release-1623234895724\cookies.sqlite"
Extracted 460 cookies from firefox
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version @.*** from yt-dlp/yt-dlp-nightly-builds [974d44403] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg N-106936-gdd99d34d67-20220519 (setts), ffprobe git-2020-06-28-4cfcfb3, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, mutagen-1.47.0, requests-2.31.0, sqlite3-3.35.5, urllib3-2.2.0, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1833 extractors
[debug] Loading archive file '..\..\archive.txt'
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Latest version: @.*** from yt-dlp/yt-dlp-nightly-builds
yt-dlp is up to date @.*** from yt-dlp/yt-dlp-nightly-builds)
[orf:tvthek] Extracting URL: https://tvthek.orf.at/profile/Nawalny/13894011/Nawalny/14214196
[orf:tvthek] 14214196: Downloading webpage
ERROR: [orf:tvthek] 14214196: Unable to extract playlist; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp\extractor\common.py", line 718, in extract
  File "yt_dlp\extractor\orf.py", line 185, in _real_extract
  File "yt_dlp\extractor\common.py", line 1266, in _search_regex

Kind regards

Kurt (Austria)
Gesendet: Samstag, 17. Februar 2024 um 12:36 Uhr
Von: "dirkf" @.>
An: "yt-dlp/yt-dlp" @.>
Cc: "khk58" @.>, "Author" @.>
Betreff: Re: [yt-dlp/yt-dlp] unable to extract playlist (Issue #9231)

Probably #2649.
If the same content is available through on.orf.at, try the nightly.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

For nightly to work you would have to use the URL at on.orf.at, which might be https://on.orf.at/video/14214238/nawalny, though I don't know that this is the same as the content you wanted. (I looked in the "Dokus" link from the main on.orf.at pae and it was the first link, for some reason). Separately, is that a delivery of VX-flavoured caviar for a Mr Putin of Moscow?

Thanx for this info, this worked!!!

Kind Regards

Kurt

Gesendet: Sonntag, 18. Februar 2024 um 14:40 Uhr
Von: "dirkf" @.>
An: "yt-dlp/yt-dlp" @.>
Cc: "khk58" @.>, "Author" @.>
Betreff: Re: [yt-dlp/yt-dlp] unable to extract playlist (Issue #9231)

For nightly to work you would have to use the URL at on.orf.at, which might be https://on.orf.at/video/14214238/nawalny, though I don't know that this is the same as the content you wanted. (I looked in the "Dokus" link from the main on.orf.at pae and it was the first link, for some reason). Separately, is that a delivery of VX-flavoured caviar for a Mr Putin of Moscow?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


zsh: command not found: mysqladmin5

In my .zshrc plugins=(   git   mysql-macports   node   npm   python   pip   django   docker   zsh-autosuggestions   extract   z   zsh-syntax-highlighting )
I enter # mysqlstatus,
i got zsh: command not found: mysqladmin5

The mysqlstatus is an alias for mysqladmin5 -u root -p ping. Do you have mysqladmin5 installed?

how to install mysqladmin5? Can i run brew install mysqladmin5 ?

The plugin is supposed to be used with macports' mysql5.


Using System.Text.Json for C# client generation causes warnings in combination with Nullable Reference Types

The generated JsonInheritanceConverter has two NRT warnings that could be surpressed with to prevent build warnings:
#pragma warning disable 8602 // Disable "CS8602 Dereference of a possibly null reference"
#pragma warning disable 8600 // Disable "CS8600 Converting null literal or possible null value to non-nullable type"

PR #4964


Reconsider Bobby

CurseForge link
https://www.curseforge.com/minecraft/mc-mods/bobby
CurseForge Mod Distribution
Allowed
Modrinth link
https://modrinth.com/mod/bobby
Source/other link
https://github.com/Johni0702/bobby
What it does
Bobby is a Minecraft mod which allows for render distances greater than the server's view-distance setting. It accomplishes this goal by recording and storing (in .minecraft/.bobby) all chunks sent by the server which it then can load and display at a later point when the chunk is outside the server's view-distance. (Credit: NebelNidas via #46)
Why should it be in the modpack
Comparing to FarSight, Bobby is:

Open Source
On Modrinth
Customizable (has a config flie)

and provides OF parity by supporting 32+ render distances
Why shouldn't it be in the modpack
FarSight was voted as a mod better than Bobby
Categories
Replaces an existing mod, Works like OptiFine
Additional details
On the public vote made in the past, the only thing the FO community was asked to vote about is performance. Since the beta version which included for the last time Bobby, more performance optimizations have been added, via Debugify and other mods too. Also, the mod's configuration allows you to disable the chunk caching. Fabulously Optimized aims on OptiFine parity and 32+ render distance is another feature FarSight does not cover. To sum up, I see no point for Bobby not to be in the modpack. Please don't forget that I had voted for FarSight too (The link is from the Discord Server). See also #46, the original issue.

Counterpoints:

Open-source is good, but I would not consider it a significant argument here (because performance is a bigger reason)
Farsight plans to publish to Modrinth as well when Modrinth implements payments (as do hundreds of other mods)
Modrinth is currently not entirely supported by FO anyway, so it does not matter that much yet
There is a config file for Farsight, but I can't recall what options it had

It works by having the client keep the chunk data, even when the server unloads that chunk up to a max distance of 32 chunks(configurable).

This is the only option

And Farsight can in fact cover 32+ render distance as said in config file

Bobby allows you to do that directly via the video settings.

You can do an issue to farsight dev to add a config GUI.

(Well there are but not really. By default FO don't want to keep cache but i understand bobby could be included to give an option to the user to keep cache. But This can be done currently by user that want that by simply swapping the mods.)

Wouldn't it be better to toggle that a few clicks?

Well maybe yes. But alone it's, i think, a too small reason to add bobby.

(The other reason i can see is that farsight is currently not on Modrinth but this will be resolved as soon as modrinth give payouts.)

Bobby doesn't seem to care for payouts on mod distributing platforms, which is good. You see, that's the good thing with open source software in general; It's not depended on money.

OK i agree with this point but maybe the man just want to get a reward for his mod which is understandable. (Yes i'm a bit the devil's lawyer here but...)

However if bobby had an update in the future to make the performance better, i think we should reconsider it.

Since the beta version which included for the last time Bobby, more performance optimizations have been added, via Debugify and other mods too...

Here you don't seem to understand me. If the performance of bobby is boosted by an external factor then farsight has a really high probability to be boosted too.

In my opinion, OSS is a big factor. Without people looking at code, you have no idea what nasty bugs could be lurking in the shadows.
My question is, why not OSS, what is there to hide?

In my opinion, OSS is a big factor. Without people looking at code, you have no idea what nasty bugs could be lurking in the shadows.
My question is, why not OSS, what is there to hide?

Is boycotting closed-source mod worth losing performance ?

I would actually recommend https://www.curseforge.com/minecraft/mc-mods/distant-horizons instead.
Since it not only keeps chunks but also applies LOD, improving performance drastically while doing it.

I would actually recommend https://www.curseforge.com/minecraft/mc-mods/distant-horizons instead. Since it not only keeps chunks but also applies LOD, improving performance drastically while doing it.

Planned, cannot add yet: #240

I want performance tests that claim this is now comparable with Farsight.

Well I have one thing that for me makes Bobby better. It has the option to remember the chunks after you logged out or disconnected. For me this is a very important thing, as I like to log in and see all the surroundings of my base without having to do a fly by every time.

Well I have one thing that for me makes Bobby better. It has the option to remember the chunks after you logged out or disconnected. For me this is a very important thing, as I like to log in and see all the surroundings of my base without having to do a fly by every time.

I understand that some people like that, but it would not be enabled in FO anyways, because I want to keep people's privacy, abide by servers' rules and not fill up the hard drive.

Well I have one thing that for me makes Bobby better. It has the option to remember the chunks after you logged out or disconnected. For me this is a very important thing, as I like to log in and see all the surroundings of my base without having to do a fly by every time.

I don't know where's the message
(someone link it here pls)
But as the option of caching chunk would be off by default to not make MC take more and more size.
I kinda like the option in some cases but they are mostly edge cases.
However if you like this mod you can change farsight for it.
And if you are with MMC-autoupdate look in the wiki on how to do it.

Well I have one thing that for me makes Bobby better. It has the option to remember the chunks after you logged out or disconnected. For me this is a very important thing, as I like to log in and see all the surroundings of my base without having to do a fly by every time.

I understand that some people like that, but it would not be enabled in FO anyways, because I want to keep people's privacy (not keeping any server files they don't know of), abide by servers' rules (I think some don't allow world download mods... haven't found such ones though) and not fill up the hard drive.

The fact your talking about banning world download mods is just silly. You are always downloading the world so you can view it. Is there really any difference between ram and SSD? You can't use the chunk files anyway without a level.dat and other things.

Alright, I'll admit that point was indeed a bit silly. Other points are valid though.

The thing about Bobby is that you can say how long it should keep the chunks saved. If it's said something like 5 days for example, it would delete the data from the chunks you haven't being in a while, all while keeping the ones you see more often. That essentially means that the size won't be creeping up that much.
In my view, anyway, it's better to have a feature but don't want to use them, then to want a feature that isn't there.

The thing about Bobby is that you can say how long it should keep the chunks saved. If it's said something like 5 days for example, it would delete the data from the chunks you haven't being in a while, all while keeping the ones you see more often. That essentially means that the size won't be creeping up that much.
In my view, anyway, it's better to have a feature but don't want to use them, then to want a feature that isn't there.

Yes, i totally agree with that, if it doesn't decrease performance.
If at one point i find in my testing that bobby surpass farsight, i'll directly use and give the results.

If at one point i find in my testing that bobby surpass farsight, i'll directly use it for myself and give the results here.

Ok! I hope this issue will have your review soon!

I think this issue should be closed, new features can be suggested in HTC.


Fix link for embedded youtube video

This PR fixes link for an embedded youtube video (as explained on StackOverflow, links to youtube videos in iframes embedded in external sites should have pathname that contains the word embed instead of watch).
Before fix:

After fix:

Link to page in a sandbox:
http://ves-hx2-76.ebi.ac.uk:8410/info/genome/genebuild/haplotypes_patches.html

@ens-st3 Unless I am missing something, these are regular links; they aren't iframe sources. Nothing extra required for them. As far as I can tell, there was only one embedded youtube video in public-plugins codebase.


[BugFix] Fix stream cascade merge may got a wrong result

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Problem Summary(Required) ：
Now our merge is edge-triggered. If a source is_data_ready, but the actual data has not yet arrived, it will cause a result error
Signed-off-by: stdpain<EMAIL_ADDRESS>Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[ ] 2.2

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3


fix mssql metric name type

found that I had one metric with an extra s in _genstatss_, making it
inconsistent with it's siblings.

:+1:


Use UID of HelmRelease as dry run release name

Fixes #1703
This prevents dry run issues when a user has a release name that is 51+
characters long, as our previous strategy was to add '-temp' to the
given release name and Helm has a self imposed release name character
limit of 53 (51 + '-temp' = >53).

If the dry run fails how can you tell from the logs what HelmRelease was it? I assume you'll see just a UID

@stefanprodan see the change to the logger.

That logs the info not the error


[Question] Extending marketplace permission

Hi there,
I was thinking about extending marketplace permissions, I need a "buyer" permission. so not every address can buy a listing. it is possible to do that?

Hey @tarikhagustia,
There's a approveBuyerForListing function which can be used here. You can set a listing as reserved when creating it, and then approve buyers for the listing.


fix: listener callback should bind this

when disconnect from wallet app,this lib will throw  below error:
TypeError: this.resetAccount is not a function
    at EventEmitter.syncAccount (appKitClient.js:93:1)
    at EventEmitter.emit (events.js:158:1)
    at Zt.deleteSession (index.es.js:1:1)
    at async EventEmitter.<anonymous> (index.es.js:1:1)

It is because forget bind this for listener callback

Hello, how can I reproduce the issue?

Steps to reproduce

open playground web page, open chrome devtools console
slect wallet > More options > WalletConnect
select a wallet, for example MathWallet, scan and connect ,page show connect success
after connect success, click disconnect inside MathWallet app
chrome devtools console will show error

@glitch-txs Can you reproduce the issue according to my steps ?

Thank you, I'll add a direct binding in other PR, I'm also adding you as contributor on release notes 👍

closing in favor of https://github.com/WalletConnect/walletconnect-solana-adapter/pull/39


Stabilize menu/content left position with this one-line fix

Try to go to the settings page and then to some longer page ie. Book Chapter or Book Page, between page navigation.
To fix it add this on line 7 of "BookStack/resources/assets/sass/_reset.scss"
overflow-y: scroll;

@ssddanbrown

Can you provide a screenshot of what you're seeing?  I don't see the problem.  I did try adding that line and it started screwing up a lot of other areas that I don't think you intended.

I think I see the problem but I'm not really sure it's a problem that requires solving. It's basically just to force the standard vertical scrollbar to always show to prevent the width jumping about when going between pages with long and short amounts content.
Putting it on line 7 will add a scrollbar to way too many elements. Putting it only on the body (or html?) element should do the trick.
Personally I like the current browser method of the scrollbar hiding on shorter pages because it makes it obvious you can't scroll whereas a scrollbar showing up signifies scroll-ability on that page.
I'll leave this issue open as I'm open to discussion on how others feel about this one.

I think that "jumping width" when page length changes is really more annoying than the fact that you will see a scroll-bar all the time, so simple overflow-y: scroll on body would be really nice. In fact, when you Really look at the scroll-bar to determine if you can scroll? :)

I had a play about with the permanent scroll scroll-bar and it was a nicer experience so I've added this in. The only place without it is the page editor as that uses a different flexbox-based layout and already has a scrollbar in the edit box itself.
Thanks @tpetrauskas for bringing this issue up.


Error running app

Not able to access to the app.
It shows 'Oh no.
Error running app. If this keeps happening, please contact support.'
Any thoughts? I tried using different browsers but did not help.

It's happening again -- maybe the post on Hacker News is overloading the app?

Thanks! Testing, but so far locally with new env it works :/

Yeah seems to just be overwhelmed. One positive out of this, it's now available under both the old url and prettymapp.streamlit.com


Feature Request: cmedb creds show hosts IP where creds came from

First of all, great work and thank you so much for CME!
At the end of an engagement it would be very helpful to be able to see in the cmedb where the creds came from so I can find affected hosts. For example, when reporting where plaintext credentials in memory - Mimikatz, that info doesn't seem to be available in the db, like it is in Empire and Metasploit.
Thanks again!

Hey there, sorry for the late response.
So this is already implemented in CME v4, once you're in database just type creds <cred_id> and you should see a Pillaged From HostID column. To get the IP of that host all you do is type hosts <host_id> here's an example:

If you don't see that column, it ptobably means you're using CME < 4.0.
Cheers

No, I wasn't using version 4. I was using the older version available in the Kali repositories. Thanks!


[Addition]: 4 hotspots exploiting beacons

Hotspot b58 Addresses
1125W1198mETdZ6u2aheYaApGPkMGkBjFJM1bnGcNFJSrB8gc153
112moLq3CPqm7rG1fDmza8duDwWVF1ssP8sMnzLBPRwhPzM4FksD
112irKg9rRpnzafBHwT17fX3Bgd3ymJFYJB61HuRdvu46oFCMyb1
11Wkd46AuMrRZxX9dmx2ydi6eVH5oo8WAtQv7hJGViTknDhcTQW
Discord Handle
moguy1973#8805
Reason(s)
All 4 hotspots are in the same wallet.  They only witness each other and no other hotspots.  They are broadcasting massive numbers of beacons.  Exploiting some sort of beacon bug.

We have taken into account this addition request in the next iteration of the denylist. Some or all of the Hotspots listed may be added to the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Async crates improvements

Make the streams for all variants of SignalsInfo
Remove unnecessary parts of examples

Codecov Report

Merging #129 (8dfe750) into master (205d686) will decrease coverage by 1.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
- Coverage   69.10%   68.04%   -1.07%
==========================================
  Files          16       16
  Lines         764      676      -88
  Branches      121      105      -16
==========================================
- Hits          528      460      -68
+ Misses        141      135       -6
+ Partials       95       81      -14

Impacted Files
Coverage Δ

signal-hook-async-std/src/lib.rs
100.00% <ø> (+35.71%)
:arrow_up:

signal-hook-tokio/src/lib.rs
100.00% <ø> (+31.25%)
:arrow_up:

src/iterator/backend.rs
70.00% <0.00%> (-6.20%)
:arrow_down:

src/lib.rs
100.00% <0.00%> (+11.76%)
:arrow_up:


Rule fuzzing_corpus for collecting corpus files

Commit Message:
Rule fuzzing_corpus for collecting corpus files
Additional Description:
Example usage:
fuzzing_corpus(
name = "my_fuzzer_corpus",
srcs = [
"file1.txt",
"path/to/file2.txt",
],
)
Testing:
Local testing and CI testing
Docs Changes: N/A
Fixes #12
Signed-off-by: tengpeng<EMAIL_ADDRESS>modified:   fuzzing/cc_deps.bzl
modified:   fuzzing/common.bzl

I am using linux command cp to collect the corpus file. In my local test, the <name>_corpus is only available to collect the files under examples/ and the its sub-directories. I am not sure that is what we want. Even it is, I am still curious about how to collect the files outside the directory of the target.
From my experiments, absolute path like "/usr/home/path/to/file.txt", which starts with '/', is not allowed. A relative path like "../parent/dir/file.txt" is also not acceptable. I am wondering what is the right way to pass the path of a file in the other directories?

Apologize for keeping pushing new little updates.
PTAL now!

In #39 we use kwargs to forward attributes to cc_test in cc_fuzz_test. Since cc_test doesn't accept corpus but we need corpus here, we need to delete corpus entry in **kwargs after calling launcher_corpus

Added _corpus_zip rule to create the zip file. PTAL!

Tengpeng, could you add a test (or example corpus for an example fuzz test) for this? I think after building a cc_fuzz_target, you should be able to expect a corpus in bazel's output directory.

Of course! Do you mean adding an target into examples/ with corpus argument not being empty, and then making a CI test for building this target?

Of course! Do you mean adding an target into examples/ with corpus argument not being empty, and then making a CI test for building that target?

Yep, either with a new target or with an existing target. (It could also be part of #44, but I think an example test or usage of this feature would be needed. For eg, specifying separate individual files, specifying all files in a subdirectory, or both)


[TrackingItem] Redirection templates shown while posh tools installed.

Version 14.0.22728.1 D14REL
EricJ mentioned that it should be fixed in latter builds of VS, put a tracking item here that we need verify it.

Fixed


Change or remove tenant

How to remove or change tenant host name

@ayman-elmalah What you can do is simply delete the Hostname or edit it?
The default models used are just Eloquent Models so you can use basic Eloquent queries on them.
You can find the models by looking at your config:
config/tenancy.php

I know, but i just asked for package function to do that

Well, we don't really need to provide that as Eloquent simply provides that?
What you can do though is use the HostnameRepository in order to update your hostnames.

You can load the hostname model you want to change and then update the fqdn field. Once you changed that property, use the HostnameRepository::update method to trigger necessary cache flushes etc.
There's nothing else to it.

So when i update the hostname, how can i tell server to add it at hosts to make the host watch the domain ?

@ayman-elmalah What exactly do you mean?


arc/flat COUNTBINS drill down broken

Testing http://desi-1.kpno.noao.edu:9090 using code from desispec #702 (now tag 0.25.0) and qlf #253, mostly looking at Observing History window and drill-downs.
arc/flat exp 3571/3672 Step 2 COUNTBINS drill down "Missing RA and DEC"
(which is true because RA/DEC don't apply for arc/flat calibration exposures).
This does work for science exposures.  These plots could be done in x,y focal plane coordinates instead of RA,dec sky coordinates.

@sbailey please verify if this issue is fixed on http://desi-1.kpno.noao.edu:9090

It no longer shows an error (good) and includes the fiber status vs. fiber number (also good).  It would be even more helpful to also include the status vs. focal plane location like the science exposures, but plotted in x,y instead of ra,dec.  Some problems could be localized on the focal plane (thus the xy plot) and others could be localize on the CCD (thus the fiber number plot).  Feel free to close this and open a new ticket if you prefer, since the problem is no longer a bug where something is broken, but rather a new feature request now.

@sbailey no problem we will discuss this with the team and add it to the backlog.
@felipeaoli any thoughts before we close this?

Thanks for the review @sbailey, I will include the xy plot.
@felipelm If you agree, I  would to like to work in the suggested plot using a new branch.

@felipeaoli ok let's do that just keep a reference to https://github.com/desihub/qlf/issues/266#issuecomment-434527296 on your PR please.


update codeowners file

Thanks for contributing!
Notice:

[ ] unite test or E2E test
[ ] do not forget essential code comment and log
[ ] document for the PR
[ ] release note label
"release/none"
"release/bug"
"release/feature"
[ ] read about  Contribution notice: https://spidernet-io.github.io/spiderpool/latest/develop/contributing/

What issue(s) does this PR fix:

Fixes #
Special notes for your reviewer:

There is one commit incorrectly signed off. This means that the author of this commit failed to include a Signed-off-by line in the commit message.


Camera mode does not work - blueprint

I just created an automation using the newly fixed blueprint - and configured it for camera mode.
Added an motion trigger and the automation is triggered as expected.
But - the automation ends every time after checking if Frigate-mode is used - see the picture.

I'm not an expert in this, so hopefully someone is able to tell me, what I have done wrong. I've checked the settings multiple times and everything looks ok...
/Lars.

Thanks for reporting! The issue should now be fixed. Please reimport the blueprint and try again. If the issue persists feel free to reopen this issue!


Uncaught Error: [vuex] must call Vue.use(Vuex) before creating a store instance.

I'm getting the error:
Uncaught Error: [vuex] must call Vue.use(Vuex) before creating a store instance.
Here's my thoughts:
Perhaps direct-vuex has its own VueX installation?
Here's what fixes my problem:
myproject/src/main.ts
import Vue from 'vue'

const { store, rootActionContext, moduleActionContext } = createDirectStore(
  (vuex: any) => Vue.use(vuex),
  {
      ...
  }
)

direct-vuex.esm.js
import Vuex from 'vuex'

function createDirectStore(callback, options) {
    callback(Vuex)

Here's my code:
myproject/src/main.ts
import Vue from 'vue'
import { createDirectStore } from 'direct-vuex'
import vuex from 'vuex'
import tabs from '@/store/modules/tabs'
import favourites from '@/store/modules/favourites'
import VuexPersistence from 'vuex-persist'

Vue.use(vuex)

const vuexLocal = new VuexPersistence<any>({
  storage: window.localStorage,
  reducer: state => ({ favourites: state.favourites }),
})

const { store, rootActionContext, moduleActionContext } = createDirectStore({
  modules: {
    tabs,
    favourites,
  },
  plugins: [vuexLocal.plugin],
} as any)

I tried yours and see no problem.  Could you share the minimum reproducible one?


Use typescript's --checkJS to validate preact doc comments

Community doesn't have to maintain a separate .d.ts, this could be generated from code.
Preact already uses jsdoc, and --checkJS can actually validate the typings specified in jsdoc.
Anyone making changes to preactjs will get excellent code completion and intellisense if they are using a typescript plugin in vscode, sublime, atom, vim e.t.c

Wow, had no idea it could do that. Definitely liking this idea.

This is new in the TypeScript 2.3 release.

Community doesn't have to maintain a separate .d.ts, this could be generated from code.

I'm not sure that's possible. Does TypeScript's JSDoc support allow defining types in one module and then re-using them from another module? From my limited testing this appears not to be the case. On that basis, shared interfaces would still need to be defined in the .d.ts file but could then be referenced by JSDoc comments.

Update: there's a feature in the works, apparently nearing completion:
https://github.com/Microsoft/TypeScript/issues/7546
https://github.com/Microsoft/TypeScript/pull/15911

+1 on the idea!
btw, how about moving preact code base to typescript?
it can be done gradually (like 10% -> 30% -> ...), IDEs starts to help devs with auto-suggestions & refactoring goodies, etc.

@devmacmari you can read about it here https://github.com/developit/preact/issues/639#issuecomment-302917848

We discussed moving to TS a few times internally and some of us even attempted to do that on our own. Two of us are quite the fans of TS, but we both came to the realization that it doesn't work well with our code. We employ some tricks to determine the type of a variable which are failing in TS (checked with 3.3.3333).
Here is a simple example that breaks:
function isType(arg: object | any[]) {
    // Will break here
    if (arg.pop) return "array";
    return "boolean"
}

As much as I'd love to add this (or enable checkJs) we are blocked by this.


Solve video can't be downloaded issue

For those who might find the problem that video can't be downloaded like me (similar to #25 ).
It seems like the download URL doesn't work in the original way.
To resolve this issue, try changing this line of code: https://github.com/mbuckler/youtube-bb/blob/a0749efce7818b8ecc69366e9cc51e3e3e262151/youtube_bb.py#L152 to
'https://www.youtube.com/watch?v=' + vid.yt_id

then wait for a long long time 🙂.

Still nothing in the download directory!!

I think this is the root cause (and solution): https://github.com/yt-dlp/yt-dlp/issues/6247#issuecomment-1433096554


failed to compile on windows with no-engine build openssl

What version of gRPC and what language are you using?

v1.27.0

What operating system (Linux, Windows,...) and version?

Windows

What runtime / compiler are you using (e.g. python version or version of gcc)

Visual Studio 2017

What did you do?

build and install openssl with no-engine, no-hw, no-dso
try to build gRPC with the aforementioned installed openssl

What did you expect to see?

successful build

What did you see instead?
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(594): error C3861: 'ENGINE_load_dynamic': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(595): error C3861: 'ENGINE_by_id': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(599): error C3861: 'ENGINE_by_id': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(605): error C3861: 'ENGINE_ctrl_cmd_string': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(606): error C3861: 'ENGINE_ctrl_cmd_string': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(607): error C3861: 'ENGINE_ctrl_cmd_string': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(608): error C3861: 'ENGINE_ctrl_cmd_string': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(609): error C3861: 'ENGINE_ctrl_cmd_string': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(615): error C2065: 'ENGINE_METHOD_ALL': undeclared identifier [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(615): error C3861: 'ENGINE_set_default':identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(620): error C3861: 'ENGINE_init': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(625): error C3861: 'ENGINE_load_private_key': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]
c:\dev\build-win64-vs15-static\grpc\src\grpc\src\core\tsi\ssl_transport_security.cc(637): error C3861: 'ENGINE_free': identifier not found [c:\dev\build-win64-vs15-static\grpc\src\grpc-build\grpc_cronet.vcxproj] [c:\dev\build-win64-vs15-static\third-party\grpc\grpc.vcxproj]

Anything else we should know about your project / environment?

everything is statically build and linking against the static msvc runtime

Jan, you're probably the most familiar with Windows build system issues.  Can you please take a look?  Thanks!

@markdroth probably Jan didn't get notified due to the missing @ mention

He got notified because I assigned this issue to him.
Jan, can you please respond to the user here?  Thanks!

we worked around the issue by patching ssl_transport_security.cc. Renamed OPENSSL_IS_BORINGSSL to OPENSSL_NO_ENGINE.
OPENSSL_NO_ENGINE is defined by openssl include files when it is compiled with no-engine.
You will need to check for both in your case

The question is whether we want to support openssl build with no-engine. This is a question for security folks.
The OPENSSL_IS_BORINGSSL part is closely related to https://github.com/grpc/grpc/pull/20316 by @akshayku
CC @yihuazhang @jiangtaoli2016
(Btw, I don't see why this issue would be windows-specific).

@yihuazhang / @jiangtaoli2016  can one of you follow up on this one?

It should be noted that the reason we do this is because the wiki page mentions it.

no-engines | Disables hardware support (useful on mobile devices)

https://wiki.openssl.org/index.php/Compilation_and_Installation

no bot, leave it

Sorry for the trouble with the stalebot.

Hi, I'm having a similar problem on mac, are there any plans to support openssl built with no-engine?

For the record, I have the exact same undeclared identifiers, when building OpenSSL for iOS (ios64-xcrun) and iOS-Simulator (iossimulator-xcrun), as seen below:
grpc/src/core/tsi/ssl_transport_security.cc:620:5: error: use of undeclared identifier 'ENGINE_load_dynamic'
     ENGINE_load_dynamic();
     ^
grpc/src/core/tsi/ssl_transport_security.cc:621:14: error: use of undeclared identifier 'ENGINE_by_id'
     engine = ENGINE_by_id(engine_name);
              ^
grpc/src/core/tsi/ssl_transport_security.cc:625:16: error: use of undeclared identifier 'ENGINE_by_id'
       engine = ENGINE_by_id("dynamic");
                ^
grpc/src/core/tsi/ssl_transport_security.cc:631:12: error: use of undeclared identifier 'ENGINE_ctrl_cmd_string'
       if (!ENGINE_ctrl_cmd_string(engine, "ID", engine_name, 0) ||
            ^
grpc/src/core/tsi/ssl_transport_security.cc:632:12: error: use of undeclared identifier 'ENGINE_ctrl_cmd_string'
           !ENGINE_ctrl_cmd_string(engine, "DIR_LOAD", "2", 0) ||
            ^
grpc/src/core/tsi/ssl_transport_security.cc:633:12: error: use of undeclared identifier 'ENGINE_ctrl_cmd_string'
           !ENGINE_ctrl_cmd_string(engine, "DIR_ADD", ".", 0) ||
            ^
grpc/src/core/tsi/ssl_transport_security.cc:634:12: error: use of undeclared identifier 'ENGINE_ctrl_cmd_string'
           !ENGINE_ctrl_cmd_string(engine, "LIST_ADD", "1", 0) ||
            ^
grpc/src/core/tsi/ssl_transport_security.cc:635:12: error: use of undeclared identifier 'ENGINE_ctrl_cmd_string'
           !ENGINE_ctrl_cmd_string(engine, "LOAD", NULL, 0)) {
            ^
grpc/src/core/tsi/ssl_transport_security.cc:641:37: error: use of undeclared identifier 'ENGINE_METHOD_ALL'
     if (!ENGINE_set_default(engine, ENGINE_METHOD_ALL)) {
                                     ^
grpc/src/core/tsi/ssl_transport_security.cc:646:10: error: use of undeclared identifier 'ENGINE_init'
     if (!ENGINE_init(engine)) {
          ^
grpc/src/core/tsi/ssl_transport_security.cc:651:19: error: use of undeclared identifier 'ENGINE_load_private_key'
     private_key = ENGINE_load_private_key(engine, key_id, 0, 0);
                   ^
grpc/src/core/tsi/ssl_transport_security.cc:663:26: error: use of undeclared identifier 'ENGINE_free'
   if (engine != nullptr) ENGINE_free(engine);

As suggested by @ksophocleous, I modified ssl_transport_security.cc to account for the no-engine option, with the following patch:
diff --git a/src/core/tsi/ssl_transport_security.cc b/src/core/tsi/ssl_transport_security.cc
index ad9f570..c4469bc 100644
--- a/src/core/tsi/ssl_transport_security.cc
+++ b/src/core/tsi/ssl_transport_security.cc
@@ -141,7 +141,7 @@ struct tsi_ssl_frame_protector {
 static gpr_once g_init_openssl_once = GPR_ONCE_INIT;
 static int g_ssl_ctx_ex_factory_index = -1;
 static const unsigned char kSslSessionIdContext[] = {'g', 'r', 'p', 'c'};
-#ifndef OPENSSL_IS_BORINGSSL
+#if !defined(OPENSSL_IS_BORINGSSL) && !defined(OPENSSL_NO_ENGINE)
 static const char kSslEnginePrefix[] = "engine:";
 #endif

@@ -592,7 +592,7 @@ static tsi_result ssl_ctx_use_certificate_chain(SSL_CTX* context,
   return result;
 }

-#ifndef OPENSSL_IS_BORINGSSL
+#if !defined(OPENSSL_IS_BORINGSSL) && !defined(OPENSSL_NO_ENGINE)
 static tsi_result ssl_ctx_use_engine_private_key(SSL_CTX* context,
                                                  const char* pem_key,
                                                  size_t pem_key_size) {
@@ -665,7 +665,7 @@ static tsi_result ssl_ctx_use_engine_private_key(SSL_CTX* context,
   if (engine_name != nullptr) gpr_free(engine_name);
   return result;
 }
-#endif /* OPENSSL_IS_BORINGSSL */
+#endif /* !defined(OPENSSL_IS_BORINGSSL) && !defined(OPENSSL_NO_ENGINE) */

 static tsi_result ssl_ctx_use_pem_private_key(SSL_CTX* context,
                                               const char* pem_key,
@@ -696,11 +696,11 @@ static tsi_result ssl_ctx_use_pem_private_key(SSL_CTX* context,
 static tsi_result ssl_ctx_use_private_key(SSL_CTX* context, const char* pem_key,
                                           size_t pem_key_size) {
 // BoringSSL does not have ENGINE support
-#ifndef OPENSSL_IS_BORINGSSL
+#if !defined(OPENSSL_IS_BORINGSSL) && !defined(OPENSSL_NO_ENGINE)
   if (strncmp(pem_key, kSslEnginePrefix, strlen(kSslEnginePrefix)) == 0) {
     return ssl_ctx_use_engine_private_key(context, pem_key, pem_key_size);
   } else
-#endif /* OPENSSL_IS_BORINGSSL */
+#endif /* !defined(OPENSSL_IS_BORINGSSL) && !defined(OPENSSL_NO_ENGINE) */
   {
     return ssl_ctx_use_pem_private_key(context, pem_key, pem_key_size);
   }

Note: I don't actually set the no-engine option, but I presume that the ios64-xcrun and iossimulator-xcrun targets do it in OpenSSL.
With these changes, it now builds for me :+1:. I honestly do not know about the security implications, but it seems to me that I don't have a choice: the iOS OpenSSL build just seems to come with no-engine.

@JonasVautherin Thanks for the patch. I will merge a PR to fix this issue.

great to see this fixed! I'm closing the issue


Error running tests in wasm32-unknown-emscripten environment.

Problem

Testing the wasm32-unknown-emscripten target fails due to a filename mismatch in the compilation output:
$ cross test --verbose --target wasm32-unknown-emscripten
+ "rustup" "target" "list"
+ "cargo" "fetch" "--manifest-path" "/home/adam/dev/ldcore/Cargo.toml"
+ "rustc" "--print" "sysroot"
+ "docker" "run" "--userns" "host" "--rm" "--user" "1000:1000" "-e" "CARGO_HOME=/cargo" "-e" "CARGO_TARGET_DIR=/target" "-e" "USER=adam" "-e" "XARGO_HOME=/xargo" "-v" "/home/adam/.xargo:/xargo" "-v" "/home/adam/.cargo:/cargo" "-v" "/home/adam/dev/ldcore:/project:ro" "-v" "/home/adam/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu:/rust:ro" "-v" "/home/adam/dev/ldcore/target:/target" "-w" "/project" "-it" "japaric/wasm32-unknown-emscripten:v0.1.14" "sh" "-c" "PATH=$PATH:/rust/bin \"cargo\" \"test\" \"--verbose\" \"--target\" \"wasm32-unknown-emscripten\""
   Compiling ldcore v0.1.0 (/project)
     Running `rustc --edition=2018 --crate-name ldcore src/lib.rs --color always --emit=dep-info,link -C debuginfo=2 --test -C metadata=ecd9c7299ab7d7c3 -C extra-filename=-ecd9c7299ab7d7c3 --out-dir /target/wasm32-unknown-emscripten/debug/deps --target wasm32-unknown-emscripten -C incremental=/target/wasm32-unknown-emscripten/debug/incremental -L dependency=/target/wasm32-unknown-emscripten/debug/deps -L dependency=/target/debug/deps`
    Finished dev [unoptimized + debuginfo] target(s) in 5.63s
     Running `node-wasm /target/wasm32-unknown-emscripten/debug/deps/ldcore-ecd9c7299ab7d7c3.js`

running 1 test
test tests::it_works ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running `node-wasm /target/wasm32-unknown-emscripten/debug/deps/ldcore_ecd9c7299ab7d7c3.wasm`
module.js:487
    throw err;
    ^

Error: Cannot find module '/target/wasm32-unknown-emscripten/debug/deps/ldcore_ecd9c7299ab7d7c3.wasm'
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.Module._load (module.js:437:25)
    at Function.Module.runMain (module.js:605:10)
    at startup (bootstrap_node.js:158:16)
    at bootstrap_node.js:575:3
error: test failed, to rerun pass '--lib'

I checked the output directory and confirmed the file does not exist, but one named ldcore-ecd9c7299ab7d7c3.wasm does (note the hyphen instead of the underscore):
$ ls target/wasm32-unknown-emscripten/debug/deps/
ldcore-0c4924d1705b39bd.d   ldcore-ecd9c7299ab7d7c3.wasm      libldcore-0c4924d1705b39bd.rlib
ldcore-ecd9c7299ab7d7c3.d   ldcore-ecd9c7299ab7d7c3.wasm.map  libldcore-0c4924d1705b39bd.rmeta
ldcore-ecd9c7299ab7d7c3.js  ldcore-ecd9c7299ab7d7c3.wast

I've confirmed that building the project outside of the cross environment produces the same file with a hyphen, not an underscore:
$ cargo clean && cargo build --tests --target wasm32-unknown-emscripten
   Compiling ldcore v0.1.0 (/home/adam/dev/ldcore)
    Finished dev [unoptimized + debuginfo] target(s) in 7.04s
$ ls target/wasm32-unknown-emscripten/debug/deps/
ldcore-a3bffe884c1348b1.d   ldcore-a3bffe884c1348b1.wasm      ldcore-a3bffe884c1348b1.wast
ldcore-a3bffe884c1348b1.js  ldcore-a3bffe884c1348b1.wasm.map

Steps (Linux)

Install and activate the Emscripten SDK (latest).
Install cross: cargo install cross (make sure you also have Docker installed and available)

I had to modify the container image that cross uses to run WASM so it would have a newer version of emsdk. This is needed for it to work properly, but it is not fixed upstream yet.

Create a library project from the template: cargo new --lib foo && cd foo
Build the tests using cross or cargo (either one will reproduce the error): cargo build --tests --target wasm32-unknown-emscripten
Attempt to run the the tests: cross test --target wasm32-unknown-emscripten

Possible Solution(s)

I did some digging, and to my surprise, this is probably caused by a specific case implemented just for WASM: the should_replace_hyphens field. Some more research needs to be done to figure out exactly why this was added and if it is still required by some supported configurations. There may have been some update to the Emscripten SDK that made it start emitting hyphens - in that case, I'm not entirely sure how we will detect and fix that.
Notes

Output of cargo version: cargo 1.38.0-nightly (e853aa976 2019-08-09) (having trouble updating to latest nightly, but will edit this when I've reproduced it.

Emscripten SDK version: 1.38.31 (latest as of the time of posting)

After thinking about it further, I'm skeptical that Cargo should even be trying to run the WASM file. It already runs the JS file which appears to be a wrapper around the WASM.


Manual/userguides requiring no technical knowledge to access the data

Issue:
Tried to follow the manual for installing the Model Framework but couldn't as it assumes high development skills.
Solution
The manual procedure requires low to no development skills. Manual covering both OS and Windows machines recommended setups so that it's possible to download and run models from Model Framework.
Preliminary Feedback
IR: Is this for model consumers? If so, I agree. We need to think more about our customers -- the consumers of the models/data. Don't forget the customers!
AT: Yes, this is for model consumers or experienced financial professionals that just start to build coding skills.

There is now a credmark-client package that can be installed and used to run models. See https://developer-docs.credmark.com/en/latest/api.html#api-client-for-consumers


[Proposal] Add support for Baichuan1 and Baichuan2

Hello, does this analysis tool support Baichuan1 and Baichuan2?
Hello, I want to use this tool to analyze the Baichuan1 and Baichuan2 models. I don’t know if it is supported.

TransformerLens does not currently support Baichuan. Looking at their page on HuggingFace, it appears to be pretty similar to LLaMA, which means that it should be relatively easy to add, barring any surprises. @StarrySeas1 if you would like to use this model, I can walk you through how to do it, and you can give it a shot. It should be a matter of adding an alias to llama, and then adding a configuration block to loading_from_pretrained to make sure it matches the config on hugging face https://huggingface.co/baichuan-inc/Baichuan-7B/blob/main/config.json.

Here are the modifications I made:

Added "Baichuan-13B-Chat" to the "OFFICIAL_MODEL_NAMES".
Added the configuration for Baichuan in the "convert_hf_model_config" function.

elif "Baichuan-13B" in official_model_name:
        cfg_dict = {
            "d_model": hf_config.hidden_size,
            "d_head": hf_config.hidden_size // hf_config.num_attention_heads,
            "n_heads": hf_config.num_attention_heads,
            "d_mlp": hf_config.intermediate_size,
            "n_layers": hf_config.num_hidden_layers,
            "n_ctx": 2048,  # Capped due to HF Tokenizer Constraints
            "d_vocab": hf_config.vocab_size,
            "eps": hf_config.rms_norm_eps,
            "act_fn": hf_config.hidden_act,
            "initializer_range": hf_config.initializer_range,
            "normalization_type": "RMS",
            "positional_embedding_type": "alibi",
            "post_embedding_ln": True,
            "positional_embedding_type": "alibi",
        }

Added a function "convert_baichuan_weights"

def convert_baichuan_weights(baichuan, cfg: HookedTransformerConfig):
    state_dict = {}

    state_dict["embed.W_E"] = baichuan.model.embed_tokens.weight

    assert cfg.d_mlp is not None  # keep mypy happy

    for l in range(cfg.n_layers):
        state_dict[f"blocks.{l}.ln1.w"] = baichuan.model.layers[l].input_layernorm.weight

        W = baichuan.model.layers[l].self_attn.W_pack.weight

        W_split = W.T.reshape(cfg.d_model, cfg.n_heads, 3, cfg.d_head)

        W_Q, W_K, W_V = W_split[..., 0, :], W_split[..., 1, :], W_split[..., 2, :]
        W_Q = einops.rearrange(W_Q, "m n h ->n m h", n=cfg.n_heads)
        W_K = einops.rearrange(W_K, "m n h ->n m h", n=cfg.n_heads)
        W_V = einops.rearrange(W_V, "m n h ->n m h", n=cfg.n_heads)
        state_dict[f"blocks.{l}.attn.W_Q"] = W_Q
        state_dict[f"blocks.{l}.attn.W_K"] = W_K
        state_dict[f"blocks.{l}.attn.W_V"] = W_V

        state_dict[f"blocks.{l}.attn.b_Q"] = torch.zeros(
            cfg.n_heads, cfg.d_head, dtype=cfg.dtype, device=W_Q.device
        )
        state_dict[f"blocks.{l}.attn.b_K"] = torch.zeros(
            cfg.n_heads,
            cfg.d_head,
            dtype=cfg.dtype,
            device=W_Q.device,
        )
        state_dict[f"blocks.{l}.attn.b_V"] = torch.zeros(
            cfg.n_heads,
            cfg.d_head,
            dtype=cfg.dtype,
            device=W_Q.device,
        )

        W_O = baichuan.model.layers[l].self_attn.o_proj.weight
        W_O = einops.rearrange(W_O, "m (n h)->n h m", n=cfg.n_heads)
        state_dict[f"blocks.{l}.attn.W_O"] = W_O
        state_dict[f"blocks.{l}.attn.b_O"] = torch.zeros(
            cfg.d_model, dtype=cfg.dtype, device=W_O.device
        )

        state_dict[f"blocks.{l}.ln2.w"] = baichuan.model.layers[l].post_attention_layernorm.weight

        state_dict[f"blocks.{l}.mlp.W_in"] = baichuan.model.layers[l].mlp.up_proj.weight.T
        state_dict[f"blocks.{l}.mlp.W_gate"] = baichuan.model.layers[l].mlp.gate_proj.weight.T
        state_dict[f"blocks.{l}.mlp.b_in"] = torch.zeros(cfg.d_mlp, dtype=W_O.dtype)

        state_dict[f"blocks.{l}.mlp.W_out"] = baichuan.model.layers[l].mlp.down_proj.weight.T
        state_dict[f"blocks.{l}.mlp.b_out"] = torch.zeros(cfg.d_model, dtype=W_O.dtype)

    state_dict["ln_final.w"] = baichuan.model.norm.weight
    state_dict["unembed.W_U"] = baichuan.lm_head.weight.T
    state_dict["unembed.b_U"] = torch.zeros(cfg.d_vocab, dtype=W_O.dtype)

    return state_dict

The following method calls can be used to load without error
model = transformer_lens.HookedTransformer.from_pretrained(ckpt_path,fold_ln=False,
        center_writing_weights=False,
        center_unembed=False,device="cuda:6",trust_remote_code=True)

Question:

Are the above additions correct?
I compared the Attention Patterns of Baichuan-13B-Chat and the Baichuan-13B-Chat after SFT on the same sample, and they are basically not much different, with the values on the first few Tokens being very large. I'm not sure if this is a problem with the model training or if my addition of support for Baichuan is incorrect.

Thank you very much for doing the work here to get it to work in TransformerLens! Your implementation from a glance seems correct. However, it's hard to say if the discrepancies are a result of something being slightly off, or if it is something of no concern without playing with the code directly. Can you setup a PR with these changes in it, so that we can double check everything together?


[BEAM-8937] Add Jenkins job definitions for GroupByKey Java load test on Flink

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Retest this please

ignore this comment

cool, this triggered a RAT check, but not other checks :(

retest this please

Not sure which suites were supposed to trigger here since it only changes groovy files

Run Seed Job


Make Contributions.md!!

Hey, I would like to recommend adding contributors.md file so the name of contributors will be visible on the main repo.
Screenshots

assign me please

I am already working on it and soon I will generate PR for same

Hey @AKACHI-4. This is really good idea can you do the same with the profile pictures that will be amazing.

Okay @Rajdip019 why not

Closing this as it is already solved.


feat: support custom npm bin

close #21

@egamma  document is update, plz review again.

Looks good, thanks 🌹


What's the purpose of laplacian/gaussian_param?

First of all, thanks for the great work which point out a framework for high resolution on GAN!!
But I don't understand what's the purpose of the functions "laplacian_param" and "gaussian_param" in gp_gan.py?
Could you give me some hint?

The two functions try to generate laplacian filter and Gaussian filter in 2D Fourier space respectively.

Thanks for the answer, but I still don't know:

why do the laplacian and gaussian filter need to be resized and rolled before fft2?
why do you divide "(dct2(L[:,:, i]) + color_weight*Xdct)" with param? What's the meaning of the "divide" operation?

Please see this paper for details: "A method for enforcing integrability in shape from shading algorithms"

Okay, I'll check this paper, but could you tell me what's the meaning that you do np.roll alone horizontal and vertical in laplacian and gaussian filter?
Thanks!!


Fix exception in error logging when we get an address that fails to parse.

Also reducing this in importance to an INFO logging

👍


How to fix an initialization for PPO2

I was using PPO2 to train my agent. And I found that different runs of my training yielded very different results using the same hyperparameters: some yielded pretty good results but some were totally broken (not learning at all). So I guess it's probably due to a different initialization of weights during each run. If my guess is correct (and please let me know if others have encountered similar problems), may I know how to fix an initialization for PPO2?

This is expected from RL agents, both because of the random initialization but also because of stochastic environments. See reproducibility in docs.
You may close this issue if that answers your question.

From the "RL tips and tricks" section:
"
Reinforcement Learning differs from other machine learning methods in several ways. The data used to train the agent is collected through interactions with the environment by the agent itself (compared to supervised learning where you have a fixed dataset for instance). This dependence can lead to vicious circle: if the agent collects poor quality data (e.g., trajectories with no rewards), then it will not improve and continue to amass bad trajectories.
This factor, among others, explains that results in RL may vary from one run to another (i.e., when only the seed of the pseudo-random generator changes). For this reason, you should always do several runs to have quantitative results.
"

These advice are helpful! Thank you!
I'll close this issue once I've experimented with the suggestions.


reproducible build concern

The kernel-releases.json file changes pretty frequently. This means that a git clone of the repo requires that the user build kernels from scratch. While this might be ok for the main branch, for released tag branches and maybe optionally with a flag on main, build the HookOS artifacts by using all existing images in quay.io/tinkerbell.
Expected Behaviour

Current Behaviour

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Operating System and version (e.g. Linux, Windows, MacOS):

How are you running Tinkerbell? Using Vagrant & VirtualBox, Vagrant & Libvirt, on Packet using Terraform, or give details:

Link to your project or a code example to reproduce issue:

#233


cli command docs skip h2s

overview
cli command page headers go straight from h1s to h3s (example).  header tags should never be skipped - this is both a concern for seo and accessibility.
screenshot

For whoever takes this on, we generate these files in the CLI Docs Build workflow of this repo (pulumi/docs) using the gen-markdown subcommand of the Pulumi CLI.


Don't send default quality with Imgix auto compression

This PR changes the behaviour for the configuration imgixConfig.default.defaultParams.
Current situation
'defaultParams' => [
    'auto' => 'compress',
],

The auto=compress and q=80 parameters are sent to Imgix. Even though we didn't set the q parameter. So it is currently impossible to only use auto=compress.
Pull request
'defaultParams' => [
    'auto' => 'compress',
],

Only the auto=compress parameter is sent to Imgix. You let Imgix decide on the quality q. According to the Imgix docs, the quality will be set to 45 if auto=compress is used.
'defaultParams' => [
    'auto' => 'compress',
    'q' => '80',
],

You can still set q if you want to make the decision yourself.

So basically. This PR makes it possible to let Imgix decide on the q quality parameter if you're using auto=compress.

This has been merged. Thanks for contributing!


Vertex / Face colors

Just wondered if anyone had got vertex or face colors to work? Variations of the following code always seem to give me a white sphere:
Three.SphereGeometry spheregeometry = new Three.SphereGeometry(5.0, 10, 10);

spheregeometry.colors = new List<Three.Color>(spheregeometry.vertices.length);

// First, assign colors to vertices as desired
for (int i = 0; i < spheregeometry.vertices.length; i++)
{
   spheregeometry.colors[i] = new Three.Color().setRGB(77, 115, 153);
}

for (int i = 0; i < spheregeometry.faces.length; i++)
{
   var face = spheregeometry.faces[i];

   // face.color.setRGB(77, 115, 153);

   // Determine if face is a tri or a quad
   int numberofsides = (face is Three.Face3) ? 3 : 4;

   face.vertexColors = new List<Three.Color>(numberofsides);

   // Assign color to each vertex of current face
   for (int j = 0; j < numberofsides; j++)
   {
      var vertexindex = face.indices[j];

      // Initialize color variable
      face.vertexColors[j] = spheregeometry.colors[vertexindex];
   }
}

Three.Material material = new Three.MeshBasicMaterial(
      color: 0xffffff, shading: Three.FlatShading,
      vertexColors: Three.VertexColors);

Three.Mesh spheremesh = new Three.Mesh(spheregeometry, material);

scene.add(spheremesh);

Anything obvious I'm doing wrong?

what does spheregeometry.faces.length do?

Gives you the number of faces in the sphere geometry I'm assuming. It's just a list.

thank you - by the way, how does it different from spheregeometry.vertices.length?

So it steps through each face and determines whether it's a tri or a quad face and then steps through each vertex of the face and assigns a color to it. I think you should also be able to set a color to all the vertices in the face, which is the commented out line. However, this doesn't work either.

spheregeometry.vertices.length is the total number of vertices that make up all the faces. Each face has 3 (tri) or 4 (quad) vertices.

very nice, thank you indeed. one more question, do you know how I can select the neighboring faces of the selected face?

i.e. if I select the face[38] I also want to get neighboring face such as face[36] face[37]

I'm not sure if there is an easy way to do this. You may have to go through each face and test for adjacency of all the other faces which will be quite an expensive operation if you have a lot of faces.
Did you manage to vertex/face colors working then as per the original issue?

not yet though. I just found out the indices of the neighboring faces have a pattern. For an example, if the index of the selected mesh is odd number X, the indices of the neighboring faces would go like X-1, X+1, X+1+20. In other case, if the index of the selected mesh is even number Y, the indices of the neighboring faces would go like Y-1, Y+1, Y-1+20. I think I will start from writing a new function from here which would be called recursively.

OK - sounds good. Let me know if you manage to get the vertex colors working as I haven't had any success.

sure, I will do :)


still working ?!

is this still working ?! i followed instructions as mentioned ! even created webhook perfectly so
https://api.telegram.org/my_token/getWebhookInfo
returns data perfectly expect one thing !
"last_error_message": "Wrong response from the webhook: 404 Not Found",

Hi, 404 error is returned from webhook when it can retrieve your bot


https://github.com/MicrosoftDocs/microsoft-365-docs-pr/compare/live...master?expand=1

Docs Build status updates of commit dee34a0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-OrganizationConfig.md
:bulb:Suggestion
View (exchange-ps)
Details

exchange/exchange-ps/exchange/New-MailUser.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-OrganizationConfig.md

Line 0, Column 0: [Suggestion-disallowed-html-tag] HTML tag 'CountryOrRegion' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Renamed some Modifier enum names

Two of the modifier names were, IMO, not correct.  This is a highly pedantic pull request to rename those modifier enum names to be more accurate.  :)
Note: I am only comfortable in a couple of these languages - in particular I'm not 100% sure of the Ruby changes.  I assume that the values in the Modifier module are just arbitrary labels and not some sort of language specific codes...

Thanks a ton @EricWittmann , much appreciated 🙏


How to get message sender JID while prining a message?

templateData: {
name: d,
displayName: Candy.Util.crop(d, Candy.View.getOptions().crop.message.nickname),
message: e,
time: Candy.Util.localizedTime(g || (new Date).toGMTString())
}};

You missed where I said we have a mailing list.


add carueda/cfg Scala wrapper

Cfg is an annotation implemented in Scalameta for both configuration specification and access in a type safe manner and free of boilerplate.
(http://www.typesafe.com/contribute/cla already signed from a previous merged PR)

Thanks!


Retrieve Google attributions automatically via their API and provide an example showing how to add the Google logo

Fixes issue #15589
To avoid adding to OL's bundle size, I decided not to include the Google logo in the source code. It also allows developpers to chose the logo they prefer/need (color/white and high DPI or not).
Instead I added an example of how to add a custom OL Control in the bottom-left corner, with a simple logo added as base64.
For the attributions, they are now retrieved via their API. Using ViewHint.ANIMATING / ViewHint.INTERACTING was still resulting in many identical API calls, so I added a little memory of the last API parameters requested, to avoid requesting the same viewport informations many time.
The new example can be tested easily by entering your own Google API Key. Then you need to zoom in somewhere to see the attributions change:

On Google's documentation, section "Displaying third-party data attributions":
https://developers.google.com/maps/documentation/tile/policies#displaying-third-party-data-attributions
It says:
+++++
When you use the Map Tiles API to display a Google Maps data as a basemap and overlay third-party (non-Google) geospatial data, you may be required to display attribution from the third-party data provider. You must not overlap or obscure the Google data attribution in any way with the data attribution of the third-party data. The attribution of third-party data must clearly be disassociated from Google's data attributions. In addition, it must be clear that Google's logo and Google's data attribution are associated with the basemap and with each other.
+++++
So I added a "Google : " prefix, to avoid conflict with other source attributions.
To be able to retrieve attributions in an async way, I changed the way the setAttributions function works, adding the patch from @ahocevar in the original issue.
I also added more information in the text of the example, to explain how you can comply with Google's policies and linked back to documentation.
I tried running the linter but on Windows, it gave me thousands of errors:

So hopefully the formatting is OK

@fnicollet Thanks for your effort on this! Can you please run npm run lint -- --fix to auto-fix the linter errors?

So I added a "Google : " prefix, to avoid conflict with other source attributions.

By default attributions for all sources appear as a single line so adding a prefix might not help. Multiple attributions can be shown on separate lines by adding
.map .ol-attribution li {
  display: block;
}

to the application css.

To avoid the large base64 string in the example code you could link the logo https://developers.google.com/static/maps/documentation/images/google_on_white.png directly (in the same way as the IGN logo in https://openlayers.org/en/latest/examples/wmts-ign.html).

Following @mike-000 comments, I removed the prefix (devs can add the CSS to display lines as block in its on code, if mixing with other sources) and linked to the image URL instead.
@ahocevar ran the linter, which fixed a couple of issues (didn't the the thousands of errors when using the --fix option)

didn't the the thousands of errors when using the --fix option

It looks like your editor is changing line endings from LF to CRLF.

Great, thanks both of you for your help on this!

Thank you @fnicollet for taking time to read the terms of service, open an issue, and contribute this enhancement.  And thanks @mike-000 and @ahocevar for the time spent reviewing.


Turbolinks not working properly with react-router

I found this bug earlier when I was messing around with the dummy app. To reproduce:

cd to spec/dummy and start the app with foreman start -f Procfile.static
Navigate to localhost:3000/react_router
Click on "Router First Page"
Go back
Go forward
Go back again

At this point, the first page component will still be visible on the page, even though we're at /react_router. When going back the first time, turbolinks unloads and reloads the page. I would expect this navigation event to be handled by react-router. Is this a bug, or expected behavior?

@jtibbertsma This might be impossible to solve. We'd have to ensure that only turbolinks or react-router listens to the back button.
In terms of your example, you can create /spec/dummy-no-turbolinks so your example does not have it.
We are not using turbolinks at https://www.friendsandguests.com/ right now due to this issue.
CC: @alexfedoseev

And see #611 and #612.

I see. I don't think I'll need to create a new dummy app though, since this doesn't break any of my test cases.

I think I can shed some light on this. The way Turbolinks knows that a PopState event is created by itself is by adding a small marker in the history state. That typically looks like this:
> console.log(window.history.state)
{turbolinks: {restorationIdentifier: "9661db46-ae69-494a-9151-9023a5db0aa6"}}

The problem is react-router also uses the history state for the same purpose, and when it's creating a new location it sadly overwrites whatever state that was already in there, so after visiting a page managed by react-router, your history state will look something like this:
> console.log(window.history.state)
{key: 'unbqk4'}

What you want to do is to listen to the "popstate" events in the browser and to call Turbolinks programmatically whenever the history object doesn't contain the 'turbolinks' property. You can typically do this in your startup file. Here is how I do it:
// client/registration.js
import Turbolinks from 'turbolinks'
import history from 'history'
Turbolinks.start()

addEventListener('popstate', (o) => {
  if (!window.history.state.turbolinks) {
    Turbolinks.visit(o.pathname)
  }
})

I haven't tested this thoroughly and there may be some edge cases, but in my small test case the back button seemed to work well enough.

Closing for now.

@volkanunsal
What If i want to use react-router inside a certain view e.g. /audits
To get to this view I use Turbolinks.visit() which is fine. but inside this view i'm using react-router for routing. Now lets say i go to another view /account and from there I press back. how do I trigger Turbolinks.visit() from there back to /audits?
The problem is we are migrating slowly from Trubolinks to react-router and we want to be able to jump between the views through the back button.


feat(i18n): localize about view

add i18n in tests
refactor code
signed-off-by: Pavan Joshi<EMAIL_ADDRESS>

Thanks for your contribution however we have first fix our CI/CD due to #135 before merging any PRs

@pavanjoshi914 you can rebase your branch since we have finished migrating to flutter 2.5.

@manjotsidhu

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

112 of 112 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 74.184%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
4023

Relevant Lines:
5423

💛  - Coveralls

squashed and rebased .. kindly review @manjotsidhu


0.2 website build broken

https://github.com/microsoft/autogen/actions/runs/11306652542/job/31447488389

Resolved by #3772