fix(plugin-react): remove querystring from sourcemap filename

Description
The variable used in this sourcemaps fix in plugin-react might contain a querystring, in which case it does not work. This PR removes the querystring passed to Babel.
Additional context
In Hydrogen, some React components end in component.client.jsx?no-proxy and it shows the warning sourcemap for xyz points to missing source files just like in #5438 due to this querystring.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Is this indirect related to #5761 ?


[Feature Request] On all videos?

Hi!
Could you make it work on most videos on the web? Not just on youtube or vimeo.
That would be awesome. I don't think there's an extension on Firefox that do this.
Thank you for your consideration.

This is really a bit difficult to support every video site.
Maybe you can try another add-on: Maximize Video.
Thanks for your feedback.

What about just supporting generic HTML5 videos like the original extension did? That would be a good step.

Most of the videos on the Internet are embedded in iframe. The receipt of the contents of the iframe prohibits the CORS.
Probably it's possible to do if the video tag is directly on the page. It can be moved to the markup of the window that plugin opens.

How did Opera achieve it? Is that something that only FF can do and not an option with the extensions?

I think Opera feature is working a completely different manner. I would guess that they do some magic with the video output (mirror it in a popup) since you need the original tab open and video plays in sync (popup and tab). For this you probably need to bypass some web extension restrictions.


Expiration dates for events

Events may need an expiration date; If you get a backlog due to some error or congestion, due to parallelism you may end up with early bad data overwriting later good data as the stack races to resolve. Example: A customer changes their address, notices a typo, changes it again. If there is a backup in the stack, and then it unravels, it’s possible the first change arrives after the correction, and the typo becomes the ‘last in’ data.

You are correct, order is not guaranteed with this implementation ( or most webhooks implementations ). If you are worried about order you can add a date or identifier to your event so that you can determine the correct order.

That would require any system using the event broker for data updates to
implement its own sub system for managing messaging continuity, essentially
creating another event broker behind the first.
Maybe I'm off base, but it seems like capturing data updates is going to be
a common use case for this tool. Wouldn't it make more sense to handle it
here?
On Mar 9, 2016 4:27 PM, "Michael Rudduck"<EMAIL_ADDRESS>wrote:

You are correct, order is not guaranteed with this implementation ( or
most webhooks implementations ). If you are worried about order you can add
a date or identifier to your event so that you can determine the correct
order.
—
Reply to this email directly or view it on GitHub
https://github.com/AmbitEnergyLabs/Spoke/issues/5#issuecomment-194540707
.

There are numerous use cases, however the async nature of the broker would make an order guarantee very difficult.
Handling this on the consumer should be as simple as having an effective date or a history to see the date related to events you have already processed to determine whether or not to process the event you received.
If you have an idea on how you would implement some sort of order guarantee i would be happy to discuss it with you.

I've got some ideas I'm kicking around, but I'm not familiar enough with
the design goals to say if they are appropriate. You want to meet up and
hash it out?
On Wed, Mar 9, 2016 at 6:11 PM, Michael Rudduck<EMAIL_ADDRESS>wrote:

There are numerous use cases, however the async nature of the broker would
make an order guarantee very difficult.
Handling this on the consumer should be as simple as having an effective
date or a history to see the date related to events you have already
processed to determine whether or not to process the event you received.
If you have an idea on how you would implement some sort of order
guarantee i would be happy to discuss it with you.
—
Reply to this email directly or view it on GitHub
https://github.com/AmbitEnergyLabs/Spoke/issues/5#issuecomment-194581236
.


Update chiaharvestgraph.c

Added support for analyze Chia's forks logs

Looks good. What's the name of the fork that you are using?

Looks good. What's the name of the fork that you are using?

I'm using Spare and Flax.


Auto-update Reaction Avatars

Fixes #233
Test it on: https://github.com/sindresorhus/refined-github/issues/74
Or on this post :)

I tried opening two tabs and 👍 this PR in one tab, but the other tab didn't update. Am I doing something wrong?

I mean this:

Before this PR, clicking reactions would remove all the avatars because GitHub fetches the whole block again.

Got it. Looks good then :)


[SPARK-25882][SQL] Added a function to join two datasets using one column with join type parameter

What changes were proposed in this pull request?
Added a function to join two datasets using one column with join type parameter

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

So we introduce a new API just to save typing Seq(...)? Maintaining an API has cost.

Thank you for the advice, @gatorsmile and @cloud-fan .
@arman1371 . It's a trade-off issue and the benefit looks smaller than the benefit. If you don't mind, could you close this PR?

@dongjoon-hyun . yes

I think the problem is real, maybe we should not use Seq in the end-user API, but always use Array to be more Java-friendly. This can also avoid bugs like https://github.com/apache/spark/pull/22789
cc @rxin @hvanhovell what do you think?

Yea good idea (prefer Array over Seq for short lists)


WFLY-14142 Update Google Guava to 30.1-jre

Fixes WFLY-14142
This PR depends on WFCORE-5203 PR 4406. Tests will fail as long as WFCORE model-test/pom.xml changes are missing.

These failures are related to the upgrade.

@jamezp

These failures are related to the upgrade.
Yes, that's why I mentioned the dependency to WFCORE PR4406. Or I'm I wrong and it is missing more than the changes in WildFly Core here?

These failures are related to the upgrade.

Yes, that's why I mentioned the dependency to WFCORE PR4406. Or I'm I wrong and it is missing more than the changes in WildFly Core here?

The core dependency seems to only be used for testing. This version is the one that ends up in a module. I'm seeing CNFE's though so I assume we're either missing a dependency or something has changed. This looks like a fairly large jump forward.

I tested the several combinations to be sure. I have attached all results to the JIRA https://issues.redhat.com/secure/attachment/12505580/WFCORE5203_WFLY14142_test_analysis.7z

Version
WildFly-Core Testsuite
WildFly-TestSuite
Detail folder

WFCore14.0.0.Beta5-SNAPSHOT Plain Vanilla / Standalone
OK
Not relevant
./.

WFCore14.0.5203.Beta5-SNAPSHOT / Standalone
OK
Not relevant
WFCORE5203_standalone

WFCore14.0.5203.Beta5-SNAPSHOT integrated in WFLY22.0.0.Final-SNAPSHOT
Not relevant
OK
WFCORE5203_patched_but_wfly_plain_vanilla

WFCore14.0.0.Beta5-SNAPSHOT integrated in WFLY22.0.0.Final-SNAPSHOT Plain Vanilla
Not relevant
OK
./.

WFCore14.0.0.Beta5-SNAPSHOT integrated in WFLY22.0.14142.Final-SNAPSHOT
Not relevant
failed
WFLY14142_patched_but_wfcore_plain_vanilla

WFCore14.0.5203.Beta5-SNAPSHOT integrated in WFLY22.0.14142.Final-SNAPSHOT
Not relevant
OK
WFCORE5203_WFLY14142_both_patched

The fails in WFLY14142_patched_but_wfcore_plain_vanilla, which are the same as happened on TeamCity are org.wildfly.test.integration.elytron.certs.ocsp.OcspTestCase. "test.log" shows the following exception:
08:59:11,897 ERROR [org.jboss.as.arquillian.container.ServerSetupObserver] (main) Setup failed during setup. Offending class 'org.wildfly.test.integration.elytron.certs.ocsp.OcspTestBase$OcspServerSetup@1ae11dd3': java.lang.NoClassDefFoundError: com/google/common/util/concurrent/internal/InternalFutureFailureAccess
        at java.base/java.lang.ClassLoader.defineClass(ClassLoader.java:1017)
        at java.base/java.security.SecureClassLoader.defineClass(SecureClassLoader.java:174)
        at ....
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
        at org.mockserver.integration.ClientAndServer.<init>(ClientAndServer.java:18)
        at org.wildfly.test.integration.elytron.certs.ocsp.TestingOcspServer.start(TestingOcspServer.java:123)
        at org.wildfly.test.integration.elytron.certs.ocsp.OcspTestBase.startOcspServer(OcspTestBase.java:362)
        at org.wildfly.test.integration.elytron.certs.ocsp.OcspTestBase$OcspServerSetup.setup(OcspTestBase.java:444)

There is only one reason: This class is in the new dependency, which is missing, but the new version is present:
[INFO] org.wildfly:wildfly-testsuite-shared:jar:22.0.14142.Final-SNAPSHOT
[INFO] - org.wildfly.core:wildfly-core-testsuite-shared:jar:14.0.0.Beta5-SNAPSHOT:compile
[INFO]    - org.wildfly.core:wildfly-subsystem-test-framework:jar:14.0.0.Beta5-SNAPSHOT:compile
[INFO]       - org.apache.maven:maven-resolver-provider:jar:3.5.4:compile
[INFO]          - org.apache.maven:maven-model-builder:jar:3.5.4:compile
[INFO]             - com.google.guava:guava:jar:30.0-jre:compile
Background: Google separated some classes out of the regular guava distribution into failureaccess in Guava 27.1. Before they were part of the main guava jar. They wanted to ease use in Android dev.
To fullfill the dependencies:

Merge https://github.com/wildfly/wildfly-core/pull/4406
Upgrade WildFly-Core version in WildFly matching to 1
Merge this PR

Ah okay sorry. I should have looked at the failure closer :)

@bstansberry
The test failure seems unrelated, guava is not part of a SSL handshake.
org.wildfly:wildfly-ts-integ-manualmode] [ERROR] testDatabaseCertLoginModule(org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase)  Time elapsed: 30.712 s  <<< ERROR! [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] javax.net.ssl.SSLHandshakeException: Remote host terminated the handshake [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] 	at sun.security.ssl.SSLSocketImpl.handleEOF(SSLSocketImpl.java:1310) [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] 	at sun.security.ssl.SSLSocketImpl.readApplicationRecord(SSLSocketImpl.java:1097) [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] 	at sun.security.ssl.SSLSocketImpl.access$200(SSLSocketImpl.java:72)

@bstansberry
The test failure seems unrelated, guava is not part of a SSL handshake.
org.wildfly:wildfly-ts-integ-manualmode] [ERROR] testDatabaseCertLoginModule(org.jboss.as.test.manualmode.web.ssl.DatabaseCertLoginModuleTestCase)  Time elapsed: 30.712 s  <<< ERROR! [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] javax.net.ssl.SSLHandshakeException: Remote host terminated the handshake [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] 	at sun.security.ssl.SSLSocketImpl.handleEOF(SSLSocketImpl.java:1310) [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] 	at sun.security.ssl.SSLSocketImpl.readApplicationRecord(SSLSocketImpl.java:1097) [13:18:26] [org.wildfly:wildfly-ts-integ-manualmode] 	at sun.security.ssl.SSLSocketImpl.access$200(SSLSocketImpl.java:72)

@bstansberry All preconditions (wfcore updated, tests passed) are fine. Please review and merge.

@bstansberry All preconditions (wfcore updated, tests passed) are fine. Please review and merge.

Thanks, @boris-unckel !


error: no member named 'asString' in 'ZXing::ByteArray'

I tried to build LibreOffice (for Android) with zxing-cpp v1.4.0.
I get:
/home/user/libo-android/workdir/UnpackedTarball/zxing/core/src/aztec/AZDecoder.cpp:332:40: error: no member named 'asString' in 'ZXing::ByteArray'
                res.applicationIndicator = res.bytes.asString(0, 1);
                                           ~~~~~~~~~ ^
/home/user/libo-android/workdir/UnpackedTarball/zxing/core/src/aztec/AZDecoder.cpp:338:40: error: no member named 'asString' in 'ZXing::ByteArray'
                res.applicationIndicator = res.bytes.asString(0, 2);
                                           ~~~~~~~~~ ^
2 errors generated.
make[1]: *** [/home/user/libo-core/solenv/gbuild/LinkTarget.mk:404: /home/user/libo-android/workdir/GenCxxObject/UnpackedTarball/zxing/core/src/aztec/AZDecoder.o] Error 1
make[1]: *** Waiting for unfinished jobs....
/home/user/libo-android/workdir/UnpackedTarball/zxing/core/src/Content.cpp:209:32: error: no member named 'asString' in 'ZXing::ByteArray'
        if (bytes.size() > 6 && bytes.asString(0, 4) == "[)>\x1E" && std::isdigit(bytes[4]) && std::isdigit(bytes[5]))
                                ~~~~~ ^
1 error generated.
make[1]: *** [/home/user/libo-core/solenv/gbuild/LinkTarget.mk:404: /home/user/libo-android/workdir/GenCxxObject/UnpackedTarball/zxing/core/src/Content.o] Error 1
make: *** [Makefile:289: build] Error 2

== Development environment ==
OS: Ubuntu 22.04 LTS
Version(Android SDK Build-Tools ):  33.0.0
Version(Android NDK): 19.2.5345600

The problem is most likely that the Android NDK/SDK is not c++-17 conform and does not provide the __cpp_lib_string_view feature macro that is tested for in https://github.com/zxing-cpp/zxing-cpp/blob/f89db69e34543c508f80e807535dab3012eea7ad/core/src/ByteArray.h#L33.
You have the following options:

define the macro yourself in the header or in your build environment
remove the 2 #ifdef __cpp_lib_string_view lines in ByteArray.h
use HEAD instead of 1.4


Zarf Package Schema Lacks Clarity

Is your feature request related to a problem? Please describe.
The Zarf Package Schema is not clear. Things like the metadata.architecture have type string without example on what is valid or preferred. New parties interested in creating Zarf packages aren't guided by the schema as it is defined.
Describe the solution you'd like
Schema fields that are enumerable strings should be enumerated like this requiring a change in the package type.
Other fields' descriptions should be improved to help new Zarf converts. Non-enumerable string fields descriptions should clearly indicate potential values.
Describe alternatives you've considered
Improving and expanding the Examples in the Docs would help. However, it is unlikely that a single example will align with all Zarf package developer's goals. Developers will be left pulling together information from many examples. It is preferable to bolster a single authoritative source: the schema itself.
Additional context
Architecture field definition in Zarf package schema on the zarf.dev webpage

Docker's officially supported architectures (not that aligning to Docker is a goal but it is a jumping off point) are arm32v6, arm32v7, arm64v8, amd64, windows-amd64. There are also options like i386, riscv64, s390x, and ppc641e.
Is there an argument to focus on field descriptions that include likely examples over adding rigid enums?

The arch metadata field isn't very common as its job is to actually restrict what arch a package uses. But we could add an enum like we do a few lines above it: https://github.com/defenseunicorns/zarf/blob/main/src/types/package.go#L9

@mikevanhemert I made some tweaks to the schema to include examples and reflowed a lot of it to make CTRL-F more likely to work, and to make the TOC more consistent.  Let me know if you have feedback:
https://deploy-preview-1381--zarf-docs.netlify.app/docs/user-guide/zarf-schema
cc @bdfinst @Madeline-UX and @Jessy-Morris

@Racer159 I have some broader concerns about the navigation of this page and some ideas for improvement but i think this change adds value now so i added those thoughts to a new issue so we do not hold up this PR


Rename "Torque Heat" to "Heatmap"

Also removes the check for Heat feature flag, since this feature is irrevocably public  :stuck_out_tongue:

@saleiva @javisantana

Frontend tests were OK :+1: (details)

retest this please

Frontend tests were OK :+1: (details)

:es:


Update virtual-machine-scale-sets-faq.md

Proposed by CSS escalation engineer Chad Mathiason <NORTHAMERICA\chadmat>, approved by Cheryl McGuire.

#sign-off

@gatneil - As the author, please review these proposed changes for your article. If you approve the changes, please add your sign off comment. Thanks
#hold-off

I'm actually not sure if a public IP on a scale set VM can be static or not (I know private can't). @gbowerman do you know?

Okay, did a bit of digging. This is indeed not currently supported, but I disagree with the second sentence about VMs constantly being scaled up and down. This is just one pattern that scale set customers have; there are plenty of customers that don't ever scale up or down: they just use the scale set management APIs on static infrastructure. So could we do this instead?
### Is it possible to have a public static IP address for a VM in a scale set?
No, this is not currently supported.

Agree.

From: Neil Sant Gat<EMAIL_ADDRESS>Sent: Tuesday, October 3, 2017 2:30 PM
To: MicrosoftDocs/azure-docs
Cc: Guy Bowerman; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Update virtual-machine-scale-sets-faq.md (#3254)
Okay, did a bit of digging. This is indeed not currently supported, but I disagree with the second sentence about VMs constantly being scaled up and down. This is just one pattern that scale set customers have; there are plenty of customers that don't ever scale up or down: they just use the scale set management APIs on static infrastructure. So could we do this instead?
Is it possible to have a public static IP address for a VM in a scale set?
No, this is not currently supported.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/3254#issuecomment-333984463, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AED8YYKRLLMJWtZBW3reCh3rzxaHLW8Zks5soqdcgaJpZM4Pn0cT.

@JarrettRenshaw  Jarrett, would the updated text work? Please work with  @gatneil  to finalize the text. Thanks!

@gatneil @JarrettRenshaw - I'm closing this as it's been two weeks since the last interaction on this PR. Jarrett, you should be able to reopen if you decide to move forward with this.


feat(expressive): update checkbox, accordion, tile, and tabs components to expressive theme

Related Ticket(s)
#873 Expressive – Tabs
#825 Expressive theme – Checkbox
#861 Expressive – Accordion
#871 Expressive – Tile
Description
Edit styles for checkbox, accordion, and tabs for expressive theme.
See related tickets for details.
Changelog
Changed

bump carbon-components-react and carbon-components versions to get latest
add accordion, tile, and tabs files

@annawen1 it looks like there's something going on with the patterns react package that's failing, I saw this in the logs:

This looks good, but should probably check on that build failure first before we merge this in.

Fixed the failed build - but LeadSpace now looks weird...checking that


New

👷 Deploy Preview for crpp-ai-oncology processing.

Name
Link

🔨 Latest commit
92fed2b3f2e3206441430ef88e47a8b38cddbd22

🔍 Latest deploy log
https://app.netlify.com/sites/crpp-ai-oncology/deploys/638625f87c563d00071c2017


[Redirect] don't import entire history module

Hello, I was looking through my webpack bundle and found this optimization to reduce bundle size.
With these changes, my bundle goes from 797.3kb to 782.9kb, a savings of 14.4kb.
Before:

After:

I've also updated the jest config so the unit test failures pass. These unit tests were failing prior to this PR -- they are not a regression introduced by this PR.

@timdorr Are you sure about this one? The current import style was chosen intentionally in #5589.
This new PR prevents webpack from being pointed at the ESM build of history, and since react-router now contains some imports from 'history' (which will correctly point at the ESM build) and some from 'history/LocationUtils' (which will not), an ESM build will actually contain LocationUtils and its imports twice, once as ESM and once as CJS.
A better fix would be to add {sideEffects: false} in the history module's package.json, for which there is an open PR: https://github.com/ReactTraining/history/pull/597. This would provide the best of both worlds.
Please consider reverting this before releasing a new version of react-router. 🙂


[Enh]: adding the method min_horizontal

We would like to learn about your use case. For example, if this feature is needed  to adopt Narwhals in an open source project, could you please enter the link to it below?
No response
Please describe the purpose of the new feature or describe the problem to solve.
In an implementation I'm working on using Narwhal, having a method like min_horizontal, as in Polars, would be very useful.
Suggest a solution if possible.
No response
If you have tried alternatives, please describe them below.
No response
Additional information that may help us understand your needs.
No response

Thanks for your request! Definitely welcome to add this


Fix parsing 'client_oid' in websocket feed

This field is showing up as client_oid:"" but the empty
string could not be parsed as a UUID and was causing the parse to
fail. We are ignoring this field if it cannot be parsed as a UUID now.
It has become "opaque" in that case.

Thanks for catching this. I believe you mean (.!=), it is definitely more idiomatic. Will make the change.


Display the network that the game will be on

Does this display the network that the game will be on? I search Google to see what games will be on for the week and also on what network they will be live on. I don't have NBATV. I only watch games on ESPN or TNT so that's what I'm normally searching Google for.

Hi @MrLuisAmador
The API I used does have this information, so this feature should be easy to accomplish. I’m busy these weeks, so PR is welcomed!

Just wanted to check in on this. Broke out what I have done so far below. One thing to note is how I handled national broadcasts vs local broadcasts. What I do is always use the local broadcast network but if there is no local broadcast network then I check the national broadcast network and if there is one I list that. If there are neither local or national for some reason I list N/A as the broadcast. From what I see there will never be a scenario for N/A being listed as broadcast network. So really I just have some code clean up I want to do since I am sort of new to using javascript in this way. Once I have that complete I will submit a pull requests. Let me know any thoughts/concerns/changes to the below you may have. Thanks!
Schedule
I add a command line options to go with the game/g command that is either -n or --networks that will have the broadcast networks show on the schedule (see below screenshot).

Live Game
When selecting a live game the broadcast network will show below each respective team record (see below screenshot).

Game Preview
When previewing a game the home/away broadcast will show below arena/city information (see below screenshots).

Completed Game
When looking at a completed game the broadcast networks will show below the arena/city information. I am not sure if broadcast network information is necessary for a completed game? (see below screenshot).

That's really awesome!!! Thanks for your contribution.
I have only one thought. The location of network in LIVE game is just a little bit strange. I prefer it 's placed below the stadium or above game date. Will this cause any inconvenience?

Nope I do not think that will be a problem at all. I will look into doing that.

Went ahead and made that change for the LIVE game as seen in the two screenshots below. Let me know what you think. If it looks good to you then I am going to work on some code cleanup and should have a pull request ready in a couple days.

It looks great! Perfect!

@xxhomey19 @justinrsmith
I'm testing the area "which game do you want to watch" and I'm not seeing the broadcasts channel like in the above image demo.

@MrLuisAmador glad you got it. I agree with @xxhomey19 that it should be option purely due to how wide it can make the table depending on screen/terminal size it could cause some issues. We can always revisit later if that is something that is decided it should always show.

@justinrsmith, thank you for adding this feature in the first place. Wish I could contribute but I just can't wrap my head around the code. I'll keep messing around until I get a grip on how the source code works.
@xxhomey19 Thank you too for such a great cli tool.

@MrLuisAmador You are welcome. Ya it took me a bit of time to get my mind around it too. Definitely was a learning experience!


Fix issue 904

Pre-commit hooks failed:
[INFO] Initializing environment for https://github.com/gruntwork-io/pre-commit.
goimports................................................................Failed
- hook id: goimports
- files were modified by this hook

modules/terraform/var-file.go

Please run goimports!

@brikis98 sorry for the inconvenience, hopefully now the Pre-commit hooks will not fail
Run goimports
Rebased on latest master and pushed

Great, thanks! I'll re-run tests now.

Hm, I think CircleCi is having some sort of outage right now...

@brikis98 The CircleCI Jobs queuing and slow UI loading incident  was resolved and the build-and-test workflow run successfully

https://github.com/gruntwork-io/terratest/releases/tag/v0.35.1


Features: Configurable affinity, tolerations, and topology constraints

Feature Request: Support admin defined affinity and tolerations - source @ryanfaircloth
Moves the static affinity to values.yaml as the default to avoid breakage and correctly combines the requirements so arm64 and ppc nodes can not be selected.
Feature Request: Add TopologySpreadConstraints support to the humiocluster CRD - source @ryanfaircloth
This PR fixes #342 with the addition of TopologySpreadConstraints to the CR. Node instances of operator deployed by helm will not automatically update the CR for those instances the CR must be applied manually

@jswoods agree how tests are built isn't easily understood for a new contributor if you have pointers I don't mind doing the work


Am I supposed to be able to read "Open Collective" on the popperjs homepage?

Steps to reproduce the problem

Go to https://popper.js.org/
Have adblocker enabled
Attempt to read the text shown above

What is the expected behavior?
People are able to read the text
What went wrong?
People are not able to read the text
Any other comments?
Website: https://popper.js.org/
Full screenshot:

Yellow is #FFE69D Background is #FFE4E0
Screenshot from https://accessible-colors.com/:

Thank you in advance for looking into this.

Yeah I figured out afterwards that this is the link color used across the website, so that makes sense.
I forked the repo and found in src/components/Layout.js line 300 it has color: #ffe69d; for a; I'm assuming this is the correct location (I'm bad at navigating larger projects with complex structures).
I would suggest changing it to #c83b50 / rgb(200, 59, 80). This is the color used in the title below, and has sufficient contrast with the background to be legible, as well as sufficient contrast with the rest of the text to be noticeably different in function (as a clickable item), and of course maintains the brand colors.

In the screenshot I also changed the border-bottom color to rgba(200, 59, 80, 0.25), as the previous yellow color also has 0.25 opacity. (Current border-bottom is set on line 303 of the same file to border-bottom: 2px solid rgba(255, 228, 148, 0.25)
Of course this should only be changed in the .carbon-text span, in order to maintain the link styles for the rest of the links. I believe I should modify carbonAds.css in the same components folder. Screenshot of potential changes in carbonAds.css:

.carbon-text {
  font-size: 14px;
}

+.carbon-text a {
+  color: #c83b50;
+  border-bottom: 2px solid rgba(200, 59, 80, 0.25);
+}

.carbon-text,
.carbon-poweredby {
  text-decoration: none;
  color: inherit;
}

Let me know if this is acceptable. I figured rather than spend my time figuring out how to test locally, create a pull request, etc, I'd at least confirm that both the color change is acceptable and that I have the right file. Please let me know how to "build" the project so I can test locally.

Yeah I figured out afterwards that this is the link color used across the website, so that makes sense.
I forked the repo and found in src/components/Layout.js line 300 it has color: #ffe69d; for a; I'm assuming this is the correct location (I'm bad at navigating larger projects with complex structures).
I would suggest changing it to #c83b50 / rgb(200, 59, 80). This is the color used in the title below, and has sufficient contrast with the background to be legible, as well as sufficient contrast with the rest of the text to be noticeably different in function (as a clickable item), and of course maintains the brand colors.

In the screenshot I also changed the border-bottom color to rgba(200, 59, 80, 0.25), as the previous yellow color also has 0.25 opacity. (Current border-bottom is set on line 303 of the same file to border-bottom: 2px solid rgba(255, 228, 148, 0.25)
Of course this should only be changed in the .carbon-text span, in order to maintain the link styles for the rest of the links. I believe I should modify carbonAds.css in the same components folder. Screenshot of potential changes in carbonAds.css:

.carbon-text {
  font-size: 14px;
}

+.carbon-text a {
+  color: #c83b50;
+  border-bottom: 2px solid rgba(200, 59, 80, 0.25);
+}

.carbon-text,
.carbon-poweredby {
  text-decoration: none;
  color: inherit;
}

Let me know if this is acceptable. I figured rather than spend my time figuring out how to test locally, create a pull request, etc, I'd at least confirm that both the color change is acceptable and that I have the right file. Please let me know how to "build" the project so I can test locally.

Looks good! Please send a PR and I'll merge it.

Looks good! Please send a PR and I'll merge it.

Sure, thank you.
I tried running yarn then yarn develop (took a long time lol) but going to public/index.html it's an empty page with the following errors logged in console:
GET file:///C:/socket.io/socket.io.js net::ERR_FILE_NOT_FOUND
/C:/favicon-32x32.png?v=fee0a7bffcef99cffef49f85c7525731:1 GET file:///C:/favicon-32x32.png?v=fee0a7bffcef99cffef49f85c7525731 net::ERR_FILE_NOT_FOUND

full cmd output:
Microsoft Windows [Version 10.0.19042.685]
(c) 2020 Microsoft Corporation. All rights reserved.

C:\Users\justi\OneDrive\Documents\GitHub\website>yarn
yarn install v1.22.10
[1/4] Resolving packages...
[2/4] Fetching packages...
warning<EMAIL_ADDRESS>Invalid bin field for "url-loader".
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@emotion/styled ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@pauliescanlon/gatsby-mdx-routes > gatsby ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>|| ^16.0.0".
warning "@pauliescanlon/gatsby-mdx-routes > gatsby > @typescript-eslint/eslint-plugin ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta".
warning "gatsby-plugin-emotion > @emotion/babel-preset-css-prop ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "gatsby-plugin-emotion ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "gatsby-plugin-emotion > @emotion/babel-preset-css-prop ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "react-helmet ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.14.0 || ^15.0.0 || ^16.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...
$ patch-package
patch-package 6.2.2
Applying patches...
<EMAIL_ADDRESS>✔
Done in 93.80s.

C:\Users\justi\OneDrive\Documents\GitHub\website>yarn develop
yarn run v1.22.10
$ gatsby develop
╔════════════════════════════════════════════════════════════════════════╗
║                                                                        ║
║   Gatsby collects anonymous usage analytics                            ║
║   to help improve Gatsby for all users.                                ║
║                                                                        ║
║   If you'd like to opt-out, you can use `gatsby telemetry --disable`   ║
║   To learn more, checkout https://gatsby.dev/telemetry                 ║
║                                                                        ║
╚════════════════════════════════════════════════════════════════════════╝
success open and validate gatsby-configs - 0.102s
success load plugins - 3.064s
success onPreInit - 0.026s
success initialize cache - 0.022s
success copy gatsby files - 0.216s
success onPreBootstrap - 0.022s
success createSchemaCustomization - 0.012s
warn code block or inline code language not specified in markdown. applying generic code block
warn Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db
success source and transform nodes - 14.363s
success building schema - 0.522s
success create redirects
success createPages - 0.042s
success createPagesStatefully - 2.938s
success onPreExtractQueries - 0.003s
success update schema - 0.039s
success extract queries from components - 0.712s
success write out requires - 0.214s
success write out redirect data - 0.009s
success Build manifest and related icons - 0.213s
success onPostBootstrap - 0.232s

info bootstrap finished - 28.549 s

success run queries - 2.492s - 42/42 16.85/s
success Generating image thumbnails - 16.624s - 74/74 4.45/s

You can now view gatsby-starter-default in the browser.

  http://localhost:8000/

View GraphiQL, an in-browser IDE, to explore your site's data and schema

  http://localhost:8000/___graphql

Note that the development build is not optimized.
To create a production build, use gatsby build

success Building development bundle - 34.555s

————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
38 pages                                          SUCCESS                                         gatsby-starter-default

Sure, thank you.
I tried running yarn then yarn develop (took a long time lol) but going to public/index.html it's an empty page with the following errors logged in console:
GET file:///C:/socket.io/socket.io.js net::ERR_FILE_NOT_FOUND
/C:/favicon-32x32.png?v=fee0a7bffcef99cffef49f85c7525731:1 GET file:///C:/favicon-32x32.png?v=fee0a7bffcef99cffef49f85c7525731 net::ERR_FILE_NOT_FOUND

full cmd output:
Microsoft Windows [Version 10.0.19042.685]
(c) 2020 Microsoft Corporation. All rights reserved.

C:\Users\justi\OneDrive\Documents\GitHub\website>yarn
yarn install v1.22.10
[1/4] Resolving packages...
[2/4] Fetching packages...
warning<EMAIL_ADDRESS>Invalid bin field for "url-loader".
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@emotion/styled ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@pauliescanlon/gatsby-mdx-routes > gatsby ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>|| ^16.0.0".
warning "@pauliescanlon/gatsby-mdx-routes > gatsby > @typescript-eslint/eslint-plugin ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta".
warning "gatsby-plugin-emotion > @emotion/babel-preset-css-prop ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "gatsby-plugin-emotion ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "gatsby-plugin-emotion > @emotion/babel-preset-css-prop ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "react-helmet ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.14.0 || ^15.0.0 || ^16.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...
$ patch-package
patch-package 6.2.2
Applying patches...
<EMAIL_ADDRESS>✔
Done in 93.80s.

C:\Users\justi\OneDrive\Documents\GitHub\website>yarn develop
yarn run v1.22.10
$ gatsby develop
╔════════════════════════════════════════════════════════════════════════╗
║                                                                        ║
║   Gatsby collects anonymous usage analytics                            ║
║   to help improve Gatsby for all users.                                ║
║                                                                        ║
║   If you'd like to opt-out, you can use `gatsby telemetry --disable`   ║
║   To learn more, checkout https://gatsby.dev/telemetry                 ║
║                                                                        ║
╚════════════════════════════════════════════════════════════════════════╝
success open and validate gatsby-configs - 0.102s
success load plugins - 3.064s
success onPreInit - 0.026s
success initialize cache - 0.022s
success copy gatsby files - 0.216s
success onPreBootstrap - 0.022s
success createSchemaCustomization - 0.012s
warn code block or inline code language not specified in markdown. applying generic code block
warn Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db
success source and transform nodes - 14.363s
success building schema - 0.522s
success create redirects
success createPages - 0.042s
success createPagesStatefully - 2.938s
success onPreExtractQueries - 0.003s
success update schema - 0.039s
success extract queries from components - 0.712s
success write out requires - 0.214s
success write out redirect data - 0.009s
success Build manifest and related icons - 0.213s
success onPostBootstrap - 0.232s

info bootstrap finished - 28.549 s

success run queries - 2.492s - 42/42 16.85/s
success Generating image thumbnails - 16.624s - 74/74 4.45/s

You can now view gatsby-starter-default in the browser.

  http://localhost:8000/

View GraphiQL, an in-browser IDE, to explore your site's data and schema

  http://localhost:8000/___graphql

Note that the development build is not optimized.
To create a production build, use gatsby build

success Building development bundle - 34.555s

————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
38 pages                                          SUCCESS                                         gatsby-starter-default


Slightly incorrect time displayed

I just published a package to hex.pm, and noticed that the time displayed in the site is slightly incorrect:

For some reason, it's assuming it will be published 7 minutes in the future, which is odd, to say the least! This seems to eliminate timezone problems, because it's not by a whole hour or anything close to that, just an odd number of minutes. I looked through the source code trying to find a culprit, but didn't manage to find anything that looked suspicious...

Very mysterious, I'm not sure what this might be.

It might just be that hex.pm is returning an incorrect/too imprecise date

A simple way to prevent this visual glitch would be to show something like "Just now!" when a package is less than one hour old, for example

Hey that's smart. We should do that


Async middleware

I have a use case to make an internal service call before all intents. One enhancement to this library I'd like to see to support this is for ActionsSdkApp.middleware to take async functions. I've had to hack around this in the time being in an Express middleware.

Yes, I would expect app.middleware to behave similarly to app.intent and support async. I'd be surprised if you couldn't make this enhancement to app.middleware in a backwards compatible way.

It's added now in 2.2.0
Try it and let me know us know of any feedback.

Thanks @Canain. I've already made the switch and rolled it out. Looking good.


App Crashes when Camera Opens in MI Note 7s

I am Encountering an issue with Flutter upload image functionality. when I attempt to upload using Mi brand phones, the camera abruptly shuts down and the application restarts. Currently, I am using the Redmi Note 7S, and this problem seems to be specific to Mi phones. On other brands of phones the upload feature works fine. Unfortunately, there are no specific error messages displayed. It showing  loss of device connection in logs. It could possibly be an issue with the Flutter plugin. I am using "image_picker_plus: ^0.5.6+2".

try to use #65 and check updates


Documentation updates

Just minor documentation updates

Thanks for this @GottfriedHerold ! These all look great.
Addressed some CI issues with 309a4eb and straightened up some multiline comments that looked ugly after make format with 2d73475. I will merge later today.


Advanced expressions randomize incorrectly

Currently, !roll 100 is the same as !roll d100 or !roll 1d100, but !roll 100+1 always gives 101, instead of rolling 100 and adding, unlike !roll 1d100+1

!roll 100 is a special-case format that gets interpreted as !roll d100. It will not be possible to omit the d for anything other than a single plain number, as anything else is considered a full dice expression. You will need to do !roll d100 + 1.

That's exactly my point it should be considered an invalid expression but currently it just does the math instead of throwing an error, allowing for the possibility of cheating if you are crafty. Desired behavior is either to throw an error or work exactly the same way as with the data identifier.
This is a very easy mistake to make accidentally, my group did it 3 times in one roll before we figured out that it's a bug.

It's behaving exactly as intended, and is definitely not a bug. There's no way to distinguish whether you would want d100 + 1, d100 + d1, 100 + d1, or 100 + 1. In this case, there is one obviously correct choice, but that won't always be the case. What about 5 + 4, for example?

Yeah, that's kind of my point, no matter which of those expressions you intended, nobody is expecting it to add the numbers together when you are invoking a !roll command. There is no correct interpretation here, so the sensible thing is to instead refuse to do anything.


add malloc fail check & fix memory leak

Signed-off-by: Hongze Zhu<EMAIL_ADDRESS>

+1 from me; @mattcaswell do you want to submit it?

Merged. Thanks for your contribution.

thank you:)


[SPARK-21023][Submit] Ignore to load default properties file is not a…

… good choice from the perspective of system
What changes were proposed in this pull request?
The default properties file spark-defaults.conf shouldn't be ignore to load even though the submit arg --properties-file is set. The reasons are very easy to see:

Infrastructure team need continually update the spark-defaults.conf when they want set something as default for entire cluster as a tuning purpose.
Application developer only want to override the parameters they really want rather than others they even doesn't know (Set by infrastructure team).
The purpose of using --properties-file from most of application developers is to avoid setting dozens of --conf k=v. But if spark-defaults.conf is ignored, the behaviour becomes unexpected finally.

All this caused by below codes:
  private Properties loadPropertiesFile() throws IOException {
    Properties props = new Properties();
    File propsFile;
    if (propertiesFile != null) {
    // default conf property file will not be loaded when app developer use --properties-file as a submit args
      propsFile = new File(propertiesFile);
      checkArgument(propsFile.isFile(), "Invalid properties file '%s'.", propertiesFile);
    } else {
      propsFile = new File(getConfDir(), DEFAULT_PROPERTIES_FILE);
    }

    //...

    return props;
  }

How was this patch tested?
Modify the UT of SparkSubmitCommandBuilderSuite.java

Can one of the admins verify this patch?

See discussion in the bug; we shouldn't change the current behavior; but adding this as a new feature (e.g. a separate spark-submit parameter, or a user-specific config file with a default location that can also be overridden somehow) is fine.
Also, this is not the only place where that properties file is loaded.

Move the comment to here

I suggest to change the current behavior and offer a document to illustrate this. --properties-file will overwrite the args which are set in spark-defaults.conf first. It's equivalent to set dozens of --conf k=v in command line. Please review and open for any ideas.

I know change an existing behavior is painful. But the purpose is making the default configuration loaded anytime.

Please keep the comments in the bug. A breaking change a lot of times makes it a no go - and this is one of the cases.

Also, this is not the only place where that properties file is loaded.

Yup, will continue contribute.

Open a new ticket SPARK-21048 as another solution. Close it cause the purpose is invaild.


update just-the-docs theme(fix from Ron)

Domino REST API Documentation Update
Change description
Due diligence

[ ] tested locally using bundle exec jekyll serve

Review/Merge

[x] Ready for review
[x] Ready for merge

Follow-up work
List any out-of-scope work that you identified

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Will not work


Add a comma to the workflow for artifacts uploads

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Add a comma to the workflow for artifacts uploads
Issues Resolved
#3012
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #3176 (aa29bf3) into main (3f43167) will decrease coverage by 0.31%.
The diff coverage is n/a.

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

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #3176      +/-   ##
==========================================
- Coverage   93.19%   92.89%   -0.31%
==========================================
  Files         167      167
  Lines        4614     4614
==========================================
- Hits         4300     4286      -14
- Misses        314      328      +14

Impacted Files
Coverage Δ

src/system/temporary_directory.py
64.44% <0.00%> (-28.89%)
:arrow_down:

src/system/os.py
94.73% <0.00%> (-5.27%)
:arrow_down:

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


Remove fake::Kernel::new's name argument, use #[track_caller] instead.

Instead of asking each unit test to name its kernel, we can instead use Rust's #[track_caller] feature to identify each kernel instance. This reduces the amount of boilerplate used in each unit test, and eliminates the chance of copy+paste resulting in two tests with identically-named kernels.
I will manually deconflict this with my other PRs.

bors: r+


[BUG] sed syntax error in creating tsduck.pc pkgconfig file with NOCURL=true

Bug description:
If I build tsduck with NOCURL=true, the "make install" step cannot create the tsduck.pc pkgconfig file, due to an sed syntax error.
How to reproduce:
make install DESTDIR=/mybuildinstallroot INSTALL="/usr/bin/install -p" install-devel NOTEST=true NODEKTEC=true NOCURL=true NOEDITLINE=true NOPCSC=true NORIST=true NOSRT=true SYSPREFIX=/usr ETCDIR=/etc USRLIBDIR=/usr/lib64 UDEVDIR=/usr/lib/udev/rules.d SYSROOT=/mybuildinstallroot

Expected behavior:
tsduck.pc is created and build succeeds.
 Partly because I can, partly because librist isn't available on Fedora mainline. (Yes, I'm aware of https://github.com/tsduck/rist-installer .)

Fixed in commit a0ad9310bfd346515aef1aef6e96758e3bdf2a5c

Obviously, in src/libtsduck/config/Makefile, the line
$(if $(NOCURL),-e 's|libcurl||g',)
should be
$(if $(NOCURL),-e 's|libcurl||d',) \

I assume you mean the opposite. It was "d" and it should be "g".

Just BTW: I'm creating a "more modern" RPM SPEC file, and adding SPEC options to disable each feature separately.

I am curious of what you mean by "more modern spec file".
Here, the update of the .pc file shall be done during make install since the installation shall be independent from RPM.

Partly because I can, partly because librist isn't available on Fedora mainline.

The absence of librist is already automatically detected, NORIST is automatically set and the dependency to librist is automatically removed from the spec file. Indeed, the presence of "Requires: librist" in the spec file is provisional since librist is not present in any Linux distro for now. I do not even know if it will be named like that if/when it is added.

Automatically closed after 290 days without update and "close pending" label set.


Is this project dead?

No updates for months. Is this project still alive?

no, it's not. I forked the repo and a community team will continue to develop RAIN. :)


Is it possible to know the rows length without XLSX.read?

Hi,
I have to know the rows length without reading the file at all - because reading the file in order to get the rows length takes me a lot of time.. Is there such a possibility?
Edit: I have found out that it takes me A LOT more time if the my excel file has macros in it.
Is there any possibility to send a flag to not pay attention to the macros?
Thanks

@ronilitman As I understand it, the worksheet self-reports its range.  XLSX stores the cell range in the <dimension> tag: https://github.com/SheetJS/js-xlsx/blob/master/bits/67_wsxml.js#L16
The range may not be correct.  Excel will "do the right thing" by ignoring the dimension field, but that requires reading the whole sheet to get the correct range.
Related issue https://github.com/SheetJS/js-xlsx/issues/189
@sheetjsdev is it theoretically possible to scan the entire sheet and get the addresses without having to generate a cell object for every cell?

@ronilitman As I understand it, the worksheet self-reports its range. XLSX stores the cell range in the <dimension> tag: https://github.com/SheetJS/js-xlsx/blob/master/bits/67_wsxml.js#L16
The range may not be correct. Excel will "do the right thing" by ignoring the dimension field, but that requires reading the whole sheet to get the correct range.
Related issues #189 #82
@SheetJSDev is it theoretically possible to scan the entire sheet and get the addresses without having to generate a cell object for every cell?

@ronilitman by the way, how to get the progress when reading the file ?

The technical answer depends on file format:
Some formats like CSV don't report the range anywhere and have variable sized rows, so the only way to know the total number of records is to effectively parse the whole thing.
Other formats like DBF have readily computable record counts based on the size since the header tells you how large each row payload must be.
The interesting formats generally have a way of self-reporting ranges but these are self-reported.  A number of third party generators are known to hack around this.  Third party hacks have made the data source unreliable, and resolving #1601 will involve changing the behavior anyway.
So the complete and unfortunate answer is "no, it's not possible to correctly determine the number of rows without scanning the entire worksheet".
As @reviewher mentioned, it is possible to just avoid generating cells, but it's unclear if the payoff is worth it (especially if the file will have to be re-parsed to actually extract the data)
@VN666 #632 is tracking "progress" related issues


How can I do add in row adapter recyclerview?

I want to add mosaic layout in row adapter, it's same the row newfeed's facebook.
View Image
Please help to do this. Thanks.

@phamthangnd yes you can. If you still can't do it, tell me to discuss it further..

@adhamenaya If you can, Can you add these to "how to use", at this time, I custom basic layout to do my case. Thanks for hard working.

@adhamenaya please provide a working sample/code snippet of using AndroidMosaicLayout with RecyclerView. I have spent a significant time to figure out but needs your expert views in this regard. Thank you.

@adhamenaya can we provide Pattern at run time? As I need only Big and small items. and that I came to know from backend that items ll be big or small.

Hi @CodeBrewRishi, you can set the the pattern in the runtime, but you can only choose from predefined options of patterns, as mean you can set a list of all possible patterns, and switch between them according to the data type you get from the backend. For example if you have all big images, you can use a repeated pattern of 2 BIG block and so one!
Notes! you need to be aware of the order of the data returned, I mean BIG, SMALL, BIG ..etc... because the pattern order a group of blocks according to how you define your patterm, so you need to pay attention to the fitting your images to the blocks in the selected pattern.

Hi @Adham Enaya  Thanks for quick response. Actually the thing is I am not
aware about the pattern. I can get Big item any time. So don't have any
specific pattern.
One thing I can do is like I have a paging of 20 20 items then I can send
all big item before and then small out of 20 items.? ll it work for you.?
On Wed, Dec 28, 2016 at 10:58 PM, Adham Enaya<EMAIL_ADDRESS>wrote:

Hi @CodeBrewRishi https://github.com/CodeBrewRishi, you can set the the
pattern in the runtime, but you can only choose from predefined options of
patterns, as mean you can set a list of all possible patterns, and switch
between them according to the data type you get from the backend. For
example if you have all big images, you can use a repeated pattern of 2 BIG
block and so one!
Notes! you need to be aware of the order of the data returned, I mean BIG,
SMALL, BIG ..etc... because the pattern order a group of blocks according
to how you define your patterm, so you need to pay attention to the fitting
your images to the blocks in the selected pattern.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/adhamenaya/AndroidMosaicLayout/issues/4#issuecomment-269510315,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMrDoY-j3ucPVdIhbKAK8B-mVXKVGgYcks5rMpxSgaJpZM4H4o9d
.

--
Thanks & Regards
Rishi Paul Sharma
Android Application Developer
CODE BREW LABS


missing conway parameters in protocol parameters update

Current main
cardano-cli --version
cardano-cli <IP_ADDRESS> - linux-x86_64 - ghc-9.2
git rev 20b7b2b1e81cbb580736c93062be85485ac98ded

The command to propose protocol parameter changes presents the following issues:

~No conway-era parameters~
There is no way to provide proposal deposit
There is no way to provide stake address (for the deposit return)
No way to provide an anchor for the proposal.

cardano-cli conway governance action create-protocol-parameters-update
Usage: cardano-cli conway governance action create-protocol-parameters-update --epoch NATURAL
                                                                                [--min-fee-constant LOVELACE]
                                                                                [--min-fee-linear LOVELACE]
                                                                                [--max-block-body-size NATURAL]
                                                                                [--max-tx-size NATURAL]
                                                                                [--max-block-header-size NATURAL]
                                                                                [--key-reg-deposit-amt NATURAL]
                                                                                [--pool-reg-deposit NATURAL]
                                                                                [--pool-retirement-epoch-boundary EPOCH_BOUNDARY]
                                                                                [--number-of-pools NATURAL]
                                                                                [--pool-influence RATIONAL]
                                                                                [--treasury-expansion RATIONAL]
                                                                                [--monetary-expansion RATIONAL]
                                                                                [--protocol-major-version NATURAL
                                                                                --protocol-minor-version NATURAL]
                                                                                [--min-pool-cost NATURAL]
                                                                                [--price-execution-steps RATIONAL
                                                                                --price-execution-memory RATIONAL]
                                                                                [--max-tx-execution-units (INT, INT)]
                                                                                [--max-block-execution-units (INT, INT)]
                                                                                [--max-value-size INT]
                                                                                [--collateral-percent INT]
                                                                                [--max-collateral-inputs INT]
                                                                                [--utxo-cost-per-byte LOVELACE]
                                                                                --out-file FILE

as discussed, it's incomplete.

Resolved in https://github.com/input-output-hk/cardano-cli/pull/369


Invitation from aca-py agent with aca-py mediator between

Hey.
I'm trying to make a connection (from https://github.com/hyperledger/aries-mobile-agent-react-native) with another agent with a mediator between.
I can connect from react  native app (using code from this repo) to aca-py mediator, but had to move the invitation to another server (/invitation path does not exist in aca-py admin). I don't know if it's good, but it works.
The issue is when I want to connect with another aca-py agent with mediator between. When I try to receive invitation on mobile I'm getting message in react native app:
New Connection Record
{
    "_didDoc":
    {
        "@context": "https://w3id.org/did/v1",
        "authentication":
        [
            [
                undefined
            ]
        ],
        "id": "Cv5XNG6AW7bQNSz49M3a8h",
        "publicKey":
        [
            [
                undefined
            ]
        ],
        "service":
        [
            [
                undefined
            ]
        ]
    },
    "_invitation":
    {
        "@id": "859fec9e-ad20-4fe1-bc5b-2585e52636cc",
        "@type": "https://didcomm.org/connections/1.0/invitation",
        "label": "label",
        "recipientKeys":
        [
            "G9JytRnX8tx6uSetjPyYuSkwwVrDFm23LB8GGK7ifB4k"
        ],
        "routingKeys":
        [
            "8zNjW3SZt9Tc94dmHjpGdm5qGL8EFQNPMXVCHqJR88ff"
        ],
        "serviceEndpoint": "http://x:3006/"
    },
    "_theirDidDoc": undefined,
    "alias": undefined,
    "autoAcceptConnection": true,
    "createdAt":<PHONE_NUMBER>615,
    "did": "Cv5XNG6AW7bQNSz49M3a8h",
    "endpoint": undefined,
    "id": "d12851bc-f05c-46af-8c7a-721023b1a960",
    "role": "INVITEE",
    "state": "requested",
    "tags":
    {
        "invitationKey": "G9JytRnX8tx6uSetjPyYuSkwwVrDFm23LB8GGK7ifB4k",
        "verkey": "7VfLb6mWcLz3apc3Z1dQxcytAghm4E4c2iKppVYdLXB3"
    },
    "theirDid": undefined,
    "type": "ConnectionRecord",
    "verkey": "7VfLb6mWcLz3apc3Z1dQxcytAghm4E4c2iKppVYdLXB3"
}

But I'm getting an error on edge agent (mediator looks good - it shows messages regarding forwarded messages):
2021-04-10 07:17:57,143 aries_cloudagent.core.conductor WARNING Cannot queue message for delivery, no supported transport

Invitation created by edge agent looks like this:
{
  "connection_id": "1281a500-aa9b-49ba-b8b8-7dba030772ba",
  "invitation": {
    "@type": "https://didcomm.org/connections/1.0/invitation",
    "@id": "16be3a5c-4cc7-4c2f-a182-6defe6665930",
    "recipientKeys": [
      "4hxjdd6u3iHyi9WnPfesh9nDvwAKmGZiskw1NiriWMNr"
    ],
    "label": "Edge",
    "serviceEndpoint": "http://x:3006/",
    "routingKeys": [
      "8zNjW3SZt9Tc94dmHjpGdm5qGL8EFQNPMXVCHqJR88ff"
    ]
  },
  "invitation_url": "http://x:3006/?c_i=invitation base 64"
}

I copy invitation url, then paste it to qr generator and scan it.

Hi @papkie, I helped work on the mediator code and would like to help. Could you post more error logs, nothing is jumping out with the current details.

Whoops I missed your message @papkie! I think the problem here is that you're trying to connect to an ACA-Py mediator. We don't fully support the mediator coordination protocol yet which means we can't request mediation from ACA-Py....
I believe @JamesKEbert is working on this, but until then you can only use an aries framework javascript mediator inside the mobile agent

@burdettadam I have investigated it a bit more later, and it's probably the issue with aca-py agent. I'm getting message related to unsupported batch pickup type:
aries_cloudagent.core.dispatcher ERROR Message parsing failed: Unrecognized message type https://didcomm.org/messagepickup/1.0/batch-pickup, sending problem report on aca-py agent

Whoops I missed your message @papkie! I think the problem here is that you're trying to connect to an ACA-Py mediator. We don't fully support the mediator coordination protocol yet which means we can't request mediation from ACA-Py....
I believe @JamesKEbert is working on this, but until then you can only use an aries framework javascript mediator inside the mobile agent

Understood, thank you!

Sadly, batch-pickup is not implemented in ACA-py. ACA-py uses websockets for mediation. Currently, they are being implemented in AFJ #218
I have used a workaround for this. You can use AFJ mediator instead of ACA-py mediator. Then you can connect to ACA-py agent using that mediator.

@icc-romeu
I have tried to use AFJ as a mediator (using the script in samples/mediator.ts).
I use samples/mediator.ts code to set up the mediator.
Edge agent (mobile one, javascript) has no issues connecting to it, but there is an issue with connecting this mediator to the ACA-Py edge client.
I'm trying to receive the invitation created by the mediator (AFJ - on /invitation path) on the ACA-Py edge client. ACA-Py sends a request with an empty body to /msg endpoint, so AFJ is not able to handle it.
When I try to receive an invitation created by ACA-Py on Javascript mediator, AFJ throws UnhandledPromiseRejectionWarning: Error: Trying to save the message without theirKey! error.
Do you have some example code somewhere?
If not, I'll just wait for WS support.

Maybe I do not understand what you are trying to do @papkie  but you do not need to connect ACA-py to the mediator. The process on  should be:

Connect from the mobile AFJ to the AFJ mediator via the invitation. This works as you say.
Connect from the mobile AFJ to the ACA-py using an ACA-py invitation. When the ACA-py starts it shows you an invitation URL. That's the one you should use once the mobile is setup with the mediator. AFJ will handle the connection with ACA-py and it will respond to your mediator instead of the mobile.

Maybe I do not understand what you are trying to do @papkie but you do not need to connect ACA-py to the mediator. The process on should be:
Connect from the mobile AFJ to the AFJ mediator via the invitation. This works as you say.
Connect from the mobile AFJ to the ACA-py using an ACA-py invitation. When the ACA-py starts it shows you an invitation URL. That's the one you should use once the mobile is setup with the mediator. AFJ will handle the connection with ACA-py and it will respond to your mediator instead of the mobile.

Maybe I do not understand what you are trying to do @papkie but you do not need to connect ACA-py to the mediator. The process on should be:
Connect from the mobile AFJ to the AFJ mediator via the invitation. This works as you say.
Connect from the mobile AFJ to the ACA-py using an ACA-py invitation. When the ACA-py starts it shows you an invitation URL. That's the one you should use once the mobile is setup with the mediator. AFJ will handle the connection with ACA-py and it will respond to your mediator instead of the mobile.

Thanks, I have tried this scenario, unfortunately, it didn't work.  I see there is mediation protocol implementation in progress, so I'll wait for that, thank you anyway!

Hi @icc-romeu, it has been two years and I think Aries Mobile agent has progress alots. so, it is possible to connect Aries Mobile Agent React Native to Mediator of ACA-Py? Currently, I cannot connect Aries Mobile Agent with Mediator of ACA-Py because Aries Mobile Agent sent the did:peer to Mediator of ACA-Py while ACA-Py accepts only Indy DID.

I believe, you can connect Aries bifold and Aca-py mediator. Because I did it last month . I follow this link to get my mediator up and running "https://github.com/hyperledger/aries-mediator-service".
Try for yourself and see if it works

@darapich92 you need to make sure to not use the did exchange protocol. You can either use connection invitations in the ACA-Py mediator, or use out of band invitations where the handshake_protocols does not include the did exchange protocol

Hi @icc-romeu, it has been two years and I think Aries Mobile agent has progress alots. so, it is possible to connect Aries Mobile Agent React Native to Mediator of ACA-Py? Currently, I cannot connect Aries Mobile Agent with Mediator of ACA-Py because Aries Mobile Agent sent the did:peer to Mediator of ACA-Py while ACA-Py accepts only Indy DID.

Sorry for the late reply.
I have been trying myself and I'm in the same point as you, no luck

Can you provide me with a reproduction? Having the message be undelivered can be because of a number of reasons, and I'd need to dig into it a bit more

Sorry @TimoGlastra I was wrong. Mediation does work. What does not work is pickup protocol.
 DEBUG  DEBUG: Agent received message
 INFO  INFO: Received message with type 'did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/notification/1.0/problem-report', recipient key z6MkmQpVr4xD2a5TejpDZv2zAfPgHQRM2DnjRN3cM8PtehJG and sender key z6MkgUYKsfH7WYYH3FEwUcKFsyN4T7mCDNEaWkzNEo61aagZ {
  "@type": "did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/notification/1.0/problem-report",
  "@id": "287a87ed-450f-4cba-b587-74167981102c",
  "~thread": {
    "thid": "8123f295-7666-4091-ab8e-e32387e08142"
  },
  "description": {
    "en": "Unrecognized message type https://didcomm.org/messagepickup/2.0/status-request",
    "code": "message-parse-failure"
  }
}
 ERROR  ERROR: Failed to process message {
  "error": {
    "name": "AriesFrameworkError",
    "message": "Not sending problem report in response to problem report: No message class found for message type \"https://didcomm.org/notification/1.0/problem-report\"",

Okay, thanks for clarifying. Are you using the pickup plugin for the ACA-Py mediator? https://github.com/Indicio-tech/acapy-plugin-pickup

Okay, thanks for clarifying. Are you using the pickup plugin for the ACA-Py mediator? https://github.com/Indicio-tech/acapy-plugin-pickup
Also, if you're mostly looking for a mediator for development, you can also use the docker image from this repo: https://github.com/animo/animo-mediator

WOOOPS! Maybe that's the issue 😅 We will check, thank you!

Are you using a customer ledger? In that case you need to add it to the ledgers.json in bifold

@TimoGlastra, Yes i use the customized ledger. so I need to modify ledgers.json in the configs/ledgers/indy. thank you for your information!

Can you share the config for your ledger?

Hi @TimoGlastra, I found the problem. i deleted some code because I want to have only one genesis that's the reason that it was stuck. I tried to have multiple genesis and it works now. thank you very much for your help!


PMM - unnecessary text in Promt for split_order_levels_enabled

Describe the bug
PMM - unnecessary text in Promt for split_order_levels_enabled
Steps:
Create/import pureMM strategy
Type config split_order_levels_enabled
Press TAB key to see promt
Actual:
There're unnecessary text
(This acts as an overrides which replaces order_amount, order_spreads, ""order_level_amount, order_level_spreads) (Yes )
instead of just Yes
Expected:
There's the (Yes/No) promt  (like for other true/false configs)
Screenshots

Release version
Staging v1.2

Also exists on the development 1.3.0 branch


How do we handle in case of aggregation backend error?

Hi Krakend Team,
Assume that I have 2 backend service (upstreams), I would like to know that what setting should be done in krakend.json to prevent the error whole request error?
for example:
I have following compoentns

endpoint: /a

backend /b and /c

I expected that if /c is failed, it should be return {} empty object, or krakend will be automatically handle error to {} empty object.
and finally response will be
response : {
  b: { success: {} }
  c: {}
}

Do you have any suggestion?

Hi, @iamgoangle
KrakenD is a very pluggable framework, so there are ways to customize almost every part of it. The default implementation will let the clients know if the response is incomplete via HTTP headers.
You can inject your custom proxy.ResponseCombiner into the register and update your endpoint configurations properly. Your implementation could check if every response has the IsComplete property enabled and return an empty response if so.
An easy alternative could be the static response feature. You can define a struct to be returned every time a backend fails.
There is a reason why the feature you commented is not supported out of the box: we think returning an incomplete response when some backend fails is a desired feature for the api gateway pattern, giving tons of resilence to the stack by supporting graceful degradation. With the current implementation, the endpoint A will fail just if and only if both backends B and C fail.
PeA = PeB x PeC
if PeB = PeC = 10%, PeA = 1%
With the behaviour you are looking to achieve, your endpoint will work if and only if both backends work, increasing the error rate dramatically.
PeA = 1 - ((1-PeB) x (1-PeC))
if PeB = PeC = 10%, PeA = 19%

Check out the existent literature on the net.
Cheers!

🤔  After reading your question one more time, I think with the static partials and groups you could get what you want.

I'm trying to create custom proxy, my solution is that when http request to backend failed (container down, connect lost, not only 4xx, 5xx returned), my proxy will be return it as empty object and moving forward to next() backend.
Honestly, I have no idea where should I started to create custom proxy and inject to proxy_factory.go

it does not work for me maybe I configured something wrong. I expected that if
endpoint: /
backend: /a /b

if /a failed maybe container not started, 4xx or 5xx returned it should not breaker all whole request.
response: {
  a: {},
  b: { ... success object ...}
}

Hi @iamgoangle let us check because there might be a problem here. Will get to you ASAP

Thanks @alombarte I would like to add some more feature about circuit breaker fallback. for example I can set default value i.e. empty response object. In case of circuit break in some backend die, before /proxy/merge.go do some action.
Do you have any suggestion which part of project should I looking for?

@iamgoangle it looks like you found a bug!
I'm updating the framework right now and the fix will be in the release planned for next week
thanks

Fixed!

Thanks!!!


XEP-0191: Blocking Command (for spam)

Users should be able to block messages. The most common use case is for preventing SMS spam. Ideally a block action can trigger an external script for adding the phone number to a blacklist (presumably to block phone calls through a PBX).

Upon further reading, XEP-0191 doesn't make sense for an external component. A client sends stanzas without a to attribute so the server handles them directly. An external component has no way of knowing when one entity has blocked another. Clients can still use XEP-0191 to block spam SMS locally but we can't use it to notify an external blacklist.
I'd still like the component to facilitate blocking of abusive phone numbers, but we'll need another way to do it. Maybe a "/block" command or something.


As a trainer, I want to be able to edit my training templates

Template edit button on profile page opens an editor popup window

[x] #345
[x] #346
[x] #347
[x] #348 -Laurel

[x] #349 -Laurel

[x] #350
[x] #351

Approved by customer


[Doc Improvement] [Update - Developer preview app manifest][3661653]

Backfilling existing dev preview and GA manifest reference docs with snippets of top-level elements that are missing.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
104
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
104
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
104
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Instead of copying over the schema description as the value for these properties, I think it'd be much more useful to show customers plausible example values to simulate how they might actually write their app manifest if they were using these features. For example, for the actions section, you could use values similar to the ones in actions sample manifest.
Longer term, as a separate doc improvement effort, it'd be awesome to ensure the entire manifest sample has plausible example values, rather than descriptions from the schema.

I agree, we should add examples instead of description.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
103
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
103
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
102
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
103
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
103
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
103
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
103
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/schema/manifest-schema-dev-preview.md
83
130
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Add Blacklist feature fixes #192

Allow the user to specify cites that Saka-key will be disabled on. This is done though the input of comma separated regular expressions. For example, if  "^https://google.com, git" is the input all pages that start with "https://google.com" or contain the "git" anywhere in the url will not have any Saka-key bindings.
I still need to add information to the tutorial (git-book), I am going to being doing that now and will make another pull request when finished.

for the next person: after entering text into that box, click on some other saka key popup element, then it saves it in the browser-extension-data.


About the training data

Great Work! It seems that ModelAngelo organizes a training set with good quality. Do you have plans to release them? I think it's a great resources for the computational CryoEM community.

Hi @Qmi3 ,
Sorry for the late reply! The original dataset with all the processing already done etc is too large for us to be able to release, due to the size of the maps associated with it.
What I can offer instead is the following list of PDB ids used for training. I can also (if there is interest) provide some cleaned scripts for the PDB parsing, some constant shifts required to register cryo-EM maps with these models, and some pointers for processing the maps.
Please find the attached list of PDB ids:
model_angelo_train_pdbs.txt
Best,
Kiarash.


Update Youtube Link!

https://github.com/curran/screencasts/issues/new
Update the link for the screencasts. Provided link doesnt seem to work.

Which link are you referring to?


Build with VS2019

Need to check this: https://www.appveyor.com/docs/windows-images-software/

superseded by #98


More details on --skip-assignment

More details on what --skip-assignment means would be helpful. What is a "default assignment" and why would one want to skip it?

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

ID: e99d25ff-03e2-e015-37f4-a29b59ec50d4
Version Independent ID: 1a8e88ed-72c7-fbe7-b0ad-68f23b3946cd
Content: az ad sp
Content Source: src/command_modules/azure-cli-role/azure/cli/command_modules/role/_help.py
Service: active-directory
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Note that this clarified —skip-assignment, but not what a ‘default assignment’ is earlier in the docs per OP’s recommendation.
I just ran into this as well and its extremely unclear that create-for-rbac grants permissions to the subscription by default. That’s a broad permission and deserves an explicit callout whenever ‘default assignment’ is mentioned.

@stewartadam, I am updating the command to output a message of creating such a role assignment.
Also the help of --skip-assignment has already mentioned the default assignment.


Trim after AOT

https://learn.microsoft.com/en-us/aspnet/core/blazor/webassembly-build-tools-and-aot?view=aspnetcore-8.0#trim-net-il-after-ahead-of-time-aot-compilation

Broke the app. Might be fixed in .NET 9: https://github.com/dotnet/runtime/issues/92238.


[CAMEL-8748] DozerBeanMapper cannot instantiate DozerThreadContextCla…

…ssLoader

This has been merged, do you mind closing this?


Use in-tree scripts in the jenkins builders

Currently, the shell script that is run for the CI jobs is defined in the jenkins/jobs/builders.yml. This makes it difficult to change what is run in CI, since it requires pushing the jenkins change to see the result. The only way to test in advance what a change to the shell script will look like is to manually create POC jobs.
Alternatively, we can create a top level ci-scripts directory with the following scripts:

get-node.sh - Responsible for getting a test node and setting the VIRTHOST env variable
collect-logs.sh - Responsible for collecting and archiving the logs
return-node.sh - Responsible for returning the test node

Then the builders would be simplified to:
bash get-node.sh
bash quickstart.sh <args for specific builder> $VIRTHOST
bash collect-logs.sh
bash return-node.sh

This would then allow us to easily iterate on simplifying the process to get and return a node to CI. It would be good to move the log collection from the khaleesi tree to its own ansible role, either in the oooq tree, or its own ansible-role-tripleo-log-collect repo.

resolved by https://github.com/redhat-openstack/tripleo-quickstart/commit/389d2e9af6eb3fa00fc5096246de33b71279e3a5


No module named "JumpscaleLib.clients.intercom"

  File "jstest.py", line 48, in <module>
    compare('j', jgen, dj, 2)
  File "jstest.py", line 46, in compare
    compare("%s.%s" % (tree, subname), subobj1, subobj2, depth-1)
  File "jstest.py", line 44, in compare
    subobj1 = getattr(obj1, subname)
  File "/usr/local/lib/python3.6/dist-packages/jumpscale.py", line 18, in _lazyprop
    setattr(self, attr_name, fn(self))
  File "/usr/local/lib/python3.6/dist-packages/jumpscale.py", line 1356, in intercom
    return Intercom()
  File "/usr/local/lib/python3.6/dist-packages/jumpscale.py", line 1058, in Intercom
    from JumpscaleLib.clients.intercom.Intercom \
ModuleNotFoundError: No module named 'JumpscaleLib.clients.intercom'
[Thu09 14:18] - ErrorHandler.py   :257 :j.core.errorhandler  - ERROR    - No module named 'JumpscaleLib.clients.intercom'

ok needed python3-intercom installed....


Go mod local replace no longer linked dynamically in 1.18, requires constant vendoring

What version of Go are you using (go version)?

$ go version
go version go1.18.0 darwin/arm64

Does this issue reproduce with the latest release?
Latest release only!
What operating system and processor architecture are you using (go env)?

Overwrite a module in go mod go mod edit -replace=github.com/dapr/components-contrib=../components-contrib
Make changes to local copy of module
Build project and notice that changes do not appear
Run go mod vendor and build project and notice changes do appear
Repeat from step 2... fully reproducible.

What did you expect to see?
As recently as Go 1.17.8 it was not necessary to run go mod vendor to sync my locally replaced modules, these were just dynamically linked. Building my project would always use the latest version.
What did you see instead?
The locally replaced module is still cached with whatever version was last vendored.

As far as I can tell it has always worked this way:
» exa -T
.
├── dep
│  ├── dep.go
│  └── go.mod
└── main
   ├── go.mod
   ├── main.go
   └── vendor
      ├── example
      │  └── dep
      │     └── dep.go
      └── modules.txt

0:20:48 ~/tmp/51932 0:00:00
» cat dep/dep.go
package dep

var V = 2

0:21:09 ~/tmp/51932 0:00:00
» cat main/vendor/example/dep/dep.go
package dep

var V = 1

0:21:22 ~/tmp/51932 0:00:00
» cd main

0:21:23 ~/tmp/51932/main 0:00:00
» cat go.mod
module example

go 1.17

require example/dep v0.0.1

replace example/dep => ../dep

0:21:25 ~/tmp/51932/main 0:00:01
» go1.16.14 run .
1

0:21:27 ~/tmp/51932/main 0:00:00
» go1.17.8 run .
1

0:21:29 ~/tmp/51932/main 0:00:00
» go1.18 run .
1

Setting the go directive in go.mod to go 1.13 will get the behavior you're describing, this is documented in go help build.
Closing as I believe this is working as documented.


[Event Hubs Client] Track Two: Fourth Preview (Misc Cleanup)

Summary
The intent of these changes is to perform some miscellaneous clean up within the code, converting batch creation to return ValueTask and applying formatting-related refactoring to increase internal consistency.
Last Upstream Rebase
Thursday, September 19, 2019 1:48pm (PDT)

CI failure is due to the artifact existing for this build.   Considering it a non-fatal error as the build itself was successful and tests are passing.

/azp run net - eventhub - ci


[Enhancement] Kafka Support for Manual Ack (updateOffset API)

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request
Kafka Support for Manual Ack (updateOffset API)
Describe the solution you'd like
Implement the updateOffset API
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

close via #1264


matTooltip Create Leak memory when mouse over it

mattooltip create leak memory

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate


Adding progress bar to CVAT uploads

What changes are proposed in this pull request?
Adds:

Progress bar in CVAT upload.

Resolve #1695
How is this patch tested? If it is not, please explain why.
A simple feature that affects only the terminal visualization.
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[  ] App: FiftyOne application changes
[  ] Build: Build and test infrastructure changes
[  ] Core: Core fiftyone Python library changes
[  ] Documentation: FiftyOne documentation changes
[ x ] Other

@ehofesmann great thank you!

@ehofesmann did some formatting changes, way simpler than before! Let me know if prefer it this way

@ehofesmann did some formatting changes, way simpler than before! Let me know if prefer it this way

Nice find on the pb.start() method! Unfortunately, the ProgressBar is designed as a context manager meaning it needs to be called using the syntax with fou.ProgressBar(...) as pb:, otherwise the pb.close() method may not be called correctly resulting in errors.
Basically, the way you had it originally was good.
Something that would be nice is to have the progress bar only print if there is more than one batch being uploaded. In many cases, only a single batch of images is being used and the progress bar will not actually update until the end. For single batches, it should probably just print out "Uploading samples to CVAT...." without printing the bar.

Something that would be nice is to have the progress bar only print if there is more than one batch being uploaded. In many cases, only a single batch of images is being used and the progress bar will not actually update until the end. For single batches, it should probably just print out "Uploading samples to CVAT...." without printing the bar.

^the cleanest way to implement this would be to add it to the ProgressBar class, which is implemented in a different package. Instead, perhaps a good compromise is to set iters_str="tasks" so that the bar at least conveys that the progress was trivial because it was tracking task creation and there was only one task (the default for image datasets, which is the most common use case).
I imagine you'd like a progress bar at download time too, right? That would be added here 🤗

@ehofesmann makes sense! I went back to the first style, and now the bar is enabled only when there are multiple batch sizes! I thought that the log Uploading samples to CVAT.... should be printed also when the pbad is printed, as the first batch can take some time, and the user would have an indication that the program didn't lag. However, we do have the option to print the log only when the pbar is quiet. Let me know!
@brimoor I would prefer the iters_str="samples", it would calculate the upload per image, which I believe its more insightful, but also the user sees how many images he will upload in total, regardless of the batch size, which also serves as a hint to the user that he is uploading the correct data
Now when there is a single batch, it will print:
Uploading samples to CVAT...

and when there are multiple:
Uploading samples to CVAT...
 100% |█████████████| 10/10 [5.7s elapsed, 0s remaining, 1.8 samples/s]

Thanks for the updates! I agree, I also prefer always printing Uploading samples to CVAT... but only providing a progress bar for samples when there are multiple batches.
There are a couple more formatting suggestions, but you can resolve those automatically by using our pre-commit hooks described here. You just need to install them from inside your cloned repository with:
pre-commit install

Note that there were recently some updates to the versions of these pre-commit hooks, so you may need to merge voxel51/develop into your branch first before installing them.

@ehofesmann  great, thank you! And thank you very much for the comments and for contributing to this great tool!
I hope I can contribute more as we move forward!


Become a host

Become a host subpage
The main menu on the left will remain
The page will have the following components:

Register your event: this will link to a form where people can register
How to host an event: this will have text info on useful tips
Workflow of the event
Resourced you can request

As this is done so far, I'm closing this in favor of #26 for the real content.


[BUG]: Error on launching after clean installation

Describe the bug
Error on installation
To Reproduce
pip install pytgpt
Expected behavior
Error on launching:
Traceback (most recent call last):
File "/home/jules/.local/bin/pytgpt", line 5, in
from pytgpt.console import main
File "/home/jules/.local/share/pipx/venvs/python-tgpt/lib/python3.12/site-packages/pytgpt/init.py", line 2, in
import g4f
File "/home/jules/.local/share/pipx/venvs/python-tgpt/lib/python3.12/site-packages/g4f/init.py", line 6, in
from .models import Model
File "/home/jules/.local/share/pipx/venvs/python-tgpt/lib/python3.12/site-packages/g4f/models.py", line 5, in
from .Provider import IterListProvider, ProviderType
File "/home/jules/.local/share/pipx/venvs/python-tgpt/lib/python3.12/site-packages/g4f/Provider/init.py", line 35, in
from .HuggingChat      import HuggingChat
File "/home/jules/.local/share/pipx/venvs/python-tgpt/lib/python3.12/site-packages/g4f/Provider/HuggingChat.py", line 5, in
from curl_cffi      import requests as cf_reqs
ModuleNotFoundError: No module named 'curl_cffi'
Desktop (please complete the following information):

OS [e.g. Windows, Mac OS, Linux]: Ubuntu
Python-tgpt version [v0.5.3]: 0.7
Binary config [x86, x64, x32]: ?
Python version [If running from source]: running from source

Try reinstalling alongside all dependencies pip install python-tgpt[all]

Yep, tried bu the issue is the same