packageDeb fails to run.

Describe the bug
When running packageDeb Gradle Task, an error will occur:
External tool execution failed:
* Command: [/opt/jdk-<IP_ADDRESS>+1/bin/jpackage, @/home/balakay/Projects/Launcher/Tritium/build/compose/tmp/packageDeb.args.txt]
* Working dir: []
* Exit code: 1
* Standard output log: /home/balakay/Projects/Launcher/Tritium/build/compose/logs/packageDeb/jpackage-2024-01-17-10-15-43-out.txt
* Error log: /home/balakay/Projects/Launcher/Tritium/build/compose/logs/packageDeb/jpackage-2024-01-17-10-15-43-err.txt

Affected platforms

Desktop

Versions

Kotlin version*: 1.9.20
Compose Multiplatform version*: 1.5.11, 1.5.10
OS version(s)* (required for Desktop and iOS issues): Linux Mint 21.2 (Kernel 5.15.0-76-generic)
OS architecture (x86 or arm64): x86
JDK (for desktop issues): <IP_ADDRESS>+1

To Reproduce
Steps and/or the code snippet to reproduce the behavior:

Create a Compose Project.
Run Task packageDeb.

Expected behavior
Deb Package to be created.
Screenshots
Additional context
jpackage-2024-01-17-10-15-43-err.txt

Do you have fakeroot package installed?

i got similar error. fakeroot already installed
IntelliJ IDEA 2023.3.2 (Ultimate Edition)
Build #IU-233.13135.103, built on December 20, 2023
Runtime version: 17.0.9+7-b1087.9 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 6.5.13-7-manjaro
GC: G1 Young Generation, G1 Old Generation
Memory: 4008M
Cores: 8
Registry:
  ide.balloon.shadow.size=0
  ide.experimental.ui=true
  editor.minimap.enabled=true
Non-Bundled Plugins:
  socrates.tabshifter (0.34)
  com.nasller.CodeGlancePro (1.7.8)
  com.vecheslav.darculaSolidTheme (1.0.3)
  wu.seal.tool.jsontokotlin (3.7.4)
  com.intellij.ko (233.199)
  com.vermouthx.idea (1.15.2)
  IdeaVIM (2.7.5)
  io.cacher.intellij (1.1.1)
  com.c5inco.modifiers (1.0.14)
  com.intellij.ideolog (<IP_ADDRESS>)
  com.markskelton.one-dark-theme (5.9.0)
  com.godwin.json.parser (1.7.1)
  com.levinzonr.arch.jetpackcompose.plugin (1.1.1)
  Key Promoter X (2023.3.0)
  indent-rainbow.indent-rainbow (2.2.0)
  com.wakatime.intellij.plugin (14.3.10)
  org.intellij.RegexpTester (2.2.0)
  com.intellij.nativeDebug (233.13135.65)
  String Manipulation (9.12.0)
  GrepConsole (12.22.1-IJ2021.1)
  net.seesharpsoft.intellij.plugins.csv (3.2.3-233)
  com.intellij.ml.llm (233.13135.116)
  com.intellij.lang.jsgraphql (233.13135.65)
  izhangzhihao.rainbow.brackets (2023.3.9-233)
  org.jetbrains.compose.desktop.ide (1.5.11)
  zielu.gittoolbox (500.1.5+222)
  mobi.hsz.idea.gitignore (4.5.2)
  cn.yiiguxing.plugin.translate (3.5.6)
Kotlin: 233.13135.103-IJ
Current Desktop: KDE

I have tried jdk 17.10 (timurrin), corretto-<IP_ADDRESS>, oracle openjdk 21 for the Gradle task, all they have created the deb package correctly on Ubuntu 22.04.03 with fakerooi installed. May you try to launch jpackage manually and fire the bug to OpenJDK?

Thanks for check. i also think it is related with jpackage.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Update create_donation_and_payment.robot

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

@rponti-sforg and @skristem Out of curiosity, does (or can) this test handle the newer payment validation logic being built:

Prevent a Payment record from having both Paid and Written-Off Checked (currently in QA)
Require a Payment Date on Payments marked as "Paid" (related to Kiet's work item that won't be merged until next week)

@force2b I and @rponti-sforg can look at those WIs and see if we need new test cases or update existing ones to cover those scenarios.


About Vimeo Dataset.

What is the frame rate for the videos in the dataset? Also, what is the sampling frequency for the triplet and septuplet datasets?

I'd also like to know what the FPS is.


Jhauser 2022 0.3.1

Updates for version 0.3.1-draft.

@jhauser-us Can create a release tag for 0.3.1-draft spec ?

Patience.  I'm not ready to do that yet.


Postprocessing of the composite

Implement logic according to the spec
p. 3, p. 4

[ ] fix UUID


Make the grammar PCRE-compatible

This pull request changes 1 regular expressions in an attempt to make the grammar PCRE-compatible. While Atom uses an Oniguruma engine, github.com (which rely on this grammar for YAML highlighting) uses a PCRE-based engine. The two engines interpret \h differently, and only Oniguruma engines support implicit count modifiers ({,n}).

Merged and tagged as a bugfix release. Thanks!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/35833

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/59510


A Question about crop operation

I raise a question when learing from your code. You implemented _crop() in img_process.py but I don't know why you crop image's width for the second time in return line. It seems that you have cropped the width. Is that a mistake? Thanks for your answer.
def _crop(self, img):
     r"""
     Load an raw image given its id.
     Every loader should implement its own method.
     """

     # Image shape (H, W, C)
     crop_left, crop_right, crop_top, crop_bottom = self.crop_proportion
     crop_left = int(img.shape[1] * crop_left)
     crop_right = int(img.shape[1] * (1 - crop_right))
     crop_top = int(img.shape[0] * crop_top)
     crop_bottom = int(img.shape[0] * (1 - crop_bottom))

     # Crop the image
     img = img[crop_top:crop_bottom, crop_left:crop_right, :]
     print(img.shape)

     allowed_height = int(
             img.shape[1] * self.target_height / self.target_width)

     return img[:allowed_height, crop_left:crop_right, :], crop_top, crop_left

Hi @parallelarc
You are right. It was a mistake and I just noticed it in our internal project several days ago.
Thanks very much for pointing it out.  I revised it in the commit f1cbf95.

Thanks again!


fix: inputs dep in configuration page

Jira: https://jira.splunk.com/browse/ADDON-37812
Changes:

Remove dependency on inputs config when configuration page is visited.

Fixes https://github.com/splunk/addonfactory-ucc-base-ui/issues/37.

License check is fixed in the main branch.


[AUTO_SCHEDULER] Supported CSE (variable definitions) in feature extraction

Add supported for LetStmts in feature extraction. A stack of variable definitions is maintained and added to the arithmetic analyzer at the appropriate points. The buffer access analysis now creates a new arithmetic analysis context per set of loops to avoid redefining variables which is unsafe in the presence of let statements.
@tqchen @junrushao1994 @merrymercy

@junrushao1994 Could you review again.

CC @tqchen for a second look


Visualization of plan optimization

Visualize the query plan as a graph helps a lot when understanding and debugging the optimization plan with many rules to apply.
petgraph is preferred.
A minimal web console is also required.

Currently using dot to generate the dot file on the fly is already sufficient.
Introducing a full featured graph library may be too heavy.


0.18 alpha: refer to value, not v, inside date to stringing

Problem
From https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/elm-dev/LZslB1keLhE/u0Faw3PwAwAJ
tl;dr when trying to use a date inside debugging mode, an error is thrown as v is undefined
TODO: the code to reproduce is incoming
Solution

v should've been value
I checked for any other mismatches, and it looks good to me

Thoughts
💭  maybe v should be replaced with value in Utils toString too, to keep them in sync?

Thanks for chasing this down. I've put an example that triggers the error here

Confirmed this is fixed based on that example. Thanks!


failed to send message to server, giving up, 0xc0000041

An interactive ctf exploration tool by @taviso.
Type "help" for available commands.
Most commands require a connection, see "help connect".
ctf> connect
The ctf server port is located at \BaseNamedObjects\msctf.serverDefault2
NtAlpcConnectPort("\BaseNamedObjects\msctf.serverDefault2") => 0
Connected to CTF<EMAIL_ADDRESS>Handle 00000238
ctf> scan
failed to send message to server, giving up, 0xc0000041
ctf>

same 0xc0000041 error at scan

STATUS_PORT_CONNECTION_REFUSED

Same  0xc0000041 error at scan

same here on Windows 10.
Please tell me if need more info about.


Can't run the code, SyntaxError: invalid syntax

I suppose I'm the only one with this problem, seeing everyone else can actually run the program.
I tried both
python3 main.py --network_type rnn --dataset ptb --controller_optim adam --controller_lr 0.00035 \
               --shared_optim sgd --shared_lr 20.0 --entropy_coeff 0.0001

python3 main.py --network_type rnn --dataset wikitext

but an error comes out with
File "main.py", line 28
    raise NotImplementedError(f"{args.dataset} is not supported")
                                                               ^
SyntaxError: invalid syntax

Does anyone have any idea why?  I'm using Ubuntu on Docker, with PyTorch installed (as well as those listed in requirement.txt, except pygraphviz (due to installation error, but this shouldn't raise any errors until it's actually called in utils.py, which I commented out anyway).

f"{args.dataset} is not supported"
This is Python 3.6+ syntax. Trying removing the f or upgrading your python

PyTorch 0.4.0 was released the other day with the addition of 0 dimensional tensors. In general, this will probably break all PyTorch tutorials and code.
I assume because of that error you have 0.4.0 but I could be wrong.

@beduffy That is indeed the issue. The program now runs fine when I use PyTorch 0.3.1 instead of 0.4.0.  Thank you.


Landing Page

I will design landing page .
please assign this issue to me .

Clippy is made for ease of access and quickly sharing message or files, so a handling page just harms the experience. So, there is no need for a Landing Page currently.


Is there any update on Wifi Adapter?

Is your feature request related to a problem? Please describe.
I have OBD2 wifi, so can I use this library for the Wifi?
Describe the solution you'd like
That would be great if we could connect to Wifi OBD2 and have documentation around that.

I've added wifi, you can use it by setting OBDService(connectionType: .wifi). I haven't added documentation yet because when I tested it on a cheap wifi adapter I kept getting errors, but I'm purchasing a better one soon. You can test it out if you run into issues let me know and I'll tweak the wifi class

I have ordered the Veepeak Mini Wifi Wi-Fi adapter, and I will receive it on April 8th. I will check it out once it arrives. https://www.amazon.in/Veepeak-Scanner-Adapter-Diagnostic-Trouble/dp/B00WPW6BAE

Are there any developments on this issue?

@habipyesilyurt17 The wifi connection seems unreliable and tends to response with no data to valid request sometimes, nothing I did helped so I am working on implementing a retry mechanism that resends the command a couple more times. I'm not done that yet but I have the next 3 days off where I would try to complete it. This is what I have so far
class WifiManager: CommProtocol {
    func scanForPeripherals() async throws {
    }

    let logger = Logger(subsystem: Bundle.main.bundleIdentifier ?? "com.example.app", category: "wifiManager")

    var obdDelegate: OBDServiceDelegate?

    @Published var connectionState: ConnectionState = .disconnected
    var connectionStatePublisher: Published<ConnectionState>.Publisher { $connectionState }

    var tcp: NWConnection?

    func connectAsync(timeout: TimeInterval, peripheral: CBPeripheral? = nil) async throws {
        let host = NWEndpoint.Host("<IP_ADDRESS>")
        guard let port = NWEndpoint.Port("35000") else {
            throw CommunicationError.invalidData
        }
        tcp = NWConnection(host: host, port: port, using: .tcp)
        try await withCheckedThrowingContinuation { (continuation: CheckedContinuation<Void, Error>) in
            tcp?.stateUpdateHandler = { newState in
                switch newState {
                case .ready:
                    print("Connected")
                    self.connectionState = .connectedToAdapter
                    continuation.resume(returning: ())
                case let .waiting(error):
                    print("Waiting \(error)")
                case let .failed(error):
                    print("Failed \(error)")
                    continuation.resume(throwing: CommunicationError.errorOccurred(error))
                default:
                    break
                }
            }
            tcp?.start(queue: .main)
        }
    }

    func sendCommand(_ command: String) async throws -> [String] {
        guard let data = "\(command)\r".data(using: .ascii) else {
            throw CommunicationError.invalidData
        }
        logger.info("Sending: \(command)")
        return try await withRetry(retries: 3, delay: 0.3) { [weak self] in
                    try await self?.sendCommandInternal(data: data) ?? []
        }
//        return try await withCheckedThrowingContinuation { (continuation: CheckedContinuation<[String], Error>) in
//            self.tcp?.send(content: data, completion: .contentProcessed { error in
//                if let error = error {
//                    self.logger.error("Error sending data \(error)")
//                    continuation.resume(throwing: error)
//                }
//
//                self.tcp?.receive(minimumIncompleteLength: 1, maximumLength: 500, completion: { data, _, _, _ in
//                    guard let response = data, let string = String(data: response, encoding: .utf8) else {
//                        return
//                    }
//                    if string.contains(">") {
//                        self.logger.info("Received \(string)")
//
//                        var lines = string
//                            .components(separatedBy: .newlines)
//                            .filter { !$0.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty }
//                        lines.removeLast()
//
//                        continuation.resume(returning: lines)
//                    }
//                })
//            })
//        }
    }

    private func sendCommandInternal(data: Data) async throws -> [String] {
            try await withCheckedThrowingContinuation { (continuation: CheckedContinuation<[String], Error>) in
                self.tcp?.send(content: data, completion: .contentProcessed { error in
                    if let error = error {
                        self.logger.error("Error sending data: \(error)")
                        continuation.resume(throwing: error)
                        return
                    }

                    self.tcp?.receive(minimumIncompleteLength: 1, maximumLength: 500, completion: { data, _, _, error in
                        if let error = error {
                            self.logger.error("Error receiving data: \(error)")
                            continuation.resume(throwing: error)
                            return
                        }

                        guard let response = data, let string = String(data: response, encoding: .utf8) else {
                            self.logger.warning("Received empty response")
                            continuation.resume(throwing: CommunicationError.invalidData)
                            return
                        }

                        if string.contains(">") {
                            self.logger.info("Received response: \(string)")

                            var lines = string
                                .components(separatedBy: .newlines)
                                .filter { !$0.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty }
                            lines.removeLast()

                            continuation.resume(returning: lines)
                        } else {
                            self.logger.warning("Incomplete response received")
                            continuation.resume(throwing: CommunicationError.invalidData)
                        }
                    })
                })
            }
    }

    private func withRetry<T>(retries: Int, delay: TimeInterval, task: @escaping () async throws -> T) async throws -> T {
            var attempt = 0
            while true {
                do {
                    return try await task()
                } catch {
                    attempt += 1
                    if attempt >= retries {
                        throw error
                    }
                    logger.warning("Attempt \(attempt) failed, retrying in \(delay) seconds...")
                    try await Task.sleep(nanoseconds: UInt64(delay * Double(NSEC_PER_SEC)))
                }
            }
    }

    func disconnectPeripheral() {
        tcp?.cancel()
    }
}

let me know if the new commit fixes the issue for you

I have tested the wifi connection and getting sensors data. Everything is working as it should if you have any other issues with it let me know.


(YAML) Expand allowed characters in keys

Describe the issue
👋 Hello hello from GitHub Docs! We use highlight.js and are very grateful for your maintenance.
https://docs.github.com/en/copilot/managing-github-copilot-in-your-organization/configuring-content-exclusions-for-github-copilot#example-of-repositories-and-paths-in-organization-settings
We have a little syntax highlighting problem with a few YAML keys.

You can see GitHub.com (Ruby) handling it a little better:
# Ignore all `.env` files in any location.
# This could also have been written on a single line as:
#
# "*": ["**/.env"]
"*":
  - "**/.env"

# In the `octo-repo` repository in this organization:
octo-repo:
  # Ignore the `/src/kernel.rs` file.
  - "/src/kernel.rs"

# In the `primer/react` repository on {% data variables.product.prodname_dotcom %}:
https://github.com/primer/react.git:
  # Ignore any files in or below the `/e2e` directory.
  - "/e2e/**"

# In the `copilot` repository of any {% data variables.product.prodname_dotcom %} organization:
git@github.com:*/copilot:
  # Ignore any files in or below the `/__tests__` directory.
  - "/__tests__/**"
  # Ignore any files in the `/script` directory.
  - "/script/*"

# In the `gitlab-org/gitlab-runner` repository on GitLab:
git@gitlab.com:gitlab-org/gitlab-runner.git:
  # Ignore the `/main_test.go` file.
  - "/main_test.go"
  # Ignore any files in any `packages` directories, anywhere in this repository.
  - "**/packages/*"
  # Ignore any files with names beginning `server` or `session`, anywhere in this repository.
  - "**/{server,session}*"
  # Ignore any files with names ending `.md` or `.mk`, anywhere in this repository.
  - "**/*.m[dk]"

I believe the relevant code would be here:
https://github.com/highlightjs/highlight.js/blob/583cb33627474ff577305dc787f9a2a09eaf3b88/src/languages/yaml.js#L16-L30
For our purposes we would need @ and * supported, but I believe there's a wider range of supported characters for YAML keys.
Which language seems to have the issue?

YAML
Are you using highlight or highlightAuto?
highlight
Sample Code to Reproduce

See description above
Expected behavior

See description
Additional context

N/A

I'd say a PR would be welcome for this.

Hello! @joshgoebel. I'm interested in this issue. Can I ask for your help if I need it during the resolution?

Sure.

is this issue still open ?

ive managed to fix the issue and ive added a PR for the same @joshgoebel  please review
[(https://github.com/highlightjs/highlight.js/pull/4096)]


Issues #76, #118, #39, #155 Resolved

issue #39: removed style from internal components of GraduatedBar and applies style only on root element
issue #155: added a new prop currentValueStyle in daq.Tank component which takes the same input dict as style attribute.
issue #118: replaced the 'on' value passed in the persisted props with 'value'
issue #76: added a new prop/attribute fullSize in daq.Slider component which takes in boolean value True/False. for vertical=True you will need to provide a custom-defined height in units (px, em, etc...) otherwise fullSize=True will not work.

@vinicvaz or @luiztauffer can you please review?


Webots

You did not provide a description. Do you have a question or problem?

@gviollaz In the interest of keeping the issue tracker clean, I'll close this issue now but please do not hesitate to reopen it, perhaps along with a description as @dbscoach suggested :slightly_smiling_face:
Thanks!


Publish Site: dataset entry: no authors entry on paper form

The form for adding papers to a dataset has no field(s) for adding authors.

Oops. I actually missed seeing the fields for adding authors. This is NOT an issue: the fields are there.


fix kubeletconfig API description

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.10 Features: set Milestone to 1.10 and Base Branch to release-1.10
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

This is a dup of https://github.com/kubernetes/website/pull/7561.

This is a dup of #7561.

Thanks for point it out.


[x] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Closing in favor of https://github.com/langchain-ai/langchain/pull/28768 (maintainers don't have permission to push to this branch).


docs

Update the README because it got a little messy when we added .plan.

@nichoth Thank you :)


Issues with 2017.3.0p3 patch for UWP build

I've tried using both .NET 3.5 and 4.6 with the newest Unity patch and this package. 3.5 gives me a ton of errors:
Assets/Xbox Live/Scripts/UnityTaskExtensions.cs(5,24): error CS0234: The type or namespace name Tasks' does not exist in the namespace System.Threading'. Are you missing an assembly reference?
Assets/Xbox Live/Scripts/UserProfile.cs(9,24): error CS0234: The type or namespace name Tasks' does not exist in the namespace System.Threading'. Are you missing an assembly reference?
Assets/Xbox Live/Scripts/Leaderboard.cs(71,13): error CS0012: The type System.Object' is defined in an assembly that is not referenced. Consider adding a reference to assembly System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'
Assets/Xbox Live/Scripts/Leaderboard.cs(81,13): error CS0012: The type System.Collections.Generic.IEnumerable1<Microsoft.Xbox.Services.Social.Manager.XboxSocialUser>' is defined in an assembly that is not referenced. Consider adding a reference to assembly System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' Assets/Xbox Live/Scripts/Leaderboard.cs(81,13): error CS0012: The type System.Collections.IEnumerable' is defined in an assembly that is not referenced. Consider adding a reference to assembly System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' Assets/Xbox Live/Scripts/UnityTaskExtensions.cs(11,57): error CS0246: The type or namespace name Task' could not be found. Are you missing an assembly reference?
Assets/Xbox Live/Scripts/UnityTaskExtensions.cs(21,75): error CS0246: The type or namespace name Task' could not be found. Are you missing an assembly reference? Assets/Xbox Live/Scripts/UnityTaskExtensions.cs(40,33): error CS0246: The type or namespace name Task' could not be found. Are you missing an assembly reference?
Assets/Xbox Live/Scripts/UnityTaskExtensions.cs(60,12): error CS0246: The type or namespace name Task' could not be found. Are you missing an assembly reference? Assets/Xbox Live/Scripts/UnityTaskExtensions.cs(87,33): error CS0246: The type or namespace name Task' could not be found. Are you missing an assembly reference?
Assets/Xbox Live/Scripts/UnityTaskExtensions.cs(91,16): error CS0246: The type or namespace name Task' could not be found. Are you missing an assembly reference? Assets/Xbox Live/Scripts/UserProfile.cs(102,64): error CS0012: The type System.EventArgs' is defined in an assembly that is not referenced. Consider adding a reference to assembly `System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'
4.6 gives me just one:
Assets\Xbox Live\Scripts\UserProfile.cs(148,49): error CS0234: The type or namespace name 'UserPicker' does not exist in the namespace 'Windows.System' (are you missing an assembly reference?)
Any ideas would be welcome.

I'm using the latest SDK version and am getting these errors with 3.5.
What import settings do I need to change, and what do I need to change them to?

Why not just switch to 4.6?


Improvement: add switch to inteoduce 'use' statements for functions instead of backslash

It can be argued that code would be kept cleaner by adding at the top of the modified file 'use' statements for each of the optimizeable functions instead of backslashes in the calls.

I agree, but it's not cleanness what this code is aiming for, but (micro) performance optimization.
Loading a class in the use, makes it loaded beforehand, not when needed down the lines.


Error when using _ as name for Bound argument in class method

We are using _ as an argument name for the &Bound<'_, PyType> argument to a classmethod and I get the following error:
error: Expected typed argument
   --> project/src/lib.rs:107:9
    |
107 |         _: &Bound<'_, PyType>,
    |         ^

error: could not compile `nrf-probe-py` (lib) due to 1 previous error

If I rename it to bound or _bound then the error goes away. The argument isn't used in our current implementations.
Is that expected?

I guess it causes from #[pyfunction] rather than #[gen_stub_pyfunction].
#[pyfunction]
fn test_empty(_: Bound<'_, PyType>) {
    println!("test_empty");
}

will causes
error: wildcard argument names are not supported
  --> examples/pure/src/lib.rs:89:15
   |
89 | fn test_empty(_: Bound<'_, PyType>) {
   |               ^


redirect issue creation to api repo

Signed-off-by: Stephanie<EMAIL_ADDRESS>What does this PR do?
This issue deletes the issue template in library, and redirects issue creation to api repo(which is our issue tracking repo)
What issues does this PR fix or reference?
https://github.com/devfile/api/issues/472
Is your PR tested? Consider putting some instruction how to test your changes

Want to add more links after issue templates have been create/updated in api repo, as part of https://github.com/devfile/api/issues/472
Convert this PR to draft PR for now.

Closing this pr since we want to disable the issue creation for devfile/library to be consistent with other repos


Unable to delete registry value

Hi,
I created a started app with Yeoman using<EMAIL_ADDRESS>Here is the console output:
`npm start desktop

<EMAIL_ADDRESS>start C:\Users\xxx\excel-addin
office-addin-debugging start manifest.xml "desktop"

Debugging is being started...
App type: desktop
Enabled debugging for add-in 05c2e1c9-3e1d-406e-9a91-e9ac64854143. Debug method: 0
Starting the dev server... (webpack-dev-server --mode development)
The dev server is running on port 3000. Process id: 22816
Sideloading the Office Add-in...
Error: Unable to start debugging.
Error: Unable to sideload the Office Add-in.
Error: Unable to delete registry value "C:\Users\xxx\excel-addin\manifest.xml" in key "HKCU\SOFTWARE\Microsoft\Office\16.0\Wef\Developer".
ProcessUncleanExitError: DELETE command exited with code 1:
Erreur�: Erreur�: le syst�me n'a pas trouv� la cl� ou la valeur de Registre sp�cifi�e.`
This error is preventing the debugging from starting. After digging aroud a bit, I think issue lies with the RegistryKey class. The deleteValue function (line 112) is matching the error message to a specific string in english, but my system is returning an error message in french.
If I create the value it is looking for in the registry before starting the debugger I do not have an issue.

Thanks for providing the information that this was related to the language of the error message.
I've made a pull request to fix this: https://github.com/OfficeDev/Office-Addin-Scripts/pull/216

The fix has been published to office-addin-dev-settings@next so if you use "npm install -D office-addin-dev-settings@next" to get the latest version, you should be able to confirm that it has fixed this issue.


新增nacos-kingbase插件

新增nacos-kingbase插件
升级nacos版本2.3.0-SNAPSHOT ==> 2.4.2

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add missing changelog entries

Travis CI's infrastructure for iOS builds has been at capacity for almost 20 hours, so I'm merging this without waiting for the CI tests to finish.


ci: using extended-runner for tests

Description
Forcing the use of extended-runner for running tests. This will help to random errors in ci pipelines.

May I ask what's the extended-runner is that something custom?

May I ask what's the extended-runner? is that something custom?

Yes. A runner with more resources than default one https://github.com/organizations/argilla-io/settings/actions/github-hosted-runners/7?viewing_from_runner_group=false


[Bug] Appears a threshold value of zero is being treated like no data.

Describe the bug
I have a boom table panel that I use as a Key so viewers can tell what icons mean when they appear in other panels. Using the Font Awesome character like a wrench, or check, or hour glass.
While testing deployment options that grab the latest plug ins I found that the icons were not displaying properly in some cases. I was seeing lots of "No Data" when I was expecting a font awesome character.
In the case of my Key panel the data value is 0. S0 thresholds are setup that a value of zero should display a green check mark. Instead I get "No Data".
It looks to me like a value of zero is being interpreted as  no data/missing data.  Even when it is a legitimate value.
Screenshots
BoomTable v1.0.0 (current version we use)

Boom Table v1.4.0
(Also shows the sort order difference mentioned in another issue)

Version Details:
Grafana version : 7.2.1
Plugin version : 1.4.0
Additional context

after upgrade grafana 6.5.x->7.3.3 and then plugin to 1.4.0
have the same issue
all zeros turned into "No Data" values :(

same, Influxdb Value of Zero, treated as Null / No Data

same, Influxdb Value of Zero, treated as Null / No Data

Same issue. Prometheus metric with a value of 0 is treated as Null / No Data.
Grafana 7.2.0 and plugin 1.4.0
I have a simple query checking if a service is up/down.
node_systemd_unit_state{instance="test-server",name="firewalld.service",state="active" results in a 0 as the service is down, but is shown as N/A on the current Boom Table.

This seems to be fixed in the latest in GitHub.
@yesoreyeram can the version in Grafana be updated?
In the meantime, try pulling this repo and building and deploying it to your Grafana cluster yourself.

If this fix can be released it would be great, I have a lot of Status checks that return 0/1 for fail/success.

Hi there!
Also, please fix this bug.

The v1.5.0-alpha.3 release seems to fix this, as reported by mcbirse in a test here: https://github.com/jasonacox/Powerwall-Dashboard/issues/49#issuecomment-1179510741
@yesoreyeram if v1.5.0 is still too alpha, would it be possible to branch v1.4.1 tag and pull just this "No data" fix into a v1.4.2 release?


Compaction Iterator Bug

I am not sure if this is a bug with the FileCompactor, the SlowIterator or not a bug at all. While testing 2.1 with my changes in #2541, I have Uno setup to run the SlowIterator during a compaction of some CI data. The compaction will start up but it won't get registered in ZK under the fate directory. It will never make it past this section of code in FileCompactor:

  while (itr.hasTop() && env.isCompactionEnabled()) {
      mfw.append(itr.getTopKey(), itr.getTopValue());
      itr.next();
      entriesCompacted++;

      if (entriesCompacted % 1024 == 0) {
        // Periodically update stats, do not want to do this too often since its volatile
        entriesWritten.addAndGet(1024);
      }
    }

I assume the source of the SlowIterator will always return true for hasTop(), hence why it never makes it past this code. It could just be a bug when using the SlowIterator with compactions. But if a user creates an iterator that does something similiar, I wonder if it will run indefinitely. Also, I only have the sleepTime of the SlowIterator configured for 5ms.

It is possible that this was due to the number of entries, which was at about 3 million.

3 Million * 5ms = 15000000ms
15000000ms = 250 minutes = 4+ hours

@milleruntime Only one way to find out. You definitely have to run it for 4+ hours now...

I'm not seeing the link between the code and the compaction not showing up in ZK.

I'm not seeing the link between the code and the compaction not showing up in ZK.

Yeah I don't know how the FileCompactor was running without starting a FATE Tx. I am doing some tracing through the code, which is way more complicated in 2.1 and this is as far as I got:

InternalCompactionExecutor.submit()
- new InternalJob
- threadPool.execute(internalJob);
InternalJob
- compactable.compact()
CompactableImpl.compact()
- stats = CompactableUtils.compact(tablet, job, cInfo, compactEnv, compactFiles, tmpFileName);

I'm not seeing the link between the code and the compaction not showing up in ZK.

Yeah I don't know how the FileCompactor was running without starting a FATE Tx. I am doing some tracing through the code, which is way more complicated in 2.1 and this is as far as I got:

I thought FaTE was only used to coordinate/schedule user-initiated compactions. Regular compactions due to a trigger in the compaction strategy inside a tserver shouldn't need FaTE to compact. The tserver can handle those on their own. Did something change in this regard? Or am I misunderstanding how this is supposed to work? Was this apparent bug observed for a user-initiated compaction?

I thought FaTE was only used to coordinate/schedule user-initiated compactions. Regular compactions due to a trigger in the compaction strategy inside a tserver shouldn't need FaTE to compact. The tserver can handle those on their own. Did something change in this regard? Or am I misunderstanding how this is supposed to work?

No, you are correct.

Was this apparent bug observed for a user-initiated compaction?

I am pretty sure it was a user compaction, with the testing I was doing. But it is possible it was a system compaction.

The user compaction eventually makes it through:

root@uno> listcompactions
 SERVER               | AGE       | TYPE  | REASON | READ  | WROTE | TABLE           | TABLET                                   | INPUT | OUTPUT                              | ITERATORS | ITERATOR OPTIONS
 ip-10-113-12-25:9997 |    40m58s |  FULL | SYSTEM |  475K |  475K |              ci | 1<;79999999999999a5                      |     5 |   /1/default_tablet/A000001q.rf_tmp |        [] | {}
 ip-10-113-12-25:9997 |    40m19s | MAJOR | SYSTEM |  469K |  469K |              ci | 1;266666666666666a;2000000000000003      |     5 |        /1/t-000005m/C000002q.rf_tmp |        [] | {}
 ip-10-113-12-25:9997 |    40m17s | MAJOR | SYSTEM |  469K |  469K |              ci | 1;733333333333333e;6cccccccccccccd7      |     5 |        /1/t-000005y/C000002t.rf_tmp |        [] | {}
 ip-10-113-12-25:9997 |  12s128ms | MAJOR |   USER | 2.05K | 2.05K |              ci | 1;666666666666667;6000000000000009       |    10 |        /1/t-000005w/C00000it.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |     2m45s | MAJOR |   USER | 31.7K | 31.7K |              ci | 1;399999999999999f;3333333333333338      |    10 |        /1/t-000005p/C00000fj.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |     2m16s | MAJOR |   USER | 26.6K | 26.6K |              ci | 1;1333333333333335;0cccccccccccccce      |    10 |        /1/t-000005j/C00000fk.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |      2m3s | MAJOR |   USER | 23.6K | 23.6K |              ci | 1;4000000000000006;399999999999999f      |    10 |        /1/t-000005q/C00000fl.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |     1m43s | MAJOR |   USER | 19.5K | 19.5K |              ci | 1;59999999999999a2;533333333333333b      |    10 |        /1/t-000005u/C00000fm.rf_tmp |        [] | {}
root@uno> fate print
2022-03-08T10:44:29,015 [conf.SiteConfiguration] INFO : Found Accumulo configuration on classpath at /home/mike/workspace/uno/install/accumulo-2.1.0-SNAPSHOT/conf/accumulo.properties
txid: 7bd4ad2463222699  status: IN_PROGRESS         op: CompactRange     locked: [R:+default, R:1] locking: []              top: CompactionDriver created: 2022-03-08T15:10:15.416Z
 1 transactions
root@uno>

I think I'm still unsure of what you were expecting to see that you are not seeing. If you want to test the user compaction without having to deal with the system compactions, just set the compaction ratio really high.


training issue:

I am having trouble in training the model; kindly look into it, thanks!
Not using distributed mode
Traceback (most recent call last):
File "D:\Initital_code\CSA-main\CSA-main\train.py", line 174, in
main(args=args)
File "D:\Initital_code\CSA-main\CSA-main\train.py", line 32, in main
dataset_names = list(eval(args.dataset_names))
File "", line 1, in
NameError: name 'rSfM120k' is not defined

I have updated the code a few days ago, you can re-download and try.

I tried with updated code; getting same error

The current code works fine on my machine, you need to check out yourself.

The current code works fine on my machine, you need to check it out yourself.


chore!: bump cma lib to v10. bump min node to 14

bump contentful-management dependency to v10 to make sure that tests in contentful-management repo run against the modern version
bump minimum node version to 14 to match the requirement of contentful-management@10

Please revert package-lock.json to v1

Please revert package-lock.json to v1

thanks for the reminder. Done


Multiple replace in one call.

It seems I can't replace multiple texts in one call.
For example when I have to replace 7 texts with another 7 texts, I have to call fnr.exe 7 times.
It could be done adding expressions find1, find2, find3, find4... find99 and replace1, replace2, replace3, replace4... replace99.
find1 and replace1 can be added without removing --find --replace for backward compatibility, like this.
fnr --cl --find1 "texttofind1" --replace1 "texttoreplace1" --find2 "texttofind2" --replace "texttoreplace2"
fnr --cl --find "texttofind1" --replace "texttoreplace1" --find2 "texttofind2" --replace "texttoreplace2"

Hello @bxyhxyh ,
Thank you for your suggestion.
We will look at it to check if we could implemented something like this or not.
Best Regards,
Jonathan

Hello @bxyhxyh ,
We investigated this issue this Friday. Due to time limitation and how the library has been coded, we currently prefer to don't support this option yet.
However, we recommend you to create a bat file, it could make it easier to support multiple finds and replace instead to type each one.
Best Regards,
Jonathan

Thanks for the reply,
I was using a bat file when I'm writing this issue.
Then that "wrong console behavior" happened. Console windows appeared 7 times for that and that was very annoying.
I've checked it on new 1.7.3 version on my bat file and that windows problem has fixed for my batch file.


Unable to reassert location

TestFlight 84, OG Hotspot and Genesis hotspot unable to reassert location (error, see video posted in discord).

https://discord.com/channels/404106811252408320/733699201887764511/812079779229466676


Add Support For AAJ TAK News

Checklist

[ ] I'm reporting a new site support request
[ ] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that none of provided URLs violate any copyrights
[ ] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description

Hello YouTube-DL
Please Add Support For AAJ TAK URL [ https://aajtak.intoday.in/video/pcr-coronavirus-latest-research-claims-that-july-august-will-see-a-peak-covid-19-cases-news-in-hindi-1-1202230.html ] .

Is this still an issue?


Editor Tags & Syntax Tree pads no longer work

When I open either of these pads on VSMac 8.2, I see a blank pad, and an exception is logged in Ide.log. As far as I know, both of these pads worked before the 8.2 update. I would appreciate it if this could be looked into, as this is blocking my work on https://github.com/mono/monodevelop/issues/8198. Thanks!
Ide.log
VSMac Version
Version 8.2 (build 1124)
Installation UUID: d7383566-ba67-46ee-9615-cd5de9a2edd2
	GTK+ 2.24.23 (Raleigh theme)
	Xamarin.Mac <IP_ADDRESS> (d16-2-fix-mac-updates / 86973979)

	Package version: 600000311

Mono Framework MDK
Runtime:
	Mono <IP_ADDRESS>1 (2019-02/494641b300c) (64-bit)
	Package version: 600000311

Xamarin Designer
Version: <IP_ADDRESS>1
Hash: 216490cf5
Branch: remotes/origin/d16-2-xcode11
Build date: 2019-07-25 17:42:00 UTC

NuGet
Version: <IP_ADDRESS>13

.NET Core
Runtime: /usr/local/share/dotnet/dotnet
Runtime Versions:
	3.0.0-preview7-27806-02
	2.1.9
SDK: /usr/local/share/dotnet/sdk/3.0.100-preview7-012285/Sdks
SDK Versions:
	3.0.100-preview7-012285
	2.1.505
MSBuild SDKs: /Library/Frameworks/Mono.framework/Versions/6.0.0/lib/mono/msbuild/Current/bin/Sdks

Xamarin.Profiler
Version: 1.6.10
Location: /Applications/Xamarin Profiler.app/Contents/MacOS/Xamarin Profiler

Updater
Version: 11

Apple Developer Tools
A valid Xcode installation was not found at the configured location: '/Applications/Xcode.app'

Xamarin.Mac
Version: <IP_ADDRESS> (Visual Studio Community)
Hash: d095f754
Branch: xcode11
Build date: 2019-07-25 06:36:54-0400

Xamarin.iOS
Version: <IP_ADDRESS> (Visual Studio Community)
Hash: d095f754
Branch: xcode11
Build date: 2019-07-25 06:36:54-0400

Xamarin.Android
Not Installed

Microsoft Mobile OpenJDK
Java SDK: Not Found

Android Designer EPL code available here:
https://github.com/xamarin/AndroidDesigner.EPL

Android Device Manager
Version: <IP_ADDRESS>
Hash: d03e3ea
Branch: remotes/origin/d16-2~1
Build date: 2019-07-19 20:55:38 UTC

Xamarin Inspector
Version: 1.4.3
Hash: db27525
Branch: 1.4-release
Build date: Mon, 09 Jul 2018 21:20:18 GMT
Client compatibility: 1

Build Information
Release ID: 802001124
Git revision: 0797bb4dfcd2cbad030d7fce609e247e23f855ca
Build date: 2019-07-26 12:13:29+00
Build branch: release-8.2-xcode11
Xamarin extensions: d663a25405cd5e7d0b0deefd2a26d548680f16f5

Operating System
Mac OS X 10.15.0
Darwin 19.0.0 Darwin Kernel Version 19.0.0
    Tue Jul 23 01:19:36 PDT 2019
    root:xnu-6<IP_ADDRESS>.1~4/RELEASE_X86_64 x86_64

Enabled user installed extensions
Ook Language Binding 1.0
MSBuild Editor <IP_ADDRESS>
AddinMaker 1.5.0

This has since been fixed, if I build the AddinMaker addin (which contains these pads) from source. Closing.


Standardize address component

All displays of the address should use the AddressViewComponent. We may need to strip down the styles inside of that component and accept a classname so consumers can customize it.

What exactly should we display at AddressViewComponent?

Account
Identicon
Address
Copy

Yes. The objective is to ensure we re-use UI components between different parts of the repo. At the moment, it's duplicated code.

@grod220 do we need to put 4 different displays with different designs into 1 component?

If two places are using the same design (like in the extension and send tab), they should be sharing the same component.
For those others, the design should not be shared, but a component should be written so that the shortening and copy is shared between it. Something like:
<ComponentA>
   <AddressView />
   <CopyIcon />
</ComponentA>

<ComponentB>
   <AddressView />
   <CopyIcon />
</ComponentB>

@Vadim121197, did you start this? Please drag to in progress if so.


Change connection to a list in API

I want to merge this change because resolve #3012 #3011 #3008
In API we should use lists instead of connections where pagination is not required.
Related fields that I think should be changed to lists:

[x] Fulfillment.lines
[x] User.addresses
[x] Checkout.payments
[x] Order.payments
[x] ProductVariant.images
[x] Product.collections
[x] Product.images
[x] Product.variants

Screenshots

Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] GraphQL schema and type definitions are up to date.

@dominik-zeglen This PR is ready from the backend side. We need to update Dashboard 2.0 accordingly.


automatically show logs for pods that are not progressing during deploy

example:
knative-v0.4.0 requires particular kubernetes version, hence some pods will log and fail
pod/controller-f84547646-gqs69 logs:
...
{"level":"fatal","ts":"2019-03-19T00:25:40.784Z","logger":"controller","caller":"controller/main.go:118","msg":"Version check failed: kubernetes version \"v1.10.11-gke.1\" is not compatible, need at least \"v1.11.0\"","stacktrace":"main.main\n\t/go/src/github.com/knative/serving/cmd/controller/main.go:118\nruntime.main\n\t/root/sdk/go1.12rc1/src/runtime/proc.go:200"}

progress log:
5:26:10PM: waiting on update deployment/controller (apps/v1) namespace: knative-serving
5:26:11PM:  L waiting on replicaset/controller-f84547646 (extensions/v1beta1) namespace: knative-serving ... done
5:26:11PM:  L waiting on replicaset/controller-cf84485f7 (extensions/v1beta1) namespace: knative-serving ... done
5:26:11PM:  L waiting on replicaset/controller-5b569b8fb9 (extensions/v1beta1) namespace: knative-serving ... done
5:26:11PM:  L waiting on pod/controller-f84547646-gqs69 (v1) namespace: knative-serving ... in progress: Condition Ready is not True (False)

pod/controller-f84547646-gqs69 status:
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: 2019-03-19T00:19:53Z
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: 2019-03-19T00:22:54Z
    message: 'containers with unready status: [controller]'
    reason: ContainersNotReady
    status: "False"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: 2019-03-19T00:19:53Z
    status: "True"
    type: PodScheduled
  containerStatuses:
  - containerID: docker://b87c0cf02ff2b04f9a3dac7c74c0ef04ae5fa8eddcec3c709209e5215dd20037
    image: sha256:ce2543ec455281f6904cbbf3304f840a619c0adaefc4bfa81964a1fadbfcaf13
    imageID: docker-pullable://gcr.io/knative-releases/github.com/knative/serving/cmd/controller@sha256:c9987a7b21400bd3afa01eeed54f390a7d1d24b25d87219803cdfb294a969379
    lastState:
      terminated:
        containerID: docker://b87c0cf02ff2b04f9a3dac7c74c0ef04ae5fa8eddcec3c709209e5215dd20037
        exitCode: 1
        finishedAt: 2019-03-19T00:25:40Z
        reason: Error
        startedAt: 2019-03-19T00:25:40Z
    name: controller
    ready: false
    restartCount: 6
    state:
      waiting:
        message: Back-off 5m0s restarting failed container=controller pod=controller-f84547646-gqs69_knative-serving(b7dc08dc-49dc-11e9-9fb1-42010a8001ed)
        reason: CrashLoopBackOff
  hostIP: <IP_ADDRESS>
  phase: Running
  podIP: <IP_ADDRESS>
  qosClass: Burstable
  startTime: 2019-03-19T00:19:53Z

for now added a kapp.k14s.io/deploy-logs annotation (https://github.com/k14s/kapp/blob/master/docs/apply.md) to inline deploy logs for debugging etc.


Use tox lint job and document it

use 'lint' instead of 'linters' for both tox and zuul jobs, so
they are consistent across all our projects. zuul job name is
already named ansible-navigator-tox-lint so it does not need change.
remove leftover type from envlist in tox as it was removed, as is
part of lint.
remove output lines from lint command that are not only outdated
but also irrelevant for docs inclusion.

If you could move changing the controversial parts to a separate PR, I don't see why the rest couldn't be merged.

If you could move changing the controversial parts to a separate PR, I don't see why the rest couldn't be merged.

I do not think that listing outdated linting output is controversial or that it should be outside this change. This change is about ensuring that documentation is correct too. What am I supposed to do replace linters in those incorrect output lines with lint and merge it like this, knowing that  user will see a different out when calling the tool?  Even worse, I already see how I would be asked to add another annoying changelog entry for fixing few lines of documentation, basically doubling the effort.

I do not think that listing outdated linting output is controversial

Your opinion is different from mine. It is the very definition of "controversial" :man_shrugging:

that it should be outside this change. This change is about ensuring that documentation is correct too.

Correcting the output is in the scope of this PR, deleting it isn't.

What am I supposed to do replace linters in those incorrect output lines with lint

That is exactly what I was asking you to do.

and merge it like this, knowing that user will see a different out when calling the tool?

I don't think it's different. I was originally adding a generic tox output one would see on the first run. Do you mean you see that the env is reused on the second invocation because it's already cached on your machine?

Even worse, I already see how I would be asked to add another annoying changelog entry for fixing few lines of documentation, basically doubling the effort.

I don't think that a fragment is necessary for such changes.

How about this? I will remove these lines from this review but you will fix them with PR?

Just replace the env name for now. If it'll still be different, I'll check locally and correct that.

recheck


ARROW-13307: [C++][Python][R] Use reflection-based enums for function options

@github-actions crossbow submit -g r

I wonder if there is a way to keep the enum (or enum class) paradigm and build a separate traits-like mechanism on the side to handle automated conversions/reflection.

This is certainly possible, but doesn't reduce the boilerplate as much. Also, IMO the mental overhead of requiring subsequent developers to mimic our EnumType declarations is equivalent to that of requiring them to mimic EnumTraits specializations

I was thinking about users of the C++ compute API. With an enum they don't have to care about the additional EnumTraits scaffolding.

Fair enough. IMO it's not too confusing to construct these from string literals, but if we determine that they're too confusing for users then we can revert to a normal enum. I'll write to the mailing list

cc @JohanMabille for opinions.

Also cc @emkornfield

Hi,
@pitrou I think redefining the enums this way is counter intuitive for regular C++ programmers (at least it is for me, and I am used to hard core metaprogramming). Even if it removes some boilerplate, the complexity it adds to such a basic concept does not seem to be worth it (and besides it is less efficient than a traditional enum, although this might not be relevant here).
A traditional approach of enum class and operators overloading would be more "standard". Providing a traits mechanism or equivalent that users can use to implement these operators is a nice to have but should not be the only way to plug addtitional enums in the system.

Due to discussion on the mailing list and here, we're not going to pursue this further. Thanks all for the input!


Update array expressions

const_err is now deny-by-default (not sure how this didn't get noticed by Rust's CI)
Indexes are checked on non-const arrays.
Fix some typos

💟 Thanks!


Fix InstallerUrl for eMClient.eMClient

Microsoft Reviewers: Open in CodeFlow

cc @palenshus @denelon: "Azure-Pipeline-Passed" label needs to be added to this PR.


Sync

Docs Build status updates of commit a0dc682:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update mkchromecast to 0.3.7

After making all changes to the cask:

[x] `brew cask audit --download {{cask_file}}` is error-free.
[x] `brew cask style --fix {{cask_file}}` left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 5570a70dca9d20b3675b83e068e3fa0f1f2877e0. Your contribution is still credited to you.


BugSnag type definitions cause error after update to 3.2.0

Hi,
I am using BugSnag in a NPM project and importing it via import 'bugsnag-js'; in my codebase. After updating to version 3.2.0 I get the following error when compiling.
Error at node_modules/bugsnag-js/src/bugsnag.d.ts:150:13: Subsequent variable declarations must have the same type. Variable 'Bugsnag' must be of type 'any', but here has type 'BugsnagStatic'.

I honestly have no real idea on how to fix it. I updated TypeScript at the same time and my first thought it was due to the update. I tried resetting both individually. After resetting bugsnag-js to 3.1.0 the error disappeared - error occurred with both TypeScript versions (2.3 and 2.4). No critical thing for me right now but I was wondering if I did something wrong. I am not sure what type of additional info might be needed.
Any help is appreciated.
Kind regards

Thanks for the report, @TobiasKrogh. We'll look into it.

Hi @TobiasKrogh
I've tried to replicate the issue on my end, but so far everything seems to be working fine. I'm also using Typescript 2.4 and Bugsnag JS 3.2.
The following produces no errors when running tsc.
src/main.ts
import bugsnag = require('bugsnag-js');

bugsnag.apiKey = 'foobar';

package.json
{
  "scripts": {
    "build": "tsc src/main.ts"
  },
  "dependencies": {
    "bugsnag-js": "^3.2.0",
    "typescript": "^2.4.2"
  }
}

I can get some other kind of error messages if I try to reference multiple instance of the bugsnag.d.ts file, or if I import the package like import bugsnag from 'bugsnag-js' - all without defining a tsconfig.json.
You may have tried this already, but do you get the same issue when nuking the node_modules folder?
Also, do you have anything in your typings.d.ts file for Bugsnag? Any additional information you have may help 😛

@jmshal thanks for the hint. Yes for whatever reason there was an entry about Bugsnag in the typings.d.ts. I should have checked that as well. Will not happen again 😉
Have a nice day


Update install-on-kubernetes.md

I propose to expand on the topic of variables that are taken from a Kubernetes configmap or secret, personally it was very difficult for me to find this information because it does not exist anywhere, I had to read the classes in charge of this operation, I also think that the syntax is not very clear, so I made a small modification. I remain of you.
Purpose

Issues

Fixes #

Maintainers: Check before merge

[ ] Manual verification

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


RFE: Adding logon account to dependencies

Hi everyone,
Would it be possible, once we add a dependency account to a master account to link a logon account to this dependencies?
I'm mainly using Domain Service Account on which I setup Windows Service usage account. But as those account can't connect to the server, I need to link a logon account to it in order to rotate the password on the service.
I will have to create at least 10 to 20 services a day, therefore having to do the link manually is a no-go for me.
Is there already a way with your script, or is it already in your roadmap?
Cheers,
V.

You can do this with PoShPACLI:
Add-PVFileCategory -safe $safeName -folder Root -file $fileName -category ExtraPass1Name -value $LogonAccountName
Add-PVFileCategory -safe $safeName -folder Root -file $fileName -category ExtraPass1Folder -value 'Root'
Add-PVFileCategory -safe $safeName -folder Root -file $fileName -category ExtraPass1Safe -value $LogonAccountSafe

Thanks for the info, unfortunately (unless I don't know about it), this won't do the trick for me.
As the script will be launch from the CPM, it means that I need a cred file on the CPM as well to launch the PoShPACLI right? and therefore the user in the cred file will be managed by CyberArk as well so whenever the password is rotated I will have to rotate the cred file as well.
Or am I missing something and there is a way to launch PACLI from the CPM without cred file ?
Cheers,
V

See https://github.com/pspete/PoShPACLI#example-connecting-to-a-vault.
But I am a bit confused: why not add the logon account using Add-PVFileCategory right after the onboarding? It would be a part of the same process and you can run everything from the same script.
Something like
Import-Module .\Add-PASDependentAccount.ps1

Add-PASDependentAccount `
        -MasterPassName "Operating System-ioSHARPWindowsDomainServiceAccount-iosharp.lab-serviceAccount01" `
        -SafeName Windows `
        -PlatformId INIFile `
        -address server01.iosharp.lab `
        -platformAccountProperties @{'FilePath' = 'C:\Example\example2.ini'; 'ConnectionType' = 'Windows File Sharing'; 'INISection' = 'Main'; 'INIParameterName' = 'Password'} `

Add-PVFileCategory -safe Windows -folder Root -file $fileName -category ExtraPass1Name -value $LogonAccountName
Add-PVFileCategory -safe Windows -folder Root -file $fileName -category ExtraPass1Folder -value 'Root'
Add-PVFileCategory -safe Windows -folder Root -file $fileName -category ExtraPass1Safe -value $LogonAccountSafe

Thank you very much for your answer. I will try that, and on top of that we may install the CCP so it would solve the issue as well.
Thanks again, I will put this issue as solved
Cheers,
V.


Corrupt gauntlet

Description:
Update time to be more accurate.
Changes:
Made it so you can make the new weapons
Made skilling reqs same as in game
Made time more accurate
Other checks:

[x] I have tested all my changes thoroughly.

Does this need to be updated to the new osjs version?

Does this need to be updated to the new osjs version?

yes

This should be updated, and tested to make sure its alright (after having updated it)

updated


gccgo: erroneously accepts method expressions with named pointer types

gccgo accepts this Go source:
package x

type T int
type P *T

func (T) t() {}

func _(p P) {
        // Equivalent to "p.t()", which gccgo correctly rejects.
        P.t(p)
}

cmd/compile and gotype reject both the original source above and the suggest alternative.
/cc @ianlancetaylor

The problematic code for this bug is in Selector_expression::lower_method_expression, on or around lines 11422 - 11441. The issue there if the receiver expression has pointer type, it goes directly to the type pointed to in order to look up the method. This works for calls like "(*T).x" but not with the scenario in the bug where you have separately named pointer type.
I have a tentative fix, which is to capture the named object when creating type expressions, then check the type expression to see whether it has a named object before the method lookup. This seems to do the trick.
Still working on setting up a new testcase (gccgo test suite is not as nice to work with as regular go test suite).


chore(main): release 1.2.0

:robot: I have created a release beep boop
1.2.0 (2022-09-06)
Features

adding another README feature to test release-please (99c7ef8)
adding new README feature to test release-please (67269e1)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/dan-hill2802/github-action-kitchen-terraform/releases/tag/v1.2.0 :sunflower:


Support logging of DNS requests

Is it possible to have a flag or config item that would enable the gof5 application to log the DNS requests/responses it processes without using the full debug logging?
Being able to view the domain name requested, the server that provided the response used, and the IP addresses provided in response by that server, would help a lot in debugging split routing settings.

you can modify the source code accordingly:
diff --git a/pkg/dns/dns.go b/pkg/dns/dns.go
index 781b20c..1e2f2b9 100644
--- a/pkg/dns/dns.go
+++ b/pkg/dns/dns.go
@@ -58,7 +58,7 @@ func dnsHandler(w dns.ResponseWriter, m *dns.Msg, cfg *config.Config, proto stri
        c.Net = proto
        for _, suffix := range cfg.DNS {
                if strings.HasSuffix(m.Question[0].Name, suffix) {
-                       if cfg.Debug {
+                       if true || cfg.Debug {
                                log.Printf("Resolving %q using VPN DNS", m.Question[0].Name)
                        }
                        for _, s := range cfg.F5Config.Object.DNS {

Feel free to submit a PS with different levels of logging.

My hero, thanks!
I've never written anything in go before, but I'll see if I can't figure it out when I have some free time. Great app, appreciate your having made it, and your help. 😊


Travel Advice Publisher End-to-end Tests

This branch will add end to end test for Travel Advice Publisher to publishing-e2e-tests.

The work here has been refactored into https://github.com/alphagov/publishing-e2e-tests/pull/32


Using prepared statements

First, really enjoy all 'vellum-*' packages!
It would be nice to be able to use prepared statements
(vellum:copy-from :postmodern
  '(:select primary_dgns_cd anchip_icd10d_codes
            pac1 readm_90_days death_in_postdisch
            history_d_10
      :from lds2019
      :inner-join first_pacs
      :on (:= (:type lds2019.episode_id varchar)
               first_pacs.episode_id)
      :where (:and (:= mdc $1) (:not $2))) ;; <-
 :columns '(primary-dgns-cd anchip-icd10d-codes pac1 readm-90-days death-in-postdisch history-d-10)
 :query-values '(mdc death_in_anchor) ;; <-
)

Not sure if this is the best way to do it, or, if it would cause problems with vellum:copy-from.
Let me know if you have any thoughts on this.
I can try to make a PR.

Hi @diasbruno I will have to read up on the postmodern manual on how to handle this because, honestly, I have no idea right now. What I usually do is to use good old `(:select ,column-name :from ,table-name) and that works, although is not efficient I guess.

https://sites.google.com/site/sabraonthehill/postmodern-examples/postmodern-parametized-statements
Ok, this does not seem to be overly complex, but I would say that defprepared seems to be a more common/useful use case. What do you think?

It's something nice because it can translate the type into the sql representation on the query. Like:
(postmodern:query (:select 'a :from 'b :where (:= 'b.id '$1)) 1)

Exactly.

My question is if you have a better way to pass this values down to copy-from that would be weird in other cases.
Not sure it passing something like :query-values '(1 "ok") would be the best way.

 It you want to implement it, sure go ahead. Adding vellum-postmodern:defprepared macro also sounds a like a cool idea to me.

Ok, so what we are doing? Wanna finish this up or perhaps you want me to take over?

I'm currently busy with other stuff. If you can take this branch and finish, please do.

I will take a look on the weekend. thx for your code.

Sure. Thanks for helping me with that PR.
I use this project everyday, so hope to contribute more soon.

Oh, how awesome is that. :-) Happy to see my stuff being usefull.

This took longer than expected, but I finally found some time. I think that the simplest way is the most appropriate here so I just decided that the user ought to pass the name as the query and t as :prepared. Query should be prepared before executing, and I see no benefit in wrapping cl-postgres in that respect. I still have to test my solution.

Nice. I wrapped in this package just so people wouldn't need to learn cl-postgres.
Looks great this solution tough.

Well, I suspect that this could be done better but frankly specifics of how postmodern manages prepared statements confuse me.


Make Dispose() synchronous again

Continuing off from https://github.com/ppy/osu/pull/10200, disposal doesn't seem to be a huge issue - most of the time is taken up by UnbindAllBindings which runs synchronously anyway.

Not sure I'll be able to meaningfully comment of the validity of (a)syncness of disposal when it comes to the overarching design, but as for the failing test:

The failing test is setting up a drawable to be during load when it's being removed from the hierarchy:
https://github.com/ppy/osu-framework/blob/8a60e3fa3d4cd4561f3307a678c2cfcca44387a2/osu.Framework.Tests/Containers/TestSceneLoadComponentAsync.cs#L45

The test is then supposed to wait some and then check that composite is not disposed:
https://github.com/ppy/osu-framework/blob/8a60e3fa3d4cd4561f3307a678c2cfcca44387a2/osu.Framework.Tests/Containers/TestSceneLoadComponentAsync.cs#L47-L51
The test fails on the first assert, but that's actually a red herring. The Clear call actually blocks until composite is disposed (which it can't be, as it's blocked on a load). The async-loading composite throws an exception after 10 secs:
https://github.com/ppy/osu-framework/blob/8a60e3fa3d4cd4561f3307a678c2cfcca44387a2/osu.Framework.Tests/Containers/TestSceneLoadComponentAsync.cs#L135-L140
which makes the load terminate, therefore allowing disposal to continue and finish up, failing the asserts.

The fact that there was a test for this is unnerving, as things could get potentially deadlock-y if we have consumers (the game, in particular) relying on this behaviour.

Even without deadlocking, it could manifest by disposing a drawable that's in the process of loading a large online resource...
Not sure about this change anymore. May be best to expose something synchronous (UnbindAllBindables) for specific cases when you need to drop resources immediately, and call it a day.


Improve documentation

We need to describe all Vektonn features with detailed usage examples for major ones. Also it would be nice to have documentation page with service architecture overview.
Proposed navigation structure:

Overview. This is the main page which purpose is to "sell the product". Page tells what we did, why, reveals the advantages in more detail. Links to support and license also remain here.
UseCases. High-level description of tasks that are solved using vector spaces and Vektonn. Here, along the way, basic terms are introduced, such as Vector, Attribute, DataPoint, DataSource, Index.
QuickStart. Describes a basic scenario with the simplest index configuration inside and out, without details.
Architecture. Give user a glimpse of Vektonn architecture overview. Place data flow diagram here.
Tutorials. Shows when and how to use Vektonn major features:

Filtering (aka Index Splitting)
Sharding
DataSource and Index versioning

Developer Guide:

DataSource configuration
Index configuration
HTTP API reference

Also, this gist may help.

This is partially solved by #10.


Unwanted extra new lines

StepS is talking from Discord, while ryojimochizuki is on Matrix.
Notice the extra new lines in StepS's messages.
Source of a message from StepS:
{
  "age": 2701687,
  "content": {
    "body": "loading decrypted eboot/modules via repatch",
    "msgtype": "m.text",
    "formatted_body": "\u003cp\u003eloading decrypted eboot/modules via repatch\u003c/p\u003e\n",
    "format": "org.matrix.custom.html"
  },
  "event_id": "$153227233422767EgQHb:t2bot.io",
  "m_is_internal_pagination_token": false,
  "m_sent_state": "SENT",
  "origin_server_ts":<PHONE_NUMBER>974,
  "room_id": "!SabTNcrymKhDfHdoaQ:xyz.is",
  "sender": "@_discord_283238983650246657:t2bot.io",
  "type": "m.room.message",
  "unsigned": {
    "age": 2701687
  },
  "user_id": "@_discord_283238983650246657:t2bot.io"
}

Notice the <p> and \n that shouldn't be here.

I believe it is a symtom of using marked as our markdown engine. I guess we can manually remove the  tags.

That's actually a riot render bug, afaik

It kinda is, but I also don't feel like <p> tags or newlines are exactly great things to include.

It seems to be fixed on version 0.8.17.

Yep


Update index.astro

👷 Deploy Preview for famous-marshmallow-cfbd61 processing.

Name
Link

🔨 Latest commit
2725aaba2dabcbcf6f4605d9348f4daf2b4eede4

🔍 Latest deploy log
https://app.netlify.com/sites/famous-marshmallow-cfbd61/deploys/64daeb6a9b26a7000800588f


FIx for "error with sudo after reboot"

Fixes #1032
Issue is that after reboot all pyinfra's sudo files in /tmp will be wiped, but pyinfra don't know about it
Solution is to manually clean that info

@Fizzadar
hey)


New version: tecwindow.SoundTranscriber version 1.3.3 (

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

There is currently 2 versions (one merged) in the PR so i cannot validate, but i trust 1.3.3 to not have changed drastically

Absolutely correct! We've made improvements to the code and added the Russian language, but there have been no changes to the operation or installation of the program.


Feat/motivation apply

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

[ ] Feature
[ ] Refactor
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[x] Other (please specify here: Component Creation)

Description of what this Pull Request does
Motivation to Apply Component
The Component should include a brief description of motivation to applying for a badge, description of badge levels and a table consisting of badge, levels and percentage of requirements
Related Tickets & Documents

Closes #61

QA Instructions, Screenshots, Recordings
(In this section, describe how to QA/review your  PR. Feel free to use screenshots and/or short videos)
Motivation to Apply Component Screenshot

Added/updated tests?

[ ] Yes
[ ] Not Yet, and here is why: explain here
[ ] I need help with tests
[x] No, and here is why: explain here

@kaxada kindly review


Edit: BT Sport 10 [UK]

Channel ID (required)
BTSport10.uk
Channel Name
BT Sport 10
Alternative Names
No response
Network
No response
Owners
No response
Country
UK
Subdivision
No response
City
No response
Broadcast Area
No response
Languages
No response
Categories
No response
NSFW
FALSE
Launched
No response
Closed
No response
Replaced By
No response
Website
https://www.tntsports.co.uk/
Logo
https://github.com/tv-logo/tv-logos/blob/main/countries/united-kingdom/tnt-sports-10-uk.png?raw=true
Notes
"BT Sport 10" has been rebranded and is now called "TNT Sports 10"

Please update your request:

the size of the logo should not exceed 512x512px

https://github.com/iptv-org/database/blob/master/CONTRIBUTING.md#channels

Changes made


WIP: Parameterize node/edge columns

This depends on the merger of #488 so that I can simplify how weights are ignored for layout.
Fix #493

Looks good so far, though I wonder if it would be simpler if you just changed some of the standalone functions to which you are passing params into methods, so that they could access the params directly.

I agree about changing how parameters are accessed. It was messy passing the dictionary around.
I fixed it for both bundling and layout by assigning the overridden parameters to the local instance parameters. It required a few lines of boilerplate for each class, so there may be a cleaner way to accomplish it.

Looks great now, thanks!


[request] more examples in the README

I've come across this project by pure chance, after looking into the pywayland project,
I was looking for a rusty fzf to replace the now dormant skim, it was happy surprise for me.
I can see that sweep offers good functionalities the like of which fzf has, but I couldn't get some to work for me,
like the --preview option, I couldn't find the proper way for it to work. Is it like fzf: --preview 'bat {}' or is there any other way.
I tried to run the demo python -msweep demo, it didn't work
Traceback (most recent call last):
  File "<frozen runpy>", line 189, in _run_module_as_main
  File "<frozen runpy>", line 159, in _get_module_details
  File "<frozen importlib._bootstrap_external>", line 1074, in get_code
  File "<frozen importlib._bootstrap_external>", line 1004, in source_to_code
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/salo/Documents/sweep-rs/sweep/sweep/sweep.py", line 126
    class SweepSelect[I]:
                     ^
SyntaxError: invalid syntax

From this output I couldn't figure out the problem, is it some dependency missing?
I use opensuse Tumblweed and python-3.11
Thank for all your hard work, you've put into this utility

It used to be that --preview did a different thing, but I have decided to change it to be more inline with the fzf, so now you can run ls | sweep --preview 'cat {}' it should work as expected. For now only colored output is supported, I will add more functionality in the future
with regards to syntax error, sweep.py assumes python3.12 and higher.

Thanks again for your hard work;
Yes, it works just fine.
For a fun story, I pulled the latest git version, compiled with cargo and run the example ls | sweep --preview 'cat {}', and got : Error parsing option '--preview' with value 'cat {}': provided string was not true or false.
wandering why it didn't work was about to report it, until I remembered that I did run it with old version. :)
Last, I couldn't start the demo again with python3.12 -msweep demo, I got:
usage: sweep.py [-h] [-p PROMPT] [--prompt-icon PROMPT_ICON] [--query QUERY] [--nth NTH] [--delimiter DELIMITER] [--theme THEME] [--scorer SCORER] [--tty TTY] [--height HEIGHT]
                [--sweep SWEEP] [--json] [--no-match {nothing,input}] [--keep-order] [--input INPUT] [--tmp-socket] [--log LOG] [--border BORDER]
sweep.py: error: unrecognized arguments: demo

And I tried  ls | python3.12 -msweep and did run correctly.
Thanks again.

To run the demo you need to run it from sweep-py directory

Sorry about that,


Nomad ACL bootstrap process

Creating a draft PR that can run tests while I write documentation. Please do give feedback if you got any.
Closes #147

Suggestion 1
I would suggest in template/test/Makefile add extra vars nomad_agent_acl_enable=false
up:
	vagrant up --provision
	$(MAKE) clean
	# Option 1 how to turn on ACL
	ANSIBLE_ARGS='--extra-vars "consul_agent_acl_default_policy=deny" "nomad_agent_acl_enable=true"' vagrant up --provision
	#$(MAKE) clean

And required fix
Terraform will fail without the token.
TASK [Terraform] ***************************************************************
fatal: [default]: FAILED! => {
    "changed": false
}

MSG:

Terraform plan could not be created
STDOUT: Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

nomad_job.countdash: Refreshing state... [id=countdash]

STDERR:
Error: error checking for job: Unexpected response code: 403 (Permission denied)

Suggestion 1
I would suggest in template/test/Makefile add extra vars nomad_agent_acl_enable=false
up:
	vagrant up --provision
	$(MAKE) clean
	# Option 1 how to turn on ACL
	ANSIBLE_ARGS='--extra-vars "consul_agent_acl_default_policy=deny" "nomad_agent_acl_enable=true"' vagrant up --provision
	#$(MAKE) clean

And required fix
Terraform will fail without the token.
TASK [Terraform] ***************************************************************
fatal: [default]: FAILED! => {
    "changed": false
}

MSG:

Terraform plan could not be created
STDOUT: Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.

nomad_job.countdash: Refreshing state... [id=countdash]

STDERR:
Error: error checking for job: Unexpected response code: 403 (Permission denied)

This is important, yeah. I think this should be its own PR, because this is easily fixed when the nomad secret backend is enabled and roles have been created in vault.


Game example(s)

Given the asynchronous nature of Rx and the performant aspect of this library makes it a possible candidate for rapid prototyping of games which are just transforming a stream of user interactions to a stream of graphics.
There is an example of Reactive programming used in a very elegant way, just composing together all the parts in a single statement, but it is in Scala.
There was also interest in this style of programming, at Microsoft in the 90s, for animation and multimedia of which the aforementioned Scala library is an example (still need to read the paper, which has examples in Haskel).
If Rx is as suitable as I think for games, it would be an awesome tool for game jams once a library of reusable components was built up. Starting with SDL2, I think, would make a good start and I'd be keen to help out there where I can.
(I confess I only recently started trying to write simple 2D games in order learn C++, albeit I have 20+ years as a software engineer and I've been itching to find an application of Rx I could get into)

Awesome!
I will check resources.
also I’ve checked SDL a bit to find a best way of integration. Currently I can offer to wrap SDL_PollEvent under special scheduler and use via this object to built pipes over events from SDL and schedule custom user’s events via SDL_Push underhood to use same queue.
What do you think about it? Or do you expect some examples without extra SDL-RPP specific objects?

we need some kind of observable over SDL_Events from SDL_PollEvent. It can be easily created with help of subject or create+publish. Next we only need to pass it everywhere it needed, for example, with help of singleton
I offer to create Scheduler which would push schedulables to SDL_Push. Why do we need it? Because in games every update of game logic or game state should happen in main thread, isn’t? But some updates prepend hard computation. For example,
we obtain click
then we need to check database or socket. It is long process, but we want to avoid blocking mail loop, so we schedule this computation to another thread
computation is ready and we need to update some state. We need to do it in main loop. So we need to schedule it to main thread somehow. There is two options: manual scheduler, then we need to extract events in main loop event and execute it OR create scheduler which would submit it to SDL loop via SDL_Push

I started simple example for now. It looks like this.
#include <rpp/rpp.hpp>

#include <SDL.h>
#include <cstdio>
#include <iostream>
#include <optional>
#include <ostream>

//Screen dimension constants
const int SCREEN_WIDTH = 640;
const int SCREEN_HEIGHT = 480;

struct InitData
{
    //The window we'll be rendering to
    SDL_Window* window{};
};

std::optional<InitData> init()
{
    if (SDL_Init(SDL_INIT_VIDEO) < 0)
    {
        printf("SDL could not initialize! SDL_Error: %s\n", SDL_GetError());
        return {};
    }

    InitData res{};
    //Create window
    res.window = SDL_CreateWindow("SDL Tutorial", SDL_WINDOWPOS_UNDEFINED, SDL_WINDOWPOS_UNDEFINED, SCREEN_WIDTH, SCREEN_HEIGHT, SDL_WINDOW_SHOWN);
    if (res.window == NULL)
    {
        printf("Window could not be created! SDL_Error: %s\n", SDL_GetError());
        return {};
    }
    //Get window surface
    auto screenSurface = SDL_GetWindowSurface(res.window);

    //Fill the surface white
    SDL_FillRect(screenSurface, NULL, SDL_MapRGB(screenSurface->format, 0xFF, 0xFF, 0xFF));

    //Update the surface
    SDL_UpdateWindowSurface(res.window);

    //Wait two seconds
    SDL_Delay(2000);

    return res;
}

auto create_events_observable()
{
    return rpp::source::create<SDL_Event>([](const auto& sub)
          {
              SDL_Event event{};

              while (sub.is_subscribed())
              {
                  while (SDL_PollEvent(&event))
                  {
                      sub.on_next(event);
                  }
              }
          })
          .publish();
}

void create_logic_for_mouse_move(const rpp::dynamic_observable<SDL_Event>& ev)
{
    ev.filter([](const SDL_Event&    ev) { return ev.type == SDL_MOUSEBUTTONDOWN; })
      .subscribe([](const SDL_Event& ev)
      {
          std::cout << "Mouse button clicked!" << ev.type << std::endl;
      });
}

void exec(const InitData& data)
{
    auto events = create_events_observable();

    create_logic_for_mouse_move(events.as_dynamic());

    // main-loop of processing. Block on this call till QUIT
    events.ref_count()
          .take_while([](const SDL_Event& event)
          {
              return event.type != SDL_QUIT;
          }).subscribe();
}

int main(int argc, char* args[])
{
    const auto init_state = init();
    if (init_state.has_value())
    {
        exec(init_state.value());
    }
    //Destroy window
    SDL_DestroyWindow(init_state.value_or(InitData{}).window);

    //Quit SDL subsystems
    SDL_Quit();

    return 0;
}

There I created toy implementation of such a class which can handle events and provide it for other subscribers. One thing that I try to solve: such an event provider need to notify subscribers about start of frame and about end of frame. i mean
SDL_Event event{};
while (sub.is_subscribed())
{
    <THERE START OF FRAME>
    while (SDL_PollEvent(&event))
    {
        sub.on_next(event);
    }
    <THERE END OF FRAME>
}

Best way of this one i see now: create user-define events for this one and send to subscribers. @awhillas , what do you think?

Wow, this is great!
Will try out your code on a simple example and see how I goes.
(Sorry about the late reply had family visiting for a week)
I was just thinking about the Indigo Scala framework and they have this idea of "game time" which solves a whole bunch of problems like:

Pause the game and then resume.
Serialise the current game state and load it/resume
set the frame rate
handle asynchronous events which are at different rates

With SDL2 you can create your own events at set intervals. I was thinking you could have a stream of these which is just a tick counter since the first frame. You can then ether merge this with another stream, like:

mouse events i.e. only act on asynchronous mouse events when the game tick comes in
animations which can make their calculations off this game time i.e. started at "tick 4" its now "tick 17" figure out which frame to render.

The advantage of this is you don't need the blocking SDL_Delay and the tick stream can be blocked/paused easily i.e. if they pull up the game menu or revert to a saved game. Serialising the time is just recording the tick counter value.
Indigo also talks about composing games together so you might have a score counter game which just animates and updates the score in the corner of the window, the background animations, player movement etc which might be all thought of as steams that can be combined... but haven't thought that enough yet.

Looks very interesting ;-)
If I get you correct, do you want to add custom event about ticks since last frame? As classic way of handling changes. Like...
old_time=now();
while(1)
{
cur_time=now;
update(cur_time-old_time);
old_time = cur_time;
}

About SDL_AddTimer: yeah, i've thought about using it as underhood mechanism for special SDL scheduler which would schedule tasks to "main" SDL loop.
Not sure, that i understand your thoughts about singletons and and subprocess, could you rerphase it and provide some details?

Closed due to inactivity


what  the pytorch version is?

what  the pytorch version is?

If you want to work on latest pytorch version then go through this #66.

If you want to work on the latest pytorch version then go through this #66.

@ViswanathaReddyGajjala you are right, it is working with last one.


cannot catch "date" information in traditional chinese version

System Information:

Version: 2.5
Language: zh-TW
Timezone Offset: +08:00
User Agent: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.111 Safari/537.36

What steps will reproduce the problem?

add event in traditional chinese version

What is the expected result?
should catch "date" information
What happens instead?
can only catch "time" information, but put "date" in the description of the event instead.
ex: " meeting on Feb 20, 6:30PM" --> today 6:30PM " meeting on Feb 20"
" meeting on 12/2/2016" --> today 8:16PM "meeting on 12/2"
" meeting on Wed, 6:30PM" --> today 6:30PM " meeting on Wed"
no matter the "date" information is in chinese or in english.
Screenshot URL:
Logs:

Same issue here. Is there any way to make it recognize the information in English?

OK I've found the problem, it's Google Calendar API not supporting quick add if your account language is Chinese, you can try it here, so currently the only workaround would be changing language of your account to English.

Im putting in date in a english version of google and it keeps adding the event on current day

solved my issue, i checked native google settings and where set to Dutch.... so for confussion


Will the Owl framework be the future js framework of odoo?

@windedge I'd rather not speak publicly on this topic before Odoo Experience, but this is clearly something that we are investigating.  Feel free to contact me privately if you want to discuss more on this.

I added my email address on my github profile


Prevent default on keys that can trigger browser functionality like fast search

Remark uses several keyboard shortcuts with no accelerator key (eg. p for presenter mode). In some off-the-shelf browser configurations this triggers functionality (for instance, to search for matching links). As a general rule, keys handled for actions should not also trigger their default behaviour.
This patch simply calls preventDefault() in applicable cases.

That is indeed a valid point, thank you!


docs(cn): translate content/docs/reference-react-dom-client.md into Chinese

This is my first PR, I like React, I hope i can make contribute to open source.

新版文档已经发布，将关闭这个 Pull Request


Validate TRIXO form step 4

Scope of changes
This implements the validation method for the TRIXO questionnaire in the registration form.
Type of change

[ ] bug fix
[x] new feature
[ ] documentation
[ ] other (describe)

Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible
Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[ ]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[ ]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

@pdeziel sorry, I keep introducing merge conflicts …

@pdeziel sorry, I keep introducing merge conflicts …

It's alright, I'm almost past the conflicts in this PR so hopefully part 5 is a piece of cake 😅


Add register-to-vote promo on transaction done pages

Only on completed transaction format (though we'll extend it to transaction
pages in the near future). Exclude register to vote done page, as there is
not a lot of point promoing something you've just done.
As some done pages will have register-to-vote links in their related links
we need to exclude those, and if they're the only link in a group, the
group as well.
Filtering the register-to-vote link out of the related_artefacts before
grouping makes this much easier, but should only be done for pages where
we're showing the promo.
Note: register-to-vote is filtered out by content id, as the related
artefacts don't have path/slug field, and the full web_url is pretty
long, and can vary per environment, so use the content_id, which should
be stable.
Note: the changes to the tests are not super elegant, but this change is
likely to be reverted wholesale once the promo is complete, this isn't a
pattern we're planning to keep around.

Just checked with @dsingleton how this differs from https://trello.com/c/dXgKpUBZ/340-register-to-vote-promotion-on-done-pages-3, which is in progress (see the ticket comments). This is short-lived and only going to be around for a few weeks, and this is applied to every done page and content designers can't choose. The changes in our story to make it be able to be chosen are still valid, but not as urgent, so we'll carry on with them at normal speed.

Front-end bits :+1:

@boffbowsh Updated. Otherwise are you happy with this?

Back end bits 👍


grub2: make localization resources deterministic

Description of changes
by taking them from the release tarball instead of from the (nondeterministic) translation portal.
See also https://github.com/NixOS/nixpkgs/issues/245356
Fixes https://github.com/NixOS/nixpkgs/issues/263095
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Maybe we should just build the whole thing from the tarball then? I was going to do that for the release, but then I completely forgot :(

Maybe we should just build the whole thing from the tarball then?

I personally like building from VCS source, makes it much easier to apply patches etc..
A bit more far-fetched, it might also leave fewer opportunities for an attacker to hide anything in generated artifacts (like the configure script, I suppose?)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixos-reproducible-builds-minimal-installation-iso-successfully-independently-rebuilt/34756/9