Updated linux build link

Updated to redirect to docs page, as current link is a 404.
PR Summary

PR Checklist

[ ] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Change is not breaking
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

File 'docs/installation/linux.md' was removed in a093a47 which broke the link.

File 'docs/installation/linux.md' was moved to Installing-PowerShell-Core-on-Linux.md in the PowerShell/PowerShell-Docs repo.


Question / no issue

Hello
I took over the handling of an existing site created by picocms. Looks great so far but I am not familar with this yet. But I am a bit under pressure to clearify some things and therefore I choose this way. As I didn't find a forum I will ask you.
I can see that the site creates a session (PHPSESSID) cookie. I need to know what is it for but have no clue where to start my investigation.
Would you please be so kind to assist me with this or you let me know whom to contact for some assistance with this?
Thank you in advance
Kind regards
//neph

Hello
Thank you so much. Looks like it came from the contact plugin. It helped a lot. :-)
There is another question. Google offers a JS based Maps API which is explained here. But it is not working in an MD file just by copying and pasting it into the content page. Do you see a way in Pico to get this code implemented and working well?
Kind regards
//neph

This should work just fine, there's likely rather something wrong with what you've copied. Anyway, if you just want a include a map, use Google Map's "Share" feature instead (https://support.google.com/maps/answer/144361, resulting in a simple ). If you really want to use the JavaScript API, you should rather create a appropiate Twig template and use appropiate meta variables to change the map's behavior.

I have already a map embeded as advised. Unfortunately it is cookie based. But I want to get rid of all cookies and adding a JS based map will solve this.
I have added this to the page:

And this is what I get displayed:

In case I made something wrong, I dont find the error.
Kind regards
//neph

You musn't split HTML tags over multiple lines.


Look for PyPI tokens in keyring when uploading

The username for keyring should look like pypi_token:project:<project_name> (using the normalised name), or pypi_token:user:<username>. If neither are found, it falls back to looking for a password (but if you have 2FA enabled on PyPI, uploads have to use tokens now).
I'd like to make some more tooling for conveniently creating tokens, but there isn't a PyPI API for it yet.
While I'm looking at this code, I've also simplified things a bit so that FLIT_USERNAME and FLIT_PASSWORD environment variables override details in .pypirc even if you use --repository, and it figures out the repository details before building the packages.

I'm going to try this branch the next time I have a package to upload, before I merge it.

I forgot about this for a long while, and then for a while longer I didn't upload any packages through Flit. Now I just have again, and putting a token in my keyring with username pypi_token:project:pep517 worked. :-)


Fix PeeringConnection:

When we have a vpc that belongs to some other account, we can't lookup it.

Any chance on getting this merged in ?


Checksum verifications

Some checksum were never verified

From commit message:

Thus return one's complement already from the right
place instead of

And this right place is ... (please add to commit message).


GUI is not showing

Description
After clicking on tray icon and opening Settings or View data, empty System Bridge window appears with errors in backend console. Image and logs attached.
system-bridge-gui (settings).log
system-bridge (settings).log
system-bridge-console (settings).log

.
What Platform / OS are you running?
Windows
What version are you running?
3.2.1
Anything in the logs or a references that might be useful?
Besides the main issue I can see that there is also errors related to database and SQL transactions in backend console log.

Additional information
No response

Hi @timmo001 , I'm afraid I don't really see much improvement.
First I removed data folder to start fresh. (c:\Users\Mitja\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\Local\timmo001)
Still some SQL errors on startup and the GUI is still blank. I clicked on "show settings" on 14:53:14 timestamp. Everything before that was simply me running python command and watching the output (no input from my side).
system-bridge.logs.zip

I also have this problem.  Additionally, I get an error of:

The code execution cannot proceed because Qt6WebEngineCore.dIl was not found. Reinstalling the program may fix this problem.

pyside6 is installed without problem, confirmed by running python -m pip install --upgrade pyside6
System Bridge 3.4.1 is installed.
I can open the GUI log, grab the URL, and open it in a browser though.

FYI, same occurs with 3.4.4

I had an issue with a laptop that had pyside6 installed but not fully. I ended up running:
python -m pip install --force --upgrade PySide6

which fixed the issue of PySide7 not being found

I figured out my problem.

Don't use Python from the Windows Store - it is a bad idea and not recommended by the Python devs.
Install Python directly from the Python website, and ensure to check the checkbox which says "Add Python [VER] to PATH" - this is critical for Python to work correctly from the CLI and for third-party dependent libraries to load, like Qt6WebEngineCore.dIl.  @timmo001 - I would recommend stressing this in your documentation (Windows Store does not setup PATH and does not give you an option to).

I also use Python from Windows Store. Solved missing Qt6WebEngineCore.dll by adding new OS environment variable
QTWEBENGINEPROCESS_PATH=%USERPROFILE%\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\PySide6\
But the GUI from within the app is still not shown. It shows the main window, it blinks once and then this is the logged error. I can still access the GUI from the browser but would be nice to have this working as well.
2022-08-17 13:26:36 INFO (MainThread) [systembridgegui.window.main.MainWindow] Open URL: PySide6.QtCore.QUrl('http://localhost:9170/app/settings.html?apiKey=d73c9a19-4069-43f2-add1-1e972839dc32&apiPort=9170')
qt.webenginecontext:

GL Type: desktop
Surface Type: OpenGL
Surface Profile: CompatibilityProfile
Surface Version: 4.3
QSG RHI Backend: OpenGL
Using Supported QSG Backend: yes
Using Software Dynamic GL: no
Using Multithreaded OpenGL: yes

Init Parameters:
  *  application-name python3
  *  browser-subprocess-path C:\Users\Mitja\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\PySide6\
  *  create-default-gl-context
  *  disable-es3-gl-context
  *  disable-features ConsolidatedMovementXY,InstalledApp,BackgroundFetch,WebOTP,WebPayments,WebUSB,PictureInPicture
  *  disable-speech-api
  *  enable-features NetworkServiceInProcess,TracingServiceInProcess
  *  enable-threaded-compositing
  *  in-process-gpu
  *  use-gl desktop

QOpenGLShader::compile(Fragment): ERROR: 4:1: 'textureSampler' : syntax error syntax error

*** Problematic Fragment shader source code ***
#version 110
#ifdef GL_KHR_blend_equation_advanced
#extension GL_ARB_fragment_coord_conventions : enable
#extension GL_KHR_blend_equation_advanced : enable
#endif
#define lowp
#define mediump
#define highp
#line 1
varying highp vec2 uv;uniform sampler2DRect textureSampler;uniform bool swizzle;uniform highp float opacity;void main() {   highp vec4 tmpFragColor = texture2DRect(textureSampler,uv);   tmpFragColor.a *= opacity;   gl_FragColor = swizzle ? tmpFragColor.bgra : tmpFragColor;}
***

This is Windows 10 headless server and I tried from RDP session as well as from TeamViewer (thinking that display driver is to blame).

I've done all of my testing via Remote Desktop.  As I mentioned in my previous posting, do not use Python from Windows Store - it's part of what caused my problems.
Here's some more reading on the weirdness of fake pathing that the Store does to its packages:
https://github.com/python/cpython/blob/main/Doc/using/windows.rst#the-microsoft-store-package


fix update zstack account password error

这个 PR 实现什么功能/修复什么问题:
修复更新zstack账号密码失败问题
是否需要 backport 到之前的 release 分支:

release/2.10.0

/cc @swordqiu @yousong

/approve
/lgtm

/hold

/hold cancel

/lgtm

Codecov Report

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

@@            Coverage Diff            @@
##           master   #1301      +/-   ##
=========================================
+ Coverage    7.83%   7.85%   +0.01%
=========================================
  Files         372     372
  Lines       58357   58357
=========================================
+ Hits         4573    4584      +11
+ Misses      53384   53374      -10
+ Partials      400     399       -1

Flag
Coverage Δ

#aFlag
7.85% <ø> (+0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
66.15% <0%> (ø)
:arrow_up:

pkg/util/rbacutils/rbac.go
74.76% <0%> (+3.42%)
: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 b08aa5d...90176d4. Read the comment docs.


Preorder traversal added

Have you read the Contributing Guidelines on Pull Requests?
(Yes)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(#178 )

Please review sir @HarshCasper @atarax665 @ankitaggarwal23 @gat786

Sir @iamrajiv please review

Please review @nidh-ish

All changes done sir @iamrajiv

All changes done @nidh-ish


Should a version number be associated with repository submissions?

Not sure if it necessary but worthwhile to think about. We have the situation where the minimal standards will likely undergo refinements, yet we should soon expect that studies will be submitted to repositories following MiAIRR. How to handle if incompatible changes are introduced? Should a spec version number be associated with repository submissions? This might also be important for APIs that repositories expose. How could this be implemented?
I can see how tools that perform automated submission could insure that a version number is attached, not sure how to do that for manual submissions.

Good point.  We could simply add a field that lists the AIRR version number. If done manually, I imagine the user would have some NCBI-related templates to fill out.  Since these are versioned, they can include their version string for the field.

Closing as moot since MiAIRR is now in maintenance mode


chore(spinner): missing spinner stories added

PR Checklist
Please check if your PR fulfills the following requirements:

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

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[x] Other... Please describe: Missing storybook file added

Which package are you modifying?

[ ] accordion
[ ] alert
[ ] alert-dialog
[ ] aspect-ratio
[ ] avatar
[ ] badge
[ ] button
[ ] calendar
[ ] card
[ ] checkbox
[ ] collapsible
[ ] combobox
[ ] command
[ ] context-menu
[ ] data-table
[ ] date-picker
[ ] dialog
[ ] dropdown-menu
[ ] hover-card
[ ] input
[ ] label
[ ] menubar
[ ] navigation-menu
[ ] popover
[ ] progress
[ ] radio-group
[ ] scroll-area
[ ] select
[ ] separator
[ ] sheet
[ ] skeleton
[ ] slider
[x] spinner
[ ] switch
[ ] table
[ ] tabs
[ ] textarea
[ ] toast
[ ] toggle
[ ] tooltip
[ ] typography

What is the current behavior?

spinner is missing in storybook but it is implemented

What is the new behavior?

spinner is present in storybook

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Thanks for adding this! Spinner is kind of an artifact from the beginning of the project. However, since people seem to use it I think we should keep it! What do you think?

Yes, I think we should keep it. People are using it, so it adds value!
I just was thinking, if we should also make it more public by adding a page for the spinner?

@elite-benni I think that's a good idea!


Make clear that applicants for iQTS are not eligible for student finance

Make clear that applicants for iQTS are not eligible for student finance
Trello card
https://trello.com/c/wodYlmcD/137-update-iqts-page-with-student-finance-info
Context
Applicant was wrongly awarded student finance which now needs to be repaid.
Changes proposed in this pull request
Clearer guidance applicants are not eligible.
Guidance to review

Ready for review when you get a sec @EllieNodder


Support AVRO events

What is your idea?
Support pattern matching on AVRO events. AVRO support is a reasonable next step because:

It has wide adoption in streaming systems like Pulsar and Kafka. Any streaming system could use not only the performance characteristics of ruler (which are much needed) but also the semantics of pattern matching.
Would serve as a good reference example/stepping stone implementation for other binary formats (or any events with formal schema for that matter).

Would you be willing to make the change?
Maybe
Additional context

Message/streaming systems are lacking a killer pattern/expression/filter language - it could use a de facto "ruler pattern" language, just like SQL is for DBs.

User who choose to define schemas for their events, expect all interactions to respect schema. Having pattern matching respect data types and fields as defined by the "active or a previous" schema seems like a natural fit for those use cases.

This is a great idea. The thing to watch out for is the performance of turning Avro messages into field name/value pairs, in the case of Jackson this is the most expensive part of the matching task.  Note the comments on https://www.tbray.org/ongoing/When/202x/2021/12/03/Filtering-Lessons where people say that it should be possible to do this very efficiently. Ideally it would be nice to have a Jackson-compatible tokenizer, which would allow for a lot of code re-use. Hey, check this out: https://github.com/FasterXML/jackson-dataformats-binary - this could be the basis for killing a lot of birds with one stone.

Thanks @timbray for the implementation hints! It is promising to see the possibility to add binary format support without compromising performance.

I like the idea of supporting jackson-dataformats-binary. Updated the title to highlight this. Code-change wise, I think this'll mean a new method in Event.java that creates a JsonParser [1] for the right format.  We then bubble up the interface from various places clients can use (GenericMatchine, Ruler). Then we need to also add tests cases + benchmarks.

Just need to be sure that this supports the nextToken() API, not just ObjectMapper.  Also need to watch out for schemas, most of these binary formats can't be parsed without accessing schemas. Will need to cache schemas where possible.  CBOR doesn't need a schema. Avro relies on a 4-byte field in the Kafka wire format header that identifies the schema. Which is to say, this feature is going to need some design thinking, even if the implementation isn't that hard.

https://github.com/FasterXML/jackson-dataformats-binary seems to be extending JSONParser, so it "should" be supporting nextToken() but definitely worth a second look. There's some tests in the pkg implementing, so hopeful.  I hadn't thought beyond this yet.
I find the idea of extending a Flattener interface from Quamina https://github.com/timbray/quamina/#flattening-and-matching. I'm hoping there's a similar interface we can have to allows for extension. This interface should be in addition for built-in support for Avro and other schemas.

Also 👋 Ayman, missed ya.

Just had a look at the Jackson Avro code. I was worried that it would implement nextToken() by deserializing into an object then traversing that, but it looks pretty efficient actually.  Also, it looks pretty complicated, wouldn't want to implement one of these from scratch.
Also, hey there Ayman.

Seems nextToken() has huge code reusability advantage. I was wondering though if it limits the ability to 1) consult the schema if the pattern field exist in the event and 2) look it up in constant time). The O(1) lookup part should be doable in protobuf - not sure about Avro. This thing maybe beneficial when Pattern fields are << Event fields OR just premature optimization  ¯_(ツ)_/¯
fwiw I was actually doing research on event matching options for Pulsar - I stumbled upon few options like JMS selectors, JSTL, and even fully fledged SQL "WHERE" conditions! At the time, I wanted to also experiment with Ruler as it would beat other options performance-wise at least (but the Java version was not OSS. And, the non-aws community is very schema-full). SOOO, big thank you everyone behind this initiative!
Also, 👋 Rishi, 👋 Tim! Hope all is well!


[Terra Form Validation] Background color of the select (with required error) is not matched with the input backround

Bug Report
Description

After submitting a form, the select bacgkround color should match with the input one. (white on default theme)
Steps to Reproduce

Open the following link: https://engineering.cerner.com/terra-ui/components/terra-form-validation/form-validation/validation-events
Set the background to #F7ECFA
Press the Submit button
The input field background is white.
The select background color is transparent (not white).

Additional Context / Screenshots

Expected Behavior

The form validated selects background should be the same as the input ones.
Possible Solution

Environment

Component Name and Version: terra-form-validation 1.69.6
Browser Name and Version: Chrome newest
Node/npm Version: [e.g. Node 8/npm 5] -
Webpack Version: -
Operating System and version (desktop or mobile): windows 10

@ Mentions

UXPLATFORM-6598 has been created for this.

up.

@KunigundaLauly  We have an internal Jira logged for this so even if this github issue gets closed for being stale we will still include this request within our list of projects to prioritize for future work.

@chrismichalewicz thank you! Will we get somewhere notified when this ticket is resolved & published?


examples/ollama_functions_example: improve system message for better response

For llama3, the example seemed to work but for some other models like qwen2 and friends, it was not working properly. I faced issues where getCurrentWeather tool worked but not the finalResponse tool. With these changes, I can get the desired behavior.
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[ ] Describes the source of new concepts.
[ ] References existing implementations as appropriate.
[ ] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

@corani Can you PTAL?

friendly ping

Looks fine to me, but it's up to @tmc to decide 😉


Added function to edit existing pizzas, for fun

Click on the header on a pizza in the sidebar to open it again in the configure dialog.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: jchristman75 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Update JAVA_VERSION to JDK_VERSION for compiling job

Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

Fix test build jobs like:
https://ci.adoptopenjdk.net/view/Test_openjdk/job/openjdk11_j9_openjdktest_x86-64_linux/28/console


export iOS zip file corrupt

OSX 10.9.4
Getting an error when trying to unzip this file:
http://www.godotengine.org/builds/release/GodotiOSXCode.zip
"Error 2 - No such file or directory"

I can't reproduce with the 1.1 export template, so I'll assume this issue is no longer valid.


Polybar loads wrong font

I know there is a Wiki page and also dozens of issues regarding font loading, but after screening all of that I believe my case was not covered yet.
I want to use Font Awesome 5 icons, so in my config I have:
font-0 = FantasqueSansMono:pixelsize=10:bold:antialias=true;2
font-1 = "Font Awesome 5 Free:pixelsize=10:style=Regular;2"

The font is installed and can be matched with this name:
$ fc-list | grep "Font Awesome 5 Free"
/home/sergey/.fonts/nix-fonts/opentype/Font Awesome 5 Free-Solid-900.otf: Font Awesome 5 Free,Font Awesome 5 Free Solid:style=Solid,Book
/home/sergey/.fonts/nix-fonts/opentype/Font Awesome 5 Free-Regular-400.otf: Font Awesome 5 Free,Font Awesome 5 Free Regular:style=Regular
$ fc-match "Font Awesome 5 Free"
Font Awesome 5 Free-Regular-400.otf: "Font Awesome 5 Free" "Regular"

However, when I launch Polybar with this config and -l info, I get the following output:
* Loaded font "FantasqueSansMono:pixelsize=10:bold:antialias=true" (name=Fantasque Sans Mono, offset=2, file=/home/sergey/.fonts/nix-fonts/opentype/FantasqueSansMono-Bold.otf)
* Loaded font "Font Awesome 5 Free:pixelsize=10:style=Regular" (name=DejaVu Sans, offset=2, file=/usr/share/fonts/truetype/dejavu/DejaVuSans.ttf)

For some reason, Polybar loads DejaVu instead, but believes it loaded the correct one. (Here is the relevant part of the output with FC_DEBUG=1.)
I tried different variations of the font name (with and withou quotes, with and without style=Regular part), but none worked in Polybar, though each was possible to fc-match.
Any suggestions?

Interesting case. What does fc-match "Font Awesome 5 Free:pixelsize=10:style=Regular" give you?
Can you also try font-1 = Font Awesome 5 Free Regular:style=Regular

It matches correctly:
$ fc-match "Font Awesome 5 Free:pixelsize=10:style=Regular"
Font Awesome 5 Free-Regular-400.otf: "Font Awesome 5 Free" "Regular"

I tried to use your proposed variant, but it does not change anything, still loads DejaVu instead. In fact, whatever random string I put there, it reports:
* Loaded font "foobar" (name=DejaVu Sans, offset=0, file=/usr/share/fonts/truetype/dejavu/DejaVuSans.ttf)

Hm, so it looks like polybar and fc-match don't use the exact same way to find fonts. I think the problem is that your fonts are in your home directory. For some reason fc-list also looks in your ~/.fonts directory. According to the Arch Wiki ~/.fonts/ is deprecated, so maybe that's it. What distro are you using?
Also what is the output of xset q?

I'm on Ubuntu, but extensively use Nix package manager to get up-to-date apps and libraries. My fonts are also managed by Nix, though it puts installed otf files in some obscure location, so I created $HOME/.fonts and made a symlink inside.
Thanks for your help!

still strange that it picked up your first font but not font awesome, even though they are in the same folder.

It happened to me too. After installing a new Font 'Source_Code_Pro_Yahei-Hybrid.ttf', polybar loads it though I did not configure to use.Then poly bar performs strange...
I tried to switch font-0 and font-1 then polybar pickup the right font now.

@Yves0 Have you confirmed with fc-match that all of your font patterns are recognized?


code cleanup based on PVS-Studio findings

Overview
Quick fixes based on PVS studio findings

Checklist

[ ] Change is covered by automated tests.
[ ] JavaDoc is provided for all new/changed public APIs.
[ ] User Guide is updated.
[x] CI builds pass.

@inponomarev What is PVS-Studio?

@packpaul PVS-Studio is a proprietary static analysis tool. I'm testing it while preparing my conference talk about devops view on static analysis.


motif search documentation

It's not too far off self-documenting, but:

add motif search doc page linked to main documentation page
update link in FAQs

Can I use BLAST to find S. pombe sequences similar to my query sequence?
Can I use BLAST to search for short sequences?

... maybe wait for #1225 outcome?

... maybe wait for #1225 outcome?
yeah good plan...

maybe a glossary of symbols if it exists? Apparently they come in slightly different flavours for logical defs(?)
I'm pretty sure someone in the workshop tried to use curly brackets to mean something which worked in fungiDB but not PomBase.

We use the standard symbols, so I think it's fine. It's clear what the syntax  for our search should be from the examples. Curly brackets are used for number ranges.

...which is broken, so that would be a bug, needs reporting?

maybe a glossary of symbols if it exists? Apparently they come in slightly different flavours for logical defs(?)

I don't fully understand this? What is needed that isn't on the current front page?

...which is broken, so that would be a bug, needs reporting?

What's broken?

...which is broken, so that would be a bug, needs reporting?
What's broken?

Ah, got it.  Curly brackets don't seem to work.  I'll fix that.

Thanks for the documentation.  I've added a link to it from the motif search page.


feat: adjust video quality

feat: adjust video quality
Description
we don't need to create a new MediaSource when the user switches video quality.
we remove the old buffer and append new video data When the user switches video quality.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[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
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #65 into master will decrease coverage by 0.69%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master      #65     +/-   ##
=========================================
- Coverage   52.08%   51.39%   -0.7%
=========================================
  Files         123      123
  Lines        1703     1726     +23
=========================================
  Hits          887      887
- Misses        816      839     +23

Impacted Files
Coverage Δ

packages/griffith-mp4/src/player.js
0% <0%> (ø)
:arrow_up:

packages/griffith-mp4/src/mse/controller.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Bugfix/resolve path

#41

Pull Request Test Coverage Report for Build 156

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

Totals

Change from base Build 151:
0.0%

Covered Lines:
22

Relevant Lines:
22

💛  - Coveralls

Thank You :+1: .


Remove trailing slash from SPDX license exception URL

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

The URL for the SPDX license exception webpage is currently https://spdx.org/licenses/exceptions-index.html/:
https://github.com/Homebrew/brew/blob/39f07841080c3396bb560237a3e208ebef2779f7/Library/Homebrew/dev-cmd/audit.rb#L391
Because of the trailing slash, the SPDX website attempts to load—
https://spdx.org/licenses/exceptions-index.html/index.html
—which results in a 404 error. This pull request removes the trailing slash.

Good catch, @nwhetsell! Looks good to me. CI is failing but it looks like it's unrelated. I've triggered the CI to run again and we can see if there's still an issue.

Thanks, @nwhetsell!


@jupyterlab/katex-extension. Add settings section.

Add settings section to katex-extension.
For now, it contains just one entry, macros, a collection of custom  LaTeX macros.
Cheers

You are perfectly right @ian-r-rose! Now, it is way better.

Added! :-)

Thanks @AlbertHilb!

Published as<EMAIL_ADDRESS>


[Feature Request]: Calling an already setup Graph via the API

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Is your feature request related to a problem?
Can not calling Graph via the API right now.

Describe the feature you'd like
Needs to calling the already setup Graph via API #1102
Describe implementation you've considered
No response
Documentation, adoption, use case
No response
Additional information
No response

I have the same requirement! Is it solved yet?

Solved in #1888


 For some reason if your sliding menu code isn't visible (hidden by a div on top) it doesn't load. Any help appreciated! :)
(your code starts at "Artist" as I'm using your demo html in my code running your plugin):

@dirtandrust Hi. Can you show me the code?

Hi sure! I've posted the code here: http://dirtandrust.com/wp-content/uploads/super_menu/step4.html
Steps to reproduce:

Go to 320 viewport width
Click menu icon
First tab has your plugin
Click first item, then open the browser viewport to maybe 600px (don't go past 767px or you'll break the test)
I see your code is working but it's just not showing where i want it

Maybe the issue is conflicting css? Not sure. Any help appreciated.
Nathaniel

I think the only issue that needs to be solved in my case is for your plugin to be responsive - currently it only fires once to detect size but should fire on any window change.

Ah, I feel you. Will get on it soon.

@dirtandrust I've finally updated the plugin. Please check if your issue has been resolved now. Also, help me test the plugin for any bugs, if you're still using it.


Rename tests to a convention

We need to select a test naming convention. Should be set by the testing framework. Behave & whatever we select.

@canwaf do we want to get rid of this issue now that we have #93?

Duplicate of #93.


performance analysis of MonoVertex

When running the below spec
apiVersion: numaflow.numaproj.io/v1alpha1
kind: MonoVertex
metadata:
  name: simple-mono-vertex
spec:
  source:
    udsource:
      container:
        image: quay.io/numaio/numaflow-java/source-simple-source:stable
    # transformer is an optional container to do any transformation to the incoming data before passing to the sink
    transformer:
      container:
        image: quay.io/numaio/numaflow-rs/source-transformer-now:stable
  sink:
    udsink:
      container:
        image: quay.io/numaio/numaflow-java/simple-sink:stable

We see the below latencies; we need to understand what is causing this.
2024-08-13T18:16:11.445617Z  INFO Forwarded 14500 messages at time 2024-08-13 18:16:11.445616763 UTC
2024-08-13T18:16:11.447598Z  INFO Read batch size: 500 and latency - 1ms
2024-08-13T18:16:11.460127Z  INFO Transformer latency - 12ms
2024-08-13T18:16:11.477020Z  INFO Sink latency - 16ms
2024-08-13T18:16:11.478153Z  INFO Ack latency - 1ms
2024-08-13T18:16:11.479804Z  INFO Read batch size: 500 and latency - 1ms
2024-08-13T18:16:11.493092Z  INFO Transformer latency - 13ms
2024-08-13T18:16:11.510400Z  INFO Sink latency - 17ms
2024-08-13T18:16:11.511496Z  INFO Ack latency - 1ms
2024-08-13T18:16:11.513326Z  INFO Read batch size: 500 and latency - 1ms
2024-08-13T18:16:11.527290Z  INFO Transformer latency - 13ms
2024-08-13T18:16:11.544914Z  INFO Sink latency - 17ms
2024-08-13T18:16:11.546031Z  INFO Ack latency - 1ms
2024-08-13T18:16:11.548117Z  INFO Read batch size: 500 and latency - 2ms
2024-08-13T18:16:11.562704Z  INFO Transformer latency - 14ms
2024-08-13T18:16:11.580568Z  INFO Sink latency - 17ms

we are able to get > 200K TPS from a single mvtx pod.


fix issue with running Java 8

:loudspeaker: Type of change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring

:scroll: Description

After increasing the setup-java to v4, it needs distribution as a mandatory input parameter
:green_heart: How did you test it?
Ran the pipeline in my local and it is able to download the version and proceed further
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated reference documentation to reflect the change
[x] All tests passing
[x] No breaking changes

@rajadilipkolli apologies for late reply. We should go with 17. But we also have to make a decision either we remove using SDKMan here, or we use version provided by SDKMan. I don't really have a strong opinion. @MatejNedic do you have any preference or good reason to use SDKMan in Github actions, considering all Java versions we need are available out of the box?


[Datasheets] Fix the UUID in order to have compatibility between Datasheets, Platform Configurator and Adaptive Questionnaire

@vcutrona

As agreed during the last technical meeting with the coordinator, I focused on the user on-boarding part of the flow, which is the most critical for the next demo.

Testing setup
OS Ubuntu 22.04.3 LTS
Browser Firefox 117.0.1 (Mozilla Firefox Snap for Ubuntu)
Adaptive Questionnaire (AQ) version: 1.6.6
Platform Configurator (PC) version: 0.5.0
Issues to report

Completing the AQ survey takes too long (~10 secs to move to the next question). However, the server seems to be slow in general (it takes quite a while to load other applications, e.g., Datasheets and Keycloak -- maybe @karikolehmainen can check this point)
As per the discussion we had right before the additional review, the AQ final page should be improved to explain:

the meaning of the plot (e.g., labels on axes are missing, the semantics of values in [0,1] is not clarified)
the next steps for the users (e.g., 'By clicking the "find a kit for your need" button, the SME-oriented Platform Configurator will automatically build a kit featuring the tools that fit your needs the best. Then, you will be automatically redirected to RAMP, where you can start the negotiation phase with the Kit Integrator and further refine the recommended kit.')

When clicking "find a kit for your need", the user is redirected to RAMP (in my test, to https://ramp.eu/#/service-request/create/a6be2340-a440-4ada-b719-8f5ac5d98401), which however shows a blank page.

By checking the console log, I can find an error with code 400 (Bad request) and the message JSON parse error: Cannot deserialize value of type ``java.util.UUID`` from String "88f87e4602d33f92c67ed078f1b7e0d0": UUID has to be represented by standard 36-char representation.
The id 88f87e4602d33f92c67ed078f1b7e0d0 is not formatted as a UUID, but its value is read by PC directly from the tool datasheet database (field information.component_uuid). Suggestions for @karikolehmainen @RyanKelvinFord :

To update the datasheet database and fix tools IDs to match the UUID format
To implement a new field validation in the datasheet application so that the user is prevented from entering non-UUID ids in the field "Component UUID"

Additional context
For details about the request payload as sent by the PC to RAMP, check the 'KitSetDTO' payload at https://kitt4sme.collab-cloud.eu/platform-configurator/docs

Findings from the demo flow:

Try Adaptive questionnaire takes to RAMP login page, should be something else?
RAMP login/registration page Look-and-feel adapted to Kitt4SME?
AQ: Question numbers away cause they cause confusion
AQ is slow… more resources for VM?
AQ Internal server error!!

2023-10-02 07:30:26.357 INFO  ch.idsia.adaptive.backend.controller.SurveyController            Request status for token=e8738aa0-788e-465a-b87e-d9779634311c
2023-10-02 07:30:27.078 INFO  ch.idsia.adaptive.backend.controller.SurveyController            User with token=e8738aa0-788e-465a-b87e-d9779634311c request a new question
2023-10-02 07:30:27.412 ERROR org.apache.catalina.core.ContainerBase.[Tomcat].[localhost].[/aq].[dispatcherServlet] Servlet.service() for servlet [dispatcherServlet] in context with path [/aq] threw exception [Request processing failed; nested exception is java.lang.IllegalArgumentException: Cannot load status: no model for token=e8738aa0-788e-465a-b87e-d9779634311c] with root cause java.lang.IllegalArgumentException: Cannot load status: no model for token=e8738aa0-788e-465a-b87e-d9779634311.           	at ch.idsia.adaptive.backend.services.SurveyManagerService.lambda$getSurvey$1(SurveyManagerService.java:104)at java.base/java.util.Optional.orElseThrow(Optional.java:403

Platform configurator:

Error on POD logs:

2023-10-02T10:40:52.721006434+03:00
An error occurred while creating a tool from this JSON: {'information': {'component_accronym': 'REORDINIS', 'provider': 'https://www.enterprise-wisdom.com', 'component_name': 'Advanced Reconfiguration Optimization for Resource Distribution and Inventory Integration System'}, 'context': {'industry': ['26'], 'productiveaxis': {'ai_hri': False, 'ai_quality': False, 'ai_manualactivity': True}, 'category': {'reasoning': False, 'decisionmaker': True}, 'features': {}, 'benefits': {'quality': {'1': False, '2': False}, 'operator': {'1': False, '2': False, '3': False}, 'machine': {'1': False, '2': False}, 'production': {'1': True, '2': True}}, 'usecase': {}, 'description': 'Our AI-based Reconfiguration Optimization for Resource Distribution and Inventory Integration System (REORDINIS) addresses the challenge of complex decision-making in production and inventory management. Our solution consists of a set of AI technologies, namely simulation & search algorithms and knowledge-based models & queries, which optimize production plan (re)configuration and inventory management decisions to support operators and engineers. Our solution stands for a reconfiguration framework for production planning and inventory management enabling production plants to be robust, flexible and modular to handle uncertainties in the market, deviations during production, and systems failures and disruptions. '}, 'module_properties': {}, 'skills': {'basic': {}, 'crossfunctional': {}}, 'support': {}, 'technicalrequirements': {'multiuser_support': {'checkbox': False}, 'multienancy_support': {'checkbox': False}, 'gpu_unit': '1', 'ram_unit': '1', 'disk_unit': '1'}, 'datamodel': {'input': {}, 'output': {}, 'datamodel_links': {}}, 'public_endpoints': {}, 'keycloak_id': '7d579e92-6d52-4cd3-addd-cbf3be8a018c'}
'component_uuid'

INFO:     <IP_ADDRESS>:45045 - "GET /kits-ramp?sid=c2b6b4d6-fed6-4e35-8367-24ccf8157b61 HTTP/1.1" 307 Temporary Redirect

@karikolehmainen about the PC error:
the given datasheet does not match the current schema; indeed, the key information.component_uuid is missing from the json (and thus, it is impossible to create the payload to send to RAMP, which requires the tool uuid).
As I suggested in my previous comment, we need to fix some entries in the database (which possibly have been inserted by using an old version of the datasheets application), otherwise the PC will skip all of them if not providing the required properties (the exception is caught and the datasheet is discarded)

Schema is now fixed in datasheets backend. API is added to retrieve datasheets with valid RAMP UUID's also new datasheets must have valid UUID when created so issue should not arise again. (two out of 15 datasheets currently do not have valid UUID as the components have not been added to martketplace)


Created merged kube config file in the clientsholder.

Serialize the merged kube config into a temporary file and save the file path so it can be read on demand.
This could be used by Brandon's wip #631 for the preflight lib integration.

PANIC  [Nov  7 16:49:12.053][clientsholder.go: 136] Failed to create k8s clients holder: failed to create merged kube config file: failed to create temporary file: open /tmp/tnf-merged-kubeconfig-2690352657: no such file or directory

Yep, I already saw that error... not sure why yet, though. Seems like the /tmp folder in this github ubuntu runners is not writable (by default?) or it does not exist... I'm investigating it.

/dci-rerun


[Event Request] Page 5708 "Get Shipment Lines" - New publisher

Hello, we need a new Event publisher in OnQueryClosePage trigger. Can you please check that? Thank you!
trigger OnQueryClosePage(CloseAction: Action): Boolean
var
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeOnQueryClosePage(CloseAction, IsHandled);
    if IsHandled then
        exit;

    if CloseAction in [ACTION::OK, ACTION::LookupOK] then
        CreateLines;
end;

New Event:
[IntegrationEvent(true, false)]
local procedure OnBeforeOnQueryClosePage(CloseAction: Action; var IsHandled: Boolean)
begin
end;

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. 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.


Support for CloudFormation Templates With SSM Documents

The CloudFormation Template parser in Visual Studio is not aware of AWS::SSM::Document resources. It highlights the object with red underlining and when you hover over the Type statement the following is displayed: "Type key is invalid for this object "
This was a recent addition, so other resources like KMS are likely also missing. Please see this posting for a list:
https://aws.amazon.com/about-aws/whats-new/2015/12/aws-cloudformation-adds-support-for-aws-codepipeline-aws-config-and-more/

Thanks for pointing that out. I'll get the schema updated.

The CloudFormation schema has been updated. You should see the changes the next time you restart Visual Studio.

I restarted Visual Studio, and indeed the Type no longer has an issue, however the rest of the SSM document schema may need to be updated as well.

Sorry about that, restart Visual Studio and you should see that fixed. Note, for the actual SSM document we only validate that it is valid json.


Return type per HTTP status code?

[SwaggerWcfResponse(HttpStatusCode.OK, typeof(ClassA)]
[SwaggerWcfResponse(HttpStatusCode.BadRequest, typeof(ClassB)]

Is something similar possible?

Sure
New release with option for:
        [SwaggerWcfResponse(HttpStatusCode.OK, "Books found, values in the response body", responseTypeOverride: typeof(Book))]
        [SwaggerWcfResponse(HttpStatusCode.NotFound, "Book not found", false, responseTypeOverride: typeof(Book[]))]

Please note that parameter emptyResponseOverride will have higher priority

Thanks a lot, just tried it and it works exactly as expected 👍


[BUG] when on last page of a table and you delete all items, it errors out

I believe it is trying to pull data from the backend with the wrong page number but that page doesnt exist anymore so it errors out. Should probably just redirect user to the first table page if that is the case.

fixed as of #103


User Access and Deletion Permissions

It looks like I can delete others accounts in the "Manage users and rights" section. I haven't tried performing the deletion, so there may already be a security measure in place. Is this behavior tied to the beta-test conditions or linked to the privileges of a manager role?

Perfectly right. Good catch. I modified and now people can only see account they created. Only root can access all account.

Crap. No, people need to see other accounts. I made the wrong fix :)

Ok, you can see all the account but only delete those you have created, except you are root


Remove Beta label from site main page

Front page still says Beta

https://github.com/SumoLogic/sumologic-documentation/pull/825


[Feature Request] Progress Updates Lag During Buffering

Hi there, I'm trying to implement an interface like Apple Podcasts (when you move the slider, the time changes in real time). When I change the position on the slider, TrackPlayer.seekTo(val)
does not work instantly, because of this the slider twitches. I am using the @react-native-community/slider library. I tried adjusting the onSlidingStart and onSlidingComplete, and setting setTimeout, it improves the situation, but the twitching still remains. All examples available online also have this problem. Replacing the slider library doesn't solve the problem either. Could you please suggest how this problem is solved? Or maybe there is an example of a correct implementation with a slider? Thanks a lot!
Tested on physical iphone 13 pro max, ios 16.
 <Slider               value={progress.position}               minimumValue={0}               maximumValue={progress.duration}               minimumTrackTintColor={palette.ACTION_PRIMARY}               maximumTrackTintColor={palette.BORD}               onSlidingComplete={async (val: number) => {                 await TrackPlayer.seekTo(value)               }}             />
Version:  "react-native-track-player": "^2.1.3",
System:
OS: macOS 12.4
CPU: (8) arm64 Apple M1
Memory: 115.11 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.0.0 - /opt/homebrew/bin/node
Yarn: 1.22.17 - /opt/homebrew/bin/yarn
npm: 8.6.0 - /opt/homebrew/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK: Not Found
Android SDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7678000
Xcode: /undefined - /usr/bin/xcodebuild
Languages:
Java: 11.0.16 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.68.1 => 0.68.1
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
update ->
Answer by gavrichards from the discord. But I didn't manage to solve the bug:
"Hey, I have the same thing in my app. I have a boolean state isDragging, which I use to set the value of the slider, like this:
const showPosition = isDragging ? draggingPosition : trackPosition;
So if it's dragging, the slider shows the dragging position, otherwise it shows the actual track position.
onSlidingStart, I call setIsDragging(true) and set the dragging position.
onSlidingComplete, I seek to the new position, and then set a 1 second timeout before calling setIsDragging(false).
This is the trick. It keeps the slider position in the dragged position for 1 second until the library then reports the correct new track position."

@MrNapcae , I've not seen this issue before, and it doesn't seem to exist in the example project. When I run the example project on an iPhone SE Gen 2 with iOS 15.5 , the progress slider is very smooth.

https://user-images.githubusercontent.com/1452066/186220652-990b2cbb-80dc-45d7-943c-e0573913bdf0.MOV
Here's a recording of me scrubbing quickly. I see no issues.

If you're able to send me a consistent reproduction of this based off of the example app, I'll take another look at it.

If you're able to send me a consistent reproduction of this based off of the example app, I'll take another look at it.

the only change I made was to put in a longer video from the internet, duration: 1066 sec
https://user-images.githubusercontent.com/42024544/186224425-de08d857-0568-44ef-ac48-9b4f979f441a.MP4

@MrNapcae what you're seeing looks like its the result of your audio file buffering. I don't think it has anything to do with the .seekTo being slow like you initially thought. When you scrub to the desired position it takes a few MS to actually buffer the file at that position before playback begins. My recommendation would be to observe the playback state, and if it's loading/buffering pausing the update of your progress bar slider.
Because the progress events are asynchronous between the native code and JS we can't guarantee a progress event won't show up during buffering.
@dcvz it's possible we could solve something like this in the useProgress hook?

@jspizziri Is it possible to download the full audio before playing it in order to avoid this?

@MrNapcae yes, you can download and play offline. I'm not 100% sure that will solve your problem, buffering the file from disk should be faster than buffering it from network, it just depends on the speed.

@jspizziri then maybe you can find out the status that buffering is in progress?

@MrNapcae I'm not sure what you mean by that.

@jspizziri Is there a way to receive a status that audio file is being loaded and it’s not available for playing yet?

@MrNapcae yes, this is what I was mentioning here.  But there's no way I'm going to read through that massive text dump and try and interpret what you're asking me.

@MrNapcae I don't want to be rude. But there's no way I'm going to read through that massive text dump and try and interpret what you're asking me.

sorry, updated

Hi @jspizziri I have the same problem when I go from pause to play the status of the player changes from loading/buffering playing/ready/paused
only on iOS on Android no problems
Thanks

Hi , i have the same issue, i never get State.Buffering,  am i missing something?
It works with TS v2.1.3 but with any higher versions than that it doesnt.

@mekaVR @MrNapcae @izyspania
Please try the nightly build which we believe resolves the buffering not being emitted issue.
See here for more detail
https://github.com/doublesymmetry/react-native-track-player/issues/1720#issuecomment-1246485024

Sadly it does exactly the same , no Buffering state. "version": "3.1.0-7b247404c2b54df0550379277949a1cab5ddf47c",


fix: process incoming IP packets in a seperate task

This fixes receiving STUN packets during an endpoint update. Before they processing of incoming STUN packets would be interrupted due to the main actor loop waiting on the endpoint update.

One change is while before there was a receive_stun_packet fn that would enqueue the msg, now there is a channel that you can grab to send msgs. Was there anything wrong with the approach before, or is this just nicer?
Also, before the netcheck actor had its own message enum with just a single case. You removed that now, but I didn't really mind it.

Calling async fns from inside an actor is usually frowned upon.

that seems very unrealistic for most uses I have for this pattern

Calling async fns from inside an actor is usually frowned upon.

that seems very unrealistic for most uses I have for this pattern

🤷♂ I don't make the rules. https://en.wikipedia.org/wiki/Actor_model
"In response to a message it receives, an actor can: make local decisions, create more actors, send more messages, and determine how to respond to the next message received."
Doing async io inside the message handler can in my opinion be perfectly valid. I am not religious about this. But then it is no longer something that conforms 100% to the actor model.

But then it is no longer something that conforms 100% to the actor model.

fair, I honestly just call these actors because I have no better name for it 😅


Update low emission discount email contents

Description
Update low emission discount email contents.
Context
PV-771
How Has This Been Tested?
Manually with local email-backend.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (be9acf5) 84.23% compared to head (972f862) 84.25%.
Report is 1 commits behind head on develop.

Files
Patch %
Lines

parking_permits/views.py
85.71%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #477      +/-   ##
===========================================
+ Coverage    84.23%   84.25%   +0.01%
===========================================
  Files          160      160
  Lines         9809     9832      +23
===========================================
+ Hits          8263     8284      +21
- Misses        1546     1548       +2

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


Release Google.Api.Gax version 4.0.0

As a new major version, GAX v4 contains several breaking changes -
but we do not expect these to affect many users, as they are
typically used by the generated API-specific client libraries.
Most significant changes since 3.7.0:
GAX now targets netstandard2.1 and net462. (Changed from
netstandard2.0 and net461.) Older versions of .NET Core and .NET
Framework are no longer supported by Microsoft.
The default GrpcAdapter implementation is now GrpcNetClientAdapter.
Both GrpcNetClientAdapter and GrpcCoreAdapter are now part of
Google.Api.Gax.Grpc (so the other adapter packages will not be
updated to v4). Grpc.Net.Client is now a dependency of
Google.Api.Gax.Grpc, and GrpcNetClientAdapter is effectively the
default adapter, with a fallback to GrpcCoreAdapter for environments
where GrpcNetClient adapter does not work (such as on most .NET
Framework environments). There is no dependency on Grpc.Core;
GrpcCoreAdapter accesses the classes it needs via reflection. API
client libraries will ship with a conditional dependency on
Grpc.Core, only on .NET Framework. Over time, we expect usage of
Grpc.Core to fall to a level where we can remove support in a new
major version of GAX.
GrpcAdapter itself is more of a black box now, with just a single
public member - the static GetFallbackAdapter(ServiceMetadata)
method. This allows for more flexibility to evolve the adapter API
internally over time.
The new ServiceMetadata and ApiMetadata classes are used to
allow more metadata about an API to be generated, then used by GAX
intelligently, for example to pick the RestGrpcAdapter where
necessary.
Extension methods have been introduced to add GrpcAdapter
implementations into MS-based dependency injection. Doing this with
GrpcNetClientAdapter allows Grpc.Net.Client to use the same
service provider, propagating logging, HttpClientFactories etc.
Client builders now support MS-based dependency injection, usually
via a generated extension method. Where a client is configured using
dependency injection, the credentials, logger and gRPC adapter can
all be obtained from the DI container.
Additionally, gRPC client builders support a new GoogleCredential
property which uses the default scopes, self-signed JWTs and any
configured quota project.
gRPC client builders also expose any channels created in the course
of creating a client, enabling callers to dispose of those channels
appropriately. While the area of resource cleanup is still not where
we want it to be, this is a step in the right direction.
The gRPC call layer of GAX now supports logging, which will enable
a simpler diagnostic process when investigating problems.
GAX no longer has a dependency on the Grpc.Gcp NuGet package, but
includes a refactored version of the same functionality within
Google.Api.Gax.Grpc; this means the Google.Api.Gax.Grpc.Gcp package
will not be updated to v4.

Spanner integration tests are still running with a locally-built release candidate, but I expect them to pass.


Enable running only tests without build phase

TODO:

[x] break the copr-build -> test dependency
[ ]  verify correctness of the format of payload if skip_build=True
[x] refactor tests

Fixes #1078

TBD

@lbarcziova thanks for working on this! :rocket:
What about having a PR on hello-world with this usecase?

We should also document this. Once in prod, we can even remove the spec file in RedHat-SP-Security/tests and finally, I'd love to announce this to QE (together with @thrix) since this is a big feature.

What about having a PR on hello-world with this usecase?

Definitely, I had it in the plan.
I would like to properly test it at first (also existing functionality, since there were refactoring changes) and then if we see it works as expected, add documentation and announce it.


[cPP Correction] (D)TLS Sever Specifies Client Requirements

Provide the location of the issue
Section A.7.1.1, Application Note 47
Section A.7.1.2, Application Note 53
What is the correction request for the cPP? Please describe.
The Server SFR specifies client requirements, “This requirement adds that the client must be capable of presenting a certificate to a (D)TLS server for (D)TLS mutual authentication.”
Describe the solution you'd like
Recommend reverting this to NDcPPv2.2e text, "This requirement adds that the TSF must include support for client-side certificates for (D)TLS mutual authentication."

Another alternative is "This requirement adds that the TSF must include support for (D)TLS mutual authentication."

I tend to favor less words, but good with either update from Kenji or Rick here.

Agreed to go with Ricks wording on today's TLSWG call.


feat: consolidated yolov5s effnetb0

PR Checklist

[x] Added label to the Pull Request for easier discoverability and search
[x] Commit Message meets guidelines as indicated in the URL https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
[x] Every commit is a single defect fix and does not mix feature addition or changes
[ ] Unit Tests have been added for new changes
[ ] Updated Documentation as relevant to the changes
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with repository license and clearly outlined the added dependency.
[ ] PR change contains code related to security
[ ] PR introduces changes that breaks compatibility with other modules (If YES, please provide details below)

What are you changing?
adding consolidated yolov5s eenetb0 script
Issue this PR will close
close: #issue_number
Anything the reviewer should know when reviewing this PR?
Test Instructions if applicable
test the script inside the container
If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/automated-self-checkout )

@jim-wang-intel as we have discussed this morning regarding your comments, the plan is to merge this PR into @brian-intel 's consolidateDLStreamer, then we will work on the ENV to replace the current parameter. with that discussion, can I resolve your comments here?


The resource failed to load

demo page https://www.composerize.com/

Hi,
Thanks for this issue. By the way:

carbon.js is an ad system added by original author of this project. Will not remove myself. Failed to load because blocked by something like uBlock Origin
count.js (goat counter) is a small "visit counter", privacy aware as far as I know, also failed to load because blocked by something like uBlock Origin
did not reproduce loading failure of fontawesome.woff in Chrome

By the way, nothing to fix at the time being


Added abstractions to OpenGL

Summary of the PR
Added simple abstractions to core OpenGL functions/objects. also adds a new Example Called "Further Abstractions".
Related issues, Discord discussions, or proposals
Links go here.
Further Comments
What turned me away from OpenGL in the first place was the complicated API and everything you needed to do just to get a triangle. this PR tries to make it very simple to draw vertices with OpenGL. you would create a GLObject and pass the vertices, indices and a refrence to a GL object. the base Absrtactions used in the examples were used for the GLObject class.
GLObject object = new GLObject(GL.GetApi(window), verts, indices);
void OnLoad(){
object.CreateResources("PathToVertexShader", "PathToFragmentShader");
}
void Render(double obj){
object.Render();
}
void OnClose(){
object.DisposeResources();
}

Ignore the SHFB stuff, Its been removed.

@RhysWhy you open to giving this a quick lookover?

@Perksey I took a look over the meat of this one as asked, just to give you a quick summary of my findings, the two main parts of this are as below:
a) Adding the Abstractions csproj with its corresponding classes
b) Adding a new Tutorial to the examples folder that relies on and demonstrates the objects added to the Abstraction csproj
As you've requested that the abstractions project be removed as its not (yet) desired then the additional example also would need removing, which is essentially this entire PR

Gotcha, thanks for taking a look through anyway and apologies for delegating this to you, figured you knew the tutorials better than anyone and it's better than it sitting around for months until i can get some quality time to look through it.
@WholesomeIsland Per Rhys' findings, I'm going to close this pull request as we have our own plans to achieve something similar to this.
Please feel free to use either the Discussions tab or our Discord server to discuss these changes with us before making them so that we can tell you before you make the change whether we think it's something suitable for Silk.NET.


Update to 1.21 Please

I just wanted to test it with other mods a bit and discovered that there is literally no content mods for Fabric 1.20.6, lol. And all "modpacks" are basically vanilla + sodium + REI

1.21 now would be great 😊

I will not touch 1.21 untill its finished (that is, 1.22 starts to come out).  And Im still thinking about porting villager trades mod... And about porting RELT for forge... Idk, too much routine work for a game that I barely even play nowadays

Yet it's finished, if you plan support for minor versions with possible big changes (ouf), that's on you.
But I get your decision 😊

I have updated my fork of this mod to 1.21, if you are interested in that you can grab the release from here.
Additionally, I have opened a pr so the update might get integrated in the mod itself

Thank you so much @tyrannus00!
I tested your fork right now, works fine on the Client but my server got this crash: https://pastebin.com/raw/Yf2eUAt7
Tested on newest REI release "RoughlyEnoughItems-16.0.744-fabric.jar".
Can soon be closed when @GrigLog accepts your PR 😄


ref #803: remove window open on wysiwyg download links

Q
A

Branch
7.1.x

Bug fix?
yes

New feature?
no

BC breaks?
yes

Deprecations?
no

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#835

License
MIT

I marked it as BC break, but only because it removes a feature. But this feature is outdated and produces problems, so i think we can live with it. You would not implement a window open as inline onclick event today.


min-storage advanced option does not compile with MPI

the operator in the MFE below does not compile when run with DEVITO_MPI=1
from devito import Grid, TimeFunction, Eq, Operator
grid = Grid(shape=(41,41,11))
p0 = TimeFunction(name='p0', grid=grid, time_order=2, space_order=8)
update_p = (p0.dx).dx + (p0.dy).dy + (p0.dz).dz
op = Operator(Eq(p0.forward, update_p), name='MFE', opt=('advanced', {'min-storage': True}))
op.apply(time_M=10)

Leads to  gcc-related loop definition errors
fore 'x'
  191 |         for (int x = x0_blk0 - 4, xs = 0; x <= x0_blk0 + x0_blk0_size + 3; x += 1, xs += 1)
      |                                           ^
/var/folders/mx/qs0dn9rx7zn6dz2zvwv7tkk00000gn/T/devito-jitcache-uid501/229a49fd56fb5533d63a353d9945a04b9700d2be.c:211:45: error: expected iteration declaration or initialization before 'y'
  211 |           for (int y = y0_blk0 - 4, ys = 0; y <= y0_blk0 + y0_blk0_size + 3; y += 1, ys += 1)

Need to double check than icc still fine with this one


latest version

Does latest version is 0.12.8 (on Apr 6, 2016) or 0.7.5 (on Dec 21, 2016)?
my node v7.5.0, "npm i node-inspector@latest -g" return 0.12.8 and not works.
Thanks

Please try again, the latest version is 1.0.0 now.


Suppress default hover_data

Is there any way to suppress the default hover_data when the hover_data argument is passed? For example, my plot doesn't need the x, y, and color args in the hover_data, but I can't figure out how to remove them.

At the moment there's no way to do this from the arguments, but this is something I'll look into. You can modify the hoverlabel contents fairly easily by modifying the hovertemplate attribute of the traces.

Right now the default value of hover_data is None, and it accepts an array of columns to add to the existing set of key-value pairs listed (which currently include every value mapped to some visual attribute in the chart). In terms of API, I could imagine either adding a new kwarg hover_data_exclusion or having hover_data accept a dict which would control if/how values are displayed. Someone recently suggested the idea of this kind of API to control the precision/formatting of numbers in hover labels with e.g. hover_data={"column": "format"} and this would fit in pretty nicely as hover_data={"column": False}

Will be completed in https://github.com/plotly/plotly.py/issues/1774

At the moment there's no way to do this from the arguments, but this is something I'll look into. You can modify the hoverlabel contents fairly easily by modifying the hovertemplate attribute of the traces.

For px.choropleth_mapbox, there is no hovertemplate property, right? I hope to do custom formatting like in go.Choroplethmapbox.

I have the same issue with px.scatter_mapbox as well. Currently, lat, lon and color are also being included in the hover text and there is no corresponding hovertemplate.


Dashboard for tracking surveillance and management progress for feeding RIPARIAS KPIs

Now that we have this dashboard for exploring the baseline distribution of the RIPARIAS target species, the idea is to extend this dashboard with some indicators at the level of the project. This is a development that will serve the following actions and can also help assess the KPI's for the project:

D.1 Monitoring the effectiveness of the early detection and rapid eradication
D.2 Monitoring the effectiveness of management actions for widespread IAS (at a later stage)

To do this, the coordination group decided on the idea of an online dashboard where some indicators to track progress (as 'distance from baseline') in a more or less real-time way. From the slides presented at that meeting:

Therefore, the idea is to perform some overlays and do some analysis on the baseline data. Notably, after discussion with @bramdhondt and @jrhillae we decide to visualize the following 'indicators':

the number of observations per species compared to the baseline (barplot with 2 values per species), for the 3 basins (with a drop down possibility to select river basin subunits) - this should e.g. show increases in detections for specific species after bioblitz events or communication actions boosting surveillance
the occupancy (1km grid cell) of the species (I propose to use a 1km grid resolution using Inspire proof EEA reference grid) (barplot with 2 values per species) - this would give a crude indication of the evolution of the spatial distribution (and show that actions at local level for established species not necessarily impact distribution at higher spatial scales downstream)
an explicit measure of surveillance effort (~time spent in the field, to be reported by RIPARIAS partners that are responsible for surveillance: ANB, VMM, CR), visualized at river basin unit and river basin subunit level
Percentage of well prospected 1 km squares (for plants) where well prospected is defined as the % of 1km  squares in a unit/subunit with >100 plant species reported - this gives a general appreciation per subunit of the survey intensity

All this should be organised per river basin subunit as these are used in RIPARIAS for organising management and surveillance.

To get new observations run the easin - flow (Create_Parameter_Files.Rmd -> gbif_prep.Rmd -> merge_data.Rmd -> copy all files from ./Data/Spatial/T999/Batch2_1_3_999 to /master/data/spatial/baseline)

the number of observations per species: baseline runs 1/01/2000 - 31/12/2019
cube is 1km resolution

https://github.com/inbo/riparias-prep/blob/d80ec369174f1c21147aeaec9277f7f04c11399f/scripts/baseline_maps_prep.Rmd#L83-L88
add chunck like the one above with subset > 2020

remark: absences available for e.g. Hydrocotyle (manscape dataset), possibility to visualise these on the baseline map?

@timadriaens @SanderDevisscher : Occupancy and observations are included in the shiny app. Normally, the output should be visible when clicking 'run app' for the script 'baseline_maps_ui_D1.R' in the branch 41_extending_baseline_map. I am planning to add 'Percetange of well prospected 1km squares' very soon. (Hopefully on Friday). Any feedback is welcome.

@timadriaens knows, but maybe all the others not yet. I created the new data cubes last week and deposited on zenodo. Cubes for all IAS taxa from GRIIS Belgium at Belgian level, cubes at class level at Belgian level and a subset of modelling species (including the ones from the RIPARIAS target list) at European level
I have also updated the cubes at species level with ALL species present in several countries, a.o. Belgium: https://zenodo.org/record/6341688.

@jrhillae Looks good, Jasmijn. Only minor things after a quick check: (1) maybe add a unit for Occupancy in the graphs (e.g. (1km² grid cells)); (2) for A0_CODE, maybe also a drop-down menu (instead of an up-down menu) ?

@timadriaens @SanderDevisscher@bramdhondt : An extra subpanel has been added with relative occupancy. The graph resembles the graph of absolute occupancy, only the axis differs. Maybe I can try to combine both (absolute and relative occupancy) in one graph? For instance by displaying relative occupancy on the axis and showing absolute occupancy as a label per bar?

@SanderDevisscher : In script occupancy_rel_prep I get the followinng error when running this line:

EEA_1km_in_RBSU <- raster::intersect(EEA_1km, RBSU)
Loading required namespace: rgeos
y[subsy, ] is invalid
Attempting to make y[subsy, ] valid by zero-width buffering
There were 15 warnings (use warnings() to see them).

To be checked if this is a real issue. So far, the intersect appears to work correctly.

If i remember correctly this means some of the polygons are not closed and thus invalid. Intersect tries to fix this by applying a zero width buffer and apparently it worked.

An elegant way to solve this is provided by sf package:
https://r-spatial.github.io/sf/reference/valid.html

@damianooldoni : Thank you for the tip, I added the fix to the code.

@timadriaens:
Except  for the trend graphs (which Sander created in a separate branch) the changes to the dashboard are finished. I still have a few questions/suggestions (I will make separate issues for some of them but a short overview is also handy):

When determining the grid cells with high surveillance effort I calculate the sum of total plant species observed in a grid based on the cube, but I do not add a filter based on year. Is it logical to maybe add a filter stating that only observations after 2000 are taken in account? (Now, any observation at any time is taken in account).

-If I am correct (@SanderDevisscher : you can correct me if I am wrong) the input files that are used to generate the dashboard do not yet contain all the species of the Riparias checklist. @SanderDevisscher : Maybe we should adapt the original script that you used to  recreate the baseline.geojson file and current_state.geojson file so that they contain all species of the checklist?
-How to deal with species of the Riparias checklist not yet observed in the area? For the moment only species which are observed within the Riparias area can be selected in a dropdown menu. The code is not yet robust enough to be run in case of absences. Maybe we can keep it like this but add an extra tab to the dashboard, called Riparias checklist, which states per species whether or not it is already observed in the Riparias area. Further, we can add a caption below each page stating that only species can be selected which are already observed in the Riparias area. Would that be ok?


Memory issue with timer and lambda

Describe the bug
Memory issue with a lambda that references a local variable.
To Reproduce

Run vim --clean (or gvim --clean, etc.)
Edit example.vim

" Show PID, for monitoring memory
echom getpid()

function! Function(timer) abort
  let l:cmd = 'let x = 2 + 2'
  call timer_start(1, {-> execute(l:cmd)})
endfunction

let timer = timer_start(20, 'Function', {'repeat': -1})

Source example.vim

source example.vim

Monitor memory

watch 'pmap <PID> | grep total'

Describe the error

Memory continuously increases.
Expected behavior
Memory would remain constant.
Environment (please complete the following information):

Vim version:

$ vim --version
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Feb 15 2021 12:29:39)
Included patches: 1-2434
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>

OS: Ubuntu 21.04
Terminal: Konsole 20.12.3

Additional context
The memory leak does not occur when the executed command is not stored in a variable.
function! Function(timer) abort
  call timer_start(1, {-> execute('let x = 2 + 2')})
endfunction

valgrind log

Daniel Steinberg wrote:

Describe the bug
A timer with a lambda that references a local variable leaks memory.
To Reproduce
Detailed steps to reproduce the behavior:

Run vim --clean (or gvim --clean, etc.)
Edit example.vim

" Show PID, for monitoring memory
echom getpid()

function! Function(timer) abort
  let l:cmd = 'let x = 2 + 2'
  call timer_start(1, {-> execute(l:cmd)})
endfunction

let timer = timer_start(20, 'Function', {'repeat': -1})

Source example.vim

source example.vim

Monitor memory

watch 'pmap <PID> | grep total'

Describe the error

Memory continuously increases.
Expected behavior
Memory would remain constant.
Environment (please complete the following information):

Vim version:

$ vim --version
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Feb 15 2021 12:29:39)
Included patches: 1-2434
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>

OS: Ubuntu 21.04
Terminal: Konsole 20.12.3

Additional context
The memory leak does not occur when the executed command is not stored
in a variable.
function! Function(timer) abort
  call timer_start(1, {-> execute('let x = 2 + 2')})
endfunction

I cannot reproduce it.  When I run the script under valgrind it does not
report any leak (compiled with -DEXITFREE).  I also tried writing a test
with this mechanism and run it under valgrind, didn't report a problem
either.
Possibly it's not a real leak but inefficient use of memory.  Can't
guess how.
--
"Space is big.  Really big.    I mean, you may think it's a long way down the
road to the chemist, but that's just peanuts to space."
-- Douglas Adams, "The Hitchhiker's Guide to the Galaxy"
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I encountered this issue using an earlier version of Vim.
After compiling the latest version from source, I still encounter the continuously increasing memory usage (monitored with watch 'pmap <PID> | grep total').
The Valgrind leak summaries differ:
Leak summary using pre-installed Vim on my system:
==123893== LEAK SUMMARY:
==123893==    definitely lost: 0 bytes in 0 blocks
==123893==    indirectly lost: 0 bytes in 0 blocks
==123893==      possibly lost: 342,051 bytes in 5,601 blocks
==123893==    still reachable: 5,800,465 bytes in 23,704 blocks
==123893==         suppressed: 0 bytes in 0 blocks

Leak summary with Vim compiled from source:
==123454== LEAK SUMMARY:
==123454==    definitely lost: 520 bytes in 1 blocks
==123454==    indirectly lost: 132 bytes in 6 blocks
==123454==      possibly lost: 0 bytes in 0 blocks
==123454==    still reachable: 77,961 bytes in 873 blocks
==123454==         suppressed: 0 bytes in 0 blocks

definitely lost: 520 bytes in 1 blocks

There are some bytes lost, but that's clearly not your issue. This really does not look like a memory leak.
Can you share the complete valgrind log?

"Can you share the complete valgrind log?"

👍 Sure. Attached are both logs, for pre-installed Vim and Vim compiled from source. For the latter, I uncommented LEAK_CFLAGS = -DEXITFREE in src/Makefile, based on the instructions in :help valgrind. This was presumbably not done for the pre-installed Vim, based on there being no -DEXITFREE shown in :version output.
valgrind-compiled-vim.txt
valgrind-pre-installed-vim.txt

"Possibly it's not a real leak but inefficient use of memory."

"This really does not look like a memory leak."

I've changed the title and description accordingly.

Okay, I did not see any leaks that come from Vim directly. I skimmed through it and it looks all the leaks where caused by some libraries (libXt, libX11 etc). I do wonder, if you stop the timer, does perhaps garbage collector skip in and releases some memory later?

"I do wonder, if you stop the timer, does perhaps garbage collector skip in and releases some memory later?"

When I run :call timer_stop(timer) to stop the outer timer, the memory that grew from earlier is not released.
I also tried retaining the inner timers (those created in Function that run once) in an array and later calling timer_stop on all the IDs in the array. That also had no effect on the memory that grew from earlier.

You can see a big difference in "still reachable" between the run with and without EXITFREE.  That means it's not a real leak, but Vim does keep a lot of things in memory.  It might indeed be that that the garbage collector is not triggered, or that there is a dependency cycle.  This may relate to register_closure() and current_funccal->fc_funcs

For the title and description, I mentioned timers, as that was the context under which I noticed the memory increase, and I thought there was a relationship.
However, the following steps also result in unreleased memory, without the use of timers.
To Reproduce

Run vim --clean (or gvim --clean, etc.)
Edit example.vim

" Show PID, for monitoring memory
echom getpid()

function! Function() abort
  let l:string = 'hello world!'
  return {-> l:string}
endfunction

Source example.vim

:source example.vim

Monitor memory

$ watch 'pmap <PID> | grep total'

Call the function many times

:for _ in range(1, 500000) | call Function() | endfor

Describe the error

Vim's memory usage increases to over 1GB.

What happens if you delete the function: :delfunc Function

"What happens if you delete the function: :delfunc Function"

Memory usage remains the same (slightly above 1GB in the example above) after running :delfunc Function. I also tried :call garbagecollect(), which also did not effect the memory usage.

I've simplified the example further below. It's not necessary to use a string local variable, nor return the lambda from Function.
To Reproduce

Run vim --clean (or gvim --clean, etc.)
Edit example.vim

" Show PID, for monitoring memory
echom getpid()

function! Function() abort
  let x = 0
  let Y = {-> x}
endfunction

Same as above
...

I've simplified the example further below. It's not necessary to use a
string local variable, nor return the lambda from Function.
To Reproduce

Run vim --clean (or gvim --clean, etc.)
Edit example.vim

" Show PID, for monitoring memory
echom getpid()

function! Function() abort
  let x = 0
  let Y = {-> x}
endfunction

Same as above
...

This clearly is a circular dependency: the local variables of Function()
are used by the lambda, and the lambda is stored in a local variable.
Thus the lambda can't be freed because it is referred to by a local
variable, and the local variables cannot be freed because they are
referred to be the lambda.
Not sure how to solve this.  We should somehow detect that "Y" isn't
used.
--
The only backup you need is the one that you didn't have time for.
(Murphy)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Perhaps just mention to avoid circular dependencies and if in doubt simply use unlet! to free local variables that are no longer needed.

Am 24.06.2021 um 22:18 schrieb Bram Moolenaar @.***>:

I've simplified the example further below. It's not necessary to use a
string local variable, nor return the lambda from Function.
To Reproduce

Run vim --clean (or gvim --clean, etc.)
Edit example.vim

" Show PID, for monitoring memory
echom getpid()

function! Function() abort
let x = 0
let Y = {-> x}
endfunction

Same as above
...

This clearly is a circular dependency: the local variables of Function()
are used by the lambda, and the lambda is stored in a local variable.
Thus the lambda can't be freed because it is referred to by a local
variable, and the local variables cannot be freed because they are
referred to be the lambda.
Not sure how to solve this. We should somehow detect that "Y" isn't
used.
--
The only backup you need is the one that you didn't have time for.
(Murphy)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net \
/// \
\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\ help me help AIDS victims -- http://ICCF-Holland.org ///
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

"Perhaps just mention to avoid circular dependencies and if in doubt simply use unlet! to free local variables that are no longer needed."

When using unlet!, the memory is released for the simplified example from above (without the return).
function! Function() abort
  let x = 0
  let Y = {-> x}
  unlet! Y
endfunction

I also tried it with an adapted version of the function shown in :help closure (adapted to create and call Bar many times), which returns the closure from the function, which results in the memory not being released.
function Foo(arg)
  let i = 3
  return {x -> x + i - a:arg}
endfunction

for _ in range(100000)
  let Bar = Foo(4)
  call Bar(6)
  unlet! Bar
endfor

The memory only increases the first time running. That is, if I re-run the for loop, I don't see a subsequent increase in memory.

Daniel Steinberg wrote:

"Perhaps just mention to avoid circular dependencies and if in doubt
simply use unlet! to free local variables that are no longer
needed."

When using unlet!, the memory is released for the simplified example
from above (without the return).
function! Function() abort
  let x = 0
  let Y = {-> x}
  unlet! Y
endfunction

I also tried it with an adapted version of the function shown in
:help closure (adapted to create and call Bar many times), which
returns the closure from the function, which results in the memory not
being released.
function Foo(arg)
  let i = 3
  return {x -> x + i - a:arg}
endfunction

for _ in range(100000)
  let Bar = Foo(4)
  call Bar(6)
  unlet! Bar
endfor

The memory only increases the first time running. That is, if I re-run
the for loop, I don't see a subsequent increase in memory.

The garbage collector only runs in the main loop, not halfway executing
code.  And most systems don't release (virtual) memory obtained before,
so the size won't go down.  Thus this is normal.
--
"You know, it's at times like this when I'm trapped in a Vogon airlock with
a man from Betelgeuse and about to die of asphyxiation in deep space that I
really wish I'd listened to what my mother told me when I was young!"
"Why, what did she tell you?"
"I don't know, I didn't listen!"
-- Arthur Dent and Ford Prefect in Douglas Adams'
"The Hitchhiker's Guide to the Galaxy"
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Had another look at this, and I see two separate problems.
First is for the original case: the timer calls the function over and over again.  In this case memory will be freed as soon as you type something, so that the main loop is encountered and the garbage collector is invoked.  You can see this by sourcing the script, wait until the number starts going up, then type some command repeatedly, e.g. "jk".  You will see the number stops rising and may also go down, until you stop typing for a while, then it goes up again.
I would consider this normal behavior.  We have had problems before calling the garbage collector when not in the main loop, since it's very difficult to track everything that might be in use somewhere halfway a command.
The second is the case where we have the circular reference: A local variable holds a reference to a closure, and the closure uses a variable in the function context.  It's very difficult to detect that this cycle has no outside references.   It might be possible, but quite hard to implement and test.  I'll add a note in the help to avoid this.  It normally would not happen anyway, the example is not useful code.

Thanks!


[🐞]  Issue with list item keys not updating in DOM

Which component is affected?
Qwik Runtime
Describe the bug
I am experiencing an issue where list items are not updating correctly in the DOM when using the Qwik framework. Despite using unique keys for each list item, the changes are not reflected in the rendered output. The key attribute seems to be ignored, and instead, I see data-qwik-inspector attributes with no q:key present in the <li> tags.
Reproduction
https://stackblitz.com/edit/qwik-starter-qsgti2?file=src%2Froutes%2Fadd-names%2Findex.tsx
Steps to reproduce

Use the button to add the input field's text to the list.
Observe that the list in the DOM does not contain q:key which force the client to render all the items

System Info
System:
    OS: Linux 5.10 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
    CPU: (4) x64 Intel(R) Core(TM) i7-5500U CPU @ 2.40GHz
    Memory: 3.03 GB / 7.67 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 20.9.0 - /usr/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/bin/npm
  Browsers:
    Chrome: 110.0.5481.177
  npmPackages:
    @builder.io/qwik: ^1.2.17 => 1.2.17
    @builder.io/qwik-city: ^1.2.17 => 1.2.17
    @builder.io/qwik-react: 0.5.0 => 0.5.0
    undici: ^5.26.0 => 5.27.2
    vite: ^4.4.11 => 4.5.0

Additional Information
I have followed the recommended practices for list rendering in Qwik, but the issue persists. Any guidance or assistance in resolving this would be greatly appreciated.
Thank you for your help!

To see the q:key you need to pause the container.
It seems to work correctly for me, how do you know everything re-renders?
You can copy this code into the playground, that's faster to play with and you can see the console logs from server and client to detect renders.
Iterating over the list will always cause re-rendering of the parent component.

Thanks for your response. I'm a bit confused. If an  item has a unique key, doesn't that prevent it from re-rendering when we push a new item to the array? I'm concerned about the potential performance impact on the client side when rendering a large number of list items

@samwalters3 See https://qwik.builder.io/playground/#v=1.2.17&f=7VZNS8QwEP0rsQhtIdsFL%2BqyKXj07kWWRYS6rFjtghSR0v%2Fue5Nk0hYPexXsqW3mI5O8efMmoLm5vv0NNFZHrjVnIchXfM%2Fx42YwGoRqrBDQWLqa5zCd0bk%2Ffxx9DiPalhPMta%2F1IJ7bNV4BuPMgGzSG0LvTUgouGSRBXz6gfzYmz63%2FRXbcmN0epBVEy27PJSJc4z03TShwmgveB1NIrkpDV4jxDpsL55DE441PMGO66tR%2FHpd%2BSDczjP%2BRM8dpc2VM25rjiVItXrw4KirAWj3kzSImA%2FmHDekyUkqm%2F7oP0VOXbiierMHgJ3dJ0cs9Y29huZJEZQp8aqFCjh0x47JHihFOqJREZKTLvlYHDjLwuCiJ1VW0SEj2%2BjPJriFcxljfNRiyneGEV5kavfo2tUYsX84eYq0o%2BBoxJ4WpLVzlpt9evp0H8CgYwQdzeh9d0V2iv7T47Tom9yI6NN8%2FY%2F5JxvwB
To iterate over the list, the render function needs to run, but as you can see each list item only renders once when added. Iterating over the items happens in the vDOM only.


[ocaml] integrate variable-length API for poseidon (feature/snappsprotocol branch)

see https://github.com/MinaProtocol/mina/issues/9032

This is an issue for o1js, not the protocol. Closing.


Unpin cairo version in setup.py

This line:
https://github.com/curtacircuitos/pcb-tools/blob/cf4a61244519f0ba0889beab3d5a3db61c44362d/setup.py#L43
Pins cairocffi at 0.6.0, but the latest is 1.1.0: https://pypi.org/project/cairocffi/
This means that Python applications that use pcb-tools are stuck with an older version. Is there a specific reason for pinning this to a 6 year old version?

This is also currently pinned in requirements.txt as well:
https://github.com/curtacircuitos/pcb-tools/blob/cf4a61244519f0ba0889beab3d5a3db61c44362d/requirements.txt#L2


Pin mongodb-k8s charm version for release 2022 05 05

Pin mongodb-k8s charm version for previous Legend bundle release.

xref: https://github.com/finos/legend-juju-bundle/pull/34
Same as the PR above for release-2022-05-05 branch.

Failure is unrelated, it will be addressed in an another PR, but this PR is necessary for the integration tests to pass.


Deploy database migration

Database migration is already on the deploy branch, but this reverts the controller logic so that the migration can happen first.
Merging this will cause a Heroku deploy that only contains the DB migration

@asfalcone Approves


perf(routes): check accept header is supported before parsing req body

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
587

Relevant Lines:
587

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
587

Relevant Lines:
587

💛  - Coveralls


Create User Account Creation Capability

We need people to be able to create use accounts. We'll probably use OmniAuth and/or Devise and CanCanCan.

Turns out this is functionally a duplicate of the OmniAuth (Omniauth being dependent on Devise) issue so I'm closing this.


dbt==0.20.1 commands break on python 3.9.7

Many dbt commands (dbt run, dbt test, ...) are breaking on Python 3.9.7 released yesterday: https://docs.python.org/release/3.9.7/whatsnew/changelog.html.
Here is a traceback:
2021-09-01 14:55:10.133377 (MainThread): Running with dbt=0.20.1
2021-09-01 14:55:11.273707 (MainThread): Encountered an error:
2021-09-01 14:55:11.273998 (MainThread): Profile() takes no arguments
2021-09-01 14:55:11.275664 (MainThread): Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/dbt/main.py", line 125, in main
    results, succeeded = handle_and_check(args)
  File "/usr/local/lib/python3.9/site-packages/dbt/main.py", line 203, in handle_and_check
    task, res = run_from_args(parsed)
  File "/usr/local/lib/python3.9/site-packages/dbt/main.py", line 242, in run_from_args
    task = parsed.cls.from_args(args=parsed)
  File "/usr/local/lib/python3.9/site-packages/dbt/task/base.py", line 157, in from_args
    return super().from_args(args)
  File "/usr/local/lib/python3.9/site-packages/dbt/task/base.py", line 75, in from_args
    config = cls.ConfigType.from_args(args)
  File "/usr/local/lib/python3.9/site-packages/dbt/config/runtime.py", line 233, in from_args
    project, profile = cls.collect_parts(args)
  File "/usr/local/lib/python3.9/site-packages/dbt/config/runtime.py", line 211, in collect_parts
    profile = cls._get_rendered_profile(
  File "/usr/local/lib/python3.9/site-packages/dbt/config/runtime.py", line 190, in _get_rendered_profile
    return Profile.render_from_args(
  File "/usr/local/lib/python3.9/site-packages/dbt/config/profile.py", line 423, in render_from_args
    return cls.from_raw_profiles(
  File "/usr/local/lib/python3.9/site-packages/dbt/config/profile.py", line 388, in from_raw_profiles
    return cls.from_raw_profile_info(
  File "/usr/local/lib/python3.9/site-packages/dbt/config/profile.py", line 338, in from_raw_profile_info
    return cls.from_credentials(
  File "/usr/local/lib/python3.9/site-packages/dbt/config/profile.py", line 242, in from_credentials
    profile = cls(
TypeError: Profile() takes no arguments

For more details please see the following Slack thread: https://getdbt.slack.com/archives/C99SNSRTK/p1630506829031300

Can confirm, I faced the same issue. I wonder if it's this change in the Python 3.9.7 release causing the issue?

bpo-44806: Non-protocol subclasses of typing.Protocol ignore now the init method inherited from protocol base classes. And the PR in question.

The [Profile] class in dbt ultimately inherits from a Protocol class, and I guess something's going wrong with the dataclass-generated __init__ there?
IANAPG (I Am Not A Python Guru) so I'm mostly just guessing though! :man_shrugging:

Thanks for jumping into this one, everyone! And thanks for opening the python issue, @julianfortune.
I'm not yet sure whether this represents:

an unintended/unsupported mechanism of protocol inheritance/instantiation, in which case, we should refactor the places where we use Protocol (there's a lot of them)
a regression in python 3.9.7 that will be fixed in a future patch release, at which point we should update python_requires to mark 3.9.7 as unsupported

As a temporary workaround you can patch dataclasses._set_new_attribute method:
if sys.version_info >= (3, 9, 7):
    import dataclasses

    original_set_new_attr = dataclasses._set_new_attribute

    def patched_set_new_attr(cls, name, value):
        if name == '__init__' and any(getattr(c, '_is_protocol', False) for c in cls.__bases__):
            del cls.__init__

        return original_set_new_attr(cls, name, value)

    dataclasses._set_new_attribute = patched_set_new_attr

@uriyyo ’s cpython patch

@uriyyo Thank you for the workaround, the fix, and a new test case to prevent the regression in the future! 🎉

@julianfortune it's not clear, is it possible to fix the issue now by upgrading dbt or python 3.9 ? Or are we waiting for either  dbt or python one to make a new public release ?
If I want to patch manually, where can I find dataclasses._set_new_attribute ?

@Startouf -- the latest patch release of dbt implements a workaround (#3855), so you can upgrade dbt to 0.20.2 and not worry about the Python bug.


(Preview) TI map Domain entity to DnsEvent

Is your feature request related to a problem? Please describe.
I have turned on my TI map Domain entity to DnsEvent analytic rule. I get duplicate value even after I have investigated IP address. How can I stop receiving the alerts for the IP which I have investigated in past.
Describe the solution you'd like
I would like to avoid getting same IP address for investigation every hour when the rule is run.
Describe alternatives you've considered
The alternative I have been using is created manual excel sheet of all the IP addresses which have been evaluated. Rather than checking in various tools it is easy to just search for IP address in excel.

Hello Jeremy Tan, I am trying to implement this query as a rule in our environment, but we do not have DnsEvents data source (table). Is it possible to modify this query using only ThreatIntelligenceIndicator data source and get only top 10 malicious-activity IPs which are not repeated?


FieldData -> add typeName field and length field

As per pull request
https://github.com/codeigniter4/CodeIgniter4/pull/3287
added field type name.
had to redo it as I had to re-fork the repo and that broke the previous pull request.
Will return NULL if mapped field type name not found
the DB version of the type returned by driver is stored in the type column
Where applicable length is stored or made equal to max_length as in for example MySQL the max_length is based on
the data returned and not the DB schema where length is closer to the truth but still not 100%
cant think of a alternative that wont be affect the performance of the procedure. Is a need if want to build schema
from this data in the future
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Yay! We made it finally 🥳
Thank you @devorama!


Role based access control and other fixes

You can now define various query modifiers on a per role basis.  For example a user with an admin role can have a certain filter applied while users with other roles can have other modifiers applied.
The roles_query config along with the match config is how a role is dynamically applied to a user. For example if a user with id 1 comes along the id = 1 match config will ensure he gets the admin role applied to him.
roles_query: "SELECT * FROM users as usr WHERE id = $user_id"

roles:
  - name: anon
    tables:
      - name: products
        limit: 10

        query:
          columns: ["id", "name", "description" ]
          aggregation: false

        insert:
          allow: false

  - name: user
    tables:
      - name: users
        query:
          filters: ["{ id: { _eq: $user_id } }"]

  - name: admin
    match: id = 1
    tables:
      - name: users

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.