Bump ICE to v0.5.14

Codecov Report

Merging #818 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #818      +/-   ##
==========================================
+ Coverage   76.42%   76.47%   +0.04%
==========================================
  Files          59       59
  Lines        4246     4246
==========================================
+ Hits         3245     3247       +2
+ Misses        728      725       -3
- Partials      273      274       +1

Impacted Files
Coverage Δ

dtlstransport.go
68.02% <0%> (+1.16%)
:arrow_up:

Continue to review full report at Codecov.

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


Add Tests for SocketModeReceiver

Description
Need to write some tests for SocketModeReceiver after figuring out an appropriate test harness.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[x] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:
node version:
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

Probably, I will start working on this soon. If anyone in the community has thoughts or ideas about this, sharing them with us would be greatly appreciated!


External Login Sample causing Access Denied redirect if initial external login fails

I've been using the Quick Start sample for an Implicit Flow with External Login.  I've used the ExternalLogin and ExternalLoginCallback methods as shown in the AccountController.
However, I've found that if the ExternalLoginCallback fails prior to signing out the External Cookie, the user can't do another login with that provider, until they clear their cookies.  If they do, they're repeatedly redirected to the Access Denied page.  Is this expected behaviour or am I missing something?  Should the ExternalLoginCallback method wrap everything in a finally block to ensure that the external cookie is signed out?

Yep, this is a MSFT bug.

Ah thanks.  Do you have a link to that issue?  Would the recommended workaround be a SignOut call in the finally block?

Not off hand... somewhere in here: https://github.com/aspnet/security/issues

Ah okay, I think it's this:  https://github.com/aspnet/Security/issues/1067

Closing since it's a Microsoft issue.


Icon font to SVG migration part 1 #6107

Assorted icon font -> SVG conversions as part of #6107. This is missing a few areas of the CMS still, which will come in a "part 2" (#10278). In particular workflows, ModelAdmin, and listings will be done in a separate PR.
This might be easier to review commit-by-commit – I’ll add comments directly in the PR for specific changes. At a high level:

Icons have been switched to inline SVGs where possible, and SVG mask-image where inline SVGs would be impractical.
Some icons will render with different alignment after this change. I’ve decided to not adjust styles here, as we are revising the visuals of almost all icons in #2349, which will involve further alignment changes anyway.
There should be no backwards compatibility issues with those changes.

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?
[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[x] Please list the exact browser and operating system versions you tested: latest Chrome Safari Firefox on macOS 13.2
[x] Please list which assistive technologies [^3] you tested: None

[ ] For new features: Has the documentation been updated accordingly?

Thank you @lb-! The CI failures are just because I’ve cancelled the GitHub Actions builds, it’s all green in CircleCI (which is all that’s relevant for this kind of change)


Remove Render configuration & AWS resources

Just some lingering cleanup from our previous experience on Render. This removes the blueprint file that is no longer used, and some of the AWS resources that were specific to Render.

This also does not actually remove the Render-related domains and CloudFront because we already did that through configuration. The code supporting those that is being removed here is already effectively dead code.


Extend Perf Compare for Adopt Perf Builds

Problem Description
As part of https://github.com/AdoptOpenJDK/openjdk-test-tools/issues/136, we enabled the extraction of perf results from Adopt Perf builds. While the backend will be able to parse builds and store results, we will need to update our front end tool, Perf Compare, which currently doesn't have the support to show the results for multiple tests under one Jenkins build.
In Adopt, we run multiple tests in one Jenkins build. Hence, we need to update Perf Compare, so that it can show the results for all tests and not just one test.
Relevant Snippet
https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/52fbb9e5bd279ea12ae1b83821e795cd77c0bd47/test-result-summary-client/src/PerfCompare/lib/ExtractRelevantJenkinsTestResults.js#L30
Proposed Changes
We are exploring what might be the most efficient way to do so. We could either add some loops in Perf Compare or do the processing in the backend, which could possibly be used for other tools as well.

Closing this since it's complete.


:bug: tenancy: only delete LogicalCluster if needed

Taken from @ncdc's rebase https://github.com/kcp-dev/kcp/pull/2945

/lgtm
/approve

/approve


Add tagging taret to make file

Automate tagging of new release

nit: typo in commit message - Add tagging target ...

fixed, thanks!


Taskbar X appearance Settings after every restart in Windows dark theme only.

Describe the bug
The icon positions don't change, but the appearance settings change after every shut down. This didn't happen when I had Windows Light Theme, only after I enabled dark theme it started acting like this.
To Reproduce
Steps to reproduce the behavior:

Go to TaskbarX configurator and set the appearance of your taskbar as you like. (<IP_ADDRESS>)
Set is as Startup Application.
Set Windows theme to Dark mode.

Taskbar icon settings don't change, but appearance change to (0,0,0,0) White.

Expected behavior
It should not reset it appearance and remain saved.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Version: 20H2

Acrylic turns to transparent in dark mode too.

Further discussion at #641


How to train FasterSeg with customized labels?

Hello,
it's very interesting to train and use FasterSeg with own custom data.
To get information about that, I read the comments of this issue description.
Based on the description linked above, I did the following steps:

prepare my own labels in labels.py
create my own annotated pictures with createTrainIdLabelImgs.py
2.1 make sure that hight and width are divisible by 32
prepare file name lists as "dataset files" for data loading see examples here
adjust the config file with number of classes

Are there any other points in the FasterSeg repository which I need to adjust for using customized labels?
For example:
In cityscapes.py, camvid.py and bdd.py are some methodes like get_class_colors() and get_class_names() which return color or class names of the cityscapes data.
Is it neccessary to add the customized labels to this methods?
For which purpose are this methods?
It would be great if you can give me some hints to answer this questions, so I can run the training process with customized labels.
Best regards

Hi @KHeap25 !
Thank you very much for your interests in our work!
First, I have to admit that I have never run this repo for a brand new dataset from scratch. What I can do is try to help you with and bug or problem.
Some comments to your steps:

As I remember the gound truth files in cityscapes/bdd/camvid are single-channel image files with integers. Each integer represents the class index for each pixel. That means as long as you prepare your ground truth files in the same way, you can run our code and don't have to write your own labels.py and createTrainIdLabelImgs.py (unless you want to make your dataset as standard as cityscapes)
During training please make the height and width of cropped patches divisible by 64. Sorry I made mistakes in responses to previous issues (I already corrected them).
It is recommended to implement your get_class_names() function, as it will be used to print the results to the terminal (see here for usage). The function get_class_colors() is only used when you want to generate predictions in color (something like the front teaser images in our Readme) (see here for usage).

Hope that helps.

Thanks @chenwydj for your reply!
In relation to your recommendation above, I implement a new version of the get_class_colors() and the get_class_names()  methods in cityscapes.py, camvid.py and bdd.py based on the customized labels.
Furthermore there are some lines in test.py that need to be adjust for using the custom labels.
After passing the step pretrain the supernet, I got the following error during the step search the architecture.

It looks like something with the "TensorRT" latency test went wrong.
Maybe in darts_utils.py.
Do you have an idea how this error can be fixed?
If I remove "TensorRT" from the system, I am able to run all the training and evaluation steps.
But then, PyTorch is used for the latency tests.

Are there any important differences between TensorRT and PyTorch for the latency test?
I look forward to hearnig from you.
Kind regards

I would guess the tensorrt meets some problem, although you have installed it.
Here is the place where the function using tensorrt is imported, you may want to comment out this part:
https://github.com/VITA-Group/FasterSeg/blob/master/search/operations.py#L25

Hi @chenwydj,
firstly let me summarize the points which I adjusted for using own customized labels.

prepare the own custom labels in labels.py
adapt the get_class_colors() and get_class_names() methods in cityscapes.py, camvid.py and bdd.py, which return color or class names of the labels.
change the trainID2id mapping in test.py for the customized labels
set the number of classes in config_search.py and config_train.py
the annotated pictures can be created with createTrainIdLabelImgs.py

After doing the steps, I was able to run a training process with customized labels (with PyTorch for latency test).
Based on that, I have some questions about the logger for TensorBoard monitoring.

Are you logging the loss and some other things like mIoU, etc. for all steps in each epoch?
How should I interpret the graphs from step 1.2? See the following picture for example.

Do you use some cunstructions to identify or overcome overfitting?

It would be great if you can give me some advice, which would help me to understand the results of the training steps better.
Kind regards

Hi @KHeap25,
The "objective" indicates Eq. 5 in our Appendix B. It is a combined target of accuracy and latency, adopted from Tan et al., 2019.
The other parts is FPS, and the code is here. Arch0 indicates teacher net, arch1 the student. FPS0 indicates the architecture that aggregates outputs from [1/8, 1/32] branches, and FPS1 the aggregation from [1/16, 1/32].
Hope that helps!

@KHeap25 can you share your code to see how your modifications got afterall?

Hey @emersonjr,
@KHeap25 was in the project with me. You can find our code here: https://github.com/Gaussianer/FasterSeg


Add child content example to label API docs.

This change adds an additional example to show the new child content property for the label component.

@enchev, thanks for adding the child content property for labels! It is a far more general solution than what I included in https://github.com/radzenhq/radzen-blazor/pull/761, and it works great.


HTTP.POST is sending a GET request

Hi,
I am having an issue where, depending on the URL I am sending the request to, SwiftHTTP will either perform a GET or a POST, even though I am requesting a POST.
Here is my code:
        do {
            let call = try HTTP.POST(Configuration.API.sendMessageURL, parameters: ["username": Configuration.Saved.user!.guide!.username!, "id": "\(booking!.id!)", "message": text], headers: Helper.generateWSSEHeader())
            call.start { response in
                if let error = response.error {
                    print("Got an error \(error) => \(response.description)")
                    return
                }
                print(response.text)
                })
            }
        } catch let error {
            print("Got an error \(error)")
        }
    }

The only differences between the two URLs is one is HTTPS and the other not.
If I set the request to PUT it sends a PUT in both cases.
Testing with CURL, I was able to confirm the issue resides with SwiftHTTP and not the server.

Sorry, definitely not SwiftHTTP's fault, when I replaced the URL with https://httpbin.org/post it worked fine.


how to run Nuxt in production

Hey! i'm question, how to deploy Nuxtjs on production with Docker.

Hey. We have not an official guide for docker (yet). But this Dockerfile would be a good reference to get idea how to do that.

@pi0 I circled your Dockerfile, but it did not work.

So would you please share more about errors? What's failing?

I'm running exactly your Dockerfile, however when I access the HOST: DOOR the request is not complete. The container is running normal, but does not access via HOST: PORT

On play-with-docker.com show the Error forwarding request..

The instance should be reachable with internal port 3000 (as is preconfigured in mentioned Dockerfile). So:
# run in project root with Dockerfile
docker build -t nuxt-example .

# Run instance in foreground
docker run -it --rm -p 8080:3000 nuxt-example

I'll try to provide more detailed examples later.

Also about play-with-docker not personally tried it. Maybe @Tabrizian should give us some help :)

I think the problem in here is that there is a missing ENV HOST <IP_ADDRESS> in the Dockerfile and also there is a middleware redirecting http requests to https. If you add ENV HOST <IP_ADDRESS> in the Dockerfile of your project root I think your problem should be solved.

@Tabrizian this is my Dockerfile:
FROM node:8-alpine

MAINTAINER Rai Siqueira

ENV HOST <IP_ADDRESS>

RUN echo "--->>> creating app folder" && \
    mkdir -p /usr/src/app

WORKDIR /usr/src/app

COPY package.json /usr/src/app

RUN echo "--->>> Installing packages" && \
    npm install

COPY . /usr/src/app

EXPOSE 3000

CMD ["npm", "start"]

That's my docker-compose.yml, and this work :rofl:
version: "3"
services:
  front:
    image: node:7-slim
    working_dir: "/var/www/app"
    environment:
      - HOST=<IP_ADDRESS>
    ports:
      - "80:3000"
    volumes:
      - ./app:/var/www/app
    command: bash -c "npm install && npm run dev"

Use banian it's better :))
On Jul 7, 2017, at 10:08 PM, Rai Siqueira <notifications<EMAIL_ADDRESS>wrote:
That's my docker-compose.yml, and this work 🤣
version: "3"
services:
front:
image: node:7-slim
working_dir: "/var/www/app"
environment:
- HOST=<IP_ADDRESS>
ports:
- "80:3000"
volumes:
- ./app:/var/www/app
command: bash -c "npm install && npm run dev"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/nuxt/nuxt.js/issues/1038#issuecomment-313746639, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJoxV6jIcuxsZCjxpJUNJVO_SKMhz88Hks5sLm0XgaJpZM4OPhf1.

run in project root with Dockerfile
docker build -t nuxt-example .
Run instance in foreground with exported port 8080
docker run -it --rm -p 8080:3000 nuxt-example
then  view
<IP_ADDRESS>:8080
no any  Response

But how to deal with baseUrl = http://localhost in production enviroment on real server?


Update skip list

Issue #, if available:
Description of changes:
Adding async inference notebook to the skip list
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Need to merge because of a launch and the failing CI is blocking the team


[Extensibility Enhancement] Event OnAfterLogInteraction Codeunit 5051 SegManagement

Please change the signature of Event OnAfterLogInteraction in function Log interaction to allow Extensibility.
procedure LogInteraction(SegmentLine: Record "Segment Line"; var AttachmentTemp: Record Attachment; var InterLogEntryCommentLineTmp: Record "Inter. Log Entry Comment Line"; Deliver: Boolean; Postponed: Boolean) NextInteractLogEntryNo: Integer
    var
        InteractionTemplate: Record "Interaction Template";
        InteractLogEntry: Record "Interaction Log Entry";
        Attachment: Record Attachment;
        MarketingSetup: Record "Marketing Setup";
        TempDeliverySorter: Record "Delivery Sorter" temporary;
        InterLogEntryCommentLine: Record "Inter. Log Entry Comment Line";
        AttachmentManagement: Codeunit AttachmentManagement;
        FileMgt: Codeunit "File Management";
        WizardAction: Enum "Interaction Template Wizard Action";
        FileName: Text;
        FileExported: Boolean;
    begin
        OnBeforeLogInteraction(SegmentLine, AttachmentTemp, InterLogEntryCommentLineTmp, Deliver, Postponed);

        ....
        OnAfterLogInteraction(SegmentLine, InteractLogEntry); // OLD CODE
       OnAfterLogInteraction(SegmentLine, InteractLogEntry, Deliver, Postponed); //NEW CODE
    end;

[IntegrationEvent(false, false)]
    local procedure OnAfterLogInteraction(var SegmentLine: Record "Segment Line"; var InteractionLogEntry: Record "Interaction Log Entry"; Deliver: Boolean; Postponed: Boolean)
    begin
    end;

Thank You!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


flutter-devicelab-linux-6 can't discover its tethered device

https://ci.chromium.org/p/flutter/builders/prod/Linux flutter_gallery__transition_perf_hybrid/637
https://ci.chromium.org/p/flutter/builders/prod/Linux flutter_gallery__transition_perf_e2e/646
[flutter_gallery__transition_perf_e2e] [STDOUT] stderr: adb: failed to install build/app/outputs/flutter-apk/app-profile.apk:
[flutter_gallery__transition_perf_e2e] [STDOUT] stderr: Application failed to start on attempt: 1
[flutter_gallery__transition_perf_e2e] [STDOUT] stderr: error: device 'ZY223D5F28' not found
[flutter_gallery__transition_perf_e2e] [STDOUT] stdout: Installing build/app/outputs/flutter-apk/app-profile.apk...          2ms
[flutter_gallery__transition_perf_e2e] [STDOUT] stderr: Error: ADB exited with exit code 1
[flutter_gallery__transition_perf_e2e] [STDOUT] stderr: Performing Streamed Install
[flutter_gallery__transition_perf_e2e] [STDOUT] stderr:
[flutter_gallery__transition_perf_e2e] [STDOUT] stderr: error: device 'ZY223D5F28' not found

Device seems to be up and running now:
flutter-devicelab-linux-6$ adb devices
List of devices attached
ZY223D5F28	device
@jmagman can you let me know if you still see any problems on this device.


🌐 Add Ukrainian translation for docs/uk/docs/fastapi-people.md, docs/uk/docs/python-types.md, docs/uk/docs/tutorial/index.md

Missed documentation on docs/uk/docs/external-links.md because it is translated into #4037 and docs/uk/docs/features.md/docs/uk/docs/index.md as they are translated into #1791

Thanks for your interest!
It's best to have each translation file as a separate PR, that way each one can be independently reviewed, and as soon one of the translations is ready it can be merged. But with all of them on the same PR, they would all have to be ready and perfect for the whole PR to be merged. And also, reviewers would have to be able to make time to review all the changed files before approving the whole PR, so it would take a lot longer. So, please separate the translations into separate PRs. :pray:
Thanks!

Maybe you or others can take parts/files of this PR and make them as independent PRs. That would work. :nerd_face:

As this PR had requested changes to be applied but has been inactive for a while, it's now going to be closed. But if there's anyone interested, feel free to create a new PR.


Merge to stable request

We would like 30719b37a9397cc38610e71a804236ec314535ec and d2b39d1a07a18572bdafedb79ce68ef5bf9e64bf cherry-picked to the new stable branch.
/cc @athomas @dcharkes

Do you mean merge to dev? Stable would mean you intend to publish this as
2.5.3?
On Thu, Oct 17, 2019, 19:48 Martin Kustermann<EMAIL_ADDRESS>wrote:

We would like 30719b3
https://github.com/dart-lang/sdk/commit/30719b37a9397cc38610e71a804236ec314535ec
and d2b39d1
https://github.com/dart-lang/sdk/commit/d2b39d1a07a18572bdafedb79ce68ef5bf9e64bf
cherry-picked to the new stable branch.
/cc @athomas https://github.com/athomas @dcharkes
https://github.com/dcharkes
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/38952?email_source=notifications&email_token=AEYRJB44T53VJRHFNT2RHXTQPCQPFA5CNFSM4JB47XKKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HSRBL2A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEYRJB76M4B7TKKCSI54NBDQPCQPFANCNFSM4JB47XKA
.

Well that depends: Did we branch for a D26 stable release or dev release?

That's not quite how Dart releases work.
We merge to dev to start the cherry-pick season (2.6.0-dev.8.0). We cherry-pick to dev until we have something we want to release (2.6.0-dev.8.1). Then we merge to stable and release (2.6.0). Then we cherry-pick to stable and release hotfixes (2.6.1).
Currently, 2.5.2 is at the tip of stable and 2.6.0-dev.8.0 is at the tip of dev.
More details are here:
https://github.com/dart-lang/sdk/wiki/Branches-and-releases

Pretty sure we mean merge-to-dev here...

Merged into 2.6.0-dev.8.1 (61d12745a8a2fedfde8e5644a43baa45ae1a5d9a).


Switch from &str to String in ReflectPathError

Objective

ReflectPathError uses non-owned strings, meaning that often when returning errors lifetime issues are encountered.
Things like the ? operator need to often be used with map_err call to work

Solution

Let the error type own the string, this increases ergonomics at negligible cost.
I don't think these errors are thrown a lot (at least in my use case)

Migration Guide

Change type from &str to String in InvalidField, UnexpectedToken and ExpectedToken variants of bevy_reflect::ReflectPathError

I've changed this to Cow<'a,str> now, that seems like the perfect compromise.
How does this end up on the hotpath exactly?

I've changed this to Cow<'a,str> now, that seems like the perfect compromise.

You can probably replace 'a with 'static and remove the lifetime parameter from it. This was mostly to avoid allocating when we have literals.

How does this end up on the hotpath exactly?

We read the field path from an animation clip alongside values of the appropriate type, then use GetPath to use Reflect to write the animated value to it. This can happen 10+ times per animated entity per frame. You might see a few hundred to a few thousand of these used per frame. And because it's coming from a saved asset and not code, if it fails, it'll fail repeatedly and quickly.

Thanks James; that context is super helpful here. I like the use of Cow here, and like you said, I think we may be able to get rid of the lifetime.

Alrighty, Done.
Now I didn't want to touch the external API much so there are two to_owned() calls:

https://github.com/makspll/bevy/blob/e8db50d9572e5f2e0b78ac96aaceb17f68667756/crates/bevy_reflect/src/path.rs#L242-L269
https://github.com/makspll/bevy/blob/e8db50d9572e5f2e0b78ac96aaceb17f68667756/crates/bevy_reflect/src/path.rs#L271-L298

Both only happen for the ReflectPathError::InvalidField error variant,  could maybe be avoided if Cow's are passed around instead of &strs

Hi @makspll! Are you planning to add benchmarks for this change? If you need help, maybe reach out to the community on Discord (I'm sure someone there has a good idea of how to go about this well)? Or we could also move this over to S-Adopt-Me for someone else to just pick up and finish the benchmarking. Up to you!

Hi @MrGVSV Apologies, I completely forgot about this PR, at this point yeah it's probably better adopted :)

Backlog cleanup: closing due to inactivity, could still be adopted but ReflectPathError has changed shape quite a bit since then.


feat: Parametrize metadata_options.http_protocol_ipv6

what
This adds option to disable http_protocol_ipv6 for launch templates.
why

One of our workloads requires this setting to be disabled.

/test all

The new var.metadata_http_protocol_ipv6_enabled variable should default to false because the launch_template resource defaults it to false. (See: https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/launch_template#metadata-options.)
In fact, some configurations (particularly with Xen-backed instances) will not allow it:
Error creating Auto Scaling Group: ValidationError: You must use a valid fully-formed launch template. The HttpProtocolIpv6 metadata option cannot be enabled for Xen-backed instance types.

I've taken the liberty to update this.

/test all

Added a no-release flag to consolidate this alongside #81 in a 0.30.0 release.


future install_aliases is broken in python3.12

fixes #13

pinging @vsoch and @rwblair, could this pull request be merged/release be made? Otherwise we can point to this branch for NiMARE's release.

I'm not exactly sure why install_aliases was there in the first place, but I suspect it's not needed. I left Poldracklab in 2016 and don't maintain this anymore so I defer to @rwblair.


Fix: Thread blocking

Closes #37

Updated rust version in Dockerfile
Replaced blocking code with async variants

<EMAIL_ADDRESS>crate, that was used to watch the logfile, has only blocking api and the newer version has issues with the tokio async runtime.
Good news is that we don't really need this crate's functionality as we can check if the file was removed by querying the amount of hard links to it via https://doc.rust-lang.org/stable/std/os/unix/fs/trait.MetadataExt.html#tymethod.nlink.

@dddenis Very nice PR! Thank you very much!


rate change pages return user to previous page

closes #92
When accessing entry_rates, transition_rates, and exit_rates, the back/return to x page/clear all rate adjustments buttons now returns the user to the page they accessed them from (adjusted or costs).
The obvious choice would be to set prev_page with request.META.HTTP_REFERER but when you save your changes to a transition rate table, the previous page is the page containing the table (e.g. if we save the entry_rates page, request.META.HTTP_REFERER gets entry_rates not adjusted/costs).
I'm not certain saving it in the session as "prev_page" is a good key name as it's not always the previous page, but I'd like guidance on whether I'm tacking this the right way before I figure out what to do here. I could delete the session key once the prev_page variable is set on a given rate change page but I'd need to work out where to do it.

Using the session makes sense with the code as it stands (in that it looks like it is intentionally done such that saving the form does not automatically return the user to a previous page). From a naming perspective, perhaps something more verbose could be useful e.g. entry_rates_origin_page. And then in the html I think we'll need something cleaner in the button (as I anticipate the current {{prev_page}} would output a button name that looks more like a URL segment)

@christina-moore Thanks for the feedback.
I've changed the prev_page variable to rate_change_origin_page based on your suggestion which I think is a better explanation of of the variable.
I've also changed the logic on the button that returns you to the previous page from the rate adjustment pages to have a better button text. I have two points/questions(?) on this:

Is it okay to put the logic in the html template, or would it be better to have it in the views and pass the button text as a variable to the template in the session as I did the rate_change_origin_page string?
I've used lower case 'adjust forecast' and 'projecting spend' text on the buttons, in keeping with other buttons on the site, but I've also added the word page to the end which isn't on buttons elsewhere in the site because it felt more syntactically correct, but I'm no grammarian, and I don't know if there is a best practice for button naming, I'd love to hear suggestions?


Publish 1.4.0 to npm

Would you be able to publish 1.4.0 to npm?
We're facing an issue that would benefit from the change in #56
Thanks.

@eamodio Would you mind doing that?

https://github.com/microsoft/vscode-test/blob/master/CHANGELOG.md is already updated
yarn version for 1.4.0
npm publish --otp xxxxxx

@eamodio do you know when the next release might happen? I'm also very eager to be able to depend on #56.

@ckant @jcreedcmu Sorry for the delay -- 1.4 has now been published


Registering Domain Events

Hello,
Thank you for this project!
How do you register a listener for domain events in an existing class (as oposed to creating a new class to implement IHandle)?
In a previous example I've seen you do it with the Register method in a Domain Events Class, (e.g.: DomainEvents.Register(Handle);) , but in this project you seem to have left it out.
How do you achieve it here, and why did you leave it out?
Thank you,
Damião

I prefer to use the convention approach of using handler classes that implement IHandle<SomeEvent>. These are wired up automatically, so they don't require an explicit call to Register. You can still use that code if the convention doesn't work for some reason (e.g. for an Aggregate event where the aggregate root wants to register for events raised by its children). I didn't include an example of that because this is meant to be a solution template, not a full sample application, and I wanted to minimize how much code folks would need to delete whenever they started a new solution from it. HTH.

Thank for your pointer. It clarifies the issue.
The difficulty I'm finding seems to come from the exact exception you referenced.
I have and aggregate root, where I need to listen for the events raised by it's children (in this case to manage the updates), that I have declared as:
public class ConfApp : BaseEntity, IHandle<ChildUpdatedEvent>
My hope was to catch the event of the child update on the aggregate root and decide on what happens depending on the condition of the aggregate (with changes to the aggregate being persisted in the same transaction).
When the DomainEventDispatcher.Dispatch tries to dispatch the event I get an exception on var handlers = _container.GetAllInstances(handlerType);
Based on the result message (pasted at the end) it would seem that Structure Map tries to reinstanciate the aggregate root, and fails because it would need parameters for that. Starting eith the Id...
If this is the case, would this justify including the Register method for the Domain Events, and how could it be updated for this codebase?
Hope this is not too off track... Thanks!

Result Message:
StructureMap.Building.StructureMapBuildException : Error while building type System.Guid.  See the inner exception for details
1.) new Guid(*Default of Byte[]*)
2.) System.Guid
3.) Instance of System.Guid
4.) new ConfApp(*Default of Guid*, *Default of Guid*, *Default of IEnumerable<ChildUpdatedEvent>*)
5.) SIIC.Core.Models.ConfApp.ConfApp
6.) Instance of IHandle<ChildUpdatedEvent> (SIIC.Core.Models.ConfApp.ConfApp)
7.) Container.GetAllInstances(IHandle<ChildUpdatedEvent>)

---- System.ArgumentException : Byte array for GUID must be exactly 16 bytes long.
Parameter name: b

I think I see the problem. You're trying to use an aggregate event with a post-persistence domain event pattern (the one I include in this sample by default). I personally distinguish between pre- and post-persistence events, because there are cases in which you want each. Specifically, I classify aggregate events (events raised by children and handled by the root) as pre-persistence since you will likely want to confirm the aggregate is in a valid state prior to persisting it. For the sake of simplicity, I don't include a pre-persistence domain event implementation in this sample. However, you can basically copy Udi's DomainEvents.cs static class from here to get one:
http://udidahan.com/2009/06/14/domain-events-salvation/
Copy that into your SharedKernel and use it from your aggregates. Call DomainEvents.Register in the root app to register for the event and DomainEvents.Raise in the child. Be aware that if you have multiple aggregates in the same thread, by default they will handle one another's events, so you'll want to include some way to detect that an event belongs to your child, not some other aggregate's child.
Hope that helps.

Pre- and post- persistence events make much more sense. Thank you!
I was convinced that the aproach followed in this project was substituting Udi's DomainEvents.cs static class, but now I understand that they can both be combined.
I was also not aware of the pitfall that multiple aggregates in the same thread can represent and will take that into account. HT!
On question. How do you import the IContainer created of the Startup.cs to Udi's DomainEvents.cs static class?
Because it is a static class it can't be passed as a parameter in the constructor for DI as I would normally do. And the aproach Udi used (below) with the ObjectFactory is incompatible with version 4+ of Structure Map, that is used in this project. Is there a workaround?
static DomainEvents()
{
    Container = StructureMap.ObjectFactory.Container;
}

Make container a public property and set it in Startup after you configure the container.

On Jul 20, 2018, at 19:12, hugodamiaodias<EMAIL_ADDRESS>wrote:
Pre- and post- persistence events make much more sense. Thank you!
I was convinced that the aproach followed in this project was substituting Udi's DomainEvents.cs static class, but now I understand that they can both be combined.
I was also not aware of the pitfall that multiple aggregates in the same thread can represent and will take that into account. HT!
On question. How do you import the IContainer created of the Startup.cs to Udi's DomainEvents.cs static class?
Because it is a static class it can't be passed as a parameter in the constructor for DI as I would normally do. And the aproach Udi used (below) with the ObjectFactory is incompatible with version 4+ of Structure Map, that is used in this project. Is there a workaround?
static DomainEvents()
{
Container = StructureMap.ObjectFactory.Container;
}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

That did it!
Thank you for all the help. This is my first project using DDD, and your material has been most helpful!
In a future iteration of this project, if you find it follows the spirit in which this project was created, you may consider including both types of domain events, as both will likely e needed.


upload file with multipart/form-data does not work

I followed this example: https://github.com/AnWeber/httpyac/tree/main/examples#multipartform-data to make a file upload request.
request.http:
### upload file
POST http://<IP_ADDRESS>:9167/testing/v2/uploadExcel HTTP/1.1
Content-Type: multipart/form-data; boundary=--WebKitFormBoundary

--WebKitFormBoundary
Content-Disposition: form-data; name="file"; filename="2.xlsx"
Content-Type: application/xlsx

< ./2.xlsx
--WebKitFormBoundary

but got the following output:
=== upload file ===
POST http://<IP_ADDRESS>:9167/base-billing-manager/v2/receipt/uploadExcel
accept: */*
accept-encoding: gzip, deflate, br
content-length: 20458
content-type: multipart/form-data; boundary=--WebKitFormBoundary
user-agent: httpyac

HTTP/1.1 200 OK
access-control-allow-credentials: true
access-control-allow-headers: Origin, X-Requested-With, Content-Type, Accept, Cookie
access-control-allow-methods: GET, POST, PUT, DELETE, OPTIONS
access-control-allow-origin: *
access-control-max-age: 3600
connection: close
content-type: application/json;charset=UTF-8
date: Wed, 21 Jul 2021 07:21:42 GMT
transfer-encoding: chunked
x-application-context: application:9167

{
  "code": 10000001,
  "message": "Required request part 'file' is not present",
  "data": null
}

It seems the body part is missing in this request, could you help me find out if I've missed something in this request?

Exactly this error also cost me a good hour yesterday. Cause with me a missing blank line between < ./2.xlsx. and the end for the multipart/form-data. The following then worked for me (spring endpoint)
### upload file
POST http://<IP_ADDRESS>:9167/testing/v2/uploadExcel HTTP/1.1
Content-Type: multipart/form-data; boundary=--WebKitFormBoundary

--WebKitFormBoundary
Content-Disposition: form-data; name="file"; filename="2.xlsx"
Content-Type: application/xlsx

< ./2.xlsx

--WebKitFormBoundary

I wanted to look at it again and find what the spec for multipart/form-data says about this case, but haven't had the time yet.  Also, my example of this, I will check again, not that I accidentally inserted the blank line there.

Exactly this error also cost me a good hour yesterday. Cause with me a missing blank line between < ./2.xlsx. and the end for the multipart/form-data. The following then worked for me (spring endpoint)
### upload file
POST http://<IP_ADDRESS>:9167/testing/v2/uploadExcel HTTP/1.1
Content-Type: multipart/form-data; boundary=--WebKitFormBoundary

--WebKitFormBoundary
Content-Disposition: form-data; name="file"; filename="2.xlsx"
Content-Type: application/xlsx

< ./2.xlsx

--WebKitFormBoundary

I wanted to look at it again and find what the spec for multipart/form-data says about this case, but haven't had the time yet. Also, my example of this, I will check again, not that I accidentally inserted the blank line there.

I switched to Rest Client in vscode, and it worked with following modification:
change "--WebKitFormBoundary" to "----WebKitFormBoundary"
Then it worked.
Also, I've succeed posting with httpyac with adding a new blank line. Thanks.

My PDF example was apparently special because it had a last blank line. This has obscured the error. The cause is that the file import removes a line break, but it is needed.

Error is fixed with version 2.21.0. thx:-)


Empty fields are not getting validated

I've encountered an issue with validator generation in the case of oneof fields - the validator does not seem to check if the field is nil or not.
The proto I've tested this with is https://github.com/TheThingsNetwork/lorawan-stack/blob/895d4920ebd1a2dc5207f45b70c3b83eb896df7d/api/applicationserver_pubsub.proto#L48-L53 (I've seen that the same validation is used for ApplicationUp).
The resulting validator code does not check for nil fields - see https://github.com/TheThingsNetwork/lorawan-stack/blob/895d4920ebd1a2dc5207f45b70c3b83eb896df7d/pkg/ttnpb/applicationserver_pubsub.pb.validate.go#L322-L349 . If Provider is set to nil, the default path is never taken, since nats is added in case no sub fields are specified.

The links are broken
This issue belongs in https://github.com/TheThingsIndustries/protoc-gen-fieldmask


update documentation install steps

Documentation should be updated to instruct users to run npm install --save moment-msdate.  Also, see about momentjs updating their documentation located here: http://momentjs.com/docs/#/plugins/msdate/

When I originally put this together the momentjs site was simply forked and
PR'd back into their repo.
On Monday, August 10, 2015, Brian Baker<EMAIL_ADDRESS>wrote:

Documentation should be updated to instruct users to run npm install
--save moment-msdate. Also, see about momentjs updating their
documentation located here: http://momentjs.com/docs/#/plugins/msdate/
—
Reply to this email directly or view it on GitHub
https://github.com/markitondemand/moment-msdate/issues/4.

--
Sent from my iPhone

Yes @brianbaker, see this for reference. Hope all's well!


use compiler to ensure interface is implemented

simple code to ensure interfaces are actually implemented. compiler will detect method signature divergence, and flag farcical implementation.

No objections, so gonna yolo this puppy.


pend() called for no reason inside executor poll

https://github.com/embassy-rs/embassy/blob/main/embassy-executor/src/raw/mod.rs#L412
dequeue_expired() call wake_task() on all expired tasks, moving them into run_queue, but if run_queue is empty wake_task() will call pend() (resulting in another call to poll() when using InterruptExecutor).
A possible fix could be creating a function wake_task_no_pend() that call directly self.run_queue.enqueue() instead of self.enqueue().
I'll be happy to open a PR if the bug is confirmed.

fixed in https://github.com/embassy-rs/embassy/pull/1748


AJ-1658: Prefer CollectionId over {UUID,String} collectionId

PR got too big.  Closing it down.  Smaller subset in:  https://github.com/DataBiosphere/terra-workspace-data-service/pull/812


Update build-local.sh

WHAT
A brief description of the pull request:
This script adds logging for each major step of the process and checks if npm is installed before attempting to run npm install. Additionally, it stores the directory of the script in a variable for clarity and consistency throughout the script.
HOW
Steps to test or reproduce:
N/A

Hi @bunnydevx - I see you've forked this repository, did you mean to open the pull request here or on your fork?

Hi @bunnydevx - I see you've forked this repository, did you mean to open the pull request here or on your fork?

Hi! I did mean to open it here - i had to fork it to requst an update

Hi @bunnydevx - I see you've forked this repository, did you mean to open the pull request here or on your fork?

Hi! I did mean to open it here - i had to fork it to requst an update

OK. Unfortunately we can't accept external contributions on this repository at the moment, so I'll have to close this pull request. Thanks for your understanding!


Not able to connect to userinfo endpoint to read claims

This is how my config.cs file looks on IS4
Part 1 https://puu.sh/C7zhg/5bf6017551.png
Part 2 https://puu.sh/C7zh0/a65723cdbb.png
This is my client and this is how I call IS4 userInfo endpoint https://puu.sh/C7zhM/772febf446.png
Here is the error: https://puu.sh/C7zka/3c5b7a55d7.png
Can someone please help?

You are using the client credentials grant type. In this grant type there is no user, and thus you can't call userinfo.


All the mapper bugfixes together for real.

Current dependencies on/for this PR:

#539  👈
main

This stack of pull requests is managed by Graphite.


feat: Implement DatetimeControl and add tests

@DrewHoo Looks like I need your review on it too since you previously requested changes.

Looks like dependencies still aren't updated? Also the DateControlOptions ought to be differentiated instead of added to a union with TextControlOptions. Something like this might work:
type JsonSchemaTypeToControlOptions<
  T,
> = T extends { type: infer U; format?: infer V }
? U extends "object"
    ? unknown
    : U extends "string"
      ?  V extends "date" ? DateControlOptions : TextControlOptions
      : ...

@DrewHoo Is this no longer required? If not we can close this PR

Yeah I think so!


Chapter 17 Question 10 - possible solution contribution

I've included my solution to Chapter 17 Question 10 in which I build a simple VAE for an RGB image dataset of flowers.  The generated flowers aren't as good as I had hoped, but I think there are some interesting results.
The original version of this can be found here:  https://github.com/KirkDCO/HandsOnML_Exercises/blob/main/Ch17_Q10.ipynb
In that version, I also train a standard Convolutional Autoencoder and look at reconstructions of images.  I left that portion out for brevity and not to distract from the VAE aspect.  I would like to take this extended version and create a Kaggle notebook to share, as long as that is OK.  I don't want to create any conflict of interest between content of this notebook and one on Kaggle with much the same content.
I appreciate any comments on code.  I don't expect you to fix any errors and I respect your option to decline this PR if the results aren't good enough.
I may work on this more in the future, and if I come up with enhancements, I'm happy to share those.
I will likely use the same dataset to develop a GAN for Question 1.  Once I have some interesting results, I'm happy to share/contribute that code as well.

Hi @KirkDCO ,
Thanks a lot for the PR! Really appreciate your help. I haven't had time to look into it yet, I'll try to do this early next week.

No hurry whatsoever!  I'm still toying with it a bit to try and improve the fake flower images, but haven't been able to improve upon it yet.  I will probably start working on Question 11 and application of a DCGAN to the same dataset soon.

I modified my VAE to have a dramatically reduced coding layer size.  The results are much better, but still not quite flower like.  Regardless, I'm hopeful it demonstrates a useful VAE on a different dataset.

I just added a candidate solution for Q11 in which I work through a DCGAN.  Again, images won't fool anyone, but it is a working version.

Hi @KirkDCO,
Thanks again for your contribution, I finally had some time to look at it today, you spent quite some time on this, I really appreciate it.
Here are a few comments regarding the VAE:

I like the description of your tests and the difficulties you ran across, but I'd like to keep the solutions fairly tutorial-like. I suggest you could have the full notebook in your github repo, which this notebook could point to for the readers who want the full details (and attribution to you), and in this repo we could shorten the solution to keep only the second try, which looks much better. Would that work for you?
There are many images, which more than doubles the size of the notebook (from 6.3M to 13M). I'm trying to limit the size of the repo, so it would be great to output only a few images.
The flowers dataset is already introduced in chapter 14, so it would be best to load it in the same way. It would be simpler than requiring the user to download it on their Google Drive, and it would work on local Jupyter, not just Colab.

What do you think?

I adjusted Q10 to use the TensorFlow Flowers dataset, but the results are much, much worse.  There is always the chance that I've broken something in the process, but in working with it, I cannot seem to improve the performance.
I will continue to work on it, but I may cancel the PR as these results are not really worth including, in my opinion.

Q11 was adjusted to use tf flowers dataset.  GAN results are much poorer with this dataset, despite a lot of effort on tuning.
Happy to defer to your judgement to keep/reject either submission.


Simplify configuration

To simplify configuration I switched to using the project id as a key instead of an arbitrary string and instead of creating bots based on the projectBots file I create them by looping over the channels config file and checking what bots the channels need created. This allows us to not need an entry in projectBots if there are no named mappings (or in the future other configuration) making projectBots an optional definition. This will make it simpler for artists to set up their channels quickly without having to work in two config files. I will be extending the projectBots config file in an upcoming change to allow a contract field. This will be used by PBAB partners to denote that the project id is not from the Art Block contracts.
closes #216

@Asupkay letting you merge in case you want to watch for anything during deployment

can be a separate PR, but we can reflect these recent updates in our ./.github/ISSUE_TEMPLATE/* files as well!

Opened #221. Thanks!!


Refine allowed methods typing

As implemented in: buoy_protocol.erl

Hi, thanks for your contribution. I'll add some tests for HEAD requests and merge this PR.

Turns out adding tests is non trivial since HEAD requests only work when content-length is chunked. I'm working on a fix, should be done by EOW.

Closing in favour of https://github.com/lpgauth/buoy/pull/34.

@javiergarea FYI, pushed a new released tagged 0.2.4


Merge repetitive conditionals

So much easier to read and probably faster this way too.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

So much easier to read and probably faster this way too.

Definitely easier to read, but I'd be surprised if creating an array and calling a method on it is faster than the bare conditional.

It is actually quite a bit slower per the benchmarks.


GH-993: Conversion Interceptor Improvements

Fixes #993
If the converter cannot convert due to a lingering content-type header, take a second
shot with the header removed.
If we still fail, throw a MessageConversionException.

These changes look good to me.


url /pub/job backwards compatibiltiy

Hi @LookACastle
Regarding the fatman -> job degreasing, we had discussed backwards compatibility. This has been interpreted in the widest possible terms in the Lab and hence people have scripts calling the api endpoints, e.g. /pub/fatman///health, like so:
curl -X 'GET' \   '<dev url>/pub/job/<name>/<version>/health' \   -H 'accept: application/json'
which now fail with:
{"detail":"Not Found"}
I suspect this is due to the api endpoint no longer being hosted at /pub/fatman but only at /pub/job.
Should we add backwards compatibility here?

It definitely should be backwards compatible, and I was under the impression that I had made sure it was. Thanks for the bug report, I'm on the case.

I'm confident that we kept the old endpoint for backward compatibility in PUB.

Ah, maybe python3 / mllab plugin is worth to check, cause maybe it's not an error raised by PUB, but the Job itself.

I just tested it locally with adder (via make up test) and it does not seem to work. pub/router.go lines 26-32 should add the endpoints though, so I'm not quite sure what's happening there.


Fix OpenAPI deprecated endpoints in Kubernetes 1.14

Deprecated endpoints are removed in Kubernetes 1.14.

/lgtm
/approve


Update sql-database-long-term-retention.md

added a couple of FAQ's around SQL

Hi @andyjballgit, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@andyjballgit - Hi, thanks for your contribution. We are in the process of refactoring the Azure technical articles, so we can’t merge this PR. When the refactoring is complete, we can give you a link to the new location of the article you updated so you can resubmit your update, if you want to.

@andyjballgit - Thanks for your contribution to the Azure docs. On November 16th, we announced the availability of our Azure technical documentation in a new location on docs.microsoft.com. You can read about it on our team blog. This change means the article source lives in a new GitHub repository. Because there are some changes in the source markdown as a result of the move, we can’t pull your contribution over to the new repo without creating a ton of merge conflicts. We invite you to resubmit the change (if you are willing) to the new version of the article at https://docs.microsoft.com/en-us/azure/sql-database/sql-database-long-term-retention. Thanks for your understanding.

Done ! ☺
From: Kristine Toliver<EMAIL_ADDRESS>Sent: 17 November 2016 05:44
To: Azure/azure-content<EMAIL_ADDRESS>Cc: Andy Ball<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-content] Update sql-database-long-term-retention.md (#7941)
@andyjballgithttps://github.com/andyjballgit - Thanks for your contribution to the Azure docs. On November 16th, we announced the availability of our Azure technical documentation in a new location on docs.microsoft.comhttps://docs.microsoft.com. You can read about it on our team bloghttps://docs.microsoft.com/teamblog/docs-november-update. This change means the article source lives in a new GitHub repository. Because there are some changes in the source markdown as a result of the move, we can’t pull your contribution over to the new repo without creating a ton of merge conflicts. We invite you to resubmit the change (if you are willing) to the new version of the article at https://docs.microsoft.com/en-us/azure/sql-database/sql-database-long-term-retention. Thanks for your understanding.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-content/pull/7941#issuecomment-261160789, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGTMu714AJaqLzG4jEuhWgEu3FuWLTaaks5q--mcgaJpZM4KwZYZ.


[FEATURE] Example showing disk + Amazon EKS encryption

Describe the solution you'd like
Would be possible to enhance (or add) one example to show disk + EKS encryption using KMS?
It should be quite easy for Amazon EKS encryption using cluster_kms_key_arn variable, but example for worker node disk encryption is probably not trivial, because it may require creation of launch template.
Anyway - any example for Amazon EKS + worker node disk encryption would be welcome...

Thanks @ruzickap for raising this issue. @ricardosouzamorais any updates on the PR?

Depends on https://github.com/aws/containers-roadmap/issues/185


Messages containing arrays don't get sent from roslibjs

As mentioned in https://github.com/v-kiniv/rws/issues/30#issuecomment-2360085645, I've gotten around to setting up a small test case for this odd issue. The process to reproduce is as follows:
git clone https://github.com/MoffKalast/vizanti.git -b ros2-rws-arraytest

Source, compile and all that. Then open three terminals with the following subscribers which each have a message type that contains some sort of array somewhere:
ros2 topic echo /path nav_msgs/msg/Path
ros2 topic echo /floatarray std_msgs/msg/Float32MultiArray
ros2 topic echo /posearray geometry_msgs/msg/PoseArray

Then run:
ros2 launch vizanti_server vizanti_server.launch.py

Which will launch vizanti with rosbridge, and upon opening http://localhost:5000, three test messages will be automatically sent to those topics, which should arrive fine in the terminals.
Then clear the subscriber terminals and run:
ros2 launch vizanti_server vizanti_rws.launch.py

The subscribers don't receive any messages, and RWS prints the following:
[rws_server-1] [WARN] [1728306011.007179379] [vizanti_rws_server]: Failed to parse JSON: [json.exception.type_error.305] cannot use operator[] with a string argument with array

Interestingly this problem is only one way, roslibjs can receive arrays just fine. I'm wondering if I'm doing something wrong on the publishing side tbh, since there are some cases (like getting header sec/secs wrong) that rosbridge handles that aren't in spec, but I'm not sure what else to try.
I've tested the above on both humble and jazzy with respective branches of RWS, with Cyclone. The /path topic is set up as latched in roslibjs so it should be received again any time a subscriber connects, but alas that detail doesn't seem to work with rosbridge either.

Thanks for the prepared repo, saved me a lot of time.
When I found the bug, I had doubts if I had made such a blunt mistake or if I forgotten why I needed to prepend array index with the field name, and removing it would cause other problems:
index 2671a2c..9d2dc7f 100644
--- a/src/translate.cpp
+++ b/src/translate.cpp
@@ -263,7 +263,7 @@ static void json_to_serialized_message(cycser & ser, const MessageMembers * memb
           }

           for (size_t index = 0; index < array_size; ++index) {
-            json_to_serialized_message(ser, sub_members, field[member->name_][index]);
+            json_to_serialized_message(ser, sub_members, field[index]);
           }
         }
         break;

I tested the changes with my old project and nothing seems to be broken. But as always, please test and let me know if anything is broken on your end before we merge PR https://github.com/v-kiniv/rws/pull/38.

Well that was an easy fix 👍
I can confirm it working on jazzy with the test subscribers, I've also ran my set of test nodes for the rest and it all checks out:

I'll test on humble as well when it's backported.


MM-14117 Fix opengraph and reactions for older servers

Summary
Fixes backwards compatibility with openGraph and reactions
Ticket Link
https://mattermost.atlassian.net/browse/MM-14117

cherry-picked for 1.16 and 1.17


Go error

Just a heads up that starting today builds of the demo server started to fail for me. The error given matches the one referenced in Errors in rest.go #4
Anything I can be doing to help myself ?

./rest.go:366: not enough arguments in call to lxdDaemon.LocalCopy

Ah yeah, likely the client needs to be updated against master since some client functions gained additional arguments.

Aha. I thought something was up when i had multiple mentions of it in my inbox. The demo is included as a sandbox tool in Shark Distro. I have an installer release going out today so Im gonna axe it for now and look at it for re-inclusion on the next ISO build

@Shark-Linux, not that client api breaks will be way less frequent once @stgraber's client rewrite lands. This will enable us to give a lot stronger client api guarantees. :)

Fixed, and yeah, as soon as I land the first pass on the new client library, I'll port lxd-demo-server over to it which should make things much more stable going forward :)

Devine Intervention!
Best of luck with the new client, Ill keep an eye out for updates.
Cheers,


Added the ability to explicitly set type names and some QoS settings

ROS types can now be specified explicitly in either direction

When type is specified explicitly the relavent publisher/subscriber is made as soon as possible, rather waitng for an mqtt message or matching publication

Added explicit QoS settings

The default for publisher is system default

Added "auto" QoS support for subscriptions where subscriptions find a matching publication and use their QoS

Resolves: https://github.com/ika-rwth-aachen/mqtt_client/issues/48

There were a lot of combinations of explicit/deduced types and explicit/auto QoS in each direction tested. Please let me know what kind of evidence you would like me to provide.  Its gonna end up being a lot of screenshots if you want all tested combinations.

I should also be forward that there is a ninja nerf in here that is kinda ugly.
Whenever we are dealing with fixed_types I changed the encodings of bools in the primitive mqtt messages to be "0" or "1" instead of "true" or "false".
In my particular use case this is because this is how a device I am communicating with encodes those things and I couldn't figure out a clean way to make either or supported without a bunch of typing.

i do wonder if a future pass through this code would do better to refactor the code into 2 separate nodes, perhaps utilizing a common library. One seems to be focused more on using mqtt as some sort of intermediary transport between two ros systems (eg, ros <-> mqtt <-> ros) vs the other which acting as an adapter between mqtt and ros.
Not, sure exactly but when I look at https://github.com/ika-rwth-aachen/mqtt_client/pull/51 I am thinking that some sort of biggish refactor would be desirable sometime soon

Thank you very much, your changes are very welcome! Unfortunately it took me a while to find time to review. I have left a few comments that you could address.
Regarding testing: you said that you had basically tested a lot of QoS combinations? I don't require proof for all possible combinations, but it would be great if you could reaffirm that you are confident with the changes. :)
If a particular setup is not working, someone will eventually run into that problem and can then file a bug report.

If you'd like me to provide some sort of OQE (objective quality evidence), I can capture some screenshots or termincal captures of what I  have done. But the basic methodology for testing all these combinations has been to modify one of the example configurations (mainly params.ros2.primitive-fixed-type-and-qos.yaml) and then run something like:
ros2 topic pub /ping/primitive std_msgs/msg/Float32 'data: -3.14' --qos-reliability reliable --qos-durability volatile
to test the auto discovery, and generally adding in ros2 topic echo /pong/primitive to test the end to end system.
Auto QoS was verified using the command line tool to very the subcription qos matched my expectation. Eg:

I did actually have a bug in there with the explicit QoS on the ros subscription side. I did rerun those tests with a couple of QoS variations and virified those using the cli.

Looks good from my side and I believe that you have performed thorough testing! I checked a couple minimal setups myself.
I'm ready to merge, please confirm that you are finished with the PR as well. :)
Again, thank you very much! Merging this in now would allow me to directly release it for the new ROS 2 Jazzy.

I'm happy if you're happy. The only comment not covered was that one place you asked for an error message, but that is relatively minor.
Merge away.


[Snyk] Upgrade @tokens-studio/tokens from 0.0.24 to 0.1.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @tokens-studio/tokens from 0.0.24 to 0.1.1.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:sparkles: Snyk has automatically assigned this pull request, set who gets assigned.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 23 days ago, on 2024-03-28.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
👩💻 Set who automatically gets assigned
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

https://github.com/tokens-studio/style-dictionary-configurator/pull/103


setup.py install/build fails when requests is not installed

setup.py imports pyes, which imports requests.  This causes setup.py to exit prematurely and be unable to define requests as a dependency.
Please consider adding your meta to a different module package with no dependencies of its own, or use something like I do here https://github.com/ex-nerd/prep/blob/master/setup.py (IIRC a technique mostly borrowed from Celery) to read the config info straight from the .py file.

ok, I'll do


Expected to find a single "csproj" file in folder

Repro steps:

used CLI to do func init Office365Functions
cd Office365Functions
code .
when code started got the prompt for "Detected an Azure Functions Project folder, that may have been created outside of VS Code. Initialize for optimal use with VS Code?"
hit Yes
from the command bar drop down selected c#

Action: azureFunctions.validateFunctionProjects
Error type: Error
Error Message: Expected to find a single "csproj" file in folder "Office365Functions", but found zero or multiple instead.
Version: Unknown
OS: win32

Thanks for reporting. As a workaround, you can create a C# project directly in VS Code: https://github.com/Microsoft/vscode-azurefunctions#create-new-project
That being said, what would be your ideal experience for the scenario above? We check the csproj file for the targetFrameworkVersion, which affects your experience in VS Code (for example, creating an HttpTrigger for .NET Core is different than creating one for .NET Framework).
For example, would you like us to create a csproj file for you? AFAIK - the func cli only supports C# Scripts, so you would never get a csproj file from func init or func new

Closing due to lack of response. Feel free to reopen with more info


Incorrect typespec

@lennartschoch
https://github.com/Bounceapp/elixir-improvmx/blob/816db639253c05bcee61fe29ca127df83889887c/lib/improvmx/domains.ex#L25
Since the underlying request function raises, the typespec for ImprovMX.Domains.add should have a return type of no_return() instead of {:error, term()}

Alternatively, we can allow this to return {:error, error} and handle it in the project code.

@StareIntoTheBeard Oh, right! I actually don't know if I like that the client is raising an error... I'll take a look, thanks for the heads-up :)

Sorry @StareIntoTheBeard, gotta admit I kind of forgot about this. Fixed it here.

Looks great!


CHAT_CONVERSATIONAL_REACT_AGENT never uses more than 1 tool per turn.

Issue you'd like to raise.
The effect of tools with return_direct=False for CHAT_CONVERSATIONAL_REACT_AGENT only causes the LLM to generate a final answer based on tool observation, but never in another tool invocation.
The same is not happening with CONVERSATIONAL_REACT_AGENT which seems able to generate new tool queries after the first one.
Is this a feature that can be fixed simply acting on the agent policy prompt, or is there a better way to enable such feature?
Thank you very much.
Suggestion:
No response

@dosu-bot but then why CONVERSATIONAL_REACT_AGENT  can use more tools per turn? They both inherit from BaseSingleActionAgent.


Sorted datatable

Hi @Asaf95
I found another way to get a runtime error.  Try adding more than 9 different resources.  (it runs out of unique colors for the index)  How would you like to handle that?
I think a user would be surprised to see the table automatically reload when they delete all the lines. It may be better to add a default row instead.  If you would like a more obvious way to reload the table than refreshing the browser, perhaps add a "refresh" button?
For button, let's use dbc.Button rather than html.Button .
If you make the browser window narrow, then the table doesn't stay in the container.   One way to fix this is by adding:
style_table={"overflowX": "auto"},

With Bootstrap, it's a best practice to use a dbc.Container as the outter container rather than an html.Div.   More info here. So please take out all the new left and right margins and start with this:
app.layout =  dbc.Container(
    [
       ....
    ],
    fluid=True
)

The conditional formatting doesn't work in the table.  The idea is to make the Finish column with a grey background to show it's disabled.  Try:
style_data_conditional=[
    {
        "if": {
            "column_id": "Finish",
        },
        "backgroundColor": "#eee",
    },
],
style_header_conditional=[
    {
        "if": {
            "column_id": "Finish",
        },
        "backgroundColor": "#eee",
    },
]

Still TODO:  Comments and Style
Just a heads up, for the Example Index we want to use minimal comments and minimal style, but we can get to that last.

Great suggestions @AnnMarieW.
I like the suggestion of adding a default row when all rows are deleted. There are a lot of good things to learn in the app. So I think adding a default row would be a simpler better way to move forward than a refresh button in this case.

Hi @Asaf95
You can fix the bug by initializing the data in the table:
data=__get_default_table().to_dict("records"),

And set prevent_intial_callbacks:
app = Dash(__name__, external_stylesheets=[dbc.themes.SPACELAB], suppress_callback_exceptions=True, prevent_initial_callbacks=True)

Then you may not need to check for the None condition in the callback :-)
I like how you changed the colors=px.colors.qualitative.Alphabet,  however, now you will get a runtime error if you have more than 26 different Resources. :cry:
A couple option - maybe a try except block to catch the "PlotlyError" and display a nice message to the screen.  Or you can limit the choices of Resources by using dropdowns in the DataTable https://dash.plotly.com/datatable/dropdowns
The error message may be the most flexable, then people can change the name of the Resources in the table. - But either one would be good examples.

Yeah, I have a couple tricks up my sleeve still.  I think we can call this substantially complete.  What you have looks great. :100:
Tomorrow, I'll do a version with a couple workaround for the dropdown. issue.  I'll also do some style updates -- only so it's consistent with other apps in the Example Index.  (Not because there is anything wrong with the style in your version)
I think this is awesome - nice work! :tada:

One more update is needed.  I noticed that the figure factory get_gantt() function is deprecated and it's recommended to use px.timeline.  Would you like to try updating the figure?
https://plotly.com/python/gantt/

Check out this Gist:  https://gist.github.com/AnnMarieW/5fdd10cf2f0ff4dbd7b8b1300ac232ba
Try putting the content of the pages-timeline-gantt.py in a file called timeline-gantt.py in the pages folder: https://github.com/AnnMarieW/dash-app-gallery/tree/main/pages.  Then run app.py from the root directory. You can find your app near the bottom (it doesn't have an image yet)  You will see that it's now using the css from the assets folder.
Next, you can update  the app in the examples folder with timeline-gantt.py from the Gist.
Here are some of the updates:
I think it's a bug that the style_table={'overflowX': 'auto'},  isn't compatible with the dropdowns.  I'm sure there is a solution with CSS, but I haven't figure it out (yet). A workaround is to use other ways to control the table width.  This version uses shorter column headings -so now the entire table fits in the smallest browser window.
When you delete a Resource using the dropdown, it caused a runtime error.  I added clearable: False to the dropdown to fix this.
I moved the add rows button to the top of the table.  I think it looks better on the bottom where you had it, but it's a little annoying to chase it around when you add/delete rows and the table changes size. :-)
I removed the background color from the table to make the disabled column more visible, and from the background of the app so it would be easier to use with different themes.
The ids are now updated for production.  All ids in the Example Index need to be unique, so we use a prefix of "-x-"
The other changes are just to make the coding  and comment style similar to other apps in the dash docs and the Example Index.  For example, I kept the comments that explained the "why" and removed the comments that just described what the code does. I think the code is really clear - I especially like where you made separate functions for updating the table and the chart in the callback.
Let me know if you would like to make any other changes - otherwise, I think it's ready for a final review from Adam. :confetti_ball:
Still TODO - update the figure to px.Timeline.  Adam - can we update the csv file so it's not 2016?

Nice!  However, it looks like it doesn't sort properly with px.timeline :cry:
We are looking into how to make the sort work.   Worst case, we can move back to the Figure Factory function.

Adam found the answer :trophy:
Try this:
gantt_fig.update_yaxes(categoryorder="array", categoryarray=updated_table_as_df["Task"])

Super! :confetti_ball:
@Coding-with-Adam This app is ready for final review

Thanks for letting me know, @AnnMarieW
And thank you for this @Asaf95 .
I should have time to review this app in several hours. I'll keep you both posted.

@Asaf95 , Adam is off for a bit,  so  I'll pass along the rest of his comments and help get this app ready to merge.

The dropdown works in production, but it should also work if you copy the code from the Example Index and run it locally. There are two possible solutions - one is to add comments on how to add the CSS to an assets folder.  The other  (simpler way) is to add to the css prop in the table.  For example:

css=[
    {"selector": ".Select-value", "rule": "padding-right: 22px"}, # makes space for the dropdown caret
    {"selector": ".dropdown", "rule": "position: static"} # makes dropdown visible
]

Let's use the naming convention of dff for the filtered or modified df.  So  add_finish(dff)  rather than add_finish(df)

Can remove this line since it's unused:

style_table=DATA_TABLE_STYLE.get("style_table"),

The Example Index is designed for beginners, and this app has a more advanced coding style.  Here are a couple suggestions to simplify:

Many beginners are not familiar with function annotation, so it would be better to remove these. This would also make it consistent with other apps in the Dash docs and  Example Index.

The dropdown in the table could be simplified to use fewer resources - then the resource_map could  be removed.

dropdown={
    "Resource": {
        "clearable": False,
        "options": [
            {"label": i, "value": i} for i in ["A", "B", "C", "D"]
        ],
    },
}

Thanks for the update.
Did you miss the this comment?  What do you think about removing the function annotations:

Many beginners are not familiar with function annotation, so it would be better to remove these. This would also make it consistent with other apps in the Dash docs and Example Index.

I like the DATA_TABLE_STYLE name - and it's a good idea to add the css style there as well.  If you want to change a name, maybe DATA_TABLE_COLUMNS instead of DATA_TABLE_SCHEMA ?  but I don't think that's too important.
The other comments, I'll make in-line

Looks Awesome!  Thanks for all your hard work :1st_place_medal:
I'll merge it and add the files necessary for production.  Should be live soon after Adam is back.
:woman_dancing:

🥳  Congrats @Asaf95. Your app is officially online: https://dash-example-index.herokuapp.com/timeline-gantt
Thank you for your support, @AnnMarieW .


"azure appservice [plan|web]" commands return no data

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: pip (python 3.6.0)
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here: 2.0.2
OS Version: What OS and version are you using?
Answer here: Windows 10
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here: cmd.exe

Description
The command az appservice plan list returns an empty json array, even though I have App Services visible in the portal.  Similar problem with az appservice web list.  az appservice plan show -g MyGroup -n myname returns nothing at all, but typing an invalid name returns an error.
I successfully created a plan via the command line, but it does not appear in the list (though it appears in the portal):
> az appservice plan create -n TestCLI -g MyResourceGroup
 => {
  "adminSiteName": null,
  "appServicePlanName": "TestCLI",
  "geoRegion": "South Central US",
  "hostingEnvironmentProfile": null,
  ...
}
> az appservice plan list -g MyResourceGroup
=> []

However it is listed in azure appserviceplan list --resource-group MyResourceGroup
Also, az webservice web list returns nothing.  Not sure if that's related.

The difference between CLI 1.0 and 2.0 is the api-version. CLI 1.0, azure, uses an older version. Do you mind running them in verbose mode and send the output to me yugangw at microsoft.com
azure appserviceplan list --resource-group MyResourceGroup -vv
az appservice plan list -g MyResourceGroup --debug

I sent you the output, thanks!
BTW, the az commands also do not work on Linux with python 2.7.

BTW, the az commands also do not work for me on Linux with python 2.7.

I am using 2.7 on linux all the time and it works. Could you share a bit error details?

@yugangw-msft the az output looks to be the same on linux.

@mikebridge, got it. I misunderstood you meant a more generic issue on linux. And yep, this is expected for the same command failing in linux as this is a service side issue, which I will forward to webapp team.

I have the same issue. Repro script here below.
az group create --name GroupCentralUS --location centralus
gitrepo=https://github.com/Azure-Samples/app-service-web-dotnet-get-started.git
az appservice plan create --name MyWebAppCentralUS --resource-group GroupCentralUS --sku FREE
az appservice web create --name MyWebAppCentralUS --resource-group GroupCentralUS --plan MyWebAppCentralUS
az appservice web source-control config --name MyWebAppCentralUS --resource-group GroupCentralUS --repo-url $gitrepo --branch master --manual-integration
show the website (returns nothing)
az appservice web list --resource-group GroupCentralUS

@markjbrown , I tried your repro, still no repro :-(

fixed by #2758


Register kaist-server.is-not-a.dev

Added kaist-server.is-not-a.dev using the CLI.

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA

recheck


Import everything from the parent scope in derives

This removes the warnings without breaking anything that wasn't broken before. Turns out that extern crate in a function scope has some strange behaviors, so I'm not going to do the function change just yet. I will change #111 not be about the hygiene warning, but just un-breaking cases where #[derive(...)] is already broken.

bors r+


Use Bluebird

...unless this project is specifically a test case for native async/await.
Bluebird's implementations are still faster & more memory efficient than Node v7's version. Relying on Bluebird would also open the door to more Node versions.

Thanks for your effort in creating this issue!

...unless this project is specifically a test case for native async/await.

But this is correct. I want to help the node/v8 community find and fix bugs in the native implementation of async/await. Also it makes the project's structure much clear! We're already transpiling most of our other packages and I noticed it creates a lot of overhead.

I want to help the node/v8 community find and fix bugs in the native implementation of async/await.

I can get behind that! :)
(There wouldn't be any compilation steps fyi; could run as it.)


Denormalized data: optimize joins with a union of class definitions

Common when multiple class mapping entries involving the same IRI template and the same class exist, due to denormalized data.
##Example
Data
CREATE TABLE "profileapp1" (
"id" INT NOT NULL PRIMARY KEY,
"name" VARCHAR(40)
);

CREATE TABLE "profileapp2" (
 "id" INT NOT NULL PRIMARY KEY,
  "name" VARCHAR(40)
);

INSERT INTO "profileapp1"
("id","name") VALUES
(1, 'John');

INSERT INTO "profileapp2"
("id","name") VALUES
(1, 'John'),
(2, 'Roger');

Mapping:
[PrefixDeclaration]
:		http://example.org/profiling/voc#

[MappingDeclaration] @collection [[
mappingId	profile app1
target		<http://example.com/profile/{id}> a :Profile ; :name {name} ; :app <http://example.com/app1> .
source		SELECT * FROM "profileapp1"

mappingId	profile app2
target		<http://example.com/profile/{id}> a :Profile ; :name {name} ; :app <http://example.com/app2> .
source		SELECT * FROM "profileapp2"

]]

SPARQL query:
PREFIX : <http://example.org/profiling/voc#>
SELECT * {
  ?p a :Profile ; :app <http://example.com/app1> .
}

After optimization, the resulting SQL query should contain no union and no distinct.

Implemented b https://github.com/ontop/ontop/commit/9e64a4f01d6dc3450d27aa1c40791b67c3485bb5


Clojars deployment

This looks a little less trivial than the other Clojars deployment PRs, just curious about the reasons for shuffling around the stuff in deps.edn (I'm sure there are good ones, just curious)

The gentests are dependent on Datasim, which itself relies on Pan. As you probably know, updating Pan on DS has been far more complicated than expected, so that process may still take a while.

It seems like it'll only be a matter of time before Datasim itself is updated, though, so I'll wait until that happens then revert these temporary changes.


dev

Type

[x] Bug Fix: add .js file extension to defaultMessageNotifications file
[ ] Functions: $pruneStatus[days;guildID;roleid:roleid:roleid...] get amount of members that can be pruned, all fields optional
$pruneMembers[days;guildID;reason;roleid:roleid:roleid...;returnCount(yes/no] prune members
$customEmoji now have guildID field
$nodeVersion check node.js version
[ ] Callbacks:
[ ] Handlers:
[ ] Others:

Dependencies (Third Party Modules) needed:
Want a credit? Discord tag or other social media link: blox#8880
Referenced Issue:
Description
i already explained things above xd

wait no wrong

wrong branch sry


chore(CategoryTheory/Bicategory): create Kan directory

I plan to put some files in this new directory.

bors merge


Has 'time to crack' gotten shorter with more GPUs?

The Ars Technica article
http://arstechnica.com/security/2013/05/how-crackers-make-minced-meat-out-of-your-passwords/2/
said that they quickly bruteforced all passwords of 6 or less, plus the all-upper or all-lower passwords of length 7 or 8 in just 41 seconds.
Yet the 'crack time (display):' for an all caps 7 char password is 5 years. I suspect more GPU power has made cracking faster.

Closing this out for now, but it's on my roadmap to remove password storage assumptions + cracking hardware assumptions from zxcvbn scoring, such that a hardware-independent guessability estimate is emitted instead. See #58 for more discussion.


[BUG?] Getting Error while installing plugin

Bug report
Error during processing of action! Attempting to revert...
Failed to install 'cordova-plugin-firebasex': CordovaError: Uh oh!
File already exists at destination "/Users/{{WORKSPACE}}/app/platforms/ios/{{APP}}/Resources/GoogleService-Info.plist" for resource file specified by plugin cordova-plugin-firebasex in iOS platform
    at install (/Users/{{WORKSPACE}}//{{APP}}//app/platforms/ios/cordova/lib/plugman/pluginHandlers.js:64:23)
    at ActionStack.process (//app/node_modules/cordova-common/src/ActionStack.js:56:25)
    at PluginManager.doOperation (//app/node_modules/cordova-common/src/PluginManager.js:114:20)
    at PluginManager.addPlugin /app/node_modules/cordova-common/src/PluginManager.js:144:17)
    at Api.addPlugin (/app/platforms/ios/cordova/Api.js:245:10)
    at handleInstall (/.nvm/versions/node/v10.18.0/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:581:10)
    at
/.nvm/versions/node/v10.18.0/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:349:28
    at process._tickCallback (internal/process/next_tick.js:68:7)
for resource file specified by plugin cordova-plugin-firebasex in iOS platform

conifg.xml
    <plugin name="cordova-plugin-firebasex" spec="^9.1.1">
        <variable name="FIREBASE_ANALYTICS_COLLECTION_ENABLED" value="false" />
        <variable name="FIREBASE_PERFORMANCE_COLLECTION_ENABLED" value="false" />
        <variable name="FIREBASE_CRASHLYTICS_COLLECTION_ENABLED" value="false" />
    </plugin>

Expected behavior:
Plugin should install
Steps to reproduce:
cordova plugin add <plugin_name>
Screenshots

Environment information

Cordova CLI version

cordova -v

9.0.0<EMAIL_ADDRESS>
Cordova platform version

iOS

Dev machine OS and version, e.g.
OSX(Latest)

I tried removing that file and re-installing, won't work.

This could be caused by a conflict with another plugin or another issue specific to your project.
Please try building the example project to confirm the plugin builds correctly in isolation.

@dpa99c I don't know why it would.
Here is the list:
    <plugin name="cordova-plugin-whitelist" spec="~1.3.4" />
    <plugin name="cordova-plugin-ionic-keyboard" spec="~2.1.3" />
    <plugin name="cordova-plugin-camera" spec="^4.1.0" />
    <plugin name="cordova-plugin-camera-preview" spec="~0.11.2" />
    <plugin name="cordova-plugin-keyboard" spec="~1.2.0" />
    <plugin name="cordova-plugin-splashscreen" spec="~5.0.3" />
    <plugin name="cordova-plugin-statusbar" spec="~2.4.3" />
    <plugin name="cordova-plugin-customurlscheme" spec="~4.4.0">
        <variable name="URL_SCHEME" value="nutri" />
    </plugin>
    <plugin name="cordova-plugin-ionic-webview" spec="~4.1.3" />
    <plugin name="cordova-plugin-device" spec="^2.0.3" />
    <plugin name="cordova-plugin-safariviewcontroller" spec="~1.6.0" />
    <plugin name="cordova-plugin-enable-multidex" spec="~0.2.0" />
    <plugin name="cordova-plugin-firebasex" spec="^9.1.1" />

that's all I am using, to be honest, there are no other plugins that conflicts with Google Firebase.

Ok, I did some more digging around: Use instructions here:
https://stackoverflow.com/questions/51131825/how-to-reinstall-plugins-already-recoded-in-config-xml-in-cordova

Followed these steps:

Delete folder ios android inside platforms.
Delete All plugins under plugins folder
Run cordova prepare

@dpa99c remove all plugins and only kept this one, still ran into the issue.

Ok, I did some more digging around: Use instructions here:
https://stackoverflow.com/questions/51131825/how-to-reinstall-plugins-already-recoded-in-config-xml-in-cordova

Followed these steps:

Delete folder ios android inside platforms.
Delete All plugins under plugins folder
Run cordova prepare

@dpa99c Still keep this issue in mind.
I will revert to 9.1.0 and try again

I was unable to fix it, after trying a few times. It gave me an error to run pod install.
I give up. Maybe will try the alternative that doesn't do magical configuration behind the scenes.
https://github.com/chemerisuk/cordova-plugin-firebase-messaging#readme

@dpa99c This is one of the best plugins for firebase, I have been using this for a long time. Off late, installing this in ios has become a night mare. I get it going after multiple platform removes, 9.0.1-cli and what not. Can this be a little streamlined like before

@dpa99c Also build time takes 3x longer with this installed.

@theavijitsarkar this is open-source software: if you are not happy with it, submit a PR to make it better

@dpa99c I am extremely sorry for putting this out in a wrong way. I am a big fan of your work, and this is an awesome plugin. I wish someday I have the talent to be able to contribute. Totally respect that this is an open source software, just was pointing out to my difficulty. Thanks again for putting this out there.

Need to support this update
https://developer.android.com/distribute/play-policies
Thanks @dpa99c


[Feature Request] API to create an outline (table of contents) for PDF readers

This should allow users to display a visual outline that is linked to different parts of the pdf.

This was mentioned in issue #127 but I think the suggested solution only works with an earlier version of pdf-lib.
I’d love to find out how you can do this with the later version.

+1 for this

@leverarch-app, I have just posted in issue #127 my updated code for @Hopding's solution with the current version.

@achordia20 @leverarch-app
I created a module that has a high level api and uses pdf-lib to add outline to outline-less pdfs. You can find it here.

Added this to the roadmap for tracking: https://github.com/Hopding/pdf-lib/discussions/998.


Console output of the full query, with headers

Just changed the console output for a better understanding of WarpScript substitutions.
For example, multiple selection on a variable are translated in a string separated by + in WarpScript:

This is not obvious...

dist/ pushed.
little differences in .d.ts files, can't figure out why.

Strange, I cannot reproduce either with node docker image with version 10/12/13.
Maybe build instructions, I use
npm install
npm run-script build

It would be better to keep only one version of those files (even if it should work as it is). Will document the building process after that :wink:

I used npm install and yarn build after a git reset of dist/
Maybe little differences in the npm package version used.
Anyway, it should work.


What is the relationship between image resolution and minimum size

minsize = 60 , I can run 720P， but minsize = 40 ,I cannot run 720P,  thx

I think you can debug the error, or show the error

now, it has been solved, I added some codes to mtcnn.cpp.
if you set the minsize too small, the resize scale will be small too,
if the smallest scale *maxside of image<12, error occurred , because the Pnet input size is 12.


docs: Add a docstring example in narwhals.dtypes for Datetime

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[x] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #1504
Closes #1504

Checklist

[x] Code follows style guide (ruff)
[ ] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below

@MarcoGorelli can you help me out with the failing check? It doesn't seem to be an effect of some code I wrote :thinking: (I may be wrong)


incorrect information about galaxy

Issue Type

Bug report

Summary
https://github.com/ansible/molecule/blob/master/docs/getting-started.rst#creating-a-new-role
Molecule does not use ansible-galaxy to create the role structure. They're actually created by cookiecutter.

Right you are! I wouldn't be against reconsidering hard-coding a role structure there. We should probably be using ansible-galaxy to do this, no? That is probably for another ticket ...


Add example input that makes the test program crash when dereferencing a null pointer

Hi Angus, here's example input that makes the current program crash.

Thanks Juha. I'll fix shortly. Anyhow I've at least simplified the problem sample to this:
CAPTION: 2.
CLIPTYPE: UNION
FILLRULE: POSITIVE
SOL_AREA: -1
SOL_COUNT: -1
SUBJECTS
129418,19467 129422,19467 129422,19468 129427,19468 129420,19457
129410,19450 129420,19467 129422,19467 129422,19468 129428,19468 129439,19450
129422,19467 129425,19467 129425,19468 129425,19467 129427,19467 129428,19467
129423,19467 129425,19467 129425,19468 129425,19467 129427,19467 129428,19467

It seems I'll have to write an app that can quickly simplify problems.
I wrote one in delphi a while ago, but it didn't do a very thorough job.

I actually stumbled upon this issue when trying to find the union of more than 27,000 polygons.
Then I semi-programmatically trimmed the input down to just these 4.
At that point didn't anymore have the energy to further simplify the individual polygons. :)
(Also didn't think I had the required know-how for doing it efficiently.)

At any rate the point was that I have some code for doing that, too. Next time around, I might try and polish it so that it could be added to the repo, unless you have already done something similar by then.

Next time around, I might try and polish it so that it could be added to the repo, unless you have already done something similar by then.

I've just about finished a Delphi utility which trims paths almost to the bare minimum while still producing an error.
And I don't expect the translation would take very long. But before that I'll be fixing your reported bug 😁.


googleapiclient import error

In terminal:
mkdir test
cd test
python3 -m venv env
source env/bin/activate
pip install cirq

I then create the following python script (test_script.py):
import cirq

Running the script gives:
(env) dkafri@dkafri:~/Desktop/test$ python test_script.py
Traceback (most recent call last):
  File "test_script.py", line 1, in <module>
    import cirq
  File "/usr/local/google/home/dkafri/Desktop/test/env/lib/python3.7/site-packages/cirq/__init__.py", line 38, in <module>
    from cirq import (
  File "/usr/local/google/home/dkafri/Desktop/test/env/lib/python3.7/site-packages/cirq/google/__init__.py", line 42, in <module>
    from cirq.google.engine import (
  File "/usr/local/google/home/dkafri/Desktop/test/env/lib/python3.7/site-packages/cirq/google/engine/__init__.py", line 21, in <module>
    from cirq.google.engine.engine import (
  File "/usr/local/google/home/dkafri/Desktop/test/env/lib/python3.7/site-packages/cirq/google/engine/engine.py", line 38, in <module>
    from apiclient import discovery, http as apiclient_http
  File "/usr/local/google/home/dkafri/Desktop/test/env/lib/python3.7/site-packages/apiclient/__init__.py", line 22, in <module>
    __version__ = googleapiclient.__version__
AttributeError: module 'googleapiclient' has no attribute '__version__'

I am able to fix this by downgrading google-api-python-client:
pip install google-api-python-client==1.7.11

Seconded - but you only need to downgrade to google-api-python-client==1.8.0

What version of python3 are you using?
3.7.6

Looks like this is due to a new release of https://github.com/googleapis/google-api-python-client/issues/870.

This issue has already been fixed in the latest version.  You can either downgrade google-api-python-client as mentioned or install cirq-unstable (the latest version).  This will be fixed in the cirq stable version (pip install cirq) once we release a new version.

Assume this is now fixed in move to cirq version 0.8.0. Closing, please reopen if I'm wrong


listview & detailview for lists

idea: name folder equal to list, iterate over its child-files (each describing one child). Alternatively, iterate over child-folders, each describing one child with multiple files (f.e. values & images).
Optimal: allow both in a mixed way.
in the main values file, reference the Listname (=foldername) - or make this obsolete with autodetection (might require special folder for lists). Or name the list-folders in a special way ("list_xy" f.e.).

might also add a warning for content that is unused

-> currently impeded, because copy-pasting the files is also possible for smaller projects (which are the main driver behind this PoC)

might also add a warning for content that is unused

Moved this to a dedicated issue #12.

From #18:
When working with the list feature, it is annoying to copy-paste the same version of the index.html.template over all folders each time.
One solution for this would be to have a list,html.template and a element.html.template. The index.yamls could be moved one level up (to the same location as the aforementioned *.html.templates) when they are renamed to their previously containing folder name plus file-ending.

Naming scheme idea:
index.html.single.template
index.html.list.template
-> they are renamed/copied (depending on element type; dir==copy&rename&move to each folder, file==copy&rename for each file) after rendering them.

Added with this commit.

Also added {{ .Item }} and {{ .ItemPath }} for those single-view items right now