Fuse allgather requests across adapters and q, k, v to reduce small network requests

Feature request
The current approach to tensor parallelism from #5 is not latency optimized. We make an allgather call for every adapter, which will be quite slow for many adapters. Additionally, we don't fuse together the q and v matrices, which would further halve the number of allgathers.
A better approach would be to pre-allocate a large tensor and then slice in and out the individual tensors, as shown here:
https://discuss.pytorch.org/t/concatenate-tensors-without-memory-copying/34609
Motivation
No response
Your contribution
No response

Following #79, this is only applicable to the loop fallback code path for very small ranks.


Implement the BallerinaTypeDescTypeBuilder in Types API

Description:
$title
Describe your task(s)
Related Issues (optional):
Subtask of #35415

Completed by #35850


Genericize Blazor wrappers

😯 Problem to Solve
Certain Blazor wrappers support a single type (e.g. radio buttons are string type), but we would like them to be generic.

Closing as low ROI


Using hashHistory causes browser to reload entire page on router.push()

Confirmed that this is a bug since it surfaces in Chrome and Edge not in Firefox (things work fine in firefox i mean).
Full details on this Stackoverflow post.
I can copy/paste here too if you want.

This seems to be happening with browserHistory too.
Here is a video demoing it:
http://screencast.com/t/PLWsYI6JD
It seems the route is loaded and then somehow there is a refresh.
The eventual 404 is cause the page is reloaded fully and i havent configured the server to use browserHistory

You have something misconfigured somewhere on your side – this works fine in our examples and code. The issue tracker is not a support forum and this is not the first time you've been told that it's not appropriate to ask for support here.


[feat]: implement a 'theme' property to the Form Component

Hey, explorer! 🚀
Currently the Form.astro component in the project's astro-reactive-form package only accepts one input property (i.e., formGroup).
You can find this file in packages/astro-reactive-form/Form.astro. Follow the instructions in our contributing guide to get the project running.
In this issue, we add a new theme input property which will dynamically add a new class to the <form> element.
Depending on the value passed to this input, we can later implement dynamic themes that will change the look and feel of our Form component.

Hi @ayoayco, can you brief this issue? It will be helpful in understanding and working for it.

Hi @Chitvan14 ! Glad to have you on board! 🚀
Right now, if you run the project demo app, you will see that we have a Form component that is currently unstyled.
We want to keep it unstyled by default, but also allow our users to use a 'theme' property if they want, like so:
<!-- if they want a dark theme -->
<Form theme="dark" formGroups={[form]} />

For this issue, the goal is to add the 'theme' property to our Form.astro component. You don't have to fully implement the themes yourself.
For more information on Astro and implementation of components, you can check the documentation
Don't hesitate to ask, if anything is unclear! :)

I updated the issue description to add a clear Acceptance Criteria :)

Hey @ayoayco would love to work on this, can you assign this to me?

@ayoayco is anyone currently working on this issue? If not, I would like to help 😀.

Hi @fazzaamiarso! This has been inactive for some days, I think it's safe to say you can work on it :)


Create dataset loader for NusaParagraph Rhetoric

NusaCatalogue: https://indonlp.github.io/nusa-catalogue/card.html?nusa_paragraph_rhetoric

Dataset
nusa_paragraph_rhetoric

Description
NusaParagraph is a human-written paragraph datasets which covers 10 local languages in Indonesia. The dataset consists of around 50,000 paragraphs each with aroung 100 tokens resulting in a total of 6M tokens. The dataset is labelled with topic, emotion, and paragraph type.

License
CC-BY-NC-SA 4.0

Closed in https://github.com/IndoNLP/nusa-crowd/pull/359


Surely this should work?

So essentially all i'm trying to do is "post" a collection of status update (DTO's) for transactions (Entities) in the db.
Since i don't know how big the status updates collection might be i decided to "batch" the db calls limiting the number of references I lookup to 1,000 per call.
I then go about pulling out of my db a "batch of transactions", expanding in to all the stuff I will need to apply the status update.
I have also added to the dbContext a property called "User" which pulls the current user from the system and expands in to that users roles like this ...
user = GetAll<B2BUser>(true)
    .Include(u => u.Roles)
        .ThenInclude(r => r.Role)
    .FirstOrDefault(u => u.Id == userName);

... this value is stored as a private property inside the context for the life of the context.
(in case that's relevant)
so ... moving on "foreach update" I pass the update and a batch of transactions in to another method which finds the transaction for the updates reference and then does a simple "t.Status = update.Status" and adds new item to the audit trail returning that in the process.
once all the updates are done I return the audit trail items that were created during this process ...
public async Task<IEnumerable<DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>>> UpdateStatusForBatch(IEnumerable<TransactionStatusUpdate> updates)
{
    if (updates != null && updates.Count() > 0)
    {
        var results = new List<DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>>();

        foreach (var subBatch in updates.BatchesOf(1000))
        {
            var references = subBatch.Select(u => u.Reference).ToArray();
            var transactions = Db.GetAll<T>(true)
                .Where(i => i.References.Any(r => references.Contains(r.Id)))
                .Include(i => i.AuditTrail)
                .Include(i => i.References)
                .Include(i => i.Buckets)
                    .ThenInclude(b => b.Bucket)
                        .ThenInclude(b => b.Roles)
                            .ThenInclude(br => br.Role)
                                .ThenInclude(r => r.Users)
                .ToArray();

            foreach(var bi in subBatch)
            {
                try
                {
                    var result = UpdateStatusFor(bi, transactions);
                    await Db.SaveChangesAsync();
                    results.Add(result);
                }
                catch (Exception ex)
                {
                    var result = new DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>
                    {
                        Item = bi,
                        Success = true,
                        Details = new[] { transactions.First(t => t.References.Any(r => r.Id == bi.Reference)).Audit(Objects.AuditItemLevel.Error, User, ex.Message) }
                    };

                    results.Add(result);
                }
            }
        }

        return results;
    }
    else
        return new List<DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>>();
}

So to my question
I get this exception from EF ...

The instance of entity type 'B2BUserRole' cannot be tracked because another instance with the same key value for {'UserId', 'RoleId'} is already being tracked. When attaching existing entities, ensure that only one entity instance with a given key value is attached. Consider using 'DbContextOptionsBuilder.EnableSensitiveDataLogging' to see the conflicting key values.

... but at no point am I editing adding or removing a B2BUserRole instance to or from anything so i'm little confused as to the cause of this other than "there might be some overlap in the batching queries." (these entities are pulled to do security checks before applying the change to the root transaction entity).
As a test for that case I pulled the batching logic out and just sent a couple of status update to the endpoint that receives this call (one referencing a transaction that does not exist).
As per best practice with EF the aspnet stack contructs a new scoped instance of the context and gives it to my controller which runs the logic above.
So that left me thinking "oh ok, it must not like that I have the user query and the transaction queries happening to return the same B2BUserRole instances or something".
so I wrapped the user call in ...
ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.NoTracking;
...
ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.TrackAll;

... and got the same result.
So the only conclusion I could draw was "oh ok maybe the user is in 2 roles that happened to be attached to this transaction query", and I was right.
So I can't pull related rows even if I don't edit them and apply a change to something related to them then call save changes.
That left me baffled, how do I solve this?

So I found the issue just inside the foreach loop where i loop through the batches I was doing this ...
var references = updates.Select(u => u.Reference).ToArray();

... confusingly though, although this is pulling more out of the DB than intended I don't understand why this would cause the exception shown.
The line of code correction should be ...
var references = subBatch.Select(u => u.Reference).ToArray()

Ok now i'm even more confused ...
It's like EF won't allow an entity instance to appear twice from the same query result, I altered the code like this ...
public async Task<IEnumerable<DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>>> UpdateStatusForBatch(IEnumerable<TransactionStatusUpdate> updates)
{
    if (updates != null && updates.Count() > 0)
    {
        var results = new List<DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>>();

        foreach (var subBatch in updates.BatchesOf(1000))
        {
            foreach(var bi in subBatch)
            {
                using var ctx = Db.NewInstance() as IB2BDataContext;
                var transactions = ctx.GetAll<T>(true)
                    .Include(i => i.AuditTrail)
                    .Include(i => i.References)
                    .Include(i => i.Buckets)
                        .ThenInclude(b => b.Bucket)
                            .ThenInclude(b => b.Roles)
                                .ThenInclude(br => br.Role)
                                    .ThenInclude(r => r.Users)
                    .Where(i => i.References.Any(r => bi.Reference == r.Id))
                    .ToArray();

                try
                {
                    var result = UpdateStatusFor(bi, transactions);
                    await ctx.SaveChangesAsync();
                    results.Add(result);

                    log.Debug($"Status updated for {typeof(T).Name}:\n{result.ToJson(ObjectExtensions.JSONSettings, Newtonsoft.Json.Formatting.Indented)}");
                }
                catch (Exception ex)
                {
                    var result = new DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>
                    {
                        Item = bi,
                        Success = false,
                        Message = ex.Message,
                        Details = new[] { transactions.FirstOrDefault(t => t.References.Any(r => r.Id == bi.Reference))?.Audit(Objects.AuditItemLevel.Error, User, ex.Message) }
                    };

                    await ctx.SaveChangesAsync();
                    results.Add(result);
                    log.Debug($"Failed to update status for {typeof(T).Name}:\n{result.ToJson(ObjectExtensions.JSONSettings, Newtonsoft.Json.Formatting.Indented)}");
                }
            }
        }

        return results;
    }
    else
        return new List<DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>>>();
}

I'm doing 1 get call per context instance here, and the user i'm using for permission checks is from the request scoped instance context so that's literally the only data i pull from the db from a single call of ...
            var transactions = ctx.GetAll<T>(true)
                .Where(i => i.References.Any(r => references.Contains(r.Id)))
                .Include(i => i.AuditTrail)
                .Include(i => i.References)
                .Include(i => i.Buckets)
                    .ThenInclude(b => b.Bucket)
                        .ThenInclude(b => b.Roles)
                            .ThenInclude(br => br.Role)
                                .ThenInclude(r => r.Users)
                .ToArray();

The change applied is then in this method on the transaction entity ...
[PrivilegePerspective(Perspective.Debtor)]
public virtual IEnumerable<TAuditItem> Hold(B2BUser user, string externalState)
{
    var result = new List<TAuditItem>();

    if (user.Can(GetType().Name.ToLower() + "_debtorhold", this as IAmBucketManaged))
    {
        result.Add(ApplyStateChange(TransactionState.Held, Perspective.Debtor, user));
        DebtorExternalState = externalState ?? "Held";
    }

    if (result.Count == 0)
        throw new SecurityException("Access Denied!");

    return result;
}

... the code i have left out for berevity is literally just mapping the update info to this call by doing ...
var transaction = transactions.FirstOrDefault(t => t.References.Any(r => r.Id == update.Reference));
var result = update.NewState switch
                {
                    (int)TransactionState.Hold=> new DetailedResult<TransactionStatusUpdate, IEnumerable<TAuditItem>> { Item = update, Success = true, Details = transaction.Hold(User, update.ExternalStateString) },
                   ...
               }

"User" is coming from Db.User (as described above.

For berevity, to increase visibility of the problem, the extracted "work" now being done on a single DbContext looks something like this for each update ...
foreach(var update in updates)
{
	using var ctx = Db.NewInstance() as IB2BDataContext;
	var transactions = ctx.GetAll<T>(true)
		.Where(i => i.References.Any(r => references.Contains(r.Id)))
		.Include(i => i.AuditTrail)
		.Include(i => i.References)
		.Include(i => i.Buckets)
			.ThenInclude(b => b.Bucket)
				.ThenInclude(b => b.Roles)
					.ThenInclude(br => br.Role)
						.ThenInclude(r => r.Users)
		.ToArray();

	var transaction = transactions.FirstOrDefault(t => t.References.Any(r => r.Id == update.Reference));
	transaction.DebtorState = "New State";
	await ctx.SaveChangesAsync();
}

I'm completely lost as to why this should trigger any sort of issue at all with UserRoles, the last "ThenInclude" call is purely included for the permissions check which I now do another context instance (variable named Db in the original code block).

@TehWardy Thanks for posting. There is a lot here to read--we will get to it when we can.

thx @ajcvickers  ... Bit of an update ...
I discussed this problem with the team, and the key things that came out were, pull the query ...
// this "false" adds .AsNoTracking() to the Set<T>() that GetAll<>(T) wraps
var user = GetAll<B2BUser>(false)
                        .Include(u => u.Roles)
                            .ThenInclude(ur => ur.Role)
                        .FirstOrDefault(u => u.Id == userName)

So straight up, EF should not care about the results of that query any more.
Then we hit the transaction query ....
So I refactored the logic a bit here allowing me to reduce the query to this:
	var transactions = Db.GetAll<T>(true)
		.Where(i => i.References.Any(r => references.Contains(r.Id)))
		.Include(i => i.AuditTrail)
		.Include(i => i.References)
		.Include(i => i.Buckets)
			.ThenInclude(b => b.Bucket)
				.ThenInclude(b => b.Roles)
					.ThenInclude(br => br.Role)
		.ToArray();

Having done that I ran the same update logic which is something like ...
transactions.ForEach(t => t.Hold(User));
await saveChangesAsync();

There's a bit more to "figuring out" the status updates (which I partially touched on above) but that's all DTO work that doesn't touch EF ... the net result is that on each transaction I do a call with an internal priv check which is something like:
public virtual IEnumerable<TAuditItem> Hold(B2BUser user, string externalState)
{
   if(t.Buckets.All(b => b.Roles.Any(br =>br.Role.HasPriv("foo") && br.Role.HasUser(User))))
       ApplyChanges(TransactionState.Held);
}

... i'm deliberately reducing the code down at this point to further clarify the EF interactions so that if clause is basically what user.Can() does from the previous code samples. I built Methods on each level of the chain so for example Roles have a Method called Allows which checks if a given user has a given priv keeping all the role logic inside the role Entity.
All of this stuff is ultimately just simple "is this joined to that" which is why i pull so many tables for the transactions I need to update.
So I refactored the user.Can() "question" to something like ...
t.Buckets.All(b => b.Roles.Any(br => user.Roles.Any(ur => br.RoleId == ur.RoleId) && ur.Role.HasPriv("foo")));

This meant that I could reduce the previous query for getting the transactions down to the one at the start of this comment (not requiring me to pull the rows from the table "B2BUserRoles" that were linked through the chain to the individual transactions) which was ultimately the cause of my initial headache.
The transaction status changing method would ask the security question and if the result was true change a string property value on the transaction and add an AuditTrail Item, so Hold() would ultimately just do something like this after the priv check ...
this.DebtorState = "Held";
this.AuditTrail.Add(new AuditTrailItem() { ... });

... then save changes would be called (initially for a whole batch, but as I hit issues I tried refactoring for each item).
I'm now getting the exception -

Violation of PRIMARY KEY constraint 'PK_Roles'. Cannot insert duplicate key in object 'Security.Roles'. The duplicate key value is (b22055fe-67db-41e7-2310-08d7740618e4).
The statement has been terminated.

Which seems odd since I did get the Role and I did read some information on it, but I didn't update it nor have I added anyhting to any role collections that would trigger an insert.
I'm now convinced that there's something about either how the ChangeTracker works / my usage of it that's causing it / me confusion.
Hopefully i'll hear from you guys soon, sorry about the information overload in this post but I often get tickets shut down for not providing enough information so i'm trying to be as thorough as possible this time.

Ok I figured out where the Role add is coming from, finally!!!
I think this is all down to a complex set of conditions in my code that resulted in an untracked entity being attached to a new one thus it did exactly what it was designed to do and tried to insert the entire entity tree in to the db again.
I'm still doing cleanup and confirming that this is the cause but for now at least I'm confident that it's not in fact a bug in EF Core.
I will confirm then close soon hopefully.

Yup that's the culprit ... one line of code out of place ... typical huh!


Add shebang capability

This pull request adds a new flag (-r) that can be used to directly execute a bish file
Either add -r before the filename
$ bish -r examples/fib.bish

or add
#!/bin/bish -r

to the top of the file
This would close #7 (Unless @reklis wanted a full REPL, but I don't think that was the intent)

This is great, thanks for doing this. There seems to be an issue though. I tried running it on this fib program:
def fib(n) {
  if (n < 2) {
    return 1;
  }
  return fib(n-1) + fib(n-2);
}
fib(15);

By saying ./bish -r fib.bish. And I get the following bash error: bash: line 7: syntax error: unexpected end of file. When I run ./bish fib.bish | bash I don't see that error. Could you look into that before I merge this?
Additionally, I get a compiler warning (clang) with your changes:
src/bish.cpp:83:14: warning: using the result of an assignment as a condition
      without parentheses [-Wparentheses]
    while (n = is.readsome(buf, sizeof(buf))) {

What you meant is perfectly clear, but to silence the warning could you just add parens around that assignment?

I'm not getting this EOF error anymore... could you try adding
fwrite(buf, 1, n, stdout);

at line 84 (right below while ((n = is.readsome(...))) and tell me the output?
I'm getting
#!/bin/bash
# Autogenerated script, compiled from the Bish language.
# Bish version 0.1
# Please see https://github.com/tdenniston/bish for more information about Bish.

function bish_fib () {
    if [[ $1 -lt 2 ]]; then
        echo 1; exit;
    fi;
    echo $(($(bish_fib $(($1 - 1))) + $(bish_fib $(($1 - 2))))); exit;
}

function bish_main () {
    bish_fib 15;
}

bish_main;
987

987 being what it outputs after all that

Here is what I when I replace printing to bash with printing to stdout:
$ ./bish -r fib.bish
#!/bin/bash
# Autogenerated script, compiled from the Bish language.
# Bish version 0.1
# Please see https://github.com/tdenniston/bish for more information about Bish.

function bish_fib () {$

It seems like readsome is not reading the entire contents. Since the input does not need to be read asynchronously, perhaps the solution is to use read instead?

I didn't realize that readsome was async... I thought the only difference was that it returned the amount of data it read. It was prettier too, but oh well, hope this works!

Hmm. I might prefer the while as opposed to the do-while. Couldn't it be possible that the stream is not "good" before the first iteration? In that case there would be an exception thrown. With the while form, we wouldn't try to use a "bad" stream.

is.gcount() would be 0 if the stream was bad, no? Then fwrite wouldn't try to write anything. Passing run_on_bash an empty std::stringstream works fine, but I don't know how to make a bad stream for testing.

All right, thanks for doing this!

No problem! Glad I could help :)


[INTERNAL] JSDoc: Add check for copyright notice

Checks for a copyright notice at the beginning of each file.
The comment doesn't need to be the first comment but it must be placed
before any code starts.
This implements the same check as the internal validation rule
DOCUMENTATION__COPYRIGHT_NOTICE_MISSING.
Adds a jsdoc exclude configuration for sap.ui.model.odata.datajs which
is a legacy redirect file that only exists in the internal core overlay.
It is configured to be excluded from the internal validation, so it also
needs to be excluded from the jsdoc build so that no error is caused.
JIRA: CPOUI5FOUNDATION-329
Cherry picked from https://github.com/SAP/openui5/commit/bcb511074319aca894a026379bd2979627fb69c7

Coverage remained the same at 94.721% when pulling 926ad2b57ae3620c682ddff5520d0e365bca38c7 on jsdoc-plugin-check-copyright-notice into b86f0fa933c2813c2ae563fc3ee68dd3433e6d15 on master.

Thanks. I've updated the commit message.
Yeah it seems that exit codes 0 and 1 are both resolving the promise. I couldn't find a hint why this was done in https://github.com/SAP/ui5-builder/pull/42.

I've added a bullet-point to the list of breaking changes for 3.0

... I couldn't find a hint why this was done in #42.

I think with older versions of JSDoc, we observed the same exit code for success and error. I can't say whether this was still the case for JSDoc 3.5.5 (with which we started in #42), but I guess that was the reason why we ignored the exit code.

Anyhow, we now rely on the exit code in the SAP internal build and should rely on it in the tooling as well (breaking -> 3.0)

Addressed via https://github.com/SAP/ui5-builder/pull/845


Added a pure Java implementation

Not sure if this is the correct place to put this, but I added a pure Java implementation at: https://github.com/sunshower-io/sunshower-arcus/tree/master/arcus-identicon (MIT License)

Thanks for letting me know. I have added a link to your repo in the README.


[feature] Could we only bump versions and make tags for some packages ?

Affected Packages
@changesets/cli
Problem
I use monorepo structure suggested by Turborepo like this:
--apps
  |--app1
  |--app2
--packages
  |--ui

app1 and app2 are just some SPAs, not packages. However, changesets will attempt to publish every recognized packages which is not necessary in this case.
Proposed solution
Maybe provide a flag just to bump versions in package.json and make git tags, skipping the publish step. I think this kind of use case is pretty common nowadays.

@awxalbert do your app's package.json have private: true set?
I might be misunderstanding your problem, but if you want to stop them being published that should work as per docs/config-file-options.md#ignore-array-of-packages.

Yeah, in this case, the private: true seems like the most straightforward solution unless there are some other constraints for the given use case.

It would be nice to be able to ignore version bumping specific packages on the command line.
I'm also working in a monorepo with multiple apps (already "private": true) and publishable packages. When I version bump app1 in preparation for releasing an update of that app, it doesn't mean I'm ready to version bump app2 and app3.
Something like this would be nice:
changeset version --exclude @mycompany/app2 --exclude @mycompany/app3

It would be a slightly complex feature which would involve:

Splitting changesets
Refusing to exclude version bumping dependencies of changed packages

Yeah, I'm glad that you see the complexity involved in this :p It's definitely doable but I'm afraid that I don't have manpower to implement this right now on my own. I would happily discuss the API, implementation details and review a PR though

@Andarist @jsinclair-indeed Thanks for your reply. However, private: true doesn't resolve my problem, the original intention was to prevent changeset from publishing private apps because it's not necessary, I only need tags during a publish phase.
But now I found my problem is resolved in this release<EMAIL_ADDRESS>and it's documented here, it's exactly what I need, I'll close this issue.


feat: add viewermilestone usernotice event

Type: Feature

Fixes: #542

Please add a corresponding event to @twurple/easy-bot as well.


Add section about psa_empty_group_action

when describing action selectors or tables with action selector implementations
in particular add this restriction for v1.0: psa_empty_group_action has to be const to avoid making the TableEntry message too complex

Moving this to P4Runtime v1.0.0-rc4, since this table property is not part of a released version of the PSA spec yet. Maybe PSA 1.1 will be released in October before we release P4Runtime v1.0.0-rc4, otherwise we can always reference the PSA working draft.


Refs #24215 -- Improved error message for unhandled lazy ops.

Before #4130, the condition in StateApps where lazy field operations were pending after all models loaded gave an error message which listed the field pointing at the missing model. Due to the refactor that introspection into the lazy operation was lost, and the error message was simplified to simply show the name of the missing model.
This restores the more informative error message, and lists all missing models and related fields, not just the first encountered.

Ok, made those changes and fixed the tests in Python 3. Thanks for your feedback!

Don't know what's going on with Jenkins here. It seems to be using old versions of certain files for some test runs.

Alright, rebasing a couple of commits seems to have fixed that.

Done.

In this still work in progress?

IIRC, I marked this as WIP because Markus was working on something related, which meant that this might have to change. @MarkusH can you remember what the plan was here?

Markus, still waiting for your feedback. Seems like it would be nice to have a decision for 1.9 so we don't have a regression in the error message.

I honestly don't remember what I might have been working on. I'd say, get it in and if my code -- whatever it's doing -- breaks, I got to fix my stuff.

merged in 25c157e4ccc27702883b0c3a53e0e9b44e19757d, thanks!

Alex, the new test seems to fail on Python 3.5. Do you have a chance to take a look?
======================================================================
FAIL: test_dangling_references_throw_error (migrations.test_state.StateTests)
----------------------------------------------------------------------
ValueError: Unhandled pending operations for models:
  migrations.author
  migrations.publisher

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/tim/code/django/tests/migrations/test_state.py", line 667, in test_dangling_references_throw_error
    project_state.apps
  File "/opt/python3.5c1/lib/python3.5/contextlib.py", line 77, in __exit__
    self.gen.throw(type, value, traceback)
  File "/home/tim/code/django/django/test/testcases.py", line 611, in _assert_raises_message_cm
    self.assertIn(expected_message, str(cm.exception))
AssertionError: 'Unhandled pending operations for models:\n  migrations.author (referred to by fields: migrations.Book.author)\n  migrations.publisher (referred to by fields: migrations.Book.publisher)' not found in 'Unhandled pending operations for models:\n  migrations.author\n  migrations.publisher'

Bisected the Python 3.5 failure to https://hg.python.org/cpython/rev/7839681ca931.
operators.func is returning <function RelatedField.contribute_to_class.<locals>.resolve_related_class at 0x7f668d6fd268> instead of <class 'functools.partial'> so operation.func.keywords raise AttributeError: 'function' object has no attribute 'keywords'. I'm not sure if this is a regression in Python or something we need to fix in Django.

Interesting. Maybe we could explicitly annotate the field name when creating the lazy operation and avoid introspection.

In case we construct nested partials, this is a regression in Python, IMO. Furthermore, https://hg.python.org/cpython/rev/7839681ca931#l1.7 seems to lack a check that func is a partial type.

Yes we do construct nested partials. I'll try to better understand the issue tomorrow unless you can create an issue in the Python tracker to explain it.

I opened a ticket on Python's tracker: http://bugs.python.org/issue25137

Seems like the Python ticket indicates we shouldn't rely on the behavior of Python < 3.5.
Collin, does https://github.com/django/django/pull/5294 implement what you had in mind?

Yes. I think so.


Fix TestLoggedProgress

This was fixed in https://github.com/argoproj/argo-workflows/pull/10097, hence it can be closed ?


Grouping flags?

Hi,
is it possible to group flags so that they appear together in the --help text? I'm using pflag in my backup program, there I have for example the options to expire backups according to a policy. Before (with the go-flags) library the options were nicely grouped together (in the order they were defined):
$ restic forget --help
Usage:
  restic [OPTIONS] forget [snapshot ID] ...
[...]
[forget command options]
      -l, --keep-last=    keep the last n snapshots
      -H, --keep-hourly=  keep the last n hourly snapshots
      -d, --keep-daily=   keep the last n daily snapshots
      -w, --keep-weekly=  keep the last n weekly snapshots
      -m, --keep-monthly= keep the last n monthly snapshots
      -y, --keep-yearly=  keep the last n yearly snapshots
          --keep-tag=     alwaps keep snapshots with this tag (can be specified multiple times)
          --hostname=     only forget snapshots for the given hostname
          --tag=          only forget snapshots with the tag (can be specified multiple times)
      -n, --dry-run       do not delete anything, just print what would be done

Now with pflag and cobra they are printed in alphabetical order:
[...]
Usage:
  restic forget [flags] [snapshot ID] [...]

Flags:
  -n, --dry-run                do not delete anything, just print what would be done
      --hostname string        only forget snapshots for the given hostname
  -d, --keep-daily int         keep the last n daily snapshots
  -H, --keep-hourly int        keep the last n hourly snapshots
  -l, --keep-last int          keep the last n snapshots
  -m, --keep-monthly int       keep the last n monthly snapshots
      --keep-tag stringSlice   always keep snapshots with this tag (can be specified multiple times)
  -w, --keep-weekly int        keep the last n weekly snapshots
  -y, --keep-yearly int        keep the last n yearly snapshots
      --tag stringSlice        only forget snapshots with the tag (can be specified multiple times)
[...]

Is there a way to group some flags so that they are printed together?

If #113 will be merged, you will be able to order flags how you want.

That'd be awesome! Unfortunately it looks like this library is dead :cry:

Fixed in #113

@bogem @eparis Thank you very much!

Hm. I don't have any idea on how to use it with cobra. The sample program at https://gist.github.com/1b3ccbf904682e0b9fd113ee42a47471 always prints the flags in alphabetical order:
go run main.go -h
Usage:
   [flags]

Flags:
  -a, --alpha string     help for alpha (default "empty")
  -e, --epsilon string   help for epsilon (default "empty")
  -z, --zulu string      help for zulu (default "empty")
err: <nil>

Hm, very interesting. I am trying to figure out not what's wrong.

Hm, it seems that cobra creates new flag sets all over the place...

There's also a bug: When you call cmdRoot.LocalFlags() before defining any flags, the help message is empty but the flags are still valid: https://gist.github.com/9cc52f7c0f5fc9d2cb84a1fcb43b11dc
$ go run main.go -h
Usage:
   [flags]
err: <nil>

$ go run main.go -a foobar
do something
err: <nil>

I think, we should create PR to make SortFlags work correct in cobra

There's also a bug: When you call cmdRoot.LocalFlags() before defining any flags, the help message is empty but the flags are still valid
Yes, it's cobra's issue. You can make issue in cobra, so I will try to figure out and fix it.

cmd.Flags().SortFlags() = false doesn't work for cobra. I think, we should add one more property to cobra.Command to switch the sort of flags.

I'll create two issues in cobra, so we can continue the discussion there

https://github.com/spf13/cobra/issues/403
https://github.com/spf13/cobra/issues/404


Change Assimp to Silk.NET.Assimp implementation

PR Details
This PR completes johang88 attempt to migrate from native C++ Assimp code to Silk.NET bindings (link to his PR).
Changes

I checked previous code and can tell it's a 1-1 recreation of the native MeshConverter. However ProcessMeshMaterial() was incomplete so I wrote missing part.
Also, my PR includes small changes that simplify MeshConverter and changes proposed by other contributors (those I could understand)

Overall, I have not noticed any regression or difference in the import of objects, but I would be happy to hear any feedback :)
Once, merged, I believe it will unlock the possibility of further development of the asset compiler for platforms other than Windows and work on glTF.
Related Issue
#1394
#360

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[x] Breaking change (fix or feature that would cause existing functionality to change).

I think if you add [WIP] to the title it is enough. But don't know.

Regarding the spaces, https://github.com/stride3d/stride/blob/master/.editorconfig is present, but not sure if it is in the right location?

To avoid this kind of problems with tabs and spaces I like to use this extension from Microsoft Labs. Highly recommended.

Yeah, of course. My review is only suggestions to point to typos, indentation, comments, etc. Then comments about what we should aim for in a future PR.

Question: should this be marked as a draft? There's lots of TO-DOs.

Although I havent cross referenced anything myself, there were a bunch of TO-DOs in the C++ files as well.

Why have you closed the PR?

I am sorry, I force pushed the wrong branch on the remote by mistake and PR was automaticaly closed 😢
At least I have a lesson to avoid that.

Alright, so I refactored and moved the classes to separate files and applied all the review suggestions I could.
I think my changes are ready to be reviewed again.

Ok. Some more comments. I hope you don't hate me too much for some of these ones 😁.

@Ethereal77 Not at all. That's the purpose of code review, to make sure everything is alright 😅
Thanks again for all suggestions. I think that's the good point to have identation consistency, so I changed for every file to space instead of tabs. Would be also nice to take care of .editorconfig file in future. Other than that I think reviewed again (I hope didn't miss anything 😂)

Nice attitude 🙂

If there are no objections, I think changes are ready to merge 😊

Thanks a bunch for taking the time to look at this @Ethereal77, and @NicusorN5 for providing additional insight !
I'm guessing this one will be taken care of in another issue/pr @Jklawreszuk ?
https://github.com/stride3d/stride/pull/1719/files#r1285751424

And of course, thank you for going through this stuff @Jklawreszuk

Thanks a bunch for taking the time to look at this @Ethereal77, and @NicusorN5 for providing additional insight !

Also, we should thank @johang88 for his big effort 🫡🫡 . I'm impressed.

I'm guessing this one will be taken care of in another issue/pr @Jklawreszuk ?

Definitely, we need now to discuss what to do next with FBX importer, because having cpp compiler as requirement during building is pain and I think potential PR would cover additional changes to assimp importer.


raidboss: E9S Individual Phaser Callouts

This looks good to my eyes!  Let me know how testing goes.


5048-ES and Eco mode - what is going wrong with my set up?

Hello,
I've read through https://github.com/mletenay/home-assistant-goodwe-inverter/issues/164 but my issue is different.
I recently started using this experimental version (master) of the integration to make use of the eco_charge mode. Mostly it works wonderfully for me. I can change to eco_charge mode and set ECO SOC and ECO power mode without issue and it works at the correct values for the settings I set.
The issue I'm facing is that every now and then the battery will not carry load after changing between eco_charge and general mode. E.g. DOD (On-Grid) is set to 65%, SOC is at 70%, mode is General. The battery should discharge to carry load until DOD is reached (so 35% of battery can be used). It does not and load is instead carried by the grid. Changing to other modes and back again (backup, eco, eco_charge, eco_discharge), does not help and load is still carried by the grid. I've also tried to see if switching off the battery and switching it back on works. It does not. Forcing off-grid mode (disabling grid physically) lets battery carry load, but when grid is available again, load is carried by the grid. The only way to get it to behave as expected is to completely restart the inverter and then when it is back on again, it works as expected and load is carried by the battery until DOD is reached. A day or two later, the behavior changes again and the battery will not carry load.
Please note that I've only seen this behavior since starting to use eco_charge mode. Previously (I was on HA official version) I was not changing settings via HA so I'm not sure whether it is a problem with my inverter, this integration etc.
I am on firmware 2424H and ARM v17, which Goodwe tells me is the latest.
Has anyone else experienced anything like this?
I've enabled the debug logging but I don't know where to find the output. Sorry, a bit of a noob.
Here is a screenshot of what I'm experiencing:

Look around 16:00.

DOD is set to 65%
SOC is at 65%
PV generation has dropped below load, so battery should start carrying load until DOD is reached.
Instead, grid starts carrying load.

Around 20:30 is when I restarted the inverter. I had to set up DOD values again. Battery started taking load.
Automation:
Automation
And sensors:
custom sensor
custom sensor

Okay, so after some further testing I can conclude the following:

Using Solar Go and changing between eco modes and general mode works as expected with the battery taking load if required after changing back to general mode.
If HA was used to set eco (charge) mode and is used to change it back to general mode, the issue this ticket was logged for occurs.
If eco mode is set using Solar Go and HA is used to change it back to general mode, the issue this ticket is logged for does NOT occur.

It seems to be that something in the way that this integration sets eco_charge mode, prevents the inverter from fully returning to general mode. Perhaps some value is changed that is not returned to it's previous value when the mode is changed back to general?
Would really like to hear your thoughts on this @mletenay . I'm available on Discord as well if you want me to test or share more info.

have you tried to set the DoD again after changing back to General mode?

I do. I set it after changing back to general mode:

I am curious, does the Eco charge SoC value change when using Solargo?
Eg. before changing from eco charge to general, try setting the eco charge SoC to similar value you have for DoD.
So like, if you have DoD set to 65%, set the SoC to 35%. Just in case its actually stuck in eco charge mode still.

What worked yesterday isn't working today. Now matter what I do in Solar Go, I cannot get the battery to start discharging:

I had issues with version 17, where before on version 13 I could change between Backup and General in HA and it would function as expected. With 17 it had some issue, I can't remember the exact symptoms, but I think it was something similar to what you are experiencing, so I decided to have them roll back to v13.

@PilgrimToHyperion
Hello
Have you solved the problem ? can you switch from eco_charge_mode back to general mode?

@dl6dbh , hello. It seems to be working normally now, but also, I've not paid much attention to it as I've not received feedback on my question. It seems intermittent.


Update microsoft-edge-security-cert-verification.md

Extend policy lifetime and update known issues section.

I think for the docs we are supposed to say "Microsoft Edge" and not just "Edge"
@dan-wesley Can you take a look?

Good call. I updated it.


Update CODE_OF_CONDUCT.md

Fixed typo as seen on Twitter

Merged!! Thank you @Gedankenstuecke!

Love the gif, that totally wants me to contribute more to see more of them!


The document needs to address environment that already has Retail Server and Channel database

This document should address the setup for a Dev box that has Retail Server and Channel database already installed also and not just a CSU.
As written in another document for setting up CSU it says:" Do not install a CSU on a developer environment, which typically already has a configured Retail Server and Channel database."
And for Norway it applies as written in this document:
"Because of limitations of the new independent packaging and extension model, it can't currently be used for this localization functionality."
So the default for Norway would be a developement boks that already has Retail Server and a Channel database..
One thing I found out when implementing this is that the current standard code and setup does not support that the certificate has a password.  So I had to update the TryGetCertificateFromKeyVault method in the CertificateProvider Class to work around this. Would have been better if the password for ceriticate would be part of the setup, done in the same way as the certificate setup.

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

ID: 32819466-5d8f-cbab-d5b6-b972668f8cf6
Version Independent ID: 32819466-5d8f-cbab-d5b6-b972668f8cf6
Content: Deployment guidelines for cash registers for Norway - Commerce | Dynamics 365
Content Source: articles/commerce/localizations/emea-nor-fi-deployment.md
Service: dynamics-365-commerce
GitHub Login: @EvgenyPopovMBS
Microsoft Alias: epopov

Created bug 656127 to track this issue.

@Halldor10, thanks for the feedback. We will update this documentation when we implement the support for the new independent packaging and extension model, which is planned for one of the next updates.
As for the support of certificates with passwords, that's correct: the functionality does not support such certificates. If you need to use a certificate with a password, you have to implement a customization.


Spike: basing it all on supabase

The idea of supporting N backend types is nice, but If we doubled down on supabase, that might increase velocity and focus.
Auth, storage, database, graphql, permissions, rest etc
And it's a good bet for the long run.
Spike it out

[x] fire up supabase free
[ ] migrate the schema into existence
[ ] test the graphql api
[ ] wire in the data editor to issue mutations back to supabase

[ ] rework backend/api/src/routes/models.ts to work with one supabase postgres
[ ] credentials can be in environment variables for now, ie no need to store secrets yet
[ ] check the graphql api in supbase works
[ ] test we can make a model in model editor, and apply it to supabase, and see it in supabase console etc

Not basing it on supabase gql any more.  That requires a schema per tenant which is not the supabase model.  So we're going to try multi-tenanted model and record storage based on json, with a multi-tenanted gql api dynamically generated from that


Have a cheat sheet

The purpose, or rather the main benefit, is that people easily could try out the language. One for far-fetched idea, would be to have a cheat sheet available in a live compile environment (in another area of the visible screen) or even auto-fill when starting typing. However, this I would guess is way of scope. But a docs section containing comparisons between 'known', 'common' langs could be in scope, I would think. Just an idea!
Docs

Filter and/or search field

Table (comparisions) such as

| 007/Alma  | Perl6 | C      | JS                              | Java  |
|--------------|---------|--------|---------------------------|-------|
| say           | print   | printf | process.stdout.write | print |

Cheat sheet [name of operator, method (...)] - [short, simple example]

Web

Compile projects live! (Include field for 'inputs', similar to a browser console)

Gonna pare down this bit for now. It's a noble goal, but we already have it covered in other issues. (Loose plan: rewrite the compiler/interpreter in TypeScript, and then build a web frontend for it.) Better to keep issues separate and with a narrow focus.

Not sure how front-and-center the comparison table should be. If we're talking about the cheat sheet, I think the room on there is too precious. Probably 25% of the cheat sheet would be spent on syntax/basics, and 75% on macro lore. (Which also seems to be the rough proportions for #67, even though most of it is as yet unpublished.)


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1641

Questions ?
Please refer to the Backport tool documentation


Add support for more java.time objects

In this issue, support for java.time.Instant was added.
Please can support for more of java.time be added, specifically java.time.Duration would be helpful.

This is really frustrating for us as well.


Add serathius as owner

/assign @brancz

/lgtm

/verify-owners

/lgtm

/verify-owners

/verify-owners

@brancz can you approve?
Thanks

/approve


feat: adding form and getFieldState to mdx

Adding getFieldState + form MDX files
I'm still not super happy with the inline code but this will unblock some moving tables with code.
Missing 6 more pages to migrate to MDX

formstate

register

usecontroller

usefieldarray

useformstate

crashed

The root cause is the sidebar is not being resolved. I'm interested in why it's happening now since this change doesn't seem to have anything that could affect that, but I have a local change to fix it

There seems to have been 2 issues.

SideMenu showing up null
git file naming casing. I changed it locally to be lowercase and it worked as expected, but git doesn't detect those types of changes 😞

  const doc = allDocs.find(
    (doc) => doc._raw.flattenedPath === params?.slug?.join("/")
  )


On Wed, 14 Mar 2018, 22:42 Tim Smith<EMAIL_ADDRESS>wrote:

@tas50 commented on this pull request.
In content/membership.md
https://github.com/sous-chefs/website/pull/30#discussion_r174632548:

Dan Webb (@damacus)

Owners
-Folks with complete control over the sous-chefs organization. This currently consists of cookbook maintainers.
+Folks with complete control over the sous-chefs organisation. This currently consists of cookbook maintainers.
crazy!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/sous-chefs/website/pull/30#pullrequestreview-104025077,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACfUpcFUiEJq8GyA2Oq1UQTkKlr-Q-pks5teZzngaJpZM4SrRF7
.


Zen browser ?

Hi, thanks for making this.
Do you know if it would work on Zen browser (firefox-based)?
Also, is a restart required after installing or does it work immediately?

Hi!
It should work as long as it uses the same local files for saving state.
You should not need to restart.
Disclaimer: this repo is pretty much dead. If you need some modification feel free to ask or submit a PR.


Creating Configuration Recorder failed: MaxNumberOfConfigurationRecordersExceededException

I have problem with module i get that error message:
`Error: Creating Configuration Recorder failed: MaxNumberOfConfigurationRecordersExceededException: Failed to put configuration recorder 'test' because the maximum number of configuration recorders: 1 is reached.
on .terraform/modules/config/terraform-aws-config-2.5.4/config-service.tf line 24, in resource "aws_config_configuration_recorder" "main":
24: resource "aws_config_configuration_recorder" "main" {`
i base configuration:
`module "config" {
source  = "trussworks/config/aws"
version = "2.5.4"
config_name        = "test"
config_logs_bucket = "test"
}`
`provider "aws" {
region     = "eu-central-1"
version = "~> 2.53.0"
}
terraform {
required_version = "~> 0.12.18"
}`

This error is due to you already having a running instance of AWS config in eu-central-1. AWS has a hard cap of 1 config recorder instance per region. So you'll either need to delete the running AWS config instance or tell terraform to setup an instance in another region.

True story bro.... thanks for info it is working now :)


Intranet URLs not accessible in Chrome when targeted by WIP

My devices are hybrid Azure AD join.
Have targeted a WIP Policy with Enrollment.
Even added /AppCombat? in Cloud resouces.
When trying to access organization Internal websites it's getting blocked in Chrome & when trying to access the same by IE or EDGE which are marked as managed apps, it's working fine.
Want to know, is it possible to access Intranet sites by Google Chrome when targeted by WIP.
Came across below add-in, does it can help me to access Intranet websites from chrome.
https://chrome.google.com/webstore/detail/windows-10-accounts/ppnbnpeolgkicgegkbkbjmhlideopiji?hl=en

@officedocsbot assign @mypil

@ankush271017 - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

@ankush271017 : At first, I thought that this question should be asked in one of the Google Chrome forum pages, but then I remembered that Microsoft has launched a Chromium-based version of Edge and that this might become very relevant to Microsoft too.

https://www.microsoftedgeinsider.com/en-us/download/

Could you perhaps, in addition to telling us which Microsoft Docs page is in need of improvement as a reference page for your issue, also test if the insider builds of Microsoft Edge Chromium will face the same issue as you have found using Google Chrome?

In my case, Google Chrome is added in exempt apps list.
But the intranet links are getting blocked for version Windows 1803 & the same is working for 1809 version.
I'm unable to understand the difference in behavior.
I've added the intranet URLs in Cloud resources.
Just wanted to understand the behavior & how exactly it should work.
If we try to access internal links on Edge or IE, it's working for both the versions.
I think now Chrome has capability of understanding Conditional Access.
https://docs.microsoft.com/en-us/windows/security/information-protection/windows-information-protection/app-behavior-with-wip
https://docs.microsoft.com/en-us/windows/security/information-protection/windows-information-protection/enlightened-microsoft-apps-and-wip

@ankush271017 - Thank you for providing additional details.
I think the best way forward is if you can open a service ticket so this can get resolved ASAP. Based on outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for Windows 10 products:
https://support.microsoft.com/en-us/hub/4338813/windows-help?os=windows-10
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@officedocsbot close


WIP: Remove mutating global state in blackbox-exporter objects

Same as #857 but for blackbox-exporter.
Additionally this adds resource requests/limits for kube-rbac-proxy and unifies assigning labels for objects.
Part of #566.
WIP until #857 is merged and I rebase this PR.

#857 is merged, this PR is ready for review.
/cc @prometheus-operator/kube-prometheus-reviewers

#857 is merged, this PR is ready for review.
/cc @prometheus-operator/kube-prometheus-reviewers

I merged this as CI was green, approvals were here, and the only change was a rebase.

I merged this as CI was green, approvals were here, and the only change was a rebase.


Error during datasource creation

I faced the issue during an application starting
Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'healthIndicatorRegistry' defined in class path resource [org/springframework/boot/actuate/autoconfigure/health/HealthIndicatorAutoConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.boot.actuate.health.HealthIndicatorRegistry]: Factory method 'healthIndicatorRegistry' threw exception; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'dbHealthIndicator' defined in class path resource [org/springframework/boot/actuate/autoconfigure/jdbc/DataSourceHealthIndicatorAutoConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.boot.actuate.health.HealthIndicator]: Factory method 'dbHealthIndicator' threw exception; nested exception is java.lang.ClassCastException: net.sf.log4jdbc.sql.jdbcapi.DataSourceSpy cannot be cast to com.zaxxer.hikari.HikariConfig 	at org.springframework.beans.factory.support.ConstructorResolver.instantiate(ConstructorResolver.java:627) ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
The lib version:

com.integralblue
log4jdbc-spring-boot-starter
2.0.0

Spring Boot version: <spring.boot.version>2.1.8.RELEASE</spring.boot.version>
I tried to change version to 1.0.2 but unsuccessfully.
Without the dependency the application is started. Please, help.

I have same issue too. any suggestion?


DOC: Change "lower" to "smaller" in atol/rtol documentation

Lower implies negative, whereas smaller implies magnitude which is the
correct interpretation in this case.
Reference issue
N/A
What does this implement/fix?
This clarifies the documentation for IVP methods a little. The word "lower" was confusing because "lower" doesn't imply magnitude which is what we actually want to imply here. "smaller" fixes this.
Additional information
N/A

Why change lower to smaller and lowest to smallest? It doesn't make sense to me. On the contrary, it may be more common to use lower or lowest when describing absolute/relative tolerance or residual norm or bounds in some journals such as SIAM.

-5 is lower than 2 but not smaller. 2e-1 is both lower and smaller than 2 but it seems to me that the fact that it's smaller is the more relevant fact. Also, elsewhere in the text we use "larger" so it would make sense to be self consistent.
Can you point out examples where people use "lower" to refer to absolute/relative, and where it could not be replaced with "smaller"?

Would most users understand this distinction between lower and smaller?
For me both refer to inequality x < y or x <= y.
There is nothing about absolute value in either smaller or lower.
("The lower bound is the smallest value that is valid" is also a proper statement if the lower bound is - 5.)
if values cannot be negative like rtol, atol, then it's also the magnitude, but not otherwise (in my readings)

Would most users understand this distinction between lower and smaller?
For me both refer to inequality x < y or x <= y.
There is nothing about absolute value in either smaller or lower.
("The lower bound is the smallest value that is valid" is also a proper statement if the lower bound is - 5.)
if values cannot be negative like rtol, atol, then it's also the magnitude, but not otherwise (in my readings)

@josef-pkt You're right in that lower is not strictly incorrect, especially since we're talking about nonnegative numbers here, but I maintain that 'smaller' is perhaps the better choice of word here.

@josef-pkt You're right in that lower is not strictly incorrect, especially since we're talking about nonnegative numbers here, but I maintain that 'smaller' is perhaps the better choice of word here.

@nbelakovski IMO, using "lower" or "smaller", even "less than (or equal)" is not an essential issue of correct and incorrect. A large number of articles use them. If not necessary, no modification is required. Here are the papers that use the "lower" or "low" keyword. Just FYI.
[1] J. J. Dongarra, et.al., Proceedings of the Fourth SIAM Conference on Parallel Processing for Scientific Computing, SIAM, 1990.
(search for the key sentence: The tolerance TOL for the stopping criteria was set to 2x10^(-7), slightly lower than machine precision)
[2] Jim E. Jones and Carol S. Woodward, Newton-Krylov-Multigrid Solvers for Large-Scale, Highly Heterogeneous, Variably Saturated Flow Problems, Lawrence Livermore National Laboratory, 2020.
(search for the keywords: the tolerance is low)
[3] O. Nasraoui, S. Goswami, Mining and Validating Localized Frequent Itemsets with Dynamic Tolerance, Proceedings of the 2006 SIAM International Conference on Data Mining (SDM), 2006.
(search for the keywords: a lower tolerance)
[4] W. Gropp, D. Keyes, L. C. McInnes, M. D. Tidriri, GLOBALIZED NEWTON-KRYLOV-SCHWARZ ALGORITHMS AND SOFTWARE FOR PARALLEL IMPLICIT CFD, The International Journal of High Performance Computing Application, 2000.
(search for the key sentence: For problems in which convergence to a low-residual steady state is required)
[5] J. Zhou, W. Liu, X. Li and Z. Qi, 3D transient electromagnetic modeling using a shift-and-invert Krylov subspace method, J. Geophys. Eng. 15 (2018) 1341–1349 (9pp).
(search for the sentence: the relative residual is lower than a given tolerance value tol)
[6] Jean-Franc¸ois Lemieux et.al., Using the preconditioned Generalized Minimum RESidual (GMRES) method to solve the sea-ice momentum equation, JOURNAL OF GEOPHYSICAL RESEARCH, 2008.
(search for the key sentence: The GMRES iteration is stopped when the residual norm (Rj) is lower than a chosen tolerance)
[7] V. Simoncini, D. B. Szyld, FLEXIBLE INNER-OUTER KRYLOV SUBSPACE METHODS, SIAM J. Numer. Anal., 2003.
(search for the keywords: a lower residual norm)
[8] Hypre documents, LLNL.
(search for the key sentence: converge to very low relative and/or absolute tolerances)

For completeness here are papers or book that use "smaller" when discussing tolerance:
[1] Flournoy, N., Rosenberger, W. F., Wong, W. K. (1998). New Developments and Applications in Experimental Design: Selected Proceedings of a 1997 Joint AMS-IMS-SIAM Summer Conference. United States: Institute of Mathematical Statistics.
(search for the sentences: "relative tolerance is smaller than" and "smaller than a prespecified tolerance")
[2] SIAM Journal on Numerical Analysis: A Publication of the Society of Industrial and Applied Mathematics. (1999). United States: The Society.
(search for the sentence: "with the understanding that a smaller choice of tolerance produces a more accurate result")
[3] SIAM Journal on Scientific Computing: A Publication of the Society for Industrial and Applied Mathematics. (2008). United States: SIAM.
(search for the sentence: "discarding entries smaller than a certain drop tolerance")
[4] Clarke, A. S., & Wiley, J. D. (1987). Numerical simulation of the dense random packing of a binary mixture of hard spheres: Amorphous metals. Physical Review B, 35(14), 7350.
(search for: "until the maximum overlap drops below some other (smaller) tolerance")
There are, of course, more.
Note how "smaller" cannot be replaced with "lower" in some of these cases without potentially changing the meaning, whereas in the previous examples if "lower" is replaced with "smaller" the sentences still make sense.
More to the point, if we were talking about a paper for publication to a journal read by professionals in the field, this difference would not matter. However, the audience of scipy documentation consists of both professionals and newcomers to the field of numerical integration. For people who are learning about the concept of numerical integration for the first time, we should strive to be as precise as possible. Additionally, some of those people may not have English as a first language and so the extra precision may be helpful to them as well.
I think we can argue about this all day but what we really need is an executive decision from the maintainers of scipy. I welcome whatever decision they'd like to make.

Thanks @nbelakovski. I agree that "lower" suggests "closer to negative infinity" whereas "smaller" suggests "closer to zero", which is the more relevant aspect here. I also agree that since we're talking about a non-negative number, the distinction may not be very important to all readers. However, a decision is needed. I do not always write precisely, but I think it's good practice to do so, so I'll merge.


Support for Lambda Layers

Set the right artifact paths for Lambda layers after TypeScript Compile

Can you help pushing this change?

Does this fix help resolve issue mentioned here? - https://github.com/serverless/serverless/issues/5556
Can this get merged in?

@schickling ^^

@medikoo I resolved these conflicts in https://github.com/serverless/serverless-plugin-typescript/pull/267, would love for you to take a look. Thanks!

@mmeyers-xomly There's still conflicts, can you fix them?

Hey @medikoo and @micheleangioni, I think these are resolved in my PR. I can't update this PR here since I don't have access, but it looks like mine has them all resolved: https://github.com/serverless/serverless-plugin-typescript/pull/267

Amazing, thanks @mmeyers-xomly . This should be good I think, @medikoo


chore(docs): create deployment steps (1-4) on top of foundation

Adds instructions on how to deploy genai repo on top of existing foundation instance (i.e. modifications made for steps from 1-org to 4-projects).
validated and tested the instructions on foundation v4.0.0

Conflicts that need to be resolved

fixed by restoring conflicting files to origin/main.


[BUG-Python] Enum not hashable

Enum not hashable
enum types, specifically array/list are not hashable and cannot be used in combination with GroupBy or Maps. Maybe reflect .NET usage by using hashSeq function internally.

Is this issue still up to date? ISA core tests in python branch are all succeeding besides one caused by https://github.com/fable-compiler/Fable/issues/3717


Update phoneMethods property to use PUT method

Fix #54
According to https://docs.microsoft.com/en-us/graph/api/phoneauthenticationmethod-update?view=graph-rest-beta&tabs=http the method is PUT not PATCH.
This PR intends to fix this behavior.

Salut Jb!
Thanks for your contribution, we really appreciate it.
This file is being automatically generated, so we don't take manual changes for it.
However, I believe the service should be using PATH instead of PUT and is not following the OData conventions.
@darrelmiller any thoughts?

Salut Jb!
Thanks for your contribution, we really appreciate it.
This file is being automatically generated, so we don't take manual changes for it.
However, I believe the service should be using PATH instead of PUT and is not following the OData conventions.
@darrelmiller any thoughts?

It was initially created as PUT and we provided guidance to the team that PATCH should be supported.  My understanding was that they were going to be doing that.  I will follow up with the PM.

It was initially created as PUT and we provided guidance to the team that PATCH should be supported.  My understanding was that they were going to be doing that.  I will follow up with the PM.

I close this one as I hope it will be fixed in other ways soon...


Canary releases?

Hi,
The current available open source canary deployment tools don't seem like they would play well with helmfile.
Flagger for example, creates/modifies a deployment gradually into a state that ends up being different than the state in the helmfile releases., so your source of truth has changed.
Since the industry is going towards canary deployments, and they are indeed an invaluable tool in production, how can helmfile play along natively?

@dudicoco Hey! Helmfile and Flagger works perfectly fine together.
Flagger works by watching the K8s deployment resource and the Canary resource managed by Helmfile. Once the deployment or the canary is changed via a helmfile apply, Flagger detects the change and updates Flagger-managed primary and canary deployments, coordinate traffic, and so on for canary deployment.
I can recommend reading https://docs.flagger.app/usage/how-it-works to see how it works. Also, please feel free to ask me anything if it still doesn't make sense.

Hi @mumoshu,
So I've read https://docs.flagger.app/usage/how-it-works a couple more times.
Does flagger work by actually overriding the default k8s deployment controller?
So if I change the image tag in my standard k8s deployment objects, flagger will detect this change and perform the canary deployment while disabling the k8s standard deployment rolling update?

Does flagger work by actually overriding the default k8s deployment controller?

No

disabling the k8s standard deployment rolling update?

No. Flagger doesn't use the relicas of your deployment managed by Helmfile.
See https://github.com/weaveworks/flagger/blob/c367e65672b88104a8dcb44756b9d02782cdbbc1/pkg/canary/deployment_controller.go#L233-L258 - Flagger copy the targeted deployment(managed by e..g Helmfile) spec to generate the primary deployment(managed by Flagger), except the replicas field.
You usually don't send real traffic to the one managed by Helmfile. So, you can set replicas of the deployment managed by helmfile to e.g. 1 or even 0.

Ok I finally got it :)
Thanks!

@dudicoco Glad to help! Also - I'd greatly appreciate it if you could share your experience once you get it working 😄

Will do @mumoshu!
How was your experience with Flagger? Are you using it in production? With or without Istio?

Gonna try this out soon as well. Thanks for the topic risen!


CoreFoundation: add US/Pacific TimeZone mapping for Windows

Windows does not have the same timezone information available as the
other platforms.  We import a subset of the CLDR (due to duplicate
mappings) into Foundation to provide the translation.
TestCalendar/test_nextDate usees US/Pacific for a test case.
Explicitly add this to the Olson -> Windows mapping.  We cannot provide
a Windows -> Olson mapping entry for this value as that would have
multiple values for the canonical America/Los_Angeles key.  This
repairs the TestCalendar/test_nextDate test case on Windows.

@swift-ci please test and merge


Add Gradient Styles for Text View

Ticket: #583
In this PR, I added gradient styles for the text view.
Text:
  styles:
    textStyle:
      fontSize: 20,
      color: green,
      fontWeight: w900,
      gradient:
        {
          colors: [0xFF675FF3, 0xFF4B93E7, 0xFF48CFD7],
          start: centerLeft,
          end: centerRight,
        }
  text: Ensemble App Developer

NOTE: Text color will be excluded from the text styles if you have a gradient.

Please remember to add the schema. Make sure to reuse the same schema style that Row/Column's gradientBackground use.

Can you just review my PR? I updated new changes and added schema


Resolve naming inconsistency with CLI.

In the CLI, this is a called a "chain".  Here it's called "DB".  This is a confusing inconsistency.
Given that our marketing is promoting that we make it easier to work with blockchains, it makes more sense to me to call this a "chain" as well.  "DB" implies database and a blockchain is not going to meet the expectations of coders who think they're getting a database.

coming soon.


Add How-D - DASH (ft. Peter Griffin) [Official Music Video] by HowD

Song Name
How-D - DASH (ft. Peter Griffin) [Official Music Video]
Artist Name
HowD
Source
Youtube
Youtube Link
0KlwHJpUC24
Direct File Link
No response
Song ID
21
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 97457564

Please reopen it
github-actions[bot] @.***> schrieb am Do., 29. Aug.
2024, 04:25:

This submission is stale because it has been open for 14 days with no
activity. If it is still relevant, please leave a comment and we will
reopen it.
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/2839#issuecomment-2316591420,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDORJA23DIR7ZNF5C73OMLDZT2BCZAVCNFSM6AAAAABMQQHO7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWGU4TCNBSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

accept


Executable ca.crt error

Hi i have these 2 errors how to fix them please?

Thanks

First problem your urls.txt or search.txt is empty.
Second problem I'm not sure yet why this error is happening for some user. Try running from source code.

All right, I guess I found the problem.  I can't reproduce the error on my machine.
youtube_viewer.zip

Très bien, je suppose que j'ai trouvé le problème. Pouvez-vous exécuter cet exe et me dire si ce problème est résolu? Je ne peux pas reproduire l'erreur sur ma machine.
youtube_viewer.zip

Thanks except that now I have this problem and I can't find Chrome version 90

I faced this when I tried to run it on azure. For some reason undetected chrome driver download v90 chrome driver. But in my local machine it download v89. So what I did I downloaded v90 beta Google Chrome and uninstalled v89 Google Chrome. And in C:\\Program Files\\Google you will find Google Beta I guess. Rename whatever the folder name it is to Google


Error: setObjectForKey: key cannot be nil

First of all I want to say that EasyMapping in the best mapping tool I ever used! Thank you for it!
I think that source if these bug hiding somewhere on back-end, because my app works fine and then bam and i get error.
May be you know what these error can mean? And how can i solve this?
Here log from my app.
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** setObjectForKey: key cannot be nil'
*** First throw call stack:
(
	0   CoreFoundation                      0x0360e746 __exceptionPreprocess + 182
	1   libobjc.A.dylib                     0x03297a97 objc_exception_throw + 44
	2   CoreFoundation                      0x034ffc9c -[__NSDictionaryM setObject:forKey:] + 940
	3   MY_APP                              0x00301bbd _ZL58__arclite_NSMutableDictionary__setObject_forKeyedSubscriptP19NSMutableDictionaryP13objc_selectorP11objc_objectS4_ + 68
	4   MY_APP                              0x00110927 -[EKCoreDataImporter cacheObject:withMapping:] + 423
	5   MY_APP                              0x00111188 -[EKManagedObjectMapper objectFromExternalRepresentation:withMapping:] + 632
	6   MY_APP                              0x001124cf -[EKManagedObjectMapper arrayOfObjectsFromExternalRepresentation:withMapping:] + 463
	7   MY_APP                              0x00111f61 __75-[EKManagedObjectMapper fillObject:fromExternalRepresentation:withMapping:]_block_invoke50 + 369
	8   CoreFoundation                      0x0353d01a __65-[__NSDictionaryM enumerateKeysAndObjectsWithOptions:usingBlock:]_block_invoke + 106
	9   CoreFoundation                      0x0353cf1c -[__NSDictionaryM enumerateKeysAndObjectsWithOptions:usingBlock:] + 204
	10  CoreFoundation                      0x03534575 -[NSDictionary enumerateKeysAndObjectsUsingBlock:] + 53
	11  MY_APP                              0x001115b7 -[EKManagedObjectMapper fillObject:fromExternalRepresentation:withMapping:] + 919
	12  MY_APP                              0x00111135 -[EKManagedObjectMapper objectFromExternalRepresentation:withMapping:] + 549
	13  MY_APP                              0x001124cf -[EKManagedObjectMapper arrayOfObjectsFromExternalRepresentation:withMapping:] + 463
	14  MY_APP                              0x00111f61 __75-[EKManagedObjectMapper fillObject:fromExternalRepresentation:withMapping:]_block_invoke50 + 369
	15  CoreFoundation                      0x0353d01a __65-[__NSDictionaryM enumerateKeysAndObjectsWithOptions:usingBlock:]_block_invoke + 106
	16  CoreFoundation                      0x0353cf1c -[__NSDictionaryM enumerateKeysAndObjectsWithOptions:usingBlock:] + 204
	17  CoreFoundation                      0x03534575 -[NSDictionary enumerateKeysAndObjectsUsingBlock:] + 53
	18  MY_APP                              0x001115b7 -[EKManagedObjectMapper fillObject:fromExternalRepresentation:withMapping:] + 919
	19  MY_APP                              0x00111135 -[EKManagedObjectMapper objectFromExternalRepresentation:withMapping:] + 549
	20  MY_APP                              0x001124cf -[EKManagedObjectMapper arrayOfObjectsFromExternalRepresentation:withMapping:] + 463
	21  MY_APP                              0x00113ff3 +[EKManagedObjectMapper arrayOfObjectsFromExternalRepresentation:withMapping:inManagedObjectContext:] + 915
	22  MY_APP                              0x0009e2e7 __45-[APIHelper addObjectsToDataBaseFrom:table:]_block_invoke + 487
	23  CoreData                            0x0081357b developerSubmittedBlockToNSManagedObjectContextPerform + 203
	24  libdispatch.dylib                   0x03ae7bef _dispatch_client_callout + 14
	25  libdispatch.dylib                   0x03acbd61 _dispatch_queue_drain + 544
	26  libdispatch.dylib                   0x03acb9cb _dispatch_queue_invoke + 212
	27  libdispatch.dylib                   0x03acf012 _dispatch_root_queue_drain + 615
	28  libdispatch.dylib                   0x03ad0b70 _dispatch_worker_thread3 + 115
	29  libsystem_pthread.dylib             0x03e2c43e _pthread_wqthread + 1050
	30  libsystem_pthread.dylib             0x03e29f72 start_wqthread + 34
)
libc++abi.dylib: terminating with uncaught exception of type NSException
(lldb)

Fabric.io aka Crashlytics said that it crashes in EKCoreDataImporter.m line 261.
I'm sure that EasyMapping works fine and problem somewhere in JSON which i get, i just hope that you as creator can know the reason of error. Thank you!
UPD: Also i've discovered that i get this error only when i try to map to many relationship like [mapping hasMany:[Address class] forKeyPath:@"addresses"];

Damn, i supposed from the start that the problem is that primariKey = nil, and i was right. By great "метод тыка", problem was founded and resolved! Thank you!


GlowBrush="{DynamicResource AccentColorBrush}" is causing a ComponentModel Exception in UpdateCore

testing the current alpha versions of mahapps.metro I currently resolve an error using a GlowBrush.
The error thrown is a "invalid window handle" in
MahApps.Metro\MahApps.Metro.Shared\Controls\GlowWindow.xaml.cs line 274.
At the moment I'm not able to reproduce that error in the sample apps contained in the mahapps.metro repository but with my own applications for example "TestUI" from https://github.com/evilbaschdi/EvilBaschdi.Core/tree/develop but I have no Idea were to search for the problem. with 1.5.0 everything's working fine.

currently I'm using 012, it also happened with 009 until 011.
I think I just found the problem my selfe. There was a AllowsTransparency="True" in my code and I don't know why it is there. But with 1.5.0 that problem does not occur.

@evilbaschdi MahApps uses now ControlzEx dll. The Problem is by NuGet which doesn't get always the highest dependency from ControlzEx for MahApps. This can be set in the NuGet package manager in VS (or you use the paket manager).

I know about that. And I've been on 3.0.66 (or what is the current) ControlzEz. I've even referenced the your repository directly for better debugging.

@evilbaschdi This should be fixed now and is available with 1.6.0-alpha013

yes, it's working now. thx!


Chart Layout Issues in V2

Could anyone explain how to layout the chart? for example, I tried to put the chart into a table (so that I can put some label along y and y2), however, the popup window always on the very top, in fact, it's already out of the chart!
second, sometimes the chart is trying to change its size dynamically. how to disable it? and how to set up the size in percentage?
thanks!

Hi @samwzm
could you provide sample Codepen please, so I can take a look at it and try to solve your problem! You could start with editing this pen http://codepen.io/chaosmail/pen/zrPJYb
Best,
Christoph

Any updates on this ?


Fixed form in dark theme

Hello PA @akshitagupta15june
Issue resolved - #1841

@akshitagupta15june kindly review this

@akshitagupta15june ??


Fix the step number

Issue #, if available:
Description of changes:
Fix step number.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

👍 thanks!


Modify the AST of the vue code in batches, and then based on the modified AST, you can directly generate the Vue code

What problem does this feature solve?
You can automatically add some tags to the vue template. Currently, these tags are manually added, but the accuracy and uniqueness cannot be guaranteed manually. In addition, manual addition is trivial, so we hope that we can modify or add corresponding to the Vue AST specification AST, then you can generate new Vue code
What does the proposed API look like?
 const data = ast.createIdentifier("data-sz");
const str = ast.createStringLiteral("123");
 return ast.createAttribute(data, str);

If you are requesting new features, you should go through an RFC, https://github.com/vuejs/rfcs.

If you are requesting new features, you should go through an RFC, https://github.com/vuejs/rfcs.


Hangs on async benchmarks

kanal hangs for me 99% of the time in all async benchmarks, both with the official benchmarks and with Tachyobench.
I was initially suspecting a problem with async notifications, but since it uses 100% CPU on all threads when hanging, I start to think it might actually be due to the use of spin locks, though I don't know for sure.

thank you for your report, whats your hardware specs?

which rust version? it it hang on any specific test?

ok, thanks, currently i can't reproduce it on my PC and laptop.

ok, thanks, currently i can't reproduce it on my PC and laptop.

Well, this is weird... Do you have intel hardware?
I tried it on an EC2 instance and it had the exact same problem, so the issue is probably widespread.
This was an EC2 t2.micro instance (exact hardware was Intel Xeon E5-2676 v3) so you can reproduce it using a free AWS starter account.
Investigating further I noticed that tests that reserve enough capacity for all messages, such as:
run_async!("bounded_mpsc(empty)", mpsc::<BenchEmpty>(Some(MESSAGES)));`

actually run.
In general, increasing the capacity seems to decrease the probability of hanging. Also, the SPSC test sometimes work even with small capacity.
I managed to sometimes get the test running with capacities of 10 or 100, but then the perf of kanal-async is not that good compared to other flume-async or async-channel, which also suggests some issues with the notification primitives.

Yes, there is UB somewhere in code and I'm investigating it, I think there is a problem with our stack borrow rules, I should fix that first, I'm gonna keep you updated to run your tests again then.
Are using nightly or stable Rust?

U

Yes, there is UB somewhere in code and I'm investigating it, I think there is a problem with our stack borrow rules, I should fix that first, I'm gonna keep you updated to run your tests again then. Are using nightly or stable Rust?

Using stable only.

Could you please rerun your tests, with the latest benchmark repo and the latest Kanal repo?

I could only test on my laptop for now, but it seems to work fine with commit #774a05f :-)
Likewise on tachyobench, so I plan to merge support for kanal once you make a release.

Thank you!

Sorry I write here, I could not find a way to PM you so I hope you won't mind I ask here.
I would like to update my own benchmarks for an MPSC channel I wrote, but there is no official release of Kanal that I can benchmark since the pre2 release has this issue with async. I don't want to misrepresent the potential of Kanal, so do you feel that the master branch of Kanal is suitable or would you prefer that I don't benchmark Kanal just yet?

Thank you for your professionalism, Kanal async is not ready yet, I need to redesign some parts of it, if you like to benchmark the sync version, the master branch is ok. but if you like to run benchmarks for async, I think it will be ready in pre3.

No problem, I understand!


append_url_extension unignored

I found that this header variable:
append_url_extension: json
is not correclty used within Page.php rendering section.
Here is the fix!

I need to confirm that this won't have the unintended result of appending the default extension when it's not wanted.

I would like to bring focus on this again: after the last update the problem is still here.
I use this patch in a production website and there is no problem at all.


(autohotkey) Create virtual, v1 and v2 packages

AHK v2 is dramatically different from AHK v1. Given the importance of this package for unattended installations, I propose the following:

Current ID autohotkey should become virtual package
Rename current autohotkey to autohotkey1
Create new package, autohotkey2
Via streams, make autohotkey to contain all releases from both versions.

Example for this methodology is done once by me in package PostgreSql and seems to work fine as one can choose major version but also choose not to care and entire thing is easily discoverable.
Alternatively, we could leave autohotkey as it is, and create set of new packages with different names: ahk, ahk1, ahk2 and deprecate autohotkey to target ahk1. Seems more clean and I like shorter name.
Also, this would have to handle portable variants. So its some work, but it could be done as single package with 5 streams.
If done, this would make #1663 and its PR obsolete.
Thoughts ?

This issue is valid/relevant.
Currently:

many packages depend on autohotkey.install and use version 1 syntax for their AHK scripts.
autohotkey.portable is currently at version 2
autohotkey.portable places the version 2 'AutoHotkey.exe' into $env:ChocolateyInstall\bin
And packages that use autohotkey.install and expect version 1 will call 'AutoHotkey', which then runs the version 2 executable from above and the user receives popups of syntax errors.

I've reopened this issue as I noticed 1.1 is deprecated so we can'yt just leave everything at v1.
If a package has a dependency on autohotkey or autohotkey.install (latest is <IP_ADDRESS>), and they install another package that  requires autohotkey.portable (latest is 2.0.2) then the Auothotkey software will be upgraded to 2.0.2 overwriting the <IP_ADDRESS> version. You can't have both versions of Autohotkey installed.
We have:

autohotkey.portable at version 2.0.3.
autohotkey.install at version <IP_ADDRESS>.
autohotkey at version <IP_ADDRESS> (it takes autohotkey.install as a dependency.

This is a bit of a mess.
It appears that you can install both v1 and v2 of Auothotkey and v2 will therefore be able to execute v1 scripts. I don't know how this works with portable versions as I haven't looked at that yet. However, I have submitted #2246 for the autohotkey.install v2 version that includes backwards compatibility with v1 scripts.


Possible bug for [TermAttribute] with multiple params values

Hello
So I'm trying to model possible <option> values for a <select> element. The HTML is basically:
<select data-unique-identifier="3fae10ba-dbe5-432f-a4b5-e031f699fa3c" class="status">
  <option value="TODO" selected="">To do</option>
  <!-- Can also be: <option value="To do">To do</option> -->
  <option value="COMPLETED">Completed</option>
  <!-- Can also be: <option value="Completed">Completed</option> -->
  <option value="WAIVED">Waived</option>
  <!-- Can also be: <option value="Waived">Waived</option> -->
</select>

The enum I've written is:
public enum BringUpStatusType
{
  [Term(TermMatch.Equals, "TODO", "To do")]
  ToDo,
  [Term(TermMatch.Equals, "COMPLETED", nameof(Completed))]
  Completed,
  [Term(TermMatch.Equals, "WAIVED", nameof(Waived))]
  Waived
}

// is referenced and used as:
[FindByColumnIndex(6), FindByCss("select[data-unique-identifier].status")]
[SelectsOptionByAttribute("value")]
public Select<BringUpStatusType, TOwner> StatusForBringUp { get; set; }

However, when I try to change the value of a given <select>, using the Set() method I get the following error:
OpenQA.Selenium.NoSuchElementException: 'Unable to locate element:
- By: XPath ".//option[normalize-space(@value) = 'TODO/To do']"
- Search time: 7.022s
- Search options: {Visibility=Any, Timeout=7s, RetryInterval=0.25s, IsSafely=False}

Context element:
- Tag: select
- Location: {X=877, Y=696}
- Size: {Width=105, Height=22}
- Element ID: 29C0372F6690CAEC11D11B975AD5A6B2_element_96
- Text:
To do,
Completed,
Waived;

For documentation on this error, please visit: https://www.selenium.dev/documentation/webdriver/troubleshooting/errors#no-such-element-exception'

So I was able to fix it by updating the enum:
public enum BringUpStatusType
{
  [Term(TermMatch.Equals, "TODO")]
  [Term(TermMatch.Equals, "To do")]
  ToDo,
  [Term(TermMatch.Equals, "COMPLETED")]
  [Term(TermMatch.Equals, nameof(Completed))]
  Completed,
  [Term(TermMatch.Equals, "WAIVED")]
  [Term(TermMatch.Equals, nameof(Waived))]
  Waived
}

And it works with the above change, but I wanted to log this issue anyway as the error message reported an XPath value of .//option[normalize-space(@value) = 'TODO/To do'], which seems kind of wrong?
[Term] accepts multiple string values which I just assumed meant it would search using something like LINQ's .Any(), but it's concatenating the values with a /.

Hi @mamift,
Yes, you are right, that's a bug. I will provide a fix for multiple enum terms, like [Term(TermMatch.Equals, "TODO", "To do")].
However the following usage should not work the way you expect:
[Term(TermMatch.Equals, "TODO")]
[Term(TermMatch.Equals, "To do")]
ToDo,

Atata always take only one first TermAttribute, so the second one will not be considered at all.
Also, if option values can differ, you can maybe rely on [SelectsOptionByText] if text values for all select elements are equivalent. In this case you will not need multiple terms.

Oh you're right, that didn't fix it - it just suppressed the error. The Title cased terms "To do" were for an older version of the webpage, and yes, the second [Term] didn't match. Anyway, I got around it by creating a new enum, and new model types.

@mamift, the fix has been released in Atata v2.14.0.


New file for creating synthetic data to add more training data to the tesseract.

Description
This file will be used to create synthetic data for tesseract to have more data to train on.
Trello Card
Loom Video
Type of change

[ ] 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)
[ ] This change requires a documentation update

Change Status

[ ] Complete, tested, ready to review and merge
[x] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Has This Been Tested

[x] Yes
[ ] No
[ ] Not necessary

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

 As a developer, I want to increase the accuracy of the Tesseract model by finding the best preprocessing steps.

Not sure why it didn't commit with the code but I tried again and it worked. Thanks for pointing it out!

Hey Josh - Looked at the video and the generator looks awesome, great work! Unfortunately, the data generator in this commit doesn't have any lines of code. Not sure if you uploaded from a different branch. Can you try uploading it again so I can review the actual code? I tried fetching the branch but no luck from that front either.
Not sure why it was blank, but I recommitted it and it seemed to work. Thanks for letting me know!

There is the whole folder of the characters and I'm not sure if I should commit all of those or not. I felt like it might be too much clutter but if you want to test I can send you the folder.

 Created function for adding image files to folder with Niger's help.

Need to figure out if the lfs urls worked on the images I added or not. Else figure out why it's not.

Ran locally with files in story ground truth. Going to do a few more tests before comparing results.


Can we use jsonb-rs in stable rust?

Hi there, enabling the use of jsonb-rs in stable Rust would certainly make it more accessible and easier to adopt for users. Have you considered providing a stable-compatible version? This could greatly benefit the wider Rust community.

Great advice, thank you!