Uplink NPM Feature Similar to Verdaccio

Feature Description
I have been an avid user of Gitea for managing my NPM packages, and I truly appreciate the simplicity and effectiveness of your platform. One feature that I find particularly useful in the realm of NPM package management is the Uplink feature that is offered by Verdaccio. The Uplink feature allows users to proxy external NPM registries, enabling efficient caching and improved access to external dependencies. This not only enhances package retrieval speed but also ensures more reliable package availability.
Given the benefits of the Uplink feature in Verdaccio, I wanted to suggest that Gitea consider implementing a similar Uplink feature within your platform.
Reference: https://verdaccio.org/docs/uplinks/
Screenshots
No response

Duplicate of #21223


Dead branches not being detected for inferred types

If I declare an object by specifying its type, Flow correctly throws errors for dead branches:
type Action = {
  type: "ACTION"
}
declare var action: Action

// Both of the following statements correctly raise flow errors, as flow knows that
// action.type is the string literal "ACTION"

const a: "hi" = action.type

if (action.type === "hi") {
}

However, if Flow infers the type from a variable declaration, it fails to identify the dead branch.
const action = {
  type: "ACTION"
}

// The first statement correctly raises a Flow error, but the second one doesn't.
const a: "hi" = action.type

if (action.type === "hi") {
}

I'm using Flow 0.59. Let me know if anything doesn't make sense. Thanks very much!

It's because in the second example action.type is string. Flow does not force you to use only initialized value, you may mutate that object to make action.type === 'hi' truly. So it's correct. However const a: "hi" = action.type is an error since you are trying to cast two incompatible values. But if you mutate action.type to hi flow won't throw. So, everything is correct here.

Ah interesting, thanks. But presumably it'd still be possible for Flow to infer the fact at that particular point in the code, action.type has not been mutated to equal "hi"?

Well, yeah. Flow could infer this.


REPLAY-1890: Implement pool of reusable bitmaps

What does this PR do?
Adds a reusable bitmap pool for base64.
Motivation
Reduce the number of new bitmaps created.
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1554 (f703ef9) into feature/base64-jmoskovich (dcd6805) will decrease coverage by 0.11%.
The diff coverage is 62.12%.

@@                      Coverage Diff                      @@
##           feature/base64-jmoskovich    #1554      +/-   ##
=============================================================
- Coverage                      83.49%   83.38%   -0.11%
=============================================================
  Files                            439      442       +3
  Lines                          14977    15015      +38
  Branches                        2258     2261       +3
=============================================================
+ Hits                           12504    12519      +15
- Misses                          1886     1909      +23
  Partials                         587      587

Files Changed
Coverage Δ

...nreplay/internal/recorder/base64/Base64LRUCache.kt
84.85% <0.00%> (+13.42%)
:arrow_up:

...id/sessionreplay/internal/recorder/base64/Cache.kt
0.00% <0.00%> (ø)

...android/sessionreplay/internal/utils/CacheUtils.kt
0.00% <0.00%> (ø)

...ssionreplay/internal/recorder/base64/BitmapPool.kt
58.82% <58.82%> (ø)

...roid/sessionreplay/internal/utils/DrawableUtils.kt
97.44% <95.00%> (-2.56%)
:arrow_down:

...nreplay/internal/async/RecordedDataQueueHandler.kt
90.00% <100.00%> (+0.61%)
:arrow_up:

...eplay/internal/recorder/base64/Base64Serializer.kt
88.06% <100.00%> (+0.18%)
:arrow_up:

... and 14 files with indirect coverage changes


Fix startMasters() from receiving unchecked exceptions

FaultTolerantAlluxioMasterProcess calls startMasters when gaining primacy, which can throw a NotLeaderException if the master has already lost the leadership, crashing the master. This PR handles this error

Codecov Report

Merging #14353 (96f00ce) into master (0edc8b1) will decrease coverage by 23.93%.
The diff coverage is 2.05%.

@@              Coverage Diff              @@
##             master   #14353       +/-   ##
=============================================
- Coverage     40.72%   16.78%   -23.94%
+ Complexity     9032     2673     -6359
=============================================
  Files          1484     1487        +3
  Lines         86554    86726      +172
  Branches      10351    10368       +17
=============================================
- Hits          35246    14557    -20689
- Misses        48463    70965    +22502
+ Partials       2845     1204     -1641

Impacted Files
Coverage Δ

...uxio/master/FaultTolerantAlluxioMasterProcess.java
0.00% <0.00%> (-59.38%)
:arrow_down:

...a/alluxio/proxy/s3/ChunkedEncodingInputStream.java
0.00% <0.00%> (ø)

...in/java/alluxio/proxy/s3/DeleteObjectsRequest.java
0.00% <0.00%> (ø)

...ain/java/alluxio/proxy/s3/DeleteObjectsResult.java
0.00% <0.00%> (ø)

...in/java/alluxio/proxy/s3/S3RestServiceHandler.java
0.00% <0.00%> (-3.20%)
:arrow_down:

...ava/alluxio/multi/process/MultiProcessCluster.java
0.00% <0.00%> (ø)

...alluxio/proxy/s3/logging/RequestLoggingFilter.java
20.00% <20.00%> (ø)

...on/src/main/java/alluxio/master/MasterContext.java
64.28% <66.66%> (ø)

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 658 more

Continue to review full report at Codecov.

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

Failed test:
Java 11 & Java 8:
S3ClientRestApiTest.listAllMyBuckets

Java 11:
FileSystemMasterRestartIntegrationTest.updateAccessTimeAsyncFlushAfterDelete
BlockMasterRegisterStreamIntegrationTest.reregisterWithDelete
BlockMasterIntegrityIntegrationTest.deleteOrphanedBlocks

Java 8:
ConfigCheckerIntegrationTest.multiNodes
RocksInodeStoreTest

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.7 please


Feature request in Zen Mode!

VSCode Version: 1.22.2
OS Version: Windows 10 Pro

Feature Request:

Most of the time I work in Zen Mode. But I am missing the opened files tabs in top screen. I can use Show opened Editors or other functions/commands to show the opened files but it is a little awkward for me. If you could make the Opened Editors visible on Zen Mode like this (http://prntscr.com/j7p2mu) I will be glad to have such a amazing editor in my PC.

Please use the following settings
"zenMode.hideTabs": false


0.8 Client.Stop does not send disconnect

In 0.7 NetManager.Stop() has sent disconnect message, but in 0.8 it stops without any message sent. Is it intended or bug? I see the code netPeer.Shutdown(null, 0, 0, true); and that the last param force = true causes not to send the disconnect message.
Library version: commit id cd84459 0.8indev
Framework:  .NET Core
OS: Linux, Windows

@xmedeko this i forgot to return to "false" after some tests.

Furthemore, I've tested 100 clients to connect to the server, then called 'Stop()'. The server has received OnPeerDisconnected Timeout just for some of them. It does not detect the Timeout for most of the clients and keeps them in the pool.

@xmedeko can you show this sample code?

More info: I've tried a bit debugging. After stopping all clients and waiting for 15 sec (more than DisconnectTimeout), the server reports: PeersCount: 30, but GetPeers((ConnectionState)0xff) returns always just one connection in the Disconnected state.
Waht about to add cleanup of disconnected peers in the NetManager just as an error fallback?

@xmedeko thanks for debugging. I will fix this soon.

@xmedeko fixed.

@RevenantX I've tested it and it's OK. Thanks.


feat: add sensision source

 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.


Google pay golang

Referencing #75
This is an initial implementation of the Google Pay library in Golang. Currently only decryption of the token from google pay is supported.

Note: the ECIES files are from the ethereum golang implementation repository

Thanks for working on this. I think this is very valuable. The PR is huge, so I wonder if you can split it up in smaller PRs, making the review easier.
For example, you can have individual PRs that implement:

Hkdf: this should stay in subtle/kdf/hkdf.go (package is kdf)
Common elliptic curve functions like key generation, decoding, encoding, computing shared secret, etc: this should stay in subtle/elliptic_curves.go (package is subtle)
EciesHkdfRecipientKem and EciesHkdfSenderKem: similar to Java, should stay in subtle/hybrid/
PaymentMethodTokenHybridEncrypt, PaymentMethodTokenHybridDecrypt, etc.: similar to Java, should stay in apps/paymentmethodtoken/go/.

What do you think?
PS: can you also revert the changes in the *.java files? Thanks!

thank you will do

Why is this PR closed? Is there a google pay decryption app which is in java available for golang?

Why is this PR closed? Is there a google pay decryption app which is in java available for golang?

There is a second version of the Google Pay API which came out after this PR and the code versioning for this never got any clear info on merging. There was also some initiative on refactoring the golang library which would have changed all the APIs and which made too many merge conflicts to make it worth retrying.
I made a separate clone here and put the PR work here: https://github.com/Saqibm128/tink
This should target v1 of the google pay api and passes one of the example unit tests from the java library, but I have no clue if it would ever work with the production api; @vladgoncharenko contacted me a while back and said he used the library, you might want to reach out to him.

I tried using this PR but it is not working Verify() method is failing for all google public keys. I will checkout the second PR and see if it works.


SSR support (render)

We should support render, for SSR: https://vue-test-utils.vuejs.org/api/#render

Migrating from vue 2 to vue 3, and this seems to be a missing feature in the test utils. Is this something that is actively planned to be reintroduced?

@samturrell Yes, there is an open PR from a contributor that should land as a first step to support SSR. See https://github.com/vuejs/test-utils/pull/1572

@samturrell if you want to pick up where the original author in #1572 left off, that'd be great! I think we just need to rebase and get his branch green, and we can release something to get feedback and testing.

This is a good addition, but I assume it's an emulated function rather than an adapted version of the proper renderToString function since it fails in a NodeJS envrionment. For example:
// @vitest-environment node
at the top of your test file gets you a ReferenceError: document is not defined from vue-test-utils.esm-bundler.mjs.

Hm, should document be defined in that context? Test Utils assumes document exists.
Any recommendation for how renderToString should be implemented here?


Way to generate csv output using java

Is there is any way to generate CSV output as input stream at java layer.

It's not provided.


#626: Implement parsing DDI 3.2 elements

This PR adds the necessary logic to parse DDI 3.2 instance documents.
This PR adds the XMLMapper class. This class holds the logic needed to parse metadata from DDI elements. XPaths has been expanded to hold instances of XMLMapper for each CMM metadata field.
DDI 3.2 Lifecycle instance documents are now fully supported by the OSMH indexer. An exemplar document has been added to the test suite to verify parser functionality.
This closes cessda/cessda.cdc.versions#626.

@john-shepherdson Could you take a look?


Add CPack configuration for Debian packaging.

LPCNet equivalent to drowe67/codec2#169 and drowe67/freedv-gui#87.

Doesn't bother me any as I'm a Fedora guy but I have noticed that on the RPM distro sided that the artifacts produced are not of high quality if if they "work".  I don't know if this is a deficiency in CMake or whoever setup CPack. Obviously I'm looking at this in a pretty pedantic way as a packager.

Doesn't bother me any as I'm a Fedora guy but I have noticed that on the RPM distro sided that the artifacts produced are not of high quality if if they "work".  I don't know if this is a deficiency in CMake or whoever setup CPack. Obviously I'm looking at this in a pretty pedantic way as a packager.

@hobbes1069, @drowe67 and I are going to check with AA4HS and see what he thinks about the Debian side since he's worked on those packages before. There might be a way to ensure that the generated packages are still acceptable.

@hobbes1069, @drowe67 and I are going to check with AA4HS and see what he thinks about the Debian side since he's worked on those packages before. There might be a way to ensure that the generated packages are still acceptable.

@hobbes1069, @drowe67, seems that Debian uses a different mechanism for producing packages but these PRs shouldn't interfere I think.
BTW, I figured out how to generate separate -dev, etc. packages and use Debian standard naming and updated the PRs accordingly.

@hobbes1069, @drowe67, seems that Debian uses a different mechanism for producing packages but these PRs shouldn't interfere I think.
BTW, I figured out how to generate separate -dev, etc. packages and use Debian standard naming and updated the PRs accordingly.

Thanks @tmiw just let me know when you are ready to merge the CPack series of PRs...

Thanks @tmiw just let me know when you are ready to merge the CPack series of PRs...

@drowe67, I think we're good if the README.md changes I just pushed look fine. 👍

@drowe67, I think we're good if the README.md changes I just pushed look fine. 👍


Single @php Blade directive does not work correctly when followed by a @endphp tag

Laravel Version
10.17.1
PHP Version
8.2.7
Database Driver & Version
No response
Description
The documentation states that we can use a single @php directive like this:
@php($counter = 1)

It works fine until there is another block of raw PHP that uses the full syntax after.
@php($index = 'test')

@php echo $index; @endphp

In the ouput file the PHP end tag is missing:
<?php($index = 'test')

@php echo $index; ?>
<?php /**PATH /Users/marcbelletre/Sites/laravel/resources/views/welcome.blade.php ENDPATH**/ ?

I also tried with older versions of Laravel (8 & 9) and got the same issue.
Steps To Reproduce

Create a blade template with a single line @php directive and a block of raw PHP using the @php ... @endphp syntax
Add a route that renders the blade template
Open the URL in the browser

For such usage it would be best to just use:
@php
$counter = 1;

echo $counter;
@endphp

or even
@php($counter = 1)

{{ $counter }}

A similar PR has been rejected before: https://github.com/laravel/framework/pull/45390#issuecomment-1369849576

Ok but I think there should be a warning about this in the docs if this is a known issue. I struggled for a while because I couldn't see what was wrong in my code before noticing it was coming from a bug in Laravel itself.
I encountered this with roots/sage which is a package that makes a bridge between Wordpress and Laravel.
The default templates are using the following syntax:
@extends('layouts.app')

@section('content')
  @while(have_posts()) @php(the_post())
    ...
  @endwhile
@endsection

Because of this bug it is impossible to use any raw PHP tags inside the loop which is quite a problem. The only workaround is to replace the default @php(the_post()) call to @php the_post() @endphp


FIX: Part of Popover is outside boundaries of viewport

Fixes #19279.

In order for this to adjust the popover arrow wouldn't you have to change the popover arrow back to utilizing the .popover-arrow class instead of the pseudo elements it was changed to in this commit?

Sorry but we removed Tether for Popper see : #22444


Create analyte_type_enumeration_examples_v2.yaml

Adding new enum representation proposal document with simpler view of latest preferred proposals

@mbrush Should I go ahead and merge these changes?


Allow custom tags when using the cli

I am using mustache via the cli, and would like to specify custom tags as a command line parameter.
maybe something like:
mustache -c "<%,%>" dataView.json myTemplate.mustache > output.html

where my custom tags are a two item comma separated list.

Surely makes sense to have support for custom tags, or "delimiters" as they've been called in the README 👍
More than happy to review a PR with the changes needed to make that happen.


getMeasurementXY is too loose

https://github.com/cheminfo/analysis/blob/274c2a7326a3046be976d6bd5e3613ad09d00e3e/packages/base-analysis/src/util/getMeasurementXY.ts#L26
There are a lot of cases where the function can return undefined and then this pollutes the Analysis class where most methods can also return undefined and the user always has to check if a value was returned.

I will prefer to thould throw on most of this cases, I think that is easier to handle as well

I will probably work on it because I need to evaluate if this should not return an array that could be of length 0 if there is no match by the selector.


coprocessor: select response of dag don't need row meta.

compatible with https://github.com/pingcap/tidb/pull/4348

 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.

Please run make dev before any commit

CI is fixed, PTAL @AndreMouche

/rebuild

/rebuild

/rebuild

/rebuild

rest LGTM.

/cc @ilovesoup

LGTM.

/run-all-tests

/run-integration-test tidb=pr/yiding/handle

/run-integration-test tidb=pr/yiding/handle

/run-integration-test tidb=pr/4348

/run-integration-test tidb=pr/4348

/run-integration-test tidb=pr/yiding/handle

/run-integration-ddl-test tidb=pr/yiding/handle

/run-integration-ddl-test tidb=pr/yiding/handle

/run-all-test tidb=pr/yiding/handle

/run-unit-test

/run-all-test tidb=pr/yiding/handle


Swagger API Docs

Accessed via http://localhost:4200/api-docs/

Thanks, @briangormanly.
@Caleb-Rogers I just want to confirm - the new commits do not have submitted changes for lines 387, 438, 502, and 579. I saw these suggestions in particular were collapsed in this thread, so it's possible you didn't see them. Take a look and let me know!


Bring the new LU toolchain

The new LU tool-chain is the blocking issue of  Managed-LU and Linting & Validation.  This includes
BF-CLI side

[ ] publish bf-lu (whatever the name is called today) package with new parser (means new features and bug fixes), CURD support and section support
[ ] (not hard requirement) merged the features baked for composer into master

@feich-ms as the POC
Composer side

[ ] Update LuBuild to use the new bf-lu
[ ] integrate the new bf-lu and lubuild, in luIndexer, luPublisher, luLanguageServer

addressed in #1608


new playbook for production deploy to PUL VM

see https://github.com/Princeton-CDH/CDH_ansible/pull/56.

see https://github.com/Princeton-CDH/CDH_ansible/pull/56.


Rendr + jspm

Hi,
can someone provide us an example or a simple approach to compile a Rendr project with jspm.
Thank you.

Hmm, I haven't actually used jspm before, project looks pretty interesting.  It seems like it's a replacement for using browserify or webpack?  you probably need a polyfill for CommonJS / AMD requires. https://www.npmjs.com/package/jspm-loader might do the trick

Thank you @saponifi3d .


Breaks after certain number of blocks

If I try to block a large number of people, it starts displaying errors roughly after 3-400, and then when the block is finished, I need to erase all login data to fix it. I use the brave browser.

I've the same problem. I think is related with two factor auth.

I've the same problem. I think is related with two factor auth.

It has nothing to do with 2FA, I don't have it enabled on my account and I still have this problem. It's probably because of API rate limiting.

Hi.
Twitter has throttling mecanisms for api requests and will invalidate your session cookies if you try to go beyond 500 effective blocks in just a matter of seconds. after that, lots errors could arise until you delete the cookies and log in again.

Is it possible to add the 'sleep' feature in the plugin options section? therefore, we can specify the rate of the block.

API rate limiting.
There needs to be a rate limiting feature for larger amounts of tasks to keep it under the API limit.
For now, just clear your cookies whenever this happens, and avoid blocking over a few hundred people at a time.


[SPARK-50648][CORE] when the job is cancelled during shuffle retry in parent stage, might leave behind zombie running tasks

What changes were proposed in this pull request?

This is a problem that Spark always had. See the following section for the scenario when the problem occurs.
When cancel a job, some tasks may be still running.
The reason is that when DAGScheduler#handleTaskCompletion encounters FetchFailed, markStageAsFinished will be called to remove the stage in DAGScheduler#runningStages (see https://github.com/apache/spark/blob/7cd5c4a1d1eb56fa92c10696bdbd8450d357b128/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala#L2059) and don't killAllTaskAttempts.
But DAGScheduler#cancelRunningIndependentStages only find runningStages, this will leave zombie shuffle tasks, occupying cluster resources.
Why are the changes needed?

Assume a job is stage1-> stage2, when FetchFailed occurs during the stage 2, the stage1 and stage2  will resubmit (stage2 may still have some tasks running even if stage2 is resubmitted , this is as expected, because these tasks may eventually succeed and avoid retry)
But during the execution of the stage1-retry , if the SQL is canceled, the tasks in stage1 and stage1-retry can all be killed, but the tasks previously running in stage2 are still running and can't be killed. These tasks can greatly affect cluster stability and occupy resources.
Does this PR introduce any user-facing change?
No
Was this patch authored or co-authored using generative AI tooling?
No

cc @cloud-fan

This is a good catch! cc @jiangxb1987 @Ngone51

Thanks, merged to master!

Thank you all for the review


fix issue 16372 - Broken links in documentation

Also fixing another mistakenly highlighted "format".

Thanks for fixing, we should really come up with a solution that prevents broken links a priori :/

Now that we have travis should be simple enough: grab the changed files list from DAutoTest, run a web scraper to get all a tags, ping all of the urls

Now that we have travis should be simple enough: grab the changed files list from DAutoTest, run a web scraper to get all a tags, ping all of the urls

I am not so sure whether it's that simple, but I agree quite doable, some potential problems that come to my mind:

we are about to switch to CircleCi, but that's just a minor detail
you can use the Github API to get the available CI results, but unfortunately they are quite heavily rate limited (see eg https://github.com/dlang/dmd/pull/6053)
IIRC there are some cases when the resulting diff gets quite large

Do you want to play with it?


Stop creating boundaries for paths not providing pending or error components

Describe the bug
Currently it appears the router creates a Suspense boundary for each parent route with  as the fallback if no pendingComponent is provided. It seems more intuitive to just not render the suspense boundary for any route without a pendingComponent , which would then automatically use the nearest suspense boundary and its configured fallback.
The same idea should be done for errorComponent. Currently the router is creating a CatchBoundary even if no errorComponent is provided.
This should remove the need for configuring a defaultPendingComponent and/or defaultErrorComponent instead they would just be defined on the root route.
Your Example Website or App
https://discord.com/channels/719702312431386674/1055081579761893496/1055081579761893496
Steps to Reproduce the Bug or Issue

Create a route structure such as __root__ > section > index.
Provide a pendingComponent on the root route.
Add a loader to the index route.

Expected behavior
When the loader is running I expect the nearest path with a pendingComponent to be displayed.
Currently, Anonymous (blank) component is rendered because it is defined on the section route, due to there being no explicit pendingComponent provided.
Screenshots or Videos
No response
Platform
macOS, Chrome
Additional context
No response

This is now fixed. Routes that do not provide an errorComponent will not get wrapped with an errorBoundary. By default, all routes will get wrapped with suspense though. If you don't want a route to get wrapped with suspense, you can pass the react-router-only option wrapInSuspense: false to the route options.


Migrate APIView into a set of VS Code extensions that could be enabled in VSCode.dev

Evaluate the feasibility / effort to migrate APIView into a set of VS Code extensions that could be enabled in VSCode.dev for a comeback to executive leadership.

What would a user gain from this?

This would make it easy for developers (any developer of any library) to see in real time what the APIView would look like (i.e. what changes would occur in the public API surface) for changes they are making to their library. It is a "shift left" feature.

We could host the APIView as a web view within VSCode ... this might be the lowest cost approach.  But we would need to spend some resources to size this effort.
We may want to make the same feature available as a Visual Studio extension (a different thing from a VSCode extension).
Every language as it's own way of showing the public surface area, but we don't use that. But we wrote our own parsers for some languages like Python because they miss some important features.  And we wrote our own linter rules. ...
The main value prop for APIView is in reviewing and providing comments / feedback / approvals.


create_test_mode_payment is not a signed operation.

#41  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @zhenlu and the rest of your teammates on  Graphite

Merge activity

Mar 28, 12:15 AM EDT: @zhenlu merged this pull request with Graphite.


[Bug] Expander and CollectionView/ListView

There are a lot of reports about the Expander used together with the ListView or CollectionView not working properly.
We've added to the Docs: Expander and CollectionView/ListView should not be used together. I'd love to have a solution for this but we don't have one right now. I'm closing all the separate issues and have a combined list here so we can track it in case someone wants to solve this.

#572
#571
#568
#569
#608
#926
#933

I also faced this but don't know much about it.
I put this in touch argument and it's working
--linkskip=Xamarin.CommunityToolkit

@kdbhalala then it should work even w/out this) Because expander is 100% cross-platform

Does the expander work as expected within a StackLayout using the BindableLayout.ItemsSource property?


Context manager for AvailableData

closes #68
This is an alternative to #70.
This is in draft until I feel more confident about the tests and the napari plugin. But I think it's close.

#132 strikes again.

I wonder if there's a way to avoid the if. null context manager? @andy-sweet

Generally looks fine. We could consider adding support for an empty AvailableData, which might help clean up some of the Python examples and avoid the need for some of the checks in the with statements, but I think that can be done later too.

Yeah, I agree that empty AvailableData should be supported, but it should be a separate PR. I can make an issue of it after this is merged and hold off on cutting a 0.3.0 rc until after that makes it in.

I think I found a few missed lines that could be deleted now. We could also consider removing the TODO at the end of __init__.py, though this only adds context manager for available data and not for runtime/start/stop.

I've removed those del / = None statements, but I'm going to leave the TODO in for now.


[dart2js] RTI need doesn't take closure equality into account

The RTI need computation in dart2js doesn't take equality of closure into account. This means that when two instantiations of the same function with the same type arguments are compared at runtime, they might not be considered equal.
For instance, the program will not see that create needs its type arguments and the test there fails:
class Class<T> {}

Class<T> create<T>() => new Class<T>();

equals(a, b) {
  if (a != b) throw '$a != $b';
}

test<T>(f) {
  Class<T> Function() g = create;
  equals(f, g);
}

main() {
  Class<int> Function() f = create;
  test<int>(f);
}

This is the underlying issue why language/constructor/tear_off_test fails.

/cc @rakudrama @fishythefish , note that Johnni already prepared a fix https://dart-review.googlesource.com/c/sdk/+/211908

sorry, my bad, the CL addresses a different issue and adds a test for this issue, but it doesn't address yet the closure equality problem.

Was fixed by d49d5a96f74ec9aa2b775b82acc3b83a546d7d73


Any plan for npm?

http://phonegap.com/blog/2015/05/26/npm-plugins-available/
Will you put your plugin on NPM?THANKS.

+1

+1

+1

+1

cordova-plugin-socialsharing is taken (for a while now), that's what's holding me back. So perhaps I'll settle for cordova-plugin-x-socialsharing instead. Suggestions are welcome.

cordova-plugin-socialshare ?
+1

:+1:
Would be great to see it with any name on npm along with cordova-plugin-actionsheet (you should hurry up while it is not yet taken)

@roberthovhannisyan good call on actionsheet - published just now.
And SocialSharing is following later today, I settled in cordova-plugin-x-socialsharing, just like I had to do with toast. I don't want to use cordova-plugin-socialshare because that may conflict with a colleague's plugin.

OK, done: cordova-plugin-x-socialsharing is up, use it on phonegap build with source="npm".

thanks @EddyVerbruggen !!!

thanks  @EddyVerbruggen


Add appinsights to helium-java

Type of PR

[ ] Documentation changes
[ ] Code changes
[ ] Test changes
[x] CI-CD changes
[ ] GitHub Template changes

Purpose of PR
Pulling in the app insights jar file into the docker image so we have telemetry as the new app insights jar file does a "codeless attach"
Review notes
Issues Closed or Referenced

Closes # (this will automatically close the issue when the PR closes)
References #164  (this references the issue but does not close with PR)

Moving it later in the file was causing issues with wget as in the middle of the file we switched the user over to helium and that user is very restricted and could not copy the file to even /app, the home doctor


Please add ARM binaries

I'm trying to run etcd on raspberry pi.

@mingfang Can you build arm binary yourself for the moment?

@xiang90 I would like to try.  Are there any instructions available?

Setup your GO env and GOPATH. Then you should be able to build etcd by using build script.

I am closing this. etcd does not officially support arm right now. We might support arm64 and release binary soon.


feat: Enable docker layer caching

Please read CONTRIBUTING.md for additional information on contributing to this repository!

What this PR does / why we need it

Jira ID
[XX-XX]

Notes for your reviewers

Also, please fill out the PR description with the motivations behind this change (optionally a link to the slack context, but since this is internal it might be best to not).


What else need for onClick's events?

Hello David,
I have some problem's with onClick events....

Created activity and implements FlexibleAdapter.OnItemClickListener, FlexibleAdapter.OnItemLongClickListener.

Created adapter like this:

public class Adapter extends FlexibleAdapter<AbstractFlexibleItem> {
    public Adapter(@Nullable List<AbstractFlexibleItem> items, @Nullable Object listeners) {
        super(items, listeners, true);
    }
}

Created simple Item object extending AbstractFlexibleItem<Item.ViewHolder> ...
Created instance of this adapter with empty items list like: new Adapter(emptylist, this);
Added some items

Try to fire onClick events, tapping items on the screen... Nothing :(
What's wrong?
Thank for help.

Hi @rusxakep, did you extend the ViewHolder from FlexibleViewHolder? What do you expect it happens in your implementation of the callback?
Also, remember that you have logs, to see if the event is fired in the FlexibleViewHolder you need VERBOSE level.

Sure, extend.
static class ViewHolder extends FlexibleViewHolder {
...
}
I expect Log.v("onClick"); :-)
Yes, log  with verbose:
I/FlexibleAdapter: Running version 5.0.0-rc2 built on 2017.07.11
I/FlexibleAdapter: Initialized with StableIds=true
I/FlexibleAdapter: Setting listener class Activity as:
I/FlexibleAdapter: - OnItemClickListener
I/FlexibleAdapter: - OnItemLongClickListener
V/FlexibleAdapter: Attached Adapter to RecyclerView
V/FlexibleAdapter: addItem delegates addition to addItems!
D/FlexibleAdapter: addItems on position=0 itemCount=1
V/FlexibleAdapter: addItem delegates addition to addItems!
D/FlexibleAdapter: addItems on position=1 itemCount=1
V/FlexibleAdapter: addItem delegates addition to addItems!
D/FlexibleAdapter: addItems on position=2 itemCount=1
I/FlexibleAdapter: Mapped viewType<PHONE_NUMBER> from TokenItem
V/FlexibleAdapter: onViewBound    Holder=ViewHolder position=0 itemId=4999
V/FlexibleAdapter: onViewBound    Holder=ViewHolder position=1 itemId=5220
V/FlexibleAdapter: onViewBound    Holder=ViewHolder position=2 itemId=6524

Ok @rusxakep, then the only reason I can see from there is the check for enabled state. If it isn't, you should debug and see why the callback is not called.

Items in default state, I do not use setEnabled() method...
Ok. If I made that's all correct, I'll be debug this. Thank you.

Hmm... Add onClick and onLongClick at ViewHolder item class and working only LongClick.
OnClick still don't fired.
@davideas, where is problem?

@rusxakep, can you show me the ViewHolder and the Item classes?

I found my mistake. In onBindViewHolder:
    holder.cardview.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
        }
    });

Thank for fast answers. Close issue.


fix #1716: enable code coverage check for master branch

How do you know whether alias is effective or ignored?


Exec tachikoma update<PHONE_NUMBER>5055

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 97.645% when pulling 194cdb550f7d4e6e5ca754970c83107112b1f7fa on pazjacket:tachikoma/update-20171003175055 into 1d885b4f49b2db2fa783bba38465cdcdc8ba0154 on sue445:master.


Unsupported state or unable to authenticate data

Hello,
I don't understand why i'm getting this error on Windows 10...
Everything is fine when i encrypt file but when I want to decrypt it, I get this error...
Thank's.

Hi, can you provide the WarShield's version you're using? (warshield -V)
And also give the full logs when using verbose mode? (warshield -v or select "verbose" on the UI)


[chrono 2.x] Add WithContext function

WithContext function will be added to be able to pass context to scheduled task.
Executor and Runner don’t have context parameters. When we add WithContext function, we should be aware this change will be breaking backward compability.

still relevant or is the project abandoned?


Disallow array parameters in JSONRPC methods

What
Disallow array parameters in JSONRPC methods
Why
Close #13 Close #40
Do not merge until we want to cut a protocol 21 release

@Shaptic it seems like javascript is still using array parameters. Could that be the case?

@Shaptic it seems like javascript is still using array parameters. Could that be the case?

It looks like it was updated, so you just need to update the system tests to reference the most recent SDK release

Uhm, it seems the CLI doesn't like it either ...

@willemneal Could it be that the CLI uses array arguments in JSONRPC calls?

Here are some of the jsonrpc requests from a trace from the CLI deploying a contract, which all use the array for "params"
[
  {
    "jsonrpc": "2.0",
    "id": 0,
    "method": "getLedgerEntries",
    "params": [
      [
        "AAAAAAAAAADh9C8tjVls3DqO4pComSAu7Uowvdz14Q1ANPs6kFTv6A=="
      ]
    ]
  },
  {
    "jsonrpc": "2.0",
    "id": 0,
    "method": "getTransaction",
    "params": [
      "303066cd68a6e8a669ca3fc22ca60f07011f9c99c4b73c046ca6e72818d207b5"
    ]
  },
  {
    "jsonrpc": "2.0",
    "id": 0,
    "method": "simulateTransaction",
    "params": {
      "transaction": "AAAAAgAAAADh9C8tjVls3DqO4pComSAu7Uowvdz14Q1ANPs6kFTv6AAAAGQAAADOAAAABwAAAAAAAAAAAAAAAQAAAAAAAAAYAAAAAQAAAAAAAAAAAAAAAOH0Ly2NWWzcOo7ikKiZIC7tSjC93PXhDUA0+zqQVO/oTPSTOwMvaooNPLS9FedDzC5S3DQ6BNZLYmDtS17OKIoAAAAAgPljDahC0uEgl8NOZZOM4yWQttNLhzyz5tyKYt0bA5YAAAAAAAAAAAAAAAA="
    }
  },
  {
    "jsonrpc": "2.0",
    "id": 0,
    "method": "sendTransaction",
    "params": [
      "AAAAAgAAAADh9C8tjVls3DqO4pComSAu7Uowvdz14Q1ANPs6kFTv6AAD2uEAAADOAAAABwAAAAAAAAAAAAAAAQAAAAAAAAAYAAAAAQAAAAAAAAAAAAAAAOH0Ly2NWWzcOo7ikKiZIC7tSjC93PXhDUA0+zqQVO/oTPSTOwMvaooNPLS9FedDzC5S3DQ6BNZLYmDtS17OKIoAAAAAgPljDahC0uEgl8NOZZOM4yWQttNLhzyz5tyKYt0bA5YAAAABAAAAAAAAAAEAAAAAAAAAAAAAAADh9C8tjVls3DqO4pComSAu7Uowvdz14Q1ANPs6kFTv6Ez0kzsDL2qKDTy0vRXnQ8wuUtw0OgTWS2Jg7UteziiKAAAAAID5Yw2oQtLhIJfDTmWTjOMlkLbTS4c8s+bcimLdGwOWAAAAAAAAAAEAAAAAAAAAAQAAAAeA+WMNqELS4SCXw05lk4zjJZC200uHPLPm3Ipi3RsDlgAAAAEAAAAGAAAAAXmbAq7ocZCaTIPZo0Kjumf6ulOoMG8FEPT+i5O+lw+/AAAAFAAAAAEALqtnAAAXDAAAAGgAAAAAAANZxAAAAAGQVO/oAAAAQBFkNvKWhKrv0JXUgfELdpRjXtebp4b6NU8yDljeRvNGhtIwPz0lLSXlLctKmSQMYQ7nN3smlMnpk3WsxS3CGAc="
    ]
  }
]

Looking at the rust crate it uses rpc_params![] which returns an array defined here: https://github.com/paritytech/jsonrpsee/blob/87a37a719a378b0700f519beeee3c21f95bc6495/core/src/client/mod.rs#L216

Ahh, it's defined in the rpc-client crate. I will try to fix it.

I think we first need to update the CLI code to use the object parameters. I will create a separate PR for that

I think we first need to update the CLI code to use the object parameters. I will create a separate PR for that

Replying to https://github.com/stellar/soroban-rpc/pull/125#issue-2227442542:

Also, it's a good idea to release an updated CLI version first, to allow users to upgrade before disallowing array parameters for good.

I agree it'd be ideal to release the CLI and for there to be a period of time where the breakage is not experienced so as to reduce it's impact and give folks time to upgrade.
When we planned #121 for this upcoming release we thought that we had already updated every client that was using arrays weeks/months ago, meaning there was plenty of time to pull the plug on arrays. But it'll be pretty rough to pull the plug now if we're only providing a week or two, and we break all existing users.
Maybe that's not a big deal if there's other necessary breaking changes already in play.

we thought that we had already updated every client that was using arrays weeks/months ago

I though so too 🤷

@willemneal will you let me know once the CLI is updated to use the rpc client create version 20.3.5? That should unblock this PR.
Thanks!

@2opremio Here is the PR to update the deps: https://github.com/stellar/soroban-cli/pull/1281


Fix for development shell

The pure development shell broke when the rust toolchain was updated.

A workaround was implemented, the nix develop mina works after the nix/pin.sh script is executed. This something that has to be fixed from nixpkgs.
I will raise an issue in the near future and link it here with technical details.

The problem arose from these changes where the SYSROOT is determined based on the librustc_driver-*.so file.
There was a "FIX " for this in the nixpkgs-mozilla repository, however this isn't enough for our case.
We would have to copy the librustc_driver.so around because we are sharing components between mutliple sysroots. Here is a more detailed explanation on the matter.
This workaround works, as mentioned in the previously linked content, because we do not pass --sysroot twice.

Merging is blocked because of permission issues

Here is the nixpkgs issue

!ci-build-me


allow for one isosurface

Algorithm needs to be adjusted so that it is possible to use just one isosurface value and have two resulting volumes. Currently requires at least 2 values.
GanttIgnore

fixed by #33


[Bug] Invalid path for moved / renamed files

I am not sure if it's a problem in danger or the plugin.
When a file get moved from a directory to another one, I guess GitHub sent something like this to the modified files, that causes the following error, because obviously that's not a valid path.
No lintable files found at path 'myProjectTests/{ => Views}/MyViewTests.swift'
To reproduce it, move a file so that GitHub generates something like this in the diff:

Hmm, that's an interesting bug! What version of the plugin do you have?

latest published danger-swxftlint 0.1.2

0.2.0 was posted four days ago. Until #16 is addressed, it only can only lint files defined in the .swiftlint.yml file, so this issue should go away. I'd like to keep the issue opened though. The version you're using uses the modified_files danger function, so this might be a bug in Danger. @orta have you seen modified_files contain errors in renamed file names?

Just wanted to add to this old issue:
I believe they are working on this at https://github.com/danger/danger/pull/681
For now, I'm doing something like this (credit to this comment):
files = git.added_files + git.modified_files
files_fixed = files.map { |path| path.gsub(/{(.*) => (.*)}/, '\2') }
swiftlint.lint_files(files_fixed)

This is is still reproducing for a moved file
18:38:56  Fatal error: 'try!' expression unexpectedly raised an error: Error Domain=NSCocoaErrorDomain Code=260 "The file "MovedSwiftFile.swift" couldn't be opened because there is no such file." UserInfo={NSFilePath=/path/to/initial/location/MovedSwiftFile.swift, NSUnderlyingError=0x7fa14dd53a60 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}: file /private/tmp/swiftlint-20190708-9813-

Using danger 6.0.9
Using danger-plugin-api 1.0.0
Using danger-prose 2.0.7
Using danger-swiftlint 0.23.0


updates image reference

nearly friday lets go can you approve?

Closing as this was used for GitHub Demo Days only.

Closing as this was used for GitHub Demo Days only.


Adding elements to arrays.

Sorry to bother with howto questions in issues section, but couldn't find any other place to ask it.
I'm using  DatabaseSearchTreemaNode.extend(MyTreemaNode); to create a list of available values to add  as a new array element.
After click on one of search result rows - it's being added as a

"element name"
 But to really push this element to array - user must activate this new row by mouse clicking on it (creates input element atm) and than press enter key to submit it.
I tried simulate same events with $, but it ain't work.
Is there any native way to skip this 'double-submit' and add new elements to array right after clicking on search result rows?

I see you closed this issue, did you figure out what you needed? Some thoughts:

It might work better for you to make a copy of the code for DatabaseSearchTreemaNode and modify it to work the way you need.
It might also help to test by triggering onClick directly with a mock UI event.
If you're testing elements which haven't been inserted into the DOM, the global $ won't work. You'll need to trigger through the treema node element directly.

Don't know if this method looks ok to you, but this is how i solve it:
            ProductTreemaNode.prototype.onClick = function (e) {
                var newSelection;
                newSelection = $(e.target).closest('.treema-search-result-row');
                if (!newSelection.length) {
                    return ProductTreemaNode.__super__.onClick.call(this, e);
                }
                this.getSelectedResultEl().removeClass('treema-search-selected');
                newSelection.addClass('treema-search-selected');
                this.saveChanges();

                if (!this.addingNewProperty()) {
                    return ProductTreemaNode.__super__.onEnterPressed.call(this, e);
                }
                return this.tryToAddNewChild(e, true);
            };

Maybe it will help someone else.


Updated API documentation and examples

@Sean-Holcomb made the updates based on your feedback. There have been a couple other API changes made that need to be documented, so this is a first of many and I'll add the other /cloudCost endpoints in a subsequent PR


Transforming results for a query Result with a page block with multiple tags in properties causes an exception

For some reason properties with multiple values are kept in a list. This causes an exception when using result transformation presumably due to the eval on the line 150 of datascript.cljs

npm 8.19.2 logseq 0.8.7
I don't know whether lq ran the same with previous versions of logseq. With -t sometimes id is the only column displayed. lq sq ... always raises exception.
node:internal/process/esm_loader:97
internalBinding('errors').triggerUncaughtException(
^
Cm [Error]: ' is not ISeqable

@Verneri I recall that being an issue with :block/properties-order which is why it's filtered out. Could you provide an example query and output that's causing this? This most likely needs to be fixed upstream in logseq. I thought we had fixed all :block attributes returning lists

@MagePrometheus Sounds like a separate issue and would recommend opening a separate one. I'll need specific steps on what you're typing and what you're seeing to help answer your issue


store key OpenWhisk configuration values in ConfigMaps

Go back to using ConfigMaps to store key cluster and database
configuration parameters and modify deployments to access the data
from those maps. Having explicit config maps makes it easier to
inspect the cluster configuration and enable separating the core
system and the providers into separate charts (subsequent PR after
this is merged).
Note: database and system authentication information continues to
be stored in secrets; that is unchanged by this PR.

@dgrove-oss I think we should also update document to affect the changes. We should document how to set up these configure maps. Do you want to use a separate PR to update document, or you want to include in this PR ?


Update GeoStylerContext.unsupportedProperties

This replaces the legacy UnsupportedPropertiesContext with the GeoStylerContext.
It also fixes the RuleTable example and adds an unsupportedProperties prop to the GeoStylerContext example:

closes #2172
Do you introduce a breaking change?

[x] Yes
[ ] No
[ ] I am unsure (no worries, we'll find out)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[v2] Styling bug when using he default preset for algolia search

🐛 Bug Report
The styling for algolia search seems to be broken in some cases when using docusarus v2.
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce
(Write your steps here:)

Go to https://react-isomorphic-data.netlify.com/
Type "with" or "use" into the searchbar.
The styling for the search result is broken

Expected behavior

I would expect the algolia search results to be displayed correctly, without the big gap in the center
Actual Behavior

Reproducible Demo
The repo for the site can be seen here: https://github.com/jackyef/react-isomorphic-data/tree/master/docusaurus

Related css here https://github.com/facebook/docusaurus/blob/master/packages/docusaurus-theme-search-algolia/src/theme/SearchBar/styles.css

I did some digging into the issue and I think the root cause could be one of these two:

Algolia changed some HTML and CSS but because we cloned their CSS in docusaurus-theme-search-algolia/src/theme/SearchBar/styles.css, our CSS is outdated and no longer matches the updated styles.
Algolia currently has a bug.

See the attached screenshots. I was comparing the difference between the working HTML and the buggy HTML - the buggy HTML seems to have missing classes that were present in other rows. Adding those two classes fixed the layout issue.

@endiliey any reason why we cloned Algolia's CSS here? Is that the correct way to use Algolia Docsearch?

Its reactjs.org css 😊.
The default one wasnt as nice and iirc is only available through cdn. Also colocated the css so that its swizzle-able easily.
P.S: waiting for stable release of docsearch v3 which is so much smaller in bundle size and came with dark/light theme css. They rewrote it from scratch and no longer relies on zeptojs


Simplify error handling

Various services had different ways of catching and dealing with errors. This PR streamlines the exception handling across all services - every error that extends either RESTError or AWSError is now caught by Moto, and the appropriate response format is determined uniformly.

Codecov Report

Merging #4936 (46718f3) into master (eed32a5) will increase coverage by 0.03%.
The diff coverage is 93.88%.

@@            Coverage Diff             @@
##           master    #4936      +/-   ##
==========================================
+ Coverage   95.53%   95.56%   +0.03%
==========================================
  Files         619      619
  Lines       65332    65001     -331
==========================================
- Hits        62417    62121     -296
+ Misses       2915     2880      -35

Flag
Coverage Δ

servertests
36.18% <16.66%> (-0.01%)
:arrow_down:

unittests
95.50% <93.88%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/appsync/responses.py
100.00% <ø> (ø)

moto/awslambda/responses.py
91.74% <ø> (-0.29%)
:arrow_down:

moto/cloudfront/responses.py
100.00% <ø> (ø)

moto/mq/responses.py
100.00% <ø> (ø)

moto/pinpoint/responses.py
100.00% <ø> (ø)

moto/s3control/responses.py
99.14% <ø> (-0.09%)
:arrow_down:

moto/xray/responses.py
60.41% <33.33%> (+0.82%)
:arrow_up:

moto/acm/responses.py
79.20% <84.21%> (-4.03%)
:arrow_down:

moto/sagemaker/responses.py
94.42% <90.16%> (+3.30%)
:arrow_up:

moto/apigateway/responses.py
94.80% <100.00%> (-0.32%)
:arrow_down:

... and 15 more

Continue to review full report at Codecov.

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

Always great to see a PR that deletes more code than it adds.  Nice work.  I think there are plenty more opportunities like this in the code base--I hope to get some time to work on some myself this year.

Always great to see a PR that deletes more code than it adds.

Agreed - it's very satisfying. The first time I came across the error_handler decorator I thought it was quite an elegant solution, but after copy/pasting it 5 times across to other services... not so much :slightly_smiling_face:


Clear method doesn't work

Clear method is missing .apply() call on editor.

Thank you for your report. This issue has been addressed in version 1.3.0.
In addition, there is clearRx() method available for using this feature in more reactive manner.


Repackage from org.researchobject to org.commonwl.viewer

Fixes #6

Looks good, just set artifactId to cwlviewer, otherwise you get viewer.jar
and "viewer" project in the IDE. You can merge it yourself thereafter.
On 24 Nov 2016 1:56 pm, "Mark Robinson"<EMAIL_ADDRESS>wrote:

Assigned #9 https://github.com/common-workflow-language/cwlviewer/pull/9
to @stain https://github.com/stain.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/common-workflow-language/cwlviewer/pull/9#event-871127031,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPd5cocuBuphFkrQC7eQ9nKvvumlSgTks5rBZdJgaJpZM4K7rQQ
.


migratemonitors.py: supply mapping file for (private) minions

Summary
Allow mapping of minion locations when migrating monitors. This is particularly important when migrating between regions, for example US to EU, as the private minion locations will not have the same location.
Desired Behavior
When migrating monitors allow the mapping of monitor locations, which is especially important when migrating between regions.
Possible Solution
An optional command line flag can be passed to reference a mapping file JSON dictionary, e.g.
{
  "1234567-aws_minion-D96": "9876543-nr_cpm_aws-556",
  "1234567-aws_minion-696": "9876543-nr_cpm_aws-F30"
}

Additional context
This may be a duplicate of issue #31. A PR with optional config flag and supplied mapping file, with example will be submitted shortly for consideration.

Fixed in https://github.com/newrelic-experimental/nr-account-migration/pull/39


[Bug][Flink-Kubernetes-V2] fix serviceaccount

What changes were proposed in this pull request
fix #2884

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

cc @wolfboys @Al-assad


ARROW-17290: [C++] Add order-comparisons for numeric scalars

See https://issues.apache.org/jira/browse/ARROW-17290

@pitrou, could you review?
cc @icexelloss

@bkietz Also curious about your opinion on this.

@rtpsw Do we need this for the AsOfJoin improvement?

I developed this PR's code while experimenting with designs for AsOfJoin; however, the design I chose to proceed with does not need this PR's code. So, I'm fine if this doesn't go through. I just figured I'd share the code to see what people think. I noticed that Arrow does not implement ordering for numeric Scalar types, but I wasn't aware of the considerations that @bkietz describes - perhaps this means the corresponding Jira, and not just this PR, is being questioned. The bit twiddling idea for ordering floats crossed my mind, but I opted to start with something simpler.

If this is intended for use in AsofJoin then we should definitely not be using Scalars at all since extracting them from a join key column, boxing them in the type-erased Scalar, then unboxing them again to do type check and comparison would be wasteful compared to doing the comparisons in-place against elements of each column.

I experimented with adding support for updating a Scalar in-place, which potentially could have avoided the above costs in my code, but ran into implementation problems (changes needed in too many places across Arrow) and decided to give up on it.

Tangent: I don't know how applicable this will be, but at least for testing purposes it might be handy to use something like Rust's float_ord to cast floating point numbers into something less subtle to sort. For example: https://gist.github.com/bkietz/8e2ef182883b886e532ffde8e537f7a3

These resources are interesting - thanks for sharing! I also found http://stereopsis.com/radix.html (linked from https://docs.rs/crate/float-ord/latest) interesting.

@rtpsw Do we need this for the AsOfJoin improvement?

No.

Just to clarify, I'm holding on looking into specific comments on this PR until and if we decide to move forward with it.

If we don't currently or in the near future require support for comparison of Scalars in the way this PR provides, then I'd say we should close this PR and mark the JIRA wontfix.


having the plugin in the folder make the port 8080 problem

having the plugin in the folder make the port 8080 problem
I guess the plugin need an update

I don't think that is correct as this plugin does nothing with ports, so that would be an issue with millennium then. Closing this.

If i reopen steam it this plugin enabled I get the port 8080 problem and if i remove it then millennium is working dont know what else I need to as
also when i have it on it just not working for me


Platform change!

Hi guys, is it possible to somehow change the platform from which the game will run, mac os or lunux? Is it possible?

For Gnu/Linux there is https://github.com/PaulCombal/SamRewritten

For Gnu/Linux there is https://github.com/PaulCombal/SamRewritten

Thank you pal!


Bugfix in propagate positions

Solves issue #1304

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (e4df024) 89.22% compared to head (5961d72) 89.22%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1305   +/-   ##
=======================================
  Coverage   89.22%   89.22%
=======================================
  Files          51       51
  Lines        7595     7595
=======================================
  Hits         6777     6777
  Misses        818      818

Flag
Coverage Δ

unittests
89.22% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lark/parse_tree_builder.py
96.42% <100.00%> (ø)

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


Post Feed Image Conversions

Having trouble displaying the "conversion" image on the "posts feed" card in any related view.
Upon further inspection i realize that when uploading images to the gallery, they get stored perfectly in https://example.com/storage/x/123.jpg but for some reason when publishing posts, the thumbnail image does not get stored in what should evidently be https://example.com/storage/x/conversions/123.-thumb.000jpg
Could this somehow be related to my permissions - which i doubt; or is it something else? Please kindly advice on how i can resolve it...
P.S: I would suggest a link in admin below "edit thumbnail image" to the path of the actual thumbnail, similar to the gallery setup; all issues apart, you've done an exceptional job. Thanks

same IS

Hum, I will check it out,
Open a PR if you find a solution please
Thank you

I fix it in https://github.com/guillaumebriday/laravel-blog/commit/f25a6374fcfc4ae37743b034e56f093dc06ec6f0
Thank you guys !


Update RHEL8 install guide noting 3.0 will be available starting in 8.1

Problem encountered on https://dotnet.microsoft.com/download/linux-package-manager/rhel8/sdk-current
Operating System: Linux RHEL 8 - x64
The instructions are wrong as there is no dotnet 3.0 in the repository of RHEL 8 there is dotnet-sdk-2.1.x86_64 and dotnet-sdk-2.1.5xx.x86_64  only these two versions in RHEL 8 repository please check..

Duplicate of #3491

If anything the documentation on the download page needs clarifying?

@leecow is this your area of expertise?

Will get the site fixed up.

Can this be closed @leecow?

This topic in regards to .NET 5 should be discussed...

Is this still an issue for .NET 5 or can this be closed? /cc @adegeo

I think this can be closed.


extra changes on cards-es.coffee

Some extra things I've seen after the files were merged and uploaded:
-Mandalorian Mercenary (Scum Firespray) and Wild Space Fringer (YT-2400) weren't translated due no entry on the file.
-Black Sun Enforcer was missing in the builder due the StarViper name change, as he didn't had a ship assigned.
-Airen Cracken had the same problem due the Z-95 name change.
-Patrol Leader (Decimator) was missing due the ship name change, but he also didn't have an entry on the file.
-R4 Agromech and R4-B11 had blank cards due no entry. Added both.
-IG-88B had a typo on his %CANNON%, showing the misspelled word instead the icon.
-Small typos in a few other lines.
And I think that is all. Except for the "SCUM ONLY" (on Greedo, i.e.) that I have no idea where it is. It should be "SOLO ESCORIA".

I've just noticed I missed three more cards, appearing blank as they doesn't have an entry. Added.
-Autoblaster Turret
-Outlaw Tech
-K4 Security Droid
Can't see anything else missing, at least for now :P


No i18n support: components should provide an object with Labels as props

Some components render localized text, but provide no way to alter it outside of the component.
Example: https://github.com/reactioncommerce/reaction-component-library/blob/33651f18c8dae7333fb3a019ac6bb878657c70fe/package/src/components/CartSummary/v1/CartSummary.js#L196
The component library should not be responsible for i18n itself.
It could accept an object with labels as props, in a similar manner compnents are passed and overrriden. The current strings could serve as fallback.

This is a known deficiency that will eventually have a solution. The workaround in the meantime is to override the entire component using the components context, providing a version of it that you've modified to use whatever i18n solution your app uses.

Yes, that would work in the meantime, thank you.
Would you accept PRs that would allow passing own strings(so still i18n-library-agnostic), or does this need further internal discussion?

@nnnnat Any thoughts here? Some components already accept label overrides (e.g., otherAddressLabel prop on AddressChoice). I think it makes sense to accept PRs that do this same thing for other components that have English text built into them. It's the simplest way to support translations without buying into any particular framework.

Can labels be added for this issue? I think its a fairly easy starting issue.

Hey, this issue doesn't seem to get a lot of traction. When it comes to components every single component that has text would need to get an extra prop for translation.
Is there any possibilty where we can slowly tackle this issue by doing components in batches?
Also, could I be assigned to this issue? We already did a big part when it comes to translations in our component library.

Did you guys ever make any progress on this issue @nnnnat @aldeed ?

@nnnnat I am currently adding all of the components with individual pull requests. Each of the components will have an override label so we can set i18next translation from the frontend side

@nnnnat could you check if the current PR's are viable to be implemented?

@HarisSpahija I approved PRs. One needs a small change. Someone with ability to override the snyk checks will have to merge them, or you can merge master into them after the Snyk-related PRs have been merged.

Some components have strings that are possibly gramatically different in other languages. For example:
<StockWarning />
return Only {inventoryQuantity} in stock;

In Dutch one would say "Slechts 7 op voorraad", while in Bosnian you could say: "7 Na Lageru". Grammar should be counted in when implementing i18next.

All components should have translation options now with a couple of exceptions. @janus-reith we could move to adding i18n to example-storefront now

@HarisSpahijaPon For those that need interpolation, you can update to take a fn prop instead.
// default props
getLowStockText: (data) => `Only ${data.inventoryQuantity} in stock`

const { getLowStockText } = this.props;

return {getLowStockText({ inventoryQuantity })};

When using in translated app, you'd pass through to i18next in the function prop:
getLowStockText={(data) => i18next.t("lowStockText", data)}

And your translation file would have "Only {{inventoryQuantity}} in stock" as the English value, for example.
See https://www.i18next.com/translation-function/interpolation

okay thanks! I will update them @aldeed

I think pretty much all components except the interpolation items have been set now. Is it time to implement a basic example to the storefront for i18n?
@janus-reith , would you be interested in helping with this issue? I am not sure what exactly is needed to make a sufficient example for users to test. Would be cool if we can implement it before 3.0 release.

@HarisSpahijaPon Great work, sure we can collaborate on this.

Hey guys @HarisSpahijaPon @janus-reith , I'm also interested in collaborating on this, please let me know how i can help.

@CristianCucunuba Id say check out this issue: https://github.com/reactioncommerce/example-storefront/issues/524
You could fork the master branch and create an example using the new label props to show translations. I think its best if we start with one component and one language. Preferably English.

I noticed that some components are using other components from the component library. Components such as MiniCart make it so labels cannot be passed to MiniCartSummary without either overriding componentContext or passing props from the parent. For example:
 <MiniCartSummary displaySubtotal={summary.itemTotal.displayAmount} />

Can be translated with the override labels
 <MiniCartSummary displaySubtotal={summary.itemTotal.displayAmount} checkoutButtonText={"test"} footerMessageText={"test"}/>

Because the component is nested there is no way we can reach the override labels. This should be fixed in the component library or an extra props should be added to pass nested props.
What do you think would be the right approach to this @aldeed ?

@HarisSpahija I tend to prefer specific props like miniCartSummaryCheckoutButtonText, etc. which are passed down. This allows specific prop validation whereas a miniCartSummaryProps prop opens the door to hundreds of potential combinations that would need to be tested and supported.
That said, I don't know that Reaction has a specific policy on this. @mikemurray or @nnnnat may have more to say.
It's not quite correct to say there's no way to do it, though. If you use the components context to inject translations for MiniCartSummary (and all components), which would be the best and recommended way, then it will be translated wherever it is used without any need for passing down props. That's really the main reason why this package supports a component context: ability to override nested components.

I tried setting an override in component context, but I couldnt get the syntax to work without sideffects. Could you elaborate how to do this with an example?

@HarisSpahija I didn't test this, but it should work as long as you provide a component (e.g. a function that returns JSX).
{
  MiniCartSummary: (props) => <MiniCartSummary checkoutButtonText={i18next.t("MiniCartSummary.checkoutButtonText")} footerMessageText={i18next.t("MiniCartSummary.footerMessageText")} {...props} />
}

You'd have to make a slightly more complicated component that uses useTranslation hook to get a properly reactive version of i18next.t function, but this is the general idea.

Some components require some references or not every component is listed in the componentContext file, we should refactor those to make it easier for developers to override the components.

Agree that no components should directly import any other components unless there's clearly no use case for overriding it. Submitting individual GH issues for each case of that would be best.


avoid requiring tmp_path introduce in pytest 3.9.0

Reverts #243 and avoids the use of tmp_path introduced by #230 instead.
Easier to read whitespace-ignoring diff: https://github.com/colcon/colcon-core/pull/246/files?w=1
@rotu FYI.

Codecov Report

Merging #246 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #246   +/-   ##
=======================================
  Coverage   80.04%   80.04%
=======================================
  Files          54       54
  Lines        3107     3107
  Branches      514      514
=======================================
  Hits         2487     2487
  Misses        582      582
  Partials       38       38

Continue to review full report at Codecov.

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

Thanks for pointing out the problems!


is CONTRIBUTING.md up to date?

is CONTRIBUTING.md up to date?
Hi,
I wanted to contribute a new article post with a PR to this repo but I noticed that in the https://github.com/pnp/blog/blob/main/CONTRIBUTING.md
point 8 we have:

but checking the repo contents I noticed we rather store Images in the same location where we keep the blog post.
For example

Not sure if I understood this properly but seems just not inconsistent 🤔.
Would you mind if I open a small PR and correct this guidance? or maybe you want to do it yourself with a quick commit 😉? Or maybe it is ok, and I just don't understand it properly 😋 (in that case sorry for the stupid question 😉)

hey @Adam-it  we needed to change the approach to make it easier for people to contribute, and you as an early adopter already noticed this :-)
We appreciate your help, please PR :-)
You rock! 🚀

So it's a new thing... that's cool 👍. I don't know if my opinion counts but IMO a step in the good direction 😎. Maybe now when I won't need to leave VS code to add a blog post I will try to contribute even more 😋.
I am writing a new post even now so I will go over the contribution guide point by point and if I will notice anything else I will also include it in this PR (which I will try to open today during late night 😉).
BTW thanks ... I always wanted to rock 🤩🤩


Add the ability to draw an Image object with a width and height

Added with 92a5ada56e850eaa00cdb237ef6015bad46ef6a4 but needs testing


Is there an example to read data from?

Is there an example to read data from?

There are some samples provided for Android, UWP, and iOS in the ./samples/ folder. @bipar, Is there something more specific you're looking for? If so, the community could create a sample.
https://github.com/angelcalvasp/BluetoothPlugin/tree/master/samples

There isnt an example yet for reading I havent worked on it yet, dont have devices to try reading with yet

@angelcalvasp it is a very good plugin, but as @biapar asked/suggested, a read method for IBluetoothDevice interface is really missing...
If you want to experiment before implementing Read method, why don't you use your smartphone and laptop, where laptop (with BT adapter) is the BT device to read from ?
On laptop it is unbelievable easy to create an application (for example in .NET) to use BlueTooth (the serial port profile, SPP). If you want, I can provide a sample.

I will try it out over the week, thanks for the snippet

Thruth is, Ive been really busy with work, and barely gives me any time to work on this

angelcalvasp.
Thank for this project, it is a very good plugin.
I'm working on an enterprise project and need it connect to several mobile Bluetooth scanner and I'm using yours and is easy to use.
I completed the reading process and work perfectly, but I adapted to my requirements.
This is the changes I  made to my needs.
`public class BluetoothDataReceivedEventArgs : EventArgs
{
public BluetoothDataReceivedEventArgs(string data)
{
Data = data;
}
    public string Data { get; set; }
}`

In the Bluetooth ConnectionThreat.cs the ReceiveData  is changed to this.
`private async void ReceiveData(BluetoothSocket socket)
{
var socketInputStream = socket.InputStream;
//byte[] buffer = new byte[256];
        string res = "";
        // Keep looping to listen for received messages
        while (!IsInterrupted)
        {
            byte[] result;
            int length;
            length = 50;
            result = new byte[length];
            try
            {
                await socketInputStream.ReadAsync(result, 0, length);
            }
            catch ()
            {
             }
            char lst = ' ';
            for (int i = 0; i < result.Length; i++)
            {
                var chr = Convert.ToChar(result[i]);
                lst = chr;
                if (result[i] >= 48 && result[i] <= 57)
                    res += chr;
                else if (chr == '\r' || chr == '\n' || result[i] > 250)
                {
                    DoReceivedData(res);
                    res = "";
                    break;
                }
            }

        }

    }`

In my case I only require numbers reading from the scanner, that is why I limits the reading to 0~9.
Best Regards,
BenNelcha.


Implement zoneout

http://arxiv.org/abs/1606.01305

Hi! I interested in this issue.
Here is my idea.

Make upward and lateral of link.LSTMBase configurable.

For example,
    def __init__(self, in_size, out_size,
                 lateral_init=None, upward_init=None,
                 bias_init=0, forget_bias_init=0,
                 upward=None, lateral=None):
        if upward is None:
            upward=linear.Linear(in_size, 4 * out_size, initialW=0)
        if lateral is None:
            lateral =linear.Linear(in_size, 4 * out_size, initialW=0),
                                  initialW=0, nobias=True),
        super(LSTMBase, self).__init__(
            upward=upward,
            lateral=lateral,
        )

Upward and lateral can be any link.
This is not only for Zoneout, but also for convolutional or any other links. (See #1288).

Zoneout implemented as link.

3)Implement ZoneoutLSTM.
Its __init__ starts with
super(LSTM, self).__init__ with ZoneOut(linear.Linear(in_size, 4 * out_size, initialW))
How about this plan?
If OK, I ready to implement Zoneout.

It lookg zoneout has already been implemented, but is not added to the docs.

Solved via #1457 and #3957


Crash to Log-in screen

The system crashes to login screen when pressing the Super key. Doesn't happen 100% of the time.
Here are the relevant logs:
Nov 20 08:27:05 localhost.localdomain gnome-shell[20464]: JS ERROR: TypeError: this._extension.settings is undefined
updateWidgetWidths@file:///home/octo/.local/share/gnome-shell/extensions/mediacontrols@cliffniff.github.com/player.js:541:30
initWidgets@file:///home/octo/.local/share/gnome-shell/extensions/mediacontrols@cliffniff.github.com/player.js:246:18
updatePlayer@file:///home/octo/.local/share/gnome-shell/extensions/mediacontrols@cliffniff.github.com/widget.js:493:33
unhidePlayer@file:///home/octo/.local/share/gnome-shell/extensions/mediacontrols@cliffniff.github.com/widget.js:555:26
_playerPropsChanged@file:///home/octo/.local/share/gnome-shell/extensions/mediacontrols@cliffniff.github.com/player.js:353:41
@resource:///org/gnome/shell/ui/init.js:21:20
Nov 20 08:27:05 localhost.localdomain gnome-shell[20464]: Window manager warning: Trying to remove non-existent keybinding "mediacontrols-toggle-trackinfomenu".
Nov 20 08:27:05 localhost.localdomain gnome-shell[20464]: Window manager warning: Trying to remove non-existent keybinding "mediacontrols-toggle-trackinfomenu".
Nov 20 08:26:30 localhost.localdomain gnome-shell[20464]: Window manager warning: Trying to remove non-existent keybinding "mediacontrols-toggle-trackinfomenu".
Nov 20 08:26:30 localhost.localdomain gnome-shell[20464]: Window manager warning: Trying to remove non-existent keybinding "mediacontrols-toggle-trackinfomenu".
Some info about the System:
System:
Kernel: 6.6.1-1-default arch: x86_64 bits: 64 compiler: gcc v: 13.2.1
Desktop: GNOME v: 45.1 tk: GTK v: 3.24.38 wm: gnome-shell dm: GDM v: 45.0.1
Distro: openSUSE Tumbleweed 20231117

This has been fixed in v30, which is in review.


Ignore filter not being applied to translations on upload

We have the following configuration file, working with github sync:
files:
  - source: /docs/src/**/*.md
    ignore:
      - /**/%file_name%-%two_letters_code%.md
      - /docs/src/pages/getting-started/page-layout-examples/**/*
      - /docs/src/pages/premium-themes/**/*
    translation: /%original_path%/%file_name%-%two_letters_code%.%file_extension%
  - source: /docs/translations/*.json
    ignore:
      - /**/%file_name%-%two_letters_code%.json
    translation: /%original_path%/%file_name%-%two_letters_code%.%file_extension%

(Original: https://github.com/mui-org/material-ui/blob/next/crowdin.yml)
However, on running crowdin upload --dryrun while in the project root directory, it indicates that it would upload all files, including translations:
src/modules/components/backers-de.md
src/modules/components/backers-es.md
src/modules/components/backers-fr.md
src/modules/components/backers-ja.md
src/modules/components/backers-pt.md
src/modules/components/backers-ru.md
src/modules/components/backers-zh.md
src/modules/components/backers.md
src/pages/css-in-js/advanced/advanced-de.md
src/pages/css-in-js/advanced/advanced-es.md
src/pages/css-in-js/advanced/advanced-fr.md
src/pages/css-in-js/advanced/advanced-ja.md
src/pages/css-in-js/advanced/advanced-pt.md
src/pages/css-in-js/advanced/advanced-ru.md
src/pages/css-in-js/advanced/advanced-zh.md
src/pages/css-in-js/advanced/advanced.md
[etc...]

In addition, as well as wanting to upload the translations, it appears to want to upload all source files new, changed and existing, rather than just new or changed files.
Thanks.

Matt, sorry for this ;(
It is indeed weird behavior
Also passing it to the team. Hope it will be fixed shortly

No worries. Just to mention, we had a similar issue with the GitHub integration (with this same YAML file and directory tree), so it might be related.

Thank you, Matt :)

Hello Matt,
Ignore parameter might not be working since placeholders like %two_letters_code% are not supported there - could you please change it to ** instead and check whether it works as expected?

Hi, no, neither - /**/%file_name%-**.md nor - /**/%file_name%-*.md work. All translations are still listed.
Even if they did, we unfortunately have source files ending "-xx.md" that would be filtered out for github sync.
With hindsight we should perhaps have used a different delimiter for the two letter codes, such as:
translation: /%original_path%/%file_name%_%two_letters_code%.%file_extension%

but too late now...

Hi @mbrookes! If everything alright with the project? We would really like to help, so, please, let us know if there is anything we may assist with!

Hi @Gosteeva, yes, everything is great using this config with GitHub, but unfortunately not with crowdin-cli. It seems to suffer from the same issue that we had originally with the CrowdIn GitHub integration (now fixed).

Hi @mbrookes ,
Thanks for the additional information! Forwarding it to the team for further investigation
We'll keep you updated on the matter

Hi, I see in another issue that there is a new release of the CLI planned. Will this issue be fixed in that release? Thanks.

Hi @mbrookes,
Sorry for the delays. We remember about this issue and the task on the matter is created.
I've just set the high priority to it and we will do our best to provide a fix in the next version of CLI

Actually, the way we are using Crowdin now, this is less critical, so please don't worry if it can't make the next release. I was just interested to know. Thanks!

@mbrookes  Thanks for the understanding, I'm really happy that the current issue is not a blocker for you. Anyway, we will do utmost to provide the fix as soon as we can

Hi @mbrookes !
This issue is fixed in CLI 3


1.18.2 can't connect to server

Describe the bug
[Render thread/INFO] [modflared/]: Disconnecting from server, stopping cloudflared...
[Render thread/INFO] [modflared/]: Cloudflared stopped successfully
[Render thread/INFO] [net.minecraft.client.gui.screens.ConnectScreen/]: Connecting to test.redacted.com, 25565
[Server Connector #2/INFO] [modflared/]: Connecting to test.redacted.com/redacted:25565
[Server Connector #2/INFO] [modflared/]: Using tunnel for test.redacted.com
[Access#1/INFO] [modflared/]: [windows-amd64.exe access tcp --hostname test.redacted.com --url localhost:26345]
[Access#1/INFO] [Cloudflared/]: 2024-01-15T06:25:23Z INF Start Websocket listener host=localhost:26345
[Server Connector #2/ERROR] [net.minecraft.client.gui.screens.ConnectScreen/]: Couldn't connect to server
io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: no further information: localhost/[0:0:0:0:0:0:0:1]:26345
Caused by: java.net.ConnectException: Connection refused: no further information
I have a tunnel correctly configured, can connect over cloudflared on client.
I can connect via <IP_ADDRESS>:26345 to the server. localhost:26345 does not work.
I am currently using forced_tunnels.json, however I have added to correct CNAME and TXT DNS records on subdomain "test" with value "cloudflared-use-tunnel"
Which version of the mod/game are you using?
Minecraft: Forge 1.18.2
Mod Version: 1.0.0 Alpha
To Reproduce
Steps to reproduce the behavior:

Start game
Click on multiplayer
Add server "test.redacted.com"
Add server "localhost: whatever modflared gives you "
Add server "<IP_ADDRESS>: whatever modflared gives you "
See error

Expected behavior
test.redacted.com has a connection
Screenshots
Available if asked for, will have to redact information
Additional context
Tried looking through other issues but the only 1.18.2 issue had a similar problem but was marked complete due to a lack of response, hoping to work through the issue. I think it might have to due with forge and an inability to connect to localhost, only <IP_ADDRESS>, but maybe that's a me problem

Because of this issue I changed the address in the new versions to <IP_ADDRESS>. If I get home I will make an update that changes it to <IP_ADDRESS>. Hopefully this fixes that issue.

I uploaded a new version can you check if it now works new version

I am now getting stuck at "encrypting". I can see that I attempt to join the server, but I eventually just time out.

That's normal when using Cloudflare tunnels for playing Minecraft. You can try restarting Cloudflared on the server.

never had the issue running cloudflared directly on the client, ill try a few more times though

was able to connect running cloudflared on the host (cloudflared access tcp --hostname test.redacted.com --url <IP_ADDRESS>:25565), but cannot connect via "test.redacted.com" through modflared, keeps getting stuck at encrypting and timing out. Is there any good way to fix this?

The problem is that I don't know exactly why this problem occurs. For me the error occurred for 5 minutes and disappeared again without any changes made.

Got it, looks like the connection at least works to a degree. I can make a separate ticket if I continue to never connect.


[BUG]: Paypal Button Click opens auto:blank#blocked popup

Is there an existing issue for this?

[X] I have searched the existing issues.

🐞 Describe the Bug
We have implemented Paypal integration in our web code. But there is a very weird issue coming, whenever we click on the button of Paypal it opens auto:blank#blocked popup on top of Paypal popup. See the screenshot attached. Once we close this blocked popup than we see the Paypal popup to proceed with the payment.

😕 Current Behavior
This is the code that is implemented
import { PayPalScriptProvider, PayPalButtons } from "@paypal/react-paypal-js";
<PayPalScriptProvider
options={{
clientId:
process.env.NEXT_PUBLIC_PAYPAL_CLIENT_ID,
debug: true,
currency: "USD",
intent: "capture",
}}
>
<PayPalButtons
style={{
layout: "horizontal",
color: "blue",
shape: "pill",
height: 40,
tagline: false,
label: "pay",
}}
onClick={async (data, actions) => {
const formErrors =
await formikProps.validateForm();
                            formikProps.setTouched({
                              ...formikProps.touched,
                              ...formikProps.errors,
                            });
                            if (JSON.stringify(formErrors) !== "{}") {
                              return false;
                            }
                          }}
                          createOrder={async (data, actions) => {
                            return await createPaypalOrder(
                              formikProps.values,
                            );
                          }}
                          onApprove={paypalBuyAcknowledgement}
                        />
                      </PayPalScriptProvider>

                      ![Screenshot 2023-07-21 at 6 22 20 PM](https://github.com/paypal/react-paypal-js/assets/129676978/a1894fed-9631-457d-a051-916f9faab80b)

🤔 Expected Behavior
As per the documentation, user should see this type of behaviour but see some other behaviour

,
🔬 Minimal Reproduction
Please let us know if you need specific url and credential to test

🌍 Environment
| Software         | Version(s) |
| ---------------- | ---------- |
| react-paypal-js  |  8.1.0     |
| Browser          |Chrome, Edge|
| Operating System | Mac OS Ventura3.4.1 (c)        |

Relevant log output
NA

Code of Conduct

[X] I agree to follow this project's Code of Conduct

➕ Anything else?
NA

The issue might be related to "Cross-Origin-Opener-Policy". Could you check the console warning related to the Paypal domain?

Same issue. Only got this in console:

Hi @praveencloudalgo, please share a reproduction url. My initial thought is along the same lines as @xenotime-india where there it may be related to using "Cross-Origin-Opener-Policy"

@praveencloudalgo Since we haven't had a response to more information, I'm going to close this issue without action. If you are still facing this issue, please re-open and provide the information requested.

Hi @praveencloudalgo, please share a reproduction url. My initial thought is along the same lines as @xenotime-india where there it may be related to using "Cross-Origin-Opener-Policy"

what's the fix if it's related to this? or it's just not possible on paypal sdk at all?


Investigate test failures: PythonExecutableService Importing with a valid PYTHONPATH from .env file should succeed

Windows & Python 2.7

windows10 python3.6  used .env  it's error


local development environment .env file should include the development maps api key

reproduction steps

follow the docs to setup local docker development environment
access the local app at http://localhost:8080

expected

show the map

actual

javascript errors, map is not shown

implementation notes

should include the development api key in the secret .env file which is added as part of the development environment seutp
the relevant template should be modified to use the api key from env var (with default being for production environment)

@carmelp16 - this is a small non-urgant task - when you have some time please take this one


long line of code can bring up weird menu

new empty notebook
add JS cell
press enter and then on the 2nd line, enter a very long line, long enough that the thumb part of the scrollbar only fills half of the width of the cell.
while the cell is scrolled to the right, click on the 2nd line.
a weird menu pops up:

i can't find the issue but i think it was sugested that we not use scrolling at all and just wrap long lines in code cells.  that would probably solve this problem.

https://github.com/twosigma/beaker-notebook/issues/3423
let's try fixing this by avoiding the problem.

OK we avoid it but if you manually turn off word-wrap, then this can still happen, so we still need to fix it.


Add oci_adm_knowledge_base and oci_adm_vulnerability_audit tables

Integration test logs

Logs
Add passing integration test logs here

Example query results

Results
Add example SQL query results here (please include the input queries as well)

Reference - https://github.com/turbot/steampipe-plugin-oci/pull/442


Add python bindings for sdf::Element

🎉 New feature
Addition to https://github.com/gazebosim/sdformat/issues/931
Summary
Test it
ctest -R pyElement
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

I've retargeted this to main to speed up development in gz-sim.


Globally change maxRuleApplications?

Is it possible to globally change maxRuleApplications to something less than 200?
As it is, my editor becomes a bit too slow while waiting for unsuccessful aesop calls.
I suspect that all my successful calls succeed in less than 200 applications (can I confirm this?).

Fair question. At the moment, your only choice is to write a wrapper macro that calls Aesop with specific options. But I think it would make sense to use the Lean options framework for this, i.e.
set_option aesop.maxRuleApplications 50
I already use this style for some options, might as well use it consistently.
Jun 2, 2022 10:28:14 Asta H. From @.***>:

Is it possible to globally change maxRuleApplications to something less than 200?
As it is, my editor becomes a bit too slow while waiting for unsuccessful aesop calls.
I suspect that all my successful calls succeed in less than 200 applications (can I confirm this?).
—
Reply to this email directly, view it on GitHub[https://github.com/JLimperg/aesop/issues/11], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAEBE5UAUDJKJMFXKNROTJLVNCEDZANCNFSM5XUQFWHQ].
You are receiving this because you are subscribed to this thread.[Tracking image][https://github.com/notifications/beacon/AAEBE5QZMJBEJKQJQJFH5OLVNCEDZA5CNFSM5XUQFWH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4SX3PMOA.gif]


Update documentation

Summary:
This PR updates the content of file readme.md to emphasize the use of locally stored datasets.
This also provides documentation about the use of new command line option exclude with short name -x.
Please make sure these boxes are checked before submitting your pull request - thanks!

~[ ] Run the unit tests with gradle test to make sure you didn't break anything~
~[ ] Format the title like "Fix #<issue_number> - " (for example - "Fix #1111 - Check for null value before using field")~
[ ] Linked all relevant issues
~[ ] Include screenshot(s) showing how this pull request works and fixes the issue(s)~

Question: we would like to encourage our software users to validate locally stored GTFS datasets rather than validating GTFS datasets stored on openmobilityda.org.
In this documentation, more precisely in the sample usage that we provide, should we totally disregard this last functionality and only provide sample usage to validate locally stored GTFS datasets? Or should we make this doc the most generic possible: provide 2 sample usages:

one explaining how to validate a locally stored dataset,
then another one detailing how to validate a GTFS dataset stored on openmobilitydata.org?

Re: Question
I think the advisable thing would be to describe the url capability without pointing to openmobilitydata.org


sql_exporter: 0.3.0 -> 0.4.0

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110206 run on x86_64-linux 1

1 package built:

prometheus-sql-exporter

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110206 run on x86_64-linux 1

1 package built:

prometheus-sql-exporter

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110206 run on x86_64-darwin 1

1 package built:

prometheus-sql-exporter

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110206 run on x86_64-darwin 1

1 package built:

prometheus-sql-exporter

Thanks everyone!

Thanks everyone!


⚠ Bofrost.at has degraded performance

In c1d0d17, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1248 ms

Resolved: Bofrost.at performance has improved in 7ccc146 after 18 minutes.


Cleanup main & WR branches

The main branch should be for README. The production and dev branches will have respective configuration files.

Removing WIP or Revert branches also.

Need all files removed except the README.

done.


enable comint mode for terraform apply

terraform apply requires typing 'yes' to actually execute the change

I thought compile could handle stdin prompts, but I guess not? Thanks in any case!


feat: running command synchronously

Closes #95
Add sync arg after the TSTools command to run the command synchronously.

Looks and works ok, thanks for contrib!

sorry, but how to use this feature


Render Helper w/out Sinatra

There are definitely parts of Padrino, particularly the helpers, that don't seem to be limited to Sinatra. Their usefulness certainly isn't limited to Sinatra and in many cases the implementation does not require Sinatra.
For example, Middleman, the popular static site generator, uses Padrino-based helpers so you can use those helpers for generating static sites. Other places where they helpers might be useful include Grape, or in Jekyll tag plugins.
I'd like to use them in Thor generators. Those are the generators that power everything from "bundle gem" to "rails generate". I have generators that build documentation or HTML reports, so the Padrino helpers are useful.
I haven't had any trouble with most of the helpers - I've used the Output, Tag and Asset helpers without any problems, but I am not able to easily use the Render helper because I'll get a method missing error.
It seems that Middleman was able to support Padrino's render helper, but had to implement a Sinatra/Padrino compatible render interface.
I'm wondering if either:

You could include assumptions in the helper documentation about the interface implemented by the class they are mixed into, like the required signature of render to use the RenderHelpers.

or

Fallback to Tilt or some other approach if render is not implemented (i.e. not inside Sinatra or a Sinatra-compatible class).

Another option would be to just implement the shim ourselves. I see the issue that Padrino basically provides the only proper view engine that is halfway independent of a framework in the Ruby space.

Yeah this is an interesting point I've thought about before, there isn't much that actually ties the helpers module to Sinatra, they are just generally useful HTML generation methods for the most part (except for the rendering and partial logic). Thanks for bringing up this issue for us to think about.

I just switched some Thor code that looked like this:
Crosstest.manifest.scenarios.values.each do |scenario|
  @scenario = scenario
   template 'templates/_test_report.html.tt', "details/#{scenario.slug}.html"
end

Over to this:
template_file = find_in_source_paths("templates/_test_report.html.haml")
template = Tilt.new(template_file)
Crosstest.manifest.scenarios.values.each do |scenario|
  @scenario = scenario
  add_file "details/#{scenario.slug}.html" do
    template.render(self)
  end
end

It was interesting to see that since the Padrino helpers were loaded the template was an instance of Padrino::Rendering::HamlTemplate rather than Tilt::HamlTemplate. This change also means I'm not using Thor's template rendering at all, it's all Tilt/Padrino. The only thing I use from Thor is find_in_source_paths.
I can't use partials, though, because:
NoMethodError: super: no superclass method `render' for #<Crosstest::Command::Generate::Dashboard:0x007fc3b544ad38>
from /opt/boxen/rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/padrino-helpers-0.12.4/lib/padrino-helpers/output_helpers.rb:33:in `render'

So I'm not sure about how the Sinatra stuff works, but in terms of pure-Tilt setup it seems like it would just be useful to customize the search paths for templates, but once Padrino finds the partial it seems like Tilt.new(partial).render(locals) would work rather than each Tilt-based framework having a render method.
Hope that makes sense... I wasn't exactly sure what sort of a shim you had in mind.


利用GitHub Pages + jekyll快速搭建个人博客 - 阿望的窝 | Awang

https://awang0608.github.io/2019/11/09/利用-GitHub-Pages-快速搭建个人博客/
Every failure is leading towards success.

I admire the infinite heart animation in https://awang0608.github.io/sweet.html
May I consult what kind of technology do you use to implement that?


feat: Thread names specified using enum

Good call on adding names to the threads!  I think this should be good to merge.

I've added additional commits

@ChosunOne it looks like the latest branch is now 'collator' rather than 'basic_cli'. are you happy with my further changes?

I'm going to create a separate PR into 'collator'