Side Switch Verification when we do unrecoverable error

@lkammath Whenever we do unrecoverable error (Other than SBE error injection), We also need to verify that we are booting with the same side. We should not see any side switch.
Below is output of one version. But unfortunately not recreating now. Requested @SrideviRamesh to try it out.
root@witherspoon:/tmp/pdgb_tools/lib# cat test_journalctl.txt | grep -i 'Setting SBE seeprom side to'
Jul 28 18:40:02 witherspoon openpower-proc-control[1471]: Setting SBE seeprom side to 0
Jul 28 18:42:52 witherspoon openpower-proc-control[1998]: Setting SBE seeprom side to 1
Jul 28 18:53:20 witherspoon openpower-proc-control[2508]: Setting SBE seeprom side to 0
Jul 28 19:02:23 witherspoon openpower-proc-control[3032]: Setting SBE seeprom side to 0
Jul 28 19:13:00 witherspoon openpower-proc-control[3543]: Setting SBE seeprom side to 0
Jul 28 19:19:00 witherspoon openpower-proc-control[4073]: Setting SBE seeprom side to 1
Jul 28 19:24:40 witherspoon openpower-proc-control[4533]: Setting SBE seeprom side to 1

retried the scenario by injecting MCACALFIR_UE at IPLIng through BMC
oot@witherspoon:/tmp/pdgb_tools/lib# cat test3_journalctl.txt | grep -i 'Setting SBE seeprom side to'
Jul 28 18:50:31 witherspoon openpower-proc-control[1452]: Setting SBE seeprom side to 0
Jul 28 18:54:55 witherspoon openpower-proc-control[1984]: Setting SBE seeprom side to 0
Jul 28 19:05:04 witherspoon openpower-proc-control[2495]: Setting SBE seeprom side to 0
Jul 28 19:06:33 witherspoon openpower-proc-control[3017]: Setting SBE seeprom side to 1
root@witherspoon:/tmp/pdgb_tools/lib#

20170824054746061086_MyFFDCLogs.zip
test3_journalctl.txt

Design is that SBE side switch occurs if 2 IPL failures occur in a row.  This can be any type of IPL failure (pgood, host watchdog, checkstop).

https://github.com/openbmc/openbmc/issues/2114

More test info here https://github.com/openbmc/openbmc-test-automation/issues/965

Lakshmi tested few combination of side switch scenarios and also opened a defect https://w3.rchland.ibm.com/projects/bestquest/?defect=SW409624

Code automated.


test: make fixtures cross-platform compatible

Git on Windows changes the end of lines from LF to CRLF by default.
This changes the bytes of fixtures that a are text files. Change
the Git repository setting to checkout all files in test/fixtures
as they are, without platform specific modifications

Rather than having this change basically everywhere we have fixtures, it might be better to just document that if you want to run tests on windows for js projects, you'll have to configure Git to use input crlf instead of default.
Basically, user's dev env should run git config --global core.autocrlf input before cloning the repositories.
What do you think?

I actually ran into this and a similar issue where levelDB fails because of line-endings in fixtures during my first week of development: https://github.com/ipfs/js-ipfs/issues/1204#issuecomment-367064044. It caused me to switch the issue I was working on because I couldn't figure it out in a couple hours. There are so many points of entry to js-ipfs land that it is hard to ensure that developers will see a warning. While I also don't like adding this to all fixtures, this problem has a broad indirect impact and reduces friction for newcomers.

@VictorBjelkholm: Having two different workflows for running tests and not seems counter-intuitive. I would rather add .gitattributes to all fixtures. I even have writing a script that checks all repos which have text files in there and make a pull request with a .gitattributes file on my TODO list.


User sent to fresh new page upon contact form submission without Backend

Fixes Issue #1057
I added a feature without back end with which user would be sent to a fresh new page on clicking Send button where the message is displayed as "Thank you We received your submission , Return ". Also on clicking to Return button of that fresh new page, user is redirected to home page.
https://user-images.githubusercontent.com/61961107/123539142-dc1cd080-d755-11eb-80a9-78ffab50b096.mp4

Hey @anushbhatia kindly check this PR and please delete that old one as when I re-cloned it then it deployed on vercel too


Fix clippy lints

Coverage remained the same at 69.406% when pulling 8c12ead6b3715d892db27080a7f0a8806dd2f1ce on clippy-fix into a27e532fd2fc2f6a9899ce41419c64f2c7c01318 on master.


Automate and test out transactions at the shard and not table level.

Currently the library supports supports transactions at the table level. Reducing it down to the shard level will allow it to divide the work up faster and ingestion deployments to deploy at a much faster rate than before. This will also give users the ability to lock at the shard and not table level which is more useful.

@Override
  public List<ColumnId> getColumnIds(String tenant, String table) {
    TableMetadata tm = getTableMetadata(tenant, table);
    if (tm == null) {
       return new ArrayList<>();
    }
    return new ArrayList<>(tm.getColumnIds());
  }


SFTP Browser not visible in sidebar after VS Code update

Hi,
after updating my Visual Studio Code to 1.38.1 the SFTP browser is not visible in the sidebar anymore (no icon, no entry in the context menu). The "View: Show SFTP" command takes me to the file browser instead of the SFTP browser.
Any ideas what could be wrong?

Try a fresh restart.

Of Visual Studio Code you mean? I had tried that without success.

Deactivating and activating the extension fixes the problem until restarting VS Code. Also, my configuration is gone.

I thought I had this issue too (maybe something different) but I had to make sure I had View -> Appearance -> Show Activity Bar enabled.

Check to see if your sftp.json file is deleted under .vscode
I saw this issue when I used a git-commit to ignore/remove this file and it got deleted. Then the icon was lost and installing, reinstalling didn't help until I ran: SFTP:config

Check to see if your sftp.json file is deleted under .vscode
I saw this issue when I used a git-commit to ignore/remove this file and it got deleted. Then the icon was lost and installing, reinstalling didn't help until I ran: SFTP:config

Hey @JustAGrumpyGuy sorry for resurrecting this topic, but I've tried to run the following command and couldn't configure the SFTP extension. I've tried all the things that OP did, without success.
I'm using Natizyskunk's version. Currently my vscode is 1.93.1


Method for Clifford RB to return gates-per-Clifford

There currently is not a method in the clifford RB implementation that returns the number of gates per clifford operation, which is relevant for mapping the per-clifford error rate estimated in RB to an approximate per-gate error rate.

@coreyostrove Do you remember exactly what your use case/intended solution for this looked like?
A recap of the relevant code path: The CliffordRBDesign takes a list of depths that passes those depths on to create_clifford_rb_circuit, which samples the correct number of Cliffords and then uses a CompilationRules object to convert it into native gates.
Given that, it seems like there are two things you could want:

The CliffordRBDesign knows the number of Cliffords sampled, as well as the length of the circuit in the terms of native gates. This could give an average number of native gates per Clifford.
We could look up the number of native gates per Clifford from the CompilationRules object directly (which again are given as lengths of "compiled" circuits).

Depending on whether we wanted some sort of member function for CliffordRBDesign or just a utility function may determine which strategy we want to take.

I was just the messenger for this one (iirc from a list of suggestions/requests brought up in an IRL roadmapping session). I believe @jordanh6 was the brainchild behind this (or else may recall who was) and may be able to provide some context.
For what is is worth, I think having both items 1 and 2 would likely be useful, but I don't personally have an articulated need readily in mind.

I don't recall suggesting this, so it was probably @tjproct.
In my opinion both of your solutions are reasonable, but I also don't have a use case in mind beyond what's in the initial comment (rescaling the RB error rate). I have personally used approximately solution (1) to rescale RB error rates before on 3+ qubits, where you'd really rather not check the compilation length of each n-qubit Clifford gate. However, I think the utility function described in (2) could be nice to have also.

Closing as this is merged in develop with #443.


Fix 4 Style, 1 Maintainability issues in multiple files

CodeFactor found multiple issues:
Trailing spaces not allowed.
test\api-gateway.test.js:18
test\api-gateway.test.js:24

Newline required at end of file but not found.
test\api-gateway.test.js:42
src\api-gateway.js:33

Unnecessary semicolon.
src\api-gateway.js:32

resolved


Support for controllers descriptions

Hi,
I could not find a way to get controller descriptions in swagger?
This should be supported in swagger, I have seen various examples how to do this with @Api annotation.
Ideally,  XML tag on controller would be parsed to controller description.
Thanks,
Mario

I think this has been added with https://github.com/domaindrivendev/Swashbuckle.AspNetCore/commit/da987073ff212e1ed60b8d0b35798b05fe4cdfe8
@domaindrivendev I'm also looking forward to this feature. Thanks for adding it!! 🍺 Will there be a release anytime soon?

Should be available as of official release - v2.5.0
NOTE: You'll need to explicitly opt-in. This is because in a Swagger document, there's no such thing as a "controller" - just operations with tags. By default, Swashbuckle uses controller name as the tag but you can change this via the TagActionsBy setting. So, this feature only makes sense so long as you're using that default. See below for the opt-in syntax:
services.AddSwaggerGen(c =>
{
    ...

    c.IncludeXmlComments(xmlCommentsPath, true);
});

@domaindrivendev Thank you very much!


Provides ability to pass scoped packages

Provides ability to pass scoped packages (see: https://www.npmjs.com/package/@ninjaneers/nsp-reporter-owasp)

The Node Security Platform has been acquired by npm, Inc.
On April 10, 2018, the Node Security Platform joined npm, Inc., which operates the npm JavaScript package registry.
Learn more here.
The service remains operational for current users in its current state. No new features or fixes will be implemented.


COMException (0x8000FFFF) from Files.App.Data.Models.SelectedItemsPropertiesViewModel.set_ItemCreatedTimestampReal

Description
Application: Files.exe
CoreCLR Version: 7.0.1323.51816
.NET Version: 7.0.13
Description: The process was terminated due to an unhandled exception.
Exception Info: System.Runtime.InteropServices.COMException (0x8000FFFF)
   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|39_0(Int32 hr)
   at ABI.Microsoft.UI.Xaml.Controls.ITextBlockMethods.set_Text(IObjectReference _obj, String value)
   at Files.App.Views.Properties.GeneralPage.GeneralPage_obj1_Bindings.GeneralPage_obj1_BindingsTracking.PropertyChanged_ViewModel(Object sender, PropertyChangedEventArgs e)
   at Files.App.Data.Models.SelectedItemsPropertiesViewModel.set_ItemCreatedTimestampReal(DateTimeOffset value)
   at Files.App.Views.Properties.GeneralPage.UpdateDateDisplayTimer_Tick(Object sender, Object e)
   at ABI.Windows.Foundation.TypedEventHandler`2.Do_Abi_Invoke[TSenderAbi,TResultAbi](Void* thisPtr, TSenderAbi sender, TResultAbi args)
--- End of stack trace from previous location ---

Steps To Reproduce
Sadly, I do not know what I was doing when this occurred. If I reproduce it again, I will update this report.
Requirements
Exception should be caught and handled (or the cause of the exception corrected).
Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2428
Log File
debug.log

Thanks for the feedback, I see a number of these reports in AppCenter but I haven't been able to repro it.

I'd like to work on this.


Tag values (except names) should start with a lowercase character

E.g. colors and combo options.
See also #133.

And see also #139.
Done in 0.7.

Not a good idea to be that blanket. Traffic sign codes atart with upper case letters. Vis traffic_sign=GB:601 for example.

It's not blanket, see https://github.com/Zverik/every_door/blob/main/lib/models/field.dart#L149 for the list.


How to process my own data?

Thanks for the great work. How can I process my own data like yours?

Can you send me an email? I will send you the code behind the working demo.


Merging criteria

TODO:

If we consider the gap, how to deal with the weight?
where should distance calculation include?
where should the maintenance of a "total" profile T include?
Clarify tree structure


Add information about communication between islands

Sometimes there is a need to communicate between hydrated components, or islands. For example, we might want to update <CartDropdown/> from <AddToCartButton/>.
It would be great if there was some guidance on how to accomplish this with Astro.
@matthewp suggested on Twitter using regular events or using nanostores.

Thanks for making this issue, and as we discovered on Twitter, Ben is already writing something about this! I'm expecting to see a PR very soon (today?), so I'll close this issue in anticipation of that. (Yes, a little early, but just getting a head start on triaging and trying to manage all these outstanding issues!) We will hold Ben to his promise.  :)


Add more actions like turn on climate or charger??

Is there a way to add a "climate switch"?
As I know this works on the audi app it should also work on the ha integration, but there is no example or wiki about it...

You can make automations (or scripts) and use this in the action.
I created a button in HA to immediately start the climatisation and another to start it within 30 minutes.
I even created an automation that sends a message to my phone when I leave the car (= bluetooth connection drops) AND when the outside temperature is below 15 degrees; the message asks if I want to start the climatisation within 30 min of 60 min or....
  - service: audiconnect.execute_vehicle_action
    data:
      vin: <your VIN code>
      action: start_climatisation

Closing this issue as its more than 2 years old and is solved with HA service calls.


Lease

Adding some basics to common repo, and trying to start set up lease logic to be used from it.

WDYT about having a lease manager with a clear interface? Something like (just a quick shot, details might vary):
type LeaseManager interface {
	CreateOrGetLease(ctx context.Context, node *corev1.Node, duration time.Duration, holderIdentity string, namespace string) (*coordv1.Lease, bool, error)
	UpdateLease(ctx context.Context, node *corev1.Node, lease *coordv1.Lease, currentTime *metav1.MicroTime, leaseDuration, leaseDeadline time.Duration, holderIdentity string) (bool, error)
	InvalidateLease(ctx context.Context, nodeName string, leaseNamespace string) error
}

type leaseManager struct {
	client.Client
	log logr.Logger
}

// enforce interface implementation
var _ LeaseManager = leaseManager{}

func NewManager(cl client.Client, log logr.Logger) LeaseManager {
	return &leaseManager{
		Client: cl,
		log:    log,
	}
}

func (lm *leaseManager) CreateOrGetLease(...) ... {...}

I think every public method should have a Context as first arg, and return an error as last value, even when it's not used today. Would prevent annoying API changes in case we need it later 🙂

Thanks for adding more checks to the Makefile. What about adding a github workflow to call them? 🙂

Thanks for adding more checks to the Makefile. ~What about adding a github workflow to call them? slightly_smiling_face~
edit: sorry, I saw there is, I was confused by not seeing it executed thinking

Yep I was wondering about that too, I've figured that it has to do with origin been completely empty now and that we'll have the checks running in the second PR - but it could be something else I'm missing :man_shrugging:


Added 11.0.7469 to buildref json

Type of Change

[X] Bug fix (non-breaking change, fixes #3752 )
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Add missing item to build ref
Approach

Commands to test

Screenshots

Learning

thank you very much 🌮 !

I see this new build number entry in the buildref json in dbatools 0.9.369, but running Test-DbaSqlBuild against an instance with that build is still returning "not recognized as a correct version", even after restarting the PowerShell environment.  Any ideas?

What is your exact version string?
On Thu, Jul 5, 2018, 14:41 SQLGB<EMAIL_ADDRESS>wrote:

I see this new build number entry in the buildref json in dbatools
0.9.369, but running Test-DbaSqlBuild against an instance with that build
is still returning "not recognized as a correct version", even after
restarting the PowerShell environment. Any ideas?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/pull/3753#issuecomment-402831445,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYUFJC7cIDU0EUMDeEIbDpLjuYCRq9Waks5uDmv0gaJpZM4U_-_7
.

Get-DbaSqlBuildReference reports the following:
SqlInstance    :
Build          : 11.0.7469
NameLevel      : 2012
SPLevel        : {SP4, LATEST}
CULevel        :
KBLevel        : 4057116
SupportedUntil : 7/12/2022 12:00:00 AM
MatchType      : Approximate
... and t-sql select @@version reports this:
Microsoft SQL Server 2012 (SP4-OD) (KB4091266) - 11.0.7469.6 (X64)   Feb 28 2018 17:47:20   Copyright (c) Microsoft Corporation  Enterprise Edition (64-bit) on Windows NT 6.3  (Build 9600: ) (Hypervisor)

needed to bump date at the top, too. lemme redo this PR to make it happen

you can now run the same command with -Update that due to https://github.com/sqlcollaborative/sqlcollaborative.github.io/blob/master/assets/dbatools-buildref-index.json being updated, will fetch the latest version of the build reference file.
/cc @potatoqualitee : remember to keep both repos in sync when you merge here

Thanks for the -update parameter tip; that worked for 0.9.369, and I suppose one of the next module updates will include the revised json with updated "LastUpdated" date.  Thanks again!

what you see on https://sqlcollaborative.github.io/builds is what -Update sees, if you have an internet connection. We made it this way 'cause doing so we're not tied to module upgrade to be on par with SQL releases.


pull request for e2e test at 11/18/2016 04:26:08

:x: Validation status: errors

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


replaces avatar-image with e-concurrency task

Codecov Report

Merging #1395 into master will decrease coverage by 0.13%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1395      +/-   ##
==========================================
- Coverage   60.67%   60.53%   -0.14%
==========================================
  Files         181      181
  Lines        3987     3986       -1
  Branches      914      913       -1
==========================================
- Hits         2419     2413       -6
- Misses       1568     1573       +5

Impacted Files
Coverage Δ

...erehows-web/app/components/avatars/avatar-image.ts
100% <100%> (ø)
:arrow_up:

wherehows-web/app/components/purge-policy.ts
81.48% <0%> (-18.52%)
:arrow_down:

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 cd94e5a...5d0c1e3. Read the comment docs.

Coverage remained the same at 22.752% when pulling 5d0c1e3b93a4a53b8a7c8a3da9576185651209eb on theseyi:avatar-task into cd94e5af6fc574323f3f25ed0c2a490cd5f8aaae on linkedin:master.

@cptran777


Aarch64 codegen: represent bool true as -1, not 1.

It seems that this is actually the correct behavior for bool types wider
than b1; some of the vector instruction optimizations depend on bool
lanes representing false and true as all-zeroes and all-ones
respectively. For b8..b64, this results in an extra negation after a
cset when a bool is produced by an icmp/fcmp, but the most common
case (b1) is unaffected, because an all-ones one-bit value is just
1.
An example of this assumption can be seen here:
https://github.com/bytecodealliance/wasmtime/blob/399ee0a54c3be0f89ae07a0af5104ba929a9eba4/cranelift/codegen/src/simple_preopt.rs#L956
Thanks to Joey Gouly of ARM for noting this issue while implementing
SIMD support, and digging into the source (finding the above example) to
determine the correct behavior.
This PR also makes an unrelated cleanup: it appears that we had two ways of saying
cset, namely Inst::CSet and Inst::CondSet. Probably an artifact of our
implementation sprint with multiple people adding instructions at once. Removed
Inst::CondSet and settled on Inst::CSet.

cc @jgouly

Thanks! As noted above, I want to come back to the width/defined-bits optimizations later, but I'll go ahead and merge this now for correctness.


[Master] Make port_macs as the optional argument in guest_register API

Sorry that the previous PR https://github.com/openmainframeproject/python-zvm-sdk/pull/76
has bug on handling port_macs as the optional argument in guest_register API,
this PR makes it as the optional argument and also provides the fix for the key error
if mac key doesn't exist in nics_info or port_macs argument.
All local check passed with 'tox -v' as attached and local function test also passed and here is the test code:
#!/usr/bin/env python
import sys
from zvmconnector import connector
import six
from nova import exception
conn = connector.ZVMConnector('<IP_ADDRESS>', port=8080, token_path='/etc/zvmsdk/token.dat')
results = conn.send_request('guest_list')
if results['overallRC'] != 0:
print("zVM Cloud Connector request failed. Results: %(results)s",
{'results': six.text_type(results)})
raise exception.ZVMConnectorError(results=results)
print "Success! Result1: %s" % six.text_type(results)
print "============================================="
print "============================================="
#results = conn.send_request('guest_register', 'fba', 'rhel7', '1')
#results = conn.send_request('guest_register', 'fba', 'rhel7', '1', 'portttt')
results = conn.send_request('guest_register', 'fba', 'rhel7', '1', {'mac1': 'port1'})
if results['overallRC'] != 0:
print("zVM Cloud Connector request failed. Results: %(results)s",
{'results': six.text_type(results)})
raise exception.ZVMConnectorError(results=results)
print "Success! Result2: %s" % six.text_type(results)
print "============================================="
results = conn.send_request('guest_deregister', 'fba')
if results['overallRC'] != 0:
print("zVM Cloud Connector request failed. Results: %(results)s",
{'results': six.text_type(results)})
raise exception.ZVMConnectorError(results=results)
print "Success! Result3: %s" % six.text_type(results)
master-fix.log

some comments above
Thanks and all addressed with code updated.

master-all.log

good to go, just please fix merge conflict

@jichenjc @bjhuangr can you please help to review and merge it? thanks


Excelsior Config Changes

Adding support to build PX4 Firmware to run on the Excelsior board.

@julianoes Please review and merge.

@jywilson The changes look good on a first pass. However, I'm really quite confused about what this all is? Future readers that are not informed about Snapdragon, Eagle, etc. will be struggling even more.
Could you add some comments that explain what sdflight, eagle, and excelsior mean? Thanks.

@julianoes The Eagle board is our first generation Snapdragon Flight board.  We are currently in development on another board, code named Excelsior.
We want to be sure that PX4 is supported at the moment Excelsior is released, and we will be pushing changes to address any issues we find.  My hope is that they are not Excelsior specific issues and that anything we upstream will also benefit the Eagle board.  We needed these config files though to address a few build differences.

@julianoes Could you test on hardware including arming and add comments? I hope that I have the param fix on master by the time @davids5 Has his fixes in.

@LorenzMeier yes will do.

Closing this PR, the commits have been rebased in #4442.


The transaction is showing twice on Donation account.

The transaction is showing twice on Donation account.
Tested on staging app v2.0(349)

Unable to replicate this issue on staging app v2.0(349).
For now I'm closing this issue.


Upload in chunks, to mitigate poor network conditions

Current behavior :

app tries to upload each resource in 1 big PUT request
it may fail completely, after a quite long time trying (e.g. 2mn)

Desired behavior :

app tries to upload resource chunks in several small PUT or POST requests
when a chunk upload fails, other succesful chunks are kept server-side
failed chunk upload is retried (some "sensible number" of times)
resource is reconstructed when all chunks have been uploaded

The reconstruction may happen server-side, or even client-side (in target browser) if [#60 Streaming] is implemented.

See also #94 Download encrypted chunks early

WhatsApp does a nice job of uploading whatever part of resource can be uploaded...


Set up hardware-in-loop testing

We have already set up self-hosted runners for GitHub actions.
We will need to better define what is required in order to bring this to fruition.

Having issues with probe-rs, same error as reported here: https://github.com/probe-rs/probe-rs/issues/1829

The probe-rs issue is now fixed (https://github.com/probe-rs/probe-rs/pull/1832) but not yet released

I have managed to flash and run tests on-device using probe-rs and defmt, so we're unblocked and back on track here. I've written a simple test for GPIO so far, which I've played with a bit and seems to be working as expected.
I will continue writing tests, and also start working on getting the CI workflow set up to invoke them.

Test cases that would be useful, a working list:

embedded-hal DMA driver returns error when transfering from ROM (ongoing work)
short waits should not block embassy (systimer driver is/was buggy, workaround in #906)

This week had some unexpected work pop up, so I unfortunately have not made as much progress and I had hoped. However, I will still provide some updates on the current status of things:

I have written a CI workflow which should handle building the tests and running them on-device via our self-hosted runners. This needs a bit of work still, but is mostly complete.
There are some logistical/devops issues which need solving still, which I will discuss with a colleague who works with CI as soon as possible. These are not technically blocking issues, however, but still need to be addressed eventually.
The plan is to initially start with with just the single GPIO test binary, to prove out the process/workflow, and then additional tests can be added.
@t-moe has opened a PR adding RISC-V support to defmt-test and panic-probe: (https://github.com/knurling-rs/defmt/pull/796). Once this is merged we will use this.

Things are moving fast in the embedded testing world, and only two weeks later things are looking a bit different!

I have switched from defmt-test to embedded-test

https://github.com/probe-rs/embedded-test

I have simplified much of the configuration, and came up with solutions for passing the chip and target to the test runner
I have begun documenting how to write and run tests
GPIO tests are still working, planning to write simple UART and Full-Duplex SPI tests too, probably, just to help get things started
I still need to sort out some details with the CI workflow, but I'd say it's probably 80% of the way there

Minor update, we already had our first successful HIL CI runs: https://github.com/esp-rs/esp-hal/actions/runs/8204261710/job/22440035135 :rocket:

Quick update regarding HIL:

Its working fine for H2 and C6 and testing (at least, partially)  aes, gpio, uart and spi_full_duplex, : https://github.com/esp-rs/esp-hal/actions/runs/8280334758
We had several issues with HIL VMs:

GH actions jobs were running under tomas user. Now they run under  espressif.
Some VMs had problems detecting (or they were disappearing after being detected) the targets and reloading them.

We still dont know the root cause of this. Tomas removed an unused VM (with a rustboard) and everything started working.

probe-rs was disappearing after running the HIL workflow:

swatinem/rust-cache action has a known issue:

The cache cleaning process currently removes all the files from ~/.cargo/bin that were present before the action ran (for example rustc).

Either we don't use the action or we make the file immutable sudo chattr +i /home/espressif/.cargo/bin/probe-rs

Board support:

Maybe tomorrow we will have a rebased version of the probe-rs branch that adds support for probe-rs test which will enable C3 and C2.
semihosting and embedded-test do not support Xtensa

Documentation could be improved, atm we document the wiring and VM setup in:  https://github.com/esp-rs/esp-hal/tree/feat/hil-testing/hil-test#running-tests-remotes-ie-on-self-hosted-runners

Thank you again for your work on this @SergioGasquez, I'm really excited to have this at our disposal. Hopefully the probe-rs issues can be resolved soon and we can start to move forward with this; I think we're pretty close to being able to open at least a draft PR and start scrutinizing things a bit.
I guess we still have not really decided which event(s) will trigger this workflow. I still don't think our merge queues are configured quite right, but I had originally thought of this somehow being a final check before merging. Not sure if this is possible with the merge_queue event or not, or if we need to think of something else. I think trying to run HIL tests each time a PR is updated will quickly lead to problems, so we need to find some happy medium.

I think we're pretty close to being able to open at least a draft PR and start scrutinizing things a bit.

Definitely, I think with what I have I can already create the first PR (as a draft at least), so we can start giving feedback.

I guess we still have not really decided which event(s) will trigger this workflow.

This is one of the things that we still need to figure out. Each time a PR gets updated seems a bit too much to me too, and I have not experience with merge_queues, as you mention having something that runs just before merging and that if fails it does not merge would be very sweet. Will have a look at merge_queues

I've gathered a list of the esp-hal modules and how hard/easy I think it would be to test those (take it with a grain of salt as I may have misevaluated some). Also, some modules can't be tested, or it does not make sense:

[x] aes - Easy

[ ] DMA
[ ] Non- DMA

[ ] analog - Hard
[ ] assist_debug - Hard
[ ] clock -
[ ] delay - Easy
[ ] dma -
[ ] ecc - Easy
[ ] fmt -
[ ] gpio - Easy
[ ] hmac - Easy
[ ] i2c - Hard
[ ] i2s - Hard
[ ] interrupt -
[ ] lcd_cam -
[ ] ledc -
[ ] mcpwm - Hard
[ ] otg_fs - Hard
[ ] parl_io - Hard
[ ] pcnt - Hard
[ ] rmt - Hard?
[ ] rng - Hard
[ ] rom -
[ ] rsa - Easy
[ ] rtc_cntl -
[ ] sha - Easy
[ ] soc -
[ ] spi

[ ] Half duplex - Hard
[ ] Full duplex - Easy
DMA

[ ] Half duplex
[ ] Full duplex

[ ] systimer - Hard
[ ] timer - Hard
[ ] trace - Hard
[ ] twai - Hard
[ ] uart - Easy
[ ] usb_serial_jtag - Easy?

Just opened a PR: https://github.com/esp-rs/esp-hal/pull/1297

Minor update: C3 is now also being tested on ci https://github.com/esp-rs/esp-hal/actions/runs/8326344742

Initial HIL has been merged! https://github.com/esp-rs/esp-hal/pull/1297
Current board support

C3, C6 and H2 are supported
We need to setup the S3 machine and include it on the CI workflow.
C2 should be supported soon
Xtensa devices would require adding Xtensa support in semihosting and probe-rs

See https://github.com/taiki-e/semihosting/pull/9 and https://github.com/probe-rs/probe-rs/issues/2301
S3 is now supported! We still need to upstream changes and setup a VM with it, so it will probably be in another PR.

Current tests

Only aes, gpio, uart and spi_full_duplex are partially being tested. See full list of peripherals pending in: https://github.com/esp-rs/esp-hal/issues/809#issuecomment-1999308797

I think we can probably close this issue at this point. There is definitely still work to be done, but I would say that HIL has been "set up", and the remaining work can be split out into their own issues.
@SergioGasquez if you think this is premature please feel free to reopen.


Feature/solcx binary path

What was wrong / missing?

There was an option missing for configuring/overwriting solc binary installation path

How was it fixed / added?
You can now configure the path either using an environment variable SOLCX_BINARY_PATH or by providing an additional parameter to install method.
Cute Animal Picture
Sorry, don't have one.

Mh, ok, I think I missed the part for providing the path to the compile methods...

Looks good overall, can you fix the linting issues tho?

Done :)

Thanks for merging :)

Thanks for the contribution!  I'll have a new release out later today.

That would be great, thank you so much!

v0.8.0 is released and available on pypi :) thanks again for your contribution!


fixFix column cached stats size bug.

Fix column cached stats average size calculate bug. Average size is double type, when calculating, it is calculated by totalSize/rowCount. We need to cast totalSize to double, otherwise the result is not accurate, only keep the integer part.

run buildall

run buildall

run buildall

run buildall

run performance


Clean up logging in runtime code

Refactor the #if VERBOSE logging code to use our LOG_DEBUG macro instead.

LGTM.


Advanced Log in Ledger shows wrong prefix for xpub

In Ledger Live, the Advanced Logs object shows an xpub value with the regular cosmos account human-readable prefix (cosmos) instead of the bech32 public key prefix (cosmospub).
If I recall, an earlier version did in fact display the correct prefix so there might be a regression?
Please let me know if this is the wrong place to file this issue! Thanks!

This bug applies to Ledger Live, not the cosmos app.


VS code not working

Type: Bug
My operating system is windows11. VS code use to work fine till now but suddenly started to show a message "Visual Studio Code on Windows 8/Windows Server 2012 will soon stop receiving updates. Consider upgrading your windows version". It's been nearly 2 days but the problem is not fixed
VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 6.2.9200
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i5-13450HX (16 x 2611)

See also https://github.com/microsoft/vscode/issues/205699

Please change the compatibility mode of the program away from Windows 8. Closing in the meantime considering OP deleted their account.


'connect ECONNREFUSED'

I'm trying to run some tests, and after I click the 'Save test case', I receive this connect error displayed at the top of the page: 'connect ECONNREFUSED'

https://github.com/jsperf/jsperf.com/issues/459#issuecomment-363657708


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/signal-r-service/web-pub-sub
Module Details
CARML to AVM
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
No response
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@segraef will migrate this module from CARML, without becoming its owner. The module will be marked as orphaned once the migration is complete, unless someone volunteers to own it.
If someone wishes to become the owner of this module, please indicate your intent in a comment on this issue.

In development.

@matebarabas afaik @krbar wants to take over this module migration as well.

That's ok, you can assign it to me

@matebarabas PR is in review: https://github.com/Azure/bicep-registry-modules/pull/1219

[x] PR merged; Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.

Thank you for your work @krbar! I'm closing this issue now.
Since this module was migrated from CARML without an owner, an orphaned module request was also created: https://github.com/Azure/Azure-Verified-Modules/issues/734


adding some clarifications

@berenss added a little bit more clarity on 2 points. Appreciate if you can weigh in!

Thanks @berenss  - changes done.


read-only role for project does not provide kubectl access for pods

Turn on Auth
Have a cluster with 2 projects
Add another user on 1 project with the read-only role
log in as the other user
using kubectl shell or kubeconfig, unable to do kubectl get pods

$ kubectl get pods
Error from server (Forbidden): pods is forbidden: User "user-lr2bn" cannot list pods in the namespace "default"

@dramich is correct. I did not pay attention to the namespace/project.


Task improvements for the registry tool

Background
The registry tool makes heavy use of the worker pool for concurrent execution of subtasks associated with various commands. Currently if a task returns an error, the worker pool will terminate the command entirely. Some tasks intentionally avoid returning errors to prevent this.
Typically commands will define a specialized struct that implements the Task interface and generate instances of these tasks to handle operations like uploading OpenAPI specs, uploading Discovery specs, uploading Protobuf specs, uploading specs from a CSV row, uploading specs with new annotations, or uploading specs with new labels. Commands that work with a variety of resource types often define one task per resource type, e.g. annotating APIs, annotating API Versions, and annotating API Specs. In other cases tasks are used for computation, executing external applications, and more.
When commands are generating tasks for the worker pool, they're often produced from resources in the registry or from files in one or more directories. When several resource types from the registry are permitted, commands usually determine the argument's resource type before fetching those resources.
Ideas

Configurable worker pools so commands can ignore, log, or immediately halt when tasks return an error. See #435.
Centralized logging for tasks to improve consistency of command output.
Define reusable tasks for common operations like creating/updating resources.
Define reusable task generators for common arguments like files and resource names.

#435

#435 addresses the functional issue of controlling worker pool execution.

Right, I remembered the discussion and forgot it was tracked.

Let's consider putting worker pool support (with some or all of these improvements) in a new package under /pkg.


Critical Infrastructure Issues in dotnet/dnceng

[x] New Critical Infrastructure Issues will need to be opened in dotnet/dnceng these are created manually, not from a link in Build Analysis
[x] Update Critical Infrastructure Issue documentation to point to dotnet/dnceng
[x] Ensure the labels necessary for Critical Infrastructure issues are migrated to dotnet/dnceng
[x] Update Known Issue code to search for Critical Infrastructure issues in dotnet/dnceng
[x] Update functionality that adds Critical Infrastructure Issues to the Known Issue GitHub project (if this is the current process, I can't remember)

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

Updated documentation in the internal wiki.


Changing order of opened chat channel tabs

It would be nice if you could change order of opened channels tabs same way as in firefox/chrome etc
Also make sure to open aeolus first?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Would we need that if we could #133


Cannot Use Common Search Operator "-" to Block Keywords or Phrases

Description
When using Indeed.com search box, you can add the dash to remove keywords or phrases.
For example:  "Administrative Assistant" -"Master's Degree"
This will look for any Administrative Assistant jobs that do not include the words "Master's Degree" in the listing. This is very helpful to cut down on job listings to sift through.
However, when using JobFunnel, there is no option to do this, so I attempted to simply include it before the keywords as normal.
I received this error:
\lib\site-packages\yaml\scanner.py", line 258, in fetch_more_tokens     raise ScannerError("while scanning for the next token", None, yaml.scanner.ScannerError: while scanning for the next token found character '\t' that cannot start any token   in "settings.yaml", line 37, column 1
Environment

Build: 1909
Operating system and version: Windows 10

That looks like an issue with JobFunnel parsing the YAML config file. So what I recommend is posting the entire yaml configuration file you are using in a comment on this thread. My gut tells me you have a tab somewhere in your configuration file. YAML does not play well with tabs last I checked.
In regards to this indeed feature; typing keywords into any site's(Indeed, GlassDoor, etc) search box, is not the same thing as adding keywords to your config file. They achieve the same thing, but the search box is in the frontend. Who knows what goes on in the frontend. JobFunnel is building a link with the keywords you give it and sending a request. Not the same thing as you typing words into the search box. There might be some JS in there that does voodoo magic with the string that has the "-" operator attached to it. I can't confirm that because I've never used the feature, but that's my best guess.
This would be an interesting feature to implement, but I fear it adding unnecessary complexity to JobFunnel. I say this because I have a feeling this might be an indeed-only feature and it might not be worth the trouble. One thing we could do to implement a similar feature is bring all the jobs down and have some "exclude keywords" key in the YAML file and filter all of the jobs against that. Not sure how feasible this is, just something to think about.
Anyway hope this helps!
Cheers!

we could implement this, similar to the flag we added earlier to enforce the presence of all keywords, it just requires constructing the query URL with respect to that website's encoding.
I think this would be worth doing.

I have reinstalled to post YAML config which is here. This time I downloaded with wget instead of using copy/paste, and only changed things slightly, so as not to disturb the YAML. Same error though, even with a normal search term. I'm also using VirtualEnv if that makes a difference.
Also, I checked Google, Google jobs, and Indeed, and they all use straightforward link-building when using the dash to filter, but Monster does not have the option to filter even with a gui. Here's an indeed link with a couple filters included as an example: https://www.indeed.com/jobs?q=admin -"Master's Degree" -Executive&vjk=467cc14b42dafd01
+1 for implementing, because I consider it a required feature when it comes to job scraping. It can really help save time when your normal search term brings up a lot of listings.
Thanks for the help! No pressure on this issue, as I'm getting a lot more hits on Google Jobs!

Marking duplicate #80


ryujinx: 1.0.6416 -> 1.0.6448

Motivation for this change
This adds support for the soundio audio backend, and disables the auto updater which does not work on nix.
The reason for removing it instead of simply ignoring it, is that by default it shows a popup asking if you want to update which is quite annoying.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

/marvin opt-in
/status needs_reviewer


fix: Removing database support of Employee Scheduling

Description of the change
This PR removes the database support for the Employee Scheduling quick start.

Checklist
Development

[x] The changes have been covered with tests, if necessary.
[x] You have a green build, with the exception of the flaky tests.
[x] UI and JS files are fully tested, the user interface works for all modules affected by your changes (e.g., solve and analyze buttons).
[x] The network calls work for all modules affected by your changes (e.g., solving a problem).
[x] The console messages are validated for all modules affected by your changes.

Code Review

[x] This pull request includes an explanatory title and description.
[x] The GitHub issue is linked.
[ ] At least one other engineer has approved the changes.
[ ] After PR is merged, inform the reporter.

Something is wrong with the constraints. See that the score is 0/0, yet there are clearly broken constraints.

Good catch! When using JPA, the persistence context ensures that there will be single instances of Employee, and they will be equal to Joiners.equal(Shift::getEmployee, Availability::getEmployee). I had to add the equals and hashCode methods to compare the instances properly.


[Bug] Getting non-specific error when trying to run InternVL2/WSL2

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.

Describe the bug
I'm trying to run InternVL2 and making no progress even getting a small amount of code to run without error (5 lines to test the qual- ity of the model).  I am in fact running it in WSL2 which may or may not be a problem.  I seem to recall that Triton doesn't run on Windows, even WSL2 and although it's giving me a Triton version I wonder if this may be a silent issue.
Environs: Linux kernel 5.15, Nvidia 4090, Ubuntu 2204.  Cuda 12.x , nvcc 12.6, FA installed via pip.
The first question I have is- where is the Chinese writing coming from?  My prompt literally says "describe the image".  And then a preamble pre-injected that roughly translates to "You are a scholar multimodal model developed by Shanghai Artificial Intelligence Laboratory and SenseTime Technology.".
2024-08-19 02:36:30,594 - lmdeploy - INFO - ImageEncoder done 1 images, left 0 images.
2024-08-19 02:36:30,629 - lmdeploy - INFO - prompt='<|im_start|>system\n你是由上海人工智能实验室联合商汤科技开发的书生多模态大模型，英文 名叫InternVL, 是一个有用无害的人工智能助手。<|im_end|>\n<|im_start|>user\n<IMAGE_TOKEN>\ndescribe this image<|im_end|>\n<|im_start|>assistant\n', gen_config=EngineGenerationConfig(n=1, max_new_tokens=512, top_p=0.8, top_k=40, temperature=0.8, repetition_penalty=1.0, ignore_eos=False, random_seed=10246247738811414702, stop_words=[6, 7], bad_words=None, min_new_tokens=None, skip_special_tokens=True, logprobs=None), prompt_token_id=[6, 1328, 144, 51943, 13326, 4510, 13992, 15290, 5777, 59977, 61094, 3540, 4419, 29361, 59661, 59691, 60131, 60106, 59647, 11443, 101, 59568, 14877, 30347, 3187, 1318, 53581, 97, 141, 4748, 32574, 59828, 60323, 53892, 4740, 44307, 102, 7, 144, 6, 2942, 144, 68, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 etc
then:
44, 6, 14135, 144], adapter_name=None.
2024-08-19 02:36:30,630 - lmdeploy - INFO - session_id=0, history_tokens=0, input_tokens=3381, max_new_tokens=512, seq_start=True, seq_end=True, step=0, prep=True
2024-08-19 02:36:30,634 - lmdeploy - INFO - Register stream callback for 0
[TM][INFO] [forward] Enqueue requests
[TM][INFO] [forward] Wait for requests to complete ...
[TM][WARNING] [RejectInvalidRequests] Skipping invalid infer request for id 0, code = 6
[TM][WARNING] [forward] Request failed for 0, code 6
2024-08-19 02:36:30,717 - lmdeploy - INFO - UN-register stream callback for 0
The program exits out with no output describing the image.
Reproduction
python test.py
code:
from lmdeploy import pipeline, TurbomindEngineConfig
from lmdeploy.vl import load_image
model = 'OpenGVLab/InternVL2-40B-AWQ'
image = load_image('https://some.jpg.img.url.com/image.jpg')
backend_config = TurbomindEngineConfig(model_format='awq')
pipe = pipeline(model, backend_config=backend_config, log_level='DEBUG')
response = pipe(('describe this image', image))
print(response.text)
Environment
Model : OpenGVLab/InternVL2-40B-AWQ
Pytorch was probably installed via pip

sys.platform: linux
Python: 3.10.12 (main, Jul 29 2024, 16:56:48) [GCC 11.4.0]
CUDA available: True
MUSA available: False
numpy_random_seed:<PHONE_NUMBER>
GPU 0: NVIDIA GeForce RTX 4090
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 12.6, V12.6.20
GCC: x86_64-linux-gnu-gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
PyTorch: 2.3.1+cu121
PyTorch compiling details: PyTorch built with:
  - GCC 9.3
  - C++ Version: 201703
  - Intel(R) oneAPI Math Kernel Library Version 2022.2-Product Build 20220804 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v3.3.6 (Git Hash 86e6af5974177e513fd3fee58425e1063e7f1361)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 12.1
  - NVCC architecture flags: -gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_90,code=sm_90
  - CuDNN 8.9.2
  - Magma 2.6.1
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=12.1, CUDNN_VERSION=8.9.2, CXX_COMPILER=/opt/rh/devtoolset-9/root/usr/bin/c++, CXX_FLAGS= -D_GLIBCXX_USE_CXX11_ABI=0 -fabi-version=11 -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -DNDEBUG -DUSE_KINETO -DLIBKINETO_NOROCTRACER -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wall -Wextra -Werror=return-type -Werror=non-virtual-dtor -Werror=bool-operation -Wnarrowing -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-stringop-overflow -Wsuggest-override -Wno-psabi -Wno-error=pedantic -Wno-error=old-style-cast -Wno-missing-braces -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=2.3.1, USE_CUDA=ON, USE_CUDNN=ON, USE_CUSPARSELT=1, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_GLOO=ON, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=1, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF, USE_ROCM_KERNEL_ASSERT=OFF,

TorchVision: 0.18.1+cu121
LMDeploy: 0.5.3+
transformers: 4.44.0
gradio: Not Found
fastapi: 0.112.1
pydantic: 2.8.2
triton: 2.3.1

Error traceback
preamble..
44, 6, 14135, 144], adapter_name=None.
2024-08-19 02:36:30,630 - lmdeploy - INFO - session_id=0, history_tokens=0, input_tokens=3381, max_new_tokens=512, seq_start=True, seq_end=True, step=0, prep=True
2024-08-19 02:36:30,634 - lmdeploy - INFO - Register stream callback for 0
[TM][INFO] [forward] Enqueue requests
[TM][INFO] [forward] Wait for requests to complete ...
**[TM][WARNING] [RejectInvalidRequests] Skipping invalid infer request for id 0, code = 6
[TM][WARNING] [forward] Request failed for 0, code 6**
2024-08-19 02:36:30,717 - lmdeploy - INFO - UN-register stream callback for
+

@pinballelectronica
Looks like you are running out of memory for KV cache blocks. Can you post the full log with INFO log level?

@pinballelectronica
Looks like you are running out of memory for KV cache blocks. Can you post the full log with INFO log level?

Sorry I didn't reply sooner, the response got buried :(
I don't even remember what I was doing at the time but adding more GPU memory should help?  Thank you.


Jest errors out when expecting a mock to be called with a string matcher

Repository: https://repl.it/repls/PleasedSubmissiveAfricanbushviper
function doWork() {
  const dispatch = jest.fn();
  dispatch(null);
  dispatch({});
  dispatch('blah');
  dispatch(() => {});
  return dispatch;
}

test('passes with exact string', () => {
  const dispatch = doWork();
  expect(dispatch).toBeCalledWith('blah');
});

test('fails with matcher', () => {
  const dispatch = doWork();
  expect(dispatch).toBeCalledWith(expect.stringContaining('bla'));
});

Output:
FAIL  ./add-test.js
  ● fails with matcher

    TypeError: Cannot read property 'includes' of null

      at Object.<anonymous>.test (add-test.js:17:20)
          at Promise (<anonymous>)
      at Promise.resolve.then.el (../../usr/local/share/.config/yarn/global/node_modules/p-map/index.js:42:16)
          at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:169:7)

  ✓ passes with exact string (6ms)
  ✕ fails with matcher (2ms)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 passed, 2 total

Jest should skip the values which don't match and only fail if none match. In my example, the second test should pass because expect.stringMatching('bla') passes on the third call with 'blah'. Basically, the same behaviour as asserting with an exact value.
Tested with
Jest v20.0.4 node v7.4.0 linux/amd64 (repl.it)
Jest v21.2.1 node v8.9.1 macOS X 10.13.1 (locally)

@mrm007 thanks for reporting, I opened a PR for it :)


FontAwesome Helper for Alias Details

Adds Helper Modal for easier selection of FontAwesome Icons
Description
Note sure if anyone noticed but the link in the helpBlock span is dead (FontAwesome changed URL Structure).
I never really liked navigating to another website/tab to select an icon anyway, so I wrote a Helper Modal/Function.
Click "PICK", scroll through all the available icons, click on one.

Fixes # (FontAwesome Link changed, and may change again in future)
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x] I did the tests, it does the things

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas

This looks really cool, thanks a lot!
I'll have a closer look at it later.

Think i royally fkd the base branch selection when i submitted this PR and dont know if i can fix it now without redoing it. You can tell i don't hubgit often.
Any suggestions for what i should do there? I've done it on another PR too

We can rebase it, that is not too big of a problem.

We can rebase it, that is not too big of a problem.

sweet! i thought i could just switch the base by editing it from here but then i think the issue is that the rest of the code base is behind develop. it didnt look right so i switched it back

Closing this PR due to expanding on it and making it part of another, better PR


feat(rust): add secret_attributes_get to ffi

I have read the CLA Document and I hereby sign the CLA

recheckcla

I have read the CLA Document and I hereby sign the CLA

recheckcla

recheckcla


[CODEOWNERS] Update Cosmos Service Owners

Summary
Updating the contacts for Cosmos, by request.

//fyi: @ronniegeraghty


🐛 Quotes within quotes of Formatter IR isn't escaped

Environment information
This is running Biomejs playground with no special options enabled.

What happened?

Had a string in Biome.js playground: https://biomejs.dev/playground/?code=YQB3AGEAaQB0ACAAIgBhACIAIAA%3D

await "a"

Double quotes is not escaped or enclosing string is not single quoted (as in Prettier Formatter IR):

This also causes broken syntax highlighting

Expected result
Expected result is either the double quotes surrounding "a" are escaped:
["await \"a\";", hard_line_break]

Or single quotes used for enclosing quotes instead:
['await "a";', hard_line_break]

Code of Conduct

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

@ematipico
Thanks for your hard work! I have a question about this issue!
I'm looking at a lot of implementations right now, and the one that needs to be fixed is https://github.com/biomejs/biome/blob/main/crates/biome_js_formatter/src/js/expressions/await_ In the :expression.rs, am I correct? :eyes:.

@ematipico Thanks for your hard work! I have a question about this issue! I'm looking at a lot of implementations right now, and the one that needs to be fixed is https://github.com/biomejs/biome/blob/main/crates/biome_js_formatter/src/js/expressions/await_ In the :expression.rs, am I correct? 👀.

The bug is around how the formatter IR is stringified in the playground regardless of what the inputted code is. I think this will be somewhere in the playground packages, so you can start by bundling that and looking at how the WASM libraries run the actual Rust code. I went along that a bit and found the https://github.com/biomejs/biome/blob/75dacbb757d99b2420c7a44152ebcd32094845df/crates/biome_wasm/src/lib.rs#L98 function to be related.

@Yash-Singh1
Thanks for the advice!
I expected that there must be a further conversion process in getFormatterIr, so I went deeper and found the format_node, is this related...?
https://github.com/biomejs/biome/blob/75dacbb757d99b2420c7a44152ebcd32094845df/crates/biome_formatter/src/lib.rs#L1062-L1132

Hi @Yash-Singh1 , apologies for let you stranded with this PR.
The issue isn't related to the playground, it's related to our internal formatter.
First, head to crates/biome_js_formatter/tests/quick_test.rs and run the quick_test test. You might need to remove the #[ignore] macro and add the -- --show-output argument if you plan to run test from the CLI. If you use a IntelliJ IDE, you just run it from the UI and it should work.
Inside src put the code await "a" and the test should print the IR and the formatted code. You will notice that IR isn't correct.
How to debug it? Use the playground and inspect the CST. You will notice that it's a JsAwaitExpression. Then go in biome_js_formatter/src/js and look for JsAwaitExpression and eventually you will find the actual implementation, which is here:
This is where we format the
It's possible that we aren't handled string literals correctly.

Also, the expected result should be the same as prettier, so use prettier's IR as solution

@ematipico
Sorry, I'm sorry for all the explanations, but I felt I couldn't solve this problem myself...
I am very sorry, but I would like someone else to tackle this problem :pray:

@ematipico (sorry for pinging, this is not urgent)
I have taken a look at the issue, and if there seems to be a solution I would like to try to implement it.
First of all, I think this would be a reasonable testcase to try to make pass:
await "a";
await 'a';
await "'a'";
await '"a"';
f("a");
f('a');
f("'a'");
f('"a"');

https://biomejs.dev/playground/?code=YQB3AGEAaQB0ACAAIgBhACIAOwAKAGEAdwBhAGkAdAAgACcAYQAnADsACgBhAHcAYQBpAHQAIAAiACcAYQAnACIAOwAKAGEAdwBhAGkAdAAgACcAIgBhACIAJwA7AAoAZgAoACIAYQAiACkAOwAKAGYAKAAnAGEAJwApADsACgBmACgAIgAnAGEAJwAiACkAOwAKAGYAKAAnACIAYQAiACcAKQA7AA%3D%3D
There are two issues as you have mentioned:

String literals aren't formatted correctly
await literals do not emit the same IR as prettier.

The second issue should be (I'm not sure) easier to solve, but the first issue isn't.
I have taken a look at the code and since the code formats correctly, I believe that the issue lies in the trait implementation of std::fmt::Display of Document. Digging deeper, I think this part of the code looks suspicious.
https://github.com/biomejs/biome/blob/789793506040e1018cdbeb5f27875e236266b192/crates/biome_formatter/src/format_element/document.rs#L237-L256
It seems that we are adding StaticText("\""), LocatedTokenText("\"a\""), StaticText("\"") to the buffer, and it is stored in a VecBuffer through the call to biome_formatter::format! which delegates to biome_formatter::format. We have then lost the context that "a" is nested in double quotes.
I am not so sure what we can do in order to fix this problem. Some potential solutions I can think of are

Introducing a new FormatElement::String variant that wraps a Vec<FormatElement> or Box<[FormatElement]> that represents strings wrapped in strings. This probably is not the solution as it would not be used outside IRs and also it would make invalid state like FormatElement::String(Box::new([Tag(...)])) representable.
Escape each text in FormatElement::StaticText | FormatElement::DynamicText | FormatElement::LocatedTokenText before passing it to write_element. This encodes the knowledge that these tokens are in text before passing them to the formatter, but I am not sure if this is correct.

Please kindly advise if there are any ideas, or feel free to take over the implementation.

Thank you @southball for looking into this.
This is indeed a tricky issue, and you did a great job to narrow down the culprit!
Between the two solutions, my gut tells me to attempt the second solution, mostly because adding a new element to the IR might have more side-effects (memory, performance, etc.)

Thank you!
I may have time to implement this at the end of this month / at the beginning of next month, so if anyone wants to send a PR please feel free to do so! Otherwise I will send a PR as soon as I can get to it!

I'd like to give this a shot.


2023.9.1-123: Scroll icon along with long text

https://github.com/lubeda/EspHoMaTriXv2/issues/123

#123

I tried it and it's exactly what I wanted!
It works very well
Thank you!

i`ll take look at the PRs in the next days

Found a little bug.
If I send the text witch in too long to fit in screen with icon, but at the same time too short for scrolling (<32px),
then icon will partially extend beyond the screen, but the screen will not scroll.
Only half of the icon will be visible and there will be no scrolling.

I'll take a look tomorrow.

Made the changes, will test today, if it works I'll bring it back from draft state.

Looks like it's fixed.

Looks like it's fixed.

Just tested this one, and its looks like fixed! Thank you
LGTM ;)


Remove Python 2

Since Python 2 is formally unsupported by the Python Software Foundation, and Python 3 has been available in Cloud Shell for a long time, we plan to remove Python 2 support in an upcoming update. The exact time for removal and process for warning users is TBD. Will update this issue with details as we progress.

Cloud Shell now has Python 3 support.


Add alokmtri-g as codeowner for select features

@alokmtri-g

Pull Request Functional Test Report for #3489 / 09e78896d1fcef81881f9e779f88b067a403f3bd
No tests identified for validation.
Help

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

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 55.268%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
1983

Relevant Lines:
3588

💛  - Coveralls

Also FYI @amrindrr,  added you for feature/platform folder


Shuffle data

Add default shuffle (either to dataset or .fitDataset); if the data is not shuffled we saw some strange oscillatory behavior during early training.

Using the native shuffle method of Dataset objects does not solve the accuracy issue encountered when training the "simple face" task. Instead a custom shuffle function instead seems to solve the issue.

dusplicate issue

you might need to keep the number of local steps (per round) quite small.
and/or make sure each class is available on each of the two clients say
On Mon, Apr 25, 2022, 20:58 Hugo @.***> wrote:

Using the native shuffle method of Dataset objects does not solve the
accuracy issue encountered when training the "simple face" task. Instead a
custom shuffle function instead seems to solve the issue.
—
Reply to this email directly, view it on GitHub
https://github.com/epfml/disco/issues/288#issuecomment-1108926924, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIMRZTWAAJU2GX3VMURNDVG3TNFANCNFSM5UCZGCCQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Document/blog post demystifying the SET for Rekor entries

Including:

Use cases
Payload (as described here)
How to correlate SET with <--> UUID of entry

Basically a post walking someone through this for a Rekor entry.
Slack convo ref

Resources:

https://docs.sigstore.dev/cosign/verify/#verify-a-signature-was-added-to-the-transparency-log
https://github.com/sigstore/cosign/blob/main/specs/SIGNATURE_SPEC.md

We should also document what the SET is not, a proof of an entry being in the log. This has come up a few times that the SET is sufficient for offline verification, because certificate transparency has the same structure (an SCT). Trusting an SET without ever querying the log requires trust in the log, which is antithetical to the design of transparency logs being auditable.

Hello @lkatalin @haydentherapper I would like to work on this. I did a bit of research and looked into the code, this is the information I currently know. Please could you let me know if I'm on the right path, so I can get started writing a walkthrough. I would also love more information on the correlation between UUID's and SET's other than for verification. Thanks.
The SignedEntryTimestamp (SET) can be thought of as a receipt, that is gotten back from Rekor when an entry has been included into the log. It is a signed inclusion promise provided by the transparency log and can act as a trusted guarantee by the log that the entry has been added to it.
For verification, Cosign queries Rekor to compare the public key attached to the generated certificate (from Fulcio) and checks the Timestamp on the signature against the entry in the transparency log. The signature is valid if its timestamp falls within the small window of time that the key pair and certificate issued by the certificate authority were valid
RekorBundle holds metadata about recording a Signature's ephemeral key to a Rekor transparency log.
type RekorBundle struct { 	SignedEntryTimestamp [ ]byte 	Payload RekorPayload }
The Payload consists of all fields required to verify the SignedEntryTimestamp (SET): The body of the Rekor Log Entry, the integratedTime, the logIndex and the logID.

Hi @EstherPhebe , thanks for working on this. I will have to dig into this a bit more before I can be helpful, so thanks for your patience.


is it possible to run shell script on docker stop?

lets say I want to do database backup when disposing of the container, is it possible to run shell script on docker stop or docker-compose down ?

I've run into the same problem and have just submitted a PR for it:
https://github.com/phusion/baseimage-docker/pull/329

fantastic, thanks!

I started making an updated fork of this project, containing my fix for this.
Have a look here:
https://github.com/hyperknot/baseimage16

@hyperknot Would you mind kindly submitting a PR for your changes? I see you had one previously. Thank you!

I'll be merging this one in after #400.

This has actually been fixed in 0.9.22 via #402.


Allow users to label Scans with chain of Connected Points

Expected Behavior
Users should be able to label their Scans with chain of connected points. Such points will be then assigned to a given tag.
Example task: "Using Connected Points please mark border of the right kidney."
Tasks to do:

[Frontend] New Selection tool that will allow to enter multiple Connected Points,
[Backend] Extend existing structure and allow to add above Connected Points to the Labels.

Points should be draggable! Everyone's not perfect :(
Connected Points should allow to create a loop (somehow)!
Proposal for JSON representation in the API (to be discussed):
{
  "points": [{
    "x": 0.5,
    "y": 0.5
  }, {
    "x": 0.5,
    "y": 0.2
  }, {
    "x": 0.1,
    "y": 0.5
  }],
  "slice_index": 0,
  "tag": "SPINE_CENTER",
  "tool": "CONNECTED_POINT",
  "loop": true
}

Proposal for UI design (to be discussed):

Closed by mistake by #330.


Add license files and trivial readme

@fairingrey I'll need your signoff to retroactively license your changes so far under MIT OR Apache-2.0.

LGTM, I'm fine with my contributions under MIT/Apache-2.0.


[CS2103T-T17-2] SWEe!

SWEe! is a desktop app for CS2103T students to manage their learning progress mainly through flashcards. It is optimized for CLI users so that frequent tasks can be done faster by typing in commands.

Codecov Report

Merging #56 into master will decrease coverage by 17.98%.
The diff coverage is 65.50%.

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


[FLINK-30792][state/changelog] make state changelog pre-emptively upl…

…oad not persisted older data
What is the purpose of the change
make state changelog pre-emptively upload not persisted older data, discussed in FLINK-30792
Verifying this change
This change added tests and can be verified as follows:

FsStateChangelogWriterTest#testPreEmptivelyUpload

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes, checkpointing)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@zoltar9264 Thanks for the PR, overall it LGTM. I left some minor comments about the test, PTAL, thanks.

Thanks for the update, CI test failed because of https://issues.apache.org/jira/browse/FLINK-30972, maybe we can rebase this into PR.


Option to disable connect/disconnect notifications

This information is usually conveyed by both the browser action icon and the blue halo around the active text field. Personally, I find the the notifications to be a bit intrusive and annoying.

That’s a good idea. For now you can use Stylish or Adblock to target and hide the element because I don’t know when I’ll have time to add the options.

Fair enough.

I've blocked the the element with uBlock Origin, thanks for the tip, but the bigger issue remains - the fact that notification appears at all also makes the browser become the front app when I close the tab in the editor, for example, which is unwanted

I've added this option if you'd like to help testing it by manually loading the extension from this folder
https://github.com/eugenesvk/GhostText/tree/tmp/distribution


IG updates identified when defining relationships

The following IG updates were identified when defining relationships:

describe encounter-timing relationship
describe activity-timeline relationship
describe scheduledactivityInstance-timeline relationship
describe ordering of eligibility criteria

I added the following text to the IG:
Regarding encounter timing relationship in section Study Timing:
A timing may be referenced from an Encounter using the scheduleAt attribute allowing for a specific encounter timing and corresponding windowing to be defined and presented in a scheduled of activities. An Encounter timing might potentially overarch multiple scheduledInstances representing different blocks of activities within an encounter.
Regarding the activity timeline (also added scheduledTimeline to Activity UML):
The Activity class can be linked to 1 or more procedures (see Section 4.12), 1 or more biomedical concepts (see Section 4.13), 1 or more groups of biomedical concepts, 1 or more surrogate biomedical concepts and/or a sub timeline. A sub timeline referred to from an activity would typically be a sequency of actions covered by the activity description (e.g. blood glucose profiles, sitting/standing vital signs sequences etc.).
Regarding the scheduledActivityInstance-timeline relationship => I think this was already addressed by the diagrams and following sentence in the Study Timing section:
The timeline can also be attached to a ScheduledActivityInstance from another timeline using the timeline reference, thus allowing timepoints within a visit to be constructed, as shown in the following figure.
Regarding the ordering of eligibility criteria:
Eligibility criteria inherit from the Syntax Template class, allowing for referencing any item stored in the USDM, such as assessments stored as BCs or an indication stored in the Indication class. They are defined within a study version which allows reuse within different study designs and different cohorts. The previous and next attributes define the presentation ordering within an eligibility criterion category or overall. The identifier attribute may be used to store the short name used for mapping to SDTM TI domains (see Creation of SDTM Trial Design Domains).


Develop

Added Documentation around Rake tasks, Processes, Appium and POM. Also some minor cleanup of project files.

Approve pull request


:bug: Fix: Change contract import of generated file to import type {Contract}

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1382  👈
#1380 : 1 other dependent PR (#1381 )
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


fix: incorrect stripe table join

What kind of change does this PR introduce?
This PR is to fix #113 . It will keep remote scan result and add implementation for re_scan() callback to reset iteration index when in nested loop join.
What is the current behavior?
If the query joins 2 Stripe foreign tables, the result data isn't correct.
What is the new behavior?
Stripe foreign table join should work correctly.
Additional context
We may need to implement the same re_scan() callback for other FDWs to support join on foreign tables, but this PR only deals with Stripe FDW for now.

@burmecia this is still not working in the supabase sql editor.
This returns 1 result:
select * from stripe.customers c where c.email = 'xxx'
This returns 5 results:
select * from stripe.payment_intents where customer = 'xxx'
However, this query returns only 1 result instead of 5 and payment intents is always null
select * from stripe.customers c left join stripe.payment_intents p on (p.customer = c.id) where c.email = 'xxxx'

can you try use customer.id as the filter instead of email?
select * from stripe.customers c left join stripe.payment_intents p on (p.customer = c.id) where c.id = 'cus_xxxx'


Allow Event Listeners to track applied events

Currently the AppliedEventsLogRepository is the only authority that tracks at what position of an event stream an Event Listener is.
For listeners/projectors that store their state in the database, this is a good default. But for other scenarios (elastic search projector, file based states, ...) listeners should be allowed to provide custom backends.

Let's please try to somehow propagate the system to store the position of an EventListener directly together with it's state, rather than further abstract those two apart.
The single one reason to always do this: As long as your single state write/update is atomic (which it mostly is), your eventlistener immediately gains exactly-once processing without additional complexity!
If you store the position in another storage, you need to wrap the state and the position storage within a transaction - which isn't easily possible unless both are within a single ACID DB. Have one stored on A and the other on B and it all breaks loose, unless you can somehow wrap both inside a costly 2PC protocol - which most of the time you can't because you don't have access to the necessary lowlevel primitives.
See https://event-storage.js.org/#exactly-once-semantics
This can for example be achieved either by abstracting the state storage into a very basic k/v logic that wraps user data with the position (like I did above), or by having the tupel (position, data) abstracted and leaving storing that to the underlying single storage implementation.

Let's please try to [...] store the position of an EventListener directly together with it's state

That's exactly the purpose of this PR:
By default your projections / listeners uses the same DB connection as the applied events storage, but if the EL state is based on a different storage (e.g. elastic, file based, different db, ..) you should be able to use that to store the position.
Or what exactly do you suggest to change?

btw: "node-event-storage" is a very interesting project, I need to read through it asap!
And just to be clear: I totally agree that the current implementation is not very straight forward right now.. To achieve "exactly once semantics" one would have to do sth like this currently:
final class SomeEventListener implements EventListenerInterface, ProvidesAppliedEventsStorageInterface, BeforeInvokeInterface, AfterInvokeInterface
{
    public function beforeInvoke(EventEnvelope $eventEnvelope): void
    {
        $this->storage->startTransaction();
    }

    public function getAppliedEventsStorage(): AppliedEventsStorageInterface
    {
        return $this->storage;
    }

    public function afterInvoke(EventEnvelope $eventEnvelope): void
    {
        $this->storage->commitTransaction();
    }
    // ...
}

Or what exactly do you suggest to change?

Nothing from that point of view, but more from the developer-facing API.

one would have to do sth like this currently

This (beforeInvoke, afterInvoke and return the storage connection) makes a lot of sense for an RDBMS storage - maybe missing a clearer API on where to place the rollback. But for simple cases, this overcomplicates things a lot. Simplest case: How should things look if I just want to project a couple of events into a single JSON file (or CSV, or ...)?
Is it easily possible to store position + data within this single file with one write, when I need to separate the position from saving the state (because one is supposed to happen manually inside the handle method, the other explicitly through another method)? If I just naively implement the AppliedEventsStorageInterface, I would end up having two write operations for state and position. In order to make that work, I would have to keep an internal UnitOfWork within my file writing abstraction layer and defer the actual write until the saveHighestApplied....
What I (optimally) want though is roughly the following:
interface ProjectionStorageInterface
{
    protected function getStateAndPosition(string $key): StateAndPosition;
    protected function setStateAndPosition(string $key, StateAndPosition $value, int $expectedPosition = 0);
}

StateAndPosition would just consist of a mixed state and int position. Then inside the EventListener:
function initialize()
{
    $stateAndPosition = $this->storage->getStateAndPosition($this->identifier);
    $this->state = $stateAndPosition->state;
    $this->expectedVersion = $this->position = $stateAndPosition->position;
}

function flush()
{
    $this->storage->setStateAndPosition($this->identifier, $this->state, $this->position, $this->expectedVersion); // expectedPosition to handle the update with optimistic concurrency if supported
    $this->expectedVersion = $this->position;
}

function getPosition()
{
    return $this->position;
}

function whenFoo($event, (...), $newPosition)
{
    ... // adjust $this->state

    $this->position = $newPosition;
}

EventListenerInvoker:
$eventListener->initialize();
$position = $eventListener->getPosition();
foreach ($events as $event) {
    if ($position > $event->sequenceNumber) {
      // Idempotency case: we already handled that event, if < we have a missing event and things are messed up
      return;
    }
   $eventListener->$handleMethod($event, ..., $eventSequence);
}
$eventListener->flush();

Notice how this one flow works both for handling new incoming events and catching up - just where the list of events come from is different. For a replay, you do a reset and catchUp.
Now of course, the implementation of the setStateAndPosition($data, $position, $expectedPosition) is not trivial for a file storage, as it needs to be atomic. But how this is handled is now fully independent of the event listener process - for the RDBMS you would just do the whole transaction (SELECT lastVersion, check against $expected, INSERT/UPDATE) roundabout within that method.
Optimally, the when* handler method should not even call the persistence method, but just project it's state in memory, then the persist call is done separately as above. That way, for the catchUp it hence greatly increases the performance of that process. Catching up a thousand events inmemory is pretty fast, but when having to do a DB roundtrip or file write for every single one makes that slow of course, while it's only really needed for the last one or maybe every couple hundred. It's not necessary that the catchUp process is persistent in every single step.

@albe Thanks a lot for the elaborate feedback once again :)
Storing the position directly in the when* methods would be possible already, because they can have access to the "raw event":
public function whenFoo(Foo $event, RawEvent $rawEvent): void
{
    // persist new state including $rawEvent->getSequenceNumber();
}

But it would be quite tedious to do so in all handler methods.
Alternatively, with the suggested PR (and the last commit 72fe75ffbd5f4c105345e674c1b654389c21f963), you could do this:
final class SomeEventListener implements EventListenerInterface, AppliedEventsStorageInterface
{

    public function reserveHighestAppliedEventSequenceNumber(): int
    {
        // fetch last applied sequence nr, start transaction
    }

    public function releaseHighestAppliedSequenceNumber(): void
    {
        // release lock on sequence nr (if any)
    }

    public function whenFoo(Foo $event): void
    {
       // change state (not yet persisted)
    }

    public function saveHighestAppliedSequenceNumber(int $sequenceNumber): void
    {
       // commit transaction
    }
}

I don't think that's too bad actually.
Tested example with file storage (and flock): https://gist.github.com/bwaidelich/b83a030b35e78b29408c8230c57782dc

@albe we merged this (after Sebastian looked over it as well) for now. The plan is to come up with an initial 2.0 release as soon as possible and this was (kind of) the last missing feature that I really wanted to have in there.
If you have suggestions how to change this for 3.x, I'd be happy to discuss that with you again. We plan to work on 3.0 right after the 2.0.0 release


Add configurable API Timeouts

Multiple limitations exist with existing code:

Human operator cannot configure Apache Timeout, so it's fixed at the defaults of 60 seconds.
This is not a big issue since there's only 1 sync operation, but for consistency we adopt the same approach as Cinder and Glance.

Default HAProxy timeout is 30 seconds, which is different than the default in OSP17 (60 seconds)

Human operators would need to sync values between 3 places: HAProxy, Apache, oslo.messaging sync RPC (call).

This patch adds an apiTimeout field to the ManilaSpecBase to allow human operators to simultaneously configure the timeouts for HAProxy, Apache, and the rpc_response_timeout.
The apiTimeout defaults to 60 seconds, to mimic the behavior present in OSP17.
Having different timeouts for HAProxy, Apache, and rpc_response_timeout is possible setting the HAProxy timeout in the apiOverride, the Apache timeout with apiTimeout, and setting rpc_response_timeout in the top level Manila customServiceConfig.
To be able to change the HAProxy value based on the apiTimeout with any update (and not just the first time) the code adds a custom annotation "api.manila.openstack.org/timeout" with the value that was initially set, this way flags it as being set by the manila-operator.

/test manila-operator-build-deploy-kuttl


Use Spring Cloud Stream Parent directly

Since the intent for spring-cloud-stream-dependencies to
become the user-facing dependency management BOM, binder
implementations should switch to the Spring Cloud Stream
parent for version management.
This also ensures that they inherit the build lifecycle of
Spring Cloud Stream - e.g. checkstyle validation.

LGTM - merged.


CDN services do not have the 1.8 version

Documentation refers to using CDN services as detailed below:
CSS
<link rel="stylesheet" type="text/css" href="//cdn.jsdelivr.net/jquery.slick/1.8.0/slick.css"/>
JS
<script type="text/javascript" src="//cdn.jsdelivr.net/jquery.slick/1.8.0/slick.min.js"></script>
However, the 1.8 versions are not available on jsdeliver.net nor cdnjs.com/libraries/slick-carousel.
Thanks.

There are working 1.8.0 links in the readme. jsDelivr website (and cdnjs too, I believe) doesn't list 1.8.0 because it isn't on npm. Related: https://github.com/kenwheeler/slick/pull/3093#issuecomment-328850045

+1. Please fix up the docs!

On https://cdnjs.com/libraries/slick-carousel the latest version is 1.7.1. Please upload the 1.8.0 version.

Published, should propagate soon

"Couldn't find the requested release version 1.8.0." Appearing as of 21 days after this issue was closed.

@EvilJordan what file/url were you trying to access?

try: https://unpkg.com/slick-carousel

@kukac7 https://cdn.jsdelivr.net/npm/slick-carousel works too...

Direct from the website "Get it now" section (http://kenwheeler.github.io/slick/#go-get-it), neither of these work:
cdn.jsdelivr.net/jquery.slick/1.8.0/slick.css
cdn.jsdelivr.net/jquery.slick/1.8.0/slick.min.js

@EvilJordan thanks, I'll send a PR to fix the website. The correct links are:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I'll send a PR to fix the website

Or not, as I can't find the correct repo. @kenwheeler ?


Create aryacollege

Created pull request for Arya College of Engineering and I.T.

@pchaigno check it now


Quantum Card with enabled restock crashes server

With installed singularity to wt and Restock enabled (btw, "No network" message appears).

Build Quantum Bridge
Insert Quantum Entangled Singularity
Try to open Quantum Bridge

OR install singularity without restock and after enabling:

Rebuild Quantum Bridge
Try to insert Quantum Entangled Singularity

OR install singularity without restock and after enabling:

Open Terminal
Close terminal

Crash reports: https://gist.github.com/dmytrohoi/47ebb9c84ba6392640c552cc32ee9f6e

(btw, "No network" message appears)

that is a known issue (#221)

I can't reproduce this.
are you sure it is related to ae2wtlib?
the crashlog seems more like an issue with Applied Mekanistics, specifically an import bus importing some chemicals

It's occurs only when WT restock enabled...

This issue has been duplicated in Applied Mekanistics repo - https://github.com/AppliedEnergistics/Applied-Mekanistics/issues/45


Look into optimizing the token size calculation for the context_window

I feel like I cut some corners on how the token size is calculated for the context_window.

Unfortunately there is still no Go SDK. Let's stick with the current estimation.


fix(v2): fix recent baseurl issues

Motivation
Some things don't currently work well with a baseUrl due to recent changes
This fixes the new issues + makes the Netlify CI build with baseurl

How do I see the preview with the baseUrl?


cocomac: Blacklist +1<PHONE_NUMBER>

cocomac requests the blacklist of the number +1<PHONE_NUMBER>. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


Fix parsing of unicode escape sequences in YAML

A YAML scalar value can contain unicode escape sequences like \u0051. Currently, this throws off the parser.

@nmck257 In https://github.com/openrewrite/rewrite/pull/2264 you added logic to handle escape sequences. Unfortunately the tests inadvertently got disabled when we switched from Kotlin to Java. In this PR I have re-enabled the tests again, but I had to remove some of the cases, since the escape sequences appear to be invalid (at least according to the SnakeYAML parser).
I was wondering if you could have a look at this PR and make sure that the implementation still meets your requirements.

@knutwannheden yep, no concerns. I'll gladly trust the SnakeYaml parser; maybe some patch version over the past ~year helped it catch invalid edge cases which it did not notice before.


Crash

Microsoft PowerToys version
0.66.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
i was typing in powertoys run, had just disabled input smoothing and was testing responsiveness
✔ Expected Behavior
it doesnt crash
❌ Actual Behavior
it crashed after i mashed a bit
error message:
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22621.0
IntPtr Length: 8
x64: True
Date: 7 February 2023 8:25:30pm
Exception:
System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.
   at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

log file: https://pastebin.com/9b2X3iLr
Other Software
Was using everything plugin from https://github.com/lin-ycv/EverythingPowerToys

This is a duplicate of #18166. It should be fixed with the latest release (0.67.1).
/dup #18166

For any users that are able to replicate this consistently, can you please update to 0.68.1 and disable thumbnail generation to verify if it fixes the issue? Please let us know on #18166!


Custom Metadata Feature

Add possibility to define custom metadata at ingestion time.

This will be possible without any additional effort with the upcoming v0.3.0 release.


WIP new file:   backend/examples/crabfeeder.rs

a SMTP client for injecting emails into mailcrab

Marked as Work In Progress because the actual sending has still to be implemented.
Reason for the early release is preventing code duplication.

The examples/crabfeeder.rs considered complete.  ( Label WIP removed. )
This merge request also brings in examples/multiple_rcpt_to.rs (https://github.com/tweedegolf/mailcrab/pull/82/commits/c342fc9c726656da713bb909a7f3e9235e06cc70).

cross reference to #69

We already have e mechanism for sending test messages, as a Rust test case - see https://github.com/tweedegolf/mailcrab/blob/main/backend/src/main.rs#L303
This will send all files in the samples/ directory as an email, interpreting the first two lines as sender en recipient addresses respectively.


URL truncation issue

When url bar autocompletes to a long address, can't make it shorter, whole browser just moves:

This is because of electron/electron#3009.

try brave/electron#13?

That's in Brave's fork of Electron, so they would need to upstream it into the main Electron repository for Min to use it.

This has been added to Electron (in electron/electron#5557), so it would be possible to fix this now after upgrading Min's Electron version.

Fixed in 5fd4a53446e3e65bbba58bf64a97524bc198d1c0.


Add publishing to the init plugin

Expected Behavior
It would be great if the init script generator supported at least Maven publishing for Java libraries. That is to say: apply the maven-publish plugin, create a default Maven publication and a "local" repository (in the build directory for example).
Something like:
plugins {
    // Apply the java-library plugin to add support for Java Library
    id 'java-library'

    // Apply the `maven-publish` plugin to add support for publication on Maven repositories
    id 'maven-publish'
}

repositories {
    // Use jcenter for resolving dependencies.
    // You can declare any Maven/Ivy/file repository here.
    jcenter()
}

dependencies {
    // This dependency is exported to consumers, that is to say found on their compile classpath.
    api 'org.apache.commons:commons-math3:3.6.1'

    // This dependency is used internally, and not exposed to consumers on their own compile classpath.
    implementation 'com.google.guava:guava:29.0-jre'

    // Use JUnit test framework
    testImplementation 'junit:junit:4.13'
}

publishing {
   repositories {
       maven {
          // Publishes to a file system repository in the build directory
          url "${buildDir/repo}"
       }
   }
   publications {
       // Publishes the default Java publication
       maven(MavenPublication) {
           from components.java
       }
   }
}

We should make this an option for the "library" ini type.
Then we can replace these samples:

https://docs.gradle.org/current/samples/sample_groovy_library_publishing.html
https://docs.gradle.org/current/samples/sample_java_library_publishing.html

By samples we generate with init and automatically get a "library publishing" sample for all JVM languages.

No, stale bot, don't close this

This should be an important feature, e.g. when oss projects don't publish their code to maven central.
Gradle forums responses are at least 3 years old and don't seem to work with the Kotlin DSL in  gradle 8.5


Fixed validate_packages script to work with test environment

Codecov Report

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

@@           Coverage Diff           @@
##           master     #428   +/-   ##
=======================================
  Coverage   86.04%   86.04%
=======================================
  Files          56       56
  Lines        4429     4429
  Branches      481      481
=======================================
  Hits         3811     3811
  Misses        465      465
  Partials      153      153

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 8d35972...4f6ce7f. Read the comment docs.


#184752235 Setup API documentation using swagger

What does this PR do?
Api documentation using swagger
Description of Task to be completed?
Setup API documentation using swagger,
Have tested working example  :example.docs.yaml
How should this be manually tested?
clone the project add your .yaml file to be tested in docs folder . or simply use provided example to test.
run in browser [http://localhost:{port}/api-docs/
Any background context you want to provide?
This will combine each .yaml documentation file that will be added on project folder [docs] to be displayed swagger docs  homepage of the project
What are the relevant pivotal tracker/Trello stories?
#184752235

well structured


Blog card on homepage not clickable.

Only the blog card title is clickable to reach the blog detail. I would like to make the clickable link extent/stretch to the entire blog card.

We need to reconfigure the blog cards to more standard bootstrap 5 classes.  Currently I have them customized and I believe this is the wrong route to take when we are using bootstrap already. The cards on the News page and the home page could use the same styles to eliminate any unnecessary css.

This issue is related to issue #14.


Feature/cairo cli integration

Changes:

Initial CLI implementation for juno, juno-cli, which integrates commands from the Starknet Feeder Gateway.

Types of changes

[x ] New feature (non-breaking change which adds functionality)

Testing
Requires testing

[ ] Yes
[x ] No

Currently implementation simple enough to compare directly to pure HTTP requests.
Further comments
See idea behind PR here https://www.notion.so/nethermind/Cairo-CLI-Implementation-f524ca0c7f404b8eb151842c5f6d2065
And living docs here

Ok I'm just going to let this be. I have tried "go mod tidy" and forcing it to load modules back in but it's just not working. Would appreciate some guidance when anybody has the time!

Closed in favour of https://github.com/NethermindEth/juno/pull/215


List ranking restructuring

close #150
close #148
loads dynamic text data in CardCollection and looks for similar AGS there as well

error with loading elements


Account deactivation doesnt work properly

When someone deactivates their account by going to /account/settings  it just says Account has been deactivated!
If you refresh the page, everything still loads, it should remove user and redirects to a page, either home or sign in

In the demo site, I have added a flag on deactivation (it doesn't actually delete/deactivate the demo account) it's just for demo purposes. We'll add a note in that case. Good catch! 👍

Closing this as this is not supported in demo :)
If you wish to actively deactivate your account, please don't hesitate to contact.