Improve Documentation on adding-images-fonts-and-files.md

Is your proposal related to a problem?
When looking at the documentation here: https://create-react-app.dev/docs/adding-images-fonts-and-files/
It talks about adding in images, fonts and files.
However all examples/info is about Images and SVGs.
Describe the solution you'd like
I'd like to see more examples of using fonts here as that is part of what the page talks about.

Does this issue needs implementation? If so I'll love to do it.

+1 This page lacks examples of fonts.


PanGesture

I'm not sure this is an issue but rather a question. I'm trying to use a panGesture in my Controller to handle a transition to dismiss the CameraController. I've set swipeToZoom to false but that just turns off the execution of the code within the selector. I'm trying to remove the pangesture all together. I know I can just comment out
    //let panGesture = UIPanGestureRecognizer(target: self, action: #selector(panGesture(pan:)))
    //panGesture.delegate = self
    //view.addGestureRecognizer(panGesture)

Is it possible to remove the panGesture from within my controller without modifying your code.
Is it possible to leave your panGesture but also make sure the code in my Gesture executes? (mine is a swipe left to right) whereas Zoom in top to bottom.

In the newest version, I have exposed both the panGesture and pinchGesture used for zoom. You should be able to add an additional action to them, and they will not interfere. No need to add a second panGesture
Let me know if it is still not working
Andrew


feat: add RLS filters for dbt models (FC-0033)

This adds row-level security filters for datasets created from dbt models by applying the existing can_view_course macro. The schema is listed as None as that is what appears for the schema attribute when working with the Superset library. I've tested that these policies are created as expected but I am still working on confirming that they are applied when a user has neither the Admin nor Alpha roles.

Thanks for the pull request, @SoryRawyer!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

@SoryRawyer It's working, I'm just missing filters for course_names, course_overviews, and course_blocks. Can you add those?

@SoryRawyer 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


TNT-46398 Support .NET 6 and .NET 7

Description
Support .NET 6 and .NET 7
Related Issue
TNT-46398

Motivation and Context

Support .NET 6 and .NET 7
How Has This Been Tested?
Tests are passing and sample app is running on .NET 7

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Should we change the version in global.json?

Should we change the version in global.json?

We should indeed, so that dotnet CLI also works with latest .NET 7, good catch, thanks!


Resolves #14 - Passes Feathers params to service hooks

Summary
Resolves the issue set forth in #14, where Feathers params specified in REST and Socket middleware are not passed through to the user service during authentication.

Thank you for the pull request. Is this fix working for you? The problem is that user services should discard the password field when params.provider is set (see https://github.com/feathersjs/generator-feathers/blob/master/generators/service/templates/hooks-user.js#L29) so there is no way for local authentication to compare password when passing it through. We could make a copy or params and omit(params, 'provider')

Ahh yes, I understand. It does work for me, but that's an excellent
point. I'm on the road at the moment, but once I'm settled I'll update my
PR accordingly.
On Apr 14, 2017 1:08 PM, "David Luecke"<EMAIL_ADDRESS>wrote:

Thank you for the pull request. Is this fix working for you? The problem
is that user services should discard the password field when
params.provider is set (see https://github.com/feathersjs/
generator-feathers/blob/master/generators/service/
templates/hooks-user.js#L29) so there is no way for local authentication
to compare password when passing it through. We could make a copy or
params and omit(params, 'provider')
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/feathersjs/feathers-authentication-local/pull/15#issuecomment-294192939,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkeu9EUx_BikVf8emJPxRLqytaeikL3ks5rv6gAgaJpZM4M9zaW
.

I want to be very careful with this PR. There are potential side-effects (especially security ones) that could occur by passing feathers params to the users service during an authentication call.
As per my comment https://github.com/feathersjs/feathers-authentication-local/issues/14#issuecomment-294237514, the lack of params passing was done intentionally so that one can't call authenticate with query params and potentially get another user's id injected into their JWT access token.

@thomas-p-wilson thanks for fixing this up. I'm okay to merge this PR but I still also feel like this is precisely the reason we have custom verifiers, if you need to do something "custom" other than username and password.
Second opinions from others are welcome otherwise I'll just merge this in a bit.

If you'd really rather leave it as a custom verified I won't be offended :)
On Jun 21, 2017 5:38 PM, "Eric Kryski"<EMAIL_ADDRESS>wrote:

@thomas-p-wilson https://github.com/thomas-p-wilson thanks for fixing
this up. I'm okay to merge this PR but I still also feel like this is
precisely the reason we have custom verifiers, if you need to do something
"custom" other than username and password.
Second opinions from others are welcome otherwise I'll just merge this in
a bit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/feathersjs/feathers-authentication-local/pull/15#issuecomment-310212863,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkeu3DGEcyzT6k0E5or6jNGiul3QXJ2ks5sGY1ugaJpZM4M9zaW
.

@thomas-p-wilson could you resolve the small conflict with the latest master?

Yep. No problem. Will do in the morning!
On Jun 22, 2017 2:19 AM, "Marshall Thompson"<EMAIL_ADDRESS>wrote:

@thomas-p-wilson https://github.com/thomas-p-wilson could you resolve
the small conflict with the latest master?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/feathersjs/feathers-authentication-local/pull/15#issuecomment-310286669,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkeu_2EIbxTmARYm-dY8mtf3XfhK1BAks5sGgd0gaJpZM4M9zaW
.

Pulled down, and merged in master, only to have tests fail. So I pulled
down master on its own, and tests are failing in master.
On Thu, Jun 22, 2017 at 4:19 AM, Thomas Wilson<EMAIL_ADDRESS>wrote:

Yep. No problem. Will do in the morning!
On Jun 22, 2017 2:19 AM, "Marshall Thompson"<EMAIL_ADDRESS>wrote:

@thomas-p-wilson https://github.com/thomas-p-wilson could you resolve
the small conflict with the latest master?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/feathersjs/feathers-authentication-local/pull/15#issuecomment-310286669,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkeu_2EIbxTmARYm-dY8mtf3XfhK1BAks5sGgd0gaJpZM4M9zaW
.

Rebased my PR. Tests passing. Should be good to go!

Ha!  This happened to me last night.  It was weird.  The tests passed in Travis.  I pulled them down and they all failed.  Wiped out node_modules and reinstalled, but it still failed.  When I wiped out the entire project folder and re-cloned it worked fine.
Thanks.  That was really fast!

released as<EMAIL_ADDRESS> Thank you, @thomas-p-wilson!

Cheers!
On Thu, Jun 22, 2017 at 8:43 AM, Marshall Thompson<EMAIL_ADDRESS>
wrote:

released as<EMAIL_ADDRESS>Thank you,
@thomas-p-wilson https://github.com/thomas-p-wilson!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/feathersjs/feathers-authentication-local/pull/15#issuecomment-310368778,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkeu4OltI0fU8wDtB8QX0T2xgV1VJQgks5sGmFjgaJpZM4M9zaW
.

Thanks guys! Nice work!


Add support for a 'default' theme

Add a bit of JS to the app-component.ts to check the system dark theme and enable it in the changeDarkTheme method rather than just checking the app setting.

This will move to dialog-settings.component.ts.


Add confirmation prompts for modifications of lots of tasks

Fixes #179.

(posted on the right PR this time!)
Change looks good, although I'm not too sure about the dialoger library's prompt - in a narrow terminal on macOS Terminal.app it doesn't wrap text, and repeats the line on input:

It works okay in iTerm2, haven't checked on any Linux terminals yet

It would be the same on any terminal.  I'll switch back to the simple theme :)

(and fix the test)

That works better, but still a little weirdly - on input, the two line prompt gets overwritten and repeated:

I wonder if we should:

Merge this PR pretty much as is

Then, fix this problem in upstream library - doesn't seem like there is many other options, and otherwise the prompt seems quite nice

Possibly make the prompt shorter by moving the note about the setting to after the prompt, maybe something like:
 ✔ Operation will modify at least 6 tasks; continue? · no
 Cancelled (note: prompted because config "modification_prompt_count" is 3)

Would be nice if ta config set ... could be worked into that prompt, but gets quite wordy.. but maybe that doesn't matter if we:

Store the modification_prompt_count as Option<u64>. If None then the prompt appears and is treated as the default value of 3. If Some(...) then the prompt is suppressed for neatness (since the user is hopefully aware of the config option as it's been changed)
This be useful for other options too - it can be helpful to tell if a setting is at the default, or if the user has explicitly configured it (which might or might not be to the same default value).
Basically something like loaded_settings.thingo.unwrap_or(default_settings.thingo) instead of loaded_settings = Settings.default(); loaded_settings.update(...). This also means it's easy to then store only the modified settings in the config file, which in turn makes it easier if we ever want to update default settings

Also, pedantically, I wonder if the setting should be named modification_count_prompt (since it's a prompt based on the modification count, not a count of modification prompts)

Also weird failure in the docs build,

Warning: This mdbook preprocessor was built against version 0.4.9 of mdbook, but we're being called from version 0.4.10

Also weird failure in the docs build,

Warning: This mdbook preprocessor was built against version 0.4.9 of mdbook, but we're being called from version 0.4.10

That's fixed on main.

I suspect option 2 is difficult -- detecting wrapping in terminal UIs is rather hard.  I did the other things though!

Thanks for the reviews here -- it's definitely improved the PR so far :)

Looks good! Wrapping weirdness still happens, but now only in an unreasonably narrow terminal, so not worth worrying about
I also noticed ta config set doesn't work if the config doesn't already exist, but that is definitely a separate issue!


NATIVE-616: Fix release notes

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword NATIVE.

I'm not sure what the bug was this tries to fix about the release notes

I have to merge this now as this fixes the releases: https://issues.tuerantuer.org/browse/NATIVE-618


Dealing with exomiser application properties

I would prefer this:

We provide access to a generic application.properties file in the input-dir
Anything we need to set in that file prior to run (data-dir) should be included in the config.yml
The exomiser runner itself parses the options that need to be overwritten from the config.yml and rewrites application.properties as needed.

This isolates all experimental factors of a run into a single file, which is more transparent when interpreting results down the line.

I can expand what I already have in the Exomiser runner (overwriting the data directory path in the application.properties). I think I may have wrapped my head around removing --config-file and moving its contents into the config in the input-dir. I am guessing there would be field that could be written in the config.yml by the tool developer and that can be parsed into methods specific to the runner?

I am guessing there would be field that could be written in the config.yml by the tool developer and that can be parsed into methods specific to the runner?

Exactly! There could be a generic config file reader that loads the whole thing as part of pheval, and then a "getApplicationSpecificConfig()" method that could be used to retrieve them. But note, the developer does not have to use config.yml -> they have their directory, and that has all the files (including config files!) that they want! I just like the transparency coming with adding these parameters to the config.yml file.

Okay, that sounds good! Makes more sense to me now!

I just want to make sure I have the idea in my head right after the last meeting. Should I still be expecting a generic application.properties and editing this in the --input-dir or am I generating an application.properties with all the desired fields within the runner?

The latter. You get all the info through config.yml and write your application.properties file yourself!

got it!


Error During executing bot Error

It looks like there is an error in your account.js, it says "You Have 2 Acc but provide 1"

Have 2 account but provide 1 proxy


Maximum number of scenes/effects?

Govee Device SKU
H6078
Govee2MQTT Version
2024.04.29-30cf7732
Describe the issue
Does the add on have a hard limit of how many scenes (or specifically DIY scenes) it imports/supports? The last 40 are listed and are controllable, but before this, they do not appear. They are useable and visible in the Govee app but not via the add on. When I try to set an effect not listed via an automation, the log shows the error ‘not available for this device’.
From testing, I’m getting an issue after 40 DIY scenes. The 41st oldest scene is never visible free the add on has been restarted. Wondering if this is a hard limit set or part of the Govee api? If anyway to raise or remove it would be appreciated here if possible.
Startup Diagnostics
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
GOVEE_API_KEY=REDACTED
GOVEE_MQTT_HOST=<IP_ADDRESS>
GOVEE_TEMPERATURE_SCALE=C
GOVEE_EMAIL=REDACTED
GOVEE_PASSWORD=REDACTED
GOVEE_MQTT_PORT=1883
++ cd /app
++ exec /app/govee serve
[2024-07-09T18:52:00 INFO  govee::commands::serve] Starting service. version 2024.04.29-30cf7732
[2024-07-09T18:52:00 INFO  govee::commands::serve] Querying platform API for device list
[2024-07-09T18:52:02 INFO  govee::commands::serve] Querying undocumented API for device + room list
[2024-07-09T18:52:04 INFO  govee::service::iot] Connected to IoT: CONNACK code 0: Connection Accepted.
[2024-07-09T18:52:04 INFO  govee::commands::serve] Starting LAN discovery
[2024-07-09T18:52:04 INFO  govee::commands::serve] Waiting 10 seconds for LAN API discovery
[2024-07-09T18:52:04 INFO  govee::service::iot] IoT (re)connected with status CONNACK code 0: Connection Accepted.
Additional Logs
No response
Home Assistant Logs
2024-07-09T18:55:22 ERROR govee::service::hass] While dispatching Message { topic: "gv2mqtt/light/E143C6383032245D/command", payload: "{"state":"ON","effect":"GHS Fire"}", qos: AtMostOnce, retain: false, mid: 0 }: mqtt_light_command: state.device_set_scene: Scene 'GHS Fire' is not available for this device
[2024-07-09T18:55:22 INFO  govee::service::hass] Command for Cylinder Floor Lamp (E1:43:C6:38:30:32:24:5D H6078): {"state":"ON","effect":"GHS Fire"}
[2024-07-09T18:55:22 INFO  govee::service::state] Using Platform API to set Cylinder Floor Lamp (E1:43:C6:38:30:32:24:5D H6078) to scene GHS Fire
[2024-07-09T18:55:22 ERROR govee::service::hass] While dispatching Message { topic: "gv2mqtt/light/E143C6383032245D/command", payload: "{"state":"ON","effect":"GHS Fire"}", qos: AtMostOnce, retain: false, mid: 0 }: mqtt_light_command: state.device_set_scene: Scene 'GHS Fire' is not available for this device
Anything else?
No response

There's no inherent limit in govee2mqtt, but we can only return what the various govee apis are returning to us


fix(policies): removes mandatory constraint in policy providers

This bug affects to any deployment that wanted to switch to the new url providers configuration. host was still mandatory, but both url and host are not allowed.

Thanks @jiparis
Please make sure that you add in the description of the PR
Fixes 1365


Downgrade Webpack, Add Tests

What's New?

Downgraded Webpack for now - some of our plugins were not ready for the upgrade yet.
Added tests for the FBOS version tracker middleware from yesterday.

Coverage increased (+0.07%) to 89.785% when pulling a7895dfec6e1cb9e634f81486b3aedb75869e1f1 on RickCarlino:deprecations_ii into 4f02e7c108e1b70349647e6f170388ea6379e538 on FarmBot:staging.


Paper on SentiLex?

Dear Wladimir,
is there a paper describing the SentiLex project? Do you have suggestions about the best parameter setting of vec2dic?
Thanks in advance.
Best,
Rachele

Hello Rachele,
Glad that you were interested in the project.
Yes, there is a paper (mainly on the re-implementations of existing algorithms).  It was presented at the PEOPLES workshop and is also available on arXiv; you also can find the BibTeX data of this publication below:
@article{Sidarenka:16,
  author    = {Uladzimir Sidarenka and Manfred Stede},
  title     = {Generating Sentiment Lexicons for German Twitter},
  journal   = {CoRR},
  volume    = {abs/1610.09995},
  year      = {2016},
  url       = {http://arxiv.org/abs/1610.09995},
  archivePrefix = {arXiv},
  eprint    = {1610.09995},
  timestamp = {Mon, 13 Aug 2018 16:46:10 +0200},
  biburl    = {https://dblp.org/rec/bib/journals/corr/SidarenkaS16},
  bibsource = {dblp computer science bibliography, https://dblp.org}
}

There might be another paper on NWE approaches in the next year; and they also will be described in my dissertation, which is about to appear in the next two weeks.  I'll post a link either here or in the README file, once it has been published.
As to word-embedding-based algorithms, so far, I've obtained one of the best results with the k-NN approach and the seed set by Kim and Hovy (2004):
./bin/vec2dic --type=1 VECTOR_FILE data/seeds/kim_hovy_2004.txt

The VECTOR_FILE should be in word2vec plain text format with tab-separated fields.  I only experimented with word2vec, but you can try other vectors, such as fasttext or BERT, maybe they will yield better scores.

Dear Wladimir,
thanks a lot for your kind and quick reply!
Looking forward to reading your thesis!
Best,
Rachele

Hello @RacheleSprugnoli ,
You can find the thesis here and cite it as:
@phdthesis{Sidarenka2019,
  author      = {Uladzimir Sidarenka},
  title       = {Sentiment analysis of German Twitter},
  type        = {doctoralthesis},
  pages       = {vii, 217},
  school      = {Universit{\"a}t Potsdam},
  doi       = {10.25932/publishup-43742},
  year        = {2019},
}

if you wish.
I think you might be most interested in the third chapter.  Let me know if you will also be interested in the results of the linear projection method.

Thanks a lot! I will use it for the paper I'm writing.
Best,
Rachele


Uncaught TypeError when field is empty

I have a gallery model with a SortedManyToManyField consisting of photos, pretty standard.
When I go to create a new gallery in Django admin, and the SortedManyToManyField has no photos to display, the following error appears in the web console:
Uncaught TypeError: Cannot read property 'match' of undefined
Specifically it is referring to this line in the static widget.js file:
id = label.attr('for').match(/^(.*)_\d+$/)[1];
This error means that when adding a new item to the SortedManyToManyField via the popup window, the window never closes after saving and the widget is not populated with the new item.
A browser refresh is required to populate the widget. The error no longer occurs once there is at least one photo created.
The issues as I have identified it are:

When I first load the change form, the label has no for attribute and there is no input to go with it, so the match function will always fail.
Even when I manually specify the label for, the regex fails.

If I change the code and ensure it doesn't fall over getting the ID, everything works as intended.

It seems like there are broader issues at play with current Django compatibility so I can't use this package at the moment. I'm going to close this issue.


Notebook '04' showing no Training Effect

Hi Leandro,
first of all many thanks for the amazing work on the library. I've found your documentation very easy to get into - especially paired with your talk at the Reinforcement Learning Meetup in Zurich.
I was toying around with your notebooks and noticed that notebook '04' was not showing any training effect for me. Please see the WANDB-Outputs HERE for reference - this is the result of simply executing the notebook with the provided training parameters, with the exception of a fix to a Key Error I explained in Git Issue #37. I toyed around a bit with learning rates, batch sizes etc. but could not clearly identify a learning effect.
Could you provide guidance on how I can address this issue? Was the fix for Git Issue #37 inappropriate?
Thanks in advance and best,
Philip

What is the value of the Batch size you use?

See #41


Add CURL and Threads as dependency in CMake config.

Use find_dependency() to depend on CURL and Threads so CMake users don't have to
do it themselves.

Checklist
Not all of these might apply to your change but the more you are able to check
the easier it will be to get your contribution merged.

[x] CI passes
[x] Description of proposed change
[ ] Documentation (README, code doc blocks, etc) is updated
[ ] Existing issue is referenced if there is one
[ ] Unit tests for the proposed change

CMake compatibilty

CMakeFindDependencyMacro and find_dependency() are available in 3.10:
https://cmake.org/cmake/help/v3.10/module/CMakeFindDependencyMacro.html.
FindCurl is available in 3.10:
https://cmake.org/cmake/help/v3.10/module/FindCURL.html.
FindThreads is available in 3.10:
https://cmake.org/cmake/help/v3.10/module/FindThreads.html.

Reasoning
Since curl and threads are not optional, they should be required by
restclient-cppConfig.cmake, in my opinion. It makes the CMakeLists.txt of
users a bit cleaner and shorter and has no downsides, as far as I'm aware.
Before:
find_package(CURL REQUIRED)
find_package(Threads REQUIRED)
find_package(restclient-cpp 0.5 REQUIRED CONFIG)

After:
find_package(restclient-cpp 0.5 REQUIRED CONFIG)

thanks for the patch!


Windows gui: window position is changed when invoking :vert & :tab under certain conditions

I sometimes use gvim 7.3 @ win7 (64 bit). When I move the gvim window by means
of the <windows-key + left/right> key combination to the left or right half of
the screen, it seems gvim doesn't properly notice this change. When I invoke
any command with :vert prepended (e.g. :vert split), gvim moves the gvim window
back to the previous position on the screen.

Steps:
1. Open gvim -> gvim window opens at position #1
2. Press <windows-left> -> gvim window is moved to the left (position #2)
3. Type :vert split

Result: The gvim window is back at position #1.

This problem seems to affect only the win gui. At least with gnome/gtk
everything seems ok.

The problem also affects :tab-related commands. It was verified by a user. See
https://groups.google.com/d/topic/vim_dev/MiKswL7hN6E/discussion

Original issue reported on code.google.com by minilith on 13 Mar 2013 at 8:19

I close this, because this is stale and we have a new issue for this.
Let's keep discussion on #2180 and #703.


TweetinviAPI.WebhooksPlugin

I am upgrading TweetInvi to the latest version in my ASP.NET webforms project that targets 4.6.2.
All has gone well however when I install TweetinviAPI.WebhooksPlugin via nuget I get the following error:
Could not load file or assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
I can see no reference to System.Runtime, Version=<IP_ADDRESS>.

I'm also having lots of conflicts when trying to use Tweetinvi.
Is there aversion for 4.6 framework maybe ?

Hi got the same error a couple of days ago, all i did was do a binding redirect in the web.config like:

One for @linvi
The nuget package is for .NET Standard, but the Tweetinvi.Webhooks project targets .NET Core, so I'm confused :)

@awisey The .NET standard nuget package should work on both .NET Core & .NET Framework, so that's already done.
Sorry if I wasn't clear in my last comment, I just can't see the code for the nuget package as the code I looked at in master targets .NET Core.

@JoshKeegan The reason why this project is using .NET Core as opposed to .NET Standard, is because it has dependencies on Microsoft.AspNetCore.All.
This is also the reason why "plugins" were created in version 4.0.
Plugins allow developers to add features to Tweetinvi without having to have dependencies on some libraries that they do not want (in that case ASP.NET).

@remy-burney

I'm also having lots of conflicts when trying to use Tweetinvi.
Is there aversion for 4.6 framework maybe ?

Tweetinvi 4.0 only supports .NET Framework 4.6.1 and above as it targets .NET Standards 1.4.


Create a website

Draft website is here: https://exaworks.org/psi-j-python/#index
Welcome comments on the structure, text, and missing components.


The variance of residual when create regression node

There is calculating the max variance of residuals in your code, I think it should be min. So why you did that ? An could you send me your best fddb roc curves to me<EMAIL_ADDRESS>

Yes, it should be the minimum variance, so I calculate the maximum reduction of variance.
For the ROC curves, sorry, As the README said, I didn't have a fully trained model yet. It's very tough.
If you are interested in JDA, you can join the QQ group mentioned in README. welcome~

Thinks.


Sidebar Disappearing

Hi Frappe Team,
I have a problem that the sidebar suddenly disappears when I open the wiki page, as shown in the picture below:

Thank you in advance

Can you replicate this issue with the latest wiki version?

I wasn't able to reproduce the issue. Feel free to reopen if the issue persists.


chore: bump swc to 0.76.48

Summary

Test Plan

Is this update just a routine for swc or trying to fix some bugs?

Is this update just a routine for swc or trying to fix some bugs?

It is intended to fix a resolving issue from @underfin, but it's yet to be confirmed. You may regard it as a routine upgrade.

we need a rule for upgrade routine swc since it's risky

Any update?


Make '@backstage/core-app-api' a dev dep on all plugins

:heavy_check_mark: Checklist

[ ] Added tests for new functionality and regression tests for bug fixes
[ ] Screenshots of before and after attached (for UI changes)
[ ] Added or updated documentation (if applicable)

The dependency is used in 3 places on the application code. You need to modify those to work in a different way.
plugins/frontend/backstage-plugin-github-pull-requests/src/components/Router.tsx
plugins/frontend/backstage-plugin-travis-ci/src/Router.tsx
plugins/frontend/backstage-plugin-buildkite/src/components/Router.tsx

I believe that you can just use a Router from react-router in place of FlatRoutes for your usages here.


Borrow Chromium's ScopedOpenSSL types.

With a few mods to make it work with OpenSSL and use unique_ptr directly.

LGTM


Add hybrid scan config for the max number of deleted files

What is the context for this pull request?

Tracking Issue: #159
Parent Issue: #150
Dependencies: n/a

What changes were proposed in this pull request?

With the current performance limitation of Hybrid scan operation for delete dataset, we limit the number of  deleted files to avoid regression from Hybrid scan. If the number of deleted files is larger than this config, the index is disabled and cannot be a candidate for Hybrid Scan.
  val INDEX_HYBRID_SCAN_DELETE_MAX_NUM_FILES =
    "spark.hyperspace.index.hybridscan.delete.maxNumDeletedFiles"
  val INDEX_HYBRID_SCAN_DELETE_MAX_NUM_FILES_DEFAULT = "10"

Does this PR introduce any user-facing change?

Yes, a new config is added for Hybrid scan delete dataset.
How was this patch tested?

unit test

LGTM, Thanks @sezruby


Improve tracing document

What this PR does:
improve tracing document
If there is no problem, I will add the English document again.
Which issue(s) this PR fixes:

Fixes #278
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@ZLBer Thanks for your contribution !
I modifed some description in your branch,please review it.
Should I merge this PR now or wait for you to modify the english doc?

@seeflood  wait  a while, I will translate immediately


Adjust PHPDoc type for causer and subject

Per the database schema and the PHPDoc type for causer_id and subject_id, the causer and subject property may be null. I update the PHPDoc to reflect it.

Thanks!


[vSAN Data-In-Transit Encryption] Documentation for Running Scripting Commands to enable/disable the feature.

PR contains addition of steps to perform enabling/disabling the feature and information around it's performance impact as well.

@jjaygbay1
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@jjaygbay1
Can you review this old PR and determine whether it needs to be closed or merged?


Investigate what happens when we try to spend collateral

Using mustSpendPubKeyOutput on the result of getCollateral call makes no sense, we should check what happens and if the error message is clear enough.
This comes from a discussion with our users.
Ideally we should provide an informative error message during constraints evaluation

How does this make no sense? In theory, any input of the transaction can be collateral, because they're consumed mutually exclusively. Either the tx passes phase 2 and the inputs are used normally, or it fails and the inputs may be used as collateral - there's no double spending

It makes no sense if a script is used. If we spend the collateral, the validator can't run. And it may not be obvious, why. I didn't check the exact error message yet.

I have a feeling if built/submitted by cardano-cli, it would be able to run. I haven't tested it though. I think CIP30 wallets might not allow it because of the special treatment of collateral.

It makes no sense if a script is used. If we spend the collateral, the validator can't run.

I've found a source that says it's fine:

The collateral inputs get removed from the ledger UTXO only if any script fails phase-2 validation. If all scripts pass, the specified transaction fee amount gets collected, as in previous eras. In particular, the amount comes from the regular, non-collateral inputs, and the collateral inputs are simply ignored. And, good news! It is permitted to use the same inputs as both collateral and regular, since only one of the two sets ever gets removed from the UTXO.
- https://iohk.io/en/blog/posts/2021/09/07/no-surprises-transaction-validation-part-2/

Also in the alonzo changes pdf

Strange. I've got a report stating that there was an error in this scenario from one of our users. Let's close then.


[e2e]: generic release main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v24.0.462
Run: https://github.com/slsa-framework/example-package/actions/runs/8274411162
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.release.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.release.main.default.slsa3.yml
Trigger: release
Branch: v24.0.462
Date: Thu Mar 14 02:13:06 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v24.0.463
Run: https://github.com/slsa-framework/example-package/actions/runs/8304576417
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.release.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.release.main.default.slsa3.yml
Trigger: release
Branch: v24.0.463
Date: Sat Mar 16 02:12:55 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v24.0.464
Run: https://github.com/slsa-framework/example-package/actions/runs/8336460616
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.release.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.release.main.default.slsa3.yml
Trigger: release
Branch: v24.0.464
Date: Tue Mar 19 02:24:33 UTC 2024
Tests are passing now. Closing this issue.


Issue with in_app_purchase plugin documentation

On this file there are several errors in the example code;
The line:
for (PurchaseDetails purchase : repsonse.pastPurchases) {
Has two errors:

":" should be in
"repsonse" is should be "response" as per the variable name above

Also the line:
if (_isConsumable(productDetails)) {
References the method "_isConsumable()" which isn't defined in the library or the scope of the example.

@NickoftheSouth if you would like to submit a PR to fix these, I would be happy to review it!

@Hixie This is fixed. The _isConsumable part is supposed to be like this I think because it is up to the user to decide if a product is consumable or not (might be wrong, just a thought).

This issue is already fixed.

close this


Update README.md

Add keys property to currency conversion example

Coverage increased (+1.4%) to 95.399% when pulling c0825ac2b79b7aeae6ebc3d9ddcc676f9cd62f07 on matejthetree:patch-1 into cf0f96d80383f0be3a9d2be35dd64cd9abe8d712 on dashersw:master.


extra args & kwargs to pass to the models contructor.

and importing image from cgi.FieldStorage

Could you merge this rebased, single commit pull request?

Merged.  Thanks!

Thanks a lot


Images not displaying in line with text

I am using markdown for my posts, and when I insert an image it doesn't sit in the centre or match the width of the post. When previewing the markdown, the images sit in line with the text. The MD I am using is below:
![The Blast Premier Spring Final stage, from the back of the arena](https://oliverb21.github.io/blog/img/posts/09_old_seats.jpg)
The result can be seen on this page:
https://blog.oliverbouchier.com/2024/06/18/blast-spring-finals-2024.html
TIA :)

fixed by using HTML tag instead of MD


Comments on Sessions

A few small things:

The name appears to be a relative URL starting with //. Is that intentional or should it be https://?

The spend example doesn't have an error message for when there is insufficient balance to spend. That would be helpful to define

I was thinking of the session balance as being tracked in two separate fields, paid and spent, so that you can have more insight into the details of the session. I'm not strongly opposed to the way balance is tracked in the proposal but I like the idea of two monotonically increasing balances rather than one that goes up and down.

For 1, see https://github.com/adrianhopebailie/open-payments/issues/7 and https://aip.dev/122

The spend example doesn't have an error message for when there is insufficient balance to spend. That would be helpful to define

Have added that in.

I was thinking of the session balance as being tracked in two separate fields, paid and spent, so that you can have more insight into the details of the session. I'm not strongly opposed to the way balance is tracked in the proposal but I like the idea of two monotonically increasing balances rather than one that goes up and down.

Have also added this


Fix edge case in range syntax sugar

Handle special case where the first character in the range is lowercase
and the second is uppercase. This is necessary because uppercase letters come
before lowercase letters in the ASCII table, and lets us support ranges like
a-Z.

Just thought of some problems with this fix. Fixing the fix now.


Update otcs.py

In case of no categories assigned to node, assign_category function fails due to "TypeError: argument of type 'NoneType' is not iterable". If empty list returned, assing_category works as expected.

Thank you for the feedback and the suggested fix.
I took your change and implemented it in the main source code repo.
This Github Repo is only a copy of released versions, so I will close this MR, but be assured the next version will contain this fix.


Correct license year.

Fix #38

Fails

:no_entry_sign:
PR body should be 'Fix #N' where N is the issue number

  Generated by :no_entry_sign: dangerJS against 07ad7d6a1acf89969b5ff700aa9d464ff9a7ada2

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

1.0.0
GitHub release

Your semantic-release bot :package::rocket:


Update PlayBilling version to V6

WIP: API updates for Play Billing V6 migration

To check, you've run the tests locally and they still pass, right?

To check, you've run the tests locally and they still pass, right?

Yes, tests pass locally


Not compatible with the version of your running IDE (IntelliJ IDEA 2024.2)

Unfortunately, I can't install the plugin because JetBrains recently updated the versions of their applications, making the plugin no longer installable.
Please adjust the plugin so that it can be used with the new version.

This is a major update, wait for me.

idea Plugin SDK 1.x -> 2.x


added test cases for proxyUtils

Closes: https://issues.redhat.com/browse/RHOAIENG-2159
Description
added test cases for frontend/src/api/proxyUtils.ts

How Has This Been Tested?
npm run test

Test Impact
test coverage:

Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/lgtm
/approve


Add Error Metric

It would be useful to add a warning and error metric to the global metrics.
This way if an error or warning occurs we increment these counts. Often in Lighthouse, we dont emit logs for discovery and a variety of errors and warnings can go unnoticed.
If we had these in the metrics we could just monitor metrics for discovery to observe if anything is going wrong.
I think it would be good to add the following metrics:

Error count
Warning count
Error Type - Give some string context as to which type of error this is. As the debug logs are not saved and we see a large error/warning count it would be good to provide some context into what these errors are.

Error Type - Give some string context as to which type of error this is. As the debug logs are not saved and we see a large error/warning count it would be good to provide some context into what these errors are.

Is the intent here for this to collect the actual error types themselves (i.e., Vec<ErrorKind>) or maintain counters for each type?

Upstream users can build their own as was done in lighthouse.


Missing CIS flag.

There seems to be flag for EU but not for CIS:
https://en.wikipedia.org/wiki/File:Flag_of_the_CIS.svg

EU is a known reserved ISO 3166 code. CIS on the other hand isn't even a two-letter code.

:)

EU is a known reserved ISO 3166 code. CIS on the other hand isn't even a two-letter code.

Your point being? That's it's not a flag? Does inclusions of an union flag depend on ISO code registry?

I would assume, yes. This entire repository is based on ISO 3166-1-alpha-2 country codes.

Seems unnecessarily limiting but sure, I can add one svg file to my own fork.

There are thousands of flags in general.. we simply can't get them ALL in one project.. :)


MolochCapture will not start "ERROR - OUI file /data/moloch/etc/oui.txt bad line"

I have initialized the database, and my config.ini file shows elasticsearch=http://<IP_ADDRESS>:9200
Elasticsearch version: 5.6.8
Moloch version: 1.6.1-1
jre - 8u192-Linux-x64
OS Name: CentOS 7 (3.10.0-862.x86_64
Installed via RPM
Stack Trace.
[root@localhost etc]# gdb /data/moloch/bin/moloch-capture

Reading symbols from /data/moloch/bin/moloch-capture...done.
(gdb) run /data/moloch/bin/moloch-capture
Starting program: /data/moloch/bin/moloch-capture /data/moloch/bin/moloch-capture
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
[New Thread 0x7fffe2b3d700 (LWP 3120)]
[Thread 0x7fffe2b3d700 (LWP 3120) exited]
Dec  3 09:55:18 http.c:283 moloch_http_send_sync(): 1/1 SYNC 200 http://localhost:9200/_template/sessions2_template?filter_path=**._meta 0/80 6ms 29ms
Dec  3 09:55:18 http.c:283 moloch_http_send_sync(): 1/1 SYNC 200 http://localhost:9200/sequence/sequence/fn-localhost 0/91 0ms 3ms
Dec  3 09:55:18 http.c:283 moloch_http_send_sync(): 1/1 SYNC 200 http://localhost:9200/sequence/sequence/fn-localhost 2/161 0ms 35ms
Dec  3 09:55:18 http.c:283 moloch_http_send_sync(): 1/1 SYNC 404 http://localhost:9200/stats/stat/localhost 0/68 0ms 5ms
Dec  3 09:55:18 http.c:283 moloch_http_send_sync(): 1/1 SYNC 200 http://localhost:9200/fields/field/_search?size=3000 0/13157 0ms 8ms
Dec  3 09:55:18 db.c:1900 moloch_db_load_oui(): ERROR - OUI file /data/moloch/etc/oui.txt bad line ''
[Inferior 1 (process 3116) exited with code 01]
Missing separate debuginfos, use: debuginfo-install moloch-1.6.1-1.x86_64

I would suggest deleting the bad line.  Please come over to slack for questions since this isn't a bug.

Full error is:
ERROR - OUI file /data/moloch/etc/oui.txt bad line ''

Looks like the file didn't get downloaded correctly, you could try running moloch_update_geo.sh again


ovh-angular-ui-confirm-modal

Migrate ovh-angular-ui-confirm-modal component
The goals of the migrations is:

Drop the Bower support
Apply ESLint Airbnb rules
Provide clean and concise README file.

We might consider using the share rollup configuration (cf: component-rollup-config)
Requirements
👀 Does the component is still used?

yes! on ovh-manager-telecom only.

💅 Does the component must be renamed?

yes! must be renamed to ng-ovh-ui-confirm-modal

🚨 Does the component has a dependency to another pending migration?

no.

⚓ Related pull request
Please list here the related pull request.

Listing of usage from ngReallyClick directive.
On ovh-ux/ovh-manager-telecom:
client/app/telecom/pack/resiliation/pack-resiliation.html
client/app/telecom/pack/xdsl/access/ip/pack-xdsl-access-ip.html
client/app/telecom/pack/xdsl/access/ipv6/pack-xdsl-access-ipv6.html
client/app/telecom/pack/xdsl/access/pack-xdsl-access.html
client/app/telecom/pack/xdsl/access/portReset/pack-xdsl-access-port-reset.html
client/app/telecom/pack/xdsl/modem/bridgeMode/pack-xdsl-modem-bridgeMode.html
client/app/telecom/pack/xdsl/modem/managedByOvh/pack-xdsl-modem-managedByOvh.html
client/app/telecom/pack/xdsl/modem/reboot/pack-xdsl-modem-reboot.html
client/app/telecom/pack/xdsl/modem/reset/pack-xdsl-modem-reset.html
client/app/telecom/pack/xdsl/modem/reset/pack-xdsl-modem-reset.html
client/app/telecom/pack/xdsl/modem/wifi/pack-xdsl-modem-wifi.html
client/app/telecom/pack/xdsl/resiliation/pack-xdsl-resiliation.html
client/app/telecom/telephony/alias/portability/portabilities/telecom-telephony-alias-portability-portabilities.html
client/app/telecom/telephony/service/voicemail/options/telecom-telephony-service-voicemail-options.html
client/components/notification/notification-list.html

On ovh-ux/manager:
packages/manager/modules/freefax/src/faxConfiguration/freefax-faxConfiguration.html
packages/manager/modules/freefax/src/notifications/freefax-notifications.html
packages/manager/modules/overthebox/src/details/overTheBox-details.html
packages/manager/modules/overthebox/src/details/overTheBox-details.html
packages/manager/modules/overthebox/src/remote/overTheBox-remote.html
packages/manager/modules/overthebox/src/remote/overTheBox-remote.html

Since the import of the current controls panels as started, the migration of this component isn't a top requirement so that's why I close this ticket in order to turn off the associated project.
Thanks!


[BUG] Webpack 5 Broken Bundle

Bug report
I would build bundle by command npm run build.
After that I get complete bundle.
But when I change some value I get error in to console.
I would get this error when I didn't setup any loader for some types of files (like .css, .js, .vue etc)
I use Vue + Vuetify build.
I can share a full code base with you if you want.
What is the current behavior?
Sounds a little weird. But it really is.
Editing files that are not related to the main (bundle), it somehow breaks. Then it gives an error as if there are no loaders (loaders) that upload the necessary files.

by google translate

If the current behavior is a bug, please provide the steps to reproduce.
I'm not sure. However, I think it is possible to repeat something like this:

Install Webpack 5.4.0
Install Vue 2.6.12
Install Vuetify 2.3.16
Install loaders (css, vue, sass and etc by vuetify docs)
Setup code for load Vue, Vuetify and etc.
Run build bundle.
Change code.
Get broken bundle.

What is the expected behavior?
I didn't get broken bundle by edit one of js files.

Other relevant information:
webpack version:  5.4.0
Node.js version:  12.18.3
Operating System: Windows
Additional tools:
Screenshots
There I have fully work page without any changes:

Then I change by adding comment line:

Then I was restart page:

Profit! Get error.
The same error but when I load min.css file (vuetify)

The same troubles I have without this css files.
for example when I remove this css file. I get second error:

For fix this error I should rebuild bandle:

After that all is work. IDK how that work.
Loaders:

Sorry if I forget add something important.
Call me and I will try add this fast how I can.

Hi all who saw this issue.
Today, after awake, I see what this trouble has been disapear manualy.
Idk why. I didn't change anything.
So, I should close this issue?

Yes, looks problem in Vuetify, nothing to do on webpack side


Add ignorer to .env.test variable in NodeJS

Reasons for making this change:
while working on my project, I realized that I always needed to add an ignore in the .env variable of tests, where I call .env.test, and I thought that many other people using this pattern go through the same thing, so I decided to try to sub.
Links to documentation supporting these rule changes:
I dont have a specific documentation link talking about having to use this pattern, but it's almost a convention, separating .env from tests by .env.test
If this is a new template:
None

@lmedson thanks!


Problems handling docker images with path deeper than one directory

docker-registry-frontend cannot handle docker images added to the registry which have a deeper path than one directory. selecting a image with deeper path won't show tags, but will be redirected to the frontend home directory
{
 "repositories":
   [
   "anaconda/cijenkinsnode",
   "centos",
   "dbcompiler/ci/oraclejava",
   "dbcompiler/cijenkins",
   "dbcompiler/cijenkinsnode",
   "dbcompiler/cioraclejava",
   "dep/jenkins/r2d2",
   "layer/dmc",
   "ci_buildenv",
   "phusion/baseimage",
   "ubuntu"
   ]
}

Examples:

images without issue:

centos, ci_buildenv, ubuntu,
dbcompiler/cioraclejava, phusion/baseimage ...

images whcih cannot be viewed correct in the web interface:

dbcompiler**/ci/**oraclejava
dep**/jenkins/**r2d2

Expected behavior is to be able to walk through the path to get the tags shown on the web page.
At the moment we are running Docker 1.12.0 on RHEL 7.2 with docker-registry-frontend:v2

@othews you're right, currently this is not supported. To cite the registry API referenc:

Classically, repository names have always been two path components where each path component is less than 30 characters. The V2 registry API does not enforce this. The rules for a repository name are as follows:

A repository name is broken up into path components. A component of a repository name must be at least one lowercase, alpha-numeric characters, optionally separated by periods, dashes or underscores. More strictly, it must match the regular expression [a-z0-9]+(?:[._-][a-z0-9]+)*.
If a repository name has two or more path components, they must be separated by a forward slash (“/”).
The total length of a repository name, including slashes, must be less the 256 characters.

+1 on this.
Sonatype Nexus 3 implements Docker v1, v2 APIs
and does not restrict repos to / path.
I would like to have repos with name like
upstream//
and docker-registry-frontend does not support it.

It seams that docker registry api support multi-slash
curl -k https://registry:5000/v2/xxx/beta/ubuntu/tags/list
{"name":"xxx/beta/ubuntu","tags":["0.10.0"]}

+1 on this. would be nice, if this issue could be resolved

+1

+1

+1

+1


Move NetworkManager GoRoutine Refresh Interval to Command Line Config

#1206  👈
feature/proof-of-stake

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tholonious and the rest of your teammates on  Graphite


Output values not consistent with the library

dependencies": {
"react": "16.11.0",
"react-native": "0.62.0",
"react-native-device-info": "^5.6.1"
}
Values returned in console log:

Not sure why the values are being returned in JSON instead of single values. I am sending this values to server and it fails. Any suggestions ?

APIs return a Promise which you are referencing directly, you need to resolve the promise either with async/await or API_you_called().then(() -> { /* etc */ }
When I first started coding async javascript these results were always a puzzle, now I recognize them immediately as what a Promise looks like when you send it directly to console.log
Burn that visual pattern of little underscore-numbers into your mind so that you can recognize them immediately as well, it will save you time later :)

Thank you !! it works.
What is the difference between getSerialNumberSync() vs getSerialNumber() ?

The difference is exactly as described in the documentation, and demonstrated in the example


fix: Correct typo in test.txt file

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


Error when running deployed create-react-app

Description
My local app is running fine, initialises from storage etc. However, when using create-react-app build and deploy, I get the following errors:

I can't figure out what is going on... does anyone know if I need to add some config to tell the compiler what to do? the duplicate name here seems to be an empty string (or  )

Fixed this by adding a name to the persistent reducers. Strange that this only surfaced in the prod build
persistentReducer(counter, {name: 'custom name'});


s/fireeye/mandiant

[x] this repo
[x] flare-floss-testfiles

closed via #443 and https://github.com/mandiant/flare-floss-testfiles/pull/1


[added] gr-rose related projects for cc-rx and gcc environment.

Description

Please add GR-ROSE related project to the rx_development_202012.00 branches on renesas amazon-freertos repo.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I also merged RSKRX65N GCC environment comes from Singapore Xiao Xuan-san into this pull request.

thanks to @VanNamDinh


[FEATURE] Roll logs by execution instead of size

Is your feature request related to a problem? Please describe.
Currently, logs are rolled depending on the size of the log file. This makes it hard to identify what logs are related to what command. Additionally, we may run into race conditions or even writing issues when two commands are executed simultaneously, since both of them will be trying to write to the same file.
Describe the solution you'd like
Create separate log files for each command executed. This ensures that no two commands will try to write to the same file, and also makes it easier to identify what logs are important for debugging a specific issue with a command.

closed by https://github.com/mlcommons/medperf/pull/425


Update VR to autoresync only when moving VR as secondary due to failover

In all other cases of moving to secondary it is a relocate, and hence a known primary is being demoted and does not require a auto resync.
If being set as the primary, then there is no resync operation to control anyway.
This adapts to changes in csi-addons here to ensure there is not force resync when demoting a volume during a relocate operation due to any spurious errors.

Update: Stuck with some e2e failures possibly related to ceph-csi changes

Update: Stuck with some e2e failures possibly related to ceph-csi changes

e2e completed with required csi changes. Merging this PR!


⚠ Processors has degraded performance

In 2e901f9, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 50 ms

Resolved: Processors performance has improved in e4903ab.


fix: allow for IUPAC codes in GetCanonicalMotif()

partially addresses #238
This will allow TRTools's GetCanonicalMotif() to work for sequences that have IUPAC codes. However, this will not entirely solve the problem described in #238, since any function that calls GetCanonicalMotif() will now have to handle the IUPAC codes somehow.

Thanks for the suggestions! I tried my best to implement them in c1f78eeb73393a9642398911b9c585496b173af4
I think the use-case is to use GetCanonicalMotif() to discover whether variant records from different callers are mergeable in EnsembleTR:
https://github.com/gymrek-lab/EnsembleTR/blob/43104bfac4f98e5ea7500e2220500c21e6f337c5/ensembletr/vcfio.py#L230

great, thanks for the review, Jonathan!


fix: :error consistency in remove_keyword_key and argument_equals? in Config.configure

Contributor checklist

[ ] Bug fixes include regression tests
[ ] Features include unit/acceptance tests

This PR makes sure that Keyword.remove_keyword_key/2 always returns :error when it fails. It also modifies try_update_three_arg/5 and try_update_two_arg/5 to use argument_equals? for keys instead of argument_matches_pattern?.

🚀 Thank you for your contribution! 🚀


Doesn´t work with file-matcher

I have js files in differents directories, and for each file I want "mix" this. To find these files I am using a third party lib (https://www.npmjs.com/package/file-matcher), and the mix api does not work in that callback.
The following code, works normaly:
let mix = require('laravel-mix');
mix.js('src/file.js', 'dest/file.js');

But, the next code, does not work:
let mix = require('laravel-mix');
let fm = require('file-matcher');
let fileMatcher = new fm.FileMatcher();
fileMatcher.find({
    path: path.resolve(__dirname, 'src'),
    recursiveSearch: true,
    fileFilter: { fileNamePattern: '**/*.js' } })
    .then(function (files) {
         for (file of files){
            mix.js(file, 'dest/js/');
        }
    });

It doesn´t throw any errors or messages, just no one file is compiled.
Can any good soul help me?

We can not force mix to run multiple time in a single run.
However original webpack allows.

@ankurk91 tks!


update template (GSI-632)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
698

Relevant Lines:
741

💛  - Coveralls


Keep default-detekt-config.yml synced is a mess

Expected Behavior
I don't need to execute a command and commit it's output when working with detekt.
Current Behavior
When I modify things related with rule configurations or add a new rule I need to execute generateDocumentation task and commit the changes.
Context
#7745 is a case where this strange behavior makes the contributions more difficult for the maintainers and for the person opening the PR too.
Even I get this type of errors. I know how to fix them fast but it would be nice to avoid them in the first place.
 Configure the gradle tasks correctly so the file is generated when needed.
I see a pro and a con with this.

Pro: we get users that find the default-detekt-config.yml on our repo and uses it instead of using the task generateConfig. This creates issues because the file default-detekt-config.yml is in sync with main instead of the version that the user is using. Users had open issues related with this. If we remove the file this problem can't happen
Con: The configuration changes are easier to review (on PRs) from the yml file than from the code that it generates it. But if this is a huge problem I imagine that we could find another way to solve it.

I don't need to execute a command and commit it's output when working with detekt.

Isn't this the same behavior as a lot of other tools (to name one: binary-compatibility-validator and the .api files).
If we move to auto-generating default-detekt-config.yml without committing we'll lose a lot of niceties coming from having that file in the git history. For example now you can easily look through the history of all the changes in the detekt repo that resulted in a configuration change.

I don't need to execute a command and commit it's output when working with detekt.

Isn't this the same behavior as a lot of other tools (to name one: binary-compatibility-validator and the .api files).
If we move to auto-generating default-detekt-config.yml without committing we'll lose a lot of niceties coming from having that file in the git history. For example now you can easily look through the history of all the changes in the detekt repo that resulted in a configuration change.

You need to do the same but I think that it's different. No one wants to change the .api nor a baseline.xml. But you need to have in the repo the current state of the API or to the "known issues". Otherwise when you change the code there is no way to know how the api was before or which ones are the fresh code smells.
In the case of the default-detekt-config.yml we don't care that much about how it was before. We don't want to compare how it was vs how it is in the current commit.

For example now you can easily look through the history of all the changes in the detekt repo that resulted in a configuration change.

You are right. We will lose some things. But I think that they are minor use cases. For example, in the past we also commit the documentation (the task that generates the default-detekt-config.yml is still called generateDocumentation for this fact). When we stopped we "lost" exactly the same but with the changes on the documentation.
TL;DR: Yes, we will lose some things but I think that the change will help us more than hurt. The idea behind this is to make the project easier to contribute. If we don't want this solution we could use danger to give advice when the workflow verify-generated-config-file fails.


[FLINK-8079][tests] Stop end-to-end test execution after first failure

What is the purpose of the change
This PR modifies the travis maven script to stop the execution of end-to-end tests once the first failure was detected.

I like the change+1
Minor: Is there a way to write if code != 0 return in bash? Might be easier to read...


Possible race condition in documentation deploy workflow

Describe the bug
Since the github actions jobs run in parallel there might be a case of two git commits trying to be pushed "at the same time"
I suggest moving the multiple deploy stages from the separate doc build jobs into a separate deploy job that depends on the successful builds of all docs. Could potentially do something fancy by deploying to separate "local" git branches and then merging them on the deploy branch.
To Reproduce
See https://github.com/nasa/cFS/pull/248/checks?check_run_id=2406018793
Expected behavior
All documentation can be deployed
Additional context
Add any other context about the problem here.
Reporter Info
Full name and company/organization if applicable

Note #234 made it such that the action wasn't running twice, but there are still two deploys from within the document action.  As suggested above, moving the deployment from the two jobs into a deployment job at the end should solve.


Convert unstructured document to conversation format

See https://github.com/Glavin001/data-maker-llm


Fix aspnet/Mvc#6296 sanitize class and namespace

The new @namespace directive isn't sanitizing class and namespace names
when generating them. This means that a file-system-legal character like
'-' will show up in a class name, and that's not good.
Note that the old code paths (document classifiers) already had tests for
this and did it properly. It was only missing from @namespace.

🚢 :it:


Cannot Unencrypt Wallet after Ubuntu Backup on Windows

Describe the issue
I am able to unlock, stake, and send qtum from my wallet on a linux computer. When I export the wallet.dat file to my windows computer, I am unable to unencrypt the wallet with the same password. I am able to see the correct account balances and transactions.
Can you reliably reproduce the issue? Yes
If so, please list the steps to reproduce below:

Back up wallet on Ubuntu computer
Replace wallet.dat file on windows computer with the Ubuntu wallet.dat file
Start qtum on windows computer, unable to unencrypt wallet with same password

Expected behaviour
I expect to be able to unlock the same wallet on my windows computer to make transactions.
Actual behaviour
I am able to unlock the wallet on my linux (origin) computer but not on my windows computer.

Hello, we haven't been able to reproduce this issue, please make sure there are no caps locked on the windows computer.

I got it to work after using the salvagewallet instructions.


Client Allocation Request for: IPFS Found

Client Allocation Questions
Core Information

Name: IPFS Found
Website / Social Media: https://filecoin.in/
DataCap Requested: 100GiB
Addresses to be Notarized: f1s6s7yfxg3gpj5t7hybuf4aug4ugk742bdmoka6q
Notary Requested: NeoGe-IPFSMain

Hey @IPFSUnion, thanks for the request. I have some questions regarding your request. Please describe answers as much specificity as you can.

Can you introduce yourself? Where can I find you online? (Github, Twitter, Linkedin handle, Wechat, etc.)
If you represent an organization, what is the name, nature, registration country, time since inception, and named officers of your organization?
In which region(s) do you intend to use the DataCap?
What is your use case for the DataCap being requested?
Do you plan to specify advanced features when utilizing a DataCap in your storage deal? If so, please specify the feature.
How many miners do you plan on allocating the DataCap? If you have a specific miner(s) you would like to allocate a DataCap, please specify.
Do you agree on providing transparency of DataCap allocation decisions & execution details in public to prevent excessive/unfair self-dealing?

Before starting to answer questions, I encourage you to take a look of our allocation strategy here which shows how we utilize the assessment.
Thanks.

Hey @IPFSUnion, thanks for the request. I have some questions regarding your request. Please describe answers as much specificity as you can.

Can you introduce yourself? Where can I find you online? (Github, Twitter, Linkedin handle, Wechat, etc.)
If you represent an organization, what is the name, nature, registration country, time since inception, and named officers of your organization?
In which region(s) do you intend to use the DataCap?
What is your use case for the DataCap being requested?
Do you plan to specify advanced features when utilizing a DataCap in your storage deal? If so, please specify the feature.
How many miners do you plan on allocating the DataCap? If you have a specific miner(s) you would like to allocate a DataCap, please specify.
Do you agree on providing transparency of DataCap allocation decisions & execution details in public to prevent excessive/unfair self-dealing?

Before starting to answer questions, I encourage you to take a look of our allocation strategy here which shows how we utilize the assessment.
Thanks.

Thank @NeoGe-IPFSMain for your review,
Can you introduce yourself? Where can I find you online? (Github, Twitter, Linkedin handle, Wechat, etc.)
IPFS Found  is developed by the IPFSUNION team, Based on Filecoin network, IPFS Found is designed as a storage and retrieval to help users store various types of files, including text, picture, audio, video, etc. And it supports storage of multiple data sizes. We aim to build more decentralized, stable and robust data storage platform.

About IPFSUNION:
Official website：https://ipfsunion.cn/
Self-developed Filecoin browser：https://filscout.io
Twitter: https://twitter.com/ipfsunion_cn
Github：https://github.com/IPFSUnion
Zhihu：https://zhuanlan.zhihu.com/p/173753718

If you represent an organization, what is the name, nature, registration country, time since inception, and named officers of your organization?
IPFSUnion - Focus on Internet IPFS distributed storage and Filecoin technology research and development and scenario application solutions。
IPFSUnion is registered in the Shanghai, China, the specific information can beem seen in the QCC.com
In which region(s) do you intend to use the DataCap?
We will search the availiable miners from mainet.  The IPFS Found base on the China.
What is your use case for the DataCap being requested?
IPFS Found is always storing the real data from curated data to the Filecoin network during the Slingshot Competition , it has stored the following datasets:
https://proofs.filecoin.io/
https://trusted-setup.filecoin.io/
https://ghtorrent.org/
https://storage.googleapis.com/openimages/web/index.html
https://www.kaggle.com/allen-institute-for-ai/CORD-19-research-challenge
https://www.kaggle.com/paultimothymooney/chest-xray-pneumonia
https://www.kaggle.com/borismarjanovic/price-volume-data-for-all-us-stocks-etfs
http://hlmxy.file123.pro:9006
https://ghtorrent.org/downloads.html

For now  we are storing the  “Landsat 8” -   it is an ongoing collection of satellite imagery of all land on Earth produced by the Landsat 8 satellite.
In the future, we hope to help people store the the private data.
Do you plan to specify advanced features when utilizing a DataCap in your storage deal? If so, please specify the feature.
IPFS Found plan to send data to the online miners for the small data. And when the miners support the offline deal, We will use it as a priority.
How many miners do you plan on allocating the DataCap? If you have a specific miner(s) you would like to allocate a DataCap, please specify.

IPFS Found will store the data from curated dataset.

And  the dataCap will be assigned to different miners, less than 20 percents datacap will be assigned to my own miners, one file will be stored in 10 copies for different miners to ensure data redundancy and decentralized.(we have 5 stable official miners f023530, f01231, f016398, f021479, f070932 f0131822, f0133505 with total 76.51 PiB power until now.)

For now, I have been looking for more large and stable miners.

Do you agree on providing transparency of DataCap allocation decisions & execution details in public to prevent excessive/unfair self-dealing?
Yes, each posted transaction info will include:
deal_id,
miner_id,
dataCID,
address,
deal_status,
file_name,
file_format,
file_size,
date,
dataset_from_source

Thank @NeoGe-IPFSMain for your review,
Can you introduce yourself? Where can I find you online? (Github, Twitter, Linkedin handle, Wechat, etc.)
IPFS Found  is developed by the IPFSUNION team, Based on Filecoin network, IPFS Found is designed as a storage and retrieval to help users store various types of files, including text, picture, audio, video, etc. And it supports storage of multiple data sizes. We aim to build more decentralized, stable and robust data storage platform.

About IPFSUNION:
Official website：https://ipfsunion.cn/
Self-developed Filecoin browser：https://filscout.io
Twitter: https://twitter.com/ipfsunion_cn
Github：https://github.com/IPFSUnion
Zhihu：https://zhuanlan.zhihu.com/p/173753718

If you represent an organization, what is the name, nature, registration country, time since inception, and named officers of your organization?
IPFSUnion - Focus on Internet IPFS distributed storage and Filecoin technology research and development and scenario application solutions。
IPFSUnion is registered in the Shanghai, China, the specific information can beem seen in the QCC.com
In which region(s) do you intend to use the DataCap?
We will search the availiable miners from mainet.  The IPFS Found base on the China.
What is your use case for the DataCap being requested?
IPFS Found is always storing the real data from curated data to the Filecoin network during the Slingshot Competition , it has stored the following datasets:
https://proofs.filecoin.io/
https://trusted-setup.filecoin.io/
https://ghtorrent.org/
https://storage.googleapis.com/openimages/web/index.html
https://www.kaggle.com/allen-institute-for-ai/CORD-19-research-challenge
https://www.kaggle.com/paultimothymooney/chest-xray-pneumonia
https://www.kaggle.com/borismarjanovic/price-volume-data-for-all-us-stocks-etfs
http://hlmxy.file123.pro:9006
https://ghtorrent.org/downloads.html

For now  we are storing the  “Landsat 8” -   it is an ongoing collection of satellite imagery of all land on Earth produced by the Landsat 8 satellite.
In the future, we hope to help people store the the private data.
Do you plan to specify advanced features when utilizing a DataCap in your storage deal? If so, please specify the feature.
IPFS Found plan to send data to the online miners for the small data. And when the miners support the offline deal, We will use it as a priority.
How many miners do you plan on allocating the DataCap? If you have a specific miner(s) you would like to allocate a DataCap, please specify.

IPFS Found will store the data from curated dataset.

And  the dataCap will be assigned to different miners, less than 20 percents datacap will be assigned to my own miners, one file will be stored in 10 copies for different miners to ensure data redundancy and decentralized.(we have 5 stable official miners f023530, f01231, f016398, f021479, f070932 f0131822, f0133505 with total 76.51 PiB power until now.)

For now, I have been looking for more large and stable miners.

Do you agree on providing transparency of DataCap allocation decisions & execution details in public to prevent excessive/unfair self-dealing?
Yes, each posted transaction info will include:
deal_id,
miner_id,
dataCID,
address,
deal_status,
file_name,
file_format,
file_size,
date,
dataset_from_source

Hey @IPFSUnion, sorry for the delays during the holiday season. Future requests would be processed at a faster pace. My allocation strategy is to build trust progressively. So I could allocate 100 GiB to you for the first time. Feel free to request more DataCap when you use up the allocation.
To better understand how future allocation looks, please take a look here for my allocation process.

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzacec5zkayvselijlppwzh5l5x2qqvbhhydja5rsn5gzjwqcztuslg2o

Address

f1s6s7yfxg3gpj5t7hybuf4aug4ugk742bdmoka6q

Datacap Allocated

107 GiB

You can check the status of the message here: https://filfox.info/en/message/bafy2bzacec5zkayvselijlppwzh5l5x2qqvbhhydja5rsn5gzjwqcztuslg2o


Prerequisites for Artifactory & Jenkins migration/upgrade

This fixes an apparent syntax issue in a newer Jenkinsfile and attempts to refresh the support to test a different Artifactory URL via alternativeResolutionRepo in a gradle.properties copied (manually due to something else that broke) from the templates directory to the project root
Tested over at https://jenkins.nanoware.us/job/repatriation/job/terasologyengine/job/mbdev/ (building Nanoware/Terasology/mbdev) as infra migration / upgrades are sort of tricky to test normally. Plus the Nanoware org job line in the current Jenkins is actually empty, partly because moving from the fancy Cloudbees Jenkins lost us a plugin that let you set easy properties at the folder level - which allowed all Nanoware jobs to automatically swap to a different set of Artifactory registries :-(
Or rather, lol, it would be tested there, but apparently I've run up against an SSD quota within GCP since I have two 200 GB Jenkins disks and some smaller stuff (including the build agent for the current test) and apparently the default limit is 500 GB? Who knew. Submitted a request to increase and will also decrease the Jenkins disk as we haven't needed that much space for a long time at this point.
Odds are this PR will fail testing here, and I may have to merge it on the spot when I get a chance to try swapping DNS over to the new stuff to start validating and rebuilding all the things. I count it a breaking change not to the game but logistics-wise - some stuff is likely to blow up till everything is trickled out to everywhere.
I've largely kept https://github.com/MovingBlocks/Logistics/tree/repatriation updated with new infra config and docs, but there's a bit more to come

Got the new Jenkins online. Merging this to test


@BatchMapping annotated functions do not support ReactiveSecurityContext

ReactiveSecurityContextHolder.getContext().awaitSingle()

throws a NoSuchElementException in a @BatchMapper annotated method while it is accessible in @SchemaMapping annotated methods for the same query. Which means that there is no direct way to access the Principal in a @BatchMapper annotated method.
I would expect this security context to still be accessible within the BatchMapper.

@jurriaan this should work in the latest snapshot, if you want to give it a try, thanks!


TIL: "How to get the expiry date of an HTTPS certificate"

🚀 Deployed on https://66d35b539886f0547e2f8c12--alexwlchan.netlify.app


scope_manager not accessable via opentracing

In the opentracing API the scope manager is stored in the property _scope_manager and can be accessed as scope_manager.
Currently, the property _scope_manager is not set in the FlaskTracing class. This leads to issues with opentracing instrumentation that needs access to it, e.g. the boto3 part of the https://github.com/uber-common/opentracing-python-instrumentation. (When I have more time, I will try to add a minimal example where failures actually occur.)
I could imagine two fixes to this a) adding
@property
def _scope_manager(self):
    return self.tracer._scopemanager

or b) explicitly doing:
self._scope_manager = self.tracer.scope_manager

in the initializer.
Happy to write the PR for this, if you provide me your preferences :slightly_smiling_face:

I am surprised FlaskTracing extends opentracing.Trace, it seems unnecessary, since it allows access to the tracer via tracer() property. I would not recommend using FlaskTracing as a replacement for Tracer, especially since the real thing is available anyway.

This package is no longer being maintained, users should migrate to the opentelemetry-api package.
Closing.


Add CPAN to communities section

I don't think this is true as stated. As far as I can tell, eg https://libraries.io/search?order=desc&platforms=CPAN&sort=rank it's mostly Artistic 1.0 or GPL 1.0+. This is not a great example for others to learn from so I'd rather not feature it on the site.

@mlinksva No worries. Thanks for your time :)


Setup tests.

Summary

Add travis CI yml
Add travis ci build badge
Add coveralls integration for coverage badge
Add nyc, mocha, chai and ignore-styles, enzyme, jsdom & react-addons-test-utils npm packages
Add npm scripts cover and cover:report
Add test/setup.js to setup dom for enzyme.mount()
Add tests for number_widget.jsx

Todo for upstream maintainer:

Update README for urls to coveralls and travis ci to accurate badge numbers.

Badges are driving by
https://coveralls.io/github/adilfulara/handsome
https://travis-ci.org/adilfulara/handsome

Thanks @adilfulara, sorry this took so long for me to get round too

@davefp I forgot to do the changes suggested by @stephenyeargin.

I'll do it directly 👍


[Enhancement] disable pre-read when orc io coalesce read enabled

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Running time of following SQL can be cut down from 43s to 16s

select max(lo_orderkey) from lineorder_flat_1000_10_orc;

MySQL [emr_test.dla_scan]> select max(lo_orderkey) from lineorder_flat_1000_10_orc;
+------------------+
| max(lo_orderkey) |
+------------------+
|        600000000 |
+------------------+
1 row in set (43.88 sec)

MySQL [emr_test.dla_scan]> select max(lo_orderkey) from lineorder_flat_1000_10_orc;
+------------------+
| max(lo_orderkey) |
+------------------+
|        600000000 |
+------------------+
1 row in set (16.18 sec)

The reason is there are lot of excessive reads caused by empty stripe. Looking at following output of orc meta

there is a gap between stripes:

you can see stream 37 ends at 24818212 + 127 = 24818339
next strip starts at 24818991

And this gap actually is a  very small stripe, and following code will make excessive reads.
-        // If this stripe is small, probably other stripes are also small
-        // we combine those reads into one, and try to read several stripes in one shot.
-        if (scope == orc::InputStream::PrepareCacheScope::READ_FULL_STRIPE) {
-            length = std::min(_length - offset, cache_max_size);
-        }

the small strip is 127 bytes
but we will make it up to 8M bytes.

Another reason why it was slow, is because there are a lot of stripe whose size is actually around 8MB. So we have to read whole stripe up, but actually use a little bit. So for performance consideration, we disable pre-read stripe when we enable io coalesace read
  if (streamIORangesEnabled) {
      contents->stream->clearIORanges();
  } else {
      contents->stream->prepareCache(InputStream::PrepareCacheScope::READ_FULL_STRIPE, currentStripeInfo.offset(),
                                     stripeSize);
  }

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

Is IO range from different stripe still able to be merged

no. because we are not sure if we will read stripe or not. maybe it will be filtered by predicate. so there is over-read


Retain psc list option

https://companieshouse.atlassian.net/browse/IDVA3-1742

CI: No security warnings found


make tokio dependency optional

The h3 crate depends on tokio, but only uses it for the unbounded channel, which signals request completion. I would like to replace the use of tokio::sync::mpsc with a lighter alternative, or at least make it optional.
This PR adds an optional dependency on async-channel and makes tokio optional. Alternatively any other mpsc implementation or even a Arc<Mutex<(VecDeque, Waker)>> should be sufficient.

Thanks for sending us the PR!
Does the channel you suggested actually "weigh" less? Currently, h3 only depends on the sync feature of Tokio, so it's not compiling the runtime part. I would assume they are similar amounts of code, and so then would tend to lean toward depending on Tokio's because it's usage by other is extensive.

Does the channel you suggested actually "weigh" less?

Depends by what metric. In terms of code being downloaded from crate.io it's roughly 50 times lighter. In terms of performance, and compiled size I have no idea (but I can check...).
A motivation was to make it obvious that the crate is runtime independent and doesn't spawn tasks. With tokio as a dependency that is tricky to tell for most people, since some things in tokio require running inside a tokio runtime, while others don't. It's not the case here, but I almost decided not to use h3 because I didn't look close enough at first.

Currently, h3 only depends on the sync feature of Tokio, so it's not compiling the runtime part. I would assume they are similar amounts of code, and so then would tend to lean toward depending on Tokio's because it's usage by other is extensive.

Same with async-channel. That's the one async-std and smol use. That was my reasoning behind making it optional.
Maybe I should have used Arc<Mutex<(VecDeque, Option<Waker>)>>...

I did a quick benchmark that seemed reasonable for this case and futures-channel was about 70% faster than tokio. It's also a much smaller download and as frequently used (according to crates.io) as tokio. We already depend on the futures-rs  ecosystem

Benchmark code
```rust
use futures_util::StreamExt;
fn main() {
let w = noop_waker::noop_waker();
let mut cx = std::task::Context::from_waker(&w);
let cx = &mut cx;
#[cfg(feature = "async-channel")]
let (s, mut r) = async_channel::unbounded();
#[cfg(feature = "tokio")]
let (s, mut r) = tokio::sync::mpsc::unbounded_channel();
#[cfg(feature = "futures-channel")]
let (s, mut r) = futures_channel::mpsc::unbounded();

for i in 0..10_000_000u64 {
    #[cfg(feature = "async-channel")]
    let x = {
        let _ = r.poll_next_unpin(cx);
        s.try_send(i).unwrap();
        r.poll_next_unpin(cx)
    };
    #[cfg(feature = "tokio")]
    let x = {
        let _ = r.poll_recv(cx);
        s.send(i).unwrap();
        r.poll_recv(cx)
    };
    #[cfg(feature = "futures-channel")]
    let x = {
        let _ = r.poll_next_unpin(cx);
        s.unbounded_send(i).unwrap();
        r.poll_next_unpin(cx)
    };
    std::hint::black_box(x);
}

}

</details>

Performance characteristics for channels really depends on usage. They all chose different pros and cons, such as using a linked list versus arrays and blocks, instrusive collections, locks vs spinning, preferring allocating in send or recv or reserve, etc. Depending on how contended they'll be, how big the message type is, how often an item is sent, etc.
For that reason, I tend to find Tokio's channel preferable because it sees more usage, and there are several active maintainers available to fix any bugs that are reported.

A motivation was to make it obvious that the crate is runtime independent and doesn't spawn tasks. With tokio as a dependency that is tricky to tell for most people, since some things in tokio require running inside a tokio runtime, while others don't. It's not the case here, but I almost decided not to use h3 because I didn't look close enough at first.

Independent of the channel, I think it is a good idea to address in the readme that h3 is runtime independent. I will submit a PR for this.

Independent of the channel, I think it is a good idea to address in the readme that h3 is runtime independent. I will submit a PR for this.

Thx, that helps.

Performance characteristics for channels really depends on usage.

That's why the benchmark looks the way it does. I suspect the channel signalling request ends will pretty much always be empty and at least half off the polls result in Poll::Pending.

I tend to find Tokio's channel preferable because it sees more usage, and there are several active maintainers available to fix any bugs that are reported.

I would have assumed that anything in futures-rs project is quite actively maintained as well and the recent download numbers on crates.io are practically identical. But as a contributor to both of them, you are obviously more qualified to comment on that.
To be clear, I only sent this PR because of the size and the "looks like depends on tokio runtime" concern, not performance. I just wanted to alleviate potential performance concerns with the benchmark. I personally would still prefer the smaller dependency, but it is pretty subjective, so feel free to close. I'm happy to send a PR for any variant mentioned in the thread otherwise.


Warn when programs.zsh is enabled system-wide

Whenever you have zsh plugins in home-manager and programs.zsh enabled system-wide, you run into a cache-invalidation issue that makes the shell start dramatically slower. I lost my benchmark results but they were all in the 1.6-2.0s range.
Disabling programs.zsh in the global config makes things work beautifully:
❯ for i in $(seq 1 10); do time zsh -i -c exit; done
zsh -i -c exit  0.06s user 0.03s system 103% cpu 0.079 total
zsh -i -c exit  0.06s user 0.02s system 104% cpu 0.078 total
zsh -i -c exit  0.05s user 0.03s system 103% cpu 0.078 total
zsh -i -c exit  0.06s user 0.02s system 103% cpu 0.079 total
zsh -i -c exit  0.06s user 0.02s system 103% cpu 0.079 total
zsh -i -c exit  0.05s user 0.03s system 103% cpu 0.079 total
zsh -i -c exit  0.06s user 0.03s system 103% cpu 0.081 total
zsh -i -c exit  0.06s user 0.02s system 103% cpu 0.083 total
zsh -i -c exit  0.06s user 0.02s system 103% cpu 0.081 total
zsh -i -c exit  0.05s user 0.03s system 103% cpu 0.081 total

I think solving this proper is a hard problem, and was previously discussed in #108, so in lieu of fixing it proper I wonder whether it's something we could warn against at generation-time?

CC. @uvNikita who seems to have his hand in zsh stuff

Just wanted to chime in to say that HM cannot reliably tell whether a configuration is being generated for a NixOS system or some other Linux system. Maybe it might be possible to detect this issue and print a warning at activation time?

Source /etc/os-release and [ "$ID" == "nixos" ] ?

To add to @peterhoeg's proposal there is also /etc/os-release which for NixOS looks like this:
NAME=NixOS
ID=nixos
VERSION="20.03pre209091.100012e55bc (Markhor)"
VERSION_CODENAME=markhor
VERSION_ID="20.03pre209091.100012e55bc"
PRETTY_NAME="NixOS 20.03 (Markhor)"
LOGO="nix-snowflake"
HOME_URL="https://nixos.org/"
DOCUMENTATION_URL="https://nixos.org/nixos/manual/index.html"
SUPPORT_URL="https://nixos.org/nixos/support.html"
BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"


Could you share your code?

I'm quite interested in your ICCV work. I wonder whether you prepare to release the code.

Apologies, I've been busy working open sourcing our NAFs code recently.
I've just uploaded the core of the differentiable ray casting code. This code includes the embree3 acceleration structure construction, differentiable geometric functions, the differentiable blending code (depth/silhouette), triangle expansion code, and a small usage example (written a while ago).
This code enables a gradient to flow to vertices. To enable gradients from vertices to flow to SDF network parameters you need to tie this with MeshSDF's framework.
I'll upload a small example tying together with MeshSDF once the current busy period ends.
You can reach me at afluo[A.T.]andrew.cmu.edu


fix: update the PulsarTopic doc to remove the partition limitation.

(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
From the https://github.com/streamnative/pulsar-resources-operator/pull/185, we supported updating topic partitions in the PulsarTopic object.
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

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

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

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

Documentation
Check the box below.
Need to update docs?

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

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

[ ] doc
(If this PR contains doc changes)

@labuladong Can you help take a look and approve this PR?


Make it transport in-dependent.

I think we should abstract out the transport layer details and make the transport details implementations in two adapters Reliable and Unreliable. This will work wonders for cases where you'd want to use WebRTC with SDP in UDP mode. Or in cases where you can make lower level UDP sockets to connect to the node instance directly.
Binding it to websocket adds TCP and HTTP overhead and has negative performance impact as TCP garuntees the packet order.
DTLS / SDP used in WebRTC present a great framework which can be used to communicate on non-reliable channels in browsers. Additionally, the webrtc identity architecture is a great way to use a 3rd party idp using standard protocols to authenticate via a separate context.

Would be nice to support WebRTC indeed.
In order to allow reliable/unreliable transport layer, the serialization of the game state also would need to be configurable and changed completely. Currently, data patches must come in the right order to the clients to be able to apply the modifications correctly.

Indeed that would be a significant change however for a start we can abstract and convert the incoming message mechanism so that you pass a reliable adapter to colyseus which then uses that as transport layer. That way the door will open for api authors to tap into the server.
Ideally for unreliable channel we'll need a lot more work on transport side but assuming a transport has just one job and that is to deliver data, the transport adapter can take responsibility of the fashion it chooses the transport data with.
Packet loss in TCP will result in higher ping and snowballing while the same in UDP will end up in corrupt state both are unplayable. We can rely on Source like interpolation https://developer.valvesoftware.com/wiki/Source_Multiplayer_Networking#Entity_interpolation which I believe (I have just quickly glanced over your linked paper not read it in-depth) is very similar to the timeline approach.

It would be nice to have serialization and a patch module interfaces. At the moment, I had to remove msgpack and fossil-delta-js from the project and use json patch since this option works on all platforms for me. I would also like to have UDP to transmit data that do not need patches.

@derwish-pro I really like that! Having a different serialization/patch modules would drastically change in the overall client/server communication. Would be awesome to have it though.

The simpler way to do that would be to break the project into different parts and maybe use an interface like node-streams. That way all of these transformations, state patches, etc can be implemented as a stream pipeline.
[just a suggestion]

Having a barebones TCP server instead of WebSocket would simplify a lot for the Game Maker Studio 2 client.

@endel Please support both TCP and UDP as we can use UDP for send moving of player for the smooth transition. Thanks!

@takaaptech I'd like to vouch a note of caution, simply supporting UDP won't make the game smooth, UDP is by definition a lossy protocol and will run into hiccups ideally you need some strategy to sync. I am more excited if we can support this, we can then use projects like valve/netcode instead.

BTW, FCM also makes sense in some cases

This issue confuses me because some tend to think of Colyseus as a netcode library rather than a game engine as Colyseus was first defined as. This requires two different approaches; if a netcode library you need to focus most on a stable network API (Server class), where if a game engine you need to focus most on a stable framework for serving games (MatchMaker, Room, Presence, etc). You COULD split the two, creating a Colyseus io framework and Colyseus Game Franework. Other transports can then use Colyseus Game Framework by leveraging Matchmaker. For example, a Boost.Asio based server over Redis.

Hope this helps, seems like a pretty clean interface on top of WebRTC: https://github.com/Efoi/WebRTC-Server-Client-Datachannel

Closing as the structures for supporting other transports is there, although only ws and uWebSockets.js are supported atm.


Fix python version build warnings

This is to fix the following warning we've been getting in the build pipeline:
"Specifying an exact version is not recommended on Microsoft-Hosted agents. Patch versions of Python can be replaced by new ones on Hosted agents without notice, which will cause builds to fail unexpectedly. It is recommended to specify only major or major and minor version (Example: 3 or 3.9)"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 41.757%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/book/bookTreeView.ts
1
36.91%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
30827

Relevant Lines:
69082

💛  - Coveralls


ggplot histogram query change

Describe your changes
Now it mimics R ggplot
Issue number
Closes #751
Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://jupysql--754.org.readthedocs.build/en/754/

ggplot_hist_check.zip
I checked the shape of histograms in the test files with ggplot in R, and they match.


Instructions for Ubuntu?

I just installed the script and it seems to have installed without issues. Also, I have the service started:
➜ journalctl --user-unit figma-agent.service
Dec 22 01:23:42 wonderland systemd[2093]: Starting figma-agent.service - Figma Agent for Linux...
Dec 22 01:23:42 wonderland systemd[2093]: Started figma-agent.service - Figma Agent for Linux.
Dec 22 01:23:42 wonderland figma-agent[361211]: [2023-12-21T19:53:42Z INFO  actix_server::builder] Starting 1 workers
Dec 22 01:23:42 wonderland figma-agent[361211]: [2023-12-21T19:53:42Z INFO  actix_server::server] Actix runtime found; starting in Actix runtime

I was expecting to find the installed fonts in Ubuntu show up in search in Figma's browser version in Ubuntu. Am I misunderstanding something? Is there anything more I should do to get this working?

I had the same issue, I checked the network tab, and see if it's blocked

And it turns out it's actually my Brave Shield blocking the request:

If it still doesn't work, try this command to see if the server is still responding:
curl --header "Origin: https://www.figma.com" "http://<IP_ADDRESS>:18412/figma/font-files"


MCP - URL Asset Import date columns not working as intended

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM 6.4 SP3
[ ] ACS AEM Commons Version: 3.19
[ ] Reproducible on Latest? yes

Expected Behavior
Header column with the @date should store as a date in the metadata.
Actual Behavior
if you have a @date and the content is like 2016-02-12T14:47:41.922-05:00
the metadata isn't imported at all.
if you set the cell to be a date in excel, and have an actual date, the date is being store as a binary!
Steps to Reproduce
Simply include a column as a date with data in it. such as:
dc:modified@date
value
2016-02-12T14:47:41.922-05:00 (this is skipped)
1980-03-14 (this is imported as a binary instead of date)

I see this issue as well.

It's not actually a bug, it's a feature request.  Reason I say this is that Excel doesn't actually recognize your DB date format as anything other than General.  If it were to format the cell as a date then this wouldn't actually be an issue.  I'll see if I can detect DB date formats since they're likely common all the same.

augh, ok so it's both a bug (date should be calendar objects, easy fix) and a feature request (add support for DB date format)

Not sure why I named that branch incorrectly, but it fixes this issue.  Sorry for the confusion.  Let me know if this works for you folks.  There's also now a unit test for the data importer tool.

@badvision can you provide the fix version for this?

It was fixed in 4.1.0