Cleanup secure-element-se05x page

Applied style guide rules and made minor changes for readability.
Built and checked page for correct rendering, edited with linter and spellcheck on.
This commit applies to FFTK-2795
Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Only minor changes made
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[x] Run spelling and grammar check, preferably with linter.
[x] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[x] Review for wordiness
[x] Match tone and style of page/section.
[ ] Run make linkcheck.
[x] View HTML in a browser to check rendering.
[x] Use semantic newlines.
[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

[x] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for ac56c15e5ec95cc9327f09a5aedb23e3e977e68a are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2585/docs/artifacts/html/index.html


fix(ci): Fixing broken Domains Test

Summary
The domains test was flaky due to changes in how search ranking is performed. This change addresses the issue. It also makes the test more resilient to re-runs by generating a unique urn on each run.
Status
Ready for review
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

due to changes in how search ranking is performed

Out of curiosity, what exactly was going wrong?


Add boot to hip builder

Fixes #
Adding boot to HIP builder

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)(=)=InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a958dc498b76a5a92077da9532960199d10f6ddb628HPX Datetime2022-05-31T12:57:29+00:002022-06-21T14:28:53+00:00Hostnamenid01193nid00375EnvfileDatetime2022-05-31T15:13:01.357969+02:002022-06-21T16:41:14.285410+02:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch(=)InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a958dc498b76a5a92077da9532960199d10f6ddb628HPX Datetime2022-05-31T12:57:29+00:002022-06-21T14:28:53+00:00Hostnamenid01193nid00375EnvfileDatetime2022-05-31T15:13:18.026239+02:002022-06-21T16:41:31.116255+02:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)-(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01a8dc498b76a5a92077da9532960199d10f6ddb628HPX Datetime2021-11-10T19:14:21+00:002022-06-21T14:28:53+00:00Hostnamenid00120nid00375EnvfileDatetime2021-11-10T20:28:18.266961+01:002022-06-21T16:41:46.480515+02:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Explanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

@akheir Boost is now found ok. The new problem is related to libatomic not being linked against the executables. I'll have a look.

Also, FWIW, the CUDA builders are fine now as well.

retest lsu

The HIP problem is fixed now.


Users can't unlink a glossaries terms from their object

Affected module
backend, UI
Describe the bug
I have limited the rights of users to change the glossary term (DENY rule for Edit All). At the same time, users can link this term to their objects - this is acceptable behavior.
But if users want to unlink this term from their objects, they can't do it.
To Reproduce

forbid the user to change the term (DENY rule for Edit All)
link this term to the object that the user owns
try to unlink this term

Expected behavior
It is necessary that users can unlink terms from their objects (as well as link) regardless of the rights to change terms.
Version:

OpenMetadata version: 1.2.3

@chirag-madlani lets verify and close it out


Fix code style

Ah thanks! That's what I get for trying to make changes in the GitHub browser editor 😄


ask

how can i change default dark mode color? i wanna change to bg-gray-900 like previous version.

As in change the background color of the dark mode? It's in this line: https://github.com/timlrx/tailwind-nextjs-starter-blog/blob/master/pages/_document.js#L37
It is already set to bg-gray-900. If you just want to change the gray palette, you can change this line: https://github.com/timlrx/tailwind-nextjs-starter-blog/blob/master/tailwind.config.js#L23

Look into tailwind's config.

Alright thank you very much 🙏🙏


Fix default pnpm workspace ignores

Going to defer to @nathanhammond on this one, but it looks like we're explicitly testing for the current behavior. Even if we don't proceed with this particular change, we definitely need a comment as to why we have the current behavior.

I believe that the "pnpm ignores **/tests/**" is actually a red herring. From what I can tell it doesn't ignore **/tests/** at invocation, only the library code that they use has that as a default ignore. If we trace through pnpm's invocation:

This is the code that looks like it is adding **/tests/** to the pnpm ignores..
But it only uses those conditionally.
And the function call site always passes in a different set of options, not including **/tests/**.


Add Fixed Mode Hide Chat plugin

Moved from my original PR: https://github.com/runelite/runelite/pull/3566
Preview:

Signed-off-by: Tomas Slusny<EMAIL_ADDRESS>

Fixed the typo


Python 2.7.12rc1

Do we want to get a tag going for the rc? Or just wait until GA?
https://www.python.org/downloads/release/python-2712rc1/

Given that it's not very easy for us to autodiscover these, I'm leaning towards a -1 unless there's definite demand for it (especially since this is "caveman Python" :smile:)

Yeah, that's why I opened issue rather than submit a PR for it. :)
I'll rename this for 2.7.12 release and I'll get PR for that when time comes.

Sounds good! :+1:


Should Detect not synced up node

I think mwc713  should provide more info about the node.
I am trying to send some funds from mwc713 to mwc-wallet  with file.
I did that and there was no error messages.
mwc713 said that transaction is confirmed. But mwc-wallet printed that funds awaiting finalization.
The reason was that for the node it takes time to download all blocks. It is not clean that it is not done.
NodeInfo command prints Height for itself and for peers.  mwc-qt-wallet need to compare those values and print in status "Syncing".
Also we should at start check the node status and emit critical level message and maybe show Message Box that explain that users can't trust transaction/output status.

Also, one thing to understand is that there is no single source of truth.
That's part of how cryptocurrency works. According to the node that mwc713
was connected to (mwc713.mwc.mw which is a very well connected and always
synced node) the transaction is confirmed, but according to your node
(which was not synced yet) the transaction is not confirmed. This is
correct and expected behavior. There's really no way that mwc713 in this
case would know that the receiver's node is not synced since it doesn't
know which node the receiver is connected to.
On Sat, Aug 24, 2019 at 5:05 PM bayk<EMAIL_ADDRESS>wrote:

I think mwc713 should provide more info about the node.
I am trying to send some funds from mwc713 to mwc-wallet with file.
I did that and there was no error messages.
mwc713 said that transaction is confirmed. But mwc-wallet printed that
funds awaiting finalization.
The reason was that for the node it takes time to download all blocks. It
is not clean that it is not done.
NodeInfo command prints Height for itself and for peers. mwc-qt-wallet
need to compare those values and print in status "Syncing".
Also we should at start check the node status and emit critical level
message and maybe show Message Box that explain that users can't trust
transaction/output status.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mwcproject/mwc-qt-wallet/issues/50?email_source=notifications&email_token=ABXFVN2BCRFPIAKAVCCMV6DQGHED3A5CNFSM4IPHRG2KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HHGTDYA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABXFVNY6D3MM4Y4ICERSITTQGHED3ANCNFSM4IPHRG2A
.

Yes, behavior is expected, but it is hard to spot it with wallet.
This issue is about detection of that fact that peer node has much higher block height. It is mean that node not in sync yet. That is why user should expect that transactions are not confirmed because of that.

What we could do if your full node is still in the "syncing" state it reports this to mwc713 and a warning is printed. Another message can be printed when syncing is done.

Messages at message box are not user friendly. I think we can eliminate them.
If node out of sync at the start, we can  start from the Node Page, so user will see that node status - out of sync.
If node was at sync and go out of sync during the session. We will print the regular warning. This message in not different from others that mwc713 emitting.
I think that will be more user friendly.

I meant on mwc713. These can be processed by mwc-qt-wallet as you mentioned.
On Sat, Aug 24, 2019, 7:29 PM bayk<EMAIL_ADDRESS>wrote:

Messages at message box are not user friendly. I think we can eliminate
them.
If node out of sync at the start, we can start from the Node Page, so user
will see that node status - out of sync.
If node was at sync and go out of sync during the session. We will print
the regular warning. This message in not different from others that mwc713
emitting.
I think that will be more user friendly.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mwcproject/mwc-qt-wallet/issues/50?email_source=notifications&email_token=ABXFVN6IADRWY4CWRD2MEFDQGHVB5A5CNFSM4IPHRG2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CK6BI#issuecomment-524594949,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABXFVNYVH27JDQGQTCVD3KLQGHVB5ANCNFSM4IPHRG2A
.


Request for instructions

I'm trying to use the code for my own project, but I do not know which is the function needed to be called or where is the generated uuid located. Can we get a short step-by-step or shot explanation on how to use it?

You probably figured out, but this is the function and it returns the uuid-v4 as a String
https://github.com/binogure-studio/godot-uuid/blob/master/uuid.gd#L23
But I agree with some explanations 👍

Thank you for the report.
It's been fixed


camera pose

Are the new perspective images rendered in blender? Can you provide a specific rotation matrix to facilitate calculation of the camera pose?

I try to render in blender and got the following calculation method for the new camera pose. I don’t know if this is correct. Of course, I know that if you use blender to render directly, you don't actually need a specific camera pose, you just need to set it directly in blender. But this is very important for my downstream tasks.

Sorry but I don't quite understand your question.
The training data is rendered in blender.


barakmesika1 gds file

added my gds file

added my gda file v1

added my gds file v1

uploading a newer version


Command line cleanup rework

Fixed version of #4701

@hardening could you check and merge this? I've only fixed the memory leak @informatimago added in the command line parsing.


[protobuf] Update to 3.7.0

Tested with:

[x] x86-windows
[x] x86-windows-static
[x] x64-windows
[x] x64-windows-static
[x] x64-linux

Test project: https://github.com/myd7349/Ongoing-Study/tree/master/cpp/protobuf/person_example

@myd7349, thanks for contributing to vcpkg!
Here is fail triplets(x86-windows and x64-windows ) and regression(3fd) with your PR.
 Processing x64-windows                                          849 vs 850
     3fd                           **regression**                Fail vs Pass
 Processing x86-windows                                          829 vs 830
     3fd                           **regression**                Fail vs Pass

Hi! @PhoebeHui Thanks for your reply.
I am a bit confused about the CI result. Does it mean that after this update the installation of google-cloud-cpp and orc will fail?
I will check it out.

any update on this ?

I tried to install google-cloud-cpp on my PC and it failed:

E:\vcpkg\buildtrees\google-cloud-cpp\x86-windows-dbg\external\nlohmann_json\include\google/cloud/storage/internal/nlohmann_json.hpp(3605): error C2039: '_strtoui64': is not a member of 'std'

I have created a PR to fix this problem: https://github.com/protocolbuffers/protobuf/pull/5864

@myd7349, thanks for the updates! Check latest cl results, orc failed with x64-osx triplet:
 Processing arm-uwp                        412 vs 412
 Processing arm64-windows                  472 vs 472
 Processing x64-linux                      617 vs 617
 Processing x64-osx                        609 vs 610
     orc                           **regression**                Fail vs Pass
 Processing x64-uwp                        440 vs 440
 Processing x64-windows                    886 vs 886
 Processing x64-windows-static             780 vs 780
 Processing x86-windows                    866 vs 866

OK. I will try to resolve it.

orc build error on osx: install-x64-osx-dbg-out.log
Seems an known issue: https://github.com/protocolbuffers/protobuf/pull/5881

So here is the case:
My patch is applied to master branch but not v3.7.x.
protocolbuffers/protobuf#5881 is applied to v3.7.x branch but not master.

After some thoughts, I decided to switch the HEAD_REF to 3.7.x in that https://github.com/protocolbuffers/protobuf/pull/5864 can be fixed temporarily with a small patch but https://github.com/protocolbuffers/protobuf/pull/5881/files needs more patching work.
By the way, I noticed that orc has been updated by this PR: https://github.com/Microsoft/vcpkg/pull/4786 yesterday. And it seems there are some building errors on Windows with this new version of orc.

Hey @myd7349. I believe that HEAD_REF in this case should continue to point to master. The reason being that when users specify the --head option, they should always be given the latest version of regardless of its validity.
After looking over the triplet tables to see the status of orc, it seems that it's building fine on Windows right now. Are you able to reproduce this?

Hey @myd7349, I've updated the REF to point to the latest commit as opposed to the merge commit as we prefer to take the latest when possible

Hi! @grdowns I have updated this PR.

Thanks for the updates! All triplets built successfully.

OK. I think I should update it to 3.7.1

Shall I squash these commits to keep the commit message clean?


Provide arm/arm64 images

Please provide ARM/ARM64 images to be able to run kraan on ARM architecture compute.

@kishoregv Begin v0.3.11, Kraan will support both arm64 and amd64. Please try the latest version. Thank you.


Status Code 422 When Publishing

Managed to get the configuration working and know that it's talking to my repository (part of a GitHub Organization) - as I had to work through a few 401s (Auth Issues, resolved with Personal Access Token) and 404s (Action / Workflow file name was wrong).
I am however completely stuck now though as when I click the "Publish" button on the GitHub Publishing page, I'm getting a Status Code 422 failure:
[2021-06-23T02:39:34.698Z] error Error: Request failed with status code 422     at createError (/home/ubuntu/strapiV2/node_modules/axios/lib/core/createError.js:16:15)     at settle (/home/ubuntu/strapiV2/node_modules/axios/lib/core/settle.js:17:12)     at IncomingMessage.handleStreamEnd (/home/ubuntu/strapiV2/node_modules/axios/lib/adapters/http.js:260:11)     at IncomingMessage.emit (events.js:326:22)     at endReadableNT (_stream_readable.js:1241:12)     at processTicksAndRejections (internal/process/task_queues.js:84:21)
I have tried this both locally (OS X) and published out on a server (AWS EC2 - Ubuntu 20.04), still getting this error.  Can I get some advice, as I'm not sure where else to look to resolve this?  Thanks!

I finally got around to looking at this, and the issue resulted in how the workflow config file was setup (.yml).  In order for this plug-in to initiate the workflow, I had to ensure the event (on:) was setup as follows ... Basically needed to ensure that you have the ability to manually trigger the workflow.
on:   workflow_dispatch:   repository_dispatch:
I didn't see this anywhere in the documentation, so wanted to at least post this here in the event someone else ran across this.  Would recommend that this is included in the README for others.
More on GitHub manual workflows: https://docs.github.com/en/actions/reference/events-that-trigger-workflows#manual-events

@stickchartkeith Thanks for this.  I ran into the same problem.
Even after your fix, I was still getting a 422 error.  I additionally needed to remove this section from my strapi config:
    inputs: {
      // Optional inputs to pass through to the GitHub workflow
      some_input: 'Some value',
      some_other_input: 'Some other value',
     },

I had copied these inputs from the documentation for this plugin.  I wasn't using the values, and assumed they'd just be silently ignored.  Turns out including them was preventing the github action from working.

I finally got around to looking at this, and the issue resulted in how the workflow config file was setup (.yml). In order for this plug-in to initiate the workflow, I had to ensure the event (on:) was setup as follows ... Basically needed to ensure that you have the ability to manually trigger the workflow.
on: workflow_dispatch: repository_dispatch:
I didn't see this anywhere in the documentation, so wanted to at least post this here in the event someone else ran across this. Would recommend that this is included in the README for others.
More on GitHub manual workflows: https://docs.github.com/en/actions/reference/events-that-trigger-workflows#manual-events

this worked for me, thanks @stickchartkeith


One plus 7 plus. Slipping heaps

Goggles v1
Phone: oneplus 7 plus
Android 11
Video skips and will no run smooth
https://photos.app.goo.gl/FpoT9uKQhfG2gBcx7

Please put this in issue #12 🙂 I think other people reported it works ok on their OnePlus 7, so it might be a cable issue.

we used the same cable & goggles on everyone elses phone and it worked seemless. will try again next weekend. I might of opened the app first


[AutoPR automation/resource-manager] Remove 4xx responses and change schema to file

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3363

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/2220


use docker hostnames

Since we are using docker-compose we don't have to fix the IP.

Thanks for providing this PR

Thank you for providing this setup. It was very helpful !


[vulnerability][fix]: Add unordered transaction inputs

Description
From #2279,

The wallet uses ordered inputs of transactions (src/wallet/wallet.cpp, line 2628 - 2640), which may incur privacy risks like fingerprinting the wallet to observers.

This PR change the original order by shuffling the inputs.

Merging this into 1.14.4 will immediately identify all 1.10.x and 1.14.x based wallets that may have vulnerabilities that we fix in 1.14.4 and any future releases. I would prefer to not merge this, but instead fix all the vulnerabilities as long as we have an active 1.14-maint, and only introduce this change in wallet behavior with 1.21. That way, we do not directly link wallet behavior to potential vulnerabilities.
Thoughts?

Sure. Should we start merging them on 1.21-dev (including some refactoring PRs)?

Should we start merging them on 1.21-dev (including some refactoring PRs)?

Already part of 1.21-dev: https://github.com/dogecoin/dogecoin/commit/fd44ac1e8b

+1

Restarting CI

Ran a bundle of transactions with this on testnet, seems good as far as I can tell.

I'm still NAK on this because I heard no argument against my assertion up there

@patricklodder I agree with you.

Merging this into 1.14.4 will immediately identify all 1.10.x and 1.14.x based wallets that may have vulnerabilities that we fix in 1.14.4 and any future releases. I would prefer to not merge this, but instead fix all the vulnerabilities as long as we have an active 1.14-maint, and only introduce this change in wallet behavior with 1.21. That way, we do not directly link wallet behavior to potential vulnerabilities.
Thoughts?


chore: publish resource and navigation events to bus

Rewrite of #440

Purpose
This PR creates the structure necessary for sharing information between plugins.
Uses cases:

Have PluginA find eventId's created by PluginB etc. Useful for LCP attribution
WebVitalsPlugin tracks current count of in-flight Xhr and Fetch requests

Revisions
Revision 1
I have created EventBus with topic-based filtering. I implemented our own instead of taking on a dependency because we only need a few extremely simple features.

Plugins can subscribe to any topic via context.bus.subscribe(topic, callback)
Plugins can publish to any topic via context.bus.dispatch(topic, message)
All events are published on every record by EventCache via bus.dispatch(eventType, message)

Revision 2

Plugins are responsible for dispatching events
EventCache is not responsible for dispatching events
EventCache.record returns a ParsedRumEvent so plugins can access things like eventId, metadata, details, etc.
Read this thread for full context https://github.com/aws-observability/aws-rum-web/pull/445#discussion_r1317539603

Examples
Use case 1: PluginA finds eventId's of events from PluginB
class WebVitalsPlugin {
   imageEvents = []
   constructor() {
      this.context.bus.subscribe(PERFORMANCE_RESOURCE_EVENT, this.storeResourceEvent)
   }
   storeResourceEvent(event) {
      if (event.fileType === ResourceTypes.IMAGE) {
         this.imageEvents.push(event)
      }
   }

   ...
   onLCP(payload => {
       ...
       findMatchingLCPEvent()
       // teardown
       this.imageEvents = []
       this.context.bus.unsubscribe(...)
   })
}

Use case 2: Count in-flight fetch/xhr requests
// FetchPlugin.ts
...
private fetch() {
   ...
   this.context.bus.dispatch('http_open_count', {payload: 1})
   return original()
   .apply()
   .then(() => {
        this.context.bus.dispatch('http_open_count', {payload: -1})
    })
   .catch(() => {
        this.context.bus.dispatch('http_open_count', {payload: -1})
    })
}

...

// WebVitalsPlugin
class WebVitalsPlugins {
    ...
    http_open_count = 0
    handleHttpCountChange({payload}) { this.http_open_count += payload }
    constructor() {
        this.context.bus.subscribe('http_open_count', this.handleHttpCountChange)
   }
   teardown() {
       this.context.bus.unsubscribe('http_open_count', this.handleHttpCountChange)
   }
   ....
}

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

high level question: could the addition and usage of an event bus impact the memory/performance footprint of the web client on web applications?

high level question: could the addition and usage of an event bus impact the memory/performance footprint of the web client on web applications?

We are currently only monitoring the size and relying design principles to limit memory and cpu usage. I have also been thinking about how to objectively monitor this performance.
But from a design perspective, I think the impact is minimal.

My thoughts on (1) vs (2):
If doing (1), this topic allows any plugin to listen for any event. Publish all events under an ‘events’ topic.
If doing (2), this topic is for the specific purpose of tracking resource/nav event Ids.  Return the event id from recordEvent and publish an <event id, url> tuple.


Localization not working properly?

Hi, thanks for a neat library.
I have placed the Localizable.strings from the English and Danish under a Supporting Files group, and I can see they have been added to en.lproj and da.lproj. I can see they are picked up:

The pop-up does not seem to be translated however, it will just say "takePhoto" for example. I know the Localizable.strings files are working, because I have this line
label.text = NSLocalizedString("loading", comment: "The loading message when pulling data")

and that changes correctly if I choose Danish or English.
Any idea if I am doing something wrong here? I copied them into the project, no reference link. I do not use CocoaPods. Content of the English Localizable.strings:
/* Decline to proceed with operation */
"cancel" = "Cancel";

/* Option to select photo from library */
"chooseFromLibrary" = "Choose from library";

/* Option to select photo from photo roll */
"chooseFromPhotoRoll" = "Choose from photo roll";

/* There are no sources available to select a photo */
"noSources" = "There are no sources available to select a photo";

/* Option to take photo using camera */
"takePhoto" = "Take photo";

/* Option to take video using camera */
"takeVideo" = "Take video";

/* The loading message when pulling data */
"loading" = "Loading...";

Thank you.

Could you please confirm first that the demo app is working for you in Danish?

Running the app:

Please note "Operatør" in the top left, changed language to Danish inside the simulator (and scheme too just to be sure). It should have been "Vælg fra kamerarulle" instead of "chooseFromLibrary", right?

Thank you, I am still looking at this.
Does anyone else have advice on how to fix this?

I'm sorry for having the breaking bug in here so long.
The cocoapods updates are killing me and the normal approach of "pod lib create FDTake then start from scratch" each time did not work this time.

Was fixed and then broken again in Swift 3.0. Help requested, new issue at #100.


unsubscribe all subscriptions

Hello,
it may be useful (to avoid to do it on the user/dev side to be able to unsubscribe all subscriptions (before disconnecting)
def unsubscribe(self, subcription_key=None):
    """Unregister the Subscription associated to the
    specified subscription_key.
    if no subcription_key is given it unsubscribe all.
    """
    if subcription_key is None:
        subscriptions = self._subscriptions.copy() # To avoid a RuntimeError: dictionary changed size during iteration
        for subcription_key in subscriptions:
            self._unsubscribe(subcription_key)
    else:
        self._unsubscribe(subcription_key)

So instead of keeping sub_key and do
lightstreamer_client.unsubscribe(sub_key)

we will be able to do
lightstreamer_client.unsubscribe()

Kind regards

Hi,
such feature would not be compliant with the actual Lightstreamer Unified Client API model.
Please have a look here
Anyway, in the near future we will take in consideration your proposal in order to extend the API model. Thanks a lot for your suggestion!

Hi,
So that's probably better to call it unsubscribe_all ?

Hi,
the problem is not related to name of the method, but to the feature itself, which is not provided by the current Unified API.

Ok understood


Change bash scripts permissions

What type of PR is this?
bug
What this PR does / Why we need it?
AppSRE build pipeline is failing as it cannot read/execute the bash scripts in the repo. This PR updates the file permissions to 755.

/lgtm


Make minishift/login-docker-registry is brittle

I think it probably needs some sleep...
I regularly see the error below. But not every time I make minishift. Error is at the end.

`$make minishift
downloading minishift
curl -L https://github.com/minishift/minishift/releases/download/v1.15.1/minishift-1.15.1-linux-amd64.tgz | tar xzC local-dev/minishift --strip-components=1
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   622    0   622    0     0    590      0 --:--:--  0:00:01 --:--:--   590
100 5360k  100 5360k    0     0  1170k      0  0:00:04  0:00:04 --:--:-- 2161k
starting minishift with name lagoon
./local-dev/minishift/minishift --profile lagoon start --cpus 6 --memory 2GB --disk-size 30GB --vm-driver virtualbox --openshift-version="v3.7.2"
-- Starting profile 'lagoon'
-- Checking if requested OpenShift version 'v3.7.2' is valid ... OK
-- Checking if requested OpenShift version 'v3.7.2' is supported ... OK
-- Checking if requested hypervisor 'virtualbox' is supported on this platform ... OK
-- Checking if VirtualBox is installed ... OK
-- Checking the ISO URL ... OK
-- Checking if provided oc flags are supported ... OK
-- Starting local OpenShift cluster using 'virtualbox' hypervisor ...
-- Minishift VM will be configured with ...
Memory:    2 GB
vCPUs :    6
Disk size: 30 GB
-- Starting Minishift VM ............................. OK
-- Checking for IP address ... OK
-- Checking for nameservers ... OK
-- Checking if external host is reachable from the Minishift VM ...
Pinging <IP_ADDRESS> ... OK
-- Checking HTTP connectivity from the VM ...
Retrieving http://minishift.io/index.html ... OK
-- Checking if persistent storage volume is mounted ... OK
-- Checking available disk space ... 0% used OK
Importing 'openshift/origin:v3.7.2' ........... OK
Importing 'openshift/origin-docker-registry:v3.7.2' .. OK
Importing 'openshift/origin-haproxy-router:v3.7.2' .. OK
-- OpenShift cluster will be configured with ...
Version: v3.7.2
Starting OpenShift using openshift/origin:v3.7.2 ...
OpenShift server started.
The server is accessible via web console at:
https://<IP_ADDRESS>:8443
You are logged in as:
User:     developer
Password:
To login as administrator:
oc login -u system:admin
-- Applying addon 'admin-user':..
replacing IP in local-dev/api-data/api-data.gql and docker-compose.yaml with the IP '<IP_ADDRESS>'
eval $(./local-dev/minishift/minishift --profile lagoon oc-env);
oc login -u system:admin;
bash -c "echo '{"apiVersion":"v1","kind":"Service","metadata":{"name":"docker-registry-external"},"spec":{"ports":[{"port":5000,"protocol":"TCP","targetPort":5000,"nodePort":30000}],"selector":{"docker-registry":"default"},"sessionAffinity":"None","type":"NodePort"}}' | oc --context="default/$(./local-dev/minishift/minishift --profile lagoon ip | sed 's/./-/g'):8443/system:admin" create -n default -f -";
oc --context="default/$(./local-dev/minishift/minishift --profile lagoon ip | sed 's/./-/g'):8443/system:admin" adm policy add-cluster-role-to-user cluster-admin system:anonymous;
oc --context="default/$(./local-dev/minishift/minishift --profile lagoon ip | sed 's/./-/g'):8443/system:admin" adm policy add-cluster-role-to-user cluster-admin developer;
Logged into "https://<IP_ADDRESS>:8443" as "system:admin" using existing credentials.
You have access to the following projects and can switch between them with 'oc project ':
default
kube-public
kube-system

myproject
openshift
openshift-infra
openshift-node

Using project "myproject".
service "docker-registry-external" created
cluster role "cluster-admin" added: "system:anonymous"
cluster role "cluster-admin" added: "developer"
wait 60secs in order to give openshift time to setup it's registry
sleep 60
eval $(./local-dev/minishift/minishift --profile lagoon oc-env);
for i in {10..30}; do oc --context="default/$(./local-dev/minishift/minishift --profile lagoon ip | sed 's/./-/g'):8443/system:admin" patch pv pv00${i} -p '{"spec":{"storageClassName":"bulk"}}'; done;
persistentvolume "pv0010" patched
persistentvolume "pv0011" patched
persistentvolume "pv0012" patched
persistentvolume "pv0013" patched
persistentvolume "pv0014" patched
persistentvolume "pv0015" patched
persistentvolume "pv0016" patched
persistentvolume "pv0017" patched
persistentvolume "pv0018" patched
persistentvolume "pv0019" patched
persistentvolume "pv0020" patched
persistentvolume "pv0021" patched
persistentvolume "pv0022" patched
persistentvolume "pv0023" patched
persistentvolume "pv0024" patched
persistentvolume "pv0025" patched
persistentvolume "pv0026" patched
persistentvolume "pv0027" patched
persistentvolume "pv0028" patched
persistentvolume "pv0029" patched
persistentvolume "pv0030" patched
make minishift/configure-lagoon-local push-docker-host-image
make[1]: Entering directory '/home/jeff/projects/lagoon-learning/lagoon'
if ! which oc; then eval $(./local-dev/minishift/minishift --profile lagoon oc-env); fi;
oc new-project lagoon;
oc adm pod-network make-projects-global lagoon;
oc -n lagoon create serviceaccount openshiftbuilddeploy;
oc -n lagoon create -f openshift-setup/clusterrole-openshiftbuilddeploy.yaml;
oc -n lagoon adm policy add-cluster-role-to-user openshiftbuilddeploy -z openshiftbuilddeploy;
oc -n lagoon create -f openshift-setup/shared-resource-viewer.yaml;
oc -n lagoon create -f openshift-setup/policybinding.yaml | oc -n lagoon create -f openshift-setup/rolebinding.yaml;
oc -n lagoon create serviceaccount docker-host;
oc -n lagoon adm policy add-scc-to-user privileged -z docker-host;
oc -n lagoon policy add-role-to-user edit -z docker-host;
oc -n lagoon create serviceaccount cronjob;
oc -n lagoon policy add-role-to-user edit -z cronjob;
bash -c "oc process -n lagoon -f openshift-setup/docker-host.yaml | oc -n lagoon apply -f -";
bash -c "oc process -n lagoon -f openshift-setup/docker-host-cronjobs.yaml | oc -n lagoon apply -f -";
echo -e "\n\nAll Setup, use this token as described in the Lagoon Install Documentation:"
oc -n lagoon serviceaccounts get-token openshiftbuilddeploy
/home/jeff/.local/bin/oc
Now using project "lagoon" on server "https://<IP_ADDRESS>:8443".
You can add applications to this project with the 'new-app' command. For example, try:
oc new-app centos/ruby-22-centos7~https://github.com/openshift/ruby-ex.git
to build a new example application in Ruby.
error: managing pod network is only supported for openshift multitenant network plugin
See 'oc adm pod-network make-projects-global -h' for help and examples.
serviceaccount "openshiftbuilddeploy" created
clusterrole "openshiftbuilddeploy" created
cluster role "openshiftbuilddeploy" added: "openshiftbuilddeploy"
role "shared-resource-viewer" created
rolebinding "shared-resource-viewers" created
error: unable to recognize "openshift-setup/policybinding.yaml": no matches for /, Kind=PolicyBinding
serviceaccount "docker-host" created
scc "privileged" added to: ["system:serviceaccount:lagoon:docker-host"]
role "edit" added: "docker-host"
serviceaccount "cronjob" created
role "edit" added: "cronjob"
deploymentconfig "docker-host" created
service "docker-host" created
cronjob "cronjob-update-push-images" created
cronjob "cronjob-prune-images" created
All Setup, use this token as described in the Lagoon Install Documentation: oc -n lagoon serviceaccounts get-token openshiftbuilddeploy
eval $(./local-dev/minishift/minishift --profile lagoon oc-env);
bash -c "oc process -n lagoon -p IMAGE=docker-registry.default.svc:5000/lagoon/docker-host:latest -p REPOSITORY_TO_UPDATE=lagoon -f openshift-setup/docker-host-minishift.yaml | oc -n lagoon apply -f -";
bash -c "oc process -n lagoon -p IMAGE=docker-registry.default.svc:5000/lagoon/docker-host:latest -p REPOSITORY_TO_UPDATE=lagoon -f openshift-setup/docker-host-cronjobs.yaml | oc -n lagoon apply -f -";
deploymentconfig "docker-host" configured
service "docker-host" configured
cronjob "cronjob-update-push-images" configured
cronjob "cronjob-prune-images" configured
eval $(./local-dev/minishift/minishift --profile lagoon oc-env);
oc login --insecure-skip-tls-verify -u developer -p developer $(cat minishift):8443;
oc whoami -t | docker login --username developer --password-stdin $(cat minishift):30000
Login successful.
You have access to the following projects and can switch between them with 'oc project ':
default
kube-public
kube-system

lagoon
myproject
openshift
openshift-infra
openshift-node

Using project "lagoon".
Error response from daemon: Get http://<IP_ADDRESS>:30000/v2/: dial tcp <IP_ADDRESS>:30000: getsockopt: connection refused
Makefile:605: recipe for target 'minishift/login-docker-registry' failed
make[1]: *** [minishift/login-docker-registry] Error 1
make[1]: Leaving directory '/home/jeff/projects/lagoon-learning/lagoon'
Makefile:581: recipe for target 'minishift' failed
make: *** [minishift] Error 2
`

However, immediately following that with:

`$make minishift/login-docker-registry
eval $(./local-dev/minishift/minishift --profile lagoon oc-env);
oc login --insecure-skip-tls-verify -u developer -p developer $(cat minishift):8443;
oc whoami -t | docker login --username developer --password-stdin $(cat minishift):30000
Login successful.
You have access to the following projects and can switch between them with 'oc project ':
default
kube-public
kube-system

lagoon
myproject
openshift
openshift-infra
openshift-node

Using project "lagoon".
Login Succeeded
`

This suggests to me that a small sleep might help. Perhaps 1 or 2 seconds before the body of minishift/login-docker-registry?

minishift/login-docker-registry is deprecated in Lagoon 2


error happen when run the first step to convert model

System Info
model qwen-chat-7b/qwen-chat-7b-int/qwen1_8B
Who can help?
@n
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
run following script:
nohup python build.py --hf_model_dir /home/wanghaikuan/.cache/huggingface/hub/models--Qwen--Qwen-7B-Chat/snapshots/62ee27f810e7f59905ee1dac4ccc86d593dd57f4/
--dtype bfloat16
--remove_input_padding
--use_gpt_attention_plugin bfloat16
--enable_context_fmha
--use_gemm_plugin bfloat16
--output_dir ./tmp/Qwen/18b/trt_engines/fp16/1-gpu/ > t5.log 2>&1 &
Expected behavior
convert to model successfully
actual behavior
[01/23/2024-21:08:32] [TRT] [W] Unused Input: position_ids
[01/23/2024-21:08:32] [TRT] [W] [RemoveDeadLayers] Input Tensor position_ids is unused or used only at compile-time, but is not being removed.
[01/23/2024-21:08:32] [TRT] [I] [MemUsageChange] Init cuBLAS/cuBLASLt: CPU +0, GPU +0, now: CPU 6058, GPU 12149 (MiB)
[01/23/2024-21:08:32] [TRT] [I] [MemUsageChange] Init cuDNN: CPU +2, GPU +0, now: CPU 6060, GPU 12149 (MiB)
[01/23/2024-21:08:32] [TRT] [I] Global timing cache in use.
Traceback (most recent call last):
File "/home/wanghaikuan/code/tt/TensorRT-LLM/examples/qwen/build.py", line 673, in
build(0, args)
File "/home/wanghaikuan/code/tt/TensorRT-LLM/examples/qwen/build.py", line 645, in build
assert engine is not None, f'Failed to build engine for rank {cur_rank}'
AssertionError: Failed to build engine for rank 0
additional notes
None

Thanks you @whk6688 for the feedback. May I ask which model are you running exactly, the Qwen-7B-Chat or Qwen-1_8B-Chat? If it is the 1.8B one, what is your build command exactly?

qwen-chat-7b/qwen-chat-7b-int/qwen1_8B  means,  no matter which one, the error is same.

would u please try our latest code base to see if the issue still exists?
And do u still have further issue or question now? If not, we'll close it soon.


Test cluster provisioning on AWS with Dual-Stack - Test Release KubeOne 1.6

Works as expected, tested for both in-cluster dual-stack services and ingress + egress dual-stack traffic.


was.rds.Cluster gets stuck in failing update state if shorthand engine_version string is supplied

I have encountered what seems like a bug in aws.rds.Cluster, at least with Aurora and in Python. If you specify an engine_version of, say '5.7' it will happily create the cluster for you. But from that point on, on any update it thinks that the cluster version has changed (ostensibly because AWS converts it to a fully qualified form in the resource) and tries to update it, and the update fails because the resource says that you are attempting to  change to the same version. It seems to only work properly if a fully qualified version string is supplied; e.g., '5.7.mysql_aurora.2.07.1'

This is - unfortunately - a behaviour of the upstream https://github.com/terraform-providers/terraform-provider-aws and of AWS itself.  They both are liberal in what they accept here, but then will show (benign) diffs if you don't fully specify what you really want.
The solution should be just to encode the specific version you do really want, or else add { ignoreChanges: ["engineVersion"] } to suppress any diffs related to this property after creation.
Hopefully that helps - if not let us know.


Fix concurrent writes in the update operation

Details

The update operation was written in such manner that it fired all database operations at once. Some of the operations internally rely on other promises, meaning they will get re-scheduled on another tick. This causes unpredictable behavior as you cannot guarantee the order of the operations. The solution is clear and is the only one, all DB writes must respect the order of the operations.
The incorrect ordering of the operations being committed to the DB causes other random behavior around the app as well. Linked the two issues.
Related Issues

https://github.com/Expensify/App/issues/28737
https://github.com/Expensify/App/pull/28584
Automated Tests

Added one more test that makes sure the correct data is persisted. It failed with the previous implementation, now it passes.
Linked PRs

No, I haven't tested the app. I can try to get some metrics but I already asked in slack if there is any particular call to update that is heavy.

I added some react-native-performance measures on app start, I cannot tell which one exactly is OpenApp but they are all basically negible. Writing to DB should not be the bottleneck, and even then, correctness is not negotiable.

OK, I just created a large account for testing things like this, so let me give it a try with this today.

Doing some testing now...

Alright well - it is defying my expectations a bit, but when timing the calls to Onyx.update() they are running way faster synchronously 😅
Sign in trials
With sync updates:
1
Timing:development.new.expensify.flushQueue 458
Timing.js:49 Timing:development.new.expensify.flushQueue 0
Timing.js:49 Timing:development.new.expensify.flushQueue 407
2
Timing:development.new.expensify.flushQueue 468
Timing.js:49 Timing:development.new.expensify.flushQueue 1
Timing.js:49 Timing:development.new.expensify.flushQueue 191
3
Timing:development.new.expensify.flushQueue 311
Timing.js:49 Timing:development.new.expensify.flushQueue 0
Timing.js:49 Timing:development.new.expensify.flushQueue 226
Without sync updates (but on latest react-native-onyx):
1
Timing:development.new.expensify.flushQueue 488
Timing.js:49 Timing:development.new.expensify.flushQueue 4147
Timing.js:49 Timing:development.new.expensify.flushQueue 962
2
Timing:development.new.expensify.flushQueue 420
Timing.js:49 Timing:development.new.expensify.flushQueue 1126
Timing.js:49 Timing:development.new.expensify.flushQueue 170
3
Timing:development.new.expensify.flushQueue 464
Timing.js:49 Timing:development.new.expensify.flushQueue 4729
Timing.js:49 Timing:development.new.expensify.flushQueue 976
This was a web-only test. Haven't tested on iOS or Android - but looks promising.


match operator / pattern matching on functions

Hello,
do you think doing something like https://github.com/natefaubion/sparkler#how-does-it-work would be possible with babel-plugin-tcomb ?
Thanks for the hard work !

Closing this since this plugin is moving to support flow


Add IPv4 mode for test servers & clients

The changes add an IPv4 mode to connection.c/.h as well as a
corresponding command-line parameter -4 to lwm2mclient.c, as outlined in previous pull request #83 .
I updated the implementation to pass ai_family as parameter instead of using a global. Also, I ported the changes to dtlsconnection.c/.h, and added the same command-line option to the server, lightclient, secure client and bootstrap server.
Note: There is one issue I encountered: It is not possible to use "localhost" as server address in IPv4 mode on my system. Instead, "<IP_ADDRESS>" needs to be specified explicitly.
The underlying root cause is using AF_UNSPEC in the hints structure when looking up the server address in connection_create(). getaddrinfo on my OS X system will return [::1] as first IP address for "localhost" in this case. The way to fix this would be to use either AF_INET or AF_INET6 depending on the mode of the client, but I'd still need to investigate a bit to find an elegant way to bring this information into connection_create().

Note: There is one issue I encountered: It is not possible to use "localhost" as server address in
IPv4 mode on my system. Instead, "<IP_ADDRESS>" needs to be specified explicitly.

So can you please give us some overview about your passed (and failed) test scenarios including the used OS for the tests?

May be adding the address familiy also to
connection_create()

will help to solve that issue.
If so, please obey, that the function is called from a "callback". Usually a callback should access additional data via a "context", which is passed in using lwm2m_init.
So extending e.g. "client_data_t" with the address familiy would be a good way (from my view).

So can you please give us some overview about your passed (and failed) test scenarios including the used OS for the tests?

The OS is OS X 10.11.3, with standard network settings (i.e., IPv6 enabled) and an IPv4 connection to the outside world. Here are my results:
Current master, local lwm2mserver:
./lwm2mclient                          # Successfully connects to local server (as expected)
./lwm2mclient -h localhost             # Successfully connects to local server (as expected)
./lwm2mclient -h ::1                   # Successfully connects to local server (as expected)
./lwm2mclient -h <IP_ADDRESS>             # Fails to connect to local server (as expected)
./lwm2mclient -h leshan.eclipse.org    # Fails to connect to leshan server (as expected, no IPv6 connection)
Pull request #83, local lwm2mserver in IPv6 mode:
./lwm2mclient                          # Successfully connects to local server (as expected)
./lwm2mclient -h localhost             # Successfully connects to local server (as expected)
./lwm2mclient -h ::1                   # Successfully connects to local server (as expected)
./lwm2mclient -h <IP_ADDRESS>             # Fails to connect to local server (as expected)
./lwm2mclient -h leshan.eclipse.org    # Fails to connect to leshan server (as expected, no IPv6 connection)
I.e. - test results are exactly the same as for current master
Pull request #83, local lwm2mserver in new IPv4 mode (lwm2mserver -4):
./lwm2mclient -4                       # Successfully connects to local server (as expected)
./lwm2mclient -4 -h localhost          # Fails to connect to local server this is the case I am talking about above
./lwm2mclient -4 -h ::1                # Fails to connect to local server (as expected)
./lwm2mclient -4 -h <IP_ADDRESS>          # Successfully connects to local server (as expected)
./lwm2mclient -4 -h leshan.eclipse.org # Successfully connects to leshan server (as expected)

@crenz
Thank's for the tests! I hope this helps a lot of user and developer to use it proper.

It looks much better now. But it looks like a Java code =) What do you think about following snippet?

Thanks for your suggestion, I updated the code.

Thank you. Could you rebase your branch and squash all commits into one?

Rebased & squashed. Still not familiar with all git intricacies, hopefully I did the right thing (I followed these instructions: https://github.com/edx/edx-platform/wiki/How-to-Rebase-a-Pull-Request )

Everything is cool. I just want to note about ai_family. Look around. There are a lot of variables, whose names are corresponded to camelCase convention.

Everything is cool. I just want to note about ai_family. Look around. There are a lot of variables, whose names are corresponded to camelCase convention.

That's true, however, the meaning of the variable is not something newly invented within wakaama, but taken directly from the addrinfo struct - where the name is given as "ai_family". So IMHO it makes sense to use exactly the same name to convey that the meaning is exactly the same.

From crenz:

Note: There is one issue I encountered: It is not possible to use "localhost" as server address in IPv4 mode on my system. Instead, "<IP_ADDRESS>" needs to be specified explicitly.
The underlying root cause is using AF_UNSPEC in the hints structure when looking up the server address in connection_create(). getaddrinfo on my OS X system will return [::1] as first IP address for "localhost" in this case. The way to fix this would be to use either AF_INET or AF_INET6 depending on the mode of the client, but I'd still need to investigate a bit to find an elegant way to bring this information into connection_create().

I'm still afraid, that this is not only the case for "localhost", it may be the case for all named host.
So this PR will only work with numeric address (e.g. xxx.yyy.nnn.mmm), when ipv4 is switched on.
And modifying "connection_create()" may fix this. How the parametes are passed in is described (
If so, please obey, that the function is called from a "callback". Usually a callback should access additional data via a "context", which is passed in using lwm2m_init.
So extending e.g. "client_data_t" with the address familiy would be a good way (from my view).)

boaks, you are right. This issue will appear with any host that has both IP6 and IP4 DNS entries.
I have implemented your suggested addition, and the clients seem to work fine on my machine now.

Hmm. Something went wrong during rebasing, it seems. The change for pull request #84 got tied up in this commit as well.


Defaulting to user installation if not writeable

if the global install is not writeable then install in .local

add in v0.0.6


ARMv7 support

Please add ARMv7 support

I'm sure you can do it. Here is the list of architectures, just add one and open a PR

Here you are:
https://hub.docker.com/layers/moleus/domru/devel-feat-push-dockerhub/images/sha256-cb22a74c69ad8f747cc0d0964504cbb88441b11ad40751f240de4b09fb8621b0?context=repo


Bot crashed after creating a poll in it

Hi, I'm fairly new to programming. I created a bot that gives a random answer from array, made few ReplyKeyboardMarkup's in it. It was working fine but after creating a poll in telegram itself( in the bot, not in the code) and sharing this poll with a user, my bot crashes everytime. Logs from heroku give me this:
INFO -- : Incoming message: text="#Telegram::Bot::Types::Poll:0x0000557a70f1c288" uid=
over and over, just different numbers.
Could you please help me out because it was working fine before this poll.

It seems that your bot can handle only certain kinds of messages and can't handle poll message.
I guess your bot just fetches the same message from Telegram Bot API every time because it crashes on handling and doesn't send acknowledgment to the API.
You can remove any handling logic that forces your bot to crash and redeploy it to Heroku. This way your bot will correctly receive the message and acknowledge it to the API.

I guess your bot just fetches the same message from Telegram Bot API every time because it crashes on handling and doesn't send acknowledgment to the API.

That's what I thought but couldn't explain myself. Thanks!

You can remove any handling logic that forces your bot to crash and redeploy it to Heroku. This way your bot will correctly receive the message and acknowledge it to the API.

I will leave basic stuff like /start and /stop, I guess it should work.  Thanks, I'll try it and get back to you.

It helped! I removed most of the code, left simple stuff. Then redeployed everything, fixed few bugs and all good.
Thanks a lot!


Auto-Args to main in Azure function App are not generated at new bindings

Routed from: https://github.com/Azure/azure-functions-python-library/issues/60

According to the documentation, we don't support auto-args. I'm not sure that we ever did, so I think that the documentation may have been incorrect.
From the documentation:

After the binding is defined, you can use the name of the binding to access it as an attribute in the function signature. By using an output binding, you don't have to use the Azure Storage SDK code for authentication, getting a queue reference, or writing data. The Functions runtime and queue output binding do those tasks for you.
Update HttpExample_init_.py to match the following code, add the msg parameter to the function definition and msg.set(name) under the if name: statement:


Too Many TimerQueueTime objects retained by DBConnectionPool of sqlClient

We are running an API server that is using Microsoft.Data.SqlClient 4.1.0
The api is async all the way and here is a sample that runs a stored procedure.
            using (var con = new SqlConnection(sConn))
            {
                try
                {
                    await con.OpenAsync().ConfigureAwait(false);
                    command.Connection = con;

                    IEnumerable<T> result;

                    var queryParams = new DynamicParameters(m_Parameters);
                    result = await con.QueryAsync<T>(procedureNameWithDbVersionPrefix, queryParams, commandType: CommandType.StoredProcedure).ConfigureAwait(false);

                    await con.CloseAsync().ConfigureAwait(false);
                    return result;
                }
                catch (Exception ex)
                {
                    LogException(ex);
                    return null;
                }
            }

.NET target: Core 3.1
over time with a load of 5K requests per second we are seeing our service ramps up to 80% cpu and then keeps rising even though the load is the same .. until process gets to 100% cpu.
in addition memory keep rising and over 15 min it gets to to ~1.2Gb
then we took a mem dump and this is the result

after inspecting this TimerQueueTimer in perfView we see this ref:

what could cause such a behavior ?

What happens if you remove Dapper from your sample code?

Closing the issue since it's not a SqlClient issue.
Feel free to reopen it or keep posting here.


GBC/GBA vs GBA/NDS slot

Hi,
I have a question about GBC/GBA slots, what is the difference compared with GBA/NDS?
The link you posted goes to a GBA/NDS instead, but I can't tell any differencies against other options.
Regards!

There is a slightly different form factor, in this listing this listing you can see some more pictures that show small bumps on the bottom of the slot. These bumps align with holes on the PCB to keep it aligned and secure.

I printed the linked 3D case but I don't see the bump correspondent wholes in it, but even more bumps (one big and one small), or are slot bumps only for spacing to the case wall instead?
Sorry for being so annoying XD & thanks for the quick response.

Thanks for the quick response, I am now ordering everything needed!
If the 3D case is not perfectly fit, I may modify it in the future.

No problem, the case should fit fine but any improvements are always welcome.


temp and root partition swapped

For two weeks or so the root partition and the temp partition has been swapped on around 20% of the runners.
Which results in temp getting bigger and root staying small:
/dev/root                     73G   73G  100M 100% /

Please see https://github.com/easimon/maximize-build-space/issues/39#issuecomment-1935591779
Can this be an explanation for what you see?
The tmpfs you mentioned is not the temp partition, but a ramdisk, mounted to /tmp. The temp disk should be mounted at /mnt.

Thank you!
Yes, that's probably it. However, I'm confused on what the fix is. Whatever I specify in temp-reserve-mb will end up getting added to /dev/root.
But if try to make temp-reserve-mb very large it'll throw fallocate: invalid length value specified.

What are you trying to achieve? The action combines all free space on / and /mnt into an lvm volume, which is then mounted as the workspace directory. The workspace is the place where most build jobs require the most space.
The reserve parameters are to control how much space is not consumed by the lvm volume, so you can still add files outside the workspace... E.g. to download docker images, install further packages via apt-get etc.
By increasing the reserve, you decrease the amount available in the workspace folder.
The temp reserve is something you should seldomly need to adjust, since by default nothing (I know of) writes to /mnt.

I'm getting this too: fallocate: invalid length value specified.
What I think @michabp is suggesting is: check if /mnt mountpoint really exists and do the calculations accordingly, otherwise its inconsistent for both configurations.
If we reserve space on /mnt and / but /mnt doesn't really exist, the reserve total space should be merged, so that it doesn't underflow/overflow on our situation, which is exactly what is happening right now.
It used to be 20% of the runners but now its almost all runners, from my experience.

I added #48, which to my knowledge describes the general situation: provided disk space and layout is fluctuating.
Closing this.


Stable Sort

Stable sorts provide desirable features over a non-stable sorts that is necessary a some cases. Since the Array.prototype.sort does not provide any such guarantee and, according to this, browsers do not agree in the implementation, there is no guarantee about Ramda's.
Does ramda care about this? If not, should ramda contain a stable sort?

i think this is a desirable feature. certainly worth exploring.

Agreed.  And since Ramda doesn't really do any internal sorting, we could easily decide, even if this ended up less performant than the native non-stable versions to nontheless offer it as an alternative version for those who need it.

Ramda already includes a very similar function and in most cases unstable sort it not a problem. And if #994 gets solved I think the need would be even smaller.
The complexity/size of such a function would not be insignificant.
It is not hard to find implementations elsewhere compared to which Ramda would not offer any benefits.
It doesn't really have anything to do with functional programming but is a general purpose algorithm.

It is not hard to find implementations elsewhere compared to which Ramda would not offer any benefits.

ok, fair point.

It doesn't really have anything to do with functional programming

it supports referential transparency: any input list is guaranteed to have the same output list on any platform.

This would be either in addition to sort or in place of it, although now that I look at it, we would probably also need to consider sortBy, which dampens my enthusiasm a bit.  As to whether a non-stable sort is an issue, that's to my mind, and issue for the user to decide.
The complexity does not have to be great at all.  We can easily map elements onto a {elt, idx} structure, sort according to the original comparator, break ties by index, and then pluck the elements off the results.  Clearly this is not as efficient as a good mergeSort implementation, but it demonstrates that we can choose the level of complexity that fits.
For almost all our functions, one can find easily find other implementation.  The main questions are usually about how well they fit with the rest of Ramda
Many of our other functions also have little to do with FP, but are useful utilities that one might want to have offered with a functional API.

Closing in favor of $1456


REFACTOR: change pounds per person per day to grams per person per day on the server side and domain

Done. Changed poundsPerPersonPerDay to gramsPerPersonPerDay to accuratlet reflect that it is collecting grams


add access reader log wire

About the PR
adds a LOG wire to most access readers that disables logging when cut or pulsed (temporarily)
Why / Balance
door logging right now is really strong for detectives since few people hack maints doors you can basically trace an antags path through the station and easily find a suspect for a crime every time.
the previous counter was either emagging it which is bad because emag isnt wanted to be a required purchase for syndies, hacking it open or deconstructing it after to wipe the access. the latter require insuls and arent something you can do preemptively.
if you are planning on killing cmo you might disable logging on all the maints doors near med then make your epic getaway, and there is deeper counterplay to this. if an engineer sees that all the doors in an area have logging disabled he can tell the detective to help piece together a story without being so "heres the bad guys name go kill em"
Technical details
added LoggingDisabled bool to AccessReader which the wire action changes
Media
blue light for log

with log disabled, opening it doesnt create any logs

and its enabled again, look at the valid

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
no
Changelog
:cl:

tweak: Doors access logging can now be disabled by snipping or pulsing a LOG wire.

To make cutting and pulsing more distinct (and to jive with how pulsing works in my headcanon), consider having the pulse action also leave behind an obvious but unidentifiable log entry (like some scrambled text or "error: access reader overloaded").

This seems incredibly easy to do and strong so not sure on it, might agree with tayrtahn.

gaming complete @metalgearsloth


Add new implementation based on the official Prometheus plugin

What does this PR do?
Add new implementation based on the official Prometheus plugin: https://docs.konghq.com/hub/kong-inc/prometheus/
Motivation
Customer requests

@ruthnaebeck Can you instead open a PR to https://github.com/DataDog/integrations-core/blob/master/datadog_checks_dev/datadog_checks/dev/tooling/templates/configuration/instances/openmetrics.yaml?


Feat(eos_cli_config_gen): Support custom User Roles

Change Summary
Add possibility to create custom User Roles
Related Issue(s)
N/A
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
Support for adding custom user roles
How to test
N/A
Checklist
User Checklist

N/A

Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

thanks for your feedback Claus. In oder to make your suggested changes, I'll have to close this PR, do the changes and then add a new PR, right?
According to the Datamodel: Sounds good: I can also rework the ACL part in a similar way as next step

thanks for your feedback Claus. In oder to make your suggested changes, I'll have to close this PR, do the changes and then add a new PR, right?

If you just continue pushing to your own fork (same branch), it will update the PR automatically.

According to the Datamodel: Sounds good: I can also rework the ACL part in a similar way as next step

Re ACL, we will not merge changes to existing data models until 4.0 since they would break existing deployments. We are already planning to rework all the dicts with variable keys for 4.0. If you are up for it when we get closer, I can assign you some of them.
Btw please drop me a mail on holbech at arista.com, so we know who contributes. Thanks


Fix Zulu version stringly typing

Before this PR
The different pieces of code that combined then split zulu versions was not correct 🤦🏻
After this PR
==COMMIT_MSG==
It is fixed.
==COMMIT_MSG==
Possible downsides?

Released 0.3.0


Project missing LICENSE file

Thanks for catching this @kshithijiyer ! We've updated with an Apache 2.0 license.


Testrpc won't work if ran in background mode

If I run a testrpc with & in bash script and then I try to truffle migrate, it won't wait until the contract is deployed to save the address in contract's JSON
Example:

if testrpc_running; then
  echo "Using existing testrpc instance"
else
  echo "Starting our own testrpc instance"
  # We define 10 accounts with balance 1M ether, needed for high-value tests.
  testrpc > /dev/null &
  testrpc_pid=$!
fi

cd contracts_src && ../node_modules/.bin/truffle compile && ../node_modules/.bin/truffle migrate && echo "done"

Expected Behavior
I should be able to deploy contract even it commanded from bash script.
Current Behavior
I get TxId as an output, but all transactions are not mined in testrpc
Steps to Reproduce (for bugs)
Run the script above with some contracts from truffle located at contracts_src
Your Environment

Version used: EthereumJS TestRPC v4.0.1 (ganache-core: 1.0.1)
Server type and version:
Operating System and version: Mac Os Sierra

I figured out, I have to have at least some period of mining blocks: testrpc -b 0.1 in order to guarantee that addresses will be available in json files

@rstormsf You shouldn't need to set a block time. The behavior you're describing sounds like you're using async migrations in truffle.
In order to make this work correctly, you'll need to follow the following pattern in your migration file:
module.exports = async function(deployer) {
  return deployer.then(async function() {
    // async deploy logic goes here
  }
}


Small fix to avoid detectors being stuck on blpop if token erased

Description of PR

What component(s) does this PR affect?

[X] Back-End (Database, Microservices, Containers, etc)
[ ] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

Does the PR require changes on other components? If yes, please mark the components:

[ ] Back-End (Database, Microservices, Containers, etc)
[ ] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

Related Issue

Resolves #
Solution

Timeout in non-pipeline blpop, retry if timeout is exhibited.
Type

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

Checklist:

[X] I have read the contributing guide and my code conforms to the guidelines.
[ ] This change requires a change in the documentation.
[ ] I have updated the documentation accordingly.

@pgigis @slowr
https://github.com/FORTH-ICS-INSPIRE/artemis/blob/master/backend/core/detection.py#L782
This is because another instance may render a hijack e.g., outdated exactly when before the blpop operation gets "unstuck", potentially trapping a detector instance for ever. It is preferrable that we simplyu return in that case (the BGP update is anyway deprecated or sth else in that case, not related to an ongoing hijack, therefore benign).


Use tiered projections for hash aggregates

Signed-off-by: Jim Brennan<EMAIL_ADDRESS>Closes #5085.
This implements a tiered approach for input projections in hash aagregations.  This is a performance optimization to reduce computation by pulling out common sub-expressions and doing them once.  The general approach is to provide an alternative to bindGpuReferences, currently called bindGpuReferencesTiered, which returns a case class that contains a tiered list of references, and the corresponding method in that case class that applies the tiered projections in order. The tiered list of references are created by recursively pulling out common sub-expressions from the initial list of expressions and rewriting expressions to refer to the output of the sub-expressions.
This feature is enabled by default, but there is a configuration property, spark.rapids.sql.tiered.project.enabled that can be used to disable it.
See #5085 for more discussion and performance.  If this works well for aggregations, we will consider enabling it for other projections in the plugin.

build

I pushed up a commit that addresses some of the review comments.  I am still doing some additional rework, so please hold off on merging for now.

build

Updated to address more review comments.  I've done a significant rewrite of getExprTiers to make it more efficient and to fix a problem I was seeing where it was making aliases of aliases in each tier, increasing the number of columns (at least column references) pretty significantly.  It no longer does this.  A simple example is from an earlier comment, these expressions: ((a + b) + c), (a + b)  produced this set of expression tiers:
(a+b) as tmp1
(tmp1 + c) as tmp2, tmp1 as tmp3,
tmp2, (tmp2 + d), tmp3

Now this will produce:
(a+b) as tmp1
(tmp1 + c) as tmp2
tmp2, (tmp2 + d), tmp1

The redundant tmp1 as tmp3 is removed.  It's small in this example, but in some queries, like q66, there can be hundreds of these.  I don't think it was causing much extra work, because they are all aliases, but it's much cleaner now.
This update does not include removing eclipsed columns.  I will address that in a follow-up PR.

build

I added some debugging to print out number of columns added and batch sizes for each tier when tiered project is happening.  I ran nds2.0 power run on spark2a and captured this data.  I split it out into three files, one for 2 tiers (1 sub-expression tier), one for 3 tiers (2 sub-expression tiers), and one for 4 tiers (3 sub-expression tiers).  In each of these files, each line looks something like this:
cols(in,diff,diff,out)  5  2  2 13      sizes(in,diff,diff,out)           5,044           3,984           7,968          13,510

This line shows 5 columns in the input batch, 2 columns added in the first tier, 2 columns added in the second tier, and 13 columns in the output batch.
It also shows a batch size of 5,044 for the input batch and 13,510 for the output batch.
The other batch sizes are differentials from the previous column, so the first tiered project added 3,984 bytes, and the second tier added 7,968 bytes.
The files are sorted by the last differential batch size column.
batch-data-2-tiers-spark2a.txt
batch-data-3-tiers-spark2a.txt
batch-data-4-tiers-spark2a.txt

Thanks for the reviews @revans2 and @abellina!


Update contributor ladder index

Describe your changes
This change synchronizes the changes made on the contributor ladder term to the Spanish version.
Related issue number or link (ex: resolves #issue-number)
NA
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@raelga @92nqb pueden revisar esta actualización?


Various 500 errors on deck detail page from Key Errors

Example. At /decks/8317/
7940

Reported on decksite by logged_out
Request Method: GET
Path: /decks/8317/?
Cookies: {}
Endpoint: deck
View Args: {'deck_id': '8317'}
Person: logged_out
Referrer: None
Request Data: {}
Host: www.pennydreadfulmagic.com
Accept-Encoding: gzip
Cf-Ipcountry: US
X-Forwarded-For: <IP_ADDRESS>, <IP_ADDRESS>
Cf-Ray: 409ba51b9a2d959e-IAD
X-Forwarded-Proto: https
Cf-Visitor: {"scheme":"https"}
Cache-Control: no-cache
Pragma: no-cache
Accept: /
From: bingbot(at)microsoft.com
User-Agent: Mozilla/5.0 (compatible; bingbot/2.0; +http://www.bing.com/bingbot.htm)
Cf-Connecting-Ip: <IP_ADDRESS>
X-Forwarded-Host: www.pennydreadfulmagic.com
X-Forwarded-Server: pennydreadfulmagic.com
Connection: Keep-Alive--------------------------------------------------------------------------------
KeyError
Stack Trace:
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/discord/.local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "./decksite/auth.py", line 116, in decorated_function
return f(*args, **kwargs)
File "./decksite/main.py", line 58, in deck
view = Deck(d, auth.person_id(), auth.discord_id())
File "./decksite/views/deck.py", line 25, in init
self.matches = match.get_matches(d, True)
File "./decksite/data/match.py", line 45, in get_matches
m.opponent_deck = decks_by_id[m.opponent_deck_id]

I have more. A handful since yesterday.

Other ids are involved -  7998 and more.

I think this is caused by the season boundary issue.  If the created date is slightly off from the season start date then we are no longer loading it in that season but if you played them ...
I don't know why some S7 decks have S6 created dates so figuring that out is the answer to figuring this out, I suspect.

This no longer happens because I rewrote load_opponent_stats to get around another issue. Also it's called load_competition_data now.


Android / iOS support

Are there any plans to support android and or ios in the near future?

I don't have any plans for it no. I would guess porting the macOS version to iOS wouldn't be that much work except there is really no support for touch in there. For Android I haven't no expertise with so I can't say there.

Maybe I will use minifb for the UI-Toolkit OrbTk. There I need also android and ios support. When I'm ready I could try to write a android and ios port for minifb.

I'm don't find the time to do a port of minifb for android or iOS. But it is still important for OrbTk. If someone could do it, I would be really happy.

Have their been any updates on this? When i try to install minifb using Termux on my android i get this:
error occurred: Failed to find tool. Is aarch64-linux-android-ar installed?

I think OrbTK has moved away from minifb (?) and I have no plans of adding this myself so I'm closing the issue for now.


wkhtmltopdf module added

xdebug to 1000 recursion limit
wkhtmlto pdf module added

Closing as dupe of PR #8.


macOS One-click Installer didn't work when downloaded.

Just as stated above. Clicked on the links here:
https://h2o-release.s3.amazonaws.com/h2ogpt/Nov2023/h2ogpt-osx-m1-cpu
Downloaded a file that macOS tries to open as textedit file.
Also, is there supposed to be a difference between the links?

Thansk


Consider compressing images

The logo image is 1.1MB. Running it through the optimiser at tinypng.com reduced it to 10kb and it looks just as nice. :)

@KharitonOff can you make sure to reduce the images and load time for the redesign such that this is fixed?
@spadgos :+1: for pointing this out.

done in the new design version


No content in commits 😱

Describe the bug
Reverting changes locally and running updatecli again creates a new commit and is pushed to the branch even there is no diff in the two commits.
It means I cannot use rebase and in these cases I don't want to use squash either 😅

WAIT what? The PR has no content using 0.0.15. This seems to go for all PRs.

Also why are changes being applied to local files in your git repo? Seems counter intuitive when testing our new updatecli files. I think something foobar is going on. The file changes and commits are not happening in the right folders.

Expected behavior
Only one commit per updated version and PR actually have content.
Info
$ updatecli version

VERSION

Application:    v0.0.15
Golang     :    1.14.2 linux/amd64
Build Time :    Fri, 17 Jul 2020 11:46:45 +0200

The code should properly error on empty commits 😅

Oups This is definitely not wanted and  I guess I did something wrong while testing the changes, let me look at it

No worries 😅
I love the tool and what it does.
Sorry for not contributing more with code 🥵

This should be solved now, I am still not certain about the best way to not commit empty commits and won't have the time to work on that today  but at least the tool is working again.

I may be able to look into erroring on empty commits :)
Thanks for fixing the issue!

Awesome, just in case I am using this git library  go-git


Question: support for any other database

What would you like to be added:
Support for any other type of database other than sqlite.
Why is this needed:
I was wondering if there is any plan to support any other database than sqlite.
Additional context:

Hi @hectorj2f, thanks for your question. We aren't currently planning on supporting any other database than sqlite at this point. Do you mind sharing a bit more about your reasons for needing support for another kind of database? We'd be happy to discuss.


Bucket sort

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
I have added python file of Bucket Sorting Algorithm and updated the readme file also.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

@SKAUL05 Please review my PR

@SKAUL05 I have made changes as you said, Please review it

@santushtisharma10 Copied from here

Exactly same, in some places only variable name is changed.
Don't just copy & paste.
Do it by your own
Note: It's very important to do legit work on your own. In case of repeated violations, you will be blocked to contribute in Neo Algo

@SKAUL05 the code of geeks for geeks works only for numbers between 0 and 1 and my code works for large as well as small numbers

I used these as referece https://www.youtube.com/watch?v=geVyIsFpxUs&t=355s and https://en.wikipedia.org/wiki/Bucket_sort

Moreover in geeks for geeks code they always used 10 buckets and since i took the help of wikipedia to understand the algorithm i created the buckets of the same size of unsorted array

@SKAUL05 please look into this matter

@SKAUL05 thank you i will keep it in mind

@HarshCasper @atarax665 @Kajol-Kumari
Review Required on this PR for plagiarism check and modifications.
Thanks.

@Abhijit2505 Sure


Automated cherry pick of #239: dep: update yunion.io/x/log

Cherry pick of #239 on release/2.6.0.
#239: dep: update yunion.io/x/log

/lgtm
/approve


What if I want to use xgboost as first layer model?

If I want to use xgboost as first layer model, what should I do?

Me too :)
I am still trying to integrate that .  You have to w8 for it  or try to add meta predictions with xgboost to your training data as in here  in the script create_files_v1.py . On a positive note I am expecting to add this this week - if everything goes as planned. But it will be fairly untested :(

Thx! I'm looking forward to seeing this new feature.
Another question:How could I extract the out-of-fold predictions and the test prediction of first layers?Because I want to use them to train a second layer model, like xgb, lgb, keras etc...

Xgboost has been added. I am closing this.


Refactor OpenAI initialization and add API base URL setting

Hi, can you explain what the purpose of these changes is? I'm not sure why you're adding another setting for the user.

Hi, can you explain what the purpose of these changes is? I'm not sure why you're adding another setting for the user.

Adding this new setting allows us to customize the host.
Some users, including myself, do not use the official API host. They might use a relay API or custom API that proxies OpenAI.
You might not be familiar with this concept. These relay services provide a simplified way for people to pay for and access the GPT models without directly paying OpenAI.

Hi, sorry for the late response. Thanks for the explanation. I'll merge your pr during the next release.


OpenPathSampling 1.0

I was waiting until we made a decision on the license for this. Minor functionality improvements since 0.9.6. Expect this to be followed quickly by 1.0.1 or 1.1, which will involve merging in contributions from people who weren't authors of the paper.
Any further API cleanup will be part of OPS 2.0, which we will release next year (early 2020).
If no discussion, I'll do the release Sunday night (21 July)
Below are the release notes:

OpenPathSampling 1.0 is ready for release! Major changes since 0.9.6 include:

New license! OPS is now under the MIT license. Versions of OPS prior to 1.0 are still licensed under LGPL 2.1 or later.
Officially supported Python versions are now 2.7, 3.6, and 3.7
Significant refactoring to clean up many parts of the code. In unusual circumstances, these could lead to API breaks. For details, see PRs linked under "Miscellaneous improvements -> Refactoring and API cleanup" below.

Deprecations: Use the generic Details object instead of MoveDetails, which adds no additional functionality.
New features

"Visit all states" ensemble (#826)
Make WHAM's lnZ accessible from outside (#825)
Add engine property to OneWayShooting (#805)

Miscellaneous improvements

Switch to MIT License (#844)
Switch to 2.7/3.6/3.7; deploy docs on 3.7 (#815, #824)
Update developer install process (#840, #842, #843)
Improve performance on trajectory_from_mdtraj (#839) #performance
Additions and improvements in the documentation (#820, #825, #828, #831, #838)
snapshot_from_pdb uses same code path as other functions (#833)
Improve continuous integration (#808, #811, #814, #829)
Shooting selector tests (#827)
Refactoring and API cleanup (#809, #816, #818, #821, #823)

No comments yet.... giving until tomorrow (whenever I get around to it) before the API for 1.0 is fixed! (No stress; we'll probably release an API update in 2.0 in early 2020).

Looks good. Go ahead!


Docker execution error

I'm trying to use TextQL from Dockerfile command.
Simple example:
FROM golang:1.10
RUN go get -u github.com/dinedal/textql/...
CMD ["textql", "-sql", "SELECT 1"]
TextQL installs properly, but the final command returns code 1 and weird error output:
2018/04/20 14:29:53 EOF
If I connect to the container and run TextQL from there, everything works properly.
Can you tell what is the reason why TextQL is not able to run properly from Dockerfile command? It can be executed only after container construction but not in it. That's quite impractial.

I got it. It must be connected with TTY, which is possible with -t parameter in docker run command.

For anyone with the same issue with a command in the crontab, if your job ends with return code 1 without any message except:
2023/11/22 14:33:01 EOF

the "proper" workaround is to use screen to allocate a pseudo-tty like so:
0 0 * * * user screen -S myscreename -d -m textql...

Source: http://www.linta.de/~aehlig/techblog/2011-06-16.html


Adding experimental client API

This PR adds the ExperimentalPublicClientApplication.ts class, which will be used for future bleeding edge features without disturbing the current PublicClientApplication.ts surface.

Codecov Report

Merging #3855 (fc0da9e) into dev (b154798) will decrease coverage by 0.03%.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
95.81% <ø> (ø)

msal-browser
84.93% <ø> (-0.13%)
:arrow_down:
Carriedforward from be9a1f4

msal-common
85.31% <ø> (ø)

Carriedforward from be9a1f4

msal-core
75.09% <ø> (ø)

Carriedforward from be9a1f4

msal-node
82.52% <ø> (ø)

Carriedforward from be9a1f4

msal-react
94.11% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

...ib/msal-browser/src/app/PublicClientApplication.ts
85.71% <0.00%> (-4.68%)
:arrow_down:

Missing an export of EPCA from the index.ts (unless you are intentionally leaving that out until we add the first experiemental feature.

Missing an export of EPCA from the index.ts (unless you are intentionally leaving that out until we add the first experiemental feature.

Yes, I was waiting until we had something to put in here before we start exporting this.


iOS Hybrid app WebRTC Stack for Cordova, Amazon Erizo Controller Fix and Misleading Comments Fixed

First of all this is my first time using Licode - it looks awesome, great work!! :)
I've added a WebRTC stack for those who want a WebRTC enabled hybrid iOS app with cordova and the iosrtc plugin found here:
https://github.com/eface2face/cordova-plugin-iosrtc
If cordova the application will load a stack for an iOS hybrid app and if it's not default to the Bowser stack! It's fully tested and I now have a fully working WebRTC iOS hybrid app using Licode!!
There are some slightly misleading comments for those using an EC2 as they have to place their Private IP in the Public IP area.
I've also fixed an error those on an EC2 will get where all the parameters in the Amazon Erizo Controller  (addNewAmazonErizoController) aren't defined and therefore gives you the following error:
/home/ubuntu/licode/node_modules/aws-lib/node_modules/xml2js/lib/xml2js.js:216
          throw ex;
                ^
ReferenceError: hostname is not defined
    at /home/ubuntu/licode/nuve/nuveAPI/cloudHandler.js:126:52
    at /home/ubuntu/licode/node_modules/aws-lib/lib/aws.js:62:14
    at Parser.<anonymous> (/home/ubuntu/licode/node_modules/aws-lib/lib/aws.js:99:19)
    at Parser.emit (events.js:95:17)
    at Object.saxParser.onclosetag (/home/ubuntu/licode/node_modules/aws-lib/node_modules/xml2js/lib/xml2js.js:183:24)
    at emit (/home/ubuntu/licode/node_modules/aws-lib/node_modules/sax/lib/sax.js:148:33)
    at emitNode (/home/ubuntu/licode/node_modules/aws-lib/node_modules/sax/lib/sax.js:152:3)
    at closeTag (/home/ubuntu/licode/node_modules/aws-lib/node_modules/sax/lib/sax.js:226:5)
    at Object.write (/home/ubuntu/licode/node_modules/aws-lib/node_modules/sax/lib/sax.js:567:29)
    at Parser.exports.Parser.Parser.parseString (/home/ubuntu/licode/node_modules/aws-lib/node_modules/xml2js/lib/xml2js.js:211:31)
    at Parser.parseString (/home/ubuntu/licode/node_modules/aws-lib/node_modules/xml2js/lib/xml2js.js:6:61)

Why do you close your pull request?

Didn't look like it was getting any attention, probably due to two commits within the same PR so pulled it down :)

reopen it immediately!

I would greatly appreciate if you could reopen this pull request. Because I have actual need for iOS clients, then I am willing to look after this functionality after it gets pulled.
I have tested what you have build and it works wonderfully. Big thank you for that.
I would be definitely happier if this would get incorporated into the Licode main.
Thanks in advance.

Okay, re-opened! 👍

Thank you!
I greatly appreciate your work. As I wrote, I am willing to follow the changes and their functionality in the future.
By the way, we have a very good in-house-solution app up and running with Licode and your code.
Thank you for your excellent work! The code worked at once. Just throw it in and it works. Excellent!

:+1:
@ArtisanCodesmith sorry for the late news, can you resolve the conflicts with master? I think some of the issues are currently solved and we've changed several lines of code recently.

@jcague Will get this done tonight! :)

@ArtisanCodesmith any updates on this PR?

I have real need for this functionality. Hence I am willing to have a look at this. I am, however, a bit busy. Therefore I do not promise fast results.

Hi, any updates on this PR?

Unfortunately not from me. I have been able to get partial functionality working, but I have not been able to get everything to work correctly. I am not familiar enough with the tools for a fast solution.
Maybe ArtisanCodeSmith could spare some time for this? He did the original excellent work.

@ArtisanCodesmith any updates on this PR?

I have not tried using adapter.js in order to get things working, but in the discussion of Issue
https://github.com/lynckia/licode/issues/915  menelike writes that with adapter.js and the Chrome stack things work fine. Or at least most of them.

@jjahifi
What we actually did was (simplified):

use https://github.com/BasqueVoIPMafia/cordova-plugin-iosrtc
get rid of https://github.com/lynckia/licode/blob/70b3e8eb7d218a6e2ed7658051e77f4689b86f3b/erizo_controller/erizoClient/src/Stream.js#L98-L116
copy ChromeStableStack.js to iOSStack.js and change

      WebkitRTCPeerConnection = cordova.plugins.iosrtc.RTCPeerConnection,
      RTCSessionDescription = cordova.plugins.iosrtc.RTCSessionDescription,
      RTCIceCandidate = cordova.plugins.iosrtc.RTCIceCandidate,

add the the new ios stack into https://github.com/lynckia/licode/blob/70b3e8eb7d218a6e2ed7658051e77f4689b86f3b/erizo_controller/erizoClient/src/Connection.js#L23 like

    } else if (that.browser === 'ios'){
        L.Logger.debug('iOS Stack');
        that = Erizo.iOSStack(spec);

and
    } else if (window.navigator.userAgent.match('AppleWebKit') !== null && window.cordova) {
        browser = 'ios';

You'll need to pass a browser specific videoOpt yourself as we removed this from licode.

for chrome we use something like this:
      videoConfig = {
        width: videoWidth, // can be also an object of {min, ideal, max}
        height: videoHeight,
        frameRate,
        aspectRatio: { ideal: 16 / 9 },
        deviceId: bowser.ios ? videoDeviceId : { exact: videoDeviceId },
      };

With the iOS stack we actually do not want to use adapter.js, so we disable it with
    // disables adapter.js
    if (bowser.ios) videoConfig.mandatory = {};

https://github.com/webrtc/adapter/blob/726fac514360bcd86e77d950b33f646da12b3b69/src/js/chrome/getusermedia.js#L19
Keep in mind that sceensharing does not work well with adapter.js since you need to work with the mandatory prop which disables adapter.js.
This works quiet well for one year on electron, chrome, cordova iOS, cordova Android.
Hope this helps.

All, iosRTC now works "out of the box" with the latest code of licode using adaptter.js. No need to add a separate iosStack or change Connection.js.  In your Cordova App - please remember  to add registerGlobals() as follows:
// Just for Cordova apps.
document.addEventListener('deviceready', function () {
// Just for iOS devices.
if (window.device.platform === 'iOS') {
cordova.plugins.iosrtc.registerGlobals();
}
});

Yes. Most of the functionality seems to work. There are, however, some strange errors in the logs (or xcode), but they do not seem to have any impact on the functionality (at least not in my case).
In any case, this is really good news.

Anyone has fixed the conflicts? I can not get Cordova ios rtc to work with Licode v5.


Feature Request: allow passing logger to makeSchema and pass it forward to plugins / etc

Relying on console.* is not great if the client is using structured logging.

Do you have a logger interface proposal?

I like the pino logger interface, but generally the winston interface is the standard.
https://github.com/winstonjs/winston/blob/master/index.d.ts


Board Review: Azure Mixed Reality Authentication client library

Thank you for submitting this review request. Thorough review of your client library ensures that your APIs are consistent with the guidelines and the consumers of your client library have a consistently good experience when using Azure.
The Architecture Board reviews Track 2 libraries only. If your library does not meet this requirement, please reach out to Architecture Board before creating the issue.
Please reference our review process guidelines to understand what is being asked for in the issue template.
To ensure consistency, all Tier-1 languages (C#, TypeScript, Java, Python) will generally be reviewed together.  In expansive libraries, we will pair dynamic languages (Python, TypeScript) together, and strongly typed languages (C#, Java) together in separate meetings.
For Tier-2 languages (C, C++, Go, Android, iOS), the review will be on an as-needed basis.
Before submitting, ensure you adjust the title of the issue appropriately.
Note that the required material must be included before a meeting can be scheduled.
Contacts and Timeline

Responsible service team<EMAIL_ADDRESS>Main contacts: crtreasu, virivera, rgarcia, ariye
Expected code complete date: 01/29
Expected release date: 03/01/2021

About the Service

Link to documentation introducing/describing the service: https://review.docs.microsoft.com/en-us/azure/object-anchors/overview?branch=release-preview-aou
Link to the service REST APIs: https://github.com/Azure/azure-rest-api-specs/tree/master/specification/mixedreality/data-plane/Microsoft.MixedReality/preview/2019-02-28-preview
Link to GitHub issue for previous review sessions, if applicable:  https://github.com/Azure/azure-sdk/issues/2005

About the client library

Name of the client library: Azure Mixed Reality Authentication
Languages for this review: JavaScript, Java, Python

Artifacts required (per language)
Please read through “API Review” section here to understand how these artifacts are generated. It is critical that these artifacts are present and are in the right format. If not, the language architects cannot review them with the SDK Team’s API review tool.
.NET
n/a already completed

APIView Link:
Link to Champion Scenarios/Quickstart samples:

Java

APIView Link: https://apiview.dev/Assemblies/Review/f3a4bb684ffd4badada05eff7de952d5
Link to Champion Scenarios/Quickstart samples:

Python

APIView Link:
Link to Champion Scenarios/Quickstart samples:

TypeScript

APIView Link: https://apiview.dev/Assemblies/Review/4917626415bc448c8e2534e00c6f3a17
Link to Champion Scenarios/Quickstart samples:

For all other languages, send a request to the Architecture Board to discuss the best format on individual basis.
Thank you!

scheduled for 2/3

scheduled for 2/3

Recording[MS INTERNAL ONLY]


about train_sony code

In the  train_sony  code,how to modify the  batch_size?

We don't support batch sizes of more than 1 due to the memory issue. I suggest you redesign the flow to meet your requirements. However, I could tell you the easiest way to change batch size is to append multiple patches before you feed into the model.

您好，在这个训练代码中，这句[:,yy:yy+ps,xx:xx+ps,:]语句中的四个参数代表的含义我不太理解，希望能够解答一下input_patch = input_images[str(ratio)[0:3]][ind][:,yy:yy+ps,xx:xx+ps,:]
------------------ 原始邮件 ------------------
发件人:                                                                                "lavi135246/pytorch-Learning-to-See-in-the-Dark"                                                                                @.>;
发送时间: 2021年3月23日(星期二) 下午2:25
@.>;
<EMAIL_ADDRESS>主题: Re: [lavi135246/pytorch-Learning-to-See-in-the-Dark] about train_sony code (#14)
We don't support batch sizes of more than 1 due to the memory issue. I suggest you redesign the flow to meet your requirements. However, I could tell you the easiest way to change batch size is to append multiple patches before you feed into the model.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


textarea appear as empty

how i can show tinymyce editor

I've fix this issue. check the develop branch once pull request is merged. you just need to change the theme of tinymce from modern to silver as it is not available in latest version.

Resolved.


screengrab is working on Android now

Hi @mmoll - thanks for the contribution :+1:
I'm not sure I understand the intent relative to the PR title, though. :smile: Is the intent to allow screengrab to work on linux?

@mfurtak Ah, yes... I'm running it on Linux and yes, I did mislabel, fixed.

Cool, let's do it! Happy screengrabbing :camera: :sparkles:


Cluster Bootstrap and Node Initialization Sequence

I would love some explanation about cluster bootstrap and initialization process under the following scenarios -

When cluster is being bootstrapped by starting N number of nodes around the same time with the same configuration(list of nodes).
Node is being restarted which were part of a cluster.

My understanding is that -

a Node need to initialized with only itself.
Add the other nodes as learners
Change the membership to include all the nodes

Every node needs to be running the same sequence. Is that correct?

In this PR I added some instructions for initializing a cluster:

#971

Did that help clear up your confusion?

FYI:
https://docs.rs/openraft/latest/openraft/docs/faq/index.html

@drmingdrmer Thanks for the docs! I am closing the issue. This is super helpful :)


Is it possible to have "if doesn't contain" conditions?

I looked at the wiki and I saw that you can add conditions that add text  if the image contains a tag from a custom token? Is it also possible that it adds additional text if an image doesn't contain any tags from a custom token? I'd like it to have "None" if it doesn't have any of the tags.

That is currently not possible but not very hard to implement, I'll add it it next release. :smile:

Looks like next release will be very not soon.

@MasterPetrik Grabber.zip

In 5.3.0, to do so you'll just have to prefix your conditions with a exclamation mark ! symbol.
Example:
<image does not contain the tag !"tag1"><!"unknown" is not one of the image tags> %md5%.%ext%
%md5% -<!%artist% no artist>.%ext%

Not to forget to add this info in Grabber wiki

Good call, I just updated it: Filename - Inverting conditions