[WIP] Lexical beam

Codecov Report

Merging #410 into master will decrease coverage by 0.27%.
The diff coverage is 54.71%.

@@            Coverage Diff             @@
##           master     #410      +/-   ##
==========================================
- Coverage   69.43%   69.16%   -0.28%
==========================================
  Files          74       74
  Lines        6357     6440      +83
==========================================
+ Hits         4414     4454      +40
- Misses       1943     1986      +43

Impacted Files
Coverage Δ

onmt/translate/DecoderAdvancer.lua
8.08% <0%> (-0.35%)
:arrow_down:

onmt/translate/Translator.lua
22.36% <12.5%> (-0.26%)
:arrow_down:

onmt/translate/Beam.lua
80.29% <38.09%> (-3.37%)
:arrow_down:

onmt/data/Batch.lua
69.39% <58.33%> (-1.13%)
:arrow_down:

onmt/translate/BeamSearcher.lua
88.63% <71.15%> (-6.99%)
:arrow_down:

onmt/data/Dataset.lua
89.04% <83.33%> (-0.82%)
: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 1f64569...1022f8b. Read the comment docs.

Any description and/or read me on the logics behind all of this would be very welcome.
thanks

@vince62s , I just added documentation to make it clearer.


header comments are too long

This causes writing a FITS file to die upon calling calibrate_science
Basically the Uncertainty of atmospheric extinctions and Uncertainty of calculated zeropoints need shortening

I'm curious about this as I'm using astropy version 5.1 and don't get this error. Do you have any idea why it's an issue for you? I wonder if it's the particular system you're calibrating causing long float values for the uncertainties (24.8999999999997 sort of thing where it should be 24.90000, obviously adding a lot of extra characters to the header card)

no, that's not it...
In [59]: from astropy.io import fits

In [60]: fits.Card
Out[60]: astropy.io.fits.card.Card

In [61]: import astropy

In [62]: astropy.__version__
Out[62]: '5.1'

In [64]: c = fits.Card('ATM_EX_E', '0.05, 0.05, 0.05, 0.05, 0.05', 'Uncertainty
    ...: on atmospheric extinctions.')

In [65]: str(c)
---------------------------------------------------------------------------
VerifyWarning                             Traceback (most recent call last)
Input In [65], in <cell line: 1>()
----> 1 str(c)

File /usr/local/lib/python3.9/dist-packages/astropy/io/fits/card.py:206, in Card.__str__(self)
    205 def __str__(self):
--> 206     return self.image

File /usr/local/lib/python3.9/dist-packages/astropy/io/fits/card.py:505, in Card.image(self)
    503     self.verify('fix+warn')
    504 if self._image is None or self._modified:
--> 505     self._image = self._format_image()
    506 return self._image

File /usr/local/lib/python3.9/dist-packages/astropy/io/fits/card.py:1008, in Card._format_image(self)
   1006         output = self._format_long_image()
   1007     else:
-> 1008         warnings.warn('Card is too long, comment will be truncated.',
   1009                       VerifyWarning)
   1010         output = output[:Card.length]
   1011 return output

VerifyWarning: Card is too long, comment will be truncated.

p.s I'm not sure why the warning gets elevated to an exception, but it does...

because:
https://github.com/Alex-J-Brown/cam_cal/blob/5e6519d2a4a520831e799de5dd5e300b3b82eca9/cam_cal/observation.py#L29

Ok this should be fixed now


How to contribute to the website?

It's not clear how to send PR's for the website. There is a tourmaline-website repo and this repo has a gh-pages branch. I see multiple typo's in de code examples or examples that do not work anymore.
It would help if this was clearly documented and if the website-repo is outdated it would be wise to remove it.
Thanks

Sorry it took me a while to get to this, I stepped away from Crystal and Telegram for a while. As far as contributing to the website:
This project uses mkdocstrings-crystal, so you'll need install the dependencies with pip install -r requirements.txt first.
You'll then need to make sure you have shard depencies installed using shards install.
Run mkdocs build to build the files, and mkdocs serve to start a local development server so you can test your changes.
 Once your PR is merged into master a Github action will run that builds the docs and pushes them to the gh_pages branch.


Home page shows "posts" folder not posts.

Describe the bug
When running hugo server locally and testing my site I am not shown my posts only a single posts that when clicked on shows an index of my other posts.
To Reproduce
I was able to reproduce by running hugo server in the example site directory.
Expected behavior
I expected different content items each showing one of my posts.
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.
Your project
https://bitbucket.org/LucasSymons/lucassymons.net/
Hugo version
Hugo Static Site Generator v0.59.0/extended linux/amd64

Hi, Have you looked at PR #75? I manually applied the changes in the PR to my site and it fixed the issue for me.

Hi, Have you looked at PR #75? I manually applied the changes in the PR to my site and it fixed the issue for me.

Looks like that's it! thanks

Closing this as PR is now merged.


HSEARCH-3311 Test (and document) alternatives to analyzer discriminators in Search 6

https://hibernate.atlassian.net/browse/HSEARCH-3311
I also included an experimental implementation, annotated with @Incubating. I think it's worth it, be it only to reduce the amount of code users need to write.
I believe the solution will evolve in the future, in particular if we address HSEARCH-3903. But for the time being... it works.

Merged, thank you!


[Question] CDPSession to talk to raw Chrome Devtools Protocol

Is there a java equivalent of CDPSession to talk to the raw Chrome Devtools Protocol?
Nodejs Equivalent => https://playwright.dev/docs/api/class-cdpsession/

No, CDPSession is not exposed in the java API. Now that we have connectOverCDP there should be much less demand for this. Can you tell us more about your use case and where Playwright API is lacking to that you have to turn to raw CDP?

I have the following use cases to use CDP,

Use CSS.getPlatformFontsForNode to see if network font or system font is being used for a element
Use Network.setRequestInterception to modify network responses

Is it doable via connectOverCDP ?

Thanks for sharing the details!

Is it doable via connectOverCDP ?

These are not currently possible but we can add an API for those that would work across all browsers.

Use CSS.getPlatformFontsForNode to see if network font or system font is being used for a element

Can you file a feature request for this in https://github.com/microsoft/playwright repo?

Use Network.setRequestInterception to modify network responses

This is being tracked in https://github.com/microsoft/playwright/issues/1774 and we are working on this (there is an open PR adding initial support upstream), so hopefully it will make it to the next release.

Thanks. I'll raise a feature request.


Ng4 support

Hi...
not sure this is the best to place to ask but...
Is this module usable with Angular4? Do you have any plan to shift to this version any soon?
Thank you!

@heppy83 AFAIK, Angular4 is backwards-compatible. There should be nothing stopping an Angular2 component from working. I think the owner of this project should upgrade the peerDep.

I get the error below after upgrading to 4 but after webpack's reload it's working fine.
ERROR in Error encountered resolving symbol values statically. Calling function 'InjectionToken', function calls are not supported. Consider replacing the function or lambda with a reference to an exported functi
on, resolving symbol ANALYZE_FOR_ENTRY_COMPONENTS in<EMAIL_ADDRESS>resolving symbol JsonSchemaFormModule.forR
oot in /Users/drorrash/work/side/ngMaterialApp/node_modules/angular2-json-schema-form/src/library/json-schema-form.module.ts, resolving symbol AppModule in /Users/drorrash/work/side/ngMaterialApp/src/app/app.modu
le.ts, resolving symbol AppModule in /Users/drorrash/work/side/ngMaterialApp/src/app/app.module.ts

Would stil be nice if all dependencies  are updated to the current versions.

As stated by @ZheyangSong , it works on Angular 4, but, yes, we do plan to release an updated version soon.


gnav: links within website need to be relative

links in gnav here: https://business.adobe.com/blog/ seem to include fully qualified hostnames, which should be changes to be relative to the host if they point to a .page, .live and/or business.adobe.com domain.

Fixed via: https://github.com/adobe/business-website/pull/148


Fix unicode bug for py2

Fixes bug created in #141 cc @skuda

Thanks! I was just about to commit the same fix :)


Issue with Aeon Labs - Home Energy meter after update to 5.4.2 (Latest version as of 7/23 - Docker) from 5.2.2

Hi,
I have been using HEM v1 for sometime with zwavejs2mqtt + HA without any issue. I have set group 1 reporting for both clamps and set it as 30 seconds. This setup has been working fine on 5.2.2, as soon as I updated to 5.4.2 HEM stopped updating the values automatically. If I refresh from zwavejs2mqtt dashboard by click refresh the values are fetched otherwise it just does not refresh.
I made sure old configuration values for Group 1 are still there like 30 sec refresh which I tried changing few times to kickstart HEM. Restarted all of the devices but still did not work. Eventually I came back to 5.2.2. Is there any change in HEM v1 driver or anything else which might be causing this issue?
Thanks!

Please attach zwavejs debug logs


Dropping HHVM

Reference: https://github.com/laravel/docs/blob/5.3/upgrade.md#php--hhvm

Thanks man!


Applies bug fixes to UI screens

Description

Before - archived models list label:

After - archived models list label:

Before - archived models list search filter:

After - archived models list search filter:

Before - archived models list pagination:

After - archived models list pagination:

Before - close button border radius:

After - close button border radius

Before - archive version modal

After - archive version modal

How Has This Been Tested?

Model Registry list view -> Actions menu next to "Register Model" button in toolbar -> Click "View Archived Models" -> Archived models list view -> Click on a model in the list -> Verify label text color is correct
Model Registry list view -> Actions menu next to "Register Model" button in toolbar -> Click "View Archived Models" -> Archived models list view -> Verify search filter and pagination has correct styling applied
Model Registry list view -> Actions menu next to "Register Model" button in toolbar -> Click "View Archived Models" -> Archived models list view -> Labels column -> click on "1 more" overflow chip. Verify close button border radius is round.
Click on a kebab menu in any list view to toggle a modal. Verify modal styling has been updated.

Merge criteria:

All the commits have been signed-off  (To pass the DCO check)

[x] The commits have meaningful messages; the author will squash them after approval or in case of manual merges will ask to merge with squash.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work.
[x] Code changes follow the kubeflow contribution guidelines.

If you have UI changes

[x] The developer has added tests or explained why testing cannot be added.
[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Verify that UI/UX changes conform the UX guidelines for Kubeflow.

/lgtm


requestData.url not properly encoded as utf8

reproduce:
var webPage = require('webpage')
var page = webPage.create();
address = "http://www.google.com/ロリ/index.html";

page.onResourceRequested = function (reqData, netReq) {
    console.log(address);
    console.log(reqData.url);
    netReq.abort()
};

page.open(address, function (status) {
    phantom.exit();
});

The first output is the correct utf8 address; but the reqData.url output line seems to be an ascii encoding.
tested with phantoms-2.1.1 release on mac osx 10.11.3

Darn it, I thought I fixed all of these.
Thank you for the bug report.  We don't need any more information, but I'm afraid it may be some time before anyone gets around to writing a patch.


Fix tooltip bug when content is null

This PR fixes page crash when content is null for tooltip

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

Perf comparision

Scenario
Fluent TPI
Fabric TPI
Ratio
Iterations
Ticks

Button.Fluent
1.18
0.12
9.83:1
5000
5878

Checkbox.Fluent
1.38
0.23
6:1
5000
6900

Icon.Fluent
0.25
0.03
8.33:1
5000
1263

Slider.Fluent
1.76
0.26
6.77:1
5000
8802

  Generated by :no_entry_sign: dangerJS

@delprzemo looks good to me, please update to master and verify if changelog entry gets to the right place (unreleased)

@jurokapsiar  Great :) Branch updated to master - changelog entry in the unreleased section. I don't have write access so could you merge it?

So, fix for this was merged in #2332. Closing.


[Bug] Prevent deleting default env

Describe the bug

Because once it's deleted, I can't set it:
t env delete default
Deleted env: default

t env set default.address localhost:7233
Error: env not found: default
('export TEMPORAL_CLI_SHOW_STACKS=1' to see stack traces)

Hmm.. it should be okay to empty the contents of the default env.
In any case temporal env set default.address localhost:7233 should not fail if the default env doesn't exist.

I get the same
Error: env not found: default

when running most commands

I got that too.
I think we can make an exception and treat missing default as empty default.

In the meantime, how do I fix? Tried reinstalling, still get the error on
all commands 😄
[image: image.png]
On Tue, Jun 13, 2023 at 1:26 PM Roey Berman @.***>
wrote:

I got that too.
I think we can make an exception and treat missing default as empty
default.
—
Reply to this email directly, view it on GitHub
https://github.com/temporalio/cli/issues/262#issuecomment-1589732454,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5LGAPES6HCDSV2GWVTEDXLCPCVANCNFSM6AAAAAAZCRNBYE
.
You are receiving this because you authored the thread.Message ID:
@.***>

if that's a quick fix i'll send a PR. Meanwhile feel free to delete the config ~/.config/temporalio/temporal.yaml

Deleting worked, thanks!


Add Yogi optimizer

Summary
This PR adds the Yogi optimizer (see [1] below) which, according to its authors, leads to even better performance than Adam with similar theoretical guarantees on convergence. It was presented at NIPS 2018.
[1] Adaptive Methods for Nonconvex Optimization
Related Issues
(None)
PR Overview

[ y ] This PR requires new unit tests [y/n] (make sure tests are included)
[ y ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[ y ] This PR is backwards compatible [y/n]
[ n ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR.
At the end of the docstring, can you add:
If you open a issue or a pull request about the Yogi optimizer, please add 'cc @MarcoAndreaBuchmann' to notify him.
I cannot be an expert in all the features which comes in this repo. Which means that you'll be notified if something is going on in github about it. This is to help out as it will not be easy for me to maintain everything. Would that work for you?

Thanks for the quick feedback, @gabrieldemarmiesse ! I've added the requested line to the docstring so future contributors know whom to contact.

Thanks for the pull request!


Fixes 'no module named emailer' issue running pytest.

If you don't have emailer in your installed packages
and you run pytest in the emailer root folder.
Then Pytest will fail with the error 'no module named emailer'
https://stackoverflow.com/a/43559758/92584
I think this is because the tests are not in a module, so they don't look
to the root folder to see if there is a module named emailer.  But
by making the tests a module, then they know to look in the pwd for a
module named emailer. 🤷

Oh I see in the .travis.yml you're using pytest tests and I was just using pytest.
Well, this makes just pytest work lol.

Thanks Robert!


Added Multiply_Strings.java

this is amazing @taranjotsingh23
thanks for the contribution in my repo and please share in social media and tag me on twitter


Some Help please object inside array

Sorry I can't understand how I can do this.
My code is like below
              out_message["params"]={
                    {url},
                    {
                      {"dir",save_loc},
                      {"out",file_name },
                      {"continue", true},
                      {"split", connections},
                      {"max-connection-per-server", connections},
                      {"user-agent", user_agent},
                    }

                };

I want to add
header object with Cookie right after user-agent array which is an object.
I've been  using https://github.com/meoow/aria2rpc/blob/0c8e2ffbbea496415d418238e94d8c118a5c09ed/aria2rpc.py#L54
as my guide.
Thanks in advance.

If you see any mistake feel free to point them out.

Also another question.
let's say I have
json["huge_array"]={   {"varx","1234"}, };
how can we just do it like this
for(int i=0;i<10;i++)
json["huge_array"][i]= {"varx","1234"};

Moved to gitlab https://gitlab.com/eidheim/Simple-WebSocket-Server/issues/113

I had made a mistake, sorry about that

This is how it should look like
{
    "jsonrpc": "2.0",
    "id": "meoow/aria2rpc",
    "method": "aria2.addUri",
    "params": [
        "http:google.com",
        {
            "continue": "true",
            "max-connection-per-server": 15,
            "split": 15,
            "min-split-size": "10M",
            "user-agent": "Mozilla/5.0 (X11; Linux; rv:5.0) Gecko/5.0 Firefox/5.0",
            "header": [
                "Cookie: a cookie over here"
            ]
        }
    ]
}

How do I add the header ?

Solved with
                        {"dir",save_loc},
                        {"out",file_name },
                        {"continue", true},
                        {"split", connections},
                        {"max-connection-per-server", connections},
                        {"user-agent", user_agent},
                        {"header",{"Cookie: "+cookies}}

For the other question. I fixed it using pushback(json::array())


test: added test for hook useBotIdInternal  #132

Description
added test for hook useBotIdInternal
Closes #132
What change does this PR introduce?
Please select the relevant option(s).

[ ] 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 not work as expected)
[ ] Documentation update (changes to docs/code comments)
[x ] Add unit test

What is the proposed approach?
Please give a short overview/explanation on the approach taken to resolve the issue(s).
Followed other unit tests structure and make the jest tests to test the changes
Checklist:

[ X] The commit message follows our adopted guidelines
[ X] Testing has been done for the change(s) added (for bug fixes/features)
[ X] Relevant comments/docs have been added/updated (for bug fixes/features)

Hi @tjtanjin  ... PR updated ... let me know if you need something else !!


Sync templates

Reviewers Checklist

[ ] Appropriate logging is done.
[ ] Appropriate error responses.
[ ] Handle every possible exception.
[ ] Make sure there is a docstring in the command functions. Hint: Lookout for
botcmd and re_botcmd decorators.
[ ] See that 100% coverage is there.
[ ] See to it that mocking is not done where it is not necessary.

Travis tests have failed
Hey @Naveenaidu,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
1st Build

     python -m pytest

============================= test session starts ==============================
platform linux -- Python 3.6.3, pytest-3.6.4, py-1.5.2, pluggy-0.6.0
rootdir: /home/travis/build/coala/corobo, inifile: setup.cfg
plugins: requests-mock-1.5.2, xdist-1.22.5, travis-fold-1.3.0, timeout-1.3.1, reqs-0.1.0, reorder-0.1.1, profiling-1.3.0, mock-1.10.0, instafail-0.3.0, forked-0.2, error-for-skips-1.0.0, env-0.6.2, cov-2.5.1
_____________________ ERROR collecting plugins/coatils.py ______________________
/opt/python/3.6.3/lib/python3.6/doctest.py:933: in find
    self._find(tests, obj, name, module, source_lines, globs, {})
/opt/python/3.6.3/lib/python3.6/doctest.py:992: in _find
    if ((inspect.isroutine(inspect.unwrap(val))
/opt/python/3.6.3/lib/python3.6/inspect.py:509: in unwrap
    while _is_wrapper(func):
/opt/python/3.6.3/lib/python3.6/inspect.py:503: in _is_wrapper
    return hasattr(f, '__wrapped__')
../../../virtualenv/python3.6.3/lib/python3.6/site-packages/ramlient/core.py:132: in __getattr__
    raise ResourceNotFoundError(attr)
E   ramlient.errors.ResourceNotFoundError: No such Resource found: __wrapped__
collected 43 items / 1 errors / 3 skipped

travis_fold:start:py-5737.cov.0
----------- coverage: platform linux, python 3.6.3-final-0 -----------
Name                                Stmts   Miss Branch BrPart  Cover   Missing
-------------------------------------------------------------------------------
answers/__init__.py                     0      0      0      0   100%
plugins/__init__.py                     0      0      0      0   100%
plugins/constants.py                    5      0      0      0   100%
utils/__init__.py                       0      0      0      0   100%
utils/backends.py                       0      0      0      0   100%
plugins/lmgtfy.py                       4      1      0      0    75%   15
plugins/nevermind.py                    5      1      0      0    80%   14
plugins/ship_it.py                      7      1      0      0    86%   28
plugins/the_rules.py                    5      1      0      0    80%   19
plugins/deprecate_bot_prefixes.py       6      3      4      0    30%   15-17
plugins/ghetto.py                      10      5      2      0    42%   20-28
plugins/spam.py                        13      5      2      0    53%   19, 27-35
plugins/pitchfork.py                   12      6      2      0    43%   18-39
plugins/coala_lowercase_c.py           12      7      4      0    31%   11-26
plugins/searchdocs.py                  14      7      2      0    44%   23-30
plugins/explain.py                     20      9      6      0    50%   30-44
plugins/wolfram_alpha.py               19     12      6      0    28%   17-18, 25-34
utils/mixin.py                         20     14      6      0    23%   7-9, 12, 18-27, 30-38
plugins/answer.py                      31     20      8      0    28%   20-27, 31-61
plugins/git_stats.py                   51     43     19      0    11%   17-18, 25-77
plugins/coatils.py                    115    100     67      0     8%   20-21, 25-32, 46-97, 108-126, 130-140, 144-159, 172-241
-------------------------------------------------------------------------------
TOTAL                                 349    235    128      0    24%
travis_fold:end:py-5737.cov.0

FAIL Required test coverage of 100% not reached. Total coverage: 24.32%
=========================== short test summary info ============================
ERROR plugins/coatils.py
SKIP [1] /home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/_pytest/doctest.py:356: unable to import module local('/home/travis/build/coala/corobo/answers/extraction.py')
SKIP [1] /home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/_pytest/doctest.py:356: unable to import module local('/home/travis/build/coala/corobo/answers/final.py')
SKIP [1] /home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/_pytest/doctest.py:356: unable to import module local('/home/travis/build/coala/corobo/answers/service.py')
!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!
====================== 3 skipped, 1 error in 1.95 seconds ======================


"Open Webview Developer Tools" does nothing in 1.67

Issue Type: Bug
Ctrl-Shift-P, select it... expected: tools open... actual: nothing happens
disabled all extensions & tried again, same result... yet it works in 1.67 Insiders... and I can't find any relevant log entries
I'm happy to respond with more info if there's something I'm missing here that could be relevant
VS Code version: Code 1.67.0 (57fd6d0195bb9b9d1b49f6da5db789060795de47, 2022-05-04T12:06:02.889Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.89GB (21.73GB free)

Process Argv
--file-uri file:///c%3A/src/AF4JM.code-workspace --crash-reporter-id 06338080-eec2-498d-98e0-cf3111dc1cde

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebug:30475216

I can't reproduce this in insiders. Have you tried the command in the command palette?

that's what I was doing... and I closed & restarted VSCode & it still did nothing... but I just tried it again & it worked... very strange
and I leave VSCode open all the time, so if it was some sort of perf issue it would've opened eventually... anyway, it works now!


[BUG] Wingdings font not created in Mac OS

Description
When creating typeface for wingdings font, the skiasharp returns as Helvetica.
Font is available in the location : ~Library/Fonts
Code
SKTypeface typeface = SKTypeface.FromFamilyName("Wingdings", SKFontStyleWeight.Normal, SKFontStyleWidth.Normal, SKFontStyleSlant.Upright);
Expected Behavior
It must be return wingdings typeface.
Basic Information

Version with issue:  1.59.3
IDE:  Visual Studio 2017
iOS:  12.1
macOS:  10.14

You need to load the font from stream if the font isn't installed on the system.

I had a go at this, and it appears on my machine.
Try this and see what happens:
// this gives me 3 variations wingdings fonts
var windings = fonts.Where(f => f.Contains("Wingdings")).ToArray();


WIP: Conditional StartTLS elevation based on UseSSL

StartTLS workaround similar to Openshift. Would fix #1666 and #1894.
I have been able to test this locally using LDAP (simple auth) authentication type. #2349 made the testing an absolute pain though.
Other changes:

Migration to prevent breaking existing installations. UseSSL==true ldap authentications will jump to using StartTLS if no actions are taken to prevent it.
Documentation updates
UI field name updates

This is a bit weird. If I'm reading this correctly, the following UI selections (on the left) translate to the following LDAP connection (after the =):

SSL, SkipVerify = LDAPS
SSL no SkipVerify = LDAPS
No SSL, SkipVerify = Unencrypted
No SSL, no SkipVerify = StartTLS

This doesn't make a lot of intuitive sense to me. The "SkipVerify" UI field has no effect if we select SSL, and when we don't select SSL, not selecting SkipVerify results in the connection actually using SSL.
What should just let the user select one of three options:

Use LDAP over TLS (LDAPS)
Use StartTLS
Use unencrypted LDAP

You are absolutely right about. Having those choices would require changes to every component from UI to DB making it quite invasive.
The "SkipVerify" field should work as expected when using ldaps. The original choices were

SSL, SkipVerify = LDAPS
SSL no SkipVerify = LDAPS
No SSL, SkipVerify = Unencrypted
No SSL, no SkipVerify = Unencrypted (unused/illegal state)

It looks more like utilizing an unused field. Changing the checkbox labels could also clear things up. This is pretty much the only way to get StartTLS working without doing larger changes to the DB.
The other option would be to use port number to select the encryption scheme that the user probably wants to use and try the other one if neccessary. E.g. port 636 -> ldaps first, port 389 -> StartTLS first.

Yeah, I didn't like the previous UI either, but at least there was one option, whether to use SSL/TLS, since the verify option did nothing. Using the port is a bad idea. There's nothing special about the port numbers other than being the defaults. Treating them specially is sure to be error prone.
Making the changes I suggest shouldn't be too big of an issue since everything goes through an ORM-ish layer.

The port numbers in the second suggestion were there only to select the first method to try. Most of the installations are using default ports anyways and non-standard ports would only result in degraded performance due to multiple tries.
Having the extra fields in data model is the cleanest solution. However, it is not the simplest one either. @Unknwon: What do you think?

From what I understand, does @SergioBenitez suggests that replace checkbox of skip verify to a dropdown list with three options?

Use LDAP over TLS (LDAPS)
Use StartTLS
Use unencrypted LDAP

@Unknwon: No, not necessarily. I think the "Skip Verify" option doesn't make a lot of sense here.
I think what I would really like is the following:

Consolidate the two LDAP (simple auth, bind DN) options into one "LDAP" option.
Add a "Mechanism" (or whatever...) drop down selection with two options:
Simple
Bind DN
Add a "LDAP Protocol" drop down selection with three options:
Unencrypted
Secure (LDAPS)
StartTLS
Remove the "Enable TLS encryption" and "Skip TLS verify" checkboxes.

The form should update automatically to show only the relevant fields when either "Simple" or "Bind DN" are selected.
This requires a bit more work, of course, but I think it's likely worth it. For now, we could simply do 3 and 4 above, which was my original suggestion.

Thanks @SergioBenitez !
Agree to 3, but I don't think Skip TLS verify option could be removed?

A quick summary of what I'm planning to do:
Add an "LDAP Protocol" drop down selection with three options as @SergioBenitez suggested:

Unencrypted (hide Skip TLS verify checkbox, no TLS)
LDAPS (show Skip TLS verify checkbox, TLS verify on choice)
StartTLS (show Skip TLS verify checkbox, TLS verify on choice)

This approach requires auto-migration:

When UseSSL is true, treats it as LDAPS.
When UseSSL is false, treats it as Unencrypted.

Do not need to care StartTLS since Gogs does not support it before the point.
Thoughts?

Why can't we simply not show the Skip TLS verify options for LDAP?

Unencrypted will hide Skip TLS verify option as I listed.

Right, I understood that. But, we don't need the Skip TLS verify option at all for LDAP if we have the three options (Unencrypted, LDAPS, StartTLS).

For LDAPS, we need it to support self-signed certificates.
For StartTLS, we need it do determine whether to call StartTLS or not.

Ah, I see. So you're saying the Skip TLS verify option is currently being used to skip checking against known CAs? Any chance you could point me to the section of code that does this? If this is the case, then the option should be there for StartTLS as well. When a user selects StartTLS, we should always attempt to use StartTLS and error out if it doesn't work.

Here you go: https://github.com/gogits/gogs/blob/master/modules/auth/ldap/ldap.go#L202
By default, Go does not allow self-signed CAs.

Ah, excellent.
Okay then! Your comment:

Unencrypted (hide Skip TLS verify checkbox, no TLS)
LDAPS (show Skip TLS verify checkbox, TLS verify on choice)
StartTLS (show Skip TLS verify checkbox, TLS verify on choice)

looks perfect then!

:+1:

It looks like a decision has been reached. Some other points:
ldap.Source struct: LDAPS/StartTLS/Unencrypted selection looks enumeration based to me. ConnectionType string ("ldaps"|"starttls"|"ldap") or int based ids? Judging by the general database model separate table for types is probably not going to happen.
Other things is the migration strategy.

UseSSL == true -> ldaps, SkipVerify remains
UseSSL == false -> ldap, optionally set SkipVerify to false to remain consistent.

@vilppuvuorinen just int type should be good enough.

PS: I'll merge your PR first then start working on other matters.


TAA reprojection

initial version of re-projection of history buffer to the current buffer
issue with WebGPU upside-down, which needs to be solved
initial stage of adding a velocity buffer, unfinished as not needed initially

Velocity buffer can be very useful for WebXR Space Warp technique. https://developer.oculus.com/documentation/web/webxr-space-warp/
I guess the format of vector data and the way it is stored is important here. If it matches between implementations - that would be just amazing!

Velocity buffer can be very useful for WebXR Space Warp technique.

created an issue: https://github.com/playcanvas/engine/issues/6116


Status View of Localnet, add to CI for testing

Confirmation

[X] I have checked for similar issues.

Problem
As of now, when you run localnet, a lot of setup is occurring.  It's hard to find out what is going right or wrong.
Create a "view" that allows us to see the status of localnet.  This should have two formats: human-readable and json. With the json format, we can run automated tests against the state of localnet.
The human readable can look something like this (not complete yet):
+----------+-------------+
| bitcoind | not running |
| bfgd     | not running |
| bssd     | not running |
+----------+-------------+

we want to use this for CI testing, this are the metrics we want to show and test against:
First ETH block containing Batcher publication
First L1 ETH block from batcher address to batcher inbox address
Most recent ETH block containing Batcher publication
same as above, but most recent
number of Batcher publications
a count of the above
repeat the above for Publisher state roots
Total $HEMI paid out for PoP
a balance query of the PoP Miner address to the GovernanceToken contract
Total $ETH deposited/withdrawn (once we get that working in e2e testing)
not yet
again, using the json output, we can use this in automated testing and CI
Suggested solution
use go pretty https://github.com/jedib0t/go-pretty
Additional Information
No response

I am putting this in review, though there is no PR yet, just confirming with @max-sanchez on final test cases, but this is running in CI on a branch.  You can see the results here https://github.com/hemilabs/heminetwork/actions/runs/8883094859/job/24389209865#step:6:14

I believe this was resolved by #106, please reopen the issue if this is wrong.


[FLINK-7316][network] always use off-heap network buffers

What is the purpose of the change
For now, network buffers may be on-heap or off-heap along with Flink memory settings. As a step towards passing our own (off-heap) buffers through netty to avoid unnecessary buffer copies, we make network buffers always off-heap.
Note that this PR is based on #4447 and #4506 .
Brief change log

always use off-heap buffers for the NetworkBufferPool
move memoryType from NetworkEnvironmentConfiguration to TaskManagerServicesConfiguration
adapt heap size calculations in bash scripts and Java source code

Verifying this change
This change is already covered by existing tests, such as: TaskManagerServicesTest for the heap szie calculations; tests under flink/runtime/io/network for most other aspects of the direct use of network buffers, especially flink/runtime/io/network/buffer; all integration tests with a full stack and non-local communication.
Actually, we even increase the test coverage since most network buffer tests only tested on-heap so far which now does not exist anymore. These tests now cover the only existing option: off-heap network buffers.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (yes - as in the network communication part)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes - memory settings, but they effectively do not change except for the network buffers being off-heap now)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (docs, JavaDocs)

actually, I need to fix the test failures in ContaineredTaskManagerParametersTest and some failure in the flink-yarn-tests first...

ok, one test fixed, the other is not so simple but maybe @tillrohrmann can help with it:
Inside ContaineredTaskManagerParameters#create(), we calculate the amount of off-heap space that we need and for yarn, we use exactly this amount for setting the -XX:MaxDirectMemorySize JVM property without letting room for other components and libraries. This worked so far for the network buffers when memory as a whole was set to off-/on-heap and the flink-reserved memory was not completely used. Now, however, if set to on-heap, the -XX:MaxDirectMemorySize is too sharp. I'm unsure about the solutions:

remove setting -XX:MaxDirectMemorySize and let the JVM adjust automatically, or
add some "sane" default to our off-heap usage?

The same may apply to Mesos if ResourceProfile(cpuCores, heapMemoryInMB, directMemoryInMB, nativeMemoryInMB) is used. At the moment, only the other constructors are used leading to solution 1.

by cherry-picking the commits from #4506, plus some fixes for code which was changed in the wrong way previously, the failing yarn tests should now be fixed

FYI: I just rebased this PR onto current master to make this mergable and support further extensions

rebased again - this should be good to go. @StefanRRichter can you continue with this?

LGTM 👍

Will merge this now.


Can SENet be converted to encoder-decoder architecture?

Hello, Mr. Kikuta,
Thank you for sharing your code of SENet based on Keras.
I'm curious if this SENet can be converted to encoder-decoder architecture with skip connections?
Just like U-Net which can do the segmentation work?
Thank you.

In principle, YES.
See https://arxiv.org/abs/1803.02579 as an example of a combination of the SE block and U-Net.
But I have no plan to provide such kind of models in this repository, sorry.

In principle, YES.
See https://arxiv.org/abs/1803.02579 as an example of a combination of the SE block and U-Net.
But I have no plan to provide such kind of models in this repository, sorry.

Thank you, I will have a look on it.


Message new Branch(es) are ready to be merged

Feature Description
As a developer, I want the ability to create pull requests fast. Therefore, I create a branch and switch then to the UI to click the button create a pull request. To improve my workflow, I would appreciate it if there was a banner to quickly create Pull Requests, like there is on GitHub.
Notes:

these messages should only appear for really newly created Branches, e.x. 1 hour
It is available on the main page of the code repository
Example:

Screenshots
No response

Many people use git UIs without looking the command line output when pushing so I support this

Do you know Gitea will return a link to the git client, so you can just click the link to enter the process to create a pull request?

I never used that

I have to agree with @yardenshoham, I got used to the VSC Git Integration since I was kinda forced by my team to use a similar workflow and I don't have any console output for that. Thats why I need a button to do this for me 🙂

I'm not against this feature but just mentioned we have another one I used it frequently. I just sent a PR to improve the documentation. #23744

Additionally, the problem with the existing feature is that it is only triggered on the first commit.
Oftentimes, you want to have multiple commits before creating a PR.

Duplicate of #13196. There is a draft PR #14003

Oh, but at least we can make it alive again :)

Let's keep this one since there more discuss here. And I copied the content from that issue for @kolaente

As a user, I would like to see a prompt on the home page or repository page to create a new PR if I recently pushed to a new branch for which no PR exists yet so that I save clicking on the "new PR" button on the repo page.

This could be done purely through the db without any git calls since we already have all the data in the action table.
I think we can always assume the user has the rights to create a PR if they have the right to push to a new branch in that repo, so no need to check that.
We would just need another check if the repo pushed to is a fork to check with the head repo if a PR exists there and not in the fork repo.


Remove 'zip' example from update_info_plist

It doesn't seem to be relevant.

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[ ] I've read the Contribution Guidelines
[ ] I've updated the documentation if necessary.

Motivation and Context

Description

Ha, nice catch, thanks 👍  Can you address the comments from the bots?

Yes, I'll get to it ASAP!

Thanks ❤

@googlebot I signed it!

'ASAP' took a while - sorry about that. Checks seem happy now 👍

@KrauseFx Reviewed and all seems good, are you happy to merge?


any  quick doc on how to user? title seems super promising...

How about this?
https://github.com/wix/react-native-sane-listview/blob/master/README.md#usage

Something like this?
import SaneListView from 'react-native-sane-listview';

and then in your class
constructor(props) {
	super(props);
	this.state = {
		data: [{name:'Dan'},{name:'Lisa'}]
	};

	reload = () => {
		this.setState({'data':[{name:'John'}]});
	};

	setTimeout(reload, 3000);
}
render() {
	return (
		<SaneListView style={{top:20}}
			rowsById={this.state.data}
			renderRow={(rowData) => <Text>{rowData.name}</Text>}
		/>
	);
}

@DanielZlotin small PR for this #2  😄


Footer UI is changed

Pull Request
Title

Footer is changed
Description

I have added the hover effects and the animations
Related Issues

#90
Changes Made

Checklist

[ ] I have tested the changes locally
[ ] Documentation has been updated (if necessary)
[ ] Changes are backward-compatible

Screenshots (if applicable)

Additional Notes

Please label the Hacktoberfest-accepted and Gssoc-etd

make it right..the alignment.. @jayanththalla ..excellent work..keep it up

make it right..the alignment.. @jayanththalla ..excellent work..keep it up

i had changed this pr only can u review it?

can u label the hacktoberfest and the Gssoc ext


double-beep: Watch +237673215039

double-beep requests the watch of the watch_number +237673215039. See the MS search here and the Stack Exchange search here.

Approved by Makyen in Charcoal HQ


KAFKA-17718: Build code with java 17 instead of 21

Build code with Java 17 instead of 21 to avoid errors caused by module spotless not supporting Java 21.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Let us try https://issues.apache.org/jira/browse/KAFKA-16896 before merging this PR

close this PR as #17423 works well


"Find And Replace: Select All" hangs 1.3.2 and results in unexpected selections/cursors

Atom 1.3.2 on OS X 10.10.5.
I have a Clojure (.clj) file whose sections are marked with the text Section:  I want to select all section headings, copy them, and paste them at the top of the file.
I didn't get very far, though.  Selecting ^;; .*Section:.* seemed to hang Atom and I had to forcefully terminate it.  This reproduced reliably.
While working towards a minimal test case, I searched for section: instead, and found that it's more likely that it's just taking a very long time, because the resulting selections are quite numerous.  They're also rather... unexpected.

Notice that there are a ton of resulting selections - or perhaps cursors.  I'm not entirely sure what I'm looking at, but it doesn't seem to be all instances of section: having been selected.
Let me know if you'd like the original file.

There's 18 result found for section, so search is probably working well.
However there was work to change the data structure holding marker for search result. You possibly found a bug in that.

This issue was moved to atom/find-and-replace#627

@jeancroy Searching isn't the problem.  It's doing "Find And Replace: Select All" afterwards.


add reactor subscription example

Hello
I think subscription example should offer reactor publisher too.
I've tested on my playground, It works same as rx-java one.

@seongahjo Could you fix the build problems so I can go ahead and merge it?

@oliemansm sure

Thanks!


GAMES WONT WORK

Everytime I try and connect to a game it does work.  Attempts to reconnect 5 times then gives up.

Apparently the servers are down

The SSL certificate on the game node server had expired.  It was renewed but the server hadnt been restarted.  Fixed now


chore: v2.0.0-rc.0: update adversarial test: generic generator

next step in release process
https://github.com/slsa-framework/slsa-github-generator/blob/main/RELEASE.md#generic-generator-verifier-test
slsa-framework/slsa-github-generator#3576

failed with the correct error
https://github.com/slsa-framework/example-package/actions/runs/8743688384/job/23994819420#step:2:214
Fetching the builder with ref: refs/tags/v2.0.0-rc.0
Builder version: v2.0.0-rc.0
BUILDER_REPOSITORY: ramonpetgrave64/slsa-github-generator
verifier hash computed is 5891b5b522d5df086d0ff0b110fbd9d21bb4fc7163af34d08286a2e846f6be03
Error: Process completed with exit code 4.


schedule failed to startup the rest instances when one instance failed to startup

I created an instance scheduler and add to 8 instances, I noticed 8 instances don't startup during office hours. Further troubleshooting found one of the instances had EBS problem. Shouldn't scheduler skip that problem instance and startup the rest of instances?

Hi @daijing01
Currently the solution batches up the instances which are to be started and sends a batch request, so if one of the instance in the batch has an instance whose configuration is missing and is in a state where it cannot be started, all the instances in the batch will not be started. Also the batching is important it reduces the number of API calls required by the EC2 service and also improves the performance of the solution in a large scale. As a workaround there is an environment variable 'START_EC2_BATCH_SIZE' in the lambda function '<solution_stack_name>-InstanceSchedulerMain' which will reduce this batch size to 1 will skip the problematic instance and startup the rest of the instances. downside to this approach is which increases the API calls required by the EC2 services and it may also impact performance of the solution.
Thanks,
Praveen

Praveen,
Thanks, we will set the environment to 1.
Jing


auto join discovery room at registration using synapse parameter

self explanatory.

Test : after deploying this modification, I registered a new user and checked that the discovery room was joined (using our discovery room scripts)

Regarding https://github.com/eimis-ans/eimis-synapse/issues/130


slice scroll parallel support

Is there any support for slice scroll in parallel?
#44 and  #142 is simple common scroll example.
just like this api https://www.elastic.co/guide/en/elasticsearch/reference/6.8/search-request-scroll.html#sliced-scroll

The package provides access to the Scroll API (docs), so you can pass the slice parameter in the request body, as outlined in the example you've linked to.
As of now, there's no high-level helper for "scanning" an index, similar to the esutil.BulkIndexer helper, but it is planned for future versions.


Tiny persist.lua refactorings

This fixes some small issues with luarocks.persist module:

persist.load_into_table (used for loading rockspecs) used to read file twice on Lua 5.2/5.3;
persist.save_from_table (used for saving manifests, etc.) used to produce broken code when table contained strings with backslashes or Windows-like newlines.

Cool!
Since I gave you commit access recently, I take it as a request for code review. :) The write_table change looks good, thanks!
For fixes like the load_into_table change, feel free to commit directly. (In fact, since this whole PR fixes behavior only and doesn't change APIs, I think you could have committed directly the whole thing, but thanks for asking anyway :) )

For fixes like the load_into_table change, feel free to commit directly. (In fact, since this whole PR fixes behavior only and doesn't change APIs, I think you could have committed directly the whole thing,

Great, will do! I just decided to be careful the first time since some teams require that even small changes are approved by at least one collaborator other than the author.


chore: use m1 machines in ci

This PR resolves PHIRE-251
Description

Circleci is deprecating non-m1 machines: https://discuss.circleci.com/t/macos-intel-support-deprecation-in-january-2024/48718
Plus they claim big speed ups!
Draft until tested.
PR Checklist

[ ] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

use m1 machines in ci - brian

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (use m1 machines in ci - brian)

  Generated by :no_entry_sign: dangerJS against 0c48dbe63efaba704b5daef3642b2d118e789e0f


Properly handle exact routing; fixes #39

Here we go!

We had one test fail:
<Router> can be nested indefinitely

Yeah, I'll be looking into it soon, thank you!


itertools.compress

implement itertools.compress

Hi @pitachips , thanks for this nice addition! Please check the clippy build on travis. This build should be fixed. The azure build is broken on master, so that probably has a cause unrelated to your change.

Might want to rebase to kick the CI pipeline and see if it passes.

Leaving a reference to https://github.com/RustPython/RustPython/issues/1361 so that we can see from that thread when it is merged.


Build Error

Version is master
git clone https://github.com/roleoroleo/yi-hack-Allwinner-v2
git submodule update --init
then I do:
./scripts/compile.sh
or sudo ./scripts/compile.sh

Something went wrong in the command
git submodule update --init
Did you enter in the yi-hack-Allwinner-v2 folder?

Try again with this sequence and check the errors:
# git clone https://github.com/roleoroleo/yi-hack-Allwinner-v2
# cd yi-hack-Allwinner-v2
# git submodule update --init
# ./scripts/compile.sh
# ./scripts/pack_fw.all.sh


Bump transport to 8.8.1

:broken_heart: Build Failed

Buildkite Build
Commit: e0df70084071bfdbe9ac13231e3aec1f33632214

Failed CI Steps

:junit: Test results

Integration tests failed for unrelated reasons.


Fix indent level of a block quote that belongs to a bullet item

PR Summary
This PR fixes inconsistent indent levels among the translated and the original text.

PR Context
Otherwise, block quote does not belong to item 1.

@atusy Thanks for fixing it. ありがとうございます！


Update LAB[PL-400]_Lab02_Model_Driven_App.md

Replace with Click back
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by Tattedgeek on behalf of MOC Labs Review


Natural language FPs with rule 932240 PL2

This issue is part of the ongoing quantitative testing natural language FPs improvement task. See the main issue for details: #3392
Description
Rule 932240 struggles with false positives on natural language input. This is because it matches on apostrophes and quote marks in the middle of sentences (either single ' or double ").
This rule, 932240, is one of the new Unix RCE rules:
Remote Command Execution: Unix Command Injection evasion attempt detected
(generic detection, PL2, targets generic evasion attempts)
I believe this is the rule that tries to replicate (and improve upon) ModSecurity's cmdline transformation function. This rule is designed to detect evasion-like patterns:
- non-existing vars: cu$@rl, una$$me -a, uname$$u -a
- vars + spacing: v='u';cu$v\r\l google.com
- globbing pattern expansion: {n$u\c$u,-nlvp,777}
- globbing: garb=cur[l];$garb+google.com

Logs
Number	Payload
300	All told, people were taken with the concept of Stadia, but the execution just didn't work.
Rules triggered
  [Thu Nov 30 00:08:18.160748 2023]
      ModSecurity: Warning. Match of "rx [0-9]\\\\s*\\\\'\\\\s*[0-9]" against "MATCHED_VAR" required.
      [file "/etc/modsecurity.d/owasp-crs/rules/REQUEST-932-APPLICATION-ATTACK-RCE.conf"]
      [id "932240"]
      [msg "Remote Command Execution: Unix Command Injection evasion attempt detected"]
      [data "Matched Data: didn't found within MATCHED_VAR: All told, people were taken with the concept of Stadia, but the execution just didn't work."]
      [severity "CRITICAL"]
      [ver "OWASP_CRS/4.0.0-rc2"]
      ["paranoia-level/2"]
      [unique_id "ZWfSchXIgioxuQ8PfmG0PwAAAAA"]

Number	Payload
1430	But three years before that movie came out, he appeared in the episode "Bad Blood" of as Sheriff Hartwell.
Rules triggered
  [Thu Nov 30 00:29:31.637490 2023]
      ModSecurity: Warning. Match of "rx [0-9]\\\\s*\\\\'\\\\s*[0-9]" against "MATCHED_VAR" required.
      [file "/etc/modsecurity.d/owasp-crs/rules/REQUEST-932-APPLICATION-ATTACK-RCE.conf"]
      [id "932240"]
      [msg "Remote Command Execution: Unix Command Injection evasion attempt detected"]
      [data "Matched Data: Blood\\x22 of found within MATCHED_VAR: But three years before that movie came out, he appeared in the episode \\x22Bad Blood\\x22 of as Sheriff Hartwell."]
      [severity "CRITICAL"]
      [ver "OWASP_CRS/4.0.0-rc2"]
      ["paranoia-level/2"]
      [unique_id "ZWfXa2kRdL2E9hp7KX-dcQAAAIU"]

Number	Payload
5270	Now it's the older and more seasoned sprinters' turn.
Rules triggered
  [Thu Nov 30 00:29:39.646090 2023]
      ModSecurity: Warning. Match of "rx [0-9]\\\\s*\\\\'\\\\s*[0-9]" against "MATCHED_VAR" required.
      [file "/etc/modsecurity.d/owasp-crs/rules/REQUEST-932-APPLICATION-ATTACK-RCE.conf"]
      [id "932240"]
      [msg "Remote Command Execution: Unix Command Injection evasion attempt detected"]
      [data "Matched Data: it's found within MATCHED_VAR: Now it's the older and more seasoned sprinters' turn."]
      [severity "CRITICAL"]
      [ver "OWASP_CRS/4.0.0-rc2"]
      ["paranoia-level/2"]
      [unique_id "ZWfXcxXIgioxuQ8PfmG1GwAAAAQ"]

How to reproduce the misbehavior (-> curl call)
curl https://sandbox.coreruleset.org -H 'x-format-output: txt-matched-rules' -H 'x-crs-paranoia-level: 2' --data "payload=All told, people were taken with the concept of Stadia, but the execution just didn't work."

Your Environment

CRS version (e.g., v3.3.4): v4.0.0-rc2
Paranoia level setting (e.g. PL1) : PL 2
Operating System and version: CRS modsec2-apache container

Confirmation
[x] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.

To correctly fix this, we need to know which payloads are we catching with ' and ". I checked information contained in:

this issue
issue #2632 (based on which was rule 932240 created)
tests for rule 932240

I was able to find only one payload which uses " and none which are using ' (but we can count one for " of course):
""p""y""t""h""o""n""3"".""1""0"" ""c""m""d""
Do we know about any other payloads which needs to be matched using ' and/or "?

The original expression that matches ' and " comes from the cmdline.py script that was used to generate regular expressions in a way that was close to the t:cmdline transformation. t:cmdline removes any ', ", \ (plus some additional stuff).
Unfortunately, we have no idea what the exact patterns are. The original transformation function was developed by Marc Stern (pinged on Slack, maybe he'll answer).
I have the impression that we could replace the simple detection of quotes with detection of more than one consecutive quotes. That should take care of the following class of evasions: c''o''m""mand.
That still leaves the class 'command' foo, which we probably can't match without too many FPs.

That should take care of the following class of evasions: c''o''m""mand.

I was thinking about this, too.

That still leaves the class 'command' foo, which we probably can't match without too many FPs.

We can do this on higher PL.

Hi all. the t:cmdline transformation removes quotes to allow to check your pattern without evasion (with quotes). That doesn't mean that quotes are malicious. It's totally different approaches.
Indeed, two single/double quotes could be detected as an attack pattern (or at least suspicious), but a quote in the middle of letters is very common ("user's" or in French "l'utilisateur", etc.).
Also, an evasion could be kill '-'9; quotes are not inside a word.
All of this to explain that both approaches have to be combined: try to detect evasion mechanisms, then try to "decode" the evasion to check attack patterns.

Thanks @marcstern. Let's weaken the detection by detecting only patterns that we know to be evasion attempts.

I've opened #3494 which fixes the immediate issues. Please take a look, maybe it's enough, maybe we can improve on it, but at least it's something to start from.

The results of testing PR #3494:
Before:
Freq.   ID #   Paranoia Level
    822 932240 PL2

After (this PR):
Freq.   ID #   Paranoia Level
    229 932240 PL2

So, it's an improvement, but rule 932240 only drops from the 2nd worst rule to the 3rd worst.

…maybe it's enough, maybe we can improve on it, but at least it's something to start from.

Does it make sense to accept the proposed changes in the PR and then tackle the remaining FPs (pairs of quote marks)? Or is that all part of the same problem/pattern?

Let me check what part of the expression causes the match, maybe we can turn one more knob.

I wasn't restrictive enough with one of the patterns. I've fixed it in the PR. Would you care to run the numbers again?

@theseion On it…

(Posted in the issue, but cross-posting here for completeness:)

Freq.
ID #
Paranoia Level

11
932240
PL2


[core] fixing karma timeout error in travis (#666)

We are moving away from PhantomJS as the memory leak crashes Travis, and the maintainer stepped down from the project. We are now using headless Chrome.  This change assumes that the development machine has Chrome Canary installed. I'd prefer to keep the browser for dev/build server the same (that is, both Chrome). This may not be flexible enough though depending on what we assume about the development machine. Open to feedback on that.
I also had to change up the popover test, as the subsequent calls to anchor the popover only needed transform values relative to position set by the previous call. That was not the case with phantomJS. This issue is fixed by setting the transform to none before each call. I also cleaned up the expected value to be calculated based on variables. I'm not 10% confident all the math is right. It tests for the same values as before and passes the tests.
Fixes #666.
Signed-off-by: Jeeyun Lim<EMAIL_ADDRESS>

Thanks @mathisscott! Yes, I also think it should go in 0.9.3.


Lazily calculate the hash

We had the following use-case: around 70 feature flags, most of which are either fully turned on or off.
Because md5 hash is calculated for every feature flag, getIdentityFlags with local evaluation ended up taking 30-40ms.
The pr lazily calls getHashedPercentateForObjIds only when needed, and skips it for cases when it's not necessary (0 and 100%). With the patch getIdentityFlags executes in ~3ms.
Please let me know if something is missing or needs changing. Have a nice day!

Surprised that calculating these hashes takes so long! Thanks for the PR - will take a look.

Btw, is there a timeline for publishing a new version?
PS. I see that the last published version is 3.0.1 but there was a bump to 3.1.0 recently: https://github.com/Flagsmith/flagsmith-nodejs-client/pull/129.

OK 3.10.0 released - thanks!


ReferenceError: require is not defined

After installing this plugin, everything seems to work correctly on my local machine. However, when I deploy my Sanity studio to my staging environment and try to access it via my browser, I see the following error and the app does not load:
ReferenceError: require is not defined
    at https://my-domain.com/static/sanity-xxxx.js:xxxx:xxxx

I'm deploying my app via Vercel and targeting Node 18.x.
The error appears to be coming from this require("isomorphic-fetch") line, in my Sanity bundle:
function E$n(e) {
    if (!e || typeof e != "string")
        throw new TypeError("An ApiKey must be provided when initiating the Pexel's client.");
    return {
        typeCheckers: x$n,
        photos: _$n(e),
        videos: S$n(e),
        collections: C$n(e)
    }
}
require("isomorphic-fetch");

Hello @markplewis . May I know what configuration of the plugin are you using? Are you specifying an API_KEY or using the proxyClient ?

Fixed in version 3.0.2
Can you please try again using that version?
Thanks a lot for your help.

Thanks @dorelljames! Everything looks good now!


Increase reusability of actions

@utam0k , do you plan on continuing your work on this, or should we close this for now, and come back some other time?

@YJDoc2 Yes, this is still on my TODOs. Are you interested in this?

@YJDoc2 Yes, this is still on my TODOs. Are you interested in this?

I was just wondering if you were planning on continuing on this. I can try this if you want, let me know what exactly you wanted with this :+1:

I guess this isn't an enjoyable task, so I will take it.


pip instruction

I strongly suggest changing the installation instructions from
pip install ImagingReso
to
python3 -m pip install ImagingReso
Currently, if you run the above command, it uses whichever version of python it finds first.  I have both python2 and python3, and it defaulted to python2.

Updated the instruction.


[WIP] Omniauth GitHub integration

Todo:

Add test for auth
Add tests for integration

Closing this. Change of OAuth strategy to GitLab


F# 4.1 compiler raises warnings

I created a new console project in VS 2017 RC, add FSharpPlus NuGet dependency into it, copied the follwing code into it:
open FSharpPlus

[<EntryPoint>]
let main argv =
    let lst11n21n12n22 =
        monad {
            let! x1 = [1;   2]
            let! x2 = [10; 20]
            return ((+) x1 x2)}

    let neLst11n21n12n22 =
        monad {
            let! x1 = { NonEmptyList.Head =  1; Tail =  [2] }
            let! x2 = { NonEmptyList.Head = 10; Tail = [20] }
            return ((+) x1 x2)}

    let some14 =
        monad {
            let! x1 = Some 4
            let! x2 = Some 10
            return ((+) x1 x2)}
    0

and try to compile. It's compiled successfully, but with some strangely looking warnings:

1>------ Rebuild All started: Project: FSharpPlusTest, Configuration: Debug Any CPU ------
1>	C:\Program Files (x86)\Microsoft SDKs\F#\4.1\Framework\v4.0\fsc.exe -o:obj\Debug\FSharpPlusTest.exe
1>-g
1>--debug:full
1>--noframework
1>--define:DEBUG
1>--define:TRACE
1>--doc:bin\Debug\FSharpPlusTest.XML
1>--optimize-
1>--tailcalls-
1>--platform:anycpu32bitpreferred
1>-r:D:\github\FSharpPlusTest\packages\FsControl.1.0.9\lib\net40\FsControl.Core.dll
1>-r:C:\Program Files (x86)\Reference Assemblies\Microsoft\FSharp.NETFramework\v4.0\<IP_ADDRESS>\FSharp.Core.dll
1>-r:D:\github\FSharpPlusTest\packages\FSharpPlus.0.0.4\lib\net40\FSharpPlus.dll
1>-r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework.NETFramework\v4.6.1\mscorlib.dll
1>-r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework.NETFramework\v4.6.1\System.Core.dll
1>-r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework.NETFramework\v4.6.1\System.dll
1>-r:C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework.NETFramework\v4.6.1\System.Numerics.dll
1>--target:exe
1>--warn:3
1>--warnaserror:76
1>--vserrors
1>--utf8output
1>--fullpaths
1>--flaterrors
1>--subsystemversion:6.00
1>--highentropyva+
1>C:\temp.NETFramework,Version=v4.6.1.AssemblyAttributes.fs
1>AssemblyInfo.fs
1>Program.fs
1>
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 878 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 879 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 880 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 881 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 1118 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 1141 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 1164 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>FSC: warning FS3186: An error occurred while reading the F# metadata node at position 1256 in table 'ivals' of assembly 'FsControl.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The node had no matching declaration. Please report this warning. You may need to recompile the F# assembly you are using.
1>	FSharpPlusTest -> D:\github\FSharpPlusTest\FSharpPlusTest\bin\Debug\FSharpPlusTest.exe
========== Rebuild All: 1 succeeded, 0 failed, 0 skipped ==========

@vasily-kirichenko this is related to this issue reported in FsControl which at the same time is linked there to an issue in the F# compiler.
This issue has no effects and you can avoid those warnings with a #nowarn 3186, they just switched the compiler to display these warnings in F#4.0 in order to fix the underlying problem.
Still I'm surprised this is not fixed in VS2017, since I remember having seen it fixed.
Anyway if you're trying to measure the speed-up in the compile time be aware that in VS2017 only the initial speed up was merged.
From what I can see the other 3 PRs were still not merged for the RC.

Here's the issue in the F# compiler but as you can read there is supposed to be fixed.
If it's not we should report them.
In any case bear in mind that these warnings are "inoffensive" in these projects, you can safely ignore them.

@vasily-kirichenko Now that RC 2 is available I'm using it to compile the project and tried removing the #nowarn and the warning doesn't show anymore.
Have you tried it with RC 2?


Memory leak with querier rows

Hello,
We're running Promscale 0.6.2 and lately we've been having memory issues. Normally promscale uses about 7GiB, but suddenly it started to increase its memory usage over the course of 4.5 hours to about 65GiB before OOMing. Unfortunately we don't have graphs for promscale's memory usage directly, but we do have monitoring for the node's memory usage (there's about 15GiB memory used by other stuff on the node):

I managed to take a heap profile before it died (SVG attached). Most of the allocation seems to be from the two sync.Pools in pkg/pgmodel/querier/row.go. I don't think the traffic was unusual or causing linearly increasing amount of work, so this smells like a memory leak to me. Do you know what the issue could be, or if there's anything we could do to mitigate?
Thank you!
profile001.svg.gz

Thanks @knrafto for the detailed bug report. I've opened a PR to resolve the issue.

Thanks for the quick fix!
Sorry I'm not too familiar with promscale (or sync.Pool) but I'm trying to learn. Can you explain what the issue was? I thought that forgetting to Put() and item back into the sync.Pool would defeat the purpose but would be ultimately harmless, since the item would get GC'ed after whatever's holding it goes out of scope (in this case when the querier goes out of scope at the end of the query). Is there another reference to the rows somewhere?

I believe that sync.Pool keeps a reference to all items given out and not returned, which prevents the GC from collecting them.


Switched swift version to 3.0.1

no code changes needed.  Worked on travis and docker-compose on image i built using 3.0.1

@BDHernand I see several warnings from the compiler. Most (if not all) of these warnings are coming from SwiftyJSON. Looks like Kitura-Starter is still pointing to an old version of this library. Can you update Package.swift so that Kitura-Starter is using the latest SwiftyJSON and Kitura related libraries as well? For instance, we would need:
.Package(url: "https://github.com/IBM-Swift/SwiftyJSON.git", majorVersion: 15)

@rolivieri package.swift file updated.  No warning or errors reported now


Clipboard monitor window can be resized vertically

Did you want the Clipboard monitor window to be resizable only horizontally?  When resizing vertically it just exposes empty sapce below the instructions.
Or did you mean for the replicant to be able to have multiple entries of history?  That would be cool, if the replicant could be expanded vertically to show the last 5, 10, whatever entries in history.

No, the replicant stands alone, apart from the rest of Clipdinger and is only supposed to show the current clipboard contents. I briefly considered making the whole history list into a replicant.
I decided against it, because the replicant has to be visible on the screen to be useful. While that's possible with a one-liner, esp. on the right side of the screen where it won't be often obscured because of the yellow tab of windows leaving that space open, for several lines that's increasingly not the case.
Also, in my modus operandi, I use Clipdinger almost exclusively by keyboard. I'm already entering text and instead of ALT+C, I press SHIFT+ALT+C to insert from history.
To the core of this issue: The replicant is supposed to be made almost arbitrarily narrow. If it gets very narrow, the instructions line-break and the window has to be able to resize vertically to make it readable again. Most users probably won't bother, having red them once when first opening the window, but it's nicer than to just have a fixed vertical size.
Thanks for the note though.

No, the replicant stands alone, apart from the rest of Clipdinger and is only supposed to show the current clipboard contents. I briefly considered making the whole history list into a replicant.
I decided against it, because the replicant has to be visible on the screen to be useful. While that's possible with a one-liner, esp. on the right side of the screen where it won't be often obscured because of the yellow tab of windows leaving that space open, for several lines that's increasingly not the case.
Also, in my modus operandi, I use Clipdinger almost exclusively by keyboard. I'm already entering text and instead of ALT+C, I press SHIFT+ALT+C to insert from history.
To the core of this issue: The replicant is supposed to be made almost arbitrarily narrow. If it gets very narrow, the instructions line-break and the window has to be able to resize vertically to make it readable again. Most users probably won't bother, having red them once when first opening the window, but it's nicer than to just have a fixed vertical size.
Thanks for the note though.

Ah I see what you mean about resizing the window. How about using a split view to resize the replicant?  Or use a tooltip?

Ah I see what you mean about resizing the window. How about using a split view to resize the replicant?  Or use a tooltip?

It's possible, of course. I don't see a significant gain, though, only to be able to restrict the resizing vertically. A spliview has the small inconvenience, that if you wanted to make the replicant view wider, you'd have to resize the window first, then maybe make it smaller again with the split handle. Can use just the window for resizing from the start.
A tooltip would work of course, but normally I'd use a tooltip for infos that otherwise would clutter the GUI. In this case, it doesn't and it's immediately discoverable other than with a tooltip, where the user might take some time trying to figure out what this is before getting the idea that there might be tooltip.

It's possible, of course. I don't see a significant gain, though, only to be able to restrict the resizing vertically. A spliview has the small inconvenience, that if you wanted to make the replicant view wider, you'd have to resize the window first, then maybe make it smaller again with the split handle. Can use just the window for resizing from the start.
A tooltip would work of course, but normally I'd use a tooltip for infos that otherwise would clutter the GUI. In this case, it doesn't and it's immediately discoverable other than with a tooltip, where the user might take some time trying to figure out what this is before getting the idea that there might be tooltip.


Support failover worker while reading

What changes are proposed in this pull request?
Failover to an available worker while the current reading worker goes down
Why are the changes needed?
Fix #16295
Does this PR introduce any user facing changes?
No

@dbw9580 Please take a look.

UnavailableException is already a subclass of IOException, so wrapping an UnavailableException in an IOException does not make much sense. Ideally we need to fix the factory method of BlockWorkerClient to not wrap UnavailableExceptions.
https://github.com/Alluxio/alluxio/pull/15837 we have a PR that makes the BlockWorkerClient a subclass of AbstractClient which all master clients extend from. Supposedly AbstractClient is the sole place responsible for handling underlying exceptions. Could you please give it a shot and see if the it fixes the problem? @secfree

Could you please give it a shot and see if the it fixes the problem? @secfree

It looks like "yes" as it does not convert the UnavailableException to IOException anymore.
I think tests are still needed to confirm it.

Could you please give it a shot and see if the it fixes the problem? @secfree

It looks like "yes" as it does not convert the UnavailableException to IOException anymore. I think tests are still needed to confirm it.

A simpler fix would be to just remove the exception wrapping in BlockWorkerClient.Factory.create().

A simpler fix would be to just remove the exception wrapping in BlockWorkerClient.Factory.create().

Yes, I can test it. But is it still needed as #15837 is coming?

It would be best if #15837 can fix the problem and does not cause
regressions. If it does cause regressions, then a simpler fix on
BlockWorkerClient.Factory.create will be a more conservative solution. Your
valuable feedbacks would be much appreciated 👍!
Best regards,
Bowen
secfree @.***> 于 2022年10月25日周二 12:03写道：

A simpler fix would be to just remove the exception wrapping in
BlockWorkerClient.Factory.create().
Yes, I can test it. But is it still needed as #15837
https://github.com/Alluxio/alluxio/pull/15837 is coming?
—
Reply to this email directly, view it on GitHub
https://github.com/Alluxio/alluxio/pull/16296#issuecomment-1289946112,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVM5HGLIF4MMLXUJWTXKGLWE5LX7ANCNFSM6AAAAAARCFIVSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

A simpler fix would be to just remove the exception wrapping in BlockWorkerClient.Factory.create()

Tested this and it works. So this can be a regression base for the coming #15837 .

alluxio-bot, merge this please


Dark Theme contrast issues

Version 2.2 master (11/26/18)
What kind of request is this (question/bug/enhancement/feature request):
Bug, Usability
Results:

[x] Global Menu search box has poor contrast (light blue on medium gray)

[x] Text contrast in settings value fields is poor (Settings page)

Text in search box field not fixed.

Had to roll back some changes on master this will be on hold until we can merge again.

Version: Master 2.2 (2/6/19)
Confirmed fix.
Search box now has highly visible text:

Text in text fields such as on settings page now has highly visible text:


feat: containerinsights 3.1.18 release

What type of PR is this?
/kind feature
What type of PR is this?
/kind feature

What this PR does / why we need it: Release 3.1.18 container insights image
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
(https://github.com/microsoft/Docker-Provider/blob/ci_prod/ReleaseNotes.md)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2292

Relevant Lines:
2887

💛  - Coveralls


Update token generation TODO item

Maybe I'm just particularly dumb, but I had to spend some time figuring out why my GitHub releases were returning a 401. Turns out I encrypted my key like this:
travis encrypt GH_TOKEN=MY_TOKEN

as specified in the file, while I should have done
travis encrypt MY_TOKEN

Because Travis encrypts the env variable, too.

Oh duh I just noticed #51

Ah, my bad. I documented that wrongly. Thanks!


Add server caching of user's data

To avoid querying the DB repeatedly

cache (or return?) the next page of search results upon rendering current page of results
Goal is to decrease load time

We'll need to add some sort of time based autodelete functionality as we build out the cache too


Suggestion: allow partial applications

This is going to be a tough one. Consider a poor-man partial application hack:
interface Partial { f: Function; args: any[]; }
interface Partial0<r> extends Partial { 'know r': r; }
interface Partial1<a, r> extends Partial0<r> { 'know a': a; }
function partial<b, a, r>(f: (one: b, two: a) => r, one: b) : Partial1<a, r>;
function partial<b, a, r>(f: (one: b, two: a) => r, one: b, two: a): Partial0<r>;
function partial(): any {
    return {
		f: arguments[0],
		args: Array.prototype.slice.call(arguments, 1)
	};
}
function call<r>(partial: Partial0<r>): r;
function call<a, r>(partial: Partial1<a, r>, two: a): r;
function call() : any {
    var f : any = arguments[0].f;
    var args : any[] = arguments[0].args;
    return f.apply(undefined, args.concat(Array.prototype.slice.call(arguments, 1)));
}

function add(x: number, y: number) { return x + y; }
var addTo10 = partial(add, 10);
alert(call(addTo10, 5));

It works although it's rather cumbersome and awkward. The motivation behind this example is that creating data objects is less expensive than creating closures. It would be really nice if TypeScript could emit this design pattern in response to the syntax like this:
function add(x: number, y: number) { return x + y; }
var addTo10 = add(10, ..);
alert(addTo10(5));

@danquirk While I'm a big fan of functional programming I'm not sure this feature is really valuable enough at the moment given the work required, especially considering how few existing JavaScript libraries will have been written in a way that you can usefully take advantage of currying.
Partial application is not the same as currying.
Currying takes last parameter and makes a new function.
Partial application can be applied in any position.
Example:
function f(a:number, b: number, c: number) { return a + b + c; }

function g(x: (q:number,w:number) => number) { return x(1,2) }

g((q,w) => f(q,w,3))
// Same with partial application
g(f(_, _, 3))

// But we can use partial in any place, this is not easily done with currying
g(f(_, 3, _))
g(f(3, _, _))

It is not a complex feature.


Update Composer dependencies (2023-12-11-00-13)

Visual regression test passed!
View the visual regression test report


fix migration 3dda56f1c

This PR fixes the issue with slices that have no params and pass None to be decoded by json.loads.
@betodealmeida @graceguo-supercat @mistercrunch  @john-bodley

Codecov Report

Merging #5468 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5468   +/-   ##
=======================================
  Coverage   59.11%   59.11%
=======================================
  Files         372      372
  Lines       23751    23751
  Branches     2758     2758
=======================================
  Hits        14041    14041
  Misses       9695     9695
  Partials       15       15

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 fee5023...6b4a189. Read the comment docs.


similar(::Columns) should return a Columns of the same size

This was breaking this line because the call to similar above doesn't specify a length.

Codecov Report

Merging #47 into master will decrease coverage by 0.22%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
- Coverage    92.2%   91.98%   -0.23%
==========================================
  Files           6        6
  Lines         834      836       +2
==========================================
  Hits          769      769
- Misses         65       67       +2

Impacted Files
Coverage Δ

src/IndexedTables.jl
92.3% <100%> (ø)
:arrow_up:

src/columns.jl
93.02% <100%> (ø)
:arrow_up:

src/utils.jl
94.17% <0%> (-1.87%)
: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 cb4494c...5a7e323. Read the comment docs.


Switch to non-deprecated percent encoding methods (#34)

Fix for #34

@kylef Can we merge this please? This should fix the warning we are having for those of us who are using Mockingjay.

@kylef any updates?

@sssilver Nope, there are no updates.

@kylef There's an open pull request, is there a reason it isn't getting merged?

Is there a reason it isn't getting merged?

It may get merged, once I have found sufficient time to review the pull request.

Thanks for the updates @oysta!

No problems @kylef! Thanks for this library—I'm happy to help out with other things if ever needed.


Ensure no duplicates for Random themes

Sometimes the same random theme will play for movie titles. This is more prevalent when only a few themes present. It would be good to ensure that random themes are not duplicated as infrequently as possible. (even with smaller selections of random themes)

This is unavoidable, however i have taken steps to try and minimise this by attempting to get a new theme if the currently selected one has already been used. The best recourse is to add more theme MP3's.


cleanup: resolves gofumpt for cmd

This commit resolves gofumpt linter for
cmd folder
Updates: 1586
Signed-off-by: Yati Padia<EMAIL_ADDRESS>

/retest ci/centos/k8s-e2e-external-storage/1.21

@Madhu-1 need your review here.

@Madhu-1 need your review here.

/retest ci/centos/k8s-e2e-external-storage/1.21

/retest ci/centos/mini-e2e-helm/k8s-1.19

/retest ci/centos/mini-e2e-helm/k8s-1.20

/retest ci/centos/mini-e2e/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.19


Chrome Instances Clutter Dock on macOS When Running Flutter Projects

Description:
When running Flutter projects in Chrome on macOS, each debug session spawns a new Chrome instance which appears as a new icon in the dock. This behavior clutters the dock with multiple Chrome icons, particularly during frequent debug sessions.
Steps to Reproduce:

Run any Flutter project on macOS using the command flutter run -d chrome.
Observe that each new debug session adds a new Chrome icon to the macOS dock.

Expected Behavior:
Repeated runs of Flutter projects in Chrome should reuse the existing Chrome instance or at least not add new icons to the macOS dock.
Actual Behavior:
Each run of a Flutter project results in a new Chrome icon appearing in the dock.
Possible Solution:
Perhaps Flutter can include an option to reuse browser instances for macOS users, or better manage how new instances are handled with respect to macOS's dock.
Flutter Doctor Output:

Additional Information:

macOS Version: 14.3.1
Chrome Version: 124.0.6367.202

Thank you for looking into this issue!

Thanks for the report. This doesn't seem to be specific to chrome, but for desktop as well, I see new instance created for each debug session.

I looked at if there's an existing issue for this, but couldn't find one, so will keep it open for team's attention.

There is a Dart Debug Extension - https://chromewebstore.google.com/detail/dart-debug-extension/eljbmlghnomdjgdjmbdekegdkbabckhm - which could be used to run apps in debug mode from an existing Chrome instance. The tool could simply have an option --wait-for-browser-connection or --use-existing-browser, which would instruct the tool to wait for the browser to connect to it rather than opening a new browser automatically. The extension is necessary because there's no other way to create debug connections. You either have to launch a pre-configured browser instance, or you have to use the extension.


[Q] Howto change cursor's speed

Please fill out the below information:
Windows 10 version 1803 build 17134.112

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
I feel slow when moving cursor around, I need to increase cursor's speed to improve workflow.
Use case: I install archlinux on vmware box and then open wsl (actual fallback to bash.exe of Ubuntu) then ssh to archlinux box, doing vim and other stuffs, the problem that I couldn't increase cursor's speed, I changed console and xorg cursor's speed but both didn't work.

What's wrong / what should be happening instead:
As mentioned above, I expect a way to increase cursor's speed of wsl, howto?

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).

For WSL launch issues, please collect detailed logs.

See our contributing instructions for assistance.

No takers on the question. Generally speaking, you can ssh into remote Linux box (VM or otherwise) and use vim without any particular "slowness" (if that wasn't the case there'd be screeching to the contrary). If you are experiencing unusual slowness there isn't any knob to tweak to make it faster. Nothing is made artificially slow. There is not enough to go on here regarding the nature of the ask for any concrete answer. Bonne chance.

Idk but we have these to increase speed of curosr on console: https://wiki.archlinux.org/index.php/Linux_console/Keyboard_configuration#Adjusting_typematic_delay_and_rate
and on X: https://wiki.archlinux.org/index.php/Xorg/Keyboard_configuration#Adjusting_typematic_delay_and_rate
I tried both but bash.exe terminal seem didn't recognize, so probably it lacks of support.

If you wanted to change the keyboard typematic delay that is unrelated to "then ssh to archlinux box, doing vim and other stuffs". It is related to your keyboard settings. You want this.

Exactly what I need but even with short of Repeat delay still feel in slow motion, but thanks, problem partial solved.

but even with short of Repeat delay still feel in slow motion

I think I should probably clarify. What I meant was "if you wanted to change the typematic delay" that is how you do it, but I do not think that is actually what you want.
Changing your keyboard settings will change the behavior (user experience) of notepad.exe in Windows, as well.  You are complaining that ssh from WSL into a Linux VM feels like slow motion. I can't tell you why that is the case, because there is not enough to go on here regarding the nature of the ask for any concrete answer. I can tell you that, generally speaking, you can ssh into remote Linux box (VM or otherwise) and use vim without any particular "slowness" (if that wasn't the case there'd be screeching to the contrary). Which was the original answer given.

I don't think so, with two above links that I mentioned to change speed rate of cursor, I tested on native archlinux and result cursor's speed changed.
But the same configs, same os, everything is the same except that this time archlinux is on vm on windows10 host, two above setups didn't affect the cursor's speed.
Changing keyboard typematic delay of windows10 host does change cursor's speed in general and affect all windows10's applications, base.exe is also included.
Windows10 applied/forced speed rate of cursor on all its applications OR bash.exe terminal didn't respect/simulate archlinux's settings.
I expect bash.exe terminal has it own implementation to change cursor's speed. This is not a complaint but sound alike.

I expect bash.exe terminal has it own implementation to change cursor's speed. This is not a complaint but sound alike.

Appreciate the follow up. If you meant "I expect WSL [to have] it own implementation to change cursor's speed" then you could have reported the absence of /dev/port in the OP. Which is self explanatory and I took that you understood that for granted. Instead you said "still feel in slow motion" after changing your keyboard repeat rate and I made the (regrettable) mistake of attempting to clarify.


Clarify magic values in yolov5.go

I enjoyed your yolov3 library, and used it in my application successfully.
Was hoping to take advantage of some potential speed increases by trying out yolov5 but ran into an issue inside of processOutputs
Can you help me understand the magic values here?
Should this based off of source resolution?
https://github.com/wimspaargaren/yolov5/blob/master/yolov5.go#L207-L208
It looks like in v3 the loop was based on the number of elements in the outputs variable:
https://github.com/wimspaargaren/yolov3/blob/master/yolov3.go#L198

Hey @arwineap, the output of the yolov5 model is structured differently than the yolov3 one.
It's basically one big array consisting of parts of 90 numbers.
The first 4 numbers represent the bounding box, the fifth number represents the confidence and the other 85 numbers represent the confidence score per category.
Must say that I don't recall the exact reason for using 25200 rows 🙈 I hacked this together in like half an hour. I think rows should be equal to len(data)/90.
Let me know if that helps :)

Hey Alex @arwineap did you get this to work? I used rows := len(data) per @wimspaargaren response, but still have the same issue. Could you let me know. I posted another ticket that was related to out of bounds before I seen yours =) Tnx.

Hey Alex @arwineap did you get this to work? I used rows := len(data) per @wimspaargaren response, but still fails, it appears the confidence vals returned in the dataset are not valid either, all the confidence values are negative using the yolov5s.onnx model for the + 4 offset. It looks like the magic number of 25200 came from here https://github.com/doleron/yolov5-opencv-cpp-python/blob/main/cpp/yolo.cpp which I've been looking at to see where the Go problem is in this build.
confidence, row
-10.778692, 0
-10.662949, 1
-11.715211, 2
-12.660248, 3
-13.028133, 4
-13.262653, 5

Hey ever fix this?

After taking some further look at the code I seem to have found the issue. I don't know too much about yolo however the problem seemed to be that the data array was formatted differently than the way it was presented in the code. Replace the function with the one below, and it'll fix any problems you may have. I'll do a git requests sometime this week if the changes aren't made, glad to help anyone since this gave me a lot of headaches :)
`// processOutputs process detected rows in the outputs.
func (y *yoloNet) processOutputs(frame gocv.Mat, outputs []gocv.Mat, filter map[string]bool) ([]ObjectDetection, error) {
// FIXME add filter functionality
_ = filter
detections := []ObjectDetection{}
bboxes := []image.Rectangle{}
confidences := []float32{}
data, err := outputs[0].DataPtrFloat32()
if err != nil {
	return nil, err
}

rows := 25200
//stepSize := 85
nclasses := len(y.cocoNames)

for i := 0; i < rows; i++ {
	startIndex := (5 + nclasses) * i
	if startIndex == len(data) {
		break
	}
	confidence := data[4+startIndex]
	if confidence >= .4 {
		endIndex := (nclasses + 5) * (i + 1)

		scores := data[startIndex:endIndex]

		classID := getClassID(scores)
		confidences = append(confidences, confidence)
		boundingBox := calculateBoundingBox(frame, data[0+startIndex:3+startIndex])
		bboxes = append(bboxes, boundingBox)
		detections = append(detections, ObjectDetection{
			ClassID:     classID,
			ClassName:   y.cocoNames[classID],
			BoundingBox: boundingBox,
			Confidence:  confidence,
		})
	}
}

if len(bboxes) == 0 {
	return detections, nil
}
indices := make([]int, len(bboxes))

gocv.NMSBoxes(bboxes, confidences, y.confidenceThreshold, y.DefaultNMSThreshold, indices)
result := []ObjectDetection{}
for i, indice := range indices {
	// If we encounter value 0 skip the detection
	// except for the first indice
	if i != 0 && indice == 0 {
		continue
	}
	result = append(result, detections[indice])
}
return result, nil

}`

Hi @kampaidev, I inserted your change, but get the following panic now on the bird sample. Does this work for you?
panic: runtime error: index out of range [1632026] with length 1632000

Hi @kampaidev, I inserted your change, but get the following panic now on the bird sample. Does this work for you?
panic: runtime error: index out of range [1632026] with length 1632000

Where do you get this error? What line

@kampaidev, it's actually here on this line below in the yolov5.go. It appears the index is out of range. I haven't had time to investigate it.
confidence := data[4+startIndex]


update istioctl pc workload command for latest ambient wds

Please provide a description of this PR:
follow up to https://github.com/istio/istio/pull/44199
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ X ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ X ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

example output after fix:
NAME                                          NAMESPACE    IP           NODE                                            WAYPOINT               PROTOCOL
event-exporter-gke-755c4b4d97-f2mjf           kube-system  <IP_ADDRESS>   gke-gihanson-ambient-default-pool-090bfae6-6dj4 None                   TCP
httpbin-797587ddc5-b74np                      default      <IP_ADDRESS>  gke-gihanson-ambient-default-pool-090bfae6-qrkz httpbin-istio-waypoint HBONE
httpbin-istio-waypoint-6d86d5bd9-9pn8v        default      <IP_ADDRESS> gke-gihanson-ambient-default-pool-090bfae6-grql None                   TCP
istio-egressgateway-789d9959fd-r2zgt          istio-system <IP_ADDRESS> gke-gihanson-ambient-default-pool-090bfae6-grql None
...
...

/easycla


Can https://stemedplus.commons.msu.edu/ be redirected to https://stemedplus.hcommons.org/?

The old site is outdated and Julie is wondering if we can redirect the old site to the new one.

Sure, that will go live on the next update