docs(ngMock): use master installation template

What kind of change does this PR introduce?
docs update
What is the current behavior? (You can also link to an open issue here)
Google CDN e.g. {% code %}"//ajax.googleapis.com/ajax/libs/angularjs/X.Y.Z/"{% endcode %}

What is the new behavior (if this is a feature change)?
Google CDN e.g.
"//ajax.googleapis.com/ajax/libs/angularjs/X.Y.Z/angular-mock.js"

Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Other information:
The inline installation can't display the proper context. On the website, it shows something like
Google CDN e.g. {% code %}"//ajax.googleapis.com/ajax/libs/angularjs/X.Y.Z/"{% endcode %}

Hm, that's bad. But we cannot use the default installation notes because they will make it look like you should include ngMocks into your application code wheras it should be included in your test runner script.

Oh well. The markup is wrong. I'm not familiar with the syntax. How do I fix it? 😄

We can either use static version numbers or change how the template is
handled.

Basically, we want to let nunjucks parse the docs.installation here.
\cc @petebacondarwin :grin:

@petebacondarwin, this is still not 100% fixed. Things like {$ doc.packageName $}, {$ doc.packageFile $} are still literally appearing in the output.

I think that 921f80e1eabf04ecd4dcfda85e7ac801a8f17728 fixes these outstanding issues.

So we don't need context to feed the templates but just hardcode the actual file name and versions?

Generally, he installation instructions are automatically generated based on a template (and fed with dynamical values, e.g. package name, file name etc). There are a couple of cases (including ngMock) where we need specific, non-generic installation instruction, so we provide them directly. In such cases, we can (and should) pass the exact content, including package name and file name.


MTP-1478 add capping check in amount form

oh and i've remembered something from the jira story: @mattbee suggested using a second environment variable to have as a switch for this feature. otherwise we can't deploy this change until policy takes effect.

oh and i've remembered something from the jira story: @mattbee suggested using a second environment variable to have as a switch for this feature. otherwise we can't deploy this change until policy takes effect.

We realised the same on Friday! So we'll get the feature flag in today.


translate description to english?

will you allow me to translate description in english ?

No problem, thanks


Why I cant change subtotal from update method?

When I tried to update the subtotal from update method, the setted value in update method is overriding.
Cause in the method updateRow() inside Shoppingcart/Cart.php, after the foreach, there is an if that recalculates the value of subtotal with quantity * price.
But, if I need change de subtotal after apply some discount per item? What I need to do?
I think that, if I am trying to change de subtotal, this math (quantity * price) needs to be ignoring.
Is there a business rule that explains this?

Why closed? @Crinsane

Sorry, forgot to comment. I've done a majpr rewrite which makes subtotal a calculated field, so it can't be set either way. Also, adding discount to a product is a feature that will be implemented in the next commit.

@Crinsane is there any progress for this?


Doc on installing the paraview module

Yes, we can use in a "headless" mode without starting the GUI. Not always, but there are some ways where it would work from the standard Python / IPython console:

The most generic way is to conda install -c conda-forge paraview
In ArchLinux, the paraview package is built with system python: sudo pacman -S paraview; python -c 'import paraview' works
The Paraview way of using pvpython tool is a bit complicated. And the most common hack is to append the virtual environment site-packages path to sys.path or $PYTHONPATH. This would be the last resort.

Originally posted by @ashwinvis in https://github.com/exabl/snek5000/pull/7#discussion_r589602818

@akhoubani I think it is a good issue for you :slightly_smiling_face:  Could you please work on it?
Like that, you will learn how to modify the documentation.
You can start once #91 will be merged.
Note: with hg, I think you have nothing to do about the change master -> main. Just pull and update to main before starting this work. If you like, you can delete the old master bookmark like this hg book -d master.

@akhoubani I think it is a good issue for you slightly_smiling_face Could you please work on it?

Yes, I'll work on it.

Like that, you will learn how to modify the documentation.
You can start once #91 will be merged.
Note: with hg, I think you have nothing to do about the change master -> main. Just pull and update to main before starting this work. If you like, you can delete the old master bookmark like this hg book -d master.

Thanks.

#91 has been merged so it is now time to work on this one!

@akhoubani do you need some help for this one ?

@akhoubani Please put the high priority label for this task. This is also a way to thanks @ashwinvis for its great help on what we are doing. If you need help, just tell me.

To be honest, it is not clear for me where should I add this explanation https://github.com/exabl/snek5000/issues/87#issue-1044475626. If I add it here is not convenient in my view. Here talks about post-processing but it is about something else. At this page which is under user guide menu, it is some how convenient, but I dislike to mix things. What I propose is that to allocate a section here for installing modules.

You can either

create a section in install.rst or
create a standalone document say install_opt.rst and link it in install.rst using a toctree directive

I prefer the latter.

You can either
* create a section in `install.rst` or

* create a standalone document say `install_opt.rst` and link it in `install.rst` using [a toctree directive](https://www.sphinx-doc.org/en/master/usage/restructuredtext/directives.html?highlight=toctree#directive-toctree)

I prefer the latter.

I'll follow the later. Thanks.

Some more tips:

Run make autobuild under ./docs to automatically render the result and preview it in the browser
You can write in both Markdown or reStructuredText. That is up to you. Some useful docs:

rST: https://docutils.sourceforge.io/docs/ref/rst/directives.html, https://www.sphinx-doc.org/en/master/usage/restructuredtext/directives.html
Markdown: https://myst-parser.readthedocs.io/en/latest/syntax/reference.html


Fix build with two-phase-borrows (aka NLL)

Why this change btw?

Because otherwise we would mutably borrow from something behind a const pointer, and that's not allowed and mistakingly works with the old borrow checker :)

Ok! Thanks!


Add Support for Azure CLI #695

Adds Support for Azure CLI #695

Includes Support for

Sub-Command Expansion
Target/Source Branch expansion
Parameter=Value expansion

Updates VSTS CLI Support

Fills in gaps for missing commands
Feature parity with AZ (branch/parameter expansion)

Both AZ/VSTS can be installed at the same time (with different aliases)
Added Tests for all cases

Awesome! For the old vsts CLI I've already created a module but adding the az CLI would be great (although I'm not sure if it should rather be its own standalone module)
https://github.com/bergmeister/posh-vsts-cli


Intellisense is broken after literals.

If you type:
1.
You see all classes instead of integer members.
Similarly typing:
"aaa".
Shows all classes. For string literal case pressing  backspace and pressing dot again fixes the issue.

Known problem in FSAC, reported in https://github.com/fsharp/FsAutoComplete/issues/120

Yes sorry for missing it though in my case it also doesn't work for string literals too.


Implicit broadcast of scalar values to vector values in the MathBuilder constructor

The math dialect in MLIR supports operations on scalar or vector inputs, but not a mixture of the two.
When generating SIMD code, we would like to generate a single source of code that can then be used to generate both SIMD code for the innermost SIMD loop, as well as scalar code for the remaining few iterations, if any.
The code to generate SIMD will load memrefs either as scalar or vectors; but often there are operations that needs constants, such as "compare to zero", "max of zero"... the scalar are needed then as a scalar in scalar code, but as a splatted constant when generating SIMD code.
The most robust method to provide this functionality is to provide implicit "splat" for each of the MathBuilder operations.
This PR does this.
This code will later be put to good use in a subsequent PR.
As an illustration of how this is used, here is a preview of where this new capability in this PR will be useful.
            emitSimdLoopIE(create.vec, simdUB, VL, // loop from 0 to simdUB by VL
                {X}, {inputAF}, // list of input values to load prior to the kernel, with their respective access function.
                {Y},  {outputAF}, // list of output values to save after the kernel is executed, with their respective function
                [&](VectorBuilder &vb,  // kernel functions with
                    ArrayRef<Value> inputVals,  // input values pre-loaded (either in scalar or simd mode)
                    SmallVectorImpl<Value> &resVals) // output values to be saved later (either in scalar or simd mode)
                 { // kernel code
                  MultiDialectBuilder<MathBuilder, VectorBuilder> create(vb);
                  Value x = inputVals[0];
                  // Scale
                  Value scaleX = create.math.div(x, scale);
                  // Round
                  Value roundX = create.math.round(scaleX);
                  // Adjust
                  Value adjustX = create.math.add(roundX, zeroPoint);
                  // Saturate
                  Value saturateX = create.math.clip(adjustX, qMin, qMax);
                  Value res = create.math.cast(quantizedElementType, saturateX);
                  resVals.emplace_back(res);
                });

In the above code, scale, zeroPoint, qMin, qMax are scalar that are needed in scalar or simd mode. We don't want explicit splats as we don't know in which mode we run, so we want them to be implicitly added when the inpputVals[*] are actually simd vectors.

@tungld implemented the scheme which we agreed on, thanks for the input.

Jenkins Linux ppc64le Build #14314 [push] Implicit broadcast of sc... started at 15:28

Jenkins Linux s390x Build #15289 [push] Implicit broadcast of sc... started at 15:16

Jenkins Linux amd64 Build #15284 [push] Implicit broadcast of sc... started at 14:16

Jenkins Linux amd64 Build #15284 [push] Implicit broadcast of sc... passed after 1 hr 7 min

Jenkins Linux s390x Build #15289 [push] Implicit broadcast of sc... passed after 1 hr 42 min

Jenkins Linux ppc64le Build #14314 [push] Implicit broadcast of sc... passed after 2 hr 15 min


related to compatibility Artemis: some specific configurator methods

related to PR: https://github.com/MassTransit/MassTransit/pull/2660
Adding Artemis compatibility (district09) #2660
Added some extra specific configurator config methods to open up generic extension points during config
Also added an extra test for those methods

Merged.


Exhume on Snap should exhume next turn if snapping at end of turn

Version
Slay the spire: BETA
Mod version: v0.49.0
Describe the bug
Cards which "Exhume when you Snap" return to your hand in the middle of end-of-turn processing if you snap at the end of turn 7, but this isn't actually the desired effect despite being accurate as written; we want those cards available during turn 8.
To Reproduce

Use Will on something
Wait for turn 7 to end
Observe that the cards exhume at the end of turn (when you snap), and are then immediately discarded (as end-of-turn finishes)

Expected behavior
Cards should be available during turn 8

Fixed by 960b424


Fix: GCP asset internet exposure NO_VALUE exception bug

This PR solves Issue 760.
Description:
The GCP asset internet exposure analysis job produced an exception with the following error in on statement #7: ERROR:cartography.graph.job:Unhandled error while executing statement in job 'GCP asset internet exposure': Expected a numeric value but got: NO_VALUE
This was caused by an empty allow_rule.fromport and allow_rule.toport.

You might need to close and reopen this PR to restart the CLA check

oh I forgot to sign the CLA, now I did. Sorry for that.

Thanks for the contribution!


Single contributor model epic test

We've used a single contributor rules-based model to produce a list of browser IDs which are likely to favour single contributions.
This is a hardcoded AB test to use that list. We hope to measure 2 things:

£AV/impression (is the variant better at making money?)
Number of targetable browsers (how useful is browser ID based data?)

The list of IDs has been manually uploaded to S3. Each SDC instance loads the list into memory on startup.
If a client request contains a browserId, and it appears in the list, then they are put into the test.
DCR will send the browserId if the user has consented: https://github.com/guardian/dotcom-rendering/pull/3545
I tested this in CODE with 3m (fake) browserIds and the t4g.small still has plenty of spare memory, so no need to change the instance type.
Control (UK/AUS)

Variant (UK/AUS)
Has different highlighted text and defaults to single:

EU/ROW has different first sentence

US has different highlighted text in the control only

Hi @tomrf1, @marialani, @michaelbjacobson and I are mobbing on testing defaulting choice cards to 'Single' contributions in the US. We noticed that in this PR, when the contribution defaults to single, an amount isn't automatically selected. Is this intentional?

Hi @tomrf1! @marialani, @michaelbjacobson and I are mobbing on testing defaulting choice cards to 'Single' contributions in the US. We noticed that in this PR, when the contribution defaults to single, an amount isn't automatically selected. Is this intentional?

Good spot! I'll investigate

Fix here - https://github.com/guardian/support-dotcom-components/pull/578


Added two new channels to YouTube section

Added two channels for the youtube section.  One specifically for career advice on applying for jobs in the industry. - How to apply to jobs. And another for PHP specific web development courses. - Code Course:General web development tutorials focused around PHP.

KGB.  Mzbg.


TiFlash failed to compile because of raftstore-proxy in tiflash-proxy

General Question

Based on Ubuntu18.04, multiple compilations using Clang-13 using readme instructions failed.  Error: Duplicate definition in raftStore-proxy linking.
I have tried this several times in release 6.0, as well as in the master branch, and the problem is repeatable and unavoidable. I doubt that raftstore-proxy, as intermediate code linking rust and c, would have potential compilation issues.
Compiling raftstore-proxy v0.0.1 (/root/ren/TiCluster/TiFlash/tiflash/contrib/tiflash-proxy/raftstore-proxy)

error: linking with `cc` failed: exit status: 1

 |
  = note: "cc" "-Wl,--version-script=/tmp/rustct6hrdj/list" "-m64" "/root/ren/TiCluster/TiFlash/tiflash/contrib/tiflash-proxy/target/x86_64-unknown-linux-gnu/release/deps/raftstore_proxy.17cgr5h210wspq13.rcgu.o" "/root/ren/TiCluster/TiFlash/tiflash/contrib/tiflash-proxy/target/x86_64-unknown-linux-gnu/release/deps/raftstore_proxy.addr2line-707913eaee2e7894.addr2line.d31b1425-cgu.0.rcgu.o.rcgu.o" "/root/ren/TiCluster/TiFlash/tiflash/contrib/tiflash-proxy/target/x86_64-unknown-linux-gnu/release/deps/raftstore_proxy.addr2line-f1073b39b9204403.addr2line.7a31b1b2-cgu.0.rcgu.o.rcgu.o"
......

 = note: /tmp/rustct6hrdj/libopenssl_sys-d85dc704372de778.rlib(b_addr.o): In function `BIO_ADDR_new':
          b_addr.c:(.text.BIO_ADDR_new+0x0): multiple definition of `BIO_ADDR_new'
          /tmp/rustct6hrdj/liblibrocksdb_sys-2494246a976b1fbd.rlib(b_addr.o):(.text+0x0): first defined here
          /tmp/rustct6hrdj/libopenssl_sys-d85dc704372de778.rlib(b_addr.o): In function `BIO_ADDR_free':
          b_addr.c:(.text.BIO_ADDR_free+0x0): multiple definition of `BIO_ADDR_free'
          /tmp/rustct6hrdj/liblibrocksdb_sys-2494246a976b1fbd.rlib(b_addr.o):(.text+0x60): first defined here
          /tmp/rustct6hrdj/libopenssl_sys-d85dc704372de778.rlib(b_addr.o): In function `BIO_ADDR_clear':
          b_addr.c:(.text.BIO_ADDR_clear+0x0): multiple definition of `BIO_ADDR_clear'
          /tmp/rustct6hrdj/liblibrocksdb_sys-2494246a976b1fbd.rlib(b_addr.o):(.text+0x80): first defined here
          /tmp/rustct6hrdj/libopenssl_sys-d85dc704372de778.rlib(b_addr.o): In function `BIO_ADDR_make':
          b_addr.c:(.text.BIO_ADDR_make+0x0): multiple definition of `BIO_ADDR_make'

error: could not compile `raftstore-proxy` due to previous error
Makefile:169: recipe for target 'build' failed
make[5]: *** [build] Error 101
Makefile:185: recipe for target 'release' failed
make[4]: *** [release] Error 2
contrib/tiflash-proxy-cmake/CMakeFiles/tiflash_proxy.dir/build.make:944: recipe for target '../contrib/tiflash-proxy/target/release/libtiflash_proxy.so' failed
make[3]: *** [../contrib/tiflash-proxy/target/release/libtiflash_proxy.so] Error 2
CMakeFiles/Makefile2:6783: recipe for target 'contrib/tiflash-proxy-cmake/CMakeFiles/tiflash_proxy.dir/all' failed
make[2]: *** [contrib/tiflash-proxy-cmake/CMakeFiles/tiflash_proxy.dir/all] Error 2
CMakeFiles/Makefile2:9107: recipe for target 'dbms/src/Server/CMakeFiles/tiflash.dir/rule' failed
make[1]: *** [dbms/src/Server/CMakeFiles/tiflash.dir/rule] Error 2
Makefile:2647: recipe for target 'tiflash' failed
......

Finally, I found that the above problem occurs only when cmake is set to -DCMAKE_BUILD_TYPE=DEBUG, while it compiles successfully if nothing is set.
However, it is still certain that tiflash-proxy has some problems, if I set the debug parameter for cmake, clang does not compile successfully, but if I change the tiflash-proxy compiler to g++, it does.  So the current makefile for compiling tiflash-proxy with clang (llvm) should be missing some parameter settings.


#645 Paste image from clipboard

Thank you for your contribution to the KodaDot NFT gallery.
Let's do a quick check before the merge.
Your contribution will be an eternal part of the community codebase
Before submitting this PR, please make sure:

[x] Code builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I didn't break any original functionality

Optional

[ ] I've tested it on mobile and everything works

PR type

[ ] Bugfix
[ ] Feature
[x] Refactoring

What's new? (maybe part of changelog)

closes #645
Have added an event listener for paste, fetched the required info and sent it to the common function

If issue contains bounty label ($,$$,$$$)

Your KSM address: EzGc4s9PgCPx1YnF3fqzhLzVHpHMTL4LWPScwpDrR8JKgSU

Community participation

[x] Are you at KodaDot Discord? If no, let's join us - https://discord.gg/35hzy2dXXh

Screenshot

[ ] Whenever your fix will change something on UI, a screenshot is more than welcome

@prachi00 can you take a peak at this so we can resolve it and merge it?:)

@yangwao Have done all the mentioned changes

Oh, I miss that we support that functionality. Others really don't know that exists.
I guess adding text Paste from Clipboard is crucial here, I'll merge it for now but looking for a new PR @prachi00 :)


New Port: Biolab Disaster

New Port for Biolab Disaster
Game Information

Title: Biolab Disaster
URL: https://github.com/scemino/z_biolab

Notes
Thanks to Dominic Szablewski for creating this awesome game and game engine.
Controls

Button
Action

DPAD
Move

A
Shoot

B
Jump

Submission Requirements
CFW Tests
Ensure your game has been tested on all major CFWs:

[x] AmberELEC
[x] ArkOS
[x] ROCKNIX
[x] muOS

Resolution Tests
Test all major resolutions:

[ ] 480x320
[x] 640x480
[ ] 720x720 (RGB30)
[ ] Higher resolutions (e.g., 1280x720)

Only 640x480 has been validated ?

Only 640x480 has been validated ?
Looking at the testing thread I suppose more resolutions are ok. But you know more than me :)

Sorry I validated 480x320 with my RG351M.

✅ 353VS / OGA Arkos, Ace Rocknix and 35xxSP muOS. The game seems to run fullscreen no matter what, so its a bit stretched on 4:3 and I bet it will look hillarious on 1:1 screens, but plays fine nonetheless

I think you can add high resolution (Ace test) 🙃

Portfiles are not in the correct structure, please reference this:
* portname/

  * port.json
  * README.md
  * screenshot.jpg
  * gameinfo.xml
  * cover.jpg (Optional)
  * Port Name.sh
  * portname/

    *
    * LICENSE Files

What file is not in the correct place? I don't see my error.

Seems like the instructions on the website arent correct. Your screenshot and cover look good now. About the structure, you have the cover/screenshot/xml/json all in the portfolder. They should be one folder higher, next to the README and script.

Seems like the instructions on the website arent correct. Your screenshot and cover look good now. About the structure, you have the cover/screenshot/xml/json all in the portfolder. They should be one folder higher, next to the README and script.

This seems all good no? They are next to the README file.

Interesting, i have a different view on my end.

Interesting, i have a different view on my end.

This is the same thing, the biolab folder is not expanded, so the files are not in the biolab subfolder.


[Feature] SQL Agent job details default date to DESC order

Version: 1.6.0-insider (user setup)
Commit: 504d5c91bc9d1781ea9f9eb6c245d43475941fef
Date: 2019-04-04T07:36:57.980Z
VS Code 1.31.1
Electron: 3.1.6
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.14393

Steps to Reproduce:

Open ADS
Connect to server
View SQL Agent job
Click on the job for history details
Dates are shown in ASC order, so it requires scrolling all the way down to find the latest execution.

SSMS defaults to job history in DESC order.

@wsmelton Thanks for the suggestion. Will have the suggested change in the coming release of the Agent extension.


feat: ability to download the list of received events

Add the ability to download the list of received events with their payloads.

Implemented in v3.9


Ability to specify the timezone for the fscalendar.

Exposed the timezone property of FSCalendar.
Today is based on the timezone.

When this will be pushed to master?

@WenchaoD would you consider merging this in?

Thanks


JSON Formatter

Hi, great package! It would be handy if there was also a formatter which creates a JSON response out of the error message. I could imagine a response like:
{
    "success": false,
    "class": "\RuntimeExcetion",
    "message": "",
    "code": 0,
    "trace": [...],
    "request": [...],
    "server": [...]
}

There is also RFC 7807 which describes such a format. This would simplify to use this handler also for API cases.

I really like this! Will definitely look into it soon. Thanks!

@chriskapp Hey Christoph, have a look at #6 and let me know what you think before I merge, please :)

Merged and published in 1.1.0.

@victorstanciu, thanks, the PR looks really good, I will try to use the renderer in my project.


Do we need to close request.sessions underlying each instance of Airtable()?

is there a way to manage/close underlying request.sessions for each instance of Airtable?  or is that done automatically behind the scene?
If Im using Airtable() as part of an API backend which (running FastAPI) is used to connect front-end to AirTable -- do I need to do anything to clean up sessions that are opened with each new instance of Airtable()?

You can manipulate the session object directly however there is no persitent connection.
Session object from requests is just used to set all request params and authentication in one place
https://github.com/gtalarico/airtable-python-wrapper/blob/master/airtable/airtable.py#L135


Any lod count <7 breaks the cpu height system

You can see in main test scene if lod count <7 the boat doesnt rock back and forth its just stationary

It seems to work for me for 5 & 6.
For 4 the available lods are quite small and the boat won't move due to the minObjectWidth setting in the animated wave settings, which is an optimisation that stops readback of small textures. By default its set to 3 which is wider than the row boat. If I set this to 0 or reduce it then i get the floating behaviour back.

Closing for now, let me know if it should be reopened


Add the resolver to the DOI which is copied to the clipboard

Is your enhancement request related to a problem? Please describe.
There is the nice feature to copy the DOI to the clipboard for projects, samples and spectra see:

This puts 10.57992/nmrxiv.p2.s273 to the clipboard
Describe the solution you'd like
I would like to have a resolver added to that DOI e.g. https://doi.org/ or https://dx.doi.org/ as I want to use this to reference a page I want to send to someone and this should be hyperlink. I agree that there are two solutions to this a) I add this manually b) I leave this to the person I send the DOI.
Describe alternatives you've considered
If the DOI is there for reasons but not the DOI including a resolve, another button next by should copy the DOI including a resolver to the clipboard.
Best,
Tillmann

I can confirm that the resolver is added now.

Thanks @tilfischer this issue can be closed now.


Make empty list visible on drag?

I understand that the current workaround to being able to drop into an empty list is to set a minimum height/width to that lists container.
However, is it possible to make it so that the empty list is only "displayed" (min height/width set) when you're dragging an element over top of where that list should be? Essentially the min height and width would only be set when you drag an item over that area.
I ask because if you want a nice, tight list hierarchy the UI doesn't look all that great with the empty lists being shown.

@gdalam - maybe you could add mouseover/mouseout events that add a class that contains the min-height. Perhaps something like: http://jsfiddle.net/rniemeyer/zLusyfcu/ . The fiddle includes a sample hoverClass binding to simplify adding/removing a class. I don't see anything else built into jQuery UI sortable to aid in this scenario. Hope this helps!


What is a reasonable training time on kitti dataset?

Hi, thank you for sharing the implementation and I have been testing the script as well as referencing the paper to have a better understanding of the entire implementations. Just wondering what is a reasonable expected training time on the 80G Kitti dataset? So far on my 64G memory machine with NVIDIA 2060 GPU, it takes around 25 minutes per epoch. I have tried with different batch_size but it didn't seem to improve the training time either.
I understand to train with 50G worth of data would not be any where quick but 25 minutes per epoch for 10000 epochs seem to be really out of the normality.
Thanks for your help!

Hi @g1y5x3 ,
It is hard to say what the right duration is, as it highly depends on the chosen learning-rate.
But for sure I never trained it for more than 200 epochs (you will also see that the loss kinda converges after that amount of time).
Best,

Thank you for your clarification. So the 10,000 epochs that was in deployer/trainer.py was just a generic configuration but not necessarily the one used for reproducing the results right?

Actually, would you mind share the training parameters that you used in the paper?

yeah, usually I deployed it on servers and wanted to run it as long as possible rather than being killed by an internal epoch limit.

Which parameters do you mean? model parameters (i.e. weights)?

There is an example checkpoint for kitti provided here: https://github.com/leggedrobotics/DeLORA/tree/main/checkpoints


Interface MTU

Issue Type

[ ] Bug
[x] Feature Request
[ ] Question

Problem and possible solution
In linux OSs interface MTUs are in /sys/class/net/$iface/mtu.
Can you please add this to networkInterfaces?

Thank you Alexis! Makes sense! Will add this.

Should work now. Version 4.0.0 published


fix: renovate config for development

:tada: This PR is included in version 1.0.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Is it possible to merge more than teo kubeconfigs?

I have 4 clusters

For now : it is not possible.
I'm planing to update the program soon, i'll add the support of many kubeconfig files.


Add a ping checker

Similar to this, but not necessarily tied to SQL.

@sagikazarmark can you elaborate?
Do you mean something like open a connection to a host, and fail on error?
Add an option to send payload, and expect a response?
Support UDP?

Here is a simple interface that (among others) the sql package implements:
type Pinger interface {
	PingContext(ctx context.Context) error
}

I'm suggesting to add a checker to this package, which accepts an instance of that interface, so that people don't have to implement a custom one. It has no external dependencies, so it's not a heavy implementation.

@sagikazarmark want to take a look?
https://github.com/AppsFlyer/go-sundheit/pull/6

Sure!


Compiler warnings when building highs with gcc 11.2.0.

Building the main branch of scipy (using dev.py) with gcc 11.2.0 on a Linux machine results in over 130 compiler warnings when the highs code is compiled.  Note that I don't get these warnings if I build our copy of highs directly with cmake from a separate build directory, so perhaps the scipy build script is using different compiler flags than those used by the highs cmake configuration file.
Almost all of the warnings are repetitions of these:
[1442/1565] Compiling C++ object scipy/optimize/_highs/libhighs.a.p/.._..__lib_highs_src_ipm_IpxWrapper.cpp.o
In file included from ../scipy/_lib/highs/src/simplex/HEkk.h:22,
                 from ../scipy/_lib/highs/src/lp_data/HighsLpSolverObject.h:21,
                 from ../scipy/_lib/highs/src/lp_data/HighsSolution.h:25,
                 from ../scipy/_lib/highs/src/ipm/IpxWrapper.h:25,
                 from ../scipy/_lib/highs/src/ipm/IpxWrapper.cpp:14:
../scipy/_lib/highs/src/util/HighsHash.h: In instantiation of ‘class HighsHashTable<long unsigned int>’:
../scipy/_lib/highs/src/simplex/HEkk.h:140:28:   required from here
../scipy/_lib/highs/src/util/HighsHash.h:816:41: warning: ‘this’ pointer is null [-Wnonnull]
  816 |       reinterpret_cast<Entry*>(0)->value())>::type;
      |       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
../scipy/_lib/highs/src/util/HighsHash.h:791:12: note: in a call to non-static member function ‘const T& HighsHashTableEntry<T, void>::value() const [with T = long unsigned int]’
  791 |   const T& value() const { return value_; }
      |            ^~~~~

[1467/1565] Compiling C++ object scipy/optimize/_highs/libhighs.a.p/.._..__lib_highs_src_mip_HighsDebugSol.cpp.o
In file included from ../scipy/_lib/highs/src/mip/HighsCliqueTable.h:21,
                 from ../scipy/_lib/highs/src/mip/HighsDebugSol.h:26,
                 from ../scipy/_lib/highs/src/mip/HighsDebugSol.cpp:13:
../scipy/_lib/highs/src/util/HighsHash.h: In instantiation of ‘class HighsHashTable<std::pair<HighsCliqueTable::CliqueVar, HighsCliqueTable::CliqueVar>, int>’:
../scipy/_lib/highs/src/mip/HighsCliqueTable.h:103:61:   required from here
../scipy/_lib/highs/src/util/HighsHash.h:816:41: warning: ‘this’ pointer is null [-Wnonnull]
  816 |       reinterpret_cast<Entry*>(0)->value())>::type;
      |       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
../scipy/_lib/highs/src/util/HighsHash.h:778:6: note: in a call to non-static member function ‘V& HighsHashTableEntry<K, V>::value() [with K = std::pair<HighsCliqueTable::CliqueVar, HighsCliqueTable::CliqueVar>; V = int]’
  778 |   V& value() { return value_; }
      |      ^~~~~

Another warning that shows up:
../scipy/_lib/highs/src/mip/HighsImplications.cpp: In member function ‘bool HighsImplications::runProbing(HighsInt, HighsInt&)’:
../scipy/_lib/highs/src/mip/HighsImplications.cpp:167:46: warning: ‘implicsdown’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  167 |       if (implicsup[u].column < implicsdown[d].column)
      |                                              ^

I don't know whether or not these warnings indicate a serious issue.  Even if they don't, the volume of compiler warnings makes it difficult to notice other warnings that might be true problems.

For reference:

https://github.com/rgommers/scipy/issues/120 was an issue about fixing compiler warnings in the meson build.
In https://github.com/ERGO-Code/HiGHS/pull/681, @Smit-create made a PR to HiGHS to fix one of the warnings, but the HiGHS devs didn't accept it because the warning only occurs with old versions of gcc.  (I haven't looked into why I get this warning with gcc 11.2.0.)

Yes, we should fix these upstream. I think they're harmless, but they're annoying nevertheless.

Thanks @rgommers.  I don't see the warnings now, so I'll close the issue.


Moved constructed nightly to the right spot.

Thanks!

Your are welcome and thanks for merging. Hope I don't send too much PRs. :speak_no_evil:

The more the better :pizza:


[ZEPPELIN-1144]Zeppelin home page should only list notebooks with read or write permission

What is this PR for?
If logged in user does not have Read and Write permission for a notebook, user should not see the notebook in the zeppelin home page.
What type of PR is it?
Bug Fix
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1144
How should this be tested?

unit test
online test

Screenshots (if appropriate)
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

CI error
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 76.102 sec - in org.apache.zeppelin.integration.SparkParagraphIT

Results :

Tests in error:
  ZeppelinIT.testAngularDisplay:118->AbstractZeppelinIT.waitForParagraph:70->AbstractZeppelinIT.pollingWait:96 쨩 Timeout
  AuthenticationIT.testGroupPermission:179->AbstractZeppelinIT.pollingWait:96 쨩 Timeout

Tests run: 15, Failures: 0, Errors: 2, Skipped: 0

Thank you for fixing it @mwkang
CI failure looks somehow relevant, you can not reproduce it locally?

@bzz I will check it. Thanks for review.
When I fix CI, I'll mention you.

@bzz https://github.com/apache/zeppelin/pull/1330#issuecomment-239997747 It did not reproduce in local. Can you check this?

@mwkang could you please try close and then re-open this PR to trigger the CI and see if the same issue persists? Thanks!
\cc @khalidhuseynov for review.

@mwkang thanks for the contribution! actually this is related to multi-user note management and there's some work going on under #1392 or #1390 .
In your case, i tested it, and there's a bit tricky moment with notes reloading and broadcasting. For example if you login as user1 with noteA and user2 with noteB and initially everything looks as expected, and then you reload your notes as shown below with refresh icon:

then if you refresh for user1 then noteA will appear in the list of all users, including user2 in our case. Same would happen if you reload for user2, meaning noteB will appear in the list of user1.

If the current single notebook-authorization.json  is managed by user (with multiple files, and no more with a single one), how would you handle it?

@mwkang could you re-trigger CI?

I think it doesn't solve all problems, but definitely in the roadmap. LGTM

Thank you @mwkang @khalidhuseynov !
Will merge to master, if there is no further discussion.

@mwkang @khalidhuseynov Could you handle or at least create issues for expected problems after merge this PR? I can at least think following two issues,

Note list broadcasting message is not aware user session, so notebook list can display notes that supposed to not see when other user refresh the notebook list.
When a user changes the permission of a note, note list need to be broadcasted immediately so all connected users can have updated list of notebook. Otherwise, there's still some chances that a user able to see some note on notebook list that they can not access.

@Leemoonsoo issues are created under ZEPPELIN-1437 and ZEPPELIN-1438 correspondingly

Thanks, Merge to master if there're no further discussions.

I'm sorry for no-reply long time. my plate was full.
I will finish other 2 PRs and then If ZEPPELIN-1437 and ZEPPELIN-1438 are status is unresolved, I will
try to resolve.
If is it okay..


Make the build a composite build

GitHub Actions is having problems. I suppose that's why the build action isn't running here. I don't want to merge this until that's resolved.

Fixes #32.


egui integration

Summary:
The egui UI framework is an extremely ergonomic one, allowing to easily build complex UIs for almost any purpose. I've been trying to integrate it myself into our game, unfortunately seems like it is a tad above my skill level. You can look at my current attempt here: https://github.com/Elinvynia/Enfaria/blob/tetra/enfaria-game/src/egui.rs which produces results looking like this:

Why is this needed?
This change doesn't necessarily need to happen in tetra, but could be a separate crate in case users wanted to use something different for handling the UI. I'm happy with any of my code being used for that purpose. But it would certainly make building slightly more complex 2D games much easier and more fun.

I don't think I'd want to have third-party UI crates integrated into Tetra directly (there's a lot of them, and everyone has a different favourite one 😄), but making it possible to use them together is definitely something I care about. This was one of the main reasons I added the Mesh API (and why some of the changes to that API were made in 0.6).
Specifically with regards to egui, I've been meaning to play around with that library for a while! If I get it working with Tetra, I will definitely publish an example at the very least - not sure yet if I want to maintain a full crate for it myself, we'll see 😅 I also have some imgui code I need to revisit.
I think for your example, the main bit you're missing is uploading the egui texture data and then binding that to your Tetra mesh - that's where it gets all the fonts and stuff from. egui gives you a Vec of the alpha values for each pixel of the texture, which you'd need to turn into a Vec of RGBA data. You can then upload that via Tetra's Texture::from_rgba constructor/Texture::replace_data method.
In the web backend, they call this function on every paint, if you need an example of the logic. It seems to basically boil down to:

Check if the texture actually needs updating by checking the version.
If it does, convert the alpha data into full pixel data.
Update the texture on the GPU with the new data.

Wow, I can't even begin to describe how helpful that was! I've managed to fix it now (though extremely inefficiently, just to get it running 😅 ) but it works regardless!
I guess something like a semi-official example would be nice to save time for people looking to do this in the future?

Example integration here: https://github.com/Elinvynia/tetra-egui

Thank you! I will take a look at the code when I get a sec and see if there's any pain points/missing stuff in Tetra :)


Upgrade to c2pa-rs 0.26.0

(Fixes nightly build issues with openssl.)

Codecov Report
Patch and project coverage have no change.

Comparison is base (c81b590) 83.91% compared to head (385d7cc) 83.91%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #143   +/-   ##
=======================================
  Coverage   83.91%   83.91%
=======================================
  Files           3        3
  Lines         373      373
=======================================
  Hits          313      313
  Misses         60       60

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


Fix StateMachine transition Sync not working on looped animations

Fixes https://github.com/godotengine/godot/issues/79505
Also properly shows the statemachine node progress on looped animations (originally static and constantly maxed out)

You have spaces instead of tabs for indentation, you will need to fix this

Superseded by #87171. Thanks for the contribution!


Deploy botbuilder-config instead of botbuilder-luis

Pull Request Test Coverage Report for Build 973

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 66.606%

Totals

Change from base Build 963:
0.04%

Covered Lines:
2239

Relevant Lines:
3199

💛  - Coveralls


Our reformatting of gtf may render the gene annotation for isoform not usable.

One of the crucial step for GTEX's sqtl pipeline to work is to generate the phenotype_group table, which needs an exon list, which in turns was generated by following functions  qtl.Annotation.annotation()
When applying this function on our gtf, no errors were thrown, but the output object is empty.
annot = qtl.annotation.Annotation('/mnt/vast/hpc/csg/snuc_pseudo_bulk/sqtl/reference_data/genes.reformatted.gtf')

The exact cause is still under investigation.

For some reason, rerunning it makes it work...


Semaphore 2.0 hangs in workflow with nested promotions

https://immersivelabs.semaphoreci.com/workflows/437fc128-2d7b-4be2-be14-3b769fac1cf4
Opening this workflow, the pipeline status never gets out of "Fetching...". Attempting to open the first promotion brings up the loading box (looks like the top left corner of a grey box) and the words "Loading pipeline...". It never gets past this state, despite the fact the promotion did in fact succeed

This is going to sound crazy, but through trial and error I was able to track this down to the promotion being named "Staging deploy"
Changing the name to "Staging deploys", "Stagingg deploy" or "staging deploy" fixes the problem. Is "Staging deploy" somehow a reserved promotion name?


Make default keybindings same as JTree

Two things here:

keys should be the same
behaviour should be the same

Easy to correct the keybindings for expand/collapse/toggle.
JTree also may have other behaviour (such as collapse on a leaf node moves to parent; have to check behaviour).

JTree navigation behaviour

left arrow - collapses current node; if already collapsed moves to parent node.
right arrow - expands current node; if already expanded moves to first child, but pressing again does not move to next child.
up arrow - move up in the tree, but doesn't affect expand or collapse.
down arrow - move down in the tree, but doesn't affect expand or collapse.
home - move to first node
end - move to last node
page down - move down a page
page up - move up a page

apparently also Enter toggles node expansion, but that doesn't seem to work in the JTree I'm using to test with.

Differences to TreeTable
TreeTable currently defines:

expands node

collapses node

space - toggles expansion collapse of node.

In addition, the underlying Jtable provides behaviour for:

up arrow - move up in the tree, but doesn't affect expand or collapse.
down arrow - move down in the tree, but doesn't affect expand or collapse.
home - move to first node
end - move to last node
page down - move down a page
page up - move up a page

So to change this, we need to:

redefine collapse to left-arrow, and implement move to parent logic.
redefine expand to right-arrow and implement move to first child logic.

Unclear whether we should replace space by Enter.  Think I prefer space for toggling, so I will leave that as it is, given Enter doesn't seem to work in JTree anyway.

Not sure I totally love the JTree behaviour.  It conflates tree navigation with expand/collapse.
Would also be nice to just be able to move up or down the parent-child structure without affecting whether they are expanded or collapsed.
In that world, there would be separate keys for expand collapse vs. navigate up/down tree structure.  If you set the same key for expand as "go to children", and the same key for collapse as "go to parent", then we could reproduce the JTree logic maybe?

OK, I think I like having explicit expand/collapse keys (+/-).  We can separately implement navigation up/down the tree structure with left arrow / right arrow.
We can make whether navigation also expands or collapses during navigation configurable.  So we would have:

left arrow - move to Parent node.  If "collapse children" is true, then collapse current parent instead of moving to next parent (same as JTree behaviour), if that node is currently expanded.
right arrow - move to Child node.  If "expand children" is true, then expand the current node if it is not expanded, otherwise halt.

This allows to either navigate the current tree structure without affecting expansion, or to replicate the JTree behaviour.

Added code to do this.


chore: remove workaround logics in verify-examples-kind.sh

Signed-off-by: Chao Zhang<EMAIL_ADDRESS>
What type of PR is this?

/kind feature
/kind cleanup
What this PR does / why we need it:
Previously, we have some workaround logics in the hack/verify-examples-kind.sh, it modifies the image tag in config/webhook/admission_webhook.yaml, so that the latest commit of admission webhook can be used.
This PR removed the hack logic by using the customization feature in Kustomize, see https://kubernetes.io/docs/tasks/manage-kubernetes-objects/kustomization/#customizing for the details. We create the config/webhook/kustomization.yaml file temporarily (and remove it when cleanup), which replaces the webhook image tag from v0.5.0 (or others in the future) to latest.
Which issue(s) this PR fixes:

Fixes #1366
Does this PR introduce a user-facing change?:

NONE

/ok-to-test

Thanks @tokers!
/lgtm
/approve


Request to add Charger

My charger appears not to be supported and wondered if it could be added.  It goes by a few different names.  Mine is an EVSwitch http://evswitchstore.com.au/ however it is also known as the EV+ Caro https://www.en-plustech.com/product/caro-pro-series/
Also, do you plan to add the BYD Atto 3 to your vehicle list?
thanks
Chris

It says OCPP is supported.
Should work. Have you tried it?

@MrB-AU I have another branded en+ wallbox and it's working just fine. The firmware and setup screenshots all look identical.
Not sure about byd cars

Ok. So when setting up EVCC do I just choose the Generic OCCP charger as my option?  I have done this but get a load point error.
Do I need to manually edit anything in the yaml files?
Fully declaring my limited technical knowledge here. On a steep learning curve of entering/setting all this up through terminal on my Mac.


Wrong device in average_metrics function

🐛 Bug Report
Since average_metrics function is being called from backward that should be called from every gpu,  the devise that is being created here should be equal to the current rank, otherwise torch.distributed.all_reduce will be stucked forever.

We use Dora for all of our experiments which typically calls torch.cuda.set_device at the beginning of the training with the proper device. That allows to use 'cuda' everywhere after without worrying about the rank of the gpu.


Fix ubuntu build errors

This fixes a variety of build errors on Ubuntu (clang and gcc), although some would occur on other platforms.
The std::atomic_int32_t and std::atomic_unt64_t type aliases are optional and were not available for me, so I replaced with std::atomic<std::int32_t> and std::atomic<std::uint64_t> which are always portable.
I also fixed several unused variable warnings because of code that was only used in debug mode, and
Added a magic // fall through comment that is the most portable way to tell compilers to be okay with
a switch case fallthough.
I also added a default virtual destructor to EngineFactoryBase (virtual ~EngineFactoryBase() = default;) which, it should have anyway, to prevent a warning with my system and compiler mix.
I've tried to keep the impact of these changes as low as possible. I should also mention that I have only been able to compile the OpenGL backend so far, (I've had to make my own bazel BUILD file, so it has been some effort).
Let me know if you think these changes don't belong upstream, I can always just maintain an internal fork if needed.
Also, at HLSL2GLSLConverterImpl.cpp:1774 I fixed a -WParentheses warning, but I'm not sure I interpreted your intent correctly.

What is your build set up that produces all the these issues? We build the engine in 23 different configurations with werror and all run clean. On Ubuntu in particular we build with both clang and gcc.
There are some variables that are used in debug checks and are unused in release builds. There is a lot of them, so the unused variable warning is disabled in release, but not in debug build.

What is your build set up that produces all the these issues? We build the engine in 23 different configurations with werror and all run clean. On Ubuntu in particular we build with both clang and gcc.
There are some variables that are used in debug checks and are unused in release builds. There is a lot of them, so the unused variable warning is disabled in release, but not in debug build.

I understand if you would rather reject this change, since my situation is a little odd.
I've been porting this project to the bazel build system for use at my job (Zoox), while I've tried to keep things as close to your cmake setup as possible, I don't yet have anything to use different flags for different types(release/debug) of builds (for various reasons that's quite annoying to do in my situation). Some other issues are just compiler and C++ std version related though, for example std::atomic_int32_t is not available for me.
If you take nothing else from this patch the std::atomic_int32_t change would be the best since I can't fix that by disabling more compiler diagnostics, and I think that is the most likely to affect other people.

I've been porting this project to the bazel build system for use at my job (Zoox)

This may work for OpenGL backend, which is the simplest one and does not have many source files or third-party dependencies. When you get to Vulkan though, things may get really complicated.
Can you build the shared libs using CMake and then use them in Bazel?

I don't yet have anything to use different flags for different types(release/debug) of builds (for various reasons that's quite annoying to do in my situation)

This may be quite an inconvenience. In debug/development Diligent performs a lot of validation and catches virtually all API usage errors printing error messages. In release build all these checks are disabled for performance reasons and invalid API usage will most likely result in a black screen or app crash.

If you take nothing else from this patch the std::atomic_int32_t change would be the best since I can't fix that by disabling more compiler diagnostics, and I think that is the most likely to affect other people.

Yes, I think fixing atomics makes sense. I can do this since this should be done consistently for all backends, not only OpenGL. For unused variable warning, you should disable it in the compiler setting. Wrapping all debug checks in #if DILIGENT_DEBUG only clubbers the code without any benefit other than silencing the warning.

I fixed atomics and a couple of other issues. Unused variables should be fixed using the compiler flags.

Seems like you have fixed the parts that make sense.


Improve class UserId

Here's what we'd like to see there:

method equals()
method hashCode()
getters for name, email, comment
proper formatting if name and comment are empty or null
better builder

will post a PR

0.2.0-alpha8 has been released :)

Thank you!


[vcpkg-tool] fetch downloads 7-zip, ignores VCPKG_FORCE_SYSTEM_BINARIES

Describe the bug
vcpkg fetch 7zip downloads 7-zip even if the system version is sufficient, and even if 'VCPKG_FORCE_SYSTEM_BINARIES' is set.
Environment

OS: Windows 10

PowerShell Core 7.2.2

Additional information (all executed from the same PoSH session):
% $env:VCPKG_FORCE_SYSTEM_BINARIES

1

% 7z | grep 7-Z

7-Zip 21.07 (x64) : Copyright (c) 1999-2021 Igor Pavlov : 2021-12-26

% Set-Location C:\Dev\vcpkg\root.git && git rev-parse HEAD

6704f4c7bb1c01eb901e6783198700e71890b29a

% .\vcpkg.exe --version

vcpkg package management program version 2022-07-21-a0e87e227afb536c62188c11ad029954f28fdb22

To Reproduce
Steps to reproduce the behavior:

Bootstrap vcpkg .
Execute ./vcpkg.exe fetch 7zip

A suitable version of 7zip was not found (required v21.7.0). Downloading portable 7zip v21.7.0...
Downloading 7zip...
https://www.7-zip.org/a/7z2107-extra.7z -> \downloads\7z2107-extra.7z
Extracting 7zip...
\downloads\tools\7zip-21.07-windows\7za.exe

Expected behavior
vcpkg fetch 7zip finds the installed version of 7-zip.
Failure logs

N/A

Additional context
ToolCacheImpl  (src/vcpkg/tools.cpp) has no way of finding tools defined in vcpkgTools.xml unless there is a corresponding implementation of ToolProvider.

Is there any update or branch for this issue?

I have some DNS issues when downloading 7zip, which can be related to my provider restrictions.
This host doesn't work for me and fails with ERR_NAME_NOT_RESOLVED.
error: https://www.7-zip.org/a/7z2301-extra.7z: WinHttpSendRequest failed with exit code 12007
Nevertheless, I have the latest 7zip installed 23.1 in my build pipeline, but it still fails. How to override the 7zip URL or binary location? I tried to set VCPKG_FORCE_SYSTEM_BINARIES and it doesn't change anything

Any ideas on how to fix this?

@demianmnave-pti
according to the used env var
VCPKG_FORCE_SYSTEM_BINARIES
it is not designed to support all the binaries in vcpkg, but just 2:
https://learn.microsoft.com/en-us/vcpkg/users/config-environment#vcpkg_force_system_binaries

@greenozon According to the implementation, it is not intented to be limited. There are really many more tools.

interesting! then we have mis-alignment between official doc portal and the tool itself
are there any steps to update official MS doc info for this env var?

Found the list of supported system tools, it does not have 7z in it...
https://github.com/microsoft/vcpkg-tool/blob/4063a94150c5d691a4fb39353bd9d48dc940b22b/src/vcpkg/tools.cpp#L861
means this env var does not cover 7z, right?

are there any steps to update official MS doc info for this env var?

Read the notes in the docs footer, or go to https://github.com/microsoft/vcpkg-docs/

Found the list of supported system tools, it does not have 7z in it...
https://github.com/microsoft/vcpkg-tool/blob/4063a94150c5d691a4fb39353bd9d48dc940b22b/src/vcpkg/tools.cpp#L861
means this env var does not cover 7z, right?

You are jumping into conclusions. I deliberately omitted the reference to the code because I was afraid of that.

Since the latest vcpkg update it no longer downloads one 7-zip package, but two of its packages:
vcpkg package management program version 2024-09-18-5f5da308dd1dd3cc49641c60eafff176e248a9dd
[...]
Additional packages (*) will be modified to complete this operation.
A suitable version of 7zip was not found (required v24.8.0).
Downloading https://github.com/ip7z/7zip/releases/download/24.08/7z2408-extra.7z
Extracting 7zip...
A suitable version of 7zr was not found (required v24.8.0).
Downloading https://github.com/ip7z/7zip/releases/download/24.08/7zr.exe

https://github.com/curl/curl/actions/runs/11163546704/job/31030787352#step:5:34
(This is when all packages are coming from the cache.)

The issue has been fixed in the latest version of vcpkg, and the test results are normal.


solid-ui share pane broken in mashlib 1.0.6

NSS 5.2.1 with mashlib 1.0.6
solid-ui share pane is broken : tried on https://bourgeoa.inrupt.net/public/
acl-control.js:101 Uncaught (in promise) TypeError: Cannot read property 'http://www.w3.org/ns/auth/acl#Append http://www.w3.org/ns/auth/acl#Control http://www.w3.org/ns/auth/acl#Read http://www.w3.org/ns/auth/acl#Write' of undefined     at b (acl-control.js:101)     at acl-control.js:1128     at acl.js:423     at acl.js:453     at fetcher.js:938
At first it seems that acl-control.js is not in line between solid/solid-ui and mashlib/node_modules

Two duplicates:

https://github.com/solid/solid-panes/issues/193
https://github.com/solid/solid-panes/issues/194

The issue is probably in solid-ui, see https://github.com/solid/solid-panes/issues/193#issuecomment-549777549 for more analysis.

closed with https://github.com/solid/solid-ui/pull/153
integrated in mashlib 1.1.7


Build env

Merged Build_env to master. Encountered no conflicts.

This should have likely not been merged in since there are still several pending issues in it:

IntelliJ metadata still committed to project
Generated bundle still committed to project
Looks like a good number of the files are from a generated JSDoc report and should be ignored
Bootstrap CSS is still present in the project


Feature request

Hi Mike, do you accept a feature request?
I'm searching for a way to clear the output window with a shortcut.

Hi @derricck, I think this is only possible through a custom command. Here's how I did in Visual Studio 2019:

Go to Tools > External Tools....
In the External Tools dialog, click Add.
Set the Title to something like Clear Output.
Set the Command to cmd.exe.
Set the Arguments to /c cls.
Optionally, set the Initial directory to $(SolutionDir).
Check 'Use Output Window' box.
Click OK to save.
Assign a Keyboard Shortcut:
Then,
Go to Tools > Options.
In the Options dialog, navigate to Environment > Keyboard.
In the Show commands containing: field, type Tools.ExternalCommand followed by the number corresponding to the position of your newly created external tool (e.g., Tools.ExternalCommand1 for the first external tool).
In the Press shortcut keys: field, press the keyboard shortcut you want to use (e.g., Ctrl+Shift+L).
Click Assign, then OK to save.

Let me know should you need further assistance.


PageSpeed Badge

Seems there could be a badge for this:
Web Test
https://developers.google.com/speed/pagespeed/insights/?utm_source=analytics&tab=desktop&url=https%3A%2F%2Fbadgen.now.sh%2F
API Explorer
https://developers.google.com/apis-explorer/#p/pagespeedonline/v4/pagespeedonline.pagespeedapi.runpagespeed?url=https%253A%252F%252Fbadgen.net&_h=1&
API Endpoint
https://www.googleapis.com/pagespeedonline/v4/runPagespeed?url=https%3A%2F%2Fbadgen.net

While, it's not fun to put URLEncoded url (https://example.com) in url (https://badgen.net/pagespeed/...). Close this for now.


fix: update hostname docs

Description
Title

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.56.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add support for passing Target Capabilities in QIR generation and job submission

The first commit fixes the issue when submitting a QIR job and when the targetCapability parameter was not being sent to the Azure Quantum service.
The second commit fixes a limitation when generating QIR from a Q# callable, in which it was not possible to pass the target or the target capability.
It also introduces a _repr_qir_() method to the Q# callable, which returns the QIR bitcode (instead of returning the QIR text as in the as_qir() method does).
The _repr_qir_() will be used by the azure-quantum Python package to submit Q# jobs.
The last 2 commits add some telemetry events for QirMagic and EntryPointSubmission.

@vxfield based on the spec review we had earlier in the day, I think the changes that introduce _repr_qir_() should not be part of this PR. Since introducing this new function would not result in new functionality, I think it would be better to wait to introduce it (since how we generate QIR for Q# might change considerably). Additionally, if it is introduced, it is significant enough to be its own PR.

/azp run

/azo run

/azp run


Mixpanel initialization issue

With npm run dev, I get the following error in the JS console; after which no more API requests are made.
VM171 common.018977bc92544fb071c0.js:4900 Uncaught TypeError: Cannot read property 'register' of undefined
    at MixpanelLib.webpackJsonp../node_modules/mixpanel-browser/build/mixpanel.cjs.js.MixpanelLib.register (VM171 common.018977bc92544fb071c0.js:4900)
    at mp (VM172 app.b8e2e3f89a27fd54c815.js:18003)
    at Function.Vue.use (VM171 common.018977bc92544fb071c0.js:15686)
    at Object../plugins/mixpanel.js (VM172 app.b8e2e3f89a27fd54c815.js:18016)
    at __webpack_require__ (VM168 manifest.19169956f61be47e1409.js:711)
    at fn (VM168 manifest.19169956f61be47e1409.js:116)
    at Object../.nuxt/index.js (VM172 app.b8e2e3f89a27fd54c815.js:1127)
    at __webpack_require__ (VM168 manifest.19169956f61be47e1409.js:711)
    at fn (VM168 manifest.19169956f61be47e1409.js:116)
    at Object../.nuxt/client.js (VM172 app.b8e2e3f89a27fd54c815.js:72)

Added default Mixpanel key to make the register property work. The env var still needs to be set in in nuxt.config.js for Mixpanel to work properly.


commit_version_bump settings doesn't read SRC_ROOT

New Issue Checklist

[X] Updated fastlane to the latest version
[X] I have read the Contribution Guidelines

Issue Description
When using the param settings: true in a fastfile, it does not appear to include ${SRC_ROOT} from Xcode in the file path, so fails to locate if this is not in the root of the git repo
See commit_version_bump.rb#L94
Complete output when running fastlane, including the stack trace and command used
Note: /Users/domster83/workprojects/TrilbyTV-iOS/TrilbyTVPlayer is my Xcode project root.
[16:55:32]: fastlane finished with errors

[!] Exit status of command 'git add /Users/domster83/workprojects/TrilbyTV-iOS/TrilbyTVPlayer/TrilbyTVPlayer.xcodeproj/project.pbxproj /Users/domster83/workprojects/TrilbyTV-iOS/TrilbyTVPlayer/TrilbyTVPlayer/TrilbyTVPlayer-Info.plist /Users/domster83/workprojects/TrilbyTV-iOS/TrilbyTVPlayer/TrilbyTVPlayer/TrilbyTVPlayer-Dev-Info.plist /Users/domster83/workprojects/TrilbyTV-iOS/Settings.bundle/Root.plist' was 128 instead of 0.
fatal: pathspec '/Users/domster83/workprojects/TrilbyTV-iOS/Settings.bundle/Root.plist' did not match any files

Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
Fastfile:
ommit_version_bump(
      message: "Deploying version #{friendly_version}.",
      xcodeproj: 'TrilbyTVPlayer.xcodeproj',
      force: true,
      settings: true
    )

Environment
fastlane version (run fastlane -v):
1.105.0
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)?
no
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)?
rbenv

@dombarnes Thanks for reaching out about this and sorry to hear that you are running into this. Which folder are you running fastlane from? Thanks!

In my case, I'm running it inside /Users/domster83/workprojects/TrilbyTV-iOS/TrilbyTVPlayer, which contains the entire Xcode project, as well as fastlane/
The root of my git repo is /Users/domster83/workprojects/TrilbyTV-iOS

@dombarnes Can you provide a sample project with a fastfile and repro steps? This seems reproducible and I want to send this out to the core team. Thanks!


Update rust.yml GitHub Action to handle sled_transaction_timeout_* te…

…sts separately
resolve #718
Detach sled_transaction_timeout_* tests and run that separately with only using a single thread.
sled_transaction_timeout_* tests often fails in GitHub Action when it runs test in parallel using multiple threads.
So, to make our rust.yml action more stable, update the action to handle that tests separately.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30739

Relevant Lines:
31400

💛  - Coveralls


Add travis CI

This will include linter and unit test check for each pull request. This is required as a new feature which gets added doesn't guarantee it will not break the project.

Closed with #36


Change repo name

I would like to make this repository available as a product that can be installed and can be built against. That might require a few changes to the Makefile and maybe a pkgconfig file. However, we would also need to rename the repository because its name would be visible without the mpiwg-abi context. What about mpi_abi_stubs?
Alternatively I could create a separate repository for this, but that seems overkill.

I expect it will move to the Forum GitHub and possible change name once it's voted into MPI 4.2.
I have no opinion on the project name and welcome suggestions.

@eschnett I'm working on it, look at this branch.

I would like to make this repository available as a product that can be installed and can be built against.

See #14.


[#1004] clean up compiler warnings

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

[x] Make sure there is a GitHub issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a GitHub 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 [#XXX] - Fixes bug in SessionManager,
where you replace #XXX with the appropriate GitHub issue. Best practice
is to use the GitHub 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.
[ ] add fixes #XXX if merging the PR should close a related issue.
[x] Run mvn verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] 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.
[x] Committers: Make sure a milestone is set on the PR

Trivial changes like typos do not require a GitHub 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
[ ] In any other case, please file an Apache Individual Contributor License Agreement.

I am still new to GitHub PRs. Most of my experience is with BitBucket. How do I assign a milestone, and how do I know which milestone to use?

That’s for committers. Don’t worry about that

@lprimak
Would I not be considered a committer?

Not yet :) You will be a Contributor.

Thank you for your contribution!

@lprimak Alright. So everything else applies?

@lprimak If you don't mind me asking, what is the procedure for responding to PR comments. Should I just make updates, commit, and push?

If you don't mind me asking, what is the procedure for responding to PR comments. Should I just make updates, commit, and push?

yes :)

Alright. So everything else applies?

Not sure what you mean there. The issue is to fix compiler warnings, so if you fix the compiler warning that applies then :)
Try not to do too much. I did see some class renaming. I am pretty sure that would be out of scope.

Try not to do too much. I did see some class renaming. I am pretty sure that would be out of scope.

Oh, I'll have to take a look at that.
By the way, would it be expected I resolve the issue or just make progress? I know some places prefer frequent pull requests while others prefer them to be basically when the entire issue is resolved.

I would say to air on the side of resolving the issue in full

Alright. I might retract this PR then. I'm not sure I want to hanging around while I work on that

I would just keep working on it. No harm there in draft

Good point. Would i have to keep squashing or could I just keep my future commits separate?

I would not worry about squashing until you are ready at the end

Alright. Thanks for taking the time to reply to me! I find it really encouraging.

No problem. Looking forward to your contributions!

Try not to do too much. I did see some class renaming. I am pretty sure that would be out of scope.

Could you point out where that is. I don't believe I've renamed any classes, but I'm sure I may be wrong.

Also, it seems that subclasses of HashedCredentialMatcher have been deprecated. I believe removing the implementation of those classes is out of the scope of this issue.

Just leave the questionable stuff for last

It seems like most errors are outdated documentation that includes deprecated items. I don't think I have sufficient knowledge of the code base to update the documentation. Maybe I could publish the PR without the documentation changes. Let me know what would be the best course of action.

I would concentrate on resolving unchecked casts and raw types, by making them actual generics or using @SuppressWarnings where appropriate

Thank you. That's clears up what direction I should take for this.

This effort is abandoned for now, Anyone willing please feel free to take it over.

Superseded by #1233


Rewrite of TabView using modern v2 capabilities

(this was originally in https://github.com/gui-cs/Terminal.Gui/pull/3808, but that got screwed up. So this is a new PR with just the changes).
TabView is a great control for testing the library. It has a lot of capabilities that stretch the library. The v1 version was internally very complex in order to work around limitations in TG. In theory, v2 should enable making this control a lot simpler.
This PR is a WIP/Prototype that I started to determine if we're on the right track with these v2 capabilities:

Content scrolling - The TabRowView can/should just be a horizontally scrolling View where each Tab is just a subview. All the complex TabView logic for renering the Tabs and supporting scrolling can go away.
Layout - Dim.Auto, deferred layout, etc... should reduce the need for a lot of the internal complexity. Especially the complex layout math scattered around.
Drawing - There should be no need for TabView to draw lines directly. Each element can just be a View and leverage Border an automatic line joins currently enabled by SuperViewRendersLineCanvas
Navigation - How a user navs with the keyboard across Tabs, TabRowView, and the tab content is complex. The new v2 Nav system should make this easier.

Depends on

#3798

In #3798 auto-line join across Views is currently broken. I'm pausing work on this PR until I fix that.
Fixes

Proposed Changes/Todos

[x] Prototype TabRowView that just uses content scrolling
[x] Remove ContentView
[ ] Remove all complex LineCanvas stuff and just rely on Border

Pull Request checklist:

[ ] I've named my PR in the form of "Fixes #issue. Terse description."
[ ] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[ ] My code follows the Terminal.Gui library design guidelines
[ ] I ran dotnet test before commit
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[ ] My changes generate no new warnings
[ ] I have checked my code and corrected any poor grammar or misspellings
[ ] I conducted basic QA to assure all features are working

That is how the PR #3828 does.

No, #3828 does not do what this PR does. Not even close. Please study this PR first.

The image is similar than I included in SetLayoutTests.cs unit test.

Ahhh, you didn't notice this:
https://gui-cs.github.io/Terminal.GuiV2Docs/api/Terminal.Gui.LineCanvas.Exclude.html#Terminal_Gui_LineCanvas_Exclude_Region_
In addition, auto-linejoin/SuperViewRendersLineCanvas is still not working quite right.
PLUS, Border is still doing custom LineCanvas work which makes it inflexible for this use: https://github.com/gui-cs/Terminal.Gui/issues/3407
If each side of the Border is a Line that can be separately configured we should be able to nail this easily.

But I'm still curious how you will accommodate a Tab.View property as TabView subview with border but with a limited height, as TextField has, for the case where the available TabView height is grater.


Solved ui overflow issues in donate page,recycle page and dashboard page

Resolved UI overflow issues. In donate page, dashboard page and recycle page.
Screencast from 2024-05-15 17-04-42.webm

Great job on resolving the UI overflow issues in the Donate, Dashboard, and Recycle pages! Your efforts have significantly improved the user experience and polished the project.
Just a friendly reminder, please make sure to star the repository and click on "Watch" to stay updated with the latest developments and changes in the project.
Keep up the fantastic work!

Great job on resolving the UI overflow issues in the Donate, Dashboard, and Recycle pages! Your efforts have significantly improved the user experience and polished the project.
Just a friendly reminder, please make sure to star the repository and click on "Watch" to stay updated with the latest developments and changes in the project.
Keep up the fantastic work!

Thank you mam

Do put your project changes made with some post on Linkedin as we are going
to give batches in the next week and it will also help in growing your
leaderboard score.
On Wed, May 15, 2024 at 5:55 PM Pra-kas @.***> wrote:

Great job on resolving the UI overflow issues in the Donate, Dashboard,
and Recycle pages! Your efforts have significantly improved the user
experience and polished the project.
Just a friendly reminder, please make sure to star the repository and
click on "Watch" to stay updated with the latest developments and changes
in the project.
Keep up the fantastic work!
Thank you mam
—
Reply to this email directly, view it on GitHub
https://github.com/sanika391/SpoonShare/pull/21#issuecomment-2112387198,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A34TW2ZGGY45ZNQBWF74SO3ZCNH3NAVCNFSM6AAAAABHX756YCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGM4DOMJZHA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Also u can tag GSSOC and the respective mentors on whose project you have
made changes
On Wed, May 15, 2024 at 5:57 PM Sanika Chavan @.***>
wrote:

Do put your project changes made with some post on Linkedin as we are
going to give batches in the next week and it will also help in growing
your leaderboard score.
On Wed, May 15, 2024 at 5:55 PM Pra-kas @.***> wrote:

Great job on resolving the UI overflow issues in the Donate, Dashboard,
and Recycle pages! Your efforts have significantly improved the user
experience and polished the project.
Just a friendly reminder, please make sure to star the repository and
click on "Watch" to stay updated with the latest developments and changes
in the project.
Keep up the fantastic work!
Thank you mam
—
Reply to this email directly, view it on GitHub
https://github.com/sanika391/SpoonShare/pull/21#issuecomment-2112387198,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A34TW2ZGGY45ZNQBWF74SO3ZCNH3NAVCNFSM6AAAAABHX756YCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGM4DOMJZHA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Do put your project changes made with some post on Linkedin as we are going to give batches in the next week and it will also help in growing your leaderboard score.
…
On Wed, May 15, 2024 at 5:55 PM Pra-kas @.> wrote: Great job on resolving the UI overflow issues in the Donate, Dashboard, and Recycle pages! Your efforts have significantly improved the user experience and polished the project. Just a friendly reminder, please make sure to star the repository and click on "Watch" to stay updated with the latest developments and changes in the project. Keep up the fantastic work! Thank you mam — Reply to this email directly, view it on GitHub <#21 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A34TW2ZGGY45ZNQBWF74SO3ZCNH3NAVCNFSM6AAAAABHX756YCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGM4DOMJZHA . You are receiving this because you modified the open/close state.Message ID: @.>

Ok mam

@sanika391  Mam this PR is not reflected in my performance. Kindly add label for this mam.

Please check it now.
On Thu, May 30, 2024, 9:16 PM Pra-kas @.***> wrote:

@sanika391 https://github.com/sanika391 Mam this PR is not reflected in
my performance. Kindly add label for this mam.
—
Reply to this email directly, view it on GitHub
https://github.com/sanika391/SpoonShare/pull/21#issuecomment-2140009307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A34TW2ZJ3RQBNUX4YK7AHD3ZE5CUZAVCNFSM6AAAAABHX756YCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBQGAYDSMZQG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Use image from Internet in firebase_ml_vision

how to be able to use images from the internet

@kendy29
Your question seems to be related to firebase plugin which has a separate repo for tracking : https://github.com/FirebaseExtended/flutterfire/issues.  Please consider to open this issue there along with detailed description of your question.


Fix SE with  non FQDN hosts

/cc @GregHanson

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

can you describe the issue, add a test, etc? No idea what this change does without context

@howardjohn FYI #13436

@howardjohn mind adding a label?

We should not depend on the hostname to get name/namespace, it is totally wrong.
For different registry, the hostname is different. And for serviceEntry, the hostname can be anything, like google, github, etc.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu, rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu,rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hzxuzhonghu could we have a test for this?
Is this only for non http, https or TLS case?   for example, this test can use wikipedia as hosts instead of the current "www.wikipedia.org".
https://github.com/istio/istio/blob/master/tests/e2e/tests/pilot/testdata/networking/v1alpha3/service-entry-tcp-wikipedia-cidr.yaml#L14

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
c34d385f6c999b3b6eecc47f2ab1d8ef5e2e1b1f
link
/test istio-integ-k8s-tests

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.

could we have a test for this?

Can do that.

Is this only for non http, https or TLS case?

No, it does not care about the protocol. It fits for all protocols.


CMake issues when building TFLite

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04.5 LTS
TensorFlow installed from (source or binary): source
TensorFlow version: 8a643858ce174b8b (master)
Bazel version (if compiling from source): CMake 3.18.2
GCC/Compiler version (if compiling from source): GCC 7.5.0 / LLVM 6.0

Describe the problem
I'm seeing various issues whilst trying to build libtensorflow-lite.a. I have a fix for the first but not sure what the remedy is for the others.
1] Missing dependency on sparsity required by lite/kernels/density.cc and lite/kernels/fully_connected.cc
This fixes
diff --git a/tensorflow/lite/CMakeLists.txt b/tensorflow/lite/CMakeLists.txt
index ac7e373f18..b731401fc1 100644
--- a/tensorflow/lite/CMakeLists.txt
+++ b/tensorflow/lite/CMakeLists.txt
@@ -264,6 +264,10 @@ populate_tflite_source_vars("kernels/internal/reference/integer_ops"
 populate_tflite_source_vars("kernels/internal/reference/sparse_ops"
   TFLITE_KERNEL_INTERNAL_REF_SPARSE_OPS_SRCS
 )
+populate_tflite_source_vars("tools/optimize/sparsity"
+  TFLITE_TOOLS_OPTIMIZE_SPARSITY_SRCS
+)
+

 # Common include directories
 include_directories(
@@ -294,6 +298,7 @@ add_library(tensorflow-lite
   ${TFLITE_KERNEL_SRCS}
   ${TFLITE_NNAPI_SRCS}
   ${TFLITE_SRCS}
+  ${TFLITE_TOOLS_OPTIMIZE_SPARSITY_SRCS}
 )
 target_link_libraries(tensorflow-lite
   PUBLIC

2] The built library libtensorflow-lite.a doesn't itself include all the dependencies it needs as build in [build dir]/_deps
3] Enabling TFLITE_ENABLE_XNNPACK gives the following error when built with GCC (build's OK with LLVM)
/home/gavin/CK-TOOLS/lib-tflite-src-static-gcc-7.5.0-v2.3.91-with.xnnpack-linux-64/build/xnnpack/src/f16-clamp/gen/neonfp16arith-x16.c: In function ‘xnn_f16_clamp_ukernel__neonfp16arith_x16’:
/home/gavin/CK-TOOLS/lib-tflite-src-static-gcc-7.5.0-v2.3.91-with.xnnpack-linux-64/build/xnnpack/src/f16-clamp/gen/neonfp16arith-x16.c:32:44: warning: passing argument 1 of ‘vld1q_dup_f16’ from incompatible pointer type [-Wincompatible-pointer-types]
   const float16x8_t vy_min = vld1q_dup_f16(&params->min);
                                            ^
In file included from /home/gavin/CK-TOOLS/lib-tflite-src-static-gcc-7.5.0-v2.3.91-with.xnnpack-linux-64/build/xnnpack/src/f16-clamp/gen/neonfp16arith-x16.c:12:0:
/usr/lib/gcc/aarch64-linux-gnu/7/include/arm_neon.h:17359:1: note: expected ‘const float16_t * {aka const __fp16 *}’ but argument is of type ‘const uint16_t * {aka const short unsigned int *}’
 vld1q_dup_f16 (const float16_t* __a)
 ^~~~~~~~~~~~~

4] Building TFLITE_ENABLE_XNNPACK with LLVM looks like it builds all the right things but doesn't affect performance.

Did you get a chance to refer Build TensorFlow Lite with CMake guide?

Hi @ymodak, yes as per the instructions. Bit tricky to read, but it's wrapped up in this script -
https://github.com/ctuning/ck-tensorflow/blob/master/package/lib-tflite-cmake/scripts.linux/install.sh

Could you make a PR on 1 & 2?
Regarding 4, you need to enable xxnpack delegate to get the performance improvement.
https://blog.tensorflow.org/2020/07/accelerating-tensorflow-lite-xnnpack-integration.html

CMake build should be fine now.
https://www.tensorflow.org/lite/guide/build_cmake


Integration test pipeline takes wrong branch for build artifacts

This integration test run was triggered manually with master branch as a source for the build artifacts. But when checking the (failed) run here, one can see that it didn't take the artifacts from a master build but rather from a forked master build which can be found here.
The Run ID where the artifacts were fetched from: https://github.com/keptn/keptn/actions/runs/1571087826
This is clearly not a build from our master which indicates to me that the resolution of the last successful master build in the pipeline step here has a bug that is probably related to forked CI pipeline builds that have a PR against keptn/keptn with master branch as source.

@odubajDT did this occur again since we created the issue?

Closing this after discussion with @odubajDT since it's not reproducible


option: targetLang + analysisLang

The targetLang option should allow the user to pass an IETF language code to use as the default value of the lang attribute in the HTML.
When applying this to the phonetic transcription, include the -fonipa subcode.

closed by a8399e6de37b97410ea81b2158f4459fc37a22a4 and 08203f19489106183bc9687ca1137b198e79ded3


Add support for GET /users?blocked=true #241

As documented in #241 this PR adds support for GET /users?blocked=true

Thanks @michaellihs! LGTM!


Real Verilog tables from Chisel Vecs

Tables in Chisel are Vecs, but the Verilog output generates a priority MUX. This leads to the fact that Chisel generated tables result in LUT logic in an FPGA although they can be implemented in on-chip memory. This needs more resources and has a lower maximum frequency.
In my tutorials on Chisel I usually mention it as a big advantage that we can use Scala to generate logic tables, e.g., for microcode for a state machine, code for a processor, or a lookup table for functions. All those large tables should end up in a on-chip RAM in an FPGA.
Therefore, I request that tables (Vecs) generate a case statement that is understood by FPGA synthesize tools to generate on-chip ROM.
I've put up an example of a 1 KB table to illustrate the case:
git clone https://github.com/schoeberl/chisel-playground.git
cd chisel-playground
make table-verilog
and run the synthesize with Intel Quartus (from quartus/issue-table.qpf) for a real Verilog table.
Results:
10 logic elements and 8192 memory bits, fmax 250 MHz (restricted to this)
Generate the Chisel based table with:
make table-chisel
and synthesize:
870 logic elements, no memory bits, fmax 191 MHz
As long as (Quartus) synthesize tools cannot extract a simple lookup table from priority MUX code, I strongly propose that Chisel/FIRRTL shall spill out a case based table for a Vec.
Cheers,
Martin

An update on the issue: with Chisel 3.4 the generated code is a bit better than 3.2, but still a priority MUX. This led to the following regression from updating the Patmos processor from Chisel 2 to 3, when synthesizing to an (old) FPGA
Chisel 2: fmax 80 MHz, fetch stage 740 LCs
Chisel 3: fmax 67 MHz, fetch stage 3170 LCs
Chisel 2 generated Verilog can be put in on-chip memory, not the Chisel 3 code.
Chisel 2 Verilog is:
  always @(*) case (T20)
    0: T18 = 32'h87c20000;
    1: T18 = 32'h87c40000;
    2: T18 = 32'h3e1000;
    3: T18 = 32'h2402026;
    4: T18 = 32'hf0010000;
...

Chisel 3 Verilog is:
  wire [31:0] _GEN_8 = 9'h1 == addrEven[9:1] ? 32'h20700 : 32'h54; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_9 = 9'h2 == addrEven[9:1] ? 32'h20800 : _GEN_8; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_10 = 9'h3 == addrEven[9:1] ? 32'h2402025 : _GEN_9; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_11 = 9'h4 == addrEven[9:1] ? 32'h87c20000 : _GEN_10; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_12 = 9'h5 == addrEven[9:1] ? 32'h2821085 : _GEN_11; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_13 = 9'h6 == addrEven[9:1] ? 32'h2021062 : _GEN_12; // @[Fetch.scala 100:26 Fetch.scala 100:26]

Cheers,
Martin

An update on the issue: with Chisel 3.4 the generated code is a bit better than 3.2, but still a priority MUX. This led to the following regression from updating the Patmos processor from Chisel 2 to 3, when synthesizing to an (old) FPGA
Chisel 2: fmax 80 MHz, fetch stage 740 LCs
Chisel 3: fmax 67 MHz, fetch stage 3170 LCs
Chisel 2 generated Verilog can be put in on-chip memory, not the Chisel 3 code.
Chisel 2 Verilog is:
  always @(*) case (T20)
    0: T18 = 32'h87c20000;
    1: T18 = 32'h87c40000;
    2: T18 = 32'h3e1000;
    3: T18 = 32'h2402026;
    4: T18 = 32'hf0010000;
...

Chisel 3 Verilog is:
  wire [31:0] _GEN_8 = 9'h1 == addrEven[9:1] ? 32'h20700 : 32'h54; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_9 = 9'h2 == addrEven[9:1] ? 32'h20800 : _GEN_8; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_10 = 9'h3 == addrEven[9:1] ? 32'h2402025 : _GEN_9; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_11 = 9'h4 == addrEven[9:1] ? 32'h87c20000 : _GEN_10; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_12 = 9'h5 == addrEven[9:1] ? 32'h2821085 : _GEN_11; // @[Fetch.scala 100:26 Fetch.scala 100:26]
  wire [31:0] _GEN_13 = 9'h6 == addrEven[9:1] ? 32'h2021062 : _GEN_12; // @[Fetch.scala 100:26 Fetch.scala 100:26]

Cheers,
Martin

With Chisel v3.4 you can initialize memories (emitted as an init block in Verilog).
So you could try to build a ROM like this: https://scastie.scala-lang.org/QmN0vvjwRda0MBdx5YLxmw
The one bug that still needs to be fixed is that the init block is omitted when SYNTHESIS is defined (which is the case for some FPGA tools).

With Chisel v3.4 you can initialize memories (emitted as an init block in Verilog).
So you could try to build a ROM like this: https://scastie.scala-lang.org/QmN0vvjwRda0MBdx5YLxmw
The one bug that still needs to be fixed is that the init block is omitted when SYNTHESIS is defined (which is the case for some FPGA tools).

Thanks for your proposal. But it looks a bit like a workaround to a Verilog emission issue. Using a plain Vec for a logic table is more elegant than the preloading of a ROM.
And the same issue pops up when using a switch statement. I have an example of a small ALU using a switch statement that doubled the number of logic cells when coding it in Chisel compared to direct coding in Verilog. The fmax got a hit as well.

Thanks for your proposal. But it looks a bit like a workaround to a Verilog emission issue. Using a plain Vec for a logic table is more elegant than the preloading of a ROM.
And the same issue pops up when using a switch statement. I have an example of a small ALU using a switch statement that doubled the number of logic cells when coding it in Chisel compared to direct coding in Verilog. The fmax got a hit as well.

Using a plain Vec for a logic table is more elegant than the preloading of a ROM.

I think using a Memory to model a ReadOnlyMemory makes a lot of sense. In general Chisel could have gone the route of trying to infer memories from Vec instead of adding an explicit Mem construct.
I do agree that your ALU should not blow up. We could probably write a compiler pass that detects switch statements and tries to emit them as such.

Using a plain Vec for a logic table is more elegant than the preloading of a ROM.

I think using a Memory to model a ReadOnlyMemory makes a lot of sense. In general Chisel could have gone the route of trying to infer memories from Vec instead of adding an explicit Mem construct.
I do agree that your ALU should not blow up. We could probably write a compiler pass that detects switch statements and tries to emit them as such.

On 15 Jan, 2021, at 0:07, Kevin Laeufer<EMAIL_ADDRESS>wrote:
Using a plain Vec for a logic table is more elegant than the preloading of a ROM.
I think using a Memory to model a ReadOnlyMemory makes a lot of sense. In general Chisel could have gone the route of trying to infer memories from Vec instead of adding an explicit Mem construct.
I do agree that your ALU should not blow up. We could probably write a compiler pass that detects switch statements and tries to emit them as such.

That would be very good.
Yes, I remember an older proposal to use an array for a Vec of regs to automatically be able to infer a memory. However, this might be similar to inferring memory from stylish written VHDL and Verilog. A bit brittle, at least.
I agree on having a dedicated ROM component is good. However, still the generated code for a Vec should be non-priority MUXes. Vec and (combinational) switch are not so different.
Chers,
Martin

On 15 Jan, 2021, at 0:07, Kevin Laeufer<EMAIL_ADDRESS>wrote:
Using a plain Vec for a logic table is more elegant than the preloading of a ROM.
I think using a Memory to model a ReadOnlyMemory makes a lot of sense. In general Chisel could have gone the route of trying to infer memories from Vec instead of adding an explicit Mem construct.
I do agree that your ALU should not blow up. We could probably write a compiler pass that detects switch statements and tries to emit them as such.

That would be very good.
Yes, I remember an older proposal to use an array for a Vec of regs to automatically be able to infer a memory. However, this might be similar to inferring memory from stylish written VHDL and Verilog. A bit brittle, at least.
I agree on having a dedicated ROM component is good. However, still the generated code for a Vec should be non-priority MUXes. Vec and (combinational) switch are not so different.
Chers,
Martin


Is interactively zooming no longer available for Axis3?

Recently, I have been trying some Makie plots. In some videos from the recent past (summer 2021), I saw that there was the possibility to interactively zoom using GLMakie. However, as of right now, that does not seem to be a possibility anymore. Is this intended behaviour?

Axis3 never had zoom, you need to use LScene for that...


Fix: filenames too long

Types of changes

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

Motivation and Context / Related issue
Rare bug, but there are experiments for which some plots fail due to filename too long.
How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

There is a better solution: the length is on the basename, not on the full path.
I'll be back.


Fix olx.source.OSMOptions#url default value documentation

See https://github.com/openlayers/ol3/blob/v3.17.1/src/ol/source/osmsource.js#L31

Looks good to me, thanks and please merge.

Thanks for the review


Feature Request: Bidirectional Auto-Connect for Seamless Wallet-DApp Integration

Enhance P2P connector lib with a bidirectional auto-connect feature, enabling seamless wallet-DApp integration regardless of the order they're opened.
Key improvements:

Add a configurable auto-connect option to the DAppPeerConnect instance (enabled by default).
Automatically attempt connection to the most recently connected wallet.
Allow specifying an alternative wallet address if needed.

@marcuspuchalla has all of this been covered by #27 ? 😊

@marcuspuchalla have all of this been covered by #27 ? blush

Yes, it includes all the changes required for this to work.
A second meerkat on the wallet side is created. The address is sent to the DApp on the first successful connection. After that point, the DApp can connect to this Meerkat instance, signaling that it has started and that the wallet can initiate a regular connection.

Nice sounds good 😊


560 external registy and repository

I borked up the conflicts during this rebase, going to try again in fresh branch.


<EMAIL_ADDRESS>

The package-lock.json includes<EMAIL_ADDRESS>which includes malicious code: package-lock.json#L1090-L1095.
See also: https://github.com/dominictarr/event-stream/issues/116
The event-stream package looks unsafe. It's better to find an alternative in my 2 cents.

After the mess, the dependency to flatmap-stream was removed and the event-stream is maintained by the NPM team now.

Thank you for the heads up. I have locked the dependency to<EMAIL_ADDRESS>which gets rid of flatmap-stream from the dev dependencies.

Hi, I'm new to MEAN stack and can anyone please tell me how to safely remove or update this dependency from package-lock.json using NPM

@shivam183 here is how I did it : remove your package-lock.json, lock your dependency to event-stream at version 3.3.4 by adding "event-stream": "3.3.4", to your package.jsonrun npm i (that should work if your flatmap-stream dependency comes from event stream, if not you can repeat those steps for the respective 'ancestors')


Add Jenkins User Handbook (official)

What does this PR do?
Add resource
For resources
Description
Jenkins Guide
Why is this valuable (or not)?
It's useful for DevOps practitioners.
How do we know it's really free?
It's available at jenkins.io site
For book lists, is it a book? For course lists, is it a course? etc.
It's a book
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!