HLS freeze issue in Chrome

This is my test code (that runs on cdn.clappr.io):
var playerElement = document.getElementById("player-wrapper");

var player = new Clappr.Player({
  source: 'https://s3.amazonaws.com/tng-local-prod-6/tests/269/PL-HLSv3.m3u8',
  baseUrl: '/latest',
  mute: true,
  height: 360,
  width: 640
});

player.attachTo(playerElement);

On Chrome it freezes at every 15 second interval.
It works in Safari and Firefox.  There is probably something different about this particular video, not sure what it is, but the combination of this video, with Chrome seem to not work.
Any ideas?

Hi @croepha I think this might be related to your hls stream, just try with this stream instead.
"http://www.streambox.fr/playlists/x36xhzz/x36xhzz.m3u8"

Apple has a tool to validate hls stream, try to validate yours.
https://developer.apple.com/library/ios/technotes/tn2235/_index.html

I ran the apple validation tool on both streams:
My video:
Error: Measured peak bitrate compared to master playlist declared value exceeds error tolerance
--> Detail:  Measured: 3054.41 kb/s, Master playlist: 3469.00 kb/s, Error: 13.57%
--> Source:  https://s3.amazonaws.com/tng-local-prod-6/tests/269/PL-HLSv3.m3u8
--> Compare: tng-v4-HLS-auto-540-.m3u8
Error: Interlaced video detected
--> Detail:  Track ID 551
--> Source:  tng-v4-HLS-auto-720-.m3u8 - tng-v4-HLS-auto-720-00000.ts
--> Detail:  Track ID 551
--> Source:  tng-v4-HLS-auto-720-.m3u8 - tng-v4-HLS-auto-720-00001.ts
...
Warning: #EXT-X-ALLOW-CACHE has been deprecated
--> Detail:  Line: #EXT-X-ALLOW-CACHE:YES
--> Source:  tng-v4-HLS-auto-720-.m3u8
...
And the big buck bunny video:
Error: Measured peak bitrate compared to master playlist declared value exceeds error tolerance
--> Detail:  Measured: 490.30 kb/s, Master playlist: 246.44 kb/s, Error: 49.74%
--> Source:  http://www.streambox.fr/playlists/x36xhzz/x36xhzz.m3u8
--> Compare: url_2/193039199_mp4_h264_aac_ld_7.m3u8
...
Error: Format change without discontinuity tag detected
--> Detail:  (null)
--> Source:  url_2/193039199_mp4_h264_aac_ld_7.m3u8 - url_566/193039199_mp4_h264_aac_ld_7.ts
...
Error: Different target durations detected
--> Detail:  Target duration: 11 vs Target duration: 10
--> Source:  url_4/193039199_mp4_h264_aac_7.m3u8
--> Compare: url_2/193039199_mp4_h264_aac_ld_7.m3u8
...
My video actually had less validation issues ...
The reference stream url plays perfectly in Chrome. its got to be something with my specific video.  (although my video plays fine in Safari and Firefox)
I am going to try to see if I can re-transcode to see if there was some sort of fluke problem, also, i might try to force de-interlacing to see if that fixes it...
We can close it if you don't think its a bug, I don't know enough about how the player works to troubleshoot it.  Do you know where in the code I would start looking to try to see where it might be hanging up?

Actually im having the same issue at the hls.js demo page, im closing here and going to try on hls.js, Thanks for the help troubleshooting

@croepha one thing you can do is to force clappr to use Flashls instead.


Fix elasticsearch query

Fixes #1162

Codecov Report

Merging #1163 into 2017.10 will decrease coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           2017.10    #1163      +/-   ##
===========================================
- Coverage    68.89%   68.87%   -0.02%
===========================================
  Files          113      113
  Lines         6163     6163
  Branches       751      751
===========================================
- Hits          4246     4245       -1
  Misses        1746     1746
- Partials       171      172       +1

Impacted Files
Coverage Δ

saleor/product/models.py
89.27% <ø> (ø)
:arrow_up:

saleor/search/backends/elasticsearch.py
0% <ø> (ø)
:arrow_up:

saleor/search/backends/dashboard.py
0% <0%> (ø)
:arrow_up:

saleor/core/utils/random_data.py
81.57% <0%> (-0.38%)
:arrow_down:

Continue to review full report at Codecov.

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


jsx-a11y/label-has-for never correct

I have recently upgraded our eslint and airbnb config amonst other things.  I keep getting the lint error
error  Form label must have associated control  jsx-a11y/label-has-for
My code looks like this, but I also get the error when using just a string (instead of inputID).
<input
    type="text"
    id={inputID}
    className="form-control u__hidden-sm-down"
    value={inputValue}
    onChange={this.handleInputChange}
    onKeyUp={this.handleKeyRelease}
/>
<label
    htmlFor={inputID}
    className={formStyles.someRandomClass}
>
    Search...
</label>

Am I doing something stylistically incorrectly here? I have a few examples of this happening in our application.
Order doesn't seem to matter.

If your receiving this error during linting versus being logged to the console during runtime then that is an issue with the eslint-plugin-jsx-a11y plugin and you can file it here: https://github.com/evcohen/eslint-plugin-jsx-a11y
Our repos are similar and complimentary but are distinctly separate.


improve baselines  Closes #55, Closes #47, Closes #80, Closes #117

This pr is trying to fix #80 #117  #55  #47.
I get an improved baselines based on a forked version of det3d. And now I move some of the changes from my repo to the original one.
PointPillars:
Performance Improvement:
before this commit:
mAP: 0.2992
mATE: 0.4408
mASE: 0.2704
mAOE: 0.9446
mAVE: 0.5465
mAAE: 0.2187
NDS: 0.4075
Eval time: 194.4s

Per-class results:
Object Class    AP      ATE     ASE     AOE     AVE     AAE
car     0.732   0.236   0.161   0.788   0.261   0.213
truck   0.336   0.470   0.210   0.702   0.298   0.249
bus     0.395   0.518   0.188   0.953   0.851   0.266
trailer 0.197   0.697   0.221   0.698   0.222   0.183
construction_vehicle    0.038   0.802   0.467   1.328   0.138   0.328
pedestrian      0.543   0.328   0.282   1.450   1.013   0.213
motorcycle      0.109   0.292   0.237   1.477   0.983   0.274
bicycle 0.005   0.290   0.265   1.043   0.605   0.023
traffic_cone    0.265   0.273   0.365   nan     nan     nan
barrier 0.372   0.503   0.308   0.063   nan     nan

After this commit:
mAP: 0.4179
mATE: 0.3630
mASE: 0.2636
mAOE: 0.3770
mAVE: 0.2877
mAAE: 0.1983
NDS: 0.5600
Eval time: 81.2s

Per-class results:
Object Class    AP      ATE     ASE     AOE     AVE     AAE
car     0.801   0.215   0.159   0.191   0.251   0.205
truck   0.457   0.411   0.205   0.159   0.192   0.223
bus     0.558   0.444   0.192   0.126   0.532   0.308
trailer 0.278   0.617   0.202   0.478   0.157   0.172
construction_vehicle    0.103   0.757   0.434   1.159   0.114   0.346
pedestrian      0.717   0.163   0.277   0.432   0.244   0.088
motorcycle      0.321   0.230   0.246   0.354   0.599   0.215
bicycle 0.053   0.193   0.265   0.376   0.213   0.030
traffic_cone    0.437   0.188   0.358   nan     nan     nan
barrier 0.455   0.411   0.297   0.117   nan     nan
Evaluation nusc: Nusc v1.0-trainval Evaluation
car Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
67.75, 81.12, 85.15, 86.46 mean AP: 0.8012047920593498
truck Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
23.71, 45.39, 54.92, 58.94 mean AP: 0.45739047966566226
construction_vehicle Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
0.14, 5.21, 15.55, 20.37 mean AP: 0.10317209147626494
bus Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
26.23, 54.24, 69.76, 72.88 mean AP: 0.5577483330153783
trailer Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
5.09, 21.19, 35.53, 49.41 mean AP: 0.2780347493252949
barrier Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
25.15, 45.56, 53.82, 57.33 mean AP: 0.45465027132389
motorcycle Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
27.79, 32.90, 33.57, 33.96 mean AP: 0.3205635980429202
bicycle Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
5.04, 5.26, 5.28, 5.45 mean AP: 0.05258777460814078
pedestrian Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
68.81, 70.60, 72.70, 74.61 mean AP: 0.7168225670676128
traffic_cone Nusc dist<EMAIL_ADDRESS>1.0, 2.0, 4.0
39.71, 41.46, 44.16, 49.28 mean AP: 0.4365392833724403

Summary of changes

bug fixes in pillar encoder (this boosts to 39.1 map / 50.2 nds)
cos / sin encoding in contrast to directional encoder (both the original repo and my current one can't get directional classifier work on nuScenes) -> (35.4 map / 52.4 nds)
flip along x/y axis during training + regression weight from 1 to 0.25 + filtering zero points box during training -> (41.8 map / 56 nds)

In my repo(which will come out next Monday), I can get (45.5 map / 58.4 nds). The remained changes to my knowledge are:

I use a heavier head with 3 conv-bn-relu. (this should matter the most)
I use l1 loss in place of smooth-l1 loss (minor accuracy difference)
I initialize the final classification conv layer's bias to -2.19 (following focal paper, not sure how much improvement)

Changing to a heavier head requires rewriting mg_head so I didn't do this here.
CBGS
results will be added once finish.(tomorrow night)
To use this newest commit, you need to regenerate all those info files to filter out zero point boxes during training. The other should be the same (no need to regenerate the gt database if you already have one).
I will provide pre-trained models once the pr is merged.

Can you provide corresponding checkpoints and training logs?

yes, the pointpillars model and checkpoint are available here. I will add cbgs in 12 hours.


[CHEC-736] Feature/Add new icons

added marketing icons that haven't been added
added new docs icons

edited label to 'checkout-lg' and added social icons as @john-raymon is working on the docs footer

Yep it's lg.

oh shoot was renaming all prefixes, but no big deal now can do later on! thanks!


Favicon is not working

Favicon is not loading for me.
Adding this would fix it (for everybody):
<link rel="icon" type="image/png" href="{{ url('user://images/favicon.png') ?: url('theme://images/favicon.png') }}" />

Original theme provides ico so i changed it to reflect theme changes. Thanks.


Replace Mandrill with SendGrid

Why:
Mandrill is getting rid of their free Heroku add-on. SendGrid still has a free plan as of today.
Closes #827

Thanks @monfresh. Sorry I didn't get to this myself.


Ensure $(sysconfdir) exists before installing a file to it

This fixes a failure when building a Debian package in a chroot, for example, when the $(sysconfdir) doesn't already exist.

Thank you for your PR, @swt2c !!
Could you inform me the location of the debian package and it's recipe?

Could you inform me the location of the debian package and it's recipe?

I'm just building it privately at the moment.

OK. That is the missing peace for a long time.
I put some pkg stuff on my private ID on another place. So it is very helpful if you can make it public.
See detail in https://github.com/LinearTapeFileSystem/ltfs/wiki/Quick-start#packaging.


Parse Single File not working as expected

It's easy for me to spot single files the suffer from a problem.
I can insert specific IDs to track what is happening in the parse or publish phases.
But, my debug cycle is around 4 minutes.
We have the parse_single_file utility, but it currently appears to produce DITA for lots of files, and it appears to not produce DITa for the specified file.
Command: python parser/parse_single_file.py data data/Wales_Composite/Wales_Composite.html
Output:

(Note: while we've had parse_single_file for a couple of weeks, I didn't have chance to use it in anger, due to a MS-Windows specific python library issue).

Yes, parse_single_file was broken - it was working when first implemented, but then we changed various things we were doing and it broke. I fixed it just now in #492.


⚠ Minecraft has degraded performance

In bbaf857, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1263 ms

Resolved: Minecraft performance has improved in e2b0cc4.


CleanMediaNet adapter missing from the download page

The updated CleanMediaNet adapter is not available on the prebid.org download page.  Is there anything else we need to do for it to show up there?

confirmed this is on the download page now


Added Jenkinsfile

@param8500 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@param8500 Thank you for signing the Contributor License Agreement!

Please remove this PR as this is the PR to the main project. If you are following tutorials please don't pollute the original project!.


Added jenkins manager rewritten to not use spring

This is the jenkins manager moved from EI. It has been rewritten to work without spring.

Applicable Issues

Description of the Change

Alternate Designs

Benefits

Possible Drawbacks

Sign-off

Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
Signed-off-by:

Why java as a part of the repo name? It will limit what is shared (could besides java be scripts, javascripts etc). The name is somewhat clashing (except the padding with -java) an other library, this would be confusing. What about eiffel-commons (Apache style naming convention)?


Corrected typo

https://github.com/MicrosoftDocs/office-docs-powershell/issues/3825

Docs Build status updates of commit a630ce4:
:clock10: Full build: the average full build time is 11 min(s) 53 sec(s), this is based on the last 837 build(s) for this repository.

Docs Build status updates of commit a630ce4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Grant-CsTeamsUpgradePolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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.

Docs Build status updates of commit c008696:
:clock10: Full build: the average full build time is 11 min(s) 53 sec(s), this is based on the last 847 build(s) for this repository.

Docs Build status updates of commit c008696:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Grant-CsTeamsUpgradePolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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.


[MRG] Add jupyter-offlinenotebook extension

This adds the offlinenotebook extension to the base install. With this
users can download their notebook even after the server has gone away.
The extension came out of ideas in https://github.com/jupyterhub/binderhub/issues/1003.
I think we should give this a try and see what people say to it. You can demo it on https://github.com/manics/jupyter-offlinenotebook (has a binder link).

I want to be unsubscribed from this thread.
Thanks
On Wed, Feb 5, 2020 at 5:22 PM Tim Head<EMAIL_ADDRESS>wrote:

This adds the offlinenotebook extension to the base install. With this
users can download their notebook even after the server has gone away.
The extension came out of ideas in jupyterhub/binderhub#1003
https://github.com/jupyterhub/binderhub/issues/1003.
I think we should give this a try and see what people say to it. You can
demo it on https://github.com/manics/jupyter-offlinenotebook (has a
binder link).
You can view, comment on, or merge this pull request online at:
https://github.com/jupyter/repo2docker/pull/845
Commit Summary

Install jupyter-offlinenotebook extension

File Changes

M repo2docker/buildpacks/conda/install-miniconda.bash
https://github.com/jupyter/repo2docker/pull/845/files#diff-0 (7)

Patch Links:

https://github.com/jupyter/repo2docker/pull/845.patch
https://github.com/jupyter/repo2docker/pull/845.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jupyter/repo2docker/pull/845?email_source=notifications&email_token=AMAN2DFE6SYN7ERVS4C7BW3RBM3ZTA5CNFSM4KQTBUZKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ILLB5WQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMAN2DFM5R3BUN5UBKLDGYDRBM3ZTANCNFSM4KQTBUZA
.

--
Assistant Professor
Department of Mathematical Sciences
United States Military Academy
West Point, NY

👍 to trying it out.


Application not opening

Describe the bug
When I click the link to open the web app, an error page opens up saying "Application error".
To Reproduce
Steps to reproduce the behavior:

Go to 'README.md'
Click on 'Visit Website'
See error

Expected behavior
To open and navigate the web app
Screenshots

Additional context
NA
Are you contributing under any open-source program ?
NA

Note:

If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
It is adviced to take up/ work on only 2 issues at the same time.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.

@Asmit2952 Thank you for the concern, we'll work on it and it will be fixed very soon.

Issue resolved via #1100

@Asmit2952 Let me know if it is working fine now

@Pranav016 Yess👍🏻...works perfectly fine


No kneeboard generated on dedicated server

Describe the bug
Decide to try the 0.4.1 on a dedicated server. Works great! However, there was no kneeboard created for the mission.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Running, DCC on a non dedicated server had different results.
Not sure if it was a result of the latest 2.9 update or result of the server running the dynamic campaign.
Screenshots
No taken. Sorry
Desktop (please complete the following information):

OS: Windows 10
DCC Version OB2.9

Additional context
Add any other context about the problem here.

Can you please reproduce the situation (generate a new mission where Kneeboard is missing) and afterwards create a support.zip (Top menu ? -> About -> Create Support zip) and upload it here


au-target-id

I have more of a question rather then issue, when inspecting DOM (runtime) I see many au-target-ids attributes with the same values on different elements, what is the purpose of au-target-id?

The view compiler tags elements which have bindings, custom attributes or custom element behavior. Each gets an id so that it can be looked up in the compiled view's internal compilation instruction list whenever a new view instance needs to be created.

so If I use  element inside the loop (repeat), shouldn't au-target-id be different for each resulting widget's view? I expect new widget instance on each iteration.

No. Those aren't instance ids, they are definition ids.

understood, thank you.

how can console.log() these IDs for that particular custom element ?


Check content dir for for file and folder name validity

The goal is to have more consistent and readable URLs, without underscore in them.
Currently the check finds the following names which will be covered in extra branches:
The following file/folder names use invalid characters. Only lowercase letters, digits, hyphens and period are allowed.
- FILE src/content/vintage/platform-overview/security/Starboard-Scanning-Monitoring.svg
- FILE src/content/vintage/platform-overview/architecture/CAPI_resources.png
- FILE src/content/vintage/platform-overview/architecture/CAPI_architecture.png
- FILE src/content/vintage/advanced/gitops/apps/add_app_template.md
- FILE src/content/vintage/advanced/gitops/apps/app_sets.md
- FILE src/content/vintage/advanced/gitops/apps/automatic_updates_appcr.md
- FILE src/content/vintage/advanced/gitops/apps/update_appcr.md
- FILE src/content/vintage/advanced/gitops/apps/add_appcr.md
- FILE src/content/tutorials/continuous-deployment/apps/add_app_template.md
- FILE src/content/tutorials/continuous-deployment/apps/app_sets.md
- FILE src/content/tutorials/continuous-deployment/apps/automatic_updates_appcr.md
- FILE src/content/tutorials/continuous-deployment/apps/update_appcr.md
- FILE src/content/tutorials/continuous-deployment/apps/add_appcr.md

See https://github.com/giantswarm/docs/actions/runs/12318801597/job/34384349891

Waiting for https://github.com/giantswarm/docs/pull/2428


Diagnostic for incompatible overload signature doesn't explain why

Bug Report
🔎 Search Terms
function incompatible overload implementation signature assignability
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about function overloads

⏯ Playground Link
Playground link with relevant code
💻 Code
function example(overloaded: string): string;
function example(actual: number, option?: boolean): string {
  return `${actual}${option}`;
}

🙁 Actual behavior
A single line of error message that doesn't indicate which parameter or return type is the issue.
function example(overloaded: string): string;
//       ~~~~~~~
// Error: This overload signature is not compatible with its implementation signature.

🙂 Expected behavior
Can we elaborate on that error message with the first assignability issue?
Bikeshedding on the error message specifics welcome...
function example(overloaded: string): string;
//       ~~~~~~~
// Error: This overload signature is not compatible with its implementation signature.
//   Overload parameter 'overload: string' is not assignable to implementation parameter 'actual: number'.

Please put TS2394 in the search terms section.


Generation of atom lists for fixed molecules

provide checks for compositions that are not in-line with min/max number of atoms
skip random atom list generation for fully fixed molecules

Done.


🌐(i18n) update french translations

Purpose
In order to create version 4.0.0-beta.8 we want to update the french translations.
Proposal

[x] update french translations

:ok_hand:


Floating window refocuses randomly

Hyprland Version
Hyprland, built from branch  at commit 96d555e8e794627bfc561e294e148ab8a9961fcc dirty (). Tag: v0.29.1  flags: (if any)
Bug or Regression?
Bug
Description
I use Firefox's "Picture-in-Picture" mode to watch videos on YouTube etc. I use it as pinned floating window when I am coding on Android Studio. When I'm typing keys, suddenly picture-in-picture window gets focused randomly. This is not happening all the time I type a key but after like 15-20 secs typing it happens. I don't know how to debug this and understand what is the problem so I would be happy if you advice some debugging methods.
Here below are my picture-in-picture window configs:
windowrulev2 = float, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = pin, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = nofullscreenrequest, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = nomaximizerequest, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = size 470 265, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = move 1420 785, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = noanim, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = keepaspectratio, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = bordersize 1, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = noshadow, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = nodim, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = noinitialfocus, class:(firefox), title:^(Firefox|Picture-in-Picture)$
windowrulev2 = bordercolor rgb(ffccaa), class:(firefox), title:^(Firefox|Picture-in-Picture)$

I set the title as "^(Firefox|Picture-in-Picture)$" because pic-in-pic window appears with "Firefox" initial title sometimes.
How to reproduce
I don't think that this happens with every window. For example, when I'm reporting this issue, the same picture-in-picture window is active and I have no problems with it. But when I switch to Android Studio and try to code, it focuses picture-in-picture window randomly. This problem may be something specific with Firefox Picture-in-Picture or maybe with XWayland windows. So trying Firefox picture-in-picture window with my config on top of some XWayland application may work.
Crash reports, logs, images, videos
No response

do you have misc:focus_on_activate enabled? if so, what if you disable it?

No, it was set as default. Now I tried both setting it enabled and disabled but nothing changed.

Still an issue

As a workaround - windowrulev2=dimaround, floating:1 forces the focus on the last opened floating window.


chore: improve CRDs template

would hold this for now, maybe we find later a better approach to not persist or send it if not necessary


Typo in settings s/weather/whether/

VSCode Version:Version: 1.51.1 (user setup)
Commit: e5a624b788d92b8d34d1392e4c4d9789406efe8f
Date: 2020-11-10T23:34:32.027Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.17134
OS Version:10.0.17134 N/A Build 17134

Steps to Reproduce:
1.CTRL+,
2.search for weather
Editor › Smart Select: Select Leading And Trailing Whitespace
Weather leading and trailing whitespace should always be selected.

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

/duplicate of #110661 and fixed in upcoming release, due out next week.


drivers: counter: Fix Smartbond counter get_rate

When timer calibration for SmartBond(tm) was added, opaque type clock_control_subsystem_t that used to have device tree ordinal number was changed to enum to allow values that are not in device tree (like no clock selection).
During this process counter driver for SmartBond(tm) was not updated accordingly, resulting in wrong frequency being reported to counter driver.
Failure could be seen when samples/drivers/counter/alarm was execute on da1469x_dk_pro board.
With this fix counter driver uses correct type when clock_control_get_rate is called and counter test works again.

@andrzej-kaczmarek please take a look


Document winget install path for PowerShell and Python, and others if possible

The built-in Windows package manager winget is stable as of two weeks ago. Python and PowerShell 7 are available on it, as well as Git.

The default installation isn't really appropriate, you can't configure anything. Will need to pass git executable the --override string from winget something like /components ... look at /verysilent flag and others, and this.

Also this...
https://github.com/git-for-windows/build-extra/blob/5ffa2186171595299a63bdc12a4ae79cca40507c/installer/install.iss#L103-L115

Python default install is also not appropriate. It's added to PATH which is risky.

Here's a wiki page https://github.com/git-for-windows/git/wiki/Silent-or-Unattended-Installation

Also need to override PoweShell install to add context menu entries.

The proper command to install PowerShell with all the options enabled is below. The /quiet install option had to be omitted due to Installer failed with exit code: 1603, which suggests that the app is already installed, even though it isn't. Omitting /quiet ticks all the appropriate boxes, and the user can be instructed to just click "Next" through the process.
winget install --id Microsoft.PowerShell -s winget --override "ADD_EXPLORER_CONTEXT_MENU_OPENPOWERSHELL=1 ADD_FILE_CONTEXT_MENU_RUNPOWERSHELL=1 ENABLE_PSREMOTING=1 REGISTER_MANIFEST=1 USE_MU=1 ENABLE_MU=1 ADD_PATH=1"

Need to see when winget upgrade is run whether override is necessary on upgrading, if context menu options will disappear, etc.

The proper command to install Terminal is below.
winget install --id Microsoft.WindowsTerminal -s winget

This text file was created by Git<...>.exe /SAVEINF and can be loaded from a file with /LOADINF. Should add /VERYSILENT to the args, then try it from invoking the EXE directly. Then, see if it can be done with a here-string. Then, try that with a winget override.

Need to document installing winget first-of-all, by trying to install from Microsoft Store first and, failing that, installing from the GitHub Releases. For instance, the installer for the latest winget at time of writing is available at this URL.
https://github.com/microsoft/winget-cli/releases/download/v1.4.10173/Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle

Need to work on installing C++ build tools while we're at it. Maybe not for default install, but an extended option?
winget search --id Microsoft.VisualStudio.2022.BuildTools -s winget

Will leave a stub for Linux and Mac setup, and link to an issue to develop it later, or request help.

Python install (e.g. Python 3.11) looks like this.
winget install -e --id Python.Python.3.11 --override "/quiet PrependPath=0"

Bypass remote script signing requirement by having the user copy/paste the setup script into a file they've created themselves. Also discuss Developer setup.

Alright, this is done now that the contributing guide details ground-up machine setup. Some more detail for non-Windows setups may be useful to add in the future, but this part is done.


plans support for WebView2

How to use WebView2 in cppwinrt ? Also, specifically, how to convert wrl::callback like the following to cppwinrt:
 HRESULT hr = CreateCoreWebView2EnvironmentWithDetails(nullptr, userDataDirectory.c_str(),
        L"", Callback<ICoreWebView2CreateCoreWebView2EnvironmentCompletedHandler>(
            [this](HRESULT result, ICoreWebView2Environment* env) -> HRESULT
    {
        RETURN_IF_FAILED(result);

        m_contentEnv = env;
        HRESULT hr = InitUIWebViews();

        if (!SUCCEEDED(hr))
        {
            OutputDebugString(L"UI WebViews environment creation failed\n");
        }

        return hr;
    }).Get());

This is not a WinRT API - I'd post this question on Stack Overflow and see whether someone can share a simple C++/WinRT wrapper.

There's a idl file with WebView2. Can we use this to generate C++/WinRT headers?
Or is there another way to generate code instead of manually write it?


allow to raise Errno::EACCESS to bind sockets already bound

Windows may raise Errno::EACCESS for such case, when
bind operation fails to access port/file descriptor.
This change is to improve test stability.

Let me merge this now... it's just about testing.


Select component applies the testID to the input, not the select.

Description
I would expect when providing a testID to Select, the HTML  element would receive this testID, not an underlying
CodeSandbox/Snack link
https://snack.expo.dev/@justinwarner/select-test-id-on-wrong-element-in-native-base
Steps to reproduce
Look at the source code and notice that the test id repro-test-id is on an <input> and not a <select>. This is unfortunate because some test libs (ie: playwright) has support for <select> directly.
NativeBase Version
^3.4.17
Platform

[ ] Android
[ ] CRA
[ ] Expo
[ ] iOS
[ ] Next

Other Platform
web
Additional Information
Relates to a closed issue- https://github.com/GeekyAnts/NativeBase/issues/4537

Thanks for reporting. We will look into this


Are shared mailboxes supported?

Hello :),
I hope you're doing well, I need your input whether this cmdlet (Enable-App, as well as New-App, Remove-App ...) should work when dealing with shared mailboxes.
I would like to be able to manage add-ins that are assigned to shared mailbox but I don't seem to be able to do that

When I run Get-App -OrganizationApp I see a list of add-ins, but this same list I cannot find in the GUI (Admin portal etc...). Can you tell if it's manageable from the GUI?

When I run this cmdlet on a shared mailbox (Get-App -Mailbox Sharedmailbox@domain.com), I get a list of apps.
Some of the apps that I get when I run (Get-App -OrganizationApp) are not present when I run (Get-App -Mailbox Sharedmailbox@domain.com).
I then try to add an app (previously listed in Get-App -OrganizationApp) to the shared mailbox using (Enable-App -Mailbox<EMAIL_ADDRESS>-Identity "AppId")
I don't get an error, but nothing happens.

See screenshot:

Thank you
Br

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

ID: d467ff3f-e206-3e0f-d3e1-87f79c03ba20
Version Independent ID: a0592470-94b5-fd12-ca80-85763c181329
Content: Enable-App (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Enable-App.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@riohedfield, you should open a support ticket about this. Report back here with any information that can help improve the documentation.

@riohedfield Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. You can contact Microsoft support via the need help section on the admin center. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


fix make test on os type darwin

What this PR does / why we need it:
use make test on darwin OS
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
  Unexpected error:
      <*fmt.wrapError | 0xc00007db40>: {
          msg: "unable to start control plane itself: failed to start the controlplane. retried 5 times: fork/exec /GolandProjects/whereabouts/bin/etcd: exec format error",
          err: <*fmt.wrapError | 0xc00007db20>{
              msg: "failed to start the controlplane. retried 5 times: fork/exec /GolandProjects/whereabouts/bin/etcd: exec format error",
              err: <*fs.PathError | 0xc0000a9c50>{
                  Op: "fork/exec",
                  Path: "/GolandProjects/whereabouts/bin/etcd",
                  Err: <syscall.Errno>0x8,
              },
          },
      }
      unable to start control plane itself: failed to start the controlplane. retried 5 times: fork/exec /GolandProjects/whereabouts/bin/etcd: exec format error
  occurred

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
927

Relevant Lines:
1425

💛  - Coveralls

I don't think this PR makes sense anymore, since we've removed controller runtime dependencies in the master branch - i.e. we no longer use that etcd binary.
Please confirm.

I don't think this PR makes sense anymore, since we've removed controller runtime dependencies in the master branch - i.e. we no longer use that etcd binary.
Please confirm.

confirm. Then you need to remove the dependency on gcr.io/kubebuilder/thirdparty-linux in the makefile

I don't think this PR makes sense anymore, since we've removed controller runtime dependencies in the master branch - i.e. we no longer use that etcd binary.
Please confirm.

confirm. Then you need to remove the dependency on gcr.io/kubebuilder/thirdparty-linux in the makefile

Could you review https://github.com/k8snetworkplumbingwg/whereabouts/pull/273 ?
I originally thought I could remove the entire thing - but I forgot we still need controller-gen to generate the IPPools clientset / informer / lister.


[Technical] Common dev backend and dev moodle

Technical
Create a common backend and database and LMS for the developers.
Problem description

Setup overhead for each developer is starting to explode.
We do not have a common base to reproduce errors and behavior

Possible approach

API for getting a cookie by supplying admin credentials for development purpose. This endpoint can be skipped to register in production using .env flags.
Setup dev moodle and dev backend and dev postgres for frontend development

Acceptance criteria

[ ] backend dev
[ ] moodle dev
[ ] postgres dev
[ ] api dev enpoint

Definition of Done

[ ] All acceptance criteria are met
[ ] The implementation of the User Story has been successfully accepted by the Product Owner
[ ] The implementation is in GitHub
[ ] A code review has been performed (or created in pairing program)
[ ] Code conventions are followed
[ ] Unit test coverage must be greater than 90%
[ ] All tests must have passed
[ ] Technical documentation created (Feature documented in GitHub Wiki)
[ ] UML class diagrams and component diagrams created and up-to-date
[ ] Documentation in realization created (source code)
[ ] Changelog created (Clear description in merge request)
[ ] There are no known bugs
[ ] Rendering behaviour is optimized

Hey team! Please add your planning poker estimate with Zenhub @DimitriB01 @juli-txt @Platura


Feature Request: Transaction button is missing post_url configurability

Describe the solution you'd like
The frame spec mentioned that tx actions can include another post_url which can be used to handle successful transactions. Note this is different than the target, which is where the txn response is requested from.

https://www.notion.so/warpcast/Frame-Transactions-Public-9d9f9f4f527249519a41bd8d16165f73
This could be useful to compose with a frame that specifically handles the transaction flow.
Currently, the FrameButtonMetadata type does not allows specifying this:
export type FrameButtonMetadata =
  | {
      action: 'link' | 'mint' | 'tx';
      label: string;
      target: string;
    }
  | {
      action?: 'post' | 'post_redirect';
      label: string;
      target?: string;
    };

Would suggest a change something like this to allow it (and the corresponding change to actually parse and include in metadata):
export type FrameButtonMetadata =
  | {
      action: 'link' | 'mint';
      label: string;
      target: string;
    }
  | {
      action?: 'post' | 'post_redirect';
      label: string;
      target?: string;
    }
   | {
      action: 'tx';
      label: string;
      target: string;
      postUrl: string;
    }

Describe alternatives you've considered.
No response

This should do https://github.com/coinbase/onchainkit/pull/235 :)

I believe this is close now.


replace finish with close

Tested it and I can now run the FluxTraining.jl integration example successfully, thank you!

Ping @avik-pal


remove "mkdir lstm-model" step while downloading the model

https://github.com/awslabs/mxnet-model-server/tree/master/examples/lstm_ptb
You are already creating a directory while downloading files using wget

This issue has been already resolved


[SPARK-47063][SQL] CAST long to timestamp has different behavior for codegen vs interpreted

What changes were proposed in this pull request?

When an overflow occurs casting long to timestamp there are different behaviors between codegen and interpreted
scala> Seq(Long.MaxValue, Long.MinValue).toDF("v").repartition(1).selectExpr("*", "CAST(v AS timestamp) as ts").selectExpr("*", "unix_micros(ts)").show(false)
+--------------------+-------------------+---------------+
|v                   |ts                 |unix_micros(ts)|
+--------------------+-------------------+---------------+
|9223372036854775807 |1969-12-31 20:59:59|-1000000       |
|-9223372036854775808|1969-12-31 21:00:00|0              |
+--------------------+-------------------+---------------+

scala> spark.conf.set("spark.sql.codegen.wholeStage", false)

scala> spark.conf.set("spark.sql.codegen.factoryMode", "NO_CODEGEN")

scala> Seq(Long.MaxValue, Long.MinValue).toDF("v").repartition(1).selectExpr("*", "CAST(v AS timestamp) as ts").selectExpr("*", "unix_micros(ts)").show(false)
+--------------------+-----------------------------+--------------------+
|v                   |ts                           |unix_micros(ts)     |
+--------------------+-----------------------------+--------------------+
|9223372036854775807 |+294247-01-10 01:00:54.775807|9223372036854775807 |
|-9223372036854775808|-290308-12-21 15:16:20.224192|-9223372036854775808|
+--------------------+-----------------------------+--------------------+

To align the behavior this PR change the codegen function the be the same as interpreted (https://github.com/apache/spark/blob/f0090c95ad4eca18040104848117a7da648ffa3c/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala#L687)
Why are the changes needed?

This is necesary to be consistent in all cases
Does this PR introduce any user-facing change?

No
How was this patch tested?

With unit test and manually
Was this patch authored or co-authored using generative AI tooling?

No

@planga82 Could you confirm, please, that the issue exists in branch-3.5 and branch-3.4.

@planga82 Could you confirm, please, that the issue exists in branch-3.5 and branch-3.4.

@MaxGekk Yes, I tested it in branch-3.5 and branch-3.4 and the issue exist

Thanks all, merged to master, 3.5.2 and 3.4.3

Hi all, I'm revisiting this PR and worried about the behavior change. I 100% agree that codegen and interpreted must have the same behavior, but it's arguable which behavior is more correct.
For non-ANSI mode, I think the Java overflow behavior is more common in Spark. In addition, codegen behavior is more widely seen by end users and it's safer to keep the codegen behavior unchanged. What do you think?

I am conflicted on this. I agree with @cloud-fan that changing the behavior to be more like the interpreted code is not ideal in a big fix release is we are worried about consistency for the user. But at the same time this is totally a corner case and I cannot think of a real world situation where the truncation is not better than overflowing. The overflow starts to happen sometime in the year 292278994, so effectively no legitimate timestamp, should ever hit this. Yes, estimates for the heat death of the universe are after that.
So, then I have to think what are the cases where this could become a problem for a user. Perhaps detecting and filtering out bad dates/timestamps? I know that this can happen in practice. But the simplest way to do that is to compare the timestamp against an allowed range, and that does not involve casting the timestamp to seconds since the epoch. So it would have to be a case where a user wants seconds since the epoch and is doing filtering/cleanup after the conversion. But in this case the truncation makes it 100% guaranteed to catch all bad timestamps because overflow could covert a timestamp back to a "valid" one. Unless your "valid" range is outside of the overflow/underflow range.
The only use case I can think of where the overflow is better is if someone is trying to detect the overflow to mark the conversion as bad (essentially look for a change in the sign of the value after the conversion). But that would also imply that the timestamp is good, which I really doubt. Perhaps there are some physics or sci-fi datasets out there where this really would be valid. I just don't know.
For me I would vote to keep this change as is, but perhaps I have too limited of a view on how this is being used.

@revans2 This is a corner case that probably no one cares about. But the problem is that Spark's behavior is the standard and making this behavior change means other Spark native engines need to adjust the behavior accordingly. Ideally people should use ANSI mode and it's already the default in Spark 4.0

I agree it is a pain for a native engine to have to change behavior in a point release. My concern at this point is that this PR went into 3.5.2 and 3.4.3, both of which are already technically released. So for good or bad if we change it back then it is going to be a bigger pain for anyone trying to deal with multiple versions of Spark than just keeping it this way moving forward.


tree_transpose does not validate tree structure of input

It looks like only the number of nodes is validated, meaning you can transpose trees that have the wrong input structure:
import jax

outer_treedef = jax.tree_util.tree_structure(['*', '*', '*'])
inner_treedef = jax.tree_util.tree_structure(('*', '*'))

tree = [1, [2, [3, [4, [5, [6]]]]]]  # Definitiely not outer_treedef x inner_treedef

print(jax.tree_util.tree_transpose(outer_treedef, inner_treedef, tree))
# ([1, 3, 5], [2, 4, 6])

As simple fix to this issue, I was thinking to add the following in the current implementation:
  inner_size = inner_treedef.num_leaves
  outer_size = outer_treedef.num_leaves
  expected_treedef = outer_treedef.compose(inner_treedef)
  if treedef.num_leaves != (inner_size * outer_size) or expected_treedef != treedef:
    raise TypeError(f"Mismatch\n{treedef}\n != \n{expected_treedef}")

Original code:
https://github.com/google/jax/blob/0da9b69285811446f069e0ef765cd8e0a8100bf4/jax/_src/tree_util.py#L378-L395
This leads to a more strict treedef verification and previous code where we can freely use tuple/list in the treedef and the input, now this will raise an error:
import jax
outer_treedef = jax.tree_util.tree_structure(['*', '*', '*'])
inner_treedef = jax.tree_util.tree_structure(['*', '*'])   # Here inner tree def uses list

tree = [(1, 2), (3, 4), (5, 6)]   # but input inner tree has tuples
print(jax.tree_util.tree_transpose(outer_treedef, inner_treedef, tree))
# on main: [[1, 3, 5], [2, 4, 6]]
# but with `expected_treedef != treedef`, we have an error like
# TypeError: Mismatch
# PyTreeDef([(*, *), (*, *), (*, *)])
#  !=
 #PyTreeDef([[*, *], [*, *], [*, *]])

Another point is that, there are also 3 tests from the tests are failing, for example: testTranspose21 where we have the following:
pytree_to_transpose: FlatCache([1, 1, 1])
outer_treedef: PyTreeDef(CustomNode(FlatCache[PyTreeDef(*)], [*]))
inner_treedef: PyTreeDef([*, *, *])
treedef: PyTreeDef(CustomNode(FlatCache[PyTreeDef([*, *, *])], [*, *, *]))
expected_treedef: PyTreeDef(CustomNode(FlatCache[PyTreeDef(*)], [[*, *, *]]))

Another observation is that inner_treedef = tree_structure(outer_treedef.flatten_up_to(pytree_to_transpose)[0]) would also fail for the same test input if we set input inner_treedef as None. Error message:
ValueError: Mismatch custom node data: PyTreeDef(*) != PyTreeDef([*, *, *]); value: FlatCache([1, 1, 1]).

Any hints, thoughts @jakevdp (when you have time to check this ticket) ?


data extraction for point time series for large number of points

I have the following code:
from vds_api_client import VdsApiV2

vds =<EMAIL_ADDRESS>'password')

# Latitude list extracted from the .csv file
lat_list = [43.90623]
# Longitude list extracted from the .csv file
lon_list = [-94.40173]

vds.set_outfold('testdata/csv')  # Created if it does not exist
# Define the date range and average window.
vds.gen_time_series_requests(products=['SM-AMSR2-XN-DESC_V003_100'],
                             start_time='2012-07-24', end_time='2020-02-03',
                             lons=lon_list, lats=lat_list, av_win=10, clim=True)

# When requests are already made, they will not be resubmitted.
if not vds.uuids:
    vds.submit_async_requests()

vds.download_async_files()

Question: if the python lat_list and python lon_list variables are increased up to 500 points, is this then calculated in parallel, if so, to what extent? And what is the expected waiting time? One source informed me that for 150 points the time series extraction took 48H. I expect that they used the date range that is given in the code; since 2012.

The jobs are submitted simultaneously. In the background are 75 workers available to process these requests in parallel. If there are no requests pending, this should take around 20 minutes or so. Otherwise you might have to wait indeed. Though 48 hours seems dis proportionally long, and is generally not accurate in my experience

Thanks!
I just got confirmation that the latitude and longitude points were used as single pairs in stead of using them as a batch - so one by one computation. The user has been notified of this feature and should extract data at a quicker rate.


No static error when iterating over a generic type, T.

void main() {
  example1(5);
  example2(3);
}

void example1<T>(T item) {
  // No static error.
  // Runtime error (in Dart2JS):
  // TypeError: 5: type 'JSInt' is not a subtype of type 'Iterable'
  for (final x in item) {
    print(x);
  }
}

void example2<T>(T item) {
  // Static error: The method 'iterator' isn't defined for the class 'Object'.
  final iterator = item.iterator();
  while (iterator.moveNext()) {
    print(iterator.current);
  }
}

I'd expect the same or similar error in both cases.
/cc @leafpetersen

@leafpetersen @lrhn - is this a language issue or an implementation one?  Right now, it looks like none of the implementations report a static error for the for.

Issue moved to dart-lang/language #383 via ZenHub

I commented on the language issue - this seems pretty clearly an implementation bug to me.  The spec is unusually clear here, and I can't see any way it allows this.
Probably this should just be fixed as part of the NNBD implicit cast elimination though.

@leafpetersen - We expect a static error for argument type T. What do we expect for argument type Object?

Unfortunately, Object is explicitly carved out as permitted by the spec.  :(
The no implicit cast/NNBD release should fix this.

Assigning to both front ends.  It probably does make sense to fix as part of NNBD.  That said, it doesn't appear to be marked as an implicit cast by the analyzer right now - i.e., no static error with --no-implicit-cast.

dartanalyzer --enable-experiment=non-nullalbe reports correctly:
  error • An expression whose value can be 'null' must be null-checked before it can be dereferenced. • 37125.dart:10:19 • unchecked_use_of_nullable_value
  error • The type 'Object' used in the 'for' loop must implement Iterable. • 37125.dart:10:19 • for_in_of_invalid_type
  error • The method 'iterator' isn't defined for the type 'Object'. • 37125.dart:17:25 • undefined_method
3 errors found.


Error running Setup.ps1

Hey, I'm trying to start up a docker container with your LocalMultiplayerAgent.exe and before running it i'm supposed to run ./Setup.ps1, but I get the following errors:
Creating new subnet <IP_ADDRESS>/16 with gateway <IP_ADDRESS>
Error response from daemon: plugin "nat" not found
Network creation for <IP_ADDRESS>/16 / <IP_ADDRESS> failed.
Creating new subnet <IP_ADDRESS>/16 with gateway <IP_ADDRESS>
Error response from daemon: plugin "nat" not found
Network creation for <IP_ADDRESS>/16 / <IP_ADDRESS> failed.
Creating new subnet <IP_ADDRESS>/16 with gateway <IP_ADDRESS>
Error response from daemon: plugin "nat" not found
Network creation for <IP_ADDRESS>/16 / <IP_ADDRESS> failed.
I looked up the error response and found some comments saying the nat driver has been removed from docker as of <IP_ADDRESS>.  Is there some other way to start this server?
Thanks

Amazing, thank you so much for your quick replies!  One last question while I have your attention, not sure if you know anything about the Mirror example, but is there a reason why it's split into a client and server project?  Mirror is supposed to be both in the same project so I'm wondering if its possible to put them back together or if that's something the playfab sdk prevents.

Nope, no particular reason - the original intention was to keep the code separate (client+server implementation) to give emphasis on the GSDK code and its integration with Unity engine. Open to feedback to make it a single project, though :) Do you think that this would help?

Yeah, I think it would.  Mirror is supposed to run as client/server sharing the same codebase to save on production time.  I originally used the one DapperDino did a tutorial for in youtube, but that version uses an out of date version of Mirror and the playfab sdk, and trying to update seems to require a lot of work.  A similar structure would make a lot of difference.

thanks! Would you mind opening an issue on the MpsSamples repository here? https://github.com/PlayFab/MpsSamples So we can close the issue and continue the discussion there. Much appreciated, thanks!

Np, i'll close now


Starter project

There are two main ways to fix the problem #45. One is previously knowing all fieldnames and then create .csv file directly. The other is reorganizing the immature .csv file into a satisfied .csv file.
The first idea is little like offline algorithm. The main problem is how can we get all information before creating new .csv file. The most costing time method is to iterate loops one time, get all information and then iterate again. And according to different kinds of input, it's possible to collect information previously and easily. However, for me, I don't know what input will be in practice. Thus, I use the second way to solve the problem.
The second method is little like online algorithm, I have to reorganize the .csv file after the first iteration. In order to achieve all of the desired behaviour, I update three source file of dowel: csv_output.py, logger.py, tabular_input.py.
After finishing the work, I have several questions to the original branch:

The headers order in original .csv file is inverted. I keep the rule. But why do we want the order is inverted?
The TextOutput always adds new content behind the old content instead of deleting content of old file and create new content. Does this operation have special requirement?
When I set missing value blank, it will meet bug in TensorBoardOutput. But I don't fix it until now. We can fix the bug by some string operations.

In the final, I roughly estimate the cost time of the code.
Iteration            New_code           Old_code              Ratio
10000                   17.82s                   15s                   1.18
50000                    87s                       76s                   1.14
100000                  174s                     151s                 1.15
The new code might be acceptable :)

Hi, @krzentner. I have updated the code according to your suggestions. I'm so sorry that it is so late to update code. That's because my keyboard was DESTRUCTED by water this afternoon!!!  I tried to fix it, but I failed......
Thanks for your suggestions, they're so helpful!


the Migration part gives wrong examples

What docs page needs to be fixed?

Section: Migrating from react-use
Page:

https://react-hookz.github.io/web/?path=/docs/migrating-from-react-use--page#side-effects
https://react-hookz.github.io/web/?path=/docs/migrating-from-react-use--page#side-effects

What is the problem?
useAsync would only export execute function, for now, if you want to execute the function when the component is mounted, you should call it in effect
What should be changed to fix the problem?
we should give the correct examples

:tada: This issue has been resolved in version 10.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Inconsistent installation instructions between installation page and main page

On the main page, installation instructions are as follows (for MacOS)
https://github.com/duckdb/duckdb/releases/download/v0.7.1/duckdb_cli-osx-universal.zip

On the installation page, homebrew is mentioned:
Homebrew: brew install duckdb
GitHub Binary: https://github.com/duckdb/duckdb/releases/download/v0.7.1/duckdb_cli-osx-universal.zip

This should probably be unified

Can probably use jekyll includes for this? https://jekyllrb.com/docs/includes/


add LICENSE file

This adds the official CC0 license text as seen in CFPB]s open source project template and project open data.

Looks good to me.


Update to egui 0.28.0

egui 0.28.0 release notes: https://github.com/emilk/egui/releases/tag/0.28.0

Ok, this should be ready now!


Dynamic types are not supported in .NET Standard build

Dear Team,
While using dynamic types in evaluation string give subject errors. Can it possible or I am doing in wrong way?
Data Object:
{     "id": "TEST10001",     "name": "TEST CUSTOMER",     "party_type": "PUBLIC",     "type": "CUSTOMER",     "addresses": [],     "createdby": "SC1005",     "createddate": "2019-08-05T00:00:00",     "updatedby": "SC1005",     "updateddate": "2019-08-05T00:00:00",     "_rid": "RwgzALX24FcFAAAAAAAAAA==",     "_self": "dbs/RwgzAA==/colls/RwgzALX24Fc=/docs/RwgzALX24FcFAAAAAAAAAA==/",     "_etag": "\"00000000-0000-0000-4ed9-1978d73901d5\"",     "_attachments": "attachments/",     "_ts":<PHONE_NUMBER> }
Want to load address dropdown once the selection changed in customer dropdown e.g. customer.addresses.

Can you provide more info: error message, code snippet, ...
Consider that ExpandoObject (ie. C# dynamic) is not supported in .NET Standard.

Hi @davideicardi, Thanks for your response. Please note, I am using c# dynamic but in .Net Core 3.0. Is it possible in it or not?
Dynamic Class:
class DynamicClass : DynamicObject, INotifyPropertyChanged
    {
        readonly IDictionary<string, object> data;

        public DynamicClass()
        {
            this.data = new Dictionary<string, object>();
        }

        public DynamicClass(IDictionary<string, object> source)
        {
            this.data = source;
        }

        public override IEnumerable<string> GetDynamicMemberNames()
        {
            return this.data.Keys;
        }

        public override bool TryGetMember(GetMemberBinder binder, out object result)
        {
            result = this[binder.Name];

            return true;
        }

        public override bool TrySetMember(SetMemberBinder binder, object value)
        {
            this[binder.Name] = value;

            return true;
        }

        public object this[string columnName]
        {
            get
            {
                if (this.data.ContainsKey(columnName))
                {
                    return this.data[columnName];
                }

                return null;
            }
            set
            {
                if (!this.data.ContainsKey(columnName))
                {
                    this.data.Add(columnName, value);

                    this.OnPropertyChanged(columnName);
                }
                else
                {
                    if (this.data[columnName] != value)
                    {
                        this.data[columnName] = value;

                        this.OnPropertyChanged(columnName);
                    }
                }
            }
        }

        private void OnPropertyChanged(string propertyName)
        {
            if (this.PropertyChanged != null)
            {
                this.PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
            }
        }

        public event PropertyChangedEventHandler PropertyChanged;
    }

Binding the initial JSON object given above in dynamic object and trying to evaluate. Please advice !!

No, for now dynamics are supported only on .NET Framework. If I remember well the problem was that on .NET Standard there isn't the Microsoft.CSharp assembly that is used to perform create the expression. Maybe now there is some other solution but I'm not investigated it.

@Fantomik
But you are able to compile this also for .NET Standard? In the past Microsoft.CSharp was not available for .NET Standard. This lib was present only for .NET Framework I think...
But if now it is available I think that we can use it as in .NET Framework.
Can you try to remove #if here and add that dependency (see DynamicExpresso.Core.csproj, remove condition).
If you are able to do a PR I will merge it. Thanks!

@davideicardi I'm able to do that. Can I create PR for my changes?

@davideicardi I'm able to do that. Can I create PR for my changes?

Yes sure.

Great PR enhancement! This was required from net.adamec.lib.common.dmn.engine.

Hi @davideicardi , was there a PR for this? I modified the project locally and it works fine with dynamics. This feature would be great in the latest release.

No, I didn't receive any PR. But appreciate it if someone was able to enable dynamics also for .NET Standard.

PR done! 👍  Supporting JObjects would be an awesome feature too, if someone out there has the time for it.

Thanks @dHamburgler ! New release 2.3.3 just published!

Closed by #125


test data

We need a way to access some data to be used for the test and is my understanding we want to avoid storing data in this repository. Does it make sense to add a method to fetch resources online? we could have a download_test_dataset() which can download the relative xslt and reference XML from the mmd repository - will something like that work for you?

I presume this one is solved now. @epifanio - please reopen if I'm wrong :)


[BEAM-625] A few memory and IO optimizations in Avro and FileIO

Using a buffer when decompressing with Snappy in Avro in order to avoid unnecessary copies during slicing of large strings.
Removing somewhat confusing cStringIO aliasing.
Aligning _CompressedFile.read_size with that of GCSIO to avoid reading deficiencies for compressed files. This effectively increases the read_size for _CompressedFile from 16KB to 16MB.

R: @chamikaramj,  can you please take a look?

LGTM

Hey @dhalperi, could you merge in this PR?

LGTM, merged, thanks.

Thanks Dan, closing.


Fix CLI export when export_path is in preset.

Export presets contains the export_path option, to specify the default export location, but the CLI export option disregarded that, and always required and export path to be specified.
After this commit, if the export path is not specified in the command, the one in the preset will be used, erroring only if it's not present or invalid.
Bugsquad edit: Closes https://github.com/godotengine/godot-proposals/issues/774
Fixes #38531

Thanks!

Cherry-picked for 3.2.4.


Flav/soft delete app

Description

This PR depends on https://github.com/dust-tt/dust/pull/7688.
This update fully uses the newly introduced soft deletion concept for AppResource. Previously, apps had a soft deletion feature implemented via a visibility field. This pull request revisits this logic to adopt the new soft deletion mechanism. The migration included in this PR sets the "deleteAt" timestamp to NOW() for all apps marked as "deleted" in the visibility field.
The visibility field has been removed from the API body since it's no longer used after transitioning to vault endpoints. We can likely remove this field from the database once this PR is live, as it is currently unused (verified in our production database). Additionally, the name App has been changed to AppModel to adhere to naming conventions.
This PR does not affect the existing process and is a NO-OP regarding the current deletion flow.
Risk

Safe to rollback.
Deploy Plan

[ ] Apply SQL migration
[ ] Deploy
[ ] Apply SQL migration again (in case some apps were deleted during the deploy)

Tested locally.

Closing in favor of https://github.com/dust-tt/dust/pull/7875.


Accessibility Disability Simulator

The purpose of this PR is to create a new wrapper component that will allow a user to be able to simulate different scenarios to show what users with different disabilities see when using the content that's wrapped inside.

@margaree I'm more or less happy with the current state of this, let me know if there's anything you feel like needs to be changed or added before I mark this as ready for review.

Nope, feel free to mark it ready for review


[CQ-4348956] added support for disable and readonly attributes in pan…

…el and its child components....

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

corresponding pr in af2-web-runtime
https://git.corp.adobe.com/livecycle/af2-web-runtime/pull/367


Create Apache Airflow DAG to run new phylopic.py script.

Fixes #307
Description
The DAG to run phylopic.py daily. It has the following parameters: catchup = False, concurrency =1 and max_active_runs = 1.
Smoke test written for the DAG.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[x] My commit messages follow best practices.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@mathemancer I have written the DAG to run the phylopic.py script, and the test for the workflow as well. Please review the changes.
Thanks!

@mathemancer I purposely removed the phylo_pic.py and test_phylo_pic.py files so that there's only one script of phylopic present, i.e. phylopic.py and test_phylopic.py. Was I not supposed to remove them yet?
I've merged master into my branch.

@mathemancer Have you submitted that PR, since I don't see it on my repo yet?
As per my current understanding, we only have to remove phylopic from the DAG_VARIABLES dictionary in config.py to fix this, right?

Yes, that is correct.  I'm not sure why the PR didn't show up in your repo.  Sorry I'm a bit behind, I'm doing my best to catch up on GH comments.  All that needs to be done is to remove the phylopic section from DAG_VARIABLES as you said.  Thank you!


FIX: unittest arbitrary wave ports

@svandenb-dev  @SMoraisAnsys I fixed the bug in unittest. I don't understand how it did pass.


get-helm-3 script does not allow helm to be downloaded into directory not on system path

Observed behavior:
When using get-helm-3 to download helm into a directory not on the system path, the script fails with the following output:
curl -s "https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3" | DESIRED_VERSION=v3.5.2 HELM_INSTALL_DIR=/home/circleci/project/bin USE_SUDO=false bash
Downloading https://get.helm.sh/helm-v3.5.2-linux-amd64.tar.gz
Verifying checksum... Done.
Preparing to install helm into /home/circleci/project/bin
helm installed into /home/circleci/project/bin/helm
helm not found. Is /home/circleci/project/bin on your $PATH?
Failed to install helm
	For support, go to https://github.com/helm/helm.

Expected behavior:
I would expect get-helm-3 to allow me to download helm into whatever directory I choose. In fact, it takes HELM_INSTALL_DIR as an environment variable.
But then it performs the following check in testVersion that enforces the constraint that the HELM_INSTALL_DIR is on the system path:
https://github.com/helm/helm/blob/4e447d87cd3222aa5e5ea2b38170e9d80b99c0b4/scripts/get-helm-3#L258-L262

NOTE: Actually, if HELM_INSTALL_DIR is not on the system path, but there is some other helm executable already on the system path, then the testVersion check will pass.

I think it is a perfectly valid use case to use get-helm-3 to download helm into a directory not on the system path so that it can be invoked explicitly.
As an example, Kubebuilder scaffolds a Makefile with phony targets that download tools into a local bin directory where they can be invoked directly, and it is useful to integrate Helm with this scaffolding.

Using ${HELM_INSTALL_DIR}/${BINARY_NAME} instead of ${BINARY_NAME} can fix this.


feat(ruby): add Gemfile.Lock updater

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #1768 🦕
Adds support for updating the ruby gem version in a committed Gemfile.lock file. This helps ensure that bundle install works correctly after a version.rb version change. There is also some additional logic introduced to simulate how ruby Gem::Version handles - with prerelease semvers (e.g. 1.0.0-alpha is parsed as `1.0.0.pre.alpha).
A future update could make use of the new common stringifyRubyVersion to translate the version into the more commonly used . prerelease seperatorwhich is treated "as is" and avoids the .pre. replacement of -.

@chingor13 - Question. What is the typical cadence for releases of release-please-action? I see https://github.com/google-github-actions/release-please-action/pull/681 has been open for a few days now.

I think this PR just broke my CI. It's now overwriting random gem versions in Gemfile.lock. With our without a specified version file.
Example from a release-please PR request in a Rails repo:
# Gemfile.lock
- actioncable (7.0.4)
+ actioncable (0.0.1)

Release please config:
    steps:
      - uses: GoogleCloudPlatform/release-please-action@v3
        id: release
        with:
          release-type: ruby
          version-file: "version.rb"
          bump-minor-pre-major: true

@kjellberg - Sorry to hear this. It looks like you haven't set a package-name in the release action config. My initial guess this could be related. If you set this does the problem go away? If so, I think a quick fix would be to bypass this feature if the package-name isn't set. I can throw up a PR to fix if that's the case.

It looks like this is likely the problem. Putting together a PR to fix the issue.

It looks like this is likely the problem. Putting together a PR to fix the issue.

Makes sense.. thanks :)


Google drive link provided not working

the google drive link provided in the readme file is not working. Please provide a new link for the models

the google drive link provided in the readme file is not working. Please provide a new link for the models

the google drive link provided in the readme file is not working. Please provide a new link for the models

the link is working in my system

the google drive link provided in the readme file is not working. Please provide a new link for the model weights


Replace Search Bar with Datepicker Filter

Replace the search bar with a date picker Filter

Is this wireframe layout too complicated to be developed, as it is the screenshot as when "Today" is clicked, the calender daypicker is shown.

https://cloud.justinmind.com/usernote/prototype/409c2169e2352b1c7e098cf4eb60cdd28081fca44bed1e724fed0b369223f40f
It will be keeping updated until it is finished.

https://www.figma.com/file/1FyQ6nWP8v4wJRaiLND7i0/betbetter-wireframe?type=design&node-id=130%3A6&mode=design&t=zzQYuo7uMUgE0dzB-1


cannot install python api

I am installing python api on my jetson tx2 by running "python3 get_python_api.py". However it seems that I can't connect to the download page.
The output is below:
`vidia@nvidia:/usr/local/zed$ python3 get_python_api.py
-> Downloading to '/usr/local/zed'
JETPACK 45
Detected platform:
jetsons
Python 3.6
ZED SDK 3.4
-> Checking if https://download.stereolabs.com/zedsdk/3.4/jetsons/jp45/py36 exists and is available
Traceback (most recent call last):
File "/usr/lib/python3.6/urllib/request.py", line 1325, in do_open
encode_chunked=req.has_header('Transfer-encoding'))
File "/usr/lib/python3.6/http/client.py", line 1281, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1327, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1276, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1042, in _send_output
self.send(msg)
File "/usr/lib/python3.6/http/client.py", line 980, in send
self.connect()
File "/usr/lib/python3.6/http/client.py", line 1442, in connect
server_hostname=server_hostname)
File "/usr/lib/python3.6/ssl.py", line 407, in wrap_socket
_context=self, _session=session)
File "/usr/lib/python3.6/ssl.py", line 817, in init
self.do_handshake()
File "/usr/lib/python3.6/ssl.py", line 1077, in do_handshake
self._sslobj.do_handshake()
File "/usr/lib/python3.6/ssl.py", line 689, in do_handshake
self._sslobj.do_handshake()
OSError: [Errno 0] Error
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "get_python_api.py", line 270, in
urllib.request.urlretrieve(whl_file_URL, whl_file)
File "/usr/lib/python3.6/urllib/request.py", line 248, in urlretrieve
with contextlib.closing(urlopen(url, data)) as fp:
File "/usr/lib/python3.6/urllib/request.py", line 223, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.6/urllib/request.py", line 532, in open
response = meth(req, response)
File "/usr/lib/python3.6/urllib/request.py", line 642, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python3.6/urllib/request.py", line 564, in error
result = self._call_chain(*args)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 756, in http_error_302
return self.parent.open(new, timeout=req.timeout)
File "/usr/lib/python3.6/urllib/request.py", line 526, in open
response = self._open(req, data)
File "/usr/lib/python3.6/urllib/request.py", line 544, in _open
'_open', req)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 1368, in https_open
context=self._context, check_hostname=self._check_hostname)
File "/usr/lib/python3.6/urllib/request.py", line 1327, in do_open
raise URLError(err)
urllib.error.URLError: <urlopen error [Errno 0] Error>`
What should I do to solve the problem?

You should check that the time and date are correctly set on your jetson, this can trigger certificate issues. You can also check if you can access the URL in a browser or using wget.
A workaround is to install the package manually, eg

wget https://download.stereolabs.com/zedsdk/3.4/jetsons/jp45/py36 -O pyzed-3.4-cp36-cp36m-linux_aarch64.whl

python -m pip install ./pyzed-3.4-cp36-cp36m-linux_aarch64.whl

Same error here, on Windows 10! I'm getting this error:
urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate has expired (_ssl.c:997)>
on line 175, urllib.request.urlretrieve(whl_file_URL, whl_file).
Any suggestion? From my browser the access is denied.


Finish patient list command

Summary
Resolves #55
Add list t/patient command to the application.
Additional

Remove warning in LightTheme.css
Update asUnmodifiableSortedByIdObservableList() in UniqueAbstractPersonList as before it is not sorted.

Pictures / Gif

Checks

[X] Link PR to issue
[X] Add Reviewer
[x] Pass CI

oh wait lol I merge it alrdy sorry2


[feature/select-mode] Continue commit 1ee7d583

cleanup code.
Whether or not to support PlutoSelectingMode.Horizontal when copying and pasting multiple selects.

Hello, @bosskmk , Thank you for your package.
I am not sure the difficulty to select a area as excel.
It is common with a mouse to select some cells in one row(column) or several and then to copy, paste or delete contents of the cells.
I just searched days and found the grid need be wrapped in a GestureDetector. I wonder that whether you have implented such work.

Thank you.

I just saw a description on other  website, it shows pluto_grid can handle the selection like excel. Great.
I will try to figure out the mode. Thank you.
Selection
Row mode - Select rows.
Square mode - Select a square area like Excel.
None mode - Not selectable.
(Row mode) - Select Row by Control(Meta on MacOs) + Click.
(Row, Square mode) - (Shift + arrow keys) or (shift + click) or (long tapping and move) to select.
Select all rows or cells. Control(Meta on MacOs) + A.


fix test suite, hoping for discussion.

#625, has caused future Travis CI builds to fail because of failing test:
 FAIL  src/test/forms/ManagedNumberField.test.js
  ● should change local and app state correctly when user inputs empty string

    TypeError: Cannot read property '1' of undefined

@snyderc, I wanted to see if the change in this PR to put back in the value = "0" was acceptable.
I left the blur change in the code, hopefully it still achieves your objective.
Either the blur has a job but this code needs back in,
the blur is not doing its job but could do the job.
or the test needs to be rewritten for the new changes.
If we cant get this resolved maybe #625 should be reverted and that pull request be submitted again with another code review.
Hopefully this will bring up positive discussion. Im having a hard time understanding, because the blur does bring in the values. But does not change the state in the test. Could this be because the blur only fires after a value has changed on the predictions page and a click away from the value column, maybe the test is before that event.

#629 is also related.
Thank you for flagging this! The reason this test is failing is that I wrote some tests based on handleChange (#622), and then on the same day and in a separate branch submitted #625 to address a bug. (I just added to #625 some explanatory text in a comment.) #625 deliberately moves certain functionality from handleChange to handleBlur, but the unit test from #622 expects the functionality to be in handleChange, so that's why you're seeing this test fail. (In retrospect, I should have waited for one PR to be approved before submitting the other PR.)
I recommend that the failing test be removed, as that will fix the problem being reported in this PR. Then, I'll need to write a new unit test, either based on #625 or #629 (the latter of which has some open discussion).

@snyderc, is it possible to get these changes added in a new pr?
As the context of removing the test and writing a new test for these issues is one that I am unfamiliar with. Then I would be happy to close this pull request.
also @knod, I know you've been busy at the moment but do we know others that should be aware of these changes so that when synderc submits a new pr it is ready for acceptance/review.

I think we're all on the same page, but I'm going to put it out there - my general take on tests that are failing are to not remove the test in order to get rid of the red 'x', but to get solve the failing test by figuring out what's wrong and implementing our desired behavior. @ethanbb has created a PR dealing with the issue, though it needs to be tweaked to conform with the behavior described in #630.
After that, I think it'd be great to write some thorough tests for that behavior. This is a component that's given us plenty of trouble.
Does that answer the concerns or have I missed the mark? Do folks have more thoughts on the issue?

@knod, cool. I submitted this pr for a quick fix alternative. In case #630 was rejected. Glad @ethanbb is knocking it out. Ill keep his branch up to date with my own and report in #630 when the tests start passing with the new changes.


Custom URL for pathnames doesn't work after last upgrade

Hello,
after upgrade to latest version, I'm unable to use custom URL (as shown in https://github.com/xiaoxinghu/gatsby-orga/blob/master/gatsby-node.js where code is commented now).
My Org pages are stored in src/help and I want them accessible on https://siteurl/help/<title>/. Problem is that after migration from orga to OrgContent it looks like like slugs are created directly in plugin (owner of slug field is orga transform plugin) and I'm no longer able to add slug field through onCreateNode.
IMHO this should work:
const { createFilePath } = require(`gatsby-source-filesystem`)

// Add custom url pathname for blog posts.
exports.onCreateNode = ({ node, getNode, actions }) => {
  const { createNodeField } = actions

  if (node.internal.type === `OrgContent`) {
    const path = createFilePath({ node, getNode })
    const slug = `/help${path}`

    createNodeField({
      node,
      name: `slug`,
      value: slug,
    })
  }
}

But I'm getting error Error: A plugin tried to update a node field that it doesn't own
Is there any chance to get old behaviour back or is there correst way how to proceed? I tried many approaches so far without success.

Hi, thanks for the feedback. Yes I changed that behaviour because orga-transformer is a little bit more complicated than then remark-transformer (markdown plugin for gatsby) when it comes to dealing with slug. Markdown files are 1 to 1 mapped to the final pages. But orga support 1 to many mapping with #+ORGA_PUBLISH_KEYWORD in buffer setting. Basically you can pick and choose subtrees to publish into standalone pages. In order to unify the slug logic, there are 2 concept: #+EXPORT_FILE_NAME and #+CATEGORY, so the slug eventually will be [CATEGORY]/[EXPORT_FILE_NAME]. With the current implementation, you can't change that logic, because it's in the plugin. The purpose is to take away the complexity from users. But it's still flexible enough to achieve your goal. You can simply add this in the beginning of your org files:
#+CATEGORY: help

If you want customised file name too, add #+EXPORT_FILE_NAME: my-file-name, then the plugin will pick that up. Without that option, the plugin will use the real file name, this is useful when you want to keep your permalink stable, but you have to change your file names. In theory, you can even be a little bit hacky by doing this: #+EXPORT_FILE_NAME: help/my-file-name.
Please let me know if it doesn't work as expected. Thanks!

Thanks for quick answer, I thought that new "category wise splitting" might be the reason.
I think your proposed solution will work, but seams really like anti-pattern in our case. We want to include our Org documents as part of Gatsby site on specific prefix and adding category and export_file_name to every Org file seams ugly.
Anyway, I'm not sure what would be best approach to allow behavior we want. Specially because I still didn't get whole logic behind gatsby transformer plugins :)
First two ideas comming to my mind:

add something like slug-prefix (or something like this) for gatsby-transformer-orga in gatsby-config
generate some "default" slug in orga transformer and still allow to change it through onCreateNode

Second approach is more appealing to me as I would be able to generate whole slugs, not only prefix them. As we are structuring our org files in sub-directories, it is little redundant to add #+CATEGORY too. It would also allow us to add more logic to slugs generation. Problem is I'm not sure how this should work, I tought that source plugin is creating default slug which is used when there is no field.slug present, but maybe it isn't the case.
Let me know what you think. It would be great if you could add support for this but I'm also looking for feedback in case you don't have time but you will be willing to merge PR.
Thanks

@kepi You are right. The do-it-all-for-you design maybe a good thing in a consumer facing product, but it is not suitable for a power tool, not to mention a software plugin. I removed the login from the plugin itself, now I'm sticking to however the markdown plugin is doing. Plus, Gatsby now introduced the concept of theme, maybe that's a better place to do opinionated things. Take a look at the www folder for the latest example.


SNS - Enforce 'Message too long' exception when publishing messages

Coverage decreased (-0.3%) to 92.017% when pulling 9ca3c2f0759d3a6a234d6be38b2f319c27d0cd08 on iainb:enforce-message-size into<PHONE_NUMBER>4ee78affc07643d16fae6bb31b4014 on spulec:master.

Thanks!


The database extension overrides CSS styles from core

The database extensions loads Yahoo's prue.css library which styles elements regardless of classes and as a result one must always ensure that core got styles which take priority to avoid side effects.
To Reproduce
Inspect any element on either the index or project page and check for styles coming from prue.css.

It would be great to unify this, in particular the database import tab whose style is visually very different from the rest of the UI.

Here is a real world case of "defensive" CSS: https://github.com/OpenRefine/OpenRefine/pull/5549/files#diff-9e4f4a212289cd6dae9f95ec0125b852018db93e3f85544208f532e3a4d1748dR290

It would be great to unify this, in particular the database import tab whose style is visually very different from the rest of the UI.

I think it's OK for extensions to be differently styled since they are not core.

There are actually two different issues:

The different styling: yes, it is in general okay, but for extensions which we ship by default, I think it makes sense to keep a consistent style.
The fact that the included CSS leaks to the rest of the application: if the database extension wants a different styling, the CSS rules it includes should be qualified so that they only apply to the UI elements defined by the database extension, which is not the case

Hmm, and we should move the DB importer into core and not an extension...but that can happen later.


Adding Automatic Number Counter widget to the website

I would like to add an automatic number counter feature to this section of the website. This will enhance user experience. It will add an element of dynamism and interaction that can make the website more compelling and memorable to visitors.
Please assign this task to me under gssoc'23 .

They are dynamic already , they are not hard coded
Please find an another improvement/issue
Happy Coding !


change license to Apache 2.0

Issue Number

Overview
Change license from MIT to Apache 2.0.

[x] I have updated LICENSE file and changed from MIT to Apache 2.0 in each file.

Comments

There is something to be confirmed here though - it concerns the following files :
lib/bech32/src/Codec/Binary/Bech32/Internal.hs
lib/bech32/bech32.cabal
lib/bech32/src/Codec/Binary/Bech32.hs

Can the license be changed if copyright is :
copyright:     2017 Marko Bencun, 2019 IOHK
license:       Apache-2.0

?

Yes. The change of license only take effects at the moment you change it. So in practice, the license in 2017 & 2018 was MIT, and starting from 2019 it's Apache 2.0.


Feature/sc 91574/deskproapps template change from yarn to pnpm

@iTweeno please could you fix conflicts and I'll get this merged ASAP

@AshleyDawson done


Puppeteer support for cross browser testing and mobile browsers

Hi,
Need to know that what is the support of puppeteer with other browsers and online cross browser platforms?

https://github.com/puppeteer/puppeteer#q-what-is-the-status-of-cross-browser-support


HDFS-14244. Refactor the libhdfspp cmake build files.

485

 Quality gate: Failed
59 Issues

 8 Vulnerabilities
 51 Code issues

Details


FilePickerControl features like new event and fixes upload behavior between page update

Description
The FilePicker component exhibits strange behaviors when the webpage is refreshed. The internal "files" structure, which is responsible for managing previously selected files, becomes null during the component's update process. This issue affects the file upload functionality, causing it to fail.
Additionally, after refreshing the webpage, the upload process does not execute as expected, leading to further problems with file uploads.
New Feature Added: A new functionality has been introduced: an event for completed uploads. This event should trigger correctly once the upload process is finalized, providing feedback on the status of the upload.

Feel free to adjust the details according to your specific requirements!
Test Code
# Test code for the review of this PR

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I signed the CLA.

[x] My code follows the style guidelines of this project

[x] I have performed a self-review of my own code

[x] I have commented my code, particularly in hard-to-understand areas

[x] My changes generate no new warnings

[x] New and existing tests pass locally with my changes

[ ] I have made corresponding changes to the documentation (if applicable)

Summary by Sourcery
Enhance the FilePicker component by adding a new event for completed uploads and fixing the file upload behavior during page refresh. Refactor the file upload logic to improve the component's reliability.
New Features:

Introduce a new event for completed uploads in the FilePicker component, allowing feedback on the status of the upload process.

Bug Fixes:

Fix the issue where the FilePicker component's internal 'files' structure becomes null during page refresh, causing file upload failures.

Enhancements:

Refactor the file upload logic in the FilePicker component to improve reliability and maintainability.

Thanks for your contribution.
Will you mind sharing some code for the proper review of this PR?

Hi Feodor, if you try to upload a file, it works for the first time. If you refresh the Page, the file uploader doesnt work correctly. Tomorrow i will make a example to reproduce this behavior.

Can anyone test file_picker on google tv streamer? It is no longer working at all on that platform. I cannot figure out if this is a google issue or a recent update to android APIs.


Change settings page

This moves the settings under Webmentions if that page is set. Depends on a PR in Webmentions.
https://github.com/pfefferle/wordpress-webmention/pull/180

Can you try to use these hooks? https://github.com/pfefferle/wordpress-webmention/pull/180

Will give it a shot.

@pfefferle Completely redid this. Fixed the fact register settings was never called also simplified the code immensely.

@pfefferle I think merging this one would make a 3.8 release ready?

Sure... what do you think about removing the semantic-linkbacks references and show the settings as webmention settings? Is this misleading?


The number of total downloads of packages on bitplatform website is not up to date

as you can see in the following image the number of downloads which is +3.5M at the moment is not up to date:

Thanks for contacting us. We're investigating this issue.
We'll let you know if it's possible to work on this issue.

The issue is accepted and planned. Resolving it will start ASAP.

Resolving this issue has started. It will be announced when this issue is resolved.

This issue has been resolved and it will be available in the next pre-release version. It will be announced when the pre-release becomes ready.

The pre-release has been published. Please give it a try and give feedback.


Does this also support syncing two readarr instances?

And if not, could that be added?

I would also love to see readarr support for ebooks & audiobooks

This would be good for me as well.

Yes, please do this! It would be excellent for managing ebooks and audio books


VSCode window gets filled with background color when connecting or disconnecting a screen on Windows 10 in specific cases

VS Code Version:

First found on version 1.53.2
Version: 1.53.2 (user setup)
Commit: 622cb03f7e070a9670c94bae1a45d78d7181fbd4
Date: 2021-02-11T11:48:04.245Z
Electron: 11.2.1
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Reproduced on the latest Insiders version
Version: 1.54.0-insider (user setup)
Commit: a699ffaee62010c4634d301da2bbdb7646b8d1da
Date: 2021-02-15T05:17:48.792Z
Electron: 11.2.3
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

OS Version:

Windows 10 Home version 20H2 (os build : 19042.804)
Windows 10 Pro version 1909 (os build : 18363.1316)

Steps to Reproduce:

Create a new VSCode window
Set the window as full screen on the "main" screen
Switch to another full screen window of another program (e.g. Chrome) on the "main" screen, hiding the VSCode window
Connect or disconnect another screen to the computer
Click on the VSCode icon in the taskbar (seems a bit more consistent) or Alt-tab back to the VSCode window (seems a bit less consistent)
The VSCode window is completely filed with the background color (see screenshot below)

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

Extra information :

Dragging the top of the window, reconnecting the screen, or starting a "snip" screenshot operation re-displays everything properly.
This issue didn't seem to happen in the 1.52 version
The issue also seems to happen in the alt-tabbing window previews (see screenshot below)

The issue does not seem to happen when the VSCode window is not full screen
The main issue does not happen every time / consistently (often enough to be slightly annoying during normal use, and inconsistently enough to be annoying to screenshot when trying to make a report). However, the above partial issue of alt-tabbing seems to be 100% consistent.
Graphics cards and screen resolutions of the main and connected screens do not seem to matter (I reproduced it on two different computers with wildly different configurations and with different connected screens). However, for my tests I could reproduce it when connecting an extra 1920x1080 screen (on a 1920x1080 main screen) and an extra 1360x768 screen (on a 1920x1080 main screen).

After digging through the commit history and doing some more testing with local compilations, it seems like this issue appeared in 94142bd7e0dbc073fe6abf130d7b7066a7beb1d6 :
- commit 962bedc4f47b26e4a6ddc917ee2a4d9445531fb9, right before the commit, works as expected (1.52.1 behaviour)
- commit 94142bd7e0dbc073fe6abf130d7b7066a7beb1d6 didn't work (new behaviour with bug)
This was the Electron update from 9.3.5 to 11.0.3, so the issue might be coming from Electron itself.

我遇到了相同的问题，必现。
按照您的提示，我下载了1.52.1，终于可以正常使用了。
I encountered the same problem, which is inevitable on my computer.
According to your tips, I downloaded 1.52.1, and finally it can be used normally.

The issue stops happening when using the code --disable-features=CalculateNativeWinOcclusion flag.

/duplicate https://github.com/microsoft/vscode/issues/114459


Create dice

Adds plugin to introduce on-screen dice rolls to the game for a table-top like experience.

This does not truly remove the widget from the parent: https://github.com/bogstandard/dice/blob/5841a593b16937fa087c0b2517ff74a7354a4857/src/main/java/com/dice/DicePlugin.java#L101

You can use Widget#getChildren -> copy the array without your widget -> Widget#setChildren

just parent.getChildren()[child.getIndex()] = null;

Thanks both, hopefully that's done it? I assumed that the arrays were working by reference to their child objects, so nullifying the child would in-turn cause the reference within the array to be nullified too.

I assumed that the arrays were working by reference to their child objects, so nullifying the child would in-turn cause the reference within the array to be nullified too.

fwiw, java objects are "by-reference" but those references are not shared, so setting some variable/field to null will not also null out other references to the object i.e. in an array somewhere or in another variable


Hotfix/remove nfegpa

This change is

Reviewed 9 of 9 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


How does this work?

Please add a short description in the readme about what is the algorithm for converting normal text to emojified text.

I've added a description to the README :+1: