Decompiler doesn't use non-default address space for data refs

I have 5 overlayed blocks at the same address. My code loads different data at that address and then uses (jumps or reads) it.
If you have overlayed blocks, Ghidra shows such option to select address space:

For jumps and calls, when you change this option to non-default AS, it changes the decompiler output.
But if you have a reference to not executable code (like at the screenshot), then the decompiler doesn't use that info:

Relaunching analysis doesn't help.

Removing data at the default address space will follow to this:

I am having a similar issue but with a computed call to a function in an overlay. I can update the reference to point to the correct overlay but the label/ref in the decompiled view always points to the original location.

Any Updates?

Any Updates?

Is there any update on this issue? This affects setups with multiple overlays that reference each other. Currently, even though sub-overlays should be able to "see" the main overlay, we can't specify Ghidra to account for these references during analysis, resulting in missed cross-references from sub-overlays to the main overlay.
Ghidra 11 introduced a new feature that allows overlay memory blocks to be moved, which allows for the sub-overlay block to be moved into the main overlay's address space and partially addresses this issue. However, this approach is cumbersome in shared projects, requiring an exclusive checkout to move memory blocks, and supports only one overlay at a time. Additionally, moving an overlay block out of the main overlay’s space removes its cross-references, further limiting the effectiveness of this method. Resolving this issue would be an improvement until a more comprehensive solution for modeling inter-overlay visibility or a more ergonomic way to move memory blocks in shared projects are implemented.

@Mc-muffin, By main overlay, do you mean the base address space, or do you actually have a defined "main" overlay space?
If you are sharing a data block among multiple overlay spaces, then the best thing is to have that shared data area not in an overlay, but in the real underlying default address space.
Also in Ghidra 11.x, you can make multiple memory blocks in one overlay space.  Both the decompiler and reference analysis will create/use references to the blocks that are in the same overlay space.  If there is no matching block then they will drop down to the real address space.
You can also create special data type pointers that refer to a particular named address space including overlays. For example if when calling a particular function that function always uses a particular address space, you can type the parameter as a pointer to that named address space.  Local variables can also use these data type pointers.

Okay, so the program is layout as a main binary that's always loaded which can then load other overlays which can have overlays of their own, the specifics of the program I'm looking at aren't that important and the problem can be simplified to a main binary with 2 overlays, 1 of which can further overlay other binaries on itself, so program layout could be
Main binary
 |- Overlay 1

Or
Main binary
 |- Overlay 2
     |- Overlay 3

Or
Main binary
 |- Overlay 2
     |- Overlay 4

Overlay 3 and 4 share load address and can see the data and functions of Overlay 2 and the main binary, now moving Overlay 3 or Overlay 4's address space into Overlay 2's works but means the need to be analyzed one at a time as I think you can't have an overlayed space on top of another, so Overlay 3 and 4 can't be moved into Overlay 2's address space at the same time.
Now, we can sorta avoid having to move the blocks around (and needing an exclusive checkout) by changing the references in Overlay 3/Overlay 4 to point into Overlay 2's address space, which works for functions but not for data references.

You can also create special data type pointers that refer to a particular named address space including overlays. For example if when calling a particular function that function always uses a particular address space, you can type the parameter as a pointer to that named address space. Local variables can also use these data type pointer

I didn't know about this feature, need to test it out. How can I make these typed pointers with an address space?

As far as moving memory blocks around, I don't think there will be a general solution that doesn't require you to make drastic changes to program in a project without an exclusive checkout. The ramifications of certain changes cause irreconcilable problems when working on a program in a multi-user project, and attempting to merge new work that conflicts with others work.

I see, it was mostly a different idea which could also solve the problem but it makes sense that an exclusive checkout is required to change the memory layout.

It can be a little cumbersome to create.
In the data type manager, first create a pointer to the datatype (can be char, void, structure, etc.)
Then create a typedef on the pointer.  The name will look a little strange, which you can change.
Then edit the Settings... for the typedef pointer and change the AddressSpace setting to your overlay.  The address space could also be a data space or a made up IO/SPI address space.

You would need to use the data type on a variable in the decompiler (param/local) or even to a data pointer in memory or within a structure.
There is still some work to be done on the analysis side to pay attention to the setting, but I believe the decompiler will use it in most cases.


bouncePreviewForDrawerSide doesn't pass completion block

bouncePreviewForDrawerSide passes nil to the completion block handler:
-(void)bouncePreviewForDrawerSide:(MMDrawerSide)drawerSide completion:(void(^)(BOOL finished))completion{
    NSParameterAssert(drawerSide!=MMDrawerSideNone);
    [self bouncePreviewForDrawerSide:drawerSide distance:MMDrawerDefaultBounceDistance completion:nil];
}

Got it, thanks.


Fix plugin docs, ToC and links

Description
Fixes #3927
This fixes missing links and pages (and ToC entries) corresponding to #3916. Also fixes a rendering issue in https://napari.org/plugins/stable/guides.html
Type of change

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

References
Completes the work in #3916
TODO:

[ ] There is a missing file (hello.png) - this should be added to the repo
[ ] The ToC at napari.github.io must be reordered.

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

@melissawm, the npe2 includes (like on the contributions page) are still not rendering, and the toc is still wrong... should this have fixed that?
https://napari.org/plugins/stable/guides.html

It should, I'll investigate, thanks for the heads up!

@melissawm, the npe2 includes (like on the contributions page) are still not rendering, and the toc is still wrong... should this have fixed that?
https://napari.org/plugins/stable/guides.html

This actually loads now, but extremely slowly. Like just a screen at a time. Very odd behavior.

@melissawm, this is fixed now (#3949, #3950, #3951) and the plugin tocs was fixed in https://github.com/napari/napari.github.io/pull/326
see also the note in https://github.com/napari/napari/pull/3951#issuecomment-1014653705 ... which is definitely a fundamental problem in the current copy-docs pattern that will almost always require a two repo PR to update the tOC


Fix: Auto-detect chrome binary path on MacOS

On MacOS systems, the chrome binary path is different. This PR aims to fix that issue.

If we're going to add a feature to auto discover the chrome binary based on the OS, wouldn't it be better to have a separate method to do the discovery instead of a single setChromeBinary? This method is doing 2 things.
Maybe something like this would be better:
    public function __construct(string $chromeBinary = null)
    {
        $this->chromeBinary = $chromeBinary ?? $this->getDefaultChromeBinaryPath();
    }

    private function getDefaultChromeBinaryPath(): void
    {
        if (array_key_exists('CHROME_PATH', $_SERVER)) {
            return $_SERVER['CHROME_PATH'];
        }

        if (PHP_OS_FAMILY === 'Darwin') {
           return '/Applications/Google Chrome.app/Contents/MacOS/Google Chrome';
        }

        return 'chrome';
    }

The method could be modified to work with Windows latter on, or even query the OS to find the correct path.

The method could be modified to work with Windows later on, or even query the OS to find the correct path.

AFAIK, chrome works fine on Windows

Pull in the symfony php 7.2 polyfill, please.

What's the reason for that? If it's just for PHP_OS_FAMILY I'd rather not since for MacOS the result is exactly the same.

Oh yes, I didn't spot that change.

AFAIK, chrome works fine on Windows

It doesn't, at least on my machines. I have to use the full path.

What's the most common path to 64-bit chrome on Windows? We might as well throw that in here too.

What's the most common path to 64-bit chrome on Windows? We might as well throw that in here too.

Apparently C:\Program Files (x86)\Google\Chrome\Application\Chrome.exe. Could you please confirm that, @enricodias ?

@GrahamCampbell it varies a lot. On my current machine it's in Program Files (x86)\Google\Chrome\Application\chrome.exe but the "Program Files" folder itself gets translated depending on the installed language. The full path also varies between different Windows versions.
It's probably better to find the correct path using the Windows registry somehow.

The "Program Files" folder only has alias in the Explorer, but it's always "Program Files" from the CLI. About Windows versions, I don't believe that changes...

@CViniciusSDias On Windows 7 it's in C:\Program Files (x86)\Google\Application\chrome.exe, on Vista I believe it was in C:\Users\UserName\AppDataLocal\Google\Chrome. I also think that I've seen it in LocalAppData before aswell.
Using "Program Files (x86)" worked here in the CLI, but you'll also need to guess the drive letter used. Mine for example wasn't using the default c:/

Ah, I didn't take into consideration the possibility of using something different that C:.
Do you know how to get the path from the registry, @enricodias ?
If not, this will probably need some further investigation and I believe it would be better to have it in a different PR, @GrahamCampbell

Also, I believe we'd need an extension to get values from Windows registry via PHP...

Using the env variable %ProgramFiles(x86)% also worked here in the old cmd, but it doesn't return the correct drive letter.
I'll have to take a look on how to query the registry using php without admin permissions. I'm not 100% sure if it's possible.

start chrome also worked in the cmd, but it doesn't work as a path to start the process.
I found the amphp/windows-registry package that seems to be able to read keys from the registry. I'll test it later.

On Windows 7 it's in C:\Program Files (x86)\Google\Application\chrome.exe

That is not 64-bit chrome?

That is not 64-bit chrome?

Even though the chrome download page says 64bit, the automated installer always used the x86 version for me. The installer itself doesn't show any options during the setup. That's one more thing to guess; even if the user is using a 64bit system, chrome could be x86.

Thanks.


Update assessment-appliance requirements

To include disk sizing

@tjsullivan1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rayne-wiselman  Why is this labeled as backup?

Thanks for the feedback! Since there were conflicts in this PR I made the change in a new PR. The change should show up in the docs later today. Closing this one
#please-close


Documentation Improvements

This PR is a work in progress, needing discussion.
I'd like to offer some help with the documentation, as @dikaiosune has mentioned that there is a lot to be improved. I have started with some low-level clarity and stylistic changes, but I'd like to open the discussion around adding Troubleshooting sections and bring FAQ style information upfront as discussed in #67.

I think that'll do for now. I'll see if I can address a Troubleshooting section in another PR. Ready for review.

Wonderful, thank you!


Allow Conda Environment to use the "base" environment

Conda has a global "base" environment where installed packages go if you install outside of an environment.  When running on a Microsoft-hosted agent, there's often no reason to create a separate environment.
This also makes using Conda Environment more similar to using Use Python Version, which intentionally does not create virtual environments.
Testing

Manually with new definition
Manually with old definition (check for breaking changes)
Added L0 test

Now the packageSpecs and otherOptions inputs are overloaded depending on the value of environmentName, and cleanEnvironment never does anything if environmentName is not set.  I thought about adding a radio button like "Environment: o base o custom", but that didn't feel very fluid to me.  But we can go that route if this seems confusing now.

Does it make sense to hide/disable "Clean environment" if base is specified?

I don't think there's a way to do conditionally-visible fields based on the contents of a text box.  That's why I considered making the radio button.

I thought you could just use "visibleRule": "cleanEnvironment != \"\"", not real sure though.

Unfortunately not :(

What do you think about this as a way to show/hide relevant fields?
Base environment

Custom environment

@DavidStaheli I think the checkbox is good.  One minor thing, the "additional arguments" fields will have to move out of "advanced."  For whatever reason, you having fields in a "group" conditionally visible on fields in another group is disallowed.


Inconsistent Response MIME Type Configuration for ChatGoogleGenerativeAI and ChatVertexAI in LangChain

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain.js documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain.js rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
import { ChatVertexAI, ChatGoogleGenerativeAI } from "@langchain/google-genai";
const vertexAIChat = new ChatVertexAI({
responseMimeType: "application/json" // or "text/plain"
});
const googleGenerativeAIChat = new ChatGoogleGenerativeAI({
responseMimeType: "application/json"
});
// This results in the error:
// Object literal may only specify known properties, and 'responseMimeType' does not exist in type 'GoogleGenerativeAIChatInput'.ts(2353)
Error Message and Stack Trace (if applicable)
No response
Description
I'm trying to use the responseMimeType configuration option with the new ChatVertexAI class, and it works as expected. However, when I try to use the same option with the new ChatGoogleGenerativeAI class, it results in the error:
Object literal may only specify known properties, and 'responseMimeType' does not exist in type 'GoogleGenerativeAIChatInput'.ts(2353)
System Info
yarn info langchain
└─ langchain@npm:0.2.5
├─ Instances: 1
├─ Version: 0.2.5
│
└─ Dependencies
├─ @langchain/core@npm:^0.2.5 → npm:0.2.7
├─ @langchain/openai@npm:~0.1.0 → npm:0.1.3
├─ @langchain/textsplitters@npm:~0.0.0 → npm:0.0.3
├─ binary-extensions@npm:^2.2.0 → npm:2.3.0
├─ js-tiktoken@npm:^1.0.12 → npm:1.0.12
├─ js-yaml@npm:^4.1.0 → npm:4.1.0
├─ jsonpointer@npm:^5.0.1 → npm:5.0.1
├─ langchainhub@npm:~0.0.8 → npm:0.0.11
├─ langsmith@npm:~0.1.30 → npm:0.1.32
├─ ml-distance@npm:^4.0.0 → npm:4.0.1
├─ openapi-types@npm:^12.1.3 → npm:12.1.3
├─ p-retry@npm:4 → npm:4.6.2
├─ uuid@npm:^9.0.0 → npm:9.0.1
├─ yaml@npm:^2.2.1 → npm:2.4.5
├─ zod-to-json-schema@npm:^3.22.3 → npm:3.23.1
└─ zod@npm:^3.22.4 → npm:3.23.8
  "@langchain/community": "^0.2.5",

"@langchain/core": "^0.2.5",
"@langchain/google-genai": "^0.0.21",
"@langchain/google-vertexai": "^0.0.17",
"@langchain/google-vertexai-web": "^0.0.17",

Is there a reason you're trying to use both the google-genai package and the google-vertexai packages?
You should be able to use the google-gauth or google-webauth packages to access both using a single library. (The google-vertexai package is mostly a wrapper that locks it into Vertex.)

Is there a reason you're trying to use both the google-genai package and the google-vertexai packages? You should be able to use the google-gauth or google-webauth packages to access both using a single library. (The google-vertexai package is mostly a wrapper that locks it into Vertex.)

Vertex AI does not allow setting the threshold to 'BLOCK_NONE'. I want to use Google GenAI, but it's not possible to set the responseMimeType on it.

The google-genai and google-webauth packages can be used to access the "GenAI" platform. You just need to have the API Key set. It doesn't use Vertex AI in that case.

The google-genai and google-webauth packages can be used to access the "GenAI" platform. You just need to have the API Key set. It doesn't use Vertex AI in that case.

If I try to use the genai api key with the "new ChatVertexAI"  I get the error [Error: APIs that require a project ID cannot use an API key]

Use the "google-webauth" package and new ChatGoogle. These classes work for both GenAI and Vertex.
See either of the following (same article on different platforms):

https://medium.com/@afirstenberg/langchain-js-and-gemini-getting-started-fa602a45bee4
https://code.iaflw.com/2024/06/langchainjs-and-gemini-getting-started.html

@rossanodr

The google-genai and google-webauth packages can be used to access the "GenAI" platform. You just need to have the API Key set. It doesn't use Vertex AI in that case.

If I try to use the genai api key with the "new ChatVertexAI" I get the error [Error: APIs that require a project ID cannot use an API key]

Use the "google-webauth" package and new ChatGoogle. These classes work for both GenAI and Vertex.
See either of the following (same article on different platforms):
https://medium.com/@afirstenberg/langchain-js-and-gemini-getting-started-fa602a45bee4
https://code.iaflw.com/2024/06/langchainjs-and-gemini-getting-started.html

@rossanodr

The google-genai and google-webauth packages can be used to access the "GenAI" platform. You just need to have the API Key set. It doesn't use Vertex AI in that case.

If I try to use the genai api key with the "new ChatVertexAI" I get the error [Error: APIs that require a project ID cannot use an API key]

Use the "google-webauth" package and new ChatGoogle. These classes work for both GenAI and Vertex.
See either of the following (same article on different platforms):

https://medium.com/@afirstenberg/langchain-js-and-gemini-getting-started-fa602a45bee4
https://code.iaflw.com/2024/06/langchainjs-and-gemini-getting-started.html

Thank you!!! save my life


The node was low on resource: ephemeral-storage

Environment

OS: CentOS 8
Kubernetes/K3s: K3s
AWX Operator: 1.1.4

Description
We are getting multiple failures due to below Issue . POD is getting terminated intermittently.
Following error shows up while we describe POD, Can we get the solution for it. how to assign or increase temporary storage.
Can someone guide on it.
Name:             awx-75cb6698f9-c9swp
Namespace:        awx
Priority:         0
Service Account:  awx
Node:             *************************************
Start Time:       Thu, 23 Mar 2023 17:56:25 +0530
Labels:           app.kubernetes.io/component=awx
app.kubernetes.io/managed-by=awx-operator
app.kubernetes.io/name=awx
app.kubernetes.io/operator-version=1.1.4
app.kubernetes.io/part-of=awx
app.kubernetes.io/version=21.11.0
pod-template-hash=75cb6698f9
Annotations:
Status:           Failed
Reason:           Evicted
Message:          The node was low on resource: ephemeral-storage. Container awx-ee was using 195868Ki, which exceeds its request of 0. Container awx-web was using 21208Ki, which exceeds its request of 0. Container awx-task was using 68236Ki, which exceeds its request of 0. Container redis was using 2432Ki, which exceeds its request of 0.

@Nilesh486
K3s consumes disk space under /var/lib/rancher for containers' file system. Ensure your /var/lib/rancher has enough free space.

Thankyou this helps

Does it possible to use clean up for this consumption.
i found below solution over multiple blogs, is it reliable to use it for our k3s setup to clean up old images, does it free the space for /var/lib/rancher this path. We have custom EE and instance group.
sudo k3s crictl images to see what images have been pulled locally
sudo k3s crictl rmi --prune to delete any images no currently used by a running container

It's safe to run sudo k3s crictl rmi --prune on your k3s, but it depends on your usage that your disk (/var/lib/rancher) will be freed up or not.

Thankyou  for clarification


line tool should be on by default when you enter sketch mode for new sketch

thank you @Irev-Dev !


fix(accordion): fixing font-family styling, fixed height demo

What I did

Updated the font-family for pf-accordion to "Red Hat Text"
Updated the demo page to expand the fixed height demos first panel automatically on load

Related Issues

#2397
#2416

I'm going to close this because really it's out of scope.

If we want to change the font stack, we need to do that upstream
If we want to change the fonts only on our website, we should do that in /docs


PROM heading

New heading is planned:  https://projects.invisionapp.com/share/CSBN9XEYX#/screens/278187375
We suppose to get this kind of fields in responses to API requests:
GET  List
Respons
[
{
name: string,
score: number,
dateCreated: number,
source: string,
sourceId: string
},
...
]
GET  Detail
Respons
{
name: string,
records: [
{
date: number,
name: string,
source: string,
sourceId: string,
type: string,
typeTitle: string,
}
]
score: number,
dateCreated: number,
specific-q1: string,
specific-q2: string,
specific-q3: string,
specific-q4: string,
source: string,
sourceId: string
}
PUT/POST edit/create  Detail
Respons
{
name: string,
records: [
{
date: number,
name: string,
source: string,
sourceId: string,
type: string,
typeTitle: string,
}
]
score: number,
dateCreated: number,
specific-q1: string,
specific-q2: string,
specific-q3: string,
specific-q4: string,
source: string,
sourceId: string,
userId: number
}
Related issue:  https://github.com/PulseTile/PulseTile-React/issues/119

thanks @anna-otrokhova
Hi @freshehr
Could you take a look at this requirement at the UI /API level here and let us know what openEHR template we could use to persist this generic PROMS data please?
thanks
T


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/120 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit 56b097f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

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


Adds constant time (for length) operations.

This adds equals, compare, and isZero operations to Util.java that are comparable to sodium/util.h versions.
Running these functions in Java is much faster than going though to the overhead to the native libsodium implementations.

memcmp -> https://github.com/jedisct1/libsodium/blob/1.0.8/src/libsodium/sodium/utils.c#L98-L117
compare -> https://github.com/jedisct1/libsodium/blob/1.0.8/src/libsodium/sodium/utils.c#L132-L155
is_zero -> https://github.com/jedisct1/libsodium/blob/1.0.8/src/libsodium/sodium/utils.c#L158-L167

@bendoerr sorry it took so long to get back to you. I understand the motivation behind it, but just can't see where these methods are being used. If such implementation is faster, I'd say, let's add test cases for it and replace those native functions.

It's not drastic and really only visible in the hundreds of thousands of operations for for example https://travis-ci.org/abstractj/kalium/jobs/124269713#L1920-L1923 (note the benchmark library uses currentTimeMillies rather than nanoTime). In my earlier benchmarks using a different testing method the difference was more substantial.  I suppose it seemed silly to invoke a FFI call for such simple operations which is why I added the native Java version.
As for where they are being used, I don't have a great use-case for them other than the fact that libsodium exposes them and I'd like to see Kalium expose as much of the libsodium API as possible.

Second commit here is just to run some JMH benchmarks for the constantTimeIsZero.
Benchmark                                           Mode Thr     Count  Sec         Mean   Mean error    Units
o.a.k.c.UtilConstantTimeBench.testFfiNotZero       thrpt   1       200    1     3115.281       40.181   ops/ms
o.a.k.c.UtilConstantTimeBench.testFfiZero          thrpt   1       200    1     2758.366       17.170   ops/ms
o.a.k.c.UtilConstantTimeBench.testNativeNotZero    thrpt   1       200    1    89103.738      784.031   ops/ms
o.a.k.c.UtilConstantTimeBench.testNativeZero       thrpt   1       200    1    88679.720      916.732   ops/ms

It's pretty clear that the native java implementation is substantially quicker. As for where they are being used, I don't have a great use-case for them other than the fact that libsodium exposes them and I'd like to see Kalium expose as much of the libsodium API as possible. But I suppose if that isn't the goal of Kalium and it's only interested in providing a subset of the API in an OO fashion then these really are not needed.

@bendoerr If I run mvn clean install, I get something like this:
[INFO]
[INFO] --- exec-maven-plugin:1.4.0:exec (run-benchmarks) @ kalium ---
# Fork: 1 of 10
Exception in thread "main" java.lang.IllegalArgumentException: No route to host
        at org.openjdk.jmh.ForkedMain.main(ForkedMain.java:68)
WARNING: Forked process returned code: 1
Exception in thread "main" java.lang.IllegalStateException: Acquiring the null result
        at org.openjdk.jmh.link.BinaryLinkServer.getResult(BinaryLinkServer.java:132)
        at org.openjdk.jmh.runner.Runner.doFork(Runner.java:426)
        at org.openjdk.jmh.runner.Runner.runSeparate(Runner.java:369)
        at org.openjdk.jmh.runner.Runner.runBenchmarks(Runner.java:303)
        at org.openjdk.jmh.runner.Runner.run(Runner.java:201)
        at org.openjdk.jmh.Main.main(Main.java:117)

Is any additional setup needed? Also, I'd suggest to create a separated profile to run such benchmarks.

This was my first time using jmh and google is coming up empty on that error. Curious if you are using JDK6 since the Travis build failed for that JDK. Also I wonder if the JVM is mucking up it's IPv6 ::1 vs IPv4 <IP_ADDRESS> detection of localhost.

@bendoerr I'm using OpenJDK 1.8, but this project compiles to 1.6 https://github.com/abstractj/kalium/blob/master/pom.xml#L27. Maybe, that's the issue?
Btw, I think now we are in 2016 is pretty much possible to drop the support for Java 6. If we are willing to bump up to 1.0.0 :)

I'm closing for now, because I believe this PR is out of date. But feel free to reopen, if you would like to add these changes.


irmin-pack: improve control file's atomic write properties

Currently, we assume that writes are atomic because the control file is small enough to fit within a page. We could improve this by using a known mechanism for atomic updates, like writing to a temporary file and renaming it.
Context https://github.com/mirage/irmin/pull/2180/files/7b9e971b221b577c26dd533f9f8853bf77967325#r1090672123

Implemented in #2206


[Epic] Extend the collection library

This includes

#9 Tree transformation
#102 Traversable.groupBy ✓
#110 Traversable.countOccurrences
#113 Tree extends Traversable
#170 Immutable HashMap ✓
#214 Immutable TreeSet ✓
#256 Re-organize collection.Tree
#271 More common TraversableOnce methods ✓
#278 Immutable Stack ✓
#279 Immutable Queue ✓
#295 Immutable Array ✓
#428 Add an immutable TreeMap to the javaslang.collection package
#429 Add an immutable HashSet to the javaslang.collection package ✓

Useful links:

http://en.wikipedia.org/wiki/List_of_data_structures
http://rosettacode.org/wiki/Rosetta_Code
http://xlinux.nist.gov/dads/

I've solved functionaljava/functionaljava#149 already for Javaslang but it will be interesting to track it anyway...

An overview of the next version of Javaslang, especially the collection API (work in progress):

RedBlackTree and HashArrayMappedTrie are backing data structures for the high-level collections Map/SortedMap, Set/SortedSet etc. I think it does not make sense to implement Traversable.
Here I want to share some thoughts on how to provide an appropriate abstraction for such data structures.
Idea

The interfaces should be kept slim. We may add functionality to special interfaces like RedBlackTree in form of static methods. This functionality is called by high-level collections to implement the Traversable interface. I've seen this in Scala's TreeSet which is used in conjunction with a RedBlackTree companion object.

Parent references will not be held.

To be discussed: I'm not sure if we will define general Tree methods which do not align very well with the nature of specific implementations. E.g. getChildren() makes not much sense for binary trees. Here we have a left and a right child. If we want a special Tree, like a BinaryTree, behave like a general Tree, maybe we should implement some sort of Views here. On the other hand - if we make our backing data structures public API, a full fledged general Tree interface would be nice.

Interfaces
We may introduce a general Tree interface containing values of type T:
interface Tree<T> {

    boolean contains(T value);

    // not sure about this, because of leaf trees with values only at leaves
    // T getValue();

    Tree<T> add(T value);
    Tree<T> remove(T value);

    boolean isEmpty();
    int size();

    // ...
}

RoseTree is the most general implementation of Tree.
Also we will need a top level interface Trie:
interface Trie<K, T> {

    boolean containsKey(K key);
    boolean containsValue(V value);

    Option<V> get(K key);
    Trie<K, V> put(K key, V value);
    Trie<K, V> remove(K key);

    boolean isEmpty();
    int size();

    // ...
}

Both, Tree and Trie, do not inherit from each other and have no common base interface.
RedBlackTree will implement an interface BinarySearchTree:
interface BinarySearchTree<T> extends Tree<T> {

    @Override
    BinarySearchTree<T> add(T value);

    @Override
    BinarySearchTree<T> delete(T value);
}

And then:
interface RedBlackTree<T> extends BinarySearchTree<T> {

    @Override
    RedBlackTree <T> add(T value);

    @Override
    RedBlackTree <T> delete(T value);
}

and
interface HashArrayMappedTrie<K, V> extends Trie<K, V> {

    @Override
    HashArrayMappedTrie<K, V> put(K key, V value);

    @Override
    HashArrayMappedTrie<K, V> remove(K key);
}

Trie.containsValue() is a very strange method. It can be implemented by naive search along whole trie or by storing values in additional structure. I believe it must be removed from interface

Yes, makes sense. I also think it belongs to the Map interface.

I thought much about the Tree collections today and came to the following insight:
Javaslang's goal is to keep things as simple as possible - when in doubt, leave it away.
Therefore HAMT and RedBlackTree will not part of the standard collection type hierarchy. Instead they form an architectural layer below our high-level collections (HashMap et. al.) as backing collections. However, they will definitely be public API because they have a high degree of reuse.
Given this, there is not need for the interfaces Trie and BinarySearchTree (for now). If they are needed in future, we will add them.
Under this condition, I will merge Tree and RoseTree and name it just Tree. Simple.


ReactorMongoEventStore | Fixed test case for filtering events by source.

Added persisting events with different sources.

Nice catch! Thanks a lot! :)


Limit bans to within the last 24 hours.

https://github.com/MicahZoltu/discord-mass-ban-bot/blob/0655d6159c459700d323f964deb0c7b73224e793/index.ts#L79
Currently, a rogue moderator can ban everyone who has ever joined.  This isn't ideal, and a simple 24 hour limit would mitigate this almost entirely.

Alternatively, limit the range of time between first message and last message.  This would also help protect against mistakenly large ban ranges.

Fixed in 3b4630320db700be0f015efa08878f6b69b0268c


Resolve issue256

Connect #256.
This issue reveals a corner case when creating a dependent resource with --force-on-exists on:
Suppose resource B can only be created on top of resource A (schedule on top of unified job templates in this case), The endpoint for creation is thus /A/pkA/B/. The modification endpoint, on the other hand, is /B/pkB/. A create method with --force-on-exists on converts creation to modification if the resource B exists, and the PATCH endpoints is simply concatenated into /A/pkA/B/pkB/, whose format is not a valid API for now.
The solution is dependent on API convention. Considering the possibility of further API convention changes (although highly unlikely ---- it makes little sense to have endpoints for individual related resource), the implementation here might change accordingly.

Coverage decreased (-0.03%) to 94.479% when pulling dff05d30f82840a8be047464483f9cd8217849f3 on jangsutsr:issue256 into c5d246a1bfa5dcc56f716c7d5b0cac8acb4ad7cd on ansible:master.

I kind of don't like this. It's just hard to read and if I happened to come by this point in code I wouldn't really be able to guess what it was doing.
I would prefer to outright refactor this:
        # Get the URL and method to use for the write.
        url = self.endpoint
        method = 'POST'
        if pk:
            url += '%s/' % pk
            method = 'PATCH'

        # If debugging is on, print the URL and data being sent.
        debug.log('Writing the record.', header='details')

        # Actually perform the write.
        r = getattr(client, method.lower())(url, data=kwargs)

Even this isn't super clear to read. What if we introduced something like a get_xxxx_url method, put in in the ResourceMethods class, and then over-ride this in the schedule class?
Refactor example:
        # Get the URL and method to use for the write.
        url = self.get_post_url(pk)
        method = 'POST'
        if pk:
            url += self.get_patch_url(pk)
            method = 'PATCH'

        # If debugging is on, print the URL and data being sent.
        debug.log('Writing the record.', header='details')

        # Actually perform the write.
        r = getattr(client, method.lower())(url, data=kwargs)

Coverage decreased (-0.8%) to 93.725% when pulling 940e9c5e9b28f14160603cbeb3f8397f311353a9 on jangsutsr:issue256 into c5d246a1bfa5dcc56f716c7d5b0cac8acb4ad7cd on ansible:master.


Upd dependencies

Issue # N/A
Description of changes:
Dependabot PR's combined.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: 753c07c6be593158fd64d3c2f79bd0439eab11b5
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


RangeError in /clan/alpinestar-y09v02rr

Error in ClashLeaders
RangeError in /clan/alpinestar-y09v02rr
Maximum call stack size exceeded
View on Bugsnag
Stacktrace
webpack:///node_modules/vue/dist/vue.esm.js:935 - observe

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been reopened in Bugsnag
RangeError in /clan/amare-bellum-9822jquu

An error linked to this issue has been marked as fixed in Bugsnag
RangeError in /clan/amare-bellum-9822jquu

An error linked to this issue has been reopened in Bugsnag
RangeError in /clan/battle-furry-pr88r0j8

An error linked to this issue has been marked as fixed in Bugsnag
RangeError in /clan/gui-cheng-feng-du-q8u0qu9

An error linked to this issue has been reopened in Bugsnag
RangeError in /clan/war-snipers-2-3-2qu90u2u

An error linked to this issue has been marked as fixed in Bugsnag
RangeError in /clan/chi-mo-qing-cheng-winter-tm-l2urglrl


Installation on Big Sur via Homebrew failed

with the following error:
[8/9] Compiling SwiftSyntaxHighlighter SwiftSyntaxHighlighter.swift
[9/10] Compiling swift-highlight main.swift
ld: warning: dylib (/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/macosx/lib_InternalSwiftSyntaxParser.dylib) was built for newer macOS version (10.12) than being linked (10.10)
[10/10] Linking swift-highlight
install: /private/tmp/swift-syntax-highlight-20201219-3997-1mtdrat/.build/release/swift-syntax-highlight: No such file or directory
make: *** [install] Error 71

I'm using Swift 5.3.2

@svanimpe This should now be fixed with version 1.2.0.


APIView - Improve the instructions to get access to APIView

APIView is accessible only if user a member of Azure or Microsoft GitHub org and this org membership has to be publicly available. Currently instructions on APIView is as follows when authentication fails.

This should be improved with more instructions and links and more instructions on how to change it to public to help them easily change org membership to public.
I think it’d help if we make the links more specific like:

Azure
o	Join Org: https://repos.opensource.microsoft.com/orgs/Azure
o	Make Public: https://github.com/orgs/Azure/people?query=your-alias
Microsoft
o	Join Org: https://repos.opensource.microsoft.com/orgs/Microsoft
o	Make Public: https://github.com/orgs/Microsoft/people?query=your-alias

I thought we changed the permissions model to check the Microsoft directory, so that public membership was not required.


Update cli.sh

The old usage block <order|hash> may be misleading. The result of cli.sh block $hash is null,  only can get the block by the command cli.sh blockbyhash $hash.

@blocklee
following are all the working examples, the block command support both order and hash as input arguments.
./cli.sh block 120
./cli.sh block 0x4595a9a8a924e3c799a0b54555c2f2ff8fdcad15c8fd3b0f413450831d2174a1
./cli.sh blockbyhash 4595a9a8a924e3c799a0b54555c2f2ff8fdcad15c8fd3b0f413450831d2174a1
./cli.sh blockbyhash 0x4595a9a8a924e3c799a0b54555c2f2ff8fdcad15c8fd3b0f413450831d2174a1

but without adding 0x prefix when calling the block command results an error, (using -D to see the error message)
./cli.sh block 4595a9a8a924e3c799a0b54555c2f2ff8fdcad15c8fd3b0f413450831d2174a1
{"jsonrpc":"2.0","error":{"code":-32600,"message":"invalid character 'a' after array element"}}

Please refactoring your PR.


Implement backup/restore test suite

We want to enhance the E2E test suite to support ability to provision different types of applications and take backups and restores of the apps while verifying they are functional and if they are stateful the data is preserved.
Demo Applications:
https://github.com/konveyor/mig-demo-apps
https://github.com/konveyor/velero-examples
Workflow:

Provision CI cluster
Install sample application
Run backup
Verify backup successful
Delete application (namespace)
Run restore
Verify restore successful
Verify any stateful data was preserved

For reference: https://github.com/konveyor/mig-controller/tree/master/tests


Where is this 'handy tool'?

In the section "How can I clear all the values on a push dataset?" a 'handy tool' is mentioned, but there is no link or reference available?

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

ID: a46e4476-1fc8-de9c-dc49-ffb8358d22b0
Version Independent ID: 7af1eac8-5069-8beb-cee4-29c413f7b957
Content: Real-time streaming in Power BI - Power BI
Content Source: powerbi-docs/service-real-time-streaming.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment @karmamule, and sorry for the delay and confusion. That handy tool is not available, and I've removed reference to it. Sorry for the inconvenience.


Helm dependency update prints Pulled: and Digest: lines into stderr

Output of helm version:
version.BuildInfo{Version:"v3.13.2", GitCommit:"2a2fb3b98829f1e0be6fb18af2f6599e0f4e8243", GitTreeState:"clean", GoVersion:"go1.20.10"}
Output of kubectl version:
Client Version: v1.24.1
Kustomize Version: v4.5.4
If you execute helm dependency update, all output is written to stdout as expected, except the lines at the end starting with "Pulled:" and "Digest:". These seem to be written to stderr with no real reason. They flood our logs and make finding the real error message harder.

"Pulled:" and "Digest:". These seem to be written to stderr with no real reason.

Helm under the hood is using ORAS SDK for OCI operations. I suspect this is naively logging to stderr (without checking the code).

execution wrapper about deciding when a helm command can be considered success or failure.

I recommend using structured output (or the SDK) for such use-cases (helm install --output ..). But I suspect the underlying issue here (ORAS library writing to stdout) will still persist.

See https://github.com/helm/helm/pull/11480 ans related issues

@jouve what a very weird pr, instead of omitting these lines when --output json is set, the solution is to print everything to stderr. It baffles me how this solution got accepted.

I still see this issue as relevant :/

Our customers are seeing this issue too - can we get an update on if this will be fixed or not?


return expandedPreset, not preset

Small bug where we were returning the wrong variable from the expandBuildPresetVariables method.
Fixes the testing done here https://github.com/microsoft/vscode-cmake-tools/pull/3905#issuecomment-2311910578.
Also fixes #4011
Thanks @v-frankwang!
@qarni FYI.

@gcampbell-msft @qarni  We verified both issues on CMake Tools: v1.19.47 (pre-release) and they have been fixed:
Actual result：
Issue：https://github.com/microsoft/vscode-cmake-tools/issues/2934

Issue：https://github.com/microsoft/vscode-cmake-tools/issues/4011


feat(compaction): limit memory usage for compaction read

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
LRU-Cache can not limit memory strictly, although we can make insert LRU-Cache fail when memory-limit full, but it's difficult to make write request block until memory enough.
So I used an independent sstable-store and memory-limiter to limit memory usage
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)

This PR could reduce half of memory but it can not still explain why our system use much more memory than statistic.
main branch:

current branch:

Question: is it possible to eliminate caching on the compactor? I don't think it's helpful because every being read will be compacted into another SST and the previous files won't be read (compacted) any more.

eliminate

Yes. I would not cache any data in compactor any more in this PR.
I only cache meta-file during compaction job just for several sub-compaction job share the same meta-file.
I do not use LRU-Cache for data file and each sub-compaction just read their own data, which may be some range of a data file.

@hzxa21 please take a look again


Changelog?

Today, I noticed a new version of shadow, 0.11.0.
I wanted to check whether it makes sense to upgrade the dependency in my Cargo.toml, but there is no changelog, and the GH releases aren't very informative either.
Is there any changelog I can find? If no, could you please consider keeping one for future versions?

@jaskij DONE. Use this tool:https://github.com/orhun/git-cliff


Collapsing folders tree

Expected Behavior
When I use ctrl + alt + mouse left click on folder in left sidebar, I expect that the all folder tree will open / unfold.
Current Behavior
It does not occur if theme enabled (I use "Boxy Yesterday"). Everything works if the standard theme.
Your Environment
Boxy Yesterday v2.0.3
Sublime Text 3 r3114
Ubuntu 14.04.4
P. S. Sorry for my bad english

Thanks for the issue. I'll try to fix it.

@PsiXdev Ok, I have some info for you. This is not theme related bug. Maybe you should use Alt key only with Mouse Left click.

Strangely. But with the standard theme ctrl + alt + Mouse Left is work.
Yes, I change some system configs and alt + Mouse Left began to work. Thank you.


LinkageChecker: Add libmvec.so.1 to the white list [Linux]

==> Downloading https://homebrew.bintray.com/bottles/md5deep-4.4.high_sierra.bottle.tar.gz
curl: (6) Could not resolve host: homebrew.bintray.com
Error: Failed to download resource "md5deep"
Download failed: https://homebrew.bintray.com/bottles/md5deep-4.4.high_sierra.bottle.tar.gz

https://travis-ci.org/Linuxbrew/brew/jobs/422885626#L517
Merging despite this transient error.


New upstream feature

Not a bug, but I've added functionality upstream to allow for the case-insensitive regex flag, so /foo/i would work now. Hope it might help here.

I've had a stab at switching to this, and although it does make the grammar easier to read, it introduces some errors, mostly around disambiguating plain identifiers and Fortran's unreserved keywords. Unfortunately, these do crop up in the real world.
See my branch for implementation, and this test file for some regressions.
One issue I realised is that we can't simply replace, for example, whiteSpacedKeyword('else', 'if') with just /end *if/i because it's possible to put a continuation line in between the keywords, but this is an orthogonal issue.

Closing since Fortran's language quirks cause issues.


Update watson-developer-cloud

@cfjedimaster commented on Fri Jan 06 2017
OpenWhisk is using a 1.* version of the watson-developer-cloud package, which is now at 2.*. Please see if you can upgrade to that version.

closed as going forward no vendor libraries should be added to the base.
The vendor and operators can extend this image an offer a nodejs kind with included vendor libraries like IBM Functions do today for nodejs:8 kind it includes watson, cloudant, and other sdks


[Blockstore] 2/3 reading for mirror-3 disk

In addition to the regular reading operation, mirror-3 disk could read a checksum from another replica, compare cehcksums and, if the comparition fails, perform a resync.

if the comparition fails, perform a resync.

this part will made the code more complex
I decided to simply return E_REJECTED
if the data is the same in 2 out of 3 replicas, one of the next retry attempts will hit those 2 replicas and succeed
there's no point optimizing this scenario since it should never happen (or almost never)
the discrepancy will be eventually fixed by the scrubbing process


An error occurred while starting the kernel.

Description of your problem
Till now I was using spyder very smoothly. But Recent update with conda has created this issue. I am using windows 8.1.
I have tried conda --reset but it didnt worked.
I have tried conda update setuptools still the kernel
What steps will reproduce the problem?

updating conda

What is the expected output? What do you see instead?

Please provide any additional information below
Versions and main components

Spyder Version:2.3.4
Python Version:2.7
Qt Version:
PyQt Version:
Operating system: win 8.1
Conda:4.3.13

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:

Please update Spyder to 3.1.3 (its latest version) and try again. That error should be fixed now.


fix: don't cache files between runs

This allows the lib to run in a long running process (eg: API that dynamically scans different repos).
fixes #12

The problem with this change is that walk is called recursively so allFiles and allFolders get cleared per subfolder.

I've added the change in a commit https://github.com/Nixinova/Linguist/commit/542baf76123744c27512af7e5808fe178283ffb8 because it needed a few more changes


[Client][CPP] Python client for Windows

Search before asking

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

Motivation
I try to build cpp client  with static flag, and it seems that link progress will be fail.
Here is my pr: https://github.com/apache/pulsar/pull/17507/files
cmake \
  -B ./build-1 \
  -G "${{ matrix.generator }}" ${{ matrix.arch }} \
  -DBUILD_PYTHON_WRAPPER=ON -DBUILD_TESTS=OFF \
  -DVCPKG_TRIPLET=${{ matrix.triplet }} \
  -DCMAKE_BUILD_TYPE=Release \
  -DBUILD_STATIC_LIB=ON \
  -S .

The workflow run is here:
https://github.com/apache/pulsar/runs/8223165825?check_suite_focus=true#step:12:239
Maybe we need to fix CMakeLists.txt to support the compile.
Solution
No response
Alternatives
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

The root cause might be

LINK : fatal error LNK1104: cannot open file 'python310.lib' [D:\a\pulsar\pulsar\pulsar-client-cpp\build-1\python_pulsar.vcxproj]

Pulsar Python client build requires a Python library installed on Windows. Could you help set up the Python in the workflow file? @yaalsn

I have confirmed again, the python3 dependency has already been added to vcpkg.json in your PR, I think it might be something wrong with the CMakeLists.txt.

I checked this issue today. It requires the changes to CMakeLists.txt because the dynamic library of Python client (_pulsar.so) should be linked statically, i.e. with the -DLINK_STATIC=ON CMake option. Unfortunately, when LINK_STATIC option is enabled, it only tries to find the *.a library, which is a static library in Linux and macOS, as the static library, while it should also be *.lib on Windows. See https://github.com/apache/pulsar/blob/8441f6724b1aa502df580518ae14f0c559f53547/pulsar-client-cpp/CMakeLists.txt#L144
BTW, @yaalsn your CI workflow is wrong. The BUILD_STATIC_LIB option means building a static library of Pulsar, but the LINK_STATIC option means the Pulsar library, which could be either static or dynamic, links to a static library.
For Linux build:

BUILD_STATIC_LIB=ON: libpulsar.a will be generated
LINK_STATIC=ON: If libpulsar.so or libpulsar.a is generated, the dependencies are statically linked. For example, for the libcurl dependency, libcurl.a is linked (just in compile time because it's statically linked) while libcurl.so is not used.

I'm working on this issue in my own branch: https://github.com/BewareMyPower/pulsar/commits/bewaremypower/cpp-static-link. Now the Pulsar libraries can be built successfully on MSVC with LINK_STATIC=ON, but the Python build still failed. I will continue fixing it.

@BewareMyPower  Thanks for your help!


how to use the teacher model inner the model.py

I saw the  implemented Distilling model in your code. So, how to use the inner teacher model  of the model.py.

what you mean by using the teacher model? Teacher model and distilled model are the same.


'MauiAppBuilder' does not contain a definition for 'ConfigureJailbreakProtection'

I created a new Maui app using VS 2022 and .NET 7.
I added the NuGet package 'banditoth.MAUI.JailbreakDetector' and followed the steps from the readme.txt file.
When building the app I get 2 errors:
Error 1:
The type or namespace name 'banditoth' could not be found (are you missing a using directive or an assembly reference?)
Error 2:
'MauiAppBuilder' does not contain a definition for 'ConfigureJailbreakProtection' and no accessible extension method 'ConfigureJailbreakProtection' accepting a first argument of type 'MauiAppBuilder' could be found (are you missing a using directive or an assembly reference?)
I check in the project file and discovered that adding the NuGet package creates these 2 groups:
<ItemGroup Condition="'$(TargetFramework)' == 'net7.0-android'">
  <PackageReference Include="banditoth.MAUI.JailbreakDetector">
    <Version>1.0.0</Version>
  </PackageReference>
</ItemGroup>

<ItemGroup Condition="'$(TargetFramework)' == 'net7.0-ios'">
  <PackageReference Include="banditoth.MAUI.JailbreakDetector">
    <Version>1.0.0</Version>
  </PackageReference>
</ItemGroup>

Nothing added for Windows.
I tried adding one of these for Windows, but then I get a bunch of building errors stating that Windows is not supported.
At this point I'm stuck. Apparently this package can't be used, no matter if I select Windows, Android or iOS.

You can easily circumvent this problem by wrapping the using and .ConfigureJailbreakDetection block into #if !WINDOWS #endif preprocessor blocks.

You can easily circumvent this problem by wrapping the using and .ConfigureJailbreakDetection block into #if !WINDOWS #endif preprocessor blocks.

Thanks for the answer on this problem. This is the solution.


MAISTRA-1224 Skip flaky test

/retest

I think we should make the test not flaky.  This test is actually useful for what I'm doing with MAISTRA-1236, as it verifies the complete processing of a single event through the reconciler.  If the issue is with the timeout, then we should increase that.  It may be better to just use the normal test timeout instead of specifying it in the test case.  (Default test timeout is 30s.  I think I added this because I wanted to test not seeing objects, which I was using to gate the end of the test, but that was before I added code to synchronize reconcile completion, so the only "hole" is whether or not another reconcile has been triggered.)

+1 on skipping for now. I'll try to fix the test if no-one else is looking at it yet.

There are two issues with the test.  The primary issue we are seeing is that it times out when running in prow.  I'm working on removing the timeout now, so it will just use the normal go test timeout.  If you look at the results for the failed prow tests, this is all of the failures not caused by legitimate test errors.
The other I have seen is a race condition with the waitForReconcileCompletion() call, but that only happens on really fast systems (basically, Wait() is called on the WaitGroup before Add() has been called.  I suppose this could also happen if wait is called in between events on the reconcile queue, as Add() and Done() surround Reconciiler.Reconcile().  The ideal solution is to integrate synchronization into the reconciler's queue, as that would tie up all the loose ends in the test harness.  That said, I have only seen this timing issue a handful of times.

Found the issue with the hanging tests.  There is only one issue left, which occurs very rarely (I've only seen it a handful of times over the last few months) and may be related to poorly written test cases, or cases that should fail in other ways (i.e. reconcile queue is not empty).

@dgn I believe you can now close this.


fix: add support on mode light Astro's logo

Fixes #77
https://github.com/hngngn/shadcn-solid/assets/116913262/742cb936-90bc-4241-a361-d67f53571700

thanks!


JMXPort option?

Ours is on like 16199..
I tried hardhacking
"-p","16199",
in the various nodetool calls, but no dice. is -h necessary if I'm local to the cass node?

-h is necessary if you have Cassandra binding to a specific IP address and not <IP_ADDRESS>.  By default, nodetool will talk to Cassandra on localhost, which won't work so well in that case.
Is 16199 your JMX port?  Because if so, yeah, you're going to need to specify the port in the nodetool command, and the easiest way to do that right now is:
cat > ~/bin/nodetool_port <<EOF
#! /bin/sh
exec nodetool -p 16199 "$@"
EOF
chmod +x ~/bin/nodetool_port
range_repair -n ~/bin/nodetool_port ...

It might make sense to allow an arbitrary set of nodetool options to be passed? Others missing are username/password for JMX authentication.


Fix nadam optimizer

Resolves issues #15035 and #13980.

Thanks for the PR! Could you add a test that would have failed without this change.

Thanks for the review! I updated the test.
The test wasn't failing before because the "sparse" test was creating a sparse vector that was actually dense (i.e. it's of size 2 and values are being specified for both indices). FWIW it looks like a number of other optimizers (at least Adam, LazyAdam) have the same shortcoming in their tests.

Alex, it looks like you reviewed the original nadam PR https://github.com/tensorflow/tensorflow/pull/9889. I'm not familiar with the nadam algorithm.

Hey Emma!

Hey Alex, thanks for the review!

Nagging Assignee @case540: It has been 17 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@strubell could you pull rebase and push again?

Sorry I've been travelling -- will do this soon. Thanks for taking a look!

Nagging Assignee @case540: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@strubell could you pull rebase and push again?

Please send this PR to tensorflow/addons instead; contrib is deprecated and stated to be removed so we're not making new changes to it.


add apexsystemsinc.com

See screenshot below - basically all the wrong things

"Hello _____ " (my name isn't even there)
"Found my resume online" but didn't look that I started a new job < 1 month ago on LinkedIn (it's public),
Referred to dated skills (stuff I haven't touched in years) and lack of any particular attention to career focus or direction.

This isn't just spray and pray, this is worse - it's automated spam.

Thanks. I should add that the Unsubscribe link included in the signature was broken on clicking it.
An Error Occurred
We're sorry, but there was a problem with your request. Please try again. If the problem continues, you can check http://status.bullhorn.com for up to the minute system status, including notices from our support and operations team.

If you require assistance with this issue, please contact Bullhorn Support using The Bullhorn Resource Center available under the Support menu.


Update install_all.sh to add -y option to apt-get install

This allows the install script to run to completion without user input.  This change was required to include in a docker build.

I made some necessary changes to the install script, to get the tagged branches of folly and fbthrift rather than the trunk this was necessary to fix build breaks. also added the running of ldconfig after installing folly, seems to work. Did not add install -y, but adding it now.


fix beforeDestroy (typo)

:tada: This PR is included in version 1.6.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


spec: Add JavaScript API specifications

for OIC, AIO, GPIO, I2C, PWM, SPI, UART.

+1
@bdilly and others, @zolkis will add node.js integration to the soletta main repo, starting with docs and then code and all :-)

yeah, we've talked a bit in these last weeks.
API / code snippets seem good to me.
I'll ping a few guys here to take a look at least in the parts they're working on. If no complains are raised, I think we're good to merge it as is.
Thanks @zolkis

Looks good for me!

Thanks @brunobottazzini for reviewing! Should I amend the commit because mainly white-space errors, or just should I push another for keeping these comments visible/trackable?

@zolkis, I'm used to amend on my PRs and github updates the PR automatically.

Yes. Done :).

the oic specs look good to me

merged, thanks


Remove Header trait

The Header trait is only used in tests, and doesn't serve a clear purpose. We should remove it.

In ibc-go use ClientMessage to Update ClientState, replace Header.


rocprofv2: aqlprofile API table load failed

Describe the bug
When trying to use rocprofv2:
❯ rocprofv2 ./a.out 10
aqlprofile API table load failed: HSA_STATUS_ERROR: A generic error has occurred.
/nix/store/c5iy8c6w34y4jfgqlv8c6scxrilc4djv-rocprofiler-5.7.0/bin/rocprofv2: line 264: 77294 Aborted                 (core dumped) LD_PRELOAD=$LD_PRELOAD:$ROCM_DIR/lib/rocprofiler/librocprofiler_tool.so $*

This might have something to do with some predefined paths in rocprofv2
Steps To Reproduce
Steps to reproduce the behavior:

Compile an executable with hipcc
Open it with rocprofv2

Notify maintainers
 @NixOS/rocm-maintainers
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.5.7, NixOS, 23.11 (Tapir), 23.11.20231016.ca012a0`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - nixpkgs: `/run/current-system/nixpkgs`

I need to do some work with rocProfiler.

I need to make a patch that strips hsa-amd-aqlprofile-bin.
Apparently I need to fix rocProfiler itself.
I wonder if it's because I'm stripping out the test directories?

I need to do some work with rocprofiler.
1. I need to make a patch that strips `hsa-amd-aqlprofile-bin`.

2. Apparently I need to fix `rocprofiler` itself.

I wonder if it's because I'm stripping out the test directories? Also yes, it looks like you're right. You can fix that by appending ROCM_DIR=/nix/store/rocprofiler for now. Although it seems to need hsa-amd-aqlprofile in ROCM_DIR too, so I'm not entirely sure it'll work. Try setting ROCM_DIR to rocmPackages.meta.rocm-developer-tools. You may just be able to run it from that as well.

I don't think that removing hsa-amd-aqlprofile-bin is a good idea, because without it, it loses performance counters support. I think there should at least be a rocprof derivation with aqlprofile.

My bad, I forgot to add that the patch is optional.

Right.
Could you tell me your test environment and the source of the hip executable (or just some stub) so I can replicate this?
I've never actually used rocprofiler.

Right. Could you tell me your test environment and the source of the hip executable (or just some stub) so I can replicate this? I've never actually used rocprofiler.

You can try this, it just prints 10 numbers, it can be compiled with hipcc without any flag. I am using a radeon rx6600m(gfx1032).
#include <hip/amd_detail/amd_hip_runtime.h>
#include <stdio.h>

__global__ void gene(int *array) {
  int tid = hipBlockDim_x * hipBlockIdx_x + hipThreadIdx_x;
  array[tid] = tid;
}

int main() {
  int device;
  hipDeviceProp_t prop;
  hipGetDevice(&device);
  hipGetDeviceProperties(&prop, device);
  printf("DEVICE: %s\n", prop.name);
  int thread_per_block = prop.maxThreadsPerBlock;
  printf("Max thread per block: %d\n", thread_per_block);

  int n_elem = 10;
  // 1 device array and 1 host array
  int *d_array, *h_array;

  int array_size = n_elem * sizeof(int);

  hipMalloc(&d_array, array_size);

  int numblocks = (n_elem + thread_per_block - 1) / thread_per_block;

  gene<<<numblocks, thread_per_block>>>(d_array);

  hipDeviceSynchronize();
  hipHostMalloc(&h_array, array_size);
  hipMemcpy(h_array, d_array, array_size, hipMemcpyDeviceToHost);
  hipDeviceSynchronize();

  for (int i = 0; i < n_elem; i++) {
    printf("%d\n", h_array[i]);
  }

  hipFree(d_array);
  hipFree(h_array);

  return 0;
}

@Tungsten842 see  #262798


Default api is not ready yet, do not forget to call initialize from 'vscode/extensions'

Has anyone encounter this problem?
I followed this example https://github.com/TypeFox/monaco-languageclient/blob/main/packages/examples/main/src/common.ts
Though Im getting this

Im using https://github.com/python-lsp/python-lsp-server/tree/develop as my language server
WebSocket

        connectToPythonLSP(){
            const url  = "ws://localhost:6083"
            const webSocket = new WebSocket(url);
            webSocket.onopen = () => {
                const socket = toSocket(webSocket);
                const reader = new WebSocketMessageReader(socket);
                const writer = new WebSocketMessageWriter(socket);
                const languageClient =  this.createPythonLanguageClient({
                    reader,
                    writer
                });
                languageClient.start();
                reader.onClose(() => languageClient.stop());
            };
            return webSocket;
        },

Monaco Language Client

            return new MonacoLanguageClient({
                name: 'Client',
                clientOptions: {
                    documentSelector:  ["python"],
                    errorHandler: {
                        error: () => ({ action: ErrorAction.Continue }),
                        closed: () => ({ action: CloseAction.DoNotRestart })
                    }
                },
                connectionProvider: {
                    get: () => {
                        return Promise.resolve(transports);
                    }
                }
            })

I tried to make the error message as self explanatory as possible, is it still not good enough? :grin:

Actually, I tried to call the initialize from the vscode/extensions but still the error appears.

Yes, I tried both using with and without await but when I use await it never proceeds to the codes below it

I created a console printing console.info("====initServices started!====") and console.info("====initServices done!====") as marks but it seems that when I call await initServices() it never proceed printing console.info("===initServices done!====")

            console.info("====initServices started!====")
            await initServices({
                enableThemeService: true,
                enableTextmateService: true,
                enableModelService: true,
                configureEditorOrViewsServiceConfig: {},
                enableKeybindingsService: true,
                enableLanguagesService: true,
                enableQuickaccessService: true,
                enableOutputService: true,
                enableAccessibilityService: true,
                debugLogging: true
            })
            console.info("===initServices done!====")

            this.initMonaco(file.language,file.contents);

It seems that having multiple monaco editors is causing the initServices to hang in my app. I was able to make it work with some errors like No file system provider found for resource 'inmemory://model/.vscode'. I'll try again in some other time.

What do you mean having multiple editor causes the hang? You're not supposed to create editors before service initialisation either

My guess is all of my editor components are loading at the same time and I'm only applying initServices in one component. I'll try next time moving the initServices in the parent component to make sure I load it first before initializing all the editors.


WFLY-18804 + WFLY-18805 + WFLY-18806 Upgrade to Hibernate Search 7.0

WFLY-18806: Upgrade to Lucene 9.8
WFLY-18805: Upgrade to Elasticsearch client 8.11
WFLY-18804: Upgrade to Hibernate Search 7.0

Creating as draft, because:

This only upgrades to Hibernate Search 7.0.0.CR2; the Final hasn't been released yet.
We need to upgrade to Hibernate ORM 6.4 first (and probably at the same time, since Hibernate Search 6.2 isn't officially compatible with Hibernate ORM 6.4).

Thanks @yrodiere. Re ORM 6.4, @scottmarlow is tracking that and had filed https://github.com/wildfly/wildfly/pull/17447. But he closed it for now while we wait for 6.4.1.

Thanks @yrodiere. Re ORM 6.4, @scottmarlow is tracking that and had filed #17447. But he closed it for now while we wait for 6.4.1.

Thanks, let's wait for that upgrade then.
In the meantime, Hibernate Search 7.0.0.Final was just released, so I updated this PR accordingly.


Connect-Spotify Unable to Redirect

Hi there, love your code and gave it my best shot combing it through it to see the error. It looks like it may not be passing any properties correctly the the redirect URI under the Connect-Spotify process. As a result im getting bad page requests after submitting my username and password. It never shows the auth screen for me to confirm it can have access and there never connects and/or gets a token. let me know if im doing anything wrong.
Also, i noticed the strings file the system is pulling from and referencing is empty. not sure if that's intentional or not.

Hey Backlash,
It should be working fine. The RedirectURI is defaulted to https://localhost:8001, so make sure that is set in your Spotify dev portal. Of course you can use your own if you wish.
FYI, I just made a massive update to the code base before a conference at expertslive. A lot of updates you might like.


refactor: remove unused env property from ExpressoConfig interface

Pull Request Guidelines
Our guidelines for submitting a pull request.
Before submitting a Pull Request, please make sure you have verified the following:

[x] The commit message follows our guidelines:

A good commit message should be two things: meaningful and concise. It should not contain every single detail, describing each changed line—we can see all the changes in Git—but, at the same time, it should say enough to avoid ambiguity.
We use Microverse's commit message convention
The convention stablish that a commit message has to be in the present tense, imperative and lowercase.
Example: fix typo in README.md

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?
Please describe the current behavior that you are modifying, or link to a relevant issue.
New library to group all common functionalities between expresso libs such as core, adapter, cli etc.
Issue Number: N/A
What is the new behavior?
Describe the new behavior or link to a relevant issue.
Does this PR introduce a breaking change?

[x] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications below.
Other information
Any other information that is important to this PR.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.62%. Comparing base (5995b1a) to head (97663e8).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   79.62%   79.62%
=======================================
  Files           6        6
  Lines         265      265
  Branches       64       64
=======================================
  Hits          211      211
  Misses         46       46
  Partials        8        8

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


Alarm.com ADC-T2000 Configuration Parameter 5

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
The configuration for parameter 5 has some form of mask/offset, so that the valid range is not actually -100 to 100. For example, 704643072 in the parameter means 0 offset. As a result, zwavejs gives an error when trying to set this configuration parameter:

Here is how it should be set written by the developer of the device handler from smartthings:
https://community.smartthings.com/t/release-alarm-com-z-wave-thermostat-adc-t2000/74833/28
Device information
Manufacturer: 0x0190
Model name: adc-t_2000
How are you using node-zwave-js?

[ ] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[X] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
8.11.6
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
No, it never worked anywhere
If yes, where did it work?
No response
Attach Driver Logfile
to be attached when required.

I wonder if this is the same as for the ADC-T3000, that was added here: https://github.com/zwave-js/node-zwave-js/pull/4131
I guess parts of the new config from there need to be carried over. @EiNSTeiN- you don't have a T2000 by any chance? :)

I have 2 of those but not connected to HA right now... Last time I connected one, I did notice the zwave float values and I made a note to open a PR when I get ready to transfer my T2000 thermostats to HA.
If someone else wants to tackle this first, it should be a matter of adding a 0xffff00 bitmask like this on every configuration parameter that represents a temperature in the adc-t2000 config.

I don't think that you're trying to set a valid value. Our parameter matches the zwave alliance DB, OpenHAB, and the SmartThigns handler that you referenced. At SmartThings, the parameter is the same as ours:

Calibration Temperature Range (in deg. F) Precision is tenths of a degree.
Range: -100 to 100
Default: 0

I've tried to set it with -100 to 100 in the beginning. It says the parameter has been updated, but the thermostat doesn't reflect the change, and if I refresh the Z-Wave device configuration page, I see the number is reverted. See below screenshots:
This is when I set it to 20. As you can see, no error is shown:

The thermostat reading is not reflected after the change:

And if I go back to the device configuration page, the number has been reverted. (note this used to say 704643072 when I initially opened this issue. It has recently changed to truncate the digits)

Let me know how I can submit a more detailed report/log if you suspect I've done something wrong instead of the bitmask issue as suggested by @EiNSTeiN- EiNSTeiN

That looks accurate to me? It's at 22.4 and you've set two tenths precision. That allows it to exceed the set temp for the defined range (dead band). Part of the difficulty in telling is that the device is looking for Fahrenheit but you're HA is in Celsius.
This wouldn't be the first device to report back an improper value. We'd need to see the zwavejs logs at a silly level.

Keep in mind the value is encoded as a zwave float. The top 8 bits are the size/scale/precision, the bottom 8 bits are unused, the middle 16 bits are the actual value. Everything except the middle 16 bits are read only so all you need to do is shift the value you want to set by 8 bits (multiply by 256). Instead of 100, enter 25600 and that should set that zwave float value to 10.0.

That is not true. Not on this device per the manual or the Smartthings handler.

Yes it is. What else does 704643072 decode to if not a zwave float. It is 0x2A000000, where 0x2A is the correct precision/size/encoding for tenths of a degree, Fahrenheit, 2 byte value.

We've seen lots of devices before that report back something completely erroneous. All I'm saying is so far we have a screenshot with one value being set. The fact it's reporting that back doesn't mean that is the correct format to set it.

Gotcha, I agree I haven't tested. Can do it tomorrow as I've recently connected my T2000 to HA

@EiNSTeiN- any new insights?

Sorry for the delay
I had to increase maxValue for one of the parameter to test this. I changed parameter 5 from 704643072 (0x2A000000) to the new value 704645632 (0x2A000A00) which should set the calibration temp to 10 (in 0.1F increments, so 1 degree).
I tried setting the value to 704668672/0x2A006400 (or how I expect the value 100 would be encoded) for parameter 5, this works and the value sticks, but anything above 100 (e.g. 101 would be encoded as 0x2A006500) does not stick and reverts back, which points to what I would expect if the value is encoded into the second and third bytes like for the ADC-3000.
I also tested setting the value back to 0x2A000000 and then to 2A0064FF (i.e. 100 for the value, and all bits set for the lower byte). This sets the value to 704668672/0x2A006400 as we would expect if the bottom byte is unused or readonly.

Can someone clearly explain how to translate these "Z-Wave float" values into something sensible? I'm not following. I have an ADC T-2000 and currently, several parameters contain these types of values (e.g., 704643072) – it's unclear how to alter them. Thanks!


Fix object deletions not working

Package is now required for CLAS deletion for RS_CORR_INSERT but was not correctly determined for object deletions because of different field names when using MOVE-CORRESPONDING and another missing assignment.
Fixes #5593
Would be nice if someone could double check this change.

Ran into this issue when pulling the new abap2xlsx after the removal of ZCL_EXCEL_APACK.
I can confirm that the change works!

thanks, merging


Toast not working when running from app directory.

Current Behavior
When using the next-expo-solito template with app directory, toasts will not work on web. useToastController seems to return an empty object. Same thing works just fine when running from pages directory.
I also noticed that @tamagui/config is needed for the app directory version, but not included in the package.
Expected Behavior
No response
Tamagui Version
latest (1.38.1)

Reproduction
See reproduction in this CodeSandbox:

https://codesandbox.io/p/sandbox/sad-panini-z7zjsm?file=%2Fapps%2Fnext%2Fnext.config.js%3A12%2C1

What I did:
- changed `experimental.appDir` to `true` in `next.config.js`
- got rid of `pages` directory (renamed to `_pages`)
- installed `@tamagui/config` into `apps/next` (that package was missing, so `TamaguiProvider.ts` would not work)
- commented out `Solito` related things that would also break within the `app` directory since the latest stable version (3) does not support it

System Info
No response

Second. I've run into the same issue where useToastController returns an empty object making it so .show() is undefined.

bump, I'm also encountering this

For me also the @tamagui/conifg is not importing correctly

Bump / still an issue in 1.75.7

I believe I found the issue.  In the file apps/next/app/TamaguiProvider.tsx the provider is different from the provider used in the pages version of things.  After working backwards through the labyrinth of nested abstractions, I didn't see the toast provider anywhere in there, which would explain the .show() is undefined error.
In the end, I just imported and used the same provider component the pages dir version was using.
So, in apps/next/app/TamaguiProvider.tsx just import { Provider } from 'app/provider' and then comment out/remove the TamaguiProviderOG usage and pass in the newly imported provider.  Here's an example of the return of apps/next/app/TamaguiProvider.tsx
  return (
    <NextThemeProvider
      onChangeTheme={(next) => {
        setTheme(next as any)
      }}
    >
      <Provider disableRootThemeClass defaultTheme={theme}>
        {/* <TamaguiProviderOG config={config} themeClassNameOnRoot defaultTheme={theme}> */}
        {children}
        {/* </TamaguiProviderOG> */}
      </Provider>
    </NextThemeProvider>
  )
}

The appDir setup is no longer included in the starter, but this is most likely an issue with your setup.


Add custom dependabot config

Closes WRAP-587

I'm fairly sure the syntax is correct (comparing it to the docs), so I'm going to merge this, and see what dependabot thinks.

Looks like I was right: https://github.com/etchteam/recycling-locator/runs/25219950109 😅


Virtual kubelet: add shadow pod label to offloaded pods

Description
This PR reintroduces the addition of the shadow pod label to offloaded pods through the virtual kubelet. This allows to reduce the overall RAM requirements of the different components, thanks to filtering at informer level.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[x] Manual testing
[ ] Unit testing (new + existing)
[x] E2E testing (existing)

/test


build-support/fetchgitlab: enable to pass private_token as an option

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

This may put the token world readable in the corresponding .drv file.

Our use case is providing token from environment variable like:
token = builtins.getEnv "GITLAB_PERSONAL_ACCESS_TOKEN";
Does this protect the token from the world?
I would appreciate some instructions how this should be implemented here.

Passing environment variables is unlikely to work because of the sandbox.
Even if you disable the sandbox, the build is executed by the daemon, not by the nix-build process for which you set the environment variable.
Also if you do so, anyone with enough privileges to run builds can leak the environment variable.
No security is gained overall.


AWS KMS SDK ThrottlingException isn't surfaced

Describe the feature
Add a ThrottlingExcepton type to the KMS SDK.
Use Case
We'd like to use the library (with retries turned off) so we can surface that the caller of our code is being throttled and let them deal with it however they'd like to. Our particular use case is symmetric encrypt/decrypt operations, which have a combined limit of 50000 op/s, which is hard for us to test locally.
Proposed Solution
Add support for ThrottlingException to the KMS SDK, which is an existing error type on several other SDKs.
Other Information
According to KMS docs, KMS endpoints can return ThrottlingException. That kind doesn't exist in the KMS Error type and as far as I can tell reading the code isn't surfaced by any of the above layers (general smithy stuff or retries).
Acknowledgements

[X] I may be able to implement this feature request
[ ] This feature might incur a breaking change

A note for the community

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Hey @skeet70, thanks for submitting this issue. The service team hasn't gotten around to modeling this error variant so we filed an issue with the KMS team. Luckily, there's no need for you to wait. When a service sends an SDK client an error that isn't modeled, we deserialize it to an Unhandled variant. In your case, you should be able to call the .code() method on the throttling error and see if it's ThrottlingException. I imagine the code would look something like this:
let res = <your encrypt opertation>.send().await;

if let Err(err) = res {
  if err.code() == Some("ThrottlingException") {
    // handle throttling errors
  } else {
    // handle other kinds of errors
  }
}

Let me know if that approach works for your use case.

I haven't had a chance to come back and try the workaround yet, but since the code is preserved in the Unhandled variant that will likely work for us for now while we wait for a real error variant.

Opened an issue in aws/aws-sdk which is used to track issues across multiple sdks. Closing this issue


luci-theme-openwrt-2020The updated interface is very cluttered！！！

This looks like an unofficial modification (the temperatures are not part of the offical status page). I suppose this modification introduces invalid markup.


Add support for export and import of inflight items

This pull request adds buttons to export and import inflight items.

Hi,
Is it also possible to export the whole app's configuration, starting from the server, publishes, subscribes?

Jou can copy the entire content of directory "C:\Users[USER].MQTTnetApp" between systems. It contains everything which is stored for this application.


Change the transmission buffer length

Hi
I have a problem with the published message buffer length. A message like this:
ex. {"meaning":"state","value":0} ,is properly published.
But if I send in a unique message with several Json (ex: [{"meaning":"state","value":0},{"meaning":"humidity","value":33.4},{"meaning":"noise","value":4576},{"meaning":"temperature","value":15}]) I can't, is too long.
How can i do?Can i change the transmission buffer length?How?
Thanks you.

Hi,
please see http://pubsubclient.knolleary.net/api.html#configoptions - MQTT_MAX_PACKET_SIZE
You need to edit PubSubClient.h to change that value.


Fix PR-37 external hcloud manager

Quick fix to just remove the extra flag --cloud-provider=external on provisioning external node.
Not fully tested right now as the build is broken, see #38

Ok now we have the behaviour that we want as we don't add the kubelet option to specify external cloud, I also added the option to pass a SSH port on the external node add command.
Also on the remove command we were not capturing the passphrase so it was not able to delete the node from kubectl directly.

So after recreating a cluster from scratch and trying to add my eternal node, everything works as expected, kubelet is started without --cloud-provider=external but still the node gets deleted automatically, so not enough apparently.

This is because of the hcloud-controller-manager.
Until the issue at that project, the only workaround is to delete that deployment.
You could build a check, if the deployment is present and in that case, remove it before joining the new node


UserService/PersonService/ProjectService: use backend mongodb directly

Fixes #923
Done:

Implemented alternate UserService that directly connects to authority database in mongodb
Implemented alternate PersonService that directly connects to authority database in mongodb
Implemented alternate ProjectService that directly connects to authority database in mongodb
Added viper argument mongodb-uri

TODO:

Add environment variable BIBLIO_BACKEND_MONGODB_URI and set it to something like mongodb://user:password@localhost:27017/?maxPoolSize=20&w=majority&connectTimeoutMS=1000&serverSelectionTimeoutMS=1000
check mongodb connections as the defaults allow for too large timeouts (which leaves the websites hanging). I noticed that connectTimeoutMS and serverSelectionTimeoutMS are set to 30s by default..

Notes:

no need for reconnection to mongodb. Is done automatically

@nicolasfranck great thanks
is this ready to be merged? it has a TODO => you can make the timeouts smaller
btw did you check if the nomad machines have access to mauriac/mauriac9/mauriac0?

@nics access to the mongo cluster needs to be checked indeed, and variable needs to be added. I guess we control those variables, right?

Dependent on firewall access: https://redmine.inuits.eu/issues/104646

I have added the missing (search) services that were still part of the biblioClient.
This requires add additional environment variable BIBLIO_BACKEND_FRONTEND_ES6_URL,
which has the exact same syntax as BIBLIO_BACKEND_ES6_URL. Not sure if it is necessary
to specify a separate source url (I suspect backend and frontend ES may be located else where in the future?)


added jacoco dependency

added Sonarlint visual studio code extension

configured the sonarqube connection in sonarlint visual studio code extension

created new project in sonarqube portal in the kubernetes pod
generated the token and copy the command to the terminal
mvn clean verify sonar:sonar \   -Dsonar.projectKey=JUnit-Practice \   -Dsonar.host.url=http://<IP_ADDRESS>:32001 \   -Dsonar.login=sqp_2fddd8c773774462351226ebe346e469d43a2a2d

executed the above maven command in previous comment

sonarlint is working in the IDE

Measures in SonarQube and its purpose
Coverage = percentage of lines of code covered by tests
Duplications = identical lines of code or the same blocks of code
Reliability = Coding Error that will break your code immediately
Security = Codes that hackers can exploit
Security Review = Security sensitive code
Maintainability = Set of conditions to enforce quality policy for your organization
Cognitive Complextiy = Code that can be confusing or difficult to maintain
Debt or Technical Debt = Estimated time takes to fix all code smells


[6.4.x] JBPM-5024 - error after timer causes the process to abort

(cherry picked from commit 3619fd9928d9eede814e1d3509e137894f1a3791)
back port of #422

jenkins retest this

jenkins retest this

merged into 6.4.x


When will Go be supported？

alphazero , muzero, Gumbel MuZero  in go game

Hello! Regarding the support on the Go environment, we have been working on this PR (https://github.com/opendilab/LightZero/pull/65/files) for a while. This PR includes the initial implementation of the regular self-play version of AlphaZero for chess games like Go/Gomoku and a league training version. Although the performance did not meet expectations previously due to limitations in computational resources, we plan to further optimize this PR in the coming weeks. We welcome your discussion and contributions and look forward to your participation and feedback!


client handles errors differently when given heartbeat 0,0 vs 5000,5000

It seems the stompit client does not handle errors the same way when configured to heart-beat at 0,0 vs 5000,5000. With numbers larger than 0, the client correctly emits an error which can be caught and used to reconnect. It seems with 0,0 the error will automatically go to uncaughtException.
The code I'm using looks similar to the following:
    const manager = new stompit.ConnectFailover(servers, reconnectOptions);

    manager.connect((err, client, reconnect) => {

      if (err) {
        // terminal error, given up reconnecting
        this.emit('error', err);
      }

      console.log('connected to bus');

      client.on('error', (err) => {

        console.log('stompit client error (reconnecting now): ', err);

        this.reconnecting = true;

        reconnect();

      });

      this.client = client;

      if (this.reconnecting) {
        console.log('reconnected to bus');

        // replay and sends that did not yet reach the broker
        console.log(`re-sending ${this.pendingSends.size} _sends`);
        this.pendingSends.forEach((pendingSend) => {
          setTimeout(pendingSend, 500);
        })
        console.log(`reapplying ${this.subscriptions.length} subscriptions`);
        // resubscribe to any disconnected subscriptions now that we've reconnected
        this.subscriptions.forEach((subscription) => {
          subscription();
        });
      } else {
        this.emit('connected');
      }

      cb();

    });

In both scenarios, I see console.log('stompit client error (reconnecting now): ', err); logged. But with 0,0 I immediately see a log statement from my process.on('uncaughtException and the additional reconnect logic never gets a chance to run. With 5000,5000 the reconnect logic works as expected.

If you see console.log('stompit client error (reconnecting now): ', err); then the reconnect function is definitely called. Are you sure the uncaught exception is not caused by this.emit('error', err);?
The stompit client has the same expected usage in both scenarios.
The implementation of Client and ConnectFailover listen to catch errors events from their underlying objects and this behaviour isn't changed by heart-beating being enabled or not.
To help debugging we need details on the uncaught exception. What is the error message and what is the call stack on the Error object and the call stack of the uncaughtException handler?


Fix the value of VsixOutDir in sign.proj file

Hi @Faizan2304, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Faizan Ahmad). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;