Support Server-Side Encryption

https://forums.aws.amazon.com/ann.jspa?annID=5434
https://aws.amazon.com/blogs/aws/new-encryption-at-rest-for-dynamodb/
CreateTable parameter is {"SSESpecification": {"Enabled": true}}
DescribeTable field is "SSEDescription": {"Status": STRING_ENUM} where STRING_ENUM is one of {ENABLING, ENABLED, DISABLING, DISABLED}
Like stream and ttl, This will be specified on the Meta object:
class User(BaseModel):
    id = Column(String, hash_key=True)
    expire_after = Column(Timestamp, dynamo_name="exp")

    class Meta:
        stream = {"include": {"new", "old"}}
        ttl = {"column": "expire_after"}
        encryption = {"enabled": True}

As with all optional meta config, when encryption isn't provided an existing value won't be validated.  However a value is specified, it will be validated against the return value when binding the model.

DescribeTable won't always return SSEDescription so sanitize_table_description needs to inject when missing with "Status": "DISABLED"


linter test failure

From @jmesserly on October 15, 2015 17:3
I'm seeing a linter failure when I pub run test -j1
00:06 +9 -1: test/driver_test.dart: Driver linter generates lints

  Expected: contains '[lint] Name types using UpperCamelCase.'
    Actual: 'Analyzing [test/data/linter_project/test_file.dart]...\n'
    'No issues found\n'
    ''

  package:test                 expect
  test/driver_test.dart 129:9  main.<fn>.<fn>.<fn>

Not sure if this was caused by the Analyzer update?
Copied from original issue: dart-lang/analyzer_cli#71

From @jmesserly on October 15, 2015 17:7
This might be unrelated, but the tests in linter package have "style_guide" key in between, does that matter?
rules:
  style_guide:

Fixed.


Unable to access 'https://github.com/openthread/wpantund.git/': Couldn't connect to server.

I am trying to install ot-br-posix in Raspberry Pi 3B. I am encountering an issue regarding cloning of wpantund repository. PFA for reference.
Version which I am trying to build:
commit 699aa6a59308f6e913e3efbe57099113c8aaf744
Author: Yakun Xu<EMAIL_ADDRESS>Date:   Wed Apr 29 14:07:51 2020 +0800

I checked following:

Internet is working fine.
When I try to clone wpantund repository manually by executing git clone https://github.com/openthread/wpantund.git command, it gets cloned successfully.

What is the raspbian version?

@bukepo
Linux 4.19.42-v7+ #1219 SMP Tue May 14 21:20:58 BST 2019 armv7l GNU/Linux
PRETTY_NAME="Raspbian GNU/Linux 9 (stretch)"
NAME="Raspbian GNU/Linux"
VERSION_ID="9"
VERSION="9 (stretch)"
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

@tarunsmalviya we just removed wpantund support in this project, could you try the latest code?

@bukepo You mean to say stretch version is not supported. Right?

@tarunsmalviya , with #424, wpantund is no longer used in master branch. As a result, the wpantund issue you are facing should no longer exist. As @bukepo suggested, can you try installing with the latest master branch? Note that the Border Router Setup Guide has been updated to reflect master's migration to using RCP mode.

@jwhui #424 was merged 12 days ago and I was facing this issue before that.
Also, as per @bukepo's suggestion, I tried installing with latest master branch. I discovered that there are many changes done in the new version.
I have following queries:

Also, in the Border Router Setup Guide, there is no mention of NCP. Is this because it does not exists now? RCP came into picture now?

Until now, I was using 0.08.00d (/0.07.01-341-g1f65656; May 11 2020 21:06:03) version of ot-br-posix. When I updated ot-br service, wpanctl was miss behaving even when the wpantund still exists. Sometimes, the output of sudo wpanctl status command shows offline state and sometimes uninitialized state. Also, NCP was attached but thread network was not coming up in RPi.

Is it so that earlier version is now not deprecated and migration from older to newer version is not handled? We will be needing RCP instead of NCP.

‘’‘Also, in the Border Router Setup Guide, there is no mention of NCP. Is this because it does not exists now? RCP came into picture now?
Yes, RCP is now the recommended radio peripheral rather than NCP.

In the wpantund solution otbr shouldn't interfere with Thread network creation. You need to check the wpantund log to identify the error.

```Is it so that earlier version is now deprecated and migration from older to newer version is not handled? We will be needing RCP instead of NCP.

There is a feature to migrate ncp network data to rcp in OpenThread. You can find how to use it [here](https://github.com/openthread/openthread/blob/master/script/check-ncp-rcp-migrate).
After you've acquired the network data on the host machine, you can form the Thread network with this data and the network shall be migrated.

Closing stale issue.


add support for mocha

allow magellan to run test with mocha
update logger

Codecov Report

Merging #5 into master will increase coverage by 8.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   88.05%   96.61%   +8.55%
==========================================
  Files           5        4       -1
  Lines         134      118      -16
  Branches       18       19       +1
==========================================
- Hits          118      114       -4
+ Misses         16        4      -12

Impacted Files
Coverage Δ

lib/logger.js

lib/profile.js
94.93% <0%> (+0.65%)
:arrow_up:

lib/executor.js
100% <0%> (+8.69%)
:arrow_up:

Continue to review full report at Codecov.

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


Quickstart starts a Datahub version without matching PyPi packages

Describe the bug
When running the datahub docker quickstart it seems it now uses acryldata/datahub-frontend-react:head (which seems to be v0.14.1?) and the when trying to use ingestion it cannot create a virtual env since there is no corresponding PyPi packages. Below is how this problem presents itself when trying to ingest metadata from Postgres.
To Reproduce
Steps to reproduce the behavior:

Follow Datahub Quickstart
Login using demo "datahub" account
Optional: to generate the below logs: Try to run metadata ingestion for Postgres

~~~~ Execution Summary - RUN_INGEST ~~~~
Execution finished with errors.
{'exec_id': '1d1c49ba-6f39-4b31-8601-56dd6ac5143a',
 'infos': ['2024-09-18 11:02:38.902353 INFO: Starting execution for task with name=RUN_INGEST',
           "2024-09-18 11:02:43.006269 INFO: Failed to execute 'datahub ingest', exit code 1",
           '2024-09-18 11:02:43.008867 INFO: Caught exception EXECUTING task_id=1d1c49ba-6f39-4b31-8601-56dd6ac5143a, name=RUN_INGEST, '
           'stacktrace=Traceback (most recent call last):\n'
           '  File "/datahub-ingestion/.venv/lib/python3.10/site-packages/acryl/executor/execution/default_executor.py", line 139, in execute_task\n'
           '    task_event_loop.run_until_complete(task_future)\n'
           '  File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete\n'
           '    return future.result()\n'
           '  File "/datahub-ingestion/.venv/lib/python3.10/site-packages/acryl/executor/execution/sub_process_ingestion_task.py", line 400, in '
           'execute\n'
           '    raise TaskError("Failed to execute \'datahub ingest\'")\n'
           "acryl.executor.execution.task.TaskError: Failed to execute 'datahub ingest'\n"],
 'errors': []}

~~~~ Ingestion Logs ~~~~
Obtaining venv creation lock...
Acquired venv creation lock
venv doesn't exist.. minting..
Using Python 3.10.12 interpreter at: /usr/bin/python3
Creating virtualenv at: /tmp/datahub/ingest/venv-postgres-3cbb1ad0ed8a0388
Resolved 3 packages in 690ms
Prepared 3 packages in 268ms
Installed 3 packages in 972ms
 + pip==24.2
 + setuptools==75.1.0
 + wheel==0.44.0
+ uv pip install 'acryl-datahub[datahub-rest,datahub-kafka,postgres]==0.14.1'
  × No solution found when resolving dependencies:
  ╰─▶ Because there is no version of acryl-datahub[postgres]==0.14.1 and
      you require acryl-datahub[postgres]==0.14.1, we can conclude that your
      requirements are unsatisfiable.

Expected behavior
Ingestion in the quickstart version works. Quickstart uses a fully supported version. Alternatively update the documentation to cover this issue.
Desktop (please complete the following information):

OS: Windows 11 using WSL
Browser Microsoft Edge Version 128.0.2739.79

The issue itself resolved as the PyPi packages were updated with the 0.14.1 version. However the root cause still exists and should be fixed.


Fix for signature exceptions in ptransform_fn.

For some cases (e.g. builting functions) signature retrieval fails. In this case, don't attempt to patch the (non-available) signature.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @damccorm


Shall we add a series for insulation to the total costs chart?

Now the insulation costs are nested under 'heat production & network', shall we create a separate series for them?

@AlexanderWirtz @antw @jorisberkhout @MartLubben @redekok  @michieldenhaan @ChaelKruip @DorinevanderVlies

👍

Yes! And add them to the investment table


Conda PATH entries added to end of $PATH

Environment data

VS Code version: 1.28.2
Extension version (available under the Extensions sidebar): 2018.19.1
OS and version: macOS 10.14
Python version (& distribution if applicable, e.g. Anaconda): python 3.7, Miniconda
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda

Actual behavior
The conda PATH entires are added to the end of $PATH (or that's what the integrated terminal tells me) meaning which python gives /usr/bin/python on macOS.
[rdonnelly@0558 conda-build]$ echo $PATH
/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Applications/VMware Fusion.app/Contents/Public:/opt/conda/bin
[rdonnelly@0558 conda-build]$ which python
/usr/bin/python
[rdonnelly@0558 conda-build]$

Expected  behavior
The conda PATH entries should be added to the front of $PATH.
I should point out that when debugging, the right python interpreter is run, however print(os.environ['PATH']) shows the same problem, and during execution of, say, conda-build, when it fetches sources from a git repo the too-old system git executable gets found instead of the more modern one that has been installed into the conda environment.
Steps to reproduce:

XXX

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)
XXX

The prom extension does not control the environment in the terminal.
You might want to report this upstream in Conda or VSCode, on how you should set up your paths.

It is not a conda issue (I work for Anaconda, Inc on packaging, conda-build and sometimes conda). Conda only every appends to the front of path so something else must be going on.
I don't know anything about the internals of VSCode, how do extensions send PATH modification information back to the app? Is there any way to specify that it should be prepended and not appended? How and where are multiple plugin path entries combined?

I use the newish conda stuff here too btw.


refactor: add 'your alerts' to settings app

The type of this PR is: Enhancement

This PR addresses GRO-654
Description

The wording in the ticket specifies that the code should be migrated, but this seems to have already been addressed. Here I added a tab to the settings app for easier navigation.

Looks great 👍 , mergin'


feat(using last-trained-or-published instead of last-trained-and-last…

…-publish command):

:tada: This PR is included in version 7.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


the tunnel will become offline a few hours later. I need to turn off and turn on the tunnel to make the tunnel online.

Type: Bug
1.turn on tunnel from gui.
2.a few hours later,it will be disconnect and can't reconnect with the 401 error
2024-09-27 15:21:09.190 [info] *

2024-09-27 15:21:09.191 [info] * Visual Studio Code Server

2024-09-27 15:21:09.191 [info] *

2024-09-27 15:21:09.191 [info] * By using the software, you agree to

2024-09-27 15:21:09.191 [info] * the Visual Studio Code Server License Terms (https://aka.ms/vscode-server-license) and

2024-09-27 15:21:09.191 [info] * the Microsoft Privacy Statement (https://privacy.microsoft.com/en-US/privacystatement).

2024-09-27 15:21:09.191 [info] *

2024-09-27 15:21:09.191 [info] [2024-09-27 15:21:09] warn Command-line options will not be applied until the existing tunnel exits.

2024-09-27 15:21:09.191 [info] Connected to an existing tunnel process running on this machine.

2024-09-27 15:21:09.191 [info]

2024-09-27 15:21:09.191 [info] Open this link in your browser https://vscode.dev/tunnel/xxx

2024-09-27 15:21:09.191 [info]

2024-09-27 18:06:58.822 [info] [2024-09-27 18:06:58] info [tunnels::connections::ws] error reading websocket: WebSocket protocol error: Connection reset without closing handshake

2024-09-27 18:06:58.822 [info] [2024-09-27 18:06:58] warn Tunnel exited unexpectedly but gracefully, reconnecting

2024-09-27 18:07:04.962 [info] [2024-09-27 18:07:04] warn Error refreshing access token, will retry: failed to lookup tunnel for host token: response error: HTTP status 401 Unauthorized from https://usw3.rel.tunnels.api.visualstudio.com/tunnels/neat-fog-ctp684q?tokenScopes=host&api-version=2023-09-27-preview (request ID c4f355f1-6b1f-42d2-a458-11de7c90d2cb):

2024-09-27 18:07:15.142 [info] [2024-09-27 18:07:15] warn Error refreshing access token, will retry: failed to lookup tunnel for host token: response error: HTTP status 401 Unauthorized from https://usw3.rel.tunnels.api.visualstudio.com/tunnels/neat-fog-ctp684q?tokenScopes=host&api-version=2023-09-27-preview (request ID d1a1d734-64d8-4389-919c-e9d19c1d988d):

VS Code version: Code 1.93.1 (38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40, 2024-09-11T17:20:05.685Z)
OS version: Windows_NT x64 10.0.26100
Modes:

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i5-12500 (12 x 2995)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
23.70GB (8.51GB free)

Process Argv
--crash-reporter-id 8e70160f-a1a2-44d6-b665-77820c8e25c0

Screen Reader
no

VM
0%

Extensions (46)

Extension
Author (truncated)
Version

vscode-dbf
ape
0.1.2

vscode-intelephense-client
bme
1.12.6

esbuild-problem-matchers
con
0.0.3

vscode-eslint
dba
3.0.10

githistory
don
0.6.20

gitlens
eam
15.5.1

prettier-vscode
esb
11.0.0

copilot
Git
1.234.0

copilot-chat
Git
0.20.3

vscode-github-actions
git
0.27.0

vscode-pull-request-github
Git
0.96.0

go
gol
0.42.1

gc-excelviewer
Gra
4.2.62

gitlab-mr
jas
1.4.3

intellij-idea-keybindings
k--
1.7.2

rainbow-csv
mec
3.12.0

vscode-docker
ms-
1.29.3

vscode-language-pack-zh-hans
MS-
1.93.2024091109

remote-ssh
ms-
0.114.3

remote-ssh-edit
ms-
0.86.0

remote-wsl
ms-
0.88.4

powershell
ms-
2024.2.2

remote-explorer
ms-
0.4.3

remote-server
ms-
1.5.2

vscode-speech
ms-
0.10.0

vscode-speech-language-pack-zh-cn
ms-
0.4.0

material-icon-theme
PKi
5.11.1

fabric8-analytics
red
0.9.5

java
red
1.35.0

vscode-xml
red
0.27.1

intellicode-api-usage-examples
Vis
0.2.8

vscodeintellicode
Vis
1.3.1

vscode-boot-dev-pack
vmw
0.2.1

vscode-spring-boot
vmw
1.57.0

vscode-gradle
vsc
3.16.4

vscode-java-debug
vsc
0.58.0

vscode-java-dependency
vsc
0.24.0

vscode-java-pack
vsc
0.29.0

vscode-java-test
vsc
0.42.0

vscode-maven
vsc
0.44.0

vscode-spring-boot-dashboard
vsc
0.14.0

vscode-spring-initializr
vsc
0.11.2

volar
Vue
2.1.6

markdown-all-in-one
yzh
3.6.2

material-theme
zhu
3.17.5

vscode-proto3
zxh
0.5.5

(6 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
f3je6385:31013174
a69g1124:31058053
dvdeprecation:31068756
dwnewjupytercf:31046870
newcmakeconfigv2:31071590
impr_priority:31102340
nativerepl1:31139838
refactort:31108082
pythonrstrctxt:31112756
flighttreat:31134774
wkspc-onlycs-t:31132770
wkspc-ranged-c:31125598
cf971741:31144450
autoexpandse:31146404
iacca2:31144504
5fd0e150:31146321

VS Code tunnels are unstable and require manual stop/restart constantly since early August. It's a mess.

VS Code remote used to be rock-solid, but is now horribly unreliable since early August. I used to be able to keep tunnels running for days, but now, they fail typically within 24 hours have to be restarted.  It seems the tokens don't last or become invalid
2024-10-01 14:29:27.110 [info] [2024-10-01 14:29:27] error failed to keep token alive: StatusError { url: "https://api.github.com/user", status_code: 401, body: "{\"message\":\"Bad credentials\",\"documentation_url\":\"https://docs.github.com/rest\",\"status\":\"401\"}" }

2024-10-01 14:29:27.111 [info] [2024-10-01 14:29:27] error access token is no longer valid, exiting: error requesting https://api.github.com/user: 401 {"message":"Bad credentials","documentation_url":"https://docs.github.com/rest","status":"401"}

2024-10-03 03:25:35.024 [info] [2024-10-03 03:25:35] info [tunnels::connections::ws] error reading websocket: WebSocket protocol error: Connection reset without closing handshake

@kingpalethe
For me, there's no problem using GitHub, but when I use a Microsoft account, it ends up like this: after a few hours, it shows as expired and then returns a 401 error.

Yes, same problem I am reporting here: https://github.com/microsoft/vscode/issues/230223
Maybe we can develop a hack or script to constantly run to try to convince the tunnel not to disconnect?

Anyone find a workaround for this? My tunnels used to work great, but I am close to giving up on VS Code tunnels since they became unreliable in early August.

I think it's just that Github won't allow you to stay logged in for more than a couple days. So VS Code tunnels isn't a stable solution if you need access long-term...

Unfortunately Github doesn't have a "refresh token" endpoint. From talking to them, at least last year, they essentially keep an LRU and have the last N access tokens be valid. To help with this the CLI will make a request with the access token every hour while the tunnel is running, but this might not be enough in every case.

@connor4312 Thanks for the tip on this. Maybe we just need a script that will make this request several times per hour instead of just every hour?

@connor4312 Also -- do you think this problem will be more unlikely if we switch to using a Microsoft account instead of a Github account? Also -- the mystery for me is why this problem started in early August. Prior to that, I was using tunnels for months -- maybe years -- and it seemed to be rock-solid and I never got logged out like this.... Is there something that changed?  Thanks!

Microsoft may be a bit better -- their token expires, but there is a proper refresh endpoint that we hit to make sure it's renewed


TypeError: count() got an unexpected keyword argument 'contig'

Hi,
Thanks for making ConFindr.
I'm getting the following error:

2018-10-11 15:53:11  Welcome to ConFindr 0.4.2! Beginning analysis of your sam                                                                                ples...
2018-10-11 15:53:11  Beginning analysis of sample MSHR0120...
2018-10-11 15:53:11  Checking for cross-species contamination...
2018-10-11 15:54:33  Setting up genus-specific database for genus Burkholderia                                                                                ...
2018-10-11 15:57:08  Extracting rMLST genes...
2018-10-11 15:57:38  Quality trimming...
2018-10-11 15:57:40  Detecting contamination...
Traceback (most recent call last):
File "/home/aaziz/bin/miniconda3/bin/confindr.py", line 889, in
base_cutoff=args.base_cutoff)
File "/home/aaziz/bin/miniconda3/bin/confindr.py", line 529, in find_contamina                                                                                tion
rmlst_report=rmlst_report)
File "/home/aaziz/bin/miniconda3/bin/confindr.py", line 351, in find_rmlst_typ                                                                                e
gene_alleles_to_use[gene] = [allele, bamfile.count(contig=contig)]
File "pysam/libcalignmentfile.pyx", line 1071, in pysam.libcalignmentfile.Alig                                                                                nmentFile.count (pysam/libcalignmentfile.c:13063)
TypeError: count() got an unexpected keyword argument 'contig'

The command I ran:
confindr.py -i PATH -o output -t 8 -k -fid _1_sequence -rid _2_sequence -d PATH
Also ran confindr on example dataset but received same error:
confindr.py -i example-data -o output_example -t 8 -fid _1_sequence -rid _2_sequence -d PATH
Running on:
Linux cheetah06 2.6.32-431.5.1.el6.x86_64 #1 SMP Wed Feb 12 00:41:43 UTC 2014 x8                              6_64 x86_64 x86_64 GNU/Linux
It was installed using conda as recommended. As the cluster is behind a firewall, I had to download the database manually but it seems to have been detected just fine. I've attached the log file that was created from the example dataset.
confindr_log.txt
Any help is appreciated!

I'm not at a computer today so I can't look into this in too much detail (but will tomorrow), but my first guess would be that the version of pysam being used isn't correct (I may need to update the contact recipe).
What's the output of pip list within your confindr conda environment?

Hi @lowandrew
Good call, updating pysam from version <IP_ADDRESS> to the latest fixed the problem and ConFindr is working.
While trying to update pysam, I ran into the issue of different versions in pip/conda. I'm not knowledge enough with Conda to figure out why ConFindr is using the pip install of pysam (which was outdated, causing the error) instead of the Conda version (which was up to date, probably updated during ConFindr installation).
pip, conda, and /confindr.py are all located in ~/bin/miniconda3/bin/

Also, found a small bug (I think). Nucleotide position of SNVs reported in example__contamination.csv file are off by 1, maybe due to python's indexing starting at 0?
Example:

BACT000030_569,203,T:46;G:6,52
BACT000030_569,206,G:49;T:3,52
BACT000030_569,425,C:84;A:2,86

Inspecting these SNVs manually in Tablet shows their position is actually +1.
Thanks

I'm also not sure why the incorrect version of the pysam package would be used, but I'll do some digging into it and hopefully get it sorted out.
As for the off by 1, you are correct - I'll update the positions in the next release of ConFindr. Thanks for pointing it out!

The off by 1 error has been fixed in recent releases - as for the incorrect version of pysam being used, do you have a $PYTHONPATH set? Apparently having that set may cause some issues.

That's probably the cause. You can unset $PYTHONPATH to stop anaconda from using other python versions' packages. Unfortunately, I can't test the solution because I don't have permissions to uninstall pysam from /usr/local/.
Thanks for your help!
On a different note completely, I've tested ConFindr on a small batch of libraries and it works very nicely. It found a mixture that we previously didn't know about. There were a few false positives (with default parameters), but these would probably be fixed by specifying -br as you've suggested.


Crash reports are still sent to the whatandroid email

You may want to change this to an email address you manage so you can add people to be forwarded email it gets for crash reports and feedback.

Thanks will, I'll get this done asap.

Fixed in 1.1.0


Exposed malloc_size on Mac

This change exposes malloc_size, which is available in libc on MacOSX.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.
Please see the contribution instructions for more information.

Could you also add it on https://github.com/rust-lang/libc/blob/master/libc-test/semver/apple.txt?

I may suggest to add malloc_good_size as well since they re somewhat related, but no worries it can be added in a separate PR otherwise :).

@JohnTitor I've added it to apple.txt. I've got to admit I am a little confused though; what is apple.txt actually used for?
@devnexen That is a good idea; I may as well add it while I am doing this, since they tend to be used together.

Let me check if it's spurious or not, @bors retry

:hourglass: Testing commit ada0d398b6d7345905e903f14c174ff6e579aaa7 with merge 35cd8f02aa2be40c8ed2d6c33089f2f3da216e7c...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 35cd8f02aa2be40c8ed2d6c33089f2f3da216e7c to master...


Go pro microservices

Hi,
Could you explain more about this issue?

Hi, only if you give me job
Вс, 12 нояб. 2023 г. в 17:04, Mehdi Hadeli @.***>:

Hi,
Could you explain more about this issue?
—
Reply to this email directly, view it on GitHub
https://github.com/mehdihadeli/go-ecommerce-microservices/issues/94#issuecomment-1807106425,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGXB6JHG4SA3KDJ55OYHSLYEC3LTAVCNFSM6AAAAAA7FHGITKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBXGEYDMNBSGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Abandonware?

No updates in a while.  Should this repo be considered abandoned and archived?

Yeah, the quality is quite trashy, it not functioning anymore the intended ware ... should be properly deleted ...

Pro-move, abandon the issues on the abandoned repo and never close them.


targets: filter by target

Previously, a user could only filter by the endpoint or namespace. Endpoints are generally the same between components and not useful for filtering, as well as being opaque. Namespaces are useful but not precise. Filtering by the target name (PodMonitor or ServiceMonitor) will be more useful for narrowing down the list.

/hold
As discussed on Slack, this is unfortunately more complicated because the monitors data is loaded asynchronously. It makes several requests to get the monitors data, which and can take a while. Allowing filtering by data that has not yet loaded likely leads to confusing UX 🙁

@kyoto can you think of any other middle ground here? I hope my commit message (and personal experience with this page) show how the filtering UX is almost not useful right now :|

/close

Created Jira ticket so that we don't forget this feature request: https://issues.redhat.com/browse/OBSDA-531


Matching detected virus with cell barcode

Hi,
Is there a way of matching a detected virus with the barcode of the cell from which it originated?
Thanks and good day.

@juychen


Contracts/core audit1.0

code clean for audit.

Codecov Report

Merging #193 into contracts/DX-2.0-No-Frontend will decrease coverage by 0.2%.
The diff coverage is 100%.

@@                       Coverage Diff                        @@
##           contracts/DX-2.0-No-Frontend     #193      +/-   ##
================================================================
- Coverage                           100%   99.79%   -0.21%
================================================================
  Files                                 6        6
  Lines                               472      496      +24
  Branches                             82       90       +8
================================================================
+ Hits                                472      495      +23
- Misses                                0        1       +1

Impacted Files
Coverage Δ

contracts/ForTestingOnly/InternalTests.sol
100% <ø> (ø)

contracts/DutchExchange.sol
99.76% <100%> (-0.24%)
:arrow_down:

contracts/Tokens/TokenTUL.sol
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

We will work with smartContractAudit


Reinstall products are deleted

When you deinstall the package and reinstall it again the products are deleted. I am not sure if this is intended. But would it not be better that the products and other settings are not removed from the database?

There's nothing we've changed about the uninstall. So it seems like C5 has changed that. I don't know what is the most acceptable solution and what peoples expectations are for an uninstall. That said, I feel like most people would expect a clean slate after an uninstall. So maybe it's not bad.
Rerunning the upgrade essentially runs much of the install script anyways.


Add Alameda, CA please

Thank you! 🖤

Hi @knick247365,
To add your city, please provide email addressed and an email that we can add. Check out the guidelines here and submit an new request: email guidelines
Thanks!


Add ability to run on provided data, rather than on mlSearch directly

It would make this library more versatile if it could take data instead of an mlSearch service. It allows finding own ways to compose data for display. It would also give more control when the directive reaches out to MarkLogic, preventing access attempts before login happened for instance.
Ideally it would take (an array of) facets, values or tuples data, showing all in one chart..

I second that.

It would also help making sure the directive won't try to make back-end calls too eagerly (e.g. before login happened)..


[Bug]: Modules docs don't render properly in Ansible Galaxy web

Expected Behavior
The generated dostrings should be in proper format for Ansible Galaxy. The Galaxy website should parse the DOCUMENTATION etc markdown and render nice html.
Basically, it seems that with the currecnt version of Galaxy web, the format changed and ansible-specdoc-generated docs are somehow wrong.
Thanks for your work on this useful project!
Actual Behavior
Only "Synopsis" is rendered.
https://galaxy.ansible.com/ui/repo/published/linode/cloud/content/module/instance/
Steps to Reproduce
No response
Error Output
No response

Thanks for reporting this, I've created a critical ticket internally and we'll tag this issue in the PR.

@jriddle-linode
I looked at it a bit and in our case the problem was rendered RETURN - it was quoted a lot, quite messy and maybe wasn't valid YAML.
https://github.com/equinix-labs/ansible-collection-equinix/blob/main/plugins/modules/metal_project.py#L76
ansible-docs tests the DOCUMENTATION and RETURN by loading it as an Ansible yaml:
https://github.com/ansible/ansible/blob/devel/lib/ansible/parsing/plugin_docs.py#L156
We had quite a mess rendered there, so I'm guessing ansible-doc was silently failing and skipping the galaxy docs render. I am not sure why your module documentation isn't rendered. I noticed you don't have DOCUMENTATION etc even injected in commited code in ansible_linode.
I think this bug is a bit invalid, but it would be a nice touch if ansible-specdoc validated that injected DOCUMENTAION, EXAMPLES and RETURN are valid yaml.


Fix failing ci error related to sparse_names containing features that are not part of the model's schema

The new loader is returning all features specified in the dataset.schema. T4Rec was built on a previous convention where the data loader was loading only features specified in the schema passed to the Trainer (that matches the schema used to construct the model).
A quick fix was implemented in a previous PR that filters out features from dataset.schema before calling the dataloader. However, sparse_names argument needs also to be filtered out to keep only the features of the schema used to build the model.

Note: I was able to catch another issue related to the types of features in the dataset stored in the drive (and used by ci test). In fact, the data contained a mix of float64 and float32 input features which breaks the input block of T4Rec as it expects all continuous features to have same type (to aggregate them into one vector).  So I updated the data by casting all columns to float32 and load it back to the drive.

I suspect some version of this error might be why saved TF models from Merlin Models are reporting all features returned by the dataloader as inputs. Seems like the dataloaders broke a lot of stuff this release cycle. 😅


uid must be an unsigned int

pnpm is awesome.. but I'm having issue on installing.. please help.
Issue:
binding.chown(pathModule._makeLong(path), uid, gid, req);
^
TypeError: uid must be an unsigned int
at TypeError (native)
at fs.chown (fs.js:1082:11)
at /usr/local/lib/node_modules/pnpm.js/node_modules/tar-fs/index.js:215:7
at FSReqWrap.oncomplete (fs.js:83:15)

How'd you get that? Can I see your package.json?
On Wednesday, February 3, 2016, Francis Torda<EMAIL_ADDRESS>wrote:

pnpm is awesome but I'm having issue on installing please help
Issue:
bindingchown(pathModule_makeLong(path), uid, gid, req);
^
TypeError: uid must be an unsigned int
at TypeError (native)
at fschown (fsjs:1082:11)
at /usr/local/lib/node_modules/pnpmjs/node_modules/tar-fs/indexjs:215:7
at FSReqWraponcomplete (fsjs:83:15)
—
Reply to this email directly or view it on GitHub
https://github.com/rstacruz/pnpm/issues/83.

And what's your os and npm/node versions?
$ npm version
On Wednesday, February 3, 2016, Rico Notifications <
<EMAIL_ADDRESS>wrote:

How'd you get that? Can I see your package.json?
On Wednesday, February 3, 2016, Francis Torda<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

pnpm is awesome but I'm having issue on installing please help
Issue:
bindingchown(pathModule_makeLong(path), uid, gid, req);
^
TypeError: uid must be an unsigned int
at TypeError (native)
at fschown (fsjs:1082:11)
at /usr/local/lib/node_modules/pnpmjs/node_modules/tar-fs/indexjs:215:7
at FSReqWraponcomplete (fsjs:83:15)
—
Reply to this email directly or view it on GitHub
https://github.com/rstacruz/pnpm/issues/83.

Turns out this was an issue with the tar-stream package required by pnpm's tar-fs dependency. tar-stream didn't support parsing large uids and gids, instead returning NaN values.
I've submitted a pull request https://github.com/mafintosh/tar-stream/pull/47 and this issue should be fixed once it's merged and published to npm.
This command now succeeds when you force npm to use my patched version of tar-stream:
docker run --rm node:latest /bin/bash -c "npm install -g brotchie/tar-stream pnpm; pnpm install<EMAIL_ADDRESS>

The change in tar-stream has been merged and and a release has been made, which has automatically propagated to pnpm, so I believe this issue can be closed.

Nice work @brotchie!

Good job James..
On 18 February 2016 at 01:29, Rico Sta. Cruz<EMAIL_ADDRESS>wrote:

Nice work @brotchie https://github.com/brotchie!
—
Reply to this email directly or view it on GitHub
https://github.com/rstacruz/pnpm/issues/83#issuecomment-185314200.


bumping universe release for marathon 1.5.0

Unlike previous minor version updates, this release will be tough to compare.  The https://github.com/mesosphere/marathon-stuff/tree/master/universe-creator tool was used to create the latest version of files.
Tool updates: https://github.com/mesosphere/marathon-stuff/pull/20/files
include:

flag change from max_tasks_per_offer => max_instances_per_offer
dcos version updated to 1.10

The version from generation was changed from snapshot to v1.5.0 for universe package version and for the marathon docker version.

Does this not conflict with #1403?

depends who lands first :) .  I thought that landed already... we will have to update this!  thanks!

@triclambert, it does. Please merge #1403, and I will update this one afterwards.

as the original creator I can't approve... however I approve!  the changes Ivan made!


Continent/Region/Island column and sort

Is it possible to add continent column to table and sort it according to it? To make it easier going trough merchants. I have 7 chars and i just put one on each continent to make it faster to go trough merchants on each one.

Compact mode for merchant grid (settings) specifies the continent. And it's a lot easier to follow. Hovering over locations when submitting or clicking on those posted pop-ups the map.


1dffa7c: "Tried to handle a committed transaction in the pool but its nonce doesn't match the head of the queue for that sender.

(Failure\n
\"Tried to handle a committed transaction in the pool but its nonce doesn't match the head of the queue for that sender.\")\n
(\"Raised at file \\\"stdlib.ml\\\", line 33, characters 17-33\"\n
\"Called from file \\\"src/lib/network_pool/transaction_pool.ml\\\", line 257, characters 14-107\"\n
\"Called from file \\\"list.ml\\\", line 117, characters 24-34\"\n
\"Called from file \\\"src/list0.ml\\\" (inlined), line 22, characters 40-81\"\n
\"Called from file \\\"src/lib/network_pool/transaction_pool.ml\\\", line 229, characters 8-1023\"\n
\"Called from file \\\"src/lib/pipe_lib/broadcast_pipe.ml\\\", line 89, characters 25-28\"\n
\"Called from file \\\"src/pipe.ml\\\", line 827, characters 4-56\"\n
\"Called from file \\\"src/job_queue.ml\\\" (inlined), line 131, characters 2-5\"\n
\"Called from file \\\"src/job_queue.ml\\\", line 171, characters 6-47\"\n
\"Caught by monitor coda\"))"

coda_crash_report_2020-01-25_13-49-28.401870.tar.gz

Dup of #4298. Closing.


Use purescript-simple-dom

The API now exposes simple-dom's HTMLElement and the examples now use
appendToBody and setInterval.
Fixes #67

:+1: Looks nice.

Ack, sorry, I thought this would be fine, but I just broke your PR. It should be a simple merge though.


Test Debug - stat.c

per #150.
Note: This test requires "std=c99" to compile. This is not a critical issue but a fix might be preferable for future ease of use.
This test currently does not fail, but it wrongly returns "size: 0". (Result from natively running the test: "size: 16392")
Full verbosity (-vvvv) output can be found in:
stat_output _verbose.txt

Looking into this.
Rust-gdb output:
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/usr/lib/libthread_db.so.1".
[2321,4151445504:06:55:30.068740] BYPASSING ALL ACL CHECKS
[New Thread 0x7ffff771f640 (LWP 2322)]
[New Thread 0x7ffff770e640 (LWP 2323)]
[New Thread 0x7ffff76fd640 (LWP 2324)]
[Switching to Thread 0x7ffff76fd640 (LWP 2324)]

Thread 4 "sel_ldr" hit Breakpoint 2, rustposix::safeposix::cage::Cage::stat_syscall (self=0x555555b39f50, path="/lind/stat.nexe", statbuf=0x7ffff76fc7d0) at src/safeposix/syscalls/fs_calls.rs:370
370             let truepath = normpath(convpath(path), self);
@ [#4:rustposix::safeposix::cage::Cage::stat_syscall()] b NaClSysStat
Breakpoint 3 at 0x555555577c0e: file src/trusted/service_runtime/nacl_syscall_common.c, line 1365.
@ [#4:rustposix::safeposix::cage::Cage::stat_syscall()] r
The program being debugged has been started already.
@Start it from the beginning? (y or n) y
Starting program: /home/lind/lind_project/lind/lindenv/bin/sel_ldr -a -- "/lib/glibc/runnable-ld.so" --library-path "/lib/glibc" /lind/stat.nexe
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/usr/lib/libthread_db.so.1".
[2325,4151445504:06:56:46.795631] BYPASSING ALL ACL CHECKS
[New Thread 0x7ffff771f640 (LWP 2326)]
[New Thread 0x7ffff770e640 (LWP 2327)]
[New Thread 0x7ffff76fd640 (LWP 2328)]
[Switching to Thread 0x7ffff76fd640 (LWP 2328)]

Thread 4 "sel_ldr" hit Breakpoint 3, NaClSysStat (natp=0x555555b36f90, pathname=0xfffeffef <error: Cannot access memory at address 0xfffeffef>, buf=0xfffefd50) at src/trusted/service_runtime/nacl_syscall_common.c:1365

^ It seems that the address is being sent wrong-or cannot be accessed for another reason.
Full output: Stat_RustGdbOut.txt

The issue was with rust having a missing variable, pad0. Adding it to the structure and setting it as 0 when initializing fixes it.
[1:0 0:29] 08:10:13 Sat Nov 13 [lind@08b6d3585784: +1] ~/lind_project
@(1:29) $ ./stat stat.nexe
running stat("stat.nexe")
size: 214579
20834/32040MB   0.00 0.00 0.00 1/575 1480       (0)
[1:0 0:29] 08:10:15 Sat Nov 13 [lind@08b6d3585784: +1] ~/lind_project
@(1:29) $ lind /lind/stat.nexe

executing: [sel_ldr -a -- "/lib/glibc/runnable-ld.so" --library-path "/lib/glibc" /lind/stat.nexe]
[1482,4075390976:08:10:22.365332] BYPASSING ALL ACL CHECKS
usage: /lind/stat.nexe <files>
running stat("/lind/stat.nexe")
size: 214579


MINOR: Add @cluster annotation to StreamsNamedRepartitionTopicTest

The StreamsNamedRepartitionTopicTest system tests did not have the @cluster annotation and was therefore taking up the entire cluster.  For example, we see this in the log output:
kafkatest.tests.streams.streams_named_repartition_topic_test.StreamsNamedRepartitionTopicTest.test_upgrade_topology_with_named_repartition_topic is using entire cluster. It's possible this test has no associated cluster metadata.
This PR adds the missing annotation.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Tested locally to confirm that the added size is the correct one.

@rondagostino can we run a branch builder for this PR as a sanity check?

Thanks, @bbejeck .  Test passed: https://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2021-05-17--001.1621281781--rondagostino--systest_add_cluster_annotation_streams_test--b20e203a0/report.html

This PR only touched a python system test file, failures are unrelated.

merged #10697 into trunk


[Github] Support of Webhook Content Type 'application/x-www-form-urlencoded'

Does this code support webhook content type 'application/x-www-form-urlencoded' for github?

@aashakabra no it only supports json

@aashakabra been thinking about this and it may not be that bad to add, want to change some things around before implementing and so kicking it to v6

Is it still valid? I hope to working on this


Not working as expected -or- I don't get it...

I tried the samples but either I am expecting something completely different or it just doesn't work.
Take the minimal API example: If I access an endpoint it just throws me a 401 but no auth redirect, no nothing. Shouldn't the middleware take care of that already?
The same is true for all other example projects. If I try to implement it myself I get the same result.

It is the default behavior for Web API, the library is built to support the Web API in mind, so if you want to develop some MVC application with redirects and everything you need to override the default and add additional authentication schemas.

Right.... I assumed that at some point. But then again when I run the most basic identity setup with keycloak the middleware takes care of redirects. When I try the examples (not the API one) I don't get redirected even though the controller setup suggests that something along those lines should happen.
I mean this is a great project and I learned a lot about keycloak specific things while browsing the source but I don't really get how to benefit from the effort you put in as a whole when using it as a package. Maybe if you could provide a small example on how to just integrate with keycloak and have the default identity behavior set up, that would be great since, again, I cannot see it working with the included examples as a custom implementation of a specific identity provider should behave.
The default and expected behavior should be that at any point when an endpoint of any sort is accessed, which is decorated with the [Authorize] attribute, the middleware should perform a redirect to the login page, when the user returns and does not have the correct permissions, a 401 should be returned.

Yeah, it is something that we could do once I have time. I have an example of how to setup Keycloak + SPA-like application  (Blazor WASM)
I guess you want to see an example of how to use and configure Keycloak + MVC

oh i know that problem too well... take your time ;)

Hi there,
I provide a free demo instance of keycloak: https://keycloak.nicedemo.de/
you can login with admin-admin.
It would be great if you could show us an example there. Best thanks for your'e great work.
If someone is interested, I created a nuxt 3 and vue 3 repo with keycloak authentification
https://github.com/Kingside88/nuxt3-primevue-starter-auth


Section for Single session operating system

Hello Team,
We have been notified that the document does not include information for Windows 10 Single Session OS when mentioning the steps:
Prerequisites
To use FSLogix search roaming functionality, you must be using one of the following versions of Windows:
Windows Server 2012 R2
Windows Server 2016
Windows 10 (prior to 1809)
Configure single user search roaming
Note
Applies to Windows 10 (prior to 1809) only.
Can we include this one as well.
Regards,
Rupali

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

ID: 120fcf61-9a2b-a111-9ebd-a8cc37082c34
Version Independent ID: 120fcf61-9a2b-a111-9ebd-a8cc37082c34
Content: Configure Windows Search database roaming - FSLogix
Content Source: fslogix-docs/how-to-configure-search-roaming.md
Product: fslogix
GitHub Login: @msft-jasonparker
Microsoft Alias: japarker

The article does NOT differentiate between single and multi-session Windows 10. The article is inclusive of both versions of Windows 10.

Hello Jason,
Thanks for your response.
Just to confirm, search is configured on Windows 10 Single session operating systems as well and no action is required from the setup perspective?
I found this difference when I setup FSLogix in multisession os and single session os - Windows 10 20H2:


Add Rough Sleeper by Burial

Song Name
Rough Sleeper
Artist Name
Burial
Source
Youtube
Youtube Link
https://www.youtube.com/watch?v=h1ZOSQ-aIXM
Direct File Link
No response
Song ID
720034
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 96919888
Downloaded: Yes

reject
replacing a song thats used a lot


Normalisation rule to filter out deleted records.

I have an OAI feed that includes deleted files. How can I filter out the deleted files and only import the files that I need?

Hi, please add an example of the OAI metadata so we can see what we are working with. Thank you.


Relationship between compression and Lee Cronin's "Assembly Theory"

I came across your Twitter post today and had some notes kicking around on my laptop that might be relevant background literature to your project...and perhaps a connection to a broader literature.
tl;dr "Assembly Theory" is just Lemepl-Ziv-Welchcomplexity / compression.

Cronin and co-workers have put forward a Molecular Assembly method for describing molecular complexity (Nat Comm 2020 10.1038/s41467-021-23258-x )
They define an "assembly pathway" arxiv:1907.04649 arXiv:1804.06972 as "sequences of joining operations, that start with basic building blocks (in this case bonds) and end with a final product. In these sequences, sub-units generated within the sequence can combine with other basic or compound sub-units later in the sequence, to recursively generate larger structures ." [Cronin Nat Comm 2020] and a "molecular assembly number" (MA) as the length of the shortest of these pathways.
This is equivalent to the Lempel-Ziv complexity measure Lempel Ziv '76], in which a source string is decomposed to a set of patterns which suffice to reconstruct the source string by copying and insertion operations.
The patterns (LZ vocabulary) are the "assembly pool" in the terminology of Cronin & co. https://dx.doi.org/10.26434/chemrxiv.13476597.v2
Cronin & co. are aware of the relationship to the Lempel-Ziv-Welch compression algorithm (it is alluded to in Ref.  https://arxiv.org/abs/1907.04649 as its relationship to the Kolmogorov complexity)
Cronin & co. (correctly) reject Kolmogorov complexity as being uncomputable.
Many string representations of molecules exist (e.g., SELFIES, SMILES, InCHI)—A challenge for SELFIES and SMILES is the lack of single intrinsic canonical order; in other words, there are multiple possible string representations, and in the computation of MA one is free to choose the one that leads to the shortest possible LZ complexity.

I have become involved in this work and completely missed this comment, sorry!
Thank you this is a very nice analogy to connect and motivate why Gzip indeed works well for chemical problems. It would also be super interesting to correlate the MA number versus the compression density we can achieve with Gzip.
Do you think one could also build an ML representation that only looks at the bonds identified with the MA algorithm and checks for redundancy - similar to the patterns of the LZ vocabulary? So essentially performing the compression manually, only permitting a vocabulary inspired by chemistry ("bonds" or higher order "fragments")?
Because would interesting to see how much the patterns identified by Gzip (which might not be "chemistry intuitive") add to the accuracy.

All good questions; no obvious answer is known to me....

Yes indeed there is a nice linear correlation between molecular assembly number and the size of the gzip compressed SMILES string :)  - even thoughts it look a little noisy the trend is there :)

(took the MA number data from https://www.nature.com/articles/s41467-021-23258-x#code-availability figure 2 )


Chore merge 9th of February

[x] https://github.com/twbs/bootstrap/commit/33ccbc23e383d9b5a9c53379791892cc73d72f21
~~https://github.com/twbs/bootstrap/commit/9bd7228ed6f3a92d679fbb4eee6f36d6d9d26984~~: nothing to do since we don't have any Dropdowns example
~~https://github.com/twbs/bootstrap/commit/bae3260188a6d09db42362145f77a60213692193~~: nothing to do here since socket.io-parser deps has already been bumped by Dependabot
[x] https://github.com/twbs/bootstrap/commit/04edf521c6e9a373f838f608762733993d980a6d
[x] https://github.com/twbs/bootstrap/commit/8ba98bc9f09d0abd69044a20f5505f485096ae6e
[x] https://github.com/twbs/bootstrap/commit/fa37da4d44d23e324b6df7aaaa1fb66fca225ab4
[x] https://github.com/twbs/bootstrap/commit/d891159950f695c69761551715fee5182a008d33
[x] https://github.com/twbs/bootstrap/commit/b613cfb58bf4b3c2acc8f46f661f1814744620da

⚠ I chose to use https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/blob/main/site/static/docs/5.3/assets/img/guides/parcel-dev-server-boosted.png, https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/blob/main/site/static/docs/5.3/assets/img/guides/vite-dev-server-boosted.png and https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/blob/main/site/static/docs/5.3/assets/img/guides/webpack-dev-server-boosted.png as thumbnails because we don't have the equivalent of the nice Bootstrap images on top of the Parcel, Vite and Webpack guides

~~https://github.com/twbs/bootstrap/commit/492187de6468efe8ac774ff25f00522181364af1~~: nothing to do since we don't have any Cheatsheet example
~~https://github.com/twbs/bootstrap/commit/21956ee6f45a0590711c7b08cdf05ffd008dbb9b~~: nothing to do since we don't have such image in our homepage
[x] https://github.com/twbs/bootstrap/commit/98ebf3a33612d270e7b4e03ca17566334f531db1
[x] https://github.com/twbs/bootstrap/commit/6295d3ba36045252729772b90a60d857dfa13aa9
[x] https://github.com/twbs/bootstrap/commit/39c51aa3eb50cc0b74895cf36038b658bd579d18
[x] https://github.com/twbs/bootstrap/commit/eb4819474de3d1dd3bc1a3d6df052fc9c35a9b2b
[x] https://github.com/twbs/bootstrap/commit/993ad97fdc0bcbc0fb63dcafb466f81e5be004e6
[x] https://github.com/twbs/bootstrap/commit/f9827ac8d83415a12a8611b1bef238c334aa422e
[x] https://github.com/twbs/bootstrap/commit/5d9c09ccbaff4d1c89398eec3b077da412dec956
[x] https://github.com/twbs/bootstrap/commit/a99234d528b12bd080096041ff0612672782cdd3
[x] https://github.com/twbs/bootstrap/commit/752be8657d934d2b44716ca2bb5bfff9193ca45f
[x] https://github.com/twbs/bootstrap/commit/c4155c4f15e8c8c01cf32d82392507bc3d025a10
[x] https://github.com/twbs/bootstrap/commit/6b9ed938f118e9ad0dd28ff3722f80cc1a844235
~~https://github.com/twbs/bootstrap/commit/406ab2abe66415f51dbe40079ec8c042854567c7~~: nothing to do since score card is removed right after
~~https://github.com/twbs/bootstrap/commit/97576345b16751a3f4f54afa5aaa581cff323c89~~: nothing to do since we didn't include score cards in previous commits

New batch of commits ready for your review @louismaximepiton 🙏


[Safari] Unexpected keyword 'const'. Const declarations are not supported in strict mode.

Do you want to request a feature or report a bug?
bug
What is the current behavior?
In Safari 9 it outputs the error
Please mention other relevant information such as the browser version, Node.js version, Operating System and programming language.
This is the line at which the error is thrown:

const parse  = __WEBPACK_IMPORTED_MODULE_1__custom_es6__["parse"].bind({ metadata: __WEBPACK_IMPORTED_MODULE_0__metadata_min___default.a })
/* harmony export (immutable) */ exports["parse"] = parse;

Is it ok for Webpack to use const in the output files?
I'm using ["es2015", { modules: false }]

The relevant compiled file (an npm dependency package) is:

import metadata from './metadata.min'

import
{
	parse as _parse,
	format as _format,
	is_valid_number as _is_valid_number,
	as_you_type as _as_you_type
}
from './custom.es6'

export const parse  = _parse.bind({ metadata })
export const format = _format.bind({ metadata })

export const is_valid_number = _is_valid_number.bind({ metadata })
export const isValidNumber   = is_valid_number

export const as_you_type = _as_you_type(metadata)
export const asYouType   = as_you_type

Ok, I get it now: this index.es6.js file isn't transpiled with Babel, and so the library author should just add 'use strict' at the top.
Webpack 2 only handles imports, it doesn't do any transpilation.

https://github.com/webpack/webpack-dev-server#caveats

Version 2.8.0 introduced a change which included ES6 keywords const and let
Those wishing to support oldIE should stick with version 2.7.1

Makes sense to rely on something with ES6 syntax when having transpiled to to ES5 before. WTF @webpack 👎


Invalid JSON v5 if versionType is blank

When adding Versions, if a lessThan (or similar comparison) is included then a versionType is required by the schema, but vulnogram doesn't warn on this and you end up with non-validating json:
i.e.
      "affected": [
        {
          "versions": [
            {
              "status": "affected",
              "version": "3.0.0",
              "lessThan": "3.0.3"
            }
          ],
          "defaultStatus": "unaffected"
        }

fixed, it nows show errors for those cases.


consume multiple versions for crd in galley

Changes in Galley to support multiple version Crd
Does not change the metadata for now as the galley/tools/gen-meta might change

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ozevren
If they are not already assigned, you can assign the PR to them by writing /assign @ozevren in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS

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

@ozevren

Do we have any decision about where to place all our galley config?  Since the galley/tools/gen-meta will be Deprecated so I have not changed the meta files galley/pkg/metadata/kube/types.go

I have tested this change in Istio with default mcp enabled for all other components. It works well to consume multiple versions crds like

:
`
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
name: virtualservices.networking.istio.io
labels:
app: istio-pilot
chart: istio
heritage: Tiller
release: istio
annotations:
"helm.sh/resource-policy": keep
spec:
group: networking.istio.io
names:
kind: VirtualService
listKind: VirtualServiceList
plural: virtualservices
singular: virtualservice
shortNames:
- vs
categories:
- istio-io
- networking-istio-io
scope: Namespaced
versions:

name: v1alpha3
served: true
storage: true

name: v1alpha2
served: true
storage: false
`

The question here is do we want to make galley Must for all other component starting from 1.2? If this is the case, I think we are safe now to support multiple versions crd. If not, I will work on new PRs to make sure pilot/mixer/citade can work well with multiple versions crd.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
6a7fbc90ba6571854993d21dfa2a15f7d0885dd7
link
/test istio-pilot-multicluster-e2e

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

Maybe I am missing something, but how is this different than simply having the version field?
I think we need the following to happen in Galley:

For Istio resources, when starting up listeners, we should do a discovery and find the highest version available that the current Galley instance supports. Then, we should serve that version out of Galley, potentially, through a single collection in MCP, but later, under multiple collection names, based on version. This latter part can be implemented later, but will need the first part.
For K8s resources, there is a version range that Istio supports. For the K8s resources we're listening to, we should listen to the highest version of the resource that is supported by the lowest version of K8s. That should be good enough and avoid having to deal with multiple different versions of K8s resources.

I want to remove the metadata file and codegen eventually, but should be keep it in good shape and not try to work around it.  I think we should add the Versions field and remove the Version field from the metadata.
For 1.2, I don't think we can make Galley required. That should be 1.3. So we also need to make sure the current CRD solution (at least for Istio resources) keep working. That should be simple though, since we can still keep the older versions of CRDs around and make components keep reading that.

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
6a7fbc90ba6571854993d21dfa2a15f7d0885dd7
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
7be842eb0f153963811c2d31e92a49430071f063
link
/test istio-unit-tests

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

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
7be842eb0f153963811c2d31e92a49430071f063
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
7be842eb0f153963811c2d31e92a49430071f063
link
/test istio-pilot-multicluster-e2e

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
4bb5a061a74c4ae4cf715310b895d83ef90fbe11
link
/test istio-pilot-multicluster-e2e

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

@ozevren  I want to use a new way to solve the problem. so let me close this temporarily.


Cannot load some indicators in newer versions

I am seeing issues with a number of the candlestick patterns in newer versions with error

Cannot read property 'hasPattern' of undefined

For example:
let doji = require('technicalindicators').Doji
let values = ...
doji.hasPattern(values)

The above will throw the error and I am seeing this across a number of patterns, not just Doji. The last version available on npm that seems stable is 0.2.17.

Ok with the 0.2.20 version we migrated things to typescript and there are few breaking changes, technically it should be a major bump, since not a lot of people are using I didn't major bump it. If you are using the latest version in npm, your example should be like
let doji = require('technicalindicators').doji
let values = ...
doji(values)

We need to update this in runkit.
I also pushed a Playground with code completion just now, which has code completion which you can used to test instead of the runkit.


Add config for release-integration team

This PR adds a config file for the Release Integration team.
Thanks,
Dave

Thanks! Be sure to invite @interrupt to your channel(s) so it’s ready to hear mentions.

Thanks @dpb587!


Add files via upload

Youtube Video Downloader Without Any Ads Free

@shivamsdk Create issue first..then do PR


Improve activities performance

Especially from app launch till Activities tab is available.
Primarily by only ever, loading a subset of activities. 1000 for now, as well as transactions in the same period.

@James-Sangalli there's this issue #2140. Otherwise, I don't mind.


Difference between RenderingStrategies

There are currently two RenderingStrategies, namely MYBATIS3 and SPRING_NAMED_PARAMETER. What are the differences between them? How/when should they be used?

The library renders SQL and parameters for use both by MyBatis3 and by Spring's Named Parameter JDBC template. I think of those two things as "SQL execution engines". The library includes utilities to make it easy to use the generated SQL with either of those engines.
You should use the rendering strategy that matches the SQL execution engine you are using. There are lots of examples of using the library with both underlying tools. You can see pointers to the different example here: https://github.com/mybatis/mybatis-dynamic-sql


(maint) Eagerly evaluate OS in analytics client

Previously, we spun evaluation of the client OS into another thread to
avoid slowing down the main execution thread by needing to require
facter. This caused a race condition in facter between the thread
compting the OS and the main thread loading PAL which also needs to
evaluate some facts. As it turns out, by the time the analytics client
is instantiated we've already required facter anyway, so we don't have
to pay that cost. Evaluating the OS fact is negligible (~25ms), so we
now do it in the main thread to avoid the race condition.

CLA signed by all contributors.

Can you add a CHANGELOG commit?


Make jumpscalex tests run green

See logs in the chatbot:  https://t.me/jumpscaleX_tests

Failed Tests:

[ ]  j.data.bcdb.test()
[ ] j.clients.sshagent.test()
[ ] Sal.ubuntu test_apt_install_version
[ ] Markdown test_include_from_other_repo
[ ] Markdown test_include_custom_link

Builders

[ ] Openresty
[ ] Digitalme
[ ] Python
[ ] Bitcion
[ ] Etcd
[ ] Capnp
[ ] Coredns
[ ] Syncthing

Schema

[ ] Ipport
[ ] Iprange
[ ] Url
[ ] Numeric
[ ] Guid
[ ] Hash
[ ] Multiline
[ ] Yaml
[ ] Enum
[ ] Binary
[ ] Unique attributes

commit: 9c455233a8e70154adc41bac92e934fd3b00799f
Fix Sal.ubuntu test_apt_install_version
Fix schema unique attributes

problem with openresty builder https://github.com/threefoldtech/jumpscaleX/issues/456

Commit: f0152cb25eb7070ad387b7ada07c508fd0b310dc
Fix capnp builder and python builder
Commit: a59f27d6a5c898d33fc62405737dab4065e2e00f & f0152cb25eb7070ad387b7ada07c508fd0b310dc
Fix bitcion builder

Commit: https://github.com/threefoldtech/jumpscaleX/commit/2adab7c0d71a6068e696d23120cff04af5de103f
fix openresty problem https://github.com/threefoldtech/jumpscaleX/issues/451

schema tests #460

@BolaNasr
He fixed bcdb test commit: 790e0297c37813b72bb6677e0e15834927b38f85
@Dina-Abd-Elrahman
She fixed syncthing builder (stop method) commit: 1ae45f66cd4aebc0997d716394cba84e4c29db87
@abom
He fixed etcd builder (start method) commit: 315c65806ca8702e3437623d5e51234268082dfe

Now we have all tests passed except schema and ssh tests
Result

@AhmedHanafy725  now schema tests Green

@rkhamis @ramezsaeed
Now we have all tests passed except sshkey.test()
But builder tests sometimes fail because:

it couldn't pull a repo.
timeout (time limit = 10 min per execution).

Another thing builder tests take at least (in case of one single push) about one hour and if there are more pushes it takes about 1-2 hours.
So I suggest not to run builder tests with every push and run them once a day.

sshKey test fix : https://github.com/threefoldtech/jumpscaleX/commit/608e96c0cf5acf0f8f533cce3cda7f7b95072635

@AhmedHanafy725 build tests should only run nightly, I agree

Nightly builders test passed here


revised PR#80, added new submissions schema and seed data

Description
this PR creates a new table in the db to replace the current Submissions table
Fixes #80

What work was done?
columns deleted from previous submissions table, new columns added in their place
Why was this work done?
booleans were removed, replaced with timestamps, some info will be stored in (yet to be created) 'Pages' table
What feature / user story is it for?
https://trello.com/c/J7brkjtW/464-schema-changes-to-the-submissions-table
Any relevant links or images / screenshots

Type of change
Please delete options that are not relevant.

[X] New feature (non-breaking change which adds functionality)
[X] This change requires a documentation update

Change Status

[X] Complete, but not tested (may need new tests)

Has This Been Tested

[X] No

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] My code has been reviewed by at least one peer
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[X] There are no merge conflicts

Great work! The comments explain the code fairly well and I can understand what is going on step by step.


Switch to ESM

Change the module from CommonJS to ECMAScript Module (ESM). Some JS Frameworks support ESM now and importing CommonJS modules may get complicated.

Just to clarify here... there's two ESM related things that are possible here:

Switch this package to use ESM so that it's package.json contains type: module
Update this package so that it doesn't assume all packages can be require-ed in since node will freak out if you require an ESM npm package

Which of these were you describing with this ticket?

I'm trying to buld Google cloud functions with webpack with enabled ESM. But external modules added with webpack-node-externals are commonjs by default and webpack compiles them as:
/***/ "pino":
/*!***********************!*\
  !*** external "pino" ***!
  \***********************/
/***/ ((module) => {

module.exports = require("pino");

/***/ }),

As you can see webpack is dumb and uses require even if it's configured to output ESM module.
What webpack-node-externals options should I set to tell webpack that all externals should be imported as ES modules?

Used webpack-node-externals option importType: 'module' to pack my project with ESM support.
More details are here: https://github.com/nrwl/nx/issues/7872#issuecomment-997460397

importType: 'module' should be added to @types.

We'd like to use some CJS dependencies (e.g. lodash) and some ESM-only ones (like d3). Now my issue is that neither of the importType options work for us. When using the default importType: 'commonjs' I'm getting
module.exports = require("d3");
^

ReferenceError: require is not defined in ES module scope, you can use import instead

and with importType: 'module':
console.log((0,lodash__WEBPACK_IMPORTED_MODULE_2__.reverse)([(0,_util__WEBPACK_IMPORTED_MODULE_0__["default"])(), d3__WEBPACK_IMPORTED_MODULE_1__.scaleLinear().domain([3, 6])(4)]));
                                                           ^

TypeError: (0 , lodash__WEBPACK_IMPORTED_MODULE_2__.reverse) is not a function

Without webpack-node-externals I can run the bundle without issues. Any ideas what do do in this case except downgrading the ESM-only packages to older, unsupported versions with CJS support?

Webpack 5 supports ESM format for configurations

@cbix
I worked around the issue with this code:
	importType: function( request ) {
		if( request === '@errorx666/query' ) {
			return `module ${request}`;
		} else {
			return `node-commonjs ${request}`;
		}
	}

More properly, the code should probably check package.json for type: module.


Add database schema and entities for validation flow

Address https://github.com/NuGet/NuGetGallery/issues/4651.
The resulting SQL is:
CREATE TABLE [dbo].[PackageValidationSets] (
    [Key] [bigint] NOT NULL IDENTITY,
    [ValidationTrackingId] [uniqueidentifier] NOT NULL,
    [PackageKey] [int] NOT NULL,
    [RowVersion] rowversion NOT NULL,
    [Created] [datetime2](7) NOT NULL,
    [Updated] [datetime2](7) NOT NULL,
    CONSTRAINT [PK_dbo.PackageValidationSets] PRIMARY KEY ([Key])
)

CREATE UNIQUE INDEX [IX_ValidationTrackingId] ON [dbo].[PackageValidationSets]([ValidationTrackingId])

CREATE INDEX [IX_PackageKey] ON [dbo].[PackageValidationSets]([PackageKey])

CREATE TABLE [dbo].[PackageValidations] (
    [Key] [uniqueidentifier] NOT NULL DEFAULT newsequentialid(),
    [PackageValidationSetKey] [bigint] NOT NULL,
    [Type] [varchar](255) NOT NULL,
    [ValidationStatusKey] [int] NOT NULL,
    [ValidationStatusTimestamp] [datetime2](7) NOT NULL,
    CONSTRAINT [PK_dbo.PackageValidations] PRIMARY KEY ([Key])
)

CREATE INDEX [IX_PackageValidationSetKey] ON [dbo].[PackageValidations]([PackageValidationSetKey])

ALTER TABLE [dbo].[PackageValidationSets] ADD CONSTRAINT [FK_dbo.PackageValidationSets_dbo.Packages_PackageKey] FOREIGN KEY ([PackageKey]) REFERENCES [dbo].[Packages] ([Key]) ON DELETE CASCADE

ALTER TABLE [dbo].[PackageValidations] ADD CONSTRAINT [FK_dbo.PackageValidations_dbo.PackageValidationSets_PackageValidationSetKey] FOREIGN KEY ([PackageValidationSetKey]) REFERENCES [dbo].[PackageValidationSets] ([Key]) ON DELETE CASCADE

Closing for now. I think it is best to start in a separate DB.


[base-cluster] user management

We should be able to manage various user accounts via the helm chart
we need the following;

[x] list of user accounts
[x] (Cluster-)Roles?
[x] (Cluster-)Rolebindings?
[ ] Namespaces?

~We probably can't automate the creation of the tokens because of the "recent" changes in k8s v1.23~
We can just create the secret manually 🤷
~Therefore~ Still, we should print a command that generates the kubeconfig, including the creation of the token

@marvinWolff  do we need the unchecked items?

Hm, good question.
What Roles and Rolebindings were used in the old cluster?
But anyway, i think it's hard to make general assumptions about what roles are needed for a specific cluster.
What i think can be done is a role which limits the access of the created users to the created namespaces in the cluster?

Hm, good question.
What Roles and Rolebindings were used in the old cluster? But anyway, i think it's hard to make general assumptions about what roles are needed for a specific cluster.

This should allow for a general configuration solution not for a specific cluster nor a default set of roles.

What i think can be done is a role which limits the access of the created users to the created namespaces in the cluster?

What do you mean by that? 🤔
I was thinking of something like this;
roles:
  admin:
    get:
      v1:
        - services
clusterRoles:
  admin:
    list:
      v1:
        - services
users:
  test:
    roles:
      admin:
        - kube-system
    clusterRoles:
      - admin

Or maybe a field type on the role? 🤔

In the past we used a namespace "accounts" to store them.


Underscores anger the semver process

What steps did you take:
vendir tools sort-semver -v "v1.17.16_vmware.2-tkg.1  v1.17.16_vmware.2-tkg.2  v1.17.16_vmware.3-tkg.1  v1.18.16_vmware.1-tkg.1  v1.18.16_vmware.1-tkg.2  v1.18.16_vmware.3-tkg.1  v1.18.17_vmware.1-tkg.1  v1.18.17_vmware.2-tkg.1  v1.19.12_vmware.1-tkg.1  v1.19.12_vmware.1-tkg.2  v1.19.12_vmware.1-tkg.3-tf-v0.10.1  v1.19.16_vmware.1-tkg.1  v1.19.8_vmware.1-tkg.1  v1.19.8_vmware.1-tkg.2  v1.19.8_vmware.3-tkg.1  v1.19.9_vmware.1-tkg.1  v1.19.9_vmware.2-tkg.1  v1.20.11_vmware.1-tkg.1-tf-v0.10.1  v1.20.14_vmware.1-tkg.1  v1.20.14_vmware.1-tkg.2  v1.20.14_vmware.1-tkg.4  v1.20.14_vmware.1-tkg.4-tf-v0.11.2  v1.20.14_vmware.1-tkg.5  v1.20.4_vmware.1-tkg.1  v1.20.4_vmware.1-tkg.2  v1.20.4_vmware.3-tkg.1  v1.20.5_vmware.1-tkg.1  v1.20.5_vmware.2-fips.1-tkg.1  v1.20.5_vmware.2-tkg.1  v1.20.8_vmware.1-fips.1-tkg.2  v1.20.8_vmware.1-tkg.2  v1.20.8_vmware.1-tkg.3  v1.21.2_vmware.1-fips.1-tkg.1  v1.21.2_vmware.1-tkg.1  v1.21.2_vmware.1-tkg.2  v1.21.5_vmware.1-tkg.1-tf-v0.10.1  v1.21.8_vmware.1-tkg.1  v1.21.8_vmware.1-tkg.2  v1.21.8_vmware.1-tkg.4  v1.21.8_vmware.1-tkg.4-tf-v0.11.2  v1.21.8_vmware.1-tkg.5  v1.22.5_vmware.1-tkg.1  v1.22.5_vmware.1-tkg.3  v1.22.5_vmware.1-tkg.3-tf-v0.11.2  v1.22.5_vmware.1-tkg.4  "

Versions

Version

Succeeded

What happened:
Nothing is properly detected as semantic versioned.
What did you expect:
Replacing + with _ is a common change since container tags cannot have a + in them.  In fact most systems (like containerd) will actually replace the + into a _ for you.
vendir tools sort-semver -v "v1.17.16+vmware.2-tkg.1  v1.17.16+vmware.2-tkg.2  v1.17.16+vmware.3-tkg.1  v1.18.16+vmware.1-tkg.1  v1.18.16+vmware.1-tkg.2  v1.18.16+vmware.3-tkg.1  v1.18.17+vmware.1-tkg.1  v1.18.17+vmware.2-tkg.1  v1.19.12+vmware.1-tkg.1  v1.19.12+vmware.1-tkg.2  v1.19.12+vmware.1-tkg.3-tf-v0.10.1  v1.19.16+vmware.1-tkg.1  v1.19.8+vmware.1-tkg.1  v1.19.8+vmware.1-tkg.2  v1.19.8+vmware.3-tkg.1  v1.19.9+vmware.1-tkg.1  v1.19.9+vmware.2-tkg.1  v1.20.11+vmware.1-tkg.1-tf-v0.10.1  v1.20.14+vmware.1-tkg.1  v1.20.14+vmware.1-tkg.2  v1.20.14+vmware.1-tkg.4  v1.20.14+vmware.1-tkg.4-tf-v0.11.2  v1.20.14+vmware.1-tkg.5  v1.20.4+vmware.1-tkg.1  v1.20.4+vmware.1-tkg.2  v1.20.4+vmware.3-tkg.1  v1.20.5+vmware.1-tkg.1  v1.20.5+vmware.2-fips.1-tkg.1  v1.20.5+vmware.2-tkg.1  v1.20.8+vmware.1-fips.1-tkg.2  v1.20.8+vmware.1-tkg.2  v1.20.8+vmware.1-tkg.3  v1.21.2+vmware.1-fips.1-tkg.1  v1.21.2+vmware.1-tkg.1  v1.21.2+vmware.1-tkg.2  v1.21.5+vmware.1-tkg.1-tf-v0.10.1  v1.21.8+vmware.1-tkg.1  v1.21.8+vmware.1-tkg.2  v1.21.8+vmware.1-tkg.4  v1.21.8+vmware.1-tkg.4-tf-v0.11.2  v1.21.8+vmware.1-tkg.5  v1.22.5+vmware.1-tkg.1  v1.22.5+vmware.1-tkg.3  v1.22.5+vmware.1-tkg.3-tf-v0.11.2  v1.22.5+vmware.1-tkg.4
"

Versions

Version
v1.17.16+vmware.2-tkg.1
v1.17.16+vmware.2-tkg.2
v1.17.16+vmware.3-tkg.1
v1.18.16+vmware.1-tkg.1
v1.18.16+vmware.1-tkg.2
v1.18.16+vmware.3-tkg.1
v1.18.17+vmware.1-tkg.1
v1.18.17+vmware.2-tkg.1
v1.19.8+vmware.1-tkg.1
v1.19.8+vmware.1-tkg.2
v1.19.8+vmware.3-tkg.1
v1.19.9+vmware.1-tkg.1
v1.19.9+vmware.2-tkg.1
v1.19.12+vmware.1-tkg.1
v1.19.12+vmware.1-tkg.2
v1.19.12+vmware.1-tkg.3-tf-v0.10.1
v1.19.16+vmware.1-tkg.1
v1.20.4+vmware.1-tkg.1
v1.20.4+vmware.1-tkg.2
v1.20.4+vmware.3-tkg.1
v1.20.5+vmware.1-tkg.1
v1.20.5+vmware.2-fips.1-tkg.1
v1.20.5+vmware.2-tkg.1
v1.20.8+vmware.1-fips.1-tkg.2
v1.20.8+vmware.1-tkg.2
v1.20.8+vmware.1-tkg.3
v1.20.11+vmware.1-tkg.1-tf-v0.10.1
v1.20.14+vmware.1-tkg.1
v1.20.14+vmware.1-tkg.2
v1.20.14+vmware.1-tkg.4
v1.20.14+vmware.1-tkg.5
v1.20.14+vmware.1-tkg.4-tf-v0.11.2
v1.21.2+vmware.1-fips.1-tkg.1
v1.21.2+vmware.1-tkg.1
v1.21.2+vmware.1-tkg.2
v1.21.5+vmware.1-tkg.1-tf-v0.10.1
v1.21.8+vmware.1-tkg.1
v1.21.8+vmware.1-tkg.2
v1.21.8+vmware.1-tkg.4
v1.21.8+vmware.1-tkg.5
v1.21.8+vmware.1-tkg.4-tf-v0.11.2
v1.22.5+vmware.1-tkg.1
v1.22.5+vmware.1-tkg.3
v1.22.5+vmware.1-tkg.4
v1.22.5+vmware.1-tkg.3-tf-v0.11.2

Highest version: v1.22.5+vmware.1-tkg.3-tf-v0.11.2

Succeeded

Anything else you would like to add:
I don't want to solutionize here, since folks might intentionally have the _ there and it's not meant to turn into a +, just illustrating that in this isolated event treating the _ as a + would fix things.
Environment:

vendir version (execute vendir --version): 0.26.0
OS (e.g. from /etc/os-release): Fedora 35

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Thanks @voor  - it seems like the library we're using is enforcing the semver 2.0.0 BNF grammar which includes the + as you show above, and - (hyphens) (to designate prereleases) but not underscores.

I've updated the title to reflect what I actually need.  If I remove the _vmware.1 part from the versions then sorting can proceed:
vendir tools sort-semver -v "v1.19.12-tkg.1  v1.19.12-tkg.2  v1.19.12-tkg.3-tf-v0.10.1  v1.19.16-tkg.1  v1.20.11-tkg.1-tf-v0.10.1  v1.20.14-tkg.1  v1.20.14-tkg.2  v1.20.14-tkg.4  v1.20.14-tkg.4-tf-v0.11.2  v1.20.14-tkg.5  v1.20.15-tkg.1  v1.20.8-tkg.2  v1.20.8-tkg.3  v1.21.11-tkg.1  v1.21.2-tkg.1  v1.21.2-tkg.2  v1.21.5-tkg.1-tf-v0.10.1  v1.21.8-tkg.1  v1.21.8-tkg.2  v1.21.8-tkg.4  v1.21.8-tkg.4-tf-v0.11.2  v1.21.8-tkg.5" --prerelease

I need some sort of find/replace capability on the versions prior to attempting sorting.  Whether that's changing _ to +, or in my case removing a chunk of erroneous text from the versions.


[Event Request] Table 83 "Item Journal Line" - OnSetUpNewLineOnBeforeLastItemjnlLinePostingDate

Hi, in the Table 83 "Item Journal Line", in the procedure SetUpNewLine, we would like to add our code at a specific place. To do so, we would need an event publisher and a IsHandled flag, like this :

With the event integration :

Do you think it is possible ?
Thank for your work.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


false change item id exception

Provide a general summary of the issue here
When rendering / not rendering item in collection based on state flag, rac throws an exception claiming item id changed. Code is slight variant on the rac menu sample:
      <MenuTrigger>
        <Button aria-label="Menu">☰</Button>
        <Popover>
          <Menu onAction={() => setUser((p) => !p)}>
            {user ? (

                <Item id="account">Account</Item>
                <Item id="signout">Sign out</Item>

            ) : (

                <Item id="signin">Sign in</Item>

            )}
          </Menu>
        </Popover>
      </MenuTrigger>

When select item, flips state and swaps sections rendered, resulting in:
Error: Cannot change the id of an item
    at $7135fc7d473fd974$export$dc064fe9e59310fd.setProps (http://localhost:3000/build/routes/_index-4NWDFMCL.js:7009:15)
    at http://localhost:3000/build/routes/_index-4NWDFMCL.js:7369:63
    at commitAttachRef (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:17271:28)
    at commitLayoutEffectOnFiber (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:17159:17)
    at commitLayoutMountEffects_complete (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:17976:17)
    at commitLayoutEffects_begin (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:17965:15)
    at commitLayoutEffects (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:17916:11)
    at commitRootImpl (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:19349:13)
    at commitRoot (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:19273:13)
    at performSyncWorkOnRoot (http://localhost:3000/build/_shared/chunk-GIAAE3CH.js:18891:11)

🤔 Expected Behavior?
Able to conditionally render node in menu component tree based on state
😯 Current Behavior
Unble to conditionally render node in menu component tree based on state
💁 Possible Solution
No response
🔦 Context
No response
🖥 Steps to Reproduce
Repo available at following repro:
https://github.com/rolanday/react-aria-change-id

Clone
Install node modules
npm run dev
Open menu, click item, observe result in devtools

Version
"react-aria-components": "^1.0.0-beta.0",
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
macOS
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

I've reproduced it locally, will have to dig further into what exactly the RAC collection node processing is doing here but it seem item order specific. Conditionally rendering something like this:
          {user ? (
            <Item id="account">Account</Item>
          ) : (
            <>
              <Item id="account">Account</Item>
              <Item id="signin">Sign in</Item>
            </>
          )}

works but
          {user ? (
            <Item id="account">Account</Item>
          ) : (
            <>
              <Item id="signin">Sign in</Item>
              <Item id="account">Account</Item>
            </>
          )}

doesn't. @devongovett any insights as to what is going on here?
That being said, rendering your Sections/Items dynamically seems to work just fine:
  let items1 = [
    {
      id: 'a',
      children: [
        {id: 'account', name: 'Account'},
        {id: 'signout', name: 'Sign out'}
      ]
    }
  ];

  let items2 = [
    {
      id: 'b',
      children: [
        {id: 'signin', name: 'Sign in'}
      ]
    }
  ];

  let [user, setUser] = React.useState(true);
  return (
    <MenuTrigger>
      <Button aria-label="Menu">☰</Button>
      <Popover>
        <Menu onAction={() => setUser((p) => !p)} items={user ? items1 : items2}>
          {section => (

              {item => <Item>{item.name}</Item>}

          )}
        </Menu>
      </Popover>
    </MenuTrigger>
  );

This is a bit of a gotcha with React itself. React's diff algorithm will reuse DOM nodes by default, so when the Sign in item is clicked, the DOM node is reused for the Account item. That's why it looks like the id is changing from React Aria's perspective.
We might be able to make some improvements, but it's a bit complicated. In the meantime, you can solve this by adding the key prop to your items in addition to id. React will see that the keys are different and will create a new item instead of reusing. This is actually more semantically correct about what's happening anyway: the Account and Sign in items are actually distinct, not the same item being updated. Here's an example: https://codesandbox.io/s/strange-bush-mqksm4?file=/src/App.js

Thank you both for looking into this an replying. I had a hunch node was being reused but didn't know if expected behavior or not. This (conditionally rendering item in a menu tree) is a pretty familiar pattern for me and first I've run into similar issue for statically added items (including not seeing same on hooks menu implementation previously used). It's simple enough to add a key. Thank you again!


fix utf16 in bookmarks

Fix for https://github.com/jung-kurt/gofpdf/issues/307
Tested on this code:
	pdf := gofpdf.New("P", "mm", "A5", "")
	pdf.AddUTF8Font("HAN NOM A", "", "hn.ttf")
	pdf.SetFont("HAN NOM A", "", 16)
	pdf.AddPage()
	pdf.Bookmark("1段落", 1, -1)
	pdf.Cell(0, 6, "2段落")
	pdf.Ln(50)
	pdf.AddPage()
	fileStr := example.Filename("Fpdf_Bookmark")
	err := pdf.OutputFileAndClose(fileStr)
	example.Summary(err, fileStr)
	// Output:
	// Successfully generated pdf/Fpdf_Bookmark.pdf

Result PDF:

I was unable to find ttf version of the font in original issue so used different one, but I think it won't be a problem.

Many thanks, @SuddenGunter!


Stale Branches

Basic info:
Apologize in advance as did not know where to place this. Upon updating a repo on an old laptop that had an outdated master I received the following branch updates. What is our strategy for dealing with this because the number of branches is much larger than the last time the machine pulled within the same time frame. Feel free to close immediately. But in NYC our motto is "If you see something...say something."
Please advise...
Solution:
A poor-person's solution would be just to encourage branch deletion from the PR authors at minimum. Encouragement to reviewers at medium. Stale branch deletion procedure at maximum.

Upon further review I was working on updating a web-platorm-test and got the same result list. IIRC there is a git submodule resourced to WPT within JSDOM. If this is the case then perhaps this is an issue for WPT (if even an issue at all).

Yes, this is for WPT.


There is a problem with Chinese garbled code

There is a problem with Chinese garbled code

Thanks for bringing it up. I always wanted to check this out, since I was somewhat sure it will not work due to the way the drawing is done. Hopefully, I find some time to revisit the Go text drawing or an alternative.

I've solved it. Just change the font.

so far,I use a not-so-trival way to catpture terminal with Chinese / emoji:
use https://github.com/tsl0922/ttyd select unicode font, and take a browser screenshot for that page.


Multi-page export to PDF does not resize larger diagrams to fit one page

Single page export (using crop) works.
By default, if technically possible, please make the multi-page export save each diagram page as a 'cropped' diagram to resize them to fit on their PDF page.
business-plan-Business Model Canvas.pdf
business-plan.pdf
Diagram link

As a work around, you can change the page size to fit all contents before exportinng

Notices this as well, but it's only happening on the desktop version. Exporting it via browser works fine.


[WIP]: Add markdown in README.rst

This PR fixes #576

Codecov Report

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

@@           Coverage Diff           @@
##           master     #578   +/-   ##
=======================================
  Coverage   95.12%   95.12%
=======================================
  Files         302      302
  Lines        2503     2503
=======================================
  Hits         2381     2381
  Misses        122      122

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 f4b493f...29f3f0e. Read the comment docs.


Highlight.JS Inline Code Syntax Highlightning

Highlight.js is a great highlightning tool. But now i want to know if it is also possible to make inline highlighting, so not like a block (like at http://tekkkz.com/posts/articles/arch_installation.html) but inside a text paragraph?
If it is possible, how?

The answer, for version 11 and up.
If using version 11 or newer of highlight.js you can use the following to also highlight <code> elements.
hljs.config({ cssSelector: 'code' });;
hljs.highlightAll();

Otherwise, check our README (this can be done with just a few lines of code) or skim below for possibly more specific examples.

Yes, you can use highlight.js not only with <pre><code> blocks, but with any other, see README#custom-initialization.

Problem is:
.hljs {
    display: block;
}

The code won't be displayed as inline.

Just pointing out, the ideal solution is to have pre > code.hljs on the library for display: block; and padding rules, but I solved this in my case with two lines, though I will expand the JS in here to make it clearer.
Intialize like this (Without jQuery):
for (i in document.getElementsByTagName("code")) {
  console.log(document.getElementsByTagName("code")[i]);
};

Or like this (With jQuery):
$('pre code').each(function(i, block) {
  hljs.highlightBlock(block);
});

And add this CSS: p code.hljs, li code.hljs { display: inline; padding: .2em }
The padding change is necessary for Solarized Dark, with .5em it overlaps for more convoluted examples, at least on my blog, and feels utterly ugly.

Ping from the year 2021. This solution above is still valid. Add the CSS as described.
pre code.hljs {
  display: block;
}

code.hljs {
  display: inline;
}

This thread is still one of the top google search results for those trying to solve this for WordPress
here's what fixed it for me:
// alternate init script to enable inline <code> tag highlighting
// written on version 2.7 of Prismatic (10.5.0 of highlight.js)
// it may break in future versions if highlight.js's API changes
window.addEventListener('DOMContentLoaded', event => {
  event.target.querySelectorAll('code')
  .forEach(hljs.highlightBlock);
  event.target.getElementById('prismatic-highlight-css')
  .sheet.insertRule('code.hljs:not(pre code) { padding: initial; display: initial; }');
}, false);

FYI: hljs.highlightBlock
This API is already deprecated (and will be removed in the future) in favor of highlightElement

The following worked for me with version 11.5.1:
hljs.configure({ cssSelector: 'code' });;
hljs.highlightAll();


update quality match

example:

Test.S02E04.2160p.WEB-DL.DDP5.1.DoVi.H.265-xyz
Test.S02E04.2160p.WEB-DL.DDP5.1.HDR.H.265-xyz
Test.S02E04.2160p.WEB-DL.DDP5.1.Atmos.DoVi.HDR.H.265-xyz

Please update quality match for:
"2160p webrip|webdl hdr|hdrplus"
to match number 2 and 3
currently this quality matches only number 2
"HDR" text is present in 3 so it should match that "hdr|hdrplus"

The issue is that 3 is getting detected as dolbyvision, which is considered a separate higher quality than plain hdr. You could change hdr|hdrplus to hdr|hdrplus|dolbyvision or hdr+ or >=hdr to allow matching it.

"hdr|hdrplus|hdr+"  will match only 1
(my TV does not support dovi in mkv, so that's why i need  2 or 3 and to avoid 1)

Hmm. I guess I don't know what the tags mean. When it has 'dovi' and 'hdr' in the title, does that mean it has 2 separate audio streams? The issue is that our parser only picks the highest matching quality, it isn't built to support files that have multiple streams contained.

For reference, we already had this discussion on #4042 but they failed to mention it on the OP

Ahh. Yeah, our parser is not compatible with multiple streams is the answer. And it isn't an easy thing to change. (It also isn't an easy thing to know based on the title. Some files might have 'dovi.hdr' and just be dovi, since dovi is a type of hdr.)

You could use the regex plugin instead to do a more naive match. Just reject everything that doesn't have "hdr" in the title, and use quality plugin to do the other bits.


Add TravelApp-KMP to KMP samples

Hi,
Wanted to add TravelApp-KMP to the samples. You can find that live here: https://seabdulbasit.github.io/TravelApp-KMP/
It is a simple UI showcase project.
Here is the repo link: https://github.com/SEAbdulbasit/TravelApp-KMP

Hi @SEAbdulbasit, thanks for the PR. We try to keep the number of examples for particular areas somewhat limited. I believe your music app with the points mentioned in #59 would be a great fit, but I think the Travel App is a bit too barebones for our sample gallery, and another app of yours, the Recipe App (that is already part of this list) has a lot of the same areas already covered very well. Thanks again for contributing that one! 🎉
So I hope you'll understand that I'll close this PR for now :)


Support name table format 1

It apparently allows encoding arbitrary language tags now.
/cc @brawer

Do we have any test fonts?
I couldn't find any on Mac or Debian. (But I've plenty of fonts with 'ltag' tables, which is almost the same, but of course slightly different... sigh).
Here's an ad-hoc script for finding fonts with name table format 1. If you've got some Windows box, can you run it there, please, and send me any matching fonts? Thanks in advance.
https://github.com/brawer/fonttools/tree/find_fonts_with_name_format_1

I'm guessing that this should be Windows-10 only.

Using your script, I didn't find any on Windows either.  I'll ask on OpenType mailing list.

And, has anyone found any sample fonts?

Nope.  Ignore this for now.


HTTPS traffic over a proxy is sent unencrypted

Hi,
There's a problem when trying to transfer https traffic over an http proxy. The current implementation does not send the CONNECT message to the proxy that would establish a tunnel for the end-to-end https connection. Instead the data is initiated with a GET message and the rest of the data is sent unencrypted between the client and the proxy.
There's already a pull request that would fix this issue by using an agent, which would take care of the proxy connection (https://github.com/EventSource/eventsource/pull/143). I decided to create an issue in case the pull request was accidentally overlooked.
Of course, if the server used http then the agent should be HttpProxyAgent instead. Alternatively, the old proxy functionality could be kept for the http server but the documentation should point out that the that it shouldn't be used for the https traffic.
diff --git a/lib/eventsource.js b/lib/eventsource.js
index 69c5525..76e9621 100644
--- a/lib/eventsource.js
+++ b/lib/eventsource.js
@@ -112,7 +112,10 @@ function EventSource (url, eventSourceInitDict) {
     // If specify http proxy, make the request to sent to the proxy server,
     // and include the original url in path and Host headers
     var useProxy = eventSourceInitDict && eventSourceInitDict.proxy
-    if (useProxy) {
+    var agent = eventSourceInitDict && eventSourceInitDict.agent
+    if (agent) {
+      options.agent = agent
+    } else if (useProxy) {
       var proxy = parse(eventSourceInitDict.proxy)
       isSecure = proxy.protocol === 'https:'

Closing this as the new implementation in v3 offloads all proxy/https options to the user/fetch implementation.


Update bug report issue templates

This PR is similar to the one created here

This PR introduces GitHub's new style of issue forms. (source)
Using these templates will lay the foundation for better reports from the community and allow the development team to focus on what they do best!

@GenericNerd I created a seperate commit for the Browser part of the issue template as I'm not sure if this is really relevant for an API bug.
If you agree and want to remove this from the issue template, just revert this commit before merging

Legendary

Legendary

Cheers for the merge!
Confirming you saw my comment above about the Browser section?


fix several issues (partially close #42)

RouterCallImpl: fix decoding responce from storage_ref
RouterCallImpl: fix decoding responce from storage_map
BucketDiscovery: check res for nil
BucketStat: decode bsInfo by ptr
Add tnt tests for disvoery logic
Add tnt tests for RouterCallImpl
Add tnt tests for RouterMapCallRWImpl

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 6.66667% with 28 lines in your changes missing coverage. Please review.

Project coverage is 32.24%. Comparing base (7938584) to head (8284b21).

Files with missing lines
Patch %
Lines

api.go
3.84%
25 Missing :warning:

discovery.go
33.33%
2 Missing :warning:

replicaset.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
- Coverage   32.39%   32.24%   -0.15%
==========================================
  Files          10       10
  Lines         673      676       +3
==========================================
  Hits          218      218
- Misses        439      442       +3
  Partials       16       16

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


Remove version: from docker-compose files

version: is no longer a "thing" in docker-compose, so our version: "3.6" just gets in the way.

Remove from all ddev
Remove from ddev-contrib
Do this on a major version upgrade so it doesn't mess up too many projects.

We'll try this in a year or so. Right now, too many people are using docker-compose versions that don't support it.