ci: zephyr: execute (real hardware) HIL tests in matrix jobs

So far bash 'for' loop was used to iterate through all tests, then build
them and run pytest. This produced massive amount of logs and as a result
was very hard to find a problem when such occurred.
Add job (matrix) that dynamically generates matrix based on the list of
tests to execute. Then use matrix strategy to build those tests and run
pytest on HIL.
Split building and runtime execution into separate steps, so that it
improves:

visual feedback in GitHub Actions UI when investigating failed jobs
no $EXITCODE handling
faster execution of "Re-run failed jobs" when only single test on a
single platform has failed (it makes it more likely to pass)

This PR slightly increases time of the initial run. However since it is very
common that some of those tests do not pass, it is much faster to restart
subset of tests instead of all of them in a loop. In the end this should
reduce time spent on hardware testing.

Can we get some rough numbers on how much time this adds for a complete test run (ignoring retries)?  Let's also identify areas where we can cut down on the per-test overhead. Some of the ones that come to mind (but there could be others):

erasing the board after each run
Setting up python/west dependencies

Can we get some rough numbers on how much time this adds for a complete test run (ignoring retries)?

Number directly calculated from CI, based on succeeded on Oct 10 in 2m 30s strings reported by GitHub (e.g. here https://github.com/golioth/golioth-firmware-sdk/actions/runs/11274763684/job/31359278655?pr=641):

branch
time (s)
time adjusted (s)
% (adjusted)
link

hil-split
1343
1089
170
https://github.com/golioth/golioth-firmware-sdk/actions/runs/11274763684/job/31359278655?pr=641

main
642
642
100
https://github.com/golioth/golioth-firmware-sdk/actions/runs/11284048073/job/31384742968

Adjusted time was based on sams_orange_pi taking around 120s to install Python dependencies, while tims_orange_pi taking only 33s. Also there was one test executed on tims_orange_pi that resulted in long container bring-up (probably cold cache).

There is one nice comparison now to make. Right now there is an issue with nrf9160dk with ota HIL test, which is a nice use case to describe.
This OTA test takes 3min 49s in total, on hil-split branch: https://github.com/golioth/golioth-firmware-sdk/actions/runs/11953157487/job/33320699596?pr=641
All tests (including failing OTA test) take 20min 42s, on another branch: https://github.com/golioth/golioth-firmware-sdk/actions/runs/11942272439/job/33326973074?pr=682
This means that restart will be much faster in execution. This means less time spent on blocking other GitHub Actions invocations.
Another important feature is that with the first link (hil-split branch) it is much easier to figure out which test has failed (OTA), whereas with the second link it is not that easy, as whole step log needs to be analyzed in order to see that only OTA test has failed there.


Implement termux colorschemes

This implements basic support for termux's colorschemes (~/.termux/colors.properties) through the new option terminal.colors. It takes an attribute set of strings and naively converts that to the key-value format termux expects, e.g.
terminal.colors = rec {
  background = color0;
  foreground = color15;
  cursor = color15;

  color0 = "#000000";
  # color1-14 here :)
  color15 = "#FFFFFF";
};

would become
background=#000000
color0=#000000
color15=#FFFFFF
cursor=#FFFFFF
foreground=#FFFFFF

I wasn't sure whether to use attrsOf or lazyAttrsOf, so I chose the latter as a guess.
I've mostly just copied over the implementation from what fonts do, I hope that's correct.
There are a couple issues with this as it stands:

Colors aren't strictly checked, you can input any string.
Attributes aren't checked, you can input any attribute (whereas I believe termux only ever uses background, foreground, cursor and color0-15).
There isn't any option to use a preexisting file.
The changes don't immediately apply and require opening a new session, but as I understand it that's blocked by #221.
Since both font and colors share most of the same code, maybe it could be pulled out into a separate function/functions?
My documentation here could probably be better.

It works enough for my usecase, but there's certainly a lot that could be improved.

Hello and sorry for not getting to this PR for so long.
I took the liberty of rebasing, bumping copyright year, squashing, limiting allowed attrnames and adding some tests. Please take a look.

One more thing we'll need is a changelog entry.


repl support

谢谢大佬的api
https://github.com/dsdanielpark/Bard-API/assets/91452062/393bda5b-78f0-4ddd-905d-c64c868698a5

This resolves #13


AutoSave not consistently syncing changes to Markdown files saved to OneDrive

VSCode Version: Code 1.4.0 (6276dcb0ae497766056b4c09ea75be1d76a8b679, 2016-08-04T16:45:31.680Z)
OS Version: Windows_NT ia32 10.0.14393

I discovered this issue when I opened a file on a second computer and the most recent changes, which were large and important, were not visible. They were still visible on my first computer, where I still had the file open in VSCode.
Note: After I disabled extensions (including vscode-icons and Spelling and Grammar Checker), autoSave seemed to be correctly. But for now I am still hesitant to trust VSCode to autoSave changes.
Steps to Reproduce:

Enable autoSave in Workspace settings. I use "files.autoSave": "afterDelay", and
"files.autoSaveDelay": 10,
Create new and save with a .md extension to OneDrive
Edit file. File changes should be synced automatically to OneDrive given that autoSave is enabled. Open the file on onedrive.com to check if the edits have been synced.

I am very sorry to hear you are having issues with VS Code. Reading your description it seems that the issue originates from an extension and not VS Code itself. Thereby I will close this issue and kindly ask you to report this issue against the extension repository.
Most extensions are on GitHub as well so it should be easy to file it against the extension. If you open an extension on the market place, click on the Support link to find out more.

Thanks again for your feedback!


fix: transfer cache in prepareNextSlot epoch transition

Motivation
While investigating #6063 I found we only transfer cache when verifying block https://github.com/ChainSafe/lodestar/blob/421fd1a8db6b1e4e381b51e5d211b619e6cbe4da/packages/beacon-node/src/chain/blocks/verifyBlock.ts#L66 as part of transition
However at epoch boundary when getPreState() we get the checkpoint state instead of state of previous epoch to save us an epoch transition https://github.com/ChainSafe/lodestar/blob/421fd1a8db6b1e4e381b51e5d211b619e6cbe4da/packages/beacon-node/src/chain/regen/queued.ts#L150
Description

Also transfer cache at prepareNextSlot() when we cache Previous Root Checkpoint State

related to #6063
Test Result
Epoch transition of this branch vs unstable group

type of node
feature branch
unstable

mainnet.
0.96s
1.1s.

64
0.73s
0.81s

16.
3.3s
4s

sm1v
2.15s
3.25s

novc
4.6s.
1.6s

on all types of node I see lower epoch transition time except for novc where unstable is significantly faster, not sure why it is like that but epoch transition on novc is inconsistent between groups so I'd not trust it

after testing for 4 more days, the difference on epoch transition time is quite random (mainnet node is better but others are worse) so we can't count on that metric. @dapplion do you know what other metrics should I look into?

@tuyennhv there are metrics to check how often the validator and balance caches are populated in the state used in state transitions. If the rate of cache hits is higher, then the PR should be merged.
https://github.com/ChainSafe/lodestar/blob/2b5935a35342837496c803a1c58dbd824c73d2cd/packages/state-transition/src/metrics.ts#L11-L14

it really shows that mainnet node has less missed validators node on this branch, that's why I see better epoch transition time on this node

for other goerli nodes, the ratio is just the same

:tada: This PR is included in v1.13.0 :tada:


Added Microsoft ASP.NET Overview

Added Microsoft ASP.NET Overview link: https://docs.microsoft.com/en-us/aspnet/overview

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: We do not accept PRs which only add links when there are already valid reference links for the article.
Thank you and happy coding.


[MR] Markdown column type

Allow obsidian links inside the cells of a new type of column

This issue is covered with 1.0.0


docsUpdate spark_load.md

Proposed changes
Issue Number: close #xxx
https://doris.apache.org/zh-CN/community/design/spark_load/

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Install the Microsoft.Net.Compilers package

Added the package to fix the CSC compiler error when the site first runs.

Linked issue - https://github.com/umbraco/Package.Templates/issues/3

Installing this dependency might have fixed the linked issue (https://github.com/umbraco/Package.Templates/issues/3), but not the actual problem. This actually replaces the compiler that's included in the IDE (e.g. Visual Studio), see: https://github.com/umbraco/Umbraco-CMS/issues/6495.


Skip video folder sd card

is it possible to switch directly from car stream video player from one video folder to another?
Thank You.

Yea, but what about cases where there are folders inside folders and they both have videos in them.
The best solution is to scan through all folders and show everything on the disk as one big list I guess. Something which all music players do


Add timestamp field to SegmentMetadataEvent

Add timestamp field to SegmentMetadataEvent
Description
Every other metrics emitted by Druid has a timestamp field. The fact that the SegmentMetadataEvent metric does not have a timestamp field can cause problem for the metric system like Druid. If you are ingesting Druid's emitted to Druid, then timestamp is very important. If you are running Druid version earlier than 27 (which doesn't have https://github.com/apache/druid/pull/14413), then ingestion will fail if there is no timestamp. While if you are running Druid version later than 27, you may be dropping these no timestamp metrics. (Note that the above is assuming you do not have missingValue set for your Druid's supervisor that is ingesting Druid emitted metrics).
Anyway, let's just add a timestamp so that all metrics are consistent.
This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

@maytasm , there are some test failures. Please take a look.


config(dm): add on-duplicate-logical, on-duplicate-physical

Signed-off-by: lance6716<EMAIL_ADDRESS>
What problem does this PR solve?

Issue Number: ref #3510
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
add on-duplicate-logical, on-duplicate-physical for load unit configuration

/run-dm-integration-test

/cc @buchuitoudegou @okJiang

ptal

ptal @buchuitoudegou @okJiang

/merge


hi i want to use your model but i have some problem OTL

i take the input data 8 * 17421 in this model but i receive the output data's row is not 17421. so i try to change the test function in exp_diviner but the output is not change. can you help me?
so i want to change the test batch size 1. but when i change the !python code's batch_size=1, but the model's output is 336. can you give me some solution?

View any run.sh demo in the repository, such as
checkpoints/ETTm1/24/run.sh
python -u main.py --model=diviner --data=ETTm1 --predict_length=24 --enc_seq_len=4 --out_seq_len=1 --dec_seq_len=4 --dim_val=48 --dim_attn=24 --dim_attn_channel=48 --n_heads=6 --n_encoder_layers=3 --n_decoder_layers=2 --batch_size=32 --train_epochs=100 --use_gpu --smo_loss --dynamic --early_stop --shuffle --verbose --out_scale --load_check_points=.\checkpoints\ETTm1\24\diviner_checkpoints.ckpt --test
By setting the predict_length paramether, the output length can be changed
Yuguang Yang
@.***
原始邮件
发件人："kim young il"< @.*** >;
发件时间：2023/10/18 14:35
收件人："CapricornGuang/Diviner-Nonstationary-time-series-forecasting"< @.*** >;
抄送人："Subscribed"< @.*** >;
主题：[CapricornGuang/Diviner-Nonstationary-time-series-forecasting] hi iwant to use your model but i have some problem OTL (Issue #4)
i take the input data 8 * 17421 in this model but i receive the output data's row is not 17421. so i try to change the test function in exp_diviner but the output is not change. can you help me?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Besides, I review my code and if you want to explore the code logit of series length, you can follow the below steps:

In data/data_loader.py, any dataset is a subclass of DatasetBasic
DatasetBasic has the attributes of self.enc_seq_len, this is a key variable to control the input length.  In terms of the output length, self.out_seq_len variable might help you

oh really thank you god to bless for you


Fix lockup

The use of assert is questionable here and it keeps us in a blocked state when false.
Also possible fix for #26

I suck at python and it still works with the change so I will assert that you know what you're doing more than I do... (pun intended)

I suck at python and it still works with the change so I will assert that you know what you're doing more than I do... (pun intended)


Add detailed instructions for plugin git repositories

When developing a plugin, we can either use a public git repository or a private one. We can also use any git host.
Clarify how this is possible by giving an absolute URL and the required format.
Thanks to Tom Watt for the pointer, this commit should help others developing plugins and hitting similar pitfalls.

Hi @deniz-lktr, thanks for raising this! It looks like this information is covered under Plugin sources on the Using plugins page. Do you think that's enough information, or is there more you think we should include there? 😊


Selected items in a treeview should be displayed in other treeview.

Hi,
I have tried displaying the Selected treeview items in other treeview on a button click , but somehow both treeview items are binded(values in the both treeview items are same always).
Please help me in above scenerio.

Take a screenshot please. Thanks.

Ngx-treeview.docx.
Screen shots are in the document..

You need to write more codes to meet your requirement. I think it is not hard

Im calling below function on the button click:
this.postitems is the one that im binding to the bottom treeview....
addClick() {
let checkedItems: any[] = [];
this.postitems = [];
console.log(this.values);
this.selectedItems = [];
this.values.forEach(element => {
  let parent: DownlineTreeviewItem = element.parent;
  if (!_.isNil(parent) && !checkedItems.includes(parent.item)) {
    debugger;
    this.parentTemp = parent.item;
    for (let i = 0; i < parent.item.children.length; i++) {
      if (parent.item.children[i] != element.item) {
        TreeviewHelper.removeItem(this.parentTemp, parent.item.children[i]);
        i--;
      }
    }
    //this.parentTemp.children.push(element.item);
    checkedItems.push(this.parentTemp);
  } else if (checkedItems.includes(parent.item)) {
    if (this.parentTemp == parent.item) {
      this.parentTemp.children.push(element.item);
    }
  }
});
this.postitems = checkedItems;

}

and my Html looks like:

      <button type="button" class="btn btn-primary btn-single pull-right" (click)='addClick()'>Add</button>

    <ngx-treeview
    [config]="config"
    [items]="postitems"
    (selectedChange)="postValues = $event">
    </ngx-treeview>

My html looks like:

Add

I ran into this same issue. If you copy the node from the original tree some how it binds to the original nodes state. But if you create a new node with the same data an push that data into the new tree it appears to break the binding issue you are running into. I ran into this same issue.

Thanks @mardon1975 , Now im able to display the selected ones in another treeview.


Update package-lock.json

What does this PR do?:
Bump semver version following:
https://github.com/advisories/GHSA-c2qf-rxjj-qqgw
For Datadog employees:

[ ] If this PR touches code that signs or publishes builds or packages, or handles
credentials of any kind, I've requested a review from @DataDog/security-design-and-guidance.
[x] This PR doesn't touch any of that.

Unsure? Have a question? Request a review!

This appears to be bumping nan, not semver? 🤔

This appears to be bumping nan, not semver? 🤔

Indeed, the nan thing was a previous omission and I forgot to add the actual update of semver...
Fixed.


goldmark v1.1.19 -> v1.1.23

vendor update ...

Codecov Report

Merging #10519 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10519      +/-   ##
==========================================
+ Coverage   43.68%   43.68%   +<.01%
==========================================
  Files         586      586
  Lines       81475    81475
==========================================
+ Hits        35590    35593       +3
+ Misses      41474    41470       -4
- Partials     4411     4412       +1

Impacted Files
Coverage Δ

modules/indexer/stats/queue.go
62.5% <0%> (-18.75%)
:arrow_down:

modules/indexer/stats/db.go
40.62% <0%> (-18.75%)
:arrow_down:

modules/git/utils.go
65.67% <0%> (-4.48%)
:arrow_down:

services/pull/temp_repo.go
31.62% <0%> (-2.57%)
:arrow_down:

modules/queue/workerpool.go
56.93% <0%> (-1.07%)
:arrow_down:

modules/git/repo.go
46.78% <0%> (-0.92%)
:arrow_down:

routers/repo/view.go
36.82% <0%> (+0.67%)
:arrow_up:

modules/log/event.go
65.64% <0%> (+1.02%)
:arrow_up:

services/pull/pull.go
36.07% <0%> (+1.17%)
:arrow_up:

modules/indexer/issues/indexer.go
58.9% <0%> (+1.36%)
:arrow_up:

... and 3 more

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 e46f7aa...968d550. Read the comment docs.

make lgtm work


When running matrix for multiple environment the commenter only work for one environment

Hi
I am using the commenter to add comments for multiple environment with a PR. But this is only adding comment for the first environment when running a matrix in parallel.
- uses: GetTerminus/terraform-pr-commenter@v3
      if: github.event_name == 'pull_request'
      env:
        GITHUB_TOKEN: ${{ github.token }}
        TF_WORKSPACE: ${{ inputs.environment }}
      with:
        commenter_type: plan
        commenter_plan_path: ${{ inputs.tf_directory }}/tf_plan_${{ inputs.environment }}.txt
        commenter_exitcode: ${{ steps.plan.outputs.exitcode }}

matrix details
strategy:
      max-parallel: 4
      matrix:
        environment: [dev,prd]

both  dev and prod has run but only dev comments are added

Dev Run details below

Run docker run
INFO: Found commenter plan file.
INFO: Found 1 page(s) of comments at https://api.github.com/repos/LiberisFinance/Liberis.Quote.Api/issues/493/comments?per_page=100.
INFO: Looking for an existing plan PR comment.
INFO: No existing plan PR comment found.
INFO: Found 1 page(s) of comments at https://api.github.com/repos/LiberisFinance/Liberis.Quote.Api/issues/493/comments?per_page=100.
INFO: Looking for an existing outputs PR comment.
INFO: No existing outputs PR comment found.
INFO: Writing 1 plan comment(s)
INFO: Adding plan comment to PR.
INFO: Writing 0 outputs comment(s)
Prd Run details below

Run docker run
INFO: Found commenter plan file.
INFO: Found 1 page(s) of comments at https://api.github.com/repos/LiberisFinance/Liberis.Quote.Api/issues/493/comments?per_page=100.
INFO: Looking for an existing plan PR comment.
INFO: No existing plan PR comment found.
INFO: Found 1 page(s) of comments at https://api.github.com/repos/LiberisFinance/Liberis.Quote.Api/issues/493/comments?per_page=100.
INFO: Looking for an existing outputs PR comment.
INFO: No existing outputs PR comment found.
INFO: Writing 0 plan comment(s)
INFO: Writing 0 outputs comment(s)

issue fixed when I have added -input=false -no-color to the terraform plan option.

issue fixed when I have added -input=false -no-color to the terraform plan option.

The issues seems to be intermittent as well

@RamBNarayanan , i've forked this and made it compatible with opentofu, so if you're using opentofu instead of terraform, this will work:
https://github.com/phoenix-actions/opentofu-pr-commenter


Follow up on query performance failures

Query performance checks started failing on or around #1717
See if it's query or db related and fix or bump costs.

FWIW, it's passing again: https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/actions/workflows/query-performance.yml

Closing this for now.


Howto get verbose output (from errors)

Running the contributed command: 'pro-deployer.upload' failed.

You can check Output -> PRO Deployer.


wrote tests and fixed bugs in reducers

npm test to run tests
to add more tests create a dir in the desired place call tests/ and jest will find them and run them

yay im bad at ' " thats why i wanted to add a linter
to.have.property ive never used it but it seems neat
ill fix the '


integrations: Add webhook payloads for Intercom

integrations: Add webhook payloads for Intercom

@ViRu-ThE-ViRuS: This looks great, but your commit messages just need to be cleaned up!  Here's a breakdown of how you can go about doing this:

Run git rebase -i HEAD~4
Replace pick with squash for the last 3 commits so that all the commits get squashed into the first.
When you save, it will prompt you to change the commit message. Replace the commit message as I explained above and make sure that the commit message ends in a period. :)
At the end run, git push origin master --force. That should do it!

Feel free to reach out to us on chat.zulip.org if you need help with any of this! :)
Thanks!

@eeshangarg  here is the link to the new pr i made with the cleaned commit history. Thanks for working this out with me :-)


[fatal][tid:main] Unable to execute JS call: __fbBatchedBridge is undefined

It was working less than 2 weeks ago but now it's show " Unable to execute JS call: __fbBatchedBridge is undefined"

@hotarzhang Most likely an issue with trying to connect to the packager with Android.
If your trying to run on it on a device, make sure you run adb reverse tcp:8081 tcp:8081

@christopherdro Thanks for getting back so quickly. But it's running with mac.setp is:
open MyReactNative.xcodeproj in Xcode
Product -> Run

and then  the same error happens when clicking Run.

@hotarzhang Can you post the solution, please?

@hotarzhang  Can you post the solution, please?

Switching iOS devices in the simulator was the fix for me

@jDeppen kind of a workaround, but resetting the simulator fixes it for me as well


WIP: virtualbox: 5.2.22 -> 6.0.2

Motivation for this change
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Things to do

~Build with hardening doesn't start~
Tests

Changes: https://www.virtualbox.org/wiki/Changelog-6.0#v0

I've tested this PR on my machine, after upgrading to 6.0.4 https://github.com/bendlas/nixpkgs/commit/1a73f8b307f5db56bd4cd753454f32337034e2db
It doesn't seem to find its extension pack (installed in libexec due to a non-hardened build)

I removed the WIP status, but I couldn't get the tests to work.
Most likely it's because nested virtualization stopped working (tests run VirtualBox guest inside of QEMU host).
Everything else should work, including 3D accel.

What's blocking this?

I tried it, it works on my side. @Infinisil I'd like to merge this if it's alright with you.

I see three open questions:

hardening
extension pack
nixos tests

I'm not entirely against merging this while these don't work yet (it doesn't currently build with linuxPackages_latest at all), but we should still pick this up :-)

Please merge #55596 before or with this merge as that PR implies that this would break vagrant since the current version in unstable doesn't support virtualbox 6.

@wkral thank you for letting me know, I will merge #55596 after this one.
@flokli Let's hear from @Infinisil first in this case!

@flokli As far I can tell, hardening and extension pack work.
I wasn't able to get the NixOS tests to run and fixing them is beyond my abilities.

@aszlig do you have time to dig into the virtualbox tests?

As far I can tell, hardening and extension pack work.

Have you tested adding USB 2/3 and other advanced hardware? That failed for me i.e. I couldn't open any of my saved machines w/o deactivating all the hardware. Said it couldn't find the extension pack.
If you think this is working, I'll be happy to give it another shot.

I also had a problem when recovering saves (it talked about pgm). It may have been a change in the state save format, since when I discarded and restarted the machine everything was working, including NVMe and USB 2.0. The extpack is also listed as functioning and installed.
$ VBoxManage list extpacks
Extension Packs: 1
Pack no. 0:   Oracle VM VirtualBox Extension Pack
Version:      6.0.4
Revision:     128413
Edition:
Description:  USB 2.0 and USB 3.0 Host Controller, Host Webcam, VirtualBox RDP, PXE ROM, Disk Encryption, NVMe.
VRDE Module:  VBoxVRDP
Usable:       true
Why unusable:

If we have different problems, what do you get when you list extension packs?

So I gave this a spin. It's pretty much unusable for me, I had to rollback.
Issues:

Video was flickering, everything is much slower
Guest additions doesn't install, gets an network error
Couldn't get oracle vm extension to work for usb 2.0 support

virtualbox also discontinued supporting i686-linux with version 6, and claim they'll still support version 5.2 (funnily inside not inside their release notes, just on the website)

Please also use version 5.2 if you still need support for 32-bit hosts, as this has been discontinued in 6.0. Version 5.2 will remain supported until July 2020.

So we should drop i686-linux from meta.

@flokli I removed "i686-linux" from meta.
Maybe it is worth considering adding a separate package with VirtualBox version 5.2 for this platform.

Have you tested adding USB 2/3 and other advanced hardware?

@bendlas Yes, I tried with USB2/3.
I also looked at your nixos configuration.  You try to enable extension pack with virtualbox.enableExtensionPack = true; . It never worked for me this way, so I enabled it using virtualbox host module:
{
  virtualisation = {
    virtualbox.host = {
      enable = true;
      enableExtensionPack = true;
    };
  };
}

I also just realized that I disabled hardening in my configuration. I will try to rebuild and fix any problems.

I packaged virtualbox 5.2.26 in https://github.com/NixOS/nixpkgs/pull/56210, which should remove the 19.03 blocker at least.

Considering how VirtualBox 6 drops support for i686, and NixOS still supports i686, we may want to make virtualbox versions 5 and 6 different packages, then just use virtualization.virtualbox.host.package to decide between them

@artemist in fact, I was planning on abandoning cfg.package, and instead just have switches for hardening and extension pack, easing some parts of the wrapper to solve https://github.com/NixOS/nixpkgs/issues/34796#issuecomment-365070156.
Unfortunately, I never found the time to polish that up, and probably won't get to that until 19.03 branch-off. I'm also not sure if we can fix virtualbox 6 until then either.
Maybe better stick with virtualbox 5 for 19.03? On the other hand, how much do people rely on virtualbox on i686?

I merged down #56210 and have removed the blocker label from this pull request. We can work on getting this out after 19.03.

I also looked at your nixos configuration. You try to enable extension pack with virtualbox.enableExtensionPack = true; . It never worked for me this way, so I enabled it using virtualbox host module:

ok, this is embarrassing. with the system config, everything seems to work now. (the menu-based download for the guest additions fails with repeatably with unspecified network errors, probably not something we should fix)
sorry for not a/b testing and thanks for debugging my configuration @ptrhlm ;-)
as far as I'm concerned, the Extension Pack issue is resolved.

A few updates since I last tried this:
William Casarin<EMAIL_ADDRESS>writes:

So I gave this a spin. It's pretty much unusable for me, I had to rollback.
Issues:

Video was flickering, everything is much slower

The flickering was happened with 3d/2d acceleration turned on. When I
turn it off, no more ficker

Couldn't get oracle vm extension to work

I didn't have the nix option set, this works now.

@ptrhlm this doesn't apply anymore, can you rebase?
Is there anybody who can dedicate some time to debug the virtualbox test breakage?

Is there anybody who can dedicate some time to debug the virtualbox test breakage?

FWIW, here is a log of the nixos tests failing: https://gist.githubusercontent.com/bendlas/02111b84e4cce6916988047f5fc9ff12/raw/9d64e28bc6106d70a2f7795b9c667babf6c1415c/vbox-tests.log

So, what's happening with this?

IMHO, we can't bump virtualbox to 6.x if we can't get the tests to work.
There's too much that can break randomly and to test manually.
I'm not sure what exactly was dropped in virtualbox 6.0 - docs mention 32bit support (which is why they keep 5.2 around until 2020), but maybe this broke nested virtualization too.
@devhell if you can devote some time to debug this, it'd be much appreciated :-)

@flokli Thanks for the info. As so many of us, I wish I had the time, and in this case also the skills required to work out the tests. :(

I'm happy to test this on macOS, although I'm a net new to Nix.  Can someone guide me on the appropriate process to check this out, and test it?

I've created a Darwin specific package, here https://github.com/tonylotts/nixpkgs/commit/d0d87fcb4d5d7ed81f5c32d5a338c8322a662c22

@tonylotts I don't think virtualbox 6.0.4 completly broken for linux, it's just that the nixos vm tests don't work anymore (maybe due to the nested virtualization happening here) so we don't spot things suddenly breaking, which is why I'm hesitant to bump it without fixing the tests too…
Note the darwin-specific virtualbox6 derivation you proposed doesn't build VirtualBox, but simply unpacks a .dmg.

@flokli, yes I've purposely used the pre-built binary; although I acknowledge that the community preference is to build from source.  As I understand it, pre-built binaries are cached.  When one   In the case that VirtualBox is already available in pre-built form, can/should that be cached?

Even when there's a from-source build in nixpkgs, it should be built by Hydra and available from cache.nixos.org.
This currently isn't working for virtual box with extension pack enabled, as IIRC its license forbids redistribution of the extpack itself, but for the non-extpack variant it should be in the cache.

Got it.

It turns out that the macOS version of VirtualBox provides kernel extensions.  A macOS derivation for VirtualBox would require nix-darwin, and support for kernel extensions https://github.com/LnL7/nix-darwin/pull/91.

@tonylotts so no matter what variant we use, this depends on https://github.com/LnL7/nix-darwin/pull/91 or a successor to be merged?

6.0.6 has been released. Not that it matters that much, I guess, right now.

@flokli, yes

@devhell I tried bumping to 6.0.6, but it seems some patches need to be adjusted. Did you do that already somewhere?

The release notes mention some nested (AMD) virtualization fixes - I guess we still should see if things have improved.

@flokli I've not been able to, no. Sorry. :/

@ptrhlm could you get to that?

@flokli I've removed unnecessary kernel module patches. Also, I moved patching of kernel module sources to main package, so now unaligned patch will fail earlier.
Tests still fail on my Intel machine.

@ptrhlm thanks for the cleanup! I gave the tests a try on an AMD machine too, unfortunately, they fail there, too:
nix log /nix/store/pnbnmxz36kxdd33b8mj9v6qh2i7hmiws-vm-test-run-virtualbox-headless.drv looks most scary:
machine# !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
machine# !!
machine# !!         VCPU0: Guru Meditation -2403 (VERR_TRPM_DONT_PANIC)
machine# !!
machine# !! TRAP=0e ERRCD=0000000000000010 CR2=00000000a082bf60 EIP=a082bf60 Type=0 cbInstr=ff
machine# !! EIP is not in any code known to VMM!
machine# !!
machine# !!
machine# !!


host firewall: duplicate policy verdict events

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Problem:
Host firewall policies (https://docs.cilium.io/en/v1.12/policy/language/#host-policies) product duplicate policy verdict events. Reproduced on multiple versions.
Expect:
Single policy verdict event per flow
Actual:
Two (2) policy verdicts per flow
Reproduction:

Follow quick installation guide (https://docs.cilium.io/en/v1.12/gettingstarted/k8s-install-default/#quick-installation)
Follow instructions to create a GKE cluster (presumably kind or other would also reproduce the issue)
alias k=kubectl
k config set-context --current --namespace=kube-system
Enable host firewall:k patch cm cilium-config --type=json -p '[
  { "op": "add", "path": "/data/enable-host-firewall", "value": "true" }
]'

Create host firewall policy (https://docs.cilium.io/en/v1.12/policy/language/#host-policies):k apply -f - <<EOF
apiVersion: "cilium.io/v2"
kind: CiliumClusterwideNetworkPolicy
metadata:
  name: "node-allow-all-ingress"
spec:
  nodeSelector: {}
  ingress:
  - fromEntities:
    - all
EOF

k rollout restart ds/cilium && k rollout status ds/cilium
 k exec -it $( k get pod -l=k8s-app=cilium -ojsonpath='{.items[].metadata.name}' ) -- cilium monitor --type policy-verdict
deploy busybox podk apply -f - <<EOF
apiVersion: v1
kind: Pod
metadata:
  name: busybox-sleep
spec:
  containers:
  - name: busybox
    image: busybox:1.28
    args:
    - sleep
    - "1000000"
EOF

k exec -it busybox-sleep -- ping -c1 $( k get pod -l=k8s-app=cilium -ojsonpath='{.items[].status.podIP}' )

Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 1, ingress, action allow, match all, <IP_ADDRESS> -> <IP_ADDRESS> EchoRequest
Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 1, ingress, action allow, match all, <IP_ADDRESS> -> <IP_ADDRESS> EchoRequest

k exec -it busybox-sleep -- wget $( k get pod -l=k8s-app=cilium -ojsonpath='{.items[].status.podIP}' )

Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 6, ingress, action allow, match all, <IP_ADDRESS>:55180 -> <IP_ADDRESS>:80 tcp SYN
Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 6, ingress, action allow, match all, <IP_ADDRESS>:55180 -> <IP_ADDRESS>:80 tcp SYN

Cilium Version

Client: 1.12.2 c7516b9 2022-09-14T15:25:06+02:00 go version go1.18.6 linux/amd64
Daemon: 1.12.2 c7516b9 2022-09-14T15:25:06+02:00 go version go1.18.6 linux/amd64

Kernel Version

Linux ${NODE_NAME} 5.10.127+ #1 SMP Fri Jul 8 00:39:24 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Kubernetes Version

Client Version: v1.24.2
Kustomize Version: v4.5.4
Server Version: v1.23.8-gke.1900

Sysdump
If this is important, we'll work on a way to provide.
Relevant log output
ICMP:
> Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 1, ingress, action allow, match all, <IP_ADDRESS> -> <IP_ADDRESS> EchoRequest
> Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 1, ingress, action allow, match all, <IP_ADDRESS> -> <IP_ADDRESS> EchoRequest

> 19:06:34.750785 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 18944, seq 0, length 64
> 19:06:34.750900 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo reply, id 18944, seq 0, length 64

TCP:

> Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 6, ingress, action allow, match all, <IP_ADDRESS>:55180 -> <IP_ADDRESS>:80 tcp SYN
> Policy verdict log: flow 0x0 local EP ID 930, remote ID 5912, proto 6, ingress, action allow, match all, <IP_ADDRESS>:55180 -> <IP_ADDRESS>:80 tcp SYN

> 19:06:18.153217 IP <IP_ADDRESS>.46046 > <IP_ADDRESS>.80: Flags [S], seq<PHONE_NUMBER>, win 42600, options [mss 1420,sackOK,TS val<PHONE_NUMBER> ecr 0,nop,wscale 7], length 0
> 19:06:18.153326 IP <IP_ADDRESS>.80 > <IP_ADDRESS>.46046: Flags [R.], seq 0, ack<PHONE_NUMBER>, win 0, length 0

Anything else?

Checked existing issues and didn't find a duplicate issue.
This only happens with host firewall policies.
confirmed via tcpdump that the node is only receiving single flow

Code of Conduct

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

I'm unable to reproduce on v1.13.1. I deployed the same policy you did, on a GKE cluster with host firewall enabled, and I'm only seeing one policy verdict per SYN or ICMP "connection".
Could you try latest v1.13? If you are still able to reproduce, it will be useful to get a sysdump to find what differs with my reproduction attempt.

Thanks Paul for the update, let me try to reproduce it on 1.13.


Feature Request: Add option to add location of organization

Hi Amruth,
Your app is really awesome. It is the best resume software I have come across and I use it whenever I need to create/update my resume.
I just wanted to request you to please add a feature in the Reactive Resume.
Currently the option to add the location of the organization is not available in the software.
In case it is possible and in case you can spare some time, may I request you to please add a feature which enables us to add location of the organization in the resume.
Thank you!
Regards,
Sugam Khetrapal

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :)
Thank you!


fix: Show rmrk2 on landing - beta.kodadot.xyz

PR Type

[x] Bugfix

Context

[x] Closes #5618

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 10 usd

Should we also show rmrk2 on kodadot.xyz? If not, I need to do some extra check.

actually no 😅 but this one should be ok?

This pr would also enable rmrk2 items to be shown on landing on kodadot.xyz. So we should avoid releasing it to kodadot.xyz or make another fix for this limitation.


Fixed port binding on docker-compose.yml

Description
The ports binding on the docker compose file hasn't changed. However since port 5000 is exposed on the Dockerfile and the port is implicitly forced with dotnet run argument "--urls" we need to do the same with the docker compose file.

Description
The ports binding on the docker compose file hasn't changed. However since port 5000 is exposed on the Dockerfile and the port is implicitly forced with dotnet run argument "--urls" we need to do the same with the docker compose file.

Could it be possible instead to not force the use of 5000?
Making the port 5000 the default, but if the port on the docker compose is something else than 5000 use it, cause right now, you cannot use a different port than 5000.

I don't really understand the point of this. Inside the docker container, the service is listening on TCP port 5000.
If you start the docker container via
docker run --name openbullet2 --rm -p 8069:5000 -v C:/OB2/UserData/:/app/UserData/ -it openbullet/openbullet2:latest

You can reach it at http://localhost:8069, where 8069 is an arbitrary port, effectively mapping port 5000 to whatever you please. The same can be done in docker-compose. The docker-compose.yml file is not meant to be used as-is, but it can be customized depending on your specific situation after you made a local copy of it, for example by setting up your custom port.


How to enable git-flow completion (zsh-completion)  in zim?

zmanage info output :
Zim commit ref:   3c2730d
Zsh version:      zsh 5.1.1 (x86_64-ubuntu-linux-gnu)
System info:      Linux sudoman 4.8.0-46-generic #49~16.04.1-Ubuntu SMP Fri Mar 31 14:51:03 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

gist of .zimrc: https://gist.github.com/supirman/3f5b00571e031de10c36e9c545de0b33#file-zimrc
gist of .zshrc: https://gist.github.com/supirman/3f5b00571e031de10c36e9c545de0b33#file-zshrc

Description
I try to use git flow completion with zim. It already include inside completion. But I have no luck to make it works. Do I need other configuration to make it works?
Steps to reproduce

Enable completion in .zimrc
Type "git flow "
Press Tab, regular directory completion will shown

It seem manually run autoload -U compinit && compinit make the completion works

Cannot reproduce. I'm using:
Zim commit ref:   dca1803
Zsh version:      zsh 5.3.1 (x86_64-apple-darwin15.6.0)
System info:      Darwin ericsmacbookpro.local 15.6.0 Darwin Kernel Version 15.6.0: Fri Feb 17 10:21:18 PST 2017; root:xnu-3<IP_ADDRESS>.1~1/RELEASE_X86_64 x86_64


[#66] Fix metrix server and gateway server bootstrap

Motivation:
Fix metrics server's bootstrap and gateway server's bootstrap
Modification:
Add spring-boot-maven-plugin for metrics and gateway server
Result:
Fixes #66
If there is no issue then describe the changes introduced by this PR.

https://github.com/sofastack/sofa-lookout/pull/65  @xzchaoo It will be closed.


Exponentiation primop

Describe the feature you'd like
This is a follow-on from #6335, and relates to prior requests in #4168 and here, as well as #6344 (as it used exponentiation, both modular and not, internally).
Currently, if you want exponentiation, you have to resort to exponentiation by squaring. While for arbitrary semigroups (or monoids, or groups), this is indeed the best we can do, in the specific case of BuiltinInteger (which is ultimately what is interesting to us here), we lose considerably. Thus, we propose that a builtin non-modular exponentiation operation be added, allowing us to provide this functionality efficiently, while also answering some community requests for functionality.
Firstly, even if we just compare a typical exponentiation-by-squaring implementation to Haskell's built-in ^ for Integer, we see we're falling behind, even on small numbers:
      bgroup
        "exp"
        [ bench "naive" . nf (naiveExp 7) $ 1024,
          bench "expBySquaring" . nf (expBySquaring 7) $ 1024
        ]

-- Produces this result:
--     naive:         OK
--      582  ns ±  47 ns, 1.2 KB allocated,   0 B  copied,  33 MB peak memory
--    expBySquaring: OK
--      630  ns ±  50 ns, 1.2 KB allocated,   0 B  copied,  33 MB peak memory

This would only become more of a problem the larger the numbers become.
Secondly, there are notable optimization paths we could take. Firstly, there is the classic case where raising 2 to a positive power is in fact a shift. This is not an optimization GHC seems to perform for Integer at least:
      bgroup
        "power-two"
        [ bench "naive" . nf ((2 :: Integer) ^) $ (10 :: Integer),
          bench "shift" . nf (unsafeShiftL (2 :: Integer)) $ 10
        ]
-- Produces this result
--     naive:         OK
--      73.6 ns ± 6.4 ns, 191 B  allocated,   0 B  copied,  33 MB peak memory
--    shift:         OK
--      27.3 ns ± 2.6 ns, 127 B  allocated,   0 B  copied,  33 MB peak memory

We also noticed that if we know an exponent is an exact power of two, regardless of base, we could beat exponentiation:
{-# INLINE naiveExp #-}
naiveExp :: Integer -> Integer -> Integer
naiveExp = (^)

{-# INLINE kSquare #-}
kSquare :: Integer -> Int -> Integer
kSquare x = go
  where
    go :: Int -> Integer
    go k
      | k == 0 = 1
      | k == 1 = x
      | otherwise = sqrInteger (go (k - 1))

-- Given these benchmarks
b = bgroup
        "exp"
        [ bench "naive" . nf (naiveExp 7) $ 1024,
          bench "k-square" . nf (kSquare 7) $ 10
        ]
-- Produces this result
--     naive:         OK
--      582  ns ±  47 ns, 1.2 KB allocated,   0 B  copied,  33 MB peak memory
--    k-square:      OK
--      172  ns ±  11 ns, 682 B  allocated,   0 B  copied,  33 MB peak memory

We can use both of these facts to gain significant advantage on Integer exponentiation by making further observations:

For any even $i$, we can separate it into $2^k \cdot j$, where $j$ is odd, rapidly (effectively by counting trailing zeroes and shifting right)
When we have $(2^k \cdot x)^y$, we can break it down into $2^{ky} \cdot x^y$ (making use of shifting and reducing the work in exponentiation)
When we have $x^{2^{\ell}y}$, we can break it down into $x^{2^{\ell}} \cdot x^y$ (making use of k-squaring and reducing the work in exponentiation)
Observations 2 and 3 combine (with big reductions)

These optimizations require low analysis costs (a trailing zero count effectively), and would work for most inputs (assuming a completely random choice of base or exponent). Further than this, if we were to implement a non-modular exponentiation operation directly, a future move away from GHC 8.10 would permit us to implement this operation mutably (requiring the new ghc-bignum interface), which would significantly improve our performance by requiring only one up-front allocation for the result.
This operation would not only answer these (in some cases long-standing) concerns, but also optimize a useful operation.
Describe alternatives you've considered
The current options consist of either:

Exponentiation by squaring; or
Trying a similar strategy to what we described above, using CIP-121, CIP-122 and CIP-123 primitives

The first has the inefficiencies we described, while the second unfortunately is drowned out by the fact that CIP-121 conversions are currently quadratic, and even with ghc-bignum's interface, cannot be made better than linear due to pinning issues. Both of the issues linked above essentially make this point, as in their use cases, these inefficiencies are unacceptable.

@kwxm a gentle reminder.

I'm not sensing a definite "won't do" here, so I'm going to retriage this as "Low priority" (and "won't do" is better known as closing the issue anyway).

Another issue is that it's difficult to take account of special cases in costing: we have to cost the worst case of builtin evaluations, so if some case such as calculating a power of 2 can be implemented particularly efficiently, we'd still have to charge for the non-optimal case: the function might run more quickly in certain cases, but this wouldn't be reflected in the charges paid by the user.

I think from the implementation point of view this should be fairly straightforward, want me to provide a proof-of-concept? From the spec point of view it's gonna be hard, I'm sure.

I think from the implementation point of view this should be fairly straightforward, want me to provide a proof-of-concept?

Not really!  I think it'd be problematic because we'd have to match up special cases in the costing with special cases in the  implementation and keep them synchronised. We already do a certain amount of this kind of thing and it's a bit annoying.  You also have to add extra logic to detect the special case before you call the function, which will add extra overhead.  In addition we'd need different sets of costing benchmarks for the different special cases, and again these would have to be kept in sync with the implementation.

OK, yes, that's what I foolishly called "fairly straightforward".
Given some additional feedback elsewhere, it seems like my original assessment of this not being a "won't do" is wrong. I'm closing the issue then, we'll reopen it if ever want to reconsider adding such a builtin.
@kozross thank you for the report and sorry we aren't able to fulfill your request.


Having trouble running "make" command

Hi,
I'm having trouble running the "make" command for NxTrim. When I run this command, I get the following error message.
/usr/bin/ld: cannot find -lboost_program_options
collect2: error: ld returned 1 exit status
make: *** [mergeReads] Error 1
I don't know how to fix it as I'm very new to programming. Please advice. Thanks!
-Taruna

Hi Taruna,
You need to have the boost libraries installed on your system and set the BOOST_ROOT environment variable appropriately.  See here.
I appreciate building boost can be a pain, perhaps a solution is for me provide some pre-compiled nxtrim binaries on here.  What operating system are you using?
best,
Jared

Hi Jared,
Thank you for your reply. I'm using MacOSX (10.10.2) and I did run the boost codes on your NxTrim site, but I got nowhere. I'm sorry if that's too vague. I'm just so new that I'm not helpful perhaps. I will read the info about boost libraries you sent me. Please let me know if you need additional information from me. Thanks!
-Taruna

Hi Jared,
I forgot to tell you that I'm trying to run NxTrim on a "super computer" through my school which I can access via terminal. I think that changes the whole reason why I can't install Boost libraries, doesn't it? Regardless, I installed NxTrim on my own computer and tried to run it, but I got a similar message.
g++ -Wall -g -I/usr//include -c fastqlib.cpp
In file included from fastqlib.cpp:1:
In file included from ./fastqlib.h:2:
./utilityfunc.h:10:10: fatal error: 'boost/iostreams/filtering_stream.hpp' file
not found
#include <boost/iostreams/filtering_stream.hpp>
^
1 error generated.
make: *** [fastqlib.o] Error 1
Tarunas-MacBook-Pro:NxTrim Taruna$ ./nxtrim
-bash: ./nxtrim: No such file or directory
Sorry for the trouble, but I appreciate your help!
-Taruna

Hi Jared,
It's working now. One of my advisors suggested that I run the following command and it worked!
export BOOST_ROOT=/usr
make
Thank you for your help though! I appreciate it!
Best,
Taruna


fix: virtualize collectibles list

What it solves
Large collectible lists freezing the Safe
How this PR fixes it
The collectibles list is now virtualized, rendering only what is visible in the view port. It no longer freezes the Safe and is much more performant.
How to test it
Open the Collectibles section of a Safe with a large number of collectibles, e.g. eth:0xc6400A5584db71e41B0E5dFbdC769b54B91256CD (prod.) and observe no lag in navigating the list.
Screenshots

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 20 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 34.868%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Balances/Collectibles/components/Item.tsx
0
2
0.0%

src/routes/safe/components/Balances/Collectibles/index.tsx
0
18
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
3381

Relevant Lines:
8786

💛  - Coveralls

The collectibles list is now virtualized

What's the meaning of this? 😅

What's the meaning of this? 😅

It only renders what is in the viewport and the rest is 'virtualized'. See here.

It works fine.
Tried with the safe given in the example and it scrolls fine.
I tried the same in prod and the page directly crashes there


Devspace tries to clone dependencies when told to skip them

What happened?
CI fails when there are dependencies because CI doesn't have access to those dependencies. When told to skip them via --skip-denpencency it still tries to clone them anyway resulting in a failure.
What did you expect to happen instead?
Only the dependencies that aren't skipped to be cloned.
How can we reproduce the bug? (as minimally and precisely as possible)
My devspace.yaml:
version: v2beta1
name: example

dependencies:
  fake-dep:
    git: git@github.com:example/this-does-not-exist.git
    branch: main

# add the above to any devspace.yaml

Run the following to illustrate:
docker run -it --rm -v $(pwd):/app -v ~/.kube/config:/root/.kube/config -v $(which devspace):$(which devspace) -v $(which loft):$(which loft) -w /app ubuntu:latest bash

Then inside the container,
./devspace build --skip-dependency fake-dep

Possible errors:
fatal deploy dependencies: resolve dependencies: git not found in path. Please make sure you have git installed to clone git dependencies

fatal deploy dependencies: resolve dependencies: clone repository: Error running 'git clone git@github.com:example/this-does-not-exist.git REDACTED --branch main --depth 1'
Warning: Permanently added the ECDSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

Note that we are using a repository that doesn't exist as a proxy for a private repository. The error is the same.
Currently, the workaround is to give the CI runner access to the dependencies even though all it does is clone them.
Local Environment:

DevSpace Version:  6.0.0-beta.0
Operating System: linux
ARCH of the OS: AMD64
Kubernetes Cluster:
Cloud Provider: other
Kubernetes Version: NA

Anything else we need to know?

/kind bug

@withinboredom thanks for reporting this issue, you are correct, seems like DevSpace is still trying to resolve those. We will fix this!


Waiting for item for X ticks

It would be very neat if the TAS reported how many ticks it waited to perform an action

As guessing the amount is tiresome

I usually just generate a new step file with amount slightly changed until it doesn't give the warning. But it is still wasteful, and i am sure we can find a way count the ticks 😎

I haven't taken a look yet, but your idea sounds possible. Regardless, we should have the "if" statements set up, so i guess we just need to keep track of a number.


Issue tracker pull request links

Rationale
The links in the issue tracker pull request text box aren't clickable.  While we're at it, let's make them nice short github pull request references.  Even better, let's attempt to fetch the PR title from github.  The current method of fetching the PR title only will work for public pull requests.
Changes

Render the pull request column as a set of short PR reference links (e.g., platform#1234)
Attempt to fetch the pull request title

I'm all for making the links clickable but it makes me nervous to be issuing additional github requests to pull in information that may not be that useful. At the very least, we should have a way to disable this functionality or just get rid of the piece that tries to pull the title from the PR

Removed the github title fetching.


Save-DbaKbUpdate - Using Architecture and some other fixes

Type of Change

[x] Bug fix (non-breaking change, fixes # )

When downloading cumulative updates for SQL Server 2014 I noticed, that also the x86 files where downloaded.
Reason is that the filtering for Architecture was not working, as $item in $InputObject.Link was not a list of all files for the KB, but only a single file - with no chance to see the other files for that KB. So the "Could not find architecture match, downloading all" never worked here. Maybe worked before pipeline input from Get-DbaKbUpdate was enabled.
So I reduced the two loops into one - sorry, this makes the diff not very readable again.
I also found other issues with the tests and the way the filename was build.

This is my use case code to download the cumulative updates in a lab environment, that I will put in "Discussions / Show and tell" after this is released:
Import-Module -Name dbatools

function Get-CU {
    param(
        [Parameter(Mandatory)]
        [ValidateSet('2019', '2017', '2016', '2014')]
        [string]$Version,
        [regex]$BuildBaseRegex,
        [int]$Last = 999,
        [string[]]$Exclude,
        [string]$Path = '.'
    )
    if ($null -eq $BuildBaseRegex) {
        $BuildBaseRegex = switch ($Version) {
            '2019' { '^15' }
            '2017' { '^14' }
            '2016' { '^13.0.5' }  # Based on SP2
            '2014' { '^12.0.6' }  # Based on SP3
        }
    }
    if ($Version -eq '2019') {
        $Exclude += 'CU7'  # CU7 is not available any more
    }
    $buildrefFile = Join-Path -Path (Get-DbatoolsConfigValue -Name 'Path.DbatoolsData') -ChildPath "dbatools-buildref-index.json"
    $buildrefData = (Get-Content -Path $buildrefFile -Raw | ConvertFrom-Json).Data
    $cuData = $buildrefData |
        Where-Object -FilterScript { $_.Version -match $BuildBaseRegex -and $_.CU -ne $null -and $_.CU -notin $Exclude } |
        Sort-Object -Property KBList |
        Select-Object -Last $Last
    foreach ($cu in $cuData) {
        $kbNr = $cu.KBList
        $cuName = $cu.CU
        $filePath = Join-Path -Path $Path -ChildPath "SQLServer$Version-KB$kbNr-$cuName-x64.exe"
        if (-not (Test-Path -Path $filePath)) {
            Save-DbaKbUpdate -Name $kbNr -FilePath $filePath
        }
    }
}

Update-DbaBuildReference
$patchPath = 'D:\Software\SQLServerPatches'
Get-CU -Version 2019 -Path $patchPath
Get-CU -Version 2017 -Path $patchPath -Last 5
Get-CU -Version 2016 -Path $patchPath -Last 5
Get-CU -Version 2014 -Path $patchPath


Add step definition for cms list

Issue: https://github.com/silverstripe/silverstripe-cms/issues/632

Merge on green, then we'll re-run CMS tests.


PLAT-128480: Add UI and unit tests for MediaPlayer

Enact-DCO-1.0-Signed-off-by: Stanca Pop<EMAIL_ADDRESS>Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Resolution

added unit tests for agate/MediaPlayer
added UI tests for agate/MediaPlayer

Links
PLAT-128480
Comments

Jenkins passed:
/job/enact-ui-tests/1028/

Jenkins passed:
/job/enact-ui-tests/1028/


[Bug]: "test_graph_info_load_from_s3" failed under build type of Release

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When I use cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=ON .. and then make test, I got an error:

Start  6: test_graph_info_load_from_s3
Test   6: test_graph_info_load_from_s3 ................***Exception: SegFault 10.66 sec

Expected Behavior
pass this test
Minimal Reproducible Example
cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=ON ..
make test

Environment

Operating system: macOs Ventura 13.3.1
GraphAr version: v0.5.0

Link to GraphAr Logs
No response
Further Information
No response

@acezen could you please help to have a look at this issue?

@acezen could you please help to have a look at this issue?

Ok, I'll take this and try to reproduce the bug.

seems it's arrow 12.0.0 bring the bug
https://github.com/apache/arrow/blob/fab79f50caf6a80eccaa3bbae79d518af7b408e5/cpp/src/arrow/filesystem/s3fs.cc#L2597-L2603

En... It seems than arrow bring a AwsInstance to manager the S3 filesystem in all arrow threads BUT arrow did not take care the life cycle of AwsInstance, they give the life cycle control to user, and user need to call EnsureS3Finalized when the S3 job is done

Consider to make patch to GraphAr that add FinalizedS3 static function to FileSystem and let user to finalize the S3 system handly.

Consider to make a patch to GraphAr that add a FinalizedS3 static function to FileSystem and let user to finalize the S3 system manually. @lixueclaire what do you think?

I agree that it would be helpful to add such a function to FileSystem in GraphAr to allow users to finalize the S3 system manually. And we could document this feature in the documentation to ensure that users are aware of this special processing requirement for the S3 system.

agree, thanks for the advise.

Open a keep track of upstream fix issue #187 ,close this one.


Unable to change track for local training

In the run.env file whenever I change the value for DR_WORLD_NAME before training, when I upload the model to the aws deepracer console after training it shows the model track to be reinvent_2018 track even when I changed the track to something else.

The console does not remember which track you trained on when you upload. (It also does not remember the color of your car...) This is not having anything but cosmetic impact, and is nothing that we can change in DRfC.


Allow staff to override some ticket props

Let staff fix bait-and-switch tickets

Let the student change them as well. So you can remove the access control from apply_ticket_update.


Handle Redaction - return a correct message if results are redacted

Must support merged queries i.e. if one set is redacted, the others should return.

Handle in baseline also.


Migrate YARN to PNPM

Make sure to only merge this PR once the entire website has been completed. Also, ensure that whoever merges this branch, updates all the packages using pnpm update.

I will resolve the merge conflicts and merge this


Add Tesla using official vehicle command library

Fix https://github.com/evcc-io/evcc/issues/10460, refs https://github.com/evcc-io/evcc/issues/10789

[x] depends on https://github.com/teslamotors/vehicle-command/issues/28 (or patch Tesla Library)
[x] add template
[x] use client id during build, replaceable by ENV
[x] fix token handling
[x] depends on https://github.com/evcc-io/tesla/issues/3 (@naltatis)

How to get Tesla Integration working?

generate a new pair of API tokens here https://tesla.evcc.io
switch to the latest nightly build of evcc
change your template from tesla to tesla-command.

vehicles:
 - name: mytesla
   type: template
-  template: tesla
+  template: tesla-command
   accessToken: [...] # generetad by tesla.evcc.io
   refreshToken:  [...] # generetad by tesla.evcc.io
   ...

⚠ The template name tesla-command might change before the next stable release. But for now, this is the working version.
Using local builds is still possible, but now requires a Tesla Developer account. Use the TESLA_CLIENT_ID env.
Optional:

[ ] implement commands via API server

Very cool!
Is it still not an option to get the climate/preconditioning state with this new API because it has to be woken up each time which drains the battery?

Does this already work? Tesla deactivated the old API as of last week for me, making EVCC unusable with a TWC3.

Yes, but not for twc3 for reasons explained in https://github.com/evcc-io/evcc/issues/10460#issuecomment-1889196385

any idea when this is released? I cannot use my Tesla vehicle anymore because the old API is no more working.

When testing I get:
.Soc:     vehicle not available: cannot create vehicle 'template': invalid vehicle type: tesla-command

I don't understand why you're doing this:
https://github.com/evcc-io/evcc/pull/10802/files#diff-f3de7bfcd152edd6a1a65a3b65c58d9ee26c3ff5b32ee1c91af6e516854d7bbfR29

When generating the new API token using https://tesla.evcc.io/, which check boxes do I have to tick? All or only the ones with wakeup and lading data?

I also get: "creating vehicle Infinus failed: cannot create vehicle type 'template': cannot create vehicle type 'tesla-command': cannot get vehicles: unexpected status: 421 (Misdirected Request)"
It also changed vehicle name to infinus (offline)

@infinus please open issue with trace log for Tesla.

@infinus please open issue with trace log for Tesla. Which region are you located in? Sounds like teslamotors/vehicle-command#131.

Yes exactly what I am thinking and what I referenced in my other post.  I will open an issue.

Anyone can point me to a description on how to change to another(nightly) build? Running evcc as AddOn on HA.
Thanks!

Closing here since this has been implemented.


Column support for array concat

Which issue does this PR close?

Ref #6804
Rationale for this change

Array concat with columns! Behavior is similar to Postgres.
Concat Operator a || b is not included in this PR!
What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Not all kinds of tests are passed but I think they can be solved in another PR. I think we can review this PR first.

@izveigor @alamb Ready for review! Thanks!


Create rubiin.is-a-fullstack.dev

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description

Link to Website

rubiin.pages.dev

heads up, file must have .json at the end, go into files changed, and 3 dots on the top left of file editor then click edit and add ".json" at the end of the file

Fixed @MaragiDev . Thanks for the headsup


Pagination Support

Right now, pagination only works with selection queries with order by
SELECT foo, bar FROM myTable
  WHERE baz > 20
  ORDER BY bar DESC
  LIMIT 50, 100

Paginate the selection results from the ‘a’ th results and return at most ‘b’ results.
The below query doesn't paginate the results
SELECT count(*), foo, bar FROM myTable
  WHERE baz > 20
 GROUP BY foo
  ORDER BY bar DESC
  LIMIT 50, 100

Pagination for Pinot is something that we need to correctly design because the underlying data changes constantly (especially more often for the

will this be supported in the future?

@kishoreg how do we up vote this issue?

+1 is there any update on this?

Yes, it's a bit challenging but we have received this request multiple times. Please consider upvoting the feature

where do we upvote?

@kishoreg as discussed offline, I wj take this up

At LinkedIn, we have started to work on pagination on priority considering multiple requests we have received internally.
At a high level, our customer requirements are around the fact that they want to run query in Pinot that can potentially return a large response and user app does not want to accept the entire response in memory at once and want the ability to paginate the response as multiple result sets (size per result set dictated by the user app).
The current pagination implementation in Pinot (even if it is just for selection query) is sub-optimal in the sense that it takes each query as a fresh query and executes the query again and again for every pagination window, discard the results outside the window and provides the result within the M, N window that user has asked for.
The main thing to note about pagination is that it has to be treated as a single query.
Our customers don't want to run a one-off pagination query OFFSET M, FETCH N where M and N are completely random in which case it is not possible to reason about the results and it's even hard for the user to decide M as a one-off starting point.
The semantics that we want to provide is that "I want to fetch 10 million records from Pinot for a query and want to fetch 100K at a time"
So the customer will typically start with M as 0 and might just keep N fixed (say at 10000 or so) and just keep paging the results through multiple calls from their app which simply changes M during every call (and they potentially refresh the results in UI etc returned by Pinot in every call).
I think we should look at the pagination problem from this perspective as opposed to a random one-off pagination query because M and N don’t make any sense for a random query. Result of a random pagination query doesn’t add any value to the user since they want to look at the entire result as a continuous stream of results with the will to stop anytime.
We are trying to tackle the problem from the above perspective when trying to provide pagination semantics. Detail design discussion is in progress.
Some more thoughts slightly related to this --
Now, one problem is that users who run such queries may have the tendency to think that support for pagination means they can run "any" query in Pinot that can be very long running and Pinot is guaranteed to finish it and provide results. This can easily cause OOM (out of memory) and bring down the cluster.
Pinot is unlikely to enter the territory of running very long running queries and getting the entire 100% accurate result by spilling to disk and avoiding OOM at all times. Presto should be used for those cases.
However, for some of our users (who are ok with multi-second latency and prefer slightly more accurate response for GROUP BY queries),  as a follow-up / next phase, we want to consider enhancing support in Pinot for queries that return large responses and/or process / aggregate more than usual amounts of data. We want to do this by doing some of the memory intensive query execution operations in off-heap (direct) memory.

Hi @siddharthteotia Your comment looks good to me as our requirements are very similar. I will add - currently, we use the JDBC client to access our pinot databases via a spring application, but if we could get results paged directly from pinot to postgres (what we use for interim storage), that would meet our needs as well.

Hi all,
This is the proposed design doc on pagination support:
https://docs.google.com/document/d/1sNch7Ck-H06De2mb0Bj67ajEBMawrLJn_MxNzDEIxPU/edit?usp=sharing
Please take a look at it and feel free to put any comments. Thanks!
Best,
Jack

cc @walterddr ^^

Hi @jackjlli, wondering if you are still working on this and if so, any timeline for the same?

hey @mayankshriv  - yes we are working on this.

Hey @siddharthteotia any update on this feature?

@mayankshriv  - Any update on this feature? Your help will be really appreciated.

any update here?

Any update here?

cc @vrajat


osm/smi: update dir structure and add test

Updates to collect osm custom resource meshconfig per latest osm release
Fixes collection of smi crds (rbac in cluster-role)
Adds naming structure to filename: meshName/resourceName_resourceType and smi/resourceName_resourceType>
Tested with kustomize kubectl apply -k deployment

Codecov Report

Merging #118 (5b854c3) into master (118d62a) will decrease coverage by 0.53%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
- Coverage   31.46%   30.92%   -0.54%
==========================================
  Files          12       12
  Lines         518      527       +9
==========================================
  Hits          163      163
- Misses        340      349       +9
  Partials       15       15

Impacted Files
Coverage Δ

pkg/collector/osm_collector.go
0.00% <0.00%> (ø)

pkg/collector/smi_collector.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

💡 🙏 Thanks for this, there are few comments, especially the yaml changes are something which need re-look. I see mainly PR is just the name and folder structure change and if you have tested this and only thing left is the yaml changes which is not needed in the core. Thank you for adding very initial test class for this as well.
Can you please open a tracking item for the conversion of these collector to client-go we have already implemented logic clients and podDecribe For example a sample reside here: https://github.com/Azure/aks-periscope/blob/master/pkg/collector/kubeobjects_collector.go#L54
for future purpose regarding the client-go > api based implementation, I can help and implement this the conversion with you but I will need someone who understand the SMI/OSM scenario to test them alongside. 🙏☕
Thank you so much,

That is correct, I've now updated the title of the PR to reflect the same and opened an issue to track the conversion to client go here #124. Thanks for your help with this.


osd commands support "/dev/disk/by-path/".

osd arguments can contain disk definitions by path.
/dev/disk/by-path/pci-0000:00:1f.2-ata-1.0
But these require : to delimit the host, disk and potentially journal.
Since argparse swallows "' characters. But since " and ' evaluate
to " and ' respectively we can use shlex to handle this correctly.
Hence a valid ceph-deploy command can now be:
ceph-deploy osd prepare --zap \
    ceph-node3:\'/dev/disk/by-path/pci-0000:00:1f.2-ata-1.0\'

or:
ceph-deploy osd prepare --zap \
    ceph-node3:\"/dev/disk/by-path/pci-0000:00:1f.2-ata-1.0\"

Signed-off-by: Owen Synge<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-deploy-pull-requests/1751/
Test FAILed.

Am I correct in believing that the tests are failing for no good reason?
They seem to fail for me on both master and with this patch.

@osynge I would say they are failing correctly with assertion Errors. I would also suggest that this patch needs unit tests for the new logic/behavior added

Everything passes for me in master:
$ tox
...
  py26: commands succeeded
  py27: commands succeeded
  flake8: commands succeeded
  congratulations :)
$ git status
On branch master
Your branch is up-to-date with 'origin/master'.

Running tox I get the output:
2029 passed, 16 skipped in 46.46 seconds
But I do get some errors that I am not able to find python 2.6

@osynge you need to have Python 2.6 and 2.7 installed to be able to run all the environments that ceph-deploy is configured to execute with tox. Again: this PR has valid failures that you need to correct, and I would strongly suggest adding unit tests to support your changes.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-deploy-pull-requests/1786/
Test PASSed.

@alfredodeza I did have a bug, annoying issues with touple and list that had no impact, but made the test suite fail.
I followed your advice and added some tests to support the current behaviour and the changes.
Unrelated to this patch : would you recommend adding the same / similar tests for "ceph disk create" and "ceph disk activate" or is adding them for "ceph disk prepare" enough?


update bagel bids subcommand to play nice with new derivatives

TODO
update the bagel bids command to

[x] accept either a pheno.jsonld (output of bagel pheno) or a pheno-derivatives.jsonld (output of bagel derivatives #345) as input
[x] update help text
[x] find and match existing imaging sessions between a provided pheno-derivatives.jsonld and a raw BIDS dataset so that we do not recreate imaging sessions if the derivatives subcommand has already created them

[x] if session is missing, add to potentially existing "ses-nb01"?

Related to #345

Closed by #349


add additional fields to kubectl get vm/vmi

What this PR does / why we need it:

add fields running and primary volume to kubectl get vm
add fields phase, ip and node name to kubectl get vmi

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

kubectl get vms is enhanced so that default output contains additional running and primary volume
fields
NAME      AGE       RUNNING   VOLUME
windows   3d        true
xx        3d        true

kubectl get vmis is enhanced so that default output contains additional phase, ip and nodeName

NAME      AGE       PHASE     IP            NODENAME
magic1    3d        Running   <IP_ADDRESS>   node01.default
magic2    3d        Running   <IP_ADDRESS>   node01.default
windows   3d        Running   <IP_ADDRESS>   node02.default
xx        3d        Running   <IP_ADDRESS>   node02.default

Please add a release note :)

@karmab can you please provide some example output of kubectl get?

This is just so awesome.

Test failures seem unrelated - merging this as it's to awesome.


Add sorting support for track by $index

This addresses an issue that exists today where the table is not sorted correctly. The following conditions must be met for this to occur:

The repeater uses track by $index
Chrome is the browser
There are more than 10 items in the array

The items should be sorted according to their index in the original array.  However, that is not the case. Here is a plunk to reproduce the issue as well as the fix that I am submitting with this PR.
https://embed.plnkr.co/vnpmSm/
The fix I am proposing identifies the track by $index clause and sorts the data accordingly.

@mattiash Any chance this could be merged in soon?

Included in 1.6.0. Thanks for the patch!


No rDNS updated with pre-created floating ip

https://github.com/linux-noob/hcloud-k8s/blob/43447bc7094ac9ebafd426f3dc3a7bb9ad2402c2/roles/tf-infrastructure/terraform/main.tf#L79
does not update rDNS for floating ip when use_my_ip = 1

I think that is expected feature when you have already that IP, you probably also want manage the rDNS manually on it?
To change already created rdns, you need to import that ip somehow. Or Dunno. Maybe we could get the id of ip somehow like data.hcloud_floating_ip.lbipv4.id? Need to test.
Do you want work on this, or do I check it?


Logging of validation metrics should not include median filtering

Currently, the training and validation metrics are treated differently and are logged with a median window - this is clearly something we want to avoid.

Likely addressed in https://github.com/directgroup/direct/commit/7c6bbebe69d1d8d544a0e7f8abce16ab2f55351f - need to verify

Has been addressed.


Add MANIFEST.in

The minimal docs should be part of the source tarball.
Could you please make a new release and push it to PyPI after merging? Thanks
Depends on #15

Hi @fabaff ,
I will update the PyPI package soon, I will let you know when it's updated.

Hi @fabaff ,
I just updated the package on PyPI: https://pypi.org/manage/project/ajpy/release/0.0.5/
Let me know if it misses something.

Thanks. Fedora package is updated now.


Use cordova-ios 4.1.0 in Milestone 1.7.0

There is a bug in cordova-ios 4.0 that can cause the app to lock up. Force people to upgrade to cordova-ios 4.1.x

For future reference or mention: https://issues.apache.org/jira/browse/CB-10530

@fredgalvao thanks for the JIRA link. I hate searching through that mess.


Support for pattern matching?

starting from version 2.6, SMTLib supports pattern matching.
Should we support pattern matching as well?
For example
IF isPoint2D(p)
  THEN x(p) >0 /\ y(p) > 0
  ELSE IF isPoint3D(p)
            THEN x(p) > 0 /\ y(p) > 0 /\ z(p) > 0
            ELSE False  -- Not possible
          FI
FI

can become (something like)
CASE p OF
      Point2D x y    -> x > 0 /\ y > 0
      Point3D x y z  -> x > 0 /\ y > 0 /\ z > 0
ESAC

Also code generation becomes simpler and maybe a little bit faster.

Do we want to support wildcards in pattern matching?
Haskell uses _
I would prefer ANY see #247.


Unnecessary slash is being inserted into assets URLs when baseHref option doesn't have one

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Area
- [x] devkit
- [ ] schematics

Versions

My operating system is macOS High Sierra.
Repro steps

I have a font with an absolute URL in my SCSS file:

 @font-face {
    font-family: 'some-font';
    src: url('/assets/some-font.woff') format('woff');
  }

In my angular-cli.config I have this configuration for styles:

      "styles": [
        "styles/styles.scss"
      ]

Styles.scss imports the file with a font-face definition.

In all of the cases listed below (disregard what value is being used for base-href) I'm geting the same results for fonts url.

Build commands:
- ng build --base-href '$(SOME_VAR)/'
- ng build --base-href '/$(SOME_VAR)'
- ng build --base-href '$(SOME_VAR)'
- ng build --base-href '/$(SOME_VAR)/'

The build output I'm getting for my styles:
url('/$(SOME_VAR)/assets/some-font.woff') format('woff');
The log given by the failure

No error log
Desired functionality

There should not be any additional slashes in case there is a base-href option explicitly provided. The thing is, $(SOME_VAR) already has the required slash in my case, but it is only being replaced during CI when deploy is happening (which is after the ng build).
Right now, I'm getting //base-path/assets/ path for fonts on my website, which browser interprets as http://base-path/assets/, while what I need is http://my-site.com/base-path/assets/ and it breaks the deployment.

The CLI has been updated significantly since version 1.6.8.  The current stable version is 8.3.5.  If this problem is still being encountered with the current version, please consider opening a new issue with updated details.


Raise exception in String#ord when given invalid UTF-8 byte sequence

CRuby:
[2.6.3] > "\xFE".ord
Traceback (most recent call last):
        5: from /usr/local/var/rbenv/versions/2.6.3/bin/irb:23:in `<main>'
        4: from /usr/local/var/rbenv/versions/2.6.3/bin/irb:23:in `load'
        3: from /usr/local/var/rbenv/versions/2.6.3/lib/ruby/gems/2.6.0/gems/irb-1.0.0/exe/irb:11:in `<top (required)>'
        2: from (irb):2
        1: from (irb):2:in `ord'
ArgumentError (invalid byte sequence in UTF-8)

rirb:
>>> "\xFE".ord
=> nil

CRuby:
[2.6.3] > "a\xFE".ord
=> 97

rirb:
>>> "a\xFE".ord
=> nil

CRuby:
[2.6.3] > "💪".ord
=> 128170
[2.6.3] > "💪\xFE".ord
=> 12817

rirb:
>>> "💪".ord
=> 128170
>>> "💪\xFE".ord
=> nil


delete your api key

your api key to airtable is open to everyone

good.
you should also rotate it. see https://support.airtable.com/hc/en-us/articles/219046777-How-do-I-get-my-API-key-

already did -- thanks for looking out!


ci: enable gofumpt with extra

Signed-off-by: Furkan<EMAIL_ADDRESS>Ran $ gofumpt -w -extra .
-w	write result to (source) file instead of stdout
-extra
    	enable extra rules which should be vetted by a human

Summary

Ticket Link

Fixes
Release Note

Looks like this has a new lint issue caused by the change!

hello, any updates on this PR? thanks!

Can someone please approve? Thanks!

@Dentrax you need to rebase and fix the conflicts :)

Missed that! Rebased.

@Dentrax can you please rebase? just to get the latest from the main branch? thanks

need a new rebase 😓

i will do a rebase to we get this merged


Merge pull request #1 from jamesagnew/master

update version

Hi @dqsoft - I'm assuming this was accidental?

Closing this


When I finished cardano-db-sync migration , Cardano-graphql logs show me some strange log.

Ask a question
inputoutput/cardano-graphql:7.0.0
inputoutput/cardano-graphql-hasura:7.0.0
cardanosolutions/ogmios:v5.2.0-mainnet
inputoutput/cardano-db-sync:13.0.4

Finished migration, and then I bump up cardano-graphql with restart.
then logs show me like below.
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"ChainFollower","msg":"Rolling back to genesis","time":"2022-09-13T09:49:06.030Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"ChainFollower","msg":"Deleted 0 assets","time":"2022-09-13T09:49:06.041Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:49:27.640Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:49:46.122Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:49:59.442Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:50:20.682Z","v":0}
(node:1) UnhandledPromiseRejectionWarning: Error: database query error: {"response":{"errors":[{"extensions":{"path":"$","code":"unexpected"},"message":"database query error"}],"status":200},"request":{"query":"query {\n          epochs (limit: 1, order_by: { number: desc }) {\n              adaPots {\n                  reserves\n              }\n          }\n          rewards_aggregate {\n              aggregate {\n                  sum {\n                      amount\n                  }\n              }\n          }\n          utxos_aggregate {\n              aggregate {\n                  sum {\n                      value\n                  }\n              }\n          }\n          withdrawals_aggregate {\n              aggregate {\n                  sum {\n                      amount\n                  }\n              }\n          }\n      }"}}
    at GraphQLClient.<anonymous> (/app/node_modules/graphql-request/dist/index.js:170:35)
    at step (/app/node_modules/graphql-request/dist/index.js:63:23)
    at Object.next (/app/node_modules/graphql-request/dist/index.js:44:53)
    at fulfilled (/app/node_modules/graphql-request/dist/index.js:35:58)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:1) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:50:38.905Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:50:59.167Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:51:20.440Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:51:39.442Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:52:00.601Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:52:21.774Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:52:43.603Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:53:03.395Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:53:24.230Z","v":0}
{"name":"cardano-graphql","hostname":"14ce5486b14e","pid":1,"level":30,"module":"Server","msg":"Sync Progress: cardano-db-sync: 100% | Asset: 0%","time":"2022-09-13T09:53:45.979Z","v":0}
.............

Those are intended?
Necessary to wait?
For the reference , my migration process like below.

Create postgres14 container.
Create User & password with database name cexplorer
Following process came from cardano-db-sync ( meant that using snapshot)
ref : https://github.com/input-output-hk/cardano-db-sync/blob/release/13.0.x/doc/state-snapshot.md
Then looked like migration process has been done. Restart cardano-db-sync.
Cardano-db-sync has been reached latest height according to its log

Is Hasura running? Please provide some logs from that container around the time of this error being triggered.

Show your log from PostgreSQL. It should contain the query error


Revert "Move class property transformation into new transformer file"

Breaks RWC tests. #31805 solved #31792, but there were more failures in the RWC suite.

@typescript-bot test this

this reverts https://github.com/microsoft/TypeScript/pull/30467


rest-api docu

Hi,
just have been through the doco after a while and in the error section it says:
{
    'type': "error",
    'status': "Failure",
    'status_code': 400,
    'metadata': {}                      # More details about the error
}

Isn't it more like this:
{
    'type': "error",
    'error': "This is an error",
    'error_code': 400,
    'metadata': {}                      # More details about the error
}

We had a plan to use a common terminology for all return values but that never quite happened. I'd be happy to merge a branch which makes the rest api doc reflect reality.


Implement fix to support jupyter-lab notebooks

Please confirm that this PR has done the following:

~Tests Added~ (no tests for Jupiter-lab notebooks)
[x] Documentation Added
~Name of contributors Added to AUTHORS.rst~
[x] Description in RELEASE_NOTES.md Added

Description of PR
This PR adapts the special handling when importing pyam from a notebook. As reported by @stefaneidelloth in #591, the deactivation of in-cell scrolling for Jupyter notebooks causes a critical error in jupyter-lab notebooks.
closes #591

Tested this locally on both Jupyter notebooks and jupyter-lab notebooks on Mac OS with Python 3.9 and the following (selected) packages and it works as expected.
ipykernel           6.4.1
ipython             7.30.1
jupyter-client      7.1.0
jupyter-core        4.9.1
jupyter-server      1.12.1
jupyterlab          3.2.4
jupyterlab-pygments 0.1.2
jupyterlab-server   2.8.2

@stefaneidelloth, could you please install from this branch using
pip  install git+https://github.com/danielhuppmann/pyam.git@notebook/jupyterlab

(recommended in a new environment) and let me know if this solves the problem on your machine?

Yes, solves the problem. Thank you!


[1Password] ...

Extension
https://github.com/raycast/extensions/tree/34c65c7f190ad172e46c75384600aa938f932e98/extensions/1password/
Description
Error: Worker terminated due to reaching memory limit: JS heap out of memory
NodeError:errors:393:5
[kOnExit]:worker:277:26
Worker..onexit:worker:199:20
Steps To Reproduce

Just tried to access a password with the 1p command.

Current Behaviour
No response
Expected Behaviour
No response

Do you still have this issue @JonCrawford

Looks ok today.