Move CRDs into CRD folder as by helm RFC

This ensure the CRDs are deployed first, before the helm chart (or a wrapper helm chart) creates any CRs.
Reference docs
https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#method-1-let-helm-do-it-for-you
Also  remove templates from crds.yaml since i expect labels to be of lesser value on CRDs then the proper deployment of those.

Needs to be discussed since the CRDs are yet requiring templates (labels). What is the actual use of labels on CRDs? Is there a real use for that - since this would be a blocker for deploying the CRDs the proper helm way

Any chance to get an answer on this. This PR might look big, but it is not - just moving files without touching them except:

remove ability to dynamically label CRDs since this would require gotpl which would block the usage of the CRD deployment

Thanks!

I believe we visited this before but had some concerns. cc @jenting

Thanks for your PR @EugenMayer
However, the helm does not handle the upgrade CRD if you put the CRD manifest into the crds/ folder.
That's why we keep the CRD manifest inside the template/ folder.

@jenting thanks for the follow up. Interesting point, i'am entirely unaware, could you explain or link docs for that?
AFAIU deploying the CRDs from the /crd folder on day,
a) then updating one CRD in /crd/foo.yml
b) adding a new CRD in crd/bar.yml
And then redeploying on existing chart installation will not reapply / change those CRDS / add that CRD?
Could you explain if both cases are problematic, a) and b)? Somewhat i assume a) to be problematic in itself and should most probably never be done. Rather using a new CRD with v2..v3 .. but if b) is not working etiher, i understand that this is a huge issue.
Thanks for you time!

@jenting thanks for the follow up. Interesting point, i'am entirely unaware, could you explain or link docs for that?

You could follow this issue https://github.com/longhorn/longhorn/pull/3224

AFAIU deploying the CRDs from the /crd folder on day,
a) then updating one CRD in /crd/foo.yml b) adding a new CRD in crd/bar.yml
And then redeploying on existing chart installation will not reapply / change those CRDS / add that CRD?

You could try add some change on one of the CRD yaml manifest like crds/foo.yml, and then run helm upgrade. You could see that the updated crds/foo.yml won't be reflected in the cluster.

Could you explain if both cases are problematic, a) and b)? Somewhat i assume a) to be problematic in itself and should most probably never be done. Rather using a new CRD with v2..v3 .. but if b) is not working etiher, i understand that this is a huge issue.
Thanks for you time!

Besides that, if you're interested in contributing the helm chart enhancement, please file the PR to here?
Because this repo is mirrored from the https://github.com/longhorn/longhorn/tree/master/chart, thank you.

That is a shocker. Is this really intended by helm chart or a known bug to be fixed? It really seems that the entire CRD topic has not be tough through in the deployment phase, just the API level / meta-model seems to be well designed and flexible. But when it comes to deploy them, within a helmchart for the app you are currently deploing, or even updating them, it all falls apart. Similar issues in terraform with the kubernetes_manifest resource.
So bottom line, using /crd is even worse then deploying CRDs as normal templates, since the former is not only not updateable (which i could understand, that deployed CRDs are immutable), but even adding new ones is not possible.
I guess opening a PR on https://github.com/longhorn/longhorn/tree/master/chart will not make any sense, i think your arguments really are more then good enough to use the "better worse". So from my POV, first of thank you for explaining, second i guess we can close the PR?

Yes, as far as I know, helm does not handle CRD upgrade in helm 2 or helm 3.
According to helm best practice, the best approach is to separate the CRD into another chart at day 1.
If it does not separate at day 1, the current way/workaround we found is to keep the CRD manifest inside the templates/ folder and helm install/upgrade will make sure all the resources inside the templates/ folder will be applied/updated.
And, we have to make sure the CR is applied/updated after the CRD manifests are applied/updated, we could achieve this by helm hooks. You could refer to how to tackle this for Longhorn v1.3.0 https://github.com/longhorn/longhorn/blob/251789f/chart/templates/customize-default-setting.yaml#L6-L7.
Last but not least, I think this PR could be closed because I tried this approach before but it's not working for helm upgrade not upgrading the CRD manifest.

Full ack, close anytime.

ref: exactly like @jenting described, https://github.com/helm/helm/issues/6581
Let's close this first, but thanks @EugenMayer contribution 👍


New Added support for node label alignment

This PR introduces a new feature allowing users to have greater control over the node label alignment in the graph by defining custom not label positions i.e. top, left, right, bottom

@tonilastre Can you pls review the PR.


Async cache fix in multi-thread-environment for GitHub downloads

No comments.

Hehe... first some solution and then more optimized. The last one looks quite nice.
let internal memoizeAsync f =
    let cache = System.Collections.Concurrent.ConcurrentDictionary<'a, System.Threading.Tasks.Task<'b>>()
    fun x -> // task.Result serialization to sync after done.
        cache.GetOrAdd(x, fun x -> f(x) |> Async.StartAsTask) |> Async.AwaitTask

thanks!


How DeduplicationHandler works?

Hello!
Have I understood correctly, that DeducpliationHandler lets all log messages through but deduplicates only messages that have level greater or equal to what has been set to DeduplicationHandler::$deduplicationLevel?
I.e. if I have set deduplication level to error and I have 10 debug messages and 10 error messages, all 10 debug messages are passed to wrapped handler and only 1 error message?
Br, Miika

Yeah the point is really to use this in combination with a FingersCrossedHandler IMO so that the dumping of debug info etc only happens when a deduplicated error reaches the FingersCrossedHandler. But this may depend on your use case.


Allow getting snapshots for periods where the CloudWatch Metrics aged out

I suspect a number of the Performance Insights CloudWatch Metrics are high-resolution ones that aren't available going further back compared to the rest of the snapshot data where I've configured a 31 day retention and can still get at it.
I'd rather have the snapshots without the CloudWatch Metrics that aren't unavailable rather than being blocked altogether.
  var cwData = await getCWMetrics(generalInformation)
  // Check if cloudWatch returns data
  if (cwData.MetricDataResults.find(v => v.Id === 'dbLoad').Values.length === 0) {
    console.log('No performance data available from CloudWatch for the selected timeframe. Please choose a timeframe with application load or user activity.')
    process.exit(1)
  }

I implemented something like this as a short-term workaround, though it's not ideal. It doesn't distinguish no data available from a true 0 value and could potentially confuse the compare report and AI analysis.
This would be a great official improvement that also is understood by the compare and AI analysis.
const handleNoPerformanceData = function() {
  console.warn('No performance data available from CloudWatch for the selected timeframe. Using default values.');
  return {
    MetricDataResults: [
      { Id: 'dbLoad', Values: [0] },
      { Id: 'auroraEstimatedSharedMemoryBytes', Values: [0] },
      { Id: 'networkThroughput', Values: [0] },
      { Id: 'storageNetworkThroughput', Values: [0] },
      { Id: 'writeThroughput', Values: [0] },
      { Id: 'engineUptime', Values: [0] },
      { Id: 'replicationSlotDiskUsage', Values: [0] },
      { Id: 'snapshotStorageUsed', Values: [0] },
      { Id: 'transactionLogsDiskUsage', Values: [0] },
      { Id: 'bufferCacheHitRatio', Values: [0] },
      { Id: 'volumeBytesUsed', Values: [0] }
    ]
  };
}

Agree, very good observation. We need to think how we can improve it. I will also test your suggestion.


Fix powershell build script

The OS built-in PowerShell complains when omitting those braces

Note: This PR is targeting the winui-build branch :-)

I did not notice any issues, but looks good.

I'm using the stock PS included with Win11, did you perhaps install the new cross-platform powershell?
Mine is:
PS c:\> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.22621.2506
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22621.2506
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

I have PS Core installed as well, but usually I take the stock one. Not sure though what I used back then. Anyways, if there was a problem, it's good that it's fixed now.


VACUUM doesn't create a commit in transaction log

When you vacuum a table with delta-rs, it  doesn't create a commit in the delta transaction log, stating the VACUUM operation.

Does it do that in Spark? I wouldn't expect it to create a transaction.

@wjones127 apparently they do since this PR: https://github.com/delta-io/delta/pull/1552

@wjones127 I checked my tables written with databricks, and they all have the VACUUM START/END commits


feat: Make containerd restart its own patch

We have multiple patches that configure containerd. We need to restart containerd after all such patches, and we should restart it exactly once.
Although we've been configuring http proxy for the containerd systemd service, we haven't been restarting it, and I believe the configuration wasn't being applied. This PR should fix that, too.
NOTE: This depends on #16 and #17, because the tests fail otherwise. Do not merge until both of these have merged.

fixed unit test and rebased: https://github.com/d2iq-labs/cluster-api-runtime-extensions-nutanix/commit/17ce0a8a9abc9d79ae67336bd592b2bda744859b

rebased from main after https://github.com/d2iq-labs/cluster-api-runtime-extensions-nutanix/pull/32 was merged.


Add set_cursor_position method to Window

This allows programatically setting cursor position in Window.

OK. I added error! calls, but I don't feel like adding the whole Events<WindowError> mechanics just to handle these unlikely errors. ;-)

I can do that if you want, as a different pr


Programmatically change visible options

Hi there! Thanks for the library!
Is there a way to programmatically add/remove or at least show/hide each individual FabOption?
I see it's possible to enable/disable which is good (but not what I need), while unfortunately the parent's show/hide methods do not seem to have any effect, as the view visibility seems to be ignored as well.

Hey @nasuellia,
Unfortunately that's not a current feature of the expandable-fab library. The easiest way to achieve what your looking for would probably be to follow the guidance of this answer.
This functionality has been asked for quite a bit though, so I think I'm going to add it to the library soon in a future update.

Hey @nasuellia,
Good news, the feature you requested has been added to the library! Please use version v1.1.1 now available on Maven Central to access the feature (or clone the latest changes from the master branch on this repo).

Closing due to inactivity.


Minor: render mermaid to image before publish?

Additional context
npmjs.com doesn't support mermaid rendering, but I and maybe others go to npmjs.com/package/packagename when I can't remember who authored it.
GitHub:

npmjs:

Not sure if there's an easy mermaid->svg/image renderer out there that could be run in prepack or something, but if there is it might be worth considering. Or just a link to github for that section - it's a v useful diagram! Feel free to close if that sounds like more effort than it's worth though.

I would consider this is something NPM should improve. I think prepacking is too much complexity just for this. To workaround it, I think we could add a link to say, "The follow graph does not renderer? Check it on GitHub". If that sounds good to you, would you like to send a quick PR? Thanks


Possible dispatch group issue?

We're seeing an issue in one of our apps related to dispatch groups. I don't know that it's ProcedureKit related, but we don't use dispatch groups in this app, and it looks like some of the ProcedureKit code that execution is passing through here does. Here's the stack trace, most of which is complier-generated:
Application Specific Information:
BUG IN CLIENT OF LIBDISPATCH: Unbalanced call to dispatch_group_leave()

Thread 0 name:  Dispatch queue: com.apple.main-thread
Thread 0 Crashed:
0   libdispatch.dylib       0x000000018770fd24 dispatch_group_leave$VARIANT$mp + 144
1   <APP>                   0x0000000100ad3abc __95-[redacted]_block_invoke + 899772 ([redacted].m:235)
2   <APP>                   0x0000000100c4beac partial apply for thunk for @escaping @callee_unowned @convention(block) (@unowned Procedure, @unowned NSError?) -> () + 2440876 (<compiler-generated>:0)
3   <APP>                   0x0000000100c4bf1c partial apply for closure #1 in closure #1 in Procedure.continueOnMainQueue(withBlock:) + 2440988 (<compiler-generated>:0)
4   <APP>                   0x0000000100cdccec thunk for @escaping @callee_guaranteed () -> () + 3034348 (<compiler-generated>:0)
5   libdispatch.dylib       0x000000018776d6c8 _dispatch_call_block_and_release + 24

The method Procedure.continueOnMainQueue(withBlock:) that appears in the stack trace is a helper extension that I wrote to allow completion handlers to be added from Objective-C code, and it looks like this:
  public func continueOnMainQueue(withBlock block: @escaping CompletionHandler) {
    self.addObserver(DidFinishObserver(didFinish: { (op, error) in
      DispatchQueue.main.async {
        block(op, op.interoperableError)
      }
    }))
  }

Because this whole part of the app is a big stack of callback functions, the redacted method call there looks like this (with completion being a block passed in to this method). The line number in the stack trace points to the last line of this snippet:
    [operation continueOnMainQueueWithBlock:^(Procedure *op, NSError *error2) {
      if( nil == error2 ) {
        completion(UIBackgroundFetchResultNewData, nil);
      }
      else {
        completion(UIBackgroundFetchResultFailed, error2);
      }
    }]; // STACK TRACE POINTS HERE

I know it's kind of a long shot, but does any of this seem familiar? (Or obviously wrong?)

OK, this looks like a false alarm. Found usage of dispatch groups in another of our dependencies that appears to be the culprit (along with a code path where one of these completion handlers can get called twice).
Sorry for the noise!


sendPing and onPong

I was wondering if it would be possible to add the possibility for custom ping and pong handeling. This has been discussed quite a few times in the past. (Most issues I found about this topic are a few years old by now)
Custom ping/pong handeling would add support for a few uses cases that are for example mentioned in the WebSockets
Living Standard section 5. Specifically:

User agents may send ping and unsolicited pong frames as desired, for example in an attempt to maintain local network NAT mappings, to detect failed connections, or to display latency metrics to the user.

There was a PR (#6730) about a year ago that suggested an implementation for better ping/pong support. In the replies this got from maintainers it seems like there was genuine interest in improving/offering better support for these use-cases. Sadly though, the suggested changes didn't seem like the right way to go and the PR was closed for the time being. The main pain point was a breaking API change it introduced. I agree with that decision and in my opinion the suggested changes in that PR still lack an easy and intuitive api for better control over ping/pong.
I want to suggest 4 functions with the following signatures:
For WebSocket:

sendPing(text: String): Boolean
sendPing(bytes: ByteString): Boolean

For WebSocketListener:

onPong(webSocket: WebSocket, text: String)
onPong(webSocket: WebSocket, text: ByteString)

People who are currently using pings through the built in pingIntervalMillis can continue to use that with no problems. People who need fine grained support for ping and pong messages can use this new API also with no problems. This solution would not make any breaking changes to the current API. It provides a clean and intuitive way for developers to take advantage of the ping and pong functionality introduced by RFC 6455.
I would be happy to help with the implementation. Looking forward to your feedback.

What's your use case? We intend to keep pings for connection health and not for application layer use.

As mentioned one use for it is measuring latency. Also most other WebSocket interfaces expose the ping/pong functionality in a way like this instead of having it be handled exclusively internally by only providing an interval you can set. It would also provide easier interoperability with WebSocket implementations written for the server. This feature would be especially useful when you are writing a client for an existing server that is not under your control. For example when this server needs to interact with multiple different client implementations. It will likely make use of these features defined in the spec. It might also have specific requirements for clients regarding pings and pongs.
After thinking about it a bit more I think there should also be methods for onPing and sendPong added to support all the uses cases that the spec has to offer.
I think that adding this will be useful for a bunch of different use-cases but I will now explain my specific reason for requesting this feature. I realise that my reasoning for wanting this is not enough on its own but, there are a lot of other use-cases like I explained above that would greatly benefit from having this added. Also hopefully if this won't be added you might be able to give advice for this specific case that I am about to describe.
React native currently uses OkHttp for its android implementation of their 'cross platform' WebSocket api. Due to the fact that OkHttp does not expose methods for ping/pong this is also not possible to do when writing apps with react native. When looking at other implementations of WebSockets (on iOS or server implementations) this method of only being able to set a ping interval is quite unusual.
What are your thoughts?

Browsers don't offer pings and pongs and that makes me reluctant to offer them in OkHttp.

I can understand that but, I think it would be great for many people if these features would be considered.
Browsers and the WebSockets Living Standard do indeed not offer pings and pongs. They also don't offer an option to set a pingInterval or any internal ping support and yet OkHttp does.
Also the standard mentions that pings and pongs are not exposed in the api but does not give a reason why. Actually the standard does the opposite. It mentions reasons why it would be good to expose this functionality (even though it does not).

I don't think any of the reasons listed are exposing these to clients. The user agent here is OkHttp (debatable).
"User agents may send ping and unsolicited pong frames as desired, for example in an attempt to maintain local network NAT mappings, to detect failed connections, or to display latency metrics to the user."
It's way easier to commit to additional public APIs, when they follow existing dominant conventions by browsers. When they don't we risk  implementing features that only work in the most carefully controlled environments, and suddenly stop working when a server changes it behaviour.

Closing, rather than keeping open hope that we will eventually get to this.

Hello! Circling back on this topic - Is this something that the team would care to revisit? I +1 the benefit of exposing a way for application-level ping/pong-ing
There is a current gap (at least to my current understanding) in natively checking for connection health of a given websocket connection via okhttp. At the moment, there seems to be no direct way to understand if a connection is truly healthy (at the app-level) aside from heartbeating/ping-pong'ing. This seems like trivial functionality to expose on top of the background mechanism via the pingInterval() configuration.
Ideally:

We should be able to setup a pingInterval() at the configuration level for periodic/background heartbeating
We should be able to manually heartbeat/ping-pong (and handle the response or lack-thereof) via some exposed methods potentially as originally proposed above by @jobpaardekooper.

@MarioBolivar would you consider doing heartbeat messages at the application layer instead? That way it’ll also work with web browsers which also don’t offer user control to these internals.


fix updateMetadata with new Metadata DAO;

@hmunfru @jframos

Coverage remained the same at 47.0% when pulling af450f5f454e579a8a85279f0a51e791f8daec32 on jesuspg:updatemetadatas into e0afbc1be01e264db362fdc4e8160e09905f6c68 on telefonicaid:develop.

LGTM

LGTM

LGTM
(To be tested in the Experimentation Environment)


Error in uploading multiple images! how to upload multiple images?

I was creating a upload form which sends multiple images to API. But i'm getting error whenever i'm attaching more than 1 image.
Single file upload with data is working but multiple files with data are not working.
Here is the piece of code:
RNFetchBlob.fetch('POST', 'https://www.test.test/api/v1/test.php', {
      'Content-Type' : 'multipart/form-data',
    }, [
      { name : 'image1', data: this.state.uploadFiles[0].uri},
      { name : 'image2', data: this.state.uploadFiles[1].uri},
      { name : 'image3', data: this.state.uploadFiles[2].uri},
      { name : 'image4', data: this.state.uploadFiles[3].uri},
      { name : 'image5', data: this.state.uploadFiles[4].uri},
      { name : 'image1_content', data: String(this.state.imageContent1)},
      { name : 'image2_content', data: String(this.state.imageContent2)},
      { name : 'image3_content', data: String(this.state.imageContent3)},
      { name : 'image4_content', data: String(this.state.imageContent4)},
      { name : 'image5_content', data: String(this.state.imageContent5)},
      { name : 'id', data: String(this.state.id)},
      { name : 'main_content', data: String(this.state.content)}
    ]).uploadProgress((written, total) => {
        console.log('uploaded', written / total);

    })
    .then((resp) => resp.json())
    .then((responseJson) => {
      console.log(responseJson);
    })
    .catch((err) => {
      console.log(err);
    })

I always get this error whenever i upload multiple files:

SyntaxError: Unexpected token < in JSON at position 0

@mdmush , does it happen on both Android and IOS ? From the code snippet I suppose the image content are encoded into BASE64 string ? If you're going to upload files using their URI, you'll need filename and also use RNFetchBlob.wrap to wrap the URI.

Actually i was testing on Android device for now. And yeah, it is BASE64 string.
Thanks for the solution. I've added filename and wrapped the URI with RNFetchBlob.wrap.
Everything works fine on my end, but now the problem is i'm not getting any data in API. The data remains null. So tell me what to do?

@mdmush , thanks for the information. I have just test your code with in my end. I and it works fine. Could you provide raw request body from server ? It'd be helpful.

Okkay.
here is my server code:
`<?
define('UPLOAD_DIR', 'content/');
$final= array();
$img1 = $_REQUEST['image1'];
$img2 = $_REQUEST['image2'];
$img3 = $_REQUEST['image3'];
$img4 = $_REQUEST['image4'];
$img5 = $_REQUEST['image5'];
$final["img1"] = $img1;
$final["img2"] = $img2;
$final["img3"] = $img3;
$final["img4"] = $img4;
$final["img5"] = $img5;
echo(json_encode($final));
?>`
Right now, i'm just checking if the data is reaching to the API or not. If this is wrong, please send me the code you tested.

there?

FYI, if you need to post files as array in PHP, you need to specify your arguments as follows:
[
  { name: 'image[]', filename: '...', data: '...' },
  { name: 'image[]', filename: '...', data: '...' },
  // ...
]

In PHP, you'll receive and get something like this:
$files = $_FILES['image'];
$filename1 = $files['name'][0];
$filename2 = $files['name'][1]; // $tmp2 = $files['tmp_name'][1];
// ... so on
// ... or better use foreach

This is how we process $_FILES when uploading multiple files in the browser so might be more compatible if we send/accept the same way.
Hope this helps someone somehow 😄

My question is same but my images are dynamic like if user select 1 image then only one image has to be post in API and so on. I am creating array in state for Images. any dynamic solution for this ???
here is my code.
ImagePicker.showImagePicker(options, (response) => {
                console.log('Response = ', response);

                if (response.didCancel) {
                    console.log('User cancelled image picker');
                }
                else if (response.error) {
                    console.log('ImagePicker Error: ', response.error);
                }
                else if (response.customButton) {
                    console.log('User tapped custom button: ', response.customButton);
                }
                else {
                    let source = { uri: response.uri };
                    setTimeout(() => {
                        this.setState({
                            Images: [...this.state.Images, response.data]
                        }); // set images array in state I want to push that array in API
                    }, 300);
                }
            });

I finally find a way to do it 💯
here is the code may it help someone somehow 😄
RNFetchBlob.fetch('POST', '***ur-url***', {
                        'Content-Type': 'multipart/form-data',
                    }, [
                            { name: 'image[]', filename: 'image1.png', type: 'image/png', data: this.state.Images[0] },
                            { name: 'image[]', filename: 'image2.png', type: 'image/png', data: this.state.Images[1] },
                            { name: 'image[]', filename: 'image3.png', type: 'image/png', data: this.state.Images[2] },
                            {
                                name: 'formData', data: JSON.stringify([{
                                    "plant": selectedPlant,
                                    "description": description,
                                    "user_id": userID,
                                    "plant_id": selectedPlantID,
                                    "method": "submitData"
                                }])
                            }
                        ]).then((response) => response.json())
                        .then((responseData) => {

                            if (responseData.status) {
                                this.showAlert('Success', 'Data Submitted Successfully');
                                setTimeout(() => {
                                    this.setState({
                                        description: '',
                                        selectedPlant: '',
                                        Images: [],
                                        loadCat: false
                                    })
                                }, 300);
                            } else {
                                this.showAlert('Error', responseData.message);
                            }
                            // ...
                        }).catch((err) => {
                            // ...
                        })


Owner authority shouldn't be required for voting / changing memo key

Current GUI always includes owner and active fields when crafting account_update_operation, which is usually used for voting. However those two fields are optional by design. If one of them is set, the operation requires owner authority. Otherwise the operation requires active authority only.

Update: according to https://github.com/cryptonomex/graphene/issues/556, change of active authorities doesn't require owner authorities any more.


Possibility to look in node_modules for abi files

It would be great to have the possibility to specify one or more npm packages where typechain will look for abi's.
Until now, we have different projects for DAPP and contracts and include the abis as an npm package to make it easier to reference the contracts, instead of pointing outside the project. Makes it less a hassle when working with CI tools to just package the contracts and include them in node_modules. Its also more adaptable when on different systems/machines etc.
I am somehow not able to put an enhancement tag to this issue so please forgive me that.

I think you should be able to accomplish it by using custom glob as mentioned in readme:
typechain "./node_modules/(packageA|packageB)" (we use https://www.npmjs.com/package/glob)
Have you tried it?

I can confirm that it works. Just specify outDir to push all contracts in one place:
typechain --outDir app/lib/contracts "./node_modules/neufund-contracts/build/contracts/*.json"


CNI and calico folder creation in ExecStartPre (1.11)

/cc @databus23

Hi @onuryilmaz, sorry I already fixed this and had a commit in my local master branch. I somehow forgot to push.
I did that no so this is already fixed.

See here: https://github.com/sapcc/kubernikus/commit/7d6d3bab13801c6504df16a4071c73224980bd6a


FI-1619 - Readd resource mismatch but downgrade to informational.

Based on further discussion with ONC, it is a good idea to provide an 'informational' message when there is an unexpected resource type returned from a search.  See #46 where we removed the check.  The reasoning is that it seems that likely they shouldn't be doing this, even if they are technically allowed, so its nice to point out that we see this happening (without implying they are wring, as a warning would).  Also, I can see us reviewing this sometime in the future and not realizing that we decided to remove it, and by mistake just readding it... by having this in there we will know that we intentionally aren't failing systems for type mismatches.
To test, the easiest thing to do is add the following line at 535, in fetch_all_bundled_resources in search_test.rb, to overwrite valid_resource_types to something wrong (for other than the Patient resource):
      valid_resource_types = ['Patient', 'OperationOutcome']

Then, run US Core v3.1.1 test suite, with the reference server as a preset, and run any of the Profile searches (except Patient).
Then, you'll see the following informational start showing up:

Code looks good. One question, do we need to add option to enable/disable Info messages?

I'm almost positive that we do not suppress info messages in g10 that come out of tests.  We only suppress info messages coming out of the validator, if that's what you mean.  For example, we show the informational messages that come out of the TLS test kit when the TLS tests are run.


A suggestion to modify the updateSegmentation.sh

If we make the assumption that the input text is going to be UTF-8 file format, I think that The following needs to be modified from
https://github.com/goodatlas/zeroth/blob/3cbb45c0dd41476e6d6278afbc84586f32cb1500/s5/local/updateSegmentation.sh#L30
	morfessor -l $lmDir/zeroth_morfessor.seg -T - -o - \
to
	morfessor -e 'utf-8' -l $lmDir/zeroth_morfessor.seg -T - -o - \
This is because morfessor defaults to the assumption that the text is ASCII.

Thank you, @akshayc11. yes, I think this is good point.
According to the morfessor's help, they said.
-e <encoding>, --encoding <encoding>
    encoding of input and output files (if none is given, both the local encoding and UTF-8 are tried)

so that's why I provide locale setup at https://github.com/goodatlas/zeroth/wiki/Requirements
Anyway, I'll do a test for sure.
And If it is possible to you, plz make an official PR.


Allow for a configurable mapping of fields

Allow a YAML configuration that defines field mappings.
This is useful when using KGX with a source that might not be completely biolink-model compliant. For example, reading Dipper TTLs from Monarch.

I think there's a lot of potential for this particular feature. What would be involved in implementing it?

Implementation is fairly easy.
But this requires a discussion and I would like @cmungall to chime in regarding this functionality.
Right now, KGX tries to check for BioLink Model compliance for KGs that it tries to read from.
Implementing this feature means we are relaxing that check for compliance, which may or may not be desirable.


Test against Python 3.3,3.4,3.5,3.6

The tests currently fail because Python 2.6 has reached end-of-life. So let's stop testing against Python 2.6.
Since this supports Python 3.x, test against it too.

While I value your willingness to contribute, unfortunately I cannot accept this pull request (at least not as a whole). There are two main reasons.
Firstly, there is a purpose for cElementTree being in the tests: the comments mention that once upon a time, it caused a warning message. It is a regression test, put there specifically to ensure that some particular bug doesn't turn up again. You can't just ignore cElementTree under Python 3 because it doesn't install, because then we won't know it if the bug returns under Python 3. The only proper way to fix the test for Python 3, is to research why cElementTree was added in the first place and then decide what to do next. We may find either that the regression test is not relevant anymore, in which case we can remove it entirely, or that the test is still relevant, in which case we need to find a replacement for cElementTree which does install under Python 3. See also 9c6a53a. (If this seems overly conservative to you, please read this).
Secondly, pip-review is a remnant of an old version of the pip-tools package. Many of its current users probably know it from that era, when Python 2.6 was still fully alive. Some of those users might still be stuck with Python 2.6 for other reasons than laziness (see e.g. #2). Right now, pip-review still works under Python 2.6 without requiring any additional effort. As long as this is the case, I want to see it from the tests, so those users who are stuck in the past aren't needlessly left in the dark. I also explained this in #15. Concluding, it is too early to remove Python 2.6 from the tests. The problem with Python 2.6 should instead be solved by adding an optional line to the test so that the pip warning message about Python 2.6 isn't unexpected anymore.
I do think that some of the changes you propose would be useful. I would be grateful if you could submit those changes in a separate, smaller pull request. Please see my comments in the "Files changed" tab for some details.

You're right, I didn't put much thought into the cElementTree test. I've now looked into it more, and it seems to come from this commit: https://github.com/jgonggrijp/pip-review/commit/243cc0cf74b047565a3ec2d727a68dc5758c5959
This commit was inherited from pip-tools, and it seemed to have been trying to fix this issue: https://github.com/nvie/pip-tools/pull/9
However, I can't get pip freeze to print out that comment even under Python 2.6 and pip 1.5.6.
Interestingly, review.t used to contain these lines:
Warning: cannot find svn location for cElementTree==1.0.5-20051216
cElementTree==* is available (you have 1.0.5-20051216) (glob)

However, this commit changed that: https://github.com/jgonggrijp/pip-review/commit/fe420151da84bf1ff71f87cf88d9614f2dcf567f
So it seems that this test is no longer testing what is supposed to be testing anyway, under Python 2 or  Python 3.
That's as far as I've got time to investigate, but I think tests should be enabled for Python 3.x in any case.

As for supported an unsupported legacy Python version (2.6), that's your choice.

Thank you for the valuable research. I'll look into it further.


Refactor lint

When using a Task we can't reference the atom module or global property from within the child process.
This removes references to atom from the lint function, including references to the text editor

Looks good 👍


Integrate the latest version of JS-POLO

Description
The current integration of JS-MOI-SDK within our project encounters an exception when attempting to depolarize an empty Map. The resolution to this is upgrading new JS-POLO to a latest version
Logs
Error: load convert fail: not a compound wire
at ReadBuffer.load (readbuffer.js:49:1)
at Depolorizer.newLoadDepolorizer (depolorizer.js:78:1)
at Depolorizer.depolorizeMap (depolorizer.js:296:1)
at Depolorizer.depolorize (depolorizer.js:109:1)
at depolorizer.js:349:1
at Array.forEach (<anonymous>)
at Depolorizer.depolorizeStruct (depolorizer.js:346:1)
at Depolorizer.depolorize (depolorizer.js:111:1)
at Depolorizer.depolorizeArray (depolorizer.js:223:1)

closed in #49


Revert continuation indent to be 4

Task/Issue URL: https://app.asana.com/0/414730916066338/1201492317527942/f
Description
In this task we make some improvements but changed some of the IDE configs we have before, like the continuation indent that went from 4 -> 8
This PR reverts that value to be 4
Steps to test this PR

pull this branch
Open the IDE
verify that continuation indents are not massive anymore

Current dependencies on/for this PR:

develop

PR #1616  (this PR)

PR #1617

This comment was auto-generated by Graphite and will continue to be automatically updated while this PR remains open.


fix for missing endpoint in config

Issue #, if available:
When updating the endpoint for pinpoint, the endpoint is missing
Description of changes:
This is because we didn't add that _ensureCredentials into the updateEndpoint function.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #373 into master will decrease coverage by 0.06%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #373      +/-   ##
==========================================
- Coverage    89.4%   89.33%   -0.07%
==========================================
  Files          54       54
  Lines        2595     2597       +2
  Branches      530      531       +1
==========================================
  Hits         2320     2320
- Misses        264      266       +2
  Partials       11       11

Impacted Files
Coverage Δ

packages/aws-amplify/src/Analytics/Analytics.ts
63.69% <33.33%> (-0.77%)
: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 f443eae...53d9f64. Read the comment docs.


Android Studio project no longer builds:

Since the recent changes, I can no longer build my Android project. I get this error "SDL2_mixer depends on undefined modules: modplug".
I'm not sure what to do. It seems the modplug library is missing from the mixer source code? is this because it's changed to a github sub-project, so I have to add it manually now? Where do I save it to, what directory name? Do I need to edit my .mk files to do this? I download the .zip file of the code and save it to a specific directory, I don't use any github software to pull the source code.

I downloaded the modplug source and managed to get it to build, but now I get another error in logcat "Couldn't find function SDL_main in library".
This all worked no problem before the recent changes to mixer.

Did you run download.sh in the external directory?
There shouldn't be any relation to SDL_main in SDL_mixer. I'm not sure what's happening there. Do you have a link to your project available?

I can't share unfortunately because it's a commercial game. I'm currently dissecting the project and I think I know what's gone wrong and it's probably not SDL's fault. Will update when I've found the problem.

Sorted it, it's not SDL - sorry! The sub-project github link for modplug made me chase down a rabbit hole.

I did notice something else. Do we no longer need to @Override getLibraries() to add "SDL2_mixer"? Seems to work without it now?

I did notice something else. Do we no longer need to @OverRide getLibraries() to add "SDL2_mixer"? Seems to work without it now?

I think you do need to override getLibraries(). If you're not seeing that, it's worth investigating why that is.


Suggestions/Snippets stopped working from 4.4.0

Hi! suggestions have stopped working since last update (rafc), with 4.3.2 now it works perfectly. Im using vs code on ubuntu 20.04, i have also emmet with "javascript: javascriptreact" enabled from last days if it helps.
Im been changing some emmet options without result from another issues that I have seen here.
Thanks for your awesome work!

Same problem for me too. Suggestions and snippets no longer work for me either on Windows 10, VsCode v1.64.2, Extension v4.4.0

@rpayeras should be fixed in v4.4.3


Why doesn't Protocol use app.config to get its settings

Sanic passes all settings such as REQUEST_MAX_SIZE as arguments via a chain of functions from run_server to Protocol constructor, rather than accessing them via app.config where needed. Is there a purpose to this?

I suppose this could be removed in a future PR if anyone doesn't have objections to that. The APIs affected by this seem to be internal, as app methods such as create_server do not take these arguments.

@Tronic the reason Protocol doesn't use app.config is simple, the app.config feature didn't exist when the run_server and Protocol constructor code was written in May 2016.

That should now be sorted out. It's a large and quickly done patch, though, so I would suggest careful review. I did not remove the loop arguments because that would require more work and has potential to break apps that do weird things with multiple loops.


[REFACTOR]: Read-only on complex type

Summary
I have some cases where ESLint throws warnings Parameter should be a read only type.(@typescript-eslint/prefer-readonly-parameter-types). Nonetheless, some of these warnings can't be fixed (In my knowledge).
Motivation
This is a simple example of this warning impossible to fix :
import React from 'react';

interface IMyState {
    anchorEl: HTMLElement | null;
    isOpen: boolean;
}

export default function MyComponent(): JSX.Element {

    const [OPEN_AND_ANCHOR, SetOpenAndAnchor] = React.useState<IMyState>({
        anchorEl:null;
        isOpen:false;
    });

    const MyFunction = (event: React.MouseEvent<HTMLElement, MouseEvent>):void =>{
      ...OPEN_AND_ANCHOR,
      anchorEl: event.currentTarget,
      isOpen: !OPEN_AND_ANCHOR.title.isOpen,
    }

    return(
        <Menu open={} anchor={} onClose={}>
            <MenuItem>Hello</MenuItem>
        </Menu>
    )
}

Implementation
No response
Checks

[X] I am running the latest version
[X] I checked the documentation and found no answer
[X] I checked to make sure that this issue has not already been filed
[X] I'm reporting the issue to the correct repository

Hi, I have the very same problem on radon-comment with a recursive type. I think this problem does not come from our side.
I think we might want to disable this rule.


MINIFI-261: Allow std::string to be used for log messages

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

reviewing

changes and tests look good.  definitely like the convenience.  will get this merged.


Add form validation

Eg give feedback if the email signup input isn't a valid email, or if not all the required fields are filled in for the GraphQL Support request

Interesting, I did the email signup, then only my first name on the Support request, but the request still worked because the analytics remembered my browser's email.


swift 2.1 support

the code is broken on the latest version of swift (2.1).
it gives an error at this line:
bv!.setTranslatesAutoresizingMaskIntoConstraints(false)
please add support for swift 2.1

@uknowmeright thanks for addressing this. I haven't been doing a great job of maintaining this repo but, I'll make sure I get this issue sorted out in the coming days.

@uknowmeright I can't seem to replicate this error. If you're still using version 0.2.1 then I would try updating to version 0.3.0. That version should have swift 2.0 and 2.1 compatibility. If your already on the latest version would you mind posting the full error your getting? In the meantime I'll update the README to reflect instructions for the latest version. Thanks!

I'm closing this issue because these errors should not persist in version 0.3.0.


💡 [Feature]: Refactor action naming

🎯 Aim of the feature
Due to new actions that will come, like the upgrade app action, which may be confusing with the current upgrade project action we should rename the current Actions to make the naming more descriptive.
I propose the following

The naming on the UI may be corrected in https://github.com/pnp/vscode-viva/blob/main/src/panels/CommandPanel.ts in actionsTreeView method
We should also align/correct the name of the equivalent VS Code commands for each of those actions and those we may find in the package.json
📷 Images (if possible) with expected result
No response
🤔 Additional remarks or comments
No response

@Adam-it can you please assign it to me?

@Adam-it can you please assign it to me?

All yours 👍.
You Rock 🤩

@maximkol how's it going?
I noticed your PR still requires some updates.
Let me know if you need any help 👍? Don't be shy, we are all here to help you succeed and learn and help us 😊.
Also if anything changed and you are unable to commit more time on this issue also please be honest and lets us know so we may unblock this and maybe someone else would like to take it over 👍. No problem if anything changed, totally understandable 😊.
Have an awesome day!
Cheers

@maximkol I finished of this change if a separate commit
https://github.com/pnp/vscode-viva/commit/c03bd3f91f47e29dea81305d136d661b5a4c70df
Thank you for taking an interest in this repository 👍
You Rock 🤩


Scalar mul could conditionally negate instead of subtract

An idea that I wasn't yet able to explore when working on scalar multiplication in o1js is the following:
Instead of

representing the scalar as (t_is_odd, t_high)
and scaling by 2*t_high + 1 + 2^255
and conditionally subtracting to get t + 2^255

we could

represent the scalar as (t, t_negate), where 2*t + 1 + 2^255 < 2^254, and we might negate the whole expression to enforce that condition
scale by 2*t + 1 + 2^255
and in the end, conditionally negate the result

This should be more efficient because negation is cheaper than curve addition, and conditionally picking a field element (the y coordinate for negation) is cheaper than conditionally picking a point

I had completely forgot this, lovely idea, but indeed low priority


svg xlink:href causes error but should not

Example SVG:
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
  <g>
    <a xlink:href="#anchor">
      <text>Link</text>
    </a>
  </g>
</svg>

Links cause the following error

WCAG2A.Principle4.Guideline4_1.4_1_2.H91.A.Placeholder

I would expect this to understand the namespace and find the href behind xlink.
Thank you :)

This Grunt task integrates of AccessSniff to perform the validation. You can ask for improving the Grunt integration here.
If you want to fix or change the validation engine, you need to open an issue for HTML_CodeSniffer, which used internally by AccessSniff.

closing - out of date


Add Cool Custom Hooks

If you have a cool idea about a useful generative ai utility offered through custom react hooks, send in a PR.

I added a pull request--  Create useStoryGeneration.js #2
Kindly look into it. Thank You.


istio-proxy sidecar slow to drain on SIGTERM

Bug Description
Thanks in advanced 🙏
Background
We have recently updated our mesh to have
    defaultHttpRetryPolicy:
      retryOn: connect-failure,refused-stream

To prevent non-idempotent retries.
Problem
We are now seeing cases where a pod responds UNAVAILABLE in the case where the Envoy is up but the serving container is down (due to pod shutdown for upgrade/deployment).
The SIGTERM signals hit the containers at similar times (within 1-2ms), but Envoy-proxy takes a long time from SIGTERM to actually log that it is draining, even though --drain-strategy immediate is passed to Envoy.
It appears Envoy handles signals asynchronously:

So the reason is that dispatcher handles signals asynchronously. it generates event to dispatch the queue and then returns so that the following signals can be handled
https://github.com/envoyproxy/envoy/issues/15467#issuecomment-804237539

so it seems there is unpredictable duration between the kernel sending SIGTERM and when envoy decides to deal with it.
From "Agent draining Proxy" (SIGTERM)
To "Envoy proxy is NOT ready: server is not live, current state is: DRAINING" (our only signal that the Envoy is in fact draining)
We see delays of more than 1s.
Here is a sanitized example:

This violates our expectations: we would like the istio-proxy to go into DRAINING really "immediately"
Solutions
We are unsure how to best approach - we can see two avenues:

Slower workload binary shut down: Artificially slow down the SIGTERM to binary shutdown in our containers by 1s or 2s - then it would shut down after istio-proxy has gone into draining mode.
Pros: in our control
Cons: total hack

Earlier term to Istio-proxy: Kubernetes pre-shutdown hook to do TERM against istio-proxy earlier
https://www.envoyproxy.io/docs/envoy/latest/operations/admin#post--drain_listeners?graceful
https://cloud.google.com/blog/products/containers-kubernetes/kubernetes-best-practices-terminating-with-grace
Pros: probably the "complete solution"
Cons: should be implemented by Istio? ALSO: Would this also terminate the Forward proxy for downstream requests from the workload while it is still operating? The istio-proxy is both the reverse and forward proxy for the workload.

Something else?

Thank you!
Version
$ istioctl version
client version: 1.15.0
control plane version: 1.15.1
data plane version: 1.15.1 (xxx proxies)

$ kubectl version --short
Client Version: v1.25.1
Kustomize Version: v4.5.7
Server Version: v1.22.15-gke.100

### Additional Information

_No response_

There should be no expectation that in a distributed system anything happens immediately. The solution is to have your app continue handling requests, you shouldn't exit on immediate SIGTERM. You would have the same issue without Istio

Thanks- 100% - draining is entirely up to the mesh. Sorry for the noise.


Convert resnet50_v1 MXnet to IR

Hello, I'm trying to convert from ResNet50_v1 to the IR using this command:
mmtoir -f mxnet -n resnet-50-symbol.json -w resnet50_v1-0000.params -d resnet50v1 --inputShape 3 224 224
But I'm getting this error:

/usr/local/lib/python3.5/dist-packages/mxnet/module/base_module.py:56: UserWarning: You created Module with Module(..., label_names=['softmax_label']) but input with name 'softmax_label' is not found in symbol.list_arguments(). Did you mean one of:
data
warnings.warn(msg)
Warning: MXNet Parser has not supported operator null with name data.
Warning: convert the null operator with name [data] into input layer.
Traceback (most recent call last):
File "/usr/local/bin/mmtoir", line 11, in
sys.exit(_main())
File "/usr/local/lib/python3.5/dist-packages/mmdnn/conversion/_script/convertToIR.py", line 184, in _main
ret = _convert(args)
File "/usr/local/lib/python3.5/dist-packages/mmdnn/conversion/_script/convertToIR.py", line 102, in _convert
parser.run(args.dstPath)
File "/usr/local/lib/python3.5/dist-packages/mmdnn/conversion/common/DataStructure/parser.py", line 22, in run
self.gen_IR()
File "/usr/local/lib/python3.5/dist-packages/mmdnn/conversion/mxnet/mxnet_parser.py", line 262, in gen_IR
func(current_node)
File "/usr/local/lib/python3.5/dist-packages/mmdnn/conversion/mxnet/mxnet_parser.py", line 515, in rename_BatchNorm
IR_node.attr["axis"].i = MXNetParser._convert_axis(IR_node, int(source_node.get_attr("axis", "1")))
File "/usr/local/lib/python3.5/dist-packages/mmdnn/conversion/mxnet/mxnet_parser.py", line 79, in _convert_axis
ndim = len(IR_node.attr['_output_shapes'].list.shape[0].dim)
IndexError: list index out of range

If anyone has any solution for this problem, I would be delighted. Thank you

Hi @CarlosE97, could you please upload your model?

Hello @rainLiuplus, thank you for your reply.
Here is the JSON file containing the model's architecture.
resnet50_v1-symbol.zip
Thanks.

Hello @rainLiuplus, after updating MMdnn, this error was fixed and I got another error which was similar to issue #208. Thank you


Accessing to the public score of python scripts in Kaggle

Hey folks,
Thanks for providing this nice API.
Using it, I have written the following code to download python kernels from various Kaggle competitions. The API also lets us download some metadata related to the kernel.
However, I don't see the public score as a part of metadata. My requirement is that to see what was a python script score in the leaderboard also, and by score, I mean the evaluation metric that has been used in the competition such as accuracy, etc.
It doesn't matter what are the competitions or how old they are. I just need data science python scripts with an evaluations metric associated to it (accuracy/precision/F-1, …).
`import time
import kaggle
max_number_of_competitions = 100
default_page_size = 20
for category in kaggle.api.valid_competition_categories:
if category == 'all':
    continue

print('category: ' + str(category))

max_number_of_pages = int(max_number_of_competitions / default_page_size)
for page_index in range(max_number_of_pages):
    competitions = kaggle.api.competitions_list(category=category, page=page_index)
    time.sleep(1)

    print(str(len(competitions)) + ' competitions found in page ' + str(page_index))

    for competition in competitions:

        kernels = kaggle.api.kernels_list(competition=competition.ref,
                                          language='python',
                                          kernel_type='script',
                                          sort_by='scoreDescending',
                                          page_size=100)
        time.sleep(1)

        print(str(len(kernels)) + ' python script kernels found in ' + competition.ref)

        for kernel in kernels:
            kaggle.api.kernels_pull(kernel.ref, './' + category + "/" + competition.ref + '/' + kernel.ref + '/',
                                    metadata=True)
            time.sleep(1)`

Unfortunately that Kaggle folks wish to keep this as proprietary data I guess.   I'm not sure why they don't explain why that is, it's not like we'd behoove them some attempt to profit off our labor.   They do so much already.


Get user info inside LuisDialog

I want to get user info (for instance his/her Skype username), but i'm doing that inside a LuisDialog.
How can I access the parent activity inside an intent handler function?. I saw this answer https://github.com/Microsoft/BotBuilder/issues/1084#issuecomment-242543615 that @willportnoy did but passing IAwaitable<IMessageActivity> activity as a parameter to the intent handler throws me an exception where it message its just the name of the intent (it doesnt matter which text you pass to it, it just throws the exception always).

I ve thought of maybe using any of the IBotDataBag inside the context (PrivateConversationData,UserData,ConversationData), but I don't know which could be the key name of the property, so I am just guessing.
I also thought of passing the original Activity to the Dialog but I cant store it as a property because its non-serializable and that would crash my bot, and storing just the username isn't exactly what I need, I want to be able to use all user data not just one property.
Any workaround for this?
Thanks!

You can use the context.Activity property to access the original incoming activity.

That property doesn't appear on my context, on which version was that introduced?

Just updated BotBuilder (Microsoft.Bot.Builder) to 3.4.0 and the property appeared, it seemed that I was using an old version of the framework.
Thanks

How to update microsoft.bot.builder ???


Java 2.0

Has anyone got this cookbook to work with the latest Java community cookbook? I'm running into a few issues. NoMethodError        -------------        undefined method []' for nil:NilClass
   Cookbook Trace:
   ---------------
     /tmp/kitchen/cache/cookbooks/java/recipes/openjdk.rb:61:in `block in from_file'
     /tmp/kitchen/cache/cookbooks/java/recipes/openjdk.rb:57:in `from_file'
     /tmp/kitchen/cache/cookbooks/java/recipes/default.rb:22:in `from_file'
     /tmp/kitchen/cache/cookbooks/artifactory/recipes/default.rb:12:in `from_file'
     /tmp/kitchen/cache/cookbooks/tw-artifactory/recipes/default.rb:66:in `from_file'

   Relevant File Content:
   ----------------------
   /tmp/kitchen/cache/cookbooks/java/recipes/openjdk.rb:

    54:    notifies :write, 'log[jdk-version-changed]', :immediately
    55:  end
    56:
    57:  java_alternatives 'set-java-alternatives' do
    58:    java_location jdk.java_home
    59:    default node['java']['set_default']
    60:    priority jdk.alternatives_priority
    61>>   bin_cmds node['java']['jdk'][node['java']['jdk_version']]['bin_cmds']
    62:    action :set
    63:    only_if { platform_family?('debian', 'rhel', 'fedora', 'amazon') }
    64:  end
    65:
    66:  if node['java']['set_default'] && platform_family?('debian')
    67:    include_recipe 'java::default_java_symlink'
    68:  end
    69:
    70:  # We must include this recipe AFTER updating the alternatives or else JAVA_HOME

`

Hey Oscar! Stephen wants an artifactory server so I'm studying this.

try changing default['artifactory']['java']['jdk_version'] = 8 to
default['artifactory']['java']['jdk_version'] = '8'
java cookbook expecting it as string.

@oscar123mendoza @srjayep
I've updated the cookbook to the latest version in chef supermarket. The latest version now is 0.4.0.
This also removes the issue with the string / integer.
I will close this issue.


The date for older posts shows up as required now

Requirement according to @andywaldrop
10 minutes ago
3 hours ago
yesterday
3 days ago
6 days ago
.................
9/10/15
8/25/15
Screenshot

@aleffert pl review

@aleffert pl review...
The tests all pass locally... Enforcing the time zone, but still this issue persists...

:+1:


Build bar chart panel (default to Ordered List options)

Name is still TBD, i.e. whether we call this panel kind: 'BarChart' or something more opinionated (DataDog has a Top List widget for example)--interested in opinions on this

v1 of this new panel type was implemented in #1307 but we need to keep iterating based on feedback, support more panel options, write Grafana migration CUE mapping, and adjust the default ECharts properties in our charts theme

A dedicated issue for the grafana migration part was created here https://github.com/perses/perses/issues/1525.

I guess we can close this one. Migration script and BarChart panel has been implemented. Improvement or bugfix should go to a new issue


Improve type declarations for Pick and Omit

Demo: Omit, Pick

Published<EMAIL_ADDRESS>Thanks! 🚀


Refactored edge loader

/hold until the CI of alibaba/GraphScope#855 turned green.

Could you please brief the major changes in this pull request in the description?

Sorry I put only the description in the GraphScope side. I will copy it to here.


Could not resolve host

I am testing drone on kubernetes, drone can't clone my repository, this error occurs: fatal: unable to access 'http://gogs:3000/test/test.git/': Could not resolve host: gogs. I am using the flannel network, what should I do to solve this problem, I hope to get the answer from you, thank you!

:wave: hey there, per our issue template, for end-user support please use one of our support channels, such as our chatroom or our mailing list.


GTK 3.20 build failure

Host Environment

OS: Windows 10 Home
Compiler: VS 2019 Community

To Reproduce
Steps to reproduce the behavior:
./vcpkg install gtk:x64-windows
Failure logs
The following packages will be built and installed:
gtk[core]:x64-windows
Starting package 1/1: gtk:x64-windows
Building package gtk[core]:x64-windows...
-- Downloading https://ftp.gnome.org/pub/gnome/sources/gtk+/3.22/gtk+-3.22.19.tar.xz...
-- Downloading https://ftp.gnome.org/pub/gnome/sources/gtk+/3.22/gtk+-3.22.19.tar.xz... Failed. Status: 56;"Failure when receiving data from the peer"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:175 (message):
  Failed to download file.
  If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
  variables to "https://user:password@your-proxy-ip-address:port/".
  Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
ports/gtk/portfile.cmake:5 (vcpkg_download_distfile)
scripts/ports.cmake:94 (include)
Error: Building package gtk:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: gtk:x64-windows
Vcpkg version: 2020.01.17-nohash
Additionally, attach any relevant sections from the log files above.
Additional context
Tried "./vcpkg update" & "git pull"

It looks like your network can't connect to the source, please download the source manually and prevent it in VCPKG_PATH/downloads, then rebuild gtk.
Thanks.

If this issue is still not resolved, please reopen it.


[MEDIUM] Axios Cross-Site Request Forgery Vulnerability

Github reported a new dependabot security alert at: https://github.com/cdk8s-team/cdk8s-cli/security/dependabot/14

Stale

@iliapolo
It appears that the [MEDIUM] CVE-2023-45857 vulnerability is still detected even with the latest version (2.198.175).

@daisuke-yoshimoto Is this still relevant? I don't see any CVEs in this repo


Child elements not added to

Elm fails to construct HTML <template> elements, probably because you're supposed to call appendChild and such on the element's content property instead of the element itself.
Example:
import Html exposing (..)

view =
	node "template" [] [
		p [] [ text "Test" ]
	]

This will result in a <template> element with an empty #document-fragment in Firefox (109.0.1). In Chrome (110.0.5481.100) the p gets added outside of the #document-fragment, which isn't right either.

Might be more accurate to open this issue in elm/virtual-dom or elm/html

Ah, I forgot that elm/virtual-dom also contained kernel code. I will reopen the issue there. Thanks :)


about Indicator of Completion

I want to ask if the process of importing the epub is really as quick as the Indicator shows?
Or is there any other background processing being done even after the Indicator of completion of importing the epub?

I want to know this so that I can delete the epub file as soon as it has been imported.

Maybe something will be done in the background, but it has nothing to do with the original .epub file. The data has been transferred to a temporary folder, which will be automatically deleted when the task ends.

Does this also means that I can import individual epubs back to back without having to worry that the processing of one temporary epub will not interfere with the processing of the other temporary epub?

What are your needs? Do you need to import many epub files in batches?The, you can use the Import all epub from libraries to your vault command.

I have 130 epubs in my Library folder. Last time I tried importing all of them at once, only 5 got imported. So I'm importing all of them one by one.

I have 130 epubs in my Library folder. Last time I tried importing all of them at once, only 5 got imported. So I'm importing all of them one by one.

It should be a bug, I will fix it.

Last time I tried importing all of them at once, only 5 got imported. So I'm importing all of them one by one.

Did you encounter any bugs when you imported these epubs one by one? I'm wondering if it's because of a bug in the import of a certain epub that it never caused the entire command to terminate.

As of now I haven't as I haven't completed importing all of them. If I
do, I'll be sure to report it to you. It's possible that there's an
issue with the structure of one of the epubs. Kindle also rejects
certain epubs if they are not properly encoded before they are sent to
the cloud storage. Therefore, sometimes I use this -
https://kindle-epub-fix.netlify.app/.
On 12/22/23, aoout @.***> wrote:

Last time I tried importing all of them at once, only 5 got imported. So
I'm importing all of them one by one.

Did you encounter any bugs when you imported these epubs one by one? I'm
wondering if it's because of a bug in the import of a certain epub that it
never caused the entire command to terminate.
--
Reply to this email directly or view it on GitHub:
https://github.com/aoout/obsidian-epub-importer/issues/16#issuecomment-1867345004
You are receiving this because you authored the thread.
Message ID: @.***>

I just released the new version, now the log and notice will be more clear, maybe you can open the console and run the command I just released the new version, now the log and notice will be more clear, maybe you can open the console, see what the output will be and show me the screenshot so I can get more useful information.

I just released the new version, now the log and notice will be more clear, maybe you can open the console and run the command Import all epub from libraries to your vault, see what the output will be and show me the screenshot so I can get more useful information.


Crash after purchase succeed

Hi,
I'm testing a game using the iap extension, it works well on Android but on iOS it's crashing after the user makes a purchase.
Here are the details of the exception:
-[NSBundle appStoreReceiptURL]: unrecognized selector sent to instance 0x1e5b73e0
Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[NSBundle appStoreReceiptURL]: unrecognized selector sent to instance 0x1e5b73e0'
Also, before the exception occurs, I'm seeing this on the console:
Successful Purchase
which makes me think that the issue is here:
https://github.com/openfl/extension-iap/blob/master/project/iphone/InAppPurchase.mm

(void)finishTransaction:(SKPaymentTransaction*)transaction wasSuccessful:(BOOL)wasSuccessful
{
if (!manualTransactionMode) [[SKPaymentQueue defaultQueue] finishTransaction:transaction];
if(wasSuccessful)
{
NSLog(@"Successful Purchase");
NSString* receiptString = [[NSString alloc] initWithString:transaction.payment.productIdentifier];
  NSURL *receiptURL = [[NSBundle mainBundle] appStoreReceiptURL];
  NSData *receipt = [NSData dataWithContentsOfURL:receiptURL];
  NSString *jsonObjectString = [receipt base64EncodedStringWithOptions:0];

  sendPurchaseFinishEvent("success", [transaction.payment.productIdentifier UTF8String], [transaction.transactionIdentifier UTF8String], ([transaction.transactionDate timeIntervalSince1970] * 1000), [jsonObjectString UTF8String]);

}
else
{
NSLog(@"Failed Purchase");
if (transaction.error.code != SKErrorPaymentCancelled)
{
NSLog(@"Transaction error: %@", transaction.error.localizedDescription);
}
sendPurchaseEvent("failed", [transaction.payment.productIdentifier UTF8String]);
}
}

I'm testing on an iPod touch with iOS 6.1.6
I read that it happened on the share extension as well and was already fixed but I prefer to ask you directly before trying to fix it myself.
In advance thank you for your help.

Also the same code was tested on iOS 9.1 and the same crash happened

How often does this happen?

Take a look at the documentation for NSBundle appStoreReceiptURL. This method is only available in iOS 7.0 and later. The IAP extension has a minimum requirement of iOS 7 (not sure if this is documented or not) ... and there isn't much use in supporting anything below iOS 7 in general.
That doesn't explain why you would see the crash on iOS 9.1. Is it the exact same "unrecognized selector" error with the same method?

Thank you for you reply, sorry for the delay.
You are right about the version supporting, I didn't know that, I actually
do not have a lot of experience with iOS native code but I found something
that probably could help improve the current code:
float version = [[[UIDevice currentDevice] systemVersion] floatValue];
NSData* data = nil;
if (version < 7) {
    data = transaction.transactionReceipt;
} else {
    NSURL* receiptUrl = [[NSBundle mainBundle] appStoreReceiptURL];
    if ([[NSFileManager defaultManager]

fileExistsAtPath:[receiptUrl path]]) {
data = [NSData dataWithContentsOfURL:receiptUrl];
}
}
I also understand this is not really necessary considering that a lot of
people are using > 7 versions of iOS but I guess it will prevent having
this issue again.
I could add it to the code if you want.
About the 9.1 issue, I'm not completely sure it was caused by this, I will
check and let you know.
Thank you for the support.
2015-12-08 3:58 GMT+09:00 Bryan Chacosky<EMAIL_ADDRESS>
Take a look at the documentation for NSBundle appStoreReceiptURL
https://developer.apple.com/library/ios/documentation/Cocoa/Reference/Foundation/Classes/NSBundle_Class/index.html#//apple_ref/occ/instp/NSBundle/appStoreReceiptURL.
This method is only available in iOS 7.0 and later. The IAP extension has a
minimum requirement of iOS 7 (not sure if this is documented or not) ...
and there isn't much use in supporting anything below iOS 7 in general.
That doesn't explain why you would see the crash on iOS 9.1. Is it the
exact same "unrecognized selector" error with the same method?
—
Reply to this email directly or view it on GitHub
https://github.com/openfl/extension-iap/issues/36#issuecomment-162624180
.

--
Henry Fernández
Fiery Squirrel
http://www.fierysquirrel.com/

I'm not sure if that change is enough to resolve the issue. When iOS 7 was released, Apple began migrating their receipt system to use a global receipt to hold the other receipts (which is what appStoreReceiptURL returns) and deprecated the "receipt per purchase" style. More so to the point, there are less than 5% of users on iOS 6.0 and under - which  would yield very little return on a change to add support.
If you'd like to make a pull request, I'll leave it to other contributors to determine if the request should be pulled in or not. But my stance is to not bother with it.

It's ok, I don't really think it's an important issue, this is the device I
have at the moment to test but I guess I'll get a newer one.
Thank you very much for your help.
2015-12-10 14:02 GMT+09:00 Bryan Chacosky<EMAIL_ADDRESS>
I'm not sure if that change is enough to resolve the issue. When iOS 7 was
released, Apple began migrating their receipt system to use a global
receipt to hold the other receipts (which is what appStoreReceiptURL
returns) and deprecated the "receipt per purchase" style. More so to the
point, there are less than 5% of users on iOS 6.0 and under
https://david-smith.org/iosversionstats/ - which would yield very
little return on a change to add support.
If you'd like to make a pull request, I'll leave it to other contributors
to determine if the request should be pulled in or not. But my stance is to
not bother with it.
—
Reply to this email directly or view it on GitHub
https://github.com/openfl/extension-iap/issues/36#issuecomment-163488106
.

--
Henry Fernández
Fiery Squirrel
http://www.fierysquirrel.com/

@fierysquirrel If you truly don't think it's important then please close unless you're planning to push the PR for it. This repo doesn't get as much love as I wished it did.


Placeholder for Kinesis apps: remove unnecessary parts of AWS Java SDK from build

For instance, Kinesis-S3 has lots of unnecessary EMR classes in.

done in r92, closing


SEND_EVENT not working

Hello,
I am trying to trigger built-in welcome event when a user clicks chat button. I am using Dialogflow for the NLU platform. Latest webchat is not sending event request as shown in the sample. I followed 15.d.backchannel-send-welcome-event sample. Dialogflow has WELCOME event so I modifed the sample code as.
if (action.type === 'DIRECT_LINE/CONNECT_FULFILLED') {
    dispatch({
        type: 'WEB_CHAT/SEND_EVENT',
        payload: {
            name: 'WELCOME', //also tried webchat/welcome
            value: {
                language: window.navigator.language
            }
        }
    });
}

Also, i found directLine postActivity example which didn't work either (though i see trigger sent in console). Here is the postActivity code i am using,
directLine.postActivity({
        from: user,
        name: 'Welcome',
        type: 'event',
        value: ''
    })
    .subscribe(function(id) {
        console.log('"trigger requestWelcomeDialog" sent');
    });

Any suggestions?
Note: WEB_CHAT/SEND_MESSAGE is working fine.

Your code looks good to me. And I also tested sample 15.d thru the demo link and looks good.
Few things to investigate:

Are you using /master/webchat.js build? The SEND_EVENT is not in /latest/webchat.js (production 4.2.0) yet. But only on /master/, a.k.a. dev build.
Open development tools, go to the network tab. Could you see a POST call to /activities/ for your event activity?

@compulim Thank you for the quick response. My responses below,

Yes, I tried both master and latest.
Yes, I see the POST call to /activities/. Here is the screenshot,

Here is the network screenshot of the POST call,

Everything looks good to me too.
I assume one of the POST /activities/ is your event, right?
Could you do 2 more investigations?

Find the POST call that send the event. Could you screenshot/dump us the request payload?
On your bot side, can you dump all activities in console.log to make sure it is not received?

Right.

Here is the screenshot with request and response details for the POST sending event,

Since I am using Dialogflow for the NLU platform, I can't access bot side code and can't see backchannel activities (as far as i know).

Sorry got caught from something this afternoon.
It seems this screenshot does not have the POST body. For example, this is one I captured from Chrome on my side.

I am new to Dialogflow for the NLU platform. Do you have any pointer I can take a quick look at it and understand the issue?

@compulim No worries :)
Sorry, Firefox shows it differently. Here is the screenshot with params captured on Firefox,

I understand. Here is the documentation on Dialogflow's events Dialogflow Events. Basically, I am trying to trigger Dialogflow's built-in WELCOME event. I was able to get the welcome event response via POST without using webchat but I don't know how to dump that into webchat (or if it is even possible). Here is the link to Dialogflow's API I used to send the event request outside webchat Dialogflow API
I am not sure if this will help you but I really like webchat and would like to see it working completely on more platforms other than Microsoft's native platform. Please let me know if you need further information. You're doing a great job. Keep it up :)

Hi @compulim, were you able to find anything?

😵 Love to make it work. Challenge accepted. 💪
So, you want to send a custom POST request, via Web Chat, to Dialogflow API. And the POST request looks like,
{
    "queryInput": {
        "event": {
            "name": "WELCOME",
            "languageCode": "en-US"
        }
    }
}

Is that correct?
I am a bit disconnected here. You want to use Web Chat to connect to Dialogflow platform, that talk a totally different protocol, right? And if they are different protocol, why "SEND_MESSAGE" currently work in your case?
If you want to use Web Chat to connect to third party services, the easiest way is to implement your own DLJS, and you need to implement some API listed in this interface, minimally:

connectionStatus$ is an ES Observable (prefer core-js over rxjs)

0 = Not connected
1 = Connecting
2 = Connected

activity$ is an ES Observable

When subscribed, start your connection
Emit activities when they arrive

end is optional
referenceGrammarId, is for speech, not used in your case
postActivity() will return an ES Observable on send activity success

{ id: "id-of-the-activity-sent" }
Web Chat using different logic to measure if an outgoing activity successfully send or not

This observable must return positive result
The activity sent over the wire, will need to be echo-ed back from the remote service, and checked against the content of channelData

getSessionId(), is for OAuth, not used in your case

@compulim Just to note - this feature works perfectly for me in the dev branch, thanks!  Looking forward to its release.

@compulim I want the SEND_EVENT to work with Dialogflow, not send the POST request (POST was just an example to give you an idea on how i got the WELCOME event response). Everything on webchat seems to be working fine with Dialogflow except SEND_EVENT and i was wondering if you could find anything related to that.
So there is not possible? and no simple way to achieve this?

@dividor What are you using for NLU platform? Is it Dialogflow?

Sorry I am completed booked until late April.
If anyone able to pick this one up, please investigate how to use Dialogflow with Bot Framework and Web Chat.

@compulim and @xenflux :- Exactly same am too expecting. (Dialogflow NLP and UI to be Microsoft Bot framework) But from my side I have been developed like this. Refer the link for your reference.
https://typescript-v97enh.stackblitz.io/
But actual I need bot has to trigger first in the microsoft webchat where NLP is from Dialogflow.
If any one fixed please share me the information thanks.

per @Kaiqb, I am closing this issue since we do not have a plan to support DialogFlow in the near future. Thank you!


Error importing svelte-markdown: "'Lexer' is not exported by node_modules/marked/lib/marked.umd.js, ..."

[!] Error: 'Lexer' is not exported by node_modules/marked/lib/marked.umd.js, imported by node_modules/svelte-markdown/src/markdown-parser.js
https://rollupjs.org/guide/en/#error-name-is-not-exported-by-module
node_modules/svelte-markdown/src/markdown-parser.js (1:9)
1: export { Lexer, Slugger } from 'marked'
            ^
2:
3: import {
Error: 'Lexer' is not exported by node_modules/marked/lib/marked.umd.js, imported by node_modules/svelte-markdown/src/markdown-parser.js
...

What is your build set up? It seems your setup is trying to import marked from the UMD build rather than the ESM build.

Svelte + Rollup

Do you have a reproduction for this? This is not happening to me with 0.2.2 and Svelte's default template.

@pablo-abc, did you solve this issue?

@LilianaEverett I never managed to reproduce this on my projects. Is this happening to you as well? Do you have a repo or something where I can see this happening?

I know this is an old issue but I'm having the same problem using sveltekit and it only occurs if it tries to render it on the server for static generation or ssr.


本地部署报错

运行npm i提示
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-terser
npm WARN deprecated<EMAIL_ADDRESS>Please use @jridgewell/sourcemap-codec instead
npm WARN deprecated<EMAIL_ADDRESS>This module has moved and is now available at @rollup/plugin-replace. Please update your dependencies. This version is no longer maintained.
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-node-resolve.

added 160 packages, and audited 161 packages in 33s

20 packages are looking for funding
  run `npm fund` for details

5 vulnerabilities (4 high, 1 critical)

To address issues that do not require attention, run:
  npm audit fix

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.
npm notice
npm notice New minor version of npm available! 9.5.1 -> 9.8.1
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.8.1
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice

再运行npm audit fix
up to date, audited 161 packages in 5s

20 packages are looking for funding
  run `npm fund` for details

# npm audit report

lodash  <=4.17.20
Severity: critical
Regular Expression Denial of Service (ReDoS) in lodash - https://github.com/advisories/GHSA-x5rq-j2xg-h7qm
Prototype Pollution in lodash - https://github.com/advisories/GHSA-4xc9-xhrj-v574
Prototype Pollution in lodash - https://github.com/advisories/GHSA-fvqr-27wr-82fm
Prototype Pollution in lodash - https://github.com/advisories/GHSA-p6mc-m468-83gw
Command Injection in lodash - https://github.com/advisories/GHSA-35jh-r3h4-6jhm
Regular Expression Denial of Service (ReDoS) in lodash - https://github.com/advisories/GHSA-29mw-wpgm-hmr9
Prototype Pollution in lodash - https://github.com/advisories/GHSA-jf85-cpcp-j695
No fix available
node_modules/lodash
  scp2  >=0.1.0
  Depends on vulnerable versions of lodash
  Depends on vulnerable versions of ssh2
  node_modules/scp2

marked  <=4.0.9
Severity: high
Inefficient Regular Expression Complexity in marked - https://github.com/advisories/GHSA-5v2h-r2cx-5xgj
Inefficient Regular Expression Complexity in marked - https://github.com/advisories/GHSA-rrrm-qjm4-v8hf
Regular Expression Denial of Service (REDoS) in Marked - https://github.com/advisories/GHSA-4r62-v4vq-hr96
No fix available
node_modules/marked
  rollup-plugin-md  *
  Depends on vulnerable versions of marked
  node_modules/rollup-plugin-md

ssh2  <1.4.0
Severity: high
OS Command Injection in ssh2 - https://github.com/advisories/GHSA-652h-xwhf-q4h6
fix available via `npm audit fix`
node_modules/scp2/node_modules/ssh2

5 vulnerabilities (4 high, 1 critical)

To address issues that do not require attention, run:
  npm audit fix

Some issues need review, and may require choosing
a different dependency.

运行npm run build提示错误
src/components/PlayListCover.svelte
38: </script>
39:
40:

    ^
41:

42:     <Lazy height={220}>
[!] Error: Unexpected token (Note that you need @rollup/plugin-json to import JSON files)

请大佬指点

参考 #9 试试。

试了一下，npm run dev本地部署成功了，但是npm run build会报错。html和build文件夹下的html也没办法直接打开，本来还想在Vercel部署

给个思路吧，保留src，把 Rollup 换成 Vite。
Vite 包含了全套工具，应该就没问题了。搞好也是自己学习的过程。

感谢大佬指点，我在试试吧，刚开始玩这种，不太了解

_ html和build文件夹下的html也没办法直接打开_
build 出来的文件是不能直接打开的，你得起一个 web 服务啊，不管是本地还是服务器。


[FEATURE]: [BitPay] Currency Unit Conversion

:memo: Feature Description

Each currency can be described in terms of base or minor units.
For instance, when dealing with USD currency, the Cent is considered as the minor unit, while the Dollar is considered the base unit.
In Hyperswitch, the amount value is expected to be always provided in minor units.
For example: In USD, If the amount is 1 in base unit (Dollar), then it will be equivalent to 100 in minor units (Cent)
Some of the connectors integrated require the amount to be converted to a desirable unit before being passed to them.
We have functions to_currency_base_unit and to_currency_lower_unit in place to convert the minor unit amount to its decimal equivalent and vice versa. These conversions are handled explicitly based on the connector.

:hammer: Possible Implementation

ConnectorCommon trait have been implemented for the connector.
This trait contains get_currency_unit method. This method needs to be implemented.
It will define what type of conversion needs to be done during connector_router_data creation. Concurrently handle the creation of connector_router_data to be passed for the connector request body creation.
You can check this PR for further reference https://github.com/juspay/hyperswitch/pull/2196

:bookmark: Note: All the changes needed should be contained within hyperswitch/crates/router/src/connector/
:package: Have you spent some time checking if this feature request has been raised before?

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

:package: Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

:sparkles: Are you willing to submit a PR?

Could you please assign the issue to me ?

Hey @mohit07dec , sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with

Hey @mohit07dec  ,
Let us know if you face any issues. Happy to help! : )

Hey @mohit07dec  ,
Let us know if you face any issues. Happy to help! : )

I had a chat with Sanchith Hegde about a recurring issue. The hyperswitch-server-1 container keeps getting killed due to memory constraints. Sanchith mentioned our large codebase and the limitations of a 4 GB Windows machine. I'll tackle the problem once I get a new laptop.

Sure @mohit07dec
If any assistance is needed from our end, please let us know.
Thanks.  : )

Hey @mohit07dec ,
Are you still working on this issue?
Please let us know if you need any help:)

Hey @mohit07dec ,
Are you still working on this issue?
Please let us know if you need any help:)

Thank you for your concern, @Sakilmostak
I have an old laptop that isn't capable of running Hyperswitch locally. If anyone is willing to address this issue, please assign it to them. I plan to work on other available issues once I purchase my new laptop.

Hey @swangi-kumari , could you please assign me this issue if anyone is not working on it?

Hey @adarsh-jha-dev , sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with

@swangi-kumari could you please assign this issue to me?

Hey @TejasMate , I have been assigned this issue already and am working on it. Kindly look for other issues. Thank you.

Hey @TejasMate , I have been assigned this issue already and am working on it. Kindly look for other issues. Thank you.

No worries 😀


Your first program: Android Basics with Compose

im new to this
but the function println( )
doesnt seem to follow the camel case typing you discussed earlier
shoulndnt it be
printLn (  )
im a beginner and this threw me off for a second
Additional information
Include screenshots if they would be useful in clarifying the problem.

@bhminton
Thank you for reaching out to us.
The function println() is part of Kotlin standard library not Android.
Please take a look at the official documentation: https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.io/println.html
There is another similar function called readln() and here is the official documentation: https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.io/println.html
We don't have control over the library functions, we can only name the functions in the source code you own and code.

Closing this issue, since there is no response back.
Please create a new one, if you need help.


Greek Letters - Wrong Braille dots

It seems Braille dot definitions for both normal and capital Greek letters in “unicode.yaml” file are wrong:

"ς": [t: "GL⠎"]              # 0x3c2
"σ": [t: "GL⠞"]              # 0x3c3
"τ": [t: "GL⠥"]              # 0x3c4
"υ": [t: "GL⠋"]              # 0x3c5
"φ": [t: "GL⠯"]              # 0x3c6
"χ": [t: "GL⠽"]              # 0x3c7
"ψ": [t: "GL⠺"]              # 0x3c8
This might be the two sigma letters so gets the tool auto-calculated wrong
Same issue for Greek letters from line 471.

I couldn't find this issue in the Nemeth and UEB folders, however the issue appears to still exist in the Vietnam folder.
Should I work on checking the braille codes in the unicode and unicode-full files there?

I recently fixed this problem which I came across when checking against the Accessible Equation Editor's list of chars. I forgot that I should make the fix in the vi directory also.
Note: I think
"ς": [t: "GL⠎"] # 0x3c2

is correct. The others were wrong.


Edit hello world example

incorporated the feedback we were aligned on into #872, so closing this PR


core.shm additions

Expose struct counter in core/counter.h in order to be able to use it in link.h (in my SnabbTop porting branch I let core.link increment the shared counters directly (rxbytes, ...)
Allow passing of readonly parameter from counter.open to shm.map
Make value parameter of counter.add defaut to 1

Maybe that suggestion doesn't make sense with respect to your link.h changes.
How about including these changes in a SnabbTop PR to next so that we can look at it in context?


tekton: triggers policy: OpenID Connect authentication and authorization

See:

https://github.com/tektoncd/triggers/issues/610

stale, didn't get any time/traction.


Debug a flutter app on an iOS simulator

Developer should be able to easily debug (using service protocol) the Dart code in their app running in the simulator.

cc @chinmaygarde

I was able to pause isolates and do other debugging things from observatory after https://github.com/flutter/engine/pull/2167


[FEATURE] Rule Editor Backend Layer

Is your feature request related to a problem?
The security-analytics plugin will provide a rule editor which will allow users to edit/create custom rules in Sigma format & then use these rules in detectors to generate findings.
This issue is created to track the progress of exposing backend apis of Sigma Rule Engine for frontend development. The following scenarios are tracked as part of this issue:

List pre packaged rules for a particular rule category.

List custom rules for a particular rule category.

Create a new custom rule

Edit an existing custom/pre-packaged rule (direct edits with versioning will not be supported, when user edits an existing rule, a new rule will be created by copying the existing rule)

Delete an existing custom rule

Link new created/edited custom rules with a detector

What solution would you like?
A high-level block diagram of the sub-system is shown below.

The api definitions for the rule editor backend will be defined soon.
Create a new custom Rule
POST /_plugins/_security_analytics/rules?category=windows

Header:
Content-Type: application/json

Body:
title: Remote Encrypting File System Abuse
id: 5f92fff9-82e2-48eb-8fc1-8b133556a551
description: Detects remote RPC calls to possibly abuse remote encryption service via MS-EFSR
references:
    - https://attack.mitre.org/tactics/TA0008/
    - https://msrc.microsoft.com/update-guide/vulnerability/CVE-2021-36942
    - https://github.com/jsecurity101/MSRPC-to-ATTACK/blob/main/documents/MS-EFSR.md
    - https://github.com/zeronetworks/rpcfirewall
    - https://zeronetworks.com/blog/stopping_lateral_movement_via_the_rpc_firewall/
tags:
    - attack.lateral_movement
status: experimental
author: Sagie Dulce, Dekel Paz
date: 2022/01/01
modified: 2022/01/01
logsource:
    product: rpc_firewall
    category: application
    definition: 'Requirements: install and apply the RPC Firewall to all processes with "audit:true action:block uuid:df1941c5-fe89-4e79-bf10-463657acf44d or c681d488-d850-11d0-8c52-00c04fd90f7e'
detection:
    selection:
        EventLog: RPCFW
        EventID: 3
        InterfaceUuid:
          - df1941c5-fe89-4e79-bf10-463657acf44d
          - c681d488-d850-11d0-8c52-00c04fd90f7e
    condition: selection
falsepositives:
    - Legitimate usage of remote file encryption
level: high

Response:
{
  "_id": "jdrLhYMBsRfxq4n2FcK0",
  "_version": 1,
  "rule": {
    "category": "windows",
    "title": "Remote Encrypting File System Abuse",
    "log_source": "application",
    "description": "Detects remote RPC calls to possibly abuse remote encryption service via MS-EFSR",
    "tags": [
      {
        "value": "attack.lateral_movement"
      }
    ],
    "references": [
      {
        "value": "https://attack.mitre.org/tactics/TA0008/"
      },
      {
        "value": "https://msrc.microsoft.com/update-guide/vulnerability/CVE-2021-36942"
      },
      {
        "value": "https://github.com/jsecurity101/MSRPC-to-ATTACK/blob/main/documents/MS-EFSR.md"
      },
      {
        "value": "https://github.com/zeronetworks/rpcfirewall"
      },
      {
        "value": "https://zeronetworks.com/blog/stopping_lateral_movement_via_the_rpc_firewall/"
      }
    ],
    "level": "high",
    "false_positives": [
      {
        "value": "Legitimate usage of remote file encryption"
      }
    ],
    "author": "Sagie Dulce, Dekel Paz",
    "status": "experimental",
    "last_update_time": "2022-01-01T00:00:00.000Z",
    "rule": "title: Remote Encrypting File System Abuse\nid: 5f92fff9-82e2-48eb-8fc1-8b133556a551\ndescription: Detects remote RPC calls to possibly abuse remote encryption service via MS-EFSR\nreferences:\n    - https://attack.mitre.org/tactics/TA0008/\n    - https://msrc.microsoft.com/update-guide/vulnerability/CVE-2021-36942\n    - https://github.com/jsecurity101/MSRPC-to-ATTACK/blob/main/documents/MS-EFSR.md\n    - https://github.com/zeronetworks/rpcfirewall\n    - https://zeronetworks.com/blog/stopping_lateral_movement_via_the_rpc_firewall/\ntags:\n    - attack.lateral_movement\nstatus: experimental\nauthor: Sagie Dulce, Dekel Paz\ndate: 2022/01/01\nmodified: 2022/01/01\nlogsource:\n    product: rpc_firewall\n    category: application\n    definition: 'Requirements: install and apply the RPC Firewall to all processes with \"audit:true action:block uuid:df1941c5-fe89-4e79-bf10-463657acf44d or c681d488-d850-11d0-8c52-00c04fd90f7e'\ndetection:\n    selection:\n        EventLog: RPCFW\n        EventID: 3\n        InterfaceUuid:\n          - df1941c5-fe89-4e79-bf10-463657acf44d\n          - c681d488-d850-11d0-8c52-00c04fd90f7e\n    condition: selection\nfalsepositives:\n    - Legitimate usage of remote file encryption\nlevel: high"
  }
}

or,

{
  "error": {
    "root_cause": [
      {
        "type": "security_analytics_exception",
        "reason": "{\"error\":\"Sigma rule must have a log source\",\"error\":\"Sigma rule must have a detection definitions\"}"
      }
    ],
    "type": "security_analytics_exception",
    "reason": "{\"error\":\"Sigma rule must have a log source\",\"error\":\"Sigma rule must have a detection definitions\"}",
    "caused_by": {
      "type": "exception",
      "reason": "java.util.Arrays$ArrayList: {\"error\":\"Sigma rule must have a log source\",\"error\":\"Sigma rule must have a detection definitions\"}"
    }
  },
  "status": 400
}

Edit an existing custom/pre-packaged rule (direct edits with versioning will not be supported, when user edits an existing rule, a new rule will be created by copying the existing rule)
If I edit existing rule A and it is saved as rule B, will rule A be deleted?


Name 'index' must not begin with '', which is reserved by GraphQL introspection

Seeing this error here:
https://fivethirtyeight.datasettes.com/graphql/fivethirtyeight?query={
  generic_ballot_polls(filter%3A{pollster_rating_name%3A{eq%3A"YouGov"}}) {
    totalCount
    nodes {
      pollster
      sponsor_ids
      sponsors
      partisan
      display_name
      pollster_rating_name
      fte_grade
      population
      population_full
      methodology
      office_type
      start_date
      end_date
      tracking
      created_at
      notes
      url
      stage
      dem
      rep
      ind
      question_id
      poll_id
      cycle
      state
      pollster_id
      pollster_rating_id
      sample_size
      seat_number
      sponsor_candidate
      nationwide_batch
      rowid
    }
  }
}

Found the source of the error here: https://github.com/graphql-python/graphql-core/blob/619a77a7d79415aca63da5d7070249ed0a7ac3b5/src/graphql/type/validate.py#L192
            if name.startswith("__"):
                self.report_error(
                    f"Name {name!r} must not begin with '__',"
                    " which is reserved by GraphQL introspection.",
                    ast_node,
                )

One of the causes of this is https://fivethirtyeight.datasettes.com/fivethirtyeight/pollster-ratings~2Fpollster-ratings which has a column called:
  "# of Polls for Bias Analysis" INTEGER,

I'm going to try turning a prefix __ into _0_.


[DO NOT MERGE] Introduce ETW events for Operation start/stop

DO NOT MERGE! For review only at this point. We still need to verify that this is sufficient for Application Insights Profiler.
The goal of this PR is to generate new, paired ETW start/stop events for operations (requests or dependencies) so that Application Insights Profiler can extract them from profile data. This will let Application Insights Profiler work in applications that are not hosted in ASP.NET (e.g. WebJobs).

Hi @pharring, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Paul Harrington). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@SergeyKanzhelev All my testing is complete. This is ready for review/approval now.

@SergeyKanzhelev I just had a through that these new "Operation" events should actually be fired from within OperationTelmetryExtensions.Start() and Stop(). It would be more low-level and would catch 'naked' uses of "Start"/"Stop". What do you think?

@SergeyKanzhelev @dnduffy  This is ready for merging and I'd like it to go into 2.3. Approve?

Please update CHANGELOG.md

And update it for your previous change of StartOperaiton API modification as well

Should it be under "Version 2.3.0-beta3" ?

Confirmed. It's not a concern for Service Profiler.