[Neue] Base

[ ] Only show to users who joined prior to

Nit: I think moving the description for Font to under the options would look a bit cleaner

Lets give this a spin on device. Things look good to me so far.

I haven't actually tested this PR in any capacity, but my understanding is that the Inter font does "Contextual Alternates" by default. Here's their demo:

Personally, I really don't like these. I find the modifications unexpected, potentially altering the meaning of text.
Anecdotally, I use the Element matrix client, which also uses Inter with Contextual Alternates - every time I've noticed it kicking in, it's been "incorrect" (e.g. inserting a multiplication symbol into something that was never a multiplication in the first place)
This CSS should fix it: font-variant-ligatures: no-contextual


Added subjects formatter class

This PR introduces duplicate code. Please re-use this formatter in https://github.com/jstack-eu/newyse/blob/master/Criteria/ReservationCriteria.php


Wrong Description in Example 1 & 3

The example states " If a file named company.aspx already exists, it won't perform the move." But the example code refers to company.docx.   Therefore I believe the example should state " If a file named company.doc already exists, it won't perform the move."
Example 3 has same type of issue....the code and description do not match.

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

ID: cc0498c8-0815-f23b-39ea-7692bf4bab16
Version Independent ID: 7bc9ccc7-b9f9-bbae-7c80-aa5dedb0d3f2
Content: Move-PnPFile (sharepoint-pnp)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Move-PnPFile.md
GitHub Login: @techwriter40
Microsoft Alias: kirks

@officedocsbot assign @yogkumgit

Hello @dh4099
Thanks for your feedback.
I need to kindly redirect you to open PnP cmdlets issues here a member of the PnP community will handle your request there.
Thank you!

@get-itips  Thank you very much for the contribution and sharing this explanation. @dh4099
Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Role template: Switching from Config to YAML editor breaks

Describe the bug
When switching editor between Config and YAML for the Role Template, it breaks. The issue may be related to https://github.com/rancher/dashboard/issues/12052, although the UI does not break in that case.
To Reproduce

Navigate to User & Authentication > Role Templates (/c/_/auth/roles#GLOBAL)
Edit Config for an existing user role, e.g. Administrator
Switch view to YAML

Result
Nothing happens, error in console:
entry-helpers.js:40 Error: Cannot find resourceFields for Schema management.cattle.io.templateversion (schemaDefinitions have not been fetched)
    at get resourceFields (steve-schema.ts:103:1)

Expected Result
It switches the view, just as it does for all the other resources.
Screenshots
https://github.com/user-attachments/assets/46de3314-2d04-47bd-91c0-92817215ee45
Additional context
Cluster editor, Workload editor and probably all the other work as expected.

@richard-cox does this look like part of your schema diet changes? Was this one of the schemas that was removed?

It means the resourceFields was accessed without the call to /schemaDefinition to populate it. There's plumbing in generic components that handle this though, so if it's a vue3 regression there's probably a hook somewhere that's not being called

It seems this issue is not blocking the Edit Yaml anymore, but you can still see that error on the console when you're redirected to the Role Templates page after saving/cancelling.
https://github.com/user-attachments/assets/e583e293-91cf-4099-b3ad-f9ea0b782354

The fetch errors only happen when navigating back to the list.
I suspect that this behavior is related to #11986. It looks like shell/components/ResourceDetail/index.vue is the culprit for the error message.
https://github.com/rancher/dashboard/blob/9e9e25b087d9499e6c31cfafaf066eee52f36102/shell/components/ResourceDetail/index.vue#L321-L338

No longer seeing error when cancelling/navigating back to list. Tested in:

Rancher v2.10-0cddb5f146709bf471b863e6715b0a7538572e74-head
Dashboard master 73bff03


Static Files Are Not Visible

When we try to deploy the application to AKS cluster, static files are not downloaded

Please check your .gitignore file, it's probably because it's being ignored in that file @reginmoses

@arkwrn - I can see the static files being present in the AKS pods.
Please refer the attached screenshot for reference.
But when we access the web application via the browser, the static files are not shown in the website.

reviewed.

@reginmoses Do you use the official containers or are you building your own? Which version are you running?

Hi Klar,
The Helm installation steps provided in the documationat did not work for
us.
If you have a better alternative solution to deploy to Azure K8s, please do
let us know.
We do the following to make it run in our cluster now:
*kompose convert *
kubectl apply -f defectdojo-data-pv.yaml
kubectl apply -f mysql.yaml
kubectl apply -f initializer.yaml
kubectl apply -f rabbitmq.yaml
kubectl apply -f celeryworker.yaml
kubectl apply -f celerybeat.yaml
Then create a custom image of defect dojo with the below changes:
·         Update Dockerfile.django
https://git.eon-cds.de/esp/esp-devtools/poc-dev/defectdojovm/-/commit/067fc8f6b31987c7af80978e42d6728fc554fac3
to Dockerfile
·         Replace username as root instead of 1001 in the dockerfile
·         Replace DD_UWSGI_MODE value to http instead of socket
·         Added the below lines
RUN chmod -R 777 /entrypoint-celery-beat.sh
RUN chmod -R 777 /entrypoint-celery-worker.sh
RUN chmod -R 777 /entrypoint-initializer.sh
RUN chmod -R 777 /entrypoint-uwsgi.sh
RUN chmod -R 777 /entrypoint-uwsgi-dev.sh
RUN chmod -R 777 /entrypoint-uwsgi-ptvsd.sh
RUN chmod -R 777 /entrypoint-unit-tests.sh
RUN chmod -R 777 /entrypoint-unit-tests-devDocker.sh
  RUN chmod -R 777 /wait-for-it.sh

Deployed the below 2 yaml files:
kubectl apply -f uwsgi.yaml
kubectl apply -f ingress.yaml
On Tue, Jul 7, 2020 at 8:36 PM alles-klar<EMAIL_ADDRESS>wrote:

@reginmoses https://github.com/reginmoses Do you use the official
containers or are you building your own? Which version are you running?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DefectDojo/django-DefectDojo/issues/2143#issuecomment-654928266,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGTC65WAT2BCQXW5DACM6DR2M2XJANCNFSM4L3TMRMA
.

--
Thank you.
Regards,
J. Regin Moses


fix property id

What's the purpose of this pull request?
When adding a product to cart with additionalProperties we were receiving the following error:
GraphQLError [Object]: Variable "$additionalProperty" got invalid value { propertyID: "4ba0a4d37b7f733da5ddb281ea50d883", name: "activeSubscriptions", value: "abonnements", valueReference: "SPECIFICATION" }; Field "propertyID" is not defined by type IStorePropertyValue.

This PR addresses this issue

Doesn't this defeat the whole purpose of not having to serialize the attachment id on the frontend?

Not really. We still don't need to serialize anything on the frontend.

Doesn't this defeat the whole purpose of not having to serialize the attachment id on the frontend?

Not really. We still don't need to serialize anything on the frontend.

It's not clear to me how it wouldn't. This is a required property on input, and we use it for the validateCart mutation. When people are sending attachments to the cart with the items, they don't have the id yet because they're creating this object on the frontend, and to have it, they must serialize it.

Doesn't this defeat the whole purpose of not having to serialize the attachment id on the frontend?

Not really. We still don't need to serialize anything on the frontend.

It's not clear to me how it wouldn't. This is a required property on input, and we use it for the validateCart mutation. When people are sending attachments to the cart with the items, they don't have the id yet because they're creating this object on the frontend, and to have it, they must serialize it.

Ok, I think I've precipitated myself on openning this PR that early, however I think it's working now.
We had two issues. The first one is that when fetching a product, additionalProperties was fetched alongside the propertyID. When adding this product to cart, this propertyID would be send to the backend, causing this error:
GraphQLError [Object]: Variable "$additionalProperty" got invalid value { propertyID: "4ba0a4d37b7f733da5ddb281ea50d883", name: "activeSubscriptions", value: "abonnements", valueReference: "SPECIFICATION" }; Field "propertyID" is not defined by type IStorePropertyValue.

To fix this issue, I added a non required field on IStorePropertyValue
The second issue was that adding a product to cart with additionalProperties would result in the item being removed from cart. This was because of the orderForm item lacking specifications and only having attachments. Adding a filter to getId functions seems to have fixed this issue.
A good case for this PR is this store , that has specifications set and after this PR, should have the bug fixed and the cart should work as intended


[WFRP4e System] Better Background for Gm Notes

Would it be possible to have another Background for Gm notes, Is really hard to see the text written.
https://cdn.discordapp.com/attachments/614126366442651679/774061492348780544/unknown.pngFrom: danil0wfrp4e: 3.0.0wfrp4e-core: 1.0wfrp4e-rnhd: 1.0wfrp4e-starter-set: 1.0

GM Notes is not needed in WFRP, as the system already provides it.


solve issue #1635

Ⅰ. Describe what this PR did
fixes issue #1635
Ⅱ. Does this pull request fix one issue?

fixes issue #1635
Ⅲ. Describe how you did it
mock input compare output
Ⅳ. Describe how to verify it
check expected result
Ⅴ. Special notes for reviews
no

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


Replace dfm update while loop

replaced while loop for dfm timesteps by single call, can you plse review ?

nice, but the tests fail

The main pipeline doesn't work with lumbricus anymore.
You will have to fix https://dpcbuild.deltares.nl/buildConfiguration/iMOD6_Coupler_TestbenchCouplerWin64Lumbricus?mode=builds and use that instead.


Fix Broken Link

Fix Broken Link of Apress Contract

@TalAter Any update on this?

Thank you so much for the fix 👍


fix: issue #391 remove applicationId and transactionServiceGroup from BranchSession

Ⅰ. Describe what this PR did
issue #391 remove applicationId and transactionServiceGroup from BranchSession
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #393 into develop will decrease coverage by 0.22%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop     #393      +/-   ##
=============================================
- Coverage      30.59%   30.37%   -0.23%
+ Complexity       615      611       -4
=============================================
  Files            166      166
  Lines           6727     6697      -30
  Branches         806      802       -4
=============================================
- Hits            2058     2034      -24
+ Misses          4457     4455       -2
+ Partials         212      208       -4

Impacted Files
Coverage Δ
Complexity Δ

...m/alibaba/fescar/server/session/SessionHelper.java
73.07% <ø> (-1.93%)
4 <0> (ø)

...m/alibaba/fescar/server/session/BranchSession.java
89.09% <ø> (+2.13%)
30 <0> (-4)
:arrow_down:

Continue to review full report at Codecov.

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


[dvsim] Fix for missing coverage & improvements

Fixed and optimized the logic that tracks the status of currently
dispatched processes. There was a bug in the existing code the caused
dvsim to prematurely exit before all coverage jobs were done.

Changed the way the progress timer is incremented - rather than incrementing seconds, the new logic now increments in hh:mm:ss format. This is better than the old approach with get_etime() method that added divide and modulo operations to obtain hh:mm:ss from seconds.

Signed-off-by: Srikrishna Iyer<EMAIL_ADDRESS>

@weicaiyang, PTAL. CI is now passing.


sync phone-number

Solves issue #800

Long time no see. I have started this PR. But would like some feedback if I have understood the tasks correctly.
I have done following:
Ran bin/configlet sync -u -e phone-number --yes --docs --filepaths --metadata --tests include
Added uuid to test cases
Remove strict types comment in example code and tests file.
Testcases with uuid: 2de74156-f646-42b5-8638-0ef1d8b58bc2 and 4a1509b7-8953-4eec-981b-c483358ff531 are not implemented, as they would break current community solutions. So how to deal with those?
I don't see which 4 tests cases that aren't implemented according to https://github.com/exercism/problem-specifications/blob/main/exercises/phone-number/canonical-data.json .
A pointer would be nice.

Thank you for adjusting the DocBlocks. Don't forget to un-draft the PR, so it can be merged.
For compatibility remove testing for the exception message or use expectExceptionMessageMatches() with a RegExp that can handle both texts. I'd prefer not testing for the message.

For compatibility remove testing for the exception message or use expectExceptionMessageMatches() with a RegExp that can handle both texts. I'd prefer not testing for the message.

Will do one of the next days. Don't have time right now. But will take care asap.


Add a license

This will reduce ambiguity and provide ease-of-mind for companies and developers that want to use/improve this tool.
The Qdrant DB uses Apache 2.0, maybe keep going with that?

Thanks for pointing this out. I've suggested this license in a PR here: https://github.com/qdrant/qdrant-web-ui/pull/58


Georefencing by lat/lon using string columns page get stuck

Context
Creating a map from an excel file with lat/long columns
Steps to Reproduce
Please break down here below all the needed steps to reproduce the issue

Uploading a xls file with lat/long columns that were loaded as string.
sin-cabecera-listado-longitud-latitud-municipios-espana.xls.zip
Create a map with that file
Geolocate by lat/lon

Current Result
Neverending roulette
Expected result
Points geolocated properly as it happens when lat/lon columns are numeric.

More info: https://github.com/CartoDB/cartodb/issues/11511


Add status indicator to applicant badge component

Release Type: Breaking change
Description
These changes add a status indicator to the applicant badge component.
This status indicator is a circle that can have three colors depending on the status:

Accepted:  green
Excluded:  red
Pending: yellow

Checklist

[x] set yourself as an assignee
[x] make sure your code lints (npm run lint)
[x] Flow typechecks passed (npm run typecheck)
[x] manually tested the app by running it in the browser and checked nothing is broken and operates as expected!

Impacted Areas
Applicant Badge Component.
Screenshots

Suggested changes have been applied. Great tool the name that color one. Thanks!


fix: hide promotion section, change contribute link

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface
/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Hide card with link to promotion section, change link of contribute button
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: bullet03
Once this PR has been reviewed and has the lgtm label, please assign vjjmiras for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

layouts/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bullet03, vjjmiras
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

layouts/OWNERS [vjjmiras]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 10acb0deba5bafe14bf464c9d110ff1cc4a1955a


MutableInode  reference bug

Alluxio Version:
2.8.1
Describe the bug
if we set
alluxio.master.metastore=ROCKS
alluxio.master.metastore.inode.cache.max.size=0
copyFromLocal will get
Cannot persist an incomplete Alluxio file
Additional context
update inode by inode tree use the same reference with lockedInodePath only work in heap or caching .
the LockedInodePath will got a dirty value when use that config

@elega @lucyge2022 Could you take a look? Thanks!

Thanks for reporting the issue. The fix is WIP
https://github.com/Alluxio/alluxio/compare/master...elega:alluxio:yimin/fix-reference-issue?expand=1

fixed in #17145


Fix mail-sink Kamelet

In fact mail-sink only supports smtp protocol. Others like pop3 and imap do not make sense for sending mails
SMTPS protocol is also not supported because SSL context is missing
to:{{protocol}} stopped working with latest versions so remove protocol property and always use "smtp"
Also fix YAKS test

Good to backport on 0.9.x. Thanks

@oscerd thanks! could you please merge? I need this for another PR


Simplify reth db drop

Describe the feature
Currently the reth db drop command requires us to update it with an up-to-date list of tables, but in reality we can just delete the file pointed to by the --db flag, which would lower maintenance burden.
Additional context
The command is in bin/reth/src/db/mod.rs

Hi, I'm just starting on this project and would be interested in taking this one. I that ok?

Yep! I've assigned you to the issue. If you feel stuck, open up a draft PR with your questions and someone will help you. If you are prevented from solving the issue for any reason, let me know and I will unassign you :smile:

Thank you!

I guess we'd still want the previous functionality if we wanted to just drop a single table?

Yes @gakonst


Allow generation of multiple clients

Hello,
I have 2 different API servers that I need to generate API clients for. I've created a separate folder for each of them with in it the conf file and the swagger spec.

MyApiOne

openapi-generator-config
openapi.yaml

MyApiTwo

openapi-generator-config
openapi.yaml

I'm using the plugin to generate the client code at build time, but the second one keeps overriding the first one (because there can only be one Client in the project
I think to cover that issue the class name should somehow bear the target name inside, like class MyApiOneClient and class MyApiTwoClient

If they're in separate modules, MyApiOne.Client and MyApiTwo.Client should be able to disambiguate them.
However, I have also been concerned about some of the generic generated names being exposed and potentially polluting the namespace of consumers.
Writing this by hand, I would consider a namespace-style enum, which would give:
enum MyApiOne { /* everything in here */ }
enum MyApiTwo { /* everything in here */ }

and then they'd always be accessed like MyApiOne.Client with all of the support types similarly namespaced under MyApiOne as well.
I could imagine this via something like a namespace: option in config?

This is a common workflow, touched on in https://swiftpackageindex.com/apple/swift-openapi-generator/0.2.3/documentation/swift-openapi-generator/api-stability-of-generated-code#Create-a-curated-client-library-package
As @porglezomp, you want to generate the two APIs not just into separate directories, but into separate modules in your package.
So if you have this today:
Package.swift
Sources
  - MyModule
    - MyApiOne
      - openapi-generator-config.yaml
      - openapi.yaml
    - MyApiTwo
      - openapi-generator-config.yaml
      - openapi.yaml

Change this instead to:
Package.swift
Sources
  - MyModuleForAPIOne
    - openapi-generator-config.yaml
    - openapi.yaml
  - MyModuleForAPITwo
    - openapi-generator-config.yaml
    - openapi.yaml

Then you can create a third module that imports MyModuleForAPIOne, MyModuleForAPITwo, or both, and you use the fully qualified names MyModuleForAPIOne.Client/MyModuleForAPITwo.Client in any files that import both.


docs: Adds ADR 10. Localization

Description
Documents our decision to translate Superset assets.
Supporting information
Part of https://github.com/openedx/openedx-aspects/issues/61
Author Notes & Concerns

Were there other rejected options I should mention?
Should I go into any more detail about how we ended up translating the assets?

Thanks for the pull request, @pomegranited! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.


podvm: remove sudo if not required in Makefile

Some sudos were added which are not required. We shouldn't force sudo unless it is required to not mess up local dev environments.

@mkulke ,
still needing super-privileges? PermissionError: [Errno 1] Operation not permitted

@mkulke ,
still needing super-privileges? PermissionError: [Errno 1] Operation not permitted

yup, there is probably a problem with an earlier step being run as root. let's see...

Ah, maybe that mkosi error we are seeing it what I saw and why I added sudo in the first place?

gah, ubuntu 24.04 blocks userns by default, so you cannot build rootless unless you do:
sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0

this will not get picked up however in this PR, so we need #2199 with just this fix 😭

https://github.com/confidential-containers/cloud-api-adaptor/pull/2199 is merged now \o/


Request reviews from all core/app engineers

Context
Breakfast discussion w/ @cmwaters @ninabarbakadze
Proposal
@cmwaters @ninabarbakadze don't like how the auto PR request reviewer action only requests reviews from two people on core/app.
I personally don't have a strong opinion for or against this.
Proposal
They would prefer if the PR request reviewer action requested reviews from all core/app engineers.

thanks! I think @evan-forbes should have the perms for this

we just need to change this line
https://github.com/celestiaorg/celestia-app/blob/main/.github%2Fauto_request_review.yml#L18


Publisher: add more abstract methods

Is your feature request related to a problem? Please describe.
Publisher expects more methods which are not abstract currently in any base class or interface and therefore it is a bit difficult to know that they need to be implemented.
Describe the solution you'd like
At least these methods should be added:

install
~uninstall~
get_context_data
update_context_data

And maybe:

~list_instances~ see comment lower
~remove_instance~

These were expected for SubsetManager (for host where is not totally clear which publishable instances are there). SubsetManager is probably obsolete and replaced with Publisher.
[cuID:OP-4339]

Few notes:

install should not be abstract but should be available with docstring
uninstall is pointless method that was never used in openpype
get_context_data and update_context_data is abstract in IPublishHost
list_instances and remove_instance is only for SubsetManager which will be removed with new publisher

uninstall is pointless method that was never used in openpype

I've actually started using uninstall() in Substance Painter integration since it's installed through an Substance plug-in which the user can disable too. Not sure if that's the way to go in the long run, but just FYI.


Hyperdimension Neptunia Re;Birth2 SISTERS GENERATION [PCSE00508]

App summary

App name: Hyperdimension Neptunia Re;Birth2 SISTERS GENERATION
App serial: PCSE00508
App version: 1.00

Vita3K summary

Version: v0.1.9
Build number: 3338
Commit hash: https://github.com/vita3k/vita3k/commit/44d8c5da
CPU backend: Dynarmic
GPU backend: OpenGL

Test environment summary

Tested by: hp
OS: Windows 10/macOS/Linux Distro, Kernel Version?
CPU: AMD/Intel?
GPU: AMD/NVIDIA/Intel?
RAM: 16 GB

Issues

Screenshots

Log
Recommended labels

A?
B?
C?

it crashes my emulatwer

lack of info


Handles opened by the application

Hello guys!
I saw that the exporter application connects to the QMGR and opens 5 handles by queue (subscribe to each metric like openclose, put, get, inqset, conndisc). As we have environments with hundreds of queues, we had a problem with de maxhands parameter of the qmgr, that needed to be strongly increased.
It may represent some problem, because we had to alter the maxhands to 2100 and it opens the possibility to other applications to work in a innapropriated behaviour, like a looping of handles in some queues.
Other problem is that as the number of queues grow, we need to increase de maxhands parameter too.
Do we have anyway to alter the application logic so it opens (for example) the limit of 250 handles per connection? In that way, we would never have to alter the maxhands of the qmgr.
Using this example, the application could establish a connection with the qmgr and starts to open the handles. If the number of handles reaches 250, the application establishes a new connection and starts to open new handles with this new connection, and so on.
Hope I made myself clear, guys :D
If I need to explain something else, just say so :)
Kind regards,
Alex

Hi @AlexAlvesJr I think I found problem as you mention, in my environment I increase maxhands from 256 (Default) to 625 due to I need to monitor more object in MQ.
My system found error that they cannot create more process and generate many dump log (FDC) until file system full.
I think it's because of maxhands parameters, some application retry to MQ with wrong config and retry every 10 secs.


in_app_purchase : purchase is made but status is not completed and verified on iOS 14

I added in_app_purchase plugin for payment in my flutter app. It works perfectly in android and sandbox, but after publishing the app, it didn't work correctly with real card. Payment is completed successfully, coins should have been added, but didn't.
Here's my code:
 _initialize() async {
    _available = await _iap.isAvailable();
    if (_available) {
      await _getProducts();
      _subscription =
          _iap.purchaseUpdatedStream.listen((purchases) => setState(() {
                print('New purchase');
                _purchases.addAll(purchases);
                Platform.isIOS ? _getPastPurchases() : _verifyPurchase();
              }));
    }
  }

  Future<void> _getPastPurchases() async {
    QueryPurchaseDetailsResponse response = await _iap.queryPastPurchases();
    print(response.pastPurchases);
    for (PurchaseDetails purchaseDetails in response.pastPurchases) {
      print('Purchase status: ${purchaseDetails.status}');
      _iap.completePurchase(purchaseDetails);

      if (purchaseDetails.status == PurchaseStatus.purchased) {
        _verifyPurchase();
      }
    }
    if(mounted) {
      setState(() => _purchases = response.pastPurchases);
    }
  }

// the problem somewhere in _getPastPurchases method, I try to complete purchases, and then verify it

    _verifyPurchase() {
    PurchaseDetails purchaseDetails = _hasPurchased(widget.productID);
    if(purchaseDetails != null && purchaseDetails.status == PurchaseStatus.purchased) {
         // my logic to add purchased coins
     }
  }

//Probably, purchaseDetails.status == PurchaseStatus.purchased condition is false somehow, //but I don't know what's wrong and then why in sandbox is working correctly

@ShamsFaraj
There's similar open issue describing your case, 59763.
Please follow-up there for updates and any questions.
Closing this as duplicate.

No, my case is different. In my case, purchase completed successfully (user is charged) but purchase status is not completed and verified. And also it’s happening on  ios 14 devices but it work on ios 12,13
From<EMAIL_ADDRESS>Sent: Thursday, October 8, 2020 5:43 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [flutter/flutter] in_app_purchase : In iOS working in sandbox but not working in real payment (#67630)
@ShamsFarajhttps://github.com/ShamsFaraj
There's similar open issue describing your case, 59763https://github.com/flutter/flutter/issues/59763.
Please follow-up there for updates and any questions.
Closing this as duplicate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/flutter/flutter/issues/67630#issuecomment-705577551, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARJR7DBGMLDU24JVGFSGMS3SJW6YDANCNFSM4SIVPRJA.

@ShamsFaraj Thanks for clarifying. Reopening.

Thanks a lot
From<EMAIL_ADDRESS>Sent: Friday, October 9, 2020 10:21 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [flutter/flutter] in_app_purchase : In iOS working in sandbox but not working in real payment (#67630)
@ShamsFarajhttps://github.com/ShamsFaraj Thanks for clarifying. Reopening.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/flutter/flutter/issues/67630#issuecomment-705993149, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARJR7DDSHCIDNIHC452RPILSJ2TU5ANCNFSM4SIVPRJA.

May be related to 59763

No, my case is different. In my case, purchase completed successfully (user is charged) but purchase status is not completed and verified. And also it’s happening on ios 14 devices but it work on ios 12,13 From<EMAIL_ADDRESS>Sent: Thursday, October 8, 2020 5:43 PM To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [flutter/flutter] in_app_purchase : In iOS working in sandbox but not working in real payment (#67630) @ShamsFarajhttps://github.com/ShamsFaraj There's similar open issue describing your case, 59763<#59763>. Please follow-up there for updates and any questions. Closing this as duplicate. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#67630 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARJR7DBGMLDU24JVGFSGMS3SJW6YDANCNFSM4SIVPRJA.

Did you find a solution??

No, my case is different. In my case, purchase completed successfully (user is charged) but purchase status is not completed and verified. And also it’s happening on ios 14 devices but it work on ios 12,13

Did you find a solution??

Yes, I changed my getPurchaseDetails function and passed purchases from initializer as parameter, not by iterating respones.pastPurchases. I don't know why, but pastPurchases method didn't fetch items that I need. I share updated codes below:
_initialize() async {
_available = await _iap.isAvailable();
if (_available) {
await _getProducts();
_subscription = _iap.purchaseUpdatedStream.listen((purchases) => setState(() {
print('New purchase ${purchases.map((e) => e.status).toList()}');
analytics.logEvent(name: 'new_purchase', parameters: {'value': purchases.map((e) => e.status.toString()).toString()});
_purchases.addAll(purchases);
Platform.isIOS ? _getPastPurchases(purchases) : _verifyPurchase();
}));
}
}
Future _getPastPurchases(List purchaseDetails) async {
for (PurchaseDetails detail in purchaseDetails) {
_purchases = [];
_iap.completePurchase(detail);
_purchases.add(detail);
if (detail.status == PurchaseStatus.purchased) {
_verifyPurchase();
}
}
}

Thanks for that - so was your in app purchases ever working? Mine were working perfectly on iOS and android and then I updated my app (didn't change the purchase code) and not it won't work on iOS 14 devices

No, my case is different. In my case, purchase completed successfully (user is charged) but purchase status is not completed and verified. And also it’s happening on ios 14 devices but it work on ios 12,13

Did you find a solution??

Yes, I changed my getPurchaseDetails function and passed purchases from initializer as parameter, not by iterating response.pastPurchases. I don't know why, but pastPurchases method didn't fetch items that I need. I share updated codes below:
_initialize() async {
_available = await _iap.isAvailable();
if (_available) {
await _getProducts();
_subscription = _iap.purchaseUpdatedStream.listen((purchases) => setState(() {
print('New purchase ${purchases.map((e) => e.status).toList()}');
analytics.logEvent(name: 'new_purchase', parameters: {'value': purchases.map((e) => e.status.toString()).toString()});
_purchases.addAll(purchases);
Platform.isIOS ? _getPastPurchases(purchases) : _verifyPurchase();
}));
}
}
Future _getPastPurchases(List purchaseDetails) async {
for (PurchaseDetails detail in purchaseDetails) {
_purchases = [];
_iap.completePurchase(detail);
_purchases.add(detail);
if (detail.status == PurchaseStatus.purchased) {
_verifyPurchase();
}
}
}

Thanks for that - so was your in app purchases ever working? Mine were working perfectly on iOS and android and then I updated my app (didn't change the purchase code) and not it won't work on iOS 14 devices

It was my first implementation, IOS was updated to 14 during my app development, but before correction I checked in several devices, it wasn’t working only on IOS 14, but older ones were just fine.
From<EMAIL_ADDRESS>Sent: Monday, February 1, 2021 2:14 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [flutter/flutter] in_app_purchase : purchase is made but status is not completed and verified on iOS 14 (#67630)
Thanks for that - so was your in app purchases ever working? Mine were working perfectly on iOS and android and then I updated my app (didn't change the purchase code) and not it won't work on iOS 14 devices
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/flutter/flutter/issues/67630#issuecomment-770741463, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARJR7DEJSQT36P5JIKCETA3S4Z5GXANCNFSM4SIVPRJA.

Are you having to send the updated version to app store each time you have to check if it's working in production?

It was my first implementation, IOS was updated to 14 during my app development, but before correction I checked in several devices, it wasn’t working only on IOS 14, but older ones were just fine. From<EMAIL_ADDRESS>Sent: Monday, February 1, 2021 2:14 PM To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [flutter/flutter] in_app_purchase : purchase is made but status is not completed and verified on iOS 14 (#67630) Thanks for that - so was your in app purchases ever working? Mine were working perfectly on iOS and android and then I updated my app (didn't change the purchase code) and not it won't work on iOS 14 devices — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#67630 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARJR7DEJSQT36P5JIKCETA3S4Z5GXANCNFSM4SIVPRJA.

Thanks so much for providing that --> Would you mind sharing your _verifyPurchase() code please?

I'm having the same issue, if you run the example app on iOS 14, subscribe to an auto-renewable subscription and wait until the subscription is automatically disabled after a few minutes (we're using here the sandbox mode, it simulates the subscription getting renewed automatically a few times before simulating the user cancelling it). Now close the app and if you try again to purchase the same subscription after it has been disabled/cancelled you get the error:
// some data has been censored with xxxx
flutter: PlatformException(storekit_duplicate_product_object,
There is a pending transaction for the same product identifier.
Please either wait for it to be finished or finish it manually using `completePurchase` to avoid edge cases.,
{applicationUsername<EMAIL_ADDRESS>  requestData: null,
  quantity: 1,
  productIdentifier: com.xxxx.monthly_renewable_subscription,
  simulatesAskToBuyInSandbox: false},
null)

Our backend verification simply hits the endpoint from apple and gets the data sent by them, to determine if a purchase is verified we simply check that the subscription is active by checking if the expiration_intent is not present and from latest_receipt_info the expires_date_ms is after the current date and cancellation_date_ms is not present.

The problem described in this issue is caused by the incorrect use of the queryPastPurchase method. The queryPastPurchase method should be used to restore purchased content on a new device or in case the user had to reset their current device.
Analysing the provided code by the OP, the queryPastPurchase method is used to refetch the purchase and then complete it. This is not how queryPastPurchase is intended to be used. And will never really work since iOS will only return purchases that have been completed (i.e. which are confirmed by the App by calling the completePurchase method). Because these purchases have never been completed, they will never be returned by the queryPastPurchase method.
The correct way to handle it would be as follows (on both Android and iOS):

Listen for purchases on the purchaseUpdatedStream;
When a purchase arrives first verify it and deliver the content (if verification is successful);
Complete the purchase by calling the completePurchase method;

Taking the provided code sample, it could be updated as follows:
 _initialize() async {
    _available = await _iap.isAvailable();
    if (_available) {
      await _getProducts();
      _subscription =
          _iap.purchaseUpdatedStream
            .listen((purchases) => setState(() {
                print('New purchase');
                _purchases.addAll(purchases);

                // Verify the purchase
                if (purchase.status == PurchaseStatus.purchased) {
                  _verifyPurchase();
                }

                // Make sure the purchase is completed.
                if (purchase.pendingCompletePurchase) {
                  _iap.completePurchase(purchase);
                }
              }));
    }
  }

  _verifyPurchase() {
    PurchaseDetails purchaseDetails = _hasPurchased(widget.productID);
    if(purchaseDetails != null && purchaseDetails.status == PurchaseStatus.purchased) {
         // my logic to add purchased coins
     }
  }


Error: Linking tmp mounts to host

I just tried to follow the basic example from install.md document. Unfortunately, this fails right at the start.
root@ishikawa ~# podman run --privileged --pid=host --net=none --security-opt label=type:unconfined_t ghcr.io/cgwalters/c9s-oscore bootc install --target-no-signature-verification /dev/nbd0
ERROR Linking tmp mounts to host: Exchanging /tmp <=> /tmp.tmp: Invalid argument (os error 22)

root@ishikawa ~# rpm -q podman
podman-4.2.0-9.el9_1.x86_64
root@ishikawa ~# cat /etc/os-release
NAME="Red Hat Enterprise Linux"
VERSION="9.1 (Plow)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="9.1"
PLATFORM_ID="platform:el9"

Ok, a comment in the code states that this part is to avoid forcing the user to use bind mount for /tmp and /var/tmp. But that should also allow such case, not forbid it entirely. I'm perfectly capable of adding those two bind mounts. ;)

So, the funny thing is that I'm using OpenZFS as a container storage which, currently, doesn't support renameat2(). That is why I'm getting this error. renameat2() will (probably) come with the 2.2.0 release of OpenZFS, so for now, XFS for this particular use case will have to be enough.

This is probably fixed by https://github.com/containers/bootc/pull/225

The problem is still present trying to run the -to-filesystem install on a fresh ubuntu setup, just for reference. I was doing some tests here and there with debian based distros.


WIP: gfx 0.14 support and renderer implementation

I cannot use glium backend in my environment (Ubuntu16.04 on Parallels on OSX. It crashes.)
This branch works fine even in the environment! I can't wait the moment when this is merged.

Looks like there's quite a lot of interest in this, so I'll try to finish this branch later this week. :smile:
I still need to upgrade to gfx 0.15 and do some cleanup. There's also something wrong with the SRGB handling because everything is too bright.

Great news for me!


Set default wallet payment method when one card/account exists

Problem
Coming from this conversation, a user is forced to pick the bank account/card they want to transfer their balance to, even when they only have one bank account/card associated with their Expensify account. This is an unnecessary step in the likely situation where they want to transfer to the one account/card they added.

Solution
Let's set the default payment method when only one bank account or card is associated with a user's Expensify account, but not show the "Default" badge until there are multiple accounts/cards to choose from.

Technically this is a few issues - but I'll use this one to track them:

[ ] Make sure to set a default automatically whenever it is added (back end change)
[ ] Only show a default badge when there are more than 1 payment methods (front end change)
[ ] Skip account selection screen when there is only one option (front end change)

Ah, so first bullet is on prod now.
Joel has a PR that should solve the second bullet that's in review.
I can look into the default badge change next - should be pretty small.

Just merged https://github.com/Expensify/App/pull/7767 which fixes

Only show a default badge when there are more than 1 payment methods (front end change)

And waiting for it to go on staging. Joel's PR here is also on Production now so that takes care of the second bullet.

Last PR still on staging.

Looks like all related PRs are done here.


Support Motion Photos

https://medium.com/android-news/working-with-motion-photos-da0aa49b50c

https://timojyrinki.gitlab.io/hugo/post/2021-03-30-pixel-motionphoto-microvideo-file-formats/
https://github.com/calangoni/MotionPhotoSplitter/blob/master/main.cpp


Add support for CocoaPods

I think there are a few things that need to be done before this is ready to go:

[ ] Add a new 0.1.3 (or whatever) release
[ ] Update .podspec to point to that release
[ ] Verify that emails are correct in the .podspec (I guessed)
[ ] Verify that deployment targets are correct
[ ] Run pod spec lint to make sure it can be built with the latest code
[ ] Upload spec to trunk (here is a helpful guide)

I just did some more digging and it's actually not as simple as this. Because the project is structured to treat the individual local libs as separate targets I think this might not compile when someone tries to pull this in as a dependency. Given my general lack of experience maintaining pods I'm not 100% sure how to replicate what you're doing in Package.swift inside of a .podspec, but I don't think something this simple will do it.

Hey DJ! Sorry for not being more responsive! It's been a busy week. We're very interested in figuring out these kinds of dependency situations in general, but we've had trouble ensuring that CocoaPods and Carthage work in the longterm for most of the open source projects we've worked on. Lemme talk to @mbrandonw about how we can try to ensure that libraries we work on, even experimental ones like the Prelude, can have support for CocoaPods and other dependency management systems.

Another update :) We got around to figuring out XcodeGen for a repo to battle some SwiftPM weirdness and to unlock Carthage support: https://github.com/pointfreeco/swift-snapshot-testing/pull/77
We'll be wanting to further explore how we can use this approach in all our repos. Hopefully soon!

From #107: 😅

And the world turns :) We've been slowly phasing out support for non-SPM tools, and Prelude's been in a bit of a "maintenance mode," so we're going to finally close this out. Thanks for coming along on the journey 😄


Support creating a new task of a particular type programmatically

In our extension we would like to guide the user to create a task of a specific type. It's already possible to read all the tasks and determine whether one of the desired type exists, however the programmatic flow seems to require executing workbench.action.tasks.configureTaskRunner which exposes all task types.
Would it be possible to expose a function in the API to create a specific task (e.g. by executing _configureTask) or accepting a filter in workbench.action.tasks.configureTaskRunner similar to runTask?

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Enhong Liu lab profile

Hi Miel, I uploaded the qmd file and a profile pic to the folders. Please review and let me know if they look good.

Added Enhong


Optional use of yml for data

THere are cases that it is needed long pieces of text as data in a project, and the yaml format makes it easier to handle large strings of text (like a blog post entry)

You can use the [dataFunction] config to customize this: https://github.com/vigetlabs/blendid#datafunction

Great! It seems now that the most important thing for blendid right now is improve the documentation and examples with copy&paste snippets. I hope this weekend I can get into that


Add missing specs for ->var.foo semantics with assignments

It is one of the reason of our confusion. Don't you think so, @waj?

Co-authored-by works. Thank you @asterite.

I don't know why I removed this from the milestone back in the day, when it was already accepted. We can probably just merge it, it's only adding specs =)
@asterite Since I requested your review, maybe take a look?


restrictedUrlPattern does not exist in the latest published npm package 3.1.0

I noticed restrictedUrlPattern was added in this PR, but it has not been released yet to npm. Could you release this feature?

In general, as we're deprecating the project, you should look into alternative approaches to rendering on the web.


Obey git hooks?

SublimeGit is ignoring my post-commit hook in my projects .git/hooks folder. Should this be working? Is SublimeGit running git in some way that might prevent this from working?

I've determined a problem with this where I'm only able to run my git hooks if I close a commit without typing anything into it. I've verified the $PATH variable in sublime is correct using import os; print(os.environ['PATH'])
Here is what my output looks like when I ctrl+w out of a commit message and don't type anything
### Checking puppet syntax, for science! ###
site/common/manifests/test.pp looks good

### Checking if puppet manifests are valid ###
OK: site/common/manifests/test.pp looks valid

### Checking if ruby template syntax is valid ###

Everything looks good.

Here is what happens when I commit properly with a commit message
[master 054d1b0] test
 1 file changed, 48 insertions(+)
 create mode 100644 site/common/manifests/test.

I'm interested in the prepare-commit-msg hook.
Following @miped's explanation I played around a bit and agree that SublimeGit needs some adjustment to accommodate this hook's expected behavior. The explanation is several comments up:

Basically, SublimeGit runs git commit -F - <commit message>

When -F is used, git collects the commit message from a file or stdin before the prepare-commit-msg hook is executed.
vim-fugitive does an interesting thing to work around this situation: it calls git commit twice:

The first time to run these hooks, capturing any errors and also capturing .git/COMMIT_EDITMSG
The second time with -F, reading the commit message from the first run.

Someone wrote a nice breakdown of this behavior on StackOverflow.
I believe this solution applies to the pre-commit hook too (since it's also affected by -F), but does not apply to other hooks.


miscalculation for popover - div.popover

For pages which are very wide (>1600px), when the page is scrolled the left position of div.popover is not set properly. The popover arrow is placed properly under the element that is clicked, but the content itself is placed far from the arrow.
I am working around using the following code - (in the click handler of the button element; after calling the popover 'show' following lines are added)
if ($(document).scrollLeft()>0){
$('.arrow').css('left','');
$('.popover').css('left', parseInt($(this).position().left)-$('.popover').width()/2+5);
}

What version of Bootstrap are you using?

I am using Bootstrap v3.3.5

Punting as WontFix per https://github.com/twbs/bootstrap/issues/15590#issuecomment-113321235
Patches would be welcomed, but the Bootstrap Core Team doesn't have the time/bandwidth to write such a patch ourselves.


Using SAHI with yolov5 in large image slicing

Hi @kkarahainko I think it would be very useful and awesome if we can port the SAHI library https://github.com/obss/sahi to do image slicing for large images before detection which will make the this library much more useful.
Do you think or have an idea where we can use similar functionality in this library here.
I do appreciate any input you can provide.
Thanks.

@kkarahainko Please see below two images (one is a cropped from the big one or let's say sliced):

and the big one is:

Model .pt file download location:
[https://we.tl/t-g4eayUYZTY]

@kkarahainko while I check what you have said above now I am attempting to use a large Yolo Model (yolov5l). When exported to ONNX it does not detect anything as it was before. I am sure there some parameters that need to be changed in the MODEL file in c#. Can you please point me on how to do that.
Thanks.

@alijundi nop, it should work as yolov5s, the same model config.

@kkarahainko Thank you. it is working as yolov5s -- it seems that I copied the wrong file!
I will also try P6 models once I complete a good training on this model


Care empty file

To fix #90

[x] Add a replication test
[x] Make test pass

Coverage increased (+0.01%) when pulling ea3e43944868e0bcf5b8c497b3503981fb407191 on care-empty into befbc4097e568af906844e3814f34b39fc7eeff9 on master.


fix: #897 frontend minor style changes

Fix #897:

update fontsize in side nav, and change the unit from px to rem
remove the border radius for the speific add user permission button
fix user data table background height problem
update to use the latest nr-theme package
remove unused class name

Style changes for button on focus is in the shared stylesheet

Hi Catherine, thanks for resolving these minor adjustment we discussed last time.
Maybe this one can adjust a little more (not sure if it is my local issue):

Hi Catherine, thanks for resolving these minor adjustment we discussed last time. Maybe this one can adjust a little more (not sure if it is my local issue):
May also reduce table margin to the top of background to be equal. Thanks.:

Hi Ian, the input height fix is done in the shared stylesheet, will publish a new version and import again later. The table margin is like that in the design, but I saw next version the margin will be reduced when we add tap, maybe we can change when work on the next version.

Hi Catherine, thanks for resolving these minor adjustment we discussed last time. Maybe this one can adjust a little more (not sure if it is my local issue):
May also reduce table margin to the top of background to be equal. Thanks.:

I see, I am confused what style in where but other than that sounds good. Thanks.

We still left some minor style changes (button on focus effect, input height) in the common theme repo, but those are really tiny ones, we could wait to release the new theme version later.


Missing make in centos/devtooset-7-toolchain-centos7

$ docker run -ti --rm -v $PWD:/opt/app-root/src:z centos/devtoolset-7-toolchain-centos7 make
/usr/bin/container-entrypoint: line 5: exec: make: not found

I have the same problem. The docs uses it as an example, but it's actually missing...

File a PR and add make here https://github.com/sclorg/devtoolset-container/blob/master/7-toolchain/Dockerfile#L24


"You must bind Parsley on an existing element." on a blank page ?

Hi,
I'm upgrading parsley to version 2.
I have a blank page that load a jQuery and parsley. After that load, I got a warning in console: "You must bind Parsley on an existing element."
Why I'm getting that warning?
Here is a fiddle: http://jsfiddle.net/xtSK5/

@peef is right. still there when using js init.

This will happen when calling parsley() on an empty jQuery set.
What should it do in that circumstance?

This error message is hard-coded at two distinct places:

src/parsley/main.js line 50.  I don't see the relevance in emitting a warning when operating on a valid jQuery set that happens to be empty, which is a very common situation.

src/parsley/factory.js line 30 is internally involed by the above.  Clearly if that point is reached, issuing a warning does make sense.

I'll try a PR.


End background task

Excuse me, where can I end the background task? Because I can cancel the task I am inquiring, but I can't finish the query. When the window is closed, the background task is still "requested to cancel" and keeps turning;

Looks like a bug. To reproduce (7.0.4):

Open Background Tasks
Open SQL Editor  for Postgresql
execute select pg_sleep(10);
Close  SQL Editor.

Expected: the query is interrupted in Background Tasks
Actual: continuation of query execution is shown
Note: Pressing Cancel operation btn doesn't help  in Background Tasks if SQL Editor window with the query was closed before.

Can't reproduce.
Is see following dialog:

Parhaps you checked "Don't ask again" and clicked No. You can check this in Preferences->User Interface->Confirmations.

I cannot see this popup. I set always in Confirmations of User interface. The issue still can be reproduced. Please see

In fact problem was caused by incorrect active job cancel after editor close.
Fixed.

verified

In fact problem was caused by incorrect active job cancel after editor close. Fixed.

Cant we cancel a job without closing the editor.?


Decide whether we need to use badger for Relayer also

After implementing this https://github.com/celestiaorg/orchestrator-relayer/issues/91, the orchestrator now uses badger as a store for the P2P network. However, we did not add it to the relayer because the relayer will be listening on the network and submitting the signatures it finds to an EVM chain.
We need to think whether we need the relayer also to store the received signatures on disk? or no need.

@evan-forbes What do you think about this? Does the relayer need to store the signatures it gets on disk? or just keep using the in-memory store ?
Pros would be that the relayer will have all the signatures locally and will be able to provide them offline if needed.
For the orchestrators, they will only have their signatures, not others. So, we can see the relayers having all the signatures.


create_view should create DRF Request not Django HttpRequest

See: https://github.com/cancan101/django-rest-swagger/commit/d80f426179d0a3b0eefcec819efa44309ac3cd3d

Yep. This is breaking in a fairly major way for me.
I'm getting a failure accessing either request.QUERY_PARAMS or request.query_params so the introspection phase fails.

:+1:  This bug is really annoying and it seems to be easily fixed with @cancan101's proposal.
Would you consider merging this to master ?

If you submit a pr, sure


detect impact location of bullet

I would to get the impact location of the bullets (when it hit's the walls etc.).
As far as I have seen something like that does not exist yet.
Would this be the right location to implement?
https://github.com/EHVAG/demoinfo/blob/master/DemoInfo/DP/Handler/GameEventHandler.cs#L120

It would be the right location, if the data was saved in demofiles.
However, it isn't saved, so there is no way to do this properly.


audit: resolve IF-INTERLAY2-11 discrepancies

Signed-off-by: Gregory Hill<EMAIL_ADDRESS>

Could you link the issue in the PR?


Update related_website_sets

Could you please rebase your PR? The commit history currently includes a lot of changes that seem unrelated to your set, which is likely being caused by a need to rebase.


Update youtube videos and sitemap

Change the PR to be agains George's PR and append the list id to each of the videos
&list=UULF8cj8_eJROxAXsOjhbvduLw


[BUG] The data in 'view data' is wrong when setting filters

Describe the bug
I set a range filter and render a bar graph, however the data in 'view data' is wrong.
To Reproduce
Steps to reproduce the behavior:

set a range filter
render a bar graph
click 'View Data' on a bar
check the data, There will be some data out of the filter

Expected behavior
The data in 'view data' should be in range of the filter.
Screenshots

Versions

pygwalker version: 0.4.6
python version: 3.8
browser: chrome latest

Additional context

Thanks for your feedback.
We'll fix it within 7 days.

fixed it.
pip install --upgrade pygwalker


(test) testing CI for operator

Testing the CI run for operator.

We expect the CI to fail in all these tests:
Test 1: Panic on main function
Local behavior: Operator crashes completely
CI behavior: Checks failed 53c113a
Test 2: Panic on sp1 verification function
Local behavior: Operator crashes completely
CI behavior: Checks failed e68cf10
Test 3: Proof data alteration
Local behavior: Verification error is logged.
CI behavior: Checks failed e7e0b0d


Chronograf display Kapacitor tickscript status

Chronograf is advertised as "the UI for kapacitor" but there is no way to view the currently 'active' alerts that have been triggered by defined tickscripts in kapacitor. There is no reason an alert rule should have to be created for an already defined tickscript just to see what it's state is.
Example:
If i defined a tickscript in kapacitor (or even create a new one in chronograf) and set it's parameters to fire off in a state I cannot view that alert state in any way in chronograf. This seems to be overlooked and the only kind of alert that can be viewed is a alert rule.

Wrong repository, although thinking about it. The documents say you can visualize alerts from kapacitor, which is currently only the case if you define a alert rule. (which is chronograf specific)


npm run watch in Hybrid mode not working

Following the documentation 2-HybridDev.md :

npm run watch

causes :

Hi @gekko68,
Please have a look at this docu: https://cap.cloud.sap/docs/node.js/typescript#cds-tsx
Best


No Shard marks explained

Hi, - Very nice tool!!
Thanks a lot.
But please can you add some explanations to Shard marks?
Green and Red
Also what does Relative% column means?

done! will be in next update 😄

fixed in 1.3.2


Make sabers more visible during gameplay

Hello,
I, like many people, have recently made the jump from Camera Plus to Camera2. I'm really enjoying it so far due to its great performance and ability to change common settings in-game. However, there is one thing that's really bugging me about Camera2, and that is the fact that sabers are much harder to see in gameplay when compared to Camera Plus. I'm not sure why this is (maybe it's the FOV?), but when comparing stock Camera Plus with stock Camera2, you can't see nearly as much of the bottom of the saber especially when using Camera2. Is there any settings you'd recommend I tweak, and, if so, is there any way you can make this the new stock setting so those who are transitioning over from Camera Plus have an easier time?
Thank you in advance!

I dont see how this would be possible - Both dont have a Z-offset by default (You could set a Z-Offset of -0.3 to -0.5 which is what a lot of people do) and Camera2 even has a higher FOV by default which should cause more stuff to be visible rather than less. You're also the first one to bring this up which makes me extra confused.
Do you think you could maybe show me some screenshots showcasing your issue?

I don't have any screenshots, but I have 2 videos on my YouTube channel showing this behavior. Notice that in the Camera2 recording, the sabers feel much closer than in the Camera Plus recording, leading the especially the bottom of the sabers being cut off. When the action picks up, sometimes you can only see the saber trails in the Camera2 recording.
Camera Plus recording
Camera2 recording
Thanks again for your help!

I would suggest you to place your headset and one controller in a stationary position and use the other controller to go into a song, you can easily compare cam2 / cam+ this way. Camera2 and CameraPlus for that matter are really just fancy configuration interfaces of Unity's built in cameras which handle the entire rending - If you use the same settings in both it should pretty much look the same, and I dont really see an obvious difference in both of your linked recordings to be honest.

Ok, I'll play around with the Z-offset values and FOV like you suggested in your original comment, and I'll get back to you with my results. I recently had to RMA one of my controllers, so it might be a little while before I can get around to testing this again.
Thanks again for your help!

Closing as I guess its not an issue any more.


Bank OVERVIEW UI update

We are giving more content hierarchy to the way the Banks OVERVIEW information is read. Please view the mocks and include the blocks listed.
Blocks listed

Tx Fee Block (independent block)
Confirmation Services (independent block)
IP Address, Port, Protocol, Version, Node Type
Bank Network ID & Bank Account Number (Include secondary button 'Add Signing Keys (For DEVOPS)

https://www.figma.com/file/ObSUx1rum1MIeZkFT3XIn1/Bank-User?node-id=837%3A4160

Moved to https://github.com/thenewboston-developers/Account-Manager/issues/354


Bug with Russian number

When number is 8 (964) 534-35-44 and i delete last symbol it becomes 8 (896) 453-43-54 One more 8 added, and it's really strange

same issue +1

+1

+1`

It because of replaced awesome-phonenumber lib to libphonenumber-js
So you can use some workaround
valueMutator: ({ ref, inputVal }) => {
        if (!ref.phoneObject.valid) {
          return
        }
        const result = new PhoneNumber(inputVal, ref.phoneObject.countryCode)
        if (!result.isValid()) {
          data.value.user.phone.local = inputVal
          return
        }

        data.value.user.phone.international = result.getNumber('international')

        return result.getNumber('national')
}


Add tests to Kitura TestResponse and TestRouteRegex

PR https://github.com/IBM-Swift/Kitura/pull/753 was merged.


Antivirus relies on connection close to get ICAP result

Describe the bug
ICAP supports reusing a connection. When ICAP is not explicitly configured to close the connection (= send FIN) after a scan, the antivirus service hangs.
Steps to reproduce
Steps to reproduce the behavior:

Have a oCIS with antivirus service
Have a ClamAV which does not close the connection after a scan
Upload a file

Expected behavior
The virusscan is finished as soon as the ICAP response is received.
Actual behavior
The virusscan takes as long as the connection is not interrupted, timed out or closed. (= eg. receives FIN). In the case of the wireshark dump below with connection reuse and keepalive it takes until 47 / 300 seconds for the antivirus service to actually use the result from 6 / 0.002026783 seconds.
Wireshark dumps
ICAP with disable connection reuse
1	0.000000000	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	57616 → 1344 [SYN] Seq=0 Win=64240 Len=0 MSS=1460 SACK_PERM TSval=442762913 TSecr=0 WS=128
2	0.000071674	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	1344 → 57616 [SYN, ACK] Seq=0 Ack=1 Win=65160 Len=0 MSS=1460 SACK_PERM TSval=3142759142 TSecr=442762913 WS=128
3	0.000104416	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	57616 → 1344 [ACK] Seq=1 Ack=1 Win=64256 Len=0 TSval=442762913 TSecr=3142759142
4	0.000725205	<IP_ADDRESS>	<IP_ADDRESS>	ICAP	329	REQMOD icap://localhost:1344/avscan ICAP/1.0
5	0.000793593	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	1344 → 57616 [ACK] Seq=1 Ack=264 Win=65024 Len=0 TSval=3142759143 TSecr=442762914
6	0.002201844	<IP_ADDRESS>	<IP_ADDRESS>	ICAP	179	ICAP/1.0 204 Unmodified
7	0.002269051	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	57616 → 1344 [ACK] Seq=264 Ack=114 Win=64256 Len=0 TSval=442762915 TSecr=3142759144
8	0.002631282	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	1344 → 57616 [FIN, ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142759145 TSecr=442762915
9	0.002995268	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	57616 → 1344 [FIN, ACK] Seq=264 Ack=115 Win=64256 Len=0 TSval=442762916 TSecr=3142759145
10	0.003055210	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	1344 → 57616 [ACK] Seq=115 Ack=265 Win=65024 Len=0 TSval=3142759145 TSecr=442762916

ICAP with connection reuse and keepalive
1	0.000000000	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	49534 → 1344 [SYN] Seq=0 Win=64240 Len=0 MSS=1460 SACK_PERM TSval=442349969 TSecr=0 WS=128
2	0.000071965	<IP_ADDRESS>	<IP_ADDRESS>	TCP	74	1344 → 49534 [SYN, ACK] Seq=0 Ack=1 Win=65160 Len=0 MSS=1460 SACK_PERM TSval=3142346199 TSecr=442349969 WS=128
3	0.000098485	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	49534 → 1344 [ACK] Seq=1 Ack=1 Win=64256 Len=0 TSval=442349970 TSecr=3142346199
4	0.000697672	<IP_ADDRESS>	<IP_ADDRESS>	ICAP	329	REQMOD icap://localhost:1344/avscan ICAP/1.0
5	0.000732788	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	1344 → 49534 [ACK] Seq=1 Ack=264 Win=65024 Len=0 TSval=3142346199 TSecr=442349970
6	0.002026783	<IP_ADDRESS>	<IP_ADDRESS>	ICAP	179	ICAP/1.0 204 Unmodified
7	0.002056869	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	49534 → 1344 [ACK] Seq=264 Ack=114 Win=64256 Len=0 TSval=442349972 TSecr=3142346200
8	15.700939197	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442365670 TSecr=3142346200
9	15.700991416	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142361899 TSecr=442349972
10	31.060941244	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442381030 TSecr=3142361899
11	31.061009903	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142377259 TSecr=442349972
12	46.421110326	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442396391 TSecr=3142377259
13	46.421374874	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142392620 TSecr=442349972
14	61.781359678	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442411751 TSecr=3142392620
15	61.781413920	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142407980 TSecr=442349972
16	77.141104139	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442427111 TSecr=3142407980
17	77.141187075	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142423340 TSecr=442349972
18	92.501252805	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442442471 TSecr=3142423340
19	92.501342394	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142438700 TSecr=442349972
20<PHONE_NUMBER>91	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442457830 TSecr=3142438700
21<PHONE_NUMBER>49	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142454059 TSecr=442349972
22<PHONE_NUMBER>27	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442473191 TSecr=3142454059
23<PHONE_NUMBER>61	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142469420 TSecr=442349972
24<PHONE_NUMBER>58	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442488551 TSecr=3142469420
25<PHONE_NUMBER>02	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142484780 TSecr=442349972
26<PHONE_NUMBER>41	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442503911 TSecr=3142484780
27<PHONE_NUMBER>04	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142500140 TSecr=442349972
28<PHONE_NUMBER>62	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442519270 TSecr=3142500140
29<PHONE_NUMBER>15	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142515499 TSecr=442349972
30<PHONE_NUMBER>36	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442534631 TSecr=3142515499
31<PHONE_NUMBER>66	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142530861 TSecr=442349972
32<PHONE_NUMBER>31	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442549991 TSecr=3142530861
33<PHONE_NUMBER>04	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142546220 TSecr=442349972
34<PHONE_NUMBER>28	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442565351 TSecr=3142546220
35<PHONE_NUMBER>87	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142561580 TSecr=442349972
36<PHONE_NUMBER>06	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442580711 TSecr=3142561580
37<PHONE_NUMBER>54	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142576940 TSecr=442349972
38<PHONE_NUMBER>69	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442596070 TSecr=3142576940
39<PHONE_NUMBER>44	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142592299 TSecr=442349972
40<PHONE_NUMBER>95	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442611434 TSecr=3142592299
41<PHONE_NUMBER>36	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142607663 TSecr=442349972
42<PHONE_NUMBER>59	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442626790 TSecr=3142607663
43<PHONE_NUMBER>21	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142623020 TSecr=442349972
44<PHONE_NUMBER>66	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive] 49534 → 1344 [ACK] Seq=263 Ack=114 Win=64256 Len=0 TSval=442642150 TSecr=3142623020
45<PHONE_NUMBER>62	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	[TCP Keep-Alive ACK] 1344 → 49534 [ACK] Seq=114 Ack=264 Win=65024 Len=0 TSval=3142638379 TSecr=442349972
46<PHONE_NUMBER>12	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	49534 → 1344 [FIN, ACK] Seq=264 Ack=114 Win=64256 Len=0 TSval=442649970 TSecr=3142638379
47<PHONE_NUMBER>04	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	1344 → 49534 [FIN, ACK] Seq=114 Ack=265 Win=65024 Len=0 TSval=3142646200 TSecr=442649970
48<PHONE_NUMBER>60	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	49534 → 1344 [ACK] Seq=265 Ack=115 Win=64256 Len=0 TSval=442649971 TSecr=3142646200

@kobergj Can you provide more context?

@wkloucek How important is this in regards of Release 3.1.0?

@wkloucek How important is this in regards of Release 3.1.0?

Not important at all

Can you provide more context?

This is an issue of the icap lib we are using (https://github.com/egirna/icap-client) It is rather unmaintained (last commit 2020) and we planned to replace it with an alternative. However back then there were no alternatives. I see two fixes for this issue, but both are more a story than a bugfix.
A) Find an alternative for the used icap-lib and use this.
B) Write an icap lib by ourselves (we only need to implement the features we really need) and use this.

The used icap Library is very "basic" and unmaintained.
See the proposal from @kobergj
Does anybody know a better library?
I would like to avoid developing an ICAP golang library on our own.

Don't know if it matters, but the ICAP client in here: https://github.com/egirna/icapeg/tree/master/icap-client seems to have some changes newer than the stuff here: https://github.com/egirna/icap-client.

Might wrapping a c library be an option? If yes this library is also used by many distributions: https://github.com/c-icap/c-icap-server. It also features a CLI: https://c-icap.sourceforge.net/install.html (which I used to test my ICAP server, for the record /usr/bin/c-icap-client -f /root/eicar.com -s "srv_clamav?allow204=on&force=on&sizelimit=off&mode=simple" -i <IP_ADDRESS>)

Might wrapping a c library be an option?

From my little experience, this should be the last option, if any other option has failed.
There are multiple drawbacks:

Safety features from the Go language won't be available in the wrapper. We need to be extremely careful with the wrapper code
Debugging code will be very complex. I don't think we'll be able to debug anything inside the wrapped library.
Some features, such as callbacks, might be difficult to make in the wrapper.
Need to pay special attention on memory management (variables "malloc"ed from C code must be manually freed), and transferring data (C types vs Go types)
The library could have additional limitations (thread-safety) which might condition the development of the wrapper or anything on top of it.

Due to these drawbacks, wrappers are usually an extremely thin interface to call C methods in order to reduce the "problematic" surface. Any complexity is managed by creating a class in Go that will use the simple methods of the wrapper to achieve its goal.
Note that the user might still need to have installed the icap shared library in his system, which might be another problem. I haven't look into it too much, but there should be an option to compile the library code statically to include the library code into the binary.

From what I remember:
KeepAliveTimeout 0

should force a connection close after the request.
So if you have KeepAliveTimeout > 0 or even -1 you'll get a late / no scan result


typer-web to run typer cli as a web server (like click-web did)

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Typer documentation, with the integrated search.
[X] I already searched in Google "How to X in Typer" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to Typer but to Click.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import typer
from typer_web import create_fastapi_web_app

app = typer.Typer()

@create_fastapi_web_app
@app.command
def main(name: str):
    print(f"Hello {name}")

if __name__ == "__main__":
    app()

Description
Is it possible to create a typer-web module, which supports both the typer cli mode and the fastapi server mode? Click-web project https://github.com/fredrik-corneliusson/click-web can be used to generate a web app for click cli in a very simple way.  It'd be great if typer can provide such a tool that supports both cli mode and web-server mode transparently.
Wanted Solution
As the description.
Wanted Code
NA

Alternatives
No response
Operating System
Linux
Operating System Details
No response
Typer Version
0.6.1
Python Version
3.7
Additional Context
No response

This would be a great feature to have, or alternatively some official guidance on how to use typer and fastapi at the same time (they are siblings, but do they get along?)

This is what Hug does, which the FastAPI documentation mentions. Hug supports using the same function for both a Web API and a CLI (the following is from their docs):
"""An example of writing an API to scrape hacker news once, and then enabling usage everywhere"""
import hug
import requests

@hug.local()
@hug.cli()
@hug.get()
def top_post(section: hug.types.one_of(('news', 'newest', 'show'))='news'):
    """Returns the top post from the provided section"""
    content = requests.get('https://news.ycombinator.com/{0}'.format(section)).content
    text = content.decode('utf-8')
    return text.split('<tr class=\'athing\'>')[1].split("<a href")[1].split(">")[1].split("<")[0]

So, the question is really, can you do this with Typer and FastAPI? I think yes:
from typer import Typer
from fastapi import FastAPI

cli_app = Typer()
webapp = FastAPI()

@webapp.get("/hello/{name}")
@cli_app.command()
def hello(name: str):
    print(f"Hello {name}")

@webapp.get("/goodbye/{name}")
@cli_app.command()
def goodbye(name: str, formal: bool = False):
    if formal:
        print(f"Goodbye Ms. {name}. Have a good day.")
    else:
        print(f"Bye {name}!")

if __name__ == "__main__":
    cli_app()

There are two ways to use this file, which let's say is called hybrid.py:

If the file above is run with python -m uvicorn hybrid:webapp, running this works as a FastAPI application.
If the file above is run with python hybrid.py hello --help, it works as a Typer application and we get:

 Usage: hybrid.py hello [OPTIONS] NAME

╭─ Arguments ────────────────────────────────────────────────────────────────────────────────╮
│ *    name      TEXT  [default: None] [required]                                            │
╰────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Options ──────────────────────────────────────────────────────────────────────────────────╮
│ --help          Show this message and exit.                                                │
╰────────────────────────────────────────────────────────────────────────────────────────────╯

Since the type annotations contain most of the business logic, not the decorators, even a pretty complicated Typer or FastAPI configuration should work with minimal repeated logic; a few exceptions of course, like if both the FastAPI endpoint and Typer command had to be deprecated, that would have be be specified in both decorators.
To ensure there is really a single source of truth for every shared behavior of these two interfaces, someone would need to write a single class that handles both.
The single class - HybridAPI, say - would need to follow the ASGI spec (Uvicorn docs), which means its __call__ method would have to have the signature __call__(self, scope, receive, send):.... This rules out the possibility of using the documented Typer API if __name__=="__main__": app().
Minimal working example of getting around that issue:
from typer import Typer
from fastapi import FastAPI

class HybridAPI(FastAPI):

    def __init__(self, *args, typer_args=[], typer_kwargs={}, **kwargs):
        self.cli_app = Typer(*typer_args, **typer_kwargs)
        super().__init__(*args, **kwargs)

    def get(self, *args, typer_args=[], typer_kwargs={}, **kwargs):
        fastAPIGet = super().get(*args, **kwargs)
        cliCommand = self.cli_app.command(*typer_args, **typer_kwargs)
        return lambda func: fastAPIGet(cliCommand(func))

app = HybridAPI()

@app.get("/hello/{name}")
def hello(name: str):
    print(f"Hello {name}")

@app.get("/goodbye/{name}")
def goodbye(name: str, formal: bool = False):
    if formal:
        print(f"Goodbye Ms. {name}. Have a good day.")
    else:
        print(f"Bye {name}!")

if __name__ == "__main__":
    app.cli_app()

All Typer and FastAPI functionality works. Of course, this only implements get (omitting post, put, etc.), and the Typer.__call__ (or any other Typer method) must be called via the cli_app field. Sharing positional or keyword arguments for the two applications would have to be implemented in the Frankenstein decorator functions.
To be clear I think this is a bad idea, but it was interesting to think about! Using two decorators, with some repeated information between them, is a small price to pay for defining business logic once.

Typer- and FastAPI-specific type annotations should work just fine:
import typer
import fastapi
...
@app.get('/test')
def test(
    boolparam: Annotated[bool, typer.Option(prompt="Are you sure?"), fastapi.Body()],
):
    ...

For all of the people that stumbles on this issue.
Nothing new needs to be implemented! You can in fact use click-web.
Take your Typer() instance and to the following
commands.py
import typer

typer_app = typer.Typer()

main.py
from click_web import create_click_web_app
import commands
import typer

typer_app = typer.Typer()

cmd = typer.main.get_command(commands.typer_app)
cmd.name = "cli"
app = create_click_web_app(commands, cmd)

start.sh
export FLASK_ENV=development
export FLASK_APP=app.py
flask run


x-transformers: init at 1.32.2

Description of changes
A concise but fully-featured transformer, complete with a set of promising experimental features from various papers.
Metadata

homepage URL: https://github.com/lucidrains/x-transformers
source URL: https://github.com/lucidrains/x-transformers
license: mit

Closes https://github.com/NixOS/nixpkgs/issues/332704

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[X] sandbox = true

[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Since this package is a python module, please start the commit message with python312Packages.x-transformers.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 332726

x86_64-linux

:white_check_mark: 2 packages built:

python311Packages.x-transformers
python312Packages.x-transformers

aarch64-linux

:white_check_mark: 2 packages built:

python311Packages.x-transformers
python312Packages.x-transformers

x86_64-darwin

:white_check_mark: 2 packages built:

python311Packages.x-transformers
python312Packages.x-transformers

aarch64-darwin

:white_check_mark: 2 packages built:

python311Packages.x-transformers
python312Packages.x-transformers


feat(dev/SprintGraph) : made sprint progress graph

Functionality:
Sprint progress graph
Solution:
The progress width increases with increase in time
Risk level:

[ ] high
[x] medium
[ ] low

How to test:
ng serve
firebase emulators:start

I need demo on this.


Create [POST-SCAN SUMMARY] output in CLI

Probably it'll be better to improve report's general information content


MVP Workshop

I think we should have a hackathon or workshop to brainstorm our first 1-3 course-paths.
Originally posted by @mrmetaverse in https://github.com/AngellXR/metaversedev/discussions/9#discussioncomment-2503792

I think we have identified some technology to set up in a hackathon, which I am happy to take responsibility for deploying the blog engine. Maybe we could get some other folks to participate and create content, design, style, contribute code or whatever and try to hackathon a launch plan together.

@lalalune I made a little post on Twitter a short while ago, spread the word?
https://twitter.com/mrmetaverse/status/1548322673993740290?s=20&t=VsvQByztT3WV2rEfIwDuvw

Archiving to focus on #22 and #21


Support Deno

Clear and concise description of the problem
Deno has higher performance.
Suggested solution
https://deno.land/
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Duplicate of #109

things have changed. deno has a node compat mode: https://medium.com/deno-the-complete-reference/node-js-compatibility-in-deno-a7cb8384a8d7
maybe it can be reconsidered?

@aladdin-add check this issue https://github.com/denoland/deno/issues/12577
vite support is on their todo

things have changed. deno has a node compat mode: https://medium.com/deno-the-complete-reference/node-js-compatibility-in-deno-a7cb8384a8d7
maybe it can be reconsidered?

thanks


bug: ExtendedAPIPlugin incorrectly causes 'cannot use [chunkhash] error'

Do you want to request a feature or report a bug?
Bug.
What is the current behavior?
Usage of [chunkhash] for both output.filename & output.chunkFilename along with ExtendedAPIPlugin causes incorrect compilation error.
If the current behavior is a bug, please provide the steps to reproduce.
Demo: https://github.com/hulkish/webpack-ExtendedAPIPlugin-bug

In your webpack config, set your output to:

output: {
  filename: '[name].[chunkhash].js',
  chunkFilename: '[id].[chunkhash].js'
}

Add ExtendedAPIPlugin:

plugins: [new ExtendedAPIPlugin()]

Run the build & get this error:

ERROR in chunk main [entry]
[name].[chunkhash:5].js
Cannot use [chunkhash] for chunk in '[name].[chunkhash].js' (use [hash] instead)

What is the expected behavior?
Should not cause this error.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Quick update - after a little debugging I found it's coming from this line

@sokra Do you think removing that line would break something else? I could send a PR.

Is there any workaround for this? Any 3rd party libs?

@gabrielhpugliese  this seems to be the only work around at the moment that im aware of. I'm unsure of what impact commenting out that line will have, though.

This should have a solution :)
I don't want to have this gets closed

Can anyone have a look at this please? This stops from using cache busting and chunkhash properly. @sokra @ooflorent

There is a PR to improve the error message: #6942
However, I don't understand the limitation.

bug analyze
Compare with APIPlugin.js and ExtendedAPIPlugin.js, we will find this line:
ExtendedAPIPlugin.js#L35
compilation.mainTemplate.plugin("global-hash", () => true);

so, useChunkHash fn always return false
MainTemplate.js#L229
useChunkHash(chunk) {
	const paths = this.applyPluginsWaterfall("global-hash-paths", []);
	return !this.applyPluginsBailResult("global-hash", chunk, paths); // always return `false`
}

then noChunkHash always true
Compilation.js#L1266
file = this.getPath(filenameTemplate, {
	noChunkHash: !useChunkHash,
	chunk
});

then asset-path plugin will throw the error
Cannot use [chunkhash] for chunk in '${path}' (use [hash] instead)

TemplatedPathPlugin.js#L63
// replacePathVariables
if(data.noChunkHash && REGEXP_CHUNKHASH_FOR_TEST.test(path)) {
	throw new Error(`Cannot use [chunkhash] for chunk in '${path}' (use [hash] instead)`);
}

by commenting out the following line in the ExtendedAPIPlugin.js#L35, it works fine.
hack fix:

create ExtendedAPIPlugin.js

/*
	MIT License http://www.opensource.org/licenses/mit-license.php
	Author Tobias Koppers @sokra
*/
"use strict";

const ConstDependency = require("webpack/lib/dependencies/ConstDependency");
const ParserHelpers = require("webpack/lib/ParserHelpers");
const NullFactory = require("webpack/lib/NullFactory");

const REPLACEMENTS = {
	__webpack_hash__: "__webpack_require__.h", // eslint-disable-line camelcase
	__webpack_chunkname__: "__webpack_require__.cn" // eslint-disable-line camelcase
};
const REPLACEMENT_TYPES = {
	__webpack_hash__: "string", // eslint-disable-line camelcase
	__webpack_chunkname__: "string" // eslint-disable-line camelcase
};

class ExtendedAPIPlugin {
	apply(compiler) {
		compiler.plugin("compilation", (compilation, params) => {
			compilation.dependencyFactories.set(ConstDependency, new NullFactory());
			compilation.dependencyTemplates.set(ConstDependency, new ConstDependency.Template());
			compilation.mainTemplate.plugin("require-extensions", function(source, chunk, hash) {
				const buf = [source];
				buf.push("");
				buf.push("// __webpack_hash__");
				buf.push(`${this.requireFn}.h = ${JSON.stringify(hash)};`);
				buf.push("");
				buf.push("// __webpack_chunkname__");
				buf.push(`${this.requireFn}.cn = ${JSON.stringify(chunk.name)};`);
				return this.asString(buf);
			});
			// compilation.mainTemplate.plugin("global-hash", () => true);

			params.normalModuleFactory.plugin("parser", (parser, parserOptions) => {
				Object.keys(REPLACEMENTS).forEach(key => {
					parser.plugin(`expression ${key}`, ParserHelpers.toConstantDependency(REPLACEMENTS[key]));
					parser.plugin(`evaluate typeof ${key}`, ParserHelpers.evaluateToString(REPLACEMENT_TYPES[key]));
				});
			});
		});
	}
}

module.exports = ExtendedAPIPlugin;

import the custom plugin in webpack.config.prod.js

const ExtendedAPIPlugin = require('./ExtendedAPIPlugin');
// ...
plugins: [
    // ...
    new ExtendedAPIPlugin()
]

bug analyze
Compare with APIPlugin.js and ExtendedAPIPlugin.js, we will find this line:
ExtendedAPIPlugin.js#L35
compilation.mainTemplate.plugin("global-hash", () => true);

so, useChunkHash fn always return false
MainTemplate.js#L229
useChunkHash(chunk) {
	const paths = this.applyPluginsWaterfall("global-hash-paths", []);
	return !this.applyPluginsBailResult("global-hash", chunk, paths); // always return `false`
}

then noChunkHash always true
Compilation.js#L1266
file = this.getPath(filenameTemplate, {
	noChunkHash: !useChunkHash,
	chunk
});

then asset-path plugin will throw the error
Cannot use [chunkhash] for chunk in '${path}' (use [hash] instead)

TemplatedPathPlugin.js#L63
// replacePathVariables
if(data.noChunkHash && REGEXP_CHUNKHASH_FOR_TEST.test(path)) {
	throw new Error(`Cannot use [chunkhash] for chunk in '${path}' (use [hash] instead)`);
}

by commenting out the following line in the ExtendedAPIPlugin.js#L35, it works fine.
hack fix:

create ExtendedAPIPlugin.js

/*
	MIT License http://www.opensource.org/licenses/mit-license.php
	Author Tobias Koppers @sokra
*/
"use strict";

const ConstDependency = require("webpack/lib/dependencies/ConstDependency");
const ParserHelpers = require("webpack/lib/ParserHelpers");
const NullFactory = require("webpack/lib/NullFactory");

const REPLACEMENTS = {
	__webpack_hash__: "__webpack_require__.h", // eslint-disable-line camelcase
	__webpack_chunkname__: "__webpack_require__.cn" // eslint-disable-line camelcase
};
const REPLACEMENT_TYPES = {
	__webpack_hash__: "string", // eslint-disable-line camelcase
	__webpack_chunkname__: "string" // eslint-disable-line camelcase
};

class ExtendedAPIPlugin {
	apply(compiler) {
		compiler.plugin("compilation", (compilation, params) => {
			compilation.dependencyFactories.set(ConstDependency, new NullFactory());
			compilation.dependencyTemplates.set(ConstDependency, new ConstDependency.Template());
			compilation.mainTemplate.plugin("require-extensions", function(source, chunk, hash) {
				const buf = [source];
				buf.push("");
				buf.push("// __webpack_hash__");
				buf.push(`${this.requireFn}.h = ${JSON.stringify(hash)};`);
				buf.push("");
				buf.push("// __webpack_chunkname__");
				buf.push(`${this.requireFn}.cn = ${JSON.stringify(chunk.name)};`);
				return this.asString(buf);
			});
			// compilation.mainTemplate.plugin("global-hash", () => true);

			params.normalModuleFactory.plugin("parser", (parser, parserOptions) => {
				Object.keys(REPLACEMENTS).forEach(key => {
					parser.plugin(`expression ${key}`, ParserHelpers.toConstantDependency(REPLACEMENTS[key]));
					parser.plugin(`evaluate typeof ${key}`, ParserHelpers.evaluateToString(REPLACEMENT_TYPES[key]));
				});
			});
		});
	}
}

module.exports = ExtendedAPIPlugin;

import the custom plugin in webpack.config.prod.js

const ExtendedAPIPlugin = require('./ExtendedAPIPlugin'); // your file path
// ...
plugins: [
    // ...
    new ExtendedAPIPlugin()
]

@ICELI Sounds like you might have an actual fix, why not make pr?

Is there any resolution for this? Looks like #7987 got closed.. 🙁

If I'm understanding this correctly:

The ExtendedAPIPlugin adds __webpack_hash__ to the runtime[1].
Because the global hash is included in the runtime, the chunkhash of the runtime needs to change with any change to the compilation.
I'm assuming that the chunkhash of the runtime is calculated on source before the global hash is included, so it's not changing (?).
It's not possible to set a different filename/chunkFilename to be used by the runtime only, which would have allowed us to just use hash there.

[1] There could be multiple runtimes.

https://github.com/webpack/webpack.js.org/pull/3185

bump


Some API functions are not part of the swagger file

The following calls are not part of the swagger file.
/system/info
/system/health
/v2/api-docs
Is there any reason for that?

Those endpoints are meta endpoints and are not part of the actual API:

/system/info, /system/healh are from spring-actuator, and are meant for monitoring.
/v2/api-docs is where swagger-ui loads the live swagger.json