[release-1.2] fix: correctly check dry-run requests

What this PR does
Manual backport https://github.com/kubevirt/kubevirt/pull/12696.
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/lgtm
/hold
release-1.2 first https://github.com/kubevirt/kubevirt/pull/12747

/unhold


Java Client Poll scheduler not work as expected

Describe the bug
I have more than 100 workers in an application which poll task from conductor server every 10ms. Some tasks never polled for a long time. It seems ScheduledThreadPoolExecutor with 100+ threads can handle properly.
any suggestion to improve the polling performance?
@apanicker-nflx

@BrandonDotLin can you provide details about your client configuration? A code snippet of how you build TaskRunnerConfigurer would also help.


About 't1-extensive'

Will the section on 't1-extensive' be updated?
If so, when will it be updated?

Hi @czTLG,

Will the section on 't1-extensive' be updated?
If so, when will it be updated?

This repository will be archived soon. It's keep like this only for historical reasons.
If you want to get a more recent version, please visit clinicadl and/or clinica repositories, there are several tools to perform t1-extensive preprocessing.


dnd-zone with Vuetify v-data-table weird behaviour

Hi,
I would like to have a v-data-table as a draggable item. It's working almost OK, I have some issues with the click for page navigation. Depending on how or where exactly you click, dnd-zone react on the click preventing or not the page navigation to happened.
Here a short video to show what I'm talking about. Any idea on how to avoid this behaviour?
https://user-images.githubusercontent.com/40483301/132459150-071347f6-71ad-4ce1-91a4-4964d37558d6.mov
Thanks again for your work!

Looking into it

Well to avoid this i would suggest using a handle class (so only tapping the handle will trigger a drag process)

Hi @supraniti ,
Thanks a lot for your time and answer. Any chance you could take some more time to show me a code example please? I reckon I understood what you mean but not sure on how to achieve this.
Did I say thanks already? ^^

Add a prop named handle-class to the dnd-zone component
 .....
This will make only elements with a class named 'handle' draggable


Network Dashboard improvements and bug fixing

add explicit requirements for Python version at least 3.9
add constraint on Shared VPC service projects and host project to both be in the monitoring scope
fix issue with itertools.groupby which assumes the iterator to be already sorted when grouping by keys

The latter issue was causing duplicated metric to be written on Cloud Monitoring resulting in the following error:
One or more TimeSeries could not be written: One or more points were written more frequently than the maximum sampling period configured for the metric

@simonebruzzechesse I'm trying to target Python 3.10 for everything in the repo. Would that make sense for you?

@simonebruzzechesse I'm trying to target Python 3.10 for everything in the repo. Would that make sense for you?

Sure! Function code works with Python 3.10 (tested both locally and on CF), should I explicitly pin the required version in README or is it fine >= 3.10?


Name is not rendered correctly in DM

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary
Name is not rendered correctly in DM when I have received new message
Environment

Operating System: mac
Mattermost Desktop App version: 4.1.2 (<IP_ADDRESS>)
Mattermost Server version: 5.4.0

Steps to reproduce

ask someone send you private message using direct message

name before that

name after there is new message

name after you click to open the users messages

Expected behavior
Expected is that name should be rendered correctly. I have tested this in webbrowser as well, but the problem is only in mattermost-client. I have asked from other users as well and they can confirm that the problem exist only in client.
Observed behavior

name looks like this
Possible fixes

Comment from one of our team members:

I think I just saw this happen. It seems to happen in direct messages. It's applying the "no unread messages" style on top of the "there are unread messages" style.

We can confirm the same problem after server upgrade to v5.4  with the latest official macos app (4.1.2)

Hi all, we have a ticket open here to address this bug: https://mattermost.atlassian.net/browse/MM-12493.

Issue in client version 5.5 has been resolved for us. Thank you!

*With the 5.5 server/webapp.
At this time, the desktop app is numbered 4.2.0 :)


⚠ ClamAV has degraded performance

In ab2c475, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 554 ms

Resolved: ClamAV performance has improved in e240eb4.


Cannot build cuda container

Describe the bug

STEP 54/108: COPY containers/cuda/intel-oneapi.repo /etc/yum.repos.d/oneapi.repo
Error: building at STEP "COPY containers/cuda/intel-oneapi.repo /etc/yum.repos.d/oneapi.repo": checking on sources under "/var/tmp/libpod_builder4210657837/build": copier: stat: "/containers/cuda/intel-oneapi.repo": no such file or directory

Error when building
To Reproduce
Steps to reproduce the behavior:

Follow instructions on https://github.com/instructlab/instructlab/tree/main/containers
Run make cuda CONTAINER_PREFIX=quay.io/ai-labs/instructlab
Get above error

Expected behavior

Expect it to build it :)
Screenshots

N/A
Device Info (please complete the following information):

Hardware Specs: Mac M1
Python Version: N/A
InstructLab Version:  N/A

Using:
dev/ai/instructlab  main ✔                                                                                10h41m  ⍉
▶ podman version
Client:       Podman Engine
Version:      5.2.4
API Version:  5.2.4
Go Version:   go1.23.2
Git Commit:   76d0859d9f6e5d7c84392da9dcd2fae3447f789d
Built:        Mon Oct  7 12:04:55 2024
OS/Arch:      darwin/arm64

Server:       Podman Engine
Version:      5.2.5
API Version:  5.2.5
Go Version:   go1.22.7
Built:        Thu Oct 17 20:00:00 2024
OS/Arch:      linux/arm64

Additional context

@cdrage are you still hitting this error?
Seems like it should have been able to find this file locally: https://github.com/instructlab/instructlab/blob/main/containers/cuda/intel-oneapi.repo

@jaideepr97 https://github.com/instructlab/instructlab/pull/2581 fixes it, but it is not merged in yet

@jaideepr97 #2581 fixes it, but it is not merged in yet

ah got it, thanks


MWPW-132977: Support global navigation link group modifier 'blue'

Description
The bacom customer success stories have a "last link blue" feature which was previously achieved using section metadata. As we introduced the new global navigation, we also introduced a so called link group feature which acts and behaves like a block. The section metadata will have limitations in the future when adding second-last-link-red for example and wouldn't be very extensible. To have a more natural milo-style of authoring, we want to add modifiers.
Authoring experience
Allow authors to add modifiers to link groups

Output
Style dropdown items, with this PR allow links to become blue.

Resolves: MWPW-132977
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/osahin/gnav-1
After: https://mwpw-132977--milo--mokimo.hlx.page/drafts/osahin/gnav-1

Verified on https://mwpw-132977--milo--mokimo.hlx.page/drafts/sghimpos/gnav-1
Ready to be merged


feat!: improve breakpoints overlapping

Description
Make sure media breakpoints from config don't overlap (related to https://github.com/modulz/stitches/issues/901)
Split PR from https://github.com/traefik/faency/pull/232

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ Bofrost.at has degraded performance

In 9829f1b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2053 ms

Resolved: Bofrost.at performance has improved in 967193e after 18 minutes.


Multiple labeled elements malformed in Labeled Element(s) column in included template on Animal Details page

BUG DESCRIPTION
Problem
It appears that when there are multiple labeled elements (possibly also requiring multiple tracers with multiple labeled elements), the Labeled Element(s) column is incorrectly populated (contains a repeated element, once with a space and once without).
Steps to reproduce

Observed an animal that has a Lysine-[13C6,15N2] tracer, e.g.: https://tracebase.princeton.edu/DataRepo/compounds/446/

Current behavior
The labeled elements column displays as C,N, C.

Expected behavior
Should be either C, N or C,N.  Probably with a space is better.
Suggested Change
I tried following the code, but had difficulty tracking down the source of the problem, though I don't think it's a data problem because elements associated with that tracer/animal display correctly in the advanced search.
Comment

None

ISSUE OWNER SECTION
Assumptions

List of assumptions made WRT the code
E.g. We will assume input is correct (explaining why there is no validation)

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected Components

change: File path or DB table ...
add: Environment variable or server setting
delete: External executable or cron job

Requirements

[ ] 1. List of numbered conditions to be met for the feature
[ ] 2. E.g. Every column/row must display a value, i.e. cannot be empty
[ ] 3. Numbers for reference & checkboxes for progress tracking

DESIGN
GUI Change description

None provided
Code Change Description

None provided
Tests

[ ] 1. A description of at least one test for each requirement above.
[ ] 2. E.g. Test for req 2 that there's an exception when display value is ''
[ ] 3. Numbers for reference & checkboxes for progress tracking

I suspect this occurs when multiple tracers are included in a single infusate, and one tracer is dual labeled (ie. 13C, 15N) and a second tracer is single labeled (ie 13C).  This then appears in the table as C,N, C.
I'm imagining that if we had an infusate like:

[13C-15N]-lysine
[13C]-valine
[2H-13C]-glucose
the table would display C,N, C, H,C

The purpose of this labeled element(s) column is to indicate what forms of labeling are possible in the measured compounds.  So, in the original example, it would only show C, N.  In the theoretical example, it would show C, N, H.

I suspect this occurs when multiple tracers are included in a single infusate, and one tracer is dual labeled (ie. 13C, 15N) and a second tracer is single labeled (ie 13C). This then appears in the table as C,N, C.
I'm imagining that if we had an infusate like:

[13C-15N]-lysine
[13C]-valine
[2H-13C]-glucose
the table would display C,N, C, H,C

The purpose of this labeled element(s) column is to indicate what forms of labeling are possible in the measured compounds. So, in the original example, it would only show C, N. In the theoretical example, it would show C, N, H.

This issue is documented and I believe it's in the beta 3 release milestone (either it is or we agreed to move it there). I did a little shallow digging into it awhile back. I couldn't pin it down exactly, but I confirmed that the data in the database is correct. I suspect the issue may be located in the dataframes code, but I can't say for sure.

I figured out what's going on here.  This happens when an infusate has multiple tracers and not all tracers have the same labeled elements.  You end up with a string of elements where the space-separated ones are the unique combination of elements for 1 or more tracers with that complement of elements.  So in the example above:

[13C-15N]-lysine
[13C]-valine
[2H-13C]-glucose

lysine results in C,N
valine results in C
glucose results in H,C
...and they're joined together with a comma-space.
Not sure yet how to fix it.
This is happening because there was an assumption made that all the tracers would have all the same labeled elements (or it was overlooked that it could be different).  The combination of labeled elements in each tracer is made into an ordered comma-delimited string, then unique is called on those strings, so if we had:

[13C-15N]-lysine
[13C]-valine
[2H-13C]-glucose
[13C]-glutamine

we would still end up with C,N, C, H,C.

@mneinast - Just was speaking with Fan about this issue in the meeting this morning and realized I might have misunderstood what the intention was for the Labeled Elements column. I had worked on this issue to "fix" it so that something like C,N, C, H,C would instead appear as C, H, N. But Fan pointed out that the purpose of displaying it as C,N, C, H,C was intentional: to represent that combinations of elements that exist among the tracers.
I had learned during my "fix" that that's why it displayed the way it did.  So I guess my question is (and I'll ask you about it when we meeting later): do you want the labeled element combinations that exist among the tracers, or a unique list of labels?  If you want the combinations, I can just close my PR without merging, though we might consider addressing the header name to somehow explain the meaning of the data in that column, because it wasn't clear to me why it seemed to show repeated elements.


Switch from __resource.lua to fxmanifest.lua

Hoping this one goes better than my last pull request

Yikes that was a bit harsh lol

Lmao, no toxicity here. Approved.

I have a few things I would like to add, then it will be included in the next release.


Gpairs numba_dpex_k fails validation with preset = "M"

With the larger problem size of "M" the gpairs implementation using numba_dpex_k no longer passes validation.

Validation failure does not occur with preset='M' (see log below). Closing issue.
================ Benchmark GPairs ========================
================ implementation numba_dpex_k ========================
implementation: numba_dpex_k
framework: numba_dpex
framework version: 0.20.1+54.g43920fb3
setup time: 191.236737ms (191236737 ns)
warmup time: 981.360405ms (981360405 ns)
teardown time: 185.307μs (185307 ns)
max execution times: 469.136166ms (469136166 ns)
min execution times: 469.136166ms (469136166 ns)
median execution times: 469.136166ms (469136166 ns)
repeats: 1
preset: M
validated: Success


Reuse fibers when possible

At the moment, every handler spawns a new fiber. Although it's the right way of doing things from the theoretical POV, we could use our own stack for effects that don't manipulate the control flow.

Turned out it's always possible since now every effect provider has a stack frame.


Invoke Lambda MissingRegion: could not find region configuration

I'm submitting a ...  (check one with "x")
[X] bug report => search github for a similar issue or PR before submitting
[] feature request
[] support request
[] general question

Current behavior (how does the issue manifest): If the payload of the Invoke Lambda activity is mapped as an object, all items that appear in the mapping array after the payload are ignored.
Expected behavior: No mappings are ignored
Minimal steps to reproduce the problem (not required if feature enhancement):

Create a new flow with the Invoke Lambda activity
Set the payload to an object

Sample
{
    "type": "object",
    "value": {
        "Greeting": "Hello World"
    },
    "mapTo": "payload"
},
{
    "type": "literal",
    "value": "us-west-2",
    "mapTo": "region"
}

In this case the literal mapping for region is not passed into the engine. If you switch the order (have region appear before payload) it works perfectly fine
What is the motivation / use case for changing the behavior?
Please tell us about your environment (Operating system, docker version, browser & web ui version, etc): Windows 10
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
flogo version
flogo cli version [v0.5.4]
Your project uses github.com/TIBCOSoftware/flogo-contrib version v0.5.4
Your project uses github.com/TIBCOSoftware/flogo-lib version v0.5.4
Your project uses github.com/retgits/flogo-components branch master and revision 6c85d7d9c68d9ad58550bf4778f7a3bb2eb70ac1

Additional information you deem important (e.g. issue happens only occasionally):

@retgits can you test this with the master branch? This commit should have resolved the problem: https://github.com/TIBCOSoftware/flogo-lib/commit/78a727e6a8b3bccec34ce56338b1c30d2c653b6b

Yup, that does indeed seem to work! Closing this issue


Enable snap sync

https://github.com/ethereum-optimism/optimism/pull/6243#issuecomment-1784396213
Base nodes would be able to sync much quicker with snap sync but seems like it is disabled by default https://github.com/base-org/node/blob/main/geth-entrypoint#L62.
Also, it should have bootnodes for op-geth to connect each other.

Snap sync has yet to be sufficiently tested for op chains which is why it's disabled for now. Would be good to get it tested and activated however.

@roberto-bayardo Could you enable --state.scheme=path option for op-geth? It will reduce disk usage by more than 50% with my benchmarks

looking forward to enabling path-based state db as well when it's more battle tested. geth team recommends leaving it disabled for lower risk profile for the time being.

@kaliubuntu0206 Are there any updates about full mode snapshot or snap sync?


Add Private RSA and DSA keys

Encryption keys take specific and regular format. We can scrape them out of binaries too, and this might help with encryption method analysis.

Made part of the integrate with Yara project.


Standalone God mode

Standalone OpenWhisk now supports quite a few optional features which can be enabled by  various options (--kafka, --api-gw, --counchdb, --user-events) etc.
This PR introduces a --all mode which would enable all the above features via single argument and launch all optional services (Kafka, Prometheus, Redis etc)
Description
Run all supported OpenWhisk Standalone features
java -jar openwhisk-standalone.jar --all

Currently this mode would launching following services (in 7 docker containers)
===============================================================
Launched service details

[ 3233  ] http://localhost:3233 (Controller)
[ 6379  ] http://localhost:6379 (whisk-redis)
[ 9000  ] http://localhost:9000 (whisk-apigw, Api Gateway - Api Service )
[ 3234  ] http://localhost:3234 (whisk-apigw, Api Gateway - Management Service)
[ 5984  ] http://localhost:5984 (whisk-couch)
[ 9092  ] localhost:9092 (kafka)
[ 9091  ] <IP_ADDRESS>:9091 (kafka-docker)
[ 2181  ] Zookeeper (zookeeper)
[ 64231 ] http://localhost:64231 (whisk-kafka-drop-ui)
[ 3235  ] http://localhost:3235 (whisk-user-events)
[ 9090  ] http://localhost:9090 (whisk-prometheus)
[ 3000  ] http://localhost:3000 (whisk-grafana)

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4670 into master will decrease coverage by 4.65%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #4670      +/-   ##
=========================================
- Coverage   83.26%   78.6%   -4.66%
=========================================
  Files         192     192
  Lines        8686    8686
  Branches      599     599
=========================================
- Hits         7232    6828     -404
- Misses       1454    1858     +404

Impacted Files
Coverage Δ

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-96.17%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0% <0%> (-94.74%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedListener.scala
0% <0%> (-86.67%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0% <0%> (-76.48%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-74.08%)
:arrow_down:

...a/org/apache/openwhisk/common/ExecutorCloser.scala
0% <0%> (-66.67%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
7.69% <0%> (-50%)
:arrow_down:

... and 31 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 5667fb0...4399790. Read the comment docs.


TF 2.0 Documents how BatchNormalization update is handle in TF 2.0

When training model that need specific update such as batch norm mean and variance, it is not clear
from the current version of documentations how we can handle this.
This docs https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/keras/Model suggests that we have to call model.updates
While here https://github.com/tensorflow/docs/blob/master/site/en/r2/tutorials/generative/dcgan.ipynb there's nothing about model.updates
System information

TensorFlow version: 2.0-preview

model.updates is not in TF2 now. Thanks


Update README.md

add information about A11ygator being only available as Google Chrome browser extension.

@puresick thanks for pointing out!


⚠ Bofrost.de has degraded performance

In 9143a7d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1328 ms

Resolved: Bofrost.de performance has improved in 67ad4c6.


Orphaned space-views when resetting blueprint

When resetting a space-view blueprint, such as for plots, we end up with 10 space-views in the blueprint timeline.
seems like a bunch get created on the first frame, then lost and replaced with others that have the same content.
easy to repro by running the plot example and resetting the blueprint

@Wumpf wondering if this is somehow related to the new hash tracking logic


outputs-pull / push

Would it make sense to create an outputs-pull / outputs-push command in the Makefile along the lines of what we already have for inputs? This would be useful if someone wants to work with processed outputs instead of implementing the whole pipeline.
(or perhaps this use-case is addressed via metaflow)

It is addressed and needs to be addressed via. Metaflow (or a similar versioned DAG solution) because...
outputs/ change when we run code (either because the code changes as we iterate on it or sometimes because the code is non-deterministic e.g. running a clustering algorithm on multiple cores) then we can't just push/pull to S3 because if I run some code and push results to S3 then that might overwrite results you just ran and pushed.
In the past we've basically just dealt with this by only storing data/models in S3 for a project that took a long time to run.
This isn't ideal because:

You either end up overwriting other/old results like I mentioned above OR you end up creating an ad-hoc (bad) versioning system
Most of your models/data are still local only

But Metaflow pipelines automatically version our data/models in S3 and allow them to be easily fetched.
N.B. Metaflow  does lots of other things for us too, e.g. easily scaling up compute, but we're just talking about outputs/ here.


fuzzel: 1.3.0 → 1.4.0; fcft: 0.4.3 → 2.2.5

Motivation for this change
Update fuzzel to the new release which fixes a crash and adds new features. Also update dependency fcft which is now required in a higher version.
Changelog
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] 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.

@GrahamcOfBorg build fuzzel

Updated the PR to update to 1.4.1 which also includes a minor (typo) fix.

Updated the PR to update fcft to 2.2.6 which fixes font rendering issues with NixOS's default freetype package if subpixel rendering was used, e. g. fuzzel was started with an completely opaque background: fuzzel -b 000000ff -t ffffffff -s a0a0a0ff.


The experimental results

after complete the training and save the model,I did not find the completion result, how should I see the completion result.thank you

Please check the results in the folder output/benchmark.


iOS: switch back to app: feed view changed, all unread are now read

Hey there. I’m on iPad (Pro 9.7”), latest iOS, NNW 6.0 (606).

Browse feeds from ‘All Unread’. Pick article, read article.
Get curious about Teterboro Airport, head over to Safari & Maps for a minute or so.
Switch back to NNW.
Am now I’m in the list for a specific feed – not the one whose article I was reading – and all of my unreads are now marked read.

Are you able to recreate this consistently or did it just happen one time?

Just the once, alas. Doesn’t look reproducible.

Oh sorry, I forgot this was open. GitHub should send you some sort of weekly digest of all of your old issues.
No, never again, all good. Thanks!


[admin] Publish

Docs Build status updates of commit 9005a71:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/add-in-development-best-practices.md
:white_check_mark:Succeeded
View

docs/design/contextual-tabs.md
:white_check_mark:Succeeded
View

docs/design/fabric-core.md
:white_check_mark:Succeeded
View

docs/design/keyboard-shortcuts.md
:white_check_mark:Succeeded
View

docs/design/using-office-ui-fabric-react.md
:white_check_mark:Succeeded
View

docs/develop/auth-with-office-dialog-api.md
:white_check_mark:Succeeded
View

docs/develop/authentication-pnp.md
:white_check_mark:Succeeded
View

docs/develop/authorize-to-microsoft-graph-without-sso.md
:white_check_mark:Succeeded
View

docs/develop/create-addin-commands.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-nodejs.md
:white_check_mark:Succeeded
View

docs/develop/dialog-api-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/develop/overview-authn-authz.md
:white_check_mark:Succeeded
View

docs/develop/sso-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/develop/support-ie-11.md
:white_check_mark:Succeeded
View

docs/develop/troubleshoot-sso-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/develop/update-your-javascript-api-for-office-and-manifest-schema-version.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-authentication.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-batching.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-debugging.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-overview.md
:white_check_mark:Succeeded
View

docs/excel/excel-data-types-concepts.md
:white_check_mark:Succeeded
View

docs/excel/excel-data-types-overview.md
:white_check_mark:Succeeded
View

docs/includes/alert-use-https.md
:white_check_mark:Succeeded
View

docs/includes/quickstart-yo-start-server-excel.md
:white_check_mark:Succeeded
View

docs/includes/start-web-sideload-instructions.md
:white_check_mark:Succeeded
View

docs/includes/tutorial-excel-start-server.md
:white_check_mark:Succeeded
View

docs/includes/tutorial-word-start-server.md
:white_check_mark:Succeeded
View

docs/includes/using-preview-apis-host.md
:white_check_mark:Succeeded
View

docs/includes/using-preview-apis.md
:white_check_mark:Succeeded
View

docs/outlook/apis.md
:white_check_mark:Succeeded
View

docs/outlook/authenticate-a-user-with-an-sso-token.md
:white_check_mark:Succeeded
View

docs/outlook/authentication.md
:white_check_mark:Succeeded
View

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

docs/outlook/extract-entity-strings-from-an-item.md
:white_check_mark:Succeeded
View

docs/outlook/get-attachments-of-an-outlook-item.md
:white_check_mark:Succeeded
View

docs/outlook/implement-sso-in-outlook-add-in.md
:white_check_mark:Succeeded
View

docs/overview/explore-with-script-lab.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-code-samples.md
:white_check_mark:Succeeded
View

docs/quickstarts/excel-custom-functions-quickstart.md
:white_check_mark:Succeeded
View

docs/quickstarts/onenote-quickstart.md
:white_check_mark:Succeeded
View

docs/quickstarts/outlook-quickstart.md
:white_check_mark:Succeeded
View

docs/quickstarts/powerpoint-quickstart.md
:white_check_mark:Succeeded
View

docs/quickstarts/project-quickstart.md
:white_check_mark:Succeeded
View

docs/quickstarts/sso-quickstart-customize.md
:white_check_mark:Succeeded
View

docs/quickstarts/sso-quickstart.md
:white_check_mark:Succeeded
View

docs/quickstarts/word-quickstart.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/excel-api-requirement-sets.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/word-api-requirement-sets.md
:white_check_mark:Succeeded
View

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


Undefined global function. on rocFFT complex 2d FFT example\n

What is the expected behavior

to compute complex FFT

What actually happens

terminate called after throwing an instance of 'std::runtime_error'
what():  Undefined global function.

How to reproduce

Environment

Hardware
description

GPU
device string

Radeon 580

Name:                    gfx803

Marketing Name:          Ellesmere [Radeon RX 470/480/570/570X/580/580X/590]

| CPU | device string |
Intel(R) Core(TM) i7-5820K CPU @ 3.30GHz
| Software | version |
5.5.13-arch2-1
| ROCK | v0.0 |
3.3.0
| ROCR | v0.0 |
3.3.0
| HCC | v0.0 |
3.3.0
| Library | v0.0 |
3.3.0

I used test.sh in the repository and most of them works. Didn't saw any runtime errors, just hotspot freeze the kde render. Even if i add some kernel with global tag like vector_add in the example works.

@gmaleshkov
I assume when you said "test.sh" it is test_all.sh in HIP example. It looks your env should be ok. As I don't have gfx 803 on hand, I could not try to reproduce your problem.
Still more questions might help to detect the problem:

Could you build rocFFT library itself?
How do you build the 2d example? Is this problem specific to 2d case?
Dou you have chance to try older ROCm stack? or the build has no prblem on the old ROCm release?

@feizheng10
Library is compiled without any issues: warnings and errors. Strange is that, if i compile it like static all works and gives right results but when is compiled as dynamic - shows the error. I tried multiple examples even this one the repository when is dynamic trows runtime error and undefined global function when is static works correctly.

First, good to hear that no problem to build the library.
From the build config, it looks like you only enabled BUILD_CLIENTS_RIDER to build.
So, I would guess the 2D example code you mentioned is not the sample code we provide in the repo. If in this case, could you try to build -DBUILD_CLIENTS_SAMPLES=ON first?

Sorry for the delay, but apparently i can't build samples with cmake:
Make Error at clients/samples/CMakeLists.txt:38 (add_subdirectory):
The source directory
/home/*****/.cache/yay/rocfft/src/rocFFT-rocm-3.3/clients/samples/CMakeFiles

does not contain a CMakeLists.txt file.
Tests and selftest works only samples gives this error.

You should be able to clean your cache and print samples_subdirs after https://github.com/ROCmSoftwarePlatform/rocFFT/blob/af3d56120329332315670e7d4c3de7ece7e4169c/clients/samples/CMakeLists.txt#L35

We haven't had any cmake issues for quite a while; I'm closing this, but please re-open if this is still an issue.


Update Zebra for aes 0.7

Is your feature request related to a problem? Please describe.
When dependabot tries to update Zebra to aes 0.7.1, the build fails with the following error:
error[E0599]: the function or associated item `new` exists for struct `fpe::ff1::FF1<aes::Aes256>`, but its trait bounds were not satisfied
   --> zebra-chain/src/orchard/keys.rs:50:29
    |
50  |     let ff = FF1::<Aes256>::new(&K, radix).expect("valid radix");
    |                             ^^^ function or associated item cannot be called on `fpe::ff1::FF1<aes::Aes256>` due to unsatisfied trait bounds
    |
   ::: /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/aes-0.7.1/src/autodetect.rs:127:1
    |
127 | define_aes_impl!(Aes256, aes256, U32, "AES-256 block cipher instance");
    | -----------------------------------------------------------------------
    | |
    | doesn't satisfy `aes::Aes256: cipher::block::BlockCipher`
    | doesn't satisfy `aes::Aes256: cipher::block::NewBlockCipher`
    |
    = note: the following trait bounds were not satisfied:
            `aes::Aes256: cipher::block::NewBlockCipher`
            `aes::Aes256: cipher::block::BlockCipher`

https://github.com/ZcashFoundation/zebra/pull/2128/checks?check_run_id=2545047975
Describe the solution you'd like
We should update Zebra so it handles these new traits correctly.
Describe alternatives you've considered
We can stay on aes 0.6.
Additional context

Duplicate of #2952.


decimal part of AI2!

Hi.
Actually is there any way to remove decimal part of shape code
example (417.78) I want it without "." like this "417"
so is it possible ?
thank you.

Making that change would require modifying the code, but there's more to it than that.
The problem is a truncation or even rounding policy for ensuring the output coordinates are integers loses a lot of precision, which ends up making the shapes look wrong, either subtly or obviously.
Decimal shape coordinates have been supported in xy-vsfilter and libass since, I think, 2013 or so. I don't see a reason to support this use case.
Why do you need to remove the decimal part of the coordinates?

Actually AssDraw not supported decimal part of code, so I was looking for something or If can modify the code (script), maybe there's lua script can do that, something like
round = function( Shape )
Shape = Shape:gsub( "%-?%d+[%.%d]*",
function( number )
number = math.ceil( number )
return number
end
)
return Shape
end

or In java
DecimalFormat df = new DecimalFormat("##0");
df.format((Math.round(doubleValue * 100.0) / 100.0));

I don't prefer decimal part of shape code, it's increase the size of ass file a little bit =)
thanx

Shape:gsub("(%-?%d+)%.?%d*", "%1")
No rounding but what you want ("remove decimal part") in a fast way.
Instead of math.ceil maybe you wanted math.floor(number + 0.5) for rounding?

Yes, I want to remove decimal part.

I have no interest in supporting ASSDraw, an ancient and really janky tool. You're welcome to write whatever scripts you want to perform whatever conversions you want outside of AI2ASS.

Amen.


remove HttpData from all public APIs.

This isn't required. Infact HttpData is a very powerful low level primitive which can be used to serve content quickly and efficiently.


add dynamic index naming bug description

cc @jackkoenig @mwachs5

Sorry I realized that we missed the thing that would make this actually show up on the "table of contents" on the Cookbook page. I didn't realize that was a manual process


Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[ ] iOS

Feature Library

[ ] HTTP Transfers

Version of Library
Version of OS(s) listed above with issue
XCode 11.3
Steps to Reproduce

Create an api to upload images
Add many images into the Http Transfer Manager queue
Observe , say out of 5 uploads 3 or more will be duplicate

Expected Behavior

While uploading 5 images, it should be 5 different images, not duplicate entries expected

Actual Behavior

Out of a number of images being uploaded, one or two images are getting duplicated. Doubting the thread safety of Http Transfer.

I have no idea what you mean here.  Can you please upload some sort of sample.

Following code sample shows the scenario where a number of http transfer are created  in a loop. End result is the file upload creates duplicate entries at server side, not distinct ones as per the loop.
foreach (var file in files)
{
Dictionary<string, string> hdr = new Dictionary<string, string>();
hdr.Add("x", x);
hdr.Add("xx", xx);
                                HttpTransferRequest request = new HttpTransferRequest("https://xxxxxxxxxx.net/api/xxx/uploader", file.Path, true)
                                {
                                    HttpMethod = System.Net.Http.HttpMethod.Post,
                                    Headers = hdr,

                                };
                                HttpTransfer tranfer = await _httpTransferManager.Enqueue(request);
                            }

I did not reproduce this.  I see all of the uploads going in just fine.  I reinspected the upload code and don't see how what you are saying is even possible since they all use different requests too.


Giving error

Sometime when people try and place a book it give this error
http://pastebin.com/R5UpkpPB

That's not an error with CE, it's somethign else.

Is there anymore to that error?

Yes, Im sorry. I thought I got it all.
http://pastebin.com/ArfR2fzy

Hmmm, that error still doesn't show me anything. Do you know when the error happens?

They where trying to place a book from CE on to a sword. and it won't let them

Could you pastebin your enchantments.yml.

http://pastebin.com/bg1f6QVV

Hmmmm, it would help if the error would show me more. Can you check for a line in the errors for something that says "me.badbones69.BlaBlaBla"

Ok , So I found what cause it
In the CustomeEnchantments.yml I have a  for the Name Titan:Cyborg
it looks like the colon in the name causes the error.
However in the Enchantments.yml this isn't a problem
I have to have the colon for your plugin to work with my slimefun
can you fix?


Cleaned and organized the Profinet PCAPs folder.

DCP messages run above RT, all the RT packets in these captures are DCP, so I'm merging them and deleting a s7comm pcap that was in the Profinet dir.

Thanks!


User location control button for map view

Currently there's no easy way recenter the map view back to the users current location. Add a "user location" control component to the map view overlay that on click sets the map view back to the user's location.
Suggested location: right side, about 1/3 away from the bottom of page or right about the attribution section of the map (bottom right).

Hey I'd like to give this a shot.


Add missing id attributes

This PR sets the @id attribute for all sources, filters and outputs that currently do not specify it.
This change should prevent high cardinality data in Prometheus by avoiding object:<random-id> values in the plugin_id label, as well as make the fluentd logs themselves more readable.

Looks good. Thanks!


Create insert node at end in a doubly linked list

this is code for inserting a node at the end in the doubly linked list
please accept my code for hacktoberfest

Please contribute to the right repository!


Fix Event field name ("name" -> "path")

Without this, when using Secret.changed_event and similar properties, a TypeError is raised:
>       return Event(name="secret_changed", secret=self)
E       TypeError: Event.__init__() got an unexpected keyword argument 'name'

Probably means there should be tests of these code paths too, but I'm too lazy to add that here -- just flagging the issue. :-)

Oops, sorry, I'd forgotten about those fixes (and that they were on a different branch). Closing.


depends only fog-core

Just backlog.

@sawanoboly Could you do this asap? And open issues at fog or fog-core for you issues so we can all track it down?

@plribeiro3000
Maybe, does this issue is related to https://github.com/fog/fog/issues/3347?
I'll try to clear all problems for this issue. It'll takes about 1 or 2 weeks or so, Is that all right with you?

@sawanoboly Awesome. Thank you for the fast answer and work.
This is indeed related to fog/fog#3347. We need to make the providers work in a manner they won't break each other. Since fog depends on all fog-providers, having a provider depending on fog might/will cause problems.
Take your time to fix it and let me know if you need any help with it.

@sawanoboly I guess we can close this now, right?

@plribeiro3000
Yes, the fog-sakuracloud no longer depends on the fog! only depends on the fog-core.
We can remove fog-sakuracloud from fog dependencies.
And I've released as v1.0.0 https://rubygems.org/gems/fog-sakuracloud just now.

@sawanoboly :clap:
Fog should depend on fog-sakuracloud. Just fog-sakuracloud tha should not depend on fog. =)


missing verto.js

can you share where to get this file?

That's now available in https://github.com/freeswitch/verto-client/blob/master/js/src/jquery.verto.js - does it help you @daniel-lucio ? Best.

That's now available in https://github.com/freeswitch/verto-client/blob/master/js/src/jquery.verto.js - does it help you @daniel-lucio ? Best.

That's now available in https://github.com/freeswitch/verto-client/blob/master/js/src/jquery.verto.js - does it help you @daniel-lucio ? Best.

Hi Giacomo, Thanks for you hard work, the version is for HTML, JavaScript, not usable from NodeJS, do you know if someone ported it as NodeJS module ? or I need to write it from ground ?
Best
Alessandro

Any update on this?

Any update on this?  Does https://libraries.io/npm/vertojs work?

I don't have any update, sorry; my focus was on the interaction on the server side and the client was only a PoC.
The verto protocol is simple, and there is an excellent guide here: https://evoluxbr.github.io/verto-docs/
If anyone develops a version for nodejs please share it here. Thanks.


DOC: Add example for haroldpolydiv

Added an example for the haroldpolydiv function. With and without a
remainder.

Nice! Yesterday I've enabled the intersphinx extension so now you can (hopefully) write
 :func:`scipy.signal.deconvolve`

to link to the SciPy documentation automatically which I apparently forgot 😃

Example here https://harold.readthedocs.io/en/latest/generated/harold.lqr.html#harold.lqr for the rendering in the notes section

Ok, I'll bring this in too :+1: My lectures start again next week so I'll probably be able to use this library in my "Signals and Systems" class :smile: Maybe I can contribute some more examples after that :yum:
And I have a question regarding haroldpolydiv and haroldpolymult. polydiv uses the scipy method to deconvolve and polymult uses the numpy method is there a reason for that? Or is there even a difference? It's just a bit weird because there is also a scipy method for convolving.

Ok, I'll bring this in too 👍 My lectures start again next week so I'll probably be able to use this library in my "Signals and Systems" class 😄 Maybe I can contribute some more examples after that 😋

That would be great to have first hand feedback. Good luck with your studies!

And I have a question regarding haroldpolydiv and haroldpolymult. polydiv uses the scipy method to deconvolve and polymult uses the numpy method is there a reason for that? Or is there even a difference? It's just a bit weird because there is also a scipy method for convolving.

Not really a big difference but SciPy convolve has a bit of an extra overhead since that is a more general function. And since this is used a lot in the MIMO transfer functions, I've taken NumPy's version. Also historically I started to use NumPy poly1d objects which was not accepted by SciPy version but then I've stopped and resorted back to plain arrays.


Using ".dependencies." infix as a part of property

Due to cloud-zookeeper uses ".dependencies." in some of properties e.g. spring.cloud.zookeeper.dependencies.ribbon.enabled during auto-configuration when property is set org.springframework.cloud.zookeeper.discovery.dependency.ZookeeperDependencies field dependecies is polluted by w this property. That may lead to wrong behaviors.

Fixed in https://github.com/cameleeck/spring-cloud-zookeeper/commit/84b3f5d32345a2bb385b8cb191b6910547a97dcf


Fix soft delete feature on linux by using cross-platform NewLine

Found another bug in the soft delete feature.
The newline \r\n was hardcoded in the code.
It is problematic as it's platform dependent so it currently only works on Windows.
Replaced it with Environment.Newline to fix that.

Thx for contrib.


Workspaces does not capture "LibreOffice Calc" Windows

Microsoft PowerToys version
0.84.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
Workspaces
Steps to reproduce
I have 3 windows of LibreOffice Calc open on monitor 1, and 2 windows of Vivaldi on monitor 2.
✔ Expected Behavior
Workspaces only saved the Vivaldi windows.
❌ Actual Behavior
Workspaces saved the Vivaldi windows but not the LibreOffice Calc windows. I had expected the ability to add applications to a workspace but I could not find that function.
Other Software
No response

@WeirdBeardDev
Hi!
I've tested the issue with our latest PT version, 0.84.1, which is a hotfix released recently.
On my system the LibreOffice Calc app has been captured and also launched.
Could you please verify if the issue has been solved on your system with the new hotfix?
Thanks!

Updated to PT v0.84.1 and Workspaces does capture LibreOffice Calc, however, it says it's in "(Safe Mode)" and it isn't and the icon isn't correct.

I have a similar issue, but instead of LibreOffice (Safe Mode), any LibreOffice apps I open are recognized as LibreOffice Base. My PowerToys version is the latest stable release, v0.87.1.


#train

Hello, when I was executing the training steps of this code and running the command(I trained on 7 scen):
sh train_nefes. sh
I encountered an error:
No such file or directory: '/ logs/nefes_stairs_color_only/000600.tar'，
Where did you obtain this file? thank you

Hi, if you want to train the second stage model in the train_nefes.sh, you just need to replace the --ft_path argument with your pretrained first stage color only checkpoint.
The first stage training script is this:
python run_nefes.py --config config/7Scenes/dfnet/config_stairs_stage1.txt
And after training, it will save the model at logs/nefes_stairs_color_only/000600.tar
Then you could lunch the second stage training:
python run_nefes.py --config config/7Scenes/dfnet/config_stairs_stage2.txt


Doesn't it work with laravel 5?

Hi,
Sorry i don't know, i don't tested laravel 5.
If you do a Pull Request for laravel 5, i will merge it


HTW weather data for pvlib simulation #64

Add code to use HTW weather data in the simulation. Results are too high, the problem might lie with pvlib.irradiance.erbs or time series resampling.
Using g_hor_cmp6 instead of g_hor_si did not fix the problem

Please delete branch if not needed any more.


Update scala-library to 2.12.15

Updates org.scala-lang:scala-library from 2.12.14 to 2.12.15.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.12.14).
You might want to review and update them manually.
.github/workflows/ci.yml
docs/changelog.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

labels: library-update, semver-patch, old-version-remains

Superseded by #357.


62d33efa-c71c-46de-a5a1-bc56ca9b91be

this is for automated testing

5e0f1697-1dc4-4ce9-9512-d13dcb7c2042


Refine README.

Codecov Report

Merging #10 (fea7cb0) into main (cf11beb) will decrease coverage by 2.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   96.96%   94.84%   -2.13%
==========================================
  Files           6        6
  Lines         330      330
  Branches       44       44
==========================================
- Hits          320      313       -7
- Misses          7       14       +7
  Partials        3        3

Impacted Files
Coverage Δ

mkkey/cli.py
89.51% <0.00%> (-4.90%)
:arrow_down:

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 cf11beb...fea7cb0. Read the comment docs.


Removing unused CSS and normalizing indents in docs/conf.py.

@stephenplusplus Can you (or another from node side) confirm what github.min.css was used for?

@stephenplusplus or @ryanseys Can you confirm what github.min.css was used for? (Our template was forked from yours by @silvolu)

Looks to me like it was used for syntax highlighting in our docs. I'm assuming it followed the color scheme of GitHub syntax highlighting, hence the name github.min.css.  I didn't actually add the file so I'm not sure whether it was ever used / to what extent it was used.

More context: https://github.com/pc035860/angular-highlightjs/blame/master/README.md#L23

Sorry for letting this one slip off my radar @dhermes. Ryan's right, it sets the theme of the highlight.js code blocks to match GitHub's color scheme. If you're not using highlightjs in your docs, then it's safe to remove.

No worries. Thanks for the info!
@tseaver WDYT?

LGTM


Buffer added immediately after input buffer

Describe the bug
A buffer is added immediately after an input buffer for a signal that goes through (not registered) the macro below. It is as if the buffer after input is added without regards to the existing buffer.
To reproduce, run make DESIGN_CONFIG=designs/asap7/mock-array/Element/config.mk with https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/1794
To view the problem, run:

untar  extra-buffer.tar.gz
Run ./run-me-mock-array_Element-asap7-base.sh
Examine the timing path for endpoint io_lsbOuts_6
Observe the back-to-back buffers on the input

The buffer is inserted during synthesis. Commenting out insbuf removes the middle buffer, but also the input & output buffers.
https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/blob/b6f7d318f0eb186069acd9af85898e7dcfa64d3a/flow/scripts/synth.tcl#L78

Expected Behavior
Remove the superflous buffer
Environment
OpenROAD v2.0-12143-g4aabf0d3c

To Reproduce
See above
Relevant log output
No response
Screenshots
No response
Additional Context
No response

The buffer is kept from synthesis as it directly connects an input port to an output port (you can tell by the name 776).  Later we buffer the input port and the output port so we get the three buffer chain.  I believe this has come up before.  The location of the buffer is just whatever global placement happens to pick (from an hpwl perspective the X coordinate has no effect).
We could do something like don't buffer the input if it only drives buffer(s) currently.  We would still buffer the output resulting in a two buffer net (prior to any repair).  Is that acceptable?

@maliberty That would be great, but it is a separate concern from being able to answer the question of what happens w zero buffers. I think both have value.

zero

I don't understand what you comment about zero buffers relates to above.  Having zero buffers is a bad idea.

zero

I don't understand what you comment about zero buffers relates to above. Having zero buffers is a bad idea.

For the mock array, there is a discussion to be had as to how many buffers are needed. Maybe not for every Element. Therefore, it is interesting to do a measurement of what zero buffers would yield as a limit. If with zero buffers one can not.reach acceptable numbers, then that is a useful negative result for architectural feedback.


[Jira] Add notifications

Extension
https://www.raycast.com/raycast/jira
Description
It could be great to see notifications inside Raycast.
Who will benefit from this feature?
Everyone using Jira.
Anything else?
No response

Thanks for the feature request. Would you like to work on it?

@raycastbot assign me


Unnecessary use of jQuery global

It seems to me that bootstrap still has an artificial dependency on the jQuery global that seems unnecessary.
At the top of the file jquery is imported but then the jQuery global is used.
https://github.com/twbs/bootstrap/blob/a4667be152a3a4fed4e42f2a9efbf831bb5946b6/js/src/index.js#L1
But then the "check" uses the global
https://github.com/twbs/bootstrap/blob/a4667be152a3a4fed4e42f2a9efbf831bb5946b6/js/src/index.js#L21-L32
This pattern is repeated in other files as well, the top imports jquery as $ but then uses jQuery in code. Should it not use $ instead?
I'm asking from a build tool perspective which wouldn't have any issues if the imported object was used instead of the global?

@Johann-S: should we also check for $ in index.js? This should be more flexible.

yep agreed with @thheller


[FLOC-1159] Design sketch - config persistence

https://clusterhq.atlassian.net/browse/FLOC-1159
Depends on #1045 being merged first.

Thanks.  This basically looks good - but please clarify in the two places where I've left questions.  (To be clear, "rejecting" this design but basically only because I'd like to make sure we're on the same page in those two places before accepting it.  I'll probably accept it once you answer those questions.)


Changes required for Beet client PR181

The Beet wallet is undergoing a significant uplift: https://github.com/bitshares/beet/pull/181
The changes to the Beet wallet required changes to this library, such as:

Removing index storage - library users should store this data with their own method eg local storage libraries

Still used in the Beet wallet itself, it's just not imposed on the NFT.next.js implementation

Replacing hand crafted websocket implementation with socket.io for reduced maintenance burden/complexity
Removed webpack, uses esm rather than commonjs
Refactor of code structure, now exporting connect & link functions, which after running export the active BeetConnection class to the dev for direct blockchain interaction.
Using @noble/ed25519 instead of eliptic for ECDH key agreement (primarily due to eliptic's dev inactivity)

To see this in action check out the following example implementation:
https://github.com/BTS-CM/NFT.next.js/blob/main/pages/beet.js
https://github.com/BTS-CM/NFT.next.js/tree/main/components/beet

The BTS JS injection example when implemented as a web component returns the following error:
{
  "code": 10,
  "message": "Execution error: Assert Exception: now <= trx.expiration: ",
  "data": {
    "code": 10,
    "name": "assert_exception",
    "message": "Assert Exception",
    "stack": [
      {
        "context": {
          "level": "error",
          "file": "db_block.cpp",
          "line": 735,
          "method": "_apply_transaction",
          "hostname": "",
          "thread_name": "th_a",
          "timestamp": "2022-05-18T17:00:00"
        },
        "format": "now <= trx.expiration: ",
        "data": {
          "now": "2022-05-18T17:00:00",
          "trx.exp": "1970-01-01T00:00:00"
        }
      },
      {
        "context": {
          "level": "warn",
          "file": "db_block.cpp",
          "line": 770,
          "method": "_apply_transaction",
          "hostname": "",
          "thread_name": "th_a",
          "timestamp": "2022-05-18T17:00:00"
        },
        "format": "",
        "data": {
          "trx": {
            "ref_block_num": 49362,
            "ref_block_prefix":<PHONE_NUMBER>,
            "expiration": "1970-01-01T00:00:00",
            "operations": [
              [
                1,
                {
                  "fee": {
                    "amount": 48260,
                    "asset_id": "1.3.0"
                  },
                  "seller": "1.2.1808745",
                  "amount_to_sell": {
                    "amount": 1,
                    "asset_id": "1.3.0"
                  },
                  "min_to_receive": {
                    "amount": 1,
                    "asset_id": "1.3.6073"
                  },
                  "expiration": "2022-06-18T16:59:54",
                  "fill_or_kill": false,
                  "extensions": []
                }
              ]
            ],
            "extensions": [],
            "signatures": [
              "signature"
            ]
          }
        }
      },
      {
        "context": {
          "level": "warn",
          "file": "db_block.cpp",
          "line": 278,
          "method": "push_transaction",
          "hostname": "",
          "thread_name": "th_a",
          "timestamp": "2022-05-18T17:00:00"
        },
        "format": "",
        "data": {
          "trx": {
            "ref_block_num": 49362,
            "ref_block_prefix":<PHONE_NUMBER>,
            "expiration": "1970-01-01T00:00:00",
            "operations": [
              [
                1,
                {
                  "fee": {
                    "amount": 48260,
                    "asset_id": "1.3.0"
                  },
                  "seller": "1.2.1808745",
                  "amount_to_sell": {
                    "amount": 1,
                    "asset_id": "1.3.0"
                  },
                  "min_to_receive": {
                    "amount": 1,
                    "asset_id": "1.3.6073"
                  },
                  "expiration": "2022-06-18T16:59:54",
                  "fill_or_kill": false,
                  "extensions": []
                }
              ]
            ],
            "extensions": [],
            "signatures": [
              "signature"
            ]
          }
        }
      }
    ]
  },
  "digest": "digest",
  "transaction": "transaction"
}

The parent TRX has 1970 as expiration, but the operation itself has the correctly formatted expiration. Should a second expiration to overwrite the 1970 value be included? Or is this a bitsharesjs/bitshares-core issue?

The BTS JS injection example when implemented as a web component returns the following error:
{
  "code": 10,
  "message": "Execution error: Assert Exception: now <= trx.expiration: ",
}

The parent TRX has 1970 as expiration, but the operation itself has the correctly formatted expiration. Should a second expiration to overwrite the 1970 value be included? Or is this a bitsharesjs/bitshares-core issue?

It should have been overwritten already: https://github.com/bitshares/bitsharesjs/blob/94ae9df594897becc1defaa119ed03a1c726dec6/lib/chain/src/TransactionBuilder.js#L153-L156
                            if (this.expiration === 0)
                                this.expiration =
                                    base_expiration_sec() +
                                    ChainConfig.expire_in_secs;

Perhaps call set_expire_seconds(sec) explicitly? See code
https://github.com/bitshares/bitsharesjs/blob/94ae9df594897becc1defaa119ed03a1c726dec6/lib/chain/src/TransactionBuilder.js#L311-L317
    /** optional: there is a deafult expiration */
    set_expire_seconds(sec) {
        if (this.tr_buffer) {
            throw new Error("already finalized");
        }
        return (this.expiration = base_expiration_sec() + sec);
    }

How are you creating a browser compatible bundle to include in plain html/javascript?

I've reintroduced the webpack compiled bundle for plain javascript use

Reintroduced HTTPS and added a new nft_object signing function and example

Was there a reason why you not only upgraded dependencies but also refactored pretty much everything? :) It's breaking my head trying to review change, I pretty much need to review the new code as its own..

The move to ESM and Vue3 is to blame for the big refactor, lol.


Introducing Puzzler-bot

Hi @yegor256.
Let me introduce my new tool for managing puzzles - @puzzlerbot. It is supposed to work either alone or in pair with 0pdd. It's aim is to provide new ways of organizing PDD approach in GitHub.
Hope it would be helpful for our EO community.

@yegor256/z please, pay attention to this pull request

@skapral/z this pull request is too small, just 2 lines changed (less than 10), there will be no formal code review, see §53 and §28; in the future, try to make sure your pull requests are not too small; @yegor256/z please review this and merge or reject

@skapral thanks

The job is not in WBS, won't close the order


feat: added link for workspace in top nav

Description
Now there is a link to the current workspace in the top navigation.
What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Closes #2675
Mobile & Desktop Screenshots/Recordings

Steps to QA

Click on the workspace link in the top navigation
It loads the current workspace
Notice the Workspace link is orange (active) when the page loads.

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Converting to draft as the link should only be visible to a logged in user.


Deprecated findUrlFactory deprecation notice refers to self.

      /**
       * @return UriFactoryInterface
       *
       * @throws Exception\NotFoundException
       *
       * @deprecated This will be removed in 2.0. Consider using the findUrlFactory() method.
       */
      public static function findUrlFactory()
      {
          return static::findUriFactory();
      }

I'm looking at fixing this deprecation notice, but since the suggestion appears to be saying I should be using the deprecated function itself, I'm not sure what to make of this.

that was a typo in the deprecation notice which has been fixed in master but apparently not in the version you have installed: https://github.com/php-http/discovery/blob/master/src/Psr17FactoryDiscovery.php#L130
use the findUriFactory - that is consistent with it returning an UriFactoryInterface


Why territory 001 for Europe/Berlin )and not DE?

Hi,
great mapping between Windows and IANA time zones, thanks a lot!
Is there any reason why Europe/Berlin has not territory "DE"?
		**<mapZone other="W. Europe Standard Time" territory="001" type="Europe/Berlin"/>**
		<mapZone other="W. Europe Standard Time" territory="AD" type="Europe/Andorra"/>
		<mapZone other="W. Europe Standard Time" territory="AT" type="Europe/Vienna"/>
		<mapZone other="W. Europe Standard Time" territory="CH" type="Europe/Zurich"/>
		<mapZone other="W. Europe Standard Time" territory="DE" type="Europe/Berlin Europe/Busingen"/>

From IANA tz:
DE	+5230+01322	Europe/Berlin	Germany (most areas)

The file is from unicode.org. But if you look at, for example, London or Paris you'll notice the same.

The file is from unicode.org. But if you look at, for example, London or Paris you'll notice the same.


Adding file existence checks

The script has copy commands for files that may not exist. This left a bug which would cause the unattended installer to exit prematurely. Added file existence checks to these copy commands to address this issue.
Addresses issue: https://github.com/pivpn/pivpn/issues/902

I am sorry, Bunch of conflicts at the moment,
Will have to ask you to make the PR again considering the new Test branch commits.
I will be more than glad to take a look at it after that.
Sorry for the inconvenience,
Kind Regards.


Incorrect File Autocompletion When Using FileDialog Filters

Godot version: 3.2.1.stable.official
OS/device including version: Windows 10 Home Version 1903
Issue description: When the filters property for FileDialog nodes is set, weird autocompletion occurs when the dialog is in use. If you click the dropdown menu near the bottom right corner of the dialog, and select a specific file type (eg. the filter *.png ; PNG Files), the file LineEdit is autocompleted incorrectly (in this example, it would be ".*.png ; png files").
If you do this multiple times (for example, switching between file types), the file extension will continue to be appended at the end; for some weird reason, the semicolon and lowercase description only ever appears once at the end and gets overwritten if used multiple times. See below for an image showing the result.

Steps to reproduce:

Create a scene with a FileDialog. In my code, I made it popup when a TextureRect is pressed, and the bug showcase project uses a Button, but any way should work.
Set the filters property. Define at least two file types. I used *.png ; PNG Files and *.jpg ; JPG Files.
Run the program and open the FileDialog.
Select one of the two custom filters by selecting the dropdown menu near the bottom right corner. Observe the weird autocompletion.
Do this a few more times. Observe the additional autocompletion.

Minimal reproduction project: FileDialog Bug Showcase.zip

Is there a workaround for this?

Is there a workaround for this?

Just use extension in filter. No star, no point, no description.
just : PNG
instead of :  *.PNG ; PNG Picture
works for me but no format description anymore


fix: missing comma introduced in #4

This PR fixes a missing comma introduced in #4 which subsequently causes the linter to fail

LGTM. Thanks


Allow destructured object entries with default parameters not to be optional

Suggestion
🔍 Search Terms
default value optional object destructure
✅ Viability Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
If a default value is set on a required field in an object, I wish Typescript allowed an undefined value to be passed in without throwing an error.
I don't think this would be a breaking change because it will only affect situations that are currently errors.
I feel like there's probably something fundamentally weird about the way I'm thinking about it, but I wanted to at least create an issue so that other people could find it, as I wasn't able to find anything on this.
📃 Motivating Example
const {a = 1, b}: {a: number; b: number} = {b: 1};

This throws an error, but I believe that this is readable, and as far as I understand it Javascript could never allow the value of a to be undefined in such cases.
💻 Use Cases
The primary use case is for React prop interfaces:
interface Props {
  a: number;
  b: number;
}

const Component: React.FC<Props> = ({a = 1, b}) => {
  return <>{a + b}</>;
};

<Component b={1} />;

This shows an error unless a is made optional, but it doesn't seem like there's anything wrong or confusing about this to me.

Previously discussed at #30488.
By design, type annotations are always the last word on anything. This is important for predictability and for performance.

For those unable to follow, like me without a lot of head-scratching:
@RyanCavanaugh seems to be saying that the following two lines should both throw the same error:
const {a = 1}: {a: number} = {a: undefined};
const {a = 1}: {a: number} = {};

I respectfully disagree, I don't think either of them should throw an error, since it is promised that a will be a number and so that is what we get. My way of thinking about it seems quite reasonable unless we're allowed to put type hints in the destructuring syntax. Then there would be a way to distinguish between these cases, though I still don't understand why one would mind (maybe a case where you reuse the assignment operator in a statement to chain it in a weird way?):
const {a = 1}: {a: number} = {}; // error cause we're expecting an object that contains a as a number, which {} is not
const {a: number = 1} = {}; // fine cause we're implicitly expecting any object, and we know that a is guaranteed to be a number

Personally I'd want to see a case where it's ambiguous what the author intended and the transpilation is more complicated than just stripping out everything after the colon, which is generally what I think of as the defining feature of typescript. I don't know about performance but it seems unlikely that you need a whole bunch of overhead just to remember whether the destructuring had a default parameter or not when you check if an undefined value was passed in.

Personally I'd want to see a case where it's ambiguous what the author intended

I think the intent of this line is extremely clear:
const {a = 1}: {a: number} = {};

I want to initialize a binding with an object
The value that must initialize the binding is { a: number }
If not, I have made a mistake and should see an error
After that, if the value for a is undefined, set it to 1

Step 3 is unambiguous and what happens at step 4 is immaterial. Further proof of this intent is counterfactual: had I wanted a to be optional, I could have written { a?: number } instead, but I didn't.

I think the disconnect is that as a maintainer, you see the behavior of a type annotation as saying something about what objects can enter the zone it's responsible for, but as a user, I am thinking of its purpose as giving me guarantees about the code that follows. So to me as long as there's no earthly way for me to interfere in the middle of an expression, I want the annotation to tell me what types the expression gives me, not what I can pass into it.

const {a = 1}: {a: number} = {};

self-evidently and unambiguously (:p) says that we are destructuring an object and that what we'll have after is a number called "a", which is true

It depends on what you think destructuring is a sugar for. If you wrote
const obj: {a: number} = {};
const a = obj.a === undefined ? 1 : obj.a;

then the type error is still there.
TS is consistent about this view of what destructuring means:
const {a = 1}: Partial<{a: number}> = {};
a // correctly number, not number | undefined

Obviously there are multiple ways to interpret the situation, but both have trade-offs and your view of this being an incompleteness is someone else's view of a possible lack of error here being an unsoundness.

Yeah that's what I'm trying to get at, you said this was "discussed" but in both cases it was just a post by you that's basically impenetrable to users. I understand the need for maintainers to have a consistent philosophy, I just think it's helpful to have it written out what the disagreement is because users rarely have to think of it that way.

I'm not sure what you want me to do?

I wanted you to make that comment that explains it! Thank you, unironically!

Happy to help 😅


Best behavior of the "see sequence" button

Clicking on the "eye" icon opens the Sequence tab and displays the sequence of the associated source. When going back to the Cloning tab, the "eye" icon has turned green, indicating that is the currently open sequence in the editor. However, when you click again on the green "eye", it turns grey again and, in the background, clears the editor in the Sequence tab.
I understand this is the "logical" programmatic way for it to work, but I think the user may expect the sequence to open again. In other words, end users will not click on the green "eye" to clear the editor which they are not seeing. What do you think? What was your idea when you designed this "toggle" behaviour, versus a simple button that opens the sequence in the editor?

Yes, that's a good option. The green eye could still show which sequence is open (the last sequence the user was working with), but clicking on it could open the "Sequence" tab again.


Add Window and Buffer overloads to allow specifying maximum time length and element count when window/buffer opening is controlled by another sequence

Basically, add the following overloads to Window (respectively for Buffer):
public static IObservable<IObservable<TSource>> Window<TSource, TWindowOpening>(
    this IObservable<TSource> source,
    IObservable<TWindowOpening> windowOpenings,
    TimeSpan timeSpan,
    IScheduler scheduler
)

public static IObservable<IObservable<TSource>> Window<TSource, TWindowOpening>(
    this IObservable<TSource> source,
    IObservable<TWindowOpening> windowOpenings,
    int count
)

public static IObservable<IObservable<TSource>> Window<TSource, TWindowOpening>(
    this IObservable<TSource> source,
    IObservable<TWindowOpening> windowOpenings,
    TimeSpan timeSpan,
    int count,
    IScheduler scheduler
)

This would allow using external events to specify the opening of overlapping windows while using the traditional element count and time length conditions to automatically close the windows.
Note that so far as I understand it, there is no trivial implementation for the _count_ overload, even if you wanted to implement this using the existing overload with window closing factory. It would be exceedingly useful to have this built into the framework for completeness.

Sounds like an uncommon usage to me. Would an implementation in a 3rd party library work for you?

Unless I am misunderstanding the purpose of the Window operators in general, this is not an uncommon usage at all.
Window operators are common and fundamental for all kinds of data analysis and signal processing. Above are some examples of the kind of stuff you can do with various windowing strategies. The last line is what is currently missing from the toolkit, and it's actually one of the most common applications of windowing in science and engineering.
Defining regular regions of interest to analyse patterns around known events is one of the most common data analysis technique for time-series data. Basically it allows you to ask questions of the sort: "what tends to happen to [seismograph, stock market, brain activity, insert your favorite time series here] data whenever this event happens?".
That this was considered important by the designers of the original Window API is revealed in the design of current TOpening, TClosing overloads. Unfortunately, for analysis purposes in most cases you don't really care about dynamic closing, as you want to average or visualize across comparable time intervals or comparable element counts, so it's an overkill. This proposal aims to rectify this and bring the convenience of count and timespan boundaries to the dynamic slicing behaviour of the Window operator.
If there is no time to implement this, feel free to leave the issue open and I might try to submit a pull request for future review.

I'm not questioning the existence of an use case, I'm questioning why this should be in Rx.NET proper. Just because there are other Window operators doesn't justify the inclusion of more. Also it is not just a question if we can write it but are we willing to keep supporting it once the original poster left. Third, my experience in the ReactiveX world tells me that users tend to overestimate how widely certain operators would be actually needed.

Here is an example how this could be implemented anywhere with the existing Window operator by using a feedback subject:
{
class Program
{
    static void Main(string[] args)
    {
        new int[] {
            100,
            600, 610, 620, 630, 640, 650,
            1100, 1700
        }.ToObservable()
        .SelectMany(t => Observable.Timer(TimeSpan.FromMilliseconds(t)).Select(_ => t))
        .Window(Observable.Never<int>(), 5, TimeSpan.FromMilliseconds(500), Scheduler.Default)
        .Do(_ => Console.WriteLine("== New window =="))
        .Merge()
        .Subscribe(Console.WriteLine);

        Console.ReadLine();
    }
}

static class Ops
{
    public static IObservable<IObservable<T>> Window<T, U>(
        this IObservable<T> source,
        IObservable<U> boundary,
        int maxSize,
        TimeSpan maxTime, IScheduler scheduler)
    {
        return Observable.Defer(() =>
        {
            var boundarySubject = new Subject<int>();
            var index = 0;

            var complexBoundary = boundarySubject.Synchronize()
                .Merge(boundary.Select(v => 3))
                .Publish(shared =>
                {
                    return shared.Merge(
                        shared.Prepend(-1)
                        .Select(_ => Observable.Timer(maxTime, scheduler))
                        .Switch()
                        .Select(_ => 1)
                    );
                })
                .Do(_ => Interlocked.Exchange(ref index, 0))
                ;

            return Observable.Window(
                source,
                complexBoundary
            )
            .Select(window => {
                return window.Select(v => {
                    var obs = Observable.Return(v);
                    if (Interlocked.Increment(ref index) == maxSize)
                    {
                        obs = obs.Concat(Observable.Defer(() =>
                        {
                            boundarySubject.OnNext(0);
                            return Observable.Empty<T>();
                        }));
                    }
                    return obs;
                }).Concat();
            });
        });
    }
}

The judgment of whether or not this is a common case is subjective, and since we clearly come from very different domains it's probably not going to be useful at this point to argue about it.
I take your point about maintenance, which to be honest is enough to leave the issue in the backlog.
However, I stand by my original point that this is not about adding extra cases to Window. This is about taking the existing overloads which have already been presented as a convenience, and rounding up the options of the operator semantics for common variations.
At some point someone thought having both a Window(count) and a Window(timeSpan) was useful, rather than forcing every consumer of Window to write custom sequences just to slice an input stream into fixed size chunks. This indicates to me that the original designers of the API had exactly the applications I presented above in mind, but I may be confused, so if anyone can clarify to me why Window was made in the first place, and what were in fact its intended applications, that should help me to understand whether this issue should stay or go.
Finally, for reference, I leave here a more concise implementation of all overloads using existing combinators:
public IObservable<IObservable<TSource>> Window<TSource, TWindowOpening>(
    IObservable<TSource> source,
    IObservable<TWindowOpening> windowOpenings,
    TimeSpan timeSpan,
    IScheduler scheduler)
{
    return source.Window(windowOpenings, x => Observable.Timer(timeSpan, scheduler));
}

public IObservable<IObservable<TSource>> Window<TSource, TWindowOpening>(
    IObservable<TSource> source,
    IObservable<TWindowOpening> windowOpenings,
    int count)
{
    return source.Publish(ps => ps.Window(windowOpenings, x => ps.Take(count).Count()));
}

public IObservable<IObservable<TSource>> Window<TSource, TWindowOpening>(
    IObservable<TSource> source,
    IObservable<TWindowOpening> windowOpenings,
    TimeSpan timeSpan,
    int count,
    IScheduler scheduler)
{
    return source.Publish(ps => ps.Window(windowOpenings, x => Observable.Merge(
        Observable.Timer(timeSpan, scheduler),
        ps.Take(count).LongCount())));
}


VS Code support

Hello guys. It would be wonderful if you add support for Visual Studio Code.

Yes, please. Looking forward for a VS Code support.


ClientModel: investigation for StreamingClientResult

Closing in favor of https://github.com/Azure/azure-sdk-for-net/pull/43392


Sort1

challenges 1 and 2 pass!

challenge 1 passes! Feel free to complete and resubmit for additional credit. Thanks

challenges 1-6 pass!


media3 integration, handle audio focus gain/lost

Media3 pros:

same player interface, which ease switching from playing in activity to playing in service
out of the box integration with wear watches, google assistant, bluetooth headset etc. (by implementing mediasessionservice)
exoplayer as core, so switching from exoplayer is straightforward

Cons:

Need to refactor
New bugs, such as crashing when playing in mediasessionservice. (which forces to use media metadata+medialibraryservice+mediaitemfiller).

all is clear, thanks for the research @ahamula . Seems media3 really doesn't provide anything of use now. Maybe we'll go for it when we need something with wear os, but even then we'll need to compare it with alternatives. Possibly they will force this upon us and deprecate the ExoPlayer.


Feat: add support to 0.4.x breaking changes

While:

adopt and polish realm-agnostic migration scripts
get the Request bytecode initialized after construction
keep a whole copy of last successfull Witnet.Response

Closes https://github.com/witnet/internal/issues/89.

To be dealt in further issues:

XauEurPriceFeed is broken (its sources are no longer live). We may want to remove this feed from the repo.
There are some feeds that are only relevant to specific networks, e.g. CFX/USD on Conflux. We shouldn't deploy those on other networks, as we can't commit to keep them running.


vite config root is ignored on workspace

Describe the bug
Though we know maintaining cwd is tricky, can we respect explicit root config such as this?
// packages/client/vite.config.ts

import { defineConfig } from 'vitest/config';
import { join } from 'node:path';

export default defineConfig({
  // neither works
  root: './dir',
  // root: join(import.meta.dirname, './dir'),

  plugins: [
    {
      name: 'dump-root',
      configResolved(config) {
        // this is always "/abs-path-to/packages/client"
        // regardless of `root` config
        console.log('[dump-root]', config.root);
      },
    },
  ],
});

For example, this can affect Vite's project absolute path resolution as in the reproduction below.
Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-rwukz2?file=packages%2Fclient%2Fvitest.config.ts

npm run test fails
npm run test-no-workspace succeeds

System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.20.3 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/local/bin/npm
    pnpm: 8.15.6 - /usr/local/bin/pnpm
  npmPackages:
    @vitejs/plugin-react: ^4.2.1 => 4.3.3
    @vitest/ui: latest => 2.1.4
    vite: latest => 5.4.10
    vitest: latest => 2.1.4

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I think we might need this line:
https://github.com/vitest-dev/vitest/blob/cf0cbf6a37a461f9040d456aa934d13aa7fb9617/packages/vitest/src/node/create.ts#L42
Although it looks very weird, I would prefer if we rewrote this to a config hook that respects user's root option.

Is this issue related to it?
We're having trouble getting the workspace to work from the root directory (both in CLI and the extension), whereas running the projects individually works fine.


[OPENSTACK-2482]Remove certs and keys for deleted ssl profiles.

@<reviewer_id>
What issues does this address?
Fixes #
WIP #
...
What's this change do?
Where should the reviewer start?
Any background context?

Do you have an JIRA id?
Added

JIRA id need to be save in commit message

Added.


Tab ordering issue

If I display <calendar-date> inside a popup (in a similar way to your Shoelace integration example) then when the popup opens I can call 'focus' on the calendar-date component to correctly set focus. This works as expected as the user is then immediately able to use the keyboard to navigate around and choose a date.
The problem occurs if the user wants to tab to the 'change month' buttons, currently they have to know to use 'shift tab' to send the focus backwards. The native Chrome input date time picker allows the user to use just 'tab' so that focus then shifts to the 'non date selecting' controls, however with <calendar-date> this causes focus to leave the component. You can see this exact same behaviour with your Shoelace example code.
I've tried putting some form of focus trapping in to detect this and refocus the component, but this is "risky" code as it involves querying down into Cally's shadow DOM to pass focus directly to the "previous" button.
I'm not sure how this can be easily fixed, I like the default Cally behaviour of focusing the 'date' part first as I believe this is the thing a keyboard user will most likely want to change so I think it should retain the primary focus but it would be nice if the tab ordering could be "tweaked" so that the other interactive elements gain focus before it leaves the component.
See https://www.w3.org/WAI/WCAG21/Understanding/no-keyboard-trap.html (Calendar widget) for some of the motivation behind this request :)

I see what you mean. Though I'm not sure changing the focus order to mismatch the visual order is a good fix, since that's a WCAG violation in itself.
One solution would be to focus the popover itself?

I see what you mean. Though I'm not sure changing the focus order to mismatch the visual order is a good fix, since that's a WCAG violation in itself.
One solution might be to focus the popover itself? But then you're trading better tab order at the cost of some ergonomics.

Its tricky for sure, I'm trying to avoid anything hacky/complicated that falls apart if someone looks at it wrong ;)

I think i'm going to extend the focus() method to allow e.g.:
calendar.focus({ target: "prev" });
calendar.focus({ target: "next" });
calendar.focus({ target: "day" }); // same as...
calendar.focus()

Just to add, the expected arg extends native FocusOptions, so it'll also allow you to do stuff like:
calendar.focus({
  target: "previous",
  preventScroll: true
})


Unable to prepare setup for platform o365.calendar

Ever since I upgraded to version 2022.10 and higher, my o365.calendar would give me the following error:
ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform o365.calendar: Platform not found (cannot import name 'CalendarEventDevice' from 'homeassistant.components.calendar' (/usr/src/homeassistant/homeassistant/components/calendar/init.py)).
Any idea what's causing this in the latest versions? It works in 2022.9.7 but not higher than that...

What version of the o365 integration are you using, are you on the latest version? Please post the full error, I think you have only posted the first line.

This release (v3.1.0) in May solves your issue - https://github.com/RogerSelwyn/O365-HomeAssistant/releases/tag/v3.1.0

@RogerSelwyn thanks for your swift response! I figured I just had to update the application including the custom component files and configuration. Works like a charm now on the latest version of HA!


Dev

How does the app not respect the config? Please get rid of all the flag stuff in main, otherwise it will be undeployable


Feedback: Consider making an IDE for lyte2D similar to amulet's browser IDE

At least for mobile devices that would make it easy to program with lyte2D directly on the mobile device. The amulet browser IDE even lets one export the code as .wasm directly from the IDE in the browser:
https://www.amulet.xyz/editor.html
Fyi, and thanks for considering.

This is a cool idea, and something like this is on the backlog.
I see two approaches here. I'm leaning towards (1) and Amulet and some others are like (2).

Write the editor in Lyte2D itself (think how Godot's editor is written in Godot using it's UI facilities.)  This can be made to work in WASM easily. And the editor/plugins would be in Lua.
Use an established open source web IDE (I'm thinking Code Mirror 6.) There are examples and should be doable.

Perhaps both of these can be done and may have their places. Also note that both of these may require some features from Lyte2D itself.
I'm thinking perhaps (2) can be approached by community as well. Perhaps someone can take an initial stab at this one. If you have interest/time in something like this, let me know!

A super cool bonus for (2) would be to integrate it with Lyte2D website itself (whose source code is also in this repo) so that live editing of samples would be possible.

Thanks for considering this! I would like to help but don't think I have the programming skill to do so yet. I'll keep working on my programming ability and keep a lookout for how/when I can contribute :-)

Hello! Slightly related, I wanted to try out to develop a simple IMGUI for lyte. Slowly but surely and I really do not know much about gui. Any tips and resources are welcome. I've got some starting code at this gist starting idea

@lplume very cool! I don't have much guidance, other than to say that I have used love2d library SUIT in the past and it was fine. But I don't have much other experience with imguis.
Please let me know if you hit issues, or find something that should exists but doesn't in the API.

nice to point out SUIT! At a first glance looks like I got more or less the same idea! Stay tuned! More to come

Based on the chat with @lplume on #40, I'm adding some ideas.
Editor-as-a-lyte2d-app thoughts (from requirements perspective):

Filesystem access style: rxi's "lite" (current folder only) or "litexl" (everything)

For former, more PHYSFS functions can be exposed. (ls, mkdir, is_directory(), etc.)
For latter, we need either LFS or implement some of it ourselves
Do we need a "native" load/save dialog support?

How does font rendering really look? Is lyte2d's font rendering good enough for long term text? Some testing and comparison may be needed. Comparison to love2d, lite and litexl would be interesing.
Lyte2D currently doesn't expose events directly. "You check something if you care about it."  Is this a problem for UI tools? I don't think so, but I'm open to thinking more about this

Is Lyte2D's input functionality good enough for this? See #39 for example.

Some for from Lyte2D will be needed for this in most cases. Let's identify these.
Thoughts?

Also slightly related to font issue: https://github.com/orgs/lyte2d/discussions/16

My bits:

filesystem access: about the editor itself it could benefit some sort of filesystem access, i would suggest to think about "would lyte2d benefits from filesystem access?", at the moment i have no idea. It is possible to access file via lua's io, i do not think about directory listing; thinking of an editor it sure could use it.
events and UI's: well... there are no ideal solution. If lyte2d do not fire events directly a library/wrapper (in pure lua probably) could be written to manage events. Im not so deep into lyte2d dev'ing,  might change my mind but so far i like how it is shaped.
Constraints usually opens to creativity, i'd like to hear from other dev who's actually experimenting the engine to gamedev


Fix typo in README

Merged https://github.com/rain-1/linenoise/commit/184b81296aa1514c9a2533db90e9a1d529e7abae


How to specify custom classpath?

One of our apps require some Java scripting, and for the scripts project, we only want to add the selected dependencies. Is it possible to override the classpath? I don't want to be using Gradle or Maven for this, but how does that work?

What that server can do - syntax compatibility, whether it supports formatting or outlines etc - is outside of our control

You'll need to ask this question upstream to the underling Eclipse JDT that does most of the work https://github.com/eclipse/eclipse.jdt.ls


Rely on PhEDEx (or something else) for the storage pledged/available/used space

Impact of the new feature
ReqMgr2 MSTransferor and likely MSMonitor
Is your feature request related to a problem? Please describe.
As part of the Unified logic for input data placement, we need to have a mechanism to reliably monitor:

how much storage is available at each PNN for the DataOps group
how much data is commited to each PNN

such that it can drive the location selection logic for input data placement (also part of the Unified black box, as requested by issue #9138 )
Describe the solution you'd like
Create the client API under the Services package and integrate it to the MicroServices where needed. Certainly in the MSTransferor algorithm to see which PNNs are eligible to receive data or not.
Describe alternatives you've considered
It could be handled through reviewing the Transferor logic in Unified and integrating whatever hasn't yet...
Additional context
See latest changes in this PR: https://github.com/dmwm/WMCore/pull/9330
and also some hints as discussed in this ticket: https://github.com/CMSCompOps/WmAgentScripts/issues/439#issuecomment-522857388

After chatting with Eric about this use case using Rucio, it works in a similar manner as PhEDEx does; however, in Rucio quotas are defined in an account and RSE basis. Thus, the PhEDEx group will be the Rucio account we use for production activity.
Here is the API and an example of its output:
>>> pprint(list(client.get_account_usage("transfer_ops")))
[{u'bytes':<PHONE_NUMBER>8955,
  u'bytes_limit':<PHONE_NUMBER>000000,
  u'bytes_remaining':<PHONE_NUMBER>31045,
  u'files': 42322,
  u'rse': u'T3_US_NERSC',
  u'rse_id': u'1cb4cc86c6674c17b20463fa36c4aa9e'},
 {u'bytes':<PHONE_NUMBER>511,
  u'bytes_limit':<PHONE_NUMBER>0000,
  u'bytes_remaining':<PHONE_NUMBER>489,
  u'files': 10857,
  u'rse': u'T1_DE_KIT_Disk_Test',
  u'rse_id': u'dc967804cd6a407a8603b54ddc73adf7'},
 {u'bytes':<PHONE_NUMBER>893,

and if we want to get the account usage against a specific RSE, then we can provide it as a 2nd parameter, e.g.:
>>> pprint(list(client.get_account_usage("transfer_ops", "T2_US_MIT_Test")))
[{u'bytes':<PHONE_NUMBER>954,
  u'bytes_limit':<PHONE_NUMBER>0000,
  u'bytes_remaining':<PHONE_NUMBER>046,
  u'files': 9071,
  u'rse': u'T2_US_MIT_Test',
  u'rse_id': u'd479089e01434deead1ee377e949276b'}]


Update help.py

updated width and height and idk if it is already counted as one character but now one line  is one more character so help command dont work if a lot of commands.

should be good now ( this is a suck fix ig but it works for me)