Package won't work with React 17

React 17 will deprecate componentWillReceiveProps which this package uses.
In the actual version of React, you get a warning. In react 17 componentWillReceiveProps should be prefixed with UNSAFE_ to work.
I don't know enough of reactlifecycle/blaze/ this package to help with the rewrite to avoid componentWillReceiveProps.

Hey @vjau, nice to see you here and thanks for the early heads up.
From a quick look, it should be enough to move the contents of componentWillReceiveProps() to the start of shouldComponentUpdate().  It will be a little while until I can look at this properly, but at least we have a good idea of what to do.
Funnily enough, componentWillReceiveProps, with all it's "unsafe" patterns in traditional React use, actually was a perfect fit for this use case, with no side effects.  But it should work just as well in shouldComponentUpdate, even if less semantically clear.
What's news from Meteor land?  What kind of projects are you still using this with?

Hi @gadicc, i don't know if you have heard, but MDG has sold Meteor to Tiny. That means the development will be able to go on.
Your package is still the recommended way by the Meteor guide to use the login package in React apps,  since it has not been ported to React.

Hi @gadicc, i don't know if you have heard, but MDG has sold Meteor to Tiny. That means the development will be able to go on.
Your package is still the recommended way by the Meteor guide to use the login package in React apps,  since it has not been ported to React.

Hi @gadicc, i don't know if you have heard, but MDG has sold Meteor to Tiny. That means the development will be able to go on.
Your package is still the recommended way by the Meteor guide to use the login package in React apps,  since it has not been ported to React.

Yes, I heard about the acquisition, hope it will go in a good direction.  And thanks, didn't realise this was still the best way to use the account packages.
I made the change and all tests are still passing the test suite (actually, hardest part was upgrading Meteor and all related packages and test packages :)).  I published as<EMAIL_ADDRESS>I'd be grateful if you could try this in a bigger package and see if everything still works as expected!

Hi @gadicc , sorry for the late answer, and thank you for your reactivity.
I have just tested your fix with my limited use case which is the login gui module, but for this use, it seems to work. Thank you very much !

FWIW, I've tried it in a few places in my app, and it works well. Perhaps worth finally releasing as a non-beta?

FWIW, I've tried it in a few places in my app, and it works well. Perhaps worth finally releasing as a non-beta?

Right you are, @edemaine, thanks for the nudge :)  And thanks again @vjau (belatedly) for your help with this issue!  Released as 1.4.3.

Right you are, @edemaine, thanks for the nudge :)  And thanks again @vjau (belatedly) for your help with this issue!  Released as 1.4.3.


Create IG Route: /accounts/{AccountId}/statements/{StatementId}/transactions

Description
The statements resource is used by an AISP to retrieve the:

statements information
statement information for a StatementId in json and non-json(file) format.
transactions for a selected StatementId
For a specific account identified by AccountId or retrieve statement information for all accounts that the PSU has consented to.

Issues related

https://github.com/SecureBankingAccessToolkit/SecureBankingAccessToolkit/issues/114

Route Specifications

read/write API: Account and Transaction
Specification link: GET /accounts/{AccountId}/statements/{StatementId}/transactions v3.1.8
Endpoint: /accounts/{AccountId}/statements/{StatementId}/transactions
Method: GET
API version spec: Statements v3.1.8
API uri: /rs/open-banking/v3.1.*/aisp/accounts/{AccountId}/statements/{StatementId}/transactions

Acceptance criteria

[x] Functional tests passed in nightly environment
[x] Functional tests passed in dev environment
[x] Proper response expected by the spec
[x] Errors expected match with the OBIE specs
[x] Route documented

[x] Route source commented
[x] Add the route documentation to the wiki route section

Functionality implemented in: /securebanking-openbanking-demo/blob/master/config/7.0/obdemo-bank/ig/routes/routes-service/11-ob-account-access.json


fix: normalize the rawPath without locale before attempting to asserting the 404 page

Thanks! We might want to add i18n to the e2e app to better test these scenarios.

I've updated the examples/pages-routes project to include some i18n config. I looked at the current e2e and they would already cover this bug since they assert the data json calls are resolving.
Would that be sufficient enough?

Thanks for adding the configs. I think it'll be useful to test the actual internationalization of the e2e apps to, that is, to add [lang] folder, eg: https://github.com/vercel/next.js/tree/canary/examples/app-dir-i18n-routing/app/[lang]

Thanks for adding the configs. I think it'll be useful to test the actual internationalization of the e2e apps to, that is, to add [lang] folder, eg: https://github.com/vercel/next.js/tree/canary/examples/app-dir-i18n-routing/app/[lang]

Hmm, looking at the example I'm not sure it would work seeing that the middleware takes care of all the i18n stuff with the app router. This means that I wouldn't know the i18n configuration to start with so my new "code" would never be executed 🤔
TBH not using the app router at all atm, we stick with good old Pages 🤷

It shouldn't be necessary to change anything for app dir, the middleware rewrites should already be applied at this point


Configure Renovate

Welcome to Renovate! This is an onboarding PR to help you understand and configure settings before regular Pull Requests begin.
🚦 To activate Renovate, merge this Pull Request. To disable Renovate, simply close this Pull Request unmerged.

Detected Package Files

providers.tf (terraform)
terraform.tf (terraform)

Configuration Summary
Based on the default config's presets, Renovate will:

Start dependency updates only once this onboarding PR is merged
Enable Renovate Dependency Dashboard creation.
Use semantic commit type fix for dependencies and chore for all others if semantic commits are in use.
Ignore node_modules, bower_components, vendor and various test/tests directories.
Group known monorepo packages together.
Use curated list of recommended non-monorepo package groupings.
Apply crowd-sourced package replacement rules.
Apply crowd-sourced workarounds for known problems with packages.
Group all updates together.

🔡 Do you want to change how Renovate upgrades your dependencies? Add your custom config to renovate.json in this branch. Renovate will update the Pull Request description the next time it runs.

What to Expect
It looks like your repository dependencies are already up-to-date and no Pull Requests will be necessary right away.

❓ Got questions? Check out Renovate's Docs, particularly the Getting Started section.
If you need any further assistance then you can also request help here.

This PR has been generated by Renovate Bot.

Renovate is disabled
Renovate is disabled because there is no Renovate configuration file. To enable Renovate, you can either (a) change this PR's title to get a new onboarding PR, and merge the new onboarding PR, or (b) create a Renovate config file, and commit that file to your base branch.


Can you ad LPG and Pellets to the type of fuel?

Is there a way you can ad LPG and Pellets to the differnet type of fuels available?

LPG would be an option I guess, not sure of which site to get pellets prices

Lpg is fine.Tought carbu could also show pellet prices. But I was wrong. My mistake.Verzonden vanaf mijn Galaxy
-------- Oorspronkelijk bericht --------Van: myT @.> Datum: 7/09/2023  21:27  (GMT+01:00) Aan: myTselection/Carbu_com @.> Cc: TimmieVL @.>, Author @.> Onderwerp: Re: [myTselection/Carbu_com] Can you ad LPG and Pellets to the type of fuel? (Issue #32)
LPG would be an option I guess, not sure of which site to get pellets prices
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

added as of release 7.4.0


Add tests for HTTP API

Add tests for all HTTP API entry points and see they produce valid responses.

Fixed in 3db4f58092f8f0d4369176e40a3e1c4f8eadcafe.


add pull after checking out master

It appears that in sometimes master branch is out of sync.
Like here https://gitlab.parity.io/parity/mirrors/wasmi/-/jobs/1868177#L29
$ git checkout master
Previous HEAD position was 2efb5fe add missing import after merge
Switched to branch 'master'
Your branch is behind 'origin/master' by 2 commits, and can be fast-forwarded.
  (use "git pull" to update your local branch)

This PR fixes that

Thanks @sergejparity for the quick fix! Let's wait for the benchmark results.

Main reason is to deliver trustworthy results. So we can be sure that difference between tests executed against master and pr is not the result of random factors like some another jobs started on the runner at the same time.

sounds reasonable, okay :+1:


Bug 2015459: Enable health check for storage driver

When the registry pod cannot reach its storage, it should report about problems.

/bugzilla refresh

/lgtm

/label qe-approved

@dmage  Hi, will cherry-pick to release 4.9 right?

/lgtm cancel
I want it to be reviewed by somebody from engineering.

@wewang58 most likely we won't backport it

/cancel qe-approved

/qe-approved cancel

/hold

/lint

/bugzilla refresh
/hold cancel

/retest

/assign @ricardomaraschini

/retest


Updated from VGGLoss to LPIPSLoss

There are still errors on module.py
File "/content/drive/MyDrive/SRGAN-PyTorch-master-N-Funcional/train.py", line 589, in
main()
File "/content/drive/MyDrive/SRGAN-PyTorch-master-N-Funcional/train.py", line 153, in main
main_worker(args.gpu, ngpus_per_node, args)
File "/content/drive/MyDrive/SRGAN-PyTorch-master-N-Funcional/train.py", line 391, in main_worker
args=args)
File "/content/drive/MyDrive/SRGAN-PyTorch-master-N-Funcional/train.py", line 543, in train_gan
content_loss = content_criterion(sr, hr.detach())
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "/content/drive/MyDrive/SRGAN-PyTorch-master-N-Funcional/srgan_pytorch/loss.py", line 207, in forward
loss = torch.nn.functional.mse_loss(self.features(source), self.features(target))
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
TypeError: forward() missing 1 required positional argument: 'in1'

I testing....


Infrastructure Cleanup

Description
This card outlines the process for streamlining our current infrastructure to improve efficiency and eliminate any redundant or outdated components. The goal is to ensure our infrastructure is lean, up to date (as much as it safely can be in 2 sprints!), and documented, with all changes properly tested and verified.
Work Items:

[ ] Separate out  all services that can be created locally in localstack so true local testing can take place.

[x] Back office Lambdas
[x] S3 Buckets
[x] Secrets Manager
[x] File Upload Scanning

[ ] Ensure event triggers are automatic between back office lambdas and do not require manual triggering.
[ ] Configure AWS SAM to leverage terraform so development on Lambdas can continue to leverage real time builds.

Localstack could include the following services as well if we buy Pro licenses:

[ ] RDS
[ ] Elasticache (Redis)
[ ] Cognito

This would remove any reliance on leveraging Staging services and manually running docker containers for local development.
Acceptance criteria:

[ ] Review the current infrastructure setup to identify redundancies, outdated services, or configurations
[ ] Identify any dependencies that need to be updated or removed

@craigzour to test the local env now that Bryan pushed an update


Support meshes

Needs support for meshes. Currently only works for Sprites and TextureAtlasSprites.

This is difficult bc we can't read meshes like a texture unless we have a way to write what it would be into a texture and then read from that.

Given that the shapes used to generate meshes all generate using a topography of TriangleList, we could just break up the mesh into it's smaller triangles. This would change all the calculations significantly though, it might be better to have a separate system for it.

An alternative route, is to use the aabb bounding box to estimate the size of the texture, create the texture and render the mesh into it. But is this even possible?

At my wit's end, there just doesn't seem to be a straightforward way to do this for meshes.
I don't like the break into its triangles method since it imposes too many caveats, restrictions, and inconsistencies. But none of these are issues if we just render the mesh into a texture. But that seems impossible with Bevy, I can't even find what code in Bevy is directly responsible for the actual rendering of the mesh. I found how they are prepared using RenderAsset, creating a GpuMesh which docs say is the form meant to be processed directly by the GPU. But that's it. I could not find where CommandEncoder takes the GPUMesh (or RenderAsset in general) and actually do the work. Otherwise, I could pull the device directly, do the same thing, except instead write the output to a texture.

Supposedly rendering a Mesh to a Texture is not an operation that can be done in the confines of Bevy.

See #4

merged #9


can't remove old custom javascript payload, and can't disable screenshots

Hey!
I'm quite unsure what could have caused this, but I've got disabled the "Screenshot" checkbox, and got nothing in my "Custom javascript" input. However, it seems like the following js is always present on the payload page:
function ez_r() {
alert(document.domain)
}

And the "html2canvas" stays present on the payload page all the time? Is this supposed to be?
What could be causing this? And how could I fix it?
Thanks!
For reference:

was looking at the wrong panel, sorry!


Match trailing comma in Source and Declaration Files

TypeScript Version:  2.4.1
Code
enum Colors {
    Blue = "BLUE",
    Green = "GREEN",
    Red = "RED"
}

tsconfig.json
{
    "compilerOptions": {
        "declaration": true
    },
    "include": [
        "**/*.ts"
    ]
}

Expected behavior:
The TypeScript Declaration file should match the source file regarding trailing commas. Thus, the output should be this:
declare enum Colors {
    Blue = "BLUE",
    Green = "GREEN",
    Red = "RED"
}

Actual behavior:
Trailing comma for Red.
declare enum Colors {
    Blue = "BLUE",
    Green = "GREEN",
    Red = "RED",
}

This triggers an issue for TSLint's no-trailing-comma rule.
#16974

FWIW, trailing commas are gaining wider acceptance in ES2017. See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Trailing_commas.

*.d.ts files output from 2.9.2 seem to have dropped the trailing commas... can't find mention of the issue/PR where that happened though (nor if there's some sort of flag to control this automatic change which surprised me a bit).


CosmosDB: 7 day continuous backup restore window?

This page says

that all accounts incur an additional cost for continuous backup
that the restore window is 30 days and can't be changed

However the pricing page currently says that 7 days of continuous backup is free, and only 30 days cost extra – implying that there is also a 7 day restore window available.
Which one is correct?

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

ID: f97e4b07-cad6-3650-0434-78e2a09a7edb
Version Independent ID: 122fc486-e88c-f7ea-2625-b653b0a12d3a
Content: Continuous backup with point in time restore feature in Azure Cosmos DB
Content Source: articles/cosmos-db/continuous-backup-restore-introduction.md
Service: cosmos-db
GitHub Login: @kanshiG
Microsoft Alias: govindk

@hho Thank you for the feedback. We are actively investigating and will get back to you soon.

Thanks @kanshiG for answering this question.
@hho, we are going to close this issue as there's no action item to take until the new tier is rolled out.
#please-close


final deploy

Signed-off-by: rachejazz<EMAIL_ADDRESS>Will work once the secrets get updated

I thought about this some over the weekend, and came the conclusion that we don't want to keep our deployment files in the core repository. Others who download and deploy this tool are likely to use their own deployment mechanism, and this won't work for them.
As such, I've gone ahead and requested a deployment repository: https://github.com/openmainframeproject/foundation/issues/207
This repository will include the deployment file included in this commit, instructions for how to use GitHub as a deployment mechanism, and make use of GitHub Secrets to store our SSH credentials for our deployment.

That'll be great :D
Keeping this PR open, till we get that down.
🎉 Eager for the completion

Closing this, new repo will have new permissions, so will build a new workflow accordingly


Address book encryption

Blocked on changes to the SDK to expose ZIP 32 Arbitrary key derivation.

Note
This code review checklist is intended to serve as a starting point for the author and reviewer, although it may not be appropriate for all types of changes (e.g. fixing a spelling typo in documentation).  For more in-depth discussion of how we think about code review, please see Code Review Guidelines.

Author

[ ] Self-review your own code in GitHub's web interface[^1]
[ ] Add automated tests as appropriate
[ ] Update the manual tests[^2] as appropriate
[ ] Check the code coverage[^3] report for the automated tests
[ ] Update documentation as appropriate (e.g README.md, Architecture.md, CHANGELOG.md, etc.)
[ ] Run the app and try the changes
[ ] Pull in the latest changes from the main branch and squash your commits before assigning a reviewer[^4]

Note
It is good practice to provide before and after UI  screenshots in the description of this PR. This is only applicable for changes that modify the UI.

Reviewer

[ ] Check the code with the Code Review Guidelines checklist
[ ] Perform an ad hoc review[^5]
[ ] Review the automated tests
[ ] Review the manual tests
[ ] Review the documentation, README.md, Architecture.md, etc. as appropriate
[ ] Run the app and try the changes[^6]

[^1]: Code often looks different when reviewing the diff in a browser, making it easier to spot potential bugs.
[^2]: While we aim for automated testing of the application, some aspects require manual testing. If you had to manually test something during development of this pull request, write those steps down.
[^3]: While we are not looking for perfect coverage, the tool can point out potential cases that have been missed. Code coverage can be generated with: ./gradlew check for Kotlin modules and ./gradlew connectedCheck -PIS_ANDROID_INSTRUMENTATION_TEST_COVERAGE_ENABLED=true for Android modules.
[^4]: Having your code up to date and squashed will make it easier for others to review. Use best judgement when squashing commits, as some changes (such as refactoring) might be easier to review as a separate commit.
[^5]: In addition to a first pass using the code review guidelines, do a second pass using your best judgement and experience which may identify additional questions or comments. Research shows that code review is most effective when done in multiple passes, where reviewers look for different things through each pass.
[^6]: While the CI server runs the app to look for build failures or crashes, humans running the app are more likely to notice unexpected log messages, UI inconsistencies, or bad output data. Perform this step last, after verifying the code changes are safe to run locally.

Force-pushed:
-Rebased on main.

Separated the version encoding spaces. The unencrypted address book format is now versioned separately from the encryption scheme.
Introduced a file identifier derived from the address book key.
Fixed the encryption key derivation process.


Fixing REDIS look up in Cloud foundry

The redis look up failed as it was working outside the flask application context.

Codecov Report

Merging #78 into master will decrease coverage by 0.61%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
- Coverage   97.65%   97.04%   -0.62%
==========================================
  Files          25       25
  Lines         640      642       +2
==========================================
- Hits          625      623       -2
- Misses         15       19       +4

Impacted Files
Coverage Δ

response_operations_ui/__init__.py
78.04% <0%> (-6.57%)
:arrow_down:

response_operations_ui/cloud/cloudfoundry.py
66.66% <50%> (-11.12%)
: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 3c9454f...a12c88d. Read the comment docs.


Images

TODO:

[x] Update to Crafty 0.7.1
[x] Switch to WebGL
[x] Add Gfx for water
[ ] Add Gfx for bridge
[x] Add Gfx for city backgrounds
[x] Add Gfx for mine
[x] Add Gfx for rocket
[ ] Add cloud
[x] Refactor loading of assets for entities
[ ] Make seperate sprite animation for eye of 'large drone'
[ ] Make damage sprites for 'large drone'
[ ] Update city horizon
[ ] Add city horizon without water

The rest of the graphics will be done at a later time, in other pull requests (this one is getting too large)

Code wise it it's complete now, only new sprites required


Pass high level procedure vars along the launch msg

Pass "high level" vars to Flor by POSTing to Flack's /messages : { point: 'launch', tree: t, vars: vars }.
Aligns with Flor:Scheduler:launch(source_or_path, opts={}), where opts can receive vars:

Should there be a spec covering this?

A spec would be welcome.
Why deleting the vars from the message while domain and tree are just read?

Hi,
I 'll work up a spec for this.
I'll also change the behavior not to delete the vars. I though I had good motivations to do so, but as I'm working on this reply, I realize deleting them isn't sound.

Thanks a lot!


Small fixes to landing page

Purpose
small clean ups and fixes to landing page
Changes

Ordering happens at the node list view, to allow for proper fetching of additional nodes without adding duplicates
Prevent showing help text that implies that the user is looking at the my projects page (including the tooltips for collections and contributors)
Current user should be a filter in institution's contributors if they are a contibutor to one of those institutions nodes

Good to merge when the build passes.


Broken link in "Using ScalaTest with sbt" user guide

In the Using ScalaTest with sbt section Include and Exclude Tests with Tags the link to the documentation for Suite is broken.

PR submited:  https://github.com/scalatest/scalatest-website/pull/80

PR merged


feat: use aria-describedby for TextInput and error text

closes https://github.com/gravity-ui/uikit/issues/860

Could U add this changes to TextArea too?

Could U add this changes to TextArea too?

Ready


[CSM-458] Query postponeUpdate endpoint when postponing the update

These changes should make Daedalus query postponeUpdate endpoint on postponing the update. The endpoint is introduced in https://github.com/input-output-hk/cardano-sl/pull/1586. This is needed to avoid problem with postponing the update described in https://issues.serokell.io/issue/CSM-458?query=%23CSM -Resolved

Reviewing this one now...


liquidation script V1

Naive method of performing liquidation --
iterates through all contracts, for each contract, issue liquidation requests for each account. Therefore, the check for whether to actually liquidate is only done at the contract level.
Moving forward, to reduce the compute needed on chain, we should only issue liquidation requests for accounts actually underwater (requires maintaining account state/positions off chain).

Closing this due to inactivity, feel free to re-open and ping me if it needs another review.


Public Release for Fall24 meta-release

What type of PR is this?

subproject management

What this PR does / why we need it:
Public release (r1.2) for 0.2.0 version of the KnowYourCustomer API family.
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Test plans are still in PR but are expected to be merged before the creation of the Release
Issues that are still in PR pending to be merged, but already included in changelog:
2.1 Update endpoint documentation in https://github.com/camaraproject/KnowYourCustomer/pull/138
2.2 Remove countryMatchScore property in https://github.com/camaraproject/KnowYourCustomer/pull/137

Changelog input
 Public release of kyc-match v0.2.0 and kyc-fill-in v0.2.0

Additional documentation
https://wiki.camaraproject.org/display/CAM/Meta-release+Fall24
https://wiki.camaraproject.org/display/CAM/kyc-match+v0.2.0
https://wiki.camaraproject.org/display/CAM/kyc-fill-in+v0.2.0

Looks already quite good to me. Please add @camaraproject/release-management_maintainers as reviewer. I have created the review issue already ;-)

Test plan for kyc-fill-in still needs to be merged before or incorporated to PR

Thanks for pointing this out.  The PR #142 for kyc-fill-in test plan was merged at around 15hr UTC, so, I believe this point has been cleared.
Thanks,
Toshi
KDDI

@ToshiWakayama-KDDI @GillesInnov35
We need at least one approval from another codeowner to merge the PR

@GillesInnov35 @ToshiWakayama-KDDI
I had to resolve conflicts with main and it dismissed the approvals... please approve it again 😄


Sidebar "Scale Down" Animation

Whats the jquery for the scale down animation of the sidebar?

$('.ui.sidebar')
  .sidebar({
    transition: 'scale down'
  })
;

Thanks, where is this in the docs?

http://semantic-ui.com/modules/sidebar.html#/settings

@jlukic Ah I see. You should add a part explaining that the settings are just key:value pairs, as it wasnt obvious looking at the docs. Thanks for the fixes.


Ignore non-hashbrown Mongo databases

When I fired up Hashbrown for the first time I was presented with all my MongoDB databases, even though none of them were Hashbrown related. Wondering whether Hashbrown should be filtering the displayed databases to only include those created by Hashbrown? Maybe based on presence of settings collection and contained value?

Absolutely, this was originally the intended behaviour. Will fix this.


feat: Add kubernetes.io/cluster/<cluster_name>="owned" tag to the cluster security group

…t with worker group
PR o'clock
Description
https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1416
Checklist

[ ] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

After digging a bit in AWS docs, I found this:

It mention "One, and only one, of the security groups associated to your nodes should have the following tag applied.". I don't know if we're not breaking something here.

Issue #1441

basing on docs and issue #1441 I think this PR should be closed. I checked and this security group aws_security_group.cluster is not assigned directly to workers (only API control plane), but this that never-less we shouldn`t follow practice which is against docs.
@antonbabenko what you think?

@daroga0002 I don't have enough understanding of the implications (docs are too long) so I would close this one.

Closing as proposed approach doesn't follow AWS documentation and EKS requirements.


Unable to delete action/phrase

For some reason I am not able to delete this saying, and/or action.

I had similar phrases, and was able to delete them, then even deleted the category. However, this phrase, and the action connected to it is not able to be deleted (nothing happens when clicking.)Though I was able to delete the category "thank you" (not "small talk" as it was a mistake), the category box remains blank for that saying, and I can't assign a different category either.
No errors in console.

Did you download a release or pull from Master? Which release? Pull, 0.21
What operating system are you running? 10 Pro
Are you running locally or in the cloud? Local
Which browser are you using? Firefox, tested on edge

Update: For some reason that particular saying has been deleted from he dialogue, no longer there. I guess this happened after multiple training sessions, when continuing with the script.

@RussellSPG did you see this the second time around on your script?

yup, I have to delete directly from swagger


Uniwirt scraping is not working

Hi Guys,
The section for UNIWIRT only shows the word "information" nothing else.
Cheers & Thx

Hi, I created a fork which currently is maintained: 👉 https://food.fabian-feichter.at/.

https://github.com/Felfio/AAUFood/pull/103

Thanks for reporting and fixing. Has been merged and deployed. :)


The RaSCSI logs do not have entries for disk image creation/deletion

Info

Raspberry Pi 3A+, 3B
Latest and Pull request #229
v2.3 Full Featured
N/A

When using the web ui to create or delete an image, there is no feedback logged on the operation itself or whether it completed successfully. In the case of the RPi's file system becoming read only or full, the user would not receive any indication why the image creation or deletion failed.
If logging could be added along the lines of the ones present when mounting or dismounting an image, that would satisfy this issue.

@rdmark Would you agree that the image creation should be a part of rascsi, instead of being done by a client? I don't know if the web UI has alogfile of its own, but if rascsi were creating the images it could simply log any issues to the regular logfile.

@uweseimet I think this is a great feature to have in rascsi.  I don't know if you've kept up with #rascsi-dev on Discord, but I've been working on expanding image creation functionality of the frondend, together with a 'sidecar' configuration file format that goes with the image so that you can create, say, a 100MB "Iomega Zip" image which when attached will get the right product and block size properties loaded automatically from the sidecar file.  I have a rudimentary implementation in place in the webapp code in develop, and would love your thoughts on this approach. In particular, do you think it makes sense to do more of this on the rascsi backend, perhaps the ability to pass the config file as a parameter?

@rdmark IMHO rascsi should essentially provide more or less basic functionality, typically atomic operations. More complex functions (e.g. combinations of different rasci operations) should better be handled on the client side.

@uweseimet Agreed this is the correct design principle.

@rdmark Please note that I updated the protobuf interface in the feature_image_creation branch to use maps with named parameters instead of parameter lists. The initial approach is too error prone and also less explicit when you have operations that expect several parameters. In addition, the error handling can be more precise.
Please see the .proto file (PbOperation message) for the parameter names supported by each operation. The functionality as such has not changed.

@uweseimet I think that would be a cleaner solution, and make better use of parameters which is a bit underutilized right now.

@rdmark One should not forget, though, that the parameters are not type-safe, in constrast to explicit protobuf message fields.

@uweseimet Well noted, I'll make sure the Python code does extra thorough type validation when using the parameters. (The woes of a weakly typed language.)

@rdmark After sleeping a night on it I decided not to convert more explicit parameters to named parameters, due to the loss of type-safety, which also results in more and more string to integer (and back) conversions in the code. If you convert too many parameters to strings you end up with a string-based free-text interface as it was before using protobuf ;-).
Note that I added two interface operations for making image files read-only or writable. I added these because the R/W status of an image file is checked in the rascsi code, so it should be possible to control these attributes via the interface.
After some more testing I will create a PR for the changes discussed here.

@uweseimet Sounds good, then I don't have to rewrite as much of the web ui code.  I'll start integrating your feature branch in my feature branch now to start the integration of the new protobuf and image creation functionality.

@uweseimet BTW did you have a particular usecase in mind when you implemented the 'read_only' property for created images? I'm just wondering how it could be leveraged in the UI, if there is a common use case for protecting your images.

@rdmark rasci (also the old code) checks image file permissions, and because the permissions are relevant from the rascsi perspectiveI added the means to modify them.

For attaching/inserting image files the parameter name is "file", for SCBR/SCDP interfaces it is "interfaces". Please see the .proto file (PbOperation message) for the other parameter names for each operation.

@uweseimet  I think the proto file is missing inline documentation for the "file" and "interfaces" named parameters (for ATTACH in particular, and I assume INSERT should accept the "file" parameter as well)

@rdmark Indeed, thank you. I just updated the comments.

@uweseimet Thanks for the quick fix!
Now before I go too far into the rabbit hole, is my reading of the code correct in that the named parameter format is any number of strings containing name value pairs with equal signs as separator?
I.e. "file=harddrive.hda"

@rdmark It's a protobuf map (key/value pairs), see the definition in the .proto file.

@uweseimet I see. I'm trying to wrap my head around the Python representation of protobuf maps, but I'm really struggling so far. The documentation isn't much help https://developers.google.com/protocol-buffers/docs/reference/python-generated#map-fields
I'll keep trying for a bit longer, but I very much prefer the previous implementation. ;)

@rdmark Looks like a good opportunity to learn something new about python and protobuf ;-). It was the same for me with C++ and Kotlin, where I had never used protobuf before. (Before only with Java.)

Alright, alright, I'll keep trying. I think I figured out at least the basic usecase. You're supposed to use the name of the parameter as an "undefined key" of the map dictionary, and then assign the value to that.

@rdmark I'm afraid I cannot really comment on the Python mapping, because I hardly know Python. How some protobuf features are mapped may depend a lot on the programming language used. The C++ mapping is quite complicated, mainly because of memory managment requirements. The Kotlin mapping is disappointing, because it does not support protobuf enums, which means it's useless for me. I had to switch to the Java mapping instead, which can also be used with Kotlin without issues.

@uweseimet I think there's a bug now where if you attach a daynaport device with rasctl it gets the 'file' parameter instead of the 'interfaces' parameter.  Are you seeing the same behavior?  Using the protobuf interface directly is fine, so I assume the bug is in rasctl.

pi@rpi3b:~ $ rasctl -i 5 -c attach -t scdp -f wlan0:<IP_ADDRESS>/24
pi@rpi3b:~ $ rasctl -i 5 -c show
5:0  SCDP  Dayna:SCSI/Link:1.4a  file=wlan0:<IP_ADDRESS>/24

Finished implementing file creation and deletion using the new interface. PR in https://github.com/akuker/RASCSI/pull/242

@rdmark Fixed.

@uweseimet Thanks for the fix! I've tested it in my environment. What do you say we get this merged to develop now?


⚠ Bofrost.lu has degraded performance

In d400f9f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1725 ms

Resolved: Bofrost.lu performance has improved in 67198c4 after 6 minutes.


Update sbt-scoverage to 2.0.5

Updates org.scoverage:sbt-scoverage from 2.0.3 to 2.0.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scoverage", artifactId = "sbt-scoverage" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report
Base: 89.63% // Head: 89.53% // Decreases project coverage by -0.09% :warning:

Coverage data is based on head (6287301) compared to base (4d9a72d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1087      +/-   ##
===========================================
- Coverage    89.63%   89.53%   -0.10%
===========================================
  Files          119      119
  Lines        10016    10016
  Branches       695      695
===========================================
- Hits          8978     8968      -10
- Misses        1038     1048      +10

Impacted Files
Coverage Δ

...in/scala/com/twitter/algebird/scalacheck/Gen.scala
91.66% <0.00%> (-8.34%)
:arrow_down:

.../main/scala/com/twitter/algebird/Applicative.scala
53.33% <0.00%> (-3.34%)
:arrow_down:

.../main/scala/com/twitter/algebird/BloomFilter.scala
93.50% <0.00%> (-1.74%)
:arrow_down:

.../main/scala/com/twitter/algebird/Approximate.scala
88.70% <0.00%> (-1.62%)
:arrow_down:

...src/main/scala/com/twitter/algebird/Interval.scala
76.19% <0.00%> (-1.59%)
:arrow_down:

...a/com/twitter/algebird/immutable/BloomFilter.scala
95.27% <0.00%> (-0.79%)
:arrow_down:

.../scala/com/twitter/algebird/immutable/BitSet.scala
97.06% <0.00%> (+0.20%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Restore log dumps for gradle OOM crashes, and set a value for MaxMetaspaceSize

Re-sets two jvmargs that were getting cleared because we set a value for -Xmx. Could help with https://github.com/flutter/flutter/issues/142957. Copied from comment here https://github.com/flutter/flutter/issues/142957:

Two random things I ran into while looking into this that might help:

Gradle has defaults for a couple of the jvmargs, and setting any one of them clears those defaults for the others (bug here https://github.com/gradle/gradle/issues/19750). This can cause the "Gradle daemon to consume more and more native memory until it crashes", though the bug typically has a different associated error. It seems worth it to re-set those defaults.
There is a property we can set that will give us a heap dump on OOM (-XX:HeapDumpOnOutOfMemoryError)

Mostly just a find and replace from find . -name gradle.properties -exec sed -i '' 's/\-Xmx4G/-Xmx4G\ \-XX:MaxMetaspaceSize=2G\ \-XX:+HeapDumpOnOutOfMemoryError/g' {} \;, with the templates and the one test that writes from a string replaced by hand. I didn't set a value for MaxMetaspaceSize in the template files because I want to make sure this value doesn't cause problems in ci first (changes to the templates are essentially un-revertable for those who flutter create while the changes exist).
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Does the instance in android_plugin_example_app_build_test.dart also need to be updated?

Yes, thank you!

https://github.com/flutter/flutter/issues/143103


Callback argument type not inferred for union of interfaces

🔎 Search Terms
"implicitly has an" inference
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about "inference"

⏯ Playground Link
https://www.typescriptlang.org/play/?target=5&ts=5.5.3#code/KYDwDg9gTgLgBASwHY2FAZgQwMbDgQQGcBPJbAJWEIFcAbGAYU1toCMcBrAHgBUAaOAFE4AXiFQo0AHxwA3gFgAUHBVwAFGigB+AFxC4AHzhI6tAVCp0YuuDwCUegG4QEAEwDcSgL5KloSLCIKGhYuAQkZJQ09ACSqFCYMNC8AuQCwmKCEtJySqrqCKgAtnr8cNjMbJx6RKQUlvRMLOzY3GlCUg5wzm6eij6K-tDwyPGheLWRDTBxaInQAAqSRQiEwClw5DIKyqpqhcAltl1LECtrXFt9A0quwNi0mBZw6NRkMAgQSHBFmGAAMggVjANu0MuJJFApGo8qonlBSgBtAC6fFhKloQMKehMRVYaDRuxUBwSSQRKkm9WiM3i8ygp3O6zKW0M4TqUSss1JyWZ6SkhJOy1W63IKKkfSUvwBWJgakRAEY+AAmPgAZlRcFVAkwEWw6iQdlE23RcAsMGoUG+3wAVNa4ErrnZ3EA
💻 Code
export interface AsyncResultCallback<T, E = Error> {
    (err?: E | null, result?: T): void;
}

export interface AsyncResultIterator<T, R, E = Error> {
    (item: T, callback: AsyncResultCallback<R, E>): void;
}
export interface AsyncResultIteratorPromise<T, R> {
    (item: T): Promise<R>;
}

declare function mapLimit<T, R, E = Error>(
    arr: T[],
    limit: number,
    iterator:  AsyncResultIteratorPromise<T, R> | AsyncResultIterator<T, R, E>,
): Promise<R[]>;

mapLimit([1,2,3], 3, async (n) => {
    return n ** 2;
});

🙁 Actual behavior
The type of n in the mapLimit callback is inferred as any:
Parameter 'n' implicitly has an 'any' type. (7006)

🙂 Expected behavior
I would expect the type of n to be inferred as number.
Additional information about the issue
This example is based on the types for the async package:
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/b9820410f85f259cbad6065d8cb6f23f403109ac/types/async/index.d.ts#L321-L325
Note that if the type for iterator in mapLimit is changed to either just AsyncResultIteratorPromise<T, R> or just AsyncResultIterator<T, R, E> (that is, no longer a union type), things work as expected and n has type number.
I found a very old discussion of what looks to be the same issue in the DefinitelyTyped repo: https://github.com/DefinitelyTyped/DefinitelyTyped/issues/24297. The issue described there is exactly the one I'm facing. One commenter claimed that things worked correctly, and another commenter claimed that changing the way that async was imported had some effect? Neither of those sounds plausible to me.
I found some issues that may describe the same or a similar problem:

https://github.com/microsoft/TypeScript/issues/56881
https://github.com/microsoft/TypeScript/issues/55632
https://github.com/microsoft/TypeScript/issues/52047
https://github.com/microsoft/TypeScript/issues/47599

However, my knowledge of TypeScript internals/terminology isn't strong enough to say for sure that this isn't a novel issue I'm facing. If this is in fact a duplicate, please close it as such and I'll add this example to the relevant issue. Thank you!

Simpler repro:
function test<T>(
  arr: T[],
  cb: ((item: T) => void) | ((item: T, extra: unknown) => void),
) {}

test([1, 2, 3], (item) => {});

Currently, getContextualSignature returns undefined in this situation because the 2 signatures that are present here don't pass the compareSignaturesIdentical check in that function


fix Prevent SynEvent from crashing FF test on getSelection

On Firefox, selectionStart throws an error on an checkbox for instance (NS_ERROR_FAILURE)

:ok: 4901c06bdf7fabcc282d9d297c6f4b2efe2834f2 seems fine for me


WIP: Extend ShowOnDashboardAttribute with configurable command

Fix for #98

@CraigHawker - would appreciate a review of this. The main thing I'm not sure about is adding a RunCommandId does nothing unless you also set ShowRunCommand = true. This could be kind of implicit but that feels a bit odd to me.

@CraigHawker - would appreciate a review of this. The main thing I'm not sure about is adding a RunCommandId does nothing unless you also set ShowRunCommand = true. This could be kind of implicit but that feels a bit odd to me.

If you truly wanted it to be implicit then, if RunCommandId is not empty/null, render the button regardless of ShowRunCommand?  I'm not sure whether I like or dislike that approach, but it's an option.


Crawl / manually curate / map Dataset resources into individual metadata records based off of schema

Map resource list to Dataset schema.  Should also rely on crawler.biothings.io to grab datasets from Figshare, Harvard Dataverse, etc.

List of generalist repositories hosting COVID data and code at this link https://datascience.nih.gov/news/nih-to-host-webinar-on-sharing-discovering-and-citing-covid-19-data-and-code-in-generalist-repositories-on-april-24. Sites included:

vivli.org: https://vivli.org/how-to-share-your-research-on-the-vivli-covid-19-portal/
Figshare: https://covid19.figshare.com/
GitHub: https://github.com/github/covid-19-repo-data
Dryad (no direct link)
Zenodo: https://zenodo.org/communities/covid-19/?page=1&size=20
Harvard Dataverse: https://projects.iq.harvard.edu/chinadatalab/resources-covid-19
Mendeley Data: https://data.mendeley.com/research-data/elsevier-covid19

FYI the link to Harvard Dataverse is one dataverse from the China Data Lab, not all the dataverses.
crawler.biothings.io should be able to grab Figshare, Dataverse, and Zenodo if updated.
current repo counts:
vivli: 0 studies (?)
figshare: 223
github: enormous list of repos containing data
dryad: a few
zenodo: 214
dataverse: 87
mendeley: 4

COVID-specific API links:
Figshare: https://covid19.figshare.com/api/institutions/857/items?types=&licenses=&orderBy=published_date&orderType=desc&limit=40&search=&categories=&itemTypes=

Figshare importation could be tackled with two approaches (or a hybrid of the two).

Some pages have [https://search.google.com/structured-data/testing-tool/u/0/#url=https%3A%2F%2Ffigshare.com%2Farticles%2FData_on_the_COVID-19_outbreak_in_Romania%2F12045243](embedded schema.org json metadata) in a <script type="application/ld+json"> tag.  So, using the API, we could get the URLs, scrape the ld+json data, add a couple of outbreak-resource-specific things, and upload.

BUT... other records don't have this tag, meaning we need to pull the metadata some other way.

Pull all the data from their API and map to schema.org properties (for instance publishedDate to datePublished).

Personally, approach 2 seems like more work, but if we're going to have to do it anyway for records without embedded metadata, we might as well do it for everything.

MVP developed w/ Zenodo crawler, Figshare parser. Closing and creating separate issues for other data sources as they arise


jira board / gitlab project mapping is not needed any more

Describe the bug

this feature was deprecated, need to be remove.

Will have this removed from form.


Check that last action is not listen

Proposed changes:

Fixes https://github.com/RasaHQ/rasa/issues/5356

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

sure makes sense


More friendly reply when a review does not belong to a reviewer or committer

ti-chi-bot replies @tisonkun: /lgtm is only allowed for the reviewers in list. if I review a pull request where I am not a reviewer or committer.
https://github.com/pingcap/tidb/pull/24463#issuecomment-837742624
... while the previous ti-sre-bot replies Thanks for your review. The bot only counts LGTMs from Reviewers and higher roles, but you're still welcome to leave your comments.
https://github.com/pingcap/tics/pull/1896#issuecomment-839585845
It should be better if we keep the friendly reply.
cc @hi-rustin

This is a good improvement! Maybe the response to invalid /merge needs to be improved too.


Senate tweets

Hi Keyon,
Thank you very much again for the well-documented repository. I've tried to reproduce the results of the paper, but I cannot find the voxgov_tweets.csv file on the website https://voxgov.com. Where can I find it?

Hi Josue,
Thanks for the note. The Voxgov data was shared with us privately and so unfortunately we're unable to release it. One option is to use another scraped datasets of tweets, e.g. this Reddit post contains a scraped dataset of U.S. Senate tweets.

Thanks for the quick answer, I'll try with that dataset.


Encrypting the configuration (passwords and tokens)

Currently the Gush configuration (passwords and tokens) are stored in plain-text!
From a security point-of-view this is horribly insecure.
Passwords and tokens should (must) be stored encrypted using a master-password.
But then you would need to provide a password every time you use Gush!
The first solution that came to mind was a configuration-agent (like the ssh-agent)! When you start Gush, an agent will be started (if not already running) which keeps the passwords and tokens decrypted in memory. So you only have to provide the master-password once per session.
There is already a very good library to do the encryption safely: https://github.com/defuse/php-encryption
Personally I would like to use https://github.com/jedisct1/libsodium-php as it also provides memory clearing (removing traces from memory) but this requires that extension is installed, so this should only be used as a recommendation and considered 'upgraded security'.
Some things I wonder about:

Should we encrypt the whole file or only the authentication info?
Its recommended (by defuse) to not use passwords but use a key derived from the password, but as this configuration is only local should this be applied?
Is memory clearing really mandatory? I know it improves security, but what is the actual risk of not doing this.

@ircmaxell as a security expert of the open-source community do you have any other suggestions or comments on this idea? :+1:

I like the idea of a passphrase to secure ssh private key like behavior for gush

From his Twitter feed it seems Anthony is on a vacation :) I will ping him again when he gets back.

there are other experts that can advise too @enygma or ... well i don't know many :blush:

I know @padraic (Pádraic Brady), @defuse (Taylor Hornby) creator of the php-encryption library, and @paragonie-scott (Scott) :+1:

@ircmaxell, i know you are very busy and probably you care little about this project, but if we can still get some kind of response from you would be great to have some closure on this. Thanks in advance.

Why not just use the systems keychain (on OSX: http://blog.macromates.com/2006/keychain-access-from-shell/ on Linux: http://askubuntu.com/questions/262698/how-do-i-get-passwords-from-the-keyring-in-the-terminal-for-usage-in-scripts etc).
Basically, generate an encryption key, store it in the system's keychain, then encrypt the config file (the whole thing) with that key. You should create a second config file (or wrapper) which contains the keychain info.
Then simply decrypt and move on.
Creating your own agent is going to be problematic (though it is an interesting proposition).
Also, as an aside, the documentation really never says clearly what this project does. I had a very hard time figuring out why you need all of those credentials...

Thanks for the heads-up and information.

Also, as an aside, the documentation really never says clearly what this project does. I had a very hard time figuring out why you need all of those credentials...

We communicate with the GitHub API (and others) to open/close/modify issues/pull-requests and manage branches.
The (old) configurator actually allowed to use a username/password and store this in plain text! The new one always generates a new token, but it got lost during some version switching (I still got the commit but need to restore it).

I like @ircmaxell option to just use keychain, but for now i don't think this is as important for development. If I get my laptop stolen I am in big trouble anyways. I believe that is what 2fa is for if you require extra security.

OK, it's a bit to much work for now. Lets keep it open If anyone is willing to work on it :+1:

:+1:


As a non-tech savvy user, I want directions on how to navigate this application.

Feedback from end user:

Make it easy as possible to navigate the site.
Have directions clear on what they need to do with appropriate hyperlinks

Information contained in the wiki


Snapshot Prometheus disk after tearing Prometheus down (1.18)

Gracefully tear down Prometheus retaining PD, then snapshot and delete PD.
Cherry-picks: #1178, #1199
Fix: #1020
/cc mm4tt tosi3k mborsz

/lgtm
/approve


Lab1 - Fibonacci

Fixes #
Review of colleague's PR #
Changes proposed in this PR:

solution for Lab1 - fibonacci
Tests for the fibonacci

-

Codecov Report

Merging #202 into lab1 will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##           lab1   #202   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files         3      3
  Lines        43     45    +2
===================================
+ Hits         43     45    +2

Impacted Files
Coverage Δ

01_fib/yayaluo/main.go
100% <100%> (ø)

01_fib/kasunfdo/main.go

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 fdefd4f...e51fc7b. Read the comment docs.


Add the ability to attach custom properties to the Metadata Store

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Describe changes
This PR reflects the initial changes required as we want to ensure that our custom properties are tracked within the ML Metadata Store. This is achieved by adding a new ContextSpec to PipelineNodes within the Pipeline proto message. Importing to note: This PR only covers the Stack element. As our internal discussion proved that we can extend this even further, this is considered to be the initial step to a more generic solution.
Other minor changes:

LocalDagRunner is removed.
KubeflowOrchestrator and KubeflowDagRunner are adjusted.
The inheritance to the TFXRunner is removed.

@bcdurak will do a proper PR tomorrow but could you change the name of the PR to something that would look nice in the release notes?

@htahir1 changed the name accordingly! thanks for the tip

@htahir1 It sort of depends on what you would like to add. Things like the git sha might be treated as a execution parameter (meaning that a change in the git sha would mean that it is required to reexecute the code), whereas the stack is treated as a 'context' of the pipeline node (meaning that a change in the stack does not invalidate caching). That's why for both cases, the flow of how we track these values is different.
As of now, we are just tracking the stack as a context within the pipeline nodes of each pipeline, thus anything that you would put in a stack will be tracked. However, this is just the first step and it is quite open to improvement and should be extended.
@schustmi Good catch. I will take a look at the AirflowOrchestrator and implement some tests.

Makes sense @bcdurak ! I am wondering where to put the Schedule

Quick update: The AirflowOrchestrator is now working with the new changes.

@schustmi I have added a test covering the stack context as well.


[FEATURE REQUEST] Choose between available cameras

It would be nice to have a selector with available cameras on device to select, or a button to switch between them... for example on a mobile phone you could let the camera switch between front or back camera.
Thanks in advance!.

You can already select the target camera during the Camera Authorization screen as shown on the attached image. This is browser specific as each vendor implement its own UI.

My request and suggestion is to be able to select between each available camera every time you start an enrollment, this would allow to enroll users with a phone in a much easier way.


#60 Time_slice() added in epoch, event and irregularsampledsignal

With tests for each.

Coverage increased (+1.05%) to 53.135% when pulling fc4f4ba74c689fc323c7a59c9e39c17a9ccbae27 on jgonin:issue60 into 52e8fd33e623e0cb2d3f2f178d03b75cf765fe1c on NeuralEnsemble:master.

Merged by hand, due to conflicts with earlier PR (#284) addressing part of the same issue


Make Custom allocators Opt-in & Parallel Benchmarks & Jemalloc on Unix

In this PR:

[x] Custom allocators are disabled by default and are hidden behind feature flag custom-alloc
[x] When custom-alloc is activated, mimalloc will be used on Windows and jemalloc will be used on unix based systems.
[x] Build CLI: additional_flags add to both CLI arguments and user configuration to enable activating features in build process.
[x] Build CLI: Add custom-alloc to additional-features to enable custom allocators feature in rs-bindings
[x] Add benchmarks to replicate running producer sessions multiple times in parallel with various memory allocators to cover their performance on multi-threaded scenarios.
[x] Add jemalloc benchmarks to consider using it on Unix based systems.
[x] Build CLI: Increase Version & Update README and CHANGELOG

@marcmo @DmitryAstafyev I've finished the changes and the PR is ready for review.
I'll test it on Windows machine and post the results


Maintainer

Can maintainer work on the main repo to edit the files?

yes

yea

As far as I know the maintainer shouldn't directly edit files in the main repository, and should fork the repository and work there the same as other members. The only time you should directly edit files that will be in main is when you have to edit pull requests to fix incoming files/resolve conflicts


add service-worker.js router

service-worker.js can be accessed.
http://localhost:3002/admin/ still returns 404.
PS: While admin signed up successfully, http://localhost:3002/oauth/login API returns 302, Redirecting to /admin/, so please help to check the redirection URL or modify the routing rule to allow /admin/.
Thanks!

It works well now.


Permissions issue when downloading artifacts

I have the following job in my component
@component(
    base_image="python:3.10",
    packages_to_install=["google-cloud-aiplatform==1.24.1"],
    output_component_file=str(Path(__file__).with_suffix(".yaml")),
)
def custom_train_job(model: Output[Artifact],...):

   gs_model_dir = model.path.replace("/gcs/", "gs://")
   job = aip.CustomContainerTrainingJob(
        project=project_id,
        location=project_location,
        display_name=job_name if job_name else f"Custom job {int(time.time())}",
        container_uri=train_container_uri,
        model_serving_container_image_uri=serving_container_uri,
        model_serving_container_health_route="/health",
        model_serving_container_predict_route="/predict",
        model_serving_container_args=["--model_uri", gs_model_dir],
        staging_bucket=staging_bucket,
    )

    cmd_args = [
        f"--train_data={train_data.path}",
        f"--valid_data={valid_data.path}",
        f"--test_data={test_data.path}",
        f"--metrics={metrics.path}",
        f"--hparams={json.dumps(hparams if hparams else {})}",
        f"--model={model.path}"
    ]

    training_output_dir = gs_model_dir.rsplit("/", 1)[0]

    uploaded_model = job.run(
        model_display_name=model_display_name,
        parent_model=parent_model,
        is_default_version=(not parent_model),
        args=cmd_args,
        replica_count=replica_count,
        machine_type=machine_type,
        accelerator_type=accelerator_type,
        accelerator_count=accelerator_count,
        service_account=service_acount,
        base_output_dir=training_output_dir
    )

I can see the artifacts correctly saved in my bucket, however, i cant download them when i try to  deploy (as you can see i pass  the path like this model_serving_container_args=["--model_uri", gs_model_dir] to the serving container).
My serving script api.py (called in the entrypoint) is something like this
parser = argparse.ArgumentParser()
parser.add_argument("--model_uri", default=os.getenv("AIP_MODEL_DIR"), type=str, help="")
args = parser.parse_args()

model_gcs_path = args.model_uri + "/model.joblib"
print("model_gcs_path:", model_gcs_path)
bucket_name = model_gcs_path.split("/")[2]
print("bucket:", bucket_name)
source_blob_name = "/".join(model_gcs_path.split("/")[3:]) #removes gs://bucket_name
print("source_blob_name:", source_blob_name)

local_model_path = "model.joblib"

storage_client = storage.Client()
bucket = storage_client.bucket(bucket_name)
blob = bucket.blob(source_blob_name)
blob.download_to_filename(local_model_path)

app = Flask(__name__)
...

As i said, the last part is not working and gets the following error
<EMAIL_ADDRESS>does not have storage.objects.get access to the Google Cloud Storage object. Permission 'storage.objects.get' denied on resource (or it may not exist).: ('Request failed with status code', 403, 'Expected one of', <HTTPStatus.OK: 200>, <HTTPStatus.PARTIAL_CONTENT: 206>).
I have already set my service account in the training job and confirmed the path to the file is the right one.  From the documentation I think i need to modify AIP_STORAGE_URI but i don't know how. My serving dockerfile is the following
FROM python:3.10

WORKDIR /root

RUN pip install --upgrade pip
RUN pip install -r requirements.txt

COPY api.py /root/api.py

ENTRYPOINT ["python", "api.py"]

Any ideas?
Best regards

For anyone interested, you need to add a service account when deploying

For anyone interested, you need to add a service account to add permissions to your particular bucket (not the one used by vertex ai agent i think)
So, if you want to deploy your custom model trained in a previous pipeline component can do the following
@component(
    base_image="python:3.10",
    packages_to_install=["google-cloud-aiplatform==1.24.1"],
    output_component_file=str(Path(__file__).with_suffix(".yaml")),
)
def deploy_model(
    model_name: str,
    model: Input[Model],
    service_account: str,
):
    from google.cloud import aiplatform as aip
    logging.info(f"listing models with display name {model.metadata["name"]}")

    model = aip.Model(model_name=model.metadata["resourceName"])
    model.deploy(service_account=service_account)


chore: remove a dbg! forget to remove

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
Just remove a dbg!() which I forget to remove when merging, will cause printing additional information when create flow
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Summarize your change (mandatory)
How does this PR work? Need a brief introduction for the changed logic (optional)
Describe clearly one logical change and avoid lazy messages (optional)
Describe any limitations of the current code (optional)

Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

Clippy failed.
https://github.com/GreptimeTeam/greptimedb/actions/runs/9155477518/job/25167937524?pr=3990


[Spike] Proof of concept for element media qeuries

Proof of concept for media queries of panels within the CMS.
https://github.com/marcj/css-element-queries
We need to test wether or not this would be a suitable solution for improving the responsiveness of the CMS. Would need to be tested on forms in both smaller and larger panels throughout the CMS and AssetAdmin.

Responsiveness in the CMS is not a major concern at the moment. This can be reopened when/if we look at CMS responsiveness again.


PYNQ file system to add DPU, DPU part is not displayed?

Here I'm using the zcu102 board.
I imported the vivado project containing DPU into the petalinux project, and generated the corresponding image through the petalinux tool, and successfully ran the demo. However, after packaging the project successfully bsp, I successfully transplanted the PYNQ file system according to the way of modifying the homemade board. However, I cannot find the file system Settings of DPU, such as the library of vitis ai added to the file system. I would like to know whether the file system Settings of the original project are retained during the PYNQ transplantation process, or whether the dpu parts of the original project are also changed?

PYNQ doesn't support using the DPU integrated in Vivado.  If this is still an issue re-open it as a [discuss.pynq.io](https://discuss.pynq.io/) post.


add fractional seconds support for time columns (only supported in my…

…sql 5.6.4 and above)
@BrianIp @jcesario

:+1:


Makes the Barcode widget text editable

In some circumstances (low lights, partially damaged barcode) a barcode maybe be impossible to scan. This PR improve the usability of the Barcode widget by making its content editable by the user.
This PR simply replaces the TextView widget by an EditText.

Hmm... I'm not entirely convinced this is something we want in every use case - presumably many projects want a barcode to be acquired only directly with no room for human interaction/error. I'd think this should be added as a configurable appearance setting, similar to the way appearance="acquired" configures image questions

Agree that as a global change we wouldn't want people to be able to
manipulate the barcode scan.
Will: Barcode is a datatype, right? that means it should be pretty easy to
set this as an appearance, right?
On Mon, Mar 2, 2015 at 9:34 AM, wpride<EMAIL_ADDRESS>wrote:

Hmm... I'm not entirely convinced this is something we want in every use
case - presumably many projects want a barcode to be acquired only directly
with no room for human interaction/error. I'd think this should be added as
a configurable appearance setting, similar to the way appearance="acquired"
configures image questions
—
Reply to this email directly or view it on GitHub
https://github.com/dimagi/commcare-odk/pull/162#issuecomment-76721289.

The read-only aspect of the BarCode widget makes lot of sense, I agree. However, what should be the user experience / way out in case of unreadable / damaged BarCode ?
This happened to me quite a lot with low specs smartphones : scanning a barcode requires to be outdoor to maximize the light hitting the sensor. Indoor / night scanning wouldn't work at all. When this happens I generate a barcode in a Linux terminal and scan my laptop screen…
Could a setting like "Change Locale" do the trick ? By default Barcodes would be RO, but it would be possible to set them RW if required on per user / per handset basis ?

I think you could accomplish this the equivalent of this by adding a String
question, either below the barcode question in a field list or possibly
toggled by a single select that asks if you want to use the barcode scanner
or not and updates the relevancy of the barcode/string questions
accordingly. You could configure this at the app level on a per user/device
basis via custom user data.
Would that be sufficient in the cases you're thinking of?
On Mon, Mar 2, 2015 at 11:03 PM, Charles Flèche<EMAIL_ADDRESS>wrote:

The read-only aspect of the BarCode widget makes lot of sense, I agree.
However, what should be the user experience / way out in case of unreadable
/ damaged BarCode ?
This happened to me quite a lot with low specs smartphones : scanning a
barcode requires to be outdoor to maximize the light hitting the sensor.
Indoor / night scanning wouldn't work at all. When this happens I generate
a barcode in a Linux terminal http://linux.die.net/man/1/qrencode and
scan my laptop screen…
Could a setting like "Change Locale" do the trick ? By default Barcodes
would be RO, but it would be possible to set them RW if required on per
user / per handset basis ?
—
Reply to this email directly or view it on GitHub
https://github.com/dimagi/commcare-odk/pull/162#issuecomment-76883420.

Thanks @wpride, this would work. However, shouldn't this Barcode issue resolved at an higher level ? I mean, at the moment, what do you guys usually do when you can not read barcodes on the field ?

I don't have a great answer to that, likely better question for
commcare-users than for us devs, I've not heard of this being a blocking
issue for people before
On Tue, Mar 3, 2015 at 10:16 PM, Charles Flèche<EMAIL_ADDRESS>wrote:

Thanks @wpride https://github.com/wpride, this would work. However,
shouldn't this Barcode issue resolved at an higher level ? I mean, at the
moment, what do you guys usually do when you can not read barcodes on the
field ?
—
Reply to this email directly or view it on GitHub
https://github.com/dimagi/commcare-odk/pull/162#issuecomment-77089966.

Fair enough. I'll ask commcare-users. Thanks @wpride

closing due to https://github.com/dimagi/commcare-odk/pull/283


Single-point energy calculations

Hello All,
Please how can I do Single-point energy calculations in PySCF Like can we do in Gaussian software?
Thanks.

Most calculations in PySCF are "single-point energy calculations". Please
see the User Guide documentation for details on how to perform single-point
energy calculations at many possible levels of theory:
https://pyscf.org/user.html
On Wed, Jul 20, 2022 at 5:02 PM ISSAD Moumena Elkebich <
@.***> wrote:

Hello All,
Please how can I do Single-point energy calculations in PySCF ?
Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/1332, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC6VRCJ3ATG3DC4VCTTVVBSOXANCNFSM54FERI7A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Process and tools to support communication with service partner regarding follow-up across the release plan activities

Communication with service partner regarding follow-up across the release plan activities

Currently there is no clear/easy way to guarantee that during the release process we are following up with the service team. Although this can be slightly improved with the outcome of the work on #4045 we need to make sure that tooling is supporting the follow up too.
For example, when a user requests feedback on a OpenAPI PR, who follows up? what is the time frame?
We also have a bot that is looking at the state of the PRs and issues in our repo, how is this contributing?
There are several teams channels for management plane that service partner's use to engage our team and ARM.  Service team's tend to use these channels to get help with PRs and Shift Left/API Spec pipeline enablement. Should links to these teams channels be available via the release plan at the appropriate step?
1.Service API Toolset
2.API Spec Review
Does the API Stewardship Board use their teams channel as a way for service teams to follow-up or ask questions during or after API Reviews?
Is APIView the tool that the arch board uses for follow-up after the SDK Reviews?
What are the SLAs associated with follow-ups for the following activities?

API Reviews.
API Spec/TypeSpec pull requests (PRs).
SDK reviews.
Naming approvals for new initial Beta SDKs for management plane and data plane.
Arch Board approval for initial stable SDK release.
SDK breaking change review for new SDK versions for management plane and data plane.
APEX launch criteria attestations/signoffs.
Release planner bugs reported by service partners.

Notifications.  Which activities should have notifications?  How and where should these notifications be?  Who should recieve the notifications?

We can do a lot here, for example, notification, dashboard.  However, currently, this issue is a requirement, yet being translated into features.  This is not ready to hand over to engineer.

I place the milestone to be '2023-01'. This could be closed when Release Planner can cover both Data Plane and Management Plane SDK release.  Notification and dashboard requirements can be tracked with separate epics.

These are all great ideas and most of it has been covered by independent efforts. Where we can, the docs have been updated to reflect the point of contact for each step.
As there are no concrete action items for the Eng Experience team, I am closing this issue


Support for Laravel 6.

Added support for Laravel 6.0
Release scheduled for September 3rd , 2019 (https://laravel.com/docs/6.0/releases).
Dropped support for Laravel < 5.5.
5.5 is the latest LTS release.

There's no reason to change anything in this file besides for adding in 6.0.
I pushed that here: https://github.com/JosephSilber/page-cache/commit/f9038d2b3a05702ffa3e587a2e6be40d55c13820


[BUG] NextJS APP router causes enableAutoTracking to be called a lot of times

Description/Screenshot
When using the nextJS Approuter to navigate between routes, enableAutoRouteTracking sends way to many track exceptions causing the navigation times to slow down after a while.
I have a nextJS application that uses Application Insights. Due to having the need to stop navigation when some custom listeners are active, I am using useRouter from next/navigation to handle navigation. To do this I am using a mix of useRouter.push(url) and useRouter.refresh() (to make sure api calls are called after saving data).
When I navigate between these routes using my custom navigation code I get around 500 to 1000 track pageview exceptions, which can be seen when checking the network tab in the browser inspector.
Below is an image of the way I am handling navigation

Image of the APP Insights setup

Image of Track's in browser Insights after first load

Image of Tracks in browser Insighst after using navigation item

I managed to fix this issue by replacing the navigating component with

router.replace(url);

And by running the Router.refresh only when it needs to be refreshed, for example after saving data that does not update in a server component.
This doesn't completely fix the issue but decreases the amount of track's that are processed after a navigation.
Steps to Reproduce
make sure enableAutoRouteTracking is true.
Make basic code to navigate between 2 nextJS pages using useRouter.push(url of otherpage) followed by a useRouter.refresh().
Switch between the pages using the router while the network tab is open.

OS/Browser: Opera GX/Edge/safari/chrome
SDK Version [e.g. 17]: 17.3.3
How you initialized the SDK:

Expected behavior
I expect autoRouteTracking to only give back one or two track exceptions when switching routes and not hundreds of them.
Additional context
Add any other context about the problem here.

I get around 500 to 1000 track pageview exceptions

What is the contents of the exception?
This sounds like either

You are getting a cascade of exceptions where and whatever is causing the exception is causing more exceptions
Or you are creating / initializing MANY instances (with the same configuration) of the SDK and each instance is listening, capturing and sending the exception

This scenario occurs if you are creating / initializing the SDK in a base component that actually is getting called for every object (attached to the DOM or part of the shadow DOM), so it won't JUST be the exceptions but ANY automatically captured events.

Thanks for the reaction.
The SDK was indeed being initialized multiple times causing every route chance event to be captured multiple times.
I have now changed my code to make sure it is only initialized once and am now only getting one track event per route switch which looks to be the intended behaviour.
The initialization code was being called inside of a custom made AppInsightsProvider which was rerendering a few times thus causing the SDK to be reinitialized over and over again.


read source filename from dotnet run command

having the script source filename embedded in the input and output path symbols in symbols.txt isn't very convenient. Explore the alternative of specifying the script name in the dotnet run command which compiles the compiler and runs it. If we can figure out how to make this name available to the compiler, it can embed it in the source and object paths that are in the symbols file.

references:
(https://learn.microsoft.com/en-us/dotnet/csharp/fundamentals/program-structure/main-command-line)
https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-run
(note the use of -- to separate c# compiler arguments from the application arguments)

some information from the compiler source code:
  static void Main(string[] args)
  {
     // the script compiler is compiled and run by the terminal command:
     //      dotnet run -- filename
     // where filename does not contain .txt or .js or any path information
     // where the source file is called filename.txt
     // and the object file is called 01__filename.js
     // the I and O symbols define the input file and object file paths respectively

     string fname = args[0];


Countering system

The current countering system is incomplete and unstable. It does not interact properly with skills that affect multiple targets and can cause unexpected chakra changes. There are several under-the-hood countering mechanisms, and their behavior is not entirely consistent.

Just as an addition to the issues: When a skill that creates stacks is countered, it still generates a stack:
Expected: Guy puts counter on Gekko, GEkko uses move and is countered, since the skill didn't do anything, the stack doesn't generate
Actual: Gekko is countered, but he generates a stack that makes him deal 5 more damage

There is now an actual countering system, which should address the above concerns. It still needs work and testing, but at least it is stable, so I'm switching this to 'progress' for now.

Work and testing have been applied, and things seem to be going smoothly. I'm closing this, and if in the future any specific problems relating to counters occur, they are to be separate issues.


SDK test resource random name potential collision

I think there may be a logic error in how the test SDK generates random names for resources.
As an example, consider: test_mgmt_compute_test_containerdb8c0d41, which is the name of the resource group in this recording.
The problem, as I see it, is illustrated by the following, which is how get_resource_name creates the "random" name for the resource:
In [9]: hex(zlib.adler32(b'test_mgmt_compute.test_container') & 0xffffffff)
Out[9]: '0xdb8c0d41'

Note that the suffix appended to the fully qualified test name is just the checksum derived from the fully qualified test name itself! This means that there is no point in using both the fully qualified test name and the checksum: any resources created for that test and named using get_resource_name will have the same name.
This doesn't seem to cause any huge issues so I think it makes sense to resolve it piecemeal, by handling it better in azure-devtools. As I understand it, the intent here is that it should be possible to use the same name for resources in different tests but differentiate them based on the fully qualified test names as above to avoid collisions. If our resource name generator takes a prefix and then augments it with either the fully qualified test name OR the checksum, that should be enough as long as we don't use the same prefix more than once in a test.

@huguesv you were the original designer of this. @v-iam seems right to me, or I miss something?

The implementation of get_resource_name looks correct to me:
https://github.com/Azure/azure-sdk-for-python/blob/69da9801a7abb98b322df60ce4fcc0410863eccf/azure-common/testutils/common_recordingtestcase.py#L98
The name becomes redundant if you pass in the qualified test name as the name parameter, which is what is done here:
https://github.com/Azure/azure-sdk-for-python/blob/94ed6ed3858e00420b95e244860c87e692d4a6b4/azure-mgmt/tests/mgmt_testcase.py#L70
It probably doesn't need to pass in the fully qualified name there, it could pass in a shorter "prefix" instead.
Note that there is some value (at least there was back then) to having a name that can be traced back to a particular test/test class. When you use the Azure portal and see a ton of resources or resource groups that didn't clean up properly, it's nice to be able to tell where they came from.  If the full name is just a checksum, it's more difficult.

To clarify, I would probably change mgmt._testcase.py to do this instead:
self.group_name = self.qualified_test_name.replace('.', '_')

Closing, since we didn't get any issues so far, assuming this is correct :)


Cannot read property "addEventListener" of null image, because image not found by setup

Sorry, but I couldn't get the image Resizer working until I removed a '#' character in line 63.
The error was thrown by the first line of addEventListener because image was null, but the problem seems to be in the setup() function. This is from the console log within setup().
The variable map was set fine, but image wasn't.
map
   <map id="add-img-maps-1022-full" data-from="content,thumbnail" data-image-id="1022" data-url="http://orwell/blog/wp-content/uploads/2013/03/featured-image-horizontal.jpg" data-size="full" name="add-img-maps-1022-full">"""<area shape="rect" coords="170, 120, 410, 180" href="http://horizontalimage.com/" alt="Horizontal featured image"></map>
image
   null

The problem seems to be that the selector is treating the map name as if it were itself a selector, and adding a superfluous '#'.
map.name
  "add-img-maps-1022-full"
document.querySelector('img[usemap="#'+map.name+'"]')
   null
document.querySelector('img[usemap="#add-img-maps-1022-full"]')
   null
document.querySelector('img[usemap="add-img-maps-1022-full"]')`
   <img width="580" height="300" src="http://orwell/blog/wp-content/uploads/2013/03/featured-image-horizontal.jpg" class="attachment-post-thumbnail size-post-thumbnail wp-post-image" alt="Featured Image (Horizontal)" usemap="add-img-maps-1022-full">

I'm guessing some browsers are more forgiving of this departure than others. Here's the full log dump:
imageMapResizer.js?ver=0.1.0:49 Uncaught TypeError: Cannot read property 'addEventListener' of null
    at addEventListeners (imageMapResizer.js?ver=0.1.0:49)
    at HTMLMapElement.scaleImageMap (imageMapResizer.js?ver=0.1.0:74)
    at Function.each (jquery.js?ver=1.12.4:2)
    at a.fn.init.each (jquery.js?ver=1.12.4:2)
    at a.fn.init.$imageMapResizeF [as imageMapResize] (imageMapResizer.js?ver=0.1.0:133)
    at add-img-maps-public.js?ver=0.1.0:14
    at dispatch (jquery.js?ver=1.12.4:3)
    at r.handle (jquery.js?ver=1.12.4:3)
addEventListeners @ imageMapResizer.js?ver=0.1.0:49
scaleImageMap @ imageMapResizer.js?ver=0.1.0:74
each @ jquery.js?ver=1.12.4:2
each @ jquery.js?ver=1.12.4:2
$imageMapResizeF @ imageMapResizer.js?ver=0.1.0:133
(anonymous) @ add-img-maps-public.js?ver=0.1.0:14
dispatch @ jquery.js?ver=1.12.4:3
r.handle @ jquery.js?ver=1.12.4:3

Just had a look at this and I get the exact opposite, if I take it out it then fails!

I've made it try both, first with # and then if nothing found without it.
v1.0.6


Retrieve original size image

I am not able to extract original image size. The image gets resized when i debug it inside the delegate method. Any solution for this?

Have you tried using the callback with the PHAsset? That will give you more control. Here is the Swift function below. You can use the selectAssets:(([PHAsset]) -> Void) closure to convert to UIImage and also get additional information about the Photo/Video.
presentOpalImagePickerController(_ imagePicker: OpalImagePickerController, animated: Bool, maximumSelectionsAllowed: Int = 10, numberOfExternalItems: Int, externalItemsTitle: String, externalURLForIndex: @escaping (Int) -> URL?, selectImages: (([UIImage]) -> Void)? = nil, selectAssets: (([PHAsset]) -> Void)? = nil, selectExternalURLs: (([URL]) -> Void)? = nil, cancel: @escaping () -> Void, completion: (() -> Void)? = nil)