Dynamic form: Fields of type Note don't get disabled

Category
[ ] Enhancement
[x] Bug
[ ] Question
Version
Please specify what version of the library you are using: [3.9.0]
Expected / Desired Behavior / Question
Adding internal field names to the disabledFields property should disable the field when the form is shown.
Observed Behavior
When the internal name of a field of type Note gets added to the disabledFields array it does not get disabled.

I had a look at the code to see if I could fix this myself but am a bit stumped right now. DynamicField.tsx tries to disable the RichText control using this property: isEditMode={disabled} which should be !disabled I believe. But regardless of the field being in the disabledFields property it is always enabled. I debugged the code to the point where the RichText control gets created and verified that it gets created with isEditMode true and false resulting in the same behavior (editable control). So there seems to be a second bug (the first being the missing ! in DynamicField) which is in the RichText control.


Delete docs manifest

This should help the docs tests pass.

Looks like it may still fail, but probably better not to keep that manifest around anyways!
I wonder if there's something odd about the registry server that doctests are using, because the main CI tests seem to see the new version just fine


Messages won't send

Need to fix soon. not sure if project or library issue.
If anyone knows the problem, help is appreciated :)

The issue probably came to be at v<IP_ADDRESS>-dev
please use discordsharp-v<IP_ADDRESS>-dev for the time being


Move away from emails in typing indicators.

We're moving toward having the client send user_ids.  @eeshangarg will take this further in his branch related to id/email changes for regular messages.

One thing worth noting on this is that the Zephyr mirroring setup will still need to use emails for this API.

(I guess one possible alternative solution to that problem is that if we are trying to send to an email address not present in people.js, we could do a call to the server to try to autocreate a Zephyr mirror user for that email, rather than having the send_message code path itself create the user; but that's it own larger refactor.)
We don't need typing notifications for not-yet-generated users, so possibly that isn't relevant for this PR, just for follow-up work in the direction of doing this same change for compose itself.

It's not clear to me whether further action is needed on this.  For the zephyr use case, the call to get_user_ids() should just silently return either nothing or just the ids that map to existing users, which should be fine for typing indicators, correct?
BTW this is related to #10391.  If we can merge this, then Eeshan can cut over to ids for typing indicators pretty easily on that branch.

@showell I think that's not quite right for the zephyr case; here's the scenario I'm imagining: A User starts composing to OldUser (already has a Zephyr account) and NewUser (does not).  get_user_ids will return just the ID for OldUser, and so we'll send typing notifications as though the user is composing a PM to just OldUser (unless I'm missing something).  If they send a message, NewUser will have a mirror_dummy account created and all will be well, but until then, we send an invalid typing notification.  So I think the correct change here is that either we use a modified version of get_user_ids or some at other stage we should confirm that all of the user IDs converted into actual users; if any didn't, we should just skip sending the typing notification.
The reason I care about this is not so much Zephyr in particular, but that this code path is what we'll need if we ever do future features where the browser doesn't have data on all users in the organization.

I think the worst case scenario is that Alice is typing to Bob/Carla, Bob has a known user id, and Carla is unknown.  Carla won't be any the wiser, which is probably a total non-issue, since she's presumably not even on Zulip yet.  Bob will see something like "Alice is typing you a message", which I agree is slightly misleading.  Possibly the change here is to just see if there's an unmappable email and punt altogether.

The latest batch of commits verifies that the user has only valid user-id-based pills before sending typing indicators.  This prevents false positives for Zephyr and situations where you may still be drafting your PM list (don't want to tell the wrong "Tim" you're typing).
For the Zephyr case presumably once the conversation starts, users will get real user ids, so typing indicators will kick in at that point.

I merged all but the last commit.  For the last commit, can you explain why removing compose.get_invalid_recipient_emails is correct?  It seems to check various (haven't read too carefully).

The reason you can get rid of get_invalid_recipient_emails is that if you get past the other new check (has_unconverted_data) you can rely on the pills to have user_ids that represent valid emails (insofar as the client can validate them, the ultimate arbiter is still the server).

I see, because that eventually calls:
exports.is_valid_email_for_compose = function (email) {
    if (people.is_cross_realm_email(email)) {
        return true;
    }

    var person = people.get_by_email(email);
    if (!person) {
        return false;
    }
    return active_user_dict.has(person.user_id);
};

Hmm, comparing with the user_pill.js code which uses people.get_by_email, do we handle sending a PM to cross-realm bot users like welcome-bot correctly here?

Hrm.  I think this is somewhat orthogonal to my PR, since any problems with the pill logic would already be bugs, and my PR doesn't change anything.
I think that validation function is clumsy but correct.  In order for is_cross_realm_email to return true, it must be the case that get_by_email returns true:
636 exports.is_cross_realm_email = function (email) {
637     var person = people.get_by_email(email);
638     if (!person) {
639         return;
640     }
641     return cross_realm_dict.has(person.user_id);
642 };

And that's what we call in `user_pill.js`:

8 exports.create_item_from_email = function (email, current_items) {
9     // For normal Zulip use, we need to validate the email for our realm.
10     var user = people.get_by_email(email);
11

So user pills should allow cross-realm emails.

It's possible we're letting non-active in-realm users get pilled, though?

And then there are the other pill uses cases, like making search and user groups, but they seem to use different functions.

OK, makes sense, merged, thanks @showell!  We should definitely investigate the story with deactivated users and pills.  @synicalsyntax @shubham-padia FYI for that discussion.
I think the correct behavior depends on the context; for search, we probably want deactivated users, but obviously not for compose or user groups.


Added signing utilities for vc-jws, with the appropriate content type.

This fixes https://github.com/TBD54566975/ssi-sdk/issues/272

Codecov Report

Merging #273 (5b4d48c) into main (5743e2e) will decrease coverage by 0.14%.
The diff coverage is 45.24%.

@@            Coverage Diff             @@
##             main     #273      +/-   ##
==========================================
- Coverage   60.48%   60.34%   -0.14%
==========================================
  Files          39       40       +1
  Lines        4398     4440      +42
==========================================
+ Hits         2660     2679      +19
- Misses       1311     1328      +17
- Partials      427      433       +6

Impacted Files
Coverage Δ

crypto/jwt.go
48.50% <0.00%> (-1.24%)
:arrow_down:

credential/signing/jws.go
51.35% <51.35%> (ø)

I wonder if we could add a go build tag for "experimental" or similar.

We could do this. A simpler approach would be to make that known in the godoc of the functions introduced. Thoughts?

agree with that


Cherry-picking : Allow the memberlist shutdown even if networkdb leave fails

Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

LGTM


engine: Detect bad return value from app start() method

Raise a specific error instead of allowing a bad value to propagate in the engine (leading to a more confusing error message).

Merged into max-next.


Device api keys are no longer used in the registration process

The commit message is a little off btw, since you can still pass a deviceApiKey to use whilst registering, just the SDK doesn't support it

@telphan is there a reason this hasn't been merged?


Support for LLaMA or Baichuan models

Hi, I was wondering if this method can be used for trimming large vocabulary in LLMs. Can vocab trimmer be extended to LLMs?

It should be possible to apply vocabulary trimming (VT) to mono-lingual LMs, but it would have little affect on the model size. Mainly because for monolingual LLMs, the ratio of embedding matrix to the whole parameters is very little, and in fact their vocabulary is already quite small (eg. LLaMA has 32k tokens).
The reason why VT is effective for small multilingual LMs (XLM, mT5, mBART), is because the ratio of embedding matrix to the whole model parameter is very dominant (eg. the ratio is 80% for mT5 small), and their vocabulary is extremely large (eg. mT5 has 250k tokens).
Although, it sounds an interesting direction to apply VT to monolingual LMs! I might work on it if I would be more convinced and had some spare time on it!

I agree it has a little effect on LLaMA. But for bilingual LLMs such as Baichuan-2 has 120k tokens. Many multi-lingual LLMs are emerging and this technique would be very useful to make them monolingual and users can finetune in their target languages without worrying about other languages.
I am currently trying to to extend the library to Baichuan-2. Facing many issues and debugging them... I will release it here after I finish that. May be we can extend this to any LLM on hugging face using Auto Casual LM inference.

Support for MistralForCausalLM would also be great!


Adding the "Say" block first causes a "server error"

Describe the bug

A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

Select ev3/ev3dev
Add a Say block
Click on <> to see the code
A popup "Attention Server-Error" appears

If I add a different block first (e.g. Drive forward), it works.
Expected behavior

A clear and concise description of what you expected to happen.

Screenshots
causes "server error"

works:

Device information

Type Desktop
OS: debian linux
Browser: chrome 78.0

After some investigation this happens because I removed the isSayTextUsed() and replaced that by adding a dummy VOICE actor to the used actors when the say block is used. Then, when creating the used actor map no corresponding configuration component exists.
I think the real solution would be to add an internal configuration component for the voice capability, however this would only be easy for the new configuration.
For now there are two workarounds:

go back to isSayTextUsed()
check for the VOICE actor during map creation

I think the second option is best for now as it can be removed once we switch to the new configuration.

Works with say block as first block, though it might need a corresponding config block in future (e.g. language setting)


Incorrect error message

Hi.
Sinek could produce incorrect error messages while loading native rdkafka client. E.g. here https://github.com/nodefluent/node-sinek/blob/master/lib/librdkafka/NProducer.js#L73 original error is swallowed and new custom error is thrown.
However as rdkafka is native module it could throw not only "Module not found" exceptions. It may also throw exceptions like: "Error relocating...  symbol not found". And this is important to preserve original error message or at least show it in debug.

Hi @SkeLLLa good find, I have released @6.16.0 with additional logs for this; also the error message is appended.


Fix revive related linting issues

Changes
Somehow, these slipped when I was moving things over to revive.
/assign @dprotaso

/lgtm
/approve


<C->

how to close <C->
use toggleterm and vim-tmux-navigator
i dont need dotoprevieious window how close it?

There's a handful of notes in the README about customizing the keybindings -- see this section on TPM installation (if not using TPM you can just remove the keybinding from your tmux.conf) and this section on configuring the vim keybindings as examples


Can I use github application token instead of PAT?

Hi Chinamy,
As described in GitHub official document, the runner's installer only supports PAT, so this wrapper can't use application token.


Add initial value to empty object for rkeconfig.configMap[hostname] d…

Summary
issue
Fixes https://github.com/rancher/dashboard/issues/8849
Areas or cases that should be tested

Create custom rke2 cluster without select the Skip TLS certificate option.
Edit the rke2 cluster and select the Skip TLS certificate option.
When saving the RKE2 cluster, it is found that the "Skip TLS certificate" value is not passed to the backend.

Areas which could experience regressions

Vue.js's watcher mechanism is based on the getter and setter of JavaScript objects. When an object is set to be reactive, Vue rewrites the getter and setter of the object through Object.defineProperty() or ES6's Proxy so that it can receive notifications and trigger the corresponding watcher when the property value changes. The issue occurs when we bind an empty object to the data object of a Vue instance because Vue does not automatically set the getter and setter for the properties of the object. Therefore, when we add a property to the object and assign it a value during component creation, Vue is not aware of this change because the property of the object has not been set to be reactive. As a result, it is necessary to initialize the corresponding property value.

This is a general bug fix, no special UX required

@smallteeths Please note we are approaching bug complete for our next milestone 2.7q3

@smallteeths looks good! i'll shepherd the PR through our CI and merge once passed.

@smallteeths I don't have permission to push changes to this PR. Could you make the following change
--------------- cypress/e2e/po/pages/explorer/api-services.po.ts ---------------
index dec45a912..c2786e126 100644
@@ -16,5 +16,5 @@ export class APIServicesPagePo extends PagePo {

   waitForRequests() {
-    APIServicesPagePo.goToAndWaitForGet(this.goTo.bind(this), ['/v1/apiregistration.k8s.io.apiservices']);
+    APIServicesPagePo.goToAndWaitForGet(this.goTo.bind(this), ['/v1/apiregistration.k8s.io.apiservices?exclude=metadata.managedFields']);
   }

It will be fixed in master soon, but this will avoid a rebase when it does

@smallteeths I don't have permission to push changes to this PR. Could you make the following change
--------------- cypress/e2e/po/pages/explorer/api-services.po.ts ---------------
index dec45a912..c2786e126 100644
@@ -16,5 +16,5 @@ export class APIServicesPagePo extends PagePo {

   waitForRequests() {
-    APIServicesPagePo.goToAndWaitForGet(this.goTo.bind(this), ['/v1/apiregistration.k8s.io.apiservices']);
+    APIServicesPagePo.goToAndWaitForGet(this.goTo.bind(this), ['/v1/apiregistration.k8s.io.apiservices?exclude=metadata.managedFields']);
   }

It will be fixed in master soon, but this will avoid a rebase when it does

done


KAFKA-14397; Don't reset producer sequence number after delivery timeout

When a batch is failed after reaching its delivery timeout, we do not necessarily know whether the broker actually committed it to disk. For example, in the case of a REQUEST_TIMEOUT error, it is possible the leader wrote the batch locally, but failed to commit it to the ISR before expiration of the timeout. Later that batch could still become committed. In this case, the producer should not adjust the sequence numbers of subsequent batches until we know whether or not they could also have become committed.
Prior to this patch, had logic to handle this case when the retry limit was reached, but we did not update it to account for delivery timeouts as well. This patch fixes the issue and tries to improve test coverage.
Note this bug was caught by the new tests added in https://github.com/apache/kafka/pull/12873.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Note I'm holding off on merging this patch because I realized when writing the test case that Justine suggested that the code was not correctly handling the case when the next in-line batch fails due to retries or timeouts correctly. Basically we do not reset the next expected sequence number and some other state. I'll try to work on this in the next couple weeks.


an error re type definitions

An attempt to extract id property from request.params object results in an error saying Property 'id' does not exist on type 'Obj | undefined'. Further attempt to fix it by adding <{id:string}> results in another error saying Type 'Obj | undefined' has no signatures for which the type argument list is applicable..
Could you please suggeset what would be the best thing to do in  this kind of situation?

Closing this, as all previous TS -related issues may or may not apply in the new version 3 (released today)

We'll re-open if it remains an issue!


BUG: index out of range

https://github.com/mathnet/mathnet-numerics/blob/f54d7192168a99bcb8c0e2f1ede506cfd1ac28d6/src/Numerics/Random/Xoshiro256StarStar.cs#L161
for buffer length < 8 out of range exception raises because of "buffer.Length - 3" it sould be  "buffer.Length - 7" ulong is 8 byte.
line should be for (int bound = buffer.Length - 7; i < bound;)
Test code
var buffer = new byte[7];
var randStarXX = new MathNet.Numerics.Random.Xoshiro256StarStar(9999);
randStarXX.NextBytes(buffer);

Fixed in #897. Thanks for reporting!


bug: import keyfile fails

Describe the bug
when trying to import a keyfile, it errors with Request body deserialize error: missing field password_hash at line 1 column 602. this is likely due to a field being changed on the frontend but not the validation endpoint
To Reproduce
Steps to reproduce the behavior:

Import keyfile
Fail

Expected behavior
Importing the keyfile successfully.
Desktop (please complete the following information):

OS: Linux
Browser Chrome
Kinode version 0.6.2 (also happens on later versions)

Additional context
request body looks like:
payload: <the keyfile here>
password: <the password hash here>

but it expects password_hash instead of password

Thanks for this report. Definitely an oversight when we changed the field name a couple versions ago. Will be fixed in next release


Driver ignores errors raised in stored procedure (with multiple statements)

Driver version
6.4.0, 7.4.1
SQL Server version
Microsoft SQL Server 2016 (SP2-CU8) (KB4505830) - 13.0.5426.0 (X64)   Jul 21 2019 21:16:47   Copyright (c) Microsoft Corporation  Standard Edition (64-bit) on Windows Server 2016 Standard 10.0  (Build 14393: ) (Hypervisor)
Client Operating System
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.18362 N/A Build 18362
JAVA/JVM version
1.7, 1.8
JDBCDriverTest.zip
java.logging.0.log.0.zip
Table schema
any schema
Problem description

Expected behaviour:
Driver passes to the caller every error raised during stored procedure execution
Actual behaviour:
For stored procedure with single statement driver passes the exception as expected.
For SP with multiple statements, if error is raised in any but first statement, then the exception is ignored.
Error message/stack trace:
For example "ignoring database error:" - full logs attached
Any other details that can be helpful:
In the detailed logs of the driver there is a message “ignoring”.
There is a workaround for the issue – you can use “SET NOCOUNT ON” at the beginning of the stored procedure body, however this sometimes can be problematic. Especially when migrating  project with multiple stored procedures (without “SET NOCOUNT ON”) to the Microsoft JDBC driver (from jTDS).
The problem exists in version 6.4.0 (for JRE 7 and JRE 8), as well as in the 7.4.1.

JDBC trace logs
Full trace logs attached, the "ignored" entries filtered below.
2019-10-24 14:15:41,178 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@3619e211 (SQLServerCallableStatement:3 executeXXX): close ignoring error processing response: Violation of PRIMARY KEY constraint 'PK_Parent'. Cannot insert duplicate key in object 'dbo.Parent'. The duplicate key value is (1).
2019-10-24 14:15:42,497 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@7660aac9 (SQLServerCallableStatement:11 executeXXX): close ignoring error processing response: The INSERT statement conflicted with the FOREIGN KEY constraint "FK_Child_ParentId". The conflict occurred in database "TEST_SP", table "dbo.Parent", column 'Id'.
2019-10-24 14:15:43,299 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@22f573e3 (SQLServerCallableStatement:17 executeXXX): close ignoring error processing response: Cannot insert the value NULL into column 'NumberValueSmall', table 'TEST_SP.dbo.Parent'; column does not allow nulls. INSERT fails.
2019-10-24 14:15:43,584 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@51c7b45e (SQLServerCallableStatement:19 executeXXX): close ignoring error processing response: Violation of PRIMARY KEY constraint 'PK_Parent'. Cannot insert duplicate key in object 'dbo.Parent'. The duplicate key value is (1).
2019-10-24 14:15:43,916 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@7886d48e (SQLServerCallableStatement:21 executeXXX): close ignoring error processing response: String or binary data would be truncated.
2019-10-24 14:15:44,179 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@6e520b3c (SQLServerCallableStatement:23 executeXXX): close ignoring error processing response: Arithmetic overflow error for data type smallint, value = 32768.
2019-10-24 14:15:44,433 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@5a4fe25e (SQLServerCallableStatement:25 executeXXX): close ignoring error processing response: Divide by zero error encountered.
2019-10-24 14:15:44,687 FINEST [com.microsoft.sqlserver.jdbc.TDSCommand close] - TDSCommand@49cd367c (SQLServerCallableStatement:27 executeXXX): close ignoring error processing response: The INSERT statement conflicted with the FOREIGN KEY constraint "FK_Child_ParentId". The conflict occurred in database "TEST_SP", table "dbo.Parent", column 'Id'.
2019-10-24 14:15:44,950 FINEST [com.microsoft.sqlserver.jdbc.SQLServerStatement processResults] - SQLServerCallableStatement:29 ignoring database error: 208 Invalid object name 'dbo.ParentX'.
2019-10-24 14:15:45,235 FINEST [com.microsoft.sqlserver.jdbc.SQLServerStatement processResults] - SQLServerCallableStatement:31 ignoring database error: 207 Invalid column name 'IdX'.
Reproduction code
Attached simple test java project (JDBCDriverTest.zip), which simulates the invalid behavior of Microsoft JDBC Driver – there is a script
src\test\resources\CreateTestDatabase.sql
which creates test database – at the bottom there is “testuser” user name used – it has to be changed to the existing user in SQL server.

Hi @jkucharski-objectivity, you need to parse the results from the server. We have an article on how to do this here on msdn.
Replacing cs.execute() with the following bit of code causes an error to be thrown:
boolean resultsAvailable = cs.execute();
int updateCount = -2;
while (true) {
        updateCount = cs.getUpdateCount();
        if (!resultsAvailable && updateCount == -1)
                break;
        if (resultsAvailable) {
                // handle ResultSet
        } else {
                // handle Update Count
        }
        resultsAvailable = cs.getMoreResults();
}

Hi @rene-ye, thank you for the answer.
It works fine using clean JDBC API.
Using the hibernate to execute the stored procedure does not give the opportunity to read all results, so we have to use the "SET NOEXEC ON" workaround.

Hi @jkucharski-objectivity ,
The behavior above is enforced by the server and it is the same for every other SQL Server driver (jTDS, ODBC, OLEDB, SQLClient...). At TDS protocol level, the server uses data stream packets for result set, errors and info messages. This means clients have to read any residual result sets and messages after a statement execution. Please see #826 , #937 for detailed explanation.

See also this Stack Overflow question:
https://stackoverflow.com/questions/42169951/how-to-get-everything-back-from-a-stored-procedure-using-jdbc

Hi @jkucharski-objectivity ,
I am closing the issue for now, please let us know if you need further clarification.


Provide some mechanism for backups

Currently, android:allowBackup is set to false and app does not provide any way of syncing or backing up user data. Ideally, compatibility with some web-based/desktop FOSS client would allow for much easier access/management of tasks, but something as simple as (automated) export functionality would be pretty helpful too.
(BTW, thanks for this beautiful app! :slightly_smiling_face:)

Manual database backup and restore functionality and importing and exporting of individual items for sharing is scheduled to be in the 2.0.0 release. However, the 1.2.0-beta01 and 2.0.0-beta01 releases has the features but isn't stabilized yet. :)
Thanks for using my app.... :)

Update: This feature is now live with the release of 2.0.0-stable :)


opt task/cron time

Avoid running tasks at the same time,
that will cause high CPU load.

Timing in prime numbers as much as possible


Bump backstage to 1.14.1

Also aligned some of the aws-sdk dependencies to the ones that are used in the new backstage version.
Tested with docker compose and yarn start and seems to be working.
I will need to adjust the dependencies later in our backstage instance, just in case. But no major issues are expected
Also bumped the dependencies of aws-sdk from v2 to v3, as done upstream: https://github.com/backstage/backstage/issues/16470
This required a few changes that will break the plugin if no region is set from now on. Also needed to adjust a little bit the code to the new functions and types, and fixed a minor UI issue when the pageSize was changed, since it didn't work out completely fine at the moment.

The commits here are a bit mixed up, is it possible to put the paging change into it's own commit?


Integrate ARS/Anclap into the offramp flow

Context
This ticket is to enable ARS offramp on the backend, which would include setting up the SEP 24 flow for offramping ARS
Requirements

When user selects ARS as offramping currency and agrees to the quote
The user should be redirected to ARS anchor i.e Anclap
TOML file linked, has all the details of issuer and endpoint to hit

Note
https://api.anclap.com/.well-known/stellar.toml

@pendulum-chain/product can this be worked already?

@pendulum-chain/product can this be worked already?

I will get this ready and estimated till then

Hey team! Please add your planning poker estimate with Zenhub @b-yap @bogdanS98 @ebma @gianfra-t @TorstenStueber

I'll assign myself to this since I can test the flow with my bank.

Just want to point out that ARS is not available yet on the AMM. But this should not be a blocker for this ticket here.

Yes @TorstenStueber thanks. Since the changes are very small, I'll wait for ARS to be "ready" to work on this and test an off-ramp.

@gianfra-t FYI, this has been unblocked now as ARS is now available on Nabla: https://github.com/pendulum-chain/tasks/issues/401#issuecomment-2412361777

@gianfra-t if and when you are unblocked for testing. please record the entire flow and share the recording

Should we close this ticket? the linked PR is outdated

Yes, can be closed.


[BUG]: Text not visible in Dark Mode.

Describe the bug
When clicking on E-learning the page which loads , there half of the text "Learn with fun on any schedule " is not visible in dark mode.
Expected behaviour
The entire text should be visible.
Screenshots

Additional context
No response
Code of Conduct

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

Even the site owner text is not visible.
Here is the proof.

I can do it


Error Page Documentation Doesn't Explain how Non-Get Failures Work

Currently, the exception handler documentation shows how to write a custom exception handler in the OnGet method of the error page.
What isn't covered is that the Error page method executed will mirror the type of operation that generated the failure. For example, if the error occurred in a post request, the Error page's OnGet will never be called, and instead an OnPost method will be called on the error handler page.
So, for example, you could add something like this to the example:
public void OnPost() # for failures in POST requests.
    {
        RequestId = Activity.Current?.Id ?? HttpContext.TraceIdentifier;

        var exceptionHandlerPathFeature =
        HttpContext.Features.Get<IExceptionHandlerPathFeature>();
        if (exceptionHandlerPathFeature?.Error is FileNotFoundException)
        {
            ExceptionMessage = "File error thrown";
            _logger.LogError(ExceptionMessage);
        }
        if (exceptionHandlerPathFeature?.Path == "/index")
        {
            ExceptionMessage += " from home page";
        }
    }

And then add a note indicating that the handler fired depends on the verb where the error occurred.

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

ID: 0e03fdf4-b19b-17c3-04f4-336ab6d2ee06
Version Independent ID: 38515dfb-91a5-b395-db9d-084bbaf095c8
Content: Handle errors in ASP.NET Core
Content Source: aspnetcore/fundamentals/error-handling.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@serpent5  this sounds like a fun one.

@serpent5  this sounds like a fun one.

@Rick-Anderson Sure, I'll sort this one.

@Rick-Anderson Sure, I'll sort this one.


Why I'm not able to run examples/face_deformation_video.html?

It keeps on showing "wait..Loading..video". Please help !

There are many ways we can view our face by clicking on either of the Unwell or Cheery face button , therefore I was wondering if there is any way that I can save the deformed video in the database .

Maybe, but I'm not sure how. You would have to do some kind of screen capture, which I'm not sure if webRTC supports. :/

Well, I've been trying to have all the facial changes in the video but those facial modes aren't getting trapped sadly . Thus, I wanted to know solution from you if you could suggest something on it . Tried Webrtc but don't know , as of now , not working .

I mean I am able to save video, but just plain video recording , no facial modes are getting stored .

No, the facial deforming is done by a webgl canvas that is overlayed the
video, so recording from the video stream won't help. You'll have to blend
the webgl canvas and the video somehow, so I guess you'd have to use some
screen capture solution, but I don't know if such a solution exists.
On Fri, Jan 9, 2015 at 9:17 PM, Bismoy Murasing<EMAIL_ADDRESS>wrote:

I mean I am able to save video, but just plain video recording , no facial
modes are getting stored .
—
Reply to this email directly or view it on GitHub
https://github.com/auduno/clmtrackr/issues/30#issuecomment-69392440.

I tried screen capturing, like taking a photo. But for having the complete video part, to my understanding, as you just said, two canvas need to merged into one working video . Am I right ?

Yes, that's correct, you'd have merge the two canvases into one video. Did
the screen capture work?
On Fri, Jan 9, 2015 at 9:26 PM, Bismoy Murasing<EMAIL_ADDRESS>wrote:

I tried screen capturing, like taking a photo. But for having the complete
video part, to my understanding, as you just said, two canvas need to
merged into one working video . Am I right ?
—
Reply to this email directly or view it on GitHub
https://github.com/auduno/clmtrackr/issues/30#issuecomment-69393587.

Screen capture also gave some trouble, ended up implementing taking the whole screen shot which was clear enough by the way, but merging two canvas ? well no idea yet . perhaps you know something about that part ? :)

I don't, sorry. :(
On Fri, Jan 9, 2015 at 9:36 PM, Bismoy Murasing<EMAIL_ADDRESS>wrote:

Screen capture also gave some trouble, ended up implementing taking the
whole screen shot which was clear enough by the way, but merging two canvas
? well no idea yet . perhaps you know something about that part ? :)
—
Reply to this email directly or view it on GitHub
https://github.com/auduno/clmtrackr/issues/30#issuecomment-69394957.

If you come across something useful related to this, please do let me know here :) Thanks.

Hi bismoy2013,
I got the same problem of loading video when using chrome. Can you tell how to fix this problem when running locally?

you have to run it as a server from eclipse , also go thru some google app engine tutorials. you will get it.

Thank you and it works after setting local server.

Welcome :) -- Happy coding --


[JENKINS-60354] Add flag to FlowInterruptedException to indicate actual interruptions

See JENKINS-60354.
4-5 downstream PRs incoming.

Closing/reopening in the hopes that ci.jenkins.io will see the PR.

Weird, it looks like everything succeeded, and the incremental build was published, but then the agent got killed before the build completed?
Cannot contact aci-maven-t12f8: hudson.remoting.ChannelClosedException: Channel "unknown": Remote call on JNLP4-connect connection from <IP_ADDRESS>/<IP_ADDRESS>:15297 failed. The channel is closing down or has closed down
19:41:27  Agent aci-maven-t12f8 was deleted; cancelling node body

Rebuilding to try to get an incremental build published. The last build got a 500 internal server error, and I don't see any useful messages in the log.

The incrementals publisher is hosed, see INFRA-2379, so I deployed a snapshot manually as 2.22-20191211.162318-1.


RuntimeError: The basic tracker is not publicly available yet,

Trying to use the object tracker in coral_camera_demo.py (--use_tracker=True) and I get the error:
'''
RuntimeError: The basic tracker is not publicly available yet, please reach out to repo maintainers for early access. If this error persists, it may mean your CPU architecture is not yet supported.
'''
Notes - this is with Raspberry Pi 4 and Coral USB

@GOBish  I think it a case of  accessibility of the code.You have not been granted the permission to use the code or there might be problem while cloning the code.
I hope this might help you!

@sarthak3248 I re-cloned the code and I still get the error
File "/home/gbenel/automl-video-ondevice/automl_video_ondevice/object_tracking/mediapipe_object_tracker.py", line 31, in
'The basic tracker is not publicly available yet, '
RuntimeError: The basic tracker is not publicly available yet, please reach out to repo maintainers for early access. If this error persists, it may mean your CPU architecture is not yet supported.

@GOBish  In line 31 of the code that you are cloning , there is a pre-requisite that you should be a co-developer and since the original "owner" of the code doesnt allow that to be available for the public,the same error is reoccuring even after you reclone it and the error keeps on popping!

@GOBish can you run the following:
cd automl_video_ondevice/object_tracking/mediapipe_tracker/aarch64/

python -c "import mediapipe_tracker"

And let me know what you get? This tracker should be available for aarch64 by now.


fuse: device not found, try 'modprobe fuse' first

Environment

Windows build number: [run cmd.exe /c ver]

Microsoft Windows [Version 10.0.18363.1316]

Your Distribution version: [On Debian or Ubuntu run lsb_release -r in WSL]

lsb_release  -a
LSB Version:    core-11.1.0ubuntu2-noarch:security-11.1.0ubuntu2-noarch
Distributor ID: Ubuntu
Description:    Ubuntu 20.04.1 LTS
Release:        20.04
Codename:       focal

Whether the issue is on WSL 2 and/or WSL 1: [run cat /proc/version in WSL]

cat /proc/version                                                                                Linux version 4.4.0-18362-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #1049-Microsoft Thu Aug 14 12:01:00 PST 2020
Steps to reproduce
I was following this guide
https://docs.microsoft.com/en-us/azure/storage/blobs/storage-how-to-mount-container-linux
But stumble upon this error: (anonymizes username)
$ sudo blobfuse /mnt/blob/ --tmp-path=/mnt/tmpblob  --config-file=/home//fuse_connection.cfg -o attr_timeout=240 -o entry_timeout=240 -o negative_timeout=120
fuse: device not found, try 'modprobe fuse' first
$ modprobe fuse
modprobe: FATAL: Module fuse not found in directory /lib/modules/4.4.0-18362-Microsoft
Is there something i need to install ?

You must use WSL2.

Thanks. I will try that.

Also the linux i'm running has to be "wsl 2 compatible" ? or can i switch wsl 1 linux into wsl 2 ?

after installing wsl 2 and converting distro into wsl 2 , it works. thanks.


Truncate file name in the display after uploading

Limit the visible file name length to something like 20-30 characters for a better user experience.

Resolved with latest commit.


[RFC] Integer signedness story

Most GTK APIs use gint whenever an integer is needed, largely for C-specific reasons. Since Rust has more strict integer semantics my desire is to use u32 (or u16, or Option<u32> where appropriate) wherever negative values are invalid.
This approach has been applied to a few classes so far: EntryBuffer, Notebook, partly to List/TreeStore.
It may follow that unsignedness needs to be applied as far as the basic structs like GtkAllocation aka GdkRectangle aka cairo_rectangle_int_t (width and height at least). This discussion makes me think it has merit.
Before any further effort is made on this it'd be nice to get some opinions on whether the direction is right or the above classes should be reverted to the upstream C convention (i32 everywhere).

cc @csoriano89

How are you going to deal with valid unsigned values when they are invalid as gint?
Say if the uint uses the signed bit as a value.
I would say, just follow what gtk+ does to avoid problems with gtk+... (although some decisions might be not the best ones).

It's not unsecure to provide an invalid value to gtk since it'll return an error value. However it's not perfect since rust is supposed to provide full safety. But u32 can allow to have greatest value than i32, so how are we supposed to handle it (that's what @csoriano89 said)?

How are you going to deal with valid unsigned values when they are invalid as gint?
Say if the uint uses the signed bit as a value.

Catch with asserts probably because typically an out of range value would be a "programmer error". Note that in some cases I have to add some checks anyway, e.g. gtk_tree_store_reorder is plain unsafe (but maybe it shouldn't have been exposed at all?)
@GuillaumeGomez GTK+ relies on optional asserts to handle various edge cases. If they're not compiled in, those edge cases (even as simple as calling gtk_main_quit outside a main loop) suddenly become unsafe.

@GuillaumeGomez GTK+ relies on optional asserts to handle various edge cases. If they're not compiled in, those edge cases (even as simple as calling gtk_main_quit outside a main loop) suddenly become unsafe.

Bad news.

Catch with asserts probably because typically an out of range value would be a "programmer error". Note that in some cases I have to add some checks anyway, e.g. gtk_tree_store_reorder is plainly unsafe (but maybe it shouldn't have been exposed at all?)

Not necessarily. It's always possible to find cases where a developer isn't directly responsible. However the assert should do the trick. I'm still not a fan of this solution but if we do it, no other choice...

An other possible alternative is using some U31 type. Not sure if there is a popular implementation, but there's this.

Not sure this is a nice idea to add a new type (even if we use it only internally). But let's keep an eye on it just in case.

"Catch with asserts probably because typically an out of range value would be a "programmer error". "
That's the thing, a big uint value is not an overflow, but the developer will see errors coming from gtk+ about negative values when they used just a guint. Sounds more confusing than what it tries to fix. In case you use a i32 and set a negative value the developer will catch it just printing it, but not a u32.
However if you use a u32 and make some asserts internally with a message explaining to the developer that it's a big value that cannot be processed and the maximum value is 2^31, then why not.

Yeah, I wouldn't want a negative value to reach gtk+. To be clear the approach I had in mind is
fn get_foo() -> u32 {
    let ret = gtk_get_foo();
    assert!(ret >= 0, "Unexpected negative value");
    ret as u32
}

fn set_foo(val: u32)
    let val = val as c_int;
    assert!(val >= 0, "Value bigger than i32::max_value");
    gtk_set_foo(val);
}

and documenting the presence of this assert.

Then looks like a good idea to me and no downsides on top of my head from the gtk+/c side.

There doesn't seem to be anything actionable here anymore. To some degree this is done manually now, but as there is no annotation for any such thing it will have to be done manually on a case-by-case basis in the future too.


An ability to run on Knative along with OpenWhisk

Introducing changes to NodeJS runtime to run on Knative along with OpenWhisk.
If you are consuming this runtime on OpenWhisk, there are no impacts or changes in the runtime.
We are introducing a flag called __OW_RUNTIME_PLATFORM to identify what platform is the runtime serving on, default is openwhisk.
For more information, please follow the docs PR at #120 and tests PR at #121.
Docs PR has a README explaining in detail how to run OpenWhisk NodeJS action on Knative.
P.S. These changes are being added from https://github.com/apache/incubator-openwhisk-devtools/tree/master/knative-build

Thanks Priti, this PR showcases the use of a platform factory that can, via the build process, either create the traditional OpenWhisk-native implementation which provides the same, identical code path (i.e., init/run endpoints) we have used since forever or create a Knative impl. which supports  supports a single endpoint and furthermore includes HTTP request pre-processing and response post-processing to provide an equivalent function experience where possible.

a quick round of feedback after inspecting service.js which should be semantic preserving.

Thanks a bunch Rodric, I have tried to address your comments, waiting for it to build. Looking forward to hearing on your overall experience.


Problem with tls elasticsearch

Hi,
When I try to connect to my secure cluster I get following error:
WARNING: An exceptionCaught() event was fired, and it reached at the tail of the pipeline. It usually means the last handler in the pipeline did not handle the exception.
io.netty.handler.codec.DecoderException: javax.net.ssl.SSLHandshakeException: General SSLEngine problem
My configuration.json file:
{
"web_port": 0,
"elastic_port": 9200,
"elastic_host": "esclient01prod",
"elastic_tls": true,
"authentication": true,
"basic": "elastic:*******"
}

Hello,
Is the server using a certificate that is signed by a trusted CA?
If the server certificate is self-signed it needs to be trusted by the JVM.
Regards,


 This will set expectations about acceptable behaviour, avenues of reporting a problem, and repercussions. We want the Louisville.io chat to be a safe, friendly, environment where people feel welcome.

I'm happy to steal a CoC from somewhere else if it's good.

Examples:
https://www.djangoproject.com/conduct/
http://www.rust-lang.org/conduct.html
https://github.com/nodejs/io.js/blob/master/CONTRIBUTING.md#code-of-conduct

http://contributor-covenant.org/

The irc.perl.org network has a decent set of rules that is short and pretty clear. Check out the "Community Policies" section at http://www.irc.perl.org/rules.html

I've read through all the linked ones and found this:
http://citizencodeofconduct.org/
It's my favorite of all of them.

Agreed. I made some custom edits. Once we have something online, we can send it to slack upon first sign-on.
https://github.com/myared/policies/pull/2

I like that one as well, Eric.  Looks like just a couple edits are needed.

From @barake : https://github.com/todogroup/opencodeofconduct/blob/gh-pages/index.md

:+1: for doing this!

Done, closing: http://louisville.io/code-of-conduct.html


[SPARK-14410][SQL] Push functions existence check into catalog

What changes were proposed in this pull request?
This is a followup to #12117 and addresses some of the TODOs introduced there. In particular, the resolution of database is now pushed into session catalog, which knows about the current database. Further, the logic for checking whether a function exists is pushed into the external catalog.
No change in functionality is expected.
How was this patch tested?
SessionCatalogSuite, DDLSuite

Test build #55089 has started for PR 12198 at commit a3601fb.

Test build #55089 has finished for PR 12198 at commit a3601fb.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

@yhuai

Test build #55093 has started for PR 12198 at commit 6abe10b.

Test build #55093 has finished for PR 12198 at commit 6abe10b.

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

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

Merged build finished. Test FAILed.

Test build #55146 has started for PR 12198 at commit a10e942.

Test build #55146 has finished for PR 12198 at commit a10e942.

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

Merged build finished. Test PASSed.

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

LGTM. Merging to master. We will take care the exception handling in getFunctionOption with the work of https://issues.apache.org/jira/browse/SPARK-14414.


Can only load data from disk

data sets should be loadable over the networks
Imported from Georgia Tech github:
{"created_at":"2019-04-18T11:18:15Z","updated_at":"2019-04-18T11:18:30Z","closed_at":"2019-04-18T11:18:30Z","html_url":"https://github.gatech.edu/ekobrin3/2019-Spring-CSE6242-flowgridviz/issues/47","id":78872}

Fixed in e8a119c12ac79bbf2d6d0782c42dea65a5587740
Imported from Georgia Tech github:
{"created_at":"2019-04-18T11:18:30Z","updated_at":"2019-04-18T11:18:30Z","html_url":"https://github.gatech.edu/ekobrin3/2019-Spring-CSE6242-flowgridviz/issues/47#issuecomment-42132","id":42132}


Enable xml doc generation

Will need to do this once https://github.com/aspnet/Mvc/pull/4088 is in.

028c0fb13164bcdda2e22e01e9093ac8a5853b06


When using recurse_over_arrays true, adding and populating hash in an array doesn't work in some cases

Even when defining the object with recurse_over_arrays set to true, certain ways of filling the data in an array don't work. (the nicer , more object oriented ones)
Option A - works
obj.spec = {}
obj.spec.ports = [{ 'port' => 3000,
'targetPort' => 'http-server',
'protocol' => 'TCP'
}]
Option B - doesn't work
obj.spec = {}
obj.spec.ports = []
obj.spec.ports << { 'port' => 3000,
'targetPort' => 'http-server',
'protocol' => 'TCP'
}
Option C - doesn't work
obj.spec = {}
obj.spec.ports = []
obj.spec.ports[0]= {}
obj.spec.ports[0].port = 3000

Could you provide examples what the expected results are for each of your Options, and how they break down?
At present, the way recurse_over_arrays works is that the first time you call an accessor method that refers to an Array, it memoizes a copy of that array where each hash within the original array is replaced with an ROS. If you use << to mutate such an array, any hashes already in the original array will be converted to ROS, but any new hashes will only be placed in the memoized array, and they won't be converted into ROSes.
I believe the original purpose of the memoization was to ensure that two accessor method calls will return the same ROS object instead of returning a different one each time.
One way of solving Option B would be to subclass Array for use within ROS (to reimplement methods like <<), but I'm concerned that this will open yet another can of worms.
Another way might be to check the memoized array to see if it contains any hashes that have not been replaced yet.
I also suspect that modifications to the memoized array may not show up under certain circumstances when .to_h is called on the parent ROS object, or when .dup is used to create a deep-duplicate of the object.

@aetherknight,  my expectation would be that option B and option C will work without getting a "no method error", so the user can build those structures in a nice object oriented way.
Consider a use case where the user constructs ROS without having a json file, but simply one attribute at a time, just as populating another class in an object oriented way.

I made a change to how the getter works so that it will always recompute the array that might contain ROS objects, and tested it against the 3 use-cases mentioned above. However, it does not modify the underlying hash @table such that the parent ROS objects @tables are properly modified. This means that to_hash on a parent ROS object is broken.


Internationalized (i18n) sub-path routing

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d1y61lmg5x2yx8
AWS Region
us-east-1
Amplify Hosting feature
Redirects
Describe the bug
As it is written into the documentation  should be supported by Amplify.
At this moment, for me, it is not working
Expected behavior
Given the following configuration
// next.config.js
module.exports = {
  i18n: {
    locales: ['en', 'it'],
    defaultLocale: 'en',
  },
}

If I have a pages/login.js the following urls would be available:
/login
/it/login
Reproduction steps

Create next.js app (version 13>)
Add i18n feature
Deploy to amplify

Build Settings
version: 1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - npm ci
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: .next
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

Log output

# Put your logs below this line

Additional information
App created 11/23/2022, 10:55:03 AM

We are also facing similar issue. Is there anything needs to be configured or any work around available?

Hi @dr-star and @shekar28, there is a bug with the automatic locale detection feature so the app's language will not switch based on the browser's language setting unfortunately. However, you can still use internationalized sub-path routing to switch your application's language.


DSP-293 Update contact information on the landing page

On landing page, update contact information for:

email address with default subject
post address
phone number

resolves DSP-293

@flavens Thanks. I will do it from next time. I will merge the PR now.


update epubcheck and fix some tests

I tried bundle exec rake and several tests failed in my environment.
The tests that failed were epubcheck related, so I fixed them and updated epubcheck to the latest version (because I want to use --locale options, which has been supported since epubcheck 4.2).

Coverage decreased (-0.03%) to 90.19% when pulling ffa09656311d8c4186cf24d6fa1110d020be4f48 on takahashim:fix-epubheck into 66685c7d6accfb0e758105e9ff607881f75cbd54 on softcover:master.

Great, thanks! The updated gem is now available as softcover 1.6.3.

Thanks for merging!


Fix install StarWarsBDAddon

Ref https://github.com/KSP-CKAN/CKAN/issues/1477

It is truly amazing how many extensions to BDArmory are only compatible with a KSP version that BDArmory is not compatible with. This looks fine, given the dependency.


DuckDBOpen failed: IO Error: File is already open

When we try to open the connection after closing it, the database file remains locked.
Version: 0.10.2
DuckDB.NET.Data.DuckDBException
HResult=0x00000001
Message=DuckDBOpen failed: IO Error: File is already open in C:\Workspace\platform-v4\src\Api\bin\Debug\net7.0\Api.exe (PID 6876)
Source=DuckDB.NET.Data
StackTrace:
at DuckDB.NET.Data.Internal.ConnectionManager.GetConnectionReference(DuckDBConnectionString connectionString)
at DuckDB.NET.Data.DuckDBConnection.Open()
at Data.Sql.CustomConnection.Open() in C:\Workspace\platform-v4\src\Data\Sql\CustomConnection.cs:line 47
at Data.Sql.CustomConnection.ExecuteCommand(String query) in C:\Workspace\platform-v4\src\Data\Sql\CustomConnection.cs:line 167
This exception was originally thrown at this call stack:
[External Code]
Data.Sql.CustomConnection.Open() in CustomConnection.cs
Data.Sql.CustomConnection.ExecuteCommand(string) in CustomConnection.cs

This code worked perfectly in release 0.9.2, but in release 0.10.2 it returns the error mentioned in this ticket.
using DuckDB.NET.Data;

var connectionStringReadWrite = $"Data Source=C:\\Temp\\mydatabase.db;access_mode=read_write";
var connectionStringReadOnly = $"Data Source=C:\\Temp\\mydatabase.db;access_mode=read_only";

var duckDBConnectionReadWrite = new DuckDBConnection(connectionStringReadWrite);
var duckDBConnectionReadOnly = new DuckDBConnection(connectionStringReadOnly);

// Open a read/write connection
duckDBConnectionReadWrite.Open();

// Opena a read only connection
duckDBConnectionReadOnly.Open();

// Execute a DDL command
var command1 = duckDBConnectionReadWrite.CreateCommand();
command1.CommandText = $"CREATE TABLE mytable_{DateTime.Now.Ticks}(foo INTEGER, bar INTEGER);";
_ = command1.ExecuteNonQuery();

// Close read/write connection
duckDBConnectionReadWrite.Close();
duckDBConnectionReadWrite.Dispose();

// Execute a SQL command
var command3 = duckDBConnectionReadOnly.CreateCommand();
command3.CommandText = $"SELECT 1;";
_ = command3.ExecuteReader();

// Close read only connection
duckDBConnectionReadOnly.Close();
duckDBConnectionReadOnly.Dispose();

// Reopen a Read/Write Connection - here returned a IO Exception
duckDBConnectionReadWrite.Open();

var command2 = duckDBConnectionReadWrite.CreateCommand();
command2.CommandText = $"CREATE TABLE mytable_{DateTime.Now.Ticks}(foo INTEGER, bar INTEGER);";
_ = command2.ExecuteNonQuery();

duckDBConnectionReadWrite.Close();
duckDBConnectionReadWrite.Dispose();

I don't know why it worked in 0.9.2 but you need to Dispose command and reader objects, or even better, put them in using. In such case, it works as expected.


[feature][broker] Support HTTP endpoint consume messages

Motivation
According to PIP-64, We have to support consume messages by HTTP endpoint.
Modifications
This pull request supports some endpoint for the persistent topic.

Support Create consumer by HTTP endpoint.
Support Delete consumer by HTTP endpoint.
Support Consume messages by HTTP endpoint.
Support Acknowledges messages by HTTP endpoint.

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[x] doc-not-needed

One reason for not adding this to Pulsar Brokers is that it will break Pulsar load balancing. @joefk Did a good write-up about this in https://lists.apache.org/thread/b7yjwn7cycp0r45mop4onfrr4mmp2tbd .
Quoting @joefk 's email:

First, broker throughput and performance becomes unpredictable. The
current Pulsar load model (and it is used in the load manager for load
balancing) becomes unusable. Not only that, there will be no pre-computed
model that can be used in the load manager. Since the producer and
consumer randomly decide on what is the business logic,and the computation
can change based on the data, the model itself becomes dynamic and the
load manager has to rebuild the model anytime an user updates the business
logic. That is a tall order, worth years of work to implement.

The context was slightly different in that discussion, but I believe that it could be applied to this case too. We shouldn't be adding more business logic to brokers such as HTTP endpoint for consuming message. That could be implemented in a separate component which isn't part of the broker.

It might not be a good idea to add the implementation to the main Pulsar Admin API at all.
HTTP consuming would be better to handle in a separate component. PIP-64 doesn't determine that this should be part of Pulsar Admin API and we should revisit this decision. I think it's a bad idea to add HTTP consuming to Pulsar Admin API and brokers.
We need another discussion on the dev mailing list for deciding.

Yes, I also consider moving this part of the HTTP producer-consumer to pulsar-proxy, which will make the broker clear and reduce communication between brokers.
However, moving this part to the pulsar proxy will require the user to deploy another component, especially if the user is not running the pulsar in k8s.
I will draft a discussion in the dev mail list.
Best,
Mattison

Yes, I also consider moving this part of the HTTP producer-consumer to pulsar-proxy, which will make the broker clear and reduce communication between brokers.

you can build a "Proxy Extensions" and a "Protocol Handler", this way your code can run on the Proxy and on the Broker.
See how @cbornet did this for RabbitMQ:

https://github.com/datastax/starlight-for-rabbitmq/blob/master/starlight-rabbitmq/src/main/java/com/datastax/oss/starlight/rabbitmq/GatewayProtocolHandler.java
https://github.com/datastax/starlight-for-rabbitmq/blob/master/starlight-rabbitmq/src/main/java/com/datastax/oss/starlight/rabbitmq/GatewayProxyProtocolHandler.java

We chose another way to implement it, and Please see the mail list<EMAIL_ADDRESS>So, close this PR.

@mattisonchao What's the current progress？Could share the link?

I found https://github.com/apache/pulsar/issues/2467 but it also does not keep track of the status on PIP-64

Not completely the same but note that we now have a Sink to an HTTP endpoint : https://pulsar.apache.org/docs/2.11.x/io-http-sink/


Add 1D version of GeoClaw solver

This is something I put together several years ago by straightforward modification of the 2D Geoclaw solver.  I've been using it ever since but somehow never added it to the main repository.
I think @rjleveque has also been developing a 1D version of the Geoclaw solver that might be superior to this one but I don't know the status of that.

If this is updated to match the newest versions of the GeoClaw 2D solvers I would almost favor updating what's in geoclaw_riemann_utils.f as well.  We have been meaning to update those and possibly update them to be faster but have not gotten around to it.  This at least would make it in F90.

The 1D solver I was working on has been incorporated into GeoClaw (as of v5.10.0), see https://www.clawpack.org/geoclaw1d.html for documentation.  There's no AMR but mapped grids are supported, which is very useful to go from large cells offshore to small cells onshore for realistic topo.
Note that a Boussinesq version is also available, https://www.clawpack.org/bouss1d.html.
There are some examples using both codes in
$CLAW/geoclaw/examples/1d_classic, shown in the gallery here.  More examples would be good at some point.
These use the Riemann solver in $CLAW/geoclaw/examples/1d_classic/rp1_geoclaw.f90 and I realize now I should have put this in the riemann repository.  It uses riemann_aug_JCP from $CLAW/riemann/src/geoclaw_riemann_utils.f.
@ketch, I'll talk a look at how your version compares to what I've been using.  I think one or the other (or some combination) should be put in the riemann repository, but let's take a closer look before merging this.
I agree with @mandli that at some point we need to clean up these solvers, and I've also been talking with @dlgeorge recently about this in relation to the D-Claw solvers, which are similar but were branched off from GeoClaw at some point and so some things aren't identical that should be.

I only recall there being an issue with common blocks, was there something in addition to that?

We have common blocks in a lot of solvers and those work fine with f2py.  But it's been 10 years since I looked at this stuff so I don't recall what the issues are.


Please change ShibaBCH logo

I would like to change the ShibaBCH logo to be more within keeping with the sBCH Community.

Thanks, added


High memory usage when consistently using new HTTP client instances for each request

Guzzle version(s) affected: 7.7.0
PHP version: 8.2.7
cURL version: 7.88.1
Description

I have a long running PHP script deployed within a docker service, which over its lifetime sends many HTTP requests.
I have noticed that the memory usage reported by docker stats keep increasing and settles at around 2GB of used memory - If I remove the line sending the HTTP requests, then the memory stays low and stable.
Inspection with memory_get_usage() in the PHP runtime, does report a significantly lower memory usage than docker stats does.
I can reproduce a similar memory gap between the two tools, by simply creating multiple curl handles and never closing them - Leaving me to the thinking that the high memory usage is caused by hanging curl handles?
If I create a single guzzle client instance and reuse that client for all requests, then I do not see any large memory usage.
How to reproduce

Deploy at docker service with the following script
while (true) {
    $guzzle = new Client();

    try {
        $guzzle->get('https://some_url_you_do_not_mind_spamming');
    } catch (Throwable) {
        // Do nothing
    }
}

And inspect the memory usage with docker stats
The host system is Ubuntu 22.
Possible Solution

It might be related to some curl handles not being closed?
Additional context

I have attempted calling gc_collect_cycles() within my script, which does nothing for the high memory.
The high memory usage can also be measured on the host system using tools such as btop to measure the difference between "Used memory", "Available memory", "Cached memory" and "Free memory". Killing the docker service after it reaches the 2 GB peak, I can see a similar drop in "Used memory". No affect is present on "Cached memory".

I have submitted a PR for the suggested solution at https://github.com/guzzle/guzzle/pull/3165

Fixed in v7.8.0.


Add wrappers for HTTP PATCH

As discussed in https://github.com/slamdata/purescript-affjax/issues/76

Thanks!


Math equations do not work when imported from another file

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Math equations do not get parsed/styled as they should when they are imported from another md file.
Reproducible demo
https://codesandbox.io/p/devbox/crimson-breeze-23sstv?file=%2Fdocs%2Fintro.md%3A10%2C1
Steps to reproduce

Create a md file in the static folder.
Add a math equation in it.## some title

some text

$$
2 + 4
$$

Import that file in a docs file.

---
sidebar_position: 1
---

# Tutorial Intro

$$
{MATH} + 4
$$

import Page from<EMAIL_ADDRESS>
<Page/>

Expected behavior
I expect to see 2 equations:

the one from my docs file.
the one from my imported file.

Actual behavior
Only one equation is shown correctly:

Your environment

Public source code: https://codesandbox.io/p/devbox/crimson-breeze-23sstv?file=%2Fdocs%2Fintro.md%3A10%2C1
Public site URL:
Docusaurus version used: 3.1.1
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[ ] I'd be willing to fix this bug myself.

Your codesandbox is private so I can't access it.
Your remark plugins are config specific, and only apply to the folder of the plugin you configured them in (./docs for example).
They do not apply to the static folder, for which we don't have a way for now to provide custom remark plugins. Although technically you could use the configureWebpack hook to add a custom mdx loader for that path and add custom remark plugins, it's not super straightforward to do so, nor robust or documented.
The solution is likely to have a global siteConfig.markdown.remarkPlugins option

I'm running into this issue as well, where I have a monorepo and I want to reuse all of the README.md files of the packages throughout the repositories, but still have them be processed with plugins.
The siteConfig.markdown.remarkPlugins option sounds like a great idea, and a nice addition to the configuration overall.

Sorry for the permission issue, fixed it now.
Thanks for the info about plugins work and are configured. I'll change things around but I also like your idea about a global option 👍

At the moment this is working as intended. The global option is tracked in https://github.com/facebook/docusaurus/issues/5999.


Block Devide Mapping even from Image Menu

Hello,
I tried to launch a new instance from the Image menu, as suggested in Issue #1 but I still get the
"Error: Failed to perform requested operation on instance "dLab3", the instance has an error status: Please try again later [Error: Build of instance fd211bc7-616f-4fa1-b9e3-b64feb9c9e37 aborted: Block Device Mapping is Invalid.]. "
Is there anything else I can try?

I think this is an SSC/OpenStack error, can you contact SSC support?
Mvh
Andreas
————————————————————————
Andreas Hellander
Associate Professor in Scientific Computing
Project Leader SNIC Science Cloud
Department of Information Technology, Uppsala University
<EMAIL_ADDRESS>
On 16 May 2017, at 21:52, dbampalikis<EMAIL_ADDRESS>wrote:
Hello,
I tried to launch a new instance from the Image menu, as suggested in Issue #1 https://github.com/SNICScienceCloud/LDSA-Spark/issues/1 but I still get the
"Error: Failed to perform requested operation on instance "dLab3", the instance has an error status: Please try again later [Error: Build of instance fd211bc7-616f-4fa1-b9e3-b64feb9c9e37 aborted: Block Device Mapping is Invalid.]. "
Is there anything else I can try?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/SNICScienceCloud/LDSA-Spark/issues/5, or mute the thread https://github.com/notifications/unsubscribe-auth/ACQT5qxgYKT1ACzZZYvMJgUiaRalXrGIks5r6f6DgaJpZM4Nc_yl.

I tried once again, with xlarge size and it seems to work now


Revamp testing framework to alleviate downloads from Allen

The issue
The first test build of #67 broke due to timeout errors attempting to connect to the Allen Institute API. This is the first instance of such an error, and it's very possible it's an isolated event, but it's worth considering alternative methods for running the test suite.
Currently, for every test build, we download microarray expression data for two donors from the Allen Institute. The data are cached and used for every test in the build, but with six builds (x2 for push and PR) this can get quite onerous.
Beyond the human microarray expression downloads, the abagen.mouse test suite makes a number of calls to the Allen API to query different data on mouse microarray expression. None of these calls are particularly "heavy" but the scale of them can be something that might be...less than ideal.
Proposed solution
Human microarray data
Perhaps it would be worth adopting something like how nilearn tests their dataset fetchers (e.g., here). They pre-generate (empty) files that are named like the real files such that when they call the dataset fetcher function the empty files are returned without ever having to download anything. This would greatly speed up the amount of time spent in the tests since we wouldn't be waiting for the downloaded data!
Unfortunately, in order to run the large majority of the tests, we do need some sort of data to work with. One option is to simply cache the microarray data with Travis. An alternative is to generate simulated data "on-the-fly" that looks like the AHBA data, but is simply hard-coded into the tests. This would have the added benefit of probably being much, much smaller than the real data and therefore also making tests go a bit faster.
Mouse microarray data
I'm honestly not sure of a good alternative to testing the calls to the Allen API! Will need to think on this...

This was largely resolved with the move to CircleCI in #121.


Style option isn't working

I'm trying to change Style option to anything but auto and it does nothing. AFAI tested the other options are fine. It's just Style.
Specially I would like to use expand option.

same here, can reconstruct it: option - style not working:
sass: {
dist: {
options: {
style: 'compressed'
},
files: {
'_target/style.css': 'app/application.scss',
}
}
},

neither do I.
I tried to remove others options and even on other computer but it's not ok.

Yeah, same issue. Any chance this will get resolved?

Same issue here no matter what i pick i get "%" is not an allowed value for option "style". please fix the issue or remove the documentation that says you can use the other values.

same issue here, still open?
"compact" is not an allowed value for option "style".

same issue here

solved problem using grunt-sass instead.

if this is abandon ware please mark as no longer supported or help needed.

Why aren't the style options: nested & compact working?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/1007 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit 2073520:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-source' (branch: Release_Archive_main_2022-01-25-03-34-24) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Object.resolveValues and Object.settleValues

const {
  one, // 1
  two, // 2
} = await Object.resolveValues({
  foo: Promise.resolve(1),
  bar: Promise.resolve(2),
});

const {
  one, // { status: 'fulfilled', value: 1 }
  two, // { status: 'rejected', reason: 2 }
} = await Object.settleValues({
  one: Promise.resolve(1),
  two: Promise.reject(2),
});

It seems weird to have these on Object instead of Promise, since they produce a Promise, not an Object.

It does not solve problems that should solve this proposal and it could cause many new potential bugs - for example, order with integer keys.
Or this is a bug in the names of keys in the first example?

(the first example with integer key)
const {
  one,
  two,
} = await Object.resolveValues({
  foo: Promise.resolve(1),
  1: Promise.resolve(2),
});

How it should work?

oh, in that case the example's just wrong :-) it'd need to be:
const {
  1: one,
  foo: two,
} = await Object.resolveValues({
  foo: Promise.resolve(1),
  1: Promise.resolve(2),
});

(depending on which was meant to be which)

In this case, it's just a name option for #7.


RUMM-1051 Use the last known network and user info into the Ndk Crash log

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

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

Codecov Report

Merging #495 (248d269) into master (5678f98) will increase coverage by 0.07%.
The diff coverage is 90.77%.

@@             Coverage Diff              @@
##             master     #495      +/-   ##
============================================
+ Coverage     89.50%   89.57%   +0.07%
- Complexity     1373     1407      +34
============================================
  Files           160      163       +3
  Lines          4684     4765      +81
  Branches        531      545      +14
============================================
+ Hits           4192     4268      +76
  Misses          318      318
- Partials        174      179       +5

Impacted Files
Coverage Δ
Complexity Δ

...m/datadog/android/core/internal/utils/MiscUtils.kt
58.62% <0.00%> (-2.09%)
0.00 <0.00> (ø)

...droid/log/internal/user/DatadogUserInfoProvider.kt
83.33% <66.67%> (-16.67%)
3.00 <1.00> (ø)

...og/android/log/internal/user/UserInfoSerializer.kt
78.57% <78.57%> (ø)
10.00 <10.00> (?)

...android/core/internal/data/file/AsyncFileWriter.kt
80.00% <80.00%> (ø)
4.00 <4.00> (?)

...id/core/internal/net/info/NetworkInfoSerializer.kt
83.33% <83.33%> (ø)
6.00 <6.00> (?)

...dog/android/core/internal/data/file/FileHandler.kt
95.08% <89.47%> (-0.16%)
15.00 <5.00> (+6.00)
:arrow_down:

...n/com/datadog/android/core/internal/CoreFeature.kt
98.58% <100.00%> (+0.25%)
43.00 <3.00> (+1.00)

...d/core/internal/data/file/ImmediateBucketWriter.kt
85.19% <100.00%> (ø)
11.00 <4.00> (?)

...roid/core/internal/domain/ConsentAwarePersister.kt
100.00% <100.00%> (ø)
6.00 <6.00> (?)

...id/core/internal/domain/FilePersistenceStrategy.kt
96.77% <100.00%> (ø)
7.00 <0.00> (ø)

... and 18 more


[Question] Some optimization suggestions for Kafka client monitoring metrics.

Question
Currently, Hertzbeat offers two methods for monitoring Kafka: one is based on the JMX protocol, and the other uses Kafka clients. For Kafka clients, I have two optimization suggestions:

In the "Topic Offset" module, the second column is intended to represent the partition number corresponding to the topic, not the number of partitions, as shown in the figure below. This can cause confusion for Chinese users. I suggest changing the title to "分区号" in the Chinese display. This would require modifying the monitoring template.

Currently, Kafka client metrics are limited. I recommend adding a new module called "Consumer Group Status," which would include metrics such as consumer group ID, subscribed topic names, partition offsets, total consumption lag, and other related fields.

If necessary, I am willing to assist in implementing these changes.

+1👍 welcome to optimize this.


Compilation warning turns into truffle error

It turns out that this contract doesn't really do what I hoped it would, but still I would guess that allowing for warnings in compilation is a good idea.
contract arrayswitcher {
    address[] public oldArray;

    function deleteElement(uint256 index) {
        address[] newArray;
        for (uint256 i = 0; i < oldArray.length; i++) {
	        if (i!=index) {
	            newArray.push(oldArray[i]);
	        }
        }
        oldArray = newArray;
    }
}

Compiles with the warning: uninitialized storage pointer, but in truffle it gives error.

Fixed on master. Cheers!


Generate the output directly in a zip file

Rather than writing many files to disk and then parsing and zipping them, add an option to generate_algorithm_template that takes an open zip_file handle and write the generated files directly there. It should be exclusive from output_path and exactly one of those two variables should be defined.

Huh, that inverts things nicely. Instead of generating things towards disk and then streaming it into a memory zip_file it would allow directly pushing things into a ZIP. Smart!

Copy and render would then look something like:
def copy_and_render(
    *,
    templates_dir_name,
    output_zip_file,
    context,
    prefix=None,
):
    source_path = PARTIALS_PATH / templates_dir_name

    if not source_path.exists():
        raise TemplateNotFound(source_path)

    env = get_jinja2_environment(searchpath=source_path)

    for source_file in source_path.rglob("*"):
        if not source_file.is_file():
            continue

        check_allowed_source(path=source_file)

        output_file = source_file.relative_to(source_path)

        if prefix:
            output_file = prefix / output_file

        if source_file.endswith(".j2"):
            # Render Jinja2 template
            template = env.get_template(
                name=str(source_file.relative_to(source_path))
            )
            rendered_content = template.render(**context)

            # Write rendered content to output file (without .j2 extension)
            output_file = output_file.with_suffix("")
        else:
            with source_file.open("r") as f:
                rendered_content = f.read()

        if output_file.suffix == ".py":
            rendered_content = black.format_str(rendered_content, fast=False, mode=black.Mode(), write_back=black.WriteBack.YES)

        output_zip_file.writestr(
            str(output_file),
            rendered_content,
            zipfile.ZipInfo(str(output_file)).from_file(
                str(source_file), strict_timestamps=False
            ),
        )


"Error: connect ETIMEDOUT"

When I use MySQLDriver to connect my MySQL server, mysql2 package and MySQLDriver class have bug or problem.
Please help me or fix it.
Error: connect ETIMEDOUT
    at PromisePool.query (/home/runner/...../node_modules/mysql2/promise.js:351:22)
    at MySQLDriver.prepare (/home/runner/...../node_modules/quick.db/out/drivers/MySQLDriver.js:36:25)
    at new QuickDB (/home/runner/...../node_modules/quick.db/out/index.js:35:37)
    at module.exports (/home/runner/...../start/4-database.js:24:16) {
  code: 'ETIMEDOUT',
  errno: undefined,
  sql: undefined,
  sqlState: undefined,
  sqlMessage: undefined
}

Hey, that isn't a problem on quick.db's end. We test the connection on every release and I just tested the MySQLDriver again right now to make sure.
ETIMEDOUT happens when it cannot reach the database. make sure the database is running and that the connection options like the ip/hostname and everything else is set correctly

OK Try it again

Hey, that isn't a problem on quick.db's end. We test the connection on every release and I just tested the MySQLDriver again right now to make sure.
ETIMEDOUT happens when it cannot reach the database. make sure the database is running and that the connection options like the ip/hostname and everything else is set correctly

It doesn't fixed :(

Hey, that isn't a problem on quick.db's end. We test the connection on every release and I just tested the MySQLDriver again right now to make sure.
ETIMEDOUT happens when it cannot reach the database. make sure the database is running and that the connection options like the ip/hostname and everything else is set correctly

It doesn't fixed :(

I tested the mysql connection myself to make sure it is working. It is in fact working. If you want you could include more information about your setup and also the database version. You are getting a network error. Your connection or database setup is probably wrong. I cannot do much to assist you in verifying those things

OK, thanks for helping👌🏻


Show information about the logged in user

Currently it is impossible for the user to know with which account he is logged in.
When the user clicks in the rightmost topmost icon (see figure), it should show there its name and email.

Uploaded information through local storage (Tried to use useState but we reload the page the information would be gone)


Switch - Toggle not working with "Enter" key

Description
According to the docs, "Switch" toggles on both the "Space" and "Enter" keys.
However, it only works with "Space" and not the other one.
I confirmed it with ark-ui's own Switch component.
Link to Reproduction (or Detailed Explanation)
https://ark-ui.com/react/docs/components/switch
Steps to Reproduce

Vist the "Switch" component page.

Focus on the preview component.

Try toggling it with "Enter" key.

Ark UI Version
4.1.0
Framework

[X] React
[X] Vue
[X] Solid

Browser
Google Chrome Dev
Additional Information
No response

This is by design. The switch is built on top of the checkbox component which only toggles with the space key or pointer click.
The enter key is reserved for form submissions.

The enter key is reserved for form submissions.

Thanks for the quick response @segunadebayo!
I only mentioned it because it was mentioned in the docs. I think we can remove it then!?


Improved Andy's Streaming.  Added NetCDF.  Added 24 hour caching

@oceanzus If this looks good to you, do you want to dockerize and test on AWS?


Report when a time series is filtered out during ingest, try#2

Report dropped-by-filter when a filter drops a time series during ingest. This fixes a "XXX:" comment.
Also updated tests.

Current coverage is 45.18% (diff: 16.66%)

Merging #124 into master will decrease coverage by 0.01%

@@             master       #124   diff @@
==========================================
  Files           598        598
  Lines         15421      15427     +6
  Methods           0          0
  Messages          0          0
  Branches       1585       1585
==========================================
+ Hits           6970       6971     +1
- Misses         7989       7994     +5
  Partials        462        462

Powered by Codecov. Last update eef540e...472eea2

Thanks! Decrease in test coverage is due to the added lines in the semantic module. Ignoring them for now.


Logo

If anyone can make a logo for awesome-svelte in compliance with the awesome look. That would be awesome :)

Assign this to me, I will do this tomorrow 😄

Thanks


Use a different dockerhub account

GraphDB, the triplifier, and the webapp are al using my personal dockerhub account; we should switch these to somewhere more official.

One possibility is to not use dockerhub at all, and instead use the GitHub package repository to store images (either on our public repo or on our GHE server). I tried it with some GitHub actions and it was really easy. Haven't tried it from k8s but maybe it would be similar?

I think I'm reading the billing page correctly - the allocated resources for free and pro accounts are pretty low.
500mb disk space and 1gb of transfer per month for free
2gb disk space and 10GB of transfer per month for pro
Additionally, the storage & network transfer are added into the GitHub Actions quota.
We had talked in the past about deploying something along the lines of docker registry; I think that GHE package registry is a pretty nice alternative since it's bundled within the GitHub deployment and we can publish more than just docker images. We'd also have much much more storage, image management becomes less of a hassle with different accounts,  and we won't have any direct costs. It also gives us a little protection against overly relying on GitHub's service which might fluctuate in free features.

I'd asked nick to get GHE working for Actions and the package registry, but he ran into some issues with configuration of the S3 store IIRC. In any case, can we start with the public github.com package registry, and then if needed we can add or replace it with GHE registries, which might take a bit to set up?


Amplify password less authentication. Error received as AuthError: Incorrect username or password

Describe the bug
I have configured the iOS app using amplify documentation. Not used Amplify CLI, but used Cognito User Pool and Identity Pool in amplifyconfiguration.json file.
Installed latest library using pod as(version 1.27.0)
pod 'Amplify'
pod 'AmplifyPlugins/AWSCognitoAuthPlugin'
Pod lock file:

Used only sign in method as

I have amplifyconfiguration.json same as in android and credential used for this API is also the same so there is no issues in that. But in the iOS application received error as
▿ AuthError: Incorrect username or password. Recovery suggestion: Check whether the given values are correct and the user is authorized to perform the operation.   ▿ notAuthorized : 3 elements     - .0 : "Incorrect username or password."     - .1 : "Check whether the given values are correct and the user is authorized to perform the operation."     - .2 : nil
In appdelegate added
try Amplify.add(plugin: AWSCognitoAuthPlugin())
try Amplify.configure()
After running app, I received log as
Amplify configured with auth plugin
Added configuration sample just masked some ids with xxx

Steps To Reproduce
Steps to reproduce the behavior:
1. Just follow amplify doc as in https://docs.amplify.aws/lib/auth/existing-resources/q/platform/ios/
2. Install latest amplify lib using pod
3. Add manually configured amplifyconfiguration.json in project
4. Hit sign-in method with password i.e. pass nil or "" string with only username in it.

Expected behavior
As all steps mentioned above is working in Android application then same should work in iOS as well.
Amplify Framework Version
1.27.0
Amplify Categories
Auth
Dependency manager
Cocoapods
Swift version
5.0
CLI version
Not used cli version
Xcode version
Xcode 13.1 (13A1030d)
Relevant log output
<details>
<summary>Log Messages</summary>

INSERT LOG MESSAGES HERE

```
Is this a regression?
No
Regression additional context
No response
Device
On All device
iOS Version
Tried in iOS 12 and and 15
Specific to simulators
No response
Additional context
No response

Not that the Amplify-iOS library starts the custom auth flow with SRP. Please check here if you need to skip the SRP step as shown. - https://docs.amplify.aws/lib/auth/signin_with_custom_flow/q/platform/ios/#lambda-trigger-setup

Thank you for your response. I debugged this issue with web team and they found that even if I am sending "authenticationFlowType": "CUSTOM_AUTH", they are receiving SRP_A from iOS device only. So aws is expecting for password and I am passing it as nil, so aws is giving response as Incorrect username or password.

iOS SDK currently starts the custom auth flow with SRP and the link I mentioned in my previous comment addresses that. We would need to bypass the SRP challenge something on the lines:
 if (event.request.session &&
    event.request.session.find(attempt => attempt.challengeName == 'SRP_A')) {
    event.response.issueTokens = false;
    event.response.failAuthentication = false;
    event.response.challengeName = 'CUSTOM_CHALLENGE'; // by pass to custom challenge
  }

With above changes it is working fine.

Hi , Im am integrating Amplify on iOS
using  do {
let signInResult = try await Amplify.Auth.signIn(username: userId, password: password)
switch signInResult.nextStep {
case .confirmSignInWithSMSMFACode(let deliveryDetails, let info):
print("SMS code send to (deliveryDetails.destination)")
print("Additional info (String(describing: info))")
                // Prompt the user to enter the SMSMFA code they received
                // Then invoke `confirmSignIn` api with the code

            case .confirmSignInWithCustomChallenge(let info):
                print("Custom challenge, additional info \(String(describing: info))")

                // Prompt the user to enter custom challenge answer
                // Then invoke `confirmSignIn` api with the answer

            case .confirmSignInWithNewPassword(let info):
                print("New password additional info \(String(describing: info))")
                DispatchQueue.main.async {
                    NotificationCenter.default.post(name: NSNotification.showResetPasswordViewForFirstLogin,
                                                    object: nil, userInfo: [:])
                }

                // Prompt the user to enter a new password
                // Then invoke `confirmSignIn` api with new password

            case .resetPassword(let info):
                print("Reset password additional info \(String(describing: info))")

                // User needs to reset their password.
                // Invoke `resetPassword` api to start the reset password
                // flow, and once reset password flow completes, invoke
                // `signIn` api to trigger signin flow again.

            case .confirmSignUp(let info):
                print("Confirm signup additional info \(String(describing: info))")

                // User was not confirmed during the signup process.
                // Invoke `confirmSignUp` api to confirm the user if
                // they have the confirmation code. If they do not have the
                // confirmation code, invoke `resendSignUpCode` to send the
                // code again.
                // After the user is confirmed, invoke the `signIn` api again.
            case .done:
                do {
                    let session = try await Amplify.Auth.fetchAuthSession()

}
catch let error as AuthError {
print("Fetch auth session failed with error - (error)")
}
} catch let error as AuthError{
print ("Sign in failed (error)")
completion(false, error.errorDescription, error)
} catch {
print("Unexpected error: (error)")
}
Im getting error as follows when i deliberately enter an incorrect password..  AuthError: Incorrect username or password. Recovery suggestion: Check whether the given values are correct and the user is authorized to perform the operation. ▿ notAuthorized : 3 elements - .0 : "Incorrect username or password." - .1 : "Check whether the given values are correct and the user is authorized to perform the operation." - .2 : nil
 My question here is how can i extract the error message form here. as i want to have redable userexperinece  .
 Another question , why is it throwing error in that last catch Block and not the error in AuthError block


Https on Localhost connection issues

Cannot connect to https service running on localhost with version 3.5.36. This used to work fine in previous releases.
I cannot see much on the logs, it seems my service is  not even hit (service works fine with Postman requests).
When I switch back to http with localhost, it just works fine.

As I'm not seeing a port number in your URL, I am assuming that you are running this via local IIS?  Or are you setting your port number to 80 some other way?
Also which SDK are you using?

I am using port 443, but when tested on different port I see the same issue.
Using the latest SDK <IP_ADDRESS> bit I experienced it before updating the  nuget packages from 3.15.0 as well.
Using the external IP or dns name of the same service instance same port works fine.
As far as I remember iso

Do you have localhost mapped to anything else in your hosts file?

Not sure which hosts file I shall check that, but I have not changed any hosts files recently. Note, it works with http and localhost.

@emadelwany is there a workaround for this, since this has been tagged as a bug?

@emadelwany double-checking on this one, as it was tagged a bug.  Can you advise a workaround or is this a wont-fix?

This was addressed in #1779


JSON error?

Sams-iMac:kadenze-dl sam$ cd /Users/sam/Desktop/kadenze-dl/kadenze-dl
Sams-iMac:kadenze-dl sam$ python3.7 kadenze-dl.py
Signing in www.kadenze.com ...
Selected courses for download:
real-time-audio-signal-processing-in-faust
machine-learning-for-musicians-and-artists-v
introduction-to-programming-for-the-visual-arts-with-p5-js-vi
creative-programming-for-audiovisual-art-i
programming-max-structuring-interactive-software-for-digital-arts-i
exploring-the-mixed-reality-landscape
Parsing course: real-time-audio-signal-processing-in-faust
Traceback (most recent call last):
File "kadenze-dl.py", line 10, in
main()
File "kadenze-dl.py", line 6, in main
client.download_all_courses_videos()
File "/Users/sam/Desktop/kadenze-dl/kadenze-dl/kadenzeclient.py", line 83, in download_all_courses_videos
self.download_course_videos(course)
File "/Users/sam/Desktop/kadenze-dl/kadenze-dl/kadenzeclient.py", line 63, in download_course_videos
sessions = self.list_sessions(course)
File "/Users/sam/Desktop/kadenze-dl/kadenze-dl/kadenzeclient.py", line 34, in list_sessions
sessions = helpers.get_sessions_from_json(response)
File "/Users/sam/Desktop/kadenze-dl/kadenze-dl/helpers.py", line 33, in get_sessions_from_json
json_string = json.loads(response)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/json/init.py", line 348, in loads
return _default_decoder.decode(s)
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Sams-iMac:kadenze-dl sam$
This is after changing the path, putting my correct login details, installing requirements.txt.
I have also tried using "python kadenze-dl.py" however it spits out a syntax error if I do so, so I used python3.7 to ensure its using the latest version
Sams-iMac:kadenze-dl sam$ python kadenze-dl.py Traceback (most recent call last):   File "kadenze-dl.py", line 1, in <module>     from kadenzeclient import KadenzeClient   File "/Users/sam/Desktop/kadenze-dl/kadenze-dl/kadenzeclient.py", line 2, in <module>     import helpers   File "/Users/sam/Desktop/kadenze-dl/kadenze-dl/helpers.py", line 71     print(s, end='', flush=True)                 ^ SyntaxError: invalid syntax

I haven't tried 3.7 yet but it could be a change on the site. Im not home for a couple days, I will check that this week end, thanks for the report.

Okay after a quick review, it seems the cause is "real-time-audio-signal-processing-in-faust" not having started yet (scheduled) so it doesn't have any sessions or videos.
You should disable it for now and listing only those you need. I will work on a fix to detect that kind of course in the next week or so.

Fix is available, just update from the repository and you're good to go :)


Override default

Hi @L3MON4D3  I saw this in @tjdevries  config, should we upstream it?
(In that case it will be the first real "push requests" in history 😄 )
https://github.com/tjdevries/config.nvim/blob/c16bab0565253f862da0cef6a288399640b3c567/lua/custom/snippets.lua#L41

override_builtin doesn't mean anything to me, am I missing something? :sweat_smile: (I'm assuming I am, please educate me :D)

That single line is incorrect, but the rest of the file is replacing the built-in snippets of vim. So my guess is that that's the API @tjdevries was expecting for that same usecase.

Ahh I see..
Yeah that seems like a good idea, quickly swap out snippet engines, and hooking vim.*-functions is very well-known. I like it :)
I'd like to mention that implementing this as a simple api-function would work as well, but is probably more cumbersome for the user (setting .setup({...})-values can be done in the lazy-configuration directly)


Jupyter Notebook: Docker VM

This really doesn't need formatting.
100 batch support for graphics with large VRAM counts
RTX 4090, supports approx 70 batch in 32k audio (twitch)

Q
cono

Batch size is not about VRAM but about the size of the dataset and the training goal.

Description
This Jupyter Notebook sets up the environment for running Applio, including downloading and installing Miniconda, creating a Conda environment, cloning the Applio repository, and installing necessary dependencies. It also starts Tensorboard and the Applio application.
Motivation and Context
This setup is required to ensure that all necessary dependencies and environments are correctly configured for running Applio. It automates the setup process, making it easier for users to get started with Applio.
How has this been tested?
The setup process has been tested by running the notebook cells in sequence on a compatible system. Each step includes checks to ensure successful completion, such as verifying the existence of Miniconda and the Conda environment, and confirming the installation of dependencies.
Screenshots (if appropriate):

Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

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

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
If you're unsure about any of these, don't hesitate to ask. We're here to help!

[x] My code follows the code style of this project.
[-] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[-] I have added tests to cover my changes.
[x] All new and existing tests passed.

README changes are not accepted. Info could be added inside the notebook/applio docs