density-dependent editing zoom limit

Currently, editing is allowed only at z16 or greater.  In some areas where there is less data, I think z14 is OK (tested in Chrome).

Ugh looks like we have to actually solve this now.   At a minimum we'll need to handle a response with "400 Bad Request" and the response error message like "You requested too many nodes (limit is 50000)"
iD does not work when loading this very densely mapped place (lots of indoor mapping happening here):
https://www.openstreetmap.org/edit#map=19/41.89317/12.49330

The node density script I made at the time has been superseded by @joto's https://github.com/osmcode/osmium-contrib/tree/master/node_density program. My plan at the time was to regularly generate an analysis and modify the download limit on the OSM server to "preflight" the request against the node density "database" (essentially an 8bpp PNG file with 1 pixel per z14 tile or so), dropping the 0.25°² limit and replacing it with a true estimate of how many nodes there will be.
However that was intended to help editing in sparsely mapped areas, allowing you to download larger areas at once; what we have here is a problem with densely mapped areas. The pre-flighting would at best enable a fast-fail on the server but not help the client much.
Perhaps the node density lookup could be made accessible to the editor directly so the editor would know what's possible, and force the user to zoom in when in a dense area...

This issue seems to be about allowing users to edit at a zoom that better suited what they were doing, not changing how data is downloaded.

This issue seems to be about allowing users to edit at a zoom that better suited what they were doing, not changing how data is downloaded.

Yes, this is different but related.  This problem of requesting data from the API needs to be solved first anyway (see @jfirebaugh's comment above about "iD currently downloads data tiles at a fixed zoom level (z16). Ideally, this would vary depending on density. But how do you know if an area is dense or not without downloading something first?")
Maybe instead of hardcoding a tilezoom, we could determine a reasonable starting tilezoom based on latitude, then adjust if the server responds with "400 Bad Request"?  Thankfully we get back that result really quickly and could easily break a dense tile up into z+1 quads and re-request those instead.
We need to store tile statistics somewhere too and make it available to other parts of the iD code - this relates to https://github.com/openstreetmap/iD/issues/2248#issuecomment-282740296 too, which has been on my mind.

CC @kriscarle for visibility - relates to maphubs/moabi work.

Would the data from http://osm-analytics.org/ help with this, or making our own density tiles based on the QA tiles?  I guess if a HOT activation significantly increased the density of an area overnight it might cause a problem? Maybe a safety check that disables it somehow in that scenario, if the node counts downloaded are significantly higher than what is in the density tiles?
Another option could be to combine it with a config in the url API, off by default, so only projects that are doing rural mapping enable it, possibly by adding a toggle to tasking manager? HOT building mapping tasks wouldn't need this anyway? For those of us mapping roads/rivers in remote parts of the rainforest, the density is not going to change that rapidly, just don't want to melt the servers if the user happens upon a denser area.

Would the data from http://osm-analytics.org/ help with this, or making our own density tiles based on the QA tiles?

Making density tiles isn't really a problem, there's half a dozen ways to do it. https://github.com/osmcode/osmium-contrib/tree/master/node_density is probably the best. It's what to do with that data that's in question.

The discussion above seems to around whether to have iD

react to dense data after it encounters it and adapt, or
making a density dataset available to iD so it can determine when it is safe to go above z16

Option 1 sounds like a complicated build? Plus it still means downloading the data on the initial load before figuring out how to optimize later requests, and that could still put a strain on both server and client?
For option 2 it was mentioned a couple times that "maybe we should make an API" followed by months/years of silence, because that is a hard thing to do :) OSM Analytics is basically that API now? The buildings vector tiles are a density aggregate down to z12 (before switching to actual buildings). What I'm proposing is that iD download the z12 tile from OSM Analytics, count the density pixels in the tile, if they are greater than an agreed upon threshold, then iD sticks with the z16 limit, otherwise we can allow z14, or possibly even a sliding scale up to z12.
@tyrasd it would be awesome to get your thoughts on this, just realized were are on this thread back in 2013 :)
The next issue to me seems to be dealing with the inevitable lag between any density dataset and OSM. That could be an issue when an area changes rapidly and there are hundreds of people loading that area in iD, like during HOT activations mapping buildings in urban areas that were previously unmapped. To handle that, we can allow this feature to be toggled off (or until we are happy with it, off by default and toggled on), and HOT can decide if they want it on by default in the links generated from tasking manager.

Glad to see the activity on this ticket! I feel like if we can store quadkey densities as part of the OSM API that's probably the best and let other editors also use this information. OSM Analytics could give this as a byproduct, but is dependent on QA tiles but I somehow think this should in API core.
Do we know what densities translate to what zoom level? It sounds like to me that even z16 is soon going to be difficult.


Rename uin8_t => bool

Hello @simondlevy ,
could you provide more information about this issue?
Best Regards,
Carlo

Closed as no feedback and issue malformed.


Upgraded and having error with some relation

ERROR: relation "motor_api_configs" does not exist LINE 8: WHERE a.attrelid = '"motor_api_configs"'::regclass ^

@silva96 please check:
https://github.com/motor-admin/motor-admin-rails/issues/63

Oh im sorry i didn't see that, shame on me!
Thanks!


Added variant for Gradle 7

Then I will add for Kotlin

The implementation format is valid for at least Gradle 5 and up. Maybe just replace with the new format?

Hm.. You can also rewrite to a new format

Just second


Update Composer dependencies (2021-07-04-00-04)

Visual regression test passed!
View the visual regression test report


Executing on Heroku

Hello - Your git is a major time saver from using iText. I am using it to fill PDF forms with data from a document in mongo db. However, I am running into issues when I deploy this to Heroku.
I am able to execute this hardcoded command and create the filled pdf on heroku bash.
'pdftk in.pdf fill_form data145.fdf output filledout.pdf flatten'
But it does not work within my node api. I get a 'Process exited with status 143'.
I wanted to see if you were able to deploy your app to heroku and create pdfs.
Thanks a bunch

Does it work locally for you?

I should have closed this. This works in heroku for us now. One of my developers fixed it. The PATH variable was incorrectly set.

@infos how did you get this to run on heroku? I've added this buildpack to heroku: https://github.com/shake-apps/heroku-buildpack-pdftk and I'm not getting any errors when running pdfFiller.fillForm, however no file gets created after invoking that method. This works perfectly on my local machine, so there must be something that I missed.


HDDS-9151. Close EC Pipeline when container transitions to closing

What changes were proposed in this pull request?
In testing we have found an issues in the ECWritableContainerProvider.
For EC a pipeline is used for only one container, when the container gets closed, the pipeline also gets closed. At the moment, the only place in the code which closes the EC piplines which no longer have an open container is inside the ECWritableContainerProvider. It first gets the list of open piplines and enforces the pipeline limit, then for all open pipelines, it tries top find one the client can use.
If the client has had problems writing to the pipelines (eg it was given a container/pipeline and then the write failed as the container was closed on the DN), the pipelines get added to the exclude list. Then we can get into a situation where many pipelines need to be closed on the write path, slowing down block allocation.
Ideally, when a container transitions to CLOSING in SCM, if the container is an EC container, we should also close the associated pipeline to avoid it counting toward the limit and to avoid needing to close it during the write (block allocation) path.
This could be achieved relatively simply inside the PipelineManagerImpl.removeContainersFromPipeline() method which is called as soon as the container transitions to CLOSING via ContainerStateManagerImpl.updateContainerState() when it executes the containerStateChangeActions. Wrapping the container close and pipeline close in a lock inside PipelineManagerImpl ensure we have a consistent "ec container close" flow and it should avoid the ECWritableContainerProvider needing to close the pipelines internally. However we can leave that code in place in ECWritableContainerProvider incase some pipelines slip through somehow.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9151
How was this patch tested?
New unit test added to ensure EC pipelines get closed and Ratis do not.

Thanks @sodonnel work on this, I think it's a good idea, this can close the pipeline in advance, so that the pipeline in the open state can be used to apply the ec block after getting it in advance.

The javadoc suggests that we also need to catch PipelineNotFoundException in PipelineManagerImpl#removeContainerFromPipeline when getting a pipeline:
Pipeline pipeline = stateManager.getPipeline(pipelineID);

I think you are correct. It would be safer to catch the PNF exception from the getPipeline call too. I have updated it.
I didn't even notice the replicate annotation, as it was annotating the interface rather than the implementation, and I did not check the interface!


Create iai-togo.txt

Adding my Educationnal university for make all students and teachers take advantage on using Jetbrains IDEs.
Website
School Mail Adress
Personnal Mail Adress
Thank you.

@bienvenu-elisis Pull request merged. Thank you!


Return source from condition

sample({
  source: $store,
  clock: event,
  target: condition({
    if: (value) => value > 1,
    then: firstBranch,
    else: anotherBranch,
  })
})

Already implemented in #17


Hamster Profile Avatar

When i clicked on top players profile picture i found a link.
i hope it will help developers to identify how they change profile picture.

https://www.youtube.com/watch?v=p4aPSN_mVZI

https://www.youtube.com/watch?v=p4aPSN_mVZI

It's not working dear

It probably takes some time to update.
On Fri, 20 Sept 2024, 14:33 Ashwin paul, @.***> wrote:

https://www.youtube.com/watch?v=p4aPSN_mVZI
It's not working dear
—
Reply to this email directly, view it on GitHub
https://github.com/masterking32/MasterHamsterKombatBot/issues/482#issuecomment-2363751211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHJHWLYRTLNKGVRRJ6AHX73ZXQP4NAVCNFSM6AAAAABORQTUR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTG42TCMRRGE
.
You are receiving this because you commented.Message ID:
@.***>


Improve bqplot classic theme

Bqplot 0.11 had two different skins
A dark theme and a light theme.
The original light theme is similar to ggplot, with a light gray background and white gridlines.

The original dark theme is a more "classic" wireframe style.

Bqplot 0.12 introduced the coloring based on JupyterLab colors
Since 0.12, bqplot makes use of the JupyterLab CSS variables for the coloring of the plot background to match the environment.
However, we kept two options for the bqplot styling: the "gg" style, similar to ggplot, and the "classic" style with the wireframe look. Roughly, "gg" looks like the original light theme in the with the JupyterLab light theme, and "classic" looks like the original dark theme in the JupyterLab dark theme
0.12 light "gg"

0.12 light "classic"

0.12 dark "gg"

0.12 dark "classic"

In this PR
In this PR, I rework the colors to be closer to the original, especially in the dark theme. We still use JupyterLab variables.
New light "gg"

New light "classic"

New dark "gg"

New dark "classic"

I think these changes improve the visual look of the dark theme, especially in the "gg" theme. In the classic mode, the axis lines are more highlighted and grid lines a bit less visible

@kaiayoung for awareness!

@SylvainCorlay many thanks for doing this.
The removal of plot background is a huge plus especially when you stack multiple charts using hbox/vbox.
I am proposing a few changes to gridlines and ticks as follows
Current dark (as done in this PR):

What I am proposing:

preferred gridlines stroke: #3b3b3b
preferred tick text fill: #b3b3b3
Please see if you can as close to this from the existing colors you have. Happy to chat further.

classic looks good btw.

New version with dimmer grid lines and axes:
light "gg"

light "classic"

dark "gg"

dark "classic"

@ChakriCherukuri I think this is pretty close to the sceenshot you shared (except for the notebook background which is part of JupyterLab). I will be merging this when all is green.

LGTM


node_modules not copied over

I am trying to do React Native development on a symlinked node module (npm link). Apparently this is problematic, and I was routed here from https://github.com/facebook/react-native/issues/4968.
Basically, what I'm trying to do is sync a folder that contains my node module, containing all it's node_modules into my React Native folder.
So I set it up like this (for this example renamed my module to the-module)
wml add ../the-module ./node_modules/the-module
Now, the folder at ../the-module is an NPM module and has it's own node_modules folder.
The problem is that the node_modules from ../the-module aren't synced over to ./node_modules/the-module/node_modules.
I am selecting No when the wml add prompt asks me whether I want to ignore the node_modules from the source location.

Same here

Hi, sorry for taking so long, I forgot to turn on notifications on this repo 🙄
I can't reproduce what you're describing, can you please post the contents of ../the-module/.watchmanconfig?
Thanks!

I had this same problem and fixed it like this:
1.wml stop
2. Make sure node_modules are not in ../the-module/.watchmanconfig (I think wml may create this if you ever yes to the wml add prompt)
3. Stop watchman (watchman shutdown-server)
4. wml start
Then you can enjoy your node_modules again.

@hendrikswan have you figured out how to solve this issue? im still stuck... 😢😒😢

I got the same problem too.


autoFocus doesn't work

autoFocus doesn't work
 <TrixEditor
 autoFocus={true}
 onChange={(html, text) => setEdit(html)}
 onEditorReady={(e) =>console.log("editor ready ", e)}
 placeholder="Type your comment here..."
 />

fixed in v0.8.0.


React Native Flipper performance plugin

📝 Description
Implementation of React Native Flipper performance monitor
❓ Context

Impacted projects: `` N/A
Linked resource(s): `` https://github.com/bamlab/react-native-flipper-performance-monitor#install-for-non-expo-projects

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo
https://user-images.githubusercontent.com/16896559/207097751-1c614659-3c7a-42b7-9e67-f81febb750d7.mov
🚀 Expectations to reach
Must work on Android and iOS

✅  Tested on Android & iOS simulators

You just need to rebase and do a pnpm i to resolve the conflicts on the pnpm-lock.yaml


fix(cloud_firestore): remove listener is available

Description
Ensures that listeners are removed only if they're available.
Related Issues
Fixes #3354
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Thanks for this!
Any idea when this will make it into a release?

Thanks for this fix, same crashes here
When can this change make it into a release?


URL is being passed to the vector layer without subdomain template

This is one of the reasons why tiles are being downloaded slowly.
Ref. https://github.com/CartoDB/d3.cartodb/issues/103
cc @alonsogarciapablo @rochoa

The URLs of the tiles we're generating are similar to the following:
https://cartocdn-ashbu.global.ssl.fastly.net/editor3/api/v1/map/004cac3905280a4a17824121ccb4d089:1455041643866.75/0/3/2/2.geojson

On one hand, it looks like CartoDB.js is not using subdomains in the tileJSON URLs when running trough HTTP. I'm not sure why...
Also, we're serving these tiles through the CDN (cartocdn-ashbu.global.ssl.fastly.net)...
What'd be the URL template for this case? @rochoa @javisantana

There has been no activity on this issue for several months and that's why we are closing it (automatically). If you think this still needs to be addressed please reopen it. The good news is that we are now working on the next version of CARTO.js, stay tuned.


Remove joda-time and use the Java 8 date time APIs instead

This will further reduce the dependency footprint and artifact size

Hey, You are still working on this repo? Are you accepting PRs ?

Yes to both Ishan!
On Sat, Dec 19, 2020 at 8:29 PM ishanarora04<EMAIL_ADDRESS>wrote:

Hey, You are still working on this repo? Are you accepting PRs ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kislayverma/Rulette/issues/66#issuecomment-748485381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPYCN6CWQIKAB2XW7IESPTSVS5WPANCNFSM4MT6HLOA
.


Druid 31.0.0 release notes

Release and upgrade notes for Druid 31.0.0
This PR has:

[x] been self-reviewed.

It doesn't look like it's in the milestone. Should I add it to the milestone too?


nixos-option broken and 18.09 beta

Issue description
cc @samueldr @vcunat is this a release blocker?
$ nixos-option networking.useDHCP

This attribute set contains:
error:
syntax
error,
unexpected
:,
expecting

Steps to reproduce
Run nixos-option on current 18.09 beta (7f70ebf30b8). Unstable seems to be affected too.
Technical details
Seems to be a fairly simple case of nixos-option capturing all the output of nix-instantiate, including the missing-NIX_PATH-entry warning. I'll have a look at fixing this.

system: "x86_64-linux"
host os: Linux 4.14.71, NixOS, 18.09beta696.7f70ebf30b8 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.1
channels(root): "nixos-18.09beta696.7f70ebf30b8, nixos-unstable-19.03pre154105.e1605d9b21b"
channels(steam): ""
channels(linus): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

I suppose this is just a consequence of #40165, combined with poor handling of the warning by nixos-option.

I expect this is fixed since ee582a32d25 then.  I'm unable to reproduce it on 18.09beta676.66fd61163a8, even with sudo etc.

@vcunat no, it's not — this still happens when it's expanded correctly if ~/.nix-defexpr/channels doesn't exist.

@lheckemann: can you also re-test this patch: de93b32?

Yep that fixes it!


Add support for parsing ToB interface for PBs

Currently, Chat Commands will parse the Adventurer's Log to obtain new PBs if the client missed the message for whatever reason. This covers most activities in the game except for ToB, which for whatever reason has it by the entrance to the raid.

This is problematic since ToB is a big part of Trailblazer for some users, and since profiles were only recently added, their main game PBs have been overwritten with the insane times possible in the league.

I think we should just have the plugin able to capture pbs from getting a tob kill:


upper bound

Current coverage is 94.90%

Merging #1145 into master will not affect coverage as of 8c1d737

@@            master   #1145   diff @@
======================================
  Files           79      79
  Stmts         9178    9178
  Branches      1707    1707
  Methods          0       0
======================================
  Hit           8710    8710
  Partial         91      91
  Missed         377     377

Review entire Coverage Diff as of 8c1d737

Powered by Codecov. Updated on successful CI builds.


Create main.yml

Creating workflow to execute Eslint, Prettier and type checking when a PR is raised.

@ShreyaPrasad1209 So the version change fixed the dep's error but there is no script for styles in package-lock.json so the this command is failing


Refactor logic from CLI to separate clases, Fix #37

Checklist:

[x] Added tests for changed code where applicable.
[ ] Documentation reflects the changes where applicable.
[x] Updated the CHANGELOG.md file with your changes.
[x] My PR is ready to review.

Resolves: #38
Resolves: #37
Description of the changes being introduced by the pull request:

Codecov Report

Merging #40 (e70372f) into master (fe2ca54) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   99.18%   99.19%
=======================================
  Files          15       16    +1
  Lines         369      371    +2
=======================================
+ Hits          366      368    +2
  Misses          3        3

Impacted Files
Coverage Δ

fastapi_mvc/cli/cli.py
100.00% <100.00%> (ø)

fastapi_mvc/cli/commands/new.py
100.00% <100.00%> (ø)

fastapi_mvc/cli/commands/run.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Access issues

I tried to access the lectures for this module but the page was not found.

@BINGZHEZHOU Some links in README are broken. Fixed. Please check them again.

Dear Dr.Wei
Thanks for your quick response. I have double-checked the repo again, it
seems that the linked page is still not available. When I clicked the link
provided in the README file, the page was not found again. Two images
attached to the e-mail present the issues that I have met.
Kind regards
BINGZHE
hengxin(Hengfeng Wei) @.***> 于2021年6月10日周四 下午4:05写道：

@BINGZHEZHOU https://github.com/BINGZHEZHOU Some links in README are
broken. Fixed. Please check them again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/courses-at-nju-by-hfwei/compilers-lectures/issues/4#issuecomment-858407052,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALFJGBTDEQ2ODVQQDKC5ZPLTSBW5BANCNFSM46L4FNEQ
.

@BINGZHEZHOU It works for me. (Refresh your pages?) I do not receive your email. Another way is to clone or download this repo.

Thanks, I will try again.


Crash after cancel

Bot write "Canceled" and crash with error
Unhandled Exception: System.IO.IOException: The process cannot access the file 'test100Mb.db' because it is being used by another process.    at System.IO.FileSystem.RemoveDirectoryRecursive(String fullPath, WIN32_FIND_DATA& findData, Boolean topLevel)    at System.IO.FileSystem.RemoveDirectory(String fullPath, Boolean recursive)    at System.IO.DirectoryInfo.Delete(Boolean recursive)    at Downloader_Bot.Program.Bot_OnMessage(Object sender, MessageEventArgs e) in C:\Users\Иван\Download-bot\Downloader Bot\Program.cs:line 197    at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state) --- End of stack trace from previous location where exception was thrown ---    at System.Threading.ThreadPoolWorkQueue.Dispatch()

If you still face this issue please tell me to reopen this issue.


Do not fail when rrule is not valid

Using invalid RRULE results in undefined local variable or method `rrule' for #<Icalendar::Recurrence::Schedule..>
Will provide a failing test later.

Thanks for the report @espen. Failing tests would be great :ok_hand:.

https://github.com/espen/icalendar-recurrence/tree/invalid_rrule


feat: add Task interface & its common implementations

Add the sghi.task.Task interface together with its common implementations such as:

sghi.task.Chain
sghi.task.Consume
sghi.task.Pipe

Also included is a special kind of Task- sghi.task.ConcurrentExecutor, used to execute other tasks that share an input concurrently.

:tada: This PR is included in version 1.0.0-dev.1 :tada:
The release is available on:

v1.0.0-dev.1
GitHub release

Your semantic-release bot :package::rocket:

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

v1.0.0
GitHub release

Your semantic-release bot :package::rocket:


Un-minimizing a window on some Linux distros opens a transparent window

Describe the bug
Un-minimizing a window on Ubuntu Budgie opens a transparent window until you adjust the size of the window.
To Reproduce
Steps to reproduce the behavior:

Open a new window
Minimize it
Un-minimize it
See that the window is transparent

Expected behavior
Un-minimizing a window should cause the window to re-paint
Screenshots

Desktop (please complete the following information):

OS: Ubuntu Budgie 23.10
Version: 0.5.3

Additional context
Add any other context about the problem here.

I am no longer seeing this, closing


fix: the URL passed with the --release option is not used

 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.


base64 wrong in PATH

correct path for base64
ldapsearch from oud but base64 from system

already fixed by first looking for base64
BASE64_BIN=$(find /usr/bin -name base64 -print) # executable for base64
BASE64_BIN=${BASE64_BIN:-"$(command -v -p base64)"} # executable for base64


\Zend\Feed\Writer\Feed output author into Rss feed seems incorrect

\Zend\Feed\Writer\Feed has author property on channel level. But it processes it incorrectly at least in case of Rss feed type. "" tag is pointed as invalid by w3org validator. Rss 2.0 doesn't contain author tag in channel. Zend uses Atom namespace so probably writer should produces "atom:author" tag rather than "".

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Create end-to-end test for Feast 0.3

Need to create an e2e test which tests

Registering feature set

Ingesting feature data

Retrieving feature data from online serving

Monitoring feature availability

Retrieving feature data from warehouse

Added here: https://github.com/gojek/feast/tree/0.3-dev/tests/e2e


Cannot find module 'zod/lib/src/parser'

Describe the bug
The latest resolvers package with zod gives a typescript error.
node_modules/@hookform/resolvers/zod/dist/types.d.ts:3:34 - error TS2307: Cannot find module 'zod/lib/src/parser' or its corresponding type declarations.

3 import type { ParseParams } from 'zod/lib/src/parser';

To Reproduce

Install latest version of @hookform/resolvers and zod in a TS project
Run tsc to get the error. (not skipLibCheck should be false)

Expected behavior
No ts error.
Additional context
ParseParams can be now imported directly from zod root to avoid any potential directory structure changes in the future

Thank you for reporting, someone already reported it: https://github.com/react-hook-form/resolvers/issues/154
We currently support Zod v1, here is the PR for Zod v3: #200

oh, I thought that one from April was already resolved.
Do you need any help with the migration?
I found also that it's not very clear that Zod v3 is not yet supported.

Fixed with 2.8.0!


Problem with flask db upgrade. Altering table 'history' executes with error

PDA version
0.4.0
Python version
3.10
Steps to Reproduce
On new version flask db upgrade fails with error.
I've executed following commands
git clone https://github.com/PowerDNS-Admin/PowerDNS-Admin.git /opt/web/powerdns-admin
cd /opt/web/powerdns-admin/
python3 -mvenv ./venv
source ./venv/bin/activate
pip install --upgrade pip
pip install -r requirements.txt
cp /opt/web/powerdns-admin/configs/development.py /opt/web/powerdns-admin/configs/production.py
python3 -c "import os; print(os.urandom(16))"
(made some db-connection config over there corresponding to guide)
vim /opt/web/powerdns-admin/configs/production.py
export FLASK_CONF=../configs/production.py
export FLASK_APP=powerdnsadmin/init.py
flask db upgrade
Expected Behavior
INFO  [alembic.runtime.migration] Context impl MySQLImpl.
INFO  [alembic.runtime.migration] Will assume non-transactional DDL.
INFO  [alembic.runtime.migration] Running upgrade  -> 787bdba9e147, Init DB
INFO  [alembic.runtime.migration] Running upgrade 787bdba9e147 -> 59729e468045, Add view column to setting table
INFO  [alembic.runtime.migration] Running upgrade 59729e468045 -> 1274ed462010, Change setting.value data type
INFO  [alembic.runtime.migration] Running upgrade 1274ed462010 -> 4a666113c7bb, Adding Operator Role
INFO  [alembic.runtime.migration] Running upgrade 4a666113c7bb -> 31a4ed468b18, Remove all setting in the DB
INFO  [alembic.runtime.migration] Running upgrade 31a4ed468b18 -><PHONE_NUMBER>77, Upgrade DB Schema
INFO  [alembic.runtime.migration] Running upgrade<PHONE_NUMBER>77 -> 0fb6d23a4863, Remove user avatar
INFO  [alembic.runtime.migration] Running upgrade 0fb6d23a4863 -> 856bb94b7040, Add comment column in domain template record table
INFO  [alembic.runtime.migration] Running upgrade 856bb94b7040 -> b0fea72a3f20, Update domain serial columns type
INFO  [alembic.runtime.migration] Running upgrade b0fea72a3f20 -> 3f76448bb6de, Add user.confirmed column
INFO  [alembic.runtime.migration] Running upgrade 3f76448bb6de -> 0d3d93f1c2e0, Add domain_id to history table
Observed Behavior
In the process of the installation on step when i execute command 'flask db upgrade' it finishes with error
INFO  [alembic.runtime.migration] Context impl MySQLImpl.
INFO  [alembic.runtime.migration] Will assume non-transactional DDL.
INFO  [alembic.runtime.migration] Running upgrade  -> 787bdba9e147, Init DB
INFO  [alembic.runtime.migration] Running upgrade 787bdba9e147 -> 59729e468045, Add view column to setting table
INFO  [alembic.runtime.migration] Running upgrade 59729e468045 -> 1274ed462010, Change setting.value data type
INFO  [alembic.runtime.migration] Running upgrade 1274ed462010 -> 4a666113c7bb, Adding Operator Role
INFO  [alembic.runtime.migration] Running upgrade 4a666113c7bb -> 31a4ed468b18, Remove all setting in the DB
INFO  [alembic.runtime.migration] Running upgrade 31a4ed468b18 -><PHONE_NUMBER>77, Upgrade DB Schema
INFO  [alembic.runtime.migration] Running upgrade<PHONE_NUMBER>77 -> 0fb6d23a4863, Remove user avatar
INFO  [alembic.runtime.migration] Running upgrade 0fb6d23a4863 -> 856bb94b7040, Add comment column in domain template record table
INFO  [alembic.runtime.migration] Running upgrade 856bb94b7040 -> b0fea72a3f20, Update domain serial columns type
INFO  [alembic.runtime.migration] Running upgrade b0fea72a3f20 -> 3f76448bb6de, Add user.confirmed column
ERROR [root] Error: All MySQL CHANGE/MODIFY COLUMN operations require the existing type.
While I do the same steps on 0.3.0 - the db upgrade passes correctly
INFO  [alembic.runtime.migration] Context impl MySQLImpl.
INFO  [alembic.runtime.migration] Will assume non-transactional DDL.
INFO  [alembic.runtime.migration] Running upgrade  -> 787bdba9e147, Init DB
INFO  [alembic.runtime.migration] Running upgrade 787bdba9e147 -> 59729e468045, Add view column to setting table
INFO  [alembic.runtime.migration] Running upgrade 59729e468045 -> 1274ed462010, Change setting.value data type
INFO  [alembic.runtime.migration] Running upgrade 1274ed462010 -> 4a666113c7bb, Adding Operator Role
INFO  [alembic.runtime.migration] Running upgrade 4a666113c7bb -> 31a4ed468b18, Remove all setting in the DB
INFO  [alembic.runtime.migration] Running upgrade 31a4ed468b18 -><PHONE_NUMBER>77, Upgrade DB Schema
INFO  [alembic.runtime.migration] Running upgrade<PHONE_NUMBER>77 -> 0fb6d23a4863, Remove user avatar
INFO  [alembic.runtime.migration] Running upgrade 0fb6d23a4863 -> 856bb94b7040, Add comment column in domain template record table
INFO  [alembic.runtime.migration] Running upgrade 856bb94b7040 -> b0fea72a3f20, Update domain serial columns type
INFO  [alembic.runtime.migration] Running upgrade b0fea72a3f20 -> 3f76448bb6de, Add user.confirmed column
INFO  [alembic.runtime.migration] Running upgrade 3f76448bb6de -> 0d3d93f1c2e0, Add domain_id to history table
INFO  [alembic.runtime.migration] Running upgrade 0d3d93f1c2e0 -> 0967658d9c0d, add apikey account mapping table
INFO  [alembic.runtime.migration] Running upgrade 0967658d9c0d -> fbc7cf864b24, update history detail quotes
INFO  [alembic.runtime.migration] Running upgrade fbc7cf864b24 -> 6ea7dc05f496, Fix typo in history detail

@roamingengineer311 please read the contribution guide. This issue has already been reported multiple times and has been corrected in the dev branch. Therefore, I'm rejecting this report on the basis of it being a duplicate and already solved.


State TTL  cleaner

At the moment we're only expiring entries that are touched but every state store has to also be continuously scanned for expired entries. This should not be very intensive but instead of creating a thread and say iterating though all entries with some time to sleep it should reuse akka scheduler every x seconds during which y records are checked.
First of all we'll need to ensure (in the MemStore contract) that all implementations provide iterator that is weakly consistent, as in to interfere with the performance of foreground operations.
However, we already have ttl support in MemStoreRocksDb and MemStoreMapDb. They do not support injecting custom timestamps in their structures but it may be enough and we can provide another MemStore implementation that does support controlling the record timestamp. So before embarking on this reconsider the usecase.

ttl cleaning delegated to each storage and memstore impleentations


Update the translation widget

Hi,
i have the translation widget running and everything works fine so far.
I just want to know how to update it. Do i have to package it everytime i want to add a key with its translation to the widget?
Because when i change the es.js file on the webserver for example and i add a new key with its translation, the widget does not recognize the new key.
When i change it and i package it with npm and register it with the portal again it works.
But this cant be the right way?
Thanks in advance for your help.

If you are using the widget in the portal, this may be a cache issue. In development, can you try this:

In the network panel, check the Disable cache
In the application panel, select Service Workers, check the Bypass for network


chore(localize): updated translation from Crowdin

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


Enable !users command and add a table formatter

Listen to wildcarded events, currently handle RPL_NAMREPLY.
Add a formatter to present nick lists from IRC as a table.

Thanks for your work. I'd like to merge it, but I think the config fits better under [mattermost] instead of [irc]
(Also the case with your previous PR)

@42wim sure, if that's all, it should be quickly done. :smile:

Ok, have been testing it now ;-)
The header also shows NicksPerRow columns

Is prettier if that's not the case ;-)

Closed in favor of #14.


Fixed score submission not disabling.

Moved the insert to before the second if block, and removed the empty check to make sure score submission works properly.
I've done some testing and this works adequately, although I should probably test with multiple mods just to be sure.

Testing with another mod appears to work as well.


UI doesn't show all training data

I've trained on several hundred items, but the UI only shows the first 25. Confirmed that even after I add another one in the UI, it says there are 25, and if I delete one - still stays 25 :).

This is vanna-flask, specifically.

Ah, here's the offending code in app.py
"df": df.head(25).to_json(orient='records'),

Thanks -- this is a duplicate of this issue:
https://github.com/vanna-ai/vanna/issues/182


Master

update README
fix return type (float -> long)

This is great! Thanks for the PR. What kind of performance do you get with numpy?

I'll test this and probably update the PyPI package tomorrow.

Not that much but as I have many rows in my pandas data frame structure and that I perform many requests against it
even a small improvement is good to take :D
(around 15% speed increase)
Best regards
Olivier

Le 19 mars 2015 à 06:33, Geoffrey Fairchild<EMAIL_ADDRESS>a écrit :
This is great! Thanks for the PR. What kind of performance do you get with numpy?
—
Reply to this email directly or view it on GitHub https://github.com/gfairchild/pyxDamerauLevenshtein/pull/3#issuecomment-83337425.

Oliver, have you seen #4? Have you seen this issue?

Yes, the dtype specified is incorrect, he should use dtype='S' instead of 'U'
(as specified in the documentation) :D

yep, I answered in the issue tracker
dtype=’S’ instead of ‘U'
Best regards
Olivier

Le 25 mars 2015 à 17:23, Geoffrey Fairchild<EMAIL_ADDRESS>a écrit :
Oliver, have you seen #4 https://github.com/gfairchild/pyxDamerauLevenshtein/issues/4? Have you seen this issue?
—
Reply to this email directly or view it on GitHub https://github.com/gfairchild/pyxDamerauLevenshtein/pull/3#issuecomment-86103569.

Hey, guys, thanks a lot for this library and for adding NumPy capability to it! :-)
I could certainly put it to good use.
It's probably not worth opening another ticket on but I wanted to clarify if the algorithm used by the library calculates the true Damerau–Levenshtein distance or a more restricted version that the Wikipedia article calls the Optimal string alignment distance?
Judging from the example from Wikipedia article it might be the latter:
>>> from pyxdameraulevenshtein import damerau_levenshtein_distance
>>> damerau_levenshtein_distance("CA", "ABC")
3

I just pushed this to PyPI. Thanks!


Hybrid app built with ng-slider doen't render properly

The same web site works correctly on PC web browser, but on Phone it does not ,

hi, can you check current version and give some screenshots if still issues, thx

no details enough


CASM-3727: Using a newer version of cray-nls: 0.9.17.

Summary and Scope
Using a newer version of cray-nls: 0.9.17.
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Resolves [CASM-3727]
Merge after https://github.com/Cray-HPE/cray-nls/pull/145

Testing
List the environments in which these changes were tested.
Tested on:

Not sure how to test this.

Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)? No.
Were continuous integration tests run? If not, why? No
Was upgrade tested? If not, why? No
Was downgrade tested? If not, why? No
Were new tests (or test issues/Jiras) created for this change? No

Risks and Mitigations
Low
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[x] HPC Product Announcement prepared, if applicable

I'm going to suggest that you bump the helm chart version all the way to 1.4.49, because there are already 1.4.47 and 1.4.48 helm charts in CSM artifactory: https://artifactory.algol60.net/ui/native/csm-helm-charts/stable/cray-nls/
Presumably built from the cray-nls repo before we knew we were supposed to update the chart here.

Done. Chart version is 1.4.49.


feat(frontend): Add License URL in menu

Motivation
We include the License agreement link in the menu as per design. To do that, we extract the License Agreement URL in a separate link.

[OISY-122]


Remove Underline from Filter menu items

As Charlie pointed out in a previous Thursday meeting, the underlined options in the filter menu option make them look like links, when really the user should just be checking the boxes. Could we remove the underline from those? Only if it's not too hard - this is low priority.

Oops, typo in the commit message.
Here's the commit that fixed it: https://github.com/PEDSnet/PEDSpace/commit/224d49c18335fbf6cb8640c2c12ff010e9805a21


⚠ Bofrost.es has degraded performance

In 968d05f, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1534 ms

Resolved: Bofrost.es performance has improved in 695f6ec after 9 minutes.


Prioritization of the leaf crates stabilization and dependency graph

Summary
Let's focus on stabilizing the most important crates: units, primitives, keys, address, in this order.
Motivation
The above mentioned crates provide fundamental types used in inter-crate communication. By being unstable a breaking change in our crate is a much bigger annoyance then it'd be otherwise: the whole ecosystem breaks and everyone is waiting on someone else to upgrade. This can take many months. Therefore it should be our priority to get them ready ASAP.
Further, these crates have most of the API in the state where I'm confident could be stabilized. Being used almost everywhere, their API is also battle-tested, so its stability is not just a theory.
Design constraints
A huge care needs to be taken when doing this as we could cause accidental breakage or paint ourselves into a corner unable to develop the downstream crates. I've identified these constraints:

No crate may publicly depend on an unstable crate (obviously)
To make development possible we need a bitcoin-unstable cfg; as discussed it needs to be a cfg, not feature
The downstream crates should not require the bitcoin-unstable cfg to implement any of the existing functionality (would be way too annoying otherwise)
Do not duplicate non-trivial code, doing so often leads to horrible problems
Prefer not to create more micro-crates if possible (containing one trait or so)

Challenges
The constraints lead to some difficulties around consensus encoding and hashes but I think I figured them out.
One issue is that Transaction needs serialization code to implement compute_txid/compute_wtxid however the push_decode part is highly unstable and we don't want to stabilize the existing traits either. This can be solved by putting AsRawTransaction similar to AsTransaction to the internals crate which also implements transaction encoding over this trait. But rather than implementing AsRawTransaction for Transaction it'll be implemented over a private wrapper.
Another issue is that the ideal Hash trait definition requires advanced Rust features and we don't want to wait for super-high MSRV. I think that we can actually get rid of the trait and just use inherent items. The API will be less annoying anyway. This means we need to stabilize the output of the hash_newtype! macro. I think we can realistically do that once it supports inherent items.
Public keys need to be parsed for that we need secp256k1. We don't need it to be stable if we don't expose the internal key but it must have non-context API for critical operations.
Proposed dependency graph
(Only parts relevant to this issue.)

Everything depends on internals (non-optionally)
internals optionally depends on push_decode (activated for primitives and consensus-encoding)
primitives depend on units and hashes
consensus_encoding depends on primitives
address depends on keys and optionally on primitives

Agree with the prioritization and high-level observations. Can you say more about:

What AsRawTransaction would do? (Maybe this is not the issue for it.)
What features we'd want from Hash trait ideally. Somewhere I recall we came to a rough consensus that our 1.0 MSRV would be 1.6x or thereabouts.

What AsRawTransaction would do?

Effectively a copy of AsTransaction from #2347 with only modification being use of raw types (integers, byte slices) to resolve the dependency loop. It'd be considered an implementation detail and only stay there as long as other crates are unstable. Once stable, the implementation would change and the trait would get removed.

What features we'd want from Hash trait ideally. Somewhere I recall we came to a rough consensus that our 1.0 MSRV would be 1.6x or thereabouts.

Allow writing fn to_byte_array(self) -> [u8; Self::LEN], though IIRC it may require the feature that is not even in stable and is somewhat broken in nightly too. (I might be confusing this with something else.) But from what I can observe so far: both GAT and weak dependencies will be a must when fully stable, so 1.65 IIRC. But the leaf crates don't need this. Actually that's what I wanted to mention above and forgot.

I think we should try the to_byte_array thing. It doesn't look like it needs GATs, just regular ATs, which have been around for a long time.
The AsRawTransaction thing makes sense, thanks. Though I wonder if we want to make this a permanent part of the API (and have a corresponding Raw type for some other types, notably Block and BlockHeader). It seems like it may be useful in some obscure low-level usecases.
OTOH it would be easy to add (or expose) post-1.0 so this isn't a decision we need to make now.

I don't see when it'd be useful. Script is already freely convertible, so only Witness is a bit obscure type. We could do something about that too. I was already thinking of having unsized Witness for zero-copy deserialization however it'd loose the index which is non issue when serializing but can be a problem when accessing.


Evaluate potential for animated tracks

The idea would be to stream GeoDataFrames to track object movement, similar to the examples in https://hvplot.holoviz.org/user_guide/Streaming.html
Feature requests in the corresponding libraries have been filed at:

https://github.com/python-streamz/streamz/issues/325
https://github.com/holoviz/hvplot/issues/436

https://github.com/anitagraser/movingpandas-examples/blob/main/3-tech-demos/folium-timestamped-geojson.ipynb


Refresh on job page scrolls view away

If you're looking down the page on the job page and it refreshes it scrolls back up to the top so you have to move the page back to where you were looking before.

I want to add that with a lot of jobs this becomes really annoying - at this point I can't tell what the status is of jobs I'm trying to check since by the time I find them the page scrolls back to the top .

I agree. It always frustrates me!

On Nov 25, 2015, at 9:04 AM, olsonanl<EMAIL_ADDRESS>wrote:
I want to add that with a lot of jobs this becomes really annoying - at this point I can't tell what the status is of jobs I'm trying to check since by the time I find them the page scrolls back to the top .
—
Reply to this email directly or view it on GitHub https://github.com/PATRIC3/patric3_website/issues/548#issuecomment-159634551.

Duplicate of another ticket, but this is fixed.


Add sequel adapter

Hey,
First commit allows not install mongo, memcached, redis, gems in local environment. All tests suite is spining up in docker containers and this could be reached with 1 command. Please check in README.md file.
Second commit add sequel adapter.

I'll split up these features.
What do you think about rubocop integration with Travis Ci?

I'll split up these features.

:cool:

What do you think about rubocop integration with Travis Ci?

Sounds like a good idea but not sure what all it would mean. Perhaps a separate PR that we can look at on its own?

@jnunemaker PR is updated please check.


Implement new Merkle Tree specification

Implement the sparse Merkle Tree and refactor the Mekrlee Tree code.
Use the new hash function (mimc7) in the Merklee Tree.
resolve #39

I've just force-pushed an amend with the changes we discussed offline.


Thousands ad thousands of defunct ssl client orphans...

:lady_beetle: Describe the bug

Not really sure TBH. I just noticed this when doing my monthy system updates.
:computer: Components impacted

Dockerized ntfy server. Running behind Traefik, which in turn is behind Cloudflare proxy (it was a bitch to get to run properly at first, but it's been running fine for ages now).
:bulb: Screenshots and/or logs

❯ dps
    CONTAINER ID   NAMES            SIZE                     STATUS
<PHONE_NUMBER>ab   container        0B (virtual 606MB)       Up 2 weeks
    83a65aa860c2   ntfy             0B (virtual 53.2MB)      Up 2 weeks (unhealthy)
<PHONE_NUMBER>ab   container        23.8kB (virtual 564MB)   Up 3 weeks
    629eb6192dc3   traefik          62.9kB (virtual 148MB)   Up 3 weeks
<PHONE_NUMBER>ab   container        0B (virtual 6.27MB)      Exited (2) 4 weeks ago
<PHONE_NUMBER>ab   container        78.8kB (virtual 765MB)   Up 3 weeks
<PHONE_NUMBER>ab   container        64.9kB (virtual 765MB)   Up 3 weeks
<PHONE_NUMBER>ab   container        2B (virtual 387MB)       Up 3 weeks
<PHONE_NUMBER>ab   container        145kB (virtual 491MB)    Up 2 days
<PHONE_NUMBER>ab   container        0B (virtual 38.7MB)      Up 3 weeks (healthy)
<PHONE_NUMBER>ab   container        0B (virtual 412MB)       Up 3 weeks (healthy)
<PHONE_NUMBER>ab   container        51.5MB (virtual 195MB)   Up 3 weeks
<PHONE_NUMBER>ab   container        22.2kB (virtual 203MB)   Up 3 weeks

❯ pstree -p
systemd(1)-+-ModemManager(544846)-+
           ├─containerd-shim(1526704)─┬─ntfy(1526724)─┬─ssl_client(1527025)
           │                          │               ├─ssl_client(1527129)
           │                          │               ├─ssl_client(1527555)
           │                          │               ├─ssl_client(1527620)
           │                          │               ├─ssl_client(1527734)
           │                          │               ├─ssl_client(1527802)
           │                          │               ├─ssl_client([...])
           │                          │               ├─ssl_client(2700900)
           │                          │               ├─{ntfy}(1526760)
           │                          │               ├─{ntfy}(1526761)
           │                          │               ├─{ntfy}(1526762)
           │                          │               ├─{ntfy}(1526763)
           │                          │               ├─{ntfy}(1526764)
           │                          │               ├─{ntfy}(1526776)
           │                          │               ├─{ntfy}(1527782)
           │                          │               └─{ntfy}(1559488)

:crystal_ball: Additional context

Not really sure what I'm expecting with this post, perhaps that someone else has run into a similar thing or perhaps just to log my issue if it happens time and time again.
But basically, I have a few NTFY servers running here and there, this has happened on one of them and I don't think there are any config differences between them all.
After the initial headache getting them to run properly behind cloudflare's proxied DNS and traefik this was running fine. I then initiated an upgrade a few weeks ago (ie. 'docker compose pull && docker compose down ntfy && docker compose up ntfy -d') and made sure the service spun up again properly.
Have since left it alone as it's seemingly been working just fine - as in, my android client app is still receiving notifications just fine and has been doing so thruout. Only noticed this today when I was doing the monthly system updates. When I did notice the million or so defunct ssl sessions I immediately tried the web UI only to be greeted by a completely blank page, which makes no sense as the andorid client uses the web to connect, no? But either way, the web ui is no longer showing me data whereas the andoing app received a notification as recent as this morning.
I have since rebooted the entire server as there were kernel upgrades and the likes, but...

Just logged for future.


Pruning/Integrate new-developed pruning API with old one

Signed-off-by: wenhuach21<EMAIL_ADDRESS>Type of Change
feature or bug fix or documentation or validation or others
API changed or not
Description
detail description
JIRA ticket: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

@chensuyue  when the preci gets passed, you can merge it

@PenghuiCheng as this PR will impact the pruning interface a little, pls work on Ella's refactor https://github.com/huggingface/optimum-intel/pull/118 to enable the pruning functionality


Fix deprecation warnings

Reopening because this needs additional verification.

What's the desired language target — Swift 5.10 or Swift 6? For iOS, the build currently emits 46 warnings; switching to Swift 6 and setting Continue building after errors to true results in 10 errors and 25 warnings.

For 6.2 we’re building with Swift 5.10. For 6.3 we’ll be building with Swift 6.

The only Mac deprecation warning right now is for NSSharingService.sharingServices — which we need in order to create a custom menu. So far there’s no replacement for it. So we can’t deal with that one yet.

There are a few iOS deprecation warnings — but they don’t need to be dealt with immediately. Though I hate to leave in any warnings at all, I’m going to put off dealing with this and move this to the 6.3 milestone.


Graph does not support DeconvolutionLayer

There is no DeconvolutionLayer support in Graph

The Graph library is used only for tests purposes, which is why it currently only contains the nodes we've needed to implement the example networks.

We've added this request to our backlog, however it's not high priority so can't guarantee if / when it will be done.
If someone else in the community wants to pick it up, please let us know

Fixed on 18.08 release. Reopen if needed.


Fix rendering to body warning in example projects.

After the React 14 update, the example projects give the warnings:

Warning: render(): Rendering components directly into document.body is discouraged, since its children are often manipulated by third-party scripts and browser extensions. This may lead to subtle reconciliation issues. Try rendering into a container element created for your app.

and

Warning: React.render is deprecated.  Please use ReactDOM.render from require('react-dom') instead.

@davidmdem Looks fine, thanks :+1:


Separate API Auth and Anypoint Headers  (and some fixes)

Features
Separate Anypoint Headers from API Auth

metrics-accelerator-api.raml: moved anypoint headers to Anypoint headers trait and removed from securedBy.  The headers are the Anypoint Platform security, NOT necessarily the security for this API.  Keeping them out of the Authorization header allows for implementers to use their own security such as OAuth.  The Anypoint security headers must still be provided in addition to any security added by the implementer.
Added feature to allow Base64 or plain text secrets in both secret/password Anypoint headers.

Added properties and loggers for API and Dash

Added API and Dashboard HTTP values as properties in app-dev.yaml and api.xml.
Added loggers to dashboard choices for debugging.  Added 501-Not Implemented if dashboard is disabled.

Fixes

Fixed api.xml flow names to match flows Studio generates from metrics-accelerator-api.raml.  Updated MUnit flows to match.
Updated latest version of dependencies
api-call-api-platform: Added default empty string to avoid null error when no response code provided.

Reverted dependencies versions. Should be a different feature. Also, I've been facing this issue, result of using MUnit Maven Plugin 2.3.5.

Latest tests execution after changes

Tests re ran after fix on Base64 for Connected Apps client secret

Connected App Login after change:

Everything looks good, great job adjusting this PR @panizzag @richardmckinley, if we are ready with this we can merge it and from develop run some last tests after the release


Fix concurrent CreateNetwork in bridge driver

The CreateNetworkin the bridge driver was not able to properly
handle concurrent operations causing 2 issues:

crash from nil pointer exception. (Fixes: moby/moby#34515)
not proper handling of conflicting configuration

This commit addresses the 2 previous mentioned issues
and adds a test for it.
The test with the original code has a low failure frequency
to confirm the fix I had to add a time.Sleep in the body of the
CreateNetwork so to have a 100% failure
This PR is a follow up of this: https://github.com/docker/libnetwork/pull/1909
Signed-off-by: Flavio Crisciani<EMAIL_ADDRESS>

@vieux @allencloud addressed comments PTAL, @mavenugo @abhinandanpb can also someone of you take a look?

LGTM

Is it possible to get this merged in? Seeing this issue manifest quite frequently.

we will merge is as soon as @mavenugo reviews it 😈

@vieux @fcrisciani :( I have no place to hide now :) ... Will review it today.

Thanks all for your efforts :)

@mavenugo based on the review I rewrote the patch in a different way, please have a look

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ba46b92). Click here to learn what that means.
The diff coverage is 74.5%.

@@            Coverage Diff            @@
##             master    #1918   +/-   ##
=========================================
  Coverage          ?   38.03%
=========================================
  Files             ?      137
  Lines             ?    27138
  Branches          ?        0
=========================================
  Hits              ?    10323
  Misses            ?    15548
  Partials          ?     1267

Impacted Files
Coverage Δ

drivers/bridge/setup_firewalld.go
71.42% <0%> (ø)

drivers/bridge/bridge.go
54.94% <76%> (ø)

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

Bump. Is there any prediction when this will be fixed? I have CI system depending on this.


add clusterset crds

/assign @qiujian16

I think you would want to add the two crds in hack/init.sh so we can verify the change on them.

added

/approve
/lgtm


Command Bug

when using the command 'pay' you must mention someone,
when you do not mention a valid user it says it's not a user but then dialogue comes up to pay 'Misaki' the specified amount

when using the command 'brain' and supply no args it will stay on a loading loop.
possible fix:
if (args.length == 0) return message.response(undefined, 'Invalid Command usage: \${this.help.usage}\');

What steps do you use to recreate your first glitch?

York knows but cant find why it’s doing it here’s what you can use,
‘m.pay @hdhshshdgdhdhg 200’ when the mention isn’t a user it pays Misaki

Fixed with proof.

Brain is also fixed, but no proof provided.


Settlement Scholastic Feat

The Settlement Scholastic feat should be an archetype feat but is listed as a skill feat and thus cannot be correctly used.

It is an archetype skill feat.
See https://2e.aonprd.com/Feats.aspx?ID=1881 and https://github.com/foundryvtt/pf2e/wiki/The-CounterFAQ#i-am-using-the-free-archetype-variant-rule-and-am-trying-to-add-the-medic-archetypes-treat-condition-feat-to-an-archetype-feat-slot-why-cant-i-do-this-in-foundry

Correct, the pulldown in the compendium has it set as a skill feat which is an error.


Improve error message when loading pretrained fasttext with ngrams

Description
For some pretrained embeddings no subword / binary / ngram models were published
and trying to load them will fail. Currently "Cannot find pre-trained source X
for token embedding fasttext" will be raised, which is wrong given that the
source exists but simply no subword features are available for that source. Now
"No ngrams are available for {}." will be raised instead.
Checklist
Essentials

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage
[X] Code is well-documented

Changes

[X] Improve error message when loading pretrained fasttext with ngrams

Comments
@eric-haibin-lin @szha we may want to do a patch release on Tuesday if we find more small usability issues like this

Job PR-307/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-307/2/index.html


Kenny's notebook renders with different font on the KTH server

http://hera.physchem.kth.se/~scipy-2017-codegen-tutorial/notebooks/08-the-easy-hard-way.html

this is really weird, I've looked at the generated HTML, the %%cython --anotate magic emits a whole new nested  environment. My guess is that the CSS set there clashes somehow. I think we could live with this though?

Yeah we can live with it. Sounds like something that should be reported upstream.

Yeah, let's look into doing that at the conference? (closing for now)


test PR with Github API

Please pull this in!

Docs Build status updates of commit 3a134a7:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 71d64a3:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/index.md
:white_check_mark:Succeeded

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 97fdc15:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/index.md
:white_check_mark:Succeeded

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 75709f1:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/index.md
:white_check_mark:Succeeded

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 441d698:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

sdp-test/page-model-documents/Tutorial.yml
:warning:Warning

Details

sdp-test/index.md
:white_check_mark:Succeeded

sdp-test/page-model-documents/Tutorial.yml

[Warning] Unable to find uid "challenge-uid".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


No output file generated for openScenario

While running openScenario based scenarios in scenario runner, the result files are not getting generated. Tried the following commands:
python scenario_runner.py --openscenario srunner\examples\LaneChangeSimple.xosc  --reloadWorld --output --json and also tried
python scenario_runner.py --openscenario srunner\examples\LaneChangeSimple.xosc  --reloadWorld --output --file
A file named CARLA of size 0kb is getting created everytime I try with these commands. The result files are getting generated for basic scenarios but not for openscenario. Is result file not supported for openScenario?
As shown in the screenshot, the json files are generated foe basic_scenario whereas the file named CARLA is getting generated for openscenario execution.
Desktop

:
OS: [Windows 10]
CARLA Version [CARLA 0.9.10]
Python version [3.9.0] [Only python 3.9 is installed in system]
Scenario_runner Version [0.9.10]

Hey @TeeManiac, sorry for the delay. Checking the issues I thought this one was already solved. Did you manage to solve it? I just checked and it's working fine for me. A file named CARLA:CyclistCrossing2020-11-18-09-57-58.txt was created, with all the info needed.
However, I am working on Ubuntu, and not Windows so that might be an issue (maybe the : have a specific meaning there?). If you haven't solved it yet please try the following

Change the starting name of the OpenScenario file from CARLA: to CARLA_ (1st line of the .xosc)
Go to *srunner/scenarioconfigs/openscenario_configurations.py and change the check at _set_scenario_name() for CARLA_

Let me know if this fixes it, because we might want to change that part if it is a Windows related problem

Hey @TeeManiac, sorry for the delay. Checking the issues I thought this one was already solved. Did you manage to solve it? I just checked and it's working fine for me. A file named CARLA:CyclistCrossing2020-11-18-09-57-58.txt was created, with all the info needed.
However, I am working on Ubuntu, and not Windows so that might be an issue (maybe the : have a specific meaning there?). If you haven't solved it yet please try the following

Change the starting name of the OpenScenario file from CARLA: to CARLA_ (1st line of the .xosc)
Go to *srunner/scenarioconfigs/openscenario_configurations.py and change the check at set_scenario_name() for CARLA

Let me know if this fixes it, because we might want to change that part if it is a Windows related problem

I fixed it by replacing ":" with "" in config_name and its working now.

@TeeManiac Thanks for this info. Maybe we should just remove this "CARLA:" for the output file, as it is more or less just an internal way of handling the different coordinate systems.

@TeeManiac Thanks for this info. Maybe we should just remove this "CARLA:" for the output file, as it is more or less just an internal way of handling the different coordinate systems.

@fabianoboril Yes, I too think so as the json file for basic scenario just has the Scenario name + current time for the json file name.


Mocking Router.events

I need to trigger Router events in my unit tests - and Im very unsure how to do this.
If I have a React hook that does the following:
useEffect(() => {
    Router.events.on('routeChangeComplete', latestPath => {

      // First work out if the latest route change is actually a route change
      // As we can have many useQueryParam hooks executing simulatenously -
      // we don't want 3 route changes for the same route, emptying the queue
      if (latestPath === previousPath) {
        return;
      }

      previousPath = latestPath;

      // Route change complete, shift the queue
      valueQueue.shift();
      // Start the next Route change
      if (valueQueue.length > 0) {
        valueQueue[0]();
      } else {
        queueExecuting = false;
      }
    });

    return () => {
      Router.events.off('routeChangeComplete');
    };
  }, []);

My unit test relies on this callback executing , how can I trigger this callback using a jest mock?

@danielcooke1996 Did you figure this out?

Apologies yes - it was quite simple actually.
Just store the callback into a variable so you can "trigger" the event whenever you wish.
let routeChangeComplete;
Router.default.events.on = jest.fn((event, callback) => {
    routeChangeComplete = callback;
  });


Conflict version of swagger-core on springdoc-openapi-ui

Versions:
- Vaadin / Hilla version: 23.2.1/ 1.2.1
- Java version:1 7

If project config to use hybrid mode which is both Vaadin flow and Hilla. To enable swagger UI, we add
<dependency>
            <groupId>org.springdoc</groupId>
            <artifactId>springdoc-openapi-ui</artifactId>
            <version>1.6.11</version>
        </dependency>

And will get exception:
java.lang.NoSuchMethodError: 'io.swagger.v3.oas.models.media.ComposedSchema io.swagger.v3.oas.models.media.ComposedSchema.addAllOfItem(io.swagger.v3.oas.models.media.Schema)'
	at dev.hilla.generator.OpenAPIObjectGenerator.lambda$parseNonEndpointClassAsSchema$24(OpenAPIObjectGenerator.java:524) ~[endpoint-1.2.1.jar:na]

Root cause:
springdoc-openapi-ui uses swagger-core and swagger-models which Hilla also uses. This one causes version conflicting. So, swagger does not work.
The workaround is to exclude swagger-core and swagger-models from springdoc-openapi-ui.
However this solution also has potential issue when springdoc-openapi-ui needs to use newer version of swagger-core and swagger-models .
<properties>
        <java.version>17</java.version>
        <vaadin.version>23.2.1</vaadin.version>
        <hilla.version>1.2.1</hilla.version>
    </properties>
<dependency>
            <groupId>org.springdoc</groupId>
            <artifactId>springdoc-openapi-ui</artifactId>
            <version>1.6.7</version>
            <!-- avoid conflict with hilla. Note: change springdoc-openapi-ui version to 1.6.11 will not work -->
           <exclusions>
                <exclusion>
                    <groupId>io.swagger.core.v3</groupId>
                    <artifactId>swagger-core</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>io.swagger.core.v3</groupId>
                    <artifactId>swagger-models</artifactId>
                </exclusion>
            </exclusions>
        </dependency>

so i would suggest to add springdoc-openapi-ui as default lib in parent POM of Vaadin and Hilla needs not indicate version for swagger-core and swagger-models.
More details, can try to run vaadin demo project below:
https://github.com/letrthang/vaadin23-flow-pro-components.git
In above demo project, in POM to change springdoc-openapi-ui version to 1.6.11 and will see the swagger UI not working.

Any better solution ?

Hi, thanks for submitting and for the workaround. Unfortunately, it’s not currently possible to upgrade swagger dependencies for us because of https://github.com/vaadin/hilla/issues/462
Let me close this for now. We could reconsider this once swagger has a fix for https://github.com/swagger-api/swagger-codegen-generators/issues/956


Flask extension demystify

https://flask.palletsprojects.com/en/2.0.x/extensiondev/

This class is used to control the SQLAlchemy integration to one or more Flask applications. Depending on how you initialize the object it is usable right away or will attach as needed to a Flask application.
There are two usage modes which work very similarly. One is binding the instance to a very specific Flask application:
app = Flask(name)
db = SQLAlchemy(app)
The second possibility is to create the object once and configure the application later to support it:
db = SQLAlchemy()
def create_app():
app = Flask(name)
db.init_app(app)
return app
The difference between the two is that in the first case methods like create_all and drop_all will work all the time but in the second case a flask.Flask.app_context has to exist.
By default Flask-SQLAlchemy will apply some backend-specific settings to improve your experience with them.


[Proposal] QRCoder2

Proposal for #242. This is just an example -- I didn't move everything that could be moved. For the most part I left the code as-is after moving it. I targeted .net 6 without putting much thought into it. I'm sure we could target a bit lower if desired. My goal was to keep dependencies out of QRCoder2.
Payloads
I pulled payloads out of PayloadGenerator into their own classes. Other libraries can expose new Payloads by referencing QRCoder2 and extending PayloadBase.
QR Code data
QRCodeGenerator translates a PayloadBase to QRCodeData. I didn't change anything here, except moving ECCLevel and EciMode to the Payloads namespace.
Renderers
I added a renderers namespace. I put a couple of the "easy" QR codes that had no dependencies there: PngByteQRCode and BitmapByteQRCode. I think the *QRCode classes should then be renamed *QRCodeRenderer. As an example, I added a library for QRCoder2.Renderers.ImageSharp which has a dependency on ImageSharp (but does nothing at the moment). Other library authors could provide their own renderers using only QRCoder2 as a dependency.

Looking for a QR Code generator, and I'm too using .NET6 - the sample code here is not working with the Nuget package that is supplied https://github.com/codebude/QRCoder version 1.4.3... so I'm thinking it not ready for .net6 ?  Just started to look into the source and see if I can help.. but thought I would drop this comment if you wanted to work on it together

Happy to help out too.  A have a BIG project and it has all upgraded to .net 6 quite successfully, except for this.

Happy to help too.
I love this project and I because I wanted to use it on my .NET 6 project, I've written the SVG and PNG generator classes locally, as new generators.
I'm using ImageSharp and ImageSharp.Drawing.
This allows me to produce not only PNG but many other formats.
The only downside is that I've ImageSharp dependencies. It's not a problem for me because the generators are within my project, and not in this library.
Would it become acceptable to add dependencies to this library?
PS: Happy to share the classes if you need them.

I also wont to use this project in my own .NET 6 project, howewer now this is not cross-platform solution. Do you plan to make this cross-platform and when?
PS: I think using ImageSharp is grate idea becouse it's meets these requirements.

The only downside is that I've ImageSharp dependencies. It's not a problem for me because the generators are within my project, and not in this library.

@itarizin I think a clean approach would be to invert the dependency. Have something like an IQrCodeRenderer interface exposed publicly in the core lib. Implementation projects/libs would then reference the core lib, providing an implementation using any dependency they wish.
E.g. ImageSharpRenderer : IQrCodeRenderer or SystemDrawingRenderer : IQrCodeRenderer or InsertImplHereRenderer : IQrCodeRenderer.
This assumes a common renderer agnostic contract can be extrapolated. Something like IQrCodeRenderer<T> (where T is the rendered output type) might even provide greater flexibility.
Something similar for Payloads like IQrCodePayload might be useful as well?
Disclaimer: I'm just spit-balling here. I haven't actually looked at the code in a few years 😀.


[Inputs.json]: Keyboard focus moves to non interactive element in DatePicker under Due Date.

https://microsoft.visualstudio.com/OS/_workitems/edit/38004603

As per new guideline we are enhancing the severity. so we are closing this issue and logged new bug: [Inputs.json]: Keyboard focus moves to non interactive element in DatePicker under Due Date. · Issue #8468 · microsoft/AdaptiveCards (github.com)


614 remove sayg from match

Resolves #614

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

61 of 72 (84.72%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 96.235%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

CourageScores/Services/Command/DeleteTournamentMatchSaygCommand.cs
50
51
98.04%

CourageScores/ClientApp/src/components/division_fixtures/tournaments/TournamentRoundMatch.js
0
10
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
8732

Relevant Lines:
8815

💛  - Coveralls


Documentation

Hey! This looks really cool :)
Do you have any documentation on using this with a generic OAuth2 provider not just Azure AD? What type of authentication flow is this using, and what scopes do I need to give this client etc? please?
Thanks!

@willfurnell
i am using this with a NetIQ (now called Microfocus) AccessManager and works fine.
Still fiddling around, but this gives successfull authentication :
{     client-id: "<your Client-ID>",     client-secret: "<your Client Secret>",     redirect-url: "urn:ietf:wg:oauth:2.0:oob",     scopes: ["email"],     endpoint-auth-url: "https://<your AccessManager>/nidp/oauth/nam/authz",     endpoint-token-url: "https://<your AccessManager>/nidp/oauth/nam/token",     extra-parameters: {         acr: "saml/secure/form/password/smartphone"     },     username-format: "%s", }
As you can see i have adapted the 'extra-parameter' to request a specific authentication contract from my box.
If you were to remove it it will authenticate you with the default authentication contract.

As another data point that could be documented, here is a working config file using this PAM module to connect to a Keycloak server:
{
   client-id: pam,
   client-secret: <REDACTED>,
   scopes: ["profile"],
   endpoint-token-url: "https://keycloak.example.com/auth/realms/<REALM_NAME>/protocol/openid-connect/token",
   extra-parameters: {
   },
}

This uses a custom client setup on the Keycloak end called "pam" with settings that look like this:


NaN validation loss when benchmarking

The models vesteinn/ScandiBERT and mideind/IceBERT-mC4-is get NaN validation loss when benchmarking.
We need to check if this is due to the framework or the models themselves. If it's the models then an InvalidBenchmark should be thrown instead of getting ~0% scores on everything.

This seems to be mixed precision causing this. When disabling fp16 it stops occuring. We need to figure out what the solution is in this case. Either we would explicitly look for NaN validation loss, in which case it should restart the evaluation without fp16, or alternatively fp16 should never be enabled by default, and instead by enabled if we're running out of memory.
@peterbjorgensen @AJDERS @KennethEnevoldsen Thoughts?

@KennethEnevoldsen I would guess that we would be able to catch the NaN loss if it occurs early. Maybe one way could be having a check within the compute_metrics function, and throw an error if the val loss is NaN. The benchmarking script could catch that error and restart the benchmark with fp16 set.
But this assumes that these models all exhibit NaN loss in the first evaluation (step 30). I noticed that xlm-roberta-base also has weird behaviour now - I'd have to check if that's due to NaN loss too.

Hmm right. if we do 32 as default, but switch to fp16 if there is insufficient memory though that would make it notably slower (apparently up to 8 times). I would probably just set a custom flag for the specific models and raise a warning if you try to call them with fp16.


Correct ArgumentListMatcher instrumentation to make it work with any_args

The ArgsHelper module did not instrument the ArgumentListMatcher correctly, hence it was not possible to use any_args with the be_queued/have_queued matchers.
original code:
module ArgsHelper
  private

  def match_args(expected_args, args)
    arg_list_matcher = RSpec::Mocks::ArgumentListMatcher.new(expected_args)
    arg_list_matcher.args_match?(args)
  end
end

Within the ArgumentListMatcher you did end up in the fuzzy_matcher comparing lists of lists instead of lists.
   28          expected_list.zip(actual_list).all? do |expected, actual|
   29            values_match?(expected, actual)
   30          end

(rspec-support-3.0.0/lib/rspec/support/fuzzy_matcher.rb:28)
So have_queued(:foo, :bar) ends up to be compared as expected_list == [[:foo, :bar]] instead of [:foo, :bar]
Naturally, that works as values_match?([:foo, :bar], [:foo, :bar]), but not intentionally. What we'd like to see was zipping and matching all?  values_match?(:foo, :foo) and values_match?(:bar, :bar).
The Problem arises when using any_args matching vs no args, nil, :foo and multiple args
Resque.enqueue(Person)
# breaks
it { should be_queued(any_args) }

Resque.enqueue(Person, nil)
# breaks
it { should be_queued(any_args) }

Resque.enqueue(Person, :foo)
# breaks
it { should be_queued(any_args) }

The fix is simple, desplatting before instrumenting the ArgumentListMatcher:
module ArgsHelper
  private

  def match_args(expected_args, args)
    arg_list_matcher = RSpec::Mocks::ArgumentListMatcher.new(*expected_args)
    arg_list_matcher.args_match?(*args)
  end
end

Cheers
SH

Mmh master is broken with the debugger install error - couldn't fix this on travis (only for 1.9.3).
In my VM both rubies (1.9.3-latest, 2.0.0-latest) build correctly.

Hi @serioushaircut,
Thanks!


added python no_arch for nglview

Do we need to increment the build number too, or did this not build successfully before?

I think if one switches from an arch to a no arch package, one does not need to increment the build number to trigger a new build.

But won't the old arch-based packages still be on anaconda cloud and have the same build number? Or would you manually delete them?


zsh-fzf-tab: Support darwin platform

Motivation for this change
Support darwin platform
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

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

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

[x] Fits CONTRIBUTING.md.

I haven't macos thus there is no way to test. The original one can work on nixos without any patches.


plus: ensure all L2s have a defaultChild set up for them, ideally named "Resource"

To support escape hatching using ApiObject.of() (see #372).

@iliapolo just realized that using Default can cause issues because "Default" is ignored by the construct path address calculations.


Feature: Close multiple git submodules/repositories at the same time

At work we use a lot of git submodules. I often want to close multiple submodules/repositories in vscode, which is not possible at the moment - I have to right-click and close them one by one.
To solve this I'd suggest implementing at least one of the following:

A) Close all repositories that are selected instead of just the one right-clicked.

B) Add an option to close all other repositories (similar to how browsers provide an option to "close other tabs")
#130931

Hey @lszomoru, considering that this has been open for almost a year now, would it be possible to merge #130931 in the meantime?
The PR only implements one part of the suggested solutions but would already be very beneficial for the described use-case and the code changes are kept to a minimum. Let me know if there is something that I can do to resolve this issue.

I often want to close multiple submodules/repositories in vscode, which is not possible at the moment - I have to right-click and close them one by one.

The option to close multiple repositories would be really welcome.
It's been almost two years since this issue was open.... :'(

+1 on being able to close multiple selected repositories.
This would be very useful for dealing with dozens of repositories and being able to quickly hide the ones we're not currently making any changes to.
It's disappointing that, despite selecting multiple repositories, the Close Repository right-click action will only close the one under the mouse cursor. To close them all we have to spam the action multiple times.
Reopening closed repositories is very easy, so I don't see any UX downsides to just plain closing all selected repositories.
If this is still a concern, a separate action like Close All Selected Repositories could also be an option.

+1 to "close all repositories".  I suffer from this frequently...

@Stabledog, could you please elaborate on your scenario in which you would like to close all repositories?

@narimiran, thanks for the information. Given the information, do you really want to close all repositories, or would you like to just close all git-submodules? Also, have you considered setting git.autoRepositoryDetection to subFolders  so that repositories from open files are not being opened?

@Stabledog, could you please elaborate on your scenario in which you would like to close all repositories?

I'm a teacher.  I have a bunch of repos which act as a host environment for repos that belong to students.  I have scripts that do things like "show me the server.py file for everything in students/*/project".   Each one of those is its own repo with student code.
So now I have 50 source files open, and I can page through them rapidly, comment on the code, etc.   But if I want to do anything specific with any one of the git working copies, I must do that from the command line because the VScode git plugin is a hopeless sea of open repo nodes, with no ability to filter or sort, no ability to hide or close in bulk... it's just useless in this situation.
Having a "close all" would allow me to at least clean the slate so I can pick individuals  to study more closely.

"Close Other Repositories" has been added to the context menu in https://github.com/microsoft/vscode/pull/200893.

Given the information, do you really want to close all repositories, or would you like to just close all git-submodules?

No, not all. There are two good options (and both should be available, IMO):

Close (all) other repositories (this is what https://github.com/microsoft/vscode/pull/200893 does, AFAICT)
Close (multiple) selected repositories. User manually selects (control+click or shift+click) multiple repositories that they would like to close.

Option 2. listed above is tracked here - https://github.com/microsoft/vscode/issues/176717

I implemented the second option in this PR: #201073


fix typo in quassel

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

Hi,This is my PR...https://github.com/kubernetes/charts/pull/4789.  The two are exactly the same...Thanks :)

/ok-to-test
/lgtm


Cow: Watch keshwaplay.net

Cow requests the watch of the watch_keyword keshwaplay\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

keshwaplay\.net has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by double-beep in Charcoal HQ


Dash Docset - (Wishlist)

Would be nice if someone with the skills could create a Dash Docset for arrow. I don't have the skills to do it.
I use arrow in Pythonista 3 (py3.6) iOS app ( I am still a nubie). Just starting to use it in a flask app with flask-restful.
It's a great lib.

arrow docs are available here arrow.readthedocs.org - it is unviable to create any other type of documentation right now


Email reappears as soon as it's deleted [disappears eventually]

Hello,
An annoyance more than anything really.
When I delete an email from the interface, the mail in question disappears and shortly reappears. It then sits there for a varying amount of time before eventually vanishing permanently (I'm pretty sure it vanishes, but I may have been messing with the mbox to see why it wasn't).
This is a stock, brand new default installation on 14.04.3 of Postfix, Dovecot and Apache2.
Here's a log snippet:
[09:43:19.615][25dc0c89] INFO[DATA]: [DATE:07.10.15][RL:<IP_ADDRESS>3][PHP:5.5.9-1ubuntu4.13][IP:<IP_ADDRESS>][PID:12998][Apache/2.4.7 (Ubuntu)][apache2handler]
[09:43:19.616][25dc0c89] INFO[DATA]: [Suhosin:off][APC:off][MB:on][PDO:mysql][Streams:tcp,udp,unix,udg,ssl,sslv3,tls]
[09:43:19.616][25dc0c89] REQUEST[NOTE]: [POST] http://<IP_ADDRESS>/?/Ajax/&q[]=/_lm8a2IG8FSj8mblbP3xGCoTLaqS4rl39epMH2S9X5iD0NwwKqPJic9NmxNXOsFN32ROU2NoPt7E_cI78n-r-I2gMpKg0R9ZgNi29LviGZwO5P8prwtc65shMqaiFgF12rDs4tMuWC5LtMgk1XvnZwPeGGRDLmTrzArCuRENY58p5OWdRZLN4cys4TbPaeFvZJ7_7x5pULLYSqBAmUQj9AnooAu4BnS2ohAF7bbaq1ABbdkHKI7Z-QWfjI3mYcTRJDdZjymYyVJ5zhvv5T3w6VW8AjLA7BTJbNp6vsX8rNI-6-XQax8--uQ../
[09:43:19.616][25dc0c89] AJAX[NOTE]: Action: DoMessageMove
[09:43:19.616][25dc0c89] POST[DATA]: {"FromFolder":"INBOX","ToFolder":"Trash","Uids":"1","MarkAsRead":"1","Learning":"","Action":"MessageMove","XToken":"5ad51d61dc07017b2db7c3fa75cb445e"}
[09:43:19.620][25dc0c89] IMAP[NOTE]: Start connection to "tcp://localhost:143"
[09:43:19.620][25dc0c89] IMAP[NOTE]: Connected (success)
[09:43:19.631][25dc0c89] IMAP[DATA]: < * OK [CAPABILITY IMAP4rev1 LITERAL+ SASL-IR LOGIN-REFERRALS ID ENABLE IDLE STARTTLS AUTH=PLAIN] Dovecot (Ubuntu) ready.\r\n
[09:43:19.632][25dc0c89] IMAP[SECURE]: > TAG1 LOGIN "b" "********"\r\n
[09:43:19.652][25dc0c89] IMAP[DATA]: < TAG1 OK [CAPABILITY IMAP4rev1 LITERAL+ SASL-IR LOGIN-REFERRALS ID ENABLE IDLE SORT SORT=DISPLAY THREAD=REFERENCES THREAD=REFS THREAD=ORDEREDSUBJECT MULTIAPPEND URL-PARTIAL CATENATE UNSELECT CHILDREN NAMESPACE UIDPLUS LIST-EXTENDED I18NLEVEL=1 CONDSTORE QRESYNC ESEARCH ESORT SEARCHRES WITHIN CONTEXT=SEARCH LIST-STATUS SPECIAL-USE BINARY MOVE] Logged in\r\n
[09:43:19.652][25dc0c89] IMAP[DATA]: > TAG2 SELECT "INBOX"\r\n
[09:43:19.654][25dc0c89] IMAP[DATA]: < * FLAGS (\Answered \Flagged \Deleted \Seen \Draft)\r\n
[09:43:19.654][25dc0c89] IMAP[DATA]: < * OK [PERMANENTFLAGS ()] Read-only mailbox.\r\n
[09:43:19.654][25dc0c89] IMAP[DATA]: < * 1 EXISTS\r\n
[09:43:19.655][25dc0c89] IMAP[DATA]: < * 0 RECENT\r\n
[09:43:19.655][25dc0c89] IMAP[DATA]: < * OK [UIDVALIDITY<PHONE_NUMBER>] UIDs valid\r\n
[09:43:19.655][25dc0c89] IMAP[DATA]: < * OK [UIDNEXT 2] Predicted next UID\r\n
[09:43:19.655][25dc0c89] IMAP[DATA]: < * OK [HIGHESTMODSEQ 6] Highest\r\n
[09:43:19.655][25dc0c89] IMAP[DATA]: < TAG2 OK [READ-ONLY] Select completed (0.001 secs).\r\n
[09:43:19.655][25dc0c89] IMAP[DATA]: > TAG3 CAPABILITY\r\n
[09:43:19.656][25dc0c89] IMAP[DATA]: < * CAPABILITY IMAP4rev1 LITERAL+ SASL-IR LOGIN-REFERRALS ID ENABLE IDLE SORT SORT=DISPLAY THREAD=REFERENCES THREAD=REFS THREAD=ORDEREDSUBJECT MULTIAPPEND URL-PARTIAL CATENATE UNSELECT CHILDREN NAMESPACE UIDPLUS LIST-EXTENDED I18NLEVEL=1 CONDSTORE QRESYNC ESEARCH ESORT SEARCHRES WITHIN CONTEXT=SEARCH LIST-STATUS SPECIAL-USE BINARY MOVE\r\n
[09:43:19.656][25dc0c89] IMAP[DATA]: < TAG3 OK Capability completed.\r\n
[09:43:19.656][25dc0c89] IMAP[DATA]: > TAG4 UID MOVE 1 "Trash"\r\n
[09:43:19.660][25dc0c89] IMAP[DATA]: < * OK [COPYUID<PHONE_NUMBER> 1 3] Moved UIDs.\r\n
[09:43:19.660][25dc0c89] IMAP[DATA]: < TAG4 OK Move completed.\r\n
[09:43:19.660][25dc0c89] IMAP[DATA]: > TAG5 STATUS "INBOX" (MESSAGES UNSEEN UIDNEXT HIGHESTMODSEQ)\r\n
[09:43:19.660][25dc0c89] IMAP[DATA]: < * OK [HIGHESTMODSEQ 6] Highest\r\n
[09:43:19.660][25dc0c89] IMAP[DATA]: < * STATUS INBOX (MESSAGES 1 UIDNEXT 2 UNSEEN 0 HIGHESTMODSEQ 6)\r\n
[09:43:19.660][25dc0c89] IMAP[DATA]: < TAG5 OK [CLIENTBUG] Status on selected mailbox completed.\r\n
[09:43:19.661][25dc0c89] AJAX[DATA]: {"Action":"MessageMove","Result":["INBOX","1ce2158d8688db608d4f6e79ecdea118"],"Time":49}
[09:43:19.661][25dc0c89] IMAP[DATA]: > TAG6 LOGOUT\r\n
[09:43:19.661][25dc0c89] IMAP[DATA]: < * BYE Logging out\r\n
[09:43:19.661][25dc0c89] IMAP[DATA]: < TAG6 OK Logout completed.\r\n
[09:43:19.661][25dc0c89] IMAP[NOTE]: Disconnected from "tcp://localhost:143" (success)
[09:43:19.661][25dc0c89] INFO[MEMORY]: Memory peak usage: 1.25MB
[09:43:19.661][25dc0c89] INFO[TIME]: Time delta: 0.049572944641113

Seems to indicate the mailbox is in readonly mode.
Video: http://gfycat.com/IndolentLoathsomeErne
Thanks,
Jason

I've seen this happen but just on occasion, the same way some selected messages get read and unread again. Does it always happen to you?

I can replicate this with every email that comes in, yes. I can also confirm that no emails today have eventually deleted, so I guess that was just me tinkering on the commandline.

[09:43:19.660][25dc0c89] IMAP[DATA]: < * STATUS INBOX (MESSAGES 1 UIDNEXT 2 UNSEEN 0 HIGHESTMODSEQ 6)\r\n
[09:43:19.660][25dc0c89] IMAP[DATA]: < TAG5 OK [CLIENTBUG] Status on selected mailbox completed.

Status response says that the folder still contains one message. I guess, RainLoop thinks that nothing happen and shows your message list from cache.
I have made some fixes and most probably it will be fixed in the next release.


Implement ClientSettingsPolicy

Proposed changes
Write a clear and concise description that helps reviewers understand the purpose and impact of your changes. Use the
following format:
Problems:

As a Cluster Operator, I want to set defaults for client settings that will work for most applications so that most Application Developers will not have to tweak these settings.
As an Application Developer, I want to be able to configure client settings for my application based on its behavior or requirements.
As an Application Developer, I want to override the defaults for client settings set by the Cluster Operator because the defaults do not satisfy my application's requirements or behavior.

Solution: Implement ClientSettingsPolicy API.

Cluster operators can create a ClientSettingsPolicy for a Gateway to set defaults for client settings that apply to all routes attached to that Gateway.
App devs can create ClientSettingsPolicies for their routes and specify client settings that override the defaults set by the cluster operator.

Testing: Manually tested the following cases:

attaching to Gateway
attaching to HTTPRoute
attaching to GRPCRoute
merging and inheritance behavior
conflict handling

Closes #1792 #1760
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING doc
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that all unit tests pass after adding my changes
[x] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Introduces ClientSettingsPolicy CRD. This CRD allows users to configure the behavior of the connection between the client and NGINX.

I wonder if we want a tech-debt issue to rework BackendTLSPolicy at some point to use this new framework.

Is the PolicyAncestorStatus supposed to apply to Inherited policies? I thought I read that it only applies to direct policies.

Is the PolicyAncestorStatus supposed to apply to Inherited policies? I thought I read that it only applies to direct policies.

Yeah, currently, it is only required for direct policies, but there's been some discussion to include it on inherited policies. Plus, the spec is moving towards collapsing direct and inherited policies into one policy type with a strategy field.
@pleshakov talked me into including PolicyAncestorStatus on the ClientSettingsPolicy here: https://github.com/nginxinc/nginx-gateway-fabric/pull/1793#discussion_r1551878502

I wonder if we want a tech-debt issue to rework BackendTLSPolicy at some point to use this new framework.

This might be tricky. We'd have to create a wrapper class to pass it around like an NGF Policy. Then, we'd have to figure out how to genericize the ConfigMap resolving part. I'm sure it can be done, but it's not an easy lift.


Update with Selectable Dropdown

Need to improve this component on Filters/src/Desktop/desktop_filter_style_2.html
Improvements required :
Replace Hoverable + non selectable dropdown with clickable and selectable dropdown
Any reference image?
desktop_filter_style_2.htm

@RavindraP04 go ahead


[BUG] Azure Identity - Risky Cryptographic Algorithm CWE ID 327

Veracode scan resulting below flaws from Azure SDK v3.9.0 com.azure.spring:azure-spring-boot-bom:3.9.0
Use of a Broken or Risky Cryptographic Algorithm CWE ID 327

CWE Id
Module
Location
Exploitability

327
azure-identity-1.3.6.jar
com/azure/identity/implementation/intellij/Aes.java 61
Likely

327
azure-identity-1.3.6.jar
com/azure/identity/implementation/intellij/IntelliJCryptoUtil.java 52
Likely

Tried to override version to latest com.azure:azure-identity:1.3.7 and same flaws were listed in v1.3.7 too.
To Reproduce
Add dependencies com.azure.spring:azure-spring-boot-bom:3.9.0, com.azure.spring:azure-spring-boot-starter-keyvault-secrets:3.9.0 to project and that will bring dependency com.azure:azure-identity:1.3.6.
Code Snippet
Add the code snippet that causes the issue.
Expected behavior
No flaws from Veracode.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ X] Bug Description Added
[ X] Repro Steps Added
[ ] Setup information Added

@g2vinay  @joshfree .. do you have any updates or suggestions ?  This is blocking quality gates for our app release.