Error :arguments_wrong_length on preload

Hey
Nice to see an Ecto 3 SQLite library. I'm thrilled about this! :)
I just tried to use it with an existing project and after some modifications (remove ALTER TABLE migrations, remove :map type field) it seemed to work.
Until I hit this error.
Git repo with branch: https://github.com/lawik/noted/tree/exqlite
Trying to run this code:
  def list_notes(user_id) do
    query =
      from n in Note,
        where: [user_id: ^user_id],
        preload: [:tags, :files]

    Repo.all(query)
  end

iex(2)> Noted.Notes.list_notes(1)
[debug] QUERY OK source="notes" db=0.0ms queue=0.1ms idle=1857.2ms
SELECT n0.id, n0.body, n0.title, n0.user_id, n0.inserted_at, n0.updated_at FROM notes AS n0 WHERE (n0.user_id = ?) [1]
[debug] QUERY ERROR source="files" db=11.7ms queue=0.2ms idle=28.1ms
SELECT f0.id, f0.mimetype, f0.path, f0.size, f0.note_id, f0.inserted_at, f0.updated_at, f0.note_id FROM files AS f0 WHERE (f0.note_id IN (?,?)) ORDER BY f0.note_id [[2, 1]]
[error] GenServer #PID<0.411.0> terminating
** (DBConnection.ConnectionError) client #PID<0.487.0> stopped: ** (WithClauseError) no with clause matching: {:error, %Exqlite.Error{message: :arguments_wrong_length}, %Exqlite.Connection{db: #Reference<0.1830523906.2284716050.207239>, path: "database.db", status: :busy, transaction_status: :idle}}
    (exqlite 0.4.1) lib/exqlite/connection.ex:443: Exqlite.Connection.execute/4
    (db_connection 2.3.1) lib/db_connection/holder.ex:316: DBConnection.Holder.holder_apply/4
    (db_connection 2.3.1) lib/db_connection.ex:1272: DBConnection.run_execute/5
    (db_connection 2.3.1) lib/db_connection.ex:1359: DBConnection.run/6
    (db_connection 2.3.1) lib/db_connection.ex:557: DBConnection.parsed_prepare_execute/5
    (db_connection 2.3.1) lib/db_connection.ex:550: DBConnection.prepare_execute/4
    (exqlite 0.4.1) lib/ecto/adapters/exqlite/connection.ex:46: Ecto.Adapters.Exqlite.Connection.prepare_execute/5
    (ecto_sql 3.5.4) lib/ecto/adapters/sql.ex:692: Ecto.Adapters.SQL.execute!/4
    (ecto_sql 3.5.4) lib/ecto/adapters/sql.ex:684: Ecto.Adapters.SQL.execute/5
    (ecto 3.5.8) lib/ecto/repo/queryable.ex:229: Ecto.Repo.Queryable.execute/4
    (ecto 3.5.8) lib/ecto/repo/queryable.ex:17: Ecto.Repo.Queryable.all/3
    (ecto 3.5.8) lib/ecto/repo/preloader.ex:229: Ecto.Repo.Preloader.fetch_query/8
    (elixir 1.11.3) lib/task/supervised.ex:90: Task.Supervised.invoke_mfa/2
    (elixir 1.11.3) lib/task/supervised.ex:35: Task.Supervised.reply/5
    (stdlib 3.13) proc_lib.erl:226: :proc_lib.init_p_do_apply/3

    (db_connection 2.3.1) lib/db_connection/connection.ex:183: DBConnection.Connection.handle_cast/2
    (connection 1.1.0) lib/connection.ex:810: Connection.handle_async/3
    (stdlib 3.13) gen_server.erl:680: :gen_server.try_dispatch/4
    (stdlib 3.13) gen_server.erl:756: :gen_server.handle_msg/6
    (stdlib 3.13) proc_lib.erl:236: :proc_lib.wake_up/3
Last message: {:"$gen_cast", {:stop, #Reference<0.1830523906.2284716033.212081>, %DBConnection.ConnectionError{message: "client #PID<0.487.0> stopped: ** (WithClauseError) no with clause matching: {:error, %Exqlite.Error{message: :arguments_wrong_length}, %Exqlite.Connection{db: #Reference<0.1830523906.2284716050.207239>, path: \"database.db\", status: :busy, transaction_status: :idle}}\n    (exqlite 0.4.1) lib/exqlite/connection.ex:443: Exqlite.Connection.execute/4\n    (db_connection 2.3.1) lib/db_connection/holder.ex:316: DBConnection.Holder.holder_apply/4\n    (db_connection 2.3.1) lib/db_connection.ex:1272: DBConnection.run_execute/5\n    (db_connection 2.3.1) lib/db_connection.ex:1359: DBConnection.run/6\n    (db_connection 2.3.1) lib/db_connection.ex:557: DBConnection.parsed_prepare_execute/5\n    (db_connection 2.3.1) lib/db_connection.ex:550: DBConnection.prepare_execute/4\n    (exqlite 0.4.1) lib/ecto/adapters/exqlite/connection.ex:46: Ecto.Adapters.Exqlite.Connection.prepare_execute/5\n    (ecto_sql 3.5.4) lib/ecto/adapters/sql.ex:692: Ecto.Adapters.SQL.execute!/4\n    (ecto_sql 3.5.4) lib/ecto/adapters/sql.ex:684: Ecto.Adapters.SQL.execute/5\n    (ecto 3.5.8) lib/ecto/repo/queryable.ex:229: Ecto.Repo.Queryable.execute/4\n    (ecto 3.5.8) lib/ecto/repo/queryable.ex:17: Ecto.Repo.Queryable.all/3\n    (ecto 3.5.8) lib/ecto/repo/preloader.ex:229: Ecto.Repo.Preloader.fetch_query/8\n    (elixir 1.11.3) lib/task/supervised.ex:90: Task.Supervised.invoke_mfa/2\n    (elixir 1.11.3) lib/task/supervised.ex:35: Task.Supervised.reply/5\n    (stdlib 3.13) proc_lib.erl:226: :proc_lib.init_p_do_apply/3\n", reason: :error, severity: :error}, %Exqlite.Connection{db: #Reference<0.1830523906.2284716050.207239>, path: "database.db", status: :busy, transaction_status: :idle}}}
State: {Exqlite.Connection, %Exqlite.Connection{db: #Reference<0.1830523906.2284716050.207239>, path: "database.db", status: :busy, transaction_status: :idle}}

I don't have an opportunity to sit down and figure if this is all WHERE x IN y or whether it has to do with Ecto preloads or if my schemas are screwy. It worked fine on Postgres and explodes on this.
If you want to reproduce with my code, set up the app, get a Telegram bot API key (not too complicated if you have Telegram, just talk to the BotFather bot) put that and the bot name in environment variables as noted in the README and use the bot to authenticate and then send a photo. This breaks the list_notes function call with the above error.

It should also be noted https://sqlite.org/np1queryprob.html#n_1_queries_are_not_a_problem_with_sqlite
But this is still something I'll look into.

Interesting, hadn't thought of that. But yeah, the error remains :)

I'm able to recreate this. Working on a fix now.

There is also a set of preload.exs integration tests we can onboard in integration_test/, for testing out the fix.

@kevinlang that is exactly what I was doing. Although I'm surprised there isn't a test within the connection_exs that doesn't check for a preload join. I'm gonna add one there as well.

@lawik a work around right now would be to do
  def list_notes(user_id) do
    query =
      from n in Note,
        select: n,
        where: [user_id: ^user_id],
        preload: [:tags, :files]

    Repo.all(query)
  end

Notice the select:. I think this may be an issue with ecto itself. I need to do some more testing with ecto to figure out where the issue resides.


Add "userChangedPasswordOnPremises" to riskDetail property in riskdetection.md

Instructions: Add any supporting information, such as a description of the PR changes, here.
I observe this value as valid, at least in Entra ID logs (table AADUserRiskEvents).
Please, could you check if it was really missing in the documentation?
Thank you.

[!NOTE]
The following guidance is for Microsoft employees only. Community contributors can ignore this message; our content team will manage the status.

After you've created your PR, expand this section for tips and additional instructions.

do not merge is the default PR status and is automatically added to all open PRs that don't have the ready to merge label.
Add the ready for content review label to start a review. Your PR won't be reviewed until you add this label.
If your content reviewer requests changes, review the feedback and address accordingly as soon as possible to keep your pull request moving forward. After you address the feedback, remove the changes requested label, add the review feedback addressed label, and select the Re-request review icon next to the content reviewer's alias. If you can't add labels, add a comment with #feedback-addressed to the pull request.
After the content review is complete, your reviewer will add the content review complete label. When the updates in this PR are ready for external customers to use, replace the do not merge label with ready to merge and the PR will be merged within 24 working hours.
Pull requests that are inactive for more than 6 weeks will be automatically closed. Before that, you receive reminders at 2 weeks, 4 weeks, and 6 weeks. If you still need the PR, you can reopen or recreate the request.

For more information, see the Content review process summary.

@ep3p , verified; thanks for your contribution.


Add workaround for webgpu blur backend

WebGPU blur sample exposes a GPU memory leak issue in WebGPU.
When using WebGPU canvas as source input to VideoFrame, the GPU memory leaks.
This PR add a workaround for this issue by reading gpu data back to CPU and
use 2D canvas as VideoFrame source input.
Description
Purpose

@huningxin @Jiawei-Shao PTAL, thanks!

@huningxin  and @Jiawei-Shao  PTAL again, thanks!

Yes, I did some test locally. Hope you could also have a try when it is landed.

@shaoboyan , thanks for your work on the workaround and the chromium fix.
I tested your PR, it fixes the memory leak issue. However, I found FPS drop (30FPS -> 20FPS) after applying this workaround. I guess it is because memory is readback to CPU for processing. As you know, this sample is mainly for the investigation of the full-GPU-only video processing pipeline, ideally it should keep the data on GPU along the way.
Given the chromium fix has good progress, I would propose to merge this workaround into a separate branch. People can use it if they meet the memory leak. And once the chromium fix is landed, I'll let people know and use the latest Chromium.
WDYT?

@huningxin  I think it is Ok for me if people are using it. And I think that CL should be landed soon.


[submodule-sync] bot-submodule-sync-branch-24.06 to branch-24.06 [skip ci] [bot]

submodule-sync to create a PR keeping thirdparty/cudf up-to-date.
 If failed, it will remain open until test pass or manually fix.

This is caused by the removal of GTest from libcudf as an exposed dependency.  #1983 was supposed to fix it, but apparently we're still missing something.


Update relative links for Chapter 03 TOC

The relative links for several sections in the table of contents were not functioning. Updated to reflect current anchors on page.

Hi, these links are generated by assemble, actually template-toc. So I fixed it in a way that I do not format these headers as code (which was probably a wrong approach anyhow).
Thx a lot!
Regards
Stefan


Migrating the RN template to Typescript

This issue wants to coordinate the conversation around the existing default template that is used when doing npx react-native init.
Context
It seems that the "inciting incident" for this conversation was a report by someone about the fact that the current templates "contains" some Flow typing https://github.com/facebook/react-native/issues/31936.
Flow is the JS typing of choice at FB, but not as much used anymore in the broader community. This has led @safaiyeh to offer to remove them, which evolved in a conversation which led to this issue.
Current status
Currently, the React Native CLI command init uses by default the template found in this repo (here) as blueprint to generate the project. As indicated above, it indeeds contains some Flow typing.
There is an optional template for Typescript already maintained in the RNC github org: https://github.com/react-native-community/react-native-template-typescript which is also documented in the docs.
That said, the TS typings are maintained separately in the DefinitelyTyped repo. We are not aware (as of now) of any one specific person maintaining them.
Proposal
Given how Typescript is very widely used in the Javascript world, it was proposed to replace the existing template with the ts-based one. Which should basically boil down to

add a tsconfig
add the typescript dependency to package.json
add a dependency on @types/react and @types/react-native

We can do these and leave the file extension as .js, which makes it easy for both vanilla JS users and TS users.
This would mean that, by default (in an hypothetical future, >= 0.66) folks doing npx react-native init would get newly generated projects already set up for Typescript.
It would also mean (I think?) that the separate repo would get discontinued in favour of the folder within the main react-native repo (this one).
Concern
In discussing this option, one main concern surfaced: @types/react-native is currently not maintained in sync with the core repo. In particular, typings live in an external repo outside of the monorepo requires waiting for someone else to review to land.
That said:

it should(could?) be addressable
There is some longer term work to generate the TS typings dynamically, which would remove the need for someone to manually put in effort to update them.
currently the template is not tested (AFAIK) in any CI, so there's no actual blocker to day-to-day development, it would only be a "manual" effort to update the types during a release process
it looks like it should be possible to have some test/pre-release types being set up if the template would start to be tested on CI
as per mention above, making the TS template the default would "add work" to the release process in the RC phase. (but it sounds reasonable to have to handle it)

pinging folks who were involved in the earlier conversations: @NickGerleman, @acoates-ms, @safaiyeh, @radko93, @satya164, @TheSavior

Why can't we just make the init template pure JS, let the consumer decide later what he wants, typescript or flow or pure JS. Why do we want to complicate stuffs?

@ravirajn22 if you don't want to use TS you can simply not care about the extra config file and typescript. It complicates nothing for people who don't want to use TS.

Extra config file, extra dependencies and probably TS warnings the very first time you open app.js file. If you plan to convert this file to app.tsx, then unfamiliar syntax to developers who does not know TS. When there is a simple alternative, I feel its a burden for both react-native template maintainers and consumers.
Anyways I am not maintaining any of these so just my opinion, its up to you people.

When I was a beginner, I found it very confusing when I came across a file written in ts.  I believe that it makes react-native more difficult to use for beginners.

@bell-steven the App.js file itself will stay in plain js. The only change is that the template will be configured to be "out of the box" ready to accept TS/TSX files.

Thanks for setting up the issue. I'll be looking into this the next couple of days to propose a change to the template.

We at Zulip Mobile strongly appreciate Facebook's development of Flow, and React Native's support for it. We're happy to say more about why we prefer Flow, here or somewhere else (maybe on a video call if you'd prefer that). 🙂
That said, it sounds like there might be a reasonable way to lower barriers for TypeScript users, while still offering Flow support for current and new projects that prefer it. Making the template-app App.js into plain JS sounds pretty small and reasonable to me. If doing so, I suggest adding a code comment or a note in a doc saying that people can set up Flow very easily if they want. (And, if you like, saying that TypeScript's initial setup is easy, too, I guess, for the benefit of people who'd prefer that. 😉)

I'm going to close this off as the conversation basically ended up being to just rely on the separate template https://github.com/react-native-community/react-native-template-typescript


Improve Time-to-goal visibility in xp tracker

Is your feature request related to a problem? Please describe.
Visibility of time-to-goal is currently hidden behind hovering the XpInfoBox - this information should be available "at a glance".
An option should be added to make this one of the 4 default labels displayed.
Describe the solution you'd like
Add an option which would add time-to-goal as one of the default labels displayed in the XpInfoBox.
This should be disabled by default.
I believe the "Actions" display is the most suitable to replace if the feature is enabled.
Option:

Implementation:

Would really like this as well, but I would rather want the option to 'add' Time Left not 'replace'.

This has been possible since 3cb2f68725c1e33b1f130f067032b9dc56137a41


[SPARK-7913][Core]Make AppendOnlyMap use the same growth strategy of OpenHashSet and consistent exception message

This is a follow up PR for #6456 to make AppendOnlyMap consistent with OpenHashSet.
/cc @srowen @andrewor14

Merged build triggered.

Merged build started.

Test build #35121 has started for   PR 6879 at commit dd4385b.

Merged build triggered.

Merged build started.

Test build #35123 has started for   PR 6879 at commit 912c0ad.

Test build #35121 has finished for   PR 6879 at commit dd4385b.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #35123 has finished for   PR 6879 at commit 912c0ad.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #35124 has started for   PR 6879 at commit 912c0ad.

Test build #35124 has finished for   PR 6879 at commit 912c0ad.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.


CPU health check failed: Exception: Unsupported platform: msys_nt-10.0-25188-x86_64!

Steps to reproduce

Install and create serverpod app.
Push to myapp_server
Run the main.dart
After a few while you will see the error.

Logs
Connecting to VM Service at http://<IP_ADDRESS>:54149/xO0W_27hZU4=/
SERVERPOD version: 0.9.10 mode: development time: 2022-09-01 21:16:37.094741Z
Insights listening on port 8081
Server default listening on port 8080
Webserver listening on port 8082
CPU health check failed: Exception: Unsupported platform: msys_nt-10.0-25188-x86_64!
#0      _filename
#1      libsysres.<anonymous closure>
#2      libsysres
#3      libsysres (package:system_resources/src/dylib.dart)
#4      SystemResources.cpuLoadAvg
#5      defaultHealthCheckMetrics
#6      performHealthChecks
#7      HealthCheckManager._performHealthCheck
<asynchronous suspension>

CPU health check failed: Exception: Unsupported platform: msys_nt-10.0-25188-x86_64!
#0      _filename
#1      libsysres.<anonymous closure>
#2      libsysres
#3      libsysres (package:system_resources/src/dylib.dart)
#4      SystemResources.cpuLoadAvg
#5      defaultHealthCheckMetrics
#6      performHealthChecks
#7      HealthCheckManager._performHealthCheck
<asynchronous suspension>

System Info
Edition	Windows 11 Pro Insider Preview
Version	22H2
Installed on	31-08-2022
OS build	25188.1000
Experience	Windows Feature Experience Pack 1000.25188.1000.0

Looks like system_resources doesn't have Windows support yet.


fix/disambiguate-property-meaning, make JSON valid in basic-javascript.json

Pre-Submission Checklist

[x ] Your pull request targets the staging branch of FreeCodeCamp.
[x ] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x ] You have only one commit (if not, squash them into one commit).
[x ] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[ ] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[x ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x ] Tested changes locally.
[x ] Closes currently open issue: Closes #8955

Description

… Notation JSON: I revised the explanations from line 4322 to line 4328 in order to define JSON and provide brief explanations. I modified the code on line 4343 by adding quotation marks around the boolean value in order to format the object as valid JSON. This did not break the functionality of the existing published code (as of 6:15 PM on June 11, 2016) when I tested that, but please double check that. Further, for the title Record Collection: I revised the instructions from line 4502 to line 4506 in order to disambiguate the meaning of key from the meaning of property. I added to the notes from line 4510 to line 4511 to provide a link to MDN which explains the push method. I modified the code from line 4517 to line 4538 by adding quotation marks around the keys in order to format it as valid JSON. This change at first DID break the functionality of the existing published code when I tested it. So I changed line 4542 from var collectionCopy = JSON.parse(JSON.stringify(collection)); to var collectionCopy = JSON.stringify(collection);.

@LindsayBrown81 As @erictleung mention on gitter you need to use html links instead of markdown.
Also you have two commits, you need to squash them. Perhaps after you make your next changes, then squash all three before you push again.
Here is the Travis CI error:
error [SyntaxError: /home/travis/build/FreeCodeCamp/FreeCodeCamp/seed/challenges/01-front-end-development-certification/basic-javascript.json: Unexpected token ]]
Looks like a syntax error, somewhere there is an extra or erroneous ]

Thank you so much. Will do on Wednesday.

On Jun 14, 2016, at 8:53 PM, Rob j Loranger<EMAIL_ADDRESS>wrote:
@LindsayBrown81 As @erictleung mention on gitter you need to use html links instead of markdown.
Also you have two commits, you need to squash them. Perhaps after you make your next changes, then squash all three before you push again.
Here is the Travis CI error:
error [SyntaxError: /home/travis/build/FreeCodeCamp/FreeCodeCamp/seed/challenges/01-front-end-development-certification/basic-javascript.json: Unexpected token ]]
Looks like a syntax error, somewhere there is an extra or erroneous ]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@LindsayBrown81 You still need to squash your commits.
Git Squash on FCC Wiki
git rebase -i HEAD~4

Then edit the file changing all the pick's to squash's except the one that describes your fix, that one stays as pick

Rob,
Thank you. As this was my first attempt at squashing, I made a mistake and asked for help on https://gitter.im/FreeCodeCamp/HelpContributors at 3:58.
I pasted my gitter message below which outlines the problem I have created:
@erictleung I made a mistake and need advice on how not to screw up anyone else's PRs. I managed to squash MY commits into one commit; however, when doing git rebase on my own fix/ branch, I reset the HEAD back too many index positions. I assumed that if I entered in a number that was greater than the number of commits I pushed to my own branch, then I would receive some type of error. What it actually did was include commits from 3 other developers that did their commits either weeks before I ever created my own branch or to a entirely different challenge file. I did not make any edits to their code at all, which the diffs reflect. So now I have a PR that says I want to merge 4 commits into FreeCodeCamp: staging. How do I correct this?
Please advise what I should do next to remove those other three developers' commits from my branch and/or PR.
Thank you again,
Lindsay
--------- Original Message --------- Subject: Re: [FreeCodeCamp/FreeCodeCamp] fix/disambiguate-property-meaning, make JSON valid in basic-javascript.json (#9096)
From: "Rob j Loranger"<EMAIL_ADDRESS>Date: 6/15/16 3:47 pm
To: "FreeCodeCamp/FreeCodeCamp"<EMAIL_ADDRESS>Cc: "L Brown"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>@LindsayBrown81 You still need to squash your commits.
Git Squash on FCC Wiki
git rebase -i HEAD~4
Then edit the file changing all the pick's to squash's except the one that describes your fix, that one stays as pick
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Rob,
When I did git rebase -i HEAD~4 and then in VI, changed all the pick's to squash's except the one that describes my fix, I got the following error:
Cannot 'squash' without a previous commit
I really got myself into a pickle. The three commits I tried to squash are ones I did not commit myself. Other developers did them.
My best guess at what I should do next is:
git reset --soft HEAD~1
git rm -r
git push -f
I believe that should put me back to having just my own three commits to squash (includes the two already pushed and the last set of changes I made this afternoon). Does this approach sound good?
Thanks again,
Lindsay
--------- Original Message --------- Subject: Re: [FreeCodeCamp/FreeCodeCamp] fix/disambiguate-property-meaning, make JSON valid in basic-javascript.json (#9096)
From: "Rob j Loranger"<EMAIL_ADDRESS>Date: 6/15/16 3:47 pm
To: "FreeCodeCamp/FreeCodeCamp"<EMAIL_ADDRESS>Cc: "L Brown"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>@LindsayBrown81 You still need to squash your commits.
Git Squash on FCC Wiki
git rebase -i HEAD~4
Then edit the file changing all the pick's to squash's except the one that describes your fix, that one stays as pick
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@LindsayBrown81 If you haven't done anything since the rebase, i would try git reset --hard ORIG_HEAD as git saves your HEAD to ORIG_HEAD when doing rebase, merge and reset

If I use HARD, will that wipe out my local changes I made to basic-javascript.json?

On Jun 16, 2016, at 10:43 AM, Rob j Loranger<EMAIL_ADDRESS>wrote:
@LindsayBrown81 If you haven't done anything since the rebase, i would try git reset --hard ORIG_HEAD as git saves your HEAD to ORIG_HEAD when doing rebase, merge and reset
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Oh yes, sorry. Use --soft
Oops
Rob Loranger
250-589-4806
Saanich, B.C.
www.robloranger.ca
On Jun 16, 2016 11:43 AM, "L Brown"<EMAIL_ADDRESS>wrote:

If I use HARD, will that wipe out my local changes I made to
basic-javascript.json?

On Jun 16, 2016, at 10:43 AM, Rob j Loranger<EMAIL_ADDRESS>wrote:
@LindsayBrown81 If you haven't done anything since the rebase, i would
try git reset --hard ORIG_HEAD as git saves your HEAD to ORIG_HEAD when
doing rebase, merge and reset
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/pull/9096#issuecomment-226575663,
or mute the thread
https://github.com/notifications/unsubscribe/AMqbrlo9jS5JtW1gzox6IaCs_b9nZNEkks5qMZk8gaJpZM4Izqts
.

@LindsayBrown81 Looks good :+1:

Thank you. What an adventure! Someone might want to double check how I reworded lines 4505-4508. I haven't completed that challenge myself because I (and other campers) found the wording so confusing.

@LindsayBrown81, you currently have merge conflicts. Please checkout the tutorial linked :smile:

@atjonathan Okay. I ran git pull --rebase upstream staging. My terminal gave me the message below so at this point, it looks like I need to fix a patch. Would you please point me to a helpful web page about how to do that?
...
Patch failed at 0001 fix/disambiguate-property-meaning, make JSON valid in basic-javascript.json #9096
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem, run "git rebase --continue".
If you prefer to skip this patch, run "git rebase --skip" instead.
To check out the original branch and stop rebasing, run "git rebase --abort".
...
I also double checked to make sure my remotes are set correctly. They look right to me (see below), but please let me know if something is amiss. Thank you!
Lindsays-MacBook-Pro-2:FreeCodeCamp lindsaybrown$ git remote -v
origin	https://github.com/LindsayBrown81/FreeCodeCamp.git (fetch)
origin	https://github.com/LindsayBrown81/FreeCodeCamp.git (push)
upstream	https://github.com/FreeCodeCamp/FreeCodeCamp.git (fetch)
upstream	https://github.com/FreeCodeCamp/FreeCodeCamp.git (push)

@LindsayBrown81, those remotes are fine. http://stackoverflow.com/questions/161813/fix-merge-conflicts-in-git has some useful pointers!

LGTM :+1:
Thanks for contributing :tada:


Tabs: Select tab on click on the entire tab, not just the text

Detailed description

Describe in detail the issue you're having.

Currently you have to click on the text stroke of each tab to activate them which very often leads to clicking on a tab doesn't activate it. It's not even just the text box, you have to click on place where there's a letter beneath or the click won't register.
I think the entire tab header should be clickable including the padding.

Agreed, this should be as easy as moving the (click) binding higher up in the ibm-tab-headers list


Load Dataset from the Hub (NO DATASET SCRIPT)

Load the data from any Dataset repository on the Hub
This PR adds support for loading datasets from any dataset repository on the hub, without requiring any dataset script.
As a user it's now possible to create a repo and upload some csv/json/text/parquet files, and then be able to load the data in one line. Here is an example with the allenai/c4 repository that contains a lot of compressed json lines files:
from datasets import load_dataset

data_files = {"train": "en/c4-train.*.json.gz"}
c4 = load_dataset("allenai/c4", data_files=data_files, split="train", streaming=True)

print(c4.n_shards)
# 1024
print(next(iter(c4)))
# {'text': 'Beginners BBQ Class Takin...'}

By default it loads all the files, but as shown in the example you can choose the ones you want with unix style patterns.
Of course it's still possible to use dataset scripts since they offer the most flexibility.
Implementation details
It uses huggingface_hub to list the files in a dataset repository.
If you provide a path to a local directory instead of a repository name, it works the same way but it uses glob.
Depending on the data files available, or passed in the data_files parameter, one of the available builders will be used among the csv, json, text and parquet builders.
Because of this, it's not possible to load both csv and json files at once. In this case you have to load them separately and then concatenate the two datasets for example.
TODO

[x] tests
[x] docs
[x] when huggingface_hub gets a new release, update the CI and the setup.py

Close https://github.com/huggingface/datasets/issues/2629

This is ready for review now :)
I would love to have some feedback on the changes in load.py @albertvillanova. There are many changes so if you have questions let me know, especially on the resolve_data_files functions and on the changes in prepare_module.
And @thomwolf if you want to take a look at the documentation, feel free to share your suggestions :)

I took your comments into account thanks !
And I made aiohttp a required dependency :)

Just updated the documentation :)
Let me know if you have some comments

Merging this one :)
We can try to integrate the changes in the docs to #2718 @stevhliu !

Baked this into the docs already, let me know if there is anything else I should add! :)


Fix promise-handling bug in creation of planning documents

[#181124054]
Fixes a copy/paste/logic bug in planning document promise-handling which likely explains the occurrence of extraneous planning documents in some circumstances.
@scytacki I also added tests that would have caught the bug, officially closing the barn door. Adding the tests required setting up offline mocking of firebase that should prove useful for writing additional tests down the road.

The database mocking is complex. I could see it being hard to maintain. It would probably fail in a weird way if some code under test makes an unexpected firebase call. But lets see how it goes. :)

It's coupled tightly to the code under test, which certainly makes it more fragile, but is also why I restricted the mocking to within each individual test. At least this way when you're restructuring some code the tests of that specific code are likely to break, but it won't require updates to a shared mock that then cascades to other failing tests. In this particular case, the important point is that it tests the promise-handling logic, which was the point of the exercise.


Cannot get livereload to work

Keep getting this error:

I'm using svelte + tailwindcss 2.0:
This is my rollup.config.js:
import svelte from 'rollup-plugin-svelte';
import resolve from '@rollup/plugin-node-resolve';
import commonjs from '@rollup/plugin-commonjs';
import livereload from 'rollup-plugin-livereload';
import { terser } from 'rollup-plugin-terser';
import postcss from 'rollup-plugin-postcss'

const production = !process.env.ROLLUP_WATCH;

function serve() {
	let server;

	function toExit() {
		if (server) server.kill(0);
	}

	return {
		writeBundle() {
			if (server) return;
			server = require('child_process').spawn('npm', ['run', 'start', '--', '--dev'], {
				stdio: ['ignore', 'inherit', 'inherit'],
				shell: true
			});

			process.on('SIGTERM', toExit);
			process.on('exit', toExit);
		}
	};
}

export default {
	input: 'src/main.js',
	output: {
		sourcemap: true,
		format: 'iife',
		name: 'app',
		file: 'public/build/bundle.js'
	},
	plugins: [
		postcss({ extract: false, use: ['sass'] }),
		svelte({
			// enable run-time checks when not in production
			dev: !production,
			// we'll extract any component CSS out into
			// a separate file - better for performance
			css: css => {
				css.write('bundle.css');
			}
		}),

		// If you have external dependencies installed from
		// npm, you'll most likely need these plugins. In
		// some cases you'll need additional configuration -
		// consult the documentation for details:
		// https://github.com/rollup/plugins/tree/master/packages/commonjs
		resolve({
			browser: true,
			dedupe: ['svelte']
		}),
		commonjs(),

		// In dev mode, call `npm run start` once
		// the bundle has been generated
		!production && serve(),

		// Watch the `public` directory and refresh the
		// browser on changes when not in production
		!production && livereload(),

		// If we're building for production (npm run build
		// instead of npm run dev), minify
		production && terser()
	],
	watch: {
		clearScreen: false
	}
};

And my dependencies:
"devDependencies": {
    "@rollup/plugin-commonjs": "^14.0.0",
    "@rollup/plugin-node-resolve": "^8.0.0",
    "autoprefixer": "^9",
    "postcss": "^7",
    "postcss-import": "^12",
    "rollup": "^2.3.4",
    "rollup-plugin-livereload": "^2.0.0",
    "rollup-plugin-postcss": "^3.1.8",
    "rollup-plugin-svelte": "^6.1.0",
    "rollup-plugin-terser": "^7.0.0",
    "svelte": "^3.0.0",
    "tailwindcss": "^2.0.1-compat"
  },
  "dependencies": {
    "sirv-cli": "^1.0.0"
  }

Any hint on this one?
I have the same issue.

Two items that might help:

Make sure your package.conf has the line: "dev": "rollup -c -w", in your scripts section
Make certain port 35729 is exposed/opened (live-reload loads a module on the backend and listens on port 35729)

Lastly, run npm run dev

Closing this issue due to lack of activity, and because it seems it's something specific about OP's setup. If this issue still exists feel free to let me know.

If anyone still has the problem, I solved it by simply changing the url in the browser from http://<IP_ADDRESS>:8080/ to http://localhost:8080/
This way the livereload worked normally.


advisor-overview Update

I updated the FAQ section under What permissions do I need to access Advisor?: You would need to enable Azure Security Center to recieve security recommendations when you're using Azure Advisor for the first time.
and also the first Note section > [!NOTE]

Azure Advisor generates recommendations for subscriptions where you are assigned the role of Owner, Contributor, or Reader.
Please note: Azure Advisor would not work if your account has been suspended or if you're using a free account.
It's also not available for the DreamSpark account as this service is still in preview.

Hi @falabs, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@falabs : Thanks for your contribution to the Azure documentation! The author, @kumudd, has been notified to review your proposed change.

@falabs Thanks for your input! You bring up a valid point. I am unable to accept your changes as is. I will work with the product group to add the appropriate notes to this topic. Closing this PR.
#Please-close


Max retries exceeded with url, No connection could be made because the target machine actively refused it

I've to write an accurate OCR script in python, and as a proof of concept I'm able to extract text from "Rich Text" PDFs or scanned PDFs using textract. I wish to use tika in python, in order to compare their efficiencies. Here's my basic code and the error that's thrown, and I'm unable to pinpoint the problem.
Also, the error is the same even if I dont use localhost, and pass the URL as mentioned in readme ('http://tika:9998/tika')
import os
import tika

from tika import config
directory = os.path.normpath("C:\\Users\\ajjain\\autom\\Samples")
filename = 'residential-rental-agreement-format.pdf'

tika.initVM()
from tika import parser
parsed=parser.from_file(os.path.join(directory,filename))
print(parsed["content"])

the traceback is
Traceback (most recent call last):
File "test.py", line 10, in
parsed=parser.from_file(os.path.join(directory,filename))
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\tika\parser.py", line 36, in from_file
jsonOutput = parse1('all', filename, serverEndpoint, headers=headers)
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\tika\tika.py", line 316, in parse1
headers, verbose, tikaServerJar, rawResponse=rawResponse)
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\tika\tika.py", line 525, in callServer
resp = verbFn(serviceUrl, encodedData, headers=headers, verify=False)
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\requests\api.py", line 126, in put
return request('put', url, data=data, **kwargs)
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\requests\api.py", line 58, in request
return session.request(method=method, url=url, **kwargs)
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\requests\sessions.py", line 508, in request
resp = self.send(prep, **send_kwargs)
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\requests\sessions.py", line 618, in send
r = adapter.send(request, **kwargs)
File "C:\Users\ajjain\autom\tika_textract\lib\site-packages\requests\adapters.py", line 508, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=9998): Max retries exceeded with url: /rmeta/text (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x05EBDA70>: Failed to establish a new connection: [WinError 10061] No connection could be made because the target machine actively refused it',))

@Tehlonewolf  I have ran into a very similar issue.  For me this was an issue of hitting max retries before the url was presented by the server.  Once the tika-server has 20-30 seconds to setup the url it works fine.

See https://github.com/chrismattmann/tika-python/issues/166 for some thoughts. Looks like a firewall or /etc/hosts (C:\Windows\System32\hosts) or binding on <IP_ADDRESS> issue.

@chrismattmann Thanks for taking the time to look through this. I had to change my approach to OCR from open source to google vision API in favour of simplicity to code and having less dependencies. But thank you for taking the time again.


build linking issue...

When I run the build process 'npm run build' all seems good...
...until I open/run the dist/index.html in a browser.  Nothing happens.
Perhaps there's a 'linking' issue within the build process?  No doubt a simple fix, but I'm just beginning this adventure.
By the way, thanks for providing this workflow!
edit - I'm working with the current/original files; I've made not further adjustments.

@minstrelMan, you must run your index.html from ./dist by server. Put files in the root of the server or install https://www.npmjs.com/package/http-server. Then run server by this command: http-server ./dist and open in browser localhost:8080

Thanks...sorted it out with solution found in issue/question #20.


How to set status of users?

Is it possible to update/set the status (locked, suspended, pending review etc) of a user by the admin? I couldn't find a method to do so.

Thanks for your feedback!
You are right, there’s indeed no method in the Administration interface yet that allows you to do that. Sorry! We should definitely add this in the future.
Until then, changing the status should still be as easy as executing the custom SQL query
UPDATE users SET status = ? WHERE id = ?

with the first parameter being one of the values
\Delight\Auth\Status::NORMAL;
\Delight\Auth\Status::ARCHIVED;
\Delight\Auth\Status::BANNED;
\Delight\Auth\Status::LOCKED;
\Delight\Auth\Status::PENDING_REVIEW;
\Delight\Auth\Status::SUSPENDED;

and the second parameter being the user ID, which can be accessed like this:
$auth->getUserId();

Hope that helps!

Thanks

Bumping this old thread and vouching for function that allows user status manipulation :) . Thank you Ocram!


ClientSecretAuthenticationProvider does not check for expired client credentials and grants authentication

@gnom7 Closing this issue as invalid. Please log a new issue with details and if you feel this is a bug then please provide a test that reproduces the issue.

@gnom7 RegisteredClient.clientSecretExpiresAt was provided for custom implementations to make use of it if required. However, the framework does not use it at the moment so this would be a feature enhancement. Feel free to log a new issue as a feature enhancement, however, this will likely be lower on the priority as we have a few other tasks that we're planning for upcoming releases.


can't run hello-world in dev container

Expected Behavior
I can debug the node hello-world app in a dev container
Actual Behavior

Error: A compatible version of Dapr has not been initialized. You may need to install a more recent version.

Steps to Reproduce the Problem

clone repo in wsl
cd quickstarts
code-insiders .
update dev container image to 1.6.9, reference: Rvm default gid conflicts with user gid syncing
say yes when prompting if you want to open a dev container
open the hello-world folder
run debugger on node app

This is a good case for resurrecting #448

Hey @spottedmahn - sorry about this.  we need to dig into why.  I can see that daprd is not by default where it's expected.  this looks like an incomplete install from dockerfile.
To workaround for now you can do this from the VS-Code Terminal:
dapr uninstall --all
dapr init
You should no longer see the dapr_placement container already exists because the uninstall step cleaned that up.  then you should be able to do this with success:
dapr run --app-id nodeapp --app-port 3000 --dapr-http-port 3500 node app.js

@spottedmahn any chance you want to test and approve the PR I have above?

I believe this is fixed, but please reopen if not


feat: Add support for creating CloudWatch log groups

Description
Add cloudwatch log group management.
Motivation and Context

Resolves #340

Breaking Changes
No.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

looking forward to this PR.

CI is green so merging!


Make the website responsive

The website is currently not responsive, you may make it responsive for mobile and tablet sizes

@Nikhita28  can you assign me this issue ?

Assigned, go on with it.


Deprecated API on build

Hello,
Whent I'm building app this error appear. It don't affect my  app but I'm afraid that at some point my app won't start anymore because of this.
Note: /home/usermon/development/flutter/.pub-cache/hosted/pub.dartlang.org/package_info-0.4.0+13/android/src/main/java/io/flutter/plugins/packageinfo/PackageInfoPlugin.java uses or overrides a deprecated API.

Many thanks in advance

Hi @durduliu2009
I see there's an issue addressing the case you described.
Please follow up on this issue.
I can make a guess that this fix may resolve your problem, but to better address it more detailed info required.
I'm closing the current one as duplicate.
If you disagree please write in the comments
and I will reopen it.
Thank you


chore: quiet dependabot

Followup to #1549
This should hopefully solve the issue.
IDK why the ignore approach wasn't working TBH, this is just a different - hopefully more supported - way of doing the same thing. What the increase-if-necessary strategy does is that it only changes package.json if the new package version falls out of the current range constraint. That should mean that wherever we have carets, we'll only get notified of major updates.

As I said, I don't know either, what you did looked good to me. But maybe this one will do the trick. 🤷


Output mapping functionality

Would you be open to the addition of some form of node mapping functionality added on top of the existing serialization? Here are some use cases that I've run into where something like this could be really useful:

Unwrapping HOC components so that the correct component name can be displayed. It's pretty easy in Enzyme to unwrap the root component but much more difficult (and likely unnecessary) to unwrap descendants. If these could be unwrapped for snapshot output though it would make the output much prettier.
Ignoring specific props. In some cases you may not want to pretty-print complex object graphs being passed through props.

I'm sure there are all sorts of other use-cases that people could come up with. It would be awesome if you could specify a function somewhere that would be called for each transformed node and would have access to the raw node as well as the default JSON output which it could transform.
If this is something you would be amenable to I'd be happy to open a PR.

Hi @jbrantly!
I am open to any kind of improvement, but I don't think I understand your use cases clearly. Can you provide code examples? As far as I know, if you can do something with Enzyme, you can do it with enzyme-to-json 🤔
Thanks!

@adriantoine Hey, I think this would be really useful for people using CSS-in-JS tools. A lot of them are creating a lot of unnecessary noise in the snapshot files.
For example, with styled-jsx:
exports[`<Overlay.Control> click trigger 1`] = `
<div
  className="OverlayControl"
  data-jsx={3299129060}
  onClick={[Function]}
>
  children
  <_class
    css=".OverlayControl[data-jsx=\\"3299129060\\"] {display: inline-block;position: relative;}"
    styleId={4150426368}
  />
  <_class
    css="
                  .OverlayControl .Overlay {
                      display: none;
                  }
              "
    styleId={3597040161}
  />
</div>
`;

Or with glamorous:
exports[`<Button> renders text 1`] = `
<button
  className=""
  data-css-1qmv7zm=""
>
  My Button
</button>
`;

Having less verbose snapshot would certainly be welcomed by a lot of these tools users. Right now they make snapshot a pain to review.

@sboudrias-aa it does look like an interesting feature. @jbrantly has started a PR a while ago, I'll see if I can implement it myself.
Thanks for your feedback.

@adriantoine Another way to ignore specific props is by toMatchObject assertion in Jest, illustrated by the description of a sample project. The best place to start reading is here:
https://github.com/pedrottimark/whimper#interlude-about-snapshot-testing
If you are willing for the API of enzyme-to-json to include mountToDeepJson and mountToShallowJson then I can make a pull request for your review.

Fixed by #56

I'm reopening that as it's not really what was implemented in #56, I'll work on a solution for v3.0 (see #67)

Ok well I'm closing this again 😁 Just to let you know that the mapping feature will be in 3.0 all the details there: https://github.com/adriantoine/enzyme-to-json/issues/67#issuecomment-330368493


Fix a bug in output variable name list, replacing (PR#577)

This PR fixing variable name entry issue#535
Additions

Removals

Changes
include/realizations/catchment/Bmi_Multi_Formulation.hpp
src/realizations/catchment/Bmi_Multi_Formulation.cpp
Testing
Unit Tests
Run ngen tests
Screenshots
Notes

Todos

Checklist

[x ] PR has an informative and human-readable title
[x ] Changes are limited to a single goal (no scope creep)
[x ] Code can be automatically merged (no conflicts)
[x ] Code follows project standards (link if applicable)
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[x ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist (automated report can be put here)

Target Environment support

[x ] Linux

This is the only place where all the data are accessible. The availableData
variable actually contains all variables in the sub-modules as well as
input forcing, so nothing is left out. If you run test on the example
realization files you provided, they all work as they are supposed to.
Somehow, in Bmi_Formulation or Bmi_Module_formulation, not all data are
accessible at the same time, so a validity comparison could not be made.
On the unit test part, I agree that it does look a bit awkward. But the
unit test code uses both original and remapped output variables in the
codes. Since the original output variables is already remapped, it does not
exist anymore, so the ngen code generate a "throw" by design. It seems to
be trapped.
On Tue, Aug 15, 2023 at 10:59 AM Matt Williamson @.***>
wrote:

@.**** requested changes on this pull request.
This is indeed cleaner, but it goes back to only fixing the problem for
Bmi_Multi_Formulation where the problem in #535
https://github.com/NOAA-OWP/ngen/issues/535 can occur in any BMI
formulation. This should be possible to do up in Bmi_Formulation or at
least with only a little duplicate code in Bmi_Multi_Formulation and
Bmi_Module_Formulation
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/ngen/pull/610#pullrequestreview-1578916468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA4SRNTXG32LGOBEMLJ4VTXVOMFJANCNFSM6AAAAAA3RD7PJ4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sure will do.
On Thu, Aug 17, 2023 at 8:24 AM Matt Williamson @.***>
wrote:

@.**** commented on this pull request.
In src/realizations/catchment/Bmi_Multi_Formulation.cpp
https://github.com/NOAA-OWP/ngen/pull/610#discussion_r1297218217:

@@ -292,6 +292,11 @@ string Bmi_Multi_Formulation::get_output_line_for_timestep(int timestep, std::st
throw std::invalid_argument("Only current time step valid when getting multi-module BMI formulation output");
}

// check if a requested output variable name is valid, if not, stop the execution
if (timestep == 0) {

   check_output_var_names();

I think this will work if this function is called at the end of
create_multi_formulation instead of in the time loop... can you try it
there instead?
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/ngen/pull/610#pullrequestreview-1582617377,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA4SRPKE4KIE6TDJFXTHS3XVYLRXANCNFSM6AAAAAA3RD7PJ4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Change columns names

df <- data.frame(old1 = LETTERS, old2 = 1)
df_tbl <- copy_to(sc,df,"df")

df_tbl <- df_tbl %>% dplyr::rename(old1 = new1, old2 = new2)

returns:
> head(df_tbl)
Error: `new1`, `new2` contains unknown variables

Is there an easy way to change the column names? since the regular R's funcitons doesn't work:
new_names <- c("new1", "new2")
names(df_tbl) <- new_names
colnames(df_tbl) <- new_names

>sessionInfo()
R version 3.4.0 (2017-04-21)
other attached packages:
[1] sparklyr_0.6.0   dplyr_<IP_ADDRESS>00

@igornoberto right now you have to do
df_tbl <- df_tbl %>%
  rename(new1 = old1) %>%
  rename(new2 = old2)
df_tbl
# # Source:   lazy query [?? x 2]
# # Database: spark_connection
#     new1  new2
#     <chr> <dbl>
# 1     A     1
# 2     B     1
# 3     C     1
# 4     D     1
# 5     E     1
# 6     F     1
# 7     G     1
# 8     H     1
# 9     I     1
# 10     J     1
# # ... with more rows

Ideally we'd like to do following but it's currently breaking:
df_tbl %>%
  rename(new1 = old1, new2 = old2)
# Error in names(select)[match(old_vars, vars)] <- new_vars :
#   NAs are not allowed in subscripted assignments

You can track the issue at https://github.com/tidyverse/dplyr/issues/2860 and https://github.com/tidyverse/dplyr/issues/2896
Also related: https://github.com/rstudio/sparklyr/issues/678

@kevinykuo
I made the same question on stackoverflow, and user8371915 suggested to use select. And it worked.
https://stackoverflow.com/questions/45514906/sparklyr-change-columns-names-in-a-spark-dataframe
The solution:
> df_tbl %>% select(new1 = old1, new2 = old2)
# Source:   lazy query [?? x 2]
# Database: spark_connection
    new1  new2
   <chr> <dbl>
 1     A     1
 2     B     1
 3     C     1
 4     D     1
 5     E     1
 6     F     1
 7     G     1
 8     H     1
 9     I     1
10     J     1
# ... with 16 more rows

@igornoberto yep that works in this case; please keep in mind that select() only returns the columns specified, whereas rename() returns all columns of the dataframe with the specified one renamed, so if you have a dataframe with many columns but just want to rename a couple, rename() would likely be easier.

To systematically change all column names, see suggestions from @user6910411 using select along with [.dots(setName]
(https://stackoverflow.com/questions/45622262/sparklyr-change-all-column-names-spark-dataframe). It worked for me.

Paging @lionel-... what would be the idiomatic tidyeval way to rename multiple columns with a character vector now that SE verbs are deprecated?

Since select/rename verbs now support character vectors, it should simply be
x <- c(new1 = "cyl", new2 = "am")
rename(mtcars, !! x)

but it doesn't currently work. I have submitted an issue: https://github.com/tidyverse/dplyr/issues/3030.
In the meantime you can do it the usual way by turning your character vector to a list of symbols and splice it in the rename() call:
rename(mtcars, !!! syms(x))

Odd, this works for me, it barked about id but when I put it in `` it worked.
tbl_cache(sc, 'taskrelation_archive')
taskRelation <- tbl(sc, 'taskrelation_archive')
taskRelation <- taskRelation %>%
rename(taskRelationId=id, ts = time_stamp)

colnames(taskRelation)
[1] "taskRelationId" "relationid"     "taskid"         "iswhat"         "isdeleted"      "ts"

Confirmed fixed with devel dplyr. Thanks @lionel- !


Bot forgets who i am

When setting up the bot within the k8s cluster and telling the bot the thornode address, etc - everything is fine. After a few hours (?) tho, the bot seems to "forget" data and when clicking on e.g. 📡 MY NODES the logs show this error:
│ 2020-10-07 08:16:54,217 - telegram.utils.promise - ERROR - An uncaught error was raised while running the promise                                                                                │
│ Traceback (most recent call last):                                                                                │
│   File "/usr/local/lib/python3.7/dist-packages/telegram/utils/promise.py", line 57, in run                                                                                │
│     self._result = self.pooled_function(*self.args, **self.kwargs)                                                                                │
│   File "bot/thornode_bot.py", line 159, in show_thornode_menu_handler                                                                                │
│     show_thornode_menu_new_msg(update, context)                                                                                │
│   File "/bot/helpers.py", line 39, in show_thornode_menu_new_msg                                                                                │
│     user_data = context.user_data if context.user_data else context.job.context['user_data']                                                                                │
│ AttributeError: 'NoneType' object has no attribute 'context'

After issuing the /start command and also adding the Thornode, it works again.

Additionally when adding the bot to a Telegram group, only the person who send the /start command can talk to the bot. The bot ignores all other people's messages and returns the same error as shown above. So it seems like the bot gets only initialized for this single person to have a converstation with. The bot should get initialized regardless of who called /start and then be able to talk to anyone with this state.

As the bot was intended for 1on1 usage, this is desired behaviour.
But we can change it to properly function in groups as well.

This issue still persists in v1.11, after /start @thornode_chaosnet_bot forgets all the nodes i watched before @mr-rooftop

That's correct. Did anybody indicate that this issue is closed?

What? No, it seems the issue still persists.


Down dirty Studio test

Maybe we support Studio??? Not sure yet.
However, this is meant to be run in the console.
You need to get bearer Token from another request
(async () => {
  // Define the start time
  const startTime = Date.now();
  console.log("Start time:", new Date(startTime).toISOString());

  // Define the bearer token and document body
  const bearerToken = "";
  const environment = "sylvain";
  const documentBody = (await SDK.document.getCurrentState()).data

  // Make the initial fetch call
  try {
    const response = await fetch(`https://${environment}.chili-publish.online/grafx/api/experimental/environment/${environment}/output/pdf?&engineVersion=1.4.1`, {
      headers: {
        "accept": "application/json, text/plain, */*",
        "accept-language": "en-US,en;q=0.9",
        "authorization": "Bearer " + bearerToken,
        "content-type": "application/json",
      },
      body: documentBody,
      method: "POST",
    });

    const responseData = await response.json();
    const taskInfoUrl = responseData.links.taskInfo;

    // Function to make GET request to the taskInfoUrl
    const checkTaskStatus = async () => {
      const statusResponse = await fetch(taskInfoUrl, {
        method: 'GET',
        headers: {
          "accept": "application/json, text/plain, */*",
          "accept-language": "en-US,en;q=0.9",
          "authorization": "Bearer " + bearerToken,
        },
      });

      if (statusResponse.status === 200) {
        // Success, log the time
        const endTime = Date.now();
        console.log("End time:", new Date(endTime).toISOString());
        console.log("Duration (ms):", endTime - startTime);
        // console.log(await statusResponse.text());
      } else if (statusResponse.status === 204 || statusResponse.status === 202) {
        // Task is not yet complete, try again after a delay
        checkTaskStatus()
      } else {
        // Handle other HTTP status codes as needed
        console.error("Received unexpected status code:", statusResponse.status);
      }
    };

    // Start checking the task status
    checkTaskStatus();

  } catch (error) {
    console.error("Error during fetch operation:", error);
  }
})();

In a production usecase I would suggest adding a timeout before rechecking to put less strain on the backend and the JS thread


Text snapshots

Being able to take a text snapshot.
For example:
expect(someHtml).toMatchSnapshot()


Why is stable branch ahead of master branch?

The stable branch is ahead of master, which is not a good practice.
Expected Behavior

Make sure that stable only accepts changes that passed the tests. Also, master should be ahead of stable, and stable is behind master.
Current Behavior

This branch is 20 commits ahead, 51 commits behind master.
Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. Python 3.6.1):
Operating System and version:

It's not a good practice, totally agree! But there are times when this is can occur temporarily, as is the case right now. We had to fix something in stable to quickly release a new version, this has broke synchrony between master and stable.
There's a new PR #407 intended to improve this situation though.

The changes are synced, and there is no conflict. We are temporarily holding some files around for various backward compatibility reasons. The situation will improve in a few weeks when we won't have to anymore.


Missing API definition.

Please would you point to the full API documentation for Azure/static-web-apps-deploy@v1
e.g. what are the possible values for action?

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

ID: b4cbd10e-57b8-de97-b84f-6e46f865c662
Version Independent ID: 533b11b0-5ada-f9c6-65f1-5966df142fa8
Content: Build configuration for Azure Static Web Apps
Content Source: articles/static-web-apps/build-configuration.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@Dev1thing Thanks for the feedback! We are currently investigating and will update you shortly.

@Dev1thing Since we have not heard back from you we will now proceed to close this thread. Let us know.


NotImplementedError

Not sure if this is a new issue or if some kind of setting was added to the webui that broke the extension, but I'm getting this error:
NotImplementedError: No operator found for `memory_efficient_attention_forward` with inputs: query : shape=(1, 4096, 8, 40) (torch.float16) key : shape=(1, 8192, 8, 40) (torch.float16) value : shape=(1, 8192, 8, 40) (torch.float16) attn_bias : <class 'torch.Tensor'> p : 0.0 `cutlassF` is not supported because: attn_bias type is <class 'torch.Tensor'> `flshattF` is not supported because: attn_bias type is <class 'torch.Tensor'> `tritonflashattF` is not supported because: attn_bias type is <class 'torch.Tensor'> triton is not available requires A100 GPU `smallkF` is not supported because: dtype=torch.float16 (supported: {torch.float32}) max(query.shape[-1] != value.shape[-1]) > 32 unsupported embed per head: 40

Seems it is an xformers compatibility issue. Any other cross-attention optimizer does not show the error.

Hi Izardy, it does indeed seem to be related to xformers. In fact, it seems to be trying to use flash attention, which I’m not sure is supported. Are you requesting to use flash attention with your command line options or something like that?


[BUG] Ploto is enumerating folders that it should not

Describe the bug
In the config, I'm clearly stating what folders are to be used for plotting.  However, I notice that Ploto is enumerating additional directories that it has no business looking into.
To Reproduce
Run Ploto as a user that only has access to the temp1, temp2 and outdrive directories.  When you try to run it, you will see it break as it tries to look into other directories.
{
"PlotterName":  "Beastly Plottorama",
"PlotterUsed":  "Stotik",
"PathToUnofficialPlotter":  "O:\tmp\ChiaStuff\madMAx43v3r_chia-plotter_win_v0.1.1\chia_plot.exe",
"EnableAlerts":  "true",
"EnablePlotoFyOnStart":  "true",
"ChiaWindowStyle":  "hidden",
"PathToPloto":  "C:\_Projects\Ploto\Ploto.psm1",
"EnableMover":  "false",
"PathsToMovePlotsTo":  "F:\chia_nft_plots,G:\chia_nft_plots,H:\chia_nft_plots,I:\chia_nft_plots,J:\chia_nft_plots,O:\chia_nft_plots,P:\chia_nft_plots",
"DiskConfig":  [
{
"TempDrives":  "D:\chiaTmp\",
"Temp2Drives":  "",
"OutDrives":  "D:\chiaPlots\",
"EnableT2":  "false",
"ReplotDrives":  ""
}
],
"SpawnerConfig":  [
{
"ReplotForPool":  "false",
"ReplotPlotsOlderThan":  "18.6.2021",
"IntervallToCheckInMinutes":  "1",
"InputAmountToSpawn":  "9999",
"WaitTimeBetweenPlotOnSeparateDisks":  "1",
"WaitTimeBetweenPlotOnSameDisk":  "10",
"MaxParallelJobsOnAllDisks":  "2",
"MaxParallelJobsOnSameDisk":  "1",
"MaxParallelJobsInPhase1OnAllDisks":  "1",
"StartEarly":  "true",
"StartEarlyPhase":  "4"
}
],
"JobConfig":  [
{
"KSizeToPlot":  "32",
"BufferSize":  "",
"Thread":  "3",
"Buckets":  "256",
"Bitfield":  "false",
"FarmerKey":  "87b3b3455ed3a3d34ec9cdf96da04e4db0e38462ddf51bc67e55277bba9c69867141a54b655e25db2da7ee87b8fd016e",
"PoolKey":  "",
"P2SingletonAdress":  "xch1wrfc3crmqtdsef0vgzqhzk8hykl8pp7tr0td6vl29taynfjy8unq3jgq76"
}
],
"SpawnerAlerts":  [
{
"DiscordWebhookURL":  "https://discord.com/api/webhooks/84716456547229767456645444796/7_BZdVY0-lvhKhsb4EXw1E56bykHhDVu8FIJTPZVTTLdoBamLM6lBG_r3d_3pTauyttu",
"WhenJobSpawned":  "true",
"WhenNoOutDrivesAvailable":  "true",
"WhenJobCouldNotBeSpawned":  "true",
"PeriodOfReportInHours":  "24"
}
]
}
Expected behavior
Ploto should contain itself to the directories that are explicitly mentioned in the config.
Screenshots

This is same issues as #112


[QUERY] AmqpEventHubConsumer.IdleTimerExpired in Java EventHubConsumer SDK

Query/Question
I am getting ERRORS of the type
The link 'G17:0000000:15_b00c67_1589608397028' is force detached. Code: ServerError. Details: AmqpEventHubConsumer.IdleTimerExpired: Idle timeout: 00:30:00.

This ERROR is not thrown back to the application. Does this mean I am not supposed to handle it?
Along with this, a few other ERRORS are also popping up.

05:32:17 [single-1] ERROR c.a.c.a.i.ReactorReceiver - connectionId[MF_2e3c67_1589612249281] linkName[15_b00c67_1589608397028] entityPath[click/ConsumerGroups/eventhub-sdk-test/Partitions/15] Error occurred in link.
The link 'G17:6442759:15_b00c67_1589608397028' is force detached. Code: ServerError. Details: AmqpEventHubConsumer.IdleTimerExpired: Idle timeout: 00:30:00. TrackingId:af954c3400018d94002b871e5ec0c559_G17_B5, SystemTracker:some-namespace:eventhub:click~16383|eventhub-sdk-test, Timestamp:2020-05-17T05:32:17, errorContext[NAMESPACE: some-namespace.servicebus.windows.net, PATH: click/ConsumerGroups/eventhub-sdk-test/Partitions/15, REFERENCE_ID: 15_b00c67_1589608397028, LINK_CREDIT: 0]
05:48:39 [single-1] WARN  c.a.c.a.i.h.ConnectionHandler - onTransportError hostname[some-namespace.servicebus.windows.net:5671], connectionId[MF_2d05f1_1589608387087], error[Connection timed out]
05:48:39 [single-1] ERROR c.a.c.a.i.ReactorConnection - connectionId[MF_2d05f1_1589608387087] Error occurred in connection handler.
Connection timed out, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
05:49:39 [single-1] WARN  c.a.c.a.i.h.ConnectionHandler - onTransportError hostname[some-namespace.servicebus.windows.net:5671], connectionId[MF_3e0104_1589608297028], error[Connection timed out]
05:49:39 [single-1] ERROR c.a.c.a.i.ReactorConnection - connectionId[MF_3e0104_1589608297028] Error occurred in connection handler.
 Current: 3. Max: 4.
05:53:50 [elastic-20] ERROR r.c.p.Operators - Operator called default onErrorDropped
java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'takeUntil' (and no fallback has been configured)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:289)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:274)
	at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:396)
	at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
	at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
	at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)
05:53:50 [elastic-20] ERROR r.c.s.Schedulers - Scheduler worker in group main failed with an uncaught exception
java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'takeUntil' (and no fallback has been configured)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:289)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:274)
	at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:396)
	at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
	at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
	at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)

I am unable to reproduce it on my own and the issue seems to be happening intermittently with no pattern (or atleast I haven't been able to recognize it).
What am I supposed to do for these ERRORs?
Why is this not a Bug or a feature Request?
Couldn't reproduce it on my own.
Setup (please complete the following information if applicable):

Version of the Library used: 5.1.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

@shubhambhattar any errors that are retry-able will not be surfaced to the calling application unless all retry attempts have failed. For errors that are not retry-able, the calling application will be notified.
In this case, it looks like the receiver has been idle with no events to consume for more than 30 minutes. So, the service closes the connection and the processor client retries to reconnect based on the configured retry policy. The application will be notified through the error handler if all retry attempts fail.

@srnagar The errors that I posted above actually persisted for hours. Also, new messages were available for consumption because the EventHub from which I consumer is always receiving new data. This is why I brought this up.
Retry wasn't happening for hours and the application was only throwing these logs for hours.

@srnagar Any help here would be appreciated. I am getting these randomly and not able to reproduce it exactly. And just to be sure, it doesn't resolve by itself even if left for hours.

@srnagar Got this issue again:
08:05:43 [single-1] ERROR c.a.c.a.i.ReactorReceiver - connectionId[MF_e21315_1590471144782] linkName[16_af18a7_1590471144782] entityPath[eventhub/ConsumerGroups/eventhub-sdk-test/Partitions/16] Error occurred in link.
The link 'G25:17658338:16_af18a7_1590471144782' is force detached. Code: ServerError. Details: AmqpEventHubConsumer.IdleTimerExpired: Idle timeout: 00:30:00. TrackingId:99593e2b00018d950056060a5eccc6cf_G25_B14, SystemTracker:some-namespace:eventhub:eventhub~17407|eventhub-sdk-test, Timestamp:2020-05-26T08:05:43, errorContext[NAMESPACE: some-namespace.servicebus.windows.net, PATH: eventhub/ConsumerGroups/eventhub-sdk-test/Partitions/16, REFERENCE_ID: 16_af18a7_1590471144782, LINK_CREDIT: 0]

And here's the Grafana screenshot for 3 hrs which shows that the application isn't able to recover from this:
https://imgur.com/a/8gGUadW
There are 2 consumers running and one of the consumer is only consuming from Partition 16. After the initial spike, the consumption stopped and never started again. This ERROR was also not propagated to the calling application and as we can see, recovery is also not happening.
Consumer- RPS is a graph which represents number of messages consumer from a partition per second.
Lag is a gauge, hence whatever last value was fetched is being reported throughout.

Till now, I don't have a code to reproduce the issue but the issue is definitely there.
It also says that it's a ServerError and the link was force detached.

@shubhambhattar The error seems to happen when the network connection is lost temporarily and the receiver fails to recover even after the connection is restored. I have a PR open to fix this issue.
https://github.com/Azure/azure-sdk-for-java/pull/11432

Hi @srnagar, any idea when this PR will me merged (and subsequently new version of the SDK will be released)? I am seeing another new type of Error but I feel it is a consequence of this issue:
04:00:32 [pool-2-thread-1] ERROR a.e.h.Error - errorCondition[amqp:connection:framing-error]. description[org.apache.qpid.proton.engine.TransportException: connection aborted], errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
04:00:42 [pool-2-thread-1] WARN  c.a.m.e.PartitionBasedLoadBalancer - Load balancing for event processor failed - errorCondition[amqp:connection:framing-error]. description[org.apache.qpid.proton.engine.TransportException: connection aborted], errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
errorCondition[amqp:connection:framing-error]. description[org.apache.qpid.proton.engine.TransportException: connection aborted], errorContext[NAMESPACE: some-namespace.servicebus.windows.net]

@shubhambhattar The new version is expected to be released next week. I will update this issue when it's released.

@shubhambhattar The new version is released to Maven central.
https://mvnrepository.com/artifact/com.azure/azure-messaging-eventhubs/5.1.1
https://mvnrepository.com/artifact/com.azure/azure-messaging-eventhubs-checkpointstore-blob/1.1.1

@srnagar I think the issue is still present. I am consuming from an EventHub (with 32 partitions) using latest 5.1.1 version of SDK and consumption from some partitions are still stopping after a certain time: https://imgur.com/a/Fio3vs9

@srnagar The issue is still present. I am consuming from an EventHub (with 32 partitions) using latest 5.1.2 version of SDK and consumption from some partitions are still stopping after a certain time:
For all partitions: https://imgur.com/a/ZU51zU1
For partition 6 from which nothing is being consumed: https://imgur.com/a/k4wjOjj
Errors in log:
2020-08-14 20:12:12,474 [parallel-1] WARN  c.a.c.a.i.RetryUtil - Retry attempts are exhausted. Current: 4. Max: 3.
2020-08-14 20:12:12,475 [boundedElastic-19] WARN  c.a.m.e.PartitionPumpManager - Error receiving events from partition 6
Did not observe any item or terminal signal within 60000ms in 'takeUntil' (and no fallback has been configured)
2020-08-14 20:12:12,475 [boundedElastic-19] ERROR c.e.e.i.ProcessError - Error while consuming from EventHub partition 6
java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'takeUntil' (and no fallback has been configured)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:288)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:273)
	at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:395)
	at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
	at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
	at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)
2020-08-14 20:45:11,771 [single-1] WARN  c.a.c.a.i.h.ConnectionHandler - onTransportError hostname[some-namespace.servicebus.windows.net:5671], connectionId[MF_cdeb28_1597415285877], error[Connection timed out]
2020-08-14 20:45:11,772 [single-1] ERROR c.a.c.a.i.ReactorConnection - connectionId[MF_cdeb28_1597415285877] Error occurred in connection handler.
Connection timed out, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
2020-08-14 20:46:11,773 [single-1] WARN  c.a.m.e.i.EventHubConnectionProcessor - Retry #1. Transient error occurred. Retrying after 4511 ms.
Connection timed out, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
2020-08-14 20:46:11,807 [single-1] WARN  c.a.c.a.i.h.ConnectionHandler - onTransportError hostname[some-namespace.servicebus.windows.net:5671], connectionId[MF_a397ff_1597430316894], error[Connection timed out]
2020-08-14 20:46:11,807 [single-1] ERROR c.a.c.a.i.ReactorConnection - connectionId[MF_a397ff_1597430316894] Error occurred in connection handler.
Connection timed out, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
2020-08-14 20:46:17,301 [parallel-1] WARN  c.a.m.e.PartitionBasedLoadBalancer - Load balancing for event processor failed - Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
2020-08-14 20:46:17,301 [parallel-1] ERROR c.e.e.i.ProcessError - Error while consuming from EventHub partition NONE
java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:288)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:273)
	at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:395)
	at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
	at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
	at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)
2020-08-14 20:47:11,808 [single-1] WARN  c.a.m.e.i.EventHubConnectionProcessor - Retry #1. Transient error occurred. Retrying after 4511 ms.
Connection timed out, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
2020-08-14 20:47:11,870 [single-1] WARN  c.a.c.a.i.h.ConnectionHandler - onTransportError hostname[some-namespace.servicebus.windows.net:5671], connectionId[MF_4cbaca_1597437911772], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
2020-08-14 20:47:11,870 [single-1] ERROR c.a.c.a.i.ReactorConnection - connectionId[MF_4cbaca_1597437911772] Error occurred in connection handler.
org.apache.qpid.proton.engine.TransportException: connection aborted, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
2020-08-14 20:47:17,302 [parallel-1] WARN  c.a.m.e.PartitionBasedLoadBalancer - Load balancing for event processor failed - Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
2020-08-14 20:47:17,302 [parallel-1] ERROR c.e.e.i.ProcessError - Error while consuming from EventHub partition NONE
java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:288)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:273)
	at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:395)
	at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
	at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
	at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)
2020-08-14 20:48:11,871 [single-1] WARN  c.a.m.e.i.EventHubConnectionProcessor - Retry #2. Transient error occurred. Retrying after 14575 ms.
org.apache.qpid.proton.engine.TransportException: connection aborted, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
2020-08-14 20:48:11,911 [single-1] WARN  c.a.c.a.i.h.ConnectionHandler - onTransportError hostname[some-namespace.servicebus.windows.net:5671], connectionId[MF_8b5d55_1597437971807], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
2020-08-14 20:48:11,911 [single-1] ERROR c.a.c.a.i.ReactorConnection - connectionId[MF_8b5d55_1597437971807] Error occurred in connection handler.
org.apache.qpid.proton.engine.TransportException: connection aborted, errorContext[NAMESPACE: some-namespace.servicebus.windows.net]
2020-08-14 20:48:27,303 [parallel-1] WARN  c.a.m.e.PartitionBasedLoadBalancer - Load balancing for event processor failed - Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
2020-08-14 20:48:27,304 [parallel-1] ERROR c.e.e.i.ProcessError - Error while consuming from EventHub partition NONE
java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'flatMapMany' (and no fallback has been configured)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:288)
	at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:273)
	at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:395)
	at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
	at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
	at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
	at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)


Missing QEMU tool

Hi,
I'm trying to build the platform and it fails when trying configure qemu-orp.
Can you please share a link to the qemu-orp project?
Thanks,
Ori

What tools are you using to build it ? What HW you want to deploy it on ?
Thanks

I just followied the readme files inside the project to build it.
The HW is not relevant at the moment, that's why I need the QEMU.

I've added qemu-orp to the repository. It contains a README.md with build and usage instructions. Thank you for your patience and please let me know how it works!

The tools and OS were built successfully after this update, thanks!


Handle introductionpoint death (DO NOT MERGE YET)

Kill TorServiceHost and inform it about the death of IntroductionPoints. Currently, TorServiceHost doesn't care about IntroductionPoints' deaths, and continues even when it shouldn't.
This PR  should not be merged because it hasn't yet been debugged/tested.

3 nits:

Please put a description for PR (you can copy+paste from one of the commit messages, the most important one; or do a summary of both).
Please state the reason why this can't be merged yet.
You have a typo in your commit: pionts -> points.

Handle introductionpoint

I believe you meant IntroductionPoint in the title of the PR.

When overwriting the branch of this PR, shouldn't you guys say if it's ready to review/merge or why it's not? @aarani @parhamsaremi

Yes this should be ready to review


Fix/connect home to data

I'd encourage looking at this with fresh eye, versus how we handle balances on the extension. There are a few difference with how we store accounts here, that hopefully makes it a little easier. Though, I do wonder just how much we can reuse (at least in its current state)
I'm thinking something like useQueries rather than useQuery that just accepts an array of all the accounts a user has would be ideal?

I'd encourage looking at this with fresh eye, versus how we handle balances on the extension. There are a few difference with how we store accounts here, that hopefully makes it a little easier. Though, I do wonder just how much we can reuse (at least in its current state)
I'm thinking something like useQueries rather than useQuery that just accepts an array of all the accounts a user has would be ideal?
Generally think it's worth setting the expectation that this is going to be quite hard to get right.

Thanks @kyranjamie 👍 yes I just wanted to flag that it's a non trivial task and could be tricky to get right.
We could also potentially share the solution in the extension where that passes an array with just the one accounts we have there as I guess we will need to support multiple accounts there too.
It seems like something worth discussing and planning as a team to make sure we get it right (I'll close this draft for now)


[ie/manyvids] Fix ManyVids extractor after website update. Fixes #8268

Fix ManyVids extractor after website update

A while ago, ManyVids updated their website and the extractor was broken. I've updated this to use data available from the API instead of scraping it from the page using regex.
Fixes #8268
Template

Before submitting a pull request make sure you have:

[X] At least skimmed through contributing guidelines including yt-dlp coding conventions
[X] Searched the bugtracker for similar pull requests

I found https://github.com/yt-dlp/yt-dlp/pull/9089, but the PR is abandoned, and only addresses teasers/previews. My PR addresses paid content too.

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[X] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

@bashonly I updated the PR a while ago, would be great if this could re-reviewed, thank you!

@bashonly would appreciate a review again when you get a moment, thank you!


Gate: Implement service logic for generic business partner

For the generic business partner model (GBPM, see #382) implement the service logic that is needed by the endpoint.
This includes logic for upserting BPs and querying the input and output stages of BPs.

Should we close this one ? Its already merged @nicoprow


Remove downloaded plugins after installation

Currently we only remove the temporary staging dir, we should also remove ZIP artifacts from the temp folder.

@mesosphere-ci run integration tests


[SPARK-36848][SQL][FOLLOWUP] Simplify SHOW CURRENT NAMESPACE command

What changes were proposed in this pull request?

This is a followup of https://github.com/apache/spark/pull/34104
SHOW CURRENT NAMESPACE is a very simple command that does not involve v2 catalog API, does not need analysis, does not have children. We can simply use RunnableCommand to save defining the physical plan.
Why are the changes needed?

code simplification
Does this PR introduce any user-facing change?

no
How was this patch tested?

existing tests

@huaxingao

Test build #143678 has started for PR 34128 at commit 9222d6e.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48192/

Thanks! Merging to master.

Merged to master for Apache Spark 3.3.

Test build #143678 has finished for PR 34128 at commit 9222d6e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class ShowCurrentNamespaceCommand() extends LeafRunnableCommand

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143678/


Convert Utf8View/BinaryView --> Utf8 / Binary at output

Is your feature request related to a problem or challenge?
Part of https://github.com/apache/datafusion/issues/11752
We are trying to change DataFusion to use StringViewArray by default when reading parquet (and, for example, when it makes more sense such as the substr function), StringView enables many  interesting optimization opportunities. However, as StringView is still being adopted across the rest of the arrow ecosystem, if DataFusion begins to emit StringViewArray in some places, it may cause issues with other parts of the ecosystem (e.g. flight clients may not be able to interpret data sent by a server using DataFusion)
Describe the solution you'd like
I would like DataFusion to retain maximum compatibility at the interfaces, but be able to use StringViewArray internally when it improves performance
Describe alternatives you've considered
I recommend a config flag that makes it possible to convert Utf8View/BinaryView --> Utf8 / Binary at the query output and I think this conversion should be done by default.
For example we might add this configuration flag:
datafusion.optimizer.expand_views_at_output=true

If this flag is true,

add code in the Analyzer (maybe in the TypeCOercion code)
check the output columns of a plan, and if any are DataType::Utf8View or DataType::BinaryView, add ProjectionExecthat converts them to Utf8/Binary (by adding a cast toDataType::Utf8orDataType::Binary` respectively

Additional context
We already have to do something similar in flight with dictionary arrays

I recommend a config flag that makes it possible to convert Utf8View/BinaryView --> Utf8 / Binary at the query output and I think this conversion should be done by default.

Sounds reasonable.
One question -- do we need a flag though?

One question -- do we need a flag though?

My rationale is that converting from Utf8View --> Utf8 is not free. Thus users should have the option of not paying the cost if they want performance over compatibility.
I will admit I don't have a specific usecase in mind

BTW with this conversion in place, I think we could contemplate more interesting changes, like substr always outputing Utf8View even when the input was Utf8

My rationale is that converting from Utf8View --> Utf8 is not free.

it is not
but transmitting non-compacted string views isn't free either.
also, we're transitioning from a state where DF didn't use string views (eager compaction) to a state where DF uses string views (deferred compaction). Thus, if we convert to non-view types on output, we do not risk regressing anything. And we can revisit later whether returning view types could be an improvement (and introduce a flag, if needed & worth it).

Thus, if we convert to non-view types on output, we do not risk regressing anything.

Right -- I agree so this is why in my mind having the default be "Utf8" is important

Thus, if we convert to non-view types on output, we do not risk regressing anything. And we can revisit later whether returning view types could be an improvement (and introduce a flag, if needed & worth it).

I would not be opposed to a PR that just hard codes the existing behavior (convert to Utf8 on output). I changed the title of this PR to reflect this.
I still think a config option offers the most flexibility but I do agree we could add it later if needed

@findepi -- are you already working on this ticket? If not, I would like to pick it up.

@wiedld awesome, go for it

take


[Impeller] AHB to KHR swapchain fallback causes flicker in the Android Google Maps example app.

And likely others.
We should figure out why the fallback happens because the experience falling back isn't good.

cc @jonahwilliams for thoughts on if we understand why this happens.

This is: https://github.com/flutter/flutter/issues/152263


Version for armv8 (aarch64)

Would it be possible to make pagermon compatible with aarch64 to run on Rpi 4 ?
I switched my raspberry that holds all the RTLSDR sticks to Ubuntu 22.04 64 bits and performance is improved.
I tried running pagermon on it but  I get no Pocsag messages.
I guess there must be an incompatibility since there is a specific armv7 version in addition to the standard one.

 - They are compiled against armhf
I can look at building a AArch64 image of rasbian - I don't think it needs anything special

Yes, this would be great!

Hi, does this mean as I'm trying to install this on my RPi4 (64bit) right now, it's not going to work?


Finetuned mBART model for Spanish-English produces lengthy incorrect results for short inputs

❓ Questions and Help
Hi,
I have mBART model that was finetuned on Spanish to English custom data set with lengthy sentences (avg sentence length of about 25 words.) Its working well for long sentences (both on custom data set and general sentences.) However when I type shorts sentences (like 2-3 word ) as inputs , its giving me strange incorrect prediction in a different language.
Here is one for simple spanish input Dame comida. (meaning Give me food ) , which gives incorrect results:

However, long sentences work(like more that 10 words):
e.g. No me gusta mucho el sushi, pero soy fanática de la comida tailandesa. meaning I don't really like sushi, but I am a fan of Thai food.

I played around with several parameters --unkpen, --max-len-a, --max-len-b and --match-source-len etc. But I still could not get rid of the incorrect language predictions.
Is there any way I could troubleshoot what has gone wrong in the model ? Has the model overfitted too much on long sentences that it cannot handle simple short sentences like the one above ? Is there a way to fix this with out retraining the model again ?
Appreciate any help !
Thanks

I have the same problem! It seems mBART doesn't work well for the dataset with short sentences! Can you please help @yinhanliu?

@fahimeh-saleh  Did you happen to get it for the same language pair (Spanish-English) ?
It works fine for English-Spanish. Only in Spanish to English model I see this discrepancy. But, I'm guessing it happens in general from what you say.
Note: I created two model, one for each direction (En-->Es and Es-->En) , by just interchanging the source and target parameters with the same command.


Extensions not loading in dev container

Upon container initial build, get these messages:
Cannot activate the 'Flake8' extension because it depends on the 'Python' extension, which is not loaded. Would you like to reload the window to load the extension?

Cannot activate the 'isort' extension because it depends on the 'Python' extension, which is not loaded. Would you like to reload the window to load the extension?

Cannot activate the 'Pylance' extension because it depends on the 'Python' extension, which is not loaded. Would you like to reload the window to load the extension?

Reformatted devcontainer.json to have python settings in a "[python]" block which seems to help. Still getting Cannot activate the 'Pylance' extension because it depends on the 'Python' extension from 'Microsoft', which is not installed. Would you like to install the extension and reload the window?


Report audit: don't record empty

To support https://github.com/impactasaurus/app/issues/512 and https://github.com/impactasaurus/metrics/issues/2, we need to only audit report generations which are successful. Currently we check that the process didn't error, but we do not check whether the report contained any data. Adjust this logic so the report generation auditing fulfils the intended purpose.

 Onboarding 2.0


Followed the steps in this document in VStudio 2022, and the app throws a fatal exception at: WindowsXamlManager::InitializeForCurrentThread()

Type of issue
Code doesn't work
Feedback
Using Visual Studio 2022, I followed all the steps listed in this article, and the application throws a fatal exception error at the call to WindowsXamlManager::InitializeForCurrentThread(). I searched on the internet for similar reports but couldn't find any. I am at a loss as to what might be causing this. I am using Windows-11.
Page URL
https://learn.microsoft.com/en-us/windows/apps/desktop/modernize/host-standard-control-with-xaml-islands-cpp
Content source URL
https://github.com/MicrosoftDocs/windows-dev-docs/blob/docs/hub/apps/desktop/modernize/host-standard-control-with-xaml-islands-cpp.md
Author
@Karl-Bridge-Microsoft
Document Id
c58a8006-f1f3-4345-d5fb-205b4c782320

My name is Atul Majithia. If you have any suggestions for me to try out, would you please email me at<EMAIL_ADDRESS>