I met some Errors on Example Project

Hi.
I have some errors on example project using react-native-openalpr.
Error is following:
"error: bundling failed: Error: Unable to resolve module react-native-openalpr from App.js: react-native-openalpr could not be found within the project.
If you are sure the module exists, try these steps:

Clear watchman watches: watchman watch-del-all
Delete node_modules: rm -rf node_modules and run yarn install
Reset Metro's cache: yarn start --reset-cache
Remove the cache: rm -rf /tmp/metro-*
at ModuleResolver.resolveDependency (E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\node-haste\DependencyGraph\ModuleResolution.js:186:15)
at ResolutionRequest.resolveDependency (E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\node-haste\DependencyGraph\ResolutionRequest.js:52:18)
at DependencyGraph.resolveDependency (E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\node-haste\DependencyGraph.js:282:16)
at Object.resolve (E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\lib\transformHelpers.js:267:42)
at E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:426:31
at Array.map ()
at resolveDependencies (E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:423:18)
at E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:275:33
at Generator.next ()
at asyncGeneratorStep (E:\workspace\ReactNative\react-native-openalpr\example\node_modules\metro\src\DeltaBundler\traverseDependencies.js:87:24)
BUNDLE  [android, dev] ./index.js ▓▓░░░░░░░░░░░░░░ 12.8% (122/341), failed.

::ffff:<IP_ADDRESS> - - [27/Oct/2021:09:56:42 +0000] "GET /index.bundle?platform=android&dev=true&minify=false HTTP/1.1" 500 - "-" "okhttp/3.12.1"
"
please help me..
I have double checked steps of that..

Closed for inactivity.


[BUG] lua/neotest-dotnet/strategies/netcoredbg.lua:62: attempt to index field 'dap' (a nil value)

NeoVim Version
NVIM v0.10.0-dev-2507+g3df1211ebc
Build type: RelWithDebInfo
LuaJIT 2.1.1702233742
Run "nvim -V1 -v" for more info
Describe the bug
Run tests by issuing the following command
require("neotest").run.run({ strategy = "dap" })

Sorry for not providing any additional information but the bug is too obvious :)
For some reason this line
https://github.com/Issafalcon/neotest-dotnet/blob/c19df2a139d88c5b4130b830d2cbe63a2c6c6c0c/lua/neotest-dotnet/init.lua#L150C27-L150C35
was not changed in the commit where the line
https://github.com/Issafalcon/neotest-dotnet/blob/c19df2a139d88c5b4130b830d2cbe63a2c6c6c0c/lua/neotest-dotnet/init.lua#L7
was changed.
So it causes an error on this line
https://github.com/Issafalcon/neotest-dotnet/blob/c19df2a139d88c5b4130b830d2cbe63a2c6c6c0c/lua/neotest-dotnet/strategies/netcoredbg.lua#L62

Did you mean to open this issue in https://github.com/Issafalcon/neotest-dotnet?

Huge facepalm. Sorry, my bad. Closing the issue.


Added Bootstrap class

Note: Please use present tense for commit titles, instead of past tense. E.g "Add", "Modify", "Remove" etc. Thank you! :)

We will also add that into the contributors file.


[Table]: Filter toggle icon should switch to blue if filter toolbar is currently active

To more clearly indicate which toolbar is toggled on, these buttons should change colors when they're toggled on.

:tada: This issue has been resolved in version 1.19.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.19.1 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


Unclear where to report issues related to the product

It is unclear who/where feedback for the package generation functionality in the new project system goes. Who owns this feature such that I can report issues/feedback? The feedback I have right now related to not only "missing" properties to control generation but also how the package gets generated itself. It is unclear whether this is a NuGet, MSBuild or Project System area.
The feedback link is for the docs but it doesn't clarify where feature feedback should go. This would be nice to include in the docs. Right now there are too many GitHub repos to figure it out.

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

ID: cb4428fc-215c-78e1-5ff0-fe73f7a08260
Version Independent ID: 6741b1de-38a4-fbb1-d79d-acc47d549a2d
Content: Additions to the csproj format for .NET Core
Content Source: docs/core/tools/csproj.md
Service: unspecified
Product: dotnet-core
GitHub Login: @blackdwarf
Microsoft Alias: mairaw

If you see in the bottom of each topic, there are two buttons: one for content, one for product.

Because of the reasons you mentioned (too many repos, unclear where to file the bug), we settled on https://developercommunity.visualstudio.com/spaces/61/index.html for product feedback. The product team will triage those and move them to the appropriate owners.

Also, if you have a guess of what repo should be, you can try to filing on that repo and the team will move if it's the wrong one.


"Format SQL" incorrectly separates PostgreSQL "@>"  array operator

Hi,
I noticed a minor issue with the "Format SQL" right click command of the SQL Editor window: it incorrectly separates PostgreSQL "@>"  array operator by inserting a space, resulting in subsequent syntax error when you try to execute the query.
E.g.
ARRAY[1,4,3] @> ARRAY[3,1]
becomes:
ARRAY[1,4,3] @ > ARRAY[3,1]
after choosing Format SQL
https://www.postgresql.org/docs/9.6/static/functions-array.html

Thanks for report


Some pubsub messages have been lost with log message "non-retriable error returned from app while processing pub/sub event "

In what area(s)?

/area runtime

What version of Dapr?
ver=1.7.2
Hi guys,
We have started to use Dapr in our new project and it seems like everything is going well.
But after deployment in AWS EKS we have faced a strange issue.
We have /api/PubSub/workflow  endpoint (.net core 6)
and it is look like this:
[HttpPost("workflow")]
public async Task WorkflowEventSubscribe([FromBody] JsonElement input)
{
_logger.LogInformation("Pubsub callback request");
try
{
// some code
return Ok(new { status = "SUCCESS" });
}
catch (Exception ex)
{
_logger.LogError($"Error in pubsub receiver - {ex.Message}");
return Ok(new { status = "RETRY" });
}
}
Sometimes Dapr executes /api/PubSub/workflow well, but sometimes
it returns non-retriable error returned from app while processing pub/sub event .
it is hard to imagine a case when WorkflowEventSubscribe returns not 204.
It can happen just in case of a bad body of request and not ability to bind action arguments, but I don't see such a case in logs.
This log shows that 2 of 3 requests were failed on the Dapr side.
.Net hasn’t evidences that (2) requests have been arrived to PubSub  controller.
time="2022-05-24T12:22:44.769563172Z" level=debug msg="1 message(s) received" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.contrib type=log ver=1.7.2
time="2022-05-24T12:22:44.769614453Z" level=debug msg="Processing SNS message id: %!s(*string=0xc000a64ec8) of topic: imr-workflows" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.contrib type=log ver=1.7.2
time="2022-05-24T12:22:44.771684998Z" level=error msg="non-retriable error returned from app while processing pub/sub event : . status code returned: 404" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.runtime type=log ver=1.7.2
time="2022-05-24T12:22:44.984083092Z" level=debug msg="1 message(s) received" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.contrib type=log ver=1.7.2
time="2022-05-24T12:22:44.98413998Z" level=debug msg="Processing SNS message id: %!s(*string=0xc000e9bb88) of topic: imr-workflows" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.contrib type=log ver=1.7.2
time="2022-05-24T12:22:44.984856096Z" level=error msg="non-retriable error returned from app while processing pub/sub event : . status code returned: 404" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.runtime type=log ver=1.7.2
time="2022-05-24T12:22:45.009936636Z" level=debug msg="1 message(s) received" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.contrib type=log ver=1.7.2
time="2022-05-24T12:22:45.010012401Z" level=debug msg="Processing SNS message id: %!s(*string=0xc000b7c208) of topic: imr-workflows" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.contrib type=log ver=1.7.2
time="2022-05-24T12:22:46.998608068Z" level=debug msg="No messages received, continuing" app_id=imr-workflow instance=imr-workflow-75cd9b47d5-2bxrs scope=dapr.contrib type=log ver=1.7.2
Finally, we have that subscription works, but some of messages were lost (msg="non-retriable error returned from app while processing pub/sub event )
Expected Behavior
All event should be processed with /api/PubSub/workflow endpoint
Actual Behavior
Some of events were not processed with /api/PubSub/workflow endpoint. It happens randomly. Dapr returns msg="non-retriable error returned from app while processing pub/sub event  for some of events randomly.
Steps to Reproduce the Problem

Deploy Dapr as side car for application container
Configure Dapr

apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
name: {{ include "template.name" . }}-subscription
spec:
topic: {{ .Values.dapr.pubsub.topic }}
route:  /api/PubSub/workflow
pubsubname: {{ .Values.dapr.pubsub.name }}

Publish massage to SNS from AWS step functions

##Important note:

Everything woks fine on local development machine. (docker, tye)

Thank you guys in advance for your response.

The message non-retriable error returned from app while processing pub/sub event occurs only when Dapr is trying to send a message to your app's configured endpoint that should receive the event and getting back a 404 NOT FOUND error.
This might occur because your app container is restarting or otherwise unavailable at the time of the request. Do you see container restarts in your Pod during the time period of these Dapr log events? Do you have a way to check if your webserver is restarting at times or why it might be giving a 404?

Hi Yaron, thank your for your response.
app container is working without restarts.
I see origin of 404.
I have snssqs-pubsub dapr  component
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
name: snssqs-pubsub
namespace: default
spec:
type: pubsub.snssqs
version: v1
metadata:
- name: accessKey
value: ""
- name: secretKey
value: ""
- name: region
value: "us-east-1"
and this component is shared between two apps for subscription :

apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
name: imr-workflows-subscription
spec:
topic: imr-workflows
route: /api/pubsub/workflow
pubsubname: snssqs-pubsub

apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
name: imr-textbook-subscription
spec:
topic: imr-textbook
route: /imr-textbook
pubsubname: snssqs-pubsub
And see strange thing....  darp side car (imr-workflows-subscription) is trying to run subscription from imr-textbook-subscription.
it is log from imr-workflows app:
2022-05-25T12:33:16+03:00       Request starting HTTP/1.1 POST http://<IP_ADDRESS>:80/imr-textbooks application/cloudevents+json 944
2022-05-25T12:33:16+03:00 dbug: Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware[1]
2022-05-25T12:33:16+03:00       POST requests are not supported
2022-05-25T12:33:16+03:00 dbug: Microsoft.AspNetCore.Routing.Matching.DfaMatcher[1000]
2022-05-25T12:33:16+03:00       No candidates found for the request path '/imr-textbooks'
2022-05-25T12:33:16+03:00 dbug: Microsoft.AspNetCore.Routing.EndpointRoutingMiddleware[2]
2022-05-25T12:33:16+03:00       Request did not match any endpoints
2022-05-25T12:33:16+03:00 dbug: Microsoft.AspNetCore.Server.Kestrel.Connections[9]
2022-05-25T12:33:16+03:00       Connection id "0HMHU3RS1HCUF" completed keep alive response.
2022-05-25T12:33:16+03:00 info: Microsoft.AspNetCore.Hosting.Diagnostics[2]
2022-05-25T12:33:16+03:00       Request finished HTTP/1.1 POST http://<IP_ADDRESS>:80/imr-textbooks application/cloudevents+json 944 - 404 0 - 0.1616ms
I think it has half of my puzzle :)
Have you any ideas, why it is happening and how to resolve it properly.
Thank you in advance for your response.

Additionally I am still searching lost messages in SNS/SQS
I am sending two approve events for each workflow, but in most cases i have got jut one event in the subscription.

Important to know that I am sending events to SNS/SQS from AWS StepFunctions.
Thank you again for your attention

Hi Everyone.
Dapr is great.
I have resolved my issue with adding Scopes in subsciptions
apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
name: imr-workflows-subscription
spec:
topic: imr-workflows
route: /api/pubsub/workflow
pubsubname: snssqs-pubsub
scopes:

imr-workflows     # !!!!!!

apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
name: imr-textbook-subscription
spec:
topic: imr-textbook
route: /imr-textbook
pubsubname: snssqs-pubsub
scopes:

imr-textbooks  # !!!!!!

Those changes push Dapr to make subscription request to proper application which has proper endpoint to handle subscription.
Thank you Yaron and everyone who keep Dapr documentation fresh.

The message non-retriable error returned from app while processing pub/sub event occurs only when Dapr is trying to send a message to your app's configured endpoint that should receive the event and getting back a 404 NOT FOUND error.
This might occur because your app container is restarting or otherwise unavailable at the time of the request. Do you see container restarts in your Pod during the time period of these Dapr log events? Do you have a way to check if your webserver is restarting at times or why it might be giving a 404?

Hi,
I may have this issue.
Will this happen also if the dapr container inside the pod gets restarted multiple times?
The retry works fine but sometimes it happens that for some processes it won't retry and unfortunately the logs are lost


Added a unit test for directives with arguments

Hey,
This seemed pretty cool so I started playing around with it.  I was having trouble getting directives which took arguments to work so tried adding a unit test for it.
Turns out it works fine, I just didn't realize you needed to declare which arguments your directive takes.
Anyway here is the unit test I wrote 😄

Tks @yaworsw so much!


About pitch extract on multi speaker or singer database

Hi,
I want to train a multi speaker VC or SVC model based on fish diffusion. However, I noticed that there is no speaker-related f0 mean or std stored in the preprocess pipeline.
But the f0 range of female and male singer are different, they share the same normalize (pitch_to_scale), is there any problem on multi singer training? Or How to resolve this problem on fish diffusion?
Thanks!

That was not a problem.The f0 analysis range is from 20 to 16,000Hz.It covers all the range which human can produce.

Oh,I have the mistake.Normally I use the configure that f0_min=20Hz,f0_max=1600Hz.
Under normal circumstances, f0 analyzers are sufficient for analyzing these frequency bands (most of the human vocal frequencies are in this range). Higher frequency analysis is meaningless, and neither the acoustic model nor the f0 analyzer can handle it.

OK, thanks for your info. And, if I want to use this f0 process on expressive speech(like the dialogue in a tvplay), not singing, can the f0 in fish diffusion handle this scene where there is roaring now and then?

OK, thanks for your info. And, if I want to use this f0 process on expressive speech(like the dialogue in a tvplay), not singing, can the f0 in fish diffusion handle this scene where there is roaring now and then?

There is multi f0 analyzers,the best is crepe but it is pretty slow.You can try these analyzers (pm,dio,harvest,crepe)and find the f0analyzers can handle it.But sometimes for some extreme audio, no f0 analyzer can accurately extract, so good luck.:)

OK, thanks for your info. And, if I want to use this f0 process on expressive speech(like the dialogue in a tvplay), not singing, can the f0 in fish diffusion handle this scene where there is roaring now and then?

There is multi f0 analyzers,the best is crepe but it is pretty slow.You can try these analyzers (pm,dio,harvest,crepe)and find the f0analyzers can handle it.But sometimes for some extreme audio, no f0 analyzer can accurately extract, so good luck.:)

OK , thanks!


Qtest Code Enhancement

Code Enhancement for MultiAmlConnection,AlmItemFields,DefectProfile

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7

See the complete overview on Codacy


No way to clear cookies

There's no way to clear the cookies, meaning the deep link that have been intended to be opened. This is an issue in situations like:

a link is clicked that opens a web view that sets a cookie to go to item A, but the app is not opened
a link is clicked and directly open the app to item B. This will link to item B, and won't clear the past intent to go to item A. So the link to item A will be trigger after which is confusing and hard to handle well.

Let me look into this.  Can you give me a few more details?
Are these the same kind of links?  Is the first a universal link and second an app scheme link (a link like myapp://open?link_click_id=...)?

Both are universal links. One is coming from what I think is a cookie set when going to the deepview, and the other is passed directly at app launch when opening the app from a link in iMessage. There's a little time in between. Like not a lot but it's definitively not happening back to back. My best guess is that the cookie is retrieved with some delay, and probably asynchronously (you probably know better) and that the direct opening doesn't clear the cookie.
I'd except the app to only have one pending deep link at a time (which is the case with cookie based ones since going to a deep view removed the previous intent if any) but here it's not the case.

Your expectations are correct -- Only the most recent deep link should be handled.
I'm not able to reproduce your problem immediately.  I'll try a few other things too.  Maybe something weird is happening in iMessage.

Hey @gsabran - I wanted to chime in on your use case here. It's not a problem with this library, so I'm going to close this issue out. It's more a function of how the Branch deferred deep linking system works. We don't refer to them as "cookies" for the click, we refer to them as "outstanding matches". If you think this is a common use case for you, we actually give you the ability to control the duration of how long a match will remain outstanding. Our default is two hours, but you could lower this to 5 minutes if you were super concerned about it. You can control it at a global level or on a per link basis. Please review highlighted section at the bottom of these docs for more info: https://dev.branch.io/getting-started/matching-accuracy/overview/

Ok thanks for the clarification. But it seems unexpected that handling one outstanding match won't remove other matches that might be less outstanding, which is what I think @E-B-Smith was mentioning.

I see your point and it's a valid one. To do this, we'd need to confident that the outstanding click (tied to a cookie) is associated with the native app session (IDFA) is the same person. We have the data to do this a certain percentage of the time. However, this whole double outstanding match should hopefully be a pretty rare scenario. Let me add this request to our infra backlog in the mean time. Thanks!


loader was not supporting options on multiselection

I still thing there was a reason why we didn't allow that, but that was more than year ago so I guess if it works why not to merge?


Getters/setters not supported correctly

https://jsmonk.github.io/hegel/try#MYGwhgzhAECC0G8BQ1XQOYFMAu0BGAFAJSIprkBOOArhQHbQBMA3GagL5vQQ74EBuYENUwAubtgoBLOuiLjqdACaYAZjMxLEnTkmAB7OhFxhoAXmh1MAdzjFWAegfRsATwAOmbgAt91EFp4XnTUALZBFPjUuFIwMqqYFFRakNDE5gB8lmEReobG0FoWYAB0eEA

getter property type is inferred as () => number instead of number
if I try to define both getter and setter - there is an error Duplicated property definition!

Ideally, it should check that input type for setter and return type of getter are the same.

Currently, getters/setters are not supported. But it will be. We will close the issue after adding the support. Thank you for your contribution ^_^.

Hey, before you get to it, I thought I'd include some ideas here:
Some people would love to be able to specify the types of getters differently from the setters.
For example, the following is something we can normally do in plain JavaScript:
class Vec2 {
  x: number = 0
  y: number = 0
  add(other: Vec2) {}
  sub(other: Vec2) {}
  // etc
}

class Thing {
  _pos = new Vec2

  get position(): Vec2 {
    return this._pos
  }
  set position(val: [number, number] | Vec2) {
    if (Array.isArray(val)) {
      this._pos.x = val[0]
      this._pos.y = val[1]
      return
    }
    this._pos = val
  }
}

const thing = new Thing

const pos = thing.position // pos is type Vec2

console.log(pos.x, pos.y) // 0 0

thing.position = [1, 2] // this is ok, no type error

console.log(pos.x, pos.y) // 1 2

thing.position = new Vec2 // We can also do this, no type error

Here's the (looong) TypeScript issue (with many upvotes): https://github.com/microsoft/TypeScript/issues/2521
In particular, this comment has a good summary of why it is needed. Basically, there are many existing APIs, especially in the DOM, that already use these patterns in practice, but TypeScript can not type them properly.


feat: add jsonschema2pojo

Motivation
Add jsonschema2pojo install helper, its special because its java application.
Changes

add jsonschema2pojo install helper

Might be superseded by #259


Update to windows-sys 0.52

Unfortunately, due to the C dependencies, I couldn't locally verify this builds.
I did use the GitHub CI to verify it builds and the tests passed, yet due to the size of the CI and the fact I on-purposely didn't add a new vet statement which made the CI self-cancel, I'm unsure this is completely valid at this time.
On the belief this is a valid patch, my question is how should vet be handled. Personally, I see no reason to trust the windows-sys 0.48 published by Microsoft yet not windows-sys 0.51 given its FFI bindings, not some original library. If an attestation the changes are as expected is needed, I'd ask someone who has experience doing said reviews and whose name actually means something to this project performs said review. If no such explicit review is needed, I'd be happy to update the existing vet attestation if informed how.

No worries about the audits, I've posted those to https://github.com/bytecodealliance/wasmtime/pull/7634 and you can rebase this PR on top of that when it merges.
Otherwise thanks for this!

Rebased :+1:

Mind undoing the changes to the CI configuration to get CI unblocked here?

Oh. So sorry. I did my CI changes on a branch, yet it appears a block from there slipped in here. Fixed.

I'm unfortunately not familiar enough with everything in play here to know how to fix the CI failure.

This says jobs were fixed, not that any failed :thinking:  The amount of jobs also had GitHub cancel it on my fork, presumably some DoS protection. I wonder if because I'm making the PR, it's using my personal line of trust instead of wasmtime's...
I believe rerunning 'failed' jobs will preserve the existing passes and may move a few more forward.

The short summary is just 7 successful, 12 skipped.

Ah you can chalk this up to GitHub UI. What you're looking at is this PR's CI which is separate from the merge queue's CI. The PR's CI does not contain the failure, the merge queue does. This is because the merge queue is running more tests than the PR does. More information about this can be found here

Yet no other error than a very generic 'linking failed'.

I'm as baffled as you! I don't know what's going on here. It may be MinGW-specific, not sure.

It's trying to link against a MSVC lib. Out of all the specified libs in the CI, I'm unclear which is the MSVC. I don't see any .lib extensions.
My immediate concern is the C:\\Users\\runneradmin\\.rustup\\toolchains\\1.74.0-x86_64-pc-windows-msvc\\lib\\rustlib\\x86_64-pc-windows-gnu\\lib\\rsbegin.o paths, yet those do still specify GNU and I don't have an immediate reference point. Just noting my in-progress debug commentary.

The .drectve warnings are not actually relevant (see https://github.com/rust-lang/rust/issues/112368) but may be masking an actual error. The warning spam is fixed in gnu binutils >=2.40.

https://github.com/microsoft/windows-rs/issues/2614#issuecomment-1684152597 suggests a newer binutils may resolve this. I'll look at what version the CI is using...
(thanks to @ChrisDenton for the context)

Thanks for the tip! Unfortunately though I don't actually know how to update gcc and/or binutils here. We use whatever's already on GitHub Actions builders, so we're not already downloading something with a version number to bump.

I wasn't asking for help. I was noting it had to be done. The above commit may achieve it. I'll fix any issues with it in about an hour, as I have to step out immediately.

I believe this one is finally correct, based on its validity on a branch running the presumed relevant subset. Apologies for any exposure to y'all for the variety of misc attempts I had to suffer through. While I could write a few paragraphs of oddities I encountered, the final solution appears quite compact and barely introduce further dependencies (due to not using any additional actions, using a package manager already present in the CI, yet yes, downloading the updated package via Msys's infrastructure).
Feel free to request I squash the commit history to tidy it up with a proper message. I'm unsure if the history or the cleanliness is preferred.

https://github.com/kayabaNerve/wasmtime/actions/runs/7229601178/job/19700662991 for the relevant CI run.

Happy to! Thanks for your patience.


Best way to provide mirror URLs?

Hello, love the project!
Came across this whilst trying to find a reliable source of UT files.
I have a crapton of space available on my Wasabi.com account, and would like to help mirror the entire content.
I'm curious if you have any tips on how I can bulk edit the yaml files to add my mirrored URLs in a pull request?
I don't mind doing manually, but might take some time!
Cheers,
Ben

Hey, thanks for the interest and offer :).
Unfortunately, there's no common or friendly way to set up an actual mirror and update all the data with that new mirror at the moment (and to keep it up to date).
The only option right now is for you to clone the data repo, load up the YAML data and download the files, upload them to your storage, and write a new entry into the download list of each (being careful to not disrupt the existing data :joy:). Then open a PR with all the changes.
Depends how adventurous you're feeling.
However, I see that Wasabi uses the AWS S3 API, and the S3 API is probably a storage backend which should be supported anyway, so I'm going to implement that into the unreal-archive binary so we can more easily sync files to mirrors on that API and manage updating all the YAML etc.

Tracking work on this in unreal-archive/unreal-archive#34 and unreal-archive/unreal-archive#35.

Ace! Thank you for working on this, look forward to testing it out !

@vohzd Hopefully a working solution is now in place. See Mirroring section of the README: https://github.com/unreal-archive/unreal-archive#mirroring
I hope it makes sense.
You can download the latest binary from here: https://code.shrimpworks.za.net/artefacts/unreal-archive/latest/ (the one without an extension can just be made executable and invoked directly). Requires Java 11.
For Wasabi, the following parameters should be used:
./unreal-archve mirror \
  --store=s3 \
  --s3-bucket=bucket-name \
  --s3-key=YOURKEY \
  --s3-secret=YOURSECRET \
  --s3-endpoint=https://s3.BUCKET-REGION.wasabisys.com/ \
  --s3-url=https://s3.BUCKET-REGION.wasabisys.com/__BUCKET__/__NAME__

It won't download to intermediate files on your machine, so you don't need to make sure you have > 200gb of local space.
When it's all mirrored, just create a PR from your fork of the data repo and everything should be cool.

Hello @shrimpza Thanks & fantastic work on this.
All installed fine, and the sync is chucking along without issues.
I expect it to take a while before I raise a PR, but will give you a shout when complete.
From spot checking a few and re-downloading them, they seem to be publicly accessible and uncorrupted.
Will give a more thorough test soon.
Cheers!

@shrimpza I've created a PR for this here:
https://github.com/unreal-archive/unreal-archive-data/pull/211

That's awesome! Thanks!
Before I merge it and it goes public, just wondering if you want me to add a hostname mapping to some sort of site name or something, or if it's cool as-is. For example, by default your links will appear as follows:

Can be "Vohzd's Mirror" or anything, or we can leave it.

Interesting question, I got into UT under the name 'Sabre', but prefer 'Vohzd' as that what I'll be putting new stuff out under... so to answer your question... put it under 'Sabre' as that's what everyone will remember me as!
You're doing a grand service dude, thank you very much for building this!

Done!
Mirror name remapped, and the mirror is now listed on the downloads.
Thank you for contributing to keeping all this old stuff available for a bit longer :).
I'm going to also add a --since=2020-04-01 type syntax to the mirror command, so if you or anyone else ever feels the urge to sync up new stuff, the option will be available.

New option was added per above (see readme), so mirroring new content should be easy too.
Closing this issue as hopefully mirroring is now resolved (aside from new storage implementations which might need to be added later).
:+1:


Player is always in walking speed

Mod: Sprint Sprint Sprint
Stardew Valley version 1.4 breaks the function of the mod. No logged errors and/or any logged info or trace messages on the SMAPI log. Player will always be in default walking speed.
See Also:

Issue posted on Nexus Mods In 1.4, does not work

Mod folder now renamed to Sprint Sprint Sprint and unique ID is now Jessebot.SprintSprintSprint2 so it can check if the old version exists. You must delete the old 1.0.4 or unreleased 1.0.5 version

2.0 Rewrite coming later, version 1.0.5 fixes this bug

Sprint Sprint is the rewrite


Implement table.insert and table.remove

Codecov Report

Merging #180 into lablua will increase coverage by 0.22%.
The diff coverage is 91.33%.

Impacted Files
Coverage Δ

titan-compiler/ast.lua
95.38% <100%> (+0.07%)
:arrow_up:

titan-compiler/builtins.lua
100% <100%> (ø)

titan-compiler/scope_analysis.lua
89.65% <100%> (-0.12%)
:arrow_down:

titan-compiler/symtab.lua
100% <100%> (ø)
:arrow_up:

titan-compiler/types.lua
84.5% <33.33%> (-2.26%)
:arrow_down:

titan-compiler/checker.lua
91.08% <89.23%> (+0.49%)
:arrow_up:

titan-compiler/coder.lua
85.13% <93.65%> (+0.92%)
:arrow_up:

Continue to review full report at Codecov.

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


KAFKA-15363: Broker log directory failure changes

Extend the ReplicaAlterLogDirsThread to send an AssignReplicasToDirsRequestData RPC before promoting the future replica in JBOD mode with KRAFT.
Extend AssignmentsManager and DirectoryEventHandler to attach a callback to the events which will run when the RPC is completed.
The state of RPCs is kept in memory in ReplicaAlterLogDirsThread, and it gets updated using the callback that is sent to AssignmentsManager. Once we promote the future replica we clear the state of the RPCs related to this replica in question

Update:

During cancellation of the assignment if the ReplicaAlterLogDirsThread had already sent RPC for queuing then it will fire the revert assignment first before it removes the fetch state of this partition.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Seeing this locally:
org.mockito.exceptions.misusing.InvalidUseOfMatchersException:
Invalid use of argument matchers!
3 matchers expected, 2 recorded:
-> at kafka.server.ReplicaAlterLogDirsThreadTest.shouldRevertAnyScheduledAssignmentRequestIfAssignmentIsCancelled(ReplicaAlterLogDirsThreadTest.scala:461)
-> at kafka.server.ReplicaAlterLogDirsThreadTest.shouldRevertAnyScheduledAssignmentRequestIfAssignmentIsCancelled(ReplicaAlterLogDirsThreadTest.scala:461)

This exception may occur if matchers are combined with raw values:
    //incorrect:
    someMethod(any(), "raw String");
When using matchers, all arguments have to be provided by matchers.
For example:
    //correct:
    someMethod(any(), eq("String by matcher"));

For more info see javadoc for Matchers class.

	at kafka.server.ReplicaAlterLogDirsThreadTest.shouldRevertAnyScheduledAssignmentRequestIfAssignmentIsCancelled(ReplicaAlterLogDirsThreadTest.scala:461)
...

Seeing this locally:
org.mockito.exceptions.misusing.InvalidUseOfMatchersException:
Invalid use of argument matchers!
3 matchers expected, 2 recorded:
-> at kafka.server.ReplicaAlterLogDirsThreadTest.shouldRevertAnyScheduledAssignmentRequestIfAssignmentIsCancelled(ReplicaAlterLogDirsThreadTest.scala:461)
-> at kafka.server.ReplicaAlterLogDirsThreadTest.shouldRevertAnyScheduledAssignmentRequestIfAssignmentIsCancelled(ReplicaAlterLogDirsThreadTest.scala:461)

This exception may occur if matchers are combined with raw values:
    //incorrect:
    someMethod(any(), "raw String");
When using matchers, all arguments have to be provided by matchers.
For example:
    //correct:
    someMethod(any(), eq("String by matcher"));

For more info see javadoc for Matchers class.

	at kafka.server.ReplicaAlterLogDirsThreadTest.shouldRevertAnyScheduledAssignmentRequestIfAssignmentIsCancelled(ReplicaAlterLogDirsThreadTest.scala:461)
...

fixed now

@OmniaGM The commit is worded as KAFKA-15363: Broker log directory failure changes.  Is this correct?  The branch is KAFKA-15357.  So I'm a bit confused as to what is correct.  The branch doesn't matter, but the commit should be reworded before this gets merged if it is currently not correct.

The branch name is wrong. But the commit message matches the Jira ticket. Do we want to name the Jira ticket?

Ah, ok, we can keep it as it is, then.  WiIl wait for build to finish and them merge assuming the test failures can be discounted.

66 tests have failed.  There are 33 new tests failing, 33 existing failing.
Many of them are common issues in recent build, and I ran several other test classes locally -- they all passed.  Merging.


What is sites_file used for?

You have a requirement of a sites_file, in the QUALITY_CONTROL_VCF workflow, however it appears unused. Could you please clarify what the purpose of this input is? Thanks!
Also, do you have a 1000G Phase3 v5 link for cloudgene install ? Thanks!

nvm


fix(metrics): not report status code 200 when http client returns error

What type of PR is this?
fix(metrics): do not report status code 200 when http client returns error

What this PR does / why we need it (English/Chinese):
en: Currently, http client report metrics status code as 200 when client returns an error.
zh: 当前在http客户端返回错误时，metrics上报的status code是200。

Which issue(s) this PR fixes:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jianzhang.kk seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
Jianzhang.kk seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@2011aad  CLA 需要处理下哈


Emoji-button - No on-click reaction (Widget not showing)

Hi,
Our reaction button got broken after the latest update. By clicking it, nothing happens. No JS-errors is thrown in the console, by clicking it. I can't really tell what's going on.
We're on the latest build, with all plugins updated, too.

Having the same issue. Seeing nothing have changed in this repository since a while back, I would guess something changed in Discourse.
If I find any way around it I'll post here :)

Yep, I think the latest update may have broken it.

Yep, I think the latest update may have broken it.

There is a patch which allows it to show, but there are quite a few changes to the internal emoji selector code, so it's not working as good as before. A lot better than nothing at all, but could use some well deserved CSS magic from someone who have a good eye to that stuff! :)

This is now fixed: https://github.com/gdpelican/retort/commit/272fe4c643864a9a419248abc2fd2e81258d3478


Cubestrip coordinates slightly inaccurate

Tested the EAC Format for cubemaps but the video has gaps in the join of the cube. Maybe it needs altering slightly.

Hello, @webciter!

The above screenshot is captured in our demo, is that the same issue?
I'll take a look at that.

Okay, I'll check this issue.
I'll leave a comment here after it's fixed :)

@webciter
We've just released v3.5.0 that fixes this issue.
You can try setting cubemapConfig.trim option to remove those graphical glitches between texture tiles.
I've also removed the above glitch in our demo :)

thanks works ok now.


Improve constraints of conditional types applied to constrained type variables

With this PR we compute more accurate constraints for distributive conditional types applied to constrained type variables. For example:
type IsArray<T> = T extends any[] ? true : false;

function f1<U extends object>(x: IsArray<U>) {
    let t: true = x;   // Error
    let f: false = x;  // Error, but previously wasn't
}

Previously, we'd obtain the constraint of IsArray<U> by applying the distributive conditional type to the constraint of U, which resolves to false (because object doesn't extend any[]). But really, it ought to be boolean (i.e. true | false) since clearly something constrained to object both could or could not be an array.
A type variable represents any possible type within its constraint. So, given a type parameter T with the constraint C, the constraint of T extends X ? A : B is

A when C is known to always extend X,
B when C is known to never extend X, or
A | B when C possibly extends X.

We previously didn't consider the third possibility. With this PR, we choose the third outcome when C is not assignable to X, but X (or, if X is a union type, some constituent of X) is assignable to C.
Fixes #30152.

@typescript-bot test top100
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this faster

@typescript-bot pack this

@typescript-bot test top100
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this faster

Latest commit changes checking for the "possibly extends" case to use a reverse assignability check. This check is not quite as permissible as a comparability check, meaning we'll less often hit the "possibly extends" case where the constraint is a union. Still, since constraint types are typically close-to-top types (like object or {}), it isn't likely to matter. And, certainly, performance with the reverse assignability test is much more acceptable.

@typescript-bot test top100
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this faster

I don't know if this is intentional or a regression but the following code used to typecheck, but no longer does with this PR:
export type DeepReadonly<T> = T extends boolean | string | number | null | undefined
    ? T
    : T extends ReadonlyArray<infer Item>
        ? ReadonlyArray<DeepReadonly<Item>>
        : {readonly [K in keyof T]: DeepReadonly<T[K]>}

type Shape = {x: number};

export function getKey<T extends Partial<Shape>, K extends keyof Shape>(shape: DeepReadonly<T>, key: K) {
    return shape[key];
}

Playground with TS 5.2
Playground with this PR

@typescript-bot pack this

@MichaelMitchell-at That was an earlier copy of the PR. It works with a current build.

I tried the current build in our code base and ran into a couple of errors that I'm having trouble creating a minimal reproduction that I can share. In general they seem to arise when generics are involved together with complex conditional types. I wonder if I'm having trouble producing a minimal repro because in the actual code we're hitting one of the depth limits that exists in the type checker.
Unrelated, but just mentioning while I have your attention with the hope y'all will keep in mind, is on nightly I'm encountering  a new The inferred type of this node exceeds the maximum length the compiler will serialize error.  One of the reasons to use the schema library is to avoid needing to write out the type by hand. It would probably only help us temporarily if the max length were increased; A more long-term approach might be some ability to give users more control over when type aliases are used/expanded in types when serialized/displayed.

As an FYI in case anyone follows this issue, it turned out that this caused a regression in type-fest and I'm still debugging why and/or how to fix, but want to mention here in the meanwhile, might file an issue later: https://github.com/sindresorhus/type-fest/issues/784


For statblocks with minor problems, handle exceptions where possible and do partial imports in the case that exceptions cannot be handled.

Summary:
Exceptions in imperfect statblocks should be handled wherever possible, because due to the frequency this is a problem, people may otherwise give up on using this module.  Worst case should be to simply not import the part of the statblock that cannot be parsed -- but still import the NPC or PC with warning messages.
Details:
Keep in mind that most of the audience for this tool (module) does not have a programming background -- and (separate issue) does not know what a "standard" statblock should look like.
So, for example, in #285 (Can't import Bestiary - Rakshasa ...) the comment was the following for closing this issue:
The HeroLab one works, the others are missing the Perform Type, e.g. "Perform +16" needs to be "Perform (Sing) +16".
There's some problems with this:

The Bestiary was used to help test this module.  The reality is that GMs still need to import NPCs for the Adventure Path they are using in their campaign -- even though the Compendium has almost everything from all the Bestiaries.  It's also not desirable to try to use the Compendium while running a game because it's just too slow.

That statblock was chosen as a regression test -- most of the Bestiary was not retested.  This means many NPCs might not now import because they wrote the Bestiary this way.

Most GMs probably won't care if "Perform" imports at all -- but would very much like to have the NPC import because they need it for their game.  The alternative of manually inputting the information by manually creating the NPC is very not desirable.

One choice for this case would be to do some exception handling and import this as "Perform (Unknown)" or just don't import it but warn the GM.

If this module breaks on imperfect statblocks often and doesn't create an NPC at all -- eventually people will give up on it.

So, for the example #285 given:
After closing that one, i created #290, as that is the underlying reason for #285 to fail. When #290 is implemented, the issues you noted should resolve itself, as all statblocks with undefined subskills for craft/perform/etc. will parse.
For the overlaying issue of usable feedback before starting a conversion thats doomed to fail, see issue #4 .
Work on that has started, see an early iteration here: https://imgur.com/VaHefhd

Ok.  I see.  Sounds good.  It goes without saying that no one will ever fix these statblocks.  Oops!  I said it.  :)

v2.0.0 is up and includes the preview. Next are more meaningful error messages.


Can I enable 'showLineNumbers' as default like 'copy' button?

As the title says, is there a way I can enable showLineNumbers as default.
In case this functionality does not exist, can you give me some guidance on how to implement this. I can work on a PR. There is already support for copy=false, so I think adding this functionality should be pretty easy.

Please request this feature in https://github.com/atomiks/rehype-pretty-code

I think this feature already exists on rehype-pretty-code, take a look at the "Line numbers" section on their site


Hardware Details: Implement TreesCard

Use image bellow as a reference:

Card name: Trees
Show each tree in the card and count for success/failed/inconclusive

It will not be implemented as the status count will be displayed in the header


Split according to transaction date (and transaction type)

I have a large file, and would like to split out activities that contain a transaction with a transaction-date/@iso-date on or after a specified date
Added bonus, if I can also specify the transaction-type in the query, it'd be very helpful
For example:

Split out all activities that have an Outgoing Commitment (code=2) transaction-type transaction and a transaction-date/@iso-date on or after 20180930

Thanks

Implemented basic transaction filtering. Note the following new command-line options:
--transaction-type=TYPE - include only activities with at least one transaction of the specified type.
--transaction-start-date=YYYY-MM-DD - include only activities with at least one transaction on or after the specified date (after filtering by --transaction-type, if specified).
--transaction-end-date=YYYY-MM-DD - include only activities with at least on transaction before or on the specified date (after filtering by --transaction-type, if specified).

@stevieflow - please test, and then close the issue if the new functionality is working for you.

Thanks @davidmegginson !
I'm not quite getting results - using the following:

iatisplit -n 1000 -transaction-type 2 -transaction-start-date 2018-09-30 inputfile.xml

Could you post an example , please?

The options you posted should split into groups of 1,000 activities (max), including both humanitarian and non-humanitarian activities, but only when they have at least one transaction of type "2" with a transaction date on or after 2018-09-30. What are you seeing?
Note that you need double hyphens at the beginning of the long options, so --transaction-type and --transaction-start-date

Additional question: currently, the script is filtering activities by transaction, but when a matching transaction is found, it includes the activity with all its transactions. Were you looking at filtering the transactions themselves, as well?

Thanks @davidmegginson
OK - using

iatisplit -n 1000 --transaction-type 2 --transaction-start-date 2018-08-30 --verbose iati-activities-Humanitarian.xml

I get

iatisplit: error: unrecognized arguments: --transaction-type --transaction-start-date 2018-08-30 iati-activities-Humanitarian.xml

And then for just

iatisplit -n 1000 --transaction-type 2 --verbose iati-activities-Humanitarian.xml

I get

iatisplit: error: unrecognized arguments: --transaction-type iati-activities-Humanitarian.xml

In terms of the other question:

Were you looking at filtering the transactions themselves, as well?

Nope.  Getting back the whole activity, which will contain qualifying transactions, but all others, would be fine

What OS are you using, @stevieflow ?
Can you try installing the latest version (0.4) and then run
iatisplit --version

to confirm?

Im on 0.4 now - which looks to have resolved it!
Thanks @davidmegginson - I've successfully filtered and split


Document Message::getPublishTime

The value returned by Message.getPublishTime is not clearly defined in the comments. I did not find the documentation about timestamps in Hazelcast. The Java-Doc gives the same information.
Proposal for modifications:
@return the time when the message is published in milliseconds since 1970.01.01.

I agree on the update. Submitted the PRs: https://github.com/hazelcast/hazelcast/pull/14583 and https://github.com/hazelcast/hazelcast-cpp-client/pull/535


What does yarp::dev::IVelocityControl::stop() actually do at the lowest level?

We all know that velocity commands are integrated by the FW to retrieve position references to follow by means of the PID.
Whenever a yarp::dev::IVelocityControl::stop() is issued, what actually happens at the lowest level? Two possibilities are given:

The FW simply stops integrating as it received a velocityMove(0.0).
The FW sets the internal position reference equal to the current encoder reading.

Is there any third one?
I'm a bit concerned about hypothesis 1: because of a poorly tuned PID, the joint will be still moving for a while after the stop command is received.
I hope the implementation resorts to 2, indeed.
Second question
Does the joints subset version of the stop command travel over the rpc channel or over the streaming channel? Can we be sure that the stop command is never lost?
/cc @randaz81 @barbalberto

Backstage
I'm asking this because while teleoperating the iCub, I've realized that the fingers - which are controlled in velocity and whose control is a bit degraded - do not stop exactly upon stop reception but, instead, proceed moving for a short while afterwards.

I can answer about the joints controlled by 2FOC boards (thus not regarding the fingers), where the stopping method is 2 (new reference = encoder reading).

@ale-git cool thanks :+1:
What about the MC4 (controlling the fingers) then?

All stop command are send using rpc.

Thanks @barbalberto :+1:
Thus, only one doubt (the main one) remaining.

The implementation 2 is already used on MC4 boards.
	if (_control_mode[axis] == MODE_IMPEDANCE_POS || \
		_control_mode[axis] == MODE_POSITION || \
		_control_mode[axis] == MODE_IMPEDANCE_VEL || \
		_control_mode[axis] == MODE_VELOCITY || \
		_control_mode[axis] == MODE_MIXED) \
		{ \
		    _set_vel[axis] = 0; \
		    _set_acc[axis] = 0; \
            _desired_vel[axis] = 0; \
			init_trajectory (axis, _position[axis], _position[axis], 1); \
		} \

Great, thanks @randaz81
Hence I'd need to better inquiry what I've been observing with the fingers!


Refresh Token

Hi,
I'm using Xamarin.Auth to authenticate users with OAuth2. But the component doesn't seem to accept refresh_token which is important to keep users logged in without asking them to re-log in.
Is this feature currently not supported?

See #79

Thanks @hvaughan3


Questions about error indicators

Hello, when I read the code, I found that there was a scale before data input, but there was no anti-standardization when output, which made mae and mse very small. I calculated mape again and reached about 10. mape is not mentioned in the original paper. Is it just the standardization of this operation that makes the non-percentage error like mae seem small? I look forward to your reply and answer

Thank you for raising these two questions.
1）Instance normalization does indeed include an anti-standardization step. This is handled in PatchMixer.py with the line x = self.revin_layer(x, 'denorm').

We primarily compared baselines using metrics such as MSE and MAE. For other objective functions like the MAPE you mentioned, the performance might not be ideal enough. This could require incorporating the loss function for optimization and training.

Yes, there is "norm" and" denorm" in your model PatchMixer, which is OK, but there is a scale operation in your data_loader before this, and then it is not transformed back, may I ask why？


Add option to output Gradescope questions that take up less space

When trying to fit quizzes onto a single page its useful to have the option to print questions to take up less space. However, its still better to leave the default the way it is, because for long-form exams its cleaner looking.
This pull request adds a command line flag that switches Gradescope questions to take up as little space as possiblle.

Looks good -- thanks!


LaTeXRenderer: fix escaping on % sign

approved

@jasondenton, good catch, thank you. It looks like the original author(s) simply overlooked escaping of this % character, didn't they? But what about _, shouldn't it be escaped as well? I base this on https://tex.stackexchange.com/questions/34580/escape-character-in-latex which is from 2011. I'm no expert on LaTeX though.
Next, I would like to ask you to:

Make the commit message more concise, like: fix(latex-renderer): add escaping of % sign. I'm not sure though whether this is possible to easily do within an existing GitHub PR. So just ignore this if it is not possible.
Add some unit test for your change. I think that just extending the existing test_raw_text() test method should be enough.

I would merge your PR afterwards. Thank you.

Ad escaping _: It looks like PR #93 (by @sebqq) suggests this, together with not escaping within inline code section... So maybe we should combine it everything together.

I think that sounds about right. _ needs to be escaped or the results
are likely to be weird at best and break latex compile at worst.
Inline code blocks are, almost by definition, something that isn't being
subject to the regular markdown rules. So they should definitely be
exempt from escaping. I'm not sure what that change looks like in the
code, so I will leave it to you.
Regards
Jason Denton
Petr Bodnar wrote on 9/16/21 2:24 PM:

Ad escaping |_|: It looks like PR #93
https://github.com/miyuchina/mistletoe/pull/93 (by @sebqq
https://github.com/sebqq) suggests this, together with not escaping
within /inline/ code section... So maybe we should combine it
everything together.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/miyuchina/mistletoe/pull/112#issuecomment-921258842,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZLFYBFTAMWN4WVWLHTB33UCJN7LANCNFSM5D653Y7A.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi Jason, thanks a lot for your feedback. I will therefore continue with the broader PR #93 and close this one.

Hi Jason, thanks a lot for your feedback. I will therefore continue with the broader PR #93 and close this one.


Define standard parameters through individual mixins

Alternative to https://github.com/zytedata/zyte-spider-templates/pull/34 using mixins.

Codecov Report

Merging #46 (dd2b025) into main (9ddb62a) will increase coverage by 0.02%.
The diff coverage is 100.00%.

:exclamation: Current head dd2b025 differs from pull request most recent head 1568cf6. Consider uploading reports for the commit 1568cf6 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   98.81%   98.83%   +0.02%
==========================================
  Files          12       13       +1
  Lines         506      516      +10
==========================================
+ Hits          500      510      +10
  Misses          6        6

Files
Coverage Δ

zyte_spider_templates/params.py
100.00% <100.00%> (ø)

zyte_spider_templates/spiders/base.py
100.00% <100.00%> (ø)

zyte_spider_templates/spiders/ecommerce.py
100.00% <100.00%> (ø)


bamfile without index

Hi I'm trying to run SVTyper on the vcf file I got after running Lumpyexpress. Here is the code that I am using:
svtyper
-i sv.vcf
-B sample.bam
-l sample.bam.json
> sv.gt.vcf
I keep getting this error code: ValueError: fetch called on bamfile without index.
Am I suppose to index the bam file that came out of BWA-MEM? I am using svtyper 0.6.0. and pysam 0.15.0.  I was also wondering if its required to run svtyper on the Lumpyexpress vcf file or if its an optional step.
Thank you for your help!

Yes, an index for your BAM file is required.
 Otherwise, we would recommend running SVTyper on the VCF from Lumpy.

Thank you so much for the response! So I ran svtyper using the indexed bam file and now I get a new error code:
Error: failed to build insert size histogram for paired-end reads.
Please ensure BAM file (sample.bam) has inward facing, paired-end reads
I used this code from Lumpy github to align my data:
Align the data
bwa mem -R "@RG\tID:id\tSM:sample\tLB:lib" human_g1k_v37.fasta sample.1.fq sample.2.fq
| samblaster --excludeDups --addMateTags --maxSplitCount 2 --minNonOverlap 20
| samtools view -S -b -
> sample.bam
Thank you!!!!

The error seems to indicate that svtyper is having trouble finding enough paired-end reads to build an insert size distribution. Since you're obviously aligning paired reads, I'd check that the order of the two files is the same, they contain the same number of reads and, in the event that you were creating these from an aligned BAM/CRAM, that they've been reverse complemented appropriately when converted back to unaligned data.


I want to add contributors.md file

I hope that makes sense
(SSOC)

There's no need for a list of contributors, I suppose. We can add a section on the website to indicate that it's open source and display the current contributors.

There's no need for a list of contributors, I suppose. We can add a section on the website to indicate that it's open source and display the current contributors.

That's necessary @kom-senapati


D4PG's vmin and vmax paramaters

Hi @fastturtle
Would it be possible to share more info about hyper-parameters D4PG? For example, D4PG paper doesn't include information about all of important hyperparameters in the paper such as vmin and vmax . Although there is a note says how to find them, it can still lead to very different results and it is kinda unclear.
If it is not possible to share them, can you clarify a bit the note about vmin and vmax?
"A good rule of thumb is to set vmax to the discounted sum of the maximum instantaneous rewards for the maximum episode length;" What policy should be used to gather this info and when? random policy? or we should train an agent for a while with another method and then use that policy it to get maximum reward? how this vmax/vmin related to the final an agent's returns and optimal policy? For example, final return of Acrobot(Swingup) is ~400 (Figure 2 in the paper), so should vmax be very close to 400?
Thanks.

Hi rasoolfa,
Thanks for your question! Indeed the problem of setting the vmin/vmax in D4PG (and C51) is not obvious especially when the environment's reward signal is completely unknown. Thankfully, in the dm_control suite the rewards are all standardized so let me use that as an example for clarification.
The instantaneous rewards (reward obtained at each timestep) in any dm_control task is bound in (-1,1). If the distributional Q function is meant to fit the distribution of discounted returns, ie the discounted sum of instantaneous rewards, the maximum possible value is a discounted sum of 1s. Therefore the maximum possible return is simply the value of the geometric series with ratio gamma = 0.99 as set in all our experiments, which leads to a vmax of 100. Historically, a value of 150 was used to give it a bit of slack but you could probably get away without it. (Let us know if you try it!)
As for your specific question about acrobot:swingup: first of all, you're missing the discounting in your calculation of vmax; secondly, just because we haven't found a policy that reaches higher returns doesn't mean that there doesn't exist one, therefore our Q function had better be able to evaluate such a policy. This is why we use the theoretical limits set by the dm_control designers and fix vmin|vmax for all tasks therein.
Hope that helped!
Bobak

Thanks for your detailed response, it is very helpful indeed.


[draft] CFG implementation

squashed version of closed PR #24

CFG implementation is moved to new feature branch, new PR #39


1.15 charts crush for Android 6 / 7

Commit https://github.com/ruuvi/com.ruuvi.station/commit/fbabf885db766a0f7153e40c917b4bc4ac97c8af

QA passed


how setting contract send params

Hi , there
I have a contract method
> DropFromSweepList(address[])

$txdata = json_decode('{"nonce":"0x125","gasLimit":"0x186a0","gasPrice":"0x77359400","to":"0x7bAe46657b90919B2C5F00aF7f783095F5e67a0E","value":"0x00","data":"0x902615f0000000000000000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000000000000000000000000030000000000000000000000008c7294523a49029248ef00c8ff167ca6fddd2a0a0000000000000000000000005aa4d978e10eacdfabb7cd0533a85c0d1b43159a0000000000000000000000007f534003e479b43c88be714c961734e63a204440"}', true);

I'm trying to use contract(abi)->at(address)->send("DropFromSweepList", $txdata, $callback) call it.
but it gives me an error.
Please make sure you have put all function params and callback.

If it's like the call() function, then you need to include all contract method parameters one by one after the contract method name.
e.g. my code for call() is something like this:
`
$contract_metadata = json_decode(file_get_contents($this->_file_name));
$this->contract = new Contract($this->_server, $contract_metadata->abi);
$this->contract->at($contract_metadata->networks->{$this->_network_id}->address);
$this->contract->call("getItem", $item_id, function ($error, $data) use (&$output) {
$output = $this->contractCallback($error, $data);
});
`
Just like my getItem() method requires a parameter for $item_id, I imagine your DropFromSweepList method needs a parameter for the address.


Universal ATM DUPE!

Hi, notabadminer. On this week i found that some players get duped diamond blocks on my server. Today my friend found this video on youtube:
http://www.youtube.com/watch?v=25r0FihyeBA
I've saw that Universal ATM will DUPE items if I place stackable items in top left five slots on my inventory and SHIFT+Left Click on first of it they will stack to 64. Its so TERRIBLE! Please FIX IT!

Screenshots:
BEFORE: http://pixs.ru/showimage/duppng_6466071_15939351.png
AFTER: http://pixs.ru/showimage/dupecoplit_6809094_15939357.png

Which version are you running? I recall this being an issue, but I thought it was fixed with an update.

Ok, I've updated mod to latest version and I've found that you fixed that bug. Really nice.
Here is updated version of languages and textures:
http://pastebin.com/jtRCHA68
http://pastebin.com/cgdSLyu2
Textures: https://www.dropbox.com/s/hn40ad031mfnsx5/textures.zip?dl=0
I've changed money types like this:
Coin, Coin Stack, Dollar, Dollar Stack, Money Bag

I just updated with a release that doesn't crash the server when crafting a Vending Frame. I'm still not sure about changing to the different money types. I am leaning towards keeping the generic coin types. I do like the textures though. They are cleaner than the stuff I threw together. At the least, I will update the russian lang and give you credit.

I think that new money types will be better, also for them exists textures.
But you can make banknote instead of dollar (Just rename it in Lang)

Its will be nice 😁


Enable did:tz and did:web for WASM

Depends on https://github.com/spruceid/ssi/pull/120
I'm not sure the tests I ran were actually using WASM, is https://github.com/spruceid/didkit/tree/main/lib/web#asm-target relevant to me?

Now also relates to https://github.com/spruceid/ssi/pull/129, I added a commit to this PR to handle the changes to DIDTz and add WASM tests.
Squashed all commits.

Only changed the ssi commit in the CI.
For what it's worth, the compilation for asm.js worked for me, but I didn't run any test after that.


Enable PDS compliance

The CLI binary has been moved to bin to achieve PDS compliance.

Should be good to go now.

thanks


the app stuck on "pending" forever after dragging the folder

Before opening, please confirm:

[X] I have checked the custom domains troubleshooting guide.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d2outlzd2gbtvh
Region
eu-central-1
Step
updating build by dragging folder
Status
Pending
Issue or question
The app stuck on "pending" forever after dragging the folder
Error message
No response
Additional information
No response

Hi @repepto, I was not able to reproduce this behavior. Can you provide a sample repository that produces this bug? Were you able to successfully update your application before?

Hi. I v noticed that happens sometimes.  The app may be updated several times very fast but sometimes it stucks on pending for a very long time.  About 10 hours or even more.  Reuploadings don't solve issue. So waiting is only way.Serhii,Best
-------- Original message --------From: hloriana @.> Date: 4/5/22  02:01  (GMT+05:30) To: aws-amplify/amplify-hosting @.> Cc: repepto @.>, Mention @.> Subject: Re: [aws-amplify/amplify-hosting] the app stuck on "pending" forever after dragging the folder (Issue #2676)
Hi @repepto, I was not able to reproduce this behavior. Can you provide a sample repository that produces this bug? Were you able to successfully update your application before?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

So is this only happening after repeatedly updating the app in a short period of time?

Yes. And sometimes, not every time. Last few updates went smoothly

On 9 May 2022, at 22:22, hloriana @.***> wrote:
So is this only happening after repeatedly updating the app in a short period of time?
—
Reply to this email directly, view it on GitHub https://github.com/aws-amplify/amplify-hosting/issues/2676#issuecomment-1121484057, or unsubscribe https://github.com/notifications/unsubscribe-auth/AANS4T6BNYWA6YBYE4AX75DVJFQWZANCNFSM5SCP4AOQ.
You are receiving this because you were mentioned.

Hi @repepto 👋 , thanks for your continued patience and apologies for the delayed response on this thread.
Please refer to this FAQ in our troubleshooting guide: https://github.com/aws-amplify/amplify-hosting/blob/main/FAQ.md#deployments-or-jobs-are-stuck-with-a-pending-status-in-the-amplify-console which outlines some potential reasons why the manual deployments can fail while using the drag and drop functionality in the Amplify Console.
Our recommendation would be to upload a zip file to an Amazon S3 bucket as this avoids file uploads from the Amplify Console which should offer a higher reliability for manual deployments: https://docs.aws.amazon.com/amplify/latest/userguide/manual-deploys.html#amazon-s3-or-any-url

Closing the issue due to age.


What actually is considered "recent" by the recent files API?

Could you define "recently used" in terms of last accessed time ?

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

ID: 1a8c752b-abbb-c5d5-e4b5-771604aea187
Version Independent ID: 8ca45129-5a6c-a98f-775f-6dfdd3aa0b35
Content: List Recent Files - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/drive-recent.md
GitHub Login: @rgregg
Microsoft Alias: rgregg

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Support mean in NNVM to Relay converter.

Support mean in NNVM to Relay converter.

Thanks, @lixiaoquan , this is now merged


Update wrong links in README.md

some links is wrong in README.md

Thank you. The beginning of the README page refers to the typesetting of easydict.


No output file

When running:
python3 inference_video.py --ext=4 --video=video.mp4 --scale=1 --fps=240
It renders more frames, and does its thing.
But there is no output file, do you know how to fix this?
I am using --fps=240 to not try to merge audio.
Thanks!

It should not be --ext ...
--exp

ah yes, thanks!


Add all Python stubs in tantivy.pyi

This is a follow-up of #168.
I added all stubs for currently exposed classes/public methods in tantivy-py.
It seems this works fine for me - I checked that with VSCode.

Thanks!

@cjrh Thank you for the quick responce!


IAM role in README should also indicate start instance.

I just used this in a pair of lambda functions to start and stop a game server.  Works great!  I don't have experience with lambda either.
Anyways, I'm pretty sure that the IAM role document example in the readme should be allowed to start instances.  See commented line below and let me know if I misunderstand.
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "MyStatementId",
            "Effect": "Allow",
            "Action": [
                "ec2:DescribeRegions",
                "ec2:DescribeInstances",
                "ec2:StopInstances",
                "ec2:StartInstances"     /// <-- Include this in your README?
            ],
            "Resource": [
                "*"
            ]
        }
    ]
}

Ah yes can you please make a PR and I will merge it. Thanks :)


Stop container by container name

First of all: Thanks for the awesome Docker Maven plugin - we use it for integration and performance tests and the linking, cleanup, etc. works just great.
We would like to use it as well for running the "latest and greatest" of our containers and do so in a nightly Jenkins job.
The plugin almost provides everything for that:

namingStrategy: alias
fixed port mappings

The only thing which is missing would be to stop containers by their container name.
docker:stop would therefore need another parameter:
containerNameToStop (or similar)
Is that possible?

How about using a extra pom or profile for integration servers?
I can see inside the documentation the remove and build targets can be manipulated global .

a profile may or may not help in this case, it depends on if you have other container definitions and how they are managed in the overall pom. see issue #207.
what's the use case for this? how do you see the jenkins job working?

how do I define the docker stop in the pom.xml file?
currently i need to run in command line : mvn docker:stop


docker compose example

Could you create please some docker-compose examples?

Heres what it would look like
version: "3.9"  # optional since v1.27.0
services:
  proxybroker2:
    image: bluet/proxybroker2
    command: serve --host <IP_ADDRESS> --port 8888 --types HTTP HTTPS --lvl High --min-queue 5
    ports:
      - 8888

I'm not really sure what else you would use as a service.

@Heggeg problem solved?


Add icon and metadata

Summary
This PR adds an icon for the Marketplace and some additional metadata to the manifest.

Heads up: I also updated the bot icon per https://github.com/mattermost/mattermost-plugin-confluence/issues/55.


Python throwing error while training the model

When I try to train model using command
python learn.py
Python throws below error.
Prateek-Mac:gesture-keyboard-master Prateek$ python learn.py
Loading the dataset from 'data'...
Traceback (most recent call last):
  File "learn.py", line 56, in <module>
    number = ord(category) - ord("a")
TypeError: ord() expected a character, but string of length 0 found

I've experienced that error. It happens when you copy the dataset to some types of file systems and the "space" char gets removed in the file name.
I will solve this problem in the next release, but in the meanwhile you can solve it in this way:
change this line:
category = name.split("_")[0]
with this line:
category = name.split("_")[0]
if category=="":
    category = " "

You just tell the program to replace the void character with a space.
That should work.


libvirt-coreos kube-addon.service failed

Hi,
I am using Kubernetes 15.0 with kvm. I followed the getting started guide for libvirt-coreos : https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/getting-started-guides/libvirt-coreos.md
But when I do kube-up.sh not a single node is added to Kubernetes:
Starting cluster using provider: libvirt-coreos
... calling verify-prereqs
... calling kube-up
KUBE_MASTER_IP: <IP_ADDRESS>
KUBE_MASTER: kubernetes-master
--2015-04-23 10:23:17--  http://alpha.release.core-os.net/amd64-usr/current/core
os_production_qemu_image.img.bz2
Resolving alpha.release.core-os.net (alpha.release.core-os.net)... <IP_ADDRESS>
8, 2a00:1450:4013:c00::80
Connecting to alpha.release.core-os.net (alpha.release.core-os.net)|<IP_ADDRESS>
28|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 142383897 (136M) [application/octet-stream]
Server file no newer than local file ‘cluster/../cluster/libvirt-coreos/coreos_p
roduction_qemu_image.img.bz2’ -- not retrieving.

Pool kubernetes refreshed

Network kubernetes_global created from cluster/../cluster/libvirt-coreos/network
_kubernetes_global.xml

Network kubernetes_pods created from cluster/../cluster/libvirt-coreos/network_k
ubernetes_pods.xml

Vol kubernetes-minion-1.img created

Pool kubernetes refreshed

Domain kubernetes-minion-1 created from /tmp/tmp.pJXbGPKXfV

Vol kubernetes-minion-2.img created

Pool kubernetes refreshed

Domain kubernetes-minion-2 created from /tmp/tmp.085mHBEZT8

Vol kubernetes-minion-3.img created
Pool kubernetes refreshed

Domain kubernetes-minion-3 created from /tmp/tmp.1qkILoK4fw

Vol kubernetes-master.img created

Pool kubernetes refreshed

Domain kubernetes-master created from /tmp/tmp.dIho46QC3A

current-context: ""
Running: cluster/../cluster/libvirt-coreos/../../cluster/../cluster/libvirt-core
os/../../platforms/linux/amd64/kubectl --server=http://<IP_ADDRESS>:8080 config
set-cluster libvirt-coreos --server=http://<IP_ADDRESS>:8080
open /root/.kube/.kubeconfig: no such file or directory
current-context: ""
Running: cluster/../cluster/libvirt-coreos/../../cluster/../cluster/libvirt-core
os/../../platforms/linux/amd64/kubectl --server=http://<IP_ADDRESS>:8080 config
set-context libvirt-coreos --cluster=libvirt-coreos
open /root/.kube/.kubeconfig: no such file or directory
current-context: ""
Running: cluster/../cluster/libvirt-coreos/../../cluster/../cluster/libvirt-core
os/../../platforms/linux/amd64/kubectl --server=http://<IP_ADDRESS>:8080 config
use-context libvirt-coreos --cluster=libvirt-coreos
open /root/.kube/.kubeconfig: no such file or directory
Wait for cluster readiness
Nb ready minions: 0 / 3
Nb ready minions: 0 / 3
Nb ready minions: 0 / 3
Nb ready minions: 0 / 3
Nb ready minions: 0 / 3
Nb ready minions: 0 / 3
Nb ready minions: 0 / 3
Nb ready minions: 0 / 3

When I ssh to the master I can see that kube-addons.service failed.
here is sytemctl status kube-addons.service :
● kube-addons.service - Kubernetes addons
   Loaded: loaded (/etc/systemd/system/kube-addons.service; disabled; vendor pre
set: disabled)
   Active: failed (Result: exit-code) since Thu 2015-04-23 08:23:52 UTC; 14min a
go
     Docs: https://github.com/GoogleCloudPlatform/kubernetes
  Process: 655 ExecStart=/opt/kubernetes/bin/kubectl create -f /opt/kubernetes/a
ddons (code=exited, status=1/FAILURE)
  Process: 647 ExecStartPre=/bin/sleep 10 (code=exited, status=0/SUCCESS)
  Process: 569 ExecStartPre=/bin/bash -c while [[ "$(curl -s http://<IP_ADDRESS>:80
80/healthz)" != "ok" ]]; do sleep 1; done (code=exited, status=0/SUCCESS)
 Main PID: 655 (code=exited, status=1/FAILURE)

Apr 23 08:23:38 kubernetes-master systemd[1]: Starting Kubernetes addons...
Apr 23 08:23:52 kubernetes-master kubectl[655]: Error: [502:  (unhandled http status [Temporary Redirect] with body []) [0], 502:  (unhandled http status [Temporary Redirect] with body []) [0]]
Apr 23 08:23:52 kubernetes-master systemd[1]: kube-addons.service: main process exited, code=exited, status=1/FAILURE
Apr 23 08:23:52 kubernetes-master systemd[1]: Failed to start Kubernetes addons.
Apr 23 08:23:52 kubernetes-master systemd[1]: Unit kube-addons.service entered failed state.
Apr 23 08:23:52 kubernetes-master systemd[1]: kube-addons.service failed.

And the journalctl _PID=655 is not very helpful and display the same error.
If it's not a bug, can someone help me with what is wrong?

The error message you get looks like the issue reported in #6756
@MCyprien : Could you check if you have the same “502: (unhandled http status [Temporary Redirect] with body []) [0]” errors in the kube-apiserver with journalctl -u kube-apiserver
If so, could you check which version on etcd is running?
ps aux | grep etcd
<the_path_you_found>/etcd --version

If you have etcd 0.4.9, that’s the reason of your problem.
Basically, CoreOS starts with an etcd 0.4.9 and kubernetes uses a 2.0.4 client.
I sent a fix in order to have etcd 2 server inside CoreOS.
This fix is #6756
This fix is in the master branch, but not on the v0.15.0 tag.
You should either try a version which contains that fix or try to apply the patch of that PR to your v0.15.0.

thank you! that was the problem!


refactor: restructure the code to align with other odpf projects

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
7785

Relevant Lines:
10015

💛  - Coveralls


Card flickers on HA 2022-11

Is anyone having an issue with the radar card flickering/reloading when the starting the sequence of radar images?
This card worked with the last version, 2022-10 but exhibited this problem after updating to 2022-11.
For me, the problem is on Chrome, Edge and Firefox. Cleared the cache in all and no change.
My config is:
type: custom:weather-radar-card
data_source: RainViewer-Original
show_zoom: true
show_marker: false
static_map: false
show_playback: false
square_map: false
extra_labels: false
show_scale: false
show_range: false
zoom_level: 8
show_recenter: true
It's on a Panel (1 card) view.
Video of the problem in OneDrive https://1drv.ms/u/s!ArXzPwEK-0K_nYx3AL4yK0EmiYvZwQ?e=sC219g

Can you please test a pre-release 2.1.1 that hopefully fixes this.

Installed 2.1.1 and it looks good.
Thanks for the awesome product!


NgModel with custom directives within tabs

Bug description:
Well known technique that for saving form validation on models used binding ngModel via require ngModelController in directive. But if custom directive, which used inside, has similar solution parent scope does not take changes.
Link to minimally-working plunker that reproduces the issue:
https://jsfiddle.net/ow40dpo3/1/
Version of Angular, UIBS, and Bootstrap
Angular: 1.6.1
UIBS: 2.3.2
Bootstrap: 3.3.7

I found problem. question is closed. sorry


Take advantage of Postgres native JSON support

Fixed with: https://github.com/creactiviti/piper/commit/033f3bef26b0c9a13b3cbecc8fd29985bb711494


[ECS] Force New Deployment

Fixes #8219

@bblommers ready for review

@bblommers ready for review

Not sure what's up with the linting errors. MyPy ran without issue locally, and all of the errors are from services I didn't touch

@bblommers Ready for launch


Generate EXECUTE SCRIPT slonik

DBSteward has historically not included the EXECUTE SCRIPT commands to run stage 1 and stage 3 schema changes after all slonik stage 1 and 3 commands have been executed.
Update slonik output to include EXECUTE SCRIPT statements to run these files. There is no reason not to be doing this at this point.

Implemented in https://github.com/nkiraly/DBSteward/pull/90
I tested this by developing https://github.com/nkiraly/dbsteward-maven-plugin/issues/1
Working as of https://github.com/nkiraly/dbsteward-maven-plugin/commit/1900aaa1613715d0a0755931da8b297c6342ad1b  1.4.0-SNAPSHOT


Refactor TransactionsComponent to accept any query type and value

Refactored TransactionsComponent to accept any query type and value (thereby decoupling it from hard-coded query types)
In transactions template, added links to addresses and transactions
Added an AddressPage with placeholders

Codecov Report

Merging #787 into ionic will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            ionic     #787   +/-   ##
=======================================
  Coverage   72.72%   72.72%
=======================================
  Files          15       15
  Lines         143      143
  Branches        2        2
=======================================
  Hits          104      104
  Misses         32       32
  Partials        7        7

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 8852924...22351df. Read the comment docs.


[Event Request] PAGE 234 "Apply Employee Entries" OnBeforeHandledChosenEntries

local procedure HandlChosenEntries(Type: Option Direct,GenJnlLine; CurrentAmount: Decimal; CurrencyCode: Code[10]; PostingDate: Date)
var
.
.
.
begin

IsHandled := false;
OnBeforeHandledChosenEntries(Type, CurrentAmount, CurrencyCode, PostingDate, AppliedEmplLedgEntry, IsHandled);
if IsHandled then
exit;

    if not AppliedCustLedgEntry.FindSet(false, false) then
        exit;
.
.
.

[IntegrationEvent(TRUE, false)]
local procedure OnBeforeHandledChosenEntries(Type: Option Direct,GenJnlLine; CurrentAmount: Decimal; CurrencyCode: Code[10]; PostingDate: Date; var AppliedEmplLedgerEntry: Record "Employee Ledger Entry"; var IsHandled: Boolean)
begin
end;

This is the method's signature as we see it now:
local procedure HandlChosenEntries(Type: Option Direct,GenJnlLine; CurrentAmount: Decimal)
Are you also asking for parameters CurrencyCode and PostingDate to be added?

I'm sorry, my mistake. No need for additional parameters:

IsHandled := false;
OnBeforeHandledChosenEntries(Type, CurrentAmount, AppliedEmplLedgEntry, IsHandled);
if IsHandled then
exit;

[IntegrationEvent(TRUE, false)]
local procedure OnBeforeHandledChosenEntries(Type: Option Direct,GenJnlLine; CurrentAmount: Decimal; var AppliedEmplLedgerEntry: Record "Employee Ledger Entry"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


[WIP] Profile Pages

WIP

I updated the branch with a rebase in order to trigger the rspec workflow.


⚡ Always run server-class JVMs inside our containers

In the default cloud-platform-environments configuration, we have 1GB memory allocated to each pod.
Since JDK 10, this fails the criteria of "server class machine", so the JVM starts with serial garbage collector. (Server class is defined as ">= 2 physical CPU's and >=2GB of memory")
The serial GC is a "stop-the-world" garbage collector and so creates unnecessary pauses in production.
The effect of -XX:+AlwaysActAsServerClassMachine is that the default GC will be the server-class GC (G1 GC in most cases).
References:

https://github.com/openjdk/jdk/blob/jdk-17%2B35/src/hotspot/share/runtime/os.cpp#L1674-L1675
https://stackoverflow.com/a/52477898/621923

Pulls in changes done in:

https://github.com/ministryofjustice/hmpps-interventions-service/pull/852

This can be tested by adding command: ["java", "-XX:+PrintFlagsFinal", "-javaagent:/app/agent.jar", "-jar", "/app/app.jar"] into the Deployment configuration and monitor logs
If you see
bool UseG1GC                                  = false                                     {product} {default}
{snip}
bool UseSerialGC                              = true                                      {product} {ergonomic}

it confirms your app is running serial GC.


logInWithCredentials: fails for macOS

Goals
What do you want to achieve?
Login to realm with a username/password pair and synchronize across iOS/Mac.
Expected Results
Realm logs in correctly on iOS and Mac and synchronizes.
Actual Results
What did happened instead?
RLMSyncUser is successfully created for iOS and fails for Mac. The code is exactly the same:
Steps to Reproduce
What are steps we can follow to reproduce this issue?
- (void)enableRealmForSession:(NSString *)username {
    if (!username) {
        return;
    }
    NSString *realmUser = [NSString stringWithFormat:@"REALM_USER_%@", username];
    NSString *realmPass = [NSString stringWithFormat:@"REALM_PASS_%@", username];
    RLMSyncCredentials *usernameCredentials = [RLMSyncCredentials credentialsWithUsername:realmUser
                                                                                password:realmPass
                                                                                register:YES];
    [RLMSyncUser logInWithCredentials:usernameCredentials authServerURL:SERVER_URL timeout:30 onCompletion:^(RLMSyncUser * _Nullable user, NSError * _Nullable error) {
        if (user) {
            [self openRealmForUser:user andUsername:username];
        } else if (error) {
            // handle error
        }
    }];
}

logInWithCredentials: succeeds for iOS and fails for Mac with: Error Domain=io.realm.sync Code=3 "(null)" UserInfo={statusCode=400}
Version of Realm and Tooling
In the CONTRIBUTING guidelines, you will find a script,
which will help determining these versions.
Realm version: 1.0.0-BETA-4.2
Xcode version: 8.1
iOS/OSX version: 10.1.1 / 10.11.6
Dependency manager + version: Using frameworks downloaded from realm.io for Objc on the client side and ubuntu + apt-get for server

According to your code sample you register an account each time you try to log in. Please register an account only once and then use an existing one by specifying register:NO for logInWithCredentials method.
The error reporting was improved in https://github.com/realm/realm-cocoa/pull/4246.

How would I go about checking if a user is already created from a new client?
Ideally shouldn't the realm server deal with this appropriately and create a user if needed otherwise return an existing user when register is YES?

Currently the only way to detect if something is wrong with the credentials is to check the statusCode of the returned error, but unfortunately right now it doesn't contain the info about what exactly went wrong. #4246 introduces the new error codes:

RLMSyncAuthErrorInvalidCredential
RLMSyncAuthErrorUserDoesNotExist
RLMSyncAuthErrorUserAlreadyExists

@stel RLMSyncAuthErrorUserDoesNotExist and RLMSyncAuthErrorUserAlreadyExists will never be returned in practice, as that would leak private details about which accounts exist.

Then for the current version of realm, the way to go about this is first perform a login with register:YES and then attempt to retry with register:NO if it fails?

No, if you expect the user with the provided credentials to exist, you should pass in register: NO. You probably shouldn't be synchronizing username and password across versions of your app before you know that they're valid.

I hope that answered your question @dsanghan!
If you have additional follow-up questions, please feel free to re-open this issue and ask them in here. :)
Happy holidays!

Couldn't the error description be something like:

The provided credentials are invalid or a user with these credentials already exists.

instead of just

The provided credentials are invalid.

?
As it's pretty confusing now. And the "new" message is pretty standard and doesn't leak private details.

Thanks for the suggestion @revolter, I've forwarded the request to our server team.

What if Error Domain=io.realm.sync.auth Code=3 "(null)" UserInfo={statusCode=400} is returned when trying to log in using SyncCredentials.facebook(token: facebookAccessToken)?


Update UG

Details

Restructured the whole UG as per CS2101
Reformatted the whole UG as per CS2101
Edited the help window

Merge first and edit your respective parts accordingly to the format. The user-centricity of the UG is currently done to my preference. If it feels out of place, please highlight it and suggest improvements so that I may change it.

Codecov Report

Merging #166 (dbeac22) into master (45a1087) will increase coverage by 19.34%.
The diff coverage is 24.46%.

@@              Coverage Diff              @@
##             master     #166       +/-   ##
=============================================
+ Coverage     32.14%   51.48%   +19.34%
- Complexity      170      341      +171
=============================================
  Files            67       83       +16
  Lines          1148     1482      +334
  Branches        111      157       +46
=============================================
+ Hits            369      763      +394
+ Misses          761      661      -100
- Partials         18       58       +40

Impacted Files
Coverage Δ

src/main/java/seedu/trackermon/AppParameters.java
75.00% <ø> (ø)

src/main/java/seedu/trackermon/Main.java
0.00% <ø> (ø)

src/main/java/seedu/trackermon/MainApp.java
0.00% <0.00%> (ø)

...ain/java/seedu/trackermon/commons/core/Config.java
68.18% <ø> (ø)

...ava/seedu/trackermon/commons/core/GuiSettings.java
42.30% <ø> (ø)

...java/seedu/trackermon/commons/core/LogsCenter.java
78.37% <ø> (ø)

...n/java/seedu/trackermon/commons/core/Messages.java
0.00% <ø> (ø)

...in/java/seedu/trackermon/commons/core/Version.java
82.50% <ø> (ø)

...ava/seedu/trackermon/commons/core/index/Index.java
100.00% <ø> (ø)

...on/commons/exceptions/DataConversionException.java
100.00% <ø> (ø)

... and 104 more

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 45a1087...dbeac22. Read the comment docs.


Entity mapping from one DbContext instance is incorrectly reused in a second DbContext instance

So when mapping a view there is a workaround that can be taken.

when creating/installing the db dont map the view (to avoid EF creating a table named by the view)
when using the db map the view

This can be done by passing a bool into the custom DbContext, so the same context can be used from the installation context and the usage context
public class MyDataContext : DbContext
{
    bool mapViews;
    public DbSet<MyView> MyViews { get; set; }
    public DbSet<MyTable> MyTables { get; set; }

    public MyDataContext(DbContextOptions<MyDataContext> options, bool mapViews) : base(options)
    {
        this.mapViews = mapViews;
    }

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.Entity<MyTable>();

        // only map View when not in install mode.
        // otherwise EF incorrectly creates a table named after the view
        // and the view creation scripts will fail with a name conflict
        if (mapViews)
        {
            modelBuilder.Entity<MyView>(entity => { entity.HasKey(e => e.Id); });
        }
        else
        {
            modelBuilder.Ignore<MyView>();
        }
    }
}

However when instantiating two different instances of said context, one with mapViews: false and one with mapViews: true, the entity mapping from the first context seems to be re-used in the second context.
[Fact]
public void ReGenSqlExpress()
{
    using (var context = new MyDataContext(GetOptions(), mapViews: false))
    {
        context.Database.EnsureCreated();
    }

    using (var connection = new SqlConnection(connectionString))
    {
        connection.Open();
        using (var command = connection.CreateCommand())
        {
            command.CommandText = @"
IF (NOT EXISTS (SELECT 1 FROM sys.views WHERE name = 'MyViews'))
BEGIN
EXECUTE('CREATE VIEW MyViews AS SELECT * FROM MyTables');
END";
            command.ExecuteNonQuery();
        }
    }

    using (var context = new MyDataContext(GetOptions(), mapViews: true))
    {
        var myTable = new MyTable
        {
            Title = DateTime.Now.ToString()
        };
        context.MyTables.Add(myTable);
        context.SaveChanges();
        var myViews = context.MyViews.ToList();
        Debug.WriteLine(myViews);
    }
}

This results in a  InvalidOperationException : Cannot create a DbSet for 'MyView' because this type is not included in the model for the context.
note that, after the first run to create the db, if you comment out the following
using (var context = new MyDataContext(GetOptions(), mapViews: false))
{
    context.Database.EnsureCreated();
}

The usage of the view will work
full repro: https://github.com/SimonCropp/EfViewMappingRepro
Targeting SQL Server 2016 on windows 2012R2
repro'd with the following nuget combos

Microsoft.EntityFrameworkCore 2.0.2 + Microsoft.EntityFrameworkCore.SqlServer 2.0.2
Microsoft.EntityFrameworkCore 2.1.0-preview2-final + Microsoft.EntityFrameworkCore.SqlServer 2.1.0-preview2-final

also verified on both netcoreapp2 and net461

note that there seems to be some static "type to mapping" caching going on. since if u split the data context into a base and two implementations (https://github.com/SimonCropp/EfViewMappingRepro/tree/inheritanceWorkaround) it seems to work as expected

@ajcvickers thanks for the prompt reply

@ajcvickers given this is very difficult to debug. wouldnt it be better to detect and log that there are conflicting context both loaded into the static context?

Re-purposing this issue to improve the logging for this, as suggested by @SimonCropp.

Thanks @ajcvickers


please release 0.0.12

Could you please release a 0.0.12 so we can use the value1/2/3 again with ifttt.
Thanks

previous version with following codes work well with value1,2,3.
But the latest 0.0.11 use request.post.
Please check and help release new version. thanks.

 request({

   uri: 'https://maker.ifttt.com/trigger/' + node.config.eventName + '/with/key/' + node.key.credentials.key,

   method: 'POST',

   json: {

     value1: msg.payload.value1,

     value2: msg.payload.value2,

     value3: msg.payload.value3

   }

===========

+1 passing values to IFTTT doesn't work for me either

Please update the package.json version in master to 0.0.12 and publish to NPM. This module does not work with the current 0.0.11 published in NPM.  The code was fixed last October, and it's time to push those fixes to NPM so people installing from the node-red console can enjoy this module.
Thank you!

Released 0.0.12 today, fixing the issue that prevented value1-3 from being sent to IFTTT.


Feature Request: preload Method

It would be great to have a preload(url) on the imageLoader Service to allow to cache an image before a view with <img-loader src="MYURL"></img-loader> was rendered. This would be good to preload images if you are offline.

Great idea, will add it when I get a chance.

You can use ImageLoader.getImage(..) to do this, but I just added a preload method so it's more clear. Will be available soon in v1.4.0