BUG: concurrent mode 'task' crashes node process. FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory

Use case
I want to benchmark how my server (runs on aws) performs (a stress test).
I need to run certain number of concurrent requests from my machine to this server. For example, I want to run 100 concurrent requests (tasks) for 60 seconds.
I tried to test how it might work using settings from the docs
from docs
import { Bench } from 'tinybench';

const bench = new Bench({ time: 100 });

bench
  .add('faster task', () => {
    console.log('I am faster')
  })
  .add('slower task', async () => {
    await new Promise(r => setTimeout(r, 1)) // we wait 1ms :)
    console.log('I am slower')

// options way (recommended)
bench.threshold = 10 // The maximum number of concurrent tasks to run. Defaults to Infinity.
bench.concurrency = "task" // The concurrency mode to determine how tasks are run.
// await bench.warmup()
await bench.run()

// standalone method way
// await bench.warmupConcurrently(10, "task")
await bench.runConcurrently(10, "task") // with runConcurrently, mode is set to 'bench' by default

it crushes during warmup or run.
Reproduction
I cloned repo to debug this test
import Bench from "src";
import { test, expect, vi } from "vitest";

test("should run concurrently using default time and iterations", async () => {
  const bench = new Bench();

  bench.add("foo", async () => 1);

  bench.concurrency = "task";
  bench.threshold = 2;

  await bench.run(); // FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
});

I found that it runs this line promises.push(limit(executeTask)); runs endlessly because while condition is always true:
while (totalTime < time || ((samples.length + limit.activeCount + limit.pendingCount) < iterations)). Specifically: totalTime < time where totalTime = 0 and time = 500.
I can fix my problem by using time: 0, warmupTime: 0, in options like it is done in your tests. I find it to be confusing. If I misconfigured a benchmark it should throw an error instead of leaking memory until it crashes.
After Theo promoted your library as benchmark js replacement I think it should have a more friendly api for concurrent tests (or more comprehensive docs) for wider adoption.

That's an amazing reproduciton, do you have a potential fix you may want to send a PR for?

One can compare each tasks against the noop benchmark and if it's below it, adapt the default settings.


Rewrite [ElmPackage]

Ref #1358

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS


Fix for 8.0.1 patch

COMBAT_LOG_EVENT_UNFILTERED event no longer provides a loadout. CombatLogGetCurrentEventInfo should be used OnEvent to retrieve that info.

Do you have a MobKillCount.lua whit your code i can download?
Thanks 👍

@Tntdruid, yeap.
Or you can open that commit and see the fix ;)

Works, Thanks for the fix!


illegal instruction error

Hi,
I am trying to run racon on some nanopore yeast data but it crashes with the error:
14645 Illegal instruction     $raconhome/bin/racon /s288c_ontreads_pass_plus2r9.fq mappings.paf layout_miniasm.fasta consensus.fasta
I was able to run a previous version of racon on the same data and using graphmap (github commit 28980bec3e98189853ed919764d5a8a9e6291264), but if I try the newest version with minimap, as suggested in the README I get the above error. Minimap seems to work fine and the contigs from miniasm look reasonable, so I don't know what can be wrong, can you help me figure it out? Thank you in advance for any help!
I ran it in this way:
# create contigs from minimap+miniasm:
minimap -Sw5 -L100 -m0 -t8  s288c_ontreads_pass_plus2r9.fq  s288c_ontreads_pass_plus2r9.fq  |  gzip -1 > reads.paf.gz
miniasm -f s288c_ontreads_pass_plus2r9.fq  reads.paf.gz > reads.gfa
awk '$1 ~/S/ {print ">"$2"\n"$3}' reads.gfa > layout_miniasm.fasta
# align raw reads to raw contigs:
minimap layout_miniasm.fasta $reads > mappings.paf
# run racon:
$raconhome/bin/racon  s288c_ontreads_pass_plus2r9.fq   mappings.paf  layout_miniasm.fasta consensus.fasta
The output is in the attached file.
output.txt
I downloaded and installed racon like this:
git clone https://github.com/isovic/racon.git  && cd racon && make modules && make tools && make -j
I also tried to run racon on  the same data using as template the script example2-paf-pipe-lambda.sh, this created a consensus file, but crashed with the same error:
...
...
[12:21:48 ConsensusDirectFromOverlaps] 	[25] utg000026l 28 alignments, contig len: 12064
[12:21:48 ConsensusDirectFromOverlaps] 	[26] utg000027l 67 alignments, contig len: 35001

[12:21:48 ConsensusDirectFromOverlaps] Started processing contig 1 / 27 (3.70%): utg000001l
[12:21:48 ConsensusDirectFromOverlaps] (thread_id = 0) Aligning overlaps for contig 1 / 27 (3.70%): utg000001l
[12:21:58 AlignOverlaps] Aligning overlap: 2547 / 2548 (100.00%), skipped 0 / 2548 (0.00%)
./example2-paf-pipe-lambda.sh: line 39: 37897 Done                    $raconhome/tools/minimap/minimap $contigs $reads
    37898 Illegal instruction     | $raconhome/bin/racon -M 5 -X -4 -G -8 -E -6 --bq 10 -t ${threads} ${reads} - ${contigs} ${consensus}
Racon exited.

Hi,
hmm this is a bit strange.
Are you perhaps running it on a 32-bit architecture?
Best regards,
Ivan.

Hi,
no, using 64bit.
Just checked more closely the consensus from when using the  example2-paf-pipe-lambda.sh as template script, and it is identical to the initial contigs from miniasm.
Thanks,
Francesca

Hi Francesca,
Would it be possible for you to tell me which exact model of the CPU you are using?
Also, are you running a VM or bare metal?
And, have you by any chance compiled racon on another machine and just copied it to the one you're getting this error, or have you built it on the spot?
This info might help resolve the issue,
Best regards,
Ivan.

Hi Ivan,
thanks for the comment, yes you're right I compiled it on a different machine from the one I was trying to use it.
I was compiling on a  AMD Opteron(TM) Processor 6272 and using it on a Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz.
Thanks a lot!
Francesca

Interesting combination - but I don't think we can reproduce your setup here...
Could you by any chance try recompiling on the target system and report back if it works?
Best regards,
Ivan.

Hi,
yes, I did that, and now it works fine.. weird though..
Thank you!
Francesca

That's great to hear, thanks! :-)
I guess the instruction set between the CPUs is a bit different - we never had problems copying the binary from an Intel to another Intel, as long as the OSs were compatible.
Best regards,
Ivan.


Auto exit after 3 CTRL+C in a row

Automatically close the huhnitor if the user presses CTRL+C (and only that) 3 times in a row.

How do you feel about keeping the exit command, just so we don't need to rely on saying spam ctrl + c to close the program?

Yes the exit command stays available, this is just a addon feature.

Another additional way to exit could be q. I'm getting used to use q to exit from other terminal applications because it's annoying to type out exit all the time.


Empty legend of hist which only has edgecolor

Description
The hist legend doesn't work for edgecolor.
Steps to reproduce
import proplot as pplt
import numpy as np

# Sample data
M, N = 300, 3
state = np.random.RandomState(51423)
x = state.normal(size=(M, N)) + state.rand(M)[:, None] * np.arange(N) + 2 * np.arange(N)

# Sample overlayed histograms
fig, ax = pplt.subplots(refwidth=4, refaspect=(3, 2))
ax.hist(x[:, 0], pplt.arange(-3, 8, 0.2), filled=True, edgecolor='indigo9', edgewidth=1, facecolor='none', legend='ul')
ax.hist(x[:, 1], pplt.arange(-3, 8, 0.2), filled=True, edgecolor='gray9', edgewidth=1, facecolor='none', legend='ul')
ax.hist(x[:, 2], pplt.arange(-3, 8, 0.2), filled=True, edgecolor='red9', edgewidth=1, facecolor='none', legend='ul')

Expected behavior: [What you expected to happen]
Showing correct legend.
Actual behavior: [What actually happened]

Proplot version
3.5.1
0.9.5.post202

I think it is because you didn't specify the label keyword for pplt.hist.
Maybe you should try something with pplt.hist(..., label='whatever', ...).

@syrte Ha, you're right!


As a user I want  the wallet designed to be similar to the rest of the 3bot apps

What
Currently all the apps in the 3bot connect app have a different look and feel.
As a part of this improvement we can also improve the usability of the wallet.
How
Main changes are:

No rounding
More breathing space
Consistent CTA-location
See mockup: https://www.figma.com/file/DDDVntYmF38Q20RIBKhG38/Wallet?node-id=0%3A1

Fixed with 2.2 redesign


Correct type for std::array size() result

The std::array API uses std::size_t for the size() nelts count.
This is a larger sized int than 'int', which causes this code
to break on Windows compilation.
Use the stdcxx type for conformance and portability.
Signed-off-by: Sunjay Bhatia<EMAIL_ADDRESS>Signed-off-by: William A Rowe Jr<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

This generally seems like the right thing to do when considering array sizes etc but is this the only case where this fails to compile on Windows? There are a lot of examples of for (int i = .. in the code base.

@bryce-anderson This pattern of changes around std::size_t are not restricted to this PR, there are more but we've been trying to do them in small batches to ensure we don't have big changesets. We've also not delved into all the extension code but will do so.
As for the for (int i = .. portion that was changed, that was required here b/c of the comparison between an int and std::size_t, signed and unsigned types. If we need to continue to change those as they come up we can do so

While I'm not a core maintainer so my opinion is mostly for funzies, I generally approve of cleaning up size types to be more consistent with container size types so I'm 👍 on this.

@bryce-anderson This pattern of changes around std::size_t are not restricted to this PR, there are more but we've been trying to do them in small batches to ensure we don't have big changesets. We've also not delved into all the extension code but will do so.

Sounds good. I assumed so, but was curious.

As for the for (int i = .. portion that was changed, that was required here b/c of the comparison between an int and std::size_t, signed and unsigned types. If we need to continue to change those as they come up we can do so

Sorry, I wasn't clear. I understand why you made the change in this commit and was just noting that this should probably be done in most places where there is a for (int i = .., which sounds like part of your plan since they may not compile on windows. 😄

I [...] was just noting that this should probably be done in most places where there is a for (int i = .., which sounds like part of your plan since they may not compile on windows. 😄

In many cases this is simply not necessary. For example, int can often be the index without ambiguity, it is the size of size_t or smaller, and depending on the comparison the signedness isn't an issue.
We are resolving this in a number of other files, but they don't need to be corrected en-mass.

We are resolving this in a number of other files, but they don't need to be corrected en-mass.

And sometimes the best solution will be an auto. But not in this case, where it is the declarations of args or retval types that are at issue. The alternate patch to this file is to downcast to int, which introduces it's own set of issues.

After looking at your observations, it is pretty clear the main source/ was correct. Our test/ was supposed to emulate main(argc, argv), so I think the current PR is the more correct solution.


Refactor main window code

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
Refactor main window code.
Now the main window is managed by src/main/mainWindow.js.
Issue link
N/A
Test Cases
All cases.
Additional Notes
https://circleci.com/gh/yuya-oc/desktop/195#artifacts

@yuya-oc can you help clarify which areas should be tested? Would the following be sufficient?

help menu bar
Settings page (both server management and app options)

Sorry, the description was not good. Need to test the following.

Close/Minimize the window
Start on Login
Tray icon menu

Works on Windows 👍

@jnugh Would you be able to help with Linux testing when you get the chance? I'll see if I can find someone on our team to test Mac, which only involves closing/minimizing the app.

Tested the following:

Settings work (can change options, add remove servers)
Can switch Servers using the Server Tabs
Can post to each server (Mattermost renders)
Can add Servers using the + button

Anything else?
Looks good to me :+1:

Thanks @jnugh! Much appreciated!
Can you also try that these actions work:

Close/Minimize the window
Start on Login
Tray icon menu

All those (+ minimize to tray on close) work flawlessly :+1:!

This one is good to merge as well @yuya-oc

Forgot to post testing results for Mac: no issues found

Minimized, sent DM from another user, saw number badge in dock as expected, and clicking dock icon restored app to desktop
Closed window with red X (with existing unreads), saw unread badge stay visible in dock as expected, and clicking dock icon restored app to desktop
Right-click dock icon options all work as expected:

Options > Keep in Dock on: Icon stays in dock after quitting app, Keep in Dock off: Icon disappears from dock after quitting app
Options > Open at Login opens app after restarting machine and logging in
Options > Show in Finder displays the Finder folder where the app lives
Quit closes app
Hide hides the app window
Show All Windows (after Hide) displays hidden window, which can be clicked to be restored

Thanks for detailed reports!


Issue #907 - Allow Site tab from FIlePicker to pick documents from a different site

…different site

Q
A

Bug fix?
[ ]

New feature?
[ X ]

New sample?
[ ]

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Addition of a new parameter to receive the URL of the site to be read, as well as new methods to gather data. If the parameter is empty, the component uses the URL of the current site as usual.

Thank you @kadu-jr for these changes.
I have one proposal though: in other pickers (for example - ListPicker) we have a property called webAbsoluteUrl which is basically the same as defaultSiteUrl you've added. I would rename the new property to webAbsoluteUrl as well to be more consistent across components.

Hi @AJIXuMuK,
No problem. I've pushed the new commit to the pull request. Feel free to add any sugestion.

Thank you @kadu-jr!


fix: fix import ts from node_modules

as https://github.com/swc-project/swc-node/pull/727#pullrequestreview-1760882920 describes, when import .ts files from node_moduels, it throws error TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts".
it works well in 1.6.7
example scene: monorepo import .ts file in project,  or server framework wants dynamic load routes in development

@Brooooooklyn please take a look

Yes, would be nice if this change can be merged.

In order to load  typescript file under node_modules, you also need change file register.js, remove the following codes:
    if (options.files && options.files.length) {
        if (PLATFORM === 'win32' &&
            options.files.every((file) => filename !== (0, path_1.resolve)(process.cwd(), file))) {
            return sourcecode;
        }
        if (PLATFORM !== 'win32' && options.files.every((file) => !filename.endsWith(file))) {
            return sourcecode;
        }
    }

These codes will limit only compile files in current project and all ts files under node_modules will be filtered out and emit to node. This is not suitable in this context.

I think this code can be changed into:
const TS_EXTENSIONS = [ts.Extension.Ts, ts.Extension.Tsx, ts.Extension.Mts];
const JS_EXTENSIONS = ["js", "jsx", "cjs", "mjs"];

export const resolve = async (specifier, context, nextResolve) => {
    const isTS = TS_EXTENSIONS.some((ext) => specifier.endsWith(ext));
    const isJS = JS_EXTENSIONS.some((ext) => specifier.endsWith(ext));
    // entrypoint
    if (!context.parentURL) {
        return {
            format: isTS ? 'ts' : undefined,
            url: specifier,
            shortCircuit: true,
        };
    }
    // import/require javascript module
    if (isJS) {
      // - local project non-TS file
      return nextResolve(specifier);
    }
    const { resolvedModule } = ts.resolveModuleName(specifier, fileURLToPath(context.parentURL), tsconfig, host, moduleResolutionCache);
    // import from local project to local project TS file
    if (resolvedModule && TS_EXTENSIONS.includes(resolvedModule.extension)) {
        return {
            format: 'ts',
            url: pathToFileURL(resolvedModule.resolvedFileName).href,
            shortCircuit: true,
        };
    }
    // import from local project to either:
    // - something TS couldn't resolve
    // - external library
    return nextResolve(specifier);
};

This code will deal with soft links of monorepo subprojects under node_modules, in this cache, the specifier has no extension at all.

add another condition to detect js files is unnecessary. these would be coved by final case.

In order to load typescript file under node_modules, you also need change file register.js, remove the following codes:
    if (options.files && options.files.length) {
        if (PLATFORM === 'win32' &&
            options.files.every((file) => filename !== (0, path_1.resolve)(process.cwd(), file))) {
            return sourcecode;
        }
        if (PLATFORM !== 'win32' && options.files.every((file) => !filename.endsWith(file))) {
            return sourcecode;
        }
    }

These codes will limit only compile files in current project and all ts files under node_modules will be filtered out and emit to node. This is not suitable in this context.

I think it works as expected. files should be added into options.files, or leaves the option empty. or please provide reproduction case it it has problem in your situation


Using 2 types for same Header can cause a panic

impl Header for MyFoo {}
impl Header for YourFoo {}

let mut headers = Headers::new();
headers.set(MyFoo);
headers.get(YourFoo); // panic

#[macro_use] extern crate hyper;

use hyper::header::{Headers, From};

fn main() {
    header! { (MyFoo, "Foo") => [String]  }
    header! { (YourFoo, "Foo") => [String] }

    let mut headers = Headers::new();
    headers.set(MyFoo("bar".to_owned()));
    headers.get::<YourFoo>(); //panic
}

When headers.set() is called and a new Item is created with Item::new_typed(), Item raw is only initialized to None.
Then when Item::typed() is called in headers.get() and there isn't a matching tid, there is a expect  on the type's item's raw, but as its None it panics.
And if you intialize it with Item::raw(), everything seems to work fine.
But I'm not confident in my solution, so would it be better to have it in Item::new_typed()?
pub fn new_typed(ty: Box<Header + Send + Sync>) -> Item {
        ...
        let item = Item {
            raw: OptCell::new(None),
            typed: map,
        };
        item.raw();
        item
    }

Or is it better to use raw() in this fashion, in Item::typed()?
    pub fn typed<H: Header + Any>(&self) -> Option<&H> {
        ...
                match parse::<H>(self.raw()) {
                // replacing: self.raw.as_ref().expect("item.raw must exist")
                    Ok(typed) => {
                        unsafe { self.typed.insert(tid, typed); }
                        self.typed.get(tid)
                    },
         ...
    }

For now, the new version of hyper won't be including the typed headers directly, so I'm closing this here as a won't fix.


stable_set.get_operator()

Information

Qiskit Aqua version:  '0.8.2'
Python version: Python 3.8.3
Operating system: Mac OS X (Identical Error also on Windows)

What is the current behavior?
I believe the stable_set.get_operator() function is incorrectly interpretting the  adjacency matrices handed to it. For the simplest case -  a path graph of 3 nodes - an incorrect qubitOp is created.
Steps to reproduce the problem
# A library that uses the qiskit own interpretation of the stable set probem to do check answers

import numpy as np

from qiskit import Aer
from qiskit.optimization.applications.ising import stable_set
from qiskit.aqua.algorithms import VQE, NumPyMinimumEigensolver, QAOA
from qiskit.aqua import aqua_globals
from qiskit.aqua import QuantumInstance
from qiskit.optimization.applications.ising.common import sample_most_likely
from qiskit.optimization.algorithms import MinimumEigenOptimizer
from qiskit.optimization.problems import QuadraticProgram
from qiskit.optimization.applications.ising.common import parse_gset_format, random_graph, sample_most_likely

import graph_plot as gp
import networkx as nx

path = nx.to_numpy_array(nx.path_graph(3))

print("path\n",path)

qubitOp, offset = stable_set.get_operator(path)
print('Offset:', offset)
print('Ising Hamiltonian:')
print(qubitOp.print_details())

# mapping Ising Hamiltonian to Quadratic Program
qp = QuadraticProgram()
qp.from_ising(qubitOp, offset)
qp.to_docplex().prettyprint()

(EDIT: Code Edited to show full import paths)
gives

which is not the correct function as the weighting of the different nodes are not equal.  The weights of the quadractic term do not have any particular bearing on the functional outcome, as long as it is greater than 1 ( this is the unweighted problem). The simplest solution is just to set all quadractic weights to 2.  [1].
What is the expected behavior?
A correct  docplex function would instead be:

Suggested solutions
I'm afraid the code is impenetrable to me.
[1]

which comes from this paper https://arxiv.org/pdf/1004.2226.pdf.

@a-matsuo I made a comment above on the problem. If this is something that we could/should fix in Aqua for its last deprecated release we can do that too. I wanted to ensure though we fixed any issue here that might have been brought over from Aqua and might still exist - though of course the code is rather different now in creating a QP.

Apologies, I didn't realise that aqua and optimisation were seperate modules. To be clear, I've edited my code to show where I've imported my various functions from. The class you linked too seems different so I  will have a look with that !

Forgive me if this is the wrong place to ask, but is there a difference between qiskit.optimization and qiskit_optimization? When I try using pip install qiskit-optimization I get the requirements already satisfied message, but then why the difference formatting : " _ " ?

qiskit.optimization is a package inside qiskit-aqua. qiskit_optimization is a package inside the qiskit-optimization library. They are contained in different pip installable libraries (qiskit-aqua & qiskit-optimization). qiskit-aqua is being deprecated. The new code is being added to qiskit-optimization.

I recalculated and figure it out. The code forgot to divide some coefficients by 2. It should be as the following code.That's why the converted Ising Hamiltonian doesn't make sense (The converted Ising Hamiltonian is wrong. from_ising is working correctly and the new stable_set.to_quadratic_program() is working correctly as well). Shall I fix it in Aqua?
    num_nodes = len(w)
    pauli_list = []
    shift = 0
    for i in range(num_nodes):
        for j in range(i + 1, num_nodes):
            if w[i, j] != 0:
                x_p = np.zeros(num_nodes, dtype=bool)
                z_p = np.zeros(num_nodes, dtype=bool)
                z_p[i] = True
                z_p[j] = True
                pauli_list.append([1/2, Pauli(z_p, x_p)])
                shift += 1/2
    for i in range(num_nodes):
        degree = np.sum(w[i, :])
        x_p = np.zeros(num_nodes, dtype=bool)
        z_p = np.zeros(num_nodes, dtype=bool)
        z_p[i] = True
        pauli_list.append([1 / 2 - degree/2, Pauli(z_p, x_p)])

@a-matsuo Thanks for checking this out. Since this is really just a bug in Aqua I will move the issue back there.  It would be nice that it were fixed for the next/final release even though the code will all be deprecated. Since you figured out what needs doing it should be quick to do right.


Show Sveltia CMS Version Number in Editor

Please display the current version of the editor in the UI. Maybe directly linked to this repository.
Maybe here as example:

Most people don’t care about the version number, so it doesn’t need to be prominently displayed in the UI. I’ll add the version to the Release Notes label when the Developer Mode is enabled.


Backend/create-dotenv-file

dotenv file created, added to package.json and kept in gitignore. Also user name and password were altered after a secret leak. All updated in .env

It looks good to me.


Game crashes while opening the profile page

2024-02-01T17:17:38-0800
Whoopsies! An unhandled exception has occured.
It appears that the crash occurred while executing code from the "b1rtek.gddlintegration" mod. Please submit this crash report to its developer (B1rtek) for assistance.

== Geode Information ==
Loader Version: v2.0.0-beta.12
Loader Commit: 9cb3cf1
Bindings Commit: a684cc0
Installed mods: 2
Problems: 0

== Exception Information ==
Faulty Module: C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\geode\unzipped\b1rtek.gddlintegration\b1rtek.gddlintegration.dll
Faulty Mod: b1rtek.gddlintegration
Exception Code: c0000005 (EXCEPTION_ACCESS_VIOLATION)
Exception Flags: 0
Exception Address: 5E4145EA (b1rtek.gddlintegration.dll + 245ea)
Number Parameters: 2

== Stack Trace ==
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\geode\unzipped\b1rtek.gddlintegration\b1rtek.gddlintegration.dll + 245ea
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\geode\unzipped\b1rtek.gddlintegration\b1rtek.gddlintegration.dll + 2300e
 - 011A3817
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\GeometryDash.exe + 106a7d
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\GeometryDash.exe + f42f7
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\GeometryDash.exe + f3e4f
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\libExtensions.dll + 169a4 (cocos2d::extension::CCHttpClient::dispatchResponseCallbacks + 148)
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\libcocos2d.dll + 107031 (cocos2d::CCTimer::update + 97)
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\libcocos2d.dll + 106ec4 (cocos2d::CCScheduler::update + 276)
 - 011A2536
 - 011A2357
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\libcocos2d.dll + 103223 (cocos2d::CCDirector::levelForSceneInStack + 131)
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\GeometryDash.exe + 3c12d8
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\Geode.dll + c6596 (gdMainHook + 406 | D:\a\geode\geode\loader\src\platform\windows\main.cpp:58)
 - C:\Program Files (x86)\Steam\steamapps\common\Geometry Dash\GeometryDash.exe + 3c267e
 - C:\Windows\System32\KERNEL32.DLL + 1fcc9 (BaseThreadInitThunk + 25)
 - C:\Windows\SYSTEM32\ntdll.dll + 67c6e (RtlGetAppContainerNamedObjectPath + 286)
 - C:\Windows\SYSTEM32\ntdll.dll + 67c3e (RtlGetAppContainerNamedObjectPath + 238)

== Register States ==
EAX: 17d731a0
EBX: 180e8850
ECX: 00000000
EDX: 7475622d
EBP: 0115fb50
ESP: 0115fad8
EDI: 00000000
ESI: 1775dc48
EIP: 5e4145ea

== Installed Mods ==
x | [v1.0.3] b1rtek.gddlintegration
x | [v2.0.0-beta.12] geode.loader

Solution -> add Node IDs as dependency


USB connection lost

First of all thanks for providing this package!
I have a USB connection problem and would like to know if you can help me.
I am using the amfitrack on Linux:
$ python --version
Python 3.12.3
$ pip list
Package            Version
------------------ --------
amfiprot           0.1.11
amfiprot_amfitrack 0.0.1a14
cobs               1.2.1
crcmod             1.7
libusb-package     <IP_ADDRESS>
pip                24.0
pyserial           3.5
pyusb              1.2.1

Here is what I get when I try to run python ./amfiprot/examples/read_config.py :
USB device: Amfitrack Hub<PHONE_NUMBER>31)
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Traceback (most recent call last):
  File "/home/demo/git/./amfiprot/examples/read_config.py", line 23, in <module>
    cfg = dev.config.read_all()
          ^^^^^^^^^^^^^^^^^^^^^
  File "/home/demo/venv/lib/python3.12/site-packages/amfiprot/configurator.py", line 13, in read_all
    for cat_index in range(self._get_category_count()):
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/demo/venv/lib/python3.12/site-packages/amfiprot/configurator.py", line 86, in _get_category_count
    packet = self.device._await_packet(ReplyCategoryCountPayload)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/demo/venv/lib/python3.12/site-packages/amfiprot/device.py", line 120, in _await_packet
    raise TimeoutError("Packet not returned.")
TimeoutError: Packet not returned.
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!

Few months ago, everything worked very well.
Any idea of how I can deal with this problem?

I try using previous version of amfitrack and managed to get it working again using 0.1.8
(pip install --force-reinstall -v "amfiprot==0.1.8").
All version > 0.1.8 don't seem to work.
I will use version 0.1.8 as a workaround from now on.

$ python --version
Python 3.12.6

$ pip list
Package                 Version
----------------------- -----------
absl-py                 2.1.0
AHRS                    0.3.1
amfiprot                0.1.8
amfiprot_amfitrack      0.0.1a14
annotated-types         0.7.0
anyio                   4.6.2.post1
appnope                 0.1.4

Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.

I am reporting the exact same issue as above, even after downgrading to 0.1.8.
please advise, thank you!

$ python --version
Python 3.12.6

$ pip list
Package                 Version
----------------------- -----------
absl-py                 2.1.0
AHRS                    0.3.1
amfiprot                0.1.8
amfiprot_amfitrack      0.0.1a14
annotated-types         0.7.0
anyio                   4.6.2.post1
appnope                 0.1.4

Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.
Write Task Reconnecting...
Write Task Connection re-established!
Write Task USB connection lost.

I am reporting the exact same issue as above, even after downgrading to 0.1.8.
please advise, thank you!

If you see the same exact output after downgrading to 0.1.8, then the downgrade has not gone right, because the debug text has been added in a later version.
We are currently working on a fix for running version >0.1.8 on Linux and macOS, and expect to have a new version ready early next week.

Amfiprot version 0.1.12 is now live. This new version should fix this issue on Linux and macOS.


vim-tmux-navigator stops working randomly

I am using neovim 0.3.2-953 on Ubuntu 16.04 LTS with tmux 2.7. I have the plugin installed for both tmux and vim and for most part it works fine, however on random occasions i get stuck inside vim(even if there are no vim splits) and and the navigation to different tmux panes does not work until i exit out of vim.
Please let me know if more information is needed from my side. I have not seen any pattern when this behavior is exhibited which is why I am sorry for not being able to provide any so troubleshooting information.

So :TmuxNavigatorProcessList shows just zsh and nvim. Navigation is stuck between vim and tmux panes means i can move fine b/w vim splits however i am not able to move out of vim to tmux panes. Mappings for ctrl-[hjkl] have not been changed, just checked in the vim.

NVIM v0.3.5-7-g947069ba1
tmux 2.6
I get the same as above but interestingly it's consistent in that everything's fine initially and the connection only breaks when I source my vim config from within vim. I'm guessing somehow the reloading of this module causes it somehow. The process list output is the same whether it's working or not.

I have the same problem. When creating a new split with prefix % or prefix ", I can easily navigate between other splits until I focus the vim split and I'm stuck again until I create a new split to get out.
:TmuxNavigatorProcessList output is
S bash
S nvim

:TmuxNavigatorProcessList i get the same output as you get except my shell is zsh . In my case i have to exit out of nvim by :q and then only i can move into other tmux splits. I am not sure if this issue is specific to nvim when used in conjuncture with tmux or does it happen with vanilla vim as well, this is something i am going to test out soon until @christoomey comment on it.

I have the same issue with vanilla vim. I haven't been able to pinpoint what causes the issue, but it's random and happens to me a few times per day. My process list output is:
Ss   -zsh
S+   /opt/homebrew/Cellar/macvim/9.0.472_1/MacVim.app/Contents/MacOS/Vim
S+   tmux

This is a really infuriating issue and it's taken me this long just to stumble upon this thread 😅 I can click on other tmux panes in the same window and navigate to vim from any other pane, but once I'm in the vim pane there is no navigating either across vim panes or outside of the tmux pane housing vim.

@brianbolnick Does this plugin work for you when the process list looks like what you shared, or is that an example from when it's not working. I'd expect it to work with that process list.

but it's random and happens to me a few times per day

This makes me think some other plugin is shelling out in a way that changes your process list. If you can capture the process list when it's working and when it breaks, that would likely help narrow down.

For me adding the changes in the issue https://github.com/LunarVim/LunarVim/issues/2745#issuecomment-1165905419 helped out. Changing the regex in my tmux conf solved the issue somehow.

Try running :TmuxNavigateDown or any of the other navigation commands manually within vim to see whether the functionality is completely broken or if the default key binds are being overridden somehow. I had the same issue and manually re-mapping the default keybindings fixed for my config.

For me this happens sometimes in an old vim window and this is the error messages that show up on the bottom of my screen.
line    4:
E282: Cannot read from "/var/folders/3s/ljg21x_s17xbzjd_j514j7240000gp/T/v6BCizH/1158"

If I manually run the :TmuxNavigateDown then I get the same error. It's like a temp file or folder is being referenced but has been deleted.


Implementation of ReportFormattingGuidelinesComponent class that creates a part of a report

Class is to implement in guide.en.py module that creates a part (reporting component - formally child of ReportComponent class) of guide or report for `reporting module.

from dynpy.utilities.components.guides.en import ReportFormattingGuidelinesComponent

ReportFormattingGuidelinesComponent(None);


CI check to check the results of CI tests

Is your feature request related to a problem? Please describe.
We would like to ensure that the quality of the open source component is guaranteed by the used CI tests.
Describe the solution you'd like
The CI check should check if actual tests are running in the CI and should check the results of the tests.
Describe alternatives you've considered
None
Additional context
There are two steps of this approach
1. Check if what is running is a test
For this we could check the name of workflows in a repo and identify if there is a workflow for testing (best idea to do this so far was to use a set of keywords). Maybe an alternative could be just to extend the keywords already checked.
2. Check the results of the tests
Verify that the execution of CheckRuns was successful for the previous N commits.

We are happy to create a pr if the idea works for the community.

Can you describe what the short comings are of the current CI-Tests check are? Is your concern repos that use GitHub actions to label issues but don't have any tests?

check if actual tests are running in the CI and should check the results of the tests.

Step 1 is the big one here, I think Step 2 would fall into place easily after that.
Different repos structure and name their tests in all sorts of ways. With or without Makefiles (or similar), in workflows, in scripts, in re-usable workflows, etc.


About the source code

Will the source code be announced in the near future?

Hi, thanks for your interest.
The code will be available after the Christmas break, i.e., soon in the new year. Let me know if that's a problem.

I put the code in this repo. Let me know about any other issues.


Feature/optimise ranges when filtering large number

Closes: #128

Codecov Report

Merging #133 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #133   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          336       367   +31
=========================================
+ Hits           336       367   +31

Impacted Files
Coverage Δ

django_property_filter/utils.py
100.00% <100.00%> (ø)

django_property_filter/filters.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Game Over ( SpaceInvaders - Python ) #36

Issue #36
Bullet firing after game over issue is fixed
Once the game is over, the bullet_state is changed to "not_ready" The bullet_state should be equal to "ready" to be able to fire
@mekken2 @Subham-Panda @ashikka @nimishjn @3xpl0itk1t

This issue was already solved by @aryan16x


Draft: dynamic path/tag variables

Description
This is POC/Draft for discussion only.
Implements https://github.com/kluctl/kluctl/issues/345
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update
[x] This change requires a new example

@codablock pls review/comment so I can finish it, thx

This will be important for you as well: https://github.com/kluctl/kluctl/pull/416
It moves to using encoding/json and json tags instead of yaml tags. It then uses conversion between json/yaml to achieve yaml support. It's basically what k8s is doing since forever and I decided to go the same route from now on.
For you this means that your structs need to be updated with json tags.

remved "draft" prefix, but still want to add some documentation & example.
I kept two topics open for @codablock for another round of discussion.

Also, a rebase is required here

Sure, will check your updates.One think I would like to add it is to have DI.name for even git reference - which either will fix missing include tag for it. item.Path == "." i was actually about tu place it separate. "di" Is nice and for full name we could have an alias. PMi22. 5. 2023 v 16:59, Alexander Block @.***>:
@codablock requested changes on this pull request.
I've create a few commits that I think make sense here:
d8049b0
1bd9651
0cc97f2
The first one already renames the "d" variable to "di", but I'm still unsure if this is the best naming. I'm also considering the fully written version "deploymentItem"...
In pkg/deployment/deployment_project.go:

  if item.Path != nil {

  	// Validate or fix item.Path, as k8s labels has constrains

  	// regex used for validation is '(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])?')

  	if *item.Path == "." {

Can you put this into a separate PR so that it properly appears in the changelog later?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Also, a rebase is required here

removed,  rebased

Latest commit fix di.relPath for latest kluctl rebase master and fix behavior on latest kluctl.
If the kluctl is in subfolder of the git repo, then the DeployItem relative dir (original variable) is path within the git repo - this was recently introduced as a fix on main branch.
which somehow with code:  for deployments.yml in PATH/IN/REPO folder:
deployments:
  - path: ./compose
    onlyRender: true

ends up to upload "compose" to  ./build/0/PATH/IN/REPO/compose vs. original behaviour ./build/0/compose (or today if the kluctl is not executed from "git repo".
So I had to somehow get the relative only to the project dir - which ended up as new definition for DeploymentProjectVars.TopPath variable.
@codablock I am now not any way dependant on this PR. But still seams to me nice feature. You want to make some changes as (vr renames/better path handling).


Test and stability fixes

This combines the long-standing tweaks from the #31 Remove Keys branch with the Modal dialog behavior fixes.  Speak now or forever hold your peace!

I found a bug in HorizontalExpandForm() which I think is a merge conflict -- I'll commit the fix tonight.  Tweaking a little bit of stuff in the examples just for polish, and i'll add a LayoutDialog example as well


Pre-Spike:  Early Assessment to discover which gems to focus on

This issue is to help capture the pre-spike investigation for new instrumentation.
The goal of pre-spike is for each developer to answer:

Which gem(s) make the most sense to instrument?
Without a deep dive into the code, does this make sense at a high level for our agent to instrument?
Why is it valuable for our customers?
Could the data it collects fit into our UI? Where would the new instrumentation appear?
Is there an existing feature that could be used for an end-to-end test of the instrumentation?
Ex: We know DB drivers will show up in the UI for example.
Is instrumenting this gem accomplishable within the two sprints (4 weeks) that we have available?

Ref Internal Doc
Please add internal links to this ticket when done.
Acceptance criteria: Brief reports or docs that capture the answers to the questions above.

As we move into the instrumentation spike, the devs are focusing on the following questions for their research.
Please add other questions we should consider as comments.

What methods do we want to instrument?
What metric names do we want to use?
What events should be generated?
Are there other things, like samplers, that would be good to add?
Where could conflicts with our existing instrumentation and structure come up?
Are there any rules that should be considered to determine when this piece of instrumentation should load?
How far back do we want to support a given gem? What version should we start with?

Do we need to limit versions (ex - super old ones) for our instrumentation? What do most customers have downloaded?


Content in TabBarIOS.Item overlapped by TabBar?

I have combined the AwesomeProject tutorial example with the movie list with a TabBarIOS and I'm now rendering the movie list in one of the tabs. When I scroll down all the way in the ListView, I can see that the TabBar is overlapping the last item in the list.
Shouldn't the children of the TabBarIOS.Item somehow only extend to the available device-height minus the TabBar?
What is the best-practice approach to shrink down the actual view-content so that it is not overlapped by the TabBar?

This is intended behavior on iOS 7+. You should set the bottom padding / margin / content inset yourself depending on how you want your app to look.

Alright, thank you very much for the fast reply!

Sorry for resurrecting this topic, but how do you guys handle the content padding/margin/content inset when the keyboard appears and you don't want any kind of space at the bottom, since the tabbar doesn't appear along with the keyboard..


Bug: Create bookmark functionality on the cdd (res 1024 +)

Test Case
Test Case: TC698
Steps to Reproduce
Issue 1: Bookmark button freezes system

Navigate to the cdd
Select Emissions data type
Apply required filter criteria
Preview dataset
Select change
Select a new data type
Select bookmark button

The system freezes.
Issue 2: Pop up Data type formatting

Navigate to the cdd
Apply required filter criteria
Preview your dataset
Select the bookmark button '

When the system displays the data type in the pop up it is in all caps. Formatting is not consistent with the wireframes

Issue 3 : Time formatting should be consistent with the wireframes

Actual Results
Requirements
User Story
As a custom data download user I want to bookmark my results so that I can share data and access my query faster
Acceptance Criteria
Given that a user has accessed the custom data download on 1024 + res
When the functionality is displayed
Then the bookmark functionality is displayed in accordance with the  Wireframes
Given that a user has accessed the custom data download
When the functionality is displayed
Then the bookmark functionality is disabled until a user previews their dataset
Given that a user has previewed their dataset
When the data table is removed (ex. add/remove filter criteria)
Then the bookmark functionality is disabled until a user previews their dataset again
Given that the bookmark tool tip is activated
When the functionality is displayed
Then the tool tip displays the following message:   Bookmark button will be disabled until query is previewed.
Given that the bookmark functionality is enabled
When the bookmark functionality is selected
Then the system displays a pop up in accordance with the  Wireframes
AND the bookmark link is copied to the user's clipboard
Development Notes

The scope of this ticket is for resolutions 1024 and above

Book Mark Requirements
Book Mark Wireframes
Book Mark interactive prototype
Bookmark functionality is disabled until at least one filter is applied
Book mark will have a tool tip
Once the book mark is selected the system will display copy the link to the users clip board and display a pop up
Pop Up
After the user has selected the book mark functionality then the system will display a pop up. The pop up will have the following:
will display the applied data type and sub type
contain the link to the query/ data table
date and time that the bookmark was created
language that the book mark was created/ copied to the clip board

users can close the pop up using the x or ok

Open Defect in TestQuality

Test Create bookmark functionality on the cdd (res 1024 +) in Run Create bookmark functionality on the cdd (res 1024 +) (linked to Cycle Bookmark functionality on CAMPD).


Custom UA support. Fallback for non-retrievable images.

Added support for:

Defining a custom user agent to be used when retrieving content. Definition in config.json.
Replacing images that couldn't be retrieved (e.g. forbidden) by a placeholder image. Previous behaviour was to abort.

In essence this PR makes sense, but I'm not completely on board with the contents.
A few things before I can merge:

[ ] config.json - the UA can be included directly in the source, and overriding the UA can be a optional argument to the utility. Putting in a config doesn't make sense because the user can't actually change it without making a change directly to the source code (unless I've completely missed something, but I still would rather make this an optional override).
[ ] PR doesn't follow CONTRIBUTING requirements (commit messages and no tests) - needs updating
[ ] I understand assets that can't be accessed might want to fall back to something else, but falling back to a 1K SVG doesn't make sense, as the image should be broken. The PR is correct though, the entire process shouldn't bail - just the image shouldn't load. The work here should be:

[ ] Don't reject on statusCode !== 200
[ ] Either insert a tiny transparent gif: data:image/gif;base64,R0lGODlhAQABAIABAP///wAAACwAAAAAAQABAAACAkQBADs= or replace the src with something like about:blank ... I'm not 100% sure here.

In future, I think it makes sense to be able to send a cookie jar with the inliner request to access some of these blocked items, but there's no priority for that as yet.

Is this PR still active? If not, I'm happy to close. Cheers.

Yes, I will look into it in the next few days.
2015-11-15 11:04 GMT+01:00 Remy Sharp<EMAIL_ADDRESS>
Is this PR still active? If not, I'm happy to close. Cheers.
—
Reply to this email directly or view it on GitHub
https://github.com/remy/inliner/pull/59#issuecomment-156798012.


Add support to provide password for a connection with an external command

Is your feature request related to a problem? Please describe.
When using AWS RDS IAM Authentication to connect to database server in AWS (MySQL, MariaDB, PostgreSQL), a temporary password can be created using the AWS CLI to connect to databases. https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.IAMDBAuth.html
This means much better security as the passwords are shortlived and generated on demand so that access can easily be revoked without having to change the password.
DBeaver can be used together with this only if you create a custom script and run it through the commandline using a script that generates the temporary password and then opens DBeaver passing this information on the commandline. But this is a quite cumbersome to work with.
Describe the solution you'd like
I would like to have a way to call an external program to retrieve the password before each connect is done. The command should have the database hostname, port and username available, for example as environment variables so it can be called similar to this:
aws rds generate-db-auth-token --profile myaccount-profile --hostname %DB_HOSTNAME% --port %DB_PORT% --username %DB_USERNAME%

The stdout of the command could be used as raw input to be used as a password.
If the password is cached in DBeaver, the expiration should be configurable (AWS RDS IAM credential is valid for 15 minutes).
This feature would not only support AWS RDS IAM Authentication, but potentially other solutions that can generate or retrieve temporary passwords.
Describe alternatives you've considered
Current approach with scripts works, but it's an ugly workflow
Additional context
pgAdmin have this feature (password exec command in the advanced properties of a connection, see here: https://www.pgadmin.org/docs/pgadmin4/8.12/server_dialog.html) and they're doing the improvement on adding environment variables I'm suggesting above as well: https://github.com/pgadmin-org/pgadmin4/issues/6794

I see that this is natively supported in the paid version, so I assume you don't want to implement it?

Yes, PRO version of DBeaver support this. We won't implement it in DBeaver Community


Issues/22

What(What have you changed/added/removed?)
Why(Why did you change/add/remove it?)

Codecov Report

Merging #26 (01dd1bb) into main (e4be193) will decrease coverage by 0.19%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
- Coverage   53.55%   53.36%   -0.20%
==========================================
  Files           5        5
  Lines         239      238       -1
==========================================
- Hits          128      127       -1
- Misses         89       90       +1
+ Partials       22       21       -1

Impacted Files
Coverage Δ

modfile.go
37.77% <0.00%> (-2.23%)
:arrow_down:

ote.go
0.00% <ø> (ø)

Continue to review full report at Codecov.

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


[Umbrella] WebUI Support for Kyuubi Server

Code of Conduct

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

Search before asking

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

Describe the proposal
WebUI Support for Kyuubi Server gives us a user-friendly view for a running Kyuubi server or maybe historical datas
Task list

[ ] Basic UI Framework

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Postpone to 1.8, because this feature is not under rapid development, and it's not supposed to be accomplished in a short time.


cancan slowing down page loads

Hi,
I was wondering if there are smarter ways to optimize cancan. I see for example that once the permission has been satisfied, cancan keeps querying for other permission definitions. For example:
  can :manage, :all if user.has_role?:admin
  # this is true
  can :manage, Event if user.has_role? :event_manager

I'm finding that even though my current user has role "admin", it is hitting both queries on every page load, really slowing down my page load time.
Any pointers in regards to cacheing / optimizing / best practices would be most appreciated. Thanks!

@jdkealy An ability is initialized for the current request. To do this it instantiates your ability class. Everything in MyAbility#initialize will run to determine what a user can or cannot do.
Instead of defining:
  can :manage :all if user.has_role? :admin
  #etc

try short circuiting it with
  if user.has_role? :admin
    can :manage, :all
  else
    # can manage other things possibly maybe

Or check out the defining abilities with blocks guide on the wiki to have it run at check time.

You can also eagerload your roles with includes to prevent additional queries.


Rework the way we use Kafka

This PR includes a bunch of changes in the way we use Kafka:

Provision per-application Kafka topics for events automatically – This allows to properly process application data, without the need to filter through all events of all applications. It also introduces a persistent work queue, for which I have mixed feelings.
Directly consume commands from Kafka – This fixes the problem that only one pod of an endpoint would receive a command, assuming you have scaled up the number of pods
Directly consume device registry events from Kafka – This allowed to drop a few more dependencies, as most of the code was already present from the command consumer implementation. It also means that we no longer require KNative for running a core Drogue Cloud instance. We also save some more RAM on not running instances for the KafkaSources. Still Knative is being used on the application side.

Can you elaborate on your "mixed feelings" about the persistent work queue?

It works. It focuses and what is necessary. There are probably some bugs in there, which we can fix. … Still, it feels a bit like implementing a broker.


MDSpinner can't change color

Description of the Bug
I can't change spinner color

OS: Manjaro
Python: 3.8
Kivy: 1.11
KivyMD: 0.104.2

@Landris18 Please, Write good and clear description, paste code sample and, if possible, attach a screenshot.

@Kulothungan16k, okay           MDSpinner:                 id : loader                 size_hint: None, None                 size: dp(20), dp(20)                 pos_hint: {'center_x':0.75, 'center_y':0.29}                 active: True                 palette: 0.38627450980392155, 0.8431372549019608, 0.596078431372549, 1                 color: 0.48627450980392155, 0.8431372549019608, 0.596078431372549, 1
I want to change the color but it doesn't work !


Local variables not narrowed in lambda function bodies

Hello all, love your work with the language. I suspect I may have found a bug:
Local variables that are a union type (such as type | undefined) at declaration aren't being narrowed in lambda functions. Parameters and local aliases of those variables are.
TypeScript Version:  2.2.2 / 2.3.2 / nightly (2.4.0-dev.20170506)
Code
// A *self-contained* demonstration of the problem follows...

// example type
interface Test {
	label:string
}

// Function operates without errors, 'parameter' is correctly
// narrowed to Test rather than Test|undefined
function testParameters(parameter:Test|undefined) {
	if (parameter) {
		console.log(parameter.label);
		Promise.resolve().then(() => parameter.label);
	}
}

// Local variable is not narrowed in the lambda/arrow
function testLocals(parameter:Test|undefined) {
	let local = parameter;
	if (local) {
		console.log(local.label);
		Promise.resolve().then(() => local.label);
		//^ Error "Object is possibly 'undefined'."
	}
}

// Useful but annoying workaround
function workaround1(parameter:Test|undefined) {
	let local = parameter;
	if (local) {
		let localLabel = local.label;
		console.log(local.label);
		Promise.resolve().then(() => localLabel);
	}
}

// Useful but annoying workaround #2
function workaround2(parameter:Test|undefined) {
	let local = parameter;
	if (local) {
		let localPrime = local;
		console.log(local.label);
		Promise.resolve().then(() => localPrime.label);
	}
}

Expected behavior:
I expect union types to be narrowed in lambda functions defined inside of a block with a type guard.
Actual behavior:
Variables that are narrowed in the block a lambda is created are not interpreted as their narrowed type inside the body of the lambda function. The above examples also apply to union types with truthy variables like number | string and their appropriate type guards and to functions written as function(){...}.

Hi! This is intended (#8541), as (in general) there is no way to see whether the callback is invoked immediately and thus in the scope of all the if conditions that can assumed to be true, or much later, when the if conditions don't have to hold anymore.

I forget to mention: It works if you use const instead of let for local, as any assumptions about const values stay true regardless of the scope.

Thanks @hediet that helps. So if I understand you correctly we could realistically have a situation where the variable is reassigned before the callback executes so the type guard isn't necessarily valid at the time the callback executes. That seems fine, but then why does the compiler narrow function parameters which are also mutable?
If I change my earlier example about parameters to something that does mutate the parameter like this:
function testParameters(parameter:Test|undefined) {
	if (parameter) {
		console.log(parameter.label);
		setTimeout(() => parameter.label, 200); //Error "Object is possibly 'undefined'."
	}
	setTimeout(() => {
		parameter = undefined;
	}, 100)
}

Then the error comes up, which is pretty cool. If I remove the assignment to parameter, there are no errors.
Could the kind of analysis that's being done here be applied to local variables as well?

You don't need to wrap the assignment to parameter in a callback - just writing to it anywhere is sufficient:
function testParameters(parameter:Test|undefined) {
	if (parameter) {
		console.log(parameter.label);
		setTimeout(() => parameter.label, 200); //Error "Object is possibly 'undefined'."
	}
	parameter = undefined;
}

Theoretically, there is no reason why this analysis wouldn't do for variables introduced with let. But why would you declare a local variable with let over const when you don't intend to reassign it?

@hediet thanks for the explanations.
Is there any easy/convenient way to get typescript to stop complaining about this in certain cases? Something that would have 0 effect on the output code? I know I could introduce another const variable to capture, but that would alter the output just to silence what I know is not a valid error (in certain cases).

No, not that I know of. Sadly, there are no type assertions in TypeScript. You always have to call a function for that or directly check for null/undefined.

Is there any easy/convenient way to get typescript to stop complaining about this in certain cases? Something that would have 0 effect on the output code? I

The ! postfix operator (local!.label) is exactly what you want - it removes null and undefined from the type of its operand.

@RyanCavanaugh that is awesome! Thank you!


Implement Game Menu

User Story
As a user I want to be able to navigate through different pages of the app from a menu
Acceptance Criteria

[x] The user must be able to navigate between pages in a well-designed menu
[x] The user must be able to exit the game from the menu

Finished both Accpetance Critieria


[type hierarchy] Make problematic default methods abstract

Our interfaces are currently treated like abstract classes. Default methods implement 'default' functionality that may not be correct for all implementors of that interface.
Example:
interface Traversable {
    // that might not be true for a class that implements Traversable
    default boolean isSequential() {
        return false;
    }
}

The goal is to make all these default methods abstract, for all interfaces of Vavr.

[ ] We start with introspection methods in #1754
[ ] Related: in #1825 we make Either, Option, Try, List, Stream, ... classes (not abstract!) that have a private constructor.
[ ] After that we make all interface methods abstract that are problematic (as described above)

Out of scope. This is not a bug, it seems to be a Java 'feature'. Maybe the behavior already changed with recent JDKs?
Eventually we will utilize a code generator for the collection hierarchy. Then the issue will be automatically solved. Until then, we will live with it.


Documentation for installing custom certificates in sidecar containers

What content needs to be created or modified?

Inform the users how to make the Dapr sidecar trust custom certificates. This can be useful when a self-signed certificate is required to make sidecar to external-world calls, e.g., when using HTTP bindings, using a proxy, etc. The document should explain the steps needed to configure the same for both Windows and Linux containers and some things-to-note (for e.g., Windows container will use Administrator permissions to install the root certs).
Describe the solution you'd like

Add this to a new documentation page.
Where should the new material be placed?

The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos

https://github.com/dapr/dapr/pull/4741
Additional context

Resolved


WSL2 Ubuntu lunar font rendering issue

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version:
Version: 1.80.2
Commit: 2ccd690cbff1569e4a83d7c43d45101f817401dc
Date: 2023-07-27T20:41:52.615Z
Electron: 22.3.14
ElectronBuildId: 22695494
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

OS Version:
keesj@vlsi:~$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 23.04
Release:        23.04
Codename:       lunar

Steps to Reproduce:

Install code  wsl2 inside ubuntu
upgrade to the latest ubuntu (lunar in my case)
Start vs code

As can be seen in the screenshots something strange is happening with the fonts making the software not usable. The rending is not an issue with other graphical packages (e.g. firefox) hence I do not know how to fix it. I also tried with

Can you check if the issue is present with our exploration build  https://github.com/microsoft/vscode/issues/184021#issuecomment-1648219893 based on a newer version of the runtime.

Thanks for confirming, looks an issue in the runtime then.

This bug has been fixed in the latest release of VS Code Insiders!
@keesj, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 35be9bf683eace09796e59d54f1f225bbc3a7866 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

@keesj - Are you able to test this on insiders to see if it was resolved with that update?

I can confirm that the font rending issue is not longer present in the insiders build
Version: 1.82.0-insider
Commit: e7756c8870ee1df7360e6624e220534174039b02
Date: 2023-08-31T05:33:40.029Z
Electron: 25.7.0
ElectronBuildId: 23434598
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2


fix: bot creation on an unexpected location

Description
Fix some bugs found during bot creation.

The location got reset during the process of decoding url. The location is set to '' and sent to server. So the server create the new bot at default location.

The preset name is undefined when save as a bot because the templateId is not passed to that component. The corresponding url is also changed to projectId/templateId/save

Add/Modify some Unit test.

Task Item
#2798
Screenshots

Hi @srinaath, can you help to verify whether this pr breaks https://github.com/microsoft/BotFramework-Composer/pull/2758. And I changed the url in Save as bot operation. Is this OK?

Checking it now @liweitian !

@liweitian  PR looks good. I checked out the branch and it doesn't break the bfcomposer protocol stuff.


This is a dummy issue to test the auto-assign workflow

Is your feature request related to a problem? Please describe.

Describe the solution you would like

Additional context

take

take


ChaCha20 Cipher

A modern stream cipher that offers better security and performance compared to RC4. It's used in modern protocols like TLS.
Add input and output field so that people can experiment

@shrinidhihegde3  Please assign it to me.

Assigned

I've opened a PR already please review @shrinidhihegde3


PDS4: Implement import of PDS4 data products

We will need to convert the import pipeline to import from PDS4 products instead of (or in addition to) PDS3. This includes the ability to do nightly incremental updates.

The first data to import might be Dick French's new ground-based occultations of Uranus.

We will also need to add new PDS4-specific search fields.


Update what-are-bots.md

Added a code sample

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/what-are-bots.md
80
16
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 6b5863e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/what-are-bots.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/bots/what-are-bots.md
80
16
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 7947e40:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/what-are-bots.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 86f0c59:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/what-are-bots.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Iconize plugin not loading, icons not displaying

Description:
I'm experiencing an issue with the Iconize plugin in Obsidian. The icons are not displaying correctly, and I'm receiving an error message that says "Failed to load plugin: Obsidian Plugin".
Steps to reproduce:
Install the Iconize plugin in Obsidian
Enable the plugin
Attempt to use the plugin (e.g. add an icon to a note)

Expected behavior:
The plugin should load correctly and display icons as expected.
Actual behavior:
The plugin fails to load, and icons are not displayed. An error message is shown: "Failed to load plugin: Obsidian Plugin".

I've run into the same problem. Are you using the latest Obsidian release?
I have a hunch it has something to do with this: https://docs.obsidian.md/Plugins/Guides/Understanding+deferred+views

If you are using the latest version, see #566

Agreed, This is probably a duplicate. Disabling the Bookmarks in Core plugin seems to work. I only had two bookmarks in the vault that was causing problems, and the issue went away when I deleted them.


Updates to support reads in data palettes

Also includes some code cleanup and improvements to documentation.  Tests continue to pass, but coverage is pretty low.

Looks good.


Evaluation status flow on Kyma upgrade

Description
Injects pre- and post- logic during Kyma upgrades. Specifically, sets Avs evaluation monitors status to Maintenance on upgrade initialization, and reverts to original eval status on finish.
Related issue(s)

PLease note and prepare for situation when external evaluation does not exist. Which is the case for recently provisioned clusters and trial clusters.

PLease note and prepare for situation when external evaluation does not exist. Which is the case for recently provisioned clusters and trial clusters.

/retest

/retest

/retest

/retest


commit

Summary

Context

@minjoo95 FYI you need to change the remote repository you keep pushing this commits to. It should point to your Git repo and not mmistakes/minimal-mistakes.
https://docs.github.com/en/get-started/getting-started-with-git/managing-remote-repositories

@mmistakes Out of curiosity, how do you know one's pushing to the wrong repo, other than them keeping on submitting wrong PRs?

It came up before in the past and that was the fix. I seem to remember someone noticed it when pushing their commits and not specifically opening a pull request.
The fact that it's so common with forked repos has me thinking there's a guide somewhere that new users to MM are following to get up and running and it's botching something.
That or it's just an unfamiliarity with how git works.

It could also be due to how easy GitHub tries to make submitting pull requests after pushing up a new branch. I don't often work that way but feel like there's a big green open pull request button that shows up on github.com encouraging you.
Which if you're new to git seems like something you need to press.


ADBDEV 1060

There was an error when starting cluster with enabled LOCK_DEBUG macro:
FATAL:  (XX000) predefined guc name: trace_locks contain neither sync_guc_names_array nor unsync_guc_names_array
HINT:  Process 188336 will wait for gp_debug_linger=30 seconds before termination.
	Note that its locks and other resources will not be released until then.
          LOCATION:  gpdb_assign_sync_flag, guc_gp.c:4983
          Traceback 0:  /home/t1mursadykov/gp/6x/bin/postgres(errstart+0x469) [0x557faaa78bf4]
          Traceback 1:  /home/t1mursadykov/gp/6x/bin/postgres(gpdb_assign_sync_flag+0x153) [0x557faaab6e72]
          Traceback 2:  /home/t1mursadykov/gp/6x/bin/postgres(build_guc_variables+0xa18) [0x557faaaa1b5a]
          Traceback 3:  /home/t1mursadykov/gp/6x/bin/postgres(InitializeGUCOptions+0x16) [0x557faaaa203e]
          Traceback 4:  /home/t1mursadykov/gp/6x/bin/postgres(PostmasterMain+0x1c6) [0x557faa84a183]
          Traceback 5:  /home/t1mursadykov/gp/6x/bin/postgres(+0x86bc07) [0x557faa733c07]
          Traceback 6:  /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf3) [0x7f7b65bfb0b3]
          Traceback 7:  /home/t1mursadykov/gp/6x/bin/postgres(_start+0x2e) [0x557faa3a416e]
FATAL:  (XX000) predefined guc name: trace_locks contain neither sync_guc_names_array nor unsync_guc_names_array
HINT:  Process 12014 will wait for gp_debug_linger=30 seconds before termination.
	Note that its locks and other resources will not be released until then.
          LOCATION:  gpdb_assign_sync_flag, guc_gp.c:4983
          Traceback 0:  /home/t1mursadykov/gp/6x/bin/postgres(errstart+0x469) [0x560d5c19ebf4]
          Traceback 1:  /home/t1mursadykov/gp/6x/bin/postgres(gpdb_assign_sync_flag+0x153) [0x560d5c1dce72]
          Traceback 2:  /home/t1mursadykov/gp/6x/bin/postgres(build_guc_variables+0xa18) [0x560d5c1c7b5a]
          Traceback 3:  /home/t1mursadykov/gp/6x/bin/postgres(InitializeGUCOptions+0x16) [0x560d5c1c803e]
          Traceback 4:  /home/t1mursadykov/gp/6x/bin/postgres(PostmasterMain+0x1c6) [0x560d5bf70183]
          Traceback 5:  /home/t1mursadykov/gp/6x/bin/postgres(+0x86bc07) [0x560d5be59c07]
          Traceback 6:  /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf3) [0x7f8b26a6c0b3]
          Traceback 7:  /home/t1mursadykov/gp/6x/bin/postgres(_start+0x2e) [0x560d5baca16e]
...

This error occurs due to missing guc variables. Variables are described  here.
Also added variables which aren't related to this error

wal_debug
log_btree_build_stats (To use this variable the BTREE_BUILD_STATS macro was added)

Are there any other GUCs from guc.c or guc_gp.c that are not present in unsync_guc_name.h/sync_guc_name.h files?

I think the commented out declaration of BTREE_BUILD_STATS macro in pg_config_manual.h could be proposed to PG community.

GUCs from guc.c that are not present in files:

optimize_bounded_sort and DEBUG_BOUNDED_SORT macro which turn on it
trace_syncscan

All variables from gp_guc.c are presented in files.

@t1mursadykov can you check that enabling DEBUG_BOUNDED_SORT/TRACE_SYNCSCAN macros causes the same error as we got with LOCK_DEBUG? If my suspicions are correct we should fix optimize_bounded_sort and trace_syncscan as well.

@darthunix, there are the same errors in logs with DEBUG_BOUNDED_SORT/TRACE_SYNCSCAN macros.
Enabling of DEBUG_BOUNDED_SORT macro causes a compilation error. To deal with this compilation error, we can add a declaration of the variable optimize_bounded_sort to cdbvars.h.
tuplesort_mk.c: In function ‘tuplesort_set_bound_mk’:
tuplesort_mk.c:1008:7: error: ‘optimize_bounded_sort’ undeclared (first use in this function); did you mean ‘optimizer_enable_sort’?
 1008 |  if (!optimize_bounded_sort)
      |       ^~~~~~~~~~~~~~~~~~~~~

@t1mursadykov add any declarations that are required for a working compilation and show the result in a new commit.


chore: update Nodejs authentication file to point to current authentication content

Update links to point to Cloud authentication documentation.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

/gcbrun


"Davidești" is listed in the drop-down menu

Description:
"Davidești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Davidești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Davidești, Argeș (Davidești)".


Could not get unknown property 'android' for project ':app' of type org.gradle.api.Project.

downgraded gradle and now this issue .All I have done was install@latest and react-native link 'project;
Your Environment

Plugin version:
Platform: iOS or Android
OS version:
Device manufacturer and model:
React Native version:
Plugin configuration options:
Link to your project:

Context

Expected Behavior

Actual Behavior

Possible Fix

Steps to Reproduce

Context

Debug logs

Issue occurs with firebase and google not having the same exact version. using versionNumber + was causing a issue.

Anyone have solution for this ?

I'm having the same. I just enabled gradle3 like it made in there https://github.com/mauron85/react-native-mauron85-background-geolocation-skeleton
I'm not experienced with android development (web only). Can anyone help me please? Thanks.

I had to abandon my project. Just because I couldn't solve this issue :/

https://github.com/mauron85/react-native-background-geolocation/issues/217

I downgraded buildToolsVersion and supportLibVersion
ext {
compileSdkVersion = 26
targetSdkVersion = 26
buildToolsVersion = "26.0.2"
supportLibVersion = "25.3.1"
googlePlayServicesVersion = "11.8.0"
//    oreoEXPERIMENTAL = "yes"
gradle3EXPERIMENTAL = "yes"
}
`
so now it builds but appcrashes right on start.
if I comment this string in MainApplication.java

then no crash on start, but geolocation does not work. ((

@j-mendez Why'd you close this? What was the solution?

OK Turns out for me the issue was in the android-common submodule:
I needed to change:
    if (findProject('..:app') != null) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

to:
    if (findProject('..:app') != null && project('..:app').hasProperty('android')) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null && project(':app').hasProperty('android')) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

@tmaly1980 very nice find.
I think the issue is in settings.gradle
I really wonder why can't it see the android property tho, because for my project it is there.

@tmaly1980 sorry, should have posted the solution. Setup your project for AndroidX and it should work.

OK Turns out for me the issue was in the android-common submodule:
I needed to change in VERSIONS.gradle:
    if (findProject('..:app') != null) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

to:
    if (findProject('..:app') != null && project('..:app').hasProperty('android')) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null && project(':app').hasProperty('android')) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

This worked for me on RN 0.62.2 - thank youj.

OK Turns out for me the issue was in the android-common submodule:
I needed to change in VERSIONS.gradle:
    if (findProject('..:app') != null) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

to:
    if (findProject('..:app') != null && project('..:app').hasProperty('android')) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null && project(':app').hasProperty('android')) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

This worked for me on RN 0.62.2 - thank you. I wonder if I could turn it into patch-module..

Why i don't have file VERSION.gradle in my project?

@tmaly1980 sorry, should have posted the solution. Setup your project for AndroidX and it should work.

can you say detail about steps to setup?Please

@caolong0204 update that code in<EMAIL_ADDRESS>

OK Turns out for me the issue was in the android-common submodule:
I needed to change in VERSIONS.gradle:
    if (findProject('..:app') != null) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

to:
    if (findProject('..:app') != null && project('..:app').hasProperty('android')) {
        applicationId = project('..:app').android.defaultConfig.applicationId
    } else if (findProject(':app') != null && project(':app').hasProperty('android')) {
        applicationId = project(':app').android.defaultConfig.applicationId
    }

This worked for me on RN 0.61.2 - thank you
@caolong0204
++

Same question as @jureczkyb, when will this be merged?

where is versions.gradle file?


Bad output when existing dependabot file does not end with a newline

Describe the bug
When there is an existing dependabot file that does not have a new line after the last entry, the new entries start on the existing last line and produce an invalid file
To Reproduce

Expected behavior

Screenshots
No response
Additional context
No response

I just wrote a test to reproduce this scenario. I'll look at fixing this bug which should result in getting the test to pass.

A fix is ready for review: https://github.com/github/evergreen/pull/232

Wow. Lightning fast turnaround. Thank you.
On Thu, Sep 26, 2024, 19:34 Zack Koppert @.***> wrote:

Closed #231 https://github.com/github/evergreen/issues/231 as completed
via #232 https://github.com/github/evergreen/pull/232.
—
Reply to this email directly, view it on GitHub
https://github.com/github/evergreen/issues/231#event-14420998729, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKFFIBLBMS76345IQ2Y2WDZYRHSPAVCNFSM6AAAAABO5EUOE6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGQZDAOJZHA3TEOI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Mouse right click on tree node not working

System information:

Ubuntu Mate 19.10 - 64bits
DBeaver version 6.2.3

If I select any node in any tree view, like database navigator or projects view, and I click on right mouse button, the contextual menu is not appearing and I should always use keyboard to display it.
The right contextual menu is very critical to update changes inside tree view because dbeaver doesn't automatically apply changes to tree nodes after doing any change like creating/deleting  tables.

Duplicate of #6489
Thanks for the bug report anyway


Windows user

Dude, it's not exactly an issue. But yes, a doubt. I'm a bit new to Docker and SQL, can you tell me how I can run the program on Windows?

I'm also curious about this. I'm running it without issue on Manjaro, but I'd like to run it on Win11.  Big noob here. I installed GitHub Desktop and GitBash but I have a hard time running this.

I don't have access to a windows machine at the moment. However I am planning to test this on windows and if possibly provide a pre-build binary with instructions.


ci: replace generated headers by templates

Currently, coverage analysis runs particularly on generated headers. However, since these are not readily available for Coveralls their source cannot be displayed prohibiting their online inspection.
This PR patches the coverage files by replacing the paths of generates headers by their template sources.

@drigz I got in touch with the Coveralls support team because the PR comments still do not appear. They were very helpful at tracing the problem to a disabled setting in the web interface. Specifically, the PR comments must be enabled at https://coveralls.io/github/google/glog/settings under "Pull Requests Alerts":

Could you please check the setting and enable it as necessary?


as and is operator does not work on generic types.

We should extend as and is operator so that one can use it to test if a generic type T is a concrete type.

Fixed in #3672.


Can't import

I use canary 4.1 8, and I can't import and run this sample:

It looks like you've imported the root folder—try importing from the Jetnews folder.

Indeed. Usually projects have the root as the one to import.
Now I tried and it works, but something is weird:
Pressing the back key doesn't go back. It just closes, and even if I press the "up" button, it reloads the main screen. In fact it happens even on orientation change.
See:
device-2020-05-19-191838.zip
I've also noticed that there is a navigation drawer, yet it was hard to notice because there is no "hamburger" icon. The navigation drawer forgets its state upon orientation change, and  when choosing "interest" it has tabs but no viewPager to swipe between them.
I guess it's more about Compose than how to handle apps nicely, but still... Seems you've put a lot of effort on other various things, and the purpose of Compose is mainly about UI.

This repo is intended to be a collection of samples; there's only one, Jetnews, right now but we plan to expand it.
Thanks for highlighting those issues. We are aware and working on providing advice on navigation (integrating with the system back button), configuration changes etc. Compose is still pre-alpha so not all parts are there yet.
Re the hamburger menu, this is the design, will feedback to our designer.

I see. I was actually quite overwhelmed by the amount of things there that I'm not familiar with.
So much that I thought it's available for a long time that people use it.
Are there any apps that use it already?
What is your suggestion about an already existing app? Especially one that doesn't even use navigation component yet (too huge and too little time for that).

Yes, a new UI Toolkit is a large undertaking. We'll be updating our documentation, samples, codelabs etc to help you learn but need to balance this against product stability.
I can't comment on current adoption or future plans but it is a goal for Compose to work well with other Jetpack libraries so adopting those would be a good step.

Would it replace the current development on Android? Eventually, nobody will use XML anymore for layouts (and maybe drawable?) ?
Is it already possible to implement everything using Compose?
Would it mean there won't be any more android-x support libraries, etc?
I actually kinda liked using XML. Got used to it quite quickly...
And I liked the division between it and normal code.

Would it replace the current development on Android? Eventually, nobody will use XML anymore for layouts (and maybe drawable?) ?
Is it already possible to implement everything using Compose?
Would it mean there won't be any more android-x support libraries, etc?
I actually kinda liked using XML. Got used to it quite quickly...
And I liked the division between it and normal code.

I'm not a maintainer for Jetpack Compose, but having been an Android Engineer for quite some time, everything you mentioned about XML layouts is actually seen as a downside.
It's a bit out of the scope of my comment, but do a little research on declarative UI like Compose, React JS, and Swift UI. You'll find that there's a load of great reasons for Android to move in this direction.
As for answering your questions. I do believe we will continue to see people use XML and the old View based UI for some time. If only because legacy codebases will call for it, and Compose is compatible with the old way.
Arguably though, Compose is targeted at removing the need to do XML or View class based UI. So in the not so distant future, new apps would prefer starting with Compose, and never touch the old UI framework.

I see. I should start learning about it soon, then.


docs(iroh-net): Improve pkarr discovery docs

Description
This tries to improve the pkarr discovery docs by trying to describe
enough about how it works that users do not need to go looking
elsewhere.
Breaking Changes
none
Notes & open questions
There are so many code overlaps and naming inconsistencies here.  But this was
part of docs friday so I'm just documenting things for now.

Discovery mechanisms are not named particularly consistent:
PkarrPublisher is probably a PkarrRelayPublisher
PkarrResolver is also a PkarrRelayResolver
DnsDiscovery only does resolving -> DnsResolver
DhtDiscovery can also use relay server, but at least it does both publish and resolve
The DhtDiscovery is pretty generic, it can probably be made to do both replace PkarrResolver and PkarrPublisher, maybe with custom constructors.

Change checklist

[x] Self-review.
[x] Documentation updates following the style guide, if relevant.
[ ] Tests if relevant.
[ ] All breaking changes documented.

@Frando @rklaehn ping

@flub what's missing here to merge this?


More sensors in sensors_plugin

Why are there only gyroscope and accelerometer?
When will other sensors have been implemented?
And could I implement only Android-side sensors?

Closing as sensors is deprecated in favor of sensors_plus.


[REQUEST]: DM BON Assistent documentation

Please provide an updated version of the dmbon assistent documentation
currently documentation is located in 2 files: dmbon.html & dmbon.md (with navigation currently going to dmbon.html) --> could this be located to one file?
+please also update dmbon in _data/software-applications.md

let us just keep this in md, no?
(removing the html file and making sure the md takes its place...)


Capture the YAML Template and design for CEC Emulator

Initial design of the emulator. Thoughts on YAML configuration.

you going to merge this after updating?