Start-Process on Windows: -Environment is quietly ignored when -Verb RunAs is also specified, combination should be prevented.

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Note:

I'm assuming that the underlying .NET / Windows APIs don't support this combination; if so, an error should be reported.

Indeed the .NET APIs actively prevent the analogous combination  (not sure about direct use of the WinAPI): .Verb = "RunAs" requires .UseShellExecute = true, which in turn actively prevents modifying the environment via .Environment or .EnvironmentVariables (exception "The Process object must have the UseShellExecute property set to false in order to use environment variables.").

Start-Process -Verb RunAs cmd -Environment @{ foo='new' } '/c "echo [%foo%]" & pause' -Wait

Expected behavior
An error indicating that the specific parameter combination isn't supported.
Actual behavior
The process is launched, and the -Environment argument is quietly ignored.
[%foo%]

Error details
No response
Environment data
PowerShell 7.4.0 on W11 22H2

Visuals
No response

Indeed the .NET APIs actively prevent the analogous combination
Sounds like its an issue with .NET, have you opened an issue in there repo?

@StevenBucher98:

I can't speak to whether the by-design limitation in .NET is ultimately an unavoidable limitation in the Windows API.

I am not personally interested in researching this further.

If someone does, and if it turns out that the limitation could be overcome via a change in .NET, there's no guarantee that such a change would be made and, even if so, it would take a loooonng time for it to become available in PowerShell.

Therefore, I don't think the Resolution-External label is appropriate here.

Instead, this is an Issue-Bug, due to allowing you to specify unsupported parameter combinations without so much as a warning.

Another way to resolve this would be for PowerShell itself to work around the .NET limitation, via direct WinAPI calls if even possible.

Marking up for grabs.


Team PRs on myteam page

Currently the team PRs can only be seen if we open the team page through the leaderboard.
It will be easier for team members to see their PRs through my team dashboard.
I will implement this. :)

Go for it!

This would be super cool. I had no idea you could even view PRs! I was about to suggest this should be a feature, so clearly we don't know where to go to find PRs. Having it on the team page makes a lot of sense.


RepositoryCreateForkOptions is should be body parameters

Hi Team!
RepositoryCreateForkOptions is not working. because it is not url params but body params.
docs.github.com/en/rest/repos/forks#create-a-fork
Sorry I didn't realize this when I added it before.
I will create a pull request for this issue.
Thanks.

@valbeat Any progress on this issue?

@TruRelax it is awaiting review. could you review this one? https://github.com/google/go-github/pull/2490


Remove NewLine in TextBox only by one backspace press, not two

I suppose this happens due to NewLine represented as two chars \r\n.

I have fixed this in my text editor, will see if code is compatible and submit a pr, remind me if I forget.

Hi, @danwalmsley. Do you have any news related to this feature? )

Any news on this @danwalmsley ?

Ah I looked at it, but the way it was implemented in my text editor depended on my document model, which would be overkill for textbox. Does textbox split the text into lines? Or could textbox check for newline chars when backspace or delete is pressed?


[PROJECT PROPOSAL] Reddit Sentiment Analysis

Project Request
I would like to make a sentiment analysis on the Reddit data using pytorch.

Field
Description

About
Sentiment analysis on Reddit data

Github
karthikbhandary2

Email
<EMAIL_ADDRESS>

Label
Project Request

https://github.com/karthikbhandary2

Define You

[ ] GSSOC Participant
[ ] Contributor

Reddit sentiment Analysis using pytorch

Description
In this project, I am going to be performing sentiment analysis on the Reddit data which is available on Kaggle. I am also going to predict whether a given comment is positive or negative as well.
Scope
This project is going to be used to analyze the sentiment and then perform prediction on the given comment to find out if it is positive or negative.
Timeline
22-06-2023 to 02-07-2023
Video Links or Support Links
[Links that can support the project in anyway]

Please assign me this task. @adithya-s-k


Empty authorized_keys after delete user key

Hi,
i have multiple ssh-keys in my authorized_keys file on the server. If i delete a key from gitea user in the web frontend my authorized_keys file is empty. Deleting a user key in the frontend does not delete only the gitea key, it also deletes all other keys in authorized_keys...
Thanks for your work.
Robert

This is the current behavior, you should anyway use a dedicated user for Gitea. I will keep this issue open, maybe we will add some kind of marker to this file to prevent the deletion of existing keys in the future.

Sure normal behavior would be to use a dedicated user. But some of us (like me) use gogs/gitea in a shared hoster environment (for me it is uberspace.de) so it is not possible to run applications with a different user. Also there is an other fact (maybe im wrong) but if i run gitea with a generic user for example "git" the SSH keys for multiple gitea users are in the same authorized_key file (/home/git/authorized_keys) right? If one user deletes bis SSH key from the web gui, all other users will lose his keys too because all keys are in the same file? If this is right there is a big Problem for multi user environments.

But the behavior is the same for Gogs and Gitea. And the file is shared for all Gitea users, but it gets regenerated if a user deletes his key from the UI.

You are right. Nice to here the file will be regenerated, that makes much more sense :)

Would it be possible to add a custome file to the config? In this file users can add static non-git ssh-keys and if gitea regenerates the authorized_keys the keys from this file will be merged with the git-ssh keys.

IMHO some marker within the file makes more sense

It's more like a bug rather than enhancement. If gogs dos it too then it also has a bug.
Why bug? Because it does things that is not suppose to do - gitea should not touch things that belongs to others.
I've got my authorized_keys wiped out without using any "key" feature in gitea.

Agreed, it's a bad bug.
I think we should get "tagged" entries in authorized_keys ASAP,
providing a way to add missing tags and backport to 1.0 too.
Was there already an issue about this ? Please check and if so link
here (somethig about "Rewrite authorized_key task wiping out keys")

Sounds like #906

Yeah, this issue will be resolved when https://github.com/go-gitea/gitea/pull/906 gets merged.


Feature: Output compiled schema as JSON – "Schema.Marshal()"

The Problem
In my current project we manage many schemas, subschemas, and schemas for schemas, etc. and put them together as needed with AddResource()/Compile(). When creating new schemas, I find that finding possible mistakes is that much harder without seeing the compilation result. Hunting down possible mistakes with JSON shards is painful, with the feedback loop being too long for efficient development.
The Solution
Schema struct could have a method Marshal() which is essentially the same as, for example, stdlib function json.Marshal(). The resulting byte slice could then be easily inspected for errors in the compiled schema.
A bonus would be MarshalIndent() to pretty-print the schema JSON.

i guess, if there is an error in another schema which is ref-ed from original schema, currently the error returned does not say which schema file has error. it always says the error is in main schema file.
if it is so, let me know. that is a bug and needs to be fixed.

The trouble has been mostly about understanding how the refs end up pointing, but the more I work on the tougher issues, the more I grok JSON Schema. If I run into such nested-errors-reported-at-top-level issues, I will open another issue here.
Closing this, thanks for listening :-)


feat: filtering for relation widget (#2405)

Summary
This PR adds a filters field for the Relation widget which allows adding filters by which the available options are filtered on. This was requested in issue #2405 (and also by myself :)).
How it works is that you can add "filters" which are a pair of field and the allowed values, where the widget will only show options (collection items) that satisfy all the "filters". An collection item satisfies a filter if the value of field is one of the values in values.
Example
collections:
  - name: posts
    label: Posts
    folder: _posts
    create: true
    fields:
      - label: Title
        name: title
        widget: string
      - label: Draft
        name: draft
        widget: boolean
        default: true
      - label: Body
        name: body
        widget: markdown
  - name: restaurants
    label: Restaurants
    folder: _restaurants
    create: true
    fields:
      - label: Title
        name: title
        widget: string
      - label: Body
        name: body
        widget: markdown
      - label: Posts
        name: posts
        widget: relation
        collection: posts
        multiple: true
        search_fields: [title]
        display_fields: [title]
        value_field: '{{slug}}'
        filters:
          - field: draft
            values: [false]

In this example the relation widget in the restaurants collection will only show and allow options that are posts which are not a draft (i.e. draftfield is false). Say, we have 20 posts in the CMS of which 5 have draft set to true, then we will only see the other 15 posts as option in the relation widget.
Multiple filters can be added:
filters:
  - field: draft
    values: [false]
  - field: title
     values: ['post about cats', 'post about dogs']

In this case only the posts with draft set to false and a title that is either 'posts about cats' or 'post about dogs' will be options seen in the relation widget.
Test plan
Added the following unit tests:
with filter
      ✓ should list the 10 out of 20 option hits on initial load using a filter on boolean value (532 ms)
      ✓ should list the 5 out of 20 option hits on initial load using a filter on string value (528 ms)
      ✓ should list 4 out of 20 option hits on initial load using multiple filters (523 ms)
      ✓ should list 0 out of 20 option hits on initial load on conflicting filter values (25 ms)

After adding the following filter to the dev-test config.yml:
filters: [ {field: "draft", values: [false]} ]
These are the results of the filter:

Out of the 23 posts items that are in the posts collection by default in the dev-test cms, the filter on draft with value false gives the following possible options: (note that without the filter you would see posts 1 to 20 by default)

Choosing, deleting, clearing options works the same:

Checklist
Please add a x inside each checkbox:

[x] I have read the contribution guidelines.

A picture of a cute animal (not mandatory but encouraged)

my 15 week old pup :)

This is so cool, kudos for adding e2e tests! I'd release this along with two other PR's this week. @martinjagodic would you mind helping with the docs update?


Resolved build issue with jacoco-maven-plugin

Bumped plugin version to 0.8.2.

Coverage remained the same at 50.152% when pulling 624330bb3342e9479f05ec379bcc79bc8511fd70 on rendits:master into 1fd3ec005d744f99a84db695569c639ae2b93103 on alexvoronov:master.


Tmp

Hello. :) Looks like you already closed this pull request. If you want to contribute, feel free to ask questions. You can use discord (link on readme), discussions, or just the chat in a pull request to discuss.


Some of HTTPX results will become "Encountered a bad command exit code"

So, I'm checking 300k hostnames, I split the file to 50k files as another friend suggested to avoid shadowclone to get stuck. However, some of the results that return will be "already printed"

Server, 8 vCPUs , 16 GB ram , Bandwidth 1.6 Gbps
Why this is happening and how to avoid it?
Thanks 🙏

which server are you using?

Scaleway


Egg animation added

Title and Issue number

I have added the project .Can it be merged now?
Close #5073

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md

@MohitGupta121 ,@TusharKesarwani Please review my PR.


BN_rand: fix documentation

Add missing formatting code so BN_rand docs render properly (man page, website, etc.)
Before:
"...in the range 0 <lt>= rnd < range."
After:
"...in the range 0 <= rnd < range."

just pushed into master, 1.0.2, and 1.0.1; thanks.


Angular Material Button Component W3C Errors for Generated HTML

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/fork/angular-w3c-errors-mdqmtt
Steps to reproduce:

Open the page generated by the angular code in the stackblitz link above.
https://angular-w3c-errors-mdqmtt.stackblitz.io/
Open developer tools (F12) in the page in step 1
Copy the  tag and everything inside of it
Navigate to the W3C Nu HTML Checker
Select the "Source" checkbox, and "text input" from the "Check by" dropdown
Replace the  tags with the code you have copied from the angular page.
Click the "check" button
Scroll to the errors that say "Element div not allowed as child of element button in this context."

Expected Behavior
What behavior were you expecting to see?
There should not be any markup errors thrown for the structure of the HTML generated by angular.
Actual Behavior
What behavior did you actually see?
There are markup errors thrown for the HTML structures of buttons generated by Angular Material. "Element div not allowed as child of element button in this context."

Environment

Angular: 8
CDK/Material:
Browser(s): Chrome, Firefox and IE
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

This isn't something that we support.


UpdatingStore  InMemoryDataStore support for CachedDataStoreprovider

make it possible to provide a custom store for updating

Installer lab build 18.2.301.15 includes commit UpdatingStore  InMemoryDataStore support for CachedDataStoreprovider #259, that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS


partially successful/warning condition

If a task fails with continue on error the job is triggered as partially successful. Is there a condition to pick up on this? I tried with failed() which didn't trigger and succeeded() which worked. But i don't really want it to trigger on true succeeded...

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

ID: 3f151218-9a11-0078-e038-f96198a76143
Version Independent ID: 09c4d032-62f3-d97c-79d7-6fbfd89910e9
Content: Conditions - Azure Pipelines
Content Source: docs/pipelines/process/conditions.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@msfe Did eq(variables['Agent.JobStatus'], 'SucceededWithIssues') work for you?

Hi,
No that didn't work for me. However, it was possible to extract in this way.
dependsOn: JobName1
condition: eq(dependencies.JobName1.result,'SucceededWithIssues')

Hope this helps!


java.io.IOException: An established connection was aborted by the software in your host machine

as title states
Modpack - Prominence rpg 2.8.0hf
Velocity version- Latest 3.3.0Snapshot
Fabric proxy and cross stitch installed
https://mclo.gs/ucclH7A - Client log
https://mclo.gs/GqS1r5v - Hub log
https://mclo.gs/Dj0AYNs - Proxy log

I have tried a lot in here btw, but the only real fix was not having fabric proxy, and cross stitch, or just removing all the mods 😢

more news, couple hours ago was getting an issue saying velocity isnt set up right to be forwarding which was not the case as everything was perfect, and then i attempted to try running a paper server with velocity to see if it was the proxy or modpack, wasnt the proxy so reinstalled modpack and infinite crashes on something with fabric networking api and alloy forging, removing either one doesnt solve the situation https://mclo.gs/k9iu1Cq

this issue with not launching was fixed

this is fixed, the error was the proxy not having a correct startup flag.


autocomplete not support Chinese

When I input a Chinese word, even the json file has the data, but autocomplete list is not showing ...

is this problem solved??


Empty type can't be changed.

I'm using #4017 simple type definition feature, so I can set type to empty. It will throw exception when I click Change Type.

Chrome console output:
react-dom.production.min.js:4408 TypeError: Cannot read property '0' of undefined
    at EditTypeModal.js:55
    at Array.reduce (<anonymous>)
    at hg (EditTypeModal.js:53)
    at Qi (react-dom.production.min.js:3274)
    at Ia (react-dom.production.min.js:3899)
    at Yo (react-dom.production.min.js:5514)
    at qo (react-dom.production.min.js:5536)
    at Rc (react-dom.production.min.js:5958)
    at Oc (react-dom.production.min.js:5925)
    at Lc (react-dom.production.min.js:6020)
uo @ react-dom.production.min.js:4408
n.callback @ react-dom.production.min.js:4773
no @ react-dom.production.min.js:4271
to @ react-dom.production.min.js:4259
Uo @ react-dom.production.min.js:4999
Go @ react-dom.production.min.js:5123
(anonymous) @ react-dom.production.min.js:5975
t.unstable_runWithPriority @ scheduler.production.min.js:274
xc @ react-dom.production.min.js:5974
Rc @ react-dom.production.min.js:5958
Oc @ react-dom.production.min.js:5925
Lc @ react-dom.production.min.js:6020
On @ react-dom.production.min.js:1737

EditTypeModal.js:55 Uncaught TypeError: Cannot read property '0' of undefined
    at EditTypeModal.js:55
    at Array.reduce (<anonymous>)
    at hg (EditTypeModal.js:53)
    at Qi (react-dom.production.min.js:3274)
    at Ia (react-dom.production.min.js:3899)
    at Yo (react-dom.production.min.js:5514)
    at qo (react-dom.production.min.js:5536)
    at Rc (react-dom.production.min.js:5958)
    at Oc (react-dom.production.min.js:5925)
    at Lc (react-dom.production.min.js:6020)
(anonymous) @ EditTypeModal.js:55
hg @ EditTypeModal.js:53
Qi @ react-dom.production.min.js:3274
Ia @ react-dom.production.min.js:3899
Yo @ react-dom.production.min.js:5514
qo @ react-dom.production.min.js:5536
Rc @ react-dom.production.min.js:5958
Oc @ react-dom.production.min.js:5925
Lc @ react-dom.production.min.js:6020
On @ react-dom.production.min.js:1737

It's not yet clear how Ratel should be handling empty types since the feature hasn't been released yet and not clear what version it will be released in.
It shouldn't crash when empty types are there, full support shouldn't be too hard to implement


v1.15 Backports 2024-05-11

[ ] #32367 (@sayboras)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 32367

/test-backport-1.15


bug: previous attachments cannot be loaded when editing the release

Gitea version (or commit ref): 1.13.1
Git version:
Operating system: docker

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No

Log gist:

Description
When trying to edit the release, the attachements cannot be shown(the upload region is blank), so cannot be deleted.
I have tried to save the modification, the attachements still remain, so only edit text description is okay just now.
Screenshots

duplicate of #677

duplicate of #677


[BackPort-to-1.3]Updating multi-field mapping at follower (#671) (#680)

Updating multi-field mapping at follower

(cherry picked from commit 4d12d6cfb17e2dbac02fdee815aff9bc95062b0a) (cherry picked from commit 475d8da7456fca22c0f022027b8802a14ddb0150)
Description
Updates multi-field mapping at follower side (map _type is handled in 1.x versions)
Issues Resolved
#622
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

security tests are failing with reason cp: cannot stat 'build/distributions/opensearch-security-<IP_ADDRESS>.zip': No such file or directory

Codecov Report

Merging #770 (a069caa) into 1.3 (79bf8b7) will decrease coverage by 1.44%.
The diff coverage is 76.19%.

:exclamation: Current head a069caa differs from pull request most recent head 36bfe8a. Consider uploading reports for the commit 36bfe8a to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                1.3     #770      +/-   ##
============================================
- Coverage     72.24%   70.80%   -1.44%
  Complexity      978      978
============================================
  Files           141      141
  Lines          4590     4611      +21
  Branches        507      513       +6
============================================
- Hits           3316     3265      -51
- Misses          954     1012      +58
- Partials        320      334      +14

Impacted Files
Coverage Δ

...rch/replication/task/index/IndexReplicationTask.kt
67.83% <76.19%> (+0.17%)
:arrow_up:

... and 8 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Restore widget unit tests

In PR #1461 the widget tests were deleted as they relied on SHA hashes which weren't very informative and platform dependent. It would be good to add some improved widget unit tests.

We now have a number of widget unit tests.


Support State.Error field in inspect command

What is the problem you're trying to solve
nerdctl inspect cannot return State.Error field today.
$ nerdctl run -d public.ecr.aws/amazonlinux/amazonlinux:2
2fd4c9d197320279055bfca5a109433935e762fc91bcbe371815a6cf7f39fe03
$ nerdctl inspect --format {{.State.Status}}:{{.State.Error}} 2fd4c9d197320279055bfca5a109433935e762fc91bcbe371815a6cf7f39fe03
WARN[0000] failed to inspect NetNS                       error="failed to Statfs \"/proc/4058/ns/net\": no such file or directory" id=2fd4c9d197320279055bfca5a109433935e762fc91bcbe371815a6cf7f39fe03
FATA[0000] 1 errors: [template parsing error: template: :1:26: executing "" at <.State.Error>: map has no entry for key "Error"]

Describe the solution you'd like
State.Error exists in the inspect response, and it is an existing TODO in comment. https://github.com/containerd/nerdctl/blob/e09d7f8a419ee2d1152386ab654afc55f6e2201b/pkg/inspecttypes/dockercompat/dockercompat.go#L170
Additional context
https://github.com/runfinch/finch/issues/242

Closed by https://github.com/containerd/nerdctl/pull/2073


lastgenre plugin: genres canonicalization

add genres canonicalization: when a last.fm tag is found but rejected (because not in user whitelist), try to     found a parent tag that is accepted.
I did use http://en.wikipedia.org/wiki/List_of_popular_music_genres as reference to build the tree and did read specific genres pages to better comprehend filiations between genres and move nodes in the base tree.

This has been merged (even if GitHub didn't automatically detect it).
I changed the plugin to use the "canonical:" configuration value to set the path to the YAML dictionary.
Canonicalization is also disabled by default. This is to prevent a dependency on pyyaml for people who don't need canonicalization. See the plugin docs (also written for the merge) for specifics.
http://readthedocs.org/docs/beets/en/latest/plugins/lastgenre.html#canonicalization


Largest bst

finding the largest BST in a given Binary Tree. Using C++
ISSUE #1355

@plazzy99 @tarun26091999 @rudrakshi99 Please review

Add proper indentation, with spaces around >>,<<,= operators


parse JFIF APP0 sections and expose Density with DensityUnits

See https://github.com/image-rs/image/issues/1067
Possible improvements

Rename to match png::PixelDimensions, which is not really an accurate name because it doesn't represent the size of a pixel, but the density of an image.
Remove the thumbnail stuff completely (I added it because I was there, and maybe someone wants it? I drew the line at allocating a buffer for the packed thumbnail data.)
Add tests? The current infrastructure doesn't appear capable of testing the whole range of possible inputs as PNG's density information has fewer units options. But I did verify with a dbg!() that the reftest images with JFIF APP0 sections could decode the density information.

This PR generally looks good to me, though I agree that it should probably either fully decode thumbnail images or just ignore them.


Show errors for quoted keys in entity selectors

e.g. say @e["tag"=foo]

:tada: This issue has been resolved in version 1.8.1 :tada:
The release is available on:

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Npm package not found

While doing (as per the instructions)
npm install --save-dev ember-table

Npm errors with a message saying ember-table cannot be found, also neither ember-table-addon
What's the current installation process in ember-cli? (if any)

This is expected: I haven't published to npm yet. Please see install instructions here: https://github.com/Addepar/ember-table/issues/300.


Application form doesn't support unicode

When filling out the application form, users will receive an error if they include non-ASCII characters in their answers.
Additionally, they recieve an very unhelpful error message "Could not create application!"

Fixed in https://github.com/unicsmcr/hs_application/commit/2e47edc3b3560cdba1072051f3e56668aa8872d0
Remember, if creating the DB manually to set the collate settings in MYSQL


INPUT_SDL2_ALLOW_* -> INPUT_SDL2_ALLOW_EXT

Currently we've got individual configs for INPUT_SDL2_ALLOW_GUIDE and INPUT_SDL2_ALLOW_MISC1; guide and misc1 respectively. SDL2 supports: touchpad, paddle1, paddle2, paddle3, paddle4, misc1, guide.
Library users voted down touchpad (#194) but I think in anticipation of Steam Deck it is in our interest to support the lot.

#243


Add Options to Postgres Connector?

Hi, this is sorta a question and potentially a connector update request. See below for the answers to the GH issue template, but in short, I'm wondering if it's currently possible to pass postgres connection options?
I'm using the postgres wire protocol, but not an actual postgres server, so it's helpful for configuration to be able to pass options at connection time. E.g. what I do w/ psycopg2 in python:
conn = psycopg2.connect(
    host=HOST,
    port=5432,
    user=EMAIL,
    password=PASSWORD,
    dbname=DBNAME,
    options="organization=public",
)

So basically having the options parameter available.
I also quickly tried a URL encoded connection string, but that didn't work. If it's indeed possible, and I'm just dumb, I'll have another look :).
Being able to issue commands at connection-time may also work (e.g. SET organization ...).
Thanks!
What do you use this database for?
Postgres, which is already supported
Are there any popular and well-maintained clients for this database?
pg
Are you interested in contributing this database connector?
Maybe, if it's easy :)

So we'll need to add the new setting to the options object that is exported by the postgres connector;
https://github.com/evidence-dev/evidence/blob/next/packages/postgres/index.cjs#L269-L277
This informs the UI of it's configuration options, you can make sure that the field appears properly in your local project's settings page.
Once that's appearing properly in the UI, it should just be a matter of adding it to the credentials that are passed to the postgres library
This is all on the next branch, so you'll want to branch off of that, and PR back to that.

Thanks -- I tried to test my changes locally as described in CONTRIBUTING.md, but am running into:
pnpm run dev:example-project

> @ dev:example-project /Users/thauck/personal/code/github.com/tshauck/evidence
> pnpm --filter ./sites/example-project dev

><EMAIL_ADDRESS>dev /Users/thauck/personal/code/github.com/tshauck/evidence/sites/example-project
> vite dev --port 3000

  VITE v4.3.9  ready in 878 ms

  ➜  Local:   http://<IP_ADDRESS>:3000/
  ➜  Network: use --host to expose
  ➜  press h to show help
/Users/thauck/personal/code/github.com/tshauck/evidence/sites/example-project/src/pages/+layout.js:24
                throw new Error('Unable to load source manifest. Do you need to run build:sources?');
                      ^

Error: Unable to load source manifest. Do you need to run build:sources?
    at Module.loadDB (/Users/thauck/personal/code/github.com/tshauck/evidence/sites/example-project/src/pages/+layout.js:24:9)
/Users/thauck/personal/code/github.com/tshauck/evidence/sites/example-project:
 ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL<EMAIL_ADDRESS>dev: `vite dev --port 3000`
Exit status 1
 ELIFECYCLE  Command failed with exit code 1.

Trying what it suggests leads to:
$ pnpm run build:sources
 ERR_PNPM_NO_SCRIPT  Missing script: build:sources

Command "build:sources" not found. Did you mean "pnpm run build:docs"?

hmm, I think CONTRIBUTING is not yet updated on the next branch.
we probably should add a convenience script for this but to build:sources you probably need to
pnpm i
cd sites/example-project
pnpm run build:sources
cd ../../
pnpm run dev:core-components

and in another terminal
pnpm run dev:example-project

Thanks, that got me unblocked. There's some assumptions (e.g. big  query creds), but I think I'll have something to review after the weekend.

There's some assumptions (e.g. big query creds), but I think I'll have something to review after the weekend.

If you run with pnpm run build:sources --sources psql, then that will prevent the bigquery issues from appearing


tools.go: Fix build constraints error message for the k8s.io/code-generator dependency

Description of the change:
Update the k8s.io/code-generator dependency and explicitly specify which
packages are needed in order to update codegen/mockgen, and silence the
following error locally:
error while importing k8s.io/code-generator: build constraints exclude all Go files in ...

Signed-off-by: timflannagan<EMAIL_ADDRESS>Motivation for the change:
Cleanup: silence warning on local IDEs.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive

The e2e error is probably fixed by https://github.com/operator-framework/operator-lifecycle-manager/pull/2554


Add more sensors for main.uri, based on the context, i.e public or not

Create 2 new sensors: main.uri.mapped.public and main.uri.mapped.subnet
Requires https://github.com/apache/brooklyn-server/pull/378

@aledsage Comment addressed

Thanks @tbouron - merging now.


Jest + Preact

I bet this question has been asked a lot. I have searched for two hours and I am not able to find a solution. There doesn't seem to be much about preact with jest testing. I looked at the boilerplate and it uses Karma.
Problem
When trying to use Jest with Preact I get the following error:
FAIL  src/widget/index.test.js
  ● Test suite failed to run

    Cannot find module 'react/lib/ReactComponentTreeHook' from 'ReactDebugTool.js'

      at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:169:17)
      at Object.<anonymous> (node_modules/react-test-renderer/lib/ReactDebugTool.js:16:30)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        0.601s

My setup
I followed the guidelines to setup babel with webpack. My .babelrc is
{
    "plugins": [
        ["transform-react-jsx", { "pragma":"h" }],
        "transform-class-properties"
    ],
    "presets": [
        ["es2015", {
            "modules": false
        }]
    ],
    "env": {
        "test": {
            "plugins": [
                "transform-es2015-modules-commonjs",
                ["transform-react-jsx", { "pragma":"h" }],
                "transform-class-properties"
            ]
        }
    }
}

package.json uses identity-obj-proxy as suggested
 "jest": {
    "moduleNameMapper": {
      "\\.(css)$": "identity-obj-proxy"
    }
  }

And finally my test looks like
import Widget from './index';
import renderer from 'react-test-renderer';

test('Tests for widget to have been rendered', () => {
    const tree = renderer.create(
        <Widget href="http://test.com" target="_blank" data-variation="test"
                innerHTML="test foo bar"/>
    ).toJSON();
    expect(tree).toMatchSnapshot();
});

I am losing my mind.
Questions

Should react-test-renderer work with preact?
Is there a better way to do this?

Possibly related to https://github.com/developit/preact-compat/issues/249 and https://github.com/airbnb/enzyme/issues/715

react-test-renderer will likely never work with preact.  preact-render-to-string support very similar functionality (including shallow rendering and JSX output with support for complex props).  It doesn't currently support JSON output, which means Jest snapshots get double-encoded.
I'd love an experimental PR for JSON output if anyone has the time - I haven't had time yet since I'm only just looking into Jest myself.
FWIW I think most people test Preact components using the preact-jsx-chai plugin.  It uses preact-render-to-string under the hood, and has a nice syntax:
expect(
  <Things items={['a', 'b']} />
).to.equal(

    <Item name="a" />
    <Item name="b" />

);

Great. This helps a lot. I'll close ticket.

Might help someone: I've been testing my Preact components with Jest and html-looks-like. It works fine and is pretty easy to set up.
import { h } from 'preact';
import render from 'preact-render-to-string';
import htmlLooksLike from 'html-looks-like';

const Thing = () => (

    Nope
    Nope
    Yep

);

describe('Thing', () => {
  it('has Yep', () => {
    const actual = render(<Thing />);
    const expected = `

        {{ ... }}
        Yep

    `;

    htmlLooksLike(actual, expected);
  });
});

That is a ridiculously cool technique @rmsaksida - will be stealing it for some of my stuff haha. I love the syntax for ignoring parts of the tree!

For anyone interested in using Jest for Preact environments, I've set up a PR for this at preact-boilerplate.

Anyone could test components that use react components as childrens? I'm having a good trouble with react-intl and react-select

@armand1m shouldn't make a difference as long as you're aliasing preact-compat.

@developit So, I'm aliasing react with preact-compat into my .babel-rc and webpack.config.js. When bundling the project, everything were working nice. When running it with jest, it could render simple elements that didn't use third-party react components nicely, but when trying to run a specific component using react-select, it was asking for react to render it.
I started having this problem with 'react-intl', have migrated to 'preact-intl' and have trespassed this problem.
Then I started having problems with 'react-select'.
When trying to alias react into my package.json, inside the "jest" key, every test case was broken.
It was trowing an error like:
Type Error: ( 0, _preact.h ) is not a function

I didn't had much time and mocked the 'react-select' to render what I needed from my component.
I can send more detailed information later, but I couldn't find other easy way to render this component into a jest test harness.

Ah - that error makes it seem like your JSX Pragma setting is being applied to all of node_modules, which will not work.  I'm not overly familliar with Jest, but there must be a way to tell it to only transpile first-party code (code in your repo).  If you can get it to do that (babelrc:false maybe?) then it should work.

@developit Nice, I will try it later and give some feedback after. Thank you!

@developit Got it working. It was actually because when you specify a module to jest configuration in package.json, you're specifying actually a Regexp. When using like this:
{
  "jest": {
    "moduleNameMapper": {
      "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$": "<rootDir>/__mocks__/fileMock.js",
      "\\.(css|less|scss)$": "identity-obj-proxy",
      "react": "preact-compat",
      "react-dom": "preact-compat"
    },
    "collectCoverageFrom": [
      "app/**/*.{js,jsx}"
    ],
    "moduleFileExtensions": [
      "js",
      "jsx"
    ],
    "moduleDirectories": [
      "node_modules"
    ],
    "setupFiles": ["<rootDir>/jest-setup.js"]
  }
}

It will not even run a component, because it is looking actually at every component that has "react", for example, in its name, and replacing their calls to preact-compat.
Got it working using this jest config in my package.json:
{
  "jest": {
    "moduleNameMapper": {
      "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$": "<rootDir>/__mocks__/fileMock.js",
      "\\.(css|less|scss)$": "identity-obj-proxy",
      "^react$": "preact-compat",
      "^react-dom$": "preact-compat"
    },
    "collectCoverageFrom": [
      "app/**/*.{js,jsx}"
    ],
    "moduleFileExtensions": [
      "js",
      "jsx"
    ],
    "moduleDirectories": [
      "node_modules"
    ]
  }
}

and this .babelrc:
{
  "sourceMaps": true,
  "presets": [
    ["es2015", {
      "es2015": {
        "loose": true,
        "modules": false
      }
    }],
    "stage-0"
  ],
  "plugins": [
    ["transform-decorators-legacy"],
    ["transform-react-jsx", { "pragma":"h" }]
  ]
}

The slight difference is in the regex for aliasing react and react-dom

@rmsaksida That looks awesome! Do you have any repository with the whole setup that you can share? Thanks!

would love to see where he's used it too!
FWIW we just merged a PR that switched preact-boilerplate over to Jest. It's quite awesome!

I have been using preact-render-to-string like the example over here + jest-serializer-html-string to sanitize html output with quite some success until now

@ruyadorno that's a really nice solution. going to give it a try and maybe integrate it into the boilerplate!

@armand1m thanks. It works like a charm

@haikyuu I'm glad it helped you mate 😄


test filed with error

i tried to use ui test with app and it field
The following assertion was thrown running a test:
Object/factory with  type Repository is not registered inside GetIt.
(Did you accidentally do GetIt sl=GetIt.instance(); instead of GetIt sl=GetIt.instance;
Did you forget to register it?)
'package:get_it/get_it_impl.dart':
Failed assertion: line 370 pos 7: 'instanceFactory != null'

@zubairehman ??


Any owner can steal the group making primary ownership almost useless

If this permission can be toggled for OWNER, primary owner is almost useless because any other owner can steal the group by enabling this perm for admins, demoting themselves to admins and disabling the perm for owners. And disabling it for owners preemptively locks the ability to change the group's perms. Locking the permission for owners will prevent this.
https://github.com/CivMC/NameLayer/pull/24

I'm closing this out as something that we won't fix. It could be revisited in the future, but it's not within our capacity to work on right now.


update requirements

I just updated the requirements using grayskull pypi sepal_ui

You should never merge anything into "main" per se without changing the version numbers and the auto PR are here for that. I'll redirect this to the latest update and see if it solves the tests

closing as I was unable to redirect the branch to the correct destination. I included your modifications to the latest release and I will finally manage to add you to the maintainer list.


TransformOperation can't be used in manual OperationInput or OperationOutput implementations

Let's suppose one has a common Auth axum extractor which is responsible for authorizing incoming requests, returning 401 responses in case of error. Assuming that, for some reason the OperationOutput needs to be manual (and not generated by the OperationIo macro), one possible implementation would be to:
impl OperationInput for Auth {
    fn operation_input(ctx: &mut GenContext, operation: &mut Operation) {
        let t = TransformOperation::new(operation);
        _ = t.response::<401, http::Error>();
    }
}

Notice that, to reduce boilerplate code and leverage all existing TransformOperation convenience methods (which also provides some error checking under the hood), the implementation wraps the &mut Operation over an TransformOperation.
However, that wouldn't work. When executing the program, the following panic is raised:

already borrowed: BorrowMutError

The errors happens due to the fact that, when aide invokes the operation_input method, it mutably borrows the GEN_CTX thread local, keeping a reference until after such a method finishes. Then, inside operation_input, while the former borrow is still alive, the TransformOperation::response method also tries to borrow it, resulting in a panic.
I don't think if TransformOperation is intended for such manual implementation use cases. However, I really think it should, as it allows for a much nicer and robust interface.
I am opening this issue to try to think about options to solve this issue. See, if operation_input gets a &mut Operation, it would make sense to make one able to create a OperationTransformer over it, right?
A viable option I see is to create a new transformer, OperationTransformerWithCtx, which could be created with both &mut GenContext and &mut Operation. Since it would be more general, to avoid duplication, the current OperationTransformer could be internally reimplemented to use its with ctx counterpart.
WDYT?

I'm also running into this issue and have not found a workaround. Manually editing the operation does not seem very efficient, so it would be nice if this was supported.


feat: add rust target

Added the rust cosmos proto generation

Created a generation script writen in rust that take the proto in the directory and put it in the rust/cosmos-sdk dir and in the rust/okp4 dir
Added .toml project definition in the rust/cosmos-sdk dir and in the rust/okp4 dir
Added .releaserc infos to update the version in both toml project definition
Added pipeline steps to build & publish the package

I will remove unused deps and we are ok to make it for review :)

@warstrolo Could you please add a description on the changes?

You mean in the commits ?

@warstrolo here:

Okay will do it !

@bdeneux


interaction history

fixing undo/redo history problems with interactions, and in the process trying to make the code more sane

latest commit addresses https://github.com/vega/lyra/issues/514 and https://github.com/vega/lyra/issues/513


Initial changes after forking

To tailor to Scanner's use case.
See commit messages for details.

Meant to open this in my fork. Sorry!


OTP 23 support

Fix for #16

Hello, @nmbrone! This is your first Pull Request that will be reviewed by SourceLevel, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

Closing this as it's broken on OTP < 23

Hello @nmbrone, thanks for your pull request and for your efforts so far. I'm going to take a deeper look into this very soon.


Feed full of not wanted answer tweets

Checklist

[X] I am able to reproduce the bug with the latest version.
[X] I made sure that there are no existing issues - open or closed - which I could contribute my information to.
[X] I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
[X] This issue contains only one bug, that only occurs in Squawker

Affected version
3.8.3
Steps to reproduce the bug
Just use it as usual.
Expected behavior
Just show the subscribed tweets in the feed.
Actual behavior
Since a few days i get a lot of answer postings in the freed even when answers are deactivated. The feed is flooded with such answers. It happens with more and more accounts.
Screenshots/Screen recordings

Logs
No response
Affected Android/Custom ROM version
/e/ os
Affected device model
xiaomi mi 8
Additional information
No response

I believe I am also facing the same issue. I logged on today and I am suddenly seeing a lot of response tweets from accounts I don't follow at all.

same problem

Same issue.

Same. Just started today.

This is easier to replicate and happens more often in Groups.
It seems like it's showing the "For you" feed instead of the accounts you follow. Using an account that follows nobody on the site will display tweets that may fall under the topics you mark as "interested" when creating an account, and it's mixed with the ones you have on Squawker.

Same issue, more noticeable over the past few days. It's making X/Twitter fairly unusable at the moment. I'm not using any groups - only the feed.

Same issue for me as well. In mian feed and all my groups. It seems I am getting tweets from people who are followed by the people I subscribed too.

Anecdotally, it appears to happen (or happen more) when the accounts I am subscribed to have not posted anything new for a while yet I continue to refresh the feed.

Same issue here, unable to read my feed in a proper way

None of my groups feeds are showing up now even for groups, but when I click on the "Subscriptions" and on each person I follow, their posts show up fine.
Would it be possible to have an option to show only the followed accounts under a specified group only as a counter-measure to whatever is happening on Twitter?
For example, if I click on the "Animals" group then it will show:
CatPosts
DogPosts
HamsterPosts
RabbitPosts
TurtlePosts
BearPosts
PuppyPosts
KittenPosts
BunnyPosts
etc.
Instead of having to scroll 1,000+ followed accounts for each list when I just want to see posts with pictures of cute animals..
I could've sworn I requested this feature before, but it might've been for the client I used before switching to this one..
At any rate, this would be a good workaround to any possible/potential sabotage by "lonE skuM" especially considering the timing..

same issue. makes this app unusable

I've been experiencing this for a couple weeks too. It seems like you can temporarily fix it sometimes by toggling replies or retweets on/off (in the 3 dot menu), but it's at best inconsistent.

Hello,
I just started having this issue. A friend esportes having it several days ago, but for me this is the first time.
It's really weird because I see no pattern to it: I had my normal tweets up to around 9 AM, then dozens of tweets and replies from people I had never seen, without a connection between them; then at 12h47 a few more "normal" tweets, then 10 seconds later a random tweet, etc.
I only follow 8 accounts, most of which tweet very little. I have no groups. I tried toggling "Include replies" and "Include retweets", but to no avail. I tried erasing the app cache and reloading, but the unwanted tweets are still there.

New data: I opened the application today, and it contained a few tweets from accounts I follow, until up to 3 hours ago. When at the top of the new tweets, I dragged down to refresh, and then I had about 15 "random" tweets appear. I don't know if it's a coincidence, but I'll see if this pattern persists and try not to "manually" refresh the list.


[request] Add metadata to Documents and Tables

Something I find missing in Grist is the ability to add metadata to Documents, Tables and even Columns.
For example:

description, use cases, explanation, instructions: text field
main author with email

This feature is available in Airtable and comes very handy. It is accessible as a little "(i)" in the column header, for example.

Sounds like a good idea. Would you be interested in the ability to set up document tours, like the templates at https://templates.getgrist.com/ have?

Yes that would also be a great idea, but seems more complex to implement than plain text fields attached to main Grist objects.


common storage function

https://capacitorjs.com/docs/apis/cookies
https://capacitorjs.com/docs/apis/preferences
https://web.dev/learn/pwa/offline-data/

typed session & storage


V1 Timecard response incomplete

The endpoint for V1 Timecard has fields that are not currently returned by the SDK (eg overtime_seconds_worked). From investigation it looks like these fields are related to the payroll system which is currently only available in the US?
https://docs.connect.squareup.com/api/connect/v1#datatype-timecard

Should have been resolved some time ago. If not, please reopen.


Gocomics module does not work

When the comic landing page has a "Read *** from the Beginning" link, the script gets the "first comic" right, it downloads it but then stops with a "next/first comic will be None"
C:*> python comicbookmaker.py --comic brevity --loglevel 10
DEBUG:root:Starting
DEBUG:root:update brevity start
DEBUG:root:_create_output_dir brevity start
DEBUG:root:_create_output_dir brevity done
DEBUG:root:_load_db_from_file brevity start
DEBUG:root:update brevity last comic is https://www.gocomics.com/brevity/2005/01/03
DEBUG:root:get_next_comic brevity starting 'get_next_comic' from https://www.gocomics.com/brevity/2005/01/03
DEBUG:root:get_content (url : https://www.gocomics.com/brevity/2005/01/03)
DEBUG:root:urlopen_wrapper (url : https://www.gocomics.com/brevity/2005/01/03)
DEBUG:root:get_next_link brevity Next link is None
DEBUG:root:get_next_comic brevity next/first comic will be None (url is https://www.gocomics.com/brevity/2005/01/03)
brevity : nothing new
DEBUG:root:update brevity done
When the comic landing page has no "Read *** from the Beginning" link, the script apparently gets the last comic's url instead of the first, doest not download anything and then stops with a "next/first comic will be None":
C:*> python comicbookmaker.py --comic peanuts --loglevel 10
DEBUG:root:Starting
DEBUG:root:update peanuts start
DEBUG:root:_create_output_dir peanuts start
DEBUG:root:_create_output_dir peanuts done
DEBUG:root:_load_db_from_file peanuts start
DEBUG:root:update peanuts last comic is None
DEBUG:root:get_next_comic peanuts starting 'get_next_comic' from None
DEBUG:root:get_content (url : http://www.gocomics.com/peanuts)
DEBUG:root:urlopen_wrapper (url : http://www.gocomics.com/peanuts)
DEBUG:root:get_content (url : http://www.gocomics.com/peanuts/2019/03/30)
DEBUG:root:urlopen_wrapper (url : http://www.gocomics.com/peanuts/2019/03/30)
DEBUG:root:get_next_comic peanuts next/first comic will be None (url is None)
peanuts : nothing new
DEBUG:root:update peanuts done
and
C:*> python comicbookmaker.py --comic calvin-goc --loglevel 10
DEBUG:root:Starting
DEBUG:root:update calvin-goc start
DEBUG:root:_create_output_dir calvin-goc start
DEBUG:root:_create_output_dir calvin-goc done
DEBUG:root:_load_db_from_file calvin-goc start
DEBUG:root:update calvin-goc last comic is None
DEBUG:root:get_next_comic calvin-goc starting 'get_next_comic' from None
DEBUG:root:get_content (url : http://www.gocomics.com/calvinandhobbes)
DEBUG:root:urlopen_wrapper (url : http://www.gocomics.com/calvinandhobbes)
DEBUG:root:get_content (url : http://www.gocomics.com/calvinandhobbes/2019/03/30)
DEBUG:root:urlopen_wrapper (url : http://www.gocomics.com/calvinandhobbes/2019/03/30)
DEBUG:root:get_next_comic calvin-goc next/first comic will be None (url is None)
calvin-goc : nothing new
DEBUG:root:update calvin-goc done

I've tried several non-gocomics modules and everything works fine.

Thanks for your feedback, I'll have a look at this!

I include several "get_soup_at_url" logfiles
get_soup_at_url.zip

Thanks! I'll have a look at this in the next days :-)
Le sam. 30 mars 2019 à 15:40, bakisfigus<EMAIL_ADDRESS>a
écrit :

I include several "get_soup_at_url" logfiles
get_soup_at_url.zip
https://github.com/SylvainDe/ComicBookMaker/files/3025588/get_soup_at_url.zip
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/SylvainDe/ComicBookMaker/issues/83#issuecomment-478251963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABL0rWoYauZtXsf74i1QK2mjW0be66eBks5vb3dqgaJpZM4cTtub
.

Note: https://github.com/SylvainDe/ComicBookMaker/commit/cd785baf0312f35cd1653a3234b63b8f79398819 is related to this issue as well.

From what I can see in the raw file:

From what I can see in the content file:

It seems like a whitespace after the "sm" disappears for some reason. I do not know if this is a pretty-printing feature of BeautifulSoup...
@bakisfigus Can you give it a try after removing the final whitespace after sm in https://github.com/SylvainDe/ComicBookMaker/blob/master/comics.py#L5171 ?

Good news. After removing the final whitespace after sm in https://github.com/SylvainDe/ComicBookMaker/blob/master/comics.py#L5171, the script correctly downloaded the first comic (tried calvin-goc and peanuts, both with no nav landing page) and then stopped.
then I tried to remove also the final whitespaces in  https://github.com/SylvainDe/ComicBookMaker/blob/master/comics.py#L5176 and  https://github.com/SylvainDe/ComicBookMaker/blob/master/comics.py#L5176 and now it works like a breeze!

Thanks @bakisfigus for the feedback.
At first I suspected that maybe the OS could have an impact on this but now that I retry on a freshly created venv on Linux, I observe the same behavior as yours.
This could be related to https://bugs.launchpad.net/beautifulsoup/+bug/1787453 .
If I get a bit of spare time, I'll try to investigate further. In the meantime, I'll submit a commit to fix this while ensuring backward compatibility.

@bakisfigus This should now be fixed, thanks again for your feedback :)
I'll keep the issue opened to try to find more about this behavior change in BS4.

I've opened this issue: https://bugs.launchpad.net/beautifulsoup/+bug/1824502 .

Note: if the same kind of issue happened somewhere, it might be worth defining a Soup wrapper to handle compatibility issues like this one.

Not directly related to this bug but here are my notes if I need to tests bs4 again in the future:
     sudo apt-get install bzr
     bzr branch lp:beautifulsoup
     cd beautifulsoup/

     python3.6 -m venv bsenv
     source bsenv/bin/activate

     pip install 2to3
     pip install soupsieve

     2to3 -w bs4

     python -m unittest discover -s bs4


Wrong word count

The word count of the text analyzer section is 1 less than the actual count.
Actual word count = 6;
Initial word count = 5;

Hacktoberfest 22'
@Susmita-Dey can I get assigned to it, please?

@Susmita-Dey Hey I'm not able to work it out, can you please help me with this?

@Susmita-Dey Hey I'm not able to work it out, can you please help me with this?

Check where is the word counter in the file and add +1.


Batch size for model predict.

First of all, thank for provide such a useful algorithm to predict BVP and respiratory rate. My questions are:

Why do we need to set the batch size for model prediction in this scenario (for predict BVP and respiratory rate) ?
How do I determine the batch size?
Will the batch size affect the predict results?

just make inference faster
depend on your gpu memory
no


ci(code-scanning): add CodeQL workflow

Description
After stumbling over this project recently and seeing that #3346 was closed, I decided to give this a try. (I like CI)
This PR adds a basic CodeQL workflow with the default scanning queries for c++ and c#.
It utilizes a basic dependency cache to slightly speed up the workflow, however still has a runtime of ~25min as a whole.
Changes

adds a CodeQL workflow

Issues

n/a?

@h1dden-da3m0n Thank you! Congrats on your first PR 🎉.


[Spring] Add sdk for 23 11 01 preview

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
N.A.
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Spring


"Run Python File" does not process launch.json

As the title says, launch.json seems to be ignored when clicking "Run Python File".
The launch.json is properly read for "Debug Python File"
Since parameters for the program are defined there this means that the program will fail (or give the wrong result).
Current workaround is to toggle off all breakpoints then run in debug...

Duplicate of #11812


Prevent duplicate permitted next nodes

Manually specified permitted next nodes
It's possible for a developer to accidentally include the same node key in the list of permitted next nodes multiple times. However, from a visualisation point-of-view, we don't want the same node to appear more than once. I'm not sure there are any duplicates in the existing flows, but it is a theoretical possibility and so it seems sensible to defend against it.
Automatically detected permitted next nodes
Likewise, when auto-detecting permitted next nodes, it's legitimate for a next_node block to have multiple execution paths which return the same next node key. But we still don't want the same node to appear more than once in the visualisation. I've been working on converting all the existing flows to use the auto-detect mechanism and I know that (without this fix) some of them would indeed generate duplicates like this.
Expected changes
None

It turns out there are a couple of duplicates in the existing flows - I'm going to add a couple of commits to this PR to remove them.

I've now pushed the two extra commits to remove the duplicates and updated the PR description to explain the removal of the duplicates.

As far as I can tell, the GraphPresenter already handles the case where there are duplicate next nodes specified in the list of permitted next nodes, i.e. the same node won't appear more than once in the visualisation. I checked this by looking at the visualisation of the two flows updated in this branch:

https://www.gov.uk/calculate-statutory-sick-pay/visualise

The maximum_entitlement_reached node is only shown once

https://www.gov.uk/pay-leave-for-parents/visualise

The outcome_mat_leave_mat_pay node is only shown once

Having said all that, it certainly does seems more correct for #permitted_next_nodes to return a unique set of nodes. So, this all looks good to me.

@chrisroos: Thanks. You're right that the GraphPresenter does seem to cope - I think it must be this conditional. However, that behaviour doesn't seem to be tested and the code is pretty hard to follow. And, as you say, I think it makes more sense for #permitted_next_nodes to return a unique set of nodes.

Rebasing against master and force-pushing in preparation for merging.


Improve test data on Dydra repository

Created dedicated test repo at http://www.data.maudry.com:3030/test/sparql


all: replace collide mode by the detached call property

Work on this PR is currently in progress
Replace the currently existing straightforward approach to race triggering (that was almost entirely implemented inside syz-executor) with a more flexible one.
The detached call property tells syz-executor that it should not await until the call returns and can immediately proceed to executing the next one. The decision on what calls to mark with detacted is made by syz-fuzzer.
Ultimately this should let us implement more intelligent race provoking strategies as well as make more fine-grained reproducers.

A legit failure on CI:
                <stdin>:595:21: error: 'inject_fault' defined but not used [-Werror=unused-function]
Error: error: 'inject_fault' defined but not used [-Werror=unused-function]
                cc1: all warnings being treated as errors

                compiler invocation: /syzkaller/netbsd/tools/bin/x86_64--netbsd-g++ [-o /tmp/syz-executor260803489 -DGOOS_netbsd=1 -DGOARCH_amd64=1 -DHOSTGOOS_linux=1 -x c - -m64 --sysroot /syzkaller/netbsd/dest/ -O2 -pthread -Wall -Werror -Wparentheses -Wframe-larger-than=16384]

A legit failure on CI:
                <stdin>:595:21: error: 'inject_fault' defined but not used [-Werror=unused-function]
Error: error: 'inject_fault' defined but not used [-Werror=unused-function]
                cc1: all warnings being treated as errors

                compiler invocation: /syzkaller/netbsd/tools/bin/x86_64--netbsd-g++ [-o /tmp/syz-executor260803489 -DGOOS_netbsd=1 -DGOARCH_amd64=1 -DHOSTGOOS_linux=1 -x c - -m64 --sysroot /syzkaller/netbsd/dest/ -O2 -pthread -Wall -Werror -Wparentheses -Wframe-larger-than=16384]

Yes, I'll push some fix as a separate PR soon. It's because of this
https://github.com/google/syzkaller/blob/1d849ab4d892af13a249f75628790b47e42b7c74/pkg/csource/csource.go#L221-L223

Update #612

With clang I am getting this warning:
SYZ_CLANG=1 make executor
go list -f '{{.Stale}}' ./sys/syz-sysgen | grep -q false || go install ./sys/syz-sysgen
make .descriptions
bin/syz-sysgen
touch .descriptions
mkdir -p ./bin/linux_amd64
clang -o ./bin/linux_amd64/syz-executor executor/executor.cc \
	-m64 -ferror-limit=0 -O2 -pthread -Wall -Werror -Wparentheses -Wunused-const-variable -Wframe-larger-than=16384 -static  -DGOOS_linux=1 -DGOARCH_amd64=1 \
	-DHOSTGOOS_linux=1 -DGIT_REVISION=\"e93b0f14e29c27d88c890e9940ef586febaa593a\"
executor/executor.cc:667:7: error: variable 'has_fault_injection' set but not used [-Werror,-Wunused-but-set-variable]
        bool has_fault_injection = false;
             ^
1 error generated.

Would be good to test executor build with clang on CI as well.

@dvyukov PTAL

https://github.com/google/syzkaller/pull/2788/checks?check_run_id=3826147175
                <stdin>:594:12: error: 'inject_fault' defined but not used [-Werror=unused-function]
Error: error: 'inject_fault' defined but not used [-Werror=unused-function]

@dvyukov @tarasmadan I added two more commits. PTAL.

Rebased on top of the master after the merge of https://github.com/google/syzkaller/pull/2882 and https://github.com/google/syzkaller/pull/2904

Now the slowdown from the extra executions should be less noticeable.
Also, I decreased the minimal output area size as now we don't always collect signal.


CORGI-257 persist errata modules

The errata tool builds list requires extra processing to capture modular builds. This change looks for modular builds in the errata builds list and prepares the modular data for processing by the existing Brew.persist_module function. It also switches slow_load_errata to use slow_fetch_modular_build instead of slow_fetch_brew_build.

@RedHatProductSecurity/corgi-devs please review


Templates

I've been a lot about how annoying it is to print a little formatted string in Wren right now. I originally had the overloads in IO.print(...) so you could do:
IO.print("Hi, ", name, "! You are ", year - birth, " years old.")

 Also, it only works for print(). There's no easy way to just create a formatted string.
Something like printf()-style format strings aren't a good fit for Wren because we don't have varargs. Also, I never liked the separation between the formatted value and its position in the string.
All of this leads towards string interpolation—a way to insert expressions in the middle of a string literal that get evaluated and the stringified result is placed into the string.
Stuff like:
"Hi, #{name}! You are #{year - birth} years old."   # Ruby
"Hi, ${name}! You are ${year - birth} years old."   # Dart
"Hi, %(name)! You are %(year - birth) years old."   # Python
"Hi, {{name}}! You are {{year - birth}} years old." # Mustache et. al.

It's a nice feature, but it feels kind of special-casey to me. You'll note that even languages that have interpolation rarely use it for things like web templating. However, there is a way to really kick it up a notch: let user code control the interpolation.
This is what tagged template strings in ES6 do. I've wanted to do something similar for several years. Yesterday, I realized that adding this to Wren would actually be pretty easy. So that's what this branch is. I'd like your feedback on it.
Syntax
Any string literal can contain template expressions, delimited by %( and ):
"Hi, %(name)! You are %(year - birth) years old."

Any expression is allowed inside the parentheses, including other template strings:
"%( "%( "inner" )" ) "

Why %?
I considered #, $, and %. I think the latter is the most intuitive once we introduce format strings (below).
Why ()?
I considered (), {}, and {{}}. The latter is more verbose. {} is more common, but is also a bit more visually noisy. Given that {} is used for things like maps, I think it could look weird in some cases:
"interpolate an empty map: %{{}}"

I like that () is visually clean and it does roughly convey the right impression regarding the grammar. The contents of it are any expression, which is the same as what's allowed inside () when it's used for grouping elsewhere.
This naturally extends to possibly supporting unparenthesized templates for simple primary expressions like names:
"point = %x, %y, %z"

I'm not sure if I want to support that (it might not play nice with format strings), but it might be cool.
Also, I just think %() looks nice.
Semantics
From the user's perspective, when they write a template string, they just get a string object back with all of the template expressions evaluated and inserted in the result:
System.print("The sum is %(3 + 4)") // "The sum is 7".

What's going on under the hood is kind of interesting, and leads nicely to tagged templates below.
String templates are just syntax sugar. Given this string template:
"Hi, %(name)! You are %(year - birth) years old."

The compiler sees:
String.template([
  "Hi, ",
  Fn.new { name },
  "! You are ",
  Fn.new { year - birth },
  " years old."
])

The template becomes a list of parts. The text sections become string literals. Each template expression becomes a function that evaluates the expression and returns the result. Then this list is passed to String.template().
That method does what you expect:
static template(parts) {
  var result = ""
  for (part in parts) {
    if (part is String) {
      result = result + part
    } else {
      result = result + part.call().toString
    }
  }

  return result
}

Since functions are closures in Wren, template expressions can use surrounding variables, this, the whole works.
Tagged templates
This desugaring leads to the natural question, "Can I pass the list of parts to something other than String.template()"? The answer will be "Yes!".
I'm still figuring out the syntax here, but it will be something along the lines of:
Foo "Some %(template) string"

This gets desugared to:
Foo.template(["Some ", Fn.new { template }, " string"])

Now Foo has complete control over how those template expressions are handled. This is important because it lets you do things like:
Html "<body>%(content)</body>"

Then the Html class could, for example, ensure that the interpolated expression gets HTML sanitized before it gets placed in the resulting string.
A custom template handler could even evaluate the template parts later or multiple times, making it useful for things like data binding and other "template-y" use cases.
Format strings
I have some rough ideas for one more small addition. Even in languages with string interpolation, doing things like custom numeric formats or padding can be a chore because you have to make it a named method call in the expression:
"Some hex number %(someNumber.toHexString)"

The format specifiers in printf() are actually pretty nice for this. So what I'm thinking is we can expand the syntax to allow a "format string" between the % and (:
"Some hex number %x(someNumber)"

That gets passed to the template processor too. So the above desugars to something like:
String.template([
  "Some hex number ",
  ["x", Fn.new { someNumber }]
])

Then the template processor can handle that as it sees fit.
Pros

Much easier to create nice formatted strings out of output. It's a really handy bread-and-butter feature for little scripts and command-line programs. Likewise, for printf()-style debugging. Once you have interpolation, you find yourself using it all the time.

Can be used anywhere instead of baking it into certain printing APIs.

Because template expressions are parsed at compile time, we get all of the compile time grammar and name validation you expect.
Unlike, say, .format() in Python, template expressions in Wren should be safe and fast.

Cons

A consequence of the above pro is that template strings cannot be created at runtime. It's not a backdoor to eval. This is good for performance, simplify, and safety, but may be a limitation in some cases. For example, a Wren program could not load a template string from a file and then evaluate it.

One interesting consequence is that tokenizing is no longer regular. This makes it harder to parse Wren using some simple parser generators, and may make things like syntax highlighters a little more annoying to create.

You have to escape % in strings. I thought about having a separate syntax for "template" strings and only allow %() there, but I want to avoid a proliferation of different string types. I'd rather have one string syntax that supports everything. I picked % because I think it's fairly uncommon so escaping should be rare.

One more feature to add.

Status
Right now, I just have the basic string template syntax working. The actual code change in the VM is pretty small: 50 lines of non-comment code.
I don't think tagged templates or format strings will add much to that. (Actually implementing various numeric format strings for numbers might, though.)
I updated the benchmarks and tests to take advantage of this, and I think it really is a readability win.
What do you think?

Everything above sits well in my view, with my understanding of wren: Clean, simple, flexible.
String interpolation is way better than vararg style formatters,
because the value isn't separated from the natural expression being formed.
"This %method %is clear even %(with complex expr)
vs
"This %d isn't % longer", method, "especially when")
I've been using compile time only string interpolation a few years, yet I haven't felt it missing from runtime. I think that's because I view runtime (and especially) templating engines a bit more specific use case. Most of the ones listed like mustache don't usually deal with arbitrary expressions as much as simple context -> values, and helper functions. The facilitation mentioned above would account for a runtime convenience sharing the same code.
Feeding a context of values + functions with a string and it will run through it seems like it could be a module (if not a std library feature) anyway.
For interest sake, which you may know,
haxe string interpolation is bound to ' single quote strings (where haxe has " as well), and $variable and ${expr} are used. It's compile time only and any expression is valid.

I never liked the separation between the formatted value and its position in the string.

I think the main point here is localization. Translations will commonly swap around the position placeholders. This is why Qt uses numbered placeholders, for example. Though I personally think named ones would be even better. But it can't be direct code, otherwise you can't swap the string for a translated version.

I've been using compile time only string interpolation a few years, yet I haven't felt it missing from runtime. I think that's because I view runtime (and especially) templating engines a bit more specific use case.

Yeah, I feel the same way. There is still a use case for pure runtime templating (localization like @bjorn brings up is another big one), but for most uses, I think compile time is fine, and I like the compile-time syntax checking it gives. I figure the parser is already in the middle of parsing things at compile time anyway, so that's a good time to parse format strings to, instead of deferring it until later.

Feeding a context of values + functions with a string and it will run through it seems like it could be a module (if not a std library feature) anyway.

Yes, I think that may be worth doing eventually too. With this, I'm going for sort of the 80% common use case while still getting simple enough to stick in the core VM. (I try to keep the VM really small, whereas the modules that come with the CLI aren't really size limited.)

is bound to ' single quote strings (where haxe has " as well), and $variable and ${expr} are used.

Single quotes are an interesting idea. My hunch is that most languages that use a different kind of string for interpolation do so because they added it later and didn't want to break compatibility. Since we don't have that problem with Wren, I figured sticking with one kind of string made sense.
I considered $ as the magic character. That's what Dart uses too. But it's often a valid identifier character too, which leads to a weird edge case with things like "some string $foo$bar". I went with % partially to avoid that.

I think the main point here is localization. Translations will commonly swap around the position placeholders.

Yup. This definitely isn't a good solution for localization. But then, with that case, you're often loading the template strings from a separate data file too, and this doesn't work for that either.
You could try to use this with localization by wrapping the string in a function:
var english = Fn.new {|fields| "Hi %(fields["first"]) %(fields["last"])!" }
var serious = Fn.new {|fields| "Greetings, %(fields["last"]), %(fields["first"])." }

Then pass that function around as a "template". But my hunch is that would be a bit of a chore.

I landed simpler interpolation support without custom templates so we're going to leave it at that for now.


[feat] Add cache purge support to the flarectl CLI.

The upcoming Purge by Host functionality has been added to the library
flarectl now has a purge sub-command for purging from cache.

e.g
# Purge by host (upcoming)
flarectl zone purge --zone=example.com --hosts="repeater.example.com"

# Purge everything
flarectl zone purge --zone=example.com --everything

# Purge by filename
flarectl zone purge --zone=example.com \
  --files="http://repeater.example.com/styles.css,http://repeater.example.com/javascript.js"

@jamesog That's my 🎄🎄🎄 Holiday Project ® 🎄🎄🎄


Service 'tor_countries' timed out.

Hi there,
I'm experiencing an issue with this plugin.
From munin-node.log I see the following messages:
2016/01/04-16:40:08 [13960] Error output from tor_connections:
2016/01/04-16:40:08 [13960]     No response from Tor daemon in TorConnection.fetch()
2016/01/04-16:40:08 [13960] Service 'tor_connections' exited with status 255/0.
2016/01/04-16:40:45 [13959] Error output from tor_countries:
2016/01/04-16:40:45 [13959]     Traceback (most recent call last):
2016/01/04-16:40:45 [13959]       File "/etc/munin/plugins/tor_countries", line 337, in
2016/01/04-16:40:45 [13959]         main()
2016/01/04-16:40:45 [13959]       File "/etc/munin/plugins/tor_countries", line 332, in main
2016/01/04-16:40:45 [13959]         provider.fetch()
2016/01/04-16:40:45 [13959]       File "/usr/share/munin-tor/circuits_by_country.py", line 80, in fetch
2016/01/04-16:40:45 [13959]         countries_num = self.top_countries()
2016/01/04-16:40:45 [13959]       File "/usr/share/munin-tor/circuits_by_country.py", line 123, in top_countries
2016/01/04-16:40:45 [13959]         c = Counter(self._gen_countries(controller))
2016/01/04-16:40:45 [13959]       File "/usr/lib/python2.7/collections.py", line 449, in init
2016/01/04-16:40:45 [13959]         self.update(iterable, **kwds)
2016/01/04-16:40:45 [13959]       File "/usr/lib/python2.7/collections.py", line 530, in update
2016/01/04-16:40:45 [13959]         for elem in iterable:
2016/01/04-16:40:45 [13959]       File "/usr/share/munin-tor/circuits_by_country.py", line 110, in _gen_countries
2016/01/04-16:40:45 [13959]         for ipaddr in self._gen_ipaddrs_from_statuses(controller):
2016/01/04-16:40:45 [13959]       File "/usr/share/munin-tor/circuits_by_country.py", line 104, in _gen_ipaddrs_from_statuses
2016/01/04-16:40:45 [13959]         for desc in controller.get_network_statuses():
2016/01/04-16:40:45 [13959]       File "/usr/local/lib/python2.7/dist-packages/stem/control.py", line 427, in wrapped
2016/01/04-16:40:45 [13959]         raise exc
2016/01/04-16:40:45 [13959]     stem.SocketClosed
2016/01/04-16:40:45 [13959] Service 'tor_countries' exited with status 1/0.
2016/01/04-16:41:02 [13960] Service 'interrupts' timed out.
2016/01/04-16:41:04 [13960] Service 'interrupts' timed out.
Any idea what could be wrong on my config?
Thank you and keep the good work up!
--
mane

*** UPDATE ***
I also see these:
2016/01/04-16:50:08 [15212] Error output from tor_connections:
2016/01/04-16:50:08 [15212]     No response from Tor daemon in TorConnection.fetch()
2016/01/04-16:50:08 [15212] Service 'tor_connections' exited with status 255/0.

The script is still working for me, something in the connection to the tor daemon seems to be not working for you.
Can you try setting up cookie auth as described in https://github.com/mweinelt/munin-tor/issues/11#issuecomment-68633802?

Hi @mweinelt,
Thanks for your quick reply. I've tried as you suggested but with no success so far.
Here is what I get into the log:
2016/01/04-19:55:28 [1238] Error output from tor_dormant:
2016/01/04-19:55:28 [1238]      2016/01/04-19:55:28 [1416] User 'tor' required for 'tor_dormant' does not exist. at /usr/share/perl5/Munin/Node/OS.pm line 183

I've checked my /etc/passwd and the user is not "tor" but "debian-tor" (as it is a raspbian OS).
Let's see if it likes that.

2016/01/04-20:10:03 CONNECT TCP Peer: "[<IP_ADDRESS>]:40091" Local: "[<IP_ADDRESS>]:4949"
2016/01/04-20:11:00 [3272] Service 'tor_countries' timed out.
2016/01/04-20:11:02 [3272] Service 'tor_countries' timed out.
2016/01/04-20:11:03 [3272] Node side timeout while processing: 'config memory'

Hope this can be useful to figure out what's wrong with my config as it seems to be all working except for the graphs:
pi@raspberrypi ~ $ sudo munin-run --debug tor_dormant
Processing plugin configuration from /etc/munin/plugin-conf.d/munin-node
Processing plugin configuration from /etc/munin/plugin-conf.d/tor_
Setting /rgid/ruid/ to /114/0/
Setting /egid/euid/ to /114 114 0/0/
Setting up environment
About to run '/etc/munin/plugins/tor_dormant'
dormant.value 0
pi@raspberrypi ~ $ sudo munin-run --debug tor_traffic
Processing plugin configuration from /etc/munin/plugin-conf.d/munin-node
Processing plugin configuration from /etc/munin/plugin-conf.d/tor_
Setting /rgid/ruid/ to /114/0/
Setting /egid/euid/ to /114 114 0/0/
Setting up environment
About to run '/etc/munin/plugins/tor_traffic'
read.value 493302702
written.value 492465149

All working now.
Apparently it was just a matter of time.
For the record:
I've modified the content of /etc/munin/plugin-conf.d/munin-node and added:
[tor_*]
timeout 60
user debian-tor
group debian-tor
enabled cookie auth as described in #11 (comment)
Thank you.


run.sh with command line arguments

It would be great to have run.sh with command line arguments instead of having to fix the script.
Maybe also make the outputdir just the genome basesname with a time stamp.
Check if mummer and blast is in path and if not raise an error for the user to fix.

Done


RFE extra incentives

If x karma points earned auto-join private VIP channel

could be something like:

3 months on slack
got 20 times a karma upvote (rather than 30 points which you get by 6x 5 points)
those 20 upvotes are given by at least 5 different people
or you're an admin (somebody needs to run the show right?)