Liv 4155 plugin add improvements typings hook pre commit

[x] Clear all lint errors (no any, no empty return function...)
[x] Add pre-commit hook with lint-stage (eslint)
[x] Add paths in tsconfig.json to make easier the import of files (import Foo from '@/foo' instead of import Foo from '../.././foo')
[x] Use class with static methods instead of Object
[x] Use camelCase instead of PascalCase
[ ] Add stronger typings

Trop cool :+1:


Ensure core design tokens are documented with a description attribute as an example

Part of the official W3C design token space being proposed is a description field to give a brief human-readable string to describe each token.
Given we have nearly 1500 tokens/variables, I believe asking to add description field to all of them in one go is a bit much and out of scope.
However, I would like for us to implement the description field on at least a subset of the more important tokens (e.g., global colors, spacing, and typography) to demonstrate the capabilities and show an example of the vision for all the tokens.
See https://design-tokens.github.io/community-group/format/#description fore more details.

Maybe just colors and spacing 😄

https://github.com/openedx/paragon/pull/2112


Add a recursion limit

This PR adds a recursion limit to INCLUDEs, MACROs and REPTs, as well as EQUS expansions; it also adds a command-line option to control that limit (-r)
Excerpt from 6068b56's commit message:

The default of 64 is super overkill: even in a a project with what I believe to be above-average levels of nesting, the level only peaked at 6.
Keeping in mind the purpose of this is to catch infinite recursion, which is still caught quickly (in usual cases, anyways), this default seems sensible.

The default might even be too high, as 64 nested INCLUDEs seem to take up a whole lot of memory.
This PR also makes RGBASM track EQUS expansion, which means it should be possible to print them in error reports after this gets merged.

Other than the pre-existing undefined behavior I noticed, everything seems good to me.


name selector can potentially be harmful

I was playing with the network stats I get out of node_exporter. I wasn't exactly sure what kind of stuff is there, so I've asked for {__name__=~"node_network_transmit.+",device="eth1"}. This gave me following vector:
node_network_transmit_frame{device="eth1",instance="localhost:9100",job="node_exporter"}         0
node_network_transmit_fifo{device="eth1",instance="localhost:9100",job="node_exporter"}          0
node_network_transmit_drop{device="eth1",instance="localhost:9100",job="node_exporter"}          0
node_network_transmit_multicast{device="eth1",instance="localhost:9100",job="node_exporter"}     0
node_network_transmit_packets{device="eth1",instance="localhost:9100",job="node_exporter"}       1660881
node_network_transmit_compressed{device="eth1",instance="localhost:9100",job="node_exporter"}    0
node_network_transmit_bytes{device="eth1",instance="localhost:9100",job="node_exporter"}<PHONE_NUMBER>
node_network_transmit_errs{device="eth1",instance="localhost:9100",job="node_exporter"}          0

Happy with the results, I've decided to graph it, so I've changed the expression to:
irate({__name__=~"node_network_transmit.+",device="eth1"}[5m])

which resulted in following vector:
{device="eth1",instance="localhost:9100",job="node_exporter"}    0
{device="eth1",instance="localhost:9100",job="node_exporter"}    233.7
{device="eth1",instance="localhost:9100",job="node_exporter"}    0
{device="eth1",instance="localhost:9100",job="node_exporter"}    2.6
{device="eth1",instance="localhost:9100",job="node_exporter"}    0
{device="eth1",instance="localhost:9100",job="node_exporter"}    0
{device="eth1",instance="localhost:9100",job="node_exporter"}    0
{device="eth1",instance="localhost:9100",job="node_exporter"}    0

As you can see, you get 8 row vector, each row having identical labels. This is rather unexpected. If you ask me, the __name__ label should have been preserved, but I can see how this can throw people off in other situations.
Thoughts?

It's not correct to preserve the  name as it's no longer the same metric. Doing this sort of operation across multiple metric names is unusual, and if you'd tried an aggregation on top of it it'd have errored out.
If you really want to do this, use label_replace  to copy out the metric name to another label name.

Fundamentally there's a number of ways you can end up in this situation with any labels. It follows logically from how the language semantics, and while it may seem a little odd it is the correct behaviour. Trying to impose different semantics in these corner cases would add complexity for little gain, so I don't think we're going to change anything here.


Negative index not allowed

Hello, I'm co-author of bevy_fbx plugin for loading FBX in Bevy Engine, which is heavly based on your fbx-viewer code.
While trying to load Bistro Scene from Amazon's Lumberyard demo, I'm getting following error
2022-07-12T13:01:02.864866Z ERROR bevy_fbx::loader: failed to load "BistroInterior_Wine.fbx"

Caused by:
    0: Failed to load geometry mesh
    1: Failed to reconstruct UV vertices
    2: Negative index is not allowed: direct=-1

For us, it looks like UB caused by FBX file (v7.5) from Amazon and I would like to know if you have any info of how we could fix this problem, either in fbxcel(-dom) or our loader.
According to Autodesk forums, FBX uses negative values in PolygonVertexIndex which then have to be turned into possitive ones by reader.

the negative values in PolygonVertexIndex are just used to indicate the end of a polygon.
It will be converted to a positive value by fbxreader according to certain rules.
So you do not need to worry about them, as long as you import the FBX file, the reader will take care.

Already tried exporting it as v7.4 and changing settings in Cinema4D and that didn't helped.

Thank you for reporting!
Fixed and released as v0.0.8.
Note that v0.0.8 requires Rust 1.56 or later (I hope this won't bother you).
The relevant commit of the fix is 65b9faaf05d42de401d743a20fd30c5558676775. You can cherry-pick this if necessary.
Anyway, you are doing amazing work!
I'm happy I can see my work will contribute to more people.

Thank you very much, I really like your work. Rust version is not a problem because we always target latest stable which is 1.62 currently.


Correctly register for EPOLLRDHUP when construct EpollSocketChannel f…

…rom FileDescriptor
Motivation:
We missed to register for EPOLLRDHUP events when construct the EpollSocketChannel from an existing FileDescriptor. This could cause to miss connection-resets.
Modifications:
Add Native.EPOLLRDHUP to the events we are interested in.
Result:
Connection-resets are detected correctly.

Cherry-picked into 4.0 (5c7022d49449389f22d3387d6323b55870e28512), 4.1 (1a5dac175ef16feb58f41e595817e1b34d9f01b5) and master (22cea8a2cc7c3082478f214876e8d06161307c59)


Question about occluder

Hi, I wanna ask about the design of the occluder used in this paper.
I read from the paper that 'Without losing generality, we simulate the presence of a box-like obstacle right in front of the camera that causes a rectangular area at the center of the frame to be occluded.'. And I think it should be look like this:

However, I ran the code and found out the occluder is somehow just a black value (RGB: [0,0,0]):

I think the code told it to do so:

So I wonder whether it should be like this(i.e, simply make the occluder black) or should I implement a version the same as the paper? If the latter option is required, how should I do it?

Hi, yes, occluders are treated and processed as the codes implemented. Our intuition is that, when there is an occluder blocking the camera, blocked human body becomes invisible and undetectable by semantic segmentation models. In this way, we obtain an alpha mask that indicates transparency at the invisible parts include both background and occluded regions, which are being processed equally (blended according to bg_color) since they are no different in our methods.

I see. Thanks for you reply!


Cabal fails after install -- timer_create: Invalid argument

apt-add-repository ppa:hvr/ghc
apt-get update
apt-get install cabal-install-1.24
/opt/cabal/1.24/bin/cabal

This results in
cabal: timer_create: Invalid argument
This works on my ubuntu-14 box
ghc appears to actually run, but it's painfully slow to start.

timer_create isn't implemented, see #112

You are correct @voltagex, same thing happens with pandoc. Why am I trying to run pandoc in bash on Windows when there is already a Windows package? I have no answer :).


code-workspace: Initial commit

This is an initial commit defining a default environment for editors that recognize code-workspace file such as:

Theia (eclipse-theia/theia)
Gitpod.io (gitpod.io)
VSCodium (vscodium.com)
VSCode (code.visualstudio.com)

To enable contributors to use a standardized environment with on-demand cloud editor in the web-browser to make it easier to work with the source code.
Demo: https://gitpod.io/#https://github.com/NixOS/nixpkgs/tree/nixos-21.05 (Using theia deployed in Gitpod's infrastracture)
Signed-off-by: Jacob Hrbek<EMAIL_ADDRESS>

Motivation for this change
Being able to work on nixpkgs without maintaining my configuration with on-demand cloud editor
Workaround to No pinentry bug in NixOS to be able to sign commits (https://github.com/NixOS/nixpkgs/issues/97861#issuecomment-908388115) - Apparently was fixed, works on my end
Things done

Built on platform(s)

[INVALID] x86_64-linux
[INVALID] aarch64-linux
[INVALID] x86_64-darwin
[INVALID] aarch64-darwin

[INVALID] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[INVALID] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[INVALID] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[INVALID] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[INVALID] (Package updates) Added a release notes entry if the change is major or breaking
[INVALID] (Module updates) Added a release notes entry if the change is significant
[INVALID] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

we already have nixpkgs/.editorconfig

a standardized environment

what exactly is standardized here?
{
	"folders": [
		{
			"path": "."
		},
	]
}

IDEs should be smart enough to open a folder

Hi @Kreyren !
I'm sharing the @milahu's concern here. Could you please let us know a bit more on what this file would bring?
If you don't want to push this further, feel free to close the PR.
Thanks!


World vs Level in screens

In client/gui/screen package (which I'm mapping for past few days), there are several classes with "world"/"level" in their names. If I understand https://github.com/FabricMC/yarn/issues/89#issuecomment-778691059 correctly:

a level is a savegame with level.dat and several worlds. "World" in "Create New World"/"Select World" means what yarn calls a level.
a world is a region/DIM1/DIM-1 folder in a level. A block can be identified with XYZ coordinates. "World" in "Generating world"/"Loading world" means what yarn calls a world.

If that is the case, naming conventions in client/gui/screen package seems to be the opposite:

CustomizeBuffetLevelScreen actually customizes a world (specifically Overworld), not a level. However JSON-based "custom world" is probably custom levels.
LevelLoadingScreen is tricky, but seeing as this is used when someone travels between dimensions, it could be "world loading screen".
Things under /world all seem to be related to level, not world.

Am I understanding things correctly, and should I rename those classes and packages?

Hmm, I now think we should call save games "save game" explicitly than using ambiguous "level". I recall that now level is mainly LevelProperties (name from level.dat), which is an implementation of both ServerWorldProperties and SaveProperties.
For customization, I think we need to look at GeneratorOptions methods. There are different types of factory methods for different customizations from the screens/server.properties etc.
Honestly, after all, I haven't thought about how improper these "world" names are for the world creation screens. Maybe this "world" is just distinct from our regular worlds enough so we never thought about these conflicts.

Save game makes more sense to me than level does. I would like to see people look into that further.


Trigger start can not change even if user select another one

User change start time. But selected time will not be reflected to control.

Close by 1d5adb895de8dae834c784c8e7eecaaaeca999c5


Incorrect isfile() check in sigver.py

This is in CertHandler's constructor:
if cert_file and os.path.isfile(key_file):
    self._cert_str = self._osw.read_str_from_file(cert_file,
                                                  cert_type)

Pretty sure that test for key_file should be a test for cert_file.

Has this fix been published to pypi yet?


Merge pull request #202 from MicrosoftDocs/corranrogue9/7.10.0-releas…

…enotes
Adding release notes for 7.10.0

Docs Build status updates of commit 891476e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/changelog/odatalib-7x.md
:bulb:Suggestion
View
Details

Odata-docs/changelog/odatalib-7x.md

Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'madansr7' is not a valid Microsoft alias.
Line 1092, Column 84: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'T' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 1477, Column 3: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Adds validation for the examples

All examples must adhere to a strict format. We'll be adding a directory with "unstructured" examples that don't adhere.
TODO on this:

[x] Fix up all examples

[x] conversational-rag
[x] hello-world-counter
[x] llm-adventure-game
[x] multi-agent-collaboration
[x] multi-modal-chatbot
[x] simple-chatbot-intro
[x] streaming-overview

More work in this, this I think is good enough for now: https://github.com/DAGWorks-Inc/burr/issues/116


systemd Service won't start

@bdaehlie re systemd:
1. What version of Caddy are you running (caddy -version)?
0.9.3
2. What are you trying to do?
systemd
3. What is your entire Caddyfile?
confluence.dpsi.jp {
	proxy / confluence.dpsi.jp:8090 {
		transparent
	}
	tls<EMAIL_ADDRESS>	gzip
}

4.
Followed this: https://github.com/mholt/caddy/tree/master/dist/init/linux-systemd
5. What did you expect to see?
Caddy starting correctly
6. What did you see instead (give full error messages and/or log)?
● caddy.service - Caddy HTTP/2 web server
   Loaded: loaded (/etc/systemd/system/caddy.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Mon 2016-10-31 13:13:03 PDT; 13ms ago
     Docs: https://caddyserver.com/docs
  Process: 10935 ExecStart=/usr/local/bin/caddy -agree=true -conf=/etc/caddy/Caddyfile (code=exited, status=1/FAILURE)
 Main PID: 10935 (code=exited, status=1/FAILURE)

Oct 31 13:13:01 proxy systemd[1]: Started Caddy HTTP/2 web server .
Oct 31 13:13:03 proxy systemd[1]: caddy.service: Main process exited, code=exited, status=1/FAILURE
Oct 31 13:13:03 proxy systemd[1]: caddy.service: Unit entered failed state.
Oct 31 13:13:03 proxy systemd[1]: caddy.service: Failed with result 'exit-code'.

7. How can someone who is starting from scratch reproduce this behavior as minimally as possible?
Fresh Ubuntu 16.04.01 LTS with the above instructions.

Before this I had an issue where it was denied permission from the Caddyfile. Changing that to 544 instead of the 444 suggested in the guide fixed that issue.
Caddy runs fine when running the command caddy. Just systemd won't work. I have it working right now using init.d but that defeats the purpose of systemd.

Hey @dpsi, thanks for the report. However we don't support the init scripts in that folder, and since this isn't a Caddy bug (there was no log output provided to show otherwise), I'm going to close this. But you are welcome to ask the community about this on the Caddy forum!

@mholt but the init scripts sometimes point to bugs in caddy. For example: it seems to mkdir ., which it should not do, which can cause systemd scripts to fail when they begin in, say /, which is owned by root and not www-data.


[examples] Fix sentiment nit

Fixes a small nit on sentiment script
cc @lvwerra

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


complete part 1 of assignment

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

create a new directory

What did you learn from the changes you have made?

how to create a directory in bash

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

not that I know of

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

none

How were these changes tested?

tested through terminal

A reference to a related issue in your repository (if applicable)
n/a
Checklist

[x] I can confirm that my changes are working as intended

You did a great job here!
Please go ahead and close this pull request now. Thanks!


Web console: upgrade axios and follow-redirects

Upgrade Axios and remove explicit follow-redirects declaration to get onto the new version.
I believe the old Axios had some CVEs associated with it.

Had to upgrade Jest also and that changed how the snapshots are rendered.


Feature: add Intel logo

Would you add the Intel logo?
https://newsroom.intel.com/wp-content/uploads/sites/11/2016/04/intel-logo-default.png

If you can find a good SVG version of the icon, I can try to shrink it.


fix: Fix help generation

CLI configuration used to generate the DApp metadata

This one comes from a Comment and not an annotation. Maybe it is best to add back again the Description for every service, so we do not fallback on presenting some unwanted description that comes from a "inner" configuration that could come from an external crate

Correctly tested every service and they are consistent now, looking something like this
docker run -it cartesi/rollups-node:devel cartesi-rollups-state-server --help | head
Configuration for state-server

Usage: cartesi-rollups-state-server [OPTIONS]


[Int4-AWQ] Fix ROCm AWQ Marlin check

This commit resolves an issue with the new AWQ Marlin support added in upstream.  AWQ Marlin is not yet supported for ROCm in vllm, but vllm will override AWQ quantization with AWQ Marlin if quantization parameters are compatible without comprehensively checking for platform support.  This commit fixes this problem in the case of ROCm by adding an is_hip() check.

Wrong repo.


AutoSuggestBox stays showing all suggestions even there is no text in the field

Normally if I type in some text and the SuggestBox still works fine.

But when I clear all the text, the box will shows all suggestions, it cannot be unfocus by clicking on other controls, and sometimes it shows on top of other windows. This bug happened on the Demo App as well.

I've got the same issue. The moment the textbox is empty, the suggestions pop up and won't go away no matter what, including being shown on top of other windows when your app isn't even in focus.

The solution I found was to set this:
    protected override void OnLostFocus(RoutedEventArgs e)     {         IsSuggestionListOpen = false;         base.OnLostFocus(e);     }
Setting the binding property to false on lost focus since its a textbox.
In my case I created a subclass as workaround and overrided the method. It could be easily implemented in the actual control.

In order to close the popup even if apps are switched, I also made this:
    protected override void OnInitialized(EventArgs e)
    {
        base.OnInitialized(e);
        Window parentWindow = Window.GetWindow(this);
        parentWindow.Deactivated += ParentWindow_Deactivated;
    }

    private void ParentWindow_Deactivated(object? sender, EventArgs e)
    {
        IsSuggestionListOpen = false;
    }

So on window deactivated, it closes the popup


Add Telegram Support - Bump to version 0.1.9

Adds Telegram support as a notification provider.
Requires the following on existing databases
ALTER TABLE capcodes ADD COLUMN telegram INTEGER DEFAULT 0;
ALTER TABLE capcodes ADD COLUMN telechat TEXT;

Looks good.. All works 👍

Fairly confident this works as expected, I tested a whole bunch last night with no issues.
I also tweaked thee message format a little, if anyone can come up with anything better, feel free.
Here's how it currently looks

Awesome! Before we merge I'd like to have a think about #114 - perfect opportunity to solve that problem at the same time.
I also wanna have a quick look at the library's docos to see if we'll run into any issues with initiating new telegram on each message. If it's just setting up methods then should be fine, but if it's actually doing a pass of credentials at that point and retrieving another auth token or something, then could run into API limits later on.

I also wanna have a quick look at the library's docos to see if we'll run into any issues with initiating new telegram on each message. If it's just setting up methods then should be fine, but if it's actually doing a pass of credentials at that point and retrieving another auth token or something, then could run into API limits later on.
Good catch, the library i used doesn't seem to be the most active one, it was just the one the last fork had used, the other one seems to be more regularly developed but would have the same issue.
https://github.com/yagop/node-telegram-bot-api

Had a quick read of the current lib, it doesn't do anything fancy on its initial call, unless you have updates enabled, and they're off by default, so should be safe as is :)
It'd probably be more efficient and safer to set up the t variable up the top just after calling the library - unless there'd ever be a case where we'd want different API keys for different aliases. One key can be used for many channels, yeah?

Yep,
My understanding is the token is per bot, i can't see any use case for multiple bots so it should be safe to declare it once, the only consideration would be if it makes a call to validate - it may slow down message processing for messages without notifications
If we're going to go down that route the same probably applies to pushover, i can't ever seeing there being a need for multiple application keys (except to skirt their message limits which i wouldn't be writing in anyway) so it could be done at the top as well.

You guys wanna review again? I've pulled the DB stuff out and added some logic to upgrade the DB schema when required.
It's controlled by the 'release' int in app.js, so all we need to do to push an update to schema is update that number and add the relevant bits into db.js. It will attempt to add missing columns as required. If more complex schema changes are required, we can probably add some logic in there to drop and recreate the tables.
Also moved the t definition up to the top - I haven't tested this bit yet, only tested the DB upgrade bits

These changes look super good, Ill pull down tonight and submit a review 👍

Yep,
My understanding is the token is per bot, i can't see any use case for multiple bots so it should be safe to declare it once, the only consideration would be if it makes a call to validate - it may slow down message processing for messages without notifications

Yeah the token is per bot..  I believe you can send messages to different channels/users from the same bot but only in the same instance.. you cant have 2 bots with the same name.

If we're going to go down that route the same probably applies to pushover, i can't ever seeing there being a need for multiple application keys (except to skirt their message limits which i wouldn't be writing in anyway) so it could be done at the top as well.

Nah you shouldn't need multiple API keys as I don't we will hit the following criterias

mess More then 30 messages per seconds
mess More then 20 messages per minute

They recommend not sending more then 1 message a second which may happen if the system spits out lots of pagers at the same time but they have the above limits
Maybe we can add tho to throw a error in log/console when 429 error is detected so we know the limit is being hit.

Tested Telegram with these changes, looks all good.
DB Upgrade worked well on my production database as well (140k messages).
Added a commit for a field i forgot to change the name of in the settings page :smile: 👍

Yep Tested Telegram as well.. Works good..
DB Updated...
Can't review but good to go :)

Date: (dateofmessage) Time: (Timeofmessage)
Message: (Message)
Brigade: (Alias) Agency: (Agency)

The agency should be shown in the title of the current template, not a fan of having the date and time, telegram has that anyway.

Merging as is, we can make changes to message format later if needed :)

Okays so Documentation for Telegram API has been done.
https://github.com/s3m1s0n1c/pagermon/wiki/Telegram
I plan on doing some more documentation on the rest of Pagermon when I get time.
Is this OK or do u want more detail/pics?
Thanks

Love some good pics in guides
Could also make use of code blocks for things like examples :)
Top work as always sonic

Okays no worries.. Will add it in..

Nice, thanks dude :) Have added you to collaborators if you wanted to edit direct on the repo's wiki

Nice, thanks dude :) Have added you to collaborators if you wanted to edit direct on the repo's wiki

No worries, Thanks


Language filter: rename "C Sharp" value to "C#"

Following on from issue #24, and as per ECMA-334, the correct name for "C Sharp" is "C#".

Fixed on http://rawgit.com/finos/finos.github.io/staging/index.html


Added requirements.txt file

Added requirements.txt file

This file contains the required list of packages for the project.

Is this the confirmed list of requirements?
Can I get a +1 from @iammarco11 as well?

Okay there are still some unnecessary packages, I have listed the necessary one
certifi==2<IP_ADDRESS>
chardet==3.0.4
idna==2.9
oauthlib==3.1.0
pkg-resources==0.0.0
PySocks==1.7.1
requests==2.23.0
requests-oauthlib==1.3.0
six==1.14.0
tweepy==3.8.0
urllib3==1.25.8

Just can you check from your side as well, @ananthanandanan?

Looks good. Once the PR is updated, let me know. I will merge :)

@iammarco11  I don't think pkg-resources is needed. For reference https://stackoverflow.com/questions/39577984/what-is-pkg-resources-0-0-0-in-output-of-pip-freeze-command

Oh yea excluding that
Can you update the PR?

Does this is it LG.

Can you remove the unwanted changes in the gitignore file?
Otherwise LGTM

@Aniketh01 I guess all the required packages are there.

Yes.

Merged and close! :)


Implement gaussian elimination

Gaussian elimination is an algorithm in linear algebra for solving a system of linear equations.
depends on #8
Matlab implementation
gaussPivot.m.txt
Test file:
Test_gauss.m.txt

depends on itself?


Program end reset

taking over from #101

this will require a fix pr https://github.com/FRUK-Simulator/SimulatorCore/pull/117


CPU&memory occupation high and windows break down when using VSCode as external editor of Xilinx Vivado

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

VS Code Version:
OS Version:

Steps to Reproduce:
1.setting VSCode as the external editor of Vivado. Opening vivado, goto the menu bar Tool->Setting->Text Editor Select Custom as current editor. Fill the textbox by "path of vscode/Code.exe" -g [file name]:[line number]
2.Then open a large project by vivado, open one source file. It shold goto the VSCode immediately, However the window always break down.

We do not support this integration. So I would request to file the issue against Xilinx Vivado product to get it diagnosed. Thanks.

I don't think it is the matter of Vivado. When I use the same way to set Atom as the external editor of Vivado, Atom would never break down. Using VSCode as the external editor of Vivado is just a process of create a VSCode process and pass some argument into VSCode, Therefore VSCode process is totally independent from Vivado. I really think that it's the matter of VSCode. You should checkout the executing way of command line  code.exe -g <filename>:<line>

See open issue: https://github.com/microsoft/vscode/issues/147541

So, we can sure that this isnt a special case, almost all the computer occur this bug when using vscode as the external editor of Vivado.  Just because vscodes auto update and some bugs when produce.


[FEATURE] - upgrade to webpack 2

Coverage remained the same at 51.411% when pulling 74d4ac33a8945c47b56be3b1ee31f7ca6aebacb9 on master-update-webpack into c6b4fcd1e76a2d05a9bb7ecd76fbeba6efcff620 on master.

Codecov Report

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

@@          Coverage Diff           @@
##           master     #34   +/-   ##
======================================
  Coverage    12.5%   12.5%
======================================
  Files           1       1
  Lines           8       8
======================================
  Hits            1       1
  Misses          7       7

Continue to review full report at Codecov.

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


IAM Query fails quite often

I have below query which runs across multiple AWS accounts( as configured in steampipe config) .
select
_ctx ->> 'connection_name' as account_info,
account_id,
user_name,
'XXXXXXXXXXXXXXX' || substring(access_key_id from 15 for 3) as access_key_id,
status,
create_date,
now() - create_date as "creation_age",
access_key_last_used_date,
now() - access_key_last_used_date as "access_key_last_used_age"
from
aws_iam_access_key
where
now() - create_date >= '60 days'
order by
create_date asc
This query get success sometimes and lot of time fails giving below error. Real values replaced with XXXXXXXXXXX.
Error: <connection_name XXXXXXXXXXX>: operation error IAM: ListAccessKeys, https response error StatusCode: 404, RequestID: e799fad4-fa8e-4741-919d-55ab378809bc, NoSuchEntity: The user with name XXXXXXXXXXX cannot be found.
<connection_name XXXXXXXXXXX>: table 'aws_iam_access_key' column 'access_key_last_used_service' requires hydrate data from getIamAccessKeyLastUsed, which failed with error operation error IAM: GetAccessKeyLastUsed, https response error StatusCode: 403, RequestID: 72ca21c1-15aa-46e2-bcc6-2475fbf91bc6, api error AccessDenied: User: arn:aws:sts::XXXXXXXXXXX:assumed-role/CrossAccountRead/aws-go-sdk-1705253076847995250 is not authorized to perform iam:GetAccessKeyLastUsed on resource: access key XXXXXXXXXXX.
(SQLSTATE HV000)
Looks like when it fetched the list of users the user was there and later when trying to fetch keys for that user it was probably deleted during that time. If it is the cause, it should handle this properly and ignore that user and return result for other users.

steampipe -v
Steampipe v0.21.3

Hi @ManojDhanorkar, Appreciate your effort in highlighting this matter to us.  I'll delve into this matter and reach out if I require further details from you.

Thanks.

Hello @ManojDhanorkar,
I was digging into the issue, but somehow, It is very difficult to reproduce the error because creating such a scenario during the query execution is difficult.

Looks like when it fetched the list of users the user was there and later when trying to fetch keys for that user it was probably deleted during that time. If it is the cause, it should handle this properly and ignore that user and return result for other users.

Yes, that may be the case,

While it's true that the table should not generate a 'NotFound' error, the plugin code is expected to handle such scenarios.
However, 'AccessDenied' errors should not be managed by the plugin.
To suppress 'AccessDenied' errors, configure the ignore_error_codes block in the ~/.steampipe/config/aws.spc file.
A PR has been raised to address the handling of 'notfound' error codes for the aws_iam_access_key table.

Your feedback on the PR branch would be greatly appreciated. If there are any further issues, please inform us.
To test the PR branch, follow these steps:

Clone the repository: git clone https://github.com/turbot/steampipe-plugin-aws.git
Change directory: cd steampipe-plugin-aws
Switch to the issue branch: git checkout issue-2053
Execute make
Run your query

Thank You!

Thanks for quick turnaround..

Is there any specific hardware requirement to run make command. I am trying on t3.medium ( 2 vcpu , 4 GB RAM) , but make command hangs

Hello @ManojDhanorkar, This makes sense. Could you please temporarily disable the code from line 40 to 239999 in the parliament_iam_permissions.go file, and then execute the make command? This should help in resolving the issue, and as a bonus, the plugin building process should be quicker.
Thanks!

Hello @ManojDhanorkar, could you please confirm if the issue has been resolved by the solutions provided in this comment?

Will try to test this week

Hey @ManojDhanorkar, we're closing this issue as we haven't heard back from you yet, please feel free to reopen if you want to share anything. Thanks!


Add qwen2 to distributed model list

Fixed Qwen2 model multi-hpu execution issue in 1.17.447 build, so add Qwen2 in model to meta list

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Multiple Storable Objects

If I set multiple storable objects, the object type seems to get mismatched. This seems to happen when the data gets read back from a file, and it parsed with .from_json.
Let me know if you need more info to recreate.

@crisward can you post an example? That would help me a ton.

Will do. I've worked around this at the moment by creating a session schema object and storing everything in there as per this discussion - https://github.com/kemalcr/kemal-session/issues/38.
I'll try and add a failing test...

@crisward what version of LLVM do you have installed?

Alright, I think I figured it out. The specs as they exist today between UserTestSerialization and UserTestDeserialization will result in type mismatches when parsing the json from the storage instance. The reason this happens is that when a union type is deserialized from JSON, crystal will go through all the types within the union until one of them parses correctly. Unless the converter method throws an error, it's likely that you'll get a parse mismatch on deserialization. If you have different classes but they happen to have the same properties, like in cases where you subclass, you're always going to get the incorrect type.
Our options include:

include the actual type when serializing the Storable Object to session storage.
Only allow one storable object across a code base

I'm in favor of the first option. I think that's going to mean we're going to have to create a wrapper object to allow us to encode the type as part of the storable object. I don't think that will require the storage engines to have to change their implementations but I'm not sure at this point.
I think there's a way forward but I'm not sure when I can get some time to fix it.

I went with option no.2 in my usage and just have a big 'session' object which I've attached my other objects to, as per this discussion - https://github.com/kemalcr/kemal-session/issues/38. Sorry I never did get around to giving an example. It was a case of finding a solution and moving on. I intend to write a mysql implementation of session storage once these kind of implementation details are ironed out.


Documentation for following redirects

I noticed that following redirects isn't mentioned in the wiki documentation, so I'm submitting this as a pull request to the wiki.

Redirects
Automatically following redirects
The HTTP.follow option can be used for automatically following redirects:
response = HTTP.follow
               .get("https://example.com")

Limiting number of redirects
A maximum of 5 subsequent redirects will be allowed by default (anything more
will raise a HTTP::Redirector::TooManyRedirectsError), but you can change the
limit with the :max_hops option:
response = HTTP.follow(max_hops: 3)
               .get("https://example.com")

Unsafe redirects
If a POST, PUT or DELETE request returns a 300, 301 or 302 redirect response,
it won't be followed by default and HTTP::StateError will be raised. You can
set :strict to false if you want these redirects to be followed, in which
case the GET verb will be used for that redirect.
response = HTTP.follow(strict: false)
               .get("https://example.com")

I have unlocked WIKI from editing. Please, add your changes there! And thanks once again!

Thanks, updated!

🙇

Yes via HTTP::Response#uri:
HTTP.follow.get("http://sensortower.com").flush.uri
# => #<HTTP::URI:0x0055947e0aaf60 URI:https://sensortower.com/>

Thanks - I've updated the wiki's redirect page.


FIX: Don't target the classification head when using target_modules="all-linear"

Fixes #2027
When using a transformers sequence classification model, target_modules="all-linear" should not wrap the classification head with LoRA. This is because it is already wrapped with ModulesToSave, i.e. it will be fully fine-tuned, which is the generally desired behavior.
Before this bug fix, the classification head would be double-wrapped. With #2028, this now raises an error. With this PR, it is avoided completely. Still, keeping #2028 is good because it helps prevent other situations where double-wrapping might occur due to mis-configuration.
Note that there is no fool-proof method to detect the classification head, we have to rely on transformers convention.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


add sticky client info panel and some subtle inferno branding

Client Info bar sticks to top of screen on scroll.

There are couple more issues:

[ ] for long server urls, the server url on top now wraps.  We have code in there to ellipsisize long server urls, but the changes here break that
[ ] After you run the test, the icon doesn't appear until you scroll a bit.  It should just show up by default.

Really cool though.  Just need to tie up on the loose ends.

While reviewing I found a few issues, so I just took care of them:

the transitions were removed when you readded the ellipsis
I added a link back to the base directory for the logo
I disabled the transition when you start in the middle of the page.

Last 2 things we need to do:

[ ] Check the in IE edge
[ ] Try out inferno-site-overlay and update any css / html if necessary


AngularJS repository marked as being 51 years old

I know that this is probably caused due to broken unix timestamps in the repository, but it should be possible to exclude those broken timestamps.
angular/angular.js

Good point. In this case the broken timestamp is the unix epoch, so it's easy to detect and discard:
https://github.com/angular/angular.js/commit/866346e1ada583daeb649c253516c8f8a879c387.patch
Repos that have this timestamp will simply be cached with false and ignored

I'm wondering why it says 51 years though, as 2020 minus 1970 equals 50. Is that GitHub's <relative-time> element rounding up?

I guess it's rounding up, because we're already in June.

we could use this and filter out the 1990 ones
{
  repository(owner: "angular", name: "angular.js") {
    ref(qualifiedName: "master") {
      target {
        ... on Commit {
          history(first: 5, after: "72fbd48f2a805e789fc574176e7a9a87979418d7 8996") {
            nodes {
              committedDate
              resourcePath
            }
          }
        }
      }
    }
  }
}

Only it that replaces the current fetchDom request


Ubuntu/Debian Quick Start Guide issue: ClickHouse does not start with no password.

The problem is very simple: if I follow the Quick Start Guide for Ubuntu/Debian the last command does not start ClickHouse client and finishes with an error.
A solution is trivial but I have to google it or read a detailed log of the previous (not the last executed) command.
What should be changed:
clickhouse-client
should be replaced with
clickhouse-client --password
Below is a complete installation log for a clean Ubuntu:
xubuntu@xubuntu:~$ sudo apt-get install apt-transport-https ca-certificates dirmngr
Reading package lists... Done
Building dependency tree
Reading state information... Done
dirmngr is already the newest version (2.2.19-3ubuntu2.1).
dirmngr set to manually installed.
The following NEW packages will be installed:
  apt-transport-https
The following packages will be upgraded:
  ca-certificates
1 upgraded, 1 newly installed, 0 to remove and 192 not upgraded.
Need to get 150 kB of archives.
After this operation, 161 kB of additional disk space will be used.
Do you want to continue? [Y/n]
Get:1 http://security.ubuntu.com/ubuntu focal-security/main amd64 ca-certificates all 20210119~20.04.2 [145 kB]
Get:2 http://archive.ubuntu.com/ubuntu focal-updates/universe amd64 apt-transport-https all 2.0.6 [4,680 B]
Fetched 150 kB in 1s (149 kB/s)
Preconfiguring packages ...
(Reading database ... 161160 files and directories currently installed.)
Preparing to unpack .../ca-certificates_20210119~20.04.2_all.deb ...
Unpacking ca-certificates (20210119~20.04.2) over (20210119~20.04.1) ...
Selecting previously unselected package apt-transport-https.
Preparing to unpack .../apt-transport-https_2.0.6_all.deb ...
Unpacking apt-transport-https (2.0.6) ...
Setting up apt-transport-https (2.0.6) ...
Setting up ca-certificates (20210119~20.04.2) ...
Updating certificates in /etc/ssl/certs...
0 added, 1 removed; done.
Processing triggers for man-db (2.9.1-1) ...
Processing triggers for ca-certificates (20210119~20.04.2) ...
Updating certificates in /etc/ssl/certs...
0 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
done.

xubuntu@xubuntu:~$ sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv E0C56BD4
Executing: /tmp/apt-key-gpghome.vjwE5NXgcT/gpg.1.sh --keyserver hkp://keyserver.ubuntu.com:80 --recv E0C56BD4
gpg: key C8F1E19FE0C56BD4: public key "ClickHouse Repository Key<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1
xubuntu@xubuntu:~$ echo "deb https://repo.clickhouse.com/deb/stable/ main/" | sudo tee \
>     /etc/apt/sources.list.d/clickhouse.list
deb https://repo.clickhouse.com/deb/stable/ main/
xubuntu@xubuntu:~$ sudo apt-get update
Ign:1 cdrom://Xubuntu 20.04.3 LTS _Focal Fossa_ - Release amd64 (20210819.1) focal InRelease
Hit:2 cdrom://Xubuntu 20.04.3 LTS _Focal Fossa_ - Release amd64 (20210819.1) focal Release
Hit:4 http://security.ubuntu.com/ubuntu focal-security InRelease
Hit:5 http://archive.ubuntu.com/ubuntu focal InRelease
Ign:6 https://repo.clickhouse.com/deb/stable main/ InRelease
Hit:7 http://archive.ubuntu.com/ubuntu focal-updates InRelease
Get:8 https://repo.clickhouse.com/deb/stable main/ Release [749 B]
Get:9 https://repo.clickhouse.com/deb/stable main/ Release.gpg [836 B]
Get:10 https://repo.clickhouse.com/deb/stable main/ Packages [230 kB]
Fetched 231 kB in 2s (128 kB/s)
Reading package lists... Done

xubuntu@xubuntu:~$ sudo apt-get install -y clickhouse-server clickhouse-client
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  clickhouse-common-static
Suggested packages:
  clickhouse-common-static-dbg
The following NEW packages will be installed:
  clickhouse-client clickhouse-common-static clickhouse-server
0 upgraded, 3 newly installed, 0 to remove and 192 not upgraded.
Need to get 216 MB of archives.
After this operation, 711 MB of additional disk space will be used.
Get:1 https://repo.clickhouse.com/deb/stable main/ clickhouse-common-static <IP_ADDRESS> [215 MB]
Get:2 https://repo.clickhouse.com/deb/stable main/ clickhouse-client <IP_ADDRESS> [124 kB]
Get:3 https://repo.clickhouse.com/deb/stable main/ clickhouse-server <IP_ADDRESS> [147 kB]
Fetched 216 MB in 20s (11.0 MB/s)
Selecting previously unselected package clickhouse-common-static.
(Reading database ... 161163 files and directories currently installed.)
Preparing to unpack .../clickhouse-common-static_<IP_ADDRESS>_amd64.deb ...
Unpacking clickhouse-common-static (<IP_ADDRESS>) ...
Selecting previously unselected package clickhouse-client.
Preparing to unpack .../clickhouse-client_<IP_ADDRESS>_all.deb ...
Unpacking clickhouse-client (<IP_ADDRESS>) ...
Selecting previously unselected package clickhouse-server.
Preparing to unpack .../clickhouse-server_<IP_ADDRESS>_all.deb ...
Unpacking clickhouse-server (<IP_ADDRESS>) ...
Setting up clickhouse-common-static (<IP_ADDRESS>) ...
Setting up clickhouse-server (<IP_ADDRESS>) ...
ClickHouse binary is already located at /usr/bin/clickhouse
Symlink /usr/bin/clickhouse-server already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-server to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-client already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-client to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-local already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-local to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-benchmark already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-benchmark to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-copier already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-copier to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-obfuscator already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-obfuscator to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-git-import to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-compressor already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-compressor to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-format already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-format to /usr/bin/clickhouse.
Symlink /usr/bin/clickhouse-extract-from-config already exists but it points to /clickhouse. Will replace the old symlink to /usr/bin/clickhouse.
Creating symlink /usr/bin/clickhouse-extract-from-config to /usr/bin/clickhouse.
Creating clickhouse group if it does not exist.
 groupadd -r clickhouse
Creating clickhouse user if it does not exist.
 useradd -r --shell /bin/false --home-dir /nonexistent -g clickhouse clickhouse
Will set ulimits for clickhouse user in /etc/security/limits.d/clickhouse.conf.
Creating config directory /etc/clickhouse-server/config.d that is used for tweaks of main server configuration.
Creating config directory /etc/clickhouse-server/users.d that is used for tweaks of users configuration.
Config file /etc/clickhouse-server/config.xml already exists, will keep it and extract path info from it.
/etc/clickhouse-server/config.xml has /var/lib/clickhouse/ as data path.
/etc/clickhouse-server/config.xml has /var/log/clickhouse-server/ as log path.
Users config file /etc/clickhouse-server/users.xml already exists, will keep it and extract users info from it.
Creating log directory /var/log/clickhouse-server/.
Creating data directory /var/lib/clickhouse/.
Creating pid directory /var/run/clickhouse-server.
 chown -R clickhouse:clickhouse '/var/log/clickhouse-server/'
 chown -R clickhouse:clickhouse '/var/run/clickhouse-server'
 chown  clickhouse:clickhouse '/var/lib/clickhouse/'
 groupadd -r clickhouse-bridge
 useradd -r --shell /bin/false --home-dir /nonexistent -g clickhouse-bridge clickhouse-bridge
 chown -R clickhouse-bridge:clickhouse-bridge '/usr/bin/clickhouse-odbc-bridge'
 chown -R clickhouse-bridge:clickhouse-bridge '/usr/bin/clickhouse-library-bridge'
Enter password for default user:
Password for default user is saved in file /etc/clickhouse-server/users.d/default-password.xml.
Setting capabilities for clickhouse binary. This is optional.
 chown -R clickhouse:clickhouse '/etc/clickhouse-server'

ClickHouse has been successfully installed.

Start clickhouse-server with:
 sudo clickhouse start

Start clickhouse-client with:
 clickhouse-client --password

Synchronizing state of clickhouse-server.service with SysV service script with /lib/systemd/systemd-sysv-install.
Executing: /lib/systemd/systemd-sysv-install enable clickhouse-server
Created symlink /etc/systemd/system/multi-user.target.wants/clickhouse-server.service → /etc/systemd/system/clickhouse-server.service.
Setting up clickhouse-client (<IP_ADDRESS>) ...
Processing triggers for systemd (245.4-4ubuntu3.11) ...

xubuntu@xubuntu:~$ sudo service clickhouse-server start

xubuntu@xubuntu:~$ clickhouse-client
ClickHouse client version <IP_ADDRESS> (official build).
Connecting to localhost:9000 as user default.

If you have installed ClickHouse and forgot password you can reset it in the configuration file.
The password for default user is typically located at /etc/clickhouse-server/users.d/default-password.xml
and deleting this file will reset the password.
See also /etc/clickhouse-server/users.xml on the server where ClickHouse is installed.

Code: 516. DB::Exception: Received from localhost:9000. DB::Exception: default: Authentication failed: password is incorrect or there is no user with such name. (AUTHENTICATION_FAILED)

xubuntu@xubuntu:~$ clickhouse-client --password
ClickHouse client version <IP_ADDRESS> (official build).
Password for user (default):
Connecting to localhost:9000 as user default.
Connected to ClickHouse server version 22.1.3 revision 54455.

xubuntu :) Bye.
xubuntu@xubuntu:~$

Moved to https://github.com/ClickHouse/ClickHouse/pull/34270


Forkid timestamp

PR description
Fixed Issue(s)

This implements https://github.com/hyperledger/besu/issues/4793
Documentation

[ ] I thought about documentation and added the doc-change-required label to this PR if
updates are required.

Changelog

[ ] I thought about the changelog and included a changelog update if required.

Closed in favour of https://github.com/hyperledger/besu/pull/4841


Type name DateTime is ambiguous between DateTime and System.DateTime.

Found an issue while running the following through the updated CQL-to-ELM translator:
parameter "Measurement Period" Interval<DateTime>
The following errors occur:
[13:41, 13:48] Type name DateTime is ambiguous between DateTime and System.DateTime. [13:32, 13:49] org.hl7.elm.r1.Null cannot be cast to org.hl7.elm.r1.TypeSpecifier [13:1, 13:49] org.hl7.elm.r1.Null cannot be cast to org.hl7.elm.r1.TypeSpecifier
Per spec, this should be allowed. If I update to the following, the translator does not error:
parameter "Measurement Period" Interval<System.DateTime>
Thoughts?

The problem was that there were some incorrect redeclarations of primitive types in the QDM model info, resulting in the ambiguous types being present in the model. I've fixed the model info for QDM 5.0 and 5.0.1.


Governance Configurations are missing in Console (since 5.12.0-m18)

Describe the issue:
In IS-5.12.0-m17 console's manage tab contains these governance configurations

from m18 -to m27 they are not visible in the console app

How to reproduce:
Get 5.12.0-m18 or the latest pack
Log in to console and go to the Manage tab

This has been resolved by https://github.com/wso2/identity-apps/pull/2226


fix: Error fetching metadata when sst start boots up

I actually fixed this already - the issue Frank was seeing was because the WS server was available immediately but the proxy was not.  I started them both together in the CLI - however this PR is a good change anyway


Running the build in local k8s with minikube

Running the build in local k8s with minikube.
Fix resolving dependencies after the change in scala artifacts binary version scheme.


Hostboot broke op-build

This commit from November breaks building the Palmetto pnor https://github.com/open-power/hostboot/commit/02353bc77297c319cdb02b6f83473c4f533560b3
Reverting it fixes the build.
###############################/scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/hostboot_build_images//buildpnor.pl --pnorOutBin /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/pnor/palmetto.pnor --pnorLayout /scratch/joel/op-build/output/build/openpower-pnor-40e407735b317d0174645b7543e0a72019709ce2/defaultPnorLayoutSingleSide.xml --binFile_HBD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//PALMETTO_HB.targeting.bin.ecc --binFile_SBE /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//venice_sbe.img.ecc --binFile_SBEC /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//centaur_sbec_pad.img.ecc --binFile_WINK /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//p8.ref_image.hdr.bin.ecc --binFile_HBB /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hostboot.header.bin.ecc --binFile_HBI /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hostboot_extended.header.bin.ecc --binFile_HBRT /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hostboot_runtime.header.bin.ecc --binFile_HBEL /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hbel.bin.ecc --binFile_GUARD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//guard.bin.ecc --binFile_PAYLOAD /scratch/joel/op-build/output/images/skiboot.lid --binFile_BOOTKERNEL /scratch/joel/op-build/output/images/zImage.epapr --binFile_NVRAM /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//nvram.bin.ecc --binFile_MVPD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//mvpd_fill.bin.ecc --binFile_DJVPD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//djvpd_fill.bin.ecc --binFile_CVPD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//cvpd.bin.ecc --binFile_ATTR_TMP /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//attr_tmp.bin.ecc --binFile_ATTR_PERM /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//attr_perm.bin.ecc --binFile_OCC /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/occ/occ.bin.ecc --binFile_FIRDATA /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//firdata.bin.ecc --binFile_CAPP /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//cappucode.bin.ecc --binFile_VERSION /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_version/openpower-pnor.version.txt --fpartCmd "fpart" --fcpCmd "fcp"
TRACE:  main::loadPnorLayout: metadata: imageSize = 33554432, blockSize=4096, arrangement = A-D-B, numOfSides: 1, sideSize: 33554432, tocSize: 32768
TRACE: A-D-B: side:A HBB:32927744, primaryTOC:0, backupTOC:33521664, golden: no
TRACE: Done checkSpaceConstraints
TRACE: createPnorPartition:: 33521664
TRACE: createPnorImg:: 33521664
TRACE: createPnorPartition:: 0
TRACE: createPnorImg:: 0
TRACE: fpart --target /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/pnor/palmetto.pnor --partition-offset 33521664 --add --offset 33521664 --size 32768 --name BACKUP_PART --flags 0x0
fpart: unexpected : ../src/libffs.c(968) : (code=-1) 'BACKUP_PART' at offset 33521664 and size 32768 overlaps 'part' at offset 33521664 and size 4096
ERROR: Call to add partition BACKUP_PART failed. Aborting! at /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/hostboot_build_images//buildpnor.pl line 534.
Error running command: /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/hostboot_build_images//buildpnor.pl --pnorOutBin /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/pnor/palmetto.pnor --pnorLayout /scratch/joel/op-build/output/build/openpower-pnor-40e407735b317d0174645b7543e0a72019709ce2/defaultPnorLayoutSingleSide.xml --binFile_HBD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//PALMETTO_HB.targeting.bin.ecc --binFile_SBE /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//venice_sbe.img.ecc --binFile_SBEC /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//centaur_sbec_pad.img.ecc --binFile_WINK /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//p8.ref_image.hdr.bin.ecc --binFile_HBB /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hostboot.header.bin.ecc --binFile_HBI /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hostboot_extended.header.bin.ecc --binFile_HBRT /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hostboot_runtime.header.bin.ecc --binFile_HBEL /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//hbel.bin.ecc --binFile_GUARD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//guard.bin.ecc --binFile_PAYLOAD /scratch/joel/op-build/output/images/skiboot.lid --binFile_BOOTKERNEL /scratch/joel/op-build/output/images/zImage.epapr --binFile_NVRAM /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//nvram.bin.ecc --binFile_MVPD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//mvpd_fill.bin.ecc --binFile_DJVPD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//djvpd_fill.bin.ecc --binFile_CVPD /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//cvpd.bin.ecc --binFile_ATTR_TMP /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//attr_tmp.bin.ecc --binFile_ATTR_PERM /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//attr_perm.bin.ecc --binFile_OCC /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/occ/occ.bin.ecc --binFile_FIRDATA /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//firdata.bin.ecc --binFile_CAPP /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_pnor_scratch//cappucode.bin.ecc --binFile_VERSION /scratch/joel/op-build/output/host/usr/powerpc64le-buildroot-linux-gnu/sysroot/openpower_version/openpower-pnor.version.txt --fpartCmd "fpart" --fcpCmd "fcp". Nonzero return code of (256) returned.
package/pkg-generic.mk:267: recipe for target '/scratch/joel/op-build/output/build/openpower-pnor-40e407735b317d0174645b7543e0a72019709ce2/.stamp_images_installed' failed
make: *** [/scratch/joel/op-build/output/build/openpower-pnor-40e407735b317d0174645b7543e0a72019709ce2/.stamp_images_installed] Error 1
make: Leaving directory '/scratch/joel/op-build/buildroot'

The script and ffs itself are generally awful.
I'm working on replacing it all with simple calls to pflash. I may add 100 LoC to pflash, but otherwise remove 20,000 of ffs garbage and 994 lines of perl.
Also, the process is non deterministic in some other ways too. Not fun times.

Thanks @stewart-ibm.
How did this change get merged into op-build?

Good question. You'd expect the jenkins to catch it. But....

We compile op-build master and master-next for all 4 platforms almost every day so is it just a timing issue?  @scprek and @jazurin please take a look.

It seems to be a bug in the sorting of some map in the Perl script. Depending on the system you have a different probability of success. We haven't seen it fail on RHEL but do see it fail on ppc64le Ubuntu. I had a proposed fix but it didn't seem to resolve everything.
Patrick
Sent from my iPhone

On Dec 16, 2015, at 8:21 AM, Dan<EMAIL_ADDRESS>wrote:
We compile op-build master and master-next for all 4 platforms almost every day so is it just a timing issue? @scprek and @jazurin please take a look.
—
Reply to this email directly or view it on GitHub.

Stewart,
Before you spend a lot of time moving the script over to pflash, we need to have a discussion with regards to FSP image building. The reason why we are using ffs (and this ugly Perl script) to begin with is because the FSP build system already relies on having ffs for its own partition building.
Before we deprecate this script we have to get the fsp team to support building with pflash.
Patrick
Sent from my iPhone

On Dec 16, 2015, at 12:07 AM, Stewart Smith<EMAIL_ADDRESS>wrote:
The script and ffs itself are generally awful.
I'm working on replacing it all with simple calls to pflash. I may add 100 LoC to pflash, but otherwise remove 20,000 of ffs garbage and 994 lines of perl.
Also, the process is non deterministic in some other ways too. Not fun times.
—
Reply to this email directly or view it on GitHub.

My comments are all in the pull request #45

@shenki could you please let us know what operating system you are using to build op-build?  Is it RHEL, ppc64le Ubuntu, or something else?   As @williamspatrick said, we saw similar failures on ppc64le Ubuntu.  The failures were caught by a nightly Jenkins job that runs a master-next build test on ppc64le Ubuntu.  But the job is not part of CI, though we might consider adding it to the CI pipeline.  Thanks.

Hey @jazurin, I'm seeing the failure on our public Jenkins instance at https://openpower.xyz.
This runs the jobs inside an Ubuntu 15.10 amd64 Docker container.
https://openpower.xyz/job/openpower-op-build/distro=ubuntu,target=palmetto/lastBuild/console
I also see the failure on my work laptop, which is has an Ubuntu 16.04 toolchian (gcc 5.3, binutils<IP_ADDRESS>51211-0ubuntu2, perl v5.20.2).
I think that adding an Ubuntu builder to your CI pipeline would be very useful. Those of us on the LTC side of the fence are using modern Ubuntu and Fedora installs for our development machines as well as our own CI.
Looking at bug reports from OpenPower developers in the community, it appears Ubuntu is the most popular option, so we should ensure that they have the best chance possible of building the firmware.
Ping me, @stewart-ibm  or @csmart on IRC if you would like to reuse any of the infrastructure we have created for doing builds.

I have also seen this on Fedora 23 x86-64, Debian stable x86-64 and various Ubuntu docker containers in a PowerKVM guest on ppc64el.

@williamspatrick We could leave the script there for FSP so they don't have to switch at the same time and can do so if they wish and on their own schedule.


[doc] Improve tls authentication documentation.

Motivation
According to current documentation, for a beginner can't finish each step smoothly. So add some keys that might go wrong at these steps.
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

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

@codelipenghui Would you please help check @sijie 's comments?

@sijie @jiazhai I'm already updated the document, PTAL.

run Integration Tests
run Java8 Tests

run Integration Tests


[lldb-dotest] Set the --swift-compiler

This sets the swift compiler in the lldb-dotest wrapper. We have the
path to swiftc avaialable in CMake, so it is just a matter of updating
the python wrapper.

@swift-ci please test


Headers key should be in lowercase

Describe the bug
I tried test with code that read custom headers and use lowercase version of header key.
To Reproduce
Steps to reproduce the behavior:
// in test file
request.setHeader({ 'FOO-HEADER': 'foo', 'BAR-HEADER': 'bar'})

// in actual code
const fooHeader = request.headers['foo-header'] // return null
const barHeader = request.headers['BAR-HEADER'] // return 'bar'

Expected behavior
Headers should be in lowercase, according to node specification
node-mocks-http already support this
Environment (please complete the following information):

Node: 10.16.x
Express: ^4.17
Jest: ^24.9
Jest-Express: 1.10.1

request.ts:108
if (options.headers) {
    const o = options.headers;
    const headers = Object.keys(o).reduce((c, k) => (c[k.toLowerCase()] = o[k], c), {});
   this.headers = Object.assign({}, headers);
}

I'm new in typescript, when I tried to modify this line, I got message
Element implicitly has an 'any' type because expression of type 'string' can't be used to index type '{}'.   No index signature with a parameter of type 'string' was found on type '{}'.ts(7053)

Awesome @dels07 thank you for the work you put in on fixing this. 👍


Workspace extension has an activation time even when not installed

This extension wants me to install it, also has a reload, and has a restart button. The restart button doesn't do anything.
This is just in the VS Code repo

Can you please share the following file

F1 > Open Extensions Folder
Share extensions.json file

What do you see in Installed Extensions view?

Installed

Running view

Hover of restart

Ok, so it does appear I have this extension installed. And it wants me to restart it to uninstall it? And then to install it as a workspace extension? Clicking the restart button doesn't do anything and I didn't request the extension be uninstalled

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Remove version: sourcegit-scm.SourceGit version 8.34

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/sourcegit-scm/sourcegit/releases/download/v8.34/sourcegit_8.34.win-arm64.zip
OK
(Automated message - build 923)

https://github.com/microsoft/winget-pkgs/pull/185471#issuecomment-2436036384

The reason for deleting the old version is:

SourceGit is not a library or SDK, but simple GIT GUI client for end users. There is no point in keeping an older version with buggy features.
SourceGit is currently released once a week, which is relatively frequent. Even if there are functional bugs, it will iterate rapidly in next versions.
Make sure that the winget-pkgs repository only retains meaningful files.

Please feel free to disregard this pull request.


Dynamic configuration as part of easy deployment instructions

To get started we're forwarding people to these docs:

Easy deployment
Enable dynamic configuration (from JW Dashboard)

Suggestion: make 'enable dynamic configuration' part of the easy deployment steps.

I wouldn't suggest to customers that they turn on the dynamic unsafe option, but maybe we can add better details on how to add the ID's of the configs they want to work with.

This is complete with the ini file architecture


Use LiteUser in lots of places in the frontend

The frontend currently loads the user's whole profile in most cases, even if all it needs is the name + avatar. We discussed this at a few dev meetings, and for now I've implemented a GetLiteUser RPC that returns just the minimal info needed in the majority of cases in the backend. The main places where this does NOT work are the profile itself, and the map (that's a separate exercise).
In addition to the lite user RPC, there is also the GetLiteUsers RPC that gets many users: this allows fetching up to 200 users at a time in one backend call, which should significantly reduce the number of calls required e.g. when opening the map page.
This issue requires:

Creating a parallel useLiteUser(s) hook similat to the existing useUser(s)  hook, and
Updating many uses of the latter hook to the new former hook.

https://github.com/Couchers-org/couchers/blob/711efe78776efad8ccb331fda32eb47d6b1af13c/app/proto/api.proto#L22-L28

@nabramow : the getLiteUsers API call is in this ticket.
Related to #4951


Error running export to tf colab

Hi! I tried to run you colab for exporting OWL-ViT to tensorflow, but got the following error:
TypeError: call_module() got an unexpected keyword argument 'function_list'

@guarin thanks


Replace attribute selector note with class selector, class demo & class test

Replaced attribute selector [name=name] inline documentation (as referenced doesn't work in HenrikJoreteg/Happy.js#42 and HenrikJoreteg/Happy.js#52) with class selector documentation and added working example of class selector to demo.html and class input selector to test.js

Sorry this has been sitting for so long, i'm seeking new maintainers, let me know here if you're interested: https://github.com/HenrikJoreteg/Happy.js/issues/72

As of #73, this issue has been resolved and the documentation is accurate.


What happened to the 3.0 branch of package:collection?

I landed https://github.com/dart-archive/collection/pull/289 a while ago, but with the move to core it seems like neither the code nor the branch it was landed on made the move. Where did it end up? Can we get it merged into the main branch at some point?

@lrhn @devoncarew either of you know?  I see work landing in master around the same time - an alternative version of 1.18.0: https://github.com/dart-archive/collection/commit/f949b09b130ddc1cb59fd056e071cecb458e8fb6.
Both the stable version and this branch now require 3.0. I don't see breaking changes in there, so I don't think there would be issues w/ just re-landing these changes. cc @natebosch

Not completely forgotten, but "deprioritized" by other things. I'll try to put it closer to the top of the list.
(Next year!)


Add Sponsors and Vendors to Front Page

Need to have a section of the main page that displays a selection of vendors and sponsors.

@StevePoulen Do you have an idea on how you would like this displayed?

@StevePoulen I have finished this for sponsors. I don't think we need vendors right now but we can address it if we need to add them.


Feature/admin amaretti

Added sidemenu

Hi @SzymonSel, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Updated to work with .NET 8.0 frends process.

Issue 23

Summary by CodeRabbit

Chores

Updated Frends.Regex.Replace component to be compatible with .NET 8.0.
Incremented version number to 1.0.2 to reflect compatibility changes.

I have tested this out and it seems that the problem is in some Frends internals like migrating from .NET 6 to .NET 8. There is nothing in the task code to fix.


Remove the roaring operation functions warnings

In this PR we are just replacing the usages of the roaring operations function by the new operators. This removes a lot of warnings.

Thank you @curquiza 🧮
bors merge


Nightly testing of HF packaging

Keeps rot short-lived.

!ci-nightly-me

Looks like the new jobs still aren't listed in https://buildkite.com/o-1-labs-2/mina-end-to-end-nightlies/builds/1648#018e785c-8047-4e8c-beae-043c96a5a5f8

!ci-nightly-me


dal: prefer pristine jQuery to the django one

First of all i can't see how select2 may work without a jQuery
as a global $. Said that now when i have another copy of jQuery
on top of the django one, select2 will register to $ but dal
will set yl.jQuery to the django one instead.
If we prefer using $ instead they'll both use the same jQuery
instance and magically work.
If there's not jQuery loaded as $ the code will behave as before.
This makes dal working reliably in the admin for me.

Codecov Report

Merging #861 into master will decrease coverage by 0.78%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #861      +/-   ##
==========================================
- Coverage    97.1%   96.31%   -0.79%
==========================================
  Files         124      124
  Lines        2176     2172       -4
==========================================
- Hits         2113     2092      -21
- Misses         63       80      +17

Impacted Files
Coverage Δ

test_project/select2_outside_admin/views.py
78.57% <0%> (-14.29%)
:arrow_down:

test_project/select2_tagging/test_forms.py
93.54% <0%> (-6.46%)
:arrow_down:

src/dal/test/case.py
90.74% <0%> (-5.56%)
:arrow_down:

test_project/select2_taggit/test_forms.py
95% <0%> (-5%)
:arrow_down:

test_project/tests/test_widgets.py
95.55% <0%> (-4.45%)
:arrow_down:

..._project/select2_generic_foreign_key/test_forms.py
95.65% <0%> (-4.35%)
:arrow_down:

test_project/select2_generic_m2m/test_forms.py
91.07% <0%> (-3.58%)
:arrow_down:

src/dal/widgets.py
96.1% <0%> (-2.6%)
:arrow_down:

test_project/select2_generic_foreign_key/urls.py
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Refactor Soc trait and move boards into component

This PR splits the Soc trait into a Soc and a Board trait, moves all soc- and board-related code into the proper modules and then moves these modules into a new boards component.  This is a preparation for adding the nkpk board.
Side effects:

nrf52 is now used consistently instead of nrf52840.
The board and soc features in boards are additive so it is easier to test changes for all boards.
Some dead code is removed.
This leads to a minor binary size increase for lpc55 and decrease for nrf52 (both < 100 bytes).

awesome refactoring efforts here! lgtm!


Delete cypress and code from project

What:

Removed cypress, tests, configs, and data-cy attributes.

Why:

We are no longer using cypress for integration specs.
Moved to capybara

This is green, please merge when ready


Security Scorecards integration

See https://github.com/ossf/scorecard for more info.
Reviewer checklist

[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure correct labels applied
[ ] Ensure any appropriate documentation has been added or amended

Coverage: 97.79%. Remained the same when pulling 98e6d44fb5667d1fff8ad9034f6eee7930c7d6fd on scorecards into a04053ef7febf69955009d81afc4b723a2b2d28c on main.


Rename profiles according to their roles and add past-deadline event

This PR changes the names of the profiles according to their roles and adds the summer camp as new event used as past-deadline-event. This will improve the experience of the demo.
I could not test if this changes work, because I have no setup of the project, so please check if I did something wrong

Coverage remained the same at 100.0% when pulling 2282d45e60d11f87397e109df6065d2b21cf92e7 on adjust_sample_data into 7d9ab33e3d56aa2a180abe3dd5c83e1454649c30 on dev.

Coverage remained the same at 100.0% when pulling 2282d45e60d11f87397e109df6065d2b21cf92e7 on adjust_sample_data into 7d9ab33e3d56aa2a180abe3dd5c83e1454649c30 on dev.

I am not sure, what I did wrong, but it does not find "event_summer_camp" which I defined in events.rb
https://travis-ci.org/hpi-swt2/workshop-portal/builds/197920436

@tobiduer If you can't test things before opening a PR, please consider doing pair-programming with a developer if you feel like you want to add some code as well. We collectively voted on things that need to be done in order to open a PR in the wiki.

Coverage remained the same at 100.0% when pulling caf067c10fc89dc38702b271b0dce0dcaed01a51 on adjust_sample_data into 3f136cab0e3a9dd7b8905df575781f4654f0ac75 on dev.

Coverage remained the same at 100.0% when pulling caf067c10fc89dc38702b271b0dce0dcaed01a51 on adjust_sample_data into 3f136cab0e3a9dd7b8905df575781f4654f0ac75 on dev.

@bjrne wishes to create a new US that developers should implement instead of POs.


Match GREATEST/LEAST function behavior to other DBs

Description
Change the behavior of the GREATEST / LEAST functions to be similar to how it is implemented in other databases (as functions instead of aggregators). The GREATEST/LEAST functions are not in the SQL standard, but users will expect behavior similar to what other databases provide.
References:

PostgreSQL: https://www.postgresql.org/docs/9.5/functions-conditional.html
Vertica: https://www.vertica.com/docs/9.2.x/HTML/Content/Authoring/SQLReferenceManual/Functions/String/GREATEST.htm
MySQL: https://dev.mysql.com/doc/refman/8.0/en/comparison-operators.html#function_greatest
Oracle: https://docs.oracle.com/cd/E11882_01/server.112/e41084/functions087.htm#SQLRF00657

This PR has:

[x] been self-reviewed.
[x] added documentation for new or modified features or behaviors.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths.

can you explain what are those other databases  and if it is not a standard SQL why we have to expose the same name ?
How about actual user using this ? would this be backward compatible ?

can you explain what are those other databases  and if it is not a standard SQL why we have to expose the same name ?

I've updated the description with references to what some other databases do. The functions aren't part of the standard, but they're available in many databases.

How about actual user using this ? would this be backward compatible ?

The current behavior was added by https://github.com/apache/druid/pull/8719. Since that change is not in a druid release yet, we don't need to worry about backward compatibility.

@b-slim Let me know if you have any comments. (I'd like to get this in before the 0.18.0 branch cut in a few days.)

+1


chl reg lookup - memory error

When running the chl lookup with lots of records - it hits a silent error when it runs out of memory. The update "ALL" option does not work, instead break up the records by id ranges (groups of 100k records works).

Filters on toolbox make this a nonissue


2FA methods needs more context

Users do not know what to do with the information presented in the 2FA methods section.
Help them make a decision about what they are looking at.
Make a "security" scale

I am assuming that the scale would be if they have any 2FA they are one value but if they use a particular type of security method for 2FA then they are max on the scale.

I think you're getting closer to my thoughts.
I was thinking that if the service offers multiple options, then the strongest 2FA option would be considered for its scoring. If it defaults to using 2FA when you sign up, thats the ultimate as well. but otherwise, its about the best case scenario for that service, as we will not know what 2FA service the user of the notifier is using with that site.
This brought up another thought though, perhaps in the settings of the extension a user can indicate which services they are preferable to using. Maybe they do not care AT ALL! if a site allows SMS 2FA as they themselves do not have a personal mobile number or feel that it is secure enough. Instead they only care about all other 2FA options.

That's a great thought to explore!
I've created an issue to come back around to it once we've fleshed out some of the higher priority issues we're currently tackling.
With regards to your first point about scoring on strongest 2FA options available, could you walk me through your thoughts here? Sorry if I am misunderstanding something. I want to make sure I incorporate all feedback. I'm particularly curious about what problem this solution is solving and what value users will derive from it?
In my mind, the biggest value is that it encourages service providers to implement stronger 2FA options in order to improve appearances.
I'm also only thinking about this in terms of JUST the extension and the limited space we have on the dropdown. The original issue was intended only for the extension dropdown with the goal of helping users make themselves more secure. (I should have communicated this more clearly in the description).
But maybe this would be useful on the notification that slides out as there is even less space to help users choose the best 2FA option. Something like "website.com has strong 2FA options available" or "website has weak 2FA options available"...


always read toml file using utf-8

The spec says:

A TOML file must be a valid UTF-8 encoded Unicode document.

Builtin open() in Python 3 uses locale.getpreferredencoding(False) by default. It may differ from utf-8. To read a TOML file, the utf-8 encoding should be passed explicitly.

Thanks for the PR! I did a bit of testing and it looks like it works in both 2 and 3.


[ADAP-614] [Bug] unique_key config for snapshots

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

What page(s) or areas on docs.getdbt.com are affected?
unique_key config for snapshots

What changes are you suggesting?
I think for the snapshot hudi table, the primaryKey should be ‘dbt_scd_id’, which is used in the ‘merge into on’ statement generated by ‘dbt snapshot’ command. And the dbt_scd_id column is related to the primary key of the source table and the update_at referenced column when we reference to timestamp strategy.
But now the primaryKey of the snapshot CTAS hudi table, which is created as we run the first ‘dbt snapshot’ command, is the same with the primary key of the source table.
In my experiment, I change two parts of the sql generated by the dbt, one is the primaryKey of the snapshot CTAS hudi table using ‘dbt_scd_id’, the other one is the ‘insert [referenced columns]’ in the 'merge into insert ’ statement instead of ‘insert *’. After these changes, I get the scd2 table, stored in hudi.
Additional information
In my solution, I copy the files include/spark/macros/adapters.sql and include/spark/macros/materializations/snapshot.sql  to the macro directory in my own project

And, I change the related code, that works.

@darcyMaJiyue which of these two are you requesting / suggesting?

update the implementation of spark__options_clause and/or spark__snapshot_merge_sql in dbt-spark
update the docs for unique_key

My colleague @dataders will help determine next steps either way.

oh, It's a long time. @dbeatty10  thanks for your response. I'm suggesting the first one.

1, update the implementation of sparkt__options_clause
On Fri, Jun 9, 2023 at 10:23 PM Doug Beatty @.***>
wrote:

@darcyMaJiyue
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_darcyMaJiyue&d=DwMCaQ&c=8lBT5Jra4Bm5rFhLVR7k1wx3__gIUgr523Abjhgq6Gg&r=SD8zkOIA4htuCih_kib0-LvIkr6Lvmpu5y9sV4rAkEo&m=8bNobNxRwmCH_ICflm6NpDRZeOfyLDV9bSfDS6zaP-FEMU3J3kT1Y_gvd2DExgyJ&s=xIe7zqzz6vpHwadqEym2KO9clLrVg6yMz0dRQ250RFA&e=
which of these two are you requesting / suggesting?

update the implementation of spark__options_clause
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dbt-2Dlabs_dbt-2Dspark_blob_e741034160444eb7aa06aef7550a366cdcacc913_dbt_include_spark_macros_adapters.sql-23L41&d=DwMCaQ&c=8lBT5Jra4Bm5rFhLVR7k1wx3__gIUgr523Abjhgq6Gg&r=SD8zkOIA4htuCih_kib0-LvIkr6Lvmpu5y9sV4rAkEo&m=8bNobNxRwmCH_ICflm6NpDRZeOfyLDV9bSfDS6zaP-FEMU3J3kT1Y_gvd2DExgyJ&s=zmo5tYDtBUeurvLA5kMg7NOw2VBtmAODxqx48fOKRfU&e=
and/or spark__snapshot_merge_sql
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dbt-2Dlabs_dbt-2Dspark_blob_e741034160444eb7aa06aef7550a366cdcacc913_dbt_include_spark_macros_materializations_snapshot.sql-23L15&d=DwMCaQ&c=8lBT5Jra4Bm5rFhLVR7k1wx3__gIUgr523Abjhgq6Gg&r=SD8zkOIA4htuCih_kib0-LvIkr6Lvmpu5y9sV4rAkEo&m=8bNobNxRwmCH_ICflm6NpDRZeOfyLDV9bSfDS6zaP-FEMU3J3kT1Y_gvd2DExgyJ&s=7BoPQ9G_Yn5XsjUZFwqRM3F5obhe_Scgz1mfVhiJavo&e=
in dbt-spark
update the docs for unique_key
https://urldefense.proofpoint.com/v2/url?u=https-3A__docs.getdbt.com_reference_resource-2Dconfigs_unique-5Fkey&d=DwMCaQ&c=8lBT5Jra4Bm5rFhLVR7k1wx3__gIUgr523Abjhgq6Gg&r=SD8zkOIA4htuCih_kib0-LvIkr6Lvmpu5y9sV4rAkEo&m=8bNobNxRwmCH_ICflm6NpDRZeOfyLDV9bSfDS6zaP-FEMU3J3kT1Y_gvd2DExgyJ&s=1RjytOzCWH3dwnJ4NcYTwttoMDSOLCSaf68SUwCP95o&e=

My colleague @dataders
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dataders&d=DwMCaQ&c=8lBT5Jra4Bm5rFhLVR7k1wx3__gIUgr523Abjhgq6Gg&r=SD8zkOIA4htuCih_kib0-LvIkr6Lvmpu5y9sV4rAkEo&m=8bNobNxRwmCH_ICflm6NpDRZeOfyLDV9bSfDS6zaP-FEMU3J3kT1Y_gvd2DExgyJ&s=5pghM6yHDyhG5yhYlluvNWrJdgSG-WRlxbM6YTY7lIg&e=
will help determine next steps either way.
—
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dbt-2Dlabs_dbt-2Dspark_issues_801-23issuecomment-2D1584664062&d=DwMCaQ&c=8lBT5Jra4Bm5rFhLVR7k1wx3__gIUgr523Abjhgq6Gg&r=SD8zkOIA4htuCih_kib0-LvIkr6Lvmpu5y9sV4rAkEo&m=8bNobNxRwmCH_ICflm6NpDRZeOfyLDV9bSfDS6zaP-FEMU3J3kT1Y_gvd2DExgyJ&s=ss4yScxpf3_480m0jatn8PSe8fuK_xOCKcbbROH3vVE&e=,
or unsubscribe
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_A33REEY7AKUSAS6KHWYA4JDXKMWVVANCNFSM6AAAAAAZAW356A&d=DwMCaQ&c=8lBT5Jra4Bm5rFhLVR7k1wx3__gIUgr523Abjhgq6Gg&r=SD8zkOIA4htuCih_kib0-LvIkr6Lvmpu5y9sV4rAkEo&m=8bNobNxRwmCH_ICflm6NpDRZeOfyLDV9bSfDS6zaP-FEMU3J3kT1Y_gvd2DExgyJ&s=RWmKpX0ieaNvMsOuydk__IOBRtujcvEtLTK65MjA60k&e=
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry for the long time @darcyMaJiyue.
Could you share reproducible example?
Could you provide an example snapshots SQL + config so that we can try to reproduce the issue that you are seeing?
We'd be looking for two things:

Code of your snapshots SQL + config
Commands to execute

We'd need both in order to add new tests to dbt-spark that trigger this error so that we can be sure any fix works as we expect.
Example
Here's what those two things should look similar to:
{% snapshot orders_snapshot %}

{{
    config(
      target_database='analytics',
      target_schema='snapshots',
      unique_key='id',
      file_format='hudi'

      strategy='timestamp',
      updated_at='updated_at',
    )
}}

select * from {{ source('jaffle_shop', 'orders') }}

{% endsnapshot %}

And then also whichever command(s) trigger the error. So maybe in your case running the snapshot twice in a row causes the issue?
dbt snapshot -s orders_snapshot
dbt snapshot -s orders_snapshot

Could you confirm the necessary code changes?
To be clear, is this the only change you are proposing? i.e., remove these lines from spark__options_clause:
9,10d8
<     {%- elif options is not none and 'primaryKey' in options and options['primaryKey'] != unique_key -%}
<       {{ exceptions.raise_compiler_error("unique_key and options('primaryKey') should be the same column(s).") }}

Or is there some change needed in spark__snapshot_merge_sql, e.g.
2a3,4
>     {%- set insert_cols_csv = insert_cols join(', ') -%}
>
14c16,17
<         then insert *
---
>         then insert ({{ insert cols csv }})
>         values ({{ insert_cols_csv}})


Product tests goes out of travis memory

See: https://api.travis-ci.com/v3/job/195820691/log.txt
2019-04-26 15:46:35 INFO: [333 of 422] sql_tests.testcases.orc.select_varchar_orc_dictionary (Groups: varchar)
presto-master_1_620f02239231 | Java HotSpot(TM) 64-Bit Server VM warning: INFO: os::commit_memory(0x00000000a8000000, 301989888, 0) failed; error='Cannot allocate memory' (errno=12)
presto-master_1_620f02239231 | #
presto-master_1_620f02239231 | # There is insufficient memory for the Java Runtime Environment to continue.
presto-master_1_620f02239231 | # Native memory allocation (mmap) failed to map 301989888 bytes for committing reserved memory.
presto-master_1_620f02239231 | # An error report file with more information is saved as:
presto-master_1_620f02239231 | # /docker/volumes/logs/product-tests-presto-jvm-error-file.log
common_presto-master_1_620f02239231 exited with code 1

It failed for PRODUCT_TESTS_BASIC_ENVIRONMENT=true

This might also be related to apparent freezes like
2019-04-28 22:42:02 INFO: [266 of 422] sql_tests.testcases.hive_tpch.q19 (Groups: tpch)

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

I run experiment (https://github.com/prestosql/presto/pull/1064).
PT multinode failed 5 of 10 times:

1 time: master could not connect to master for few minutes
2 times: master failed OOM
2 times: freeze

Then I run the same setup without G1 (hey, G1 isn't suitable for 1-2g heaps) and had 9 of 9 success (1 failure from maven).


Oni portal configuration depends on filesystem layout

ie it expects that the host and docker filesystems have the /opt/... structure - we can try to fix this in our fork if we have time

There is already a partial fix for this in our code


Remove version: Jackett.Jackett version 0.20.2465

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.2465/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Feature request: update & remove notifications

As already mentioned in #215, I would like to  ask for the ability to update and remove existing notifications. This is especially useful in a smart home context because we may want to inform the user about certain required actions (e.g., you forgot to close the back door) and once this is done (door closed), the reminder (= notification) should disappear which also serves as a nice feedback. This way, the notifications can somehow serve as a "TODO list". (It's similar, if not equivalent, to persistent_notification.dismiss).
Thanks for considering!

Somehow 'tag's were helpful since today, right now there's discussion about closing the backend on unsupported features.
It wasn't pure dismiss, but you could overwrite previous message with current status like ("Door were closed').

Yes, but if I got you right, isn't 'overwriting' and updating the same? 🙈
What do you mean by "right now there's discussion about closing the backend on unsupported features."? 🤔

This issue https://github.com/home-assistant/home-assistant-android/issues/349

Great thanks!


Tb-18

Describe your changes

Updated PE file with virtual_network_name
Updated Readme, fixed wording
Removed Network Rules from Storage Account File, Dupicate
Deleted locals as they were not being used
Added account_tier/account_replication_type to locals
Added ResourceName to tags
Removed Private_link_access from Network Rules, not needed
Added smaller network_rules variable, removed all other variables.
Added Data for Existing Subnet in PE
Updated README to correct the module for Management Hub
Added new example basic with PEs.

Issue number
#18
#21
#20
#19
Checklist before requesting a review

[X] The pr title can be used to describe what this pr did in CHANGELOG.md file
[X] I have executed pre-commit on my machine
[X] I have passed pr-check on my machine

Thanks for your cooperation!


Branch color too light

Could you darken the default to black and perhaps increase the weight of the line a bit? Here's an example of a MEGA tree. Font for leaf labels could be Arial  10 or 12.

Yes, definitely!


Leaderboard "10 around me" potentially shows a duplicate entry for the current user

If the user was created prior to version 1.5, they may be listed twice in the 10 around me view of the Leaderboard component.

PR: https://github.com/NationalSecurityAgency/skills-service/pull/683

:+1:


Training with local variables is very slow in distributed tf

I used local variables on worker tasks in a distributed RNN training. Every update go through 3 steps: 1) assign local variables on each worker with PS variables 2) training local variables in several loops/batches, and accumulate the gradients from each loop 3) apply the accumulated gradients to the PS variables. The first and second step is very slow: for step 1, it takes about 12 seconds; for step 2, every loop takes about 6 seconds. If I combine the three steps together (not use local variable and go through one batch data every update), it only takes about 0.7 second. So, I suspect that it is due to the bad performance of local variables.
Environment info
Ubuntu 14.04, CUDA 8.0, tensorflow 0.12.0-rc1

How big are your local variables? There's some slowness in current version if they are too big -- https://github.com/tensorflow/tensorflow/issues/6116 . Also, what does your session.run looks like? If you do something like sess.run(assign_op) then you are fetching result to Python which can be slow (https://github.com/tensorflow/tensorflow/issues/6256), the solution is to do sess.run(assign_op.op)

Yes, my variables are very big, totally larger than 400MB. sess.run(assign_op.op) solved my problem. Thanks a lot.


(@Clas) gen9mnmlc: readds giraf restrict

@Clas: "readds giraf restrict"
Created via Iolanthe on Sun, 10 Mar 2024 03:55:43 GMT.

No changes.


Websocket with headers and cookies

Does android async support headers and cookies for a websocket connection ? if so, how ?

It does not, websocket is kind of channel we're not currently able to support with HttpClient library, use OkHttp or different library to achieve this.


Every first launch - significant lag, UI elements active but anything typed doesn't appear

VSCode Version: Code 1.17.2 (b813d12980308015bcd2b3a2f6efa5c810c33ba5, 2017-10-16T13:59:48.528Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author (truncated)
Version

xml
Dot
1.9.2

gitlens
eam
5.7.1

nuget4code
ffo
0.0.2

gc-excelviewer
Gra
1.1.15

minify
Hoo
0.3.0

Ionide-fsharp
Ion
3.12.1

csharpextensions
jch
1.3.0

hex-rgba-converter
med
1.0.4

mssql
ms-
1.2.0

csharp
ms-
1.13.0

node-debug2
ms-
1.18.5

PowerShell
ms-
1.5.0

team
ms-
1.122.0

debugger-for-chrome
msj
3.5.0

yo
sam
0.9.3

cucumber
ste
0.9.6

vscode-react-native
vsm
0.5.2

change-case
wma
1.0.0

(4 theme extensions excluded)

Steps to Reproduce:

Reproduces without extensions: Yes/No

I wonder if this issue is being caused by an installed extension. Can you try to run VS Code without extensions? From the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If you see it is an issue with the extension, please file it against the extension repository itself.