Remove unused dependency in beam-model-pipeline

DESCRIPTION HERE

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand:

[ ] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

There was a warning about unused protobuf dependency in the build but if we remove the dependency, then the module does not build any more


unnecessary cast

Current coverage is 95.07%

Merging #994 into master will increase coverage by +0.02% as of 748c635

@@            master    #994   diff @@
======================================
  Files           74      74
  Stmts         9007    9007
  Branches      1711    1711
  Methods          0       0
======================================
+ Hit           8562    8563     +1
  Partial        109     109
+ Missed         336     335     -1

Review entire Coverage Diff as of 748c635

Powered by Codecov. Updated on successful CI builds.

thx!


Cohere - change metadata to meta

Follows #6570
Now that haystack-ai has been released,
we should also update the Cohere Chat Generator, which uses the old metadata attribute of Chat Messages.

Nice one, thanks @anakin87 🚢  it


geometry.js:41 Uncaught TypeError: Cannot read property 'getOrCreateGeometry' of undefined

Looks like a cool project.
I tried it out with a brand new ember app and just a simple application.hbs:
<a-scene>
  <a-box color="#6173F4" width="4" height="10" depth="2"
         position="-10 2 -5" rotation="0 0 45" scale="2 0.5 3"></a-box>
</a-scene>

Immediately, I get an error in the browser from within a-frame:

Thanks in advance!

@lukegalea Interesting!  I haven't seen that error.  Do you know if you are using AFrame 0.3?  They did a pretty major release recently, the last version I was able to test with was 0.2.
I will try later on to test out everything later today with AFrame 0.3 and see if I get the same error.  What version of Chrome are you on?

HI @mnannola - That's 0.3.1
I'll try with 0.2 and see how it goes!
Chrome is  52.0.2743.116 (64-bit)

@mnannola Forgot to say (Sorry for the delay) that 0.2 works fine, so it's definitely an issue with 0.3


build_sysext destroys potentially important files

Description
When running build_sysext to create a sysext image for Flatcar, from line 190 on everything that is not in the /usr folder is deleted. This could potentially be, but not only:

Important udev files
Important configuration files

It would be better to move everything that is now deleted, either into a folder that is packed into the sysext image so it can be put in place when the sysext image is mounted, or it should be put into a tar ball that can then be used to create the configation of the system.
Impact
Things might simply not work because important information is missing.
Environment and steps to reproduce

Set-up: Flatcar
Task: build_sysext in the sdk.
Action(s): Follow the documentation to create a sysext image for any given package and check the content. A nice example is zfs with zfs-kmod. This includes a configuration tree and udev files and everything is lost during the build.
Error: zfs was not working on the end system.

Expected behavior
Important files need to be available, either in the documentation directory in the sysext image or in a seperate tar ball deliverd with the sysext image. Even better would be if it is put in place by the sysext image on installation.
Additional information
Please add any information here that does not fit the above format.

The main use case of build_sysext is to build OS dependent sysexts (like docker, vendor tools, kmods etc). Contrary to user-supplied sysexts, these need careful integration with the base OS. build_sysext is not meant to be a generic packaging tool; ebuilds built into sysexts with this tool will always need some adoption.
For instance, Flatcar populates /etc at provisioning time, so there needs to be a custom mechanism to account for this. This is what the mangle scripts are for. (For example, here's the script for the containerd sysext: https://github.com/flatcar/scripts/blob/main/build_library/sysext_mangle_containerd-flatcar). This is the right place to align whatever the sysext packages produce with the specifics of Flatcar.
The mechanism provided by mangle scripts make it easy to use stock Gentoo ebuilds (to benefit from upstream version updates etc) and to align the sysexts to Flatcar specifics at the same time.

Closing as "works as intended", though I absolutely agree we need to further improve the sysext developer experience of the SDK.


Non-linear increments

Scenario - I'm wanting a knob where I can select every (whole) number between 0 and 720.
Problem - It doesn't seem to be possible to set values causing the step increments to be 1.
Reproduction - With a starting setup as follows...
[min]="0" [max]="720" [startDegree]="0" [endDegree]="360" (change)="log($event)"
I would have expected each "step" of the knob to be 2 (i.e. 720/360).  However, in testing, this gives increase in output value of 6.  Assuming this to be graphical precision due to the size of the knob, I expected that Increasing the [endDegree] to "720" (i.e. 2 full rotations), each "step" of the knob would now be 3.  However, the step is still 6, the knob instead just moves twice as far (i.e. 2 degrees).  This can be more clearly seen by setting [endDegree] to "3600" (i.e. 10 full rotations).  The step is still 6 but the knob rotation occurs in very large jumps.
I'll have a work through the maths and submit a pull request if I figure anything out.

I think it comes down to the constructor and the calculation of maxDistance.  I can kinda see why the code is there (though I'm not sure why this.maxDistance = Math.round((this.maxDistance * 10) / 100); isnt just this.maxDistance = Math.round(this.maxDistance / 10);?).
Perhaps one approach would be to have pixel based rotation (boolean @input flag)?  So 1 pixel of mouse movement is +/- 1 meterValue?  That would allow the user precise knob values whereas at the moment, the precision is determined by screen width.

Hello.
First: Of course Math.round((this.maxDistance * 10) / 100) == Math.round(this.maxDistance / 10)
:laughing:
It was just an initial calculation, and it is still there.
Second: this is done mainly for "small" screens. ie, I use it in an ionic app for mobile devices, but not only,  It seems reasonable to calculate a (more or less) physical distance and put in relation with the total degrees you can move. This way, the calculated distance (maxDistance=10% of your screen) is the correlation for the maximum degrees you can move. If you put 1 pixel  for 1 degree correlation, then you will need to move 1 pixel for each degree, and it depends on the screen if this is a reasonable distance or not.  ie, if you configure a knob with 10 degrees, you only move 10 pixels?? Think in a touch movement, is much more difficult, you need a good distance to be precise.. in fact, know this distance is small (I think it should be a 20% or 30%), but too much distance could be hard for mouse users... it depends.
Third: What we can do is to put the % of the distance in a configuration variable (now is 10, but you can put 50%, 100%, whatever).. or even, just put the distance in pixels in a variable (you know, in case this variable is filled, then don't calculate the % of the screen, and just put the number of pixels you want to move for the knob).


Support exponential waiting time for the next Batch retry

It always wait 2 minutes if the job failed. It would be better you can extend constant time to exponential waiting time and it would be helpful for failures like rate limit, EC2 metadata timeout, etc..
2019-12-26 14:57:14.921 [13/hello/65 (pid 38906)] Sleeping 2 minutes before the next Batch retry
2019-12-26 14:59:15.302 [13/hello/65 (pid 38906)] [1c74e540-8513-4c41-9115-268be11c7ed1] Task is starting (status SUBMITTED)...
2019-12-26 14:59:19.606 [13/hello/65 (pid 38906)] [1c74e540-8513-4c41-9115-268be11c7ed1] Task is starting (status STARTING)...
2019-12-26 14:59:22.810 [13/hello/65 (pid 38906)] [1c74e540-8513-4c41-9115-268be11c7ed1] Task is starting (status FAILED)...
2019-12-26 14:59:22.920 [13/hello/65 (pid 38906)] Batch error:
2019-12-26 14:59:22.920 [13/hello/65 (pid 38906)] Task crashed due to CannotStartContainerError: Error response from daemon: failed to initialize logging driver: failed to create Cloudwatch log stream: AccessDeniedException: User: arn:aws:sts::xxxxxxx:assumed-role/ecsInstanceRole/i-0b7b1xxxxx7e is not authorized t .This could be a transient error. Use @retry to retry.
2019-12-26 14:59:22.970 [13/hello/65 (pid 38906)]
2019-12-26 14:59:23.395 [13/hello/65 (pid 38906)] Task failed.
2019-12-26 14:59:23.451 [13/hello/65 (pid 39105)] Task is starting (retry).
2019-12-26 14:59:23.764 [13/hello/65 (pid 39105)] Sleeping 2 minutes before the next Batch retry

This would be great -- exponential backoff with random jitter would be ideal. Depending on how this is implemented internally, https://pypi.org/project/retrying/ might be useful (though I see Savin may already have picked this up which is awesome).


[Loc] added updated strings for sql-migration

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.002%) to 42.215%

Files with Coverage Reduction
New Missed Lines
%

extensions/machine-learning/src/common/processService.ts
2
81.48%

extensions/notebook/src/book/bookTreeView.ts
2
38.21%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
27647

Relevant Lines:
61217

💛  - Coveralls


Remove space from new Java PortabilityApi Jenkins job name

Jenkins jobs should not include spaces; see:
https://brentlavelle.wordpress.com/2015/09/24/dont-put-spaces-in-your-jenkins-job-names/

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

At some point we'll incorporate Jenkins job definitions into the build and automate validation for these types of things..


Feature - exclude field name using code-first approach

From @Pinox on March 22, 2017 18:20
As requested I opened a seperate thread for this request.
In linq we can't exclude field name in the select statement. List.Select(x=> new { column1 , column2 })
using code first approach in EF it would be nice if we can say "give me all table fields except the byte[] field of 20mb. By selecting the field names in the current linq scenario changing the database (code first - adding columns) you now always have to remember to add the new columns in the select statements. A better scenario would have included those new fields but still excluded the large byte[] field. List.SelectAllExcept(x=> new { byte[] columns1 , byte[] column2 })
Using currently .net core 1.1
Thks.
Copied from original issue: dotnet/corefx#17391

From @svick on March 22, 2017 19:28
Reposting my comment from https://github.com/dotnet/corefx/issues/17376:

Also, in linq we can't exclude field name in the select statement.

I'm not sure this is something that makes sense in the general LINQ. On the other hand EF Core supports explicit Include, so maybe it could make sense if it also supported explicit Exclude. In that case, it might make sense to request this feature from EF.

Part of #3797 - lazy loading should also be available for non-navigation properties. We will need API for marking such properties as lazy.

Consolidating issues and closing as a duplicate of #1387


Ability to render primitive of requested size(capsule, box, sphere)

Done in https://github.com/Esentiel/HurricaneEngine3D/pull/39


Playground is broken in Safari

The playground seems to be completely broken in Safari 9.0.2.
Code is not being evaluated, or if it is, the panel with the result is not being shown in any case.
I don't feel this issue needs any more details. It's just broken.

It is not completely broken. You “just” have to scroll down to see it. Can’t see the exact reason for this immediately.

@nwin: Try giving it a few tries. I thought I experienced that behavior where the "result panel" was just hidden out of sight and I had to scroll, but when I posted this issue, and still, if I go and try it out now, I can't even scroll down to see the result.

I think this is addressed by #167.

Fixed by https://github.com/rust-lang/rust-playpen/pull/167, thanks @killercup!


AVX build crashed at launch while SSE3 is fine

Backgrounds

App Version: Thorium_AVX_121.0.6167.204.zip
OS: Windows 11 Pro for Workstations x64 23H2 v10.0.22631.3155
CPU: Intel Core i3-2310M
GPU: Integrated Intel HD Graphics 3000

Problems
As the title says.
Logs
Within THORIUM.BAT with this command below,
"%cd%\BIN\thorium.exe" --user-data-dir="%~dp0%\USER_DATA" --allow-outdated-plugins --enable-logging=stderr --v=99 --vmodule=metrics=99 --blink-platform-log-channels > log.txt 2>&1

I was able to obtain additional logfile beside the Crashpad folder.
There are hundreds of command switches, let me know if I miss something.
Thorium_AVX_121.0.6167.204_Logs_2024-03-06T07.24.00.zip

@Unknown78 Unfortunately, some lower end 2nd Gen CPUs have AVX, but not AES (notice AES-NI is greyed out). You will have to use the SSE3 version in this case.

@Alex313031
Could there be four variant of builds in this case?

AVX2 + AES-NI
AVX + AES-NI
AVX
SSE3

If you're too busy to do that, at least, could please kindly explain the tutorial to compile it into AVX build but not with AES-NI?
I've looked up the keyword aes in this repo, and it spread across 41 files, I don't know which one to modify.


alt+click in terminal resets scroll position and prevents click-on-link

VSCode Version: 1.37.0 (system setup)
OS Version: Win 7 (Windows_NT x64 6.1.7601)

Steps to Reproduce:

(optional) set "editor.multiCursorModifier": "ctrlCmd" so that links are clickable via alt
open terminal → hold enter until the buffer is large enough for the terminal to be scrollable → scroll up
alt+click anywhere

result: scrolls to the bottom of the terminal. If you alt+click on a link, the file doesn't open (note: if you click while the terminal is scrolled to the bottom, it works).

Does this issue occur when all extensions are disabled?: Yes
Is this fixed in insiders?: No

Terminal settings:

I can replicate this behaviour, VS Code Version: 1.37.0, macOS 10.14.6 (18G87).

If you alt+click on a link, the file doesn't open (note: if you click while the terminal is scrolled to the bottom, it works).

This is how I found the issue, links in my test results would only work when terminal was scrolled to the bottom.
Happy to help provide more debugging info or anything else that can be useful here.

+1 I have exactly same issue as @dwelle

https://github.com/xtermjs/xterm.js/pull/2381

While the alt+click on link has been fixed, the scroll-to-bottom still happens. This is a problem when you mis-click a link (happens a lot to me) -- it scrolls you down and you need to scroll back up to try and click again, which is a pain when working with long stack traces.
Can we maybe re-open this, or should I create a new issue?

@dwelle created https://github.com/microsoft/vscode/issues/80496, https://github.com/xtermjs/xterm.js/issues/2401


v0.39

Version 0.39
Released December 18, 2023

Refactor holiday categories handling (#1584 by @arkid15r)
Update Azerbaijan holidays, add l10n support (#1579 by @KJhellico)
Update Israel holidays (#1557 by @arkid15r)
Update Laos and Thailand special observed holidays (#1587 by @KJhellico)
Update Makefile: fix snapshot generation process (#1594 by @arkid15r)
Update NYSE holidays: fix New Year's Day observed (#1585 by @KJhellico)
Update Netherlands holidays: fix Liberation Day (#1592 by @KJhellico)
Update Singapore 2024 holidays (#1588 by @KJhellico)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

699 of 699 (100.0%)  changed or added relevant lines in 63 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10327

Relevant Lines:
10327

💛  - Coveralls


publish<EMAIL_ADDRESS>

Requested by: @armcknight
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave the #retract comment under this issue to retract the release (original issuer only).
Targets

[x] github
[x] cocoapods[sentryprivate-cocoapod]
[ ] cocoapods[sentry-cocoapod]
[ ] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

@kahest, I thought we wanted to do a beta and still include https://github.com/getsentry/sentry-cocoa/pull/3079, because we have some risky features in 8.8.0. cc @brustolin.


Addon by url

Is your feature request related to a problem? Please describe.
 You can then input an url and specify anything that you might need to such as usercss, schemes, includes etc.
Describe alternatives you've considered
"HEY YOU KNOW ON MARKETPLACE YOU CAN ADD YOUR THEMES HUH, EVER HEARD OF MARKETPLACE???????!?!?!?!?!?!?!??!?!!??!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!????????????"

Going to close this because we will be moving over to a mono-manifest approach where you can just make a PR to the main repo and add whatever theme you want to it.


Colaboration with siuba project

I believe your great initiative could be stronger if you join forces with siuba, which has the same philosophy, instead of building a new project from the ground up. What do you think about that?

Thanks for the suggestion.
I talked to @machow a couple of weeks ago. These two packages are focusing on different aspects. suiba is more powerful with SQL-generating support; while datar is more stringent on following the original dplyr and family's API design. And datar is now pretty complete on porting all of the APIs from dplyr, tidyr, and tibble.
I have kept an eye on suiba's development, too. You may check out some of my comments on issues of suiba.

I get it, thank you for the clarification. I just wish users could use the same package for both purposes.
By the way, what other packages do you plan to port? forcats and purrr would be great additions

Great mentions about forcats, purrr and even dbplyr, a dplyr backend for databases, which functions similarly as suiba does. And maybe other related packages.
Maybe I would make it independent of datar, or optional to datar (i.e. to be installed via pip install datar[dbplyr]). Since the backend support (https://github.com/pwwang/pipda) for the piping syntax is independent, they don't have to be bundled with datar.
I need to block out a period on my calendar to implement then, but obviously not in a near future.
Of course, contributions and PRs are welcome for those ideas.

Yes, dbplyr would be fantastic too. Nice to know you are open to it. Thank you for the feedback

You are absolutely welcome!
Leaving this open for future discussions and people who may be interested in this topic.


Missing cref documentation

Partially linked to my feedback #7623 this documentation doesn't explain or link to an explanation about how the cref attribute is constructed but a documentation exists here: https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/xmldoc/cref-attribute

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

ID: 39152ded-70ce-012d-ffc3-a8b1a78f3308
Version Independent ID: 809078f6-169a-0ffe-5f33-d5a359aeefc5
Content: Documenting your code with XML comments
Content Source: docs/csharp/codedoc.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

This should also be handled as part of the consolidation of the C# Guide.


Gray out / darken the background when clicked

It would be great if the background would darken when the action button is clicked so the attention would be centered in the sub-buttons. In some screens there's so much "noise" that the labels are unnoticeable.

There is already a prop for that, called bgColor.
If you want to darken the background, you can type something like <ActionButton bgColor="rgba(0,0,0,0.2)" ...
Hope this helps.

Am 28.01.2016 um 15:34 schrieb nbastoWM<EMAIL_ADDRESS>It would be great if the background would darken when the action button is clicked so the attention would be centered in the sub-buttons In some screens there's so much "noise" that the labels are unnoticeable
—
Reply to this email directly or view it on GitHub.

Great!!!
I thought that was for the button background...
Thanks!!


logsmith cli is not exiting the terminal

After logging into a new profile using
logsmith --login some-profile the cli is not releasing the terminal again.
One has to use the timeout command right now to release the terminal currently, like that:
timeout 10 logsmith --login some-profile
Tested on a Manjaro Linux machine
Logsmith version: logsmith 6.0.0

This should be fixed by https://github.com/otto-de/logsmith/pull/13

This should be fixed by https://github.com/otto-de/logsmith/pull/13

Can you please check if running
logsmith --login some-profile --oneshot
with version 6.1 works as you'd expect?

Can you please check if running
logsmith --login some-profile --oneshot
with version 6.1 works as you'd expect?

Thank you! It works like a charm on a Manjaro linux.

Thank you! It works like a charm on a Manjaro linux.


The Alpha-clip demo with LLAVA will constantly repeat a sentence under certain specific images.

Just like this. I would like to understand why this situation occurs.

Hi, this image is too difficult for LLaVA to understand and LLaVA itself have similar case like circular duplication. A straightforward solution is to increase duplication penalty during beam search.

Thanks


Adopters: Korifi

The Cloud Foundry project Korifi is using kpack.
While I was in the ADOPTERS.md file, I've corrected the link to open a PR against this repository.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.11%. Comparing base (49014e2) to head (5c8e0ca).
Report is 55 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1756      +/-   ##
==========================================
- Coverage   67.34%   67.11%   -0.23%
==========================================
  Files         140      144       +4
  Lines        8886     9051     +165
==========================================
+ Hits         5984     6075      +91
- Misses       2393     2456      +63
- Partials      509      520      +11

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Fix tests and remove zips

Issue #, if available:
Description of changes:

Made the unit tests work with cicd and dev deploy.
Removed the zip files. Added code to zip them during deployment.

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

If we make this change, can we replace the CICD and README commands to just run yarn-deploy instead of yarn && yarn deploy?

No. The install packages just installs the packages required to build boto3-layer.zip and es.zip. Other packages would still need to be installed via yarn.

When the packages get zipped, they will end up on the user's local disk and in the repo. We dont want these files to be committed to the repo. Could you please add the necessary zips and related artifacts into the .gitignore?

Done


Adding myself as admin for receiving debug emails.

Signed-off-by: Wesley Queiroz<EMAIL_ADDRESS>

:+1:


[ISSUE #229] Fix 404 issue due to setting log path to relative path

Fixes #229
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Change the relative reference to the logo in src/theme/Footer/index.tsx: 81 to an absolute reference
Documentation

Does this pull request introduce a new feature? (yes / no)
no

LGTM


Add a logging API

Just added a re-export of wasm_rs_dbg::dbg in the end, as there's lots of external crates for wasm logging.


Uncaught RangeError: Invalid count value

[Enter steps to reproduce:]

...
...

Atom: 1.40.1 x64
Electron: 3.1.10
OS: Mac OS X 10.13.6
Thrown From: Atom Core
Stack Trace
Uncaught RangeError: Invalid count value
At /Applications/Atom.app/Contents/Resources/app/static/<embedded>:14

RangeError: Invalid count value
    at String.repeat (<anonymous>)
    at ScreenLineBuilder.emitIndentWhitespace (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1199795)
    at ScreenLineBuilder.buildScreenLines (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1194618)
    at DisplayLayer.getScreenLines (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:95252)
    at TextEditorComponent.queryScreenLinesToRender (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:79712)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:70365)
    at TextEditorComponent.updateSync (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:67993)
    at visible.suppressUpdates.updatedSynchronously.updateScheduled.i.getScheduler.updateDocument (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:67021)
    at ViewRegistry.performDocumentUpdate (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:325676)

Commands

Non-Core Packages
atom-beautify 0.32.2
atom-html-preview 0.2.5
atom-live-server 2.2.0
atom-runner 2.7.1
atom-xcode 0.1.3
browser-plus 0.0.98
emmet 2.4.3
markdown-preview-plus 2.5.5
markdown-writer 2.7.3
pigments 0.40.2
pretty-json 2.0.1

Thanks for the report!  Can you confirm a few things for us?

What were you doing at the time of the error?
Is the error is reproducible?
If reproducible, do you see the error in safe mode (atom --safe)?


Sharing our NaturalBench

I am Zhiqiu Lin, a final-year PhD student at Carnegie Mellon University working with Prof. Deva Ramanan. Your work on MIR is very interesting!
I wanted to share NaturalBench (NeurIPS'24 D&B) in case you are looking for more benchmarks:
NaturalBench (https://linzhiqiu.github.io/papers/naturalbench/) is a vision-centric benchmark that challenges vision-language models with pairs of simple questions about natural imagery. Unlike prior VQA benchmarks (like MME and ScienceQA), which blind language models (e.g., GPT-3.5) can solve, NaturalBench ensures such shortcuts won’t work. We evaluated 53 state-of-the-art models, and even top models like GPT-4o and Qwen2-VL fall 50%-70% short of human accuracy (90%+), revealing significant room for improvement.
We also found that current models show strong answer biases, such as favoring “Yes” over “No” regardless of the input. Correcting these biases can boost performance by 2-3x, even for GPT-4o, making NaturalBench a valuable testbed for future debiasing techniques.
Check out my Twitter post about it here: https://x.com/ZhiqiuLin/status/1848454555341885808.
🚀 Start using NaturalBench: https://github.com/Baiqi-Li/NaturalBench
Best,
Zhiqiu

Hi Zhiqiu,
Thanks for your kindly sharing! We will consider to add the experiments on NaturalBench in our next version. We believe this impressive benchmark can greatly enrich the evaluation of our work!


Tabs JavaScript

I added the JavaScript for the tabs at the top of the page. I'd kind of like to try and add an animation for the selected line to move between the options, but that probably shouldn't even go on the backlog.
Unrelated Changes

Added a fetchpriority of "high" to the facts.js script as is best practice

Partially resolves: https://trello.com/c/hQX0RWs1

everything looks good. great work!


Tasks

[x] (Web5 Team) Cut preview/alpha release for acceptance testing. Version: 0.8.2-alpha-20231101-b865b10
[x] (Open Source Programs Engineering) Run Web5 tests against acceptance suite
[X] (Open Source Programs Engineering) Create new branch on site repo for the upgrade.

Branch Name: v0.8.2-release

[X] (Open Source Programs Engineering) Open PR on this branch.

Link to PR: https://github.com/TBD54566975/developer.tbd.website/pull/960

[ ] (Open Source Programs Engineering) This will trigger a Deploy Preview on Netlify.

Link to Deploy Preview on Netlify: ❌ Build is failing
https://app.netlify.com/sites/tbd-website-developer/deploys/6544257a55b5d200078020a5

[X] (Open Source Programs Engineering) Update the Web5 version to the alpha release in site monorepo on the upgrade branch
[X] (Open Source Programs Engineering) Run the monorepo tests - this will cover code snippets and examples
[ ] (Open Source Programs Engineering) Check results of tests

Link to the tests results: ❌ Fail

[ ] (DevRel) If errors, issue updates to guides, examples in the release branch until tests pass
[ ] (Web5 Team) Cut release of Web5 available on Releases Page
[ ] (Open Source Programs Engineering) Update Web5 version in site to new Web5 version on branch
[ ] (Open Source Programs Engineering) The release of Web5 will trigger tbdocs automation and open a PR for that against the site. Merge that PR in the branch.
[ ] (Open Source Programs Engineering) Rebase and merge site to main, run tests, push upstream

opened https://github.com/TBD54566975/web5-js/issues/260 for test failure

opened https://github.com/TBD54566975/web5-js/issues/259 for test readRecordByIdFromDid

send() failing. Not sure if it's because of the new record format. Asked here.
Failing tests:

sendRecordToRemoteDWNs
sendProtocolToRemoteDWNs
sendRecordToDWNOfRecipient


Filter all values in dataset <0, send patient list to Aida

No longer relevant, psuedo-imputed values have been described.

Done.


If decoding an invalid disposal value, default to Any rather than failing.

Unfortunately, malformed GIFs exist in the wild, and it's often necessary to be able to
handle them. While I don't think silently ignoring invalid disposals is ideal, I believe
this is the most useful behavior given the current API design.

@bvssvni @nwin is this crate dead?

It should not be.
This change looks OK to me. Merge?

Merging.


Left-Right arrow in treeview should open/close tree

On the left-hand side treeview, I'd like to be able to open/collapse folders with the left/right arrows on the keyboard, like in SSMS.
Here:

@ststeiger this sounds like a reasonable suggestion.  I think you can accomplish the same behavior using the space key to toggle expand\collapse state.

@kburtram: Ah, interesting. Thanks, works for me in the meantime.

Use of the space key is not sufficient when you dealing with the stored procedure folder that has 200 objects. It requires you to still page all the way up before you can collapse the folder. The same ability that the file explorer view has for arrow key control should exists in the Server View....this functionality is present in SSMS.
An example of what can be done with files....just to visualize what I'd see the Server Pane having:
vscode_example_key_file_collapsing.zip

+1 FYI, Enter also works but it's much more convenient to use the arrow keys so you don't have to either jump around the keyboard or use two hands.

From an accessibility perspective, this is also a must have.
Note that in explorer, keyboard accessibility works as normal. Is there anything holding back from using the same underlying control?

I believe this support is available in the latest ADS insiders version - targeted for May release.

I believe this support is available in the latest ADS insiders version - targeted for May release.

Meaning 1.44? Left/Right arrows still don't work for me

Support is available in latest insiders, targeted for July release.


Header section in aggregated report Insights page.

Feature Request
Description:
Add section in the Insights section: Header (Information about analysis arguments), filtering options, date of the analysis.

Released as a part of v0.10


test downloads…

using nightmare, I try test downloads on a page. depending on nightmares configuration, it behaves differently but not correct, the download does not start
I.click('Download') // triggers a request to a pdf, which should be downloaded I.acceptPopup() // does accept the download prompt
What are you trying to achieve?
if show: true is set, electron shows a download dialog but does not accept the shown dialog
if show: false is set, electron opens anyway after I.click('Download') has been clicked but the download dialog is neither shown nor confirmed.
Details

CodeceptJS version: v0.5.1
NodeJS Version: v4.4.5
Operating System: Mac OS
Nightmare
Configuration file:

{
  "tests": "./*_test.js",
  "timeout": 10000,
  "output": "./output",
  "helpers": {
    "Nightmare": {
      "url": "http://localhost",
      "show": true
    }
  },
  "include": {
    "I": "./steps_file.js"
  },
  "bootstrap": false,
  "mocha": {},
  "name": "test"
}

I have the same issue. I think there's a way using the Nightmare helper you can hook into the electron 'will-download' event and maybe auto accept. But this would be a great feature to have


LoggerMessage.Define requires all parameters are defined in the formatString.

I'm currently moving some logging over to the more performant approach of adding extensions to Actions created using LoggerMessage.Define. In some cases, our current logging is passing extra params which are not used in the log message but are recorded as structured objects using Serilog.
It doesn't appear that currently, I can define anything which doesn't include placeholders for all of the objects in the formatString. Doing so results in an ArgumentException.

System.ArgumentException: 'The format string 'xxx' does not have the expected number of named parameters. Expected 2 parameter(s) but found 1 parameter(s).'

Is there any possibility of extending this to support cases where not all of the objects are expected to be rendered into the formatted message?

@stevejgordon Any recommendations on how to achieve this?

@muratg I haven't dug into the codebase to know what's possible but I'm wondering if there could be an overload that takes a Dictionary<string, object> of additional data which is then included in the serialised log messages? Similar to what seems to be possible when using scopes?

Yes, I have been wondering about the same thing myself. The action returned from LoggerMessage.Define() will log a TState type of some internal type LogValues<T0, T1, ... TN> which encapsulates the arguments passed to action from Define(). The given LogValues type is optimized for the N number of arguments.
Example:
public static Action<ILogger, T1, T2, Exception> Define<T1, T2>(LogLevel logLevel, EventId eventId, string formatString)
{
    var formatter = CreateLogValuesFormatter(formatString, expectedNamedParameterCount: 2);

    return (logger, arg1, arg2, exception) =>
    {
        if (logger.IsEnabled(logLevel))
        {
            logger.Log(logLevel, eventId, new LogValues<T1, T2>(formatter, arg1, arg2), exception, LogValues<T1, T2>.Callback);
        }
    };
}

The usage of TState  in ILogger.Log<TState>() method is implementation specific:

Most loggers only uses the state to format message string by passing it to Func<TState, Exception, string> formatter
NLog tries to cast the TState object into a IReadOnlyList<KeyValuePair<string, object>> and if successful adds state to it's LogEventInfo class.

Thank you for your feedback. We're closing this issue as the behavior discussed seems to be by design.


Add top maps and task creation

Need to add a separate top map for each user and the ability to create tasks on this map.

Blocked from #19.


Template structure simplification

It has been a few weeks I am thinking about that, and a discussion with Kabir a few days ago finally convinced me that refactoring the Template hierarchy was a good idea.
The main reason for that refactor is the unneeded complexity of the old Template code: we only need to represent "simple" templates such as logical qubits and their sides. Being able to represent arbitrary templates using the ComposedTemplate class was nice, but not a requirement. With the need to query templates about their middle lines, sides, and likely other properties in the future that were not easily implementable (or even correctly defined) for all the possible ComposedTemplate instance, removing that intermediary seemed like a necessity.
This (rather large, sorry) commit aims at greatly simplifying the Template-related module. The interfaces are left unchanged, which reduces by a lot the possibility of introducing errors with this refactor and reduces the number of changes.
Note that this is only a draft pull request, I really want other maintainer opinions on that before even considering it mergeable.
Note also that both #71 and #274 will be impacted by this refactor.
Finally, this is also an experiment to test another idea for automatic detector computation. I will use that PR as a base for another PR on that, and see if it turns out to be a nice approach.

I'm not sure what kind of complex structures we would need, but I think both corners and junctions can be represented as a block graph?

Yes they can be represented by block graph. But when it comes to instantiate them into circuits, there is the possibility that we need to change the underlying template for more complex circuits to fit in. For example, corner/junctions in space need special care about the plaquettes arrangement for the circuit to be distance-preserving.

How do you plan to construct more complex templates like the corners/junctions? I assume that for each "composed template" now we need to implement a rather large instantiate method to specify all the plaquette indices.

The problem is the same with the current architecture. The main complexity is due to the center 2k x 2k template that has a "less regular" pattern in the case of corner or junctions. Implementing a corner / junction with the ComposedTemplate API still needs to specify that 2k x 2k template by hand, which is the main part of complexity.
In other words, because the border implementation is not really complex (and can still be factored out in a separate function), I do not see any added complexity by this approach, the same things have to be done anyway in both approaches.

I remembered that the complexity of PiecewiseLinearFunction mainly comes from the usage in ComposedTemplate. If it does not exist anymore, is it possible to simplify the scale.py/interval.py as well?

This is very much true, I will try to remove that now.

In order to demonstrate that it can be done, I implemented the 4-way junction as a new template. Note that 2/3-way junctions are not implemented yet, and there are enough questions we need to collectively answer on them to justify another PR just for that.
I think this PR is ready, but before merging I would like @KabirDubey to react here as I know he was working on #71, which is greatly impacted by this PR.

Now that I included more tests I think this PR is ready to be reviewed. As soon as it is accepted, I will merge this one and just after merge #318.
Then, I will need to fix any potential merge conflict with #305 and merge #305.
The next steps will be:

Finish #319 and merge it,
Start and finish a translation method from AbstractObservable instances to actual Observable, or even better to a new ScalingObservable.

This is also how I see it: a showcase that will need to be checked and improved. Thanks for the quick approval !


Gulp, TypeScript, and Monaco

This PR does three things:

Gets rid of Jake in favor of Gulp
Upgrades our TypeScript version to 3.7.5
Upgrades our Monaco version to 0.19.3

It also removes our dependency on pxt-monaco-typescript and replaces it with the stock TS language service. The plan is to remove that service and switch entirely over to one implemented by us in the near future.
I'm marking this as a draft PR until I've opened PRs on each of the targets to upgrade to the new TS version.

Very excited about gulp. It's a much better foundation.
I suspect we could get even more of an improvement if we could pipe output from tasks directly into each other instead of outputing files to built/ and then consuming from built/, but I would need to look into it more myself.


Typo fix on ln command

Minor typo in howto
ln -s "$py_path" "$py2_path"
should be
ln -s "$py2_path" "$py_path"

Thanks, nice catch


Print std_output if it's not empty

Actually on a second thought, maybe we should print stdout whenever it's not empty in any view. PR updated accordingly.


Development for Express

Might be an engine implementation designed for express or just a set of producers that handle requests.
TBD

partikip

With the new Engine Modules the implementation is trivial and doesn't require further extending the Engine.
See the pattern detailed here:
https://code11.github.io/engine/docs/patterns/request-response
What we might end up providing is a sandbox with a working example.


Fix three issue related to editing observations and projects

Doesn't throw up the warning stuff about constraint violations if the observation is already part of that project.
Doesn't automatically try to add the observation to projects if you're editing the observation vs. creating a new one.
Don't allow admin editing of objects of users who do not trust a project.  Note that editing of objects is only allowed for closed projects.

coverage: 94.441%. remained the same
when pulling 1a18b128dadb6b5042add98faa9d5e1ee9daf330 on njw-edit-obs-project-fixes
into 852d452fd5e9ae759b58319c01a6375b25b2b48e on main.

Not going to sweat that edge case.  Not this has to merge after njw-project-members

Suggestions (because I didn't understand all implications of Trust).

When a User clicks Trust the flash should indicate that Admins (or all Members, as the case may be) can edit the User's Observations which belong to the Project. (The flash now just says that admins can view hidden coords: "You now trust admins of this project with hidden GPS data for associated observations.")
Similar for clicking Revoke Trust
And in addition to the ephemeral flash, explain the effect of the current trust status on the ShowProject.
Something like:
"You trust Project Admins to edit, and see hidden coordinates of, your Observations that are part of this Project."
"You trust all Project Members can edit, and see hidden coordinates of, your Observations that are part of this Project."
"You do not trust Project Admins cannot edit, or see hidden coordinates of, your Observations that are part of this Project."
"You do not trust Project Admins or Members to edit, or see hidden coordinates of, your Observations that are part of this Project."

So as things are currently written if a project is closed objects associated with a project are editable by other members of the project if the owner of that object trusts the project.  I agree that adding comments to that effect makes sense.  However, in thinking more about it (and testing), I found another thing that needs to be fixed.  The main goal of not allowing editing of objects in open projects is to avoid the case where someone just joins a project and can suddenly edit stuff with no review by a project admin or the owner of the object.  However, I've realized this means if an admin wants to edit an observation of someone who trusts the project, they can do so if they switch the project to closed, make the edit, and then switch the project back.  That seems kind of odd.  I think the editing ability needs more explicit control.  I need to think this through more.

Would this work:

When a User joins/is added to a Proj, set trust regardless of Proj type. If it's an Open Proj, set trust to false.
Now if the Admin changes the Proj type, s/he cannot edit other Users' objects that belong to the Proj because the User doesn't trust the Project.

Maybe I don't grasp something about the desired behavior.
If the intent is to allow Open Proj Admins (but not other Proj members) to edit other users' Objects,
then would this work:

Multiple levels of trust: Untrusted, Admin (only), Trusted.
When User joins an Open Project, set Trust to Admin.

Once I get the coverage back up on this, I think we can merge it.  It doesn't solve the problem I've described, but it's better to have these fixing in than not.  After that I'll work on cleaning up the concept of trust for projects.

@JoeCohen I think this is finally ready to merge.


[DOC] Factorization Machines core module

what does this contain?

model equation
little theoretical support
calculations as to how order of computation changed.
gradients

what's later

once @dusenberrymw reviews & merges classification & regression, documentation will be added for them also.

Thanks a lot.

@dusenberrymw - please provide any comments as to what else needs to be added.
@krishnakalyan3 - can I get a preliminary review, just copy the changeset & paste it at upmath.me  to get real time rendering.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2275/

@j143 thanks for working on this. Initial version looks good.

@dusenberrymw , can this doc also be added as a preliminary version, for now.
We can write a full doc when scale the fm scripts, after few months. Thanks.

@j143 Can you push a copy of the built docs to a gh-pages branch in your systemml fork so that it can be viewed in web form for review?  It only takes a minute: http://apache.github.io/systemml/contributing-to-systemml#documentation.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2348/

Hi @j143
Looks like a decent intro to factorization machines. Thanks for adding.
Possible improvements could be a section on how to invoke the accompanying factorization machine script (so that some1 who is not interested in the theory could get started quickly). Please let me know in case that part of the documentation is somewhere else.

Well, if the goal is simply to add an fm layer, then I guess its fine (I wasn't suggesting adding a standalone dml implementation).
I looked for documentation on nn layers and seems like we have very sparse docs. For instance, I couldn't find any documentation explaining at an intuitive, abstract level what conv2d does. That's what I was suggesting. For the standalone dml algos we have a pdf file with such info.
So feel free to ignore my previous comment. Seems like the comments in the fm layer's implementation are supposed to serve as sufficient description.

sounds good


Lua 5.4-compatible?

It's not listed in README.md, but it seems to work!

I had planned to release a brand new stdlib, with dependencies on new broken out submodules several years ago, which would have obsoleted the current all-in-one release... but then I took a(n otherwise excellent) job which disallowed any open source work or contributions.
However, I recently switched to a more open source friendly employer, so this is all back on my TODO list!
If you have the bandwidth, I'd be interested in whether you can do the things you want using the git rockspecs (see README.md for the various other projects in this github account too) and report any issues to help me apply the final polish necessary to make a solid major release soon.
If not, I have continued to use the last luarocks stdlib release for specl, which in turn runs all the testing for all the rocks I maintain on all supported versions of Lua/luajit, and that works just fine with Lua 5.4:
% lua -v
Lua 5.4.4  Copyright (C) 1994-2022 Lua.org, PUC-Rio

% luarocks install stdlib
Installing https://luarocks.org/stdlib-41.2.2-1.src.rock

stdlib 41.2.2-1 depends on lua >= 5.1, < 5.5 (5.4-1 provided by VM)
stdlib 41.2.2-1 is now installed in /usr/local (license: MIT/X11)

What do you see?


json::to_bson does not handle unsigned 64 bit integers

Description
Trying to convert a nlohmann::json object which contains an unsigned int of 64 bit length breaks BSON conversion.
I do consider this to be a bug, as BSON should be capable of handling those values [1] , but the corresponding header-bits are not set in write_bson_unsigned. Only entry headers 0x10 (for int32) and 0x12 (for int64) are written, but 11 (for uint_64) is not.
Reproduction steps
This bug is triggered whenever one uses the json::to_bson function on any object that contains an uint64_t, that really uses all 64 bits.
Expected vs. actual results
I expect BSON to generated, but I get an error message (see below)
Minimal code example
#include <map>
#include <string>
#include <nlohmann/json.hpp>

int main(){
  std::map<size_t,std::string> m={{14631357879723465589u /*really just any large enough uint64_number*/,"To_Big"}};
  nlohmann::json j;
  j["map"]=m;
  auto bson=nlohmann::json::to_bson(j); //can not be converted
}

Error messages
terminate called after throwing an instance of 'nlohmann::detail::out_of_range'
  what():  [json.exception.out_of_range.407] integer number<PHONE_NUMBER>9723465589 cannot be represented by BSON as it does not fit int64

Compiler and operating system
Clang-13 on Mint 21 64bit, Clang Trunk (via godbolt)
Library version
3.11.2 (develop branch via godbolt) , 3.10.5
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

It seems I was confused with the alias "timestamp" when I implemented BSON. You are right - it makes no sense to not support uint64 numbers.
An extension should be straightforward, but to do it properly with tests may take some time.

@nlohmann  Can I raise a PR for this?

Sure!

This is resolved by https://github.com/nlohmann/json/pull/4380 which has been open since May - @nlohmann, any chance of a review?  Thanks!

Sorry for the delay. I added some comments.


Test on iOS devices

https://i.imgur.com/f4VEoHs.jpg

iOS 10.0.2
Chrome 67.0.3396.87
Does not work because missing polyfill for Object.values.
When polyfill added - generally functionality appears to work except the following

Day
Week
Schedule
4 days

Display a completely empty view (header still in place).
Updated Chrome to 70.0.3538.75 and the issue remains.

DIsplay issues appear to be CSS bugs that actually affects OS X safari as well.
Below screenshot of iOS iPhone simulator

As you can see, all content actually exist in the generated HTML but for a reason or another is not being displayed.
If for example the parent style "ds-week-view" is removed, the content is shown

Breaking properties in CSS
ds-week-view:
position: absolute;
ds
flex: 1 1 60%;
Once commented out, appears to work OK
👍


NIFI-5200: Fixed bug that caused the wrong InputStream to be closed b…

…y StandardProcessSession if calling Session.read() from the callback of another Session.read(); also changed default of the 'allowSessionStreamManagement' flag from 'false' to 'true' as it will provide performance benefits in some cases and NIFI-516 outlined that it should be 'true' initially but ended up being false when the PR was merged.
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Reviewing...

@mattyb149 +1 LGTM. I can merge.

ok have at it, didn't know you were reviewing :) Thanks!


Spin out Media section to its own page

Fixes #1211

It doesn't look great but we'll rework its layout later.


rename contractset.log to contractset.wal

This PR renames the contractset to have the 'wal' extension.

You should only perform this rename if contractset.wal doesn't already exist. If it already exists, you shouldn't even check if contractset.log exists.


How to use modsecparser and mlogc?

Recently I found out that the modsecparser can writes audit logs into a Postgres database,the mlogc can send audit logs to a central repository,but I can't find how to install them,I just found some configuration messages on https://github.com/Ixolit/modsecparser or https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual-(v2.x)

Hi @prince-java,
The modsecparser is an independent project maintained by @terrorobe.
The mlogc is distributed altogether with ModSecurity version 2.
Whait is the verision of your ModSecurity?

Hi @prince-java,
The modsecparser is an independent project maintained by @terrorobe.
The mlogc is distributed altogether with ModSecurity version 2.
Whait is the verision of your ModSecurity?

The verision of my ModSecurity is 2.9.3.
I find out there is a folder named "mlogc" in the tar file,but I don't know how to install、use or configuration.

@zimmerle
Thank you for response!
The verision of my ModSecurity is 2.9.3.
I find out there is a folder named "mlogc" in the tar file,but I don't know how to install、use or configuration.

Mlogc will be built upon the compilation of ModSec. Proceed with the regular compilation and the binary will be generated in that folder.

My ModSecurity has been installed.After that,there is a file named 'mlogc' in /usr/local/modsecurity/bin.
I just don't how to use it, add mlogc-default.conf in /usr/local/modsecurity-2.9.3/mlogc into httpd.conf?

There are some information here -
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual-(v2.x)#secauditlog


Create example on how to use multicore SOLR available in WunderMachina

Related to this https://github.com/wunderkraut/WunderMachina/issues/96 we should add an example to WunderTools on how a project can use multicore SOLR. This is specifically needed because we should use the SOLR config from search_api_solr module that the project is using so it needs to be done on the project level and can't be predefined. See https://ohthehugemanatee.org/blog/2013/04/18/how-to-set-up-apache-solr-multicore-for-drupal/

@aleksijohansson Would it be enough to have an example multicore role people can copy/modify? I created something for my project so we could use that.

@artursv would be nice to have support for multicore directly in the playbook. But for the starters I think the multicore example as local role would be enough to get started.

Closing as this issue hasn't been touched in a while and we are looking at using Elasticsearch even for Drupal 7 projects. Feel free to reopen if needed.


Enable FullNetworkPoliciesInRuntimeCluster feature gate locally

How to categorize this PR?

/area dev-productivity
/kind enhancement
What this PR does / why we need it:
This PR enables the FullNetworkPoliciesInRuntimeCluster feature gate (introduced with https://github.com/gardener/gardener/pull/7589) for the local setup with a version of gardener-extension-networking-calico including https://github.com/gardener/gardener-extension-networking-calico/pull/247.
/cc @ScheererJ
Release note:

NONE

After https://github.com/gardener/gardener-extension-networking-calico/pull/229 the networking-calico extension enforces a non-nil .spec.networking.nodes. We have to wait for a new release.
/hold

/unhold

/test pull-gardener-e2e-kind


Add subtitle blocks

The user is able to add new subtitle blocks at any position. When adding a new subtitle block an empty line and a TC-IN and TC-OUT is created automatically.
While clicking on the litte "plus" (Mouseover: "new subtitle block") before/after a subtitle block a new subtitle block appears. The following subtitle blocks move dynamically.
The user clicks on the "plus":

A new subtitle block with an empty line and automatically created TC-IN and -OUT appears:

@mfinger-irt One question is about the gap between two subtitles. I propose a gap that corresponds to 1 frame (so 40ms if you have 25fps).

@mfinger-irt A good content structure for adding a subtitle would be as follows:

  placeholder

The following would also be good but we can not use it for the moment because there is no way to delete a subtitle line
`

  placeholder

  placeholder

@mfinger-irt A tricky question is what style attributes should be added to the new
 and  element. At the moment only the style attributes that are present at parsing/inserting time can be edited. So, if for example, if the color attribute is not present at a  when a new subtitle block is added, text color can not be changed on the text-level.
It may be good to provide the possibility to add a TTML template where a user can specify how the content structure and application of style attribute should look like. For the moment this could be a static template that is in the repository but in the end, there may be the option to provide the template in the custom folder.
The alternative would be to specify all possible style attributes for
 and  elements. But in this case, inherited values (e.g. from
 or ) would be overwritten.

The template approach could also be a good solution to get the right choice to add a new region for the subtitle block.

@LauraEhlis In the UI I think that the "plus" sign should be highlighted visually so that you can immediately see that you have the option to add subtitles. Possibly it can be set in bold (as the "x" for the deletion of subtitles) and the font could be a bit bigger.


[now dev] Fix builder sorting when use is undefined

Fix builder sorting when use is undefined

Codecov Report

Merging #2566 into canary will decrease coverage by 0.6%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           canary    #2566      +/-   ##
==========================================
- Coverage    14.3%   13.69%   -0.61%
==========================================
  Files         257      257
  Lines        9530     9457      -73
  Branches     1114     1084      -30
==========================================
- Hits         1363     1295      -68
+ Misses       8042     8038       -4
+ Partials      125      124       -1

Impacted Files
Coverage Δ

src/util/dev/server.ts
64.74% <100%> (ø)
:arrow_up:

src/util/dev/builder-cache.ts
78.48% <0%> (-4.37%)
: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 68c1f71...3bba36a. Read the comment docs.


[ROS] Update GPG key installation step to the latest

Some of the ROS installation instructions have been updated.
This PR updates the GPG key installation step to the latest.
ref: https://github.com/ros2/ros2_documentation/pull/1189

Thanks @Tiryoh - there were a couple minor changes that needed made in this PR, see the fixes in https://github.com/dusty-nv/jetson-containers/commit/19a3805120a997e962d551820805f2cce1740d95
Rebuilding the ROS2 containers now and will update the images on DockerHub.

Oops, Thanks for fixing, @dusty-nv!


Error: Impossible to use asynchronous mapping using automapper.map(); use automapper.mapAsync() instead

Hi,
I'm getting the following error even though my mapping function is not async. I'm using the mapFrom method.
Error: Impossible to use asynchronous mapping using automapper.map(); use automapper.mapAsync() instead
I've tracked the issue down to automapper.js in the dist folder.
See below. The async property is getting set to true if there are 2 parameterNames. I'm not sure what parameterNames is, it looks like a serialization of the mapping function. There is nothing async in my mapping function.

Can you explain to me what this is trying to achieve or a way i can avoid this issue?
thanks
Mat

I'm having trouble uploading a screenshot. It's located on line 129 in the method AutoMapperHelper.getMappingMetadataFromTransformationFunction in file dist/automapper.js

I was using an arrow function and this throws the getFunctionParameters out of wack. I will change to using  proper functions

Hi Matt,
Could you please post your mapping code resulting in this exception? It would help investigating and/or clarifying the problem you are facing ;) ...
Cheers, Bert

Hi Bert,
Using arrow functions it doesn't seem to be determining the number of parameters correctly
see below
`const automapper = require('automapper-ts')
automapper.createMap('input', 'output')
.forMember('outputProp', opts => mapFromNullable(opts, 'input'))
.ignoreAllNonExisting()
console.log(automapper.map('input', 'output', { inputProp: 'Test Prop' }))`
To be honest it's no big deal. I've converted it to use explicit functions like below and it's fine
function (opts) { mapFromNullable(opts, 'input') }
thanks
Mat

I found the root of this issue. Lucky me, I was about to register the same issue with forSourceMember.
In function AutoMapperHelper.getFunctionParameters, the author intends to count the number of parameters of forMember's callback:
opts => mapFromNullable(opts, 'input')
, which in this case should be 01 (opts). However the function makes mistake and counts number of params in mapFromNullable(opts, 'input') which yields 02. I can tell that the regular expression in AutoMapperHelper.getFunctionParameters mistakes mapFromNullable(opts, 'input') (with 2 input params) for  opts => mapFromNullable(opts, 'input') (with 1 input param).
In the time being, this is a working workaround:
function transformation(opts) {
  return mapFromNullable(opts, 'input');
}

automapper.createMap('input', 'output')
  .forMember('outputProp', opts => transformation(opts))
  // .forMember('outputProp', transformation) => Don't do this. Believe me 😃
  .ignoreAllNonExisting();

I experienced the same issue. Moving from arrow function expressions to function expressions fixed this for me as well.

Hi @LWB-MatJ , @hirikarate and @gdovicak ,
I am unable to reproduce your issue using TypeScript. The TypeScript compiler compiles arrow functions to plain JavaScript functions at all times. At least, that is what happens with me. A couple of questions:

Are you using Typescript to write front-end code?
Which compiler target are you using? Is it possible that your resulting code still has arrow functions in it?

Number two would explain why changing arrow functions to plain JavaScript functions would change behavior at your end...
I hope to hear from you soon.
Cheers, Bert

Actually I am using automapper-ts for NodeJS backend code and it works pretty well except this issue (kind of minor one). Here is my tsconfig:
{
	"compilerOptions": {
		"target": "es2016",
		"module": "commonjs",
		"moduleResolution": "node",
		"sourceMap": false,
		"emitDecoratorMetadata": true,
		"experimentalDecorators": true,
		"removeComments": false,
		"noImplicitAny": false,
		"allowSyntheticDefaultImports": true
	}
}

I am doing the same as @hirikarate and am not using Typescript. #2 would explain the issue I am seeing.

Hi @hirikarate and @gdovicak ,
Unfortunately, AutoMapperTS does not fully support ES2016 (yet). The workaround in this scenario is to use a classic EcmaScript 5 function (e.g. function(opts) { ... }) instead for those mapping scenarios.
Please feel free to fix the problem and file a PR, should one of you have a little bit more spare time than I currently have.
Cheers, Bert

This is absolutely an issue. Particularly if you are using prettier/eslint/beautify, which will remove function(){} definitions and replace them with arrow functions.

This is absolutely still an issue. Particularly if you are using prettier/eslint/beautify, which will remove function(){} definitions and replace them with arrow functions.

It is something terrible… I spent 2-3 hours to understand why Angular 7 to 9 migrated project doesn't work! Issue is still here and need to be fixed!

Hi,
I used @gonza-lito fork on my project, and also I created PR there to fix incompatibility with arrow-functions. Maybe you can check that PR to fix the issue on your side.
https://github.com/gonza-lito/AutoMapper/pull/1

The main change has been done here: https://github.com/gonza-lito/AutoMapper/pull/1/files#diff-e48398614b3d36beb47c5d3ae5f42b7e

Thank you, works fine... in readme you wrote that "Remove global scope variable", please, provide at least a minimal description for suggested usage...


can compitible libtool as a archive maker?

for some reason, I must set libtool as archiver, but openssl call archiver with specified options(qc) that libtool not support, so archive stage will fail, request openssl compitible with libtool, thanks very much
ar qc libssl.a ssl/libssl-lib-bio_ssl.o... --> libtool qc libssl.a ssl/libssl-lib-bio_ssl.o...

The definitions for AR and ARFLAGS are found in Configurations/00-base-templates.conf (see the BASE_unix template defintion).  If you have a need to override those, you can set AR=libtool ARFLAGS="" on the configure command line

libtool will output libcrypto.a twice, and sencond time will override first one, lead many undefined symbols when link to libcrypto, libcrypto.3.dylib has all symbols. why libtool output libcrypto.a twice?(caused too many object files??), can compitibable this situations?
my ARFLAGS is -static -o

maybe it's libtool's problem


Single argument function types require parentheses

This is a quick fix for errors in Xcode 8 beta 3 to satisfy parens requirements

Thanks!


Add targeLabels to Prometheus ServiceMonitor CRDs

We would like to use the targetLabel functionality in the servicemonitor CRDs. This would assist with passing labels from our Services onto the metrics being scraped per team/env.
Plan to submit a PR for this request. Thanks!

This feature was added with #439. Its safe to close.


[HOP-2943] created unit test for only concatenating some fields in the stream

Following this checklist to help us incorporate your contribution quickly and easily:

[X ] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[X ] Each commit in the pull request should have a meaningful subject line and body.
[X ] Format the pull request title like [HOP-XXX] - Fixes bug in SessionManager,
where you replace HOP-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the commit message.
[X ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[not code change but a new unit test that force current code to error ] Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[X ] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.

Trivial changes like typos do not require a JIRA issue (javadoc, comments...).
In this case, just format the pull request title like (DOC) - Add javadoc in SessionManager.
If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[X ] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[X ] In any other case, please file an Apache Individual Contributor License Agreement.

Hi @jballment Thanks for the contribution!
Can you add the unit test to the existing main-0004-concat-fields.hwf?
there is no need to create a workflow per test.
We have been using the structure 1 workflow per transform and add multiple test there.
Example
Wokflow: main-0006-groupby.hwf
Unit test pipelines : 0006-groupby-all-rows.hpl, 0006-groupby-basics.hpl, 0006-groupby-no-group.hpl

Hi,
I've removed the main-004-concat-some-fields.hwf and added the unit test
into main-004-concat-fields.hwf as requested.
I then squashed the original change with these changes and pushed it to my
forked repo.  I then went to resubmit the pull request but the existing
pull request seems to now have the updated version (I'm not sure if that is
what you were expecting to happen).
Please let me know whether I've submitted this correctly; we don't do "pull
requests" and "squashing" changes in our environments.
Regards,
John Ballment
Mobile: 0418 740 871
Email: @.***
Before printing, please consider the environment.
On Wed, 9 Jun 2021 at 18:17, Hans Van Akelyen @.***>
wrote:

Hi @jballment https://github.com/jballment Thanks for the contribution!
Can you add the unit test to the existing main-0004-concat-fields.hwf?
there is no need to create a workflow per test.
We have been using the structure 1 workflow per transform and add multiple
test there.
Example
Wokflow: main-0006-groupby.hwf
Unit test pipelines : 0006-groupby-all-rows.hpl, 0006-groupby-basics.hpl,
0006-groupby-no-group.hpl
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-hop/pull/858#issuecomment-857489854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA27Y56DK76CEUPBJC7SUZ3TR4PS7ANCNFSM46LP4SMQ
.

Added code fix for issue with concat fields.  This does change it's behavior, all fields are returned + new concatenated field.  This is similar to how PDI outputs looks for the equivalent step.


#1042: Onboarding documentation

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-14829

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-16086


opacity option support

I can look into it

I noticed in the [tokyonight.nvim configuration]https://github.com/folke/tokyonight.nvim?tab=readme-ov-file#-configuration] they have a setting for transparent that does not set the background -> is this the feature your are requesting?

Yes!!

A bit busy this week, but should have some time to make a feature to configure this as a default, It will take a medium size refactor of the colorscheme, but I was planning to do it anyway, in order to have some customization features like transpaent or string_color and quite a few more
for now if you want to do quick fix you can use thic code in config,
        vim.cmd.colorscheme("lackluster-hack")
        vim.api.nvim_set_hl(0, 'normal', {
            fg = '#cccccc',
            bg = 'none'
        })
       -- this will not work if you switch colorschems but it will work if you are using lackluster as your default colorscheme :)

🎉 done :)
I added docs in the main readme on how to setup transparency, or change menu,poup,telescope bg colors
You can also now tweek some of the syntax colors too, I found with different transparent backgrounds comment needed to be altered to be readable
😆 from testing i now have a siik  dragon wallpaper

lemme know if you have any trouble :)

@slugbyte
Thanks! that transparent mode is brilliant!!
I appreciate it!!!
❤


Refactors stored accounts info

Problem
After accounts are stored, we return information about the offsets for each account, and the size. We use the size of each account written to update the AccountStorageEntry with the total size of the storage itself. This size is used by shrink, among others.
With tiered storage, the size of a single written account is not as straight forward. For one example, the owners field of each account is deduplicated in the storage file itself.
Since we only need the total size written, we can use that for both AppendVec and TieredStorage. The code just needs some refactoring.
Summary of Changes
Refactors stored accounts info.

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

Project coverage is 81.8%. Comparing base (c7a5199) to head (23a02ff).
Report is 8 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #887   +/-   ##
=======================================
  Coverage    81.8%    81.8%
=======================================
  Files         853      853
  Lines      231791   231859   +68
=======================================
+ Hits       189784   189841   +57
- Misses      42007    42018   +11


Setup 3D Visualization menu

How about renaming the section of GPA module from "Setup 3D Visualization" to "Setup Dynamic Visualization", or "Setup Interactive Visualization" Lollipop plots are also 3D.

Addressed in commit dde2c0baaff8b7a08d0568a2f863e4e520037aaa


Teleoperation not working

ISSUE TEMPLATE ver. 0.4.0

Which TurtleBot3 platform do you use?

[x] Burger
[ ] Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[ ] ROS 1 Melodic Morenia
[ ] ROS 1 Noetic Ninjemys
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[x] ROS 2 Foxy Fitzroy
[ ] etc (Please specify your ROS Version here)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Intel Joule 570x
[ ] Raspberry Pi 3B+
[x] Raspberry Pi 4
[ ] etc (Please specify your SBC here)

Which OS you installed on SBC?

[ ] Raspbian distributed by ROBOTIS
[ ] Ubuntu MATE (16.04/18.04/20.04)
[ ] Ubuntu preinstalled server (18.04/20.04)
[x] etc (Ubuntu 20.04.4 LTS)

Which OS you installed on Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[x] Ubuntu 20.04 LTS (Focal Fossa)
[ ] Windows 10
[ ] MAC OS X (Specify version)
[ ] etc (Please specify your OS here)

Specify the software and firmware version(Can be found from Bringup messages)
I have attached image file containing output of Bringup

Software version: [x.x.x]
Firmware version: [x.x.x]

Specify the commands or instructions to reproduce the issue.

ros2 run turtlebot3_teleop teleop_keyboard

Copy and Paste the error messages on terminal.

CP2102 USB to UART Bridge Controller
OpenCR Virtual ComPort in FS Mode

Please describe the issue in detail.

I am new to the turtlebot, robotis, ros. In fact I just bought the turtlebot3 and started setting it up. I did everything according to the steps here https://emanual.robotis.com/docs/en/platform/turtlebot3/overview/#overview but I just don't understand why I am stuck at bringup. The output of bringup looks fine except these two,
CP2102 USB to UART Bridge Controller
OpenCR Virtual ComPort in FS Mode

I can run the command ros2 run turtlebot3_teleop teleop_keyboard and the node is successfully launched, but the bot just won't move. I tried to test Opencr with ArduinoIDE and it works totally fine there but not on the Teleoperarion.
Asking out for help to seniors.

@ROBOTIS-Will

problem solved.
Reference #301 .


Postfix templates

There is useful thing in IDEA/another IDE's
You can write abc.if[TAB], and this expands into if(abc) {}
There are extensions, which implements this feature, i.e https://github.com/ipatalas/vscode-postfix-ts, but it would be awesome to see this as built-in

The extension is pretty good and extensions are easy to install. Copying extension functionality into the product will give people reason to call us bloaty

@jrieken, this extension supports only one language, if something like this gets official support, it can be implemented in many language plugins (since postfix templates require language support), and anyone can easily use them as a snippets without installing plugin for every language and having templates in plugin configurations


fixed mobile view of website

Related Issue
#1841
Fixes:  #1841
Description
fixed mobile view of website

Type of PR

[X] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
before :
after :
https://github.com/anuragverma108/SwapReads/assets/142989448/1535aa3c-b03a-4f4e-894f-5170358db0db
Checklist:

[X] I have made this change from my own.
[] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@anuragverma108 @huamanraj please merge the PR with gssoc and level 2 label
thanku

please resolve the conflicting files and raise new PR

@RitiChandak please check PR #1912


Conformance question

Hello, I was just wondering if Yet XAPI LRS 1.0.74 listed here https://adopters.adlnet.gov/products/e695e5c9-6542-4b68-9ddf-3a3acefc8045/ is the same as this SQL LRS?
If it is not, I was also wondering if SQL LRS has gone through the conformance testing here: https://lrstest.adlnet.gov/ and if so, if there's a certificate that is available?
Thank you,
Ilya

Hi Ilya!
The officially listed conformant LRS you mentioned is an older SaaS product, and does not apply to SQL LRS. We actually need to update that, thanks.
That said, we do run the same ADL conformance test suite (for xAPI version 1.0.3) in our CI pipeline and SQL LRS is very much conformant!
Cheers,
Milt

Thank you for the quick response and the clarification!


new module has been added

Description

Motivation and Context

Breaking Changes

How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects

tested


How to update favicon, meta tags like descripiotn, keywords, openGraph(og) etc.

How to update favicon, meta tags like descripiotn, keywords, openGraph(og) etc.
I tried to implement Head component using the link, meta tags are not updating dynamically.
Iam using Laravel 11 and Vue3.
My Sample code here:
<Head>
    <title>{{settings.site_title}}</title>
    <link rel="icon" :href="settings.site_favicon" sizes="32x32"/>
    <meta head-key="description" name="description" :content="settings.seo_meta_description">
    <meta head-key="keywords" name="keywords" :content="settings.seo_meta_keywords">

    <meta property="og:image" :content="settings.site_favicon">
    <meta property="og:image:type" content="image/png">
    <meta property="og:image:width" content="436">
    <meta property="og:image:height" content="228">
    <meta property="og:description" :content="settings.seo_meta_description">
</Head>

and I have following tages in app.vue file:
<title inertia>{{SITE_TITLE}}</title>
<meta name="description" content="my description">
<meta name="keywords" content="my keywords">
<meta property="og:image" content="{{asset('assets/logo.jpg')}}">
<meta property="og:image:type" content="image/png">
<meta property="og:image:width" content="436">
<meta property="og:image:height" content="228">
<meta property="og:description" content="my description">

Note: Title is updating dynamically, but meta tags are not updating dynamically.

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord or InertiaJS Discord for Inertia questions
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Installer on RHEL does tarball install instead of rpm install

Is this a request for help?:
No.
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST
On RHEL8, the playbook installs the pgdg and nginx-stable repos then installs postgresql-13 and nginx from those repos.  This is a awesome.  However, it then goes and does an Artifactory tarball install.  Why doesn't it install the Artifactory repo then install Artifactory from that repo just like it does for the others?
Which installer:
Artifactory, but it could apply to others.
Which product and version (eg: ansible & collection version - 1.1.2):
Ansible 2.12.6
Artifactory 7.38.10
JFrog Platform 10.6.2
Which operating system and version(eg: ubuntu & version - 20.4):
RHEL 8.6
What happened:
Playbook installed Artifactory from tar file.
What you expected to happen:
Playbook to install Artifactory rpm
How to reproduce it (as minimally and precisely as possible):
N/A

Hi @cdalessio0131
we use tar file installation to support multiple flavours not specific  to RPM based installation

I figured that was the reason. But since you needed to be RedHat specific for postgresql and nginx, why not go all the way for a RedHat install?

As admin, I prefer to have the yum/dnf logging as well as rpm verify for my installed package.

For PostgreSQL and Nginx third-party libraries we went with upstream vendor packages.
For Jfrog products, we wanted to have uniform installation method across different flavors.

Disappointed, but I understand the reason behind the decision.  Thank you.


merge 2 sorted arrays

Important notes:
This pr contain:

basic implementation
LeetCode problem 88 for usage of it


feat: authenticate using token based auth with remote Bee node

You can pass arbitrary headers using the -H or --header flag


Icon Request - YouTube Revanced

app.revanced.android.youtube
Left is yt Vanced and right Is yt Revanced

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.

The left one kind of looks cool to be honest i mean it's better than the one we have now


Update README.md

Hi, I commented the readme in a fork just for the sake of simplicity.
Just wanted to chime in to say that the autodetect function in nodered only checks if the connection is successful, not if the commands work with the device. Specifically kd8003 series of modular intercoms.
If you are interested, there is some activity going on to make a module in homeassistant, but more intruiguing is the fact that the intercom kd8003-ime1 apparently needs more parameters passed to work/accept commands (eg room number, floor...)
https://community.home-assistant.io/t/ds-kd8003-ds-kv8113-ds-kv8213-ds-kv6113-ds-kv8413-and-integration-hikvision-hikconnect-video-intercom-doorbell/238535/1172

Hi, thanks for the hint.
Yes. Doorbell is a total different story from cameras.
Unfortunately, i've no more time to spend on that node and i haven't any doorbell to test and develop for.
I'll totally remove that part of the page, because there are too much firmwares and models out there and this can lead in misunderstandings.


TypeError on addCommentGrades in case of empty request

Questions
Answers

Description?
URL like /module/productcomments/PostComment?id_product=2 is exposed on html form.  Even if the form has a method POST, some bot crawl this URL and error 500 append :Uncaught exception 'TypeError' with message 'Argument 2 passed to ProductCommentsPostCommentModuleFrontController::addCommentGrades() must be of the type array, bool given, called in modules/productcomments/controllers/front/PostComment.php on line 100' in modules/productcomments/controllers/front/PostComment.php:135 in ProductCommentsPostCommentModuleFrontController::addCommentGrades called at modules/productcomments/controllers/front/PostComment.php (100) in ProductCommentsPostCommentModuleFrontController::display called at classes/controller/Controller.php (331) in ControllerCore::run called at /var/www/prod/classes/Dispatcher.php (518) in DispatcherCore::dispatch called at index.php (28)I realized that validation are not done properly and after the persistence of data and criterions are not done.

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/Prestashop#27534.

How to test?
Just call this URL /module/productcomments/PostComment?id_product=2The response code should be 200

Thanks @clotairer & @khouloudbelguith


Support for Nvidia TensorRT-LLM Inference

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #9841
Type of Change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new notebook (that tests end-to-end)
[ ] I stared at the code and made sure it makes sense

Suggested Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

@hatianzhang omg I forgot that list exists lol. Good point

@hatianzhang specific LLMs are not really mentioned in /llama_index/docs/module_guides/models/llms/usage_custom.md ?
I linked to the notebook in /llama_index/docs/module_guides/models/llms/modules.md though


mjpeg quantization not support other than 255

I got Q 63 not supported when ingest MJPEG rtsp stream. How to support other quantization?

Hello, quantization tables other than 255 are now supported thanks to #583


Electron density betwenn a couple of molecular orbitals

Hello everybody,
There is a way to calculate de electron density (rho)  between two MOs that one choose?
being
\rho(r) = \psi_a^(r)\psi_a(r) = \sum_i \sum_j C_{i,a}C_{j,a} \phi^_i(r)\phi_j(r)
I know how to get the C_{i,a}, but not the corresponding \phi.

One option would be to use tools.cubegen.  The \rho(r) that you want is
just the elementwise product of the cube formats of the individual MOs.
On Thu, Apr 15, 2021 at 6:20 PM DanielBajac @.***>
wrote:

Hello everybody,
There is a way to calculate de electron density (rho) between two MO that
one choose?
being
\rho(r) = \psi_a^(r)\psi_a(r) = \sum_i \sum_j C_{i,a}C_{j,a} \phi^
_i(r)\phi_j(r)
I know how to get the C_{i,a}, but not the corresponding \phi.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/909, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC7QBEXZ54D6LVBEMPLTI5REFANCNFSM43ALEOHQ
.

Thanks for your answer.
I've never use the cubegen file, but, in the example:
from pyscf import gto, scf
from pyscf.tools import cubegen
mol = gto.M(atom='''
O 0.0000000, 0.000000, 0.00000000
H 0.761561 , 0.478993, 0.00000000
H -0.761561, 0.478993, 0.00000000''', basis='6-31g*')
mf = scf.RHF(mol).run()
cubegen.density(mol, 'h2o_den.cube', mf.make_rdm1())
If I use a particulas MO, por example: mf.mo_coeff[:,[2]], I have problems with the rdm1.
IndexError: boolean index did not match indexed array along dimension 1; dimension is 1 but corresponding boolean dimension is 18.
How can I choose the MO I want?

I’m suggesting to use orbital() not density(). You could create two cube
files for your two orbitals and then multiply them together by yourself
with another script. Or maybe you could modify/write your own functions
based on the code in cubegen to do this all at once.
On Thu, Apr 15, 2021 at 8:12 PM DanielBajac @.***>
wrote:

Thanks for your answer.
I've never use the cubegen file, but, in the example:
from pyscf import gto, scf
from pyscf.tools import cubegen
mol = gto.M(atom='''
O 0.0000000, 0.000000, 0.00000000
H 0.761561 , 0.478993, 0.00000000
H -0.761561, 0.478993, 0.00000000''', basis='6-31g*')
mf = scf.RHF(mol).run()
cubegen.density(mol, 'h2o_den.cube', mf.make_rdm1())
If I use a particulas MO, por example: mf.mo_coeff[:,[2]], I have problems
with the rdm1.
IndexError: boolean index did not match indexed array along dimension 1;
dimension is 1 but corresponding boolean dimension is 18.
How can I choose the MO I want?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/909#issuecomment-820817645, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC2IJVF3RNONS2YLDLTTI56FDANCNFSM43ALEOHQ
.

Thanks a lot Mr. Berkelbach.
Your suggestions was very helpful


Fix timing instrumentation logs for Skype and GooglePhotos

Bring both workloads in line with the standard way of regex searching the instrumentation logs.

Since this will be common functionality for Android UI benchmarks, is it feasible to move it into AndroidUiAutoBenchmark class in workload.py ?

It is common to the work we are doing with UX Performance workloads.  Android UI benchmarking is a broader topic.  You can find plenty of other workloads which take a different approach to instrumentation and logging.


Implement triangular&hexagonal networks

We need triangular&hexagonal networks for PhD thesis of Aboubakar.

This has been done in Moose-Algos-Graph-Generators package


Can provide a linux binary package?

Can you provide a linux binary package?
like "brew install infer".
and we can just download the binary package and use it directly.

@jvillard

This question has already been answered: #470, please comment there if you have more questions.


BigMart Sales Prediction

Description
I have added the notebook which predicts the BigMart Sales using XG-Booster algo. Please review @khusheekapoor
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

(Add x in the [] to mark as completed.)
Related Issues or Pull Requests
Fixes #419

I have updated the readme.md file. Please review @khusheekapoor

@khusheekapoor Pls review the changes.

@khusheekapoor Review the changes and give me ur suggestions.

@khusheekapoor Review the changes and give me ur suggestions.


errors.Wrap calls shadow the source of the error

When a Fybrik controller faces an error, sometimes its error is wrapped by the error.Wrap call and returned to the upper call.
As result, we can see the error in the fybrik application status.errorMessage, and we don't see the real point in the code where the error was accused.
Before returning the wrapped error, we need to print an error message into the log.

manager.log


code --diff does not support /dev/null

VS Code Version: 1.54.1
OS Version: Ubuntu 18.04.5 LTS

Steps to Reproduce:
VS code is configured as git difftool via
[difftool "vscode"]
  cmd = code --wait --diff $LOCAL $REMOTE

git init
touch foobar
git add .
git difftool --cached

Problem: instead of file diff opened in a new tab, current window is closed and new window is opened with foobar file.
Root cause is that git passes /dev/null if file was added or removed, for example in above case difftool command will be
code --wait --diff /dev/null foobar

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

Workaround
[difftool "vscode"]
  cmd = code --wait --diff $([ "${LOCAL}" = "/dev/null" ] && mktemp || echo "${LOCAL}") $([ "${REMOTE}" = "/dev/null" ] && mktemp || echo "${REMOTE}")

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


Questions regarding evaluation

Dear author,
Thank you for your great work.
I have some questions regarding the procedure for evaluating the ordinal error in your paper.
Did you first random sample 50k pairs of points, filter equidistant points, and then measure the ordinal error on the remaining points?
Or, did you random sample 50k pairs of points that are not equidistant and then measure its error?
Thanks,

Hi, thanks for your interest!
For the evaluation, we did not filter out any point pair from the 50k random samples. As written in Section 4.3 of the paper, pairs with the exact same depth have a relation of 0, which is also covered by the ordinal error metric definition, but, however, almost never occurs in the data as we do not quantize the depth maps before.

Thank you for your quick answer. And also thank you for the updates on the evaluation code, which will greatly help me out!
I have some further questions regarding the evaluation procedure. As far as I know, Ibims and DIODE datasets have their own masks for filtering pixels with valid depth. Have you used them during your evaluation? I could not find them in the current code version.
Thanks,

You can find the evaluation code in the respective branch: https://github.com/julilien/PLDepth/blob/eval
Unfortunately, we did not use these masks for iBims-1 and DIODE, only for HR-WSI (which also provides depth consistency masks; we used them for training and evaluation). What we did was assigning the depth capacity of 350 to zero values in DIODE (which reflect depths far off; see https://github.com/julilien/PLDepth/blob/8cb385e595053abc6762c536b2d807d7df784dbd/pldepth/eval/test_suite.py#L46). That being said, I would recommend to use the masks, which would result in less noisy results, but I wouldn't expect the relative order of performances changing too much (as all evaluated methods suffer from potentially distorted depths). The performances should get a bit better with the masks.

Thank you. One quick last question, is the performance gap big between a model that is trained on the sampling strategy of Sec 3.5(decreasing order of sum of depth difference) and random-only sampling, with all other factors controlled? (e.g. number of points, architecture)

It depends on the number of total sampled rankings R per image. If R is small, the chances to observe only a few particularly "informative" rankings in random-only sampling is relatively high. As we sample N*R rankings per image with N > 1 (in our experiments N=5), we get a significantly higher chance to observe particularly informative rankings as we see way more of them. In our experiments, we used R=100 to get the results as shown in Tables 2 and 3, there was a quite notable performance gap, so I would suggest you to stick to our advanced sampling strategy.

Thanks!


[REQUEST] Implement semantic verification of fields shape_pt_lat, shape_pt_lon, shape_pt_sequence and shape_dist_traveled in shapes.txt

Is your feature request related to a problem? Please describe.
These numeric fields should be user configurable so that suspicious values can be reported.
Describe the solution you'd like

[ ] Add new Warning notice:

SuspiciousShapePtLatNotice
SuspiciousShapePtLonNotice
SuspiciousShapePtSequenceNotice
SuspiciousShapeDistTraveledNotice

[ ] Add support to provide thresholds for these fields values via configuration file and/or command line

[ ] Determine default values for said thresholds

I don't think shape_pt_sequence can really be suspicious - it's either a valid unique ID for a given shape_id or it's not (which would be an error).

Good point. Plus, this one can be touchy depending on the resolution of shapes. Editing this issue accordingly.

No semantic validation of fiels yet, per discussion.


Add the web app or slot info to output logs when executing commands for them

OS: All
Build Version: 20200520.14
Repro Steps:

Execute "Start streaming logs" command for one slot.
Execute "Start/Stop/Restart" commands for web app or slot.
Deploy to web app/slot.
Check the output logs.

Expect:
The log is clear to check the detailed resource info, like below screen.

Actual:
The log is unclear, only contains the resource name.

More info:
This issue also reproduces when swapping slot.

Gonna close this one as won't-fix. It's slightly different from your "expected" example, but it still has all the most important information about the resource/slot name