Caching performance should be better

Mountpoint for Amazon S3 version
mount-s3 1.3.2
AWS Region
us-west-2
Describe the running environment
Ubuntu 22.04 and Amazon Linux 2023
Mountpoint options
sudo mount-s3 --cache /mnt/raid0 raykrueger20240124--usw2-az1--x-s3 /mnt/s3

What happened?
Using the --cache arg provides no performance benefit.
I have my cache dir set to /mnt/raid0, which is a 2 nvme raid0 mount. When I read a 10G file from my S3 mount it takes 9-10 seconds. When I read the file every time after the first, I expect that time to be the same speed as reading from that /mnt/raid0 location. Every subsequent read takes the same amount of time as the first. Also, it takes the same amount of time if I have no --cache at all.
I see the same behavior mounting directory (express) or regular S3 buckets.
Relevant log output
# mount-s3 --cache /mnt/raid0 raykrueger20240124--usw2-az1--x-s3 /mnt/s3
bucket raykrueger20240124--usw2-az1--x-s3 is mounted at /mnt/s3

# time cat /mnt/s3/data/file_1 > /dev/null

real	0m9.108s
user	0m0.007s
sys	0m4.287s
# time cat /mnt/s3/data/file_1 > /dev/null

real	0m7.780s
user	0m0.012s
sys	0m4.683s
# time cat /mnt/s3/data/file_1 > /dev/null

real	0m7.410s
user	0m0.014s
sys	0m4.509s

The 2s drop is margin for error variation.
The cache is populated.
# du -sh /mnt/raid0/mountpoint-cache/
11G	/mnt/raid0/mountpoint-cache/

This is the cache performance I expect.
# time cat /mnt/raid0/mountpoint-cache/V1/9c/eaf5ecb13463aa2b4834c069b935949355d31e8b9c201fe4f3b461ad5d0132/* > /dev/null

real	0m1.587s
user	0m0.010s
sys	0m1.576s

Based on looking at metrics in the debug logs it looks like the cache is "working", blocks are pulled and read from cache. The performance is not close to what I would expect for reading from disk. The same 10G file copied to my /mnt/raid0 disk comes off as such...
# time cat /mnt/raid0/file_1 > /dev/null

real	0m1.150s
user	0m0.000s
sys	0m1.150s

Next steps I'm going to try adding some flamegraphs to the tracing and see where this peaking.

After sprinkling in a bunch of instrument attributes, spans, and a flamegraph layer; I can confirm that all of our time is spent in deserializing at mountpoint_s3::data_cache::disk_data_cache.

Thanks for reporting this. In [1], we are introducing cache benchmarks. With this in place, we want to explore improvements by experimenting with changing the serialization library, for example.
[1] https://github.com/awslabs/mountpoint-s3/pull/761

I should mention that the tracing changes I made are in my fork here https://github.com/raykrueger/mountpoint-s3/tree/flamegraphs
I had no intention of sending a PR for the changes as they were pretty heavy handed.

We recently tested using a RAM-based cache, which eliminated the need for: 1) Serialization 2) Disk I/O.
Despite these improvements, we still observed an overhead compared to the baseline of reading the file outside of Mountpoint. This overhead seems to be attributable to FUSE itself.
Additionally, we don't expect the performance of Mountpoint to match that of the EXT4 filesystem due to the following reasons: 1) Mountpoint performs integrity checks 2) Mountpoint performs serialization.
Generally, we believe that for large files, the cache serves more as a cost saver for requests rather than a performance enhancer.


Leaving Template Tab from TemplateDetail then returning glitches out

When in TemplateDetail on iPhone, going to Trips tab, and returning, it launches the detail again and glitches out.
This can probably be fixed by an “ondisappear” set selection to none

Wait, still here, on iPad, if you do it fast enough, it returns,
Maybe try an onappear to templatehome on iPad in that orientation?

an ondisappear in templatehome seemed to work


ComplianceChecker update

This PR includes:

copy of 0.7.0 rules to 1.0.0
making 1.0.0 the default version
added checks for gradinet_accumulation_steps (#78 )
adding RNN-T rules (it includes a new check for weights_initialization key (#80 ))
adding Unet3D rules

recheck


Profile Link Broken

Bug
The profile link is not working. This is a recurring bug, and a problem because users must activate the Phone Role to answer calls.
To Reproduce
Steps to reproduce the behavior:

Log in.
Click "Profile" under the dropdown for your name
Bug: Nothing happens.
Also, there is extra white space to the left. Formatting seems incorrect.
This is a recurring bug.

Expected behavior
Profile should load
Screenshots

Desktop (please complete the following information):

Win 10, FF; Chrome

resolved in cd7b9ab
Will be live in next production release


(new/feature request)

I want to take a backup of the whole postman and export it. including env

@muhammedfurkan yes you can import postman collections and environments now one file at a time.

@muhammedfurkan You can now select multiple collection files to import. Please update to the latest version and try.


[MTSRE-1664] Fix Addon Operator package deployment - Phase I

What type of PR is this?
(bug/feature/cleanup/documentation/test/refactor)
Bug
What this PR does / why we need it?
MTSRE-1664
Which Jira/Github issue(s) this PR fixes?
https://issues.redhat.com/browse/MTSRE-1664
Fixes #
Special notes for your reviewer:
Pre-checks (if applicable):

[ ] Tested latest changes against a cluster
[ ] Ran make test-unit command locally to run all the unit tests and mock tests locally.
[ ] Included documentation changes with PR
[ ] Squashed your commits

Codecov Report

Merging #525 (de523c8) into main (9be2ac9) will not change coverage.
Report is 7 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #525   +/-   ##
=======================================
  Coverage   73.88%   73.88%
=======================================
  Files          51       51
  Lines        3240     3240
=======================================
  Hits         2394     2394
  Misses        708      708
  Partials      138      138

I'd like to see the package validation as part of this PR as well.
/hold

/retest

/retest

/unhold
Package validation will done in a separate PR to unblock deployment.


Create User Schema

Creating an issue for creating user schema.

@yuhwaa doesn't look like I can assign you.

@Caleb-Cohen thanks for opening the issue — I think you should be able to assign it after this comment.

Perfect @yuhwaa it's all yours!

Thank you! Noting discussion #9 for reference.

User Schema updated at #23

Thank you! @yuhwaa

🙏🏻


Add test coverage for various components to meet quality gate

Language.ForProject
ConnectionInformation.Ctor
SectionController
SonarQubeServiceWrapper
SolutionBindingOperation
SolutionRuleSetsInformationProvider
Various (null arg checks)

LGTM


Sensu first client(the master itself) is not shown in uchiwa

Hello,
I have recently set up the sensu,here is the env
OS= Centos7
rabbitmq-server-3.4.1-1.noarch
erlang-erts-R16B-03.17.el7.x86_64
sensu-0.20.3-1.x86_64
Sensu+uchiwa installation was done through the sensu's own repository
Created the certs based on the cert script and set in the right directory:
clients in the /etc/sensu/ssl/
servers in the /etc/rabbitmq/ssl
Opened up the required ports and all stuffs now all are running with no issues at the systemctl status command
However can't get the first client- being the server itself- into the uchiwa management no matter how hard I try...
I have spent almost 2 days various troubleshooting but still it does not budge...I can paste the config files but pretty sure that all are fine. Now the nearest I can get is the error in the:
`[root@sensu conf.d]# tail -n 10<EMAIL_ADDRESS>=ERROR REPORT==== 12-Nov-2016::17:35:17 ===
closing AMQP connection <0.1087.0> (<IP_ADDRESS>:49534 -> <IP_ADDRESS>:5671):
{handshake_error,starting,0,
{amqp_error,access_refused,
"PLAIN login refused: user 'sensu' - invalid credentials",
'connection.start_ok'}}
=INFO REPORT==== 12-Nov-2016::17:35:19 ===
accepting AMQP connection <0.1092.0> (<IP_ADDRESS>:49536 -> <IP_ADDRESS>:5671)
[root@sensu conf.d]#`
My netstat output:
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:6379          <IP_ADDRESS>:*               LISTEN      1176/redis-server 1
tcp        0      0 <IP_ADDRESS>:4369            <IP_ADDRESS>:*               LISTEN      2306/epmd
tcp        0      0 <IP_ADDRESS>:3030          <IP_ADDRESS>:*               LISTEN      3860/ruby
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN      1165/sshd
tcp        0      0 <IP_ADDRESS>:4567            <IP_ADDRESS>:*               LISTEN      3906/ruby
tcp        0      0 <IP_ADDRESS>:15672           <IP_ADDRESS>:*               LISTEN      4144/beam
tcp        0      0 <IP_ADDRESS>:25            <IP_ADDRESS>:*               LISTEN      2383/master
tcp        0      0 <IP_ADDRESS>:25672           <IP_ADDRESS>:*               LISTEN      4144/beam
tcp6       0      0 :::4369                 :::*                    LISTEN      2306/epmd
tcp6       0      0 :::22                   :::*                    LISTEN      1165/sshd
tcp6       0      0 :::3000                 :::*                    LISTEN      3936/uchiwa
tcp6       0      0 ::1:25                  :::*                    LISTEN      2383/master
tcp6       0      0 :::5671                 :::*                    LISTEN      4144/beam
tcp6       0      0 :::5672                 :::*                    LISTEN      4144/beam
udp        0      0 <IP_ADDRESS>:45160           <IP_ADDRESS>:*                           3608/dhclient
udp        0      0 <IP_ADDRESS>:43119           <IP_ADDRESS>:*                           2523/dhclient
udp        0      0 <IP_ADDRESS>:3030          <IP_ADDRESS>:*                           3860/ruby
udp        0      0 <IP_ADDRESS>:68              <IP_ADDRESS>:*                           3608/dhclient
udp        0      0 <IP_ADDRESS>:68              <IP_ADDRESS>:*                           2523/dhclient
udp6       0      0 :::38552                :::*                                2523/dhclient
udp6       0      0 :::28362                :::*                                3608/dhclient`
[root@sensu log]# tail sensu/sensu-client.log
{"timestamp":"2016-11-12T17:19:16.538793+0330","level":"warn","message":"config file applied changes","file":"/etc/sensu/conf.d/api.json","changes":{"api":{"bind":[null,"<IP_ADDRESS>"]}}}
{"timestamp":"2016-11-12T17:19:16.538845+0330","level":"warn","message":"loading config file","file":"/etc/sensu/conf.d/uchiwa.json"}
{"timestamp":"2016-11-12T17:19:16.538912+0330","level":"warn","message":"config file applied changes","file":"/etc/sensu/conf.d/uchiwa.json","changes":{"sensu":[null,[{"name":"Sensu","host":"localhost","ssl":false,"port":4567,"path":"","timeout":5000}]],"uchiwa":[null,{"port":3000,"stats":10,"refresh":10000}]}}
{"timestamp":"2016-11-12T17:19:16.538973+0330","level":"warn","message":"loading config file","file":"/etc/sensu/conf.d/client.json"}
{"timestamp":"2016-11-12T17:19:16.539022+0330","level":"warn","message":"config file applied changes","file":"/etc/sensu/conf.d/client.json","changes":{}}
{"timestamp":"2016-11-12T17:19:16.539066+0330","level":"warn","message":"loading config file","file":"/etc/sensu/conf.d/rabbitmq.json"}
{"timestamp":"2016-11-12T17:19:16.539121+0330","level":"warn","message":"config file applied changes","file":"/etc/sensu/conf.d/rabbitmq.json","changes":{"rabbitmq":{"host":["localhost","<IP_ADDRESS>"],"password":"REDACTED"}}}
{"timestamp":"2016-11-12T17:19:16.541987+0330","level":"warn","message":"loading extension files from directory","directory":"/etc/sensu/extensions"}
{"timestamp":"2016-11-12T17:19:16.663607+0330","level":"warn","message":"reconnecting to transport"}
{"timestamp":"2016-11-12T17:19:58.670682+0330","level":"error","message":"[amqp] Detected TCP connection failure"}

Hi again,
I found the issue, and fixed it. It basically boils to the fact that the uchiwa actual conf file being used by uchiwa is located in /etc/sensu/uchiwa.json while all my engaement was per the other json files in /etc/sensu/conf.d/uchiwa.json :)
It clearly is a documentation precission lack and hope it helps other people ...:)


fixed cc.isArray

objectToString 未定义，在jsb下会导致cc.isArray报错

Hi, @akira-cn
Cocos2d-JS will be moved entirely to Cocos2d-x repo, so can you post a PR to Cocos2d-x's repo ?
Related file can be found here : https://github.com/cocos2d/cocos2d-x/blob/v3/cocos/scripting/js-bindings/script/jsb_boot.js

Ported to Cocos2d-x


daser: errors with not found error

Another one

Ok, so the issue is in the header/store. The store has a pending slice of headers to be aggregated and flushed at once. The Store also has a cache of headers for reads. If a header gets requested from the Store, we check the cache first and then the disk. The problem here is that pending slice headers are only Getable from the cache(until they are written), while the cache can evict some of the pending headers due to other headers being more popular, so a very rare situation possible where the header is not in the cache and not in the disk, but only in the private pending slice. I confirmed this behavior by making Get method also read from the pending slice and logging this event out.


Missing mention of webhooks in schema section

Right at the beginning of the page there's a "schema" section and it's missing "webhooks" (which is described later on the page).

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

ID: ee4ec9d0-e0d5-4fb4-7c3e-b84abfa290c2
Version Independent ID: 3e2b80d9-30e5-0c48-49f0-4fcdfedf5eee
Content: Resources - Azure Pipelines
Content Source: docs/pipelines/process/resources.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks @NenoLoje! This will be fixed shortly.


Getting Block-scoped declaration error on run

Trying to run it, but getting this error:
SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode
Any idea?

yeah I guess I have to add use strict on the top of the file, but for now you can upgrade node.js or add it yourself.


ramen_sync_data_bytes metric

ramen_sync_data_bytes
Mostly remains the same as #707

Use-case:

Historical data on bytes transferred per sync
Can be used to figure out overall bandwidth requirements
Can be used to plot hot/cold volumes (heat maps), and leveraged for planning

Code mechanics:

lastSyncBytes

Updated when a VR reports the same causing a reconcile of VRG (and hence DRPC)

Same as lastSyncTime
Not available for VolSync

This is done.


@conda-forge-admin, please add bot automerge

Additional comment:
@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/de-tagesschau-feedstock#2.


Packing Collateral into a single slot

For ERC20Collateral, it may be ideal to make the Collateral struct packable into a single slot. However, this may require reducing the size of the amount variable to make space for timestamp.

Not relevant after https://github.com/OpenZeppelin/openzeppelin-community-contracts/pull/24


Allow Kubernetes version override for k3s

What would you like to be added:
I would like to be able to override the expected API version of the Kubernetes cluster via a global flag such as --k8s-version.
An example usage is kubectl kudo --k8s-version 1.15.4 install kafka --version 1.1.1.
Why is this needed:
I'm not certain of this, but it seems that the discovery of server version uses apimachinery version helpers that may restrict the Semantic Versioning structure to an x.x.x-(alpha|beta)x format.
This causes failures when running against the k3s server version of v1.15.4-k3s.1. The k3s server version is a valid semvar tag, but the error is definitely coming back from client-go.

@jlaswell Thanks for the issue! I'll install k3s and test this out. We're about to update client-go anyway.

@gerred Thanks for taking a look! but I didn't post this issue until after a couple of hours of tinkering with my own operator, and just now realized I should try a community operator as well.
$ kubectl kudo install kafka
instance.kudo.dev/v1alpha1/kafka-ppfvhc created

Seems everything does work fine and the issue is with how I have defined my operator. I'll debug my work from there knowing this works for kafka and others.
Sorry to post a false alarm here! and thanks for the work with the project.


RIS-2832 Removed regex removing comments from query

Removed regex removing comments from query as we want to keep comments in:

stored procedures
values containing double dashes --  (ex. cmd line parameters)

IMPORTANT:
Do not use question mark (?) nor PDO parameter syntax (:word) inside comments as this will cause PDO to fail.

Will this work? I have a feeling that this may break things because PDO can't handle -- comments (but I could be wrong).
Could you test it if you haven't already?
If it doesn't work, we may need to keep removing the -- comments in all cases except within strings, by making the regex more intelligent.

This is already in PRODUCTION proving that it's working just fine.
https://github.com/aptvision/ris/pull/1896/files#diff-4c2be43bffab03013da7a2bf20182fe6c428aed41d1d8bdfdee37d0b5d824ffbR471
Double dashes don't cause problems with PDO.
It's what get's in the comment itself.
So everything that will confuse PDO is not allowed - more info in PR descripion.

This should be fine, but it will break in case we have comments inbetween statements, so if we have something like:
UPDATE.....

-- This is a comment
ALTER TABLE....

It will treat it as start of the statement and i think that won't work, but not 100% sure.

@marcinw0 could you check this and see if it should be merged or closed?

@marcinw0 could you check this and see if it should be merged or closed?

This change has been tested with positive results. We probably could merge this PR.
It's the nature of nested dependencies that prevents us from using that code in ris. We've fallen into dependency hell - libraries simultaneously requiring different versions of the same dependency. But the problem has to be solved on the ris side so I'm merging this PR :wink:

This change has been tested with positive results. We probably could merge this PR.
It's the nature of nested dependencies that prevents us from using that code in ris. We've fallen into dependency hell - libraries simultaneously requiring different versions of the same dependency. But the problem has to be solved on the ris side so I'm merging this PR :wink:

Thanks @marcinw0!

Thanks @marcinw0!


chore: release 0.17.0

:robot: I have created a release *beep* *boop*
0.17.0 (2021-05-28)
Features

ignore search files by config (#88) (a7b1b3a)

This PR was generated with Release Please. See documentation.

Coverage remained the same at 78.307% when pulling c88553d69e444d260ab52c9070b4fe5e25b312a5 on release-v0.17.0 into a7b1b3a98bda9c72859ebc762efa5eb874dbd7e9 on master.

:robot: Release is at https://github.com/easyops-cn/docusaurus-search-local/releases/tag/v0.17.0 :sunflower:


How to initialize RDS connection during application boostrap with setting stored in AWS SSM Parameter Store / Secrets Manager

Type: Feature
Is your feature request related to a problem? Please describe.
I  am trying to setup a RDS connection during application boostrap from AWS SSM parameter store.  For that I was just trying to store the RDS parameters as key in AWS SSM parameter store. The problem is that square bracket are not allowed within keys.
Trying to store something like cloud.aws.rds.instances[0].db-instance-identifier as key in parameter store will end up with the following error:
_Error: error creating SSM parameter (/config/mySpringApp/cloud.aws.rds.instances[0].db-instance-identifier): ValidationException: Parameter name: can't be prefixed with "ssm" (case-insensitive). If formed as a path, it can consist of sub-paths divided by slash symbol; each sub-path can be formed as a mix of letters, numbers and the following 3 symbols .-__

Trying the same in AWS secrets manager is more or less the same:
_Error: only alphanumeric characters and /_+=.@- special characters are allowed in "name"_

Describe the solution you'd like
I would expect that there is way that you can initialize a RDS dynamically during bootstrap. If the square brackets are not allowed there should be a different form to specify those keys which will also respect the limitations of special characters in key for AWS SSM parameter store and AWS secrets manager.
Maybe just introduce a way to store a key such as /config/mySpringApp/cloud.aws.rds.instances[0].db-instance-identifier as /config/mySpringApp/cloud.aws.rds.instances_0_.db-instance-identifier and replace it dynamically during bootstrap into again /config/mySpringApp/cloud.aws.rds.instances[0].db-instance-identifier.
Describe alternatives you've considered
Additional context

You can store password in another key that is referenced like this:
cloud.aws.rds.instances[0].db-instance-identifier=${myRdsPassword}

Does it solve your problem?

Closing due to lack of feedback


Empty interface definitions are not checked properly

Hi,
I found that TypeDoc is falling with error message:
" interfaceReflection.children.forEach(function.....
TypeError: Cannot read property 'forEach' of undefined.... "
It happens when it runs on the definition of such type:
" ....
export interface Entity {
//No members
}
.....
"
So there should be a check if interfaceReflection really has children, something like if(interfaceReflection.children)
Best regards, Alex

Hi Alex, thank you for the report. I've included a fix for the problem.

Hi,
When this fix will be released and published to npm?
Thanks,
Sergey

Hi Sergey, sorry for the long delay, has been a hot summer :wink:
I'll publish v0.3.9 to NPM today.

Thank you very much!

@sgal Please note that version v0.3.9 that I've published earlier today contains a bug when resolving lib.d.ts, I've published v0.3.10 right now containing a fix.

This one should be fixed. Pleas feel free to open a new issue if you encounter any problems.


[FLINK-938] Auomatically configure the jobmanager address

This PR is an alternativ approach to #48 without touching the config file.  Once this is merged, I'll close #48

Mingliang could you rebase the PR on the latest master? In the meantime, the codebase changed quite a bit.

ok. I rebased the PR and modified it with scala implementation.

Great, thanks.

I've rebased this on the current master, added log messages when falling back to the default address (which I've renamed as Till suggested), and tested it.
There is one remaining issue though: When starting the task manager, you use hostname as the default JM address. As @StephanEwen pointed out in another thread [1], this will only work for certain setups. Then the question is whether we really improve deployment with this or not. @rmetzger, you reported the original issue. Do you (and others of course) have an idea about this?
[1] https://github.com/apache/flink/pull/581#issuecomment-94911176
PS: The rebased branch is here, if you have time to pick it up from there: https://github.com/uce/incubator-flink/tree/jmaddress

I've just checked EC2. The default AMIs all return the cloud internal hostname, which is accessible (given the correct security setup). I count this as an indicator that just using hostname is a reasonable approach.
Before merging, I would like to hear another opinion on this though.

This will work for most use cases where the JobManager is started within the same cluster as the TaskManagers. It will certainly fail if the JobManager resides in a different network environment where the hostname cannot be resolved by the TaskManagers.
I think it's ok to merge if we print a big warning when the job manager

OK going to merge and adding the warning. Makes it more comfortable to run a stand-alone cluster in the cloud. :) Thanks for the PR.

I waited to merge this, because of the recent ZooKeeper integration (to see whether this would be affected by it or not). Turns out this change is orthogonal, so I think we can finally merge this now if there are no objections.

Are we sure this helps, or will it add yet another way of configuring and starting the cluster. Are we getting to the point that it becomes hard to understand what happens in which situation, with all the different variants?

Yeah, I was also thinking about this.
This will result in a total of 4 combinations:

HA mode
JM address configured
JM address behaviour

0
0
Use hostname of where the JM is started (new behaviour, this PR)

0
1
Use configured hostname (current behaviour)

1
0
Ignored (current behaviour)

1
1
Ignored (current behaviour)

We would comment out the JM RPC address in the default config and most cluster setups w/o HA would require one less configuration value.
What do you say? I'm slightly leaning towards adding this, but if your gut feeling is that this complicates things I'm also happy to close this PR. :)

Not sure right now.
All setups through YARN are not affected anyways. For standalone mode, do we need it?
How reliable is this? What happens if you start the cluster from a node that has more than one network interface, multiple IP addresses and different hostnames? How will it know which one to pick?
The scenario I am thinking about is clouds with internal- vs. external addresses.

I've tested this with default EC2 and GCE instances, which by default return the cloud internal hostname. I would say that this is a reasonable configuration. If it is not configured this way, the address needs to be configured (which is the current behavior). With a good startup/error message this should be fine.
Still, I start to feel like we are discussing this issue for too long w/o anybody asking for it. Since we have to rebase this on the recent big changes anyways, let's close this PR and reconsider it the issue if someone complains/asks for it. Agree?

How would you give a good error message? It would manifest itself as the fact that the TaskManagers cannot connect to the JobManager.
I would actually agree to not add this for now.

Something like "Using address XYZ. Configure via ABC." at startup and a similar error messasge if connecting fails. You could also try to connect from the remote host before starting the task managers.
But as I've said, let's not add it for now.
@qmlmoon, can you close this PR?


Uncollapsable  Embed

If you include the embed in a bulleted list and collapse it, it still shows up.
How to Replicate:

Create a bulleted list with at least one indentation.
Embed a piece of media (e.g. tweet) in an indented bullet point.
Collapse the bullet point above it.

Sorry, if this is the wrong place but I wanted to add that this also doesn't work with headings too.

It's ok! Here's fine.
Just tested and it works for me though
## Heading 2
![](https://obsidian.md/)


records: exposing record metadata revisions

(addresses #1990)

Extends get_record with optional revision parameter
Exposes revision for record information (metadata) tab
Adds a config variable CFG_RECORD_REVISION_URL_PARAMETER
with default value of "revision"

Signed-off-by: Bogdan Kulynych<EMAIL_ADDRESS>

@tiborsimko Invenio code base have moved quite far since this PR was opened, and given the new rewrite of data model and search I think it will be likely hard to reintegrate this, so would it be ok to close the PR as someday so that we can still look up the code if we need it in the future.

Related with #3129

Another issue is the relation to forthcoming DIP store that will allow to archive more historical formats than just JSON or master ones.  (E.g. slave BibTeX format, see discussion in #1990). Hence some parts of this PR can be recycled now in @egabancho's branch, and wider user-serving parts could be revived once we start with the DIP store.

Implementation should be adapted in new Invenio-Records module (https://github.com/inveniosoftware/invenio-records/issues/10).


V1

Prepare v1 release.
Add support for OAS3.1 (Webhooks and new schema specification) as well as callbacks.

Hello! Thank you for your work! Looking forward to v1 release. Could you also add this wrapping function to apistos/src/internal/actix/route.rs?
use actix_service::{boxed::BoxService, ServiceFactory, Transform};
use actix_web::body::MessageBody;
use actix_web::dev::{ServiceRequest, ServiceResponse};

impl Route {
  // ...

  pub fn wrap<M, B>(self, mw: M) -> Route
  where
    B: MessageBody + 'static,
    M: Transform<
        BoxService<ServiceRequest, ServiceResponse, Error>,
        ServiceRequest,
        Response = ServiceResponse<B>,
        Error = Error,
        InitError = (),
      > + 'static,
  {
    let oas_version = get_oas_version();
    Route {
      oas_version,
      operation: self.operation,
      path_item_type: self.path_item_type,
      components: self.components,
      inner: self.inner.wrap(mw),
    }
  }
}

Hi, just added it.
Regarding v1, once Schemars v1 is release I plan on doing a documentation effort before releasing Apistos with a stable version.

Thanks for your work!
I just integrated the latest prerelease (here), and a quite happy with the utopia replacement, great work ! It was a bit difficult to sometimes find the right documentation, but it's understandable since it's not yet released 😅
2 things:

I might not have found the right way to do this, but I had to dump the use of actix macros to create the routes (not a big deal though)
I had trouble with some part of the API that I did not want to document (some html+icons part), and ended up by mounting them to the actix APP before calling .document(spec) so I could use actix only types and skip documentation. I'm not sure if it was the right way to tackle this though.

Hi,
Thanks for your message !

Apistos currently doesn't support actix macros (might be the case one day but it's currently not a priority) which is the main difference between this project and utopia. Out of curiosity, why did you switch ? I'm always happy to receive some feedback and I'm looking for ways to make this project better.
It is possible to skip some endpoint to keep them undocumented with something like #[api_operation(skip)] . Documentation is not yet sufficient, some effort will be made on this front in the future.


Mark all fields touched

When I submit the form and validation is failing I want all validation errors to be shown.
So it would be great to tell the form to mark every field as touched.

Thanks for raising this Bernard - and the associated PR. It's now the weekend for me (I'm in Sydney, Australia) so I won't do anything with the PR until Monday (so I can check the branch out and test it out). I've had a quick look and it looks good though so shouldn't be a problem to merge
This is definitely a good feature to add so I'm very happy that you've identified the issue and raised a PR for it - I'm really very grateful, thanks! 👍

This feature got broken again. It was fixed in 0.0.22 and got broken in 0.0.26.

Thanks for reporting this @blabno - is it still broken in 0.0.28 ? There have been subsequent changes to validation since that commit you referenced.
I have been trying to ensure we have unit tests to cover all the conditions but it might be that we have a shortfall in the tests.

Yes, it exists in 28 as well.
Sent from ProtonMail mobile
-------- Original Message --------
On 27 Nov 2018, 01:29, Dave Draper wrote:

Thanks for reporting this @blabno - is it still broken in 0.0.28 ? There have been subsequent changes to validation since that commit you referenced.
I have been trying to ensure we have unit tests to cover all the conditions but it might be that we have a shortfall in the tests.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Update aws-java-sdk-dynamodb to 1.12.319

Updates com.amazonaws:aws-java-sdk-dynamodb from 1.11.1034 to 1.12.319.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #348.


Request - On Progress Callback

It would be excellent if there was an optional onProgress callback that would return the current frame being processed.
That way you could give the users an idea of how long the gif will take to make.
const output = await encode({
  width: 200, height: 200,
  frames: [
    { imageData: '/example1.png', delay: 100 },
    { imageData: '/example2.png', delay: 100 }
  ],
  onProgress: (frame) => console.log(frame)
})

In fact, all frames need to be sampled first to generate a palette (the same palette as ffmpeg), then each frame is matched from the palette to index, crop, and encode to gif
So progress callback, I also do not know how to provide ~


set up publishing releases to npm

set each new release to automatically publish to npm

from Slack (@GreatWizard) :

create personnal npm account
publish stuff under this account
and then link the package to the PD organisation
so everyone from the organisation can publish new versions

https://docs.npmjs.com/getting-started/publishing-npm-packages

I suggest we rename layout-linter (at least inside package.json) to html-linter.
The name is important when it comes to users looking for a package on npmjs and I think html-linter is better in that case.
@peopledoc/tribe-js what do you think ?

@xcambar are you ok with renaming the repo and the module to html-linter ?
once this is sorted I will go ahead and publish to npmjs.com

Go ahead 🤘


Add GMM results; Update GMM.py to its final and working version; Add GMM models; Rename MultilayeredPerceptron result files

Add GMM-related methods.


M10 Pattern Military Police Helmet

https://cm-ss13.com/wiki/Marine_Equipment#M10_Pattern_Military_Police_Helmet

was done in #583


Updating top level README

Adding a tad more details to the README, should potential contributors or interested parties land on the repo first.

@robscott Kept this short and sweet, expecting that we update this as we develop. PTAL, thanks!

LGTM, besides the nits pointed out from others.

LGTM


Fixed #32635 -- Fixed system checks crash for invalid model field names in CheckConstraint.check/UniqueConstraint.condition with a reverse o2o relation.

https://code.djangoproject.com/ticket/32635

@sim1234 Thanks for checking :+1:


feat(cache): do not require Redis

Move all the Redis integration to http://npmjs.com/package/@contentacms/redis and http://npmjs.com/package/@contentacms/redisShare

Pull Request Test Coverage Report for Build 44

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

Totals

Change from base Build 42:
0.0%

Covered Lines:
212

Relevant Lines:
212

💛  - Coveralls

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update sys-dm-xe-sessions-transact-sql.md

Adding new columns to  the sys.dm_xe_sessions dmv.
The column names are:-

session_source
buffer_processed_count
buffer_full_count
total_bytes_generated

@DanielAdeniji : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Thanks @DanielAdeniji for contacting Microsoft with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We'll contact you if we need further information, and you'll receive an email message each time we mention you in a comment.
Thanks again for this feedback, your suggestions help improve our documentation.

@DanielAdeniji thanks again for the time and attention to submit this pull request. Your commit has been pulled into an internal PR that is in progress and under review. This is a valid issue, we are hunting down confirmation on some details which will expand the definitions you've provided. Will update you here in this space when new information is published. Your commit will be merged with others. After the updated article is published, GitHub will list you as an article contributor.

Thanks again @DanielAdeniji, just an update that a doc update is in flight and should be published in the next 24 hours.  Thanks again for your time and attention here, your feedback helps improve our docs for everyone.


Integrate autoenv module

Autoenv automatically sources (known/whitelisted) .env and .out files.
https://github.com/horosgrisa/autoenv

Hi @rjcoelho , your PR got merged in the community maintained fork of this repo.
zsh-users is an organization dedicated to host Zsh community projects and therefore became the rational home for a community maintained version of prezto.
Your contribution is greatly appreciated!
https://github.com/zsh-users/prezto

Thanks for your contribution! Unfortunately we're trying to focus on fixing bugs and improving existing modules rather than adding new ones at the moment. It's hard to keep everyone happy, but we're still doing our best to focus on the original goal of providing a sane set of defaults and a framework to build on rather than including every requested module.
I'm closing this for now because I think it would be a better fit for user configuration but thank-you for taking the time to submit this.


Created consistent spacing between tabs and tabbed content.

Hi @skttl
Great package! Only just getting round to trying it out and I spotted there could be an improvement made with a bit of the spacing between tabs and the tabbed content.
First tab with your original code - spacing is correct.

All other tabs with your original code - spacing is incorrect.

I can see you were adding a content push div (.matryoshka-tabbed-content-push) of 20px in height that was accounting for the lack of margin on top of content areas. However, this would then have a knock-on effect on all other tabs, as they would also be pushed down by the margin-bottom set to the previous content area by default in Umbraco 8.
Therefore, I have removed the height set on the content push area and instead added margin-bottom to the tabs list (.matryoshka-tabs-list). Since the tabs list (.matryoshka-tabs-list) will be a sticky item we can still set margin below it and that will create a space between itself and the items below. A sticky item is essentially still a relative item until is reached, it is at that point it becomes sticky.
All other tabs with my new code - spacing is correct.

Hope that all makes sense. Let me know how you get on with testing it but I've tested in a few scenarios in different browsers and have had consistent results so far.
Thanks
Paul

Already fixed by #26

Ah ok, I hadn't spotted #26
Although reviewing #26 it looks like it is still reliant on the empty div "matryoshka-tabbed-content-push" in order to style the positioning of each tab pane. Moving the margin to "matryoshka-tabs-list" would mean that the empty div could be removed entirely and avoids overriding styles applied to "umb-group-panel" from a global Umbraco CMS level.
Thanks
Paul


Fix Posts Datagrid

This PR resolves an issue with the datagrid on the posts management page

Oh, I just realized you are targeting master - did you intend to target the default branch development?


Rename static_intrusive_list and add new intrusive_list

Most lists start as static_* and add non-static implementations.
The intrusive_list is static, but the name does not reflect this.
Rename it to static_intrusive_list and add a new intrusive_list.

Fixed in v3.0.0


example for testing vuex store?

Would it be possible to use this to run unit tests against a vuex store?

@gafotech, Vuex state/getters/mutations/actions are just raw functions, so if you want to test them, you don't need any specific tool. Only one "issue" I've faced – how to test this.$axios inside Vuex actions. There are two ways – easy and complex.
Easy one. Just use .call(...) when trying to test Vuex action. For example, you have an action:
export const actions = {
  async fooAction({ commit }) {
     const data = await this.$axios.get('/bar-api');

     commit ('bazMutation', data);
  }
}

To test this action, you just need to mock fooAction's context, so the test will not fail with "get" of undefined (because in the test environment your this will not contain $axios).
There is an example of how to test this action:
import { actions } from './path-to-actions';

test('fooAction', async () => {
  const commit = jest.fn();
  const mockResponse = { api: 'response' };
  // here we're checking that mutation was called, and called with passed arguments
  const expectedMutations = [
    [
     'bazMutation',
     mockResponse,
    ],
  ];

  await actions.fooAction.call(
    { $axios: { $get: () => Promise.resolve(mockResponse) } },
    { commit },
  );

  expect(commit.mock.calls).toEqual(expectedMutations);
});

And in that way, you can mock any item from the Nuxt's context.
More complex solution. Just get rid of using this.$axios in the store and move all requests to the separate service, that they can be mocked with jest spy like this
const spy = jest.spyOn(productService, 'fooMethod');

beforeEach(() => {
  spy.mockImplementation(() => ({ foo: 'bar' });
});

I'm not sure about all other Nuxt's context items, because I didn't use them in my Vuex. Hope this example with Axios will be useful.

Related #95


default privileges can't be determined error

Hi,
I reached next level by first using ldapsearch and then getting parameters correct. But next I came across this below:
ldap2pg.yml.
Running in dry mode. Postgres will be untouched.
Inspecting roles in Postgres cluster...
Querying LDAP cn=developers-group,ou=d... (objectclass...
Would create ash12.
Would create ashle.
Would create tt.
Would create developers-group.
Inspecting GRANTs in Postgres cluster...
In dry mode, some owners aren't created, their default privileges can't be determined.
Unhandled error:
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/ldap2pg/script.py", line 84, in main
exit(wrapped_main(config))
File "/usr/local/lib/python3.4/dist-packages/ldap2pg/script.py", line 68, in wrapped_main
count = manager.sync(syncmap=config['sync_map'])
File "/usr/local/lib/python3.4/dist-packages/ldap2pg/manager.py", line 213, in sync
pgacl = self.inspector.fetch_grants(schemas, pgmanagedroles)
File "/usr/local/lib/python3.4/dist-packages/ldap2pg/inspector.py", line 254, in fetch_grants
rows = psql(privilege.inspect)
File "/usr/local/lib/python3.4/dist-packages/ldap2pg/psql.py", line 142, in call
self.cursor.execute(query, *args)
psycopg2.ProgrammingError: syntax error at or near "("
LINE 26: FILTER (WHERE pro.proname IS NOT NULL) AS procs
^
Please file an issue at https://github.com/dalibo/ldap2pg/issues with full log.

Here I am giving ldap2pg.yml:
postgres:
  dsn: postgres://****:pass123@localhost:5432/
  databases_query: [postgres]

privileges:
  # Define an privilege group `ro` with read-only grants
  ro:
  - __connect__
  - __execute__
  - __select_on_tables__
  - __select_on_sequences__
  - __usage_on_schemas__
  - __usage_on_types__

sync_map:
- ldap:
    base: cn=*,ou=*,dc=ldap,dc=com
    filter: (objectclass=*)
  role:
    name: '{cn}'
    options: LOGIN SUPERUSER

Any help would be appreciated.

Can you tell the postgres cluster version ?

Ok, i reproduced the bug. I'm working on it.

@testingash the fix is merged in master, and 9.3 is tested on CI. Can you confirm this ? Can you test installing a from master ?

@bersace I downloaded from master.
Starting ldap2pg 4.11.
....
Running in dry mode. Postgres will be untouched.
Inspecting roles in Postgres cluster...
Querying LDAP cn=developers-group,ou=d... (objectclass...
Would create  ash12.
Would create  ashle.
Would create  tt.
Would create developers-group.
Inspecting GRANTs in Postgres cluster...
In dry mode, some owners aren't created, their default privileges can't be determined.
Can't inspect privilege mypriv: query not defined.
Would grant mypriv on postgres.information_schema to admin.
Would grant mypriv on postgres.pg_catalog to admin.
Would grant mypriv on postgres.pg_temp_1 to admin.
Would grant mypriv on postgres.pg_toast to admin.
Would grant mypriv on postgres.pg_toast_temp_1 to admin.
Would grant mypriv on postgres.public to admin.
Comparison complete.
Fix worked.thank you.

@bersace : one thing I would like to ask is Will this tool will be helpful if I want to create same LDAP users in postgres?? This case is for already created users in LDAP server. In my understanding this tool will work as hook to get LDAP users. Please correct me if I am misunderstand tool.

Will this tool will be helpful if I want to create same LDAP users in postgres??

Yes, for sure. It's the purpose of ldap2pg. See http://ldap2pg.readthedocs.io/en/latest/roles/ .
Postgres knows how to check password using LDAP protocol. Postgres does not know how to query LDAP and create roles. ldap2pg allows you to describe how to create roles in Postgres cluster according to a centralized directory, using LDAP. Since a user creation is complete with it's default privileges, ldap2pg helps you to manage privileges too.

@testingash ldap2pg 4.12 is now on PyPI with the fix, so you don't have to use master.

thanks for information. I mean by this "Will this tool will be helpful if I want to create same LDAP users in postgres?? " -- Is I want to create/update table in postgres with those LDAP users.

I want to create/update table in postgres with those LDAP users.

Ok, sorry, I misunderstood. For a role to be allowed to create a table it must have __create_on_schemas__ privilege granted alias CREATE ON SCHEMA as well as __insert_on_tables__ in schema pg_catalog.
privileges:
  rw: [__instert_on_tables__]
  ddl: [__create_on_schemas__]
sync_map:
- grant:
  - privilge: rw
    schema: pg_catalog
    role: creators
  - privilege: ddl
    schema: myschema
    role: creators

YMMV

I have written yml as:
postgres:
dsn: postgres://:@localhost:5432/uos
databases_query: [postgres,**]
schemas_query: |
SELECT nspname FROM pg_catalog.pg_namespace WHERE nspname = '****'
privileges:
ro:

connect
execute
select_on_tables
select_on_sequences
usage_on_schemas
usage_on_types

rw privilege group lists write-only grants
rw:

all_on_tables
all_on_sequences
insert_on_tables

ddl privilege group lists DDL only grants.
ddl:

create_on_schemas

sync_map:

roles:

names:

admin1
readers
options: NOLOGIN

name: writers
Grant reading to writers
parent: readers
options: NOLOGIN
name: owners
Grant read/write to owners
parent: writers
options: NOLOGIN
name: creators
options: NOLOGIN

Now grant privileges to each groups
grant:

privilege: ro
role: readers
Let's everyone see pg_catalog
schema: all

Grants on specific schema appdb.appns:

privilege: rw
role: writers
database:***
schema: *****
privilege: ddl
role: owners
database: uos
schema: dpatch
privilege: ddl
database: ****
schema: ****
role: creators

ldap:
base: cn=developers-group,ou=developers,dc=ldap,dc=com
filter: (objectclass=*)
role:
name: '{cn}'
options: LOGIN SUPERUSER
parent:

admin1
owners

maybe my understanding can also wrong.I have table in schema which I mentioned in this yml. I want LDAP users say ash12,ash22,tt to be updated in that table .

I want LDAP users say ash12,ash22,tt to be updated in that table .

I dont understand this. Which table are you talking about ?

I am referring to table from postgresql tables which schema was used in
yml.  Please correct me if I am wrong.
On Fri, 3 Aug 2018 19:32 Étienne BERSAC<EMAIL_ADDRESS>wrote:

I want LDAP users say ash12,ash22,tt to be updated in that table .
I dont understand this. What table are you talking about ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dalibo/ldap2pg/issues/230#issuecomment-410258124, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATLO4JUmCMGa35AMuaDc8v8nZ-edRiCYks5uNFfdgaJpZM4Vr_1q
.

@testingash I don't understand what is your goal.
ldap2pg is not meant to insert neither update data in tables. ldap2pg create, alter or drop roles, and grant or revoke privileges.

ldap2pg is not meant to insert neither update data in tables. ldap2pg create, alter or drop roles, and grant or revoke privileges --- So LDAP users will be ported as roles in PG right?
I don't understand what is your goal -- I am trying to understand concept of case where customers have implemented LDAPor RADIUS we fetch all needed user details from their servers instead of creating 10,000 users again in Postgres.

@testingash In Postgres, you must create roles to grant access and privileges. Postgres server only uses LDAP to check password. Nothing more. See the first line of LDAP authentication documentation of Postgres  : https://www.postgresql.org/docs/current/static/auth-methods.html#AUTH-LDAP
ldap2pg is a tool to create/alter/drop Postgres roles according to a LDAP directory. It's a synchronisation tool to keep Postgres up to date with directory.
You have to carefully configure either cron tasks or triggers so that de delay between directory update and Postgres synchronisation with ldap2pg is low enough for your use case.
Is it clearer for you ?


IDEA: extend random string generator with UUID generator

UUID's are random strings with a defined format used in programming for different purposes e.g. object ID's or session ID's etc.
They look like HEX numbers with hyphens e.g. "550e8400-e29b-41d4-a716-446655440000".
There are some variants and these are relative easy to implement.
Details see: https://en.wikipedia.org/wiki/Universally_unique_identifier
It might be a logical extension for the nppRandomStringGenerator, if not just close the issue.

@RobTillaart will look into that, shouldn't be much work to add it.


With v7.0.8 "Clarification" individuals without a surname or that use location in their name presents issue!

With the new "clarification" the Personal Name it would seam that individuals that do not have surnames and may also have location type names will not have the use of PERSONAL_NAME_PIECES based on the new definition:

Clarify that one implication of "The Personal Name payload shall be seen as the primary name representation, with name pieces as optional auxiliary information" is that "all name parts in PERSONAL_NAME_PIECES should appear within the  payload."

Since normally I would enter in a GEDCOM the following for a person without a surename and a location name"
0 INDI
1 NAME Olaf Olsen Bruflot
1 GIVN Olaf
Without a SURN since they did not have a surname.  But I would need to enter their entire name from birth records   <location/farm name> in the the NAME tag,
His sister would be:
0 INDI
1 NAME Maria Olsdattr Bruflot
1 GIVN Maria
The problem is that without a a GIVN tag, reports would be forced to only use the NAME tag on all displays and reports where a given name was normally displayed.   This would be compounded when an individual had multiple given names producing a very long name display: Thorvald Andreas Olsson Bruflot comes to mind!

Resolved in #161


Swapping sound effects with a custom sound mutes sound effect in question

I'm trying to custom swap a soundwith a short wav file I made just to try the plugin out. I've identified the sound effect I want to swap in game with the "debug sound effects" feature, typed the sound effect ID into the Custom Sounds field, pasted the wav file with the same sound ID number as the sound I'm wanting to swap into the SoundSwapper folder in my .runelite folder, and checked the box called "Swap Sound Effects". This results in no sound playing when the sound effect should happen. I've listened back to the wav file alone to make sure it has audible sound in it (which it does), tried using a different wav file, tried closing and reopening runelite, and I've tried using different sound IDs and matching the wav file name to the ID and preforming the action in game that should produce that sound. None of these troubleshoots worked and will still result in silence. Once the box called "Swap Sound Effects" has been un-checked, the vanilla in-game sound is able to play again but I'd like this plugin to work for me and others who might have the same problem.

Hey! Sorry to hear that you're having issues.
It sounds like you have everything set up correctly. I'm testing it right now with Runelite up to date with the latest and not having issues on my end. This is on a Windows PC. Are you possibly on Mac/Linux? I haven't personally tested on other platforms.
As for how I'm testing - I put some .wav files in the SoundSwapper folder in my .runelite folder like you said

I chose 62 and 4025 since these are easy to test (close/open door and interface sound [e.g. click on different tabs in your bank]). Plugin settings look like the following:

Is your Blacklist empty as well? Typically, if you aren't getting any sound but the in-game sound is getting consumed, it means that you either have the sound blacklisted or the plugin is failing to find/load your sound file. I've also seen people accidentally give their file a double extension - e.g. they named the file 62.wav when Windows already knows it's a .wav file, so the file ends up named 62.wav.wav under the hood.

Hi thanks for the speedy reply! I am on a PC. My blacklist is empty on the plugin. I've tried naming the wav file with just the sound ID number, and also tried naming it "number".wav. When i add the ".wav" extension in the file name it definitely doesn't work because it just plays the vanilla in-game sound, but when I leave ".wav" out of the file name and simply name it the ID number of the sound effect, this is when the sound is muted/doesn't play entirely.
I looked into my .runelite folder and found a folder called "logs". I scrolled all the way down to the bottom of the most recent file called "client" and found a line of text saying "2024-05-15 23:43:01 EDT [Client] WARN  com.soundswapper.SoundSwapperPlugin - Failed to play custom sound"
I asked my friend to troubleshoot with me and he has had the same problem as I've experienced with the plugin, and he was actually the one who pointed out the log folder. We also noticed that with the Debug Sound Effects option turned on, we can see that when attempting to trigger the sound which we're trying to swap the debug panel reads the sound effect as "Custom" in a light pink colour

For clarity the sound ID im using to test is 2244, which is to equip a weapon to wear, so I'm naming my wave file "2244" and typing 2244 into the Custom Sounds field).

Could you send me the sound file you're testing with by chance? I can share one that is working from my setup as well. Github only allows you to send images in these comments so I'd recommend hoping in the Discord (https://discord.gg/F2mfSvcnaj) and we can help you debug further there. Check out #support channel.
Also another thought - where did you get the .wav file from? If you converted it from .mp3 or some other source, did you use an online converter?

Thanks, I'll follow up on discord in just a bit!

Narrowed down to an issue with the file - vodicy was able to get it working after tweaking settings in their audio program!
Only noteable difference I see is the bitrate of the files. Will update this thread if I find anything, but closing for now.


Investigate scenarios where external RPC port would be needed

If scenarios are found then keep implementation, otherwise remove this section from values.yaml.

It seems this would not be needed in any known scenario for now. We can re-open if needed in the future.


Split EKS commands and outputs for better copy and paste

Motivation
When starting an EKS cluster, I was copying the commands from the docs but the output was copied along with the commands.
Changes

Split the command and output into separate code blocks

Great that you are changing the docs right when coming across issues! For some of the changes in the PR it might make sense to use the new feature @webdev51 implemented: #804 (comment) Wit that you can disable the copying for certain sections within the command, instead of separating command and output all the time (which sometimes makes sense, but sometimes disrupts the flow).

Thanks @alexrashed for the suggestion. I definitely prefer the output as rendered, however the copied command contains a newline at the end. An alternative is to do something like
{{< command >}}
$ pwd<disable-copy>
/
</disable-copy>
{{< / command >}}

which is a little ugly, but in general I prefer to not break up the reading flow as you say.

Ah, interesting. A newline at the end really doesn't seem to be too nice when copy-pasting the commands.
Maybe we could fix the command shortcode to suppress the last newline?
Should be fixed somewhere around this line:
https://github.com/localstack/docs/blob/main/layouts/shortcodes/command.html#L6

Ah, interesting. A newline at the end really doesn't seem to be too nice when copy-pasting the commands. Maybe we could fix the command shortcode to suppress the last newline?

I have implemented this by trimming the whitespace from the copied text before sending to the clipboard.


ip camera

how to display image through camera ip HAP-nodejs?

Not possible until someone adds the new HomeKit Camera type with its relevant associated coding to HAP-NodeJS.
You'll have to wait until someone has the time to do that.
Probably have to wait until iOS 10 is released before anyone with the knowledge takes an interest 😊

Well, that's only a couple days away now.

I would be interested to see this too

Watch the IP-cam branch...  https://github.com/KhaosT/HAP-NodeJS/tree/ip-cam

@PatchworkBoy
Have you been able to add an ip camera to HomeKit using the branch above?
Any idea how I can add my camera to config.json and what parameters I would need to define?
Thanks

Nope - pointless til kT says s'all there n' ready for use.

Alright, the camera support is in v0.4.0
I'll try to document things a little bit later but the general idea is for developer to copy Camera.js to their own project and modify the Camera.js to handle snapshot request and streaming request.

@KhaosT
Thanks for the update.
Can you please elaborate on the modifications which need to be done, etc.?
Also, do the ip camera streams have to be rtp encoded from the ip camera side, or can a developer configure a http/https video stream to work?
Some documentation would be very useful. Or at least some examples

@leoneleone you can find document at here. HomeKit on iOS side requires RTP stream but technically with the help of ffmpeg, HLS should also be possible.


Cleanup streams on server shutdown

It seems like this leads to some weird state when the server shuts down mid HLS stream and then gets restarted. Not sure if it is an HLS.js error or if there is some weird state lingering around (I think it's the latter)

This won't happen proactively, but the previous stream will be cleaned up upon server restart. Might be good enough?


Performance Improvement

Having tried to generate a thumbnail of an MP4 video, I noticed that running ffmpeg took a lot of time to retrieve a thumbnail from a given time in the video. Upon inquiring, I found this comment interesting (https://superuser.com/a/821680)

Each invocation of above ffmpeg command takes a fraction of second (!) for ~1GB H.264. That is because it instantly jumps to  position (mind -ss before -i) and takes first key frame which is practically complete JPEG. There is no time wasted for rendering the movie to match exact time position.

Currently in the creation of arguments for ffmpeg the argument -ss is placed after the definition of the source file.
https://github.com/ScottyFillups/simple-thumbnail/blob/44379e88307c1aa2c8efee518dd2c1d4861eb566/index.js#L60-L68
I have therefore rearranged the order of the arguments to put the -ss before the -i as suggested in the comment above.
return [
  '-y',
  `-ss ${seek}`,
  `-i ${input}`,
  scaleArg,
  '-vframes 1',
  output
]

For the following test script with an MP4 file of 23m40s and 188Mb I give you the total execution time of this same script with the original order and the one I propose.
The script:
const ffmpeg = require('ffmpeg-static')
const genThumbnail = require('simple-thumbnail')

async function download () {
  await genThumbnail('test.mp4', 'test_s.jpg', '150x?', {
    path: ffmpeg,
	seek: '00:12:00'
  })
  console.log('Done!')
}

download()

The characteristics of the MP4 file (the return of ffprobe):
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'test.mp4':
  Metadata:
    major_brand     : isom
    minor_version   : 512
    compatible_brands: isomiso2avc1mp41
    encoder         : Lavf58.33.100
  Duration: 00:23:40.16, start: 0.000000, bitrate: 1113 kb/s
    Stream #0:0(und): Video: h264 (Main) (avc1 / 0x31637661), yuv420p, 1280x720 [SAR 1:1 DAR 16:9], 915 kb/s, 23.98 fps, 23.98 tbr, 24k tbn, 47.95 tbc (default)
    Metadata:
      handler_name    : VideoHandler
    Stream #0:1(und): Audio: aac (LC) (mp4a / 0x6134706D), 48000 Hz, stereo, fltp, 192 kb/s (default)
    Metadata:
      handler_name    : SoundHandler

The execution time of the script with the original order:
PS C:\Users\marti\Documents\test> Measure-Command {node .\index.js}

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 18
Milliseconds      : 188
Ticks             : 181882832
TotalDays         : 0,000210512537037037
TotalHours        : 0,00505230088888889
TotalMinutes      : 0,303138053333333
TotalSeconds      : 18,1882832
TotalMilliseconds : 18188,2832

The execution time of the script with the proposed order:
PS C:\Users\marti\Documents\test> Measure-Command {node .\index.js}

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 1
Milliseconds      : 480
Ticks             : 14806789
TotalDays         : 1,71374872685185E-05
TotalHours        : 0,000411299694444444
TotalMinutes      : 0,0246779816666667
TotalSeconds      : 1,4806789
TotalMilliseconds : 1480,6789

The time saving is huge for me, and I see the same time saving if I provide the MP4 file via a URL and not as a local file.
The execution time of the script with the file over the internet with the original order:
PS C:\Users\marti\Documents\test> Measure-Command {node .\index.js}

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 26
Milliseconds      : 18
Ticks             : 260180286
TotalDays         : 0,000301134590277778
TotalHours        : 0,00722723016666667
TotalMinutes      : 0,43363381
TotalSeconds      : 26,0180286
TotalMilliseconds : 26018,0286

The execution time of the script with the file over the internet with the proposed order:
PS C:\Users\marti\Documents\test> Measure-Command {node .\index.js}

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 2
Milliseconds      : 265
Ticks             : 22658406
TotalDays         : 2,62250069444444E-05
TotalHours        : 0,000629400166666667
TotalMinutes      : 0,03776401
TotalSeconds      : 2,2658406
TotalMilliseconds : 2265,8406

My tests were performed on a Windows 10 Pro 64bits with an Intel Core i7-10510U.
The order of the parameters definitely seems to have an importance on the performance of ffmpeg, unfortunately I don't have time to do more tests but I'll let you try on your side too.
This is my first issue and I hope I have respected the rules as much as possible 😢
Have a nice day !

This is honestly the most well-structured issue I have ever seen.

is this going to get fixed in the next release?


Changing address prefix doesn't work

    await this.$store.dispatch("common/env/init", {
      prefix: "foo",
    });

Still shows cosmos in UI.

a) it's addrPrefix
b) If you're using the starport module (default) , it reconfigures itself for nodes from the :12345/status endpoint and also picks up prefix from the env variables set by starport: VUE_APP_ADDRESS_PREFIX

Yes, my mistake, used addrPrefix, still can't see the change.

Does starport set VUE_APP_ADDRESS_PREFIX properly? can you comment out starport(store) from config.js and try again?

ah....spotted it, fixed in: https://github.com/tendermint/vue/commit/4c6ffd37c3b84b0b7a57299832496f2c56961089

@clockworkgr on @starport/vue 0.1.32 the prefix now doesn't get added at all. Can't seem to be able to use the app. 0.1.31 looks fine.

Fixed in https://github.com/tendermint/vue/tree/v0.1.37


Windows 2016/19 support

Does anyone know if Xsemmel run on Windows Server 2016 or 2019?

Tried it with Win Server 2016 and it works. Don't see any reason why it shouldn't work with 2019 as well.


"Incorrect table name" after publishing the vendor

I followed these steps:

Install through composer
Publish config and migrations
Migrate the database

When I try to migrate the database I get the following error:
Migrating: 2019_02_10_125533_create_friendships_table

Illuminate\Database\QueryException  : SQLSTATE[42000]: Syntax error or access violation: 1103 Incorrect table name '' (SQL: create table `` (`id` int unsigned not null auto_increment primary key, `sender_type` varchar(191) not null, `sender_id` bigint unsign
ed not null, `recipient_type` varchar(191) not null, `recipient_id` bigint unsigned not null, `status` tinyint not null default '0', `created_at` timestamp null, `updated_at` timestamp null) default character set utf8mb4 collate 'utf8mb4_unicode_ci')

Maybe I did something wrong by not following this step:
Configure the published config in
config\friendships.php

But I don't know what is meant with configure the published config? What do I have to config here? I didn't create the migrations myself like this person did here: https://github.com/hootlex/laravel-friendships/issues/94

If you have published the config files using :
php artisan vendor:publish --provider="Hootlex\Friendships\FriendshipsServiceProvider"
Go to config/friendships.php and see if it matches this -
<?php
return [
    'tables' => [
        'fr_pivot' => 'friendships',
        'fr_groups_pivot' => 'user_friendship_groups'
    ],
    'groups' => [
        'acquaintances' => 0,
        'close_friends' => 1,
        'family' => 2
    ]
];

If its the same then clear your config & compiled files by running php artisan config:clear and php artisan clear-compiled in your console and then run your migration again.
If it still doesn't work, go to bootrap/cache/ folder and delete all files inside that folder, restart the server and try again.
-TT

Running config:clear and clear-compiled worked. Thanks for you fast reply.


Adds a simple earcon to indicated boundaries of expressions.

Adds a very simple earcon. See https://github.com/zorkow/speech-rule-engine/issues/30speech-rule-engine

Awesome, will test. (But ogg only supported in Chrome and Firefox.)

This should now also support non-ogg browsers. PTAL.

I did primarily a browser test (since AT isn't really needed to check this but I did a few combinations as well, one for each AT). Works as expected in all browsers we support.


VK.com in latest version not working properly

I am trying to scrobble track on my music page  https://vk.com/audios3735913 , and only one track is scrobbling now. After I reload the page is other one track is scrobbling.

Version used: 2.2.0
Browser name and version: 56.0.2 32 bit on Windows 7 SP1 64bit.

P.S. I am not sure, probably it is not an issue with extension.

It's working here for me on 2.2.0.

See also https://github.com/david-sabata/web-scrobbler/issues/1455

I reproduced this one on FF57. Closing this in favor of #1455.


NBSNEBIUS-86: not saying that replication is in progress on volume monpage if volume config contains only garbage FreshDeviceIds

There used to be a bug (fixed a couple months ago) because of which some garbage device ids were added to the configs of some volumes. Even though the bug is fixed, some old volumes still contain garbage in their FreshDeviceIds lists and show "ReplicationStatus:in progress" on their monpages. This PR fixes the monpages for those volumes.

упало 2 флапающих теста: один - тула check_emptiness, другие 2 - тесты DM

Флапнули 3 теста:
TEST NAME	ELAPSED	STATUS
cloud/blockstore/libs/storage/disk_registry/ut_pools/[1/15] chunk chunk	0.000s	FAIL
cloud/blockstore/libs/storage/disk_registry/ut_pools/[10/15] chunk chunk	0.000s	FAIL
cloud/blockstore/libs/storage/disk_registry/ut_pools/[7/15] chunk chunk    0.000s      FAIL


React-native ios subscription issue

"react-native-iap": "^6.0.1",
"react-native": "~0.63.4",
In the ios when I click on the purchase button the first time then subscription dialog not open whenever I'm click on second time it's work fine
It's not proper work
First time, not open subscription dialog
i'm check in real device
Please follow above mention step

Yeah, with me too but is not a bug is simply because the subscription is already made and the time for some reason is not reduced. Probably just canceling go to config -> appstore -> sandbox or something like this and try to find something that cancel and say here if it worked

Same with my project. In the settings, mentioned by Ulisses1478, there is shown that the subscription is expired. But it's the same behaviour as described by the originator. This is the error, which is thrown: [Error: Product already purchased, if not returned in the active products it may be owned by a different user (restore needed)] . I am using<EMAIL_ADDRESS>which is building up on this library.


Ownbit wants to be listed

Hi Menero team,
Ownbit wants to be listed as one of 3rd wallets that support XMR. As one of the mobile wallets that support XMR, Ownbit implemented BulletproofV2 in JS by its own.
Website: https://ownbit.io
Thanks for the help.

Please follow the instructions at the top of the merchants page to add a new merchant: https://web.getmonero.org/community/merchants/
This repository is not active yet.

Opened on gitlab: https://repo.getmonero.org/monero-project/monero-site/-/issues/1079


Is libSQL a library or a RDBMS?

SQLite is a RDBMS.
To me, the name libSQL suggests a lib (library). Is libSQL a library or a RDBMS?
IMHO, I think this should be mentioned in the msin description or in a FAQ.
To me, a name create certain expectations. SQLite suggests something lite. If you saw the name libSQL for the first time, what would think of?
Thank you.

I had same question why picking a library name until I see https://github.com/libsql/libsql/issues/25.

libSQL is a fork for of SQLite, and their main page (https://sqlite.org) starts with SQLite is a C-language library that implements (...), so I dare say SQLite is a library, and so is libSQL :) Both are designed specifically to be used as an integral part of other applications, not separate database systems.

Siding with Sarna, we see SQLite (and libSQL) as an embedded library that you can add to your application, not an external database. An example of RDBMS built with it is rqlite.
I am closing this, but mostly because this is a better fit for Discussions, not an issue.
Still, hoping that the explanation is satisfactory


Expose CommandArea through property

It would be really great if you could make a release with a public property ThemedTableLayoutPanel CommandArea { get { return this.commandArea; } }
This way it would be easy to add more buttons to the command area, cuz now at classic win7 theme or windows xp classic theme added buttons are not at the same height as cancel or next buttons. Due to the fact that commandArea is docked at bottom and Cancel, as well as Next buttons have bottom anchor and my added buttons do not have this anchor to commandArea.
Originally posted: 2013-01-25T13:39:37

This could be done with Header as well (giving full control to developers of WizardControl in designer view).
Originally posted: 2013-01-29T02:18:27

As of version 2.0.6, WizardControl has an AddCommandControl method that allows for controls to be added next to the buttons in the Command Area.
Originally posted: 2014-12-12T14:28:25


Add the Campplus model and configuration file

Hi ,
Recently, the cam++ model released by Alibaba has achieved advanced results in speaker recognition tasks. I would like to merge the cam++ model into the wespeaker framework, but I'm not sure if it's compliant with the standards.  About the cam++ model

Firstly, thank you for your attempt of using cam++ model in wespeaker.
Actually, we have noticed the PR of Alibaba and reproduced the results already. Several modifications need to be done to meet the torch.jit.script requirements. Also, we would like to reuse some modules already provided by wespeaker for a unified code style.
We will update the codes in several days.

I understand, and I am looking forward to the updates.


Add explicit names for outputs of some nodes

Adding explicit names for outputs of some nodes for which there may be ambiguity

Thank you!


2.18.4 / 2.19.0 - Swagger generated Python api requires all fields for POST /endpoints

Before you start please confirm the following.

[X] Yes, I've searched similar issues on GitHub.
[X] Yes, I've checked whether this issue is covered in the Portainer documentation or knowledge base.

Problem Description
When using the Python API generated from the Swagger spec, the EndpointsApi.endpoint_create which maps to POST /endpoints api call requires that all parameters are set, even though the Swagger Docs mention that only Name,  EndpointCreationType, and EdgeTunnelServerAddress is required. Which by the way, is the EdgeTunnelServerAddress field being required as intended?
Expected Behavior
The api call succeeds
Actual Behavior
An exception is thrown
Traceback (most recent call last):
  File "main.py", line 16, in <module>
    endpoints_api.endpoint_create(
TypeError: endpoint_create() missing 15 required positional arguments: 'url', 'public_url', 'group_id', 'tls', 'tls_skip_verify', 'tls_skip_client_verify', 'tlsca_cert_file', 'tls_cert_file', 'tls_key_file', 'azure_application_id', 'azure_tenant_id', 'azure_authentication_key', 'tag_ids', 'edge_checkin_interval', and 'gpus'

Steps to Reproduce

Generate the Python api from the Swagger documentation
Install the generated python api with pip
Run the sample code below setting the PORTAINER_URL and API_KEY values
Notice the exception

Command used to create the Python API
wget https://repo1.maven.org/maven2/io/swagger/codegen/v3/swagger-codegen-cli/3.0.43/swagger-codegen-cli-3.0.43.jar -O swagger-codegen-cli.jar
java -jar swagger-codegen-cli.jar generate -i https://api.swaggerhub.com/apis/portainer/portainer-ce/2.18.4?resolved=true -l python -DpackageName=portainer_api -o portainer_api

Sample Code:
from portainer_api.api_client import ApiClient
from portainer_api.configuration import Configuration
from portainer_api.api.endpoints_api import EndpointsApi

PORTAINER_URL: str = ""
API_KEY: str = ""

client_config = Configuration()
client_config.host = f"{PORTAINER_URL}/api"
client_config.verify_ssl = False
client_config.api_key['x-api-key'] = API_KEY

api_client = ApiClient(client_config)
endpoints_api = EndpointsApi(api_client)

endpoints_api.endpoint_create(
    name="TestEnvironment",
    endpoint_creation_type=1, # Local Environment
    edge_tunnel_server_address=""
)

Portainer logs or screenshots
Traceback (most recent call last):
  File "main.py", line 16, in <module>
    endpoints_api.endpoint_create(
TypeError: endpoint_create() missing 15 required positional arguments: 'url', 'public_url', 'group_id', 'tls', 'tls_skip_verify', 'tls_skip_client_verify', 'tlsca_cert_file', 'tls_cert_file', 'tls_key_file', 'azure_application_id', 'azure_tenant_id', 'azure_authentication_key', 'tag_ids', 'edge_checkin_interval', and 'gpus'

Portainer version
2.18.4
Portainer Edition
Community Edition (CE)
Platform and Version
Docker 24.0.4, docker-compose 2.7.0
OS and Architecture
Ubuntu 20.04, AMD64
Browser
No response
What command did you use to deploy Portainer?
docker-compose up

with the following docker-compose.yml

version: "3"
services:
    portainer:
        image: portainer/portainer-ce:2.18.4
        container_name: portainer
        ports:
          - 8000:8000
          - 9443:9443
        volumes:
          - /var/run/docker.sock:/var/run/docker.sock
          - portainer_data:/data
        network_mode: host

    portainer-agent:
        image: portainer/agent:2.18.4
        container_name: portainer_agent
        ports:
          - 9001:9001
        volumes:
          - /var/run/docker.sock:/var/run/docker.sock
          - portainer_data:/data
        network_mode: host

volumes:
    portainer_data:

Additional Information
No response

This issue persists with the 2.19.0 update

I've had some feedback from our engineering team, and they've pointed out that you're using<EMAIL_ADDRESS>which is intended for Swagger/OpenAPI 3.x only. Our API docs are published using Swagger/OpenAPI 2.0, which is not directly supported (I suspect the spec conversion isn't working for our API docs) by the version you're using, as per their repo:

2.X and 3.X version lines of Swagger Codegen are available; 2.X (master branch) supports Swagger/OpenAPI version 2, while 3.X (3.0.0 branch) supports OpenAPI version 3 (and version 2 via spec conversion to version 3). Online generator of version 3.X supports both generation from Swagger/OpenAPI version 2 (by using engine + generators of 2.X) and version 3 specifications.

If you try with a 2.x version (for example 2.4.41) you should se a successful build:
wget https://repo1.maven.org/maven2/io/swagger/swagger-codegen-cli/2.4.41/swagger-codegen-cli-2.4.41.jar -O swagger-codegen-cli.jar
java -jar swagger-codegen-cli.jar generate -i https://api.swaggerhub.com/apis/portainer/portainer-ce/2.19.5?resolved=true -l python -DpackageName=portainer_api -o portainer_api

I am getting the same error with the swagger codegen 2.4.41. Arguments marked optional in the documentation are mandatory in the python generated code

It appears there may be a bug with API key authentication in 2.20 that our team are now looking into, so this could be part of the issue. I've been working with our engineering team and they've provided the following code snippet that should work using JWT authentication instead of API key auth as a workaround (it'll do the JWT generation itself, it just needs your credentials instead of a key) - are you able to give this a test and see if it works as you'd expect?
from portainer_api.api_client import ApiClient
from portainer_api.configuration import Configuration
from portainer_api.api.endpoints_api import EndpointsApi
from portainer_api.api.auth_api import AuthApi

PORTAINER_URL: str = "http://localhost:9000"

client_config = Configuration()
client_config.host = f"{PORTAINER_URL}/api"
client_config.verify_ssl = False

auth_json = {"username":"admin", "password":"mypasswordgoeshere"}

api_client = ApiClient(client_config)
auth_api = AuthApi(api_client)
auth_resp = auth_api.authenticate_user(
    body=auth_json
)

client_config.api_key['Authorization'] = f"Bearer {auth_resp.jwt}"
api_client = ApiClient(client_config)
endpoints_api = EndpointsApi(api_client)

endpoints_api.endpoint_create(
    name="TestEnvironment1",
    endpoint_creation_type=1, # Local Environment
    edge_tunnel_server_address=""
)

My appologies, I seem to have made a mistake when running the swagger generation with swagger codegen 2.4.41. I have re-run it making sure I have a clean run and it does fix the issue

Great, thanks for following up. I'll close this issue (and the other one).


No expo-signature header specifield

my clone server: https://github.com/Chuazz/self-host-expo-clone
my app: https://github.com/Chuazz/phone-book-for-old-man

Nice work!!

Nice work!!

i have research with many site but not found solution for this error

Sorry, was In a hurry and missed the error altogether.
so, in you app.json you should point the URL to an instance of the server, localhost will point to the phone itself.
focodeSigningCertificate to use a relative path to the app root, like ./code-signing/certificate.pem, where code-signing is a folder in the root of the mobile app (see example)

i have config adb reverse in script and it work, i working on ubutu when i build it location on tmp folder so it dont understand relative path

		"codeSigningCertificate": "../../../../../../home/chuazz/Workspace/freetime/phone-book-for-old-man/code-signing/certificate.pem",

place code-signing/certificate.pem inside the root of your app project and write  "./code-signing/certificate.pem" instead.
You may also try first the example, it should work out of the box

i have do it before like this image and it dont understand relative on ubuntu

i have to point it with absolute path

the path is already relative, as it starts with ../../../ ...
seems that you are working with an ejected project, in that case the certificate and metadata needs to be embedded tint the AndroidManifest.xml, look at here for example
https://github.com/umbertoghio/self-hosted-expo-updates-server/blob/master/ExampleEjected/android/app/src/main/AndroidManifest.xml

i pretty sure i on an managed project, i do  ../../../... because when on build it create a build folder on /temp/ and it dont understand where ./code-signing in temp builder folder, i have point it back to /home/user/..../code-sigining again

i think i will try on window

i don't understand but because i ignore *.pem in gitignore then eas build don't copy it into buid folder @@@, thank for your help

if you are in managed you must include the pem file, otherwise the build won't copy it

i get another error :____, i will create another issue help me if you can :3


ci: update slack webhooks

Update Slack webhooks.
We're using the nonprod team alert channel for all library CI alerts to avoid unnecessary noise in the prod alert channel. Trying to keep only alert directly related to prod that requires an immediate attention in the prod alert channel.
:tickets: TICKET-000

Upstream PRs

Downstream PRs

Changes

Notes for Reviewers

Recordings/Screenshots

coverage: 100.0%. remained the same
when pulling cb768f88d2c8677e55b0d72dd9d251ebba016df5 on PLAT-8525-update-slack-weebhooks
into 8a553b9abc372bb2a1791d71394aaab2f4dd3ad1 on main.


refreshing view?

is it possible to refresh the view without reopening git-log window?

^ This would be sooo much helpful! Btw. aworkaround can be to open git log once more (it stays in the same tab, but reloads the data), you can make a keybinding to do so:
in keymap.cson add:
'.git-log':
  'f5': 'git-log:show'

then hit f5 and enter....


bug: v0.0.7 Provider produced inconsistent result after apply: .encryption_assumed_role_identifier was null

when i create a new clickhouse cloud service with version 0.0.7 of the provider i get the following terraform error
Error: Provider produced inconsistent result after apply
When applying changes to clickhouse_service.hubble_timescape_sandbox, provider "provider[\"registry.terraform.io/clickhouse/clickhouse\"]" produced an unexpected new value: .encryption_assumed_role_identifier: was null, but now cty.StringVal("").

This is a bug in the provider, which should be reported in the provider's own issue tracker.

revert to version v0.0.6 fixes the issue and the instance gets created without errors
Terraform
version v1.6.4

can you ask the user to "please share your terraform TF file"

Hi, did you make changes to encryption_assumed_role_identifier

Good afternoon,
I'm also getting this issue.
`
Error: Provider produced inconsistent result after apply
When applying changes to clickhouse_service, provider "registry.terraform.io/clickhouse/clickhouse" produced an unexpected new value: .encryption_key: was null, but now cty.StringVal("").
This is a bug in the provider, which should be reported in the provider's own issue tracker.
Error: Provider produced inconsistent result after apply
When applying changes to clickhouse_service, provider "registry.terraform.io/clickhouse/clickhouse" produced an unexpected new value: .encryption_assumed_role_identifier: was null, but now cty.StringVal("").
This is a bug in the provider, which should be reported in the provider's own issue tracker.
`
This occurs when settings encryption_key and encryption_assumed_role_identifier" to nullor not including the attributes, if CMEK is not required. The returned state value is an empty string"", but the input value is null`, causing the discrepancy.
From a quick glance, it may be due to lines 931 and 932 in service.go:
	 state.EncryptionKey = types.StringValue(service.EncryptionKey) state.EncryptionAssumedRoleIdentifier = types.StringValue(service.EncryptionAssumedRoleIdentifier)
As the state is cast into a String, regardless of whether the value is null or not.

i indeed did not include a encryption key

Thank you. I will provide a fix for this by early next week.

I am also running into this bug:
`Error:Providerproducedinconsistentresultafterapply
Whenapplyingchangesto
module.clickhouse_service_prod.clickhouse_service.this,provider
"provider["registry.terraform.io/clickhouse/clickhouse"]"producedan
unexpectednewvalue:.idle_timeout_minutes:wascty.NumberIntVal(0),but
nowcty.NumberIntVal(15).
Thisisabugintheprovider,whichshouldbereportedintheprovider's
ownissuetracker.`
I am setting idle_timeout_minutes = 0 to disable idling in a production environment.

The fix is taking longer than anticipated.
Please use version v0.0.6 for now. Thank you.

also seem to get it with 0.0.6 when i try to use this tf code
resource "clickhouse_service" "hubble_timescape" {
  cloud_provider              = "aws"
  idle_timeout_minutes    = 5
  max_total_memory_gb  = 24
  min_total_memory_gb  = 24
  name                            = "name"
  password                     = var.clickhouse_service_password
  region                          = "eu-west-1"
  tier                               = "production"
  private_endpoint_ids   = [data.terraform_remote_state.vpc.outputs.clickhouse_endpoint.id]
  ip_access                     =  ["<IP_ADDRESS>/0"]
}

seems a genaral issue due the use of empty strings as null values
think in my case the work around is setting   encryption_assumed_role_identifier = ""


AutoSlugField changes slugs that end in numbers consufing them with autoincrementing "next" value

I have an entry titled "Articles 2010", and generate the "articles-2010" slug for it. When I update the entry, AusoSlugField will think that the "2010" part of the slug was the incremental "next" value, and when it finds that there were no 2009 entries with slug "article" before it, it will update the slug to "article", thus breaking a slug that shouldn't have changed.

This is also burning us, would much like to see it addressed.

This bug makes the AutoSlugField unusable for me.

This is what I ended up doing:
sdornan/django-extensions@3e0a018
It's definitely a hack, but seems to work okay for my purposes. You would probably run into trouble if you wanted to modify the uniqueness constraints on the AutoSlug field after the fact.


Reworking ClangImporter to not depend on Sema, part 2

This is a follow-up to https://github.com/apple/swift/pull/11795 which addresses the remaining case of a non-bridged newtype whose raw type is bridged.
Also, we now assert when attempting to infer an associated type for an imported conformance. This should no longer happen, and if it does, we've uncovered a case that might fail if we're importing a type after Sema has been torn down.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please test

I'm pretty sure we don't handle typealiases in protocol extensions correctly. If we did, you'd be able to use it for OptionSet too.


example repo topics & naming

From https://github.com/iterative/dvc.org/pull/3087#discussion_r777304564, we should:

name example repos consistently, and
have a convention for adding topics to repos

Maybe:

repo name
repo topics
description

example-*
demo, example, docs
used in official docs

blog-*
demo, blog
used in blog posts

course-*
demo, course
used in official courses

test-*
demo, test
debugging & CSE

Alternatives names & topics which are also inconsistently used include learn, eg, intro, scenario, lesson.
Repo topics make it easy to find/filter repos and are probably more important than repo name. iterative topic:example lists only one repo atm :cry:
There are ~150 repos in total and it's quickly getting impossible to keep track.

I began to believe having a dedicated Github organization like iterative-examples might be a better approach for the example repos. We can fork an example repository to a new name, modify it and use in a blog post/document. At least the fork lineage becomes trackable that way, and we can keep the tags, READMEs and whatnot.
Personally, I like the current approach better, but planning them centrally and building them with a script (like example-repos-dev) seems not scalable when teams have deadlines to meet.

-- @iesahin https://github.com/iterative/dvc.org/pull/3087#discussion_r777472797

if we want to create a plethora of content with examples, having all example repos under Iterative is not sustainable. In my opinion, the examples for the official course and documentation (if needed) should be under Iterative. Everything else - examples for blog, talks, workshops, meetups, and so on are fine to be under personal accounts. These are meant to be temporary.

-- @MKhalusova https://github.com/iterative/dvc.org/pull/3087#discussion_r778090710

These are meant to be temporary.

not sure about this to be honest. We want to have them around at least for some time with some guarantees. Personal account doesn't guarantee anything. Also, it feels that there is an easy way to "solve" this with prefixes or a separate account like folks suggested.

-- @shcheklein https://github.com/iterative/dvc.org/pull/3087#discussion_r778223452

name example repos consistently

I vote for prefixes but if people prefer a 2nd organization I don't see a problem with that either. Well, repos would still need prefixes under iterative-examples though... So not sure how it helps other than moving less important repos out of the main org. which seems like a separate question.

have a convention for adding topics to repos

Lools like https://github.com/topics/ are meant to be specific technologies for anyone on Github to explore. "DVC" would be a topic. "demo", "example", "blog" have meanings internal to Iterative so should be labels or/and prefixes.

name example repos consistently

I vote for prefixes but if people prefer a 2nd organization I don't see a problem with that either. Well, repos would still need prefixes under iterative-examples though... So not sure how it helps other than moving less important repos out of the main org. which seems like a separate question.

Prefixes are good. I don't have any objections. Nevertheless, I don't think we can enforce this. We had a very simple rule to add example as a prefix to the repository name but we can't even let people to conform to this. To my mind, if everyone doesn't conform to a convention (and it's expensive to enforce than to forgo), better to cancel it and find alternative ways to deal with the chaos.
Having a different organization at least will say "we create example repositories here, not in iterative/" Creating repositories under iterative can be made an admin-level permission that way.
I don't think we can maintain a prefix policy for repositories in the long run. Even if we write it to "company policies" and ask people to read during onboarding, the issue is small and very easy to ignore/forget.

have a convention for adding topics to repos

Lools like github.com/topics are meant to be specific technologies for anyone on Github to explore. "DVC" would be a topic. "demo", "example", "blog" have meanings internal to Iterative so should be labels or/and prefixes.

Different organization will complicate administration significantly though and potentially we'll have to pay double if at some point we'll need private repos there.
Complications like - not being able to reference a ticket from a private repo in the main org, invite everyone to two orgs, still maintain some order there.

Lools like github.com/topics are meant to be specific technologies for anyone on Github to explore.

I think (ab)using topics like so to filter within an org is another use case. Seems to address:

I don't think we can maintain a prefix policy for repositories in the long run. Even if we write it to "company policies" and ask people to read during onboarding, the issue is small and very easy to ignore/forget.

We discussed this with Ivan briefly, and renaming the repositories to conform to a naming convention seems feasible. Time to time, we can rake through the repos and rename those that looks example repositories. Github maintains redirections for the renames, so it shouldn't make much difference for the links.
We can also use topics as Casper suggested for quickly sorting/grouping the repositories.

(ab)using topics like so to filter within an org

Doesn't https://github.com/iterative?q=example work for this? If we're consistent enough with repo naming
Not convinced about the topics "misuse".

Doesn't https://github.com/iterative?q=example work for this? If we're consistent enough with repo naming (I agree with that part). Not convinced about the topics "misuse" part.

If the above convention is used, this query won't find, e.g., ˋblog-*ˋ repositories.

Guys I understand there's a decision on this spike. Can we get a summary here for now? Then we can move move to either wiki or internal docs. Or close this and go there directly? Up to you @casperdcl @iesahin thanks

AFAIK it's "push people to use topics on existing and new repos (and ideally create new repos using the naming convention too) as per the the suggestion. Renaming existing repos low priority."

I've created #3245 to add topic labels to example repositories.

I'll move the list of example repos to the wiki, you can also copy paste from #3245 for the time being. I don't think we can push people much about this issue, but having topic labels seems a good first step.

No need to add a list of repos to the wiki or anywhere really if the idea is to use GH to filter them by topic. We could add the info on naming and topics to the Wiki though... Or is that internal documentation?

The list is meant to be generated from the topic/search some time in the future. I used it as a reference and may use it again to update the attributes.

OK. I'm just trying to move the decision here to wiki or internal docs so we can close the spike 🙂

We have added example topic tag for all the repositories in the list. I think we can close this for now. @jorgeorpinel

Closing this, as we have added example topic label and https://github.com/iterative?q=topic%3Aexample&type=source lists all example repositories.


File Explorer opens when I upload/download files from browser

Description
I set Files app as default in the settings but when I try to upload a file or choose "Save as" for a file, it opens File explorer instead Files (I am not totally sure if this is expected behavior from File explorer or a bug)
Steps To Reproduce

Open any page that has download option or just right click on web page and choose "save as"

Expected behavior
It should open Files app instead of File explorer
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22543.1000
Windows Version
Windows 11 21H2 22543.1000
Relevant Assets/Logs
debug.log

Thank you for your feedback. This is the expected behavior for now.


Mini games

Add a section to bot that user can play mini games like rolling dice or play football or basketball mini games.

Done


WIP: adding relatedIdentifiers to the DataciteTableSerializer for the oai …

adding relatedIdentifiers to the DataciteTableSerializer for the oai interface in the metadata-app
see #72

tests:

[x] unittests
[x] local tests
[x] test-instance

all tests passes, ready to merge!


Fix Typo in sketch.js

HAHA I HADN'T SEEN THIS, oh my, sorry and thank you very, very much for the fix! :laughing:


Rotation of image by any arbitrary Angle

[Feature] Added Rotation function for arbitrary angles in numeric/affine.hpp
Description
The rotation function already existed however, the rotation was about the top-left vertex of the image as the origin, resulting in some parts being out of view. I have added a function that rotates the image about its center, using affine transformations, and scales it according to the dimensions of the source image.
This pull request is in reference to issue #533
References
#533
Tasklist

[ ] Add test case(s)
[ ] Ensure all CI builds pass
[ ] Review and approve

Sorry for the mess. I'll create a new PR incorporating the changes

@gkumar28, please do not keep creating new PRs for the changes. Address all the changes in the same PR. It helps in keeping track of what changes are addressed or what more needs to be done.

@lpranam I think this will conflict with your https://github.com/boostorg/gil/pull/573
What do you suggest to do?

@mloskot fortunately I am yet to touch those files :D so if this PR is ready we should get this in


[Win]Test starts - Browser opens and stays idle

Hello,
Platform: Windows 10
Code Base: Frameworkium-Examples
Steps to reproduce
1.) mvn clean verify -Dbrowser=chrome -Dwebdriver.chrome.driver=c:\path\to\chromedriver.exe
Expected: Browser opens and navigates to SUT url
Actual: Browser opens and does nothing.
I re-ran the same on a macbook pro, everything runs as expected.
Thanks Arjun

Which version of Java, Chrome, Chrome Driver and maven are you using? Also could you include the relevant parts (maybe a stacktrace?) of the frameworkium.log file? Thanks.

Also which version of frameworkium-core do you have in the pom.xml file?

Hi Graham,
Thank you for the prompt reply.
Google Chrome is up to date
Version 65.0.3325.181 (Official Build) (64-bit)
chrome driver -> 2.37
Firefox -> 59.0.2 (64-bit)
Gecko Driver -> v0.20.1
Framework-core -> 2.7.2
Code base -> Frameworkium-Examples (unchanged) @ commit b6041e829194d33da70f8ee7c59eb819403e1221

I tried again on another windows box, everything is working as expected. Must be an environment issue.
Closing. Apologies for the hassle.