Is this a Textmate Bundle?   I just confirmed this works as in IntelliJ/Pycharm as well

Would it be appropriate to make clear this project works in a number of different editors that aren't VSCode?
I was just able to verify that this works in PyCharm using the Textmate Bundle Plugin (a default plugin).
Preferences > Editor > Textmate Bundles
Hit the plus in the upper left and then select the directory this code is in.
It should Just Work™ at that point for all .cook files.

That's great, thanks for noticing that @Cabalist. Yes, probably we can update documentation and mention this.


Tug-O-War

Is there any way to not play tug-o-war with the map when using it between a few people in different locations? My buddy is in Oregon and I'm in Cali and when we scan the map, I notice the map scan location changes for both of us, so it's like we're playing tug-o-war with each other on who gets to scan a location. Any help would be greatly appreciated!

Honestly you shouldn't both use the same server. that's not how it's really meant to work. unless you're both together then maybe. but in which case still probably only need one of you running it. if you're going to share don't do it at same time. it cannot scan two diff locations at once.

@MysticalOS Is it possible to make a version that can scan multiple locations? I'm assuming that's what PokeVision used to do, right?

This app is just a front end (GUI) for https://github.com/AHAAAAAAA/PokemonGo-Map/ so it only really does what it does. that supports multiple accounts but for scanning same location. if you want to scan two diff locations same time you really just should be running two different go map servers.

You'd have to use MySQL as your database for thread safety and then just start multiple instances of the server with different port numbers but the same database.

That would make you get notifications from all locations...

Now you can use Pogom server to scan multiple location.
I close


chore(ci): skip mapping errors when upgrading the Terraform provider

Pretty much every time we upgrade the upstream Terraform provider, the upgrade fails due to one of the following:

Addition or removal of a resource/data source
Changes to examples

As a result we are, in practice, doing a manual upgrade of the Pulumi provider for almost every release of the Terraform provider.  Those manual upgrade PRs also end up being overly large and difficult to review because they also include changes to add or remove features in parity with the upstream provider, and possibly-accidental upgrades of surrounding tooling.
This updates the upgrade-provider GitHub Actions workflow to define environment variables that make the provider upgrade process more resilient to upstream changes.  The script to generate examples is also updated so that issues with example conversion result in missing examples rather than a failed upgrade.
The intent of these changes is to enable automation to do its job as well as possible, and allow follow-up contributions to focus narrowly on fixing examples or adding/removing resources.

For some reason the changes to the example script are not having the same effect in CI that they do locally.  Manually running the upgrade job on this branch fails, but when I run what I think are the equivalent commands locally, it succeeds.

For some reason the changes to the example script are not having the same effect in CI that they do locally. Manually running the upgrade job on this branch fails, ~but when I run what I think are the equivalent commands locally, it succeeds.~ I had been running the ./upstream.sh ... commands to update the submodule, followed by make tfgen, and that succeeded. When I instead installed and ran upgrade-provider, it failed but in a slightly different place than it does in CI.
I see some kind of repo checkout happening during upgrade-provider execution; does that command use scripts and/or Makefile from the main branch rather than using the local files?

Yes it checkouts to main branch
pull default branch
✓ /usr/local/bin/git ls-remote --heads origin: done
✓ finding default branch: master
✓ /usr/local/bin/git checkout master: done
✓ /usr/local/bin/git pull origin: done

It's also worth mentioning that it seems now there is more control over how examples are generated by default, perhaps by making some changes to Terraform, the script to generate examples is not needed at all.


Add support for navigational page block [PlanID:2023.3/11/2.7]

Introduce new page block type for showing one or two level of dynamic and static navigation
Inspired by bootstrap's nav component.
List configuration

[x] Add tab right of general - "Navigation"
[x] options for align, vertical, fill, pills, small, tabs
[x] Default properties for all items

[x] disabled
[x] text variant or colour
[x] background variant or colour

[x] List items can be sorted (drag & drop)

Supported navigation item types

[x] Text
[x] Delimiter (only inside dropdown)
[x] URL

[x] new window/tab

[x] Compose page in the same namespace

[x] new window/tab
[x] use to prop on <b-nav-item> to to ensure internal router is used
[x] select a page to be used, keep reference (ID)
[x] option to show list of child pages in-place (1 level only)
[x] option to show referenced page as dropdown with subpages

[x] Dropdown

[x] Only text and URL
[x] If used in combination with compose page, ensure there is no 2nd level
[x] Support drop(left|right|up) prop

Item types properties
Each item can override the default (see above) item properties

[x] Ability to remove explicit property value (and use the default)

https://github.com/cortezaproject/corteza/pull/584

THere are couple more things that needs to be addressed and it was already discussed with Tolu. So once everything is addressed feel free to move the ticket back again to in QC

Retested again and couple more issues arise here so we will try to address those and after that will need to be retested again.
The found issues for now are:

The display block as a card doesn't work as expected.

I don't see the difference between opening up things in a new window or a new tab (this is connected with the browser type, will be discussed further).

When I set navigation items of type URL, the open in same tab doesn't do anything (and if you have first a configuration for compose page and you switch to URL later on, the URL tab is still stuck with the compose page and the URL is not even shown. Tolu will be addressing this one)

Compose page doesn't show the subpages (this was related with the configuration of a subpage. This is probably fine, will be retested again)

Display options for appearance doesn't look to be doing the correct output. (this is probably fine, since the difference between the tabs, pills and small are actually behind the curtains so we cannot really see what is going on just by looking at the design.

Tested and confirming that this now works as expected. The only thing now left is the CL entry.

Changelog:
• What changed

Updated the page blocks with a navigation page block

• Why did it change

To add new support for a fully customizable navigation page block with multiple navigation item type and customizatible options for the navigation list items

• How did it change

Added a new configurator with the required configuration options for the navigation page block and the base which displays the navigational block with the styles and configuration options

Added CL


Implement difference, intersect.

See https://jena.apache.org/documentation/javadoc/jena/org/apache/jena/rdf/model/Model.html for details.

Merged feature/turtle to develop. This doesn't solve the task above, but it this done, now @GezimSejdiu ?

Yes, you are right.
This does not solve completely the issue above. But the basic implementation of intersection and difference was done already, which imply that we can close the issue for now.


Can't add extra parameters to mutect2

Version info

bcbio version 1.2.3

Hi guys,
I am trying to add the "--max-mnp-distance 0" command to mutect2 for building a PON database. I have not managed to get it to include this parameter in the final YAML file no matter which way I specify it. Not sure if there is something I am doing wrong here?
Thanks!

Tumor run template

details:

algorithm:
aligner: false
mark_duplicates: false
recalibrate: false
remove_lcr: false
variantcaller: mutect2
ensemble:
numpass: 2
variant_regions: /shared/pipeline-user/run_data/baits/sureselectv7/S31285117_Padded.bed
vcfanno: [somatic]
tools_on:

damage_filter
analysis: variant2
genome_build: hg38
description: TD1717_test

resources:
mutect2:
options: [" --max-mnp-distance","0"]
fc_date: '2020-02-10'
fc_name: 'tumor_only'
upload:
dir: ../final

or

Tumor run template

details:

algorithm:
aligner: false
mark_duplicates: false
recalibrate: false
remove_lcr: false
variantcaller: mutect2
ensemble:
numpass: 2
variant_regions: /shared/pipeline-user/run_data/baits/sureselectv7/S31285117_Padded.bed
vcfanno: [somatic]
tools_on:

damage_filter
analysis: variant2
genome_build: hg38
description: TD1717_test

resources:
mutect2:
options:
- --max-mnp-distance
- 0
fc_date: '2020-02-10'
fc_name: 'tumor_only'
upload:
dir: ../final

@naumenko-sa @roryk Guys, can anyone help me with this? I am sure it must be something simple but I have not managed to get this working. Thanks!

Hi Warren @waemm!
I think I'm following your footsteps in PureCN user story.
In bcbio, we require phenotype: tumor for mutect2,
and a normal or panel of normals. For tumor-only projects we are using vardict.
So, currently, it is not possible to create a PON for small variants in bcbio with mutect2.
I do it outside of bcbio with:
# $1 = sample_N.bam
# #2 = panel.interval_list

# -tumor is deprecated

bname=`basename $1 .bam`

gatk Mutect2 \
-R /data/bcbio/genomes/Hsapiens/hg38/seq/hg38.fa \
-I $1 \
-O $bname.vcf.gz \
--max-mnp-distance 0 \
--intervals $2 \
--interval-padding 50 \
--germline-resource af-only-gnomad.hg38.vcf.gz \
--genotype-germline-sites

tabix -f $bname.vcf.gz

vcf_files=""
for f in *.for_pon.vcf.gz
do
    vcf_files="$vcf_files -V $f"
done

gatk3 -Xmx12g \
-T CombineVariants \
--minimumN 3 \
-R /data/genomes/Hsapiens/hg38/seq/hg38.fa \
-o snv_pon.vcf \
$vcf_files

bgzip snv_pon.vcf
tabix snv_pon.vcf.gz

I'm trying to bring it to bcbio as a part of PureCN effort.
There is a complication that a PON generated by CreateSomaticPanelOfNormals does not work for PureCN,
so we might need two sorts of PONs here in bcbio.
S.

@naumenko-sa Thanks Sergey, yes I was more or less following this approach. Would be great to see it in bcbio!


RFC page description style changes

RFC description
Increased font size and removed max width from the RFC description. There is no reason to limit the text and keep it small for reading.
Before changes

After changes

RFC Tags
RFC tags change styles to be consistent with proper hover effect.

We used 2 blade components for these tags x-tag and x-tag-link with different styles, now we use only x-tag for all tags.
Removed hover effect from tags that are not links
Removed some SVG icons and replaced them with icons that we already have in our components/icons/ directory

Before changes

After changes

You voted for
Changed styles for You voted {vote} tag on home card

After changes

I see how I can make it look even better, maybe she should put them with other tags together, but I would rather make an issue in the future with some ideas, to discuss which solution is better. Plus, I'm used to small PRs.

I just noticed another issue:

Code samples are breaking the screen width on mobile, give me a minute

Should be good now

Awesome, thanks!


Fix terratest. Bump terraform-yaml-stack-config module version

what

Fix terratest
Bump terraform-yaml-stack-config module version

why

Terratest had incorrect module name in go.mod
Latest version terraform-yaml-stack-config module has a few improvements and bug fixes

/test all


Admin approval

Re: https://github.com/keboola/connection/pull/633

the tests should cover also:

super admin or maintainer cannot join organization
super admin or mainteinar cannot change organization autoJoin flag
super admin or maintainer can join project belonging to ogranization with auto join turned on
super amdin or maintainer can request access to project belonging to organization - pending status

Right, they should indeed!  thanks

To test the maintainers rights, the maintainers members endpoints http://docs.keboolamanagementapi.apiary.io/#reference/maintainers/maintainer-members would need implementation.

@pivnicek can you please doublecheck testSuperNoJoinOrganization? I'm getting
1) Keboola\ManageApiTest\OrganizationsTest::testSuperNoJoinOrganization
Keboola\ManageApi\ClientException: You don't have access to maintainer 1

/code/src/Client.php:446
/code/src/Client.php:400
/code/src/Client.php:127
/code/tests/OrganizationsTest.php:128

Caused by
GuzzleHttp\Exception\ClientException: Client error: `POST http://local.docker:8700/manage/maintainers/1/organizations` resulted in a `403 Forbidden` response:
{"error":"You don't have access to maintainer 1","exceptionId":"connection-devel-8b7a5ffd3fd4821b9cc43e65c6468bd3"}

all other tests pass, anything wrong on my end?

jo, sorry, I'll sort it.


Permission 'admin-blog-post' does not show posts by other users

Shouldn't users with admin- level be able to edit posts by other users? I've set the permissions on a group to:
self.apos.users.insert(req, {
username: emailAddress,
email: emailAddress,
title: first_name : 'title',
//anything other than admin will not let you edit (or view) others posts
permissions: [mscr == 36 ? 'admin-blog-post' : 'admin' : ''],
}, { permissions: false }, getUser(loginUser, callback));
But only users with 'admin' permission can edit other posts. Is there a way to allow a group of users to have control over blog posts (any posts) but restrict access to other portions of the site (e.g. pages or users)?

Hi Lance, the permission you want exists but it's called:
admin-apostrophe-blog
permission name, hyphen, "name" option of the piece type in question.

Thanks Tom. This relates to the answer you gave on SO here: http://stackoverflow.com/questions/39885020/article-published-permission
I have different permissions/groups now for admin and admin-apostrophe-blog (aab). We have one more user type: users who can create or edit any blog post, but cannot publish a blog post. From my testing, it looks like only admin or aab permissions can see all blog posts. In order to have an aab user who can create/edit all posts but not publish, I'm thinking Ill have to create a custom field, to  distinguish this user type, then in the admin modal hook into the view that renders the publish drop down to not display to this user type. I

Hi Tom. I know this issue is closed, but I have a related question. What is the db relationship between users and page/article permissions? Initially, because of this response ( http://stackoverflow.com/questions/39834726/auto-add-users-to-apostrophe ) I thought there was a relationship between a user record groupIds values and his ability to perform actions via Apos admin bar (e.g. create pages or edit articles). However, after more testing on the db, it looks like the groupIds isn't used but instead the permissions field is used. This db record still has access to edit pages and articles, even though the groupIds field is absent.
"_id" : "ciuhr8axs0004hwsmdx308w5h",
"username"<EMAIL_ADDRESS>"permissions" : [
    "admin"
],

Changing this record to:
"_id" : "ciuhr8axs0004hwsmdx308w5h",
"username"<EMAIL_ADDRESS>"permissions" : [
    " "admin-apostrophe-blog"
],

allows the user to create and edit articles via Apos admin bar, but not page settings. Therefore, it looks like only this permissions field needs to be update if I want to add users automatically from my own custom module.
My question is, is groupIds: [] needed then?
Thanks

Okay. I think I've gotten to the issue I was having and understand how apos uses mongo to lookup and store permissions.  On app setup/db creation, app.js runs and inserts the 'apostrophe-users': {  groups: [] } items into aposDocs. When a user is created and his permission is selected in apos admin bar, the user record is inserted into aposUsersSafe and also aposDocs. In aposDocs the user record contains the groupIds:[] field, which is the mongo record id for the group id created on app setup/db creation.
This is the relationship between users and groups. If the permission is changed in app.js, for example:
from:
'apostrophe-users': {
groups: [{
title: 'publisher',
permissions: ['admin-apostrophe-blog']
to:
'apostrophe-users': {
groups: [{
title: 'publisher',
permissions: ['edit-apostrophe-blog']
then the aposDocs group record needs to be updated also:
{
"_id" : "ciuhiskoi0005losm3jn01v1b",
"title" : "publisher",
"permissions" : [
"admin-apostrophe-blog"
],

Yes, that's a correct reading. The "simplified permissions" feature you're
referring to is built on the assumption that you won't be changing it up
much. If you really want to alter the permissions of groups, create groups
as you see fit, etc., just eliminate the groups option for the users
module altogether and you get an feature to edit groups in the admin bar.
See this discussion:
http://apostrophecms.org/docs/tutorials/intermediate/permissions.html
I think it qualifies as a bug that the code that creates the pre-configured
groups doesn't force a reset of the permissions if they already exist. I'll
open an issue on that.
On Mon, Oct 24, 2016 at 2:26 PM, Lance P<EMAIL_ADDRESS>wrote:

Okay. I think I've gotten to the issue I was having and understand how
apos uses mongo to lookup and store permissions. On app setup/db creation,
app.js runs and inserts the 'apostrophe-users': { groups: [] } items into
aposDocs. When a user is created and his permission is selected in apos
admin bar, the user record is inserted into aposUsersSafe and also
aposDocs. In aposDocs the user record contains the groupIds:[] field, which
is the mongo record id for the group id created on app setup/db creation.
This is the relationship between users and groups. If the permission is
changed in app.js, for example:
from:
'apostrophe-users': {
groups: [{
title: 'publisher',
permissions: ['admin-apostrophe-blog']
to:
'apostrophe-users': {
groups: [{
title: 'publisher',
permissions: ['edit-apostrophe-blog']
then the aposDocs group record needs to be updated also:
{
"_id" : "ciuhiskoi0005losm3jn01v1b",
"title" : "publisher",
"permissions" : [
"admin-apostrophe-blog"
],
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe-sandbox/issues/59#issuecomment-255824249,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fWD6KsBT9YTyGCghjmeHegjRdujEks5q3PhTgaJpZM4Kbr1-
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

We do plan to reintroduce this exact distinction in the near future; it was
optional in 0.5 and related to the workflow option.
On Fri, Oct 21, 2016 at 12:14 AM, Lance P<EMAIL_ADDRESS>wrote:

Thanks Tom. This relates to the answer you gave on SO here:
http://stackoverflow.com/questions/39885020/article-published-permission
I have different permissions/groups now for admin and
admin-apostrophe-blog (aab). We have one more user type: users who can
create or edit any blog post, but cannot publish a blog post. From my
testing, it looks like only admin or aab permissions can see all blog
posts. In order to have an aab user who can create/edit all posts but not
publish, I'm thinking Ill have to create a custom field, to distinguish
this user type, then in the admin modal hook into the view that renders the
publish drop down to not display to this user type. I
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe-sandbox/issues/59#issuecomment-255289679,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fbYxutKzEW_V7CfKicVDAi3kIV8bks5q2DwngaJpZM4Kbr1-
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Hi Lance, the permission you want exists but it's called:
admin-apostrophe-blog
permission name, hyphen, "name" option of the piece type in question.
On Wed, Oct 19, 2016 at 11:30 PM, Lance P<EMAIL_ADDRESS>wrote:

Shouldn't users with admin- level be able to edit posts by other users?
I've set the permissions on a group to:
self.apos.users.insert(req, {
username: emailAddress,
email: emailAddress,
title: first_name : 'title',
//anything other than admin will not let you edit (or view) others posts
permissions: [mscr == 36 ? 'admin-blog-post' : 'admin' : ''],
}, { permissions: false }, getUser(loginUser, callback));
But only users with 'admin' permission can edit other posts. Is there a
way to allow a group of users to have control over blog posts (any posts)
but restrict access to other portions of the site (e.g. pages or users)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe-sandbox/issues/59, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAB9fUiAj5Vy8g765uKWkaZGFQ_INtL7ks5q1uBIgaJpZM4Kbr1-
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com


K8SPXC-344 add only ready PXC members into the config

fix self-healing-advanced test. It is important to have non-empty wsrep_cluster_address in case when cluster in broken state

Test name
Status

scaling
passed

storage
passed

recreate
failed

init-deploy
passed

big-data
passed

restore-to-encrypted-cluster
failed

scaling-proxysql
passed

self-healing
passed

demand-backup
failed

limits
passed

upgrade
passed

monitoring
passed

self-healing-advanced
failed

scheduled-backup
failed

upgrade-consistency
passed

security-context
passed

monitoring-2-0
passed

operator-self-healing
passed

affinity
passed

one-pod
failed

auto-tuning
passed

demand-backup-encrypted-with-tls
passed

proxysql-sidecar-res-limits
passed

Test name
Status

scaling
passed

big-data
failed

init-deploy
passed

recreate
passed

storage
passed

scaling-proxysql
failed

self-healing
passed

limits
passed

upgrade
failed

restore-to-encrypted-cluster
passed

monitoring
passed

upgrade-consistency
passed

security-context
passed

self-healing-advanced
failed

demand-backup
failed

monitoring-2-0
passed

scheduled-backup
failed

operator-self-healing
passed

affinity
passed

one-pod
failed

auto-tuning
passed

demand-backup-encrypted-with-tls
failed

proxysql-sidecar-res-limits
passed

Test name
Status

recreate
passed

restore-to-encrypted-cluster
passed

demand-backup
failed

scaling
passed

storage
passed

init-deploy
passed

scaling-proxysql
passed

self-healing
passed

limits
passed

self-healing-advanced
passed

monitoring
passed

big-data
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

upgrade
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

scheduled-backup
failed

operator-self-healing
passed

upgrade-consistency
passed

security-context
passed

demand-backup-encrypted-with-tls
failed


Add Vec type

I've found myself wanting to use the equivalent of Rust's Vec when writing Sway code. I'd like to see this implemented in the SDK 😄
This probably is dependent on this issue: https://github.com/FuelLabs/sway/issues/428

Something like: https://doc.rust-lang.org/std/vec/struct.Vec.html

Hmm, doesn't the SDK already support vectors? Although only static/fixed size vectors. I remember implementing it a while ago. I.e. if you declare a Sway struct with a fixed-size vector as one of its properties, and use this struct as input in your ABI method, it should work fine 🤔
I'm imagining that that's what you're looking for? Sorry if it's not and it's something entirely different!

Oops haha @digorithm this proposal probably has you confused because I meant to post it in https://github.com/FuelLabs/sway-lib-std 😅

Hahaha that makes a whole lot more sense!


Does adding MySQL database support?

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
Does adding MySQL database support?
✔ Solution
Does adding MySQL database support?
❓ Alternatives
No response
📝 Additional Context
No response

I think this is a duplicate of https://github.com/louislam/uptime-kuma/issues/953
Also think that this might be reoslved by the v2.0 relase, as we are adding mariadb support: https://github.com/louislam/uptime-kuma/pull/2720
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

resolved by the v2.0 relase, as we added mysql/mariadb support: https://github.com/louislam/uptime-kuma/pull/2720
=> closing as resolved


fix(overthebox): use @ovh-ux/ng-tail-logs

fix(overthebox): use @ovh-ux/ng-tail-logs
Description of the Change
Replace ovh-angular-tail-logs dependency by<EMAIL_ADDRESS>7a149d7 — fix(overthebox): use @ovh-ux/ng-tail-logs
Related PR

[ ] https://github.com/ovh-ux/ng-tail-logs/pull/14 — (Prerelease 2.0.0-beta.0 done)

/cc @jleveugle @frenautvh @antleblanc

Hello @cbourgois,
Your pull request is now closed due to inactivity.
Feel free to re-open it if you have any update.
Anna Bot for OVH UX <3


Errors in smhifilter.sh

The code below:
https://github.com/georgefloros1998/MNXB11-homework/blob/88acae7c2cdd0a13c85f00608afc68141e5da056/homework4/smhifilter.sh#L157
Fails:
[2024-10-31T17:16:02+01:00 Filter]: Filtering on only measurements taken in the first day of the month, writing to
grep: invalid option -- ' '
Usage: grep [OPTION]... PATTERNS [FILE]...
Try 'grep --help' for more information.

due to a mistake in the grep expression, - is a special symbol and needs to be escaped:
grep '\-01' ...

Moreover the variable name $CLEANER_FILTERFILENAME_ONLYFIRST does not exist
https://github.com/georgefloros1998/MNXB11-homework/blob/88acae7c2cdd0a13c85f00608afc68141e5da056/homework4/smhifilter.sh#L156C85-L156C118
was actually a bug in my own script and it doesn't print out:
[2024-10-31T17:16:02+01:00 Filter]: Filtering on only measurements taken in the first day of the month, writing to
``

Addendum, just so to understand what ChatGPT gave you:
The symbol ^ used in https://github.com/georgefloros1998/MNXB11-homework/blob/29d938ff0f0956a64834d42a14439d8f75bb2dcf/homework4/smhifilter.sh#L157-L158
means: match the following string (1998-) at the beginning of a line.


Upgrade to Jdk1.7

Hi Folks,
This PR updates pom.xml to specify javac 1.7 as well as Maven compilation for 1.7.
It then implements all of the localization and charset specification required to comply with the forbidden API's plugin. Additionally it removes all references to System.out.blah which have been replaced by String exception messages and typically wrapped in slf4j logging instances.
Thanks

Bugger i just noticed that this PR also includes the bloody commits for the validSitemaps branch as well. @Chaiavi can you please merge #82 into master and I will rebase? Thanks.

This PR is now good folks, it's been updated to master.

Thanks Lewis
I ran the unit tests and all passed
On Mon, Sep 7, 2015 at 9:19 PM, Lewis John McGibbney <
<EMAIL_ADDRESS>wrote:

This PR is now good folks, it's been updated to master.
—
Reply to this email directly or view it on GitHub
https://github.com/crawler-commons/crawler-commons/pull/94#issuecomment-138351814
.

OK, will wait maybe another day and merge if all good.

Hi @jnioche PR updated to accommodate your suggestions. Quick note on existing String.format usage... we only use it within function calls which return string values. We cannot implementing slf4j logging conventions in such instances.
I've implemented slf4j conventions in a few other places as well.

we only use it within function calls which return string values. We cannot implementing slf4j logging conventions in such instances.

sure, that's not what I was suggesting. Thanks for your work on this

Hi @lewismc - I didn't see this mentioned in CHANGES.txt

Correct Ken. I used the merge function in the UI and forgot about this. Will update with adhoc commit right now.


Add a warning string is no connectionStrings are defined in the config file

I will push something regarding this one. Let me know if I misunderstood.

@rangzen Sounds good, I will assign you. FYI though, this issue is not complex enough to be considered for Hacktoberfest. Are you still interested?

@rangzen  Are you still interested in pursuing this issue?

Hello @prestonvasquez, the MR is merged, do you need something else?


Drop deprecated cordova-plugin-file-transfer testing

which consistently gives me failures on Android
this would invalidate https://github.com/apache/cordova-labs/issues/16 - Move cordova-filetransfer branch to cordova-plugin-file-transfer repo

this would invalidate apache/cordova-labs#16 - Move cordova-filetransfer branch to cordova-plugin-file-transfer repo

Not at all. This app is just one (mostly unused btw) way to run tests.

this would invalidate [...]

Not at all. [...]

Removed from description by strikeout. The original title and beginning of the description continue to stand on my part.

You having failures also is not a reason to drop a test tbh.
The plugin being deprecated might be - but that would be a different issue then and include more plugins.

The plugin being deprecated might be - but that would be a different issue then and include more plugins.

Closing now, in favor of a new issue that I am planning to raise later today.
https://github.com/apache/cordova/issues/54#issuecomment-442834293) indicates to me that using mobilespec with the cordova-filetransfer branch of cordova-labs is used for plugin testing, maybe primarily for plugin testing, maybe only for plugin testing. I was not aware of this information from recent emails and GitHub issues.


added $binDir to path tmprrly to fix mongo install

if you dont want to merge dont its tmp fix i can keep it local and u can delete the branch

seems ok


Internally Hosted Draw.IO is not usable, "This content is blocked. Contact the site owner to fix the issue."

Describe the Bug
Similar to #2285 , I am getting a gray page in Chrome that says "This content is blocked. Contact the site owner to fix the issue."
I have the following environment variables set for the container:
DRAWIO=http://<IP_ADDRESS>:8080/?embed=1&proto=json&spin=1&configure=1&stealth=1
I have also attempted to modify this environment variable:
ALLOWED_IFRAME_SOURCES=
I've tried:

http://<IP_ADDRESS>:8080
https://<IP_ADDRESS>:8443
http://<IP_ADDRESS>*

The only one that "works" is if I make it ALLOWED_IFRAME_SOURCES="*", which seems like a security vulnerability even if I'm running this on a LAN.
Note: I can access the plain old Draw.IO interface just fine: http://<IP_ADDRESS>:8080, and it loads.
Steps to Reproduce
Edit a page, click the icon to work on a Draw.io image.
Expected Behaviour
I expect to load into a Draw.IO instance.
Screenshots or Additional Context
No response
Browser Details
Chrome and Edge on Windows 11
Exact BookStack Version
v24.05.2

Hi @thickconfusion,
You shouldn't need to adjust the iframe sources since BookStack will look to automatically add any custom drawio URL, where set, to the CSP rules. Maybe our custom handling is tripping up any additional rules you're adding.
It does look though like we are not currently handling scenarios where non-protocol-standard ports are used.
I've marked this to be tested for next patch, against a custom-ported drawio instance.
Dev reference
https://github.com/BookStackApp/BookStack/blob/78ebcb6f38ee7a984b26cd56dff882ae9d7e9f95/app/Util/CspService.php#L144

Sure, I was just saying that we attempt to handle this so you shouldn't have to set the iframe sources, but we currently don't handle custom defined ports.
I've now fixed port handling via 897bb338f956245e2c86bda6cd5c6a67711f9448, with testing to cover, which will be part of the next patch release so I'll therefore close this off.
Not sure why your custom ALLOWED_IFRAME_SOURCES additions did not work, since I could work around this on my dev instance via this method, but could be down to browser specifics or configuration changes not take place when expected.
If you still have issues after the next patch release feel free to still comment here for further investigation.


Updating dependabot file to support github-actions, python and docker packages

Codecov Report

Merging #767 (e432677) into main (d322d07) will decrease coverage by 1.55%.
The diff coverage is n/a.

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

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #767      +/-   ##
==========================================
- Coverage   61.18%   59.64%   -1.55%
==========================================
  Files         139      140       +1
  Lines        4364     4574     +210
==========================================
+ Hits         2670     2728      +58
- Misses       1694     1846     +152


Some tracks open just fine, but appear to yield no samples during decoding

I wrote a pandora cli client, and recently Pandora seems to have stopped supplying mp3 URLs for tracks, so I switched to their AacPlus mp4 streams, which I'm using redlux with rodio to decode.
A fair number (maybe half?) of tracks I get from pandora work just fine, I initialize an MP4 decoder, no errors reported, and playback is great.
The other ~half open just fine with the mp4 decoder, but then immediately terminate - as far as I can tell never having played a sample.
To make sure it wasn't something I was doing wrong on my end, I cloned the redlux repo, copied one of these problem tracks into the tests/samples/ folder, updated the m4a test to point to my track instead (extended the sleep time to 2 seconds, and set the volume to 1.0), and ran cargo test.  I hear 2 seconds of silence for my track, but when pointing it at your track I hear 2 seconds of audio.
This same track plays fine with vlc.
Comparing track metadata, all the failing ones have this metadata attached to them: ©too=Lavf58.20.100 or ©too=Lavf58.45.100.
This suggests to me that the failing tracks were (trans/en)coded using libavformat versions 58.20.100 (ffmpeg 4.1.6) or 58.45.100 (ffmpeg 4.3.2).  The encoder for the working tracks is not known, but isn't listed.
Re-encoding your original test sample using ffmpeg -i Simbai\ \&\ Elke\ Bay\ -\ Energy.m4a Simbai\ \&\ Elke\ Bay\ -\ Energy_2.m4a did not render it unplayable, and following similar steps to re-encode my failing track made it work, so I'm guessing there's a particular encoding option that impacting this.
Beyond this point, I don't know what to look for or how to look for it, but I'm happy to help however I can.

Hmm, that's pretty odd. Could you provide the full metadata of the file?
If it's an issue with how it's encoded, I'm guessing it's an issue with fdk-aac, which redlux uses to decode audio samples. To be sure, it would make sense to try to somehow use the fdk-aac bindings directly.
Might be useful to know if it's actually playing any samples. Inside the test, you could try the Sink's sleep_until_end method instead of thread::sleep. If it's not playing any samples, I assume it ends instantly.

I added the sleep_until_end - it's definitely dropping it instantly.
I added some logging statements to the decode_next_sample() function, and picked up this:
2021-04-03 13:16:38:018646175 [ERROR] <redlux:156>:decode frame error: DecoderError { code: 4098, message: "The input buffer ran out of bits." }
2021-04-03 13:16:38:037397975 [ERROR] <redlux:156>:decode frame error: DecoderError { code: 4098, message: "The input buffer ran out of bits." }
2021-04-03 13:16:38:037502986 [ERROR] <redlux:240>:Decoder error: TrackReadingError

I added a bit more instrumentation, and here's the line that it's erroring out on:
let object_type = track.audio_profile().or(Err(Error::TrackReadingError))?;

Un-squelching the error from mp4 yields:
Err(InvalidData("invalid audio object type"))

I patched mp4-rust to be aware of the audio profile value that wasn't supported, and it's the value for SBR, so I did some local modifications to patch mp4-rust and redlux's adts.rs to make it support that value as well.
Now I can trace the execution just past the call to self.aac_decoder.fill() to self.aac_decoder.decode_frame(), and the input buffer is still running out of bits.

Nice find. My first thought was that SBR wasn't supported by fdk-aac, but I believe it is.
From the fdk-aac decoder PDF, these seem to be the supported audio formats:

AAC-LC Low-Complexity
HE-AAC v1 High-Efficiency
HE-AAC v2 High-Efficiency
AAC-LD Low-Delay
AAC-ELD v2 Enhanced Low-Delay

Based on the MPEG-4 Wikipedia article's list of audio object types, SBR seems to count as HE-AAC v1, so I don't know why your solution wouldn't work.
Either way, out of the formats supported by fdk-aac, rust-mp4 (and consequently redlux) only supports AAC-LC. Opened https://github.com/alfg/mp4-rust/issues/51 for that.

Alright - I found was I was doing wrong.  I was coercing SBR (5) and PS (29) object types to LTP (4) for the ADTS header instead of AAC LC (2).  Once I did that everything worked.
You can see my patch here: https://github.com/probablykasper/redlux/compare/master...compenguy:sbr-support
I have validated using my own test file (not added to the repo) on your tests that it decodes the frames, and I've validated by pointing my pandora client to build against this branch, and it has played every track so far just fine.
I only added support for SBR and PS types because I don't know if there are valid type coercions for the ADTS header, and if there are, what they would be.
As soon as there's an mp4-rust release with his fix, I'll send my work as a pull request.

The fix is out in v0.3.0


Add math binary functions

I added a support for basic binary mathematic functions such as addition, subtraction, division and multiplication.
I hope it will help others.

Thank you for your contribution. Since these are not standard JMESPath functions they belong in the jmespath-java-contrib repo, rather than in the core implementation (see https://github.com/burtcorp/jmespath-java/pull/36 for some background on that decision). Could you re-open the PR there?
Also, I can't merge the PR without tests, please see the other functions in the contrib repo for inspiration.

Please find PR:
https://github.com/burtcorp/jmespath-java-contrib/pull/5


(void)awakeFromNib missing

if loaded from nib, the initWithFrame method will not get called

Looks like this issue can be now closed.


x/crypto/ssh/knownhosts: cannot have multiple keys for same host

What version of Go are you using (go version)?

$ go version
go version go1.13.3 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users/pjt/projects/go/bin"
GOCACHE="/Users/pjt/Library/Caches/go-build"
GOENV="/Users/pjt/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/pjt/projects/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.13.3/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13.3/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/p9/y23xtnms6r90wsl5lsz2tkfh0000gq/T/go-build606914329=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version go1.13.3 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.13.3
uname -v: Darwin Kernel Version 19.0.0: Thu Oct 17 16:17:15 PDT 2019; root:xnu-6153.41.3~29/RELEASE_X86_64
ProductName:	Mac OS X
ProductVersion:	10.15.1
BuildVersion:	19B88
lldb --version: lldb-1<IP_ADDRESS>
Apple Swift version 5.1.3 (swiftlang-1100.0.282.1 clang-1<IP_ADDRESS>)

What did you do?

I'm trying to open an ssh connection to my-site.com:22 using ssh.Dial where the config uses the callback provided by knownhosts.New("~/.ssh/known_hosts"). I have a known_hosts file that looks like this:
my-site.com,host1.my-site.com,<IP_ADDRESS> ecdsa-sha2-nistp256 <public-key-1>
my-site.com,host2.my-site.com,<IP_ADDRESS> ecdsa-sha2-nistp256 <public-key-2>

What did you expect to see?
Connection  succeeds when either public key is provided.
What did you see instead?
Connection only succeeds when I happen to connect to host1.my-site.com. If it tries to connect to host2.my-site.com I get a KeyError. I can connect to either host using the ssh program.
Why did this happen?
Using knownhosts.New to build a host key callback rejects some hosts from the known_hosts file when there are multiple Public Keys of the same type. There is the assertion in the knownhosts code which says "For each key algorithm, there can be one hostkey", which I don't  believe is correct. I think we need to check keys from any line that matches the current host, rather than only ones that have key types we haven't seen yet.

knownhosts.New("~/.ssh/known_hosts")

Off topic, but I assume that's just pseudo code, as os.Open doesn't expand the tilde.

I can connect to either host using the ssh program.

Thanks for that info. I didn't know such a thing was supported.
/cc @hanwen

Off topic, but I assume that's just pseudo code, as os.Open doesn't expand the tilde.

@bradfitz Yup, just to give you an idea of what I'm doing. I actually do have a fix, but I was unsure what should be in KeyError.Want since that was originally assuming one key of each type. Do you have any ideas how people use that?

Sorry, I don't know much about this package. I'll let @hanwen handle this.


Add feature switch for JSON log enhancement

https://github.com/SumoLogic/sumologic-lambda-extensions/issues/13
Implement a feature switch to turn off log enhancement for JSON logs
Feature switch defaults to true, so behaviour is unchanged
PR Details
Add a feature switch to turn off log enhancement for JSON logs
Description
Implement a feature switch (via the enrivonment variable SUMO_ENHANCE_JSON_LOGS) which defaults to true so current behaviour remains unchanged. If the feature switch is set to false then the message sent to SumoLogic will not contain any addition metadata only the actual log message.
Related Issue
https://github.com/SumoLogic/sumologic-lambda-extensions/issues/13
Checklist

[x] Updated CHANGELOG.md. (there is no changelog file)
[x] Ran unit tests locally.

@himanshu219 please take a look when you can.

@himanshu219 thanks for the merge. Should I make a new release with it or is that your responsibility?

release is done automatically I will push the new tag

@himanshu219 do you have an ETA for that?

new release is published.


[SPARK-44275][CONNECT] Add configurable retry mechanism to Scala Spark Connect

What changes were proposed in this pull request?
This PR introduces a configurable retry mechanism for the Scala SparkConnectClient.
Parameters for the exponential backoff and a filter for exceptions to retry are passed to the client via the existing (extended) Configuration class. By default, only RuntimeStatusException: Unavailable triggers a retry - this mirrors the default retry behavior in the Python client.
One might want to move the retry logic into the GRPC stub that will potentially be introduced here.
Why are the changes needed?
There are a few existing exceptions that one might want to handle with a retry.
For example, this would allow one to not exit with an exception when executing a command while the cluster is still starting.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Tests included.

@dillitz
Since https://github.com/apache/spark/pull/41743 merged, I think it would be best to:

moving the retrier to it's own utility class, used from the blocking stub (like GrpcExceptionConverter object)
implement a non blocking stub as well
implement retries for addArtifacts. It is a client-side stream, so also needs special handling: only sending the first element in the stream should be retried.
implement functions in SparkConnectClient needed for ArtifactManager, so that ArtifactManager uses the client instead of sending rpcs directly

@dillitz Since #41743 merged, I think it would be best to:

moving the retrier to it's own utility class, used from the blocking stub (like GrpcExceptionConverter object)
implement a non blocking stub as well
implement retries for addArtifacts. It is a client-side stream, so also needs special handling: only sending the first element in the stream should be retried.
implement functions in SparkConnectClient needed for ArtifactManager, so that ArtifactManager uses the client instead of sending rpcs directly

Not sure if I managed to implement the retries for the ArtifactManager correctly. I need your expertise here @juliuszsompolski. Previously implemented retries should be successfully moved to the CustomSparkConnectBlockingStub/GrpcRetryHandler.

Merged to master.


Add cov-report to default pytest flags

Codecov Report

Merging #223 (78e78ae) into main (8e997cc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #223   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           14        14
  Lines          837       836    -1
=========================================
- Hits           837       836    -1

Impacted Files
Coverage Δ

meilisearch_python_async/index.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Enabled HTTP and HTTPS ports for Solr

This allows for providing environment variables to enable Jetty to run with both HTTP and HTTPS exposed on different ports. This will occur if a SOLR_PORT and SOLR_SSL_PORT are provided and are different from each other. By default these ports will be the same.
Additional configuration can be done in webdefault.xml to redirect requests to all or specific paths from HTTP to HTTPS.
This allows us to expose specific endpoints via HTTP such as: /solr/admin/metrics, /solr/metrics, and /api/node/health.

Should we allow http port access from localhost only?

Should we allow http port access from localhost only?

I'm not sure how we could actually implement that, we can configure Jetty to only listen on localhost, but that would be for both ports. I don't think what we expose here is any risk so I think it'll be okay as is.

Should we allow http port access from localhost only?

I'm not sure how we could actually implement that, we can configure Jetty to only listen on localhost, but that would be for both ports. I don't think what we expose here is any risk so I think it'll be okay as is.

I was just thinking that if we allow localhost only, then probably we can access all endpoints locally.
(i looked briefly jetty, and it seems we can bind server socket to localhost - but i'm sure there will be other issues)
Let's go with it for now. Thanks.


Clarify union example as to "same key" processing

Clarify the union function example to make clear which value is selected if multiple arguments have the same key.

@QuasarSE - thanks, great change!

#sign-off


Adding more unit tests for ES v6 client

What changed?
Add unit tests for Scrolll, ClearScroll and IsNotFoundErr method

Why?
Increase test coverage

How did you test it?
Unit test

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 018e15dd-b491-421c-8766-6b4cdb3af0f9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
85 unchanged lines in 21 files lost coverage.
Overall coverage decreased (-0.03%) to 63.731%

Files with Coverage Reduction
New Missed Lines
%

service/matching/matcher.go
1
91.07%

common/peerprovider/ringpopprovider/config.go
2
81.58%

common/persistence/executionManager.go
2
78.59%

common/persistence/historyManager.go
2
66.67%

common/persistence/sql/sqlplugin/mysql/db.go
2
79.49%

common/persistence/sql/sqlplugin/mysql/task.go
2
73.68%

common/task/fifo_task_scheduler.go
2
85.57%

common/task/weighted_round_robin_task_scheduler.go
2
88.56%

common/util.go
2
91.69%

service/matching/db.go
2
73.23%

Totals

Change from base Build 018e15cf-3d6d-468c-b814-72a9a96fede3:
-0.03%

Covered Lines:
93429

Relevant Lines:
146599

💛  - Coveralls


Helper functions for component slots

In the process of getting my tutorial posts up to speed, I found myself wanting a few helper functions in just about every component I was making.
If these are :+1: to your tastes, then I'll be glad to extend them to the slot' functions and write up corresponding documentation :)

I'm not too sure about these - I definitely see the motivation, I just worry a bit that they will be misused.
The idea behind using the thunk is it means you don't have to pay for constructing the component and initial state every time the parent is rendered, and also using the name initialState in the record reinforces the fact that the value is only used for the slot's initial state and that changing it will have no effect on the child component.
I'll have a think! Any thoughts from @beckyconning / @cryogenian / @jdegoes / @jonsterling would also be appreciated.

Maybe this is more of a problem with the lack of something like
mapWithIndex :: forall f a b. (Functor f) => (Int -> a -> b) -> f a -> f b

rather than an issue with Halogen?

I agree that there is a real concern here though :+1:

I appreciate that these may have cleaned up some code in an application project, but I think that the convenience functions are not of broad enough applicability to include in the Halogen project itself. (Just my 2c.)

I like indexedSlot and don't like makeSlot :smile:

Generic way to make slots indexed is cool feature. But probably we should make something like Halogen.Component.List where this helper function will live (with generic List component)
As far as I know slot signature (thunk and object) is made this way to make obvious that changing initalState won't cause rerender.

I was under the impression that the calls delegating to slot would preserve the performance benefits of the thunk -- but I'm probably thinking about it with Haskell's evaluation strategy on the mind, and could certainly be wrong. If there's no way to avoid the \_ -> { component: .... line noise then that does restrict the utility of the mkSlot helper as something to be included in the library. I do feel like it will be reimplemented many times across many components and projects -- I've already implemented something similar at least three times, and there's one in the todolist example as well.
mapWithIndex would be nice, but the general use case I had in mind is the Array Int used as a means of keeping track of slots (like renderTask in the todolist), where the index is potentially less important than the actual item in the array. I imagine that will be a pretty common use case for this sort of thing. The name is striking me as poorly chosen now -- it's not the index wrt array, it's the index wrt child component! Could probably do away with that entirely if the slot function had the arguments in a different order: map (slot component initialState <<< Slot) accomplishes essentially the same thing.

@parsonsmatt Thanks for the PR, btw. :smile:


fix-artifact-paths should provide feedback

Describe the bug
Running fix-artifact-paths with no path throws
TypeError: argument of type 'PosixPath' is not iterable
To Reproduce
Steps to reproduce the behavior:
**Expected behavior**

Something along the lines of "Please provide a path"

This one is one me ;)
Good find, thanks!

https://github.com/mlf-core/mlf-core/commit/dbddfa5527c8cec22562586fe01e3e17f830b95a


Please add LICENSE to this project

https://help.github.com/articles/licensing-a-repository/
I'd suggest the MIT one. Adding a license would be useful to spur developers to contribute to this library and maybe use it in production.

ok i will add it :)


Flashing timer icons

Is your feature request related to a problem? Please describe.
With alot of notification tracking what the notification is exactly can get confusing. yes, the notification says "your health is low", but those are rarely read, at least in my experience.
Describe the solution you'd like
The ability for the timer icons to flash when they're low instead of having a solid red
Describe alternatives you've considered
Additional context

Can you be more specific? Are you referring to notifications (eg. the Idle Notifier plugin's "Your health is low" notification) or timers? (eg. infobox with antipoison duration)

i think he is suggesting that the infobox would flash


Sending a new primary key when it already exists, should it be blocking?

While I was working in the Python SDK I realized that you can send a primary key as a aparameter while creating or updating documents. If a primaryKey is already set, the default behavior seems to be the same in MeiliSearch and in every SDK:  it ignores your 'new' primary key, and makes the addition/modification you requested. No exception or error, no notification, nothing, it is just ignored.
In my opinion, this leads the user to think that the 'new' primary key was set, and lead to confusion and introducing bugs in different parts of his code (as it "seems" to work) without understanding what happened exactly and it might be hard to debug afterwards.
I propose throwing an error/exception/panic to tell the user he's actually requesting something that can't be done!
Python current index.update_documents() method:
def update_documents(self, documents, primary_key=None):
        """Update documents in the index

        Parameters
        ----------
        documents: list
            List of dics containing each a document, or json string
        primary_key: string
            The primary-key used in MeiliSearch index. Ignored if already set up.
        Returns
        ----------
        update: `dict`
            Dictionnary containing an update id to track the action:
            https://docs.meilisearch.com/references/updates.html#get-an-update-status
        """
        if primary_key is None:
            url = '{}/{}/{}'.format(
                self.config.paths.index,
                self.uid,
                self.config.paths.document
            )
        else:
            url = '{}/{}/{}?{}'.format(
                self.config.paths.index,
                self.uid,
                self.config.paths.document,
                urllib.parse.urlencode({'primaryKey': primary_key})
            )
        return self.http.put(url, documents)

I would appreciate your opinion on this!

This may be a COre discussion (or at least before discussing anything for the SDKs).
I created an issue here: https://github.com/meilisearch/MeiliSearch/issues/738

Closing this since it's a core discussion.
A more general discussion could be open about warnings in the SDKs


Remove-Refresh data without setOption()

General Questions

[x] ECharts version；4.1.0
[x] ECharts-GL version；1.1.1

Issue Details
I'm working on a cyber threat world map.
What I want to do looks imposible with the API or eCharts properties.
I have to remove each line after a few seconds of doing its respective appendData() method.
I am representing a line with effect from region source to its destination with a scattered point.

Each line-point attack being added like:
// Config lines and scatters

newSeriesEvent() {
   const serieLine = {
      name: event.event_type,
      type: 'lines',
      zlevel: 1,
      coordinateSystem: 'geo',
      large: true,
      effect: {
        show: true,
        period: 0.5,
        symbol: 'pin',
        symbolSize:  11,
        trailLength: 0,
        loop: false
      },
      lineStyle: {
        normal: {
          width: 1.1,
          opacity: 0.8
        }
      },
      showSymbol: false,
      animationEasing: 'quadraticIn',
      animationDuration: 600,
      data: []
    }

    const serieScattered = {
      name: event.event_type,
      type: 'effectScatter',
      coordinateSystem: 'geo',
      showEffectOn: 'render',
      zlevel: 2,
      label: {
        normal: {
          formatter: '{b}',
          position: 'right',
          show: false
        },
        emphasis: {
          show: false
        }
      },
      hoverAnimation: true,
      rippleEffect: {
        brushType: 'stroke'
      },
      symbolSize: 8,
      data: []
    }

    this._series.push(serieLine, serieScattered);
}

// New event

appendData() {
      const line = {
        seriesIndex: position,
        data: [{
          coords: [srcPosition, dstPosition],
          lineStyle: {
            curveness: Math.random() * (0.4 + 0.4) - 0.4
          },
        }],
      };

      const scatter = {
        seriesIndex: position++,
        data: [{
          name: event.event_type,
          value: dstPosition,
        }]
      }

      this._chart.appendData(line);
      this._chart.appendData(scatter);
}

This release I made, refresh the whole map with this.chart.setOption({series: this.series}) after 100 events as max arrived.
Are there other ways to remove each line without refreshing the whole content??
Any suggestions?
Expected Behavior

For instance, this map shows how the attack should be working on my case:

New feature
It'll be nice to have a removeData() method like appendData() or a {duration: 2s} property for each data added with  appendData().
That can be VERY useful for realtime applications in general.
This map is receiving in realtime more than hundreds events for second!!
I'll do appreciate your help, thank you so much :beers:

I too desperately need this feature.  Another way to do this (at least for me) is to have a feature that limits the number of data points in a series.  But chartInstance.removeData({series_index, element_index_start, element_index_end}) would be nice.

I really need removeData feature as well. Any update?

@kmmertes how did you limit number of data points in a series? Is it from API or UI?


[Bug]: MsSqlBuilder constructor broken in 3.9

Testcontainers version
3.9
Using the latest Testcontainers version?
Yes
Host OS
Windows (but also occurs in Linux Build Agent)
Host arch
x86
.NET version
.NET 7
Docker version
Client:
 Cloud integration: v1.0.31
 Version:           20.10.24
 API version:       1.41
 Go version:        go1.19.7
 Git commit:        297e128
 Built:             Tue Apr  4 18:28:08 2023
 OS/Arch:           windows/amd64
 Context:           default
 Experimental:      true

Server: Docker Desktop 4.18.0 (104112)
 Engine:
  Version:          20.10.24
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.19.7
  Git commit:       5d6db84
  Built:            Tue Apr  4 18:18:42 2023
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.18
  GitCommit:        2456e983eb9e37e47538f59ea18f2043c9a73640
 runc:
  Version:          1.1.4
  GitCommit:        v1.1.4-0-g5fd4c4d
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Docker info
Client:
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc., v0.10.4)
  compose: Docker Compose (Docker Inc., v2.17.2)
  dev: Docker Dev Environments (Docker Inc., v0.1.0)
  extension: Manages Docker extensions (Docker Inc., v0.2.19)
  init: Creates Docker-related starter files for your project (Docker Inc., v0.1.0-beta.2)
  sbom: View the packaged-based Software Bill Of Materials (SBOM) for an image (Anchore Inc., 0.6.0)
  scan: Docker Scan (Docker Inc., v0.25.0)
  scout: Command line tool for Docker Scout (Docker Inc., v0.9.0)

Server:
 Containers: 0
  Running: 0
  Paused: 0
  Stopped: 0
 Images: 7
 Server Version: 20.10.24
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 2456e983eb9e37e47538f59ea18f2043c9a73640
 runc version: v1.1.4-0-g5fd4c4d
 init version: de40ad0
 Security Options:
  seccomp
   Profile: default
 Kernel Version: <IP_ADDRESS>-microsoft-standard-WSL2
 Operating System: Docker Desktop
 OSType: linux
 Architecture: x86_64
 CPUs: 12
 Total Memory: 15.51GiB
 Name: docker-desktop
 ID: ALNS:VWVK:HCLB:T4ZJ:SH23:44E2:VJ5L:ORCV:VZOD:GCDB:5K22:GMCY
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 HTTP Proxy: http.docker.internal:3128
 HTTPS Proxy: http.docker.internal:3128
 No Proxy: hubproxy.docker.internal
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  hubproxy.docker.internal:5555
  <IP_ADDRESS>/8
 Live Restore Enabled: false

WARNING: No blkio throttle.read_bps_device support
WARNING: No blkio throttle.write_bps_device support
WARNING: No blkio throttle.read_iops_device support
WARNING: No blkio throttle.write_iops_device support

What happened?
Hi!
our integration test code breaks with 3.9.
We initialize our test container like this:
    private readonly MsSqlContainer _container = new MsSqlBuilder()
        .WithImage("mcr.microsoft.com/mssql/server:2022-CU12-ubuntu-22.04")
        .WithPassword("***")
        .WithLogger(NullLogger<ContainerFactory>.Instance)
        .Build();

In our Dependabot branch to update from Testcontainers 3.8 to 3.9 this breaks. During test execution (building the project works fine) we get the
exception (see log for original, I formatted it a bit nicer here):
Method not found:
'Void DotNet.Testcontainers.Configurations.ContainerConfiguration..ctor(
  DotNet.Testcontainers.Images.IImage,
  System.Func`2<Docker.DotNet.Models.ImageInspectResponse,Boolean>,
  System.String,
  System.String,
  System.String,
  System.String,
  System.Collections.Generic.IEnumerable`1<System.String>,
  System.Collections.Generic.IEnumerable`1<System.String>,
  System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.String>,
  System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.String>,
  System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.String>,
  System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Configurations.IResourceMapping>,
  System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Containers.IContainer>,
  System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Configurations.IMount>,
  System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Networks.INetwork>,
  System.Collections.Generic.IEnumerable`1<System.String>,
  System.Collections.Generic.IEnumerable`1<System.String>,
  DotNet.Testcontainers.Configurations.IOutputConsumer,
  System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Configurations.IWaitUntil>,
  System.Func`3<DotNet.Testcontainers.Containers.IContainer,System.Threading.CancellationToken,System.Threading.Tasks.Task>,
  System.Nullable`1<Boolean>,
  System.Nullable`1<Boolean>)

I checked in your code base: In DotNet.Testcontainers.Configurations the constructor for ContainerConfiguration now takes a parameter of type IEnumerable instead of IEnumerable. I would expect that this doesn't matter since the constructor is not called explicitly and the parameter has null as a default, but here we are.
Could you check the issue? Perhaps it just needs changing back to the more general IEnumerable? We'd appreciate it it very much.
Kind regards,
Torben
Relevant log output
Message:
OneTimeSetUp: System.MissingMethodException : Method not found: 'Void DotNet.Testcontainers.Configurations.ContainerConfiguration..ctor(DotNet.Testcontainers.Images.IImage, System.Func`2<Docker.DotNet.Models.ImageInspectResponse,Boolean>, System.String, System.String, System.String, System.String, System.Collections.Generic.IEnumerable`1<System.String>, System.Collections.Generic.IEnumerable`1<System.String>, System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.String>, System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.String>, System.Collections.Generic.IReadOnlyDictionary`2<System.String,System.String>, System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Configurations.IResourceMapping>, System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Containers.IContainer>, System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Configurations.IMount>, System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Networks.INetwork>, System.Collections.Generic.IEnumerable`1<System.String>, System.Collections.Generic.IEnumerable`1<System.String>, DotNet.Testcontainers.Configurations.IOutputConsumer, **System.Collections.Generic.IEnumerable`1<DotNet.Testcontainers.Configurations.IWaitUntil>**, System.Func`3<DotNet.Testcontainers.Containers.IContainer,System.Threading.CancellationToken,System.Threading.Tasks.Task>, System.Nullable`1<Boolean>, System.Nullable`1<Boolean>)'.

  Stack Trace:
MsSqlConfiguration.ctor(String database, String username, String password)
MsSqlBuilder.ctor() line 21
ContainerFactory.ctor() line 22

Additional information
No response

Do you use multiple modules? Please make sure you update all dependencies to version 3.9.0.

My bad, you're right.
Normally I get seperate pull requests for each module of a package and immediately see when I have to update them together. Somehow I didn't get a pull request for Testcontainers.MsSql and overlooked it in ther global packages.props.
Thanks a lot! The tests worked after I increased the version for both modules.

Yes, this is an unfortunate issue. I am still unsure how to bypass it or at least mitigate it. Since modules always depend on a base version of Testcontainers, it is usually necessary to update all of them.


INT B-21490 Office users can user their secondary duty locations

B-21490
Summary
Office users are able to switch between their primary and secondary transportation office GBLOCs similar to an HQ user.
The bulk of the secondary transportation office changes were added in B-21492 and are also included in this PR.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the
Login as a

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

Warnings

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

  Generated by :no_entry_sign: dangerJS against 884e3a2f216caa59e9c37194507e9ab789cc0b61

Too many failure for it to be flaky, putting it back as draft temporarily.

Tested with SC supervisor role, Origin duty location stayed when switching GBLOCS and all moves displayed properly. Looks good for TOO as well.


CI: Add valgrind

Will try to add some valgrind tests to CI

Finally learnt to deal with subrepositories...

I think it should be good for merging now


actions slot for ChatMessage?

Any plan to add actions slot for ChatMessage? ChatMyMessage has actions slot but not ChatMessage.
Trying to figure out how to add Teams app style reaction hover menu.

There is no plan to add a slot for this. The ChatMessage is actually a custom Popover trigger https://github.com/microsoft/fluentui-contrib/blob/73f3909177ececcd0a81bd366ba00006b5360873/packages/react-chat/src/components/ChatMessage/useChatMessage.ts#L104
When you wrap a popover around a chat message it will automatically be positioned relative to the bubble. Here's an example https://codesandbox.io/p/sandbox/adoring-saha-tfh6yw?file=%2Fsrc%2Fexample.tsx%3A42%2C17


[Feature] Pawn promotion

As a user I want to have the option to promote my pawn to another piece when it reaches the "end of the board", so that I can relate to the common chess rules.

There are 2 options:

promotion to queen, rook, knight or bishop and the player can choose it
automatical promotion to queen (a statistic says that 96.9% of the promotion is to a queen)


How to use Spin for several images on a portfolio page

Hi,
I'm actually working on my website (www.christophehassel.com/portfolio). I made a portfolio using Wookmark Jquery and everything works fine, but I'd like to add a loading animation for each images for the slow connexions.
I'd like to use Spin but I don't know where to start from!
I would really appreciate a bit of help!
Thanks
Christophe

I think it would be better to ask this question on Stack Overflow. It's not an actual issue in spin.js.


Fututre incident with resolved status

Futurre incident without resolved [start_datetime:1706016000] [end_datetime:1706016180]

Incident no 41


Eloquent Collections causing FatalErrorException when using fetch() & unique() together

Hi,
I have a problem with using fetch() & unique() together.
That's because \Illuminate\Database\Eloquent\Collection::unique() overrides \Illuminate\Support\Collection::unique() with different implementation.
$user = User::find(1);
$posts = $user->posts;

$uniqueTitles = $posts->fetch('title')->unique();

It causes a FatalErrorException:
Symfony \ Component \ Debug \ Exception \ FatalErrorException (E_ERROR)
Call to a member function getKey() on string

…/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Collection.php:237

Is that an issue?
Thanks :)

fetch serializes the model into an array. If you then want to run unique on them, you should use the base collection:
$uniqueTitles = $posts->fetch('title')->toBase()->unique();

Thanks:)


fix: replace Math.abs with tenary statements

This will help improve the performance in Chrome
Hi @jankovicsandras , we have experienced "Intermittent slowdown in colorquantization function" in our application(same as #38 )
I have narrowed it down to a for loop inside of colorquantization and it seems that the Math.abs inside of that forloop is causing the performance issue. So I have replaced the Math.abs with a tenary statement and it resolved the problem. (Not sure why Math.abs is rnning slow on Chrome but not on Firefox)

Hi,
Thanks for your investigation and your PR, great work!
It still leaves me with questions, because some symptoms doesn't match in "Intermittent slowdown in colorquantization function #38".

@jminardi mentions that the first tracing is usually fast enough, but the slowdown happens after repeated function calls. Full reload fixes the problem temporarily, which points to that some wrong internal state could be the cause.

Issue #38 suggests that the slowdown is related to Gaussian blur preprocessing.

As I understand, if Math.abs() would be the cause, then we would experience constant slow performance, independent of the blur preprocessing.
But this aside, I still think it's a good idea to replace Math.abs(), because:

I believe your analysis. I haven't tested this in Chrome.
I saw articles and Stackoverflow questions which suggest slow Math.abs() in V8.
I read the source of (an earlier version of) V8 where Math.abs() seemed to handle all types and edge cases, which are not required for byte-byte subtraction. (imagetracerjs supports only RGBA8888, but it's easy to adapt to 16bit/channel)
I love ternary operators :)

But instead of this
// cd = Math.abs(palette[k].r-imgd.data[idx]) + Math.abs(palette[k].g-imgd.data[idx+1]) + Math.abs(palette[k].b-imgd.data[idx+2]) + Math.abs(palette[k].a-imgd.data[idx+3]);
var a = palette[k].r-imgd.data[idx] > 0 ? palette[k].r-imgd.data[idx] : imgd.data[idx]-palette[k].r;
var b = palette[k].g-imgd.data[idx+1] > 0 ? palette[k].g-imgd.data[idx+1] : imgd.data[idx+1]-palette[k].g;
var c = palette[k].b-imgd.data[idx+2] > 0 ? palette[k].b-imgd.data[idx+2] : imgd.data[idx+2]-palette[k].b;
var d = palette[k].a-imgd.data[idx+3] > 0 ? palette[k].a-imgd.data[idx+3] : imgd.data[idx+3]-palette[k].a;
cd = a + b + c + d;

how about this ?
cd =
	( palette[k].r > imgd.data[idx  ] ? palette[k].r - imgd.data[idx  ] : imgd.data[idx  ] - palette[k].r ) +
	( palette[k].g > imgd.data[idx+1] ? palette[k].g - imgd.data[idx+1] : imgd.data[idx+1] - palette[k].g ) +
	( palette[k].b > imgd.data[idx+2] ? palette[k].b - imgd.data[idx+2] : imgd.data[idx+2] - palette[k].b ) +
	( palette[k].a > imgd.data[idx+3] ? palette[k].a - imgd.data[idx+3] : imgd.data[idx+3] - palette[k].a );

Hi @jankovicsandras ,
Thanks for your reply!
Yes, I think your suggestion is better. I have updated the PR accordingly. : )
Regards,
Joseph

Hi @jankovicsandras , do you think we can cut a new release of ImageTracer that incorporates this change anytime soon?


Fixed Chicken Missing Chins

Updated Chicken Textures to have chins where it makes sense.

You are awesome! Thank you!


Ref's are called twice when rerendering

Simple test case:
<!DOCTYPE html>

  <meta charset="utf-8">
  <title>Preact Ref Bug</title>

When running the above test, the console.log statement is printed twice, once with null and once with the actual ref.

@marvinhagemeister Ah - you're calling bind() in render, so every render is giving a newly created bound function as a ref to the div.  Hoisting the bind should fix the issue.

@developit ohh.. you're right! One always learns something new. Thanks for the quick answer 💯

@developit sadly a lot of people from React world do so. Especially non-JS ones coming from Ruby, etc. and just doing so because it was done in some examples 😐


No valid option generated in #select when options don't start with spaces

The bug
When I run the code below, I get the following error:
ValueError: No valid option generated in #select! Please post a GitHub issue since this should not happen :)
To Reproduce
# doesn't work
guidance(
    "Hello, {{#select 'name'}}Alice{{or}}Bob{{/select}}",
    llm=guidance.llms.Transformers("gpt2"),
)()

# works
guidance(
    "Hello,{{#select 'name'}} Alice{{or}} Bob{{/select}}",
    llm=guidance.llms.Transformers("gpt2"),
)()

System info (please complete the following information):

OS: PopOS 22.04
Guidance Version (guidance.__version__): 0.0.49

Should be clone of #67 for which I opened #78 .


small typo fix

Well a newer version of translation has been updated to suggest Nuxt directly. So this pull request will be discarded.
But always thank you very much!


refactor(app): button positioning in pipette flow modals

closes RQA-1165 and RQA-1166
Overview
Button positioning for the on device display modals in the pipette flows
Test Plan
Verify that the screenshots match the ticket. I made the changes to only affect the on device display modals so also verify that the desktop modals look okay
I also noticed that the button in the Calibration error modal was incorrectly placed for the odd so i fixed that as well
(this screenshot is the bonus fix)

Changelog

added stylings to the different buttons and made sure it only affected the onDeviceDisplay modal

Review requests
see test plan
Risk assessment
low

Do you think you can fix the scrollbar in this PR?
I think we don't need to display it in a wizard.

From the design, the button should be in the right side.

From the design, the button should be in the right side.

https://github.com/Opentrons/opentrons/assets/474225/13464275-2c1d-46d7-97c0-0d6d76c7855e
I guess there would be two approaches for this.

move Exit to the right col space and flex-end
change the layout for touchscreen (1 col for exit button)

move Exit to the right col space and flex-end
change the layout for touchscreen (1 col for exit button)

thanks @koji , i'll probably move it to flex-end like i did with the other modals. Thanks for noticing that this button is misaligned, I missed it!
also I'll make sure that other modal doesn't have a scroll.


Fix autocomplete weird suggestions

This autocomplete weirdness is because

For autocomplete suggestions i used this json file. https://gist.github.com/vinaypuppal/1409b9acc7d933156644501cd79c0df5
I extracted the most frequent words from links meta data and placed them in above file.
I used many filters to eliminate unrelated words and sorted them according to their frequency.
But still they contain some weird words.
Some suggestions at end contain some weird characters, I dont know how to filter or remove them

So if anyone interested in further filtering this list can be helpful to fix this issue.
This is code i used to extract tags https://github.com/vinaypuppal/linklet-tags

Those seems to be unicode characters. Filtering them fixed the issue.


Pod affinity and Anti affinity changes

This PR addresses #51 .
Adding the E2E tests for Pod Affinity and Pod-Anti-Affinity and some minor fixes,
like namespace addition in the task descriptor and addition of missing operators.
/assign @deepak-vij

/ok-to-test


How is RE-flex different from reflex?

There exists a project with a similar name that appears to be a variant of flex as well:
https://invisible-island.net/reflex/

When releasing RE/flex back in 2016, I searched the web for "reflex" with flex but could not find any public distros and repos that matched both names with the project name being "reflex". Anyway, RE/flex is the official name of this project, suggesting a combination of "RE" (regular expression libraries) with "flex". So I don't see a problem.


[Fix #3558] Mixin autocorrection logic moved into Corrector classes

This implements refactoring discussed in #3558 .
This procedure brought forth some drawbacks, but it also encapsulated logic successfully. The result is that every cop now defines it's own #autocorrect, and nothing is hidden inside a mixin. Breakdown of #autocorrects defined in mixins from issue:

What's wrong with it?

If I want to use a mixin in a cop that shouldn't implement auto-correct, I need to add a no-op #autocorrect definition to the cop. This is pretty icky.
If I want to use two mixins, both implementing #autocorrect in a cop, which method is finally defined depends on the order of the mixins. This is very fragile.
There might be more than one #autocorrect that makes sense for a mixin, depending on the context it is being used.
There is no easy way to see if, and if so which, auto-correct is defined from within a Cop body.

Related "issue" this PR resolves
Since this meant moving around a lot of functionality in our mixins, I took this as an opportunity to make all mixin methods private where possible.  This is related to #4439 .  The remaining public methods in mixins must remain public. ... Unfortunately, these additions and movement of the private modifier leak into most commits and are not encapsulated to just one. Sorry!
Something this PR does NOT do
Catch every place where refactoring is warranted. -- I resolved a few extra places where corrector logic had seeped into mixins outside of #autocorrect methods directly. However, some alterations got complicated, particularly when dealing with how our test suite dynamically changes cop_config.  Once this PR reaches a deliverable status and is merged, I'll open up a couple issues for the places that deserve their own PR to extract Corrector logic.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[fail] Squashed related commits together.
[x] Test coverage on all bits of this code.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

Great job! 👍


Added index for seq_region_mapping see ENSPROD-7014

Description
Deduplicates row is seq_region_mapping table. see ENSPROD-7014 for details
Use case
Spotted duplicates in seq_region_mapping
Benefits
No more duplicates :-)
Possible Drawbacks
None
Testing
Have you added/modified unit tests to test the changes?
If so, do the tests pass/fail?
Have you run the entire test suite and no regression was detected?

Closing this as we are pushing forward #593


Training descriptions serialization

Related to #153
As it stands the training descriptions are made in an ad-hoc format:
https://github.com/persephone-tools/persephone/blob/aa695758e675f47e91ef79beb0f470e97f754e6e/persephone/model.py#L202-L215
There are a few concerning things here, especially the use of getting info directly out of the frame. I think this was a really smart way of dealing with this initially but I'd like to move over to use a more specific method from Tensorflow for extracting relevant variables, especially if more than one model is available at a time.
I propose that we make this a more easily parseable format when we write this to a file.
I'm thinking something like YAML or JSON works well here. Is there a preference for serialization format?
JSON comes in the standard library but has a bit less flexibility. That said flexibility isn't necessarily a benefit here.

I don't have a format preference. But whatever we do here it's important that the output file gives a complete picture of the experiment so it can be reproduced. This should include, but may not be limited to: the model information and the corpus information (as well as perhaps a hash of the data too).


AI Ethics and inadequate training sets

AI Ethics is mentioned in section 1.5.
One thing that is not mentioned, is the danger of (unconsciously) picking a biased training set (which is not mentioned in Section 1.2).
That is exemplified by Google's image recognition that misidentified pictures of Blacks as apes.
See:
https://www.theguardian.com/technology/2018/jan/12/google-racism-ban-gorilla-black-people

I cite this paper that covers these and other topics
https://ml-retrospectives.github.io/neurips2020/camera_ready/19.pdf
“Data and its (dis)contents: A survey of dataset develop- ment and use in machine learning research”.

I cite this paper that covers these and other topics
https://ml-retrospectives.github.io/neurips2020/camera_ready/19.pdf
“Data and its (dis)contents: A survey of dataset develop- ment and use in machine learning research”.


Is there a way to use a raster source ?

Hello,
I tried to no avail to plug a map to a local tileserver-gl instance serving custom raster tiles (tileJson file). After digging in the source code of nativscript-mapbox plugin & mapbox-gl-native 6.5.0 I did found an addSource method but it only support vector sources.
From here I see 3 possible solution for my problem :

I didn't dig enough and there's a way to use raster maps & tilejson files, can you point me to right direction ?
Somehow turn my aerial views into vectors
Fork this plugin to add the capability to handle raster source and maybe submit a PR but it would only be on android as I don't have any knowledge & mac to test on iOS.

Please, tell me if you need more information or if I need to rephrase my issue.
Thanks

It is already possible to use raster tiles with this plugin, take a look at the demo app as it uses OpenStreetMap raster tiles - also take a look at the MapBox style JSON spec for more information.
To use them, first off create a may style JSON file. An example can be found in the demo app here
Then with the onMapReady event you can set the map style to the style we defined in the JSON file, an example of this is in the demo app also here
Hope this helps! :)

Thank you for you reply, indeed I missed this example and didn't thought about local json file. I tried to replicate the exemple :
MyActivity.vue :
<Mapbox
        accessToken="noToken"
        mapStyle="~/app/style.json or ~/style.json" (tried booth)
        hideCompass="true"
        hideAttribution="true"
        hideLogo="true"
        @mapReady="onMapReady($event)"
        ref="map">
</Mapbox>

After a build I can see my file style.json next to my bundle in /platforms/android/app/src/main/assets/app/ but when I launch my app Mapbox can't find the style.json file nothing happen and there is this error in logcat :
E/Mbgl: {MyApp}[Setup]: loading style failed: Could not read asset
Am I missing something ?

Hi,
I think that this may be a permissions issue, unfortunately I'm not experienced in using Vue however I have attempted to have a play around. What leads me to believe that this is a permissions issue is when the JSON file is loaded in locally, the app fails however if you load the JSON file in from a remote server it works fine, give it a try using the raw JSON file from the example (here). However when I have tried this using NativeScript (Javascript) and had no such issues and loaded in just fine locally.
I've searched through the issues and it appears that other's are having the same issue, take a look here #286

Thanks everyone for your replies, after spending a lot of time trying to make this work my team and I decided to give up on nativescript-mapbox & nativescript altogether. Apart from the issues with raster sources we need feature from newest version of the SDK that are not present here and we don't have enough bandwidth to do it ourselves and submit a PR.


fix: broken links 🐛

The issue here is when astro sees src/content/docs/path/to/filename.md it renders path/to/filename/index.html.
In path/to/filename.md, linking to path/to/filename2.md is usually done by ./filename2.md but for the path to work in the web it has to be ./../filename.
An alternative solution here would be to have every file named index.md in the appropriate folders. Then the relative paths will always match correctly.

Ideally astro would just figure this out when it sees links in a markdown file. Alternatively, we can write a remark plugin to do this ( I think I've already written one for this in the past when my personal blog was in svelte ).

It's weird that cloudflare didn't post a preview here? I see the successful build on their website though.


Search disappear from SideBar

How to get back my search icon ? No extension, no theme.. No search :(

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.81.0
OS Version: Ventura 13.4

Use Command Palette to run the command to rest view locations.

Ok but if I want to use the sidebar, how ?

The exact command to run is  View: Reset View Locations


Upgrade web3 to 1.0.0-beta.48

fix #135

Pull Request Test Coverage Report for Build 179

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.2%) to 87.327%

Totals

Change from base Build 177:
-1.2%

Covered Lines:
309

Relevant Lines:
341

💛  - Coveralls

@cryptokat -- I think this PR is looking good to generate typings for<EMAIL_ADDRESS>with default typings. But dropping the legacy support isn't really good idea at this stage as there might be lot of projects out there relying upon<EMAIL_ADDRESS>I'm sure @krzkaczor will agree on this.

@eswarasai I decided to drop legacy support because of broken test/package-test that needs<EMAIL_ADDRESS>to be working, otherwise the dependencies on the sub packages of web3 such as web3-core will cause compilation error.

Dropping legacy support is a no-go. Personally, I feel like web3 0.20.x is way more stable. Please revert the removal and tmrw I will help you fixing those failing tests (sorry it's late at night here).

@eswarasai @krzkaczor I reverted the removal of legacy support and fixed the package-test by excluding it from tsc compilation https://github.com/ethereum-ts/TypeChain/pull/141/commits/ddc6ae0f61ce1563b2453934ff0dc1eb6b40fe26 . Please have a look if that's alright.

In case that you are not aware of the error I mentioned above, here's the error I was getting from package-test before excluding it from tsc compilation:

@cryptokat thanks! Frankly, I can't reproduce these problems that you were encountering but I think ignoring package-test in global tsconfig.json file makes sense.

btw. i think this is slightly related to: https://github.com/ethereum-ts/TypeChain/pull/129

@cryptokat @eswarasai PR looks fine for me. Did anyone of you tested it locally in your projects? I can merge this and publish new version today.

@krzkaczor -- I haven't tested this locally but the changes are looking good to me. I think we're missing a use case here where people might be using web3.js and the types from @types/web3 in their existing codebase and might not be easy for them to upgrade to the types within web3.js.
It might help devs to know what are the breaking changes within this release and what's not supported going forward in the README.md. Let me know if that makes sense. Thanks!

@eswarasai yeah I imagine that people would use current ie. 0.3.11 version if they want to stick with older web3.js (without embedded types).
Yes, I will for sure leave a note about it in the changelog once it's merged. Thanks for reminder 😆

Thank you for contributing! @cryptokat  🎉
Released as<EMAIL_ADDRESS>

In my project the Transaction parameter taken by the send() is the wrong interface after updating and following this thread. It requires all values to be passed in as seen here:

which leads to this error Im getting:

I think it has something to do with the Transaction object web3-core is importing in the type.d.ts file. The value should be of type any

Is anyone else encountering this issue? Fore reference I am using the latest web3 and typechain

Hey @bryanwho could you create small repo example? I will get this sorted out

@krzkaczor just sent you an invite to a skeleton repo

Hey @krzkaczor any follow up on this: https://github.com/bryanwho/TypeChainTesting

@bryanwho @krzkaczor I'm having the same issue mentioned above with the latest web3 and typechain. Please see this repo
https://github.com/jchen86/typechain-web3-pr-114/blob/2debfa4278029f30923c5c047ae4fb6c06187a0e/test/simple-storage.test.ts#L6


complete client/server demo

@christian-posta i see lots of conflicts on these files,
src/main/fabric8/configmap.yml
src/main/fabric8/deployment.yml
src/main/fabric8/envoy-admin-service.yml
src/main/fabric8/service.yml
possible for you to rebase with my code and send in another PR ?

Yah let me rebase ... there are prob going to be conflicts anyway because
of how I changed the spring mvc service. But let me take a look at what's
conflicting.
On Sun, Mar 12, 2017 at 9:12 AM Kamesh Sampath<EMAIL_ADDRESS>wrote:

@christian-posta https://github.com/christian-posta i see lots of
conflicts on these files,
src/main/fabric8/configmap.yml
src/main/fabric8/deployment.yml
src/main/fabric8/envoy-admin-service.yml
src/main/fabric8/service.yml
possible for you to rebase with my code and send in another PR ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kameshsampath/sb-webmvc-envoy/pull/1#issuecomment-285954930,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADP0XBSDs0G7juFtY1V9l2ycVweluWKks5rlBl4gaJpZM4MalRU
.

--
Christian Posta
twitter: @christianposta
http://www.christianposta.com/blog
http://fabric8.io

I made some changes yesterday and today morning - though should be simple ones adjusting ports et al.
@christian-posta i have also give you push access to my repo, so feel free to push directly if you wish ;)


redoc no longer works

openapi-enforcer docs-build e.json e.html
The output is "Built successfully" but no file is created.
CLI v0.3.5
OpenAPI Enforcer v1.12.0

LOL. That's rich. "Build successfully". Apparently not. I'll take a look.

OK, this is now working in CLI v0.3.6 and has been published to NPM.
Thanks for the bug report.


Chrisda to Master

OPS Build status updates of commit 5e205f5:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit 5e205f5:
:clock10: Incremental building: average incremental building time is 12 min(s) 23 sec(s)

OPS Build status updates of commit 5e205f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/advanced-threat-protection/Set-AtpPolicyForO365.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/encryption-and-certificates/Get-SmimeConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/encryption-and-certificates/Set-IRMConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/encryption-and-certificates/Set-SmimeConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mail-flow/Set-RemoteDomain.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailbox-databases-and-servers/Set-MailboxDatabase.md
:white_check_mark:Succeeded
View (exchange-ps)

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.


New package: NestedTests v0.1.0

Registering package: NestedTests
Repository: https://github.com/tanaylab/NestedTests.jl
Created by: @orenbenkiki
Version: v0.1.0
Commit: 952a3e4baef2491ac07889d3bd6a858429b85465
Reviewed by: @orenbenkiki
Reference: https://github.com/tanaylab/NestedTests.jl/commit/952a3e4baef2491ac07889d3bd6a858429b85465#commitcomment-123708159
Description: Run tests in nested evironments.

Looking at the details of the failed auto merge, it isn't clear what I need to fix. The errors look like this:
┌ Error: Some registries failed to update:
│     — `/tmp/jl_Xq5VUM/registries/General` — registry dirty
└ @ Pkg.Registry /opt/hostedtoolcache/julia/1.9.2/x64/share/julia/stdlib/v1.9/Pkg/src/Registry/Registry.jl:510

Which is repeated later:
┌ Error: Some registries failed to update:
│     — `/tmp/jl_PYsPdf/registries/General` — registry dirty
└ @ Pkg.Registry /opt/hostedtoolcache/julia/1.9.2/x64/share/julia/stdlib/v1.9/Pkg/src/Registry/Registry.jl:510

And leads to the final:
ERROR: LoadError: RegistryCI.AutoMerge.AutoMergeGuidelinesNotMet("The automerge guidelines were not met.")

Anything I can do to fix this?


Upgrade to SGX SDK v1.1.0

Description
This PR is to upgrade teaclave-sgx-sdk v1.1.0 and Intel SGX SDK 2.7.1. Everything is fixed and test passed on my workstation.
Fixes # (issue)
Type of change (select applied and DELETE the others)

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] API change with a documentation update
[ ] Additional test coverage
[x] Code cleanup or just sync with upstream third-party crates

How Has This Been Tested?
make sgx-test passed. CI will soon pass at https://ci.mesalock-linux.org/dingelish/incubator-mesatee/36
Checklist (check ALL before submitting PR, even not applicable)

[x] Fork the repo and create your branch from master.
[x] If you've added code that should be tested, add tests.
[x] If you've changed APIs, update the documentation.
[x] Ensure the tests pass (see CI results).
[x] Make sure your code lints/format.

CI passed
https://ci.mesalock-linux.org/dingelish/incubator-mesatee/36

BTW, can you fix this bug in this PR? https://github.com/apache/incubator-teaclave/issues/153
This may cause some issues for submodule updates.

let me try

The fix is to check:

if any of the submodule is empty
git submodule summary's output is not empty

if either (1) or (2) is satisfied, do submodule update
diff --git a/cmake/MesaTEEUtils.cmake b/cmake/MesaTEEUtils.cmake
index 8c2f0cd..709937f 100644
--- a/cmake/MesaTEEUtils.cmake
+++ b/cmake/MesaTEEUtils.cmake
@@ -20,7 +20,7 @@ endfunction()

 function(init_submodules)
     execute_process (
-    COMMAND bash -c "if git submodule status | egrep -q '^[-]|^[+]'; then echo 'INFO: Need to reinitialize git submodules' && git submodule update --init --recursive; fi"
+    COMMAND bash -c "if [[ ! $(git submodule foreach ls -A) ]] || [[ $(git submodule summary) ]]; then echo INFO: Need to reinitialize git submodules && git submodule update --init --recursive; fi"
     )
 endfunction()

New CI result is at https://ci.mesalock-linux.org/dingelish/incubator-mesatee/39