Interoperability with magic-wormhole commandline clients

The web client should interoperate with command line Go client as well as with python client via the websocket relay server.
A few possibilities:

Go: cmdline client to cmdline client, both talking via relay server TCP port.
Go: cmdline client to cmdline client, both talking via relay server websocket port.
Go: cmdline client to cmdline client, one talking to tcp port of the relay server, other talking to websocket port.
Browser client to Go cmdline client, browser connected to websocket port of relay server and cmdline client connected to tcp port.
Browser client to Go cmdline client, browser connected to websocket port of relay server and cmdline client connected to websocket port. (browser sending, cmdline client receiving and vice versa)
All the above with Python command line client instead of the Go commandline client.

As I understand it, the current difficulties are around the hints? That is, this will only ever hint at "websocket to a relay" and the {Python,Go} clients don't know that they can do "tcp to the same relay" to make the connection...?

The Go client built from https://github.com/LeastAuthority/wormhole-william/pull/20 works fine with web client.

To receive from web client:
./wormhole-william --appid myFileTransfer --no-listen --relay-url wss://mailbox.w.leastauthority.com/v1 --transit-helper wss://relay.w.leastauthority.com:443 receive

To send to web client:
/wormhole-william --appid myFileTransfer --no-listen --relay-url wss://mailbox.w.leastauthority.com/v1 --transit-helper wss://relay.w.leastauthority.com:443 send <filename>

I implemented some very minimal parsing of websocket urls etc on the python client. It does not yet support connecting to a websocket based relay, but if the relay server is listening on both tcp and websocket ports, then python client is able to connect to it via TCP while the other side connects to it via websocket, do handshakes and transfer the file.


Prog run: diagnostics dimension order consistency

We've had a heterogeneous set of prognostic runs fail with the same array dimension error when writing diagnostics to zarr. In at least one case it's because the python wrapper is not returning consistent diagnostic dimension order across ranks and timesteps, causing the zarr monitor to fail to append. This PR enforces consistent dimension order for anything written to zarr.
Significant internal changes:

Diagnostics are now required to have dimensions in z,y,x (3d) or y,x (2d) order when written.

[X] Tests added

Resolves #1767 (at least some of the cases, hopefully all)

We decided to resolve this via updates to pace-util (#1813).


Change "Ongoing Programs" tab name

Is your feature request related to a problem? Please describe.

It was suggested that we should change the “Ongoing Programs” tab name to a different one, the reason is there is a state called “Ongoing”.

Describe the solution you'd like

Come up with a different name for the tab.

Join the discussion using the link below
https://sef.discourse.group/t/come-up-with-a-different-name-for-ongoing-programs-tab/329

@anjula-sack, Can you assign me for this

Sure! go ahead


[SAC-79] Leverage KafkaStreamWriterFactory to avoid reflection while getting destination topic (Spark 2.3.x)

Modify the changeset of #80 to work along with Spark 2.3.x
Also add UTs for KafkaHarvester

Please check the difference between this patch and #80 : Micro-batch source of Kafka connector relies on DSv1, while continuous source relies on DSv2. Moreover there's some difference on DSv2 (Spark 2.4 sorted out some odd things like InternalRowXXX.)
While I added code for continuous mode, I don't expect continuous mode to trigger listener to get executed plan (it will not be finished), so spark-atlas can't work with continuous mode.

Please hold on reviewing: I found inconsistency between code in UT (which mimics some part of SparkExecutionPlanProcessor) and actual SparkExecutionPlanProcessor... Will address it.
I think it would be better to add test for SparkExecutionPlanProcessor for kafka streaming query as well, so that it can prevent any missing like this case. Will add it too.

Just updated. Please take a look. Thanks in advance!

Could you share the result in the cluster? This one needs that.

Let me spin up test cluster and validate against. Will post the result after validation.

@dongjoon-hyun
Verified with HDP <IP_ADDRESS>-190 cluster. Please take a look again. Thanks in advance!

Command:
./bin/spark-shell --jars spark-atlas-connector-assembly-0.1.0-SNAPSHOT.jar --conf spark.extraListeners=com.hortonworks.spark.atlas.SparkAtlasEventTracker --conf spark.sql.queryExecutionListeners=com.hortonworks.spark.atlas.SparkAtlasEventTracker --packages org.apache.spark:spark-sql-kafka-0-10_2.11:2.3.2

Streaming query to run:
val sourceTopics = Seq("sparksource1", "sparksource2", "sparksource3").mkString(",")
val targetTopic = "sparksink"

val df = spark.readStream.format("kafka").option("kafka.bootstrap.servers", bootstrapServers).option("subscribe", sourceTopics).option("startingOffsets", "earliest").load()

df.writeStream.format("kafka").option("kafka.bootstrap.servers", bootstrapServers).option("checkpointLocation", checkpointLocation).option("topic", targetTopic).start()

Please proceed to cherry-pick to internal repo and verify once more there.

@dongjoon-hyun Pulled the change into internal repo and verified again.

Thanks.


create new content

when I try to create one model, which implements with ReaZzon.Gutenberg.Behaviors.Gutenbergable and controller with ReaZzon.Gutenberg.Behaviors.GutenbergController
but it showed the error message
"SQLSTATE[HY000]: General error: 1364 Field 'contentable_id' doesn't have a default value (SQL: insert into reazzon_gutenberg_contents (raw_content, rendered_content, updated_at, created_at`) values (
wqerqewr

qwer

qwer
,
wqerqewr

qwer

qwer
, 2019-10-31 12:45:20, 2019-10-31 12:45:20))" on line 664 of /var/www/html/edm/vendor/laravel/framework/src/Illuminate/Database/Connection.php
`

Can you write your steps to reproduce this error?
I just did fresh install with Gutenberg from marketplace, and created plugin with Builder, and everything works fine for me.


Javascript: Fix flow argument and add support for entry.length

Flow support was totally broken '--from vim' did not work, it should either be '--from', 'vim' or like I did '--from=vim'. This also improves the error message in general and adds support for the length of underline.

@rafaelrinaldi
Does this PR work for you, too? I'd like to have a confirmation from another Flow user.

@JelteF What Flow version are you using?

I'm using the latest one (0.37.4)

@mhinz @JelteF That's weird because it works fine for me and it's actually what Facebook's Vim plugin does too. See.

Might be operating system related, that it renders to a  quoted string on
Linux. But the error format was definitely broken. I based mine on the one
from vim-flow, with some tweaks to get the full message.
On Thu, 22 Dec 2016, 18:38 Rafael Rinaldi<EMAIL_ADDRESS>wrote:

@mhinz https://github.com/mhinz @JelteF https://github.com/JelteF
That's weird because it works fine for me and it's actually what Facebook's
Vim plugin does too. See
https://github.com/flowtype/vim-flow/blob/3bd879dd7060f13a78e9238669c2e1731e098607/plugin/flow.vim#L41
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neomake/neomake/pull/884#issuecomment-268850772, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABG8JqUIoV4SX2IKnDD1GDeI6hDk-Hc4ks5rKrWQgaJpZM4LUFYg
.

FB's Vim plugin appears to use a string for the command though: https://github.com/flowtype/vim-flow/blob/3bd879dd7060f13a78e9238669c2e1731e098607/plugin/flow.vim#L50.
@rafaelrinaldi
So you are using Windows then?
Vim or Neovim?

On Windows and Vim all arguments will be joined into a string anyway: https://github.com/neomake/neomake/blob/master/autoload/neomake.vim#L227-L231.

@blueyed I'm on a Mac (macOS Sierra), running Neovim (v0.2.0-150-g33319b1b).

@rafaelrinaldi
Then 'args': ['--from vim'] shouldn't work for you, too?!
Can you show the output of a run with :let g:neomake_verbose = 3, in case it works (as in "reports errors correctly")?

@JelteF
Have you seen https://github.com/neomake/neomake/pull/590?

@blueyed But it does work for me.

Yeah, the shell is different.
Try sh -c 'flow "--from vim"', where the --from vim is passed in as a single argument.
Or `flow '--from vim``.
(It is more important in this context how Neomake handles it (the verbose log output) than your shell or how it looks in screenshots)

I could still imagine than flow parses args differently on MacOS that Linux though - e.g. by splitting single args on spaces, but that would be unusual.

@rafaelrinaldi I don't doubt that it works on whatever platform you are running. It just completely fails on mine. I'm quite sure the quoting behaviour for the args array is just be different on different platforms. With it being as simple as this:
So for linux:
['--from vim']    -> flow '--from vim'
['--from=vim']    -> flow '--from=vim'
['--from', 'vim'] -> flow '--from' 'vim'

and on mac
['--from vim']    -> flow --from vim
['--from=vim']    -> flow --from=vim
['--from', 'vim'] -> flow --from vim

So any of them should work on mac (like you said), but the first one fails on linux because it interprets it interprets it as a string literal.
@blueyed
I did not see #590 and used this from vim-flow as a base and changed it a bit so it concatenates all explanation lines (sometimes there are two explanation lines)
@rafaelrinaldi
Is your (yellow line) output actually from neomake and not from vim-flow. I've never seen an error formatted like that before. That would also explain why it "works" for you, because it's not actually neomake :wink:
PS. I'm using neovim 0.1.7 on arch linux (in xfce4-terminal).

Merged it, thanks!
We can keep on discussing and trying to figure out the behavior @rafaelrinaldi is seeing though still, of course.

@JelteF Oh sorry if it sounded that I didn't think this was an issue. I want everybody to be able to use Neomake too, regardless of platform! I just find this to be interesting. It might be something silly on the way Flow is parsing stdin args. Will take a look at that and will let you know about my findings here...
Oh and the yellow thing is from my theme configuration. For some reason one day I thought it was a great idea but seeing it now it look ugly as hell.


make it clearer to call right executable

in here: http://ritazh.github.io/devopsfun/node-app/#installing-mongodb for some reason I did not spot that it was mongod that I should run, so I got stuck awhile wondering why running mongo.exe does not work

Thanks @DrJukka! Closed via 56edb14


Bug: Professor List Page Number Text Repeats Based on Number of Times Command is Invoked

The !prof list command's "page # of ###" embedded element has a duplicating issue. Every time the command is called, an additional  "page # of ###" shows up (except only the one at the bottom updates when you change pages)
The first time the command is called:

The second time:

The third time:

AFAIK this issue only affects this specific command.

Closed in #632


General Issue: Questions about validation

Background Information
This may work better in the "Discussions" section, but I wasn't sure how to start one of those...
Hello Admiral team,
First off, fantastic work here! I really appreciate how well done this package is, and the same for the entire pharmaverse. And, to the extent any of you are involved with the R Validation Hub, I really appreciate that work as well. Which brings me to my question...
I lead the Data Science team at Metrum Research Group. One of our main projects is working on a suite of open source R packages, primarily used for Pharmacometrics. Here is a link to some of our work, if you're interested.
We are continuing to work on how to most comprehensively and most efficiently Validate these packages, for our own use and for the use of others. I know your team has also put a lot of thought and work into this and I'm interested to hear about how you are currently approaching it.
We have found much of the R Validation Hub work very valuable, and we are already using many of their principles and several of their packages. That said, the majority of their work seems focused on validating packages developed externally for their own use in their respective companies. While this work has been very helpful to us (obviously, we use many packages developed by the open source community), I am particularly looking for guidance on what validation you do for the packages you are developing internally. In particular, how is it different than the validation you do for something like the tidyverse packages?
I see several mentions of validation in your documentation, in particular this FAQ. But again, this is focused on what another company might do if they are interested in using admiral. Is this essentially the same thing you are doing internally? Statements like this one make me think that your internal processes are focused more on traditional software development and testing best practices, and that Validation portion is left to the "user". I recognize that in many cases, the "user" is also yourselves, just in a different role. This pattern makes sense and seems appealing to me, but I want to make sure that I'm not misunderstanding or assuming too much.
Thanks in advance for any help or guidance. And of course, I realize that you may not be able or willing to discuss these kinds of internal processes, especially in a public forum like this. Please let me know if there is a more appropriate place for this conversation, or if any of you are willing to connect directly. And thanks again for all of your hard work to promote open source software.
Best,
Seth Green
Definition of Done
A lively discussion on validating open source software development.

Agree Ben! There's even a validation channel there as it's always such a hot topic in the open source space. Will close this issue in anticipation of the discussion moving to Slack.
As FYI @seth127 here's some R/Pharma past sharing to help answer your question from Roche/Genentech perspective:
https://www.youtube.com/watch?v=xksxuvXVimM
https://www.youtube.com/watch?v=ZfZpypQ1jSM


fatal git error is not detected (return value ignored?) and check is reported as succesful although it failed

https://github.com/www-learn-study/saraswati.learn.study/actions/runs/6727951896/job/18286524048 from https://github.com/www-learn-study/saraswati.learn.study/pull/42 has an example of a fatal git error what was not detected (return value ignored?) and check is reported as succesful although it failed:
(...)
Checking if there are files to commit…
/usr/bin/git add --all .
/usr/bin/git checkout -b github-pages-deploy-action/ii3mg6sxx
Switched to a new branch 'github-pages-deploy-action/ii3mg6sxx'
/usr/bin/git commit -m Deploy preview for PR 42 🛫 --quiet --no-verify
Pushing changes… (attempt 1 of 3)
/usr/bin/git push --porcelain ***github.com/www-learn-study/previews.git github-pages-deploy-action/ii3mg6sxx:gh-pages
remote: Support for password authentication was removed on August 13, 2021.
remote: Please see https://docs.github.com/en/get-started/getting-started-with-git/about-remote-repositories#cloning-with-https-urls for information on currently recommended modes of authentication.
fatal: Authentication failed for 'https://github.com/www-learn-study/previews.git/'
Changes committed to the gh-pages branch… 📦
Running post deployment cleanup jobs… 🗑
/usr/bin/git checkout -B github-pages-deploy-action/ii3mg6sxx
Reset branch 'github-pages-deploy-action/ii3mg6sxx'
/usr/bin/chmod -R +rw github-pages-deploy-action-temp-deployment-folder
/usr/bin/git worktree remove github-pages-deploy-action-temp-deployment-folder --force
Completed deployment successfully! ✅
Run echo "url=https://www-learn-study.github.io/previews/pr-preview/pr-42/"  >> $GITHUB_OUTPUT
Run marocchino/sticky-pull-request-comment@v2

This looks like an upstream bug in https://github.com/JamesIves/github-pages-deploy-action - I've gone ahead and reported it: https://github.com/JamesIves/github-pages-deploy-action/issues/1472
Will leave this issue open for now and verify that the upstream fix resolves this.


[Feature] bootstrap init command

[+] generate cmd/api/main.go based on choosed framework
[+] generate Dockerfile
[+] generate .dockerignore
[+] generate Makefile

I tried to generate app with wrong flags. ex:
bccli init todo -framework laravel

This command generate will generate go.mod, go.sum, and cmd/api/main.go but the main.go will be empty and there's a message Error: templte: pattern matches no files: "tmpl/unknown_main.tmpl".
I think we should check the flags first before start creating the stucture of project

will fix

@NupalHariz fixed, you can check & test it

@NupalHariz fixed, you can check & test it

This change looks good to me. I’ve already tested it, and it works really well.
I have a question about the echo_main.tmpl: should we remove it, or should we add a case for Echo in framework.go?

itll be added later on, just keep it for now

itll be added later on, just keep it for now

Okay thanks for the information


Enum type guard

Is there a way to guard type only to enum types?
Thanks!

This is an issue tracker, not a help forum. Please ask questions on your favorite Q+A site, e.g. Stack Overflow.

I'm asking as I could not find a way - so it's more like feature request I guess.

there is not a general Enum type. Enums all are subtypes of number, but obviously this includes numbers as well.
In general, if this is a suggestion, it would be helpful to include an explanation of the scenario you are trying to address with this change, a code snippet demonstrating the original problem as well as some description of the proposed solution.


wont build

i tried to built it my self cuz the resource .jar was missing but it wont build error every time

same

you need to run as administrator


Added note about servicing/apply update failures

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.


Update to 1.2.171 + VK_KHR_synchronization2

I made some crude manipulations to work with Flag64 dont know if its good enough.
For VK_KHR_synchronization2 I followed the  general layout of the other extensions.

Funnily enough I have right about the same change locally, with changes pending to the xml parser/wrapper to propagate bitwidth. More than 32 used bits seems like a good heuristic but might not be true in every case, where a 64-bit flags type would be used up front in preparation for more flags appearing in the future. Not sure if such a case exists yet.
Please reopen this PR before force-pushing with the missing files.
I could possibly review this PR as is, but it is perhaps easier to share my attempt: https://github.com/MaikKlein/ash/commit/1125d2cdaa9a4ec1255b5cb5ec651c0006ee9238. Forgot about this for almost two full weeks :see_no_evil:


Crash with create:big cannons mod

*Translator warning
Probably a long time ago, when I ran the changed mod with create:big cannons and it crashed during the startup game phase, I thought it was a rendering issue.
Here's the crash report:
crash-2023-02-24_21.03.03-client.txt
Game version:1.18.2 Forge_40.2.1;
RAM:I think I allocated 8G;
Changed mod version:v0.10.2;
Create:Big Cannons version:beta-0.5.b;
Ps:Chinese warning if you want to view the crash report
Pss:I don't think this is a good question but I think it probably means that you'll meet this question with other mods because of rendering

Try running the mod with the latest version of changed.

Solved,but.........I think I found another issue?(should I close this a add a new issue?)
Mod lists:
Changed-m1.18.2-v0.11c
create-1.18.2-0.5.0.i
flywheel-forge-1.18.2-0.6.8
i18nupdatemod-1.18-3.0.2 (just a auto translate mod)
Problems:
I create a new world and after a few seconds I crashed.
And I don't know why I don't have a crash report.
Here's the lastest log:
latest.log
Solved?
No.
How to reproduce?
Add the mods and create a new world,then wait for a few sec.

After a morning of testing, I still didn't find the specific cause of the problem.
Maybe is just my personal problem.
Anyway,I think my first question has been solved,and this problem is not so important.
That's it.


issue 2401 - normalize "aarch64" architectures to "arm64"

hanging off issue #2401 add mapping for aarch64 to arm64

cc @stevvooe

LGTM
Long term, we can move to the containerd platforms package, which encodes all of these normalizations.


pcl_detector: 0.0.3-2 in 'kinetic/lcas-dist.yaml' [bloom]

Increasing version of package(s) in repository pcl_detector to 0.0.3-2:

upstream repository: https://github.com/LCAS/pcl_detector.git
release repository: https://github.com/lcas-releases/pcl_detector.git
distro file: kinetic/lcas-dist.yaml
bloom version: 0.8.0
previous version for package: 0.0.3-1

object3d_detector
* Merge pull request #3 <https://github.com/LCAS/pcl_detector/issues/3> from scosar/master
  Input parameter to set lidar topic
* Merge remote-tracking branch 'upstream/master'
* Included a parameter to set input lidar topic
* Contributors: Serhan Cosar, scosar

redundant


Scroll-bar is weirdly thin and matches background too much

That white part is the scrollbar. Didn't even notice at first

I'll change the width of the scrollbar.

Fixed in https://github.com/crutchcorn/eXGTK/commit/4e6f3d20293accf4daa87a0a964b6c90279d6dcf


update: codeql action

Applied mkpr on .github/workflows/codeql_analysis.yml
cat .github/workflows/codeql_analysis.yml

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.842%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
416

Relevant Lines:
426

💛  - Coveralls


Firefox issue

Great plugin, saved me some headaches!
Currently I'm having a problem with firefox and getting the blur to fire. The test page is located here: http://loiseau-noir.com/seb/home.html
If you click on the black circle, it opens up a new menu, and the image in the background should blur. In Chrome it works perfectly, however in Firefox no blur event fires.
Is it due to conflicting plugins or something else?
Thanks!

You can add 'filter': 'blur('+filterBlurRadius+'px)', below line 135 and it works a little better.  However, I still get other css issues on Firefox caused by this plugin.  Going to use jQuery.css instead:
$elem.css('filter', "blur(#{blurRadius})")
$elem.css('-webkit-filter', "blur(#{blurRadius})")


feat: mock mainnet btc blockstream requests

Try out Leather build a4e4581 — Extension build, Test report, Storybook, Chromatic

This pr mocks btc blockstream mainnet requests

When I mocked just there it also didn't fix the test, I actually had to put it directly in signInWithTestAccount, but still not sure why.


Token resolve fails on every page refresh.

Line 87 looks really strange.
Shouldn't we refresh the token if data.expiresAt < Date.now()?

@ramanpreetnara If you do find something wrong with that, a failing test would be amazing.

@ramanpreetnara that is checking that the expiration date is in the future before proceeding to schedule a token refresh, the actual time the refresh occurs is calculated after...
It was also changed slightly in a recent PR:
https://github.com/jpadilla/ember-cli-simple-auth-token/pull/33/files#diff-b3b19e97b14851cfabb1e6558abfc6daR196

@erichonkanen that clears things up, thanks!


chore: add trial button

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


Syntax error in the page script

Hello,
Seems the "for" is not completed for the unattached NIC

Thanks!

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

ID: 939e640f-6505-774f-68d9-697a596f14fb
Version Independent ID: 887d307d-5c87-c9fa-3a95-91ccc1aa9260
Content: Find and delete unattached Azure NICs - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/find-unattached-nics.md
Service: virtual-machines
Sub-service: networking
GitHub Login: @cynthn
Microsoft Alias: cynthn

@emarana I just ran the script and it is working without any errors.
Can you please share the screenshot of the error.

Hi @VikasPullagura-MSFT sorry, but I wonder how is this possible. I can't run on ISE or Powershell 5.1. Is necessary specific version?
Please look at the images:

Thanks!

@emarana As this is CLI script, I tried in Bash Shell and it worked.
Can you pleas try and let me know.

@emarana
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


ViewportTexture is available as a texture in decal but isn't supported.

Godot version
4.1 dev3
System information
Doesn't matter.
Not planned as feature according to #73400
Issue description
The texture is available but not supported and there are no plans on supporting it.
No error/warnings are raised either.
Steps to reproduce

add ViewportTexture as texture to Decal node
it doesn't work in game

Minimal reproduction project
N/A

Duplicate of https://github.com/godotengine/godot/issues/73400
Closing so that we can track the underlying issue in one place


scroll event fires too late on mobile iOS Safari

Scroll event is not fired until the scrolling action comes to a complete stop.
Handling touchmove event makes things significantly better:
unbindEvent(window, 'touchmove', validateT);

An article about it:
http://tjvantoll.com/2012/08/19/onscroll-event-issues-on-mobile-browsers/

Not fixed for Cordova web views :(

One caveat for Cordova developers
Although Apple implemented this change in iOS Safari, as well as its new WKWebView control, it did not change the scroll behavior in its old UIWebView control. And because of a major bug in the replacement WKWebView control, the Cordova team cannot upgrade to WKWebView yet.

This means that at the moment Cordova apps running on iOS 8 continue to pause JavaScript execution, and will continue to until Cordova can upgrade. And this doesn’t just affect Cordova apps. Any iOS app that uses web views — including Facebook, Twitter, and Chrome for iOS — will get the old behavior until they upgrade their apps to WKWebView. So yes, that means you could get different behavior opening the same URL from different iOS apps depending on which API they use internally.

Update (September 25th): Commenter Ben Kennedy found that, for whatever crazy reason, the old scroll behavior also applies to home screen web apps. So to summarize, apps that run in Safari or in a WKWebView get the new scroll behavior, apps that run in a UIWebView or in a home screen web app do not.

Hey jDeppen.
Yes I noticed it in chrome for IOS. Thanks for the explanation.


page break on tag ol

bug appear when js function try to page break tag li in ol, any solution?

Thanks! Could you share a screenshot of the (unexpected) result? And perhaps the code excerpt used to create this?

Thanks, that’s helpful.
Could it be that you are trying to use a manual page break like

, or just any element with class="page-break", somewhere in there?
The JavaScript adjustments – so far – run on
 only, so
 and
 should not be split up.

The JavaScript adjustments are now run on
 and
 as well, but still not within those elements. Perhaps this fixes the problem. Changed in https://github.com/delight-im/HTML-Sheets-of-Paper/commit/c48148cf8a6a33c12559dd7d7665988f61175f92 .


Footer update

Two small changes:

reworded the text which I always found somewhat confusing. Let me know your thoughts.
added a link to the Metrics page.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @PyvesB!

  Generated by :no_entry_sign: dangerJS against 2f18a3b570ebba8adf05f7c110a2460e4edae9b5


fix(material-experimental/mdc-list): rename mdc-list-* classes

caretaker note: merge when corresponding change cl/361202374 is submitted

I've updated the unit test selectors but there seems to be 4 remaining unit selection list unit tests that are failing. The recurring problem seems to be that a disabled list item is still selectable

I think something in their TS is still depending on the old disabled class. I added an extra binding so it applies both the original and deprecated class for now, I'll ask them to investigate


spinel-cli: Stability improvements to ping

This fixes the spurious issues highlighted in #517 by making the ping command completely asynchronous, as implemented in ot-cli.
This PR does not address creation of ip addresses upon receipt of prefix change indications (the original purpose of #517), nor remove the use of CTRL+C to terminate ncp-sim nodes.

LGTM 👍


Fix incorrect error type being recorded when Sinatra route raises exception

Fixes https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/292
Wait, we're deleting code? YUP!
tl;dr: The Rack instrumentation already catches exceptions, adds the data to the current span and marks the error status on it.
The primary problem is if an exception is raised in @app.call(env), response would be nil, so the error checking code in trace_response will throw a new exception for trying to use response. That new exception is then caught by the Rack instrumentation (I believe, or higher level instrumentation that I found when debugging).
So, if we were to make this work and catch exceptions, record them and then raise it again (as we should since this is middleware and we don't want to interfere with other exception handling), the higher level OTEL exception handler will also catch it and record it, giving us 2 exception events. That is what I originally did and saw 2 exception events, which is not correct.
This is a draft, please feel free to do a full review and make suggestions or ask questions. I'm new to this codebase so I could have done this completely wrong!

Thanks @arielvalentin! I'm super excited to get this working! It's one of the blockers for doing a proof of concept at my job.

This unfortunately does not add the sinatra exception to the span anymore. It would be great if we could do the error handling in the Rack instrumentation and record the exception there; but unfortunately I don't know how.

This unfortunately does not add the sinatra exception to the span anymore. It would be great if we could do the error handling in the Rack instrumentation and record the exception there; but unfortunately I don't know how.

@scbjans are you saying that when sinatra handles and exception it does not bubble it up to Rack so this PR isn't working as expected?

This unfortunately does not add the sinatra exception to the span anymore. It would be great if we could do the error handling in the Rack instrumentation and record the exception there; but unfortunately I don't know how.

AFAIK env['sinatra.error'] is the stacktrace from the exception, and that is added to the event via the Rack instrumentation. We could check env['sinatra.error'] in this sinatra instrumentation and add it to the span if we want, I can test that. It would probably duplicate what Rack is doing though.
Before this PR, sinatra.error wasn't being recorded at all due to the error described in the PR description.

@arielvalentin @rdoherty-fastly I did a quick test and it seems that since span.record_exception(env['sinatra.error']) is removed; the exception is not added to the span anymore (but the span is correctly marked as erroneous).
Before:

With this PR:

I'll look further into this next week.

Thanks again. I will keep on the lookout for an update.
@scbjans thanks for double checking this! I will scrutinize the changes a little more next time when there are interdependent instrumentations like this.

@scbjans That is weird because when I inspect the finished spans in the test case, I can see a stacktrace:
 exporter.finished_spans.first.events[0] => #<struct OpenTelemetry::SDK::Trace::Event name="exception", attributes={"exception.type"=>"CustomError", "exception.message"=>"custom message", "exception.stacktrace"=>"/Users/rdoherty/src/opentelemetry-ruby-contrib/instrumentation/sinatra/test/opentelemetry/instrumentation/sinatra_test.rb:26.....
I probably don't understand the correct method of handling exceptions, I'll step through the test cases from main and this branch to see what's different.

@rdoherty-fastly what is the instrumentation scope for the finished span? Is this possibly an artifact of rack-test vs when running plan old rack on an app server?

@rdoherty-fastly @arielvalentin
I'm so sorry, apparantly I was testing on an outdated clone.. :(
If I do test it on a freshly cloned repo; it all works as expected.
Sorry for the confusion!

@scbjans No worries! I do appreciate you doing exploratory testing since I am not a sinatra user.
I am glad we got that sorted out.

@scbjans no worries, thanks for testing it! \o/

I didn't fully understand why my simple tests did only work on a freshly cloned repo.
Although my branch was fully up-to-date before testing; I had an outdated named docker volume bundle. Resulting in testing against an old set of gems 🤦
It seems the original PR (https://github.com/open-telemetry/opentelemetry-ruby-contrib/pull/133) that added this wasn't even necessary in the first place, but since it was released simultaneously with the PR that based sinatra middleware on top of rack middleware (https://github.com/open-telemetry/opentelemetry-ruby-contrib/pull/54) I guess I just simply never noticed.
Anyway, always good to see code cleaned up where it can.
I learned (and re-learned 😄  ) stuff along the way.
Thank you both.

You're welcome @scbjans ! Thanks for reviewing!

So what is the next step on this PR? I see it was approved but I don't have privileges to merge. Someone else will have to do that for me :)

@rdoherty-fastly we have a policy of having at least 2 approvals from @open-telemetry/ruby-contrib-approvers and I am the only one who has had a chance to review this PR so far. Once we get another approval I will merge this PR and release the gem.
I have added this PR to the agenda^1 for the Ruby SIG so hopefully that will be enough of a signal boost. Please join us if you are available!

Oops ok thanks for the context @arielvalentin!


minor updates to README.md

Hello, I made some minor changes to the documentation.

Thanks Bert 👍


[BEAM-4014] Replace the Java maven based PostCommit with a Gradle based version.

https://builds.apache.org/view/A-D/view/Beam/job/beam_PreCommit_Java_MavenInstall/19300/testReport/api/json
shows the list of ITs that were executed. Note that its all the examples/java ITs and the sdks/java/io/google-cloud-platform ITs.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand:

[ ] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

CC: @tgroh

If #5048 is merged, the project names need to be updated.
CC: @swegner


textarea spec

📝 Changes

TextArea spec

what was your thinking on making this a separate component vs reusing TextField? (i think it's perfectly reasonable btw, but just curious how you've thought about it)

i think to start, ill build it as its own component, but if i end up duplicating enough code, i might extend the TextField to support text area fields, but still expose TextArea as its own component, that way it remains friendly to consumers and we don't have to handle invalid configurations.


[Network] vnet peering: deprecated --remote-vnet-id in favor of --remote-vnet

Addressed #7016:

Deprecated --remote-vnet-id in favor of --remote-vnet.
Passing remote vnet's name instead of resource ID through --remote-vnet /  --remote-vnet-id does not crash CLI.
Can also specify resource group of remote vnet if only name is provided through --remote-vnet-rg.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

Closes #7016

@tjprescott  Addressed your requests. Let me know if this looks good to you.


Homepage/nav style updates

@jacksoncvm, thanks -- is this the Bookbinder re-skin effort I've heard about?

hey @RichardJJG -- it is! but i need to revert this from master-- we are going to test in a test branch first! thanks!!


Use latest version of helm-exporter

Signed-off-by: Arvind Naidu<EMAIL_ADDRESS>What this PR does / why we need it:
This PR updates the version of helm-exporter to export the values of the appVersion which was a very recent fix.
Special notes for your reviewer:
I am currently using the latest version of the image since v0.2.0 is 2 days old but latest is 14 hours old which fixes this PR over in the original application repository.

I personally am against using the latest tag so I will create a new issue over in the original repository to address this matter and hopefully be able to add a tag for the latest Docker image.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@naiduarvind thanks for the PR.  I just cut a 0.3.0 release of helm-exporter.  The latest version changes the port so this PR will need to change the port to 9157.  I'll approve this if you can bump the version to 0.3.0 and update the port.
Thanks.

@naiduarvind thanks for the PR. I just cut a 0.3.0 release of helm-exporter. The latest version changes the port so this PR will need to change the port to 9157. I'll approve this if you can bump the version to 0.3.0 and update the port.
Thanks.

I have done the above as requested. :)

/ok-to-test

/lgtm

@sstarcher  Seems like the test has failed. Any idea why? From what I see in the dashboard, it seems like Helm timed out. Any further detail would help from you if it is an issue from my end.

/retest

@naiduarvind I would recommend running it locally and doing a helm install and see if he comes healthy.

/lgtm

I see the likely error.  It's using /metrics for the liveness/readiness and those now do the actual scraping and take longer.

I just cut a 0.3.1 that adds a separate healthcheck endpoint /healthz

@sstarcher - made the changes you listed above. However, I am still seeing the charts err out. I changed the liveness probe from /metrics to /healthz (the dummy one).

Install it into a local cluster and see what's happening.

@sstarcher - testing it tonight and will update here once done.

@sstarcher - I made the changes and ran locally. Initially, I made a mistake with an incorrect port on the service so I changed it to the correct one (9157) and reinstalled the chart. However, the liveness and readiness probe just doesn't work for both /metrics and /healthz.

@sstarcher - I also removed both the probes just to remove them as culprits but I believe the app itself is not coming up. I don't see much in the STDOUT and I don't see it reporting any information on the Grafana dashboard.

That would explain it the port is 9571... I have it listed everywhere as 9571... but I have a commit that says 9157... so bad on my part.
https://github.com/sstarcher/helm-exporter#metrics

@sstarcher - made the changes and ran it in my cluster. All looks good! 🔥

Would you be updating the dashboard for Grafana to add the appVersion?

Made the changes on the dashboard for the new Helm chart. But unfortunately, I am unable to see the appVersion.

I can, but I have not done so yet.

/lgtm

@naiduarvind I updated the grafana dashboard.

@sstarcher - unfortunately, I don't see the appVersion in the new dashboard. Is there anything that needs to be done?

No it should work.  I would start by checking your prometheus to ensure you have the appVersion metrics and verify you have the dashboard updated to the second revision.


search on search parameters

Given discussion
https://chat.fhir.org/#narrow/stream/179166-implementers/topic/AuditEvent.20search
for cases where a Audit Client wants to search for AuditEvents about Search events, with specific search parameters;
there is a need for an extended parameter to search on the .description element.

Although the .description element would include the cleaned search parameters, it is not clear what kinds of searches on AuditEvent can use this kind of a search parameter to justify the need for a search parameter extension, and thus to also drive conformance requirements for implementation.


Button for Creating New Project has moved in 4.2 compared to 4.1

Godot version
4.2 beta 3
System information
Godot v4.2.beta3 - Windows 10.0.19045 - GLES3 (Compatibility) - NVIDIA GeForce GTX 1050 Ti (NVIDIA; <IP_ADDRESS>47) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)
Issue description
Button for Creating New Project has moved in 4.2 compared to 4.1
Here is where the "New Project" button was in the godot 4.1 launcher

It got moved here in 4.2

This messes with my workflow compared to 4.1. Can an option be added to keep the old layout?
Steps to reproduce
Run the launcher for 4.1
Then Run the launcher for 4.2 and get confused on the location for the new project button
Minimal reproduction project
N/A

How does it mess with your workflow?

To give an idea, I make tons of new projects. While some were downloaded from the asset store, I have 81 project files in my project folder. I'd have to unlearn the positioning (this isn't a huge deal, though why was the position changed?)

It was changed to improve navigation of this dialog, better separating unrelated items.

I see. Just my thoughts: It makes more sense to me to have all the buttons together. (though I had actually never used the other buttons until this thread. I always just double clicked projects to open them.)

For reference, this was changed to https://github.com/godotengine/godot/pull/50674.
I'd say this is a matter of habit. I suggest using the new UI for a few weeks, and you will probably find the old UI less convenient in the long run :slightly_smiling_face:


An annotation error in /widgets/routes.dart

location : widgets/routes.dart  line 1060 ，and 1065
SlideTransition's position should be a type of Tween<Offset>，not  AlignmentTween  or TweenOffset 。

FR

SlideTransition's widget example also shows returning a Tween<Offset> rather than AlignmentTween or TweenOffset.
late final Animation<Offset> _offsetAnimation = Tween<Offset>(
https://api.flutter.dev/flutter/widgets/SlideTransition-class.html#widgets.SlideTransition.1


BE: Messages: Getting stuck on "polling partitions"

via Discord:
For topics with larger numbers of messages, clicking the "Next" button to load the next page of messages, or switching to Newest First sorting results in it getting stuck on "Polling Partitions". In the logs of the application, it shows the following
INFO [boundedElastic-1719] o.a.k.c.u.AppInfoParser: App info kafka.consumer for kafka-ui-consumer-1712975021172 unregistered
INFO [boundedElastic-1719] c.p.k.u.e.EnhancedConsumer: [Consumer clientId=kafka-ui-consumer-1712975021172, groupId=null] Kafka consumer has been closed

Screen recording:
https://discord.com/channels/897805035122077716/1228532718493761546/1232452982763687987

@Haarolean if recording still available?

@iliax updated the link

should be fixed with https://github.com/kafbat/kafka-ui/pull/406

duplicate of #299


V5 Modals: Add loading spinner before displaying content

I hope the core team adds loading spinner functionality to the modal before displaying content something similar to this example:
Live Example: https://codepen.io/themes4all/pen/oNWeYzp

That's outside of the component's scope.

That's outside of the component's scope.

it's a merge between two components

To explain it with different words, it doesn't make any sense because our modal doesn't load any content that would require to show a spinner.


Version Updates

We've been working on e2e tests (still pending) and the major Sarama conversion and have fallen behind on our use of eventing and eventing-contrib/kafka and need to catch up ; )
Note - Once we're up to date we should refactor the Dispatcher's retry mechanism to make use of the newly added DispatchMessageWithRetries() functionality!
Note - Should also look at TODO in the ConsumeClaim() in handler.go for improving the the URI Resolving to support object refs as well.

/assign @travis-minke-sap


proposal to implement a state machine instead.

Fahd proposed that instead of restricting callers to a linear workflow, we could implement a general state machine instead. I think the variant of this I like the best is where you don't give a liner list of steps. Instead, the logic for each state returns some description of the result, and the state machine is actually implemented in the state enum. There are some obvious problems with this exact implementation, but I wanted to sketch out what it would kind of look like.

:+1:

This is more note-taking than a real pull request. Closing for posterity.


need to use holdingId instead of locationId in purchase order lines

Currently, we use a Symphony holding code to FOLIO location mapping to get the locationId.
https://github.com/sul-dlss/folio-tasks/blob/8b87bc3ac1d591c03743bd0efb365601ba9f5a2a/tasks/helpers/orders/po_lines.rb#L111-L115
Need new mapping to figure out FOLIO holdingId to put in purchase order line data. See related tickets https://github.com/sul-dlss/FOLIO-Project-Stanford/issues/268#issuecomment-1374192964 and https://github.com/sul-dlss/FOLIO-Project-Stanford/issues/274

Using holdingId will be done in a post migration of inventory step, where we can lookup holdingId. The orders will be loaded with locationId for now (since it is a required field).


Assert time and audience conditions on SAML responses

SAML responses can (and normally do) supply conditions for when the assertions should be considered valid. NotBefore and NotOnOrAfter restricts to a time period. AudienceRestriction restricts to the intended audience.
The library was previously not validating these at all.

... and the idea is to PR this to clever/saml2 as well?

@brettkiefer yeah — sorry, should have made that clear. Approval to contribute to the project is pending with the legal team, so this PR on our fork is just to facilitate code review before I can open the PR back to clever/saml2.

@brettkiefer I've added a commit to allow for clock skew tolerance between servers — see commit message for details: https://github.com/trello/saml2/pull/1/commits/eefbe39f4bdbb7eff896b465b5271e5b82bd39d3
Would you kindly review that additional commit, then I can open the PR with Clever.

Closed as this was just for internal review purposes. Continued in a PR to merge back up to the source repo: https://github.com/Clever/saml2/pull/116


CountEncoder tests in tests/test_encoders.py are skipped

Expected Behavior
tests/test_encoders.py should be running for all encoders.
Actual Behavior
CountEncoder is missing from __all__ in category_encoders/__init__.py. This has the effect that tests are skipped.
11 of these tests fail for CountEncoder.

If CountEncoder is included, some of their tests are not going to pass: right now, CountEncoder doesn't implement  some of the methods that are in the rest of the encoders, such as "get_feature_names".

I see, do you know if this is mentioned in the code or elsewhere? I could not find anything, so i suggest adding some remarks. Also I am just trying to implement the missing features and would be happy about some help.

Not helping with the code. But a few notes after looking at the documentation:

Consider renaming the argument value 'count' in handle_missing to 'value'. Reasoning: 'value' string is used in other encoders, making it easier to experimentally test multiple encoders. And it allows you to use normalized=True argument without causing the conflict between the "count" string and what actually happens (calculation of the relative frequency?).

Consider adding 'value' value into handle_unknown, which would impute with 0, and set it as the default behaviour. Reasoning: It would improve compatibility with other encoders.

The doc says Defaults to None which uses NaN behaviour specified at fit time. I am not sure that is correct (if nothing else, fit method does not document the possibility to set handle_unknown).

I am not sure why _check_set_create_dict_attrs has line 'handle_unknown': 'count', when 'count' can be currently set only in handle_missing and not in handle_unknown.

The documentation repeatedly mentions or dict of.. A dict of what to what?

The documentation mentions The default name can be long ae may keep changing. What is ae?

@janmotl thanks a lot, your input is very much appreciated and helpful to me.

see PR #260
1 + 2.) I renamed and added the option 'value' as you suggested. I could get handle_missing to pass the tests but got stuck on handle_unknown. There is one test still failing test_encoders.py::TestEncoders::test_handle_unknown_value.
3.) I think the Docs refer to the way how NaN is treated in fit with handle_missing.
4.) this was/is confusing for me also
5.) I amended the docs to dict of {column: option}. As far as I understand you can choose a different option per column.
6.) I found this in a comment, it's just meant to be "and".
So the problems that still remain are handle_unknown_value and combine_min_nan_groups.

checking in with this after some while: Can we close this issue? As of today all tests work for count encoder

Great, yes we can close it!


Improving the latency

Is improving the latency of the model on the roadmap, e.g. real-time TTS?

It is certainly on the roadmap! Out of interest, what are you building with TTS today? What latency are you looking for?

We're building Lifelike, a voice AI character platform (https://lifelike.app), so we'd need latency < 500ms (the lower the better). If there's any way we can contribute to that effort over the next few months, would love to follow up over email<EMAIL_ADDRESS>

Thanks for offering to help, we would love contributions! I have emailed you for an intro call.
Btw, I tried to use your platform, but got waitlisted :)

Would also be willing to help<EMAIL_ADDRESS>

@gad2103, I have emailed you :)

Hello @sidroopdaska
Would love to contribute too. Please reach out to me at<EMAIL_ADDRESS>

@athenasaurav, I have emailed you

hey folks, try https://github.com/metavoiceio/metavoice-src/pull/71 ... it's a bit dirty, but does give a 2-4x speedup (depending on hardware)!

Closing due to resolution


Use a bot that monitors broken links

I just found this tool while browsing another "awesome" list. It can be used with Travis CI, which I believe this repo uses.
https://github.com/dkhamsing/awesome_bot
I know we currently use a script from @borgified, but I am not sure how frequently it runs. Maybe this could even be forked or modified to check for personal/affiliate links.
Pinging @vhf for thoughts

Interesting project, thanks @onebree. I'll set this up tomorrow on a branch and test it.

sorry i've been super busy w/ the new job so havent been able to contribute a whole lot lately. im running my url_checker atm and will post an update after it ends. not sure if we still need to run it or not (havent kept up to date with the latest with where we are towards automating the url checking)

Nice!

i got some free time, im gonna set up a docker container, put the script in it and publish it on dockerhub so anyone can just launch and run and have all the dependencies already taken care of

That's awesome, thanks a lot @borgified !

Anyone interested in running this regularly?

I wouldn't mind running it on my servers via a cronjob. How often though, any wish @eshellman ?

once a month?

@eshellman from time to time (every other month these days), i come by here and run my script to check for bad links, i just found out that the project isnt under vhf anymore but my question is, do you want me to continue doing this?
what it entails:

i run the script
i copy the output and create an issue here
other volunteers can click on those links to verify if those urls are indeed bad (sometimes they are  are not up all the time or not fully globally accessible)
the same volunteers can create prs to correct the appropriate files and remove bad links

I forgot to report back. Setting this up was more troublesome than I thought because we'd need to create a github account and setup API keys for it to be able to come here to create an Issue.

@borgified Yes, please! I'd be happy to add you as a team member if it would help.

sure ok (thanks @eshellman). actually in a previous iteration of the script i did manage to automate creating github issues based on my script output but that ended up spamming a bunch of contributors (example: https://github.com/EbookFoundation/free-programming-books/issues/733). i can give another go again and automate (this time being wiser about not spamming)

source code script?
El 03/08/2017 22:46, "borgified"<EMAIL_ADDRESS>escribió:

@eshellman https://github.com/eshellman from time to time (every other
month these days), i come by here and run my script to check for bad links,
i just found out that the project isnt under vhf anymore but my question
is, do you want me to continue doing this?
what it entails:

i run the script
i copy the output and create an issue here
other volunteers can click on those links to verify if those urls
are indeed bad (sometimes they are are not up all the time or not fully
globally accessible)
the same volunteers can create prs to correct the appropriate files
and remove bad links

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/issues/1849#issuecomment-320085229,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJYDQyZ8naZnJ_RFG5pRtGkBxlPP_5Bks5sUjGqgaJpZM4HdYuS
.

@kiquenet here you go: https://github.com/borgified/url_checker

One "link checker" issue is enough #2541


feat(onboarding): add context module for onboarding follows and saves

The type of this PR is: feat

This PR resolves GRO-1153
Description

Adding missing onboarding flow context module: context_module: ContextModule.onboardingFlow per this onboarding analytics spreadsheet here -->
https://docs.google.com/spreadsheets/d/12wj1rt-j5SguboPJMuaM1GqJD-5B5u5piipw8MQ5Zuk/edit?pli=1#gid=275928850

:rocket: PR was released in v4.61.0 :rocket:


analytics: Update query for remote_push_devices in remote activity.

Updates query to count the distinct pairs of user_id and user_uuid on the RemotePushDeviceToken table.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @laurynmm!


keep LeapCSharp up to date with platform 562d155

This includes the following PR:
Dispatch Connect and Init events to clients if we have already inited… #5154
Remove phone tracking and dependencies #5252

I am getting errors in the AR scene
Exception: System.NullReferenceException: Object reference not set to an instance of an object
at Leap.Config.handleConfigChange (System.Object sender, Leap.ConfigChangeEventArgs eventArgs) [0x0001f] in C:\Users\Alex\Documents\GitHub\UnityModules\Assets\LeapC\Config.cs:49
at (wrapper delegate-invoke) System.EventHandler1<Leap.ConfigChangeEventArgs>:invoke_void__this___object_ConfigChangeEventArgs (object,Leap.ConfigChangeEventArgs)   at (wrapper delegate-invoke) System.EventHandler1<Leap.ConfigChangeEventArgs>:invoke_void__this___object_ConfigChangeEventArgs (object,Leap.ConfigChangeEventArgs)
at (wrapper delegate-invoke) System.EventHandler1<Leap.ConfigChangeEventArgs>:invoke_void__this___object_ConfigChangeEventArgs (object,Leap.ConfigChangeEventArgs)   at Leap.CSharpExtensions.Dispatch[ConfigChangeEventArgs] (System.EventHandler1 handler, System.Object sender, Leap.ConfigChangeEventArgs eventArgs) [0x00006] in C:\Users\Alex\Documents\GitHub\UnityModules\Assets\LeapC\CSharpExtensions.cs:79
at LeapInternal.Connection.handleConfigChange (LeapInternal.LEAP_CONFIG_CHANGE_EVENT& configEvent) [0x0004b] in C:\Users\Alex\Documents\GitHub\UnityModules\Assets\LeapC\Connection.cs:525
at LeapInternal.Connection.processMessages () [0x00206] in C:\Users\Alex\Documents\GitHub\UnityModules\Assets\LeapC\Connection.cs:241
Both Images and Tracking are not functioning in this scene.

Confirmed that I do not get errors and do get images/tracking in the AR scene on develop

The develop is already brought up to date, closing this


feat(charts): upgrade chart to v1.7.1

Signed-off-by: Raphaël Pinson<EMAIL_ADDRESS>

Fails to sync on Exoscale 1.22.5 with

Probably due to a CRD upgrade. Fresh install (blue/green) should work better.

I remember having to disable the API check hook (but maybe it’s possible to make it work), maybe the issue is linked to it?
  # Startup API check hook doesn’t seem to work well with Argo CD
  startupapicheck:
    enabled: false


fix nightly hash

Closes #
Proposed Changes

I submit this contribution under Apache-2.0 license.

Thanks @elit-cx , Works OK ( https://github.com/Checkmarx/kics/releases/tag/untagged-25b16556f0bab895790f)


Ensure isn't being respected on location.

This is similar to the ensure => absent PR I submitted last week.  Locations support ensure, but it wasn't respected.
Tests passing in Ruby 2.1.2.   I'd be glad to change this and resubmit if I've missed something.

hi @kwolf, the change makes sense. Can you please squash this to one commit? Thanks! Once done I'll merge.

I missed up squishing my commits.  I'm going to close this and open another PR.  Sorry.


https://updater.retrodriven.com/ cert expired

Issued On	Friday, August 11, 2023 at 8:44:21 AM
Expires On	Thursday, November 9, 2023 at 8:44:20 AM
could be important

Thanks for the heads up! This has been resolved.


Accept 409 as expected behaviour in broker-test

Signed-off-by: Georgi Lozev<EMAIL_ADDRESS>

Coverage remained the same at 89.358% when pulling 8b28a752fd0eef5d0579640037087bdf2d11e3c3 on SAP:accept_confict_broker into 211e9d07f112e67025dd204af6e71de173674469 on cloudfoundry-incubator:master.


added file

test

Codecov Report

Merging #8 (bf781d7) into master (a647b61) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   98.01%   97.96%   -0.05%
==========================================
  Files         129      129
  Lines       22591    22063     -528
==========================================
- Hits        22142    21615     -527
+ Misses        449      448       -1

Impacted Files
Coverage Δ

...ornetwork/matrixproductstates/infinite_mps_test.py
84.78% <0.00%> (-1.50%)
:arrow_down:

...ckends/tensorflow/tensorflow_tensornetwork_test.py
59.75% <0.00%> (-1.17%)
:arrow_down:

tensornetwork/linalg/tests/initialization_test.py
84.61% <0.00%> (-0.65%)
:arrow_down:

tensornetwork/backends/decorators_test.py
92.64% <0.00%> (-0.51%)
:arrow_down:

...ensornetwork/tests/tensornetwork_symmetric_test.py
92.70% <0.00%> (-0.37%)
:arrow_down:

tensornetwork/backends/decorators.py
96.77% <0.00%> (-0.20%)
:arrow_down:

tensornetwork/linalg/tests/test_linalg.py
96.31% <0.00%> (-0.20%)
:arrow_down:

tensornetwork/tests/tensor_test.py
98.51% <0.00%> (-0.15%)
:arrow_down:

examples/simple_mera/simple_mera.py
95.49% <0.00%> (-0.12%)
:arrow_down:

...ors/opt_einsum_paths/path_contractors_node_test.py
89.78% <0.00%> (-0.11%)
:arrow_down:

... and 44 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 a647b61...a0f25b4. Read the comment docs.


Update to 0.04.21

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


fix checkpoint_completion_target

currently this parameter isn't being computed and on pg 14 the default value changed to 0.9.

The standard formula is (checkpoint_timeout - 2min) / checkpoint_timeout rounded to one decimal.
I know, the default for checkpoint_timeout is 600s so: 600 - 120 / 600 = 0.8


Add interpolation args on bootstrap

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #846

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added.

[ ] bumpversion patch has been called on this branch
[x] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Update doc of bootstrap parameter mainly to warn the user that there are cases where bootstrap should be left to False
Added alpha and beta parameters on climatology_bounds.attrs to treat them the same way window is handled in bootstrapping.

Does this PR introduce a breaking change?
Non.
Other information:
I'm not sure how to unit test this.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 90.243%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
12477

Relevant Lines:
13826

💛  - Coveralls


Handle closing flow sequence after explicit key

Currently after an explicit flow key '?' in a flow sequence, an immediately following closing ] is ignored by the parser:
% echo '[ ? ]' | ./tests/run-parser-test-suite --flow keep
+STR
+DOC
+SEQ []
+MAP {}
=VAL :
=VAL :
-MAP
Parse error: did not find expected ',' or ']'
Line: 2 Column: 1
% echo '[ ? ] ]' | ./tests/run-parser-test-suite --flow keep
+STR
+DOC
+SEQ []
+MAP {}
=VAL :
=VAL :
-MAP
-SEQ
-DOC
-STR

It is read correctly by the scanner as a YAML_FLOW_SEQUENCE_END_TOKEN, and the flow_level is decreased. Then it is passed to the parser where it gets ignored.
This leads to invalid YAML being accepted, and valid YAML resulting in an error.
Also the flow_level is incorrectly decreased, so you can nest sequences like that without running in to the MAX_NESTING_LEVEL.

[{?}] and [{?}]] are similar. Can you verify it fixes those?

Those are not affected by the code and work like expected already.

Tests should probably be added.

I will add tests to the yaml-test-suite.

This PR caused a problem, and is now fixed with #296


[BUG] Documentation URL for Amazon s3 incorrect in Linux reference

Describe the bug
on https://min.io/docs/minio/linux/reference/minio-mc.html the URL for S3 is incorrect; it should be https://s3.amazonaws.com
To Reproduce
$ mc alias set myAlias https://s3.amazon.com ACCESS_KEY SECRET_KEY
mc:  Unable to initialize new alias from the provided credentials. Get "https://s3.amazon.com": dial tcp <IP_ADDRESS>:443: connect: connection refused.
Expected behavior
Changing the URL to amazonaws.com fixes it:
$ mc alias set myAlias https://s3.amazonaws.com ACCESS_KEY SECRET_KEY
Added myAlias successfully.
Desktop (please complete the following information):

OS: Ubuntu

Additional context
Add any other context about the problem here.

{your-amazon-instance-URL}

See: https://docs.aws.amazon.com/general/latest/gr/s3.html

@djwfyi none of the examples in the URL you provided have either the minio documented url or my corrected url; can you describe what the link is for? Also, "{your-amazon-instance-URL}" is not in the documentation either; can you describe what you wrote that for?

@djwfyi none of the examples in the URL you provided have either the minio documented url or my corrected url; can you describe what the link is for? Also, "{your-amazon-instance-URL}" is not in the documentation either; can you describe what you wrote that for?

The URL in the example is just a dummy URL. It was never intended to be copy and pasted as is and used. You need to replace it with the URL to your s3 instance. There is no one filler URL that will work for all S3 users. So the proposal is to replace the URL in the example with something like what I noted above instead of anything that looks like it could be copy/pasted as is.

Thanks for the clarification. All the URLs follow the same pattern:
s3.us-east-2.amazonaws.com
so maybe change the documentation to
s3.{your-region}.amazonaws.com


API: Accept upper case "reserved" keyword

Depending on the kernel version, it might be uppercase or lowercase.
See https://github.com/torvalds/linux/commit/640e1b38b00550990cecd809021cd37716e45922.

Note: if I understand correctly,  TestE820ReservedCheck goes directly for the system's memmap, which is not mocked. So this is not really unit test.
To implement unit tests, I'd need to change a bit how IsReservedInE810 reads the filesystem, to be able to do dependency injection. Let me know if you want me to give it a try.

@marcoguerri I am going to refactor it, Feel free to push PRs :)


h3dnet model config parameter

what's the meaning of upper_thresh(default 100)?
why num_dims is 2,1,0 respectively in  primitive_z_cfg,primitive_xy_cfg,primitive_line_cfg？

@encore-zhou @Tai-Wang 有人回答吗

@achao2013 According to paper, primitive_z means object center, primitive_xy means face center and primitive_line means edge center, so we use different dimension of vectors to represent the primitive center offset.
As for upper_thresh, we invalidate the GT center offset by assigning the this value. For more details, you can refer to here or the official implementation.

@encore-zhou  thanks, but that's not my question.  i want to know why set num_dims is 2,1,0 respectively in primitive_z_cfg,primitive_xy_cfg,primitive_line_cfg？

Secondly, i read the code but i still  fill confused why offset center for upper_thresh(100). does it mean a infinity for centers that below the thresh。


SignatureHash does not left-shift the chain id sufficiently

There is the following code in interpreter.cpp:SignatureHash() function:
// MVF-BU begin: apply the active forkid if we're hashing to produce a signature
ss << txTmp << ((nChainId << 1) | nHashType);
// MVF-BU end

The chain id should be left-shifted by 8 bits, not 1.

Closing MVF prototype developments subsequent to release of Bitcoin Cash clients.


Update About.js

Additional explanation of change in time series data, plus style changes to comply with GDS style guide for dates, times and numbers

@paulfryers Could you please send a pull request to the development branch instead?


Proper way to upgrade Idris2?

I had trouble figuring out how to tell pack to install a new release of Idris2.  After poking around in the help and trying a few pack commands, I finally discovered that pack switch latest would download, build, and install the latest version (0.7.0, specifically 0.7.0-27780073c at this moment).  I might have missed something in the online docs or commandline help, but I didn't find anything that made it clear that this would have the desired effect.
Is this the right way to upgrade Idris2 using pack?
If so, I suggest adding a note somewhere online or in the help docs that clarifies this. I can submit a PR if that would be helpful.  I'm not sure what the best documentation strategy is.
Thanks!

(Later I found notes that I wrote to myself earlier suggesting that this in fact the right way to upgrade idris2.  I'm not sure where I got the idea from--maybe by the same route as described above, or maybe someone mentioned it on Discord.)

Using pack switch latest is indeed the correct way to do this. Unfortunately, this is indeed not yet properly documented. It would be great if we had a section about pack switch in the README, probably as a subsection of "Usage". PRs highly welcome.

OK, thanks--I'll submit something when I get a minute.  It might not be perfect, but it will be a start, and I won't mind corrections.

I have a further question about how pack works before writing something, but it's related to something that @mattpolzin wrote today in Discord, so I'll ask there.


chore(master): release 0.98.0-alpha

:robot: I have created a release beep boop
0.98.0-alpha (2024-05-06)
Features

new (b598b05)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mafqla/yalis-blog/releases/tag/v0.98.0-alpha :sunflower:


Add upstream connection time and header time metrics

Proposed changes

Add the following new metrics:

nginx.upstream.connect.time
nginx.upstream.connect.time.count
nginx.upstream.connect.time.max
nginx.upstream.connect.time.median
nginx.upstream.connect.time.pctl95
nginx.upstream.header.time
nginx.upstream.header.time.count
nginx.upstream.header.time.max
nginx.upstream.header.time.median
nginx.upstream.header.time.pctl95

Improve performance of request time metrics

Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[x] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[x] I have updated any relevant documentation (README.md)

Looks good. You just need to run make deps


Migrate to Update 6

Minimum requirement:

[x] 1. Ensure source code compiles & runs.
[x] 2. Fix all new warnings & errors.
[x] 3. There is no test design document, play with the app a bit and ensure it operates as you might expect!

@esreli the PR has been merged to master and I have published the release. Can you verify?

Repo and release look good. I'm assuming testing to support U6 was performed by the Android team. Verified.


BUG: gravity tries to send a message that has already been relayed

Details:  https://www.notion.so/volumefi/ERC-20-transfers-on-op-main-paloma-testnet-a1e235dd59b74106ad5b0651bfb087a2?pvs=4
Note that both birdlady and birdpoop2 were taken offline by stopping pigeon service to force lack of consensus to stop more messages to be sent.

We still have this persisting on testnet due to batches still in the system. Pigeons still see the bad batch.

@taariq @byte-bandit, does this mean more work will be required to fully fix this issue?

@verabehr The main problem is fixed, Validators will not keep burning ETH trying to relay the batch. So this lowers the priority here I assume. There is however still a bug in place, as Pigeons are supposed to remove a batch once its been executed, and this doesn't seem to work yet either. I assume we have to supply the same fix I did for this at another point in the code.
So this will need to be addressed before launching gravity on main net. Those batches won't cause Validators to burn ETH any longer, but they will clog the system and create noise and traffic (and also never ever disappear on their own).a

Thanks @byte-bandit. Agreed on the lower priority here. I think it makes
sense to break this into a new ticket then
On Mon, Oct 23, 2023 at 3:44 PM Christian Lohr @.***>
wrote:

@verabehr https://github.com/verabehr The main problem is fixed,
Validators will not keep burning ETH trying to relay the batch. So this
lowers the priority here I assume. There is however still a bug in place,
as Pigeons are supposed to remove a batch once its been executed, and this
doesn't seem to work yet either. I assume we have to supply the same fix I
did for this at another point in the code.
So this will need to be addressed before launching gravity on main net.
Those batches won't cause Validators to burn ETH any longer, but they will
clog the system and create noise and traffic (and also never ever disappear
on their own).a
—
Reply to this email directly, view it on GitHub
https://github.com/VolumeFi/paloma/issues/880#issuecomment-1775233700,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYIW34CEUPFRYFYSTGT6XLYAZYELAVCNFSM6AAAAAA6EJAQBOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGIZTGNZQGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Added note for compatible storage accounts

Incompatible storage accounts mentioned

@damendo : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@damendo: I'm sorry - only the author of this article, @KumudD, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@jborsecnik : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @KumudD - Would you review the proposed changes? Thanks.
@damendo - Thank you for your contribution. The article author needs to approve the revisions you suggest.

@KumudD  @ktoliver  Customers are reporting bugs and we need to get this documentation update in right away. For most docs after Jim the author has changed. How do we change the author to PM so that we can close minor enhancements asap ?   This is day in day out update for us,

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.

fixed note and resolved merge conflict. @KumudD @twooley - can you review? thanks.

@KumudD Kumud Dwivedi FTE @ktoliver Kristine Toliver (Aquent LLC) Vendor Customers are reporting bugs and we need to get this documentation update in right away. For most docs after Jim the author has changed. How do we change the author to PM so that we can close minor enhancements asap ? This is day in day out update for us,

Hi @vinynigam - In the public repo, approval by the author or a qualified reviewer is required. PRs opened in the private repo don't require approval.

#sign-off

@ktoliver  How do we get the authors changed to PM. PMs have originally authored these docs. Jim had published them due to access issues at that time.


kstars: 3.4.3 -> 3.5.0

Motivation for this change
Add new kstars required dependency (stellarsolver) to nixpkgs, and update kstars to 3.5.0. Also updates kstars to use newer version of qt (5.15) and cleans up some mistakes I made in previous update to kstars (deviating from nixpkgs wiki recommendation to use imported mkDerivation instead of wrapQtApps hook).
ksutils.cpp patch also updated to support xplanet binary in the nix store.
Also adding myself as a maintainer to receive pings about kstars, I can force push that out if it's inappropriate.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 104848 run on x86_64-linux 1

2 packages built:

kstars
stellarsolver

Result of nixpkgs-review pr 104848 run on x86_64-linux 1

2 packages built:

kstars
stellarsolver

@hjones2199 thanks for working on this. Some of your changes were already done, and there's merge conflicts now. If you could please fix them.

@hjones2199 thanks for working on this. Some of your changes were already done, and there's merge conflicts now. If you could please fix them.

Thanks for the heads up, rebased onto master and fixed conflicts


[Java] Adding Spotless to Algorithm module

Describe the enhancement requested
The main issue discussing the inclusion of Spotless plugin is discussed here.
Component(s)
Java

Issue resolved by pull request 41825
https://github.com/apache/arrow/pull/41825


[Feature] Re-export HomeHero from theme-default for customizable usages?

What problem does this feature solve?
I'm thinking of rendering <HomeHero /> for certain sub-pages. Do you think this could be a good idea?
What does the proposed API look like?
// docs/contents/zh/dir/dir/index.mdx

import { HomeHero } from 'rspress/runtime';

<HomeHero
  frontmatter={{
    hero: {
      name: "A Sub-Project",
      text: "A Sub-Project Under Main Project",
      actions: [
        {
          text: "Get Started",
          link: "/start/index.html",
          theme: "brand",
        }
      ]
    }
   }}
/>

Yeah, i think it's ok to export more automatic render abilities in theme-default. cc @10Derozan

I have re-export all internal components which in rspress, in #817 .
And after this, you can use it from 'rspress/theme'

It's in https://rspress.dev/api/client-api/api-components.html#homehero


WoWSim - Rogue Settings not saving the Professions

In the local version of wowsim (V. 0.1.31) in the Rogue Sim, saving "Settings" does not save the selected Professions!
All other settings can be saved as expected.

Still present in the latest version v0.1.35 of the simulator (downloadable AND online) Professions are not saved under "Saved Settings"if you switch between different Settings for different charaacters. all other settings are saved, so this should be an easy fix to add those two fields to the save. Please?

Closing as a duplicate of https://github.com/wowsims/wotlk/issues/3522. This will be fixed in the next release though


Candidate should not see 'New application process' blue box

The candidates should no longer see the shot above. This used to happen when they hit 'Check if you're eligible and apply'. They should not just be taken to the citizenship question below:

Hey @tomlovesgithub could you take care of this one. The view and route can be deleted


feat: added youtube link in footer!

Fixes #4047
Describe the changes you have made in this PR -
YouTube added to footer

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

This has been fixed in https://github.com/CircuitVerse/CircuitVerse/pull/4049


💡 RFC: Proper web environment support

Background & Motivation
Right now when installing the Astro extension in a web VS Code environnement such as vscode.dev you.. don't get much outside of syntax coloring because the extension isn't packaged properly and is not following the requirements for web extensions
Proposed Solution
Possible solutions
Follow the requirements at https://code.visualstudio.com/api/extension-guides/web-extensions and see how well we fare
Detailed Design
No response
Help make it happen!

[X] I am willing to submit a PR to implement this change.
[ ] I am willing to submit a PR to implement this change, but would need some guidance.
[ ] I am not willing to submit a PR to implement this change.

Interesting repo to check when we'll attack this: https://github.com/asafamr/svelte-vscode-web


Q: how to keep the  zoom constant  for a sequence of  images?

i have a squence of images, i want  if i zoom one of these images,  the images after it can keep the zoom constant and fixed position?
i know
sequenceMode: true,
preserveViewport: true,
can solve the priblem.
But, i show different DZi images like belows
function change(){
 OpenSeadragon({
	 //$("#aa").mousedown(function(){
	    //OpenSeadragon({
              id: "openSeadragon1", //the div taht will be showed
              prefixUrl: "./images/",
			  sequenceMode: true,
			  preserveViewport:true,
              tileSources: [{
              Image: {
              xmlns:  "http://schemas.microsoft.com/deepzoom/2009",
              Url: src[i],
              Overlap: "1",
              TileSize: "256",
              Format : "png",
              Size:{
                 Height: "3635",
                 Width:  "13352"
                   }
                     }
					 }]
              });
		 }}
window.setInterval("change()",200)

so, the sequence mode does not work, is there other solution?

Sure... create the viewer just once, and then just change the page in change. Something like so:
var tileSouces = [];
var i;
for (i = 0; i < src.length; i++) {
  tileSources.push({
    Image: {
      xmlns:  "http://schemas.microsoft.com/deepzoom/2009",
      Url: src[i],
      Overlap: "1",
      TileSize: "256",
      Format : "png",
      Size:{
         Height: "3635",
         Width:  "13352"
      }
    }
  });
}

var viewer =  OpenSeadragon({
  id: "openSeadragon1", //the div taht will be showed
  prefixUrl: "./images/",
  sequenceMode: true,
  preserveViewport:true,
  tileSources: tileSources
});

i = 0;

function change(){
  i = (i + 1) % src.length;
  viewer.goToPage(i);
}

window.setInterval(change, 200);

cool! thx for your reply! it works! But, now i meet a new problem .
when
'window.setInterval(change, 200); ' works,  the picture always flashes with image switching.
I think the reason is once the function belows is called, the picture will appear with flashes.
OpenSeadragon({
id: "openSeadragon1", //the div taht will be showed
prefixUrl: "./images/",
sequenceMode: true,
preserveViewport:true,
tileSources: tileSources
});
viewer.goToPage(i);
actually, i want to play the DZi images continuously, just like play a video. but now the problem of flickering pictures bothers me，
how can i fix it?

To avoid the flickering, you'll have to use the multi-image API rather than sequence mode. Here's an example: https://codepen.io/iangilman/pen/PjdyGv

thx again! sorry one more question. when i run the code, the screen (actually the video) is a bit shaking and fuzzy, especially zooming into a small area. I think the reason is  the size (15499*4084) of  my image is very large, and  when i create the dzi image , i set the  TileSize: "256" . so the the period of loading tiled image is time-consuming , which causes the shaking problem.
Addtionally, because the shaking and fuzzy problem, i set the  setTimeout {function(),80)， the framerate is about 10fps, i want to increase the framerate to 30 fps, but the smaller of  value "setTimeout "  is , the more intense the video shakes.
Looking forward to your reply soon! thx!
var num=0;
var src=new Array();
for (num=0;num<100;num++){
src[num] =  "./DZI/"+num+"_files/";
}
var tileSources = [];
var i;
for (i = 0; i < src.length; i++) {
tileSources.push({
Image: {
xmlns:  "http://schemas.microsoft.com/deepzoom/2009",
Url: src[i],
Overlap: "1",
TileSize: "256",
Format : "png",
Size:{
Height: "4084",
Width:  "15499"
}
}
});
}
var viewer =  OpenSeadragon({
id: "openSeadragon1", //the div taht will be showed
prefixUrl: "./images/",
tileSources: tileSources[0]
});
index=1;
function addFrame(){
var tileSource=tileSources[index];
index++;
if(index>=tileSources.length){
index=0;}
viewer.addTiledImage({
tileSource:tileSource,
index:0,
success: function(event) {
var tiledImage = event.item;
  var handler = function(event) {
    if (event.tiledImage === tiledImage) {
      viewer.removeHandler('tile-drawn', handler);
	  setTimeout(function() {
        viewer.world.removeItem(viewer.world.getItemAt(1));
        addFrame();
      }, 80);
    }
  }
  viewer.addHandler('tile-drawn', handler);
   }

});
}
viewer.addHandler('open', function() {
addFrame();
});

Now, when opening the html page, the video is playing automatically. how can i add a 'start' button and a 'stop' button ?
i write like belows, it does not work.
play
........
function run(){
viewer.addHandler('open', function() {
addFrame();
});
}

Now, when i open the html page, the video is played  automatically. how can i add the 'start' button and 'stop' button to control the video playing?
I write the code like below, but it does not work.
play
......
function run(){
viewer.addHandler('open', function() {
addFrame();
});}

The shaking you're seeing is because the image hasn't fully loaded when you flip to it. To know when it's fully loaded, you'll have to use the fully-loaded-change event that's not yet released. You can get it by building from the master branch of this repository, or you can wait for the release (hopefully soon, see #1094).
To have a button start the animation, you'd just have the button call addFrame directly; don't put the viewer.addHandler in the button click handler.
To stop the animation, you'd want to set a flag, and addFrame can check that flag to see if it really should add another frame or not.

@iangilman   thx! It seems that you can release the  latest version this week. And this version will add fully-loaded-change event  which addresses my shaking problem, right? If the fully-loaded-change really work,  i have 200 frames, each frame is very large as i mentioned above, will the time of loading all the files  be very long?
Additionally, i try to use addTiledImage to preload all tile files ,  but it does not improve the slow loading phenomenon. It means i have to wait for a long time until all tiled images are loaded.
Here is the link:
Will the latest version be able to handle the two problems?
Looking forward to your reply soon! thx again.

Yes, we should revisit this after 2.3.0 is released.
That said, OSD is definitely not optimized for this kind of scenario. If you want to play video, you may be better off using a video format. Of course then you wouldn't get the zooming. Anyway, just be aware that you're on the cutting edge here.
I'll give some thought to pre-loading, but it may not be worth it. At any rate, let's see how far we can push it without having to pre-load.

Thanks a lot for your support!
Sorry, i am not good at I/O event. I will wait for  2.3.0 version and do some improvement based on this. By the way, where will the 2.3.0 be released  ?  in the"issues"' or somewhere else? i'm afraid i will miss the date of 2.3.0.

Sounds good. You can subscribe to https://github.com/openseadragon/openseadragon/issues/1094 for the release announcement. We'll also announce via Twitter:
https://twitter.com/openseadragon/

Sorry. Before 2.3.0 is released,  i think is there something i can do to improve the problems above?

when i run the code, the screen (actually the video) is a bit shaking and fuzzy, especially zooming into a small area.
http://cite.nju.edu.cn/testv9.html
If i understand #1193 correctly, the updated tiledimage.js loadingCoverage loaded  loading does not need to load too many files, so does that mean when i play multi-images continuously, the framerate will be higher? and the screen will be smoother when zooming into one area？

Not really, unless you want to build master from this repository. At any rate, we're expecting to release tomorrow.

Great!
Thanks for your reply！

@iangilman  How to make  fully-loaded-change event  work? I edit the code belows, but it does not work. And if i fully loaded the image, will it take up too much time so that the video is not continous?
Here is the demo
var viewer =  OpenSeadragon({	   id: "openSeadragon1",    prefixUrl: "./2.3.0/images/",   tileSources: tileSources[0], }); index=1; var flag=1; function addFrame(){ var tileSource=tileSources[index]; index++; if(index>=tileSources.length){ index=0;} if (flag){ viewer.addTiledImage({       tileSource:tileSource,       index:0,       success: function(event) {        viewer.addHandler('fully-loaded-change', function(){ 		  setTimeout(function() {             viewer.world.removeItem(viewer.world.getItemAt(1));             addFrame();                       }, 100); 	    })       }   });   }   }
Thx!

@iangilman How to make fullt-loaded-changework? I edit my code below,but it does not work.
And, if the image is fully loaded, will it take up too much time which leads to the discontinuous dsiplay?
Here is the demo(http://cite.nju.edu.cn/testv9.html)
The code:
var viewer =  OpenSeadragon({	  id: "openSeadragon1",   prefixUrl: "./2.3.0/images/",  tileSources: tileSources[0], }); index=1; var flag=1; function addFrame(){ var tileSource=tileSources[index]; index++; if(index>=tileSources.length){ index=0;} if (flag){ viewer.addTiledImage({       tileSource:tileSource,       index:0,       success: function(event) {        viewer.addHandler('fully-loaded-change', function(){ 		  setTimeout(function() {             viewer.world.removeItem(viewer.world.getItemAt(1));             addFrame();                       }, 100); 	    })       }   }); } }
Thx!

Here's my updated demo using fully-loaded-change:
https://codepen.io/iangilman/pen/PjdyGv
Let me know if that helps!

Cool! The flickering problem has been fixed now.  Even though the framerate can not reach video framerate because of the loading time, the result is still much better. Maybe it is the best result I can get.

Yes, it may be the best we can do. One thing to try would be preloading all of the images before showing the image in the first place (showing a progress bar instead or something), but then the user will have a long time to wait in the beginning.
Beyond that, we'd probably have to invent some sort of streaming protocol for OSD, which would require a specialized server.


Gradle project sync failed

Gradle project failed. basic functionality (e.g. editing, debugging) will not work properly.
my android studio is version 2.13

This issue really belongs under mythling instead of mythling-epg.
Anyway, please note the Environment Variables section under Build and Run in the wiki dev setup guide:
https://github.com/oakesville/mythling/wiki/Developer-Setup

Thank you,very much

You're quite welcome.

Don


VSCODE APP IS MIRRORING IN MULTIPLE SCREENS ON MACOS M1

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug
step1 - Open the multiple screens with multiple repos
step2 - Use those vscode screens by iterating them
Step3 - Close and open the app
step3 - Later some time you will find the vscode screen will mirror automatically in all screens, if you delete any one screen all the screens will delete automatically.
VS Code version: Code 1.67.1 (Universal) (da15b6fd3ef856477bf6f4fb29ba1b7af717770d, 2022-05-06T12:37:16.526Z)
OS version: Darwin arm64 21.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 Pro (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 2

Memory (System)
16.00GB (0.27GB free)

Process Argv
--crash-reporter-id 85557884-35ca-4fb0-81fc-d24012659d66

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

atlascode
atl
2.10.12

prettier-vscode
esb
9.5.0

minify
Hoo
0.4.3

vscode-docker
ms-
1.22.1

vscode-yaml
red
1.9.1

LiveServer
rit
5.7.5

rust
rus
0.7.8

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vstes627:30244334
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vscscmwlcmt:30465135
cppdebug:30492333
vscaac:30438847
pylanb8912cf:30529770
vsclangdf:30486550
c4g48928:30535728
hb751961:30541207
dsvsc012cf:30540253

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

Sorry for late reply, please refer this recording for better understanding
Recording 2022-12-09 at 19.58.00.gif
https://drive.google.com/file/d/1LLMWTI4YcmYOh03JSLFr_1_h52r0LCIJ/view?usp=drive_web
On Tue, Dec 6, 2022 at 4:57 PM VSCode Triage Bot @.***>
wrote:

Thanks for reporting this issue! Unfortunately, it's hard for us to
understand what issue you're seeing. Please help us out by providing a
screen recording showing exactly what isn't working as expected. While we
can work with most standard formats, .gif files are preferred as they are
displayed inline on GitHub. You may find https://gifcap.dev helpful as a
browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling
screencast mode for the recording (Developer: Toggle Screencast Mode in
the command palette). Lastly, please attach this file via the GitHub web
interface as emailed responses will strip files out from the issue.
Happy coding!
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/157589#issuecomment-1339180543,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIUPILUQEUBZ37J5QNTM7O3WL4PLTANCNFSM557ADCNA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey @deepak1556, this issue might need further attention.
@BalanaguYashwanth, you can help us out by closing this issue if the problem no longer exists, or adding more information.

Problem still exists facing issue, whoever assigned either @deepak1556 or other developer. Please resolve it

Hey @deepak1556, this issue might need further attention.
@BalanaguYashwanth, you can help us out by closing this issue if the problem no longer exists, or adding more information.

I am unable to repro with latest stable 1.84, please give it a try.

I tried it still facing it

Are you using any custom window manager ? Can you maybe share the steps with exact commands used.

I installed and opened the vscode
Same vscode screen is mirroring in different desktop screens (sub-screens) on mac
You can check this video for reference -
https://drive.google.com/file/d/1LLMWTI4YcmYOh03JSLFr_1_h52r0LCIJ/view?usp=drive_web

I am still unable to repro following the steps in the above recording.

Can we connect in online meet ? @deepak1556

I'm unable to reproduce the issue on the latest Insiders with macOS x64, 21.6.0.

Does the issue occur with VS Code Insiders?
If no to 1, does the issue occur when you launch VS Code from the terminal with
mkdir ~/some-empty-directory && code --user-data-dir=~/some-empty-directory?
If no to 2, can you try a clean uninstall of Code followed by a reinstall and see if that works?

Let me do clean uninstall

I cleaned it, uninstalled and installed again. Still issue is there
I also removed many vscode related files which you can refer these blogs -
https://stackoverflow.com/questions/42603103/how-to-completely-uninstall-vs-code-on-mac
https://code.visualstudio.com/docs/setup/uninstall#_clean-uninstall
Also i noticed when i installed right at the moment, then it is showing created yesturday. (which should expected created today and current time - 14 Dec - 9.55 IST )

Hey @rzhao271, this issue might need further attention.
@BalanaguYashwanth, you can help us out by closing this issue if the problem no longer exists, or adding more information.

Hello @rzhao271 @VSCodeTriageBot ,
Yes we can connect via Microsoft meet or google meet, So that i can explain more about my problem in my vscode. already tried many ways not resolving.

Hey @rzhao271, this issue might need further attention.
@BalanaguYashwanth, you can help us out by closing this issue if the problem no longer exists, or adding more information.

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