Signature extraction failed: Traceback

kubanowsky@kubanowsky:~
$ youtube-dl --playlist-start 108 -x --audio-format mp3 --audio-quality 0 https://www.youtube.com/playlist?list=PLzcST9n6QAXVpLpHhm9G0cNDOMfGsy7RF --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--playlist-start', '108', '-x', '--audio-format', 'mp3', '--audio-quality', '0', 'https://www.youtube.com/playlist?list=PLzcST9n6QAXVpLpHhm9G0cNDOMfGsy7RF', '--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.08.05
[debug] Python version 2.7.9 - Linux-4.0.5-antix.1-486-smp-i686-with-antiX-15-jessie
[debug] Proxy map: {}
[youtube:playlist] PLzcST9n6QAXVpLpHhm9G0cNDOMfGsy7RF: Downloading webpage
[youtube:playlist] PLzcST9n6QAXVpLpHhm9G0cNDOMfGsy7RF: Downloading page #1
[download] Downloading playlist: Dobre nuty
[youtube:playlist] playlist Dobre nuty: Collected 112 video ids (downloading 5 of them)
[download] Downloading video #1 of 5
[youtube] Setting language
[youtube] oO10eNicsfQ: Downloading webpage
[youtube] oO10eNicsfQ: Downloading video info webpage
[youtube] oO10eNicsfQ: Extracting video information
[youtube] oO10eNicsfQ: Encrypted signatures detected.
[youtube] {22} signature length 46.43, html5 player new
[youtube] oO10eNicsfQ: Downloading js player new
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 479, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 383, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 454, in _parse_sig_js
u'Initial JS player signature function name')
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 391, in _search_regex
raise RegexNotFoundError(u'Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 479, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 383, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 454, in _parse_sig_js
u'Initial JS player signature function name')
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 391, in _search_regex
raise RegexNotFoundError(u'Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 516, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 170, in extract
return self._real_extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 860, in _real_extract
encrypted_sig, video_id, player_url, age_gate)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 489, in _decrypt_signature
u'Signature extraction failed: ' + tb, cause=e)
ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 479, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 383, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 454, in _parse_sig_js
u'Initial JS player signature function name')
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 391, in _search_regex
raise RegexNotFoundError(u'Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

You are using an outdated version of youtube-dl. Read our FAQ if you have troubles updating.


Write a JavaScript program to convert string to pathcase

Description
Write a JavaScript program to convert string to pathcase
Pathcase is a style of writing in which all letters are lowercase and spaces are replaced with forward slashes.
Input  : "hello world"
Output : "hello/world"

How to contribute

Save the solution in program/ConvertStringToPathcase.js file
Add ConvertStringToPathcase.js file in convert-string-to-pathcase folder

!assign


Support collecting dubbo thread pool metrics

Inspired by the implementation of untow_thread_pool at issue#8433
Now the skywalking-javaagent could collect the Dubbo thread pool metrics. I tested it with dubbo-2.6.9 and dubbo-<IP_ADDRESS> and ran it in our production environment with hundreds of applications, it worked fine.
Dubbo could use telnet to get the thread pool status, this meter plugin is related to this telnet implementation.
$ telnet localhost 20880
Trying ::1...
Connected to localhost.
Escape character is '^]'.

dubbo>status -l
+------------+--------+--------------------------------------------------------+
| resource   | status | message                                                |
+------------+--------+--------------------------------------------------------+
| threadpool | OK     | Pool status:OK, max:200, core:200, largest:3, active:1, task:3, service port: 20880 |
| load       | OK     | load:2.60595703125,cpu:12                              |
| memory     | OK     | max:3641M,total:1294M,used:481M,free:813M              |
| registry   | OK     | <IP_ADDRESS>:2181(connected)                            |
| server     | OK     | /<IP_ADDRESS>:20880(clients:1)                         |
| summary    | OK     |                                                        |
+------------+--------+--------------------------------------------------------+

Please take a look. Also, cause the e2e test is friendly for the trace plugin, rather than the meter plugin. There is no test code with this PR, I'm willing to add test code if there are some guidelines.

[ ] If this is non-trivial feature, paste the links/URLs to the design doc.
[ ] Update the documentation to include this new feature.
[ ] Tests(including UT, IT, E2E) are added to verify the new feature.
[ ] If it's UI related, attach the screenshots below.
[ ] Update the CHANGES log.

The meter plugin could be verified by the same way like tracing plugin.
https://skywalking.apache.org/docs/skywalking-java/next/en/setup/service-agent/java-agent/plugin-test/#expecteddatayaml
Meter and Logs are all supported actually.

https://github.com/apache/skywalking-java/blob/main/test/plugin/scenarios/hikaricp-scenario/config/expectedData.yaml#L207
Plugin tests exist as well. We usually add docs with features.

Thanks, test code will be add soon.

The dubbo-2.5.x-scenario test case runs well, but the dubbo-2.7.x-scenario test case runs failed. Because after dubbo-2.7.5, the thread pool has big changes, see https://github.com/apache/dubbo/issues/6625. Maybe separate the test case,  or let dubbo-2.7.x-plugin adapt the high version.

Maybe separate the test case, or let dubbo-2.7.x-plugin adapt the high version.

What do you mean separate test case? I can see one case for 2.5.x, and the other for 2.7.x. Do you mean 2.7.0-2.7.4 could pass the tests, but 2.7.5+ would fail due to the changes?

Yes! There are many dubbo versions that make people crazy.

Yes! There are many dubbo versions that make people crazy.

For version compatibility, it is okay if they break something internally. We just need to choose the witness class or method(s) to identify versions(not actual version numbers, but different internal APIs) in the runtime.
4. Set up witnessClasses and/or witnessMethods if the instrumentation has to be activated in specific versions in
https://skywalking.apache.org/docs/skywalking-java/next/en/setup/service-agent/java-agent/java-plugin-development-guide/#implement-plugin
If you have to rely on different versions to compile the plugin codes, we should provide two new plugins(dubbo-2.7.4-below-pool-metrics and dubbo-2.7.5-above-pool-metrics), but keep the original dubbo-2.7.x-plugin untouched for tracing only.
Meanwhile, due to the witness class mechanism, you should be able to keep test scenarios as one.

Thanks, let me check how to adapt 2.7.5+ first.

Now the dubbo scenarios run well. There is a better class to enhance. We don't need the witness class and separate the plugins. Here are the brief changes:

Change the enhance class to org.apache.dubbo.remoting.transport.AbstractServer.
Fix the wrong default port 20080 -> 20880.
Change the thread pool name to DubboServerHandler-%port%, just like the dubbo thread name DubboServerHandler-<IP_ADDRESS>:20880-thread-%. But currently, I have not added the hostname to it because the expectedData.yaml doesn't support the value operator start with, the hostname couldn't predict when running the test case.

Please take a look. :)

Could you share the screenshots about how this metric of pool likes like on the v9 UI?

Please update the changes.md too.

Could you share the screenshots about how this metric of pool likes like on the v9 UI?

We forward the metrics to other storage in the backend and still use the v8 UI. I could configure the metrics in the v9 UI, it needs takes some time.

Could you share the screenshots about how this metric of pool likes like on the v9 UI?

We forward the metrics to other storage in the backend and still use the v8 UI. I could configure the metrics in the v9 UI, it needs takes some time.

It is fine your private usage is different. But please take some time to finish the steps for the community.

Sure, I will finish it ASAP.

Here are the Dubbo thread pool screenshots at v9 UI. In this case, tomcat and Dubbo thread pool metrics are put together.

Is there anything missing? At least one, the supported-list.md should be updated.

Is there anything left before merging? At least one, the supported-list.md should be updated.

The supported-list.md has be updated. One thing I metion before, I want to make agent-test-tools support new operator start with to check the thread pool name.

The supported-list.md has be updated. One thing I metion before, I want to make agent-test-tools support new operator start with to check the thread pool name.

Feel free to send a pull request to the tool. I think this is easy to add.
And, I think that is not a block for this PR. You could enhance the test case later when the tool side is ready.

Well done, This plugin is exactly what I want🎉


Add missing gofuzz deps

Looks like I had update Godeps.json but failed to add the actual dependencies.
cc @vmarmol

Is the failure expected:
github.com/GoogleCloudPlatform/heapster/sinks/api
sinks/api/decoder_test.go:32: cannot use struct {} literal (type struct {}) as type api.AggregateData in argument to NewDecoder().Timeseries

Hehehehe :D

The test failure is real. Fixed the issue.

I am not sure why jenkins did not update the PR, but the test did pass: http://<IP_ADDRESS>/job/heapster-e2e-gce/573/

LGTM
Merging


Annotations not parsed when running Sourcery with the same template but different output files

Hi!
I have a setup where I have one AutoMockable.stencil file that I use as template for 2 configurations. One configuration outputs in a file in the unit tests target, and one in the App target.
I use this to generate mocks for SwiftUI previews and also have them for unit tests.
Another note is, I use 2 // sourcery: AutoMockable annotations:

the standard // sourcery: AutoMockable - used to output in the tests target file
a new // sourcery: AutoMockableApp - used to output in the App target file

I also have some custom flags configured in the .sourcery.yml to be able to differentiate which configuration is run (the unit tests or app) in order to customize the resulting mocks.
Finally, the super-weird part is that some times it works, but if I run sourcery multiple times consecutively, it breaks and doesn't find the annotations in one of the configurations
My .sourcery.yml looks like this
configurations:
  - sources:
      - Sources/
    templates:
      - /Stencils/AutoMockable.stencil
    output: Tests/Mocks/GeneratedMocks.swift
    args:
      isTestTargetMocks: true
      autoMockableImports: [
        "Combine",
      ]
      autoMockableTestableImports: [
        "MyApp"
      ]
  - sources:
      - Sources/
    templates:
      - /Stencils/AutoMockable.stencil
    output: MyApp/Core/Generators/GeneratedBuilders.swift
    args:
      isAppTargetMocks: true
      autoMockableImports: [
        "Combine",
      ]

The AutoMockable.stencil is the same like the template one on this repo, just modified it a bit to not generate CalledCount, etc for the App target mocks.
Mocked protocol looks like this
// sourcery: AutoMockable
// sourcery: AutoMockableApp
protocol ArticlePagedLoaderType: ObservableObject {
    var articles: [Article] { get }
    var isLoading: Bool { get }
}

Attached photos of the output in the App target
Any ideas what could be the problem?

I investigated a bit, and it turns out you can reproduce it quite easily. The setup is:
protocol SomeTypeUsingGenericConstraint: AutoMockable {
    associatedtype MyGenericConstraint: SomeTypeToBeUsedAsGenericConstraint
}

protocol SomeTypeToBeUsedAsGenericConstraint: AutoMockable {
    func doStuff()
}

You just need to run sourcery multiple times, one after the other. I tried with --disableCache but it still fails. SomeTypeToBeUsedAsGenericConstraint is not generated anymore, only SomeTypeUsingGenericConstraint
We have sourcery as a pre-build phase (don't ask why, it was there when I found it 😸 ) and since we're working with SwiftUI and Previews, you can imagine sourcery is called quite often when the previews are built.
Would appreciate any insight on this.


fix(vitest): support default coverage provider

Vitest has a default coverage provider, v8, but knip detects it as an unused dependency.
We can safely assume the user uses Vitest's coverage if the config file contains the coverage field, so this PR uses v8 as the default provider if provider is not set.
https://vitest.dev/guide/coverage.html#coverage-providers

Thanks @risu729! Any chance you could add a (something to an existing) test?

Sure! I added a test now.

:rocket: This pull request is included in v2.19.0. See Release 2.19.0 for release notes.


Missing optimizations for 2d copies in the dma system

The optimizations for batching 2d copies are currently missing in the deppart branch. The issue can be reproduced by using the following commands.
export LG_RT_DIR=/home/zhihao/legion/runtime/
cd /home/zhihao/legion/apps/cnn/
make -j 12
./cnn -ll:gpu 2 -ll:fsize 4000 -level request=2
You should be able to observe that the dma system launches a large number of 1d requests for a potential 2d copy.

@jiazhihao did this performance issue show up in the dma branch as well?

I am using the deppart API for creating partitions (e.g., create_partition_by_restriction). So it may not be easy to know if we can reproduce the issue in the dma branch.

Yeah, don't worry about it then.  I didn't realize this app only existed in the deppart branch.

Commit 60d0e80 adds a test that exercises 2d and 3d copies and reproduces the performance regression for 2d copies in the dma branch.  (The 3d copies are bad even in master, but that's a
problem for another day.)  Working on a fix now.

@jiazhihao I believe that both the dma and deppart branches have fixes for 2d and 3d copies for channels that support them (only memcpy supports 3d at the moment).  Can you try your app again and let me know if the performance is sufficiently improved?

The performance looks much better after the fix. I will close this issue for now and reopen it if I observe other related issue.


An utility method to eliminate casts used by generic methods in builders

There is a lot of places in EF nd its providers where a generic method calls its non-generic version which implements all logic. This introduces a lot of casts. To reduce code size, improve quality and to make code less error prone there could be added some methods in EF itself which will call a non generic method and cast the result back:
public static DbContextOptionsBuilder<TContext> NonGeneric<TContext>(
    Func<DbContextOptionsBuilder, DbContextOptionsBuilder> method,
    DbContextOptionsBuilder<TContext> optionsBuilder)
    where TContext : DbContext
    => (DbContextOptionsBuilder<TContext>)method(optionsBuilder);

public static DbContextOptionsBuilder<TContext> NonGeneric<TContext, T>(
    Func<DbContextOptionsBuilder, T, DbContextOptionsBuilder> method,
    DbContextOptionsBuilder<TContext> optionsBuilder, T arg)
    where TContext : DbContext
    => (DbContextOptionsBuilder<TContext>)method(optionsBuilder, arg);

// and so on... and for other types...

Example:
// before
public static DbContextOptionsBuilder<TContext> UseSnakeCaseNaming<TContext>([NotNull] this DbContextOptionsBuilder<TContext> optionsBuilder)
    where TContext : DbContext
    => (DbContextOptionsBuilder<TContext>)UseSnakeCaseNaming((DbContextOptionsBuilder)optionsBuilder);
// after
public static DbContextOptionsBuilder<TContext> UseSnakeCaseNaming<TContext>([NotNull] this DbContextOptionsBuilder<TContext> optionsBuilder)
    where TContext : DbContext
    => NonGeneric(UseSnakeCaseNaming, optionsBuilder);

The disadvantage is that is allocates delegates, but since EF allocates a lot during the configuration step maybe it's not so important.

@YohDeadfall - Adding any public surface in EF Core for "providers" means that it has to be in public class & not inside .Internal namespace. Which also exposes them to public. Given these methods live alongside methods which customers are supposed to call, it would confuse customers if we add them. If it helps you in making code DRY then you can put those methods as private in your class as implementation detail. We don't plan to add these methods due to negative value it provides to customers.

Okay, but does it make sense to introduce it in SQL Server and SQLite providers? Some internal code as you know is copy pasted to non-Microsoft providers, so it won't be a public API, but still can be used by all parties.
/cc @roji @brgrainger

Okay, but does it make sense to introduce it in SQL Server and SQLite providers? Some internal code as you know is copy pasted to non-Microsoft providers, so it won't be a public API, but still can be used by all parties.
/cc @roji @bgrainger

@AndriySvyryd - Do you believe that we should do this in our providers?

We'll consider it when working in this area, I don't think it's high priority


user configuration options cache + packr2 refactoring

The idea is to simplify code and dependencies, instead of using packr2 and caching the content of in-memory JSON files we can use go generate. It will generate Golang files with corresponding inline JSON and update templates package level variable with schemas content.

A little bit more regarding how it will work if we decide to merge this PR:
When a user runs go generate Golang triggers aiven/templates/gen.sh shell script to generate *_user_config_schema.go files with variables containing unmarshaled JSON from *_user_config_schema.json files. We include autogenerated files to the VCS as it is recommended in Golang community (though we don't have to do that).

Looks ok, rebase and I'll merge this


Why does spirv_disas, from spirv 1.1 branches, report Version 1.0?

In the generated code, says spirv version 1.0:
(master|…1) ~/git/pub-prototyping/spirv$ cat cl_kernel1.ll
; SPIR-V
; Version: 1.0
; Generator: Khronos LLVM/SPIR-V Translator; 14
; Bound: 22
; Schema: 0
               OpCapability Addresses
               OpCapability Linkage
               OpCapability Kernel
               OpCapability Int8

But everything I'm using is SPIR-V 1.1. Why the version mis-match?
llvm and spir versions:
(khronos/spirv-3.6.1|…1) ~/git/marsohodmethod/llvm$ git status
On branch khronos/spirv-3.6.1
Your branch is up-to-date with 'origin/khronos/spirv-3.6.1'.
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        build/

nothing added to commit but untracked files present (use "git add" to track)
(khronos/spirv-3.6.1|…1) ~/git/marsohodmethod/llvm$ cd tools/clang/
(spirv-1.1|✔) ~/git/marsohodmethod/llvm/tools/clang$ git status
On branch spirv-1.1
Your branch is up-to-date with 'origin/spirv-1.1'.
nothing to commit, working directory clean

spirv-tools version:
(spirv-1.1|…1) ~/git/SPIRV-Tools$ git status
On branch spirv-1.1
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        external/spirv-headers/

The disassembler is reporting the SPIR-V version word inside the SPIR-V module. It's the second word
See https://www.khronos.org/registry/spir-v/specs/1.2/SPIRV.html#_a_id_physicallayout_a_physical_layout_of_a_spir_v_module_and_instruction
This is completely independent of whatever git branch you got the SPIRV-Tools code from.


WebMercatorViewport is a constructor not a function

The ReadMe docs state usage like this:
const viewport = WebMercatorViewport({})

but it should be:
const viewport = new WebMercatorViewport({})

https://github.com/uber-common/viewport-mercator-project/blob/9aa65be379971ef53a9eb581bb58f2d4b9a9cc2c/docs/get-started/README.md
https://github.com/uber-common/viewport-mercator-project/blob/9aa65be379971ef53a9eb581bb58f2d4b9a9cc2c/docs/README.md

Thanks for reporting. Fixed on master, will be updated next time docs are republished.

Is this still relevant?


clustersync: fix missing memcpy

71c1839b / #1572 fixed an ordering problem in the clustersync controller
to address "moving" a resource from one syncset to another. In so doing,
it introduced a temporary list to look for the deletions. However, the
list was being aliased rather than copied, and the result was that
scenarios with multiple syncsets could end up ignoring changes to a
subset of them. Fix.
HIVE-1891

/hold for live verification
/assign @abutcher
/cc @machi1990

/test e2e-pool

/cherry-pick ocm-2.5-mce-2.0

/cherry-pick ocm-2.5-mce-2.0


Get rid of toEventually()'s in ListingsViewControllerTests

We should opt for synchronicity if possible.

Closing in favour of folding this into https://github.com/artsy/eidolon/issues/333 .


Update user guide and add reposense config file

Docs
This PR includes the additions to user guide as reviewed in 21/03/19's CS2101 class. There are some minor formatting changes to the developer guide as well.
Reposense
Added author names for @yichong96 and myself to _reposense/config.json so that reposense is able to detect our contributions.

Thank you Liang's  👍


[FEATURE] Ability to publish content types

Is your feature request related to a problem? Please describe.
Currently there is no way to publish SharePoint in 365 content types with Powershell. Changes coming to Content type sync coming in January 2021 will make this very helpful.
Describe the solution you'd like
Method to publish a content type given we can create them with Add-PnPContentType and are expected to use Add-PnpContentTypesfromContentTypeHub to sync in the future.
Describe alternatives you've considered
This is the only method I have found to programmatically publish content types for SharePoint in 365, it does not work with MFA.
Additional context

Hi @MDGH2021 , we have added this cmdlet Add-PnPContentTypesFromContentTypeHub.
You can use that to publish your content types.
https://pnp.github.io/powershell/cmdlets/Add-PnPContentTypesFromContentTypeHub.html

Hi @MDGH2021 , we have added this cmdlet Add-PnPContentTypesFromContentTypeHub.
You can use that to publish your content types.
https://pnp.github.io/powershell/cmdlets/Add-PnPContentTypesFromContentTypeHub.html

I tested this and it did not work. The content type was not published, it was also not added to the site. As well, I received the same error documented here: https://github.com/pnp/powershell/issues/1080
In that thread it is mentioned that despite receiving the error, the content type was still added to their site. It seems in the case of an unpublished content type it will not add.

The documentation for the cmdlet is also contrary to your statement
Adds published content types from content type hub site to current site.
This will add the content types with the ids '0x0101' and '0x01' to the current site. Latest published version of these content types will be synced if they were already present in the current site.

My bad , yeah the docs are correct.
you can push a CT from the hub to another site using that cmdlet.
As for the original question, yeah we don’t have a specific CSOM or REST method for this , so unfortunately can’t implement it.
The social technet link that you posted contains.a hacky workaround which isn’t possible using current PnP authentication methods.


HIGH and LOW functions don't work in all contexts

Not entirely sure if this is a bug or feature request. Certainly it's confusing.
The HIGH and LOW builtin functions seem to not work when evaluating expressions needed during compilation. For example:
SECTION "Example", ROM0

MYCONST EQU $1234

; This works
db HIGH(MYCONST)

; This causes a syntax error
PRINTV HIGH(MYCONST)

In contrast, other functions such as SIN() do work in both contexts.
This is low priority since I can always use x/256/x % 256 instead of LOW/HIGH but it's surprising and annoying since they work otherwise.

Yeah, I completely forgot about this case when I implemented it, thanks.


height calculation is wrong

Hi,
i am having a problem with hight of the div(container) is only 20, through its looking mess.

thanks

Hi, pls check the plugin options on the original page, you can set height!

i mean main container where the hight have to calculated depending on content, to set fixed hight is not right.

ok, got you, I'll take a look at this now and see why this happens _> as I set no height for the container

                     <?php \philippfrenzel\yii2masonry\yii2masonry::begin([
						'clientOptions' => [
							'columnWidth' => 300,
							'itemSelector' => '.item'
							]
					]); ?>

					<?= ListView::widget([
						'dataProvider' => $dataProvider,
						'layout' => "{summary}\n{items}

{pager}",
						'itemOptions' => ['class' => 'item share share-self col0'],
						'itemView' => function ($model, $key, $index, $widget) {
							return $this->render('_view', ['model' => $model]);
						},
					]) ?>

					<?php \philippfrenzel\yii2masonry\yii2masonry::end(); ?>

thanks! I'll check this! -> as the class is not mandantory.. Update will be available in about 10 hours;)

removed the class and now all container settings can be passed over by the options attribute during initialisation


Improvement/Bug: KeyError raised when invalid topic slug passed to latestposts endpoint.

In the _latestposts.py module line 37 a KeyError is raised when an invalid topic slug is passed to the latestposts endpoint. Since the latestposts endpoint is limited in regards to what values can passed to the topic_slug argument, when an invalid topic slug is passed this results in an error returned by the Medium API which ultimately results in an empty dict being returned at medium.py line 82. When the resp['latestposts'] call is made in _latestposts.py this then raises a KeyError which is a rather unhelpful error considering the reason why the error is being raised.
I propose that instead a more meaningful error message be raised in this situation so that users can be better informed in regards to how this error can be avoided. Perhaps an error message similar to the following:
An error occurred when calling the latestposts endpoint. Please ensure that you have passed a valid topic slug. If you are unsure if the topic slug you are using is valid consider using the topfeeds endpoint instead.
Alternatively, if the intention is to avoid raising an error then there should be a check to determine if the response dictionary is empty so that the KeyError is not raised.

Hey @re-alejandro-garcia, Thank you so much for pointing this out. I'll implement this error handling as soon as possible. I'm also working on other improvements in this python package, so I'll roll out everything at once in the upcoming 0.2.8 version on the PyPI.
If you're only interested in this specific improvement, I'll resolve and commit it on Github by the weekend. Then, you can download and directly install it from the source.
Happy coding!


Lucid type error when trying to run simple transaction code

Hello!
I am trying to write a simple code that sends ADA, modeling it similar to your simple transaction code. Last week, I had issues running my code in Vasil, but it seemed this error was resolved. Now, I am getting an error when trying to compile my ts code. However, the error seems to be originating from the lucid code itself. Is this an error needing to be addressed here?
node_modules/lucid-cardano/types/src/types/types.d.ts:208:5 - error TS2411: Property 'version' of type 'number' is not assignable to 'string' index type '{ [assetName: string]: NFTMetadataDetails; }'.

208     version?: number;
        ~~~~~~~

Found 1 error in node_modules/lucid-cardano/types/src/types/types.d.ts:208```

Hey! Make sure you ts compiler doesn't type check the node_module folder. Then it should work fine.
But this line
version?: number;

I could remove from the code and probably should. I just realized the @ts-ignore above that line is removed after building the library.

Thanks for the help! How do I ignore a specific folder? Is that done in the tsconfig.json file?

I guess you solved this issue since you closed it @ssbright but for future googlers, node_modules can be explicitly excluded in tsconfig.json like:
"exclude": ["node_modules"]


Questions about CorrDiff example

my machine got the error about " out of memory"

i have change the "total_batch_size" to 2，
ues "amp-fp16" ，
and use dataset "'2018-01-01' to '2018-01-10'" ten days data
Are there any other places where I can modify to change the model's memory usage?
What is the meaning of the "training_duration: 200000000"?

@MyGitHub-G training_duration is the number of (repeated) samples/images the model sees during the training. If you divide it by the number of unique samples in the dataset, it gives you the number of epochs.


fix use-t rule crashing when test() has no arguments

use-t rule crashes with the snippet below, which obviously is not a proper test file but it's what the linter sees if you run it as you type (with eg. the atom plugin)
import test from 'ava'

test()

I've fixed it and added a test for it.

Thank you :)

Thanks a lot @nfcampos!

New version published.


Disable code generated by webpack for dynamic splitting.

use dynamic import(). I need to add and remove styles (.css) depending on the route.
Before adding, I check for the existence of the tag in html markup and if it is not there, then add it.
But webpack adds the same code to html markup after me. This leads to errors.
How do I leave dynamic splinting but disable code generation that adds tags to html?

It looks like you just deleted our issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Document clean shutdown beyond the FAQ

Currently we only mention in the FAQ that a clean requires sigterm or , that should be in the general docs too and also the http endpoint.
Hmm, should we mention that in the crash recovery error message?

Perhaps it could go https://prometheus.io/docs/prometheus/latest/getting_started/  Add two more sections. "Reloading your configuration" and "Shutting down your instance"

SGTM


[Backup] az backup vault encryption update: Add support for MUA for CMK operations

Related command

az backup vault encryption update
Description

Multi-User Authorization (MUA) now works for update encryption tasks in recovery services vaults.
Testing Guide

History Notes

[backup] az backup vault encryption update: Add support for MUA for CMK operations

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

backup

Hi @zubairabid, Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.

Requesting a review within this milestone.

/azp run

please fix the CI issues.

Please add test for this change to verify if it meets your expectations.

Please add test for this change to verify if it meets your expectations.

@yanzhudd the code has been tested manually. We are not adding an automated test at the moment as it involves cross-tenant calls being made, and the existing automated tests that verify this general functionality are alright.
Resolving the CI issues.

Please add test for this change to verify if it meets your expectations.

@yanzhudd the code has been tested manually. We are not adding an automated test at the moment as it involves cross-tenant calls being made, and the existing automated tests that verify this general functionality are alright.
Resolving the CI issues.

@yanzhudd I'm working on resolving the CI issues - they're related to an unrelated RBAC change and are mostly fixed. But currently my local testing is getting DOS limited so I'll be able to push the fix for it mostly tomorrow, but I've added a unittest skip for today. Can you please approve the PR (we have been targeting this month's release).

Hi @zubairabid,
No worries. We have pushed the release back to next week. As long as these issues are resolved by 07-02, this PR will be ready to catch the upcoming release train.

/azp run

/azp run


Ajax request

How could i show the message if the partial view is loaded by ajax post ?
A bootsrap modal window is loaded via ajax request and i want to show a error message after the successful response. Do i need to add the component  seperately on the partial view

I would not advise you to use the component inside a partial to show notification. What you can do is use the toastr JavaScript API to show a toast notification on the ajax success/complete event.
//On Ajax success event in client side javascript
function(data){
  // Your logic here

  //show toast notification
  toastr.error('I do not think that word means what you think it means.', 'Error Title!')
}

Is there any way that i could find the latest added message

Can you give me more details regarding the case when you want to find the latest added message?

Check the comments in code below
//On Ajax success event in client side javascript
function(data){
  // Your logic here

  //show toast notification
 // 'I do not think that word means what you think it means.' message added on server side
  toastr.showLastError()
}

There is no such API in the toastr.js library

#23 supports this.


Read vault config from environment?

I don't want to respecify --host in every service we have.  Just have some global EnvironmentFile that sets the setting.

or even better, put consul to good use? See: https://github.com/hashicorp/consul-template
We would only have to seed it with a vault token in the EnvironmentFile  and we could even monitor for CONFIG changes with restarts...

Vault and Consul are not necessarily used in conjunction. I'd like to keep this tool generic in that sense.   If we want  to have consul integration, then we can use already existing approaches that give the same features as vaultenv + more. Namely Consul-template or confd.

I don't really understand how this pertains to vaultenv itself. You will still need a mechanism to get the EnvironmentFile on every host you want to have the same config on.
We could make things a bit easier. Some options (not mutually exclusive):

Respect VAULT_ADDR when set.
Introduce VAULT_HOST and do DNS SRV record lookups to get the port, then fall back to the default Vault port. (We probably also want to introduce VAULT_PORT for symmetry)

Thoughts?

I think we should keep this simple:
Respect either VAULT_ADDR or VAULT_HOST and VAULT_PORT when set.
Command-line flags should take priority over env variables when set.
This way we can set these env vars in the EnvironmentFile everywher and don't have to build new packages when something with vault changes.

Context for non-Channablers reading along: internally, we use the systemd services, which have an EnvironmentFile directive where you can specify a file with environment variables to make available to the current process.
This discussion is about how we can make vaultenv work nicely with this, by introducing environment variables for configuration.

I've started with this on the config-from-env branch. I've almost got things working, except I'm having trouble with the CLI flags, such as --no-connect-tls


How to create a custom dropdown in the menu

I'd like to create a custom dropdown menu item that will insert merge codes into the document at the cursor position. In the ProseMirror docs it describes doing this by defining a schema via a node spec.
In reviewing the tiptap documentation I'm not recognizing how to leverage this capability. How would this be done in tiptap? Thank you!

Actually I just found it on the github readme: https://github.com/scrumpy/tiptap#create-custom-extensions so closing this out.


just some stuff

various refactorings on our way forward

This PR has sprawled far past its original purpose, it now contains lots of housekeeping (dependency upgrades, model refactorings, more documentation for observables, more features for observables, now mypy & model tests in CI), I'm just going to land it to move forward so that any potential contributors get a more accurate baseline


test: fix iam live test

This PR updates the IAM live server test to fix the comments that described the required config properties, plus it adds a test that verifies that a refresh token can be obtained using the authenticator.

@dpopp07 @rmkeezer Dustin, Matthew... I added you both as reviewers so you're aware of the IAM live server test that's part of the Java core.   Perhaps we could add something similar for Node and Python.

:tada: This PR is included in version 9.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


search={true} triggers a warning

warning.js:44 Warning: Failed propType: Invalid prop search of type boolean supplied to Picker, expected string. Check the render method of Search.

Happens the same for me

Thanks! This is fixed in 0.4.0, just released to npm


Support annotations without appearance streams

Most annotations, even unsupported ones, render just fine in PDF.js because of their appearance stream. However, there exist PDF files with annotations that do not have an appearance stream (even though this is a deprecated practice). In the latter case, PDF.js displays nothing. We need to have a fallback to a default appearance stream in that case, or create an appearance stream ourselves using the data present for the annotation (such as quad points and color). Below is a list of known PDF files that contain annotations without appearance streams, for which the types are listed.

http://pdfkit.org/docs/guide.pdf (Underline, Highlight and StrikeOut)
http://www.plpdf.com/examples/PLPDF-SDK/Simple_Examples/37_Markup_annotation/markup.pdf (Underline, Highlight and StrikeOut)
https://drive.google.com/file/d/0B6YhVF9p-y2AU0RlYjRKakFQRjQ/view?usp=sharing (Highlight, from #6382)
https://drive.google.com/open?id=0B6YhVF9p-y2Ad3VsWDRHRkRxY2c (FreeText, from #6246)
https://drive.google.com/file/d/0B-_5BgJGB33Ea2dQZnpUY2FvYkk/view (FreeText, from #6564)
http://mirrors.rit.edu/CTAN/macros/latex/contrib/pdfcomment/doc/example.pdf (practically all annotation types, from #1973)

http://www.pdfill.com/example/pdf_commenting_new.pdf

Waiting for this feature..^^

This is still a pretty serious issue.  Apparently poppler, one of the main PDF libraries used by Okular, etc, creates annotations without appearance streams.
When loading them into PDF.js its impossible to see the annotations.

@edsontrick The question is not related to this particular issue. Please post it on IRC instead of on an unrelated issue.

Somebody should also create an issue in poppler. The /AP is required in PDF 2.0 except for Popup, Projection or Link.

The corresponding issue for Poppler seems to be this one:
https://gitlab.freedesktop.org/poppler/poppler/issues/522

Note: "Opened 9 years ago by "
... 9 years ago.  Not going to be fixed anytime soon ;)

But they did work on it, there were three patch sets submitted.

On the upstream poppler bug someone commented on the poppler issue tracker in response to someone saying the poppler bug would be responsible for projects using poppler not being able to properly add annotations:

It's not really our fault if pdf.js is buggy. This bug has nothing to do with what you're complaining though, so please don't hijack it.

So it seems not to be clear which project should/can fix what. As I understood it, pdf.js could render annotations without an appearance stream, but PDF software should create appearance streams, at least when making PDFs with version 2.0, right?
Maybe someone from this project like @timvandermeij could write a message to someone responsible in poppler to constructively clear up what poppler can and can't do to change the end user experience of not being able to show annotations created in one software when viewing in another?
That would be highly appreciated!
If that does not seem like a good idea, what can be done instead to improve the situation?

It doesn't really matter who is at fault. From a PDF 2.0 point of view, it's poppler, and from a PDF 1.* point of view, it's nobody and everyone. The head guy on poppler is "Albert Astals Cid" and he did comment on the issue so he is aware. It is useless to pressure people in OS projects on anything, the only result is that they'll feel bad. The best thing to do is to contribute yourself, if you can. I can't contribute code either, but I can help with test PDFs and strategies on the appearance content streams and giving feedback on produced PDFs because I implemented some of the annotations in Apache PDFBox.

Hi folks! Just wanted to cross link poppler issue 839, it's roughly the counter part at poppler side. No idea yet when we'll get to it though.

@haxtibal Thank you for the information! I agree that both sides need work, so Poppler should provide the appearance stream and PDF.js should be able to build the appearance stream based on the information in the data dictionary if it is missing.

Highlight, underline, squiggly and strikeout annotations without appearance streams are now supported by PDF.js thanks to the PR above. This issue remains open to track the remaining types, in particular freetext annotations.

Square and circle annotations without appearance streams are now also supported (see the PR above).

Line annotations without appearance streams are also supported now since PR #13003.

Further improvements, for many different Annotation-types without appearance streams, were made in PR #13448; these include support for opaque stroke/fill colours and handling of inconsistently specified coordinates.

Is this issue being worked on?
Is there a likelihood of it being resolved this year?
Is there currently some suggested workaround?
Thanks

FreeText annotations without appearance streams are now also supported, see PR #15587.

Is there currently some suggested workaround?

The correct thing is to generate PDF documents with appearance streams for all annotations, which is also a requirement in the PDF 2.0 specification (as far as I recall).

Given that FreeText was the primary remaining type missing here, let's close this old issue as fixed now (and further types should be evaluated/fixed separately).


clear cache browser

I have made an authentication system. When i try to logout, and press back button, or try to login with other account, it always get back to the first refreshed page.
i have used cookies, and clear cookies when logging out.
app.get('/logout', (req, res) => {
    if (req.session.user && req.cookies.user_sid) {
        res.clearCookie('user_sid');
        res.redirect('/');
    } else {
        res.redirect('/login');
    }
});

what should i add to clear caches after log out ???? and thank you.

Hello @hathemi, I would recommend you use stack overflow or any programming
forum for your question since it's not directly related to RFM-node. You'll
get a lot of good answers there.
Also, you can create a GitHub repository for your projects so I can see
your project code to give you some assistance from time to time.
Note: you'll have to remove all sensitive information such as username and
passwords

after a lot of search and like i mentioned before i want to  create a hole authentication system. I wonder is there a way to reload middleware for each time user is logged in.
//  static files from rich-filemanager module.
app.use( express.static('node_modules/rich-filemanager') );
// Static files from the demo public folder
app.use( express.static(__dirname + '/public') );
//Filemanager route
app.use( '/filemanager', filemanager( __dirname + '/uploads', config ) );


Usage example fails with exception

Hi,
Thanks for all the great work on making CRDTs generally accessible to developers.
I tried the examples snippets for client and server in the README.md but the client fails with an exception.
server.py
import asyncio
from websockets import serve
from ypy_websocket import WebsocketServer

async def server():
    websocket_server = WebsocketServer()
    async with serve(websocket_server.serve, "localhost", 1234):
        await asyncio.Future()  # run forever

asyncio.run(server())

client.py
import asyncio
import y_py as Y
from websockets import connect
from ypy_websocket import WebsocketProvider

async def client():
    ydoc = Y.YDoc()
    async with connect("ws://localhost:1234/my-roomname") as websocket:
        WebsocketProvider(ydoc, websocket)
        ymap = ydoc.get_map("map")
        with ydoc.begin_transaction() as t:
            ymap.set(t, "key", "value")

asyncio.run(client())

fails with
Task exception was never retrieved
future: <Task finished coro=<WebsocketProvider._run() done, defined at /home/chris/.local/lib/python3.7/site-packages/ypy_websocket/websocket_provider.py:29> exception=ConnectionClosedOK(Close(code=1000, reason=''), Close(code=1000, reason=''), False)>
Traceback (most recent call last):
  File "/home/chris/.local/lib/python3.7/site-packages/ypy_websocket/websocket_provider.py", line 34, in _run
    await process_sync_message(message[1:], self._ydoc, self._websocket, self.log)
  File "/home/chris/.local/lib/python3.7/site-packages/ypy_websocket/yutils.py", line 120, in process_sync_message
    await websocket.send(reply)
  File "/home/chris/.local/lib/python3.7/site-packages/websockets/legacy/protocol.py", line 635, in send
    await self.ensure_open()
  File "/home/chris/.local/lib/python3.7/site-packages/websockets/legacy/protocol.py", line 953, in ensure_open
    raise self.connection_closed_exc()
websockets.exceptions.ConnectionClosedOK: sent 1000 (OK); then received 1000 (OK)

Any pointers why this could be?
Thanks,
Chris

Hi Chris, thanks for the kind words.
I'm not sure we had this exception in the past. Maybe websockets raises it in newer versions?

I downgraded websockets to 10.1 but the exception remains on my system. If you can't reproduce, maybe the problem is with my setup.

I can reproduce, it's actually because the client immediately finishes after sending an update, and the server echoes back this update to the client, which has already disconnected. If you add await asyncio.sleep(0.1) at the end of the client function, the exception disappears.
This is just an example showing a minimal server/client setup, but feel free to open a PR if you want to improve it.

Thanks for taking the time to reproduce. Ok, haven't worked much with asyncio in python yet, but will send a PR if I have something. Cheers!


Fix tns debug ios --emulator

Fixes https://github.com/NativeScript/nativescript-cli/issues/1253

You should update the version in package.json

:+1:


Make UploadFile type codable.

Motivation
I fixed UploadFile type to confirm Codable to upload binary data either a reference of model or binary data as string a.k.a a primitive type.
This changes allows to upload following model that could not be uploaded because of UploadFile can't be encode or decode when UploadFile is contained in user defined models.
I uploaded a minimal, reproducible example here.
paths:
  /withModel:
    post:
      summary: ''
      operationId: post-withModel
      responses:
        '200':
          description: OK
      requestBody:
        content:
          application/json:
            schema:
              type: object
              properties:
                uploadData:
                  $ref: '#/components/schemas/UploadModel'
    parameters: []

...
components:
  schemas:
    UploadModel:
      title: UploadModel
      type: object
      properties:
        binary:
          type: string
          format: binary

This model is converted to this code and swift complier could not build as following error.

Trade-off
When generating specs with --option typeAliases.File:UploadFile, TestSpec could not be build because of the error below.
SwagGen/Specs/TestSpec/generated/Swift/Sources/Requests/GetFile.swift:52:49: error: cannot convert value of type 'Data' to expected argument type 'File' (aka 'UploadFile')
                case 200: self = try .status200(data)
                                                ^

To overcome this issue, we have to define public typealias File that responds to request body or response.
If there any concerns, please let me know freely :)

@yonaskolb Hi, is it possible to review this PR?


Issue with loading class and container height on load

I've searched quite widely about this but can find no definitive answer, so hoping somebody can help shed some light on this.
When an owl carousel loads, it doesn't know the height, so the boxes flicker and then pop to the size of the image as the plugin reads the size of the images.
To help get around this I originally tried adding some styles of .owl-loading, but no matter what I do none of my styles are applied there, and to be honest I can't even see it appearing on page load.
In an attempt to get around this I tried adding my own custom 'loading' class and then use the onInitialized callback like this:
onInitialized: function() {
    $('.loading').removeClass('loading');
}

That kind of works, but it fires too early. So I can see my loading class then, plugin initalises then remove my class, the carousel collapses in size then reappears as the images are loaded.
What I would like to do is only have that class removed AFTER the images are loaded, but there doesn't appear to be a callback for that. Any further ideas here?

Same here, still looking for a solution...

I am also using onInitialized, to set the height of each slide to the tallest slide. The problem is onInitialized is firing before anything in owl-stage is present.

Right now, the only solution, unfortunately, is a timeout.


Crashes when plugging in scanner via USB on Android 12

You can reproduce this by plugging in a scanner for which the app doesn't already have permission. On my device it instantly crashes 100% of the time.
According to the stack trace and decompiled code, the problem is in the USBManager.getAvailableScannersList() method. If it does not already have permission to open the attached device, it constructs an intent to get permission for that device with PendingIntent.getBroadcast(context, 0, new Intent(ACTION_USB_PERMISSION), 0).
While this has been a warning for a while now, starting in Android 12 this will crash every time - callers now have to specify either PendingIntent.FLAG_MUTABLE or PendingIntent.FLAG_IMMUTABLE when calling this method. In your case, you should probably replace this with PendingIntent.getBroadcast(context, 0, new Intent(ACTION_USB_PERMISSION), PendingIntent.FLAG_MUTABLE).
tl;dr Use PendingIntent.getBroadcast(context, 0, new Intent(ACTION_USB_PERMISSION), PendingIntent.FLAG_MUTABLE)

This was caused by me using an outdated library version (specifically, the one located in this repository). You might want to update that version to eliminate confusion, but anyway updating to the latest SDK fixed my problem.


License?

Hello, I was wondering what the license for this database is? If I include it in my own project so it can work offline, is that allowed, what's the deal with attribution, etc.
Please let me know, thanks!

Hey @kokone , thanks for your interest in the project! I just added an MIT license, so you're free to reuse and redistribute any of the code. I'd appreciate it if you credited me in some capacity, but you don't have to.

Cheers @drewthoennes ! I'll absolutely credit you.


Convenience methods to support easy response validation

I would like to validate request and response in a test. For the request this is supported very well. The Response can be validated using the OperationValidator but to use it I would need a Path and Operation object wich is typically managed by the RequestValidator.
A handy solution would be the following new method for RequestValidator:
public OperationValidator getValidator(Request request) throws ValidationException {
  Pattern pathPattern = PathResolver.instance().findPathPattern(pathPatterns.keySet(), request.getPath());
  if (pathPattern == null) {
    throw new ValidationException(String.format(INVALID_OP_PATH_ERR_MSG, request.getURL()));
  }
  Path path = pathPatterns.get(pathPattern);
  Operation operation = path.getOperation(request.getMethod().name().toLowerCase());
  if (operation == null) {
    throw new ValidationException(String.format(INVALID_OP_ERR_MSG, request.getURL(), request.getMethod().name()));
  }
  return getValidator(path, operation);
}

I am pretty sure, using are some refactoring the resulting code duplication in RequestValidator could be minimized.
With such an approach I could do the following validation:
OperationValidator opValidator = new RequestValidator(api).getValidator(request);
ValidationData<?> validation = new ValidationData<>();
opValidator.validateHeaders(response, validation);
opValidator.validateBody(response, validation);
if (!validation.isValid()) {
  throw new ValidationException(INVALID_RESPONSE_ERR_MSG, validation.results());
}

Even better the OperationValidator could offer this as convenience methods:
public void validate(Response response) throws ValidationException {
  validate(response, new ValidationData<>());
}

public void validate(Response response, ValidationData<?> validation) throws ValidationException {
  validateHeaders(response, validation);
  validateBody(response, validation);
  if (!validation.isValid()) {
    throw new ValidationException(INVALID_RESPONSE_ERR_MSG, validation.results());
  }
}

With that the validation get a one-liner:
new RequestValidator(api).getValidator(request).validate(response);

As I am not sure about your design considerations it is also possible to add the convenience methods for response validation directly to RequestValidator but it would need one parameter more for each method:
public void validate(Response response, Request request) throws ValidationException

public void validate(Response response, Request request, ValidationData<?> validation) throws ValidationException

Hi,
Thanks for opening an issue!
I don't have much time to add this QoL methods right now but I'll definitely have a look!
Don't know (remember?) why this is not available yet...

@bdremynoel If you could decide where to put the methods (RequestValidator only or combination with OperationValidator) I should be able to create a PR.

Both locations seem fine since they are complementary to each other.
OperationValidator validates all aspects of requests and responses and RequestValidator is on top to handle cache for paths and validators.


registry: Node registration process should be 2 stage

Followup from #2018:
Currently registering a compute/keymanager worker requires being able to attest with IAS.  This is thwarting attempts to restrict IAS attestations via the proxy to valid compute/keymanager workers, since the initial registration will be impossible if such an access control check were added.
The probably correct way to do this is to have a two staged initial registration process, the first that grants attestation access, followed by the actual node registration(s) as normal.

Moving over from #3413:

Before a storage node is eligible to be elected into a storage committee it should self-report that it is fully synced. This should be done by flipping a flag in the registration descriptor (could be added into a new "storage" capability).

We should probably generalize such a 2-step registration process by having some kind of a "ready to participate in a committee" flag.


Is it possible to read from excel file and create chart and have reference to original  excel file?

I want to use this library and it looks great but Is it possible to read from excel file and create chart and have reference to excel? I do see read data and read excel but I need data coming from some original excel and then have user refer to it for verification when they click on read data inside ppt slide.

1: Use the great XLSX library to read in an Excel doc, grab the rows you want
2. Show it to a user on-screen
3. Use the addChart() method to create a Presentation with charts


Ubuntu 20.04

Hey @crohr, thanks for creating this and running packager.io all these years!
The latest Ubuntu LTS was released last week - would be great to see pkgr support it.
Also, it appears the doc subdomain has an expired SSL certificate: https://doc.packager.io/documentation

Hi @ankane, thanks for the ping. Ubuntu 20.04 support is coming, and I'll fix the documentation certificate ASAP!

@ankane ubuntu 20.04 support has landed in master, and is now available on Packager.io (just use the ubuntu-20.04 target).

You're awesome, thanks @crohr!


submit blob should probably take [u8; NS_ID_SIZE] instead of u32

Closed - it's now a wrapper type.


ICD9 50 codes

I am wondering if you could share the top 50 ICD codes used in your work? Did you use DIAGNOSES_ICD to extract ICD codes? I looked into CAML_mimic3_50/preds_test.psv and found some codes, such as 37.22 and 96.72 are not in the MIMIC-III data.

The study used both DIAGNOSES_ICD and PROCEDURES_ICD. Maybe 37.22 and 96.72 are from the procedure codes?


Introduce a cell model based on Thevenin

Use the Thevenin to implement an ECM model.
To do list:

[x] Gather overpotentials from RC elements
[x] Make sure it works with a state estimator
[x] Review implementation with @c-randall
[x] Document how to make new CellModels, start a list

@c-randall , this is all ready. Not sure if you have bandwidth or desire to review, but want to let you know the first iteration is complete


Fix morphed normals.

Our morphNormal GLSL function treats each normal target as a displacement from the base normal. However the normal that is extracted from the tangent frame is actually an absolute normal, not a displacement vector.
In other words, if b is the base normal, we were doing:
b = b + w0 * n0 + w1 * n1 + ...

This is obviously wrong, since the base normal has an overpowering influence.
The fixed math looks like this:
b = b + w0 * (n0 - b) + w1 * (n1 - b) + ...

Fixes #5584.

So reasonable! Thanks for your fix!


Check for admin

(TODO)
Check if alwsl is run as admin. That should never happen, is not required, and may have catastrophic effects on the system.

Reliable check in batch:
:isAdmin
fsutil dirty query %systemdrive% >nul
exit /b

Assert that errorlevel is not 0.

Done in private dev branch.


Scheduling

Model currently inefficient (due to weak error functions)

Codecov Report

Merging #13 (3b73884) into main (53a9a2d) will decrease coverage by 6.12%.
The diff coverage is 45.16%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   90.40%   84.27%   -6.13%
==========================================
  Files           7        8       +1
  Lines         198      229      +31
==========================================
+ Hits          179      193      +14
- Misses         19       36      +17

Impacted Files
Coverage Δ

src/scheduling.jl
45.16% <45.16%> (ø)

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 53a9a2d...3b73884. Read the comment docs.


updated logicalSwitchesGlobal with queryParameters

Thanks! This looks good. At some point we will need to enhance nsxramlclient with a helper method to collect and search multiple pages. This is on my shorter term TODO list


Replace occurrences of "on-premise" with "on-premises"

This replaces #28364.
I did a global find & replace on the use of on-premise in the docs. That is technically not the correct term.
premise: a previous statement or proposition from which another is inferred or follows as a conclusion.
premises: a house or building, together with its land and outbuildings, occupied by a business or considered in an official context.

@cdhunt : Thanks for your contribution! The author, @Juliako, has been notified to review your proposed change.

we're reviewing this PR for impact.

@cdhunt - thank you for catching these! (and for re-submitting these in a new PR).


Babel consolidation

~Forked from #3874. see diff~
Hit a snag by forgetting to update the content babel. Thought it worth sharing a common content.
It also means our babel deps can be centrally managed in common.
We still need @babel/core, and the jest config as they're still running the old babel.

Updated with merge.


Line chart dataset order is wrong

This fiddle shows the problem: https://jsfiddle.net/dmig/97pxd7eL/4/
When I specify datasets, I expect them to be drawn in that order:
  datasets: [
    {
        label: "Blue",
      fill: true,
      backgroundColor: "rgba(32, 162, 219, 1)",
      data: dataPack1,
    },
    {
        label: "Red",
      fill: true,
      backgroundColor: "rgba(196, 93, 105, 1)",
      data: dataPack2,
    },
    ]

Workaround is to reverse their order, but label order in legend/tooltips is also reversed,

Similar to a bunch of other issues that are open, but I can't find them at the moment.

If you could just point me to the section where you choose the render order of datasets I could take a look, just don't have time to dig around looking.
Also, here is that similar issue: #2219

Datasets are drawn here: https://github.com/chartjs/Chart.js/blob/master/src/core/core.controller.js#L354-L358

This will be resolved in #4008 which adds the ability to fill to another dataset. This means that the draw order will become irrelevant since the correct solution for the blue dataset will be to fill to the red one instead of all the way down to 0

Tied in with my comment on #2219

Closing per my comment from march. This can be fixed using the fill to other dataset functionality. See https://jsfiddle.net/gz4ev7jr/

fill to other dataset functionality

Closing per my comment from march. This can be fixed using the fill to other dataset functionality. See https://jsfiddle.net/gz4ev7jr/

Funny how your jsfiddle doesn't work.


Null Safety or not?

I'm totally confused.
This repo has a release with the title "Update to null safety". When I download the source code and look into it it's not null safe. Also it's tagged as a 0.3.4 version, but in the pubspec.yaml it's 0.3.2.
And out of curiosity: What's the idea behind having both a main branch and a master branch?
What's going on?

It looks like commit 6f44bd62afab5aa0cdb52a9cb176a62ae7ea205f on master is the commit that updates the code to have null safety, but the release is tagged on main, which hasn't had this update merged.

Some information about the version with null safety?

Hey bro @pinkfish , there is any chance you update your pub.dev package to Null Safty ?

Opps, ok moved to main.


Formatter timeZone problem

Hi,
I have problem with Formatter. asTime method always add 2 hours.
I think that it is because php on my server date.timezone option is set on Europe/Warsaw.
But Yii Formatter have $defaultTimeZone = 'UTC' by default.
Yii should sync this settings, because for now I need to configure this option in this way:
        'formatter' => [
            'defaultTimeZone' => date_default_timezone_get(),
        ],

dates and times stored in the database are usually in UTC so the input value is considered UTC by default. If you store it in a different timezone you should configure it explicitly.
Note that using date_default_timezone_get() could result in unexpected results because the timeZone setting of the application sets this.
http://www.yiiframework.com/doc-2.0/guide-output-formatter.html#time-zones

So there are an error, becouse I do not change any configuration about timezone in database.

the fact that you did not change the config does not tell anything about whether it is correct or not.
If you have everything in one timezone you should set the configs to state the timezone explicitly.
for further questions please use the forum, github is for bug tracking.

@cebe I still think that it IS a bug. Because Yii should by default use php server timezone do convert user input and display proper time, but store in files and database in UTC format.
For now, Yii store time data directly based on user input without casting it to UTC time.

@cebe I still think that it IS a bug. Because Yii should by default use php server timezone do convert user input and display proper time, but store in files and database in UTC format.

it does exactly that:

database time is UTC:

https://github.com/yiisoft/yii2/blob/master/framework/validators/DateValidator.php#L108
https://github.com/yiisoft/yii2/blob/master/framework/i18n/Formatter.php#L82

Output is php server default:

https://github.com/yiisoft/yii2/blob/master/framework/base/Application.php#L470

For now, Yii store time data directly based on user input without casting it to UTC time.

this is not true, see links above. If it is the case in your app, you are doing something wrong.

You introduced timestampAttribute field. Why Yii does not use database data/time formats for storing data/time?
Timestamp format can not be easy filtered. (By year, month, day etc.)

I do not understand your question, please explain in more detail.

Yii should use default DATE, DATETIME and TIME types to store time information in database.
Why you force to use INTEGER as type for storing time information?
It is very uncomfortable way to store that, because databases have defined many features (like procedures and functions) to manage time information. Also UNIX timestamp have a lot of limitations. First is can not easy find records from chosen year or month, next is you can not save date before 1970 year.
Yii should provide support for database date and time data types.
Why you force me to store time in UNIX timestamp (date validator use it by default)?
I know that I can define timestampAttributeFormat, but it depend on format in database and database engine. Finally, I can not define it for entire application.
Why Yii can not determine it based on data in database?

Why you force to use INTEGER as type for storing time information?

we do not force it, it is just a default that works cross db.

First is can not easy find records from chosen year or month, next is you can not save date before 1970 year.

You can create a condition like WHERE date > 12349879 AND date < 12384536. And it can store dates before 1970, these are just negative numbers.

Yii should provide support for database date and time data types.

It does support this by timestampAttributeFormat.

Why Yii can not determine it based on data in database?

because as you said, it depends very much on the database and environment. If it were only mysql this would be quite easy but yii supports more that that.

Finally, I can not define it for entire application.

you can:
Yii::$container->set('yii\validators\DateValidator', ['timestampAttributeFormat' => 'php:Y-m-d H:i:s']);

Moreover, there are more features are missing about date and time processing in Yii right now.

please be more specific. imo datetime support is very complete right now.

Really? I do not think so.
When I use negative integer I have an exception:
'-20602' is not a valid date time value: Failed to parse '-20602' as a UNIX timestamp.

'-20602' is not a valid date time value: Failed to parse '-20602' as a UNIX timestamp.

well, this is a bug...

The problem with the negative values seems to remain. This is the report:
exception 'yii\base\InvalidParamException' with message 'Failed to parse '-154317600' as a UNIX timestamp.' in /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php:645 Stack trace: #0 /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php(558): yii\i18n\Formatter->normalizeDatetimeValue(-154317600, true) #1 /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php(462): yii\i18n\Formatter->formatDateTimeValue(-154317600, 'dd-MM-yyyy', 'date') #2 [internal function]: yii\i18n\Formatter->asDate(-154317600) #3 /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php(285): call_user_func_array(Array, Array) #4 /var/www/akadim/vendor/yiisoft/yii2/widgets/DetailView.php(155): yii\i18n\Formatter->format(-154317600, Array) #5 /var/www/akadim/vendor/yiisoft/yii2/widgets/DetailView.php(136): yii\widgets\DetailView->renderAttribute(Array, 8) #6 /var/www/akadim/vendor/yiisoft/yii2/base/Widget.php(97): yii\widgets\DetailView->run() #7 /var/www/akadim/backend/views/coach/view.php(86): yii\base\Widget::widget(Array) #8 /var/www/akadim/vendor/yiisoft/yii2/base/View.php(325): require('/var/www/akadim...')

The error message does not exist in the code after the fix so it seems you are not using the latest version. Note that this will be fixed in 2.0.5 which is not yet released.

Yes, my mistake, I watched the dates and thought that 2.0.5 was released. So this is one more reason to expect the release, thank you. :+1:

this time I have updated to 2.0.5, and test it again. It seems that the bug remains or I am doing something wrong. [I have changed the date for test purposes].
exception 'yii\base\InvalidParamException' with message 'Failed to parse '-269748000' as a UNIX timestamp.' in /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php:645 Stack trace: #0 /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php(558): yii\i18n\Formatter->normalizeDatetimeValue(-269748000, true) #1 /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php(462): yii\i18n\Formatter->formatDateTimeValue(-269748000, 'dd-MM-yyyy', 'date') #2 [internal function]: yii\i18n\Formatter->asDate(-269748000) #3 /var/www/akadim/vendor/yiisoft/yii2/i18n/Formatter.php(285): call_user_func_array(Array, Array) #4 /var/www/akadim/vendor/yiisoft/yii2/widgets/DetailView.php(155): yii\i18n\Formatter->format(-269748000, Array) #5 /var/www/akadim/vendor/yiisoft/yii2/widgets/DetailView.php(136): yii\widgets\DetailView->renderAttribute(Array, 8)

Yeah sorry, we were urged to release a security fix as 2.0.5, this release only includes the fix for the security issue, the regular release which includes this fix is now 2.0.6.

Btw, the due date for the next regular release is available here: https://github.com/yiisoft/yii2/milestones

Ok, I had understood this just now, that' s why I deleted my message, thank you for the quick response


Fix tracepoint_serializer for rvm gemsets

Motivation
Get sorbet to work nicely with RVM gemsets -- fixes #1044 -- by altering the regular expression in tracepoint_serializer.
Test plan
Added a test -- see test/snapshot/partial/local_rvm_gemset_gem

An amazing test!! Thanks!


Save Shopify discount options

As a shop owner,
I want to save the discount options as a new discount
so that it can be used in an offer for data licensing.
Get the data from the Discount Settings UI and send it to the /api/latest/discount/{product/order} to save the discount data.
The structure of the sent data should be:
discountType: string;
discountValue: number;
purchaseType: string;
appliesTo: string[];
minValue: number;
minQty: number;
maxUse: number;
onePerUser: boolean;
This endpoint will create a new Discount based on the options.

Missing fields in description
combinesWith
startsAt
endsAt


raftstore: polish the availability check on conf change requests

What is changed and how it works?

Issue Number: Close #16465

What's Changed:
When calculating the impact of conf change, include all operations into considerations.

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None

/merge

/cherry-pick release-7.5


Convert fail if meeting is longer than 3 hours

Hello,
I use your code to convert presentation to mp4 it worked very well until we tried to convert long meeting (3hours+). It seems it's a limitation from exec command.
I migrate  exec command to spawnSync and now it's work fine.
Best regards,
Romain

We have removed  exec long ago. Try the latest version & it should work perfectly.


Notes on using Lonboard in a remote (esp JupyterHub) environment

I was pairing with @wildintellect on debugging why Lonboard wasn't working in a JupyterHub environment and we came up with a couple notes.
In particular, it's very easy to get lost in environments. Generally a JupyterHub server has at least two Python environments. There's a server environment in which JupyterHub itself is installed and from which it's launched, plus one or more user client environments. As far as I can tell, a Jupyter widget needs to be installed in both environments, and presumably those versions must match.
The widget installed on the server environment is necessary to provide the JS bundle as part of the webpage the user loads, while the widget installed on the user environment is necessary so the Python model works.

Are there ways to provide helpful errors when no JS code is found? Only a JS console error is produced. Is there some way to catch that and provide a more helpful error to the user?
Is there a good way to validate that the frontend and backend versions of lonboard match?

Aside from lonboard-specific tasks, I wonder if there are tasks that JupyterHub could do that would simplify the loading of widgets so that they don't need to be separately pre-installed on the server. Additionally, ensuring versions match between the user's environment and the global server environment sounds messy.
I've noticed that Colab handles this problem quite well (I think it's with their googlecolab/colab-cdn-widget-manager project). In particular, the first time that a widget is attempted to be rendered, the JS code for that widget is dynamically added to the page. This means that
!pip install lonboard
from lonboard import viz
viz([])

will just work, even without restarting the kernel or reloading the browser tab, because the JS side doesn't need to exist already when the Python side is created.
cc @batpad

FWIW In my own IPYwidget library buckaroo, I addressed some of this.
Users frequently pip installed the library without restarting the server.  This causes a JS error with no python error.   I added a check on the MTime of the python library, and the start time of the server.
https://github.com/paddymul/buckaroo/blob/main/buckaroo/widget_utils.py#L37-L55
I'm interested to hear about other approaches around this problem.  It's a really poor experience for an initial user install.  Maybe you could source the JS from unpackage when the user would encounter the problem.

discussion on this before. See especially https://github.com/manzt/anywidget/issues/385#issuecomment-1821263446.

says "... a virtual environment (Python venv, Conda environment or what have you) where package jupyterlab has been deployed — one deploys their widget package in the same environment as where Jupyter Lab lives" (bold mine).
When I was running for example your NYC building notebook from a Jupyter Lab instance launched from my jupyterlab conda env -- to which I had made my lonboard conda env visible as a kernel with the usual ipykernel dance -- I was having rendering issues e.g.
[1] m
Map(layers=[PolygonLayer(extruded=True, get_elevation=<pyarrow.lib.FloatArray object at 0x0000028B03972680>
[
…

And similar errors for ScatterPlot layers and viz([])
In order to fix this, I just installed jupyterlab directly in my lonboard env, launched JLab from that env and bingo.
Unless there's a "JS dance" to expose the JS of conda envs in kernels exported to jupyterlab, I think that means we need to duplicate jupyterlab installs for those conda envs where there is JS.
lonboard v. '0.9.3' in Jupyter Lab 4.2.1

I just installed jupyterlab directly in my lonboard env, launched JLab from that env and bingo.

FWIW this is what I usually do personally.
For other cases, all you need is to install anywidget in the JupyterLab environment, so that anywidget's JS code exists in JupyterLab when you start the page. Then you can install lonboard separately inside a user environment.


Cannot use gittio with nvm

I installed Node.js through nvm, and I get the following error:
$ node -v
v5.3.0

$ gittio install <module>
[ERROR] gittio requires the global Titanium CLI: [sudo] npm install -g titanium

$ npm install -g titanium
/Users/user/.nvm/versions/node/v5.3.0/bin/ti -> /Users/user/.nvm/versions/node/v5.3.0/lib/node_modules/titanium/bin/titanium
/Users/user/.nvm/versions/node/v5.3.0/bin/titanium -> /Users/user/.nvm/versions/node/v5.3.0/lib/node_modules/titanium/bin/titanium
/Users/user/.nvm/versions/node/v5.3.0/lib
└──<EMAIL_ADDRESS>
$ gittio install <module>
[ERROR] gittio requires the global Titanium CLI: [sudo] npm install -g titanium

Could it be that you have tishadow installed? It used the sys module which causes a warning in Node 4 or later which corrupts the Titanium json output that ultimately can cause this error: https://github.com/FokkeZB/gittio/issues/109

Yes I do have tishadow installed. Thanks for the feedback


Uncaught Error: Cannot call function 'getPosition' on missing remote object 214

[Enter steps to reproduce:]
Clicked on "Continue Rebase" on GitHub Desktop. 1 conflicted file, drop down option "Open in Atom."
Atom: 1.42.0 x64
Electron: 4.2.7
OS: Mac OS X 10.15.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot call function 'getPosition' on missing remote object 214
At /Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:222

Error: Cannot call function 'getPosition' on missing remote object 214
    at throwRPCError (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:148:17)
    at /Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:406:5
    at EventEmitter.ipcMain.on.args (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:273:21)
    at EventEmitter.emit (events.js:182:13)
    at WebContents.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:368:21)
    at WebContents.emit (events.js:182:13)

Commands

Non-Core Packages

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/16630 so you can subscribe there for updates if you'd like (we believe that this will be fixed in a future Electron upgrade).
About your steps to reproduce, can you reproduce consistently or did it happen just one time?

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Failing to load config variable

Hi all,
I am trying to read config variable from that I defined in project config but browser complains about mot finding the config file.
In the project.config.ts :
export class ProjectConfig extends SeedConfig {
  BACKEND_BASE_URL = argv['backend_base_url'] || 'http://localhost:8080';

And in one of my service classes I have:
import {BACKEND_BASE_URL} from '../../../tools/config';  // Yes, it is a valid pass
...
return this.http.post(BACKEND_BASE_URL + '/api/login', body, options).toPromise()
...

The browser tries to pull it as http://localhost:3000/tools/config.js and of course failing with 404.
Is it even supported?

project.config.ts is only for tooling configuration.
You need to define this constant inside of the src directory in order to make it accessible by the rest of the app:
// src/config.ts
export constant BACKEND_BASE_URL = '<%= BACKEND_BASE_URL %>';

This way the value of the constant BACKEND_BASE_URL, which is defined in project.config.ts will be interpolated within the rest of the code inside src/config.ts.

Thank you, Sir!


Small bug in hist graph

the chart should not display "undefined" for x-axis in histogram chart

I think this bug is already fixed - not seeing it any more.


Changed \u and \U to reflect number of hex digits to follow

Some unicode icon strings were set with \uHHHHH which showed the wrong
characters on some implementations of zsh (linux and macOS).  Changed
\uHHHHH to \UHHHHH and conversely changed \UHHHH to \uHHHH.

Thanks for the fix, @bsidhipong! We really appreciate your contribution!


fix(config): correct valueSize for parameter 40, Qubino ZMNHTD

The new value size is noted in the device manual and has been tested on
my home network.

While you're at it with the manual, the unit for the next parameter seems wrong. Could you check that too?

Done, PTAL.

Thanks!


Solve spurious network errors in CI

Run cargo build --verbose
Updating crates.io index
warning: spurious network error (2 tries remaining): SecureTransport error: connection closed via error; class=Net (12)
warning: spurious network error (1 tries remaining): SecureTransport error: connection closed via error; class=Net (12)
error: failed to get version-sync as a dependency of package strftime-ruby v0.1.0 (/Users/runner/work/strftime-ruby/strftime-ruby)
Caused by:
failed to load source for dependency version-sync
Caused by:
Unable to update registry crates-io
Caused by:
failed to fetch https://github.com/rust-lang/crates.io-index
Caused by:
network failure seems to have happened
if a proxy or similar is necessary net.git-fetch-with-cli may help here
https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli
Caused by:
SecureTransport error: connection closed via error; class=Net (12)
Error: Process completed with exit code 101.

I have tested this in tz-rs and it seems to work.


Add utils for multimedia data

Description
Add utils for multimedia data.
Changes:

Provide some common functions in multimedia_utils.
Support image inputs in jinja template.

All Promptflow Contribution checklist:

[ ] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Please add some unit tests in the next PR.


Fix arp requests in Liqo-Gateway

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #(issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Hi @alacuku. Thanks for your PR!
I am @adamjensenbot.
You can interact with me issuing a slash command in the first line of a comment.
Currently, I understand the following commands:

/rebase:            Rebase this PR onto the master branch (You can add the option test=true to launch the tests
when the rebase operation is completed)
/merge:             Merge this PR into the master branch
/build              Build Liqo components
/test               Launch the E2E and Unit tests
/doc-build          Trigger the build of the Liqo documentation

Make sure this PR appears in the liqo changelog, adding one of the following labels:

kind/breaking:      :boom: Breaking Change
kind/feature:       :rocket: New Feature
kind/bug:           :bug: Bug Fix
kind/cleanup:       :broom: Code Refactoring
kind/docs:          :memo: Documentation

/build

/build

/test

/test

The generated artifacts appear to be out-of-date.
Please, ensure you are using the correct version of the generators (eg. controller-gen) and re-run:
make generate

Here it is an excerpt of the diff:
diff --git a/pkg/liqonet/routing/common.go b/pkg/liqonet/routing/common.go
index 4926c08..00853f2 100644
--- a/pkg/liqonet/routing/common.go
+++ b/pkg/liqonet/routing/common.go
@@ -16,13 +16,14 @@ package routing

import (
  "errors"
-	"github.com/vishvananda/netlink"
-	"golang.org/x/sys/unix"
-	"k8s.io/klog/v2"
  "net"
  "os"
  "reflect"

+	"github.com/vishvananda/netlink"
+	"golang.org/x/sys/unix"
+	"k8s.io/klog/v2"
+
  "github.com/liqotech/liqo/apis/net/v1alpha1"
  liqoneterrors "github.com/liqotech/liqo/pkg/liqonet/errors"
  "github.com/liqotech/liqo/pkg/liqonet/utils"


🐛 Bug Report: Close Icon Doesn't Work For Function Execution Modal

👟 Reproduction steps

Open a Function
Click on Execute Now
Click on (X) in the top left of the Modal
Does not dismiss the Modal, even Escape doesn't work (but highlights the X though) which works for almost all other modals across Console.

👍 Expected behavior
Dismiss the Modal.
👎 Actual Behavior
The Modal is not dismissed.
🎲 Appwrite version
Appwrite Cloud
💻 Operating system
Linux
🧱 Your Environment
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Duplicate of #854. Closing.


Library output is not tree-shakeable

I have just tested tree shaking and it seems to be not working.
Reproduction:

init fresh library

npx vue-sfc-rollup sfc-init

create duplicate of sample component.. called example
build

npm run build

In Vue app project

import { WsUikitSample } from 'your-lib';
...
components: {
    WsUikitSample,
  },
...

build app for production
WsUikitExample is present in bundle :/

I thought it will be fixed by removing all from entry.ts but following line
export * from '@/lib-components/index';

it did not help
I am still getting Example component implementation into app bundle, even the app does not use it. :/
I did upload example library https://github.com/luckylooke/vue-sfc-rollup-tree-shake
I did not upload app as i ti standard Vue app generated by Vue CLI
So I did not publish full reproduction... I can do it later, but I need to know first whether there wont be answer like: "Tree shake is not supported", so I spare some energy to reproduce it online..
thanks for understanding
And also thanks for sharing this repo.. so far the best I have found but still missing this tree shake feature.
Happy coding 🙂 👍

Seems this is something which I can partially address, but not completely. There is an auto-install that happens as part of the output for use via script tag, and this should not be in the ESM output, so I can adjust the build to omit this from the esm build.
But in addition, there seems to be some output from rollup-plugin-vue which is not flagging things properly for webpack. See this issue for an explanation.
I will look into making the esm build changes that I can, but this might not be resolved until rollup-plugin-vue gets updated as well.

Thank you 🙏 for your further investigation, nice results, it would be awsome to have tree-shakeable vue component library ona day. For now I put library as git submodule into the projects so they directly access library source *.vue files and it works. But having it as npm module one day will make me feel to have it more standard.

If you notice, the individual .vue files are actually published to npm as part of the library, so you don't have to do the subrepo idea. Your import paths would be like 'import foo from "npmmodule/src/lib-components/foo.vue";' The decision to include the raw .vue files was made because there are times when it's beneficial to have them. In those cases, npm is just the delivery mechanism, and the transpile stuff is ignored.

This issue is not complete resolved - still pending update in rollup-plugin-vue. If that doesn't seem likely, it's possible to fix here with a custom rollup plugin, but very hacky IMHO.

Just tested with latest release of rollup-plugin-vue (5.1.7) and it seems to work great now!


add useCapture props

by default useCapture is true but it will change event bubble behavior. I think it should add an option to set useCapture.

close this I will add source for pr only