Add AAA ESG

Interest Rates
What needs to be done for interest rate scenarios:

[ ] Expose easier way to use constrained NelsonSiegel curves from Yields.jl
[ ] Port logic from AAA sample notebook in Learn repository
[ ] Should return a yield curve for each generated scenario/timestep.

Equity
What needs to be done for equity scenarios:

[ ] Get working prototype of single fund (see https://github.com/JuliaActuary/Learn/blob/master/AAA_Equity_Generator.jl)
[ ] Extend prototype of multi-fund (should be as simple as re-using the Z variates and using different parameters
[ ] Implement multi-fund version into EconomicScenarioGenerators.jl

Reference: https://www.actuary.org/sites/default/files/pdf/life/c3supp_march05.pdf

Hi @alecloudenback,
cc @gchen3
I just sent you a long email, but didn't check this issue first.
I don't think the 2nd equity bullet above is correct. That would use the same return shock and volatility shock for each asset class, but the AIRG model uses one set of correlated return and volatility shocks for the Diversified Fund (one pair of shocks for each month for each of 100 years for each scenario as I read it), another (different) set of correlated shocks for the International Fund, and so on.
So I don't think you would have similar results to those they published if you reused the Z variates.

Thanks for the feedback, I edited to say more clearly what I intended.
I updated the notebook to have all four equity funds and it validates against Table 8 mean/std dev in the PDF.
https://github.com/JuliaActuary/Learn/commits/master/AAA_Equity_Generator.jl


Annotate commits by path

Is your feature request related to a problem? Please describe.
We are generating a centralized changelog for different sub-projects within a monorepo. At present, we're using --include-path to generate N separate changelogs.
However, a given commit can span multiple sub-projects, and the total "changelog" produced by unioning each individual path's result can be long and contain a lot of duplication.
I can see a few different ways of enabling the above workflow to produce a result that's much more compact.
Describe the solution you'd like

The ability to annotate "named paths" and have that be accessible inside the "context".
[named-paths]
"lib/foo" = "Foo"
"lib/bar" = "Bar"
"service/Baz" = "BaZ"

producing a "named-paths": ["Foo", "Bar", "Baz"] in the context, enabling one to customize the template to render those paths alongside each commit line-item in their respective sections

The ability to execute git-cliff against externally deserialized context
git-cliff --include-path lib/foo --output foo.json
git-cliff --include-path lib/bar --output bar.json
# jq magic that deduplicates commits and annotates them in some way that does something like the above
git-cliff --from-context combined-context.json

The ability to --include-paths multiple times (supported already), but expose the (full set of) matched paths to the context, for similar reasons to option 1.

The ability to execute git-cliff against externally deserialized context

This is now possible via --from-context, see https://git-cliff.org/docs/usage/load-context


[MRG] FIX:  path regex on windows

@Eric89GXL waiting to see if CIs are fixed...

Current coverage is 85.58% (diff: 0.00%)

No coverage report found for master at 4f948fa.

Powered by Codecov. Last update 4f948fa...ec7d688

CIs look happy, and I suspect that's the correct fix. Is it easy for you to test locally that it doesn't break Unix systems if there is a \ in the path?

I haven't tested it, can do that, but I first put it into a conditional for
windows and then realized that if you had a backslash in a Linux path you
would also need I escape it in the pattern, this just never came up because
nobody puts backslashes into paths.
On Saturday, January 28, 2017, Eric Larson<EMAIL_ADDRESS>wrote:

CIs look happy, and I suspect that's the correct fix. Is it easy for you
to test locally that it doesn't break Unix systems if there is a \ in the
path?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/3942#issuecomment-275888050,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI5a5R7Y7zIvP-XwBrZAjwBf3z_MPQmks5rW-_rgaJpZM4Lwl07
.

$ mkdir a\\a
$ ls
a\a
$ ipython
Python 2.7.13 |Continuum Analytics, Inc.| (default, Dec 20 2016, 23:05:08)
Type "copyright", "credits" or "license" for more information.

IPython 5.0.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: import re

In [2]: import os

In [3]: print [p for p in os.listdir('.') if re.match('a\\a', p)]
[]

In [4]: print [p for p in os.listdir('.') if re.match('a\\\\a', p)]
['a\\a']

LGTM
@Eric89GXL merge if it works for you

Thanks @christianbrodbeck


TCN and HybridNet are outputs 3 dim and not 2

My proposal is to transform both models to module, removing the EEGMixing abstraction


Failed to Sync - Error: Replica AdGuard Home version must be >= v0.107.0

Describe the bug
A clear and concise description of what the bug is.
I am running an instance of adguard home on a raspberry pi and trying to sync that instance to 2 other replicas. All versions are set to the latest release. When I first ran the sync, I got errors relating to redirects - I found the answer to that in issue #59 . Once I added that env variable, I now am faced with an issue where it looks like it fails to detect the version to initialize the sync...after it detects the version
To Reproduce
Steps to reproduce the behavior:

What version of AdGuardHome sync used?

Docker - Latest

What version of AdGuardHome us used?

v0.107.8

How does the configuration look?
What is the error message? (I changed all of the IP's in the log to "x.x.x.x")
2022-08-03T03:40:05.953-0700    INFO    sync    sync/sync.go:191        Start sync      {"from": "x.x.x.x", "to": "x.x.x.x"} 2022-08-03T03:40:05.964-0700    INFO    sync    sync/sync.go:199        Connected to replica    {"from": "x.x.x.x", "to": "x.x.x.x", "version": "v0.107.8"} 2022-08-03T03:40:05.964-0700    ERROR   sync    sync/sync.go:202        Replica AdGuard Home version must be >= v0.107.0        {"from": "x.x.x.x", "to": "x.x.x.x", "error": null, "version": ""} github.com/bakito/adguardhome-sync/pkg/sync.(*worker).syncTo      /home/bakito/go/src/github.com/bakito/adguardhome-sync/pkg/sync/sync.go:202 github.com/bakito/adguardhome-sync/pkg/sync.(*worker).sync      /home/bakito/go/src/github.com/bakito/adguardhome-sync/pkg/sync/sync.go:179 github.com/bakito/adguardhome-sync/pkg/sync.Sync.func2      /home/bakito/go/src/github.com/bakito/adguardhome-sync/pkg/sync/sync.go:44 github.com/robfig/cron/v3.FuncJob.Run      /home/bakito/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:136 github.com/robfig/cron/v3.(*Cron).startJob.func1      /home/bakito/go/pkg/mod/github.com/robfig/cron/v3@v3.0.1/cron.go:312

Expected behavior
A clear and concise description of what you expected to happen.

To sync as expected as I am running the latest for Adguard-home and Adguardhome-sync docker containers with minimal configurations

Found the answer myself - my adguard replicas had the admin interface set to a custom port. I needed to add that port to my replica's url in my adguardhome-sync config file. Once I changed that, the sync went as expected


Add dollar symbol prepared statement notes for Python API

This PR comes in the spirit of stepping towards addressing #429 with what I understand so far about how this syntax operates for DuckDB API's. Something I wasn't sure about was whether prepared statements work for all API's or only some. In the meantime I thought it might be important to cover that it is supported by the Python API.
Thanks in advance for any thoughts or feedback you may have!

Thanks very much! This is great. Love the duck duck goose reference...

Cheers, thank you @Alex-Monahan !


Is it possible to change the size of an infocard?

I would like to make the info-cards smaller is it possible to do so?

+1

+1


Add custom strategies under FindBy annotation

Currently to use, for example, class chain strategy for iOS we need to add additional annotation to the element @ClassChain and also keep FindBy annotation with random strategy (FindBy(xpath = ".....)).
Lets investigate ability to add different strategies directly to FindBy annotation, so the result will be like FindBy(classChain = "**/XCUITypeOther")

Ready for review https://github.com/qaprosoft/carina/pull/657

Already merged to carina 6.0.12


Error with code.

The code for the app is too old, not updated with the latest flutter version and plugins. After updating there is too many errors. Please fix it.

Please add null check to code, otherwise has too meny errors


About the func evaluate_on_gpu,

when I run the func evaluate_on_gpu, why it is soooooooooo slow???

How's your speed? What's your running env? Have you checked the output of nvidia-smi when running?

I find it is because the funtion:gpu_nms(), I have solved this problem, but I can't understand it.
Here is the code in your respority:
pred_boxes, pred_confs, pred_labels = sess.run(gpu_nms_op,
feed_dict={pred_boxes_flag: pred_boxes,
pred_scores_flag: pred_confs * pred_probs})
and in the train.py, you feed the data for gpu_nms_op.
I have changed the code as follow:
pred_boxes, pred_confs, pred_labels = nms_utils.nms(sess, pred_boxes, pred_confs*pred_probs, num_classes, score_thresh=score_thresh, iou_thresh=iou_thresh)
and in gpu_nms() func, I use boxes_, score_, label_ = sess.run([boxes, score, label]) to get the array variables: boxes_, score_, label_. Use this method, the code will run very slow.
So why it run faster when the gpu_nms_op() is defined in train.py using feed_dict,
why it run sooooo slow when I give a sess to the gpu_nms() to get variables(should return array not a tensor)

Because TensorFlow is graph based. Everything like setting up the ops should be finished before actual running. Once starting the session, you should not add more ops as it will pause to update the graph and it's slow! And it may cause OOM problems as the ops are added in every step. That's why I define a gpu_nms_op before the session.

session
Thanks for your answer! have you ever train VOC data by your model?

Not yet. Because I am really busy these days and have no time to train it on VOC dataset. If you do that, please add data augmentation and other tricks in the readme file.

@rainley123 I have made quick tests on the VOC dataset. Performance is good. Refer to comments here: https://github.com/wizyoung/YOLOv3_TensorFlow/pull/93


Support for different CFA patterns

Thanks for the nice work. It looks like the assumption of a standard 2x2 Bayer filter is assumed. Are there any plans to relax that assumption to work with other configurations such as Quad-CFA?
Some mobile phones (e.g. Xiaomi Mi 10 Ultra) are now equipped with such sensors.
Single samples (no burst) can be found here:
https://github.com/darktable-org/rawspeed/issues/256
https://drive.google.com/drive/folders/1EPZEpvBHupHehsYI600f9cS7m_MaVhSQ?usp=sharing

Hi and thank you for your interest in the project. Adding other array configurations is not planned because I think it would make the code less readable and would diverge too much from the paper.
However it is totally do-able and I encourage you to propose a fork if you feel like it may be useful. The key to make it work are :

In merge.py, one should change the 2 in local_CFA[pixel_idy%2, pixel_idx%2] by the size of the pattern (4 for quad CFA).
The function compute_grey_image in utils_image.py should still work when using the FFT because it does not uses the pattern, but I have no idea of how good the obtained grey image will look. In decimation mode, the way to obtain a grey image must be modified to work with the new pattern
compute_guide_image in robustness.py should be adapted. We currently get the rgb image by decimating bayer quads and averaging the greens, but with a different pattern it should be different.

In the end, it all boils down to figuring out how to decimate a raw image to grey or to rgb with the new pattern. The easiest may be nearest neighbour intepolation for each color channel separately, then averaging to grey.

Many thanks for the fast response. As there are no current plans for it, I'll close this issue but would re-open it if/when we have any progress to share on our side.


Update OV

Update OV to the last commit on May 30, 2021

I think this PR would not be well-served by squashing; I think we want to retain the separate upstream commits unsquashed. To that end, let's not land this directly. I plan to update the plaidml branch to the HEAD of this branch and then push to this repo to get this update landed (probably tomorrow).

I think this PR would not be well-served by squashing; I think we want to retain the separate upstream commits unsquashed. To that end, let's not land this directly. I plan to update the plaidml branch to the HEAD of this branch and then push to this repo to get this update landed (probably tomorrow).

Yes, by merging and squash, all the commits merged into one commit. You can rebase to the latest commit included in this update. https://github.com/openvinotoolkit/openvino/commit/7fadfa88d79179273062b3e9c6969ecc6ac1a28e

Ok, thanks for the update. Let's get at least one of those fixes into this PR before merging. If I understand correctly, the more complete fix doesn't particularly affect us? If that's correct there's no need to wait for it IMO.
Also, no need to rebase, the current commit structure you have seems reasonable to me.

Ok, thanks for the update. Let's get at least one of those fixes into this PR before merging. If I understand correctly, the more complete fix doesn't particularly affect us? If that's correct there's no need to wait for it IMO.
Also, no need to rebase, the current commit structure you have seems reasonable to me.

I have updated this branch to https://github.com/openvinotoolkit/openvino/commit/5c716d2afc411c69e8bca9ec82a1290aec245c25, which includes Xin's fix.
For your question, Xin's fix was based on the code before update. In order to pass  this activation tests after updated OV, probably we will need to include the complete fix . Since that test is commented out and that complete fix seems to still in review state, I think we do not need to wait for it.

Merged as discussed above, thanks!


"Authentication Administrator" and "Privileged Authentication Administrator" needs to be updated

Per MC178555, these roles are now able to reset user passwords. Please update to reflect this change.

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Administrator role descriptions and permissions - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/directory-assign-admin-roles.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@UKayo Thanks for reporting this. I have assigned this to the author to update the new changes accordingly.


Plugin cannot be installed

Hi @willasm
We have noticed that the latest release of your plugin cannot be installed currently.
This is due to the following reason:

No release found for version 1.7.0

We will remove this plugin from the community plugins if this issue is not fixed within 30 days.

The Obsidian Team

I was about to create new release then found a bug that needed to be fixed. All corrected now and new release has been created.
Thanks, William.


🐛 Documentation - Mobile version is unusable

On mobile version, the storybook documentation is unusable. Indeed, it is not possible to display another story than Primary Button.
Steps to reproduce

Open documentation
Click on sidebar to open the navigation menu
Select a story different than Primary Button
Click on canvas
💥 The canvas displays the Primary button story

Screen recording
https://github.com/openfun/cunningham/assets/9265241/cea91682-618a-4a46-b278-edc39f76a229

Hell, this seems to happen to lots of people and the fix is scheduled for v8 .. 😅
https://github.com/storybookjs/storybook/issues/23723#issuecomment-1729410044

It seems to be fixed now.


ui: student view now show both solo and group submissions for a given assignment

feat: add SubmissionsForUser class to manage user and group submissions
refactor: use new SubmissionForCourse class in state, actions
feat: enhance Lab component to differentiate between group and individual submissions
refactor: simplify ProgressBar component by removing assignmentIndex prop
feat: replace CourseLabs component with AssignmentCard for better submission handling
feat: update NavBarLabs to handle submissions for group and individual labs
style: add submission-icon styles for improved visual presentation
fix: correct user lookup in ManageSubmissionStatus component
refactor: simplify ProgressBar tests by removing assignmentIndex prop and using SubmissionsForUser helper
feat: update StudentPage to include route for group labs

Did a quick pass and it looks good. But could you take a look at the DeepSource issues; there are a few simple fixes. You don't have to fix all.


add OpenShift Hive 1.0.0

Signed-off-by: Greg Sheremeta<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[x] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[no] Links to the maintainer, source code and documentation
I didn't put links. There is only one thing called 'OpenShift Hive'. If I need to add a link somewhere, let me know.
[x] Example templates for all Custom Resource Definitions intended to be used
the User Guide has them. Is that good enough?
https://github.com/openshift/hive/blob/master/docs/using-hive.md
[no] A quadratic logo
waiting on a logo from graphic design

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

Hi @gregsheremeta, there is an error message namespaces "hive" not found.

@J0zi interesting, #1402 didn't have that?
Hive does need to be in the hive namespace. Do you have any advice for a fix here? When I was testing on OpenShift I did need to create the hive namespace manually. Is there a way to automate that?

@gregsheremeta, this is the issue. Namespace was not created.
Another thing which can be interesting:
containerStatuses:

containerID: cri-o://a09d36cdc5e88f6e45b9fd1cbfcefa64c28432500c3110b4d79d1de71d617b06
image: quay.io/openshift-hive/hive:v1.0.0
imageID: quay.io/openshift-hive/hive@sha256:25a343356881cd22fecdb7873ff88c4f28634f7a7bf1ab795a04aba6cdf3954e
lastState:
terminated:
containerID: cri-o://bec33017d309fdf209ce63967f8f90f7e73e2befa34aab20dc9ba43795a51c8f
exitCode: 1
finishedAt: "2020-03-26T20:58:52Z"
reason: Error
startedAt: "2020-03-26T20:58:52Z"
name: hive-operator
ready: true
restartCount: 2
started: true
state:
running:
startedAt: "2020-03-26T20:59:18Z"

@J0zi can you post or email me the log of the operator pod when it fails to start? I have not seen that before.

@J0zi fix for namespace thing is on the way :smile: hopefully i can update this by the end of the day

replaced with #1560


cordova-plugin-iosrtc/iosrtcPlugin.swift:620:25: Cannot convert value of type 'UIView' to expected argument type 'UIWebView'

The plugin won't compile for me with the error as shown above.
Here's the code generating the compilation error, with some additional context:
iosrtcPlugin.swift: 620
func new_MediaStreamRenderer(command: CDVInvokedUrlCommand) {
		NSLog("iosrtcPlugin#new_MediaStreamRenderer()")

		let id = command.argumentAtIndex(0) as! Int

		let pluginMediaStreamRenderer = PluginMediaStreamRenderer(
			webView: self.webView!, // ! Cannot convert value of type 'UIView', etc...
			eventListener: { (data: NSDictionary) -> Void in
				let result = CDVPluginResult(status: CDVCommandStatus_OK, messageAsDictionary: data as [NSObject : AnyObject])

For my own purposes I've added a cast:
webView: self.webView! as! UIWebView,

and it seems to compile. I'm not sure if this is the best solution in this case but it seems legitimate to me, however it's been a while since I've written anything in Swift.
The cause seems to be that CDVPlugin.h (the parent class) lists the property as a UIView * rather than a UIWebview *:
CDVPlugin.h: 57
@property (nonatomic, readonly, weak) UIView* webView;

Please check the readme, you need corodva-ios >= 4.

How do I check the current version of cordova-ios and update it? I must have confused it with cordova command line tool itself.

You can use npm ls.
Saúl
On Oct 31, 2016 09:04, "Gregory McQuillan"<EMAIL_ADDRESS>wrote:

How do I check the current version of cordova-ios and update it? I must
have confused it with cordova command line tool itself.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/eface2face/cordova-plugin-iosrtc/issues/220#issuecomment-257199073,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATYGBchD2msLvaz3Q8V_egO9q0ccAMWks5q5UyhgaJpZM4KjHdV
.


Add comments to AlarmCore.start

Please answer these questions before submitting pull request

Why submit this pull request?
Add annotation(AlarmCore.start)  for easy reading

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

Codecov Report

Merging #4703 into master will decrease coverage by 1.13%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4703      +/-   ##
============================================
- Coverage     51.39%   50.26%   -1.14%
+ Complexity     2668     2564     -104
============================================
  Files          1275     1274       -1
  Lines         27659    27615      -44
  Branches       3003     3017      +14
============================================
- Hits          14215    13880     -335
- Misses        12801    13094     +293
+ Partials        643      641       -2

Impacted Files
Coverage Δ
Complexity Δ

...king/oap/server/core/alarm/provider/AlarmCore.java
85.71% <ø> (ø)
10.00 <0.00> (ø)

...ing/oap/server/library/util/ProtoBufJsonUtils.java
0.00% <0.00%> (-90.00%)
0.00% <0.00%> (-2.00%)

...tworkalias/NetworkAddressAliasSetupDispatcher.java
11.11% <0.00%> (-88.89%)
1.00% <0.00%> (-1.00%)

...ler/v8/rest/ManagementServiceKeepAliveHandler.java
32.00% <0.00%> (-64.00%)
2.00% <0.00%> (-1.00%)

...rest/ManagementServiceReportPropertiesHandler.java
27.58% <0.00%> (-62.07%)
2.00% <0.00%> (-2.00%)

...er/core/server/auth/AuthenticationInterceptor.java
0.00% <0.00%> (-61.54%)
0.00% <0.00%> (-3.00%)

...r/storage/plugin/influxdb/query/TopologyQuery.java
3.50% <0.00%> (-59.65%)
2.00% <0.00%> (-12.00%)

...plugin/elasticsearch/query/TopologyQueryEsDAO.java
1.50% <0.00%> (-57.15%)
1.00% <0.00%> (-9.00%)

...lysis/manual/networkalias/NetworkAddressAlias.java
2.43% <0.00%> (-56.10%)
0.00% <0.00%> (-5.00%)

...p/server/core/source/NetworkAddressAliasSetup.java
33.33% <0.00%> (-50.01%)
2.00% <0.00%> (-1.00%)

... and 36 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 15f1e04...b1cbcfe. Read the comment docs.

Upgrade nacos version to 1.2.0

If you are planning to do something else, please open another PR with clean commit log.


Allow files to be viewed from amazon s3 or with query string

I have added clean up code in order to get rid of query string. It will allow user to put the s3 urls.

Hi @Suhoij, sorry for ignoring your patch so long. ViewerJS 0.5.5 has been released today, and it comes with the commits from https://github.com/kogmbh/WebODF/pull/859 , which should help a lot finding/setting the right content type. It also picks up your suggestion in the fallback to guess the type from the path.
So closing this PR as superseded now, thanks for the proposal.


Avoid duplication of task execution

Fixed a bug that causes second execution of task in a same minute

Closes #296.


[Backport release-24.11] dxvk_2: 2.4 -> 2.5

https://github.com/doitsujin/dxvk/releases/tag/v2.5 (cherry picked from commit e0d0c3932b807d332666600264dc165979b49213)
Backport failed in https://github.com/NixOS/nixpkgs/pull/355274#issuecomment-2485891991, but the branch was created - so opening the PR manually.
@reckenrode

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Once eval completes, this can be merged. There’s also a follow up for DXVK 2.5.1, which has some needed bug fixes.


[Manual Merge Request] Updating token meta data

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

Need help in merging my PR (https://github.com/solana-labs/token-list/pull/18807) manually, as I'm getting auto-merge error.
PR Change: Updated token metadata and added extension data.
CC: @keone
-Thank You.

merged


Allow reqwest to use rustls

The build fails when targeting environments that do not have openssl (AWS lambda) - fix should be as simple as adding the default-features=false to the reqwest dependencies.

Hey @fdoyon ,
To be honest I am not as familiar with what setting default-features=false does but I will look into it. Feel free to let me know if you know more about it. I assume it has to do with switching what tls library is used?
I am open to adding that to the dependencies for reqwest. I just want to make sure it doesnt cause any unintended side effects. I will get back to you soon on this hopefully. Sorry for the delay.

Looking at it further I think it would be adding default-features=false and adding someting like rustls-tls since without default-features you cannot call https requests from what I can tell (only http).

Hi, thanks for looking into it.
When you drag reqwest, by default, it drags the native-tls crate which is calling openssl-sys on most platforms.
If you want to allow your clients to choose whether they want native-tls or rustls I think you might want to do the  following:

Define your dependencies on rustls with the minimum amount of features
create a nativetls feature on your crate, that then drags reqwest.nativetls
create a rustls-ts feature on your crate, that then drags reqest.rustls-tls
create a default feature that contains all the default features of your crate (and that may include nativetls so as not to break the current default behaviour
Users that want to use rustls would then import `graph-rs  = {version="0.3", default-features=false, features = ["rustls-tls"]

That's from looking at what crates like axum do when specifying their tls flavour, but I might be wrong.

[dependencies]
reqwest = { version = "0.11", default-features = false, features = ["json", "gzip", "blocking", "stream"] }

[features]
default = ["nativetls"]
nativetls = ["reqwest/nativetls"] # define a nativetls feature of our crate that enables reqwest's nativetls feature
rustls-tls = ["reqwest/rustls-tls] # define a rustls feature of our crate that that enables reqwest's rustls-tls feature

This is from the cargo book : https://doc.rust-lang.org/cargo/reference/features.html
If the client crate already defines a dependency on reqwest, these will supercede any feature you may describe , as far as I can tell features are additive, but the consumer has priority when it comes to conflict resolution.

Ok cool. I think we are on the same page. This was my thought after looking into it as well.
Basically making features available and letting the library user decide on what to use for things like tls libraries.
This would probably need to be done in a way that prevents having to define those features across all projects that use reqwest in this repo (I think 4 off the top of my head). That will get ugly trying to maintain. Possibly the repeated dependencies could go in a separate project or even in the graph-core project which is specifically meant to be used by all projects if needed as it only defines code that is used by or can be used by all projects (technically it just defines two enums right now).
Looking into this some more. Thanks for the explanation and research on this.

I don't think you need to specify the flavour of TLS you are going to use all crates. Set the default in the crate that users will use, and only specify no-default=true for all child crates. This is because crates in the same workspace use the same dependency. You may have to set dev-dependencies if you have integration tests in the child crates.

I don't think you need to specify the flavour of TLS you are going to use all crates. Set the default in the crate that users will use, and only specify no-default=true for all child crates. This is because crates in the same workspace use the same dependency. You may have to set dev-dependencies if you have integration tests in the child crates.

Yep your right. Testing it looks like thats working.

I’ll +1 this! Thanks for the request @fdoyon

Awesome, nice bias for action!


Align Pack / UnPack with DIDComm

DIDComm WG as begun to try and standardize these. I'm going to try and get some of them to review here, and provide links to any test-vectors or source that can be used to ensure compatibility.

Great!


Bazel initial support:  Marlin

Tested on MacOS and Debian 9.
This and Zexe are mutually dependent; neither will build until both are merged.  PR for Zexe is 37

Point taken.  It's surely possible to derive cargo/Cargo.toml from the other cargo files, but I looked around online and don't see any tools for that, and writing one doesn't look entirely trivial to me.  But I'm not sure this is a major problem - the likelihood that cargo/Cargo.toml will get out of sync is probably no higher than the probability that Marlin and Zexe will get out of sync, no?  Could we write a githook that prevents divergence?

Fixed an oversight: put cargo in top-level bzl dir, to segregate pure bazel stuff and minimize top-level clutter.

Obsoleted by #71


15F070 Has Messed Up Mobile CSS

Looks like I was a little hasty with the deployment this morning, as tablets and phones have awful CSS issues.

Cannot View Dropdown to Add Image / Geolocation / Long Post
[Cancel][Send] on New Post is out of alignment

15F071 resolves these issues and then some. Yay!


"deepdive do sentences" - Taking more time without creating any rows in the sentences table

Hi Team,
I have got the following last error statement while i manually stop by ctrl+c
2017-08-21 13:26:34.376376 ++ dirname process/ext_sentences_by_nlp_markup/run.sh
2017-08-21 13:26:34.376384 + cd process/ext_sentences_by_nlp_markup
2017-08-21 13:26:34.376392 + : dd_tmp_ dd_old_
2017-08-21 13:26:34.376400 + export DEEPDIVE_CURRENT_PROCESS_NAME=process/ext_sentences_by_nlp_markup
2017-08-21 13:26:34.376407 + DEEPDIVE_CURRENT_PROCESS_NAME=process/ext_sentences_by_nlp_markup
2017-08-21 13:26:34.376415 + export DEEPDIVE_LOAD_FORMAT=tsj
2017-08-21 13:26:34.376423 + DEEPDIVE_LOAD_FORMAT=tsj
2017-08-21 13:26:34.376431 + output_relation=sentences
2017-08-21 13:26:34.376439 + output_relation_tmp=dd_tmp_sentences
2017-08-21 13:26:34.376447 + output_relation_old=dd_old_sentences
2017-08-21 13:26:34.376455 + deepdive create table-if-not-exists sentences
2017-08-21 13:26:35.423450 + deepdive create table dd_tmp_sentences like sentences
2017-08-21 13:26:38.080108 CREATE TABLE
2017-08-21 13:26:38.082247 + deepdive compute execute 'input_sql=
2017-08-21 13:26:38.082294
2017-08-21 13:26:38.082306 SELECT R0.id AS column_0
2017-08-21 13:26:38.082315      , R0.content AS column_1
2017-08-21 13:26:38.082323 FROM articles R0
2017-08-21 13:26:38.082331
2017-08-21 13:26:38.082339 ' 'command=cd "$DEEPDIVE_APP" && udf/nlp_markup.sh' output_relation=dd_tmp_sentences
2017-08-21 13:26:38.478602 Executing with the following configuration:
2017-08-21 13:26:38.478671  DEEPDIVE_NUM_PROCESSES=7
2017-08-21 13:26:38.478688  DEEPDIVE_NUM_PARALLEL_UNLOADS=1
2017-08-21 13:26:38.478703  DEEPDIVE_NUM_PARALLEL_LOADS=1
2017-08-21 13:26:38.478713  DEEPDIVE_NAMED_PIPES_DIR=/home/XXXXX/deepdive/examples/spouse/run/process/ext_sentences_by_nlp_markup
2017-08-21 13:26:38.820593 unloading to /home/XXXXX/deepdive/examples/spouse/run/process/ext_sentences_by_nlp_markup/deepdive-compute-execute.KD5A3y4/feed_processes-1: '
2017-08-21 13:26:38.820675
2017-08-21 13:26:38.820688 SELECT R0.id AS column_0
2017-08-21 13:26:38.820696      , R0.content AS column_1
2017-08-21 13:26:38.820704 FROM articles R0
2017-08-21 13:26:38.820712
2017-08-21 13:26:38.820720 '
2017-08-21 13:26:39.986708 Loading dd_tmp_sentences from /home/XXXXX/deepdive/examples/spouse/run/process/ext_sentences_by_nlp_markup/deepdive-compute-execute.KD5A3y4/output_computed-1 (tsj format)
Any suggestions?
Thanks
Bala

Hi,
It's normal that takes a lot of time compared to other steps. Did it create "dd_tmp_sentences" table or not ?
And which dataset you use? I mean how big is your dataset, number of sentence etc..?

Hi mcavdar,
Thanks for you reply.
I have extracted the contents of various documents from a repository.( nearly 700 documents of PPT,PDF,DOCX and weg pages). I have created an article.tsv file which is having all the contents along with a content_id.
I can execute the deepdive do articles and which created a table of all these contents.
But, when i execute the deepdive do sentences command, which will create the dd_tmp_sentences and sentences table.I am using deepdive 0.8 version.
On executing the deepdive do sentences,
CoreNLP processes got started  and finished with an error. I have pasted my logs here for your reference.
Last few lines for your reference.
2017-08-24 04:29:04.853776 INFO: Ignoring inactive rule: null
2017-08-24 04:29:04.854532 Aug 24, 2017 4:29:04 AM edu.stanford.nlp.ling.tokensregex.CoreMapExpressionExtractor appendRules
2017-08-24 04:29:04.854563 INFO: Ignoring inactive rule: temporal-composite-8:ranges
2017-08-24 04:29:04.854789 Reading TokensRegex rules from edu/stanford/nlp/models/sutime/english.holidays.sutime.txt
loading dd_tmp_sentences: 0:01:06    1  [15.1m/s] ([   0 /s])2017-08-24 04:29:40.044975 Loading parser from serialloading dd_tmp_sentences: 0:01:07    1  [14.9m/s] ([   0 /s])2017-08-24 04:29:40.415900 Warning: skipped malformedloading dd_tmp_sentences: 0:01:07    1  [14.9m/s] ([14.9m/s])B/s])ns, this is my first sentence"}
2017-08-24 04:29:40.458075 ERROR:  missing data for column "sentence_index"
2017-08-24 04:29:40.458141 CONTEXT:  COPY dd_tmp_sentences, line 1: "#"
2017-08-24 04:29:40.460150 /home/u5/local/util/compute-driver/local/compute-execute: line 140: kill: (50040) - No such process
2017-08-24 04:29:40.460183 /home/u5/local/util/compute-driver/local/compute-execute: line 140: kill: (50044) - No such process
2017-08-24 04:29:40.460196 /home/u5/local/util/compute-driver/local/compute-execute: line 140: kill: (50045) - No such process
2017-08-24 04:29:40.460204 /home/u5/local/util/compute-driver/local/compute-execute: line 140: kill: (50051) - No such process
2017-08-24 04:29:40.460213 /home/u5/local/util/compute-driver/local/compute-execute: line 140: kill: (50052) - No such process
ârun/ABORTEDâ -> â20170824/042830.108135291â
I also want to know the actual nlp_backup.sh content.
please help me to resolve it.
Thanks,
Bala

Hi @Balachandar-R ,

2017-08-24 04:29:40.458075 ERROR: missing data for column "sentence_index"

According to this, deepdive doesn't get clearly result/output of CoreNLP.
I suggest to run this piece of code from terminal, when your CoreNLP system is running.
wget --post-data "Hello friend!" 'http://localhost:24688/?properties={"annotators": "tokenize, ssplit, pos, lemma,ner","outputFormat": "json"}' -O -
Expected output is something like that, with sentences and index tags:

{"text":"Hello friend!","sentences":[{"index":0,"tokens":[{"index":1,"word":"Hello","originalText":"Hello","lemma":"hello","characterOffsetBegin":0,"characterOffsetEnd":5,"pos":"PROPN","ner":"O"},{"index":2,"word":"friend","originalText":"friend","lemma":"friend","characterOffsetBegin":6,"characterOffsetEnd":12,"pos":"VERB","ner":"O"},{"index":3,"word":"!","originalText":"!","lemma":"!","characterO-

Is it same for you?

To start CoreNLP, run:
export CORENLP_JAVAOPTS=-Xmx4g
deepdive corenlp start

If you didn't install Corenlp yet, install it before start:
deepdive corenlp install

Hi @mcavdar,
Thanks for your responses.
Since i am  using deepdive 0.8, Bazaar parser is doing document parsing process.So that i could start CoreNLP as a service in this version.  After exporting a variable called DEEPDIVE_NUM_PROCESSES=1 the parser can successfully running on  all the documents without any errors.
Thanks @mcavdar

Hi @Balachandar-R ,
I tried to run spouse example on Deepdive 0.8. Actually, it uses an old  Shift-Reduce Constituency Parser model. (srparser-2014-10-23-models) So I'm not sure but it doesn't seem like it finds the NER.
I hardly suggest you to update Deepdive. Thus, you can use CoreNLP either with NER functionality or with Regexner functionality. See for train your own NER. Also see Regexner example. If you are interresting just with some predefined keyword, Regexner is easy way to do.

HI @mcavdar,
Thanks man.. I have updated DeepDive and now i can create the sentences table as well.
But i have one more question.
1.I have extracted the content from various repositories and writing it in a single file(.tsv) using python.
But the fileformat of this .tsv file was
UTF-8 Unicode (with BOM) text, with very long lines
But the fileformats available in the spouse example is something like
UTF-8 Unicode text, with very long lines
And the Postgres sql also follows the UTF8 encoding scheme.
How could you see it from your perspective ?
Any Suggestion?
Thanks
Balachandar

Good morning,
I'm having the same issue, I've downloaded the stable version of deepdive and the spouse example. Unfortunately I can't pass the process of NLP_markup, it takes too much time and memory and the process is aborted by the same deepdive. I've tried to update my version because I do not have the command "deepdive corenlp *", with no luck.
Will you be so kind to tell me exactly how to make the deepdive update, please.
Thanks in advance.
My deepdive version is:
deepdive v0.8.0-79-g28a58de (Linux x86_64)
Information on this build of deepdive follows.

To build and install last version from source code: link. If you want to install from specific commit, do git checkout <commit_hash> after git clone https://github.com/HazyResearch/deepdive.git


Crossing -90 Longitude

When route crosses -90 Longitude heightraph shows some crazy data.
Even the shortest 100-meter route shows as 120000 km long

@zzptichka
could you provide a specific example please.
I couldn't reproduce this up till now:
http://maps.openrouteservice.org/directions?n1=32.292357&n2=-90.190887&n3=10&a=32.240683,-89.989014,32.379961,-90.351563&b=0&c=0&k1=en-US&k2=km
http://maps.openrouteservice.org/directions?n1=32.292357&n2=-90.190887&n3=10&a=32.379961,-90.351563,32.240683,-89.989014&b=0&c=0&k1=en-US&k2=km
which version of the Leaflet.Heightgraph are you currently using ?

I think this might be a problem with my data. I'll close the issue and re-open if I can confirm it.


DataTable: Feature request implement pagination

Hi team,
We're extensive users of Python Shiny and its such a great product!  We have small feature request to enhance the existing DataTable:
We use shiny.render.DataTable to visual our various datasets. It appears the current design sends the entire dataframe data into the browser in a single websocket message. Where the dataframes are even moderately large with thousand of rows, it results in the browser visibly hanging. With large dataframes in the hundreds of thousands for rows, it often causes the browser to fail entirely.
We have implemented our own pagination mechanism using shiny.ui.input_selectize but this is less than ideal. It requires the user to select a page dropdown where previously users can use the mouse scroll. The existing shiny data grid summary, showing the row message, no longer aligns to the dataframe shape.
Many thanks!
Versions
Python: 3.12.5
shiny: 1.0.0
Operating System: Ubuntu 22.04.4 LTS
cc: @gshotwell and the dataframe epic: https://github.com/posit-dev/py-shiny/issues/1119

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

You can add iTables to your project very easily.  They have an integration:  https://mwouts.github.io/itables/shiny.html
This is a screenshot of a very early version of an application I'm building.


fix "Cannot read property 'riskcode' of undefined"

Fixes the following:
#27
#25
#21
#16
#12

@insideou7 @izvictor I will work on getting this PRs merged and a new version released by the end of the weekend. I need to do some changes to the CI pipeline and how I release these new versions. I will work on that during the weekend. Sorry for the delay.

Thanks @kasunkv appreciate you getting back to us. Looking forward to the fix 👍

@kasunkv Any progress on getting this PR complete?

@insideou7 Thanks for the contribution :) Finally got some time to setup the CI process for the repo. Can you please update this branch with the current changes to the master branch and update the PR. Then I will  approve and get a version released. Sorry for the delay in response.

Done


Can YouTubeCenter hide/remove the channel branding "bug" overlays that some videos have?

These channel branding "bug" overlays on so many YouTube videos can be just as distracting as they've been on televisions for years now:  http://www.illumina.net/default.htm
See screenshot, from SNL's Black Widow Trailer (parody) here:

http://imgur.com/gallery/AKt6Afs/
Original YouTube video here:  http://www.youtube.com/watch?v=j_5KgpN38hM
Can YouTubeCenter have a setting to remove/hide these "bugs," please?  SPOI has this, but now that SPOI is conflicting with YTC, I had to make the choice (and of course YTC wins.)  This is the only feature I'm seriously missing now.
I'd appreciate the consideration.  Many thanks!

Why are you calling them bugs? These aren't bugs, just disable annotations and they'll be removed from the video.

Gotcha - I appreciate it! They reminded me of TV logo "bugs" as described here:
http://en.wikipedia.org/wiki/Digital_on-screen_graphic
http://www.illumina.net/default.htm
I clearly had no idea that YTC was referring to them as "annotations."  Thanks so much!

Please close the issue!

These annotations are still appearing for some videos, even with "enable annotations" unchecked.
Example video:  https://www.youtube.com/watch?v=R1nfAxkpx9M

@Fofer No idea if you are still watching this, but you can control those particular annotations with a setting located here: https://www.youtube.com/account_playback

@SuperSajuuk Yes, I am still watching, and thanks for that link!  I wasn't aware of it, obviously, and very happy to have "native" controls in YouTube's settings to turn (at least some of?) them off.
So is YTC's "enable annotations" checkbox now redundant?  Or is it meant to cover more/other bases?
I do still contend that YTC's setting should be the reverse.  I think it would be more intuitive to have us check the box to DISABLE them (as opposed to checking the box to enable them, per YouTube's default state.)
Thanks again!

@Fofer Those options have been there for years, but Google's updates over the last 1-2 years have resulted in a lot of the older "account" style pages virtually impossible to reach now.
I guess that YT Center currently only supports removal of normal, user-added annotations. The parts covering channel promotions and interactive cards don't seem to be getting tracked by YT Center and is likely due to the fact that these aren't really annotations in the sense.
The channel promotions only run for a short time anyway, so it's somewhat pointless for them to be removed since I believe they run for about 10-15 seconds maximum. The interactive cards is a relatively new feature and I guess @YePpHa hasn't had a chance to fix that bug.
It would be nice if there were MORE developers involved with YT Center though: just @YePpHa being involved in making commits means that updates to the script are so lengthy that bugs are going unfixed for ages. There is no chance of the bugs listed here being cleaned up. It would be nice if @Yonezpt had the right to make commits to the repository, but until that happens... don't expect many updates.

Disabling annotations isn't really a good way to do this since you'll, well, lose annotation. I would like an option to remove only the "bug". Also looking at the "Remove Branding Watermark" it sounds like it would to this, but it doesn't seem to, so what does it do?

@Fofer Have you tried Removing watermarks ?

Yes, I have that active.
Den 2015-07-16 20:59, IreuN skrev:

@Fofer https://github.com/Fofer Have you tried Removing watermarks
https://github.com/YePpHa/YouTubeCenter/wiki/Features#Remove_Branding_Watermark
?
—
Reply to this email directly or view it on GitHub
https://github.com/YePpHa/YouTubeCenter/issues/1854#issuecomment-122050562.

Me too.  Doesn't seem to make a universal difference.  I still get them, from time to time, on various videos.  I'm not sure I fully understand the difference between "channel branding," "annotations," and "watermarks" but ideally I'd see none of them.
YTC used to block them all, but with YouTube constantly changing things, this feels like a game of "Whack-a-Mole."


Craft 3?

Will this plugin be updated for Craft 3? If so, any timeline?

@davist11 no timeline. Eventually we will have to move the site it is on to Craft 3, but that won't happen anytime soon. Sorry!


Upgrade to kafka-clients 1.0.0

Resolves https://github.com/spring-projects/spring-kafka/issues/468

update client version
fix KStreams deprecations
fix KafkaEmbedded to be compatible with 1.0.0

deprecate unused methods where the AdminUtils is used but has been removed

OK. I see that you have just made 2.0.x branch.
So, this is already 2.1.0, therefore gradle.properties must be fixed as well.
Let me know and I'll fix on merge!

Oops - yes, of course; thanks.

We should deprecate (but leave the code) in those methods in 2.0 as well.

in those methods in 2.0 as well.

Sure! Feel free to push there or PR and I'll merge.

Merged as https://github.com/spring-projects/spring-kafka/commit/67104ed1a580c43c00bb9a80e852010ed498c040


Latest -dev Fdroid Riot get no notification/sound when called from latest -dev GPlay riot . It works OK from 2Fdroid versions.

reported by @ji-ef:matrix.org
https://github.com/matrix-org/riot-android-rageshakes/issues/1038

I asked to @Ji-eF:matrix.org whether the caller is not blacklisted or the dedicated room is not muted.

https://github.com/vector-im/riot-android/issues/1620


added chapter 11 stateful containers with portworx

added stateful containers with portworx

LGTM!
@ManoMarks this is good to be merged!

After discussion with @arun-gupta we're not going to merge this request. Thanks.


Fixed link generation error when using modules introduced in PR #32

Closes https://github.com/Carrooi/Nette-Menu/issues/33

thanks :+1:


Allow any input in to_onnx and to_torchscript

What does this PR do?

Fixes #4373
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

args (tuple of arguments or torch.Tensor) – the inputs to the model, e.g., such that model(*args) is a valid invocation of the model. Any non-Tensor arguments will be hard-coded into the exported model

According to torch.onnx.export docs, it has to be tuple or Tensor? Also I'm not sure what hard-coding non-Tensor arguments mean exactly...

Is there a test to check for list input?

Also, how will this work if the input_sample is an str or some other type?

I will change it to allow only list or tuple of tensors.

@rohitgr7 should be unblocked now!

Hello @rohitgr7! Thanks for updating this PR.

In the file pytorch_lightning/core/lightning.py:

Line 1495:121: E501 line too long (122 > 120 characters)

@tchaton is it good now?

looked at PyTorch source code and did some testing with different kinds of input (dict, list, custom batch, tuple, ...). Looks like all possible inputs are allowed in both these method. Even [Tensor, str] is allowed in torch.onnx.export as long as the second item in the list is not used in the forward method computation. For other cases, PyTorch raises some error.
Will update this PR to allow any input without any type check from pl side. Let PyTorch handle other cases.

Re-requesting reviews since the PR is updated to allow any input to these methods.

@williamFalcon this needs your approval

@williamFalcon mind review? made the changes.

@williamFalcon mind review?

This as api change shall go in 1.2


When showing an error text in references, show display values for missing fields

For example: title_primary, gen_notes -> Title, Notes.

There's no object that maps the field names to their proper names (title_primary, gen_notes -> Title, Notes) currently. This has been done previously by typing the proper name into the validators name field, but the reference validator has been coded in a way that makes this really hard. I think the reference validators have to either be rewritten entirely or there has to be an object that does the field_name -> Proper Name mapping.


Update deps and code cleanup

Cleanup code and update crates.
Adds a lint for the code format.

You merge your own PRs ?

I assumed since you merged the other PR into this one that was the intention


Create 230-Kth-Smallest-Element-in-a-BST.cs

File(s) Modified: 230-Kth-Smallest-Element-in-a-BST.cs
Language(s) Used: c#
Submission URL: https://leetcode.com/submissions/detail/753583000/

Thanks, @notauserx!


Turtlebot topics not visible, only on Ubuntu 22.04 jammy

Hi, I can't get the simulation and terminals to communicate using source run_discovery_service.sh in Ubuntu 22.04 + humble, using the jammy branch and UE5.1. After starting the simulation, I can't see topics published by the engine. Here are my outputs:

Editor

~/dev/turtlebot3-UE$ ./run_editor.sh
2024-07-05 11:26:38.919 [XMLPARSER Error] Found incorrect tag 'userTransports' -> Function fillDataNode
2024-07-05 11:26:38.919 [XMLPARSER Error] Error parsing participant profile -> Function parseXMLParticipantProf
2024-07-05 11:26:38.920 [XMLPARSER Error] Error parsing profile's tag participant -> Function parseProfiles
2024-07-05 11:26:38.920 [XMLPARSER Error] Error parsing '/home/catec/dev/turtlebot3-UE/fastdds_config.xml' -> Function loadXMLFile
### Server is running ###
  Participant Type:   SERVER
  Security:           NO
  Server ID:          0
  Server GUID prefix: <IP_ADDRESS>f.<IP_ADDRESS>f.53.49.4d.41
  Server Addresses:   UDPv4:[<IP_ADDRESS>]:11811
The daemon has been started
Increasing per-process limit of core file size to infinity.
* UE5 output *

Listener

~/dev/turtlebot3-UE$ source run_discovery_service.sh
2024-07-05 11:26:56.890 [XMLPARSER Error] Found incorrect tag 'userTransports' -> Function fillDataNode
2024-07-05 11:26:56.890 [XMLPARSER Error] Error parsing participant profile -> Function parseXMLParticipantProf
2024-07-05 11:26:56.890 [XMLPARSER Error] Error parsing profile's tag participant -> Function parseProfiles
2024-07-05 11:26:56.890 [XMLPARSER Error] Error parsing '/home/catec/dev/turtlebot3-UE/fastdds_config.xml' -> Function loadXMLFile
2024-07-05 11:26:56.892 [RTPS_PARTICIPANT Error] Discovery Server wasn't able to allocate the specified listening port. -> Function createParticipant
2024-07-05 11:26:56.893 [DOMAIN_PARTICIPANT Error] Problem creating RTPSParticipant -> Function enable
Server creation failed with the given settings. Please review locators setup.
The daemon is already running
fastdds discovery <discovery-args>

1
 fast-discovery-server tool not found!
~/dev/turtlebot3-UE$ ros2 topic list
/parameter_events
/rosout
~/dev/turtlebot3-UE$

This was not an issue in Ubuntu 20.04 + foxy under the devel branch. I have noticed that the fastdds_config.xml is the same for both devel and jammy. Might this be an issue with this config file, which is outdated for the more recent humble fastdds version?
As a workaround, not using the repo scripts and just running $UE5_DIR/Engine/Binaries/Linux/UnrealEditor turtlebot.uproject then sourcing ROS on the receiving terminal makes all topics visible and echoable.

Hi @pabsan-0 I'm also encountering this problem trying to test locally! Did running ./run_editor.sh false with the discovery server turned off work for you? Because it also doesnt seem to for me - I can't seem to get any ros topics in another terminal.
I was just reading and it seems there's issues running discovery server on localhost: https://github.com/eProsima/Fast-DDS/issues/2031#issuecomment-1176388874

Hi after some testing I stumbled upon the following post which pointed towards a solution! https://github.com/ros2/rmw_fastrtps/issues/676#issuecomment-1470432237
Essentially discovery server by default will work in the background, but all of the ros2 cli tools fail. They require the super client settings in order for the ros2 cli tools to work (https://fast-dds.docs.eprosima.com/en/latest/fastdds/ros2/discovery_server/ros2_discovery_server.html#daemon-s-related-commands). This combined with an outdated format for the fastdds config meant that you could not get a connection and instead got the Error parsing errors.
What worked for me was to update the fastdds_config.xml to the following:
<?xml version="1.0" encoding="UTF-8" ?>
<profiles xmlns="http://www.eprosima.com/XMLSchemas/fastRTPS_Profiles">
  <transport_descriptors>
    <transport_descriptor>
      <transport_id>udp_transport</transport_id>
      <type>UDPv4</type>
      <maxMessageSize>1400</maxMessageSize>
    </transport_descriptor>
  </transport_descriptors>

  <participant profile_name="participant_profile" is_default_profile="true">
    <rtps>
      <userTransports>
        <transport_id>udp_transport</transport_id>
      </userTransports>
      <useBuiltinTransports>false</useBuiltinTransports>

      <builtin>
        <discovery_config>
          <discoveryProtocol>SUPER_CLIENT</discoveryProtocol>
          <discoveryServersList>
            <RemoteServer prefix="<IP_ADDRESS>f.<IP_ADDRESS>f.53.49.4d.41">
              <metatrafficUnicastLocatorList>
                <locator>
                  <udpv4>
                    <address><IP_ADDRESS></address>
                    <port>11811</port>
                  </udpv4>
                </locator>
              </metatrafficUnicastLocatorList>
            </RemoteServer>
          </discoveryServersList>
        </discovery_config>
      </builtin>
    </rtps>
  </participant>
</profiles>

The docs mention that the RemoteServer should be set to whatever the output of fastdds discovery -i 0 is.

Then additionally you should add the following to the bottom of fastdds_setup.sh
# Restart the daemon to setup the super client config
echo "Restarting ROS2 Daemon For New FASTRTPS Settings"
ros2 daemon stop
ros2 daemon start

Which restarts the ros2 daemon with the super client enabled automatically!
Hopefully this works for you!

Edit!
In run_discovery_service.sh you may want to change the call to fastdds discovery into a subshell rather than background process. This is so that the discovery server also stops when you exit/ctrl c, so that the next time you run, the discovery server restarts correctly.
#!/bin/bash

source fastdds_setup.sh
source /opt/ros/humble/setup.bash
(exec fastdds discovery -i 0) # Run as sub shell so ctrl c will stop it
ros2 daemon start

I have had issues with the discovery server not reconnecting properly with a restarted UE5 instance.


skip_before_filter doesn't work

Hi,

class ApplicationController < ActionController::Base
  before_filter RubyCAS::Filter
end
class MyController < ApplicationController
skip_before_filter RubyCAS::Filter
def index
# But here before_filter is triggered !
end
end

How to do that ?
Regards,
Rivsc

If you'd like to use skip_before_filter, put RubyCAS::Filter in a method in ApplicationController. When you say skip_before_filter, Rails doesn't know what you're talking about if you don't. Here's an example:
class ApplicationController < ActionController::Base
  before_filter :cas_filter

  def cas_filter
    RubyCAS::Filter.filter(self)
  end

end

Then, in your other controller you may skip normally:
class MyController < ApplicationController
  skip_before_filter :cas_filter

  def index
    # No more CAS here
  end
end


Support for Grocy 4.0 API

Yet again the grocy api changed somewhat.
One thing that does not work anymore would apparently be grocy.all_products():
Traceback (most recent call last):
  File "/builds/alexwaresystems/mink/grocy-backup/export.py", line 23, in <module>
    grocy.all_products())
    ^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pygrocy/grocy.py", line 110, in all_products
    product_datas = [ProductData(**product) for product in raw_products]
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pygrocy/grocy.py", line 110, in <listcomp>
    product_datas = [ProductData(**product) for product in raw_products]
                     ^^^^^^^^^^^^^^^^^^^^^^
  File "pydantic/main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for ProductData

We just merged a fix that'll address this issue. @SebRut can we push an incremental release so the latest version supports 4.0 installations? Guiltlessly, if we do that then we can update the Home Assistant integration to leverage the new fix as well.

I kicked of new release, but it seems like the Travis CI setup is broken again on their side :/ Hopefully the support is able to resolve this quickly.
Should probably migrate to GitHub actions or something else completely with Travis breaking occasionally.

Version 2.0.0 should now be available on PyPI.


NGSW based Service Worker doesn't work in ng8 with differential builds

🐞 Bug report
Command (mark with an x)
- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description
When doing a production build with differential builds (target = es2015 in tsconfig), the service worker does not properly serve up cached esm js files. It throws an error (below in the error section) in the browser console window.
Turning off differential builds (by settings target to es5 in tsconfig) fixes the issue.

Note, this issue only appears to be in Chrome (see the end of the issue).

🔬 Minimal Reproduction
Create a new app:
ng new ng8-pwa-test (I added routing and selected CSS)
Update the bits to make sure on latest:
ng upgrade @angular/cli @angular/core
Add NGSW:
ng add @angular/pwa
Do a prod build:
ng build --prod
Use a local web server to serve up the dist/ng8-pwa-test (using (https://www.npmjs.com/package/local-web-server)[local-web-server myself]):
ws --spa index.html --static.maxage -1  -p 8080 -z -d dist/ng8-pwa-test
Visit http://localhost:8080, open dev tools and verify the service worker is running on the Application tab. Maybe refresh a few times to make sure everything is nice and primed.
Go offline via the Service Worker tab.
Refresh, and get the error down below in the error section.
Now, go into tsconfig.json and turn off differential builds by setting the target to "es5".
Do another prod build:
ng build --prod
Restart the web server:
ws --spa index.html --static.maxage -1  -p 8080 -z -d dist/ng8-pwa-test
Go back to the browser, unregister the existing service worker, clear out any cache, go back online and see the page again.
Now, go back offline, hit refresh, and see the es5 builds are properly being served from the sw cache.
Heres a link to a project already prepped https://github.com/elylucas/ng8-pwa-test
🔥 Exception or Error

ngsw-worker.js:596 Uncaught (in promise) Error: Response not Ok (fetchAndCacheOnce): request for http://localhost:8080/polyfills-es2015.e4a1185e6871d06f842f.js returned response 504 Gateway Timeout
    at PrefetchAssetGroup. (ngsw-worker.js:596)
    at Generator.next ()
    at fulfilled (ngsw-worker.js:320)
(anonymous) @ ngsw-worker.js:596
fulfilled @ ngsw-worker.js:320
localhost/:16 GET http://localhost:8080/main-es2015.a547e075e866b7c26e38.js net::ERR_FAILED
ngsw-worker.js:596 Uncaught (in promise) Error: Response not Ok (fetchAndCacheOnce): request for http://localhost:8080/main-es2015.a547e075e866b7c26e38.js returned response 504 Gateway Timeout
    at PrefetchAssetGroup. (ngsw-worker.js:596)
    at Generator.next ()
    at fulfilled (ngsw-worker.js:320)

🌍 Your Environment

Angular CLI: 8.2.0
Node: 10.15.3
OS: darwin x64
Angular: 8.2.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, service-worker

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.802.0
@angular-devkit/build-angular     0.802.0
@angular-devkit/build-optimizer   0.802.0
@angular-devkit/build-webpack     0.802.0
@angular-devkit/core              8.2.0
@angular-devkit/schematics        8.2.0
@angular/pwa                      0.802.0
@ngtools/webpack                  8.2.0
@schematics/angular               8.2.0
@schematics/update                0.802.0
rxjs                              6.5.2
typescript                        3.5.3
webpack                           4.38.0

Anything else relevant?
During testing, I noted that this issue only seems to happen in Chrome (in on v76). I tried in Firefox and Safari and they both seemed to work, but the devtools there aren't as good so not sure if I was going offline properly (i just killed the local server).
Therefore, this could be an issue in Chrome and ESM module loading in service workers.
Im also on Mac OSX Mojava.

Hi @elylucas, I downloaded your reproduction and tried it locally and I was not able to replicate the exact issue that you are reporting. I am using Chrome version Version 76.0.3809.100 on Windows.

That being said, I can see a different error which is for the manifest.webmanifest, which I think we should be adding to the ngsw.json

hi, I have similar issue like @elylucas . However, it randomly happens, sometimes I can reproduce it, sometimes I can't

Currently facing this issue as well. Happens consistently between builds.
Produce a production build, load it, let the worker cache.
Produce a new production build, then attempt to load it - will receive the fetchAndCacheOnce method error.
Once you receive this error, refreshing the browser then fetches the latest and everything is happy.
Currently looking at implementing a hack into the ngsw-worker.js to just refresh instead of throwing the error until this is resolved.

Also having this issue on a new project.

It seems for me using NPM package http-server instead of NPM package ws to host my Angular 8 production builds resolved the issue in testing.

Hello! This issue is still persist in my environment. I will really appreciate if fix is available.

Just ran into this also... target change to es5 from es2015 seems to have "fixed" it but a real fix would be nice. Also a simple reload after ngsw has download the new files also allows the app to load, which is very strange and seems like ngsw-worker.js has some interaction with Chrome (if this indeed only happens on Chrome, I haven't tested that).

This issue is now obsolete since differential loading will be removed in the next major version. See: https://github.com/angular/angular-cli/pull/21467


Adding filters to move only Epics from different projects.

Can we move only list of  epics from different projects in JIRA to Azure boards? If yes can you please let me know how to apply the filter for this.

Yes, customize the JQL query in the config file to include the issuetype, for instance
"query": "project = SCRUM and issuetype = Epic ORDER BY Rank ASC",

@vishwanathk26071981 did you manage to get this working? I'm closing the issue, feel free to re-open if you find any issues with the JQL implementation.


Error when starting

Below errors are thrown when I start the smart open through Telescope
Error executing vim.schedule lua callback: .../smart-open.nvim/lua/smart-open/util/priority_insert.lua:9: attempt to compare nil with number
stack traceback:
        .../smart-open.nvim/lua/smart-open/util/priority_insert.lua:9: in function 'priority_insert'
        ...m/lua/telescope/_extensions/smart_open/finder/finder.lua:94: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

Weird thing is this only happen for this specific CWD while in other projects it's working fine.
I'm on the 0.2.x branch

Sorry, didn't notice this until just now.  I'll take a look

I think I fixed this.  Thanks for putting in an issue


Add emoji support

Just wanted to point out that Unicode Support in Windows-based consoles is very troublesome right now.
For example, the current print line to console
click.secho("All done! ✨ 🍰 ✨", bold=True)

prints
All done! ✨ �🍰✨✨

In some versions of the console, no unicode output can be seen.
However, https://github.com/microsoft/terminal is set to be shipped as the default terminal for all windows systems sometime soon, which delivers expected outputs for Unicodes. But until then, only *nix based systems could have this feature.


Enable Kube-VIP for Target Clusters

Description: We do support bring a custom load balancer so customers should be able to deploy kubevip as a load balancer option. At the moment, no plans to ship kubevip as a load balancer sku in AKS.


Make container build use idiomatic gradle

The stage task seems to be related to deploying to Heroku. It adds
nothing to the assemble task. The assemble task seems to be a more
commonly used task name for assembling a .jar.

This depends on #255.

There is a good breakdown of the various tasks here: https://docs.gradle.org/current/userguide/java_plugin.html.
Agreed that the stage task is a Heroku thing.


You must provide an API key

    image: plugins/gitea-release
    settings:
        base_url: http://gidro.westeurope.cloudapp.azure.com:3000
        api_key:
          from_secret: gitea_token
        files: .*.tar.gz
    checksum:
        - md5
        - sha1
        - sha256
        - sha512
        - adler32
        - crc32
    when:
        event: tag

not working?

compose file
version: "3.7"

services:
  gitea:
    image: gitea/gitea:latest
    environment:
      - USER_UID=1000
      - USER_GID=1000
    restart: always
    networks:
      - gitea
    volumes:
      - ./gitea:/data
    ports:
      - "3000:3000"
      - "222:22"
  drone-server:
    image: drone/drone:1
    container_name: drone_server
    networks:
      - dronenet
    ports:
      - "8080:80"
      - "8443:443"
    volumes:
      - /var/lib/drone:/data
      - /var/run/docker.sock:/var/run/docker.sock
    restart: always
    environment:
      - DRONE_GITEA_SERVER=http://gitea:3000
      - DRONE_GIT_ALWAYS_AUTH=false
      - DRONE_RUNNER_CAPACITY=2
      - DRONE_SERVER_PROTO=http
      - DRONE_SERVER_HOST=gidro.westeurope.cloudapp.azure.com:8080
      - DRONE_TLS_AUTOCERT=false

networks:
  dronenet:

I'm using this plugin on a daily base, I can't confirm any issue as long as you configured the secrets and drone correctly.

You files setting won't work, it got to be a glob and not a regular expression.

Closing because of lacking details

I have the same problem. I get  time="2019-08-19T12:56:28Z" level=fatal msg="You must provide an API key" as well.
With API-Key I hope you mean the gitea-token? Because otherwise I've done something completely wrong

If you don't provide the gitea api key it can't work.

I generated the key according to this documentation:
https://docs.gitea.io/en-us/api-usage/
But it still says I have to provide an API key.
  PUSH:     image: plugins/gitea-release     settings:       api_key:         from_secret: gitea_token       base_url: https://buddyspencer.duckdns.org/gitea       files: gohome     when:       event: tag

I generated the key according to this documentation:
gitea
But it still says I have to provide an API key.
PUSH:
image: plugins/gitea-release
settings:
api_key:
from_secret: gitea_token
base_url: https://xxx/gitea
files: gohome
when:
event: tag

Than I would guess that you have not added the secret correctly


Added arrays and strings more questions

Added more questions of Arrays and Strings inside the array folder

Hey I had added a folder of linked list in which there a lot of questions there.
Please review it.


Specs are broken due to Loofah whitelist

Since my previous changes in the def wrap_content_in_tag :
https://github.com/refinery/refinerycms/blob/8d2778944572162be6eef8811c17be28a2cb374f/pages/app/presenters/refinery/pages/section_presenter.rb#L66-L70
Loofah strip placeholders in inputs.

Is it Loofah?
I think I was loosing placeholders a couple of weeks ago, but didn¹t get to
follow it up.
At this point I would also suspect wymeditor.
(I suspect everyone)
Anita Graham
<EMAIL_ADDRESS>

It is Loofah. i think i will provide a PR with our own whitelist config, no more Loofah strange whitelist and deps.

I have done a version of that, but not written tests (or tested it).
--
Anita Graham
<EMAIL_ADDRESS>From:  Brice Sanchez<EMAIL_ADDRESS>Reply-To:  refinery/refinerycms-inquiries
<reply+00060f3e4c2c05a9b5caace8f00e14cafe2d429ffade68a592cf00000001131c4bdc9
<EMAIL_ADDRESS>Date:  Wednesday, 6 April 2016 at 12:00 PM
To:  refinery/refinerycms-inquiries
<EMAIL_ADDRESS>Cc:  Anita Graham<EMAIL_ADDRESS>Subject:  Re: [refinery/refinerycms-inquiries] Specs are broken due to
Loofah whitelist (#168)
It is Loofah. i think i will provide a PR with our own whitelist config, no
more Loofah strange whitelist and deps.

You are receiving this because you commented.
Reply to this email directly or view it on GitHub
<https://github.com/refinery/refinerycms-inquiries/issues/168#issuecomment-2
06109304>

Nice, could you provide a PR ?
Let me know if i could help you :)

https://github.com/refinery/refinerycms/pull/3164


GKE deployment script - four microservices to begin with

What does this PR do?
It adds k8s deployment scripts for the different microservices
What JIRA task is related to this PR?
N/A
How do I test out this PR?
A test can run on Jenkins server by triggering a CI/CD pipeline from GitHub
Which end points should are ready for testing?
No endpoints available but the test can be done on Jenkins server

Who is doing the merge duty for this one? Should I go ahead and do it?@martin, you can go ahead, though I have noticed there some comments that need to be resolved.

Who is doing the merge duty for this one? Should I go ahead and do it?@martin, you can go ahead, though I have noticed there some comments that need to be resolved.

Just seen this. Could we first make sure that it has the latest changes from MASTER?

Hi @Baalmart, Looks like this PR contain only the requested changes from the reviewers. The rest of the content is available here. I will update the PR to include locate, locate-registry and device-monitoring micro-services. Thanks!

Hi @Baalmart, Looks like this PR contain only the requested changes from the reviewers. The rest of the content is available here. I will update the PR to include locate, locate-registry and device-monitoring micro-services. Thanks!

Cool, thanks.

Check out the updated PR here https://github.com/airqo-platform/AirQo-api/pull/105


set default remote and branch for git push action

fixes #1096

#1431 proposes and more sophisticated approach to this.

Did not saw that one.


Cannot be Enabled

The full text of the error message:

Cannot be enabled due to issues in required dependencies

This is with Material Deck v1.7.0 and this module v1.0.1

You need to update Material Deck to v1.7.1


chore: add Dockerfile

Description

part of #32
Checklist

[x] I have explained the need for this PR and the problem it solves
[ ] I have explained the changes or the new features added to this PR
[ ] I have added tests corresponding to this change
[ ] I have updated the documentation to reflect this change
[ ] I have verified that this change is backward compatible

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

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


update

Hi,
Have a quick question with the last update of the zigbee2mqqt it added the support for "TRUST ZYCT-202" when will the this updated on the "hassio add-on"
Thank you For the great work

+1 for this request :)

Have a look here: https://github.com/danielwelch/hassio-zigbee2mqtt/pull/128

@TheZoker Thanks


learn how to write to files without running out of memory

https://github.com/jessinthecloud/ValheimJsonExporter/blob/777f74144e5535a0edd05c4cf41d133d9e4e9ec2/ValheimJsonExporter/Docs/SpriteDoc.cs#L52

Related to #14


Facial expression enhancement

Hello,
I was wondering if it would be possible to enhance the spectrum of the facial expression recognition in your node. I would like to add some other expressions that could be detected. If this is an option please let me know.
Thank you in advance for your answer.
Best regards,
Victoria

I believe your talking about the withFaceExpressions
You could train this detector if you like.
See: https://www.tensorflow.org/tutorials/images/transfer_learning
this will get you started on how to create and retrain models. After you retrain the model just make sure its named correctly and overwrite the one Ive included in this node.

Do you need a node-red node?
There is some better work out there now for facial recognition see: https://github.com/vladmandic/human
I've just not had the time / will / want to make a new node using this work. Have to code for work and don't have the 2 weeks of free time to just make a node based on the above link


haskell lens: don't run doctests

my proposed solution to #8522

hah, wrong change. will fix in a minute

right, that's the commit i meant to commit

:+1: I did the same change to my local nixpkgs after noticing it taking forever.

Actually, I'd rather not disable the test suite for a package that's as central as lens is. Users who don't want to run it can easily add a dontCheck override in ~/.nixpkgs/config.nix to disable the tests locally, but IMHO the central builds should execute those tests.

‘It runs too long on my machine’ is not a valid justification of skipping it for everyone and this is coming from someone who spent many hours of his life waiting for those very tests to finish.
For the most part, binary caches should deal with this.

You might want to have all builds depend on an unchecked version and add a separate checked version to catch problems on Hydra. That way you can relatively easily satisfy both objectives IMHO.

Ah I didn't realize you could set dontCheck in config. That's a much better solution.

thank you all for your responses. i am closing this at this point as it does not seem desired


TMX corpus crash

I got an error one step before ...
I will dig what's wrong in my TMX (export from studio 2015).
But it would be good if the raised exception could be specific on what translation unit it crashes
INFO: (1 of 7) TMs clean-up...                                         DONE (in 21s)
ERROR Unexpected exception:
Command 'java -cp /home/moses/ModernMT/build/mmt-0.14.jar -Dmmt.home=/home/moses/ModernMT -Djava.library.path=/home/moses/ModernMT/build/lib eu.modernmt.cli.CleaningPipelineMain -s nl -t en --output /home/moses/ModernMT/runtime/nl-en-belgique/tmp/training/clean_tms --input /home/moses/ModernMT/data/nl_NL-en_UK' failed with exit code 1
Traceback (most recent call last):
File "./mmt", line 583, in
main()
File "./mmt", line 561, in main
actionscommand
File "./mmt", line 383, in main_create
debug=args.debug, steps=args.training_steps, split_trainingset=args.split_corpora)
File "/home/moses/ModernMT/cli/engine.py", line 193, in build
unprocessed_bicorpora, self._get_tempdir('clean_tms')
File "/home/moses/ModernMT/cli/mt/processing.py", line 96, in clean
shell.execute(command, stdin=shell.DEVNULL, stdout=shell.DEVNULL, stderr=shell.DEVNULL)
File "/home/moses/ModernMT/cli/libs/shell.py", line 55, in execute
raise ShellError(str_cmd, returncode, stderr_dump)
cli.libs.shell.ShellError: Command 'java -cp /home/moses/ModernMT/build/mmt-0.14.jar -Dmmt.home=/home/moses/ModernMT -Djava.library.path=/home/moses/ModernMT/build/lib eu.modernmt.cli.CleaningPipelineMain -s nl -t en --output /home/moses/ModernMT/runtime/nl-en-belgique/tmp/training/clean_tms --input /home/moses/ModernMT/data/nl_NL-en_UK' failed with exit code 1

ok I found it:
this sequence in a TU will make the preprocess crash: �

Another strange thing.
my TMX is 450k seg long.
if I cut it into 2 files it goes fine, as one file it crashes....
But I had used bigger ones in the past.

Hi @vince62s
we made some good improvements in the error logging in the latest commit of MMT, all the details are reported in the log file "traning.log". Unfortunately this was not available in previous versions.
We had some problems with very strange UTF-8 chars (like emoticons) and we found out that it was a JVM bug! So first thing I would suggest to do is update the openjdk:
$ sudo apt-get update
$ sudo apt-get --only-upgrade install openjdk-8-jdk

Then you could check if everything went well with:
$ java -version
java version "1.8.0_111"
...

If you have a Java version with suffix "_111" or higher, then you should be ok.
If this doesn't solve the problem, could you create a stub TMX with that single phrase pair that makes MMT crash, and send it to me? I will try to investigate where the problem is.
Thank you!

ok I will, should I recompile the last version of MMT too ?
because eventhough I could train my new model, it crashes every time I try to load it ....

ok then, if you can create that single-pair TMX and send me also the commands you used to create the engine, I will try to replicate (and solve, of course! :) ) the bug myself.
We haven't heavily tested "nl" tokenizer yet, but we made several bug fixes in the latest master branch, so you could try to compile from source and see if it solves the problem.

okay on issue after the other... now I can't recompile.
-- Found the Rocksdb library: /home/moses/ModernMT/build/lib/librocksdb.so /home/moses/ModernMT/build/include
-- Not Found lzma
-- Compiling with OpenMP
CMake Error at /home/moses/ModernMT/src/cmake_modules/FindModernMT.cmake:49 (message):
Could not find the following ModernMT components: common
Call Stack (most recent call first):
CMakeLists.txt:92 (find_package)
-- Configuring incomplete, errors occurred!

@vince62s just fixed! sorry for that :)

okay.
I compiled fine.
But now I realize that you changed some stuff, xconifg related so I need to rebuild my model.
right ?

Yes, unfortunately beside the config file, we also made improvements in the models that are no longer backward compatible.

ok then will run during the night. thanks.

ok only issue remaining is with these sequences
& # x 1E ;
& # x 1F ;
(of course without the spaces)
the rest seems ok.

Hi @vince62s
I've investigated the problem and I checked the XML documentation: those two XML entities are invalid in an XML document as reported here: W3C XML 1.0 - Charsets.
The question is: who's produced that XML? MMT rewrites all TMX in the first step (cleaning) so please check if the original TMX already contained those invalid sequences: if yes, the TMX are invalid, otherwise it's a MMT problem.

it's from Studio 2015, so might be an issue from SDL then.

Hi @vince62s
I've just pushed a fix into branch develop: all invalid XML entities are now replaced with whitespace.
So even if you have an invalid TMX, MMT can now parse it and read it.
Can you confirm that this fix solved your problem?
Davide

my NL config is in beta production right now, and on another building new models ....
can't test right now. do you want a small TMX by email ?
would the develop branch be fully running otherwise ?

no problem @vince62s , I've already tested the solution on a small TMX containing errors on purpose, and I've also run it on a benchmark of 300M parallel words, so I can say that the fix works as expected!
I'm closing the issue right now but if you find some other issues related to XML Char entities, please reopen it.
Thanks!

I did git pull but I am having an error. Do I need to rerun the setup or is it only in a script ?

Yes, I made some changes also in the Java part, so you need to rerun mvn clean install from src/ folder, no need to call ./setup.py script.


Use Web Crypto API instead of forge

Built in API allows usage of basic hash functions without the use of third-party library
This fix was done because the main script was not being injected unless the forge library was correctly located in dist/third_party/forge. chrome.runtime.getURL would return undefined so the callback to load the main script never fired.

thanks. (Just to clarify this was only an issue for someone who was developing extension locally, in prod version forge was included by github action.)
this is good small one purpose pr, easier to test, and review, and approve :) nicely done.
One quirk: in my tests, the code doesn't wait for createHunterIdHash() function to finish before continuing. Meaning that it's a race condition with other things that use user_id. (might be my tests are not accurate, i just added console log after function call, and it fired first instead of waiting, let me know if you get different results.)

works ok in my tests

thanks


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

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

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

Questions ?
Please refer to the Backport tool documentation


M3 1769 Dismiss domain create drawer

Purpose
This PR makes is so that DomainCreateDrawer closes upon success (after creating or cloning a domain).
Why so many changes?
Making this change was challenging because we navigate to DomainDetails upon success. BUT at the same time, we also need to close the drawer.
We currently have two instances of DomainCreateDrawer. The one mounted on AddNewMenu plays nicely with this change, because that component is top-level, i.e. the routing change didn't affect it.
But the drawer mounted on DomainsLanding doesn't work with this change, because once the routing changed happens, the drawer isn't mounted anymore. This means that it simply disappears, which looks very choppy.
Solution
My proposal is to have ONE DomainCreateDrawer which is at the App level (similar to VolumeDrawer). This means routing changes won't affect it.
My hope is that in addition to smoothing out the transition, this is better to maintain.
Notes
As this is my first foray using Redux in this codebase, feel free to especially rip this one apart.

Functionality works well. Just left some comments about syntax.


Platform/Network Images will begin loading in the current animation frame on the web

Flutter Web uses XHR to load both network and asset images. While the actual load happens asynchronously, constructing and sending the XHR is surprisingly expensive and is currently performed in the animation frame. This can potentially delay the frame end and subsequent animation fames if the app is under heavy load - a common circumstance if the image load is triggered by something like a route navigation.
In the following example from the new gallery, we can see the xhr events are responsible for about ~6.5 ms in a single frame

On the other hand, if we defer them to a separate event loop (using Future.delayed), then they have no impact on the frame workload:

This has the advantage of not blocking animations under heavy load. This may impact Flutter on mobile and desktop as well, but since the initialization of image loading seems less expensive it may not be worth it to further defer image loads.

@goderbauer @yjbanov

Hard to capture exactly, but here you can see the xhr start in didChangeDependencies:

@goderbauer and I talked a bit about scheduling today - ideally it would be nice if the scheduled task was deferred until the app was no long under heavy load - but I don't know what guarantees are possible today.
I believe on mobile/desktop we prioritize events triggered via input/frame on the engine, but on the web I don't think we can let an animation frame take precedence over Timer.run?
since these events already take place outside of the frame workload it is difficult to tell if a frame would be missed by running them


Deployment to sepolia testnet Failed

System information
Ubuntu
Commit id
x
Tools versions
N/A

Description & steps to reproduce

1. I followed all the steps as mentioned in docs to deploy on sepolia but i am facing the above errors
2. I modified the salt , I cleaned the previous enclaves
3. I have funded good enough for the accounts
Desired behavior
Should deploy to sepolia testnet
What is the severity of this bug?
Critical; I am blocked and Kurtosis CDK is unusable for me because of this bug.

Issue has been resolved


➕ Upgrade from Newtonsoft.Json to System.Text.Json

@NoobNotFound can you take a look at where I might have gone wrong?


Improve toolchain detection for samples executor

The samples executor needs to know about the various jdks as some of the samples use toolchains in different versions. AvailableJavaHomes already knows about the custom locations in our infrastructure so just reuse those installations.

@blindpirate This fixes the issues for not finding JDK14 in some agents despite them being installed. What's left is that some windows agents still miss jdk11 (either not installed or we somehow can't find them)

I've disabled the tests on master for now (7f7818f398fbf06753fb89a1e39e5d9159c7b7af). Let's discuss how to proceed with this

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


"az login --identity" doesn't show subscription name

Describe the bug
When I use MSI to login az cli, it hide all the subscription name:
az login --identity
EnvironmentName    IsDefault    Name    State    TenantId

AzureCloud         True         MSI     Enabled  xxx
AzureCloud         False        MSI     Enabled  xxx
All subscription names are shown as "MSI", not sure if this is by design.
To Reproduce
run "az login --identity"
or "az account list"
Name    CloudName    SubscriptionId                        State    IsDefault

MSI     AzureCloud   xxx  Enabled  True
MSI     AzureCloud   yyy  Enabled  False
Expected behavior
It should show subscription name same as regular login user.
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)
Installed from ubuntu apt-get,
Ubuntu 17, bash,
az --version
azure-cli (2.0.46)

this is by design for the reason that is no longer very legit (creds-sharing with CLI 1.0). I will fix it.


Remove a duplicate command line in the "Deploying a smart contract to a channel" doc

Type of change

Documentation update

Description
I noticed there were two lines of the same command exporting the same environment variable CORE_PEER_TLS_ROOTCERT_FILE in the section Upgrading a smart contract of the Deploying a smart contract to a channel doc when I was reading it. So I removed one of them.
Additional details
nil
Related issues
No related issues. Quite a small change.

Thanks for your contribution! Could you please follow the process here
https://github.com/hyperledger/fabric/pull/3305/checks?check_run_id=5720682565
to sign your commit?


I wanna change Lidar attributes!

I use 0.9.8

And I do
make launch
make PythonAPI
make rebuild

But it doesn't change.
What should I do?

@Jae-Seung-Jeon Are you still experiencing this issue?
@DSantosO Could you follow up on this please?

Hello @Jae-Seung-Jeon,
The default values are not the ones in that file that you show but are defined in the file ActorBlueprintFunctionLibrary.cpp from line 820 .


[resolves #287] Usage of camel-jaxb depends on TCCL

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/190/
Test PASSed.


NoneType' object is not iterable in resolve_fw_decl

>>> import typing
>>> from pytypes import type_util
>>> T = typing.TypeVar('T')
>>> class Foo(typing.Generic[T]):
...   pass
...
>>> type_util.resolve_fw_decl(Foo)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'NoneType' object is not iterable

When posting issue-related stacktraces would be helpful to set pytypes.clean_traceback = False (right after pytypes import) to make the pytypes-internal part of the stacktrace visible.

Oh, didn't know about that.

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".../pytypes/type_util.py", line 905, in resolve_fw_decl
    for in_tp in in_type.__args__])
TypeError: 'NoneType' object is not iterable

So __args__ is None in this case.

Alright.
Changing line 903 from
elif hasattr(in_type, '__args__'):
to
elif hasattr(in_type, '__args__') and in_type.__args is not None:
would solve this issue I suppose.
I don't have time to work on pytyes right now, so it would be great if you could test this solution (and ideally push)...

I will look into this. The #35 is in fact more problematic to me, it seems I have found also other examples where it does not work. :-(

Done: #37.

Thanks!


non-mutating slice reverse

it would be nice if .slice().reverse() (usual way to reverse without mutating) where not breaking the fp/no-mutating-methods rule (as .reverse() does).

@benjamine use 'lodash/fp/reverse'


:sparkles: Port innertube to Dart

Is it possible for you to port this library to dart so that it could be used in flutter.

Funnily enough, a couple of years back I did begin experimenting with a dart library to interact with the innertube API, however I never finished it. In case it comes in handy, here's a link to the implementation I started working on - https://github.com/tombulled-archive/dart-youtube
I'm definitely not opposed to developing and maintaining a dart version of this library, however unfortunately I don't believe this is something I'm likely to get around to in the immediate future.
In the meantime, if yourself or anyone else would like to get started on a dart implementation, I'd be more than happy to assist/support :slightly_smiling_face:

I will do work on that, can you explain me how the post request is being sent, eg: what headers and what data is to be set

Sure, so the main URL you'll need is https://youtubei.googleapis.com/youtubei/v1/, this is the "base URL" that you'll be able to add the endpoints onto the end of.
YouTube uses this concept of clients, which represent the device/system talking to the API. You'll need to specify both a client name and a client version. I've got a decent list of these available here: innertube/config.py#L26. For example, to imitate YouTube in a web browser you'd use WEB, or to imitate YouTube Music on an iPhone, you'd use IOS_MUSIC.
In terms of request headers, you can see the headers I set here: innertube/models.py#L58. I don't think they're all essential to set, however I'd recommend setting them to avoid any bot detection mechanisms Google might employ in the future.
From my testing, Google doesn't appear to be checking the contents of the User-Agent header, however it's probably best to fake the user agent in case they do in the future. For some example user agents, check out innertube/config.py#L11.
In terms of the request data, you'll need to use a combination of query parameters and a JSON body that you'll POST. To get an idea of which query parameters/body fields you'll need to use, check out innertube/clients.py#L41

The easiest way for you to understand how interacting with the InnerTube API works is to have a fiddle with it yourself. You can do easily do this by making use of your browsers Developer Tools. If you open the Network tab and look for any POST requests made to https://www.youtube.com/youtubei/v1/, you should be able to inspect the request headers, body etc.

I'm going to close this issue as Won't do for now as it's not something I'm able to commit time to at present


CHAR: Fix OTGR dependency order and activation

Closes #3795

looks good to me, thanks 👍

Wow, thanks for this PR!


Use consistently ' for @options['

The mixed usage of @options[' and @options[" makes it difficult to search
for options.

OCD run https://ocd.appscale.com:8080/job/Daily Build/1352/


Expecting a function in instanceof check, but got #

Happens at
exports.resolve = resolve;
function resolve(value) {
  if (value instanceof this) {
    return value;
  }
  return handlers.resolve(new this(INTERNAL), value);
}

I'm using Opera 36 (the newest, Chromium-based), it seems to expect this to be a function, but it is
Object {
  all: all(iterable)
  default: Promise(resolver)
  race: race(iterable)
  reject: reject(reason)
  resolve: resolve(value)
  __proto__: Object
}

ah, this assumes that resolve is a method of the Promise function, looks like you have some sort of other module loader that is changing it to default

I don't understand. Could it be Babel doing wrong things? I don't remember exactly, but I think I tried to isolate the error and run it directly in the browser console, but I hadn't considered the module loader possibility, so I probably didn't check for it, but I will.
Also, I replaced lie with bluebird and it worked -- just after that I decided to open this issue.

so this is supposed to run as Promise.resolve, that does not seem to be the case here so try importing it as import Promise from 'lie' assuming that isn't what your doing now.

I was indeed importing it with import Promise from 'lie' and running it with Promise.resolve().then(...), that was exactly when it crashed.

hm then babel is doing something weird, though I should probably simplify
how the export is done
On Thu, Apr 28, 2016 at 9:55 AM Giovanni T. Parra<EMAIL_ADDRESS>wrote:

I was indeed importing it with import Promise from 'lie' and running it
with Promise.resolve().then(...), that was exactly when it crashed.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/calvinmetcalf/lie/issues/32#issuecomment-215431139


Observable.foreach should not fork

Currently, because Observable.foreach is based on foreachL(cb).runAsync, it forces an asynchronous boundary due to the current behavior in Task.create (to be changed in issue #288).
Regardless of issue #288, we need to ensure that Observable.foreach is approximately the same as normal subscribe().

Merged on master.


Failed to upload image when error occured

i got an error when uploading a file
updated the file locally (same filename)
wanted to press save but button was disabled

Error was that the file had an invalid resolution