What is the minimum supported of the ASA SDK?

What is the minimum supported of the ASA SDK?

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

ID: 8d172218-934f-1431-be92-4ad6897a31e9
Version Independent ID: 124f517a-9531-838d-3834-a21f94da9fdb
Content: Install Azure Spatial Anchors for Unity - Azure Spatial Anchors
Content Source: articles/spatial-anchors/how-tos/setup-unity-project.md
Service: azure-spatial-anchors
GitHub Login: @msftradford
Microsoft Alias: parkerra

Acknowledged. Working on a PR for this.


Corrupt zipfiles crashes program

When doing Auto Tag on a large batch of files, the program crashes when it encounters a corrupt zipfile.
Opening the file in comictagger works fine, it's when you try to save that comictagger crashes.
2022-06-06T12:27:10 | comicapi.comicarchive | ERROR | Error reading zip archive [[Errno 22] Invalid argument]: /media/Collections/Comics/Incoming/_cbz/C/Chainsaw Man v07 (2021) (Digital) (danke-Empire).cbz :: Chainsaw Man - c053 (v07) - p000 [Cover] [dig] [In a Dream] [VIZ Media] [danke-Empire].jpg
2022-06-06T12:27:10 | comicapi.comicarchive | ERROR | Error reading zip archive [[Errno 22] Invalid argument]: /media/Collections/Comics/Incoming/_cbz/C/Chainsaw Man v07 (2021) (Digital) (danke-Empire).cbz :: Chainsaw Man - c053 (v07) - p000 [Cover] [dig] [In a Dream] [VIZ Media] [danke-Empire].jpg
2022-06-06T12:27:10 | comicapi.comicarchive | ERROR | Error reading in page 0. Substituting logo page.
Traceback (most recent call last):
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 850, in get_page
    image_data = self.archiver.read_file(filename) or bytes()
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 217, in read_file
    data = zf.read(archive_file)
  File "/usr/lib/python3.10/zipfile.py", line 1473, in read
    with self.open(name, "r", pwd) as fp:
  File "/usr/lib/python3.10/zipfile.py", line 1528, in open
    fheader = zef_file.read(sizeFileHeader)
  File "/usr/lib/python3.10/zipfile.py", line 744, in read
    self._file.seek(self._pos)
OSError: [Errno 22] Invalid argument
2022-06-06T12:27:10 | comicapi.comicarchive | ERROR | Error reading in page 0. Substituting logo page.
Traceback (most recent call last):
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 850, in get_page
    image_data = self.archiver.read_file(filename) or bytes()
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 217, in read_file
    data = zf.read(archive_file)
  File "/usr/lib/python3.10/zipfile.py", line 1473, in read
    with self.open(name, "r", pwd) as fp:
  File "/usr/lib/python3.10/zipfile.py", line 1528, in open
    fheader = zef_file.read(sizeFileHeader)
  File "/usr/lib/python3.10/zipfile.py", line 744, in read
    self._file.seek(self._pos)
OSError: [Errno 22] Invalid argument
2022-06-06T12:27:18 | comicapi.comicarchive | ERROR | Error reading zip archive [[Errno 22] Invalid argument]: /media/Collections/Comics/Incoming/_cbz/C/Chainsaw Man v07 (2021) (Digital) (danke-Empire).cbz :: Chainsaw Man - c053 (v07) - p000 [Cover] [dig] [In a Dream] [VIZ Media] [danke-Empire].jpg
2022-06-06T12:27:18 | comicapi.comicarchive | ERROR | Error reading in page 0. Substituting logo page.
Traceback (most recent call last):
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 850, in get_page
    image_data = self.archiver.read_file(filename) or bytes()
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 217, in read_file
    data = zf.read(archive_file)
  File "/usr/lib/python3.10/zipfile.py", line 1473, in read
    with self.open(name, "r", pwd) as fp:
  File "/usr/lib/python3.10/zipfile.py", line 1528, in open
    fheader = zef_file.read(sizeFileHeader)
  File "/usr/lib/python3.10/zipfile.py", line 744, in read
    self._file.seek(self._pos)
OSError: [Errno 22] Invalid argument
2022-06-06T12:27:18 | comicapi.comicarchive | ERROR | Error reading zip archive [Bad magic number for file header]: /media/Collections/Comics/Incoming/_cbz/C/Chainsaw Man v07 (2021) (Digital) (danke-Empire).cbz :: Chainsaw Man - c053 (v07) - p001 [dig] [In a Dream] [VIZ Media] [danke-Empire].png
2022-06-06T12:27:18 | comictagger | CRITICAL | Uncaught exception: BadZipFile: Bad magic number for file header
Traceback (most recent call last):
  File "/home/scuttle/.local/lib/python3.10/site-packages/comictaggerlib/taggerwindow.py", line 1086, in commit_metadata
    success = self.comic_archive.write_metadata(self.metadata, self.save_data_style)
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 817, in write_metadata
    retcode = self.write_cix(metadata)
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 1026, in write_cix
    self.apply_archive_info_to_metadata(metadata, calc_page_sizes=True)
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 1161, in apply_archive_info_to_metadata
    data = self.get_page(idx)
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 850, in get_page
    image_data = self.archiver.read_file(filename) or bytes()
  File "/home/scuttle/.local/lib/python3.10/site-packages/comicapi/comicarchive.py", line 217, in read_file
    data = zf.read(archive_file)
  File "/usr/lib/python3.10/zipfile.py", line 1473, in read
    with self.open(name, "r", pwd) as fp:
  File "/usr/lib/python3.10/zipfile.py", line 1533, in open
    raise BadZipFile("Bad magic number for file header")
zipfile.BadZipFile: Bad magic number for file header

@ScuttleSE if this happens again feel free to re-open

@lordwelch  thou has happened with me

@edgehub-cloud please open a new issue and provide logs from when the issue happened


Unable to upload a file

I am trying to upload a file using AML:
Here is my AML:

Bill Submission
1

Bill 1
G:\abc.docx

but not able to upload file.

I'm closing this as a duplicate of #180 so that we can continue the conversation there.


add exec delete timeout

add exec delete timeout

@mingfukuang you'll need to rebase off main to pull in changes to make CI checks happy across your PRs

@mingfukuang you'll need to rebase off main to pull in changes to make CI checks happy across your PRs

It's done , pls have a look. thanks.

the GitHub UI is still showing that there are merge conflicts, and it looks like your diff contains a lot of prior changes that are already on main

conflicts exists at here:
[dependencies]
async-trait = { workspace = true, optional = true }
<<<<<<< f7
protobuf = "=3.1"
protobuf = "=3.5"

main
ttrpc = "0.8"

but this conflicts could be  still  exists in commit 6a5c9c26295b1e2c1eb14aa1b22edb17f0b07f86:
crates/shim-protos/Cargo.toml :
$ git reset --hard 6a5c9c26295b1e2c1eb14aa1b22edb17f0b07f86
HEAD now  6a5c9c2 fix: Correct clippy lint suggestion part tres
$ cat crates/shim-protos/Cargo.toml |grep protobuf
protobuf = "=3.1"

I'm not sure how to handle it properly.


Non blocking

Some updates and breaking changes
updates

async call objects now have a .result, .error and .status property
wait -> await_result.
tried to make the logic for on_result() and on_error() clearer by moving it to JS.
added some client tests to the end of the module

breaking changes:

Interval - the .interval property was replaced by .delay

this means that setting delay = 0 causes the interval function to run every 0 seconds

Interval clear_interval() -> clear()

The breaking changes are there for consistency between Timeout and Interval.
I can make this backward compatible.
But I think the number of users affected by these changes might be close to zero.

And they were clearly warned!


Page is not available

https://pydfs-lineup-optimizer.readthedocs.io/en/latest
is not available

I can open the docs, maybe readthedocs had some problems

Probably was temporary problem


Forward ported CLI tests from Flask-CLI and fixed a bug with the CLI's name.

To clarify, looking at the signature of the click's Group class I found that passing the whole Flask app isn't what it expects, but just a command group name.

Excellent, thanks!


Fix multi-tuner detection on Linux and support Android device naming

Fixes #596

Hi,
There are a few things I don't understand in the code. Did you test it on Linux and Android?
The function replace() seems equivalent to UString::substitute().
ExpandWildcard() is called several times in a row. Each call erases the previous content of the output string array. Did you mean ExpandWildcardAndAppend()?
I don't understand this:
// If not specified, use frontend index for demux
if (fcount < 3) {
    demux_nb = frontend_nb;
}

// If not specified, use frontend index for dvr
if (fcount < 4) {
    dvr_nb = frontend_nb;
}

It seems that, on Linux, /dev/dvb/adapter2 will end up with demux /dev/dvb/adapter2/demux2, which seems incorrect to me.
Could you explain exactly what is the DVB device naming on Android? I assume that this is in fact "Android TV" on set-top box, right?
I thought that Android relied on a Linux kernel. So, why is the DVB device naming different? Are the DVB device drivers modified?

There are a few things I don't understand in the code. Did you test it on Linux and Android?
ExpandWildcard() is called several times in a row. Each call erases the previous content of the output string array. Did you mean ExpandWildcardAndAppend()?

Yes exactly, that's the only flaw in the PR. I hadn't tested that last change on Android yet.
We have some kind of super-static binaries that work on all (at least most) Linux distros and even on Android without doing a separate compilation. That's why I've merged the code and removed the conditional compilation (that I had added for Android) just few days ago.

The function replace() seems equivalent to UString::substitute().

Thanks, I thought you must have something suitable, but the naming is s a bit unexpected :-)

I don't understand this:
// If not specified, use frontend index for demux
if (fcount < 3) {
    demux_nb = frontend_nb;
}

// If not specified, use frontend index for dvr
if (fcount < 4) {
    dvr_nb = frontend_nb;
}

It seems that, on Linux, /dev/dvb/adapter2 will end up with demux /dev/dvb/adapter2/demux2, which seems incorrect to me.

No, that's not the case. The number behind 'adapter' will never be parsed and will always exist as part of the string only.

When device_name is /dev/dvb/adapter2, the frontend_nb will be 0 and demux_nb and dvr_nb will just be set to the same.
=> No change here

When device_name is /dev/dvb/adapter2:1, the frontend_nb will be 1 and demux_nb and dvr_nb will be set to 1 as well.
=> That's the important change: It's a much better assumption that demux and dvr will have the same number as the frontend than assuming them to be always 0.

I'm not sure whether cases even exist where demux and dvr would have different numbers, but those are definitely rare and in such cases, it's still possible to specify /dev/dvb/adapter2:1:8:8.
GetAllTuners:
GetAllTuners() will now always return device names including the frontend number as one of these patterns:

/dev/dvb/adapterX:Y
or
/dev/dvbX:Y

Could you explain exactly what is the DVB device naming on Android? I assume that this is in fact "Android TV" on set-top box, right?

Correct, but in order to access devices natively, you need either a rooted device or approval by the device vendor.

I thought that Android relied on a Linux kernel. So, why is the DVB device naming different? Are the DVB device drivers modified?

I can't say why, but it seems that both naming variants exist:
https://android.googlesource.com/platform/frameworks/base/+/master/services/core/java/com/android/server/tv/TvInputManagerService.java#108

It seems that, on Linux, /dev/dvb/adapter2 will end up with demux /dev/dvb/adapter2/demux2, which seems incorrect to me.

No, that's not the case. The number behind 'adapter' will never be parsed and will always exist as part of the string only.

Yes, you are right. There was a confusion in my mind between adapter number (never identified as an integer) and fontend number. Sorry about that.

I thought that Android relied on a Linux kernel. So, why is the DVB device naming different? Are the DVB device drivers modified?

I can't say why, but it seems that both naming variants exist:
https://android.googlesource.com/platform/frameworks/base/+/master/services/core/java/com/android/server/tv/TvInputManagerService.java#108

Did you ever encounter the "flat" structure without adapter directories? Do you think it is Android-specific or could it be some legacy Linux structure that the Android code included for completeness?
In #596, you show a directory tree of one adapter with 16 frontends under adapter0. Was it produced on Android? Is it a regular Android TV or an AOSP one?
About this one, there are 16 frontends (which makes sense for 16 tuners) but there are 23 demux, 23 dvr, 23 net and 23 ca. 23 ? Why 23 ?

Did you ever encounter the "flat" structure without adapter directories? Do you think it is Android-specific or could it be some legacy Linux structure that the Android code included for completeness?

OK, I can now answer this one. This is apparently a very old Linux legacy, prior to kernel 2.4. I have found references to that change in Dec 2004 here. I have found other references from 2005 about users being puzzled about the change from flat to hierarchical structure.
I have also found references to the reason: before some kernel version (which one?), a driver could not create device nodes and directories and a static flat structure was required. After that kernel version, drivers were able to create device nodes and directories and then came the structured DVB device nodes.
Personally, I started to use DVB tuners on Linux in 2005 and never encountered the flat structure. Since it was long before the advent of Android, I do not think any Android device will ever see the flat structure (if the references I found are correct).
So, unless there is some evidence of the use of the flat structure in the last 10 years, I think we can simply drop the flat structure.
On the other hand, the modification looking for all fontends in GetAllTuners is still necessary.

So, unless there is some evidence of the use of the flat structure in the last 10 years, I think we can simply drop the flat structure.

10 years is the Linux story, but not the Android story...
Actually I'm working with a recent device, (delivered with Android Pie) that exposes the flat structure, otherwise I would have never come to that.

In #596, you show a directory tree of one adapter with 16 frontends under adapter0. Was it produced on Android? Is it a regular Android TV or an AOSP one?

That wasn't Android at all. It was armv8 Linux. Here are the results after this PR's change:
FBC_new2.txt
When you look at the device's product page that I referenced, it should become clear that it's not intended to be used like regular tuner hardware. It's some kind of set-top box that comes with its own software and not meant to run any other application. One could also say that: "we ran tslsdvb inside the firmware of the STB".

About this one, there are 16 frontends (which makes sense for 16 tuners) but there are 23 demux, 23 dvr, 23 net and 23 ca. 23 ? Why 23 ?

I could explain 24, but not 23...
The device includes 2 PCIex modules:

1 module providing 8 DVB-S tuners
1 module providing 8 tuners for either DVB-T or DVB-C

3x8 = 24... but not 23
Eventually it's probably irrelevant as it's not the kind of device meant to be accessed via DVB API.

10 years is the Linux story, but not the Android story...
Actually I'm working with a recent device, (delivered with Android Pie) that exposes the flat structure, otherwise I would have never come to that.

OK, weird. This is the first time I see the flat structure. So, just keep support for the flat structure. Since the driver decides the device structure, it is possible that drivers for specific embedded tuners in set-top boxes use the old (very old) structure.

I could explain 24, but not 23...
The device includes 2 PCIex modules:
. 1 module providing 8 DVB-S tuners
. 1 module providing 8 tuners for either DVB-T or DVB-C
3x8 = 24... but not 23

When a tuner provides DVB-T and DVB-C, then either you have two frontends, one for each modulation, or one single fontend and you need to specify the right "delivery system" when you tune. But, in all cases, the number of demux and dvr are the same as the number of frontends.
This can be an issue with TSDuck. In the tslsdvb output you provided, the frontends 0 to 7 are identified as DVB-C and 8 to 15 as DVB-S. There is no DVB-T. There is something to analyze here.
In the last code:
    UStringVector names;
    ExpandWildcard(names, u"/dev/dvb/adapter*");

    // Android
    ExpandWildcardAndAppend(names, u"/dev/dvb*.frontend*");

    // Linux
    ExpandWildcardAndAppend(names, u"/dev/dvb/adapter*/frontend*");

Do you still need the first one? It seems redundant with the third one.
And the comments are probably not adequate. It does not seem to be a matter of Linux vs. Android but rather a matter of legacy for the flat structure.

This can be an issue with TSDuck. In the tslsdvb output you provided, the frontends 0 to 7 are identified as DVB-C and 8 to 15 as DVB-S. There is no DVB-T. There is something to analyze here.

To give you an example of a combined DVB-T / DVB-C tuner, here is the output of an AverMedia TD 310. You can see that the device is identified as "DVB-T, DVB-T2, DVB-C". There is only one tuner but it supports 3 different delivery systems. Use -I dvb --delivery-system DVB-2 for instance to select one.
Note that, on Windows, the same device appears as two distinct tuners, a DVB-T/T2 one and a DVB-C one. I think that this is due to some limitation in the DirectShow framework. Since there is only one physical tuner anyway, the "two logical tuners" on Windows cannot be used at the same time.
$ tslsdvb -v

/dev/dvb/adapter0 (Silicon Labs Si2168, DVB-T, DVB-T2, DVB-C)

tslsdvb: error reading status on /dev/dvb/adapter0/frontend0: Resource temporarily unavailable
  Delivery system ............................ DVB-T
  Frequencies:
    Min ................................. 48,000,000 Hz
    Max ................................ 870,000,000 Hz
    Step .................................... 62,500 Hz
    Tolerance .................................... 0 Hz
  Spectral inversion ........................... off
  Modulation ...................................  It seems redundant with the third one.

I apologize for the inaccuracy. I'm stuck at an earlier version (before introduction of the "guts"), so I'm just pasting the relevant changes into the GitHub web editor..

And the comments are probably not adequate. It does not seem to be a matter of Linux vs. Android but rather a matter of legacy for the flat structure.

Fixed ;-)

This can be an issue with TSDuck. In the tslsdvb output you provided, the frontends 0 to 7 are identified as DVB-C and 8 to 15 as DVB-S. There is no DVB-T. There is something to analyze here.

I don't think it's worth the effort. It's a closed system, not meant for use by applications, and as such it's sufficient that it makes sense to themselves. Maybe there's a jumper on a circuit board to switch to DVB-T, maybe a certain ioctl command or whatever it may be - it doesn't really matter.

To give you an example of a combined DVB-T / DVB-C tuner, here is the output of an AverMedia TD 310. You can see that the device is identified as "DVB-T, DVB-T2, DVB-C". There is only one tuner but it supports 3 different delivery systems. Use -I dvb --delivery-system DVB-2 for instance to select one.
Note that, on Windows, the same device appears as two distinct tuners, a DVB-T/T2 one and a DVB-C one. I think that this is due to some limitation in the DirectShow framework. Since there is only one physical tuner anyway, the "two logical tuners" on Windows cannot be used at the same time.

It's not a technical limitation. but a historical one: Some applications (including WMC) didn't provide a way to select a delivery system, and a device manufacturer wants to be compatible with all applications. That's my interpretation why it has evolved like that..


Headers parsing

Implementations of HPUB and HMSG messages.
Also includes minor code clean-up and some test improvements to reduce false positives.

Implementation LGTM - looks like a test might be flapping though because the CI failed

CI was failing because of a change on the server reporting WS port instead of standard port which broke cluster reconnect test. There was also an issue reading the server version in tests. All issues in test code and they're fixed now.


[Feature] Support globbing with includes directive

It would be amazing to be able to do !include path/to/my/subjusts/*.just and move on with life vs including each seperately.

I would love this feature also. I used it a lot with make to divide makefiles into like functionality.
Here is a workaround. The update recipe will scan a folder for all *.just files, format them, create a temporary Justfile with all the !include directives, then dump them to a single Justfile. I have found !include to be too buggy for daily use in the primary Justfile.
Initial Justfile:
!include ./justfiles/self.just
./justfiles/self.just:
justfile_dir := "./justfiles"

update:
    #!/usr/bin/env bash
    set -euxo pipefail
    cp Justfile Justfile.bak
    echo "# DO NOT EDIT. Auto-generated. Update with 'just update'." > Justfile.includes
    for file in {{ justfile_dir }}/*.just; do
        echo "!include ${file}" >> Justfile.includes
        just --justfile "${file}" --fmt --unstable || true
    done
    just --unstable --justfile Justfile.includes --dump > Justfile
    rm Justfile.includes

After making any changes to a *.just file in your watched folder, run:
just update

This seems reasonable. include has been stabilized as import, so this wouldn't technically be backwards compatible, but I think that * in paths is so vanishingly rare that we can go ahead and do it.


About the training loss

I read your code about the design for loss and found your implementation is different from that proposed in the paper. So you use the traditional loss of GAN instead of the WGAN loss? Does it mean WGAN loss might not be a good choice in practice?

The result images shown in the main body of the paper are generated with
WGAN. After the acceptance of the paper, I did some further experiments and
found that the original GAN loss actually help improves the quality of
output images, for the datasets used in the paper. Though I cannot conclude
that the traditional GAN loss is always better than wGAN loss in terms of
the DualGAN architecture.
If you have interests in further exploration, you may try WGAN-GP or other
losses.
On 11 July 2018 at 16:28, Yihong Gu<EMAIL_ADDRESS>wrote:

I read your code about the design for loss and found your implementation
is different from that proposed in the paper. So you use the traditional
loss of GAN instead of the WGAN loss? Does it mean WGAN loss might not be a
good choice in practice?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/duxingren14/DualGAN/issues/22, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACc6e6kdp3q45BB2Lvksqqnz_PcINmmRks5uFooggaJpZM4VL_lS
.

Why is noise z not used?
@duxingren14

Instead of explicitly adding random noise, I used dropout instead.


Discoverer should not retry deleting a service/endpoint that does not exist

While doing performance testing, I noticed that the discovery components are wasting cycles trying to delete services/endpoints that no longer exist. This is currently impacting other tests (which can be thought of as other gimbal teams) because the discoverer is busy trying to delete non-existent services, instead of discovering new services for other teams.
How does this happen? The performance test creates a namespace where it creates hundreds of services and endpoints in the backend cluster. These are then discovered by Gimbal and everything is fine.
Once the test is done, instead of deleting services/endpoints one by one, the performance test script destroys the test namespace on both the gimbal cluster and the backend cluster. The deletion of the namespace in the backend cluster results in watch notifications sent to the discoverers about the deletion of services. The problem is that the services/endpoints have already been deleted in the gimbal cluster, because the entire namespace was deleted.

How many threads are set up on your discoverer? Could you retest with a higher number? Just curious how that might have (a positive) effect on the performance of the queue.

Originally had 2, bumped it up to 16 but still seeing this issue. Will try bumping up even higher, but not sure how much it'll help.

Having more should process the queue faster since they should operate in parallel. But just a thought before going down a design route to see how to resolve.

I haven't thought through the implications, but I think we should be able to swallow the "does not exist" error when we are trying to delete, instead of raising it up to the queue for re-queuing.
What's unclear to me is if we can naturally get into this state by somehow processing a delete notification before processing the corresponding add, and thus end up with an extra service that should have been deleted.

These charts are interesting.
We can see when the namespace is deleted and all services are gone, but the discoverer is still trying to delete them, and thus accumulating errors.
Looks like it took approximately 20 minutes to drain the queue.

Memory usage of discoverer:

Ohh I understand now, I didn't realize that we were re-queuing each time on error. I think we could handle the "does not exist" or put a max retry on the queue, once an item is attempted 2 times, then just drop from the queue. That would solve the same scenario where maybe an update or add errors (for whatever reason) and we keep trying.

Yeah, I actually think we need both... It doesn't look like we have a max retry on the queue item. I'll pick this up and look into it more.

Not sure if we'd want a retry metric or not t track how many items are re-tried, or thrown out because if error.

Now that we know we were not actually retrying before #141, the issue title is somewhat misleading.
What is actually going on is that the namespace is deleted from under the discoverer on both clusters, resulting in the API server from the backend cluster sending a long list of DELETE watch notifications to the discoverer. This is the expected behavior.
Now that we have added retries though, we should probably not attempt to retry a deletion for a service or endpoint that does not exist.
Another idea (which needs a bit more thought): Have a separate queue per namespace, so that the operations performed by one team do not affect other teams. This would be a larger change that requires discussion and design though (cc @rosskukulinski @stevesloka).

My guess that thousands of services within a namespace is unlikely. I would think though, that bringing up a new cluster could see a high input of data, also, anytime the re-sync window is hit on a large cluster.
This was my reason for the re-queue param, you may not want to retry more than once if something fails.


TracingConfiguration.EmitTo.* to replace ILogger argument of EnableTracing()

TracingConfiguration currently sets up the activity listener to write events through to the static Log.Logger by default.
This can be changed by passing an ILogger to EnableTracing(), but since the rest of the API is now fluent, it's not very discoverable or consistent keeping this argument here.
EmitTo.Logger(ILogger) and EmitTo.StaticLogger() might be reasonable names to use?
On semantics, the default should still be to write to the static logger. Serilog configuration usually allows defaults to be re-specified, for better or worse.
There's only a single target logger to configure, so last in will win.

Frankly, for me setting this as a parameter of EnableTracing seems most logical. "You want to trace it? Show me where to trace it!"

Doing it via EnableTracing() might work better if we renamed that method to two variants - TraceTo(ILogger) and TraceToSharedLogger()


Add ex_fetch_power_state option to list_nodes().

Add ex_fetch_power_state option to list_nodes().
Description
Allows caller to skip (potentially expensive) requests for extended node
power state status.
Also set timeout on connection object used to fetch token.
Status
Ready for review
Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[x] Documentation
[ ] Tests
[x] ICLA (required for bigger changes)

@tfmorris Thank you for the review!
@tetron Can you please fix the lint, possibly by moving the status fetch in another function?

It's been 20 minute since I pushed and Travis hasn't started running tests yet.  I don't know if it is a rate limit thing or Travis is having trouble (but currently no downtime is reported on their status page).  Guess I'll try to be patient.

@tetron Heh. It did take some time to run, and there's one lint issue remaining. You can use tox -e lint to get results faster!

Codecov Report

Merging #1126 into trunk will decrease coverage by 0.01%.
The diff coverage is 44.73%.

@@            Coverage Diff             @@
##            trunk    #1126      +/-   ##
==========================================
- Coverage   85.45%   85.43%   -0.02%
==========================================
  Files         346      346
  Lines       66272    66288      +16
  Branches     5899     5905       +6
==========================================
+ Hits        56631    56635       +4
- Misses       7239     7250      +11
- Partials     2402     2403       +1

Impacted Files
Coverage Δ

libcloud/common/azure_arm.py
73.01% <100%> (ø)
:arrow_up:

libcloud/compute/drivers/azure_arm.py
49.82% <43.24%> (-0.71%)
:arrow_down:

Continue to review full report at Codecov.

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

@tetron Thanks! We just need tests for the ex_fetch_power_state=False branch to ensure coverage does not decrease.

Thanks @tetron @ldipenti this is now merged in trunk.

Great, thanks @pquentin !


New spells have buggy XSpellIsActive callbacks

Example:
https://github.com/GrognardsFromHell/TemplePlus/blob/183509ba001ac320932883df702d3250b7c48df0/tpdatasrc/tpgamefiles/scr/tpModifiers/sp_moment_of_prescience.py#L108
This will return 1 regardless of the spell enum being queried. Looks like a forgotten placeholder.

Good find.  I'll make sure something like this doesn't effect any other spells and should have a pull request on Monday.


Error Line 28

When i try ti use i recive this error:
File "asnban.py", line 28
def print_prefixes(prefixes: typing.List[str], args):
^
SyntaxError: invalid syntax

Are you using Python 3.5 or later?

O yes with python3 recive this error:
File "asnban.py", line 53, in
from providers import bgptools, cymru
File "bgptools.py", line 4, in
from bs4 import BeautifulSoup
ModuleNotFoundError: No module named 'bs4'
but i right installed beautifulsoup4 4.7.1

Not sure if this issue is stale but you may have installed it in to the wrong python version? I would recommend creating a virtual python environment with virtualenv:  https://packaging.python.org/en/latest/guides/installing-using-pip-and-virtual-environments/
I tested my code with pytho 3.8 and 3.10 so it definitely should work with those! Let me know if you have any other issues :)


Adding css unminified to src folder

Thank you for updating the src folder with the latest branch. Could you also make an unminified css file in that same folder. We have found a need to make some modifications in that regard.
Thanks so much.
Great widget.
Stewart Fishman
Allianz Life Insurance Company

Just added! Thanks!


Update your docs for testnet section

As for me some details of getting TOMNI are still unclear(and as google shows not just for me). Steps I do

Deposit account(doesn't matter what btc address I use because I have no ability to choose address 'from',  when use sendtoaddress method)
I use sendtoaddress to moneyqMan7uh8FqdCA2BV5yZ8qVrc9ikLP.
I receive some TOMNI.
Yes, acctually I receive tomni, my wallet is credited but some random address receives this tomni and not a particular address. So my question is - Is that the right case? For example, I want to get tomni on address mxpjTJy7Pv9pi64XxQ2H7TQuSiDU8XKjER, what I do now -  I search for address that was credited with tomni and then send them to  mxpjTJy7Pv9pi64XxQ2H7TQuSiDU8XKjER.
So it would be great if u add more details about tesnet usage.

Hello there,
you are correct. You received the TOMNI at a random address, because sendtoaddress just uses a random address with BTC in it for the send.
You could either then forward the TOMNI to the actual destination (e.g. with omni_send), or you may create a raw transaction, which spends BTC from the actual source you'd like to use (e.g. with listunspent to list unspent outputs, create a raw transaction with createrawtransaction, then sign it with signrawtransaction and finally send it with sendrawtransaction).


Crash after migrating from GCM to FCM

We are using this library for period scheduling of in app notification in our app for almost a year. Yesterday, we release an app update for 50% of the users in which we have migrated from GCM to FCM. Crashlytics is reporting a lot of this error. Is it really a crash or just warning. Because I didn't see any app has stopped working dialog on my one plus 5t(android pie) after the app update. and How to resolve this ?
I went through the workaround referred in this link:
https://github.com/evernote/android-job/wiki/FAQ#how-can-i-remove-the-gcm-dependency-from-my-app
but Android Studio is showing error: PlatformGcmService is not assignable to Service.
Crash is occurring even on Android Nougat and Oreo but according to above link it should occur in Android 4.x
Fatal Exception: java.lang.RuntimeException: Unable to instantiate service com.evernote.android.job.gcm.PlatformGcmService: java.lang.ClassNotFoundException: Didn't find class "com.evernote.android.job.gcm.PlatformGcmService" on path: DexPathList[[zip file "/data/app/com.tf.eros.faythTv-vjFwvbm9Y8zq3iJ6sGNdxQ==/base.apk"],nativeLibraryDirectories=[/data/app/com.tf.eros.faythTv-vjFwvbm9Y8zq3iJ6sGNdxQ==/lib/arm64, /system/lib64, /system/vendor/lib64]]
       at android.app.ActivityThread.handleCreateService(ActivityThread.java:3529)
       at android.app.ActivityThread.-wrap4(Unknown Source)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1786)
       at android.os.Handler.dispatchMessage(Handler.java:105)
       at android.os.Looper.loop(Looper.java:164)
       at android.app.ActivityThread.main(ActivityThread.java:6944)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:327)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1374)

Caused by java.lang.ClassNotFoundException: Didn't find class "com.evernote.android.job.gcm.PlatformGcmService" on path: DexPathList[[zip file "/data/app/com.tf.eros.faythTv-vjFwvbm9Y8zq3iJ6sGNdxQ==/base.apk"],nativeLibraryDirectories=[/data/app/com.tf.eros.faythTv-vjFwvbm9Y8zq3iJ6sGNdxQ==/lib/arm64, /system/lib64, /system/vendor/lib64]]
       at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:93)
       at java.lang.ClassLoader.loadClass(ClassLoader.java:379)
       at java.lang.ClassLoader.loadClass(ClassLoader.java:312)
       at android.app.ActivityThread.handleCreateService(ActivityThread.java:3526)
       at android.app.ActivityThread.-wrap4(Unknown Source)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1786)
       at android.os.Handler.dispatchMessage(Handler.java:105)
       at android.os.Looper.loop(Looper.java:164)
       at android.app.ActivityThread.main(ActivityThread.java:6944)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:327)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1374)

UPDATE 1:
So I added PlatformGcmService as mentioned in FAQ and also added gcm dependency implementation 'com.google.android.gms:play-services-gcm:16.0.0' . After that I was able to build my project after a clean build.
But is it safe to have both gcm and fcm dependency under the same project. Any possible downside of this?

You're running into this: https://github.com/evernote/android-job/wiki/FAQ#how-can-i-remove-the-gcm-dependency-from-my-app
The library is not using GCM, but only optional the GcmNetworkManager. Although the name implies it, the class is unrelated to GCM.

@vRallev thanks for the quick reply. I have applied the changes mentioned in above link and also added
gcm dependency implementation 'com.google.android.gms:play-services-gcm:16.0.0' in apps' gradle file.
I am able to build my project now. But is it safe to have both gcm and fcm dependency ?
Also, I am unable to reproduce this crash on my end. Any idea how can I reproduce it so that I can verify whether the fix you have mentioned will work or not.

@vRallev I made the changes as per link which you have shared. We released the app update last week and so far no crash was observed. Thanks for the help. you can close this issue.


Development

[ ] redeem tx
[ ] cancel vesting
[ ] rfl and refactor
[ ] remove warnings

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Functions

https://github.com/basarat/typescript-book/blob/master/docs/tips/statefulFunctions.md

https://vitest.dev/api/#describe

https://www.typescriptlang.org/docs/handbook/modules.html


[BUG]: Space between Back to Home button and footer

[x] Would you like to take up this task?
[x] Are you from GSSoC'23?

🐛
Bug report

Describe the bug
There is no space between Back to Home button and footer on error page

Location of the bug
Error page

Expected behavior
There must be some space between Back to Home button and footer

Screenshots (if any)

Please assign me this issue so that I can work on this issue under GSSOC'23.

The footer need padding from top as it has very less space from other components also


Can't mock the module in TypeScript

Hey.
I'm having a very hard time mocking the fileDownload export. Not sure exactly how to do it correctly.
Error: Error: Uncaught [TypeError: js_file_download_1.default is not a function]
My attempt to mock.
// test.tsx

import fileDownload from 'js-file-download'

jest.mock('js-file-download', () => {
    return {
        _esModule: true,
        fileDownload: jest.fn(() => 'stuff'),
    }
})

window.URL.createObjectURL = jest.fn()
window.URL.revokeObjectURL = jest.fn()

test('returns the data when clicked', () => {

    // ... test implementation that renders a button and fires a click event.

    expect(fileDownload).toReturnWith('stuff')
})

I see now where I went wrong.
The default export is the fileDownload function. I guess I was attempting to mock a module.exports object.

I'm not using TypeScript, but how did you end up mocking it? I am also having trouble... I've tried:
jest.mock('js-file-download', () => jest.fn(console.trace))
// or
jest.mock('js-file-download', () => {
  return {
    __esModule: true,
    default: jest.fn(console.trace)
  }
})

which prints a trace that indicates it is being called in the correct place, but I cannot verify with expect. I even tried:
let mockFileDownloadCalled = false
jest.mock('js-file-download', () =>
  jest.fn(() => {
    console.trace()
    mockFileDownloadCalled = true
    console.log('fileDownload mockFileDownloadCalled', mockFileDownloadCalled)
  })
)

which logs that mockFileDownloadCalled was set to true, but is false back in the testcase when trying to verify with expect.

Hey!
Been a while since I worked on this and I don't have access to the source code any more.
You could try setting the second parameter directly as a jest.fn, but I'm just firing from the hip now.
jest.mock('js-file-download', jest.fn(() => {...}))

@jestrickler Could you show me how you're implementing this in your test? Is the mock in place before your test runs?

@vilbergs I've tried a couple ways. Please see https://stackoverflow.com/questions/64465087/how-to-mock-a-3rd-party-module-that-is-just-a-function-js-file-download. I think it's actually working, but maybe a timing issue where I need to waitFor the calls, but unfortunately I'm also using react-keyed-file-browser which has some issues with waitFor. I can print fileDownload.mock.calls at the point of call in my react component and see that it was called.

Thank you @vilbergs for being helpful. If at the end of this, you find that some documentation would be useful, please feel free to tag me or make a PR.


screen share permission call twice.

Hello. I wanna use the screen capture alongside audio. So it's my code:
<ReactMediaRecorder
                screen
                render={({
                  status, startRecording, stopRecording, mediaBlobUrl,
                }) => ((

{status}

                    <button type="button" onClick={startRecording}>Start Recording</button>
                    <button type="button" onClick={stopRecording}>Stop Recording</button>
                    <video src={mediaBlobUrl} controls autoPlay loop />

                ))}
 />

but the screen share permission call twice. one after render  and one after calling the startRecording.
It is a BUG i think...

The getDisplayMedia() behaves a bit differently than getUserMedia(). The granted permission won't persist. You can read more here about it (from MDN) here, under the Usage Notes. So, each time you want to record the screen, the browser will explicitly ask for the permission.

On a second read, you mentioned it was asking for permissions on 'render'. On my end, it didn't ask. (I used the same example you provided).

It was an URGENT situation so I rewrite the whole package on my own. I think I should capture my screen to show you how it behaves. As i said, the screen capture permission calls twice and It's a bug.
I will send the video for you.
for now, my chrome version is: 85.0.4183.83

This should be fixed in #57 . New version 1.5.0


[key: string]: string of interface sub prop not match type Record<K, string> when K is extends string

Bug Report

🔎 Search Terms

TS2717 record type

🕗 Version & Regression Information

[key: string]: string of interface sub prop not match type Record<K, string> when K is extends string

⏯ Playground Link

Playground link with relevant code

Subsequent property declarations must have the same type.  Property 'groups' must be of type '{ [key: string]: string; }', but here has type 'Record<K, string>'.

💻 Code

interface A
{
	groups: {
		[key: string]: string
	};
}

interface A<K extends string = string>
{
	groups: {
		[key: string]: string
	};
}

interface A<K extends string = string>
{
	groups: Record<K, string>;
}

🙁 Actual behavior

Subsequent property declarations must have the same type.  Property 'groups' must be of type '{ [key: string]: string; }', but here has type 'Record<K, string>'.

🙂 Expected behavior

no error when K is extends string

These are indeed different. I'm not sure what else to add.


Begin/update channel points and docs

I noticed a bug with User Input from Channel Points not coming through, because of missing " in a struct tag.
Also updated the docs in Event Sub, after using them to setup some Channel Points subscriptions.

Pull Request Test Coverage Report for Build 714847793

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

Totals

Change from base Build 641999945:
0.0%

Covered Lines:
731

Relevant Lines:
838

💛  - Coveralls


New extension: AST

Can anyone explain how to install php-ast?
It's required by phan

Hello, the php-ast extension added to PHP.earth Alpine repository via https://github.com/phpearth/alpine/commit/69a3e0b91764ddcb6bf380ea609a8f17bced9588. Installation:
apk add --no-cache php7.2-ast

Or for older PHP versions apk add --no-cache php7.1-ast or apk add --no-cache php7.0-ast.
In case there are some bugs, let me know. Thanks for reporting.


Implement save/unsave post + saved list display

Hi, I needed the save functionallity so I added 'w' to save/unsave a post and /r/saved to show the saved list.

I'm a python beginer so sorry for not having complete a new test case for the new functionality. I tried to figure out how to update the cassettes files but I failed with the refresh-token part.

Hi tyjak,
Thanks for the PR. For a python beginner, your code looks very clean!
For the tests, all you have to do is point pytest to you own refresh token (the one that rtv saves after you login, ~/.config/rtv/refresh-token). Then, when you run in "record" mode, the test will use your account to perform actions. All of your authenticated requests will be stripped of sensitive information and saved in a new cassette. For example, from the project root you can run
$ PYTHONPATH=. py.test tests/test_submission.py::test_submission_view_saved --record-mode once --refresh-token ~/.config/rtv/refresh-token

Then when you get your test to pass, you can switch to using the cassette you created
$ PYTHONPATH=. py.test tests/test_submission.py::test_submission_view_saved

You then check that new cassette (tests/cassettes/test_submission_view_saved.yaml) along with the test into the repository.

@tyjak been a while. What needs to be done to get this pushed through?

Hi, I don't have too much time right now, I remembered that tests need to be write/done before to be pushed

Any updates on this? I'd love to have the ability to save comments/posts.

Hello, I made some updates, the saved comment now is displayed in the save list view. I added some test but missed one test precisely for the comment in /r/saved view. I'll add these test as soon as I have time to do it.

Hello @michael-lazar, do you think I need to make some other tests ?

I'm away for the weekend but I'll try to take a look at it when I get back
next week.
On Jul 30, 2016 9:41 AM, "tyjak"<EMAIL_ADDRESS>wrote:

Hello @michael-lazar https://github.com/michael-lazar, do you think I
need to make some other tests ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/michael-lazar/rtv/pull/187#issuecomment-236375353,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEyz68Q6pRSN8ANakrXwBpZFoYUD9hcEks5qa36mgaJpZM4HGdm4
.

Hey @tyjak, I've been testing this out and here's my feedback.

Code style and test coverage look great, thank you!
Small bug where you were checking for /r/saved instead of /u/saved
It crashed when I tried to save a really long comment and then open /u/saved. I added a check for comment title length to cut it off after ~4 lines so it didn't overflow.
I noticed that the load time got realllllly long when the /u/saved page contained several comments. After turning on logging, it was clear that the problem was that PRAW was trying to load each comment individually.

2016-07-31 22:40:09,390:DEBUG:connectionpool.py:395:"GET /user/civilization_phaze_3/saved.json?sort=new&limit=1024&t=all HTTP/1.1" 200 1353
2016-07-31 22:40:11,337:DEBUG:connectionpool.py:395:"GET /comments/4vk6oy/_/d5z8ms4.json HTTP/1.1" 200 1917
2016-07-31 22:40:13,096:DEBUG:connectionpool.py:395:"GET /comments/4vk6oy/_/d5z8joh.json HTTP/1.1" 200 1980
2016-07-31 22:40:15,020:DEBUG:connectionpool.py:395:"GET /comments/4vk6oy/_/d5z8eea.json HTTP/1.1" 200 3147
2016-07-31 22:40:17,228:DEBUG:connectionpool.py:395:"GET /comments/4vk6oy/_/d5z811w.json HTTP/1.1" 200 3277
2016-07-31 22:40:19,550:DEBUG:connectionpool.py:395:"GET /comments/4vk6oy/_/d5z7w5r.json HTTP/1.1" 200 4319
2016-07-31 22:40:21,235:DEBUG:connectionpool.py:395:"GET /comments/4vk6oy/_/d5z7jvt.json HTTP/1.1" 200 6776
2016-07-31 22:40:23,654:DEBUG:connectionpool.py:395:"GET /comments/4vk6oy/_/d5z2p8n.json HTTP/1.1" 200 8466

I found that the reason for this is that /user/{username}/saved.json only returns partial data for the comment objects. Notable fields that are missing include "submission" and "permalink".  When we try to access any of these missing fields, praw will try to auto-load the comment. I had to work around this by going a step further from coerce_saved_comment and implementing a custom loader for the new SavedComment type.

If you get a chance, please test this out on the master branch and let me know if I broke anything ;-)

Hi @michael-lazar evrything seems to be ok, thank you for your help !

Any progress on this feature?

@mrshannonyoung Yes it's been merged. Once you login you can press w to save a post and go to /u/saved to view them.


RNCWebView Crash - Completion handler was not called / presenting controller whose view is not in the window hierarchy

Bug description:
In the runJavaScriptAlertPanelWithMessage method, in RNCWebView, the following line of code can cause a crash:
[[self topViewController] presentViewController:alert animated:YES completion:NULL];
Presumably the same is possible in runJavaScriptConfirmPanelWithMessage & runJavaScriptTextInputPanelWithPrompt as well. TopViewController in the case of trying to select a file (photo/video/..) is UIImagePickerController, but it is dismissed as soon as the file selection is made, hence the crash below:
2020-04-16 20:27:22.715114-0400 homeDepot[47590:3894842] Warning: Attempt to present <UIAlertController: 0x10224f800> on <PUPhotoPickerHostViewController: 0x102238a00> whose view is not in the window hierarchy!
2020-04-16 20:27:22.721279-0400 homeDepot[47590:3894842] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Completion handler passed to -[RNCWebView webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:completionHandler:] was not called'
Top viewController in these cases should be the presentingViewController, and the alert should shown there.

To Reproduce:
Try to select a file that a particular server will not accept for some reason (file type, size of image, etc).
Expected behavior:
An alert should be displayed on the correct view.
Screenshots/Videos:
Screenshot of what would be expected:

Environment:

OS: iOS
OS version: 13.3.1
react-native version: 0.60.5
react-native-webview version: 9.2.1 (NOTE: just updated it to 10.2.3)

Proposed/Potential Solution
The solution to me seems pretty clear, in the case of UIImagePickerController, return the presentingViewController and not the presented one. I've taken the liberty of doing the simple required change here:
https://github.com/jxm2004/react-native-webview/commit/66b86887831cbcbef99a5c79d9034f4d75081074

Thanks this helped my problem. Hope it will be on next release

Same error here.

+1

This has now been merged in and is available in 10.4.1 and onwards 🎉; closing


ADC: CONFLICT! driver_ng is not allowed to be used with the legacy driver

Infinite loop on boot:
Rebooting...
ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0xc (RTC_SW_CPU_RST),boot:0x29 (SPI_FAST_FLASH_BOOT)
Saved PC:0x403787f9
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fce3818,len:0x508
load:0x403c9700,len:0x4
load:0x403c9704,len:0xad0
load:0x403cc700,len:0x29d8
entry 0x403c9880
E (332) ADC: CONFLICT! driver_ng is not allowed to be used with the legacy driver
abort() was called at PC 0x4208b103 on core 0
Backtrace: 0x403781e2:0x3fceb200 0x4037ffb1:0x3fceb220 0x40385c49:0x3fceb240 0x4208b103:0x3fceb2c0 0x42056fba:0x3fceb2e0 0x403784eb:0x3fceb310 0x403cd856:0x3fceb340 0x403cdafe:0x3fceb380 0x403c98d5:0x3fceb4b0 0x40045c01:0x3fceb570 |<-CORRUPTED

README ， https://github.com/Xinyuan-LilyGO/LilyGo-AMOLED-Series#4⃣-install-from-arduino-library-manager-recommended
Arduino ESP32 V 2.0.5 or above and below V3.0


Template gallery index.html is not included in packaging

Describe the bug
Invoking the template gallery command Typst: Show available Typst templates (gallery) for picking up a template to initialize on VSCode leads to error:

and there is no content in the template gallery tab consequently.
To Reproduce
Steps to reproduce the behavior (Library test):

Invoke the Typst: Show available Typst templates (gallery) for picking up a template to initialize command in VSCode command palette
See error

Expected behavior
The template gallery works normally as the readme and update changelog described.
Package/Software version:
VSCode version(Help -> About):
版本: 1.87.2 (system setup)
提交: 863d2581ecda6849923a2118d93a088b0745d9d6
日期: 2024-03-08T15:20:17.278Z
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22631

tinymist extension version: v0.10.3
Logs:
There is no related error can be grabbed either from the output of Tinymist server or the VSCode devtool, it is located in the output of VSCode Extension Host.
2024-03-17 03:45:31.520 [error] Error: ENOENT: no such file or directory, open 'c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\editor-tools\index.html' tinymist.showTemplateGallery {"value":"myriad-dreamin.tinymist","_lower":"myriad-dreamin.tinymist"}

And the message from popout window is quite straightforward. Looking up the directory of extension also reveals the reason: there is no existence of the required file.

Moreover, there is an error emitted frequently in VSCode devtool, but it seems not interfering critical functions and performance:
2024-03-17 03:45:28.433 [error] [Extension Host] failed to substitute vscode variables TypeError: Cannot read properties of undefined (reading 'replace')
	at n (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:44:47779)
	at av (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:49:3036)
	at c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:49:3353
	at Array.map (<anonymous>)
	at cv (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:49:3290)
	at Object.configuration (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:44:51116)

If this error is unrelated, please move this to another separated issue.

Moreover, there is an error emitted frequently in VSCode devtool, but it seems not interfering critical functions and performance:
2024-03-17 03:45:28.433 [error] [Extension Host] failed to substitute vscode variables TypeError: Cannot read properties of undefined (reading 'replace')
	at n (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:44:47779)
	at av (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:49:3036)
	at c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:49:3353
	at Array.map (<anonymous>)
	at cv (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:49:3290)
	at Object.configuration (c:\Users\Falke\.vscode\extensions\myriad-dreamin.tinymist-0.10.3-win32-x64\out\extension.js:44:51116)

If this error is unrelated, please move this to another separated issue.

The vscode variable may not process some string correctly so I try-catch it. It should not be triggered frequently, so I identity it as an enhancement: I'll implement it by myself when I schedule a time. Anyway it is not related and is an separately issue.


Show months and years only, hide days.

Hello, good work! It is possible to hide/turn off days in calendar? I need only months and years. Thank you for answer.

Hi @stanislavhosek,
Unfortunately, we don't have such feature now.

Why is there hard coded examples with only months and years in the demo then ? Seems misleading.

Thanks @hug0b ! The demo has been updated.


fix(ci): package name

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

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Anybody updated to F7v2?

Is there anything going on with app-framework?
Or do we consider this project as dead? (No pulls merged, no issues fixed, nobody has access to help out).
I like the idea of this framework pretty much, and it would be a great starting-point for any new project, but VueJS is already far into the 2.x.x and F7 made some pretty awesome changes, both of which we can't benefit from in app-framework.
So, has anybody already incuded F7v2?

@tiptronic - Andy, you are right, there is the time for App Framework 2. I kindly invite you for your ideas in the following thread: https://github.com/scriptPilot/app-framework/issues/775
As you maybe know, I became a daddy last month and beside this major project I started a new position in my company. So - there is a conflict of priorities and as there have not been many more requests than your one, for me there was no chance to complete App Framework 2 in the last weeks.
Dead I will not say ... on hold currently ... ok.

@scriptPilot Thx for your answer! I surely understand your position, but I don't want to wait until it's getting better, so adding to #775 is not a real option, if the project is 'on hold'.
To me it would work just porting app-framework to F7V2 and then collect new ideas for another next major version (which would probably take a while until it works as expected).
Right now I'm using the 'regular' F7-vue-webpack package and manage everything else in XCode/Android Studio, which is pita compared to app-framework.
Using app-framework v1 is not really an option, since it doesn't play well with the iPhoneX and v2 has a much better performance and page-/style-handling. Also, it's highly confusing, when you need to lookup something in the docs, because you have to take exceptional care, if you currently read v1 or v2 docs.
I also started to write some scripts to easify common tasks, but I'm also new to webpack and lots of things are still unclear to me (and I also don't have time to work fulltime on those).
So any constructive idea is wholeheartedly welcome.

I understand you have less time, not enough skills but want the Framework be updated asap - good combination :-D No, I am joking. So, for open source the point is not the money itself, it's time. I could reduce my ours at work to realize V2 soon, but then I need to get paid for 50 hours or so a different way - you have any idea? Second, if there should be better user contribution, the architecture could be more clearer. We should work on that in V2 as well. Referring to the post #775 - for me, the number of users contributing to it shows me in a way how much requested it is. For now, not much I understand.

Well tbh, I just need some information where to start - currently it's quite a bit information, what to change, and which consequences this has... But I'm digging through it and if I better understand what's going on, I'll here to help - no problem... there will always be a couple of hours time to do something (especially if it saves time in the long term).
As far as #775 is concerned: It's quite difficult to consider what should be going into V2, because I have a certain way how I (personally) work and what is important to me. I can add this, if you wish, but this would lead to a very stripped-down version of V1 😊 . E.g. for me, performance is #1, offline-first and state-restoration #2, Cordova/Android (plugin) handling #3 ... these are things that every application needs - everything else is optional.
Btw:Paying for 50 hours is only  just a question, how much you ask per hour 😊

Generally I don't care, if you use a payment-site or just Paypal, Patreon, Flattr, Bountysource or whatever...

 All I can say: if it saves me time setting up my next project, I am happy to share... If you're going to create the setup-scripts to MY OWN desire, I'd pay even more 😉 (in that case - just drop me a pm)

Anyway: Donorbox/Donations is surely a good option to prioritize features. But they need to go along with a reliable time-frame then. Otherwise it's kinda pointless 😉

Hello! Can a notice off Vue version be added to the top of readme, it at least somewhere visible?
"also maybe a contribution wanted" message/issue/note could be good.
Congratulations @scriptPilot, I do hope you manage to get some sleep and that everything is going well :)

@ehausen - Thanks for your interest. What is in your mind when you take about detaching from Vue?

It's possible to update directly to F7 v3 instead. I've done several experiments including upgrading the webpack, etc.

Next version will use Framework7 V3
https://github.com/scriptPilot/app-framework/issues/803


Some releases fail to clone. Run action shows set-output warning instead.

I get the following output from building an Action -
Adding v0.5.2-beta to ...
Adding v0.5.1-beta to ...
Adding v0.5.0-beta to ...
Adding v0.4.0-beta to ...
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Of the 7 releases in the source repo, the first 4 now appear as drafts in my target repo. The other 3 fail to clone. Of the 4 repos which did manage to transfer as drafts, all of them were missing binaries.
My YAML -
name: Clone
on: push
jobs:
  build:
    runs-on: ubuntu-latest
    steps:
      - name: Run action
        # Put your action repo here
        uses<EMAIL_ADDRESS>        # Put an example of your mandatory inputs here
        with:
          token: ${{ secrets.GITHUB_TOKEN }}
          src_repo: <internal_org_repo>

Expected behavior
All 7 repos transfer, including binaries.

Hi. Thanks for the issue
The action does not copy files from the releases on the target repo to your repo. It only creates releases on your repo that match the release on the target. If you want to copy release files, you would need another workflow to run on release
I'll look into the set output issue

I can edit the ReadME to note that info. Right now the text says "clone releases from a source repo to your repo". So it's unclear if binaries are included or not. Could that feature be added as an option? Including binaries is very helpful in most cases I can think of.

I'm taking a look at this today and will see what I can do.

https://github.com/andrewthetechie/gha-clone-releases/pull/15 - Fixes the set-output issue
https://github.com/andrewthetechie/gha-clone-releases/pull/17 - Adds draft to the fields copied from source Releases to the created Release.
These will be available in the v1.6.0 release once the release PR is merged
I'm still looking into copying release assets.

Please try out the 1.6.0 release and the new copy_assets input and let me know if that works for your needs.

Having an issue accessing the 1.6.0 release -
Pull down action image 'ghcr.io/andrewthetechie/gha-clone-releases:v1.6.0'
  /usr/bin/docker pull ghcr.io/andrewthetechie/gha-clone-releases:v1.6.0
  Error response from daemon: Head "https://ghcr.io/v2/andrewthetechie/gha-clone-releases/manifests/v1.6.0": unauthorized
  Warning: Docker pull failed with exit code 1, back off 2.8[9](https://github.com/Ansys-DEE/td-mc-plugin-releases/actions/runs/4973417576/jobs/8899249193#step:2:9)3 seconds before retry.

Same issue going to that link in a browser -
https://ghcr.io/v2/andrewthetechie/gha-clone-releases/manifests/v1.6.0 ---> Chrome returns --->
{"errors":[{"code":"UNAUTHORIZED","message":"authentication required"}]}

I missed a checkbox to enable un-authed access to the ghcr package. Please give it a try again and it should work for you now.

Several improvements over last time!

Container access issue fixed
Pre-release checkbox now checked
Tags are now migrating (just noticing this was not the case in v1.5.0).
Each release migrates as a completed release! Not a draft.
Some assets are migrated.
set-output() logging issue fixed.

Remaining possible improvements.

Still only migrating 4 of the 7 releases. This may be Ok for my specific case. The 3 repos being missed are much older releases... as long as recent (and all future) releases are migrated, I'm happy.
Only 2 of the 3 assets were migrated. Here is the output log -

##[debug]mcd-plugin has 7 releases
##[debug]td-mc-plugin-releases has 3 releases
##[debug]Limit: 0
Adding v0.5.0-beta-mc.22.2 to td-mc-plugin-releases
Error: Error while copying asset TD_MC_v050_BetaMC22_2.zip. 404 Client Error: Not Found for url: https://github.com/mcd-plugin/releases/download/v0.5.0-beta-mc.22.2/TD_MC_v050_BetaMC22_2.zip
##[debug]Docker Action run completed with exit code 0
##[debug]Set output addedReleases = v0.5.0-beta-mc.22.2
##[debug]Set output skippedReleasesCount = 0
##[debug]Set output addedReleasesCount = 1
##[debug]Finishing: Run action

Note the message td-mc-plugin-releases has 3 releases is because I manually deleted  v0.5.0-beta-mc.22.2 before running the Action. Otherwise it would have all 4 releases from previous testing.
Note the 2 assets which migrated successfully were the source code ZIP and TAR.GZ. The failed asset TD_MC_v050_BetaMC22_2.zip contains compiled binaries.
Lastly note when I click the GitHub URL in the error message Error while copying asset TD_MC..., a download immediately starts for that asset.

When I try to open that URL, it returns a 404. GitHub returns a 404 when you don't have permission to access a resource rather than a 403 to prevent enumeration. I'd suggest you check out the permissions of the token you're using with the action to see if, for some reason, it doesn't have access to that file.
As for the other issues, I added more debug logging in v1.6.5 (ignore the interim releases). That debug logging might give you more insight into why releases are not being cloned.

Got some useful info!
Failure to migrate the first 3 of the 7 releases.

This was caused by our early naming convention. Versions were only noted in the Tag. Names were repeated for Releases 1 - 4. So the Action only migrated Release 4, but ignored Releases 1 - 3.
For Release 5 - 7, we were noting version number in both the Name and Tag. So all of those releases were uniquely named and the Action migrated them.
It's Ok for my use case. We can name releases uniquely going forward. No "fix" required unless you think other devs would run into the same issue.

For migrated releases, only 2 of 3 assets were included.

This one I've partially figured out. Permissions are part of the story. Using secrets.GITHUB_TOKEN results in no visibility to other Private or Internal repos. No repos get migrated. This is a known bug - https://github.com/actions/setup-node/issues/49.
Using secrets.<PAT> works better. That at least migrates 2 of 3 assets per release. As before, the 2 assets which migrated successfully were the source code ZIP and TAR.GZ. The failed asset TD_MC_v050_BetaMC22_2.zip contains compiled binaries.
It's very strange that GitHub is treating some assets differently than others. Even using fine-grained PATs (which I tested), permissions can be set for all Release assets or none. There is no option for filtering certain types of assets.
Honestly I wish there were an option to filter types of assets using PATs. I would rather migrate the binaries without the source code. That's our goal for publishing. We don't want source code leaving private / internal repos, only binaries. I was planning to manually go through and delete source code ZIP / TAR.GZ assets from the migrated repos. Unfortunately GitHub doesn't appear to have an option for doing so from the browser. That may be another Action for me to investigate.
If source code assets are truly "undeletable" from a release, that is another example of them being treated differently than binary assets. I really hope there is some other part of the GitHub API intended for accessing binaries...

Please try v1.7.0 and let me know if it works as expected.
I created a private repo and was able to clone a release and copy over the assets using a Personal Access Token (PAT). You'll have to use a PAT with access to the source repo for this to work. Thank you for the link to the pip-tools issue; that helped.
Just to comment on something from earlier discussion, the action is not copying the Source Code zip and tar.gz from the source repo. Those assets are created automatically when you create a release. They will contain the source code of the destination repo, at the time the release and tag are created. I don't think you can delete these.

v1.7.0 is working!! This is extremely helpful for our team. Now we can keep source code in the private repos, while copying releases to internal / public repos. All in a simple automated way.
No further code changes to suggest. A few comments on the docs.

Note that Releases must be uniquely named or they will fail to copy.
Need to note using PATs instead of GITHUB_TOKEN with private / internal repos. I think you could just post this link rather than explain the backstory - https://github.com/actions/setup-node/issues/49. Otherwise each new user would likely take a few tries before realizing they need to use a PAT.
Consider on: push for the example workflow. This is the most common use case I can think of. Placing the workflow in the source repo ensures that any new releases are migrated as soon as they are uploaded to GitHub.
For the example workflow. In an earlier version, repo was a required parameter. Now it's src_repo.
For the example workflow. Consider locking onto a specific ubuntu version rather than ubuntu-latest. This way a bad ubuntu build will never break your code. It's happened to me before with ubuntu 22.04 breaking some of my legacy software builds. But on the other hand, pulling ubuntu-latest should mean less Linux kernel vulnerabilities. I'm on the fence about this one.

Glad it's working, I'll update some docs to mention the new feature and update that example you found.
@all-contributors please add @jrbe228 for bug and ideas


handle url.protocol on older versions of node

On older versions of node, url.parse()does not return url object with protocol property, if a protocol not present in the url string.
The condition url.parse(path).protocol !== null will always be true in those environments.

Behavior confirmed on node 0.6.  The protocol key is not present when a protocol-less url is passed.  Thanks!


Update prober alert email and add link to the message

@tkchin

lgtm

lgtm

The sender has to be a project member, so not noreply@


dist/quagga.min.js file gone

Hi, I would like to use this project using the script tag as written in readme but the  dist/quagga.min.js file is missing.
I am not familiar with NPM.
Maybe you have an updated version of the quagga.min.js I could use?
Thanks for continuing this project!

Hello there!  It doesn't really make sense to have the distribution files in the source repository, as npm is the preferred way of installing a working version, and having the distribution files in source makes for really cluttered change views.
Assuming you have node installed, you should be able to do npm install @ericblade/quagga2  from your application.. that section of the README might need a bit of an update, though. . .   If you'd like to generate the files yourself from the repo, you can run npm run build which will generate the packages as well.
Either way should work, though I think we probably need to do some changes to README with regards to how to get it to work correctly in a modern browser using npm install . . . all of my projects are using webpack and such to install modules..  so .. yeah.. maybe some changes are needed.

I just ran a quick check after doing npm install @ericblade/quagga2
<script<EMAIL_ADDRESS>correctly loads quagga into global space as Quagga

Thanks, learned something new.

how did you resolve?

As you suggested:
npm install @ericblade/quagga2- which created<EMAIL_ADDRESS>which I then copied to my project.
I only use the quagga.min.js in my project- none of the other files created by npm install @ericblade/quagga2
btw I don't use webpack- I use the asset pipeline

cool :) you could do the same with a copy of the repo and npm build


change resize to loadedmetadata since edge does not implement resize

Description
Edge doesn't seem to trigger resize so we need to use another event. loadedmetadata seems to work fine but travis will tell us if not :-)
Purpose
more green tests for Edge

Yeah this should now work since firefox has fixed their issue with only updating videoWidth/height after receiving a frame which does not happen before loadedmetadata has fired obviously. Not sure if ESR has caught up to it though but since the tests are passing we can probably assume so ;)
LGTM!


[Bug]: Column not found: 1054 Unknown column 'field' in 'field list'

What happened?
I have product model BelongsToMany collections model,
Here's my code :
ProductResource
SelectTree::make('collections')
    ->label(__('Collections'))
    ->emptyLabel(__('No results found'))
    ->relationship('collections', 'name', 'parent_id', function ($query) {
        return $query->whereLocale('name', app()->getLocale());
    })
    ->searchable()
    ->withCount(),

After saving the product record, it saves the relationship ids in product_collection, but an error appears :
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'collections' in 'field list'

update
  `products`
set
  `has_variants` = 0,
  `collections` = [ 1,
  2,
  3 ],
  `products`.`updated_at` = 2023 -12 -06 15: 44: 09
where
  `id` = 1

How to reproduce the bug

Setup BelongsToMany relationship
Setup SelectTree as mentioned in the doc
Create / update record on adding relationship ids
Save

Package Version
3.1.14
PHP Version
8.2
Laravel Version
10.x
Which operating systems does with happen with?
Linux
Notes
No response

Can you also show your Product model?

public function collections(): BelongsToMany {
    return $this->belongsToMany(Collection::class, 'product_collections')->withTimestamps();
}

Can you try to rename your relationship to something else and try again, i´m wondering if its because of the name.

It works fine if I use Select Filament component

Im not sure what is going wrong on your side, this is what my resource looks like:
public static function form(Form $form): Form
{
    return $form
        ->schema([
            Forms\Components\TextInput::make('name')
                ->required()
                ->live()
                ->maxLength(255),

            SelectTree::make('categories')
                ->label(__('Category'))
                ->relationship('categories', 'name', 'parent_id')
                ->enableBranchNode(),
        ]);
}

Blog.php
public function categories(): belongsToMany
{
   return $this->belongsToMany(Category::class);
}

Migrations

public function up(): void
    {
        Schema::create('categories', function (Blueprint $table) {
            $table->id();
            $table->string('name');
            $table->unsignedBigInteger('parent_id')->nullable();
            $table->foreign('parent_id')->references('id')->on('categories');
            $table->timestamps();
        });
    }

public function up(): void
    {
        Schema::create('blog_category', function (Blueprint $table) {
            $table->unsignedBigInteger('blog_id')->nullable();
            $table->foreign('blog_id')->references('id')->on('blogs')->cascadeOnDelete();
            $table->unsignedBigInteger('category_id')->nullable();
            $table->foreign('category_id')->references('id')->on('categories')->cascadeOnDelete();
        });
    }

In EditRecord of Filament resource,
protected function validateFormAndUpdateRecordAndCallHooks(): void
   {
       $this->callHook('beforeValidate');

       $data = $this->form->getState();

       $this->callHook('afterValidate');

       $data = $this->mutateFormDataBeforeSave($data);

       $this->callHook('beforeSave');

       $this->handleRecordUpdate($this->getRecord(), $data);

       $this->callHook('afterSave');
   }

if I use select filament, and do dd($data) after         $data = $this->mutateFormDataBeforeSave($data); I wont' find collections field, but when using your component, I find it, so after that,         $this->handleRecordUpdate($this->getRecord(), $data); will occur the error !

NB: Also, it works fine with belongsTo relationship,

Facing same issue.
Defined like this:
SelectTree::make('siteCategories')
                                            ->relationship('siteCategories', 'name', 'parent_id'),

Instead of saving to relationship, it trying saving to model itself.

Can you share your models / migrations? @abrardev99

here is code, not whol but relevant.
Accommodation.php
class Accommodation extends Model
{
    public function siteCategories(): BelongsToMany
    {
        return $this->belongsToMany(SiteCategory::class);
    }
}

SiteCategory.php
class SiteCategory extends Model
{
    protected $guarded = [];

    protected $table = 'site_categories';

    public function accommodations(): BelongsToMany
    {
        return $this->belongsToMany(Accommodation::class);
    }
}

migrations
accommodations table
Schema::create('accommodations', function (Blueprint $table) {
            $table->bigIncrements('id');
}

site_categories table
Schema::create('site_categories', function (Blueprint $table) {
    $table->bigIncrements('id');
}

pivot table
Schema::table('accommodation_site_category', function (Blueprint $table) {
            $table
                ->foreign('site_category_id')
                ->references('id')
                ->on('site_categories')
                ->onUpdate('CASCADE')
                ->onDelete('CASCADE');

            $table
                ->foreign('accommodation_id')
                ->references('id')
                ->on('accommodations')
                ->onUpdate('CASCADE')
                ->onDelete('CASCADE');
        });

I can set up a reproduction repo if you want?

@CodeWithDennis I noticed it saved the relationship, and the exception happened after it. Looks like after saving the relationship, it fills Accommodation model properties
Please take a look at the attached video.
https://github.com/CodeWithDennis/filament-select-tree/assets/54532330/4b8999d0-8bdf-4db3-86b8-6ff1a61aa4aa

@abrardev99 Yes would be great if you can make a repo!

I have the same problem 😣 i update the package to 1.3.6 and persist the problem

                                SelectTree::make('hierarchies')
                                ->label(__('Category'))
                                ->dehydrated(false)
                                ->relationship('hierarchies', 'name', 'parent_id'),

I find a solution on EDIT, but on create i have this error:
call_user_func(): Argument #1 ($callback) must be a valid callback, no array or string given

@Lucsnchz That worked. Great thanks.

This doesnt solve the real problem, could anyone make a repo for it?


Discussion: Ease the support of high contrast themes for accessibility

Discussion: Ease the support of high contrast theme for accessibility
To support the high contrast themes, the only way to test the XAML change is to change the Windows OS settings. This is slow process and it is not comfortable for a developer who has normal vision. There is a workaround to use the Windows Simulator, but there is an issue for multiple months that launching the Simulator will break OneDrive and the sync will need to be re-synched from scratch.
Suggestion: Visual Studio could provide a way to change in a faster way the high contrast theme. This can be in the design view or even better in the debugging bar that is integrated in the app at runtime.

The XAML designer should be able to be set into every possible mode the app can be run in, and perhaps even run conditions, so the app will override system defaults whilst being debugged?

The ability to set the Visual Studio XAML Designer to display different High Contrast modes would be great.
When testing for this at the moment (I currently automate changing to the different high contrast modes at a system level and then running the app while taking screenshots for later comparison) I know there are lots of challenges over accessing the appropriate system resources when the desired mode is not enabled for the OS. That makes me strongly suspect that modifications in Windows itself may be required to make this possible. Hopefully someone who knows all the specifics can tell me otherwise though.

I think this could be possible by creating a ResourceDictionary containing definitions for all the Brushes (e.g. SystemControlBackgroundBaseHighBrush, etc.) and Colors (SystemAltHighColor, etc.) that hard codes them to values equivalent to what they would be when Windows is in High Contrast mode. Then an app could test this simulated High Contrast mode by merging in this resource dictionary. I don't think it would be 100% accurate to true High Contrast mode, but it would probably be close enough to be useful for testing things without having to turn on High Contrast mode across the entire machine.

The complexity on creating a ResourceDictionary on our own, for each application is all the variations on the type of High Contrast, additionally keeping track of the growing set of colors that we add to our app themes might make editions more complex as time goes by. I would think that the easies way (from the developer's perspective) is have the designer support this in a way similar to "RequestedTheme" property of the Page, it could be inside the d namespace from Blend so that it remains only in design time, this way it is easy to spot issues and fix them. Although there is a missing piece in enabling this in design mode. Some elements of High Contrast are around interaction which require runtime, leading to the option to have this feature part of the debugger toolbar that shows up at the top of the window being debugged.

With all the recent increased focus on improving accessibility within Microsoft's own software, it would be terrible to see this be closed and deemed unimportant for those wanting to create software that is accessible to all users.


no need of OK Label

HI,
In my environment, I have to trigger alert if value decreases.
I am taking maxValue and minValue in 5 mins interval and if the value differs, is throwing alert.
but the problem facing that, after 5 mins of triggering CRIT alert, it automatically takes the alert into OK label.
Is this possible to have alert it does not throw OK alert or any other way to implement this concept?
below is the tickscript:
var data1 = stream
|from()
.database(db)
.retentionPolicy(rp)
.measurement(measurement)
.groupBy(groupBy)
.where(whereFilter)
|window()
.period(5m)
.every(5m)
|max('value')
.as('MaxValue')
var data2 = stream
|from()
.database(db)
.retentionPolicy(rp)
.measurement(measurement)
.groupBy(groupBy)
.where(whereFilter)
|window()
.period(5m)
.every(5m)
|min('value')
.as('MinValue')
var data = data1
|join(data2)
.as('data1', 'data2')
var trigger = data
|alert()
.crit(lambda: "data1.MaxValue" > "data2.MinValue")
.critReset(lambda: "data1.MaxValue" != 0)
.log('/tmp/SQLtest2.log')

Have you tried adding the no recoveries function to your alert node?
.noRecoveries

thanks for the solution!!
now I able to solve the issue of triggering OK alert.


FingerPrint Plugin Or C# Code

Please Guys Can Anyone Make a C# Who generate new fingerprint evry request I really need it

E-mail me your telegram/discord to<EMAIL_ADDRESS>I might be able to help with this

done i emailed u id u need drop telegram or discord here better

This is not an issue with OB2, closing. Use the forum next time


Http evader v4

Follows #199

makes liblzma support optional (configure does not exit if it does not find it)
rewords the commits to explain why
reverts useless change in htp_connp_is_line_ignorable

There is still to address the pending question from https://github.com/OISF/libhtp/pull/194#discussion_r272029858


Copilot X  - Full Project File Access

When I'm using Copilot X, I need to select specific code or highlight that code in order to get a solution, and sometimes it says give me the rest of the code, which I can't provide because it's in a different folder.
So it would be better if Copilot scanned all the files before answering the questions.
The engine needs to be chatgpt 4, not open AI.
If necessary, add an option to disable or grant access to .env files.

Sharing from #14: @workspace with "Used references" landed in Copilot Chat 0.10 as part of VS Code's October release

Please give it a try and file new issues for any follow-up feedback!


cmd/server: bind HTTP server with TLS if HTTPS_* variables are defined

Fixes: https://github.com/moov-io/wire/issues/54

Codecov Report

Merging #55 into master will decrease coverage by 0.07%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   88.53%   88.46%   -0.08%
==========================================
  Files          72       72
  Lines        5775     5780       +5
==========================================
  Hits         5113     5113
- Misses        414      419       +5
  Partials      248      248


[TW#26295] About component esp_https_ota problems

According to https://docs.espressif.com/projects/esp-idf/en/latest/api-reference/system/esp_https_ota.html?highlight=https,If I don't fill esp_http_client_config_t->cert_pem,the client can also communicate the https server. But in esp_https_ota.c, if i don't fill cert_pem, it will return ESP_FAIL.
I coment the check ,and it solve the problems.

This does not solve problem, this just does not check certificate validity so your connection is not secure.

This does not solve problem, this just does not check certificate validity so your connection is not secure.

So, If  I add the ssl certificate that means publick key?, that check fail ?

@zhaochengwei When it comes to normal HTTPS connection, server certificate verification is optional, but since firmware upgrade is a critical operation, we need to follow best security practices. So for best security practices, we have mandated HTTPS with server certificate verification, and it would be ideal if you provide root CA's certificate as server certificate.

@zhaochengwei When it comes to normal HTTPS connection, server certificate verification is optional, but since firmware upgrade is a critical operation, we need to follow best security practices. So for best security practices, we have mandated HTTPS with server certificate verification, and it would be ideal if you provide root CA's certificate as server certificate.

yeah,I think what you said will be better. But I meet the problems if I put my firmware on aliyun oss, It will be hard to get the ca certificate. So can you help me?

I am trying to determine how to use secure HTTPS in case of private networks. ESP32 is trying to contact to a list of servers inside the network where ip to these servers are distributed by router through DHCP and there is no local DNS running inside the network. Any advice to resolve this issue?

@zhaochengwei I found this root CA certificate for aliyun oss:
-----BEGIN CERTIFICATE-----
MIIDdTCCAl2gAwIBAgILBAAAAAABFUtaw5QwDQYJKoZIhvcNAQEFBQAwVzELMAkG
A1UEBhMCQkUxGTAXBgNVBAoTEEdsb2JhbFNpZ24gbnYtc2ExEDAOBgNVBAsTB1Jv
b3QgQ0ExGzAZBgNVBAMTEkdsb2JhbFNpZ24gUm9vdCBDQTAeFw05ODA5MDExMjAw
MDBaFw0yODAxMjgxMjAwMDBaMFcxCzAJBgNVBAYTAkJFMRkwFwYDVQQKExBHbG9i
YWxTaWduIG52LXNhMRAwDgYDVQQLEwdSb290IENBMRswGQYDVQQDExJHbG9iYWxT
aWduIFJvb3QgQ0EwggEiMA0GCSqGSIb3DQEBAQUAA4IBDwAwggEKAoIBAQDaDuaZ
jc6j40+Kfvvxi4Mla+pIH/EqsLmVEQS98GPR4mdmzxzdzxtIK+6NiY6arymAZavp
xy0Sy6scTHAHoT0KMM0VjU/43dSMUBUc71DuxC73/OlS8pF94G3VNTCOXkNz8kHp
1Wrjsok6Vjk4bwY8iGlbKk3Fp1S4bInMm/k8yuX9ifUSPJJ4ltbcdG6TRGHRjcdG
snUOhugZitVtbNV4FpWi6cgKOOvyJBNPc1STE4U6G7weNLWLBYy5d4ux2x8gkasJ
U26Qzns3dLlwR5EiUWMWea6xrkEmCMgZK9FGqkjWZCrXgzT/LCrBbBlDSgeF59N8
9iFo7+ryUp9/k5DPAgMBAAGjQjBAMA4GA1UdDwEB/wQEAwIBBjAPBgNVHRMBAf8E
BTADAQH/MB0GA1UdDgQWBBRge2YaRQ2XyolQL30EzTSo//z9SzANBgkqhkiG9w0B
AQUFAAOCAQEA1nPnfE920I2/7LqivjTFKDK1fPxsnCwrvQmeU79rXqoRSLblCKOz
yj1hTdNGCbM+w6DjY1Ub8rrvrTnhQ7k4o+YviiY776BQVvnGCv04zcQLcFGUl5gE
38NflNUVyRRBnMRddWQVDf9VMOyGj/8N7yy5Y0b2qvzfvGn9LhJIZJrglfCm7ymP
AbEVtQwdpf5pLGkkeB6zpxxxYu7KyJesF12KwvhHhm4qxFYxldBniYUr+WymXUad
DKqC5JlR3XC321Y9YeRq4VzW9v493kHMB65jUr9TU/Qr6cf9tveCX4XSQRjbgbME
HMUfpIBvFSDJ3gyICh3WZlXi/EjJKSZp4A==
-----END CERTIFICATE-----

So, can you copy it to examples/system/ota/server_certs/ca_cert.pem file and check whether simple_ota_example works or not.

Closing (do no see any issue with esp_https_ota component).


Add joint keypoints config

TODO: Read this instead of hard coding the joint names

Files where this joint set is hard-coded and needs to be changed:

[ ] https://github.com/PTG-Kitware/angel_system/blob/master/ros/angel_system_nodes/angel_system_nodes/pose_estimation/pose_estimator.py#L110
[ ] https://github.com/PTG-Kitware/angel_system/blob/99697008986ea669f5f5e5c5cbbbf8025d9b874d/angel_system/data/common/kwcoco_utils.py#L216
[ ] https://github.com/PTG-Kitware/angel_system/blob/99697008986ea669f5f5e5c5cbbbf8025d9b874d/ros/angel_utils/nodes/simple_2d_detection_overlay.cxx#L70
[ ] https://github.com/PTG-Kitware/angel_system/blob/99697008986ea669f5f5e5c5cbbbf8025d9b874d/angel_system/activity_classification/utils.py#L270
[ ] https://github.com/PTG-Kitware/angel_system/blob/99697008986ea669f5f5e5c5cbbbf8025d9b874d/ros/angel_system_nodes/angel_system_nodes/activity_classification/activity_classifier_tcn.py#L188
[ ]


[BUG] v<IP_ADDRESS> unhandled exception on start

Computer Information:

OS: Windows 10 Version 1903 (OS Build 18362.535)
SRT version: <IP_ADDRESS>
Game version: Steam latest

Describe the bug
On launch attempt an unhandled exception window comes up and the program doesn't start.
Steps to Reproduce
Steps to reproduce the behavior:

Launch RE2
Launch SRT

Expected behavior
The program to launch
Screenshots

Additional context
RE2 Windowed Mode and DX11

Everything was extracted to the same folder? You should have roughly 59 files in your srt folder. Otherwise, there may be a new requirement for .NET Core 3.1 Runtime. I'd be surprised if that was the case but it is a possibility..

Everything was extracted to the same folder? You should have roughly 59 files in your srt folder. Otherwise, there may be a new requirement for .NET Core 3.1 Runtime. I'd be surprised if that was the case but it is a possibility..

All files extracted to the same dir, will install .NET Core 3.1 and try again.

I tried explicitly installing .NET Core 3.1 x64+x86 and ASP.NET Core 2.2 Runtime x64+x86, same error dialogue.
Also additional context here is my dotnet info.
Microsoft Windows [Version 10.0.18362.535]
(c) 2019 Microsoft Corporation. All rights reserved.

C:\>dotnet --info
  It was not possible to find any installed .NET Core SDKs
  Did you mean to run .NET Core SDK commands? Install a .NET Core SDK from:
      https://aka.ms/dotnet-download

Host (useful for support):
  Version: 3.1.0
  Commit:  65f04fb6db

.NET Core SDKs installed:
  No SDKs were found.

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Also receive the same error dialogue on version <IP_ADDRESS>

One last question, did you download from the links in the ChangeLog.MD or from the Releases tab?

I believe I found the issue. The Releases tab CI automated builds are not including RE2REmakeSRT.exe.config. As soon as I copied that into the extracted folder, it started working for that build. I'll work on fixing that (if I can remember which login I used for Azure DevOps >.>).
In the meantime, use the http://dudley.gg/squirrelies/re2/latest.7z or explicit http://dudley.gg/squirrelies/re2/RE2REmakeSRT-1410-Beta-Signed-Release.7z download.

Fixed and tested the Releases tab CI. It now includes the required .config file.


MDX import results in "Module parse failed: Unexpected character..."

Bug report

Describe the bug
I'm trying to use the @zeit/next-mdx package to import .mdx files (and their metadata) into one of my pages. I'm pretty sure I'm using the plugin correctly. I'm also using TypeScript (potential source of error), but have defined an mdx.d.ts:
`
declare module '*.mdx' {
    let MDXComponent: (props) => JSX.Element
    export default MDXComponent
}

`
I believe that I've properly configured the plugin in my next.config.js. To see the project more in depth, check out the repository.
To Reproduce

clone the repository
yarn && yarn start
open the locally-served site in Chrome and make sure it displays without errors
open the project in your code editor and navigate to pages/Landing/index.tsx
uncomment lines 6 & 7
save & refresh
BUG BUG BUG

Expected behavior
The console log (line 7 in pages/Landing/index.tsx) is supposed to print out the imported MD as a JSX/TSX-compatible component and the metadata as an object literal.
System information

Version of macOS: High Sierra 10.13.5
Version of Chrome: 68.0.3440.75
Version of Next.js: 6.1.1

Additional context
Thank you so so so much for taking the time to look this over. Please let me know if you have any questions. I searched pretty hard and don't see this issue elsewhere. Apologies in advance if the answer is obvious. Hopefully this is helpful to others as well.

Hey, just checking in. I'd love to implement MDX in a NextJS site. I would really really really appreciate any suggestions or support 🙏 loving the framework and can't wait to use it in production.
Thank you,
Harry

Having the same issue here as well!

Hi hi hi

I noticed that next-mdx has a different signature with the other plugins (#320).
You should add () to your next.config.js and it should work:
// https://github.com/harrysolovay/NextJS-TypeScript-MDX/blob/master/next.config.js
const withSourceMaps = require('@zeit/next-source-maps')
const withMDX = require('@zeit/next-mdx')() // <--
const mdxImagesPlugin = require('remark-images')
const mdxEmjoiPlugin = require('remark-emoji')


”Gherghești”, ”Chetrosu”, ”Corodești”, ”Dragomănești”, ”Drăxeni” are listed in the drop-down menu

Description:
When user writes ”Gherghești”, ”Chetrosu”, ”Corodești”, ”Dragomănești”, ”Drăxeni” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Gherghești"

Expected results
The location appears in the drop-down menu as "Sat Gherghești, VASLUI (Gherghești)".

Step 3
Write in the search bar "Chetrosu"

Expected results
The location appears in the drop-down menu as "Sat Chetrosu, VASLUI (Gherghești)".

Step 5
Write in the search bar "Corodești"

Expected results
The location appears in the drop-down menu as "Sat Corodești, VASLUI (Gherghești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Dragomănești"

Expected results
The location appears in the drop-down menu as "Sat Dragomănești, VASLUI (Gherghești)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Write in the search bar "Drăxeni"

Expected results
The location appears in the drop-down menu as "Sat Drăxeni, VASLUI (Gherghești)".


🌐 Add Russian translation for docs/ru/docs/deployment/https.md

Awesome, thank you @Xewus! :bow:
And thank you for the reviews @onix2020 and @AGolicyn :coffee:


Allow tagging, querying limited by tag

Our organization uses a tagging system to track usage, for cost accounting, auditing, and etc. It would be nice to be able to assign arbitrary tags to instances. It would also be good if we could use them as limiters for cluster-wide queries, or else return them in the results so downstream tools can do so.

Metadata editing appears to be sufficient to our needs.


[chainerx] Adds hsplit

Solves part of issue #6423

Leaves a reference ticket (vsplit) as it's somewhat related. https://github.com/chainer/chainer/pull/7032

@ishanrai05 Are you still able to work on this PR and vsplit?

@hvy  I am so sorry... I had been busy.. I'll update the PR ASAP.

Now that your vsplit has been merged, could you update this PR (e.g. the tests)?

Could you fix the Travis docs failure?

You are throwing 3 DimensionErrors but could you write tests to cover these paths? You could maybe take inspiration from test_split_invalid.

Jenkins, test this please.

Jenkins CI test (for commit 21226b361a5da1bf8edea7522e470a9d75bb2841, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 21226b361a5da1bf8edea7522e470a9d75bb2841, target branch master) succeeded!

Thanks as always, LGTM!