iOS production crash

iOS production build crashes on iPhone SE 3 after adding react-native-beacon-radar 0.3.2 but simulator does not. Is it expected?
The crash happens in the following function.
void RCTNativeModule::invoke(unsigned int methodId, folly::dynamic &&params, int callId)
npx react-native info
info Fetching system and libraries information...
System:
OS: macOS 13.4
CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
Memory: 1.47 GB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 19.8.1 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 9.5.1 - /usr/local/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK:
API Levels: 30, 31, 33
Build Tools: 30.0.2, 30.0.3, 33.0.0
System Images: android-29 | Google APIs Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-31 | Google APIs ARM 64 v8a, android-31 | Google APIs Intel x86_64 Atom, android-33 | Google APIs Intel x86_64 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.10121639
Xcode: 14.3.1/14E300c - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_201 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.11 => 0.71.11
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found

Does this issue only occur in production builds?

Yes, this issue occurred only in a production build, not in simulator.

Do you have a project which I can use to reproduce this? I am unable to reproduce this in my own test projects.


Unit test for duplicated points in an STM

Issue coming from a discussion in PR #66
A duplicated coordinate (lat, lon, time) will cause the spatial temporal query enrich_from_dataset  fail. We need to create a check function to validate there is no duplicated 3D coordinates in an STM.
Also quote Sarah's comments here which are good for consideration:

@rogerkuou to check if the points are unique, the test np.unique(ds['lat'].values).shape == ds['lat'].values.shape is not enough because it only checks the duplicates in one dimension here lat. However, for example, points can be located on one line.
Instead, we need a test if there are cases where (lat, lon, time) are duplicated. Functions like xarray.Dataset.drop_duplicates and pandas.DataFrame.duplicated can be used to write a test. But these functions only work on dim and not coords. In our cases, lat and lon are coords and space is the dim. So we might need to use unstack which leads to memory problems.

and also this:

As discussed, scipy KDTree works if coordinates (lat, lon, time) are duplicated and the values of variables e.g. temperature are the same too, I added a test for this. If the values of variables are not the same for duplicated coordinates, MacOs and linux behave differently to pick up a value related to the nearest neighbor. For this extreme case, I suggest creating a new issue and fixing it in another PR.


Backport "Workaround #19542" to LTS

Backports #19561 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-19520.
Reference


fix: ephemeral builds

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

closing in favor of https://github.com/apache/superset/pull/31385

/testenv up

/testenv up


Influxdb2 eats up 41G of memory, how to optimize it

Let me talk about the environment first:
influxdb-version:2.0.7;
go-version:1.15.2;
Linux on physical machine: 64G of memory.
The problem is that now that 41G of memory is used, I'm very worried that it will take up unlimited memory and eventually cause the process to crash. The following is relevant information. It occupies 66% of the memory. The actual memory allocated to it is about 13G. It has 3 buckets and a total of 70G files are occupied. And its buff/cache usage is very high. I understand that this is the index. Regarding the memory used by data, I want to limit the high memory usage of influxdb.

How to deal with it?
Please help me and Thanks very much!

Check out this response on reddit.
Also, if you use a newer version you can take advantage of GOMEMLIMIT; but influxdb uses mmap so it is always going to "use" as much memory as possible for page cache, but that is managed by the OS and high mmap shouldn't cause a out-of-memory situation. You may have an runtime: out of memory if golang cannot allocate more memory for the heap (or stack). See the top man page for more details.

First of all, I am very grateful for your reply and confusion. For some reasons, I cannot open reddit well and cannot see the reply information to this question. Could you please elaborate more on how to optimize this problem, and do you have any other suggestions? Greetings to you.

Check out this response on reddit.
Also, if you use a newer version you can take advantage of GOMEMLIMIT; but influxdb uses mmap so it is always going to "use" as much memory as possible for file page cache, but that is managed by the OS and high mmap shouldn't cause a out-of-memory situation. You may have an runtime: out of memory if golang cannot allocate more memory for the heap (or stack). See the top man page for more details.

First of all, I am very grateful for your reply and confusion. For some reasons, I cannot open reddit well and cannot see the reply information to this question. Could you please elaborate more on how to optimize this problem, and do you have any other suggestions? Greetings to you.

I don't have anything else to suggest other than the links and information I shared above.

OK, thank you. I plan to upgrade the version to 2.7 and try it again.


The pbench-user-benchmark invocation of run_snafu seems to be incorrect

The use of pbench-user-benchmark at line 59 of workloads/templates/workload-fio-script-cm.yml appears to be incorrect.
By definition of how pbench-user-benchmark works, when run_snafu returns, pbench-user-benchmark will stop execution and all collection of tool data, and the created /var/lib/pbench-agent/pbench-user-benchmark_${config}_${datestamp} directory it created will be buttoned up.
Later on line 79, when fiotest.sh is invoked, another pbench run directory will be created for that fio test.
Perhaps consider one of two changes:

Drop the use of pbench-user-benchmark at line 59, and just invoke run_snafu without it.
Drop the use of pbench-fio at line 105 and let "snafu" orchestrate the fio workload itself.

Thank you for filing this @portante.
Seems there are a few other pieces to fio tests that are failing.
For example, there is an error at the following line.
https://github.com/openshift-scale/workloads/blob/b800a7d72714180393639a83565c658e76e3742f/workloads/templates/workload-fio-script-cm.yml.j2#L102
Mon Sep 28 16:03:35 UTC 2020 Pods for FIO I/O test created.
sed: couldn't open temporary file /opt/pbench-agent/sedN4W3bm: Permission denied
[error][2020-09-28T16:03:40.021683720] pbench-fio: package version is missing in config file

Seems that expectations around the folder structure for /opt/pbench-agent, context user, etc.


[0.8.1] I can delete the currently executing job in the spooler and it keeps running nevertheless.

I think there should probably be separate context-menu items for:

cancel currently burning job: which lets the instructions already sent to the laser complete, then puts the laser into a known state (and possibly pops up an interrupt box if there is already another job queued); and
remove queued jobs

Also remove job on the current job doesn't give an entry in the job history despite that job having partially run.

If you hit stop mostly it will do this and kill that stuff. This is an estop. Cancels the currently burnings, breaks the running, and kills any followup jobs.
You probably should not be allowed to delete the current job. You should be required to cancel it. In which case it should probably stop at the first opportunity and move on to the next job in the queue.
If removed it didn't complete and thus isn't in the history but it should probably note the state in the history. Job run, made it up to this position. Then it was killed. There's also some chance you could maybe store enough information about that job that you could resume it. Though that might be a bit non-obvious but if you knew exactly where it stopped you might be able to resume as such.

Exactly!

That should be handled better now


Changed link in CONTRIBUTING.md

Signed-off-by: Vinayak<EMAIL_ADDRESS>
What this PR does / why we need it:
Since most of us are English speakers and readers, I changed the link of git book - aliases chapter to the English language one.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
I hope the link isn't meant for Turkish readers/speakers

Please fix dco signoff

@princerachit I missed signing off the commit. How do I fix that? Do I need to remove this commit and push another one with the sign-off?

@vinayak42 you can either ammend your commit message or mention the email id in the PR description.

@kmova I have mentioned email id in PR description, still, it is showing that sign-off is missing. Also, I am not being able to amend my commit message.

Email ID<EMAIL_ADDRESS>

Hi @vinayak42 . I have sent you a mail regarding Hacktoberfest Swags. Can you please check your inbox and fill up the form

Filled it up. Thanks!


Registrator always appends port number with multiple exposed ports

On master, if I use -e SERVICE_80_NAME=http -e SERVICE_443_NAME=https I'll always get a service with the names registered as http-80 and https-443 when I'm expecting just http and https.
Is that intended?

Check the other issues - there is already a lot of discussion on this - a lot of people are trying to revert this behaviour (or make it optional)

See #245

@gtmtech are there other relevant issues besides #245?

I think this fixes it. It would be good to have a v8 release sometime @progrium  as master is always changing on dockerhub


Add devcontainer Configuration for easy integration & reproducibility

-Lokal VSCode(LatexWorkshop) configuration for correct commands
-Docker image with all dependencys nicely bundled
-Devcontainer definition with plugin installation
-Updated Thesis.pdf for comparison
Whats missing:
-documentation

This is a integration of the template into VSCode as a Devcontainer with automatic Plugin installation & configuration via Workspace.
To use:
Install VSCode with remote Container Extension (https://code.visualstudio.com/docs/remote/containers)
Open Remote Container Folder -> Select git Repository
To Compile execute VSCode command(strg + p) "Latex Workshop:Build Latex Project" on Thesis.tex
Is there interest in this or is everyone fine using a local installation ?

Danke für den PR!


Possible improvement on chunk_barcoded_bam.py

Thank you for creating this useful toolkit. When running the software on a really large combined libraries (~200k cells to consider), I found the bottleneck becomes the chunk_barcoded_bam.py part, and I found possible solutions to improve it.

transform cell barcode list from list to set bc = set([x.strip() for x in content]) which improves the speed of checking existence of barcodes a lot (~800 records /s -> ~100k records / s)
Use pysam read.get_tag, instead of the iteration way

def getBarcode(read, tag_get):
  '''
  Parse out the barcode per-read
  '''
  # for tg in read.tags:
  # 	if(tag_get == tg[0]):
  # 		return(tg[1])
  # return("AA")
  try:
    read.get_tag(barcodeTag, tag_get)
  except:
    return ("AA")

This improves the speed from ~100k records/s -> 130k records/s

Thanks for the input! Would you be able to open a PR for this?
On Jun 25, 2023, at 1:16 PM, ruochiz @.***> wrote:

Thank you for creating this useful toolkit. When running the software on a really large combined libraries (~200k cells to consider), I found the bottleneck becomes the chunk_barcoded_bam.py part, and I found possible solutions to improve it.

transform cell barcode list from list to set bc = set([x.strip() for x in content]) which improves the speed of checking existence of barcodes a lot (~800 records /s -> ~100k records / s)
Use pysam read.get_tag, instead of the iteration way

def getBarcode(read, tag_get):
'''
Parse out the barcode per-read
'''
for tg in read.tags:
if(tag_get == tg[0]):
return(tg[1])
return("AA")
try:
read.get_tag(barcodeTag, tag_get)
except:
return ("AA")
This improves the speed from ~100k records/s -> 130k records/s
—
Reply to this email directly, view it on GitHubhttps://github.com/caleblareau/mgatk/issues/73, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD32FYOWVXWRSYBGB5H4S6DXNCMARANCNFSM6AAAAAAZTK7Z5Q.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

now implemented in v0.6.8. Thank you very much @ruochiz for the contribution. You should be able to pip install the latest version of the software now.


Custiomization example fixes

🎯 Goal
Some miscellaneous items I found during preparing for the theming blog post
🛠 Implementation details
Provide a description of the implementation
🎨 UI Changes
Add relevant screenshots

:tada: This PR is included in version 3.0.0-theming2.1 :tada:
The release is available on:

npm package (@theming2 dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(ssi): adjust ssiCredentialIssuer address

Description
adjust ssiCredentialIssuer address
Why
The url of the ssi credential issuer component was configured wrongly
Issue
N/A
Checklist

[x] I have performed a self-review of my changes
[x] I have successfully tested my changes
[x] I have added comments in the default values.yaml file with helm-docs syntax ('# -- ') if relevant for installation
[x] I have commented my changes, particularly in hard-to-understand areas

https://github.com/eclipse-tractusx/ssi-credential-issuer/blob/release/v1.0.0-rc.2/consortia/environments/values-dev.yaml#L37

@evegufy that's strange... I double checked on argo, there the api seems to be available under the -backend url.
for example the SSI Swagger

Closing this PR in favour of #76 the path is adjusted there.


Out of Memory for Init Training

There are parts of Init Training when it is loading a model to be used that uses too much VRAM due to the model size, but would otherwise work for training outside of the initialization. For example when using the split mode it fails to initialize as it runs out of memory loading the entire model (flux dev fp8) which it fails here.
The card I am using has 12gb of vram, I can't get it to work on Linux but I can on Windows as only the Windows driver has Nvidia's system memory hot swapping. So it can use more than 12gb of vram it has.
Not sure if there's a way to instead do this on the CPU, if there's a way to detect that there isn't enough memory to do it on the GPU. Or add a setting that uses the CPU. There may be a similar issue for other parts as well, when I turn split mode off it then runs out of memory with caching clip models.

On the Init Flux LoRA Training node, you can try to switch split mode to on, it should bring down the VRAM usage.


Ability to add text annotations to plots

Included in this option would be the ability to set font attributes such as font size and colour.

Thanks @supernifty.
One question is how should this feature be used on the command line? Each annotation needs a coordinate and some text and maybe some settings such as font. If there are a lot of them then it could be annoying to enter them.
One option might be to have a feature which works okay for a small number of annotations, where they are listed on the command line. And another option that would support a larger number of annotations that are derived from a table (input CSV etc). The latter might be nice to allow the data to generate the points. However, it probably would be best implemented if Hatch supported multiple tables. For instance you could either read in a table of annotations, or you could compute a new table of annotations on the fly and then use that. So maybe this latter feature is another good reason to support multiple tables. It might be worth thinking more about how that would work. In the meantime can probably easily implement something that works on the command line, if a bit tediously.


Task/swagger identity api

In this pull request:

Add swagger annotations to the Identity API module.

This change is

Codecov Report

Merging #111 into swagger will decrease coverage by 0.02%.
The diff coverage is 50%.

@@              Coverage Diff              @@
##             swagger     #111      +/-   ##
=============================================
- Coverage      86.81%   86.78%   -0.03%
  Complexity      1276     1276
=============================================
  Files            111      112       +1
  Lines           3534     3535       +1
  Branches         152      152
=============================================
  Hits            3068     3068
- Misses           366      367       +1
  Partials         100      100

Impacted Files
Coverage Δ
Complexity Δ

...common/src/main/java/constants/EventConstants.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ain/java/org/oscm/rest/identity/RolesResource.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...in/java/org/oscm/rest/identity/UserParameters.java
55.55% <ø> (ø)
8 <0> (ø)
:arrow_down:

...a/org/oscm/rest/identity/OnBehalfUserResource.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

.../java/org/oscm/rest/identity/LdapUserResource.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...mon/src/main/java/constants/IdentityConstants.java
0% <0%> (ø)
0 <0> (?)

...main/java/org/oscm/rest/identity/UserResource.java
100% <100%> (ø)
6 <1> (ø)
:arrow_down:


Deployments with AppController don't mind of the EnvFrom configmap

Hi!
We're using AppController to generate all our definitions, dependencies and deployments, it works well, but ... We've recently started the use of configmaps, and I think we face a bug.
When creating the deployments (and pods), appController don't mind the configmap and create the pod without take the value referencing to the configmap.
Any idea or solution?
Thanks,
Stephen

Hi!
Here is an sample of the files that are deployed by appController (Attached, i've changed the extention to txt for the upload)
deployment.txt
After some investigation, we think that the problem can be caused by an API version update.
But can't confirm any suggestion to resolve the problem.
Thanks.

Going to try to reproduce the issue, but it's very unlikely to be caused by AppController. We do not do any manipulations with resource definitions. They are deserialized into k8s data structures and passed to the k8s as is.
The first thing I'd check is does this is sat kubectl -f deployment.yaml and see if it works correctly. If it does, I'd wrap it into AppController resource definition and do the deployment. Then I'd take created resource yaml spec from the k8s and compare it with that from deployment that was created earlier with kubectl to make sure that no fields were lost.
Btw, what k8s version do you have?

Thanks for the answer, we're using the last version of k8s (1.7.3).
I'll give you some mor informations :

The deployment using the command kubectl create -f deployment.yml works fine
Running the command kubectl apply -f deployment.yml (same file as the deployment by appController) make the deployment taking care of the configmap.
we use the script (bash) appController.txt to generate definitions (bash script :
generate_definitions.txt and dependencies.

This is the way we make our deployments.
After that, we run the commands :

kubectl exec k8s-appcontroller kubeac -- run
kubectl exec k8s-appcontroller kubeac -- deploy
to force the schedule to start, and thats it.

Hope that help to understand the problem

I will check
Why do you use kubectl apply to upload resource definitions? AppController doesn't support updates. It checks if resource exists and if it doesn't create it. So if it was created earlier, AppController won't update it even if resource definition changed since then. So changing configmap and re-running AC deployment won't help
Also, if you're using latest AC you shouldn't do kubectl exec k8s-appcontroller kubeac -- deploy, because this is what  k8s-appcontroller does in background. So when you execute this command, you actually create second instance of deployment process and they both are going to compete over your deployment

I'm using kubectl apply because we deploy a lot of microservices (arround 20 actually, will be a lot more at the therm of de development), and the docker image that's deployed change after every automatic builds, so the kubectl apply will update the deployment and recreate the pod with the new image (actually works fine).
Hum, thanks for the details about the deploy with AC, that can explain some of my previous problems :+1:

@digibankproject: so that means that we can close this issue as resolved?

the problem of the issue isn't resolved, but some of troubles that i've faced before are explained.


Closing responses kills everything

Remove it ASAP.

In prod mode: crash.
Fair enough.


Aggregate exceptions + Inner Exception bug

An aggregate exception can have multiple inner exceptions. But any of these inner exceptions can be aggregate exceptions themselves. This branching behavior is hard to represent in the current Bugsnag UI, so I've done a fix that end ups generating a new Event whenever this type of branching occurs.
This isn't ideal but at least records information about every exception generated. The only downside is that its not obvious that two events were created from same Aggregate Exception. Any suggestions on how to represent it in the UI?
Also noticed a bug with displaying inner exceptions...they weren't, they were just copies of the original
(link to line) should be ExceptionParser.GenerateExceptionInfo(currentExp, ....
But since I've rewrote that section for this fix, its not a problem here.
Also added some black box testing for some Exception types to cover these types of exceptions

Fix for #22

This looks pretty good to me. I think that having multiple events for aggregate exceptions might be a little confusing. What do you think about just flattening the list to a single list of exceptions? Its unlikely that people will actually have the big chain of aggregates and as long as we show them all the order isnt super important.
I think the bigger issue (that you've fixed) is that caused by isnt working at all right now! Thanks for fixing that @CodeHex!

Changed to add all inner exceptions in a single event.

This looks good to me too!


Found version 1.0.0 in nuget.org package source that is not current with documentation

Hello, I am happy to find this library fit for my testing requirements, however, I first installed a version 1.0.0 from nuget.org via Visual Studio.  It has a date published of May 23rd, 2024 (today) and the correct contact information for this library.  A corresponding Serilog.Sinks.InMemory.Assertions 1.0.0 version is also available.  Neither version has the supported API in documentation and tests matching the release 0.11.0 version on GitHub.  Not sure what gives but could be someone spoofing the library! See linked screenshot.

Ho hum. Let me investigate this 🤔

Did you discover what this issue is? Just updated to v1 and encountered this

It’s strange you see that because 1.0.0 isn’t listed anymore on nuget.org at all.
Are you by any chance using a package cache like CloudSmith or Artifactory?

sorry for late reply been away. Yep, looks like our ADO was caching it thanks, did not think to check that

No worries 👍 glad you were able to find it


Maximum number of backups

It would be great to be able to specify a maximum number of backups. If this number is reached the oldest backup would be deleted automatically each time a new backup is made. If your are interested in this feature I can create a PR.

Now available with:
'sylvainjule.backups.maximum' => 10, // or any integer, default is false

awesome, thanks!


Adding Website Data (non personal) to Gaia

I understand working with Gaia after reading through the info in the blockstack docs, but i dont understand how i can use it for general data and shared date. Examples of these are;
Discussion between two app users
Forum threads that contain inputs from many different users
General website data that will be accessible to all users on the website
In summary, i want to know how some info can be generalized or made accessible to a group of users based on their connection to the specific data

@theallegrarr  As you asked this on the forum as a support question and it was answered, I'm going to close this. For reference, the forum post is here:
https://forum.blockstack.org/t/how-do-i-create-shared-data-on-gaia-storage/9156/2


Move to RQRCode-Core library and clean up

Problem
Issue #3 points out that we don't really need all of RQRCode
and it's dependencies because we're only using the core components.
Solution
Import rqrcode_core by itself and use it.
Changes

add rqrcode_core dependency in favor of rqrcode
clean up the code base with rubocop

Fixes #3


Add listen_interval to wifi ClientConfiguration

The listen_interval configuration allows controlling how often the mcu wakes from light-sleep during automatic power management mode.

Alright. Will close these PR's out in that case.


ensure element exists before access it

I got the following error when I created a layout without a header.
Uncaught TypeError: Cannot read properties of null (reading 'addEventListener')
    at main.min.js:1:2293
    at Array.forEach (<anonymous>)
    at HTMLDocument.<anonymous> (main.min.js:1:2271)
Uncaught TypeError: Cannot set properties of null (setting 'innerHTML')
    at setMode (main.min.js:1:1492)
    at window.onload (main.min.js:1:1803)

so added null checks to ensure that we use these elements when it actually exists.

thanks!


Gasprice values of Transactions in JSON RPC response denoted in GWEI

Description
The json response for a block & transactions has the gasPrices denoted in different format
baseFeePerGas in wei & gasPrice of transactions in Gwei, where it should be in wei
sample
 "transactions": [
            {
                "blockHash": "0x3b75ba801136c9ee47dcbe882bb7d99c1c13e87acc65c64e7d98d1145cd1eb91",
                "blockNumber": "0x5db6f5",
                "chainId": "0x128",
                "from": "0xe471b5198df1cd42dffc2a4a83a7e4f9c74f2617",
                "gas": "0x61a80",
                "gasPrice": "0x093",

Steps to reproduce

curl --location 'https://testnet.hashio.io/api' \
--header 'Content-Type: application/json' \
--data '{
    "jsonrpc": "2.0",
    "method": "eth_getBlockByNumber",
    "params": [
        "0x5db6f5",
        true
    ],
    "id": 1
}'

Additional context
No response
Hedera network
testnet
Version
"relay/0.51.0-rc2",
Operating system
None

@Nana-EC @simsonraj
Relay's request:
{
    "method": "eth_getBlockByNumber",
    "params": [
        "0x5db6f5",
        true
    ],
    "jsonrpc": "2.0",
    "id": 1
}

Response:
{
    "result": {
        ...
        "baseFeePerGas": "0x13356219000",
        ...
        "transactions": [
            {
                ...
                "gasPrice": "0x093",
                "hash": "0x6b2527ec12817318961c7a2378ec6fd4bc75ecdb5697e46ef3be5f760d7bba03",
                ...
            }
        ],
        ...
        "hash": "0x3b75ba801136c9ee47dcbe882bb7d99c1c13e87acc65c64e7d98d1145cd1eb91",
        "parentHash": "0x3d1c46ba632ea7e89b5a6a4d31e75f2ae2cb0ebc233b82ec91878bb204d0c3b5"
    },
    "jsonrpc": "2.0",
    "id": 1
}

weibarCoef - 10_000_000_000
baseFeePerGas - 0x13356219000 = 1_320_000_000_000 (weibars) / 10_000_000_000= 132 (tinybars)
gasPrice - 0x093 = 147 (tinybars)
As far as I understand, the returned gasPrice is not in a gwei but in tinybars . The main question is whether we want to return it in a weibar (same as baseFeePerGas) or stick to the tinybars. I think we should convert it to weibars because end users (and all of the ethereum tools) are working with wei which in hedera context is weibar. Am I missing something @Nana-EC?

@Nana-EC @simsonraj
Relay's request:
{
    "method": "eth_getBlockByNumber",
    "params": [
        "0x5db6f5",
        true
    ],
    "jsonrpc": "2.0",
    "id": 1
}

Response:
{
    "result": {
        ...
        "baseFeePerGas": "0x13356219000",
        ...
        "transactions": [
            {
                ...
                "gasPrice": "0x093",
                "hash": "0x6b2527ec12817318961c7a2378ec6fd4bc75ecdb5697e46ef3be5f760d7bba03",
                ...
            }
        ],
        ...
        "hash": "0x3b75ba801136c9ee47dcbe882bb7d99c1c13e87acc65c64e7d98d1145cd1eb91",
        "parentHash": "0x3d1c46ba632ea7e89b5a6a4d31e75f2ae2cb0ebc233b82ec91878bb204d0c3b5"
    },
    "jsonrpc": "2.0",
    "id": 1
}

weibarCoef - 10_000_000_000 baseFeePerGas - 0x13356219000 = 1_320_000_000_000 (weibars) / 10_000_000_000= 132 (tinybars) gasPrice - 0x093 = 147 (tinybars)
As far as I understand, the returned gasPrice is not in a gwei but in tinybars . The main question is whether we want to return it in a weibar (same as baseFeePerGas) or stick to the tinybars. I think we should convert it to weibars because end users (and all of the ethereum tools) are working with wei which in hedera context is weibar. Am I missing something @Nana-EC?

Good investigation, I think we should be sending back weibar for consistency and equivalence, looks like we missed it as you noted.


Kafka input support

Hi,
I have added kafka input. Instead of use https://www.npmjs.com/package/kafka-node as suggest in #49 I prefer to use kafka-native cause is much align with new Consumer Client API both use https://github.com/edenhill/librdkafka.
On the wire there is  https://github.com/Blizzard/node-rdkafka and IMHO is most flexible respect other two and provide KafkaStream support, but
at the moment i was enable to use client API the for now I choose kafka-native.
From Delivery Semantics point of at the moment The consumer store it position in local file that can be configure then "at-least-once" is respected
Every suggestion are welcome
Thanks for attention
--Filippo

Thanks @fbalicchia !
Is this going to be a bit problematic, as it depends on the shared libs and such, which may or may not exist?
(looks like Travis CI is having problems with exactly this sort of thing)
Similarly, won't this cause problems where one updates librdkafka and inadvertently breaks this kafka input plugin?
Or is this common in Node.js world and people just deal with it?

Opps ! sorry @otisg on my branch I haven't enable travis and see that error. On my local machine I use node v8.1.2 instead in .travis file I don't see specify version 8. I'll fix on my branch and after that I'll reopen issue.
Thanks

I think we can configure travis to compile the lib.  A separate npm package would be possible for the plugin. Then only Kafka users would need to deal with it. See https://github.com/megastef/logagent-tcp-input/blob/master/README.md for an example.

@fbalicchia here is a travis example with enabled C++ compiler:
https://github.com/sematext/spm-agent-nodejs/blob/master/.travis.yml
I hope this helps to make .travis file for your Kafka plugin.
In this repository we could add link and documentation for the Kafka plugin.


Connect refused behind proxy

A connection error occurred when using this action behind a network proxy,
error log here:
Run seanmiddleditch/gha-setup-ninja@master
downloading https://github.com/ninja-build/ninja/releases/download/v1.10.0/ninja-win.zip
D:\github-runners\_work\_actions\seanmiddleditch\gha-setup-ninja\master\dist\index.js:310
    request.on('error', error => { throw error })
                                   ^

Error: connect ECONNREFUSED <IP_ADDRESS>:443
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1129:14) {
  errno: 'ECONNREFUSED',
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 443
}
}

Thanks.

This action downloads the ninja binaries from https://github.com/ninja-build/ninja/releases, and the binary download links apparently redirect to an AWS S3 bucket (the IP that failed for you is S3). I just double checked and even in a local browser I see the links redirect to an S3 URL, so that appears to be normal/expected behavior for github.
Can you confirm whether you have access to S3 behind your proxy?

Thanks for your reply.
I tried with this command but not working.
Invoke-WebRequest -Uri "https://github.com/ninja-build/ninja/releases/latest/download/ninja-win.zip" -o "ninja-win.zip"
After set the proxy with this command, it works.
$Wcl = new-object System.Net.WebClient
$Wcl.Headers.Add("user-agent", "PowerShell Script")
$Wcl.Proxy.Credentials = [System.Net.CredentialCache]::DefaultNetworkCredentials
Invoke-WebRequest -Uri "https://github.com/ninja-build/ninja/releases/latest/download/ninja-win.zip" -o "ninja-win.zip"

Hi @mikeshuangyan  - I pushed up a branch https://github.com/seanmiddleditch/gha-setup-ninja/tree/proxy that (hypothetically) adds proxy support via an http_proxy input variable.
I don't have a setup right now to actually test this branch... I think I should figure out how to build a simple Docker container that requires a proxy so I can add a test to CI, but that's not something I'm likely to get to soon.
Would you be willing to check out if that branch lets you solve this problem? Further, let me know if the interface (a single http_proxy input variable) is ideal for you?


Confusion Around Refresh Token (Missing Authorization Header)

Hello! After working through #197 and getting the package updated, I'm still running into an issue regarding refresh tokens that I'm starting to think is a mismatch between what my API backend expects and what is the expected behavior of ESAT.
With my current setup, both GETs and POSTs to the API are including the Authorization header with the access token (so far, so good). However, when the refresh period rolls around, there's a POST to my /auth/refresh endpoint that contains the refresh token in the body, but with no Authorization header. Conversely, the API (flask-restplus & flask-jwt-extended) appears to want a POST with no payload, but with the refresh token in the Authorization header. Whenever I do this procedure manually with Postman, I get the correct results, however when ESAT does the POST it rejects it as missing the Authorization header.
I tried building a small application from scratch implementing auth and ran into the same issue, so it seems it's client-side.
Does this sound like the expected behavior from ESATs end? I think I may not need to rely on the API's built-in refresh verification and build my own based on ESAT's behavior if this is correct.
  ENV['ember-simple-auth-token'] = {
    serverTokenEndpoint: ENV.REST_HOST + '/auth',
    identificationField: 'username',
    passwordField: 'password',
    tokenPropertyName: 'access_token',
    refreshTokenPropertyName: 'refresh_token',
    authorizationPrefix: 'Bearer ',
    authorizationHeaderName: 'Authorization',
    headers: {},

    //JWT-specific options
    refreshAccessTokens: true,
    serverTokenRefreshEndpoint: ENV.REST_HOST + '/auth/refresh',
    tokenExpireName: 'exp',
    //Set short for testing.
    refreshLeeway: 50
  };

Request headers (Authorization missing):
POST /auth/refresh HTTP/1.1
Host: <IP_ADDRESS>:5000
Connection: keep-alive
Content-Length: 291
Accept: application/json, text/javascript
Origin: http://localhost:4200
User-Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.81 Safari/537.36
Content-Type: application/json
DNT: 1
Referer: http://localhost:4200/
Accept-Encoding: gzip, deflate
Accept-Language: en-US,en;q=0.8

Assuming this is expected behavior, I ended up writing my own handler on the API side for having the refresh token in the payload vs. header. For anyone else who may be searching this out, though, I'd be interested to know for sure.

@adam820 The expected behavior is to have the refresh token sent in the payload of the request rather than in the header. I am closing this issue for now.

@adam820 I am using flask-jwt-extended and just noticed the exact same issue. Could you share the specifics of your workaround?

Nevermind - I think I got it.  Here's my solution in case somebody else needs it:
https://gist.github.com/midget2000x/8924fae4cca98a0a70884612c5f9e6d3

@midget2000x Sorry for not getting back to you sooner; a quick look at what you did basically looks like a more elegant version of what I ended up doing. Mine wasn't done as a decorator.

@tarponjargon The link is not working anymore. Can you paste the code please?

@0xecute
You could create app/authenticators/jwt.js with the following:
import { merge } from '@ember/polyfills';
import JWT from 'ember-simple-auth-token/authenticators/jwt';

export default JWT.extend({
  refreshAccessToken(token) {
    return new Promise((resolve, reject) => {
      const headers = merge(this.headers, {
        'Authorization': `Bearer ${token}`,
      });
      this.makeRequest(this.serverTokenRefreshEndpoint, {}, headers).then(response => {
        try {
          const sessionData = this.handleAuthResponse(response.json);
          this.trigger('sessionDataUpdated', sessionData);
          return resolve(sessionData);
        } catch (error) {
          return reject(error);
        }
      }).catch(error => {
        this.handleTokenRefreshFail(error.status);
        return reject(error);
      });
    });
  },
});


Swift Package Manager support

Proposal to add Swift Package Manager support to TPKeyboardAvoiding

Cheers! 👍

Thank you! Could you tag a new release? Otherwise the package isn't visible from SPM


Lazy tableSizeChangeOutput

Motivation:
tableSizeChangeOutput is an unpooled ByteBuf which is eagerly allocated for every DefaultHttp2HeadersEncoder.
We only need it when we get a SETTINGS frame that has this header, which might never be the case depending on the application.
This result in excessive allocations when having to deal with short lived HTTP/2 connections under load.
Modification:
Only allocate tableSizeChangeOutput if needed.
Result:
Less allocations

@slandelle nice catch

@normanmaurer related question: why is HttpObjectDecoder#parserScratchBuffer unpooled?

@normanmaurer related question: why is HttpObjectDecoder#parserScratchBuffer unpooled?

I suspect the idea was that it is "long-living" but I dont know for sure... @franz1981 I think you introduced this one. Thoughts ?

@normanmaurer related question: why is HttpObjectDecoder#parserScratchBuffer unpooled?
I suspect the idea was that it is "long-living" but I dont know for sure... @franz1981 I think you introduced this one. Thoughts ?

We know the lifecycle of its owner (the handler) and it's already properly released. I think we could use a pooled one here, it would reduce allocation in the case of short lived connections.

Yep ... let me do a PR.


Scrolling screenshot

Description of the new feature / enhancement
To take screenshots of contents that are beyond the viewable area of the user.
Scenario when this would be used?
It would be use full to take screenshots of 1000 lines of codes with ease, or take screenshots of scrolling webpages.
Supporting information
No response

/dup #3388


Show nodeid with Connection

To connect to node

@rootzoll
Currently we just display the nodeID in the "Show QR Code" Modal.
WIth the node01.fulmo.org node.
The connection ID (public key + address) is:
024a8228d764091fce2ed67e1a7404f83e38ea3c7cb42030a2789e73cf3b341365@<IP_ADDRESS>:9735
Currently we just display
024a8228d764091fce2ed67e1a7404f83e38ea3c7cb42030a2789e73cf3b341365
Should it stay that way / should we display both or should we only display the full connection ID?


Mongoose Uri & inMemory Check

Lets add  the mongoose connection for those who need to pass it in because of delay in the mongoose ready state. Let make sure we add back in the readyState to alert usering of potentially needing to add in the uri. When we use the URI we should use the createConnection call .
Look into the inMemory check so that if they are over the limit we store them so that we dont have to go back to the DB

Should address #8

Im gonna put a commit together are you fine with adding this as a config  setting
mongoose:{
   uri: mongodbUri
}


Rename switch --emitDeclarationsOnly to --emitDeclarationOnly

Flag originally introduced in https://github.com/Microsoft/TypeScript/pull/20735, this change removes the extra s in the flag name (https://github.com/Microsoft/TypeScript/pull/20735#issuecomment-361296688)

Thank you


Can't unsubscribe to a meeting

Looks like a bug?
Namn:            BBS Ads (Retronet)
Skapat av:       Joacim Melin #1
Skapat tid:      20160728  12:29
Mötesnummer:     165
Sorteringsvärde: 700
MAD:             Joacim Melin #1
Mötestyp:        FidoNet möte.
Tagnamn:         rtn_bbsads
Bibliotek:       mail:rtn_bbsads
Origin:          Delta City (deltacity.se, Vallentuna, Sweden)
(Läsa) nästa text --> uttr BBS Ads (Retronet)
Du gick ur totalt 0 möten
(Läsa) nästa text --> uttr #165
Du gick ur totalt 0 möten
(Läsa) nästa text -->

As discussed on NiKom BBS. This is because these commands have support for wildcards and the "(" makes it think it's a wildcard, but it doesn't match anything.
We've decided to simply remove the wildcard support since it's based on AmigaDOS pattern matching which is a bit weird and something that almost no one knows nowadays.


Hello, I have a question

Can you tell me the difference between Individual Transformer and QuantizedTF

Sry for the late response.
IndividualTF is the standard regressive model.
Individual TF take as input a sequence of real values and output sequence of real values that represent the difference in positions between adjacent timesteps, it can only output a single possible future.
QuantizedTF is a version where we treat the problem as a classification problem (think of it as sentence generation in NLP, where the next word can be one of multiple plausible words).
We first cluster the speed vectors in the training set, then when we do the prediction instead of working directly with the differences like in the IndividualTF we input the cluster ids, and output the probability of the next movement to be in a given cluster. The prediction will be less accurate due to the model only be able to predictt the cluster centroids, but it now allow to output multiple predictions.


A11y_Azure Data Studio_Query Plan (Preview)_ColorContrast: Color contrast ratio of keyboard focus of each node and its background is less than 3:1.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:
#A11yMAS; #A11yTCS; #A11yWCAG2.1; #A11ySev2; #SQL Azure Data Studio; #SH-SQL Azure Data Studio-Win32-OCT2022; #DesktopApp; #ColorContrast; #WCAG1.4.11; #FTP; #Win11; #Closed
Environment Details:
Application Name: Azure Data Studio
Windows Version: Windows 11
Repro Steps:

Launch the "Azure data studio" application.
Connect with the provided SQL server.
TAB to "New Query" button and hit ENTER key.
Enter the SQL query and TAB to "Estimated Plan" button and hit ENTER key.
"Query Plan (Preview) screen appear.
Observe that color contrast ratio of keyboard focus of each node and its background is 1.372:1 which is less than 3:1.

Actual:
Color contrast ratio of keyboard focus of each node and its background is 1.372:1 which is less than 3:1.
Expected Result:
Color contrast ratio of keyboard focus of each node and its background should be equal to or greater than 3:1.
Attachment:

Verified the issue in latest insider, issue is fixed.
Please find below screenshot for reference.

Github Tags:#Closed;


how to set the smoothing factor on the softmax function on dssm?

hi, i wonder if it is possible to set the smoothing factor on the softmax function, according to the  Equation 6 in the dssm paper: https://posenhuang.github.io/papers/cikm2013_DSSM_fullversion.pdf
thanks!

It's not implemented and I don't think we have a plan to do it in the future.


Remove IPv6 Reverse DNS Limitation

As per https://learn.microsoft.com/en-us/azure/dns/dns-reverse-dns-hosting#ipv6 it is possible to reverse lookup IPv6

Learn Build status updates of commit 2367cd9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-network/ip-services/ipv6-overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@asudbring
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


[BUG] Register-PnPAzureADApp - Request_BadRequest","message":"A null value was found with the expected type 'Edm.String[Nullable=False]'. The expected type 'Edm.String[Nullable=False]' does not allow null values.

Reporting an Issue or Missing Feature
I try to create an new Azure appregistration with a certificate with command "Register-PnPAzureADApp". I remember this script was working before. Now it is not working anymore.
Expected behavior
I expect a new Azure appregistration with name "AzureFunctionAppSP3" with a certificate and api permission "Sites.Selected".
Actual behavior
This is the response in PowerShell:
Checking if application 'AzureFunctionAppSP3' does not exist yet...Success. Application 'AzureFunctionAppSP3' can be registered.
Register-PnPAzureADApp:
Line |
   2 |   Register-PnPAzureADApp -ApplicationName "AzureFunctionAppSP3" -Tenan …
     |   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | {"error":{"code":"Request_BadRequest","message":"A null value was found with the expected type 'Edm.String[Nullable=False]'. The expected type 'Edm.String[Nullable=False]' does not allow null values.","innerError":{"date":"2023-11-20T12:18:12","request-id":"53467660-7217-4a3e-8305-3c4e3ef86baf","client-request-id":"53467660-7217-4a3e-8305-3c4e3ef86baf"}}}

Steps to reproduce behavior
See script below to reproduce:
 $password = Read-Host -Prompt "Enter certificate password" -AsSecureString
 Register-PnPAzureADApp -ApplicationName "AzureFunctionAppSP3" -Tenant myCompany.onmicrosoft.com -CertificatePassword $password -SharePointApplicationPermissions "Sites.Selected" -Interactive

What is the version of the Cmdlet module you are running?
2.2.0
Which operating system/environment are you running PnP PowerShell on?

[X ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

hello @khha , thanks for reporting this.
It has been already fixed in the latest nightly builds , please uninstall the 2.2.0 and install the nightly build. It will be available in 2.3.0 as well when we release it , currently no ETA
To download latest nightly build, you can check this link :
https://pnp.github.io/powershell/articles/installation.html#nightly-build
Closing it as duplicate of https://github.com/pnp/powershell/issues/3316

Thanks @gautamdsheth , I just have another issue when installing the nightly build.


from uniswap import Uniswap ERROR

from uniswap import Uniswap
ImportError: cannot import name 'Uniswap'
This is very strange

This is not enough detail to be a useful bug report. We need considerably more detail (how did you install, how can we reproduce, etc).

Have same Problem
fresh install Ubuntu 18 64bit
apt-get update
apt install python3-pip
alias pip=pip3
pip install uniswap-python
root@ubuntu:~# python3
Python 3.6.9 (default, Jan 26 2021, 15:33:00)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

from uniswap import Uniswap
Traceback (most recent call last):
File "", line 1, in
ImportError: cannot import name 'Uniswap'

what can i do please help


Add tests for OP_CHECKSIG

close https://github.com/yenom/BitcoinCashKit/issues/280

Codecov Report

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

@@            Coverage Diff             @@
##           master     #345      +/-   ##
==========================================
+ Coverage   44.31%   44.36%   +0.04%
==========================================
  Files         149      149
  Lines        4206     4206
==========================================
+ Hits         1864     1866       +2
+ Misses       2342     2340       -2

Impacted Files
Coverage Δ

BitcoinCashKit/Core/Crypto.swift
91.93% <0%> (+1.61%)
:arrow_up:

...hKit/Core/Scripts/OP_CODE/Crypto/OP_CHECKSIG.swift
100% <0%> (+8.33%)
: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 b250466...dab092b. Read the comment docs.


bug: Xlrelease_xld_server tries to authenticate with a wrong parameter

From @ltutar on May 22, 2015 13:9
I have the following yaml.
xlrelease::xlr_version: '4.6.0'
xlrelease::xlr_licsource: 'puppet:///modules/xlrelease/license/xl-release-license.lic'
xlrelease::install_type: 'puppetfiles'
xlrelease::puppetfiles_xlrelease_source: 'puppet:///modules/xlrelease/sources'
xlrelease::xlr_admin_password: 'xebia1'
xlrelease::xlr_rest_user: 'levent'
xlrelease::xlr_rest_password: 'tuttifrutti'
xlrelease::xlr_xldeploy_hash:
'default':
properties:
url: "http://<IP_ADDRESS>:4516/deployit/"
username: "admin"
password: "xebia1"
When I run 'puppet agent -t', I get the following error. It looks like the parameter xlrelease::xlr_rest_user: 'levent' is being used to authenticate with XL Deploy.
rror: /Stage[main]/Xlrelease::Postconfig/Xlrelease_xld_server[default]: Could not evaluate: 746: unexpected token at 'cannot send request to deployit server 401/Cannot authenticate levent:

Error 401 Cannot authenticate levent

HTTP ERROR 401
Problem accessing /repository/tree/Configuration/Deployit/default. Reason:
    Cannot authenticate leventPowered by Jetty://

'
Notice: /Stage[main]/Xlrelease/Anchor[xlrelease::end]: Dependency Xlrelease_xld_server[default] has failures: true
Warning: /Stage[main]/Xlrelease/Anchor[xlrelease::end]: Skipping because of failed dependencies
Notice: Finished catalog run in 48.94 seconds
[root@puppetagent7 ~]#
Copied from original issue: WianVos/puppet-xlrelease#7

From @ltutar on June 1, 2015 18:55
I looked into this.
The following needs to be correct in yaml:
xlrelease::xlr_rest_user: 'admin'
xlrelease::xlr_rest_password: 'xebia1'
I will close the issue.


Live server only shows list of files

I'm using Chrome on Windows 10. In my project I don't have *.html files so I followed points 4 and 5 from your readme.
Unfortunately even hen I select command Open with Live Server all I'm getting on localhost is the list of files and if I click on index.php it tries to download it.

I get the same with VS Code installed from snap on Ubuntu 20.04 and Firefox.

I get the same thing. If I try to navigate to a specific page within the working directory, the browser downloads the file. Not sure what I am missing here.

Same here. Would love a solution or work-around for this issue!


Missing preProvisioningHook in UpdateProvisioningTemplate API for CloudFormation

Hello there,
The API UpdateProvisioningTemplate has a parameter called preProvisioningHook which is missing from the update function in this repo:
https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-iot/blob/master/aws-iot-provisioningtemplate/src/main/java/com/amazonaws/iot/provisioningtemplate/UpdateHandler.java#L101
For reference, you can compare it to the create function which does have the preProvisioningHook:
https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-iot/blob/master/aws-iot-provisioningtemplate/src/main/java/com/amazonaws/iot/provisioningtemplate/CreateHandler.java#L107
Can you please add the preProvisioningHook to the update?
Thanks!

Fixed in https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-iot/commit/be69bfe93bbcf0248ed6f49232b5b895578e16a1


what should I set ldim to?

(cross posting from https://github.com/nenadmarkus/picojs/issues/33, since this has more to do with the underlying implementation, and less to do with the javascript framework)
Hi, I want to use pico to find faces on images of arbitrary width/height. I am hoping there is some sort of general rule I can uses for setting the ldim parameter based on image width & height. I have read over the explanation doc https://nenadmarkus.com/p/picojs-intro/

The parameter ldim tells us how to move from one row of the image to the next (known as stride in some other libraries, such as OpenCV).

From what I gather from the examples, it is usually set to the ncols parameter (a.k.a image width). However, I have an example image that is 400x400 pixels. Setting these params:
// where image.width === 400 and image.height === 400
  const image = {
    pixels: rgba_to_grayscale(image_data, image_data.height, image_data.width),
    nrows: image_data.height,
    ncols: image_data.width,
    ldim: image_data.width
  }

This results in zero detections. Setting ldim to the seemingly arbitrary value of 419 results in one detection (which is the desired result). Setting ldim to anything higher results in several detections all correspoding to the same face.
All the other parameters have values taken from examples/image.html
ldim: 400 (the image width):

ldim: 419 (the desired result):

ldim: 420:

ldim: 480:

full code for reference:
import pico from './pico.js'
import { decode } from<EMAIL_ADDRESS>
const facefinder_bytes = await Deno.readFile('./examples/facefinder')
const facefinder_classify_region = pico.unpack_cascade(facefinder_bytes)

/**
 * a function to transform an RGBA image to grayscale
 */
function rgba_to_grayscale(rgba, nrows, ncols) {
  var gray = new Uint8Array(nrows * ncols)
  for (var r = 0; r < nrows; ++r)
    for (var c = 0; c < ncols; ++c)
      // gray = 0.2*red + 0.7*green + 0.1*blue
      gray[r * ncols + c] =
        (2 * rgba[r * 4 * ncols + 4 * c + 0] +
          7 * rgba[r * 4 * ncols + 4 * c + 1] +
          1 * rgba[r * 4 * ncols + 4 * c + 2]) /
        10
  return gray
}

async function find_face(image_filepath: string, stride?: number) {
  const raw = await Deno.readFile(image_filepath)
  const image_data = decode(raw)
  // const image_data = image_data_flat.reduce((acc, ))
  // console.log(image_data)
  const grey_image_data = rgba_to_grayscale(image_data, image_data.height, image_data.width)
  // console.log(image_data.height, image_data.width)
  const image = {
    pixels: rgba_to_grayscale(image_data, image_data.height, image_data.width),
    nrows: image_data.height,
    ncols: image_data.width,
    // ldim: image_data.width // ? TODO
    ldim: stride ?? image_data.width
  }
  const params = {
    shiftfactor: 0.1, // move the detection window by 10% of its size
    minsize: 20, // minimum size of a face (not suitable for real-time detection, set it to 100 in that case)
    maxsize: 1000, // maximum size of a face
    scalefactor: 1.1 // for multiscale processing: resize the detection window by 10% when moving to the higher scale
  }

  // run the cascade over the image
  // detections is an array that contains (r, c, s, q) quadruplets
  // (representing row, column, scale and detection score)
  let detections = pico.run_cascade(image, facefinder_classify_region, params)
  // cluster the obtained detections
  detections = pico.cluster_detections(detections, 0.2) // set IoU threshold to 0.2
  // draw results
  const qthresh = 5.0 // this constant is empirical: other cascades might require a different one

  // this just draws the rectangles to help visualize. It is not relevant to the face tracking
  const rectangles = detections.map(([x, y, w, h]) => `rectangle ${x},${y} ${w},${h}`).join(' ')
  const proc = Deno.run({
    cmd: [
      'convert',
      image_filepath,
      '-fill',
      'none',
      '-stroke',
      'red',
      '-draw',
      rectangles,
      'preview.jpg'
    ]
  })
  const result = await proc.status()
  if (!result.success) Deno.exit(1)
  console.log(image_filepath, `(${image_data.width}x${image_data.height})`, 'found', detections.length, 'faces with ldim', image.ldim)
  return detections
}

// 419 feels extremely arbitrary
await find_face('./samples/6627147.jpeg', 400)
await find_face('./samples/6627147.jpeg', 419)
await find_face('./samples/6627147.jpeg', 420)
await find_face('./samples/6627147.jpeg', 480)

full code for reference:
import pico from './pico.js'
import { decode } from<EMAIL_ADDRESS>
const facefinder_bytes = await Deno.readFile('./examples/facefinder')
const facefinder_classify_region = pico.unpack_cascade(facefinder_bytes)

/**
 * a function to transform an RGBA image to grayscale
 */
function rgba_to_grayscale(rgba, nrows, ncols) {
  var gray = new Uint8Array(nrows * ncols)
  for (var r = 0; r < nrows; ++r)
    for (var c = 0; c < ncols; ++c)
      // gray = 0.2*red + 0.7*green + 0.1*blue
      gray[r * ncols + c] =
        (2 * rgba[r * 4 * ncols + 4 * c + 0] +
          7 * rgba[r * 4 * ncols + 4 * c + 1] +
          1 * rgba[r * 4 * ncols + 4 * c + 2]) /
        10
  return gray
}

async function find_face(image_filepath: string, stride?: number) {
  const raw = await Deno.readFile(image_filepath)
  const image_data = decode(raw)
  // const image_data = image_data_flat.reduce((acc, ))
  // console.log(image_data)
  const grey_image_data = rgba_to_grayscale(image_data, image_data.height, image_data.width)
  // console.log(image_data.height, image_data.width)
  const image = {
    pixels: rgba_to_grayscale(image_data, image_data.height, image_data.width),
    nrows: image_data.height,
    ncols: image_data.width,
    // ldim: image_data.width // ? TODO
    ldim: stride ?? image_data.width
  }
  const params = {
    shiftfactor: 0.1, // move the detection window by 10% of its size
    minsize: 20, // minimum size of a face (not suitable for real-time detection, set it to 100 in that case)
    maxsize: 1000, // maximum size of a face
    scalefactor: 1.1 // for multiscale processing: resize the detection window by 10% when moving to the higher scale
  }

  // run the cascade over the image
  // detections is an array that contains (r, c, s, q) quadruplets
  // (representing row, column, scale and detection score)
  let detections = pico.run_cascade(image, facefinder_classify_region, params)
  // cluster the obtained detections
  detections = pico.cluster_detections(detections, 0.2) // set IoU threshold to 0.2
  // draw results
  const qthresh = 5.0 // this constant is empirical: other cascades might require a different one

  // this just draws the rectangles to help visualize. It is not relevant to the face tracking
  const rectangles = detections.map(([x, y, w, h]) => `rectangle ${x},${y} ${w},${h}`).join(' ')
  const proc = Deno.run({
    cmd: [
      'convert',
      image_filepath,
      '-fill',
      'none',
      '-stroke',
      'red',
      '-draw',
      rectangles,
      'preview.jpg'
    ]
  })
  const result = await proc.status()
  if (!result.success) Deno.exit(1)
  console.log(image_filepath, `(${image_data.width}x${image_data.height})`, 'found', detections.length, 'faces with ldim', image.ldim)
  return detections
}

// 419 feels extremely arbitrary
await find_face('./samples/6627147.jpeg', 400)
await find_face('./samples/6627147.jpeg', 419)
await find_face('./samples/6627147.jpeg', 420)
await find_face('./samples/6627147.jpeg', 480)

The pixel intensity values of the image are accessed as pixels[r*ldim + c] and in your case ldim should be set to image_data.width.
In some cases, pico just fails to detect the face. This is the case you have.
Understand that pico should mainly be used with a video stream where detections get "averaged" over multiple frames and this leads to significantly better detection capabilities.

The pixel intensity values of the image are accessed as pixels[r*ldim + c] and in your case ldim should be set to image_data.width.
In some cases, pico just fails to detect the face. This is the case you have.
Understand that pico should mainly be used with a video stream where detections get "averaged" over multiple frames and this leads to significantly better detection capabilities.

ah. Thanks for the explanation. Is there a chance fiddling with any of the other defaults would help?
In the final product I will in fact be using this with frames extracted from a video to zoom-pan to the face so hopefully this will be less of an issue

ah. Thanks for the explanation. Is there a chance fiddling with any of the other defaults would help?
In the final product I will in fact be using this with frames extracted from a video to zoom-pan to the face so hopefully this will be less of an issue

You can try reducing shiftfactor and/or scalefactor. This should improve the detection rate at the cost of speed, i.e., there's a trade-off.

You can try reducing shiftfactor and/or scalefactor. This should improve the detection rate at the cost of speed, i.e., there's a trade-off.


Moving 'CreateFromInstanceType' dir under catalog

Signed-off-by: Aviv Turgeman<EMAIL_ADDRESS>📝 Description

Moving 'CreateFromInstanceType' dir under catalog directory instead of being under CreateVMHorizontalNav/components

cc:// @pcbailey

/retest

/lgtm

/lgtm


M3-6041: Add Awareness of DX tools (API-CLI ) - Create Linode flow

Description 📝
Introduces API CLI Awareness Feature to cloud manager
What does this PR do?
Enables users copy  and execute dynamic cURL and CLI commands in create Linode flow.
Preview 📷
https://user-images.githubusercontent.com/119517080/214948403-614f3fe7-55c2-4ddc-b5e9-1e91e7c69e52.mov
How to test 🧪

Click Create Linode
Click Create Using Command Line button
Should be able to inline error messages
Fill all the required fields to create Linode
Now, click on Create Using Command Line button
Should open API CLI Modal
Validate the copy and Modal as per the design
Copy the command (cURL / CLI )
Open terminal and paste
Update the command with token
Execute and observe the Linode's page should appear created Linode

How do I run relevant unit or e2e tests?

yarn test
yarn cy:run

One thing i'm noticing is that the generated CLI command does not like the tag okay!\test'okay. cURL looks good.

One thing i'm noticing is that the generated CLI command does not like the tag okay!\test'okay. cURL looks good.

I verified the changes you merged and both tag and root password look for both cURL and CLI command.

Tested with some complex strings API is working as expected and able to login into VM as well.


Enclosed the alert key with aps in apns push notification payload

Fix for ios push notifications

Automated message from Dropbox CLA bot
@kunall17, it looks like you've already signed the Dropbox CLA. Thanks!

Closing this as PyAPNs library is doing the job!


chmod +x on entrypoint.sh missing. Latest image version broken

Hi sorry forgott to write something. Cant start the container anymore because of the missing +x flag. Thanks for your help


About me page redesign

About me page redesign. Here are screenshots:
closes #206
Desktop

Mobile

mobile version screenshot looks cut out from the right side, I don't know why, Firefox is weird

I've made contributors in random order to make so that the position in contributors.json does not impact the position on the page.

Looks beautiful!

this is beautifull! 🤩

Looks awesome

These phpstan errors will be fixed if you merge the last suggestion.

Thank you @Plytas for the feedback, great suggestions

I don't know how I feel about the GitHub bot fixing styles.. It pushes changes to my working branch and it conflicts with my local work. Am I the only one who doesn't like it?

Definitely get in the habit of running composer qa or just composer pint before committing. We could even consider adding pint as pre-commit hook. Apart from that without the bot there would be no way to maintain consistent styling, because someone will forget to run it locally.

there would be no way to maintain consistent styling

I think there is a way to tell Pint so that instead of fixing styles to just error out. It will fail the checks on PR. Anyway, in which PHP version we got this syntax with 3 dots?:
Cache::remember('contributors', self::DAY, $this->getContributors(...))

I somehow missed it

in which PHP version we got this syntax with 3 dots?

I commented this on the suggestion, but it might have gotten lost. We have it since PHP 8.1 https://stitcher.io/blog/new-in-php-81#first-class-callable-syntax-rfc

I think there is a way to tell Pint so that instead of fixing styles to just error out. It will fail the checks on PR.

You are right about that, we could call the action with --test flag and then it would only error out. I feel like that would be more work for @brendt in case original PR author disappears and doesn't push pint fixes. He would have to checkout the PR, run pint and push it himself.

It may also be worth extracting getContributers to another class, perhaps FetchContributers in the Support directory. We currently cannot mock the list of contributors, nor retrieve it for output elsewhere.

You are right about that, we could call the action with --test flag and then it would only error out. I feel like that would be more work for @brendt in case original PR author disappears and doesn't push pint fixes. He would have to checkout the PR, run pint and push it himself.

I'm ok with that, I agree that dealing with style commits can be a hassle. Made an issue for it #239, I'll probably fix it tomorrow.
On the other hand: always best to run composer ca before committing.

It may also be worth extracting getContributers to another class, perhaps FetchContributers in the Support directory. We currently cannot mock the list of contributors, nor retrieve it for output elsewhere.

@joshbonnick good idea, could you make an issue for it so that we don't forget?
@SerhiiCho the page looks 🔥 can I merge this, or do you need to make changes still?

You can merge it

It may also be worth extracting getContributers to another class

Great idea, but in a separate PR

It's live :) I did notice Nikita's avatar not showing up, probably an uncomitted file?

artisan storage:link maybe?

It's live :) I did notice Nikita's avatar not showing up, probably an uncommitted file?

It's in .gitignore

I'm pretty sure it's just storage link, as the file is definitely committed, but asset() is looking in public dir.

I'm also thinking that you just need to run php artisan storage:link

Ah, the deploy script we're using clears everything from /public/public, and symlinks it to the storage folder on the server for persistent data. In other words: you cannot use that folder 😁 I've simply moved it to /public for now, and it works just fine :)


[Bug]: link broken in converting-data-to-nexus tutorial

What happened?
This link to download the examples file is broken because the token rate limit has been exceeded:
https://github.com/FAIRmat-NFDI/pynxtools/blob/6b3b3d5c372b3d0a062bdf32097929211e6c2e86/docs/tutorial/converting-data-to-nexus.md?plain=1#L30
The actual link at the moment is: https://download-directory.github.io/?url=https%3A%2F%2Fgithub.com%2FFAIRmat-NFDI%2Fpynxtools%2Ftree%2Fdocumentation%2Fexamples%2Fxps
Relevant log output
⚠ Your token rate limit has been exceeded.
Retrieving directory info
Repo: FAIRmat-NFDI/pynxtools
Directory: /examples/xps
Loading

@sherjeelshabih any ideas?

I had a quick look. The link just runs code in your browser and for our link accesses the GitHub API unauthenticated. That means whoever clicks that link can make 60 requests to the API from their IP address.


fix issue 17337

fix #17337
It seems like there is a bug with hugo when yaml is below list. The yaml will be considered as part of the last item.
For example:
The example below demonstrates the components of a StatefulSet.

A Headless Service, named nginx, is used to control the network domain.
The StatefulSet, named web, has a Spec that indicates that 3 replicas of the nginx container will be launched in unique Pods.
The volumeClaimTemplates will provide stable storage using PersistentVolumes provisioned by a PersistentVolume Provisioner.

yaml
...

I think I have seen this before. @jimangel @zacharysarah is there a best practices fix for this?
/assign @jimangel

Huh, that is weird. The empty line between the list and the yaml block should remove any indentation.
Adding empty headings is a kludge, though, and would be rough to clean up later. Is there an issue/fix on https://gohugo.io that covers this?

find some reference:
https://github.com/gohugoio/hugo/issues/4785
https://github.com/istio/istio.io/pull/1501/files
And i try to indent code by four, like below:
    apiVersion: v1
    kind: Service
    metadata:
      name: nginx
      labels:
        app: nginx
    spec:
      ports:
      - port: 80
        name: web
      clusterIP: None
      selector:
        app: nginx
    ---
    apiVersion: apps/v1
    kind: StatefulSet
    metadata:
      name: web
    spec:
      selector:
        matchLabels:
          app: nginx # has to match .spec.template.metadata.labels
      serviceName: "nginx"
      replicas: 3 # by default is 1
      template:
        metadata:
          labels:
            app: nginx # has to match .spec.selector.matchLabels
        spec:
          terminationGracePeriodSeconds: 10
          containers:
          - name: nginx
            image: k8s.gcr.io/nginx-slim:0.8
            ports:
            - containerPort: 80
              name: web
            volumeMounts:
            - name: www
              mountPath: /usr/share/nginx/html
      volumeClaimTemplates:
      - metadata:
          name: www
        spec:
          accessModes: [ "ReadWriteOnce" ]
          storageClassName: "my-storage-class"
          resources:
            requests:
              storage: 1Gi

It formats ok, but seems like part of the last item.
I think this is the correct location and format, like below:

With indent  four, it shows like this:

A best suggestion is needed, thanks.

Page preview:
https://deploy-preview-17340--kubernetes-io-master-staging.netlify.com/docs/concepts/workloads/controllers/statefulset/

This revised PR looks like a proper fix at the moment although I would prefer extracting the YAML into the examples directory.
Leaving approval to other reviewers for a double check.
/lgtm

@tengqm Thanks for the review!
@lojies Thanks for submitting this PR. Sorry for the relative silence--the chairs been discussing how best to proceed in Slack, which isn't easily visible here.
I agree with @kbhawkey and @jimangel: while the current rendering is indeed broken thanks to Black Friday, it's better to leave the rendering broken as is than to "solve" it by breaking it in a different way. In that spirit, I think it's best to close this PR and wait for a Hugo-based fix.
/close


[#136783629] Enable SSL

What
Story: How do encrypt traffic between the elb and gorouter using haproxy
Add an SSL port to HAProxy. The traffic is routed internally to the usual proxy protocol front-end. This is required because the PROXY header is encrypted inside the SSL stream so we need to decode it first.
How to review
Review alongside relevant PR on paas-cf
Who can review
Not @keymon or me

LGTM. Merging.


Need something more robust than polymorphic this typing for RxJS

I was pretty excited about this typing for RxJS, since it does have a sort of "fluent" style interface. However we have some issues with it that are discussed here: https://github.com/ReactiveX/RxJS/issues/846
Basically, we have operators that return same-shaped, but different generic types:
map for example: Observable<T>.prototype.map<R>((value: T) => R): Observable<R>
The reason we were looking at this typing the above is that map doesn't always return an Observable, it could be called on a Subject which is a subclass of Observable. In that case, it actually returns a Subject, not an Observable, because of our fancy lift mechanism:
The Observable's lift:
https://github.com/ReactiveX/RxJS/blob/2896556afe8b663d4f983da1830fe407841e7d5b/src/Observable.ts#L51-L63
The Subject's lift:
https://github.com/ReactiveX/RxJS/blob/2896556afe8b663d4f983da1830fe407841e7d5b/src/Subject.ts#L37-L41
How lift is used (in map for example):
https://github.com/ReactiveX/RxJS/blob/2896556afe8b663d4f983da1830fe407841e7d5b/src/operators/map.ts#L17
It would be fantastic if we could provide our TypeScript users with the proper type information in their IDEs. Perhaps some additional syntax like this<R> would fit the bill? It's also worth noting that operators always return a new instance, rather than the same this instance, if that makes a difference.

I think this is another instance of the higher order generic referenced in:  https://github.com/Microsoft/TypeScript/issues/1213 combined with this types. i think you want to say something like:
map<T<*> extends Observable<*>, R, U>(this: T<U>, fn: (value: U) => R): T<R>

@mhegazy that looks something like it yeah. My C# is rusty, but I think in C# it was something like map<T, R, U>(this: T<U>, fn: (value: U) => R) where T<U> : Observable<U> or the like.

I'm pretty sure C# doesn't support higher-kinded types either, so it'll fail at T<U>.

No - C# does not yet have higher-kinded typeshttps://github.com/dotnet/roslyn/issues/2212).

@weswigham bummer. Not that I think it would be too useful in C# in most cases. But since TypeScript has JavaScript as a build target, I think it would be useful here. Well, clearly, because we've hit the need.

@mhegazy, @blesh
for your example you do not need higher kinded types, f-bounded polymorphism is sufficient
your example can be expressed in C# as
static T map<T, R, U>(this T @this, Func<U, R> fn) where T : Observable<U> { ... }
which is an extension method
and in typescript (since 1.8 I think) as
map<T extends Observable<U>, R, U>(that: T, fn: (value: U) => R): T { ... }
which is unfortunately an ordinary function


Agar.io got freaking updated

Dear Maker of this Super Cool Hack. Hoperfully you read this fast. So i think that Agar.io just got updated and your Super Cool Hack doesn't work anymore (Image: : http://puu.sh/i7bIH/3e023f182a.png). Read this Story (if you want):  I was searching for a Hack/Bot for Agar.io then i found a Video (not this Video). and it actually worked. i posted it on another video, where you "must" like comment subscribe or something and then he sends you the hack, yeah so i wanted that the Viewers dont get ripped off. The Next time i was on agar.io, well this happend: http://puu.sh/i7bIH/3e023f182a.png.
Greetings from Switzerland (i was from Switzerland all along m8)

yeah, this is happening witch me too

Same

Update to the latest version.

Thx :)


Cannot use this action in a chain

Example:
  test:
    uses: ...
    if: "! inputs.skip-tests"

  test-status:
    needs: [ test ]
    runs-on: ubuntu-latest
    steps:
      - uses: re-actors/alls-green@release/v1
        with:
          allowed-skips: test
          jobs: ${{ toJSON(needs) }}
    if: always()

  build:
    needs: [ test-status ]
    uses: ...

The latter job is not run, probably because of the needs of the former

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

That works in our own CI. Perhaps, yours is misconfigured somehow 🤷♂.
Anyway, it's not something related to the action (as it only works with the inputs passed in), but is happening on the workflow definition level, which we wouldn't be able to control, even if we wanted to.
Closing as it's not a bug for us, but something on the calling side.

If you want to run a job despite test-status failing, you'll have to stick if: always() there too. Although I don't know what use case this would actually support.

I should clarify the example given. The issue is when test job is skipped, test-status of course runs, but build is skipped as if the skip status propagated through. I'll make a mwe tomorrow.
Indeed this is a usage issue, but it is not intuitive how it should be configured, so some documentation would be helpful for such scenario.

@webknjaz here is a mwe if you have any pointers about it: https://github.com/LecrisUT/all-greens-example


sdgfsdgfsdgdf

Pre-submit checklist:

Branch

[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Relevant tickets are mentioned in commit messages
[ ] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reviewer requested

Closing because noise.

Sorry!
On Sun, Aug 1, 2021 at 12:16 PM effectfully @.***>
wrote:

Closing because noise.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/input-output-hk/plutus/pull/3658#issuecomment-890571906,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSGT2OOFFGZRZ73MIN6CUTT2WMQDANCNFSM5BKNXACA
.


[stdlib][gardening] Silence a warning about unused value

It's only one of two warnings at the moment popping up when I compile the stdlib. Also easily addressed, so it's a perfect time to do some Saturday morning gardening.

@swift-ci smoke test and merge


fix: test semantic-release

What

Tests new tokens used in semantic-release

:tada: This PR is included in version 0.2.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.2.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


check if shasum binary exists

in some distributions like alpine the binary shasum is not present.

Thanks!


VM detail page issues

Virtual Machine Detail

[x] 1. There is no label to remove, the remove button is not necessary, the close and the save button are in Chinese, click the save button, the popup does not close

[ ] 2. Add a label and then remove it, but the removed label will be recovered when clicking the save button, which means it has not been deleted

[x] 3. Unable to add more than one label, add a new one and it will be erased automatically

[x] 4. The IP Address copy tooltip's background is transparent that makes the texts overlap

[x]  1. There is no label to remove, the remove button is not necessary, the close and the save button are in Chinese, click the save button, the popup does not close
[x]  2. Add a label and then remove it, but the removed label will be recovered when clicking the save button, which means it has not been deleted
[x]  3. Unable to add more than one label, add a new one and it will be erased automatically
[ ]  4. The IP Address copy tooltip's background is transparent that makes the texts overlap

@DaiYuzeng for the second issue, the user is still not able to remove the last label or annotations since u didn't send an empty array of string to the API.

@DaiYuzeng for the second issue, the user is still not able to remove the last label or annotation since u didn't send an empty array of string to the API.

Oh! Maybe i forgot to reset the bound object value,  I will fix it tomorrow! Thanks for your advice!

Validated


Reseting reverts to wrong info

Current behavior
When clicking on revert on a style, the ordered qty is not reverted to what was uploaded. Rather it seems like its changed to what the first item has.
Expected behavior
It should change back to what was initially uploaded
Steps to reproduce the issue
Upload PO, adjust qty with buffer and click reset.
Can you handle fixing this bug by yourself?

[ ] YES
[x ] NO

Verified the issue.
Instead of just quantity, all the fields such as product name, arrival date, etc are changed to what the first item has.


Allow one application to subscribe multiple times on same topic?

With the way the current publish/subscribe API is set up, it's not possible to have one application subscribing multiple times on the same topic.  I have the following example in mind:
My application hosts an "Order API".  Each time an order is created through the API, the order gets published on the "ordertopic".  For each incoming order, two parallel actions have to be executed.  In this case, I would like to subscribe twice on the "ordertopic", to handle each of the actions individually.  This is currently not possible.
As a workaround, I can of course create separate applications for that, but this would over-complicate the management, monitoring and operational side.  Is there an option to allow for example multiple "channels" (or whatever naming) within an application that subscribe on messages?
  GET http://localhost:<appPort>/dapr/subscribe

  [
      { "topic" : "ordertopic", "channel" : "action1" },
      { "topic" : "ordertopic", "channel" : "action2" }
  ]

Message delivery could be performed in this scenario on this URI:
  POST http://localhost:<appPort>/<topic>/<channel>

Interested to hear your view on this!
Thanks
Toon

Hello! This issue has not been resolved but I agree it makes sense to allow an application to subscribe multiple times to a single topic.
For example, a DeliveryCompleted event should be handled by the Notifications service that must do multiple notifications in parallel (as SMS, Push and Email). If one of these notifications fails, we do not want to retry all of them, as the user has been notified already.
The only options we have right now (at least the only ones I can think about) are:

to do all notifications in parallel within the same DeliveryCompleted event handler, adding the necessary code for handling transient errors, retries and all the boilerplate already solved with dapr;
to go with multiple services from little to no value, to allow us subscribe multiple times to the same event;
to create some Saga that coordinates this work;

In my opinion, the simplest solution is to add a subscription name to the handler, something like:
@app.subscribe(pubsub_name='pubsub', topic='delivery-completed', subscription='sms-handler')
    def sendSms(event: v1.Event) -> None:
        data = json.loads(event.Data())
        print(f'Subscriber #1 received: id={data["id"]}, message="{data["message"]}", content_type="{event.content_type}"',flush=True)

@app.subscribe(pubsub_name='pubsub', topic='delivery-completed', subscription='push-handler')
    def sendPush(event: v1.Event) -> None:
        data = json.loads(event.Data())
        print(f'Subscriber #2 received: id={data["id"]}, message="{data["message"]}", content_type="{event.content_type}"',flush=True)

Can we reopen this issue and have some discussion about it?

@marcmaranon we have a design in place that will allow that, please see here: https://github.com/dapr/dapr/issues/2582#issuecomment-883649283


Getting exception when clicking on view all under review section at product page.

Bug report
Title
Getting exception when clicking on view all in review grid at product page.
Preconditions
1. framework Version.0.2..2

Steps to reproduce
1. step1: give review to any product approve from admin
2. step2:open product and click on view all under review section

Expected result
review page should open
Actual result

framework Version.0.2.2
Fixed


Improper handling of install name for shared library on OS X

I am working on a project that has libhdfs3 as a submodule in our git repo. Since we want to keep the build process contained in a single (user-owned) directory tree, we configure with cmake -DCMAKE_INSTALL_PREFIX:PATH=$(pwd)/usr. However, after running make && make install, I then find the following incorrect behavior when I run otool.
[thomas@Mithlond] libhdfs3-cmake $ otool -D usr/lib/libhdfs3.dylib
usr/lib/libhdfs3.dylib:
libhdfs3.1.dylib

Note that since the install name is incorrectly set, linking against this copy of the library, even by absolute path, will produce a binary that can't find libhdfs3.dylib without manually altering LD_LIBRARY_PATH.

@elfprince13  - Apologies for the very delayed response. The development team managing this codebase is also responsible for Apache HAWQ (incubating) and trying to get the first release out the door. As a result, this repo has been unintentionally neglected and recently turned read-only.
This code was birthed from HAWQ and will be merged back into into that project (now that it is open source as well); mailing lists, pull requests, issues and releases related to libhdfs3 will be there. The plan is to provide tarball releases of this code separate from the larger HAWQ project until it can be relocated into a dedicated repo again (down the road) - see HAWQ-1059.
Again, thank you for the contribution and patience while we sort out these logistics.
If this issue/feature is still active, please respond otherwise it will be closed.

Yep, still active. My current work-around is adding
if [[ "$OS_UNAME" == "Darwin" ]]; then
	install_name_tool -id `pwd`/usr/lib/libhdfs3.1.dylib `pwd`/usr/lib/libhdfs3.1.dylib
fi

into a bash script that runs as a wrapper around the build task, but I'd appreciate having the official CMake files configured correctly.

Issue tracking moved here to HAWQ-1066.
Vote to close the issue here.

Sounds good, thanks!


revert dual screen

Revert #1029
Description of the changes:

Temporary revert the dual screen experiment (will be added back with new patches in a couple of days)

How changes were validated:

Manually

I can't seem to find where in the code the issue is, however the Variables edit panel does not show up anymore.


bug: meteor "new recipe" command is not using latest recipe requirement

Describe the bug
Current meteor new command does not use the latest recipe requirements.
generated-recipe.yaml
name: sample
version: v1beta1
source:
  type: kafka
  config:
    broker: "my-broker:6668"
sinks:
  - name: http
    config:
      # The url (hostname and route) of the http service
      url: http://localhost:3333
      method: POST
      # Additional HTTP headers, multiple headers value are separated by a comma
      headers:
        X-Other-Header: value1, value2

running this get below error
ERRO[0000] error running recipe                          duration_ms=0 err="failed to setup extractor: could not initiate extractor \"kafka\": urn scope is required to generate unique urn" recipe=sample records_count=0
ERRO[0000] failed to setup extractor: could not initiate extractor "kafka": urn scope is required to generate unique urn

The error is because since version 0.5.0, meteor introduce new required source.scope field on recipe.
This field is needed to make sure asset's urn is unique by using scope as some sort of namespacing feature.
To Reproduce
Steps to reproduce the behavior:

Run meteor new recipe sample -e kafka -s http > recipe.yaml
Copy printed out recipe to a file (recipe.yaml)
Run meteor run recipe.yaml
See error

Expected behavior
Update new command
meteor new recipe should include scope on the generated recipe. It could also make scope as part of required input when running the command. e.g. --scope=my-scope.
Docs updated
Somewhere in the docs, it would be helpful if the new scope field is being mentioned so that users are not stuck.

Related PR - https://github.com/odpf/meteor/pull/406.


Repository not public

mazda2mqtt Warning pull access denied for mazda2mqtt, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

You need to build locally first, using the comment docker compose build

i really need to read more carefully the docs :smiling_face_with_tear:
sorry!

i really need to read more carefully the docs :smiling_face_with_tear:
sorry!


Autorest Regen Preview to 3.0.0-alpha.20230330.1 by Dapeng Zhang from refs/pull/3207/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3207

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.


nightly release

no rush on this, would be cool to have a "nightly" channel where if someone installs nightly they get nightly builds but if they install "stable" they only get stable tags

After that extra fix #901, this Nightly channel works! :night_with_stars:
Can grab latest builds from https://dl.kittycad.io/releases/modeling-app/nightly/last_download.json


Feat: Support Trivy status filtering

What this PR does / why we need it:
Adds ability to filter by image vulnerability status through a new field ignoredStatuses in components.scanner.config.
https://aquasecurity.github.io/trivy/v0.44/docs/configuration/filtering/#by-status
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #821
Special notes for your reviewer:
Still needs to be (manually) end-to-end tested. Have only "verified" it works based on logic and the small test for the config, but have not done any manual verification.
Manual Verification Steps
Install Eraser with updated scanner

kind create cluster
make docker-build-trivy-scanner TRIVY_SCANNER_IMG=eraser-trivy-scanner:dev
kind load docker-image eraser-trivy-scanner:dev
helm upgrade --install eraser charts/eraser --values override.yaml (seen below)

Test Eraser with vulnerable image

docker pull docker.io/library/alpine:3.7.3 (pulling vulnerable image)
docker exec -ti kind-control-plane bash
crictl images (in the bash)
kind load docker-image alpine:3.7.3

Repeat this while updating the override.yaml and doing helm upgrade --install eraser charts/eraser --values override.yaml to see it taking effect.
When fixed is an ignored status, the image does not get deleted. When we don't ignore it, it gets deleted (as expected). That alpine image only has a single CRITICAL - fixed vuln.

override.yaml
runtimeConfig:
  manager:
    scheduling:
      repeatInterval: "1s"
  components:
    collector:
      enabled: true
      image:
        tag: "v1.2.1"
    scanner:
      enabled: true
      image:
        repo: "docker.io/library/eraser-trivy-scanner"
        tag: "dev"
      config: |
         cacheDir: /var/lib/trivy
         dbRepo: ghcr.io/aquasecurity/trivy-db
         deleteFailedImages: false
         deleteEOLImages: false
         vulnerabilities:
           ignoreUnfixed: false
           types:
             - os
             - library
           securityChecks:
             - vuln
           severities:
             - CRITICAL
           ignoredStatuses:
             - fixed
    remover:
      image:
        tag: "v1.2.1"

deploy:
  image:
    repo: ghcr.io/eraser-dev/eraser-manager
    pullPolicy: IfNotPresent
    tag: "v1.2.1"

@inFocus7 Thank you for the contribution! :)

Looks like helm pull secret test was failing due to PodTemplate not found error which we fixed recently. Tests pass after rebase.

@ashnamehrotra Thanks! It looks like some checks failed, not sure if they were flakes though

@inFocus7 I can't seem to re-run tests either, checking if closing and re-opening resolves this

@inFocus7 made those changes in #917! Once that is merged we should be good to rebase. There are still vulns in the trivy binary which we can ignore for now since we cannot fix those.

🥳


Release enhanced-table-1.9.0_7.6.0.zip dependency issues?

Jul 10 11:34:22 computar kibana[58369]:     ERROR in ./plugins/enhanced-table/public/document-table-vis-data-params.js
Jul 10 11:34:22 computar kibana[58369]:     Module not found: Error: Can't resolve 'ui-select' in '/blah/kibana/plugins/enhanced-table/public'

I know next to nothing about js, but the extracted enhanced-table-1.9.0_7.6.0.zip has no reference to ui-select in package.json, as opposed to the 7.6 branch; perhaps that's it?

Hi,
I just tried to install enhanced-table-1.9.0_7.6.0.zip on Kibana 7.6.0, and it works great.
ui-select is a dependency provided by kibana core in $KIBANA_HOME/node_modules/ui-select.
So there is normally no need to add it into plugin itself

Hi,
Even I don't reproduce the issue, I just released kibana v1.9.1, where packages for kibana 7.6 now have ui-select dependency embedded.


soroban-cli cannot convert RawVal to json

What version are you using?
0.8.0

What did you do?

Here is a gist with examples that would fail when executed with soroban-cli:
https://gist.github.com/vinamogit/16970b94af3463eb5989f66d90281bfe
soroban contract invoke --wasm target/wasm32-unknown-unknown/release/bug_raw_to_json.wasm --id 222 -- get_bytes
What did you expect to see?
The json output like for other types.
What did you see instead?
thread 'main' panicked at 'not yet implemented: Bytes(
    ScBytes(
        BytesM(0000000000),
    ),
) doesn't have a matching Val', /.cargo/registry/src/github.com-1ecc6299db9ec823/soroban-cli-0.8.0/src/strval.rs:492:27

Good catch val is generally not supported at the moment.  Might I ask what your use case is for it? What is the reason for returning RawVal instead of Bytes?

No real use case at the moment, more a theoretical discussion from here (https://discord.com/channels/897514728459468821/1116730668999790736).
The case of cross contract call comes in for proxies maybe.
Take these contract functions

    pub fn remote(_env: Env, input: u32) -> u32 {
        input
    }

    pub fn call(env: Env, target: Address) -> RawVal {
        env.invoke_contract::<RawVal>(&target, &Symbol::new(&env, "remote"), vec![&env, 42_u32.into()])
    }

Invokations will looks like:
soroban contract invoke --id 46d257601ebfc99ebfff3cd72c95c7646f2be116bdb1d872f0b48494e76a4e7b --network futurenet -- remote --input 13
13

soroban contract invoke --id 5afe963c342e87e418ad392f072bffb2cb1b91f6d086b9fbc0897e4372b392ca --network futurenet -- call --target CBDNEV3AD274THV7746NOLEVY5SG6K7BC263DWDS6C2IJFHHNJHHWANY
error: cannot print result U32(42): Invalid pair U32(
    42,
) Val


knife-ec2 0.14.0 breaks compatibility with older chef-client

The latest version of knife-ec2 (0.14.0) introduces a requirement for Ruby 2.2.2 or greater.
Chef began bundling Ruby 2.3 with version 12.14, released about 3 months ago. Versions of chef-client prior to 12.14 install an older version of Ruby (2.1.6p336) which does not satisfy the requirements for knife-ec2 0.14.0.

What’s New in 12.14
Upgraded Ruby version from 2.1.9 to 2.3.1 Adds several performance and functionality enhancements.
https://docs.chef.io/release_notes.html

Therefore, knife-ec2 requires chef-client 12.14 or newer and this should be described in the CHANGELOG.

@nicwaller I verified it from my end too that knife-ec2-0.14.0 requires chef-client 12.14.
So we will modify the CHANGELOG accordingly. Thanks for pointing this out.

Fixed with https://github.com/chef/knife-ec2/pull/467

Added a post on discourse: https://discourse.chef.io/t/knife-ec2-0-14-0-released/9972/2


MVP weights available?

Hey, awesome project!
Quick question: do you have the MVP model weights available somewhere? The weights have disappeared from their storage location specified in the repo...
Best, Lars

Hi Lars,
Thanks for your inquiry. Unfortunately, the MVP model weights are currently not available on our server. I will search somewhere else to see if I have a backup. If you need robotic representations, you can explore some alternative options:)
Guangqi


Wrong docs examples for keyboard

I was reading API docs. Look at this method for example:
https://github.com/GoogleChrome/puppeteer/blob/master/docs/api.md#keyboardtypetext-options
It says that I should use:
page.keyboard.type('Hello');
But In fact that's not working and I should use:
page.type('Hello');
Should be docs fixed?

@JoelEinbinder what do you think about adding info around the differences between page.type, page.keyboard.type, handler.type, etc? The docs are fuzzy on when to use each.

It seems page.type('text') no longer works, moving from 0.10.2 to 0.13.0 and that I should now use page.keyboard('text')? I'm page.click(selector)-ing before each type, so I don't need to pass a selector which page.type now requires? I'm confused and agree clearer documentation would be helpful. Thanks.

page.type went through a lot of variations. The current, stable variants are:
page.type(selector, text) : focuses the selector and then types the text
page.keyboard.type(text): types the text into whatever currently has focus
elementHandle.type(text): Focuses the element, and then types the text.
If you find any documentation that is out of date, let me know! Thanks

@JoelEinbinder nice


Characters not easy to distinguish on dark themes

As you can see on the image, many of the characters are hard to distinguish on dark themes.
Horizon

Dark+

Abyss

I don't know if it's possible but the text should be shown in the same way as the colorscheme.

Please share steps to reproduce the issue. Which color them for example?

I've disabled all extensions and just enabled vscode-docs-view.
I have a simple code test:
console.log("Testing Themes")

While placing the cursor on log and switching themes I look for changes in the docs window.
Some seem to work and some don't.
Abyss ❌

Dark+ 💹

Sometimes a theme will not work. If I then change themes back and fourth a bit it sometimes starts to work again. As you can see on Dark+ theme that didn't initially work but now do.
Abyss (built-in theme) never works.

This is what I see with regard to this issue.
theme: One Dark Pro
code:
struct {
        /* mirror selected kvdb_rparams */
        u64 csched_samp_max;
        u64 csched_lo_th_pct;
        u64 csched_hi_th_pct;
        u64 csched_leaf_pct;
    } inputs;

Should be improved with 0.0.5. Let me know if there are any themes that still have issues
Keep in mind though that because the docs view has a different background color than the editor, some colors may visually look slightly different between the two (see the much darker editor background for Kimbie Dark for example):

I tried this out, and the experience is much improved. One thing that I thought was interesting was Docs View even showed git blame documentation coming from GitLens. Thanks Matt.

The new version works flawlessly for me.
Thanks for the help!