WIP: issue #1340 - support bulk export to parquet format

Updates to fhir-operation-bulkdata and fhir-bulkimportexport-webapp
to accept and pass alternative export formats (application/fhir+parquet)

Introduction of SparkParquetWriter for using Apache Spark to
introspect a list of JSON strings (e.g. FHIR resources), infer a schema,
and write the schema and data into a parquet file. This approach writes
specific files under a single logical file which is actually a
directory, allowing the schema to change over time.

Updates to the System ChunkReader and ChunkWriter for parquet;
instead of writing to a buffer in the reader, we simply add to the
context info and return the Resource objects themselves. Then, in the
writer, we always write a file with whatever resources were passed.
This works because Spark is managing the writes to COS for us and
Parquet has this notion of a multi-part logical file, so we don't need
the tight control over multi-part uploads like in the case of NDJSON.

Also includes miscellaneous cleanup, debug, and style changes.
Signed-off-by: Lee Surprenant<EMAIL_ADDRESS>

closed in favor of #1396


Update development flow in README

Where?
README.md
Who?
Contributors
What?
Make sure it's clear that the main dev branch is dev.
Additional Context
Add any other context about the problem here.

Can take on this. @vvmnnnkv I'll ping you on Slack for some hints.


Component: Alert

Add an alert component. Mentioned by @ryan-mulrooney 🙌

Note: Alerts should be collapsible.

Rough spec:
<Alert
  closeLabel='Close dis alert'
  dismissable
  onDismiss={func}
  state='success'
/>


Err, just change the profile. Most people can figure it out.

HOW, SWAY?!


Consider adding a "contact point" fixed-length field to ad serialization

An ad is pointless if there is no way to respond to the ad.
Thus, we can expect that most ad texts will include also, some way to contact the advertiser, most likely via a website.
However, consider the cases below:

An advertiser is concerned about privacy for itself and its potential customers, thus uses a Tor v3 address.
An advertiser could not possibly care less about privacy, and thus uses a link compressor.

Not only can the ISP of the advertiser and the customer learn about the connection between them, but also the link compressor service learns about every customer that uses the compressed link.
The advertiser might inadvertently use some low-level software that includes in its logs about IP addresses of accessors of the advertiser website, which might be acquired by third parties without knowledge of the advertiser.

Thus, roughly speaking we can expect a massive loss in privacy in case 2 compared to case 1.
But advertisers using case 1 will be penalized compared to those in case 2, due to simply the number of characters involved.
Thus I propose, the addition of a required field, 33 bytes in length in the serialization, to serve as the link or "contact point" for someone to respond to the advertisement.  With Tor v3, it can instead store the ED25519 curve point, with the software automatically translating it to/from a .onion address.  For non-Tor-v3 usages, it could be a plaintext link.
The first byte could encode an enumeration, such as "no link", "Tor v3 with point having positive Y coord", "Tor v3 with point having negative Y coord", "ascii link" etc.
The principle, is that we should design our software to reduce useability-to-privacy tradeoffs.

I agree with your observation, which goes much further than we have now. I focused on proving the concept and spent the bare minimum on network serialization. It is auto-generated in CBOR format for now. If reworking that, your comments should be incorporated.


refactor(werelogs): use new API

http://ci.ironmann.io/gh/scality/Integration/2367#build-parameters end to end passed without a hitch.

:+1:

:+1:

Merging on behalf of @LaurenSpiegel to get things going forward.


False positive for testImplementation dependency with classifier

Build scan link
N/A
Plugin version
1.18.0
Gradle version
7.6
(Optional) Android Gradle Plugin (AGP) version
N/A
reason output for bugs relating to incorrect advice
> Task :E84HSP-app:reason

----------------------------------------
You asked about the dependency 'se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002'.
You have been advised to remove this dependency from 'testImplementation'.
----------------------------------------

Shortest path from :E84HSP-app to se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002 for compileClasspath:
:E84HSP-app
\--- se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002

Shortest path from :E84HSP-app to se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002 for runtimeClasspath:
:E84HSP-app
\--- se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002

Shortest path from :E84HSP-app to se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002 for testCompileClasspath:
:E84HSP-app
\--- se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002

Shortest path from :E84HSP-app to se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002 for testRuntimeClasspath:
:E84HSP-app
\--- se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002

Source: main
------------
* Exposes 27 classes, 5 of which are shown: se.micronic.bennu.kernel.gui.DisplayConditionsIfc, se.micronic.bennu.kernel.logevent.LogEventDeclaration, se.micronic.bennu.kernel.logevent.LogEventId, se.micronic.bennu.kernel.logevent.XMLLogEventSpecificationBase, se.micronic.bennu.kernel.message.Command (implies api).
* Imports 3 constants: se.micronic.bennu.kernel.message.*, se.micronic.bennu.kernel.message.CommandDeclaration.*, se.micronic.bennu.kernel.variable.* (implies implementation).

Source: test
\------------
\* Uses 1 class: se.micronic.bennu.kernel.message.threaded.ThreadedCommandTestExtension (implies testImplementation).

Describe the bug
I have this dependency, (please note the classifier = 'test'), without it my project do not compile.
  testImplementation("se.micronic.bennu:kernel:<IP_ADDRESS>-46-16c1002") { artifact { classifier = 'test' } }

So, the test code of my project depends on the test code of another project which is selected using the 'classifier = test', however the dependency analysis seems to ignore the classifier.
To Reproduce
N/A
Expected behavior
Should not give advise to remove the dependency since it is used for compilation of test code.
Additional context
N/A

Thanks for the issue. I think it relates to https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/issues/342.
@jjohannes what do you think? Is this something that would fall under the umbrella of work you'd like to do?

Yes this and #342 are related to the "capability" topic (Case 2 here https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/issues/298#issuecomment-1346159435). Using a classifier is like addressing another feature variant (only that the feature variant is not described in the corresponding POM metadata). IIRC Gradle does some tricks to still represent a Jar with classifier as a feature variants internally.
If I start looking into #298 and related issues, I can take a look if these issues can be solved together.

In the context of #854 I added a test to reproduce this.
The problem that can cause false positives is something I missed in #688.
The situation is that one component can consist of multiple Jar files. The code was initially assuming that there is always only one Jar file per component (or, to be more precise, variant of a component). With #688, the fact that there may be more than one Jar (in a non-standard setup) is respected. But there is a bug. The "capabilities" are computed for each Jar, but then the computation are merged later and the "capability details" (like which Public Methods exist) of one of the Jars is dropped. The fix is part of #854 (see https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/pull/854#discussion_r1105859793).
Now in the case of classifiers, as pointed out in the comments of #342, Gradle looses the classifier information, which is partly a conceptual problem of how this whole classifier thing (invented by Maven) works. It's better to use full-fledged "feature variants" where you can which the plugin will support with #854.
But if classifiers are at play, for the plugin it now looks as if all Jars (e.g. the kernel.jar and kernel-test.jar ) belong directly to the component. And they are treated as if they were one Jar. With the fix, this should work. Then there should be no more false positives (that lead to code that won't compile anymore).
Still advices involving classifiers can be wrong. But I would say this is something to follow on in #342.


Retrofit some test for AsyncOperation cancellation.

There are tests for individual classes inheriting from AsyncOperation, but there are none for the the noex::Table operations.
There should also be some integration tests for cancellation - we've made assumptions on how gRPC reports cancellation, but we've never executed this branch other than in whitebox unittests.

There should also be some integration tests for cancellation - we've made assumptions on how gRPC reports cancellation, but we've never executed this branch other than in whitebox unittests.

It is difficult to write consistent integration test for cancellation, This understanding is based on following observations.
Test Cancel for operation still not reached to server
One of the strategy to test cancellation is described below;

gRPC call for operation A.
Block server by some means so that it won't accept new requests.
gRPC call for operation B followed by a cancellation.

here step 2 is hard to achieve either with embedded server(example) or emulator(example).  I couldn't find a means to run single threaded server. The server picks up incoming request.
So, if client sends AsyncBulkApply followed by AsyncReadRows to server. Server picks up both the requests and starts processing both the requests.  Further Cancel request on  AsyncReadRows cancels the operation.
A use of ResourceQuota to reduce the server thread count did not help either. The server responds back immediately with grpc::Status::RESOURCE_EXHAUSTED(8) for second call.
note: Need to investigate if async server can be used for testing cancellation.
Test Cancel by terminating ongoing streaming RPC
a. One idea is to perform   AsyncBulkApply and AsyncReadRows, then Cancel the operation AsyncReadRows during the processing.
This can be easily achieved either through embedded server or emulator. This results into a callback with grpc::StatusCode::CANCELLED for AsyncReadRows. AsyncBulkApply completes the operation normally.
b. Another idea is to perform   AsyncBulkApply and AsyncReadRows, then Cancel the operation AsyncReadRows and break the stream by terminating emulator.
This results into a callback with grpc::StatusCode::CANCELLED(1) for AsyncReadRows. AsyncBulkApply ends up with grpc::StatusCode::DEADLINE_EXCEEDED(4).
so, though there is a way to test Cancel manually but it is difficult to write a consistent integration test case for it.


[go_router] Redirect examples in documentation use incorrect method signature

All of the examples in the documentation for the redirect field on the GoRouter class now use the incorrect signature.
One such example from the docs:
final GoRouter _router = GoRouter(
  routes: <GoRoute>[
    GoRoute(
      path: '/',
      redirect: (_) => '/family/${Families.data[0].id}',
    ),
    GoRoute(
      path: '/family',
      pageBuilder: (BuildContext context, GoRouterState state) => ...,
    ),
    GoRoute(
      path: '/:username',
      pageBuilder: (BuildContext context, GoRouterState state) => ...,
    ),
  ],
);

The GoRouterRedirect callback now takes a BuildContext in addition to the GoRouterState that was there before.
For the examples to compile, we would need to  change the constructor parameter to something like:
redirect: (_, __) => '/family/${Families.data[0].id}',

https://github.com/flutter/flutter/issues/102679 should be fixed for go_router as part of fixing this, to prevent this kind of issue in the future.

Hi @stuartmorgan this is in API doc string, does code excerpt supports doc string?

Ah, I didn't realize it was in code. I think the repo tooling currently only looks at md files, but it could potentially be expanded to code files.


chore(main): release 0.0.5

:robot: I have created a release beep boop
0.0.5 (2024-10-17)
Bug Fixes

fix incorrect diagonal three in a row (b9475f5)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.0.5
:sunflower:


ENH: Prefer ffmpeg

In https://github.com/pyglet/pyglet/pull/534#issuecomment-1010732272 it was pointed out that WMF is not accelerated, and ffmpeg is preferred for larger videos. However, there is no way for the user to actually make this choice AFAIK, because it's set in the order of add_decoders. Assuming it's not awful for smaller videos (where performance will be less important anyway), it seems like ffmpeg should be the preferred decoder on Windows. So this PR is really a proposal to change the priority order, feel free to close if it doesn't make sense.
An alternative would be to allow users to set an option to set the priority / preference using pyglet.options somehow, like there is for audio.
The motivation here is that the playback speed difference between these backends for large videos is drastic.

You can pass a specific decoder to the media load function.
from pyglet.media.codecs.ffmpeg import FFmpegDecoder
specific_decoder = FFmpegDecoder()
pyglet.media.load(filename, decoder=specific_decoder)

Okay, makes sense!


[Bug]: Need to refresh the screen to populate date in date picker after I use moment object in Default date

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Date do not populate on date picker widget after using {{moment()}} in edit mode.
Steps To Reproduce

Drag and drop a date picker widget
Use {{moment()}} in default date property
Observe the date do not populate on the widget
Refresh the screen and see that date gets populated

Environment
Production
Version
Cloud

@vivekverma2312 cannot reproduce, can you check this once?

@dilippitchika Adding loom video for this
https://www.loom.com/share/a9b925d70b9d48da831d6d5b2d75ca50

Ok, my bad I checked in production and not release. @sbalaji1192 are the changes in datepicker not in release but in production? Or this looks like a different issue. Please check.
@vivekverma2312 please make sure this issue is resolved before promoting a release.

@dilippitchika Doesn't seem like caused by the moment changes we did. I reverted my changes, but it is still happening.

@sbalaji1192 I am assigning this to @techbhavin to take this forward.


[BEAM-7660] Create Python ParDo load test job on Flink

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Seed Job

Run Python Load Tests ParDo Flink Batch

@iemejia  Could you take a look?

R: @iemejia Could you take a look?

@iemejia Please take a look. You can also tag someone else if you're busy

Run Seed Job

Run Python Load Tests ParDo Flink Batch

R: @lgajowy

Run Seed Job

Run Python Load Tests ParDo Flink Batch

Run Seed Job

Run Python Load Tests ParDo Flink Batch

@lgajowy I replied to your comments, please take a look once again.

@kamilwu My excuses I somehow totally missed this one, since it seems @lgajowy is quite advanced with the review I won't have much to add. Sorry again.

Sorry, I also missed this PR..


WebView2 freezes if another instance of a UWP app is suspended (such as if it's minimised or on another virtual desktop)

Describe the bug
If a UWP app supports multiple instances, and multiple instances are currently running with WebView2s, suspending (which happens automatically if a UWP app is minimised or on another virtual desktop) any one of them will cause all WebView2s in all instances to freeze, even if they are in an unminised desktop on the current virtual desktop. This issue is similar to https://github.com/microsoft/microsoft-ui-xaml/issues/6253.
Steps to reproduce the bug

Create a UWP project
Install WinUI 2.8.2
Install WebView2 1.0.1518.46
Add a WebView2 to MainPage and set its source to anything
Change package.appxmanifest to support multiple instances
Build, deploy and launch the app
Launch the app again
Minimise one of the windows of one of the instances of the app
The other instance's WebView2 will freeze

Expected behavior
WebView2s in running instances should not freeze when other instances of a UWP app are suspended.
Screenshots
No response
NuGet package version
None
Windows version
Windows 11 (22H2): Build 22621
Additional context
NuGet Package version is 2.8.2 but it's not in the above list.

Hey @aquinn39, thank you for filing the issue. Unfortunately (much like for #6253), we are currently unable to investigate WinUI2-specific issues that are not business or security critical (more on the policy here).


select signer information

It can be useful in visible signatures not to show the entire "subject" but rather a subset of it, for example the CN, OU, etc.
I suggest that you consider including an option to select which "subject" fields will be shown in the visible signatures.
I have not experience in Java, I only know a little bit of C, but if you tell me how to compile and make the jar from the sources, I dare to include that functionality by myself, and thus I learn a little bit of Java 😁
thanks to the authors for the good work done 👍

For me it would also be beneficial to be able to selectively modify the fields in the visible signature.
E.g. atm i do not require image, labels and hints. And would prefer the signee field to be really just the CN.
Another usefull option would be to be able to fully control the visible signature (e.g. some simple template with name and timestamps, etc)

It can be useful in visible signatures not to show the entire "subject" but rather a subset of it, for example the CN, OU, etc. I suggest that you consider including an option to select which "subject" fields will be shown in the visible signatures. I have not experience in Java, I only know a little bit of C, but if you tell me how to compile and make the jar from the sources, I dare to include that functionality by myself, and thus I learn a little bit of Java 😁 thanks to the authors for the good work done 👍

Hey there,
It should be relatively straight forward to get started by following the instructions here: https://github.com/open-pdf-sign/open-pdf-sign?tab=readme-ov-file#development
It's not much but really should be as simple as this. I suggest you install a java IDE though, for example https://www.jetbrains.com/idea/ or https://eclipseide.org/
If you get stuck happy to help!


Create the new UI in Figma

the new system needs more menu bars
Mimic google earth system

https://www.figma.com/design/h28MBkbOPCcXBWrljIxFgt/Panopticon?node-id=128-29&p=f&t=p7yEI7Q4pjDdcA97-0


SSD1306 I2C OLED 128x64 display not working with ESP32-DevKitC

Describe the bug
I'm trying to make a SSD1306 display work with an ESP32 w/ USB Type C, this model specifically:

I have connected the SDA and SCK pins to pins 21 and 22 respectively, as shown in this pinout diagram:

And set it up like shown in the docs:
import { SSD1306Driver } from "@devicescript/drivers"

const ssd = new SSD1306Driver({
    width: 128,
    height: 64,
    devAddr: 0x3d
})
await ssd.init()
ssd.image.print("Hello world!", 0, 0)
await ssd.show()

When I run the project, I get this:
 Unhandled exception
  message: I2C device not found or malfunctioning: I2C -1 dev=61: write [Buffer[2] 80ae], read undefined B
  at init_t main_F0 (pc:75)

NOTE: It works perfectly fine with Micropython, using this driver: https://github.com/micropython/micropython/blob/v1.19.1/drivers/display/ssd1306.py
To Reproduce
Steps to reproduce the behavior:

Connect a SSD1306 SDA and SCK pins to an ESP32 DevKit 21 and 22 pins respectively.
Copy/Paste the code from the docs example for SSD1306, and set width, height and devAddr setting accordingly.
Connect the ESP32 device
Run the project

Expected behavior
To see a "Hello world" text in the SSD1306 display.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

I suspect this is due to the missing import for the devkitc pin configuration -- see https://microsoft.github.io/devicescript/devices/esp32/esp32-devkit-c#devicescript-import .
I've added a little faq about this error message (https://microsoft.github.io/devicescript/developer/errors#i2c-device-not-found-or-malfunctioning) that will hopefully help in the future.

The esp32-devkit-c does not have an i2c configuration, so also follow the instructions at https://microsoft.github.io/devicescript/developer/board-configuration#configuration-by-code to configure the i2c pins.

I followed all sections mentioned:

I added the import for devkitc
I created a new board config with I2C setup for pins 21 and 22
I set up the I2C in-code
and still no luck, same error message.

I can confirm this works with micropyhton

Could you share your project sources?

I just release v2.14.2 which has an improved flow to collect a bug report from Visual Studio Code. After updating the extension, run "DeviceScript: Report Issue".

@pelikhan Sorry for the late response.
You can find the sources at this repo: https://github.com/varogonz95/devicescript-pgchi.git

@pelikhan any new on this? Im willing to contribute if necessary

could you carve out a minimal repro/snippet?

Sure, this is what I have, in a nutshell:
import { configureHardware, startLightLevel, startSoilMoisture } from "@devicescript/servers";
import { pins } from "@dsboard/esp32_wroom_devkit_c";

configureHardware({
    i2c: {
        pinSDA: pins.P21,
        pinSCL: pins.P22
    }
})

const ssdDisplay = new SSD1306Driver({ height: 64, width: 128 })

try {
     const characterScreen = await startCharacterScreen(ssdDisplay)
     await characterScreen.message.write("Hello world!")
} catch (error) {
    console.warn(`${SSD1306Driver.constructor.name} is not available:`, error)
}

vscode ask me to upgrade firmware to 2.15.1, which I already did, and it wont let me run anything until I do so :(

Oh right, please try again.

Unfortunately that didnt work :(
Output:
2023-08-16 22:37:25.123 [warning] dev:  Espressif ESP32-DevKitC v2.15.1; file v2.15.1
2023-08-16 22:37:25.162 [warning] dev:  start: (no name) v0.0.1-9ad74c3-2<IP_ADDRESS>.37
2023-08-16 22:37:25.191 [error] dev:  Unhandled exception
2023-08-16 22:37:25.205 [warning] dev:  Exception: DriverError
2023-08-16 22:37:25.205 [warning] dev:   message: I2C device not found or malfunctioning: I2C error -1 dev=61: write [Buffer[2] 80ae], read undefined B
2023-08-16 22:37:25.205 [warning] dev:   at init_F52 (pc:52)
2023-08-16 22:37:25.216 [warning] dev:   at startCharacterScreen_F11 (pc:24)
2023-08-16 22:37:25.219 [warning] dev:   at main_F0 (pc:515)
2023-08-16 22:37:25.220 [error] dev:  Unhandled exception
2023-08-16 22:37:25.225 [warning] dev:  stop program

WAIT. It works now!
I had to change the address tho.
Evidence:

Victory! i2c adresses....


Add a skeleton of ServiceRuntime [ECR-2939]

Overview
Separate the service runtime and service instantiation, so that the native code can be updated as soon as possible to create service runtime independently from services.
See ECR-2940 for the description of prospective updates in native.

See: https://jira.bf.local/browse/ECR-2939
Definition of Done

[ ] There are no TODOs left in the code
[x] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Public API has Javadoc
[x] Method preconditions are checked and documented in the Javadoc of the method
[x] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage increased (+0.07%) to 85.631% when pulling 044b4e32de9aeb8825e61246258df5d4e835fa44 on dmitry-timofeev:service-runtime-skeleton into 68064ba2e05d1d99a6187c3601c837d6cacf1e49 on exonum:master.


Strange GZIP bahviour

Example URL: http://st.sprashivai.net/10/081330ecc8/secure.min.js
This is js file, that must be gzipped, but:

In Chrome it works well

But in Safari, and via some online tools (http://checkgzipcompression.com/) GZIP don't work.

We use latest weed, compiled with go 1.4

Sending gzip content back to clients is conditionally turned on here:
https://github.com/chrislusf/weed-fs/blob/master/go/weed/weed_server/volume_server_handlers_read.go#L103
Maybe you can print out debug message to find out?

I've see debug info, the reason is IzGzipped function return false in my case
https://github.com/chrislusf/weed-fs/blob/master/go/weed/weed_server/volume_server_handlers_read.go#L102

Sorry, I've checking debug output and realize, that file was uploaded via weed php library WITHOUT filename - so, weed didn't match gzip criteria, and didn't gzipped it.


Add test suites and test clauses

Is your feature request related to a problem? Please describe.
The backend API lacks unit tests which make the app very bug prone.
Describe the solution you'd like
We need to introduce unit tests.
Describe alternatives you've considered
I've set up Jest as a test runner but feel free to explore other alternatives.
Additional context
Add any other context or screenshots about the feature request here.

Jest's Documentation

@Ladu-Lumori what's the status on this one?


Faster u32 parsing

This series of 5 commits refactors how the code parses u32 values.  The cumulative impact on the runtime of the decode/generated-png:noncompressed-8x8.png benchmark looks as follows:

[-7.1475% -6.9281% -6.6885%] (p = 0.00 < 0.05)
[-7.7189% -7.5262% -7.3241%] (p = 0.00 < 0.05)
[-7.4631% -7.1972% -6.9319%] (p = 0.00 < 0.05)

I expect that this PR will have minimal impact on bigger images, so I didn't even try running the other benchmarks (I assume that any changes observed in the other benchmarks would be just statistical noise).
I think that this PR turned out quite nicely from code simplification and readability perspective.  Even if there was no performance impact, I think it might be desirable to land it from this perspective.
Various notes:

I wondered whether it might be desirable to add some additional test coverage for partially read u32 values.

FWIW this path seems to be covered by decoder::tests::no_data_dup_on_finish - only one test... :-(
Once BufRead changes land, maybe we should tweak the render_images test to work with BufReader::with_capacity(1, <original input>) which should test dripping the input byte-by-byte.
I did try to run all the tests after temporarily tweaking BufReader::with_capacity calls in src/decoder/mod.rs so that a buffer of only 1 byte was used.

I wondered if U32ValueKind::Type after an IDAT chunk really needs to go through 2 state transitions (with a detour via Decoded::ImageDataFlushed during which self.state is set to the old/unchanged value).  This is probably in the unnecessary-microoptimizations territory, but I was hoping to simplify the code a bit more.  Still... I don't see how to do this cleanly.

@fintelia, can you PTAL?

:+1: This change looks good to me, but I haven't yet had a chance to do a detailed read through. Hopefully should get to that soon


any-nix-shell: 1.2.1 -> 1.2.1-unstable-2023-11-08

Description of changes

The current stable version is more than 3 years old, and is missing some important fixes, especially this one that adds support to nix run and fixes nix shell.
Also backport PR 34, that adds support to nix develop.
Misc changes include moving to pkgs/by-name and formatting it accordingly to RFC166.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

You could have just requested a new release at the https://github.com/haslersn/any-nix-shell repo.

If this is possible, please. I was assuming since there is no activity in the repository that it was dead.

Converting this PR to draft until we have a new release since @haslersn asked.


Improving the UX of mitmweb

HI,
This PR aims to improve the UX of mitmweb by fixing the position of edit button, a user had to scroll all the way up to start editing and once edited the content had to again scroll back to the top to lock the content.
Before

After

I hope this gives users a better experience

@Kriechi
got the mistake, gulp prod was the key. This is ready for review, thanks :)

Hi, please do take a look at this. Thanks


Send feature value in ingestion metric

What this PR does / why we need it:
This PR adds feature_value to the ingestion metric. This feature value metric uses StatsD histogram type so the min, max, average and percentiles of the values within the StatsD flush interval can be retrieved.
Some users require this metric in order to make sure that the features they are using are valid. For instance, someone may ingest invalid feature values to Feast. By looking at this metric, the end user can perform additional manual check before feature retrieval.
Also existing users of Feast v0.1 are already having this feature value metric but this is not available in v0.4 https://github.com/gojek/feast/pull/245/commits
Which issue(s) this PR fixes:

Does this PR introduce a user-facing change?:

Let's discuss this offline please.


Create develop.md and update Makefile

Description

Add submodule to Makefile to make it easier for developers with a clean repo
to get started.
Create documentation for developers that want to contribute to the operator.

Type of change

[X] Docs
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] This change requires a documentation update
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added a test case that will be used to verify my changes
[ ] Verified independently on a cluster by reviewer

Verification steps

Thanks, i will fix tomorrow morning

fixed


Implement Replication layer

Depends on #317

thaliSendNotificationBasedOnReplication

[ ] Implement in-memory notification list
[ ] Implement DB to persistently store notification list and load on start up

thaliPullReplicationFromNotification
thaliReplicationPeerAction

This work is done. I'm just blocked on https://github.com/pouchdb/pouchdb/issues/5114 which I have a proposed resolution to and https://github.com/pouchdb/express-pouchdb/pull/329 which I can't check in until we get the build system for express-pouchdb back up and running.


Use new QEMU test project to improve stream/message buffer tests

Description
Improve the robustness of the stream and message buffer tests - required various minor updates to StreamBufferDemo.c and MessageBufferDemo.c to account for variations in the execution pattern related to how far through a tick period tasks started executing and the priority of other test tasks.
Also check in the FreeRTOS/Demo/CORTEX_LM3S6965_QEMU project used to first investigate then update the StreamBufferDemo.c and Message BufferDemo.c files.  The QEMU project is useful for recording execution traces, using thread aware FreeRTOS plug-ins, and testing different GCC versions.
Test Steps
Run the CORTEX_LM3S6965_QEMU project as will be documented on the FreeRTOS.org site.  Leave the project running and observe the QEMU output.  Note the demo will eventually fail due to too many tick interrupts occurring between the the task that tests the stream buffer trigger level unblocking and subsequently executing - that failure does not occur on real hardware.
Related Issue
None.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

We'll likely need to add some user oriented instructions. Meanwhile, leaving a note for none Windows users.
These minor changes may be necessary --

chmod +x StartQEMU.bat
Watch out for the debug folder name. In my setup StartQEMU.bat has qemu-system-arm -machine lm3s6965evb -s -S -kernel ./Debug/RTOSDemo.elf
Correspondingly RTOSDemo Debug.launch has <stringAttribute key="org.eclipse.cdt.launch.PROGRAM_NAME" value="Debug/RTOSDemo.elf"/>

A clean Eclipse installation may need to setup path for QEMU and ARM toolchain as well.
I'm hitting "error in STRM" after a while, though per previous communication this is likely in QEMU port as we don't see it in hardware. We'll need to look into QEMU as a separate effort to this PR.


Remove 3.1 and 3.2; add 2.6, 3.5 and pypy

Also add setup test_suite to allow setup.py test to work.

Combined with #2, the results are all tests pass; see https://travis-ci.org/jayvdb/nested-dict/builds/89913563

Hello?
I am wanting to use this for pywikibot (https://gerrit.wikimedia.org/r/#/c/239040/).  If it isnt being maintained, we'll definitely need to vendor a forked version of it :/


Analytics events on artwork page.

This adds the rest of events defined here: https://docs.google.com/spreadsheets/d/1ld2IIex0SYWaDijeT10QYrPsKCDgw1uJJp6pAh2heCU/edit#gid=0
cc @anipetrov, @katarinabatina
@alloy - We seem to use to have a convention to add a special analytics- class to fields that are tracked by analytics. But we added so much of those here that I ended up not doing a separate class for each element and besides in our scenario the same class elements in different context dispatch different analytics records.
Btw... Wondering if that is something we need to think through with our new implementation of artwork/artist pages.

Btw regarding the artist/artwork pages, we’d be using react-tracking as we’ve been doing in Emission and a little in Reaction.


react-instantsearch-core introduces DOM types into react native Apps.

🐛 Current behavior
"react-instantsearch-core": "^7.13.1" introduces DOM types into react native apps. We have experienced this issue with 		"react-instantsearch-hooks": "^6.33.0" as well.
This causes invalid code in react native to be accepted as valid by tsc causing crashes at run time in our Apps. (At run time typescript/javascript support is provided by hermes.)
All variables and functions valid in DOM environment (eg. close() , the window global , ScrollY,  StyleSheet ) and thousands of other variables / functions are considered valid due to node_modules/typescript/lib/lib.dom.d.ts being included.
Via the tsconfig explainFiles flag i was able to determine the responsible node modules. https://www.typescriptlang.org/tsconfig/#explainFiles
node_modules/typescript/lib/lib.dom.d.ts
  Library referenced via 'dom' from file 'node_modules/preact/src/jsx.d.ts'
node_modules/preact/src/index.d.ts
  Imported via "preact" from file 'node_modules/instantsearch.js/..........' with packageId<EMAIL_ADDRESS>node_modules/instantsearch.js/es/index.d.ts
  Imported via 'instantsearch.js' from file 'node_modules/react-instantsearch-core/......' with packageId<EMAIL_ADDRESS>
🔍 Steps to reproduce

Add "react-instantsearch-core": "^7.13.1" to a react native App
In any source file import { InstantSearch } from 'react-instantsearch-core';
Edit any App source file and call the close() method.
run tsc (or within VSCode) . It fails to mark the close() call as an error as in a DOM environment close() calls the global window.close()

Live reproduction
https://itdoesntseemtobefeasibletoreproRN.com
💭 Expected behavior
tsc should flag close() as an undefined function when running under React Native.
Using instantsearch-code / instantsearch.js treats any code valid in a browser environment as valid, even when invalid for react-native.
Package version
"react-instantsearch-core": "^7.13.1"
Operating system
NA
Browser
NA
Code of Conduct

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

I wasn't aware of this issue, but we have a medium-term goal of splitting up InstantSearch.js into a core part that doesn't import preact and any browser code and one that does, which can then be used for React InstantSearch core too. Unfortunately for now I don't see a workaround without that amount of work


Error

📑 I have found these related issues/pull requests
I am unable to find any issues.
🛡 Security Policy

[X] I agree to have read this project Security Policy

Description

^ Above shows the full error $
I just randomly got this error.
👟 Reproduction steps

I first started Uptime Kuma
I then went and added a monitor
Then the error came up, saying about SQLite or something

👀 Expected behavior
For the monitor to do its heartbeat, like all other monitors. I did the 'push' one.
😓 Actual Behavior
2024-08-20T09:09:22Z [MONITOR] WARN: Monitor #8 'idk': Failing: No heartbeat in the time window | Interval: 60 seconds | Type: push | Down Count: 0 | Resend Interval: 0
🐻 Uptime-Kuma Version
1.23.13
💻 Operating System and Arch
Alpine Linux, v3.20.2
🌐 Browser
Opera GX
🖥 Deployment Environment

Runtime: nodejs v18.20.4, Docker 27.1.2 (i think)
Database: sqlite
Filesystem used to store the database on: disk
number of monitors: 2

📝 Relevant log output
2024-08-20T09:07:06Z [MANAGE] INFO: Delete Monitor: 1 User ID: 1
2024-08-20T09:07:06Z [DB] INFO: Delete Monitor completed in : 28 ms
2024-08-20T09:07:10Z [MANAGE] INFO: Delete Monitor: 4 User ID: 1
2024-08-20T09:07:10Z [DB] INFO: Delete Monitor completed in : 15 ms
2024-08-20T09:08:22Z [MANAGE] INFO: Resume Monitor: 8 User ID: 1
2024-08-20T09:08:22Z [MONITOR] INFO: Added Monitor: undefined User ID: 1
2024-08-20T09:08:46Z [MANAGE] INFO: Delete Monitor: 8 User ID: 1
2024-08-20T09:08:46Z [DB] INFO: Delete Monitor completed in : 1 ms
2024-08-20T09:08:53Z [MONITOR] INFO: Get Monitor: 6 User ID: 1
2024-08-20T09:09:01Z [MANAGE] INFO: Resume Monitor: 6 User ID: 1
2024-08-20T09:09:01Z [MONITOR] INFO: Get Monitor: 6 User ID: 1
2024-08-20T09:09:22Z [MONITOR] WARN: Monitor #8 'idk': Failing: No heartbeat in the time window | Interval: 60 seconds | Type: push | Down Count: 0 | Resend Interval: 0
Trace: [Error: insert into `heartbeat` (`down_count`, `duration`, `important`, `monitor_id`, `msg`, `status`, `time`) values (0, 0, true, 8, 'No heartbeat in the time window', 0, '2024-08-20 09:09:22.301') - SQLITE_CONSTRAINT: FOREIGN KEY constraint failed] {
  errno: 19,
  code: 'SQLITE_CONSTRAINT'
}
    at safeBeat (/home/container/server/model/monitor.js:1034:25)
2024-08-20T09:09:22Z [MONITOR] ERROR: Please report to https://github.com/louislam/uptime-kuma/issues

We have never found a way to semi-reliably reproduce that one.
Could you report what you did before this issue here https://github.com/louislam/uptime-kuma/issues/906?
(I am going to close this issue as a duplicate to keep the conversation on track)


Episode and Season numbering

The Plex XML output orders Seasons and Episodes. Plex JS uses this ordering to name episodes in the Plex GUI.
Should be able to set
episode_obj.episode_number = X
episode_obj.season_number = X
in the Episode class.
Also would be nice to have a definite integer attribute for season number as well, could be handled in the same way.

Expect a pull request for this added functionality in the next few hours. Working on it now.

The major problem with implementing this is the modular way that the XML parsing is done. When an item is built it doesn't have any context of where it was in the XML that came from Plex.
To pass this info down to the Season/Episode objects will require passing down an 'index' parameter into the object.
OR
after an item has been built, if it is part of a list it's index can be added to the item as an 'index' attribute, which could then be read. The downside to this is that the object is already created, and the listItems function doesn't know what type of object it is. It's a bit ugly to put in a 'if type == season' conditional in the listItems function, but it's also very ugly to persist 'index' as a parameter way down to the leaf object constructors.

I've been thinking about this problem a bit as well. The Season objects used to include index and parentTitle attributes, but seems to have been removed sometime in the last few months.  not sure about the best way to solve it.
The problem with knowing where you came from is not ideal either because you may not always to the the a Season object through it's Show. You can go directly there if you wanted. Maybe we do a combination of both.  Pass in the index if we have it, make a second request to the Show (from the Season) if we request index and do not have it yet?

Pull Request #55 should solve this issue. Only added 5 lines I think. Was pretty easy to add and I think it's a fairly elegant solution.

commit 36b5d12a192ac0574c33b0849f1087b17f805c9b
Author: Michael Shepanski<EMAIL_ADDRESS>Date:   Thu May 19 23:42:06 2016 -0400
Add back missing index value from Show,Season,Episode; Make season numbers a bit easier to fetch


Make TranslationServer singleton variable inline.

This change avoids the problem known as 'Static Initialization Order Fiasco' (SIOF).
We ran into this issue in a customized Godot build, but could hit anyone.
See the following PR for more explanation: https://github.com/godotengine/godot/pull/94683
Developed by Migeran

Thanks!


lutorpy, importing Error:

I've installed torch 7 and then I fire the command
sudo ~/torch/install/bin/luarocks install image
sudo ~/torch/install/bin/luarocks install nn
sudo pip install lutorpy
But now when I am in terminal python:
`aquib@javed:~$ python
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import lutorpy
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/lutorpy/init.py", line 17, in
lualib = ctypes.CDLL(os.path.join(TorchInstallPath, "lib/libluajit.so"), mode=ctypes.RTLD_GLOBAL)
File "/usr/lib/python2.7/ctypes/init.py", line 362, in init
self._handle = _dlopen(self._name, mode)
OSError: /home/aquib/torch/install/lib/libluajit.so: cannot open shared object file: No such file or directory

import lutorpy as lua
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/lutorpy/init.py", line 12, in
from lutorpy import torch_path
ImportError: cannot import name torch_path
`
how can I fix it?

You have problem in detecting the torch path, I suggest you do a reinstallation of lutorpy.

Hi @oeway , I tried this also but I am getting the same error:
I uninstall the lutorpy by using pip uninstall lutorpy
Then I downloaded the lutorpy from github and run the command:
sudo python setup.py install
>>> import lutorpy
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/lutorpy-1.3.7-py2.7-linux-x86_64.egg/lutorpy/__init__.py", line 17, in <module>
    lualib = ctypes.CDLL(os.path.join(TorchInstallPath, "lib/libluajit.so"), mode=ctypes.RTLD_GLOBAL)
  File "/usr/lib/python2.7/ctypes/__init__.py", line 362, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: /home/aquib/torch/install/lib/libluajit.so: cannot open shared object file: No such file or directory

can you check if you have "/home/aquib/torch/install/lib/libluajit.so" exist?

Hi @oeway , I just checked my folders and I didn't find file named "libluajit.so"

Ok, so that's the problem. Please give more details on your installation, system(mac osx is not supported so far), and did you install torch with the default way? Could you list the torch/install/lib folder see if there is a file named libluajit?

I am using Ubuntu 16.04,
git clone https://github.com/torch/distro.git ~/torch --recursive
cd ~/torch
clean old torch installation
./clean.sh
optional clean command (for older torch versions)
curl -s https://raw.githubusercontent.com/torch/ezinstall/master/clean-old.sh | bash
https://github.com/torch/distro : set env to use lua
TORCH_LUA_VERSION=LUA52 ./install.sh
New packages can be installed using Luarocks from the command-line:
run luarocks WITHOUT sudo
$ luarocks install image
$ luarocks list

Hi @aquibjaved , I see the problem, because lutorpy assume one install the luajit with torch, but you used lua5.2. The easiest solution would be install torch with luajit(the default installation), is there a particular reason for you to choose lua5.2? Otherwise we have to figure out how to use lua5.2 rather than luajit.

Hi @oeway, I don't need any specific lua version, I just want to make import lutorpy work.
What I have to do now?

How do I proceed now? Do I need to install it again from beginning?

you need to reinstall torch with luajit, not LUA5.2, you will find it on
torch website.
On Tue, 29 Nov 2016 at 03:38 Aaquib Javed Khan<EMAIL_ADDRESS>wrote:

How do I proceed now? Do I need to install it again from beginning?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/imodpasteur/lutorpy/issues/30#issuecomment-263460068,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdNy5QhPf6QpXvPVGFuJvFCHcnsWgiqks5rC5BDgaJpZM4K9mSd
.

Can I  add bash install-deps after ./install.sh, I did the installation without it, It's taking around 2 hours for the installation.

you can try ./install.sh without bash install-deps, if it succeed, then I think you don't even need  install-deps. That's because you have installed all the dependencies when you install torch for the very first time.

Hi @oeway , I did the installation again, and when I enter  th in terminal  torch window open.
But While importing Lutorpy the error now I am getting is:

Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import lutorpy
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/lutorpy/__init__.py", line 58, in <module>
    from lutorpy._lupa import *
ImportError: /usr/local/lib/python2.7/dist-packages/lutorpy/_lupa.so: undefined symbol: lua_rawlen

Try also to reinstall lutorpy with
sudo pip uninstall lutorpy
sudo pip install lutorpy

When I uninstall the lutorpy and again installed it shows me this:
Collecting lutorpy
Requirement already satisfied: numpy in /usr/local/lib/python2.7/dist-packages (from lutorpy)
Requirement already satisfied: future in /usr/local/lib/python2.7/dist-packages (from lutorpy)
Installing collected packages: lutorpy
Successfully installed lutorpy-1.3.7

I guess you forget to run source .bashrc to append torch to env, you can try:
source ~/torch/install/bin/torch-activate

I try it but it won't work.

Sorry, I can't reproduce your problem.
Could you try:
sudo pip install cython --upgrade
git clone https://github.com/imodpasteur/lutorpy.git
cd lutorpy
sudo python setup.py install --with-cython

And please also run
echo $LD_LIBRARY_PATH

and make sure you have '/home/xxx/torch/install/lib'

Hi I did this installation with cython

Maybe try
sudo ldconfig

Otherwise, I have no idea what happening.

Hi @oeway ,
aquib@javed:~$ sudo ldconfig
[sudo] password for aquib:
aquib@javed:~$ python
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import lutorpy
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/lutorpy/__init__.py", line 58, in <module>
    from lutorpy._lupa import *
ImportError: /usr/local/lib/python2.7/dist-packages/lutorpy/_lupa.so: undefined symbol: lua_rawlen
>>> exit()
aquib@javed:~$

Hi @oeway ,
Okay so finally I get rid of the error and now it's working.
So what I did is simple after installing lutorpy using cython, I hit the command   sudo pip uninstall lutorpy
and then I open python terminal and import lutorpy  and there's no error.

Thank you for the help @oeway

Great to hear that it works for you.


Bug found in path intersection code

Hi!
We're currently upgrading our dependency on canvas from commit 3711dfc3bb35 to 2f948bde1ebd, and one of our tests fails with the following panic:
panic: Bug found in path intersection code, please report on GitHub at https://github.com/tdewolff/canvas/issues with the path or paths that caused this panic.

goroutine 16 [running]:
testing.tRunner.func1.2({0x104f82240, 0x10513ea40})
	GOROOT/src/testing/testing.go:1631 +0x1c4
testing.tRunner.func1()
	GOROOT/src/testing/testing.go:1634 +0x33c
panic({0x104f82240?, 0x10513ea40?})
	GOROOT/src/runtime/panic.go:770 +0x124
github.com/tdewolff/canvas.pathIntersections(0x14002768b88, 0x14002768b88, 0x0, 0x0)
	external/com_github_tdewolff_canvas/path_intersection_util.go:614 +0x2f30
github.com/tdewolff/canvas.(*Path).Settle(0x14002768b88, 0x2)
	external/com_github_tdewolff_canvas/path_intersection.go:167 +0x398
github.com/tdewolff/canvas.(*Path).Stroke(0x1400000f170, 0x3fc999999999999a, {0x1051417e0, 0x1059ccb20}, {0x105141760, 0x1058fc840}, 0x3f8156f8c384071a)
	external/com_github_tdewolff_canvas/path_stroke.go:613 +0x260
github.com/tdewolff/canvas/renderers/rasterizer.(*Rasterizer).RenderPath(0x14002788000, 0x140026fe840, {{{0x0, 0x0, 0x0, 0x0}, {0x0, 0x0}, {0x0, 0x0}}, ...}, ...)
	external/com_github_tdewolff_canvas/renderers/rasterizer/rasterizer.go:84 +0x1a0
github.com/tdewolff/canvas.(*Canvas).RenderViewTo(0x1400007eba0, {0x10514e0e0, 0x14002788000}, {{0x3ff0000000000000, 0x0, 0x0}, {0x0, 0x3ff0000000000000, 0x0}})
	external/com_github_tdewolff_canvas/canvas.go:856 +0x294
github.com/tdewolff/canvas.(*Canvas).RenderTo(0x1400007eba0, {0x10514e0e0, 0x14002788000})
	external/com_github_tdewolff_canvas/canvas.go:841 +0x58

[... local part of stack trace censored ...]

testing.tRunner(0x1400065c340, 0x14002434080)
	GOROOT/src/testing/testing.go:1689 +0xec
created by testing.(*T).Run in goroutine 24
	GOROOT/src/testing/testing.go:1742 +0x318

The path seems to be this:
Path: len=9 data=M-0.1997406229376793<PHONE_NUMBER>925494L-0.1997406229376793<PHONE_NUMBER>3238177L-0.19974062293732664<PHONE_NUMBER>079834L-0.19999999999964735 20.77454766193328L0.19999999999964735 20.774546910730027L0.20025937706232075<PHONE_NUMBER>4458095L0.20025937706232072<PHONE_NUMBER>3238177L0.20025937706232072<PHONE_NUMBER>925494z
Intersection 0: seg=(1,2) t=(0.9999999972804681,1) pos=(-0.19974062293732664,158.8874019079834) dir=(270°,89.99994620346916°) tangent
Intersection 1: seg=(1,3) t=(0.9999999986401219,1.359651237533596e-09) pos=(-0.1997406229376793,158.88740172019808) dir=(270°,269.9998923980606°)
Intersection 2: seg=(2,3) t=(0,2.7195268509761408e-09) pos=(-0.1997406229376793,158.88740153238177) dir=(89.99994620346916°,269.9998923980606°) tangent

I'll try to reduce this down to a standalone test case, and will update this issue once I have created one.

Thank you Gregor for the report. This is a problem when resolving self-intersections of a path by "settling" (resolving or simplifying) it after applying stroke.
It looks like the stroke of an (almost) straight path and the problem appear numerical. Specifically, when checking the self-intersections in a path, it should remove the connections at the endpoints between consecutive line segments (since they always touch), but it doesn't appear to remove them in this case.
You have three line segments involved:
-0.1997406229376793<PHONE_NUMBER>925494 => -0.1997406229376793<PHONE_NUMBER>3238177
 -0.1997406229376793<PHONE_NUMBER>3238177 => -0.19974062293732664<PHONE_NUMBER>079834
-0.19974062293732664<PHONE_NUMBER>079834 => -0.19999999999964735 20.77454766193328

As you can see, the second line segment is of length 4e-7, this is still bigger than Epsilon (=1e-9 if I'm not mistaken), so I'm unsure why this is causing a problem. This may be a problem in the line-line intersection code w.r.t. numerical floating point problems. Will look into this.

I see, the problem is not so much in eliminating line-line intersections for self intersecting paths, it's that there is a numerical precision error for almost linear/parallel/overlapping lines. The commit seems to fix this issue, can you please check if it works?

The commit fixed the particular path in the initial report, now the test progresses a bit further to this one:
Path: len=9 data=M162.43449681368278 -9.999996185876773L162.43449681368278 -9.99998551284069L162.2344968136828 -9.999985512840697L162.23449681368282 -10.399974839804614L162.4344968136828 -10.3999748398046L162.43449681368278 -9.999985512840682L162.2344968136828 -9.99998551284069L162.2344968136828 -9.999996185876773z
Intersection 0: seg=(1,5) t=(0,0.9999733166977942) pos=(162.43449681368278,-9.999996185876773) dir=(90°,90.00000000000408°) tangent
Intersection 1: seg=(1,5) t=(1,0.9999999999999822) pos=(162.43449681368278,-9.99998551284069) dir=(90°,90.00000000000408°) tangent
Intersection 2: seg=(2,5) t=(0,0.9999999999999821) pos=(162.43449681368278,-9.99998551284069) dir=(180.00000000000202°,90.00000000000408°) tangent
Intersection 3: seg=(2,6) t=(0,0) pos=(162.43449681368278,-9.99998551284069) dir=(180.00000000000202°,180.00000000000202°) tangent
Intersection 4: seg=(2,6) t=(1,1) pos=(162.2344968136828,-9.999985512840697) dir=(180.00000000000202°,180.00000000000202°) tangent
Intersection 5: seg=(3,6) t=(0,1) pos=(162.2344968136828,-9.99998551284069) dir=(270.0000000000041°,180.00000000000202°) tangent
Intersection 6: seg=(3,7) t=(0,0) pos=(162.2344968136828,-9.99998551284069) dir=(270.0000000000041°,270°) tangent
Intersection 7: seg=(2,7) t=(1,6.657362817941556e-10) pos=(162.2344968136828,-9.999985512840697) dir=(180.00000000000202°,270°) tangent
Intersection 8: seg=(3,7) t=(2.6683302170291398e-05,1) pos=(162.2344968136828,-9.999996185876773) dir=(270.0000000000041°,270°) tangent
Intersection 9: seg=(3,8) t=(2.6683302170291398e-05,3.791925304638669e-18) pos=(162.23449681368282,-9.999996185876773) dir=(270.0000000000041°,0°) tangent
Intersection 10: seg=(5,8) t=(0.9999733166977942,1) pos=(162.43449681368278,-9.999996185876773) dir=(90.00000000000408°,0°) tangent

I haven't gotten around to packaging up our test case into a standalone thing, I'll do that ASAP.

FWIW, I get test failures when I run go test ./... on the main branch of this package, all of which look like the result of floating point inaccuracies.
I run my tests on an ARM64 macbook, and for our own code base we had to introduce some "fuzzyness" that we tolerate in test results because on ARM64 the Go compiler does automatic FMA fusing, but it doesn't do that on x86_64 (where our CI/production systems run).
Maybe something similar would be appropriate here as well?

Hi Gregor, thanks I'll take a look. Regarding the failing tests, there are many tests failing as they are outstanding bugs. I'm trying as much as I can to fix bugs that bother people, but there is still significant work left to fix all edge-cases, and until I have the time to do so, new bug reports will keep appearing hehe.
I didn't know about FMA fusing, thanks for that. I do check for floating point intervals, not exact values, so not sure if it is such a big problem? As I said, the tests fail as there are many outstanding bugs (and there are many commented tests that fail too) unfortunately :-(

Yeah, FMA is probably not the main problem here, you're right.
FWIW, I do appreciate your quick reaction to issue reports a lot, but by all means don't feel pressured to work on stuff just on my (resp. my employers) behalf. We're extremely grateful for the canvas package as it is already :)
I'll try and see if I can figure out what's wrong with out test case tomorrow over the day, maybe this is a "many eyeballs make all bugs shallow" thing :)

Thanks Gregor. What is your use-case?
The paths you've shown so far are quite "pathological" in the sense that they seem to be the result of something that generates suboptimal paths. The last case you sent for example, is a simple rectangle when viewing as SVG.  Would it make sense to first "clean-up" the path before applying stroke? I always meant to implement the Ramer–Douglas–Peucker algorithm, but haven't come around yet.

Our use case is plotting geographical surveys after projecting them, so cleaning up polygons is sadly not possible (because they represent real measured geographical features).
The process in general is that a user walks around with a GPS (or other GNSS) receiver and marks off positions on a construction site to track utility hookups, power/gas/water lines, etc...
We then project the measured points (which are in S^2) onto R^2, construct a set of additional lines that indicate things like "distance of this point from a vanishing line" and "distance from orthogonal of the vanishing line", resize and scale the whole thing to fit on, say, a sheet of A4 paper, and use the canvas package to render the whole thing.
The paths in my posts are all from the same test case, which admittedly is a constructed case of a smiley face, but one that could conveivably be constructed by walking across a field and marking off points.

I'm working to use this library for a very similar purpose, so better get this fixed! What ground resolution do you need? For example, if you are good with meter precision, or maybe centimeter precision, you could round all numbers to that precision first.
This bug in particular doesn't find an intersection between the end of the first line segment, and the start of the 6th line segment (position {162.4344968,-9.999985513}), due to numerical issues in the line-line intersection test (again). Trying to work on a better solution.

I've fixed part of the bug, but the Settle function outputs a bad path.


Unable to bind Cmd+` in key bindings

VSCode Version:
Version 1.16.0-insider (1.16.0-insider)
OS Version:
macOS High Sierra Beta7
Steps to Reproduce:

Press cmd+k cmd+s
Bind cmd+` to any command

Only cmd is displayed. I have disabled "Move focus to next window" option in the OS keyboard shortcuts settings, so nothing is intercepting that key combination. And I can register that key binding in atom.

Reproduces without extensions: Yes

@hgl
Are you using an ANSI, ISO or JIS physical keyboard ?

@alexandrudima American keyboard comes with MacBook Air, I guess it's ANSI?

I bound Cmd+` to Find. Interestingly, the find menu item in the top menu bar shows the keyboard shortcut, but pressing the key combination does nothing.

I'm asking about it, as Chromium has some nasty code where the scan codes of IntlBackslash and Backquote are swapped in the keyboard events for ISO keyboards -- see https://github.com/Microsoft/vscode/issues/24153#issuecomment-297986346.
We have some code which undoes the swap that Chromium does and I wanted to rule that out as a possible source for ` not working.
An ANSI keyboard would look like this (source -- https://apple.stackexchange.com/a/106059):

Yes, the keybindings labels reflect the configured keybindings. So the top level menu would show what we expect to receive.

The fact that we don't get a cmd + `  in the define keybinding box is problematic, as it indicates that we are not receiving the event. If you open this page in Chrome on your machine, is cmd + ` registered/received in there ?
https://jsfiddle.net/v1gz5c5m/1/

Pressing cmd+` gives
keydown --- code: Backquote, keyCode: 192, key: `
keydown --- code: MetaLeft, keyCode: 91, key: Meta

Chrome version Version 60.0.3112.113 (Official Build) (64-bit)

I was unclear in my description about binding to Find. I couldn't bind cmd+` to Find directly in the UI opened by cmd+k cmd+s, I manually added that binding to keybindings.json.
You're right, the event are not being received in vscode. But looks like it's fine in chrome, and I'm able to bind it in atom.

I'm sorry, but strangely it's working now. I don't think I touched any related settings.

I'm really not sure what would have caused this.


Does CodeIgniter support "exists" in SQL language?

For example,
SELECT … FROM table WHERE EXISTS (subquery)

This is a support question, and should be asked on the forum - we use github for bug tracking.
A quick search in the user guide suggests the answer to your question is "no", but I could be wrong and there could be work-arounds.


Allow tracking label to be set on namespace component.

Allow users to set the label for a searches created from a namespace.

This doesn't make sense in the current SDK, namespaces are gone.


Upgrade to spark 3.

#753 seems to not have much action, and was having issues with the xgboost upgrade.  So this PR leaves xgboost at it's current version, and focuses on Spark.  We can do xgboost as a separate PR.
Fixes #727 and sort of #751
Some notes:

Spark v3 added multicolumn input/output support for Binarizer, StringIndexer, and StopWordsRemover.  I did a "partial support" for this where we are now serializing any of the multi-column parameters like the thresholds/labels, but if we try to load the multicolumn into mleap, then an exception is thrown.

I /think/ it might need some tweaks to the spark deserialization for inputCols and outputCols.  I need to increase the test cases in a few places still but figured I'd post the PR to get any feedback.
Getting the full multicolumn support into mleap seems like it will be a larger change, so put that as out of scope for now.

There are a few new Estimators/transformers in spark 3 ml, didn't add those in since we can always do it later.

Tried to fix most of the deprecation warnings.

cc @ancasarb @voganrc @emitc2h

Ok, @voganrc @ancasarb I think we're ready for review.  Travis is passing 🎉
I think the most unanswered question is what we want to do with testing the "legacy" deserialization behaviors for the MultiInOutFormat classes mentioned in the description.  E.g., if someone tries to deserialize a spark v2 StringIndexer object into a spark v3 environment, then it should be backwards compatible -- however that logic is never tested because we can't make a spark v2 StringIndexer object in the test suite.  There are two options here:

Just don't worry about it.  Look at the code and convince yourself it is correct.
We serialize a spark v2 StringIndexer bundle (e.g., by using mleap master branch) and check it into the mleap repo so we can use it tests.

I lean towards option 1 because I think the code is pretty simple and it avoids polluting the repo with these legacy test artifacts.


Double arrows on the merge button?

I know we've talked about this before, but I was just looking at the merge button thinking that it might be nice to have arrows on both sides of it... Thoughts?

What we discussed before is that the arrow to the working directory doesn't necessarily happen. If there is a merge conflict (as there often are), then only changes are made to the working directory. Also it can be unclear where changes are coming from and going to with arrows.
Changes come from the repository (and possibly working directory, but not in most cases), which results in changes in the working directory, and either a fast forward (no new commit) or a new commit, which you might have to commit changes for.
If you think arrows on the merge button come closer to informing users that this is what Git is doing, add them, or something that does 😃

The way I see it, with or without arrows the merge button is lying about what it's doing compared to the other buttons. I don't know which is the better lie!


Specify TypeScript version in solution rather than IDE.

I would like to be able to upgrade to Visual Studio 2015 Update 1. The problem is that this will also update the TypeScript compiler to version 1.7. My code does not compile in version 1.7 for some reason so I cannot do this yet.
I think the TypeScript version should be a project option rather than tied to the IDE, that way I can upgrade Visual Studio 2015 to update 1 without having to fix any code and without having to upgrade the build server and get the whole team of devs to upgrade.
Is this possible?
Thank you.

try to use npm to install a local special version, and use glup.js to compile TypeScript for you project.
or maybe you can try to replace the tsc.exe...
here are links can help you:
#1110
Use specific TypeScript compiler version in Visual Studio

you can set the TypeScriptToolsVersion in your project to 1.6 (or which ever version that works for you) and your project will always build against this version of the compiler. The IDE plugin in is not tied to the compiler version, so upgrading the IDE is possible without upgrading the compiler.
Note that since the IDE plugin version is different, you might get different errors from that at build time.
See https://github.com/Microsoft/TypeScript/wiki/Setting-Compiler-Options-in-MSBuild-projects#toolsversion for more information about toolsVersion


Add support for Kafka backend

The tests are not yet ready.

@ralphm please review

Thanks for this! Can we come up with another name for the _kafka.py module? kafkalog ?

I could not name it kafka due to name collision with the Kafka client. I will rename it to kafkalog as you  suggested.

As a general remark, this code base uses the same coding style as the Twisted project: 2 blank lines between methods, 3 blank lines between module-level definitions (classes and functions).

@ralphm I renamed kafkalog to kafka, kafka-hosts to kafka-brokers, and fixed the style.

Turned out this solution has a problem that I discovered when I went beyond my vagrant box and deployed it in our staging environment. The problem is that the background thread created by the kafka-python library is lost during daemonization. A solution is to move kafka client creation to the startService method. See the following commit.

@ralphm @twonds are there any more comments? We have been running this branch in production for 5 days already, and everything seemed to be fine.

@twonds thanks!

Yay!


Issue running stormpath-sdk-angularjs.js (non minified) from bower

For some reason, the stormpath-sdk-angularjs.js has a unicode 0xa0 character in it right there on line 1405. It's not in the github repo, but it's in the version downloaded from bower.

Cannot reproduce with the latest version:
bower install stormpath-sdk-angularjs --save
bower stormpath-sdk-angularjs#* cached https://github.com/stormpath/stormpath-sdk-angularjs.git#1.1.0
bower stormpath-sdk-angularjs#*         validate 1.1.0 against https://github.com/stormpath/stormpath-sdk-angularjs.git#*

Tested against https://github.com/stormpath/angularjs-spring-boot-stormpath-example


Adding test for stride and write method for RadClass

Includes a test to ensure the stride variable is working correctly in RadClass. The test compares the length of the results and compares it to the expected number of samples when using a specified stride and integration. Resolves #13.
stride should advance the working timestamp, which varies independently of integration. This allows the integration window to overlap or be spaced out.
RadClass is also tweaked slightly, removing the store_data member variable. Instead, a write() method is included. Therefore, writing to file will no longer be automatic, but can be controlled by the user.

You should be able to test this without writing the CSV file. You already have the data available in the RadClass object.

Correct, I have updated each test case accordingly.

On a related note, I just realized that run_all() is where you also write data to the CSV file. That would be better as a separate method, perhaps meaning that you don't need a save_data member variable. If you had a memory issue that required you to save data as you process it (I don't think that's the case here, but maybe you are anticipating it?), things would be a little different, but you still might want to have it in 2 methods.

Good point. I think a while ago I had envisioned RadClass being used with multiple files. The user would provide a list of filenames that would all be processed by the same object and therefore their results would be saved automatically. In practice, each RadClass object tends to only handle one file, or at least one file at a time. Thus, I think the user probably would prefer control over what is and isn't written. This has been updated.

Finally, there is no way for the user to specify a different name for results.csv

Also updated, the implemented write() method takes a filename as an input.

Since this PR is small, I think it is okay to expand scope to include the introduction of the write() method.

I have changed the way that the spectra is constructed as you suggested. Now it is an nxm matrix with A_nm = n. expected is now calculated using the sum formula n(n+1)/2 with a correction for indexing being n(n-1)/2. I agree that this PR should be merged before #16.
It also looks like this may require a rebase, I will do this once we are ready to merge.

This looks good to me now.  I think it needs a rebase before tests can run

Rebase complete and tests are passing. I should note that I had to add a correction to the expected values for #19. Since collapse_rows now does averaging (i.e. divides by integration), so too must the expected values.

LGTM


Make site preview work on macOS 10.14.6

This fixes it for me, but rather than pushing it and breaking it for others, it would help to have the team members that run it locally confirm that it still works for them after this change.

This fails on 12.0 beta with
✘ Failed creating virtual environment
[pipenv.exceptions.VirtualenvCreationException]:   File "/usr/local/lib/python3.8/site-packages/pipenv/vendor/click/decorators.py", line 17, in new_func
[pipenv.exceptions.VirtualenvCreationException]:       return f(get_current_context(), *args, **kwargs)
[pipenv.exceptions.VirtualenvCreationException]:   File "/usr/local/lib/python3.8/site-packages/pipenv/cli/command.py", line 202, in cli
[pipenv.exceptions.VirtualenvCreationException]:       ensure_project(
[pipenv.exceptions.VirtualenvCreationException]:   File "/usr/local/lib/python3.8/site-packages/pipenv/core.py", line 570, in ensure_project
[pipenv.exceptions.VirtualenvCreationException]:       ensure_virtualenv(
[pipenv.exceptions.VirtualenvCreationException]:   File "/usr/local/lib/python3.8/site-packages/pipenv/core.py", line 505, in ensure_virtualenv
[pipenv.exceptions.VirtualenvCreationException]:       do_create_virtualenv(
[pipenv.exceptions.VirtualenvCreationException]:   File "/usr/local/lib/python3.8/site-packages/pipenv/core.py", line 934, in do_create_virtualenv
[pipenv.exceptions.VirtualenvCreationException]:       raise exceptions.VirtualenvCreationException(
[pipenv.exceptions.VirtualenvCreationException]: Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/virtualenv.py", line 2634, in <module>
    main()
  File "/usr/local/lib/python3.8/site-packages/virtualenv.py", line 860, in main
    create_environment(
  File "/usr/local/lib/python3.8/site-packages/virtualenv.py", line 1162, in create_environment
    install_python(home_dir, lib_dir, inc_dir, bin_dir, site_packages=site_packages, clear=clear, symlink=symlink)
  File "/usr/local/lib/python3.8/site-packages/virtualenv.py", line 1451, in install_python
    copy_required_modules(home_dir, symlink)
  File "/usr/local/lib/python3.8/site-packages/virtualenv.py", line 1341, in copy_required_modules
    dst_filename = change_prefix(filename, dst_prefix)
  File "/usr/local/lib/python3.8/site-packages/virtualenv.py", line 1273, in change_prefix
    raise AssertionError("Filename {} does not start with any of these prefixes: {}".format(filename, prefixes))
AssertionError: Filename<EMAIL_ADDRESS>does not start with any of these prefixes<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>'/Users/antares/Library/Python/3.9/lib/python/site-packages', '/Users/antares/.local/lib/python/3.9/site-packages', '/Users/antares/Library/Python/3.9/site-packages', '/Library/Python/3.9/site-packages']

python3 in PATH is a very long standing convention, can we assume that 3 as a VirtualEnv version name would also work? Nothing in this repo really requires 3.9.

How about now @michaelklishin?

I only have access to 11.6 right now but let's merge this. I can always run ./driver.py directly with any Python 3 installation on 12.0 betas.


Add naming blog post

Naming blog for Eric.
@ericrudder, questions for you:

Did you want some title other than "Names in Pulumi"?
Do you want to change the meta_desc?
Wasn't sure what tags we wanted for this, so I left it as just pulumi.
What do you think of the meta_image? I whipped this up, but probably could be better:

this is great ... a) sent you name in slack .. b) suggested edit to meta ...  c) tag is OK for now
d) the image is great!  i would just make the orange text "Infra-AsCode" or something less, um, poetic.


Fix timezone issues in scheduled messages

As far as I can tell the selected time is converted to UTC correctly in the frontend before it is send to the server. The server always uses UTC time. I'm unsure why there are still issues with the timezone ...


Create dataset loader for XED

Dataloader name: xed/xed.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?xed

Dataset
xed

Description
XED, a multilingual fine-grained emotion dataset. The dataset consists of human-annotated Finnish (25k) and English sentences (30k), as well as projected annotations for 30 additional languages, providing new resources for many low-resource languages.

Subsets
XED-VI

Languages
vie

Tasks
Sentiment Analysis

License
Unknown (unknown)

Homepage
https://github.com/Helsinki-NLP/XED/blob/master/subtitle-retrieval/students/pairs-vi.txt

HF URL
-

Paper URL
https://aclanthology.org/2020.coling-main.575.pdf

#self-assign

I think this one has Indonesian as well
https://github.com/Helsinki-NLP/XED/blob/master/subtitle-retrieval/students/pairs-id.txt
Should we include or not?

#self-assign


Question about --nbr_fracs

Hi everyone,
A wonderful package to link scRNA and scTCR!!
Recently I test the run_conga.py script and found that CoNGA scores in TCR2D was easily influenced by argument 'nbr_fracs'. When I use the default option (0.01)(above), it seems that less association between gex and tcr. If I set to 0.5 (below), obvious association was present. And you emphasized the smalllish nbr_fracs in your script. So is there any suggestions in selecting the suitable value for nbr_fracs?
by the way, is the "length" in TCR.csv necessary for analyses? I have a TCR.csv file without "length" (It is not standard output from 10x). I want to include this file in CoNGA input.
Looking forward to your reply. Thanks!

Hi there,
Thanks for trying conga! This is a really interesting question. I can see that using larger neighbor-fractions for building the neighbor lists might lead to more significant P values in some cases, but I wonder whether one might lose some "specificity" and see more generic assocations, like the tendency for CD4+ (or CD8+) TCRs to be more similar to one another, or for memory TCRs to share sequence features, versus TCR/GEX associations in smaller groups of cells. I don't think there's any correct answer. Note that the default suggestion is to combine both 0.01 and 0.1 neighbor fractions.
Take care,
Phil

From: anchormok @.>
Sent: Monday, September 25, 2023 12:22 AM
To: phbradley/conga @.>
Cc: Subscribed @.***>
Subject: [phbradley/conga] Question about --nbr_fracs (Issue #64)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Hi everyone,
A wonderful package to link scRNA and scTCR!!
Recently I test the run_conga.py script and found that CoNGA scores in TCR2D was easily influenced by argument 'nbr_fracs'. When I use the default option (0.01)(above), it seems that less association between gex and tcr. If I set to 0.5 (below), obvious association was present. And you emphasized the smalllish nbr_fracs in your script. So is there any suggestions in selecting the suitable value for nbr_fracs?
by the way, is the "length" in TCR.csv necessary for analyses? I have a TCR.csv file without "length" (It is not standard output from 10x). I want to include this file in CoNGA input.
Looking forward to your reply. Thanks!
[merge_graph_vs_graph_logos]https://urldefense.com/v3/__https://user-images.githubusercontent.com/40140273/270259679-4c3e684b-ea14-440d-9274-4307125c77c2.png__;!!GuAItXPztq0!mbI3vzLg3tBe4orrTMtgNPBrKUQB4xYO1jsODHYzd4ZQQrGXYSVorzxSiVkeLyPXoQKfQ1zi-eDxyPka_1AJ4tPW$
[test1_graph_vs_graph_logos]https://urldefense.com/v3/__https://user-images.githubusercontent.com/40140273/270259693-21355921-55ba-4e1e-9530-d2f7cf2f99f0.png__;!!GuAItXPztq0!mbI3vzLg3tBe4orrTMtgNPBrKUQB4xYO1jsODHYzd4ZQQrGXYSVorzxSiVkeLyPXoQKfQ1zi-eDxyPka_2tf-tfa$
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/phbradley/conga/issues/64__;!!GuAItXPztq0!mbI3vzLg3tBe4orrTMtgNPBrKUQB4xYO1jsODHYzd4ZQQrGXYSVorzxSiVkeLyPXoQKfQ1zi-eDxyPka_-DsSECW$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ABBNCH6VMTLMOFUDQK5W7ODX4EWMDANCNFSM6AAAAAA5FT6LNA__;!!GuAItXPztq0!mbI3vzLg3tBe4orrTMtgNPBrKUQB4xYO1jsODHYzd4ZQQrGXYSVorzxSiVkeLyPXoQKfQ1zi-eDxyPka_9xnBLur$.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks for your sugguestions!
After confirming the special cells cluster identified by CoNGA score, I used the Seurat to analyze their transcriptional characteristics and met some questions. For example, total 1400 interested cells with unique TCR sequence were identified by CoNGA (because CoNGA choose the representative cell for clone), but in my seurat object, 1585 interested cells with the share TCR sequence were found. Should I used the 1400 cells or 1585 cells for analyses? If the answer is 1585 cells, does it mean that the similar TCR sequence present the similar transcriptional characteristics?
Thanks again for your reply

by the way, I found the Top 9 DE genes logo in "graph_vs_graph_logos.png" were too crowded to present. Is there some parameters to solve this item??
Thanks a lot!

Re: 1485 vs 1500, it probably depends on your dataset, and there may not be much difference. Generally speaking, cells with the same clonotype do tend to have similar GEX profiles, which supports conga's reduction to clonotypes. But in a dynamic setting there can be substantial heterogeneity.
Regarding the logos, the genes themselves are probably getting written out somewhere to a TSV file. But the TCR logos also don't look good, so it might be worth fixing that and then seeing if the DEGs are more interpretable. I think the problem is with the conversion from SVG to PNG. If you attach the log file, maybe we could figure out which tool is being used and swap it out for something that performs better on your system. I find Inkscape to be the most reliable across linux and mac.

An excellent suggestion!!
The question about DEG and TCR logo presentation have been solved since I employed the "Inkscape" for the conversion from SVG to PNG.
Thanks for your reply!


Can't change value of NIMBLE parameter BLE_GATT_MAX_PROCS (IDFGH-5951)

Environment
Development Kit: none
Module or chip used: ESP32-WROVER
IDF version (run git describe --tags to find it):
C:\repo\ble_test\dicm\esp32>git describe --tags
HMI_Rubicon_v1.4.0-12-g0d53d8d4
Build System: CMake
Compiler version (run xtensa-esp32-elf-gcc --version to find it):
xtensa-esp32-elf-gcc (crosstool-NG esp-2020r3) 8.4.0
Operating System: Windows
(Windows only) environment type: Plain Command Prompt
Using an IDE?: No
Power Supply: USB
Problem Description
It is not possible to set parameter MYNEWT_VAL_BLE_GATT_MAX_PROCS in the config tool.
I use ESP-IDF version 4.2.2
Expected Behavior
Should be possible to alter the NIMBLE config parameter MYNEWT_VAL_BLE_GATT_MAX_PROCS
Actual Behavior
It is not possible to change parameter MYNEWT_VAL_BLE_GATT_MAX_PROCS  without altering an IDF file which means that we need to fork the IDF if we want to change the parameter.
The parameter is set in this file:
\components\bt\host\nimble\port\include\esp_nimble_cfg.h
Steps to reproduce
Code to reproduce this issue
Debug Logs
Other items if possible

Hi @JonasAxelsson ,
Currently, the options exposed via menuconfig that user can select to configure were primarily decided based on common use cases and also exposing too many options may lead  to confusion. However, will revisit the list and expose such options for user to configure.
Thanks,
Rahul

As part of new release versions, BLE_GATT_MAX_PROCS is included to be configured via menuconfig. Closing this issue.

In which version of the IDF is the fix included?

As part of new release versions, BLE_GATT_MAX_PROCS is included to be configured via menuconfig. Closing this issue.

In which version of the IDF is the fix included?


Solution to the issue #84

Solution to Fibonacci problem with involving multiple approaches having different Time complexity and Space complexity

@Sar-taj107 please let me know if any changes required
also can you tag this pr as 'hacktoberfest accepted' once merged

@Sar-taj107 HAcktoberfest shows that this repo is excluded this year


How setting about javadoc and resource?

I can‘t see javadoc and resource on vscode of maven project

but I hava javadoc and resource

@nightmare-mio What if you close the editor tab: DynamicRegistrationBean.class, reload the VS Code?

老样子,
The same as usual.

oh,today is ok.

@nightmare-mio
It's because the setting java.maven.downloadSources by default is false. Which means by default, the source jar will only be downloaded when you have open the class file at least once.
And before the source jar is downloaded, only stub source will be displayed.


Use postMessage for Discv5 worker thread

Motivation

Right now we use Observable inside threads library to inform a new enr to main thread from discv5 worker thread, the profiler shows that it takes almost 10% on a mainnet node subscribing to all subnets

Description

Use postMessage instead
Cannot see any differences after I run this branch for 3 hours so need to take a profile and a benchmark test to see

TODO

[x] Make sure this works and we can have a lot of discovered peers/ENRs
[ ] Let this branch run for a while and take another profile to see if postMessage is better or not
[ ] Write a benchmark to see this is better or not

Observable is only 2.8% slower than postMessage (68s vs 70s) if I sent 10M messages from worker thread to main thread, benchmark is in https://gist.github.com/tuyennhv/c8dfcaa8931e19b1ffa6b9558ee28de5 and https://gist.github.com/tuyennhv/d6f441da0aa174899821fc708fd9b8ad

@tuyennhv can you post a screenshot of what specifically is using the 10% of cpu time?

this isn't happening anymore after #5073 see description

@tuyennhv can you post a screenshot of what specifically is using the 10% of cpu time?

@dapplion it was originally from https://github.com/ChainSafe/lodestar/issues/5084 , I don't think it happen anymore after some enhancement/improvement @wemeetagain made


onClick listener not working on MovingAverageTooltip, but working on OHLCTooltip

Hello,
I start playing a little bit with the first example (CandleStickChartWithDarkTheme) and I'd like to add a click listener on the moving average tooltip (component name: MovingAverageTooltip).
I cant' find why my "onClick" props is never called:
<MovingAverageTooltip onClick={() => console.log('clicked')} origin={[-38, 10]} calculators={[ema20, ema50]}/>

Many thanks for your input on this!
cheers

It is a bug in 0.6 beta,
can you add this to your styles and let me know if that fixes it?
.react-stockcharts-moving-average-tooltip {
    pointer-events: all;
    cursor: pointer;
}

Once you confirm I can have it fixed.

that's it! I added it to re-stock.scss, not sure if it's the best place but I can see my click now ! yay, Many thanks :)

It is fixed in 0.6-beta.5 you can remove that css from your stylesheet

Many thanks !
于 2016年10月28日 GMT+08:00 下午6:47:35, Ragu Ramaswamy<EMAIL_ADDRESS>写到:

It is fixed in 0.6-beta.5 you can remove that css from your
stylesheet
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/rrag/react-stockcharts/issues/157#issuecomment-256891433


Scrollbar too small

The current scrollbar (I'm testing on Ubuntu) is too thin, such that it's impossible to drag without switching to the native frame. Whenever you go close, the pointer switches to resize instead of drag.
Using the native frame fixes this, but I wanted to raise the question since I imagine it's fairly simple: Can the scrollbar be made longer?

Yep, a problem I've been thinking about. I'll try to reprioritize it :) thanks for the feedback


AE fishing incompatablity

Fishing with improved camera causes the player to get stuck in first person and unable to open 3rd person perspective.

Is that with Hotfix 1?

fwiw I tested Skyrim Fishing SE and had no problems. I'm using Hotfix 1. Just letting you know.

Looking at fishing in 1.6.353 with and without IC installed, it always goes into First Person no matter what and you cannot enter third person looks like the POV switch is blocked.
It shouldn't have got stuck before Hotfix 1 was available as it is just in first person camera all the time and I see nothing untoward that would get it stuck.
If your still getting stuck then it has to be something else not Improved Camera.

Updating to hotfix 1 fixed this for me.

Awesome!


Serialize calls to PrepareLayer due to Windows bug

This is a fix for timeouts in PrepareLayer. It is a temporary workaround until a platform fix is available. It is still possible to hit this on low spec or heavily loaded systems
/cc @jhowardmsft @jstarks
Signed-off-by: Darren Stahl<EMAIL_ADDRESS>

LGTM

LGTM


Add Office 365 proper detection

Please add proper Office 365 detection so it will show Microsoft (product name) 365 in Large Icon description instead of Microsoft (product name) 2016.

I've been scanning through the documentation provided by Microsoft,
and I don't think this is possible.
I've searched for 365 and subsc in the following links:

https://docs.microsoft.com/en-us/dotnet/api/microsoft.office.core?view=office-pia
https://docs.microsoft.com/en-us/dotnet/api/microsoft.office.tools?view=vsto-2017

There's also no obvious signs of Office 365 in the MainModule.FileVersionInfo
i used this script to dump this info
Debug.WriteLine(ObjectDumper.Dump(Process.GetCurrentProcess().MainModule.FileVersionInfo));
{System.Diagnostics.FileVersionInfo}
  Comments: ""
  CompanyName: "Microsoft Corporation"
  FileBuildPart: 11425
  FileDescription: "Microsoft Word"
  FileMajorPart: 16
  FileMinorPart: 0
  FileName: "C:\Program Files\Microsoft Office\root\Office16\WINWORD.EXE"
  FilePrivatePart: 20202
  FileVersion: "16.0.11425.20202"
  InternalName: "WinWord"
  IsDebug: False
  IsPatched: False
  IsPrivateBuild: False
  IsPreRelease: False
  IsSpecialBuild: False
  Language: "Language Neutral"
  LegalCopyright: ""
  LegalTrademarks: ""
  OriginalFilename: "WinWord.exe"
  PrivateBuild: ""
  ProductBuildPart: 11425
  ProductMajorPart: 16
  ProductMinorPart: 0
  ProductName: "Microsoft Office"
  ProductPrivatePart: 20202
  ProductVersion: "16.0.11425.20202"
  SpecialBuild: ""

Unless I am dumb dumb,
I can't find obvious ways, either in the docs and Google.
If you (or anyone else) can find any way of doing this, please get back to me.

Well there's this clear indicator, but the program can't check for that


Creation of deployment from subscription can take some time

The deployment gets created when the subscription installs the kyverno
helm chart.  Sometimes this can take a while for the helm install to
start so the default timeout is really a bit short.  Increasing it to
reduce the concern here.
Signed-off-by: Gus Parvin<EMAIL_ADDRESS>

/cc @JustinKuli

@gparvin, I'm going to bring this into my https://github.com/stolostron/governance-policy-framework/pull/386 - I don't think your PR can pass without the gatekeeper fix in mine, and mine probably can't pass without the change here 😂


It seems non-ascii unicode chars are getting stripped from js files

E.g. rails-assets-d3 (I checked versions 3.5.5, 3.5.4 and 3.5.3), file app/assets/javascripts/d3/d3.js:1261 should have pi and epsilon symbols as variables names (as it used to), but recently it became not to (so syntax error is arised).
This is caused by processing files content with encode!('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '').

+1

Thanks for noticing this. It's most likely because of https://github.com/rails-assets/rails-assets/commit/742f2d12c55f2933a428fb051a884a08a6a232fa

Thanks a lot! Build passed

@sheerun, the same problem with rails-assets-moment still exists

Problem with rails-assets-moment resolved


Enhance environment variables UI

Introduces

Different tabs for Runtime and Installation variables
Based on the selected user mode hide/show azure, ldap-related config variables

Updated UI :

@jugnu-appveen


Refactor cfclient

Adjust sample to reflect changes in SDK. Should be integrated on SDK release to nuget

@milos85vasic-harness FYI


Add automatic uniform for scene background color

Our app sometimes changes the scene's background color. We want our shaders to be able to adjust fragment color based on the background color in order to ensure proper contrast.

Thanks @pmconne!  Have you sent use a contributors license agreement?  If not, please send one over so we can review this.  You can find instructions here: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/CONTRIBUTING.md#contributor-license-agreement-cla
It also looks like you have a failing unit test.

@hpinkos, @pmconne is covered under the Bentley CLA so he's good to go.

Ah, thanks @mramato.  @pmconne, can you add yourself to CONTRIBUTORS.md under the corporate CLA section?  Thanks!

Hannah, my name is in there on the 3d-tiles branch.
It's not clear to me how to interpret the output from CI. Skimming through the raw log I see the following, neither of which seem relevant to my changes (pardon my github naivete):
Renderer/Texture
âœ— can get the size in bytes of a texture
Error
at new DeveloperError (Source/Core/DeveloperError.js:44:19)
at new Texture (Source/Renderer/Texture.js:125:23)
at expectTextureByteSize (Specs/Renderer/TextureSpec.js:565:19)
at Specs/Renderer/TextureSpec.js:578:9
at Object. (Specs/customizeJasmine.js:80:30)
23 03 2017 12:50:55.730:WARN [web-server]: 404: /url
ERROR LOG: 'An error occurred while rendering.  Rendering has stopped.
undefined
foo'
ERROR LOG: 'An error occurred while rendering.  Rendering has stopped.
undefined
foo'
Electron 1.6.2 (Node 7.4.0): Executed 8608 of 8614 (1 FAILED) (skipped 6) (48.867 secs / 47.148 secs)
TOTAL: 1 FAILED, 8607 SUCCESS
23 03 2017 12:54:18.158:WARN [web-server]: 404: /url
ERROR LOG: 'An error occurred while rendering.  Rendering has stopped.
undefined
foo'
ERROR LOG: 'An error occurred while rendering.  Rendering has stopped.
undefined
foo'
23 03 2017 12:54:21.735:WARN [web-server]: 404: /base/Build/Specs/TestWorkers/createBadGeometry.js
Electron 1.6.2 (Node 7.4.0): Executed 8608 of 8614 (1 FAILED) (skipped 6) (1 min 6.343 secs / 1 min 3.496 secs)
TOTAL: 1 FAILED, 8607 SUCCESS

Okay, thanks @pmconne!  It took a closer look, and that test is failing in the 3d-tiles branch and is unrelated to your change.
@lilleyse do you want to review this PR?

Shouldn't this change go directly into master instead of 3d-tiles?

The changes all look good. Can you add a test in AutomaticUniformsSpec?

@pmconne thanks for the contribution!

Shouldn't this change go directly into master instead of 3d-tiles?

Yes, @lilleyse can you please cherry-pick these and the previous CONTRIBUTORS.md change into a PR into master?

Made a small style tweak, otherwise looks good!


Update README to improve first encounter

I had problems when I tried react-native-webview for the first time in a freshly created react native 0.60 project. It just showed a blank page because width and height were zero. Scaling the page should at least show something and improve the initial experience with this library.

I'm not sure why settings scalePagesToFit fixes your issue but please keep in mind that WebView is just a view so if you want it to fill your screen please use flex: 1

I'll update the example to reflect this


upgrade frontend-maven-plugin to 1.7.5 test

upgrade frontend-maven-plugin to 1.7.5 test

Codecov Report

Merging #2057 into dev will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #2057      +/-   ##
============================================
- Coverage     33.15%   33.14%   -0.01%
  Complexity     1553     1553
============================================
  Files           313      313
  Lines         15529    15529
  Branches       1886     1886
============================================
- Hits           5148     5147       -1
- Misses         9852     9853       +1
  Partials        529      529

Impacted Files
Coverage Δ
Complexity Δ

...scheduler/alert/template/AlertTemplateFactory.java
42.85% <0%> (-14.29%)
2% <0%> (ø)

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 8b894de...49f4677. Read the comment docs.

not work as expected, close


runtime: implement CPU capability detection for openbsd/arm64

Most arm64 runtimes make use of auxv HWCAP data in order to determine what CPU features are available. OpenBSD does not currently provide auxv - we either need a way for the kernel to provide this information, or we need to implement CPU capability detection in userland.

We'll probably face the same problem for netbsd/arm64 (#30824) and freebsd/arm64 (#24715), I assume?

on freebsd/arm64 it's done here: https://github.com/MikaelUrankar/go/commit/b46aa4ee007eedd2b7be40ac46dce507a39225b3#diff-ea01e2d7a6eb362c2114bdfca93fdaf2R42
(that code should also work on modern linux btw)

AFICT that code requires emulation of the MRS instruction in the kernel (as described in https://github.com/torvalds/linux/blob/master/Documentation/arm64/cpu-feature-registers.txt for Linux). I'm not sure if OpenBSD offers this.


Unable to make SpeechRecognizer work with RAW PCM Audio

String path = Environment.getExternalStorageDirectory() + "/input/whatstheweatherlike.pcm";
PullAudioInputStream pullAudio = AudioInputStream.createPullStream(new BinaryAudioStreamReader(path));
SpeechRecognizer reco = new SpeechRecognizer(config, ac);
assert(reco != null);
        Future<SpeechRecognitionResult> task = reco.recognizeOnceAsync();
        assert(task != null);

        SpeechRecognitionResult result = task.get();
        assert(result != null);

However there is not OUTPUT from the API END and the JVM Crashes.
I also tried AudioConfig.fromWavFileInput(test.raw);
Request to please guide, whether I am doing anything wrong.
How to send raw PCM MONO 16 HZ sample rate and 16-bit PCM audio to the REST/WEBSOCKET ENDPOINT

Are you able to run the build and run sample here? https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/master/samples/java/jre/console/src/com/microsoft/cognitiveservices/speech/samples/console/SpeechRecognitionSamples.java#L225

Yes i tried these combination. The WavStream method is looking for the headers of the wav file.... since PCM data is raw and dosent have headers it seems its not working

Please see https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/588 for a suggested working example.


[Issue 14783][pulsar-broker] Fix javadoc warnings on pulsar-broker

Fixes #14783
Motivation
Fixing the javadoc warnings on the executing maven build for pulsar-broker.
Modifications
Changes done in the Java doc which is added as part of code.
+<
- <

+>
- >

+@link:
-@link

+and
-&&

Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[ ] no-need-doc
(Please explain why)

[x] doc
(If this PR contains doc changes)

Update
Screenshots for Shedder Class Doc
ThresholdShedder

UniformLoadShedder

/pulsarbot run-failure-checks

@ankitagrahari  I think you should rebase to the master branch to get CI working.

@ankitagrahari I think you should rebase to the master branch to get CI working.

Hi @RobertIndie,
I see it is up to date. Please let me know if I am missing something

@ankitagrahari I see that the master branch of your own repo is not synchronized with the main repo(apache/pulsar).

Please synchronize it. And pull it to your local master branch.
Then you can rebase or merge your branch.

/pulsarbot run-failure-checks