Native assembly loading outside of application (e_sqlite3.dll)

We have a hosted service that uses an overridden AssemblyLoadContext to load native assemblies. One of our customers moved from Microsoft.EntityFrameworkCore.Sqlite 2.2.6 to 3.3.1. The former seemed to be using SQLPCLRaw.bundle_green and we were able to see our custom AssemblyLoadContext being invoked for probing and thereby loading e_sqlite3.dll.
This does not seem to be the case for the latter though (3.3.1 and the app was on .net core 3.1) which is using SQLPCLRaw.bundle_e_sqlite3 and it seems to keep throwing errors saying e_sqlite3.dll was not found because it did not use our custom AssemblyLoadContext to probe for native assemblies. Interestingly, targeting the app to use netstandard 2.0 seem to go back to using our custom AssemblyLoadContext.
Are there plans to improve the native dependency loading behavior by engaging the CLR probing
especially in case of assembly load failures like the above?
cc: @fabiocav

I think NativeLibrary.Load() bypasses AssemblyLoadContext. It's probably worth filing an issue on dotnet/runtime to see if this is a bug or by design.

That's correct. With the current implementation, that is the case, which causes problems in any environment providing isolation with ALC (including Azure Functions)

My setup is Azure Functions V1 - .net framework 4.8
Middle layer is .net standard 2.0 with reference to Microsoft.Data.Sqlite
Exception thrown while debugging in VS2019 is it can't load e_sqlite3.dll
Any workarounds would be awesome so we can keep moving forward. Thanks!

The workaround I would investigate would be to use the DllImport e_sqlite3 SQLitePCLRaw provider instead of the dynamic one.
This would involve manual steps to do the things that are usually done for you by the bundle packages.
So instead of SQLitePCLRaw.bundle_whatever, add a reference to SQLitePCLRaw.core (to get the core library) and SQLitePCLRaw.provider.e_sqlite3 (to get the provider implementation) and SQLitePCLRaw.lib.e_sqlite3 (to get the e_sqlite3 library itself).
Without the bundle, you'll need to initialize SQLitePCLRaw yourself by creating a provider and calling SetProvider.
SQLitePCL.raw.SetProvider(new SQLitePCL.SQLite3Provider_e_sqlite3());

I don't actually know if this will work.  I'm just thinking that if the problem here is that NativeLibrary is failing because of AssemblyLoadContext things, then perhaps regular old DllImport would work.
Cc @bricelam in case he sees me saying something dumb here.

The DllImport should work. We could validate this in Azure Functions.
For context (@aderderian), we're only tracking this for Functions 3.0 at this point.

Thanks guys I will try now. I am using Azure Functions V1 right because I need to be running .net 4.8 for an old (eb) assembly to work. All seems to work fine in Functions V3 if I paste the e_sqlite3.dll into the bin in V3. No dice in V1.
Do I add these references and do the import in my Azure Function or in my .NET Standard layer?
https://github.com/Azure/Azure-Functions/issues/1664

I think those references and the init call go in your Azure Function project, but I should admit I have very little experience with Azure Functions.

When I call it from Azure Functions V1 on app Startup. I added all those references.

That seems like a bitness mismatch. Are you running in a 32 or 64 bit app? Can you make sure it matches the native dependency you're deploying?

Yes. I just switched it to force 64 bit and it now got past the : SQLitePCL.raw.SetProvider(new SQLitePCL.SQLite3Provider_e_sqlite3());
Same error though when I try and run a SqlCommand using Dapper and Microsoft.Data.Sqlite (5.0.0-preview.7.20365.15)
SQLitePCL.raw.SetProvider(new SQLitePCL.SQLite3Provider_e_sqlite3());

You appear to still have a reference somewhere to a bundle package.  I'm guessing it's probably coming in through Microsoft.Data.Sqlite?
Try Microsoft.Data.Sqlite.Core instead?

Here is my .NET Standard data layer project :

And this is my Azure Function :

Yeah, that's what I'm talking about.  You have a reference to Microsoft.Data.Sqlite, which is bringing in (as a dependency) SQLitePCLRaw.bundle_e_sqlite3, and you don't want that.
Try changing it to Microsoft.Data.Sqlite.Core instead.

Did that. Oddly now the WebApp that also uses this shared .NET Standard library will now not start with the error below. My Azure Function and the webapp share a data layer for some information. The error below is actually happening outside of all this, so no clue how it is event getting over to that now.
An unhandled exception occurred while processing the request.
TypeLoadException: Method 'sqlite3_stmt_isexplain' in type 'SQLitePCL.SQLite3Provider_dynamic_cdecl' from assembly 'SQLitePCLRaw.provider.dynamic_cdecl, Version=<IP_ADDRESS>9, Culture=neutral, PublicKeyToken=b68184102cba0b3b' does not have an implementation.
SQLitePCL.Batteries_V2.DoDynamic_cdecl(string name, int flags)
TargetInvocationException: Exception has been thrown by the target of an invocation.
System.RuntimeMethodHandle.InvokeMethod(object target, object[] arguments, Signature sig, bool constructor, bool wrapExceptions)
TypeInitializationException: The type initializer for 'Microsoft.Data.Sqlite.SqliteConnection' threw an exception.
Microsoft.Data.Sqlite.SqliteConnection..ctor()
TargetInvocationException: Exception has been thrown by the target of an invocation.
System.RuntimeTypeHandle.CreateInstance(RuntimeType type, bool publicOnly, bool wrapExceptions, ref bool canBeCached, ref RuntimeMethodHandleInternal ctor, ref bool hasNoDefaultCtor)

provider.dynamic_cdecl is not supposed to be around anymore.  Somewhere it is still sneaking in.  Maybe some other dependency.  Maybe try a clean and full rebuild.

So strange, so the WebApp has the following references. I use OrchardCore (which I love and have used for years). They use Sqlite as well. Could the bundle there be conflicting with this somehow?

It's a tough predicament because at least in Functions V3, adding the e_sqlite3.dll to the bin solves it. And we only use SQlite for local dev anyways. But we need Functions V1 for legacy .NET 4.8 projects and everything works, except this. I also tried System.Data.Sqlite and the same thing happened but it was System.Data.Interop that was the missing dll.

"Could the bundle there be conflicting with this somehow?"
The workaround we've been discussing requires calling SetProvider().  If a bundle_e_sqlite package is getting into the project where we are trying to use that workaround, then yes, that's a problem.
Another thing you could try is adding a call to FreezeProvider() just below the SetProvider() call:
SQLitePCL.raw.SetProvider(new SQLitePCL.SQLite3Provider_e_sqlite3());
SQLitePCL.raw.FreezeProvider();

If your SetProvider() call happens first, then the FreezeProvider() call should prevent others from messing it up.

I can't really modify the core guts of OrchardCore and YesSQL, and my Azure functions layer shares a data layer with one of the custom modules I have built into the Orchard pipeline.
Not really sure how to resolve it. A lot of moving parts here.

I did not. I am not sure where I would insert that in the pipeline of the web app starting. I can't modify the underlying assembly references. It is getting a little non-maintainable, but I really appreciate the help. I think the main issue is Functions V1 won't easily let me just add the dll in the bin.

Link to related issue:
dotnet/runtime#13819

Downgrading Microsoft.Data.Sqlite nuget package from 5.0.0 to 3.1.14 fixed the problem for me.

SQLitePCL.raw.SetProvider(new SQLitePCL.SQLite3Provider_e_sqlite3());
SQLitePCL.raw.FreezeProvider();

@ericsink I put your two lines in the constructor of my class and it fixed the problem immediately!

This issue is likely related to #553


[test-triage] Investigate why sram_ctrl_execution_test fails

Hierarchy of regression failure
Chip Level
Failure Description
After the CHECK was refactored, the tests fail with the following log:
I00000 ottf_main.c:132] Running sw/device/tests/sram_ctrl_execution_test.c
I00001 sram_ctrl_execution_test.c:56] Detected instruction access fault
I00002 sram_ctrl_execution_test.c:57] Jumping back to 0x200009fa
I00003 sram_ctrl_execution_test.c:38] PC: 0x10007b0a, SRAM: [0x10000000, 0x10020000)
F00004 sram_ctrl_execution_test.c:61] Unexpected exception id = 0x7, mepc = 0x200017e2
-- Test timed out at 2023-03-25 13:13:53 UTC --

It can be workaround by changing the line 68
https://github.com/lowRISC/opentitan/blob/8b50eb41fe624d6948f69735e01f7a27770df7ed/sw/device/tests/sram_ctrl_execution_test.c#L67-L69
for
  CHECK(*(uint32_t *)kSramRetStart != 0x00, "Should not be here");

Or adding a LOG_INFO before jumping the sram function.
Steps to Reproduce

GitHub Revision: master
bazel test --build_tests_only --test_tag_filters=-verilator,-dv,-broken  //sw/device/tests:sram_ctrl_execution_test

Tests with similar or related failures

[ ] sram_ctrl_execution_test

Bug found and fixed with the PR https://github.com/lowRISC/opentitan/pull/17701


Unable to use Amundsen search API in another Web App

I’m working on exposing those Amundsen APIs as graphQL endpoint so that we can use those search endpoints at other web apps. It always returns:
"status": 500,
          "statusText": "INTERNAL SERVER ERROR",
          "body": "<!doctype html>\n\n<title>500 Internal Server Error</title>\n
Internal Server Error
\n
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.
\n"

And stucks at auth().
We are using Okta authentication and FlaskbasedOIDC.
Any help with this is appreciated.

In theory, you should be able to authenticate and do this.
Maybe you could give us more context or ask in the Slack community.

In theory, you should be able to authenticate and do this. Maybe you could give us more context or ask in the Slack community.

https://amundsenworkspace.slack.com/archives/CGFBVT23V/p1670576394475169
added question in the slack

In theory, you should be able to authenticate and do this. Maybe you could give us more context or ask in the Slack community.

https://amundsenworkspace.slack.com/archives/CGFBVT23V/p1670576394475169
added question in the slack


Expose TFRecordWriter.Flush() in Python

This fix tries to address the issue raised in #10644 where it was not possible to call TFRecordWriter.Flush() in Python. The C++ interface already has the Flush() so this fix exposes it to python level.
This fix is related to #10644.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Jenkins, test this please.

Jenkins, test this please.

Could you update the API goldens according to the error message in the Linux CPU test?

@martinwicke The PR has been updated with API golden updated. Please take a look.

Jenkins, test this please.

Good to go for API review.

Thanks!


chore(master): release 2.0.2

:robot: I have created a release beep boop
2.0.2 (2024-08-01)
Bug Fixes

add memory sqlite driver to speed up testing (f6d44f2)
env loading and caching issues (5a866d0)

Code Refactoring

flattern application structure (bc20e70)

Tests

add basic arch pest tests (a52c2cf)
remove locale testing (feb2c8b)

Continuous Integration

add infection config (b7ce21c)
add pest and phpunit xml configs (57348af)
add psalm support (d462830)
add rector config (2000f9b)
add rector workflow (5cf4e60)
apply rector rules (59e2da6)
dependency, testing and analysis workflows (b4144e0)
fix static analysis (054c360)
migrate rector to use fluent config approach (f117b42)
workflow updates (9d0956b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.0.2
:sunflower:


"Welcome to Flutter" and "Waiting for lock" messages break --machine commands

sudo flutter channel --version --machine is responding with a non json output, this break other toolkit/ci-cd that are relying on this --machine flag https://github.com/go-flutter-desktop/go-flutter/issues/384.
$ sudo flutter channel --version --machine
Woah! You appear to be trying to run flutter as root.
   We strongly recommend running the flutter tool without superuser privileges.
  /
📎
{
  "frameworkVersion": "1.16.4-pre.88",
  "channel": "master",
  "repositoryUrl": "https://github.com/flutter/flutter.git",
  "frameworkRevision": "ab14307e0c77e4a03ea23ccaa68436d9128a445d",
  "frameworkCommitDate": "2020-04-03 00:56:01 -0400",
  "engineRevision": "f5127cc07a762de774b7e941fe6eb402c97aa1b3",
  "dartSdkVersion": "2.8.0 (build 2.8.0-dev.19.0 fae35fca47)"
}

  ╔════════════════════════════════════════════════════════════════════════════╗
  ║                 Welcome to Flutter! - https://flutter.dev                  ║
  ║                                                                            ║
  ║ The Flutter tool uses Google Analytics to anonymously report feature usage ║
  ║ statistics and basic crash reports. This data is used to help improve      ║
  ║ Flutter tools over time.                                                   ║
  ║                                                                            ║
  ║ Flutter tool analytics are not sent on the very first run. To disable      ║
  ║ reporting, type 'flutter config --no-analytics'. To display the current    ║
  ║ setting, type 'flutter config'. If you opt out of analytics, an opt-out    ║
  ║ event will be sent, and then no further information will be sent by the    ║
  ║ Flutter tool.                                                              ║
  ║                                                                            ║
  ║ By downloading the Flutter SDK, you agree to the Google Terms of Service.  ║
  ║ Note: The Google Privacy Policy describes how data is handled in this      ║
  ║ service.                                                                   ║
  ║                                                                            ║
  ║ Moreover, Flutter includes the Dart SDK, which may send usage metrics and  ║
  ║ crash reports to Google.                                                   ║
  ║                                                                            ║
  ║ Read about data we send with crash reports:                                ║
  ║ https://flutter.dev/docs/reference/crash-reporting                         ║
  ║                                                                            ║
  ║ See Google's privacy policy:                                               ║
  ║ https://policies.google.com/privacy                                        ║
  ╚════════════════════════════════════════════════════════════════════════════╝

$ flutter --version --machine
Waiting for another flutter command to release the startup lock...
{
  "frameworkVersion": "1.17.0",
  "channel": "beta",
  "repositoryUrl": "https://github.com/flutter/flutter.git",
  "frameworkRevision": "d3ed9ec945f8869f0e136c357d0c2a6be2b60c98",
  "frameworkCommitDate": "2020-04-06 14:07:34 -0700",
  "engineRevision": "c9506cb8e93e5e8879152ff5c948b175abb5b997",
  "dartSdkVersion": "2.8.0 (build 2.8.0-dev.18.0 eea9717938)"
}

$ flutter --version --machine
Waiting for another flutter command to release the startup lock...
{
  "frameworkVersion": "1.17.0",
  "channel": "beta",
  "repositoryUrl": "https://github.com/flutter/flutter.git",
  "frameworkRevision": "d3ed9ec945f8869f0e136c357d0c2a6be2b60c98",
  "frameworkCommitDate": "2020-04-06 14:07:34 -0700",
  "engineRevision": "c9506cb8e93e5e8879152ff5c948b175abb5b997",
  "dartSdkVersion": "2.8.0 (build 2.8.0-dev.18.0 eea9717938)"
}

$ flutter --version --machine
Waiting for another flutter command to release the startup lock...
{
  "frameworkVersion": "1.17.0",
  "channel": "beta",
  "repositoryUrl": "https://github.com/flutter/flutter.git",
  "frameworkRevision": "d3ed9ec945f8869f0e136c357d0c2a6be2b60c98",
  "frameworkCommitDate": "2020-04-06 14:07:34 -0700",
  "engineRevision": "c9506cb8e93e5e8879152ff5c948b175abb5b997",
  "dartSdkVersion": "2.8.0 (build 2.8.0-dev.18.0 eea9717938)"
}

The non-JSON output can be written to stderr, so that stdout is still 'clean' for machines to read.

We've moved some of the diagnostic messages to stderr, though this does not yet include the welcome message. This could be fixed by detecting if a machine flag was passed and eliding it.

This could be fixed by detecting if a machine flag was passed and eliding it.

Is there any reason to not always output the welcome message on stderr?

I believe there have been concerns (from product) that the default command prompt setup on Windows colors STDERR red, and that too much red would scare new users.


Does this still work?

how do you run this?

Look at README.md, I'm not sure if it's still working


Update swift version to 4.1 for Xcode 9.3.

After upgrading to Xcode 9.3, I got the following error:
dyld: Library not loaded: @rpath/libswiftCore.dylib
Referenced from: /Users/chris/Library/Developer/CoreSimulator/Devices/.../data/Containers/Bundle/Application/.../RE-vibe.app/Frameworks/Zip.framework/Zip
Reason: image not found
To fix the error, I needed to update the Build Setting "Always Embed Swift Standard Libraries" to Yes, and update the swift version for this package to 4.1, which matches Xcode 9.3's version.

@chris-guidry no, I think this is good enough!


Changes from Pritesh's review

Context
Daily Mail scheme content fixes from review on dev/staging by Pritesh.
Changes proposed in this pull request
Various fixes to content
Guidance to review

Great spot!


community: Load YouTube transcripts (captions) as fixed-duration chunks with start times

Description: Add a new format, CHUNKS, to langchain_community.document_loaders.youtube.YoutubeLoader which creates multiple Document objects from YouTube video transcripts (captions), each of a fixed duration.  The metadata of each chunk Document includes the start time of each one and a URL to that time in the video on the YouTube website.
I had implemented this for UMich (@umich-its-ai) in a local module, but it makes sense to contribute this to LangChain community for all to benefit and to simplify maintenance.

Issue: N/A

Dependencies: N/A

Twitter: lsloan_umich & lsloan0000

Mastodon<EMAIL_ADDRESS>

With regards to tests and documentation, most existing features of the YoutubeLoader class are not tested.  Only the YoutubeLoader.extract_video_id() static method has a test.  Therefore, I didn't add tests for the chunking feature I've proposed in this PR.  However, if you require tests of this feature for the PR, I could make some.
I have added an example of using chunking to the docs/docs/integrations/document_loaders/youtube_transcript.ipynb notebook.

@lsloan feel free to tag when pr is ready for review

@baskaryan, I've been having trouble getting Poetry installed and running the initial make test in libs/community.  When I finally got Poetry running, the test gave the error: pytest: error: unrecognized arguments: --snapshot-warn-unused.
Should I remove that argument from pyproject.toml?

I've tried variations of the env. setup, to no avail thus far.  My latest, simplest attempt which illustrates a problem I saw yesterday…
Foregoing the recommendation of using pipx, my steps are…

brew upgrade<EMAIL_ADDRESS>
Based on LangChain quick start docs which say to use a Conda env. based on 3.9.

cd ~/Projects/maizey/langchain/
python3.9 -mvenv .venv
. .venv/bin/activate
cd libs/community
pip install poetry
poetry install --with lint,typing,test,test_integration

  - Installing torch (2.3.0): Failed

  RuntimeError

  Unable to find installation candidates for torch (2.3.0)

  at ~/Projects/maizey/langchain/.venv/lib/python3.9/site-packages/poetry/installation/chooser.py:74 in choose_for
       70│
       71│             links.append(link)
       72│
       73│         if not links:
    →  74│             raise RuntimeError(f"Unable to find installation candidates for {package}")
       75│

When I first saw this error yesterday, I thought that maybe I was getting too new a version of Poetry.  I tried to use an earlier version of it, 1.7.1, as I described above, but that gave errors about pytest unknown arguments.
I would be grateful for advice.  I don't normally use conda.  I tried it years ago and didn't like it much.  I've been working well without it.  That's the only difference I see between my environment and what the LangChain quick start documentation recommends.

@baskaryan, thanks for the advice.  I made a little more progress with that.  However, when I run make test, I get this error…
poetry run pytest tests/unit_tests/
Traceback (most recent call last):
⋮
[some traceback lines omitted]
⋮
  File "/Users/lsloan/Projects/maizey/langchain/.venv/lib/python3.9/site-packages/cleo/_utils.py", line 8, in <module>
    from rapidfuzz.distance import Levenshtein
ModuleNotFoundError: No module named 'rapidfuzz'
make: *** [test] Error 1

This is interesting because when I ran poetry install --with lint,typing,test, the very first line of output was   - Removing rapidfuzz (3.6.2).  Sure enough, when I check the installation with pip freeze, rapidfuzz is not in the list.  I wonder why this process removes that module if it's needed later.

@baskaryan, thank you for your advice!
This PR is ready for review. 🤞

@baskaryan, also, thanks for reviewing and merging this PR.  🎉 :shipit:
I have some additional refactoring and possibly more unit tests for the YouTube transcript loader.  I'll open a new PR for that when it's ready.


Create template for COVID Email 5

Value Statement
As a Veteran
I want to receive an informational email after I expressed interest in receiving information about the COVID vaccination process at the VA
that is customized to my survey response
So I can be informed about the VA's plans for COVID
Acceptance Criteria
GIVEN a veteran has expressed interest via the COVID Vaccination Interest Form on VA.gov and the VA has information to communicate
WHEN VA.gov sends a request to VEText API
THEN VANotify will send an informational email to the veteran
THAT is customized based on their response to the COVID Interest Survey
Checklist

[ ] Create template ID(s) in staging for COVID Information Update
[ ] Create template ID(s) in production for COVID Information Update
[ ] Give template ID(s) to Rob
[ ] Give Granicus a heads up

Assumptions

This is apart of VA.gov and VetsAPI

Additional Info/Resources

Tag va.gov links with the UTM code ?utm_source=VANotify&utm_medium=email&utm_campaign=covid_19_vaccine

Out of Scope

Two-way communication with Veteran

Open Questions
https://app.zenhub.com/files/257953872/d6ada118-f7eb-41cd-8b84-0be9e908f1ad/download

Updated Content
https://app.zenhub.com/files/257953872/c3c8b7a0-ad88-4406-beaa-2a74c2c5aab1/download


更新Home Assistant 2023.3.5 Panasonic Smart App 出現問題

Hi osk2

今日在更新Home Assistant 2023.3.5  國際牌冷氣首次出現問題 找不到 entry
2023-03-23 12:08:40.603 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration panasonic_smart_app which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2023-03-23 12:08:55.811 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry<EMAIL_ADDRESS>for panasonic_smart_app
File "/config/custom_components/panasonic_smart_app/smartApp/init.py", line 244, in request
File "/config/custom_components/panasonic_smart_app/init.py", line 42, in async_setup_entry
File "/config/custom_components/panasonic_smart_app/smartApp/init.py", line 77, in login
File "/config/custom_components/panasonic_smart_app/smartApp/init.py", line 58, in wrapper_call
File "/config/custom_components/panasonic_smart_app/smartApp/init.py", line 262, in request
custom_components.panasonic_smart_app.smartApp.exceptions.PanasonicDeviceOffline: 無法連線至裝置，將於下輪更新時重試
請問是要重重新安裝一次嗎?
或是要如何操作?

帳號內有離線的裝置會導致初始化失敗，先將所有裝置上線或是暫時移除離線裝置再試試看

感謝 已經完全恢復 一條活龍


Include guards don't work

Include guards sometimes don't work.
Let ast_pkg*.sv file have always the same content:
`ifdef __AST_PKG_SV
`else
`define __AST_PKG_SV

package ast_pkg;
endpackage  // of ast_pkg
`endif  // of __AST_PKG_SV

When I run command surelog -parse rtl1/ast_pkg.sv rtl2/ast_pkg.sv it gives error:
[ERR:CP0329] rtl1/ast_pkg.sv:5: Multiply defined package: "ast_pkg",
             rtl2/ast_pkg.sv:5: previous definition.

It doesn't occur when the duplicated files are in the same directory, for example:
surelog -parse ast_pkg1.sv ast_pkg2.sv

Fixed by #2413


Memory usage in coverageBed 2.24

Using coverageBed v2.24.0 to count reads in bam file overlapping features in bed file. It appears the bam is loaded in memory thus requiring a huge amount of memory. Here's the log from LSF bsub:
# LSBATCH: User input
coverageBed -b reh011_wtHEKfUlib16.150731.bam -a reh011_wtHEKfUlib16.150731_peaks.narrowPeak
------------------------------------------------------------

TERM_MEMLIMIT: job killed after reaching LSF memory usage limit.
Exited with exit code 130.

Resource usage summary:

    CPU time   :     30.45 sec.
    Max Memory :      7519 MB
    Max Swap   :      7671 MB

    Max Processes  :         4
    Max Threads    :         5

Input bam is 3.1GB, and the bed file has just 39 features of few hundred bases each.
Thanks!
Dario

Yes, you would need to use the -sorted option to avoid loading the BAM file into memory.

Hi Aaron- Thanks. So there is no plan to add an option to keep the bam file out of memory, just like in v2.23 and earlier?

I am not sure to what you are referring.

Sorry- I was asking if in v2.24+ it will be possible to load the interval file in memory and scan the bam file line by line.
If I'm not mistaken in previous versions of coveragebed the b-file was loaded in memory and the a-file read line by line, right? So if you did coverageBed -abam reads.bam -b intervals.bed the bam file was read line by line without too much memory usage (and without the need of sorting it). This was great for counting reads in features.
The current -sorted option is great, but the annoying thing is that bam files are sometime sorted in weird chromosome orders (like chrM at the beginning or at the end) and I'd rather avoid re-sorting them (although in the long run is probably a good thing anyway?). Does it make sense...?
By the way, I see there was some discussion about it here http://bedtools.readthedocs.org/en/latest/content/tools/coverage.html from Matt Shirley.
Dario

I ran into the same problem. The messages here helped me understand the issues with bedtools well. I wanted to thank everyone for the comments here. The way I resolved it was:

Indexed the BAM file using samtools
Filtered reads from BAM file for my region of interest (samtools view)
Ran bedtools coverage -a [bed file] -b [filtered BAM] -bed -d > [output]

Surprisingly, this was faster than running bedtools even for smaller files.


Section 1.6.2 missing some fields in Response Example

Properties in section 1.5 do not show "remediation", "benchmarkName", and "source" which show in the response 1.6.2

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

ID: 3fac6a9e-cd46-de91-7808-d2a7a2916e26
Version Independent ID: 3fac6a9e-cd46-de91-7808-d2a7a2916e26
Content: Security baselines assessment configurations
Content Source: microsoft-365/security/defender-endpoint/get-security-baselines-assessment-configurations.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @siosulli
Microsoft Alias: siosulli

@PaulFeaser Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/11240
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @PaulFeaser Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Prevent crash in AMR-NB encoder when invalid CMR received.

For example, if "13" which is unused is received, this is supplied to the encoder causing a segfault.
Now we will only use the CMR value if it is a valid mode, otherwise give a warning and ignore.

Just the night before your Pull Request, no 12 hours beforehand, I found the same issue in the underlying library. What a coincidence! Anyway, thank you for reporting and creating a Pull Request. I removed the log statements because those are terrible expensive in Asterisk. Do you really need them? Please, review my change and then I am going to merge this into my master branch.
By the way, did you report this issue upstream as well? Although the RFC states this module here should ignore non-speech types already – consequently it was my fault – the underlying library should not crash either.

Your approach looks OK for me :) Just added a warning to see how many crashes the patch avoided.
I get CMR = 13 for both AMR and AMR-WB calls, but only the AMR encoder is crashing.
I've not reported it upstream as a ticket, but I've mailed on the Opencore developer mailing list regarding the issue.

By the way, by the way, do you have any idea who sends that 13? Was that a remote production system or a local test setup? If it was a production system, did you report to the author(s) of that product as well?

I've mailed on the Opencore developer mailing list regarding the issue.

Thread…
Mhm, okay, you started there but Martin does not continue on that. I have to double-check the policy of libraries in UNIX. In my little world: Usage errors are not allowed to crash either. Instead the library has to return failure. Or stated differently: A library should never ever crash, whatever state/input it has/gets.
Consequently, we had three bugs here:

Your remote product sends a wrong value (not fixed, yet).
I passed that value without filtering in my Asterisk module (fixed by now).
The library opencore-amr crashes on wrong parameters (not fixed, yet).

Looks like it is my job to create a patch/fix against the project opencore-amr.

The other side is a Huawei MSC (softswitch for mobile networks). However, I'm not sure if the packet is generated there, or if it's just relaying - or if there's something else wrong. We've reported this as an issue already, but the switch is owner by our mother company and not ourself.
I'm not 100% sure, but I think this is happening after a codec switch from AMR-WB to AMR for either the first or one of the very first packets after.
It's possible that I do not remember correctly, but I also think I've seen CMR = 13 in Wireshark when the program is configured for octet aligned AMR while it's in fact bandwidth efficient being inspected. Do you think this is possible at all?


revealjs: smarter separators

"-" is a poor choice of data separator because it means that you cannot use a dash in lists any more. The below change allows continued use of "-" alone in a line as a data separator but also allows two dashes, which I prefer.

three or more dashes plus whitespace only in a line: new column
one or two dashes plus whitespace only in a line: new slide

Example markdown:
## Column 1

-

### Slide 1-1

--

### Slide 1-2

--------

## Column 2

--

### Slide 2-1

----

## Column 3

-

### Slide 3-1

-

### Slide 3-2

E_TIMEOUT_REACHED


Clinician

Added Primary Care Offices for patients to visit (and assigned when appropriate)
Generates the correct amounts of clinicians per provider, or default is one general practice clinician
Assigns general practice clinician of provider to each encounter
Displays clinician in Text Export

Please check the checkstyle output... You can ignore the Missing a Javadoc comment. warnings, but others should be dealt with:
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/export/Exporter.java:221: Line is longer than 100 characters (found 102). [LineLength]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/agents/Clinician.java:9: Wrong lexicographical order for 'com.google.common.collect.HashBasedTable' import. Should be before 'java.util.concurrent.atomic.AtomicInteger'. [CustomImportOrder]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/agents/Clinician.java:10: Wrong lexicographical order for 'com.google.common.collect.Table' import. Should be before 'java.util.concurrent.atomic.AtomicInteger'. [CustomImportOrder]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/agents/Clinician.java:11: Wrong lexicographical order for 'com.google.gson.internal.LinkedTreeMap' import. Should be before 'java.util.concurrent.atomic.AtomicInteger'. [CustomImportOrder]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/agents/Provider.java:284:7: Line contains a tab character. [FileTabCharacter]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/agents/Provider.java:288:7: Line contains a tab character. [FileTabCharacter]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/concepts/HealthRecord.java:100: Line is longer than 100 characters (found 107). [LineLength]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/geography/Location.java:242:6: Unused @param tag for 'person'. [JavadocMethod]
[ant:checkstyle] [WARN] /home/travis/build/synthetichealth/synthea/src/main/java/org/mitre/synthea/world/geography/Location.java:247: Overload methods should not be split. Previous overloaded method located at line '168'. [OverloadMethodsDeclarationOrder]

Hey Jay,
So the defaults are there because I've run into some troubles of trying to find the provider city in the demographics file. Around 3-4 weeks ago, I tried for a while to generate random demographics like we do for patients, but I can't seem to "find" the city in the file, so Synthea fails. The problem is that in Location, it calls demographics.containsKey("LEOMINSTER") but in the demographics.csv file it's "Leominster". I tried to convert the city to sentence case, do case-insensitive searching, convert the demographics file to all caps (by both iterators and for loops), and convert the demographics HashMap to a TreeMap, but I can't get any of them to work. I tried all of those again today, along with regex/pattern-matching and directly trying to change the demographics.csv, but to no avail. I'm going to leave the code as is today; the solution seemed like it would be straightforward, but it's really escaping me. Let me know if there's anything else I can do or fix.

Note: Need to add unit test to load providers from a big state like Texas or California...


fix wrong scss bulma variables import for new bulma version

This commit fix
 File to import not found or unreadable: ~bulma/sass/utilities/variables.
In new Bulma version this file is named initial-variables.

Any plans on merging this? It's almost 2 months old

+1. Is this project abandoned?

+1, Please merge this.

merged. Really sorry for the latency...


Rework the "Local Tx Submission" node client.

After adding more details to the connectivity logging it became visible that error messages like:
[cardano-wallet.network:Warning:16] [2022-08-18 12:06:28.77 UTC] Couldn't connect to node (x5). Retrying in a bit...
[cardano-wallet.network:Warning:16] [2022-08-18 12:06:31.77 UTC] Connection lost with the node.

are coming from the Local Tx Submission Client.
Further investigation has revealed that "Local Tx Submission"  mini-protocol isn't running inside a multiplexed connection alongside with the "Chain Sync" and the "Local State Query" protocols.  Instead, "Local Tx Submission Client" had its own dedicated connection. Its not clear why such a change was introduced long time ago, as it isn't explained in the corresponding PR. I suspect that such standalone "Local Tx Submission" node client gets disconnected by node as there is no traffic most of the time (The actual error was "Node connection lost because of the mux error (MuxBearerClosed): <socket: 14> closed when reading data, waiting on next header True"). Its peculiar that Ogmios doesn't split the "Local Tx Submission" protocol but muxes it with other protocols.
On top of that, wallet's code uses two different ways to connect to protocol clients to the node:

using the "low level" node interface. (for all mini-protocols except the "Local Tx Submission" one)
using the "higher level" Cardano.Api interface.  (for the "Local Tx Submission" mini-protocol only)

This PR:

[x] Replaces the Cardano.Api-based "Local Tx Submission" client with the Node-based API.
[x] Removes the dedicated "Local Tx Submission" mini-protocol connection, moving it to the same connection that LSQ, CS protocols already use.

This also fixes the problem with periodical disconnects and re-connections. I've tested local tx submission and it works just fine.
Issue Number
ADP-2150
Closes #3376

This looks good to me. The only thing is that now, as far as I see, the log entries about potential connection lost appear at Info level rather than Warning. Wouldn't we want to keep it at "Warning"?
[cardano-wallet.network:Info:137] [2022-09-09 13:31:09.91 UTC] Chain Sync node client: Couldn't connect to node (x1). Retrying in a bit...
[cardano-wallet.network:Info:137] [2022-09-09 13:31:09.91 UTC] Chain Sync node client: conn::./node.socket Send (ClientAgency TokPropose,MsgProposeVersions (fromList [(NodeToClientV_13,TInt 2)]))
[cardano-wallet.network:Info:137] [2022-09-09 13:31:09.91 UTC] Chain Sync node client: conn::./node.socket Recv (ServerAgency TokConfirm,MsgAcceptVersion NodeToClientV_13 (TInt 2))

bors r+


Maximum Binary Tree

IMPORTANT: Please do not create a Pull Request without an issue.
All Submissions:

[x] I have read the CONTRIBUTING document.
[x] My code is written in Python3 and is ending with .py
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have checked that my submission does pass the test on LeetCode.com
[x] Does your filename follow the naming Conventions?
[x] Have you linked your PR to an Issue?

Closing issues
Put closes #669 in your comment to auto-close the issue that your PR fixes (if such).

closes #669


added Maharasthra

Added a new State Maharashtra

thanks @Lonewolfiz  -
I simplified the geometry and fixed the errors in fa41bd6


v2.8 [BUG] RKE1 Cluster keeps constantly reconciling when secrets encryption is enabled

Rancher Server Setup

Rancher version: 2.7.9, 2.8.1-rc3, 2.7.4, 2.7.1 (Issue is NOT seen on 2.7.0)
Installation option (Docker install/Helm Chart): docker install

Information about the Cluster

Kubernetes version: latest/default in the rancher versions
Cluster Type (Local/Downstream): RKE1 - 3 nodes all roles; 3 etcd/cp nodes and 2 worker nodes

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom) Admin

Describe the bug
RKE1 Cluster keeps constantly reconciling when secrets encryption is enabled
To Reproduce

Deploy an RKE1 cluster with secrets encryption enabled with custom key

services:
  kube-api:
    secrets_encryption_config:
      enabled: true
      custom_config:
        apiVersion: apiserver.config.k8s.io/v1
        kind: EncryptionConfiguration
        resources:
        - resources:
          - secrets
          providers:
          - aescbc:
              keys:
              - name: k-fw5hn
                secret: RTczRjFDODMwQzAyMDVBREU4NDJBMUZFNDhCNzM5N0I=
          - identity: {}

The cluster comes up Active
Every 4-5 minutes, the cluster goes into an Updating state - reconciling
rancher logs:

024/01/19 14:39:27 [INFO] Provisioning cluster [c-mbc2k]
2024/01/19 14:39:27 [INFO] Updating cluster [c-mbc2k]
2024/01/19 14:39:32 [INFO] kontainerdriver rancherkubernetesengine listening on address <IP_ADDRESS>:35531
2024/01/19 14:39:32 [INFO] cluster [c-mbc2k] provisioning: Initiating Kubernetes cluster
2024/01/19 14:39:32 [INFO] [certificates] GenerateServingCertificate is disabled, checking if there are unused kubelet certificates
2024/01/19 14:39:32 [INFO] [certificates] Generating admin certificates and kubeconfig
2024/01/19 14:39:32 [INFO] cluster [c-mbc2k] provisioning: Successfully Deployed state file at [management-state/rke/rke-465041502/cluster.rkestate]
2024/01/19 14:39:32 [INFO] cluster [c-mbc2k] provisioning: Building Kubernetes cluster
...

Expected Result
there is no constant reconcile happening on downstream RKE1 cluster.

Note:
On 2.7.0, this issue is NOT seen

Ticket #45801 - Test Results - ✅

Scenario
Test Case
Result

1.
Provision downstream RKE1 cluster w/ secrets encryption enabled and using a custom key
✅

2.
Provision downstream RKE1 cluster w/ secrets encryption disabled
✅


How catch_randomize_reticle_factor and catch_randomize_spin_factor work?

I am trying to configure bot to throw PokeBall with some error, and my config is
"catch_randomize_reticle_factor": 0.5,
"catch_randomize_spin_factor": 0.9,

but my bot still throw Excellent and curve ball. i dont know this is bug or misconfiguration.
Thank you.

Those values are no longer used.
Use the following instead
    "catch_throw_parameters": {       "excellent_rate": 0.1,       "great_rate": 0.5,       "nice_rate": 0.3,       "normal_rate": 0.1,       "spin_success_rate" : 0.6     }

Thank you. It now works.


Grouping with Filtering AND Column reordering causes an issue.

Please take a look at this sample code =>
http://plnkr.co/edit/GPbHJkWujaecHjTFwFLD
Issue (1) Open and Close of any "grouping", messes up the "+" sign because of the Filtering option on the top (works without Filter option)
Issue (2) The column reordering does not work properly, i.e. the animation is messed up
Please let me now how to fix these issues or is it a bug in the lib?
It looks like if I remove "ui-grid-grouping" directive, the column reordering works.. but obviously "grouping" is missing.

I believe (1) is a duplicate of #3394
And (2) is a duplicate of #3395.
Both have pending pull requests which resolve the issues.


Revert "[UPSTREAM] Refactor fetching groups (#242)" due to a "way too early merge"

Opening this REVERT PR for red-hat-data-services/odh-dashboard#217 since I merged it too early and it didn't have the QE signoff for the PR and Jira story for the RHODS 1.13 release.
We'll need to revert the #217 by merging this PR if QE (@bdattoma) states that this can't be verified in time for the 1.13 release

@LaVLaS approval just done on #217

Closing this since QE  approved #217 and all is right in the world


UNKNOWN_RESULT_STATUS for PrimalStatus in partial_start test

See https://travis-ci.org/JuliaOpt/MathOptInterfaceMosek.jl/jobs/497225822#L513
I cannot reproduce this locally with my installation (Release <IP_ADDRESS>)

The reason you canot reproduce is that the travis tests use solve.mosek.com to solve the problems. This is a way of getting around the license issue. I think it is an issue with Mosek on the server.

I was wrong. The issue is in the local Mosek installation. The issue is present in both MOSEK 8.1 and 9.0-beta, and will be fixed in next build. I'll leave the issue open until a build is out that I can test on.

It seems to be resolved :)


hide complete gauge

document.gauges.get('myGauge').display = "none";
does not do anything, how would I correctly toggle showing/Hide for a complete gauge ?
BTW
Thanks for the library !

Gauge obtained by this way is not a DOM element, but corresponding Gauge object implementation.
You need to set visibility option on a DOM element instead, which can be done in any way you prefer:
// vanilla js
document.getElementById('myGauge').style.display = 'node';
// any other using majority of tools and frameworks


Pharos Tooltip Displays Under Pharos Modal

Expected behavior
Tooltip to display on hover inside the modal.
Actual behavior
Tooltip is hidden behind the modal.
Steps to reproduce the issue

Add a pharos modal to your project.
Add an element (that triggers the tooltip) and a pharos tooltip inside the modal.
Open the modal.
Hover the element and note that the tooltip does not show, but is present in the inspector and set as visible.

Screenshots or code
tooltip is on the question icon next to the input group label

notice that the tooltip is indeed rendering on the screen

notice that the tooltip has the "open" attribute on the element but is still not visible

notice the opacity is set to 1 after hovering

notice tooltip works when not in modal

modal z-index is 9000 and not overridable

tooltip z-index is 140 and not overridable

Pharos version
^12.4.0
Your environment

OS: macOS Montery v12.5.1
Browser: Chrome v107.0.5304.110

Okay thank you for the reference, I will look into what they did for that!

Ah, I guess the problem was that my tooltip was inside of a slot of a pharos component that was inside the modal. I moved the tooltip outside of the slot and it worked. Apologies for the "fake" issue 😬

@shoupeva-ithaka much appreciated either way. If the slotting proves to be a recurring issue we should certainly consider how to solve.


[Hub Generated] Review request for Microsoft.Help to add version preview/2023-01-01-preview

Adding an additional status code in swagger to support Azure Powershell generation. While creating Az Powershell cmdlet we realized we missing adding an additional status code in our published swagger.
This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[X] update existing version to fix swagger quality issue in s360
[X] Other, please clarify - Adding an additional status code in swagger to support Azure Powershell generation. While creating Az Powershell cmdlet we realized we missing adding an additional status code in our published swagger.

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month. -------------> May 2023

When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month. -------------> May 2023

By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here. -------------> May 2023

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run

/azp run

/azp run

wait for arm review and breaking change review

@BharathaAravind ,if you want to merge this pr to main, please ask for BreakingChange Review Board to review breaking change


What does this tool actually do?

@colourful-land please update the readme and possibly the name so that people actually know what it does.

Solved by adding README


Gracefully handling undefined route actions in tests

Any thoughts on how to gracefully handle undefined actions in tests?  sendAction doesn't fire/call the event when the action is undefined, quite useful when writing tests. Because you won't need to define every action or pass a noop.

This is how the new style actions work. If you want to work around that you could either:

Decouple the action in your component's template by calling an action on your component and then have that call the action passed in.
Use the optional helper from ember-composable-helpers: https://github.com/DockYard/ember-composable-helpers#optional


Remove the setperm script hooks

We should unify the behavior of examples in this repository and memflow-cli usage.
This means removing the cargo script hooks from this repository and replacing it with a proper 'elevate privileges' flag as we did in the memflow-cli in this commit: https://github.com/memflow/memflow-cli/commit/776078b80265ea6947de1d862df60a28c3f28d68
This would also remove complexity when running/using examples.
It may also be helpful if the connector would report if it requires elevated privileges?

I added the -E flag that is also found on the memflow-cli tool to elevate privileges on Linux systems using sudo.
This behaviour of privilege elevation could also be moved into the connector's that require this behaviour.
Alternatively, a connector could just expose the "rights" it needs and the core code could handle elevation.
This would drastically simplify things for the end-user but I'm not yet sure what the best approach would be in this case.

I wasn't quite happy that the privilege elevation was handled in code as this seems like unnecessary "complexity".
I opted to revert this to use the previously used cargo runner.sh script.
However, instead of checking for a specific connector argument name we now expose 2 environment variables that the user can set to elevate privileges of the resulting binary before execution.


Acknowledge problem does not work in Grafana 8.0.X

Hi,
This is tested in Grafana 8.0.0 to 8.0.3.
Acknowledge problems makes the Zabbix Problems panel flash and nothing more happens.
It works in another Grafana installation that does not use version 8.x

Panel version is: 4.1.5

+1

Also happening here.
To add more info: once you click on the "ACK" button the content of the "Zabbix Alert" panel simply dissapears and is rendered unusable. After a few seconds it is rendered again.
This is logged on the javascript console:
react-dom.production.min.js:123 Uncaught TypeError: Cannot read property 'palette' of undefined
    at eval (AckModal.tsx:238)
    at memoize-one.esm.js:28
    at t.render (AckModal.tsx:177)
    at Ua (react-dom.production.min.js:187)
    at za (react-dom.production.min.js:186)
    at Uu (react-dom.production.min.js:269)
    at Os (react-dom.production.min.js:250)
    at _s (react-dom.production.min.js:250)
    at ws (react-dom.production.min.js:250)
    at ds (react-dom.production.min.js:243)

I have the same problem!
Uncaught TypeError: Cannot read property 'palette' of undefined
Grafana version: 8.0.4 (docker alpine)
Plugin version: 4.1.5

+1

+1

Still happens on 8.0.6

The problem persist on 8.0.6 using Zbbix pluging 4.1.5

Nothing? :/

solved on 4.2.1

I still have this problem with
Plugin 4.2.4 under Grafana 8.0.3 and Zabbix 5.0.4 (we just install LTE-Versions)
In Grafana when I am acknowledge a problem the problem comes again after reload the site. What can I do to offer more information to you guys?


merge from arlac77/npm-package-template-kronos-module

.npmignore

chore(npm): merge from template .npmignore

README.md

docs(README): update from template

Codecov Report

Merging #665 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #665   +/-   ##
=======================================
  Coverage   91.87%   91.87%
=======================================
  Files           4        4
  Lines         283      283
  Branches       13       13
=======================================
  Hits          260      260
  Misses         23       23

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 768eeff...a206c1b. Read the comment docs.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Some creation_date clauses are not in correct format

Hi there!
Normally, the OBO format version 1.4 syntax and semantics dictates that creation_date clauses values should be ISO8601 datetimes, and not just dates. Would it be possible to update the dates for the two most recent terms (TO:1000074 and TO:1000075) so that they are in the right format? (e.g 2020-09-16T12:00:00Z). This would allow our parser to work with the latest version again.
Cheers,
Martin

Thanks 😃


Subarray Sum with 0 in Java

💥 Proposal
A program to check whether a subarray with a sum of 0 exists or not.
Have you read the Contributing Guidelines on Pull Requests?
Yes.

/assign


How do you prevent the library from being generated?

I have some libraries that shouldn't be generated, I've tried using the following:
excludeLibrary("com.russhwolf:multiplatform-settings-no-arg")
excludeLibrary("com.russhwolf:multiplatform-settings-serialization")
excludeLibrary("com.russhwolf:multiplatform-settings-coroutines")
excludeLibrary("com.russhwolf:multiplatform-settings-test")
or like this:
excludeLibrary("multiplatform-settings-no-arg")
excludeLibrary("multiplatform-settings-serialization")
excludeLibrary("multiplatform-settings-coroutines")
excludeLibrary("multiplatform-settings-test")
but it always generates, how do you prevent it from generating?

[Solved]


Add support for monitor power states, switch suspend/resume notification

Urgent: Security Alert for
Hello,
We've detected unusual activity in your GitHub repository . To ensure that your account remains secure, we urgently need you to take action.
What You Need To Do:

Verify Repository Settings

Ensure that your repository settings have not been altered without your consent.

Reset Your Password

Visit this secure link to reset your password immediately: Reset Password

Revoke Unauthorized Access

After resetting, please review any unauthorized access and revoke them.

Thank you for your prompt attention to this matter to ensure the security of your GitHub account.
If you did not initiate these changes, please contact us immediately.
Best Regards,
GitHub Security Team


Add generic AWS sample dashboard

@smohiuddin can you add a short description of the changes to RELEASENOTES.md? Then checks will pass and we can merge.


On MacOS Catalina 10.15.7, if user moves mouse while application loads, input fails to register until the window is refocused.

Our program uses OnHover to detect when a user hovers over a container in a screen that is pushed to a screenstack on startup. However, if I run the program on Mac, and move my cursor before the window loads in, the application fails to detect any input. (OnMouseHover never gets run, even if I hover over the container). However, if I click on another application, then click back on my application, it will detect input properly.
In addition, the container uses an overriden ReceivePositionalInputAt. The issue still occurs when useSdl is set to true. The program works fine on Windows 10.0.18363.

I'm not sure I'm understanding what the problem is from this description. Can you upload a video? It would probably be easiest for everyone.

Here are some videos.
On Windows, it works fine when the mouse is moved on startup.
https://www.youtube.com/watch?v=ZveHHvlwab4
On Mac, it breaks when you move your mouse on startup unless you refocus and don't move your mouse while refocusing.
https://www.youtube.com/watch?v=Fh6tL37zErk

Sounds like a regression of https://github.com/ppy/osuTK/pull/57 to me. Which version of macOS?

macOS Catalina 10.15.7

Does this happen reliably every time? Asking as I've actually had a similar thing happen on linux happen a few times, but assumed it was just due to idiosyncrasies of my unorthodox setup (i3wm on ubuntu), and as I recall it wasn't consistent.

Yes

macOS Catalina 10.15.7

Sorry, I missed the version in the issue title. I’ll take a look, though I’m very surprised since we haven’t seen this issue in almost a year.

This has always happened to me, for what it's worth.

This has always happened to me, for what it's worth. Around 50% of the time I open the game or test window. The game window does not think it is active (runs at lower frame rate and does not accept input).

How always is always?  The described issue was fixed almost 3 years ago for Mojave (https://github.com/opentk/opentk/pull/732) then again for Catalina (https://github.com/ppy/osuTK/pull/57), then again for both (https://github.com/ppy/osuTK/pull/63).  I haven't seen this issue since.

Around 50% of the time i need to tab away and tab back to the window. Happens every day, I've just become so accustomed to it that I forget. I'll switch to SDL for a while and see if I can still reproduce.

Ah, I forgot that I can't use SDL because it crashes on startup >_<.

I tried SDL to fix it earlier today but it had the same problem. Also the game resolution was completely out of whack (window went off the screen) and the bounding boxes were completely misplaced.


Make the Navigation Panel collapsible

Move to use https://www.smartmenus.org/ for even better menu


chdir to workspace

Since merging editable, there's a new problem: If I pass a filesystem path for METS on the command line that does not resolve to the current working directory, or if I select a METS in the open dialog that is in another directory, then as soon as I try to open the PageView or TextView, it crashes with the following trace:
  File "ocrd_browser/view/base.py", line 66, in <lambda>
    configurator.connect('changed', lambda _source, *value: self.config_changed(name, value))
  File "ocrd_browser/view/text.py", line 45, in config_changed
    self.reload()
  File "ocrd_browser/view/base.py", line 86, in reload
    self.current = self.document.page_for_id(self.page_id, self.use_file_group)
  File "ocrd_browser/model/document.py", line 356, in page_for_id
    image, _, _ = self.workspace.image_from_page(pcgts.get_Page(), page_id)
  File "ocrd/workspace.py", line 419, in image_from_page
    page_image_info = self.resolve_image_exif(page.imageFilename)
  File "ocrd/workspace.py", line 271, in resolve_image_exif
    ocrd_exif = exif_from_filename(image_filename)
  File "ocrd_modelfactory/__init__.py", line 32, in exif_from_filename
    with Image.open(image_filename) as pil_img:
  File "PIL/Image.py", line 2878, in open
    fp = builtins.open(filename, "rb")
FileNotFoundError: [Errno 2] Datei oder Verzeichnis nicht gefunden: 'OCR-D-IMG/OCR-D-IMG_0001.tif'

This looks like we should chdir to the workspace directory after opening it (as all processors do; most core classes were written for them).

Thanks @hnesk!


feature request: Retain a single VFO

Well, going from two vfo`s to just one, when you turn the radio off and on again, it always starts in double vfo.
Would it be possible to retain the last setting before shutting down?
Thanks for all the work and effort spent on the firmware, it's fantastic.
CT1ESJ - Henrique

Use the menu to change rx mode to main only, then set your shortcut to change rx mode.
The shortcut functions are only for temporary settings.

Hello Henrique,
The display change is temporary. To make it permanent, use the RxMode menu (54/64).
Nevertheless, I understand your request.
That's why I've added the MAIN ONLY action. Most of the time I use double VFO (MAIN TX DUAL RX). But I've added a MAIN ONLY action to the F1 (Short) button. So I can quickly switch to single display without having to go into the menus.
73'
Armel F4HWN

Yes, well seen, I hadn't made this combination of menu and keys, so it doesn't need to take up more space in the firmware.
Thank you Armel and Prokryt, for helping me find the switch, sometimes that's all it takes to make light.

You're welcome ;)


CA: Perfect Het

🐛 Bug Report
Tombs of Amascut task checking for Theatre of Blood completion

Thank you - fixed here https://github.com/oldschoolgg/oldschoolbot/pull/5259


reference: add circle() and square()

closes #131 "New circle() and square() functions are missing in the reference"

Queuing this up now so I don't forget -- checks won't pass right now because the travis fix hasn't been merged on master, but when I tested this on top of those changes it passed integration. Let me know if you want it resubmitted after that merge.


[Suggestion] forge compatible

Can it be compatible with Sinytra connector? With this mod, you can play Marium's Soulslike Weaponry in Forge!

Soon an update for the forge version will come that makes it playable.


[py2py3] Fix some file-handling problems that arose with LogArch - src

Fixes #10569
Status
in dev
Description
Some usual juggling with file opening

[x] fixed test/python/WMCore_t/WMSpec_t/Steps_t/Executors_t/LogArch_t.py:otherLogArchiveTexst.addLogArchiveFile
[ ] still broken: test/python/WMCore_t/WMSpec_t/Steps_t/Executors_t/LogArch_t.py

If this gets too complicated to fix right now, i will wait and postpone this up until its turn in the schedule #10422, which is #10544
Is it backward compatible (if not, which system it affects?)
yes
Related PRs
nope
External dependencies / deployment changes
nope

This is getting tough. When I fix a thing, other two or three problems pop up. I would rather fix this kind of problems from the bottom up (i.e. following #10422). I will mention this issue/PR in the appropriate issue and make sure this will be followed up. If it's ok with you, we can even close with a comment this PR

Hi Dario, slice 13 also reports possible issues with unit tests in test/python/WMCore_t/WMSpec_t/Steps_t/Executors_t/LogArch_t.py. So I had a look at them this morning and there are 3 tests failing, but they are all marked as integration (thus, we do not know even if they pass in python2, likely not though!). I'd suggest to skip tests marked as such

@mapellidario Dario, is this PR still relevant? I believe I have already seen some of these changes getting merged in one of your recent PRs. If this is a work in progress, then I'm afraid there are conflicts that need to be sorted.

You are correct, you already included such changes in #10585 and these unittest is no longer failing in py3 (in jenkins, it is still failing in my machine both in py2 and py3)!
I will close this PR with this comments. Thanks for the changes and thanks for checking in!


problem with UserChangeForm

hello . in case create CustomUser from AbstractUser .
in UserChangeForm i need this :
def update(request):
if request.method == 'POST':
form = UserChangeForm(request.POST , instance=request.??????)
what i must insert instead ????????????   ?

Please use Stack Overflow for these types of questions. Thanks.


How to evaluate the trained model

The same question. How to evaluate the trained model....Is there any test script?

Hey @potato23333 @morgen-star ! Thank you for your question. I apologize for the late reply. There is no specific script for running the eval log. However, the function for evaluation is stand-alone and can be easily copy-pasted/imported in a standalone script. Code 1 and Code 2
Let me know if this unblocks you. I have put this as a potential item to work as part of future iterations.


Allow multiple connections

This will probably require to add an optional "pid" argument to every user function

Just in case it could be useful I just implemented an LDAP connection pool here: https://github.com/tanguilp/ldapoolex


feat: cherry pick option to bind nvmf target iface

feat: add option to bind nvmf target iface
Signed-off-by: Tiago Castro<EMAIL_ADDRESS>
feat: allow specifying the api version
Signed-off-by: Tiago Castro<EMAIL_ADDRESS>

bors merge


TabNavigator with bug on iOS

when i use TabNavigator,the first tab flex box will have bug,this is my code.

the same code will show different result in different tab,this is example

as you know , flex: 1 should seprate the view parent, but the first tab does not show normal,how can i solve this bug

who can help me?thanks

Just replace your stylesheet with
const styles = StyleSheet.create({ second: {     flexDirection: 'row',   },   firstButton: {     backgroundColor: 'red',     flexGrow: 1,     flexBasis: 0,   },   secondButton: {     backgroundColor: 'blue',     flexGrow: 1,     flexBasis: 0,   }, });
I hope it helps :)

Thank you for your answer, but it is not helpful.In the first tab, flex not work, i think this is bug on react-navigation.

I have solved this problem.I add width property on parent view .This is my code

It wastes a lot of time , hah~

I spend some time on searching bug but I didn't find nothing. I have also created same code as you and after replacing flex: 1 with flexGrow: 1, flexBasis: 0 everything works.

Hi! Since this issue has been quiet for a while I'm going to go ahead and close it. If the issue persists in the latest version of React Navigation please re-open!


Updates to use tracing component, allowing you to switch propagation

The latest version of Brave includes a tracing component which provides
the propagation implementation. This removes the hard-coding.
Note: this breaks the BraveTracer.wrap() signature because this library
is <1.0 and doesn't need the complexity of deprecation.

Theres another issue about binary. Main thing is that I'd like to capture
it in a test case and so far no one has a format they would use. Waiting on
that issue for an example format (though there is one in finagle)
On 23 Apr 2017 09:18, "Felix"<EMAIL_ADDRESS>wrote:

@felixbarny commented on this pull request.
In src/main/java/brave/opentracing/BraveTracer.java
https://github.com/openzipkin/brave-opentracing/pull/36#discussion_r112827793
:

@@ -56,27 +57,28 @@
* Returns an implementation of {@linkplain io.opentracing.Tracer} which delegates
* the the provided Brave Tracer.

maybe Brave Tracer Brave {@linkplain brave.Tracing tracing} component?
In src/main/java/brave/opentracing/BraveTracer.java
https://github.com/openzipkin/brave-opentracing/pull/36#discussion_r112827957
:

   this.brave4 = brave4;

 formatToPropagation.put(Format.Builtin.HTTP_HEADERS, Propagation.B3_STRING);

Is the Propagation.B3_STRING constant still available for me to use in
stagemonitor-zipkin? Or should I rather use Tracing#propagation? But can I
be sure that the propagation is always B3?
In src/main/java/brave/opentracing/BraveTracer.java
https://github.com/openzipkin/brave-opentracing/pull/36#discussion_r112828102
:

 return new Builder(brave4);

}
public static final class Builder {

brave.Tracer brave4;

Tracing brave4;
Map<Format, Propagation> formatToPropagation = new LinkedHashMap<>();

This is actually not regarding this pull request, but why limit format to
TextMap so that it is not possible to register a io.opentracing.propagation.
Format.Builtin#BINARY?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/brave-opentracing/pull/36#pullrequestreview-34173906,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD61zwIwQIoaxlOK-2z97_nv6-1fB6lks5rywlggaJpZM4NFDRg
.

going out as release-0.20.0 thx @felixbarny for the review


Activating a portal with Flint and steel using dispenser crashes the game

latest.log
debug.log
crash-2021-02-18_16.06.07-server.txt

Fixed Title -

(Should be) fixed


Add suspend and resume functionality

About this change - What it does
This PR is currently an in progress PR on the suspend/resume functionality. Its currently blocked because of the following issues

https://discuss.lightbend.com/t/committing-a-kafka-cursor-after-chunk-upload-on-multipart-s3-upload/8941
https://github.com/akka/alpakka/issues/2760
https://github.com/akka/alpakka/pull/2770
https://github.com/aiven/guardian-for-apache-kafka/issues/27 (we ended up having to use a killswitch to properly test this functionality however in production you should be using the controlled shutdown)

The PR is ready to review, the main notable change since the last comment is that the configuration for the timeslice has been changed (see TimeConfiguration)

LGTM but certainly @jlprat is needed here :-)

I just rebased the original commit, I forgot the change the reference.conf for the new Backup configuration. Ontop of this I also added tests (ConfigSpec) for this new configuration to make sure that the reference.conf is working.


Scraper for VD fails when using bash

It works with dash (default /bin/sh on Debian/Ubuntu):
VD
Scraped at: 2020-03-26T10:05:28+01:00
Date and time: 24.03.2020
Confirmed cases: 2234
Deaths: 21

but not with bash
VD
Scraped at: 2020-03-26T10:05:32+01:00
Date and time: Date\tHospitalisations
Confirmed cases: l'h\u00f4pital\tD\u00e9c\u00e8s\tTotal
Deaths: de

To be investigated.

@zdavatz Out of curiosity which OS, Distro are you using that defaults /bin/sh to bash?

BTW. It is probably easier to just port it to Python, like all the other scrapers, and not deal with this dash / bash difference. (it could be a bug in one of the shells tho. who knows).

@zdavatz Out of curiosity which OS, Distro are you using that defaults /bin/sh to bash?

Gentoo Linux ;)

OK. I was expecting that.
Looks like using echo -e will help.

Closing since I merged #236


Wording mistake with regards to public IP addresses?

There's a paragraph under Architecture (https://docs.microsoft.com/en-us/azure/architecture/reference-architectures/apis/protect-apis#architecture) that reads:

"In this scenario, API Management uses two types of IP addresses, public and private. Public IP addresses are for internal communication on port 3443, and for runtime API traffic in the external virtual network configuration. When API Management sends a request to a public internet-facing back end, it shows a public IP address as the origin of the request. For more information..."

Should this not read "...Public IP addresses are for external communication..." or rather "... Private IP addresses are for internal communication..."?

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

ID: d61c0c47-bb1e-5bd4-55ed-92058c3c1e2b
Version Independent ID: 09757b3e-205b-4263-fada-4d0224048f6f
Content: Protect APIs with Azure Application Gateway and Azure API Management - Azure Reference Architectures
Content Source: docs/reference-architectures/apis/protect-apis.yml
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @fabriciosanchez
Microsoft Alias: fabriciosanchez

Much thanks for the link to that doc! I was totally confused as to how communication within the VNet would work with APIM, i.e if an App Service within the VNet called an API that's not public facing, but I understand it a bit better now.


Fixes

This addresses #322 and #281
I think what I did in Mouse.ts is a hacked together version of floor, but I could be entirely wrong. It works

Why didn't you just use Math.floor()?
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Math/floor

Because I'm young and inexperienced :P
I'll change it.

Looks good to me! I updated the title so I remember to call this out as a breaking change in the release notes. I also associated a bounty with this - so please submit a claim on bountysource (https://www.bountysource.com/issues/43365739-inaccurate-clicking). Thanks for the fix!


feat(publisher): support profile for tiup publications

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


Ensure that non-finished relation objects are engraved in the correct order

In order to ensure that different objects are engraved in the expected order Lomse makes several passes across the system's list of pending objects and engraves object of one type in each pass. However this doesn't work for objects that are not finished in the current system as those are engraved after engraving all objects that are finished in the current system. This leads to an inconsistent engraving order if unfinished relation objects are involved, for example (note that the order for a wedge and an octave shift is inverted when the octave shift spans for two systems):

This pull request moves engraving of non-finished relation objects to the same loop that does engraving of other objects which makes the engraving order more consistent between those objects:

(test file: engraving-order-example.musicxml.txt)
Of course, the situation shown in the example is not really common, but having a consistent engraving order will greatly simplify things in future. This will ensure the correct engraving order if more object types are added (for example, to ensure the engraving order between wedges and pedal lines which are typically both placed below a staff) and will make it easier to introduce vertical alignment of certain element types (for example, consecutive volta brackets, pedals, combinations of dynamic marks and hairpins etc.)

Thank you very much for this PR.

but having a consistent engraving order will greatly simplify things in future.

Totally agree. This is an important change that should have been done time ago! Thank you.

While testing PR#314 I've noticed a regression error  (e.g lilypond test 31a-Directions.xml.txt ) caused by this PR that I overlooked when testing this PR. Now, after merging this PR,  wedges, volta brackets and 8va signs are placed below lyrics lines.
It is necessary to change priorities in the engraving priority list as I've seen that new elements supported after the list was created have been added at the end without more thinking as they didn't interfere. I will fix this.

Fixed in https://github.com/lenmus/lomse/commit/909763358da623faed900ff63a24c4c457146150

Thanks, I have indeed not reviewed the engraving order here. In addition to the changes in 909763358da623faed900ff63a24c4c457146150, I think another logical change can be placing dynamic marks and wedges to adjacent positions in the engraving order table (not sure where exactly though). They both refer to dynamics and should be placed vertically close to each other or can even be vertically aligned in some cases. This would be sometimes problematic if other objects are engraved between the engraving of dynamic marks and wedges.
I can do it in my next pull request, where I plan to propose an implementation of vertical alignment of some auxiliary and relation objects (as I have briefly mentioned, not only dynamics and wedges but also voltas and later pedals, perhaps also something like dashes object and its corresponding text). This alignment would only make sense in conjunction with a consistent engraving order for these types.

I have indeed not reviewed the engraving order here.

I neither. I'm still on travel and could not check engraving order in notation books. So I just moved some notations before the lyrics line. Please feel free to change the order as you think it is better.

I can do it in my next pull request, where I plan to propose an implementation of vertical alignment of some auxiliary and relation objects ...

That will be a great improvement! Thank you


Florida County Vaccinations

Scraper to track Florida vaccinations at the county level. Data Source.
Tracks:

original variable
category
measurement
unit

First Dose (yesterday)
total_vaccine_initiated
new
people

Series Complete (yesterday)
total_vaccine_completed
new
people

Total People Vaccinated* (yesterday)
total_vaccine_doses_administered
new
doses

First Dose (total)
total_vaccine_initiated
cumulative
people

Series Complete (total)
total_vaccine_completed
cumulative
people

Total People Vaccinated* (total)
total_vaccine_doses_administered
cumulative
doses

* column is the sum of First Dose and Series Complete
Issues:
-  PDF is not always updated regularly, so using _retrieve_datetimem1d() will sometimes give incorrect date. For example, the pdf has not been updated for today, so the data is from the 18th, but the date attributed to it is the 19th. I can work to extract the date either from the PDF, or the source webpage if needed.

Updated method for getting date.

Notes:
-  used tabula-py for extracting data from PDF, it is added to requirements.txt

Replaced tabula with camelot for extracting from PDF because tabula often failed. New method has worked/past tests for the past 10 days (that's as far back as I went.)

Great work! Nice scraping of a pdf -- Now hopefully they won't change it on us too much :sweat_smile:

Great work! Nice scraping of a pdf -- Now hopefully they won't change it on us too much :sweat_smile:


Charger Current Limit implementation

Hi, according to the the VOLVO Documentation there is an API to get the current limit charging Amperage:
https://developer.volvocars.com/apis/energy/v1/endpoints/charging-current-limit/
Can it be implemented ?

As said in #181, the new endpoints are not working at this time, sorry.


fix: sample app

add missing network xml link for Spotlight
report ttfd for root screen
use isar community fork dependency instead (it's more stable since it's being actively maintained and the original isar dep doesn't work in android release for some reason with compileSdk 34)
re-generate isar sample model

#skip-changelog
Closes #2261

@denrase can you approve pls


KAFKA-14884: Include check transaction is still ongoing right before append (take 2)

This is a redo of https://github.com/apache/kafka/pull/13608
Introduced extra mapping to track verification state.
When verifying, there is a race condition that the add partitions verification response returns that the partition is in the ongoing transaction, but an abort marker is written before we get to append. Therefore, we track any given transaction we are verifying with an object unique to that transaction.
We check this unique state upon the first append to the log. After that, we can rely on currentTransactionFirstOffset. We remove the verification state on appending to the log with a transactional data record or marker.
We will also clean up lingering verification state entries via the producer state entry expiration mechanism. We do not update the the timestamp on retrying a verification for a transaction, so each entry must be verified before producer.id.expiration.ms.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I'm getting tripped up on some flaky tests -- namely
kafka.api.TransactionsBounceTest.testWithGroupId()
kafka.api.TransactionsBounceTest.testWithGroupId()
kafka.api.TransactionsTest.testBumpTransactionalEpoch(String).quorum=kraft
kafka.api.TransactionsTest.testBumpTransactionalEpoch(String).quorum=kraft
Both involve restarting brokers, so I suspect I'm missing some logic there. testBumpTransactionalEpoch was failing before, but with a different error, so I've filed https://issues.apache.org/jira/browse/KAFKA-15099 for that.


Could libzip_all.c be updated to version >= libzip 0.11 to support UTF-8 file path?

The current version(0.9) is too old to support UTF-8 file path, could you kindly update the source update-to-date ?
I download the libzip-1.1.2.tar.gz, there are a lot of files , I can not simplely replace it with new file.
Thank you!

I believe that libzip-all.c is simply all the .c files in the .tar.gz merged together. Maybe you could merge those files together and open a PR?


interface conversion: interface {} is int, not string when trying to index slice type

Hello,
When trying to upgrade expr from 1.9 to the latest version in crowdsec, some of our automated tests failed with the following error:
WARN[09-03-2023 00:17:41] runtime error : interface conversion: interface {} is int, not string (1:27)
 | "<IP_ADDRESS>" in results[0].Overflow.GetSources()
 | ..........................^

After some debugging, I managed to narrow down the issue and reproduce it with the following code:
package main

import (
	"fmt"

	"github.com/antonmedv/expr"
)

type EventsSlice []Event

func (b EventsSlice) Len() int {
	return 42
}

type Event struct {
	Foo string
}

func main() {
	env := map[string]interface{}{
		"results": EventsSlice{{"bar"}},
	}

	code := `results[0].Foo`

	program, err := expr.Compile(code, expr.Env(env))
	if err != nil {
		fmt.Printf("error compiling expression: %s\n", err)
		return
	}

	fmt.Printf("%s", program.Disassemble())

	output, err := expr.Run(program, env)
	if err != nil {
		fmt.Printf("error running expression: %s\n", err)
		return
	}

	fmt.Println(output)
}

and gives this output:
0	OpLoadFast	0	results
1	OpPush	1	0
2	OpFetch
3	OpFetchField	2	{Foo [0]}
error running expression: interface conversion: interface {} is int, not string (1:8)
 | results[0].Foo
 | .......^

It seems  that if a slice type has a non-pointer receiver,  you cannot index it (if you change the Len() definition to be a pointer receiver, expr has no issue using the variable).
This issue is probably related to https://github.com/antonmedv/expr/issues/271, which you did not manage to reproduce at the time.
Go version: go version go1.20.1 linux/amd64

Thanks for the fix! Released v1.12.2 with a fix.


complete task 3 - CORE: GET /api, minor changes to task-2 tests

Hello.
changes:
add tests for GET /api
update app.js - get api, error handling
add controller & model
add test index to link all tests requires to 1 file, keep the code DRY.
update endpoints.json
update task2 test - changed body.msg to "ROUTE NOT FOUND"

Thanks for the feedback Nick, much appreciated. Changes made #5 .


Fix taskQueue to respect useLineageBasedSegmentAllocation in default taskContext

Description
useLineageBasedSegmentAllocation in default taskContext is always overridden currently.
This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

I'm catching up on the initial PR for this still. I understand what this PR is doing and it looks good to me. but as your comment says that there is a critical bug in the legacy way of doing this, do we really want to allow the override of the new value?

@capistrant yes, because it can be useful for the replacing rolling downgrade. In the replacing rolling downgrade, you can terminate nodes one at a time and add a new node that replaces with the terminated one. Suppose you have your cluster upgraded and a task running that uses the new segment allocation protocol. But a new critical bug is found in the new version and now you want to downgrade your cluster. In this case, you first need to wait until there is no running task that uses the new protocol before you start the downgrade process because every task must use the legacy protocol during the downgrade. To do this, you will need to either downgrade the overlord first or set useLineageBasedSegmentAllocation to false in the default taskContext. If you cannot downgrade the overlord first for some reason, you should be able to set it to false.

@capistrant yes, because it can be useful for the replacing rolling downgrade. In the replacing rolling downgrade, you can terminate nodes one at a time and add a new node that replaces with the terminated one. Suppose you have your cluster upgraded and a task running that uses the new segment allocation protocol. But a new critical bug is found in the new version and now you want to downgrade your cluster. In this case, you first need to wait until there is no running task that uses the new protocol before you start the downgrade process because every task must use the legacy protocol during the downgrade. To do this, you will need to either downgrade the overlord first or set useLineageBasedSegmentAllocation to false in the default taskContext. If you cannot downgrade the overlord first for some reason, you should be able to set it to false.

thanks for the explainer! LGTM


Do not return control batch records

Control messages are used to enable transactions in Kafka and are generated by the broker. Clients should not return control batches (ie. those with this bit set) to applications. (since <IP_ADDRESS>)

This change is

A clone of Pull Request https://github.com/dpkp/kafka-python/pull/2361

Sorry for the delay on this. I've been looking into fixing the failing pylint tests.


Update prisma-1-maintenace-fq12.mdx

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


fix safe_app_nodejs path in README

The path should app/node_modules/safe-app instead of app/node_modules/safe_app_nodejs.

Thanks for the pull request, and welcome! The MaidSafe team is excited to review your changes, and you should hear from @krishnaIndia (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTOR.md for more information.


Content style toggling

Partial for #1007.
The proposal here is to toggle B/I/U styles on the content and not on the attributes of the element. The reason for this is because element styles cannot be changed by the text editor, which would make styles inconsistent between text selection and the complete text element. From the user point of view, toggling Bold on the side panel for the frame, or cmd+B in the text editor are completely interchangeable.
With this tool, toggling styles becomes:
const {toggleFlag} = useContentHelper();
const updateContentStyleToggle = useCallback((flag, value) => {
  pushUpdates(({content}) => {content: setFlag(content, flag, value));
});
...
<BoldButton onClick={(value) => updateContentStyleToggle('b', value)}>
<ItalicButton onClick={(value) => updateContentStyleToggle('i', value)}>
...

And I propose we solve the "bold" problem this way:
const {content, fontWeight} = props;  // e.g. 100, 400, 700, etc.
const {setCustomStyles} = useContentHelper();
const boldWeight = Math.min(Math.max(fontWeight + 100, 700), 900);
if boldWeight > 700) {
  contentMarkup = setCustomStyles(content, 'b', {fontWeight: boldWeight});
}

Basically this will work simply like this:

A 100-600 fonts will get bold of weight 700. That's the default anyhow.
A 700 font will get bold of 800, a 800 font will get bold of 900.
A 900 font will always just get 900. I.e. bold toggle has no effect.

I liked this @dvoytenko. I guess this proposal also solves partially #413, WDYT?
Also, what about adding a way to extract font-weights? It will be useful to bind the Font Weight dropdown as multiple or only a specific one.
const extractFontWeightsFromContentSelected = (dom, tagName, defaultFontWeight) => {
  let fontWeights = []
  const nodeList = dom.querySelectorAll(tagName);
   for (let i = nodeList.length - 1; i >= 0; i--) {
     fontWeights = [
       ...fontWeights,
       parseInt(
        nodeList[i].style.fontWeight ? nodeList[i].style.fontWeight : defaultFontWeight
       )
     ];
   }
   return [...new Set(fontWeights)]
}

It's proposing to use only a delimited content selected, but it also can extract from the whole content. Thoughts?

I liked this @dvoytenko. I guess this proposal also solves partially #413, WDYT?

It's mostly unrelated to #413, tbh. That one is specifically only about using toggle buttons on selections. All the same issues can now be encountered by simply pressing cmd+B on the selection.

Also, what about adding a way to extract font-weights? It will be useful to bind the Font Weight dropdown as multiple or only a specific one.

Maybe. We can grow this set of utilities as needed. However, I don't remember the requirement to apply the font weight selector to selections (or font, or anything besides the toggles). I could be wrong. But I'd still first focus on the toggles.

However, I don't remember the requirement to apply the font weight selector to selections (or font, or anything besides the toggles). I could be wrong. But I'd still first focus on the toggles.

@pbakaus describes exactly that:

If you have any kind of text selection and change the weight via dropdown, everything in the selection switches to that weight.

I don't know I he considered it for all the other text manipulations, though. What about text alignment? Line height? Font family or size? That's suddenly a lot of extra complexity!

I do however think this is all too complex for us to handle now, so I say we punt all this to post-alpha. It's a completely new system we need here - I do however believe we can utilize draftjs, as it's very extensible as far as I know (FB OSS).
@pbakaus?

However, I don't remember the requirement to apply the font weight selector to selections (or font, or anything besides the toggles). I could be wrong. But I'd still first focus on the toggles.

@pbakaus describes exactly that:

If you have any kind of text selection and change the weight via dropdown, everything in the selection switches to that weight.

I don't know I he considered it for all the other text manipulations, though. What about text alignment? Line height? Font family or size? That's suddenly a lot of extra complexity!

I think this should work for other properties of text, and definitely for italic, strikethrough and the likes. I'm not sure if we should allow multiple line heights for aesthetic reasons though.

I do however think this is all too complex for us to handle now, so I say we punt all this to post-alpha. It's a completely new system we need here - I do however believe we can utilize draftjs, as it's very extensible as far as I know (FB OSS).
@pbakaus?

post-alpha seems fine - which is post Monday :)

I do however believe we can utilize draftjs, as it's very extensible as far as I know (FB OSS).

We can even explore more the RichUtils.toggleInlineStyle from draftjs.

I do however believe we can utilize draftjs, as it's very extensible as far as I know (FB OSS).

We can even explore more the RichUtils.toggleInlineStyle from draftjs.

RichUtils.toggleInlineStyle could work and we should definitely try. But I think that will follow a very similar style that I'm suggesting here.

@pbakaus

I think this should work for other properties of text, and definitely for italic, strikethrough and the likes. I'm not sure if we should allow multiple line heights for aesthetic reasons though.

I think the we are likely to have a lot of complicated nuances with all the different font styles, letter spacing, etc. But I think as soon as practical we need address the basic boolean toggles B/I/U. Mostly because they are broken now and it's very noticeable. More complicated styles could take longer.

@pbakaus

I think this should work for other properties of text, and definitely for italic, strikethrough and the likes. I'm not sure if we should allow multiple line heights for aesthetic reasons though.

I think the we are likely to have a lot of complicated nuances with all the different font styles, letter spacing, etc. But I think as soon as practical we need address the basic boolean toggles B/I/U. Mostly because they are broken now and it's very noticeable. More complicated styles could take longer.

Agree.


Doesn't work

Error running process: \Downloads\stable-diffusion-webui-master\stable-diffusion-webui-master\extensions\prompt_translator\scripts\main.py
Traceback (most recent call last):
File "\Downloads\stable-diffusion-webui-master\stable-diffusion-webui-master\modules\scripts.py", line 417, in process
script.process(p, *script_args)
TypeError: Script.process() missing 1 required positional argument: 'language'

there was a problem In the code. It has been fixed. Can you pull the latest version and try again?
I am on vacation and don't have access to a PC to test.

Fixed


Add more attributes to configure how segments are stored

The base unit of permission storage is the segment. Currently segments are only split by contexts. Adding more segment-specific attributes would allow using segments to control other aspects of permissions resolution. However, a lot of the internal resolution of segments would have to become more complex.
Possible additional segment attributes:

[ ] Until (timed permissions implementation)
[ ] Inherit (whether or not values set in this context will be inherited -- could be a 'levels-of-inheritance' limit. This is similar to 1.x's prefixing permissions with # -- would have an inheritance limit of 1 when applied to groups). Otherwise a boolean.
[ ] Weight -- replace the key-value mapping of permissions used now.
[ ] Transient -- replace the separate MemoryDataStore? possibly? or just merge segments from the data stores -- so api-side they're the same implementation they're not.

To implement these changes, some more decisions have to be made, especially about how the commands interface for these changes would work.

How can users edit a segment with the current ingame model for setting permissions. Possible more long options would make sense here, which could then match against existing segment or create new one if options did not exist
Would it make sense to have these appear the same as contexts for users? Contexts could become properties of a segment rather than a separate subobjects, just remove the keys that have PEX-specific functionality? This would mean that some keys would become invalid for contexts (but this is the same as another plugin using those contexts if the keys are not stripped)
Can performance be maintained? We already cache results, so the this is not as critical as it would be if segment resolution had to occur on every permission check, but still very important.
Would it be possible to use the matches() method of all ContextCalculators for the contexts rather than using the current combinations system? Does it make sense to register ContextCalculators for specific contexts? Could this be an addition to the ContextCalculator interface?

Related to #1577?


Bitbucket integration should use the slug to build URLs, not the name of the repo

Hi!
Currently the bitbucket integration uses the name of the repo to build up the clone, git and SSH urls. This breaks if the name of the repo includes whitespace or special characters since Bitbucket generates a slug from the name of the repo to build the URLs. Using the slug returned from the Bitbucket API instead would solve this.

this was fixed by @gtaylor