Connection leak from watch timeout.

I'm using the watch timeout set to 10s. If we've not received a change notification by then I ping the downstream service anyway and then restart the watch. Unfortunately, this leaks a connection every 10s if there is no activity in etcd.
I assume this is due to the rust-etcd watch integration with hyper. Let me know if you would like any specific diagnostics.

Can you clarify what you mean by leaking a connection?

Sure, sorry.
As above I'm setting a 10s timeout on a watch and then repeating after 10s. This issues an http://etcd:2379/v2/keys/root?wait=true (from memory) every 10s with no response, as expected. If you run netstat -anp | grep -cE ":2379.*ESTABLISHED" you can see that the number of connections to etcd increments every 10s. Each connection is allocated an outbound TCP port. At that rate it takes about a week to run out of ports to connect from.
My understanding is that this is because the timeout configured in https://github.com/jimmycuadra/rust-etcd/blob/master/src/kv.rs#L531 has no way of cancelling the hyper connection, and the latter never times out. I've not really looked into hyper, but perhaps they don't think there is a problem: https://github.com/hyperium/hyper/issues/1097.
I'm using (from Cargo.lock) etcd 0.8.0 and hyper 0.11.6.

As linked above, I think this is a hyper bug which I've raised with them - or if it isn't, then I think it needs a better explanation of how to use timeouts for long-poll GETs!

Should be fixed by f2574a2f935a181961c68dd8615ae3bff06f5090.


SslHandler#unwrap must send fireChannelRead event after a notification for a handshake success

Motivation:
In some use cases SslHandler#unwrap sends fireChannelRead event before a notification for a handshake success.
Such use case is a HTTP server that supports HTTP/1.1 and HTTP/2 via ApplicationProtocolNegotiationHandler,
in this case fireChannelRead is sent before ApplicationProtocolNegotiationHandler is able to setup properly the pipeline
depending on the negotiated protocol. The issue is visible when SslProvider.OPENSSL is configured.
This is a regression caused by #11156
Modifications:

SslHandler#unwrap sends fireChannelRead event after a notification for a handshake success
Add test

Result:
Fixes #11201

@violetagg thanks a lot!

@violetagg thanks a lot!


How to make it not display info information?executorExampleBasic.cpp

I don't want to display info information when I run ExecutorInstanceBasic.cpp。What should i do? Thanks.
Info information such as：
[TensorRT-LLM][INFO] Engine version 0.13.0 found in the config file, assuming engine built by new builder API
[TensorRT-LLM][INFO] Initializing MPI with thread mode 3
[TensorRT-LLM][INFO] Initialized MPI
[TensorRT-LLM][INFO] Refreshed the MPI local session
[TensorRT-LLM][INFO] MPI size: 1, MPI local size: 1, rank: 0
[TensorRT-LLM][INFO] Rank 0 is using GPU 0
.................

export TLLM_LOG_LEVEL=WARNING;
(INFO,WARNING,ERROR,TRACE,DEBUG)


service-worker.js 404

What is the missing service-worker.js ?
Hi everyone ! I'm new here, I created a express.js app few minute ago, it's on 4.17.1.
My question is in this screenshot:

And, here is my simple code:
const express = require('express')
const app = express()

app.get('/', function (req, res) {
  res.send('Hello World')
})

app.listen(3000)

What am I missing ?

Hi @KaivnD, this doesn't appear to have anything to do with Express.  You must have an html page which is trying to load that as a script tag, and as your example doesn't even serve an html page I am not sure how it could be an express issue.

Hi @wesleytodd, this error keep coming, and I do have html page, seen below.

Yes, your html is referencing a file which does not exist.  This has nothing to do with express.


react-native run-android --variant=release

HI when i use
react-native run-android --variant=release
the audio dont play, if only use react-native run-android works fine
, how can i deploy to google play my app?

Facing the same issue, is your issue get resolved ?

Does this have a solution yet? Android seems to have issues lately :(

$ cd android
$ ./gradlew assembleRelease
my solution

Maybe you have Android 9 - then it is due to security policy - please read here how to solve it: https://developer.android.com/training/articles/security-config


Enable local archiving (and improve archiving design)

Currently, the archiving process assumes archiving to a remote host using rsync.
We should allow multiple transports.  At least, rysnc between local dirs.
The main complication here is that we not only need to transfer files, but also to check free disk space on the target directories.  Currently this is implemented by remotely executing the df command on the remote host.  So we need a clean and robust way to check free space that is coordinated with the file transport mechanism.
There's an additional complication because when transferring via rsync to a remote rsync daemon, we see virtual paths as exported by the rsync daemon configured modules, but when we execute df, we view paths as they exist natively on the remote host.  We currently have a fairly crufty mechanism for mapping between the two views of paths.  This mechanism needs to be robust, and to generalize across whatever transport mechanisms we implement.

As a request, it would be nice to be able to enable rsync over SSH in the config. This would allow for secure transfer over the internet(, of course this would bring encryption overhead). This would allow not needing to have rsync daemon running, which would inturn add security. I would also maybe opt for plotman being configured to use rsync over SSH by default, so that no one by mistake has rsync facing the internet.

will be great if can put the ssh custom port in the config.yaml
rsync --ignore-existing --remove-source-files -Pave "ssh -p 12022" source/path/here user@host:/destination/archive

As a request, it would be nice to be able to enable rsync over SSH in the config. This would allow for secure transfer over the internet(, of course this would bring encryption overhead). This would allow not needing to have rsync daemon running, which would inturn add security. I would also maybe opt for plotman being configured to use rsync over SSH by default, so that no one by mistake has rsync facing the internet.

+1


some float/double cases are not right

for example:
d_fdiv_b1_01.S
inst_13:
// rs1==f0, rs2==f5, rd==f17, fs1 == 1 and fe1 == 0x3f8 and fm1 == 0x0000000000000 and fs2 == 1 and fe2 == 0x001 and fm2 == 0x0000000000002 and rm_val == 0
// opcode: fdiv.d ; op1:f0; op2:f5; dest:f17; op1val:0xbf80000000000000; op2val:0x8010000000000002; valaddr_reg:x16; val_offset:208; rmval:0x0; correctval:0; testreg:x18
TEST_FPRR_OP(fdiv.d, f17, f0, f5, 0x0, 0, x16, 208, x17, x15, 208, x18)
0xbf80000000000000 divide 0x8010000000000002, and result is stored into f17, it is not equal 0, but you set the correct val rm_val == 0, so this case is not right.
it seems that there are lots of similar cases in riscv-test-suite/rv64i_m/D.
is there anyone who can help to update? thanks a lot~

Just to be clear: is it the actual result that is incorrect, or the comment with the result incorrect?
Note that every result reported is zero, which make me think this is a test generator artifact.

@allenjbaum the comments for the reference-value/correct-value for almost all tests must be ignored because they are not meant to be generated by CTG. They are a place holder for a fix that @MarcKarasek volunteered to fix by extracting those from the SAIL runs and pushing them back into the tests via some compile macros.
So, I would not focus on the "reference-value/correct-val" in the comments for now.

@allenjbaum  the the actual result is incorrect. @neelgala do you know how to fix it? thanks a lot~~

The results generated are from SAIL and/or Spike (default golden models). If you can show that SAIL/Spike (both of which use soft-float) infact spit out the wrong values, then this should be filed with SAIL/Spike ASAP.
Considering this repo (riscv-arch-test) has migrated to RISCOF and no longer host the references in this repository there is nothing much to be done in the tests themselves.


v5

v5

[ ] support file://* manifest URIs, not just https//* ones
[ ] support file://* nw URIs, not just https//* ones
[ ] add version options stable and latest
[ ] detect platform-arch and check if they want to directly run instead of building it
[ ] delete outDir dir post zip
[ ] verify checksum of binaries
[ ] add support for linux arm
[ ] add support for mac arm
[ ] validate nwbuild arguments and centralise it to one function (currently yargs has its own checks)
[ ] package app capability
[ ] chmod binaries
[ ] code sign binaries
[ ] improve error logs using Winston maybe?
[ ] improve JSDocs
[ ] Improve code quality by using promises and async await
[ ] configure Desktop entry file for Linux builds (#689)

delete outDir dir post zip
This one should be optional.

I'm very interested to see what you find with "chmod binaries" and "code sign binaries".
For the chmod issue in my scripts, I ended up requiring builds to be done in Linux to preserve file permissions from the NW.js tarball as well as dependencies. Otherwise, it was going to be a nightmare to determine which files needed permission changes and to get those permission changes inside a tarball generated in Windows.
I really need to implement code signing, to hopefully avoid the Windows Defender popups. Hoping you make good progress here!
Finally, one more suggested change/enhancement is to implement rcedit to modify the Windows NW.js executable (icon and metadata). I suspect something similar is also needed for macOS packages.
P.S. All of these changes sound like a 4.1, not a 5.0. 😉

For the chmod issue in my scripts, I ended up requiring builds to be done in Linux to preserve file permissions from the NW.js tarball as well as dependencies. Otherwise, it was going to be a nightmare to determine which files needed permission changes and to get those permission changes inside a tarball generated in Windows.

In Windows, chmod only changes the write permission. Possible solution is to pass in an octal value which represents that if platform is Windows.

Finally, one more suggested change/enhancement is to implement rcedit to modify the Windows NW.js executable (icon and metadata).

✅

I suspect something similar is also needed for macOS packages.

Yup, it involves updating the Info.plist files. First need a way to build and test run an app in CI so one would need to have a Mac to debug NW.js Mac apps.


does votenet suit for lidar point cloud detection?

Hi,
Thanks for your good work.
I am curious that does votenet suit for object detection in lidar e.g. kitti benchmark?
Whether the lidar point cloud is too sparse to vote out good candidates

Good question!
It is possible to apply a similar idea to KITTI. I did have some initial experiments with some positive signals. However due to the time constraint we didn’t go deep into this part.
As lidar points are more sparse, the signal to noise ratio (rate of positive votes to negative votes) is lower – some filtering or pre-segmentation may be necessary to make it work really well.

Thanks for your reply.
I think the pipeline with pre-segmentation like PointRCNN may help. Or we can upsample/completion point cloud firstly.

@godspeed1989 did you get this working? if so how were the results?

@gigadeplex Not yet, it is just a idea now.


How to add the extra toolbar in mirror container ?

Whenever an element from the container getting dragged, dragula creates the exact mirror of it and moves along with the mouse.
Now the use case is to have additional toolbar (which contains the button to move and copy item ) which is outside of container.

How can we make the toolbar as a part of mirror container so that it can look like toolbar is stick with item ?
How can we enable the drag functionality when the toolbar is out of container ?

Sample Image for toolbar -

The container is the element that contains all the draggable items, it's not draggable itself.
Aside from that, if I understand you right you want to apply a toobar to the item that is being dragged? How would you click any of those icons if you're already dragging?


Add an entry in glossary for MVP

Thanks for adding this @Richabanker
/lgtm

I get the name was changed as per discussion here
Is the feature gate: UnknownVersionInteroperabilityProxy in the process to be renamed as well?

This LGTM, thanks @Richabanker
/lgtm
However I'd still like to know if:

I get the name was changed as per discussion here
Is the feature gate: UnknownVersionInteroperabilityProxy in the process to be renamed as well?

This LGTM, thanks @Richabanker /lgtm However I'd still like to know if:

I get the name was changed as per discussion here
Is the feature gate: UnknownVersionInteroperabilityProxy in the process to be renamed as well?

Thanks @Ritikaa96
Looks like we may have to stick with the original name (UnknownVersionInteroperabilityProxy) for the feature gate according to @jpbetz's response here

This LGTM, thanks @Richabanker /lgtm However I'd still like to know if:

I get the name was changed as per discussion here
Is the feature gate: UnknownVersionInteroperabilityProxy in the process to be renamed as well?

Thanks @Ritikaa96 Looks like we may have to stick with the original name (UnknownVersionInteroperabilityProxy) for the feature gate according to @jpbetz's response here

Right. I prefer consistency as well though I understand it's due to the code freeze. 👍🏼


Rename address fields

I think we should rename 'Address' and 'Address' to 'Address 1' and 'Address 2' or, even better, we could change the logic of addresses.
For example, Amazon, we have a field set "Street", then inside it:

Street and number, P.O. box,
Apartment, suite, building, floor, etc.

We don't need to use placeholders, event just as labels "Address 1 (Street and number, ...)", "Address 2 (...)".
I get negative feedbacks on this part as being confusing, and get the questions "What is it?" and "What is address 1 and 2?.

We should only display one "Address" label. I like what Amazon and Shopify do in this regard, they display one label and use placeholders to provide hints.

@Pacu2 Can You assign me for this issue?

@patrys, @Pacu2 Can you tell me if this looking good?

@tomcio1205 Looks good to me

Looks good! 👍


Fix intuitive propagation behavior

Add small threshold to begin dragging
Don't propagate clicks to scrolled element after movement has reached threshold

Useful if you are scrolling something with lots of links or buttons in it

Don't propagate mousedown events

Useful if you have .dragscrolls embedded in other .dragscrolls

These changes break current behavior, and add a few more bytes, but it makes the behavior a lot more intuitive to me. Creating the pull request in case it seems more intuitive to anyone else.

this works perfectly for me.
I suggest to merge this.

This worked really well as a fix on a sliding navigation bar, where navigation clicks were being incorrectly detected as drags in some cases.

+1 for the works really well!
SHOULD MERGE!

Worked


Add a heap to optimize lease expiration checks

This adds a heap acting as a priority queue to keep track of lease
exiprations. Previously the whole lease map had to be iterated through
each time.
The queue allows us to check only those leases which might be expired.
When the expiration changes, we add an additional entry. If we check an
entry that isn't expired, it means that the lease got extended.
If we find a entry in the heap that doesn't have a corresponding entry in
the map, we know that the lease has already been expired or revoked.
This is our first stab at writing real Go, and we're happy to make any changes if you all think this is a good way of doing things.
/cc @jcalvert

can you write a benchmark to compare the old/new approach?

We're working on a reproducible, synthetic benchmark, but here is a chart of 3.2.11, 3.3.1 and 3.3.1 with this change under our production traffic to give you an idea of the problem we are trying to fix:

This is our application observed latency for creating a lease and making a transaction (plus some other stuff responsible for the baseline 100ms. These match our other metrics, where average latency is normal, but p99 and higher get very, very large. We should have a reproducible benchmark available soon.

@mgates Do you mean the tail latency with heap is much higher than the one with map?

Ideally, benchmark (in Go, func BenchmarkTESTNAME) should show faster expiry lease query, and the compiled etcd with heap show lower p99 latency.

Great - we'll get those benchmarks set up.

I think I'm misunderstanding - the tail latency with the heap is much much lower (and doesn't seem to grow unbounded with lease map length).

tail latency with the heap is much much lower

Then, sounds good!
Can you provide reproducible workloads that can be cross-checked in my side?

Just to let you all know, we got side-tracked, but we're still planning on getting some benchmarks done and ready.

@mgates This is missing some methods (e.g. revoke).
Let me cherry-pick your commit with more extensive benchmarks, tomorrow.

So, we don't actually think we need to revoke or update - the idea is that the heap is just candidates - we always check the map for the actual lease. If a lease is revoked, when we get to it in the heap, we find it's not in the map, and we ignore it. If a lease is renewed, an additional marker gets put in the heap, and we ignore the first one, because the lease isn't expired in the map yet.

So, we don't actually think we need to revoke or update - the idea is that the heap is just candidates - we always check the map for the actual lease.

Then how we going to handle false positives in server-side lease revoke routine? If a bunch of manual revoke happened, then expire channel could return leases with stale secondary index, and server errors. Was there any performance bottleneck updating heap on revoke?

I don't think that we'd get errors - if we get a stale lease ID, we just ignore it. My understanding is that it would be slow to iterate through the leap finding them by lease ID, but if you think it won't be we can try it and benchmark it.

Oh I see now that your code ignores it. I will take another look.

@mgates Thanks for updates!

accurately reflects the improvement in performance we've seen with the find expired leases function.

I would like to cross-check on this before we merging in. What kind of workloads did you ingest to verify this improvements?
I am busy till next week. But will try to merge this in by following, latest.

@gyuho - We cloned our traffic in a production environment to a separate cluster, for about 500 GRPC operations per second (tx, get, put, lease grant etc) with a max of about 250 leases expiring per second and a total number of keys averaging around 5 million. This same workload rendered our unpatched cluster unusable within 30 minutes of operation.

The benchmark result makes sense to me.

@jcalvert can you please clean this PR up? Then we can get it merged.

LGTM after addressing @xiang90's comments. We can merge first. I will do the benchmarks sometime later.
Thanks!

Hey @gyuho and @xiang90,
@jcalvert and I have fixed some cleanups to the PR and I think we addressed all outstanding feedback. Let us know.

@mgates @cosgroveb Could you squash commits and add license header? I have example branch https://github.com/gyuho/etcd/commits/tmp.

@gyuho Added the license header to both new files and squashed our commits.

@cosgroveb One last nit, could you amend commit titles to lease: ... (CI will complain about this)? Then LGTM.
Thanks a lot!

@mgates I just rebased from master to trigger CIs. Will add release notes as well. Thanks.

Test failures aren't related. I also manually ran full test suites and no failures. Merging.


[Feature Request] When removing users and networks in ZNC, the physical user/ and network/ directories should be deleted, or there should be an ability to do cleanup

ZNC does not seem to delete the physical user/ and network/ directories (and contents) within ~/.znc if the corresponding users and networks are deleted via *status or *controlpanel or *webadmin.
It similarly does not delete moddata/module/ directories but I find this to be a very good thing - you would want your previous module configuration to be retained if you decide to unload and reload a module.  This is specifically for users and networks which are removed.
Perhaps instead of doing this instantly, there could be a CleanConfDir command (or similar).  Such as, /msg *status CleanConfDir which would compare the physical conf directory contents to the existing users and networks defined in the conf and remove the ones which are no longer valid.
I do periodic backups of my entire ~/.znc directory and it is bothersome that there are so many users and networks which were long since deleted.
Tested on:
<*status> ZNC 1.9.x-git-108-60ea155 - https://znc.in
<*status> IPv6: yes, SSL: yes, DNS: threads, charset: yes, i18n: yes
<*status> ZNC 1.8.2 - https://znc.in
<*status> IPv6: yes, SSL: yes, DNS: threads, charset: yes, i18n: yes, build: cmake

This appears to be a duplicate of https://github.com/znc/znc/issues/260


Homekit Thermostat: Better support for temperature ranges

Support for obtaining temperature range
Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

@cdce8p now the range is 10 - 38，Can get it automatically? When using homekit control, it will be out of range, and the range of air conditioning is generally 16-30

I see what you mean. To get this into the next beta, I did the required changes (unit conversion, fallback to defaults, tests).

@roiff I'll merge this now. Can you test if all works for you? If not I can do a bugfix later.

@cdce8p Thank you, If there's a problem, I will again submit the problem to you。
there is a problem, some tropical areas, use of air conditioning is very much, homekit use mechanism does not adjust the air conditioning fan, I want to add a fan_v2 as an air conditioning accessory , Work at the same time, but this is not in line with homekit norm, how do you think this issue should be handled properly?
In addition, the accessory of the fan can't adjust the wind speed now. Is this a consideration?
My English is very poor, sorry....

Fan service for climate entities

I understand the issue you're thinking of. However, due to the current state of the climate component this might be more difficult, since it isn't that compatible with the way HomeKit is doing it. In addition there is a debate at the moment to change / improve the climate entity model https://github.com/home-assistant/architecture/issues/22.
If you nevertheless want to try it, I would suggest adding the fan as a second service. Take a look at the media_player type file (inside the homekit folder). That might help.

Fan speed settings

We are thinking about it. The issue is again that the current Home Assistant model is not really compatible with HomeKit, since one measures percent and the other low, medium, high and multiple others. We'll add that once it's possible.

Since it isn't advised to discuss something in closed PRs, I would suggest that you write me on discord if you have questions.


fix bump-version.sh (version.go file was moved)

What is the purpose of this change? What does it change?
fixes script that updates version number before release
this got broken due to moving verision.go file.

:white_check_mark: odo build 656 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/1d721e5bb1 by @kadel)


fix: External refs with overrides

The external resolver did not take into account for overrides. E.g take the following example:
$ref: 'base.yaml'
id: my_id
settings:
  $ref: 'base_settings.yaml'
  useCache: true

This failed to resolve on $ref: 'base_settings.yaml'. This is due to to root object passes $Ref.isExternal$Ref(obj) and therefore not considering the other child objects. We should still crawl the tree even though the root has a $ref.

Included in https://github.com/APIDevTools/json-schema-ref-parser/pull/321


Update printer_star_line.inc.php

Use proper variable

@holtkamp Thanks for catching this

@gare-bear thanks for the feedback, I noticed this "typo" later here as well: https://github.com/star-micronics/star-cloudprnt-for-woocommerce/blob/239d442b1bec119714f16a5d0b21a4bca609721b/cloudprnt/printer_star_prnt.inc.php#L143


Login/SignUp button in navbar not have pointer

This is not good that login/signup button not have cursor pointer i will add pointer to that so it give better experience.

@zalabhavy it has now.


Segmentation fault after running load_network from inference engine with daal4py

Describe the bug
I am using openvino inference engine and daal4py in my program. When I call the load_network function, I get a segmentation fault when the python interpreter exits.
If you import daal4py before openvino.inference_engine, then there is a problem, but if you swap the imports, then there is no error. If you do not import daal4py, then there is no error.
P.S.  Because of what the operating system loaded two different libtbb.so.
I think this issue can be closed.


Mark more features as optional

What problem does this solve or what need does it fill?
I want to use bevy as an engine for backend part of my project. For now bevy requires a lot of useless dependencies for backend stuff like window features and others.
Describe the solution would you like?
Mark as much as possible features as optional. Then end user will be able to select and use only required features.
Describe the alternative(s) you've considered?
I don't see other solution for the problem.
Additional context
No

default-features = false is enough to run Bevy on backend server.
The simplest server I use on backend looks like:
    App::build()
        // minimal plugins necessary for timers + headless loop
        .add_plugin(bevy::type_registry::TypeRegistryPlugin::default())
        .add_plugin(bevy::core::CorePlugin)
        .add_plugin(ScheduleRunnerPlugin::run_loop(Duration::from_secs_f64(
            1.0 / 60.0,
        )))
        .run();

With Cargo.toml:
[dependencies]
bevy = { version ="*", default-features = false }

@smokku Does it require to install all deps from installation guide? Like sudo dnf install gcc-c++ libX11-devel alsa-lib-devel systemd-devel?

No. Turning off default features disables the whole render pipeline.

I just checked the dependencies of my server.
$ ldd target/debug/server
	linux-vdso.so.1 (0x00007ffca1de8000)
	libdl.so.2 => /lib64/libdl.so.2 (0x00007fb76aea7000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007fb76ae85000)
	libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x00007fb76ae6a000)
	libc.so.6 => /lib64/libc.so.6 (0x00007fb76aca0000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fb76b29d000)
	libm.so.6 => /lib64/libm.so.6 (0x00007fb76ab5a000)

Thanks for the help and your investigation! Now it's clear for me.


🧑🦲 Test Current UI Iteration with Users for Some Initial Feedback on MetaData PoC

User Need
As a (type of user)
I want (some action)
so that (some result)
Value
Functional Requirements:

Non-Functional Requirements:

Acceptance Criteria:

Notes:

Aged out


AttributeError: module 'general_purpose.utilities' has no attribute 'keys_exists'

I've noticed this today while running an old script of Climate-Learning:
==Reading data==
Traceback (most recent call last):
  File "/mnt/homedafs-2.10/gmiloshe/Climate-Learning/VAE/analogue_george.py", line 167, in <module>
    if (ut.keys_exists(run_vae_kwargs, 'label_period_start') and ut.keys_exists(run_vae_kwargs, 'label_period_end')):
AttributeError: module 'general_purpose.utilities' has no attribute 'keys_exists'

Did you change something?

No I think it is something else...

There were breaking changes in the fix of #6. For the matter at hand, keys_exists was renamed to key_exists

yes, I'll do it stat

Please read the other breaking changes that were introduced for #6 and tell me if they make problems for you

I added back the keys_exists method. But if you can, I recommend rather doing replace all in the files involved.
The name of the function was misleading, as it checks only one key, so I'd rather not stick around with it too much


Calculate the ACC error during train the model

When I am training the model, there is an error in calculating ACC.
Could you provide a way for me to solve it?
Error message:Tensor holds the wrong type, it holds int, but desires to be int64_t.

pred and label type

Hello, Thanks for the issue, we don't see this problem in our env, could you please provide your paddle version ?
A quick fix is to add a type conversion in the loop where the data is loaded from dataloader, such as:
for batch_id, data in enumerate(dataloader):
    # get data
    images = data[0]
    label = data[1]
    # add data conversion to int64
    label = label.astype('int64')

P.S.,
From the screenshot I saw the main_single_gpu.py may not be the updated code in our implementation, could you please refer to our new impl https://github.com/BR-IDL/PaddleViT/blob/develop/image_classification/SwinTransformer/main_multi_gpu.py

Hi xperzy,
My env are as shown below.
Package            Version

appdirs            1.4.4
astor              0.8.1
certifi            2021.10.8
charset-normalizer 2.0.12
cityscapesScripts  2.2.0
colorama           0.4.4
coloredlogs        15.0.1
cycler             0.11.0
decorator          5.1.1
fonttools          4.33.3
humanfriendly      10.0
idna               3.3
kiwisolver         1.4.2
lmdb               1.3.0
matplotlib         3.5.1
numpy              1.19.3
opencv-python      <IP_ADDRESS>
packaging          21.3
paddlepaddle-gpu   2.2.2.post112
Pillow             9.1.0
pip                21.2.4
protobuf           3.20.1
pyparsing          3.0.8
pyquaternion       0.9.9
pyreadline         2.1
python-dateutil    2.8.2
PyYAML             6.0
requests           2.27.1
setuptools         61.2.0
six                1.16.0
tqdm               4.64.0
typing             <IP_ADDRESS>
typing_extensions  4.2.0
urllib3            1.26.9
wheel              0.37.1
wincertstore       0.2
yacs               0.1.8

P.S.,
I use the SwinTransformer model for image classification.

@VivizSun  Is your problem sovled by using : label = label.astype('int64')  ?

@xperzy Yes, thank you very much.


Feature Request: @extends for record types declarations

I'd really like to be able to do something like
/**
 * @param {ShapeMessage} message Type-safe data for constructing shape.
 * @constructor
 */
Shape = function(message) {
  this.color = message.color;
};

/**
 * @typedef {{
 *   color: string
 * }}
 */
ShapeMessage;

/**
 * @param {SphereMessage} message Type-safe data for constructing sphere.
 * @constructor
 * @extends {Shape}
 */
Sphere = function(message) {
  goog.base(this, message);
  this.radius = message.radius;
};
goog.inherits(Sphere, Shape)

/**
 * @typedef {{
 *   radius: number
 * }}
 * @extends {ShapeMessage}
 */
SphereMessage;

new Sphere({color: 'white', radius: 5})

So then Sphere constructor can call goog.base to run the Shape constructor with the message safely, and then the Sphere constructor can interpret the Sphere-specific stuff itself.  This example is dopey, but you can imagine for larger trees of inheritance that this would be useful.  We create the leaf object with a fully-formed message. The message bubbles up to the base constructor, with each constructor reading the relevant parts, and type-safety guaranteed the whole way by defining a separate type for each constructor.
I'd be totally willing to hack around on this for a bit myself if y'all think it's worth it, but thought I'd get your opinions first.
This would obviously wouldn't work if the extended type were a primitive, but this should be relatively easy to detect and reject.

+1 for this request.
I find myself wanting to do the same thing for an API response-type that has some (local-only) data mixed-in by a controller.

We will be making typing for interfaces structural, and this will provide extends for records for free.

I've skimmed through the Release Notes but haven't found the answer: if there's already a way to inherit one @typedef from the another? Like reuse one @typedef declaration but add more fields to it?

No there isn't a way to do that.

There is now experimental support for @record in the compiler.

@record is pretty stable now and can be used with inheritance.


Wrong directory path in "Compiling Mattermost"

Hi,
in the file docs/source/developer/dev-setup-compiling.rst (current master; commit  d31f09de5cc53c21d338f26bc5e45608284bf89f) one is asked to:

... change to the source directory with the following command:
cd ~/go/src/github.com/mattermost/platform

which will not work (since the folder does not exist).
Substituting platform with mattermost-server helps (hence suggested change).
Probably related to #1477.
Regards,
wp

Thanks @WyattPorter !
I've created a pull request to fix the issue.


riscv64 support in cmake is broken

Currently if architecture is not detected it falls back to x86, with SSE2 and other stuff enabled, which is incorrect for riscv64.
I'm not sure what would be the best course of actions here - either to fallback to generic or add riscv64 (or maybe both) though as I don't know what were intentions behind fallback to x86.

"generic" architecture used to be same code path as 80x86 (32-bit, little-endian, Linux)... It was never changed as all other combinations and choices were subsequently added as soon as optimized versions were added. As already mentioned elsewhere, we combined code paths for 32-bit and 64-bit processors when the instruction sets clearly overlapped.
We haven't decided yet if we should separate "generic" and "x86" cases to eliminate unnecessary checking and/or enabling of SSE2 or other x86 specific instruction sets.

What I meant that behavior of configure and cmake build systems are fundamentally different here. In case of configure if it fails to detect architecture it falls back to generic.
In case of cmake it's clearly falls back to x86 in that case.
I've changed the PR to have only one defined.

Yes... configure has more fine-grained detection... cmake support was quite unfinished when zlib-ng was forked from zlib and since that it has evolved more than the configure script... We still support using configure (and other alternatives, like nmake) as configure script has always been more advanced and cmake isn't installed by default on a lot of systems.


Multiple input HMTL files from the command line

How can I pass all HTML files to uncss on the command line? I tried with wildcards but uncss **/*.html doesn't seem to work.

Sorry, just realized I was looking at the wrong output file, the command above actually works.


Laravel 11 cannot access subdomain on multi-database setup

Bug description
After setting up the multi database and registering a user I'm always redirected from subdomain to the domain.
Seems like it in the TenancyServiceProvider the makeTenancyMiddlewareHighestPriority() breaks it. Changing the namespace fixes it.
foreach (array_reverse($tenancyMiddleware) as $middleware) {
-     $this->app[\Illuminate\Contracts\Http\Kernel::class]->prependToMiddlewarePriority($middleware);
+    $this->app[\Illuminate\Foundation\Http\Kernel::class]->prependToMiddlewarePriority($middleware);
}

Steps to reproduce
Fresh multi database setup
Expected behavior
Access to the subdomain.
Laravel version
11.1.0
stancl/tenancy version
3.8.0

You likely followed some steps here wrong https://github.com/archtechx/tenancy/pull/1180. After properly setting up tenancy in L11, this logic for me works the exact same way it did in L10.


Hot Fix for SOD

All Submissions Basics:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[ ] Have you created a .py in ~/pyod/models/?
[ ] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[ ] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build 1420

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.533%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/models/sod.py
0
1
0.0%

Totals

Change from base Build 1401:
0.0%

Covered Lines:
4406

Relevant Lines:
4612

💛  - Coveralls


The input of 3D Human

Hello
The input of  (f3D H )is 85 but you told in paper this input is 1024. why?

Please take reference from the following code. For convenience, we incorporated the calculation of f_{H}^{3D} in the full model.
https://github.com/DirtyHarryLYL/DJ-RN/blob/83c3d0f16a4a3b55545e96a8350397f6adb1ef91/lib/networks/DJR.py#L383

I dont understand.  the size of f3D is 85 but you tile it to 1024?


Error creating push database! { Error: You are not a server admin.

Error creating push database! { Error: You are not a server admin.
at Request._callback (/usr/src/app/node_modules/nano/lib/nano.js:252:15)
at Request.self.callback (/usr/src/app/node_modules/nano/node_modules/request/request.js:185:22)
at emitTwo (events.js:126:13)
at Request.emit (events.js:214:7)
at Request. (/usr/src/app/node_modules/nano/node_modules/request/request.js:1157:10)
at emitOne (events.js:116:13)
at Request.emit (events.js:211:7)
at IncomingMessage. (/usr/src/app/node_modules/nano/node_modules/request/request.js:1079:12)
at Object.onceWrapper (events.js:313:30)
name: 'Error',
error: 'unauthorized',
reason: 'You are not a server admin.',
scope: 'couch',
statusCode: 401,
request:
{ method: 'PUT',
headers:
{ 'content-type': 'application/json',
accept: 'application/json' },
uri: 'http://XXXXXX:XXXXXX@couchdb:5984/pushinfo' },
headers:
{ date: 'Fri, 15 Jun 2018 06:03:13 GMT',
'content-type': 'application/json',
'cache-control': 'must-revalidate',
statusCode: 401,
uri: 'http://XXXXXX:XXXXXX@couchdb:5984/pushinfo' },
errid: 'non_200',
description: 'couch returned 401' }

Sorry, but that's very unclear. What did you do, what had you done?

Problem solved, Thanks :)

For future googlers:
I think this is caused from trying to access the containers outside of the internal docker services or bad username/password.
(I think).


Removed "-common" packages

Problem
We need to reduce amount of npm packages for publishing YAML/JSON LSP server.
Solution
Removed "-common" packages.

made a small fix for "fileMatch" field

License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Why did package-lock.json increase by 8k lines?

We need to reduce amount of npm packages for publishing YAML/JSON LSP server.

What exactly does this mean? What problem is being solved?


No such module "YouTubePlayer"

Hi, thanks for providing the swift realization of Youtube player view.
But when I tried to used your library, when I try to call:
import YouTubePlayer
in my ViewController file, the following error is reported:
No such module "YouTubePlayer"
Can you give me some hint?
Thanks very much!

How are you including the library in your project? I'd suggest using a git submodule.
In your project's root directory:
$ git submodule add git@github.com:gilesvangruisen/Swift-YouTube-Player.git
Then drag YouTubePlayer.xcodeproj into your project in Xcode. In the "General" tab of your project's target, look for "Embedded Binaries" and click the +. If the library is included as a sub-project, you should see YouTubePlayer listed. Click that and you should be good to go!

@gilesvangruisen
Thanks very much for your hint.
My project can find the Module now! (^O^)
Right now, I am considering supporting from iOS 7.0, but your library is designed to supporting from iOS 8.0.
So can I just change the deployment target from iOS 8.0 to 7.0 in order that my app can run?
Waiting for your kindly help.
Thanks ahead!

I'm going to leave it as 8.0+ for now (might change this in the future) but there should be nothing wrong with changing deployment target to 7.0.
Best of luck!

gives me....  -bash: $: command not found

folder right click and click terminal at folder under list then paste $ git submodule add git@github.com:gilesvangruisen/Swift-YouTube-Player.git  its give me error -bash: $: command not found

ohh sorry .. now ok..

<myproject.VideoViewController 0x7fc24bc9c770> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key videoplayer.

hi man. how can i remove youtube big red button...:(

@gilesvangruisen
I have the same problem as imadeit, and I am doing as u have explained here and in the end I am adding in "Embedded Binaries" YouTubePlayer.framework. But YoutubePlayer.framework in "Embedded Binaries" is being red and I am having again the error saying: No such module 'YouTubePlayer'.
Any suggestion would be very helpful
Cheers

Updating this thread to reflect that YouTubePlayer now supports Cocoapods and Carthage. Check out the README


Option to combine -o and -g

[X] I've verified and I assure that I'm running youtube-dl 2016.04.13
[X] At least skimmed through README and most notably FAQ and BUGS sections
[X] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[X] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of your issue, suggested solution and other information
I use Kodi Media Center for all my entertainment and media management. This program is capable of playing streaming video in many formats from an url written in a .strm file. I would like to use youtube-dl to output a playlist of music videos in a specific format (say, '%(playlist)s/%(playlist_index)s - %(title)s.%(ext)s' which will put playlist files into a common folder, but instead of actually downloading the video, just echo the url (as given by the -g option) into the file. I have actually accomplished this using a bash script which parses the output of youtube-dl --get-filename --get-url but it is a very hacky solution. Any chance this could become a feature?
EDIT: To give an example, I would like to be able to run youtube-dl -f best -g -o '%(playlist)s/%(playlist_index)s - %(title)s.%(ext)s' https://www.youtube.com/playlist?list=PLY5f8vtstsfjedMi6VHRAeFV85Xc_VoKG and see afterwards a subdirectory in the working dir of youtube-dl, which is named by the title of the playlist (in this case, NateWantsToBattle Songs) and in that subfolder would be a number of files named according to their order in the playlist and title (in this case, one would be named "NateWantsToBattle Songs/01 - twenty one pilots - Stressed Out [NateWantsToBattle feat. ShueTube].mp4") and each file would contain only the plain-text url of the video as shown by -g (the example above would contain the url string shown here).

Could you give some concrete video links and your expected output?

great idea & it does have links... does github not list the time that OP's post was edited?

Well, no. Edits don't activate notifications and alerts either..

I think this can be easily achieved with a script:
#!/bin/sh
youtube-dl -j "$@" | python -c '
import sys
import json
import os
for line in sys.stdin.readlines():
    info = json.loads(line)
    fname = info["_filename"]
    dir = os.path.dirname(fname)
    if dir and not os.path.exists(dir):
        os.makedirs(dir)
    with open(fname, "wt") as f:
        f.write(info["url"])
'

I don't find it hacky, although others may disagree.

I am now using this script and it does exactly what I wanted. I forked the repository and was gonna make a PR with the required changes, but I don't know how to properly do this in the context of youtube-dl's workflow (it looks to me like a file is created in the same function as downloading something to it, and while I suppose I could probably make it download a pastebin of the url somehow, it seems, well, too hacky. How could this be integrated into the application in a more elegant way?


Pvt data update

Documentation update

Description
backport to 2.2 branch.

200 Files in this PR. Maybe manually copy the file might be better?
I was going to make some edits to the tutorial, so not sure it if necessary to cherry pick this right away.

Was trying to backport merged PR to 2.2 and got a failed merge message.  Executed the manual cherry-pick process documented here https://hyperledger-fabric.readthedocs.io/en/release-2.2/github/github.html#cherry-picking-your-pr-to-other-branches
and this is what happened.  Not intentional.

I’m going to close this PR Nik.
Sent from my iPhone

On Sep 21, 2020, at 12:27 PM, Chris Gabriel<EMAIL_ADDRESS>wrote:

200 Files in this PR. Maybe manually copy the file might be better?
I was going to make some edits to the tutorial, so not sure it if necessary to cherry pick this right away.
Was trying to backport merged PR to 2.2 and got a failed merge message. Executed the manual cherry-pick process documented here https://hyperledger-fabric.readthedocs.io/en/release-2.2/github/github.html#cherry-picking-your-pr-to-other-branches
and this is what happened. Not intentional.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Trimming of newlines or linebreaks

Hi,
I am new to react quill. I want to trim new lines or line breaks at the beginning and end of the editor's contents. Trimming of spaces and tabs is working fine. but I want to trim newlines/linebreaks as well.which are caused by clicking on "Enter" Button.
I am saving the HTML content from Editor in my component state. So I need to remove one or multiple of these

You could simply use a regular expression for that
let noNewLines = html.replace(/

<\/p>/g, '')

To remove duplicates but leave one instance.
const noNewLines = html.replace(/(

<\/p>)+/g, "$1")


Gradle plugin uses tmp as the default AppMap output directory

:tada: This issue has been resolved in version 1.2.0 :tada:
The release is available on:

v1.2.0
GitHub release

Your semantic-release bot :package::rocket:


Feature/Basic Deploy Operator

This PR will try to implement the basic structure of Deploy Operator in xline-operator.
A Deploy Operator will handle the following tasks:

Provide a CRD to add a new resource type, XlineCluster, to k8s.
Accept human operators commands on this resource (such as scaling up/down, upgrading, rolling back), such as adding an XlineCluster.
Create/Update/Reclaim resources such as StatefulSet/Secret/PersistentVolume/CronJob.
Monitor Sidecar Operator.

Some question for the name, why name it to Deploy Operator and not xline operator.
It let me confused.
https://github.com/xline-kv/xline-operator/blob/a3030cbe4de7769d50bbb11a1363e1777c6ea82d/deploy/src/crd.rs#L29-L31
And some comment for crd:

i will suggest init the version from v1Apha.
domain for xline and not datenlord.io would be greate.

Thanks for your reviewing!

why name it to Deploy Operator and not xline operator

Here is our architecture diagram:

We can see that we have two green operators. They together form the xline operator.
Specifically, the deploy operator is responsible for deploying various resources on k8s, while the sidecar operator plays a role in monitoring and recovering the cluster.

i will suggest init the version from v1Apha.

Thank you for pointing it out. I think it is reasonable to use "alpha" as a suffix while the CRD has not yet stabilized. I will make the necessary changes in the upcoming PR.

domain for xline and not datenlord.io would be greate(great?).

Yes, I will change it to xline.cloud in the upcoming PR.

@iGxnon
I will suggest name the sidecar operator  the xline sidecar ,  and it's also managed by xline operator.
For the first version, xline operator just work for deploy xline,  sidecar is the advanced features.
Some comment to FYI. :)


noImplicitReturns disallows empty return statement in an async function that does not explicitly return Promise

TypeScript Version:
1.8.5 /1.8.7
Code
async function test(isError: boolean = false) {
	if (isError === true) {
		return;
	}
	console.log(await Promise.resolve("The test is passed without an error."));
}

test();

Expected behavior:
There should be no error when compiling the above code with noImplicitReturns.
Actual behavior:
"error TS7030: Not all code paths return a value." is returned.

This is by design i would say. the whole point of noImplicitReturns is to make sure you do not "fall through" the function end unnoticed.
function foo(isError: boolean = false): SomeType {
    if (isError === true) {
        return undefined;
    }
}

so for the non async function foo above, --noImplicitReturns forces you to return a value in the other branch.
it is true that an async function would always return a value (a Promise object), but the motivation of this feature is not nullablity as much, but rather style consistency, and in that sense, i think the error you are getting is in line with the motivation.

The non-async version of this code doesn't have an error under noImplicitReturns, so I don't see what the relevant difference between an async an non-async function is when considering return expressions.

that is because the return type annotation is not there. and in that case it goes to any. in the case of the async function it is known it is always Promise.

@mhegazy Thanks for your tip. The following code can compile:
async function test(isError: boolean = true): Promise<void> {
	if (isError === true) {
		return;
	}
	console.log(await Promise.resolve("The test is passed without an error."));
}

However, why is "Promise" required?

Well, i think that is a bug :D (at least from my point of view). looks like we have an inconsistent behavior here regardless of how you look at it.

OK discussed this with @vladima  offline. looks like a bug.
// Should be an error, Promise<number>, currently retorted correctly
async function test3(isError: boolean = true) {
    if (isError === true) {
        return 6;
    }
}

// Should not be an error, Promise<any>, currently **not** working
async function test4(isError: boolean = true) {
    if (isError === true) {
        return undefined;
    }
}

// should not be error, Promise<any> currently working correctly
async function test5(isError: boolean = true): Promise<any> { //should not be error
    if (isError === true) {
        return undefined;
    }
}

// should be error, currently reported correctly
async function test6(isError: boolean = true): Promise<number> {
    if (isError === true) {
        return undefined;
    }
}

// infered to be Promise<void>, should not be an error, currently reported correctly
async function test7(isError: boolean = true) {
    if (isError === true) {
        return;
    }
}


BooleanQuery Occur::MustNot causes no docs to be matched

Describe the bug

What did you do?
Searched with a BooleanQuery with a MustNot occurrence.

What happened?
No documents mached.

What was expected?
Documented behaviour of only excluding documents matching MustNot clause.

Which version of tantivy are you using?
0.16.1
To Reproduce
I was seeing MustNot causing no documents to be returned when used as part of a BooleanQuery with other Must occurrences.
Wrote a minimal test case which does fail: https://gist.github.com/shikhar/775f34ecf80a7fd4db2b62a5b830383c
diff --git a/src/query/boolean_query/boolean_query.rs b/src/query/boolean_query/boolean_query.rs
index 513c81027..f15e46030 100644
--- a/src/query/boolean_query/boolean_query.rs
+++ b/src/query/boolean_query/boolean_query.rs
@@ -206,6 +206,8 @@ impl BooleanQuery {

 #[cfg(test)]
 mod tests {
+    use tantivy_query_grammar::Occur;
+
     use super::BooleanQuery;
     use crate::collector::DocSetCollector;
     use crate::query::{QueryClone, TermQuery};
@@ -226,6 +228,25 @@ mod tests {
         Ok(index)
     }

+    #[test]
+    fn test_must_not() -> crate::Result<()> {
+        let index = create_test_index()?;
+        let searcher = index.reader()?.searcher();
+        let text = index.schema().get_field("text").unwrap();
+        let term_a = TermQuery::new(Term::from_field_text(text, "a"), IndexRecordOption::Basic).box_clone();
+        let not_a = BooleanQuery::new(vec![(Occur::MustNot, term_a)]);
+        let docs = searcher.search(&not_a, &DocSetCollector)?;
+        assert_eq!(
+            docs,
+            vec![
+                DocAddress::new(0u32, 0u32),
+            ]
+            .into_iter()
+            .collect()
+        );
+        Ok(())
+    }
+
     #[test]
     fn test_union() -> crate::Result<()> {
         let index = create_test_index()?;

---- query::boolean_query::boolean_query::tests::test_must_not stdout ----
thread 'query::boolean_query::boolean_query::tests::test_must_not' panicked at 'assertion failed: `(left == right)`
  left: `{}`,
 right: `{DocAddress { segment_ord: 0, doc_id: 0 }}`', src/query/boolean_query/boolean_query.rs:239:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    query::boolean_query::boolean_query::tests::test_must_not

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 500 filtered out; finished in 0.02s

Maybe I'm missing something? I found this assertion https://github.com/tantivy-search/tantivy/blob/46b86a797651e1d09dc9859ee4db705272e6cfa7/src/query/boolean_query/mod.rs#L164-L167
But I would expect docs [0,1,2] to match that https://github.com/tantivy-search/tantivy/blob/46b86a797651e1d09dc9859ee4db705272e6cfa7/src/query/boolean_query/mod.rs#L36-L40

In the issue description you mentioned "with other Must Occurrences". Your unit test only has one MustNot occurrence.
The following works ok for me.
#[test]
    fn test_must_not() -> crate::Result<()> {
        let index = create_test_index()?;
        let searcher = index.reader()?.searcher();
        let text = index.schema().get_field("text").unwrap();
        let term_a = TermQuery::new(Term::from_field_text(text, "a"), IndexRecordOption::Basic).box_clone();
        let term_d = TermQuery::new(Term::from_field_text(text, "d"), IndexRecordOption::Basic).box_clone();
        let a_but_not_d = BooleanQuery::new(vec![(Occur::Must, term_a), (Occur::MustNot, term_d)]);
        let docs = searcher.search(&a_but_not_d, &DocSetCollector)?;
        assert_eq!(
            docs,
            vec![
                DocAddress::new(0u32, 1u32),
                DocAddress::new(0u32, 2u32),
            ]
            .into_iter()
            .collect()
        );
        Ok(())
    }

For the example, you joined, what you should expect is non trivial at all.
It is a bit similar to the question, should empty query return no elements or all documents... And this is something that depends on your application.
In tantivy, (-term_A) is interpreted as (emptyset - term_A).
You can get the resultset you are shooting for by using the AllQuery.

I see, indeed doing an intersection with the AllQuery gives me what I had expected...
Yes this test case is different from my app, where introducing MustNot clauses is causing no documents to be returned even though there are other types of occurrences. I will poke around a bit more on that.

no documents to be returned even though there are other types of occurrences

It was actually:
BooleanQuery {
    Must BooleanQuery { Should .... },
    Must BooleanQuery {
       Must QueryA
       Must BooleanQuery { MustNot QueryB }
    },
}

I'm seeing the expected results after turning this into:
BooleanQuery {
    Must BooleanQuery { Should .... },
    Must BooleanQuery {
       Must QueryA
       MustNot QueryB
    },
}

Thanks again!

Sweet!


Fix invalid reference in gke_github

This example does not work, so fixed invalid reference.
Error: Invalid reference

  on .terraform/modules/flux/examples/gke_github/main.tf line 132, in data "github_repository" "main":
 132:   full_name = "${var.organization}/${repository_name}"

A reference to a resource type must be followed by at least one attribute
access, specifying the resource name.

@kskumgk63 thanks for the contribution 🌹
This change needs to be done in the templates directory. Then you need to run make docsto generate the docs.
https://github.com/fluxcd/terraform-provider-flux/blob/main/templates/guides/gke_github.md.tmpl
Could you also rebase with the latest changes from main?

@phillebaba
Thanks for the review.
I've addressed it.


All az openshift commands fail with a python traceback with 2.0.66

This is autogenerated. Please review and update as needed.
Describe the bug
All az openshift commands fail with a python traceback. This appeared to start with 2.0.66 and I've observed it on both Fedora 30 and RHEL 7.6. In both cases az cli was installed from the Microsoft Azure yum repo. Other az sub-commands do not appear to be affected.
Command Name
az openshift -h
commands such as az openshift scale are also affected and the traceback is similar.
Errors:
The operation 'azure.cli.command_modules.acs.custom#openshift_scale' is invalid.
Traceback (most recent call last):
python2.7/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 507, in execute
    parsed_args = self.parser.parse_args(args)
python2.7/site-packages/knack/parser.py, ln 259, in parse_args
    return super(CLICommandParser, self).parse_args(args)
usr/lib64/python2.7/argparse.py, ln 1705, in parse_args
    args, argv = self.parse_known_args(args, namespace)
...
azure/cli/core/__init__.py, ln 470, in default_description_loader
    op = handler or self.get_op_handler(operation, operation_group=kwargs.get('operation_group'))
azure/cli/core/__init__.py, ln 511, in get_op_handler
    raise ValueError("The operation '{}' is invalid.".format(operation))
ValueError: The operation 'azure.cli.command_modules.acs.custom#openshift_scale' is invalid.

To Reproduce:
Install latest az-cli on Fedora 30 or RHEL 7.6. I have not confirmed if other Red Hat/Fedora releases are affected but I can test those if needed.
run: (or any other az openshift command)

az openshift -h

Expected Behavior
no traceback, help information is displayed in the case of az openshift -h
Environment Summary
Linux-5.1.8-300.fc30.x86_64-x86_64-with-fedora-30-Thirty
Python 2.7.16
Shell: bash

azure-cli 2.0.66

Additional Context

@marstr can this be related to #7523 ? thanks

Nope! #7523 only impacts SUSE based distributions, Fedora is more directly related to Red Hat than that.

azure-cli-2.0.64-1.el7.x86_64 appears to work fine on Fedora 30:
I installed it via:
dnf -y install https://packages.microsoft.com/yumrepos/azure-cli/azure-cli-2.0.64-1.el7.x86_64.rpm
I noticed one difference in installing this package is that it did not pull in python-wheel-wheel and python2-virtualenv as dependancies while 2.0.66 did.
[m@localhost OCP-23927]$ rpm -q azure-cli
azure-cli-2.0.64-1.el7.x86_64
[m@localhost OCP-23927]$ az openshift -h
Group
az openshift : Manage Azure Red Hat OpenShift Services.
Commands:
create : Create a new managed OpenShift cluster.
delete : Delete a managed OpenShift cluster.
list   : List managed OpenShift clusters.
scale  : Scale the compute pool in a managed OpenShift cluster.
show   : Show the details for a managed OpenShift cluster.
wait   : Wait for a managed OpenShift cluster to reach a desired state.

Curious, I just switched over to my Fedora 29 machine to try to repro this. az openshift -h seemed to behave fine for me on my main machine. I also tried installing 2.0.66 of the CLI on a fedora:30 container image in podman, seemed to work for me there to. Anything else you can tell me about your environment, @mgahagan73?
For the moment I'm going to assume this is a packaging/installation error, not directly related to the ACS command_module. I'll reassign this issue to myself for the time being.

@marstr Thanks for the quick response, it turns out that I installed a number of python2 packages on both my F30 system and the RHEL 7.6VM as they were required for some testing I was doing. One of those packages broke all the azure-cli openshift commands. There were quite a few packages and dependences and once I removed them I was able to get 'az openshift -h' working again. I think you are correct in that it is some sort of installation or packaging issue. I saved the list of packages I removed to "fix" the problem, but at the moment I don't have they cycles to go through each one to find what broke az openshift. In the F30 case it was something from the distro, in the 7.6 case it came from pip. I can share the list of packages here if you want but if you need to close this out as a one-off installation issue then I understand. Prior to removing the python packages the last version of azcli that worked for me on those two environments was 2.0.65.

On CentOS Linux release 7.6.1810 (Core)
az aks get-credentials --resource-group ...rg \
    --name ...aks01 --subscription xxxx-xxxx-xxx-xxx

failed with a very similar error.
yum remove azure-cli and yum install azure-cli-2.0.64-1.el7 seems to have fixed the issue, so 2.0.66 seems problematic for everything, not just az openshift.

@dcfsc, can you tell me anything more about your environment? I just install fresh on a Centos 7 container, and am able to use aks commands as expected.

Mine has been upgraded several times, so def not fresh.  Last system update was month ago?
I have pyenv installed (!), and
$ pyenv versions
system
3.5.6
3.6.7

3.7.1 (set by /home/dc/.pyenv/version)
$ pyenv global  system
$ python --version
Python 2.7.5

So system == 2.7.5
I am not a Python guy, so anything else that might be helpful?

Thanks for getting back to me, @dcfsc. That all looks to be in order.
Could you run the following commands and show me the output?
pip check
pip list | grep virtualenv

Also, if you could run az --version and use the absolute path of the python version it mentions to run:
${AZURE_CLI_PYTHON} -m pip list

Sure no problem. Thanks for the help!
$ pip check
No broken requirements found.
$ pip list | grep virtualenv
You are using pip version 19.0.3, however version 19.1.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
$ az --version
...
Python location '/usr/bin/python2'
Extensions directory '/home/dchwalis/.azure/cliextensions'
Python (Linux) 2.7.5 (default, Oct 30 2018, 23:45:53)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-36)]
$ export AZURE_CLI_PYTHON='/usr/bin/python2'
$ ${AZURE_CLI_PYTHON} -m pip list
DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future version of pip will drop support for Python 2.7.

Updating:
 azure-cli                                        x86_64                                        2.0.67-1.el7                                           azure-cli                                         27 M
...
  Updating   : azure-cli-2.0.67-1.el7.x86_64 1/2
  Cleanup    : azure-cli-2.0.64-1.el7.x86_64  2/2
  Verifying  : azure-cli-2.0.67-1.el7.x86_64  1/2
  Verifying  : azure-cli-2.0.64-1.el7.x86_64  2/2
Updated:
  azure-cli.x86_64 0:2.0.67-1.el7

and got the original error:
$ az aks get-credentials  --resource-group XXXrg --name prduse2aks  --subscription XXXX-XXX
The command failed with an unexpected error. Here is the traceback:

The operation 'azure.cli.command_modules.acs.custom#aks_get_credentials' is invalid.
Traceback (most recent call last):
  File "/usr/lib64/az/lib/python2.7/site-packages/knack/cli.py", line 206, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib64/az/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 488, in execute
    self.commands_loader.load_arguments(command)
  File "/usr/lib64/az/lib/python2.7/site-packages/azure/cli/core/__init__.py", line 276, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
  File "/usr/lib64/az/lib/python2.7/site-packages/azure/cli/core/commands/__init__.py", line 288, in load_arguments
    super(AzCliCommand, self).load_arguments()
  File "/usr/lib64/az/lib/python2.7/site-packages/knack/commands.py", line 97, in load_arguments
    cmd_args = self.arguments_loader()
  File "/usr/lib64/az/lib/python2.7/site-packages/azure/cli/core/__init__.py", line 473, in default_arguments_loader
    op = handler or self.get_op_handler(operation, operation_group=kwargs.get('operation_group'))
  File "/usr/lib64/az/lib/python2.7/site-packages/azure/cli/core/__init__.py", line 520, in get_op_handler
    raise ValueError("The operation '{}' is invalid.".format(operation))
ValueError: The operation 'azure.cli.command_modules.acs.custom#aks_get_credentials' is invalid.

I used pyenv to flip back to Python 3.7.1
The get-credentials failed with the same stack.
I also tried using a new K8S cluster in a different subscription, same result.

I think I figured it out. I have some experiments to run with how to resolve it, but basically if you look here:
https://github.com/Azure/azure-cli/blob/7c5f44151010b4b64d822f8cbe3e725f3525a448/scripts/release/rpm/azure-cli.spec#L40-L65
I'm creating a virtual environment, but throwing away bits of it that don't play nice with rpm. I think that the pip reinstalls of our namespace packages aren't getting invoked correctly. I'll need to experiment with some fixes. In the meantime, RPM based Linux users (myself included unfortunately, as I'm a Fedora user) should probably stick to 2.0.65 of the Azure CLI. I'll see if I can't figure out a work around for folks who need the latest functionality.

Reassigning to @zikalino as I've moved to the Notification Hubs team.

add to S166.

RPM is now using python 3 and this issue can not be replicated with azure cli 2.2.0.

Close due to inactivity. Please feel free to reopen it if you still have the same issue with the latest Azure CLI version.


Remove Empty Lines in Default Files (Issue #173)

There were three .coveragerc files with multiple empty lines which was addressed in Issue #173 . This PR contains 3 commits which remove them.

Thanks! With this, these will close the ticket.


bug(python): LanceDB on Cloud Run (GCP) using GCS bucket mount - Generic LocalFileSystem error

LanceDB version
v0.6.10
What happened?
Hello, I am using LanceDB inside a Cloud Run instance on GCP, which is mounted on a GCS bucket (using FUSE). The LanceDB connection is using a local path db = lancedb.connect(“vector_db/lancedb“) which is mounted on a GCS bucket path <bucket-name>/vector_db/lancedb.
So, I am not using directly gs://<bucket-name>/vector_db/lancedb to connect to the GCS bucket.
Also I am creating a table called texts.
However I get this error:
severity=ERROR message="fuse: *fuseops.CreateLinkOp error: function not implemented"

ERROR: LanceError(IO): Generic LocalFileSystem error: Unable to copy file from /vector_db/lancedb/texts.lance/_versions/.tmp_4.manifest_710c8ec5-9856-4086-b8e3-ab180e751bf7 to /vector_db/lancedb/texts.lance/_versions/4.manifest: Function not implemented (os error 38), /home/runner/work/lance/lance/rust/lance-table/src/io/commit.rs:609:54

Could you please let me know if there is a possible solution here?
Thank you
Are there known steps to reproduce?
No response

In order to support atomic commits, the filesystem needs to support certain atomic operations. For local file system, we rely on hard_link. This is an important feature, as without it your table could become corrupted.
GCS mounted with FUSE doesn't seem to have this, which is why it breaks. It seems like it supports rename^1, so maybe it would work if we switched to a rename syscall. But there doesn't seem to be an atomic rename if not exists syscall available in Rust that works across platforms. That's probably why we use hardlink instead. Maybe you could convince gcsfuse to support that?
Is there a reason you aren't using GCS directly? Seems like it would be the simplest workaround.

Thanks for your response. Ok I see, as far as I understand GCS FUSE is not planning to support hard_link.
The main reason I need to use fuse mount mainly has to do with the general architecture of our application, but I also had the impression that using a gs:// connection would be slower (I might be wrong here). It’s an option for sure, which as it seems I would have to implement. Would allowing for an optional fuse_compatibility argument when building the LanceDB connection be a possibility in the future (which would use a rename syscall instead of hard_link only when a user explicitly asks for it)?

I also had the impression that using a gs:// connection would be slower (I might be wrong here)

Unless GCS FUSE does some very fancy caching, I don't see why it would be any faster than directly using the GCS API.

Would allowing for an optional fuse_compatibility argument when building the LanceDB connection be a possibility in the future (which would use a rename syscall instead of hard_link only when a user explicitly asks for it)?

It's something we could think about. Perhaps we could somehow dynamically detect which is supported by the filesystem.

Also, do you think that the same problem would come up in a read-only setup too?

In a read-only setup, you shouldn't need this operation at all. It's just used when committing transactions to the table. So you could just use gs:// for writes and use the FUSE for ready workloads if you preferred that.

Couldn’t find many resources but it seems that gsutil is actually faster than gcsfuse.
Here is an interesting reference: https://gist.github.com/bobbae/1f1f2dcd3f9a2288fca56c9089d6ea37
In that case I guess that using gs:// would be a better choice between the two anyway. Therefore I will simply convert some parts of my code to use gsutil. If you ‘re planning to support fuse you could probably leave the issue as open, otherwise you can close it as far as I ‘m concerned. Thanks


Consider adding public ModelMapper getOrCreate TypeMap capability

I've thought about this numerous times, but one of the drawbacks of the current API is it forces you to think about whether you're getting or creating a TypeMap:
modelMapper.createTypeMap
modelMapper.getTypeMap

Internally though, we safely do a getOrCreate to lookup a TypeMap if it exists, else create it. It would be convenient, and might make sense, to allow users to do the same thing:
TypeMap<Foo, Bar> typeMap = modelMapper.typeMap(Foo.class, Bar.class);

or
TypeMap<Foo, Bar> typeMap = modelMapper.typeMapFor(Foo.class, Bar.class);

This might relieve the need for separate createTypeMap and getTypeMap methods altogether.
Any thoughts?

I'm not sure which is your intension about the method

replacement of createTypeMap and getTypeMap
additional methods other than createTypeMap and getTypeMap

I think you might mean the first one as you had point out

This might relieve the need for separate createTypeMap and getTypeMap methods altogether

I think it will be good api and really convenient that we only need to call typeMap or typeMapFor to create or get TypeMap.
But as a replacement of createTypeMap and getTypeMap, I think it need to have all functionality of createTypeMap and getTypeMap.
So there will be a little confused in some case,
like when we call typeMapFor(Source.class, Destination.class, config),
but there is already a TypeMap<Source, Destination> in store with different config.

I would think of typeMap(Class, Class) as an additional method for createTypeMap(Class, Class) and getTypeMap(Class, Class). We can probably also add typeMap(Class, Class, String).


this.$Progress not defined

Current behavior
When I try and click any link in fresh capybara theme I get this error in console, it cannot resolve this.$Progress. Maybe it has something to do with the components not being registered properly? 464 ... Output of my console is this: https://i.imgur.com/zUMAh8z.png
Expected behavior
Show progress bar and get to desired page/product/etc.
Steps to reproduce the issue

Install fresh current capybara theme
changing theme in tsconfig.json
runing lerna bootstrap && yarn dev

Can you handle fixing this bug by yourself?

[x] NO

Just posting for related information.
It seems the master branch of this repo has a few bugs related to missing references. (SfTab, SfFooterColumn, SfListItem, SfFooterColumn, SfBottomNavigationItem and the one most likely the culprit for this thread is: vue-progress-bar)

found in

---> <DefaultLayout> at src/themes/capybara/layouts/Default.vue```


Define exit prop in useAnimationState

Currently, we can do this:
const state = useAnimationState({
  from: { opacity: 0 },
  to: { opacity: 1 }
})

Since useAnimationState is all on the native thread, and is more performant than using props directly, it would be nice to have an exit option here too:
const state = useAnimationState({
  from: { opacity: 0 },
  to: { opacity: 1 },
  exit: { opacity: 0 } // we should add this
})

Avoiding this for now, unnecessarily complex.


Multi wan support ?

First thanks a lot for this. I deployed it as proposed but I need to update both wan and wan2 ips thru ddns in cloudflare.
UDM pro does not let you create two ddns client with the same provider (hummm) so I deployed two workers with different names dyndns and easydns, but I cannot get this to work properly. Both DNS entries always end up being updated ... with the same ip (from WAN1)
I looked at the generated inadyn.conf but indeed this conf file does not allow to specific different interfaces so I wonder how it is supposed to be able to work ? Thanks !

I still don't understand how the stock implementation in unifyos is supposed to ever be working but I was able to get it to work by modifying manually the /run/inadyn.conf to add to each block respectively
checkip-command = "/sbin/ifconfig eth8 | grep 'inet addr'"
and
checkip-command = "/sbin/ifconfig eth9 | grep 'inet addr'"
since eth8 is wan1 and eth9 is wan2
so each block looks lilke
custom dyndns.MYZONE.workers.dev {
hostname = "MYHOST.MYZONE.COM"
username = "MYZONE.COM"
password = "API_KEY"
ddns-server = "dyndns.MYZONE.workers.dev"
ddns-path = "/update?hostname=%h&ip=%i"
checkip-command = "/sbin/ifconfig eth8 | grep 'inet addr'"
}

did so does the manual config end up working??


atuin doesn't load from .zshrc when it contains the plugin zsh-vi-mode

the line eval "$(atuin init zsh)"  in my .zshrc don't take effect when the plugin zsh-vi-mode (https://github.com/jeffreytse/zsh-vi-mode) is loaded.

Is this no history recording at all, or just the shell bindings don't work? You could run atuin history list to check if the former is working

Since the default initialization mode, zsh-vi-mode will overwrite the previous key bindings, this causes the key bindings of other plugins to fail.
change eval "$(atuin init zsh)" to zvm_after_init_commands+=(eval "$(atuin init zsh)") in your .zshrc.
# Atuin
zvm_after_init_commands+=(eval "$(atuin init zsh)")

See: https://github.com/jeffreytse/zsh-vi-mode#execute-extra-commands


Complete user stories of DG

Closes #35

Codecov Report

Merging #36 (b7fc366) into master (5c2eb19) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #36   +/-   ##
=========================================
  Coverage     71.84%   71.84%
  Complexity      399      399
=========================================
  Files            71       71
  Lines          1236     1236
  Branches        125      125
=========================================
  Hits            888      888
  Misses          315      315
  Partials         33       33

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 5c2eb19...b7fc366. Read the comment docs.


Add comments to unused fields in tests (indirectly required) and remove unused field

coverage: 87.301% (-0.009%) from 87.31%
when pulling 2d9ea254b02fb29a974e58a67adfb8e068530ed7 on hazendaz:next
into 1d936a4a210719e33cbd72f8d1a4ce46e114a71b on mybatis:master.


Jurisdictional Refactor

Go through the jurisdictional refactor outlined in this TDD
QA:

Ensure existing listings have jurisdictions that match the county of their address
Try adding a new unit and ensure that the ami-charts available match the jurisdiction that the listing is assigned to.
On the public side, ensure that the notificationsSignUpUrl is functional on the listings page, and the listing detail page shows utilities included in the additional fees section (if enabled). For reference Test:Coliseum is seeded to have it enabled.

@sarahlazarich when we release this to production the following changes will be noticeable by the users:

New listing will not be required to select a jurisdiction when creating the listing (the list should be the same as county plus a "Bay area" one)
When adding a new user "jurisdictional admin" will now be an option
Preferences will need to be assigned to the correct jurisdiction (this is something we will do for them on existing preferences in the system)
AMI chart dropdown will only show the AMI charts associated to the jurisdiction selected for the listing


java: Use AP timeouts & read output file upon process exit

Based on:

https://github.com/Granulate/gprofiler/pull/264
https://github.com/Granulate/gprofiler/pull/263
https://github.com/Granulate/gprofiler/pull/262

Description
This PR includes 2 changes :sweat_smile: it was convenient to write them together.

We now pass the timeout parameter, added to AP v2.6. This ensures we prevent AP from running forever, if we don't stop it for whatever reason.
We try to read the AP output file even if the profiled process has exited during profiling. This helps when profiling short processes - as the last "duration" of profiling may be lost, if the process exits e.g 50 seconds into the 60 seconds profiling session.

Motivation and Context

AP timeouts - improves robustness; we will automatically stop Java profiling if gProfiler goes down. This completes what we started in https://github.com/Granulate/gprofiler/pull/247.
Reading output file upon process exit - we'll still have the what profiling data was already collected for the process; useful if processes are short.

How Has This Been Tested?
Added 2 tests :) So CI.

[ ] Specifically for the AP timeout, I also ran old gProfiler, pkill -9 it, then ran it again and saw the log "Found started AP". Then tried with this branch - no "Found started AP" message.

Rebased on the 3 PRs.


footnotes seem to be contradictory

source has

ASP.NET codeless attach on App Service only emits metrics in "full" monitoring mode.
ASP.NET codeless attach on App Service, VM/VMSS, and On-Premises emits standard metrics without dimensions. SDK is required for all dimensions

this says that asp.net codeless only does full on app service, then says again it only does standard on app service.  which is it?

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

ID: 2ca73876-268e-c89b-aaaa-0834234c239c
Version Independent ID: 7d408ea5-f7bc-0ed2-5e05-daeab325e294
Content: Log-based and pre-aggregated metrics in Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/pre-aggregated-metrics-log-metrics.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@drdamour
Thanks for your feedback! We will investigate and update as appropriate.

@Marwaabouawad For assistance.

Yes, looks like its confusing. Tagging @AaronMaxwell for visibility, review and document enhancement consideration as appropriate.

Thanks for your dedication to our documentation. This issue is being triaged by multiple product groups before we comment, and we apologize for the further delay that is expected. Thanks in advance for your understanding as we investigate to provide the most accurate answer and documentation updates.

The footnote has been clarified in https://github.com/MicrosoftDocs/azure-docs-pr/pull/218320. We have filed an internal work item for further clarification as this feature matures in the near future.  #please-close

that link 404's


Skip command skips two songs?

Um so i took it for my server, but the skip command seems to skip two songs at the same time when the queue is more thn 2 songs like it works just fine with two songs on queue but i will skip to the third if there is one. ye is it just me or does it do that

No it should not skip two songs unless specified I believe. I will look into it. Thank you.

No it should not skip two songs unless specified I believe. I will look into it. Thank you.

ight i tried to fix it but im so dum i couldnt even find where its given to skip a song is it in the skip command or is there a diff function?

ight i tried to fix it but im so dum i couldnt even find where its given to skip a song is it in the skip command or is there a diff function?

I believe the skip command handles the skip using the global queue list. It's been a bit since I've looked at it. I will look into it after work.

I believe the skip command handles the skip using the global queue list. It's been a bit since I've looked at it. I will look into it after work.

Thank you!

Thank you!

and uh idk if i should open another issue for it but the play playlist command doesnt work if the playlist is made from youtube links, like it works for local but doesnt work for youtube linked one

and uh idk if i should open another issue for it but the play playlist command doesnt work if the playlist is made from youtube links, like it works for local but doesnt work for youtube linked one

Yes, feel free to make an new issue for that one. As I won't be getting to that just now.

Yes, feel free to make an new issue for that one. As I won't be getting to that just now.

Skipping should be fixed now. Please give it a try after pulling changes.

Skipping should be fixed now. Please give it a try after pulling changes.

Yup it completely works, Thank you. Imma go make the playlist issue

Yup it completely works, Thank you. Imma go make the playlist issue


Make Service Instance creation wizard service plan visibility aware

Based off https://github.com/cloudfoundry-incubator/stratos/pull/2086

@richard-cox Made a few tweaks.


Take this article down. This never worked.

Retire this article and create a new one because this never worked.
[Enter feedback here]

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

ID: 8a819f2a-490b-0cdd-7d95-21139a20f4ff
Version Independent ID: 181a621d-0e83-af7d-cb8a-7e63f3195604
Content: Quickstart: Get started - create a Synapse workspace - Azure Synapse Analytics
Content Source: articles/synapse-analytics/get-started-create-workspace.md
Service: synapse-analytics
Sub-service: workspace
GitHub Login: @saveenr
Microsoft Alias: saveenr

@wallycn
Thanks for your feedback! We will investigate and update as appropriate.

@saveenr
Could you please review add comments on this, update as appropriate.

@wallycn
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close

You are all terrible in your work. Escalate my concern to the next higher up manager. I want this team abolished and reformed with better people. Reply back as I will monitor the progress of this or else chase with a higher up plus i will post negative feedback in all social media and tech communities.On 23 May 2024, at 5:57 am, Bill Anderson @.***> wrote:
Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@wallycn I'm sorry about closing the issue. I did so my accident. Thanks for letting us know. I'll reopen and try to get engineering team help to properly address this issue.
#please-open

@wallycn Hello. Can you please explain in more detail about what isn’t working in the article?
I just tested everything in the article myself. I’ll admit that the experience to download the green taxi trip records parquet file was not intuitive. However, I had no other difficulty running through the entire article.

@wallycn I'm closing this issue since we haven’t heard back from you. If you want me to reopen the issue, please provide us with more helpful feedback that we can take action on.
#please-close


Monochrome bitmap VDU command doesn't match documentation

The documentation for the bitmap commands says that VDU 23, 27, 2, ... should be for uploading a monochrome sprite to the VDP, and accept width, height, two colours and data.
The implementation however does not accept any data and accepts only a single colour.  It just creates a solidly filled bitmap.
The version of the command in the documentation is probably more useful.... I can't really imagine much of a use-case for the current implementation.
As an alternative, we may wish to consider supporting other bit depths of bitmaps than the current PixelFormat::RGBA8888 only system.  We could re-purpose VDU 23, 27, 2 for extended/enhanced bitmap uploading.

Hi @stevesims yes well spotted, documentation error. I'll consider your change request, though as it is a breaking change I will probably not use VDU 23, 27, 2 in case it is being used in the wild.

my feeling is that, at this time, there's not all that much use of any particular features in the wild - the community is still pretty small, so we can probably break a few things.  indeed the revised screen mode numbers is already a breaking change, one that's pretty easy for developers to fix their code
in the case of the current VDU 23,27,2 as I said I can't really imagine much of a use-case.  creating a solid bitmap of a single colour when there's no mechanism for adjusting the contents of that bitmap doesn't deliver much.  if the bitmap is being plotted to the screen a developer is more likely to have drawn a filled rectangle for the same functionality.  that leaves using a solid colour bitmap for a sprite frame, and that doesn't feel very useful either.  much like the change in screen modes, these are fairly easy things to work around.
then again, there's plenty more command bytes that can be used for VDU 23,27 - the only real reason to re-purpose 2 would be to keep bitmap command numbers together, separate from the sprite commands

I suggested an update to the documentation here to match what it actually does.


problem

Hello！Can you tell me where the "import imufusion" in your code comes from?

"problem" is not an appropriate title for an issue.  Please retitle this issue so that it will be useful to others.
imufusion is a Python package and is distributed on PyPI.  It is installed the same way as any other Python package, using pip install.

Okay, I've found it, thank you


dark theme for UI

creating a dark theme for the UI would make late-night hacking a lot more pleasant.

true, we would then start a css dance, but I think this should be manageable (most of the style is in style.css)

i know there are probably lots of other more pressing issues - I just wanted to add this to the 'nice to have' list :-)

Indeed, I agree that this skinable thingy would be great actually! And the dark theme is interesting, for instance I always use a dark theme in my sublime

Are we there yet?

IMHO reliability and a single, but beautiful UI is of greater importance.
so let's better improve the default white style first.


AlternateSecurityIDs should be AlternativeSecurityIds.

In the referenced troubleshooting guide, Tushar Pathak writes about the AlternateSecurityIDs attribute, but this is a typo.

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

ID: f737e278-7c89-e048-b60b-287615a829fb
Version Independent ID: 1df35258-7360-74c4-680a-e154c54291fe
Content: Microsoft Teams Guest Access checklist
Content Source: Teams/guest-access-checklist.md
Service: msteams
GitHub Login: @romanma
Microsoft Alias: romanma

@Mike-Crowley Thank you for catching this. I will make sure this gets updated.
I was able to get this into the backlog for the Teams/Skype for Business team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@Mike-Crowley Thanks for pointing this out and for your contribution.  I will investigate the issue to correct the typo.

@David-Rosenthal please can you have a look at the troubleshooting guide in the blog from Tushar Pathak. @Mike-Crowley mentions a typo in there in his message above.

I only cover Teams in Gov :) The PM for Guest Access is @somakbhattacharyya

@somakbhattacharyya could you please assist with above issue.

@lindspea  Thank you very much for the contribution and following with bog owner. @Mike-Crowley  Blog owner has confirmed that he will update this typo. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Use plugin discovery from plugin environment

Use plugin discovery from plugin environment
:heavy_check_mark: Checklist

[ ] Added tests for new functionality and regression tests for bug fixes
[ ] Added changeset (run yarn changeset in the root)
[ ] Screenshots of before and after attached (for UI changes)
[ ] Added or updated documentation (if applicable)

I dont think this needs a changeset because the change is in the example app.

@alexef fyi