add multipart form upload file, swagger generate json error

Hi all:
I use this example https://github.com/swagger-api/swagger-core/tree/v1.3.12/samples/java-jersey2-guice,
add this method in PetResource.java:
    @POST
    @Path("/{petId}/uploadImage")
    @Consumes({MediaType.MULTIPART_FORM_DATA})
    @Produces({MediaType.APPLICATION_JSON})
    @ApiOperation(value = "uploads an image")
    public Response uploadFile(
            @ApiParam(value = "ID of pet to update", required = true) @PathParam("petId") Long petId,
            @ApiParam(value = "Additional data to pass to server") @FormDataParam("additionalMetadata") String testString,
            @ApiParam(value = "file to upload") @FormDataParam("file") InputStream inputStream,
            @ApiParam(value = "file detail") @FormDataParam("file") FormDataContentDisposition fileDetail) {
        return Response.status(200).entity("true").build();
    }

but the scanner generate json is :
...
{
    "path": "/pet/{petId}/uploadImage",
    "operations": [
        {
            "method": "POST",
            "summary": "uploads an image",
            "notes": "",
            "type": "void",
            "nickname": "uploadFile",
            "produces": [
                "application/json"
            ],
            "consumes": [
                "multipart/form-data"
            ],
            "parameters": [
                {
                    "name": "petId",
                    "description": "ID of pet to update",
                    "required": true,
                    "type": "integer",
                    "format": "int64",
                    "paramType": "path",
                    "allowMultiple": false
                },
                {
                    "name": "body",
                    "description": "file detail",
                    "required": false,
                    "type": "FormDataContentDisposition",
                    "paramType": "body",
                    "allowMultiple": false
                }
            ]
        }
    ]
}
...

some thing not work, and miss some parameters.
please help me.

Any reason you're using 1.3.12?

Yes, I am using com.wordnik:swagger-jersey2-jaxrs_2.10:1.3.12

Yes, but why do you use that version and not the latest?

I'm sorry. I will try the latest right now.

Thanks, please keep us posted.

I can not find the example of swagger-jersey2-guice in the latest version, can you give it to me. Thank you

Samples have migrated - https://github.com/swagger-api/swagger-samples/tree/master/java/java-jersey2-guice

I have try this project last week.
When I add the same method in PetResource.java.
it will return JSON is :
...
{
"post": {
"tags": [
"pet"
],
"summary": "uploads an image",
"description": "",
"operationId": "uploadFile",
"consumes": [
"multipart/form-data"
],
"produces": [
"application/json"
],
"parameters": [
{
"name": "petId",
"in": "path",
"description": "ID of pet to update",
"required": true,
"type": "integer",
"format": "int64"
},
{
"in": "body",
"name": "body",
"description": "Additional data to pass to server",
"required": false,
"schema": {
"type": "string"
}
},
{
"name": "file",
"in": "formData",
"description": "file to upload",
"required": false,
"type": "file"
},
{
"in": "body",
"name": "body",
"description": "file detail",
"required": false,
"schema": {
"$ref": "#/definitions/FormDataContentDisposition"
}
}
],
"responses": {
"default": {
"description": "successful operation"
}
}
}
}
...
But the FormDataContentDisposition be generated in JSON.
Can you teach me how to deal with it?

Did you update to the latest version?

Yes, I download this example project, then add those code and run it.

The samples are not updated with the latest version. Make sure to update the dependency version, as there was a related bug that was solved in a recent version.

OK, thanks

Hi webron:
it does not work, the FormDataContentDisposition still in  JSON.
I only add register(MultiPartFeature.class). in Application.java
Need I add other configuration？

You're using Jersey 2, right?

Yes， Jersey's version is 2.21

Okay, we'll have to look into it.

Thank you very much.

Can you share your dependencies?

I have push the code to https://github.com/wangzilong/java-jersey2-guice.git
pom.xml : https://github.com/wangzilong/java-jersey2-guice/blob/master/pom.xml

Thanks for fixing this bug so quickly.
I am a git newer, can you tell me which branch can get the new code?

Hi @wangzilong, I think that the changes are still not integrated.
By the way, we are dealing with a similar situation in our solution (we are using Apache CXF but I think is similar) and our solution is to use @ApiImplicitParam to generate the swagger docs and "hide" the @ApiParam by using a custom SwaggerSpecFilter. Here [1] you can find an example (is scala but I think is easy to understand).
Hope this can help you,
[1] http://stackoverflow.com/questions/22812365/how-to-hide-a-parameter-in-swagger

@agusgr - by the way, in general, if you're looking to hide a parameter entirely, you can now use @ApiParam#hidden() to set it.

can someone post here a json file with an upload-file that i can use in swagger-codegen ?
Thanks


Pull request for command line option to select location of debug-logging file

---Please read and delete this text before creating this pull request---
Thanks a lot for the pull request you are going to do!
Please verify that you validate these points:

[x] indentation is made with 4 spaces
[x] your pull request contains only changes intended (no refactoring, line return added, ... that could make code review harder. Make another one if you think it is needed...)
[ ] run and verify that existing tests pass. Add some new units tests, if needed.
[ ] update the documentation, if needed.
[ ] update the release notes file

Added pull request per comment in #1169

I've made the changes requested, but not sure how to get them into the same pull request.


Really more of a question than an actual issue

This is sweet.  How difficult would it be for an amateur coder somewhere between beginner / intermediate to port this so that it works for Windows? Thanks

This project doesn't work anymore, as each of these marketplaces have drastically changed their listing processes since I wrote it.
But looking back at the project, even if it did work it's not a great way to solve the problem. I would recommend looking into something like Puppeteer, which will work on Windows out of the box.

Thanks for this


Authenticate API endpoints with supertoken session

Right now most endpoints are not requiring any authentication. Also, should not expect userId to come in the body / query of the request, but to be inferred after the authentication .

This is almost ready, I am just trying to fix the tests and having a lot of hard time trying to reconcile rewire with typescript, not being able to mock the authentication in the tests. @alcipir is helping me, we will see if maybe we can work on another way to solve it.


[BEAM-5327] Add support for custom dataflow worker jar in Go

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.


Change Table::ReadRows to work with StatusOr.

The google::cloud::bigtable::Table::ReadRows() function return a bigtable::RowReader.  This class has begin() and end() member functions that return an iterator of bigtable::Row.
The iterator (which is of type google::cloud::RowReader::iterator, an alias for google::cloud::bigtable::internal::RowReaderIterator), raises an exception (which can be disabled, but that is not the case we are worried about here).  Instead of raising exceptions, it should become an iterator over StatusOr<Row>.
At least, I think that is how we will fix this class to support StatusOr, I would like @devjgm to confirm.

LGTM

Greg and I already started the discussion on the iterator in the Spanner DD. Let me move it here.
TL;DR;
I'm not a big fan of iterator over StatusOr<> because we're introducing another iterator state (in case of an error we get an iterator which is not end() but indicates and end of iteration). That disallows using such iterators in standard algorithms. Instead I propose a facade, which binds the status:
Status status;
for (auto& row : table.ReadRows(...).iter(status)) {
   // handle rows
}
if (!status.ok()) {
  // oh no stream was broken...
}

Alternatives
I agree it's a bit awkward, but TBH, I thought it's the least bad idea I considered. Let me document my thought process:
Idea 1
You'd have to check status on result_set after you've finished iterating and it would tell you about an error condition while receiving the stream.
Snippet
auto row_reader = table.ReadRows(...);
for (auto& row : row_reader) {
   // handle rows
}
if (!row_reader.stream_status().ok()) {
  // oh no stream was broken
}

Pros:

range-based loops work
not very tedious to use

Cons:

user might forget to check the status at the end

Idea 2:
Rather than implementing a typical C++ iterator, create one which has HasNext() / Next() member functions (like Java iterators). HasNext() could return a Status.
Snippet
auto it = table.ReadRows(...).iterator()
StatusOr<bool> has_next;
while (has_next = it.HasNext()).ok() && *has_next) {
   auto& row = it.Next();
   // handle rows
}
if (!has_next) {
  // oh no stream was broken, handle has_next.status()
}

Pros:

pretty straight forward
hard to forget about the status

Cons:

very non-idiomatic
range-based loops no longer work

Idea 3: (proposal from the description):
Make the iterator return StatusOr<>
Snippet
for (auto& row : table.ReadRows(...)) {
   if (!row) {
     // oh no stream was broken, handle row.status()
     break;
   }
   // handle rows
}

Pros:

impossible to miss the stream status
range-based loops almost work (you need to manually break them)

Cons:

weird semantics - we'd introduce a third state of an iterator (beyond valid an invalid): it's fine to dereference it, but doesn't hold value; it's not end(), but indicates an end
you wouldn't be able to do a std::copy or any other standard algorithm

Idea 4 (AKA my proposal):
Create a facade around ResultSet, which binds the status, so that if the stream is broken, the iterators advances to end() and the bound status is set accordingly.
Snippet
Status status;
for (auto& row : table.ReadRows(...).iter(status)) {
   // handle rows
}
if (!status.ok()) {
  // oh no stream was broken...
}

Pros:

almost impossible to miss the status
as idiomatic as it gets with StatusOr
all the assumptions around iterators still work, you can use std::copy and company.

Cons:

it's a bit unusual to check errors that way

What do you think?

I am not a fan of iterators of StatusOr<> either, but FWIW, we considered all these options (or slight variations of them) when designing the GCS iterators. We picked Idea 3 after considering them.
@devjgm: we should write that discussion as a design doc or ADR, we should not be rehashing it on every iteration API.
I think we all agree that iterators with StatusOr<> are never going to be as clean as iterators with exceptions (but we cannot do that), and non-iterator APIs are worse.
If it helps, do not think of StatusOr<> as an error for the iterator, the iteration happens to stop on the first error, but that is part of the range.  Just like if you had a collection of std::optional<T> and it just happened that the last optional had no value :unamused:
Alternatively: it is not hard to write a wrapper that converts the iterator of StatusOr<T> to an iterator that raises on the first non-ok  StatusOr<>.
In any case, my comment is not binding here, we should wait for  @devjgm to chime in.

Thanks for your detailed write up @dopiera. While StatusOr certainly does make things a bit more clumsy, Idea 3 is the one that I prefer. I agree with @ocryan's explanation about an alternative way to think about an iterator of StatusOr that makes it, IMO, not so bad. Idea 3 also fits well with all the other cases where we return Status or StatusOr, i.e., we never (I think never?) return them via an out-param.

Thanks, I'll move on with option 3. I wasn't aware we already did that in GCS.


Add CSV Decoder for realtime

We only have JSON and Avro. Add support for CSV

@npawar Can take this up, working on a very similar feature.

Thanks Prashant!

PR: https://github.com/apache/pinot/pull/8658

❤ thanks everyone :)


How to save image to gallery iOS

I have image as byte array,  how to save it to gallery iOS?? help please

Hi @BoikoIlya!
By using the Save File Picker UI, you can do it like explain in the documentation:
val image: ByteArray

// FileKit Core
val file = FileKit.saveFile(
    baseName = "image",
    extension = "png",
    initialDirectory = "/custom/initial/path",
    bytes = image
)

// FileKit Compose
val launcher = rememberFileSaverLauncher() { file ->
    // Handle the saved file
}
launcher.launch(
    baseName = "image",
    extension = "png",
    initialDirectory = "/custom/initial/path",
    bytes = image
)

And you need to find a way to get the url of the gallery in iOS that you can pass it to initialDirectory parameter.
If you want to save direct bytes to a file without using picker UI, you'll need to code this yourself as it's not supported yet with FileKit.

this may help: https://github.com/fethij/Rijksmuseum/blob/main/feature/arts/src/iosMain/kotlin/Utils.ios.kt#L47


I want to render code how to write (code highlights), I have seen for a long time or no idea.

Steps to reproduce the problem (provide example input):
render the code like java
Expected behavior:
hightlights
Actual behavior:
no hightlights

Hey,
I have a simplified clone of PrismJs for Java: prism4j. It's not sophisticated and doesn't have theming built-in, but it will allow you to tokenize a supported syntax and apply styling for your needs. I use it in my Markwon library (syntax-highlight module) to provide syntax highlighting to Android applications.
PrismJs is RegEx based so do not expect something too precise, but it does help with general highlighting. If you have any questions you can create an issue there and I will try to answer it.

@qiudaozhang I'm not exactly sure what you want. But if it's syntax highlighting, see @noties pointers.
commonmark-java provides the functionality to provide custom rendering of code blocks, so you'd want to use that together with a syntax highlighting library.
Providing syntax highlighting in commonmark-java itself is not in the scope of this project.

thanks  , i use js have resolve it.

Hi @qiudaozhang how did you fixed with js?
Client side on browser?

You may have a look for highlight.js or some java libs


Example customization of the ClickToAct event

Documentation says: You can also write custom controls to raise the Channel Integration ClickToAct window event and pass any eventData as the business requirement
Could you provide an example or some best practices on how to customize and raise the ClickToAct event?

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

ID: 720f0643-164e-5bb2-6f24-e951235a5c6d
Version Independent ID: 449f179c-c9e0-a03c-d667-d0b9ad37c76d
Content: onclicktoact event (JavaScript API Reference) for Microsoft Dynamics 365 Channel Integration Framework (CIF) version 2.0
Content Source: ce/customer-service/channel-integration-framework/v2/reference/events/onclicktoact.md
Service: dynamics-365-customerservice
Sub-service: channel-integration-framework
GitHub Login: @mh-jaya
Microsoft Alias: v-jmh

Found a similar question/issue on the same documentation page but for CIF version 1.0: #1790
The reply from @susikka here https://github.com/MicrosoftDocs/dynamics-365-customer-engagement/issues/1790#issuecomment-737653715 refers to an example for the CIF provider code, which is receiving the ClickToAct event, what I'm looking for is an example on how to dispatch/raise the OnClickToAct event from let's say a Dynamics form.

Hello,
You can create a custom control. More information is available here: Power Apps component framework API reference
Here's a TypeScript sample code snippet for raising a ClickToAct event:
const data: any = {
"entityLogicalName": "someEntity",
"entityId": "{some_guid}",
"key_1": "val_1",
"key_2": "val_2"
};
        const event = new CustomEvent("CIClickToAct", { detail: data });
        window.dispatchEvent(event);

Hope this helps. For any further questions, request you to please contact product support.  To contact Support, consider one of the options described here: https://docs.microsoft.com/dynamics365/get-started/support/
Closing this issue.


Modify schema validation to allow additional properties

The issue seems to be due to the additionalProperties  being set to false in values.json.schema
https://github.com/cert-manager/istio-csr/blob/main/deploy/charts/istio-csr/values.schema.json#L4C8-L4C28
This results in not allowing additional properties being set in the values.
https://json-schema.org/understanding-json-schema/reference/object#additionalproperties
Note - Setting the additionalProperties schema to false means no additional properties will be allowed.
By having defined the additionalProperties setting to false in the chart, essentially telling Helm - or really any JSON schema validator that it's not allowed to have any additional properties defined besides those defined in their JSON schema.
Here is an example of what helm returns when we try and set an additional value:
Helm install failed for release cert-manager/csr with chart<EMAIL_ADDRESS>values don't meet the specifications of the schema(s) in the following chart(s):
      cert-manager-istio-csr:
      - (root): Additional property domain is not allowed
      - (root): Additional property bigbang is not allowed

@volvera10 could you give an example of when this would be useful? What additional values do you want to set?

We intentionally set additionalProperties: false to prevent typos.
Please reopen the issue if you have an example of where adding an unknown field to values.yaml would be useful.


[ECard Transactions] [Polish] Update Generic Receipts to use ReceiptBackground

Design Doc Section
https://docs.google.com/document/d/1iELOXPOAcnuIPZJ3XbO1S8_3qHaM31Ptf8dR4LRZMyE/edit#bookmark=kix.xnc4hi7wlvbg
Overview

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d3d6d9c663db1e73
Upwork Job ID:<PHONE_NUMBER>807113216
Last Price Increase: 2023-10-04

not overdue, this is just clean up and can be de-prioritized for now

What is it being de-priotised in favour of? Should we not finish it off?
If it should be de-prioritised @grgia please can you update the labels to reflect the cadence it should be worked on? e.g. change this to Monthly / put it on "hold"

@zanyrenney weekly is the correct label

any further update here @grgia ?

Adding monthly for now

Unassigning for now to free up. I don't think this is worth refactoring right now

shall we put this on hold?

No, let's find someone who can take on! Posting in update about it!

@mollfpr feel free to ignore, we already have @narefyev91 on this one. Thanks!

yes!

@dangrous design doc is access denied, anyone give access?
mail<EMAIL_ADDRESS>

I updated the description with (I think) all the details that are needed!

Proposal
Please re-state the problem that we are trying to solve in this issue.
LOW: Transactions - Update Generic Receipts to use ReceiptBackground
What is the root cause of that problem?
New Feature
What changes do you think we should make in order to solve the problem?

Currently we are using ReceiptUtils.getThumbnailAndImageURIs to get receipt previews when there are pending waypoints in the transaction so it will return receipt source if there is no pending waypoints So to move to EReceiptThumbnail usage we need to create a component that displays the receipt when there is no pending waypoints and display the preview according to the file type using EReceiptThumbnail when there are pending waypoints.
For instance here in DistanceEReceipt
https://github.com/Expensify/App/blob/e9f27ab4218ec21ac6ecd755d4026b85c50f67d6/src/components/DistanceEReceipt.js#L70-L75
So we should do everywhere in the code base where we display the receipt previews like MoneyRequestView , MoneyRequestView etc...
What alternative solutions did you explore? (Optional)

@dangrous @narefyev91 is no longer C+. You need to reassign someone else.

@mollfpr feel free to ignore, we already have @narefyev91 on this one. Thanks!

Maybe assign @mollfpr back

Oh whoops, will do! Also proposers, please note, I just added one more criteria to the issue (adding a text label option to the component to allow for matching the old style images). Please let me know if you have any questions!

melvin is confused; @mollfpr you're acting as C+ here - thanks!

Updated Proposal to include all the requirements.

I'll finish the review in the morning!

hey @mollfpr how is the review going? Thanks!

Sorry for the delay! I think the proposal from @FitseTLT has most of the part the right to do it. Let's go with the proposal then!
🎀 👀 🎀 C+ reviewed!

Great, I think this makes sense - we'll have to get a full list of where receipt images show up in the app to make sure the new component functions well in all cases, but I think we can move forward with this implementation.

how are we getting on here @FitseTLT when do you expect the PR to be ready? cc @dangrous @mollfpr ?

Will create PR in 2 Days

Please can you link the PR when its created? It's been 2 days so assume we should get it today? @FitseTLT Thanks!

@zanyrenney Will create and link tomorrow. Thnx!!

Looking forward to seeing the PR today @FitseTLT !

Sorry guys for the delay, it didn't go as I planned, but hopefully will finalize it tomorrow 🙏

thanks for the update, look forward to the PR today then? @FitseTLT ?

Thanks @FitseTLT! Just a heads up, given that it's been a week since the initial assignment, if we can't get the PR into review by EOD today we'll likely have to look out for other options. Hopefully that won't be a problem but wanted to call it out.

@dangrous @grgia I want to clarify the expectation in this issue. We want to replace the current eReceipt thumbnail with a new component with an eReceipt background.
So, the result will remain the same as the current thumbnail.

Am I correct?
I'm asking for clarification because the PR from @FitseTLT currently has a different result.

Result from PR
Scan doc on Staging
Scan HTML on Staging

On the PR, it will showng the same thumbnail and without the extension label.

Hey @mollfpr thanks for asking! So yes, you are correct - basically, the changes made here should result in essentially no visible changes from the existing behavior, but it should use a dynamically created images. So it might be slightly different in terms of color, exact alignment, etc., but it should have the receipt image and the file type, just like the screenshots you have from staging.
(Correct me if I'm wrong @grgia)

Color and size of text ??? @dangrous

Color, it looks like it should be the darker of the two image colors (I believe that's defined as backgroundColor here
As for text size, that's a good question, for now let's see if you can match the size in the existing components (again, it should visually be almost exactly the same), and if @Expensify/design has any alternate ideas, they are welcome!

Interesting, how are these generic receipt images created? I kinda figured we would just give you a .svg or .png of that entire thumbnail image and thus you don't need to worry about adding dynamic text? I could be wrong though!

Ha so yeah they were previously just a non-dynamic image, but as part of this doc we're using the dynamic component - https://docs.google.com/document/d/1iELOXPOAcnuIPZJ3XbO1S8_3qHaM31Ptf8dR4LRZMyE/edit#bookmark=kix.xnc4hi7wlvbg

@FitseTLT Could you attach the screenshot here so the design team can review it? Thanks!

Here is how it looks after the label is added

That feels pretty good to me, curious if others agree though!

Yup—looks pretty solid to me too!

How are we looking on this one? Do we think we'll be able to final review and merge sometime this week? Thanks!

@dangrous I just finished reviewing and testing. It looks good to me!

Only one doubt I have @dangrous is EReceiptThumnail sets its bg color dynamically based on the transaction id so it might cause some confusions, for instance, when we create a request because it is a draft transaction (with transactionID 1) on confirmation page it will always be orange and after you create the request it will have different bg color when it gets its real transaction id. So should we set static bg color or we can go with the current dynamic bg color status ???

Eh, I think it'll be fine - I'm not that concerned about that for now.
I will have time to look through the PR tomorrow morning! Looking forward to it.

payment summary
@mollfpr requires payment automatic offer (Contributor)
@FitseTLT requires payment automatic offer (Contributor)-$500

[@mollfpr] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.

I don't think we need a regression test here.
@zanyrenney I'll do manual request in NewDot, thank you!

payment summary
@mollfpr requires payment automatic offer (Contributor) to be paid $500 on ND.
@FitseTLT requires payment automatic offer (Contributor)- paid $500

$500 approved for @mollfpr


feat: Add a dry run mode

Currently, there's no way to run etcd-defrag without having it potentially perform defragmentation. Before writing any changes, one might want to observe what the effects will be, for instance when testing defrag rules. These changes add a flag for enabling dry run mode, wherein etcd-defrag will check endpoints, evaluate the defrag rule, but skip compaction and defragmentation.

Thanks @pchaseh for the PR, which looks good. Please fix the minor comment.


Kodi support? Alternatives?

Hi,
Sorry if this question as already been answered but I didn't find any, so I'm asking here.
I own a Jellyfin Server, which I share to friends. It works perfectly. But now we are looking to watch movie toguether via Syncplay. Already tried it with VLC, and it works great.
I know there is already a syncplay feature built in Jellyfin but it only works with movies played in the web version of Jellyfin. Unfortunately, 90% of my library is x265 encoded so it can't be played via the web UI. It only works when played with Kodi and its addon. So we all have to use Kodi.
But Syncplay is not compatible with Kodi so we can't use my server to watch movies together.
I am looking to find a solution for that, but after days of research, I didn't find anything. I find that very strange, because I guess movie playback syncing is something A LOT of people are asking for. But there is very little solutions available. Syncplay is one of them, but not for Kodi.
Is Kodi support planned ? Is there a addon that could do that ? Is there alternatives to Syncplay that could work for us ? Is there other solutions I may not have heard about ?
Thanks in advance for your help,
Have a good day.

It is noted at https://syncplay.pl/about/ideas/ that: "[Support for] additional media players would need to come from an individual who used that media player and was willing to maintain it, and may require modifications to the media player software itself."
Syncplay is an open source project run by unpaid volunteers. Anyone can contribute but nobody is forced to do. As such, the 'supply' (i.e. what people are willing/able to do) is at least as important as 'demand'. Usually people contribute features that they themselves want to see in Syncplay, and because they then use that feature they are able to identify problems and fix them as they arise (e.g. it is not uncommon for media players to change things in new versions which breaks Syncplay compatibility).
Back in 2015 Tari gave adding Kodi support a go but the project stalled. See https://github.com/tari/syncplay/commits/kodi-support and https://github.com/Syncplay/syncplay/pull/77
Back in 2016 someone asked for volunteers at https://forum.kodi.tv/showthread.php?tid=213926 for someone to take over the effort but nobody stepped forwards.
Nobody is currently volunteering to introduce and maintain Kodi support, and so it isn't happening. I'll leave this issue open for a while to see if anyone will volunteer for this project, but as I don't use Kodi I am not the right person to lead this project.

This topic has been open for more than a month and nobody has volunteered to introduce and maintain Kodi support. As such, I'll close this issue for now but if someone comes along who wishes volunteer then feel free to re-open the discussion.


Conditions

https://app.roll20.net/forum/permalink/3124285/

Closed as part of my new issue tracker via bitbucket


[Enhancement] azd down - add portal links to RG

Will be good to show a portal link to the RG in context. Enabling users to quickly check the resources  and decide.
azd down
? This will delete 9 resources, are you sure you want to continue? Yes
Deleting resource group rg-wiuqey... /

cc: @Petermarcu @salameer

Handled as part of https://github.com/Azure/azure-dev/pull/1924


[Feature Request] List of reactions for each message in #show-your-projects

So every message that is sent in the #show-your-projects channel, the bot sends this list of reactions in order:
👍 👎 ❤ 🚩 👀
This can be changed any way you want, but essentially whenever someone posts a message showing their project, the bot immediately sends these reactions, so that people can quickly react with those emojis without looking for them in a menu. You can even have it so you turn off the ability for people to add new reactions, only using the reactions above. An extra technical detail, since some tend to post their project as more than one message, maybe a 30-second slowmode on the channel can deter that too.
Implementation details

[ ] Write the bot command (duh)

[ ] Choose the reactions and the order

[ ] Change Permissions (optional)
[ ] Add slowmode (optional)

Additional information
This is an evergreen request.
I hope I didn't miss anything, but if I did you can leave a comment telling me what. Also if anyone has suggestions for certain emojis to use as reactions you can leave those in too.

I'm a little opposed to the idea of having a 👎 emoji on the messages. show-your-projects is frequently used by beginners to show off their first projects or whatever and if they post it and someone nasty reacts with a downvote then it could be quite demoralising.
If someone does have improvements to make to the project they should voice them in the discussion channel instead of just reacting with a downvote.
I'm fine with the other implementations of this though including the permission change and the slow mode.

@jos-b Good point, the list of emojis are definitely not final, (there was another one I was thinking of: 👍  ❤ 👀  ⭐  🚩) but I want to see what more people think about a good list of reactions.

can i use "on_message" ?

I got this.

one second, 👍 ❤ 👀 🎉 🚀 ⭐ 🚩  is the my decided list of emojis just so you know

oh okay,cool.
I too will look into those emojis and see if i can find something intresting.

Please wait for issues to have the "Approved" label before claiming

oh okay,got it.ill wait

What is the position with this @sco1 ? Is this going to be approved. It is a good idea and I would be happy to implement.

I'm going to go ahead and approve this. It doesn't actually hurt anyone to add this on, and could be a nice feature for users to have. However, I'd like to ask @DuckyQuack if he would like to tackle this issue on first, since they were the one to bring it up.

Looks like no response from Owner. Regardless, another thing you could have is a leaderboard maybe and set command? For example to submit a message in that channel you run a command like:
!submit url=... description=...
If a file is attached then no need for the URL.  Upon a message being sent the bot deletes it, processes it and posts in an embed. This way when users react we have all info such as title and description for the leaderboards. Not sure if this is something we want. Let me know.

@RohanRadia Since @DuckyQuack hasn't responded to this since it was brought up, I'm going to let you work on it.


Returning url

https://github.com/criteo/addressable-network-proposals/blob/d0c18b9a4c5b516ddb7d215b6a44a40ac78f3444/mvp-spec/operator-design.md?plain=1#L85
Why the CMP is stepping in at this stage? Why is the CMP providing the returning URL and not the Publisher directly?

Another "not-so-clear" part of the diagram, I must admit. Thanks for pointing it out.
Actually, the publisher itself provides its return URL when calling the CMP, via what I called the pubPage parameter.
I invented this keyword to emphasise that the publisher calls the CMP backend but doesn't have to know that behind the scenes, it's PAF that is called. That's why I didn't want to use the same name that is used when the CMP builds the URL to call on the operator.
But what really happens is: CMP takes the value of pubPage and puts it in returnURL.
So I realize it's just creating confusion. I will update the diagram to use returnURL at both steps.
To answer more broadly to your question, the reason why the CMP is stepping in is to sign the request that will be sent to the operator. This has to be done by a backend that (securely) owns a private key. It cannot be done by the frontend (in a browser).
Note that it doesn't have to be the CMP that signs, it just needs to be what I called an "operator backend proxy" (see https://github.com/criteo/addressable-network-proposals/blob/main/mvp-spec/landscape.md for a bit more context and https://github.com/criteo/addressable-network-proposals/blob/b17ee292010e4e29055854b0ee7b6a784291264c/mvp-spec/website-design.md for more details). In this example, this component is hosted by a CMP backend.

So if I understand it well, in this case we decided to make the CMP backend play the role of "operator backend proxy" but we could have chosen the publisher backend instead, right?
FYI I'm still not able to see the diagram  (it appears blank) https://github.com/criteo/addressable-network-proposals/blob/b17ee292010e4e29055854b0ee7b6a784291264c/mvp-spec/website-design.md whereas for the rest of mermaid diagrams I have no problem (after a refresh usually)

in this case we decided to make the CMP backend play the role of "operator backend proxy" but we could have chosen the publisher backend instead
Precisely.

For the issue with Mermaid, we'll investigate together.
Anyway at some point we want to have these diagrams to be automatically generated, I will create an issue for that.


_.extend converts source object's prototype properties to "own" properties

I'm trying to figure out (or at least get opinions on) whether this behavior is an undocumented "feature" of _.extend or a bug:
function Foo(){}

var foo = new Foo();
Foo.prototype.bar = "On the prototype!";
foo.hasOwnProperty('bar');
// => false

_.extend(foo, {});
foo.hasOwnProperty('bar');
// => true

Basically, the current version of _.extend not only iterates through the properties of passed in "source" objects, but also through the properties of the "destination" object itself, including properties on the prototypes. _.extend is intended to include properties from prototypes, AFAIK (if you want only the "own" properties, there's now _.assign for that), so that isn't the issue. The issue is whether _.extend should be including the properties on the prototype chain for the original destination object itself, so that the destination object gets new "own" properties and ends up extended even when the "source" is an empty object.
I'm inclined to think this shouldn't be the case. When the "source" is empty (an empty object), the "destination" shouldn't be changed. You can just supply the prototypes as additional arguments to _.extend if you want the prototype properties on the destination object iself.
Thoughts?
Original notice of this issue and some related discussion is here: https://github.com/jashkenas/underscore/pull/2022

This is actually a huge breaking change. Consider Backbone's .extend:
function extend(protoProps, staticProps) {
    //...

    // Add prototype properties (instance properties) to the subclass,
    // if supplied.
    if (protoProps) _.extend(child.prototype, protoProps);

    //...
};

With this change, all of the parent's instance methods will be copied onto the new subclass. This'll break things like generic base method wrapping (if it was loaded afterwards).
var View = Backbone.View.extend({});
Backbone.Model.prototype.set = wrappedSet;

View.prototype.set === wrappedSet; // => false

Great catch, @jmrog. :clap:

Good to know, @jridgewell. Thanks! I'll try to put together a pr (fix should be easy, at least at first glance) and a couple of tests shortly.

Actually, @jridgewell, can you say more (if you get a chance) about what's going on and/or supposed to be going on in that second bit of code (with wrappedSet)? I'm not immediately seeing it.

In general, you can change a parent classes prototype and have that immediately reflected in any subclasses.
var Parent = function() {};
Parent.prototype.method = function() {
    // does something...
};

var Child = function() {
    Parent.call(this);
};
Child.prototype = Object.create(Parent.prototype);

// Later on, we decide #method should throw for Parent and all its subclasses
Parent.prototype.method = function() {
    throw new Error("don't call this");
};

Under normal JS inheritance, calling Child#method will throw an error after the change. Since method won't be found on Child.prototype, it will traverse up the prototype chain until it finds it on Parent.prototype.
Since _.extend is now setting any enumerable-properties as own-properties, this means Child#method won't throw. That's because Child.prototype has an own method, and won't traverse up the prototype chain any further.

Ah, gotcha. Thanks. :+1:


feat: add data profiling

https://github.com/Kanaries/pygwalker/assets/22167673/fc06e61f-2bc6-4b8d-94f0-3da0862bbcb3


[WIP] Verible standalone preprocessor

Description:
This PR introduces takes another step to enhance the standalone preprocessor tool by implementing multiple-cu mode.
Please refer to issue #1358 for more information about the proposed design.
Goal:

In case multiple files are passed to the tool, each one of them will be compiled in a separate compilation unit, thus declarations scopes will end by the end of each file (this is the default behavior).
Compiler directives should be correctly interpreted (files inclusion, macros expansion and conditionals are a good start).

Flags description
Using the Abseil flags library allows an easy access to flags and their arguments.

Flags
Type
Default
Description
State

--strip_comments
bool
false
This replaces comments with equal number of whitespaces, to preserve the same byte offsets.
DONE

--generate_variants
bool
false
This generates all the possible variants for conditionals.
DONE

--multiple_cu
bool
true
In case of true multiple files are passed to the tool, each one of them will be compiled in a separate compilation unit, thus declarations scopes will end by the end of each file (this is the default behavior), otherwise all files will be compiled in the same compilation unit.
DONE

Positional arguments
Those mainly are the SV files to preprocess, defines and search-directory paths for includes.
Any of the following positional argument can be passed one or more times.

Argument
Description
State

<file.sv>
Specifies a SV file to be preprocessed.
DONE

+define+<foo>[=<text>]
Defines one or more macros same as  `define foo text
DONE

+incdir+<dir>
Specifies the directory to search for files included with  `include "file_name"
DONE

TODO:

[X] Fixing testcases reported in mglb's review.
[ ] Better comment placing.
[ ] const qualifier in cmd_positional_arguments.h.
[X] Fixing +define+macro+="a=b" issue.
[X] Removing the full usage message, relying on --helpfull from absl/flags.
[ ]  Better error messages.

Codecov Report

Merging #1360 (c62140b) into master (dd5e91a) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1360      +/-   ##
==========================================
- Coverage   92.91%   92.82%   -0.10%
==========================================
  Files         340      340
  Lines       23686    23709      +23
==========================================
  Hits        22008    22008
- Misses       1678     1701      +23

Impacted Files
Coverage Δ

verilog/tools/preprocessor/verilog_preprocessor.cc
51.92% <0.00%> (-41.19%)
:arrow_down:

Continue to review full report at Codecov.

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

Hello @hzeller ,
Wish you had a good vacation!
Here is a quick overview about what's going on here:

The preprocessor tool was getting arguments through a class defined in common/util/subcommand.h, It defines a subcommands such that each one has its own flags.
I changed that to the normal abseil flags to be similar to other Verible tools.
As @tgorochowik's request to standardize the way of passing arguments, What do you think about my approach?
About the generate_variants feature, I added comments so the code is understandable, but let me give you a quick overview:
Now, this feature doesn't know if a macro is defined or not, It just assumes all the possibilities and outputs the SV code for each one by constructing a control flow tree, which defines the edges between each token and the possible next tokens, then traveses with a depth first search to generate all possible vairants.

verible::CmdPositionalArguments aims to parse the preprocessor tool's positional arguments which includes:

<file.sv>
+define+<name>[=value]
+incdir+<dir>

Note:

For each type multiple arguments are supported in a single tool run.
For +define: multiple macros are supported in the same argument.
For +incdir: multiple directories are supported in the same argument.

TODO:

[ ] The arguments are accessible now from the tool, but only SV files are used, need to use the rest.
[ ] Need to implement a HandleIncludes function in verilog/analysis/preprocessor/.
[ ] Need to implement a AddDefineFromCmdLine function in 'verilog/analysis/preprocessor`.

Include directives supported usage:

[x] "file_path" can be an absolute path, or a relative one.
[x] In case of relative one, search_paths_ of verilog::VerilogPreprocess are searched.
[x] search_paths_ are passed to the tool through +incdir+<dir>[+<dir>].

Limitations:

Number of nested includes should be limited (min is 15 as per LRM).
Come up with a more efficient way to extend the included file TokenInfos life (more info is documented verilog/preprocess/verilog_preprocess.cc).
Share defined macros with the included files.
Minimal testing was done, need to add more tests.

To solve the white-spaces problem I have added VerilogPreprocess::GenerateBypassWhiteSpaces function which skips white-spaces tokens, and returns an iterator to a non-whitespace token.
TokenStreamView::const_iterator VerilogPreprocess::GenerateBypassWhiteSpaces(
    const StreamIteratorGenerator& generator) {
  auto iterator =
      generator();  // iterator should be pointing to a non-whitespace token;
  while (verilog::VerilogLexer::KeepSyntaxTreeTokens(**iterator) == 0) {
    iterator = generator();
  }
  return iterator;
}

Then replaced each generator() into GenerateBypassWhiteSpaces(generator)) in VerilogPreprocess methods.
Example:
verible-verilog-preprocessor m5.sv
Example input file:
`include "file.sv"
module m();
  wire x = 1;
`ifdef KARIM
  real is_defined = 1;
`else
  real is_defined = 0;
`endif
// pre-blank line

// post-blank line
endmodule

Example output file:
module included_module();
  wire x=1;
endmodule

module m();
  wire x = 1;

  real is_defined = 0;

// pre-blank line

// post-blank line
endmodule

Note: if added +define+KARIM then the other branch will be selected.
TODO:

[ ] Add tests for all the implemented features, didn't want to do it earlier as I will have to revisit them once I fix the white-spaces problem.

[ ] File inclusion.
[ ] Multiple compilation unit mode.
[ ] Generate variants mode.

[ ] Open issues on the known limitations.
[ ] Maybe add a flag to disable blank lines.

@kbieganski : here are some bits and pieces for the preprocessor, that have been addressed in various sub-pull request.
Can you have a look through these to make sure everything actually made it to the final preprocessor or if there are pieces missing that we should reformulate in a new PR ?


How to configure @ServiceBusListener ServiceBusReceiveMode among Spring Message

Query/Question

I am using Spring Message @ServiceBusListener to receive the Azure Service Bus Queue message.
I refer to this official page
https://learn.microsoft.com/en-us/azure/developer/java/spring-framework/using-service-bus-in-spring-applications
I know the difference between PEEK_LOCK and RECEIVE_AND_DELETE of the ServiceBusReceiveMode, but

I did not find any attribute of @ServiceBusListener to configure the above receive mode?
Meanwhile, even I have tried to configure spring.cloud.azure.servicebus.consumer.receive-mode=RECEIVE_AND_DELETE or spring.cloud.azure.servicebus.processor.receive-mode=RECEIVE_AND_DELETE at the application.properties file, the result seems still  PEEK_LOCK, that means the message does not be deleted upon receiving

my question is how to configure the ServiceBusReceiveMode for the Java springboot if you use the annotation, eg: @ServiceBusListener
Setup (please complete the following information if applicable):

Java Spring Boot

        <dependency>
            <groupId>com.azure.spring</groupId>
            <artifactId>spring-messaging-azure-servicebus</artifactId>
            <version>5.9.1</version>
        </dependency>

### Tasks

@yiliuTo could you take a look?
/cc @Azure/azsdk-sb-java

Hi @kensinzl
Thank you for reporting this issue. We have received your submission and will take a look. We appreciate your input and will review this matter as soon as possible.
Please feel free to provide any additional information or context that you think may be helpful. We'll keep you updated on the progress of our review.

Hey, is there any progress, seems no one take this issue to have a look

@yiliuTo @Netyyyy @joshfree  any progress for this ticket? thanks

I'm stuck with a similar issue where I want to configure the @ServiceBusListener to use PEEK_LOOK and to enable sessions on the consumer as the subscription has sessions enabled. Properties like
spring.cloud.azure.servicebus.processor.session-enabled=true
spring.cloud.azure.servicebus.processor.receive-mode=peek_lock
do not seem to work with the annotated listeners.

@kensinzl FYI: Managed to figure it out... Those properties don't configure the @ServiceBusListener but seem to be for other approaches like ServiceBusClient.
I've managed to configure the listeners with the following bean declaration.
@Bean
public PropertiesSupplier<ConsumerIdentifier, ProcessorProperties> propertiesSupplier() {
    return key -> {
        var processorProperties = new ProcessorProperties();
        processorProperties.setEntityType(ServiceBusEntityType.TOPIC);
        processorProperties.setReceiveMode(ServiceBusReceiveMode.PEEK_LOCK);
        processorProperties.setSessionEnabled(Boolean.TRUE);

        return processorProperties;
    };
}

@nikoladjuran
Hey man, thanks for the information.
I still use the @ServiceBusListener(Spring messaging) to monitor the service bus queue. If my understanding is correct, the @ServiceBusListener source code is using the default ServiceBusReceiveMode.PEEK_LOCK and also embed the commit to delete the message if successfully executed.
again, the document of azure is quite a mess, particularly the Java Azure SDK. Hopefully, I am on the right track.

double confirm, any progress for this ticket?? @yiliuTo @Netyyyy @joshfree, thanks so much

Hi @kensinzl, this is still in our backlog.

@kensinzl FYI: Managed to figure it out... Those properties don't configure the @ServiceBusListener but seem to be for other approaches like ServiceBusClient. I've managed to configure the listeners with the following bean declaration.
@Bean
public PropertiesSupplier<ConsumerIdentifier, ProcessorProperties> propertiesSupplier() {
    return key -> {
        var processorProperties = new ProcessorProperties();
        processorProperties.setEntityType(ServiceBusEntityType.TOPIC);
        processorProperties.setReceiveMode(ServiceBusReceiveMode.PEEK_LOCK);
        processorProperties.setSessionEnabled(Boolean.TRUE);

        return processorProperties;
    };
}

@kensinzl, have you tried this approach? If you register a PropertiesSupplier bean in your application,
@Bean
public PropertiesSupplier<ConsumerIdentifier, ProcessorProperties> propertiesSupplier() {
    return key -> {
        var processorProperties = new ProcessorProperties();
        processorProperties.setReceiveMode(ServiceBusReceiveMode. RECEIVE_AND_DELETE);

        return processorProperties;
    };
}

I understand the perfect way to let you configure it in the annotation, but could you help check whether this approach could meet your requirement? Or what you expect is to be able to configure it via annoation?

@saragluna
Thank you for the reply.
I make it work now. Let me type the following so that other people can refer if they also face the same issue.

https://learn.microsoft.com/en-us/azure/developer/java/spring-framework/using-service-bus-in-spring-applications
if you used Spring Messaging Azure Service Bus annotation way from the above official link, whose default mode is the PeekLock which is hardcode at the ServiceBusReceiverClientBuilder
I tried the properties configuration, eg: spring.cloud.azure.servicebus.consumer.receive-mode=RECEIVE_AND_DELETE or spring.cloud.azure.servicebus.processor.receive-mode=RECEIVE_AND_DELETE, But both of them DO NOT change the mode into RECEIVE_AND_DELETE from the log monitor.

if you want to use the RECEIVE_AND_DELETE mode for the Spring Messaging Azure Service Bus annotation way, you can inject a Bean like the following code. Note: you need to make the auto-complete into false.
    @Bean
    public PropertiesSupplier<ConsumerIdentifier, ProcessorProperties> propertiesSupplier() {
        return key -> {
            ProcessorProperties processorProperties = new ProcessorProperties();
            processorProperties.setReceiveMode(ServiceBusReceiveMode.RECEIVE_AND_DELETE);
            processorProperties.setAutoComplete(false);
            return processorProperties;
        };
    }

this will be injected into AzureServiceBusMessagingAutoConfiguration.
hopefully, I am at the right track


Fix file read/write encoding and bump version

It turns out the default file encoding used by Python for both reading and writing files is platform dependent, meaning if you're trying to save UTF8-encoded data on Windows it's likely to fail, as the default encoding is Windows 1252(!). This is horrid, but you can fix it by adding encoding="utf-8" to any calls to open().
Bump the project version so I can issue a new release.

Yup! The default encoding used on Linux and Mac is usually (always?) UTF8 anyway, so it doesn't make a difference. It's just Windows which is weird here.


Use "ConditionRef + index" rather than "ConditionRef" to register "Condition" in state of "Pipelinerun"

fix issue #2124
Use ConditionRef + index rather than ConditionRef in state of pelinerun to display state of taskrun of condiition.

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Use ConditionRef and index of the Condition to register Condition’s Taskrun in state of Pipelinerun.

/test pull-tekton-pipeline-integration-tests

/assign @dibyom

@dibyom
Done, thanks

Awesome! Thanks @vincent-pli
/lgtm

/test pull-tekton-pipeline-integration-tests


Why won't release 2.1.0 start my server?

I switched from directly importing a local version from the /develop branch (commit 6dae027) to the full 2.1.0 release but now I cannot start my servers, it throws two errors every time. I included Import-Module Pode in my $profile and I am running as an admin of course. I also ensured my server.ps1 script imports Pode on the first line with Import-Module Pode -MaximumVersion 2.99.99 -Force.
I've also tried importing Pode manually before calling pode start or even calling server.ps1 directly. All of this results in the following two errors every time:
PS \> pode start
Close-PodeServerInternal: C:\Program Files\PowerShell\Modules\Pode\2.1.0\Public\Core.ps1:198
Line |
 198 |          Close-PodeServerInternal -ShowDoneMessage:$ShowDoneMessage
     |          ~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'Close-PodeServerInternal' is not recognized as a name of a cmdlet,
     | function, script file, or executable program. Check the spelling of the name, or if
     | a path was included, verify that the path is correct and try again.

Convert-PodeFunctionVerbToHttpMethod: C:\Program Files\PowerShell\Modules\Pode\2.1.0\Public\Routes.ps1:745
Line |
 745 |              $_method = Convert-PodeFunctionVerbToHttpMethod -Verb $ve …
     |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'Convert-PodeFunctionVerbToHttpMethod' is not recognized as a name of a
     | cmdlet, function, script file, or executable program. Check the spelling of the
     | name, or if a path was included, verify that the path is correct and try again.

PS \>

Versions:
Pode: 2.1.0
PowerShell: 7.1.1
Windows: 10.0.19042

Hey,
If you do $error after running ./server.ps1, is there anything else in there? There could be another error triggering those two.

I reverted and worked on another feature for awhile, then tried to upgrade again without issue. I cannot reproduce the error at this point. If it comes back, I'll go down the rabbit hole of looking into $Error. Thanks!!

The error has come back to me, when I look at $Error, I get:
PS C:\Users\sijet-dot\Documents\GitHub\Pode1> $Error[0]

Invoke-PodePackageScript: C:\Program Files\PowerShell\Modules\Pode\2.1.0\Public\Core.ps1:463
Line |
 463 |              Invoke-PodePackageScript -ActionScript $actionScript
     |              ~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'Invoke-PodePackageScript' is not recognized as a name of a cmdlet, function, script file, or
     | executable program. Check the spelling of the name, or if a path was included, verify that the path is
     | correct and try again.

PS C:\Users\sijet-dot\Documents\GitHub\Pode1> $Error[0].Exception

ErrorRecord                 : The term 'Invoke-PodePackageScript' is not recognized as a name of a cmdlet, function,
                              script file, or executable program.
                              Check the spelling of the name, or if a path was included, verify that the path is
                              correct and try again.
CommandName                 : Invoke-PodePackageScript
WasThrownFromThrowStatement : False
TargetSite                  : System.Management.Automation.CommandInfo LookupCommandInfo(System.String,
                              System.Management.Automation.CommandTypes,
                              System.Management.Automation.SearchResolutionOptions,
                              System.Management.Automation.CommandOrigin,
                              System.Management.Automation.ExecutionContext)
StackTrace                  :    at System.Management.Automation.CommandDiscovery.LookupCommandInfo(String
                              commandName, CommandTypes commandTypes, SearchResolutionOptions searchResolutionOptions,
                              CommandOrigin commandOrigin, ExecutionContext context)
                                 at System.Management.Automation.CommandDiscovery.LookupCommandProcessor(String
                              commandName, CommandOrigin commandOrigin, Nullable`1 useLocalScope)
                                 at System.Management.Automation.ExecutionContext.CreateCommand(String command,
                              Boolean dotSource)
                                 at System.Management.Automation.PipelineOps.AddCommand(PipelineProcessor pipe,
                              CommandParameterInternal[] commandElements, CommandBaseAst commandBaseAst,
                              CommandRedirection[] redirections, ExecutionContext context)
                                 at System.Management.Automation.PipelineOps.InvokePipeline(Object input, Boolean
                              ignoreInput, CommandParameterInternal[][] pipeElements, CommandBaseAst[]
                              pipeElementAsts, CommandRedirection[][] commandRedirections, FunctionContext funcContext)
                                 at
                              System.Management.Automation.Interpreter.ActionCallInstruction`6.Run(InterpretedFrame
                              frame)
                                 at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(Interp
                              retedFrame frame)
Message                     : The term 'Invoke-PodePackageScript' is not recognized as a name of a cmdlet, function,
                              script file, or executable program.
                              Check the spelling of the name, or if a path was included, verify that the path is
                              correct and try again.
Data                        : {System.Management.Automation.Interpreter.InterpretedFrameInfo}
InnerException              :
HelpLink                    :
Source                      : System.Management.Automation
HResult                     : -2146233087

When I call .\server.ps1 at the terminal, it works without issue. I've attempted to debug the pode start command inside VSCode, but it does not fail in that environment, but when I follow the code, my only guess is somehow it is not importing the private functions when I execute pode start from a separate admin terminal outside of VSCode.
As a note, I do not have an import command in my PowerShell profile and I installed Pode via Choco.

I managed to get this to trigger, but only if I had Import-Module Pode -Force in the ./server.ps1 script. Removing that, or removing just the -Force fixed it.
Do you import the module in ./server.ps1?

Why yes, I do import the module at the top of the server script... I will remove it and try it next time I touch the server.
I believe importing the module was in one of the getting started tutorials, should that be removed from the tut?

Okay, I was able to test it and it is able to start without the Import-Module Pode -MaximumVersion 2.99.99 -Force statement at the top of ./server.ps1.
I can tell where I got myself confused when I first started, the Building your first Pode app indicates you should use the Import-Module statement to avoid breaking changes and I mistakenly thought the tutorial was saying to place that at the top of ./server.ps1.


Thumbnail video preview

Could you make a thumbnail video preview?

?????????????

...

Sorry, could you be a little more impatient? There's plenty of other issues to attend to as I'm sure you can see. You're not the only one around here...
If you're going to raise an issue, at least put some effort into an explanation. A thumbnail where? There's poster image functionality baked right into the HTML5 elements. If you mean in the seek tooltip like YouTube does, then you'd need to do something custom. It might be possible using canvas to load the buffered video and use that to display. Another option would be generating them server side.  This is an open source project that I maintain in my spare time.


Capstone2: Movie-website

Movie-website
In this project with my partner @Lembani, we implemented a movie app using TVmaze API and capstone API for recording the users' actions. You can comment and like any movie you see through this app.
For the demo, Please Click Here!
your review is welcome :smile:

Hi @zhadier, thanks a lot for approving our PR. We really appreciate it. 💯


152: Editor blocks and styling

Closes #152

Hides unsupported blocks from post editor.
Hides block patterns from post editor.
Hides unsupported or problematic options from supported blocks.
Hides unsupported or problematic rich text formats.
Styles editor to layout blocks similar to front-end feature layout.

To Review

[ ] Checkout Branch.
[ ] Run npm run local (or npm run refresh if you want a fresh database.)
[ ] Log into http://the-world-wp.lndo.site/wp-admin/.

...then...

[x] Edit a post.

NOTE: Due to issues still present in the content migration, images will not initially load with their correct layout.

[x] Select an image. In the sidebar, open the Settings context menu and click Reset all. This will be what images should migrate over as with default formatting.
[x] Change image alignment. Ensure layouts look closer to what would be n the frontend.
[x] Try inserting your own image.
[x] Try inserting a YouTube video.
[x] Try Inserting a Vimeo video.
[x] Try Inserting a Tweet.
[x] Try Inserting a pullquote.
[x] Note that blocks available are in line with what can currently be handled by the front-end.
[x] Note that the options for each block is in line with what can currently be handled by the front-end. (Should only see the Advanced options in the sidebar.)
[x] Select some text in a paragraph and check the formatting options.
[x] Note formatting options are in line with what is currently supported by the front-end.
[x] Mouse over space between blocks and click the + button, then click Browse all.
[x] Ensure Patterns tab is not shown.

@rpeterman-gp This works as described. There is one issue with pullquotes - that I can't replicate on live

Create a pullquote
Type in content, when you press space, the space is removed after I type in the content
I was able to replace the space after tabbing to the next field.

Video:
https://github.com/PRX/cms.theworld.org/assets/369739/c1eb3a0c-456f-452c-b0ce-560240cb8da0


Design the Gallery (new) page & new partners

Gallery page we can explore the pictures lastest events.

Add new partners ✅

#55


Add 50 & 100 distance options

ideally make them smart so that they only show up if they are needed

thanks @gkovats - PR is merged and deployed now 🥇


carbon install via pip doesn't show up in "pip freeze" Edit

Imported from https://bugs.launchpad.net/bugs/1022576

Reported by
jlintz

Date Created
Jul 09, 2012

Tags
[u'carbon', u'install', u'pip', u'setup']

We need to fix this for 0.9.13. After further consideration I just don't think it's reasonable to ask users to set PYTHONPATH like this to fix problems in both Carbon and Graphite-Web.

Sorry for the milestone tom-foolery, this is indeed 1.0.0 material due to the prefix changes required.

The installation docs now formally instruct the user to set PYTHONPATH before installing via pip. Fixed in https://github.com/graphite-project/graphite-web/commit/1904348a6adbdf3eecb0c31442285fe759061f13.


[mesos] add mesos checks

Description
Adds both mesos_master and mesos_slave checks
Notes

tests use get_check_class I don't know how that'll behave within this separate repo
tests use fixtures, we still need to figure out how they'll be found iirc. Will ship them after we clear this out

Thanks for the review @gmmeyer
It should be good now, the linting issue was due to a bug in the gem I believe, it wasn't considering the flake8 ignore_list from tox.ini. It seems to work now.


ValueError: need at least one array to stack

run the demo...
Traceback (most recent call last):
File "humor/fitting/run_fitting.py", line 439, in
main(args, config_file)
File "humor/fitting/run_fitting.py", line 175, in main
video_name=vid_name
File "/home/junwei/zjw/humor/humor/fitting/../datasets/rgb_dataset.py", line 59, in init
self.data_dict, self.seq_intervals = self.load_data()
File "/home/junwei/zjw/humor/humor/fitting/../datasets/rgb_dataset.py", line 146, in load_data
joint2d_data = np.stack(keyp_frames, axis=0) # T x J x 3 (x,y,conf)
File "<array_function internals>", line 6, in stack
File "/home/junwei/anaconda3/envs/humor_env/lib/python3.7/site-packages/numpy/core/shape_base.py", line 422, in stack
raise ValueError('need at least one array to stack')
ValueError: need at least one array to stack
what happen?

From these lines:
File "/home/junwei/zjw/humor/humor/fitting/../datasets/rgb_dataset.py", line 146, in load_data
joint2d_data = np.stack(keyp_frames, axis=0) # T x J x 3 (x,y,conf)
It seems like it cannot find the openpose keypoints so maybe you didn't provide the correct path to openpose

Yes, the error indicates the OpenPose keypoints are not being saved and/or read in properly. Are you using your own video or the one provided in the repo? I would check to make sure the keypoint json files are correctly being saved to rgb_preprocess. If not, check to make sure OpenPose is installed correctly and the --openpose flag points to the correct place. If OpenPose is running correctly, I would suggest investigating the json files to determine why it's not being read in correctly with this function (e.g. no people are detected or something like that).

Closed due to inactivity. Please re-open if issues persist.


Teltonika IO decoding

I think it is more convenient to discuss and collect it here
I've collected IO meanings from some documentation, last column from official for FMBXXX devices http://teltonika.lt/?media_dl=6079 only permanent and eventual IOs.
Teltonika Summary IO Table.xlsx
Marked green that totally compatible, marked red that is not compatible from device to device.

Uploaded to Google Docs:
https://docs.google.com/a/traccar.org/spreadsheets/d/1BTbIQ9nlkwqDO1s9AnoighdxyiwCSCCVVEBdi0dOjTU

Sounds good to me.


Allow group headings in section navigation

Currently, to produce the following lefthand navigation menu, the section navigation component has to be used multiple times for each sub-headed group. This results in multiple nav elements for what is essentially a single navigation menu.
This fails WCAG 2.1 Level A criterion 1.3.1 ('Info and relationships') because the h3 group headings aren't contained within any landmarks.
Allowing multiple lists within the component, with sub-headings, will simplify and improve the labelling of the nav landmarks on the page.
Note:
We're already handling these groups/sections in this way in the sub-navigation macro for the header component

Originally posted by @jrbarnes9 in https://github.com/ONSdigital/design-system/discussions/174#discussioncomment-5053551

Links to #242 in Service Manual


issue if using reqwest v0.12.0 and above

Hi everyone,
I've recently started to use that great library after watching on of your video @jeremychone  and I love it.
However in my case some of my testing required me to set the header for auth
therefor I've use the reqwest_builder i.e. reqwest::ClientBuilder to be use in conjonction with the httpc_test::new_client_with_reqwest.
So far so good, however the issue I want to raise is that for what ever reason if you use a version of reqwest v12 and above  you receive that annoying:

expected ClientBuilder, found a different ClientBuilder

I fafed around with different version and everything work fine until you chose v0.12.0
Max supported is v0.11.27.
Thanks for your great work and I hope I'm writting this in the right place.
Alex

@SniffleSneeze Sorry for this forever delay. I finally merged @joeftiger PR that should fix the issue.


tag blocks in guide.jade

you should like test this because i didnt

you have completely destroyed the formatting of text

#failing

don't worry, i still love you <3


Add Treadle to CI builds

Change CI to build Treadle master from source. This fixes a bug where
Treadle would be pulled from the latest snapshot (which would pull the
latest FIRRTL snapshot). For bleeding edge PRs that rely on new FIRRTL
features, using a FIRRTL snapshot may result in compilation failures.
Signed-off-by: Schuyler Eldridge<EMAIL_ADDRESS>
Related issue: https://github.com/freechipsproject/chisel3/pull/1499#issuecomment-658963659

Type of change: other enhancement

Impact: no functional change

Development Phase: implementation
Release Notes

None.

I added one commit, marked with !remove, which adds a reference to the new firrtl.ir.Formal package to show that this PR fixes the build issues mentioned in https://github.com/freechipsproject/chisel3/pull/1499#issuecomment-658963659. Pending approval, I'll strip this commit via rebase, but am leaving it in for a reviewer to verify that this does do what it's supposed to do.


Can't directly edit components within a managed solution ...

Message in Dynamics/Power Apps says:
You cannot directly edit the components within a managed solution. If the managed properties for solution components are set to allow customization, you can edit them from another unmanaged solution.
Due to this, I don't seem to be able to set the Username/Password in the Configuration area - after I click submit, nothing happens so I don't know if it has saved or not.
Should I be using the 'Unmanaged' zip file instead? (i.e. ReportRenderer_1_5_0_0.zip instead of ReportRenderer_1_5_0_0_managed.zip)

tried loading the 'Unmanaged' file but that gives a warning about not being able to uninstall the components once it's installed (which is not the case for the 'Managed' solution). So I've not loaded the Unmanaged file yet for that reason. I guess this is a standard warning for all unmanaged solutions but would still like your guidance on whether I should stick with Managed if there's a way to bypass the Managed issue from the first post? Or should I go Unmanaged?
Message:
"Changes applied by importing an unmanaged solution cannot be uninstalled. Do not install an unmanaged solution if you want to roll back the changes."

Hi @bguidinger - any guidance on the above?

Use managed. The username and password will disappear after you click submit.
Get Outlook for Androidhttps://aka.ms/ghei36

From: John Murray<EMAIL_ADDRESS>Sent: Tuesday, September 10, 2019 3:29:22 AM
To: bguidinger/Xrm.ReportScheduler<EMAIL_ADDRESS>Cc: Bob Guidinger<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [bguidinger/Xrm.ReportScheduler] Can't directly edit components within a managed solution ... (#33)
Hi @bguidingerhttps://github.com/bguidinger - any guidance on the above?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bguidinger/Xrm.ReportScheduler/issues/33?email_source=notifications&email_token=AAE7QNHMMYQZ7GK25P634H3QI5LGFA5CNFSM4ITDKIBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6KJJRQ#issuecomment-529831110, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAE7QNDAHTN6B6OB4D7NMFDQI5LGFANCNFSM4ITDKIBA.

@johnmurray01 were you able to get this working?

Hi @bguidinger Having an issue with Connector Status - says "This connection is not authenticated - Update password" - but when i click update password and then click fix connection, it tries to log me in (even though I'm already logged in) to fix it, but it doesn't work - any ideas on what password they're referring to here? My CRM login or my Report Renderer pw? I dont think I can change the Report Renderer pw, can I?

ok, i've just uninstalled and reinstalled - that seems to have got rid of the authentication issue.

Failed with error: 'AADSTS650057: Invalid resource. The client has requested access to a resource which is not listed in the requested permissions in the client's application registration. Client app ID: 787e2xxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx(Report Renderer). Resource value from request: https://xxxuat.crm11.dynamics.com. Resource app ID: 00000007-0000-0000-c000-000000000000. List of valid resources from app registration: 00000003-0000-0000-c000-000000000000. '. Please sign in again.
not sure what i've done wrong here?

ok, fixed the permissions issue above - but now when I try to setup the Flow, I'm only seeing a list of 'System Views' in the 'Render Template - Excel' dropdown. I want to be able to run Views I created myself in the 'My Views' section of Dynamics 365. Any ideas how to bring these up?

Even when i try to run a System View i get the following error:
{
"error": {
"code": "0x80040265",
"message": "Something went wrong: The remote server returned an error: (400) Bad Request.",
"innererror": {
"message": "Something went wrong: The remote server returned an error: (400) Bad Request.",
"type": "System.ServiceModel.FaultException1[[Microsoft.Xrm.Sdk.OrganizationServiceFault, Microsoft.Xrm.Sdk, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bfxxxxxxxxxxxxxx]]",       "stacktrace": "   at Microsoft.Crm.Extensibility.OrganizationSdkServiceInternal.Execute(OrganizationRequest request, InvocationContext invocationContext, CallerOriginToken callerOriginToken, WebServiceType serviceType, Boolean checkAdminMode, ExecutionContext executionContext, Dictionary2 optionalParameters)\r\n   at Microsoft.Crm.Extensibility.OData.CrmODataExecutionContext.Execute(OrganizationRequest request, ExecutionContext executionContext)\r\n   at Microsoft.Crm.Extensibility.OData.CrmODataServiceDataProvider.ExecuteOperation(CrmODataExecutionContext context, EdmOperation edmOperation, Dictionary2 parameters, Dictionary2 boundParameters)\r\n   at Microsoft.Crm.Extensibility.OData.ActionController.ProcessOperationRequest(String operationName, Dictionary2 operationParameters, EntityReference entityReference, String boundEntityName, String boundEntityType)\r\n   at Microsoft.Crm.Extensibility.OData.ActionController.<>c__DisplayClass11_0.<PostBoundAction>b__0()\r\n   at Microsoft.PowerApps.CoreFramework.ActivityLoggerExtensions.Execute[TResult](ILogger logger, EventId eventId, ActivityType activityType, Func1 func, IEnumerable1 additionalCustomProperties)\r\n   at Microsoft.Xrm.Telemetry.XrmTelemetryExtensions.Execute[TResult](ILogger logger, XrmTelemetryActivityType activityType, Func1 func)\r\n   at lambda_method(Closure , Object , Object[] )\r\n   at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ActionExecutor.<>c__DisplayClass10.b__9(Object instance, Object[] methodParameters)\r\n   at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ExecuteAsync(HttpControllerContext controllerContext, IDictionary`2 arguments, CancellationToken cancellationToken)\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Controllers.ApiControllerActionInvoker.d__0.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Controllers.ActionFilterResult.d__2.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Dispatcher.HttpControllerDispatcher.d__1.MoveNext()"
}
}
}

Hi @bguidinger -
2 Issues:

Getting error above when running a System View
Cant run or even see Views (Advanced Finds) I created myself in the 'My Views' section of Dynamics 365.

@bguidinger  bob, anybody there?

Hi John - I have a little one at home now, and my priorities have shifted.
I do not have much time to dedicate to this project at this time.
On Mon, Nov 4, 2019 at 4:57 AM John Murray<EMAIL_ADDRESS>wrote:

@bguidinger https://github.com/bguidinger bob, anybody there?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bguidinger/Xrm.ReportScheduler/issues/33?email_source=notifications&email_token=AAE7QNANE7FJSNBQQKGLP3TQR75X7A5CNFSM4ITDKIBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC63MDY#issuecomment-549303823,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE7QNF2IX5SQKTDS5VY2RLQR75X7ANCNFSM4ITDKIBA
.


Add experimental support for ARM64 for FAT-only modification

Per https://forum.mikrotik.com/viewtopic.php?t=184254&sid=69052abfd7c3fbd2b38567ae0ed7ccee#p1100176
I don't have a ARM64 Mac handy, but created no-gdisk-arm script  – which is mean just EXT2 to FAT conversion, using ARM64 CHR imagine.

It unclear what's needed.  No high demand (including from me since I used Intel ;)).

fwiw the amd64 raw image for 7.17 beta 2 worked fine with an Oracle Cloud VM running Ampere arm64 (VM.Standard.A1.Flex instance) - those shapes only support UEFI so this was very handy!  Downgraded to 7.16 after using this and all good.


Use double quotes, not single, in IDs

I don't think it causes a build issue, but this is non-standard:
[id='support-overview-remote-health-monitoring']
Should use double quotes:
[id="support-overview-remote-health-monitoring"]

Interesting to note, thanks for the feedback 👍
"As a project style rule" - Does that refer to a .vale.ini file outside of the vale-at-red-hat repo?

I think this came from a peer review comment in the OpenShift docs. As it's not bad AsciiDoc, perhaps it's better not to flag as an error in the AsciiDoc rule set.
As Aidan alluded to, if this went against style guidelines for a particular project, for example RHEL or OpenShift or whoever, you could create a rule for it and include it in the project's rule set. And yes, the .vale.ini file is how you configure Vale in a repo, for example, what rules you want turned on or off in a rule set, or what rule sets (or packages) you want to use etc.

Thanks for your comments @rohennes. I'm still getting to grips with using Vale so this info is helpful :)
In terms of helping to resolve this issue, is it a matter of moving this rule into OpenShiftAsciiDoc, or does this need to be addressed within the OpenShift docs themselves?

Typically these types of rules might hang off some project-specific guidance. We don't have a rule about this in the OpenShift style guidelines though. So it's probably solving an issue that doesn't really exist. I got a review comment about it but perhaps that was just one reviewer's preference rather then a hard rule.
Sorry if this issue was a wild goose chase but probably need to drop it unless you can find some project that really disallows this. The updates in your PR looked good otherwise!

You could try asking the mod docs people if they want to mandate for double-quotes by opening an issue? You could add it to the Red Hat rule set then. It's a bit of a long shot though. https://github.com/redhat-documentation/modular-docs/issues

Thanks @rohennes, I'll take a look :)

We support single and double quotes in the rules FYI: https://github.com/redhat-documentation/vale-at-red-hat/pull/613
I don't think we need to change this tbh


XML Key Ordering, missing namespaces, decimal conversion additions

This PR includes several fixes:

(legacy) fixes to netconf xml generation pre-tree
Added sort option to tree XML generation to order tree based on schema keys (options: None, Alphabetical, SchemaBound)

Thanks Alex!
Can I ask what the purpose for the SchemaBound ordering is? Unfortunately I do not understand it. Thanks

Of course, it came from the requirement that our NF seems to require list elements with a key defined to have the first children in the order defined in that key, which may not be the alphabetical order.
for example, given the following Yang model:
list a {
    key “id”;
    leaf id {}
    leaf abc {}
  }

In the current sort implementation, abc would be the first child and before id which would cause the config to be rejected by the NF.
This PR actually should depend on https://github.com/sdcio/schema-server/pull/134 which fixes the order of the keys sent by the schema-server to match the order given in the Yang model.

You seem to be totally right...

Key elements must be the first elements within the list entry to comply with the YANG and NETCONF standards.
we were also discussing if we should add more option flags or simply go with the sorted output... now also given this, I tend to do so.
The other discussion point was to also take data types into account and sort not just in alphabetical (string) order.
So if you like and have the time, give it a shot maybe.

02-Crud                                                               | PASS |
117 tests, 117 passed, 0 failed
==============================================================================```

i will approve schema-server and bump the version; then will update this PR with the adjusted version


Implement Ruleset class and test

Travis Python2 tests failing because of test byte strings being passed in. Not sure how to fix.

When this is ready for merging to dev, don't forget to add some detail on set-up and usage of rulesets here: https://github.com/IATI/iati.core/blame/master/README.md#L85-L87 :-)

See https://github.com/IATI/iati.core/pull/45#discussion_r136948919 for a magic string to extract.

Test_rulesets could use refactoring but it's not currently a priority within our time constraints.


xds grpc disconnect

Environment:
gateway: lastest from main branch
envoy: 1.24-latest
Logs:

Include the access logs and the Envoy logs.

@qicz do you seen this issue with gateway-dev:latest and envoy:1.25-latest ?

@qicz do you seen this issue with gateway-dev:latest and envoy:1.25-latest ?

I will watch this, feedback soon.

I have further verified this issue and may need to adjust the timeout of the grpc connection.
  - connect_timeout: 60s
    load_assignment:
      cluster_name: xds_cluster

I set the conection_time to 60s, maybe my environment is slower.
suggestion: adjusting to 10s.  you can assign this task to me. @arkodg

thanks, static xds cluster https://github.com/envoyproxy/gateway/blob/2c57f3d3adbf4ce9f64126233bea323300929d67/internal/infrastructure/kubernetes/bootstrap.yaml.tpl#L32 probably should match dynamic cluster settings
https://github.com/envoyproxy/gateway/blob/2c57f3d3adbf4ce9f64126233bea323300929d67/internal/xds/translator/cluster.go#L34 (5s)

this can be part of https://github.com/envoyproxy/gateway/blob/main/api/config/v1alpha1/envoyproxy_types.go

thanks, static xds cluster
https://github.com/envoyproxy/gateway/blob/2c57f3d3adbf4ce9f64126233bea323300929d67/internal/infrastructure/kubernetes/bootstrap.yaml.tpl#L32
probably should match dynamic cluster settings
https://github.com/envoyproxy/gateway/blob/2c57f3d3adbf4ce9f64126233bea323300929d67/internal/xds/translator/cluster.go#L34
(5s)

the dynamic cluster settings may need the k8s gateway api support or use the envoyproxy api provides the global cluster timeout. the static xds cluster adjust the .tp file only.

this can be part of https://github.com/envoyproxy/gateway/blob/main/api/config/v1alpha1/envoyproxy_types.go

bingo

fixed by #1058, close it.

wrong PR number, updated.

thanks

wrong PR number, updated.

thanks @zirain


feat: add java docs

Adds java docs for:

Action classes
Dao Classes
SDFs

Thanks @hengfengli addressed all the nits.


Want to change Dictionary and List type from $ to ^ sign for MongoDB

I want to change in NewtonSoft.Json library. As in my case, the backend is of C# and have to save and retrieve data from MongoDB which don't consider the $ sign as the type in Dictionary and List. So, we have to change that $ sign to ^  sign.
As we did this change in the old library of NewtonSoft.JSON library, but now we again have to change it in updated code base. So can you tell me in which file(s) and where I make this change?
My sample JSON style is following
{"Variables": {     "^type": "Model.Data.GamePlayVariables, Assembly-CSharp",     "QuestionMultpliers": {       "^type": "Model.Data.QuestionMultipliers, Assembly-CSharp",       "Easy": 1.75,       "Medium": 3,       "Hard": 5.25,       "Impossible": 2     },     "QuizGold": 100,     "QuizThreshold": 60,     "_id": "gameplayVariables"   }}
Otherwise, I'm getting the following error when I switched my Unity Standalone project into Unity iOS project.
Could not convert string '^type' to dictionary key type 'TutorialNames'. Create a TypeConverter to convert from the string to the key type object. Path 'Data.Tutorials.Tutorials.^type', line 1, position 267.
StackTrace:   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateDictionary (IDictionary dictionary, Newtonsoft.Json.JsonReader reader, Newtonsoft.Json.Serialization.JsonDictionaryContract contract, Newtonsoft.Json.Serialization.JsonProperty containerProperty, System.String id) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject (Newtonsoft.Json.JsonReader reader, System.Type objectType, Newtonsoft.Json.Serialization.JsonContract contract, Newtonsoft.Json.Serialization.JsonProperty member, Newtonsoft.Json.Serialization.JsonContainerContract containerContract, Newtonsoft.Json.Serialization.JsonProperty containerMember, System.Object existingValue) [0x00000] in <filename unknown>:0

I would suggest loading JSON into a JObject after serializing it and using LINQ to replace any properties that start with a $ with a ^, and vice versa.

@JamesNK can you suggest it with the code snippet.
I think it would be better if I do this in Serialize and Deserialize functions. What do you say?


Add a getSelectionBarColor options

Add a getSelectionBarColor options to dynamically change the selection bar color. As discussed in #173.
Demo: http://jsfiddle.net/mtm2xhwx/

This is very good, but I kind of want it on the other side. I want to convey that choosing a number higher than a defined amount is bad.
In other words, I have a slider that goes from 1-14. Once I slide past 10, I want things from 11-14 to turn red.

Also, do we want to add the ability to allow the tick values to be able to turn colors as well or always just keep them blue?
http://take.ms/q2LC2

Thanks for your feedback. Indeed, I forgot to implement colors for ticks, now it is done.
About your first comment and the colors logic, everything is handled inside the user controller so this is up to you (see the updated demo).
About DOM manipulation and not using ng-class, what do you mean. I have tried using ng-style to apply the color but then we are dependant of angular $digest cycle and I got some weird behavior where the color was not updated. This is the same for the ticks, since they were already constructed "manually", I add a similar manual method the color them.

Ok, I think maybe I'm thinking about the UX for this incorrectly, but what I'd like is really just to make the line between 11-14 red at all times. I'd like to signify that moving the slider into that range is 'bad'. Maybe 11-14 turns darker red if the slider is actually in that range.
Yea, I think you might get better performance out of angular if you use it as it is intended and not use .html() to push stuff into the DOM.

If you want 11-14 to always be red then you can just set the slider bar to red using css (not the selection bar).
For the ticks part, it's true that it has become quite dirty but my logic was not to add any useless stuff for people not using the ticks (or the tooltip) . If I do it the angular way, I would need to add a ng-repeat for displaying the ticks but I think I'll try to do this because it would be cleaner and easier to maintain. As this is not directly to this PR, I'll do it outside of it.

@lookfirst I have reimplemented the ticks mecanism using ng-repeat, ng-style and ng-class and it almost work. The only issue is the one I was afraid of: synchronisation. In the JSFiddle example, if you move the slider quickly the ticks sometimes don't get the correct color whereas the color specified on the object is correct.

@ValentinH Looking better.... =)
It still appears as though you are trying to do one thing with .css() and another with ng-style?

The code lokka better but the behavior is not good unfortunately.
I know that doing everything with .CSS() and .html() is not the Angular way but at least we are sure that it works, it uses less $watch internally and we don't depend on the $digest cycle.
I didn't write this library at the beginning and when I started to modify it, it was also feeling weird to me to do all that work manually but now I understand that it was not due to a lack of Angular knowledge.
The codebase of this slider almost only uses Angular specific stuff to interact with the outside world. I think this is actually good because it should be easier to upgrade to Angular 2 it even reuse it with another framework.

Shrug. =) Up to you man.

Do you see the issue on the JSFiddle example?
Here's what I get:

Yup, I totally see the issue.
As I said above, I think that sync issue is related to you using .css() and ngstyle. Pick one, not both. =)

I just tried locally to only use ngStyle to handle colors and it's worse: sometimes the color is not updated whereas the model is.
You can see on the gif below, that when I change the colored slider, then another one, the first one only gets the other color when the second is modified. This shows that there is a $digest problem...

So unless somebody can explain how to fix this issue, I'm rolling back to god old css() way :/

OK there is a deeper problem about synchronisation of model update and color update. I'll look at it better tomorrow.

Wow, I can't imagine how that is happening. Each slider should be separate from the other slider and there should not be shared state between them.

Don't worry, they are separated but the angular $digest cycle is common to the whole application.
When I stop dragging the first slider, the scope value is 9 but it is still green because the scope value was still 10 when the getSelectionBarColor was called. Then when I moved the other one, it starts a new $digest and the getSelectionBarColor of the first one is called because its scope has changed compared to the last cycle.
The internal flow is quite complicated actually and it might need some.simplification.

Ok now it should be fine. I haven't refactor the internal flow yet (this is not only related to this PR) but  now the current values are passed to the getSelectionBarColor function so we are sure we are testing the latest one. I have updated the example to show this: http://jsfiddle.net/mtm2xhwx/

@ValentinH Made a small comment on two return statements in one of the commits above.

:+1: nice work.

I wonder if there is a way to apply css3 animations to make things a bit smoother... right now it feels like the slider just jumps between ticks... I wish it could slide more...

Actually if you uncomment those lines: https://github.com/rzajac/angularjs-slider/blob/master/src/rzslider.less#L79, it might do the job. I just remember that there was some wrong stuff with it but I haven't tried it for a while.

Here's an example: http://jsfiddle.net/gc3hgf0y/ .
It works well when you click on the bar but if you drag the slider then there is a weird behaviour...
Also transition doesn't work for the ticks because they are replaced at every frame by ng-repeat...

My last commit fixes the ticks issue by using a track by in ng-repeat so the dom element are just updated.

Yea... I just came here to tell you that... ;-)

This is much cooler, but it is a little bit weird now because it follows the mouse a bit when dragging...

Yes, this is why it is not added by default. But setting a very small value like 0.1s is quite cool actually.
So let's merge?

Works for me. Nice job.

On Dec 16, 2015, at 1:16 PM, Valentin Hervieu<EMAIL_ADDRESS>wrote:
Yes, this is why it is not added by default. But setting a very small value like 0.1s is quite cool actually.
So let's merge?
—
Reply to this email directly or view it on GitHub.


Copy API from phone feature

I can't type the API key because fat fingers 😭
Can you make some way to grab it from the phone? Maybe with a companion app?
Thanks in advance!

Hey @ThrownLemon,
I apologize for the inconvenience, i am working on the same and will be pushed in coming updates !


Tutorial fix

A few issues were observed with the tutorial in issue #454, This PR aims at fixing those issues.

Pull Request Test Coverage Report for Build 278792914

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

Totals

Change from base Build 278723488:
0.0%

Covered Lines:
5509

Relevant Lines:
6070

💛  - Coveralls


Base64 decode doesn't give correct result

Please see this small example: https://jsfiddle.net/j1rp55oo/
You can check that Base64.encode(...) works correctly by decoding it's output using OpenSSL or any other base64 encoder.

I'm not at a computer for the rest of the weekend nyt base64 encoding/decoding can be performed natively by browsers by using atob and btoa.

http://caniuse.com/#search=Atob

Thanks, I know about atob/btoa, but it's not supported in IE9, unfortuntely. So having working base64 encoding in jshashes still could be useful.

Yeah, that's a problem.

It should use i++ here: https://github.com/h2non/jshashes/blob/master/hashes.js#L331~L334 .
Then decode works for ASCII words.

We should update the input length after UTF-8 convert: https://github.com/h2non/jshashes/blob/master/hashes.js#L301.
Then encode works for UTF-8 words


Segment-Anything Integration?

Is it possible to use Segment-Anything to define the pictures of the source picture and the reference picture, will it be more convenient?

We are working on it


Remove resize and rotate example because it is misleading

The nodeResizer does not work with rotations and we will not support it.
We might want to encourage people to create their own simple resizers that work with rotation (maybe with handles changes their positions based on rotation as well) It's not that hard if you don't have to support everything possible with React Flow at the same time.

[ ] remove option to resize on the rotatable node
[ ] remove option to rotate on the resizable node
[ ] simply show labels "resizable" and "rotatable" on the nodes without the checkboxes


ci: ruby_memcheck reporting leaks on ruby-head

e.g., https://github.com/sparklemotion/nokogiri/actions/runs/5551865569/jobs/10138487549
I'm creating this issue primarily to communicate that this is a known problem.
Leaks are reported due to the problem with inline caches described at https://github.com/Shopify/ruby_memcheck/issues/23. This is the root cause of the failures recorded at https://github.com/sparklemotion/nokogiri/issues/2838.
Recently, the removal of the transient heap in ruby head is resulting in these leaks being reported much more frequently.
I've been in touch with Peter, the maintainer of ruby_memcheck, who's got some ideas for a Ruby feature that will free up all memory at shutdown, including the inline caches.
In the meantime, I may just add suppressions for Ruby 3.3.

Suppressions added in #2929

Context: https://github.com/Shopify/ruby_memcheck/issues/23#issuecomment-1640748018


Export and entry point

This is a small PR to improve the experience of NPM users:

the entry point has been moved from the browserified solver to the source code. This way it is possible to build (namely browserify) a project that uses the solver.
@JWally I just erased the environment export system to replace it with a simple module.exports. I did that because of a bug I had after I browserified the solver and some code that need it, together. And then I load everything through an html page. So could you check if the simple export works with all the environment you want to support ?

@JWally last point, after this patch, could you update the version on NPM please ?

@lvenerosy when I include ./src/solver.js in a browser, I don't appear to have visibility to solver.  Are you having success getting it in a browser? I haven't had my coffee yet, so I'm probably missing an obvious step somewhere 😜
Could you describe the issue you were having a little more?
Sorry I've been absent on this project for a while. I'll try and look at this some tonight (CST).
Thanks!
-JWW

I was a bit vague in my explanation but now the html include should work.


CI: Switch to self-hosted runner

Switch to my self-hosted runner instance, hopefully pkgcheck executes faster...?
Signed-off-by: Kirin Etheridge<EMAIL_ADDRESS>

Alright, I'm a bit confused at socket.gaierror: [Errno -3] Temporary failure in name resolution, seems like for some reason the runner can't access the internet.
I use docker-github-actions-runner, so this is ran inside a docker container... I already had network issues I fixed, but I'm not sure why it can't resolve IPs, docker execing into the container and running ping, it works?

Oh, I think I know why, took a bit to figure it out. I noticed I always have to use --network=host on my containers to resolve DNS issues, but never knew why they happen... I realized I use DNS on a private class IP for DNS in resolv.conf, while the Docker daemon uses a different one, so they can't communicate... Gonna test if changing docker's config makes this work in a bit >_>

Ight now I get a new error related to the PAT, cool

For the past hour I've tried to fix the docker network issues, I really think I forgot "Edit: Happens when I make a new runner without docker as well...". I feel like this is impossible as I use nftables (screw iptables I hate it), so back to the old setup, which gives the same network errors, ughhh

Okay, well, this will be kept as a draft until either if this is a bug I fix it, or until nftables with docker properly works (yes, I do have iptables-nft)

Alright, made everything network related work, had to hard-code an external DNS for the docker daemon but that's fine with me... New run gives a new error, but docker run arguments are different (entrypoint isn't used?), weird.

Alright, this should work now...