Couldn't acquire D-Bus name: org.bluealsa from alpine container

Hi there,
I have a classic dbus issue using bluealsa.
bash-5.0# bluealsa
bluealsa: E: Couldn't acquire D-Bus name: org.bluealsa

I take a look from the other case, think I'm right but would like to clarify
I run an alpine linux 3.11 container with alsa / bluez / bluealsa installed on it
This container live on top of BalenaOS on a Raspberry Pi 3 A+
(I have a topic on it here : https://forums.balena.io/t/bluetooth-configuration-with-alpine-linux/113733)
I think my bluetooth conf is good. From bluetoothctl I'm able to pair a device :
bash-5.0# bluetoothctl
Agent registered
[bluetooth]# show
Controller B8:27:EB:14:A6:97 (public)
        Name: 5fc958d
        Alias: chambre
        Class: 0x00000000
        Powered: yes
        Discoverable: yes
        DiscoverableTimeout: 0x00000000
        Pairable: yes
        UUID: Generic Attribute Profile (00001801-0000-1000-8000-00805f9b34fb)
        UUID: A/V Remote Control        (0000110e-0000-1000-8000-00805f9b34fb)
        UUID: PnP Information           (00001200-0000-1000-8000-00805f9b34fb)
        UUID: A/V Remote Control Target (0000110c-0000-1000-8000-00805f9b34fb)
        UUID: Generic Access Profile    (00001800-0000-1000-8000-00805f9b34fb)
        Modalias: usb:v1D6Bp0246d0532
        Discovering: no
Advertising Features:
        ActiveInstances: 0x00
        SupportedInstances: 0x05
        SupportedIncludes: tx-power
        SupportedIncludes: appearance
        SupportedIncludes: local-name
[NEW] Device 6C:29:95:7C:B3:01 Chromebook_3036
[CHG] Device 6C:29:95:7C:B3:01 Connected: no
[CHG] Device 6C:29:95:7C:B3:01 Paired: yes
[bluetooth]# info 6C:29:95:7C:B3:01
Device 6C:29:95:7C:B3:01 (public)
        Name: Chromebook_3036
        Alias: Chromebook_3036
        Class: 0x00480104
        Icon: computer
        Paired: yes
        Trusted: no
        Blocked: no
        Connected: no
        LegacyPairing: no

dbus daemon is runing on the host and from my container it is working I have a reply when I try this :
bash-5.0# dbus-send --system --print-reply --reply-timeout=2000 --type=method_call --dest=org.freedesktop.timedate1 /org/freedesktop/timedate1 org.freedesktop.DBus.Properties.GetAll string:"org.freedesktop.timedate1"
method return time=1589529332.400402 sender=:1.280 -> destination=:1.279 serial=4 reply_serial=2
   array [
      dict entry(
         string "Timezone"
         variant             string ""
      )
      dict entry(
         string "LocalRTC"
         variant             boolean false
      )
      dict entry(
         string "CanNTP"
         variant             boolean false
      )
      dict entry(
         string "NTP"
         variant             boolean false
      )
      dict entry(
         string "NTPSynchronized"
         variant             boolean true
      )
      dict entry(
         string "TimeUSec"
         variant             uint64<PHONE_NUMBER>399850
      )
      dict entry(
         string "RTCTimeUSec"
         variant             uint64 0
      )
   ]

But when I launch bluealsa I get the Couldn't acquire D-Bus name: org.bluealsa error
I think the /etc/dbus-1/system.d/bluealsa.conf should be in the host BalenaOS.
It is not of course and the filesystem is lock so I can't write it there.
The strange part is when I try from another container (debian strech) it is working but the bluealsa version are different
alpine linux :
bash-5.0# bluealsa --version
2.1.0

debian :
root@5fc958d:/# bluealsa --version
1.3.0

I also try to grep some info from dbus-monitor on the BalenaOS host :
root@5fc958d:~# dbus-monitor --system | grep org.bluealsa
   string "org.bluealsa"
   string "Connection ":1.343" is not allowed to own the service "org.bluealsa" due to security policies in the configuration file"

Look like obvious that I can't use org.bluealsa bus service
But when I lauch bluealsa from the debian container (bluealsa 1.3.0) I don't have any issue on dbus-monitor. In fact I can't fing any log
Do you have any explanation ? maybe using another bus service ?
I don't know very well dbus. Can I use another one with bluealsa ?

There was a recent issue regarding use of bluealsa with Docker, which the user resolved. There may be something useful in the links that were provided there:
#313

Hi !
I will take a look soon. I don't start to play with bluealsa-aplay yet
I confirm that org.bluealsa wasn't authorized from the BalenaOS host
From the Balena forum I figured out how to enable RW on the filesystem and add the /etc/dbus-1/system.d/bluealsa.conf
I'm able to start /usr/bin/bluealsa -i hci0 -p a2dp-sink &
Next step alsa :)

finaly everything work perfectly :)
/usr/bin/bluealsa-aplay --profile-a2dp --pcm-buffer-time=1000000 00:00:00:00:00:00 &
sound perfectly ^^


Union of Sibling Classes

How do I use Union properly?
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.3
            pydantic compiled: True
                 install path: /usr/local/lib/python3.7/site-packages/pydantic
               python version: 3.7.4 (default, Sep 12 2019, 15:40:15)  [GCC 8.3.0]
                     platform: Linux-4.19.76-linuxkit-x86_64-with-debian-10.1
     optional deps. installed: ['typing-extensions', 'email-validator']

from typing import Union

from pydantic import BaseModel as PydanticBaseModel

class BaseModel(PydanticBaseModel):
    class Config:
        allow_mutation = False
        case_sensitive = True
        use_enum_values = True
        anystr_strip_whitespace = True

class VendorA(BaseModel):
    name = "VendorA"

class VendorB(BaseModel):
    name = "VendorB"

class VendorTask(BaseModel):
    vendor: Union[VendorA, VendorB]

tasks = [VendorTask(vendor=VendorA()), VendorTask(vendor=VendorB())]

print(tasks)

Output is:
[VendorTask(vendor=VendorA(name='VendorA')), VendorTask(vendor=VendorA(name='VendorB'))]
What's the best way of achieving the desired result?

This is because VendorB as a dict {'name': 'VendorB'} is valid as input to VendorA.
Solutions:

use Literal on name to force that valid which will allow pydantic to differentiate between an instance of VendorA and VendorB
wait for #619 to be implemented, AFAIK no one is working on it though
perhaps in v2 we'll change how sub-models are parsed, to avoid this problem


Imported library null fix aug18

Just a null error bug fix, plus preparation for publishing it as 2.0.4.

I'll land this because it's so tiny, and it would be nice to fix the bug and have a fast response to the submitter of https://github.com/dart-lang/reflectable/issues/145.


Update test_run.sh

Can one of the admins verify this patch?

lol no

ok to test

retest this please

ok to test


Chore add display memory_limitation_per_thread_for_schema_change_bytes on f…

…ailed status msg
Proposed changes
add display memory_limitation_per_thread_for_schema_change_bytes on failed status msg

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


/btw neutral

BTW Version: 2.1.8
Towny Version: <IP_ADDRESS>
Issue: /btw neutral currently doesn't do anything

It's a discontinue sub-command. Fixed with the next update.

It's a discontinue sub-command. Fixed with the next update.


WIP ZEPPELIN-3432 Fix results object when throwing exception while running job

What is this PR for?
Handling the issue that InterpretJob.jobRun() throws an InterpreterException
What type of PR is it?
[Bug Fix]
Todos

[x] - Change the logic to create InterpreterResult when throwing InterpreterException

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3432

How should this be tested?

Enable zeppelin.spark.sql.stacktrace option
%sql select invalid query

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

LGTM

I've tested it but found it didn't show proper messages. I'll fix it until this weekend

@jongyoul Sorry, it seems my mistake in #2926 , revert the changes in RemoteInterpreterServer.java should be able to fix this.

@zjffdu Thanks. I've checked that issue and added the original logic to Job class as it looks more clear to handle it in one class. After merging this PR, others' classes don't have to care the case that results is not InterpreterResult class.

LGTM Thanks @jongyoul

@zjffdu Thanks. I've fixed tests for adopting changes. I'll merge it after CI finishes.

@zjffdu I've changed not to unwrap InterpreterException as I've found some problems when I run an unavailable script at the first time. I need to check it finally and will tell you the results

I've checked this PR finally.

Will merge it today.


[WIP] Fill in content for additional resources tab

Closes #1698

I see that when you click on the contributor or corporate license agreements items in the left nav, PDF files automatically open up outside the browser. Is there an icon we can use in the left nav to indicate that? A download icon or at least the external link one? What do you think @Eric-Arellano @gracelindsell

Is there an icon we can use in the left nav to indicate that? A download icon or at least the external link one?

Yeah, I was hoping the external link icon (up-right arrow) would Just Work. I have to figure out how to get it to work. I'll open an issue in closed source.

Since we aren't doing migration guides yet, I think the only piece missing is the content and metadata for the open-source intro file (and a review of this PR).

I've moved the responsible-quantum-computing file into this PR to streamline efforts. @javabster approved that file in #1670


Add-TeamChannelUser -Role owner fails on shared channel

Running:  "Add-TeamChannelUser -groupid $teamid -DisplayName $kanaal -User $eigenaar -Role Owner" on a shared channel fails.

Channel is newly created
ChannelUser is already created before promoting to owner

Error message:
Add-TeamChannelUser : Error occurred while executing
Code: BadRequest
Message: $filter query is not supported while fetching members of shared channels.
InnerError:
RequestId: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx
DateTimeStamp: 2022-12-15T10:00:44
HttpStatusCode: BadRequest
At line:1 char:1

Add-TeamChannelUser -groupid "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxx" - ...

  + CategoryInfo          : NotSpecified: (:) [Add-TeamChannelUser], ApiException
  + FullyQualifiedErrorId : Microsoft.Teams.PowerShell.TeamsCmdlets.ErrorHandling.ApiException,Microsoft.Teams.PowerShell.TeamsCmdlets.AddTeamChannelUser

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

ID: cd91d571-ca88-5766-2666-d6a4d34cdbed
Version Independent ID: 2481b258-b593-e323-90f0-e9ca6e7797bf
Content: Add-TeamChannelUser (MicrosoftTeamsPowerShell)
Content Source: teams/teams-ps/teams/Add-TeamChannelUser.md
Service: teams-powershell
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

Microsoft Case # 33149197
I submitted this case about this exact issue on 9/29.
The issue is also on Remove-TeamChannelUser

Tested in 4.9.1 and it works, closing as here we are limited to documentation issues.
Thank you


Issues performing .get() to CTA_FILE_URL from behind internal firewall

Our model currently uses v1.0.5 and we are struggling to update to v1.0.6+ because the requests.get() introduced in 2c9ca5d24b834db65910324abb2886e650b6045e to get the CTA_FILE_URL from behind internal firewall fails. We can go through a semi-lengthy exemption process but below changes will make it simpler and generally safer.
this issue to discuss some possible changes

[ ] simpler/safer to return a JSON response from the API and not full Excel file (this eliminates risk of 'bad' macros in XLSX file)?
[ ] a fallback in cases where requests.get fails supported?
[ ] support ability to pass in verify= param to the request.get?

Hi and thanks for your input. I will look into the possbility of using JSON - if the SBTi has this possbility. A fallback will be introduced so that we check for a status code of 200, and if this fails we'll fetch the file included in the package instead. It might not be the very latest but I think it will be possible to manually load the file to that directory if the user can access it in some other way than through the tool.
As for the verify parameter, I believe it is set to True by default.

thanks
JSON response would definitely be simpler for us to perform the request.get but i realise you would need to make a change to how the file is hosted and served!
shall I knock up a PR to add the targets.xlsx file to the GH repo and read from local if the request.get fails? I would just put the current file into

manually load the file to that directory if the user can access it in some other way than through the tool

suggestions for where could get user to load the file to?
as for verify param: our company decrypts our HTTPS traffic and would need verify to point to our trusted root cert. And I have realised that I may be able to just use the REQUESTS_CA_BUNDLE env var to point to this certificate >>
from https://requests.readthedocs.io/en/latest/user/advanced/#ssl-cert-verification

Well here's some good news. There will soon be a new api with the CTA delivered in JSON format. I don't have a data yet, but the code to update the tool is being prepared.
The change will include a test for status == 200 and it it fails we'll read the file target-dashbard.xlsx from the directory STBi/inputs.
And as for verify, is it enough for you if we add verify=True to the .get()? That's no problem from my side.

thats great thanks - shall we leave this issue open until the API and check if status == 200 PR has been opened so can link the PR and issue?

And as for verify, is it enough for you if we add verify=True to the .get()? That's no problem from my side.

We should be able to just use the REQUESTS_CA_BUNDLE env var in our model execution environment thanks, I should have found that earlier!

Yes, let's leave it open until the API is ready. But I will add the status == 200 and perhaps leave an option to read the CTA file from the directory "data" that is created when running the tool in Colab. (If you run it in your own Jupyter environment you should be able to see the CTA file anyway.)

thanks for changes in #306 and 8426dc83a5bea6c0646435bf967d3c2daf505ea5 to add local CTA file backup.
unfort we are still facing exceptions so wrapped the requests.get in try/except in #308.
your review plz?


Merge common code

After these are completed:

[ ] #23
[x] #27
[x] #85
[ ] #96

consider these:

[ ] combine AdminSeeder class and RoleSeeder class to one
[ ] extract common code in GlobalExceptionHandler

Note: check before refactoring for the latest changes and future developments and review what and if it is still needed

@jcilacad Review my suggestion please and assign this issue to me if you agree or close it as not planned in case there is no need for that
Thanks

@panagiotisbellias  go for it.

Done


Update outdated viewport docs

For https://github.com/uber/deck.gl/issues/4280

Coverage remained the same at 83.298% when pulling e216ecee5f551e459a2cf986bcbc5a1419e4303b on x/web-mercator-viewport-docs into 9fc27f9cb1dc8d188a4bc73d99e2127e227f0f7c on master.


[Bug] Install-DbaFirstResponderKit installs the deprecated versions instead of the updated versions

Report
When installing or updating FirstResponderKit from local file or from default latest version, the Deprecated folder in the zip file are included and overwrite updates.
Host used

[X] powershell.exe
[X] ISE
[ ] VS Code
[ ] Other (please specify)

Errors Received
Steps to Reproduce
 Install-DbaFirstResponderKit -SqlInstance localhost -Database master -Verbose
Expected Behavior
Install should not include scripts listed in the Deprecated folder, which have _2005.sql appended to their filenames by default.
Actual Behavior
VERBOSE: Performing the operation "installing/updating sp_Blitz_SQL_Server_2005.sql in master." on target "localhost".
VERBOSE: Changed database context to 'master'.
SqlInstance  : localhost
InstanceName : MSSQLSERVER
ComputerName : VM-DWSQL01
Name         : sp_Blitz_SQL_Server_2005
Status       : Installed
Database     : master

Environmental information
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨
Please provide the output of the below script
dbatools latest installed : 1.0.124
Culture of OS             : en-US

SQL Server:
Microsoft SQL Server 2016 (SP2-CU15) (KB4577775) - 13.0.5850.14 (X64)   Sep 17 2020 22:12:45   Copyright (c) Microsoft Corporation  Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2012 R2 Standard 6.3 <X64> (Build 9600: ) (Hypervisor)

us_english

Maybe you have an old local copy including those files (there was a bug some versions ago).
Could you rerun the command with -Force to update the local cache?

I can't replicate this on version 1.0.125
Install-DbaFirstResponderKit -SqlInstance mssql1 -Database master -Verbose

Then checking which stored procs got installed:

I can't replicate this on version 1.0.125
Install-DbaFirstResponderKit -SqlInstance mssql1 -Database master -Verbose

Then checking which stored procs got installed:

It could have been fixed (I ran it on 1.0.124) but the deprecated versions are also named sp_blitz, sp_blitzcache, and sp_blitzindex.

When I ran it on 1.0.131, I no longer encounter the error.


Re-enable 'testModuleAliasingPrebuiltWithScanDeps'.

It appears the test setup resulted in us being unable to find modules when passing them as explicit dependencies to depending compiler invocations. Setting an explicit module cache path helps with that.
Resolves rdar://88073675

@swift-ci test


build bert: build does not load model

System Info
a100
Who can help?
@byshiue
Information

[x] The official example scripts
[ ] My own modified scripts

Tasks

[x] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
My bert model is model.safetensors. But examples/bert/build.py cant read it.
I changed this part of code and worked:
if args.model_dir is not None and os.path.exists(
                os.path.join(args.model_dir, "pytorch_model.bin")):
            state_dict = torch.load(
                os.path.join(args.model_dir, "pytorch_model.bin"))
            hf_bert.load_state_dict(state_dict, strict=False)

to
if args.model_dir is not None and os.path.exists(
                os.path.join(args.model_dir, "pytorch_model.bin")):
            state_dict = torch.load(
                os.path.join(args.model_dir, "pytorch_model.bin"))
            hf_bert.load_state_dict(state_dict, strict=False)
elif args.model_dir is not None and os.path.exists(
                os.path.join(args.model_dir, "model.safetensors")):
            state_dict = safetensors.torch.load_file(os.path.join(args.model_dir, "model.safetensors"))
            hf_bert.load_state_dict(state_dict, strict=False)

Expected behavior
read model
actual behavior
not read model
additional notes
nothing

fixed together with your other issue https://github.com/NVIDIA/TensorRT-LLM/issues/2373. Thank you!!

@symphonylyh
Note: the change only covers the case elif args.model == 'BertForSequenceClassification' or args.model == 'RobertaForSequenceClassification'. The if args.model == 'BertModel' or args.model == 'RobertaModel' case remains buggy. Please have another look at #2187, there is a fix for the latter case (the corresponding bug: #2197).

@symphonylyh  Here you said that you were planning to merge the PR


CIS Alarms not working when using name prefix

Description
CIS Alarms not triggered due to wrong metric name when using name_prefix or use_random_name_prefix pet names
Versions

Terraform: v1.0.7

Provider(s): registry.terraform.io/hashicorp/aws v3.66.0

provider registry.terraform.io/hashicorp/external v2.2.0
provider registry.terraform.io/hashicorp/local v2.1.0
provider registry.terraform.io/hashicorp/null v3.1.0
provider registry.terraform.io/hashicorp/random v3.1.0

Reproduction

Steps to reproduce the behavior:

Setting name_prefix variable
Code Snippet to Reproduce

module "cis_alarms" {
   source  = "terraform-aws-modules/cloudwatch/aws//modules/cis-alarms"
   version = "~> 2.3"
   alarm_actions     = ["your_sns"]
   name_prefix       = "test-"
}

Expected behavior

Expect that Cloudwatch Alarm to change state when metric changes
Actual behavior

Metric in cloudwatch alarms are empty an no alarms are triggered, as the prefixed name is different from the cloudwatch metric name
Terminal Output Screenshot(s)

Additional context

related to #14

This issue has been resolved in version 2.3.1 :tada:


Discontinue evm feature

This discontinues the evm feature available in crates within the workspace. It also removes chain-evm from the workspace. The last commit with the crate is stored as the git tag: https://github.com/input-output-hk/catalyst-core/tree/chain-evm-discontinued
This reduces the build time considerably.

@jmgilman any idea why the nix CI test for jormungandr is failing?

@saibatizoku Should we create a tag or branch of the system with the state before evm things were removed monocle_face?

Yeah, @minikin , I added a tag: https://github.com/input-output-hk/catalyst-core/tree/chain-evm-discontinued

Sorry I missed the ping on this. We build with prometheus by default because we rely on it for metric scraping. It appears something is broken when that feature is enabled.


Ability to open declaration/body package in a separated and large SQL Editor

Would be great open a body or declaration package in a large editor text to see more code.. Thanks

Makes sense. I need to think about possible solution (open in a separate editor, maximize editor pane in current editor or even open in external editor)..

"Open in SQL console" action will be added in the next version.


Documentation: This Repo has the potential to rule the further development of 3d printing

I really think, this repo has the potential to rule the further development of 3d printing. There are several projects out there to  add several axis to existing printers or build 5 or 6 axis printers, but all lack of a slicer.
But the documentation is lacking of information. After several tries I was able to build  it successful on ubuntu. So it would be really cool to upgrade the documentation about building it.
But now, I honestly don't know what type of file I can open in the slicer and how to convert an OFF file to a TET file on ubuntu. I also don't know what files I can open in the slicer. I thought it can use STL, but now stumbled upon TET files.
Would it be possible to improve the documentation? I would like to include my workflow how I built it on ubuntu.
I'm sure, that you and the work you have done, can be the next step for the open source community to move to more axis.
Kind regards

Hi all, I wrote a note describing the total steps for the tetrahedron model generation and the stress field generation (input) for the strength reinforcement case. please check the attached instructions.
Generation of Stress field form Abaqus.docx


Respect the to_field setting and the target field type

https://github.com/treyhunner/django-simple-history/issues/182
Hope this is enough tests? :)

I guess not :D

Is the code coverage test performed on a certain version of django? if that's the case then I can't really fix it.. the pre-1.6 code version would work in later versions of django too (getting the to_field from rel.field_name), but it's not really the right approach, I think at least?
Let me know your thoughts :) thanks

Coverage should be cumulative. I'm not sure why the commit requirement is 95%, that's pretty high. I wonder if the assumption was that it's requiring 95% coverage of the new lines being added. Anyway, those are new so I wouldn't worry too much. I'll try to look at this later this week.

Yeah I'm pretty sure it's counting the new lines - since the coverage test runs on django 1.4, 1 line out of 6 new ones that is not applicable to django 1.4 is not covered by the coverage test.. (5/6 = 83.33) :)
and thanks!


canvas and svg subtypes

Described in README.md

compiler.js@377 looks like a bug.
Merging onto branch for further investigation.


Examples of delayed slash command response with Slack

Hi
Is there any documentation or examples on how Botkit API can be used to respond to Slack, if a slash command takes more than 3 seconds?
I have a slash command that is working with a Slack bot. Before returning a response, the bot communicates with an external server (Response of the slash command depends on what the external server returns). Most of the time, the external server responds fast (within a second). Sometimes, the external server takes a long time (around 20 seconds). At that time, Slack channel shows  "darn – that slash command didn't work. Feel free to give it another go" error message.
My slash command logic is as follows:
case /my_slash_command:
external_response = contact_external_server; // Sometimes fast, sometimes slow
bot.replyPrivate(msg, "Private Message", function () { bot.replyPublicDelayed(msg, "Public message")})
I also tried variations like this one below, but not seeing any change in Slack behavior
case /my_slash_command:
bot.replyPrivate(msg, "Private Message")
external_response = contact_external_server; // Sometimes fast, sometimes slow
bot.replyPublicDelayed(msg, "Public message")

Got solution from this thread - https://github.com/howdyai/botkit/issues/595


introduced numbering for classification and added 3.8

Introduce numbering to be able to make references

a researcher may reference an item why he/she thinks an issue is a vulnerability
an organization may respond with a reference why a reported issue may not be a vulnerability

Also make clear that accessing public information is not considered a vulnerability.

Thank you for the PR. I totally missed it.


Add continuous delivery

:D

Removing the "homepage": "https://rsksmart.github.io/rif-identity-manager/", attribute breaks the original GitHub deployment that we are using for testing.

Yes.
But since yestarday we are using staging.identity.rifos.org


mac-10 lost external connection phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-10 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting machine to restore connection.


🎨 Add Ansible brand colors to the docs theme

This uses the Ansible pool logo for a splash of color, and a different color for the left-hand navigation to make it stand out a bit more.
Preview: https://awx-plugins-core--35.org.readthedocs.build.

@samccann do you think these are all the changes that would be necessary in a new theme?
I checked the color contrast with DevTools (against the background) and it's good for the dark theme variant but not ideal for the light one. Do we have a darker logo variant that would satisfy the accessibility recommendations?

So the goal here was to make as few changes as possible for now. There could be more work if we really want to customize this as he theme has a lot of flexibility on playing with colors.  I thought the Ansible pool logo had the splash of Ansible color that people could recognize.
As for the nav/toc change - I found the allwhite navigation harder to notice so tried to make that a different color that still worked with existing font colors etc.

@samccann I also noticed that the logo size got smaller. Is that on purpose?

So Firefox's accessibility tool reports contrast problems with the ToC titles in the light theme variant. It needs to be fixed before the change is acceptable UX-wise. I'd like to also compare with what Lighthouse says.

Logo size - yes that was deliberate. The existing logo imo is too large. That said, since this is awx-related, do we want to use the awx logo?

That's a good question. Personally I liked the original black-and-white variants. Anyway, I don't think I should be deciding which logo to use beyond checking the accessibility implications that I'm doing.
cc @chrismeyersfsu @TheRealHaoLiu @thedoubl3j @djyasin plz share your thoughts on this


BibTeX: add unpublished type for preferred-citation.

Coverage remained the same at 100.0% when pulling e5ae522a3eb13443f4fee45dbd2eaa14b1cb3ae0 on add-unpublished into 83f76e2c349b2af7ec2ad15cc2a9ba25462be5d3 on main.


#1362 - Fix typo in tagline for the Spanish translation

Fixes #1362
Changes:
Fixed the tagline6 that was replicated with tagline5 in the Spanish translation.

@limzykenneth, could you please review it?
Also marking the Spanish translation stewards: @Guirdo, @Andreu-G, @holomorfo

Looks good. Thanks!


Show power usage of the OSR in the main README

Given that we recommend a 4S LiPo battery, might this be the adequate place to add a small warning about not discharging your LiPo battery too far? While the dropout regulator does need more than 12V, the battery can still be drained to levels that are detrimental to battery health

good point, added that warning in!

Looks good!


Catch does not exist error

There was a new line at the end of does not exist
which was causing this to fail.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@rh-atomic-bot r+


proposal: cmd/vet: consider reporting invalid/ignored Go versions in build tags

Files may specify their own Go language version via a build tag. For instance:
//go:build go1.22

indicates that this file requires Go language version 1.22.
If the language version in the build tag contains dot-release information, say it's "go1.22.1", it is invalid.
The compiler simply ignores the version (it doesn't use, say "go1.22").
It seems (to me) that this shouldn't be silently ignored as it might be very confusing for users. go vet might be the right tool to report an error. Alternatively, the compiler could report an error, but generally, the compiler doesn't complain about errors in (pragma and build) comments.

go/analysis/passes/buildtag would be a good home for this check.

@griesemer - Kindly assign me for this.  How can I assign myself and update labels NeedsInvestigation, NeedsFix like that.
 If a specific source file should be compiled only when using a newer Go toolchain, adding //go:build go1.22 to that source file both ensures that only Go 1.22 and newer toolchains will compile the file and also changes the language version in that file to Go 1.22.
Blog:
There is also a new toolchain line in go.mod that specifies the minimum Go toolchain to use when working in a particular module. In contrast to the go line, toolchain does not impose a requirement on other modules. For example, a go.mod might say:

module m
go 1.21.0
toolchain go1.21.4

This says that **other modules requiring m** need to provide at least Go 1.21.0, but when we are working **in m itself,** we want an even newer toolchain, at least Go 1.21.4.

My Question : When working in m itself gives me a meaning like //go:build go1.21.4 - specific work is happening there for some  of those files. But other modules requiring m files still build on go 1.21. So wouldn't patch version needed for build ?

@griesemer  @ianlancetaylor - Do you consider handling tags including expression like go:build go1.21 && go1.22.3 or a single tag ?

@gopherbot - Assign this to me

It should report any Go version string that is silently dropped and ignored.
Also, before anybody starts working on this: note this is a proposal and needs to be discussed by the proposal review committee first. Thanks.

We already have a //go:build line checker. It should definitely report these.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group
The proposal is for the ‘buildtag’ checker to report build tags of the form go1.X.Y since only go1.X is recognized in build tags.

A new check added to vet seems like something that needs to be mentioned in Go 1.24 release notes, and it's not mentioned there yet. Reopening as a release blocker for tracking that.


RTIO Improvements

Several significant improvements to the existing RTIO subsys could make it more useful in common expected use cases:

[x] User mode support #51283
[x] Dealing with a contended iodev/device #51772
[ ] Direct polling for submission completion when interrupts are undesirable (e.g. SOF) Started in #51772
[ ] Test covering rtio_submit to context with pending requests in the queue
[ ] Simplify blocking consume call
[ ] Simplify blocking submit call
[ ] Showcase of DMA usage (e.g. with a SPI driver supporting RTIO)
- [ ] Flags for using DMA transfers (spi txrx)
[ ] Flush Operation (wait on all prior events)
[ ] Timeout Operation (wait on N prior events for T time)
[ ] Deadline Operation (fail subsequent events if time T has passed)
[ ] Samples with real, easily accessible, hardware across multiple devices
[ ] RFC of I2C support
[ ] RFC of SPI support
[ ] RFC of DAI/I2S support
[ ] Equivalent to the io_uring Provided Buffer feature (buffer pool for reading) :new:

@yperess added the provided buffer feature, let me know if that makes sense here or if it could use its own ticket


Add to Readme: how to use external tsc

It took me quite a while now to figure out you use a bundled tsc by default (with version still at 1.6).
After some searching I found a very easy flag to use an external (more recent) tsc, using the following line in a typescript.rb initializes:
TypeScript::Src.use_external_tsc = true

Maybe it would help others to add this option to the readme as it allows for so many up-to-date features? Especially, @types installed via yarn just work then ;-)

This was invaluable.  Thank you!


Title and Link attributes break link to launch model viewer

Description
I have a web page link that launches scene viewer on Android mobile devices. It currently works without the Link and Title attributes. Once I add the Link and Title attributes the link stops working... becomes unclickable. I'm trying to add a "buy tickets" link with title at the bottom of the scene veiwer.
CODE THAT IS WORKING:

CODE THAT IS NOT WORKING:

*I've tried swapping the v to 1.1 as well. ie "/scene-viewer/1.1?file"
Live Demo
BOTH CHUNKS OF CODE FROM ABOVE ARE LIVE AND MARKED ON THIS PAGE
https://www.oceanandcake.com/android-test/
Version

model-viewer: v1.0 and v1.1 (I think)

Browser Affected

Chrome
version: not sure, I don't have an android device.
OS
Android
AR

SceneViewer

@eugenecys @AdrianAtGoogle Do you see any syntax problem with intent above? @mfebrey You said the OS is Android, but you don't have an Android phone - how are you testing this?

@eugenecys @AdrianAtGoogle Thanks for the reply. My client has an Android and have been testing with them. I need to get an Android for this, I know. Brand new to the G side of mobile XR dev.
I don't see a syntax error. Would be most appreciative if you see something and can share.

@mfebrey I think it could be because you're missing the link before browser_fallback_url. I just tried this link and it worked on my end. (Pixel 7)
<a
          href="intent://arvr.google.com/scene-viewer/1.2?file=https://cdn.glitch.global/6c5aca2b-3427-4770-aa49-072a8e63bb2a/COF31.glb?v=1692639141263&title=Color%20of%20Fashion%20-%20Tickets%20+%20Details&mode=ar_only&link=https://www.oceanandcake.com/color-of-fashion#Intent;package=com.google.android.googlequicksearchbox;scheme=https;S.browser_fallback_url=https://arvr.google.com/scene-viewer/web?file=https://cdn.glitch.global/6c5aca2b-3427-4770-aa49-072a8e63bb2a/COF31.glb?v=1692639141263&title=Elroy+Accent+Chair+with+Wood+Legs+Natural+Linen&fdl=1&link=https://www.oceanandcake.com/color-of-fashion-xr-error;end;"
        >

Demo link: https://tar-receptive-gopher.glitch.me
Glitch code: https://glitch.com/edit/#!/tar-receptive-gopher


Get-started section text updated.

Something wrong with my repo. I'm closing and starting over again. Apologies! 👍


Fixes an example in the README

The before and after blocks also take the context in the current version. See MessageSendRecorder protocol "recording - callbacks".

Ah, we should use #cull: to evaluate these blocks ... In many recording scenarios, only the first few arguments may be relevant at all ...

Absolutely true :) I would also keep the updated example though, to document which arguments users can expect.

I just uploaded Kernel-ct.1376 to the inbox. When it gets accepted, we could use #cull:cull:cull:cull:cull: in MessageSendRecorder>>recordInto:receiver:arguments:context:. At the moment, #setRecordAllObjects is broken ...
/cc @marceltaeumel

Here is a changeset that reenables the #setRecordAllObjects functionality again when Kernel-ct.1376 has been loaded:
MessageSendRecorder-cull.1.cs

cull: would be slower. That's why I did not use it.

cull: would be slower. That's why I did not use it.

Did you measure this? :-) If you send cull:{n} to an n-arg block, it should be as fast as value:{n} since the implementation uses the same primitive by default. However, we could also "ceil" the block args in setBefore:after: to combine convenience + optimization. What do you think?

Well, since we do not know the size of the call tree in advance, we want to save every tiny bit we can. The use of cull: did not seem worth. Users can just ignore the argument if they do not need it.


Fix incorrect namespaced controller names in rspec validators

Problem
When Apipie is parsing resources it stores resources descriptions under a unique key in the @resource_descriptions hash in Apipie::Application. When generating such a key for a namespaced controller (e.g. Api::AdUsers::SessionsController) it will generate a name prefixed with the namespace to avoid collisions (e.g. ad_users-sessions). These names are generated using the Application#get_resource_name method. However, the Rspec validation helper does not use this method and instead uses controller_name, which is not prefixed. This causes all match_declared_responses calls to fail when testing namespaced resources.
Solution
Use Application#get_resource_name to generate resource names in the Rspec helper.

@seanich -- can we resubmit this PR in the @instacart org fork of apipie-rails?
https://github.com/instacart/apipie-rails/tree/ic-custom
Our ads-backend should be using that version of the fork now 👍

@seanich -- can we resubmit this PR in the @instacart org fork of apipie-rails?
https://github.com/instacart/apipie-rails/tree/ic-custom
Our ads-backend should be using that version of the fork now 👍

Right! Will do


fixes to ICPointRew

Expressing the initial condition as a percentage of the distance between bounds is more stable and prevents errors during fitting due to invalid initial conditions. See e.g. https://doi.org/10.7554/eLife.36018.001
This same logic applies to ddm.models.ic.ICPoint and ddm.models.ic.ICRange, but I did not modify those classes as I haven't used/tested them.
Thanks for a great package!

Hi ntardiff,
Thank you for your contribution!
I don't want to accept the PR as-is because there are valid reasons for doing the fitting either way, and this change will break code that other people have already written.  However, if you formulate this as a new InitialConditions object (perhaps name it "ICPointRewRatio") and add it as a new, separate option alongside the existing ICPointRew, I would be glad to merge the PR.

So, I found that I couldn't successfully fit with the current ICPointRew, because valid minvals/maxvals for x0 are dependent on the value of the bound height B, which you do not know if you are also fitting B. But maybe I was doing something wrong?

You are correct - if you are fitting bound height, and the maximum x0 is greater than the minimum bound height, then you will get an error.  For this situation, you will want to use something like the version you have.  That's why it would be useful to have both versions!

Cool. I updated the pull request accordingly.

This looks great, thank you!  I noticed one more small thing: the PyDMM convention is that x0=0 means no bias, whereas in your code, x0=.5 means no bias.  I know this convention is different from other packages, but could you please change this in the PR to maintain consistency?  (It should just be a x0 = self.x0/2 + .5 line and then a corresponding adjustment for the reward bias line.)  So -1 < x0 < 1 instead of 0 < x0 < 1.

Sorry, can you clarify what you mean by "a corresponding adjustment for the
reward bias line"? Scaling x0 as you suggested should allow the rest of the
function to stand as before, no? Or did you mean the minval/maxval in the
example instantiation in the doc?
On Thu, Apr 2, 2020 at 5:19 PM Max Shinn<EMAIL_ADDRESS>wrote:

This looks great, thank you! I noticed one more small thing: the PyDMM
convention is that x0=0 means no bias, whereas in your code, x0=.5 means no
bias. I know this convention is different from other packages, but could
you please change this in the PR to maintain consistency? (It should just
be a x0 = self.x0/2 + .5 line and then a corresponding adjustment for the
reward bias line.) So -1 < x0 < 1 instead of 0 < x0 < 1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mwshinn/PyDDM/pull/16#issuecomment-608095212, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGJEIGZOMCLLBSKSWQWUU3RKT6O5ANCNFSM4LX36QSA
.

Sorry, I wasn't very clear. You would have to do -x0 if highreward==False instead of 1-x0.  x0 should take values -1 to 1, with -1 being a bias towards the incorrect target and 1 towards the correct target.  Having 1-x0 means that the bias would take values ranging from 0 to 2 instead of -1 to 1.

Oh, OK. I thought you were suggesting in your previous email email that I
rescale the bias at the beginning of the function to 0 to 1 w/ the line x0
= self.x0/2 + .5 and just keep the remaining code as is, which I think
would work, no? I guess the alternative would be to fix the reward bias
line as you suggest and then do something like this later in the function:
shift_i = int(start*(len(x)-1)/2 + (len(x)-1)/2)
and scale the bias to the scale of the bounds. Is that what you were
thinking?
On Thu, Apr 2, 2020 at 8:30 PM Max Shinn<EMAIL_ADDRESS>wrote:

Sorry, I wasn't very clear. You would have to do -x0 if highreward==False
instead of 1-x0. x0 should take values -1 to 1, with -1 being a bias
towards the incorrect target and 1 towards the correct target. Having 1-x0
means that the bias would take values ranging from 0 to 2 instead of -1 to
1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mwshinn/PyDDM/pull/16#issuecomment-608164931, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGJEICTLCLREJ2U34G6TYTRKUU3LANCNFSM4LX36QSA
.

Oh wait, of course, you're completely right... if you rescale it at the beginning you don't need to change the highreward line.  You can disregard my previous comment about highreward.

Glad we cleared that up. :-) I've pushed the changes. Let me know if you spot anything else.

Thank you for your contribution!  How would you like to be identified in the CONTRIBUTORS.txt file?  I would normally put your full name but your github profile doesn't have a full name listed on it.

Happy to do so! Thanks again for developing the package! Ah, I guess I'm
too incognito--you can put Nathan Tardiff. Thanks again!
On Fri, Apr 3, 2020 at 10:50 AM Max Shinn<EMAIL_ADDRESS>wrote:

Thank you for your contribution! How would you like to be identified in
the CONTRIBUTORS.txt file? I would normally put your full name but your
github profile doesn't have a full name listed on it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mwshinn/PyDDM/pull/16#issuecomment-608478248, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGJEIA2L7CC4PR7ESCL743RKXZT5ANCNFSM4LX36QSA
.

All set!  Thanks again!


Sending session_id in Cookies doesn't work

Hi,
Thanks for your module this makes our work very simpler. I am working in Odoo 13
As mentioned, due to security purpose I sent the session_id in cookies but instead of returning the results I get Odoo Login page as the response (302 error - Redirection) for GET and DELETE Methods and Odoo session expired response for POST
could you please guide me on this issue?

How are you sending your cookie?.

I am also facing same issue, I am using postman to test the api, /auth route is working fine but with other route's I am facing same redirection issue and getting odoo login page as response.
Please find below postman request and Odoo logs
PostmanRequest

Odoo-Logs

Just to add more I am using postaman intercepter for sending cookies

because nested a  web/session/authenticate/ call  inside /auth, this is wrong for Odoo, there is 2 different sessions, each one is auth =none,  and/auth return the  wrong one that has not be authenticated.
BTW.   auth= none should set uid = none in session, means the user in session has not login.
there have 2 solution,
1, dont use /auth, use /web/session/authenticate  instead
2, rewrite /auth , call odoo internal call http.request.session.authenticate  directly
good lucks.


Cover all the areas in the Any% Glitchless route

Cover all routes, towns, etc in the main Any% Glitchless route. This should work even after completing the game, so it's easy to test.

Fixed in https://github.com/daplf/PokemonBDSPAutoMusic/commit/11921a22c9a3939294106e9c2dccd483566d9b9f
I forgot a couple of them, but I will add those later when I tackle the story specific stuff.


Policy documentation generator

A script to our that generates markdown files based on our policy metadata json files.  Will be displayed as part of our documentation

Picking this up @amirbenv :)


[FEATURE REQ] Allow roles specification for WebPubSubConnectionAttribute

Library name
Microsoft.Azure.WebJobs.Extensions.WebPubSub
Please describe the feature.
So the inner workings of the WebPubSubConnectionAttribute makes use of the WebPubSubServiceClient to get the access Uri
as shown here
internal WebPubSubConnection GetClientConnection(string userId = null, string[] roles = null)
{
    var url = _client.GetClientAccessUri(userId: userId, roles: roles);

    return new WebPubSubConnection(url);
}

Although it provides the roles here, in the only reference to this function no roles are provided
private WebPubSubConnection GetClientConnection(WebPubSubConnectionAttribute attribute)
{
    var hub = Utilities.FirstOrDefault(attribute.Hub, _options.Hub);
    var service = new WebPubSubService(attribute.Connection, hub);
    return service.GetClientConnection(attribute.UserId);
}

If it wasn't intented this way I wonder if it could be possible to extend the WebPubSubConnectionAttribute to be able to provide a string[] of roles for them to be used to get the client connection Uri

@Jverschoor Thanks for the feedback! We will review and update accordingly.

@Jverschoor Thanks for providing the feedback.
The issue here is input binding doesn't have good ways to input the array parameters. That is to say, even WebPubSubConnectionAttribute has the roles parameter, user still doesn't have good ways to input the values directly through input binding. And if build the access token with roles inside the function, then there's no big difference to build with WebPubSubServiceClient() directly. So we don't see benefit to introduce this parameter and will confuse customer about how to use it.


Create ashparsh-pandey.md

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 08fbc59f59091bbc638fbcdfcd1cd8d7a7db4dac at: https://6707d7006eb6fd238b6b98d9--meshery-docs-preview.netlify.app

@Ashparshp 🚀 😎


use flatMap from lodash instead of native js

https://sentry.io/organizations/fishbowl/issues/1666187786/?project=5241100&referrer=alert_email

 Use lodash flatmap instead of native (Msft Edge support)


mouse does not working in black desert mobile

phone control and other game is working fine but for this particular game it does not register mouse click

Maybe because the game does not react to mouse events?

Maybe because the game does not react to mouse events?

but when i use right click, it shows a windows asking want to leave the game or not. any add on to solve this?

vysor click working fine for this game

i use right click, it shows a windows asking want to leave the game or not

Because right click sends BACK to the device, not a click.

vysor click working fine for this game

Hmm, interesting.

Ref #635.
Just fixed it on master by 17d53be3ef9d376e48a7f9e90cf6c87cf8fbf8f5.


Allow passing a NumberFormat to Intl.plural which would be used to format the howMany argument

If you want to display the argument in the text of the message and format it in a particular way you would need to either explicitly put a formatter into the message text or else format it first and pass an extra argument. Either is cumbersome.

This issue has been moved to dart-lang/intl#47.


Sync the codebase with the latest version of the Polkadot-JS Apps master branch.

Please update the changelog

Thank you! Yes, I'll do that. Usually we do it in the release branch. You can check it here.


Unit tests for interacting with ECR (Resolver and Fetcher)

Add a bunch of unit tests covering interactions with ECR.  There's a lot here, but each commit should stand on its own and can be reviewed in sequence.
This PR currently covers the Resolver and Fetcher, but not the Pusher.  It can be reviewed now and Pusher tests can be submitted separately.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

cc @nmeyerhans @sipsma
I'd appreciate your review.  Thanks!

@sipsma

There were also a few other files you edited that seem to need an updated year in the license header.

Fixed


Make sure CS employees can manage and create S3 buckets in management cluster AWS accounts

Not relevant anymore.


feature request: add multiple datasets in import modal

The ability to add more than one dataset at a time would be killer :)
I could see a couple of workflows here:

After connecting one dataset, the modal remains open,  the file starts importing, and there is an option to 'add another dataset' so they appear as a list of imports in progress
When clicking connect dataset, a user could select multiple files on their computer and all of them would then be selected to import
From this modal, the user would have the option to bundle datasets into a map once imports are successful

cc @xavijam

cc @CartoDB/design this is a UX / speed gain for me. Being able to upload multiple datasets in one go -- without having to repeat the process of uploading one at a time -- would save time.

totally agree @andy-esch. We'll think about it :). Thanks dude.


Child of Fade with in={false} visible on mount

References #7888

I have a below snippet:
<Fade in={this.state.scrollVisible}>
  <ScrollTopButton />
</Fade>

Initially on load this.state.scrollVisible is false, but ScrollTopButton is visible anyway. After showing/hiding, it starts to work correctly, only initial load with in={false} is broken.
Adding mountOnEnter prop helps, but shouldn't be required.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Child of Fade should never be visible when in={false} on mount
Current Behavior

Use Fade component with in={false} - child will be rendered and visible while it shouldn't.
Steps to Reproduce (for bugs)

Open https://codesandbox.io/s/9ov57l1pv4
You can see that Shouldnt be visible on Load button is visible on load while it shouldn't
Click SHOW button, Shouldnt be visible on Load will still be there, another button mountOnEnter helps but shouldnt be necessery will show.
From now on, you can toggle hide/show, everything works fine, only point 1 is problematic.

Context

Your Environment

Tech
Version

Material-UI
1.0.0-beta.31

React
16.2.0

browser
any

@klis87 Thank you for the reproduction example!

@oliviertassinari Thanks for quick fix and ... decreasing bundle size :D

I'm noticing this same bug again. Could there have been something in the most recent update (v4.10.2) that brought it back?
Context:
Here's the class I've wrapped with "fade":
class Placeholder extends Component {
    constructor(props) {
        super(props);
        this.state = {};
    }
    render() {
        return (

placeholder

        );
    }
}

And here's how it looks in the code:
            <Fade in={false} timeout={1000}>
                <Placeholder />
            </Fade>

But for some reason, it's still showing up upon render.


Can't upload cert using safari

Rancher Version: v1.2.0-pre1-rc4
Steps to Reproduce:

Try to add a cert in Safari

Results: All cert files are dimmed and unselectable
Expected: Should be able to select proper files

Version - v1.2.1-rc2
Verified fixed


Apply Wagtail page seo_title or title

Simply checks for a Wagtail page's seo_title then title. If it's a sheer page, just append the bureau's title to whatever is there.
Additions

Bureau title to all page's titles.

Testing

Go to any page and inspect the title to see the bureau's title appended.

Review

@richaagarwal
@kave
@rosskarchner

@kurtw needs to be updated but looks straight forward :+1:


Hybridfox variant!

Found another fox! This one is a hybrid

Thanks for the pr. I have checked out the pr and will try to live in the colorscheme for a while.

Thanks! I love the nightfox theme but have been missing the hybrid colors.
Feel free to tweak anything, i'm not used to making themes so I've only pretty muched copied the colors from w0ng :)


Add lint GH action to the init method

Closes #5
Needs a few TODO's in the lint.yml managed before merging.

[x] Modify the lint.yml to set their primary branch as the branch name?
[x] Test the php-cs-fixer GitHub action (we've tested the other two often, but not this one)

Testing live, and OK without catching their name and assuming main for now.


Implement FastCircle component

Adresses https://github.com/ppy/osu-framework/issues/6067, https://github.com/ppy/osu/issues/21495
Pros:

No nested Box drawable (adds up in demanding scenarios)
Correct masking when used inside CircularContainer
No overhead from masking info and no batch breaking in cases when multiple circles are being drawn consecutively (though I think SSBO supposed to fix that?)

Cons (which are holding me back from replacing current Circle implementation with this one):

Circle is no longer a container (so is the Box which I think is fine)
No ability to set masking-related effects directly (again Box has the same properties)
Points above might be considered a breaking change, but easily fixeable by using CircularContainer instead
Texture coordinates abuse to pass draw size into the shader without uniform buffers
Has a separate shader

Performance comparison in demanding scenarios:

master
pr

Just reading the title of this makes me want to go "no why is this being proposed even". I'd hope we can not resort to cpp style microoptimisations like this. But maybe it's just me 🤷
I'd be much more okay with this replacing Circle as long as there is an actual perf gain and the presumed increase in draw calls from shader rebinding doesn't kill the perf gain. The tradeoffs sound fine to me.

I'd be much more okay with this replacing Circle as long as there is an actual perf gain and the presumed increase in draw calls from shader rebinding doesn't kill the perf gain. The other tradeoffs mentioned sound fine to me.

My current proposals:

We probably can do some hackery like providing some specific flag somewhere and use the same texture shader with reroute to the new one and thus shader switch will be eliminated, but I can't think of any sane and good-looking solutions
We can leave this as a side component (probably with a better name and xmldoc) amd use it only in specific cases, which is what I'm doing in provided screenshots
Wait for SSBO implementation and see performance difference then

My current position on this is I'm not sure where it fits in right now but I'm not totally opposed to it.

I'd be much more okay with this replacing Circle as long as there is an actual perf gain and the presumed increase in draw calls from shader rebinding doesn't kill the perf gain. The other tradeoffs mentioned sound fine to me.
If this "fast" circle can't be applied universally then I'm not super interested in this imo.

So I've added the "worst case" performance test scene, in which drawables are alternating between circle and the box since I haven't noticed much difference in game when replacing circle with my implementation.
In this "worst case" scenario draw call count is the same and for my machine specifically my version runs faster. If you are fine with that then I can update this pr to replace original circle instead.

Circle
FastCircle

The part that stops me from replacing Circle is the lack of a texture. It's the reason I questioned it in my review.
Slight tidbit: draw calls aren't equal. It's very likely your system will handle many draw calls much better than, say, a mobile GPU.

a separate shader is going to interrupt batching, so we need to be sure that's what we want (probably not?)

What you have to understand is that it already interrupts batch as a result of masking. It all depends on whether the gains of SSBO can be realised.

The part that stops me from replacing Circle is the lack of a texture. It's the reason I questioned it in my review.

Which current Circle doesn't have either, so we are not losing anything.

a separate shader is going to interrupt batching

There's a way to use the same shader, but I'm not sure if it's too ugly or some better approach can be implemented.

I have a working implementation which relies on unused y component of BlendRange being set to a unique value, however it's probably too ugly.

Hmm, found discrepancy in behaviour.
While Circle changes it's shape on Scale, FastCircle is always preserving the pill shape and I'm not sure which one is more expected to the end user.

Hmm, found discrepancy in behaviour

Okay, this one is easily fixable.
But also: while I can see how this component is too niche to include with the framework and potentially may cause performance issues upon misuse, I think it will be beneficial to at very least use it locally in osu! in the scenarios shown in OP, where we are sure that multiple circles will be drawn consecutively in a single draw call.
I have branches ready and can proceed further if necessary.

This PR's still in my emails fwiw, just very low priority while working on the many other tasks... I do eventually want to come back and re-read through the above comments + the capabilities offered by the existing Circle.
I may be okay blindly merging this in as FastCircle with documentation that it behaves slightly differently to Circle and is subject to change, for the specific scenarios that warrant it.

Hmm, was kinda hoping this would just replace Circle. I really don't want to have to go through code and decide when we want to use FastCircle vs Circle. Would prefer to just fix the few broken cases of Circle usage with the new implementation if that's possible.

My point is I don't know if this is the best that can be done. It still breaks the batch after all, and in a way that is much harder to prevent. 99% of cases would be fine with just using Circle, there are very specific use-cases that would currently benefit majorly from this.

Fair enough


moonlight: 3.1.1 -> 3.1.4

Motivation for this change
moonlight 3.1.2 adds support for HEVC on the pi 4. 3.1.4 is the latest version, with bugfixes and improvements
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)
[x] 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/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Tested on x86 and arm, lgtm. But HEVC support on pi will still require updates to nixos-hardware.

@ofborg build moonlight-qt


libinput: fix segfault when handling non-wlr device removal

Devices is null for some devices. . . There's way too many things called device or devices in there!

Feel free to sort it out and send a patch disambiguating the names if you like


[WIP] Add CI machinery

Fix https://github.com/ami-iit/idynfor/issues/7 .

Windows is failing with:
2022-07-14T10:56:52.6960359Z   Checking Build System
2022-07-14T10:56:52.9751368Z   Building Custom Rule D:/a/idynfor/idynfor/src/CMakeLists.txt
2022-07-14T10:56:53.2077138Z   iDynTreePinocchioConversions.cpp
2022-07-14T10:57:02.1402436Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(69,5): error C2988: unrecognizable template declaration/definition [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1403986Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(69,5): error C2059: syntax error: 'const' [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1404690Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(69,5): error C2059: syntax error: ')' [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1405428Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(69,5): error C2143: syntax error: missing ';' before '{' [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1418336Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(69,5): error C2447: '{': missing function header (old-style formal list?) [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1436983Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(70,5): error C2988: unrecognizable template declaration/definition [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1438451Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(70,5): error C2059: syntax error: 'const' [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1439403Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(70,5): error C2059: syntax error: ')' [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1440853Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(70,5): error C2143: syntax error: missing ';' before '{' [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]
2022-07-14T10:57:02.1442145Z C:\Miniconda\envs\test\Library\include\pinocchio/math/fwd.hpp(70,5): error C2447: '{': missing function header (old-style formal list?) [D:\a\idynfor\idynfor\build\src\iDynFor.vcxproj]

The strange thing is that pinocchio is being tested against that version: https://github.com/stack-of-tasks/pinocchio/blob/c2ad2c60eecc04555e265a8f80a9765d3b84f5f1/.github/workflows/windows-conda-v142.yml . However, probably we can fix this in a separate PR.

Windows.h included somewhere (why?)

I compiled with /showIncludes, and apparently the windows.h is directly included by pinocchio:
Note: including file:              C:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt\ctype.h
Note: including file:             C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Tools\MSVC\14.29.30133\include\clocale
Note: including file:              C:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt\locale.h
Note: including file:   C:\Users\STraversaro\AppData\Local\mambaforge\envs\idynfordev\Library\include\pinocchio/math/fwd.hpp
Note: including file:    C:\Users\STraversaro\AppData\Local\mambaforge\envs\idynfordev\Library\include\pinocchio/fwd.hpp
Note: including file:     C:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\um\windows.h
Note: including file:      C:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared\winapifamily.h
Note: including file:       C:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared\winpackagefamily.h
Note: including file:      C:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared\sdkddkver.h

This is something that I think can be fixed upstream in pinocchio, but for the time being we can just compiled with NOMINMAX.


Typo on venv_requirements.txt

Hi,
Just noticed a typo in venv_requirements. I think elevnlabslib should be elevenlabslib. I had the following error and this fixed it for me
ERROR: Could not find a version that satisfies the requirement elevnlabslib (from versions: none)

Thanks for raising the issue, It should be fixed now.


Add a "goto trace" for jumping around by trace index and a bookmark system

When jumping around a flow sometimes we find interesting places we want to come back later.
Being able to write down the trace-index and/or bookmark it seems useful.

https://github.com/jpmonettas/flow-storm-debugger/commit/0e9965435a6a1e5f8e8c5f0539fab237422d21fd implements the "goto trace" part of this issue

Given that a bookmark system can be accomplished with writings down the trace-idx in any text note, I don't think having a bookmark system now worth it. Maybe in the future ...


Implemented creating a link cloned VM from a template VM

Implemented CreateLinkClonedVM cloud interface to create a link cloned VM from a template VM. The code checks if the template VM has a snapshot, if no it creates it before creating a link cloned VM. If snapshot already exists, it uses it to create the link cloned VM.
Testing done:

kops cluster create goes through fine and creates the link cloned VM for the master and worker. Verified that it creates the snapshot on the template VM if it does not exists before creating a link cloned VM. In case the snapshot exists, it uses it to create the link cloned VM.
"make ci" is successful.

LGTM


Fix flake8 errors.

Merging this since there is not efficient changes in the code base and travis is happy.


Create Relaton gem

Create the Relaton gem.
x = Relaton.fetch("ISO19011")
=> Relaton::Error "No such fetcher"

require 'relaton-iso'
x = Relaton.fetch("ISO19011")
=> <# Relaton::Iso @docidentifier: "ISO 19011:2011" ... >

x.to_xml
=> <bib-item ...>

Types of bibliographic items, such as isobib, gbbib, rfcbib (called bibliographic engine) should "register" themselves to Relaton (e.g. which regular expression will this bibliographic engine handle).
This gem should also handle caching (document local, and machine global):

Relaton machine global database (e.g., "~/.relaton/db.json")
Relaton local database (e.g., "~/documents/iso-rice/relaton-bib/")
allow saving entries to global / local database (Relaton::Database.add(:global, Relaton.fetch("ISO19011")))
allow loading entries (Relaton::Database.load(:local, doc_identifier: "ISO19011"))
allow listing entries (Relaton::Database.list.map(&:to_s))
allow exporting entries (Relaton::Database.load(:local, doc_identifier: "ISO19011").to_xml / to_bibtex / to_rfcxml ...)

I am concerned about folding the GB standards codes into this sat without a universal  disambiguating prefix. Outside the context of Chinese standards, a prefix like NY is insufficiently specific. I propose to use "GB Standard" as a prefix for all Chinese standards, which is filled in by the gem for asciidoctor-gb

You're right.
But do you mean citing with a key like <<GB Standard GM/T 0009-2012>> or <<GBGMT00092012>> (Relaton.fetch("GBGMT00092012"), or ...?
Let's define proper patterns for every standard type, and we should document this so others know how to cite for auto-fetch?

The cache key and the fetch key should be the human readable key; I don't see a reason for it not to be. So GB Standard GM/T 0009-2012.
We will need to do likewise for IETF; I've already seen that ISO prefixes IETF to RFC, and RFC itself doesn't bother.
I don't believe ISO and IEC need further prefixing.

The relaton gem right now implements just Isobib calls, and just takes XML from the cache and the website. Initialising it specifies a filename for a global and a local cache. It works with asciidoctor-iso.
But the spec above is much, much bigger, and I want @andrew2net involved, rather than doing it all myself.

Done:

registering isobib to register in relaton
registering required prefix of isobib references to relaton
iterating through required prefixes in relaton to match code
initialising local database
initialising global database
loading entire database (JSON object, XML content)
saving entire database (JSON object, XML content)

To do:

returning Relaton class (isobibitem) instead of XML string as representation of entry
unmarshalling entries from database: https://github.com/riboseinc/iso-bib-item/issues/1
marshalling entries to database
loading individual entry from database
saving individual entry to database
listing entries
extending all of the above to the other two gems, gbbib, rfcbib
implementing to_rfcxml (which I'm just going to do by storing the original RFC XML)

@opoudjis do you intend to split the tasks between us?

@opoudjis I'm starting to work on returning Relaton class (isobibitem) instead of XML string as representation of entry

Thank you. The story is going to be:

Read XML from cache, when you do read it, into isobibitem class. That's what the Unmarshal task enabled
Return isobibitem class for all calls to fetch. (The recipient then has to serialise it to xml; I'll do that in asciidoctor-iso)
Explicitly marshal isobibitem class back to XML for caching
Load individual entry, and save individual entry (which can mean converting the JSON object into a key/value database)

@opoudjis saying Load individual entry do you mean we should load entries one by one and check if a key match to the code, to avoid loading entire DB into memory?

Yes, return IsoBibliographicItem from isobib.

I think Ronald invisions ass not loading all entries from the database at once, but only one by one as needed. That means that, for relaton, a reference will be loaded from the database only if we are checking whether it already exists there, for a get. You will need to load all entries for the list operation.

You may choose to do this using a key value database, instead of a json object.

The testing, I'm afraid, was not as extensive as I would have preferred, and I had to fix a fair few things to get it working with Asciidoctor-ISO and with existing ISO references; but that's all done now.


use derive macro for Default trait on Scoreboard struct and chunk optimizations

Description
A Detailed Description:
when I was reading the code I accidentally came across this:
impl Default for Scoreboard {
    fn default() -> Self {
        Self::new()
    }
}

In my opinion, this is a useless piece of code, because we can use #[derive(Default)] instead of this, for speed up
code read time.
Testing
cargo run and joining to server
Checklist
Things need to be done before this Pull Request can be merged.

[yes ] Code is well-formatted and adheres to project style guidelines: cargo fmt
[ yes] Code does not produce any clippy warnings cargo clippy

Change the commit name to something like "use derive macro for Default trait on Scoreboard"

Use x instead of yes in checkboxes
Don't lie to me, Your Code does produce clippy warns

I also just noticed you not only added derive Default for Scoreboard, You added a bunch of things. This should be split up into multiple PR's

I also just noticed you not only added derive Default for Scoreboard, You added a bunch of things. This should be split up into multiple PR's

how i can create another pull request? i can only view current

how i can create another pull request? i can only view current

You should use different branches


[SPARK-46764][DOCS] Reorganize script to build API docs

What changes were proposed in this pull request?
We are abusing Jekyll's plugin system to set flags for what API docs to build. This change maintains this overall status quo but makes the following improvements:

Rename the plugin file to be in line with its actual purpose.
Organize the code into functions so it's easier to follow and understand.
Print section headers that are easier to find in the output when building the docs.

The behavior of the documentation build otherwise remains unchanged.
Why are the changes needed?
This should make maintaining this part of the doc building workflow easier.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
In several ways:

I built the docs with various skip flags set and confirmed the build succeeds. I also manually browsed through some of the Scala, Java, Python, and SQL API docs.

The only docs I didn't test building were the R docs, because I do not have R installed locally.
Here is the build output on my machine for SKIP_RDOC=1: build-docs.log.zip
There are some errors output when building the Java unidoc, but these are present already in master.

I built the docs against master and diffed the resulting _site/ directory against the one output by this branch.

The _site/ directories are identical except for minor differences in some general SQL function example files.
Here is the diff: site.diff.zip

I also confirmed that ./dev/change-scala-version.sh 2.13 runs successfully (though I didn't try to run the modified scripts after that).

Was this patch authored or co-authored using generative AI tooling?
No.

In the future, I think it would be appropriate to convert this script into Python, as Ruby is not a focus of our community. (I don't use Ruby much myself, either.) But organizing the code into clear functions is something we'd want to do regardless of the language.

I diffed the built _site/ directory across master and this branch. They are identical except for some minor differences in some generated files. I've updated the PR description accordingly with the details.
@HyukjinKwon - I think this PR is good to go. Let me know if there is anything more you'd like me to do here.

Merged to master.


KAFKA-9080: Addresses MessageFormatChangeTest.testCompatibilty with version <IP_ADDRESS>

#7167 added a check for non-incremental offsets in assignOffsetsNonCompressed, which is not applicable for message format V0 and V1. Therefore, I added a condition to disable the check if the record version precedes V2.
Committer Checklist (excluded from commit message)

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

All MessageFormatChangeTest tests passed.

@omkreddy @guozhangwang ready for reviews

@tuvtran I took a look at the PR but still cannot understand why for V0/V1 continuous offsets no longer hold true: with V0/V1 the inner records use the base offset + relative offset which should also be continuous, right?

@tuvtran : Looking at the logic in DeepRecordsIterator, it seems there is a difference btw V0 and V1 message format. In V1, the wrapper message has the absolute base offset and each inner message has the relative offset. In V0, each inner message has the absolute offset and the offset in the wrapper message is ignored. So, we need to distinguish between these 2 cases during offset validation.

Can we revert the check added in assignOffsetsNonCompressed ?, so that we can continue with existing behavior in 2.4. We can add complete fix in next release.

Yeah I can create a separate JIRA for V0/V1 offset validation.

retest this please

@tuvtran : Looking at the logic in DeepRecordsIterator, it seems there is a difference btw V0 and V1 message format. In V1, the wrapper message has the absolute base offset and each inner message has the relative offset. In V0, each inner message has the absolute offset and the offset in the wrapper message is ignored. So, we need to distinguish between these 2 cases during offset validation.

What confused me here is that, in either V0, V1 or V2 case, the inner message offset (either it is inferred as base + relative, or it is encoded as absolute) should always be continuous right? @junrao @tuvtran

@guozhangwang : It seems the issue is that expectedInnerOffset always starts at offset 0. This will fail the test if (record.offset != expectedOffset) for V0 message since each inner offset is the absolute offset.

Can we make sure to include a unit test that verifies this fix? Generally, we should aim to do that before merging.

@guozhangwang : It seems the issue is that expectedInnerOffset always starts at offset 0. This will fail the test if (record.offset != expectedOffset) for V0 message since each inner offset is the absolute offset.

That makes sense, thanks!


VSCode configuration file related to "sort-imports" extension

@visualkhh please consider VSCode's config file (.vscode/settings.json) newly created
I described the reason here.
please check this out.

I checked. Thank you.
Added ".vcode" in ".gitignore".
You can use ".vcode" additionally in your local stage

ok, thanks


[Task]: Fix cypress test suite failures caused by PR #29794

Fix implemented in https://github.com/appsmithorg/appsmith/pull/30371


Minor readability and performance improvements

One thing that could be done at a later date is to move the check_output_path and similar into the utils file as these processes seem to be repeated in other files.

Hi Takuya,
Sorry for not getting back to you earlier. Thank you very much for cleaning and improving the code! It looks great. Fully writing numpy, pandas etc. is a matter of taste I guess. In our Oxford team we sometimes write them full to make it more clear which package is being used where.
Will start working on this package again in the summer :-)
Elco


Airplay & Pow

Hi,
Don't know why but if I get pow installed on my up to date Macbook, I can't reach any of my Airplay device. Discovery can't be done. I bet for a weird firewall issue.
I think this issue is present since a recent OSX update but I can't tell which one.
Pow version: 0.5.0
OSX version: 10.10.2

Note that installing pow also prevents public network share to appear in finder.


Update save file directory

Change the actual save and access of files using the new directory, [JAR file location]/data/edrecord.json instead of [JAR file location]/data/addressbook.json. A follow up from #59.
Related to #18.
Related to #72

Fixed by  #74


CI: Update PNPM (v8)

This seems to be using webpack v4 when it should be using webpack v5.
Probably a bug in PNPM v8.
Possible remedies:

Wait for PNPM to fix this
Update Storybook (this probably where webpack v4 is being pulled in)


[HELP] how to get image data from a cairo::Surface?

I want to get drawing result in cairo::Surface.
for example, I have tried the following
fn main() {
    let surface = cairo::ImageSurface::create(cairo::Format::ARgb32, 1920, 1080).unwrap();
    let ctx = cairo::Context::new(surface).unwrap();
    ctx.set_source_rgba(1.0, 1.0, 0.0, 1.0);
    ctx.move_to(0.0, 0.0);
    ctx.line_to(100.0, 100.0);
    ctx.stroke().unwrap();
    let surface: cairo::Surface = ctx.target();
    let maped: cairo::MappedImageSurface = surface.map_to_image(None).unwrap();
    let mut img_surface: cairo::ImageSurface = maped.clone();
    let img_surface_data: cairo::ImageSurfaceData = img_surface.data().expect("why panic here?");
    let data: Vec<u8> = img_surface_data.to_vec();
    println!("{}",data[0]);
}

but rust compiler report panic message:
thread 'main' panicked at src/main.rs:11:72:
why panic here?: NonExclusive
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

it panic at let img_surface_data: cairo::ImageSurfaceData = img_surface.data().expect("why panic here?");
is the above approach to get data from a surface is wrong? and how to get image data from a surface(I am ok to copy data in to a Vec)?

You need to make sure that there's only a single reference to the surface. Yours will have one more from the context, and another one from maped.clone().


Add Polish translation

Add Polish translation

Hey @pepsonEL.
Sorry for not providing feedback earlier.
Can you please follow Semantic Release naming conventions for commits?
feat: Description for features
fix: Description of bugfix

I have an automated versioning set up using semantic release and it depends on proper commit naming for it to work.
I also update contribution guidelines with this information for future contributors.

@pepsonEL I'll merge this PR and make the manual version bump.
Thank you for this contribution 🎉


Readme example does not simply compile

My first impression of async-std was not positive. I've added it to the project, copy'n'pasted the first example, and I've got:

a compilation error
2 | #[async_std::main]
  |              ^^^^ could not find `main` in `async_std`

and then noticed the explanation that the code used in the readme would be slow to compile. That makes it feel like it's not the best thing to use, and yet it's in the README.

It would be nicer if the first example in the README compiled with default crate features, without need to enable anything first. It's also worse by the fact that the prose just links to documentation that explains features. If the TOML markup enabling the required feature was inline in the README I'd have noticed it. But I didn't read the prose on purpose, since I wanted to jump straight to the code, and ended up disappointed.

It's also confusing because surf crate has almost identical example, but it uses #[runtime::main] attribute instead. And cargo add runtime adds a wrong version (it picks only stable ones).
So the first two async examples I've tried both failed because of a mysteriously missing magical attribute around main.

@kornelski Thanks for open this issue!
This allowed me to notice documentation issues.

I added #[async_std::main] to my main and it's also complaining with could not find maininasync_std`. What's missing?

I added #[async_std::main] to my main and it's also complaining with could not find main in async_std. What's missing?

async-std = {version = "1", features = ["attributes"]}

Examples in the documentation should be correct.


NP-641: Avoid firewalld flush of iptables

Disables firewalld access to iptables related nftables translator,
this avoids firewalld flush of iptables under any circumstance,
firewalld will work at nftables level, while MicroShift/OVN can
continue to work at iptables level (translated to nft) under the hood.

/retitle NP-641: Avoid firewalld flush of iptables

/hold I am doing more testing this morning to verify

/unhold
It seems to work well on my tests.

/retest

/retest

/approve

/lgtm

/cherrypick release-4.12


Incompatible with local image plus

I disabled the automatic rename feature ,but when the local image plus plugin dumps the network picture locally, the renaming function will still be triggered，the picture link in the md file is not updated at all.

Disable the Automatic rename attachment will only disable the rename operation when you rename the associated md/canvas file. But if you add a new attach to your notes file, it will be rename as your config (by listening the create event).

I have tested with Attachment Management v0.9.3 (enabled Automatic rename attachment) and Local Images Plus v0.15.9, it's work for me. Can you provide more info about your situation?
https://github.com/trganda/obsidian-attachment-management/assets/62204882/8e302f70-4591-4b3e-99ac-99b961192a08

https://github.com/trganda/obsidian-attachment-management/assets/58174558/6ccd49f7-0fb0-4568-95b5-75158829419f


0.7.289-stable-test   error:   "name already used as a name or title"

Neil
this is an error I got running the new 0.7.289 in the container. Note that I didn't get this error last week when I did a first successfull run.   I wanted to redo a test, same, from scratch.
I rebuilt the container, meaning I got the code from github etc.   I did a full reset to clean up everything on disk but the raw directory with the nights.
Every is fine with the PP, I start to have errors at the EXTTELL and EXTOBJ steps:
This is an example of the errors I get:
09:21:30.819-  |PROC| ***************************************************************************
09:21:30.827-@!|PROC| W[40-503-00019]: Error found for ID='1356'
09:21:30.834-@!|PROC|   apero_extract_spirou.py 2020-06-30 2497599o_pp.fits --crunfile=mini_run2.ini --program=EXTOBJ[01356] --recipe_kind=extract-science --shortname=EXTOBJ --parallel=True
09:21:30.842-  |PROC| ***************************************************************************
09:21:30.850-  |PROC|
 E[01-002-00012]: Error <class 'ValueError'>: name already used as a name or
        title
         file = '/apero-data/test_helu/asset/telluric/tapas_all_sp.fits.gz'
         function = drs_table.py.read_table()
09:21:30.859-  |PROC|
09:21:30.866-  |PROC|
09:21:30.874-  |PROC| ***************************************************************************

I did "fix" the column names (as you mentioned they were incorrect) maybe I broke something (I didn't run a mini data run as I though the change didn't require it) I will look into this.

If you were really motivated you could copy the tapas_all_sp.fits.gz from version 0.7.288 over that one and see if it works - that would speed up my job when I come to this problem.

Okay it seems to be that I indeed screwed up naming the columns and now have two columns called "trans_ch4" I'll figure out which is which and correct the v0.7.289 branch

okay I've updated v0.7.289-live and v0.7.289-stable-test so a simple git pull should fix this problem (hopefully I renamed the correct trans_ch4 column)

I'm running a new test. rebuilt everything from scratch. Container version.
The aim of this test is to check (that the bug is gone) that I do get the same results that last week.
I'm planning a second test after that: I'mm process the mini run in 2 steps: REF and CALIB first, then clean the red/ and the continue with the EXTTEL and ETCOBJ etc. just to test the ability of Apero for work with only the calib files from the calib/
One question still: we Apero builds the products, does'it pick up the balze, wave etc from calib/ or from red/ ?

the mini_run2.ini ended with no error.  So the tapas name bug is fixed.

perfect thanks for verifying! I've been too busy to run a mini test so its very helpful!

One question still: when Apero builds the products, does it pick up the blaze, wave etc from calib/ or from red/ ?

In theory everything including the postprocessing should only ever use files from the calib and telluric directory... I'm not sure about the lbl codes - I can't remember if I added a way to "copy" the science/calibration data to lbl or whether it is always just symlinked - if it is always symlinked then apero-lbl stuff may break (or run on a smaller data set) -- I'm really not sure at this point


mv: Unable to get file status (stat) for 'inferred-predicates': No such file or directory

Hello, I'm sorry to bother you. When I ran citation, I encountered a lack of inference predicates. Can you help me solve it?
(myenv) fanyongjun@lab4gpu:~/neu/neupsl-ijcai23-main$ ./scripts/run.sh citation
Data experiment::citeseer found cached, skipping download and extract.
Data experiment::cora found cached, skipping download and extract.
Running '/data/fanyongjun/neu/neupsl-ijcai23-main/scripts/../results/citation/method::neupsl/experiment__citeseer/split::0/method::simple'.
Output file already exists, skipping:
real    0m0.787s
user    0m2.637s
sys     0m0.151s
mv: Unable to get file status (stat) for 'inferred-predicates': No such file or directory
Running '/data/fanyongjun/neu/neupsl-ijcai23-main/scripts/../results/citation/method::neupsl/experiment__citeseer/split::0/method::smoothed'.
Output file already exists, skipping:
real    0m0.799s
user    0m2.602s
sys     0m0.158s
mv: Unable to get file status (stat) for 'inferred-predicates': No such file or directory
Running '/data/fanyongjun/neu/neupsl-ijcai23-main/scripts/../results/citation/method::neupsl/experiment__citeseer/split::1/method::simple'.
Output file already exists, skipping:

Please send the output provided in the following files:
results/citation/method::neupsl/experiment__citeseer/split::0/method::simple'/out.txt
results/citation/method::neupsl/experiment__citeseer/split::0/method::simple'/out.err.

Thank you for your answer. I have successfully resolved this issue when I run ./scripts/run.sh. Initially, I directly ran /citation/scripts/...., which caused this error.
Thank you for your answer.
Best wishes.


[YUNIKORN-1039] add e2e test which uses volumes

What is this PR for?
This PR adds 2 e2e tests to verify volume binding.
The first case aims to test static volume binding, where a PV and PVC are created before deploying a pod.
The second case checks dynamic volume binding. It creates an NFS provisioner to simulate an NFS storage environment. Pods can use a PVC with a specific storage class, and the provisioner automatically creates the corresponding PV.
What type of PR is it?

[x] - Test

Todos
Add more complex cases.
What is the Jira issue?
YUNIKORN-1039
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Please have a look at the lint issue. You can prevent the linter from generating the failure by adding a nolint directive if it is safe.

Hi @pbacsko thanks for the detailed review and the explanation! I misunderstood it before, and it has been modified now.


Adding cached flag for Mac Perf

Working great here for some time already.
https://docs.docker.com/docker-for-mac/osxfs-caching/#cached

I would use delegated instead for mysql and redis volumes as we don't need containers' priority :)

I can't add this as it's not compatible for windows/linux users. However I'll probably document this, mutagen, and using NFS in the docs for anyone interested.


fire up plugin

Sorry this questions but i'm newbie on video.js.
Is it possible to initialise videojs-Background plugin inside data-setup={} ?
Thanks

Hello, sorry but you can't init from data-setup, you should be able to use normal data-setup parameters to set video options as per the video.js documention though.


How to lint only the changed files?

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
My code repository contains a large number of files, and formatting the entire codebase every time would take several hours. How can I configure it to achieve the effect of "formatting only the changed files"?
Expected Behavior
formatting only the changed files
Anything else?
No response

So, what's the plan about it? You're not going to include it to the roadmap? why?

Hi @AndreyMlashkin and @acbin
This is already implemented. See the VALIDATE_ALL_CODEBASE configuration variable.


Deploying internet-pi in a k8s/k3s cluster

Perhaps this is out of scope for this particular project, however, after watching your Kubernetes series, I am interested in deploying this in my RPi 4b cluster.  Maybe this is more of an enhancement or future iteration of this project.
From what I understand running through the yaml files, this will only install and configure a single host (or RPi 4b in the YouTube tutorial);  I'm assuming I would just ignore the Ansible playbook for deploying internet-pi and run a docker-compose command on the main.yml file?

Yes... ish. The Ansible playbook templates out a number of configuration files using values defined in config.yml, and if you want things to run correctly you'd have to manually do that work with all the files inside the templates directory.
To get it into K3s/K8s, you'd need to rework a bit because multi-node K8s networking, especially around non-TCP things, can be a bit awkward.


refactor(query-builder): update schema colors

closing in favor of https://github.com/IgniteUI/igniteui-theming/pull/341


Use SHARED_MESH_CONFIG in e2e suite if specified.

Describe the feature request
E2E Test Suite should use SHARED_MESH_CONFIG when making changes to Istio MeshConfig / ConfigMap, if specified.
Currently, the testing framework patches the default ConfigMap when making changes to the MeshConfig. However, in environments / setup that use the SHARED_MESH_CONFIG, users may want the test framework to patch the local shared ConfigMap instead. This can be specified using a flag ex: --use-shared-mesh-config when running the testing command.
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

Just curious, if there are 2 configmaps why does it matter which is patched?

@howardjohn for instance, external / hosted control plane setups, to better mimic user flow. Or other setups where default ConfigMap may not be directly accessible or modifiable to user.

Should this be closed as implemented in #46074?

@mikemorris yes, will do.


Use opc.ai/repositories API or GitHub API and request Repo / Branches for clone command.

The opc.ai option would be to just use the existing JSON endpoints of https://opc.ai/repositories.json?q=jfrux as an auto complete for the fields and still let them enter their own if it doesn't find it...
opc.ai has worker processes and archives / updates all repos in the github api every hour or so.
https://opc.ai/repository_branches endpoint will need about 5 minutes of work to tweak it to respond to full-text queries and response as json.

@saranchockan Would this be something you would be interested in trying to tackle?

Another option would be to just use the existing JSON endpoints of https://opc.ai/repositories.json?q=<text> as an auto complete for the fields and still let them enter their own if it doesn't find it...

@saranchockan Would this be something you would be interested in trying to tackle?

I will look into!


Update playwright-chromium

@XhmikosR no, it isn't. The problem here is that i don't have much time to dig deeper. It's 99% that problem is in chromium rather than playwright.
I understand that we can't wait forever so let's land this. I will research sometime later.

If you work on battery, I wouldn't want to make things harder for you. We can still keep using the old version for more time.

@XhmikosR apparently it's chromium bug. And it's caused without external display with mackbook m1 devices :/ Not with the power source as I though before. Ok let's keep the old version for now


[DEP-5588] Fix architecture lib helpers

This PR fixes a couple of issues discovered when integrating the arch mismatch check into MySQL charm codebases.

It makes the WrongArchitectureWarningCharm class to raise an exception, instead of exiting with error code 0. As briefly discussed on 11-12-2024 Technical Sync meeting, there is an edge-case where, if the charm user is refreshing the revision to an arch-incompatible one, some events may be lost in the rollback process if the charm just exits.
It makes the is_wrong_architecture helper func resilient to environments where the CHARM_DIR variable is not set (i.e. when running unit tests).

I did not bump the patch version, as no one is using this library at the moment.

what were the results from testing show-status-log?

I ran the CI on the other PR setting the unit status to ErrorStatus(...). Turns out that line was raising an undesired exception, as the ErrorStatus is read only, and must not be used to set any state manually (reference).


Add play-request-was-interrupted update article

R: @mounirlamouri
Preview at https://pr-4688-dot-web-central.appspot.com/web/updates/2017/06/play-request-was-interrupted
Background: https://bugs.chromium.org/p/chromium/issues/detail?id=728294#c25

No no! Mounir didn't approve yet. I'll ping you when needed ;)
On Wed, Jun 14, 2017, 4:50 PM Pete LePage<EMAIL_ADDRESS>wrote:

Merged #4688 https://github.com/google/WebFundamentals/pull/4688.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/WebFundamentals/pull/4688#event-1123443238,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmubvF9ajDTOU9OMBKgVTJHTv9e-ujWks5sD_MxgaJpZM4N4fRh
.

sorry - already live. it didn't have the DO NOT MERGE tag, so I assumed it was good to go. Does it need to be pulled?

It's not horrible. As long as Francois addresses the issues, soon, I ok
with keeping this in.
Joe Medley | Technical Writer, Chrome DevRel |<EMAIL_ADDRESS>|
816-678-7195
If an API's not documented it doesn't exist.
On Wed, Jun 14, 2017 at 8:17 AM, Pete LePage<EMAIL_ADDRESS>wrote:

sorry - already live. it didn't have the DO NOT MERGE tag, so I assumed it
was good to go. Does it need to be pulled?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/WebFundamentals/pull/4688#issuecomment-308464474,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH0viw1fn-T59m4tru72Ub6ZaKcU5hu4ks5sD_mMgaJpZM4N4fRh
.

@jpmedley @petele See https://github.com/google/WebFundamentals/pull/4697


mergeSEs: tree merging

Tree merging part just check if tree row can be found from set of trees. It finds the smallest number of trees that can present the rows and adds them to TreeSE. However, correct way is to merge the tree into single tree https://www.rdocumentation.org/packages/ape/versions/5.8/topics/bind.tree

Create random trees:
 x <- ape::rtree(5)
 y <- ape::rtree(10)

 x <- ape::makeNodeLabel(x, prefix = "x_")
 y <- ape::makeNodeLabel(y, prefix = "y_")

 combined_tree <- tree1

Get unique taxa in both trees
 taxax <- x$tip.label
 taxay <- y$tip.label
common_taxa <- intersect(taxax, taxay)
common_mapping <- match(common_taxa, y$tip.label)

Remove common taxa from y to avoid duplication
 z <- drop.tip(y, common_taxa)
Bind the remaining part of tree2 to tree1 at the common taxa nodes
combined_tree <- bind.tree(combined_tree, z, where = common_mapping[i])

I don't have now time to check in detail, but row can be also linked with internal node. Then the row cannot be found from tip. If you then use drop.tip, it might remove also internal node that represent row. We have .prune_tree function that takes this into account. Explore that.