Can't inject using machobot

When i am trying inject dylib i am getting error:
MacBook-Pro$ inject_dylib ewqeqw "~/Downloads/some.dylib"
Traceback (most recent call last):
File "/usr/local/bin/inject_dylib", line 9, in
load_entry_point('machobot==0.1.2', 'console_scripts', 'inject_dylib')()
File "build/bdist.macosx-10.10-intel/egg/pkg_resources/init.py", line 558, in load_entry_point
File "build/bdist.macosx-10.10-intel/egg/pkg_resources/init.py", line 2682, in load_entry_point
File "build/bdist.macosx-10.10-intel/egg/pkg_resources/init.py", line 2355, in load
File "build/bdist.macosx-10.10-intel/egg/pkg_resources/init.py", line 2361, in resolve
File "/Library/Python/2.7/site-packages/machobot/command_line.py", line 10, in
import machobot.dylib as dylib
File "/Library/Python/2.7/site-packages/machobot/dylib.py", line 15, in
from .common.macho_helpers import modify_macho_file_headers
ImportError: No module named common.macho_helpers
Any help ?

Oooops, that's my fault, sorry! I'll fix it ASAP.
Thanks for letting me know about the problem! :green_heart:

@m1entus please check the new version out and see if the problem is still there.
I think you could just uninstall and install it again with pip:
$ sudo -H pip uninstall machobot
$ sudo -H pip install --no-cache-dir machobot

Thanks!

Thanks, work like a charm!


Queue media sync when media path is set

Another commit motivated by a comment:
// In a next version we should opt for a "MediaPathChanged" event,
// but let's just do this async now.

I've included a test for this and manually tested also. Works fine on my end. Let me know is there are any changes you want me to make.

I really should comment more!

Actually the reason why I didn't queue this task is Koel's web interface must know when the task is done in order to reload. Queuing – with a proper driver, if the user is savvy enough to configure one into .env – will break this logic.
We will come back to this topic when Koel is prepared.


[UT] Test gen -m, --mock FILE

Is your feature request related to a problem? Please describe.
Write UT:
Generate mock of the chosen file. The .h or .hpp file shall be used. It will then generate mock in tests/mock folder with a path mirroring the sources.
Describe the solution you'd like
UT
Additional context
UT of the ut gen - scargo.file_generators.mock_gen

Basic tests implemented


Handle RDS instances without subnet groups

I'm getting an exception during data collection when a RDS instance doesn't have an attached DBSubnetGroup. While this is unlikely, it's possible and the error condition should probably be handled by catching a KeyError here.

Good catch, thanks for letting us know!  Mind pasting the stack trace as well?

Sure
Traceback (most recent call last):
  File "/Users/dgreene/.pyenv/versions/3.6.5/bin/cartography", line 10, in <module>
    sys.exit(main())
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/cli.py", line 180, in main
    return CLI(default_sync, prog='cartography').main(argv)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/cli.py", line 161, in main
    return cartography.sync.run_with_config(self.sync, config)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/sync.py", line 129, in run_with_config
    return sync.run(neo4j_driver, config)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/sync.py", line 63, in run
    stage_func(neo4j_session, config)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/intel/aws/__init__.py", line 119, in start_aws_ingestion
    _sync_multiple_accounts(session, aws_accounts, regions, config.update_tag, common_job_parameters)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/intel/aws/__init__.py", line 60, in _sync_multiple_accounts
    _sync_one_account(session, boto3_session, account_id, regions, sync_tag, common_job_parameters)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/intel/aws/__init__.py", line 38, in _sync_one_account
    rds.sync_rds_instances(session, boto3_session, regions, account_id, sync_tag, common_job_parameters)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/intel/aws/rds.py", line 253, in sync_rds_instances
    load_rds_instances(neo4j_session, data, region, current_aws_account_id, aws_update_tag)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/intel/aws/rds.py", line 109, in load_rds_instances
    _attach_ec2_subnet_groups(neo4j_session, rds, region, current_aws_account_id, aws_update_tag)
  File "/Users/dgreene/.pyenv/versions/3.6.5/lib/python3.6/site-packages/cartography/intel/aws/rds.py", line 131, in _attach_ec2_subnet_groups
    db_sng = instance['DBSubnetGroup']
KeyError: 'DBSubnetGroup'

Gimme 1 sec to get the pre-release out to pypi..

Thanks for the fix!
A suggestion: you guys use dict indexing a whole bunch which is fine except you have to be careful about KeyErrors which I seem to keep running into. I highly recommend using dict.get as it allows for safer access to keys that may or may not exist.
If you're interested I can do a bunch of the legwork and open a PR.

Here we go: https://pypi.org/project/cartography/0.2.3rc1/
On KeyErrors: in general our thinking is that if a dict returned doesn't have a data field we're expecting to have, we're ok with letting the sync fail because it's violated a data contract.  For example, a lot of the code directly accesses an object's ARN using data['SomeARN'] as opposed to using data.get('SomeARN').  This way the sync will fail if an important ARN does not exist.
In the case of the RDS subnet group, I made the incorrect assumption that all RDS instances should have a DBSubnetGroup, so now I'm backtracking and adding that key check :).

Totally understood. I was thinking from the perspective of being a user scanning infrastructure I don't own and so, unfortunately, I'm at the whims of whether my users create their infrastructure in such a way that it fulfills the contract. If they don't, cartography bombs out and I have to start scanning all of our accounts again and we have a lot of them so it takes a long time.
My comment was more around safety when accessing nested objects that potentially create relationships between entities. If the nested object doesn't exist then the relationship doesn't exist and a safe check with an early return might be appropriate.
Ultimately your choice and I appreciate the work you're doing regardless 😀

Got it, please continue to let us know if there are other data assumptions we make that aren't correct. 🙂


Update tests to point to new privacy page URL

Task/Issue URL: https://app.asana.com/0/488551667048375/1205442219109577/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3527   👈

This comment was auto-generated by Graphite.


su path fix for auditd

V-71999: fixed incorrect path for su in auditd config
disabled slow controls in .kitchen.yml
added a nolong attributes file

Signed-off-by: Aaron Lippold<EMAIL_ADDRESS>

This needs the update for the latest simp-beaker-helpers to be released.


StreamWriter to support writing ints, floats, etc. to the cells

The SetRow function of the StreamWriter struct has the row values to be written as interface{}, i.e.:
func (sw *StreamWriter) SetRow(axis string, values []interface{}, opts ...RowOpts) error
Thus one can write any datatypes to the row as long as they are converted to interface{}. I would have expected that when writing an int, float or time value that the SetRow function would write to the cell similarly to writing a cell using the SetCellInt and SetCellFloat functions since the underlying type of the interface would be available. Instead, the values are all written as text in the general format. Thus, numbers will appear as "Number formatted as text" rather than actual numbers for which a number format can be set using the SetColStyle function.
Is there a way to write numbers properly using the StreamWriter or alternatively a fast way to get the texts converted to numbers, or does one instead of the StreamWriter function have to use the SetCell.. functions to get the values stored as proper numbers?
The idea of the StreamWriter is to load data fast to a sheet. However, the need for having proper numbers stored in the sheets is not less for larger datasets -- probably rather the other way around.

Thanks for your issue, as you says, the value paramter in the SetRow function of the StreamWriter is interface{}, so it can be accept mutipal hybrid data type values, and the value will be write in the cell as is original data type. Reference the example of the stream writer.

I was working with a use case where the number and type of columns was not static but decided at runtime. Thus, I packed the values into an interface{} slice and calling SetRow with that slice. However, the values written seemed to be text in the "General" format rather than integers, decimals, etc. The code dealing with a lot of other aspects like formatting, so I may be wrong.
Once I have my project completed, I will work out a simple example to check whether there actually is an issue or not and let  you know.
In the meanwhile, I have just used SetCellValue to write the cells and that works excellent. But I guess for large datasets the stream writer may still be preferable.

Below is some code which demonstrates the problem that I encountered. However, while I originally thought that the data from the interface slice was not written correctly to the cells according to the underlying datatype, I now see that it is formatting using SetColStyle which does not achieve the result that I expected, i.e. it seems that the styles are not applied to the cells in the columns.
The code below creates a new excel file test.xlsx which contains a sheet with five columns each of a different datatype, with three rows. Even though the col styles are applied it does not seem to affect the cells in the columns, i.e. they still have the "General" format. However, I do notice that numbers are right-adjusted, which would indicate that numbers are written correctly as such to Excel.
Go version: 1.17.1 windows/amd64
Excelize version: 2.4.1
Code:
package main

import (
	"fmt"
	"log"
	"os"
	"time"

	"github.com/xuri/excelize/v2"
)

const (
	s_fi = "test.xlsx"
	s_sh = "Sheet1"
)

type data_tp struct {
	fld_str   string
	fld_int   int
	fld_float float64
	fld_bool  bool
	fld_time  time.Time
}

func main() {

	sl_data := []data_tp{
		{"str_val1", 1000, 11000.0, true, time.Now().AddDate(-10, -1, 0)},
		{"str_val2", 2000, 22000.0, true, time.Now().AddDate(-5, -2, 3)},
		{"str_val3", 3000, 33000.0, true, time.Now().AddDate(0, -4, 5)},
	}

	wb := excelize.NewFile()

	// populate the sheet with the stream writer
	sw, err := wb.NewStreamWriter(s_sh)
	if err != nil {
		fmt.Println("Error:", err)
		return
	}
	sl_intf := make([]interface{}, 5)

	for ix, data := range sl_data {
		sl_intf[0] = data.fld_str
		sl_intf[1] = data.fld_int
		sl_intf[2] = data.fld_float
		sl_intf[3] = data.fld_bool
		sl_intf[4] = data.fld_time
		cell, _ := excelize.CoordinatesToCellName(1, ix+1)
		if err := sw.SetRow(cell, sl_intf); err != nil {
			fmt.Printf("Error: %s, row num: %d\n", err, ix+1)
			return
		}
	}
	if err := sw.Flush(); err != nil {
		fmt.Printf("Error flush: %v", err)
		return
	}

	// create styles
	s_sty := ""
	s_sty = "{\"number_format\": 3}"
	i_sty_int, err := wb.NewStyle(s_sty)
	if err != nil {
		fmt.Println("Error:", err)
		return
	}
	s_sty = "{\"number_format\": 4}"
	i_sty_fl, err := wb.NewStyle(s_sty)
	if err != nil {
		fmt.Println("Error:", err)
		return
	}
	s_sty = "{\"custom_number_format\": \"yyyy-mm-dd;@\"}"
	i_sty_dt, err := wb.NewStyle(s_sty)
	if err != nil {
		fmt.Println("Error:", err)
		return
	}
	s_sty = "{\"custom_number_format\": \"@\"}"
	i_sty_gen, err := wb.NewStyle(s_sty)
	if err != nil {
		fmt.Println("Error:", err)
		return
	}

	// apply styles to the columns
	if err = wb.SetColStyle(s_sh, "A", i_sty_gen); err != nil {
		fmt.Println("Error:", err)
		return
	}
	if err = wb.SetColStyle(s_sh, "B", i_sty_int); err != nil {
		fmt.Println("Error:", err)
		return
	}
	if err = wb.SetColStyle(s_sh, "C", i_sty_fl); err != nil {
		fmt.Println("Error:", err)
		return
	}
	if err = wb.SetColStyle(s_sh, "D", i_sty_gen); err != nil {
		fmt.Println("Error:", err)
		return
	}
	if err = wb.SetColStyle(s_sh, "E", i_sty_dt); err != nil {
		fmt.Println("Error:", err)
		return
	}

	if fileExists(s_fi) {
		err := os.Remove(s_fi)
		if err != nil {
			log.Fatal("Failed to remove file:", s_fi, err)
		}
	}
	if err := wb.SaveAs("text.xlsx"); err != nil {
		fmt.Println("Error:", err)
		return
	}
}

func fileExists(s_file string) bool {
	info, err := os.Stat(s_file)
	if os.IsNotExist(err) {
		return false
	}
	if info.IsDir() {
		log.Fatal("Error:", s_file, "is a directory")
	}
	return true
}

Sorry for my late reply. This issue is similar to issues #725, #777, and #826. SetColStyle is not a streaming API. So it won't work with a streaming API. Please don't use common API and stream API mixed, using the wb.SetSheetRow(s_sh, cell, &sl_intf) instead of stream writer. I'll close this issue. If you have any questions, please let me know.


Mathematics equations support

Hi nea,
Thank you so much for your excellent work on this plugin. I have a little question about its usage, how could I write mathematics equations?  "$$" seems not work in MarkdowViewerPP. Thank you!

Hi @SpinoPi
I just released a new version 0.8.0.
I changed the Markdown renderer to Markdig which supports basic math equations like https://github.com/lunet-io/markdig/blob/master/src/Markdig.Tests/Specs/MathSpecs.md.
I hope this helps.
Cheers

Thank you very much for this great work! @nea

If I remember correctly this holds for the HTML output yet the view pane doesn't display Math, right?


Update reports which use CustomFieldValue for Netbox 2.10

Reported at https://github.com/netbox-community/netbox/discussions/5562#discussioncomment-254943

ping?

Hi,
Now we have device.status and I think you can use
for device in Device.objects.filter(status="active"):
instead of this one:
for device in Device.objects.filter(status=DeviceStatusChoices.STATUS_ACTIVE):
And in this case you can remove:
from dcim.choices import DeviceStatusChoices
I've checked this on my Netbox v.2.10.4 and it works.

Now we have device.status and I think you can use
for device in Device.objects.filter(status="active"):
instead of this one:
for device in Device.objects.filter(status=DeviceStatusChoices.STATUS_ACTIVE):

The second is still correct, and preferred.


New private messages?

Very excited to see that TT added private messaging, which adds another (less noisy) way for bots to interact with real people.
http://blog.turntable.fm/post/18010302466/turntable-adds-private-messaging
I was hoping  bot.on('speak') would work, but looks like it doesn't. I hope to see support for it in your API, Alain!

Hey !
It has been implemented !
This is how you should listen for new private messages.
bot.on('pmmed', function (data) { console.log(data); });

This is how you can send a private message.
bot.pm('Your message.', 'RECEIVER_ID');

:)


Day start and end times

The jQuery plugin has an option for setting the start time and end time. Any plans to support all hours in a day? Thanks!

Sure thing, I'll add that in over the weekend :)

https://github.com/mattlewis92/angular-bootstrap-calendar/pull/17

The start and end time of the day view is now configurable.


The node is missing and cannot be loaded properly when I install AnimateDiff Evolved(the another t2video)

I have both versions installed and all nodes are loading correctly. Could you provide some information to aid in debugging?

Are you using the latest versions of both extensions?
Have you encountered any error logs in the console?
Which ComfyUI commit are you currently using? (You can use this command git rev-parse --verify HEAD)

awesome work！！I installed the latest version，and everything's running nice


Extension causes high cpu load

Issue Type: Performance
Extension Name: al
Extension Version: 6.6.434101
OS Version: Windows_NT x64 10.0.19042
VSCode version: 1.54.3

:warning: Make sure to attach this file from your home-directory:
:warning:c:\Users\jhayp\AppData\Local\Temp\ms-dynamics-smb.al-unresponsive.cpuprofile.txt
Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

Hi! Could you open the issue following the template and provide the requested information?


Separate units package

In some recent threads (https://github.com/openmm/openmm/issues/2997, https://github.com/openmm/openmm/issues/3002, likely many others) the idea of having a separate simtk-units/openmm-units package was proposed and met with favorable responses. An obvious use case, if somewhat niche, is installing the units module without the rest of the OpenMM API and compute layers. I've actually run across this recently in a toy repo, so I would benefit somewhat from this.
I am happy to contribute to or do the implementation of this if provided some guidance. I don't know if refactoring the current recipe to use outputs: makes sense, or if simply contributing a new package to staged-recipes is a better approach.
P.S.  I'm only concerned with the conda packaging here, but it's worth pointing out that that is an ancient PyPI project that is several years out of maintenance that could provide some confusion if people haven't forgotten about it and happen to try using pip to install it: https://pypi.org/project/simtk.unit/

outputs would be the way to go if the source is going to stay in the same repo. I'd wait until next major release, where simtk->openmm is done, to start the split packaging, though.

It might be useful to convene a brief developers chat with @peastman as well to see if there might be a good path to adding interoperability support for standard unit systems like pint as well, so that we can at least support pint as well as simtk.unit units.


Changing some docs for RC3

Some of the docs had outdated syntax so that is now updated. Added the
 to the csproj reference. More work is needed to
get the docs up to date completely especially given the new CRUD
commands.

Also added the dotnet clean command. I plan to maybe add other commands as well, such as CRUD commands.

@blackdwarf It occurs to me that one of the things missing from the documentation for the XML elements is their parent element, if they are nested.

@rpetrusha well, in MSBuild parlance, most of these are either properties or items, so by saying that we say that their parent element is either <PropertyGroup> or <ItemGroup>.

@blackdwarf But I'm not sure that customers who are beginning to use MSBuild-based project files for the first time will understand that. It just seems like a basic reference item that should be present.

@rpetrusha I'm pretty sure that this is not the place to educate them on that. There is a link at the top of the article which points them to the general docs about how MSBuild works.

@mairaw is this good to go?

You've addressed my last comment. ms.date should be updated to reflect the docs have been updated as well.

@mairaw all things should be good now, providing the build passes. I've also rebased the PR to the latest master.


Fix shell format warning in library.sh

As per title, this patch fixes shell format warning in library.sh.
It was discovered by https://github.com/knative/hack/pull/297/files but didn't fix because it didn't touch all of them.

/cc @kvmware @upodroid

Closing as the warning is produced forever and one of the reviewdog's suggestion:
[shellcheck] reported by reviewdog 🐶

Raw Output:
library.sh:286:-        local status=(`echo -n ${pod} | cut -f2 -d' ' | tr '/' ' '`)
library.sh:286:+        local status=($(echo -n "${pod}" | cut -f2 -d' ' | tr '/' ' '))

I changed it as https://github.com/knative/hack/pull/298/commits/83541465ff57a3db6481defc1f77b4245bdeb162 then integration test started failing...

You can configure shellcheck to ignore a specific line.
# shellcheck disable=SC2059 Replace it with the correct id


Services created in Kubernetes environment still show up after editing the orchestration to Cattle

**Rancher Version:**v1.1.0-dev2-rc4
OS and where are the hosts located? (cloud, bare metal, etc): Ubuntu, Cloud
Environment Type: (Cattle/Kubernetes/Swarm/Mesos): Cattle/Kubernetes
Steps to Reproduce:

Create a Kubernetes environment with few services and ingress
Switch to cattle environment

Results:

Services  created in Kubernetes environment still show up in cattle although the containers are destroyed.

Verified with Master. The services from Kubernetes environment are removed after editing the orchestration to cattle, the user created containers do show up as expected. When switched back to Kubernetes again, all the containers are cleaned up as part of Kubernetes bring up process.


Redesign: Update the inner box shadow for all focus used in the panel/library

Bug Description
The focus line is using bg.primary as the default box-shadow between the blue line and the input edge, making it seem transparent, however, the panels have different background color and many (or most) of the inputs are having visibly different colour due to that.
We should update the colours in the panel for all inputs.

Expected Behaviour

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

@samwhale FYI -- logged this issue since it looks like most of the inputs in the panel are currently using the incorrect box-shadow.

Good call!

Verified in QA


Make.go improvements

just some minor comments above, but otherwise great work. Would you also mind updating the build instructions in the README? https://github.com/drone/drone/tree/0.4.0#cloning-building-running

just some minor comments above, but otherwise great work. Would you also mind updating the build instructions in the README? https://github.com/drone/drone/tree/0.4.0#cloning-building-running

Sure, I will address that in the next commit.

@bradrydzewski I have addressed all the comments. Should be fine now :)


policy: keep parser type of first rule that applies at L7 even if all traffic is allowed at L7

Consider the following rule:
specs:
- endpointSelector:
    matchLabels:
      id: app1
  ingress:
  -  toPorts:
    - ports:
      - port: "9093"
        protocol: TCP
      rules:
        http:
        - method: ^GET$
          path: ^/metrics$
  - toPorts:
    - ports:
      - port: "9093"
        protocol: TCP

While this rule means that we allow all at L7, because there is L7 policy for that port, we must redirect through the proxy nonetheless. Do not set the proxy type in the L4Filter for this port to 'None'; instead, retain the proxy type of the first L7 rule for this port.

I disagree that this needs an e2e test, this is entirely unit testable.

I'm unsure about the semantics of L7RulesPerEp and how it interacts with L7Parser in the L4Filter. Need to sit down and discuss this to progress further.

I'm unsure about the semantics of L7RulesPerEp and how it interacts with L7Parser in the L4Filter. Need to sit down and discuss this to progress further.

Will review in details and provide feedback.

Fixed by #3732.


Document ports

Describe the change
Our book does not reference what ports are used, for what, and what should likely be publicly exposed.
These are the ports:

30303 (TCP and UDP): Peering with other nodes. Should be exposed
9001 (TCP): Metrics. Should not be exposed by default
8545 (TCP): HTTP RPC. Should not be exposed by default
8546 (TCP): WS RPC. Should not be exposed by default.
8551 (TCP): Engine API (for communication between reth and CL). Should not be exposed by default

Additional context
No response

gm, happy to help here

go ahead sir

@Evalir you still want this one?

Hey @Sabnock01 ! Yep still planning to contribute here, I'll ping you if I don't

@onbjerg I think this can be closed with #4370.

thanks @tcoratger


MPC: add root public key for clients to hardcode

We're ramping up a member of Proximity, and he was having a hard time finding the MPC root key.
I pulled master and searched globally but couldn't find it. So this issue is to make sure we include:
secp256k1:4HFcTSodRLVCGNVcGc4Mf2fwBBBxv9jxkGdiW2S2CA1y6UpVVRWKj6RX7d7TDt65k2Bj3w9FU4BGtt43ZvuhCnNt
So that folks who are creating clients are sure to hardcode it as suggested

Indeed, this is important, moreover since the key has changed for the new contract:
secp256k1:4NfTiv3UsGahebgTaHyD9vF8KYKMBnfd6kh94mK6xv8fGBiJB8TBtFMP5WWXz6B89Ac1fbpzPwAvoyQebemHFwx3


CompressAI version

Hi, thanks for your wonderful works!
Could you please provide your version of CompressAI? I have been trying to use the latest version 1.2.4, but I encountered an issue with the following error message: 'TypeError: TCM.init() got an unexpected keyword argument 'entropy_bottleneck_channels.''
I suspect that this issue might be related to a version inconsistency with CompressAI.
Could you please provide guidance on which version of CompressAI is compatible with my setup? Your help would be greatly appreciated.

Solved by refereing https://github.com/jmliu206/LIC_TCM/issues/6#issue-1739034056 . The correct version is 1.2.0


No version of NDK matched the requested version - Android platform

Hi everyone!
I'm new one in NativeScript development but I have a weird issues related to android environment. Actually when do I start to build android platform. I use the macosx to develop and my NativeScript components versions information is:
nativescript - 8.1.4
nativescript/core - 8.1.4
nativescript/ios is not installed
nativescript/android - 8.1.1
Follow the issue after command "ns run android":
_Execution failed for task ':app:stripDebugDebugSymbols'.
No version of NDK matched the requested version 20.0.5594570. Versions available locally: 22.1.7171670, 23.0.7599858
Unable to apply changes on device: [ANDROID_DEVICE_ID]. Error is: Command ./gradlew failed with exit code 1._
The most weird thing is: I have the latest version of NDK already installed (23.0.7599858). I really don't know if have any possibility to change the NDK version requested by NativeScript.
Anyone have an idea what happened in this issue?
Thanks in advance

Duplicate of #1659 . check the issue for a workaround


Blocked label in detail view is not highlighted and cannot be interacted with

Thanks to @cjcjameson for reporting:

We have the following issue: when we look at a label in the detail view (that has "blocked" in it), the label is not highlighted with any color at all, and it cannot be deleted / selected, etc.

Extracted from #5.

Thanks!!


Merge reflection class into the IBMiDb2Dialect

The reflection class is now not needed because there is only one
backend, so it was merged with the IBMiDb2Dialect
Fixes: #27

Looks like there's some conflicts that need to be resolved.


Bot see's trade (as shown in log) but does not react to it.

When I try and send a trade to the bot, it's acknowledged in the log file, but it does not do anything with the offer.
I think this may be due to the 7 day trade ban on newly logged in devices, however I can't see anything in debug/log files that would indicate whether this applys or not. Is this something to take into account with bots that use the steam API?
Thanks!

Literally minutes after opening this I see the bot does have the 7 day trade ban. My bad, closing this issue! Thanks :)


Add reported message author's profile picture to unfurl embed

Closes https://github.com/python-discord/bot/issues/2070
This PR adds the reported message author's profile picture to incident embed.

I have one small reservation about this that I need to discuss with mods first before merging.

Special thank you to @Shivansh-007 for helping me fix my git stuff 👍🏻

Looks perfectly fine!
Thank you! 😁

Because of your help!! Thank you 🥳


[Carbon X] DatePicker inner container clips content when date range selected

Detailed description
If you select a date range & then reopen either the start or end date picker, the content within is clipped. Here's a screenshot (NOTE: this is a g100 theme with some color overrides, but everything else remains unchanged from Carbon X) --

I think the problem area is this line:
https://github.com/IBM/carbon-components/blob/master/src/components/date-picker/_date-picker.scss#L852
.flatpickr-innerContainer,
  .flatpickr-rContainer {
    width: 100%;
    height: rem(336px);
  }

When I experimented with setting height: auto, it seems to resolve the problem. But I'm not sure if that's an ideal solution. 🤔
Steps to reproduce the issue

Open Carbon X date picker
Use start/end range picker to select a start and end range (try to include the first week of the month in the selected range to better notice the defect)
Reopen the end range date picker
Note the clipped content

@jendowns Thank you for spotting. Your proposed solution sounds fine to me, great if you can run some tests so the change won't accidentally break other in date picker. Thanks!

We updated to the latest carbon-components & we no longer see any clipping. Closing out this issue 🎉


Create test that saves and retrieves data from the database

Create test that saves and retrieves data from the database

Slack name: classicmeone_k
Wrote automation test for the Zuri core data api read and write and generate report using coverage module
Live link: https://music.zuri.chat/testreport/


SingletonAttribute from infra project not recognized during build

See: https://github.com/dolittle-samples/eCommerce/blob/f3b2699009d2480aa76caa630fc5d44166a45d70/Source/Common/Infrastructure.Web/Hubs.cs#L8
Should this come from Dolittle.Lifecycle then a reference should probably be added within Infrastructure.Web.csproj ?
On a similar note; the build also seems to dislike builder.Bind<ICallContext>().To<DefaultCallContext>(); within NullBindings on both Bounded Contexts

Relates to #4
We've had some namespace changes lately.  For our 2.0.0 release to go out, we need this sample compiling and working. Not complete - but at least just working :)


No way for user to navigate to verified domain from VS Code

Currently when I hover over the verified author I see the verified domain.
However there is no way for me to click on the verified domain, so I can inspect it more.
Could we introduce some simple UI gesture so users can be taken to the page?

fyi @misolori

We'd need to use the custom hover for this in order to make it clickable (we do this in the extension view hover)

We can introduce custom hover for it, but it will look odd unless we use custom hovers for other items next to it. What do you think?

Agreed to enable it for all items, I don't see any issues with doing that?

Yeah I do not think we loose a lot here.
I thought we are moving towards using the custom hover more and more in workbench.
Is that the long term vision? @Tyriar @bpasero

@isidorn that's the plan, it hasn't been discussed in the UX sync/wider team yet though

We can bring it up in the next UX sync

Brought it up in the UX sync and all were in favor, will post to the wider team to see if there's any pushback and if not we can start creating issues to get components to migrate over

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Adding the docker file with all python client dependencies as a quick start image

I checked the PR, and I have no major issues with it. However I do not have the rights to merge it, so let us wait for @nkarasiak to perform the merge.

Thank You Nicolas !!


Implement LLVM x86 AVX2 intrinsics

My hope was that we'd implement them in libstd as fallback bodies instead (https://github.com/rust-lang/miri/issues/3397), but I guess now that we have a shim impl and tests, doing that would be simpler as we have some help against messing up the fallback bodies

:hourglass: Testing commit a79b1f1f5348ff74bcda4fb94aa42b626f8f23bf with merge 9d6623e7e66e3b36a912ddc3c61eca965b1d2f4f...


Exclude null and undefined from Option()'s inferred type parameter

With strict null checks enabled, the current Option() constructor causes the Option's inferred type to include _ | null | undefined if the argument was of such type. This is not intended, as the goal of Option is to map those two types to None, as is enforced by the constructor.
This patch ensures that T does not include such union types.
I've tried and failed to write a failing test to cover this change. By definition, it's not possible to assert on a parameterized type in JavaScript since generics are erased during the TypeScript compilation. I haven't found a pattern to do it in TypeScript either.
The closest I've found is a test like
it('extracts the type of the value, ignoring undefined', () => {
  var value = 'value' as string | undefined;
  var opt: monapt.Option<string> = monapt.Option(value);
  // ^ will cause tsc compilation to fail
});

run through tsc --noEmit for type checking. It will fail to compile, but only if strictNullChecks is enabled. However monapt currently has a number of existing errors when enabling strict null checks, so that'd need to be fixed first.

Thanks!

Great, thanks for merging! Would there be any chance of doing a release with this fix? No big deal but I'm currently relying on an inline version of this patch, would be nice to use the pristine library instead :smile:

Yup, can do - I'll push out a release later this weekend.
Apologies for not being more on top of this, I plan to start doing weekly releases if needed.
And I'm happy to hear you're getting value out of monapt 👍

Rebuilding doesn't seem to update the type definitions, so a new release would be equivalent to the current release.
I wonder if this is related to strict null checks.  Any ideas?

Released in 0.7.1


Usage for mocking blockchain scenarios in Rust tests

Component
Forge
Describe the feature you would like
I'm implementing EIP-1271 using alloy in a Rust project and I'd like to write an automated test for my implementation. My idea is to write a tokio test which starts up an anvil blockchain, and deploys a mock EIP-1271 contract (perhaps the one from the spec) to the chain, and then executes the code under test using a mocked blockchain. Several test cases would check more scenarios, such as invalid signatures or invalid address.
However, I ran into some problems with this approach, namely:

Using anvil as a library does not appear to be stable. When using it with anvil = { git = "https://github.com/foundry-rs/foundry.git" } and providing the same [patch.crates-io] I get compile errors (and different ones from yesterday).
Using forge from Rust code is not possible because the main logic is inside the bin folder: https://github.com/foundry-rs/foundry/blob/master/crates/forge/bin/cmd/create.rs

Alternatively I could install the anvil and forge binaries and execute them from Rust, but this significantly more effort to pass configuration around (e.g. private keys and deployment addresses) and requires the developer to install dependencies.
Additional context
No response

something like this you mean?
https://github.com/foundry-rs/foundry/blob/1e8fb252f19b906d156f28421e79b8dfda5ed478/crates/anvil/tests/it/anvil.rs#L53-L60

Since this is just for internal testing, you should just require the anvil binary and call it through alloy-node-bindings. We don't provide any stability guarantees to any crate at the moment.

Note that the ethers crate provides an interface to anvil: https://docs.rs/ethers/latest/ethers/core/utils/struct.Anvil.html
I know that alloy is supposed to replace it and ethers is being deprecated, but until there is a better solution, maybe this is an acceptable workaround?
I do something very similar to what you describe here with anvil from ethers: https://github.com/xmtp/didethresolver/blob/a30b4da8b436cdbef6f93a17538612bc4d512e81/lib/tests/integration_util/mod.rs#L52

Thanks guys for the ideas! Ideally I'd like to do what @mattsse mentioned; i.e. call into the Anvil directly and spawn it. However, I had gotten compiler errors when trying to do this.

We don't provide any stability guarantees to any crate at the moment.

That's fine, I don't mind if the API is unstable and changes frequently. Would be nice if I could get it to compile though!
Seems like alloy-node-bindings is the best way forward for now. However I'd still request that this feature be added in the future: ability to spawn a node and deploy a contract from Rust w/o binary installation.

In the future we possibly have a better way of using Anvil as a library but for now using alloy-node-bindings is the recommended way. Marking as resolved.


Plz resolve this depedency conflicts issue

Because excel >=1.0.4 <2.0.0-null-safety depends on xml ^4.1.0 which depends on convert ^2.1.0, excel >=1.0.4 <2.0.0-null-safety requires convert ^2.1.0.
And because json_serializable >=4.0.3 depends on build ^2.0.0 which depends on convert ^3.0.0, excel >=1.0.4 <2.0.0-null-safety is incompatible with json_serializable >=4.0.3.
And because invoiceninja 0.0.9 depends on json_serializable ^4.1.0 and no versions of invoiceninja match >0.0.9 <0.1.0, excel >=1.0.4 <2.0.0-null-safety is incompatible with invoiceninja ^0.0.9.
So, because etf depends on both invoiceninja ^0.0.9 and excel ^1.1.5, version solving failed.
pub get failed (1; So, because etf depends on both invoiceninja ^0.0.9 and excel ^1.1.5, version solving failed.)
exit code 1

@fahadmustafa650
This issue is not from excel package.
Please refer to Stack-Overflow for further help.
Thanks.


WIP: Check for untapped deps when sorting formulae

I maintain at least one formula with cross-tap dependencies (osrf/simulation/gazebo7 depends on cartr/qt4/qt@4). In #91, I fixed a bug in unlink_conflicts related to this use case that rescues from TapFormulaUnavailableError, adds the missing tap, and tries again to call recursive_dependencies on a formula.
I just noticed there is an earlier place where a missing tap causes a problem: in the run function, a formulae.each loop is right after homebrew.
The formulae function calls brew deps and then sorts the formulae so the lowest level dependencies are processed first, but brew deps fails if untapped formulae aren't available:
$ brew deps gazebo7 && echo success
Error: No available formula with the name "cartr/qt4/qt@4"
Please tap it and then try again: brew tap cartr/qt4
$

I hadn't noticed this before when just building a gazebo7 bottle, but I now notice that the order is wrong when trying to build bottles for https://github.com/osrf/homebrew-simulation/pull/269, which has bumped revisions for gazebo7 and two of its dependencies that are later in the alphabet, so it builds out of order.
This PR checks $? to see if the brew deps command failed, and if so calls Formulary.factory(formula).recursive_dependencies with a rescue block to add the missing tap. I put WIP in the title because it feels messy to duplicate code from #91. I am assuming that brew deps behavior should not change to automatically add missing taps.

Seems reasonable, thanks @scpeters. @ilovezfs any thoughts?

Thanks again @scpeters!


SVS-30 SVS-19 Including SonarAnalyzer for the unconnected mode, and d…

…eactivating rules when another version is present

I'm unsure how these changes could be unit tested. Both of them are IDE specific.

I replied on how you may test most of the logic, and provided other minor comments.
Other than that  LGTM.

@eyalantzMS, @mjcheetham thanks, I've applied the quality fixes that you proposed, and also did a bit of refactoring to at least be able to test the assembly version collision checking.
Could you have a second look?

Minor comments, otherwise LGTM

downgraded the LanguageServices and CodeAnalysis references to version 1.0.0. It should be good now.
Someone could have a look at the app.config assemblyBindings. See the above comment.
And someone should still test this PR not just review it. I added different versions of SonarAnalyzer nuget to the JIRA ticket.

@tvsonar I'll create a machine with VS2015 RTM and test the VSIX with downgraded versions.

Superceded by https://github.com/SonarSource-VisualStudio/sonarlint-visualstudio/pull/14


[Space Engine] Remove the @_downgrade_exhaustivity_check hack

This hack was only needed for Swift 3 mode in a narrow case.  Flush it out of the compiler so we can simplify the space engine.

@swift-ci please smoke test

@swift-ci please test source compatibility

Good stuff!

Please test with following pull request:
https://github.com/apple/swift-corelibs-libdispatch/pull/381
@swift-ci Please smoke test Linux platform

Lost the original source compat failure.  Let's try again
@swift-ci please test source compatibility

Please test with following pull request:
apple/swift-corelibs-libdispatch#381
@swift-ci Please smoke test

Please test with following pull request:
apple/swift-corelibs-libdispatch#381
@swift-ci Please smoke test Linux platform

⛵


Support Part/Section/Section Data

It would be nice to support Part / Section/  Chapter data for long SGF collections.  This provides a way to segment content and also is necessary for book generation.

This ended up not being the right approach.


A11y_AzureDataStudioAzureAAE_AzureArcController_ConnecttoExistingAzureArcController_ScreenReader: NVDA does not announces the change in the notification when the default path is deleted from "Kube config file path" editbox

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#SH_AzureDataStudio_AAE_Jun2021;#SQL Azure Data Studio;#Win10;#FTP;#DesktopApp;#NVDA;#WCAG4.2.12;#AINotInScope;
Environment Details:
Application Name: Azure Data Studio -AAE
Windows Version: Win10
NVDA Version: 2020.4, Jaws(2020.2008.24)
Repro Steps:

Launch the application-Azure Data Studio-Insiders and install extension Azure arc and Azure data ClI
Connect to Azure Arc Controller
Tab till Azure Arc Controller tree and tab to "Create new Azure Arc Controller"
Tab to "Kube config file path" edit box start nvda and delete the default path location and observe announcement.

Actual:
NVDA does not announces the change in the notification when the default path is deleted from "Kube config file path" edit box.
The notification changes from "ENOENT: no such file or directory open, "path name" to ""ENOENT: no such file or directory, open"
Expected:
NVDA should be announcing the change in the notification as well as the original notification.
User Impact:
Screen Reader users will not get the proper information about change in notification.
https://user-images.githubusercontent.com/60880975/127480498-c1041ad2-370b-4b0e-b064-361345ad888e.mp4

Closing as not repro


Two "inputs" formats create different assets

According the samples in specification, we support two formats for "inputs" (list of
parameter):
a complex one:
  actions:
    func1:
      function: actions/function.js
      runtime: nodejs:6
      inputs:
        functionID:
          type: string
          description: the ID of function
        visited:
          type: string
          description: the visted city list

and a simple one:
package:
  name: TestSequencesCreation
  actions:
    func1:
      function: actions/function.js
      runtime: nodejs:6
      inputs:
        functionID: string
        visited: string

The complex one will create an action, with empty string "" to be the default values of parameters. The simple one will create an action with string to be the default values of parameters. See below samples.
action created by complex format
{
    "namespace"<EMAIL_ADDRESS>    "name": "func1",
    "version": "0.0.1",
    "exec": {
        "kind": "nodejs:6",
        "code": "/**\n * Return a simple string to \n * confirm this function has been visited.\n *\n * @param visited the visited function list\n */\nfunction main(params) {\n    functionID = params.functionID || 'X'\n    if (params.visited == null) {\n        params.visited = 'function'+functionID;\n    } else {\n        params.visited = params.visited + ', function'+functionID;\n    }\n    return {\"visited\":params.visited};\n}\n"
    },
    "annotations": [
        {
            "key": "exec",
            "value": "nodejs:6"
        }
    ],
    "parameters": [
        {
            "key": "functionID",
            "value": ""
        },
        {
            "key": "visited",
            "value": ""
        }
    ],
    "limits": {
        "timeout": 60000,
        "memory": 256,
        "logs": 10
    },
    "publish": false
}

action created by the simple format:
ok: got action TestSequencesCreation/func1
{
    "namespace"<EMAIL_ADDRESS>    "name": "func1",
    "version": "0.0.1",
    "exec": {
        "kind": "nodejs:6",
        "code": "/**\n * Return a simple string to \n * confirm this function has been visited.\n *\n * @param visited the visited function list\n */\nfunction main(params) {\n    functionID = params.functionID || 'X'\n    if (params.visited == null) {\n        params.visited = 'function'+functionID;\n    } else {\n        params.visited = params.visited + ', function'+functionID;\n    }\n    return {\"visited\":params.visited};\n}\n"
    },
    "annotations": [
        {
            "key": "exec",
            "value": "nodejs:6"
        }
    ],
    "parameters": [
        {
            "key": "functionID",
            "value": "string"
        },
        {
            "key": "visited",
            "value": "string"
        }
    ],
    "limits": {
        "timeout": 60000,
        "memory": 256,
        "logs": 10
    },
    "publish": false
}

I think it is a bug that the simple format creates an action with string to be the default values of parameters. We should fix it.

@daisy-ycguo @mrutkows please correct me if I am wrong,
I don't think so its a bug, here is why, in complex example, we are specifying that an action func1 has two parameters functionID and visited and both of these parameters are of type string, now by default, golang assigns zero value of string to these two parameters which is empty string, until the values are being picked up from deployment file.
Now, in case of simple example, we are specifying that an action func1 has two parameters functionID and visited and both have values literal string just like assigning any other value id123 or New York etc. and that's why the values of these two parameters are string.


Add option to hide unsupported files and directories

As the title says. This would be useful for a couple of different uses cases, such as:

Users who use the command in watch to get live updates.
Users who want a de-cluttered output.

This is a very good idea and definitely something I'd merge.
I'll probably implement this myself in a couple of days when I have the time, in the meantime I'm open to PRs.
Thanks for the suggestion!

Gonna chime in with agreement on this. Had to quickly edit the script up to have $SKIP_IMMOVABLE, sorry for the badly named hack x3
Also another pain is that uhhh, well, it would be really nice to actually have the variable info separate or such, so that you could tell it to output a .bashrc/etc drop in file, with comments detailing what it is for, and if there are any possible caveats and commenting it out by default for that
Maybe also give a "migration" command too, like, just a bunch of mvs for supported dirs that don't have "gotchas"

I'll probably implement this myself in a couple of days when I have the time

Well, sorry about that.
Anyways, it's implemented with 193c1d3 and fea5aa5, you can now use --skip-unsupported.
Let me know if you encounter any problems.


ants* tools don't return error on failures

Related to #397 , at least antsRegistration and antsApplyTransforms don't throw an non-zero return code on (some) failures.
In this case, a typo for the extension of antsRegistration's output throws an exception, but the bash script running it doesn't stop because the return code was 0:
Exception Object caught:
itk::ImageFileWriterException (0x34bc390)
Location: "unknown"
File: /home/m/mchakrav/gdevenyi/projects/quarantine/minc-toolkit/1.9.10/src/build2/ITKv4/Modules/IO/ImageBase/include/itkImageFileWriter.hxx
Line: 151
Description:  Could not create IO object for writing file /tmp/tmp.htnQTj34Fz/2/nonlin0.xfm
  Tried to create one of the following:
    NiftiImageIO
    NrrdImageIO
    GiplImageIO
    HDF5ImageIO
    JPEGImageIO
    GDCMImageIO
    BMPImageIO
    LSMImageIO
    PNGImageIO
    TIFFImageIO
    VTKImageIO
    StimulateImageIO
    BioRadImageIO
    MetaImageIO
    MRCImageIO
    GE4ImageIO
    GE5ImageIO
    MINCImageIO
  You probably failed to set a file suffix, or
    set the suffix to an unsupported type.
<Script continues despite failure of antsRegistration command>

And immediately after, antsApplyTransforms can't find the output file, but the script continues anyways:
antsApplyTransforms -i /project/m/mchakrav/quarantine/resources/mni_icbm152_nlin_sym_09c_minc2/mni_icbm152_wm_tal_nlin_sym_09c.mnc -t /tmp/tmp.htnQTj34Fz/2/linear.xfm -t /tmp/tmp.htnQTj34Fz/2/nonlin0_inverse_NL.xfm -r /tmp/tmp.htnQTj34Fz/2/denoise.mnc -o /tmp/tmp.htnQTj34Fz/2/SegmentationPrior3.mnc -d 3 -n Linear
Transform file does not exist: /tmp/tmp.htnQTj34Fz/2/nonlin0_inverse_NL.xfm
+ antsApplyTransforms -i /project/m/mchakrav/quarantine/resources/mni_icbm152_nlin_sym_09c_minc2/mni_icbm152_gm_tal_nlin_sym_09c.mnc -t /tmp/tmp.htnQTj34Fz/2/linear.xfm -t /tmp/tmp.htnQTj34Fz/2/nonlin0_inverse_NL.xfm -r /tmp/tmp.htnQTj34Fz/2/denoise.mnc -o /tmp/tmp.htnQTj34Fz/2/SegmentationPrior2.mnc -d 3 -n Linear
Transform file does not exist: /tmp/tmp.htnQTj34Fz/2/nonlin0_inverse_NL.xfm
+ antsApplyTransforms -i /project/m/mchakrav/quarantine/resources/mni_icbm152_nlin_sym_09c_minc2/mni_icbm152_csf_tal_nlin_sym_09c.mnc -t /tmp/tmp.htnQTj34Fz/2/linear.xfm -t /tmp/tmp.htnQTj34Fz/2/nonlin0_inverse_NL.xfm -r /tmp/tmp.htnQTj34Fz/2/denoise.mnc -o /tmp/tmp.htnQTj34Fz/2/SegmentationPrior1.mnc -d 3 -n Linear
Transform file does not exist: /tmp/tmp.htnQTj34Fz/2/nonlin0_inverse_NL.xfm

Eventually, Atropos segfaults because a bunch of its inputs don't exist.

I added a  bunch of EXIT_SUCCESS / EXIT_FAILURE calls a while ago, but it's possible I missed a use case. I'll look further into it if you can provide a reproducible example.

https://github.com/stnava/ANTs/commit/b2232ba768b0f4b3e33d3b1965bc0a2e0ae2b87d#diff-8600080e87b85d19d357ef64624e93d9


justering på hva som vises fra språk-objektet

A picture tells a thousand words
Before this PR

After this PR

Coverage increased (+0.3%) to 43.272% when pulling 1f1ec02c63b21e0084d2ffc88042defc20414710 on fix_#379 into 1b6c9c665fa4757f2f1a4b2c472ec241df93270a on master.


Define Mode better

The previous definition was the same as for Resource. This one is pulled straight from @kapilt in private Slack.

Ready for review, @kapilt.


Missing Dyson Sphere Program recipe

DSP has added a single new recipe, adding yet another loop to the oil production.

Thanks for the details! This is also logged as issue #761, so I'll close this one, but I hope to add the issue as soon as possible, probably by Monday or Tuesday.


Database Service\Ingestions Pagination Listing Issue.

Affected module
In my opinion, it might be a UI layer issue.
Describe the bug
Reporting a bug under Database Service\Ingestions, issue on pagination, the page is not refreshing to list all 17 ingestions even if you  change the # of items per page to 25 or 50.
To Reproduce
Create more than 10 ingestions in a Database Service

Expected behavior
It should list down the ingestions items based on the selected # of items per page.
Version:

OS: AWS EKS
Python version:
OpenMetadata version: 1.2.4
OpenMetadata Ingestion package version: 1.2.4 in AWS EKS

Additional context
Please see the attached screenshot.

Fixed with 1.3.0


RPC: unify return type for ExecuteTransaction

The key change of this PR is in crates/sui-json-rpc-types/src/lib.rs
Before
execute_transaction returns TransactionResponse, the schema of which varies between different transaction types
#[derive(Serialize, Deserialize, Debug, JsonSchema)]
pub enum TransactionResponse {
    EffectResponse(TransactionEffectsResponse),
    PublishResponse(PublishResponse),
    MergeCoinResponse(MergeCoinResponse),
    SplitCoinResponse(SplitCoinResponse),
}

After
execute_transaction now returns SuiTransactionEffectsResponse, which will always have certificate, effects, timestamp_ms field. I added an optional parsed_data field for convenience. Clients can choose to ignore this field(i.e., it does not make client side parsing harder), but some clients(e.g., sui client cli) may find the parsed data helpful. For example, for Publish Transaction, the parsed_data field may contain a package_id field. Without this field, the client would have to make additional queries for each of the created objects to find out their types.
#[derive(Serialize, Deserialize, Debug, JsonSchema)]
pub struct SuiTransactionEffectsResponse {
    pub certificate: SuiCertifiedTransaction,
    pub effects: SuiTransactionEffects,
    pub timestamp_ms: Option<u64>,
    pub parsed_data: Option<SuiParsedTransactionResponse>,
}

#[derive(Serialize, Deserialize, Debug, JsonSchema, Clone)]
pub enum SuiParsedTransactionResponse {
    Publish(SuiParsedPublishResponse),
    MergeCoin(SuiParsedMergeCoinResponse),
    SplitCoin(SuiParsedSplitCoinResponse),
}

Testing
In addition to the unit test, I also manually tested the sui client as the Display trait implementation is changed. Note that the parsed data such as Split Coin Results are still rendered.
➜ ./sui client split-coin --amounts 100 --coin-id 0x043e5935d0bd263b91de724d6f58a46b038a100b --gas-budget 1000
----- Certificate ----
Transaction Hash: re8jFpLqcQQAVkze3pE5j9JRRDA/mxgqOvufnpWvoG8=
Transaction Signature: AA==@AaW+Oy4xPVVChb4lFHxicsQe9gpWwP2Pgxn4w25z2UNAkujXjEAdHeDzGUqyQq8cPBY2dOWlJlATfQZJ5m21Cg==@UoCg/5a0f/fbC1If8AofBRAs4wSHpuzkpUbhCF7dn8I=
Signed Authorities Bitmap: RoaringBitmap<[0, 1, 2]>
Transaction Kind : Call
Package ID : 0x2
Module : coin
Function : split_vec
Arguments : ["0x43e5935d0bd263b91de724d6f58a46b038a100b", [100]]
Type Arguments : ["0x2::sui::SUI"]
----- Transaction Effects ----
Status : Success
Created Objects:
  - ID: 0xf3476a213c021166c19eab8444e43b025549724b , Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
Mutated Objects:
  - ID: 0x043e5935d0bd263b91de724d6f58a46b038a100b , Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
  - ID: 0x42aac345cc504060cab63c398bf2e4750e682b18 , Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
----- Split Coin Results ----
Updated Coin : Coin { id: 0x043e5935d0bd263b91de724d6f58a46b038a100b, value: 99999900 }
New Coins : Coin { id: 0xf3476a213c021166c19eab8444e43b025549724b, value: 100 }
Updated Gas : Coin { id: 0x42aac345cc504060cab63c398bf2e4750e682b18, value: 99999473 }

➜ ./sui client merge-coin --primary-coin 0xaef3f3c497ccf44f58ae49e5f6d296650c909727 --coin-to-merge 0x97f88e91be1b5cd9ab4b2d9257af0867eae6d474 --gas-budget 1000
----- Certificate ----
Transaction Hash: 6nkRyCP1OViLnavKhRw4dvBagw9O5zr9CauIFatoPEU=
Transaction Signature: AA==@CdKTVbjNiZCaRzj/2gIZAI4xwBOG+AClMQzUbEn0wUP8UaBwWMzTCDqolRq8z+lGVaSykMQgoRJf/vV8LpyXAg==@UoCg/5a0f/fbC1If8AofBRAs4wSHpuzkpUbhCF7dn8I=
Signed Authorities Bitmap: RoaringBitmap<[0, 1, 2]>
Transaction Kind : Call
Package ID : 0x2
Module : coin
Function : join
Arguments : ["0xaef3f3c497ccf44f58ae49e5f6d296650c909727", "0x97f88e91be1b5cd9ab4b2d9257af0867eae6d474"]
Type Arguments : ["0x2::sui::SUI"]
----- Transaction Effects ----
Status : Success
Mutated Objects:
  - ID: 0x043e5935d0bd263b91de724d6f58a46b038a100b , Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
  - ID: 0xaef3f3c497ccf44f58ae49e5f6d296650c909727 , Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
Deleted Objects:
  - ID: 0x97f88e91be1b5cd9ab4b2d9257af0867eae6d474
----- Merge Coin Results ----
Updated Coin : Coin { id: 0xaef3f3c497ccf44f58ae49e5f6d296650c909727, value: 200000000 }
Updated Gas : Coin { id: 0x043e5935d0bd263b91de724d6f58a46b038a100b, value: 99999429 }

fastnft/target/debug on  chris-unify-transaction-response [$!] using ☁  default/sui-devnet
➜ ./sui client publish --path ~/projects/toy-move-packages/fungible-tokens/ --gas-budget 30000
----- Certificate ----
Transaction Hash: 3RbcdT8t6YPTfKxXon5RQdmbDWQwjHILUx3LVUU8HGU=
Transaction Signature: AA==@TBwXTCVBacpnRgUsz19EoSbXk+HVcq0I+ZhrGDSLbIyyr/3cQBBZ19+I+OFxu/1v2k8dprU/gQnP6XUMOVhMDQ==@UoCg/5a0f/fbC1If8AofBRAs4wSHpuzkpUbhCF7dn8I=
Signed Authorities Bitmap: RoaringBitmap<[1, 2, 3]>
Transaction Kind : Publish
----- Transaction Effects ----
Status : Success
Created Objects:
  - ID: 0x816726a6501f709d202010a46bea2ccb284f3540 , Owner: Immutable
  - ID: 0xab0ece644e06f2b25a96821f77483d503c928812 , Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
Mutated Objects:
  - ID: 0x043e5935d0bd263b91de724d6f58a46b038a100b , Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
----- Publish Results ----
The newly published package object ID: 0x816726a6501f709d202010a46bea2ccb284f3540

List of objects created by running module initializers:
----- Move Object (0xab0ece644e06f2b25a96821f77483d503c928812[1]) -----
Owner: Account Address ( 0x03530db0719259f8b5ff912d05e020b2ab40745e )
Version: 1
Storage Rebate: 16
Previous Transaction: 3RbcdT8t6YPTfKxXon5RQdmbDWQwjHILUx3LVUU8HGU=
----- Data -----
type: 0x2::coin::TreasuryCap<0x816726a6501f709d202010a46bea2ccb284f3540::usdc::USDC>
id: 0xab0ece644e06f2b25a96821f77483d503c928812
total_supply:
  type: 0x2::balance::Supply<0x816726a6501f709d202010a46bea2ccb284f3540::usdc::USDC>
  value: 0

Updated Gas : Coin { id: 0x043e5935d0bd263b91de724d6f58a46b038a100b, value: 99998848 }

Checklist

[x]  Update TS-SDK
[x]  Update Explorer
[x]  Update Wallet Browser Extension

#closes #3854

Dumb question - do we return package in transaction effects? That's a must.

The changes in json sample files are really quite annoying and makes the PR look huge lol, do we still need them for TS SDK testing?

We should be able to get rid of those pretty soon as @Jordan-Mysten is moving the test data generation to the client side.


CMS updates won't deploy

There is an issue again when deploying updates through the CMS

This GitHub Action error can safely be ignored. This error is thrown because we did not remove the deprecated GitHub Action Workflow. Therefore, the site was still being deployed via GitHub Pages. The GitHub Actions Workflow has been removed from the repo, and the site is currently hosted through Cloudflare Pages.


Backend tests inexplicably fail in GitHub Actions sometimes

To Reproduce
Open or merge any PR.
Expected behavior
All tests pass consistently if they aren't broken.
Actual behavior
Sometimes backend tests won't pass for seemingly no good reason. Could be an issue with GitHub, could be an issue with hidden dependencies between tests.

Might also be a timeout issue. In #513 a test failed for this reason.


No discoverable shortcut to "Auto-format"

There was a certain shortcut for autoformatting the code in the code pane of Dartpad. But there's no in the current updated Dartpad. It would be nice to get it back. Thanks!

It's an action button next to the run button:

Please let us know if that is not showing up for you.


cant find media folder not named "movies"

My Plex instance runs using one singular folder mounted to the container, then sub-directories to find its "tv" and "movies" folders.
i mounted the subgen volume bind pointing at the same root folder but it cannot see any of the files inside. does it need to be pointing at two different folders, named "tv" and "movies"? and does plex also need to be looking for those two folders as well?

What do your volume binds look like for both Plex and subgen?  What do the subgen logs show?

Plex Bind: /mnt/Harmony:/Harmony
Subgen bind: /mnt/Harmony:/Harmony
also fails with "/mnt/Harmony/Plex Media/movies:/movies" and "/mnt/Harmony/Plex Media/tv shows:/tv"
Container logs are:
_subgen_logs.txt

You have a 404 error in your log meaning your Plex API key is wrong, or you server/webhook address is wrong in Tautulli.  Re-read the instructions for setup.

Thanks! fixed the issue with the webhook, now getting a 200 result as expected. the logs dont show the processing of any files though, is that expected?
_subgen_logs (1).txt

Dependant on your settings, whether you have trigger on add, play, or skip if existing internal sub.  Mine rarely runs (probably less than 5 times a week) as most stuff now has internal stuff embedded.
Easy way to test is enable proconplay and disable skip internal and just play something.

Solved, thank you so much!!


Add Bread Puns API

Description
This API contains random Bread Puns.
Screenshot
Optional

Checklist

[x] API used in this PR was not used in just-calling-random-api before.
[x] API used in this PR is Public or I have permission to use it.
[x] My contribution is working.
[x] My contribution doesn't expose any API keys.
[x] My changes don't generate any new warnings.
[x] My changes don't affect any work done by other developers.

To tick a box, replace [ ] with [x].

Thanks 😊


Per file options

Use if instead of filters for configs
Add per-file buildoptions

I fixed a minor bug.
Please let me know if you are unable to further maintain this project, in which case I would maintain my own fork.

yeah im not working on this repo anymore.
you can maintain your own fork or team up with other forks.


[BUG] Multi-GPU runs crashing with "invalid permissions for mapped object"

Hi, thanks @jiangtong1000 for providing the sample mpi4py+cupy AFQMC script off-GitHub earlier. This resolved✓ the issue with the MPI processes being crowded to a single GPU. I come with a different error message this time.
Describe the bug
I tried running it on my cluster for the scf.chk, hamiltonian.h5, wavefunction.h5 which I had with Li₂ (generated from PBE/cc-pVDZ). After 0-25 steps into the main QMC loop, ipie crashes with a "invalid permissions for mapped object" error.
To Reproduce
This is the script for generating scf.chk, hamiltonian.h5, wavefunction.h5. The other files were used with minimal changes (same env vars, same scheduler -- SLURM).
from pyscf import gto, dft
from ipie.utils.from_pyscf import gen_ipie_input_from_pyscf_chk

bsetName = "cc-pvdz"

N_occA = 3
N_occB = 3

L_bond = 5.051

mol = gto.M(
        atom = [("Li", (0, 0, -L_bond/2)), ("Li", (0, 0, L_bond/2))],
        basis = bsetName,
        verbose = 3,
        spin = N_occA - N_occB,
        unit = "Bohr",
        )

rks_diatom = dft.rks.RKS(mol, xc="pbe")
rks_diatom.chkfile = "scf.chk"
e_rks = rks_diatom.kernel()

gen_ipie_input_from_pyscf_chk(
    f"scf.chk",
    "hamiltonian.h5",
    "wavefunction.h5",
    chol_cut=1e-5,
    )

Expected behavior
ipie should go through all 50000 blocks, with the final energy being around -14.99 au.
Screenshots
Here's the output.
[falo-cu4:773079] Rank 0 bound to package[1][core:26]
[falo-cu4:773079] Rank 1 bound to package[1][core:27]
[falo-cu4:773079] Rank 2 bound to package[1][core:28]
[falo-cu4:773079] Rank 3 bound to package[1][core:29]
0 0 4
2 2 4
1 1 4
3 3 4
2 784
1 784
3 784
0 784
158 158
158 158
158 158
158 158
# random seed is 70881236
# Using pair_branch population control algorithm.
# target weight is 640
# total weight is 640
# ipie version: 0.7.1.dev0
# Git hash: None.
# Git branch: None.
# Calculation uuid: 7623c558-26bd-11ef-82e4-b47af137216e.
# Approximate memory available per node: 755.5523 GB.
# Running on 4 MPI ranks.
# Root processor name: falo-cu4
# Python interpreter: 3.11.9 (main, May  9 2024, 23:25:41) [GCC 12.2.0]
# Using numpy v1.25.2 from: /proj/home/ibs/tdqh/cwmyung/dcyang/src/venv-ipie/lib/python3.11/site-packages/numpy.
# - BLAS lib: openblas64_ openblas64_
# - BLAS dir: /usr/local/lib
# Using scipy v1.10.1 from: /proj/home/ibs/tdqh/cwmyung/dcyang/src/venv-ipie/lib/python3.11/site-packages/scipy.
# Using h5py v3.11.0 from: /proj/home/ibs/tdqh/cwmyung/dcyang/src/venv-ipie/lib/python3.11/site-packages/h5py.
# Using mpi4py v3.1.6 from: /proj/home/ibs/tdqh/cwmyung/dcyang/src/venv-ipie/lib/python3.11/site-packages/mpi4py.
# - mpicc: /opt/ibs_lib/apps/openmpi/4.1.4/GCC/1220/bin/mpicc
# Using cupy v12.2.0 from: /proj/home/ibs/tdqh/cwmyung/dcyang/src/venv-ipie/lib/python3.11/site-packages/cupy.
# - CUDA compute capability: 7.0
# - CUDA version: 12.02.0
# - GPU Type: 'Tesla V100-SXM2-32GB'
# - GPU Mem: 31.739 GB
# - Number of GPUs: 4
# MPI communicator : <class 'mpi4py.MPI.Intracomm'>
# Available memory on the node is 755.552 GB
# PhaselessGenericChunked: expected to allocate 0.0 GB
# PhaselessGenericChunked: using 1.253662109375 GB out of 31.7393798828125 GB memory on GPU
# GenericRealCholChunked: expected to allocate 0.0007276087999343872 GB
# GenericRealCholChunked: using 1.253662109375 GB out of 31.7393798828125 GB memory on GPU
# SingleDet: expected to allocate 0.0001310110092163086 GB
# SingleDet: using 1.253662109375 GB out of 31.7393798828125 GB memory on GPU
# UHFWalkers: expected to allocate 0.0037577003240585327 GB
# UHFWalkers: using 1.253662109375 GB out of 31.7393798828125 GB memory on GPU
# Setting up estimator object.
# Writing estimator data to estimates.0.h5
# Finished settting up estimator object.
            Block                   Weight            WeightFactor            HybridEnergy                  ENumer                  EDenom                  E
Total                  E1Body                  E2Body
[falo-cu4:773085:0:773085] Caught signal 11 (Segmentation fault: invalid permissions for mapped object at address 0x152aedcaf000)
[falo-cu4:773084:0:773084] Caught signal 11 (Segmentation fault: invalid permissions for mapped object at address 0x151465caf000)
[falo-cu4:773083:0:773083] Caught signal 11 (Segmentation fault: invalid permissions for mapped object at address 0x14cc85cab400)
[falo-cu4:773082:0:773082] Caught signal 11 (Segmentation fault: invalid permissions for mapped object at address 0x15471acab400)
BFD: Dwarf Error: Can't find .debug_ranges section.
BFD: Dwarf Error: Can't find .debug_ranges section. (this line repeated 132 more times)

Other information:

Git branch and hash: eef994f
OS: Rocky 8.6 + kernel 4.18
Version: Python 3.11.9 (system-wide), slurm 21.08.5 (system-wide), OpenMPI 5.0.3 (user-built), "relevant" parts of pip list shown below ...

ipie                          0.7.1.dev0     /home/dcyang/arc/ipie
h5py                          3.11.0
mpi4py                        3.1.6
numpy                         1.25.2
cupy-cuda12x                  12.2.0
cutensor-cu12                 2.0.1
Cython                        3.0.10
scipy                         1.10.1
pyscf                         2.5.0
pytest                        8.2.0
pandas                        1.5.1
numba                         0.59.1
plum-dispatch                 2.3.5
nvidia-cublas-cu12            <IP_ADDRESS>
nvidia-cuda-cupti-cu12        12.1.105
nvidia-cuda-nvrtc-cu12        12.1.105
nvidia-cuda-runtime-cu12      12.1.105
nvidia-cudnn-cu12             <IP_ADDRESS>
nvidia-cufft-cu12             <IP_ADDRESS>
nvidia-curand-cu12            <IP_ADDRESS>
nvidia-cusolver-cu12          <IP_ADDRESS>
nvidia-cusparse-cu12          <IP_ADDRESS>
nvidia-nccl-cu12              2.20.5
nvidia-nvjitlink-cu12         12.5.40
nvidia-nvtx-cu12              12.1.105

Additional context
This seems like a lower level error (either with hardware or SLURM) which I'm also trying to debug in the meantime ... but any help will be greatly appreciated.

hi David @dcyang , is this cuda awared mpi?

hi David @dcyang , is this cuda awared mpi?

Yes, I built a fresh OpenMPI in my user space with an explicit --with-cuda=<valid-path> because the system-provided OpenMPI was pointing to a CUDA installation path that no longer existed. Anyways, ompi_info of my current OpenMPI installation shows that it is CUDA-aware.

A cursory google suggests this is cuda aware mpi related, and potentially related to the UCX version. On your end I would suggest testing a simple MPI Send Recv operation without any ipie intergration (or some other test for cuda aware MPI) to make sure that device memory is being sent appropriately.

It turns out that the system-provided UCX installation (which my OpenMPI was using) was also pointing to the old non-existent CUDA installation path. I built and installed the 1.16.0 (newest) with --with-cuda=<valid-path-12.2> --enable-mt (apparently it treats OpenMP and multi-threading separately), and then re-built OpenMPI with --with-cuda=<valid-path-12.2> --with-ucx=<valid-path-1.16.0>.
Now the crash doesn't happen. ✓ I think any version of UCX >= 1.13 is fine. Thanks all for your help!


The pwsh freezes when using a regular expression

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Hello,
When using this regular expression ""data"(.*?|\n)+},", the PowerShell process hangs. Changing the regular expression resolves the issue, and the code executes successfully - ""data"(\n)+},"
$obj = @{
	data = @{
		data = @{
			message = ""
		}
	}
}
$obj.data.data.message = "text"
$body = @{data=$obj.data.data} | ConvertTo-Json

Write-Host $($body -replace "`"data`"(.*?|\n)+},")

Expected behavior
pwsh process not freezes

Actual behavior
pwsh process freezes

Error details
null

Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0
PSEdition                      Core
GitCommitId                    7.4.0
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

Fundamentally, please note that PowerShell's regex functionality is just a thin layer on top of the .NET regex APIs, so it's always worth checking if the problem also occurs when using these APIs directly (e.g., [regex]::Replace($body, "`"data`"(.*?|\n)+},", '')) - which is indeed the case here. If so, issues need to be reported at https://github.com/dotnet/runtime/issues
However, in the case at hand I think the only problem is that you've created a very inefficient regex that takes a very long time to evaluate (about 7 seconds on my machine). That is, there is no hang.
I assume you're looking for something like the following (I'm only guessing), which evaluates virtually instantaneously:
$body = @'
{
  "data": {
    "message": "text"
  }
}
'@

$body -replace '(?s)"data".+?\}'

Note, however, that using regexes to parse and manipulate JSON is best avoided, as it is inherently brittle.

Thanks for the issue @korovindenis! And thanks for the testing @mklement0! I also suggest opening an issue with dotnet since the issue seems to lie with them.

I've created a .NET issue here:

https://github.com/dotnet/runtime/issues/96340


Moderpython simplification + Attribute NS

Mostly split Base.py in a bunch of smaller, more specific classes, and add attributes NS.

@guillaume-alliander What is the state of this PR? Can it be merged before #20 or is there some dependency? Please assign this to me when you would like one of us to have a look.
If you would like to discuss the roadmap or maintainer role with us, feel free to reach out to @dinkelbachjan and me via mail or slack.

I would like to have confirmation from @chicco785 but this one should be merged before #20 I believe.

I would like to have confirmation from @chicco785 but this one should be merged before #20 I believe.

yes, i will rebase my changes on top of this PR


Pause and skip song with headset click

A minor suggestion that's useful for many - pause the song on single click on headset button and skip to next song on double click.
PS : I think the reason why Media action button listeners are not working properly on some device is because you haven't set a priority for RemoteControlReceiver in the manifest.

Sorry I took so long to get back to you -- I've been pretty busy with the semester ending last week.
Single clicking the headset remote button currently does toggle between play/pause on my Moto X 2nd generation and Nexus 9 running 5.1 and the N developer preview respectively, and I'm not sure why it wouldn't be working on your device. I recently found that Jockey doesn't properly capture media focus on its first launch when a song is played for the first time, but it shouldn't affect you since it's resolved after pausing playback or restarting Jockey.
RemoteControlReceiver actually isn't even being used anymore -- MediaSessionCompat with a MediaSessionCompat.Callback is being used to handle play/pause key events.
Double and triple clicks to skip would be nice to implement, but I'm not sure what the best way to go about doing this is, since the callback that handles play/pause events from headset remotes also handles play/pause events from other sources (like keyboards).

What about using KeyEvent.KEYCODE_HEADSETHOOK ?
I see you have omitted RemoteControlReceiver completely. Won't it affect playback?

I've looked into MediaSessionCompat a bit more, and I've come to the following conclusions:

RemoteControlReceiver was being used, but only on devices running API 19 and lower (which I hadn't tested before removing it). As of right now, the service crashes when it starts on pre-Lollipop devices, because MediaSessionCompat throws an IllegalStateException when it can't find a BroadcastReceiver that handles android.intent.action.MEDIA_BUTTON actions.
Android only allows one BroadcastReceiver for media buttons to be active (because pressing the play button on 10 media players would be very bad). It determines which receiver to send the broadcast to based on when it was registered, favoring the most recently added. Since MediaSessionCompat creates this receiver automatically on Lollipop and up using the MediaSessionCompat.Callback implementation, the solution you linked to wouldn't apply on API 21+.
Jockey's current use of MediaSessionCompat isn't as complete as it could be, and doesn't take advantage of a lot of extra functionality and free boilerplate code in the support libraries.

I'm working on addressing these points, but they're all intertwined to an extent and affect the service notification a bit, so it's probably going to roll into a somewhat large commit.
I'm not quite sure where KeyEvent.KEYCODE_HEADSETHOOK fits into this, because Android appears to treat it the same as KeyEvent.KEYCODE_MEDIA_PLAY_PAUSE -- and from the tests I was able to perform, the latter is the event sent from inline remotes. From what I've seen, though, the link you posted is how several others have accomplished this before the APIs added in Android 5.0.

I've created the branch m/media-session-compat to be able to work on the MediaSessionCompat implementation without affecting master. It should fix the crash on API 19 introduced in 28c655a, but it introduces a few problems of its own. Most notably, a0f50ea started the migration of the service notification to use NotificationCompat.MediaStyle, which has temporarily broken the ability to stop the service.

If u are still working on this app..Here's an implementation of headset click that you can have a look into
It seems you ave only implemented ACTION_UP and not ACTION_DOWN
https://github.com/CyanogenMod/android_packages_apps_Eleven/blob/cm-12.0/src/com/cyanogenmod/eleven/MediaButtonIntentReceiver.java

Don't worry -- I'm still working on Jockey, but I have a summer internship doing Android app development which doesn't give me as much free time as I had before, so most of the commits to Jockey will happen on weekends. The internship has also introduced me to a lot of libraries and patterns that I hope to implement in Jockey eventually, starting with in m/dagger branch, which sets up Dependency Injection and RxJava to make the in-memory music library easier to work with and simplify a lot of UI logic. In addition, I'm also moving new features into their own branch until the implementation is complete, which you've probably noticed.
With regards to ACTION_DOWN, I don't think that there's anything to implement. Each button press triggers an event when the button is both pressed and released (corresponding to ACTION_DOWN and ACTION_UP respectively). I believe that specifying behavior for ACTION_DOWN would just double-count each event.
The main thing that's left to do on m/media-session-compat is to implement killing the service on API 21 and higher by swiping away the notification. This will leave double and triple click behavior undefined, but I don't think that this can easily be done with the newer APIs.

I guess they are checking the number of clicks or the duration between clicks to determine a double click and skip to next song or triple click to switch to previous song. That's the reason for having a separate block for ACTION_DOWN.

CyanogenMod uses a frameworks_base patch to dispatch those media events to the system upon long-press of volume-up or down (it simulates a previous/next), which often called "Volume Rocker" in their changelogs. This feature is not something that could come from the application itself I'm afraid.

Can you make something out of this?
http://stackoverflow.com/questions/30942054/media-session-compat-not-showing-lockscreen-controls-on-pre-lollipop

:+1: Good link @salvator91. I didn't realize that MediaSessionCompat.Callback had an onMediaButtonEvent(Intent) method -- I thought it was all abstracted away by the support library.
I've only tested this on a phone with 5.1.1, but the following code appears to do the right thing:
@Override
public boolean onMediaButtonEvent(Intent mediaButtonEvent) {
    KeyEvent keyEvent = mediaButtonEvent.getParcelableExtra(Intent.EXTRA_KEY_EVENT);
    if (keyEvent.getKeyCode() == KeyEvent.KEYCODE_HEADSETHOOK) {
        Log.i(TAG, "Headset button pressed");
        // TODO: Count clicks and post delayed runnable
        return true;
    } else {
        return super.onMediaButtonEvent(mediaButtonEvent);
    }
}

Just uploaded a commit to m/remote-actions that should implement this. Let me know if it's working as you'd expect.

Great. Does this work on API < 21 ? I have an API 19 device to test. Hope it does work

Actually this isn't working. Playback itself has stopped for some reason. I am testing on API 19. Why have you done away with all the BroadcastReceivers in Playerservice?

Can you be more descriptive about what's not working? I just tested this in an API 19 emulator and it appears to be working as intended.
The reason that the broadcast receivers were removed, by the way, is because they weren't needed. MediaSessionCompat includes a lot of this implementation for free, which is why I switched to it. The one thing that isn't handled by MediaSessionCompat, however, is headphone disconnects, which are still responded to with a BroadcastReceiver in MusicPlayer.
On API 21+, MediaSessionCompat can directly register our MediaSessionCompat.Callback with the system, forgoing BroadcastReceivers all together. On API 19 and below, Jockey's Android Manifest specifies android.support.v4.media.session.MediaButtonReceiver as a BroadcastReceiver for android.intent.action.MEDIA_BUTTON actions. The support library's implementation looks for services with the same intent filter, and forwards Intents along to the service's onStartCommand() method. When we get this Intent, we actually just pass it right back into MediaSessionCompat by calling MediaButtonReceiver.handleIntent(), which figures out which method in our MediaSessionCompat.Callback to use for the KeyEvent that just happened, and fires it back to us to handle the event. The result of this is that we don't have to do anything with BroadcastReceivers ourselves to handle media Intents -- the support library takes care of all of it -- hence the lack of BroadcastReceivers in PlayerService. If you're interested, Ian Lake gave a presentation about MediaSessionCompat where he actually talks about this around 14:30

I've gone ahead and merged this from m/remote-actions into master with 05170f4. If there are other issues related to playback, please feel free to start a new issue.


Add options to leave spoilers in the chapter instead of moving them

Leech gives you the option to include or not include content in spoiler tags but if they are  included they are moved to a footnote at the end of the file. Some story have important information in spoiler tags so fliping back and forth to that footnote page is anyoing.
While I use a modified version of the program that just comments out the clean spoilers line a better solution would be to add an option to do this.

The current experience works out okay for me because on a Kindle the footnotes are set up to trigger footnote-popups so I don't need to jump around.
That said, an option to "clean" them by leaving them inline seems reasonable -- do you think they need some sort of markup added to indicate which bits of the content were previously hidden-as-spoilers, or would just unwrapping them and leaving them inline be sufficient?

I'm thinking that I might change the current --[skip|include]-spoilers flag into --spoilers=[footnote|inline|skip]. (I guess technically I could do endnote as well, to put them at the end of the current-chapter.)

The current experience works out okay for me because on a Kindle the footnotes are set up to trigger footnote-popups so I don't need to jump around.
That said, an option to "clean" them by leaving them inline seems reasonable -- do you think they need some sort of markup added to indicate which bits of the content were previously hidden-as-spoilers, or would just unwrapping them and leaving them inline be sufficient?

That all depends on the story sometimes just unwrapping it and putting it inline is enough other times you might need to know the spoilers title or that something was in a spoiler it all depends on what spoilers are used for so maybe make that an option as well.


docs: add indiekit-preset-fff plugin

Love it! Congrats on being the first third-party plug-in to be listed in this project’s plug-in directory 🚀


undefined method `per' for #ActiveRecord::Relation:0x007fc43bcc65a8

When I click a model name, it showed the error "undefined method `per' for #ActiveRecord::Relation:0x007fc43bcc65a8". Thanks for your help.

This helped me: http://tech-brains.blogspot.in/2012/11/kaminari-willpaginate-incompatibility.html


Show key-value component delete button for single row

Allow delete icon even if there is only 1 key-value pair.

This didn't work because of the remaining x-show on the button cell... did you test this? Will merge with my fix but please test stuff in the future when submitting a PR


PTFE-772 generate valid name

Validate the runner group name to ensure a valid runner name when generating it.

Codecov Report

Merging #369 (90f19b4) into main (1c59d6b) will decrease coverage by 19.80%.
Report is 1 commits behind head on main.
The diff coverage is 88.88%.

@@             Coverage Diff             @@
##             main     #369       +/-   ##
===========================================
- Coverage   85.87%   66.08%   -19.80%
===========================================
  Files          27       27
  Lines         906      911        +5
===========================================
- Hits          778      602      -176
- Misses        128      309      +181

Flag
Coverage Δ

api
66.08% <88.88%> (+0.40%)
:arrow_up:

unit
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

runner_manager/models/runner_group.py
62.73% <88.88%> (-29.58%)
:arrow_down:

... and 9 files with indirect coverage changes


Pointing simulation show wrong completion rate

Describe the bug
When a target list only contains medium-resolution is uploaded, the calculated completion rate for medium-resolution becomes "nan", while that for low-resolution is not zero.
Expected behavior
The completion rate for medium-resolution should be calculated properly, while that for low-resolution should be zero, nan, or something else that shows no low-resolution observations is in the target list.
Screenshots

Computer Environment:

OS: macOS 12.7.1
Browser: Chrome 119.0.6045.159

duplicate of #124 and will be fixed by #128


feat(freebsd): FreeBSD 13.0-RELEASE compat

Fix Makefile to reference correct version of git2go for FreeBSD
13.0-RELEASE
Fix scm/git.go imports to reference correct version of git2go for
FreeBSD 13.0-RELEASE
Fix scm/git.go:301 git2go TreeWalkCallback type changed return type
'int' -> 'error'
Fix util/test.go imports to reference correct version of git2go for
FreeBSD 13.0-RELEASE

Tags: 13.0-RELEASE FreeBSD

This isn't ready to merge yet (IMO).
I need to understand more about your CI/build solution to get a feel for what else needs to be done.
Also need to add some logic so as to not break existing things that work (pretty sure this would break Linux builds as it sits)


Indexing/element access on super avoids instance property checks

Discovered in conversation at https://github.com/microsoft/TypeScript/pull/55892#discussion_r1339658106
class C {
    yadda = () => {};
}

class D extends C {
    badda() {
        super["yadda"]();
    }
}

new D().badda()

Currently there is no error for this code; however, if you switched it to a property access, you'd get the error introduced at https://github.com/microsoft/TypeScript/pull/54056.
class C {
    yadda = () => {};
}

class D extends C {
    badda() {
        super.yadda(); // ❌ errors, as expected
    }
}

I imagine that there are also other bugs here. checkElementAccessExpression doesn't end up checking if it's accessing on super at any layer (from what I can see quickly) and checkPropertyAccessibilityAtLocation can produce 2 other diagnostics as well (within the isSuper branch)