Failed to compile custom app on Mac

Hi,
I compiled the SDK on Mac OS (Sierra) and did system-wide install. However there is no sudo ldconfig (which is used on Linux to update the linker cache) in Mac. What is the equivalent way to do it in Mac?
I also followed the sdk guide (https://sdk.dronecode.org/en/guide/toolchain.html) in order to compile a custom C++ app, and created a custom app and the corresponding CMakeLists.txt.
However, when I run make, I get,
[ 50%] Building CXX object CMakeFiles/formation_gs.dir/formation_gs.cpp.o
/Users/abdkadma/Desktop/src/DroneCode_apps/formation_gs.cpp:15:10: fatal error:
      'dronecode_sdk/action.h' file not found
#include <dronecode_sdk/action.h>
         ^~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
make[2]: *** [CMakeFiles/formation_gs.dir/formation_gs.cpp.o] Error 1
make[1]: *** [CMakeFiles/formation_gs.dir/all] Error 2
make: *** [all] Error 2

So, what could be the problem here? Is it because the linker cache is not updated?

Issue is solved by specifying the include and lib system directories in the CMakeLists.txt.
Would the tutorials need to be updated in order to account for this?
if(NOT MSVC)
    add_definitions("-std=c++11 -Wall -Wextra -Werror")
    include_directories(/usr/local/include)
    link_directories(/usr/local/lib)
else()
    add_definitions("-std=c++11 -WX -W2")
    include_directories(${CMAKE_SOURCE_DIR}/../../install/include)
    link_directories(${CMAKE_SOURCE_DIR}/../../install/lib)
endif()

Also, I would like to mention that I used the develop branch.

Also, I would like to mention that I used the develop branch.

That is the correct branch to use. Develop is for ongoing development, master is for the last official release.
Regarding the install directories, did you install dronecode system wide or locally?  If you installed locally, then you do need to tell the system where the includes are: https://sdk.dronecode.org/en/guide/toolchain.html#sdk_local_install
My understanding is that if you installed system wide you shouldn't have to.
IF you did and this is needed then yes, we will have to update a bunch of things, including test scripts and docs.

However there is no sudo ldconfig (which is used on Linux to update the linker cache) in Mac. What is the equivalent way to do it in Mac?

https://stackoverflow.com/questions/1451047/ldconfig-equivalent-in-mac-os-x

Please re-open if this is still a problem.


Introduce new deserialization framework upper layer's heroes

Contents
This PR:

Introduces ResultMetadataHolder, a versatile container for ResultMetadata, for use in QueryResult etc.

Introduces new lifetime-generic containers for Result:Rows response with varying degree of serialization/deserialization of metadata and rows:

RawMetadataAndRawRows,
DeserializedMetadataAndRawRows,

whose goal is to allow lazy deserialization of rows and deserializing metadata in the borrowed form (to reduce number of string allocations).

Introduces RawRowsLendingIterator (see description below).

Introduces two main heroes belonging to the upper layer of the new deserialization framework:

(new) QueryResult,
(new) RowIterator,

and prefixes their legacy counterparts with Legacy_.

Propagates new QueryResult and RowIterator down the internal code, converting them to legacy counterparts only when needed by the legacy Session's API.

Details
RawRowsLendingIterator
It is added in tandem with the already present (the low level, scylla_cql one, not the iterator.rs one!) RowIterator. The difference is that RawRowsLendingIterator owns the frame and the metadata, whereas RowIterator borrows it. The lending one is needed in session iterator interface, and the borrowing one is used with QueryResult.
Achieving old behaviour for dynamic deserialization purposes
If one still wants to deserialize arbitrary rows (as the old framework would do) and only then try to convert them dynamically to some proper types, this is doable by using the old Row struct as the target type, e.g.
let qr = session.query("SELECT a FROM ks.table")?; // new Session's API not yet implemented in this PR
if let Some(rows_deserializer) = qr.rows_deserializer()? {
  let rows = qr.rows::<Row>()?;
  for row in rows {
      // Type check and deserialization succeeds for any valid CQL type.

      // Do arbitrary dynamic magic with `row`.
  }
}

ColumnSpecs
The new ColumnSpecs struct is introduced that provides view over columns specification, but exposes only TableSpecView and ColumnSpecView structs instead of TableSpec and ColumnSpec, allowing us to modify the latter in the future without introducing breaking changes. One planned change is moving table_spec out of column_spec for reduced memory footprint (see #956), but as CQL protocol allows different table specs for different columns and we can't predict whether joins won't be introduced at some point in the future, we retain this flexibility by imposing a new layer of abstraction (the views).
New paging iterators
Instead of RowIterator (returning Rows) and TypedRowIterator (returning instances of the target type) both implementing Stream, now we have the following:

RawIterator

cannot implement Stream, because returns ColumnIterators that borrow from it,
provide type_check() and next() methods that can be used for low-level, manual deserialization (not recommended for ordinary users)
supports deserializing manually borrowed types (such as &str).

TypedRowIterator

created by calling into_typed::<TargetType>() on RawIterator,
type checks upon creation,
supports deserializing automatically borrowed types (such as &str),
does not implement Stream in order to support borrowed types.

TypedRowStream

created by calling into_stream() on TypedRowIterator,
implements Stream and hence does not support borrowed types.

TODO

#1057 is going to follow this, bringing new Session's deserialization API and migrating internals APIs, tests and examples to it.

Another step towards: https://github.com/scylladb/scylla-rust-driver/issues/462
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[x] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[x] I have provided docstrings for the public items that I want to introduce.
[ ] I have adjusted the documentation in ./docs/source/.
[x] I added appropriate Fixes: annotations to PR description.

v1.1: addressed @muzarski comments - thanks for them!

Marked as draft, because #1101 must be merged first, and then this PR will be adjusted to two lifetime parameters in the deserialization framework.

v1.2: Rebased on main.

v2.0:

rebased on #1101,
redesigned RowsDeserializer (and DeserializedMetadataAndRawRows) to better support both types of frame ownership: borrowing from QueryResult (QueryResult::rows_deserializer) or owning themselves (QueryResult::rows_deserializer_owned() and RawRowsLendingIterator in iterator framework). Now flexibility enabled by #1101 is exposed through the API layers.

Which could still be a problem if there is a lot of columns, right?

I don't know what this is about. GitHub displays it as an alone message without reasonable context.

Which could still be a problem if there is a lot of columns, right?

I don't know what this is about. GitHub displays it as an alone message without reasonable context.

That happens if, as a part of review, you respond in a discussion in some other review. In that case this comment is a response here: https://github.com/scylladb/scylla-rust-driver/pull/1093?notification_referrer_id=NT_kwDOAFrp5bMxMjg5NzkzNzk3Mjo1OTU4MTE3#discussion_r1814856912

Which could still be a problem if there is a lot of columns, right?

I don't know what this is about. GitHub displays it as an alone message without reasonable context.

That happens if, as a part of review, you respond in a discussion in some other review. In that case this comment is a response here: #1093

The link points me to the PR as a whole, so again no luck in determining the context.

v2.1: addressed another bunch of @muzarski comments.

v2.2: rebased on main.

Which could still be a problem if there is a lot of columns, right?

I don't know what this is about. GitHub displays it as an alone message without reasonable context.

That happens if, as a part of review, you respond in a discussion in some other review. In that case this comment is a response here: #1093

The link points me to the PR as a whole, so again no luck in determining the context.

I hate GitHub, nothing works well here.
This link currently works for me, does it work for you? https://github.com/scylladb/scylla-rust-driver/pull/1093#discussion_r1803499266

v3.0:

adopted @Lorak-mmk's idea to use yoke crate and gain flexible ownership over self-borrowed ResultMetadata. As a result, we have the best of two worlds:

when deserializing a Result:Rows response, ResultMetadata is always deserialized in a borrowed form!,
ResultMetadataHolder is now an owned type, so no lifetime bounds are incurred.
Thus there's no more need for dual RowsDeserializer.

RowsDeserializer is replaced with QueryRowsResult, convertible from QueryResult using to_rows_result(). QueryRowsResult exposes a superset of QueryResult's API.
as @Lorak-mmk requested, new abstractions in iterator.rs are named appropriately. No more iterators there, only streams - and even lending streams.

Huge credits to @Lorak-mmk for coming up with the idea, finding yoke and writing a PoC!

v3.1: responded to all comments but one about refraining from "heroes" term (TODO).

v3.1.1: addressed all comments.

v3.2:

addressed @Lorak-mmk another round of comments,
uncovered a severe problem about TypedRowLendingStream, which renders it unable to deserialize borrowed types. For details, see a new commit that contains a failing test of it.

This is no longer true now that metadata is always borrowed, right?

Actually, metadata is still always borrowed in scylla crate, unless the one cached in PreparedStatement is used.. Both RawRowLendingIterator and QueryRowsResult hold ResultMetadataHolder, so we get that feature thanks to yoke.

This is no longer true now that metadata is always borrowed, right?

Actually, metadata is still always borrowed in scylla crate, unless the one cached in PreparedStatement is used. Both RawRowLendingIterator and QueryRowsResult hold ResultMetadataHolder, so we get that feature thanks to yoke.

I don't understand how this comment relates to the discussion that my comment was posted in.

I don't understand how this comment relates to the discussion that my comment was posted in.

GitHub again displayed your comment as standalone, so I didn't know that it refers to any particular discussion :see_no_evil:

I don't understand how this comment relates to the discussion that my comment was posted in.

GitHub again displayed your comment as standalone, so I didn't know that it refers to any particular discussion 🙈

If you see that it is not possible to respond to a comment (there is no "Reply" text field) that means this comment was posted in some other thread which you need to find.

v3.2.1: exposed column_specs() and tracing_ids() on TypedRowStream too, similarly to TypedRowLendingStream.


Greenkeeper/prettier 1.17.1

:tada: This PR is included in version 1.30.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Switch to steamcmd/steamcmd:ubuntu-24

Move the ENV section ahead of the RUN sections in the Dockerfile to be able to use the preset PGID and PUID.
Move DEBIAN_FRONTEND=noninteractive to ENV section.
Solve for UID & GID 1000 already in use by removing ubuntu user & group. userdel handles both in one command.
Create steam group and user with PGID and PUID from ENV vars, default 1000.
Add tzdata package to insure Docker TZ environment variable functions. Without tzdata the container doesn't know how to apply timezone changes / offsets.

Hey @goofball222! Apologies for not reviewing this yet. I'm very hesitant with rebasing the image, after the slew of issues it caused before. I'll properly review this once I can find the time to adequately test it out :) Thanks!

Anecdotally, I've been running a server off a build of this Dockerfile since I submitted the PR. No issues so far that I've noticed. Server has been through several restarts, etc.
No worries, take your time on review and / or feel free to close this out. Just wanted to share my solution to the reported problems with UID/GID conflicts reported from the first go-round with the Ubuntu 24 base.

Thanks for waiting! I'm going to release this as v1.9.0 and see how we go :) Great stuff!


jestPlaywright is not defined

After following the basic setup, I am running into following issue:

Environment
node -v 12.15.0
Windows 10

Which version of jest-playwright have you installed? It was added in version 0.0.6.

Which version of jest-playwright have you installed? It was added in version 0.0.6.

Indeed bumping the version of jest-playwright from 0.0.5 to 0.0.6 solved the issue. Thanks much for the help!


Error inside createRouteAction not triggering ErrorBoundary

In the documentation, it's mentioned that if you don't explicitly handle an error on a route action then the error should bubble up to the nearest ErrorBoundary:

Given the following two components:
export function FormSection() {
  return (
    <ErrorBoundary fallback={() =>
Oh no! Error!
}>
      <TinyForm />
    </ErrorBoundary>
  );
}

export function TinyForm() {
  const [formState, { Form }] = createRouteAction(async (data: FormData) => {
    throw new Error("Something went wrong!");
  });

  return (
    <Form>
      Username:
      <input type="text" name="username" />
      <input type="submit" value="submit" disabled={formState.pending} />
    </Form>
  );
}

I would expect that the error being thrown in the route action bubbles up to the ErrorBoundary above, but it is not. The error is just swallowed by the routeAction. I can still manually handle errors using the recommended approach:
<Show when={formState.error}>

Oh no, there was an error: {formState.error.message}

</Show>

But ideally, I'd like to ensure that these hit the Error boundaries as expected so that I can ensure I'm catching unhandled errors inside of Sentry.
A sample app with a reproduction lives here

Yeah I think this was changed at some point before beta release and the docs need to update to reflect this. Relying on a property access made that very difficult, especially with things like multi-actions. The criteria to fickle. And almost never what you'd want since it would unmount the form most likely as it had to be tied to where the action was defined rather than to where it is used. Patterns used for Suspense didn't really make sense here.
At this point I think I'd say this is a documentation issue. We could try to revisit it but this is a very different type of primitive since it is basically ephemeral so it doesn't tie into the this is needed to render the UI pattern that made ErrorBoundaries a thing in the first place.


Fix doc_markdown lints in bevy_derive

#3457 adds the doc_markdown clippy lint, which checks doc comments to make sure code identifiers are escaped with backticks. This causes a lot of lint errors, so this is one of a number of PR's that will fix those lint errors one crate at a time.
This PR fixes lints in the bevy_derive crate.

I think this code is actually dead, let's remove it in #3490


Added extra cache to the cache list

hi . when i use config enabled(false) for useQuery , Requests will not be sent but in react query cache system , when mount component , add a request , In fact, there is no such request. please check

Please reopen with a codesandbox or example showing this error in action.


Casmmon 138 r1.0

Updating https://github.com/Cray-HPE/docs-csm/blob/release/1.0/operations/kubernetes/Restore_an_etcd_Cluster_from_a_Backup.md

I wonder why only two checks out of the 6 have run here.

I wonder why only two checks out of the 6 have run here.

Good question. @mtupitsyn any idea?

I rebased this and force pushed up the changes -- I want to see if it runs all of the checks this time.

Since @mantoanibroadwing hadn't touch this one in a while, I went ahead and fixed the linter complaints.

It will still need the backports that Kim mentioned above

/backport release/1.2 main

Looks like the backports will require manual intervention

A PR for release/1.2 and main is also needed.

Will create it for sure

@kimjensen-hpe is on vacation, otherwise we just need her stamp/re-review. We also need the backports created @mantoanibroadwing .

/backport release/1.2 release/1.3 main


Several issues fixed for basemap part

Added the custom scroll to the add-basemap-modal content (ticket: #11132).
Removed the re-render of the basemap carousel each time a category is selected.
Fixed a problem with the NASA date-picker, where a JS error appeared each time that tab was selected in the add-basemap-modal.
Fixed a problem with a padding in the WMS text input.

*I encourage the reviewer to check those problems in production and check the difference in the proper staging server later.
Things to test:

Check the stuff described in #11132.
Be sure the add-basemap-modal is working properly for each tab.
Check if basemap carousel is working properly after selecting a new category and a new custom map.
Check if WMS/WMTS panel works properly.

cc @oriolbx

Acceptance passing 🍔

I think this is already deployed but in the wrong board column. @MariaCheca can you confirm?


add Getter that bypasses internal nodes

Since verkle uses the path-based storage, it is possible to read the value of a leaf directly. We only need to read the intermediate nodes for updating the commitments... which can be done by a prefetcher.
This might not be useful in most cases as the snapshot is used, but there are several cases where this can make a difference:

when checking for FILL_COST
when the snapshot is missing

I am leaving here a summary of some side-channel discussion about this.
The way PBSS works is that each node in the tree has a deterministic key depending on where it lives in the tree. If a LeafNode has stem 0xAABBCCDDEE... then the key in the db could be 0xAA, 0xAABB, 0xAABBCC, or similar depending on the depth of its depth in the tree.
This means that at the API level, we can't fetch a leaf node directly without the client providing more information about the node's depth, which doesn't sound viable.
A potential option is that the PBSS key for a LeafNode is the full stem, which:

Might reduce disk churn when new internal nodes are created since leaf nodes have a stable key.
Require some extra logic to load nodes from internal nodes. To load "the next node", now you'd have two potential options depending on whether the child is an internal or leaf node (since their keys are encoded differently).

Option 1: do a double-lookup (which might have a perf hit)
Option 2: do some extra data encoding in internal nodes to signal which children are a leaf node (e.g: a bitlist) (which might have a storage overhead)

If reading a leaf node directly is helpful, as mentioned for FILL_COST, or when snapshots aren't available, we probably need to benchmark the mentioned options (or explore others).


MP4 Video not Showing in IE8

When playing MP4 videos in IE8, the video does not display even though you can hear the audio.
Environment: Windows XP / IE8 Standards Mode
This happens in the Flash portion of the player.

@alvincrespo Is this still an issue? If not can you close it please?

@alvincrespo Closing old issues. If you'd like to reopen this, let us know. Thanks!


Improve Kotlin 2.0 support

Kotlin 2.0.x rejects the current code:
Generated_Kotlin.kt:29:5 Property must be initialized, be final, or be abstract.

Initialize the property inline, with the value passed-in to the constructor, to address this.

@nil4 LGTM. I'm kind of surprised this is the only place the generated code is affected by this change.

@nil4 Since the error message mentions making the property final, did you try that? I guess it means that subclasses cannot override it which would make sense in this case.

No, I haven't tried final. This seemed like the minimal change necessary to unblock a specific use case, so I didn't looked further than that.
I see how final might be a better fit here, since subclasses aren't expected to override the property, but I'd leave that for a future enhancement.

_handle made final in https://github.com/royalapplications/beyondnet/commit/8bcc383b629fb0141babc8dc5f41fb5ad9948a48


Another UFS tidying up commit

Trying to consolidate harvey-specific code into sensible .c and .h files.
Reading is in progress, this is more prep work.
Signed-off-by: Graham MacDonald<EMAIL_ADDRESS>

Let me know if you're happy with the changes and I'll push.


Fix some erroneous recipes

This addresses items 1 and 3 from #19
Some additional changes were necessary:

I "sorted" the ingredients on each recipe. This makes it easier to find recipes that use the same ingredients.
All recipes that used Snowque had to be removed, as there were no valid recipes that created it, and I couldn't find one that did.
I added a new recipe for Waterjet, as there were no valid recipes that created it: Lawnjet + River = Waterjet
I needed a working scraper to verify which "duplicate ingredient" recipes were correct, so I fixed yours. It requires axios now, because I couldn't get the fetch API to order the headers correctly.

There are still some duplicate recipes remaining, but all of the duplicates with the wrong result are gone.

Thanks for your contribution! I'm currently busy, but will merge this in a few days :heart:. Please remove all changes in chunks, I will run chunk.js myself when this is ready to merge.

I've removed changes made by chunks.js, as requested

Thank you for your contribution!


Storage: [Do not merge] Delete old buckets and files

This is just the same that we did for Spanner, right? No need to review because you will close after?

Yes this is the same thing that we did for spanner.
Don't why it's failing with below error:
The following constructor parameters did not have matching fixture data: BucketFixture fixture

Closing this one.

That's what happens when there's an exception in the fixture.
The error is a 404 not found and the stack trace is:
  at Google.Apis.Requests.ClientServiceRequest`1.ParseResponse(HttpResponseMessage response) in C:\Apiary\2020-03-06.14-34-34\Src\Support\Google.Apis\Requests\ClientServiceRequest.cs:line 250
   at Google.Apis.Requests.ClientServiceRequest`1.Execute() in C:\Apiary\2020-03-06.14-34-34\Src\Support\Google.Apis\Requests\ClientServiceRequest.cs:line 174
   at Google.Api.Gax.Rest.RestPagedEnumerable`3.AsRawResponses()+MoveNext() in T:\src\github\gax-dotnet\releasebuild\Google.Api.Gax.Rest\RestPagedEnumerable.cs:line 50
   at System.Linq.Enumerable.SelectManySingleSelectorIterator`2.MoveNext()
   at GoogleCloudSamples.BucketFixture.CleanupBuckets() in /tmpfs/src/github/dotnet-docs-samples/storage/api/StorageTest/StorageTest.cs:line 118
   at GoogleCloudSamples.BucketFixture..ctor() in /tmpfs/src/github/dotnet-docs-samples/storage/api/StorageTest/StorageTest.cs:line 92

(It seems to be at BuckerFixture.CleanupBuckets())


If base64 contains a line break, it will be InvalidBytes

What did you implement:
A response from GutHub contains a line feed code.
But if you enter it directly into rust - base64 it will be InvalidBytes. So how about decoding after removing a line break?

I just ran into this issue as well. After doing some research, I believe this PR has the correct fix.
The base64 crate specifically does not handle whitespace stripping. In the release notes it says:

Remove whitespace stripping to simplify no_std support. No out of the box configs use it, and it's trivial to do yourself if needed: filter(|b| !b" \n\t\r\x0b\x0c".contains(b).

Unfortunately, filtering isn't optimal either since base64::decode operates on slices, not a reader.
I also tried using decode_allow_trailing_bits since it references https://infra.spec.whatwg.org/#forgiving-base64-decode, but that doesn't work either. The base64 crate simply doesn't skip over whitespace.

closing in favor of https://github.com/softprops/hubcaps/pull/215

Wow.


[ABW-2907] Change guarantees section layout

Description

changed guarantees section layout to match zeplin for pool tx previews
split TransactionAccountCard into smaller composables - no layout changes just moved them around

How to test

contribute to pool and validate new guarantees layout

Screenshot

PR submission checklist

[x] I have tested Estimated/Predicted pool contribution guarantees

Oh boy the conflicts will be difficult in this one.


[INSTALL]<EMAIL_ADDRESS>

Which software in the stack would you like installed?
scotch
What is the version/tag of the software?
7.0.4
What compilation options would you like set?
no changes
Installation timeframe: Would you like this package to be installed in an upcoming quarterly spack-stack release, or sooner?
spack-stack 1.5.0 release
Any other relevant information that we should know to correctly install the software??
None (7.0.4 provides bug fixes)

Yup, that's already included in #722 (will make the PR close the issue)

do we know if there are any changes to build settings that will need to be in scotch's package.py for 7.0.4?

None that I know of. It should just be bug fixes, so apart from no longer needing the MPI_Fint patch, I don't think it changes the build itself.

so these lines can go https://github.com/JCSDA/spack/blob/a7778813cd955229db7a8c4d48883610e3fe1f74/var/spack/repos/builtin/packages/scotch/package.py#L76-L77 ? what about the noarch variant?

No, those lines can stay, the patch just won't be applied to 7.0.4 because it's not needed. The noarch variant should stay; I seem to recall it related to compiler issues that as far as I know are not scotch's fault.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 844 ms

Resolved: Bofrost.lu performance has improved in ca34f22.


Trap receiver

resolve #2
For the following reasons, I changed some API from #2

Support for both IPv4 and IPv6
Support for community validation
Future expansion

Support for TCP transport
Support for trap of version 3

@mgenov I would like to get your feedback.

Overall: I think that things are looking really good. +2 about this.
The only think that bothers me is that there are several places where packet versioning is checked. For example, when TrapServer is initialised, it has
mp:        newMessageProcessing(V2c), // server explicitly says that it uses messages V2c

but latter in the handle func there is:
if v := msg.Version(); v == V2c {
}

so even server has decided what message processing to be used, it makes another versioning check.

The only think that bothers me is that there are several places where packet versioning is checked

That's true.
These is temporarily implementation. When support the other SNMP version, there is a need to be modified.
To be exact, it is necessary to be able to select the message-processing for the received message.

LGTM !


Override file_set_actor to retain japanese characters

Before:

After:

I've tested uploading files with Japanese filenames. "2019....xlsx" is uploaded before applying this PR and "2019忘年会_日程調整_02_DPFC.xlsx" is uploaded after applying it.

In my recognition, items already uploaded are not affected by this PR. Some PubMan items have Japanese filenames, so we will rename them later.

@nabeta you are correct, existing items are not affected as the label is set when files are added. It ought to be quite straightforward to create a rake task to run through existing file sets and decode the filenames. Let me know if you want me to have a look at that.


No Support For OkHttp (3.8.0) - Native Modules SDK Compatibility Issues

Description
Was trying to implement a Java SDK into my React Native Android project, however after adding the SDK into my build.gradle the project would not build because the Java SDK required okhttp 3.7.0 but React Native does not support that version. The motive for adding this SDK into the project was for exposing native modules that use the Java SDK. Suggest updating the version of okhttp React Native uses, this would help with compatibility with other SDK's. This issue was also filed for the Java SDK to support React Native, however it would make more sense for React Native to update, this issue could be affecting many SDK's and could preventing developers from making native modules to extend their React Native projects. The issue filed on the Java SDK can be found here.
When compiling the error will be:
Cause: com.android.dex.DexException: Multiple dex files define Lokhttp3/internal/ws/RealWebSocket$1;
Error:Execution failed for task ':app:transformClassesWithDexForDebug'.
> com.android.build.api.transform.TransformException: com.android.ide.common.process.ProcessException: java.util.concurrent.ExecutionException: java.lang.UnsupportedOperationException

Which is saying that there are two dependancies that require okhttp but different versions, therefore incompatible.
For your reference, the dependency tree of my project can be seen below:

Reproduction Steps and Sample Code
NOTE: Assuming you can create React Native applications from the command line using the react-native-cli if not, please look at here.

Create a new React Native Project: react-native init AwesomeProject

Open the Android Application and add the Watson Java SDK to AwesomeProject/android/app/build.gradle and make sure to sync Gradle:

dependencies {
    compile project(':react-native-tts')
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile "com.android.support:appcompat-v7:23.0.1"
    compile "com.facebook.react:react-native:+"  // From node_modules
    compile "com.ibm.watson.developer_cloud:speech-to-text:3.8.0" //add this line
}

Build the project, either through the CLI or using Android Studio.

When compiling the error will be:
Cause: com.android.dex.DexException: Multiple dex files define Lokhttp3/internal/ws/RealWebSocket$1;
Error:Execution failed for task ':app:transformClassesWithDexForDebug'.
> com.android.build.api.transform.TransformException: com.android.ide.common.process.ProcessException: java.util.concurrent.ExecutionException: java.lang.UnsupportedOperationException

NOTE: This example is using the Watson Java SDK, but the same error would occur if using a different SDK that needed a version of okhttp greater than what React Native uses.
Solution
Solution to this issue would be to update the version okhttp React Native uses to be greater than or equal to version 3.7.0.
Additional Information

React Native version: 0.44.0
Platform: Android
Development Operating System: macOS
Dev tools: Android Studio

It was very recently upgraded to v3.8.0: https://github.com/facebook/react-native/commit/93a1d592d61b229d3dafe57d1590d286312cc0bf
So master should have what you need.

@fkgozali Thank you for updating the version of okhttp, it now works. 👍

Will this break again with https://github.com/facebook/react-native/commit/3af9be52b272b767b480ed4f989ddfd6146a56f8 ?

@mikelambert 3af9be5 will indeed break this again. What is the reasoning for downgrading?

this was downgraded due to an issue that caused breakage to existing apps: https://github.com/square/okhttp/issues/3308
cc @emilsjolander in case there's more details to add

@fkgozali Thanks, when a fix is found it should be updated once again.

I find okhttp 3.8.1 is fixed that bug.
https://github.com/square/okhttp/blob/master/CHANGELOG.md

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


URL is erased during unsuccesfull request (error 404) (IDFGH-5406)

Environment
Module or chip used: [ESP32-WROVERKIT2]
IDF version (run git describe --tags to find it): ESP-IDF v4.2.1
Build System: [Eclipse]
Operating System: [ubuntu]
Using an IDE?: [Yes (Eclipse]
Power Supply: [external 3.3V]
Problem Description
URL address is changed during a unsuccesful request (404). During a succesfull request it stays the same.
This is a memory leak because i can no longer free the memory...
Expected Behavior
URL address remains the same, content remains the same
Actual Behavior
URL address changed, is now empty
Code to reproduce this issue
char buffer[10000] = { 0 }, *url;
asprintf(&url, "https://www.espressif.com/hi");
esp_http_client_config_t config = {
    .url = url,
    .timeout_ms 	= 10000,
    .event_handler = _http_event_handler,
.transport_type = HTTP_TRANSPORT_OVER_SSL,
    .user_data 		= buffer        			// Pass address of local buffer to get response
};
esp_http_client_handle_t client = esp_http_client_init(&config);
esp_err_t err = esp_http_client_perform(client);

But doing this works:
    esp_http_client_handle_t client = esp_http_client_init(&config);
    esp_http_client_set_method(client, HTTP_METHOD_GET);
    esp_err_t err = esp_http_client_perform(client);

Is there space for a 10k buffer in the task stack?

@Lisa999 esp_http_client will not modify the url you malloc. Maybe the url was covered. And for buffer, is it a temporary variable. Maybe auto-recycle when the function return.

@Lisa999 Closing as we suspect issue with test code here, if you are still facing problem please feel free to re-open adding more details with complete example project.


Need to remove "Server" section from Console Conditionally

Describe the issue:
Server section does not make sense for a cloud user so it should be configurable and should be able to disable in a cloud deployment
How to reproduce:

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

This is now fixed and cab be conditionally hidden via the following methods.
deployment.tom (Recommended for non-externalized Console)
[console.server]
enabled = false

deployment.config.json (File Path: repository/deployment/server/webapps/console)
{
  ...,
  "ui": {
     "features": {
            "server": {
                "disabledFeatures": [
                ],
                "enabled": false,
                "scopes": {
                    "read": [
                            "internal_governance_view"
                    ],
                    "delete": [
                    ],
                    "update": [
                            "internal_governance_update"
                    ],
                    "create": [
                    ]
                }
            },
            ...


Update sbt-sonatype to 3.9.7

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

Files still referring to the old version number
The following files still refer to the old version number (3.9.5).
You might want to review and update them manually.
project/Versions.scala

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

labels: sbt-plugin-update, semver-patch, old-version-remains

Superseded by #210.


Show error when we have scroll in both direction in flutter web

When I used the package in flutter web, I got this error in console. I see no issue in result.
══╡ EXCEPTION CAUGHT BY ANIMATION LIBRARY ╞═════════════════════════════════════════════════════════
The following assertion was thrown while notifying status listeners for AnimationController:
The provided ScrollController is currently attached to more than one ScrollPosition.
The Scrollbar requires a single ScrollPosition in order to be painted.
When the scrollbar is interactive, the associated Scrollable widgets must have unique
ScrollControllers. The provided ScrollController must be unique to a Scrollable widget.
When the exception was thrown, this was the stack:
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49      throw_
packages/flutter/src/widgets/scrollbar.dart 1508:9                                                                             <fn>
packages/flutter/src/widgets/scrollbar.dart 1531:14                                                                            [_debugCheckHasValidScrollPosition]
packages/flutter/src/widgets/scrollbar.dart 1445:14                                                                            [_validateInteractions]
packages/flutter/src/animation/listener_helpers.dart 233:27                                                                    notifyStatusListeners
packages/flutter/src/animation/animation_controller.dart 815:7                                                                 [_checkStatusChanged]
packages/flutter/src/animation/animation_controller.dart 749:5                                                                 [_startSimulation]
packages/flutter/src/animation/animation_controller.dart 612:12                                                                [_animateToInternal]
packages/flutter/src/animation/animation_controller.dart 461:12                                                                forward
packages/flutter/src/widgets/scrollbar.dart 1821:46                                                                            [_handleScrollNotification]
packages/flutter/src/widgets/notification_listener.dart 130:22                                                                 onNotification
packages/flutter/src/widgets/framework.dart 3078:18                                                                            dispatchNotification
packages/flutter/src/widgets/framework.dart 3081:13                                                                            dispatchNotification
packages/flutter/src/widgets/framework.dart 4375:24                                                                            dispatchNotification
packages/flutter/src/widgets/notification_listener.dart 60:13                                                                  dispatch
packages/flutter/src/widgets/scroll_activity.dart 94:90                                                                        dispatchScrollUpdateNotification
packages/flutter/src/widgets/scroll_position.dart 900:5                                                                        didUpdateScrollPositionBy
packages/flutter/src/widgets/scroll_position_with_single_context.dart 224:7                                                    pointerScroll
packages/flutter/src/widgets/scrollable.dart 722:16                                                                            [_handlePointerScroll]
packages/flutter/src/gestures/pointer_signal_resolver.dart 88:7                                                                resolve
packages/flutter/src/gestures/binding.dart 451:29                                                                              handleEvent
packages/flutter/src/gestures/binding.dart 425:14                                                                              dispatchEvent
packages/flutter/src/rendering/binding.dart 329:11                                                                             dispatchEvent
packages/flutter/src/gestures/binding.dart 380:7                                                                               [_handlePointerEventImmediately]
packages/flutter/src/gestures/binding.dart 344:5                                                                               handlePointerEvent
packages/flutter/src/gestures/binding.dart 302:7                                                                               [_flushPointerEventQueue]
packages/flutter/src/gestures/binding.dart 285:32                                                                              [_handlePointerDataPacket]
C:/b/s/w/ir/cache/builder/src/out/host_debug/flutter_web_sdk/lib/_engine/engine/platform_dispatcher.dart 1105:13               invoke1
C:/b/s/w/ir/cache/builder/src/out/host_debug/flutter_web_sdk/lib/_engine/engine/platform_dispatcher.dart 185:5                 invokeOnPointerDataPacket
C:/b/s/w/ir/cache/builder/src/out/host_debug/flutter_web_sdk/lib/_engine/engine/pointer_binding.dart 130:39                    [_onPointerData]
C:/b/s/w/ir/cache/builder/src/out/host_debug/flutter_web_sdk/lib/_engine/engine/pointer_binding.dart 295:14                    [_handleWheelEvent]
C:/b/s/w/ir/cache/builder/src/out/host_debug/flutter_web_sdk/lib/_engine/engine/pointer_binding.dart 559:7                     <fn>
C:/b/s/w/ir/cache/builder/src/out/host_debug/flutter_web_sdk/lib/_engine/engine/pointer_binding.dart 284:84                    <fn>
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 334:14  _checkAndCall
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 339:39  dcall
The AnimationController notifying status listeners was:
  AnimationController#a8507(▶ 0.000)
════════════════════════════════════════════════════════════════════════════════════════════════════
2
Another exception was thrown: The provided ScrollController is currently attached to more than one ScrollPosition.

Well, I add a new ScrollController for my vertical scroller (not improverd scroll bar) and the errors gone :).


Try innovating the design of navigation bar.

Hey Contributors, try to innovate the navigation bar design.

Like can you give any suggestions what all you would like to see
so we can further elaborate the thing

Hay @jheel05 I would like to work on this issue, can you assign this task to me?

I would like to work on this. Please assign it to me @jheel05

@radzhiv25 its totally on your creativity, all you have to do is make the navigation bar look beautiful as well as simple and subtle.

@radzhiv25 would you like to work on it?

Sure

Hello, I am a GSSOC participant and I would like to work on this, Can you please assign this to me :)

@AnweshaRoses sure

if this issue is still open I want to work on this issue. can you please assign me this issue? for GSSoC

Hi sorry for the delay, I had my end sems going on. Gimme around 5 days ill start working on it.

Hi,
I would like to work on this issue as I am confident that I can effectively address the problem. With my experience and skills, I believe I can provide a valuable solution. I am eager to contribute to the project and make meaningful improvements.
Please assign this issue to me. Thank you for considering my request!


Jakarta EE10/MyFaces 4

Just curious if you had plans to bump from 2.3-next to 4 on this project?
I noticed that MyFaces 4 RC1 was published to the maven repos but I would think that https://github.com/quarkusio/quarkus/issues/21574 will remain a blocker. Beyond that issue opened 21-November I hadn't read anything else about the plans at Quarkus regarding Jakarta EE; Faces in particular.
Also, do you anticipate that the myfaces-quarkus extension will flip out of experimental status before or after that namespace/upgrade from javax to jakarta? Best guess?

These are great questions.
Quarkus control is out of my hands so when quarkus updates from javax to Jakarta specs is anybody guess
Once that happens then MyFaces quarkus extension can be updated for Jakarta. And then finally this demo would be updated.
However the Quarkus project isn't really concerned about EE specs or Faces. It's thanks to the amazing team at MyFaces that it even has support.   But I will say that I personally think it's an impressive way to deploy a container JSF app!

I agree.  JSF fits very well.
My current project still has a way to go but developing on Quarkus using JSF has been pleasant. Actually, it was a little rough to start. THEN I found this/your repository. I would certainly say that anyone looking to use Quarkus and build the front with JSF should have no real issue... just stop here first. :)
Smooth sailing ever since. So, thank you, Sir!

agreed!  Glad to see my project here was useful to some people!

@gesker also I just released OmniFaces on the official Quarkiverse so you may want to stop using that AdminFaces plugin and use:
<dependency>
            <groupId>io.quarkiverse.omnifaces</groupId>
            <artifactId>quarkus-omnifaces</artifactId>
            <version>3.0.0</version>
        </dependency>

https://github.com/quarkiverse/quarkus-omnifaces

I've been following your work and its impressive. And as I love everything JSF and Quarkus,  i did implement a huge project now going to production. Although I did implement using AdminFaces, I will give a try on your omnifaces.

you may get an EL Expression error if you do just add this and I fixing in 3.0.1 release.
        <dependency>
            <groupId>org.glassfish</groupId>
            <artifactId>jakarta.el</artifactId>
            <version>4.0.2</version>
        </dependency>

See: https://github.com/apache/myfaces/pull/458

https://quarkus.io/blog/road-to-quarkus-3/
Figured I'd add this here just in case you didn't see it @melloware. I did ask on that post about Faces coming over from the Jakarta project, too, along with the name space change.

Yep I have a prototype working with MyFaces 4 and OmniFaces 4.  I have been fixing a few bugs and reported a couple to Quarkus.

Linked to: https://github.com/quarkiverse/quarkus-primefaces/issues/16

This one is next on my list.

@gesker all committed!   I have not tried GraalVM Native mode yet but at leats JVM is working on Myfaces 4.0.0 Quarkus 3.

Opened a couple of MyFaces tickets. Also was reported Hot swap deployment is not working in Dev mode and I confirmed that as well as broken.


Named query support for JPA repositories

Related problem
Currently, JPA repositories support two types of queries:

Inherited through JpaRepository (using delegates for implementation)
Using @Query annotations on the method
However, this does not support @NamedQuery which can be added to entity classes. This is already registered to Hibernate, but not used by Hartshorn.

Requested feature
Add support for named queries. This should be third in line (after delegation and @Query lookup), and should be looked up based on the entity type, followed by the method name. For example:
@Service
@DataSource("users")
public interface UserRepository extends JpaRepository<User, Long> {
    List<User> findAdults();
}

Should require a named query called User.findAdults, which would be defined on the User type:
@Entity
@NamedQuery(name = "User.findAdults", query = "select u from User u where u.age >= 18")
public class User { ... }

Implementation note: this should be expandable in the future, so preferably this should be handled through different strategies which can be registered to a context.

ProxyDelegationQueryStrategy
JakartaAnnotationQueryStrategy
EntityNamedQueryStrategy


jenkins autorelease of ontology files stuck?

http://viewvc.geneontology.org/viewvc/GO-SVN/trunk/ontology/
The last autorelease of jenkins was 10/29.
Log of /trunk/ontology/go-basic.obo
Revision 36633 - (view) (download) (annotate) - [select for diffs]
Modified Sat Oct 29 12:56:21 2016 UTC (4 days, 3 hours ago) by jenkins
File length: 32906077 byte(s)
Diff to previous 36605
auto-release
There have been successful commits to the editors version since then.  How come no public releases since 10/29?
http://geneontology.org/ontology/go-basic.obo
Can someone please check?

@cmungall @kltm : please read

Hi @tberardini - there was an issue with the build which @cmungall is fixing. See https://github.com/geneontology/go-ontology/issues/12775

Ok, I noticed that our import reports of the GO file has showed no changes in the past few days when I know that we've been editing successfully.

Release made it through, http://build.berkeleybop.org/job/release-go/132/


Gopro wifi

Some of this is also in another PR. What I did:

listener on commands.next so I can get notified when a waypoint is reached
camera class that connects to a GoPro3 over wifi (more cameras to come), and syncs time
shutter() method that engages the shutter and logs a geotag.

@peterbarker can you do some review of the APM interfaces and overall architecture plz?
@hamishwillee do you have any pointers for documentation?


Added Date format commonly found in germany

Hey, so for my usecase I wanted the library to also parse the date format commonly used in germany (and I think some other european countries).
For example:
17.07.2020
So here is a pullrequest if you want to officially support it

@N-Schaef Thanks!


First phase/step of the auth flow implementation. Also collapse the r…

…etry behaviors in to the same thing. We perform the docker style auth flow (used by all service providers it seems), right now however we are not including support for the credentials helpers to lookup username/password combos. But this is enough to now authenticate against docker hub anonymously as it requires to fetch anything
This does make some of the request semantics a bit more complex in the simplest cases as we are handling the retries, redirects, authentication in the one place.
There is an extra parameter context as it seemed like the best way to be able to make state from a wrapping method into the passed async lambda that might be called many times.
Tested this manually doing queries from docker hub and it worked.

I believe it should be a case of shelling out to the auth helpers of docker to enable full auth capabilities, but if this seems ok i'll try add that next since need to connect to azure's ones authenticated too

Updated for all the comments I think. Thanks!


update sinks on global config change

Looks good. The websocket receiver also uses the cluster name and account id. If we're already fixing it, might be also good to create a new receiver if one of those change. Checkout ConfigLoader.__reload_receiver(..)

Ill do that in a new PR


Best practice for accessing Values/OneOrMany objects?

Where 'album' is MusicAlbum, and Author is set as a Person.
Is there a better way to access the author name than this?
album.Author.Value.Value2.FirstOrDefault().Name.Value.FirstOrDefault()
Thanks!

I've just checked in some code that will allow you do do this:
List<Person> people = album.Author;

This is using implicit conversion operators. Thanks for the idea.


static get request to /undefined when first opening launcher and every time after sending message

To repoduce:

open Demo page
open console
click launcher chat icon
: check log
write message then press send button
:check log

on local dev. The error pointing to MessageList.vue

nope sorry, i just ignore it

nope sorry, i just ignore it

Thanks for the response!


error etl oracle to postgres

System information:

Operating system windows 10
DBeaver version 21.x and 22.x

Connection FROM specification:

Database Oracle spatial
version 12c (enterprise release <IP_ADDRESS>.0 64 bit)
Driver name: oracle.jdbc.OracleDriver (com.oracle.database.jdbc:ojdbc8:RELEASE[<IP_ADDRESS>])
Do you use tunnels or proxies (SSH, SOCKS, etc)? NO

Connection TO specification:

Database PostgreSQl 13.6 on Ubuntu 64 bit
extension PostGIS 3.x
Driver name: org.postgresql.Driver (org.postgrsql:postgresql:RELEASE[42.2.3.jre7]
Do you use tunnels or proxies (SSH, SOCKS, etc)? NO

Describe the problem you're observing:
Exporting spatial table from Oracle to PostgreSQL, geometry column ON Postgres DB are invalid.
Export is OK with DBeaver version 7.x

Hello @fburelli
Can you provide a source table DDL?
And screen of Columns mapping dialog?

Also, what do you mean about "geometry column ON Postgres DB are invalid"?
Please check log files.

Column table source and destination are the same. Field mapping is one 2 one for completed data export.
Geometry format on destination table, appear with list of coordinates but not correctly formatted.
I'm sorry but don't have screenshot becouse i have downgrade software without save screen.
Table layout

Table ddl

DDL text:
CREATE TABLE "SIVEP"."SVP_ZONA"
(	"ID" NUMBER(10,0) NOT NULL ENABLE,
"DESCRIZIONE" VARCHAR2(50 CHAR) NOT NULL ENABLE,
"GEOM" "SDO_GEOMETRY",
"RECAPITO" VARCHAR2(50 CHAR),
"REFERENTE" VARCHAR2(50 CHAR),
"DATA_INIZIO" TIMESTAMP (6) DEFAULT SYSDATE,
"DATA_FINE" TIMESTAMP (6) DEFAULT TO_DATE('2999-12-31 00:00:00', 'yyyy-mm-dd hh24:mi:ss'),
"ULTIMA_VERSIONE" NUMBER(1,0) DEFAULT 0,
PRIMARY KEY ("ID")
USING INDEX PCTFREE 10 INITRANS 2 MAXTRANS 255 COMPUTE STATISTICS
STORAGE(INITIAL 65536 NEXT 1048576 MINEXTENTS 1 MAXEXTENTS<PHONE_NUMBER>
PCTINCREASE 0 FREELISTS 1 FREELIST GROUPS 1
BUFFER_POOL DEFAULT FLASH_CACHE DEFAULT CELL_FLASH_CACHE DEFAULT)
TABLESPACE "TB_SIVEP"  ENABLE
) SEGMENT CREATION IMMEDIATE
PCTFREE 10 PCTUSED 40 INITRANS 1 MAXTRANS 255
NOCOMPRESS LOGGING
STORAGE(INITIAL 65536 NEXT 1048576 MINEXTENTS 1 MAXEXTENTS<PHONE_NUMBER>
PCTINCREASE 0 FREELISTS 1 FREELIST GROUPS 1
BUFFER_POOL DEFAULT FLASH_CACHE DEFAULT CELL_FLASH_CACHE DEFAULT)
TABLESPACE "TB_SIVEP"
VARRAY "GEOM"."SDO_ELEM_INFO" STORE AS SECUREFILE LOB
( TABLESPACE "TB_SIVEP" ENABLE STORAGE IN ROW CHUNK 8192
CACHE  NOCOMPRESS  KEEP_DUPLICATES
STORAGE(INITIAL 106496 NEXT 1048576 MINEXTENTS 1 MAXEXTENTS<PHONE_NUMBER>
PCTINCREASE 0
BUFFER_POOL DEFAULT FLASH_CACHE DEFAULT CELL_FLASH_CACHE DEFAULT))
VARRAY "GEOM"."SDO_ORDINATES" STORE AS SECUREFILE LOB
( TABLESPACE "TB_SIVEP" ENABLE STORAGE IN ROW CHUNK 8192
CACHE  NOCOMPRESS  KEEP_DUPLICATES
STORAGE(INITIAL 106496 NEXT 1048576 MINEXTENTS 1 MAXEXTENTS<PHONE_NUMBER>
PCTINCREASE 0
BUFFER_POOL DEFAULT FLASH_CACHE DEFAULT CELL_FLASH_CACHE DEFAULT)) ;

this is error from the screen.

this is the csv export with erro
SVP_ZONA_202204211659.csv
r

There has been no update on the issue for a long time.
If it is still actual - feel free to reopen it with additional info.


Speed up beaker wasm build on Mac Silicon

workspace-optimizer could be much faster. but we are using the amd64 type always:
Unable to find image 'cosmwasm/workspace-optimizer:0.12.6' locally
0.12.6: Pulling from cosmwasm/workspace-optimizer
...
Status: Downloaded newer image for cosmwasm/workspace-optimizer:0.12.6
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

beaker could detect the local architecture and select between workspace-optimizer and https://hub.docker.com/r/cosmwasm/workspace-optimizer-arm64/tags. Alternately, we would be happy for a PR to build multi-architecture docker images, but that requires some other tooling, like https://github.com/CosmWasm/rust-optimizer/issues/67 first

Never mind. After writing this, I found the -a option... https://github.com/osmosis-labs/beaker/blob/main/docs/commands/beaker_wasm.md#beaker-wasm-build


fail to read GISTIC file

Hello, I failed to read GISTIC file, could you help me solve the problem? Thanks a lot.
laml.gistic = readGistic(gisticAllLesionsFile = 'C:/Users/Meredith/Desktop/maftools/all_lesions.conf_90.txt',gisticAmpGenesFile = 'C:/Users/Meredith/Desktop/maftools/amp_genes.conf_90.txt', gisticDelGenesFile = 'C:/Users/Meredith/Desktop/del_genes.conf_90.txt', gisticScoresFile = 'C:/Users/Meredith/Desktop/maftools/scores.gistic', isTCGA = TRUE)

-Processing Gistic files..
--Processing amp_genes.conf_90.txt
--Processing del_genes.conf_90.txt
--Processing scores.gistic
--Summarizing by samples
Error in `levels<-`(`*tmp*`, value = as.character(levels)) :
  factor level [61] is duplicated

Session info
R version 3.6.2 (2019-12-12)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 18362)

Matrix products: default

locale:
[1] LC_COLLATE=Chinese (Simplified)_China.936  LC_CTYPE=Chinese (Simplified)_China.936    LC_MONETARY=Chinese (Simplified)_China.936
[4] LC_NUMERIC=C                               LC_TIME=Chinese (Simplified)_China.936

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] maftools_2.2.10

loaded via a namespace (and not attached):
 [1] compiler_3.6.2     Matrix_1.2-18      tools_3.6.2        RColorBrewer_1.1-2 survival_3.1-12    Rcpp_<IP_ADDRESS>
 [7] tinytex_0.23       wordcloud_2.6      splines_3.6.2      grid_3.6.2         data.table_1.12.8  xfun_0.14
[13] lattice_0.20-38

here's the files
maftools.zip

Hello,
Thank you for reporting. I have fixed it. Please try install from GitHub for patched version.
Also, please note that setting isTCGA=TRUE truncates sample names to first twelve characters - meaning primary samples and their metastatic counterparts (or from other sources) will all be collapsed into one single sample type.

THANKS!


Add pre-commit hook to automatically keep track of dependencies

Fixes #19

Sorry for not answering this for so long.
Is there a way that we can have this on the repo but at the same time make it opt-in? As in, we allow these configuration to be there but there is another small change that needs to be done to get it there to run?
I don't want to generate these files every time that I commit as I don't use pip for lock files myself so I don't want this files sitting around.
Does this make sense?

FYI from experience of this hook applied to the Datta lab, it can create some pretty consistent and nasty merge conflicts especially when multiple developers contribute to the same common package (can't see them anymore because Paul closed them, but https://github.com/catalystneuro/datta-lab-to-nwb/pull/71 and https://github.com/catalystneuro/datta-lab-to-nwb/pull/70 were fresh commits from main at the time, from different user on a different platform, with only minor changes performed in each)
In general I would say the proper solution to this situation is to just use poetry or a platform-specific conda environment files (Dendro uses the former, GUIDE uses the latter)

@h-mayorquin how about this?
Also, it has been useful for me, and really not much of a burden.

That seems like a simple solution to me.
@CodyCBakerPhD would you be OK with it? That would facilitate Paul's workflow for his solo projects and not get in the way of others that don't want to use it.
Btw, I personally use rye for this as I never liked the poetry workflow. But there seems to be a new [push recently to standarize lock file]s(https://discuss.python.org/t/lock-files-again-but-this-time-w-sdists/46593). Fingers cross it works.

I'm still not convinced why we should include it in the general cookie-cutter if it's just for one person, and if everyone else has a preferred way of doing it - surely Paul can just add it himself to any of his projects?
Also pip freeze is insufficient in conda-dependent environments (NWB Benchmarks, GUIDE) so I also don't know if we'd want to recommend this generally either

Cool, I'll have to check out rye. Also, I proposed this for the cookie cutter because I felt like it was a handy tool worth sharing, but if no one else is going to use it because everyone has their own preferred way of keeping track of dependencies then I am more than happy to just add it to each of my projects.

I was reading up on rye as well as the package guide that ben mentioned, and I found a nice Venn diagram showing all of the different python packaging tools and their various overlapping roles: diagram.
Basically, what I'm using right now is Conda for environment management and python version management, pip for package management, setuptools for package building, and twine for package publishing. That sounds like a lot of tools, but the cookie cutter already has automated github actions for package building and package publishing, so I (so far) never have needed to interact with those. And since I already know the syntax for Conda and pip, I don't really think it will be worth my time to learn rye (though it is cool that it does all the jobs in the diagram). I also prefer to play it fast and loose with my dependencies in a given environment. For example, if I'm making changes to the photometry extension for my photometry interface and I want to change the installation location from pypi to local to a github branch/commit and then back to pypi, it's a bit tedious to have to constantly be updating the requirements file (or pyproject.toml in the case of rye) and then generate a new lock file and then reinstall from the new lock. In contrast, with this hook I can freely change my installation of ndx-photometry directly using pip as many times as I like, and there's a faithful record in case I ever run into any dependency related bugs.
With all that being said, it doesn't look like anybody else is particularly interested in this hook, so I think we can just close this PR and continue with everyone using their own preferred dependency management solution.

I also prefer to play it fast and loose with my dependencies in a given environment. For example, if I'm making changes to the photometry extension for my photometry interface and I want to change the installation location from pypi to local to a github branch/commit and then back to pypi, it's a bit tedious to have to constantly be updating the requirements file (or pyproject.toml in the case of rye) and then generate a new lock file and then reinstall from the new lock. In contrast, with this hook I can freely change my installation of ndx-photometry directly using pip as many times as I like, and there's a faithful record in case I ever run into any dependency related bugs.

Interesting. Would using a different environment for local vs remote installation work in this case or is that more work?

Interesting. Would using a different environment for local vs remote installation work in this case or is that more work?

I think that would just end up being more work tbh.


error: externally-managed-environment

error: externally-managed-environment
× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
python3-xyz, where xyz is the package you are trying to
install.
If you wish to install a non-Debian-packaged Python package,
create a virtual environment using python3 -m venv path/to/venv.
Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
sure you have python3-full installed.

If you wish to install a non-Debian packaged Python application,
it may be easiest to use pipx install xyz, which will manage a
virtual environment for you. Make sure you have pipx installed.

See /usr/share/doc/python3.11/README.venv for more information.

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.

I tried reinstalling all python by installing python2-minimal python2 python-is-python3 2to3 and python3-full. Then I install  pipx. It resolvec


Fix double parenting.

The PushChildren command was always setting the Parent and PreviousParent to the same value which meant that a break (see ~line 31 of parent_update_system) preventing PreviousParent removal from executing would go off. This meant that if an entity was re-parented, it wasn't removed from its old parent's Children list.
If this is resolved we are not immediately breaking out of parent_update_system each loop. The debug_assert! begins going off - we've already performed the additions (see deleted code).
This change does the following:

Update PushChildren to properly take the Parent and set it as the PreviousParent, if it exists. Otherwise, it follows the previous behavior.
Removes the duplicate - and more complicated - Children update behavior in hierarchy_maintenance_system.

I'm curious, also, is this the wrong direction? It seems like we might want to limit PushChildren to solely update the Parent and PreviousParent components, leaving management of Children to hierarchy_maintenance_system. I did try this approach, but it did not function correctly and I wasn't making much progress debugging it. At the same time, re-parenting was completely broken and hierarchy_maintenance_system wasn't doing much, so this seems at least like a step in the right direction if that's the goal.

Also, as a note, I changed the test to use Commands as I don't expect we can provide any sort of contract to people who are fiddling with the insides of the parent/child relationships.

I've been working on some related changes (which got sidetracked due to my ECS V2 work). The idea is to make parenting completely transactional (children can only be added via World/Command builders or newly added HierarchyQuries), which means we don't need to check for duplicates, use a "hierarchy maintenance system", or maintain a PreviousParent component.
https://github.com/cart/bevy/tree/hierarchy/crates/bevy_transform
I think I want to wait and see how that experiment pans out. I'm curious to hear what you think!

(it is very much a WIP at this point)

That looks pretty cool, looks like it would solve the issue. I'm a fan of the solution for making parenting operations transactional and better change tracking will solve the transform side of things.
This PR might be worth merging in the short term just to fix parenting in cases where people re-parent, I'm definitely going to be using it locally since it is broken otherwise. Depends on when you're planning on getting back to the transforms.
I wouldn't mind helping out there a bit, but the Query code looks kind of like magic to me and I don't really understand how we are guaranteeing that we have singular write access to the two entities in the query.
Thanks for taking a look, cart!

As the carts hierarchy changes were ultimately sidelined and the 0.5 release is imminante, I think this PR should be updated, to include it in 0.5 .

As the carts hierarchy changes were ultimately sidelined and the 0.5 release is imminante, I think this PR should be updated, to include it in 0.5 .

I agree: this is a good stop-gap fix for the release.

Closing this for now in favor of a future overhaul.


Visual Studio Start Up

I have just bought the Pro version.  Is there any quick start Visual Studio GitHub project that I can download and quickly understand (and apply) CoreUI?  I am having a hard time identifying and downloading all the packages and components and npm packages. I still haven't have any success getting CoreUI to run on Visual Studio. Am begining to think this is a bad decision. It will be very useful if you have a git project I can just download and see how it works. Thanks

Apparently, there is one already at http://github.com/mrholek/CoreUI-Net

see also:
http://www.coderepo.blog/posts/building-elegant-applications-aspnet-core-mvc-2.1-coreui-2-bootstrap-4/


Networking! 🌐

Networking

Originally posted by @KookaburraShell in https://github.com/AZProductions/Kookaburra/issues/16#issuecomment-855238544

[ ] import net
[ ] binding?

HTTPS Web client?
TCP
UCP?

All feature requests are now on Jira.


out of bound memory access caused undefined behaviour instead of error

Test Case
I've uploaded the code to https://github.com/PureWhiteWu/wasmtime_ub_demo.
Steps to Reproduce
I tried to access some out of bound memory in my rust code using unsafe, and found that it didn't report error in some cases. Instead, it caused UB.
You may clone https://github.com/PureWhiteWu/wasmtime_ub_demo and just call run.sh to execute the test cases.
Expected Results
For tests cases 1m_minus_two_i32, 1m_plus_one, 1m_plus_64k, 1m_plus_64k_plus_one, 2m, should fail with out of bounds memory access.
Actual Results
wasmtime: compiling using cranelift...
wasmtime: running cranelift version...
wasmtime 64k cranelift:
0
wasmtime 64k_plus_one cranelift:
0
wasmtime 1m_minus_four_i32 cranelift:
0
wasmtime 1m_minus_two_i32 cranelift:
-201326592
wasmtime 1m cranelift:
0
wasmtime 1m_plus_one cranelift:
244
wasmtime 1m_plus_64k cranelift:
0 (note: this may change to a random number during different running)
wasmtime 1m_plus_64k_plus_one cranelift:
Error: failed to run main module `target/wasm32-wasi/release/1m_plus_64k_plus_one.wasmtime`

Caused by:
    0: failed to invoke command default
    1: wasm trap: out of bounds memory access
       wasm backtrace:
           0:  0x2b8 - <unknown>!_1m_plus_64k_plus_one::main::h16960bb87b178e85
           1:  0x293 - <unknown>!std::sys_common::backtrace::__rust_begin_short_backtrace::h048d11a00c6a35d0
           2: 0x18ba - <unknown>!__original_main
           3:  0x279 - <unknown>!_start
           4: 0xb90c - <unknown>!_start.command_export
       note: using the `WASMTIME_BACKTRACE_DETAILS=1` environment variable to may show more debugging information

wasmtime 2m cranelift:
Error: failed to run main module `target/wasm32-wasi/release/2m.wasmtime`

Caused by:
    0: failed to invoke command default
    1: wasm trap: out of bounds memory access
       wasm backtrace:
           0:  0x2b8 - <unknown>!_2m::main::h9f0049353c958a74
           1:  0x293 - <unknown>!std::sys_common::backtrace::__rust_begin_short_backtrace::h6d0aa0bccba5db1c
           2: 0x18ba - <unknown>!__original_main
           3:  0x279 - <unknown>!_start
           4: 0xb90c - <unknown>!_start.command_export
       note: using the `WASMTIME_BACKTRACE_DETAILS=1` environment variable to may show more debugging information

wasmtime end...

1m_minus_two_i32 reads from 1m-2 to 1m+2 (an i32), which should failed with out of bounds memory access, but seems it succeed with undefined behaviour.
1m_plus_one, 1m_plus_64k reads only a byte, and should fail with out of bounds memory access, but seems reads unpredictable values too.
Versions and Environment
Wasmtime version or commit: wasmtime 0.31.0
Operating system: Linux 5.4.56 Debian 10
Architecture: amd64 / AMD EPYC 7642 48-Core Processor

I just checked 1m_minus_two_i32. The compiled wasm file specifies a linear memory of 17 pages. As a wasm page is 64k, this means that the first 1088KiB of the linear memory is fully readable and writable. This is exactly 1MiB + 64KiB. 1m_minus_two_i32 and 1m_plus_one should thus succeed. 1m_plus_64k succeeding would be an off-by-one error in the bounds check of wasmtime if I did my math correctly. 1m_plus_64k_plus_one and 2m give an out of bounds error as intended.
All examples invoke UB at the Rust level as you dereference a pointer created from an arbitrary integer, but this does not translate to the wasm level. 1m_plus_64k_plus_one, 2m and I think 1m_plus_64k are defined as trapping, but the rest is fully defined at wasm level.

Thanks for the report but I  believe that this is all working as intended. Looking at the modules themselves it appears that most of them have a memory with 17 pages of memory, which is 1MB plus 64k, which I believe explains why all the tests exhibit the behavior that they do. Otherwise though I believe everything is in working order with the wasm spec, so I'm going to close this.

 This is because I invoke UB in Rust not wasmtime.
Thanks for your reply!


Numerical Input without incrementing and decrementing button

Description
There is a case that input happens to be numerical value but not strictly value that can not be increase or decrease (more detailed explanation in MDN).
But I couldn't find a way to pass through attrs like inputmode="numeric" or pattern="\d*" into Input inner <input> element.
Is there a workaround? Or could this become a feature?
Additional context
In my case I just want to prevent users input characters other than numbers (of cause without that native (maybe ugly?) incrementing and decrementing button).

All attributes are bound to the <input> automatically so this work: https://github.com/nuxt/ui/blob/3411b891916d82df91550344a7ed36f7e9b8df01/src/runtime/components/forms/Input.vue#L13

All attributes are bound to the <input> automatically so this work:
https://github.com/nuxt/ui/blob/3411b891916d82df91550344a7ed36f7e9b8df01/src/runtime/components/forms/Input.vue#L13

Thanks for your intuitive answer!


User accounts for vendors

Problem:
Currently, you can enter the email address of any registered vendor and manage their subscriptions. Theoretically, a bad actor could unsubscribe all of their competitors from RFP notifications.
Solution:
In order to re-authenticate vendors, send a link via email that must be clicked in order to proceed.

I think I'd prefer to go with full user accounts for vendors. We'll have to:

Add a new vendor role to the roles table
Remove any redundant columns from the vendor table
Link the users and vendor tables with a 1-1 association

Add a password field to beacon/signup
Remove signup form from beacon/opportunities
When viewing opportunity at /beacon/opportunities/72, if user is logged in, display "follow this opportunity" button. If user is not logged in, "follow this opportunity" should link to signup page.

Add fields from signup form to /manage so vendors can update their profile.


Persist Lightning payment data in medici transactions

Description
This PR is to add relevant lightning payment info from lnd to medici transactions in the database (see #605).
A new LnPayments collection was set up to persist these entries and they can be uniquely referenced by their paymentHash property to tie to transactions we have in medici.
The new payment property in the database looks like:

Latest commits here implement a separate LnPayments collection with a _payment property in the relevant transactions in the medici_transactions collection that has an ObjectId object with the id of the payment object in LnPayments
For migration, I'm planning to query medici transactions in our db for ones without the _payment property and to use the hash on these to ping lnd for payment info to populate LnPayments collection.

Latest commits here implement a separate LnPayments collection with a _payment property in the relevant transactions in the medici_transactions collection that has an ObjectId object with the id of the payment object in LnPayments
For migration, I'm planning to query medici transactions in our db for ones without the _payment property and to use the hash on these to ping lnd for payment info to populate LnPayments collection.
Planning to do this in a follow-up PR.

then a follow-up to this follow-up PR can be that we delete the whole payment history, instead of just the payment failure. this would help reduce lnd bbolt size

Temporarily reverting this to draft until I add the integration tests I'm working on

Added tests and reverted the extra property were adding to medici transactions. This should be good to go for review again.

force-push to tip of main

Moving to draft to test issue in update-payments flow

Ready for review again.
I changed the structure of the persisted data so that the status, paymentHash and paymentRequest properties are at the top-level, and the rest of data is placed in a paymentDetails object that gets updated if the payment moves from pending to settled, but doesn't get erased if the payment eventually fails.

@vindard this needs a rebase

@jireva can you run TEST="01|02" make reset-integration and take a look at the objects in new lnpayments collection in mongo to see if what we're doing here to persist payments looks ok or can be improved?

the second row doesn't have a createdAt for some reason:

one last set of comments, but looks good for me otherwise

the second row doesn't have a createdAt for some reason:

It seems that when a payment fails, sometimes paymentDetails is set to null while other times it get's created but is missing confirmedAt and secret.
If you dump the lnpayments collection:
$ mongoexport --uri="mongodb://testGaloy:password@localhost/galoy" --collection=lnpayments --jsonArray > lnpayments

You can use jq to compare two failed payments, one of which is missing the paymentDeatils field:
$ jq '.[0,7]' lnpayments
{
  "_id": {
    "$oid": "61a7c893c7f362f7473a08a1"
  },
  "paymentHash": "ded83fcb8cd2944a715adde74d275a1e8d141212207c0581943057f86c57b633",
  "__v": 0,
  "paymentDetails": null,
  "paymentRequest": "lnbcrt15m1ps60jynpp5mmvrljuv622y5u26mhn56f66r6x3gysjyp7qtqv5xptlsmzhkcesdqqcqzpuxqr23ssp5vkar0rfxagl6msvn84cnmjrsjc7j7vtf397qcu952tr8c6y0e5qq9qyyssq52qwfdtfeawtwlkc7tfp2mfap6aum4vkyzc0f32tchv8jdl9km042tpxdq57num282d92m5w5cn5vn6fk3f9rvhhehzzg7z2wsmkquqqyd4q7d",
  "status": "failed"
}
{
  "_id": {
    "$oid": "61a7c8a5c7f362f7473a0b4f"
  },
  "paymentHash": "f1330670b47c8215f7031624cfadc0832ad10c9332b0505bee88a7be0a798533",
  "__v": 0,
  "paymentDetails": {
    "milliSatsFee": 0,
    "roundedUpFee": 0,
    "_id": {
      "$oid": "61b2548c082b7fede7ba6885"
    },
    "createdAt": {
      "$date": "2021-12-01T19:10:26.076Z"
    },
    "destination": "0362bf71f3ff53aa074be9a19da6c7e49a50739dbf03cefaff8c4dc73d2aac6079",
    "milliSatsAmount": 1000000,
    "paths": [
      {
        "fee": 0,
        "_id": {
          "$oid": "61b2548c082b7fede7ba687f"
        },
        "fee_mtokens": "0",
        "hops": [
          {
            "fee": 0,
            "_id": {
              "$oid": "61b2548c082b7fede7ba6880"
            },
            "channel": "148x1x0",
            "channel_capacity": 1000000,
            "fee_mtokens": "0",
            "forward": 1000,
            "forward_mtokens": "1000000",
            "public_key": "0362bf71f3ff53aa074be9a19da6c7e49a50739dbf03cefaff8c4dc73d2aac6079",
            "timeout": 270
          }
        ],
        "mtokens": "1000000",
        "payment": "08004f3d447164fd2343b95d717ac28ecd2c632f4c7782f18b6b9b67648430e8",
        "timeout": 270,
        "tokens": 1000,
        "total_mtokens": "1000000"
      }
    ],
    "amount": 1000
  },
  "paymentRequest": "lnbcrt10u1ps60j9ppp57yesvu950jpptacrzcjvltwqsv4dzrynx2c9qklw3znmuznes5esdqqcqzpuxqyz5vqsp5pqqy702yw9j06g6rh9whz7kz3mxjcce0f3mc9uvtdwdkweyyxr5q9qyyssq3ankzemmgsjrxyyaqadgq8mtpequnh4m6l7fq3973204fullz058htveakpnnuwde7agdaakt7340fqcyy3awqf6nun8dnhy56uflkgq7unl42",
  "status": "failed"
}

Is the difference that some payments are pending and then fail while others never become pending?

the second row doesn't have a createdAt for some reason:

Is this the latest version of the code? There's now a nested paymentDetails object that holds all the properties that can change as a payment moves from pending to settled/failed.
The createdAt prop is currently in there but I can pull it up to the top level of the object and assign it a value in the case of the api not returning one (when payments fail without being pending)

Is the difference that some payments are pending and then fail while others never become pending?

Yes exactly, null is when the payment immediately fails and we get back no information for those fields from lnd whereas we can sometimes have information from a pending state that eventually fails. Is this the best way to handle this, or would some other way of reflecting this be better?

the second row doesn't have a createdAt for some reason:

Fixed here: https://github.com/GaloyMoney/galoy/pull/665/commits/4e0a6550c56c0cae4cf90739980e1fc59ed728cd

Yes exactly, null is when the payment immediately fails and we get back no information for those fields from lnd whereas we can sometimes have information from a pending state that eventually fails. Is this the best way to handle this, or would some other way of reflecting this be better?
The idea behind the paymentDetails object as well is to have values that can change in there, vs. other top-level properties which stay fixed during the lifetime of the payment

Yes that makes sense, then it's definitely good to move createdAt to the top level.  I'm not sure what circumstances would lead to lnd failing without returning any information but it might be useful to also know as much information as possible about these intended payments (e.g. the intended destination).
Would it be possible to simply add a field like failedImmediately to paymentDetails?

I'm not sure what circumstances would lead to lnd failing without returning any information

The hint is in this type on the library we use. The library actually returns no information whenever it fails whether it was immediate or after a pending state (see failed property). What I'm doing is persisting the transition state (from pending property) internally if we see a pending state first that eventually switches to failed.
For additional context, if a payment succeeds either eventually or immediately, we always overwrite with the data we get back from the payment property.

but it might be useful to also know as much information as possible about these intended payments (e.g. the intended destination)

This is all already encoded in the top-level paymentRequest property.

Would it be possible to simply add a field like failedImmediately to paymentDetails?

Based on what you're saying here, is it that you'd like us to decode whatever we can get from the paymentRequest and add it to paymentDetails and then also add something like a failedImmediately property to signal that immediate fail instead of simply leaving paymentDetails as null?

I'm not sure what circumstances would lead to lnd failing without returning any information

The hint is in this type on the library we use. The library actually returns no information whenever it fails whether it was immediate or after a pending state (see failed property). What I'm doing is persisting the transition state (from pending property) internally if we see a pending state first that eventually switches to failed.
For additional context, if a payment succeeds either eventually or immediately, we always overwrite with the data we get back from the payment property.

but it might be useful to also know as much information as possible about these intended payments (e.g. the intended destination)

This is all already encoded in the top-level paymentRequest property.

Would it be possible to simply add a field like failedImmediately to paymentDetails?

Based on what you're saying here, is it that you'd like us to decode whatever we can get from the paymentRequest and add it to paymentDetails and then also add something like a failedImmediately property to signal that immediate fail instead of simply leaving paymentDetails as null?

Thank you for the link to the library, I think I understand the issue much better now and I think it actually makes more sense to keep paymentDetails null for failed payments (it should also be set to null for payments that were pending but eventually failed too!).  The reason for this is that paymentDetails only describes a single payment attempt which is canonical for successful payments but not for failed payments.
With regards to decoding the paymentRequest, I don't think it's necessary to do that here.  It could be decoded as part of whatever downstream data analysis we want to do.
I think we should also persist data on failed payments (and failed attempts of successful payments) but, since it's of quite a different structure, maybe it needs to be a separate collection and PR.

Update
Based on a call we had a little while back, the open things left to do on this are:

move the update process to cronjob and use getPayments method instead for full update
edit the update we do in updatePayments to be as light as possible (can even remove if we think that's appropriate, or add an isPersisted flag to indicate that cronjob hasn't persisted as yet)

In a follow-up PR

add a delete-from-lnd step in the cronjob for payments already persisted

Closing in favour of PR #848


Add missing stdlib.h

Added stdlib.h to shell.c as a hard requirement for GCC 14.

I'm sorry, I made a mistake resolving the conflict. I'll just cherry-pick the atoi change into master

Thanks for the PR! I had not noticed that atoi was defined there before.

This change  seems to cause some issues after all. I've reverted the change with this PR: https://github.com/pspdev/psplinkusb/pull/47
Please review.


Missing ModelicaExternalC.dll in Modelica

I was excited to see this code available, however I get the following error when I load the Modelica code into Dymola and run a simulation on the XMLTest1 model:
The program can't start because ModelicaExternalC.dll is missing from your computer.  Try reinstalling the program to fix this problem.
I can find ModelicaExternalC.lib, but not the dynamic library.  Do I need to to change something in the ExternData code in order for it to look for the static library, not the dynamic library?
System specifics:
Windows 7 Enterprise SP1
Dymola 2015 and 2016 32-bit and 64-bit versions

The library should work out-of-the-box with Dymola. ModelicaExternalC.dll (and in fact all DLLs) is not required by Dymola. Only ED_XMLFile.lib and expat.lib from the library and ModelicaExternalC.lib from the Dymola installation are needed for linking an executable of XMLTest1.

That's what I thought should happen.  However, I get the error message above when I use version 1.0.2 of ExternData.

Did you open package.mo in Dymola? What happens if you delete all DLLs from the Library dirs?

You must not use ExternData\Resources\thirdParty\ITI\win32\ModelicaExternalC.lib with Dymola. This is only for building the DLLs for SimulationX from ITI. Dymola provides its own ModelicaExternalC.lib.

Deleting all of the DLL files from the ExternData Library folders seems to have fixed the problem.  The simulations run and I no longer get the error message.  Thank you for your help and your quick responses.

Glad that it works for you. But strange, I thought I checked this. I'll try to reproduce it next week with Dymola 2015 and 2016. I'll leave this issue open meanwhile.

Actually I cannot reproduce the problem you mentioned with Dymola 2016. I downloaded ExternData 1.0.2 from the releases, extracted it to an empty directory and opened package.mo in Dymola 2016. The example models work out of the box.

I can confirm this issue exists. Did a fresh clone, then tried to run XMLTest1 with Dymola 2016.

@thorade Thank you for reproducing the issue. If I only would know what goes wrong here?

I believe it might be related to another issue I once encountered:
https://github.com/iea-annex60/modelica-annex60/issues/40
If I manage to fix it, I will report a solution here.

Do you use MSVC or MinGW compiler?

In case of MSVC the linker option /VERBOSE:Lib displays a list of founded lib dependencies.

I tried Visual Studio 2013 Community Edition, all examples have the same error.
The fix suggested by jnangle works: After deleting all dll files from ExternData\ExternData\Resources\Library\win32 the examples simulate.
Where would I specify the linker option and where can I find the output?

Please set /VERBOSE:Lib in file build.bat of your Dymola\bin install directory. Edit line 197 from
set LINK=%DYMOLALINK% /stack:0x500000 /NODEFAULTLIB:LIBCD /NODEFAULTLIB:LIBC /NODEFAULTLIB:LIBCMT /OPT:NOREF  %liblink% %libdsLib% %DELAYARG1% %DELAYARG2% dymosim.obj %EXTRALIB%

to
set LINK=%DYMOLALINK% /VERBOSE:Lib /stack:0x500000 /NODEFAULTLIB:LIBCD /NODEFAULTLIB:LIBC /NODEFAULTLIB:LIBCMT /OPT:NOREF  %liblink% %libdsLib% %DELAYARG1% %DELAYARG2% dymosim.obj %EXTRALIB%

Please run the translation again and watch the output.

Thanks. Looks OK though since ED_XMLFile.lib and expat.lib are linked as expected. Need to check the linker warning about the imports.
Can you please check the generated dymosim.exe with the dependency walker (depends.exe) what DLLs it uses! The question is where the dependency of ModelicaExternalC.dll comes from?

OK, I will do that.
As deleting the dlls from the Resources\Library\win32 directory seems to fix the problem, I would assume they are the problematic part!? Are they needed or would it be an option to deliver the library without dlls?

Yes, indeed. The link instruction (line 197 of build.bat) contains this DELAYLOAD: option. Since ED_XMLFile.dll and ED_XMLFile.lib have the same name this leads to the conflict.
The reason why there are both ED_XMLFile.dll and ED_XMLFile.lib is simple. This way the lib should work out-of-the-shelf with SimulationX and Dymola (on Win).

I'll rename ED_XMLFile.dll to ITI_ED_XMLFile.dll and will release a new version soon.

Dependecy Walker gives following warning for dymosim.exe:
Warning: At least one delay-load dependency module was not found.
Warning: At least one module has an unresolved import due to a missing export function in a delay-load dependent module.

But I did not yet find which module it is.

@thorade Can you please git clone again (to a new directory) and check simulation with Dymola again. Should be fixed by 26754cb.

Yes, now it works out of the box!

Thanks.

Thanks!
I might be using the JSON reader in the future.

Sure. Tell me if you see improvement.

Argh, commit links should go to #3.


1640: Update and bundle SWIG

Update and bundle a newer version swig to allow full migration of C++11.
To test (on EL6, and Ubuntu):

Build portable: You should see swig build in the output at this step.
Install portable
Verify that it is working.
Build and install open.
Verify that it works.

Its also worth noting that the wiki page will need to be updated for portable. I will do this once the PR is merged in, as it isn't a static file that can be edited as part of the PR.


Error while adding a relationship

I am trying to extract features automatically with featuretools from synthetic data I have created. An error is raised when I add a particular relationship es.add_relationship(ips_flows) (line 12 in synethetic_flows.py). The error says that the key ip does not exist, while it does in the input data frame.
Is it a bug ? Or do the linked columns need to meet some specific constraints ?
Context :
- installation with pip install featuretools
- run with python 3.5
synthetic_flows.zip

@ab-anssi the problem here is that the relationships must be one-to-many. In your code, the relationship between main and ips is a one-to-one since the relationship is on both of their index columns.
If I remove main and build features for ips as the target entity, it all run. Attaching the code.
However, looks like your label is in main, so I'd recommend merging them into one table before DFS if you need it or just extracting the label separately after getting your feature matrix.
Does that help? Also, feel free to post question in our gitter chat room: gitter.im/featuretools/featuretools

If the code above works, you should see a feature matrix like this
       AS_number country  SUM(flows.duration)  SUM(flows.num_bytes)  STD(flows.duration)  STD(flows.num_bytes)  MAX(flows.duration)  MAX(flows.num_bytes)  SKEW(flows.duration)  SKEW(flows.num_bytes)               ...                 It works !


Synchronize changes from the kubeedge/staging directory on 2023.12.16

Synchronize changes from the kubeedge/staging directory on 2023.12.16

/lgtm

/approved


successCallback ?

i tried to investigate the events the socket triggers:
successfully sent
 writeToSocket
 socket.write <Buffer>
 onDrain called with writeBuffer.length 0
 drained

error
 writeToSocket
 socket.write <Buffer>
 onDrain called with writeBuffer.length 0
 drained
 socket on data <Buffer 08 08 00 00 00 00>
 errorCallback
 socket close

both success & error, have the first four events common. the only difference with error is that data is returned on the same socket, and then closed.
is it possible to set a successcallback (that doesnt rely on timers for an error to come back ) ?
~B

just to be consistent with how other library usually pass callback(er, result).
i have implemented notification specific errorCallback for node-apn here. but cancelled the pull-request when i realised that successcallback wasnt working.
https://github.com/bosky101/node-apn/commit/90e4d7ec2843b85e765b45ac719108690087ad0d


Create Release

Once Permission for creating releases and docker registry are done a release needs to be created with the following commit 17ac4bde75d490bef84a3b32a84d255d6f5f01c9

/remove-lifecycle stale


appropriate way to run migrations

While reading NuxtHub's docs I see a comment about only run migrations in development:
https://github.com/nuxt-hub/core/blob/b36246d0e54ab465034282a12f502835bfd37c2d/docs/content/docs/3.recipes/1.hooks.md?plain=1#L17-L30
But wouldn't this use the Cloudflare's quota? (I imagine only while in remote)
Moreover, while reading Cloudflare's docs about D1 migrations I also see that they are supported via .sql files. How complex could it be to implement a local migration folder (alongside all Nuxt's folders) that is cloned to Worker's directory and handle migrations this way?
First time checking out both NuxtHub and Cloudflare 😜

Hi @Sandros94
I actually run my migrations in development to my staging or production database using npx nuxt dev --remote
It works well so far!

I actually run my migrations in development to my staging or production database using npx nuxt dev --remote

Are you referring to the Nitro plugin for migrations or D1's .sql migrations?
I imagine the first

Nitro plugin yes @Sandros94

You can also take a look at https://hub.nuxt.com/docs/recipes/drizzle#migrations for usage with Drizzle ORM, this is what we use to build https://admin.hub.nuxt.com

You can also take a look at https://hub.nuxt.com/docs/recipes/drizzle#migrations for usage with Drizzle ORM, this is what we use to build https://admin.hub.nuxt.com

That indeed was my current plan and started learning a few things locally with Drizzle + db0, with the main goal to migrate those tests into D1.
I'm closing this issue for cleanup, also since the operations are quite simple and easy to understand

Check you emails @Sandros94 :)


Python  setup develop failed on Windows.

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
Mmcv-full has been installed successfully, but I got error when running python setup develop
13:51:48  Using c:\users\user\miniconda3\lib\site-packages\tomli-1.2.2-py3.8.egg
13:51:48  Finished processing dependencies for mmdet==2.18.0
13:51:48  C:\Users\user\miniconda3\lib\site-packages\torch\utils\cpp_extension.py:381: UserWarning: Attempted to use ninja as the BuildExtension backend but we could not find ninja.. Falling back to usi
13:51:48  ng the slow distutils backend.
13:51:48    warnings.warn(msg.format('we could not find ninja.'))
13:51:48  Error in atexit._run_exitfuncs:
13:51:48  Traceback (most recent call last):
13:51:48    File "C:\Users\user\miniconda3\lib\site-packages\colorama\ansitowin32.py", line 59, in closed
13:51:48      return stream.closed
13:51:48  ValueError: underlying buffer has been detached

https://github.com/open-mmlab/mmdetection/issues/5429
Reproduction

What command or script did you run?

python setup develop```

After `pip install requirements/built.txt`

2. Did you make any modifications on the code or config? Did you understand what you have modified?
3. What dataset did you use?

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

Got the same error after ninja                  <IP_ADDRESS>  installed

@del-zhenwu Can the #5429 solution be used directly to solve it?

@del-zhenwu Can the #5429 solution be used directly to solve it?

Maybe not the same issue I think, cause the mmcv package has been installed before


When register a user or organization account, TimeZoneId should be from user's local TimeZoneInfo instead of from config.json

Replaced the user TimeZoneId retrieved from config.json:General:DefaultTiemZone with the user's TimeZoneInfo.Local.Id in Account and Admin Controller;
Updated the corresponding unit tests.

Thanks for this @JingWendyLiu. Unfortunately, this will not work because TimeZoneInfo.Local represents the local timezone of the server process, not the local timezone of the user who made the request. In the case of the application running in Azure (or on most web hosting providers), that would be UTC. Using TimeZoneInfo.Local here works for developers running the application from Visual Studio because it will default new users to the developer's local time zone but it will not work for the hosted application in Azure, at least not in the way we would like it to.


can use this lib for react native?

hi
can i use this lib for react native app?
and work fine in android and ios?

To the best of my knowledge, yes. You should be able to use React Table in React Native.

how to fix https://github.com/tannerlinsley/react-table/discussions/2144???

react-native table, I'm going to use this one: https://github.com/meliorence/react-native-render-html


Fix SpatialConnectionCleanupTest copyright notice

Description
Corrects copyright notices in SpatialConnectionCleanupTest

As a side note, I also noticed this test doesn't have a high-level description of what it does - would be useful to add.


Error loading webview: Could not register service workers

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.70.0
OS Version: win11

Steps to Reproduce:
Error loading webview: Error: Could not register service workers: InvalidStateError: Failed to register a ServiceWorker: The document is in an invalid state..

Maybe a duplicate of  #157392

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Update seti-ui theme

This PR fixes #98586
Updated theme has a new icon for go:

Verified on Mac + Windows, updated go icon looks good.


Show Score when Game Over

When game is over, I want to see my last score next to the overall high score.
If I it is a new high score, it want to be informed that it is a new one.

closed in 2c3cb38