This PR fixes case in SS of baseline=True and baseline_spending=True

This PR solves a problem that was identified in Issue #782. The case when baseline=True and baseline_spending=True was not correctly treated in the code and was not tested anywhere in the unit testing files.
This PR will be ready for review once I confirm that the TPI works correctly in this case and I have included tests for this case.
cc: @jdebacker

Codecov Report

Merging #786 (7a1fc2e) into master (62fc89b) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #786   +/-   ##
=======================================
  Coverage   84.13%   84.13%
=======================================
  Files          19       19
  Lines        3411     3411
=======================================
  Hits         2870     2870
  Misses        541      541

Flag
Coverage Δ

unittests
84.13% <100.00%> (ø)

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

Impacted Files
Coverage Δ

ogcore/SS.py
86.81% <100.00%> (ø)

ogcore/fiscal.py
100.00% <100.00%> (ø)

@jdebacker. In Issue #782, you note that baseline_spending=True is only valid in the reform case (baseline=False) and that we cannot condition baseline_spending=True to have that requirement of baseline=False in ParamTools validation because the baseline parameter often changes within a script that performs a baseline and reform. @jdebacker proposed:

but we can put an assert statement somewhere that doesn't allow this combination of parameter values.

This PR is an alternative to the assert statement approach. To do the assert statement approach, we have to make sure we assert that 'baseline_spending == True and baseline == False` at every entry point to the model.
The approach of this PR allows the baseline_spending parameter to be more of a flag. Every if gate instance of p.baseline_spending == True is now accompanied by and not  p.baseline. This allows a user to erroneously set baseline_spending==True during the baseline run and to have the feature apply correctly to only the reform.
I am almost done running the full set of unit tests on my machine for this PR. But I am happy to go the assert statement route if you want. Awaiting discussion.

@rickecon writes:

To do the assert statement approach, we have to make sure we assert that baseline_spending == True and baseline == False at every entry point to the model.

One can do this assert approach by adding the following below line 35 in parameters.py (maybe making the message more clear/descriptive):
if baseline:
   assert not p.baseline_spending, "Cannot use baseline spending in a run of the model baseline"

I ran the full set of CI tests locally on my machine and had two failures in the test_SS.py module in the test_run_SS test function in specifications [Reform, use zeta] and [Reform, small open use zeta]. Both errors were RuntimeError: Steady state equilibrium not found, which is probably easily fixed with an adjusted starting value for initial_guess_r_SS.
ogcore/tests/test_SS.py ...........................F.F..           [  7%]
ogcore/tests/test_TPI.py ........................                  [ 12%]
ogcore/tests/test_aggregates.py ................................   [ 20%]
ogcore/tests/test_basic.py ....                                    [ 21%]
ogcore/tests/test_elliptical_u_est.py .......                      [ 22%]
ogcore/tests/test_execute.py .                                     [ 22%]
ogcore/tests/test_firm.py .......................................  [ 31%]
ogcore/tests/test_fiscal.py ..................                     [ 35%]
ogcore/tests/test_household.py ........................................  [ 45%]
ogcore/tests/test_output_plots.py ...................................... [ 53%]
ogcore/tests/test_output_tables.py .............                         [ 56%]
ogcore/tests/test_parameter_plots.py ................................... [ 64%]
ogcore/tests/test_parameter_tables.py .......                   [ 66%]
ogcore/tests/test_parameters.py .............                   [ 69%]
ogcore/tests/test_run_example.py ..                             [ 69%]
ogcore/tests/test_run_ogcore.py .                               [ 70%]
ogcore/tests/test_tax.py ...............................        [ 77%]
ogcore/tests/test_txfunc.py .........................           [ 82%]
ogcore/tests/test_user_inputs.py .........                      [ 84%]
ogcore/tests/test_utils.py ................................................................ [ 99%]
ogcore/tests/test_wealth.py ..                                  [100%]

The change in the most recent commit to test_SS.py::test_run_SS[Reform, use zeta] changes the initial guesses for r_ss and TR_SS in param_updates6 to 'initial_guess_r_SS': 0.09 and 'initial_guess_TR_SS': 0.05. This solves the first failure.
The second failure was more tricky. I tried all the permutations of  initial_guess_r_SS = [0.05, 0.06, 0.07, 0.08, 0.09, 0.10] and initial_guess_TR_SS = [0.03, 0.04, 0.05] in param_updates8 in the test_SS.py::test_run_SS[Reform, small open use zeta] test. None of these solved, and each ended with the same error RuntimeError: Steady state equilibrium not found. I am at a loss of how to solve this second failure.

@jdebacker. The question in this PR is whether to allow a user to specify baseline_spending=True as a flag or whether to require it to only be used in conjuction with the baseline=False. If we prefer the latter, then we should just go the "assert statement in parameters.py" route. Otherwise, the model code in the current PR allows the user to specify baseline_spending=True as a flag that can be in place during the baseline=True stage, even though it only applies to baseline=False. I like this second approach. One reason is for cases like in Issue #786. Another reason is that I like the flag definition and interpretation of baseline_spending. A final reason is that I think the use cases are spelled out more explicitly in the code. Happy to discuss.

@rickecon FYI, all SS tests pass for me on master branch:
(ogcore-dev) jason.debacker@JDEBACKER-2 OG-Core % pytest ogcore/tests/test_SS.py

============================= test session starts ==============================
platform darwin -- Python 3.10.2, pytest-7.0.0, pluggy-1.0.0
rootdir: /Users/jason.debacker/repos/OG-Core, configfile: pytest.ini
plugins: xdist-2.5.0, forked-1.4.0
collected 32 items
ogcore/tests/test_SS.py ................................                 [100%]
================= 32 passed, 23 warnings in 2571.40s (0:42:51) =================
(ogcore-dev) jason.debacker@JDEBACKER-2 OG-Core %

@jdebacker. This PR is almost ready for review. The errors that I was finding in test_SS.py have gone away. They were somehow an artifact of the code changes I had made previously, although I don't understand how those code changes made those tests fail. I did run all the test_SS.py tests in the master branch, and they all passed.
ogcore/tests/test_SS.py ................................                 [100%]

I re-ran the entire set of CI tests locally last night and got the following output, with only two errors in test_SS.py.
ogcore/tests/test_SS.py .....................F.F........                                     [  7%]
ogcore/tests/test_TPI.py ........................                                            [ 12%]
ogcore/tests/test_aggregates.py ................................                             [ 20%]
ogcore/tests/test_basic.py ....                                                              [ 21%]
ogcore/tests/test_elliptical_u_est.py .......                                                [ 22%]
ogcore/tests/test_execute.py .                                                               [ 22%]
ogcore/tests/test_firm.py .......................................                            [ 31%]
ogcore/tests/test_fiscal.py ..................                                               [ 35%]
ogcore/tests/test_household.py ........................................                      [ 45%]
ogcore/tests/test_output_plots.py ......................................                     [ 53%]
ogcore/tests/test_output_tables.py .............                                             [ 56%]
ogcore/tests/test_parameter_plots.py ...................................                     [ 64%]
ogcore/tests/test_parameter_tables.py .......                                                [ 66%]
ogcore/tests/test_parameters.py .............                                                [ 69%]
ogcore/tests/test_run_example.py ..                                                          [ 69%]
ogcore/tests/test_run_ogcore.py .                                                            [ 70%]
ogcore/tests/test_tax.py ...............................                                     [ 77%]
ogcore/tests/test_txfunc.py .........................                                        [ 82%]
ogcore/tests/test_user_inputs.py .........                                                   [ 84%]
ogcore/tests/test_utils.py ................................................................  [ 99%]
ogcore/tests/test_wealth.py ..                                                               [100%]

The two tests failing in test_SS.py are test_run_SS[Reform, baseline spending] and test_run_SS[Reform, baseline spending, use zeta]. Both of these tests set baseline_spending=True.  But the code in the test_run_SS() test sets baseline=True which causes the error ValueError: Parameter baseline_spending=True cannot coincide with baseline=True. The issue is that a baseline_spending=True run requires baseline=False and requires baseline output to be saved in an OUTPUT_BASELINE folder. In this test, the OUTPUT_BASELINE directory is now a tmpdir from PR #775.
I think the current tests in the master branch which are all passing are not testing what we expect. They were cheating by setting baseline=True which would negate the baseline_spending=True setting. To really test baseline_spending=True, we need to set up a different test that runs a baseline first, then the reform so that any tmpdirs are not erased after the baseline run. The current offending code in test_SS.py::test_run_SS is in lines 509-518.
baseline_dir = os.path.join(tmpdir, 'OUTPUT_BASELINE')
if baseline is False:
    p_base = Specifications(
        output_base=baseline_dir,
        baseline_dir=baseline_dir,
        baseline=True,
        num_workers=NUM_WORKERS)
    p_base.update_specifications(param_updates)
    p_base.baseline_spending = False
    base_ss_outputs = SS.run_SS(p_base, client=dask_client)

I will work on making this change. Then this PR will be ready.


Installing dependency fails

The following command fails:
composer require open-admin-ext/helpers

Trace:
Problem 1
    - Root composer.json requires open-admin-ext/helpers ^1.0 -> satisfiable by open-admin-ext/helpers[v1.0, v1.0.1, v1.0.2, v1.0.3].
    - open-admin-ext/helpers[v1.0, ..., v1.0.3] require open-admin-org/open-admin ~1.0 -> satisfiable by open-admin-org/open-admin[v1.0, ..., v1.0.27] from composer repo (https://repo.packagist.org) but open-admin-org/open-admin[dev-readme-fix, dev-main] from path repo (/var/www/open-source/libs/open-admin) has higher repository priority. The packages from the higher priority repository do not match your constraint and are therefore not installable. That repository is canonical so the lower priority repo's packages are not installable. See https://getcomposer.org/repoprio for details and assistance.

Info:

Laravel Version: Laravel Framework 10.12.0
PHP Version:  8.1.18

Closing this issue, as I think it's not a bug. @open-admin-org please have a look if in case you consider it a bug. Thanks!


Add support for Java

Currently it does not add support for java

@everanurag Please see #359. Feel free to give your input on how you'd run a Java file.


common global variable with constant.py

Made a common constant.py for global commonly used variables.

@rajendraarora16 mind remaking this pull request from a public repo? Closing for now.


how to run the exemple on my card ???

I want to test the Pipe CNN on an Attila Arria 10 GX card with an Intel SDK open CL v17.1. I did the implentation and it's working!
Stil, I want to do the demo on my card:
hwo can I do that?

Shall I download the imagNet database?
Shall I upload caffe? and since I am using ubunto 16.04 shall I upload cuda toolbox 8.0 with it?
Please help! I wan't to simulate the demo!

thank you very much for the further explanation.

Hello @Fnajjar
Install the Intel SDK for openCL. This guide might help you.(https://www.altera.com/en_US/pdfs/literature/hb/opencl-sdk/aocl_getting_started.pdf)
Install OpenCV.
As this repo supports Intel opencl v16.1, make necessary changes for v17.1.
Download the imgaenet 2012 validation set.

Thank you very much for your answer @manojrohit, but how can I download the imgaenet 2012 validation set??
I didn't find it in the ImageNet website unfortunatly

You can download by registering in http://www.image-net.org/download-images.

Thank you @manojrohit,
So ones I upload the validation images 2012 from the imageNet, do I extract the folder in the PipeCNN-master/data/imagenet?
Is there modifications to make in the file's code?

If you go through the host cpp file, in line 71 you can see
char  picture_file_path_head[100] ="/home/dwang/Work/imagenet/ilsvrc2012/ILSVRC2012_img_val/ILSVRC2012_val_";
You can extract the validation data and change the path in host program.

@manojrohit thank you so much it's finally working
that was so helpful :)

Definitely YES : )

Thank you @doonny  for this great news! :)
Do you have previous work or some advices on how shall I proceed?
Thank you very much for the help

@Fnajjar @manojrohit @doonny
I was using DE1-SoC Cyclone V board and successfully compiled and run in the board the PipeCNN using PLATFORM=arm32 and flow=hw which only read one image file.
And now, I tried to enable OpenCV to experiment loading multiple images from Imagenet 2012 validation set.
I set USE_OPENCV = 1 in the Makefile and installed OpenCV 2.4.9 in my PC (Ubuntu 16.04).
When I compiled, I got the following error that says:
libopencv_core.so :file not recognized: File format not recognized

Any suggestion on how to solve this error? Is OpenCV 2.4.9 not compatible with the current set-up of PipeCNN?
Thank you very much.

First, you need to cross-compile your opencv lib for arm. Your opencv installed is for PC not for arm

Hi Sir,
how to change cross compile OpenCV  lib for arm?

Hi Sir,
how to cross compile OpenCV lib for arm?

How to use VNC for image visualization on de10_nano board @doonny @manojrohit @roddregs

I am assuming this problem has been solved.


Webpack HTML dependency files

Suggestion provided by @gfs
Description: how to consolidate various js and cs files needed for HTML format report to package them into a reduced set of references and avoid real-time internet reference and download time
My recommendation is as follows. This eliminates all dependency on external files being present on the disc, everything will be in the binary.

Use webpack either pre-build or as part of the pre-build build tasks to compile all your js and css assets. It sounds like your html is also one file too, which is great.
Embed those into your binary. Set the build task to EmbeddedResource in visual studio.
Have a simple test to ensure that the embeds are there.
At runtime access the embedded files like you do for your rules.
Do whatever processing you need to.
Write the embedded files to disc where the user has selected.

Looks good guy. Let me know if you need any help.

Thanks for digging into this one. I hope this resolves the issue I'm running into. I'm running v1.2.58 and the HTML output is non-functional and the output looks like #264. I tried from macOS and from Ubuntu and couldn't get either to produce usable output from the CLI.
I noticed a few resources are no longer referenced in my output.html when compared to an old report sample I saw online. I'm not sure this information is helpful or related but the first item looks highly important. The other JavaScript resources (query, ace, d3, c3, popper) are now sourced from cdnjs.cloudflare.com but I no longer see these:

appinspector.js
ext-settings_menu.js
ext-modelist.js
ext-beautify.js

I will verify this and get back to you tonight.  Shouldn't see it in the 1.2.58 version. There are fewer resources reference because they are embedded in the HTML

I can't reproduce this using the dotnet tool version.
You can install that version (if you have the .NET SDK installed) with dotnet tool install -g Microsoft.CST.ApplicationInspector.CLI
PS D:\GitHub> appinspector analyze -s .\LogicalAnalyzer\
The output file does not have an html or htm extension and may not open properly in your browser
Analyze command running
100% source files processed
The output.html file size is large and may render slowly.  Consider running using alternate output format options as needed.
Opening default browser to output.html report
Analyze command completed
Additional details may be found in log file at C:\Users\Gstoc\.dotnet\tools\.store\microsoft.cst.applicationinspector.cli\1.2.58\microsoft.cst.applicationinspector.cli\1.2.58\tools\netcoreapp3.1\any\appinspector.log.txt

Thanks for looking into this. I still have the error and will provide more details in a separate issue (See #279) to not detract from this issue.

Closing in favor of rewriting UI in Blazor.


Catch corner & error cases & add default values to arguments

Fix #000 (<-- Add issue number here)
IMPORTANT: Each pull request (PR) should resolve an issue. Please create the issue if it hasn't been created, unless you're fixing typos or making minor changes to documentation or infrastructure. In that case, just leave the issue number as #000.
Details
If the PR title isn't enough to describe your PR, please provide details here.
Checks
Make sure these boxes are checked before your PR is ready to be reviewed and merged. Thanks!

[x] PR is concisely and descriptively titled 📑 and links to the original issue 🔗.
[x] Code is in uniquely-named feature branch and has no merge conflicts 📁.
[x] Tests pass -- look for a green checkbox ✔ a few minutes after opening your PR.
[ ] @mentions of the person or team responsible for reviewing proposed changes. By default, use @BadwaterBay/reviewers.
[ ] (Optional) Screenshots/GIFs are attached 📎 in case of UI updates.

We're happy to help you get this ready. Don't be afraid to ask for help, and don't be discouraged if your tests fail at first.

Thank you for contributing to this project!

Codecov Report

Merging #97 into master will decrease coverage by 1.36%.
The diff coverage is 53.12%.

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
- Coverage   63.41%   62.05%   -1.37%
==========================================
  Files           6        6
  Lines         164      195      +31
  Branches       10       16       +6
==========================================
+ Hits          104      121      +17
- Misses         59       70      +11
- Partials        1        4       +3

Impacted Files
Coverage Δ

src/core/helpers/read_write.py
34.28% <25.80%> (-25.72%)
:arrow_down:

src/core/find_route.py
89.15% <70.83%> (+2.67%)
:arrow_up:

src/core/helpers/helper.py
100.00% <100.00%> (ø)

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 ae1457a...10d44e4. Read the comment docs.


JobDSL: Upgrade Jenkins used in tests, fix unstable test

Upgraded Jenkins to the more recent version, fix missing folder in test.

Fixed deploy_jobs.groovy.


Add browser info at least user-agent to OTEL traces

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
https://opentelemetry.io/docs/specs/semconv/resource/browser/ specifies a user_agent.original attribute.  Can that be added into the traces at minimum?  The rest can be added later.
The use case is if we look at the trace from the top which is actually traefik.  We want to know as much information about the client so we don't have to dig through the lower level traces to find out information that would've been available to Traefik already.

Hey @trajano,
Thanks for your suggestion.
We are interested in this issue but are unsure about the use case and the traction it will receive, so we are going to leave the status as "kind/proposal" to give the community time to let us know that they would like this.
We will reevaluate as people respond.

I've updated the OP for use case

Fixed by #10223


24 words seed is too long

Give users the option to create 12 18 or 24 words seeds.

12-word 'recovery' should be enabled in the next fw release.
12-word 'new mnemonic' will be enabled in a future fw release.

12-word 'new mnemonic' added (as the default) in fw release 0.1.27


Mobile fixes

Still can't seem to remove the overlay from mobile :/ But the modal is full screen which looks better.


help manage my disk space

Bazel workspaces can occupy a lot of space. bazel clean will fix that, but first you have to know that Bazel is the reason you're low on disk, and how big each Bazel workspace is.

https://gist.github.com/mr-salty/a66119941e797d9eb49b15ea211ea968 has a recipe that might be useful

Dont want to parse the entire disk if we can avoid it
A large number of bazel workspaces will be siblings to the bazel repo you are working on right now. Should be able to easily see those caches without checking the entire disk
Want to remove some combination of largest / "longest since last used" bazel caches first
Prompt user before removing anything
Will need to support windows as well but can be done as a separate pass later. Make sure to make a ticket for tracking purposes

I think we can land this without the repository cache, and without looking for "other" disk cache. The current disk cache is fine.
Comment on the upstream bazelbuild issue where the disk-cache has no eviction. https://github.com/bazelbuild/bazel/issues/5139#issuecomment-941525571


Create a page that will list all topics #261

Description
This is the PR for a new page under /admin/topics which will fetch the list of topics from the API and display them
Fixes # (261)
How to test?
Checklist

[ x ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ x ] I have made corresponding changes to the documentation
[ ] This PR is ready to be merged and not breaking any other functionality

@Pallabi-ghosh Can you fix that warning please?

@Pallabi-ghosh Can you fix that warning please?


fix: show username on cert if name is private

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #40765

Ideally, should we be not even fetching cert.name, if showName is false?

Good call @ShaunSHamilton 👍 - I think I fixed it.


Fix pypi image

Needs to be a full raw.githubusercontent.com/* for the image to show up on pypi, solves the logo not showing here: https://pypi.org/project/accelerate

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Cannot find type definition file for 'lodash' error on following step described in npm site.

I followed all the steps listed in :   https://www.npmjs.com/package/toastr-ng2
I am using the fountain.js scaffolding with webpack for my angular 2 project.
I am getting the following typescript error hence my code is not compiling further :
ERROR in /project-root/node_modules/toastr-ng2/portal/portal.d.ts
(1,1): error TS2688: Cannot find type definition file for 'lodash'.

I went ahead and even did npm i lodash, and typings install lodash and
import * as _ from 'lodash'; in my project but nothing helped.
Please help !!

Sounds like a release issue. I'll come up with a fix soon. You can also try installing @types/lodash to fix now.

fixed in 1.5.6


Fix #555 - Permalinks have larger click target.

I like it, even though it kind of looks like clicking it will take me to Twitter :)

Ha - that's fair. Should we use a different symbol? like a pilcrow ? (¶)

Sorry, should have waited to merge for a response! (But hey, the stats!!) I think it's okay with #, really. But if @ryanseys comes along and thinks it's weird, I'm happy with whatever the team decides!

The # comes from the # in the anchor URL but I think that's probably not really the case here because it goes to a new page. I'm really indifferent but a pilcrow (that's a fun word) looks nice too!


Header menus are left aligned

possibly after removing ng-bootstrap and Bootstrap tags the menus got left aligned and got back to all uppercased type. @Xymmer requested type to be Capitalized since the very beginning of the project, Bill pls confirm if this is still the case.
before

after

checking on this

We've aligned the implementation with the designs from PD team, including uppercase and left alignment. I think it's fine and believe it's their domain, and it looks great, espcially on mobile from my perspective.

so it's as intended, closing


Save and Load AEGs as JSON Files

Add support for saving and loading AEGs to and from JSON files.

[x] Write a method for serializing AEGs to JSON objects using the browser JSON api
[x] Have a method and UI element that downloads the JSON file with a default name when clicked on. (Quick Download)
[x] Have a method and UI element that opens a save menu where the user can rename the file and select where to download it. (Slow download)
[ ] implement CTRL+S to call the Slow Download method.
[x] Write a method for creating an AEG from an uploaded JSON file. For typescript you will need an interface that represents the data to properly load the JSON, see here for help
[x] Implement a UI element that when clicked opens a file menu for the user to select an AEG file to load, when selected, it should try to load the AEG into the draw workspace. If the file can't be parsed as an AEG, an error popup should appear in the UI.

Forgot Control+S


Missing parameter ProxyPool description

What has happend? When running command "get-help move-csuser -full" it list a parameter: "-proxypool" and that is what we have used earlier. Now when reading online documentation, there is no comments about that at all?
But then you do have comment: "To move a user to Office 365, specify the ProxyFqdn of the hosting provider". But then there is no parameter as "ProxyFqdn".
Is it me only who think this is not fully correct?

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

ID: 15c68752-3b36-518f-8cf1-7a4e2aa8cf15
Version Independent ID: 98b9a798-e232-0b2c-ee0f-857071885c1d
Content: Move-CsUser (skype)
Content Source: skype/skype-ps/skype/Move-CsUser.md
Service: skypeforbusiness-powershell
GitHub Login: @hirenshah1
Microsoft Alias: hirshah

@officedocsbot assign @yogkumgit

Hello @Petri-X
Thanks for your feedback.
Actually, the full sentence in the online documentation says:

To move a user to Office 365, specify the ProxyFqdn of the hosting provider as the Target.

And then, in Target it says

If moving to Office 365, this must be set to the ProxyFqdn value of the hosting provider. In most cases this is sipfed.online.lync.com. The value of the ProxyFqdn can be obtained using Get-CsHostingProvider

About ProxyPool, digging into the commits, I saw that @bjwhalen was the author of the pr that removed the parameter from this article, but found no description in the pr, I see that the parameter is still present in SfB 2019, so can you shed some light here @bjwhalen and @tomkau ? We appreciate.

You are correct @get-itips ! :)
I just saw a term "proxy" and I could not separate ProxyPool and ProxyFQDN from each others :D

Hello @Petri-X
I am glad that I could help you, if you agree, we will proceed closing this one.
Thank you

@get-itips, I still would like to see what about the "-ProxyPool". That in my mind is still missing.

@get-itips, I still would like to see what about the "-ProxyPool". That in my mind is still missing.

ok for that, we depend on the current author @hirenshah1 or @bjwhalen

@hirenshah1 Please help us with your inputs on this issue.

@yogkumgit I am not aware about this change. @bjwhalen can clarify

@bjwhalen can you please add here your inputs. Sent you an email as well.

@bjwhalen Please respond to our query.

@bjwhalen can you please review this thread and respond to our comments? I appreciate

Hello @hirenshah1 or @bjwhalen can you help us here? Thanks

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @Petri-X Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Problem installing chromatic on Windows

I tried to install chromatic-lightcurves on windows and it gave me this error. I have downloaded the Microsoft Visual C++ already and have even tried running the code in the Developer Command Prompt for VS 2022.
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for batman-package
Failed to build batman-package
ERROR: Could not build wheels for batman-package, which is required to install pyproject.toml-based projects

My plan for this is to:

turn batman-package into an optional dependency, and just warn people if they want to use it
explore python-only transit codes, or at least ones that are easier to install
help @zashaavery get set up with being able to compile C code

It looks like the numpy-only version of https://github.com/exoplanet-dev/exoplanet-core might do the trick. @catrionamurray , does that seem reasonable to you as a replacement for batman? If so, could you help a little with the translation from the kinds of inputs we provide to inject_transit to what we need to provide for flux = quad_limbdark_light_curve(u1, u2, b, ror)? I think this is probably closely related to what you've needed to be doing for all your jax experiments.

@will-waalkes , do you have experience with this Microsoft C++ Build Tools error that @zashaavery 's getting? Any ideas? We installed the visual-cpp-build-tools but didn't explore too deeply down into what other things we need to install to be able to compile things via pip.

@will-waalkes , do you have experience with this Microsoft C++ Build Tools error that @zashaavery 's getting? Any ideas? We installed the visual-cpp-build-tools but didn't explore too deeply down into what other things we need to install to be able to compile things via pip.

If I remember correctly, I installed the Visual Studio tools and got things to start working but I had to play with which version - this page might help https://stackoverflow.com/questions/40504552/how-to-install-visual-c-build-tools.
I believe my process looked like installing different versions of Visual Studio and then trying to run code until one of the versions worked. My longer-term fix to coding on windows is either (a) download a Linux virtualbox (I can help with that) or there are some browser-based IDEs which might allow you to skip over the pesky windows dependencies, google collab might still be a good option for that? I haven't used it in quite some time.

It looks like the numpy-only version of https://github.com/exoplanet-dev/exoplanet-core might do the trick. @catrionamurray , does that seem reasonable to you as a replacement for batman? If so, could you help a little with the translation from the kinds of inputs we provide to inject_transit to what we need to provide for flux = quad_limbdark_light_curve(u1, u2, b, ror)? I think this is probably closely related to what you've needed to be doing for all your jax experiments.

Yep I can help with this! Shall I create a new branch + pull request using exoplanet-core instead of batman?

Added pull request for exoplanet-core method that shouldn't require any special imports like batman!


Feature request: Make it visible what code is shared

Hi!
Thanks for the great plugin indeed, also the improvements you did reg. my previous ticket #99.
I am coming with another usability request.
At some point during my sessions I've realised that ChatGPT responded in ways that was "suspiciously" specific to my code while my prompt was generic. I started to do some tests and found out that part of my code was shared with the AI.
Now this is perfectly fine and indeed there's a feature that selected code is shared. Probably i had some bits selected during the chat.
However I don't know what code will be shared with the AI? - Code selected when I start writing my prompt or code selected when I hit the send button?
Not sure about others, but while writing prompts I usually copy paste keywords or fragments from my editor. So at the end of the day not knowing exactly what other code is added to the prompt makes the situation kind of working blind.
I would really like to see the communication with the API "as it is" without the plugin adding code fragments to my prompts and hiding those in the chat UI. It makes working with the AI very confusing. Also it is easy to accidentally start writing to the chat while something is selected in the editor.
Probably showing the included code in full may bloat the interface a bit, but Id happily accept that in exchange for clarity. If there was an option to "not hide anything". We are developers after all. We don't really expect the reality to be hidden from our eyes.
Also there should be a way to see when code is shared or about to be shared. Like an icon indicating that the next message would include some automatically added code segment, so at least I'd be aware that : Oh, snap, I started chat with something selected. Also a remove option before sending the message, so I could initiate/maintain generic chat instead of some unknown code fragments polluting the discussion.
You get what I mean I'm sure.
Also it may be problematic with huge code segments selected it may consume lots of tokens by accident...
Hope you can consider this. I assume many other devs would more like to see the chat interaction in full rather than some unknown code additions in the context we are not aware of.

Hey @BenceSzalai!
Thank you for the feedback!
I believe this behaviour should be configurable. Personally, I feel that the selected code will clutter the tool window, making it difficult to find the questions I was trying to ask previously. To view the previous context, you can reopen the tab.
I will try to fix it in the next release.

I believe this behaviour should be configurable.

Sure. I don't want to suggest changes that would break other user's expectations or established workflow.

This functionality is now available from v2.2.3.

Thanks, cannot wait to see!

This is perfect, very nice work! Thank you! You are spoiling us! :)

Thank you! 🙂


General epochs

Is the idea that EpochStart is run in the BeginBlock of the nextBlock while EpochEnd is run in the EndBlock of the previous block?
Would be good to specify and document that this means blocks will be inclusive.  The first block whose timestamp is after the duration is counted as the end of the epoch.
And that the next block after that is the new epoch.
Because there's not way of knowing that a block will be the last block whose timestamp is before a certain time.

Because there's not way of knowing that a block will be the last block whose timestamp is before a certain time.

Got it,


No tags in private DNS zone links

What happened? Provide a clear and concise description of the bug, including deployment details.
The private dns zone module does not pass incoming parTags to resVirtualNetworkLink and resVirtualNetworkLinkFailover which causes weird deployment errors if there are polices like "Require a tag and its value on resources" /Microsoft.Authorization/policyDefinitions/1e30110a-5ceb-460c-a204-c1c3969c6d62 in place.
https://github.com/Azure/ALZ-Bicep/blob/main/infra-as-code/bicep/modules/privateDnsZones/privateDnsZones.bicep
Also, please consider changing the link names, "link-vqlscyvvqtsm4" is very misleading.
Please provide the correlation id associated with your error or bug.
xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
What was the expected outcome?
resource resVirtualNetworkLink 'Microsoft.Network/privateDnsZones/virtualNetworkLinks@2020-06-01' = [for privateDnsZoneName in varPrivateDnsZonesMerge: if (!empty(parVirtualNetworkIdToLink)) {
//  name: '${privateDnsZoneName}/${take('link-${uniqueString(parVirtualNetworkIdToLink)}', 80)}'
  name: '${privateDnsZoneName}/${take('dns-${parVirtualNetworkNameToLink}-link-${privateDnsZoneName}', 80)}'
  location: 'global'
  properties: {
    registrationEnabled: false
    virtualNetwork: {
      id: parVirtualNetworkIdToLink
    }
  }
  dependsOn: resPrivateDnsZones
  **tags: parTags**
}]

Relevant log output
No response
Check previous GitHub issues

[X] I have searched the issues for this item and found no duplicate

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @sergey-netdev, thanks for raising this. Would you like to submit a PR for this addition?


Improve nuxt.config.js and Nuxt modules API

Resolves #1288

Codecov Report

Merging #1447 into dev will increase coverage by 0.08%.
The diff coverage is 95.23%.

@@            Coverage Diff            @@
##              dev   #1447      +/-   ##
=========================================
+ Coverage   87.01%   87.1%   +0.08%
=========================================
  Files          14      14
  Lines         901     915      +14
=========================================
+ Hits          784     797      +13
- Misses        117     118       +1

Impacted Files
Coverage Δ

lib/builder/webpack/server.config.js
93.33% <100%> (+1.02%)
:arrow_up:

lib/builder/webpack/client.config.js
80.76% <100%> (+0.76%)
:arrow_up:

lib/common/utils.js
100% <100%> (ø)
:arrow_up:

lib/builder/builder.js
92.64% <83.33%> (-0.36%)
: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 459363b...9c39c12. Read the comment docs.

Awesome!
Thank you @Zip753
Nice to see you in the contributors now!


Re-generate with latest tree-sitter-cli.

It automatically migrated TS-related data from package.json to a new tree-sitter.json

https://github.com/vlang/v-analyzer/pull/135 already contained commit ab24881 .


problem with unquantified model

I'm trying to execute the unquantified model. I made the changes to the tflite files and tags, in addition to changing the unquantified condition, I had to change this line by adding the multiplication by 4 let bytesCount = 4 * newWidth * newHeight * newComponentsCount * batchSize so that the error of byte input size. But it does not provide the same results of the quantified model.
I hope you can help me solve this problem, thanks in advance

All you have to do is change the scaledImage function to scaledPixelArray


Upgrade test suggestion fix

When in_flight_prev is an empty string, we get an extra line. Caught here https://coreos.slack.com/archives/C01UQM3PM3L/p1633427199193600

I believe this has been addressed recently in some other commits


When a property has a value tag, that tag is used instead of the summary tag

According to StyleCop rules the summary of a property must match its accessors.
That means that the summary for properties must be something like Gets or sets …, which is a description of the .NET property and not its value.
On the other hand, properties must have a value tag with text.
Since the description in the swagger is the description of the value, it makes more sense to use this tag.
This PR Solves this.

I'm not sure if the reverse is being done, but, if not, point me in the right direction and I'll do it.

See https://github.com/RSuter/NJsonSchema/blob/master/src/NJsonSchema.CodeGeneration.CSharp/Templates/Class.liquid#L30
Maybe we should preserve the original one (custom extension) and provide a setting for this (to avoid breaking changes)?

Thanks for the pr! 👍

The reverse?

By reverse I was referring to generate summary and value tags for properties with the correct wording.

It probably should be something like:
    /// <summary>{{ property.Description.GetPropertySummaryXmlDocumentationFromDescription() | csharpdocs }}</summary>
    /// <value>{{ property.Description.GetPropertyValueXmlDocumentationFromDescription() | csharpdocs }}</value>

I'm still not familiar with DotLiquid templating. The text between the double curly braces is C# code? What does | csharpdocs mean?

Also, now, GetXmlSummaryAsync looks a bit strange now for properties, since it can be Value instead of Summary.

I tried to generate the correct documentation for properties but I had to add properties to PropertyModel for that (https://github.com/RSuter/NJsonSchema/pull/707/files#diff-4383900dc8b7b5c4e2cfe8617cd3ef18R12).
Despite the name, these two tests seem to be testing the same thing:

When_enum_has_description_then_csharp_has_xml_comment - https://github.com/RSuter/NJsonSchema/pull/707/files#diff-c05b8be927a1d3f06dd8526c840a4801R260
Task When_property_has_description_then_csharp_has_xml_comment - https://github.com/RSuter/NJsonSchema/pull/707/files#diff-c05b8be927a1d3f06dd8526c840a4801R295

@RSuter, can you take a look at this? What do you think?


Update ruff check and ruff format to default to the current directory

Closes https://github.com/astral-sh/ruff/issues/7347

I'm looking into the behavior this exhibits with include per https://github.com/astral-sh/ruff/issues/3970 — it looks like more changes will be needed to properly support the desired behavior there.


FindingImageAccessToken exception

Bug description
Not sure if this is just my setup or not.
While creating an html report for a test, to obfuscate the /media/ directory, a call is made to /finding/image/. The code that receives this request is in /finding/views.py line 1358.
def downloading_finding_pic(request,token):
This is failing with an exception and the image will not load in the report. The exception error message is
dojo.models.FindingImageAccessToken.DoesNotExist: FindingImageAccessToken matching query does not exist.
Steps to reproduce
Steps to reproduce the behavior:

Create a test, like a Pen Test
Create a finding.
In the finding page, click the hamburger drop down and click "manage images" and upload an image.
Go back to the test, click the hamburger drop down again and click "Report"
Make sure "Finding Images" is set to yes and generate a report.
Notice that the picture is broken. Open up the developer controls in the browser and look at the network tab, refresh if necessary. Notice the 403 error on /finding/image/.

Expected behavior
The image should load on html report.
Deployment method (select with an X)

[ ] Kubernetes
[ ] Docker
[X] setup/setup.bash

Environment information

Operating System: Ubuntu 16.04
DefectDojo Commit Message: 8deafd1: Added readme to services information [2019-10-29 12:49:07 -0700]

Screenshots

Console logs (optional)
Unable to download finding pic
FindingImageAccessToken matching query does not exist.
Traceback (most recent call last):
  File "./dojo/finding/views.py", line 1363, in download_finding_pic
    access_token = FindingImageAccessToken.objects.get(token=token)
  File "/opt/dojo/lib/python3.5/site-packages/django/db/models/manager.py", line 82, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/opt/dojo/lib/python3.5/site-packages/django/db/models/query.py", line 408, in get
    self.model._meta.object_name
dojo.models.FindingImageAccessToken.DoesNotExist: FindingImageAccessToken matching query does not exist.

Issue no longer exists, has been replaced by #1674


Fix a link to MVC mac tutorial

@badersur Thank you again for your contributions!

You're welcome :)


Package learning

What if the program could ask the user if the selected package is correct in cases where names are not exact. This could also be used to learn what packages are Python and ruby. This data could then be sent to a server of some sort where it will be included in the repo for other people to download. You could also get this file by running a command in ulpm

https://github.com/jakebacker/ULPM/issues/10

#10

This data could be stored on an SQL server. Each package that has been installed will contain: the package name in different package managers, if other package managers have the package, and more that I cant think of right now

Server will be running Ubuntu 16.04 and mysql 5.7.15


Rename modules to jetty-server and tomcat-server

jetty has been jetty since before we had a client, but now we have a jetty-client, so it really ought to be called jetty-server.
And everything else is *-server or *-client, so Tomcat ought to be tomcat-server, too.
(Reminded of this by @vasilmkd's comment on #2702.)
I don't think we can scalafix an sbt script.  Fortunately, these modules are both <10% usage.  We'll just have to scream in the release notes.

I'll take this if you don't mind.


Swap auto-increment IDs for ObjectID

Auto-increment IDs are a feature of most databases. They're an easy way to create an id which is unique for each row in a relational database, so easy, it's not often thought about.
However, auto-increment IDs have several undesirable features:

Any replication system with multiple masters needs to have unique IDs, otherwise replication breaks.
Auto-increment IDs leak information, although this is not always a problem.
Auto-increment IDs don't globally identify resources.
This blog post covers several more small reasons why auto increments aren't great.

When Ghost was originally built, I already identified the need to be able to identify resources globally, and so all models in Ghost have both an auto-increment id and a uuid.
Far from solving the problem, this feels like the worst of both worlds:

We still have auto-increment ids and all the downsides they bring
We have an additional uuid using uuid.v4() that we rarely leverage and which is only "probably" unique, rather than being a guaranteed globally unique id.

The proposal for Ghost 1.0.0 is to move towards using a single ID for each resource, which is guaranteed to be globally unique. After some research into node modules which provide unique id's (there are many and this one was particularly promising), and also looking at what other platforms use, we settled on the idea of using ObjectID - the ID format that is used by MongoDB, via this npm module: https://github.com/williamkapke/bson-objectid.
Note, there are some small downsides as this could have a performance impact. One interesting potential performance issue is that innoDB uses auto-increment ids to optimise inserts. However, as ObjectID is still serial I don't believe this will be an issue.
Using ObjectId can also increase field and index sizes, I believe the impact for us will be minimal. At the moment, we do a poor job of adding indexes to Ghost, so even the fact that we intend to add an optimise them is going to be an improvement over the existing system. Further, we're reducing from having two potential IDs to need indexes on, to having one. There is plenty of content showing that the performance impact is minimal or a worthwhile tradeoff.
Finally, the one place where objectID might not be nice, is for post previews, we may need to revisit the implementation of these, ref #5097.

Already started with the implementation, but waiting for https://github.com/tgriesser/bookshelf/issues/1383 right now 👍

@ErisDS Shouldn't UUIDv4 be unique enough for this use case?
From reading the UUIDv4 Wikipedia page section about collisions:
https://en.wikipedia.org/wiki/Universally_unique_identifier#Random_UUID_probability_of_duplicates
Values are:
n = 68,719,476,736 (2^36), chance = 0.0000000000000004 (4×10^−16)
n = 2,199,023,255,552 (2^41), chance = 0.0000000000005 (5×10^−13)
n = 70,368,744,177,664 (2^46), chance = 0.0000000005 (5×10^−10)

As a global identifier this should be a pretty safe bet.
To limit collisions even more you can set the uuid field as the primary key or set a unique index and thus limit same-table collisions even though global collisions would still be possible in theory.

It's been a while since I've used MongoDB but by re-reading the documentation it would seem like ObjectIds are not much more random or unique than UUIDv4s.

objectid (12 bytes) = time (4 bytes) + machine identifier (3 bytes) + pid (2 bytes) + inc (3 bytes)

time = 4 bytes
-> epoch with a precision up to seconds
(https://github.com/williamkapke/bson-objectid/blob/2f8c1ca62280c3198131b02d243ea8341a04a364/objectid.js#L146)

machine identifier = 3 bytes
-> 3 random bytes generated at runtime and for the JavaScript / Node.js driver it's using the non-cryptographicaly secure Math.random() function as a source of random bytes
(https://github.com/williamkapke/bson-objectid/blob/2f8c1ca62280c3198131b02d243ea8341a04a364/objectid.js#L2)

pid = 2 bytes
-> pids are not generated randomly and a lot of Docker containers will run the main app on pid 1 for example
(https://github.com/williamkapke/bson-objectid/blob/2f8c1ca62280c3198131b02d243ea8341a04a364/objectid.js#L4)

inc = 3 bytes
-> basically the same
(https://github.com/williamkapke/bson-objectid/blob/2f8c1ca62280c3198131b02d243ea8341a04a364/objectid.js#L3)
(https://github.com/williamkapke/bson-objectid/blob/2f8c1ca62280c3198131b02d243ea8341a04a364/objectid.js#L141)

I haven't done the collision chance calculation but my gut feeling tells me it can't guarantee a lower collision rate then UUIDv4.

The idea is to at least guarantee no collision within a single system, like Ghost(Pro) where we can ensure that pids are unique etc.
If we really need to improve this further, we could always move to our own version of ObjectID that uses a proper machine fingerprint, but it seems to me if the current implementation is good enough for MongoDB, it's probably good enough for us.

The only real advantage I remember from working with ObjectIds is that since the time is the prefix you have a natural ordering by insert date.

This is a pretty big deal, especially as we're going to be focusing our efforts on optimising for MySQL (and InnoDB). This is the main reason for not using uuid.

closed via #7495


Translate content/ja/docs/home/_index.md in Japanese

ref: https://github.com/kubernetes/website/issues/10821

/lgtm

/approve


feat: add --no-modules-global support

This PR adds pass-through support for the wasm-bindgen flag --no-modules-global, which is very useful when building/running WASM in certain scenarios (browser extensions in particular).
Supercedes #860 which was mysteriously closed last summer.
Closes #729.

Make sure these boxes are checked! 📦✅

[x] You have the latest version of rustfmt installed

$ rustup component add rustfmt

[x] You ran cargo fmt on the code base before submitting
[x] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

Hi there, any thoughts on this?

LGTM (as a random person on the internet with no approval/commit rights)

Hi there, it's been another month. Any thoughts?

LGTM

No clue, we've been using it for a while without issue.

I'd really like to have this feature as well, but apparently wasm-pack is lacking maintainers
https://github.com/rustwasm/wasm-pack/issues/928
I wonder if I should use wasm-bindgen cli directly for this while waiting (?).


zabbix_agent_listenip seems to be ignored since 1.7.0

Describe the bug
After upgrading ansible-zabbix-agent to 1.7.0 from ansible-galaxy zabbix_agent_listenip seems to be ignored:
zabbix/zabbix_agentd.conf doesn't contain ListenIP anymore on all hosts where I set zabbix_agent_listenip.
Installation method/version

Ansible Galaxy, role version: 1.7.0

Ansible Version
ansible 2.7.6
  config file = /home/stk/nextcloud/dirkfiles_martiniesrv/vagrant/baseserver/ansible.cfg
  configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/dist-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.15rc1 (default, Nov 12 2018, 14:31:15) [GCC 7.3.0]

Targetted hosts
Concerns the following OS(es):

Ubuntu 18.04

Expected behavior
zabbix/zabbix_agentd.conf should contain ListenIP=<IP_ADDRESS> on all hosts where I set zabbix_agent_listenip: <IP_ADDRESS>.
Additional context

Has been working fine on every version since #98. Thank you very much!
This is a security issue since the agent now is reachable via ip/interface where it should not.

Looks like I broke it with #228. Working on fix.

@dsteinkopf
Where you able to test the provided PR, when possible?
Which version of Ansible are you using?

Yes, thank you. See in the PR #232 for my comment.
I am using ansible 2.7.2 on ubuntu 18.04 (see above)

Hi @dsteinkopf
I pushed 1.7.1 to the Galaxy to solve your issue.


Build for Apple Silicon

I fixed the CLA.

I don't understand this change.  The darwin/amd64 build should already be included by virtue of darwin & amd64 being in the respective arrays and not being excluded.  Can you explain this change if I'm misunderstanding?

Oh sorry, it says "arm64" not "amd64".  👓


When marking an argument as deprecated it shows the python code name for the argument in the deprecated message

it should show --use-squash-merge instead of use_squash_merge.

it should be specified in the call as target='--use-squash-merge'


added header for the page, added whiteboard for noting, added toggle …

…button for open/close whiteboard, adjusted page css, all screen fit except for Galaxy Fold
!!! Carefully check src\App.js, src\App.css and src\components\WordlePanel

Double check wordle panel part (css) and check button part (js, css), I added jquery so when it is merged you need to run npm install after git pull origin main to make sure it could be run locally!


Error while installing typings

I am windows10 user and tried to install angular2.  I couldn't insatll it with npm install at first but with an modified package.json Ii was able to install it.
{
"name": "angular2-quickstart",
"version": "1.0.0",
"scripts": {
"start": "npm run lite",
"lite": "lite-server"
},
"license": "ISC",
"dependencies": {
"@angular/common": "2.0.0-rc.5",
"@angular/compiler": "2.0.0-rc.5",
"@angular/core": "2.0.0-rc.5",
"@angular/forms": "0.3.0",
"@angular/http": "2.0.0-rc.5",
"@angular/platform-browser": "2.0.0-rc.5",
"@angular/platform-browser-dynamic": "2.0.0-rc.5",
"@angular/router": "3.0.0-rc.1",
"@angular/router-deprecated": "2.0.0-rc.2",
"@angular/upgrade": "2.0.0-rc.5",

"core-js": "^2.4.0",
"reflect-metadata": "0.1.3",
"rxjs": "5.0.0-beta.6",
"zone.js": "0.6.12",

"angular2-in-memory-web-api": "0.0.15",
"bootstrap": "^3.3.6"
},
"devDependencies": {
"concurrently": "^2.0.0",
"lite-server": "^2.2.0"
}
}

It didn't created a typings folder so I ran npm run typings -- install which produced this error
F:\AngularJS\my-proj>npm run typings -- install
npm ERR! Windows_NT 10.0.10240
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\Aiden\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "run" "typings" "--" "install"
npm ERR! node v4.5.0
npm ERR! npm  v3.10.6

npm ERR! missing script: typings
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     F:\AngularJS\my-proj\npm-debug.log

What should I do to install it?

You removed the typings command from that package.json.

how could I add it?

Adding it back :P

I used the original package.json and it gave this error
F:\AngularJS\my-proj>npm run typings -- install

<EMAIL_ADDRESS>typings F:\AngularJS\my-proj
typings "install"

typings WARN deprecated 7/25/2016: "registry:dt/core-js#0.0.0+20160602141332" is deprecated (updated, replaced or removed)
typings WARN deprecated 8/23/2016: "registry:dt/node#6.0.0+20160807145350" is deprecated (updated, replaced or removed)
typings ERR! message Unable to read typings for "core-js". You should check the entry paths in "core-js.d.ts" are up to date
typings ERR! caused by Unable to connect to "https://raw.githubusercontent.com/DefinitelyTyped/DefinitelyTyped/95e782233e8e203a0b9283c3a7031faee428a530/core-js/core-js.d.ts"
typings ERR! caused by self signed certificate in certificate chain
typings ERR! cwd F:\AngularJS\my-proj
typings ERR! system Windows_NT 10.0.10240
typings ERR! command "C:\Program Files\nodejs\node.exe" "C:\Users\Aiden\AppData\Roaming\npm\node_modules\typings\dist\bin.js" "install"
typings ERR! node -v v4.5.0
typings ERR! typings -v 1.3.2
typings ERR! If you need help, you may report this error at:
typings ERR!   https://github.com/typings/typings/issues
npm ERR! Windows_NT 10.0.10240
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\Aiden\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "run" "typings" "--" "install"
npm ERR! node v4.5.0
npm ERR! npm  v3.10.6
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>typings: typings "install"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>typings script 'typings "install"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-quickstart package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     typings "install"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular2-quickstart
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular2-quickstart
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     F:\AngularJS\my-proj\npm-debug.log

That is the first time I see that error, seems like it cannot connect to that url for some reason (a self signed certificate).
I have no idea at all.

Is there any way around to create typings?

Are you behind a proxy?
See if this helps: https://github.com/typings/typings/blob/master/docs/faq.md#configuration

Hi I do not see typings in your package.json file.
{
"name": "angular2-quickstart",
"version": "1.0.0",
"scripts": {
"start": "concurrently "npm run tsc:w" "npm run lite" ",
"lite": "lite-server",
"postinstall": "typings install",
"tsc": "tsc",
"tsc:w": "tsc -w",
"typings": "typings"
};

What about the last line @hhhcube? :P

I do not understand which last line you are referring to @Foxandxss

I found the problem.. It was the proxy. I created a .typingsrc file with
{
  "rejectUnauthorized": false
}

It installed typings successfully.
$ npm run typings install

><EMAIL_ADDRESS>typings F:\AngularJS\my-proj
> typings "install"

typings WARN deprecated 7/25/2016: "registry:dt/core-js#0.0.0+20160602141332" is deprecated (updated, replaced or removed)
typings WARN deprecated 8/23/2016: "registry:dt/node#6.0.0+20160807145350" is deprecated (updated, replaced or removed)

├── angular-protractor (global)
├── core-js (global)
├── jasmine (global)
├── node (global)
└── selenium-webdriver (global)

@Altair827 this solves the problem


Missing ExecutionTimeLimit in Triggers

Missing ExecutionTimeLimit in New-ScheduledTaskTrigger
https://docs.microsoft.com/en-us/powershell/module/scheduledtasks/new-scheduledtasktrigger?view=win10-ps

@az-devops
Upon your feedback, We have updated the content with relevant changes accordingly. Thanks.

@officedocsbot close


Design changes

kole

The aspect ratio is not ready, needs to be like you open the camera, and I don't see barcodes strings.


Add a voice analyzer for shapekeys proposal

Design a proposal for a voice analyser.

Closing because not directly related to VR locomotion or networked object picking.


Get transform matrix from camera to world coordinates

Required Info

Camera Model
D455&L515

Firmware Version

Operating System & Version
Win 10

Kernel Version (Linux Only)

Platform
PC

SDK Version
{ v2.3.2 }

Language
{python }

Segment
{ }

Issue Description
Hi,
I am quite new to this camera and I am having two cameras D455&L515 for doing the 3D reconstruction, but every time I just use single camera.
I am using the algorithm called NeRF which requires the transform matrix, how can I get this one?
I have looked through quite many issues here and I saw we can get extrinsic matrix from sdk which is about aligning color to depth or vice versa. Is this matrix the "transform matrix camera to world"?
More Descrption
Firstly I am sure the format matrix I want to get is combined of rotation and translation. Or we call it camera pose
I am doing the indoor video streaming, so the camera was kept moving and imagines were captured. The camera coordinate is moving as well.
Hence, I need captured imagines and corresponding matrix from camera to world coordinate for each imagine
The world origin does not matter where it is. But i really donot know how to get this Matrix.
Thanks in advance.

Hi @Yiiii19  If you are aligning color to depth or depth to color on the same time then you should not need to get involved with extrinsics, as the RealSense SDK's align_to instruction can perform the calculations for you via its Align Processing Block.  An SDK example program for depth to color alignment in Python at the link below that is compatible with both the D455 and L515 camera models is align-depth2color.py
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/align-depth2color.py
For color to depth alignment, change line 60 from 'align_to = rs.stream.color' to align_to = rs.stream.depth
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/align-depth2color.py#L60

In regard to obtaining the extrinsic transform between a pair of sensors, you can do this in Python with the get_extrinsics_to instruction.  A RealSense team member provides a Python example code for depth and color at https://github.com/IntelRealSense/librealsense/issues/2124#issuecomment-407187377
Alternatively, you can obtain extrinsics for all supported sensor combinations including depth to color and color to depth by using the SDK's rs-enumerate-devices tool, launching it in calibration information mode with the command rs-enumerate-devices -c

Hi,
thanks for your so fast reply.

Firstly for your question at the end: I just use one camera either D455 or L515. So far i donot need any transform between thee two cameras.

One question about what you replied to me, want to make sure if I understand you correctly:
(we only talk about depth to color now)

Through align-depth2color.py the results are color and aligned depth imagines
get_extrinsics_to can give me the  extrinsic transform matrix which can align depth to color

I just added more description of my question and you can check that again please.
Thanks again!

The align_depth2color.py example program results in depth aligned to color, yes.
If using the align_to SDK instruction then you do not need to retrieve the extrinsics in order to perform alignment, as the SDK automatically performs the calculations and adjustments between depth and color for you.

Great thanks.
Can you or your collegue also answer me the question at the beginning?
Thanks!

Firstly I am sure the format matrix I want to get is combined of rotation and translation. Or we call it camera pose
I am doing the indoor video streaming, so the camera was kept moving and imagines were captured. The camera coordinate is moving as well.
Hence, I need captured imagines and corresponding matrix from camera to world coordinate for each imagine
The world origin does not matter where it is. But i really donot know how to get this Matrix.

The pose of the camera is only supported as a built-in feature on the RealSense T265 Tracking Camera model, which supports a pose stream.  It is possible to pair a T265 with another RealSense camera model (for example, on the same mounting bracket) so that the T265 can provide pose whilst the other camera provides depth information.  Intel have published a guide about this at the link below.
https://www.intelrealsense.com/depth-and-tracking-combined-get-started/
If your project cannot use a T265 then it should be possible to obtain the camera's relative position on a map using ROS and SLAM.  A guide at the link below that was created by a RealSense ROS user demonstrates a method for doing this.
https://shinkansan.github.io/2019-UGRP-DPoom/SLAM

T265 camera is possible for me to use. I just look through the example code. I do see the code dealing with pose stream but I am not sure which one is correct?
print("Position: {}".format(data.translation))
what does the position look like (I cannot test it yet cause i didnot have the T265)?
Thanks!

It looks as though the quoted line comes from the program t265_example.py.  If it does, the image below shows an output from that program.

Yes, it comes from t265_example.py

I do think this position is in camera coordinate. But what does the position mean? like of one point?
How can I get the position in world coordinate/camera pose in world coordinate? That is exactly what I really want to know

The T265's position is in world coordinates, as described in the T265 documentation link below.
https://github.com/IntelRealSense/librealsense/blob/master/doc/t265.md#sensor-origin-and-coordinate-system
I believe that when the T265 starts up, its position will be 0,0,0 and then as the T265 is moved, its position updates to reflect how many meters it has travelled in world distance from its 0,0,0 start position.

Thanks for your reply. Quite clear about the T265 camera.

If your project cannot use a T265 then it should be possible to obtain the camera's relative position on a map using ROS and SLAM. A guide at the link below that was created by a RealSense ROS user demonstrates a method for doing this.
https://shinkansan.github.io/2019-UGRP-DPoom/SLAM

I just have a look through yesterday about that repo, I unfortunately find the method to get the mapping matrix I want. Do you have any idea?

Regarding obtaining the pose of a camera without a T265, the available possibilities are limited.
Apparently in ROS, RViz has a 2D Pose Estimate button when RViz has been set up with the Navigation Stack.  This enabkes the pose of a robot to be set.  There is a tutorial about this at the link below.
http://wiki.ros.org/navigation/Tutorials/Using rviz with the Navigation Stack

Many thanks  for your help!

You are very welcome, @Yiiii19 - thanks very much for the update!


Add Update test to Client Go

Saw that this test was missing, might add some confidence when adding the status subresource.

@adiprerepa: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
64c87686b279edbba4f153e49a05159b6a200bf7
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Add Harsh to people

Add him to the README too with something he said about Devzat. cat.txt does nothing.

Oh also, thanks for the PR and for helping out Devzat!

oof lol I added him anyway


Creating Scroll up button in Home page to go to top

Creating Scroll up button in Home page to go to top.

Hello @sagarbee,
Thank you for creating an issue! ⭐
So, I'm officially assigning this task to you under SWOC. So feel free to start working on it.
Please keep me updated on your progress through images/videos in the comment section before raising the PR.
PS: Always mention the issue number in your PR in the following format: Resolve issue #<issue_number>

If you have any questions or need clarification, don't hesitate to ask in the comments or on Discord.
#Good luck! ✌

This issue is Solved.
So I closed this. ⭐
Thanks for the contribution! ✌


fix #33

Reconnect logic is now disabled when disconnect is by request (call to .end() or .destroy()).
This is not tested though, I don't have time for that. I may have missed some place where reconnect logic should be enabled back ( .retry.closed set to false ).

Would be accepted with tests. Otherwise will wait until someone with commit access has the time to write tests, which could be several months.

Well for me it could also be several months, when I'd have some time I'll
look into it, but no promises there.
--
Pozdrawiam
Bartosz Grabias
+48 793669953
2015-02-02 18:15 GMT+01:00 Charlie Robbins<EMAIL_ADDRESS>
Would be accepted with tests. Otherwise will wait until someone with
commit access has the time to write tests, which could be several months.
—
Reply to this email directly or view it on GitHub
https://github.com/foreverjs/nssocket/pull/34#issuecomment-72496258.

This PR doesn't completely solve the issue. In the case where the retry timeout has already been created, this will still end up throwing. I have a PR coming soon which will fix this issue and a bunch of others I found getting nssocket ready for my own use.

@AlmirKadric +1


Support adding necessary modules when generate tarballs.

Is your feature request related to a problem? Please describe.
I'm always frustrated when all modules are added when generate tarballs. It make the tarball too big.
Describe the solution you'd like
Support adding necessary modules when generate tarballs.
Urgency
General

@jiacheliu3 Would you like to take a look? Thanks♪(･ω･)ﾉ

@Xenorith @jiacheliu3

is your request specific to the mechanism in which a tarball can be generated via the existing scripts or is it a comment on the tarballs available for download, for example in https://downloads.alluxio.io/downloads/files/ ?
if it is with regards to the tarball generation script, are you aware of the -ufs-modules flag in the ./generate-tarballs release command?

is your request specific to the mechanism in which a tarball can be generated via the existing scripts or is it a comment on the tarballs available for download, for example in https://downloads.alluxio.io/downloads/files/ ?
if it is with regards to the tarball generation script, are you aware of the -ufs-modules flag in the ./generate-tarballs release command?

Yeap! I'll try it! Thanks♪(･ω･)ﾉ

This issue will be closed, feel free to reopen the issue if need further discussion, thanks for submitting the issue!

is your request specific to the mechanism in which a tarball can be generated via the existing scripts or is it a comment on the tarballs available for download, for example in https://downloads.alluxio.io/downloads/files/ ?
if it is with regards to the tarball generation script, are you aware of the -ufs-modules flag in the ./generate-tarballs release command?

I have tried and found some problems：

This issue will be closed, feel free to reopen the issue if need further discussion, thanks for submitting the issue!

It seems that I can't turn it open, can you help me turn it open again? Thanks♪(･ω･)ﾉ

is your request specific to the mechanism in which a tarball can be generated via the existing scripts or is it a comment on the tarballs available for download, for example in https://downloads.alluxio.io/downloads/files/ ?
if it is with regards to the tarball generation script, are you aware of the -ufs-modules flag in the ./generate-tarballs release command?

I have tried and found some problems：

When I specify -ufs-modules "ufs-hadoop-ozone-1.2.1", the compilation will report an error, because the package name that the system wants cp is not the actual package name.（Actually is alluxio-underfs-hadoop-ozone-1.2.1-2.10.0-SNAPSHOT.jar, but alert alluxio-underfs-ozone-2.10.0-SNAPSHOT.jar: No such file or directory）
Modules like fuse will continue to be compiled. Since fuse does not support arm currently, there will be some problems.

The alert is due to I added ozone configuration in dev/scripts/src/alluxio.org/build-distribution/cmd/common.go,  and the alert will not be reported after rollback.
`var libJars = map[string]struct{}{
"integration-tools-hms":        {},
"integration-tools-validation": {},
"underfs-abfs":          {},
"underfs-adl":           {},
"underfs-cos":           {},
"underfs-cephfs":        {},
"underfs-cephfs-hadoop": {},
"underfs-gcs":           {},
"underfs-local":         {},
"underfs-obs":           {},
"underfs-oss":           {},
    "underfs-ozone":           {},
"underfs-s3a":           {},
"underfs-swift":         {},
"underfs-wasb":          {},
"underfs-web":           {},

}`

Modify the ozone configuration in lib-jars, and specify lib-jars and ufs-modules at the same time to package only ozone. Other ufs have not been tried yet, and they may be the same
var libJars = map[string]struct{}{
"integration-tools-hms":        {},
"integration-tools-validation": {},
"underfs-abfs":          {},
"underfs-adl":           {},
"underfs-cos":           {},
"underfs-cephfs":        {},
"underfs-cephfs-hadoop": {},
"underfs-gcs":           {},
"underfs-hadoop-ozone-1.2.1":           {},
"underfs-local":         {},
"underfs-obs":           {},
"underfs-oss":           {},
"underfs-s3a":           {},
"underfs-swift":         {},
"underfs-wasb":          {},
"underfs-web":           {},
}

@JySongWithZhangCe it sounds like you found the solution on your own

-ufs-modules will specify which UFS jars to build. if left empty, it will build the default set of ufs modules, which is a fairly long list
-lib-jars will specify which which jars within the lib folder to be copied into the tarball. by default it will copy the list you found defined by var libJars in common.go. this is the current list of UFS jars, besides the hadoop ones, that are built by default and would be copied by default.

so your conclusion is correct. any ufs that is not built by default would need to specify both flags in order for it to be included into the final tarball. note that we cannot add the hadoop-ozone-1.2.1 underfs line as part of libJars because we would not normally build that and the tarball generation will fail with a file not found when trying to copy that path.
let me know if there are other problems i may have not addressed and/or if the issue needs to be reopened


config: unset config endpoint/unmanaged route (PROJQUAY-2049)

Only sets status.configEditorEndpoint if cluster supports Routes and the
Route component is managed.

/cherrypick redhat-3.6


[11.x] Add parameters to Str::password() for mixed case and case limited passwords

This adds two new optional parameters, $upperLetters and$lowerLetters to Str::password() to ensure that a generated password contains upper case characters and/or lower case characters.
The current $letters parameter pulls from a pool of both upper and lower case letters and does not guarantee inclusion of either upper case or lower case letters. Adding these option as new parameters ensure that the current $letters behavior remains unchanged.
There are a few benefits and reasons for this change:

Passwords can be generated by Laravel that will pass the mixedCase() validator. Currently if you are trying to generate temporary passwords for users but also have the mixedCase validator implemented you have to move to rolling your own password generation method.
Passwords can be generated for external systems that require all lower case or all upper case letters.

At this time we're not customizing this method anymore. You can write your own password method if you wish. 👍


Can't update rows with null values in keys

Fixed in calebmer/postgrest-client#8, will close this once it's merged

This is actually resolved since conditions are always the primary keys now, which can never be null.


docs: unify spelling to OpenAPI

Status

READY
Description
Update inconsistent  from  "OpenApi" to "OpenAPI"  on the document.
Related PRs
none
Todos

[x] Tests
[x] Documentation
[x] Changelog Entry (unreleased)

Steps to Test or Reproduce
none

Thanks!


:bug: Fix target card behavior and rendering

Resolves: #2022
Resolves: #2010
Resolves: #1981
Resolves: https://issues.redhat.com/browse/MTA-3094
Target card behavior and rendering on the target selection step of the analysis wizard and on the custom migrations target page have been updated:

All of the card contents render inside the card itself (#1981 / MTA-3094)
When changing the specific label selection for a target, the card selection will not changes (#2010)
The target label select now sorts the labels in numeric natural sort descending order.  OpenJDK 21 will be initially selected and appear above OpenJDK 17 in the select list. (#2022)
Layouts used on the card have been refactored to use base layouts instead of the EmptyState component
The target label select box has been updated to the current SimpleSelectBasic component for current PF5 alignment
When card selection is enabled (on the target selection step of the analysis wizard), the selection checkbox is always displayed and the normal Card hover styles have been disabled.

Some refactoring on the SetTargets component used by the analysis wizard have been made to simplify rendering code.

Custom migration targets now:


Everything is green

I'd like to appreciate your immense effort for sharing your exquisite experience. I've found the code you've had created recently.It is tremendously efficacious and constructive.However, I faced a drastic problem which is the code detects the color of all cars as green even they are of different colors. Could you please help me find a solution if it doesn't bother you.Thanks in advance. I am looking forward for your response.
Have a good day.
here's some screenshots

Hi @M0stafaAhmed,
Thank you for taking the time to provide feedback on this repository. I'm glad you find the code efficacious and constructive.
Regarding the issue you mentioned about the code detecting all car colors as green, regardless of their actual colors, I appreciate you bringing this to my attention and I'll do my best to help you resolve it.
Could you please provide some additional information to help me better understand the issue?

If you've made any modifications, please share the modified code as well.

Are you running the code on a specific platform or environment (e.g., Windows, Linux, macOS)?

Have you followed the installation instructions correctly, ensuring that all required dependencies are installed properly?

If you have any error logs or stack traces, please share them as well.


Update manifests/t/Tencent/TencentVideo/11.47.9543.0/Tencent.TencentVideo.locale.en-US.yaml

Fixes an issue where this package has multiple different monikers
Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels


bump revision to 8

Not compatible with direct <IP_ADDRESS> or greater. Will have to hotfix direct <IP_ADDRESS>.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.7 to before v4.19.220",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ec25bc04ed8e12947738468cbe2191f1529f9e39",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=676dc7d9b15bf8733233a2db1ec3f9091ab34275"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ec25bc04ed8e12947738468cbe2191f1529f9e39",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "676dc7d9b15bf8733233a2db1ec3f9091ab34275",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net/mlx4_en: Fix an use-after-free bug in mlx4_en_try_alloc_resources()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.19.220 by commit 676dc7d9b15bf8733233a2db1ec3f9091ab34275, it was introduced in version v4.7 by commit ec25bc04ed8e12947738468cbe2191f1529f9e39. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002419


Naming conventions for environmental variables

Proposal

service related: RENGA_{SERVICE}_{WHATEVER_WITH_UNDERSCORES}
global config for multiple services RENGA_SERVICES_{...}

Acceptance

[ ] authorization
[x] deployer
[ ] graph
[ ] storage
[ ] ...

--
cc @erbou @leafty @rokroskar @ebiiii

lets revisit this in a cleanup of development branches


ParsErr

Hello, when I try to run the script, I receive the following error:
ParsErr "(line 979, column 92):\nunexpected \"\\n\"\nexpecting \"\\r\\n\" or \"==========\\r\\n\""
Do you know what this can be? I appreciate your help in advance.

Hey, if you'd like you could send me your clippings file via email and I could investigate!
It is on your kindle -> "documents" / "My Clippings.txt

Mhh this one is weird, I could synchronize you file without any problem, maybe you could try from a different computer?
(please wait for the 0.2.6 release thought, I am fixing something else, should be ready in a minute)

Mhh this one is weird, I could synchronize you file without any problem, maybe you could try from a different computer?
(please wait for the 0.2.6 release thought, I am fixing something else, should be ready in a minute)

Oh okay, let me know whenever the release is ready and I'll try it, okay?

It is ready!
On Fri 16. Oct 2020 at 23:58, MariDantas<EMAIL_ADDRESS>wrote:

Mhh this one is weird, I could synchronize you file without any problem,
maybe you could try from a different computer?
(please wait for the 0.2.6 release thought, I am fixing something else,
should be ready in a minute)
Oh okay, let me know whenever the release is ready and I'll try it, okay?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/yannick-cw/notionfy/issues/16#issuecomment-710671786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS6362UQLN3YKHTK74YSSDSLC6XFANCNFSM4RY67S3Q
.


Fix test.

Needs global_changes app started and get host from chttpd.
rebar -r eunit apps=global_changes now passes 4 tests.

+1. @kxepal can you merge?

I can merge, but I can't verify because I got OOM in the middle of the test suite ):

heh

+1 Downgraded from 18.1 to 17.5 erlang to make them passed.


Removed lambda reflection because it is considered harmful.

Fixes #1112

Current coverage is 94.96%

Merging #1115 into master will decrease coverage by -0.07% as of c70a8c7

@@            master   #1115   diff @@
======================================
  Files           77      77
  Stmts         9318    9192   -126
  Branches      1729    1729
  Methods          0       0
======================================
- Hit           8855    8729   -126
  Partial         91      91
  Missed         372     372

Review entire Coverage Diff as of c70a8c7

Powered by Codecov. Updated on successful CI builds.


Allow installing without setting the JAVA_HOME variable

I want to use this plugin for a toolchain. So I want to keep the existing JAVA_HOME.

Hi!
Thanks for raising this issue!
I have the following solution in mind, that I think is flexible enough:

By default, the action will set the JAVA_HOME env var to whatever path it downloaded the JDK to.
You can override this environment variable to whatever you'd like.

Let me know, if that's okay for you.

Works for me!

Please refer to the updated README regarding the new configuration options.
You can find examples in the Build Master workflow.
Also, as this is a non-breaking change, it is now released on the v1 branch.
Please reopen the issue if you run into any problems!


refactored tips

There was an error while validating job 1ef2ccf3-83b7-4c1e-b477-65058bbf5dc8 Message:Error clonning repositoryDetails:System.ArgumentException: Error clonning repository ---> LibGit2Sharp.NotFoundException: Branch 445 not found. Available branches: origin/master
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass34.<>c__DisplayClass39.b__2e(KeyValuePair2 branch)    at System.Collections.Generic.List1.ForEach(Action1 action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass34.<CloneRepository>b__2c()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.<ExecuteAction>b__0()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetTreeAsync(String path, Boolean includeSubDirectories, String excludePath, IList1 extensionWhiteList)    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.ListSourceControlFilesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)


Add 5.1.0+trunk+stedolan+PR11103

This PR adds a benchmark run for the ocaml/ocaml PR 11103 and should partially address https://github.com/ocurrent/current-bench/issues/374
Xavier Leroy mentions here his lack of confidence in using the linear scan allocator since he's not sure how much use it has seen in production. Would it help if we also benchmark the compilation of some packages in the "wild" using this branch, like we do here?

I'm curious, are there numbers available on compile time after this change, what do they say? (I'm not very familiar with the Sandmark infrastructure, can someone point to an URL to look at?)

I had assumed that the intention in a Sandmark branch for this PR was to measure an effect on compilation time. (I don't think it would change the runtime of compiled programs in a noticeable way.) Sorry for the noise :-)


Update RemoteDesktopManagerFree download URL

Updating line 6479 download URL to reflect new URL

@mechman27 FYI: https://github.com/mechman27/RemoteDesktopManager/pull/1


chore: AD-35 move quality-dashboard deployment files to quality-dashboard repo

The aim of this PR is to have basically the same deployment, but the files should be moved to quality-dashboard repo.
Here's the diffs of yamls generated by running kustomize build components/quality-dashboard/<frontend|backend> before and after this change:
$ diff old-frontend.yaml new-frontend.yaml
7d6
<   namespace: quality-dashboard
15d13
<   namespace: quality-dashboard
23d20
<   namespace: quality-dashboard
44d40
<   namespace: quality-dashboard
96c92
<         image: quay.io/redhat-appstudio/quality-dashboard-frontend:5e4d63a8c07414a6f1ec736c1fa09d5ea433436b
---
>         image: quay.io/redhat-appstudio/quality-dashboard-frontend:9b6b1a71be8fbb7452fba572e95af9729e931c7d
126d121
<   namespace: quality-dashboard

diff old-backend.yaml new-backend.yaml
9d8
<   namespace: quality-dashboard
35d33
<   namespace: quality-dashboard
91c89
<         image: quay.io/redhat-appstudio/quality-dashboard-backend:8cf9faea4364a89aab4f6c6076aa8276844fb3fb
---
>         image: quay.io/redhat-appstudio/quality-dashboard-backend:9b6b1a71be8fbb7452fba572e95af9729e931c7d
116d113
<   namespace: quality-dashboard

From these diffs it's clear that only change is removed namespace (but that should be handled by ArgoCD from the Application definition, right?) and updated images to newest version of dashboard.

/lgtm