Vlan Group overhaul

As ...
Austin - Network Automation Engineer
I want ...
Vlan Group model to be more advanced and provide new capabilities
So that ...
I can more efficiently group my Vlans.
I know this is done when...

Vlan Group has .role
Vlan Group has GFK scope Location, Cluster, ClusterGroup
Vlan Group has min_vid, max_vid or range in format [1-10, 30-50, 99]

Related to #526 and #529
Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Apps (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
No response
External Dependencies
No response

Sorry, do you mind explaining the purpose of the scope GFK within the context of this issue? VLANGroup also already has a location attribute, would that be confusing when scope is pointed to a location?

Sorry, do you mind explaining the purpose of the scope GFK within the context of this issue? VLANGroup also already has a location attribute, would that be confusing when scope is pointed to a location?

I think group of vlans can be also related to a cluster or a rack etc. The GFK list in first post is not complete, but I think we could point vlanGroup to more objects (not only to location)

Sounds very similar to #606.


Move path before dev-specific segments; dev segments closer to git

Feel free to close but I think the prompt is more user-friendly when the CWD is in a predictable location. This change will also move the dev-specific segments (ruby/virtualenv/nvm/go) closer to the git segment. What do you think?

:+1:

Well, it actually makes a lot of sense and now i'm feeling bad for never thinking about this.
Awesome.


hi, question about the image size of .nrrd format.

Thanks for your github.
I'm testing with .nrrd format.
however, there is an issue where the resulting image is enlarged.
(ex: input size:256x256x256 --> result : 128x128x128)
Also, the image is enlarged to 32-bit or greater precision.
What part of main.py should I edit?

The size of the result will strictly correspond to the input, so please check your input and output carefully.

My input shape is 256x256x256.
and I set hparam.py like this:
crop_or_pad_size = 256,256,256
patch_size = 128,128,128

fold_arch = '*.nrrd'

save_arch = '.nii.gz'

source_train_dir = '../datasets/source_dataset'
label_train_dir = '../datasets/label_dataset'
source_test_dir = '../datasets/source_dataset_test'
label_test_dir = '../datasets/label_dataset_test'

Is there something I'm missing out on? Please advise.

The setting is correct, the results of the network would be 128128128 and the results after splicing would be 256256256.


Modules compilation order

Test cases where module_a requires module_b to be compiled first.
This can be in case module_b is parse transform, used in module_a.
On module_a use on_load with code from module_b.

Done in https://github.com/comtihon/coon/commit/9df13921556bd4023e6ee13d72897f7c17ac634d


Missing Azure UAE regions from node template list

What kind of request is this (question/bug/enhancement/feature request):
Enhancement request
Steps to reproduce (least amount of steps as possible):
Go to: /g/clusters/add/launch/azure
Open dropdown for "Region"
Result:
UAE Regions missing
I am creating a PR to correct this.

Observed in v2.4.3 that UAE regions are missing in azure node template

Validated with v2.4-head(afdea9956) and master-head(0ab59f95e). UAE regions are populated in the azure node template. UAE Central and UAED North regions are added.


IO performance is too slow.

First, Thank you for your great job!
I'm trying to deploy a bare-metal cluster using glusterfs over RDMA. But the IO performance is not fast as expected.
The glusterfs have 3 nodes using of HDDs with RAID 5 configuration.
The network is 56G Infiniband.
First. I have tested the network the speed is about 36Gbps as shown below.
$ iperf -c <IP_ADDRESS> -P 16
------------------------------------------------------------
Client connecting to <IP_ADDRESS>, TCP port 5001
TCP window size: 85.0 KByte (default)
------------------------------------------------------------
[ 18] local <IP_ADDRESS> port 46752 connected with <IP_ADDRESS> port 5001
[  3] local <IP_ADDRESS> port 46722 connected with <IP_ADDRESS> port 5001
[  4] local <IP_ADDRESS> port 46724 connected with <IP_ADDRESS> port 5001
[  5] local <IP_ADDRESS> port 46726 connected with <IP_ADDRESS> port 5001
[  7] local <IP_ADDRESS> port 46730 connected with <IP_ADDRESS> port 5001
[  6] local <IP_ADDRESS> port 46728 connected with <IP_ADDRESS> port 5001
[  8] local <IP_ADDRESS> port 46732 connected with <IP_ADDRESS> port 5001
[  9] local <IP_ADDRESS> port 46734 connected with <IP_ADDRESS> port 5001
[ 10] local <IP_ADDRESS> port 46736 connected with <IP_ADDRESS> port 5001
[ 11] local <IP_ADDRESS> port 46738 connected with <IP_ADDRESS> port 5001
[ 12] local <IP_ADDRESS> port 46740 connected with <IP_ADDRESS> port 5001
[ 13] local <IP_ADDRESS> port 46741 connected with <IP_ADDRESS> port 5001
[ 14] local <IP_ADDRESS> port 46744 connected with <IP_ADDRESS> port 5001
[ 16] local <IP_ADDRESS> port 46748 connected with <IP_ADDRESS> port 5001
[ 15] local <IP_ADDRESS> port 46746 connected with <IP_ADDRESS> port 5001
[ 17] local <IP_ADDRESS> port 46750 connected with <IP_ADDRESS> port 5001
[ ID] Interval       Transfer     Bandwidth
[ 18]  0.0-10.0 sec  1.65 GBytes  1.42 Gbits/sec
[  3]  0.0-10.0 sec  2.65 GBytes  2.28 Gbits/sec
[  4]  0.0-10.0 sec  2.87 GBytes  2.47 Gbits/sec
[  5]  0.0-10.0 sec  3.08 GBytes  2.65 Gbits/sec
[  7]  0.0-10.0 sec  2.34 GBytes  2.01 Gbits/sec
[  6]  0.0-10.0 sec  2.22 GBytes  1.90 Gbits/sec
[  8]  0.0-10.0 sec  2.76 GBytes  2.37 Gbits/sec
[  9]  0.0-10.0 sec  3.38 GBytes  2.90 Gbits/sec
[ 10]  0.0-10.0 sec  2.73 GBytes  2.35 Gbits/sec
[ 11]  0.0-10.0 sec  3.00 GBytes  2.58 Gbits/sec
[ 12]  0.0-10.0 sec  2.84 GBytes  2.44 Gbits/sec
[ 14]  0.0-10.0 sec  3.08 GBytes  2.64 Gbits/sec
[ 16]  0.0-10.0 sec  2.84 GBytes  2.44 Gbits/sec
[ 15]  0.0-10.0 sec  2.57 GBytes  2.21 Gbits/sec
[ 13]  0.0-10.0 sec  2.16 GBytes  1.85 Gbits/sec
[ 17]  0.0-10.0 sec  1.75 GBytes  1.51 Gbits/sec
[SUM]  0.0-10.0 sec  41.9 GBytes  36.0 Gbits/sec

Next, i test the normal disk speed it is about 322MB/s
$ dd if=/dev/zero of=/test  bs=1M count=1024 oflag=direct
1024+0 records in
1024+0 records out
1073741824 bytes (1.1 GB, 1.0 GiB) copied, 3.33907 s, 322 MB/s

But when i test the gluster volume speed, it is very slow (16.2MB/s)
$dd if=/dev/zero of=/mnt/gluster_vol/test  bs=1M count=100  oflag=direct
100+0 records in
100+0 records out
104857600 bytes (105 MB, 100 MiB) copied, 6.47873 s, 16.2 MB/s

My volume configuration is dirperse 2:1.
gluster volume info vol_3d582863ad2c090a625dd11d548af391

Volume Name: vol_3d582863ad2c090a625dd11d548af391
Type: Disperse
Volume ID: e6538acf-ddff-4b8e-963f-7c42a84e38ac
Status: Started
Snapshot Count: 0
Number of Bricks: 1 x (2 + 1) = 3
Transport-type: rdma
Bricks:
Brick1: <IP_ADDRESS>:/var/lib/heketi/mounts/vg_0a0e479ba2cd16ec734f9937c3c0acb0/brick_54b322354a68d728dce1306dce1e1fc2/brick
Brick2: <IP_ADDRESS>:/var/lib/heketi/mounts/vg_79369cf5dcc8f19f668b99cfa8fc1496/brick_e6a96af3ce804e4615b16f983e4fcc8a/brick
Brick3: <IP_ADDRESS>:/var/lib/heketi/mounts/vg_4d261c2ab4baaf6ba936ddb7c5ee792d/brick_a90fff90e44b4e4c7236c6792f0b109e/brick
Options Reconfigured:
config.transport: rdma
transport.address-family: inet
nfs.disable: on

I googled some posts about performance tuning but it did not help.

Please note that oflag=direct does not guarantee data to be available on disks. Gluster ensures that all data and metadata is on the disks by the usage of fsync() for write operations. Using oflag=sync could possibly give a better comparison with dd.

Thank you for your feedback. Here is the comparison when using oflag=sync. The speed is on glusterfs volume is still far away from the normal disk.
dd if=/dev/zero of=/test  bs=1M count=1024 oflag=sync
[sudo] password for thangvu:
1024+0 records in
1024+0 records out
1073741824 bytes (1.1 GB, 1.0 GiB) copied, 4.88277 s, 220 MB/s

dd if=/dev/zero of=/mnt/gluster_vol/test  bs=1M count=100  oflag=sync
100+0 records in
100+0 records out
104857600 bytes (105 MB, 100 MiB) copied, 7.37125 s, 14.2 MB/s

Not sure if you're still hunting this down, but you may be better off emailing the gluster-users mailing list.

Thank you for your support. I will do that.

@thangvubk I have the same issue, have you managed to solve it?

@thangvubk and @Segflow
I have the same issue here, Any solution?

@leonrado I didn't handle the problem. So i change to NFS.


Add Organizations API to EC2 Service Discovery

Proposal
AWS has an Organizations API that allows you to discover accounts within an organizational unit (ou).
Prometheus EC2 Service Discovery requires each account to be specified individually. This could be achieved more reliably by specifying an ou to traverse and to add each account found to the list of accounts to be checked by EC2 SD.
If this is a sensible feature to add I'm happy to help put together a PR for it.

How would that work with authentication, and the multitude of different ways that auth can work with AWS?

In the same way that EC2 SD requires a role to assume in the target account, Org SD would require a role to assume in the management account that has organizations:Describe* and organizations:List*.

My question is more once you have a list of accounts, how do you authenticate to those accounts?

Also, how would you gain access at the network level to the accounts to be able to talk to their targets?

Currently the SDConfig takes a role_arn which looks like:
arn:aws:iam::1234567890:role/MonitoringService
It then, if that role is set, assumes that role and does the ec2.DescribeInstances dance.
My suggestion is to also provide an optional:
ou_default_role
ou
ou_management_arn
And then to create a role_arn list by getting the account number from the ou and appending the ou_default_role
The config would look like:
scrape_configs:
  - job_name: 'foo'
    ec2_sd_configs:
      - ou: ou-abcdef
        ou_management_arn: awn:aws:iam:managementaccount:role/Discovery
        ou_default_role: Monitoring

The network level is the same approach as in the current discovery - i.e. just done through assumed roles.

Currently the SDConfig takes a role_arn which looks like:

That is only 1 way of the 10+ ways that auth can work with AWS. How does this work with the others?
I'm wary of adding complexity like this for something that only covers one tiny corner of the potential use cases, and requires yet more options that'll then go on to interact weirdly with all the other auth mechanisms which is something we've had issues with for AWS several times in the past.
This feels like adding discovery discovery, which would require a whole new DSL, secret handling etc. to work.

The network level is the same approach as in the current discovery - i.e. just done through assumed roles.

Scrapes have no knowledge of AWS, as discovery is an independent process. They're simple HTTP requests.

That is only 1 way of the 10+ ways that auth can work with AWS. How does this work with the others?

This proposal doesn't touch authentication.
Service Discovery currently exclusively uses roleARN for cross account authorization for describing instances in other accounts. This proposal doesn't add anything new in terms of auth and only uses what is already supported.
AWS strongly recommends against running infrastructure in the management account, so assumed roles is practically the only way people will ever want to talk to the organizations API anyway.

This feels like adding discovery discovery, which would require a whole new DSL, secret handling etc. to work.

That's exactly what it is, but there's no additional secret handling to work on as far as I can see. The simplest approach I can see is that instead of returning a target_group per account we return a target_group per ou and add the account as a __meta_ec2_account label

Scrapes have no knowledge of AWS, as discovery is an independent process. They're simple HTTP requests.

This proposal doesn't change this at all. We would still get a target group back from service discovery exactly how we do now. The only difference is instead of the targets all belonging to a single account they'd belong to a single OU.

This proposal doesn't touch authentication.

That would be the problem I'm seeing. Once you have a list of accounts, how do you authenticate to each of them to find their targets? Does that work with all the different auth methods? Based on experiences with other SDs I'd also expect to see requests to need to filter that list, or have special handling for some of them. So we'd probably be talking about a new form of relabelling on top of that, on top of two levels of AWS auth.
Discovery discovery is a whole new set of problems on top of discovery, and discovery is enough of a problem on its own (with AWS being notable for the complexity its myriad of ever growing auth methods bring). The proposed optional fields also smell like business logic, as I see none of those fields in the API docs. Business logic like that does not belong hardcoded in SD code, we can't make any presumptions about how users have laid out their organisations and accounts.
I have a suspicion that this would be quite complex to do in a proper generic fashion, and that for the level of flexibility you seem to be asking for you might be better off regenerating the config file when there's a new account.

you might be better off regenerating the config file when there's a new account.

We already do regenerate the configuration ourselves with a sidecar process, and that works fine for our purposes.
I really do think this a problem that other people will have in the future, which is why I think it would be a useful addition to service discovery.

Once you have a list of accounts, how do you authenticate to each of them to find their targets? Does that work with all the different auth methods?

The mechanism for authenticating to each account is no different to how it is now, just the method of discovering accounts changes from manually populating them in the config file to discovering them based on the OU they belong to.

The proposed optional fields also smell like business logic, as I see none of those fields in the API docs. Business logic like that does not belong hardcoded in SD code, we can't make any presumptions about how users have laid out their organisations and accounts.

OUs, if used, contain zero or more accounts. They can always be queried from the Management Account. The only assumption made here is that each account within an OU is set up similarly with regards to cross account access and roles being available. As that's pretty much the point of Organizations I think that it's rather safe.

The only assumption made here is that each account within an OU is set up similarly with regards to cross account access and roles being available.

That is an assumption, but the broader assumption is that a) you care about all OUs within the organisation and b) all those OUs are set up in this fashion. That seems like a pretty shaky assumption, particularly as one incorrectly setup account should break discovery for all the rest.

I'm not suggesting scraping the entire organisation - just the OUs of interest.  For example, a common pattern is to have separate OUs for dev, prod and staging. Each of those would be targetted individually. To clarify, an Organization contains one or more OUs. An OU contains other OUs and accounts. Using this proposal the user could start anywhere in this tree that they wanted.
An incorrectly set up account would cuase an error when attempting to assume the role in it to list instances. That could just be logged out as a warning rather than breaking discovery.

I'm not suggesting scraping the entire organisation - just the OUs of interest.

That is the question, how do you know which are of interest. Does a user have to list them all? A regex? What if they want to exclude a particular subtree or account? What about other information a OU might contain? What are the defaults?

That could just be logged out as a warning rather than breaking discovery.

Silently ignoring targets is unwise, on error a SD should continue to use the last good state and it is considered a bug for a SD to do otherwise.

I feel like something is being lost in communication here.
Let's say I have an OU for all my dev accounts called ou-123456. I might be adding a new dev account every time we bring a new dev on board, or want to try a new experiment.  I'd like to be able to run EC2 Service Discovery in all of those accounts.
I would like to simply add something like
aws_ous:
  - ou-123456

to my scrape config, and have EC2 SD run in all accounts within that OU.
The OU changes very, very rarely. Accounts which are in this OU change potentially many times a day.
There is no default, this is a feature for AWS users who are managing multiple accounts, almost always with some sort of tooling.

I understand that, I also understand from experience that that would not be sufficient for many users who want to do more complex operations based on membership of several OUs and any other metadata that AWS currently has or in future adds in relation to these. I have not just to think about the happy simple case, but every other case that users are likely going to ask for if we start going down the path of discovery discovery.

The less API calls we need to do for one service discovery, the best it is. I do not believe in discover discover, because Prometheus itself is not multi-tenant. Error behaviour is also questionable.

Happy to adjust error behaviour, I'm more interested in agreeing on functionality first.
Prometheus currently doesn't do any complex operations based on accounts other than at the label/relabel stage as I understand it. As all I'm interested in doing is providing a list of accounts to discovery with perhaps an additional account label, I believe everything that currently works will still work.

Prometheus currently doesn't do any complex operations based on accounts other than at the label/relabel stage

Prometheus doesn't do any operations on accounts at all, it knows nothing of them. We work in terms of targets which happen to be within an account but that's all hidden from us by the libraries we use.

Service Discovery works by using a role to get a list of all targets within
an account.
Account discovery would use an OU to get a list of all accounts to then get
a list of all targets.
I don’t see that the two concepts are all that different.
On Sat, 24 Oct 2020 at 11:59, Brian Brazil<EMAIL_ADDRESS>wrote:

Prometheus currently doesn't do any complex operations based on accounts
other than at the label/relabel stage
Prometheus doesn't do any operations on accounts at all, it knows nothing
of them. We work in terms of targets which happen to be within an account
but that's all hidden from us by the libraries we use.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/8101#issuecomment-715898135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE2WRCBV7YCHWK5X4WLCV3SMKXP3ANCNFSM4S4KX44Q
.

We looked at this in the bug scrub, and this is an unusual use case that is also quite complex in the general case. Accordingly we're going to close this off. You could also consider something like Consul.


fix: native a11y support for Spinner and improve docs

Paragon ticket: https://openedx.atlassian.net/browse/PAR-457

Added optional screenReaderText prop to provide accessible screenreader-only text and a role="status".
Added additional (pre-existing) examples to the Spinner docs.

PR previews:

brand-edx.org
core

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Validate/update rbk-staging recipe

Update rbk-staging recipe to match rbk-dev (with a tagged version and no hourly updates).

Issue moved to informatics-isi-edu/rbk-project #171 via ZenHub


Roadmap for 0.6.0

Before start working on 0.6.0, I want to complete some pending stuff:

[X] Complete AS3 (Adobe Air for mature iOS target)
[ ] Complete PHP targets
[ ] Stabilize and fix C++ target and make it work with GC instead of ARC (for the moment) even if slow
[x] Perform some cleanups, remove old stuff and perform some target unifications
[ ] Update documentation + videotutorials showing how to use jtransc

I'm working on integrating boehm gc.

I have implemented a typer to avoid casts for ASM backend. Still not fully optimized, since it will work better with a SSA form, but already improves a lot static targets in some small methods:
Before:
C:\Users\soywiz\projects\kor\jtransc\benchmark>..\gradlew rund
:compileJava UP-TO-DATE
:processResources NO-SOURCE
:classes UP-TO-DATE
:jar UP-TO-DATE
:startScripts UP-TO-DATE
:distTar UP-TO-DATE
:distZip UP-TO-DATE
:assemble UP-TO-DATE
:compileTestJava NO-SOURCE
:processTestResources NO-SOURCE
:testClasses UP-TO-DATE
:test NO-SOURCE
:check UP-TO-DATE
:build UP-TO-DATE
:runD
SyncVfsFile(AccessSyncVfs(com.jtransc.vfs._LocalVfs@3800fb71, C:/Users/soywiz/projects/kor/jtransc/benchmark/build/jtransc-d), '')
C:\D\dmd2\windows\bin\dmd.exe -release -O C:\Users\soywiz\projects\kor\jtransc\benchmark\build\jtransc-d\program.d
Execute command C:\D\dmd2\windows\bin\dmd.exe -release -O C:\Users\soywiz\projects\kor\jtransc\benchmark\build\jtransc-d\program.d
Execute command C:\Users\soywiz\projects\kor\jtransc\benchmark\build\jtransc-d\program.exe
JTransc 0.5.16 - drunD
Java 1.8.0_51 - 25.51-b03 - 1.8.0_51-b16
Benchmarking:
plain loops...3.0078
call static mult...30.0486
call instance mult...28.0707
call instance div...106.2825
instanceof classes...180.4823
write byte[]...53.1389
write short[]...62.1656
write char[]...57.1492
write int[]...59.1596
write float[]...57.155
write double[]...80.21250000000001
String Builder 1...106.2781
String Builder 2...61.1653
long arithmetic...1.0029
simd mutable...185.4951
simd immutable...1197.1813
simd mutable matrix mult...512.3615
StringBuilder1...182.461
StringBuilder2...173.4606
Create Instances1...109.3195
Create Instances2...125.3038
Create Instances with builder...1030.709
Java's CRC32...136.3767
jzlib's CRC32...144.3838
compress java's Deflate...76.2008
compress jzlib...81.2752

BUILD SUCCESSFUL

After:
:processResources NO-SOURCE
:classes UP-TO-DATE
:jar UP-TO-DATE
:startScripts
:distTar
:distZip
:assemble
:compileTestJava NO-SOURCE
:processTestResources NO-SOURCE
:testClasses UP-TO-DATE
:test NO-SOURCE
:check UP-TO-DATE
:build
:runD
SyncVfsFile(AccessSyncVfs(com.jtransc.vfs._LocalVfs@682ce6b9, C:/Users/soywiz/projects/kor/jtransc/benchmark/build/jtransc-d), '')
C:\D\dmd2\windows\bin\dmd.exe -release -O C:\Users\soywiz\projects\kor\jtransc\benchmark\build\jtransc-d\program.d
Execute command C:\D\dmd2\windows\bin\dmd.exe -release -O C:\Users\soywiz\projects\kor\jtransc\benchmark\build\jtransc-d\program.d
Execute command C:\Users\soywiz\projects\kor\jtransc\benchmark\build\jtransc-d\program.exe
JTransc 0.5.16 - drunD
Java 1.8.0_51 - 25.51-b03 - 1.8.0_51-b16
Benchmarking:
plain loops...3.0384
                                                                                call static mult...29.0771
call instance mult...28.0998
call instance div...105.28
instanceof classes...179.4773
write byte[]...9.023
write short[]...20.0829
write char[]...12.032
write int[]...13.0326
write float[]...20.057
write double[]...39.0964
String Builder 1...105.2797
String Builder 2...61.162
long arithmetic...0.9708
simd mutable...181.5132
simd immutable...1140.0322
simd mutable matrix mult...363.9969
StringBuilder1...166.4429
StringBuilder2...172.4584
Create Instances1...109.2903
Create Instances2...107.285
Create Instances with builder...1009.6838
Java's CRC32...134.3568
jzlib's CRC32...136.3629
compress java's Deflate...51.1667
compress jzlib...50.1329

BUILD SUCCESSFUL

Still fixing some issues but will come probably today. This will improve both: compilation time (less casts) and execution time (casts are really expensive) in all static targets: Haxe, D, C++, C# and AS3 for the moment.
In write primtype[] you can see an improvement of one order of magnitude because in those methods I was able to do casts properly without the SSA yet.


Add like , comment and save post -- these stuffs to the feed . It'd be great .

Can I contribute? Let me know then :)

i can wok on this with adding more features

Ok then I'll be working with Share and save post options.

@nsrs-78 yes you can start working with it. I'll assign this to you.

Tnxx

@nsrs-78 are you still working on the issue?

not started yet

Hello @Mayuresh-22 can you please assign me this issue i would like to work on it.

@Thanush19 are you willing to work on this issue?

Hello @Parul1606, @GaganMeshram123, @Thanush19 if you want to work on this issue reply this comment and I'll assign on the basis of FCFS

Hello @Parul1606, @GaganMeshram123, @Thanush19 if you want to work on this issue reply this comment and I'll assign on the basis of FCFS

Yes @Mayuresh-22  I want to work on these issue

@Mayuresh-22 I can work on these issue

okay @GaganMeshram123, I will assign this issue to you right away.

@Mayuresh-22 yes

@Mayuresh-22  can you guide me for these issue

Yes just start working and I'll try my best to assist you with it.

@GaganMeshram123 any updates?

Yes merging pull request


Ajax not sending data to controller.

Hi when I use ajax to send form data to controller I'm not getting the date to controller but when I disable ajax and directly submit the form, I'm getting the POST data, I enabled profiler to see if I'm getting data.
My Ajax script is:
<script>
    $(document).ready(function (){
        $("#forgotpassword").submit(function (e){
            e.preventDefault();
            var data = $(this).serialize();
            console.log(data);

            $.post( "welcome/forgotpassword", data, function( response ) {
                alert(response);
            }, "json");
        });
    });
</script>

And in controller I'm using,
public function forgotpassword()
{
	$email = $this->input->post('email');
	$this->output->enable_profiler(TRUE);

	if($this->Authentication_model->setSecurityToken($email)){
		$this->Email_model->sendForgottenPasswordEmail($email);
	}else{
		echo "Details not found";
	}
}

I tried different ways of submitting data, but no use. I'm facing this issue for all ajax calls.
I'm using WAMP 2.5 and CI v3.0.3

This is a bug tracker. Please ask your question on the http://forum.codeigniter.com/.


Unable to find SWIG 3.0 during installation on Ubuntu 18.04

The following two files have SWIG version hardcoded to 3.0 on the line "find_package(SWIG REQUIRED 3.0)"
OpenROAD/src/OpenDB/CMakeLists.txt
OpenROAD/src/OpenSTA/CMakeLists.txt
Using a different version of SWIG while installation throws an error.

Sorry, wrong repo.


create static method to clone existing sinsp_evt

Signed-off-by: vadim.zyarko<EMAIL_ADDRESS>/kind feature
Any specific area of the project related to this PR?
/area libsinsp

What this PR does / why we need it:
This PR creates a facility for cloning sinsp_evts, thus detaching it from the sinsp::next loop.
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: VadimZy
To complete the pull request process, please assign fededp after the PR has been reviewed.
You can assign the PR to them by writing /assign @fededp in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: VadimZy
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval by writing /assign @fededp in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

What's the relationship between this and https://github.com/falcosecurity/libs/pull/344?
it's duplicate for that, the original has stalled.

LGTM label has been added.  Git tree hash: 959fa003b5a7817c49aabb5016cdd1a89656c408

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, VadimZy
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[PORT] Add argument "--framework netcoreapp3.0" to Create PublishedBot.zip

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3224

Fixes "error : The 'Publish' target is not supported without specifying a target framework"
(I picked 3.0 instead of 2.1 or 2.2. It seems to work in my test build, but this ought to tell for sure.)

Not Applicable


x11docker ERROR: Image devdocker_gui not found locally.

Even the: $ docker images
REPOSITORY      TAG       IMAGE ID       CREATED             SIZE
devdocker_gui   latest    9654cbe166be   About an hour ago   2.58GB
My dockerfile is using
FROM  x11docker/mate
...
I have another Dockerfile for command line only, pretty identical FROM	ubuntu:20.04, that is working smoothly. (I start that using docker run, but otherwise quite identical flow - but x11docker just does not find the image )
Host ubuntu 20.04

Similar issue: https://github.com/mviereck/x11docker/issues/468 ....it looks like x11docker just not work with Docker Desktop. You should tell that more loudly in readme as Docker is very much pushing people to use Desktop, not Engine.

I just reread the readme - the reason is that I was using Docker desktop. My bad - it is not working :-/


Is there  a tiny model for a small GPU memory?

My machine's GPU is  RTX3060 with 6G memory,  it can run test.py process, and can not run visualize.py process, so could is there a  tiny model or some strategy to   reduce GPU memory  comsumption?

Unfortunately, we don't yet support running the model on GPUs with limited memory.


Double indention for an object in an array

Description
When formatting an object literal in an array literal, js-beautify "double" indents the body of the object (see sample below).
I'm using two spaces for indention.
Input
The code looked like this before beautification:
var test = [{
  value: 1
}]

Expected Output
The code should have looked like this after beautification (no change):
var test = [{
  value: 1
}]

Actual Output
The  code actually looked like this after beautification (extra 2 spaces are added:
var test = [{
    value: 1
}]

Steps to Reproduce
Just format the snippet above (using 2 spaces as indention) and see two extra spaces being inserted.
Settings
Example:
{
    "indent_size": 2,
    "indent_char": " ",
    "eol": "\n",
    "indent_level": 0,
    "indent_with_tabs": false,
    "preserve_newlines": true,
    "max_preserve_newlines": 2,
    "jslint_happy": true,
    "space_after_anon_function": false,
    "brace_style": "collapse-preserve-inline",
    "space_before_conditional": true,
    "break_chained_methods": false,
    "keep-array-indentation": true,
    "eval_code": false,
    "unescape_strings": false,
    "wrap_line_length": 420,
    "end_with_newline": false,
    "wrap-attributes": true,
    "indent-inner-html": true,
    "e4x": true
}

@briemens - I can't reproduce this on http://jsbeautifier.org/ . Please try it there.

Not sure what version/config is on http://jsbeautifier.org. I was able to reproduce it earlier with a real live example. I need to play with it and come back to you how I did that.
On my machine I run v1.6.14 of js-beautifier, installed via NPM, and has this problem and also the duckduckgo version of js-beautifier shows the same problem: https://duckduckgo.com/?q=jsbeautify&ia=answer
My use case is that I use js-beautifier from VIM to format my code, this I where I noticed it first.

I was able to reproduce it on http://jsbeautifier.org using the following code:
var test = [{
  value: 1,
},
{
  value: 2
}]

Which gets formatted as:
var test = [{
    value: 1,
  },
  {
    value: 2
  }
]

If I put the closing and oping brace on the same line it formats it correct:
var test = [{
  value: 1,
}, {
  value: 2
}]

What you're describing above is kind of a bug but also kind of by design - it's not a perfect design but it is intentional.
The beautifier tried to remove extra indentation where it makes sense.  What you're seeing is because there are two items in the array and there is a newline between them.   Or to put it another way, as long as the array itself is all on one line (even if the items are multi-line), the beautifier will undindent.  If not, it won't.
So, for this:
[
  a,
  {
    value: 1
  },
  c
]

The array itself has multiple lines, so it can't be unindented it.  But in this case:
[a, {
  value: 1
}, c]

The beautifier sees:
[a, multi-line-item-so-what, c]

And so that beautifies with unindent (unchanged from above original input):
And can unindent.  Going back to your examples:
var test = [{
    value: 1
  },
  {
    value: 2
  }
]

var test = [{
  value: 1
}, {
  value: 2
}]

The beautifier sees these as:
var test = [multi-line-item,
  mult-line-item]
// The 2 space indent above makes sense here.

var test = [multi-line-item, multi-line-item]
//No extra indent is need here

Now you might argue that the beautifier should figure out that theres really just to objects there and unindent.  But then you have two different result for these two inputs:
var test = [{
    value: 1
  },
  {
    value: 2
  }
]

var test = [{
    value: 1
  },
  a
]

It's comes down to the definition of a "multi-line" block. When you add the new line between items in the array, it becomes a multi-line block.  I'm open to evaluating better ways to detect when to unindent, but I haven't found one yet.


Security banner not being removed after successfull upgrade

Issue Summary
From the official documentation I have tried the forceful upgrade of ghost to latest version but even after successful upgrade, it's still showing the Upgrade banner in the ghost admin

To Reproduce

sudo -i -u ghost-mgr
cd /path/to/ghost
ghost update -f

Technical details:

Ghost Version:
Node Version:
Browser/OS:
Database:

Yeah, a self-hosted instance that I manage is suffering from the same problem. We still have the security vulnerability warning present after upgrading (without force in our case) to 4.16.0.

The banner is not a problem right now. I believe the team will fix it asap. I want to know whether my website is secure or not. If I manually have to fix some package, plz let me know

same here, same latest version

The upgrade banner in Ghost admin is not currently being automatically dismissed. We realise that doesn't inspire confidence and are tracking getting that fixed ASAP.
In the meantime:
If ghost -v and/or the "What's new" page in Ghost admin say that you are on 4.15.1 or higher, then your site is patched and secure.
If you are on 4.15.1 or higher then you are safe to dismiss the banner by clicking the ✖  on the right hand side.

If ghost -v and/or the "What's new" page in Ghost admin say that you are on 4.15.1 or higher, then your site is patched and secure.
If you are on 4.15.1 or higher then you are safe to dismiss the banner by clicking the ✖ on the right hand side.

I found that ghost -v doesn't always work - for example, I just get this output Ghost-CLI version: 1.17.3 and nothing about Ghost itself.
Also, the "✖ on the right hand side." isn't easy to see if you're in dark mode. Got it now though!

This bug was closed by https://github.com/naz/Ghost/commit/c0d59db5bee2643b8b1ab7ec36797be51f62b965


Repo sync for protected CLA branch

The pull request is created from master637480585151242059 to master to fix git push error for protected CLA branch

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


[RTL] Fix for #2235

Signed-off-by: Srikrishna Iyer<EMAIL_ADDRESS>

This LGTM.

sorry you went through all this work. I'm only noticing now that the comment stated jtag mux was temporary...

Yeah, I saw it too. I think there is still a value in maintaining the autogen pkg. If not for design, it will be useful for DV.

Thanks @sriyerg for reporting the issue and preparing a fix. Apologies for introducing this problem with #2204 . I added another DIO for pin flip feature inside usb and overlooked the dependency of those indices on the DIO ordering.
I like the approach with the package as a first step to prevent such errors of forgetting to update a hardcoded value in just one place. However, it seems that there are still questions on how this is best done.
Maybe for a quick fix it is better to just change the hardcoded values (to unblock things quickly).

as a side note, i think it makes sense to merge the contents of top_pkg.sv
into this one. But we can probably leave that as a separate exercise.
On Wed, May 13, 2020 at 11:51 PM sriyerg<EMAIL_ADDRESS>wrote:

sorry you went through all this work. I'm only noticing now that the
comment stated jtag mux was temporary...
Yeah, I saw it too. I think there is still a value in maintaining the
autogen pkg. If not for design, it will be useful for DV.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2237#issuecomment-628427032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSVN4SBDXJL2MWOAQX3RROIGBANCNFSM4NAIGD7Q
.

This LGTM. I'm pro merging this now, since it seems like without this fix, CI private will just fail.

Sadly the private CI passes. The real UART test that needs to replace the hello_world test in DV is in PR #2052 which is not yet merged. This fix blocks it.

wait really? Because in another one of my PRs, it started failing with GPIO
interrupt unknown (and mine definitely does not touch that).
When I go to head of tree and just run csr_hw_reset_test with seed = 1, it
also fails for me.
On Wed, May 13, 2020 at 11:53 PM sriyerg<EMAIL_ADDRESS>wrote:

This LGTM. I'm pro merging this now, since it seems like without this fix,
CI private will just fail.
Sadly the private CI currently passes. The real UART test that needs to
replace the hello_world test in DV is in PR #2052
https://github.com/lowRISC/opentitan/pull/2052 which is not yet merged.
This fix blocks it.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2237#issuecomment-628428173,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSX3XED73L2H4FR5NODRROIQLANCNFSM4NAIGD7Q
.

Thanks @sriyerg for reporting the issue and preparing a fix. Apologies for introducing this problem with #2204 . I added another DIO for pin flip feature inside usb and overlooked the dependency of those indices on the DIO ordering.
I like the approach with the package as a first step to prevent such errors of forgetting to update a hardcoded value in just one place. However, it seems that there are still questions on how this is best done.
Maybe for a quick fix it is better to just change the hardcoded values (to unblock things quickly).

I am inclined towards merging this as - is, considering it doesn't matter anyway. Is that okay?

I am inclined towards merging this as - is, considering it doesn't matter anyway. Is that okay?

What do you mean by "it doesn't matter anyway"?

I am inclined towards merging this as - is, considering it doesn't matter anyway. Is that okay?

What do you mean by "it doesn't matter anyway"?

Ohh, from the comment it sounded like that logic will be removed.

Ah okay.
There is one remaining question for Sam around the new autogen package (thanks for putting that into a separate commit). But I think it is okay to merge this now and solve the details in the package later.

Ah okay.
There is one remaining question for Sam around the new autogen package (thanks for putting that into a separate commit). But I think it is okay to merge this now and solve the details in the package later.

Yeah, I posted that question :) I will check with him offline.

Sounds good, thx @sriyerg .

Ok, thanks. Merging this in now.


Maps - modernize and clean up filter

Notes:
Changing the black bar that runs under the main drop down menu/header to add context and look more modern.
Updating filter to have a more seamless look with the above stats bar and making the word filter to Filters (also making this word the word that shows filters not the three line symbol). This filter bar should look like it is part of the overall site rather than seeming off as it does now and needs to be readable.

Ensure that the filter is mobile responsive also.

CPI Review 2019-10-11_UCOMM.docx

@manushah17
8.	Maps pages/header
a.	Filter – make backgrounds consistent
b.	Make line red to be consistent with header line (in thickness too)
c.	Remove odd line under the word reset
d.	Change reset to white linked text with an underline rather than button
e.	Make the light gray box square and white not light gray
f.	Make the font black (all blacks including headers if not gray need to be the black listed above)
g.	Make the filters button consistent with other buttons (if red is not visible make white)
h.	Expand the header box for the maps pages and remove information buttons
i.	Need to create an area for a snippet that gives context about the Map
Here are the notes.  Make the filters button consistent with other buttons (if red is not visible make white)
Expand the header box for the maps pages and remove information buttons
Need to create an area for a snippet that gives context about the Map
Double in size and add context there.  Need to take info text and put it in text under title of map.
Opacity 70% - 75% rich black.  Still use white text and allow to see map underneath it.


Appears to emit no valid protocol header?

I have been trying to run a simple example client against various different AMQP servers from the Apache Software Foundation (Apache ActivemMQ Artemis, Apache Qpid Dispatch Router and Apache c++ broker aka qpidd).
In all cases I am unable to connect. My test client is as follows:
import os
try:
    from urllib import urlparse
except ImportError:
    from urllib.parse import urlparse

import uamqp
from uamqp import authentication

uri = os.environ.get("AMQP_SERVICE_URI") or "amqp://localhost/foo"

def uamqp_receive_simple():
    parsed_uri = urlparse(uri)
    anon_auth = authentication.SASLAnonymous(parsed_uri.hostname, port=5672)

    message = uamqp.receive_message(uri, auth=anon_auth, debug=True)
    print("Received: {}".format(message))

if __name__ == "__main__":
    uamqp_receive_simple()

Viewing the interaction with wireshark, it appears that the client does not emit a valid AMQP protocol header at the start of the stream. I see 16 03 01 00 ab 01 00 00 seemingly every time as the first 8 bytes. The client then seems to spin in a loop, with cpu going to 100%.
If it is useful for testing, you can use:
docker run -it -p 5672:5672 docker.io/gordons/qdrouterd:1.3.0

To start an instance of qpid dispatch router listening on 5672 (no TLS, no auth required).

Thanks @grs for these details!!
I will try to repro :)
The only thing I can answer off the top of my head is the 100% CPU use - I generally assume that establishing the connection and authentication will a be quick process and therefore I made no effort to slow down iterating the connection as much as possible during this process.
Clearly this assumption is invalid in circumstances where:

The connection or authentication takes time to establish
For whatever reason the connection fails to establish

So I will revisit the design there.
Again, thanks!

Thanks for the report @egandone - I would really like to invest the time in getting this project working with these other AMQP servers - though unfortunately I wont be able to in the immediate future :(
I will hopefully be able to revisit this in the coming months as we are on-boarding more people to help with these projects.


Create new screencast automatically on docs build

Uses mkdocs-termynal to automatically create a new screencast recording when the docs build.

Add mkdocs-gen-files to run scripts as part of the mkdocs build
Add a script docs/scripts/generate-termynal.py to generate docs/_partials/termynal.md with the output from running CCDS.
Use mkdocs-include-markdown-plugin to incorporate the generated docs/_partials/termynal.md in index.md
Add the mkdocs-termynal plugin to add replay for the terminal session in the docs.
Add some CSS/JS tweaks to make things look nice and smooth.

Preview here: https://cookiecutter-data-science-pr-339.onrender.com/#example


docker-image

This is the minimum viable Dockerfile, which you can build and push to the docker hub.

Thanks for the PR @disfated - I have a couple of questions I'll add to the files.

@disfated - I put a bunch of thoughts in this PR's comments, you don't need to respond to them.  They're comments to help clear up my own thoughts, and for posterity.
In summary, I agree with a lot of what you say.  Between the two evils, I think I'd prefer evil # 2, using mounts, just because it removes a bit of complexity from the user's side.  There are some immediate improvements required to make this "mount" option more viable, which you mentioned:

add some way to populate an empty data directory from the image
add some "user preference" UI for the settings -- this will remove the need for the .env file edits entirely

When running, users would need to specify the data folder, and the backup folder ... I agree there is a chance for error there, but I don't know a way around that!  As you mentioned, the goal for this stream of work is to make Lute easy to install, but -- in my mind, at least -- the better way to do that is to spend time in the python rewrite.  (Which is a slog :-/ but I think it's necessary)

Following up.  I've spent some hours reading through and trying out volumes, and feel that they're good for many ops-type situations, but that they could easily create confusion for non-techies, and then for the inevitable support questions that will come up.
To summarize, I'd like to move forward with pre-built containers that use bind mounts, and provide simpler compose files and .env files for that, with some extra checks and documentation.  This is the only thing that I'm comfortable supporting at the moment.
Some notes below:
Volume management isn't as easy as file management
When no volume is explicitly passed in the container run command, VOLUME in the dockerfile creates an anonymous volume with a SHA id, e.g.
$ docker volume ls
DRIVER    VOLUME NAME
local     1a078b998378121f021cde0e70b1c3a3e26f37a013ad49fbd853856edef0b783
local     3e6eaac05d79693395279a6df813efc75b4ec629a4dff14b73798853ce1f7b93
... etc

While this should work fine for just getting up and running, I can't see a clear path for managing that for users easily.  You can also specify the volume name in a docker compose file or a command line, but that also requires users to get the flags right -- it sounds dead easy, but users sometimes do fun things.
Users can create and mount volumes, but given the challenges that they have already, I don't feel that's an option.
Interestingly, specifying the volume name in a compose file seems to create different volumes depending on where the file is stored, per https://forums.docker.com/t/docker-compose-prepends-directory-name-to-named-volumes/32835/9.  I created a zzdocker folder with a dockerfile and compose file:
version: "3.9"

services:
  frontend:
    image: xxxx:latest
    volumes:
      - myapp:/blah/data
volumes:
  myapp:

Running this created a named volume:
$ docker volume ls
DRIVER    VOLUME NAME
...
local     zzdocker_myapp

I then moved the compose.yml file to another location and ran compose up again, and a new volume was created.
$ docker volume ls
DRIVER    VOLUME NAME
...
local     zzdocker_myapp
local     zzzzdocker_myapp

Based on this, I'm going to go with mounts.
Mis-mount = big problems
As mentioned, mounts create some dangers: users could mount things incorrectly!  So that would be terrible.  That can be mitigated in a few ways:

The .env for the application could be baked in to the image, along with some default settings (e.g. backup enabled, 10 backups, etc etc)

a separate, much smaller .env file for docker compose to use could be created.  This would just have two things:

DATA_PATH=xxxx
BACKUP_PATH=yyyy

Then the compose file would reference that .env file, and those would be used for the mounts.  Composer does check that the directory exists when it composer ups.

We can also check the content of the mount listing if needed in the entrypoint script, e.g:

#!/bin/sh

REQUIREDDIR=/lute/data
mountdata=`mount`
mountVar=`mount | grep "$REQUIREDDIR"`

# echo $mountdata

echo "got mountVar = $mountVar"

if [ -z "$mountVar" ]
then
    echo "$REQUIREDDIR not mounted ... exit"
    # echo $mountdata
    exit
else
    echo "$REQUIREDDIR mounted"
    touch "${REQUIREDDIR}/hello.txt"
fi

So that if some users decided to mount things directly from the command line docker run -whatever-flags ... it would at least safeguard that.

Documentation!  Once things are configured.  With a big red exclamation point asking people to verify the folders they expected to get populated are in fact getting populated.

buildx is still needed
For the image to work on things like Raspberry Pi, different linux arch's etc, the build still needs to output multi-arch.  This is a separate note from the Dockerfile and compose, just specifying it here as part of the build necessary.  A good article is here: https://www.docker.com/blog/multi-arch-build-and-images-the-simple-way/
Summary
Pre-built containers is still the way to go for this version of Lute.  I've been getting some basic things down for v3 Python, and think that that will be much easier to work with, especially since sqlite is baked in to Python.

Closing this, thanks @disfated for the discussion and notes.
I've implemented pre-built docker containers in lute v3 (https://github.com/jzohrab/lute-v3/) with a semi-automated way to push images to docker hub (https://hub.docker.com/repository/docker/jzohrab/lute3/general).  The lute v3 docs (https://jzohrab.github.io/lute-manual/install.html) also refer to those pre-built images, it's a much better intro.
Also implemented are "settings" so there's no more '.env' file for people to hack at.
Phew!


No response found in the 'my-cache' cache.

Library Affected:
"workbox-cli": "^3.6.2"
"workbox-range-requests": "^3.6.2",
"workbox-sw": "^3.6.2"
Browser & Platform:
Google Chrome | 87.0.4280.88 (Official Build) unknown (64-bit)
Ubuntu 20.04.1 LTS
Issue or Feature Request Description:
The issue is during development on localhost:8000.
It seems workbox is claiming my cache has no entry matching the request:
workbox-core.dev.js:132 workbox No response found in the 'build-media' cache
due to:
The FetchEvent for "http://localhost:8000/manifest/media/e460e3d099bfa7c4538825ce705bf75f-57904.m4a" resulted in a network error response: an object that was not a Response was passed to respondWith().
Promise.then (async)
(anonymous) | @ | workbox-routing.dev.js:837

I can see from dev tools Application/Cache that it is in the cache, and I don't see any such problems running on a non-localhost server.
The oddity that is worth mentioning is that I am using cacheOnly in my sw, and my app is fetch()ing the files manually and cache.put()ing the files into the cache with a key that is the pathname prepended with '/manifest'. The app then requests the 'fake' paths as seen above. The main reason I do this so that I can get fine grained loading progress. I do clone the response and cache.put() the cloned response.
It also seems to only be a problem for the m4a files - my jpg files in the same cache seem to work 'just fine'. However, both types seem to work fine on non-localhost server.
Any ideas?

I see to get the same problem if I add a background-image: url("/manifest....") in dev tools...I get no 'hit' in the service worker 'match' callback :/

Is there a Vary: header on your responses, perhaps? C.f. https://github.com/GoogleChrome/workbox/issues/1550

Yes, the localhost responses do have a Vary header, and the ones on the real server do not:
Vary: Accept-Encoding
I followed the link you gave, and the links in that, but it's not entirely clear what I need to change.
This seemed a better link than the ones in #1550:
https://w3c.github.io/ServiceWorker/#dictdef-cachequeryoptions
IE, somehow I need to set ignoreVary to true.
I see matchOptions here:
https://developers.google.com/web/tools/workbox/reference-docs/latest/module-workbox-strategies.CacheOnly#matchOptions
and...
https://developer.mozilla.org/en-US/docs/Web/API/Cache/match
So, I tried this:
workbox.routing.registerRoute(
  ({url}) => {
    const {pathname} = url;
    const retVal = !!pathname.match(mediaRegExp);

    if (retVal) {
      console.log(`SW: resolving ${pathname} from ${mediaCacheName}.`);
    } else {
      console.log(`SW: ${pathname} not in ${mediaCacheName}.`);
    }

    return retVal;
  },
  workbox.strategies.cacheOnly({
    cacheName: mediaCacheName,
    matchOptions: {
      ignoreVary: true,
    },
    plugins: [
      new workbox.rangeRequests.Plugin(),
      new workbox.cacheableResponse.Plugin({
        statuses: [0, 200, 304]
      })
    ]
  })
);

I also attempt a cache.match() when I decide to fetch() the real file in the first place, so I also added the option there:
        await myCache.match(file, {ignoreVary: true})

I did have an unrelated problem (snafu) that was confusing matters, but all-in-all...seems to be working now, so thanks :D


Install process doesnt support latest version of terraform (0.12.x)

Micro services

Spring boot Applications:

Admin Service
Auth Service
Asset Service
Compliance Service
Statistics Service
Notification Service

Rule Engine
Rules
ETL
Webapp

OS Type: Windows/Linux/MacOS
Java version:

1.9
1.8
1.7

Summary
Install instructions say install latest version of terraform. Latest version of terraform is 0.12.x
Terraform 0.12.x introduced change to structure of resource blocks such that all tf files need to be upgraded to 0.12 version. (see https://www.terraform.io/upgrade-guides/0-12.html )
So either the pacbot install instructions should say "install terraform v 0.11.11" or the tf files need to be upgraded to be compatible with syntax employed by terraform v0.12.x
Reproduce steps
run install using terraform version 0.12.x
Expected Results
terraform to run without any issues
Actual Results
terraform plan fails

Terraform latest version support is under development and will be released soon.

great - maybe update the install instructions to mention that 0.11.11 is latest version of terraform currently supported ?

as part of  2.0 release latest version will be available


Adding Search Functionality

Implement a search bar to allow users to search for specific travel journals based on keywords, destinations, or user names.

hey@amrit03b  I would like to contribute on this pls assign this to me

hey can you assign to me

hey@amrit03b I would like to contribute on this pls assign this to me

Assigned.

hey can you assign to me

Assigned @prithu12

hey there, can you assign this task to me

hey there, can you assign this task to me

Assigned @harshit-m007


Automatic Resizing of images

Hi,
I've got a problem with the original image src size. I'm not sure why but the original image size seem to be a set size. So image the image is larger than this set size I'm getting image pixelation.
So, in this case, the "intrinsic size" is always 1024px wide. So from what I can tell this is maximum image width. But I've set this size to be 'fp-large' which is 1200px wide. So why is the image not rendering at this size? The original uploaded image size is about 1300px wide. I've attached a screenshot which shows what's happening.

Hi Frank
Check the library --> responsive-images.php file. There's a function there that limits the size of full width images. I remember having a similar issue a while ago and I think I resolved it by commenting out that function.


No warning for wrong type in addition

There is unfortunately no warning at all in PHP for using + with strings:
echo ('I want ' . ('$1000' + 'dollars')  . '!');

Outputs "I want 0!" (never write a financial accounting sofware in PHP?)
Would it be possible to add a warning about this type of type-error?
Also, would it be possible to have a sort of type checking so a function like this would be marked for invalid addition?
/**
 *
 * @param string $a
 * @param string $b
 */
function concat($a, $b) {
        return $a + $b;
}

@programmin1 some IDEs like the one I use (PhpStorm) warn you about such things:

Feel free to open a Pull Request.

Are you saying that in order to even warn about this operator problem - which Python and other languages will quit on, and Java won't even compile - you have to buy a special IDE?
The fact that things like this are legal in PHP - as well as allowing using variable names that don't exist, mistyped constants that don't exist, and only getting a warning-notice while continuing on its potentially-disasterous way, are likely one reason PHP is not remaining competitive against other languages that fail early and have quality-analysis tools that catch things like this.
I'm glad to see a good start on code-quality analysis with phpmd, but unfortunately it doesn't seem to have most of the features I'd expect - do you know if the code-checking component of PhpStorm is open-source, and if so could it be merged in with this project?

@programmin1 :+1:

Constructive PRs are always welcomed.


Remove estrin feature and set version number to 1.0.0

The feature reduced the accuracy of the functions. Since features are assumed to be additive by the compiler this meant that if this feature was enabled anywhere in the dependency tree, the crate became less accurate everywhere, making it unreliable.
This PR also sets the version number to 1.0.0. I did not do this before as I thought I couldn't since I had dependencies that have not hit 1.0.0, but the restriction is only on public dependencies: https://rust-lang.github.io/api-guidelines/necessities.html#public-dependencies-of-a-stable-crate-are-stable-c-stable.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (7cf34f2) to head (c304c2e).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #27   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines         1963      1865   -98
=========================================
- Hits          1963      1865   -98

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


Added the rewrite tag plugin

Added the ability to re-write tags

rewrite-tag-filter is widely used so adding it seems useful. Thanks!


urlon.stringify() not encoding '/' (slash)

While using this library to update url in my browser I faced an issue while converting it back to JSON.
For example:
urlon.stringify({ foo: 'ba/r' }); // OUTPUT: $foo=ba//r

But in URLs / is a valid character.
Temporarily, I'm using this to solve the problem.
encodeURIComponent(urlon.stringify({ foo: 'ba/r' })); // OUTPUT: %24foo%3Dba%2F%2Fr

It would be great if this issue can be fixed!

This library has 2 major goals:

for any JSON value J, JSON.parse(JSON.stringify(J)) is deep equal to urlon.parse(urlon.stringify(J))
urlon.stringify - to use only characters that could be shown in url without forced encoding.

As you have stated / is a valid character to be used in url.
I could try to guess an actual issue you having is that / is also a common character that is used as path delimeter.
Most times I've used urlon it was for query part, that goes after ? character. And there is no rule that disallows use of / there.
I only used urlon with primitives (number, boolean, string) when applied it to path segment part. Though it seems still can be an issue for urlon.stringify('ba/r').
Could you please clarify your use case in more details. I had some improvements in mind for a next breaking change release, so potentially could consider your case if find it valid.

Thanks for the quick response...
So in my web app (React.js), I'm generating a table using the params passed from the URL.
And while fetching them, it only fetches till /. It is considering the other part as some different path.

@apal21 do you work with url part on Frontend only side? Do you use any router library? Could you share exact snippet how do you integrate urlon in your app?

There is nothing special about the code.
I am using it like this for now
<Route exact path="/:fields/:conditions" component={HomePage} />

But as you mentioned I'll probably convert it to a query. Like ?fields=something&conditions=something
You can close this issue. And thanks again for the quick responses.

with ?fields=something&conditions=something you still may have issues.
I'd suggest to use urlon for whole query parsing parsing.
so use result of urlon.stringify({ fields: [''], conditions: {}, other_query_param: 'something' })

Ohh okay got it!


Improve retry

hi,
I have some questions about how to handle only failed data and retry after a write failure. Can we discuss adding this design pattern

`private void flush() throws IOException, InterruptedException {
    synchronized (ElasticsearchSink.class) {
        int contextRetryNum = context.getRetryNum();
        while (contextRetryNum > 0) {

            try {
                BulkRequest request = bulkRequest.build();//构建operation时候有异常 missing异常
                List<BulkOperation> operations = request.operations();
                checkpointState.clear();
                checkpointState.addAll(operations);
                BulkResponse result = esClient.bulk(request);
                if (result != null && result.errors()) {
                    contextRetryNum--;
                    handlePartialFailures(result, bulkRequest);
                }

                break;
            } catch (ElasticsearchException ese) {
                LOG.error("write es failed:" + ese.getMessage());
            } catch (MissingRequiredPropertyException missError) {
                //此异常证明没有操作需要进行 //超时异常也不管
                LOG.error("es sink error:{}", missError.getMessage());
            } catch (Exception ex) {
                // 等待一段时间后重试
                Thread.sleep(context.getSleepTime());
            }

        }

        bulkRequest = this.bulkRequestFactory.create();
        thresholdCounter.set(0);
    }

}

`

AltLayer Airdrop Season One Update

🚀 Exciting Announcement! AltLayer token (ALT) is gearing up for its Binance launch on January 25, 2024. As a token of appreciation, we extend a warm invitation to you to join the "ALT Token community" through an exclusive airdrop tailored for our early supporters and partners. Here's a quick overview:

Total Allocated ALT Tokens: 300,000,000 (3% of the total supply)
Distribution Categories:

Ethereum Mainnet
Binance Mainnet

For a comprehensive breakdown of criteria and further details, refer to the complete announcement Oficial Website.
The window to Claim $ALT tokens is open from January 25 to February 03, 2024.
Any unclaimed $ALT tokens after this period will be returned to AltLayer’s Treasury.
We deeply appreciate your contribution to AltLayer's success! Stay tuned for more exciting developments in Season One.
Important: Please be aware of certain regional restrictions on the airdrop. Review the disclaimer before participating.
ZiNai, @Ayush-Mahajan, @linguistbreaker, @AlbertoVari, @Lucas-L, @jcampuza, @atavism, @software-is-life, @bub, @guangzhixie, @fangjingping, @phumoonlight, @khanhtc1202, @dxyzx0, @yangchengjian, @Iridocyclitis562, @yaalsn, @sriharshakappala, @hoangthienan, @stes


Problem with new HLSLccExporter

Import: System.Exception: Unexpected program type DX9VertexSM30    bei DXShaderRestorer.ShaderGpuProgramTypeExtensions.GetMajorDXVersion(ShaderGpuProgramType _this)    bei DXShaderRestorer.ResourceChunk..ctor(ShaderSubProgram shaderSubprogram)    bei DXShaderRestorer.DXShaderProgramRestorer.GetResourceChunk(ShaderSubProgram shaderSubprogram)    bei DXShaderRestorer.DXShaderProgramRestorer.RestoreProgramData(Version version, GPUPlatform graphicApi, ShaderSubProgram shaderSubProgram)    bei uTinyRipperGUI.Exporters.ShaderHLSLccExporter.Export(ShaderWriter writer, ShaderSubProgram& subProgram)    bei uTinyRipper.Classes.Shaders.ShaderSubProgram.Export(ShaderWriter writer, ShaderType type)    bei uTinyRipper.Classes.Shaders.SerializedSubProgram.Export(ShaderWriter writer, ShaderSubProgramBlob blob, ShaderType type, Boolean isTier)    bei uTinyRipper.Classes.Shaders.SerializedProgram.Export(ShaderWriter writer, ShaderType type)    bei uTinyRipper.Classes.Shaders.SerializedPass.Export(ShaderWriter writer)    bei uTinyRipper.Classes.Shaders.SerializedSubShader.Export(ShaderWriter writer)    bei uTinyRipper.Classes.Shaders.SerializedShader.Export(ShaderWriter writer)    bei uTinyRipper.Classes.Shader.ExportBinary(IExportContainer container, Stream stream, Func3 exporterInstantiator)    bei uTinyRipperGUI.Exporters.ShaderAssetExporter.Export(IExportContainer container, Object asset, String path)    bei uTinyRipper.AssetExporters.AssetExportCollection.Export(ProjectAssetContainer container, String dirPath)    bei uTinyRipper.AssetExporters.ProjectExporter.Export(String path, FileCollection fileCollection, IEnumerable1 assets, ExportOptions options)    bei uTinyRipper.GameStructure.Export(String exportPath, Func2 filter)    bei uTinyRipperGUI.MainWindow.ExportFiles(Object data)

Well, new HLSLccExporter doesn't seem as good as it looked :[] Could you upload a file for test?

unfortunately im not sure which file causes it and im not that much into hlsl shaders that I could make a sample project. 😞

I thing HLSHcc doesn't support DX9. I set old exporter for this shaders in b5e19357b454aff2685fc4e48f6291e765701744. Waiting for @spacehamster to clarify this moment.

On the games I tested it on, I didn't run into any DX9 shaders, so I'm not entirely sure if HLSLcc supports DX9. The converter doesn't currently, but If you can upload the game assets that contain the shader, I can take a look into it. What game is it by the way?

It's The Forest. Is there any easy way to see which asset file is currently processed without having to insert log into the code? Doesn't seem like there is a log with more information anywhere.

resources.zip
rename zip to asset

I have that game, no need to upload the asset for that one.

Ok, i looked into it. DX9 shaders which use shader model 2 and 3 have a completely different bytecode format to DXBC (shader models 4 and 5). HLSLcc doesn't support DX9 and it would basically require its own decompiler or a DX9 -> DXBC bytecode convertor to work, which wouldn't be trivial especially considering how little documentation there is for the DX9 bytecode format.
It appears that unity uses hlsl2glsl for DX9 shaders, and it appears to be a source-to-source compiler, which wouldn't be of any help for uTinyRipper.

Well, current converter is already good enough. Of cause, it is a little sad because of lack of dx9 support but I don't think it is a big deal. Especially, since dx9 was deprecated long time ago.


Start Razor Language Server when new Razor LSP editor opens.

Included the LanguageServer core dlls as part of the VSIX install. To do this I had to expose the language server's output path via a target and then dynamically include its outputs in the VSIX output.
Given how VSIX's are built I could not privately reference our language server to ensure that it's built when the VSIX is built. To work around this limitation I added a _BuildLanguageServer target that calls MSBuild directly on the language server as part of build.
For now I'm dynamically booting the currently built language server that's deployed with the extension. I haven't given a lot of thought to what flavor of OS (x64 vs. x86) the server can run on; or if we need to ensure dotnet.exe is available.
Found that the VS LSP client and the O# language server library that we use don't inter-operate as we'd expect. The server ends up telling the client that it doesn't care about text document change events (we do). I've created a separate PR in O# to fix this: https://github.com/OmniSharp/csharp-language-server-protocol/pull/199

aspnet/AspNetCore#17789

@dotnet/aspnet-build This PR doesn't do anything special with dependencies and the one it's based off of already passed. Any idea why signing would be exploding all of sudden?

@dotnet/aspnet-build Never mind, I see why it's complaining in this PR given that I'm lifting bits into the VSIX now.


External pagination tutorial link in pagination tutorial is broken

The link for external pagination tutorial at http://ui-grid.info/docs/#/tutorial/214_pagination is broken. It is pointing to http://ui-grid.info/docs/#/tutorial/318_external_pagination but it should instead point to http://ui-grid.info/docs/#/tutorial/314_external_pagination

Thanks, this should be fixed next time the docs generate.


Add opinionated but configurable automatic updates

Let's steal some of the logic and code from https://github.com/coreos/zincati/blob/main/docs/usage/auto-updates.md

I'd suggest considering renovatebot as the framework for auto-updating. We can contribute to the project in case it doesn't support this use case.

Big +1 for renovate. Although I'm not sure how it would work in the model unless a "treefile" is somewhat tracked by GitOps

@lmilbaum as I understand it, this issue is about updating a bootc OS deployment, not about updating code dependencies in this repo. So to me that sounds rather unrelated to renovate.

@cgwalters are you planning on adding support for Cincinnati in bootc, or do you have an alternative in mind?
I think it'd be interesting to investigate ways of distributing the update metadata through OCI registries, instead of Cinci.

also ref https://github.com/coreos/fedora-coreos-tracker/issues/1263

@cgwalters are you planning on adding support for Cincinnati in bootc, or do you have an alternative in mind?

IMO no; that's exactly the kind of thing that can live as a higher level project.  A huge difference here vs e.g. FCOS is the emphasis (and raw ability) to trivially derive builds.  And one of those things that's now easy to inject is a custom tool to do OS updates.

This was fixed by https://github.com/containers/bootc/pull/179


User add/edit forms are missing form media

Issue Summary
Reported on the support slack channel: apparently form widgets with js/css media (such as AdminDateInput) don't work on the wagtailusers add/edit forms, because we aren't inserting the form media tags onto these templates.
There should be {{ form.media.js }} / {{ form.media.css }} tags, similar to the ones added here: https://github.com/wagtail/wagtail/commit/67795603fb4d80f4a82d54eeddc8d8ed55d4b8b4
Steps to Reproduce
(unconfirmed)

(for example) Start a new project with wagtail start myproject

Define a custom user model as per http://docs.wagtail.io/en/v2.4/advanced_topics/customisation/custom_user_models.html , with a date_joined date field

In the corresponding form, define this field to use AdminDateInput:
 from wagtail.admin.widgets import AdminDateInput

 date_joined = forms.DateField(
     widget=forms.AdminDateInput(),
     help_text="Date joined",
 )

Visit the 'add user' form - the date picker does not show

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: no

Fixed in #5146


Link to fzf-bin releases from fzf README?

There's no mention of "releases", "binary", or fzf-bin in the README.
I've put fzf up on webinstall.dev/fzf linking to the homepage, but if a person checks on the FZF homepage to verify that the builds are unmodified, 1st party builds directly from the official releases, as promised, they will likely go to the Releases tab and see that nothing is there, which would be confusing.
And then there's nothing in the main README to indicate that the official releases are available, but that they're in the fzf-bin repo.
I don't remember how I found out about them myself, but when I came back here looking for "releases" and "binary", I found nothing.

[ ] N/A
[ ] N/A
[x] I have searched through the existing issues

Info

OS - N/A

Problem / Steps to reproduce

The link in https://github.com/junegunn/fzf#installation points to the page.

Update: Since 0.24.0, new realeses will be available in https://github.com/junegunn/fzf/releases


Optimizing environments

We are currently running four instances of the COVID Alert server:

Sandbox
Staging
Demo
~Production~

They are currently all running the same database, compute, and memory configuration to ensure there is no drift between environments.
AC:

[x] Reduce the ECS environments for the staging environment
[x] Reduce the RDS environments for the staging environment
[x] Verify RDS Environment is db.t3.small for the staging environment
[x] Reduce the ECS environments for the demo environment.
[x] Reduce the RDS environments for the demo environment.

RDS Instances should change during next maintenance window:
Demo: 2020-11-30 Overnight
Staging: 2020-11-25 Overnight
I'll close this issue when I've verified that the DB instances downgraded successfully.

Only one of the 3 staging databases scaled down, going to manually scale down.

Didn't scale down Production as part of this task.
Will handle in a separate issue.

Confirmed Demo RDS is scaled down.


Event page: remote control 'back' button

The remote control 'back' button correctly sends the user back to the main navigation, but the whole screen is shifted to the left.

the estimate is about 1h(already completed)

done


.next directory not automatically created when doing production build.

Bug report
Describe the bug
I just upgraded to version 9.0.7 (from 9.0 I believe) and did a production build. It didn't create the .next directory which I had previously deleted and would only work if I manually created the .next directory.
As a bit of background, I got into the habit of deleting the .next directory between production builds as a simple way of removing all previous builds. Running a new build always automatically created the directory but now it seems not to recognise that the directory doesn't exist.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Delete your .next directory
npm run build
No built files can be found

Expected behavior
The built files should be in the .next directory which should have been recreated.
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: Win10
Version of Next.js: 9.0.7

Not sure what we can do to help then, maybe give support through enterprise support 🤔 feel free to email<EMAIL_ADDRESS>

Closing as no reproduction provided.


UI Shell Hamburger Menu: Needs Click Event + template addition

Detailed description

Describe in detail the issue you're having.
Would be nice if the Hamburger component supported a click() event handler.  Also, would be nice if the template of the hamburger component toggled between what it has now and the big blue x, as portrayed here:

Is this a feature request (new component, new icon), a bug, or a general issue?
Feature request

Is this issue related to a specific component?
Hamburger menu component that's part of UIShell component.

What did you expect to happen? What happened instead? What would you like to see changed?
I expected there to be a click handler such that users could toggle between the default, three-bar hamburger menu icon that's present now and the blue x shown in the attached screenshot.

What browser are you working in?
Chrome and Firefox.

What version of the Carbon Design System are you using?
V10

What offering/product do you work on? Any pressing ship or release dates we should be aware of?
Watson for Protocol Development (Watson Health).  We would like to have this available for a 3rd-quarter release.

Steps to reproduce the issue
You can see this behavior (or, rather, lack of behavior) in the storybook example (https://angular.carbondesignsystem.com/?path=/story/ui-shell--together).
Additional information

Screenshots or code
Notes

Poking through the demo and the scss it seems like they have styles for that ... Let me know if you're interested in tackling this, I'll put it in our Q2 project either way :+1:


automatic visualizations of memos in "comic book" format with nested thought bubbles

planning to pair-program with @manya-bansal

I made some progress on comic generation this evening, inspired by Manya's lab meeting talk!

https://github.com/kach/memo/blob/main/memo/comic.py


Outcome Indicator Component

Motivation:
Each client has a case and each case may have more than one charge against the client. The top level data about each charge will be represented in a "Row" of the top level data view. The row displays data like the name of the charge, whether it's a felony or a misdemeanor, but it should also have an "Indicator" which shows at-a-glance what the status of the row is.
Accepts:
chargeId (or some identifier): used to match on the specific charge that the indicator should listen to in the Context.
Renders:
An indicator, such as a circle which changes color and hovertext based on the outcome of evaluations. For example if context indicates that the charge has not been evaluated, show as grey, or if ineligible show as red.
Pushes to context:
???
Consumes from context:
State/outcome of the charge.

I'll take this one.


Found Bug 🐛: Auth and Cached decorators not working with Mounts

Describe the bug 🐛
I use a mount in my api, the code is defined in another file and then mounted. When i try to use either of the new auth decorators, or the new cached one, i get a weird error. It happens whether i'm using the decorator in the mount but not from the main server file.
To Reproduce 👨🔬
Steps to reproduce the behavior:

Create a basic ssr server
Add some code in another file (so you can mount it)
Try to use AuthBearerJWT, AuthJWT, or the Cached decorators on a route inside the mount file.
See error

Expected behavior 🤔
Mounts should work fine with the new decorators.
Screenshots 🖼

This is from the mount file.
Please complete the following information 📃

OS: WIN 11 PRO 23H2 + WSL 2 ( Ubuntu 22.04)
HappyX version 4.7.0
HappyX project type [ssr/spa]

HappyX project language = Nim
Nim version  = 2.2.0
HTTP Server Default (also fails on others too, tested on httpx and httpbeast)

@Beetroit can you provide some example of code?

main.nim file
import happyx
import test_case

serve "<IP_ADDRESS>", 5000:
    onException:
        echo e.msg
        echo e.name
        return {"error": getCurrentExceptionMsg()}

    notFound:
        return {"response": "Not found"}

    get "/":
        "Hello, World!"

    mount "/users" -> User

test_case.nim file
import happyx

mount User:
    get "/":
        return {"response": "Hello, World!"}

    @Cached(10)
    get "/testcache":
        return {"response": "Cache"}

nim.cfg
verbosity=0
hints=off
define = release
define = ssl
threads = on
deepcopy=on
define=beast

nope


Test - adding an empty file to trigger build

Testing https://github.com/CartoDB/cartodb-onpremise/issues/555 against a branch containing a trivial change

Can one of the admins verify this patch?

Works fine. Closing the PR without merging.

reopening to check PR hooks

It was not reopened as branch is deleted


Update bundler gem

We should stay up to date on this

This won't update the gem, you'd also need to update the lockfile also

I don't see a lockfile anywhere...

Ah, yeah, looks like we manually install bundler on CIs with a specific version: https://github.com/danger/danger/blob/master/.travis.yml#L16
So if this works that's fine during dev then that might be ok, but we had bundler 2.0 problems for a while

Circle passed, now just travis and appveyor

Travis is green

👍 - nice one, thanks!


Pre-built install error on maple self-update from v0.34 to v0.35

Environment (please complete the following information):

OS: Debian Bullseye
(Neo)Vim version: Vim 8.2
vim-clap version: v0.34->v0.35 plug update
Have you reproduced with a minimal vimrc: no, space-vim
Have you updated to the latest plugin version: yes, well trying to
Have you upgraded to/compiled the latest Rust binary: no this fails

Describe the bug
A clear and concise description of what the bug is.
vim-plug :PlugUpdate command results in:
x Post-update hook for vim-clap ... Vim(call):E631: channel_write_input(): write failed

EDIT: I think an edited out remark I had here earlier was from vim-lsp issue, not related.
From :Clap install-binary! command in:  Plug 'liuchengxu/vim-clap', { 'do': ':Clap install-binary!' }
Problem appears to be this line.
To Reproduce
Steps to reproduce the behavior:

Create the minimal vimrc min.vim:

set nocompatible
set runtimepath^=/path/to/vim-clap
syntax on
filetype plugin indent on

Start (neo)vim with command: vim -u min.vim

Reproducible with:

:echo term_start($HOME.'/.vim/plugged/vim-clap/bin/maple upgrade --do
wnload', {'curwin': 1, 'cwd': $HOME.'/.vim/plugged/vim-clap', 'exit_c
b': {job, status -> execute('echoe "v:shell_error="'.status)}})
:echo v:shell_error

Or:
:Clap install-binary!

Output of above is:
Retrieving the latest remote release info...
New maple release v0.35 is avaliable, downloading...
thread 'main' panicked at 'not currently running on a Tokio 0.2.x runtime.',
 /root/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.2.25/src/runt
ime/handle.rs:118:28
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrac
e
101

v:shell_error is 101
With let $RUST_BACKTRACE=1:
Retrieving the latest remote release info...
New maple release v0.35 is avaliable, downloading...
thread 'main' panicked at 'not currently running on a Tokio 0.2.x runtime.',
 /root/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.2.25/src/runt
ime/handle.rs:118:28
stack backtrace:
   0: rust_begin_unwind
             at /rustc/9f4dc0b4db892271cd0dada6e072775b5b5d6b1e/library/std/
src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/9f4dc0b4db892271cd0dada6e072775b5b5d6b1e/library/core
/src/panicking.rs:143:14
   2: core::panicking::panic_display
             at /rustc/9f4dc0b4db892271cd0dada6e072775b5b5d6b1e/library/core
/src/panicking.rs:72:5
   3: core::panicking::panic_str
             at /rustc/9f4dc0b4db892271cd0dada6e072775b5b5d6b1e/library/core
/src/panicking.rs:56:5
   4: core::option::expect_failed
             at /rustc/9f4dc0b4db892271cd0dada6e072775b5b5d6b1e/library/core
/src/option.rs:1874:5
   5: <core::future::from_generator::GenFuture<T> as core::future::future::F
uture>::poll
   6: <core::future::from_generator::GenFuture<T> as core::future::future::F
uture>::poll
   7: <core::future::from_generator::GenFuture<T> as core::future::future::F
uture>::poll
   8: <core::future::from_generator::GenFuture<T> as core::future::future::F
uture>::poll
   9: <futures_util::future::try_future::try_flatten::TryFlatten<Fut,<Fut as
 futures_core::future::TryFuture>::Ok> as core::future::future::Future>::pol
l
  10: <hyper::common::lazy::Lazy<F,R> as core::future::future::Future>::poll
  11: <futures_util::future::try_future::try_flatten::TryFlatten<Fut,<Fut as
 futures_core::future::TryFuture>::Ok> as core::future::future::Future>::pol
l
  12: <futures_util::future::poll_fn::PollFn<F> as core::future::future::Fut
ure>::poll
  13: <reqwest::async_impl::client::PendingRequest as core::future::future::
Future>::poll
  14: maple::main::{{closure}}
  15: maple::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose
 backtrace.

Should be fixed by #848, and the tests are enabled to ensure it won't break silently again.


Update troubleshoot-aad-connect-objects-attributes.md

Removing legacy information requesting customers to open support case to manage dirsyncoverrides attributes. New feature is available to customer self-service.

Reviewed and approved change.


Logotype

We already have the logo symbol but do not have a logotype to go with it. I would think this would be a requirement for a website. Would love to track this here and collectively look at type specimens and usage.

First comp from today, just basic Helvetica Neue

Yes I agree with you, great point.
@ai do we have any specific styleguide to go after when it comes to font, colors etc?

Yes I agree with you, great point.
@ai do we have any specific styleguide to go after when it comes to font, colors etc?

@stephenway looks like very nice. Maybe we should cappitalize “CSS” to? Like PostCSS?

@stephenway looks like very nice. Maybe we should cappitalize “CSS” to? Like PostCSS?

Styleguide is simple: Middle Ages and alchemy. But, of course, we should not use medieval font. Just few references.

Styleguide is simple: Middle Ages and alchemy. But, of course, we should not use medieval font. Just few references.

What about using a typo like the effect on the logo:

@MoOx That idea would depend on if the logo symbol was intended to have an "o" inside it. Mixing the symbol into the logotype makes it hard to separate the two.

Yeah sure, that was just an idea ^^

haha, that's funny.
@ai Great, maybe we can form an styleguide after the design is completed as well and double check everything so it's according to what you had in mind.

Yea style guide would be a great for adding to the website and improving communication with the community.

Yeah indded, It would be cool doing something similar to lonleyplanet or maybe we can use postcss-styleguide :wink:

Maybe this would be even more appropriate https://github.com/sapegin/react-styleguidist (and logical)

I've used that and ended up migrating to Jon's mdcss, absolutely happy with it. Comes with lots of options.

That looks neat, we should definitely try that one out.

Maybe we should make a new issue for this? Agreed @MoOx

Absolutely, that's my job to do :sweat_smile:

Anyway, logo looks great to me but as @ai pointed out maybe make the CSS text a little big bigger and try that with the same red color if you mind @stephenway.

Yea I want to post some more options asap

PostCSS sign is not good for “o”. Because it can be read as “A” to because of triangle.

This is an overview of the first type exploration from today, I've bordered 2 I think work nicely when paired with the logo symbol.

The third lgtm :)

@MoOx  That's using Google's Poppins font.

4th in right is a Cyrillic reference? :D
http://i.kinja-img.com/gawker-media/image/upload/s--GSNBcS9r--/c_scale,fl_progressive,q_80,w_800/18j1kyfaz7pb7jpg.jpg

Great work, the third or seventh in my opinion.

@ai Cyrillic, yes, kinda failed on getting a good "T", but I think it's readable. Maybe need to substitute with a "T" from another font?

but on the other hand I agree with the choice of number duo:

Styleguide is simple: Middle Ages and alchemy.

 There is a very strange medieval way to write a letters.
https://s-media-cache-ak0.pinimg.com/originals/40/84/db/4084dbadc95d6f95bbe07e0e330c752f.gif
http://eldesign.ru/upload/Image/sreda/2007-05-23/letter_r08.gif
https://s-media-cache-ak0.pinimg.com/originals/6b/b4/dc/6bb4dc58d61fc8dc7f7e182368dda295.gif
The idea is to combine symbols in calligraphy pattern, like Arabic people do.
But it is hard to do, only if @stephenway really want to make some culture investigations.
I can help you, I often use this way in henna painting:
https://pp.vk.me/c314431/v314431146/4d05/bSv3XtzICu4.jpg
https://pp.vk.me/c621221/v621221146/246aa/HM9skZNRTtU.jpg
https://pp.vk.me/c623429/v623429146/1de5f/FOZ8SJtnI_w.jpg

@stephenway you can find more by “orthodox calligraphy” or “Vyaz”
http://www.christianforums.com/proxy.php?image=http%3A%2F%2Fmedia-cache-ak0.pinimg.com%2F736x%2F74%2F47%2Fd4%2F7447d42fe887d0a99a360013db21debf.jpg&hash=b4b90ec56d2cb118c9d713f195866376
https://s-media-cache-ak0.pinimg.com/236x/fa/13/e5/fa13e5eadb29f7d7c1047ed0e8dad538.jpg

@ai Those examples are really cool, is that your own henna? I can try drawing some of that for round 2, but I will make sure that we come up with something that is easy to read too.

So far it sounds like we are looking at these options, plus more calligraphic options that I will get going soon for round 2.

@shinnn nope-nope, pseudo-cyrillic case we can remove. Cyrillic has own special philosophy and mirroring is not a right way :).

I amd OK with first. Separating “Post” and “CSS” is a good point of view.

@stephenway yeap, it is my henna, but it is not my paintings. I just copy them from some medieval book. It will be cool, if we try this Vyaz way.  This PostCSS site project should be a hobby project to test new ideas for us. We have a lot of task in our job to make a common ordinary solution.

Discussion will continue in postcss/brand#3

To all that subscribed and follow this thread. Follow the brand link for our round 2 options posted today.


Rework the Auth0 Guide

Based on feedback from multiple users, for example: https://elixirforum.com/t/onboarding-feedback-the-good-the-bad-and-the-ugly/58527/6?u=zachdaniel
Ultimately what should be done is that we should rework the guide to assume that the user has not already set up ash_authentication, and then highlight some areas that might be different if they already have set it up for password authentication, for example.

I am running into what feels like a potential dead-end after following the current guide.
I do not think this is an actual issue with the code, just with the guide, so I'm not opening a separate issue.
After following the guide, my app is vulnerable to account takeover, here are the steps to reproduce:

Enable Gmail and Email/Password connectors in Auth0
Sign up with Gmail, now my app considers the user to be<EMAIL_ADDRESS>In an incognito tab, sign up with Username/Password and choose the same<EMAIL_ADDRESS>Do not verify the email address
AshAuthentication considers me to be the same user as the one from Gmail, even though the accounts were not linked in Auth0.

What I think we might need to do
The user_info that comes from Auth0 includes a sub, these are not the same for the two users.
"sub" => "google-oauth2|redacted-numbers"
"sub" => "auth0|redacted-numbers-and-letters"
If instead of:
  identities do
    identity :unique_email, [:email]
  end

We made the identity :unique_auth_id, [:auth_id] or maybe just identity :unique_sub, [:sub]
Then the different Auth0 accounts would be considered separate.
What I do not know but I fell about 83% sure
If they are linked, will the sub change? I think they will not. So if an Ash user wants to build user linking after following a guide that caused the Ash user to be correlated to the Auth0 user by sub, they will be able to do that. I would love for someone with more Auth0 experience to weigh in.

Yeah, this seems like an oversight on my part. We do have the UserIdentity extension and resource which we can use to track a user's multiple identities, but it all still comes down to whatever we decide to use for the unique identity for the user.  At the very least I think we should update the guides to explain how this could happen in the case where there are multiple strategies enabled.

No worries, it already felt like magic when I followed a guide, got auth working, and I have the absolute minimum amount of code to maintain. The UserIdentity extension looks cool.
One more reason to use the auth0_id is that if someone changes their email address, using the email as unique identity would mean that we need to change the email in our database and on Auth0 at the same time, if the email was only changed in Auth0, we would end up creating a new user. If we use the auth0_id, the email just needs to be changed in Auth0 and then it automatically gets changed in our app on the next login.
What do you think of a PR to the guide that changes the action to:
    create :register_with_auth0 do
      argument :user_info, :map, allow_nil?: false
      argument :oauth_tokens, :map, allow_nil?: false
      upsert? true
      upsert_identity :unique_auth0_id

      # Required if you have token generation enabled.
      change AshAuthentication.GenerateTokenChange

      # Required if you have the `identity_resource` configuration enabled.
      change AshAuthentication.Strategy.OAuth2.IdentityChange

      change fn changeset, _ ->
        user_info = Ash.Changeset.get_argument(changeset, :user_info)

        changes = %{
          "email" => Map.get(user_info, "email"),
          "auth0_id" => Map.get(user_info, "sub")
        }

        Ash.Changeset.change_attributes(
          changeset,
          changes
        )
      end

I would also wan to incorporate whatever I learn from here

Absolutely.


COMPRESS-499: Fixed edge case in SeekableInMemoryByteChannel#truncate

Initial proposed patch.
It's unknown whether the bug has any actual impact because the edge case is very specific and honestly is a misuse-case in my book.
Test cases cover what we deemed to be all possible size vs new size vs position relationships.

Coverage increased (+0.1%) to 86.903% when pulling c69c93ee8c07bf817ce1957768b3fb067223f527 on trejkaz:compress-499-fix into 26c7aff58aa4cdf4943cf3ddb744f680a1fbe007 on apache:master.

As I said on JIRA, I think we should wait a while until we have figure out if SeekableInMemoryByteChannel is doing this intentionally or not.
I would say it may be better that we should be patient and have a discussion in mailing list before the code is merged.
Besides, I think it's better to have a modification on SeekableInMemoryByteChannel.repositionIfNecessary if this is identified as a problem. The problem is about the position in SeekableInMemoryByteChannel, not the size. SeekableInMemoryByteChannel.repositionIfNecessary is the method that actually changes the position, and it's also called in SeekableInMemoryByteChannel.read.
And I think a double check on other classes that implements SeekableByteChannel would be a better software practice - we shoud check other related potential problems - but not too rush, I believe a discussion before it may be better.
Hope this helps :)

My reading of the spec is that the current behavior for size was correct but not for position. I've added a bunch of tests and think I've fixed the issue with af63df69


Update project dependencies (#7151)

This closes #7151

https://github.com/ysmoradi/bitplatform/actions/runs/8297886080/job/22709968135
@msynk


Failed to connect if password contains a '?'

It is not possible if the password for the private key contains a '?' character.

Fixed by #12


dotNetFringe reflections

I'm excited to be here. Figured I should probably write about it. General notes / outlines below.
Pre-Conference -- Saturday

The idea of this conference is important.
The energy around it caught my attention.
This feels like a rebirth, and I want to be a part of it.
I feel like this was a good choice for my first professional conference
So many excellent people to learn from in one place
Smaller venue seems easier to navigate
Not focusing on mainstream MS stuff, which I can read/watch about in many places
I "grew up" as a developer on these folks and they've helped me immensely.
As I was learning to develop I was simultaneously learning that there's a lot more to it
Also a little intimidating -- these folks are generally speaking much farther along career/development wise than I am. But I'm looking forward to soaking in a lot of knowledge.
Looking forward to seeing what people are like when not on a stage (which is the only place I've seen them ever)
Meeting an entire conference full of new people feels strange. I'm a social being but not having a strong connection to anyone in the group to start will be interesting. Luckily, the group in general seems very friendly so I feel a little emboldened by that (not that I've ever been shy.)
Delta airlines. Woof. Delay then un-delay then delay then sprinting for a connection then luggage not making it.
I'm grateful to my company for allowing me to go to this conference. A lot of folks take that for granted but this is the first time I've been able to dig in like this.
There's a difficult balance between being excited and wanting to be a part of things vs. trying too hard. I thought I mastered that but times like these make me wonder. Never know how to apply that enthusiasm to a group of people that already know each other.
It feels like I'm back at the kids table again. And I don't necessarily mind that, it's just strange.

Expired; closing this.


Add a refresh method to the RouterService

This adds a refresh method to the RouterService as described in RFC 631
RFC PR: https://github.com/emberjs/rfcs/pull/631

@Windvis Possible we can upstream this to Ember.js now?

@snewcomer Hey! Can you elaborate on what you mean with "upstream this to Ember.js"? :sweat_smile:
If you mean moving the implementation into Ember itself that is already the case and the feature is enabled in 4.1.0-beta.1. If you mean moving this polyfill to the ember-polyfills org, I've asked around in the Discord already but it seems that everyone is busy :smile:.

Oh thank you!  I may have been looking at 3.28 and should have been looking at the master branch.