🚀 - Vue Support

Which @codeimage/* package(s) are relevant/releated to the feature request?
Editor Language
Description
Would love it if you could get Vue syntax support. You have the icon for it, but the Language Editor defaults to "Typescript", which isn't always the syntax case for Vue.

This may take a while since CodeMirror 6 doesn't have vue language as is core, but there is already an integration for cm5
I need to do some investigation in order to understand what's needed to mix languages and add some vue rules to the current parser 😄

Hey @kupoback, i'm not a vue user ..is this highlight ok? I tried to add both tsx and html language while using .vue extension

Hey @kupoback, i'm not a vue user ..is this highlight ok? I tried to add both tsx and html language while using .vue extension

Yeah, this looks great! I was going to suggest taking a look at the Examples documentation on Vue to see what could be combined, but using HTML/CSS and TypeScript should be just fine!
https://vuejs.org/examples/#fetching-data

Perfect, it seems the same I think 😄

I've opened a pr #429 with that demo https://codeimage-app-pr-429.vercel.app/
This will be deployed tomorrow I think

Looks good to me! Great work!

@kupoback Deployed now with 1.0.9 release


⚠ SceneTime has degraded performance

In 9548666, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 6412 ms

Resolved: SceneTime performance has improved in 7e7ebf6 after 5 minutes.


Add type specs for address functions

I recently ran into an issue where I was trying to figure out why a comma separated string wasn't working  for the to and I needed to see what type was supported when calling the to function. I think by adding these it makes it easier for users to see what options are passed but also it means that users could perform static analysis to make sure they're calling it correctly.

This is a great idea! Just left a couple comments :)

@paulcsmith updated!

LGTM! This will make it a lot easier for people to figure out what can be done with recipients. Thank you very much!


Summary
I would like to add Coston chain (testnet for Songbird)
Chain ID
16
Chain detail URL
https://chainlist.org/chain/16
RPC URL
https://coston-api.flare.network/ext/bc/C/rpc
Blockscout Client URL
https://coston-explorer.flare.network/graphiql
Etherscan Client URL
No response
Etherscan Client API URL
No response
Version
1.4.1
Address (Master copy)
0x41675C099F32341bf84BFc5382aF534df5C7461a
Deployment Tx hash (Master copy)
0x21e6ab4075ea5f47bb0c7db3c74c14f272ec5f6c527476588092c5d9d1d7d3f0
Block explorer URL (Master copy)
https://coston-explorer.flare.network/address/0x41675C099F32341bf84BFc5382aF534df5C7461a
Address (Master copy L2)
No response
Deployment Tx hash (Master copy L2)
No response
Block explorer URL (Master copy L2)
No response
Address (Proxy factory)
0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67
Deployment Tx hash (Proxy factory)
0x3268cca44b84d882b547557e01ed8aa7d2af1904b67a7653d34b32afe50e7a3c
Block explorer URL (Proxy factory)
https://coston-explorer.flare.network/address/0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67

Hi @jO-Osko!
I have not been able to find version 1.4.1 on https://github.com/safe-global/safe-deployments/tree/main/src/assets/v1.4.1. Please, could you add it first?

It has now been added: https://github.com/safe-global/safe-deployments/pull/732

/execute


New command: graph subscription add

graph subscription add [options]
Subscribes a listener application to receive notifications when the requested type of changes occur to the specified resource in Microsoft Graph.

Options

-r, --resource <resource>
The resource that will be monitored for changes

-c, --changeType <changeType>
The type of change(s) in the monitored resource that will raise a notification. Allowed values are created updated deleted

-u, --notificationUrl <notificationUrl>
The URL that will receive the notifications

-e, --expirationDateTime [datetime]
When the webhook subscription expires. Will be set to maximum value for resource if not provided

-s, --clientState [clientState]
A client state value sent in each notification

More information: https://docs.microsoft.com/en-us/graph/api/subscription-post-subscriptions?view=graph-rest-1.0&tabs=http

Nice suggestion. How would you spec it @garrytrinder?

Absolutely, I just threw it in to capture a quick thought I had this morning 😊
I’m struggling to depict —changeType as it has a defined list of options to choose from [‘created’,’updated’,’deleted’] which can be one or all.
I’d see this as a list of options that the user can tab through as it is a defined list and would look like this on the command line
graph subscription add —changeType created updated deleted —resource ...
Are there any commands that use this approach already in the CLI?

@waldekmastykarz updated the issue with an initial spec, that look ok?

Thanks @waldekmastykarz 👍🏻 spec has been updated

Looks good to me! You want to take it? 😄

Absolutely 😁

Awesome! Appreciate your help! 👍

Just an update on this, no progress has been made due to me working on the personal site command which required investigation into a bug in the SharePoint API.
I’ll be planning to work on this in the coming week however

Perfect. Thanks for the update 👍

Opening this up for someone else to pick up

Hi @garrytrinder , can I take this one ?

All yours @ypcode thanks for your help 👍🏻

Hi @garrytrinder , I am going to begin this one.
One note regarding the specs for the changeType that allow multiple combined values, I would suggest the --changeType argument's value to be a comma-separated list of values as a single string (e.g. --changeType "Created,Updated".
I'll start it in that direction, let me know if you have any remarks !

Good catch @ypcode 👍

Good catch @ypcode
My initial thought on this command is that --changeType would accept multi predefined values as it would be the easier approach for the end user, however from looking at the Vorpal docs and doing some extra research this doesn't appear to be implemented, so would require some customisations to Vorpal.
Happy with your approach, thanks for the PR 👍🏻


 Last updated by [1f3ad8d...71616df][cc-compare]
[cc-base-branch]: https://codecov.io/gh/javaslang/javaslang/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/javaslang/javaslang/compare/1f3ad8d5dfefa29bf1b991f490a84d2199af509f...71616df8dbfef2356239159f8de52cce24f2f732
[cc-pull]: https://codecov.io/gh/javaslang/javaslang/pull/1364?src=pr

Please also update the performance table in javaslang/collection/package-info.java if necessary.

Everything was constᵉᶠᶠ before in the table, even though I think they were logarithmic in reality (i.e. Map has log for remove)

@danieldietrich, @ruslansennov, @eduardmanas, @zsolt-donca, please review and try to find counter-examples that should still be tested :)

I'm trying to run this test
    @Test
    public void tt() {
        Vector<Integer> vect = Stream.range(4, 1000)
                .foldLeft(Vector.of(1, 2, 3), (v, i) -> v.append(i).tail());
        System.out.println(vect); // expected: Vector(997, 998, 999)
    }

But your implementation fails (current master works fine).

Well, my goal is to check size of HAMT in resulting Vector. Current implementation shows size = 3. I guess your implementation will show size = 1000.
Let's talk about this but not about bug that I shown above. OK, my test is a little bit synthetic and Vector can be replaced by Queue for example, but you can see the problem with memory consumption.

Thank you @ruslansennov, very useful comments, I force pushed the fixed to the bug you pointed out :) (it was an index mismatch in append).
Abut storing the underlying map, people can simply normalize it via ofAll(iterator()) any time, I consider it a separate memory optimization issue.

About storing the underlying map, people can simply normalize it via ofAll(iterator()) any time, I consider it a separate memory optimization issue.

I'm not sure that we can force the user to do the normalization. This non-obvious requirement, which looks like a limitation. My snippet code with range to Long.MAX_VALUE can runs along years and it is expected behavior, but your fails with out of memory.
And another question. The complexity of all operations depends from size of HAMT, but not from size of Vector. Let's imagine code something like this:
Vector<Integer> vect = Vector.range(0, 1_000_000).take(10)

Size of Vector is 10, but size of delegated HAMT is one million. So, all operations with vector (set, get, etc.) will be strange slow.
Sorry, my opinion is that this PR should be rejected.

Sorry, my opinion is that this PR should be rejected.

Yes, I also think that this is a major issue.

@danieldietrich, @ruslansennov, this is the same as Array, i.e. the delegate might be bigger than it appears.
If this seems like a valid problem, it could be resolved similarly to how ArrayList solves it (which has usually more allocated space than necessary also), i.e. if the masked out region is bigger than the visible one, the with method won't change the offsets, but copy the actual content instead.
If I'm mistaken correctly, this way the complexity of e.g. consuming n elements with drop(1) would be around O(2n), while before it was O(n²).
Opinions?

this is the same as Array, i.e. the delegate might be bigger than it appears.

My failt, I have not seen this
Should be reverted ASAP

:-1:

this is just my opinion :)

Could you please react to my response to your concern also?

Could you please react to my response to your concern also?

IMO previous implementation of Array was simple and fast enough for its main goals: get element and iterate along elements. All others operation should used very carefully. If someone try to get tail of Array -  OK, but it must know what he doing and must understand all about bad performance.

but it must know what he doing

I think this logic is applicable to my version also!

This PR is open for a while now. I share Ruslan's opinion that we should take memory consumption seriously.
Memory sharing makes only sense if there are actual users of different parts of a collection that form a whole. But if we keep a whole HAMT but only a little part is used at all then this could be considered as memory leak. However, we still need one reference to prevent the garbage from being collected.
We should not try to provide hacks/optimizations to an existing algorithm. Over time the algorithm will be hard to maintain because its simplicity got lost. Instead we should consider to change the algorithm - by taking a look at how other (like Scala) solved the problem. @paplorinc provided a link to another Vector impl.
We should revert these change/close the PR. We should rewrite the whole Vector implementation without remorse if, and only if, we get an overall better performance. It does not make sense to rewrite it if we only get partial better results (for some methods) and others get worse.
The same applies to the recent Array changes.

Note: In this PR not only Vector was changed but also Array. The Array changes should have been done in another PR.
Note 2: During my office hours I'm not able to answer question (even not at gitter.im). I will come back once a day / in the evening (CET).

@danieldietrich, I think I addressed @ruslansennov's concerns about memory consumption and I think these changes could be applied even if the Vector impl is changed!
Just take a look at how much faster the tail and slice operations are compared to anything else (e.g. 10x faster than Scala), while consuming the same amount of amortized memory (i.e. O(2n) worst case while providing O(1) operations for many operations).
I don't think the implementation got a lot more complicated, since I reused lots of parts internally.

I see the effort and the improvements that you've done with this PR and I really appreciate it. That's the reason I will pull it in. Please resolve the conflicts (see above) - then I will merge it (in the evening hours (CET)).
We always have to move forward, not backward. If there are other caveats with the current impl we have to solve them, right? So keep on the great work (@all).

The memory consumption of my implementation has the same amortized complexity (!), while being a lot faster for some methods (e.g. 1500x faster than Scala for 1000 elements). These optimizations should be done even if the HAMT is changed: it's unrelated.
Not sure why @ruslansennov didn't explain the "whole picture" here, so that we can talk about it, but it's your library, you decide.
Feel free to use the "unrelated" parts from this commit, I'm not planning on doing it.
Keep up the good work with Javaslang!

But isn't Ruslan's example (currently) valid?
// backing HAMT stays big
Vector<Integer> vect = Vector.range(0, 1_000_000).take(10)

Apparently result.toVector() on line 55 in Vector.java isn't trimming correctly.
If we change it to result.map(Function.identity()) instead, the above expression will have exactly 10 backing elements:
@Test
public void test() {
    assertThat(Vector.range(0, 1_000_000).take(10).delegate.size()).isLessThanOrEqualTo(2 * 10);
}

What about Vector.range(0, 1_000_000).take(500_001)?

It would still consume O(n) memory, just as
List.range(0, 1_000_000).take(550_000).toJavaList()

will have 800_000 elements internally, instead of 500_000, it's the same concept, just a different multiplier (1.5 instead of 2).

List.range(0, 1_000_000).take(550_000).toJavaList()
will have 800_000 elements internally, instead of 500_000

Good catch, thank you.
We should use something like new ArrayList(size()) instead of new ArrayList()


The cap on websocket-client is preventing consumption of security fixes

Please see https://github.com/websocket-client/websocket-client/compare/v0.40.0...master for what you're missing.
Can it be removed?  master is currently at websocket-client>=0.32.0,<=0.40.0 # LGPLv2+

@prometheanfire we need to see if the problem mentioned here is in newer versions:
https://github.com/kubernetes-incubator/client-python/issues/262

Hi ,
I tried to reproduce the issue is #262 without success.
that the pip freeze output :
cachetools==2.0.1 certifi==2017.11.5 chardet==3.0.4 google-auth==1.2.1 idna==2.6 ipaddress==1.0.18 kubernetes===4.0.0-snapshot oauthlib==2.0.6 pyasn1==0.4.2 pyasn1-modules==0.2.1 python-dateutil==2.6.1 PyYAML==3.12 requests==2.18.4 requests-oauthlib==0.8.0 rsa==3.4.2 six==1.11.0 urllib3==1.22 websocket-client==0.44.0
I run the same code as show in #262 and didn't got any certifcate exception ..

@leeyal  excellent, can you submit a PR that excludes the bad version (0.4.0?) and leaves the cap open?

the following versions need to be tested as well to see if it's still broken (these are the versions above 0.40.0 that are published to pypi and are not 0.44.0 (confirmed good). 0.42.0 0.42.1 0.43.0

well i tested it with all version from 0.40.0 between 0.44.0 and it not reproduced ,
two option :

I don't reproduced it in the current way.
the issue is in 0.43.0 that not appears in pip # pip install websocket-client==0.45.0 Collecting websocket-client==0.45.0 Could not find a version that satisfies the requirement websocket-client==0.45.0 (from versions: 0.3, 0.4, 0.4.1, 0.5.0a0, 0.5.0, 0.5.1, 0.6.0, 0.7.0, 0.8.0, 0.9.0, 0.10.0, 0.11.0, 0.12.0, 0.13.0, 0.14.0, 0.14.1, 0.15.0, 0.16.0a0, 0.16.0, 0.17.0, 0.18.0, 0.19.0, 0.20.0, 0.21.0, 0.22.0, 0.23.0, 0.24.0, 0.25.0, 0.26.0, 0.27.0, 0.28.0, 0.29.0, 0.30.0, 0.31.0, 0.32.0, 0.33.0, 0.34.0, 0.35.0, 0.36.0, 0.37.0, 0.39.0, **0.40.0, 0.42.0, 0.42.1, 0.43.0, 0.44.0)**


Fix typos

Docs for da7e46e41a167b978a184407959fd17b41a4cf7d are browesable at: https://ci.foundries.io/projects/fio-docs/builds/773/docs/artifacts/html/index.html


Give stores the ability to check for owner only

About the PR

Some stores may want to be only opened by the original account owner. Wizard's spellbook is a good example since you wouldn't want them sharing spells with apprentices or the crew.
Why / Balance

See about the PR.
Technical details

A bool called OriginalOwner was added to the StoreComponent.
The store system checks this before the UI is opened. If there's no account owner then it will assign the first person to open this store.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

goodbye uplink code hello accounts :trollface:


Handle duplicate image messaages

In 20220907 meeting we discussed several cases when users may submit images that is highly similar to existing images in the database.
We will use this ticket to track the issue and see if we should mitigate this issue further (such as using image hash as article hash; don't change article ID if we change hashing algorithm)

Another log: https://g0v.hackmd.io/nNjUnOgRTqOa2dDcqF67KA?view#重複圖-log
The only solution we can think of is to memorize that the user has already clicked "Yes" before.
Given the fact that the submit button action sends session ID, we probably can just store if the user has clicked submit in the current session.


multiple istio-ingressgateway

Hi,
How we implement multiple istio ingress (i want to implement multiple AWS certificate each cert have different domain name And I want separate load balancer for each domain).
example: I want to deploy 2 application(abc, xyz) with diff domain in same cluster, Istio ingress Create one load balancer in AWS. both application exposed service taking to that load balancer (Under Route 53, External DNS service  point record set with istio-ingress load balancer)
But I want two ISTIO ingress (each application has his own istio load balancer) liek abc.com point to one istio load balancer and xyz.com point to other load balancer  .
How we use same in ISTIO? under istio-1.0.2/install/kubernetes/istio-demo-auth.yaml. which contexts I to use again with separate name?
like in nginx ingress controller we create one more ngnix controller. with separate name. its working perfect.

I am not install istio via helm, Could you please let me know how we add multiple istio via installing with mutua auth

You can add multiple ingress gateway instances by adding entries in the "gateways" property of the helm values.
You will have to specify unique host ports but this will generate a loadbalancer for each.
See values-istio-gateways.yaml as a reference.
To reference it in the gateway you will need to make a selector that matches the ingress-gateway you need.

I am not install istio via helm, Could you please let me know how we add multiple istio via installing with mutua auth

@yciabaud https://github.com/istio/istio/issues/14625 can you please check this have similar issue, am i doing ryt?
newbee


Build fails when trying to build .vhd

Issue
from the documentation:
d2vm build [context directory] [flags]
Flags:
-o, --output string              The output image, the extension determine the image format, raw will be used if none. Supported formats: qcow2 qed raw vdi vhd vmdk (default "disk0.qcow2")
but when trying to build a .vhd the build fails with
d2vm build . --verbose --password Test12345678 --output agent-image.vhd
...
qemu-img convert /tmp/d2vm/9a280bb5-8871-46bd-8c92-1f817e9c5334/disk0.d2vm.raw -O vhd /tmp/d2vm/9a280bb5-8871-46bd-8c92-1f817e9c5334/disk0.vhd
qemu-img: Unknown file format 'vhd'

Cause
The underlying qemu-img convert argument is vpc not vhd
see https://docs.openstack.org/image-guide/convert-images.html
Side note
it would be nice to be able to use additional options for qemu-img convert etc:
qemu-img convert agent-image.qcow2 -O vpc agent-image.vhd -o subformat=fixed,force_size

Hi @Nojinverse, thanks for reporting this problem.
I may have made a typo when selecting formats for qemu-img (or missed an update), as the list of supported formats, as indicated by qemu-img --help, is as follows:

Supported formats: blkdebug blklogwrites blkreplay blkverify bochs cloop copy-on-read dmg file ftp ftps host_cdrom host_device http https iscsi iser luks nbd null-aio null-co nvme parallels qcow qcow2 qed quorum raw rbd replication sheepdog ssh throttle vdi vhdx vmdk vpc vvfat

There is no vhd but vhdx. Using vhdx as an extension in the d2vm ... command will not work because validation will reject it.
Would updating the format list to change vhd to vhdx and adding vpc solve your problem?
Image format conversion is only supported by convenience for the most common formats. For more advanced needs I think it's better to work in raw format and use qemu-img directly yourself.
Implementing the validation of these options would increase the maintenance work required for a relatively uncommon use.

@Adphi I am running into the same issue as @Nojinverse and since I also need VmWare vmdk output besides Hyper-V, I will go with raw output from d2vm (very very great tool by the way!) and calling qemu-img manually 2 times (1 for Hyper-V vhdx and 1 for VmWare vmdk).

Hey gang. I had this issue as well and just created a PR that corrects it. https://github.com/linka-cloud/d2vm/pull/49


pass request to form

Laravel Version: 5.8.2
PHP Version: PHP/7.3.1
Laravel-admin: #.#.#

Description:
how pass  [Request $request] to
 protected function form() {
 }

for retrive a GET value in form.
Steps To Reproduce:

You can get the request instance by using request() function in form.


🐛 [cloud_functions_web] _webFunctions is missing region

Bug report
In packages/cloud_functions/cloud_functions_web/lib/cloud_functions_web.dart at<EMAIL_ADDRESS>I found _webFunctions is missing region.
In this changes, functions_interop.getFunctionsInstance in deleted lines 18, 19 is called with region, but without region in added lines 31, 32.

And this causes a bug that functions always calls the "us-central1" server, like https://github.com/FirebaseExtended/flutterfire/issues/4226.
Steps to reproduce
call a function and set the region, like this
HttpsCallable callable = FirebaseFunctions.instanceFor(region: 'europe-west1').httpsCallable('function-name');
Expected behavior
functions_interop.getFunctionsInstance must be called with region.
Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.3, on macOS 11.6 20G165 darwin-arm, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.62.0)
[✓] Connected device (1 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.14.4
Flutter SDK 2.5.3
aruga_one_manager_web 1.4.11+1

dependencies:
- after_layout 1.1.0 [flutter]
- awesome_dialog 2.1.1 [flutter simple_animations flare_flutter]
- cached_network_image 3.1.0+1 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- cloud_firestore 3.1.0 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- cloud_functions 3.1.0 [cloud_functions_platform_interface cloud_functions_web firebase_core firebase_core_platform_interface flutter]
- csv 5.0.0
- dio 4.0.1 [http_parser path]
- firebase 9.0.2 [http http_parser js]
- firebase_analytics 8.3.4 [firebase_analytics_platform_interface firebase_analytics_web firebase_core flutter meta]
- firebase_auth 3.2.0 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- fl_chart 0.36.4 [flutter equatable pedantic]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_linkify 5.0.2 [flutter linkify]
- flutter_localizations 0.0.0 [flutter intl characters clock collection meta path typed_data vector_math]
- flutter_rating_bar 4.0.0 [flutter]
- flutter_state_notifier 0.7.0 [flutter meta provider state_notifier]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- font_awesome_flutter 9.2.0 [flutter]
- freezed_annotation 0.14.3 [collection json_annotation meta]
- google_fonts 2.1.0 [flutter http path_provider crypto]
- googleapis 3.0.0 [_discoveryapis_commons http]
- googleapis_auth 1.2.0 [crypto http http_parser]
- horizontal_data_table 3.4.2 [flutter provider]
- intl 0.17.0 [clock path]
- json_serializable 4.1.4 [analyzer build build_config collection json_annotation meta path source_gen]
- path 1.8.0
- photo_view 0.12.0 [flutter]
- pretty_dio_logger 1.2.0-beta-1 [dio]
- provider 5.0.0 [collection flutter nested]
- recase 4.0.0
- responsive_builder 0.4.1 [flutter]
- shared_preferences 2.0.8 [flutter meta shared_preferences_linux shared_preferences_macos shared_preferences_platform_interface shared_preferences_web shared_preferences_windows]
- state_notifier 0.7.0 [meta]

dev dependencies:
- build_runner 2.1.4 [args async analyzer build build_config build_daemon build_resolvers build_runner_core code_builder collection crypto dart_style frontend_server_client glob graphs http_multi_server io js logging meta mime package_config path pool pub_semver pubspec_parse shelf shelf_web_socket stack_trace stream_transform timing watcher web_socket_channel yaml]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]
- freezed 0.14.2 [analyzer build build_config collection meta source_gen freezed_annotation]
- mockito 5.0.15 [analyzer build code_builder collection dart_style matcher meta path source_gen test_api]

transitive dependencies:
- _discoveryapis_commons 1.0.2 [http http_parser meta]
- _fe_analyzer_shared 22.0.0 [meta]
- analyzer 1.7.2 [_fe_analyzer_shared cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml pedantic]
- args 2.3.0
- async 2.8.1 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 2.1.1 [analyzer async convert crypto glob logging meta path]
- build_config 1.0.0 [checked_yaml json_annotation path pubspec_parse yaml]
- build_daemon 3.0.1 [built_collection built_value http_multi_server logging path pool shelf shelf_web_socket stream_transform watcher web_socket_channel]
- build_resolvers 2.0.4 [analyzer async build crypto graphs logging path package_config pool pub_semver stream_transform yaml]
- build_runner_core 7.2.2 [async build build_config build_resolvers collection convert crypto glob graphs json_annotation logging meta path package_config pool timing watcher yaml]
- built_collection 5.1.1
- built_value 8.1.3 [built_collection collection fixnum meta]
- cached_network_image_platform_interface 1.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.1 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.1.0
- charcode 1.3.1
- checked_yaml 2.0.1 [json_annotation source_span yaml]
- cli_util 0.3.5 [meta path]
- clock 1.1.0
- cloud_firestore_platform_interface 5.4.5 [collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 2.5.0 [cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- cloud_functions_platform_interface 5.0.14 [firebase_core flutter meta plugin_platform_interface]
- cloud_functions_web 4.1.0 [cloud_functions_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- code_builder 4.1.0 [built_collection built_value collection matcher meta]
- collection 1.15.0
- convert 3.0.1 [typed_data]
- crypto 3.0.1 [collection typed_data]
- dart_style 2.1.1 [analyzer args path pub_semver source_span]
- equatable 2.0.3 [collection meta]
- fake_async 1.2.0 [clock collection]
- ffi 1.1.2
- file 6.1.2 [meta path]
- firebase_analytics_platform_interface 2.0.1 [flutter meta]
- firebase_analytics_web 0.3.0+1 [firebase firebase_analytics_platform_interface flutter flutter_web_plugins meta]
- firebase_auth_platform_interface 6.1.4 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 3.2.0 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core 1.10.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_core_platform_interface 4.1.0 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.2.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- fixnum 1.0.0
- flare_flutter 3.0.2 [collection flutter meta]
- flutter_blurhash 0.6.0 [flutter meta pedantic]
- flutter_cache_manager 3.1.3 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_lints 1.0.4 [lints]
- frontend_server_client 2.1.2 [async path]
- glob 2.0.2 [async collection file path string_scanner]
- graphs 2.1.0 [collection]
- http 0.13.4 [async http_parser meta path]
- http_multi_server 3.0.1 [async]
- http_parser 4.0.0 [charcode collection source_span string_scanner typed_data]
- io 1.0.3 [meta path string_scanner]
- js 0.6.3
- json_annotation 4.1.0 [meta]
- linkify 4.1.0
- lints 1.0.1
- logging 1.0.2
- matcher 0.12.10 [stack_trace]
- meta 1.7.0
- mime 1.0.1
- nested 1.0.0 [flutter]
- octo_image 1.0.0+1 [flutter flutter_blurhash]
- package_config 2.0.2 [path]
- path_provider 2.0.6 [flutter path_provider_linux path_provider_macos path_provider_platform_interface path_provider_windows]
- path_provider_linux 2.1.0 [flutter path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.2 [flutter]
- path_provider_platform_interface 2.0.1 [flutter meta platform plugin_platform_interface]
- path_provider_windows 2.0.3 [ffi flutter meta path path_provider_platform_interface win32]
- pedantic 1.11.1
- platform 3.0.2
- plugin_platform_interface 2.0.2 [meta]
- pool 1.5.0 [async stack_trace]
- process 4.2.4 [file path platform]
- pub_semver 2.1.0 [collection meta]
- pubspec_parse 1.1.0 [checked_yaml collection json_annotation pub_semver yaml]
- rxdart 0.27.2
- shared_preferences_linux 2.0.2 [file meta flutter path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 2.0.2 [flutter shared_preferences_platform_interface]
- shared_preferences_platform_interface 2.0.0 [flutter]
- shared_preferences_web 2.0.2 [flutter flutter_web_plugins meta shared_preferences_platform_interface]
- shared_preferences_windows 2.0.2 [flutter file meta path path_provider_platform_interface path_provider_windows shared_preferences_platform_interface]
- shelf 1.2.0 [async collection http_parser path stack_trace stream_channel]
- shelf_web_socket 1.0.1 [shelf stream_channel web_socket_channel]
- simple_animations 3.2.0 [flutter supercharged flutter_lints]
- sky_engine 0.0.99
- source_gen 1.0.3 [analyzer async build dart_style glob meta path pedantic source_span yaml]
- source_span 1.8.1 [collection path term_glyph]
- sqflite 2.0.0+4 [flutter sqflite_common path]
- sqflite_common 2.0.1+1 [synchronized path meta]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- stream_transform 2.0.0
- string_scanner 1.1.0 [charcode source_span]
- supercharged 2.1.1 [supercharged_dart flutter]
- supercharged_dart 2.1.1
- synchronized 3.0.0
- term_glyph 1.2.0
- test_api 0.4.2 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- timing 1.0.0 [json_annotation]
- typed_data 1.3.0 [collection]
- uuid 3.0.5 [crypto]
- vector_math 2.1.0
- watcher 1.0.1 [async path]
- web_socket_channel 2.1.0 [async crypto stream_channel]
- win32 2.2.10 [ffi]
- xdg_directories 0.2.0 [meta path process]
- yaml 3.1.0 [collection source_span string_scanner]

@nukotsuka
Do you get any error log after running HttpsCallable callable = FirebaseFunctions.instanceFor(region: 'europe-west1').httpsCallable('function-name'); ?
If I use HttpsCallable callable = FirebaseFunctions.instanceFor(region: 'europe-west1').httpsCallable(                   'listFruit', in plugin's official example app, I get internall error, but if I use us-central1, I get the data back as expected. Is that the same case at your end ?

@darshankawar
Yes, the same case is at my end.
And I got the following error in Chrome DevTools console.
❌ Access to fetch at 'https://us-central1-nukotsuka.cloudfunctions.net/listFruit'
from origin 'http://localhost:8000' has been blocked by CORS policy:
Response to preflight request doesn't pass access control check:
Redirect is not allowed for a preflight request.
❌ POST https://us-central1-nukotsuka.cloudfunctions.net/listFruit net::ERR_FAILED

And then, I got an internall error in the console running Flutter.

I'm having the same issue
This is my code
 FirebaseFunctions functionsInstance = FirebaseFunctions.instanceFor(       region: 'europe-west3',     );     HttpsCallable callable = functionsInstance.httpsCallable(       'app/players-new',       options: HttpsCallableOptions(         timeout: const Duration(           seconds: 60,         ),       ),     );     final results = await callable();
The browser tries us-central-1
https://us-central1-...........cloudfunctions.net/app/players-new

Thanks for confirming. This is replicable using latest plugin version and on latest stable and master and using plugin example.

 stable, master flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.5.3 at /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (2 days ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/dhs/Library/Android/sdk
    • Java binary at: /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (4 available)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 93.0.4577.82

• No issues found!

[✓] Flutter (Channel master, 2.6.0-12.0.pre.655, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-GB)
    • Flutter version 2.6.0-12.0.pre.655 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5ce05e6ec5 (3 hours ago), 2021-11-08 23:43:04 -0500
    • Engine revision 162d8508f8
    • Dart version 2.15.0 (build 2.15.0-285.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/dhs/Library/Android/sdk
    • Java binary at: /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[[✓] Connected device (4 available)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 93.0.4577.82

• No issues found!

I think I have the same issue
https://twitter.com/soarcreator/status/1458070623461122054
I hope this issue will be fixed soon


Update K8s operator _index.md

In my actual operation, I found that the namespace attribute needs to be declared, so I modified the document. Please review the modifications.

Didn't it create the resource in the namespace from your active context? What was the error message?

Didn't it create the resource in the namespace from your active context? What was the error message?

Sorry, I read my operation log carefully. I was trying to solve another problem at the time. I thought it was caused by this namespace. Now I have tried again and it is normal that there is no namesapce attribute. Moreover, this resource object namesapce only restricts tenants. I made a mistake, sorry.


Fix Notice: Undefined index: structs

1. Why is this change necessary?
Fix "Notice: Undefined index: structs" error on homepage.
2. What does this change do, exactly?
Add an additional check to see wether needed data is actually available.
3. Describe each step to reproduce the issue or behaviour.
I don't know exactly why this error is thrown, but it happens because certain price rules were invalid.

4. Please link to the relevant issues (if any).
/
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Just reindex ./bin/console dal:refresh:index and the error is gone :)

Just reindex ./bin/console dal:refresh:index and the error is gone :)

Doesn't seem like the error should happen at all though? 😁

You are not using normal installation. This can happen all time using the branch and not using a tag. After updating the updater will trigger reindexing so the error will not occur on real installations.

Okay. If that's the case I guess this can be closed 👍


regex negated character set doesn't work properly with character classes

 Issue by kanaka
Tuesday Oct 14, 2014 at 15:11 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/18035
This issue was labelled with: A-libs in the Rust repository

$ rustc --version
rustc 0.13.0-nightly (1c3ddd297 2014-10-13 23:27:46 +0000)

Here is my test code:
#![feature(phase)]
#[phase(plugin)]
extern crate regex_macros;
extern crate regex;

fn main() {
    let re = regex!(r#"([^\s])"#);
    println!("\n1 {} on '1 2 3'", re);
    for cap in re.captures_iter("1 2 3") {
        println!("0: {}, 1: {}", cap.at(0), cap.at(1));
    }

    let re = regex!(r#"([^\s,])"#);
    println!("\n2 {} on '1 2 3'", re);
    for cap in re.captures_iter("1 2 3") {
        println!("0: {}, 1: {}", cap.at(0), cap.at(1));
    }

    let re = regex!(r#"([^A,])"#);
    println!("\n3 {} on '1A2A3'", re);
    for cap in re.captures_iter("1A2A3") {
        println!("0: {}, 1: {}", cap.at(0), cap.at(1));
    }

    let re = regex!(r#"([^[:alpha:],])"#);
    println!("\n4 {} on '1A2A3'", re);
    for cap in re.captures_iter("1A2A3") {
        println!("0: {}, 1: {}", cap.at(0), cap.at(1));
    }

    let re = regex!(r#"([^[:alpha:]Z])"#);
    println!("\n5 {} on '1A2A3'", re);
    for cap in re.captures_iter("1A2A3") {
        println!("0: {}, 1: {}", cap.at(0), cap.at(1));
    }
}

Results:
1 ([^\s]) on '1 2 3'
0: 1, 1: 1
0: 2, 1: 2
0: 3, 1: 3

2 ([^\s,]) on '1 2 3'
0: 1, 1: 1
0:  , 1:
0: 2, 1: 2
0:  , 1:
0: 3, 1: 3

3 ([^A,]) on '1A2A3'
0: 1, 1: 1
0: 2, 1: 2
0: 3, 1: 3

4 ([^[:alpha:],]) on '1A2A3'
0: 1, 1: 1
0: A, 1: A
0: 2, 1: 2
0: A, 1: A
0: 3, 1: 3

5 ([^[:alpha:]Z]) on '1A2A3'
0: 1, 1: 1
0: A, 1: A
0: 2, 1: 2
0: A, 1: A
0: 3, 1: 3

Cases 2, 4, 5 have the misbehavior. My expectation is that every case should have the same result as 1 and 3.

Any update on this?

This is still broken. Any update?

I haven't had time to look at it yet. Hopefully soon. It's probably a bug in the parser somewhere.

This is a nasty bug. I probably won't fix it tonight. Basically, character class negation is completely broken when the class contains two or more non-overlapping ranges. The current code will match if any range matches, but when the class is negated, this should be switched to no ranges matching. I'm probably going to do this by simply inverting ranges (otherwise I'd need to invent a new VM instruction that knows how to "and" things).

@BurntSushi so, @alexcrichton fixed up mal's implementation to work with rust nightly (https://github.com/kanaka/mal/pull/26) which included fixing up rust-pcre (https://github.com/alexcrichton/rust-pcre) to work with nightly. The priority of fixing this bug in regex is probably not as urgent (at least, not for me).


[#9] [UI] [Android] As an anonymous user, I can see the login screen

Close #9

What happened 👀

Create a general survey text field and button
Create Sign In screen UI
Create SurveyNavigation and SurveyDestination to navigate from the Splash screen to Sign In screen

Insight 📝

N/A

Proof Of Work 📹
Screen_recording_20230818_124629.webm

@nmint8m I have resolved your concerns, unfortunately, we have not had any linter for the trailing commas yet, but let me do some research about it and will implement it later if possible 👍 Nice catch btw


Items is invisible

All Umbrellas is invisible
Forge 43.2.7
minecraft 1.19.2
Dave's Potioneering 0.10 - Forge 1.19.2

fixed


baseUrl not working properly in switchLocalePath()

Environment

Operating System: Windows_NT
Node Version:     v16.16.0
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, css, vite, modules, i18n
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/github/hubret/i18nbug/tree/main/nuxt-app
https://github.com/hubret/i18nbug
Describe the bug
the base url shows up after hitting 404 page. then you need to refresh
Additional context
No response
Logs
No response

Handy Stackblitz link for reproduction: https://stackblitz.com/github/hubret/i18nbug/tree/main/nuxt-app

I added a commit that hardcodes the baseurl. first version had only set the variable in env. oops!

Thank for your feedback!
Unfortunately, i18n module v8 hasn't support nuxt baseUrl  yet.
i18n module v8 is currently ported based on the existing v7.
Apparently nuxt baseUrl is not supported in i18n module v7.
We need to extend the existing implementation.

any suggestion on a workaround? some sort of string manipulation on switchLocalePath to write the baseUrl into it?

I've checked nuxt i18n module v7.
I mentioned previous comment was wrong.
nuxt i18n module v7 and nuxt i18n module v8 support nuxt baseUrl.
As rchl mentioned, you need to use nuxt-link instead of a tag.
I noticed this when I read your repro code.
Thanks!

ah. right. initially, this wasn't working with differentDomains, but I took that out with the thought that I had isolated it to just baseUrl + switchLocalePath. And I left the  tags in.
I'm going to try it with getting the current locale path and the locale domain from i18n (if its possible to access locale domain)


"Get-PartnerCustomerUser" with "-ReturnDeletedUsers:$false" returns 0 for some customers

Copied from docs issue #112
"Get-PartnerCustomerUser" with "-ReturnDeletedUsers:$false" returns 0 for some customers.
I'm sure the customer in question has users that is not deleted, and it works for 7 out of 11 customers.
I'm specifying "-ReturnDeletedUsers:$false" because I like having available parameters visible in scripts no matter if I use them or not. Easier for troubleshooting later on.
PartnerCenter module version: 3.0.7
Reproduce
PS C:\Users\OlavRønnestadBirkela>
# Without -ReturnDeletedUsers:$false
$Users = Get-PartnerCustomerUser -CustomerId $CustomerId;$Users.'Count'
$Users | Group-Object -Property 'State' -NoElement
518

Count Name
----- ----
  518 Active

PS C:\Users\OlavRønnestadBirkela>
# With -ReturnDeletedUsers:$false
$Users = Get-PartnerCustomerUser -CustomerId $CustomerId -ReturnDeletedUsers:$false;$Users.'Count'
$Users | Group-Object -Property 'State' -NoElement
0

PS C:\Users\OlavRønnestadBirkela>

The required code changes to address this issue has been addressed.


fix(MWPW-142248): Accessibility for carousels containing focusable elements.

The Accessibility for milo carousels that contain  focusable elements within them has been configured as per w3 standards.

Tab index is available only to the active slide.
Tab index for carousels are changed when the active slide is changed.

Resolves: MWPW-142248
Test URLs:

Before: https://main--cc--adobecom.hlx.page/products/premiere/explore?martech=off
After: https://mwpw-142248--milo--sharath-kannan.hlx.page/drafts/sharathkannan/Accessiblity/explore?martech=off

Code seems to fix the issue.  Can you add code coverage? Check is showing a block of your new code is not covered.

Code seems to fix the issue. Can you add code coverage? Check is showing a block of your new code is not covered.

Added code coverage.

Validation done on the below urls :
https://mwpw-142248--milo--sharath-kannan.hlx.page/drafts/sharathkannan/Accessiblity/explore?martech=off
https://mwpw-142248--milo--sharath-kannan.hlx.page/docs/library/kitchen-sink/carousel
https://main--cc--adobecom.hlx.page/products/premiere/explore?milolibs=mwpw-142248--milo--sharath-kannan
Now when the user lands on carousel, focus willbe on button inside. When the user tabs and navigated to next slide the control will be on navigations, inorder to get the focus inside user has to use shift+tab
https://screenrec.com/share/R4qPhIjk2a


Strange behavior for multiple matching completions

VSCode Version: Code - Insiders 1.19.0-insider (bd11a1f64eb870a927f3d0e5e4628b3fa066d90c, 2017-12-06T06:14:27.050Z)
OS Version: Linux x64 4.4.0-21-generic
Extensions: none

Steps to Reproduce:

Have a global variable verify, and also @types/node in scope. (This was reproduced by adding a new file to tests/cases/fourslash in the TypeScript repo.)
Start typing verify.

I would expect the expected completion to always be verify because that's always the best match, but it seems to switch to createVerify when the text is verif but change when I type y. When I try reproducing it again it's sometimes the opposite and typing verify completes to createVerify even though there's already an exact match!

This might be due to us now remember a previous choice. In case you have selected createVerify once with the verif prefix we will automatically select it again. Please try with latest, latest insiders from today which has a slightly tweak variant of this (it will stay on the selected item).

I can't reproduce this behavior any more, thanks.


Add stun to brig shield bash

Created by: MaxwellJung

moved from ow1_emulator_backup#730

mentioned in merge request ow1_emulator_backup!50

mentioned in merge request ow1_emulator_backup!49

mentioned in merge request ow1_emulator_backup!45

mentioned in merge request ow1_emulator_backup!46


Improvement

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

test

test

test

test


Link reference definitions with \r\n line endings are not parsed correctly

If you have a document like the following:
[flexmark-java] [flexmark-java issues]

[flexmark-java]: https://github.com/vsch/flexmark-java
[flexmark-java issues]: https://github.com/vsch/flexmark-java/issues

It will correctly parse when it has only \n as line separator:

flexmark-java flexmark-java issues

However, if the line separator is \r\n and you have at least two consecutive link reference definitions then it is not parsed correctly:

[flexmark-java] [flexmark-java issues]

[flexmark-java]: https://github.com/vsch/flexmark-java
[flexmark-java issues]: https://github.com/vsch/flexmark-java/issues

Can be reproduced using the following Java code:
import com.vladsch.flexmark.ast.Node;
import com.vladsch.flexmark.html.HtmlRenderer;
import com.vladsch.flexmark.parser.Parser;

public class FlexmarkTest {

    public static void main(String[] args) {
        Parser parser = Parser.builder().build();
        HtmlRenderer renderer = HtmlRenderer.builder().indentSize(4).build();

        String input = "[flexmark-java] [flexmark-java issues]\n"
                + "\n"
                + "[flexmark-java]: https://github.com/vsch/flexmark-java\n"
                + "[flexmark-java issues]: https://github.com/vsch/flexmark-java/issues\n";

        // With \n it is parsed correctly
        Node node = parser.parse(input);
        System.out.println(renderer.render(node));

        // With \r\n it is not parsed correctly
        node = parser.parse(input.replace("\n", "\r\n"));
        System.out.println(renderer.render(node));
    }

}

@Minecrell, thank you for reporting this. It is was caused by many instances of \n in regular expressions used for parsing instead of \r\n|\r|\n as needed since the original regex was from commonmark-java which normalized EOL before parsing and flexmark-java does it in rendering to keep the AST source based.
This bug is a duplicate of #75 and was fixed in version 0.18.1, current version is 0.18.2 and has a few more bug fixes.
I tried your sample and the output is:

flexmark-java flexmark-java issues

flexmark-java flexmark-java issues

confirming that this is fixed in the latest release.

You're right. I began testing with 0.10.2, then switched to 0.10.0 temporarily and forgot to change it back. I can confirm that it is fixed in 0.10.2. Thanks!

You're welcome. No problem with duplicate reports. I prefer to have duplicate reports rather than none.


Client side support for HelloRetryRequests

Please note that while we are transitioning from travis-ci to AWS CodeBuild, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Resolved issues:
resolves #1607
related to https://github.com/awslabs/s2n/pull/1940
Description of changes:

Adding client side support for HelloRetryRequest
Handles HRR requests from a server and send a valid keyshare if supported

Testing:

Unit tests and self-talk tests
Integration test is pending

Call-Outs:

SAW Tests may need to be updated
Fuzz Testing has not been covered

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

Codecov Report

Merging #1964 into master will increase coverage by 8.71%.
The diff coverage is 87.06%.

@@            Coverage Diff             @@
##           master    #1964      +/-   ##
==========================================
+ Coverage   80.31%   89.03%   +8.71%
==========================================
  Files         228      148      -80
  Lines       17037    11572    -5465
==========================================
- Hits        13684    10303    -3381
+ Misses       3353     1269    -2084


Multiple async describe causes some test to not be run.

Describe the bug
The bug is quite strange, when using multiple nested async describe, some test can be skipped for no apparent reason.
As pointed out in the repro comments, commenting some describes make the previously skipped test to run.
There is no part of the form to do it, so, thank you guys a lot, that is amazing work you are doing here! 🚀
Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-trh7ej?file=test/suite.test.ts
I isolated a minimal repro with only three describe and an async function using a timeout.
System Info
See provided StackBlitz.

Used Package Manager
npm
Validations

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

Is this supported by Jest? I only see one closed merge request in Jest saying that they will not support it.
You can use async beforeAll and beforeEach in describe and don't mark describe as async.

Is this supported by Jest? I only see one closed merge request in Jest saying that they will not support it.
You can use async beforeAll and beforeEach in describe and don't mark describe as async.

Agree, describe should not be async.

@Demivan, it makes sense. In this case, shouldn't the type SuiteFactory here : https://github.com/vitest-dev/vitest/blob/735e5074fe333053b0ba53e8860372054e5b6b99/packages/vitest/src/types/tasks.ts#L81
be changed to not accept asynchronous functions?

To sum up for everyone. @antfu, if I understand #750 correctly, async describe will be supported? 🚀


Fix header phase for static frameworks

Corrects a check that was removing all headers from static frameworks

Not averse to adding a test for this but would need some guidance

Added a changelog entry. Not sure why the change is causing CI to fail

Added a changelog entry. Not sure why the change is causing CI to fail

CI is failing because Xcode 10.3 is no longer supported it seems. #707 will resolve it for the time being.

@wag-miles please rebase on master to get the updated CI

Thanks @wag-miles!


Add support for edge attributes

At the moment, edges consist of a source vertex, a target vertex, and a weight. It would be great to have a label as well:
type Edge[T any] struct {
	Source T
	Target T
	Weight int
	Label string
}

Possible APIs

One approach to provide an API for adding a label to an edge is to implement methods such as EdgeWithLabel and WeightedEdgeWithLabel. This is a very simple approach, but the drawback is that we would end up with 4 new methods (because of the additional ByHash methods), which would make a total of 8 methods to create an edge.

The second approach would be to introduce functional options for both the weight and the label for an edge. Those functional options would be added to the Edge and EdgeByHashes functions. Since they are an optional parameter, adding them would not be a breaking change. Functional options are already being used by the New function.

Functional options
I personally prefer the second approach. The Edge and EdgeByHashes functions would look as follows:
Edge(source, target T, options ...func(Edge[T])) error
EdgeByHashes(sourceHash, targetHash K, options ...func(Edge[T])) error

For adding the weight and labels, there would be two predefined functional options:
func EdgeWeight[T any](weight int) func(Edge[T]) {
    return func(edge Edge[T]) {
        edge.Weight = weight
    }
}

Same for the Label field. Creating an edge from "A" to "B" would look like this:
g.Edge("A", "B", graph.EdgeWeight(5), graph.EdgeLabel("my label"))

As a result, the WeightedEdge and WeightedEdgeByHashes functions could be deprecated.

Change: Instead of supporting a single label, a map of attributes should be supported. This is a map[string]string and allows for better flexibility and extensibility.
These attributes will be rendered by #20.

#20 already introduces the Attributes field.


where is text_txt

Hello, dataloader.py part, where is the file in the /text_txt/ folder in this code text_path = self.args.data_dir + '/texts_txt/' + annotation + '/' + id.replace(".jpg", ".txt").

This is the dataset directory of food101. You have to download it from their website.

I can't find this file, the URL is invalid, can you compress this txt file and send it to me？ thank you very much
------------------ 原始邮件 ------------------
发件人:                                                                                "Vincent-ZHQ/UniS-MMC"                                                                                @.>;
发送时间: 2024年1月8日(星期一) 下午2:27
@.>;
<EMAIL_ADDRESS>主题: Re: [Vincent-ZHQ/UniS-MMC] where is text_txt (Issue #5)
This is the dataset directory of food101. You have to download it from their website.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hello, dataloader.py part, where is the file in the /text_txt/ folder in this code text_path = self.args.data_dir + '/texts_txt/' + annotation + '/' + id.replace(".jpg", ".txt").
Hello, I downloaded Food101 from the Internet and didn't find the text_txt file, and there were the following problems in operation:
FileNotFoundError: [Errno 2] No such file or directory: 'Path/To/Dataset_Home_Directory\Food101/texts_txt/lasagna/lasagna_666.txt'

Do you have a similar problem?
How did you solve it?


Add Left Click Banking plugin

Same functionality as the built in left click menu entry customization, but in the bank interfaces.

This is basically the same as #1883, which we decided we do not want to add.


Micro F1 Or Average F1

Hi, In your paper, you have reported micro F1 for Relation Extraction on chemprot dataset. But In the repo, the test metric is Average F1, I wonder how I can change Average F1 to Micro F1.

Hey @h-peng17, Micro F1 is the same as Accuracy, which should be one of the metrics returned

Thank you. I forget the dataset doesn''t have NA.


Verify local file before restoring

Output of restic version
restic 0.9.2 compiled with go1.10.3 on linux/amd64

What should restic do differently? Which functionality do you think we should add?
Currently, when using the restic restore command, it systematically restore data from a snapshot and eventually check the integrity of the restored data afterward if the --verify arg is used.
It would good to check the data before restoring it as well, and skip the restore part if the data present on the disk is already valid (same principle as rsync).
What are you trying to do?
It would save time when restoring backups not from scratch.
For example, we want to give our users the content of their last backup so they can do incremental backup (with logic on application level).
To do so, we're giving our users a working directory:

we restore the content of the last backup if they ask for it,
they do their backup and write in that directory,
we're backing up this directory using restic.

This directory is persisted but to guarantee that its content is not altered between two backups, we systematically restore the content of the previous backup before presenting it back to the users.
Currently, this means that we systematically entirely restore a snapshot, which can take a bit of time.
Executing the same operation as during the --verify before restoring objects would save quite a lot of time, and shouldn't add much overhead when restoring in an empty directory.

Thanks for reporting this, and for the implementation!

Duplicate of #2662.


Stack removing

Good work. I love that mod !
However I noticed an important bug : when you move a stack of nodes to the letters table formspec, and then you halve it (right-click) for move the half to the inventory, the stack in the letters table is removed and you just keep the half of the stack moved to the inventory.

Okay, thanks. I'll check that out when I get a bit of time!

Okay, thanks. I'll check that out when I get a bit of time!


Update README.md

Small grammar corrections.

Thanks for flagging this, going to close for now since we just did a bunch of README updates


Test PyPI should include the --index-url argument for the pip install command.

What's the problem this feature will solve?
Copy-pasting the pip install command on Test PyPI won't actually install from Test PyPI
Describe the solution you'd like
instead of pip install package_name we should show pip install --index-url https://test.pypi.org/simple/ package_name
Additional context
It would also be super awesome if pip just had an alias for this, e.g. pip install --with-test-pypi package_name.

Good First Issue: This issue is good for first time contributors. If you've already contributed to Warehouse, work on another issue without this label instead. If there is not a corresponding pull request for this issue, it is up for grabs. For directions for getting set up, see our Getting Started Guide. If you are working on this issue and have questions, feel free to ask them here, #pypa-dev on Freenode, or the pypa-dev mailing list.

@nlhkabu Can I try this one?

Great @fitodic

@fitodic GitHub wont let me assign this to you - so I'll assign it to me so no one else takes it :)

Removing the sprint label to reserve :)

Thanks :+1:

@nlhkabu I've just noticed there's already a pull request for this issue: https://github.com/pypa/warehouse/pull/4044
Should I move on?

ah, yes sorry.  I will remove this from the sprint list.  my mistake!

Fixed in #4941.


Wipe VMFS filesystem signatures

Change zeroStartEnd to zero 8 more bytes,

4 bytes at offset 1mib (VMFS)
4 bytes at offset 2mib (VMFS_volume_member)

Fixes #121.

Kind of looking for feedback here. This definitely does do what I was after, wiping the signatures that blkid would find. I think 2mib+4 and 1mib+4 still qualify as part of "start and end" (per the name zeroStartEnd).
I'm not sure what I think about extending that to be also wipe some known signature locations. The intent of the function was initially to make sure that tools looking at signatures did not find what anything, but we were just zeroing the 1MiB.
Thoughts?

Agreed this is still start and end.
Wiping out other known signature locations is IMO a good thing.  If you want to rename the fn to zeroKnownSignatures that would be fine.  But we know that some tools will break if you don't zero those, so it's a good thing to do.


feat(BpnDidService): Transition from API Key to Technical User Authentication for BDRS Integration (#59)

Description
We need to update the authentication mechanism for our portal's integration with the BDRS. The current implementation uses an API Key, but it is needed to switch to a new authentication method using a technical user with client ID and secret.
Why
The endpoints the portal is hitting remain unchanged, however, the error handling got improved to support the new errors provided with the technical user authentication method.
Issue
Ref: #1078
Corresponding Configuration PR
https://github.com/eclipse-tractusx/portal/pull/474
Checklist

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[x] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas

Before approving I'd wait for the PRs to adjust the configuration in the portal repo and add the technical user in the portal-iam repo

@Phil91 configuration PR has been attached
Seed data (related to Service Account of BDRS Management) still needs to be part of this PR

Requesting changes so that the PR doesn't get merged until there's an according change on BDRS side eclipse-tractusx/sig-release#959 (comment)

Just FYI: @evegufy | @Phil91,  I have added Seed Data for a technical user for BDRS Management

@tfjanjua as there was a conflict with latest seeding-data related changes on main I did rebase the branch on main and die a force-push. Please pull those changes before applying further commits.

don't merge until change from BDRS side is clarified.
I was then also under the impression that you prediscussed the change with the BDRS committers, as commented here, but that doesn't seem the case, see https://github.com/eclipse-tractusx/bpn-did-resolution-service/issues/120.


Downie 4 Causes the plugin to not work

Extension
https://www.raycast.com/mackopes/quit-applications
Description
Error: Command failed: plutil -convert json "/Applications/Downie 4.app/Contents/Info.plist" -o - checkExecSyncError:node:child_process:871:11 execSync:node:child_process:943:15 Ss:index.js:5:12411 index.js:21:646
Steps To Reproduce
No response
Current Behaviour
No response
Expected Behaviour
No response

hey! thanks for raising this issue, this seems to be related to https://github.com/raycast/extensions/issues/5356. I will try to push the fix soon.

@macOSWk This issue should be fixed now. Could you please check it out and close the issue if it has been resolved for you? :)

@raycastbot close this issue


BigInt support

Initial
Currently, BigInts (Node 10.7+) are converted to null when storing:
> s=require('sqlite3')
> d=new s.Database(':memory:')
Database { open: false, filename: ':memory:', mode: 65542 }
> d.exec('create table t(s blob)')
Database { open: true, filename: ':memory:', mode: 65542 }

> d.run('insert into t values(?)',[1234n], ()=>d.all('select * from t',console.log))
Database { open: true, filename: ':memory:', mode: 65542 }
> null [ { s: null } ]

> d.run('insert into t values(?)',[1234], ()=>d.all('select * from t',console.log))
Database { open: true, filename: ':memory:', mode: 65542 }
> null [ { s: null }, { s: 1234 } ]

Since signed 64 bit ints are a native datatype for SQLite, I propose that BigInts that fit in a 64 bit int are stored as ints, and all the rest is stored as a BLOB.
Here's some N-API code that converts BigInt to Buffer: https://github.com/no2chem/bigint-buffer
Complications
Not sure what should happen when reading integers, since BigInts and Numbers are not easy to mix in JS. Most likely sqlite3 would need a flag to allow reading integers as bigint, but even then you'd want to decide on a column-by-column basis which one is read as a bigint.

Referencing on the comment, please check https://github.com/mapbox/node-sqlite3/issues/922#issuecomment-851007196


code of conduct

To get the ball rolling...
Please have a read over it, pretty sure there are improvements to be made (see the management team comment).
#50
(And yes, so far it is essentially a find and replace on 'conference' to 'meet-up').

@tomlane thanks a lot for this - great start


Adds initial CONTRIBUTING.md

Trying to start solve #23 this PR adds an initial Contributing.md file.

I do not understand the purpose of move the Contributing file to .github directory. Seems that is a common practice keep this file in root.

@cacarrara It is a general convention and I feel it helps keeps source clear of unrelated files.

@cacarrara Just checking if you're still on this :)


Metadata validation produces formatted logging message

Describe the issue
Running CLI commands that validate RTCs, validation messages are formatted in an unexpected way:
$ elyra-metadata list runtimes
[E 2021-09-01 07:28:28,848.848] Validation failed for instance 'my_other_kubeflow_test_instance' using the kfp schema with error: 'Tekton' is not one of ['Argo'].
Available metadata instances for runtimes (includes invalid):

Schema    Instance                         Resource
------    --------                         --------
airflow   my_airflow_test_instance         /Users/patti/Library/Jupyter/metadata/runtimes/my_airflow_test_instance.json
kfp       my_kubeflow_test_instance        /Users/patti/Library/Jupyter/metadata/runtimes/my_kubeflow_test_instance.json
kfp       my_other_kubeflow_test_instance  /Users/patti/Library/Jupyter/metadata/runtimes/my_other_kubeflow_test_instance.json  **INVALID** (ValidationError)

Unexpected as in "the CLI currently does not produce messages that use the server's logging format", which is why this one (and others that might be produced).
To Reproduce
Steps to reproduce the behavior:

Create a KFP/Tekton RTC
Stop JupyterLab
pip uninstall kfp-tekton
Run elyra-metadata list runtimes

Expected behavior
A clear and concise description of what you expected to happen.
Deployment information
Describe what you've deployed and how:

Elyra version: 3.0.1/master

Unexpected as in "the CLI currently does not produce messages that use the server's logging format", which is why this one (and others that might be produced).

I'm not understanding this comment.  The CLI utilities - including elyra-pipeline - are built on code used by the Elyra server.  This log statement is produced by that shared server code and has always been this way since this tool existed.

Below are examples of how other errors are reported in the CLI. Note the absence of [E 2021-09-01 07:28:28,848.848], since the output is the console and not not a log file.
$ elyra-pipeline blah
Usage: elyra-pipeline [OPTIONS] COMMAND [ARGS]...
Try 'elyra-pipeline --help' for help.

Error: No such command 'blah'.

$ elyra-pipeline describe nosuchfile
Error: Pipeline file should be a [.pipeline] file.

$ elyra-pipeline run kfp.pipeline

────────────────────────────────────────────────────────────────
 Elyra Pipeline Local Run
────────────────────────────────────────────────────────────────

Error: Invalid runtime configuration: local
 No such instance named 'local' was found in the runtimes namespace.

The messages you reference above are all produced directly by the application code.  The message indicated in this issue is produced by the Elyra server logging buried in the metadata service validation code.  For example, if you run with the --debug flag, you see the debug logging produced by the server.  Since validation is logged as an error, it is produced sans --debug:
$ elyra-metadata list runtimes --debug
[D 2021-09-02 07:49:02,129.129] Namespace 'runtimes' is using metadata directory: /Users/kbates/Library/Jupyter/metadata/runtimes from list: ['/Users/kbates/Library/Jupyter/metadata/runtimes', '/usr/local/share/jupyter/metadata/runtimes', '/usr/share/jupyter/metadata/runtimes', '/opt/anaconda3/envs/elyra-dev/share/jupyter/metadata/runtimes']
[D 2021-09-02 07:49:02,132.132] Loading metadata instance from: '/Users/kbates/Library/Jupyter/metadata/runtimes/kf_devises1.json'
[D 2021-09-02 07:49:02,135.135] Loading metadata instance from: '/Users/kbates/Library/Jupyter/metadata/runtimes/af_teacloth1.json'
[D 2021-09-02 07:49:02,135.135] Loading metadata instance from: '/Users/kbates/Library/Jupyter/metadata/runtimes/kf_eg.json'
[D 2021-09-02 07:49:02,136.136] Loading metadata instance from: '/Users/kbates/Library/Jupyter/metadata/runtimes/kf_cloning1.json'
Available metadata instances for runtimes (includes invalid):

Schema    Instance      Resource
------    --------      --------
airflow   af_teacloth1  /Users/kbates/Library/Jupyter/metadata/runtimes/af_teacloth1.json
kfp       kf_cloning1   /Users/kbates/Library/Jupyter/metadata/runtimes/kf_cloning1.json
kfp       kf_devises1   /Users/kbates/Library/Jupyter/metadata/runtimes/kf_devises1.json
kfp       kf_eg         /Users/kbates/Library/Jupyter/metadata/runtimes/kf_eg.json


changes to vlc instructions

I forgot to mention that birdnet-go will need to connect to the vlc rtsp stream using udp. I updated the readme to reflect that.  I also simplified the vlc --sout command line parameter.

Thanks very much!


Deep initials

I believe this will conflict with mine in the v5/microstep branch, which represents initial as a transition (InitialTransitionDefinition) rather than an array of states.
The transition definition's .target property can specify 1 or more state nodes, as this PR does.
Can you compare it to that branch? I'll make a PR for that soon


fix: fix unspecified namespace failure for k8s resources

What type of PR is this?

/kind bug
What this PR does / why we need it:
This PR fixes the unspecified namespace failure for K8s resources. Kusion will now first validate the intent resource id and patch the default namespace for Kubernetes namespaced resources without specified namespace field.
Which issue(s) this PR fixes:

Fixes #411
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 76.983%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
8766

Relevant Lines:
11387

💛  - Coveralls


How can I add mapped tags?

I have filter:
    def filters = {

        all(controller: '*', action: '*') {
            before = {
                def principal = securityService.principal

                MDC.clear()
                MDC.put("PrincipalID", principal.hasProperty('id') ? principal.id : "N/A")
            }
        }
    }

And follow Config.groovy:
// log4j.appender.SentryAppender.mappedTags=['PrincipalID']
log4j.appender.GrailsLog4jSentryAppender.mappedTags=['PrincipalID']

The PrincipalID exists in Additional Data section, but tags not appeared in Sentry.
What am I doing wrong?

Fixed here:
https://github.com/agorapulse/grails-raven/commit/544ef94ae5724785e17219227bc95e40435e886c

Fixed in Grails3 branch by @donbeave .
Thanks !


Add FTPS Transform

With correct file this time.

Let's subclass instead of calling a random method on a different protocol.

@aaspring what's the status of this pull request? It has noted problems. Do you plan to fix or should we close it?


Create a skeleton News Files for 0.7.0 Release

USing 0.6.0 as a guide.  Create a skeleton News section for release 0.7.0
For examples:
New Features
General
...
Updates to Existing Functions
Break Changes
Bug Fixes
Documentation
etc....see previous news file

@bms63 hope OK if i take this one on as i'd like to use a simple issue for the demo at the community meeting later?

closing this issue (as forgot to use closes keyword so github does it automatically later when merged to main, but i figure leaving this one open is of no use to anyone as its such a trivial update!) :)


Mistral AI embedding cannot stream response. status 400

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.

Example Code
from langchain_mistralai.embeddings import MistralAIEmbeddings
all_documents = []
embeddings = MistralAIEmbeddings()
for file_data in selected_files:
with tempfile.NamedTemporaryFile(mode="wb", delete=False, suffix='.pdf') as temp_file:
temp_file.write(file_data)
file_name = temp_file.name
loader = PyPDFLoader(file_name).load()
docs = text_splitter.split_documents(loader)
all_documents.extend(docs)
db = FAISS.from_documents(all_documents,embeddings)
Error Message and Stack Trace (if applicable)
An error occurred with MistralAI: Cannot stream response. Status: 400
Description
by uploading multiple pdf i cannot do document probing. due to mistral ai embeddings erroring out.
System Info
langchain==0.1.0
langchain-community==0.0.10
langchain-core==0.1.8
langchain-experimental==0.0.49
langchain-google-genai==0.0.5
langchain-mistralai==0.0.3
langchain-nvidia-ai-endpoints==0.0.1
python 3.11.7
macos 14.2.1 (23C71) sonoma

@dosu-bot when multiple pdf are being ingested then the error comes. but there is no problem for a single pdf.


[Feature Request] on-error how to get response?

Existing Component
Yes
Component Name
Upload
Description
on-error only give (err, file, fileList) param,i want to get ajax upload response body

这个需求可以用 http-request 属性来做。


Sends only half Ecommerce transactions

Doesn't seem to work properly and sends only half of data. In javascript everything works fine. I use it like this:
$t = new \Std\GATracker('UA-12345678-1', null, $tr->userId);
$t->send('transaction', array(
  'transactionId' => $tr->id,
  'transactionRevenue' => $tr->price
));

foreach($items as $item) {
	$t->send('item', array(
	  'transactionId' => $tr->id,
	  'itemName' => $item['name'],
	  'itemCode' => $item['id'],
	  'itemCategory' => $item['category'],
	  'itemPrice' => $item['price'],
	  'itemQuantity' => 1
	));
}

tcpdump shows that all requests are actually sent to analytics server so I don't know what is happening. Any ideas?

According to this and this:

_trackPageview need to be added before
_trackTrans need to be added after

I looked through the code of universal analytics and haven't spotted that those are added automatically for ecommerce tracking.
I'm also not usre if I should use command like
_trackPageview, _addTrans, _addItem and _trackTrans
or pageview, transaction, item and

@aboritskiy the _trackPageview and _trackTrans calls are related to GA Classic ga.js and have no bearing in this PHP library.
Could you re-test, after removing your last $tracker->send('trackTransaction'...) call? That one should not be necessary.


Cannot authenticate with POST form in Forward Authentication process

Do you want to request a feature or report a bug?
Bug
What did you do?
I'm trying to configure forward authentication on my traefik install.
Web apps are accessible at https://app.mydomain.tld/, and I have a simple http authentication POST form at https://auth.mydomain.tld/. The latter is my authentication backend, ans is also accessible form https://mydomain.tld:8081/ for testing purpose.
What did you expect to see?
I'm expecting to be able to login when browsing either to https://auth.mydomain.tld/ or https://app.mydomain.tld/, the following is true for both and I'll refer to them as https://*.mydomain.tld/.
When browsing to https://*.mydomain.tld/ for the first time, the response from the forward auth service being 401, the login form is displayed. When entering a valid password, the form data is send to auth, which responds with an authenticating cookie.
In presence of the cookie, any access attempt to https://*.mydomain.tld/ should be validated by the forward auth server, and successful.
What did you see instead?

When browsing to https://*.mydomain.tld/, the authentication form is presented to me, as expected.
After entering valid data in the form, no cookie is returned in the response.

The scenario is similar to the feature request #2620, which is closed by its creator with no solution.
I suspect this is because the POST request send to the authentication backend via  https://*.mydomain.tld/ is also filtered through the forward authentication process. Because I'm not logged in when sending this request, a new request is created by Traefik and send to the authentication backend without the POST data.
A solution would be to disable the forward authentication process when the requested URL matches that of the authentication backend. With this solution I could login from https://auth.mydomain.tld/ (not from https://app.mydomain.tld/, but that's solvable with appropriate redirections).
Note that the authentication process is working correctly when login in with https://mydomain.tld:8081/, which sets the cookie correctly.
Output of traefik version:
Version:      v1.7.4
Codename:     maroilles
Go version:   go1.11.1
Built:        2018-10-30_10:44:30AM
OS/Arch:      linux/arm
What is your environment & configuration (arguments, toml, provider, platform, ...)?
# traefik.toml
debug = true
logLevel = "DEBUG"

defaultEntryPoints = ["http", "https"]

[api]
    entryPoint = "traefik"

[docker]
    endpoint = "unix:///var/run/docker.sock"
    domain = "mydomain.tld"
    watch = true
    exposedByDefault = false

[entryPoints]
    [entryPoints.http]
        address = ":80"
        [entryPoints.http.redirect]
            entryPoint = "https"
    [entryPoints.https]
        address = ":443"
        [entryPoints.https.tls]
        [entryPoints.https.auth.forward]
            address = "http://auth:80/"

# docker-compose.yml for auth backend
version: '3.2'

services:
    auth:
        [...]
        container_name: auth
        labels:
            - "traefik.enable=true"
            - "traefik.frontend.rule=Host:auth.mydomain.tld"
        ports:
            - "8081:80"
        expose:
            - "80"

I'll be happy to help on a solution to this problem. As per the suggested idea, I figured adding the following lines in middlewares/auth/forward.go could help:
func Forward(config *types.Forward, w http.ResponseWriter, r *http.Request, next http.HandlerFunc)
     if (r.Host == config.Address) {
         next(w, r)
     }
    ...

but it can't work because the config.Address takes the form of http://auth:80/.

I am experiencing the same issue here. Will use an alternative port for my auth server for now, but implementing a fix would be great.

Hello!
I close this issue as it targets an unsupported version of Traefik.
Feel free to re-open it or join our Community Forum.


Tweak Wagtail security support dates

The two most recent Wagtail release series receive security support
https://docs.wagtail.io/en/stable/contributing/security.html#supported-versions

and added that section to the docs too

Thanks 👍🏽. Filing a separate PR to see if we can make the page a bit concise.


How do you create multiple new records inside the portal with a single call?

This library is a very thin wrapper over the FileMaker Data API, so you may need to consult their docs directly for how to format this. I haven't tried this recently, but I suspect that you can simply pass an array for the portal to create multiple records.


We should use call on graphic feature to get the actual data

The current way is too implicit and error prone for unfamiliar users, the current repr makes it confusing.

done in #92


[zstd] all versions after 1.2.0, zstd.h is modified incorrectly

Is your feature request related to a problem? Please describe.
Original line is here:
https://github.com/facebook/zstd/blob/ffde100b68bcfa76bc62e3ac7421068dba927f0e/lib/zstd.h#L44C1-L44C1
VCPKG modified it and `became:
#elif 1

This looks like a mistake in that the "else" condition would never hit.
As result when we compile zstd in Windows, Android SDK, arm64, , which does not support __declspec, and always produces an error:
__declspec attribute 'dllimport' is not supported
Proposed solution
This line replaced the original condition with "1":
https://github.com/microsoft/vcpkg/blob/83444c57754d01d584a4a371a070c5147e8a5aae/ports/zstd/portfile.cmake#L53C9-L53C9
The PR that introduced this change:
https://github.com/microsoft/vcpkg/pull/1032
No additional context was in the above PR and the end result does not look logical. Please remove replacement and keep the original condition untouched.
Describe alternatives you've considered
N/A, there is no workaround
Additional context
No response

There is no port bug: The replacement is done only for Windows targets:
https://github.com/microsoft/vcpkg/blob/83444c57754d01d584a4a371a070c5147e8a5aae/ports/zstd/portfile.cmake#L51-L55

There is no port bug: The replacement is done only for Windows targets:
https://github.com/microsoft/vcpkg/blob/83444c57754d01d584a4a371a070c5147e8a5aae/ports/zstd/portfile.cmake#L51-L55

My dev environment is indeed Windows, with VS2022 + Android Studio (NDK), whose clang does not support __declspec , how do I make this work?
In the end, isn't the original condition more solid and reliable for everyone?

In the end, isn't the original condition more solid and reliable for everyone? I am not understanding the reason behind the change to wipe it out...

What's your host compiler? The replacement is done only for Windows targets, so you're probably looking at wrong include (i.e. host instead of target).
The reason behind the replacement is to prevent errors when users forget to pass -DZSTD_DLL_IMPORT=1 when using shared zstd and when they do pass -dZSTD_DLL_IMPORT=1 when using static zstd.

Turned out it picked up the wrong triplet .. so that worked for me.
Final thoughts though, I see removing the condition is to help certain scenarios, but feel it is somewhat unexpected when the header file in vcpkg deviates from the original zstd repo.


Release v1.0.0-rc3

Codecov Report

Merging #73 into master will decrease coverage by 5.45%.
The diff coverage is 11.69%.

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


Failure assuming role via source_profile in ~/.aws/config

I wanted to capture this, although I haven't been able to root cause it. We have the following setup:
[profile central-ops]
aws_saml_url = home/amazon_aws/blah/111
region = us-east-1

[profile in-acct]
source_profile = central-ops
role_arn = arn:aws:iam::456456456:role/next-role
external_id =<PHONE_NUMBER>3123

When I attempt to
aws-okta exec in-acct -- aws sts get-caller-identity
I get the following error:
AccessDenied: User: arn:aws:sts::<central-ops-account-id>:assumed-role/Admin/my-username is not authorized to perform: sts:AssumeRole on resource: arn:aws:iam::456456456:role/next-role

I am not sure if this is related to the external id or something that aws-okta does not understand about the chaining of profiles and assumptions.

I have exact the same issue - my code that worked on multiple accounts via assuming roles from the central account started throwing errors:
AccessDenied: User: arn:aws:sts::account:assumed-role/role is not authorized to perform: sts:AssumeRole on resource: arn:aws:iam::target_account:role/target_role

Any chance you guys can post debug output?


Tests still use #library #import #source

+big_integer_vm_test: Fail, OK
+collection_to_string_test: Fail, OK
+exception_implementation_test: Fail, OK
+futures_test: Fail, OK
+map_test: Fail, OK
+maps_test: Fail, OK
+queue_test: Fail, OK
+splay_tree_test: Fail, OK
+string_base_vm_test: Fail, OK
+stopwatch_test: Fail, OK
+isolate2_negative_test: Pass
+compute_this_script_browser_test: Pass
+arithmetic_test: Fail, OK
+call_constructor_on_unresolvable_class_test/none: Fail, OK
+call_constructor_on_unresolvable_class_test/01: Fail, OK
+call_constructor_on_unresolvable_class_test/02: Fail, OK
+call_constructor_on_unresolvable_class_test/07: Fail, OK
+call_constructor_on_unresolvable_class_test/03: Fail, OK
+closure_type_test: Fail, OK
+const_var_test: Fail, OK
+ct_const4_test: Fail, OK
+cyclic_import_test: Fail, OK
+function_type_alias3_test: Fail, OK
+getter_setter_in_lib_test: Fail, OK
+import_collection_no_prefix_test: Fail, OK
+import_core_no_prefix_test: Fail, OK
+import_core_prefix_test: Fail, OK
+import_core_test: Fail, OK
+intrinsified_methods_test: Fail, OK
+issue1363_test: Fail, OK
+library5_test: Fail, OK
+library_private_in_constructor_test: Fail, OK
+numbers_test: Fail, OK
+prefix101_test: Fail, OK
+prefix10_negative_test: Pass
+prefix10_test: Fail, OK
+prefix11_test: Fail, OK
+prefix12_test: Fail, OK
+prefix1_negative_test: Pass
+prefix21_test: Fail, OK
+prefix4_negative_test: Pass
+prefix5_negative_test: Pass
+prefix_new_test: Fail, OK
+prefix_test: Fail, OK
+private_member3_negative_test: Pass
+private_member_test: Fail, OK
+top_level_prefixed_declaration_test: Fail, OK
+typed_message_test: Fail, OK
+crypto/base64_test: Fail, OK
+crypto/hmac_md5_test: Fail, OK
+json_test: Fail, OK
+utf8_test: Fail, OK

This can be closed. Nothing uses this syntax any more.


Bluetooth: controller: Fix PHY update procedure cachability

Fix missing PHY update procedure cachability omitted in
commit 16dbb9a4fef0 ("Bluetooth: controller: split: Fix cmd
disallowed and collision disconnects").
Relates to #31473.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

@mtpr-ot The change in this PR will allow the PHY update to be cached if there is a "different" transaction in progress, this will still return command disallowed if there is a peer initiated PHY update (same procedure) already in progress.
Do let me know if caching is required when same procedure is in progress?

@mtpr-ot The change in this PR will allow the PHY update to be cached if there is a "different" transaction in progress, this will still return command disallowed if there is a peer initiated PHY update (same procedure) already in progress.
Do let me know if caching is required when same procedure is in progress?


How bout them qualities

Finally remembered to install all these ~secret FL extensions~ for alleged wiki-ing purposes! Would you be willing to add quality ID-displaying to this one? Not sure why but I've always found those even more annoying to dig out than item IDs.

Which qualities and where?


Docs/add warnings readme

This should go into my forked branch at https://github.com/askjaer/python-best-practices


Extend unit-test framework so it is possible to execute entire suite with one command

RESOLVED in 77bfc52c


json files imported via import directive into variable show as variable not found

import "./file" as $fileData;
$fileData

shows $fileData as not found

Should be fixed now. But note that jq-lsp currently does not show any error if the included/imported file is missing.


Should we auto-open ports for load-balanced service?

Currently (on GCE) we don't automatically open ports for a load-balancer service.  Users must manually open them  (gcloud compute firewall-rules create ingress --allow tcp:80 --target-tags=kubernetes-minion).  Mentioned this to @thockin and he suggested that maybe we should be opening them.  Should we be, or do we expect users to open firewall-rules for any services they create?
In general, I'd like to understand the philosophy here.  AWS is wide-open right now in terms of firewall-rules, so I want to lock that down asap.

I assumed that we were doing this already.  The question is whether we can do this with enough granularity to not hurt anyone.

We have no mechanism for specifying any allowed src-ip other than <IP_ADDRESS>/0 (e.g. something on Service.Spec).
We could fairly easily add a flag to kube-cloud-controller, which would control the allowed src-ip range for an LB.  So no per-service control, but I would imagine the typical use-case for not allowing everyone is to restrict access to the entire cluster to a particular subnet.  We could also have a value that would mean "don't open this for me".
On GCE, opening a port on the LB actually means allowing traffic to the nodes - there are no rules on the LB itself.  This should be safe as long as kube-proxy has set up the firewall rules for the load-balancer first.  I don't know if we can have a catch-all rule for GCE LBs that would avoid this sequencing requirement.
On AWS & I believe OpenStack, you can set firewall rules directly on the LB, so no implementation problems here I don't think.

This was previously discussed in https://github.com/GoogleCloudPlatform/kubernetes/issues/3636 where we decided that it would be better not to automatically poke holes in the firewall for external services.

Note that one of my own replies said "I don't think we should automatically
open firewalls unless the user has explicitly asked for external access."
The bug shifted from "should we open firewalls for external load
balancers" to "should we auto-start services with external load balancers
and firewalls opened".  I am against the latter (e.g. my brand new cluster
should not come up with a for-pay cloud load balancer providing worldwide
access to kibana) but lean in favor of the former.
Sadly, though, GCE's firewall rules are under-specified.  If I put a
service on port 10250 and ask for a load-balancer, which opens firewalls I
have now put all of my kubelets on the internet.
On Fri, May 29, 2015 at 1:42 AM, Robert Bailey<EMAIL_ADDRESS>wrote:

This was previously discussed in #3636
https://github.com/GoogleCloudPlatform/kubernetes/issues/3636 where we
decided that it would be better not to automatically poke holes in the
firewall for external services.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/8900#issuecomment-106746271
.

Could we default-drop inbound traffic in iptables on GCE, and then just open the ports we need?  (It probably isn't a terrible idea in general anyway - defense in depth)

It's pretty hostile to any other use of the machines, which may be OK.
Which set of ports?  Every service?  all ports?  I'm not sure how to
describe this - security is not my thing :)
On Fri, May 29, 2015 at 5:10 PM, Justin Santa Barbara <
<EMAIL_ADDRESS>wrote:

Could we default-drop inbound traffic in iptables on GCE, and then just
open the ports we need? (It probably isn't a terrible idea in general
anyway - defense in depth)
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/8900#issuecomment-106965388
.

The heapster case was different -- it exposes cluster data and therefore requires auth.
To me, a request for an external load balancer means that you want it to be visible, and we should make that happen.

I don't know how feasible this is on other cloud providers, but on GCE could we do:
(1) have an instance-tag-controller that assigns a tag-$service to each node with a pod belonging to $service (https://cloud.google.com/sdk/gcloud/reference/compute/instances/add-tags)
(2) create a firewall that with --target-tag=tag-$service
(3) when the service is deleted, the firewall and all tag-$services are also removed?
Also possible that's not a default behavior for a service or it's controlled by serviceScope: {internal,external} or createFirewallRules: true or something.

I think we will NOT open firewalls for 1.0 because of issues on GCE and unknown others.  We will instead do #9150 for now, and consider how to do this by default safely after 1.0

@alex-mohr that doesn't fix the exposure on GCE, so I don't want to do it by default.

Decision reversed, fixed for 1.0

This goes all the way back to #3636 :)
On Jul 14, 2015 9:50 AM, "Tim Hockin"<EMAIL_ADDRESS>wrote:
Closed #8900 https://github.com/GoogleCloudPlatform/kubernetes/issues/8900
.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/8900#event-355559626
.


Adds ability to specify release notes to uploaded build

Closes #1
I have validated it with bitrise test run.

Hi @fileformat, are you ok with PR? Can you release it please?

Oh, this is perfect change! Please, merge it! 💪

Thank you @skywall !

@crrobinson14 are you gonna to release it any time soon? 🤞

@IntergalacticPenguin I am not a maintainer/contributor here, sorry. No comment.

@IntergalacticPenguin  I released v1.0.1 with this update

Is the v1.0.1 available in Bitrise? The feature for including release notes is still not working 😢


SR300 lost camera calibration

System Configuration

Version
Your Configuration

Operating System
Ubuntu 14.04.5 LTS

Kernel
4.4.0-64-generic

ROS
indigo

ROS RealSense
1.7.1

librealsense
1.12.1

Camera Type-Firmware
SR300 - <IP_ADDRESS>

Expected Behavior
After launching camera I expect it to return a non-zero distortion matrix
rostopic echo /camera/color/camera_info
below are the values I get back from my r200 camera (Camera FW: <IP_ADDRESS>)
header:
  seq: 47
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 884896012
  frame_id: camera_rgb_optical_frame
height: 1080
width: 1920
distortion_model: plumb_bob
D: [-0.09274807572364807, 0.07124239206314087, -0.00021547084907069802, 0.002587695373222232, 0.0]
K: [1408.539306640625, 0.0,<PHONE_NUMBER>4375, 0.0, 1411.0552978515625,<PHONE_NUMBER>019531, 0.0, 0.0, 1.0]
R: [1.0, 0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 0.0, 1.0]
P: [1408.539306640625, 0.0,<PHONE_NUMBER>4375, 0.0, 0.0, 1411.0552978515625,<PHONE_NUMBER>019531, 0.0, 0.0, 0.0, 1.0, 0.0]
binning_x: 0
binning_y: 0
roi:
  x_offset: 0
  y_offset: 0
  height: 0
  width: 0
  do_rectify: False
---

Actual Behavior
distortion matrix is zero
header:
  seq: 88
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 770626796
  frame_id: camera_rgb_optical_frame
height: 480
width: 640
distortion_model: plumb_bob
D: [0.0, 0.0, 0.0, 0.0, 0.0]
K: [617.9534912109375, 0.0,<PHONE_NUMBER>097656, 0.0,<PHONE_NUMBER>8125,<PHONE_NUMBER>970703, 0.0, 0.0, 1.0]
R: [1.0, 0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 0.0, 1.0]
P: [617.9534912109375, 0.0,<PHONE_NUMBER>097656, 0.0, 0.0,<PHONE_NUMBER>8125,<PHONE_NUMBER>970703, 0.0, 0.0, 0.0, 1.0, 0.0]
binning_x: 0
binning_y: 0
roi:
  x_offset: 0
  y_offset: 0
  height: 0
  width: 0
  do_rectify: False
---

Steps to Reproduce
roslaunch realsense_camera sr300_nodelet_default.launch
rostopic echo /camera/color/camera_info
FYI this is a new camera, up to this point I have been doing my tests on a r200. Is is possible to rewrite the camera_info to the camera? From what I have read this is not supported yet.
I do get a warning about the camera firmware. Could this cause the problem?
[ WARN] [1488496075.059926094]: /camera/driver - Detected unvalidated firmware:
-<PHONE_NUMBER>86's current camera firmware is <IP_ADDRESS>, Validated camera firmware is <IP_ADDRESS>

Thanks,
-Marq

I believe this is because the R200 has distortion for the color stream, but the SR300 and F200 do not.  See https://github.com/IntelRealSense/librealsense/blob/master/doc/projection.md

Thanks for the link mdhorn!