Initial README

Information about:

What this project is about.
Its goals.
Its values.
A basic vision of where this project should be going.

@eripe970 Thoughts?

Additional thoughts (that might need to be put in writing in README, too):

Two types of plugins:
Those that stores the migration states for each migration. Can be stored in plain file, in database, zookeeper whatnot. I've called these repositories for lack of a better word.
Those that execute migrations, be it running SQL executions, a script etc. For lack of a better word I've called these migrators.
I'm thinking of a plugin system like https://github.com/hashicorp/go-plugin.
Pros:

Will allow anyone to write their own plugins.
Simply drop in a plugin binary and it will automatically discovered by any-migrate.
Doesn't require recompiling any-migrate to get plugins in.
Doesn't require cgo or any dynamically linked library and everything that comes with that.

Cons:

Handling backward compatability etc.

Discussion points:

Are there too many states? Too few? I think testing migrations (if possible) and supporting reverts is important.

Like the idea with different plugins! I also think there could be a need for a reporting plugin (to graphite for example) if you would like to store your migration state in mysql or text file but still would like to push information to graphite when you did your migrations.
Some other things that crossed my mind.

Do we want to support migrations of migration states to different repositories? I think so. I'm thinking that users will start playing around with a textfile and then would like to move up to a database in an easy way.

Like the idea with different plugins! I also think there could be a need for a reporting plugin (to graphite for example) if you would like to store your migration state in mysql or text file but still would like to push information to graphite when you did your migrations.

I think it's a good idea, but that's maybe something that we can leave out until version 0.2? I created 0.1 and 0.2 milestones in the issues tab and assigned #6 to milestone 0.2.
This also made me think of logging, see #7.

Do we want to support migrations of migration states to different repositories? I think so. I'm thinking that users will start playing around with a textfile and then would like to move up to a database in an easy way.

I think that would be great! However, however I propose we leave that for milestone 0.2. I've created an issue for it here.


Improve how we decide on fallbacks

I've been thinking about how to improve how we decide whether or not to fall back on another function for a while, and this is what I've settled on.
@rosbif, what do you think of this?  You've been doing a lot of work on the fallbacks, so this probably effects you the most.
It's just a WIP, I haven't done the fallbacks to 256-bit functions yet.  Just wanted to get the idea out there and get some feedback.

I think that, in general, each "level" should only fall back to the next lower "level" which can provide an efficient implementation.
However sometimes (I think fairly rarely) it is more efficient to provide a "level n-2" implementation in addition to a "level n-1" implementation.
Off the top of me head I can only think of this recent example:
For SSE4.1 simde_mm_mask_mov_epi8 falls back to movm + blendv
For SSE2 it falls back to movm + and + andnot + or
In SSE4.1 I think that blendv should fall back to SSE2 setzero + cmpgt + and + andnot + or (but it doesn't).
Falling back directly to SSE2 would avoid an unnecessary setzero + cmpgt.
Not a big deal you'll say but I couldn't think of a better example.

Sorry I hadn't realized that you wanted a review of a specific proposal rather than just a discussion.
I don't think that your suggestion will permit deciding between two fallbacks with the same vector size, like my example above.
There is also maybe the case where a bitwise identical operation is defined at two different levels for different data types.
For instance I have never understood why
_mm256_xor_ps and _mm256_xor_pd are in AVX
but
_mm256_xor_si256 requires AVX2
Aren't they bitwise the same operation?
(Worse than this I can't see a use-case for xor on floats or doubles.
Possibly and/or if you are manipulating mantissas and exponents. But xor ???)


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3464

Questions ?
Please refer to the Backport tool documentation


[CT-2661] Release 1.5.2 (final) on June 22nd

@jelstongreen Good catch. We had more changes to go in, so we decided to put out a second release candidate on June 16. Assuming all goes well with RC testing, final release is planned for this week (June 22).

Could the docker image for this be published please 🙏🏼 https://github.com/dbt-labs/dbt-core/pkgs/container/dbt-core


Add https://www.searchftps.net/ to code section

Add https://www.searchftps.net/ to code section

@edoardottt  please verify and merge the request


String column names in [0, 1] are now no longer interpreted as colors

Matplotlib considers values like "0", "0.3" or "1.0" as valid grey-scale colors. Therefore, colors.is_color_like("0") returns True. This was causing an issue (see linked) for at least one user. This PR introduces a change so that these strings are no longer interpreted as colors.
(The failing tests are fixed in https://github.com/scverse/spatialdata-plot/pull/323)

Codecov Report
Attention: Patch coverage is 83.33333% with 3 lines in your changes missing coverage. Please review.

Project coverage is 79.26%. Comparing base (e97a7fc) to head (9f70cb0).
Report is 1 commits behind head on main.

Files
Patch %
Lines

src/spatialdata_plot/pl/utils.py
83.33%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #331      +/-   ##
==========================================
- Coverage   79.27%   79.26%   -0.01%
==========================================
  Files          11       11
  Lines        1785     1799      +14
==========================================
+ Hits         1415     1426      +11
- Misses        370      373       +3

Files
Coverage Δ

src/spatialdata_plot/pl/utils.py
68.04% <83.33%> (+0.15%)
:arrow_up:


Model Builder Error: The process cannot access temp file as being used by another process.

System Information (please complete the following information):

Windows OS: Windows-10-Enterprise-21H1
ML.Net Model Builder (Preview): <IP_ADDRESS>6604 (Main Branch)
Microsoft Visual Studio Enterprise 2019: 16.10.2
.Net: 5.0.301

Describe the bug

On which step of the process did you run into an issue: Data page
Clear description of the problem: This error has always been difficult to reproduce, and I see this error occasionally, and then when I want to reproduce it, this error dialog does not appear again. From the test records, this error has occurred since the <IP_ADDRESS>1904 of the main build.  Today, I meet this error again, find that it appears on the Image classification or Object detection scenarios. The steps listed below can sometimes reproduce this error, please take a look.

To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio 2019 start window;
Choose the C# Console App (.NET Core) project template with .Net 5.0;
Add model builder by right click on the project;
Select Image classification scenario;
Select Local (CPU) or Azure environment;
Navigate to Data page, sometimes below steps will meet the error.
a. Select the image folder, pop up the error dialog as below .gif .
b. Select one image folder, then change to another image folder.

Screenshots
.gif :

Object detection:

Additional context
Close the error dialog, can continue to train and other steps.

@vzhuqin  I am not able to reproduce these issues on our latest main.

@zewditu This problem is a bit of a headache, when you want to reproduce it, it just doesn't appear, but it will pop up inadvertently.
I just met this issue on latest main build: <IP_ADDRESS>7001, please take a look.
Repro steps:

Create one new Model Builder;
Select image classification scenario;
Select Local (CPU) environment;
Input the image folder, here not issues;
Go back to repeat steps 2-3;
Input the same dataset, pop up the model builder error dialog.

Error log:
ImageLocal_MLModel1-YZVSJO.txt

@LittleLittleCloud Any ideas what could be causing this one? I'm setting it as P2 and moving to triage.

@beccamc
Might be caused by update handler? I'm not sure...

Ahhh, yes that's a good guess. We're probably on a background thread writing to this file, open a file and kick off the same write operation, but can't because the previous is still running (and wasn't cancelled).
Similarly, I think there was a request from one of the "%temp% file too large" issues to make these file write operations stop on cancel. We should probably make the operation cancellable and make sure we cancel if we either 1) cancel training, or 2) pick a new dataset (dependent on what files are generated where).

Over a year with no updates on this issue or customers reporting. Given that it's flakey to reproduce I am closing.


Curb do not trigger on_debug call back

Hi,
web mock curb implementation do not trigger on_debug callback ... because of that i did this work in initializers...
spec/support/webmock_curb_fix.rb
# frozen_string_literal: true
if defined?(Curl) && defined?(WebMock)
  module Curl
    WebMockCurlEasy.class_eval do
      def on_debug
        yield if block_given?
      end
    end
  end
end

@ravenesi any idea why it doesn't work with webmock?

@bblimke indeed our WebMock implementation do not implement proper on_debug callback other callbacks are there i can create PR is is it needed :)

a PR would be very welcome :)


Better contrast with content-links and background

The black color of content links against the dark background in the effect descriptions makes the links difficult to read. I think using the navajowhite that's used in the heading for the color of the links works well.

I am not seeing what you are seeing.

This is what I'm seeing.

It's in the SWADE system, but not with modules disabled. But that CSS selector in the visual effects CSS file is applying a black color to .content-link regardless.

This image is taken using only VAE, with SWADE v3.2.5.

I understand. This is content being pulled from a content module over which I have no control. Regardless, VAE's CSS is applying a black color to the .content-link class in the module's CSS. Is that style definition addressing a specific need?

The text in effect descriptions is set to a brighter color to be readable on the dark background. This also affects the content links, so the black was meant to reset this.
Testing in both swade and dnd5e, using color: initial; instead does the same trick. I'm prepared to make that change if that solves it for you. You can test it yourself in the console.

Setting it to initial seems to bring it back to #ddd as defined in the definition for the selector section.visual-active-effects .effect-item .effect-info .effect-info-details :is(.effect-intro, .collapsible-content, .vae-buttons). Using revert brings it to the same color as the text it seems.
If it's too much trouble to resolve, I'm fine with just overriding it with the rest of my custom styles using the Custom CSS module.

I don't know what module it is you are using, so I cannot test the interaction myself. Where you should be testing it is by editing the css of VAE. I.e., change it from color: black to color: initial, not adding a new style.

I didn't add a new style. I edited the definition in your file via dev tools.

Then I am afraid I have no solution. I cannot conditionally apply css depending on whether a module is removing the default border and background of content links.

It's fine. Like I said. I can inject custom CSS myself.


Copy-paste broken

Issue Type: Bug

Open vscode
Type "copy me"
Select "copy me"
Copy using right click, cmd+v or from the command palette and watch it slow down significantly ([Violation] 'keydown' handler took 1482ms showing in dev-tools when using cmd+v)
Paste and nothing gets pasted

VS Code version: Code 1.24.0 (6a6e02cef0f2122ee1469765b704faf5d0e0d859, 2018-06-06T17:30:30.113Z)
OS version: Darwin x64 15.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) m5-6Y54 CPU @ 1.10GHz (4 x 1200)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
8.00GB (0.44GB free)

Process Argv
/Users/bob/Downloads/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

vsc-material-theme
Equ
2.1.0

vscode-github
Kni
0.28.1

vscode-eslint
dba
1.4.12

prettier-vscode
esb
1.5.0

python
ms-
2018.5.0

sublime-keybindings
ms-
4.0.0

vscode-fileutils
sle
2.9.0

ayu
tea
0.14.0

gitblame
wad
2.4.2

change-case
wma
1.0.0

(2 theme extensions excluded)

It probably does count as a duplicate of the previous issue, but it wasn't happening at all for me in v1.23.1. Let me know if you need any more information.

I think that this was a system-wide issue that seemingly occurred at exactly the time I upgraded. I've restarted and everything is fine. Sorry for the noise!


[v1.x] Specify Arm CD ARMPL include path

In the cd arm pipeline it looks like cmake is finding the wrong header for amrpl (integrated with mkldnn)
https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-mxnet-cd%2Fmxnet-cd-release-job-1.x/detail/mxnet-cd-release-job-1.x/2035/pipeline/88
[2021-06-17T20:00:16.022Z] -- Looking for sgemm_ - found
[2021-06-17T20:00:16.022Z] -- Found BLAS: /opt/arm/armpl_21.0_gcc-8.2/lib/libarmpl_lp64_mp.so;/usr/local/gcc-8.5.0/lib64/libgfortran.so;/opt/arm/armpl_21.0_gcc-8.2/lib/libamath.so;/usr/lib/aarch64-linux-gnu/libm.so
[2021-06-17T20:00:16.022Z] -- Looking for cblas_sgemm
[2021-06-17T20:00:16.022Z] -- Looking for cblas_sgemm - found
[2021-06-17T20:00:16.022Z] -- Found CBLAS: /opt/arm/armpl_21.0_gcc-8.2/lib/libarmpl_lp64_mp.so;/usr/local/gcc-8.5.0/lib64/libgfortran.so;/opt/arm/armpl_21.0_gcc-8.2/lib/libamath.so;/usr/lib/aarch64-linux-gnu/libm.so
[2021-06-17T20:00:16.022Z] -- CBLAS include path: /work/mxnet/staticdeps/include

So it appears this line in the mkldnn repo caused the problem
https://github.com/oneapi-src/oneDNN/blob/83ebc40d86bc54f0f23e947235e53570eeacf254/cmake/blas.cmake#L75
After changing
find_path(BLAS_INCLUDE_DIR ${CBLAS_HEADERS} $ENV{CPATH} ${BLAS_LIB_DIR}/../include ${BLAS_LIB_DIR}/../../include)

to
find_path(BLAS_INCLUDE_DIR ${CBLAS_HEADERS} HINTS $ENV{CPATH} ${BLAS_LIB_DIR}/../include ${BLAS_LIB_DIR}/../../include)

CMake can find the correct header file. But without changing MKLDNN code we can also set BLAS_INCLUDE_DIR on our side just for the cd armpl build

Should this be fixed upstream too?

Should this be fixed upstream too?

reported to oneDNN here https://github.com/oneapi-src/oneDNN/issues/1100


Fix doc links

Motivation
Fixes #1495
Solution

Reference OutputSelection with correct module scope.
Move #[cfg(all(feature = "svm-solc"))] to top of compile_* methods; this enables Project::svm_compile() to be in scope within the documentation for the method.

PR Checklist

[ ] Added Tests (N/A)
[ ] Added Documentation (N/A)
[ ] Updated the changelog

No worries! Let's go with your PR as it fixes without modifying actual source code. Also, the fix in documentation makes more sense semantically.


Issue with Allstar Branch Protection Enforcement (404 errors upon action: fix)

Hello,
Over the past couple of months, I've noticed a recurring issue where Allstar returns 404 error messages when attempting to enforce branch protection, as shown in the logs below, causing the application to crash. The issue is still there with Allstar latest version (4.2)
allstar {"severity":"INFO","org":"Org","repo":"test0","area":"bot","enabled":true,"time":"2024-08-20T08:29:17Z","message":"Check repo enabled"}
allstar {"severity":"INFO","org":"Org","repo":"test0","area":"Binary Artifacts","enabled":false,"time":"2024-08-20T08:29:17Z","message":"Check repo enabled"}
allstar {"severity":"INFO","org":"Org","repo":"test0","area":"Binary Artifacts","result":true,"enabled":false,"notify":"","details":{"Artifacts":null},"time":"2024-08-20T08:29:17Z","message":"Policy run result."}
allstar {"severity":"INFO","org":"Org","repo":"test0","area":"Branch Protection","enabled":true,"time":"2024-08-20T08:29:17Z","message":"Check repo enabled"}
allstar {"severity":"INFO","org":"Org","repo":"test0","area":"Branch Protection","result":false,"enabled":true,"notify":"No protection found for branch master\n","details":{"master":{"PRReviews":false,"NumReviews":0,"DismissStale":false,"Bl
allstar {"severity":"ERROR","error":"PUT [https://api.github.com/repos/Org/test0/branches/master/protection](https://api.github.com/repos/Org/test0/branches/master/protection): 404 Branch protection has been disabled on this repository. []","time":"2024-08-20T08:29:18Z","message":"Unexpected error running policies."}
allstar {"severity":"INFO","area":"bot","count":7,"results":{"CODEOWNERS":{"totalFailed":1},"OpenSSF Scorecard":{"totalFailed":1},"Repository Administrators":{"totalFailed":1}},"time":"2024-08-20T08:29:18Z","message":"EnforceAll complete."}

After investigating, I found that this issue occurs when branch protection rules are disabled on a repository. I also noticed in the GitHub API documentation that while there is an API to update branch protection, there doesn’t seem to be one to set it initially. Could this be a new limitation from GitHub?

Currently, I’ve implemented a custom alerting system to manually re-enable branch protection rules when Allstar fails, but this is not an ideal solution. Is there a better workaround that you would recommend?
For reference, here are the current permissions for my Allstar GitHub App.

Thanks for bringing this up! I didn't know about this "disable branch protection" option. It looks like new repos have branch protection enabled by default, but just with empty rules. So that the "fix" option will just add the correct protection and everything works as long as a user didn't use this disable option.
If there is a way to re-enable via API, then we definitely want Allstar to just do that. I did a quick look through the REST docs as you did, and also the GraphQL docs, and didn't see any way to query or enable this either. It might be impossible, or might just take more digging.
Shorter term. We can update the code to expect the 404 on some repos, and just log a Warning and continue without exiting the enforcement loop.

Hi Jeff,
It would be great if you could take a look at this. If not, I can try submitting a PR to handle the 404 error by setting an exception. I believe it would go somewhere in this function: branch.go#L294-L313.
Thanks for your help!

@jeffmendoza I opened a support ticket at GitHub following the missing API and got the below answer:
Your ticket was transferred to the Ecosystem team since it's more in line with our scope of support. We agree that we should have an API endpoint for this; however, our engineering team have suggested you whatever tooling you are using should migrate to rulesets. We announced rulesets last year in this blog post, and while we don't have fixed plans to sunset branch protection rules yet, that's something we're considering.
Regardless, we'll still raise this as customer feedback since the limitation is causing you significant friction, but we can't say if or when such functionality may be available. We could only recommend that you keep an eye on the GitHub Blog and the Changelog for related updates. We understand this may be frustrating and wish we had better news to share regarding a workaround, but we're afraid this just simply isn't possible at the moment with our available APIs.
That raises the priority of this Feature Request: https://github.com/ossf/allstar/issues/475


Session Issue while running through docker

I have setup the cachet locally it works fine. But when I do the same using docker then session is not persistent. I am able to fetch someones others account in Cachet. Sometimes if someones login one account gets automatically logouts.

should have been fixed with #311


TestListAPIGatewaysV2 Test Failure in Circle CI

        	Error Trace:	/home/circleci/project/aws/apigatewayv2_test.go:38
        	            				/home/circleci/project/aws/apigatewayv2_test.go:60
        	Error:      	Could not create test API Gateway (v2): %s
        	Test:       	TestListAPIGatewaysV2
        	Messages:   	BadRequestException: HTTP APIs are not available in this AWS region

Seems like API Gateway V2 (HTTP APIs) are not available in all regions. https://aws.amazon.com/blogs/compute/announcing-http-apis-for-amazon-api-gateway/#:~:text=The API V1 namespace represents,via the AWS Management Console.
Will limit this test to us-east-1, us-east-2, and us-west-1 for now.


#16 Written error class in typescript

This branch contains apollo-error code in Typescript

@thebigredgeek have made the requested changes, please review and let me know if there are any other


[BGE-16] Adjust cooldown feature to reflect Global Cooldown on abilities

Blockgame has introduced a global cooldown of 0.5s on abilities. The cooldown system in the mod does not reflect this global cooldown and thus it can look like inputs are being dropped. A 0.5s cooldown visualization should appear on all ability-based items.
From SyncLinear.com | BGE-16

Present in v1.2.5.


[ZEPPELIN-5242] Change /interpreter/ to interpreter/**

What is this PR for?
This PR corrects the rat exclusion **/interpreter/** which is too wide. I think we should only exclude the interpreter folder from the root path.
What type of PR is it?

Bug Fix

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5242

How should this be tested?

CI

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@zjffdu Any thoughts on this amended exclusion rule?

LGTM

I will merge this PR into master and branch-0.9 on Wednesday (10/02/2021) if no further comments are received.


Window opens on a different monitor

If the main window is open on a second monitor and the Trade Management window is opened, it opens on the main monitor. Please open the window in the center of the main window. As a workaround, move the window there after opening. Please note this for all windows.

Confirmed on my multi-monitor setup also. I will fix and report back when the code is corrected.

Implemented a monitor aware centering function AfxCenterWindowMonitorAware() so hopefully all popup windows will now display on the correct monitor.

Works as expected. Thank you.


Include sample17.cshtml instead of inline code

Meant to do this originally, oops!
Thanks for merging my previous pull request; sorry for this being separate :sweat_smile:

@james1293,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@james1293  Thanks!


Miner Port

My miner port is 3032 but when I point my miner it seems to not be working. Any advise would be fantastic! Using a Linux based Claymore miner.

firewall??


Allow download of dependencies from internal sources

Is your feature request related to a problem? Please describe.
We currently work in a sealed off environment with our local Artifactory that mirrors external sources for us. There is no direct connection to the outside otherwise. We're trying to build a docker container containing the bi client, but it currently hardcodes the dependency repo - https://github.com/jfrog/build-info-go/blob/main/utils/dependenciesutils.go#L37
Describe the solution you'd like to see
It would be nice to have the dependency repo as a parameter we can set.
Describe alternatives you've considered
There are no workarounds because the value is hardcoded in the source file.

@dg424,
We recommend using JFrog CLI's jf docker pull command to pull docker images from Artifactory.

It is not a docker image. bi-cli mvn option tries to download a file directly from the internet - the line I mentioned. The ask is how to get bi-cli to use our local Artifactory instead ? Note that this is a jar file download as we're currently testing the maven/mvn option.

@ddag,
JFrog CLI uses build-info-go as a depdemcy and allows running the `jf mvn' command to resolve maven from Artifactory as well as generating build-info.

Ok, I had to do jf mvn-config first which created .jfrog/projects/maven.yml. After this, the build worked and the jar was deployed to our artifactory server, but build info was not published and I got this:
[main] INFO org.jfrog.build.extractor.maven.BuildDeploymentHelper - Artifactory Build Info Recorder: publish build info set to false, build info will not be published...

@dg424,
Here's how you can generate build-info and publish it to Artifactory for your maven build with jfrog-cli:
jf mvn install --build-name b1 --build-number 1
jf rt bp b1 1

You can read more about this in the JFrog CLI documentation. Let us know if you have follow-up questions.


Fix loss of precision warnings

What problem is this PR intended to solve?
There are multiple integer loss of precision warnings.
There are three categories of warnings fixed.

Most are spurious warnings where we cast an int to a long and then use INT2NUM rather than just avoiding the cast);
With the HTML5 parser and a file containing an error on line number 2^31 or higher (or column number 2^31 or higher), the returned errors will have the wrong line (resp. column) number.
Node pointers are cast to a ints and used as ids for node comparison. Two different nodes could have the same least significant 32 bits in their pointers which would cause them to compare equal.

Have you included adequate test coverage?
I added no test coverage. The warnings in the first category are harmless. The ones in the second category require producing input with at least 2^31 bytes. This doesn't seem worth it. Testing the third category seems extremely difficult.

Does this change affect the behavior of either the C or the Java implementations?
Yes! It fixes the behavior of two different nodes possibly comparing equal in the C implementation. It also fixes incorrect line/column number for errors in pathological cases.

I prepended a commit that updates extconf.rb to set gcc or clang flags for integer precision warnings.
The changes here also break the truffleruby build, something that shouldn't be a blocker but that I want to understand before merging.

Undoing that last commit for now. Will investigate the truffleruby breakage and then circle back on #2692

I also noticed that the test suite only exercises html5lib-tests fragment cases by using the tag name (passing context as a string) and not by passing context as an instantiated node. Would it be worthwhile to do both, since the majority of real-world cases are going to be using a context node (e.g., Node#fragment)?

Yes. We should definitely be testing with a context node. The specific context node mostly doesn't matter (beyond the type), but it certainly does in this case.

Yes. We should definitely be testing with a context node

Ok, I'll open a new issue so we don't forget to do that.

@eregon In this fix, we're using rb_unit2big to convert a pointer into a VALUE that we can use for equality comparisons. That function doesn't exist in truffleruby. Do you have any suggestions on portable ways to do this?

rb_uint2inum is implemented on TruffleRuby and has the same signature.
My understanding is rb_uint2big is not recommend since that assumes it creates a Bignum but the value might be in Fixnum range, and what the docs say: https://github.com/ruby/ruby/blob/6dd1a5f532711a28ffe815455503f5a2b1f70d23/include/ruby/internal/arithmetic/intptr_t.h#L60-L61. TruffleRuby does not allow creating Bignums in Fixnum range because that's wasteful and inefficient, so cannot really implement that C API function.

It's probably the case on every platform that Nokogiri (and probably Ruby) works on that the size of a size_t is the same as the size of an uintptr_t. It maybe sufficient to use SIZET2NUM((size_t)(uintptr_t)node).

Oh I missed @eregon's comment about rb_uint2inum(). We should just try that instead.

OK, changed it to rb_uint2inum, rebased, and force-pushed.


ENSEMBL genes are not being merged together

As noted in https://github.com/NCATSTranslator/Feedback/issues/340, Babel has multiple distinct Ensembl gene identifiers for the TNF gene:

NCBIGene:7124 (which is combined with ENSEMBL:ENSG00000232810, ENSEMBL:ENSP00000365290, ENSEMBL:ENSP00000365290.3 and UniProtKB:P01375, among others)
ENSG00000204490
ENSG00000206439
ENSG00000223952
ENSG00000228321
ENSG00000230108
ENSG00000228849
CHEMBL1825

Here are the RENCI-dev results: https://nodenormalization-sri.renci.org/1.3/get_normalized_nodes?curie=NCBIGene:7124&CHEMBL1825&curie=ENSEMBL%3AENSG00000230108&curie=ENSEMBL%3AENSG00000228849&conflate=true

I haven't had a chance to dig deeply into this yet, but it looks like the NCBIGeneENSEMBL concord is built from babel_downloads/NCBIGene/gene2ensembl.gz, which only maps NCBIGene:7124 to ENSG00000232810:
gene2ensembl.gz:9606	7124	ENSG00000232810	NM_000594.4	ENST00000449264.3	NP_000585.2	ENSP00000398698.2

However, babel_downloads/ENSEMBL/hsapiens_gene_ensembl/BioMart.tsv has many more mappings, including to the identifiers specified above, which appear to all link to HGNC:11892:
havana	TNF	ENSP00000389265	DIF	CHR_HSCHR6_MHC_APD_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228978
havana	TNF	ENSP00000389265	TNF-alpha	CHR_HSCHR6_MHC_APD_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228978
havana	TNF	ENSP00000389265	TNFA	CHR_HSCHR6_MHC_APD_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228978
havana	TNF	ENSP00000389265	TNFSF2	CHR_HSCHR6_MHC_APD_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228978
ensembl_havana	TNF	ENSP00000365290	DIF	CHR_HSCHR6_MHC_COX_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000204490
ensembl_havana	TNF	ENSP00000365290	TNF-alpha	CHR_HSCHR6_MHC_COX_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000204490
ensembl_havana	TNF	ENSP00000365290	TNFA	CHR_HSCHR6_MHC_COX_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000204490
ensembl_havana	TNF	ENSP00000365290	TNFSF2	CHR_HSCHR6_MHC_COX_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000204490
ensembl_havana	TNF	ENSP00000389490	DIF	CHR_HSCHR6_MHC_MCF_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000223952
ensembl_havana	TNF	ENSP00000389490	TNF-alpha	CHR_HSCHR6_MHC_MCF_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000223952
ensembl_havana	TNF	ENSP00000389490	TNFA	CHR_HSCHR6_MHC_MCF_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000223952
ensembl_havana	TNF	ENSP00000389490	TNFSF2	CHR_HSCHR6_MHC_MCF_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000223952
ensembl_havana	TNF	ENSP00000410668	DIF	CHR_HSCHR6_MHC_DBB_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228849
ensembl_havana	TNF	ENSP00000410668	TNF-alpha	CHR_HSCHR6_MHC_DBB_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228849
ensembl_havana	TNF	ENSP00000410668	TNFA	CHR_HSCHR6_MHC_DBB_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228849
ensembl_havana	TNF	ENSP00000410668	TNFSF2	CHR_HSCHR6_MHC_DBB_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228849
ensembl_havana	TNF	ENSP00000392858	DIF	CHR_HSCHR6_MHC_MANN_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228321
ensembl_havana	TNF	ENSP00000392858	TNF-alpha	CHR_HSCHR6_MHC_MANN_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228321
ensembl_havana	TNF	ENSP00000392858	TNFA	CHR_HSCHR6_MHC_MANN_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228321
ensembl_havana	TNF	ENSP00000392858	TNFSF2	CHR_HSCHR6_MHC_MANN_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000228321
ensembl_havana	TNF	ENSP00000389492	DIF	CHR_HSCHR6_MHC_SSTO_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000230108
ensembl_havana	TNF	ENSP00000389492	TNF-alpha	CHR_HSCHR6_MHC_SSTO_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000230108
ensembl_havana	TNF	ENSP00000389492	TNFA	CHR_HSCHR6_MHC_SSTO_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000230108
ensembl_havana	TNF	ENSP00000389492	TNFSF2	CHR_HSCHR6_MHC_SSTO_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000230108
ensembl_havana	TNF	ENSP00000372988	DIF	CHR_HSCHR6_MHC_QBL_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000206439
ensembl_havana	TNF	ENSP00000372988	TNF-alpha	CHR_HSCHR6_MHC_QBL_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000206439
ensembl_havana	TNF	ENSP00000372988	TNFA	CHR_HSCHR6_MHC_QBL_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000206439
ensembl_havana	TNF	ENSP00000372988	TNFSF2	CHR_HSCHR6_MHC_QBL_CTG1	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000206439
ensembl_havana	TNF	ENSP00000398698	DIF	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810
ensembl_havana	TNF	ENSP00000398698	TNF-alpha	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810
ensembl_havana	TNF	ENSP00000398698	TNFA	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810
ensembl_havana	TNF	ENSP00000398698	TNFSF2	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810
ensembl_havana	TNF	ENSP00000514308	DIF	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810
ensembl_havana	TNF	ENSP00000514308	TNF-alpha	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810
ensembl_havana	TNF	ENSP00000514308	TNFA	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810
ensembl_havana	TNF	ENSP00000514308	TNFSF2	6	protein_coding	HGNC Symbol	7124.0	tumor necrosis factor [Source:HGNC Symbol;Acc:HGNC:11892]	ENSG00000232810

So those might be the missing mappings we need to include in the NCBIGeneENSEMBL concord.

It looks like there is some code in Babel for generating an ENSEMBL concord:
https://github.com/TranslatorSRI/Babel/blob/f3748b881082f7f573409e8e75822cd02b6becb5/src/snakefiles/gene.snakefile#L69-L75
https://github.com/TranslatorSRI/Babel/blob/f3748b881082f7f573409e8e75822cd02b6becb5/src/createcompendia/gene.py#L25-L74
However, this code is not currently being run, because gene_concords doesn't include "ENSEMBL":
https://github.com/TranslatorSRI/Babel/blob/f3748b881082f7f573409e8e75822cd02b6becb5/config.json#L19
@cbizon Do you know if this was deactivated deliberately? I'm currently trying to re-run Babel after adding "ENSEMBL" to the list of concords to see if the concord can be generated correctly and if it includes the mappings we're looking for.

I seem to recall that the ensembl mappings led to some very unpleasant merges.  We'll want to be careful with them.


remove hash from redirectUrl

https://github.com/SolidOS/solid-logic/issues/60

NSS tested locally with Watch.
Issues resolved


New Tests & Error Logging

Improved Error Logging
Improved server responses
Add tests for ExamServlet
Add test for ExamServletResponse
Fix Checkstyle warnings

lgtm


Edit pivot attribute bug in v2.15.40

Package
filament/filament
Package Version
v2.15.40
Laravel Version
v9.27.0
Livewire Version
v2.3
PHP Version
PHP 8.1
Problem description
After upgrading to v2.15.40 pivot attributes are not updating when editing a relation in RM
If downgrading to v2.15.39 there is no issue
Expected behavior
Update notes pivot attribute
Steps to reproduce
Edit a record in RM
Reproduction repository
https://github.com/molcsab/pivotbug
Relevant log output
No errors, but the query in debugbar looks like this:
update `role_user` set `notes` = 'test' where `user_id` is null and `role_id` is null

@molcsab, could you test if this comment fixes your issue? https://github.com/filamentphp/filament/pull/3789#issuecomment-1235511597


Documentation Page Index redirect

If you access /docs URI you'll get a list of pages.
Redirect https://pts-project.org/docs/ -> https://pts-project.org/docs/prologue/introduction/

The same thing happens if you access other top level sections URLs such as:
https://pts-project.org/docs/colander/
https://pts-project.org/docs/pirogue/


Removed /dist from bower.json:ignore

Otherwise /dist/graphlib.js could not be reached only by bower install only

dist/graphlib-core.js should be used. dist/graphlib.js includes dependencies which should not be bundled in the bower distribution of graphlib.


Fix #769 'cc' and 'S' auto indent issue

This should fix the auto indent issue.
I just read this part of source code, and -[XVimDeleteEvaluator motionFixed:] is the only caller of -xvim_change: , so this change may not affect other functions in my opinion.

Please write test case for this to merge. Thank you.

It may fail in few cases, don't merge for now

Done. No failed cases anymore.

:+1:


WIP: BUG: fix inconsistent scope for tensorflow.python.layers.base.Layer

It is proposed to fix #13429, however, I'm not sure whether it's the best solution. I'll finish the PR when I get back from holiday (next week).
How to test

[ ] add test case.
[ ] pass all tests.

Can one of the admins verify this patch?

Jenkins, test this please.

@facaiy Can you rebase to master to resolve the conflict?

Adding Eugene and Francois as they've been looking at Layers code more recently (I think I skipped some changes).

Hi, @frankchn , it seems that master introduces a big change in variable_scope.py. I'll test nightly version. If bug still exists, I'll rebase master.
@lukaszkaiser The bug is reported on tf.layer at first, however, I find that it seems from a little mistake in variable_scope.

Could you make a smaller PR that only changes variable_scope first? I'd much rather do it in a 2-step way, as these changes can break things in a subtle way and it can be hard to figure out what broke what.

The case is more subtle than I expected, so let's close the PR now and go to #13429 for discuss. I'll reopen the PR until the best solution is found.


Reserved word fix + Sleep time fix

Removed the 1000 multiplier from getxTime() functions since they return the result in miliseconds already.

altered "hackableServers" var name to "vulnerableServers" due to reserved word bug in some ocasions.

Thanks, Luis.  Definitely needed or the code takes 1000x times as long.

Great fix, confirmed that this works for me on the latest steam version (1.3.0) along with the web version (1.2.0).

The script sleeps a ridiculously long time (literally hours) even after installing augments and starting over on the first set of businesses. I don't fully understand it but it appears to want to sleep for 12+ hours before it does anything.
'mainHack.ns: [4:52:09 PM] Starting mainHack.ns
mainHack.ns: [4:52:09 PM] Selected rho-construction for a target. Planning to weaken the server. Will wake up around 6:28:09 AM
mainHack.ns: [4:52:09 PM] Stock values: baseSecurity: 42; minSecurity: 14; maxMoney: $13,184,119,700
mainHack.ns: [4:52:09 PM] Current values: security: 22.78; money: $13,184,119,700
mainHack.ns: [4:52:09 PM] Time to: hack: 09:23:59; grow: 10:52:47; weaken: 13:35:59
mainHack.ns: [4:52:09 PM] Delays: 04:11:29 for hacks, 02:42:56 for grows
mainHack.ns: [4:52:09 PM] Cycles ratio: 35885 grow cycles; 3297 weaken cycles; expected security reduction: 164.85'

Glad I saw this, I was about to deep dive the scripts to figure out why they weren't working.


TWO Separate Controls - Still Issues

Hi,
this morning I noticed that now the Koolnova 1 is not available, I attach both screenshots and log.
Thank you

home-assistant-2.log

Hi,
what append since the last time ? errors occur after a simple HA restart ?
In the log, it's not koolnova 1 but koolnova 2.
Line 367: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Device-sensor already exists - ignoring sensor.koolnova_2_device
Line 368: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Address-sensor already exists - ignoring sensor.koolnova_2_address
Line 369: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Modbus-RTU-sensor already exists - ignoring sensor.koolnova_2_koolnova_2_modbus_rtu
Line 370: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC1-throughput-sensor already exists - ignoring sensor.koolnova_2_engine_ac1_throughput
Line 371: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC1-temp-order-sensor already exists - ignoring sensor.koolnova_2_engine_ac1_temperature_order
Line 372: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC2-throughput-sensor already exists - ignoring sensor.koolnova_2_engine_ac2_throughput
Line 373: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC2-temp-order-sensor already exists - ignoring sensor.koolnova_2_engine_ac2_temperature_order
Line 374: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC3-throughput-sensor already exists - ignoring sensor.koolnova_2_engine_ac3_throughput
Line 375: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC3-temp-order-sensor already exists - ignoring sensor.koolnova_2_engine_ac3_temperature_order
Line 376: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC4-throughput-sensor already exists - ignoring sensor.koolnova_2_engine_ac4_throughput
Line 377: 2024-11-20 14:53:51.333 ERROR (MainThread) [homeassistant.components.sensor] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC4-temp-order-sensor already exists - ignoring sensor.koolnova_2_engine_ac4_temperature_order
Line 378: 2024-11-20 14:53:51.334 INFO (MainThread) [homeassistant.components.select] Setting up koolnova_bms.select
Line 379: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.select] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Global-HVACMode-select already exists - ignoring select.koolnova_2_global_hvac_mode
Line 380: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.select] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Global-HVACEff-select already exists - ignoring select.koolnova_2_global_hvac_efficiency
Line 381: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.select] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC1-State-select already exists - ignoring select.koolnova_2_engine_ac1_state
Line 382: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.select] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC2-State-select already exists - ignoring select.koolnova_2_engine_ac2_state
Line 383: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.select] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC3-State-select already exists - ignoring select.koolnova_2_engine_ac3_state
Line 384: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.select] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Engine-AC4-State-select already exists - ignoring select.koolnova_2_engine_ac4_state
Line 385: 2024-11-20 14:53:51.334 INFO (MainThread) [homeassistant.components.switch] Setting up koolnova_bms.switch
Line 386: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.switch] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Global-HVAC-State-switch already exists - ignoring switch.koolnova_2_koolnova_2_global_hvac_state
Line 387: 2024-11-20 14:53:51.334 ERROR (MainThread) [homeassistant.components.switch] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Modbus-Debug-switch already exists - ignoring switch.koolnova_2_koolnova_2_modbus_debug
Line 388: 2024-11-20 14:53:51.335 INFO (MainThread) [homeassistant.components.climate] Setting up koolnova_bms.climate
Line 389: 2024-11-20 14:53:51.335 ERROR (MainThread) [homeassistant.components.climate] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Area 1-area-climate already exists - ignoring climate.koolnova_2_area_1
Line 390: 2024-11-20 14:53:51.335 ERROR (MainThread) [homeassistant.components.climate] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Area 2-area-climate already exists - ignoring climate.koolnova_2_area_2
Line 391: 2024-11-20 14:53:51.335 ERROR (MainThread) [homeassistant.components.climate] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Area 3-area-climate already exists - ignoring climate.koolnova_2_area_3
Line 392: 2024-11-20 14:53:51.335 ERROR (MainThread) [homeassistant.components.climate] Platform koolnova_bms does not generate unique IDs. ID koolnova_bms-koolnova 2-Area 4-area-climate already exists - ignoring climate.koolnova_2_area_4

Ok I found out that the situation is now the opposite, where now I can see the entities correctly under each system but now koolnova 1 is working while koolnova 2 is not working.
home-assistant-1.log
[home-assistant-2.log](https://github.com/user-attachments/files/17844825/home-assistant-2.lo

g)
I attach two logs, log 1 is before the restart of HA and the second after. I also attach the screenshots of the integration.

Log analysis shows that there was a disconnection of the modbus client during the integration setup after restarting Home Assistant.
Ligne  194: 2024-11-21 11:27:32.609 DEBUG (MainThread) [custom_components.koolnova_bms.koolnova.operations] [OPERATION] dict: {'_mode': 'Modbus RTU', '_timeout': 5, '_debug': False, 'port': '/dev/ttyUSB1', 'addr': 49, 'baudrate': '9600', 'parity': 'E', 'bytesize': 8, 'stopbits': 1}
Ligne  195: 2024-11-21 11:27:32.611 DEBUG (MainThread) [custom_components.koolnova_bms.koolnova.device] Retreive system status ...
Ligne  196: 2024-11-21 11:27:32.611 DEBUG (MainThread) [custom_components.koolnova_bms.koolnova.operations] reading holding register: 0x50 - Slave: 49
Ligne  197: 2024-11-21 11:27:32.612 DEBUG (MainThread) [custom_components.koolnova_bms.koolnova.operations] [OPERATION] dict: {'_mode': 'Modbus RTU', '_timeout': 5, '_debug': False, 'port': '/dev/ttyUSB2', 'addr': 49, 'baudrate': '9600', 'parity': 'E', 'bytesize': 8, 'stopbits': 1}
Ligne  199: 2024-11-21 11:27:32.612 DEBUG (MainThread) [custom_components.koolnova_bms.koolnova.device] Retreive system status ...
Ligne  200: 2024-11-21 11:27:32.612 ERROR (MainThread) [custom_components.koolnova_bms] Something went wrong ... Client Modbus not connected
Ligne  202:   File "/config/custom_components/koolnova_bms/__init__.py", line 69, in async_setup_entry
Ligne  205:   File "/config/custom_components/koolnova_bms/koolnova/device.py", line 303, in async_update
Ligne  208:   File "/config/custom_components/koolnova_bms/koolnova/operations.py", line 242, in async_system_status
Ligne  211:   File "/config/custom_components/koolnova_bms/koolnova/operations.py", line 78, in __async_read_register
Ligne  213: custom_components.koolnova_bms.koolnova.operations.ModbusConnexionError: Client Modbus not connected

You could play with the Timeout setting (default: 5 seconds) on the integration configuration page and increase it to 15 seconds (Max: 30 seconds).

Hi,
so I have an update.
The reason why there war an error it was related to the fact that I switched the USBs.
What I did was deleting the instances and restart the procedure. All good.
Now there is another issue, attached a screenshot.
Sometimes happens that one of the two systems goes down and all the entities goes under the first one. I need to restart the integration to see back the original situation.
Moreover, when I add une system and select the proper area, when I add the second one I cannot change and, if I try to change the area of the second system, it changes the areas of both.

Thank you.

Hi EdoTraverso,
Can you attach the logs when the issue appears to see what doing wrong, please.

Hi,
so screen 1 is the state before the refresh of each system.
screen 2 is after the refresh
Log attached.
Thank you
home-assistant-2.log

Thanks for the log, I see nothing in the log that explain your issue.
Did you re-configure the 2 instances with 2 differents name ? or did you renamed each instance after installing them ?
Normally you should see one and only one device ("dispositivo") for each instance (not 2 devices "dispositivi" for the first instance and 1 device "dispositivo" for the second).
Could you try to remove the 2 instances, and after reconfigure each (one with the name "Corridoio" without "Koolnova" in the configuration view and one with the name "Disimpegno" in the configuration view)
After that, could you send me the same screenshots (with logs) please ?

Done, logs attached.
Yes I renamed the instances, probably the issue is this.
Thank you!
home-assistant-2.log


Use global isNaN and parseInt for IE/Safari support

Unfortunately neither Safari or IE support the versions of isNaN and
parseInt that are on the Number object.

I guess the goal of @Nimelrian when using those functions was to align the splice function to JS specs. Can you ensure that moving to the global version does not change that (I know for instance that Number.isNaN and isNaN don't perform the same thing exactly.
And, if we can go back to the globals, parseInt(n, 10) should probably become +n, it's way easier.

Thanks for detailing where the change came from.  I read through the ToNumber specification and tested splice in Chrome and adapted baobab's splice to further conform to it.  It actually turns out that using parseInt resulted in some incorrect behavior because it parses booleans and empty strings as NaN whereas the spec allows those.  Reworked the commit and its message with the new behavior and some tests.

Looks like I missed some of the more specific parts of the specification there. Thanks for the fix!

Merging right now and publishing a new version soon :)

v2.4.3 is published on npm.

Great, thanks!


Buildfile: build.xml does not exist!

I tried ant build_all command to build a new jar file but i faced this error:
Buildfile: build.xml does not exist!
Build failed

I have the same error, have you found out how to fix this?

@moslem-pc @crashkonijn  if you're running on mac/linux you need to run ./build_all script, not ant.
If you're running on windows, simply run the commands inside the build_all script.

I tried downloading and using build_all, as it was shown in the README.md but I then received errors about the org.json and android.app packages not existing when compiling. I'm not using an IDE and I have installed Ant, the Android SDK and I configured the paths for Android.

@Giacom please make sure that you have android-19 installed, or change these lines to what is right for you.

Ah thanks, I'll try that.


Search Pipeline

Backend for search from string in DB.

Not relevant now, will recreate more apt issue


Making Switch Account more intutive #8345

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes  #8345
[x] ## Did your issue had any of the "$" label on it?
[x] My DOT address: Payout

He won't see it

https://github.com/kodadot/nft-gallery/discussions/8407


Dev

Merge to test for demo

IBM Cloud toolchain: Delivery Pipeline deployed LRP-web-app to prod, including commit 41f5b9513e3758b41ddcc78210778f59ed5d559a


Highlight Julia code

Originally posted by @JamieMair in James-Yu/LaTeX-Workshop#3054
When I am using a package to enable proper typesetting of Julia code, the syntax highlighting does not use the Julia syntax highlighting, but the default latex highlighting. This is shown below:

This becomes an issue when using special characters:

To Reproduce  [Required]
Insert code as shown above. This can be useful for the verbatim package as well.

@JamieMair Can you post the tex content of your example? What packages are providing the environments juliaconsole and juliaverbatim?

I am using a copy of this template - https://github.com/sisl/tufte_algorithms_book
In the introduction.TeX file in the chapters, you can find examples of the Julia related environments working (there a few different ones).
I believe they use a version of the pythontex package with a custom lexer to do the syntax highlighting in the PDF. As far as I can tell, the Julia latex environments are defined in pythontex.sty
Maybe it would be useful to have an optional setting in which you can specify the syntax highlighting for different environments. For example:
"vscode-latex-basics.syntax.environments": {
    "juliaverbatim": ".jl"
    "juliaconsole": ". jl"
... Etc
}

Perhaps this line https://github.com/jlelong/vscode-latex-basics/blob/5f26a6daf70352a6f3f40b3b4437a605f5d954ee/syntaxes/LaTeX.tmLanguage.json#L466 needs to be modified to be like https://github.com/jlelong/vscode-latex-basics/blob/5f26a6daf70352a6f3f40b3b4437a605f5d954ee/syntaxes/LaTeX.tmLanguage.json#L485
Which has support for pythontex environments.

https://github.com/jlelong/vscode-latex-basics/pull/15 - The proposed changed seemed to work fine.

Close by #15

Maybe it would be useful to have an optional setting in which you can specify the syntax highlighting for different environments. For example:
"vscode-latex-basics.syntax.environments": {
    "juliaverbatim": ".jl"
    "juliaconsole": ". jl"
... Etc
}

Unfortunately, it is not possible to use configuration variables in TextMate grammars. Only semantic highlighting can be configured by the user but for the time being we consider it out of the scope of the extension.


fix: save last finalized block

Closes #1712
Prevents potential issues with fixed finality depth on adverse conditions, there are no fixed bounds to finality depth and the actual finalized block can be earlier than a  last_finalized_block_number initialized to a fixed value.

unwinds should probably clear this one out if it goes beyond this block. this can happen when manually calling unwind

what are you suggesting? what needs to be done?

if there's a pipeline unwind that goes beyond that block, replace it with a lower one on Finish stage unwind fn

@joshieDo cool thx! we should save the lowest one that results from the unwind operation, right? either to-block or the lowest end resulting from num-blocks?

if there's a pipeline unwind that goes beyond that block, replace it with a lower one on Finish stage unwind fn
@joshieDo ok done here https://github.com/paradigmxyz/reth/pull/8473/commits/eaec49db6cb64b779232c05d4b5321cb99bb884f by introducing a FinalizedBlockProvider trait with default methods, ptal


Add per-level formatter

Currently Formatter is usable only in backend logger. In some cases (e.g. for debug messages) is preferable to apply a different Formatter without touch the backend. So, in this case, it would be useful to apply the Formatter in level functions (e.g. log.Debug("message", formatter) )

Personally I would want all my log statements to be a consistent formatter, so the way it currently works makes sense to me. I can't think of a use case where per-level makes much sense to be honest. Why do you need debug-level messages to be formatted differently? You can always just add whatever debug info you need to the message.

That's not something I will add. If you do write the code, please send a PR and I might reconsider.

@giftig for example, I want to use the formatter "%{time:2006-01-02 15:04:05.000} %{level} - Application - %{message}" for all messages. In debug messages, I want to use the formatter "%{time:2006-01-02 15:04:05.000} %{level} %{longfunc}:%{id:03x} - Application - %{message}" because is very useful to know how the message is generated without replicate the code in every debug point. Alternatively, you need to codify all messages manually, and is very boring (and error prone)

Hm, I tend to find clear log messages gives you sufficient context, but to each their own I suppose. Of course, you can simply put your formatter in a configuration file along with logging level, so you could swap between the two formatters when swapping between, say, warning / debug level just by making a simple config change. I would have thought that level of functionality would be best left to the application.

@henryx, this [pull request #60] (https://github.com/op/go-logging/pull/60) it's what you want?

@franciscocpg yes, is a good way for doing it


Test code for models

Requires to make a test code for each model.

Hi! I meet a problem when use your source code named “at_model.py” . In the seventh line， from six.moves import xrange ,

So could you tell me what wrong with the code ?  I will be very grateful to you！


C# rename namespace aliases

s:: -> sys::
proto:: -> pb::
protowkt:: -> pbwkt::

Codecov Report

Merging #1925 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1925      +/-   ##
============================================
+ Coverage     85.77%   85.77%   +<.01%
  Complexity     4792     4792
============================================
  Files           441      441
  Lines         19189    19190       +1
  Branches       2005     2005
============================================
+ Hits          16459    16460       +1
  Misses         2046     2046
  Partials        684      684

Impacted Files
Coverage Δ
Complexity Δ

...en/transformer/csharp/CSharpCommonTransformer.java
95.08% <ø> (ø)
13 <0> (ø)
:arrow_down:

...oogle/api/codegen/util/csharp/CSharpTypeTable.java
87.5% <100%> (ø)
24 <1> (ø)
:arrow_down:

...ansformer/csharp/CSharpGapicClientTransformer.java
96.43% <100%> (+0.01%)
51 <0> (ø)
:arrow_down:

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 707a42d...5bc5245. Read the comment docs.


[Experiment] TypeSafe Query Builder

There is already a SQL builder called LINQ, but I don't like its syntax.
I would like to write a syntax that is more similar to C# and SQL, and that is also type-safe.
ideal form
    public readonly record struct TableA(
        int a_id,
        string text,
        int value
    );
    public readonly record struct TableB(
        int a_id,
        string text,
        int value
    );
    public readonly record struct TableC(
        int a_id,
        string text,
        int value
    );

var a = Sql.Table<TableA>(); // Define table A
var b = Sql.Table<TableB>(); // Define table B
var c = Sql.Table<TableC>(); // Define table C

var query = Sql.From(a)
    .InnerJoin(b, () => a.id == b.id)
    .InnerJoin(c, () => a.id == c.id && a.id == b.id)
    .Select(() => new { a.id, b_id = b.id, value = Sql.Sum(c.id) }) // Select columns and aggregate function
    .Where(() => a.id == 1) // Filter rows
    .GroupBy(() => new { a.id, b.id }) // Group by columns
    .Having(() => Sql.Sum(c.id) > 10) // Filter groups
    .OrderBy(() => new { a.id, Sql.Descending(b.id), c.id }); // Order results

// CTE with query
var q1 = Sql.With(query)
    .From(query)
    .Select(() => new { query.id, query.b_id });

// Sub query
var q2 = Sql.From(query)
    .Select(() => new { query.id, query.b_id });

The directions are:
Before writing the query syntax, define the tables you will use.
Being able to treat variable names as aliases makes it much easier to create query syntax.
The same goes for subqueries and CTEs.
Note: I initially considered defining both the table and alias within the query syntax, but decided against it because I thought it would be redundant, deeply nested, and create readability issues.
Start with the From clause
The syntax of the SQL statement is intentionally changed here because it is natural to define the table and then define the selection columns.
However, you may not need to start with the From clause because you are externalizing the table declarations you use.
Requires a class called SQL
We need a SQL class with methods such as From, Sum, and Descending. However, the content must be empty because the class only needs a function name (it just converts the function name to a SQL function name). Only the argument and return value definitions are important.
The select clause is expressed in an anonymous class
Anonymous classes that allow you to declare selection columns and aliases at the same time are best.

add issue [Experiment] TypeSafe Query Builder - single table #415

add issue [Experiment] TypeSafe Query Builder - CTE #419


[zh] Deep learning

working in progress

For your information, I translated after your progress in #23 :)

@YijiaJin Oh, I see :) Thanks a lot!

@shervinea Should I preserve the number list head, like 2. ? And what about the bold ** ?

Thanks to the both of your for your hard work!
@spin6lock The format is as follows:

English blabla

⟶ Translated blabla

@spin6lock would you have time to do the changes? If not, no worries; then maybe we could continue this process directly on this PR that I previously marked as duplicate.

Sorry for the late reply, I’ll fix it in afternoon:)
Shervine Amidi<EMAIL_ADDRESS>周三08:05写道：

@spin6lock https://github.com/spin6lock would you have time to do the
changes? If not, no worries; then maybe we could continue this process
directly on this PR
https://github.com/shervinea/cheatsheet-translation/pull/49 that I
previously marked as duplicate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shervinea/cheatsheet-translation/pull/12#issuecomment-426470407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs0YN2LO7xN_9w5k4MZbduTPK4mSsR0ks5ug_8ugaJpZM4Wps0_
.

@tigerneil 递归->循环 没修改，按中文wiki的翻译，一个是时间递归（recurrent nn），一个是结构递归（recursive nn）


Banner filtering

Proposed changes
Fullfører bannere. Legger til filtrering.
Frontend:
https://github.com/TIHLDE/Kvark/pull/684
Issue number: closes #
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] CHANGELOG.md has been updated. Guide
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] The fixtures have been updated if needed (for migrations)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

https://github.com/TIHLDE/Lepton/pull/669 Tihi hadde glemt vi hadde denne

#669 Tihi hadde glemt vi hadde denne

Like ikke at den var så dymisk. Det var for mye logikk i frontend for å hente ut bannerene. Det eneste man trenger er å se synlige nå, tidligere synlige bannere og fremtidig synlige bannere. Bedre å gjøre den logikken i backend.


Convert more tests to image tests

What do you think about this? I'm leaning towards thinking we should do this. We can use higher resolution than is practical for text-based tests, and they serve as a form of documentation for what filters do.

@terror

Yep that sounds good, and since we're storing the images on GitHub we can also link them to the README as examples for each filter.

I think this is basically done.


pypi auth failed

Wheels are successfully built. But there were some issues with uploading the wheels.
https://github.com/pyscf/gpu4pyscf/actions/runs/6442244137/job/17493609742

We probably can use dynamic load to reduce the wheel size.


Branch 1.4

Has any one meet the following situation:
We observed that during the shuffle period of a sort job, the tasks distributed among several nodes are running after by after, not concurrently. e.g.
task 0~15 belongs to node0
task 16~31 belongs to node1
task  32~47 belongs to node2, they finished  in this manner: 16~31(the first wave), 0 (the second wave), 32~33(the third wave) …
Why not 0,16,32, then 1, 17, 33, then…

Can one of the admins verify this patch?

@f7753 can you please close the pull request?


Fix crashing case, add warning

This PR fixes the case where the simulation crashes and adds a warning to the log.

bors r+


Renaming class name in a code behind of a Razor page should rename the component

Direct customer ask copying here:

When I refactor the class name in a code behind of a razor page the name of the component is not refactored. With that you can “refactor” working code in not compiling anymore.

Closing this in favor of https://github.com/dotnet/razor/issues/8541, (similar to use cases 4 and 5 explained in the description)


[FEATURE REQ] Make each Azure Spring Identifier only used in one place.

Current problem
Now some Azure Spring Identifier be used in more than one place.

az-sp-kv-ct be used in AzureKeyVaultCertificateAutoConfiguration and azure-security-keyvault-jca-user-agent-value-prefix.txt.
az-sp-eh be used in AzureBlobCheckpointStoreConfiguration,  AzureEventHubsClientBuilderConfiguration, DedicatedConsumerConnectionConfiguration, AzureEventHubsProcessorClientConfiguration, AzureEventHubsClientBuilderConfiguration
az-sp-bus be used in PremiumServiceBusJmsAutoConfiguration, AzureServiceBusClientBuilderConfiguration, NoneSessionConsumerClientConfiguration, SessionConsumerClientConfiguration, NoneSessionProcessorClientConfiguration, SessionProcessorClientConfiguration, AzureServiceBusProducerClientConfiguration
az-si-sb be used in DefaultServiceBusNamespaceProcessorFactory(2 places), DefaultServiceBusNamespaceProducerFactory.
az-si-eh be used in DefaultEventHubsNamespaceProcessorFactory, DefaultEventHubsNamespaceProducerFactory.

Solution
Design new identifier for every usage scenario.
Identifier max length
Because ClientOptions#MAX_APPLICATION_ID_LENGTH=24, and we may have version like 4.0.0.beta-1 which length is 12. The remaining length is 24 - 12 = 12.
Identifier format.
Identifier prefix.
About identifier prefix, 2 options come into mind:
Option 1: Keep same with version 3.x. Use az-sp- as prefix.
Here is the identifiers used in version 3.x:

ApplicationId
Description

az-se-kv-jca
Azure Security KeyVault JCA

az-sp-kv-ct
Azure Spring KeyVault Certificates

az-sp-kv
Azure Spring KeyVault Secrets

az-sp-bus
Azure Spring Service Bus

az-sp-sb
Azure Spring Storage Blob

az-sp-sf
Azure Spring Storage Files

az-sp-aad
Azure Spring Active Directory

az-sp-b2c
Azure Spring Active Directory B2C

Pros:

Keep same with previous version.
az-sp- is not easy to have conflict with other User-Agent.

Cons:

The length of az-sp- is 6, there will be only 12-6=6 left. It may not enough to encode all identifiers.

Option 2: Use as- as prefix.
Pros:

There will be 12-3=9 left.

Cons:

The possibility of conflicting with other User-Agent is much higher.
Not same with previous version. Which means we need more efforts to maintain the logic of BI.

I decide to choose option 1: Use az-sp- as prefix.
Identifier suffix
In version 3.x, we already have identifier like az-sp-kv-ct, which means Azure Spring KeyVault Certificate. We can continue this style: Use abbreviation as suffix.
But Some services have many scenarios, for example, Now there are 10 identifiers about ServiceBus.

Usage

AzureServiceBusClientBuilderConfiguration#serviceBusClientBuilderFactory

DefaultServiceBusNamespaceProcessorFactory#doCreateProcessor#sessionEnabled=false

DefaultServiceBusNamespaceProducerFactory#doCreateProducer

DefaultServiceBusNamespaceProcessorFactory#doCreateProcessor#sessionEnabled=true

AzureServiceBusProcessorClientConfiguration#NoneSessionProcessorClientConfiguration#serviceBusProcessorClientBuilderFactory

AzureServiceBusConsumerClientConfiguration#NoneSessionConsumerClientConfiguration#serviceBusReceiverClientBuilderFactory

PremiumServiceBusJmsAutoConfiguration#jmsConnectionFactory

AzureServiceBusProducerClientConfiguration#serviceBusSenderClientBuilderFactory

AzureServiceBusProcessorClientConfiguration#SessionProcessorClientConfiguration#serviceBusSessionProcessorClientBuilderFactory

AzureServiceBusConsumerClientConfiguration#SessionConsumerClientConfiguration#serviceBusSessionReceiverClientBuilderFactory

We can use az-sp-sb-nc for Azure Spring ServiceBus Namespace No Sessioon Processor Client. and az-sp-sb-sc for Azure Spring ServiceBus Namespace Sender Client Builder Factory. But after we use abbreviations like this, it's hard to know what does az-sp-sb-nc mean.
And after the project continue developing, there maybe more identifiers come up. It's hard to manage these abbreviations.
Here is one solution: Use number instead of abbreviations. For example: az-sp-sb-01 means Azure Spring ServiceBus Client Builder Factory, and az-sp-sb-02 means Azure Spring ServiceBus Namespace No Session Processor Client.
If we add new identifiers for ServiceBus in the future, we just need to increase number: 01 -> 02 -> 03 -> ... -> 99 -> 00 (10 * 10 = 100 in total). If we still have more identifiers, we can use aa -> ab -> ac -> zz (26 * 26 in total), I think that's enough.
Identifier table.
Identifier value and description

NO
Identifer value
Description

1
az-se-kv-jc
Azure Seciruty KeyVault Jca

2
az-sp-ac-01
Azure Spring App Configuration Client Builder Factory

3
az-sp-ad-01
Azure Spring Azure Active Directory Authorization Code  Grant Request

4
az-sp-bc-01
Azure Spring Azure Active Directory B2C Authorization Code  Grant Request

5
az-sp-bs-01
Azure Spring BlobService Client Builder Factory

6
az-sp-bs-02
Azure Spring Blog Service Eveht Hub Processor Blob Service Client Builder Factory

7
az-sp-cs-01
Azure Spring Cosmos Client Builder Factory

8
az-sp-eh-01
Azure Spring EventHubs Client Builder Factory

9
az-sp-eh-02
Azure Spring EventHubs Client Builder Factory for Consumer

10
az-sp-eh-03
Azure Spring EventHubs Client Builder Factory for Processor

11
az-sp-eh-04
Azure Spring EventHubs Client Builder Factory for Produder

12
az-sp-eh-05
Azure Spring EventHubs Namespace Processor Client Builder Factory

13
az-sp-eh-06
Azure Spring EventHubs Namespace Producer Client Builder Factory

14
az-sp-kv-01
Azure Spring KeyVault Certificate Client Builder Factory

15
az-sp-kv-02
Azure Spring KeyVault Certificate JCA

16
az-sp-kv-03
Azure Spring KeyVault Secret Client Fuilder Factory

17
az-sp-qs-01
Azure Spring QueueService Client Builder Factory

18
az-sp-sb-01
Azure Spring ServiceBus Client Builder Factory

19
az-sp-sb-02
Azure Spring ServiceBus Namespace No Session Processor Client

20
az-sp-sb-03
Azure Spring ServiceBus Namespace Sender Client Builder Factory

21
az-sp-sb-04
Azure Spring ServiceBus Namespace Session Processor Client

22
az-sp-sb-05
Azure Spring ServiceBus No Session Processor Client Builder Factory

23
az-sp-sb-06
Azure Spring ServiceBus No Session Receiver Client Builder Factory

24
az-sp-sb-07
Azure Spring ServiceBus Premium Connection Factory

25
az-sp-sb-08
Azure Spring ServiceBus Sender Client Builder Factory

26
az-sp-sb-09
Azure Spring ServiceBus Session Processor Client Builder Factory

27
az-sp-sb-10
Azure Spring ServiceBus Session Receiver Client Builder Factory

28
az-sp-ss-01
Azure Spring ShareService Client Builder Factory

Identifier value and usage

NO
Identifer value
Where to use

1
az-se-kv-jc
HttpUtil.DEFAULT_USER_AGENT_VALUE_PREFIX

2
az-sp-ac-01
AzureAppConfigurationAutoConfiguration#configurationClientBuilderFactory

3
az-sp-ad-01
AADOAuth2AuthorizationCodeGrantRequestEntityConverter

4
az-sp-bc-01
AADB2COAuth2AuthorizationCodeGrantRequestEntityConverter

5
az-sp-bs-01
AzureStorageBlobAutoConfiguration#blobServiceClientBuilderFactory

6
az-sp-bs-02
AzureBlobCheckpointStoreConfiguration#eventHubProcessorBlobServiceClientBuilderFactory

7
az-sp-cs-01
AzureCosmosAutoConfiguration#cosmosClientBuilderFactory

8
az-sp-eh-01
AzureEventHubsClientBuilderConfiguration#eventHubClientBuilderFactory

9
az-sp-eh-02
AzureEventHubsConsumerClientConfiguration#DedicatedConsumerConnectionConfiguration#eventHubClientBuilderFactoryForConsumer

10
az-sp-eh-03
AzureEventHubsProcessorClientConfiguration#eventProcessorClientBuilderFactory

11
az-sp-eh-04
AzureEventHubsProducerClientConfiguration#DedicatedProducerConnectionConfiguration#eventHubClientBuilderFactoryForProducer

12
az-sp-eh-05
DefaultEventHubsNamespaceProcessorFactory#doCreateProcessor

13
az-sp-eh-06
DefaultEventHubsNamespaceProducerFactory#doCreateProducer

14
az-sp-kv-01
AzureKeyVaultCertificateAutoConfiguration#certificateClientBuilderFactory

15
az-sp-kv-02
azure-security-keyvault-jca-user-agent-value-prefix.txt

16
az-sp-kv-03
AzureKeyVaultSecretAutoConfiguration#secretClientBuilderFactory

17
az-sp-qs-01
AzureStorageQueueAutoConfiguration#queueServiceClientBuilderFactory

18
az-sp-sb-01
AzureServiceBusClientBuilderConfiguration#serviceBusClientBuilderFactory

19
az-sp-sb-02
DefaultServiceBusNamespaceProcessorFactory#doCreateProcessor#sessionEnabled=false

20
az-sp-sb-03
DefaultServiceBusNamespaceProducerFactory#doCreateProducer

21
az-sp-sb-04
DefaultServiceBusNamespaceProcessorFactory#doCreateProcessor#sessionEnabled=true

22
az-sp-sb-05
AzureServiceBusProcessorClientConfiguration#NoneSessionProcessorClientConfiguration#serviceBusProcessorClientBuilderFactory

23
az-sp-sb-06
AzureServiceBusConsumerClientConfiguration#NoneSessionConsumerClientConfiguration#serviceBusReceiverClientBuilderFactory

24
az-sp-sb-07
PremiumServiceBusJmsAutoConfiguration#jmsConnectionFactory

25
az-sp-sb-08
AzureServiceBusProducerClientConfiguration#serviceBusSenderClientBuilderFactory

26
az-sp-sb-09
AzureServiceBusProcessorClientConfiguration#SessionProcessorClientConfiguration#serviceBusSessionProcessorClientBuilderFactory

27
az-sp-sb-10
AzureServiceBusConsumerClientConfiguration#SessionConsumerClientConfiguration#serviceBusSessionReceiverClientBuilderFactory

28
az-sp-ss-01
AzureStorageFileShareAutoConfiguration#shareServiceClientBuilderFactory

Tasks

[ ] Update Java code to use new identifier.
[ ] Update BI scripts.

Closing this issue.
Use this issue to track new identifier design: Redesign: #26203


links to anchors to other headings not working

I have a markdown file with various headings and links inside the same file linking to those headings (i.e., linking to anchors in the same file) and converted this to HTML using the following code:
  gh-md-to-html ./test.md -w .

After conversion the "internal" links to other headings do not work anymore. My dirty workaround was to replace id="user-content- with id=" and class="anchor" with an empty string.
  sed -i 's/id=\"user-content-/id=\"/g' test.html
  sed -i 's/class=\"anchor\" //g' test.html

I suppose there might be a more elegant solution to this?

@fabbra Hello, and sorry for responding so late (several months, actually). I accidentally unwatched my own repository, and missed several issues in the process. I didn't realize this until right now, though.
My apologies for the delayed response; I will look into the issue (and hopefully fix it) this evening.
Did you find a workaround for yourself? I'm sorry if you had to stop using my tool due to me not fixing this in time.

I suppose there might be a more elegant solution to this?

I think the behavior is probably either caused by GitHub's web API, or by some internal code of the module. In the later case, I just need to change some code, and in the former one, I will parse the HTML and do some changes to the parsed HTML (using sed would probably cause issues in case the string "user-content- is used within a code box, for example.

Turns out the behavior to start header ids with user-content- is GitHub-specific behavior (and the offline-option that gh-md-to-html provides actually imitates that behavior on purpose). This behavior is also intended, because it ensures that the resulting HTML file can be embedded into any website, for example using a self-written static site generator, without clashing with anchors used by the rest of the website provided none of them starts with user-content-.

I fixed this now by making the tool change all internal links from  to .
Thanks again for filing this issue! :D

perfect. thanks a lot for the fix!


Bug Report

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed

Expected Behavior
I should receive a response from the device.
Current Behavior
When I try to write on a characteristic in IOS I get Characteristic XXXXXXXXXX write failed for device XXXXXXX
Steps to Reproduce

Platform: iOS.
JS logs: Reactotron Characteristic XXXXXXXXXX write failed for device XXXXXXX

const bleManager = new BleManager();
bleManager.onStateChange(async state => {
if (state === 'PoweredOn') {
const device = await bleManager.connectToDevice(deviceId);
await device.discoverAllServicesAndCharacteristics();
const details = await device.characteristicsForService(
'0000ffe0-0000-1000-8000-00805f9b34fb'
);
await details[0]
.writeWithResponse(btoa('a'))
.then(async char => console.tron.log(await char));
}
});

https://github.com/Polidea/react-native-ble-plx/wiki/=-FAQ:-I-got-a-BleError.-Now-what%3F

I stumbled upon a similar problem in production build.
The problem in my case was that btoa isn't available without running through browser debugger, so I had to install base-64 from npm and import it as so:
import {decode as atob, encode as btoa} from 'base-64'
thus avoiding refactoring code.
This was crazy since I couldn't debug in production. I was able to connect but it failed sending messages saying that it couldn't discover characteristics (???). Then I realised that without debugging activated btoa "wasn't found".


gruut-ipa~=0.12.0

This library requires gruut-ipa~=0.12.0 but I don't see that being released? The newest version is 0.11.0? https://github.com/rhasspy/gruut-ipa/releases/tag/v0.11.0

Sorry about that! Just pushed gruut-ipa 0.12.0 to PyPI.


Speedup of singlediode._lambertw function

I have an application where I need to evaluate _lambertw many many times. Even a small speedup would be helpful. Currently _lambertw calculates IV curve points like I_x, I_xx that I don't need. What about adding an input flag that allows us to skip the evaluation of some parameters?
The version I wrote to do this is below, but there are other ways this could be done as well.
def _lambertw(photocurrent, saturation_current, resistance_series,
              resistance_shunt, nNsVth, ivcurve_pnts=None,
              calculate_all=False):
    if calculate_all:
        # Compute short circuit current
        i_sc = _lambertw_i_from_v(resistance_shunt, resistance_series, nNsVth,
                                  0.,
                                  saturation_current, photocurrent)

    # Compute open circuit voltage
    v_oc = _lambertw_v_from_i(resistance_shunt, resistance_series, nNsVth, 0.,
                              saturation_current, photocurrent)

    params = {'r_sh': resistance_shunt,
              'r_s': resistance_series,
              'nNsVth': nNsVth,
              'i_0': saturation_current,
              'i_l': photocurrent}

    # Find the voltage, v_mp, where the power is maximized.
    # Start the golden section search at v_oc * 1.14
    p_mp, v_mp = _golden_sect_DataFrame(params, 0., v_oc * 1.14,
                                        _pwr_optfcn)

    # Find Imp using Lambert W
    i_mp = _lambertw_i_from_v(resistance_shunt, resistance_series, nNsVth,
                              v_mp, saturation_current, photocurrent)

    if calculate_all:
        # Find Ix and Ixx using Lambert W
        i_x = _lambertw_i_from_v(resistance_shunt, resistance_series, nNsVth,
                                 0.5 * v_oc, saturation_current, photocurrent)

        i_xx = _lambertw_i_from_v(resistance_shunt, resistance_series, nNsVth,
                                  0.5 * (v_oc + v_mp), saturation_current,
                                  photocurrent)

    if calculate_all:
        out = (i_sc, v_oc, i_mp, v_mp, p_mp, i_x, i_xx)
    else:
        out = (v_oc, i_mp, v_mp, p_mp)

    # create ivcurve
    if ivcurve_pnts:
        ivcurve_v = (np.asarray(v_oc)[..., np.newaxis] *
                     np.linspace(0, 1, ivcurve_pnts))

        ivcurve_i = _lambertw_i_from_v(resistance_shunt, resistance_series,
                                       nNsVth, ivcurve_v.T, saturation_current,
                                       photocurrent).T

        out += (ivcurve_i, ivcurve_v)

    return out

Not opposed but not excited about the specific proposal. What about a helper function _lambertw_mp that calculates the maximum power voltage and current? Then you could use _lambertw_v_from_i etc. to get Voc and Isc if desired.

Or just use Newton, or the explicit method. Why do you need Lambert W? What is your use case?

Trying to get a full of curve? You won't beat the performance of pvlib.singlediode.bishop88, you can also access this from pvsystem.singlediode by using method='newton'.

Thanks for the suggestions. I wrote a quick performance testing script and found that method='newton' is 2.4x faster than method='lambertw'. Is there a reason to prefer method='lambertw'?
import numpy as np
from time import time
from pvlib.pvsystem import singlediode

photocurrent = np.random.random(100000)*9
saturation_current = 1e-9 + np.zeros_like(photocurrent)
resistance_series = 0.5 + np.zeros_like(photocurrent)
resistance_shunt = 100 + np.zeros_like(photocurrent)
nNsVth = 1.8 + np.zeros_like(photocurrent)

start_time = time()
out = singlediode(photocurrent,saturation_current,resistance_series,resistance_shunt,
            nNsVth,method='lambertw')
print('Lambertw method elapsed time: {:.3f} s'.format( time() - start_time))

start_time = time()
out = singlediode(photocurrent,saturation_current,resistance_series,resistance_shunt,
            nNsVth,method='newton')
print('Newton method elapsed time: {:.3f} s'.format( time() - start_time))

# Lambertw method elapsed time: 0.685 s
# Newton method elapsed time: 0.281 s

I believe it's because of the possibility that Newton could have trouble converging, but I also think that concern may have been overstated, b/c these curves are all monotonic. But I think this concern is also why we provided the Brent method in addition to Newton, b/c it's guaranteed to converge for monotonic curves to any point between the bounds of Isc and Voc. However the Brent method is still using numpy.vectorize (a broadcasting loop) instead of the Brent method from scipy.cython_optimize. I never followed up and completed the work, b/c newton seems to work well enough. It's possible that a Cythonized Brent could be even faster than the vectorized Newton method. In benchmarks of scipy.cython_optimize the Cython version was over 30x faster than looping. If there's an appetite to add the Brent method, I can look at making a PR, it shouldn't require too much effort, but I'm also okay if someone else wants to do it, or just benchmark it.

My personal testing has shown that if you are after the most accurate solutions, the LambertW method is more reliable than the Newton/Brentq methods, but, the difference in accuracy is typically in the 11th and lower significant digits. I doubt most users would notice, or care, about these differences.

And solving 10^5 diode equations in a fraction of a second isn't bad performance in my opinion :)

Thanks for this advice! I think the Newton method will be accurate enough for my application. We can then avoid making any changes to the library.


Set-Up (wfntp): Merge wfntp submodule into main repo

User Story
As a dev,
I want to merge wfntp submodule into main ,
so that I can edit wfntp code within the current repo.
Acceptance Criteria

[ ] All the code from wf_ntp submodule is merge into main
[ ] keep git history of submodule (do we care?)

Definition of Done

[ ] Product Owner accepts the User Story
[ ] Acceptance criteria met
[ ] New data has been dvc push

Defined by CICD:

Code reviewed
black compliant
Unit tests passed
Functional tests passed
Non-Functional requirements met

@COTHSC this article might be useful

At the end, code was already in the PR of @COTHSC, I accept it despite It should not be acceptable.

I let you close this issue and delete the branch @cnstll


Building ReSwift as a dependency using Carthage

Hi,
I'm trying to build ReSwift using:
$ carthage update --platform iOS --no-use-binaries
But I keep getting:
Failed to write to /Users/XXX/Developer/Projects/XXX/Carthage/Build/iOS/ReSwift.framework: Error Domain=NSCocoaErrorDomain Code=260 "The file “ReSwift.framework” couldn’t be opened because there is no such file." UserInfo={NSFilePath=/Users/XXX/Library/Developer/Xcode/DerivedData/ReSwift-bmuzndysbkjrnnchodupxilcxzgk/Build/Products/Release-iphoneos/ReSwift.framework, NSUnderlyingError=0x7fc187f23b00 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}
If I clone ReSwift and build from Xcode I can build the ReSwift.framework for both the simulator (i386) and real devices (arm). Any idé as to what is wrong? (the build log does not give me a clue...)

Which Xcode version are you using?

Yeah, that's probably the issue. I'm trying to build using Xcode 10/Swift 4.2... But there is no issues when building ReSwift from within Xcode (10.0). Any ideas?

I have zero trust in the Xcode 10.0 build tools at the moment because a lot of things don't compile in one scenario (swift build from command line) while they do build from Xcode, and vice versa.

Hopefully the build tools will get fixed.

FYI, here is my script https://github.com/sejersbol/ReSwift/blob/master/build_universal_static_library.sh

Thanks for sharing! Do you apply signing later in the app?

Yes :-)


Add manual correction of tokenization in BIO format

An idea for enhancement: adding a new tab "Tokenization" or "Tokens". In this tab a user could upload a file in IOB2/BIO format and manually correct tokenization and tags. For example, a user could split or merge the tokens and modify the corresponding tags.
Motivation: the current tokenizer in MedTator for BIO export has relatively low accuracy for many special cases. In those cases the BIO files cannot be currently used for training. But even if the tokenizer will be replaced by another one (as mentioned in https://github.com/OHNLP/MedTator/issues/7), it is unlikely to have good performance for all languages and use cases. Therefore having an opportunity to fix manually the tokens and tags in BIO format would be very helpful for building a gold standard corpus.

Thank you so much for your suggestion! I think it's a very nice feature for improving the dataset quality!
My understanding is that your idea is about revising the file content of the IOB2/BIO format data, such as split/merge/delete tokens and tags, if there are any errors in tokenization or tagging. If so, I think the most challenging part is how to locate the errors in the long IOB-format file. As you described, the errors can be caused by exceptional tokenization cases, and we are not sure where they are in the file, which needs manual correction.
The editing, such as split/merge/delete tokens, can be done easily in any text editor (VSCode, Sublime Text, etc.).
I agree that a tool can be more helpful in making both searching and editing easier. Could you share some cases related to fixing issues in IOB2/BIO format files? For example, how to locate the errors and what the changes to the files. Then I think we can check how to improve this process.
I found one tool, neat, which may have similar functions for IOB2/BIO file editing. But I'm not sure if there is any other tool for reference. If you have any thoughts, please feel free to discuss :)

Indeed, neat looks like a good tool for tokenization correction. I'll test it.
With respect to locating errors and changes to the file: I tested MedTator on a text with many special characters and then converted it to BIO format, the special characters were considered as separate tokens. In this case I need to merge many tokens, for example.


code refactoring for tenant management API

This PR includes:

Refactor code to add an abstract layer provide APIs for tenant management
add vmdk_ops unit test for multi-tenency
make "name" unique in "tenants" table

Please review it @msterin @shaominchen

@lipingxue CI failing. Please fix.

@lipingxue - it looks like there are some conflicts to be resolved. and the latest changes (marked as "done" are still not in. Looking forward...

Yes.  After rebase with the master, there are some conflict with latest change in master(Bruno's change). I am addressing those conflict and rerun the test. Will do the push when it is done.

@msterin I have resolved conflict  and push the change. Please take another look.

@msterin
As we discussed last Friday, CI test failed due to DB schema existed in the testbed.
In order to make the test always start from a clean state, we decided to add code to remove auth_DB at each test. I added code for that, but it does not work.
The CI test failed with  "disk I/O error"  when accessing auth DB
I checked the log and also looked at our code, and I figured out this is reason why I see the above error.  We call function "get_auth_mgr()" to get a connection to the DB.  This function will call "thread_local._auth_mgr.connect()" ONLY if it does not have a cached value. In function"AuthorizationDataManager.connect()", we have the logic to decide whether we need to create table or not (check whether DB file exists or not).
Assume we start from the state that we have pre-exist DB in the testbed.

first time call "get_auth_mgr" to get a DB connection. Since this is the first time, it will further call AuthorizationDataManager.connect() to check whether create table is needed or not. Here
we don't need create the table since we already has the DB file in the test bed. The function
return a connection for the DB.

remove the auth DB file in the test code

second time call "get_auth_mgr" to get a DB connection. This time, it will not call AuthorizationDataManager.connect() since it already has the cached value. Therefore, it returned
a connection to staled DB file.

I think we only need to remove the old DB one time. This logic also apply to the possible DB migration.
Now I change the logic as follows:

At the beginning of  AuthorizationDataManager.connect(), check if we have DB file, it we have, and we haven't removed it (upgraded), do remove the DB.
After that, existing code will check whether we have the DB file (we don't have now since we just remove it),  and if we don't have, it will create the DB file and return the connection to the DB.

Please review this  newly added code.

@msterin I have addressed your comments, please review.

Fixed #710


refactor(choices): avoid setting activeItem on mouseenter

We were using ng-mouseenter to be able to set the hovered row as the active item, but this caused some lagging as ng-mouseenter was calling for a full $digest.
Could be better just to show a simple hover background change (using css) and avoid doing a $digest.
Demo plunker

I might be wrong, but after a quick git-grep it looks like the only place where setActiveItem was being called was from the ng-mouseenter event. Perhaps we can get rid of it in uiSelectChoicesDirective.js.

@jtheoof yes, its not needed anymore. I just removed it

:+1: Not sure what went wrong with the build.

Y just rebuilt it and Travis is fine now

Cool, I think it's good to go. Might help #88 but I do not think it's the main performance issue.

Hi,
When my ng-model is null at first with placeholder "Please select". Do we have any option to turn off active blue background on the first item of the list. Because that make people confused when user click to open the dropdown and see the first item selected with blue background.
The blue background on item should only be highlighted if ng-model has value when user select one item. Otherwise, we should leave all items in the dropdown list with white background as normal.
Thanks

+1


Producer-Consumer Problem (Java)

I would like to add producer-consumer problem in java which uses multithreading concept. I want to contribute to this project as a part of gssoc'21. Please let me know if I can work on this problem.

We are currently accepting only DSA related issues. But not specific language-based implementations.


changing header to use branding and changing term user need to public need

fixes #472 and #438
Just some small quick fixes in here @tmaybe!

:+1: This is good to merge when Travis tests pass.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 589

Questions ?
Please refer to the Backport tool documentation


[iPad] Default primary action does not dismiss the WhatsNewViewController

WhatsNewKit Environment

WhatsNewKit version: 2.0.2 using UIKit
Xcode version: 13.3
Swift version: 5+
macOS version running Xcode: 12.4
Dependency manager (SPM, Manually): SPM

What did you do?
on iPad, the continue button (primary action) does not dismiss. Works fine on iPhone.
What did you expect to happen?
The WhatsNewVC should dismiss
What happened instead?
As a workaround, I used the 'onDismiss' completion to dismiss the controller

Hi @john-work-ios,
Sorry for the late reply.
I have successfully reproduced the bug and I'm currently investigating this issue.
I will let you know when a new version is available which fixes this problem ✌

any lead on this from your investigation if we'd like to support on a fix?

Hi @john-work-ios,
I've recently tried to reproduce the bug and it seems like this issue has been fixed with either iOS 15 or iOS 16.
Seems like UIKit now passes a correct context for the PresentationMode which is called when the primary action gets tapped in order to dismiss the sheet / presented view controller.
https://github.com/SvenTiigi/WhatsNewKit/blob/641b2f5e771627d172f69ebfc5dda777ee74393d/Sources/View/WhatsNewView.swift#L215-L224
https://user-images.githubusercontent.com/11733014/222533689-b3e76139-8212-45c2-8602-a97fd0950421.mov

Closing due inactivity.
Please feel free to re-open the issue at any time.