Propagation Test in 13.9 MHz is missing MARS

I'm uncertain where the "propagation test" moniker came from that you list.  However, i do know that the primary MARS frequency is 13.927MHz. There are a few other MARS bands also.  Depending on the source of the current moniker, you might find it useful to remove it and simply list the MARS bands.
Ref: http://www.marsphonepatch.net/operation-faq

@isaacgerg Thanks for brining that to my attention! The 'Propagation Test' moniker came from US340 footnote in the FCC Table of Frequency Allocations ( https://transition.fcc.gov/oet/spectrum/table/fcctable.pdf ). I am planning an update to the bandplans that will remove these placeholder allocations, since they are no longer befitting to the purpose of a detailed bandplan.
I will probably publish a small update soon, addressing all listed issues in the repository before the larger update.

@isaacgerg 7.6335MHz, 11.407MHz, and 13.927MHz have been added. If you have any other comments/ fixes about this issue let me know, as I will close this issue as of 1 week after last activity.


Implement Counterfactual Safes creation rate limit

Summary
This PR introduces a rate limitation to creating Counterfactual Safes using the related CounterfactualSafesController endpoint. This rate limit is applied by taking the Account associated with the Counterfactual Safe to calculate the limit threshold.
Changes

Adds a rate limitation controlled by a cache key before proceeding with Counterfactual Safes creation.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

2 of 9 (22.22%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 48.014%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/datasources/accounts/counterfactual-safes/counterfactual-safes.datasource.ts
2
9
22.22%

Files with Coverage Reduction
New Missed Lines
%

src/datasources/accounts/counterfactual-safes/counterfactual-safes.datasource.ts
1
20.37%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.01%

Covered Lines:
4427

Relevant Lines:
7446

💛  - Coveralls

As with the other implementation for accounts, this seems more guard-like behaviour? It might even allow us to reduce repetition if it was in a guard. What do you think?

I've answered in https://github.com/safe-global/safe-client-gateway/pull/1800#issuecomment-2271229334, we can follow the discussion there if you are happy with it.


Start Sequentially Minting at 1 instead of 0

As described in #45  NFTs with token ID 0 can cause some headaches for developers down the line.
I renamed the currentIndex to _nextTokenId as this describes its use better as well it's an internal variable. But let me know your thoughts about it.
During development I stumbled over this function declaration
function getOwnershipAt(uint256 index) in ERC721AExplicitOwnershipMock.sol
For me it was a bit misleading. I expected _ownership to be an multidimensional array containing the history of the ownership of an NFT. However, it's a flat mapping between basically the tokenId to the TokenOwnership. I would suggested renaming the index parameter to tokenId.
https://github.com/chiru-labs/ERC721A/blob/main/contracts/mocks/ERC721AExplicitOwnershipMock.sol#L19-L21

You should probably add .vscode to the git ignore

@MrMcGoats Done. Thanks for pointing that out.
I've added new tests as well. One for tokenOfOwnerByIndex is currently failing.
Any suggestions how to adapt it?
https://github.com/chiru-labs/ERC721A/blob/main/contracts/ERC721A.sol#L85-L108

Fixed the tokenOfOwnerByIndex function now as well. I wrote some test for the. The PR is ready for review, happy to get some feedback and merge it if approved. 👍

Hi, thought of one more edge case with _exists(). I think return tokenId < currentIndex; fails when checking tokenId 0 or the last tokenId? Maybe something like this to fix return tokenId > 0 && tokenId <= currentIndex;
I'm surprised there aren't existing tests already that fail for the boundaries. Haven't had a chance to run the test suite though.

This should have the flexibility to allow users to choose between zero and one indexed tokenIds, with minimal additional gas cost and minimal API change.

@Vectorized Thanks for the review. The change requests check out, will incorporate them probably today. Not sure what you mean by Can be unchecked.? I guess it relates to the option to make the starting index flexible?
I wonder if you think it should start at 0 or 1 or if even other starting indexes should be valid as well.
@jpegdigital good call, I've added some _exists tests and adapted the function my changes yesterday already. ✌
https://github.com/chiru-labs/ERC721A/pull/66/files#diff-833552e5bd4a4be3ada89b7874eedc6762fb91ed0890dda22e82a1936e6d60d8R280-R282

@Vectorized Thanks for the review. The change requests check out, will incorporate them probably today. Not sure what you mean by Can be unchecked.? I guess it relates to the option to make the starting index flexible?

Wrap those lines in unchecked {} blocks for gas savings, since they cannot overflow and underflow.
See: https://gist.github.com/Vectorized/e6f47b0c1de008bb5983852224e41f69

@Vectorized thanks for the gist, it helped a lot. I've adapted my fork to your suggestions. Works fine now. It needed minor adjustments.
I've brought back the original test files and plainly copied them for new mock contracts which uses the override to set the starting token id to one. Not sure how I feel about the copy-pasted code, not sure how to do it more elegantly atm.

@Pczek
Thanks for your hard work. I was actually preparing to work on this, until I saw your PR.
Don't worry about copying. Just want to contribute.
We rise together.
We build together.
We grow together.

Likewise, thanks @Pczek.

Thanks @Pczek! I'm excited to see this one on the next release :)

@Vectorized @chiru-labs Who can do the final review and approve it to be merged?
IMHO the PR is ready to be merged.

Once we push the burn function in, I'll review this one :)
Thanks for all the hard work and bearing with us

@Pczek I've opened a PR to your fork with the added burn functionality. https://github.com/Pczek/ERC721A/pull/1

@Vectorized @cygaar merged https://github.com/Pczek/ERC721A/pull/1 ✅
All tests succeed ✅

Awesome thanks, this is the next PR on the docket

@cygaar that would be great! I've just resolved the latest conflict and adapted the code of mine to work with the recent changes of the main branch.
Regarding a gas report. Here's a screenshot of the current main and my fork side by side. The only difference I'm able to spot is the cost of deployment. However, these are expected as more code is deployed. 🤷

Interesting how the mint actually used lower gas on average when the starting index is 1 instead of 0.
When the index starts at 0, the very first mint will change _currentIndex from 0 to 1, which costs more gas (20000 vs 5000).

@Pczek I've opened a PR https://github.com/Pczek/ERC721A/pull/3 to your branch for adding _totalMinted function (#124).

Thanks for your PR @ahbanavi let's wait for that to get merged and then I'll do an in-depth review of this PR

@Vectorized @ahbanavi just merged your PR https://github.com/Pczek/ERC721A/pull/3
Test seem to run fine, I haven't adapted anything else since the PR of @ahbanavi .
Anything left to do? I saw you opened up a new PR @Vectorized ?

@Pczek Welcome back!
I'll close that PR first. Let's wait for @cygaar.

Going to review this today

@Vectorized @cygaar please check my recent changes to make a parameterized test. How do you like it this way?
If this is fine for you we can adapt the rest of the duplicated tests for the ExplicitOwnership extension as well.
2ba810d (#66)

LGTM. @Pczek parameterize the ownership explicit test and this is good to go. Appreciate all the work you and everyone has put in to get this out

Alright, I've parameterized now the following tests:
ERC721A.test
ERC721AOwnersExplicit.test
ERC721ABurnable.test
All of them now also run their test suite for a contract which is modified with the StartTokenIdHelper.sol contract to set the override the _startTokenId() function dynamically.
Afaik no further tasks are open on this PR until another review or it's ready to be merged.
CC @Vectorized @cygaar


Build upgrade and Release 0.20.0

This PR upgrades the build to the latest version of cats, cats-effect, circe, etc. Additionally, it ignores a random test failure.
Releases 0.20.0.

@BenFradet @fedefernandez I'm requesting again your review since I've added more changes in order to complete the Scalajs upgrade.

Can you upload the artifacts to maven, please?

They should be there, that PR deployed artifacts automatically ;)

Now I see them. Previously they were missing. Thanks


In Chrome the button to turn Do Not Sell button does not appear to consistently turn off navigator.globalPrivacyControl

Tested on Chrome and Chrome Canary and deactivating GPC does not appear to work on the latest version of the extension.

@OliverWang13 will inquire further.

Closing this issue as we are not able to replicate it. Feel free to reopen, @AramZS.


Explicitly set the health probe path

Based on the known issues https://github.com/Azure/AKS/issues/2955 & https://github.com/Azure/AKS/issues/2903 it is required to explicitly mention the health probe path to /healthz as the default is / which causes the health probe to fail and causes to backend to be marked unhealthy

@asudbring
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


npm ERR! help process exited with code: ENOENT

nvm install --lts
npm help install then got this error:
npm ERR! help process exited with code: ENOENT

npm ERR! A complete log of this run can be found in: /home/jack/.npm/_logs/2023-11-28T07_39_46_625Z-debug-0.log

npm help install is coming from npm, not nvm, so you'd need to file an issue with npm.


JSON:API Includes compatible ?

Hello,
We start to develop a nex site with next-drupal and I want to know if we can use  JSON:API Includes to reduce the verbose API return ?
Thanks.

Yes. You can use JSON:API Includes for collections.

Thanks @shadcn for this quick reply.
Have a good day.


Use Aggregated Perf Results in Perf Compare

Problem Description
With https://github.com/AdoptOpenJDK/openjdk-test-tools/issues/73, we've added the ability to aggregate the perf results from multiple iterations. We should update Perf Compare to use and display aggregated perf results in order to comply with the new design.
Proposed Changes
Perf Compare should use the aggregated data for any build that's passed in the input for test or baseline build.
It should be able to support the following comparisons:

Master Build vs Master Build
Child Build vs Child Build
Master Build vs Child Build

Since we'll be using the aggregated data for each build, Perf Compare would get a significant boost in displaying the results.
Data to Display:
For parent build:
Display the aggregated data for the parent and the raw data for each of its child build.
For child build:
Display the aggregated data and the raw data for each test iteration.
Assigned Contributors
I'll be working with Sophia (@sophiaxu0424) from my team to add this feature.

Three comparison scenarios

Master vs master

Child vs child

Master vs child

Closing this since all related work to this issue has been completed.


Highlight.js on Blogger wraps code lines automatically

At the moment I'm facing an issue integrating highlight.js with Blogger. The syntax highlight works great, but the text lines inside the pre/code tags are auto wrapping. Is this the intended behavior? If yes, can we have an option to show a horizontal scroll bar instead? Looking at the demo page I can see that some examples have a scroll bar, so I'm guessing this is a problem with Blogger instead.
I have also opened a StackOverflow question about this, so far unanswered.
Following is a screenshot of the problem I'm facing. As you can see, the code gets quite unreadable with the line wrapping:

It's not something highlight.js does itself, it depends on how you style your code blocks with CSS on Blogger. Most probably there's somewhere a rule setting white-space: pre-wrap (or similar) in your stylesheet.

Following your suggestion, I did some more fiddling with the CSS and was able to fix the problem by overriding the .hljs style, just after including the highlight.js stylesheet. Probably not the only or best fix, but this was the only one I've been able to get right. My knowledge of the Blogger API or CSS in general is limited, so the reasons it worked are also not quite clear to me.
For future reference, in case you are interested on further investigating the matter to perhaps incorporate that to the library, this is how my highlight.js include is (at the end of <head>):
<link href='//cdnjs.cloudflare.com/ajax/libs/highlight.js/8.6/styles/solarized_dark.min.css' rel='stylesheet'/>
<style type='text/css'>
	pre, code {
		white-space: pre;
		overflow-x: scroll;
	}
	.hljs {
		display: inline-block;
		overflow-x: scroll;
		padding: 0.5em;
		padding-right: 100%;
		background: #002b36;
		color: #839496;
		-webkit-text-size-adjust: none;
	}
</style>
<script src='//cdnjs.cloudflare.com/ajax/libs/highlight.js/8.6/highlight.min.js'></script>
<script type='text/javascript'>
    hljs.initHighlightingOnLoad();
</script>

Glad you had it sorted! Basically, this is how CSS is supposed to work: if a general style sheet doesn't work for you, you override it with a local rule.

Ah-ha-ha! Nice! Thanks guys! I wanted my code blocks to wrap on Google Blogger, instead of having the scroll-bar, so looking at these posts here I was able to determine I needed to add the following to force it to wrap my code with highlight.js syntax highlighting!
Add this between the <head> </head> tags at the top of the blogger theme to affect the whole website, or just to the top of a given blog post to affect that post only:
<style>
pre, code {
    white-space: pre-wrap;
}
</style>

pre-wrap forces the lines to wrap. See here: https://www.w3schools.com/cssref/pr_text_white-space.asp.

pre-wrap  Whitespace is preserved by the browser. Text will wrap when necessary, and on line breaks

Add this between the <head> </head> tags at the top of the blogger theme to affect the whole website, or just to the top of a given blog post to affect that page only

Update: actually, I can't seem to get it to affect the whole website. Adding it between the  <head> </head> tags at the top of the blogger theme doesn't seem to work.


Connections are always nullable and ignore GraphQLNonNullType

Product
Hot Chocolate
Version
14.0.0-p.93
Link to minimal reproduction
See zip below
Steps to reproduce
Repro solution: HotChocolateBugRepro.zip
Code for quick reference:
var builder = WebApplication.CreateBuilder(args);

builder
    .Services
    .AddGraphQLServer()
    .AddQueryType<Query>();

var app = builder.Build();

app.MapGraphQL();
app.Run();

public class Query
{
    [GraphQLNonNullType]
    [UsePaging]
    public List<int> Test() => new List<int>();
}

What is expected?
type Query {
  test(...): TestConnection!
}

What is actually happening?
type Query {
  test(...): TestConnection
}

Relevant log output
No response
Additional context
The order of UsePaging and GraphQLNonNullType does not matter.
Additionally, I would expect that when

This is not a bug but by design ... we will have a look another look at connections for the December release.


Touch ripple on Button prevents onClick handler

if there is onClick handler, it will not be triggered anymore, since handleTouchStart blocking the event.
Now it is required to add both onClick and onTouchStart to make it work on mobile.

Wow, so sorry. Actually we are getting rid of the handleTouchStart for the next version. It's not giving too much value and it's creating problems. For example, you can't scroll on mobile since it stops the event propagation. For users focusing on mobile who need an immediate response from the ripple is no big deal to create a component wrapper to do so.
Fixing soon, thanks for reporting!

I just updated and noticed everything stopped working on mobile. Could you explain what you mean by "For users focusing on mobile who need an immediate response from the ripple is no big deal to create a component wrapper to do so."? Or not, if it's easier for me to wait for a fix. :)

Yeah. The Ripple component was stopping the events as they were used to track the position of the event. A first approach was to make the button able to respond with a ripple onTouchStart but since it was stopping the event, when you try to scroll on mobile with an initial tap in the button, this first tap is prevented and you can't scroll. That's why we removed it from now.
The solution from React Toolbox POV is to avoid stopping the event which we need to look into deeper. If you want to use the ripple onTouchStart you can still create a wrapper component for the Button and add onTouchStart callback starting the ripple yourself.

Is it fixed in latest build, or just closed and anyone should create wrappers to implement that workaround? I suppose latter as there is commit attached.
Why not just remove stop propagation, and handle double ripples(I see this is the only problem in this case) gracefully, disable mouse event for example in case of touch fired or any other way.

It may have been fixed by d0e4135d0b34e0e491cdbcb6fe01f7628e101dac, which went into the 0.13.0 release at the same time this issue was closed.

Yup, my site is fixed at 0.13.1.

Great!


Remove version: Comix.FastMeetingLauncher version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://fs.hst.com/download/FMLauncher_V<IP_ADDRESS>.exe
Response Error: The operation has timed out
(Automated message - build 923)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 903.)

Close with reason: Duplicate;


why my loss always keep in 300 and acc under in 50%?

苏AZ3A25 1.0 <-> 苏AZ3A25 1.0
苏AJ3H16 1.0 <-> 苏AJ3H16 1.0
苏AC0378 1.0 <-> 苏AS0377 1.0
苏AY9L95 1.0 <-> 苏AX9L15 1.0
苏AZ1W57 1.0 <-> 苏AZ7X17 1.0
苏AR3595 1.0 <-> 苏AR0195 1.0
苏AI0P50 1.0 <-> 苏AI0P59 1.0
苏AE8J34 1.0 <-> 苏AE8J34 1.0
苏AO9R64 1.0 <-> 苏AO9R64 1.0
苏AC5N78 1.0 <-> 苏AC5N78 1.0
苏AU4A12 1.0 <-> 苏AU4A12 1.0
苏AN4780 1.0 <-> 苏AN4780 1.0
B77120 42.00% 100.00% loss:<PHONE_NUMBER>328125 (digits:<PHONE_NUMBER>328125, presence: 1.8921246439497583e-15) |XX X  X XXXX XX XXXX    XX XXX XX  XXX  XXXXX     |
time for 60 batches 9.751856088638306
I had train it 3 hours

Can you share your code?Thank you very much!

Hi Sir,
I have the same problem! did you solve it?

我在第三步，识别的时候遇到了这样的问题
deep-anpr/detect.py:102: RuntimeWarning: invalid value encountered in greater
for window_coords in numpy.argwhere(y_val[0, :, :, 0] > -math.log(1./0.99 - 1)):
你遇到了没有曾大佬

did you solve the problem? I have a similar problem. I follow the tutorial completely, training 7 hours, the result is that.


Add kube-aws:version to stacktags

We don't currently tag our stack or instances with the version of kube-aws that created them.  This change adds a 'kube-aws:version' tag into each stack, including the root stack, and fixes a bug where tags are not updated when stacks are updated.  These stacktags are also inherited by instances, so all ec2 instances also recieve the tag.  This makes it very easy to use AWS API/CLI/GUI to check on the version of kube-aws used to deploy a cluster.
This idea was mentioned in https://github.com/kubernetes-incubator/kube-aws/issues/1405 where it was suggested as a good idea - giving us less reason for further requests to embed it the cluster.yaml.

Awesome! Does this also work with spot instances? They do not inherit the stack tags unfortunately and kube-aws supports some other attributes for that, although I kind of which we connected them as to the stack tags since AWS doesn't.

Codecov Report

Merging #1409 into master will increase coverage by 0.03%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #1409      +/-   ##
==========================================
+ Coverage   37.53%   37.57%   +0.03%
==========================================
  Files          74       74
  Lines        4534     4546      +12
==========================================
+ Hits         1702     1708       +6
- Misses       2597     2603       +6
  Partials      235      235

Impacted Files
Coverage Δ

cfnstack/provisioner.go
0% <0%> (ø)
:arrow_up:

core/nodepool/cluster/cluster.go
46.51% <100%> (+1.27%)
:arrow_up:

core/controlplane/cluster/cluster.go
50% <100%> (+0.69%)
:arrow_up:

Continue to review full report at Codecov.

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

Hi, I haven't tested spot instances but it looks as though they are responsible for setting their own tags and the stack tags are propigated to them through their cloud-config files, so they should also set this new tag!
I've added the new 'kube-aws:version' tag to StackTags so it behaves like any other StackTag - and so I'm quite confident that it will be propigated down.  However, I am a little less confident that this is the behavoir that you desire (hopefully it is)!

We have a set of script/unit to tag spot instances by their own! https://github.com/kubernetes-incubator/kube-aws/blob/77c43b5e16912d89a9238c4a38b573303bcdb71e/core/nodepool/config/templates/cloud-config-worker#L1090-L1136
It would be nice if we could add the support for it later.

Oh, it seems like we can remove the script/unit in favor of the official cfn support for spotfleet tag specification as per #901!

Could changing the implementation of setting spot tags be another issue/pr?  I can't properly test it here.

My vote goes for separate, it's a bit delicate as kube-aws also supports separate tags per node pool as well.


JBPM-5296 - Cleaning up human-task and removing deprecated code

PR set:

https://github.com/droolsjbpm/droolsjbpm-knowledge/pull/162
https://github.com/droolsjbpm/jbpm/pull/562
https://github.com/droolsjbpm/droolsjbpm-integration/pull/609

Jenkins, please retest (SingletonRuntimeManagerTest.testInterceptorAfterRollback succeeds locally -- seems like a false negative?)

jenkins retest this

+1 to merge once build is green

Jenkins please retest this.


Recipe Difficulty isn't applied correctly/automaticaly

Recipe Difficulty for Forged Gladiator's Heraldry and Forged Aspirant's Heraldry isn't applied automaticaly.

#481 - duplicate


Provide AssetHost configuration

This can be used to support a CDN as per the tests

@toshok yeh I can do. Do you know why we forked this initially? We could look at moving to upstream completely

ah yeah, the fork exists so we can be running our fixed/improved code without having to wait for upstream.


Reconsider MySQL 5.7 support in Community until EOL, disable Teams edition preview in docker

Which version and edition of Flyway are you using?
7.15/8.0.0
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Yes, see below.
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
command line (docker image)
Which database are you using? (Type & version)
MySQL 5.7.35
Which operating system are you using?
kubernetes
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Ran a migration against MySQL 5.7
What did you expect to see?
A successful migration
What did you see instead?
Part 1:
The 8.0.0 community version removed compatibility with MySQL 5.7, despite it being supported by Oracle for an additional ~2 years. This is a pretty unfortunate decision, since you still have to go through the effort to support it in code for the Teams edition and it's not EOL yet. I was introduced to flyway in a hobbyist project that I'm working on via another contributor, and I have reasonably enjoyed using it but changes like this degrade my opinion of the product significantly. I have pinned to 7.15 for now, but ...
Part 2:
I backed out to 7.15, where I then saw:
Running kubectl logs -f -l app=migration
WARNING: This version of Flyway is out of date. Upgrade to Flyway 8.0.0: https://rd.gt/3rXiSlV

Flyway Teams Edition 7.15.0 by Redgate
Database: jdbc:mysql://mysql:3306/game (MySQL 5.7)
----------------------------------------
Flyway Teams features are enabled by default for the next 27 days. Learn more at https://rd.gt/3A4IWym
----------------------------------------

I was very surprised to see that I was temporarily granted Teams features. I am now setting the edition to Community vs. Teams via environment variable in my yaml definition for the migration job, but I'm glad I looked at the logs: I wouldn't want to accidentally create automation that relies on features that will be removed. This should not be the default to mysteriously break later if you're not paying attention. Please consider defaulting to Community edition and requiring manual intervention to upgrade to Team edition in docker containers which will be primarily used by automation.

Thanks for raising this issue
Regarding database version support - this is in-line with our own support policy. Regarding enabling Teams Edition by default - this is in order to reduce the barrier to entry, and each Teams feature is entirely opt-in and appropriately documented
We will take this into consideration moving forward but won't be making immediate changes unless there is further interest

In what manner is "each feature entirely opt-in"? If I ran against a not-Community database, if teams features were enabled by default, would this stop working in 27 days?


Alter title and heading on world_location_news pages

title change to 'UK and COUNTRY', and heading change to 'COUNTRY and the
UK'

Hi @1pretz1 👋 Could you post the link to the trello card that this PR relates to (in the PR description)?

Hi @davidbasalla, the link is - https://trello.com/c/aQPSDsVW/74-change-page-title-and-description-for-world-location-news-pages

@1pretz1 when you add the Trello card to the PR description (at the top), it will automatically pop a link to the PR into the Trello card, which is quite useful!


transaction tagging

Feature Description

The user can see the details of the transaction including its account, source and privacy tag
They can also see the user defined tags but can also:

associate new tags (from what already exist)
create new tags (and associate them to the txn)
delete a tag

A maximum of 5 unique tags are allowed. Tags cannot be repeated
’4. A user can also tag transactions when receiving. When receiving from an account, the account tag, source tag and privacy tag of the xPub is shown. If the privacy tag is Public, it can be changed to Private or Peer. More user tags can be added by the user. This is shown on the receive address screen. Txn type tag is associated accordingly.
A user can tag outgoing transactions when sending. This will also tag any incoming change transaction. When sending from a particular account, the least private tag of the UTXOs used is used as privacy tag. The user can change the privacy tag to less private tag (e.g. from peer to public). The source tag is the tag of the xPub used to generate change address. Txn type tag is associated accordingly.
Adding/ changing tags while send and receive is optional and not something which the user may want to do every time. The order of tags from least private to most private is: Public, Unknown, Peer and Private.
All output transactions show the change transactions within the details as coming in.
All transaction details screen have an option of ‘Send UTXO’ (Send coins from this Transaction)’, this takes the user to the send screen but will allow only one recipient and will send the whole UTXO to the recipient (minus the fee)
When the user is looking at the latest transactions, they can decide to ‘View All’. This will take the user to a separate screen where the user can:
a. Search transactions based on tags, period (date to and from), from, to and amount
b. They can select one or multiple transactions
c. Once selected they can: Send those UTXOs together

**Pre-requisites/ dependencies
-Designs
-Research
-Discussion with a Senior
**Task Break Down
-TASK 1
-TASK 2
**Test Cases
**Design
**Flow
**ERD

https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/11c13d2c-169f-4729-9228-c7af3f9791e7/

https://github.com/bithyve/bitcoin-keeper/issues/4682  - Tagging regarding issue has been Done Here

Replaced by ##5193 and #5195


http://react-shadow.herokuapp.com/ is offline

Your demo is offline, see attached SS:

Good spot. It should be good now. Thanks! 👍


Complete windows implementation

Finally got all commands and tests up and running for all PHP versions in Windows and Linux 🎉
I did however have to separate unit/integration tests. This was due to include'ing the global-ray-loader.php file, and then attempting to test the build command, which will replace the .phar while it's loaded, generating exceptions in certain instances.
I've also set the build-phars workflow to be dispatched manually, so they are not needlessly generated upon every push and pull request.
However, we will still need to implement the tryAsWindowsAdmin() if a user is attempting to install on Windows and is not running the CLI as an administrator:
https://github.com/composer/composer/blob/1e914f8afcb8ae2210843ac60a731456dae3755c/src/Composer/Command/SelfUpdateCommand.php#L593
I'll work on this later tonight or tomorrow 👍

Seems great! 👏
I'll have some time during the week to contribute a little bit myself! 👍

Thanks Freek! 😄


strace: disable mpers (multiple personality) support on aarch64 to fix

This fixes the build on aarch64, but may limit functionality in the long-term
that would be hidden by avoiding an underlying problem.
Or maybe this is best.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Hmm. I'll give this a try and try either this or revert, whichever is better.

Success on aarch64-linux (full log)
Partial log (click to expand)
/nix/store/z7k4qnrvab9l08aynamhwq27m36g23hz-strace-4.21

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘strace-4.21’ in /private/var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/development/tools/misc/strace/default.nix:20 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
/nix/store/lscmjgcp9g7s44wras5vjn3bp76vp768-strace-4.21

Hmm. I'll give this a try and try either this or revert, whichever is better.

Not sure I follow, but are you saying you'll take things from here? :)

Not sure I follow, but are you saying you'll take things from here? :)

Yep. :)
I googled around a bit and basically I think it's an upstream bug that the build fails fatally (by default)  when it finds out that -m32 doesn't work on AArch64, given that GCC doesn't support such a flag there (and I expect they never will). So I hope this can be removed one day in a further upstream release.
I did do s/=no/=check/
to make it a bit more futureproof (i.e. in case strace grows support for -milp32 on AArch64 one day, we don't want to unconditionally disable it): https://github.com/NixOS/nixpkgs/commit/8989a6b0f750931641e7fb0f0bc47cc7af1d3a4a

Perfect, I wasn't thrilled about the "=no" bit myself, check (assuming it does what it sounds, how is this not the default??) sounds great.
Thanks!


Fix synchronisation

This library is not thread-safe and thus does not need synchronisation. All synchronized blocks and synchronised lists etc. should be removed/replaced.
Since the WebSocket implementation does use threads internally, we still need synchronisation between it and the public API surface. I see a couple of options:

Make everything synchronized.
Introduce an API to hand in a lock from the application that is being used for pretty much everything.
Get rid of the WebSocket threads and allow this library to be run single-threaded (my preference, mid-term).
Remove the WebSocket transport. Create a transport interface and let the application create its own WebSocket transport (my preference, long-term).

I'd explore 3, potentially using a single thread executor service: https://github.com/TakahikoKawasaki/nv-websocket-client#connect-to-server-asynchronously

To be honest, I am not entirely sure how you'd structure it with a single thread executor service? Perhaps you can elaborate.
Some other idea (please feedback):

Encapsulate the whole WebSocket session and handling into separate object. So, when you would like to change the websocket client library in the future, it's less of a hassle. Currently, the SaltyRTC session and the websocket session is tightly coupled and kind of inflexible for changes.
Encapsulate the whole SaltyRTC session state in one object instance. At the moment, the state is shared between multiple threads and in combination with the callbacks I find it hard to follow the code. Could be my inexperience, though. :smile:
What I have overall in mind is more like actor-based concurrency and message passing instead of the manual locking.

Let's say we have two main actors (SaltyRTC and WebSocket). Both have their own mailbox for messages (just use a BlockingQueue). Now, we can ensure that the currently multi-threaded websocket library will nonetheless process the frames sequentially. In the callback methods we pass messages to the SaltyRTC and WebSocket actors. A little bit like Akka, just way more lightweight and way less powerful.
It would mean to create immutable messages for message passing. Each actor would run in its own thread and executing tasks that are in the mailbox sequentially.

I'm not entirely sure what you mean, @dbrgn. Can you explain how you would structure that?

For the time being #125 basically introduces @lgrahl's 1./2.


simulator tools -> better create connections

This creates the controller connections in a much nicer way, which also matches the documenation.
I think we can completely remove the old_method.
It also adds an option to use a subset of the controllers, but currently only remove those from the end (so if you have a 6 OPX setup, you can use 1-4, but not 3-6), but that can be edited.
WDYT?

I actually never use this tools so I am not very familiar with the use-cases. In any case the readme and docstring need to be updated and maybe also have some unit tests. Writing this I actually realized that there is no readme for this tool...

Thanks!
This is a very old branch, and I never got around to writing the unit tests :\ Which is probably the only reason it wasn't merged.
But Tomer recently needed it, so I came back to it.
I'll try to finalize this (and also the readme!)


Add method to clear a set log level

Java Logging supports clearing override log levels previously set by
calling setLevel(). Expose an Airlift method for doing this.

@martint and @findepi - last push uses syncronization. Note: there is now an implied contract that you can only clear a level that was previously set via setLevel().


Register using forRootAsync

I need to be able to inject the prisma service.
https://www.npmjs.com/package/nestjs-prisma-module
https://www.npmjs.com/package/nestjs-prisma
However I cannot do that unless I can register this module asynchronously. A forRootAsync method needs added.

Thanks @MichalLytek


Update build command and docker.socket

Added docker.socket to mounts
Removed the push from the Build step so push can push
Removed the if check since the whole job only runs on tag

Codecov Report

Merging #86 (f9335ea) into master (b81e17e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #86   +/-   ##
=======================================
  Coverage   12.17%   12.17%
=======================================
  Files          14       14
  Lines        1051     1051
=======================================
  Hits          128      128
  Misses        908      908
  Partials       15       15

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


"self" parameter in message callback is always false

Actual behaviour:
Using the code example in https://tmijs.com/ under "OAuth token authorization", the bot replies to itself when using the command !hello.
Expected behaviour:
The self parameter in the message callback should be set to true and the bot should not reply to itself.
Error log:
[17:14] info: Connecting to irc-ws.chat.twitch.tv on port 443..
logger.js:11
[17:14] info: Sending authentication to server..
logger.js:11
[17:14] info: Connected to server.
logger.js:11
[17:14] info: Executing command: JOIN #youblicitybot
logger.js:11
[17:14] info: Joined #youblicitybot
logger.js:11
[17:14] info: [#youblicitybot] <youblicitybot>: !hello
[17:14] info: [#youblicitybot] <youblicitybot>: @youblicitybot, heya!

Is this only supposed to work for messages sent through client.say? I'm using the bot account to run the command on the chat, through the browser.
Server configuration

tmi.js version: ^1.5.0

tmi.js will set the self argument to true when it's internally echoing the message it just sent. It's not for detecting if the user sending the message has the same username of the bot.


[Storage] az storage account hns-migration start/stop: Support migrate a storage account to enable hierarchical namespace

Description

pending on swagger merge and SDK release
depends on #19510
Testing Guide

# validate migration
az storage account hns-migration start --type validation -n teststorageaccount -g testresourcegroup
# start migration
az storage account hns-migration start --type upgrade -n teststorageaccount -g testresourcegroup
# stop migration
az storage account hns-migration stop -n teststorageaccount -g testresourcegroup

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage


Performance issue when hiding series by click on legend for charts with large number of datapoints

Steps to Reproduce

Open https://apexcharts.com/features/ and scroll to High Performance section

Choose 25000 points and click render

Click on legend series-1

Page stops responding

Looks like setAttribute is called for each dot and it cost a lot.
firefox

chrome

@junedchhipa I believe this is a duplicate of #4488, which was fixed by PR #4483. If so, it is fixed in release 3.49.2.

Yes, this is fixed and works as expected.


clearCookie Before goTo

How can i clear all cookie?
Is there any function like clearCookies?

Hi @Jehanramadhan ! Sorry for the late reply. I had 2 surgeries in the past month and couldn't make time to answer the issues.
I've just added that fn to this module and it's available in v4.3.4, you just can use that by calling

const HeadlessChrome = require('../index')

const browser = new HeadlessChrome({
  headless: true
})
async function navigateWebsite () {
  try {
    await browser.init()
    const mainTab = await browser.newTab({
      privateTab: false
    })
    await mainTab.goTo('https://github.com/LucianoGanga/simple-headless-chrome')
    await mainTab.clearBrowserCookies()
    await browser.close()
  } catch (err) {
    console.log('ERROR!', err)
  }
}
navigateWebsite()

Cheers!
And again, sorry for the late reply


Error in bot startup

Hello, I have a problem in bot and need your help. The problem is this, I installed the bot with the help of my friend, all right but when you open the cmd in the bot folder and type "python pokecli.py" he of the following error: "D: \ Documents \ Desktop \ PokemonGo-Bot-master> python pokecli.py
08/08/2016 12: 27: 10,480 [cli] [INFO] PokemonGO Bot v1.0
08/08/2016 12: 27: 10.483 [cli] [INFO] Error: No /configs/config.json or specified config
usage: pokecli.py [-h] [-cf CONFIG] -a AUTH_SERVICE [-u USERNAME]
[-ws WEBSOCKET.SERVER_URL]
[-wss WEBSOCKET.START_EMBEDDED_SERVER]
[-wsr WEBSOCKET.REMOTE_CONTROL] [-p PASSWORD] [-l LOCATION]
[-lc LOCATION_CACHE] [--forts.spin FORTS.SPIN] [-w WALK]
[-k GMAPKEY] [-e SHOW_EVENTS] [-d DEBUG] [-t TEST]
[-du DISTANCE_UNIT] [-EC EVOLVE_CAPTURED]
[-rt RECONNECTING_TIMEOUT] [-HR HEALTH_RECORD]
[-ac FORTS.AVOID_CIRCLES] [-mcs FORTS.MAX_CIRCLE_SIZE]
[--catch_randomize_reticle_factor CATCH_RANDOMIZE_RETICLE_FACTOR]
[--catch_randomize_spin_factor CATCH_RANDOMIZE_SPIN_FACTOR]
[--map_object_cache_time MAP_OBJECT_CACHE_TIME]
pokecli.py: error: argument -a / - auth_service is required "
What should I do? I do not know what to do, I'm trying to get hours, but I can not, I await answers, thanks in advance!
PS: My operating system is Windows 10.
PS²: I leave a screenshot attached.
PS³: If anyone knows speak Portuguese will help me a lot.
Hugs!

Rename config.json.example to config.json under /configs. You need to edit it first to put in your credentials, make sure to check out wiki for further information.

The file is already right, look at the picture :


Fix: OpenAPI warnings

Closes https://github.com/bullet-train-co/bullet_train/issues/734
Required by https://github.com/bullet-train-co/bullet_train-core/pull/270

Introduces .redocly.yaml config to turn off no-ambiguous-paths check, as Redocly team contributor confirmed that some paths in our API are considered non ambiguous (/projects/{project_id}/goals and /projects/tags/{id} for example).
Rewrote test/controllers/api/open_api_controller_test.rb so it fails on any warnings.

There are some test failures in the joint PR, which also show up here in the Core: tests. Going to convert this to a draft for now.


Add new APIs added in version 2.1.13

Add new APIs released in 2.1.13
Reference:

liff.getLineVersion()
liff.id

Thanks for contribute! Published in 2.1.3


What is the value range of BlendShapes for 3D models?

I checked `blender_livelinkface/pylivelinkface.py
        value: float
            Value to set the BlendShape to, should be in the range of 0 - 1 for
            the blendshapes and between -1 and 1 for the head rotation
            (yaw, pitch, roll).

I set the blendshape to 0-1 but there was no change. I want to know what the minimum and maximum values are?

The min/max values coming from the LiveLinkFace app (or any iPhone app using ARKit blendshapes) are as specified, the range (-1,1) for head rotation and (0,1) for everything else.
What do you mean you "set the blendshape to 0-1 but there was no change"?

Thank you for your reply. I’m sorry I didn’t describe the scene well before. In fact, I imported the model into unity and found that the 0-1 weight did not deform much. The reason is that the weight range in unity is 0-100.


Add proportional staging index dag

Fixes

Fixes #3488 by @stacimc
Description

This PR adds a new create_proportional_by_source_staging_index DAG as described in the IP here:

It can be used to create a new index in staging which is a subset of a staging index, but maintains the same proportion of records per source.
Note: reindexing is not parallelized with slicing, as it is for create_new_production_es_index and other DAGs (see comment thread for explanation). However it is parallelized in the sense that the individual reindexing tasks for each source can run in parallel. Both of these are acceptable because the DAG only touches staging indices, not production.
Testing Instructions

Tests assume you're starting from fresh local testing data, ie after running just api/init.

Run the create_proportional_by_source_staging_index DAG locally with default conf options. Then verify in elasticvue that you have a new es index called audio-50-percent-proportional-20240214t181238 (the final suffix is a timestamp and so will differ), with the alias audio-subset-by-source.

By default it will be based on the audio-filtered index and should have 50% of the total documents, with the same source proportions. Since the filtered index has 4912 documents, the new index should have 2456.
Inspect the logs for get_staging_source_counts to see the counts for the source index, and then the wait_for_reindex tasks to see how many were reindexed for each source in the destination index. The proportions should remain the same. For example wikimedia_audio has 3992 records in the source index, and 1996 in the new one.
Note that the remove_existing_alias step was skipped in this dagrun

Run the DAG again, but this time change the percentage_of_prod to 0.25 and the source_index to audio.

You should get a new index named something like audio-25-percent-proportional-20240214t182518
No steps should be skipped this time. The audio-subset-by-source alias should have been removed from the previous index and is now applied to this one.
The index should have 1250 documents (1/4 of 5000 from the main audio index)

Run the DAG with media type image and ensure this also works. You should get a new index with image-subset-by-source alias, a name like image-50-percent-proportional-20240214t181238, and 2328 documents (half of the 4656 that are in the filtered image index). Check the proportions are correct.
Run the staging_elasticsearch_cluster_healthcheck and create_new_staging_es_index DAGs to ensure that refactoring out shared elasticsearch utilities did not break them.

Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I think I am going to hold off on this and implement the fix for #3761 in this branch, because otherwise we have to add a new Airflow connection for the API which will just get removed later.

The fix for #3761 was added in this branch.
I've encountered another issue with auto-slicing to parallelize the reindexing step. This works fine in DAGs that don't use max_docs (like create_new_es_index), and for dagruns targeting the audio indices (which only have one primary shard). For dagruns targeting the image indices, the reindexing tasks would consistently fail with the error:
ApiError: ApiError(500, 'search_phase_execution_exception', '[slice] can only be used with [scroll] or [point-in-time] requests')

The docs state that max_docs When set to a value less then or equal to scroll_size then a scroll will not be used to retrieve the results for the operation. This disabling of scroll then causes slicing to break. Scroll size defaults to 1000. This can be addressed in a few ways:

Reduce the value of size (the scroll size). This may have a negative effect on reindexing efficiency.
Set slices to None. Reindexing will not be parallelized but the problem is avoided.
Set conflicts='proceed' (instead of the default 'abort').

After looking into the code, I realized scroll is only disabled if reindex is set to abort on version conflicts. I think we can safely set this to 'proceed', since we're reindexing into an empty index and should not expect version conflicts. This prevents the scroll issue.

I implemented the last of these options in this commit, but while this fixed the errors I encountered another issue. As noted in the docs:

using max_docs with slices might not result in exactly max_docs documents being reindexed.

And in fact you can see, if you check out this branch at the commit mentioned above, that if you run it against the image-filtered index with percentage_of_prod=0.25, we end up with a new index that has fewer records than expected because max_docs is not respected when using slices. In my test run, I got 852 records instead of the expected 1164. This is in my opinion not acceptable because it results in the proportions by source being off.
As a result, I do not think it is possible to use slicing to parallelize reindexing for this DAG, and still achieve exact record proportions. The most recent commit on this PR handles it by disabling slicing. This will necessarily be less efficient, although I'm not sure how much.

Noting this will have to be rebased when #3805 is merged.

#3805 was merged, so this has been rebased.


⚠ Bofrost.at has degraded performance

In ccf032d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1982 ms

Resolved: Bofrost.at performance has improved in 1158586 after 6 minutes.


Fix tests for PR #215

Need
PR #215 was merged and integration tests failed because the were not updated.
Definition of done
Make those integration tests green!

Need to apply migrations to db when merging prs containing migrations


DEVC: application component

Cloning https://github.com/abapGit-tests/DEVC_component into package eg. $DEVC will cause a diff
It looks like "application component" can only be set for transported packages

That seems like a non-trivial fix. I guess you would have to save the application component of the remote .xml in some extra storage and retrieve it on serialization if the package is local?

But of course it could have been changed on the remote in the meantime...

hmm, yeah, its difficult

I think we should do as @flaiker suggest above, if the class is local then persist fields COMP_POSID + COMPONENT using ZCL_ABAPGIT_PERSISTENCE_DB, logic to be implemented in class ZCL_ABAPGIT_OBJECT_DEVC

Let's not waste more than a line for this https://github.com/larshp/abapGit/pull/3635


Internal orgs appears to be unused in oc-chef-pedant

This translates to excluding tests that are tagged :internal_orgs, but there aren't any such tests any more.

:+1:


M1 Mac running an x86_64 Lima instance consistently freezes on Ventura 13.4.1 on a brew installed lima.

Description
On an M1 Pro with the following instance config (note the arch is x86_64) running on Ventura 13.4.1 on a brew installed lima.  My Ubuntu Jammy Lima VM starts, but freezes doing the following
sudo apt-get update
sudo apt-get upgrade

The instance freezes mid upgrade and later after limactl stop... restart, shell in a sudo dpkg --configure -a.
I've tried tweaking CPU and memory settings and the same thing happens.
The problem consistently happens.
Prior to upgrading to Ventura my x86_64 instances worked fine.  I'm not sure what other changes occurred prior to upgrading.  Maybe I should build and install lima from source?
Here is my config file with comments stripped for brevity (It is the template except for configuring DNS and configuring the use's home directory to be writable.):
arch: x86_64

images:
# Try to use release-yyyyMMdd image if available. Note that release-yyyyMMdd will be removed after several months.
- location: "https://cloud-images.ubuntu.com/releases/22.04/release-20220712/ubuntu-22.04-server-cloudimg-amd64.img"
  arch: "x86_64"
  digest: "sha256:86481acb9dbd62e3e93b49eb19a40c66c8aa07f07eff10af20ddf355a317e29f"
- location: "https://cloud-images.ubuntu.com/releases/22.04/release-20220712/ubuntu-22.04-server-cloudimg-arm64.img"
  arch: "aarch64"
  digest: "sha256:e1ce033239f0038dca5ef09e582762ba0d0dfdedc1d329bc51bb0e9f5057af9d"
# Fallback to the latest release image.
# Hint: run `limactl prune` to invalidate the cache
- location: "https://cloud-images.ubuntu.com/releases/22.04/release/ubuntu-22.04-server-cloudimg-amd64.img"
  arch: "x86_64"
- location: "https://cloud-images.ubuntu.com/releases/22.04/release/ubuntu-22.04-server-cloudimg-arm64.img"
  arch: "aarch64"

cpus: null

memory: null

disk: null

mounts:
- location: "~"
  mountPoint: null
  writable: true
  sshfs:
    cache: null
    followSymlinks: null
    sftpDriver: null
  9p:
    securityModel: null
    protocolVersion: null
    msize: null
    cache: null
- location: "/tmp/lima"
  writable: true

mountType: null

ssh:
  localPort: 0
  loadDotSSHPubKeys: null
  forwardAgent: null
  forwardX11: null
  forwardX11Trusted: null

caCerts:
  removeDefaults: null

containerd:
  system: null
  user: null

cpuType:
  aarch64: null
  x86_64: null

firmware:
  legacyBIOS: null

video:
  display: null

propagateProxyEnv: null

hostResolver:
  enabled: false
  ipv6: null
  hosts:

dns:
 - <IP_ADDRESS>

The instance spin up but freezes on
update-initramfs: Generating /boot/initrd.img-6.2.0-20-generic

I cannot open another shell on the instance nor can interrupt the process.  If I do open another shell prior watching processes prior to the sudo apt-get upgrade the shell will lock too.
When I go to stop the instance it doesn't stop smoothly
INFO[0000] Sending SIGINT to hostagent process 61430
INFO[0000] Waiting for the host agent and the driver processes to shut down
INFO[0000] [hostagent] Received SIGINT, shutting down the host agent
INFO[0000] [hostagent] Shutting down the host agent
INFO[0000] [hostagent] Stopping forwarding "/run/lima-guestagent.sock" (guest) to "/Users/mjwalsh/.lima/mep-amd64/ga.sock" (host)
INFO[0000] [hostagent] Unmounting "/Users/mjwalsh"
INFO[0000] [hostagent] Unmounting "/tmp/lima"
INFO[0000] [hostagent] Shutting down QEMU with ACPI
INFO[0000] [hostagent] Sending QMP system_powerdown command
FATA[0180] did not receive an event with the "exiting" status

I tried uninstalling brew installed qemu 8.0.2 and installing my own tap of 8.0.0, but the same thing happens.
I'm lost as what to do.
Is there something I can do to provide insight and are others seeing this?

Am I the only one seeing the same? @esposm03 you gave the issue a thumbs up does this indicate you are impacted as well.

Yes, I am impacted too, I just didn't want to start polluting this thread with "Me too" comments.

Thanks for reporting.
Does it work stably with vz?

Yes it does, however I need to both run and debug i386 binaries which rosetta doesn't support. My current workaround is manually running the binary I want to debug under qemu-i386 -g option, and then in another shell using gdb as normal.

I don't know if it's related but I get frequent freezes on an M2 Max with an aarch64 Ubuntu VM. This happens randomly, a few times a day (sometimes even a few minutes apart). I have 3 or 4 terminals open with zsh running in the VM, I'm usually editing code with Vim or running make, etc. and all my terminals would become unresponsive. Doing lima zsh hangs, I can only do limactl stop, and restart the VM.
I'm not familiar at all with the technology behind lima, so let me know if there is anything I can do when it freezes that could provide more information as to what the problem is.

I don't know if it's related but I get frequent freezes on an M2 Max with an aarch64 Ubuntu VM. This happens randomly, a few times a day (sometimes even a few minutes apart). I have 3 or 4 terminals open with zsh running in the VM, I'm usually editing code with Vim or running make, etc. and all my terminals would become unresponsive. Doing lima zsh hangs, I can only do limactl stop, and restart the VM.
I'm not familiar at all with the technology behind lima, so let me know if there is anything I can do when it freezes that could provide more information as to what the problem is.
Occasionally, limactl stop will also hang indefinitely on this:
$ limactl stop
INFO[0000] Sending SIGINT to hostagent process 9976
INFO[0000] Waiting for the host agent and the driver processes to shut down
INFO[0000] [hostagent] Received SIGINT, shutting down the host agent
INFO[0000] [hostagent] Shutting down the host agent
INFO[0000] [hostagent] Stopping forwarding "/run/lima-guestagent.sock" (guest) to "/Users/mdorier/.lima/default/ga.sock" (host)

When this happens, I have no other choice but to restart my computer.
Note: limactl is version 0.16.0, installed via brew.

I am encountering the same issue. Is there a good way to debug this to help identify the issue?

I don't know if it's related but I get frequent freezes on an M2 Max with an aarch64 Ubuntu VM. This happens randomly, a few times a day (sometimes even a few minutes apart). I have 3 or 4 terminals open with zsh running in the VM, I'm usually editing code with Vim or running make, etc. and all my terminals would become unresponsive. Doing lima zsh hangs, I can only do limactl stop, and restart the VM.
I'm not familiar at all with the technology behind lima, so let me know if there is anything I can do when it freezes that could provide more information as to what the problem is.
Occasionally, limactl stop will also hang indefinitely on this:
$ limactl stop
INFO[0000] Sending SIGINT to hostagent process 9976
INFO[0000] Waiting for the host agent and the driver processes to shut down
INFO[0000] [hostagent] Received SIGINT, shutting down the host agent
INFO[0000] [hostagent] Shutting down the host agent
INFO[0000] [hostagent] Stopping forwarding "/run/lima-guestagent.sock" (guest) to "/Users/mdorier/.lima/default/ga.sock" (host)

When this happens, I have no other choice but to restart my computer.
Note: limactl is version 0.16.0, installed via brew.

I'm seeing something similar on my end. My VM hangs and I'm forced to stop it.
❯ limactl stop dev
INFO[0000] Sending SIGINT to hostagent process 20385
INFO[0000] Waiting for the host agent and the driver processes to shut down
INFO[0000] [hostagent] Received SIGINT, shutting down the host agent
INFO[0000] [hostagent] Shutting down the host agent
INFO[0000] [hostagent] Stopping forwarding "/run/lima-guestagent.sock" (guest) to "/Users/chrisshyi/.lima/dev/ga.sock" (host)
INFO[0000] [hostagent] Unmounting "/tmp/lima"
INFO[0000] [hostagent] Shutting down QEMU with ACPI
INFO[0000] [hostagent] Sending QMP system_powerdown command
FATA[0180] did not receive an event with the "exiting" status

Running limactl list afterwards shows that the VM is stopped
❯ limactl list
NAME    STATUS     SSH            VMTYPE    ARCH       CPUS    MEMORY    DISK      DIR
dev     Stopped    <IP_ADDRESS>:0    qemu      aarch64    4       8GiB      200GiB    ~/.lima/dev

As far as I know, the ACPI works differently on ARM. So that could explain acpid is not able to shut down the OS correctly (it should still "pull the plug" eventually)
It should not affect running VMs, but we might want to try to execute a software shutdown on this architecture ?

I randomly experience a similar issue - the VM freezes completely and I can only force stop it.
But I use arm64 VM on vz driver on kind of the same platform 2021 M1 Pro.

@afbjorklund Thanks for chiming in. Could you elaborate on what you meant by a software shutdown on ARM?

I mean if the "hardware" power down signal is not working properly, then maybe issuing a poweroff command in the shell works better?

ah I see what you mean! But generally when the VM freezes we don't shell access, which is why I had to stop the instance from the host

That is a different issue, I was referring to the specific issue when the ACPI was not properly caught by the guest OS

I'm experiencing this as well on an M2 Max MBP. My experience seems identical to @mdorier. Happy to provide any details that would be useful for debugging.
lima version: 0.20.1 (installed via brew)
VM type: qemu
OS version: Sonoma 14.2.1
Guest VM: Ubuntu 23.10 (release 20240209)

I'm experiencing this as well on an M2 Max MBP.

Does vz work better?

I set the cpu to 1 and haven't had a problem since.


Put nginx configurations in dedicated directory

The default configuration of nginx was updated to use a configuration
directory.  This allows multiple applications to be hosted on the same
nginx instance/host as long as different port numbers are used.
Updating the install script to use this change so that users can get
going with IPXE without problems.

+1 from me


run page design

Looks great, I'm gonna merge it. Please remember to create the alternative design on figma when you got time and muse :)


Add Python trainer API

TODO:

[ ] test cases

Closes #10

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   13.77%   13.77%
=======================================
  Files           3        3
  Lines         508      508
=======================================
  Hits           70       70
  Misses        438      438

Continue to review full report at Codecov.

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


Shouldn't InputStream Helper be a Service rather than a Program add-on?

InputStream Helper appears in the Add-ons > Program add-ons menu in Kodi which is a bit annoying. Isn't InputStream Helper a Service add-on rather than a Program add-on?

InputStream Helper is in fact a library. But is has a settings page, and because a library with a settings page is not easy to find (in order to change settings) we decided on 2 things:

Add an API for other add-ons to call from their settings
Have an add-on entry script to easily go to the settings (program)

In short: ISH is not a service, but not really a program either.

InputStream Helper is in fact a library. But is has a settings page, and because a library with a settings page is not easy to find (in order to change settings) we decided on 2 things:

Add an API for other add-ons to call from their settings
Have an add-on entry script to easily go to the settings (program)

In short: ISH is not a service, but not really a program either.


kxd matrix or 1xd vector?

In section 3 of paper 'Augmenting Convolutional networks with attention-based aggregation':
···
We can easily specialize the attention maps per class by replacing the CLS vector with a k × d matrix, where each of the k columns is associated with one of the classes. This specialization allows us to visualize an attention map for each class, as shown in Figure 2.
···
But I only found 1 x d vector. Where is k x d matrix?
https://github.com/facebookresearch/deit/blob/40ae72b79cc5cd48dac2b02e1fceb03ee4192676/patchconvnet_models.py#L201

Dear @densechen,
as stated in the paper (see discussion in Section 3.2),
"By default, we use the more convenient single class token."
The code that we provide for now corresponds the main model based on the vector, which corresponds to most of our results (in Table 2 we provide the result of S60 when specializing the attention per class).

We will provide the S60 model and weights used to generate the visualization in our paper.
cc @TouvronHugo

It seems ridiculous.

one class-token per class works worse then a single class-token in Table 5.
Actually, one class-token per classes is not used in the released code or the experiments described in the paper.
Why spend so much time describing a design that doesn't work? Or just for a better visualization?
At the first glance, we think this one class-token per class is a key design which is quite different from the one in ViT.

It seems ridiculous.

one class-token per class works worse then a single class-token in Table 5.
Actually, one class-token per classes is not used in the released code or the experiments described in the paper.
Why spend so much time describing a design that doesn't work? Or just for a better visualization?
At the first glance, we think this one class-token per class is a key design which is quite different from the one in ViT.

This design is introduced as a straightforward variation of our learned aggregation layer. And yes: mostly for vizualization.
In Figure 1 most of our result are with 1 token. I am sorry if you think you find we have spend 'much time', I actually thought that we were presenting it as a straightforward variation of our main proposal: only the sentence 'We can specialize this attention per class' in Figure 1 for the last column, and a small paragraph in Section 3, where we point out some limitations of this variant. Maybe the confusing part is Fig. 3, where we could easily explain the tiny difference between both choices.
So yes, our paper has a significant focus on how to provide vizualisation and some interpretable aggregation layer to convnets, this is also true for the single-token version.
The choice of the single token is driven by a more direct relationship to this objective, as it provides the weight of each patch in the aggregation. The one-class-token provides a weight per patch for each class, independent of whether the class is selected or not by the classifier. While both have their pros and cons, we feel like they are complementary for vizualisation purposes.
Hervé

This design is introduced as a straightforward variation of our learned aggregation layer. And yes: mostly for vizualization.
In Figure 1 most of our result are with 1 token. I am sorry if you think you find we have spend 'much time', I actually thought that we were presenting it as a straightforward variation of our main proposal: only the sentence 'We can specialize this attention per class' in Figure 1 for the last column, and a small paragraph in Section 3, where we point out some limitations of this variant. Maybe the confusing part is Fig. 3, where we could easily explain the tiny difference between both choices.
So yes, our paper has a significant focus on how to provide vizualisation and some interpretable aggregation layer to convnets, this is also true for the single-token version.
The choice of the single token is driven by a more direct relationship to this objective, as it provides the weight of each patch in the aggregation. The one-class-token provides a weight per patch for each class, independent of whether the class is selected or not by the classifier. While both have their pros and cons, we feel like they are complementary for vizualisation purposes.
Hervé

Hi @densechen,
I have added the code and the model S60 for the multi-class token version. As discussed in the paper, this model is difficult to learn from scratch compared to the model with 1 class token, which explains the performance of 81.1% reported in Table 5.
This is why, in our paper (see page 3 and text-image below), we recommend to fine-tune the multi-class token model from the weights of the single-class token model. In that case the model is easier to train, you typically get comparable results to the single token-case. For instance for the S60 we obtain 82.0% for multi-token vs 82.1% for the single-token architecture.
I will add the weights next week. I hope this addresses your concern.
Best,
Hugo

@TouvronHugo Thanks for your kindly reply. Your detailed explanation makes me know better about the whole process. Thanks!


docs(*): clarify type restriction on identity fields

Concerto enforces at runtime that identifying fields must be of type String, but the documentation does not reflect this. This change updates the documentation to clarify the type restriction on identity fields.
Signed-off-by: Simon Stone<EMAIL_ADDRESS>

Duplicated by #414


Update fetchGitRoot to not halt execution

What?
This PR updates the fetchGitRoot function in files.ts to not error when attempting to runExternalProgram. Explicitly, this change involves:

Updating the return statement to return the currentWorkingDirectory if gitRoot == ""
Updating the catch (error) block to log the error, and then returning the currentWorkingDirectory instead of throw new Error.
Updating the associated test for fetchGitRoot() inside of files.tests.ts

@jimmy-lee I have a suspicion why the tests are failing, but I won't be able to get to this until later today or tomorrow morning. I'll merge this in and deploy a new version tomorrow afternoon

@jimmy-lee I have a suspicion why the tests are failing, but I won't be able to get to this until later today or tomorrow morning. I'll merge this in and deploy a new version of the uploader tomorrow afternoon

Thank you!

Re-running tests.

@drazisil-codecov I think this is just access to CODECOV_TOKEN

Enabling secret sharing with the fork in order to pass the CI. Will disable again after.

Thank you, @jimmy-lee  and @carlosdoordash


Dialog.showMessageBox() does not return the correct button index when using keyboard

I have noticed that when running the following code on Windows:
.....................
var buttons = ['OK', 'Cancel'];

dialog.showMessageBox({ type: 'info', buttons: buttons, message: 'Exit?' }, function (buttonIndex) {
	updateFooter("Exit: " + buttons[buttonIndex]);
});

and using arrow keys to navigate between the OK and Cancel buttons, and then using Enter key to close the dialog, I am always getting back 0 regardless of which button was actually selected.
When I click on the button with a mouse, the code works as expected.
Has anyone else observed the same behavior?

I'm merging this to #2090.


WebUI to manage OrbitDB and databases

OrbitDB should have a WebUI app through which OrbitDB instance can be managed.
This would include:

Create new databases
Query databases
Add databases to replicate
Network and connection status
Display database information, such as: data directory info, keys, size (bytes or entries), replication status, etc.

At the moment, as we don't have a daemon nor http server, this could simply be a browser app where OrbitDB is embedded. Later on, we can serve the WebUI via a daemon or http server and connect to and manage a "backend / server" instance of OrbitDB.
This is a fairly large task but also not hugely complicated and would be a great task for someone looking to contribute in a big and meaningful way! If you want to work on it, comment here.

@therebelrobot thanks for jumping in and sharing your thoughts!
There's not much design for this yet, so I'd be happy with whatever the implementor decides to use here. Couple of personal preferences and thoughts around it:

This will be a fairly extensive app (eventually) in terms of the frontend, so using a framework would make sense. I personally like React and would love to use it, but I'd be fine with other well-known frameworks as well. Whatever you're comfortable with.
I think we could get an MVP without the HTTP API: OrbitDB can be embedded into a browser app just like any other module. When embedded, the databases get created in the browser's storage just the same as they would on a filesystem (same for IPFS). So what we can do to get started, is to manage databases "local-to-browser". We'd be able to create, modify and replicate the databases between two browsers just as they would between two Nodejs/http-api instances of OrbitDB. Here's an example how OrbitDB can be embedded to a browser app.
As you propose @therebelrobot, we should definitely separate the layer that interacts with the http-api or the (embedded) OrbitDB instance. If we start with the embedded OrbitDB instance and separate that, we can later replace db.add("hello") with fetch("http://localhost:37373/add/<database_address>/ and it should work all the same.
Platforms: the cool thing about js-ipfs is that it works both in the browser and in Node.js. And with the browser version, we can run it on any OS and platform that has a browser, including mobile devices! :) So if we go with the embedded approach, we would have the MVP running on mobile devices too without any extra effort, which would be really cool imo.

Hope this helps! Let me know if you have more questions, suggestions, ideas or otherwise want to explore further 😄
Or if you have the motivation and time to work on this, I'd be more than happy to help you get started and otherwise. #orbitdb on IRC (Freenode) would be the best way to chat in real-time.

Hello, @haadcode I would like to work on this if you guys approve it!

Awesome! Yeah, we'd love for you to help out. You're already an old hand from your other repo; if you need to ask general questions, Gitter (which it looks like you joined recently?) is probably the best place. :) All systems go.

Thank you @RichardLitt! I'll begin to work based on the message that @haadcode wrote above if I have any question I'll let you know! Thanks!!

@natachadelarosa did you end up making any progress with this?

@phillmac  @RichardLitt I'm so sorry this just fell out completely for me. A lot of things happened, do you mind if I start again on this?

Ah, good news @natachadelarosa! You don't have to start from scratch. Check out: https://github.com/orbitdb/orbit-db-control-center

@aphelionz  I saw that most of the features are implemented and also just one issue opened, one thing I saw was that an already listed DB can be added by address is that okay? should it be fixed?

It's alright, @natachadelarosa! Thanks for following up, regardless. :)

@natachadelarosa Life happens to all of us; I'm certainly no exception. :grin: One of the goals I had in mind was to create a front end for the http api as a sort of control panel, an alternative to running a full node in browser. I've not gotten very far at all with it . One of the intermediate goals was to create a drop in replacement for the orbit-db- package that was 100% compatible, instead backed by the api. I'd still like to have a go at both of these :grin:

Sounds great @phillmac! I'll go with the HTTP API front-end!

Ok so here's the state of things, orbitdb/orbit-db-http-api is quite outdated and has a few silly bugs in it. however it's mostly working. I really need to update it.
phillmac/orbit-db-http-api-dev is my own fork with a lot of non-standard features for debugging/ manipulating base orbit-db instance.
Usually I fix bugs as quickly as I can however its definitely to be considered 'unstable' and I end up completely breaking it on occasion
There's respective docker images here:

https://hub.docker.com/r/orbitdb/orbit-db-http-api

and here:

https://hub.docker.com/r/phillmac/orbit-db-http-api-dev

if they're any use to you.
Here's an example docker-compose.yml:

https://github.com/orbitdb/orbit-db-http-api/issues/26#issuecomment-608962546

Here's an example with external ssl via lets'encrypt, certbot & nginx

https://github.com/phillmac/compose-example-orbit-db-http-api

Commenting on this issue to let anyone that might be interested: I am writing an RPC server for Orbitdb based on the ConnectRPC framework from Buff.build. It will be written in Typescript and should hopefully cover most use cases by providing native code clients for:

Browser
Node
Go
Swift
Kotlin

While also being accesible through Buf Studio to try, applications like Postman as well as normal HTTP/2 request using CURL or whatever http client.
My hope is that after doing this is building a webui similar to the old OrbitDB Control Center. The RPC daemon is a necessary step towards this since OrbitDB instances can't share the same directory concurrently for locking issues reasons.
This is the repo for the RPC daemon and the service declaration schema hosted on protobuf.
I am still in the early implementation since I had to create typings for the Orbitdb package manually since 1.0 had breaking changes with the community typings. Any advice, PRs, issues, etc is welcome. At some point I hope the orbitdb team makes it a package usable from @orbitdb/rpc or something similar, as well as the UI with @orbitdb/webui. Let's see where I can get to.


PUPIL-317: Support lessons with missing sections

Music year: 1997
Description
Supports lessons with one or more missing sections (starter quiz, exit quiz and video).
🤦♂ Includes a fix for a mistake I made when building the overview page — somehow I managed to get the count of exit and starter quiz questions the wrong way round.
How to test

Go to https://deploy-preview-2234--oak-web-application.netlify.thenational.academy/pupils/programmes/english-primary-ks1/units/writing-lower-case-letters-in-print/lessons/formation-of-v-w-x-and-z
You should see a lesson with an intro, starter quiz and video
You should be able to take the lesson and complete the 3 sections
After completing the 3 sections you should see the lesson review.

NB: There aren't any lessons available currently with a missing starter-quiz or video. So i've had to simulate these in the API. If the API contracts can be relied upon when the data is available then we should be good 🤞🏼
Screenshots
https://github.com/oaknational/Oak-Web-Application/assets/122096/0c8e4fcd-da2f-4783-8f47-e2c962dae132
👇🏼 I cheated and simulated no quizzes in my local environment to create this demo
https://github.com/oaknational/Oak-Web-Application/assets/122096/0219109b-25fa-456a-a336-e8f424c8cdc9
Theoretically according to the API we could have a lesson with no sections and only an intro. This is supported but I feel like it would be unlikely.

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

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


Add failsafe for getAssetBalance

Currently we automatically select ws endpoint or we give user option to override it, give them ability to either provide multiple endpoints or ability to switch between automatically them whenever one fails

Make this more generic use this in every scenario that generates API - xcm calls, asset queries and any else possible scenarios.
Also one way to do this is to allow endpoint to return promises within specific time and if no promise is returned switch to another API endpoint automatically.
This will resolve us having to update endpoints manually.


[video_player_platform_interface] upgraded to pigeon 1.0

The code should behave the same except the wire format has been modified in Pigeon 1.0 to support nested types and multiple arguments.
This shouldn't merge until Pigeon 1.0 is published: https://github.com/flutter/packages/pull/447
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[ ] I signed the CLA.
[ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Per previous discussion offline, based on https://docs.google.com/document/d/1BSrDA_FjLU-3T0H774RocJ4IYyc0ytMl937rAimGJdg/edit?usp=sharing the approach for updating Pigeon in video_player is going to be to move the platform channel for iOS and Android into the implementation, and then update there. Since that's the direct we want to go with platform channels, doing it first will avoid the need for a breaking change to the current platform interface (both saving us unnecessary churn, and avoiding disruption to unendorsed implementations).


Implement immediate pipeline composition for push sinks.

just compose it backwards. can still look left-to-right if ufcs is not used, eg:
mmappedFile("foo").peekPush(somePushFilter(nextPushFilter(pushSink()))

For now, all stages will be constructed in-place.


Idea - Using Dapper instead of EF

Hi Joe.
Thank you for the awesome work you have done here.
One request though is to provide a starter kit using Dapper. After using Dapper for a while EF seems real sluggish.

you are more than welcome to build a data layer using dapper


Fix for Wagtail@latest only has a favicon on the index page #281

I have done the changes hopefully this fixes #281

Oh I didn't realize there was a pull request already with the same changes my bad!


Did everything right, but no products showing up

I've followed the official instruction here step by step but seems like it's not connected with the Shopify storefront.
I doubled checked all the API keys, everything seems correct. I have also tested the webhook api using Send test notification and checking console  logs on the /api/revalidate/routes.
How can I check/figure out whats wrong? Why isn't the nextjs store showing the products I've added on the shopify store?

Hey there, sorry you're having trouble here. It's hard to tell what could be wrong in your setup. Could you try starting from scratch maybe? Maybe we can add more to the README if you determine what is missing.

Closing as stale, but let me know.

hi @leerob , the homepage is blank, but can search products:
https://nextjs-commerce-qadq.vercel.app/product/sony-alpha-zv-e10-mirrorless-vlog-camera
https://nextjs-commerce-qadq.vercel.app

My SHOPIFY_REVALIDATION_SECRET was wrong as I put there the blurred first part in https://vercel.com/docs/integrations/shopify#configure-shopify-webhooks-in-the-shopify-admin instead of what comes after secret=XYZ generated from https://www.uuidgenerator.net/guid.


Add NSW Design System detection

Website:
https://www.digital.nsw.gov.au/digital-design-system
GitHub:
https://github.com/digitalnsw/nsw-design-system
Examples:
https://www.nsw.gov.au
https://www.tag.nsw.gov.au
https://www.digital.nsw.gov.au
https://nsw.convivial.io
https://data.nsw.gov.au
https://www.asbestos.nsw.gov.au/

Thanks! I made a few changes.

Thanks! I made a few changes.


set_focus() on qt5 app does not work the first time. Window disappears from task bar

So I'm running up against a very weird issue and not finding anything relevant around. I'm trying to automate bringing an app to focus. Specifically, the app is (https://github.com/neovim/neovim/releases)[Neovim]. I'm on windows 8.1 and the plugin I'm writing is here - branch (https://github.com/raghur/vim-ghost/tree/pywinauto)[pywinauto] @ 68d37f09e0297df029f4b99639d9546bb4b600df
Now for the problem:

On first click from the browser addon, the plugin code runs this self.winapp.windows()[0].set_focus(). The neovim window disappears from the task bar.

If I try the recommended way - self.winapp.Neovim.set_focus() - I get a KeyError OR it just fails silently.
Once the window has disappeared, the process is still running and I can run the same code app.Neovim.set_focus() above in from an IPython session and the window isn't found (No windows for this process)
On the other hand, if I run app.windows()[0].set_focus(), the window is brought into focus and back in the taskbar.

For my plugin, this means that the first time interaction doesn't work - but after that everything works as intended (even without going through Ipython). So any idea why set_focus() doesn't work the first time but works after that without a hitch?
The setup is a little hard to replicate if you're not a Neovim user - so happy to do a screencast/im/video chat if needed.

So right after I posted this, some interesting updates:

If I call self.winapp.windows()[1].set_focus(), then the app window is brought to focus without the disappearing issue. However, keyboard focus isn't on it and I can't type till I click on it first.
For reference, app.windows() returns this:

In [129]: x.windows()
Out[129]:
[<hwndwrapper.DialogWrapper - 'Neovim', Qt5QWindowIcon, 14550444>,
 <hwndwrapper.DialogWrapper - 'M', MSCTFIME UI, 232785194>,
 <hwndwrapper.DialogWrapper - 'Default IME', IME, 141952228>]

I tried this to get the input focus back in my plugin code:

                self.winapp.windows()[1].set_focus()
                self.winapp.windows()[0].set_focus()

But now, I can visually see the window being brought into focus the and then immediately disappearing from the task bar (Again - first time only). Repeating the action, the window is brought into focus with keyboard focus without issues.

So I think I have finally found what works reliably:
            try:
                self.winapp.windows()[0].set_focus()
                self.winapp.windows()[0].ShowInTaskbar()
            except Exception as e:
                logger.warning("Error during _raise_window, %s", e)

I tried putting IsInTaskbar() but that returns True even when the window has disappeared from the task bar.
Would love not to have a hack - so keeping the issue open.

Sorry, we a bit busy right now. QT5 support is pretty limited in pywinauto. Any luck with UIA backend?

Hi @raghur, good investigation. I've seen similar problem for Qt5 app (WireShark) but had no much time to debug it yet. Old implementation with SetForegroundWindow(...) we used in set_focus() before is also working for WireShark all the time. Maybe need to combine these approaches properly or find a better way.

I'm getting back to this case. Running standalone nvim-qt.exe has shown several interesting things.
When using Application(backend="uia") (and Inspect.exe) there is no title text available:
>>> app_uia = Application(backend="uia").connect(path='nvim-qt.exe')
>>> app_uia.windows()
[<uiawrapper.UIAWrapper - '', Dialog, -1584082235>]

Inspect.exe shows the same: title is absent in window properties. And app_uia.Dialog.set_focus() doesn't work silently: no crash or error, but the window is not brought to focus.
When using backend="win32" (the default one), the picture is much more optimistic. .set_focus() is working perfectly and I couldn't detect any issue. Even first time connecting to the app or starting it.
app = Application(backend="win32").connect(path='nvim-qt.exe')
app.Neovim.set_focus()

Is this reproducible for browser plugin only? Or standalone app as well?

Is this reproducible for browser plugin only? Or standalone app as well?

I'm not sure I understand what you mean - it's a nvim plugin - so runs inside nvim's python plugin process and tries to control the nvim-qt window and bring it to foreground.


Dependency and CI Updates

Update to latest dependency versions
Refactor test code to Sinon 6.x API
Change TS target to ES6
Refactor to use async/await instead of promises
Update AppVeyor config with our standard configuration
Add Test Results badge to readme

Coverage remained the same at 100.0% when pulling 9eb061c8e73b4c0a08613710daa3f5a93d47061d on cc into 96859992e100eec0702e6cfb74c990348fe614b1 on master.


disable referrer by default

With stuff like https://blog.mozilla.org/security/2015/01/21/meta-referrer/ now widely available, I think PDF.JS should set the policy to origin to limit leakage of potentially sensitive PDF URIs via links clicks in the PDF.

By default all external links already use rel = noreferrer, please see https://github.com/mozilla/pdf.js/blob/7c7d2399956c25491aa7d2548c9c784cc2b3cea8/src/display/global.js#L230-L236.
Closing as answered, since it doesn't seem like there is anything more we'd need to do here.

ohh! thanks! sorry I missed it.


Use column position when using Ctrl-A in insert

With this change using Ctrl-A in insert mode will take into account the column position as well. This means that only the newest insert gets pasted.
closes #283

@benjie, could you check if this fixes issue for you?

@Shatur95 Afraid I no longer use VSCode (back to Vim + tmux for me!), but the test looks like it covers what I would do to test it if I did :+1:

@benjie, good choice, BTW :)


feat: handle MsgCreateVestingAccount

Description
Closes: #XXXX

This PR will handle the MsgCreateVestingAccount as the auth module currently only handles genesis vesting accounts.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[x] targeted the correct branch
[x] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[x] reviewed "Files changed" and left comments if necessary
[x] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2/cosmos/stargate@46a82f7). Click here to learn what that means.
The diff coverage is n/a.

@@                  Coverage Diff                  @@
##             v2/cosmos/stargate     #317   +/-   ##
=====================================================
  Coverage                      ?   50.02%
=====================================================
  Files                         ?       19
  Lines                         ?     1891
  Branches                      ?        0
=====================================================
  Hits                          ?      946
  Misses                        ?      816
  Partials                      ?      129

@huichiaotsou which chain have you tested it with and at what height? I am testing it locally with desmos but so far it can't find any msgs with MsgCreateVestingAccount type

@huichiaotsou which chain have you tested it with and at what height? I am testing it locally with desmos but so far it can't find any msgs with MsgCreateVestingAccount type

so far I know only 1 case for MsgCreatePeriodicVestingAccount which was our validator account created by Kwun on agoric, it's at height 3507144

These accounts in this transaction are all not being handled.
https://agoric.bigdipper.live/transactions/7A3AFC2DD94A023419EC41846AE5FD0C36A070581B19623B8855CB0FE326CD0C

Same issue as mentioned in PR #318, the start_time value is always null. Can you try to pass the timestamp from the tx? Other than that everything looks good and it's working correctly.

Was the problem raised by Kwun solved? Is the account now being stored correctly?

@RiccardoM Yes the issue raised by @kwunyeung was fixed in PR #318 that was directly targeting v2/chains/agoric/mainnet branch


Add svg for no content file

Proposed Changes

Potential Breaking Changes

Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1251 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1251   +/-   ##
=======================================
  Coverage   99.49%   99.49%
=======================================
  Files         356      356
  Lines        8971     8971
  Branches     2128     2128
=======================================
  Hits         8926     8926
  Misses         45       45

Continue to review full report at Codecov.

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


Add asynchronous test blocks

I use the tit snipped a lot, but whenever I have async tests, I sometimes forget to add the async keyword before the arrow function. It would be great if there were snippets that cover this use case.
This PR adds two new snippets the create asynchronous tests:

testa -> Creates a test('should ', async () => {}) block
tita -> Creates an it('should ', async () => {}) block

@bernardodomingues-hotmart Thanks for your contribution. I will release it today :)


[GitHub] Repository Search Sort By Most Stars⭐

Extension
https://www.raycast.com/raycast/github
Description
Toggle to search by most stars.
Who will benefit from this feature?
Everyone who is looking for the most reliable repo.
Anything else?

@raycastbot assign me

This is being implemented and once merged you should have these options to sort repositories by:
https://github.com/raycast/extensions/blob/5f9d04c1f3bd512841193be45e9ab3f9dfe03c16/extensions/github/src/helpers/repository.ts#L136-L139


Driver for PCA9685 + example

The PCA9685 is an I2C-bus controlled 16-channel LED controller optimized for Red/Green/Blue/Amber (RGBA) color backlighting applications. Each LED output has its own 12-bit resolution (4096 steps) fixed frequency individual PWM controller that operates at a programmable frequency from a typical of 24 Hz to 1526 Hz with a duty cycle that is adjustable from 0 % to 100 % to allow the LED to be set to a specific brightness value. All outputs are set to the same PWM frequency.

I'm going to merge this one tomorrow if no objections.


Add signatures for Sidekiq::Web

Type of Change
I added it to prevent check errors when using Sidekiq::Web.

[ ] Add RBI for a new gem
[x] Modify RBI for an existing gem
[ ] Other:

Changes

Gem name: sidekiq
Gem version: sidekiq (7.1.5)

I have signed the CLA!

@yuichi-tanaka if you add require "sidekiq/web" to sorbet/tapioca/require.rb and regenerate the RBIs a signature will be automatically added.

@yuichi-tanaka if you add require "sidekiq/web" to sorbet/tapioca/require.rb and regenerate the RBIs a signature will be automatically added.

Look like the right solution, closing this PR 👍


Add new feature: Support --config with config file

Hello, I have an idea that you can add to your project. Basically, users are now able to pass url as parameters but in case they don't want to do it every time, so I'm thinking of adding a config option to your project, so that users just have to create a file and write their configuration to the file.
Some config files for reference
  "input": "./docs"

{
  "input": "./docs",
  "output": "./web"
}

That sounds great.

/@tuanthanh2067


Generates the rpc and rset info for exported GDScript.

Improved the send rpc log message when fail.

This work has been kindly sponsored by IMVU.

Fixed and squashed commits.

If anything else, this is ready to be merged @akien-mga.

Thanks!


Can't launch Yaagl

Describe the bug
Can't launch Yaagl
Screenshots

System Information (please complete the following information):

macOS: [10.13.6]
Device: [iMac 27-inch, Late 2009 ]
CPU: [intel i7 2,8 GHz ]
Dedicated GPU (if any): [ATI Radeon HD 4850 512MB]
RAM: [12G ]

Additional context
Add any other context about the problem here.
Logs (if any)

If it's related to the launcher, upload neutralinojs.log in Yaagl Data Directory

If it's related to the game, upload `logs/game_{timestamp}.log in Yaagl Data Directory

You didn't read https://github.com/yaagl/yet-another-anime-game-launcher?tab=readme-ov-file#install


[MM-18750] Fixed a typo that was causing tray icon to constantly show up

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
The call to shouldShowTrayIcon() was not actually a call at all, it was just checking if the function existed. As a result, the tray icon always showed up. This fixes that typo.
Issue link
https://mattermost.atlassian.net/browse/MM-18750
Test Cases

Open the desktop app on Mac
In settings, turn off 'Show Mattermost icon in the menu bar'
Restart the app
See that the menu icon no longer appears.

QA will test after merge is complete.


symm rrule implementation

Adds a missing rule for BLAS.symm!. I believe that this is the last missing Level 3 BLAS rule that we were missing!

codecov failure is due to additional error handling code which I can't easily test. Merging.


Typo in README file, soruce instead of source

💩 Bug Report
A summary of the bug
Hello,
There is a typo in README file in property name. It says soruce, but it should be source instead.
There are 4 places in a file with this typo, to be precise.
Environment

This project version(s): v2.0.0

[x] I've tried to find similar issues
[x] I would like to work on a fix 💪🏻

Oops.
Thank you, I'll fix it later.