fix #22282 StatusBar.setHidden() error

Related to https://github.com/facebook/react-native/issues/22100 and https://github.com/facebook/react-native/pull/22282#issuecomment-442566154
StatusBar.setHidden() is caused that error, so I changed arguments 0 required yes -> no.
Test Plan:

[x] npm run prettier
[x] npm run flow
[x] npm run flow-check-ios
[x] npm run flow-check-android
[x] npm run lint
[x] npm run test
[x] ./scripts/run-android-local-unit-tests.sh

Changelog:
[GENERAL][ENHANCEMENT][StatusBar.js] - setHidden arguments 0 required yes -> no

We moved the StatusBar project out of React Native as part of our Lean Core effort #23313. Please submit your PR in the new repository again: https://github.com/react-native-community/react-native-statusbar


Not able to scroll the content inside BottomSheet

I have a Flatlist inside the BottomSheet but I'm unable to scroll it. The Flatlist is defined inside CommentsScreen component.
This is code for the content rendering.

This is the code inside CommentsScreen component.

This is how the BottomSheet looks currently. There are 9 entries in my Flatlist but I can see only first 5.
How can I scroll down the Flatlist considering there may be more than 9 entries in future.
The background of BottomSheet is just a Webview.
Initial View.

After Scrolling.

same here.

check this out, worked out a lot better than the suggested solution above https://github.com/osdnk/react-native-reanimated-bottom-sheet/issues/95#issuecomment-683810936


Storage Explorer hangs on loading accounts and/or auth (MacOS)

Storage Explorer Version: 1.11.1
Build Number: 20191121.1
Platform/OS: macOS Catalina
Architecture: x64
Regression From:  Issue did not occur in 1.10.1, this happened during an upgrade (may be related)
Bug Description
Storage Explorer will hang while loading accounts (Screenshot below).

Steps to Reproduce
Happened during an upgrade then a restart of the computer. Couldn't exactly replicate it, but I moved the 1.10.1 version to trash, re-downloaded Azure Storage Explorer and added the application from temp back to Applications folder.
Steps to Fix
The only way I could get it to work was by checking Developer Tools identifying that Storage Explorer was getting an ECONNREFUSED from a value within ~/.ServiceHub/ (a folder that Storage Explorer writes to outside of its Application installed path).
Note: Use at your own risk, this is how I fixed my problem and may provide direction for bug fix.

Close Storage Explorer
Remove and delete all files with ~/.ServiceHub/
Open Storage Explorer, Storage Explorer recreates ~/.ServiceHub appropriately and the files load.

Expected Experience
Storage Explorer would not hang on load.
Actual Experience
Storage Explorer hung while loading accounts. Deleting application, re-downloading, and fresh install did not fix the issue.

I have the same problem that it hangs in the same position but this solution did not help. How can I figure out what Storage Explorer is trying to do to come closer to a solution?

Same issue here - deleting ~/.ServiceHub resolved it for me.  Can we get this resolution added to https://docs.microsoft.com/en-us/azure/storage/common/storage-explorer-troubleshooting?toc=%2Fazure%2Fstorage%2Fblobs%2Ftoc.json&tabs=Windows%2C1904#sign-in-issues while we wait for a proper fix? This would have saved me a pile of time.

I merged a fix from our auth library that fixes this problem. We're shipping 1.12 soon-ish so I'm going to hold off on updating the docs.


⚠ Jerry Savelle Ministries International Australia has degraded performance

In 5cbd5f5, Jerry Savelle Ministries International Australia (https://jsmiaustralia.org) experienced degraded performance:

HTTP code: 200
Response time: 5833 ms

Resolved: Jerry Savelle Ministries International Australia performance has improved in 835d6e2 after 6 minutes.


Added prime number Calculator

Fixes Issue🛠

Closes #1808
Description👨💻

Added a prime number calculator which identifies whether the given number is prime or not
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100 can you please merge this PR ?

@rugved0102  Add your calc in main index.html file of the project in alphabetical order.

done with the changes can you plz check once ?


"Failed to copy "/workspace/NewMask-Layout/build/compile_commands.json" to ".": {2}"

In the configuration stage, it shows error "Failed to copy "/workspace/NewMask-Layout/build/compile_commands.json" to ".": {2}"

Operating System:  Ubuntu
CMake Version: 3.7.3
VSCode Version:  1.45.3
CMake Tools Extension Version: 1.40.0
Compiler/Toolchain: GCC 9

Who is attempting to copy compile_commands.json? Do you have a task?
Is this a regression with 1.4.0? When did this last work for you, with what previous release?
The CMake Tools extension generates this file as one alternative for getting IntelliSense from CppTools extension but the preferred way is to set in the workspace .vscode\settings.json:    "C_Cpp.default.configurationProvider": "ms-vscode.cmake-tools". If you are not already using that, try it and let us know if you need anything else in your workflow.

Thanks for the information.  Yes, it is the latest CMake-tool version of 1.4.0. I made a typo.
I am using clangd for the IntelliSense, which I think need to copy the compile_commands.json to the root of the workspace.  Hence, I set the option "Cmake: Copy Compile Commands" and set to the directory of my workspace.  However, it seems cmake-tools attempts to copy the compile_commands.json to the right path but fails every time. So I have to manually copy this file after the cmake configuration stage.  I was wondering if you have any solution for this?
Thank you.

Same here, I'm trying to set up ccls or clangd.
OS: Ubuntu 16.04
$ cmake --version
cmake version 3.5.2

CMake suite maintained and supported by Kitware (kitware.com/cmake).
$ code --version
1.46.1
cd9ea6488829f560dc949a8b2fb789f3cdc05f5d
x64
$ code --list-extensions --show-versions | grep -F cmake-tools
<EMAIL_ADDRESS>
User settings:
    "cmake.copyCompileCommands": "${workspaceFolder}",

Who is attempting to copy compile_commands.json? Do you have a task?

I'm launching "CMake: Configure" from Command Palette.

Is this a regression with 1.4.0? When did this last work for you, with what previous release?

I haven't used VS Code for this project before. I believe it worked as expected once. Then I've opened the copy of compile_commands.json in VS Code to have a look, wasn't happy about its contents, tweaked cmake-kits.json, launched "CMake: Configure" command again and got the error:

Failed to copy "/home/developer/disks/foo/bar/build/clang/Release/compile_commands.json" to "/home/developer/disks/foo/bar": {2}

And it has been failing since then. I've tried to remove destination file assuming CMake Tools can't overwrite it for some reason - no success. VS Code restart did not help as well.
@andreeis, please let me know if I can provide more information.

Fixed.  One needs to specify the file name. See the setting below:
"cmake.copyCompileCommands": "${workspaceFolder}/compile_commands.json".

@yuanliuus, thank you for sharing the solution! Works for me. I hope you will keep this issue open, there is clearly some room for improvement (error message, parameter description).

It seems we could auto-add compile_commands.json to the path provided if the user specifies a folder.  Any objections to that behavior?  Is there a reason anyone would want to call the compile_commands.json file anything other than compile_commands.json?
The fix would go around here: https://github.com/microsoft/vscode-cmake-tools/blob/89b9ec8b12ee6d6138cc8865b14e6d0f0ae37d02/src/cmake-tools.ts#L646

It seems we could auto-add compile_commands.json to the path provided if the user specifies a folder. Any objections to that behavior?

No objections, sounds reasonable.

Is there a reason anyone would want to call the compile_commands.json file anything other than compile_commands.json?

I'm not aware of any. compile_commands.json is a convention. Many tools generating (e.g. CMake) and using (e.g. Clang-Tidy) compilation database don't even support alternative names. Some generators (e.g. Build EAR) can output differently named compilation database, but I am not aware if any consumer of compilation database that takes file name as argument, usually it's directory where compile_commands.json is located.


Don't shadow built-ins

https://github.com/apache/www-site/blob/9f860a8bcd6e5b0172857f6587e4a5ecb82a62eb/theme/plugins/asfgenid.py#L141
The variable 'id' shadows a built-in.
It should be renamed to avoid confusion

This is fixed with a PR to the master copy in infrastructure-pelican. This will be included in future buildsite process.


No matter how many bbox,

No matter how many bbox, the thermal map should be the same. Why is the thermal map of the whole image different for different objects?

Hi @xiaoma-ux, this is due to the gradient that is taken from the classification part of each object. Unlike classification networks, object detection models have classification nodes for each object alongside the bounding box and objectness probability notes. That's why the thermal map differs for each object.

hi@pooya-mohammadi，I mean the thermal map of the whole picture should be the same。There may be multiple BBoxes on a particular layer, but the thermal map of the entire image should be the same

@xiaoma-ux The code does not return or even compute the thermal map of the whole image. The thermal is computed for each bbox then combined together

So how do you calculate the thermal map of the whole picture? Can you tell me about it?

@xiaoma-ux I'm afraid there is no way because yolo model has no classification section for the whole image


feat: add programmatic api

Description
Working on cliff-jumper it irked me that I had to call git-cliff through npx. Looking at the code I figured I could easily add a small programmatic API on top of (or technically rather below 😂) the CLI one currently provided so here we are.
Other than the actual changes in the source code I also took the liberty to:

Update dev dependencies
Switch out regular child_process with execa, primarily for its promise-based API but also refer to the other advantages it offers (FWIW it's one of many Sindre's packages so it can be trusted to be stable and good)
Added tsup as bundler so separate CJS, ESM and CLI bundles can easily be generated

The whole split of CJS and ESM is something I could go into a lot more detail for but for brevity's sake I'll just keep it at that the NodeJS ecosystem is currently shattered between the two and for proper TypeScript support 2 bundles have to be provided, related for more reading: https://github.com/arethetypeswrong/arethetypeswrong.github.io

Updated from Yarn v1 to Yarn v4
Marked the package as being an ES module, thanks to the added export mapping and the split CJS/ESM bundles from point 3 this is now possible. It also allows the source code to use modern ESM code like import.meta.resolve instead of the old require.resolve, yay!
Added the export mapping, it's the modern way in a package.json to define all the entrypoints
Modernized the tsconfig, more yay for modern code.

Important: This sets the minimum NodeJS requirement to Node 18 as it is the lowest version that supports import.meta.resolve without an experimental flag (docs v16, docs v18, docs v20). In my opinion this is acceptable because Node 16 has been End of Life since 2023-09-11 and people should as minimum update to Node 18.
Motivation and Context

Having a programmatic API is always a nice-to-have for the more advanced users of the git-cliff npm package as it allows far easier scripting integration. The primary motivation here is my own library cliff-jumper which is one such "more advanced use".
How Has This Been Tested?

I have tested this by yarn linking the local repository to my cliff-jumper repository where I then refactored my code to use the programmatic API. Where before I had to parse out the options provided to cliff-jumper to proper CLI options and call npx I can now call the programmatic API. I have ran this code and it works perfectly:
const gitCliffOptions: GitCliffOptions = {
  tag: options.tagTemplate,
  prepend: './CHANGELOG.md',
  unreleased: true,
  config: './cliff.toml'
};

if (!isNullishOrEmpty(repositoryRootDirectory)) {
  gitCliffOptions.repository = repositoryRootDirectory;
  gitCliffOptions.includePath = `${options.packagePath}/*`;
}

const githubToken = getGitHubToken(options);
const githubRepo = getGitHubRepo(options);
if (!isNullishOrEmpty(githubRepo) && !isNullishOrEmpty(githubToken)) {
  const resolvedGitHubRepo = githubRepo === 'auto' ? `${options.org}/${options.name}` : `${githubRepo}`;

  gitCliffOptions.githubRepo = resolvedGitHubRepo;
  gitCliffOptions.githubToken = githubToken;
}

await runGitCliff(gitCliffOptions);

Screenshots / Logs (if applicable)
N.A.
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation (no code change)
[x] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
(I hope so anyway, I had VSCode auto format and it should have picked up editorconfig)
[x] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt. N.A. I haven't touched rust code
[x] I checked the lints with clippy ESLint.
[x] I have added tests to cover my changes. N.A.
[x] All new and existing tests passed.

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

Project coverage is 41.30%. Comparing base (110b9b8) to head (1ce72b9).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #523   +/-   ##
=======================================
  Coverage   41.30%   41.30%
=======================================
  Files          15       15
  Lines        1034     1034
=======================================
  Hits          427      427
  Misses        607      607

Flag
Coverage Δ

unit-tests
41.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Tested it on both MacOS and Windows now. Also with a not 3 AM brain which let me to make the last 2 commits lol

Hey this looks pretty good! I will test it locally soon.
I can also create a RC release just to test this functionality and see if everything works with NPM.

I tried running yarn && yarn dev inside npm/git-cliff directory and got this error:
file:///home/orhun/gh/git-cliff/npm/git-cliff/lib/index.js:3
Object.defineProperty(exports, "__esModule", { value: true });
                      ^

ReferenceError: exports is not defined in ES module scope
This file is being treated as an ES module because it has a '.js' file extension and '/home/orhun/gh/git-cliff/npm/git-cliff/package.json' contains "type": "module". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.
    at file:///home/orhun/gh/git-cliff/npm/git-cliff/lib/index.js:3:23
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)

Node.js v18.17.0

Whereas this was working before like this:
$ yarn dev
yarn run v1.22.21
$ yarn build && node lib/index.js
$ tsc
 WARN  git_cliff > "cliff.toml" is not found, using the default configuration.

@favna any idea what's going on?

@orhun Fixed it. I overlooked the dev script calling the cli so I just had to adjust the path.

Hm I do still get an error but I'm not sure if that's related the programmatic change?
 WARN  git_cliff > "cliff.toml" is not found, using the default configuration.
 ERROR git_cliff > Git error: `could not find repository at 'F:\favware\git-cliff\npm\git-cliff'; class=Repository (6); code=NotFound (-3)`
file:///F:/favware/git-cliff/npm/git-cliff/node_modules/execa/lib/error.js:60
                error = new Error(message);
                        ^

Error: Command failed with exit code 1: F:\favware\git-cliff\npm\git-cliff\node_modules\git-cliff-windows-x64\bin\git-cliff.exe
    at makeError (file:///F:/favware/git-cliff/npm/git-cliff/node_modules/execa/lib/error.js:60:11)
    at handlePromise (file:///F:/favware/git-cliff/npm/git-cliff/node_modules/execa/index.js:124:26)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async run (file:///F:/favware/git-cliff/npm/git-cliff/lib/cli/cli.js:60:25) {
  shortMessage: 'Command failed with exit code 1: F:\\favware\\git-cliff\\npm\\git-cliff\\node_modules\\git-cliff-windows-x64\\bin\\git-cliff.exe',
  command: 'F:\\favware\\git-cliff\\npm\\git-cliff\\node_modules\\git-cliff-windows-x64\\bin\\git-cliff.exe',
  escapedCommand: '"F:\\favware\\git-cliff\\npm\\git-cliff\\node_modules\\git-cliff-windows-x64\\bin\\git-cliff.exe"',
  exitCode: 1,
  signal: undefined,
  signalDescription: undefined,
  stdout: undefined,
  stderr: undefined,
  cwd: 'F:\\favware\\git-cliff\\npm\\git-cliff',
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}

Node.js v20.11.0

I'm getting a different error:
file:///home/orhun/gh/git-cliff/npm/git-cliff/lib/cli/cli.js:22
    throw new Error(
          ^

Error: Couldn't find git-cliff binary inside node_modules for linux-x64
    at getExePath (file:///home/orhun/gh/git-cliff/npm/git-cliff/lib/cli/cli.js:22:11)
    at runGitCliff (file:///home/orhun/gh/git-cliff/npm/git-cliff/lib/cli/cli.js:48:25)
    at run (file:///home/orhun/gh/git-cliff/npm/git-cliff/lib/cli/cli.js:60:31)
    at file:///home/orhun/gh/git-cliff/npm/git-cliff/lib/cli/cli.js:64:6
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)

Node.js v18.17.0

However, I can confirm that the binary exists:
$ file node_modules/git-cliff-linux-x64/bin/git-cliff
node_modules/git-cliff-linux-x64/bin/git-cliff: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 2.6.32, BuildID[sha1]=7bc79cc471a2b79104849701c87fcc2ebcb76add, stripped

I applied this patch:
diff --git a/npm/git-cliff/src/getExePath.ts b/npm/git-cliff/src/getExePath.ts
index f1bbbd6..341087f 100644
--- a/npm/git-cliff/src/getExePath.ts
+++ b/npm/git-cliff/src/getExePath.ts
@@ -28,7 +28,7 @@ export async function getExePath() {
     );
   } catch (e) {
     throw new Error(
-      `Couldn't find git-cliff binary inside node_modules for ${os}-${arch}`
+      `Couldn't find git-cliff binary inside node_modules for ${os}-${arch} (${e})`,
     );
   }
 }

The error is:
Error: Couldn't find git-cliff binary inside node_modules for linux-x64 (TypeError: (intermediate value).resolve is not a function)

My IDE also complains about the await import.meta.resolve line:

typescript: 'await' has no effect on the type of this expression.

I was under the impression that import.meta.resolve returns a Promise but apparently it does not so the IDE was correct here.
I'll see if I can reproduce your issue. So far I've only tried with MacOS and Windows on Node 20. I'll try with Node 18.17.0 (your version as seen above) on both and leverage one of my VPSs for Linux x64 as well. I'll report back.

OS
Arch
Node version
Result of import.meta.resolve
Result

Windows 11 Pro
x86_64
v18.17.0
TypeError: (intermediate value).resolve is not a function
❌

Windows 11 Pro
x86_64
v18.19.1
file:///F:/favware/git-cliff/npm/git-cliff/node_modules/git-cliff-windows-x64/bin/git-cliff.exe
✅

Windows 11 Pro
x86_64
v20.11.1
file:///F:/favware/git-cliff/npm/git-cliff/node_modules/git-cliff-windows-x64/bin/git-cliff.exe
✅

MacOS
arm64
v18.17.0
TypeError: (intermediate value).resolve is not a function
❌

MacOS
arm64
v18.19.1
file:///Users/favna/workspace/favware/git-cliff/npm/git-cliff/node_modules/git-cliff-darwin-arm64/bin/git-cliff
✅

MacOS
arm64
v20.11.1
file:///Users/favna/workspace/favware/git-cliff/npm/git-cliff/node_modules/git-cliff-darwin-arm64/bin/git-cliff
✅

Debian GNU/Linux 11
x86_64
v18.17.0
TypeError: (intermediate value).resolve is not a function
❌

Debian GNU/Linux 11
x86_64
v18.19.1
file:///root/workspace/git-cliff/npm/git-cliff/node_modules/git-cliff-linux-x64/bin/git-cliff
✅

Debian GNU/Linux 11
x86_64
v20.11.1
file:///root/workspace/git-cliff/npm/git-cliff/node_modules/git-cliff-linux-x64/bin/git-cliff
✅

I think the pattern is fairly obvious. Looks like there is a bug in Node 18.17.0 which is fixed by 18.19.1 lol. Turns out you'll just have to update to the latest minor, or go straight to 18 then.

Thanks for the debugging this - everything works as expected now after upgrading node!


segmentation fault (core dumped)

I am getting an error of segmentation fault(core dumped) while using kenlm within the ctc beam search decoder.

I am currently having the same issue as well.

@arpit601 @lscaria hi have you guys found a way to fix this problem?

I have a custom TensorFlow Op that runs on TF 1.8 and ran into a similar issue. Please find the details  here https://github.com/kpu/kenlm/issues/154#issuecomment-386415120


output of chcp was not parsed correctly

On Windows, when wrapping a batch command with the function s:wrap_cmds(),
when calling 'chcp' to get the current code page, the code assumed that
the output was in the format "active code page: XXX" (where XXX is the
code page), whereas the actual output is localized (for instance, in
French, the output would be: "page de code active: XXX").
The parsing of the output relied on that, and this failed for a
message different from "active code page" (i.e., English).
This patch changes the parsing to split the output of chcp on the colon
instead of spaces. Assuming that the output is always in the format
"localized message: XXX", regardless of the locale, hopefully, this is
a bit more robust.

Thanks @gh4w for the fix and using deleayexpansion.


Another one struct code typo?

Repeated struct member. This page.

Fixed. Thanks for reporting this!


Adds staking pool info

a bit confused, took a look at the stake pool addresses specified on the config, both addresses are the same and they do not correspond to an valid stake pool address https://stake.cardinal.so/8Da8qPqSwc2DNenp5wwK7HWxpSrvxw8LUNoBZ2gAYjazz


Fix isTokenUtxo to non SLP transactions

Hi :)
I found that SLP.Utils.isTokenUtxo does not handle validations like SLP.Utils.tokenUtxoDetails. More specifically, the following code of SLP.Utils.isTokenUtxo does not handle null cases of x:
validations = validations.map((x: any) => x.valid)

I think that it should be done like in https://github.com/Bitcoin-com/slp-sdk/blob/master/src/Util.ts#L462,
correct?
By the way, thanks for your effort here at slp-sdk, it is a really helpful library!

Thanks for the PR! It looks like a great addition to the repository, and mirrors a similar PR I just created for tokenUtxoDetails() a couple days ago.
And thanks for alerting us to the issue. We just caught it ourselves.
The only thing we need to do before accepting the PR is for the Developer Services team to review the implications to SLP-aware wallets interacting with isTokenUtxo() if we merge this PR. SLPDB returns null for a BCH-only TXID AND it returns null for SLP transactions that it hasn't processed yet. That's the main thing we need to take into consideration.

Going to merge this PR. Thank you @thr0wn!
After this PR is merged, I'll also refactor this function to match the refactoring done to tokenUtxoDetails() in PR #86. The new behavior of this function will return confident false or true. It will only return null if SLPDB has not processed the UTXO yet and has not yet made a determination of its validity.

Created Issue #87 to capture future work.


Support custom HTTP headers in DaprClient

Describe the proposal
I've got a scenario where Service A invokes a method on Service B and Service B requires an Authorization HTTP header to be sent. Example:
await _daprClient.InvokeMethodAsync<BasketData>(
    "serviceB",
    $"api/v1/basket/{id}",
    new HTTPExtension { Verb = HTTPVerb.Get });

There's currently no way to pass the header to the DaprClient. Ideally, I'd want to be able to do something like this:
Dictionary<string, string> headers = new Dictionary<string, string>
{
    ["Authentication"] = "Bearer ..."
};

await _daprClient.InvokeMethodAsync<BasketData>(
    "serviceB",
    $"api/v1/basket/{id}",
    new HTTPExtension { Verb = HTTPVerb.Get, Headers = headers });

I've looked a bit further into this and it should not be hard to implement. We just need to pass any header information as gRPC metadata to the sidecar. I'll try and create a PR.

Somewhat related, could there also be a way of extracting HTTP headers sent by an application with the response?  For example, if the response was an opaque set of bytes, the Content-Type response header might be needed in order to differentiate between multiple types of data.  Something of this sort:

class HttpResponseExtension
{
   IDictionary<string, string> Headers { get; }
}

class MethodResponse<TResponse>
{
    TResponse Content { get; }

    HttpResponseExtension Metadata { get; }
}

class DaprClient
{
    abstract Task<MethodResponse<TResponse>> InvokeMethodWithResponseAsync<TRequest, TResponse>(string appId, string methodName, HTTPExtension requestMetadata, CancellationToken);
}

@philliphoff The service that's being called may be an HTTP service, but it could also be a gRPC service. So I'd do it slightly different:
class MethodResponse<TResponse>
{
    TResponse Content { get; }
}

class HttpResponse<TResponse> : MethodResponse<TResponse>
{
    IDictionary<string, string> Headers { get; }
}

class DaprClient
{
    abstract Task<MethodResponse<TResponse>> InvokeMethodAsync<TRequest, TResponse>(string appId, string methodName, HTTPExtension requestMetadata, CancellationToken);
}

Alternatively, at that point it may even be better to make separate invoke methods for Http and gRPC, like InvokeHttpMethod and InvokeGrpcMethod. The abstraction is already a bit leaky due to the need for the HttpExtension object.
May I suggest opening a new issue for response headers? I'd like to keep this issue/PR focused on the request side.

@amolenk May I suggest opening a new issue for response headers? I'd like to keep this issue/PR focused on the request side.

Opened #400.

@vinayada1 Can you review? This is ready to be merged.


怎样删除左上角的应用名

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Please write in English.
Please fill in the issue details properly.
Also, from the machine translation of the title, the question is not specific to this package, so please solve it yourself using Google search or ask on Stack Overflow.


[5.5] Add support for MultiSubnetFailover DSN parameter

configures Microsoft Drivers for PHP for SQL Server to provide faster detection of and connection to the (currently) active server when connecting to a SQL Server 2012 availability group listener or SQL Server 2012 Failover Cluster Instance.

How many options does SQL Server have? 😬

Many!
https://docs.microsoft.com/en-us/sql/connect/php/connection-options
Would it be worth refactoring this section to account for all of them?


Master Overall

Maybe you can put this value to scrapying?

Yes, I have added this feature in commit 47fc2bea5ca412d9d3643c90f11ba841a92a2bc7. That attribute has a key "OVR" in the player database. Thanks.


fix(indent): correct indentation in chained methods calls with generics

Description

Linked Issues
Closes #270
Additional context

Problem is partially resolve but other cases appears
I update my sample: https://stackblitz.com/edit/github-caheqy-gwabam?file=src%2Findex.ts
We still have an error: it only works for the top level items.
Inside a class for example we have the indentation bad lint

Problem is partially resolve but other cases appears
I update my sample: https://stackblitz.com/edit/github-caheqy-gwabam?file=src%2Findex.ts
We still have an error: it only works for the top level items.
Inside a class for example we have the indentation bad lint

Thanks for your reply, I will fix it soon.

Problem is partially resolve but other cases appears
I update my sample: https://stackblitz.com/edit/github-caheqy-gwabam?file=src%2Findex.ts
We still have an error: it only works for the top level items. Inside a class for example we have the indentation bad lint

Hello, sorry for reply latly.
当然，以下是这段话的英文翻译：

I carefully reviewed the cases you provided and found that they don't seem to be errors. If the type declarations of the methods are removed, it looks like this:
categorySelector
    = createSelector(
      this.sliceSelector.bind(this),
      (slice) => slice.category
    );

The CallExpression of the function is at the same indentation level as the =.
Btw, we can create a new issue so that more people can join the discussion.

Problem is partially resolve but other cases appears
I update my sample: https://stackblitz.com/edit/github-caheqy-gwabam?file=src%2Findex.ts
We still have an error: it only works for the top level items. Inside a class for example we have the indentation bad lint

I carefully reviewed the cases you provided and found that they don't seem to be errors. If the type declarations of the methods are removed, it looks like this:
categorySelector
    = createSelector(
      this.sliceSelector.bind(this),
      (slice) => slice.category
    );

The CallExpression of the function is at the same indentation level as the =.
Btw, we can create a new issue so that more people can join the discussion.


Added form UI support for editable artifact upload size

Pull Request Description
This pull request solves part of Strongbox/Strongbox#1819
Acceptance Test

[x] Building the code with ng build still works.
[ ] Building the code with ng test still works.
[x] Building the code with ng e2e still works.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[x] No

Does this pull request require other pull requests to be merged first?

[x] Yes, please see Strongbox/Strongbox#1819
[ ] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[x] No

Code Review And Pre-Merge Checklist

[x] My code follows the coding convention of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code in hard-to-understand areas.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.

Did you close this by accident ? :)


Assess correctness of derivation of seeds from mnemonic

I was surprised to see that we were using the hkdf library to transform the mnemonic seed into the LDK and on-chain wallet seeds. Furthermore, we weren't sure about the way we appear to be "tagging" the mnemonic seed to generate LDK and on-chain seeds.

@luckysori this ticket seems to be important. What is the issue with using the hkdf lib?

Can't say I remember what I had in mind when I wrote this ticket 5 months ago 🤷 I'll assign myself and add the grooming label for now.

Can we define what we want to achieve with this ticket? Correctness can have multiple meanings and different wallets implement different derivation paths.
My understanding is that the goal is to import the seed phrase in a different wallet and recover the on-chain funds.
We should be aware that not all wallets support the same derivation approach, e.g. I've read that the mnemonic phrase of electrum is not compatible with a bip39 seed phrase.
I guess a good start would be testing against bip39 test vectors.
I gave it a try in this branch.
As one can see, we are not compatible with the test vectors.


Add support for CXF interceptors

Or is CXF using Apache HttpComponents and we can just use #15 ?

CXF supports the Apache HttpComponents HttpClient as a transport: http://hc.apache.org/httpcomponents-client-4.5.x/index.html
So this can supported by just fixing #15


Multi-transaction BatchMutationDatastoreDAO

Bulk updates to Datastore are excruciatingly slow with 25-record transactions in series. This change add support for a configurable number of multiple parallel write transactions. Of course, the config parameter should be set to 1 if the order of mutations matters.

@jacksonic did you get a chance to look at this when you looked at the transactional IDBDAO? The core of the logic is the same.


Remove nightly FMT

This removes all the nightly fmt features, so we will be formatting with default fmt on stable
closes #317

I suggest updating the Dockerfile to remove the +nightly mentions


⚠ 2 Minutes de génie (staging) has degraded performance

In 38c9867, 2 Minutes de génie (staging) (https://2mdgstaging.gitlab.io/) experienced degraded performance:

HTTP code: 502
Response time: 3181 ms

Resolved: 2 Minutes de génie (staging) performance has improved in f910ef5.


Sudden breaking API changes

Describe the bug:
Spotify have made sudden breaking changes to their API, depreciating many endpoints (making them unavailable to many current users)
News: https://developer.spotify.com/blog/2024-11-27-changes-to-the-web-api
Related Issue on Spotipy library: https://github.com/spotipy-dev/spotipy/issues/1172

Fixed by using Spotipy Anon library to read/authenticate and copy Spotify owned libraries.


Update query_events version to 3.1

@jchristn, I was talking to @ebob9 today and came to know version 3.1 is available now with the acknowledgment feature. Please help to update the version.

@jchristn thank you for working on it. My use case is to get the events that have not been acknowledged but the ability to acknowledge the event is helpful too. When do you plan to make a production release?

@sumitgoelpw yes - you'll be able to add include_acknowledged: false in the query to retrieve events that are not yet acknowledged.  I'll get this packaged up today and submitted to npm.


HSEARCH-3440 Finalize the lucene DirectoryProvider SPI

https://hibernate.atlassian.net//browse/HSEARCH-3440
~Based on #2036 , which should be merged first.~ => Done

Thanks, I addressed most of your comments and rebased. Waiting for CI, then I'll merge.


Squirrel!

I am a great tracker.
My pack sent me on a special mission, all by myself.
Have you seen a bird? I am going to find one, and I am on the scent.
I am a great tracker; did I mention that?

+1


Issues with the Issues panel: link "Issues with other features:"

Here "Issues with other features: 14" should be hyperlinked to some place where the user can see those 14 features.

Or else he perhaps will think "they are probably referring to those 14 items below."
Anyway he perhaps thinks "the school principal says I have 14 demerits, but won't let me know what they are." (Even if not related to him.)

Nah, I think the target audience for iD are persons with IQ over 90 or so. We wouldn't like others to edit the map anyway; or at least, not to try fixing the map issues created by others.

I suspect it is because you only have "my edits" ticked - change that to "everything" and all will become clear

not to try fixing the map issues created by others

Really? People do it all the time. Otherwise, nothing would ever get fixed.

Closing here.  I think people can figure out that by playing with the radio buttons, they'll see different things here.
@DujaOSM Please be nice.  iD is for everyone.

@bhousel  Acknowledged, with an apology.

I see.
One would have never expected it to check things even that

And in fact "Everything" could be changed to "All edits" or "Everybody's edits".
And "Everywhere" should be changed to "Nearby" or "Within 14km" else it
means "for the whole world."

"Everything" and "Everywhere" are purposefully encompassing so users can be sure that no loaded features are being left out.

OK but it would still be good to know how far it is checking.

OK but it would still be good to know how far it is checking.

There's no set distance. "Everywhere" checks the tiles that were downloaded since starting iD, so whatever places you've viewed. If you edit in Japan and then start editing in the US, the issues you encountered in Japan will still display.

OK. I would have never ever guessed that.
One usually assumes some radius rectangle.
So perhaps a "?" or "i" information button could explain that.
Else 100% of users would never guess that.

So perhaps a "?" or "i" information button could explain that.
Else 100% of users would never guess that.

Advanced mappers could deduce this or ask us. Casual mappers (to the extent they even know about this feature) should never have to care—it's intuitive that unseen data is ignored.


[SPARK-1600] Refactor FileInputStream tests to remove Thread.sleep() calls and SystemClock usage

This patch refactors Spark Streaming's FileInputStream tests to remove uses of Thread.sleep() and SystemClock, which should hopefully resolve some longstanding flakiness in these tests (see SPARK-1600).
Key changes:

Modify FileInputDStream to use the scheduler's Clock instead of System.currentTimeMillis(); this allows it to be tested using ManualClock.
Fix a synchronization issue in ManualClock's currentTime method.
Add a StreamingTestWaiter class which allows callers to block until a certain number of batches have finished.
Change the FileInputStream tests so that files' modification times are manually set based off of ManualClock; this eliminates many Thread.sleep calls.
Update these tests to use the withStreamingContext fixture.

@JoshRosen I did a pass and I had some comments. Overall it looks pretty good. Addressing those comments should make it good enough.
Could you tell me how much is the speed up by converting the unit tests from real clock to manual clock?

Test build #25052 has started for   PR 3801 at commit b4442c3.

This patch merges cleanly.

Pushed some commits addressing most of the feedback, but I'm still struggling to remove that last Thread.sleep(1000).  I think that the problem here is that the writing of the checkpoint is asynchronous and without the sleep, we wind up in a state where batch 3 has started processing but has not finished, and the StreamingContext shuts down before a snapshot including batch 3's file info is written.  I plan to dig into this tomorrow to see whether this is actually the case.

Test build #25052 has finished for   PR 3801 at commit b4442c3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25052/
Test FAILed.

Right, that is probably the case. The checkpointing is asynchronous, and there isnt any callback hook for that, which we can use to wait on. We could wait for the expected checkpoint file to appear in the directory. That will make it deterministic.

Test build #25071 has started for   PR 3801 at commit 8340bd0.

This patch merges cleanly.

Test build #25071 has finished for   PR 3801 at commit 8340bd0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25071/
Test PASSed.

Test build #25086 has started for   PR 3801 at commit e4494f4.

This patch merges cleanly.

I am merging this. Thanks @JoshRosen for this humongous effort!


Unable to install vue cli

Version
2.6.11
Reproduction link
https://github.com/vuejs/vue-cli/issues/5181
Steps to reproduce
Vue global installation fails with 404 Not Found - GET https://registry.npmjs.org/error-ex
What is expected?
Install vue cli globally
What is actually happening?
 You can ask questions on the forum, the Discord server or StackOverflow.


Access "config" attribute of a job

Is it possible to access the "config" attribute of a job?
"config" is listed on this page as one of the attributes of "job": https://developer.travis-ci.com/resource/jobs
It seems support was added for it a while back via https://github.com/shuheiktgw/go-travis/commit/81f36c5bc2f9c0441bfb66ab10e728e01fd53050#diff-b91d5620c6bf557e598604750ebe0062 but it seems to have been dropped.
For example when running the following query each job of the response includes a config attribute as shown below
$ curl -H "Travis-API-Version: 3" \
-H "User-Agent: API Explorer" \
-H "Authorization: token <token>" \
'https://api.travis-ci.org/builds?limit=1&include=build.jobs%2Cjob.config'

"config": {
	"os": "linux",
	"dist": "xenial",
	"sudo": true,
	"group": "stable",
	"stage": "Test",
	"python": "3.8-dev",
	"script": [
		"make tests"
	],
	".result": "configured",
	"install": [
		"make install"
	],
	"language": "python",
	"global_env": "GITHUB_TOKEN=[secure]",
	"notifications": {
	        "email": false
        }
}

Thanks!

Thanks for reporting the issue! 😄
I'll work on this when I have time, but if someone else want to take it, please feel free to do so.

Thanks!


[TESTS] Make Helm version in bats test renovatable

Currently the bats tests for the helm tool are hardcoded and will not be versioned by Renovate.
We should enable all version in the bats tests to be automatically versioned by Renovate.
This probably requires a custom regex manager in the containerbase/.github repository.
Outcome of #300

Maybe we can resume the shell script regex manager config here.

I created https://github.com/containerbase/.github/pull/8 for that.
Should be straight forward as it is basically the same


fix(classes): fix code format when adding new pocket device

:tada: This PR is included in version 2.12.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add Apache SkyWalking to the ecosystem list.

SkyWalking has had native OpenTelemetry's OTLP support fo months.

Is SkyWalking a vendor or an OSS project you can self host?

OSS project.

Hmm, in this case I think it makes more sense to list it under the Ecosystem page, since vendors are typically products that you pay to use.

I can see the preview build fails due to
9:56:24 AM: [warn] content-modules/opentelemetry-go/website_docs/exporters.md
9:56:24 AM: [warn] content-modules/opentelemetry-go/website_docs/getting-started.md
9:56:24 AM: [warn] content-modules/opentelemetry-go/website_docs/libraries.md
9:56:24 AM: [warn] content-modules/opentelemetry-go/website_docs/manual.md
9:56:29 AM: [warn] Code style issues found in 4 files. Forgot to run Prettier?

But it seems not from my changes. Is something broken for main branch?

Yeah, we added formatting to CI and it affects existing PRs. If you run npm run prettier:write and commit the changes it should go green.

Those files belong to another repo(git submodule to opentelemetry-go). I bumped up the submodule(telemetry-go), it should be good now.

I think this time is good.

Please remove the changes to content-modules/opentelemetry-go, if files there are breaking the workflow we need to fix this on our end. cc @chalin

So, you want me to revert sub module change and keep CI fails? Or I need to wait for another fix? @svrnm

So, you want me to revert sub module change and keep CI fails? Or I need to wait for another fix? @svrnm

Yes, individual PRs should not touch the content modules, if the CI still fails, we need to give this a separate look.

Hi. I believe that the source of the problem is that you merged from main earlier. What you need to is is rebase from main. Give that a try and let us know how it works out (it can be a bit trick to recover from a merge).

I think I make the submodule back to the current main branch. Now it works. Thanks for your help.

Yes, good progress! But this PR still isn't a clean rebase because it contains changes to the community repo:

Please fix that as well.

Yes, good progress! But this PR still isn't a clean rebase because it contains changes to the community repo:

Please fix that as well.

Reverted.

looks good to me 👍

@svrnm, FYI, it seems that you merged as means of updating the branch:

Generally speaking, it's important to rebase, not merge:

Why? Well, we've had a number of PRs get into muddled states because of merges.
I've enabled "Require linear history" for the main branch. Hopefully that'll cause the "merge" option to be excluded. We can discuss further during today's Comms meeting if you feel the need.
Thanks!


rosbridge_suite: 1.3.2-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository rosbridge_suite to 1.3.2-1:

upstream repository: https://github.com/RobotWebTools/rosbridge_suite.git
release repository: https://github.com/ros2-gbp/rosbridge_suite-release.git
distro file: rolling/distribution.yaml
bloom version: 0.11.2
previous version for package: 1.3.1-2

rosapi
* Fix ROS2 CI for iron & rolling (#875 <https://github.com/RobotWebTools/rosbridge_suite/issues/875>)
* Contributors: Hans-Joachim Krauch

rosapi_msgs

No changes

rosbridge_library
* Interpret call_services_in_new_thread as boolean (#857 <https://github.com/RobotWebTools/rosbridge_suite/issues/857>)
* Add option to call services in a separate thread (#847 <https://github.com/RobotWebTools/rosbridge_suite/issues/847>)
* fix memory leak on service call (#774 <https://github.com/RobotWebTools/rosbridge_suite/issues/774>)
* Contributors: Sebastian Castro, hiroyuki obinata, rwhitney456

rosbridge_msgs

No changes

rosbridge_server
* Add option to call services in a separate thread (#847 <https://github.com/RobotWebTools/rosbridge_suite/issues/847>)
* Contributors: Sebastian Castro

rosbridge_suite
* rosbridge_suite: depend on ament_cmake (#848 <https://github.com/RobotWebTools/rosbridge_suite/issues/848>)
* Contributors: Jochen Sprickerhof

rosbridge_test_msgs

No changes

Holding for an upcoming Rolling sync


[BUG]: Stucked at [BackgroundWorkerService] Feature is not enabled and will not be started.

How are you running AnythingLLM?
Docker (remote machine)
What happened?
System: Ubuntu 22.04 arm64
Running other services like mongodb (port 27017), ollma (port 11434), nginx(port 80, 443) etc.
Issue:
When I start with sudo docker run mintplexlabs/anythingllm:latest or with the following options --add-host=host.docker.internal:host-gateway, it seems running correctly like the following message showed. But the message stucked at  [BackgroundWorkerService] Feature is not enabled and will not be started.
Checking with docker status by docker ps -a, it shows STATUS  Up 2 hours (healthy) but PORTS are empty.

[collector] info: Collector hot directory and tmp storage wiped!
[collector] info: Document processor app listening on port 8888
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
✔ Generated Prisma Client (v5.3.1) to ./node_modules/@prisma/client in 395ms
Start using Prisma Client in Node.js (See: https://pris.ly/d/client)
'''
import { PrismaClient } from '@prisma/client'
const prisma = new PrismaClient()
'''
or start using Prisma Client at the edge (See: https://pris.ly/d/accelerate)
'''
import { PrismaClient } from '@prisma/client/edge'
const prisma = new PrismaClient()
'''
See other ways of importing Prisma Client: http://pris.ly/d/importing-client
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": SQLite database "anythingllm.db" at "file:../storage/anythingllm.db"
SQLite database anythingllm.db created at file:../storage/anythingllm.db
23 migrations found in prisma/migrations
......
└─<PHONE_NUMBER>4346_init/
└─ migration.sql
└─<PHONE_NUMBER>5625_init/
└─ migration.sql
└─<PHONE_NUMBER>5054_init/
└─ migration.sql
All migrations have been successfully applied.
┌─────────────────────────────────────────────────────────┐
│  Update available 5.3.1 -> 5.19.1                       │
│  Run the following to update                            │
│    npm i --save-dev prisma@latest                       │
│    npm i @prisma/client@latest                          │
└─────────────────────────────────────────────────────────┘
[backend] info: [EncryptionManager] Self-assigning key & salt for encrypting arbitrary data.
[backend] info: [TELEMETRY ENABLED] Anonymous Telemetry enabled. Telemetry helps Mintplex Labs Inc improve AnythingLLM.
[backend] info: prisma:info Starting a sqlite pool with 17 connections.
[backend] info: [TELEMETRY SENT] {"event":"server_boot","distinctId":"89988ae1-940a-4088-892a-07df1b673dc4","properties":{"commit":"--","runtime":"docker"}}
[backend] info: [CommunicationKey] RSA key pair generated for signed payloads within AnythingLLM services.
[backend] info: [EncryptionManager] Loaded existing key & salt for encrypting arbitrary data.
[backend] info: Primary server in HTTP mode listening on port 3001
[backend] info: [BackgroundWorkerService] Feature is not enabled and will not be started.
Are there known steps to reproduce?
No response

[BackgroundWorkerService] Feature is not enabled and will not be started.

This is a normal message. Its just a log for a beta preview feature. The server appears to boot fine

[backend] info: Primary server in HTTP mode listening on port 3001

You should probably use the official command with your edit in it for the host gateway
export STORAGE_LOCATION=$HOME/anythingllm && \
mkdir -p $STORAGE_LOCATION && \
touch "$STORAGE_LOCATION/.env" && \
docker run -d -p 3001:3001 \
--cap-add SYS_ADMIN \
--add-host=host.docker.internal:host-gateway \
-v ${STORAGE_LOCATION}:/app/server/storage \
-v ${STORAGE_LOCATION}/.env:/app/server/.env \
-e STORAGE_DIR="/app/server/storage" \
mintplexlabs/anythingllm

[BackgroundWorkerService] Feature is not enabled and will not be started.

This is a normal message. Its just a log for a beta preview feature. The server appears to boot fine

[backend] info: Primary server in HTTP mode listening on port 3001

You should probably use the official command with your edit in it for the host gateway
export STORAGE_LOCATION=$HOME/anythingllm && \
mkdir -p $STORAGE_LOCATION && \
touch "$STORAGE_LOCATION/.env" && \
docker run -d -p 3001:3001 \
--cap-add SYS_ADMIN \
--add-host=host.docker.internal:host-gateway \
-v ${STORAGE_LOCATION}:/app/server/storage \
-v ${STORAGE_LOCATION}/.env:/app/server/.env \
-e STORAGE_DIR="/app/server/storage" \
mintplexlabs/anythingllm

Could you help me with this 'HTTP mode listening on port 3001' situation?
netstat -tuln | grep :3001 lsof -i :3001 show nothing and I think my port 3001 is available. Even if I change the port number, like -d -p 8000:3001 this message keeps showing up.

If you run that command without the --add-host does it boot up fine? It should bind to whatever parent port you like assuming that the port is available, otherwise the container will just shut down

If you run that command without the --add-host does it boot up fine? It should bind to whatever parent port you like assuming that the port is available, otherwise the container will just shut down

Thank you so much for your help.
Unfortunately, it suffers from the same problem that says, "Primary server in HTTP mode listening on port 3001" with or without adding --add-host AND/OR assign a host port by using -p option.

If you run that command without the --add-host does it boot up fine? It should bind to whatever parent port you like assuming that the port is available, otherwise the container will just shut down

Thank you so much for your help. ~Unfortunately, it suffers from the same problem that says, "Primary server in HTTP mode listening on port 3001" with or without adding --add-host AND/OR assign a host port by using -p option.~
Issue Fixed It was my environmental setting issue. Thank you so much for helping

i have the same problem, how can i fix it ?


feat: integrate scope

New Features:

`configureScope { ... }
captureMessage(message, ScopeCallback)
captureException(throwable, ScopeCallback)
and handful of tests

Scope options:

Breadcrumb
Level
Context
Tags
User
Extras

The most important class is SentryScope in the Android and iOS module which modifies the platform scopes directly and the SentryAndroid & SentryApple classes.
Other than that it doesn't contain much logic, it is mostly wrapping data around

@marandaneto I've refactored a lot of things, hopefully it looks a bit better now. I've also tested the functionality manually and it works so far on Android and iOS (I also added more tests)


SAK-44602 - embed calendar broken in master

Issue introduced in https://github.com/sakaiproject/sakai/pull/8754
Used ContentHostingService to check the availability of all Lesson item types (so this affects more than calendar).

In the switch cases, are there any other types we should add? MULTIMEDIA for instance? I'm unsure if these are backed by resources or not

Hello everyone;
First of all, I apologize for the inconvenience my pull request caused. I've been reviewing the case and the mistake was applying the filter regardless of the lesson content type.
So, based on my tests, I think the only content that should reach the "continue" and not be shown is type 7, multimedia. In the other cases contentHostingService.isAvailable () causes errors, which was the reason why the items did not appear on the screen.
My suggestion is that the valid code in ShowPageProducer.java is as follows:

And this way calendars and other items can be seen without difficulties.

Thank you.
@bbailla2
@bjones86

Original code was reverted, I expect the solution will be reimplemented in SAK-44571


[SQL Migration] Revert dependency on new Azure Core API changes

Initially, we had an issue reported where the SQL Migration extension did not work for customers in USGov. Upon further investigation, it was determined that this was due to a bug in the Azure Core API (which the SQL Migration extension relies on) which incorrectly assumed that all customers were in the public cloud.
So, in a previous PR (https://github.com/microsoft/azuredatastudio/pull/20304), we added a new getProviderMetadataForAccount function to the Azure Core API, which fixed the issue, and then we started consuming this new function all over the SQL Migration extension.
In its current state, the extension works properly on insiders, as well as on local dev builds of ADS. However, it does not work in any production version of ADS, past or present, including the v1.39 version which was just released recently. This is because the aforementioned changes to Azure Core did not actually end up making it into the product release on time. Therefore, this getProviderMetadataForAccount functionality that the SQL Migration extension now depends on is not actually available yet.
This PR removes the dependency on this functionality in the SQL Migration extension, allowing the extension to work in production versions of ADS.
This of course means we're back where we started in the sense that the extension will still not work for USGov/non-public cloud customers, which was the point of the fix to begin with, but this can still be worked around by directing customers to use insiders with a private build of the extension.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.634%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27875

Relevant Lines:
62380

💛  - Coveralls


neo4j: 3.5.14 -> 4.3.3

This is a rebase and continuation of https://github.com/NixOS/nixpkgs/pull/98443.
Running in the NixOS test the DB doesn't start up, so this is definetly not mergable yet.
Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages 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/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Closed by #179710.


[PDI-14232] "Log line timeout (days)" not deleting old log table entries (job logging)

Cleaning up log recodes ( due to log record timeout property ) after writing new info
Tests written

@brosander, review it please

Build Commands:
Build:

rm -r ~/.kettle; rm -r ~/.pentaho; echo removed hidden folders
ant -Dpentaho.coding.standards.version=1.0.1 -Djunit.jvmargs=-XX:PermSize=256m -f engine/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
engine/src/org/pentaho/di/job/Job.java
engine/test-src/org/pentaho/di/job/JobTest.java
Unit test coverage change:
org.pentaho.di.trans.steps.excelwriter.ExcelWriterStepMeta:

instructionPercentCoverageChange :-0.08%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta:

branchPercentCoverageChange : +0.53%

org.pentaho.di.core.logging.BaseLogTable:

instructionPercentCoverageChange : +1.51%
branchPercentCoverageChange : +1.35%
linePercentCoverageChange : +1.90%
complexityPercentCoverageChange : +2.65%
methodPercentCoverageChange : +2.56%

org.pentaho.di.job.Job:

instructionPercentCoverageChange : +1.32%
branchPercentCoverageChange : +0.77%
linePercentCoverageChange : +2.00%
complexityPercentCoverageChange : +0.30%
methodPercentCoverageChange : +0.58%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta:

instructionPercentCoverageChange : +0.28%
branchPercentCoverageChange : +3.33%
complexityPercentCoverageChange : +1.85%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta:

instructionPercentCoverageChange :-0.07%

org.pentaho.di.core.lifecycle.KettleLifecycleSupport:

branchPercentCoverageChange : +8.33%
complexityPercentCoverageChange : +9.09%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta:

instructionPercentCoverageChange :-0.18%

org.pentaho.di.core.logging.JobEntryLogTable:

instructionPercentCoverageChange :-0.45%
linePercentCoverageChange :-1.54%
complexityPercentCoverageChange :-3.64%
methodPercentCoverageChange :-14.29%


Coin info section

Turns the Exchange section into a Coin section of which Exchange is a subsection.
Adds a summary with basic plaintext token info so one can link to it.

I just saw that there's a blank character in the first plaintext line.
Not sure if this is originally intended, but I'd adhered to it.
Eventually I think that not the whole table of content of the official doc need to be represented in the Awesome as separate section. (But maybe it naturally wanders into a separate file once there's more project to report of in the Awesome list)

Someone corrected my on not using "token" BIND, but fyi I now see the use here too
https://github.com/Bx64/Awesome-Compendia/blob/master/awesome-blog/CompendiaCommunityFund_Proposal.md
Maybe I'll push a change in that file too

Changes approved:

awesome-blog/CompendiaCommunityFund_Proposal.md
awesome-blog/Staking.md

Changes not (yet) approved:

README.md

Alphabetical order is not something that we are willing to change, and therefore should be adhered to. Once the list grows, not having items in alphabetical orders will make the list impractical and completely unusable.
The table of contents may be changed later, but for now - with the amount of submissions - it is better off complete.
Regarding exchanges: if we are changing this section, it is better to make a distinction between BIND and wBIND markets. It does not look like Altilly is returning anytime soon, so that can be removed. Therefore please apply the following subsections:

Exchanges

BIND

No active market currently.

wBIND

BIND.exchange (Uniswap)
Switcheo

We are still discussing the section name and will get back to this later.

kk, I mean it's just a block of text so one can link Telegram people somewhere. It seem I don't really have an incentive to enter into a do-this-do-that relationship here so if you figured out how you like your list, I'd say just merge this in as is and make the changes on top of it.

We have guidelines for the format we are using and if you see getting feedback on your suggested changes as a 'do-this-do-that relationship', than I'm not sure open source project commits are a great idea.
Feel free to add the feedback, otherwise we will take your suggestions, review them and rework them how we see fit with our next update.

Yeah that's what I mean, just take it as base for a restructuring of how you see it fit.


Dockerfile incompatible with Ubuntu 12.04 i386

Unable to to build Ubuntu 12.04 32-bit using Dockerfile, got a bunch of errors. Here is a fix, I guess:

adding multiverse/universe to  /etc/apt/source.list failed, so runit and other packages could not be installed. There is a single line:
deb http://archive.ubuntu.com/ubuntu precise main

Just added the complete line:
echo 'deb http://archive.ubuntu.com/ubuntu precise main multiverse universe' > /etc/apt/sources.list

in system_services.sh syslog-ng-core install failed, though all workarounds are there. Had to force install to be true and moved workarounds after it:

 ## Install a syslog daemon.
-$minimal_apt_get_install syslog-ng-core
+$minimal_apt_get_install syslog-ng-core || true
+sed -i 's/^#\(SYSLOGNG_OPTS="--no-caps"\)/\1/g' /etc/default/syslog-ng
+sed -i -E 's/^(\s*)system\(\);/\1unix-stream("\/dev\/log");/' /etc/syslog-ng/syslog-ng.conf
+apt-get install -f || true
+service syslog-ng restart
 mkdir /etc/service/syslog-ng

I don't understand your issue. You mentioned Ubuntu 12.04, but baseimage-docker is based on Ubuntu 14.04. Or are you saying you are trying to build baseimage-docker on Ubuntu 12.04?
Can you paste the exact error message you see?


Added aniketsingh98571

@srinibasbiswal  Please review my PR, and do let me know if any changes required.

Hi @aniketsingh98571 , Can you please give the full URL for images instead of local path.

Which URL ? @srinibasbiswal

@aniketsingh98571 For the images like the first image and social links please change the url to waw.github.
For example :
For the first image : current url : https://github.com/aniketsingh98571/Web_Development/blob/master/canva.png
expected url  : https://raw.githubusercontent.com/aniketsingh98571/Web_Development/master/canva.png
Check your markdown at https://uiwjs.github.io/react-md-editor/

@srinibasbiswal  You have given the wrong name on your website, You have given Ayushi Mittal instead of Aniket Singh in my Profile readme. Please resolve this.

@aniketsingh98571 Please hard reload. Sometimes, it's messing up. I am working on that. For now, hard reloading is resolving that.

@srinibasbiswal  Done
And I have also added this to my profile

@aniketsingh98571 Please create a new merge request

@srinibasbiswal Please add the "Hacktoberfest" label to your repo and also add "Hacktoberfest-accepted" labels to the merged PR,
as the rules for Hacktoberfest have changed. It's a request to you to please add the "Hacktoberfest-accepted" label to my merged PR.


Feedback for "Environment Variables"

https://help.sap.com/docs/btp/sap-business-technology-platform/environment-variables#httpheaders
Hey,
I deployed the httpHeaders like so:

After checking the response I dont see any Content-Security-Policy nor X-Frame-Options headers. Also I am not sure if static routes are covered by the headers as well when I deploy them with the app service.
Thanks in advance.
Regards
Marcus

Hello @szymarcus,
Unfortunately, we do not understand what went wrong. The configuration of the  httpHeaders should normally work as described here on the Help Portal: httpHeaders
Did you add the httpHeaders environment variables to the application router or to the backend application?
(CDS_CONFIG is not a known application router environment  variable.)
Are you using an application router managed by SAP (provided by SAP Cloud Portal or SAP Build Work Zone, standard edition
or advanced edition)?
Best regards,
Sarah

Hello Sarah,
thanks for your quick reply.
I am using the managed app router from SAP Build Work Zone, standard edition. Trying to add the config it states
cf set-env <myApp> httpHeaders "[ { \"X-Frame-Options\": \"ALLOW-FROM http://acme.com\" }, { \"Test-Additional-Header\": \"1\" } ]"
this command. <myApp> can only be filled with my backend service though which does not make sense imo when I try to configure the app router. On the other hand I have no idea how to configure the managed app router. Do you have an idea how I can set the config for the managed app router? I imagine that this is not possible since other apps also use it in our sub account.

Hi @szymarcus,
If you are using the managed application router, you can configure response headers in the routing configuration file (xs-app.json file).
This is described here in the documentation for SAP Build Work Zone, standard edition: responseHeaders
(Or here in the SAP BTP devloper guide: responseHeaders ).
Best regards,
Sarah

Hi there 👋 There's been no activity on this issue for a while, so we're closing it. If you've further information to provide, please comment below and add that information. We'll review it and reopen the conversation if necessary. Until next time, thanks!


Question - How to replicate the AppSource Validation?

Describe the issue
We have made our app available to all countries on appsource and we would like to run the validation before submitting the app.
Now the number of countries available jumped from 44 to 65.
Is there a way to get the list of all available countries to build the list needed for the Run-AlValidation command?
Or hardcoding is the only option?
Scripts used to create container and cause the issue
$countries = "us,ch,be,cz,es,fi,fr,is,it,gb,nz,au,at,nl,no,se"

$validationResult = Run-AlValidation -licenseFile $licenseFile `
                                     -previousApps $previousApp `
                                     -apps $appFile `
                                     -validateCurrent `
                                     -sasToken $sastoken `
                                     -countries $countries `
                                     -affixes $affixes `
                                     -supportedCountries $countries `
                                     -containerName 'validation1'

All additional countries are based on w1 - meaning that if you have one country included, which is based on w1, that should be fine.
Note that if you use -validateCurrent, it only validates on Current version.
If you remove that, it will also determine the lowest compatible version and validate on that (the application dependency in app.json) - the appsource validation process works like this.

Thanks for the clarification, I really appreciate all the work and help you provide!


Derive Eq, Ord and Show instances for Settings data type

Those instances are the standard minimum, but I would also suggest adding others like Data, Read and Generic.

Thank you @k0ral ,
But, why do we need an Ord instance for Settings?

I don't right now, but an Ord instance is necessary to use Maps or Sets for instance.


The Oculus driver causes vive wireless not to work.

I want to use my gaming pc for ALVR on my oculus quest. For reference i have this up and running on my everyday pc. But on my gaming pc when i get it running then want to go back to using my og vive with wireless adapter it goes through process evrything seems fine then no pic in headset. I uninstall alvr including oculus driver then it works again.

I have written the code to avoid conflict with other drivers, but maybe it was not sufficient.
You can try "Uninstall driver" on About tab and then close ALVR to completely avoid conflict.
The log of vrserver will show some clues about the issue.
You can check it in C:\Program Files (x86)\Steam\logs\vrserver.txt

I know its not exactly the same thing, but I stop the oculus service with Oculus Tray Tool  before each use of ALVR since I also use the PC with a Rift-S.  Polygraphene, do you mean I don't have to do this in order to run ALVR?


feature flag: juicecrowd metadata configuration

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #4128   👈

This stack of pull requests is managed by Graphite.


{Telemetry} Switch telemetry bool properties to str type

Related command

Description

GDPR may not detect properties which are not str type, so this PR switch Context.Dafault.VS.Core.Istty and Context.Default.AzureCLI.SecretsDetected from bool type to str type
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Telemetry


Starting on contributor guidelines

On Wed, May 27, 2015 at 07:09:08PM -0700, Greg Wilson wrote:

A CONDUCT.md (33)

This is copied from another SWC repository, no?  Maybe mention which
one in the commit message, so we know where to look for updates.

The original source is contributor-covenant.org, so I thought I should
cite that, but yeah, linking to the lesson-template is a good idea -
I'll fix.

Shall I go ahead and merge before it goes stale?

Sorry it took so long, GH doesn't send messages about updated code in PR :/
I'm fine with this PR after you resolve conflict by rebasing against master.

Done - push the button?

:+1:


feat(i18n): setup for Chinese translation

[x] Setup for Chinese languages (zh)
[x] Translate the Home and Installation page
[x] Create the files for the operators

The next step is the operators translation.

Codecov Report

Merging #308 into master will decrease coverage by 0.83%.
The diff coverage is 25%.

@@            Coverage Diff             @@
##           master     #308      +/-   ##
==========================================
- Coverage   90.78%   89.95%   -0.84%
==========================================
  Files         116      116
  Lines         445      448       +3
  Branches        9        9
==========================================
- Hits          404      403       -1
- Misses         40       43       +3
- Partials        1        2       +1

Impacted Files
Coverage Δ

src/app/core/data/language.data.ts
100% <ø> (ø)
:arrow_up:

src/app/core/services/operators.service.ts
88.23% <0%> (-11.77%)
:arrow_down:

src/app/core/services/language.service.ts
96.55% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


NCAAB Boxscore and Boxscores returning blank for valid inputs

The following code returns nothing for either games or game_data. From reviewing previous issues, it likely could be a format change.
from datetime import date, datetime
from sportsipy.ncaab.boxscore import Boxscore, Boxscores

yr, mo, dy = date.today().year, date.today().month, date.today().day

games = Boxscores(datetime(2021,11,7), datetime(yr,mo,dy))

game_data = Boxscore('2023-01-05-19-central-connecticut-state.html')

Same here... getting null output even after trying #720

@colesims-working I ended up creating my own script to scrape historical box score data for the last 5 seasons (in my case). Hopefully this helps you. Feel free to use the script and modify it to your needs.
https://github.com/Criviere/NCAAB_Sports_Reference_BoxScore_Data_Scraper/blob/main/NCAAB_Scrape_Historical_Data.py


Fix: typescript tailwind.config.ts

Proposing following structure for tailwind.config.ts
`
import type { Config } from "tailwindcss";
const config: Config = {
darkMode: ["class"],
content: [
"./pages//*.{ts,tsx}",
"./components//.{ts,tsx}",
"./app/**/.{ts,tsx}",
"./src/**/*.{ts,tsx}",
],
theme: {
container: {
center: true,
padding: "2rem",
screens: {
"2xl": "1400px",
},
},
extend: {
colors: {
border: "hsl(var(--border))",
input: "hsl(var(--input))",
ring: "hsl(var(--ring))",
background: "hsl(var(--background))",
foreground: "hsl(var(--foreground))",
primary: {
DEFAULT: "hsl(var(--primary))",
foreground: "hsl(var(--primary-foreground))",
},
secondary: {
DEFAULT: "hsl(var(--secondary))",
foreground: "hsl(var(--secondary-foreground))",
},
destructive: {
DEFAULT: "hsl(var(--destructive))",
foreground: "hsl(var(--destructive-foreground))",
},
muted: {
DEFAULT: "hsl(var(--muted))",
foreground: "hsl(var(--muted-foreground))",
},
accent: {
DEFAULT: "hsl(var(--accent))",
foreground: "hsl(var(--accent-foreground))",
},
popover: {
DEFAULT: "hsl(var(--popover))",
foreground: "hsl(var(--popover-foreground))",
},
card: {
DEFAULT: "hsl(var(--card))",
foreground: "hsl(var(--card-foreground))",
},
},
borderRadius: {
lg: "var(--radius)",
md: "calc(var(--radius) - 2px)",
sm: "calc(var(--radius) - 4px)",
},
keyframes: {
"accordion-down": {
from: { height: "0" },
to: { height: "var(--radix-accordion-content-height)" },
},
"accordion-up": {
from: { height: "var(--radix-accordion-content-height)" },
to: { height: "0" },
},
},
animation: {
"accordion-down": "accordion-down 0.2s ease-out",
"accordion-up": "accordion-up 0.2s ease-out",
},
},
},
plugins: [require("tailwindcss-animate")],
};
export default config;
`

There is a PR for it https://github.com/shadcn-ui/ui/pull/1247

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Please explain why the library is not working? I can neither write to activity_main.xml nothing and in MainActivity.class .

the library just refuses to work. I also can't launch the application I downloaded (source code). Nothing helps

the library just refuses to work. I also can't launch the application I downloaded (source code). Nothing helps

I don't think it is maintained now.  In the past, the library is uploaded to jcenter, but not to mavenCentral. Now, jcenter is deprecated.


docs: translate the content under the general directory

Fixes: #1077
Changes: add zh-hans translations for following contents:

website/docs/general/join.md
website/docs/general/submit-issue.md
website/docs/general/security-guide.md
website/docs/general/committer-guide.md

@lunarwhite Thanks for your contribution! And Great Work！
I have reviewed and left some comments, feel free to talk with me if you have any questions^ ^


Maybe name it Boardtrix?

The idea behind this is absolutely brilliant!
So...playing on the concept of "tricks", as in, neat things which can be done (with white boards)...Plus, adding the "-trix" suffix (like so many matrix-related software has), I'm proposing for the name of this really cool application to be named Boardtrix.  (I won't be offended if it isn't adopted of course.)   ;-)

Nice that someone proposes a name idea!
I like that as well. I was thinking of something more boring like MatrixPaper oder MatrixBoard
But Boardtrix fairly nice as well.

Glad you like it!  MatrixBoard isn't bad either...as they say: it says what it does right there on the tin!  :-)


"Export without opening in Finder" plugin

So I made a quick copy of original plugin for myself that doesn't open a new Finder window every time I update something in Sketch and run export. You need to install it as a separate plugin and use via menubar or Cmd+Ctrl+Opt+P.
Feel free to merge this PR (you'd also need to update README), though I suspect there's a better way to achieve this without code duplication.

Thanks for this pull. I'm aware of how annoying it is. I wonder if there's a way to capture an additional key press in the plugin and use that as a marker.
In general I've avoided multiple files in this plugin, as managing/installing a plugin with more than one file is a PITA for sketch (with sketchtoolbox, it's less of an issue, but I don't assume most people use it). I'll think more about how to make this happen without a dupe of files. Ideally we could've circumvented sandboxing and saved the file somewhere you can access later, but as it's in temp, I wanted to err on the side of safety so people can find their files. Thanks!

maybe i could use a time throttle to only auto open it once a day

Sure. Opening Finder window only once during the day wound seem random and confusing. I believe user should have a control when to open Finder window, because it's useful when you want to add new images to Origami (doesn't make sense to me to do it only once a day).

couple solutions i've been tinkering with:

If you are running the plugin, check if you've made an export before. If you have, default to not opening the finder window. We already flash a message at the bottom, but add "Run plugin again within 3s to open the Finder window". If user runs plugin within 3s after, we open Finder window.
Using AppleScript to check current Finder paths opened - ran into sandboxing problem here.

I agree time based is confusing

A plugin can be a folder with multiple commands. Perhaps something like…

Export and reveal in finder
Export only

Yea now that the new plugin format supports a bundle I'll probably move to that format. I didn't want to have users manage two different plugins before
On Sat, Apr 25, 2015 at 1:44 PM, Aen<EMAIL_ADDRESS>wrote:

A plugin can be a folder with multiple commands. Perhaps something like…

Export and reveal in finder
Export only

Reply to this email directly or view it on GitHub:
https://github.com/tarngerine/sketch-origami-export/pull/2#issuecomment-96244784

ill be doing this this week. lol 8 months later...

actually i forgot i already did this


In ue4 media player black screen

When selecting source i see the url i want but its stuck black saying preparing

I’m getting the same thing here.

I think this was an incompatibility between NDI 3.0 and NDI 3.5 Preview. I removed the SDK files from this plug-in. They are now required to be installed manually with the official NDI SDK installer. Check the README for details.


http: handles multiple content-length headers

Logs only as warning
Takes the latest value as the valid one
Same goes for requests and responses
Fixes #1776
I assume that Suricata should parse as much as possible.
So, we should not return HTP_ERROR to keep parsing.
As Wireshark, we use the latest header as the valid one.
Is this assumption right ?
If it is the case, should I create test for suricate-verify ?

Yeah, can you create a Suricata PR if needed for the new warning and a suricata-verify PR for testing various combinations of requests and responses with CL issues?

There are already sample tests in libhtp for C-L issues : 74-response-multiple-cl.t  and 88-response-multiple-cl-mismatch.t
But I should still add some tests to suricata-verify, right ?

See #189 squashed


how to write custom DatabaseAssertionMode

I meet an assertion exception when using @ExpectedDatabase like this:
junit.framework.ComparisonFailure: value (table=account, row=0, col=update_time)
Expected :2016-06-21 20:11:56
Actual   :2016-06-21 20:11:56.027
so I want to write own DatabaseAssertionMode class to ignore microsecond for timestamp.

That is DBUnit-level issue and currently the only way to do it would be to override TimestampDataType class in your test code and add the support you want. But my suggestion would be to use a mocking framework like JMockit and just mock your date classes to return a predictable value which you would then pass as a parameter to your data set.


ES6-like string interpolation

This is related to #899 but should be much more straightforward to implement.
Essentially a syntax translation from something like:
"Fifteen is ${a + b} and not ${2 * a + b}."

to
"Fifteen is " ^ (a + b) ^ " and not " ^ (2 * a + b) ^ "."

This also seems like a good first issue for one to dip ones toes into contributing. If someone could give me some pointers I'd give it a shot.

I have something started as a ppx.  I still need to clean the code up enough to release but it may help as a starting point if you're interested.  The target is currently format strings ("%s"/"%a"-style) but anything else could be used as a target.

Related: https://github.com/facebook/reason/pull/366. Note the PRs to free
` are still open.
On Feb 6, 2017 20:26, "Hezekiah M. Carty"<EMAIL_ADDRESS>wrote:

I have something started as a ppx. I still need to clean the code up
enough to release but it may help as a starting point if you're interested.
The target is currently format strings ("%s"/"%a"-style) but anything
else could be used as a target.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/reason/issues/1040#issuecomment-277785689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEABQAGlVEtMZrecv_4WyMDX__KQ7ZSks5rZ3P_gaJpZM4L4btx
.

Wait, so there already is syntax for string interpolation with {|foo|}? How do you embed an expression in it?

If you are looking for an example, here is one. https://github.com/reasonml/rebel/blob/master/src/merlin.re#L93

@glennsl There is no interpolation in {|foo|} - that form is meant to allow unescaped content within a literal.

@hcarty %s would do fine. I had that exact same idea. It doesn't matter much to me what the exact syntax is, so if {| |} will do the trick too, that's fine by me. Otherwise I'd be happy to take a look at the ppx you've started on.

@hcarty ah, that's more commonly termed a raw string literal, no?

http://caml.inria.fr/pub/docs/manual-ocaml/extn.html#sec250 - "quoted strings" in the OCaml manual.

BuckleScript now has a simple form of string interpolation: https://bloomberg.github.io/bucklescript/Manual.html#_unicode_support_with_string_interpolation_since_1_7_0
Some sugar would still be pretty sweet though!


chore: unify nix publishing

Description
Unifies similar publishing workflows onto one single workflow`
abi-wasm, acvm-js, and noir-wasm call publish-nix-package.
noir-js, noir_js_backend_barretenberg, noir_js_types call publish-js-package.
All workflows build all packages and authenticate with NPM before publishing.
Problem*

Resolves
Summary*

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Does this PR also resolve https://github.com/noir-lang/noir/issues/2882?

Closing in favour of chore: simplify ES workspace build and publish #2989 which heavily borrows from this PR, Thanks @jonybur


EVP_PKEY_cmp() Fails Identical keys

Create a EVP_PKEY and compare it to itself. EVP_PKEY_cmp() returns 0, which means a mismatch.
See code fragment below.
	int rv = 0;

	EVP_PKEY *mac_key = EVP_PKEY_new_raw_private_key(EVP_PKEY_HMAC, NULL,
	                                                 (const unsigned char *)("0123456789"),
	                                                 strlen("0123456789"));

    rv = EVP_PKEY_cmp(mac_key, mac_key);
    if (rv != 1) {
        printf("key compare failed. rv = %d\n", rv);
        return 0;
    }
    printf("key compare passed\n");

Possible fix ...
@ -130,8 +130,14 @@ int EVP_PKEY_cmp(const EVP_PKEY *a, const EVP_PKEY *b)
                return ret;
        }

        if (a->ameth->pub_cmp)
            return a->ameth->pub_cmp(a, b);
        if (a->ameth->pub_cmp) {
            ret = a->ameth->pub_cmp(a, b);
            if (ret == 0) {
                return 1;
            } else {
                return 0;
            }
        }
    }

The issue is present in the latest 1.1.1 stable branch. And appears to be on Master (3.0) as well.

Norman Ashley wrote:

[SNIP]
Create a EVP_PKEY and compare it to itself. EVP_PKEY_cmp() returns 0, which means a mismatch.
See code fragment below.
	int rv = 0;

	EVP_PKEY *mac_key = EVP_PKEY_new_raw_private_key(EVP_PKEY_HMAC, NULL,
	                                                 (const unsigned char *)("0123456789"),
	                                                 strlen("0123456789"));

     rv = EVP_PKEY_cmp(mac_key, mac_key);
     if (rv != 1) {
         printf("key compare failed. rv = %d\n", rv);
         return 0;
     }
     printf("key compare passed\n");

But sample is only for HMAC key while report pretend ALL keys!

Possible fix ...
@ -130,8 +130,14 @@ int EVP_PKEY_cmp(const EVP_PKEY *a, const EVP_PKEY *b)
                 return ret;
         }

         if (a->ameth->pub_cmp)
             return a->ameth->pub_cmp(a, b);
         if (a->ameth->pub_cmp) {
             ret = a->ameth->pub_cmp(a, b);
             if (ret == 0) {
                 return 1;
             } else {
                 return 0;
             }
         }
     }

The issue is present in the latest 1.1.1 stable branch. And appears to be on Master (3.0) as well.

Error is in "a-method" compares specific to a particular "key type".
Why to change a "generic" method?
Quote from EVP_PKEY_ASN1_METHOD(3):
The pub_cmp() method is called when two public keys are to be compared. It MUST return 1 when the keys are equal, 0 otherwise. It's called by EVP_PKEY_cmp(3) <https://www.openssl.org/docs/man1.1.1/man3/EVP_PKEY_cmp.html>.

Of course OpenSSL does not follow strictly documentation. Also It is long story misuse of comparison functions. EC key "cmp" is another sample in this particular case.
Roumen

Yeah, I noticed that many tests fails when i ran make test with my suggested change.

I cannot reproduce this with the current master. Could you please retest this?

Tomáš Mráz wrote:

I cannot reproduce this with the current master. Could you please retest this?

What about 1.1.1 stable ?
Roumen

What about 1.1.1 stable ? Roumen

It most probably still fails there, that's why I am keeping the branch 1.1.1 label and the issue open.

Румен Петров wrote:

Tomáš Mráz wrote:

I cannot reproduce this with the current master. Could you please retest this?

What about 1.1.1 stable ?
Roumen

Quote from https://www.openssl.org/docs/man1.1.0/man3/EVP_PKEY_ASN1_METHOD.html
The pub_cmp() method is called when two public keys are to be compared. It MUST return 1 when the keys are equal, 0 otherwise. It's called by EVP_PKEY_cmp(3).
Then see (1) hmac/hm_ameth.c and (2) ec/ec_ameth.c both pub_cmp does not follow documentation. The code in (2) is harmless in context of EVP_PKEY_cmp but in (1) is used ASN1_OCTET_STRING_cmp
Outside EVP_PKEY_cmp (2) must be considered as defect.
Quote from https://www.openssl.org/docs/man1.1.0/man3/ASN1_STRING_cmp.html
ASN1_STRING_cmp() compares a and b returning 0 if the two are identical. The string types and content are compared.
Roumen

I see both on master and 1.1.1 that ec_ameth ec_key_pub_cmp returns 0 if EC_POINT_cmp() returns nonzero and 1 EC_POINT_cmp() returns 0. That is IMO correct.
The hmac implementation is broken. I'll submit PR for 1.1.1 (it is not there anymore in master at all).

Tomáš Mráz wrote:

I see both on master and 1.1.1 that ec_ameth ec_key_pub_cmp returns 0 if EC_POINT_cmp() returns nonzero and 1 EC_POINT_cmp() returns 0. That is IMO correct.

Sure but let see code:
static int eckey_pub_cmp(const EVP_PKEY *a, const EVP_PKEY *b)
{
int r;
const EC_GROUP *group = EC_KEY_get0_group(b->pkey.ec);
const EC_POINT *pa = EC_KEY_get0_public_key(a->pkey.ec),
pb = EC_KEY_get0_public_key(b->pkey.ec);
if (group == NULL || pa == NULL || pb == NULL)
return -2;
r = EC_POINT_cmp(group, pa, pb, NULL);
if (r == 0)
return 1;
if (r == 1)
return 0;
return -2; / <==== non allowed return value for EVP_PKEY_ASN1_METHOD pub_cmp() method */
}
Reviewing the code is visible that method could return -2.
At this point if EC_POINT_cmp returns values different from 0 and 1 what will be result?
Note "EC_POINT_cmp returns 1 if the points are not equal, 0 if they are, or -1 on error".
So if "EC_POINT_cmp fails method return -2. This is not allowed for pub_cmp() - see https://www.openssl.org/docs/man1.1.1/man3/EVP_PKEY_asn1_set_param.html

The hmac implementation is broken. I'll submit PR for 1.1.1 (it is not there anymore in master at all).

Roumen

I do not think it is relevant as EVP_PKEY_cmp is documented to have negative return values on errors.

And the pub_cmp() is not used outside of that call. And yeah, in theory you could obtain the ameth and obtain the pub_cmp() from it and call it directly from an application. Unless you have an example of such application I am not going to spend more time on that theoretical concern.

Fixed by ccfe5ec


Moved query error tooltip

@phish108
Here's my suggestion on how to address #58. Please tell me whether this is to your expectation.
When entering a search term twice:

When selecting an SDG twice:

Description
Moved error tooltip "The query already exists." to no longer block the existing search terms and mark the offending search term in red color.

The error tooltip is now positioned to the left of the search button ("Press Return to add")
Added styles for "optioncontainer" class to be able to have the error class which marks its children with a red background color. This currently happens when adding a search term that was added already.

Closes #58

Unfortunately, I haven't been able to run npm run test with the following error message:
><EMAIL_ADDRESS>test
> npm run css-lint && npm run css

npm ERR! Missing script: "css-lint"
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/gero/.npm/_logs/2023-02-22T16_31_55_685Z-debug-0.log

Is there something I still need to do in order to have the css-lint script?

There is not unit testing available, so npm run test must fail.

Good start.
Moving the tooltip was not what I had in mind, though. I figured that the tooltip for error messages was a mistake in the first place. In order to be consistent with the rest of the UI, the tooltips should be used only for static help messages.
When I meant overlay in the issue, I thought about having a "danger element" hidden in the DOM is set above or next to some other element. Such overlays are present in the DOM and are not dynamically created and placed. To get an idea about what I mean, click on the robot head at the bottom.  This opens the index terms overlay menu. The point of using overlays is that they do not affect the rest of the DOM and are displayed only when needed.
Also I find using the background color not too helpful, because some items come with their own background color. This is visible in the second example. That color must not be overridden, which you did not. However, A better solution for highlighting the offending element would be the border Something like to or three px wide, maybe with an orange shadow to create a hellish glow effect. This would leave the primary settings of the element intact. Bootstrap should use borderbox measures, so adding a border should not create any flickers.
I commented your code with more details on style.

@phish108 What do you think of this design instead of the tooltip? (I'm still working on the border instead of background, it isn't as trivial as I assumed)

Exactly what I imagined.
Don't bother about responsive design for the time being. Currently the dashboard is used on at least 3k displays. For a mobile use case, we first have to define the use case and then decide what interactions suits best for that use case.

@phish108 Updated the PR, the CSS solution I came up with is unfortunately not very nice, but it works :sweat_smile:


Rename instances for Indexed; add Wander instance

Seems reasonable to me. What does this instance enable?

It lets you compose an IndexedTraversal with a Traversal and get back an IndexedTraversal:
indexedTrav :: IndexedTraversal i s t m n {- = forall p. (Wander p) => Indexed p i m n -> p s t -}
trav :: Traversal m n a b {- = forall q. (Wander q) => q a b -> q m n -}

indexedTrav <<< trav :: forall p. (Wander p) => Indexed p i a b -> p s t

rather than
indexedTrav <<< trav :: forall p. (Wander p, Wander (Indexed p)) => Indexed p i a b -> p s t

👍 Gotcha, thanks.


The dist metadata should set the minimum sphinx version

$sbj here https://github.com/pradyunsg/sphinx-inline-tabs/blame/0923485/pyproject.toml#L12
Otherwise, it has the potential to break projects that have a lower Sphinx version set: https://github.com/pypa/setuptools/issues/2614.

On a side note, this import path is triggered before the Sphinx.require_sphinx() invocation at https://github.com/pradyunsg/sphinx-inline-tabs/blob/0923485/src/sphinx_inline_tabs/init.py#L12. FYI.


Quick and dirty update to 1.20

Updated fabric and minecraft for 1.20 support. Sorry for the spam pulls, this is the final.

Updating properly requires unreleased dependency updates. I already have the diff locally and will handle the update.


Fix go routines

This addresses the handling of go routines in the codebase.
Instead of using errgroup as suggest in the related issue, I used waitgroups because the former allows only to retrieve errors while we want to manage return values also.
A 1 millisecond sleeptime has been added to those esplora methods that perform mulitple calls to prevent the latter to return error. Emprically, Esplora starts complaining (aka returning errors) if more than 340/400 concurrent requests arrive at the same time.
This also makes the channels in InitWallet RPC to be closed at application rather than interface level, simplifying the whole logic.
Use of select has been removed where not necessary.
Finally, some debug-level logs have been added to the wallet service to know how long restoration took to complete.
This closes #352.
Please @tiero @sekulicd  review this.

@tiero in ci.yml I removed the testing-with-elements step since it's deprecated so far.
Originally, we used to switch between esplora and elements based on some env var for app tests, while now, in component tests, we make use of a mocked explorer.


Add parameter-file support

For workflows that have many parameters, or cases where the workflows should be run with slightly different requirements, it can be useful to have a file containing parameters instead of submitting via the command line.
This commit adds support for the --parameter-file (-f) command line option to provide a new-line-delimited file of parameters.  These parameters can still be overridden by the --parameter (-p) flags.
Comments in parameter files are also supported and denoted by a # character.
See #796 for more information

Thanks for taking this on. Have you considered making the parameter file yaml/json and using https://github.com/go-yaml/yaml.
I think this is more consistent with configuration in a kubernetes ecosystem and the package provides the parsing for you.

Thanks @wookasz for the quick review! I had thought about yaml, but I decided to start simple and see where it goes.  I'll push a new commit with that support!

I updated the PR to use YAML/JSON as the format for the parameters file, which should naturally also take care of the comment parsing issue.  The only caveat here is the files have to marshal into a map[string]string.

Thanks for your contribution!

The only caveat here is the files have to marshal into a map[string]string.

One approach is that you can unmarshal to a map[string]interface{}, then when setting the value from the interface object, you use .Value =  fmt.Sprintf("%v", itemVal)


Environment variables for Username / Password

Hello.
I try to understand, what environment variables it accepts?
Is this correct:
services:
  valkey:
    image: valkey/valkey:latest # Replace with the correct image if available on Docker Hub
    container_name: valkey
    ports:
      - '6379:6379' # Map container port 6379 (default Redis port) to host
    volumes:
      - valkey_data:/data # Mount volume to persist data
    environment:
      - REDIS_USER=x-user
      - REDIS_PASSWORD=x-pass
      - REDIS_DB=x-test
    networks:
      - valkey_network

I keep getting Unhandled error event: ReplyError: WRONGPASS invalid username-password pair or user is disabled.

Hi,
As far as I know, it does not use environment variables. You can use a custom configuration file, or additional command arguments.
I use the following to set a password for the default user with --requirepass <password>.
  valkey:
    image: valkey/valkey:8-alpine
    user: 'valkey'
    restart: unless-stopped
    command: >
      --requirepass "${CACHE_PASS}"
    healthcheck:
      test:
        [
          'CMD-SHELL',
          "echo 'auth ${CACHE_PASS}\nping' | valkey-cli -h '${CACHE_HOST}' | grep 'PONG'",
        ]
      interval: 1m
      timeout: 30s
      retries: 3
      start_period: 60s
    networks:
      - cache
    volumes:
      - cachedata:/data

.env
...
CACHE_HOST=valkey
CACHE_PASS=
...

You could do something similar with a username too (not tested):
--user [...]

https://valkey.io/topics/acl/