[presence API] Show who is present and editing a record

Enhancement
Show who is present and editing a record

https://biblio-backend-ugent-prototype-git-bootstr-307d84-ugent-library.vercel.app/biblio/researcher/publications/detail/full-view.html
Success criteria

[ ] When there is more than one browser window connecting to the same record...

[ ] I can see a badge appear for each of the browser connection
[ ] Each badge contains the initials (first- and last name if available, drop whichever is not available) of the person looking at that record.
[ ] When I hover over that badge, I see the name of the person looking at the record

I can see activity happening live:

[ ] When I am looking at a record, and someone else opens a browser window that looks at the same record, a new badge appears instantly.
[ ] When a person leaves, or I close one of my browser connections, the badge for this connection disappears instantly.

Rules

When I am alone in a record – I do not see any badge.
Do not flood people with information that is irrelevant or a given
When I am looking at a record in two browser windows – I see two badges with my own information
When I am looking at a record in three browser windows – I see three badges with my own information
When one or more person is looking at the same record – I see a badge appear for each person with their information

Automatic testing scenario
Scenario explained under Rules – anything else needed to write this?

Removed the "cypress-test-requested" label here since this requires two separate simultaneous sessions in separate windows, which is not something Cypress can do.

Duplicate of #1229 which will be picked up later


COMP: Fixes the HDF5 build on MinGW

The HDF532defs.h were changed, as the number of arguments was wrong in
Mingw32. This was referenced on #562 (comment) in #562 but wasn't
incorporated. Left it as it didn't seem to be an issue anyone else had
until yesterday.
Co-authored by: John Muschelli<EMAIL_ADDRESS>Superceeds #1216

This still needs some work for Visual Studio.

Just FYI, I believe this is still an issue without this fix: https://ci.appveyor.com/project/muschellij2/itkr/builds/27807695/job/wwjqfl5xqr8pvj6e

Did this get merged?


WINAPI PowerRegisterForEffectivePowerModeNotifications

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem.
My vs version is 1809, and I link Powrprof.lib to my project and call the function PowerRegisterForEffectivePowerModeNotifications. The project can build successful but wen I run the .exe, there is a popup yell me no entry point of this function.

https://docs.microsoft.com/en-us/windows/desktop/api/powersetting/nf-powersetting-powerregisterforeffectivepowermodenotifications
Link to article:
Tell us where exactly a problem occurs.
Problem:
What's wrong with the article?

Hi @lxy771895408 -- thank you for your feedback. We can have an author check the doc to see if we can add more helpful information, but if you need support help with your project I would recommend posting to the community forums linked on our support page.


Hover area is extended to labels

Unfortunately, didn't get any help at Stackoverflow, so I have to ask here:
https://stackoverflow.com/questions/51515030/hover-area-is-extended-to-labels
Thanks in advance for any help.

Hi, I'm very sorry you didn't get a response. However, unfortunately we don't have enough maintainers to answer questions and don't have the resources to break from our guidelines and answer questions here


Bug: Randomized benchmarking still using Ignis

Where does the issue happen?
https://learn.qiskit.org/course/quantum-hardware/randomized-benchmarking
What is the content issue?
Chapters using Ignis, which is no longer supported, should switch to Qiskit Experiments. The update was in progress in the old textbook (see https://github.com/qiskit-community/qiskit-textbook/pull/1480) but it was decided the work should be done here.
Screenshots
No response
Additional information
No response

Will be done once https://github.com/Qiskit/textbook/pull/20 is live.

This repository is no longer maintained. It has been superseded by IBM Quantum Learning.


Add Stable Diffusion XL example

Similar as https://github.com/ROCm/AMDMIGraphX/tree/develop/examples/diffusion/python_stable_diffusion_21
Screenshot of the Gradio app:

Added support for the refiner:


When in tabFocusMode, assign a different keybinding for inline suggestion acceptance

Discovered by @jooyoungseo

Open the hover for an inline suggestion
Use the keybinding to see the next suggestion
🐛 the text in the hover doesn't show the next suggestion unless I close and re-trigger the hover

cc @isidorn

We now have the accessible view, so I don't believe the hover is the preferred way to review this anymore, thus we can close this.


Use correct environment variable when setting up OpenTelemetry propagator

Thanks
I also notice the bug this afternoon (shame on me).
I included your PR + a test into the released 0.8.2


take name of current open file from usethis::compute_name()

  # TODO take

https://github.com/dataheld/elf/blob/a2c80907d1719d66ad3a0cb3aaf3d3af7bd49d39/R/examples.R#L29

blocked by https://github.com/r-lib/usethis/issues/2019

closing in favor of #4


ENHANCING THE ACCESSORIES PRODUCTS SAME LIKE OTHER PAGES

resolved issue num  #565
@Tejashri-Taral You can check for any other issues
I'd like to take any other inputs!!

Please resolve the conflicts

can you check now? @Tejashri-Taral

https://github.com/user-attachments/assets/0cd05de3-46fd-4224-8980-95e839f67248
hey @Tejashri-Taral
I've not removed any but just modified the existing products...
earlier there were total 24 products..
I've just enhanced them...(above I've added the video)
can you check once again...

@Tejashri-Taral the above video that I've sent is what i changed and its well organised
can you check again??

some row contains 4 product some 3
It should be consistent like each row should contain 4

My changes are like this for the page...I don't know what it is for you
https://github.com/user-attachments/assets/de216936-dee9-403f-8161-158764d0622b
so once again can I send the pr

can you check now @Tejashri-Taral ??

I was expecting level2 label for this
can you assign level 2 since There were many lines of code included!!
@Tejashri-Taral


Add root taxons to tracking on the link to content data

Adds 'data-track-dimension-3' to the link attributes for
the link to content-data metrics page.
It contains a comma separated list of base paths for each root taxon
linked to the content item.

 (3) Add button to Content Data from Whitehall

Already merged in another PR


Bug 1948431: Bump openshift/api to get the necessary CSIMigration feature gates

The CSI Migration tech preview feature is included in openshift/api, and I'm bumping the version here so that the FeatureGate can be propagated correctly.
Goal of this PR is to apply --feature-gate=CSIMigrationAWS=true,CSIMigrationOpenStack=true to kube-apiserver when TechPreviewNoUpgrade featureSet is enabled, which was introduced in openshift/api#880 .

/retitle Bug 1948431: Bump openshift/api to get the necessary CSIMigration feature gates

/retest
/lgtm
/approve

/retest


Updated Getting started tutorial to Moveit2

Description
Ported Getting Started tutorial to MoveIt2  #42
Checklist

[ ] Required by CI: Code is auto formatted using clang-format
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

Sorry I tried to clean this up but the remote branch was giving me issues and when I gave up and force pushed to it, it closed this PR.


3rd try to get github to not break when I submit a pull request

I stopped dropping empty hours of the day. I also took off the x axes for some of those bar plots and added their abs number in either the middle, or off to the side if it was <0.35 the max value. Screen shots in slack because I've had so many issues today with internet


SyntaxError: Unexpected token T in JSON at position 0

Hi there,
The app has been deployed from AWS Serverless Application Repository.
The error occurs when Publish message from SNS console.
{
    "errorType": "SyntaxError",
    "errorMessage": "Unexpected token T in JSON at position 0",
    "stack": [
        "SyntaxError: Unexpected token T in JSON at position 0",
        "    at JSON.parse (<anonymous>)",
        "    at Runtime.handler (/var/task/dist/handlers/forward-to-teams.js:40:30)",
        "    at Runtime.handleOnceNonStreaming (file:///var/runtime/index.mjs:1147:29)"
    ]
}

Regards,
Eugene

The error message looks like you were sending non-JSON data to the topic. Actually you would need to create the very exact JSON structure as Cloudwatch would do it.


CollapsibleTableViewHeader.swift #selector

CollapsibleTableViewHeader.swift:52:75: Argument of '#selector' refers to instance method 'tapHeader' that is not exposed to Objective-C

@objc func tapHeader(_ gestureRecognizer: UITapGestureRecognizer)
this is what needs to be done in order for this to work.
I've manually changed that inside CollapsibleTableViewHeader


note on naming restrictions

else push would fail with cryptic
retrieving manifest
Error: file does not exist
==> maybe change that in code too

Thank you for the PR!


refactor(frontend): remove build server script

Motivation
Side quest of #1994.
I tried to rebuild the entire frontend with vike from scratch, following
the current documentation. One thing that I noticed is that they have no
build step for the server code. They simply run tsx in production.
This is a good option for us, too, because we can get rid of a lot of
code and complexity.
How to test

Deploy this to the kubernetes cluster
Deploy this on our vServer
Check if both frontend and presenter are running fine

The idea here was to build the whole thing and get rid of the typescript requirement. Thats why I designed it like this.

@ulfgebhardt I rebased this and now it needs another approval.


Cannot parent anything to glTF actors

const gltf = await MRE.Actor.CreateFromGltf(this.app.context, {
    resourceUrl: Server.registerStaticBuffer('sphere.glb', gltfFactory.generateGLTF()),
    actor: {
        transform: {
            position: { y: 1, z: -1 },
            scale: { x: 0.5, y: 0.5, z: 0.5 }
        }
    }
}).value;

MRE.Actor.CreatePrimitive(this.app.context, {
    definition: {
        shape: MRE.PrimitiveShape.Box,
        dimensions: { x: 2, y: 2, z: 2 }
    },
    actor: {
        parentId: gltf.id
    }
});

The box appears at the origin at world-scale 2,2,2.

Fixed by https://github.com/Microsoft/mixed-reality-extension-unity/pull/59


[Bug] Disable Test button when datasource form is invalid, just like the Save button

Description
When some of the fields in a datasource form are in error state, the Save button is disabled, but the Test button is not.

Important Details

Version: Cloud
OS: macOS
Browser: Firefox

Closing as we should allow folks to test datasources even if the configuration is invalid.

Then I think the Save button should not be disabled as well. Why is testing invalid configurations allowed but saving is not? This inconsistency here is what's bothering me.

Agreed. Re-opening with a modified title & description.

Closing this issue since it is fixed already.


#asWord simplifications

This PR stabilises and brings together all of the #asWord-related simplifications created during various engagements.
In addition, it:

corrects some copy-pasting typos across the development.

TODO: add tests for the two new lemmas.

The tests are now in and the PR is fully ready for review.

I can also confirm that the two main engagement-related sets of proofs run marginally faster after these changes.

Can we have the code-motion and the type cleanups in a separate PR, that way I can review that quickly and have a smaller diff to review for actual new lemmas and tests? In general, we always request that code motion comes in its own PR for quicker/saner review process.

I can move the typo corrections and the movement of the auxiliary slot-update lemmas, yes, if that's what you meant specifically.
The rest I am not sure how to separate, given that the purpose of the PR is to consolidate #asWord reasoning, which involves movement of existing lemmas, changes to existing lemmas, introduction of new lemmas, and creation of appropriate tests.

All the movement/consolidation should come in it's own PR, without semantic changes to anything. Semantic changes can come after. The reason being that it's easier to review a PR if I know it's all code movements, I just have to check "where did it come from where did it go?", not the actual contents of the lemmas. Then in a separate PR, which introduces or changes lemmas, I can inspect the actual content of the lemmas without. It reduces the overall amount of review I have to do.


RTU Wrong CRC

Dear all
It's the firs time that we have used this application, but we have a lot of years using others like ModbusPoll or ModScan32, and the slave it's the firs time that we had communicate too.
The comunication is done by a Master ussing laptop by Moxa Uport 1150 and 1 slave by MGS-408
Comunication by Modbustools

The 1st configuration of the expected register address but the slave doesn't read the correct function etc.
The information about Modbus Tool is CRC Wrong.

The 2nd has modified the register address to achieve the same read data from slave than on working applications, but logically for us was wrong because we put 4x etc. (there are images about configuration)
The information about Modbus Tool is CRC Wrong.

Comunication by ModbusPoll / ModSca32

The data is working (live value)
The configuration of bus it's ok because its reading data (Modbus RTU 2W, 9600 bps, 8 Bits Data, 1 Bit Stop, Parity = None)

This is the 1st configuration of  Modbus Tool (register address as expected of other programs, but yoyu can see how the data is read by the slave) [Communication KO]

This is the 2nd configuration of  Modbus Tool (register address modified to read in the slave the same configuration as working communication with other applications) [Communication KO]

Configuration working variable ModScan and Slave display diagnostic [Communication OK]

Configuration working variable Modbuspoll  and Slave display diagnostic [Communication OK]

Thanks in advance,
Best regards.

Hello. This is an old issue that has been resolved few releases ago. So you just need to download the latest v0.3.9

Hello. If I understand correctly 40017 is holding register with number 40017 (numeration from 1). Left digit 4 (most signified digit) in ModbusTools address notation is just memory type id. 4x - for holding regs, 3x - for input regs etc. So 440017 can be interpreted like holding register with number 40017 (numeration from 1).
There is a similar thread: https://github.com/serhmarch/ModbusTools/issues/5#issuecomment-2097897458


Add support for PHP-FFMpeg/PHP-FFMpeg 1.x

Hi There,
Do you have any plans to add support for PHP-FFMpeg/PHP-FFMpeg 1.x versions.
Thanks mate.

Thank you for the report. I have just released the new version and now you can update the package:
composer require aminyazdanpanah/php-ffmpeg-video-streaming


slightly document the source, add an allSystems list

bringing over drafted material from https://github.com/numtide/flake-utils/pull/84. I think renaming this repository to "numtide/flake.systems" would be a good idea if we're going in on that naming convention.
slightly document the source
since this is the most upstream flake providing these source files, we should provide in-file documentation as to their purpose.
add an allSystems list
this should be the only other list necessary. if tooling like flake-utils is going to vendor this list, it should be centrally defined and overridable when necessary without forking flake-utils.
The presence of systems + "/all.nix" requires that a downstream ./flake.systems.nix become a ./flake.systems with the two files default.nix and all.nix. all.nix can be copied from upstream and updated whenever relevant. Any further downstreams won't notice if all.nix is outdated because they'll be overriding it anyways with either full upstream or their own file.

If we don't want to make ./flake.systems a directory, then we either need to figure out how to remove the need for allSystems to be vendored in flake-utils without adding other dependencies, or we add a flake.allSystems.nix file for the allSystems input and incur one extra input's worth of boilerplate (+1 line per flake input). Selective input tree overrides would be nice here, but that's probably a post-https://github.com/NixOS/nix/pull/6530 thing.

The main reason allSystems was introduced to flake-utils was that the list of systems wasn't externally extensible. The idea was that since the list of systems wasn't configurable, let's add all of them. Now that this is solved, what utility does it provide in exchange for the more complicated interface?

Thanks for participating. Closing as default.nix has moved to nix-systems/default.
If you see the utility of it, I'm open to creating a nix-systems/all repo.


Visualizations: Remove compute id from route

We currently include the computation id as a part of a visualization's route. Since a visualization can move from one computation to another when modifying the compute configuration, visualization URLs are unstable.
Instead, we should remove the computation id from the route (and add a redirect for URLs that still contain the computation id).

this ticket duplicates https://github.com/VEuPathDB/web-eda/issues/1545, which was closed

This issue is slightly different. The idea is that we don't want to change the URL of a visualization just because the configuration of its computation changes.

since we haven't released mbio yet, seems like it'd be best to get this done by 64 (if possible of course!) so that we don't have to worry about "old" computes?

Just a heads up that in https://github.com/VEuPathDB/web-monorepo/pull/143 I've added some convenience methods/functions to the useAnalysis payload that will be useful here.  See link below.
https://github.com/VEuPathDB/web-monorepo/blob/03877c6d22d8316d72acbb6b02c35be62d49fbdf/packages/libs/eda/src/lib/core/hooks/analysis.ts#L56-L80
For example, when rendering from a URL with just a visualization ID you can get the computation and viz with getVisualizationAndComputation(vizId).
The new functions will probably help streamline the code in computations/Utils.ts - e.g. handleConfigurationChanges and maybe others.  Note that the new addVisualization(comp, viz) will remove viz from any old computation before adding it to comp, so you can use it to move a viz from one computation to another.
While tidying up, we may want to move deleteComputationWithNoVisualizations from VisualizationsContainer into analysisState?


Basic template

Initial Commit: Set Up Template Repository Structure
Description
This PR establishes the starting point for a foundational structure for the EECA Python Template repository. The goal is to provide a standardized starting point for Python projects within EECA, equipped with pre-configured tools and workflows to streamline development, testing, and continuous integration.
What's Included:

GitHub Actions Workflows:
.github/workflows/pylint.yml: Workflow for linting code using Pylint, Black, and Isort.
.github/workflows/python-tests.yml: Workflow for running tests with Pytest and generating coverage reports using Coverage.py.
Pre-commit Hooks:
.pre-commit-config.yaml: Configuration for pre-commit hooks to enforce code formatting and import sorting before commits using Black and Isort.
Configuration Files:
.gitignore: A comprehensive ignore file tailored for Python projects to exclude unnecessary files from the repository.
pyproject.toml: Configuration for the build system and tools like Black and Isort.
requirements.txt: Placeholder for runtime dependencies.
requirements-dev.txt: Development dependencies including Pylint, Black, Isort, Coverage, and Pre-commit.
LICENSE: MIT License for the project.
Source Code Structure:
src/python_package/__init__.py: Initial package directory for project source code.
tests/test_main.py: Example test module with a sample test function to demonstrate testing setup.
Documentation:
README.md: Updated with detailed instructions on how to use the template, including setup steps, usage of pre-commit hooks, and descriptions of the provided features. Added badges for build status and linting status.

Features

Code Formatting: Enforces consistent code style with Black and Isort.
Linting: Analyzes code quality using Pylint.
Testing: Runs tests using Pytest and reports coverage with Coverage.py.
Pre-commit Hooks: Automates code formatting and linting before commits.
Continuous Integration: GitHub Actions workflows automate linting and testing on each push and pull request.

Love it!


add getIsSomeRowsExpanded instance property (v8 alpha)

A small and efficient addition to the expanded feature I use in my project.
In most cases where expanded state is not handled manually, this should be an instant constant lookup time.

I can’t remember does the row api have this already?

@tannerlinsley
My original code I was going to pr here looked just like the getIsAllRowsExpanded code, kinda like this:
getIsSomeRowsExpanded: () => {
  const expanded = instance.getState().expanded

  // If expanded is true, save some cycles and return true
  if (expanded === true) {
    return true
  }

  // If any row is not expanded, return false
  if (
    Object.keys(instance.getRowModel().rowsById).some(
      id => instance.getIsRowExpanded(id)
    )
  ) {
    return true
  }

  // None must be expanded :shrug:
  return false
},

but then I thought I could skip the loop by just looking at the expanded state instead.
The use case for this may be small, as I'm just using it for an indeterminate value, similar to getIsSomeRowsSelected


chore(doc): update FAQ

Issue #, if available:
#5
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @wchaws please take a look.


Update version to 1.2.88-2 on production

Update version to 1.2.88-2

Hey! I see that you made changes to our Form component. Make sure to update the docs in FORMS.md accordingly. Cheers!


set cgo flag on PROMU to build dynamically linked binary

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

/lgtm


Kite does not run in .tsx or .ts files

Seems Kite doesn't run in TypeScript files
Is this by design? Is there a setting I forgot to turn on?

I don't think that kite supports TypeScript but it would be great!

I also hope kite supports TypeScript

It should support typescript based on their site https://www.kite.com/blog/product/kite-launches-support-for-11-new-programming-languages/


How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url:https://maxlauncher.sourceforge.io/downloads.html
Publisher:MaxLauncher
Package Name:MaxLauncher.MaxLauncher
Description:MaxLauncher is an open source tabbed launcher. Its primary goal is to reduce the amount of time to run a program/file.
Package Version:<IP_ADDRESS>
Installer URL:https://sourceforge.net/projects/maxlauncher/files/MaxLauncher/<IP_ADDRESS>/maxlauncher_<IP_ADDRESS>_setup.exe/download  not sure if its right

@AwesomeDog Azure Pipeline seems blocked by SourceForge. Maybe downloading and installing applications via winget.exe violate ToS of SourceForge?
https://github.com/microsoft/winget-pkgs/pull/122410#issuecomment-1758616142

@AwesomeDog Azure Pipeline seems blocked by SourceForge. Maybe downloading and installing applications via winget.exe violate ToS of SourceForge?
#122410 (comment)

https://sourceforge.net/p/forge/documentation/Release Files for Download/


Saving a notebook results in editor displaying text content

Azure Data Studio Version:
Version: 1.31.0
Commit: 9ca6200018fc206d67a47229f991901a8a453781
Date: 2017-12-15T12:00:00.000Z
VS Code: 1.56.0
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.5.0

Creating a new notebook and saving it will show the notebook configuration.
https://user-images.githubusercontent.com/34872381/124048826-07a8f080-d9cc-11eb-8608-0e7e3a5db510.mov

I have a potential fix for this but I want to look into the issue a bit more before submitting - there was some major refactoring of the editor override stuff and I'd like to dig into that a bit more before going forward with this in case there's a better way to do it.
Here's the VS Code PR that changed this : https://github.com/microsoft/vscode/pull/119274
There's been a ton of fixes/changes to that original implementation since it was made though (most of which aren't included in our current sync point) so it's also possible that a fix to bring back to old behavior was done at some point so I want to look into that as well.


--embed with no alias drops the parent directory from the embedded files

When xcaddy is invoked with an --embed target without an alias, referring to a directory within the working directory, like so:
$ xcaddy --embed my-dir

…this results in the files/ directory of Caddy's embedded FS containing the contents of my-dir/, as though my-dir/* were the target, rather than a files/my-dir/ directory containing the contents of my-dir.
This behavior was unexpected for me because it diverges from the way that the go:embed directive of the Go tool treats its path arguments. Go will embed a specified directory (without a glob suffix) as a directory, and any specified subdirectories will preserve the entire directory structure that leads to that subdirectory. I.e., go:embed my-dir creates an embedded directory my-dir, and go:embed dir-a/dir-b creates an embedded directory structure of dir-a/dir-b.
This behavior results from this line:
https://github.com/caddyserver/xcaddy/blob/b72e330d5d7439ce848e29ade848760012e9e386/environment.go#L96
where d.Name is empty if there was no alias in the corresponding argument to --embed.

I see how this could be the intended design for --embed, and I also realize that the argument to --embed must not be treated identically as the arguments to a go:embed directive, since relative path elements like .. are permitted in --embed arguments for greater flexibility. If this is the case, I think this behavior should be specified in the --embed documentation.
If it's not the intended design, I suggest that an unaliased path passed to --embed that points to a directory should use the last directory name as the copy target when creating the entry in the embedded files/ dir. I.e., --embed ../other-dir/my-dir -> files/my-dir.
Either way, I'm happy to submit the patch. (I've been tinkering with --embed on a fork already.)
Thanks!

The logic is a carryover from Matthew's prototype module https://github.com/mholt/caddy-embed. He lays out the reasoning for that part in the README under the Site Root section.

Hmm, yeah, I think the reason for this is to make it so that URI paths don't have to contain a subfolder. The problem is that without the way we do it, you have to have a subfolder like /files/... in your URI, whereas with the way we do it, you may have a subfolder if you want, by naming it anything you like, within the source folder. Does that make sense?
I mean, if this is truly a problem I am open to changing things, I just figured people would want the option to choose whether a subfolder was required or not.

Yes a simple PR would be accepted! Thank you!

Did wordsmithing here: https://github.com/caddyserver/xcaddy/pull/206. Whether or not the PR goes in, I consider this issue resolved. Thanks!


ctraffic image updated to 22.04

Description

base image set to ubuntu 22.04
tcpdump path changed

Issue link
Checklist

Purpose

[ ] Bug fix
[ ] New functionality
[ ] Documentation
[x] Refactoring
[ ] CI

Test

[ ] Unit test
[ ] E2E Test
[x] Tested manually

Introduce a breaking change

[ ] Yes (description required)
[x] No

Introduce changes in the Operator

[ ] Yes (description required)
[x] No

/reverify

/reverify

/reverify


[CT-102] Use pre-commit to enforce Black and flake8

To make it easier for contributors to stick to the formatting guidelines for the repo, we will use pre-commit to help. This will let contributors know about formatting errors before committing their work locally. More info here on the integration and details:
https://ljvmiranda921.github.io/notebook/2018/06/21/precommits-using-black-and-flake8/

yes yes yes! I wrote about this here as well https://github.com/dbt-labs/dbt-core/issues/3196

I missed these issues so just closing this out in favor of the other


torch.distributed.elastic.multiprocessing.api.SignalException

Hi,
Thanks for your great work!
I trained with your code but always got the above exception after training for 1 or 2 hours.
I searched and found that was because the terminal window was closed even the model was training in the background with nohup.
Have you also met the same problem? and I just wonder how did you train the models in the background?
Thank you!

Thanks!
Sorry but i got no idea about this issue. We use tmux (similar to nohup) to train the models in the background. Maybe you could try it too.

Thank you. I tried tmux and it worked!


native_blockifier: export os_resources.json.

Move os_resources into a json file, so that it can be edited easily by scripting.
Move json TODOs near the constant definition (can't add comments to json).
Parse the json file into the constant OS_RESOURCES_JSON and export to python.

This change is

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (d50a7db) 70.39% compared to head (47783c1) 69.36%.

Files
Patch %
Lines

crates/native_blockifier/src/lib.rs
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1206      +/-   ##
==========================================
- Coverage   70.39%   69.36%   -1.04%
==========================================
  Files          58       58
  Lines        7131     6897     -234
  Branches     7131     6897     -234
==========================================
- Hits         5020     4784     -236
- Misses       1692     1694       +2
  Partials      419      419

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Some exotic constraints are missing corresponding signature constructs

Constructor constraints with more than 0 arguments and member constraints with or clauses containing at least one concrete type can be generated by F# programs, but passing the --sig option to the F# compiler results in a malformed signature file because these constructs aren't part of the grammar for signatures.
For example, given this file:
let inline f() = (^t : (new : int -> ^t) 5)
let inline g t u = ((^t or ^u) : (static member (^*^) : ^t * ^u -> ^v)(t, u)) // ok
let inline h u = g 1 u // static-typars portion of inferred constraint is (int or ^u)

--sig results in a signature file like this:
//error FS0010: Unexpected symbol '(' in member signature
val inline f : unit ->  ^t when  ^t : (( .ctor ) : int ->  ^t)
val inline g :
  t: ^t -> u: ^u ->  ^v
    when ( ^t or  ^u) : (static member ( ^*^ ) :  ^t *  ^u ->  ^v)
// error FS0010: Unexpected identifier in type constraint. Expected infix operator, quote symbol or other token.
val inline h :
  u: ^a ->  ^b when (int or  ^a) : (static member ( ^*^ ) : int *  ^a ->  ^b)

but this isn't a valid signature file because the definitions of f and h have invalid syntax.

See also #392, #393, and this UV. Support for signatures seems to have long since stopped tracking the current state of the language, which is unfortunate particularly because there doesn't seem to be any good way of controlling module exports without them, and CLR access controls (private/internal/etc.) aren't a substitute because they're often not usable at all (e.g. The value 'doofer' was marked inline but its implementation makes use of an internal or private function which is not sufficiently accessible and similar errors).
(Apologies to Don for my being responsible for that UV, yet still not getting around to working on a patch!)

@drvink There are definitely corner cases where things that can be inferred can't be written in signatures. And you're finding them :)  It's entirely reasonable to eventually address these and we would welcome PRs to address them.
@kbattocchi We should track this as a UV item .  Please make sure the UV item has a link here.  Consider any improvements in this space pre-approved :)
Could you give more detail on " The value 'doofer' was marked inline but its implementation makes use of an internal or private function which is not sufficiently accessible " - in what cases can't you label the appropriate transitive closure of things suitably private?

@dsyme A lot of the time, I want to hide implementation/helper functions that users of a module would have no need or reason to call directly (as they only exist for the benefit of functions I want to export), but it's not possible to hide them because if I make the helper functions or their enclosing module anything other than public, I get that error.

BTW if you're after another exotic one, try let inline f x = printfn "%d" x :) I don't think anyone has ever really noticed, but F# has a "monomorphic choices" constraint used specifically for %d and %f in printf.  There is no syntax at all for this except in signature printing.  There you go.
In this case, I don't think we would ever add syntax to the language for this kind of constraint - I regard it as just arising from the built-in support for printf and unless we generalize the machinery in some way we'll leave it at that (it should, however, be mentioned as a footnote in the F# language spec, which it isn't).  The world doesn't fall apart if you can't use inline and %d and signatures together.  However some of the above are slightly more significant.

@drvink   That is probably because of the F# language restriction that member trait constraints can only be satisfied by public members, though they can be satisfied by things hidden by signatures. This is a slightly separate consideration but in the same general zone.

@dsyme I actually ran into this before I started (ab)using overloads. :) I was doing something simple like:
let inline pchar (c : char) (p : char -> char -> bool) (errstr : string) =
  satisfy (p c) <?> errstr + Char.ToString c + "'"

let inline char (c : char) = pchar c (=) "character '"
let inline notChar (c : char) = pchar c (<>) "not character '"

...and I wanted to hide pchar, but couldn't, but I didn't want to make char/notChar not inline, since they were small functions in a performance-oriented parser.

Closing in favour of #393 which is being used as the master bug for all issues with --sig


feat: Series.value_count spec enhancement

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Checklist

[x] Code follows style guide (ruff)
[x] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below.
Expands on Series.value_counts available arguments to match polars 1.0

CI fails:

pre-commit: I though I fixed mypy, apparently not. Ideas on how to fix it are welcomed as I can't manage to find a way
all other errors are related to pyarrow_hotfix not being installed. Should we add it to test requirements?

sorry messed up the merge, will fix now


[Enhancement] Look into using common React hooks

Describe the solution you'd like
See:
https://github.com/streamich/react-use

Another hook to add seperately that isn't supported is something like useSystemTheme:
see package:
https://www.npmjs.com/package/react-use-system-theme
Except don't use this package directly, just implement it.


This page missing offline link for Ubuntu 22.04 WSL2

Version
NA
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
NA
Distro Version
Page
Other Software
NA
Repro Steps
https://docs.microsoft.com/en-us/windows/wsl/install-manual
Expected Behavior
The page must have a download link for WSL2 latest Ubuntu release.
Actual Behavior
The page missing the download link.
Diagnostic Logs
NA

Ubuntu 22.04 was added in the manual installation page. Direct link https://aka.ms/wslubuntu2204

I've also been looking for the AppBundle link for the proper 22.04 version
If you install the 22.04  version from the manual download instructions, then run cat /etc/os-release in Ubuntu, it still reports as 20.04, not 22.04.
If you install 22.04 LTS from Windows Store, it works properly.

Hi @pa-terrythibodeau, I don't know if this can help you: I have multiple Ubuntu installations in WSL. I needed to install 22.04 and I followed this guide: https://learn.microsoft.com/en-us/windows/wsl/install  and it worked fine, 22.04 is added as a new installation next to the other ones.

It is also missing offline link for Ubuntu 24.94 WSL2 now ;)

It is also missing offline link for Ubuntu 24.94 WSL2 now ;)

Yep, but... https://cloud-images.ubuntu.com/wsl/noble/current/

I would not upgrade to noble yet, I tried yesterday and it has some issues with dbus when used in WSL


java.lang.ArrayIndexOutOfBoundsException on generated code

Summary
Invalid deserialization code is generated for a fragment like this:
fragment FriendDetails on Character {
  ...CharacterDetails
  ... on Human {
    homePlanet
  }
  ... on Droid {
    primaryFunction
  }
}

leading to ArrayIndexOutOfBoundsException during deserialization.
Description
The issue can be reproduced against master from the pcarrier/20200212-repro branch.
Execute com.example.mixed_fragments_on_fragment.TestExecution.
(I couldn't find a better way to write a runtime test against the generated code)
RESPONSE_FIELDS is of length 3:
    private val RESPONSE_FIELDS: Array<ResponseField> = arrayOf(
        ResponseField.forString("__typename", "__typename", null, false, null),
        ResponseField.forFragment("__typename", "__typename", listOf(
          ResponseField.Condition.typeCondition(arrayOf("Human"))
        )),
        ResponseField.forFragment("__typename", "__typename", listOf(
          ResponseField.Condition.typeCondition(arrayOf("Droid"))
        ))
        )

FriendDetails.invoke goes up to index 3:
    operator fun invoke(reader: ResponseReader): FriendDetails = reader.run {
      val __typename = readString(RESPONSE_FIELDS[0])
      val fragments = Fragments(reader)
      val asHuman = readFragment<AsHuman>(RESPONSE_FIELDS[2]) { reader ->
        AsHuman(reader)
      }
      val asDroid = readFragment<AsDroid>(RESPONSE_FIELDS[3]) { reader ->
        AsDroid(reader)
      }
      FriendDetails(
        __typename = __typename,
        fragments = fragments,
        asHuman = asHuman,
        asDroid = asDroid
      )
    }
  }

Version
Reproduced by our customer on v1.3.0, and through this testbed on master.

com.apollographql.apollo.compiler.codegen.kotlin.KotlinCodeGen.toMapperFun calls mapIndexed, but FieldType.Fragments does not actually consume an index.

Pushed a fix.


Automatic method name extraction fails

Something has gone dreadfully wrong with the auto conversion here.  FsRtlIsNameInExpression is in the
extern "C"
namespace and _FSRTL_ADVANCED_FCB_HEADER is a (somewhat heavily kludged) standard C structure.

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

ID: 92a79a15-8ae0-3056-e883-994969448a60
Version Independent ID: 36a2a3cc-5b46-eaad-514b-40ad9baf1001
Content: FsRtlIsNameInExpression function
Content Source: wdk-ddi-src/content/ntifs/nf-ntifs-_fsrtl_advanced_fcb_header-fsrtlisnameinexpression.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

@rodwiddowson, I believe our build tool has been modified to fix the problem you reported here. Let me know if there is still an issue and I can reopen this. Sorry for the slow response - thanks as always for reporting issues!


[Event Request] Report 1002 Post Inventory Cost to G/L

We need to prevent the output of skipped Value Entries under specific circumstances, because they are posted by our extension.
Therefore, I would like to request an event to be able to do this.
Data Item SkippedValueEntry
trigger OnAfterGetRecord()
TempValueEntry
    PreventOutput: Boolean;
begin
    if TempValueEntry.Next() = 0 then
        Clear(TempValueEntry);

    // ++ New Code
    PreventOutput := false;
    OnPreventOutputOfSkippedValueEntry(TempValueEntry, PreventOutput);
    if (PreventOutput) then
        CurrReport.Skip();
    // -- New Code

    SetRange("Item No.", TempValueEntry."Item No.");
    SetRange("Entry No.", TempValueEntry."Entry No.");

    if Item.Get("Item No.") then;
    if "Expected Cost" then
        CostAmt := "Cost Amount (Expected)"
    else
        CostAmt := "Cost Amount (Actual)";
end;

Event Definition
[IntegrationEvent(false, false)]
local procedure OnPreventOutputOfSkippedValueEntry(TempValueEntry: Record "Value Entry" temporary; var PreventOutput: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 43560.


Issue credits to Peter

Issue 10000 credits to @PeterEHart

 Follow up
On Thursday, 28-Mar-24 19:28:52 UTC by Gov4Git dev
Issued 10000 credits to member @peterehart.


Airplay radio issue returned after 2.0.0b74 upgrade

What version of Music Assistant has the issue?
2023.6.0b3
The problem
After upgrading music-assistant/server from 2.0.0b73 to 2.0.0b74, I am experiencing an issue which also existed before, but which works without problems in 2.0.0b73.
If I steam a radio station to an AirPlay device, the stream changes from normal music to loud white-noise-ish after 78-79 minutes. I have tested this several times, using tunein and the radio stations 538 and slam. Streaming the same radio stations to a chromecast audio device goes all day without problems.
I have also tested that after downgrading to 2.0.0b73 the problem no longer occurs, after upgrading again to 2.0.0b74 the problem reappears.
How to reproduce
Upgrade music-assistant/server from 2.0.0b73 to 2.0.0b74, start a radio station using tunein (538/slam tested), play it to a AirPlay device for 78-79 minutes.
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2023.11.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Thanks for the report. This will get reviewed in due course

Aha ... I think that's what happened yesterday to me too - after playing a radio station on my Airplay speaker for some time (cannot vouch for it to be around 78 mins, but that sounds about right). As this was a first for me, I thought it was a just a one-off glitch, so didn't think much of it ...

This should be fixed in beta 75. Let us know if not.


[Bug][GitHub] Fail to add public repos to a GitHub connection

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
It threw a 500 error.

What do you expect to happen

Successfully added the public repos.
Plus, it's better to show the repos with the namespace. Otherwise, it's hard to tell whether I have added the right repos.

How to reproduce

Install DevLake v0.18.0-beta1
Create a GitHub connection
Add public repos and save

Anything else
No response
Version
v0.18.0-beta1
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Seems like the database is using the wrong collation setting

@ZhangNing10 Thanks. I'll close it.


feat: execute onBeforeLeave and onBeforeEnter when changing parameter

When all elements in the new navigation can be reused from previousChain,
router will reuse them and run onBeforeLeave and onBeforeEnter
of the element corresponding to the changed segments.
Fixes https://github.com/vaadin/vaadin-router/issues/387

This change is

This PR also upgrades the distro image in travis build because of https://github.com/travis-ci/travis-ci/issues/9361


[SPARK-33782][K8S][CORE]Place spark.files, spark.jars and spark.files under the current working directory on the driver in K8S cluster mode

What changes were proposed in this pull request?
This PR will place spark.files , spark.jars and spark.pyfiles to the current working directory on the driver in K8s cluster mode
Why are the changes needed?
This mimics the behaviour of Yarn and also helps user to access files from PWD . Also as mentioned in the jira
By doing this, users can, for example, leverage PEX to manage Python dependences in Apache Spark:
pex pyspark==3.0.1 pyarrow==0.15.1 pandas==0.25.3 -o myarchive.pex
PYSPARK_PYTHON=./myarchive.pex spark-submit --files myarchive.pex

Does this PR introduce any user-facing change?
No
How was this patch tested?
Tested via unit test cases and also ran on local K8s cluster.

@HyukjinKwon  Please review , build is failing because of un related error (since its passing on local)

Can one of the admins verify this patch?

cc @dongjoon-hyun @tgravescs @Ngone51 FYI

Does this PR introduce any user-facing change?
No

Is this true?  If I specify a file to pass along and my code references it, what did it do before?   I know normally on yarn I would do some like ./XXXX.
I'm a bit worried here about breaking people if we move location and they expected it somewhere else.

@tgravescs
https://github.com/apache/spark/pull/30735/files#r540935585 , In case K8s files are not copied to current working directory . Files are available in local but not in current working directory for driver.

ping @dongjoon-hyun @Ngone51

ping @dongjoon-hyun @Ngone51

ping @dongjoon-hyun @Ngone51 . I think this is important feature to have the same behaviour as yarn . Please review.

The change generally looks good to me. It'd be good if @dongjoon-hyun could take a look since he has more knowledge in K8s.

Thx @Ngone51  , for reviewing this , i'll do the suggested changes.
@dongjoon-hyun Please review the PR .

Gentle ping @dongjoon-hyun

Gentle ping @dongjoon-hyun

@dongjoon-hyun , please review this , it will help to have K8s similar functionality as Yarn . Since this is target for Spark3.4 (as per jira) , IMHO , it would be great if u can spend some time on it .
cc @HyukjinKwon (since jira is created by [HyukjinKwon])

@dongjoon-hyun , Have incorporated all the review comments , please look into the same.

@HyukjinKwon @Ngone51 @dongjoon-hyun
Gentle ping to please review the same.

@HyukjinKwon  , can u please reviewed or get it reviewed. Since this is important jira (opened by you) and also fix parity with yarn .

@Yikun can u please review this PR .

@HyukjinKwon  Thx for giving LGTM.  Please merge the PR

Merged, thanks for the reminder.

Thx @holdenk for your help .

Thanks for making the PR :D

Maybe this is not the correct place to ask, but this PR had created some issues for our flow. Maybe our flow was not how things were intended to be used but it goes like this:
We submit jobs on kubernetes by baking in the jar in the docker image, we copy the jar in the docker image on the workdir location.
The result of this PR is that our jar is removed, because of the Utils.deleteRecursively(dest).
Now a quick fix is to copy the jar somewhere else, but if this sounds like unintended behavior to you guys, I am willing to open up a PR to fix it.
I am however not sure what the intended behavior would be. Just remove this Utils.deleteRecursively(dest) sounds a bit too simple :)

@pralabhkumar @HyukjinKwon @holdenk
Facing issue with the new Spark version which is using Files.Copy in SparkSubmit.scala.
The latest change in the SparkSubmit.scala is causing the NoSuchFileException.
The mentioned "sample.jar" is present in the mentioned path /opt/spark/work-dir/sample.jar inside the docker image.
My same code is working fine with Spark 3.3.0.
Here is the Error Message
Files  local:///opt/spark/work-dir/sample.jar from /opt/spark/work-dir/sample.jar to /opt/spark/work-dir/./sample.jar
Exception in thread "main" java.nio.file.NoSuchFileException: /opt/spark/work-dir/sample.jar
        at sun.nio.fs.UnixException.translateToIOException(UnixException.java:86)
        at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:102)
        at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:107)
        at sun.nio.fs.UnixCopyFile.copy(UnixCopyFile.java:526)
        at sun.nio.fs.UnixFileSystemProvider.copy(UnixFileSystemProvider.java:253)
        at java.nio.file.Files.copy(Files.java:1274)
        at org.apache.spark.deploy.SparkSubmit.$anonfun$prepareSubmitEnvironment$14(SparkSubmit.scala:437)
        at scala.collection.TraversableLike.$anonfun$map$1(TraversableLike.scala:286)
        at scala.collection.mutable.ResizableArray.foreach(ResizableArray.scala:62)
        at scala.collection.mutable.ResizableArray.foreach$(ResizableArray.scala:55)
        at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:49)
        at scala.collection.TraversableLike.map(TraversableLike.scala:286)
        at scala.collection.TraversableLike.map$(TraversableLike.scala:279)
        at scala.collection.AbstractTraversable.map(Traversable.scala:108)


Move settings related for disabling console proxy to settings.yml

Move settings related to disabling console proxy to settings.yml
this PR is follow-up for ManageIQ/manageiq#12675
\cc @Fryguy

@yrudman I know we usually go alphabetical, but its really strange to see end before start.  Can you reverse it?
Otherwise, LGTM, but, @martinpovolny, I am questioning whether proxy_port makes sense as a key name.  It's not clear at all that this is actually the console proxy_port in context of the other keys under server.  Should the key name be console_proxy_port instead?

@Fryguy : console_proxy_port makes sense to me.

@Fryguy ready for merge

Removing blocker because #12675 had blocker removed.

Euwe backport details:
$ git log -1
commit feda28107471e8ed112b44cecf5d7ed43dab2b56
Author: Jason Frey<EMAIL_ADDRESS>Date:   Tue Nov 29 14:30:16 2016 -0500

    Merge pull request #12891 from yrudman/move-disable_console-settings-to-yml

    Move settings related for disabling console proxy to settings.yml
    (cherry picked from commit 076385ccea94424d153bebcdaddaf9ddcf96a49d)

    https://bugzilla.redhat.com/show_bug.cgi?id=1399677


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
5b96dd7 -> bab77a8

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.35.1 -> v1.38.1

go (source)
toolchain
minor
1.22.4 -> 1.23.1

terraform-ibm-modules/common-pipeline-assets
action
patch
v1.22.3 -> v1.22.4

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.38.1
Compare Source
Bug Fixes

deps: update gomod (#857) (87710ff)

v1.38.0
Compare Source
Features

add function to check ingress health status of a cluster (#751) - Added new function testhelper.CheckClusterIngressHealthy() which can be used in Terraform tests to validate the status of an OpenShift cluster (801dd27)

v1.37.2
Compare Source
Bug Fixes

deps: update gomod (#855) (131e322)

v1.37.1
Compare Source
Bug Fixes

deps: update gomod (#854) (a79629e)

v1.37.0
Compare Source
Features

allow deep folders in stack config path (#851) (fd252ef)

v1.36.0
Compare Source
Features

simplify testprojects TearDown logic (#845) Simplified the logic around when resources or projects should get deleted during TestTearDown in the testprojects package, also added unit tests to verify all permutations. (380757f)

v1.35.4
Compare Source
Bug Fixes

bug with stack arrays (#843) (4e09e9c)

v1.35.3
Compare Source
Bug Fixes

do not destroy the project on failure so schematic workspace logs can remain for debug (#837) (72f3c8e)

v1.35.2
Compare Source
Bug Fixes

bug, fail test on any deploy error (#833) (79923b8)

terraform-ibm-modules/common-pipeline-assets (terraform-ibm-modules/common-pipeline-assets)
v1.22.4
Compare Source
Bug Fixes

added support for trailing spaces in pipeline run command (#699) (215fa1a), closes #701

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

11 additional dependencies were updated

Details:

Package
Change

github.com/IBM-Cloud/bluemix-go
v0.0.0-20240423071914-9e96525baef4 -> v0.0.0-20240719075425-078fcb3a55be

github.com/IBM-Cloud/power-go-client
v1.6.0 -> v1.7.1

github.com/IBM/go-sdk-core/v5
v5.17.3 -> v5.17.4

github.com/IBM/platform-services-go-sdk
v0.63.1 -> v0.68.1

github.com/gruntwork-io/terratest
v0.46.15 -> v0.47.1

github.com/hashicorp/go-getter
v1.7.4 -> v1.7.6

github.com/hashicorp/go-retryablehttp
v0.7.5 -> v0.7.7

golang.org/x/crypto
v0.24.0 -> v0.26.0

golang.org/x/sync
v0.7.0 -> v0.8.0

golang.org/x/sys
v0.21.0 -> v0.23.0

golang.org/x/text
v0.16.0 -> v0.17.0

/run pipeline

/run pipeline

/run pipeline