XUnit Reporter Throws Fatal Error "test.isPending is not a function"

The XUnit reporter (/lib/reporters/xunit.js) fails on line 134 when attempting to call test.isPending(). The error reported is: Fatal error: test.isPending is not a function.
When I inspect the test object passed into XUnit.prototype.test, I do not see isPending defined on the object.
If I modify the else if condition to typeof test.isPending === 'function' && test.isPending(), all tests complete successfully and the XUnit report is written correctly.
What I cannot determine (at least not without a little help) is if this is a valid error or if this is a downstream effect caused by something my code is doing incorrectly.
I’m calling mocha from grunt-mocha for client side tests. If this is an upstream error on their side I’m happy to help track it down.

Can you narrow it down to a minimal test case that reproduces the issue? The XUnit reporter is working ok for me. As far as I'm aware, isPending is definitely supposed to be a function and not ignored by the reporter, but there could be some unusual failures that would try to report something that's not a normal test. (In the meantime I will see if I can dig up the place I previously saw such failures.)

I created a small sample app that recreates the issue. I'm running Node 4.4.1 and npm 3.9.5, if that helps any.
To run it, cd to test/client/config/, do a npm install then run grunt for the Spec reporter, which completes successfully. Then to see it fail run grunt testJenkins --force and you’ll see the fatal error test.isPending is not a function.
Mocha XUnit Reporter Sample
https://github.com/matthewriley/mocha-xunit-reporter-sample
Please let me know if I provide any further detail to help diagnose this error.

Could this be made into two separate tests, a Grunt test without Angular and an Angular test without Grunt?

I've narrowed it down. Used this test file: describe("suite",function(){it("test",function(){})}) Took the other tests and the references to Angular out of testrunner.html. Still got the error. Changed to testing the JS file directly with "grunt-mocha-test" instead of "grunt-mocha", did not get the error. So it's something to do with "grunt-mocha" or one of its components...

Correpsonding ticket on grunt-mocha: kmiyashiro/grunt-mocha#163

Closing in favor of grunt-mocha issue listed above.

Cool; glad I could help. Thanks for the test code!

I encounter the same problem when writing custom interface.
thanks @fry2k points out the change is from this commit.
I found the root suit (for example, this) is not instanceof "mocha/lib/suite".
so isPending is not on its prototype chain. it this the root cause?

I encounter the same problem when writing custom interface.

Can you provide an example of custom interface code that will cause the issue? It would be great to narrow down what sort of code will run into problems.

...the change is from this commit.

However, it doesn't break Mocha per se (that is, Mocha is setting up isPending on everything it should): I can run the same test code with Mocha and with various integrations, and only certain integrations are broken.

I found the root suit (for example, this) is not  instanceof  "mocha/lib/suite".

For anyone reading this thread and not the other one, I reply to this here: https://github.com/kmiyashiro/grunt-mocha/issues/163#issuecomment-227011271

Here is the interface I'm working on: https://github.com/cades/respec-given
And the file defines interface:
https://github.com/cades/respec-given/blob/master/mocha/index.js
version 0.0.6 already have the workaround applied, you can use 0.0.5 to re-produce that error.

Thanks! I will take a look at it.

My tests are passing using<EMAIL_ADDRESS>without issue, both with the default reporter and with the XUnit reporter.

It seems like the global install plus local plugin thing would affect any commandline Node program that takes plugins. We should find out if there's a standard solution for it rather than trying to hack together some custom workaround. (Note that the standard solution might turn out to be just "don't use local plugins with global commands, make both local or make both global." But we should find out.)
I will look into whether the original Grunt-Mocha issue is being caused by some kind of double-versioning or discrepant copies of Mocha thing among its local installs, however, since I confirmed the issue for Grunt-Mocha using all local installs.

The Grunt-Mocha issue is not caused by multiple differing versions of Mocha; when I install the test provided by matthewriley and reproduce the issue with it, there is only one copy of Mocha installed.

Does the installed mocha version is same as the globally installed one (you can check it with mocha --version)?

as for the standard solution, I think gulp and gulp-cli is a good example. They use the approach of "separate globally installed command and locally installed module". gulp-cli's responsibility will find locally installed gulp module and use it to consume gulpfile, so there's no concern of version conflict.

Does the installed mocha version is same as the globally installed one (you can check it with  mocha --version )?

What globally installed version?
If you're referring to this:

The Grunt-Mocha issue is not caused by multiple differing versions of Mocha; when I install the test provided by matthewriley and reproduce the issue with it, there is only one copy of Mocha installed.

...then I can clarify that I did not use use a globally installed Mocha, instead using the locally installed one that the project had as a dependency. (In fact, I did not even have Mocha globally installed, hence my saying that there was only one copy installed.)

I got it. I suppose everyone has one mocha globally installed, now I know it's wrong. Sorry for my misunderstanding.


Fix Stylelint issues

Description
Fix issues that Stylelint throws. This is the second step to onboard Stylelint.
Issues Resolved
Paritally resolve #630
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

There are still a few issues:
public/components/metrics/index.scss
 17:1  ✖  Modifying global selector ".mainContentTabs .euiResizableContainer" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

public/components/notebooks/index.scss
  7:3  ✖  Usage of property "font-family" is not allowed. (no_restricted_properties)  @osd/stylelint/no_restricted_properties
 24:3  ✖  Usage of property "font-family" is not allowed. (no_restricted_properties)  @osd/stylelint/no_restricted_properties

public/components/event_analytics/explorer/explorer.scss
 17:1  ✖  Modifying global selector ".mainContentTabs .euiResizableContainer" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

public/components/event_analytics/explorer/log_explorer.scss
 7:3  ✖  Modifying global selector ".euiTabs" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

public/components/metrics/sidebar/sidebar.scss
 67:3  ✖  Modifying global selector ".euiButton__content" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

public/components/event_analytics/explorer/events_views/data_grid.scss
 291:7  ✖  Usage of property "font-family" is not allowed. (no_restricted_properties)  @osd/stylelint/no_restricted_properties
 453:7  ✖  Usage of property "font-family" is not allowed. (no_restricted_properties)  @osd/stylelint/no_restricted_properties

public/components/event_analytics/explorer/events_views/docView.scss
 16:1  ✖  Modifying global selector ".doc-flyout td.eui-textNoWrap" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

public/components/event_analytics/explorer/sidebar/sidebar.scss
 55:3  ✖  Modifying global selector ".euiButton__content" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

public/components/visualizations/charts/data_table/data_table.scss
 19:1  ✖  Modifying global selector ".custom-pagination-container .euiIcon:focus" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

public/components/event_analytics/explorer/visualizations/config_panel/config_panel.scss
 116:1  ✖  Modifying global selector ".euiComboBoxOptionsList__rowWrap" not allowed. (no_modifying_global_selectors)                                      @osd/stylelint/no_modifying_global_selectors
 120:1  ✖  Modifying global selector ".euiComboBoxOptionsList__rowWrap > div" not allowed. (no_modifying_global_selectors)                                @osd/stylelint/no_modifying_global_selectors
 136:5  ✖  Modifying global selector ".euiBadge" not allowed. (no_modifying_global_selectors)                                                             @osd/stylelint/no_modifying_global_selectors
 184:1  ✖  Modifying global selector ".color-theme-combo-box-option .euiFlexItem.euiFlexItem--flexGrowZero" not allowed. (no_modifying_global_selectors)  @osd/stylelint/no_modifying_global_selectors

How should these be handled? Can they be fixed in followup PR's?


StackOverflowError in TwoWayAbsListView.dispatchDraw

I just built and installed 1.9.1 and I'm getting a StackOverflowError when I browse my TV shows in a Grid layout.
The exception happens when I move the selection all the way right till it scrolls the show list over.
I added some debug info to help understand what is happening. The problem is that the code creates a deep view hierarchy that looks like this:
01-11 22:56:03.003  9633  9633 I Alon    :  RelativeLayout<PHONE_NUMBER>
01-11 22:56:03.003  9633  9633 I Alon    :    FrameLayout
01-11 22:56:03.003  9633  9633 I Alon    :     FrameLayout
01-11 22:56:03.003  9633  9633 I Alon    :      FrameLayout
01-11 22:56:03.003  9633  9633 I Alon    :       FrameLayout
...
01-11 22:56:03.003  9633  9633 I Alon    :                                          RoundedImageView<PHONE_NUMBER>
01-11 22:56:03.003  9633  9633 I Alon    :                                          BadgeView<PHONE_NUMBER>
...
01-11 22:56:03.007  9633  9633 I Alon    :     BadgeView<PHONE_NUMBER>
01-11 22:56:03.007  9633  9633 I Alon    :    BadgeView<PHONE_NUMBER>
01-11 22:56:03.007  9633  9633 I Alon    :   ImageView<PHONE_NUMBER>
01-11 22:56:03.007  9633  9633 I Alon    :   ProgressBar<PHONE_NUMBER>
The exception looks like this:
01-11 22:56:03.023  9633  9633 E AndroidRuntime: FATAL EXCEPTION: main
01-11 22:56:03.023  9633  9633 E AndroidRuntime: java.lang.StackOverflowError
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.graphics.Matrix.preRotate(Matrix.java:204)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.View.updateMatrix(View.java:5881)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.View.hasIdentityMatrix(View.java:5857)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.drawChild(ViewGroup.java:2455)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2323)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.drawChild(ViewGroup.java:2716)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2323)
...
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.drawChild(ViewGroup.java:2716)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2323)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.View.draw(View.java:9379)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.drawChild(ViewGroup.java:2718)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.ViewGroup.dispatchDraw(ViewGroup.java:2323)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at com.jess.ui.TwoWayAbsListView.dispatchDraw(TwoWayAbsListView.java:1626)
01-11 22:56:03.023  9633  9633 E AndroidRuntime:        at android.view.View.draw(View.java:9
I've narrowed it down a bit but I still don't know what the fix is:
TVShowBannerImageGalleryAdapter.java:172 Creates a BadgeView with a target that is an ImageView.
BadgeView#init() calls applyTo() which has an else fork that happens when target is not a TabWidget.
In this fork,  it adds a newly created FrameLayout to the targets parent which is FrameLayout. This keeps happening over and over and builds a deep nesting.
Not exactly sure why yet.
Running on a Sony BRP (original model).

OK, I think I know what the problem is:
TVShowPosterImageGalleryAdapter.getView() uses convertView which already had it's ImageView re-parented and it reparents it again, deeper in the hierarchy.

Okay. Instead of re-using the convert view, which we could just tell it to create a new view.  Which should then fix that issue.   Evetually, I want to replace the Gallery view with the following.
https://github.com/NineWorlds/HorizontalVariableListView
If you want to make the pull request with the change to not reuse the convert view, I can get it merged into the 1.9.2 development branch.

I'm testing exactly that change right now :)

Tested OK.
It's been a while since I worked with Github and I don't remember how to sync my repository etc.
I created a patch instead:
-----%<----------------------------------------------------------------------
diff --git a/serenity-app/src/main/java/us/nineworlds/serenity/ui/browser/tv/TVShowPosterImageGalleryAdapter.java b/serenity-app/src/main/java/us/nineworlds/serenity/ui/browser/tv/TVShowPosterImageGalle
index 30ae5fe..a4f9c47 100644
--- a/serenity-app/src/main/java/us/nineworlds/serenity/ui/browser/tv/TVShowPosterImageGalleryAdapter.java
+++ b/serenity-app/src/main/java/us/nineworlds/serenity/ui/browser/tv/TVShowPosterImageGalleryAdapter.java
@@ -57,11 +57,8 @@ TVShowBannerImageGalleryAdapter {
*/
@Override
public View getView(int position, View convertView, ViewGroup parent) {

          View galleryCellView = convertView;

          if (galleryCellView == null) {

                  galleryCellView = showActivity.getLayoutInflater().inflate(

                                  R.layout.poster_tvshow_indicator_view, null);

          }

          View galleryCellView = showActivity.getLayoutInflater().inflate(

                          R.layout.poster_tvshow_indicator_view, null);

          SeriesContentInfo pi = tvShowList.get(position);

-----%<----------------------------------------------------------------------

For github, all you need to do is create a branch with your changes, push your branch to your github repo, and then in Github, you'll have an item to Submit a pull request.
I'd prefere a pull request as I can comment inline with any changes.  I suspect i'll need to make similar changes to the the Movie browser and episode browsers as well.
Regardless I'll try and get this changed merged in a day or two.

My develop branch is out of sync. I synced it but it's 6 commits ahead of your develop branch and I don't know how to fix it.

Looks like it worked alright.  So I merged the pull request.  Going to close this issue.  Thanks for the fix.


Validate OPTIONS argument

Scope & Purpose
Ensure that the argument to an AQL OPTIONS clause is always an object which does not contain any dynamic (run-time) values.
Previously, this was only enforced for traversal options and options for data-modification queries. This change extends the check to all occurrences of OPTIONS.
Backport of https://github.com/arangodb/arangodb/pull/12629

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :hammer: Refactoring
[x] :book: CHANGELOG entry made
[x] :muscle: The behavior in this PR was manually tested
[x] :computer: The behavior change can be verified via automatic tests

Testing & Verification

[x] This change is already covered by existing tests, such as shell_server_aql.

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11801/

Tests blue

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11860/

Tests blue.


M11075: Content blocked for localization: "article"

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/python-sdk-azure-samples-managed-disks.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

LOCBLOCKER

#sign-off

@SunnyDeng


No input for outdirectory

@pradeeban
As concore provides ability to give output in separate output directory with command:
./makestudy example/demo
In backend, build method also contains feature to take output Directory Name.
Therefore, there must be an optional input given by user in this regard.
If this issue is valid assign me this.

Yes, this is valid. We did not have this implemented because there was no integration of concore and concore-editor (until you attempted to it recently).


Bump kernel version to 4.10.11

Purpose
$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/8360182216

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/8360182216
Status: failure

Hi @sandushi, if you are to update the kernel, you need to carbon.product file as well.. You can refer a similar change in https://github.com/wso2/product-is/pull/19945.
Please update the PR with the necessary changes, and if you don't need to this change anymore, then lets close the PR.
Thanks,


Feature Request: De-select transaction row on Save

Feature Name: De-select transaction row on Save
Expected Behavior
When clicking Save on a row, I'd love for it to de-select that row. I often find myself getting annoyed that I have to go back and uncheck the row when I've moved on to another transaction, especially once that I am intending to select, since I now have two rows selected.
Steps to Reproduce the Behavior

Add/edit a transaction row
Click Save
Row should not be selected

I really like the idea and have often wished for it. Feature requests are tracked on the Toolkit Trello board. Please use the Extensions section of the YNAB forum and user bluebird8203 will create the request on Trello. The reason for using the forum and Trello is to allow many users to see the request and vote on it which allows the Toolkit developers to prioritize the work.

I agree it's annoying and would be a good feature.
@dbaldon I know I'm late to the toolkit party, so I'm just trying to get a handle on the rules and culture here, but if there is a developer willing to build this feature (e.g. me), can we just go ahead and do it, or do we have to wait for it to get enough votes on the Trello board? There are some features on Trello I think would be useful that have been languishing for months. I'd rather not maintain my own fork just to get the features I want.

You're more than welcome to develop the feature and submit a PR. No need to maintain your own fork. Personally I see writing features we want as one of the privileges of donating your time to the tool kit. At the same time I do try to balance writing new features with fixing existing issues. I want folks know we stand behind our work not to mention when the tool kit doesn't work nYNAB can really suck. :)

@datvance, welcome!
No need to wait for votes, just make sure you claim a trello card so we don't end up with two people building the same thing at the same time.
If you need access to the trello board just let me know!

This feature request has not made it to the forum or trello so I'm going to comment here.
I only see the behavior of selecting rows when editing transactions, not adding new ones. Am I missing something?
Also, it would be a nice feature on the budget screen as well where the same thing happens when editing a sub-category. Should these be two separate features, one for budget and one for accounts?
@blargity I guess access to the Trello board would be useful though I'm not sure what it means to claim a card.

You're correct that it's only when editing and not adding.
I'm not sure it's needed on the budget screen since there are no Save and Cancel buttons like on there accounts screen.  The two screens work differently enough that I've never found myself irritated that the row was still selected after editing the budget amount (the most commonly updated field). My .02

We're starting to track all enhancements in Trello.
I've added this request here. Thank you!

Thanks @joshmadewell! Also, looks like the link on the README to suggest new features is pointing to the old thread, just FYI.


P4C: Show community-college-specific text

Displays "2 year" instead of "4 year" in text around gas gauge charts when selected school's highest degree is associate.
Additions

New classes to toggle "2 year" and "4 year" text

Changes

Update a few external links to open in new tab

How to test this PR

Check out branch and run gulp
Navigate to grad path tool with community college selected, go to "Compare this school to others" tab, and verify that text around gas gauges references "2 year" instead of "4 year" colleges
e. Navigate to grad path tool with 4 year college selected, go to "Compare this school to others" tab, and verify that text around gas gauges references "4 year" colleges

Screenshots
Community college

4 year college

Checklist

[ ] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
Consider prefixing, e.g., "Mega Menu: fix layout bug", or "Docs: Update Docker installation instructions".

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Future todos are captured in comments and/or tickets
[ ] Project documentation has been updated, potentially one or more of:

This repo’s docs (edit the files in the /docs folder) – for basic, close-to-the-code docs on working with this repo
CFGOV/platform wiki on GHE – for internal CFPB developer guidance
CFPB/hubcap wiki on GHE – for internal CFPB design and content guidance

Front-end testing

Browser testing
Visually tested in the following supported browsers:

[ ] Firefox
[ ] Chrome
[ ] Safari
[ ] Edge 18 (the last Edge prior to it switching to Chromium)
[ ] Internet Explorer 11 and 8 (via emulation in 11's dev tools)
[ ] Safari on iOS
[ ] Chrome on Android

Accessibility

[ ] Keyboard friendly (navigable with tab, space, enter, arrow keys, etc.)
[ ] Screen reader friendly
[ ] Does not introduce new errors or warnings in WAVE

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Does not introduce new lint warnings
[ ] Flexible from small to large screens

I may be doing something wrong, but when I test this locally against Normandale, I'm getting both spans:

NM -- I re-ran the front-end script and restarted my dev server, and it's fine now. Sorry for the churn.

Thanks, @higs4281!


Call concurrent solver if available, to leverage multiple CPU cores

The SCIP solver supports concurrent solving, through the use of SCIPsolveConcurrent(). Note that SCIPsolveParallel is mentioned in the docs about how to run concurrent solving for version 4.0, but is marked as deprecated in favor of SCIPsolveConcurrent in the code.
For example, it is possible to build SCIP with multi-threading support on macOS, by setting TPI=tny.
However SCIPsolveConcurrent() fails when SCIP is built without parallel support rather than calling the single-threaded code.
To correctly call the suitable method for the build, there is a SCIPtpiGetNumThreads() method in tpi.h, that returns 1 when no threading support is available. So we could check if the return value is 1 and call the single threaded solve, otherwise call SCIPsolveConcurrent().
I'll include a PR, however I have little experience with bindgen and it appears SCIPtpiGetNumThreads() does not currently have ffi bindings.. Any idea how to add that?
cc @mmghannam if you might know.

@yonch I added an issue in scip-sys to include this and other functions for SCIP's parallel interface, I included an idea in the issue of how to fix this, if you're interested to work on it I'd be happy to review the change.


Autoconfiguration file for Spring Boot 2.7+ / Required by Spring Boot 3

Spring Boot 3 will not support
org.springframework.boot.autoconfigure.EnableAutoConfiguration in spring.factories
See: https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.7-Release-Notes#auto-configuration-registration
See: https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.0.0-M5-Release-Notes#auto-configuration-registration

I think this is all that's necessary for Spring Boot 3 compatibility.
If you could please release a new version once this PR has been merged, it would be very helpful for us trying out Spring Boot 3. :)
Thank you in advance!

Thanks. Looking at it. Your PR is not removing the entry from the (now deprecated) spring.factories file which means it is now in both places. This is good in the sense that then Spring Boot 2.6 will still work, but I need to test if Spring folks have guarded themselves against double invocation in the situation where a class is mentioned in both places.

@candrews :

I think this is all that's necessary for Spring Boot 3 compatibility.

He-he. Never that simple. For Spring Boot 3 a lot more was required.  Should have guessed. :-)
FYI: List of changes to accommodate Spring Boot 3:  diff
I've released version 1.3.0-RC1  which supports Spring Boot 3.  It is currently on a branch of its own. Please give this new version a spin.
Thanks for nudging me to update the project to Spring Boot 3.


allow passing additional dynamic headers to all TIN api calls

Is your feature request related to a problem? Please describe.
Yes - when we have multiple users hitting TIN's server via the app, its impossible to distinguish their calls in our logs
via the extension when we make our API calls, we supply a few headers that we create a composite logging key, and we can follow an entire request through the logs
Describe the solution you'd like if possible share the expected EDL
Allow for us to set headers (some static, some dynamic) before the invokeAPI call
Describe alternatives you've considered
Additional context could be screen shot, discord conversation, figma file
here, you can see there are 2 users, and one of the users made multiple API calls
and this allows us to trace either a request or a user's path

you can set as many headers as you like on API calls. Dynamic or statically typed in EDL

oh -- i thought @vusters mentioned this wasnt supported?
how can i do this?


How to generate result of validation and resolution.

Hi author:
I am using your hunyuan finetuning bash to finetune lora on my own dataset with original resolution of 1080p. But I find your model can only run on video with both height and weight can be divided by 32. Can the model also be trained  on video with 360p or 720p and why?

It is possible to train with any resolution of your choice. You can specify different resolution by using the --resolution_buckets flag. The arguments must be of the format {num_frames}x{height}x{width}.
For example: --resolution_buckets 49x512x768 81x512x512 161x768x1024. The resizing will automatically try and force your video into these resolution buckets based on whatever would fit best


Add region copy methods

As mentioned in #266 I'd like to add copy methods for regions.
I looked around a bit, e.g. here and here, and there's __copy__ and __deepcopy__ that we could implement.
However, looking at SkyCoord, that doesn't seem to be implemented there. So I'd suggest to follow that class, and add a copy method for regions.
@astrofrog @keflavich @larrybradley  @sushobhana or anyone interested - could you please have a look at the one example method and test fr CirclePixelRegion.copy here? Is it OK, or would you suggest some different implementation or test pattern?
I'll wait a day here for suggestions before going ahead and adding copy methods for many region classes.

LGTM, carry on!

@keflavich or @larrybradley - Could you please review this?
I've added copy for circle, ellipse, rectangle and polygon here.
And would prefer to continue with other shapes in future PRs.

We could also implement copy in a generic way if we have a list of data fields for each class. And we could allow changes to be made on copy, which is very commonly what people want: make a copy, but change a few fields. I'm not sure if this generic copy written once in the base class would work for now, for the annuli classes (see #279).
See example implementing this proposal for one class: https://github.com/astropy/regions/commit/93c36566360b1b4e2c556c59ae5192b71d1213b9
This is similar to what replace does for dataclasses

I'll merge this in now, because I've gone ahead and started to implement rotate (see #265) which requires this copy functionality. I'll open a new PR for that shortly where I also change the implementation of copy. Comments still welcome any time!


Crash [Win 8.1]

After I launch atom, as soon as I click the tree dependency menu [1], it freezes and crashes into an error message [2]:
[1]

[2]

Doesn't happen every time and reloading seems to fix the problem when it happens.
OS: Windows 8.1

Does it still happen if you open atom in safe mode (atom --safe)?
Do you see this every time, or only on occasion?
Does it happen only with this specific folder or also all kinds of other ones?

It's on occasion, I only managed to repeat the error once purposely, it happened 3 times in total.
I haven't yet had this issue with other folders but I don't think it's a specific folder problem since the crash happened without opening any of the folders. Clicking the area that I included in the red square was enough to crash.

Clicking the area that I included in the red square was enough to crash.

I think this may be significant.
I don't know what the tree view does when it gets focused, but we know it generally scans the project folder and some people have reported issues with very large and deep folder trees.

Are you still running into this issue on newer builds? If so, please re-open.


Schedular: Frequently sending emails

For weekly and fortnightly it's not updated until the next scheduled time due to which is sending emails frequently.

Merged with https://github.com/openmsupply/excel-report-email-scheduler/pull/108

This needs to go into haiti too


Add ensure macro?

Have you though about adding an ensure! macro as well in the same vein as the one in failure?
We use failure today and with quite a bit of ensure! which I find makes quite compact and explicit code, and has a nice naming connection to assert!.
Would you be open to adding ensure! to this crate?

Thanks @repi, good call. I would accept a PR to add an ensure macro.

Thanks, I'll look into it.


perf: optimize IO path for reading manifest

Fixes #2338
Partially addresses #2318
TODO

[ ] Cleanup
[ ] Check repro from 2318
[ ] Make comparison of traces for opening a dataset
[ ] Add IO unit test for loading a dataset

Codecov Report
Attention: Patch coverage is 64.77987% with 56 lines in your changes are missing coverage. Please review.

Project coverage is 79.67%. Comparing base (8c1ee00) to head (5c564bf).
Report is 4 commits behind head on main.

Files
Patch %
Lines

rust/lance-table/src/io/commit.rs
60.29%
21 Missing and 6 partials :warning:

...ust/lance-table/src/io/commit/external_manifest.rs
50.00%
9 Missing and 4 partials :warning:

rust/lance/src/dataset.rs
73.17%
11 Missing :warning:

rust/lance-io/src/local.rs
77.77%
0 Missing and 2 partials :warning:

rust/lance/src/index.rs
0.00%
0 Missing and 2 partials :warning:

rust/lance-io/src/utils.rs
85.71%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2396      +/-   ##
==========================================
- Coverage   79.99%   79.67%   -0.32%
==========================================
  Files         200      200
  Lines       54519    55015     +496
  Branches    54519    55015     +496
==========================================
+ Hits        43612    43833     +221
- Misses       8389     8642     +253
- Partials     2518     2540      +22

Flag
Coverage Δ

unittests
79.67% <64.77%> (-0.32%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Unable to connect to POSTGRESQL for running an script

Hi,
I'm trying to run a script in my connection to Postgresql.
when select run the log shows an error saying
psql unrecognized option --no--beep
I've tried to change the beep option (flagged and unflagged) without success.
I've typed as suggeste psql --help but no information found
Versions:
DBeaver: Version 4.0.6
PostgreSQL: Version <IP_ADDRESS> (<IP_ADDRESS>

It is a bug in PG script exec wizard (it messes psql with pg_dump)

Hi again,
I downloaded version 4.0.7.  and I get the same error. I'm not sure if I'm wrong.  what information do you need to evaluate it?
thanks

This bug i fixed.
Fix will be included in 4.1.0 (see milestone info).

thanks. I misunderstood it putting the resolution in 4.0.7
Thanks!!


PL doc updates for grid and reorganizing features

Separated the grid types into separate views within the PL. Moved count
status, count status grid and page header under features as they all
contain multiple components.

re: flex grid — I'm going to be thinking of this while I work on the marketplace panel (and on the page headers) today


[WSL] No such file or directory when setup environment

Windows Build Number
Microsoft Windows [Version 10.0.19041.1110]
WSL Version

[ ] WSL 2
[X] WSL 1

Kernel Version
4.4.0
Distro Version
Ubuntu 18.04
Other Software
No response
Repro Steps

append new line export NODEJS="/mnt/d/wsl/node" to the end of  ~/.bashrc
start wsl and appear: -bash: export NODEJS=/mnt/d/wsl/node: No such file or directory
but is avaliable to access this directory in wsl, see the screen capture

Expected Behavior
use file (outer of wsl) as environment variables of wsl
Actual Behavior
fail to setup
Diagnostic Logs
No response

i restore the ~/.bashrc and redo these again, and it works!
i can make sure that i alway use LF instead of CRLF in ~/.bashrc, maybe it is due to \n and \r\n ?


[SPARK-49918][CORE] Use read-only access to conf in SparkContext where appropriate

What changes were proposed in this pull request?
This PR switches all calls to SparkContext.getConf that are read-only to use SparkContext.conf instead. The former method clones the conf, which is unnecessary when the caller only reads the conf. SparkContext.conf provides read-only access to the conf.
Why are the changes needed?
Cloning the entire conf adds unnecessary CPU overhead due to copying, and GC overhead due to cleanup, and both affect tail latencies on certain workloads.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing tests.
Was this patch authored or co-authored using generative AI tooling?
No

The GA jobs actually all passed. Thanks, merging to master!


About distance map

Hi JunMa,
Thanks alot for your code.
I am just wondering as to boudary loss, should the distance map be a signed distance map (negative inside and positive outside the boundary) or an un-signed distance map (positive both inside and outside)?
Thanks in advance.
Cheers.

Hi,
You can find a clear answer in the original repo
https://github.com/LIVIAETS/boundary-loss


⚠ Telly has degraded performance

In 802e154, Telly ($TLY) experienced degraded performance:

HTTP code: 200
Response time: 1372 ms

Resolved: Telly performance has improved in 08d34de.


Responsive media object

Working with the "Media Object" has been extremely helpful in the past, but there are a couple of issues which I kept encountering, having to fix them with every project.
1 - The "media-left" part seems very close to the content. Perhaps increasing the default padding-right a bit would make things look more clean and readable?
2 - Responsiveness seems a bit tricky on mobile, especially when the media object is a bit wider than default. Perhaps a class that turns the "media object" into something more responsive would be good? Maybe a "media-responsive" class which would make the mobile version not use the table display model and moves the "media-left" above the "media-body"? For example, when using a larger media-object item like the following, the page layout will be really broken:

There are cases where we might have to use larger media-objects - like comment sections, an "about author" section on a blog post, etc.

3 - Add additional configuration variables for the left/right paddings

+1

CC: @mdo

May I piggyback on this as I've just encountered an issue with the media object as well:
3 - "media-right" does not seem to consistently work where "media-left" works in the examples in the documentation. If you take the very first example and replace media-left with media-right, it looks like this:

Same thing happens in the subsequent examples where a div is used as a wrapper. The only place media-right seems to work is inside the media-list.

+1 might I recommend implementing media-sm media-md

We won't be adding anything for this in v4 yet (at least the initial launch) and I'm unclear how much this would benefit others when you can use responsive media independent of the media classes. Put another way, there's nothing stopping you from using max-width and more with a bit of custom code to accomplish these things.

@mdo can I get a followup on this. Perhaps an example for:

using max-width and more with a bit of custom code

I'm looking to have the media object break into two rows.


Fixup: Use expanding ranges

Description
Currently fixups will use the users selected range, if it is available.
This is ok but it becomes problematic in different scenarios.
Example:
Code actions -> Do we really just need to change the code that has the error? Often we need to change much more to satisfy the error.
Refactors -> I want to change the returned value of a function, if I have only selected the return value then we should expand the range scope to include the entire function.
Implementation
I think we should investigate what the best option here is. We should look at competitors like Copilot too.
One basic idea would just be to always expand the range scope to the nearest function (if available).

At a cursory out look I saw that cody does use Prefix/Postfix text  in the comprehension of Fixup recipes.
https://github.com/sourcegraph/cody/blob/54db16dc212702ee20fb50436c02ad8849eccab7/vscode/src/non-stop/FixupController.ts#L283-L295
And even for FixupCodeAction
https://github.com/sourcegraph/cody/blob/54db16dc212702ee20fb50436c02ad8849eccab7/vscode/src/code-actions/fixup.ts#L22-L28
The broader question is that while the range expansion for input is good. How to extend it intelligently to the part where the fixup is actually applied. Like should it only be applied to EXACTLY what the user selected and if we wanna do more. Is it possible we might cause inconvenience for the user by going out of bounds?

This is a great idea, we should do it.
FWIW, anecdotally from when I built this:

Fixups performs better with the before/after context. (Unsurprising... so would a human.)
Fixups performs poorly when it would be beneficial to expand the scope. For example, you ask it to rename something but the declaration is outside the scope being edited. Typical bad behavior: repeating code outside the selection (which IIRC we now prompt to suppress); not generating output; etc.

In the first iteration I tried to let Cody push back in chat if you did that, but it was way too chatty in general so @abeatrix silenced it.
Expanding the selection to a boundary which trades off logical sense (e.g. "whole function") and amount of code sent sounds great as a first cut.
Performance can deteriorate asking it to reiterate a lot of code... we could relax the requirement to repeat all the code and use a diff to align the output we do get to the content of the file.
Hard mode: Edit multiple files/places. Might be interesting to let Cody pitch toil to the editor, for example, have it explain a multi-step refactoring with relatively abstract commands like "rename variable x to y" and then we do those precisely.

Closing as we've had multiple improvements here now (many thanks to @arafatkatze)

Thanks @umpox so glad to see how far Cody has come!!


Lo/mid-fi wireframes

Start laying out the rough layout for the app.
everything is in pace, would like some feedback on the desktop weekly view. how to improve and stay away from the excel sheet look.
just copy the mobile version ?

Yes try to do that, Frederik! Good idea:)

Perfect!

Added comment to the weekly task frame for you to discuss

I have impemented changes based on feedback and comments.


Add a markdown linter

Some rules

max length for lines
some basic check for links & headings

https://github.com/DavidAnson/markdownlint

:+1: good idea
I guess it will improve the diff & review

Yep top

nice idea!

We definitely need this.

Closing in favor of #452


Add simple loading screen to all pages (except the landing page)

Issue summary
Authenticating the current user is not instantaneous, and so for a split second the user is able to see the page load before Firebase realizes that they are not currently signed in and kicks them back to the landing page. This may be a problem if the user is able to, even temporarily, see the details of a trip itinerary. A fix for this could be a simple loading screen that hides page contents, which is only removed after the user has been authenticated.
Expected behavior
Page details are completely hidden from the user until they have been authenticated.
Actual behavior
Page continues to load even while the user is still being authenticated.

UPDATE: Using React, the page is simply not loaded until the user is fully authenticated. The browser temporarily shows a blank screen that says "Loading..." while this happens. This is left in for testing purposes, in the deployed build the loading screen should be made blank (since the authentication happens relatively quickly, the split second blank screen should be fine).
The title of this issue has been renamed from "Add simple loading screen to all pages (except the landing page)" to "Remove Loading text from the loading screen in production build".


Result image gets distorted

When uploading an image that's zoomed all the way out (or close to it), the result image sometimes becomes stretched and distorted. The further out I zoom, the more distorted the image becomes. Not sure what's causing it or how exactly to recreate the issue since it only happens with certain images.
As far as I can tell it happens on Chrome, Firefox, and IE. Has anyone else experienced this? Any ideas?
Max zoomed out (with distorted result)

Zoomed in, working normally

We need more information:

How are you getting the result image ... base64? blob? html?
How are you rendering the result image?  An img tag?  Does it have a width and height?

I have the same issue as you, I'm using croppie 2.6.2, so it happen when I zoomed all the way out and drag until viewport exist the bottom of the image, then it will happen.
And here are some of the code:
var resultWidth = 900, resultHeight = 1200;

uploadCrop.croppie('result', {
  type: 'base64',
  format: 'jpg',
  size: {
    width: resultWidth,
    height: resultHeight
  }
})

It seems like the result is not getting the correct width and height of the setting maybe? not so sure.

I'm having a similar experience. @AlexanderLyon How exactly are you swapping these variables?

@trevorlangston at the bottom of the _rotate() function, I created temp variables to store the old orientations, then just reassigned them to each other:
function _rotate(deg) {
  if (!this.options.useCanvas || !this.options.enableOrientation) {
    throw 'Croppie: Cannot rotate without enableOrientation && EXIF.js included';
  }

  var self = this,
    canvas = self.elements.canvas,
    ornt;

  self.data.orientation = getExifOffset(self.data.orientation, deg);
  drawCanvas(canvas, self.elements.img, self.data.orientation);
  _updateZoomLimits.call(self);
  _onZoom.call(self);
  copy = null;

  // Reverses image dimensions:
  let oldHeight = self._originalImageHeight;
  let oldWidth = self._originalImageWidth;
  self._originalImageWidth = oldHeight;
  self._originalImageHeight = oldWidth;
}

After applying the fix, I had some issues dragging the picture after a rotation. Moving the following two lines :
_updateCenterPoint.call(self, true);
_updateZoomLimits.call(self);

After updating self._originalImageWidth & self._originalImageHeight seems to fix them.

Do you need an extra check that the "deg" is divisible by 90/270/-90 etc?
Something like
if ((Math.abs(deg) / 90) % 2 === 1) {
   ...
}


adding missing dep on jdk for gen_ops.bzl (again)

Redoing changes in https://github.com/tensorflow/tensorflow/pull/12827
(undid by merge)

Can one of the admins verify this patch?

Closing as change has been submitted internally.


Manager should be an SSH server itself

Since Kitodo.Production will unlikely have access to a Docker installation (i.e. be able to docker run ocrd_manager something from a script task) or even a native OCR-D core installation (i.e. be able to install core in the system and then sh initialsetup.sh something) – we should use the same base recipe as in https://github.com/bertsky/ocrd_controller.

[x] install openssh-server in Dockerfile, provide unprivileged access for pseudo-user ocrd
[ ] provide some callable (say: shell script taking parameters) which will

(perhaps: transfer the process data / Vorgangsdaten to the controller)
log into the controller and run the predefined workflow
(perhaps: retransfer the results from the controller)
post-process the results by

validating the whole OCR-D workspace
determining which are the result fileGrps (from either the workflow definition or the position of the fileGrps in the METS or the timestamps of the subdirectories)
copying/moving the result files (for now: ALTO, later: PAGE / PDF / TEI / ...) to a path in the process directory where Production expects them (ocr/%08d.xml I guess), by iterating over the METS with ocrd workspace find

signalling the exit status via ActiveMQ

[ ] provide a smoke test for the callable (preferably via a Makefile):

start up the server
download test data
call the controller
(perhaps somehow receive/register the signal)

@bertsky added the openssh server and the pseudo-user implementation similar to ocrd-controller.


Add Tinder Clone

I am a GSsoc 2022 contributor and will like to clone Tinder

hey , i want to work on this , plz assign it to me

hey I would like to work on this project

Hi @sagnik2001     you can work on this issue ur project should have a working link and you should add that in the main website too once done with creating it .
Deadline -  5-6 days
after that it will be assigned to @Anish1411

hey, @sagnik2001 you crossed the deadline date now this issue is assigned to @Anish1411

@Anish1411 you have 6 days to complete

hey @gurjeetsinghvirdee , can u plz tell me i have to clone only login page or i have to clone the interface that comes after login.
if possible , send me ss , what actually i have to build

You have to build login with interface
On Sun, 13 Mar, 2022, 10:17 Anish Kirodiwal, @.***>
wrote:

hey @gurjeetsinghvirdee https://github.com/gurjeetsinghvirdee , can u
plz tell me i have to clone only login page or i have to clone the
interface that comes after login.
if possible , send me ss , what actually i have to build
—
Reply to this email directly, view it on GitHub
https://github.com/Rayman-Sodhi/Clone-IT/issues/63#issuecomment-1066025735,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARSWKZPQHQQFD4KTUU7XM53U7VXMJANCNFSM5PR2XQ6A
.
You are receiving this because you were mentioned.Message ID:
@.***>

can u plz send me screen shots

can u plz send me screen shots

@Anish1411 you commented on this issue that you want to work on, the last person who worked on this issue didn't create any PR so I can't have ss something, so built on your own idea that's your choice you want to make a design or in add some functions, if you can then reply either I have to assign @neetra

okk , i'll

okk , i'll

then start your work

@Anish1411 are you still working with this?

yeah i will submit it by tomorrow

Ok
On Fri, Mar 25, 2022 at 2:43 PM Anish Kirodiwal @.***>
wrote:

yeah i will submit it by tomorrow
—
Reply to this email directly, view it on GitHub
https://github.com/Rayman-Sodhi/Clone-IT/issues/63#issuecomment-1078810226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARSWKZPFQXPORHRQVFXWKHDVBV7VJANCNFSM5PR2XQ6A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Part 3 Question 3: Rates of Harm

I'm trying to mutate all collisions to create proportions but I keep getting errors:
dat %>%
group_by(hour) %>%
mutate(harm = Totalinjuries + Totalfatalities) %>%
mutate(all_collisions = dat$Totalfatalities >= 0 & dat$Totalinjuries >= 0) %>%
select(harm, hour, all_collisions) %>%
count(harm, hour, all_collisions) %>%
summarize(n = n()) %>%
Error: Problem with mutate() column all_collisions. i all_collisions = dat$Totalfatalities >= 0 & dat$Totalinjuries >= 0. i all_collisions must be size 388 or 1, not 28470. i The error occurred in group 1: hour = "00".

I'm not entirely sure what you are trying to work out here, but a few things to keep in mind:
When you use mutate() you do not need to refer to specific variables with $. Since they are piped in via %>% the mutate() function knows which ones you are referring to by name.
I cleaned up your code and this will run, although your function for all_collisions will return a vector of TRUE because you are specifying a logical operator with >=0 (also, specifying greater than or equal to zero will equal ALL the rows because there are no negative fatalities or injuries).
dat %>% mutate(harm = Totalinjuries + Totalfatalities,        all_collisions = Totalfatalities >= 0 & Totalinjuries >= 0) %>% select(harm, hour, all_collisions) %>% count(harm, hour, all_collisions)
What I think you want instead is the count and proportions by hour, if you look at the examples, you need to use count() to get the number of injuries and fatalities by hour. For example, try starting with:
dat %>%   count(hour, Totalinjuries, Totalfatalities)


fix #359 lazy templates on nested keys

fix #359 lazy templates on nested keys

Hello @rochacbruno! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file dynaconf/vendor/box/box.py:

Line 104:33: E241 multiple spaces after ','
Line 104:80: E501 line too long (115 > 79 characters)
Line 132:80: E501 line too long (116 > 79 characters)
Line 227:80: E501 line too long (86 > 79 characters)

@rochacbruno this looks like it accidentially hcanged a vendored file, was that intended?


Fix duplicate subscriptions

Status
READY
JIRA Ticket
MOB-107
Description
This fixes a bug where mqtt would resubscribe to certain streams every time it reconnected. Now, it only sets up subscriptions on the initial connection.

+1 emoji

👍


unable to verify the first certificate at Function.AxiosError.from

Current Situation
Since start of june 2024, ewelink plugin is no more working.
I have checked user and password but they are correct becouse I can access to the site and app.
Logs
[6/19/2024, 9:56:49 AM] [eWeLink] An error occurred during http login().
[6/19/2024, 9:56:49 AM] [eWeLink] ***** Disabling plugin. *****
[6/19/2024, 9:56:49 AM] [eWeLink] ***** unable to verify the first certificate at Function.AxiosError.from (file:///homebridge/node_modules/homebridge-ewelink/node_modules/axios/lib/core/AxiosError.js:89:14). *****

Configuration
{
    "name": "eWeLink",
    "username": "***",
    "password": "***",
    "mode": "auto",
    "language": "en",
    "countryCode": "39",
    "platform": "eWeLink"
}

Environment
Docker: yes
OS: Ubuntu Jammy (22.04.3 LTS)
nodejs: v20.14.0
homebridge: v1.8.2
homebridge UI: v4.56.2
Process Supervisor
Docker (Mention image name in Additional Context)
hb-service
Additional Context
https://registry.hub.docker.com/r/homebridge/homebridge/

Seeing exactly the same problem

Okay this seems to have fixed it on my server:

Head into the Plugin Config for Homebridge eWeLink
Expand Advanced Settings
Set HTTP Host to Auto (Recommended) <- mine was set to None
Reload Homebridge when prompted

@mikeh thank you so much. it works now. cheers

Okay this seems to have fixed it on my server:

Head into the Plugin Config for Homebridge eWeLink
Expand Advanced Settings
Set HTTP Host to Auto (Recommended) <- mine was set to None
Reload Homebridge when prompted


[Play] - Container Refactors

Issue:
The current structuring of our code is overly complicated, with three containers and a condition in which gameSession is initialised with null, forcing us to do a bunch of null checks all the way down our code.
Resolution:
After speaking with Mani and Ryan, we came up with some requirements for the new structure:
Mani's recommendations:

Use only two containers
Eliminate the need for null initialisation of gameSession object

Ryan's recommendations:

use routing to split the app into something like / for preGame and /game for gameInProgress
use either route param or localStorage to transition between the two
use a secondary container to handle the api calls
perform null logic (using loading/error/data states) in that secondary container to mitigate the need for null checks in child containers

The included code implements these recommendations. Here are the important components:

PregameContainer.tsx - handles all the joining game functionality (enter gamecode etc). When user selects avatar, team is added to gameSession object on the backend and data is stored in localStorage.
useFetchLocalData.tsx - custom hook for fetching local game session data. Exported so we can use a navigate('/') without compromising rules of hooks.
useFetchandSubscribeGameSession.tsx - custom hook for fetching the current game session and subscribing to it. Returns an object formatted as { isLoading, error, gameSession} to track game loading, error state and data populating.
GameSessionContainer.tsx - retrieves local data and subscribes to game via custom hooks. Renders appropriate components based on response from subscription.
GameSessionSwitch.tsx - holds all game pages and switches them based on gameSessionState. This is the normal Game In Progress flow.

Preview:
No errors, player joins game, game loads, player waits in lobby until teacher starts game:
https://github.com/rightoneducation/righton-app/assets/79417944/626c5e2c-b961-4854-80fe-41ba667b4c18
With error, modal pops up, player attempts to rejoin, player quits out to main menu:
New video without useEffect (june 1, 2023):
https://github.com/rightoneducation/righton-app/assets/79417944/d56a53cd-6990-4feb-b5ed-1e2bb798c34d
Old video with janky useEffect (may 31 2023):
https://github.com/rightoneducation/righton-app/assets/79417944/6af25eb2-b99c-4e30-aa86-1090aac40f5b

I'm going to merge this, Mani. An error checking PR will follow for the functions in GameSessionSwitch and maybe we can talk about the righton/networking model for IGameSession later in the week.


Question: How do I use the ScratchTracker?

The docs say to use the ScratchTracker to get a good estimate for scratch space but does not explain how to use it.

The easiest way is to use a CompositeSerializer, wrap your scratch in a ScratchTracker, serialize an instance with that serializer, and then check ScratchTracker::min_buffer_size to see how much scratch space was required.
This test can serve as example: https://github.com/rkyv/rkyv/blob/58587fdf527546bedd593c649b889898c3266daf/rkyv_test/src/test_alloc.rs#L2141-L2186

Added a new section for this in 6c2e1e762e70ee07bd6b25c8f31ff0d36dcda283.


Tests fail in swift package

Hi,
I converted my library to a swift package and made the transition to SPM for all dependencies. The issue I am facing now is that all my tests are failing because of this:
Internal error when a required entitlement isn't present, client has neither application-identifier nor keychain-access-groups entitlements.
I believe it's an error because Keychain is used. Do you know how to overcome this or is this a limitation?

I'm facing the same problem. I'm investigating a solution.

Same here. Adding Keychain groups doesn't help.

Any updates? facing exactly same issue when running tests in SPM package


ACM-2857 - Add cluster version and csv object outside gather-acm.logs (release-2.5)

Signed-off-by: dislbenn<EMAIL_ADDRESS>Related Issue: https://issues.redhat.com/browse/ACM-2857
Description of Changes: Updating must-gather to capture cluster version.
What resource is being added:  | N/A

Clusterversion

Is this a Hub or Managed cluster change?:
Hub Cluster | Managed Cluster | N/A

Hub & Managed

Notes:

/hold


it shows that " Value of type 'StorageMetadata' has no member 'downloadURL' " in user.swift at 48 no line

I have the same issue.

@jamesdang @hasankamrul The Firestorage API has been changed. Working on fix. Will push in this week

Great! Looking forward to the update.

@jamesdang @hasankamrul pushed the new version.


[Service Bus] ServiceBusClient live tests Random scheme in the endpoint from connection string failing in nightly runs

Service Bus nightly test runs are failing with:

Error message:
Unexpected number of msgs found when peeking: expected 0 to equal 1
Stack trace:
AssertionError: Unexpected number of msgs found when peeking: expected 0 to equal 1
at Object.should.equal (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/chai@4.3.4/node_modules/chai/lib/chai/interface/should.js:98:35)
at testPeekMsgsLength (/Users/runner/work/1/s/sdk/servicebus/service-bus/test/public/utils/testutils2.ts:558:10)
at runMicrotasks ()
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at Context. (/Users/runner/work/1/s/sdk/servicebus/service-bus/test/internal/serviceBusClient.spec.ts:76:7)

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1265167&view=results

This is a failure in the macos run that is much slower than usual.  It is most likely machine/network issue.  Haven't seen it recently.  Closing.  Please re-activate if this happens again.


Issue with fullTemplateTypeCheck and ng serve

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

Is this a regression?
I don't know.
Description
When having fullTemplateTypeCheck enabled in tsconfig.json and running ng serve everything runs fine initially. I might, for example, get nice error messages about my template type-check issues (with colors and everything :smiley:):
ERROR in src/app/app.component.html:38:66 - error TS2322: Type 'SearchResults | null' is not assignable to type 'SearchResults | undefined'.
  Type 'null' is not assignable to type 'SearchResults | undefined'.

38 <aio-search-results #searchResultsView *ngIf="showSearchResults" [searchResults]="searchResults | async" (resultSelected)="hideSearchResults()"></aio-search-results>
                                                                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  src/app/app.component.ts:23:16
    23   templateUrl: './app.component.html',
                      ~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component AppComponent.

Then on re-compiling (e.g. after editing a file and saving), I get the following error:
ERROR in ./src/main.ts
    at plugin.done.then ([...]\angular\aio\node_modules\@ngtools\webpack\src\loader.js:41:31)
    at process._tickCallback (internal/process/next_tick.js:68:7)
ERROR in ./src/polyfills.ts
Module build failed (from ./node_modules/@ngtools/webpack/src/index.js):
Error: [...]\angular\aio\src\polyfills.ts is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
    at AngularCompilerPlugin.getCompiledFile ([...]\angular\aio\node_modules\@ngtools\webpack\src\angular_compiler_plugin.js:918:23)
    at plugin.done.then ([...]\angular\aio\node_modules\@ngtools\webpack\src\loader.js:41:31)
    at process._tickCallback (internal/process/next_tick.js:68:7)
i ｢wdm｣: Failed to compile.

I might then get some error template type-check errors, but not as nicely formatted (or colorful) as before:
ERROR in src/app/app.component.html(38,66): error TS2322: Type 'SearchResults | null' is not assignable to type 'SearchResults | undefined'.
      Type 'null' is not assignable to type 'SearchResults | undefined'.
    src/app/app.component.html(47,63): error TS2322: Type 'NavigationNode[]' is not assignable to type 'Option[]'.
      Type 'NavigationNode' is not assignable to type 'Option'.
        Types of property 'title' are incompatible.
          Type 'string | undefined' is not assignable to type 'string'.
            Type 'undefined' is not assignable to type 'string'.
    src/app/app.component.html(47,87): error TS2322: Type 'NavigationNode' is not assignable to type 'Option'.
    src/app/custom-elements/api/api-list.component.html(24,8): error TS2322: Type 'ApiSection[] | null' is not assignable to type 'ApiSection[] | Iterable<ApiSection> | undefined'.
...

After this first error, the app is not re-compiled when I make changes (I have to stop the process and run ng serve anew).
(NOTE: The error seems only (or much more often) happen when there are template type-check errors.)
🔬 Minimal Reproduction
It started happening for me in PR angular/angular#32980. To reproduce, you could:

Check out the PR.
Reset to the commit before build(docs-infra): enable `fullTemplateTypeCheck` (and fix failures) (currently https://github.com/angular/angular/pull/32980/commits/3f228a626).
Add fullTemplateTypeCheck: true in aio/tsconfig.json > angularCompilerOptions.
Run yarn --cwd=aio && yarn --cwd=aio start. (You should see some nice template type-check error messages.)
Open aio/src/app/app.component.ts, make a change and save it. (You should see the error.)

(NOTE: If that reproduction isn't helpful, I can try to create a minimal one with a new app.)
🌍 Your Environment
(NOTE: I am on 9.0.0-next.8 now, but this happened with 9.0.0-next.6 as well.)

Angular CLI: 9.0.0-next.8
Node: 10.16.3
OS: win32 x64
Angular: 9.0.0-next.9
... animations, common, compiler, compiler-cli, core, elements
... forms, language-service, platform-browser
... platform-browser-dynamic, router, service-worker

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.0-next.8
@angular-devkit/build-angular     0.900.0-next.8
@angular-devkit/build-optimizer   0.900.0-next.8
@angular-devkit/build-webpack     0.900.0-next.8
@angular-devkit/core              9.0.0-next.8
@angular-devkit/schematics        9.0.0-next.8
@angular/cdk                      9.0.0-next.0
@angular/cli                      9.0.0-next.8
@angular/material                 9.0.0-next.0
@ngtools/webpack                  9.0.0-next.8
@schematics/angular               9.0.0-next.8
@schematics/update                0.900.0-next.8
rxjs                              6.5.3
typescript                        3.5.3
webpack                           4.41.0

cc @filipesilva

I think the errors being formatted different on rebuilds is due to https://github.com/angular/angular/issues/32213.
The Module build failed error on the first rebuild is https://github.com/angular/angular/issues/32214 I believe. I tried introducing a simple template type error as shows in the docs and, and saw that it will show the Module build failed errors only if the ng serve starts with the error. If I introduce the error after the first build, everything is fine.


Some font tweaks

Improved fontface selection
Load and save text-anchor and text x and y attributes, better recognition of "class" provided font-attributes
Some tweaks around bound-box calculation for a text

There's a fundamental flaw in wxPython to get the right fontsize.
Both GetTextExtent as well as GetFullTextextent provide the fontmetric-size as result for the font-height and dont take the real glyphs into account. That means that ".", "a", "g" and "T" all have the same height...
Consequently the size is always off... This can be somewhat compensated by taking the descent from the font-metric into account.
A 'real' height routine would most probably need to draw the string on an empty canvas and find the first and last dots on a line. I was wondering whether the imagetools provide such a functionality?!

Image tools does not but it's a pretty shallow view of Pillow which almost certainly could. If you did .bbox on a pillow image of a drawn font there's a somewhat solid chance it would find the right coords for the height there. There's also some chance that a similar thing could be done more directly in Pillow. Even some font rendering might be done in Pillow.
Other alternatives, there's some chance of getting freetype working. Which is a font rendering engine. I believe it requires the freetype python addon and a copy of the free type dll file.
#!/usr/bin/env python
# -*- coding: utf-8 -*-
# -----------------------------------------------------------------------------
#
#  FreeType high-level python API - Copyright 2011-2015 Nicolas P. Rougier
#  Distributed under the terms of the new BSD license.
#
# -----------------------------------------------------------------------------
#
# Direct translation of example 1 from the freetype tutorial:
# http://www.freetype.org/freetype2/docs/tutorial/step1.html
#
import math

import matplotlib.pyplot as plt
from freetype.raw import *
from PIL import Image

WIDTH, HEIGHT = 640, 480
image = Image.new('L', (WIDTH,HEIGHT))

def to_c_str(text):
    ''' Convert python strings to null terminated c strings. '''
    cStr = create_string_buffer(text.encode(encoding='UTF-8'))
    return cast(pointer(cStr), POINTER(c_char))

def draw_bitmap( bitmap, x, y):
    global image

    x_max = x + bitmap.width
    y_max = y + bitmap.rows
    p = 0
    for p,i in enumerate(range(x,x_max)):
        for q,j in enumerate(range(y,y_max)):
            if i < 0  or j < 0 or i >= WIDTH or j >= HEIGHT:
                continue;
            pixel = image.getpixel((i,j))
            pixel |= int(bitmap.buffer[q * bitmap.width + p]);
            image.putpixel((i,j), pixel)

def main():

    library = FT_Library()
    matrix  = FT_Matrix()
    face    = FT_Face()
    pen     = FT_Vector()
    filename= 'Vera.ttf'
    text    = 'Testings !'
    num_chars = len(text)
    angle   = ( 20.0 / 360 ) * 3.14159 * 2

    # initialize library, error handling omitted
    error = FT_Init_FreeType( byref(library) )

    # create face object, error handling omitted
    error = FT_New_Face( library, to_c_str(filename), 0, byref(face) )

    # set character size: 50pt at 100dpi, error handling omitted
    error = FT_Set_Char_Size( face, 50 * 64, 0, 100, 0 )
    slot = face.contents.glyph

    # set up matrix
    matrix.xx = (int)( math.cos( angle ) * 0x10000 )
    matrix.xy = (int)(-math.sin( angle ) * 0x10000 )
    matrix.yx = (int)( math.sin( angle ) * 0x10000 )
    matrix.yy = (int)( math.cos( angle ) * 0x10000 )

    # the pen position in 26.6 cartesian space coordinates; */
    # start at (300,200) relative to the upper left corner  */
    pen.x = 200 * 64;
    pen.y = ( HEIGHT - 300 ) * 64

    for n in range(num_chars):
        # set transformation
        FT_Set_Transform( face, byref(matrix), byref(pen) )

        # load glyph image into the slot (erase previous one)
        charcode = ord(text[n])
        index = FT_Get_Char_Index( face, charcode )
        FT_Load_Glyph( face, index, FT_LOAD_RENDER )

        # now, draw to our target surface (convert position)
        draw_bitmap( slot.contents.bitmap,
                     slot.contents.bitmap_left,
                     HEIGHT - slot.contents.bitmap_top )

        # increment pen position
        pen.x += slot.contents.advance.x
        pen.y += slot.contents.advance.y

    FT_Done_Face(face)
    FT_Done_FreeType(library)

    plt.imshow(image, origin='lower',
               interpolation='nearest', cmap=plt.cm.gray)
    plt.show()

if __name__ == '__main__':
    main()

The advantage here is that it actually drew the glyphs themselves which means we would have access to the actual curves of the glyphs. It's a bunch of dll stuff and other things I don't love but which seems to work. Though there might be different compatibility issues with other OSes.

The final option would be to maybe write my own parser. I'm told it's a lot of work but all the tables and how to parse the stuff is already online and it seems like the majority of this is the glyph tables and a few custom kerning tables for between different letters to slightly adjust the differences. The advantage here is that it would be universally compatible and could be easily adapted to Hershey fonts. Which are not TTL, but would need their own font parsing anyway. These are not fonts made out of pure glyphs but rather single line fonts where they only have the width of a single line. The importance here for laser cutting is that implementing the engraving of a hershey font (which is certainly different than normal TTF formats) is that it can engrave very small letters. I believe lightburn has this sort of capability as well as specifying directories directly.

A somewhat weirder extra possibility is that inkscape has console accessable font to path capabilities. If we wrote a tiny svg of the correct font size and such to a temporary directory we could ask inkscape if the person has it installed on the computer to convert that to a path and then deal with the font path. This is a lot less helpful but would in theory parse all the bits of the SVG font data correctly. Though it might heavily restrict editing or whatnot.


what is root password?

i can not use sudo

By default the project runs as root user but Heroku replaces the root user with their custom one which doesn't allow root access.Try running it on Google CloudShell as a Docker container but you will not get much storage in the CloudShell.


[cherry-pick] Fix task retry if tikv is down (#1207)

cherry-pick https://github.com/pingcap/tispark/pull/1207

/run-all-tests

/run-all-tests


Add back the top members section in Home tab

In the new UI updates for v3, we left out an element that was/is reported as useful, namely the individual members with their respective staked amounts and number of proposals voted on.

@ammoon if you could please add this to the Figma again. Thxs!


Search List not showing up to date info

To replicate:

go to DR protected area search
look for golden
notice golden eyes comes up, click it
details page shows golden ears (which is most up to date)

Bug:

search results showing old name "golden eyes" but park name should be golden ears
open search caching is not updating

Tested on TEST: Passed

verified that this issue has been fixed.
Passing to PO review.

search list is updating now - moving to done


chore: update readmes for docs directories

what

see above

why

🧹

oops this was a repeat of #45 but with readme changes lol


common: remove check_firewall code

Check firewall isn't working as expected and might break deployments.
This part of the code will be reworked soon.
Let's focus on configure_firewall code for now.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1541840

jenkins test luminous-lvm_osds

CI error unrelated.


Using remark with Jekyll

Thanks for the instructions in the wiki. I've noticed a couple of things...

This is more of an observation than an issue, but the formatting of the presentation.html file is pretty sensitive. The presentation.html file that is provided is formatted like so...

  <body>
    <textarea id="source">
{{ content }}
    </textarea>

I found that when the {{content}} was indented or if I put a comment preceding the mustache code, a class: center middle line I inserted to format the title slide was ignored. In addition the class: center middle line was treated as normal text and rendered at the top of the slide.

When using Jekyll, the markdown file needs to have a html ending. Is there a way around this? It would be nice for my markdown file to have markdown text coloring in my text editor, rather than html coloring. If I name the file default-presentation.md rather than default-presentation.html the slide deck is rendered as a single page rather than as a slide deck.

I find it is much more robust using an separate txt file with the markdown slides in it and then using a jekyl layout to make use of it.

<script src="https://gnab.github.io/remark/downloads/remark-latest.min.js">
</script>
<script>
  var slideshow = remark.create({
  sourceUrl: '{{page.presentationsource}}'
  });
</script>

this feature is not working for me at all. I tried putting the presentation.html file in a _layouts folder on the top level of my jekyll site... and in the gem folder
/Users/me/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/minima-2.0.0/_includes

Thanks for the instruction on the wiki
I'm trying to make it run on the gitlab.com plateform and had it work on github before.
I tried locally and the errror is similar
When deploying on gitlab it is not working probably because of an obvious mistake I can't see...

I created a layout names presentation.html containing the code displayed in the wiki
at the root of the jekyll site I created a test-prez.md file containing the code displayed in the wiki

If I add the meta permalink: /test-prez/ as written in the wiki it displays a 404 error
If I remove it it works. Well the markdown is rendered as html but the slide magic doesn't happen as the --- are rendered as hr tags
Any help would be greatly appreciated :)

This is more of a Jekyll problem since it "wants" to transform your content.
If you don't want Jekyll to touch your markdown, you can simply tell it not to generate that.
For instance, I have done the "lowest effort possible" thing in my blog by simply using another directory: https://github.com/abelards/abelards.github.com/blob/master/talks/10mnCTO.html
With my take (Jekyll don't touch this) and the wiki's, you have two very different ways to do it, so I'll close the issue, and please let me know and reopen it if it's not enough.
Cheers,

Hi,
Thanks for your answer. I'll try that and let you know if I manage to make it to work.
Cheers,

If you don't want Jekyll to touch your markdown, you can simply tell it not to generate that.

@abelards: I've looked at your example, but havent found out how you tell jekyll not to touch my markdown.. can you elaborate?

@ratnanil that's more a jekyll than a remark issue, sadly I don't have the code handy.
What I did was putting the talks in a /talks folder, so it's not in _drafts or _posts.
It fits my use and what I wanted for URLs since my "talks" somehow live their lives independently of "posts".
If you want directories with jekyll doing its magic, you specify that in the config file (see the very bottom of that page  https://jekyllrb.com/docs/structure/ ). I'm not sure about letting your remarks in _posts folder and then telling jekyll not to change them, but I guess that's possible.
Cheers


ToggleSplitButton "IsChecked" property not working with two-way binding on simple property

Describe the bug
ToggleSplitBUtton's "IsChecked" property is not allowing to two-way bind to a simple property, like I do with all the other xaml controls.
I suspect that Microsoft hasn't added the support to do it yet.
The error message says that it must be a Dependency Property!
The strange behaviour was founded both in windows.ui.xaml.controls.ToggleSplitButton and in microsoft.ui.xaml.controls.ToggleSplitButton.
This is the code I used in my app, it is nearly the same as the sample one used in XAML Controls Gallery, so it should work.
Here it's the XAML code:
<ToggleSplitButton VerticalAlignment="Top" IsChecked="{x:Bind ThemeViewModel.IsLeftPaneTransparent, Mode=TwoWay}">            <StackPanel Orientation="Horizontal">            <Grid/>       </StackPanel>       <ToggleSplitButton.Flyout>         <Flyout Placement="Bottom">             <Grid/>         </Flyout>     </ToggleSplitButton.Flyout> </ToggleSplitButton>
Here it's the property to which it is linked, in my ThemeViewModel:
private bool isLeftPaneTransparent;  public Boolean IsLeftPaneTransparent  {        get { return isLeftPaneTransparent; }        set { SetProperty(ref isLeftPaneTransparent, value);         OnPropertyChanged(nameof(IsNOTLeftPaneTransparent)); }  }
Steps to reproduce the bug
Just run the debug and it will display the error.
Expected behavior
The two-way binding should work just stright away.
Actual behavior
The error message

TwoWay binding target 'IsChecked' must be a dependency property

Version Info
Win 10 1903
Gallery version (found on Settings page):
2.2
Windows 10 version:

[ ] Insider Build (xxxxx)
[ ] November 2019 Update (18363)
[x] May 2019 Update (18362)
[ ] October 2018 Update (17763)
[ ] April 2018 Update (17134)
[ ] Fall Creators Update (16299)
[ ] Creators Update (15063)
[ ] Anniversary Update (14393)

Device form factor:

[x] Desktop
[ ] Mobile
[ ] Xbox
[ ] Surface Hub
[ ] IoT

Best regards

Thank you for your feedback @LucaLindholm . As you correctly pointed out this was an issue with the property not being defined properly. This issue was fixed in WinUI a few days ago and hopefully will be released soon. See this issue at the WinUI repo for more context and information.
Also, if you find an issue with WinUI, filing the issue over at the WinUI GitHub repository would be better as the XAML Controls Gallery is just a showcase app of WinUI controls, while WinUI is being developed in it's own repository.

@LucaLindholm Quick update: The fix for this issue has been released and is available in the latest stable release (WinUI 2.4.2) and the latest prerelease (WinUI 2.5.0-prerelease.20060900).

Closing as this was fixed per @chingucoding's note.


Can i use it in normal html website?

Hello guys
I recently working on my website. its a simple html, css website. I want to use mocka placeholder. Let me know how i can use mocka.
Thank you

Of course you can. Use as shown in its demo page, then, as soon as the content you are loading has loaded, swap out the placeholder for the content!

I don't see why not. It's only css styling.
Look at the readme for usage example.


[Suggestion] Mentioning about dbus as a required component for the agent

When I compile and run the agent on Alpine Linux 3.13, I got an error that /etc/machine-id could not be found.
It turns out that dbus is not pre-installed on certain Linux distro. I think the README should mention have a section outlining all prerequisites external dependencies.

Hi @runlevel5, I've added this to the README, thanks for pointing it out.
I think every Linux environment we've used had dbus already, so good to be reminded it's not universal!

Hi @runlevel5, I've added this to the README, thanks for pointing it out.
I think every Linux environment we've used had dbus already, so good to be reminded it's not universal!

Interesting, I didn't realise the agent relied on /etc/machine-id.
Looking at the code it seems we might only use if for tags, and only if the --tags-from-host flag is used. Were you using that flag @runlevel5 ?
If so it seems best effort and I'd love to find a way to make the error non-fatal.

Interesting, I didn't realise the agent relied on /etc/machine-id.
Looking at the code it seems we might only use if for tags, and only if the --tags-from-host flag is used. Were you using that flag @runlevel5 ?
If so it seems best effort and I'd love to find a way to make the error non-fatal.

We also use it in the registration code path, but omitting it appears non-fatal to my eye:
https://github.com/buildkite/agent/blob/3f6b183588248853ce5678d3dde65192112cf431/agent/register.go#L78-L81

We also use it in the registration code path, but omitting it appears non-fatal to my eye:
https://github.com/buildkite/agent/blob/3f6b183588248853ce5678d3dde65192112cf431/agent/register.go#L78-L81


Issue with k8s.io/docs/concepts/services-networking/connect-applications-service/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
Broken link in:

If it isn’t running, you can enable it. The rest of this section...

Page to Update:
https://kubernetes.io/docs/concepts/services-networking/connect-applications-service/

/remove-lifecycle stale


Picture does not show

flutter beta channel
code
'''
test

test2

    test

  '''

..........
now, images success .
is network wrong


Does not run on the device

Hi.
Please, somebody help me. I have been trying to get the project's first page "Usage" example to work on the device but it crashes when launching. On the computer it works fine.
Appreciate any help.

You can't provide device logs, we can't help to fix the issue.

And we don't know what "device" it is.
If you talk about Android or iOS, it does work, i'm using it for various apps on both platforms.


add multinode cpu cluster fixture

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #278

PR #295   👈

This stack of pull requests is managed by Graphite.


'Deep link / External URL' couldn't let app navigate to desired path

A specified URL in Deep link / External URL is not opening the desired path or even let the user move to safari or default web-browser
To Reproduce
Steps to reproduce the behavior:

Go to CleverTap dashboard
Send push notifications to any user, do the campaign for sending push notification
Set the required http(s) url in Deep link / External URL section e.g. https://mydomain.com/profile or https://subdomain.onelink.me/<CUSTOM_ID>/<PATH_ID>
When the user tap on such kind of notification - it opens the app and suddenly moves to safari or default web browser.

Expected behavior
It should give us control to handle the navigation for the deep link URL and keep it in the application only.
Screenshots
How to set Deep link / External URL
Environment (please complete the following information):

Xcode version: 13.2.1
CleverTap SDK Version: latest i.e. using SPM
Device: iPhone 11
OS: iOS 15.4.1

Possible Solution
Don't know
Information as per document:

Deep Link/External URL - (Reference URL: https://developer.clevertap.com/docs/ios#section-push-notifications)
Deep links allow you to land the user to a particular part of your app. Your app’s OpenURL method is called with the deep link specified here. If you wish to use external URLs, then you will have to whitelist the IPs or provide http/https before the URL so that they can be handled properly by the SDK

Additional context
If we set the Custom key-value pairs then it will open the app and we can handle the navigation, but it looks like a hack.

@g212gs We do have a protocol which lets you handle push/In-App/Inbox notification URLs. Here it is: https://developer.clevertap.com/docs/ios#custom-handling-deeplink
Do check it out and let us know if it fits.

akashvercetti - Using the above method, I could handle the URL in Deep link / External URL.
Thank you very much.
Problem solved 🤝


Journal Choice

Based on the discussions in https://github.com/oemof/organisation/issues/7 and https://github.com/simnh/addressing_challenges_oemof/issues/10
From my point of view, there are three options left if we are not aiming at Elsevier or Springer (which would be my preference, see also http://www.leopoldina.org/fileadmin/redaktion/Publikationen/Allianz/2016_12_02_DEAL.pdf why):
Energies
http://www.mdpi.com/journal/energies/about
pro: fully open access / fast: a first decision provided to authors approximately 31 days after submission / peer-reviewed / energy-relalted
contra: don't now what kind of reputation the journal has, topic of the journal fitting to energy but maybe not especially to modeling tools.
costs: 1400 CHF
International Journal of Sustainable Energy Planing and Management
https://journals.aau.dk/index.php/sepm/about/submissions#authorGuidelines
pro: open access / no fee / peer-review / 5000-8000 words
contra: AAU is promoting EnergyPlan and they already questioned oemof at our 100%RE conference, no latex style for handing in (maybe on request) / long review time?
PLOS ONE
http://journals.plos.org/plosone/s/journal-information
PLOS ONE features reports of original research from all disciplines within science and medicine. By not excluding research on the basis of subject area, PLOS ONE facilitates the discovery of connections between research whether within or between disciplines.
Submissions describing methods, software, databases, or other tools We consider submissions describing methods, software, databases, or other tools if they follow the appropriate reporting guidelines. See the submission guidelines for more information.
pro: one of the very first open access / fully open access / latex / good policies
contra: $1,495 USD (http://journals.plos.org/plosone/s/publication-fees) / not energy-realated
IEEE
http://ieeexplore.ieee.org/xpl/RecentIssue.jsp?punumber=60 (oder Power Transactions, o. ä.)
http://open.ieee.org/pubchanges.html
pro: high impact factor / international peer group / open access / peer-review / latex-hand-in mit fertigem template
contra: fees ($1,750-$1,950), old-fashioned and only partly open journal
I know that some people from our wind institute have published in Energies..

My vote:

IEEE
Energies
PLOS ONE
AAU Journal

I just think that our chances in Aalborg might not be the best since we are providing a competetive product from their point of view.

Wrong repository...too early in the morning...sorry ;)


Show the coordinates in a different form

It could be a useful addition when the application allows the user to choose a form of found coordinates.
I suggest extending command line interface with the new parameter -o respectively --output.
Supported values for suggested parameter should be:

[x] maven
[x] gradle
[x] gradle-short
[x] gradle-kotlin
[x] sbt
[x] ivy
[x] grape
[x] leiningen
[x] buildr

When the parameter will not be from the given list, the application throws an exception with explaining message.
When the parameter will be empty or blank, the application throws an exception with explaining message as well.
For sake of backwards compatibility, when an output form will not be explicitly defined, maven output form will be used by default.
Suggested parameter will be relevant only for search command and only when groupId, artifactId and version are defined.

Thank you for contributing this idea, @BranislavBeno. I  like the idea a lot!
May I suggest naming the new parameter format rather than output? To me, output sounds more like "where should the output go".
Indeed, when not specified, let's stick with the current behaviour (formatting as pom.xml snippet). For unknown values, an empty value or a blank value, an appropriate error message is a good idea.

Thank you for your feedback, @mthmulders.
I agree, that format as a parameter identifier better fits its purpose.

Thank you for your feedback, @mthmulders.
I agree, that --format as a parameter identifier better fits its purpose.
On the other hand, in case of -f as a shortened version, there could be potential confusion with --full-name of class-search command.
How to deal with it?

Hmm, you're right. Thinking out loud:

Keep using --format as the full name of the parameter, shortened to -n (for "notation")
Name the parameter --notation, shortened to -n

I don't like a short parameter which doesn't even resemble the full name (option 1). But I still feel format is the best description of what this parameter does.
What do you think about it, @BranislavBeno?

You are right, --format is a better parameter identifier.
I will stick to your first suggestion.
This will also speed up PR preparation.
However, I'm of course still open to any changes within review. :-)
Best regards
Branislav

Added this feature with a3f54fd. Thanks you, @BranislavBeno, for your contribution!

Added this feature with a3f54fd. Thanks you, @BranislavBeno, for your contribution!

The pleasure was mine. :-)

Released in v0.3.0.


fix: add redirect preview page when in edit mode

Description

When in edit mode, if the page is set to redirect to another, it shows the link to which it is redirecting.
Related resources

#8012

Checklist

[x] I have opened this pull request against develop-4
[ ] I have added or modified the tests when changing logic
[x] I have followed the conventional commits guidelines to add meaningful information into the changelog
[x] I have read the contribution guidelines  and I have joined the channel #pr-reviews on our Discord Server to find a “pr review buddy” who is going to review my pull request.

@theShinigami Great work! Thank you. Two things:

Can this go away? In the details view edit mode cannot be on. https://github.com/django-cms/django-cms/blob/565d4cc157778e2225d60cfb247cb8d12d866dec/cms/views.py#L192-L201
Can you add a test? I could imagine creating a PageContent object with redirect set. You could call the edit endpoint and make sure that the redirect message is in the response.


Open file limit not set correctly

If node['rabbitmq']['open_file_limit'] is set to anything other than the default, the resulting ulimit line is normally written in /etc/rabbitmq/rabbitmq-env.conf.  That file is usually sourced, and doing so usually prints out an error along the lines of "operation not permitted" when using rabbitmqctl, etc.
The correct place for the ulimit is in /etc/default/rabbitmq-server.  I tested this manually by applying the updated recipe to the default-centos-65 and default-ubuntu-1204 images.
I followed the same format for the minitest tests, but could not figure out how to execute them.  I'd appreciate a quick rundown on how that's supposed to work, as I'm pretty new to Chef and test-kitchen and am just learning them now.

@jjasghar I added a spec (my first time using ChefSpec), so please feel free to reject if I did it wrong.  I did have to upgrade ChefSpec above 4.0 to get the tests to start.  Since this is my first time contributing to and using Chef, is it normal best practice to perform bundle upgrade frequently to pull in updated dependencies?

@tas50 It should be fine to specify 5 digit integer for the value of mode attribute as we're using that format in default recipe.

@jjasghar I think this PR should be ready for a merge (sorry about the long delay).

@jessedavis we've made some comments on the PR, can you either respond or update the PR with those changes? Thanks!

@jjasghar I believe I've made all the changes that you and @stevecstian recommended in the comments.  Let me know if I'm mistaken or if I didn't make the changes correctly.

:+1:

:+1:


Server-side rendering broken in 0.22

It looks like 0.22 cannot be used server-side due to reliance on document. Here is a simple example that worked in 0.21.
var React = require('react');
var ReactDOM = require('react-dom/server');
var DatePicker = require('react-datepicker');

ReactDOM.renderToString(React.createElement(DatePicker));

Result:
ReferenceError: document is not defined
    at /Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:6:16300
    at Object.o (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:6:16334)
    at e (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:1:668)
    at Object.t.__esModule.default (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:6:12242)
    at e (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:1:668)
    at Object.e.exports (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:1:906)
    at e (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:1:668)
    at t.__esModule.default (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:1:755)
    at /Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:1:760
    at n.(anonymous function).exports (/Users/strawbrary/projects/datepicker/node_modules/react-datepicker/dist/react-datepicker.min.js:1:143)

Looks like this was introduced when we upgraded react-tether to v0.3.1, which has an eagerly-evaluated document.body reference.

@strawbrary, can you verify that changing the portion of referenced in the stacktrace fixes your problem?
Around react-datepicker.min.js:6:16300:
...renderElementTo:document.body...

Change to:
...renderElementTo:document&&document.body...

@strawbrary, can you verify that changing the portion of code referenced in the stacktrace fixes your problem?
Around react-datepicker.min.js:6:16300:
...renderElementTo:document.body...

Change to:
...renderElementTo: typeof document !== 'undefined' && document.body...

@rafeememon Adding typeof document !== 'undefined' works, thanks.

Thanks -- going to make a PR into react-tether

Should be fixed in v0.23.1

I'm using v0.23.1 and I'm getting the error.

@martijnrusschen, it looks like v0.23.1 didn't publish correctly, can you try deleting node_modules, re-installing and republishing v0.23.2?

Can you try this again with v0.23.2?

@martijnrusschen, it's working for me with 0.23.2. Awesome.

Woot!


the firebug icon on toolbar don't have error indicator now

i love that indicator... so much, please make sure it is back

Thanks for the report!
Florent


Version Conflict six>=1.10.0

Trying to install PokemonGo-Bot (Ubuntu 16.04) and get this version conflict:
Installing collected packages: numpy, decorator, networkx, geopy, six, protobuf, requests, future, s2sphere, pycryptodomex, gpsoauth, pgoapi, geographiclib, requests-mock, protobuf-to-dict, googlemaps, colorama, enum34, pyyaml, haversine, polyline, websocket-client, python-engineio, python-socketio, itsdangerous, click, Werkzeug, MarkupSafe, Jinja2, flask, socketIO-client, greenlet, eventlet, gpxpy, funcsigs, pbr, mock, timeout-decorator, contextlib2, raven, demjson, iniherit, Unidecode, python-slugify, yoyo-migrations, paho-mqtt, docutils, Geohash, urllib3, certifi, python-telegram-bot, discord-simple
Found existing installation: six 1.10.0
Uninstalling six-1.10.0:
Successfully uninstalled six-1.10.0
Running setup.py install for pgoapi ... error
Complete output from command /root/PokemonGo-Bot/bin/python2 -u -c "import setuptools, tokenize;file='/tmp/pip-build-A4CQjq/pgoapi/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-UkFRrJ-record/install-record.txt --single-version-externally-managed --compile --install-headers /root/PokemonGo-Bot/include/site/python2.7/pgoapi:
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-build-A4CQjq/pgoapi/setup.py", line 21, in
package_data={'pgoapi': ['lib_0_45/', 'lib_0_51/']}
File "/usr/lib/python2.7/distutils/core.py", line 111, in setup
_setup_distribution = dist = klass(attrs)
File "/root/PokemonGo-Bot/local/lib/python2.7/site-packages/setuptools/dist.py", line 320, in init
_Distribution.init(self, attrs)
File "/usr/lib/python2.7/distutils/dist.py", line 287, in init
self.finalize_options()
File "/root/PokemonGo-Bot/local/lib/python2.7/site-packages/setuptools/dist.py", line 386, in finalize_options
ep.require(installer=self.fetch_build_egg)
File "/root/PokemonGo-Bot/local/lib/python2.7/site-packages/pkg_resources/init.py", line 2306, in require
items = working_set.resolve(reqs, env, installer)
File "/root/PokemonGo-Bot/local/lib/python2.7/site-packages/pkg_resources/init.py", line 853, in resolve
raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.VersionConflict: (six 1.9.0 (/root/PokemonGo-Bot/lib/python2.7/site-packages), Requirement.parse('six>=1.10.0'))

This is no longer an issue.

try:
pip uninstall six
virtualenv .
source bin/activate
pip uninstall six
pip2 install --upgrade -r requirements.txt


Add dynamo as a backend option

Currently, the local bolt database is the only supported backend for skribe. Need to at least add a dynamo implementation

A first cut at this backend is now merged.


Add endpoint, models and handlers to fetch transportation office for duty station

Description
Add endpoint, models and handlers to fetch transportation office for duty station
Reviewer Notes
Note: Currently only returning the address and phone number on the transportation office if the phone number is a voice (not a fax number). If a transportation office does not exist for a duty station, an object with zero values are returned.
Code Review Verification Steps

[ ] All tests pass.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in
Querying the Database Safely
have been satisfied.
[ ] Any migrations/schema changes also update the diagram in docs/schema/dp3.sqs
[ ] There are no aXe warnings for UI.
[ ] This works in IE.
Any new client dependencies (Google Analytics, hosted libraries, CDNs, etc) have been:

[ ] Communicated to @willowbl00
[ ] Added to the list of network dependencies

[ ] Request review from a member of a different team.
[ ] (TRIAL) Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

@jim I agree that a 404 would be better when there is no data. While it may seem strange to fetch an object by the id of another thing, we already do that in /service_members/{serviceMemberId}/current_orders

@tinyels Do you know if we are trying to not expose the orders or Transportation Office IDs for a specific reason? Having the actual object ID is better for the client (it knows if the object exist without making a request).

@jim We decided to do it this way to avoid touching already done things at this point in the release. I can add a story to clean it up after today, but a feature that is MVP depends on this endpoint.

@tinyels @amitch23 and I talked about this and I'm good with just changing the response to a 404 when appropriate. Will approve when that lands.