Can we update xgboost4j dependency with latest spark version ?

Can we update xgboost4j dependency with latest spark version ?
The spark version 3.1.1 is pretty old.

CC @trivialfis @wbo4958

May I ask what's the motivation for upgrading spark dependency? I suppose xgboost is using the public APIs of spark, which suppose xgboost can run on any spark 3.x version. Please correct me

good point !
Unfortunately latest spark version brings breaking change. See https://github.com/dmlc/xgboost/pull/8831

We can improve CI to test against multiple spark versions instead of upgrading dependent spark version


Does seem to not work anymore with VScode 1.70.1

Does seem to not work anymore with VScode 1.70.1

Hi, @mwinterm.
Thank you for your feedback, and sorry for the very late reply and check.
It seems to work with current VS Code (1.74.2).
This extension also has a new build (v0.0.5), so please try it if you have time.


Does Not Import Emoji characters

+1 to this. I'm using left and right arrows to indicate alignment, which also don't get imported correctly. I was able to fix it by escaping the special characters in the JSON file, but it'd be nice for this to be automated.


[BUG] when i trie to download bloxstrap it gives me this error : "System.AggregateException: One or more errors occurred. (The given key 'content-platform-dictionaries.zip' was not present in the dictionary.)  [Inner Exception] System.Collections.Generic.KeyNotFoundException: The given key 'content-platform-dictionaries.zip' was not present in the dictionary."

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.5.4
What problem did you encounter?
System.AggregateException: One or more errors occurred. (The given key 'content-platform-dictionaries.zip' was not present in the dictionary.)
[Inner Exception]
System.Collections.Generic.KeyNotFoundException: The given key 'content-platform-dictionaries.zip' was not present in the dictionary.
Bloxstrap Log
N/A

Please help me fixing this thing bro i used to play roblox smoothly with bloxstrap now i can't use it because of this issue.

Anybody knows how to fix this thing? plsss

Update bloxstrap


Local environment setup enhancements

Description
Add docker-compose.yml for easier local PostgreSQL database setup locally and leverage a Makefile for shorter make commands to run the project.
QA Instructions

[ ] Step 1 Start the local env PostgreSQL Docker Container: make start-db
[ ] Step 2 Install the project Dependencies: make install-dependencies
[ ] Step 3 Initialize the Database: make init-db
[ ] Step 4 Generate Prisma ORM client: make generate-orm
[ ] Step 5 Start Project: make start

Evidence

@RiverRuby I've prepared this contribution as a small initial enhancement based on the local environment setup our team has followed in the PSE Core Program of Costa Rica. Please let me know if it brings value and if helps to keep a standard local setup.


Fineract-428 Implementing parallelizing and paging of Add accrual accounting

FINERACT-428
Parallelizing and paging of add accrual accounting

@vorburger yes, I'm interested in becoming an Apache Committer. Besides this has no conflicts at the moment and I'd kindly request @avikganguly01 to review these changes


Integrate with aio-lib-core-errors

Description

Replace the custom TvmError.js with aio-lib-core-errors

Motivation and Context
Unified error handling across the sdk

Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #5   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines          68     68
  Branches       16     14    -2
=====================================
  Hits           68     68

Impacted Files
Coverage Δ

lib/TvmError.js
100% <100%> (ø)
:arrow_up:

lib/TvmClient.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Adding xref file

@dend,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


display issue with the phenotype table

The evidence and assertion count values do not align with the column header boundaries in the phenotype table:

Which browser are you using? I'm unable to replicate this on Chrome Version 124.0.6367.119. The space on the right is where the vertical scroll bar should be. This appears to be missing in your screenshot so the table values align to fill the space. Does the scrollbar appear for you in other tables?

After chatting with @jmcmichael this might be a side effect of having scrollbars always turned on in your browser setting. Josh isn't able to replicate this issue in his browser, where his scrollbar settings are "Automatically based on mouse or trackpad".

In macOS, one may toggle scrollbar options in System Settings / Appearance / Show scroll bars.

Since this is not a bug, per se, but a limitation of the ng-zorro component library we are using in combination with a specific user system setting, there isn't really anything we can do on our end to address this short of using a different component library. Closing this issue as "wontfix".

I believe this open issue is what we are seeing.


Task description: Use self-api of linter?

I was thinking that we might have lost some possibilities using the linter API. One thing I was planning was to attach behaviors to errors and warning - say "quickfix". For instance, the command for import suggestions could be attached to the DOM element of an error if we had such an element. But typical usage of linter API doesn't provide that - we leave the entire rendering business to linter.
I now saw that we can actually probably achieve this with the Self API of linter:
https://github.com/atom-community/linter/wiki/Self-(internal)-API
That in combination with the new block decoration API could be really powerful:
http://blog.atom.io/2016/02/03/introducing-block-decorations.html
In particular:
  onDidUpdateMessages(callback:Function<{linter, messages, editor}>)
I guess would be  called of an open editor when there where messages being displayed. So here we could use decorations and attach behaviors I think, like quickfix.

Please note that compatibility of self-API is not guranteed and can be broken at any time (we've already broke most of it in the branch that is soon going to be merged).
If you want to provide buttons to linter, you can do so by giving an HTMLElement in the html field of the message, if you want to provide an autofix API for linter messages it's already plan and will soon be implemented (https://github.com/atom-community/linter/issues/1049). Usage of self-API is highly discouraged and should only be used if you really want to take on all the risks

@steelbrain Good to know! This was not on my immediate todo, but then I'll wait until something might stabilize as an official API. A button might be good enough actually. And then just have that button create a marker + decoration for some extra features or whatever.
Thanks!

I'll close this since that's probably not the way we'll do it. But #197.


[BUG] - Review and improve docker images

Describe the bug
Currently, we have 2 Dockerfiles used to build images:

conda-store, which does an editable install:
https://github.com/conda-incubator/conda-store/blob/64552a28e649d7305ec971332cfb739286769ec9/conda-store/Dockerfile#L22-L23
❗this is also the file used during releases which should not be the case
conda-store-server, which installs a local directory:
https://github.com/conda-incubator/conda-store/blob/64552a28e649d7305ec971332cfb739286769ec9/conda-store-server/Dockerfile#L49-L51

These are used by the docker-compose.yaml in:

the root dir, used by developers for local development
the examples dir intended for end-users --❗we should use released versions from PyPI or conda-forge here, or refer to the docker images built on releases

docker/docker-compose.yaml - ❗this file also has defined redis twice, which causes errors while building
docker-without-nfs/docker-compose.yaml

☝🏽Additionally, the port number 5000, where the UI is deployed for development does not work reliably on MacOS. We need to pick default ports that work on all platforms.
Expected behavior
We need to have separate images for building dev and stable versions of conda-store. The releases should use to stable versions.
Anything else?
xref: #589, #566

I think we can use the docker-compose.yaml in conda-store-ui, as is and as a ref, for the examples directory.

@dcmcand it looks like the review is complete and all of the issues have been opened as separate issues. Can we close this in favor or the individual issues?

Yes @kcpevey, we can close this one.  The only outstanding issue is #618. That should be a pretty easy one.


为 Metrics 示例增加 kubernetes 部署支持

https://github.com/apache/dubbo-samples/tree/master/4-governance/dubbo-samples-metrics-spring-boot

assign to me, thanks

Added in https://github.com/apache/dubbo-samples/pull/745

需要等  https://github.com/apache/dubbo/pull/11630 merge 后重新添加demo


Problem while trying to run examples

Hi,
I am not sure if this is a bug or I am doing something wrong but invoking
./http_client -s www.google.com -p /

results in
15:01:47.795 [WARN] [QUIC:967430E39685E122] handshake: encrypt crypto context at level forw-secure not initialized

and nothing more is returned i.e. there isn't any HTTP response in my console.
Looking into wireshark I cannot see any payload sent to me or from to google. I also tried to test it with
quic.tech:4433 or quic.tech:8443 but none of them works for me.
All tests are passing so I believe my installation of lsquic is correct.
Am I missing something? I am using version from master.

The problem is that Google uses h3-29, whereas lsquic by default will use the highest version it implements, which is h3-31.  There is no version negotiation in IETF QUIC v1, so handshake fails.  The warning message is a bug in lsquic: it fails to encrypt a packet with a CONNECTION_CLOSE frame in it.
To make the long story short, add -o version=h3-29 to your command line and things will work.

I take it back: version negotiation is attempted, but lsquic does not do it properly:
sh$ ./http_client -s www.google.com -p / -l event=debug
11:46:17.631 [DEBUG] [QUIC:F6149AB853641E1B] event: connection created
11:46:17.631 [DEBUG] [QUIC:F6149AB853641E1B] event: version negotiation: proposed version FF00001F
11:46:17.631 [DEBUG] [QUIC:F6149AB853641E1B] event: created packet 0; flags: version=1, nonce=0, conn_id=1
11:46:17.631 [DEBUG] [QUIC:F6149AB853641E1B] event: generated CRYPTO frame: offset: 0, size: 235
11:46:17.631 [DEBUG] [QUIC:F6149AB853641E1B] event: sent packet 0, type Initial, crypto: clear, size 1200, frame types: PADDING CRYPTO, ecn: 0, spin: 1; kp: 0, path: 0, flags:<PHONE_NUMBER>
11:46:17.645 [DEBUG] [QUIC:F6149AB853641E1B] event: version negotiation: supports version FF00001D
11:46:17.645 [DEBUG] [QUIC:F6149AB853641E1B] event: version negotiation: supports version FF00001B
11:46:17.645 [DEBUG] [QUIC:F6149AB853641E1B] event: version negotiation: supports version Q050
11:46:17.645 [DEBUG] [QUIC:F6149AB853641E1B] event: version negotiation: supports version Q046
11:46:17.645 [DEBUG] [QUIC:F6149AB853641E1B] event: version negotiation: supports version Q043
11:46:17.645 [DEBUG] [QUIC:F6149AB853641E1B] event: packet 0 repackaged for resending as packet 1
11:46:17.645 [DEBUG] [QUIC:F6149AB853641E1B] event: sent packet 1, type Initial, crypto: clear, size 1200, frame types: PADDING CRYPTO, ecn: 0, spin: 1; kp: 0, path: 0, flags:<PHONE_NUMBER>
11:46:17.655 [DEBUG] [QUIC:F6149AB853641E1B] event: version negotiation: agreed version FF00001D

What lsquic does is resend the ClientHello using the negotiated version of QUIC, ID-29, but wrong ALPN, h3-31.  It should regenerate ClientHello and include the correct ALPN, h3-29, for the version negotiation to work.
I don't expect this to be fixed, because a) this mechanism is likely to become defunct when QUIC v1 is released and b) version negotiation is not necessary for an HTTP/3 client, because the other side's version is communicated to it via the Alt-Svc HTTP header.  Nevertheless, let's leave this bug open for now as a record.


Proposal: caching the context

The purpose here is to speed up build iterations, especially when working with large contexts ('large' means hundreds of MB, possibly a GB).
There are some ways to mitigate the problem such as #6579 (.dockerignore to prevent uploading unnecessary files) and #5369 (to avoid copying source/intermediate files into the resulting image), but the next bottleneck is the transfer of the useful context itself. The current implementation requires to upload and process it completely for every build, even if only few files were changed since the previous build.
There have been several suggestions to mount an external volume at build (#8394 #1191 #8757), but they raise security and reproductibility issues.
Proposal
The change consist of updating the POST /build operation to allow sending the context in two steps. The first step would only include the files metadata and the sha256 sum of their content. The second step would provide the actual content of files (if requested by the server).
A build could take place as follows:

The client computes the sha256 sum of every regular file present in the context

The client sends its /build request. The query includes an option to enable server-side caching (eg. cache_context=true). The body is a 'light' tarball of the context in which the content of regular files is replaced with their sha256 sum, except the Dockerfile which is transferred as is.
POST /build&cache_context=true HTTP/1.1
Content-Type: application/tar

[...context tar archive, but with the content of regular files replaced
 with their sha256 sum...]

In the response, the first chunk sent by the server includes a "miss" item listing all files that are missing from the cache. In case of 100% cache hits, the "miss" must be present and empty. Then the build proceeds normally.
HTTP/1.1 200 OK
Content-Type: application/json

{"miss": [ ...list_of_missing_sha256_keys... ]}
{"stream":"Step 1...
 ...}

The client parses the response.

If the first chunk does not include a "miss" item, then the client must assume that caching is not supported. It must abort build and restart a new build with caching disabled.

If the "miss" item is empty, nothing particular happens.

If the "miss" item is not empty, then the client creates a tarball with the missing files indexed by their sha256sum and  opens a second HTTP connection to upload it. In the tarball, the files should be stored in the same order as in the list sent by the server.
POST /build/upload HTTP/1.1
Content-Type: application/tar

[...tarball of the missing files indexed by their sha256 sum...]

The server must verifiy the sha256 sums of the files before adding them to the cache, and then performs the build.
HTTP/1.1 200 OK

This change would allow several optimisations:

once a file has its sha256 sum validated by the server, it can be cached for subsequent builds.
when launching a new build with a slightly modified context, only the updated files need to be uploaded
ADD/COPY of cached files will not require re-hashing on server-side (the server can trust its internal cache)
if the storage driver is 'btrfs', all server-side copies (CACHE->CONTEXT and CONTEXT->CONTAINER) can rely on the copy-on-write abilities of the filesystem (cp --reflink)
in case of a rebuild, hashing files should be faster on client-side since they are likely still cached in memory by the OS (if needed it is even possible to cache the sums on client-side and re-hash the files only if their mtime is changed)
the server can start the build eagerly, before the context is fully uploaded
the server can parse the Dockerfile eagerly and build the list of cache misses accordingly:

so that files needed first are uploaded first
so that unused files are not uploaded at all

Implementation considerations:

before adding a file into the cache, the server MUST verify the sha256 sum (otherwise this would be a security hole)
the proposal does not address how the cache is managed. As for now we can just assume it is a basic LRU cache hosted by the docker engine. Sharing cache with a cluster of daemons or with an external storage facility is out of the scope.
the size of the cache should be configurable at runtime (eg. 'docker -d --context-cache=1G') the default should be rather small
instead of replacing the content of files with their sha256 sum in the tarball, it could be better to include the list of cached files as a file inside the archive, with a special name (eg. .docker-cached-files), the question is open

I like the #batteriesincludedbutremovable idea, this feature would improve so many lives, it should come included and be easy.  And I'm sure people are coming up with their own solutions.  But at the same time it adds bloat, and different people may require different solutions.
Such as if the build server knew that the project was in a git repo, it could instead fetch new changes instead of transferring from the client.
Also, instead of reinventing the wheel for efficiently sending over new changes, we should reuse something like rsync.  And after a quick Google search, it brings up librsync—a library for network delta-compression of streams and files—which is used by Dropbox, rdiff-backup, duplicity [1].
[1] http://librsync.sourcefrog.net

:+1: I like the sound of this.

If one were to cache the build context, I would like to argue it should only cache things that are added or copied in during the build. Otherwise, this could become a very massive thing that contains unrelated junk.

I made some refinements in the proposal. There are two changes:

missing files are upload in a separate http connection, thus the two requests (build an upload) can happen in parrallel
the content of the Dockerfile is uploaded immediately (instead of trying to get it from the cache)

These changes allows two additional optimisations:

the server can start the build eagerly, before the context is fully uploaded
the server can parse the Dockerfile eagerly and build the list of cache misses accordingly:

so that files needed first are uploaded first
so that unused files are not uploaded at all

@a-ba Keep in mind, the builder is being moved to the client side.

And by client side, I mean client driven.

Oooh, that's really old.
@cpuguy83 @tonistiigi Do you know if it'll be implemented?


Non-randomized placements should not grant hint points

I take it from the "server-side" tag that making the non-randomized items handled locally is not in the cards, and that they must come from the server?

I'm working on a server-side solution

New proposal for this is we will make a toggle (default off) to place vanilla items for coop purposes. I do see the value in the feature as implemented but I don't believe that it's correct as the default, as evidenced by the number of people that have complained about it

Addressed on the AP side in 0.4.5. Necessary changes to shops done in<PHONE_NUMBER>12208efbbcc41b4a2952b9ea9c28b0


Include Yarn with the php82 node 16 image

The title speaks for itself

Ah I see we did not merge this one. I've solved it in another way: I've copied yarn from the official node16 container. Closing this


feat: create new snarkjs groth16 template

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (yarn compile) was run locally and any changes were pushed
[x] Lint (yarn lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
#96 improves the Verifier.sol code, but that code is auto-generated from SnarkJS templates.

What is the new behavior?
Verifiers will be generated from our custom template, which contains the latest changes (#96).

Does this introduce a breaking change?

[ ] Yes
[x] No

@recmo could you check if everything is ok?


matter-bridge  Accessories are not displayed in Apple Home (CON-911)

I currently have a product implemented based on the official example/matter-bridge-app. My ZAP file structure has endpoint 0 as the root node, endpoint 1 as an Aggregator device type, endpoint 2 as a Thermostat device type, and endpoint 3 as a Matter Dimmable Light. Endpoint 3 serves as a bridge node, bridging four lights below it, while endpoint 2 functions as a regular Matter device (note: this configuration is allowed in the official specifications). After configuring the devices in the Apple Home app, I noticed that only the accessories for the four lights under the bridge node appeared, and the thermostat under endpoint 2 did not. When I read the properties of the thermostat under endpoint 2 using a Raspberry Pi, I can retrieve the information. Moreover, I can see the thermostat accessory in the Google Home app. Is this an issue of incompatibility with Apple HomeKit, or is there another explanation for this behavior?

The Bridged endpoints should be in the Part List of the aggregator endpoint so that it can be identified as a bridged device by eco-system of Apple or Google. we provide set_parent_endpoint() in esp-matter the set parent for bridged endpoints.

@WOLEN6914183 Can you please check if this helps?

"I have reviewed the official documentation, and our product is similar to the structure in the diagram above. On the bridge device, there is a thermostat similar to the 'ep31' in the diagram, but it does not belong to the 'Part List of the aggregator' node. I can see the thermostat accessory in the Google Home, but it is not visible in the Apple Home."

If the thermostat endpoint is not child endpoint of the aggregator endpoint(not in the PartList of aggregator endpoint), it should be visible as an independent (non-bridged) endpoint in Apple home app. If you cannot find the device in Apple Home app, I think it is Apple home app's issue.

ok, thank you for your answer despite your busy schedule. I will ask Apple's customers when I have time.


Unsupported activity types

I get an error: [ERROR]: Statements contain unsupported activity types: ['SSP']. Please, check documentation.
As I see it the SSP type is the stock split.

Yes, SSPs are the stock split events. Support for those activities is in development.

Same here.

SSP and MAS activities are now supported.
https://github.com/doino-gretchenliev/revolut-stocks/commit/e0bff741581581a85b0fe67dcc74ba40c6f39490


[source-build-release] Announcement draft month is incorrect - clarify intent

Content of announcement draft, in step 3.a, will likely have incorrect month. Draft is created during pipeline's Pre-Release stage. This stage is executed 12 days before release, which is, most often, during the month preceding the release. Real announcement draft will be generated during Release stage and contain the correct release month.
There should be a note in draft validation step, about this, to help anyone new with this process.

Fixed with https://dev.azure.com/dnceng/internal/_git/dotnet-release/pullrequest/33767


Unexpected behaviour of andd

I'm experimenting with Widget composability and decided to try out the form example given in the documentation. I created a gist containing all the code for reference.
Creating a multi form step-by-step widget using traverse works as expected. However, composing them side-by-side using andd results in surprising behaviour (at least for me), when following these steps:

Start a side-by-side composition of the form widget using andd. This results in three forms, laid out on top of each other.
Change every input field of the form into something else, for example "1", "2", "3".
Submit one of the forms by hitting the "Submit" button.
Now there are two forms on the screen, as expected, but the state of these forms returned to their initial value!

I expected each individual form to keep their changed state. Is this expected behaviour or a bug? Should I write other code to make this work as I expect?

Some background: I'm trying to rewrite the edit headings example using widgets instead of signals. Here I ran into the same problem. Probably going to switch to the Haskell version of Concur, because I need some Haskell libs on the backend. I did not try the form example with Haskell yet.

From the code snippet and your description, it does look like a bug in the
implementation of ‘andd’. I’m currently travelling so I will take a look at
this later today.
On Sat, 24 Aug 2019 at 16:57, Tim Steenvoorden<EMAIL_ADDRESS>wrote:

I'm experimenting with Widget composability and decided to try out the
form example given in the documentation
https://github.com/ajnsit/concur-documentation/blob/master/README.md#replicating-elm-architecture.
I created a gist
https://gist.github.com/timjs/eb859d552e7113281f402c7d92e45449
containing all the code for reference.
Creating a multi form step-by-step widget using traverse works as
expected. However, composing them side-by-side using andd results in
surprising behaviour (at least for me), when following these steps:

Start a side-by-side composition of the form widget using andd. This
results in three forms, laid out on top of each other.
Change every input field of the form into something else, for
example "1", "2", "3".
Submit one of the forms by hitting the "Submit" button.
Now there are two forms on the screen, as expected, but the state of
these forms returned to their initial value!

I expected each individual form to keep their changed state. Is this
expected behaviour or a bug? Should I write other code to make this work as
I expect?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ajnsit/purescript-concur/issues/28?email_source=notifications&email_token=AAA7CPRU2FZING7LQXY4GLDQGELLPA5CNFSM4IPF6D6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HHFSNJA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA7CPU7K6T4R5GZULZJ3RTQGELLPANCNFSM4IPF6D6A
.

--
Sent from my hyper-communicator

That's fine! Good travels.

On second thoughts, this does not seem like a bug.
When you set a “value” attribute on an input, it’s value will be reset on any state change. React provides a “defaultValue” attribute for this very problem. Can you try changing the attributes to “defaultValue”?

I'll try it out and let you know!

Works better, but still not as I expect:

Change every input field of the form into something else, for example "1", "2", "3".
Submit the second one, containing "2".
Now there are two forms on the screen, as expected, but their values are respectively "1" and "2", instead of "1" and "3".

With defaultValue attribute, you are letting React manage the value of the
input. So you need to provide React with some way to distinguish between
the form instances. Adding a “key” attribute to the form would help (it
also speeds up rendering).
--
Sent from my hyper-communicator

So I’ll need to generate an unique key/ID for every form in the list? I hoped I wouldn’t need to do that with Concur, kind of the big selling point when comparing it with TEA. Like in the tree example where you do not need to index of the subtrees.
Could you help me recreating this example using widgets only instead of signals? I've an implementation in this gist, but probably made some kind of mistake as I did in the form example, as state does not get saved correctly.

Should I add the key to every element? Adding it only to the div results in saving the values of the input fields, but all checkbox are cleared on a submit.

No you only need it at the top level. To get the same behaviour with
checkboxes you need to use ‘defaultChecked’ instead of ‘checked’.
You can read up more about default values in React here -
https://reactjs.org/docs/uncontrolled-components.html#default-values
On Sun, 25 Aug 2019 at 15:08, Tim Steenvoorden<EMAIL_ADDRESS>wrote:

Should I add the key to every element? Adding it only to the div
https://gist.github.com/timjs/eb859d552e7113281f402c7d92e45449#file-form-purs-L33
results in saving the values of the input fields, but all checkbox are
cleared on a submit.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ajnsit/purescript-concur/issues/28?email_source=notifications&email_token=AAA7CPW7TQ7CURHJ5UTCFGDQGJHK3A5CNFSM4IPF6D6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CQARI#issuecomment-524615749,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA7CPTTQHHEXRZ53D7ZPCTQGJHK3ANCNFSM4IPF6D6A
.

--
Sent from my hyper-communicator

Obviously! Thanks :-)

Well yeah. It’s not ideal. There is some discussion about Concur keeping
track of the components that change itself without relying on the backend
(react in this case).
However, to clarify one thing - this is only a problem because you want the
form elements’ state to persist without handling it in Concur. There is
some more discussion on better state handling in Concur which might solve
this problem.
On Sun, 25 Aug 2019 at 14:58, Tim Steenvoorden<EMAIL_ADDRESS>wrote:

So I’ll need to generate an unique key/ID for every form in the list? I
hoped I wouldn’t need to do that with Concur, kind of the big selling point
when comparing it with TEA. Like in the tree example
https://github.com/ajnsit/purescript-concur/blob/master/examples/src/Test/EditHeadings.purs
where you do not need to index of the subtrees.
Could you help me recreating this example using widgets only instead of
signals? I've an implementation in this gist
https://gist.github.com/timjs/fd48c0e9f675134b7a09a8d6424e0cb7, but
probably made some kind of mistake as I did in the form example, as state
does not get saved correctly.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ajnsit/purescript-concur/issues/28?email_source=notifications&email_token=AAA7CPRFZPBYXFG37LDCBJDQGJGCDA5CNFSM4IPF6D6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CP3OQ#issuecomment-524615098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA7CPXXTRKQTVMVR4ZYSHTQGJGCDANCNFSM4IPF6D6A
.

--
Sent from my hyper-communicator

So should I use a similar to trick with keys with the tree widget example? Or am I doing something else wrong?
PS Thanks for responding so quickly to my questions! No need to respond immediately if you're trying to enjoy your Sunday however ;-)

Could you help me recreating this example using widgets only instead of signals? I've an implementation in this gist, but probably made some kind of mistake as I did in the form example, as state does not get saved correctly.

Well you can't just andd the children together as then the widget won't save the state until all the children have finished. I created a combinator called displayList which supports widgets which can be deleted and passes any modifications to the list on to the parent. The code with that combinator is at - https://github.com/ajnsit/purescript-concur/commit/ba1a80f21b88a7d0af8a13268e227b32c2593209#diff-5d9f654f8ead8ee8848e2ba690fba9be

@timjs I'm going to close this issue. Please feel free to reopen in case you have more questions or comments!

Thanks Anupam for helping me out. This is an intuitive solution.
So, if andd does not save the state of the widgets internally, what is the intended use case of it?

I think the question of whether the "state" is saved "internally" or not,
is a red herring.
As a rule, there is no state inside Concur, apart from that captured by the
time sequencing monad (the currently executing step in the sequence), and
the (pure and immutable) arguments passed around to functions. If the state
you are looking to save is not captured in an argument, and is not
determined by the position in the monadic sequence, then Concur does not
deal with it.
Given that, it should be possible to understand the semantics of any widget
or widget combinator in Concur by just its type signature.
-- Type signature
andd :: forall a. Array (Widget HTML a) -> Widget HTML (Array a)

-- And usage
do
  ...
  result <- andd children
  ...

Here andd takes an array of widgets, and returns a single widget. Once
you have combined the widgets with andd, there is no internal state of
any internal widget that's visible outside. Only when the entire andd
returns the result are we able to examine the result and possibly save it
ourselves. Monadic sequencing cannot see inside andd since at this level
it's just an opaque single widget.
Now we know that andd does not return a value until all of its
constituents return themselves. So if one or more of the constituents are
widgets with intermediate steps, there is no way to get access to those
intermediate results from outside andd.
This might seem limiting, but it follows directly from the type signature,
and is expected. Mostly andd was created as a dual to orr, and to
provide a nice contrast with traverse in the documentation :) However, as
displayList demonstrates, it's easy to create your own generic
combinators with any semantics you want. Infact you are encouraged to do
so, as ease of composability is the USP of Concur.
Note that all is not lost, and andd could still be useful in the
following circumstances when -

You have simple widgets that either don't have intermediate steps, or
are fine with losing intermediate state.
No ancestors of the andd have any dynamic actions attached. Such as a
bunch of forms at the top level that all need to be filled.
Even if any ancestor of the andd has any dynamic actions attached,
those are expected to reset the intermediate state of the children. Such as
when you have page switching controls at the top, and it's understood that
switching pages loses the state of the forms.

Thanks,
Anupam Jain
On Tue, Aug 27, 2019 at 1:32 PM Tim Steenvoorden<EMAIL_ADDRESS>wrote:

Thanks Anupam for helping me out. This is an intuitive solution.
So, if andd does not save the state of the widgets internally, what is
the intended use case of it?

Here andd takes an array of widgets, and returns a single widget. Once
you have combined the widgets with andd, there is no internal state of
any internal widget that's visible outside. Only when the entire andd
returns the result are we able to examine the result and possibly save it
ourselves. Monadic sequencing cannot see inside andd since at this level
it's just an opaque single widget.

I totally agree with this way of thinking. It would be odd to have a way to "peak into" the current state of a widget in the array. I expect that, after feeding a list of widgets to andd a step made by one of its elements is remembered.
However, the mistake I made was that every kind of event which would change the view of, for example, an input box would be remembered. But that is not true for basic Dom elements, as you explained. I see now that andd works in the expected way when I crate my own input box widget, which remembers every character entered, like this:
inputbox :: String -> Widget HTML String
inputbox value = do
  result <-
    Html.input
      [ Html.autoFocus true
      , Html._type "text"
      , Html.value value
      , Html.placeholder value
      , Just <$> Html.unsafeTargetValue <$> Html.onInput
      , Nothing <$ Html.onKeyEnter
      ]
  case result of
    Just value' -> inputbox value'
    Nothing -> pure value

Glad to hear it!


Add fall-back font for non Latin script

No good. (Text is supposed to be Hebrew)

notify desk case 227619 when resolved


log collector does not always detect RKE2 distro

I've had many cases where the log bundle captured from a node in an RKE2 cluster only contains the following directories:
journald  networking  systeminfo  systemlogs
I have to tell customers to manually add the -r rke2 parameter for the log bundle to be complete.
curl -OLs https://raw.githubusercontent.com/rancherlabs/support-tools/master/collection/rancher/v2.x/logs-collector/rancher2_logs_collector.sh
sudo bash rancher2_logs_collector.sh -r rke2

We had an occurrence of this with another customer.  Getting stdout to attach here

@dkeightley, do you have any idea what's going on here? I guess you had implemented the detection logic in the past?

If you have an example collection please send it my way, with the output from the process list or rke2-* service logs we might be able to tell. One scenario could be a custom RKE2 install directory is used which can't be detected with the current approach.

Please reach out or reopen and we can continue solving this

Hopefully addressed by https://github.com/rancherlabs/support-tools/pull/254


Verify and update translation to spanish of basic javascript

Closes #6496

@jorgeruvalcaba it's a good practice to run npm test before pushing anything. That will catch all these issues.

@bugron Done, check again please.

@jorgeruvalcaba awesome, after you squash your commits into one commit, and @vtamara or @justinian336 verifies that everything is good we'll merge your PR.

@bugron thanks you, squash it's done. @vtamara or @justinian336 please check if everything is good.

@vtamara corrections done. Please let me know if everything it's ok.

@jorgeruvalcaba I just left a comment about the format of nameES, please check it out.

@bugron I already checked most of this file and suggested some changed that @jorgeruvalcaba applied. ¿Could you please merge?

Thanks for your work on this!


[Feature Request] Provide examples for WinAPI if possible

One of the features of ffi was the use of native WinApi functions.
Provide an example of using your implementation with WinAPI functions in the Readme. If there are any restrictions on the OS version, please indicate them.

I can't quite understand your needs. ffi don't care dynamic library file internal code.

@zhangyuang the thing is that original ffi provides the ability to use functions from WinAPI directly without having to compile your DLL, like this:
var current = ffi.Library("User32.dll", {
   'MessageBoxA': [ 'int', [ winapi.HWND, winapi.LPCWSTR, winapi.LPCWSTR, winapi.UINT ] ]}
);
current.MessageBoxA(ref.NULL, "sss", "sss", 0);

It would be nice if your project could do this too, but I don’t see an example in the doc, so I suspect that it can’t.
In any case, you have already written a useful thing, thank you. The main thing is to add more return types (especially boolean is missing).

Thanks for the info, Currently ffi-rs only provides some common types。I will provide the boolean type as soon as possible。As for WinAPI, it looks a bit complicated, but I'll try to provide it in a later release as well!

ffi-rs version 1.0.13 supports boolean-typed parameters and return values

@zhangyuang that great, thank's!
One more question... Is it possible to immediately return std:string without creating an intermediate char array?

you can describe your need more specified
发自我的iPhone
------------------ Original ------------------
From: Nick @.>
Date: Sun,Oct 1,2023 2:35 AM
To: zhangyuang/node-ffi-rs @.>
Cc: yuuang @.>, Mention @.>
Subject: Re: [zhangyuang/node-ffi-rs] [Feature Request] Provide examples forWinAPI if possible (Issue #3)
@zhangyuang that great, thank's!
One more question... Is it possible to immediately return std:string without creating an intermediate char array?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I mean the example from the Readme for the RetType.String type.
If we're using RetType.String and inside the function we're need copying the std::string to a char *cstr, but it would be nice to be able to just use a construct simple like this:
std::string error = "Error text";
return error;

The c and c++ code in the readme is just a sample. And in ffi scenarios, the return value and parameters must be of c type not c++。std::string is a c++ type not a c type

Ok got it


Tooltip

For this component I created only one story since it is now possible to play around with the tooltip's position and alignment in the Control pannel. If you think we still need to write a story for every position and alignment let me know

I think you are missing some stories.

@ralucacril yes, that's what I explained in the comment above. I am not sure but since it is now possible to play around with the tooltip's position and alignment in the Control pannel, do we still need the whole list ?

so so sorry, i missed your comment. 🙈

tbh for us might not be important to have them as a separate story. but maybe for designers could be important. might not hurt to ask Fred (he is on slack)  or Enrique if they would find the benefit of the story.

@ralucacril Thanks Raluca, I will ask Enrique 👍

@ralucacril According to Enrique there's no need to have the full list


ConnectionPool leak

hi
I encountered a problem that exhaust all the connection in the pool indefinitely.
the scenario is as follows:
the MySQLConnectionHandler.writeAndHandleError() throws a DatabaseException (line #332) due to an inactive channel.
that exception is thrown instead of being encapsulated inside a future(that the method returns) and that exception is propagated all the way up to the AsyncObjectPool.use() method which causes the giveBack() call not to happen and as a consequence the connection stays in the checkouts collection forever which eventually causes the pool to get exhausted with all the connection checked-out and none in the poolables and no new connections are being created.
that leak can be solved in two places:

in the AsyncObjectPool.use() - wrap the user function application with a try-catch block and call giveBack() hoping the connection is marked as non valid
in the MySQLConnectionHandler.writeAndHandleError() return the exception inside the future instead of throwing it

obviously those solutions do not contradict and can be applied both.
what do you think ?

@haski definitely both, didn't consider this case. gonna see if there are other places where we directly raise an exception as well to clean it up.

cool. so are you going to fix it or do you want me to create a PR ?

I'll only be able to look at this at the end of the week (holidays), if you could come up with a PR it would be merged much faster :)

NP. I'll try to cook something up :)


Fix syntax errors in Python examples

Current coverage is 85.48%

Merging #181 into master will not affect coverage as of c41e871

@@            master    #181   diff @@
======================================
  Files           10      10
  Stmts          489     489
  Branches       147     147
  Methods          0       0
======================================
  Hit            418     418
  Partial          0       0
  Missed          71      71

Review entire Coverage Diff as of c41e871

Powered by Codecov. Updated on successful CI builds.

Thanks for the correction.


Update demo with supabase 2.0

Can this also show how to apply the new supabase types generated, since the instantiation of supabase client is abstracted within the composable.
I am referring to
import { createClient } from '@supabase/supabase-js'
import { Database } from 'lib/database.types'

const supabase = createClient<Database>(
  process.env.SUPABASE_URL,
  process.env.SUPABASE_ANON_KEY
)

Demo has been updated so I close this issue. @shiftlabs1 thanks for the proposal, feel free to propose something with types generation :smile:


Figure out swipeState setting inconsistency

@kylebshr: One question I have is that I'm surprised this setter method and the property's didSet are different:
        private var swipeState: SwipeActionState = .closed {
            didSet {
                if oldValue != swipeState {
                    swipeConfiguration?.swipeView.apply(state: swipeState)
                }
            }
        }

Based on UIKit paradigm, I'd simply expect the didSet to call through to a set(state: newValue, animated:false). If the behaviour is meant to be meaningfully different, should the name of this method be different?
Originally posted by @kyleve in https://github.com/kyleve/Listable/pull/136

Crap, forgot to look at this today. I’m on a walk rn but will take a look tonight

The behavior is pretty much the same, since the function sets the property which triggers the didSet. I'm not actually sure how the setter could just call the function, since the function sets state - it would be recursive. If the function didn't need to store the state, it could work that way, but since it needs to store the value I think that's why it's set up how it is?


Contribute to upstream Git repository

Hi @adamcrume!
Thank you for your great work on this project! I was wondering one day about whether someone has ever considered implementing a version of git bisect that is robust against intermittent errors and found this repository.
I would like to kindly check with you on whether you have any plans on merging this upstream (in C, of course). Having it as a built-in feature in Git itself would be nice and much more convenient for software engineers to use. Perhaps, we could also put it behind some optional flag if this is not considered production-ready yet. I am sure that Junio Hamano would welcome this as a feature. 😄
Otherwise, if you deem this as still too experimental, then that's fine too.
Cheers!
James Raphael Tiovalen

I wouldn't say this is experimental, but porting it to C would be a decent amount of work, of course.  The trickiest part is that this algorithm doesn't (yet) support skipping commits, so that would have to be added first if this were to be merged into git bisect itself rather than being a similar but separate command.


Updated projects to .NET 8 and updated references to the latest versi…

Summary of the changes:

Updated projects to .NET 8.
Updated referenced libraries to the latest versions.
Updates to get all tests passing.
Cleaned up some compiler warnings.

Useful Changes ✅
Using latest technology ✅
Likelihood to get merged into Baget ❌


Wrote image links explicitly

This PR will resolve issue #477 .

Thank you, @96RadhikaJadhav! This looks good to me.


ci: Add support for testing a PR via a workflow_dispatch

Add a workflow_dispatch workflow to test PRs using secrets
PR Checklist
GitHub

[x] Commits signed: All commits must be GPG or SSH signed.
[x] DCO Compliance: Developer Certificate of Origin (DCO) applies to the code, documentation, and any example data provided. Ensure commits are signed off.

@adampingel Please review and approve. This change will allow us to run the tests on a PR using the secrets.


[#105] Change the word2vec model dimensions from 500 to 200.

Changed word2vec model to 200 dimension SKIP
Cleaned up STDOUT % output

[#105]

Overriding since Travis passed the build but did not communicate the pass to GitHub.


feat(backup): add backup manager skeleton

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Add backup manager in meta node, as the entry point of backup functionality.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)
https://github.com/risingwavelabs/risingwave/issues/6482

For simplicity of implementation, "reading from meta store and writing to backup store" is done in meta node's backup manager for now. Will refactor later to move it out of meta node.

The recovery part should be handled by the cloud service or any other component?

Recovery will be provided as an external binary which has a dependency on meta store and backup store (no dependency on meta node).
It will be in another PR.

Any SQL commands to issue the backup?

Not yet in this PR. I think it can be supported after stabilizing the backup/restore. Before that we need trigger backup/restore via external binary.

Some type renaming:

Backup of meta node is named MetaSnapshot
Storage for storing MetaSnapshot is named MetaSnapshotStorage, e.g. ObjectStoreMetaSnapshotStorage
Manifest tracking MetaSnapshot is named MetaSnapshotManifest.
Metadata of MetaSnapshot is named MetaSnapshotMetadata. It's used in manifest file.

@Li0k @StrikeW


Fix: Link Checker

GitHub Pages 404.html returns HTTP 404 on the new docsite site but didn't previously(?).
This drastically reduces the effectiveness of having a link checker.

Allowing a 404 response is basically turning the link checker off - we should just go fix the routes on the docsite (which are blocked due to other reasons, unfortunately)

@jkelleyrtp It's not the routes that are wrong. For some reason, the new changes on the docsite have made it no longer work with the 404.html page. The only difference between now and then that I could find is we used a custom index.html instead of the pre-generated one from DX - which would've been copied to 404.html

We set our 404 page on github pages to return the same html as index.html, so technically those routes are returning 404s.
This is because SSR is currently disabled on the docsite while I fix ISRG in #3139

Oh, I finally understand. SSR is generating all the index.htmls for each route. I thought originally that we used the 404.html for everything.


Modern Buffer Overflows Part II

Adds a post to the "Modern Buffer Overflow" series that outlines shellcoding

Rewritten, appended, and ready for review

White space issues fixed

fixup!'ed ready for more review

fixup!'ed

:gopher::squash:

squashed!


volumes

PVC watches podChanges to update its labels, PV use labels from its bound PVC
delete r.reconcileForPVCDeletion(req) in PVC controller, reconcileForPVCDeletion is an old logic to clean pv.claimRef when bounding PVC is deleted to make the PV available to be bound again, it is no longer needed, since reuse PV now works in 2 ways:

for same namespace, reuse PVC to reuse disk
for diff ns, the component controller will delete PV claimRef to make it available again

other minor changes:

compName is saved as a label in PVC
check nil before set headlessService.name

@davidqhr updated, plz check


Fix Minor Typos in README

Thanks for this really useful code.
just a few typos in the example code in the readme
&goeditorjs.BlockParagraphHandler{},
&goeditorjs.BlockListHandler{},
&goeditorjs.BlockCodeBoxHandler{},

I believe should be:
&goeditorjs.ParagraphHandler{},
&goeditorjs.ListHandler{},
&goeditorjs.CodeBoxHandler{},

Thanks for the feedback. This has been closed in 25aad9d25.


Update transformers version

I found a bug in the Trainer class of my current Transformers version (4.21.3). In order to push a model to the hub, inside init_git_repo the line 3248 is initialising a Repository object giving the private parameter. This attribute does not exist in such class, hence causing TypeError: Repository.__init__() got an unexpected keyword argument 'private'
I am not sure about the implications of this version bump, I had the version locked because of Tensorflow. I'll find out tomorrow.

Btw, I checked their master branch and this function is fixed and improved.


Problems with typescript types

Hello, I am working with the typescript library and I found some errors to be able to modify, for example, the navigation buttons that do not support a JSX.Element. Also when I want to modify the active state of the tab it only supports a number. Could it be added so that it admits a string?
Based on this, could you add the following types?
rightBtnIcon?: React.FunctionComponent | string | JSX.Element;
leftBtnIcon?: React.FunctionComponent | string | JSX.Element;
activeTab: number | string;
onTabClick: (e: React.BaseSyntheticEvent, id: number | string) => void;

Hey, of course, thank you for giving me the types already 😅
On the next alpha version, i will add them.

Hello @Mood-al
When will be next alpha version be released?

@sofialancubaREDATLAS  @ahmedatadjust  Hey, I just updated the package to 2.0.3-alpha adding the types and fixing some issues.
if you encountered any issues just open an issue with it!


Can't switch tabs with keyboard when tree view is focused

Running Atom 1.18.0 on OS X 10.10.5
Using keyboard shortcuts ⌘+⌥+← and ⌘+⌥+→ doesn't switch tabs when the tree view is focused.
I have keyboard shortcuts assigned to specific gestures on the mac trackpad and they should work everywhere in the app, regardless of where the cursor is. This is also inconsistent behavior as closing and reopening tabs with ⌘+W and ⌘+⇧+T works as intended.
Steps to Reproduce
With 2 file tabs open and the tree view visible:

Click to focus the tree view.
Use ⌘+⌥+← to switch to previous tab.

Expected behavior: To switch to previous tab.
Actual behavior: Tab switching doesn't work with that key combo.

Thanks @chbk :+1:
I can reproduce with 1.18.0 on macOS 10.12.6, going to tag as a bug for now since it seems inconsistent like you mentioned based on how other functions work with tree view focused.  I went ahead and added steps to reproduce in the issue body as well.

I went ahead andI brought this up with the team and it looks like though some folks agree that it could be useful to be able to switch file tabs with the tree view focused, things are working as expected for a dock like the tree view.  Though it could be a possible enhancement in the future (providing an option that enables this behavior), it's not something that would happen soon so we're going to close it for now.


Add a "oneshot" donation option

I only speak for myself here, but I know there are others that feel the same way.
I'm not a fan of Patreon, and instead of becoming a patron with a small monthly payment (and waste cents every month), I'd much rather donate, say, $50 once (and maybe other times, too if I'm really pleased by something I manage to achieve thanks to a piece of software).
Why don't you open a Pledgie campaign or even a simple paypal donation link?

Thanks for the suggestion, and for the offer of donation: it really is appreciated. I'll take a look at the link you gave and will follow up here.

Github now features some sites other than Patreon on the Sponsor wiki. Maybe there is a good place to look at? Or if the current system works, we can close this too

Hi! We now have a paypal link on the "Other" link which goes to https://www.paypal.com/donate/?token=djgAyLY5JtIxM_zFhiKyVY7glmGOzykV1bhX29yqI-mV60rinUcet6pkhIQAO9TUkN4pVG&country.x=US&locale.x=
Hopefully that covers your use case (assuming it's still of interest). Thanks for your thoughts and again for the kind offer. I appreciate in the last 3+ years your position may have changed :)

Thanks. A donation was made :)

Many thanks!! @iboB :)


Fixed xhamster getNextPage

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a bug fix (Fix #765)

Description
Fixed getNextPage
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Coverage decreased (-0.004%) to 38.574% when pulling 0c4ec95ca4943bb0c067ca33a944a9623913d822 on cyian-1756:xhamsterFixes into cbfd3638425282eeb4cadc15747ff04315486400 on RipMeApp:master.

Coverage decreased (-0.004%) to 38.574% when pulling 0c4ec95ca4943bb0c067ca33a944a9623913d822 on cyian-1756:xhamsterFixes into cbfd3638425282eeb4cadc15747ff04315486400 on RipMeApp:master.


feat: Upgrade chart to latest version

This would fix #8

Actually I noticed that the deployments before didn't respect the interval of 30 seconds and updated every second.
After two deployments in different AWS Accounts, we quickly hit the 5req/sec Rate Limit :D

@toabi could you describe what you mean exactly? How did you hit the 5req/sec route53 limit?

Well, external-dns checked the zone state every second…

Is that a bug in the new external-dns version which is deployed by the bitnami chart?

It was a bug in the old one, using the one in this PR fixed the issue for me.

Great! The pipeline is failing because you haven't executed pre-commit which generates the README doc. We've added some notes about this. Please check https://github.com/lablabs/terraform-aws-eks-external-dns#validation-linters-and-pull-requests. If the instrustions are not clear, let us know.

Finally installed this pre-commit thingy. Nice setup there. This would be now ready-to-merge.

@toabi we are planning to integrate the terraform-docs and fmt into the CI pipeline and commit the changes automatically. Thanks for updating it. The changes look good to me.


[WIP] volumemgr handle content tree parts

Replaces the OCI image all-in-one tar download and then load into containerd at snapshot creation time, with downloading in parts.
Notable changes:

downloads the index (if relevant), manifest, config and layers via downloader, then verifies each via verifier
when all are verified, loads them into containerd
SnapshotPrepare() has been changed to use the content already loaded into containerd

Based on @zed-rishabh's latest work on volumemgr
What does it not do well, or yet:

does not load VMs into containerd. Given the size of them, and that they need media types (which are subject to discussion), I wanted to keep that separate. It also minimizes the changes
it does not check if some element of content already exists in containerd before downloading and verifying. That is relatively simple, but should be a separate PR.
does not remove the old verified files from /persist/downloads/... after loading into containerd. We didn't do it before, so I didn't want to change it here. We definitely can, and it is simple, in a future PR

Thus far, I have tested individually with downloads and verifications, but not kicking off actually ECOs. I will do that in the integration lab with the kind help of @eriknordmark, who offered.
Open questions for reviewers:

load VMs as well: I recommend we leave that for a future PR
remove old layers after verifying and loading into containerd: we should do that in a future PR
are there too many log.Infof() statements? I had to put a lot in for debugging; would be good to get input from reviewers
progress indication is weaker than it was before. I had gotten lots of advancement in the "tar downloader", and now we bypass it. It updates the volume, but only after each blob is downloaded. I recommend we let it go in this PR, and then update in a future one (before the next release).

Finally, there is one "bug". For reasons unknown (yet!), when downloading an image, larger layers appear briefly, start to download, then are marked as missing and disappear. I can recreate it reliably, but do not yet know why it is happening. The whole download then completes except for that layer, eventually kicking in again ~10-15 mins later. This obviously cannot get merged in with that bug, but having extra eyes on it would be very helpful.
I will load up logs into a gist and share it in a comment.

Re: the bug. When downloading the alpine:3.10 referenced in the alpha controller, there are three blobs to download:

21c83c5242199776c232920ddb58cfa2a46b17e42ed831ca9001c8dbc532d22d - the binary layer, ~2.8MB
a143f3ba578f79e2c7b3022c488e6e12a35836cd4a6eb9e363d7f3a07d848590 - the manifest, 528B
be4e4bea2c2e15b403bb321562e78ea84b501fb41497472e91ecb41504e8a27c - the config, ~1.6KB

The smaller layers download immediately, verify, and are good. The bigger layer starts to download - if I watch the downloads dir, I see /persist/downloads/pending/21c83c5242199776c232920ddb58cfa2a46b17e42ed831ca9001c8dbc532d22d briefly, but then it disappears, and doesn't come back for quite a while.
I suspect some kind of race condition. All three layers get started, the small ones immediately kick off an updateStatus, the volumemgr checks the verify status of all three, then the download status of all three, just as the large layer is starting to download, and then deletes it.
I pulled off the entire /persist/rsyslog/syslog.txt, and have it. The attached gist here has the results of grep 21c83c5242199776c232920ddb58cfa2a46b17e42ed831ca9001c8dbc532d22d syslog.txt.
If you follow it through, you can see it starting the download here at 11:55:49, lots of checking the status (I suspect because there were other updates that caused the whole volumemgr to check the statuses of all of its blobs), and then it has a "stat; no such file or directory" error here at 11:55:56, ~7 seconds later.
Any input would be appreciated.

FYI, there are some extra logging statements and even an exec("ls","-l",localFile); these are for debugging purposes and will not be in the final commit.

@eriknordmark I responded to all of your comments. Please take a look.

@deitch Gentle reminder: OldVolumeConfig/Status is going away in a few days.

Yup. But as long as it is there, I need to support it.
@zed-rishabh my tests thus far all used OldVolume; I just created an appinstance from alpine:3.10 and it did it. Is there a way for me to exercise ContentTree?

In master, we are creating both old and new volumes. New volumes are using the content tree and getting stored in persist/vault/volumes. And with this PR https://github.com/lf-edge/eve/pull/1102, we are completely moving to the volume refs in application instance config.

This branch was completely rebased on master, so I’ll assume it is exercising it then.

Yes, you can also verify it from content tree and volume status in var/run/volumemgr/appImg.obj

This has added the following:

rebased on @zed-rishabh 's changes
simplified the verifier dirs somewhat: removed the /persist/downloads/appImg.obj/{verifier|verified}/<sha>/<sha> dir/file structure in favour of a simpler /persist/downloads/appImg.obj/{verifier|verified}/<sha> (just the file). I want to remove the objType too, but that is more work and we will defer
publishes BlobStatus to pubsub, to support the work @adarsh-zededa is doing with #1094

Once integration tests pass, and we get some good eyes on it, this will be good to go.
Follow-in work:

load VMs into containerd
remove all of the transient files from /persist/downloads/verified once they are in containerd; this requires doing initial load on startup and delete updates
more dynamic progress indication. I know how to do this, but I want it as a separate smaller PR


Add functional tests for docs

Use protractor to get the based site navigation tested so we can start continuous deployment.

Any reason why Protractor over Nightwatch.js?
I've used Protractor in an AngularJS project, but I've heard good things about Nightwatch as well.

We've been using it internally in Yahoo for a while now and works pretty well for our needs. I personally haven't used Nightwatch.

@redonkulus Do you guys just enable ignoreSynchronization in order to use Protractor?

@cesarandreu we did, here.  You can see the tests here: https://github.com/yahoo/fluxible.io/tree/master/tests/spec and run them with grunt func.


NoMethodError: undefined method `configure' for JazzFingers:Module

I've included jazz_fingers in the development and test environments.  I'm trying to copy the initializer shown in the README, however, I'm getting the error:
NoMethodError: undefined method `configure' for JazzFingers:Module

I get the same error if I try to run JazzFingers.configure manually in the Rails console.

Which version are you using?

Ah, apparently I was running a really old version (0.5.4).  Good call.

:+1:


Using custom solution file path throws an error when using gitlink

When the actual path to the solution file is not inside the specified source directory, build will fail when running gitlink.
For instance in my own repository I keep the solution file in the root of the repository, except for inside the src directory where both the source code and the unit tests reside.
This works as expected both when restoring nuget packages, and when building the projects.
However when executing gitlink this fails miserably just after posting the following message:
Could not find solution file: C:\projects\cake-transifex\src/Cake.Transifex.sln

Expected
I would expect the call to gitlink uses the same paths as when both when restoring packages, and building the projects.
I won't mind opening a PR for that of course, and keeping it the same as the path used in restore and build.

@AdmiringWorm thank you for reporting this, I will try to get to this tomorrow.

@gep13 I can take care of this when I get home later today if you want

@AdmiringWorm that would be great, thank you!

@AdmiringWorm thanks for making this fix.


Add support for other 338$2 values than rdacarrier  for instanceFormatIds or enable other mappings to instanceFormatIds

The script are counting occurrences of this from now on.

We have not had any clients with this so far. Waiting until any comes along.

Currently there are some checks in the code preventing this. These need to be loosened.

Closing this since the field is really for rdacarrier.


feat(FirstTests)

Summary by CodeRabbit

New Features

Introduced a test suite for the 7WondersEvaluation project, enhancing code reliability.

Bug Fixes

Adjusted the access method for the Sum property in the Index.cshtml file to improve data retrieval logic.

Documentation

Updated the project's technology stack in the README, highlighting the transition to .Net Core and usage of xUnit/Moq.

Chores

Updated .gitignore to exclude build artifacts and cache directories.
Added the 7WondersEvaluation.Test project to the solution, expanding the project's testing capabilities.

@coderabbitai review


Documentation example with setTimeout unclear

The first example from the doc (source code) is unclear:
function timerGame(callback) {
  console.log('Ready....go!');
  setTimeout(() => {
    console.log('Times up -- stop!');
    callback && callback();
  }, 1000);
}

jest.useFakeTimers();

test('waits 1 second before ending the game', () => {
  timerGame();

  expect(setTimeout.mock.calls.length).toBe(1);
  expect(setTimeout.mock.calls[0][1]).toBe(1000);
});

Result:
Ready....go!

Adding jest.runAllTimers() (or jest.runOnlyPendingTimers()):
function timerGame(callback) {
  console.log('Ready....go!');
  setTimeout(() => {
    console.log('Times up -- stop!');
    callback && callback();
  }, 1000);
}

jest.useFakeTimers();

test('waits 1 second before ending the game', () => {
  timerGame();
  jest.runAllTimers(); //// <=======

  expect(setTimeout.mock.calls.length).toBe(1);
  expect(setTimeout.mock.calls[0][1]).toBe(1000);
});

Result:
Ready....go!
Times up -- stop!

The example should state that it does not work (by default users expect examples to work OOB):
test('waits 1 second before ending the game', () => {
  let timerGameDone = false;

  timerGame(() => timerGameDone = true);

  expect(setTimeout.mock.calls.length).toBe(1);
  expect(setTimeout.mock.calls[0][1]).toBe(1000);

  // /!\ The following expectation fails /!\
  // timerGame() does not display "Times up -- stop!"
  // setTimeout() callback is never executed and timerGameDone is false
  // Use jest.runAllTimers() or jest.runOnlyPendingTimers() for
  // the following expectation to succeed
  expect(timerGameDone).toBe(true);
});

Also the sentence:

The native timer functions (i.e., setTimeout, setInterval, clearTimeout, clearInterval) are less than ideal for a testing environment

suggests that native timers work, i.e users don't need to do anything (like calling jest.useFakeTimers()).
("not ideal" means "not perfect", it does not mean "not working")
This is not the case:
function timerGame(callback) {
  console.log('Ready....go!');
  setTimeout(() => {
    console.log('Times up -- stop!');
    callback && callback();
  }, 1000);
}

// Let's use real timers
//jest.useFakeTimers();

test('waits 1 second before ending the game', () => {
  let timerGameDone = false;

  timerGame(() => timerGameDone = true);

  // <=== timerGameDone is false, "Times up -- stop!" is not displayed
  expect(timerGameDone).toBe(true);
});

Showing how this could work without jest.useFakeTimers() would be nice.

Hi, the native timers do work, but using them makes your code asynchronous. You can read more about testing asynchronous code in the docs.

Links to documentation redirect to https://code.facebook.com/

@esantini which links? All links in this issue seems to work for me.

I guess they fixed it or something because they weren't working for me yesterday :(
Should I delete the comments?


How to add test case into run?

Is it possible to add test case into run when I have a test case ID and run ID?
If yes, which method should I use?

It's not possible to add a test case to a run. A TestRail run only contains tests which contain a reference to the original case.

@ipolihas You can try doing this:
yourRun.setCaseIds(youListOfCases) and then runs().update(yourRun).execute()
Still it's not usefull since (I dont know why) I can't retrieve list of previous cases: run.getCaseIds() returns null and every time previous added cases will be removed...


socketio package seems unecessary

In socket_server.py, socketio is imported but unused

I must've forgotten to remove it while testing the server, thanks.

Fixed in this commit


New Device: Delta Pro

Device
Delta Pro
Characteristics

Service
Characteristic
Standard
Permission

Battery
BatteryLevel
✅
Read

Battery
ChargingState
✅
Read

Battery
StatusLowBattery
✅
Read

Outlet AC
On
✅
Read/Write

Outlet AC
OutletInUse
✅
Read

Outlet AC
Battery Level, %
🔲
Read

Outlet AC
Input Consumption, W
🔲
Read

Outlet AC
Output Consumption, W
🔲
Read

Outlet CAR
On
✅
Read/Write

Outlet CAR
OutletInUse
✅
Read

Outlet CAR
Battery Level, %
🔲
Read

Outlet CAR
Input Consumption, W
🔲
Read

Outlet CAR
Output Consumption, W
🔲
Read

Outlet USB
On
✅
Read/Write

Outlet USB
OutletInUse
✅
Read

Outlet USB
Battery Level, %
🔲
Read

Outlet USB
Input Consumption, W
🔲
Read

Outlet USB
Output Consumption, W
🔲
Read

Switch X-Boost
On
✅
Read/Write

Are you ready to test beta version of plugin with added device support?
🔴 No (please explain in the 'More Information' box below)
More Information
Have no device to test on. Can test only in Simulate mode

Hi Pietro,
Thanks for working on this integration! We have massive power outages in my country, so I want to make the HomeKit-integration to disable our electric kettle if we're on ecoflow, and it looks like mostly it works :)
The only major change I made – I added extra check if data is non-undefined: https://github.com/Vadimkin/homebridge-ecoflow/commit/af8f0f0d1e8aa26ac25b146575f6975953b2ef55#diff-7c50714ede734c9092354e5cb3cb81c2512b1fb13f4ba0d0eb42ea63e51f7cc2R62
When plugin is initialized, for the first message I have an event with all proper messages, but after that I see a lot of errors in the terminal.
I added logger (you can see it in the diff), so some examples how it looks at some time:
processQuotaMessage {
  addr: 0,
  cmdFunc: 0,
  cmdId: 0,
  id:<PHONE_NUMBER>204437000,
  version: '1.0',
  timestamp:<PHONE_NUMBER>962,
  params: {
    'bmsMaster.temp': 29,
    'bmsMaster.inputWatts': 0,
    'bmsMaster.outputWatts': 0,
    'bmsMaster.soc': 89
  }
}

processQuotaMessage {
  addr: 0,
  cmdFunc: 0,
  cmdId: 0,
  id:<PHONE_NUMBER>794344700,
  version: '1.0',
  timestamp:<PHONE_NUMBER>118,
  params: {
    'ems.maxChargeSoc': 90,
    'ems.maxCloseOilEbSoc': 100,
    'ems.minDsgSoc': 0,
    'ems.minOpenOilEbSoc': 0
  }
}

Stack trace (my initial idea is that updateInitialValues() has different type of fields, but it looks some other case):
[30/11/2024, 23:01:07] TypeError: Cannot read properties of undefined (reading 'bmsMaster')
    at DeltaProAccessory.processQuotaMessage (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/src/accessories/batteries/deltapro/deltaProAccessoryBase.ts:63:14)
    at Object.next (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/src/apis/containers/mqttDevice.ts:42:49)
    at ConsumerObserver.next (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/node_modules/rxjs/src/internal/Subscriber.ts:161:25)
    at SafeSubscriber.Subscriber._next (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/node_modules/rxjs/src/internal/Subscriber.ts:119:22)
    at SafeSubscriber.Subscriber.next (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/node_modules/rxjs/src/internal/Subscriber.ts:75:12)
    at /var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/node_modules/rxjs/src/internal/Subject.ts:68:20
    at Object.errorContext (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/node_modules/rxjs/src/internal/util/errorContext.ts:29:5)
    at Subject.next (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/node_modules/rxjs/src/internal/Subject.ts:61:5)
    at MqttDevice.processReceivedMessage (/var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/src/apis/containers/mqttDevice.ts:26:27)
    at /var/lib/homebridge/node_modules/@pietrolubini/homebridge-ecoflow/src/apis/ecoFlowMqttApiManager.ts:138:38

I'll test it more and will put my updates here. Thanks again for this enormous set of work!


Not works with youtube link.

For example, I have a link https://www.youtube.com/watch?v=k__gEeVPiLg&ab_channel=SweetGirl.
The parse result is:

Which I thought it should be like this:

Update.
I changed primary parser from MicroLink to JSONLink, it works.
So seems there's some bugs with MicroLink?

Update.
JSONLink not works well with "https://www.shadertoy.com". Couldn't fetch any information.

Maybe could have mode that fetch three methods together then use the longest infomation one?

For Youtube, you can use the share link provided by Youtube. That link will work fine.
For your example, https://youtu.be/k__gEeVPiLg is the link.

https://www.shadertoy.com/ seems working fine.

For Youtube, you can use the share link provided by Youtube. That link will work fine. For your example, https://youtu.be/k__gEeVPiLg is the link.

yes, share link provided by Youtube works.
Maybe could have a mode that fetch three methods at same time, then use the longest infomation one as result? Which will have a highly possibility is the right one

Sometimes the longest information is not the correct information. It depends more on the parser site, rather than the length of the results. Another question is whether should I mix all parser's result to get the longest information. For example, taking the title from one parser and the description from another. It's difficult to find a perfect way, as long as you can not modify the returns from the parser site. The best way I think is to find a better parser.


az network bastion tunnel target_ip_address

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az network bastion tunnel Extension Name: bastion. Version: 0.2.0.
Errors:
The command failed with an unexpected error. Here is the traceback:
_validate_and_generate_resourceid() missing 1 required positional argument: 'target_ip_address'
Traceback (most recent call last):
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 663, in execute
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 697, in _run_job
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 333, in __call__
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "${user.home}/.azure\cliextensions\bastion\azext_bastion\custom.py", line 153, in ssh_bastion_host
    _validate_and_generate_resourceid(cmd, bastion, target_resource_id, target_ip_address)
TypeError: _validate_and_generate_resourceid() missing 1 required positional argument: 'target_ip_address'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az network bastion tunnel --name {} --resource-group {} --target-resource-id {} --port {} --resource-port {}

Expected Behavior
Environment Summary
Windows-10-10.0.19045-SP0
Python 3.10.8
Installer: MSI

azure-cli 2.45.0

Extensions:
bastion 0.2.0
ssh 1.1.3

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

Additional Context

Same issue, unable to access to VMs
Environment Summary
macOS-13.1-x86_64-i386-64bit, Darwin 22.2.0
Python 3.10.9
Installer: HOMEBREW

azure-cli 2.45.0

Extensions:
front-door 1.0.17
azure-firewall 0.14.4
bastion 0.2.0
aks-preview 0.5.128
logic 0.1.6
ssh 1.1.3

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

route to CXP team

@lgranie I've switched temporarily to the Azure Cloud Shell and it works (I know is not the same as using the local terminal)

Workaround by downgrading to Azure CLI 2.40 resolved the issue.
I get a warning "The 'bastion' extension version 0.2.0 is not compatible with your current CLI core version 2.40.0." but at least I can connect again.

@lgranie @bollicino @perttulaurila  The fix is being worked upon and should be made available in the 0.2.1 version of bastion extension. I will keep you posted.

@lgranie @bollicino @perttulaurila As mentioned in the above comment, Could you please try updating your bastion extension (to 0.2.1) ? That should resolve this issue. Please run the below command to update your extension.

az extension update -n bastion

@lgranie @bollicino @perttulaurila As mentioned in the above comment, Could you please try updating your bastion extension (to 0.2.1) ? That should resolve this issue. Please run the below command to update your extension.

az extension update -n bastion

ciao @navba-MSFT
yes it works (sorry for late reply, I was on vacation ⛷).
thank you!

everything is ok
thk


How to pin row to top?

I want to always show a certain row with certain value on top of the table as first row regardless of current sorting. I modified the comparator for one column to always return 1 (should mean it is always the "biggest" value). It works, but when I click to sort based on another column, the same doesn't work again.
Any way to do this?

I got it sorted the way I described. I just needed to put the checks to all comparators.


Issue 146 coherent units

Update annotations including examples for the coherent unit classes.

Minor change in release notes.
I know it's not in the scope of this issue, but is there a reason why CoherentProductUnit, currently defined as
    EquivalentTo:
        gist:ProductUnit
         and (gist:hasMultiplicand only
            ((gist:BaseUnit or gist:CoherentProductUnit or gist:CoherentRatioUnit)))
         and (gist:hasMultiplier only
            ((gist:BaseUnit or gist:CoherentProductUnit or gist:CoherentRatioUnit)))

cannot be simplified to
    EquivalentTo:
        gist:ProductUnit
         and (gist:hasMultiplicand only gist:CoherentUnit)
         and (gist:hasMultiplier only gist:CoherentUnit)

Similarly with CoherentRatioUnit.

I don't understand what that reason could be, the two expressions are logically equivalent, should not matter which goes where. If out of scope here, perhaps create as a separate issue?

I'm in favor of making a separate issue for Boris's suggestion.


Add forward declare for dataview::Pod

Having a forward declaration for the dataview::Pod dependency would be nice to have and prevents the need to deal with mismatched versions.

I pushed the changes to the dev branch so it should be in the next minor release.
What you now do is:
use memflow::prelude::v1::*;

which includes the custom Pod derive macro as well as the upstream dataview dependency.
Alternatively you can include it manually via:
use memflow::dataview::*;

Which includes the proc macro from ::memflow_derive::Pod as well as the dataview crate itself.


Can't integrate Alexa Media Player

Hello,
for some days I have been unable to integrate Alexa media palyer into my Home assistant, when I try to enter my credentials, (which I am sure of, and that they work perfectly on the amazon.it website), does not log in, and writes me in red "Unknown error, please report log info ", please, help !!
my HA is
Home Assistant
0.111.2
the log:
Unknown error:
10:24:35 - Alexa Media Player (WARNING)
alexalogin.login ((<alexapy.alexalogin.AlexaLogin object at 0x7f8e639190>,), {'data': {'email'<EMAIL_ADDRESS>'password': 'XXXXXXXXX', 'url': 'amazon .it ',' debug ': False,' scan_interval ': 60,' include_devices': '', 'exclude_devices':' '}}): An error occured accessing AlexaAPI: An exception of type InvalidURL occurred. Arguments: (URL (''),)
10:24:35 - custom_components / alexa_media / config_flow.py (ERROR)

Please make sure you're using 2.8.6. The log indicates you're probably not. Also please fill out the template completely.

I've the same issue, since when i upgraded my Hassio to 111.3.
Home Assistant version: v. 111.3
Alexa Media Player  Custom components: v. 2.8.6
i tried to remove, the integration and custom component from HACS, and i reinstalled it. but when i try to create the integration and i put in the fields
My email account of amazon
My Password
amazon.it
i have this error
`2020-06-17 08:38:58 ERROR (MainThread) [alexapy.helpers] alexalogin.login((<alexapy.alexalogin.AlexaLogin object at 0x7f5def27d0>,)......
2020-06-17 08:38:58 WARNING (MainThread) [custom_components.alexa_media.config_flow] Unknown error: 0, message='Attempt to decode JSON with unexpected mimetype: text/html;charset=utf-8', url=URL('.......')
`
Any Ideas?

I had the same error...It was like during the login something went wrong...so i tried to login on Amazon website and Amazon ask me to change my password (i really don't know why)...that was the problem...i I followed the steps to change the password and then i added Alexa media player with the new credentials...that worked for me

Same here
I had a working installation which suddenly stopped working....
I noticed that my echo could not been found ... tried to do the configuration process again and failed.
Then I reinstalled alexa_media via HACs
And tried to add the echo again and it is failing.
I can not proceed at all withe amazon.de url ... none works
Error is:
Logger: custom_components.alexa_media.config_flow
Source: custom_components/alexa_media/config_flow.py:191
Integration: alexa_media (documentation)
First occurred: 17:13:11 (6 occurrences)
Last logged: 17:35:52

Unknown error: 0, message='Attempt to decode JSON with unexpected mimetype: text/html;charset=utf-8', url=URL('https://www.amazon.de/ap/signin?showRmrMe=1&openid.return_to=https://alexa.amazon.de/api/bootstrap&openid.identity=http://specs.openid.net/auth/2.0/identifier_select&openid.assoc_handle=amzn_dp_project_dee_de&openid.mode=checkid_setup&openid.claimed_id=http://specs.openid.net/auth/2.0/identifier_select&openid.ns=http://specs.openid.net/auth/2.0&')

Unknown error: 0, message='Attempt to decode JSON with unexpected mimetype: text/html;charset=utf-8', url=URL('https://www.amazon.com/ap/signin?showRmrMe=1&openid.return_to=https://alexa.amazon.com/api/bootstrap&openid.identity=http://specs.openid.net/auth/2.0/identifier_select&openid.assoc_handle=amzn_dp_project_dee&openid.mode=checkid_setup&openid.claimed_id=http://specs.openid.net/auth/2.0/identifier_select&openid.ns=http://specs.openid.net/auth/2.0&')

Again if you want help, provide logs per the template. I can't help based on the information so far.

Again if you want help, provide logs per the template. I can't help based on the information so far.

ALANDTSE Which logs do you need?

Something what could be related is, as it happens at the same time
`Logger: homeassistant.core
Source: core.py:156
First occurred: 17:17:58 (6 occurrences)
Last logged: 17:35:58
Error doing job: Unclosed client session
Error doing job: Unclosed connector

`

ALANDTSE Which logs do you need? I updated as much as I could above

Logs as described here. As you noted, you have to scrub info out before posting.

ALANDTSE Which logs do you need? I updated as much as I could above

Logs as described here. As you noted, you have to scrub info out before posting.

Thanks will do ...

One potential fix for those who are comfortable is to modify alexalogin line 278.
From:
        from aiohttp.client_exceptions import ContentTypeError

To:
        from alexapy.aiohttp.client_exceptions import ContentTypeError

One potential fix for those who are comfortable is to modify alexalogin line 278.
From:
        from aiohttp.client_exceptions import ContentTypeError

To:
        from alexapy.aiohttp.client_exceptions import ContentTypeError

@alandtse Do you know where alexalogin.py is located?

I'm getting a message from Amazon that I have to click a link the send me in a text. I'm doing that and them clicking the submit button on the screen that it's been done, but the integration won't finish setting up. Any suggestions?


About screen formatting fixes

Updates the styling on the About screen to properly format on mobile devices.

https://user-images.githubusercontent.com/53443041/169863742-bcf7add7-3219-4c6c-9096-f972c06a47a0.mov

CONFIRMED!  Much better!


sql support

some of the sql stores add some support for sql queries beyond the native$ method: they allow sql strings or an array containing sql string and params as input to the list$ method.
This has also been codified in the tests, but there's no indication of that in the specs.
Should we keep this functionality? Should it be present among tests?

@rjrodger see here for the tests https://github.com/rjrodger/seneca-store-test/blob/master/store-test.js#L457

no - this is not correct - is not allowed directly :)
a string in this position is always an opaque id
however, we can support { $native:"SELECT..."  } as the query parameter - I think this is a valid use case, as it supports generating entity objects from a native query
note that this feature now applies to all stores, but is OPTIONAL - so we now introduce the idea of optional store features....
frameworks are fun

I'm afraid opaque id feature is not mentioned anywhere, I'll add tests for that.
I'll convert the the sql tests to use the native$ option. This will break backward compatibility on some of the stores.
About being optional to some of the stores, seneca-store-test is already constructed in way that makes easy to import and run only relevant tests (e.g.: basics, but not native sql)

@rjrodger I was wondering, is the "opaque id" supposed to work with the list$ command too?

@rjrodger Additional question: how is { native$: ... } supposed to work?

list$ should be able to take an array of ids as the query... so a degenerate case is a single id - so yes, opaque ids work for list$ too
{ native$:"SELECT * FROM foo" } should execute the query, and then construct -/-/foo entities using the result set.


Add support for providing config values from environment variables

Can you add config vars for Heroku, please?

The documentation is currently mostly platform agnostic, and will probably stay that way since my aim isn't to provide official support for hosting the bot on specific platforms - the bot should work anywhere node and sqlite do, and other platform specific differences are out of the scope of this repository. You can check issues #74 and #119 for discussion/details about hosting on Heroku.

Unless, of course, you mean more generically reading config values from environment variables. If that is what you meant, I'll reopen this and edit the title!

Thats what I meant

Unless, of course, you mean more generically reading config values from environment variables. If that is what you meant, I'll reopen this and edit the title!

I forgot to quote.

It's been possible for a while now 👌


ENH: Make models inherit from base model

Make models inherit from base model.

@effigies @oesteban Following this comment https://github.com/nipreps/eddymotion/pull/166#issuecomment-2058387340, I had a look at inheriting the models from ModelBase. Things were unclear, and I ended up adding my questions as comments in the code. Not good practice for reviewing, but would be grateful if you had a look and commented.
The main point is that the BaseClass' fit and predict methods contain a lot of code that is not being call at all. The initialization method is not used either. The latter may be easy to fix with a call to the superclass init from the child classes, but it is unclear how the code in the fit and predict methods of BaseClass should be reused.
This is also related to issue #174.
So comments would be appreciated.

Gave this another go. Adjusted some docstrings.
More questions/comments (inline and below, long to digest, sorry):

I am not sure whether the parent slots are inherited by the child classes

I am not sure, and how the AverageDWModel __init__ method kwargs documentation is understood, as (i) it only contains kwargs (we may need to tell something like kwargs can contain... - not sure how this is done properly in Sphinx); (ii) the parameters are defaulted to some values if not found. So maybe they can just be list as regular keyword arguments.

Not sure if I follow the timepoint/index rationale for the PET model. Do we have multiple PET volumes for a single session? If we do not, then I am not sure to follow: this would be like requiring DWI volumes from different sessions/timepoints to be aligned, which is different from what we try to do for the DWI case (correcting data from the same session).

Following this comment about DSI, the requirement of a gtab in BaseDWIModel may need to be changed (or create a Cartesian vs Spherical acq base classes). IMHO, this should be addressed in a separate PR, otherwise the PR will get stalled.

The following:
https://github.com/nipreps/eddymotion/pull/176/files#diff-a875f501910044a7d95658fb83740e2c5c6c1693e7e6808703d282441db82be8R117
Looks like it does not apply to PET (see https://github.com/nipreps/eddymotion/pull/176/files#diff-a875f501910044a7d95658fb83740e2c5c6c1693e7e6808703d282441db82be8R382). So should it be moved to the DWI base class?

Sorry to ping you again his morning @oesteban.

Let's get #166 over the final line and then I move onto this.

@oesteban Have gone through the comments. Will wait after this https://github.com/nipreps/eddymotion/pull/176#discussion_r1631915776.
The main difficulty to make this work now lies in https://github.com/nipreps/eddymotion/pull/176/files#r1616403642. Although you answered to the thread, not sure if the question was addressed: the point is that I do not see why DTIModel and DKIModel exist in here; if they are meant to be removed, the issue related to _model_class would go away, and the inheritance would be easier I think, as it is the e.g. TrivialB0Model and AverageDWModel DWI models the ones that we are interested in subclassing from BaseDWIModel<-BaseModel.

@oesteban Had to adjust the test:
https://github.com/nipreps/eddymotion/pull/176/files#diff-b7713b8731a4d1f895063e58f31f8ab54f7bb6ca0e2b1712d85683874d94a6e6R44-R57
due to this now being applied to the TrivialB0Model:
https://github.com/nipreps/eddymotion/pull/176/files#diff-a875f501910044a7d95658fb83740e2c5c6c1693e7e6808703d282441db82be8R284-R287