[SPARK-12072][PYTHON][SQL][WIP] Python jdf dataframe schema breaks

What changes were proposed in this pull request?
For large schemas, wrap the Java version of the schema instead of parsing the JSON.
How was this patch tested?
Unit tests & manually locally run the SQL test suite wrapping all of the schemas
This is a WIP PR - hopefully the original creator of the issue can also verify this fix solves their problem.

Test build #53648 has started for PR 11856 at commit c920e19.

Test build #53648 has finished for PR 11856 at commit c920e19.

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

Merged build finished. Test FAILed.

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

jenkins retest this please

Test build #54101 has started for PR 11856 at commit c920e19.

Test build #54101 has finished for PR 11856 at commit c920e19.

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

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

Merged build finished. Test FAILed.

Test build #54205 has started for PR 11856 at commit 5eeacc9.

Test build #54205 has finished for PR 11856 at commit 5eeacc9.

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

Merged build finished. Test FAILed.

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

Test build #54209 has started for PR 11856 at commit 9fe40fe.

Test build #54209 has finished for PR 11856 at commit 9fe40fe.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #54211 has started for PR 11856 at commit 6a7c9c3.

Test build #54211 has finished for PR 11856 at commit 6a7c9c3.

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

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

Merged build finished. Test PASSed.

Test build #54880 has started for PR 11856 at commit e2e5906.

Test build #54880 has finished for PR 11856 at commit e2e5906.

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

Merged build finished. Test PASSed.

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

I'm going to go ahead and close this - happy to update to master if the original reporter can confirm this approach works for them.


uv-resolver: add some tracing logs for when we filter requirements

Specifically, these are emitted when requirements fail to satisfy
Requires-Python or the markers associated with the current fork in the
resolver.
Closes #4373

Did you check for how noisy this is?

Did you check for how noisy this is?

I tried locking with the pyproject.toml in #4133, and the result is that about 10% of all TRACE logs correspond to this addition. And that also reflects about 5% of total DEBUG logs. So I'd say this seems fine. It does seem like pretty useful debugging info.


feature/setup-deployment

Deployment
Configure client-side routing for running a Single Page Application (SPA) on Heroku.

Looks good, let's merge!


Add a Reverse Sort property to BoxContainer

Describe the project you are working on
A card game where you need to select and scroll throw many card slots
Describe the problem or limitation you are having in your project
I want a way that my card slots can overlap each other when activated and that the upper card slot in the scene tree takes the input event.
https://github.com/user-attachments/assets/ed2441da-a876-41a7-a9df-a756e990562b
Describe the feature / enhancement and how it helps to overcome the problem or limitation
A way to fix this would be having a boolean that sorts the items in reverse, same as in css flex-direction row-reverse.
This way you can have the upper item in the screen to be lower in the scene tree, so the input events will trigger in the different order.
Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams
Having a boolean named sort_reverse available in the editor
void BoxContainer::_resort() {
    ...

    for (int i = 0; i < get_child_count(); i++) {
        int index = i;
        if (sort_reverse) {
             index = get_child_count() - 1 - i;
        }
        Control *c = as_sortable_control(index);
    }
}

If this enhancement will not be used often, can it be worked around with a few lines of script?
This can not be worked around, the only way i achieved it was writing the same logic for the BoxContainer in GDScript and adding the logic of the reverse sorting
Is there a reason why this should be core and not an add-on in the asset library?
This should be directly integrated in the BoxContainer Node

Seems like a good change to me, I'm curious why this isn't a thing yet


Document localization string method for dropdown-like fields

related: https://github.com/wintercms/winter/pull/910/files

Just as a suggestion, because we're now up to the 7th way of populating options - could we perhaps create a new section for this in this particular page, outside of the field type documentation? Option population does affect some other fields, and it would make it easier to find as opposed to within the middle of the dropdown field doc.

@bennothommo I restructured as you suggested.


CodeOfConduct

As discussed elsewhere, please consider adding a Code of Conduct.  The sibling repository adopted: https://github.com/tlaplus/tlaplus/blob/master/.github/CODE_OF_CONDUCT.md

I support adding a code of conduct. Thank you for suggesting it.

Addressed in commit 77cd0c0b09dc373745816d22c7617fc01d591c93.


Dedent docker-enter script

Exit after printing usage, which gets rid of the big else block.
Diff without whitespace:
diff --git a/docker-enter b/docker-enter
index 97c6b2f..6cc75cd 100755
--- a/docker-enter
+++ b/docker-enter
@@ -19,9 +19,7 @@ if [ -z "$1" ]; then
     echo ""
     echo "Enters the Docker CONTAINER and executes the specified COMMAND."
     echo "If COMMAND is not specified, runs an interactive shell in CONTAINER."
-    exit
-fi
-
+else
     PID=$(docker inspect --format "{{.State.Pid}}" "$1")
     [ -z "$PID" ] && exit 1
     shift
@@ -49,3 +47,4 @@ else
         # "$@" is magic in bash, and needs to be in the invocation
         $LAZY_SUDO "$IMPORTENV" "$ENVIRON" "$NSENTER" $OPTS "$@"
     fi
+fi

That makes sense. Thanks!


F6 Slider ignores decimal setting

I have double range slider on number inputs that have step set to 10 and the configuration of the slider has decimal set to 0 but when I move the handles it always uses two decimal places.

I think this:
this.options = $.extend({}, Slider.defaults, this.$element.data(), options);
should be this:
this.options = $.extend({}, Slider.defaults, this.$element.data('options'), options);

It does not yet work with stepping, it's a near-future feature, expect that some time next week.
Also, it is definitely the first one. The $element.data('options')  is processed through Foundation.core

@zurbchris I think this has been fixed as of 6.0.6, yes?

the decimal issue yes, the stepping, no

Implemented with #7910


MRG: Minor fixes

Fixes 3 minor problems:

eLORETA had threshold issues if inv was saved and loaded (threshes were for float64 but I/O roundtrip makes them float32; also neg eigenvalues should be treated as zero)
base.py change should fix the issue brought up on the mailing list about coercing a filename None to unicode
Use hann instead of hanning as it's deprecated in scipy

LGTM
we should probably backport this right?
can you take care of it with also a what's new update? thanks heaps

Done


Run MathJax in C# using Javascript.Net

Is it possible to run complex JS code like MathJax in C# using Javascript.Net?

JavaScript.Net doesn't include browser-specific bits of JavaScript (like the DOM).  If the code relies on that then it won't work.  The normal standard JavaScript library (e.g. Math.max()) is present.


Improve report image appereance

The basic features of #7 are covered but the appearance of the report details needs some work.
The current implementation for generating the image and adding the test uses the Pillow library, which is not designed specifically for generating images based on a table.
An alternative approach would be to save the report data into a pandas data frame which would then be converted into a table and saved as a .png file.  I had to revert my attempt of this since the dfi.export method caused an OSError error: Chrome executable not able to be found on your machine
https://github.com/luisgc93/stock_reminder_bot/commit/c5d88fcc7c979f5a70d4d0bd6bc84fabd49553c6
The approach would be something like this:
https://stackoverflow.com/questions/35634238/how-to-save-a-pandas-dataframe-table-as-a-png
I was able to create this locally but it raised an OSError when deployed:

In order to make this work, try adding chrome through docker

In order to make this work, try adding chrome through docker

Fixed:
https://twitter.com/stock_reminder/status/1350505139375894530?s=20

Fixed:
https://twitter.com/stock_reminder/status/1350505139375894530?s=20

Is alpine chrome what you ended up using to make this work? Facing a similar issue right now, and wondering how to solve it.
Really cool project btw!

Is alpine chrome what you ended up using to make this work? Facing a similar issue right now with dataframe_image package, and wondering how to solve it.
Really cool project btw!

Hi! Glad you like the project!
I was able to save the image locally but setting up the chrome driver on heroku was an absolute nightmare. Instead, I ended up using this library:
https://pypi.org/project/dataframe-image/
If I remember correctly, what made this approach work was using table_conversion=None which bypasses the default table_conversion='chrome' and therefore doesn't need a chrome driver:
https://github.com/luisgc93/stock_reminder_bot/commit/ede93620e7ac940b7b4b5c8b00a25b113ad2e84d

Appreciate the prompt reply! Got it working with table_conversion=None. Of course it was that simple lol. Thank you for saving me from hours of googling how to deploy headless chrome on Lambda.


How to create a custom cluster for Matter device? (CON-541)

Hello,
I want to create custom cluster for matter device, how can I create the cluster? Is any there article regarding this....?

Please refer https://github.com/espressif/esp-matter/issues/427


Feat(IconDefault): retrieve all options from CSS

Should fix #4968
This PR attempts to improve the Default Icon automatic images path resolution, especially for build engines (typically webpack) which can be configured to manage images linked through image CSS data types (e.g. url(path/to/image.png)) and re-write them, hence breaking the Leaflet path detection and re-building (the previously hard coded marker-icon.png file name no longer exists after the build engine renamed it or inlined it as base64 data).
To achieve this, this PR takes the radical approach to outsource all Default Icon option values to the CSS (image url's including file name, sizes and anchor positions, for icon, shadow and their Retina version) and retrieve all these values the first time _getIconUrl is called.
That way, everything related to the default icon images is located in one place (the CSS) and can be easily modified by the app developer (and the build engine…)
Due to this shift, IconDefault no longer has default options (except for a new option classNamePrefix, more on that below), and when retrieved from CSS, the option values are applied to the instance options rather than on IconDefault.prototype.options.
That way, it makes it possible to use other instances of IconDefault with a different classNamePrefix option to target another set of CSS rules and define a new icon through that CSS. May not be very useful outside the default icon, but still an extra functionality.
There is however a change in how imagePath is applied: it must be set before the first call to _getIconUrl occurs (typically before any Marker is added to a map), because it is no longer used after that. Whereas previously it can be changed dynamically and affect re-drawn Markers.
Another tweak is the margin (top and left) used to specify the icon (and shadow) anchor: to be closer to how currently we can position an icon through its className with negative margins, I took the approach to use negative margins here as well, although the iconAnchor (and shadowAnchor) typically have positive values. Then the margin values are reverted to get the positive options. (but then Leaflet re-reverts them to negative margins to make the actual positioning…)
Finally, I replaced the use of background-image by cursor CSS rule in order to workaround the high contrast / colours overriding mode in Firefox (see https://github.com/Leaflet/Leaflet/issues/4968#issuecomment-328277840).
Note that both the normal url and its Retina version are specified as multiple images in this cursor rule, as allowed by the spec.
Tests:

Added 1 test for imagePath.
Adjusted Popup and Marker tests to account for the new IconDefault behaviour.

Quite some code for this simple feature, so I am expecting some critics in review(s)… :-)

@perliedman might you have time to review this soon? Ran into this issue again today; having the fix merged would be great!

@ghybs I remembered a bug I had once with a cursor image in IE11.
Look like your solution may not work, if leaflet stylesheet will be placed not in site's root directory.

@cherniavskii thank you for the link!
Then it is impossible to satisfy both IE11 and Firefox high contrast option, unless we duplicate the CSS rule (having both background and cursor) so that we can fallback to cursor in case the former is empty...
Sigh even more bytes to add...

@ghybs thanks for your thoughtful work and comments regarding this feature! Also sorry about taking forever to review this.
I think I generally like this - I think this approach makes a lot of sense given the issues we've had. The major downside I can see is probably that it's not entirely intuitive to figure out which CSS property that corresponds to which icon option, but I don't think we can do much more than just document the fact.
My major concern now would be backwards compatibility: if you do set icon options using the old way, will that still work? From just looking at the code, it looks like it wouldn't, and perhaps that's something we should address first.

Hi @perliedman,
Thank you for your feedback!
As for backwards compatibility with setting default icon options, I think it should work, even though there is an extra condition. I have added a test spec for that.
Given the fact that with this PR, options are applied per instance of L.Icon.Default instead of being always retrieved from L.Icon.Default.prototype, these app specific options must be applied to L.Icon.Default.prototype before the first default icon tries to retrieve options from CSS; i.e. typically before any marker is added to a map.
However I think this is how apps should work already (i.e. setting default option before adding markers to map), otherwise the result would be strange: any marker already added to map would use the previous options, and any new marker, or those removed then re-added, would use the new options.
That being said, I think it should be possible to refactor this PR so that options are no longer applied per instance, but applied on L.Icon.Default.prototype like the current situation. But like the above example case shows, I think having per instance options is more logical.
Please let me know your opinion on this.
I still need to refactor to use background-image first for IE, then fallback on cursor for Firefox high contrast setting.

@ghybs thanks, that sounds very reasonable!

Refactored this PR to use background-image first (to avoid IE bug regarding cursor URL being relative to HTML page instead of relative CSS file) and fallback to cursor (for Firefox high contrast option).
Added a test spec for this fallback mechanism.

Unfortunately I think I still missed something, somehow implied by @perliedman's previous comment.
Current situation:

Common images folder retrieved from CSS. Recorded in options.imagePath, unless already provided.
Icon URL option in JS class provides file name (but it could also include some path).
_getIconUrl concatenates imagePath and icon URL option.

New situation: (i.e. with current PR)

Individual icon path + file name retrieved from CSS. Each could be in arbitrary folder / URL. Recorded in icon URL option, unless it is already provided.
options.imagePath used to refactor the above retrieved URL (currently just in case icon URL option is not provided), replacing the entire path except for the file name.
_getIconUrl just outputs icon URL option as recorded (and possibly refactored).

Therefore the behaviour changes in case both imagePath and non-default icon URL options are provided:

Currently: _getIconUrl concatenates imagePath and icon URL option.
With this PR in current state: _getIconUrl just outputs icon URL option.

While the fix may look simple, actually it becomes complicated when we try to keep the possibility to specify a path in icon URL option as well (therefore we have to concatenate imagePath, we cannot replace the path), and we take into account the fact that each icon URL in CSS may have a completely different URL (therefore there may be no common folder to record into imagePath).
Maybe some intermediate solution could be to still retrieve full URL from CSS if no imagePath is provided, and in the case it is provided, strip the path before the file name so that it can be replaced by imagePath, and _getIconUrl works as currently in that case?

Ok should be fixed now:

If imagePath option is specified, IconDefault emulates how it currently works: only the file name is retrieved from CSS and recorded in iconUrl option (same for shadow and Retina). Then _getIconUrl appends imagePath and iconUrl, so that it also works when the latter is provided by the application (and it can even contain a partial path).

If no imagePath option is provided, the full icon URL is retrieved from CSS and recorded in options, so that they can have any arbitrary URL.

Added 2 new dedicated tests.


add margin-top Footer and remove redundancy background-color PageHeader

This is a minor issue, may I get to assign PR with this update ?
backgroundImage was doing the same job as backgroundColor, with gradient linear dark grey
result remove backgroundColor
footer and content in faq maybe need some space between them
result add mt=24px
correct me if I wrong
Thanks in advance

Hello @nanto88 👋
Thank you for your issue, so at the moment, i think for the backgroundImage and backgroundColor it's still applicable but for the footer and content in faq page, it was resolved apparently 😅

Okay i don't need to update then , thank you for this response 😄


implemented a startup page setting

This change implements a startup page setup and loading functionality which will allow users to set a startup page and have it loading automatically at app start.

Thanks for this PR! Curious to learn more about the use case for this: kiosk-style app?


publish<EMAIL_ADDRESS>

Requested by: @billyvg
Merge target: chore-replay-upgrade-rrweb-2.28.0
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).
Targets

[ ] npm[@sentry/types]
[ ] npm[@sentry/utils]
[ ] npm[@sentry/core]
[ ] npm[@sentry-internal/browser-utils]
[ ] npm[@sentry-internal/replay]
[ ] npm[@sentry/opentelemetry]
[ ] npm[@sentry-internal/feedback]
[ ] npm[@sentry-internal/replay-canvas]
[ ] npm[@sentry/browser]
[ ] npm[@sentry/node]
[ ] npm[@sentry/profiling-node]
[ ] npm[@sentry/angular]
[ ] npm[@sentry/ember]
[ ] npm[@sentry/react]
[ ] npm[@sentry/solid]
[ ] npm[@sentry/svelte]
[ ] npm[@sentry/vue]
[ ] npm[@sentry/wasm]
[ ] npm[@sentry/vercel-edge]
[ ] npm[@sentry/cloudflare]
[ ] npm[@sentry/deno]
[ ] commit-on-git-repository[getsentry/deno]
[ ] npm[@sentry/aws-serverless]
[ ] npm[@sentry/google-cloud-serverless]
[ ] npm[@sentry/bun]
[ ] npm[@sentry/nestjs]
[ ] npm[@sentry/nextjs]
[ ] npm[@sentry/nuxt]
[ ] npm[@sentry/remix]
[ ] npm[@sentry/solidstart]
[ ] npm[@sentry/sveltekit]
[ ] npm[@sentry/gatsby]
[ ] npm[@sentry/astro]
[ ] npm[@sentry-internal/typescript]
[ ] npm[@sentry-internal/eslint-plugin-sdk]
[ ] npm[@sentry-internal/eslint-config-sdk]
[ ] aws-lambda-layer
[ ] gcs[browser-cdn-bundles]
[ ] github
[ ] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

This keeps failing: https://github.com/getsentry/sentry-javascript/actions/runs/10961298461/job/30447536668#step:13:74

#retract


Allow access to writer and provider (Part II ) #176.

Description
Added the third getter by tag and level for writers from the TinyLogLoggingProvider.
Linked issue: #176
Definition of Done

[x] There are no TODOs left in the code
[x] Code style follows the tinylog standard
[x] All classes and methods have Javadoc
[x] Changes are covered by JUnit tests including corner cases, errors, and exception handling
[x] Maven build works including compiling, tests, and checks (mvn verify)
[x] Changes are committed by a verified email address that is assigned to the GitHub account (https://github.com/settings/emails)

Documentation
Additions or amendments for the public documentation:
No documentation change necessary.

Agreements

[x] I agree that my changes will be published under the terms of the Apache License 2.0
[x] I agree that my GitHub user name will be published in the release notes

You can add this getter for level and tag for the logging provider if you like ( #176). It is not very high priority but gives a bit more flexibility. If you don't want it, just reject the pull request #178 and close the issue #176.

@Git5000 Thank you for your pull request! I'm happy to merge :)


Add to https://caddyserver.com/download

Not sure what configuration is required to do this, but when I ran caddy add-package github.com/greenpau/caddy-systemd I got github.com/greenpau/caddy-systemd is not a registered Caddy module package path. Other modules written by you (caddy-security and caddy-git) seem to be listed on that page.

@ajusa , this module is not yet ready for prime time.


Table updates

Update tables to leverage new webcharts v1.9 functionality

@samussiah Should be fixed now.


Devmariam/feat/2865 new templates page [Part1]

Changes to PR for corresponding Issue
Functionality

[x] Initial Structure
[x] Final Structure

UI

[x] Title Bar
[x] Descriptio Bar
[x] Search Bar
[x] Selection box/bar component styled from scratch
[x] RIght sidebar
[x] Steps
[x] Form
[x] RWD

How to test it
Screenshots
20.01.2023

Checklist

[x] tested locally
[ ] added new dependencies

Discussed with @julioramosest today.
Everything is fine and ready to merge from UI point of view @devcatalin .


Generate new README

Checklist

[x] I've read and accepted the Contribution Guidelines
Requires PATCH version update

Context

We are introducing a new README format for steps.

Changes

Added generate-readme workflow with contrib_section (see: docs/contribution.md).
Generated new README.

We have to wait with restarting the build because we reached our upload limit to iTunes Connect 🙃


Temporary file /tmp/parsetab.py

I think when parsing qiskit writes a temporary file /tmp/parsetab.py . This poses a problem if two users use qiskit on the same machine. The temporary file should be unique to each user.
Expected Behavior
Temporary file should be unique for each user.
Current Behavior
Temporary file is not unique for each user which leads to write conflicts.
Possible Solution
Add user name to the parsetab file.
Steps to Reproduce (for bugs)
ls -l /tmp/parsetab.py

Yep, you are right!
Thanks for reporting! ... will be fixed ASAP.

Closed via #139  - as usual, thanks for reporting!

@nikomoll We have issued a new release: 0.3.9 which fixes this issue as well. You can update it via pip  (if you are using pip): pip install --upgrade qiskit


Why In SugiyamaAlgorithm nodes are aligned with top when tree depth is unbalanced

Why Node 1 is not in the same level of node 2. But when i add 3 more on top of node 1 then it will comes on the same level of Node 2.
Please help to figure this out...

i will take a look at this, can you help export this data so that i can test

i will take a look at this, can you help export this data so that i can test

I had change the file "SugiyamaEdgeRenderer.dart" as per my requirement

i will take a look at this, can you help export this data so that i can test

@nabil6391  did you get time to take a look on the issue

Try out 1.2.0 , added a new CoordinateAssignment to allow users to adjust alignment


enum doesn't work in set when sent from other isolate

If I do some computation in another isolate using compute, and return an Enum value, then try to compare this using Set, it doesn't work.
import 'package:flutter/foundation.dart' show compute;

enum X {
  a
}

foo() async {
  final s1 = Set()..add(X.a);
  final xa = await compute(bar, '');
  print(s1.contains(X.a));
  print(s1.contains(xa));
  // expected:
  //   true
  //   true
  // actual:
  //   true
  //   false
}

bar(arg) {
  return X.a;
}

$ flutter doctor -v
[✓] Flutter (Channel master, v1.1.5-pre.42, on Mac OS X 10.13.6 17G4015, locale en-GB)
    • Flutter version 1.1.5-pre.42 at /Users/steve/code/flutter
    • Framework revision 4b87e334d7 (20 hours ago), 2019-01-02 19:49:59 +0100
    • Engine revision 844c3274e6
    • Dart version 2.1.1 (build 2.1.1-dev.0.1 88e6fe0f67)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/steve/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.3.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] VS Code (version 1.30.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.21.1

[✓] Connected device (1 available)
    • iPhone XR • 1EE1625B-61BB-421E-B104-CB3C8A5B6076 • ios • iOS 12.1 (simulator)

• No issues found!

That looks like a Dart language issue that should be reported in https://github.com/dart-lang/sdk/issues
I'm going to close. Feel free to reopen if you think this is Flutter-specific.

@zoechi I'd like to re-open this. The following using the Dart APIs works fine. However, the version I initially posted using Flutter's compute fails. So, I think this is a Flutter issue.
I don't think I have rights to re-open the issue, though.
import 'dart:isolate';

enum X {
  a
}

foo() async {
  final r = ReceivePort();
  final s1 = Set()..add(X.a);
  final isolate = await Isolate.spawn(bar, r.sendPort);
  final xa = await r.first;
  print(s1.contains(X.a));
  print(s1.contains(xa));
  // expected:
  //   true
  //   true
  // actual:
  //   true
  //   true
}

bar(SendPort sendport) {
  sendport.send(X.a);
}

main() async {
  foo();
}

Sounds weird.
What is the Dart version you are using when you try with Dart directly?

$ dart --version
Dart VM version: 2.1.1-dev.0.1.flutter-88e6fe0f67 (Thu Dec 27 21:02:52 2018 +0000) on "macos_x64"

I think Flutter uses its own ReceivePort implementations — not sure where they live though.

compute implementation is quite simple, this is why I was confident that this is a Dart issue.

That makes sense. I don't think it's an issue in what compute does exactly, but around where the serialization happens.
Do you have a pointer to where objects are deserialized for Flutter? I spent 10 mins trying to figure it out, but couldn't find where Flutter does it.

I'm still pretty confident Flutter does not do anything here.
compute() is just a convenience method for one-off calculations on another isolate.
You can use isolates the same as you do in plain Dart if you need more control.

@a-siva Is it a bug in Dart that enum values do not keep their identity when send across isolates?

If the isolates were spawned using the Isolate.spawn() API then it should be possible to pass enum values across these isolates but if the Isolate.spawnUri() API then enum values cannot be passed across these isolates.

It seems to work in his simple Dart program above which sends X.a across isolates, we probably need to investigate this.

Let me have somebody from the VM team investigate this.


Eyetracking pupil diameter

Describe the problem
Will MRTK3 provide pupil diameter in the eyetracking package?
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A series of amazing functions would be enabled if we could access the pupil diameter and center coordinate of eyeballs.
A clear and concise description of what you want to happen.
Describe alternatives you've considered
We've been trying alternatives but your HoloLens2 is pretty tight with its visor space, add-on eye trackers wouldn't work smoothly. And the built-in eye tracker is taking the perfect spot. We really wish there is an alternative.
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi @hunter12580, for MRTK3 issues please file an issue in the new repository. Thanks!


Support for xDS protocol

We need the support for xDS protocol so that we can do something similar (or better).
https://cloud.google.com/blog/products/networking/traffic-director-supports-proxyless-grpc
More on xDS protocol can be found here.
https://www.envoyproxy.io/docs/envoy/latest/api-docs/xds_protocol

A good log on xDS
https://blog.envoyproxy.io/the-universal-data-plane-api-d15cec7a

The tasks we have to do are as follows:

Prepare the proposal
Implementation
Testing with a locally implemented control plane and GCP(Trafic director)
Testing with Istio


Rust functions allows for minimal required parameters from pool state

Currently, most rust functions requires the pool state object as a parameter, regardless of which fields the calculate functions require. While this is useful when e.g., we get the pool state from the chain before a rust call, it makes it difficult to manually build a correct pool state from scratch for a calculate function. Ideally, rust functions allow only taking parameters that each rust function actually needs, along with an option to pass in the pool state in its entirety.

FYI this was something @jrhea was also asking for, we scoped it a bit and it will take a sizable refactor -- the SDK is designed from the ground up with functions implemented on state.
This is also discussed in this agent0 issue: https://github.com/delvtech/agent0/issues/1355

We can do this one-by-one for the functions that are the highest priority. I haven’t run into a scenario where I would actually use this option, so it’s not clear to me which functions we’d want to prioritize.
Actually doing the refactor will be pretty easy. We can just leave the implementations where they are in the repo and move them out of the corresponding “impl” block. Basically, we’ll be left with a number of procedural functions.
I would like us to preserve the easy of calling the functions on the State object, but it’ll be easy to do this by adding a function that passed the appropriate arguments into the corresponding procedure.

The Python and WASM wrapped rust SDK removes the need to include a state by instantiating it for every call from a pool info & pool config.
Python example here: https://github.com/delvtech/hyperdrive-bindings/blob/main/crates/hyperdrivepy/python/hyperdrivepy/hyperdrive_state.py
WASM example:
https://github.com/delvtech/hyperdrive-bindings/blob/main/crates/hyperdrive-wasm/src/lib.rs

@dpaiton FWIW @mcclurejt is already working on the refactor. This seems like a nice DevEx improvement though.

The other thing to consider is that this request somewhat contradicts the implementation for https://github.com/delvtech/hyperdrive/issues/876
Given that I think we should build public stand-alone functions for key operators (max long/short; open trades; close trades; spot price; spot rate; etc) that use specific fn args and then replace the impl versions to call the stand-alone functions and derive the arguments from State.


2024 solver participant submission: OpenSMT

We are really sorry, but we made an error in the last iteration of the rules. The requirement that the submission is a Dockerfile was an idea that we forgot to remove. The submitted file must be an archive that contains the executable precompiled (statically linked is preferable). It will be executed on a computer that has the same installation as the docker image: registry.gitlab.com/sosy-lab/benchmarking/competition-scripts/user:latest
For more details on the image see https://gitlab.com/sosy-lab/benchmarking/competition-scripts/
#computing-environment-on-competition-machines.
PS: Not sure if you are on one of the following mailing lists. If not, we recommend you to join at least the smt-announce group. We sometimes post updates on the competition there including the above information.
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

We agree and we will write an email in the next days to all cloud/parallel participants about this. We haven't yet fully decided how this will look like, but it will be sufficient by the final solver deadline.

The goal is to send out the mail how this works today/tomorrow. (We still have to update the submission format so this is possible.)

@Tomaqa We have executed the latest version of OpenSMT on a randomly chosen subset of 20 single-query benchmarks from each logic where it participates. The benchmarks are also scrambled by the competition scrambler (with seed 1). You can find the results here: https://www.fi.muni.cz/~xjonas/smtcomp/opensmt.table.html#/table
Quick explanation:

Green status means that the result agrees with the (set-info :status _) annotation from the benchmark.
Blue status means that the benchmark has annotation (set-info :status unknown).
By clicking on the result (e.g. false, true, ABORTED, …) you can see the command-line arguments with which your solver was called and its output on the benchmark.
By clicking on the benchmark name (i.e., *scrambled*.yml), you can see the details of the benchmark including its contents (by clicking on the file link in input_files) and the name of the original bennchmark before scrambling (e.g., # original_files: 'non-incremental/AUFBVFP/20210301-Alive2-partial-undef/ph7/583_ph7.smt2').

Please check whether there are some discrepancies, such as missing/extra logics, unexpected aborts or unknowns, and similar. If you update the solver, let me know and I can execute further test runs. We still have plenty of time for several follow-up test runs.

We will soon run some tests for unsat-core and model-validation. However, these tests will only check whether your solvers will execute on these benchmarks without errors. In particular, they won't check the soundness and or formatting of the returned cores/models. If it helps, today we will release pre-build versions of the pre-processor, post-processor, etc. that should run in the docker image we provide you for testing.

@Tomaqa Here are also results of the test run on a small subset of incremental benchmarks: https://www.fi.muni.cz/~xjonas/smtcomp/opensmt_inc.table.html#/table
The status column has the following meaning:

DONE (X correct): the solver terminated successfully and correctly decided X (check-sat) queries before that.
SOMETHING_ELSE (X correct) (e.g., TIMEOUT (10 correct) or ABORTED (10 correct)): The solver did not terminate successfully, but correctly decided X (check-sat) queries before it crashed or was killed.
WRONG: There was at least one answer to (check-sat) that did not agree with the status specified in the benchmark.

Again, if there are discrepancies (missing/extra logics, unexpected results), please let me know.

@Tomaqa We have finished test runs of model-validation and unsat-core generation tracks. You can find the results here:

https://www.fi.muni.cz/~xjonas/smtcomp/opensmt_model.table.html#/table
https://www.fi.muni.cz/~xjonas/smtcomp/opensmt_unsatcore.table.html#/table

As before, you can click on the status of the benchmark to see the output of your solver. If you find any discrepancies or extra/missing logics, please let me know.
Note that we selected only SAT benchmarks for model validation and only UNSAT for unsat-core generation. As a result of that, some logics do not contain any benchmarks. So do not be surprised if you have subscribed to one of these logics and you do not have any result for it. In particular, the logics are:
For model-generation: QF_UFFP, QF_UFBVDT, QF_UFDTNIA, QF_NIRA.
For unsat-core: UFBVDT, FPLRA, QF_AUFBVFP, QF_ABVFPLRA, QF_UFBVDT, QF_UFDTNIA, QF_UFDT, QF_NIA, QF_SNIA.


List Widget Variable Types: Add Type as Heading when type is expanded

Is your feature request related to a problem? Please describe.

I make use of the variable lists feature to enable block-based content editing for my clients. A major issue for me is that the CMS does not show the editor the type of the widgets. Using a summary is only a partial solution, because it gets hidden if the object is expanded.
Describe the solution you'd like

I also experimented with custom components for the markdown widget. Here, the type/name of the component is clearly stated in the top left corner of the editing interface. That way, the editor always knows what type of content he is currently authoring.
Describe alternatives you've considered

I considered using markdown files with custom widgets, but this combination has some issues when combined with Gatsby. For instance, I need a gallery widget having multiple images. If I use a markdown component, Gatsby does not automatically link the respective file and requires manually adding and maintaining an "assets" field.
Additional context

Custom Markdown component with the type (formular) visible in the top left cornder:
Variable Types with no type information when expanded:

If anyone would like to pick this up the place to start is here:
https://github.com/netlify/netlify-cms/blob/879e03b6f16feb23b49ba4248f2ad1eb3da4d259/packages/netlify-cms-widget-list/src/ListControl.js#L501

@erezrokah I'd like to give this a try

Thank you all for resolving this issue in such a short amount of time!


No List Style?

Couldn't find a listStyle() function

no I don't think we have that yet, but we are open to PRs :)


Use USB Remote who is already Paired by CC41 Control Unit

Hello Rainer,
is it possible to use an already paired Centronic Stick with this integration?
Cheers

To be honest I don't know.
The stick will work. But I'm not sure if you will need to pair your shutters again with this integration.
To my knowledge the stick does not store anything. Any data is stored either in the CC41 or within Home Assistant. So why not give it a try?
You should be able to switch between CC41 and Home Assistant without any impact. You just might to pres the Stop button for each unit several times after changing between CC41 and Home Assistant in order to sync with the shutters.

Hey Rainer, i managed to pair 9 Covers flawlessly and bridge them to Homekit. Thank you for your integration! I use the Traveltimes to controll the covers and the next step is to add the existing walltransmitter to track the positions :)

Hey Rainer, i managed to pair 9 Covers flawlessly and bridge them to Homekit. Thank you for your integration! I use the Traveltimes to controll the covers and the next step is to add the existing walltransmitter to track the positions :)


Try to fix wrong resource path

Because resource path from Pods-resources.sh contains ${PODS_ROOT},
the calculated pathname is wrong vendor/Pods/${PODS_ROOT}/podname/classes/...,
which will cause copy resource files to fail.
Ref: https://github.com/amirrajan/rubymotion-applied/issues/47

/cc @MarkVillacampa

Thanks!


Release 1.1.0

First Major Update, Target PR for all sub tasks

Released the current version to appstore, because of some issues with the ollama summmary generation.


Account for situations in Wagtail 6+ where chooserUrls are not in the window object

Without this fix, you'll get an error in the console: Uncaught TypeError: window.chooserUrls is undefined
This probably isn't the ideal solution, but it seems to work and shouldn't disrupt prior versions. The 6.1 release notes document the kind of changes that should be made long-term.

Seconding this—not an ideal solution, but it fixes the Wagtail >=6 admin for now without dropping support for Wagtail 5.2.


Investigation of using multiple domain names for grafana, and deployer referencing one specific

The deployer script reads grafana.ingress.tls[0].hosts[0], selecting out hosts[0] specifically as a host to return for a cluster. If a redirect is setup, maybe there won't be issues, maybe there will be - I'm wasn't sure so I investigated.
Conclusion
I think we should be fine. urllib.request.urlopen follows redirects, and so does requests.get it seems. As long as GET and POST are similar, we'll be fine.
Action points

[ ] Perhaps update docstring of get_grafana_url to mention all existing usage of the returnvalue makes it acceptable to pick one single.

Investigation
get_grafana_url hardcodes to hosts[0]
https://github.com/2i2c-org/infrastructure/blob/d4f0fdc15d85ebae2f71c02929b678d05e0b09bd/deployer/grafana/grafana_utils.py#L12-L30
Reference sof get_grafana_url
Grafana API requests via deploy_grafana_dashboards
This function clones the https://github.com/jupyterhub/grafana-dashboards repo and passes the grafana URL to the ./deploy.py script in that project
https://github.com/2i2c-org/infrastructure/blob/d4f0fdc15d85ebae2f71c02929b678d05e0b09bd/deployer/deployer.py#L96
When ./deploy.py from that project is used, it sends GET and POST requests using this function. I think they may very well redirect correctly and we should be fine. But I don't know for sure. Some tools like curl require you to for example explicitly pass a flag to say that redirects should be followed.
It seems the following is used: from urllib.request import urlopen, Request and testing against https://google.com I note that it does follow the 301 redirect for a GET request at least.
Grafana API requests via new_grafana_token and update_central_grafana_datasources
https://github.com/2i2c-org/infrastructure/blob/d4f0fdc15d85ebae2f71c02929b678d05e0b09bd/deployer/grafana/grafana_tokens.py#L160
https://github.com/2i2c-org/infrastructure/blob/d4f0fdc15d85ebae2f71c02929b678d05e0b09bd/deployer/grafana/central_grafana.py#L101
Seems to use import requests and that seem to follow redirects at least for GET requests.
Related

QCL hub configuration with two hosts

I don't think we had issues with this in the end, closing.


New FtpSslSessionReuseException

@FanDjango what is your thought on absorbing errors like this and instead throwing a special exception with the exact advise you give out on the forums?
SSL connection failed; session reuse required -> FtpSslSessionReuseException which then has the message "You need to 1) upgrade your FTP server to the latest or 2) disable SSL session reuse in your FTP server or 3) use FluentFTP.GnuTLS which supports SSL session reuse"
It should reduce the workload of constantly answering the same damn thing when it comes up as a bug ticket.
What say? Is it possible? Is it viable?

Hi Robin,
excellent idea.
I sometimes prefer one or two lines of "ERROR" messages prior to a normal exception.
Also: Not every failure to authenticate the 2nd, 3rd, 4th, etc. connection (i.e. the data connection) is due to session resume problems, although we currently seem to be getting only these errors.
But by all means…
mit freundlichen Grüßen,
Michael Stiemke @.***)
AnSyNova GmbH
Ringstr. 13
85405 Nandlstadt
+49 8756 969 49 79
<http://www.ansynova.com http://www.ansynova.com/ >
From: Robin Rodricks @.***
Sent: Wednesday, March 1, 2023 5:23 AM
To: robinrodricks/FluentFTP
Cc: FanDjango; Mention
Subject: [robinrodricks/FluentFTP] SSL connection failed; session reuse required (Issue #1224)
@FanDjango https://github.com/FanDjango  what is your thought on absorbing errors like this and instead throwing a special exception with the exact advise you give out on the forums?
SSL connection failed; session reuse required -> FtpSslSessionReuseException which then has the message "You need to 1) upgrade your FTP server to the latest or 2) disable SSL session reuse or 3) use GnuTLS to fix this"
What say? Is it possible? Is it viable?
—
Reply to this email directly, view it on GitHub https://github.com/robinrodricks/FluentFTP/issues/1224 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMFOTW4MHNFXOTYQWIKIHZLWZ3FKPANCNFSM6AAAAAAVLROSAQ .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AMFOTWZPYDF34K2X25MUWWTWZ3FKPA5CNFSM6AAAAAAVLROSASWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHF7HSNPA.gif Message ID: @.***>

excellent idea.
I sometimes prefer one or two lines of "ERROR" messages prior to a normal exception.
Also: Not every failure to authenticate the 2nd, 3rd, 4th, etc. connection (i.e. the data connection) is due to session resume problems, although we currently seem to be getting only these errors.
But by all means…

if (connection == data_connection)...
 {message with advice
 throw (possible resume failure exception)
}

Some more thoughts on this:
Quite a few cases exist where the SSL Authentication itself does not fail, but shortly after, there is a "4xx" message saying that it was not possible to open a data connection.
"Operation not permitted" is one such message, another actually refers to "session resume". There is a whole catalog of such messages, depending on the server type.

@4nd3r5 You might want to read through this issue
https://github.com/robinrodricks/FluentFTP/issues/948#issuecomment-1289708124

I have move this to the Wiki: https://github.com/robinrodricks/FluentFTP/wiki/Future-Plans


multi: bump all dependencies to latest versions

There's a panic in the itest, will look at it tomorrow.
Putting it up as a placeholder.

Replaced by https://github.com/lightninglabs/lightning-terminal/pull/348.


Feature/advanced search filters dropdown

Description
Finished styling of search and filters menu. Added handler functions for clicking outside the filter box to close and hitting enter in the address search
Type of change

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

Change Status

[x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[x] Visual and user testing of elements
[x] Existing render tests

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

LGTM


Is it possible to use this plugin without sudo

I was looking into whether it would be possible to use this plugin without sudo packer build. My biggest problem with the current approach is that the output files and packer_cache directories end up being owned by root.
I was looking into whether it would be possible to prefix commands within the plugin with sudo so that sudo was used only when necessary but this doesn't work for os.OpenFile in pkg/builder/step_register_bin_fmt.go. Maybe there is a better way though?

yea write to bin_fmt requires root; if there a shell command we can use to do this? this way we can prefix it with sudo.
also, how does this work for you - does sudo not need a password?

Actually I've started looking into a new approach, using systemd-nspawn instead of chroot. This would not only remove the need for sudo with the chroot command but also remove a lot of mount commands (as systemd-nspawn can handle this as well). I'm not sure if you'd be open to using systemd-nspawn so I might fork this plugin instead.
Regarding step_register_bin_fmt, does this need to be done every time Packer runs? On my system, the qemu-user-static package has a post-install script that sets up binfmt:
case "$DPKG_MAINTSCRIPT_NAME:$1" in

    postinst:configure)
	for fmt in $fmts ; do

	    eval "case $fmt in $omit) magic= ;; *) magic=\"\$${fmt}_magic\" mask=\"\$${fmt}_mask\" ;; esac"
	    if [ -n "$magic" ]; then
		update-binfmts --package qemu-user-static --install qemu-$fmt /usr/bin/qemu-$fmt-static \
			--magic "$magic" --mask "$mask" --offset 0 --credential yes --fix-binary yes
	    else
		remove_binfmt $fmt
	    fi
	done
	;;

    prerm:remove)
	for fmt in $fmts ; do
	    remove_binfmt $fmt
	done
	;;

esac

i'm open to add this as an option (even one that's enabled by default, and assuming it works for everyone deprecate the old way). as for registration, it doesn't need to be done every run, i mainly do that for simplicity for the user; we can add a test if the required value is there, and only write it if its not there

Thanks for the feedback. I might work on this if I have some spare time :)


chore: bump package.json files in main so...

What does this PR do?
chore: bump package.json files in main so they're x.(y+1).0 and therefore greater than those in 1.1.x branch (RHIDP-1720)
This will allow us to do semantic-release releases from the 1.yy.x branches as well as from main, making it more clear which plugins are for RHDH 1.1.z vs. vrom 1.2, and allowing plugin vendors to have maintenance releases too. (eg., Orchestrator and Notifications have asked for this)
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
https://issues.redhat.com//browse/RHIDP-1720
How to test this PR?
This PR should trigger releases of all the plugins to npmjs.org via the semantic release process so we have versions tha are greater than those in the 1.1.x branch.
In future this change would be done by script:  https://gitlab.cee.redhat.com/rhidp/rhdh/-/commit/a88232220e9fe2b10a0858aab20f3ceee6cbb92c
PR Checklist
As the author of this Pull Request I made sure that:

[x] The Eclipse Contributor Agreement is valid
[x] Code produced is complete
[x] Code builds without errors
[ ] Tests are covering the bugfix
[x] The repository devfile is up to date and works, if one exists
[x] Sections What issues does this PR fix or reference and How to test this PR completed
[x] Relevant user documentation updated
[x] Relevant contributing documentation updated
[x] CI/CD changes implemented, documented and communicated

Reviewers
Reviewers, please comment how you tested the PR when approving it.

Thanks @nickboldt,
I liked the approach with this change fixes can cherry picked to 1.1.x and it can do a release independently from the respective branch.
For this PR I have two doubts

The job is failing https://app.snyk.io/org/janus-idp/pr-checks/804dd0e0-5485-474a-9782-de0208217d89 and couldn't see the failures but this can be due to version not found as we are bumping and in npm that version might not exist. So this PR should be the exception and get manual merge and post merge do we expect the release to trigger for this version bumped with this PR otherwise all open PR would see these failures
With the current setup release doesn't happen from 1.1.x if am not wrong so that needs to be supported as well

cc @gashcrumb @kadel

@schultzp2020 I've reverted the bumps in packages/ to keep them at 0.0.0 if they're currently 0.0.0;
@invincibleJai Here's the PR for enabling 1.yy.x semantic releases https://github.com/janus-idp/backstage-plugins/pull/1401 which we can merge after this one.


method_context_id not matching after binding trait values and separating once again

Describe the bug
Found in test-trait_bind_sep.R
To Reproduce
Steps to reproduce the behavior:
#Extract a dataset
dataset_id <- c("Falster_2005_2")
subset <- extract_dataset(austraits_5.0.0_lite, dataset_id = dataset_id)
bounded <- bind_trait_values(subset$traits)
seperated <-separate_trait_values(data = bounded, austraits_5.0.0_lite$definitions)
Expected behavior
All rows expect for value_type should be the same
subset$traits %>%
select(-value_type) %>%
arrange(dataset_id, observation_id, trait_name, value) |>
select(method_context_id)
seperated %>%
select(-value_type) %>%
arrange(dataset_id, observation_id, trait_name, value) |>
select(method_context_id)

@fontikar The problem is that each of the 4 rows of data that are being bound together by the function bind_trait_values has a different method_context_id (as in, each was collected using a somewhat different method). The function is simply taking the first method_context_id. There are three other grouping variables that should be included in the function, method_id, method_context_id, and repeat_measurements_id.
I'm also puzzled why dataset_id isn't a grouping variable. Because if the function is used with a subset of data that includes multiple datasets there will be duplicate observation_id values that belong to the two datasets.
I suspect this function was written for AusTraits 3.0 and has never been revamped! (I've never used it.)

Oh my goodness that is such a helpful comment @ehwenk I have been staring this for so long and can't figure out where the problem is - definitely lacking on the expertise on what columns to use for splitting.
I don't understand why the tests used to past and now they don't....
Are you brave enough to tackle this since you have more domain knowledge on the dataset structure? I was about to bug @dfalster!

I think I can fix it very quickly... so yes, let me try.

Branch from switch-deprecate and then I can merge your solution back in

Just push!


Need to support authentication options supported by Nancy

I haven't even looked at what is possible.  This will likely flush out into a few issues.

+1 for authentication. Would be useful for PowerShell commands to run in the context of the user/service account that made the web request. E.g. kick off some password reset process as the user

I'd love to have the ability for a user to authenticate using oAuth2 protocol (I mean using his FB, Google or Twitter accounts).


about the file---"structure_files/chembl.smiles"

I am not majoring in biology or medicine, and I don't quite understand how to find related smiles on ChemBL website. May I ask if you could provide "structure_files/chembl.smiles"? Or give some advice on how to download smiles by myself. Thank you very much!!!!
:-)

Here is a way to download data(including SMILES) from ChEMBL:

click search with an empty query

Filter with some conditions, such as

only compounds(no targets, assays, documents)
only select 'Homo sapiens' in 'Organism'

and download data in a csv format. A column named 'Canonical SMILES' is just what you want.


added avatar class components and updated the docs for avatar

Added avatar and badge component to the docs and the UI library

Hey, host your development branch on netlify, it would be easy that way for us to review. Or, put some screenshots of your page.

I have hosted the dev branch and this branch is made with dev as base .Still it's not showing the preview . Any idea ?


CI 32-bit and OPAM

Add CI workflows to test:

32-bit on Linux; this thus builds upon #297;
OPAM packages.

I've now merged this. On the main CI run all Windows workflows failed during Cygwin install:
  [...]
  connection error: 12037 fetching https://cygwin.mirror.constant.com/x86_64/setup.ini
  mbox note: Unable to get setup from <https://cygwin.mirror.constant.com/>
  unattended_mode is set at mbox: returning default value
  download/verify error in unattended_mode: out of retries
  note: Installation incomplete.  Check D:\cygwin\var\log\setup.log.full for details
  Ending cygwin install
  Error: The process 'C:\hostedtoolcache\windows\cygwin\3.4.3\x86_64\setup-x86_64.exe' failed with exit code 1

I suspect that this is just due to a temporary network failure.


Detect and fail tests when timers or other handles are not cleaned up.

We've noticed a few tests stay running for a significant amount of time after all the actual tests have completed.  This is a sign of timers or other handles (file streams, sockets, etc.) outliving our test cleanup.  Since this is technically unexpected behavior in our tests, I believe we should detect these handle leaks and fail the tests when they occur.
Basically, if a test process has not terminated 5 seconds after it finishes reporting all results, the rush cover and rush test will now fail.
To help identify the cause of these hangs, I've also setup all of our tests to use the very handy wtfnode package, which can identify open handles as well as (crucially) the function/line that created them.  However, because wtfnode adds extensive instrumentation to capture this information, it will only run/report results when tests are run with the DEBUG_LEAKS environment variable set.

@bsteinbk mind taking a look at this?


update recipe to 0.4.7

Upstream project
Parent Jira ticket (scikit-criteria)

Linter check found the following problems:
The following problems have been found:
WARNING: clone/recipe/meta.yaml:15: avoid_noarch: noarch: python packages should be avoided
WARNING: clone/recipe/meta.yaml:41: documentation_specifies_language: Use the generic link, not a language specific one
WARNING: clone/recipe/meta.yaml:42: missing_description: The recipe is missing a description
Warnings were found


Defender is flagging VSCode

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:
1.Download from https://github.com/microsoft/vscode-winsta11er/releases/download/v0.0.4/vscode-winsta11er-x64.exe
2. Observe that Smartscreen (browser) and Windows Defender instantly flag the above binary.
3.

This is resolved with 0.0.5


DataTables warning ; table id=Biens_list Ajax error

And at the end of the error message, I have : ' See : http://datatables.net/tn/7
According the console in google chrome developper tool, I have :
'GET http://crudadmin.local/biens/list?draw=2&co ...(very long string) ... 414 (Request-URI Too Long)'  ....  jquery.min.js:6
So I  reconfigure apache with a larger value for LimitRequestLine without success.
If I reduce the field number in the table, I have no error.
So How I can "increase" this field number in a table
Any help appreciated. Thanks

more than changing the apache config, I think athe AJAX request should have done with POST and not GET
Could you give us a bit more information ?
like on what page it happened, with what kind of table &data...

I have the same error when I click list
DataTables warning ; table id=Table01_list Ajax error

I've had the same error. Here's what I've done to make it work:
in web/controllers/table_name/index.php
change:
$vars = $request->query->all();
to:
$vars = $request->request->all();
and in:
web/views/table_name/list.html.twig
change:
ajax: "table_name/list",
to:
ajax: {
	"url": "table_name/list",
	"type": "POST"
},

This could be done during code generation. I don't know why nobody fixed this. If I have time, I'll make the changes and add pull request for this.

Thanks
Regards,
Keme Kenneth
Web Developer   |   twitter.com/zeelz
[image: Keme Kenneth on about.me]
http://twitter.com/zeelz
On Fri, Jul 29, 2016 at 3:39 AM, Vasilj Milošević<EMAIL_ADDRESS>wrote:

I've had the same error. Here's what I've done to make it work:
in web/controllers/table_name/index.php
change:
$vars = $request->query->all();
to:
$vars = $request->request->all();
and in:
web/views/table_name/list.html.twig
change:
ajax: "table_name/list",
to:
ajax: {
"url": "table_name/list",
"type": "POST"
},
This could be done during code generation. I don't know why nobody fixed
this. If I have time, I'll make the changes and add pull request for this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jonseg/crud-admin-generator/issues/56#issuecomment-236080459,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFWeICViYOm6PmSqjfmQDwG9eyojgbjLks5qaWfQgaJpZM4GH6pT
.

I made the change mentioned above and I still have the DataTables warning - Ajax Error
while TABLENAME/list works with the following output : {"start":0,"recordsTotal":7,"recordsFiltered":7,"data":[]}

This is a nice project but I think the developer has stopped working on it. Please try this new project: https://github.com/Muhammadinaam/speed-admin


Nterl0k - T1564.004 NTFS Alternate Data Streams abuse

testing data and script for upcoming detection submission

@nterl0k : Hello Steven! thanks for contributing this to attack data! The ps1 file in here is better suited an atomic in this project: https://github.com/redcanaryco/atomic-red-team would be an awesome contribution! I can work with you to your atomic published in that repo as some of us on STRT are also maintainers of that project. can you perhaps update this PR after removing the ps file from here? We also suggest adding the ps file as a github gist on your profile and link it in the detection references.

File removed from repo
I'll look at making PR to atomic in the future.
Regards,
Steven.
-------- Original message --------
From: Bhavin Patel @.>
Date: 12/11/23 5:34 PM (GMT-05:00)
To: splunk/attack_data @.>
Cc: Steven Dick @.>, Mention @.>
Subject: Re: [splunk/attack_data] Nterl0k - T1564.004 NTFS Alternate Data Streams abuse (PR #850)
@nterl0khttps://github.com/nterl0k : Hello Steven! thanks for contributing this to attack data! The ps1 file in here is better suited an atomic in this project: https://github.com/redcanaryco/atomic-red-team would be an awesome contribution! I can work with you to your atomic published in that repo as some of us on STRT are also maintainers of that project. can you perhaps update this PR after removing the ps file from here? We also suggest adding the ps file as a github gist on your profile and link it in the detection references.
—
Reply to this email directly, view it on GitHubhttps://github.com/splunk/attack_data/pull/850#issuecomment-1850999549, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIYP7WKIZVR56KZK6QCYU3YI6C6TAVCNFSM6AAAAAA7GR4K62VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJQHE4TSNJUHE.
You are receiving this because you were mentioned.Message ID: @.***>

wow! that was quick.  👍


Chef-client buffers the log and spits out the log in the end of the recipe

The Chef client does not put out any log when I create a machine using vagrant until very end
I saw there was ticket for it in old platform and it is marked as solved but the issue is there for me so it is NOT solved.I have newest version of everything (chef-dk, chef-client, vagrant)
link to the old ticket : https://tickets.opscode.com/browse/CHEF-1777
==> default: Starting Chef Client, version 11.16.4

chef -v
Chef Development Kit Version: 0.3.0

vagrant -v
Vagrant 1.6.5

I'm pretty certain this got fixed and was actually a vagrant issue if i recall correctly.  Please reopen if my memory is bad.


allow additional :ets config to be passed in on start up

Very cool little library! I often find myself using Cachex just because I need ttls, so I am happy to have a more light weight alternative.
I want to be able to configure the ets table a bit more, for example :read_concurrency, or :write_concurrency, so I have added that possibility. Let me know what you think 😄

Thanks @sneako 🎉


Acronyms with periods at the end of the sentence are tokenized incorrectly

If acronyms with periods are at the end of the sentence, TreebankWordTokenizer and NLTKWordTokenizer would split the last period (which serves both as part of the acronym and as a full stop):
>>> import nltk
>>> nltk.TreebankWordTokenizer().tokenize('I have been to U.S.A.')
['I', 'have', 'been', 'to', 'U.S.A', '.']
>>> nltk.NLTKWordTokenizer().tokenize('I have been to U.S.A.')
['I', 'have', 'been', 'to', 'U.S.A', '.']

Sentences with acronyms at the start or middle are handled correctly:
>>> nltk.TreebankWordTokenizer().tokenize('U.S.A. is a north American country.')
['U.S.A.', 'is', 'a', 'north', 'American', 'country', '.']
>>> nltk.NLTKWordTokenizer().tokenize('U.S.A. is a north American country.')
['U.S.A.', 'is', 'a', 'north', 'American', 'country', '.']

https://stackoverflow.com/questions/47274540/how-to-improve-nltk-sentence-segmentation

The awesome-ness of Kiss and Strunk (2006) Punkt algorithm is that it's unsupervised. So given a new text, you should retrain the model and apply the model to your text

I don't think this is a code issue, so I would believe we can close this issue.

In my use cases, I do not use nltk.word_tokenize (which would call nltk.sent_tokenize first). I call TreebankWordTokenizer and NLTKWordTokenizer directly for word tokenization task (sentence tokenization, if needed, is handled separately by other functions).

@arademaker this indeed does not concern Punkt. NLTKWordTokenizer and TreebankWordTokenizer, which are regex tokenizers. To be specific, the following regex causes the issue:
https://github.com/nltk/nltk/blob/3b72fa2b3dea0088cd6d15d4409ce52318a3d1a0/nltk/tokenize/treebank.py#L65-L68
It searches for occurrences of a non-dot, followed by a dot, followed by some token that may be after a dot (e.g. " or )), and then whitespace, before the end of the text. As a result, it matches A. in I have been to U.S.A..
This can be "fixed" (or rather, improved), by changing the regex such that the first non-dot is now a non-dot and non-capitalised letter. I.e. we get ([^\.A-Z])(\.)([\]\)}>"\']*)\s*$ as the regex.
However, this is not ideal:
Current situation
>>> import nltk
>>> nltk.TreebankWordTokenizer().tokenize('I have been to U.S.A.')
['I', 'have', 'been', 'to', 'U.S.A', '.']
>>> nltk.TreebankWordTokenizer().tokenize('I have been to USA.')
['I', 'have', 'been', 'to', 'USA', '.']

Situation after described fix
>>> import nltk
>>> nltk.TreebankWordTokenizer().tokenize('I have been to U.S.A.')
['I', 'have', 'been', 'to', 'U.S.A.']
>>> nltk.TreebankWordTokenizer().tokenize('I have been to USA.')
['I', 'have', 'been', 'to', 'USA.']

In short, this would likely be a regression for most cases.
Perhaps some more complex solutions could work, e.g. checking for texts that (don't) match ([A-Z]\.){2,}. I'm open for someone to have a look at this, but be wary of the potential regressions that could follow.

I'm wary that all solutions would introduce regressions, so I'll close this for now.


#161 Improve Quality Control

Change cloudly pixel fraction cut from 10% to 50%
Check for masked pixels on sub-images
Minor changes in image utils

Fixed confliects with base branch.
Imporved output while processing network centrality.

#161


VS Code shortcuts missing

Check for existing issues

[X] Completed

Describe the feature
I have found that some shortcuts are missing when I select VS Code keymap in Zed. I'm not sure if this is the bug or feature request.
Cmd+Shift+L - In VS Code this would select all the words in the document, similar of doing Cmd+D through entire document.
Alt+Up/Down - This would move current line up or down (mac)
If applicable, add mockups / screenshots to help present your vision of the feature
In VS Code, if you select fmt and then press Cmd+Shift+L this happens:

Partly related to #1318

Hey @mileusna, can you upvote this:

https://github.com/zed-industries/community/issues/75

Will close this as a duplicate to keep the votes in one place.

Alt+Up/Down - This would move current line up or down (mac)

@JosephTLyons is there an issue for this one?
alt-up move current line (or selected lines) up
alt-down move current line (or selected lines) down
alt-shift-up duplicate current line (or selected lines) above
alt-shift-down duplicate current line (or selected lines) below


Service testing

Add the test command for the service.
This command now run a server, start the service in parameter then listen for events (filter them if necessary) and also send task with data
closes #28

@NicolasMahe why add a random port ? how the service can communicate if it stays alive and we change the port for every tests...
If it's just to avoid the conflict with the daemon I don't agree, maybe we should provide a --port flag to fix this but definitely not a random port.
Can you explain why you added this please

good point. commit reverted.


ambient: use upstream Gateway API

Currently, we use a fork of gateway-api to remove the requirement to have listeners set. Using a fork of a ~core Kubernetes API is not viable, so this reverts this.
Now to deploy a waypoint, you do:
apiVersion: gateway.networking.k8s.io/v1beta1
kind: Gateway
metadata:
  name: namespace-waypoint
spec:
  gatewayClassName: istio-waypoint
  listeners:
  - name: mesh
    port: 15008
    protocol: ALL

Alternative is to get https://github.com/kubernetes-sigs/gateway-api/pull/1596 merged but it isn't making progress.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio_exp
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio_exp
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test lint

gencheck_istio_exp
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio_exp
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test lint

gencheck_istio_exp
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test gencheck

integ-ambient_istio_experimental-ambient
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio_exp
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test gencheck

integ-ambient_istio_experimental-ambient
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test integ-ambient

lint_istio_exp
f77c64c1f17d770ef52933b23d7099375b7ec58d
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio_experimental-ambient
092ec8ce4a739312cfde74ea62bc8aff79253952
link
true
/test integ-ambient

lint_istio_exp
f77c64c1f17d770ef52933b23d7099375b7ec58d
link
true
/test lint

gencheck_istio_exp
f77c64c1f17d770ef52933b23d7099375b7ec58d
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio_experimental-ambient
c49088468daa05f085a4ad7a7286c498140e19fa
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio_experimental-ambient
0c9bd11fe9cce4158e7da25122fc783cf5bb7dad
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio_experimental-ambient
2de2fdd2662645229cb9bd5dd9f296090e20b8e5
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn Is it intended to remove the CRD? After this PR got merged, I cannot apply gateway resources anymore. Just create a PR https://github.com/istio/istio/pull/43323 to sync the CRD back, but not sure if it is needed.


include margins in the label translate

fixes #375

rebased

A little explainer would be nice here… I’m finding it hard to follow what the logic was vs. what it should be. I feel like the current behavior was intentional (put the x-axis label as far right as possible to give it space); now I’m trying to decide what the new behavior should be, if the current one is not desirable.

The idea is to position the axis label relative to the "tip of the axis", instead of having it float to the far end. In the case of the OP, that gives:
before

after

But what is the default label margin of 20?

That was to match (most of) the existing plots.

Because normally there isn’t a second axis? It’s matching the 20 from here, I presume.
https://github.com/observablehq/plot/blob/07cf90baf428ade198d1cac5ec3ab3f531804995/src/plot.js#L118-L121

I see now that this branch works only in a few cases, and is far from working in general.
I've made an interactive here https://observablehq.com/d/79d3d39812dd3441 ; it seems to me that the behavior of the two first charts is desirable, i.e. that the label should be tied to the tip of the axis. But it would need to work in the four faceting possibilities (no faceting, x, y, x+y), for labelAnchor=left, and also for the vertical axis (which I haven't touched here either).
Maybe an alternative would be to have a configurable label.shift? (label.offset is for the orthogonal direction). I'm closing this for the time being.

Nice notebook. That helps! 👍 It’s definitely tricky and I expect we do want some tweaks to the default behavior as well as likely new options.


[backport 17.06] Fix handleEPTable log

There was an extra parameter not in the formatters
Signed-off-by: Flavio Crisciani<EMAIL_ADDRESS>(cherry picked from commit b34bc70afb5b2ce4d979cf2a27b146777510e488)

Not a critical bugfix, it is already fix in subsequent releases


Add DuckDuckGo and MSNBot browsers to the crawlers list.

Add crawlers that are routinely showing up as real hits to the crawlers list.

Thanks for submitting this. Please add this to the browsers.list file. Thanks.

Done.
Thank you for the feedback.

Merged. Thanks!


[New] Add trace utility network

Description
This PR implements Trace utility network.
URL to README here
Linked Issue(s)

swift/issues/2259

How To Test

Tap "Star a new trace"
Select a trace type
Tap various network elements to add them as starting points or barriers.

Note, to test the terminal picker look for elements with a triangle icon. You may need to zoom way in to get identify to hit them properly

Tap "Trace"

Screenshots

Can you give the reasons why you didn't split it? Maybe I'll be convinced if there is other benefits I haven't seen. Thanks.

Mainly for simplicity, brevity and it's my understanding that view models are good for making SwiftUI code more testable. Since this code won't be programmatically tested I opted to write everything all in the view. I think it might also be a bit easier for someone to follow with everything in one file.
That being said, if you prefer a view model, I'd be happy to go that route.

Merge conflicts resolved

I would say whatever layout the samples app is currently using (a view model or not), I would follow that.

I took a look at the existing samples and interestingly it's quite split, 21 with a model and 13 without. However it's quite clear that a model is used whenever a sample is longer and so this would be an outlier if it didn't so I'll go ahead and implement one. Reverting to draft.

For some reason, only the main and model files show in the picker.

I see them all on iPhone sim:

Also, when I start a new trace on iPad I get weird UI behavior, where the select trace type is cut off at the top.

I've made some UI improvements to closer match the old sample. This is ready for re-review.

Thank you both!


Iap getproductlist basic draft

This is a very basic first draft for combining android and iOS get product list.
Would like this to be solid before working into more details.

@mklim Updated with example app. Please take another look.

@mklim Updated with suggested reviews.


Inspector Pane Does Not Resize

Godot version
4.0.3.stable
System information
macOS 13.3.1 (a), M2 Pro
Issue description
I noticed today using Godot 4.0.2.stable that when an AnimatedSprite2D is selected in the Node Tree pane, and SpriteFrames is active in the bottom pane (not sure what it is called, but I am using the default layout), I could not resize  the Inspector pane. In addition, the Node Tree pane only resizes to a limit.
I upgraded Godot this morning to 4.0.3.stable and the issue still persists.
I am aware that this may be by design due to the SpriteFrames pane needing horizontal real-estate, but I am posting this bug just in case it is not.
Steps to reproduce
Create a Scene as per the screenshot below:

With the AnimatedSprite2D selected in the Node Tree, and SpriteFrames selected in the lower pane, resizing the Inspector pane, using the vertical pane divider, does not change the width of the Inspector pane.
Minimal reproduction project
MinimalBugProject.zip

This is because the SpriteFrames editor imposes a minimum width which is very wide, and therefore prevents you from resizing the inspector (due to your screen resolution being low).


Indicate which variants have comments in the variants listing page

...and allow listing/filtering of only "tagged"/commented variants

:+1:


Vive hands are not tracked in Unity 5.6

SteamVR camera no longer updates hands unless an UpdatePoses component is added to it. It's an unexpected change from unity 5.5.
http://answers.unity3d.com/questions/1299567/vive-controllers-not-tracking-in-unity-560b3.html

This is explicitly stated in the SteamVR readme.

Yeah, we are waiting for a fix from Unity in the 5.6 beta before I can re-enable the auto add that it does elsewhere.  We are seeing a GL.End error pop up because the dummy camera that is being used has its clear flags set to Nothing.

Should be fixed by #14 for every Unity version.

I am not able to see my vive controllers even with steamvr 1.2.2 and unity 5.6.1f. I may be doing something wrong but i can't figure what. Any clue to help me ?


Template no longer working after update.

I noticed that the data.json file has my old template as the "noteTemplate" but most of the items in this template are not showing up. Photo attached so you can see what it does vs what I'm expecting from the template below.
"---\ntitle: {{title}}\nauthor: {% if author %}{{author}}{% endif %}\nyear:\ntags:\npages: {{pages}}\nread:\nrating:\n---\n# {{title}}\n\n### ris:FileInfo References\nRelevant Notes:\nRelevant Books:\n\n### Summary\n\n### ris:Information Key Ideas\n\n### ris:ChatQuote Key Quotes\n\n### ris:Book3 Book Highlights\n{% for highlight in highlights %}Pg. {{highlight.page}}\n- {{highlight.text}}\n{%- if highlight.note %}{{highlight.note}}{% endif %} \n{% endfor %}\n\n#### Footnotes"
Also, how do I get rid of the ^ref at the end of each note?

You cannot get rid of the ^ref- block quote at the end of a line with an annotation. The plugin requires this reference to understand what annotations are in the file, in order to be able to resync future new highlights.

What about the template issue, how do I resolve that?

I would like an option to prevent the reference block. When I import a book, I am finished with it and it will no longer require another sync.

As I explained above, the referenced block is now required in order for the plugin to support resync in case any new annotations were added later on for a given book. Adding an option to remove the reference block will add unnecessary complexity to the plugin (both technically and from a usability perspective). I am not currently inclined to add this unless it is a popular request  by many users.

What about the template issue, how do I resolve that?

I am not sure if I understood the template issue that you are referring to. Do you mind describing it again. Can you also confirm what version of the plugin are you currently running please.

@hadynz I understand that it's required, simply saying it isn't something I prefer to have if it were possible. For now I'll use Find & Replace to edit them all out at one time.
Regarding my issue with the template... The first pst at the top shows what my old note template was for this plugin
"---\ntitle: {{title}}\nauthor: {% if author %}{{author}}{% endif %}\nyear:\ntags:\npages: {{pages}}\nread:\nrating:\n---\n# {{title}}\n\n### ris:FileInfo References\nRelevant Notes:\nRelevant Books:\n\n### Summary\n\n### ris:Information Key Ideas\n\n### ris:ChatQuote Key Quotes\n\n### ris:Book3 Book Highlights\n{% for highlight in highlights %}Pg. {{highlight.page}}\n- {{highlight.text}}\n{%- if highlight.note %}{{highlight.note}}{% endif %} \n{% endfor %}\n\n#### Footnotes"
It's the structure I use for all the books I import. Since recently updating this plugin (1.5.2) it removed my note template as the plugin was restructured in some ways.
I found my note template in the data.json file which suggests to me it is still being used as the template when I import books but the image in my original posts shows that much of what's in my note template itself is not showing up.
For example, all of this is in the data.json but isn't on the page when I import...
n### ris:FileInfo References\nRelevant Notes:\nRelevant Books:\n\n### Summary\n\n### ris:Information Key Ideas\n\n### ris:ChatQuote Key Quotes\n\n### ris:Book3 Book Highlights\n{% for highlight in highlights %}
How do I get it to work with my original template? Otherwise I'll have to restructure every single book I import which is a huge waste of time. I'd rather just go back to the previous version of this plugin without having to work on removing the location data and ^ref blocks.

@hadynz I understand that it's required, simply saying it isn't something I prefer to have if it were possible. For now I'll use Find & Replace to edit them all out at one time.

You need to be careful when you do this. If you were to add any new annotation for a book that you synced before, the plugin will detect that there is a new annotation and will trigger a resync. Because you are planning to remove all block references generated by the plugin, the plugin will make an assumption that no annotations have been synced before. The resync will then bring over all your annotations, leading to duplication.

It's the structure I use for all the books I import. Since recently updating this plugin (1.5.2) it removed my note template as the plugin was restructured in some ways.

Before 1.5.2, the plugin had two customisable templates:

Annotation template — how an individual highlight is rendered
Book template — how your entire highlight is written in the file

I removed the second template to reduce complexity in the plugin. Am I correct in my understand that this is the template that you want to bring back again?

Yeah, the book template is what I used to lay out the entire way the book was put into Obsidian so it matched up with how I write up my other notes. With how it is now I have to spend 5-10 minutes for each book to structure and format it how I do all my other notes.

Is there a way to roll back the plugin to a previous version where the previously mentioned feature still worked?

The book template saves time formatting the metadata. Can you add it again?

I can bring back the feature to format the overall book template. If people can share what some of their templates look like, it would be quite useful when looking at making this feature again.
In terms of the ^ref at each note, there is no plans to remove it or make it optional at this stage.

I can bring back the feature to format the overall book template. If people can share what some of their templates look like, it would be quite useful when looking at making this feature again.
In terms of the ^ref at each note, there is no plans to remove it or make it optional at this stage.

Here is the data I had in the original template before it was removed.
`"---\ntitle: {{title}}\nauthor: {% if author %}{{author}}{% endif %}\nyear:\ntags:\npages: {{pages}}\nread:\nrating:\n---\n# {{title}}\n\n### ris:FileInfo References\nRelevant Notes:\nRelevant Books:\n\n### Summary\n\n### ris:Information Key Ideas\n\n### ris:ChatQuote Key Quotes\n\n### ris:Book3 Book Highlights\n{% for highlight in highlights %}Pg. {{highlight.page}}\n- {{highlight.text}}\n{%- if highlight.note %}{{highlight.note}}{% endif %} \n{% endfor %}\n\n#### Footnotes"
It ended up looking something like this in edit mode.

Is this something still being considered for a future update?

Hey @FrocketGaming. I am going through all issues and just noticed this issue is yet to be resolved.
After re-reading this thread again, I think I understand the issue that you are encountering. Let me know if I understood it correctly and if my explanation/solution works for you.
When you manually open up data.json, you see the original template that you had still correctly stored against a field called noteTemplate and you are wondering why it is not longer being used?
noteTemplate is now a deprecated field. There isn't a replacement for it. You can control how an individual highlight is formatted by modifying highlightTemplate.
With that said, I will be introducing something soon to give you more control to format (a) front matter data, (b) overall template.

@FrocketGaming thanks for your patience. I've implemented what you've been asking for in release 1.6.0. You now have the ability of customising the layout of your main template as you've asked for.
If you can test it out that would be great.

@hadynz whoa. Completely forgot about this. Thanks for your efforts I'll check this out.


zephyr: coap: skip free for observations

Observations are stored in the golioth client struct so that the data may be used to reestablish observations upon reconnect. When the Golioth client is stopped, all requests in the client->coap_reqs linked list are freed. This causes the client address in the stored observation to change, leading to a crash when trying to reestablish. This commit skips freeing observations so the data remains intact. This memory should only be freed when cancelling an observation.
resolves https://github.com/golioth/firmware-issue-tracker/issues/557

This memory should only be freed when cancelling an observation.

Actually, with more thought this memory should never be freed. It's part of the client struct. There's a in_use flag that can be set to false when an observation is cancelled so the memory become available for a future observation.

@szczys Can we add a HIL test for this?

@szczys Can we add a HIL test for this?

I added a commit the puts a loop around the reestablish RPC tests. This takes the test through the client stop/start cycle 3 times. I tested it on main and it fails, but passes on this branch as expected.


Missing requirements?

On some of my smoker systems the test suite fails:
#   Failed test 'folder "./local" is required'
#   at t/01_execute.t line 34.

#   Failed test 'script or command is required'
#   at t/01_execute.t line 34.

#   Failed test 'env is configured to load libraries'
#   at t/01_execute.t line 34.

#   Failed test 'env is configured to execute binary'
#   at t/01_execute.t line 34.
# Looks like you failed 4 tests of 4.
t/01_execute.t ..
Dubious, test returned 4 (wstat 1024, 0x400)
Failed 4/4 subtests

It seems that in these cases prerequisites are missing, at least local::lib.

0.04 and 0.05 are affected.

Hmmm. I don't see any real diffs:
https://metacpan.org/diff/file?target=OKLAS/App-cpanexec-0.06/&source=OKLAS%2FApp-cpanexec-0.05

Hmmm, still no "local::lib" in the requirements...

Thank you! Shade is caught! Happy new year!


 U: .

 U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5199/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 25s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 52s
master passed

+0 :ok:
refguide
2m 40s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 45s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  5s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+0 :ok:
refguide
2m 12s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 52s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

17m 24s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5199/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5199

Optional Tests
dupname asflicense refguide spotless

uname
Linux a47ec0335484 5.4.0-1094-aws #102~18.04.1-Ubuntu SMP Tue Jan 10 21:07:03 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 95f83cb8e2

refguide
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-5199/1/yetus-general-check/output/branch-site/book.html

refguide
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-5199/1/yetus-general-check/output/patch-site/book.html

Max. process+thread count
84 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5199/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Correction for raw affiliation ENSAM, L2MC Laboratory, Moulay Ismail University, MOROCCO

Correction needed for raw affiliation ENSAM, L2MC Laboratory, Moulay Ismail University, MOROCCO
raw_affiliation_name: ENSAM, L2MC Laboratory, Moulay Ismail University, MOROCCO
new_rors: 008axfh28
previous_rors: 008axfh28;018pp1107
works_examples: W4389339343;W4377289505;W4388500177
contact: bdfc9ca27af7bfa7841fe718b5555735:7e52c284f74048b9da762988d0 @ ensam.eu

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


add get-relevance-reports

I set this file as +x. I noticed your other scripts aren't +x, but don't know if that is intentional.
Did you want this generation to be manual (or can we add a make target)?

We can add a make target. I didn't have a make target before because the tools existed before the Makefile. Having it as +x makes sense also. The other scripts are never run directly, so I didn't feel a need to make it +x.


TM Missing in Safari Zone

This is in the northern part of the Safari Zone. Not a big deal for Kaizo, but relevant for Standard/Ultimate

Fixed and deployed


Adding sourmash support

This PR will add the support of sourmash signatures plain(sig), and gzipped (sig.gz, .gz.sig). You must provide a kmer size to select while indexing (signatures might contain multiple kmer sizes).
All the signatures should be stored in a single directory.
The new kSpider CLI options:
Usage: kSpider index_datasets [OPTIONS]
  Index all sketches in a directory.

Options:
  --dir TEXT               Sketches directory (must contain only the sketches)
                           [required]
  -k, --kmer-size INTEGER  kmer size (only if using --sourmash) <--------
  --sourmash               use sourmash sigs instead of kProcessor <--------
  --fast                   if you're certain kSize is found in all smash sigs <--------

Example:
If the signatures are stored in the directory foo.

You want to index the signatures by their signature::name instead of the filename: kSpider index_datasets --dir foo --sourmash -k 51
You're certain that the kmer size exists in all sigs, and you want to index the signatures by their filename  (this will speedup the processing for large datasets) : kSpider index_datasets --dir foo --sourmash -k 51 --fast

Closing in favor of #26


Add configuration for local Kafka REST port

Set as 8082 by default, initially as readonly until the sidecar allows updating the kafkarest-uri

Marking as on hold since we need additional changes from the sidecar to support setting a non-default Kafka REST proxy port.

Won't do


Add bug fix for possibly incorrect error message in generate

Codecov Report
Base: 74.75% // Head: 74.75% // No change to project coverage :thumbsup:

Coverage data is based on head (dbf237d) compared to base (e916e5c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #249   +/-   ##
=========================================
  Coverage     74.75%   74.75%
  Complexity      556      556
=========================================
  Files            86       86
  Lines          1632     1632
  Branches        163      163
=========================================
  Hits           1220     1220
  Misses          352      352
  Partials         60       60

Impacted Files
Coverage Δ

src/main/java/gim/commons/core/Messages.java
0.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Better validation of notification orders

Description
Implements validation of more order fields based on constraints from downstream providers. I came across this since I was wondering what the input constraints was when developing the SMS client in apps
Related Issue(s)
N/A
Verification

[ ] Your code builds clean without any errors or warnings
[ ] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@acn-sbuad since this would be a breaking change I'm not sure if this is something you want. But I think a change like this is good because

Users get bad request errors early (as opposed to only noticing when reading the status, I'm assuming errors due to service limits would appear somewhere in the status/delivery reports).
Unbounded input is not a good idea in terms of security, upstream applications could affect service availability and performance (either due to bugs or security issues) I think

If we do something like this, we should document the limits as well