Use hypothesis testing framework

As a follow-up on #4. Can we use the hypothesis framework in testing?
What I'm going for is much more diverse data.
All help appreciated!
https://hypothesis.readthedocs.io/en/latest/

I'm going to look into it.


Add favicon and Shorter title to the website

PLEASE ASSIGN ME THIS ISSUE

Please assign it to me

which website
On Sun, Oct 30, 2022 at 9:06 PM Avinash Singh @.***>
wrote:

Assigned #214
https://github.com/avinash201199/Portfolio-Collection/issues/214 to
@riddhitech https://github.com/riddhitech.
—
Reply to this email directly, view it on GitHub
https://github.com/avinash201199/Portfolio-Collection/issues/214#event-7699295425,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVYVZUIUIDSNWPCRP4D7RQDWF2IX3ANCNFSM6AAAAAARMMYAM4
.
You are receiving this because you were assigned.Message ID:
<avinash201199/Portfolio-Collection/issue/214/issue_event/7699295425@
github.com>


[BUG] Text overflowing

🙁 Actual behavior

The Verse of the day container on the home screen is not expandable.

The About us page is not enclosed with the safe area, i.e., It flows into the status bar.

🙂 Expected behavior

Either the container must expand if we have a big verse, or the font size must reduce to adjust itself inside the container.
Enclose the about us page in the safe area.

🔢 Steps to Reproduce the Problem

Install and visit the home page and scroll the about page.

I would like to fix this myself.

Thanks for identifying this bug Ravinder. That's very helpful since there are like 3500 active users and a lot of them might be facing these errors.

Please assign me this problem

Hi, I would like to work on this issue.


Wrenformer

This PR adds a new variant of the Wren model called Wrenformer, a rewrite that drops the custom self-attention in favor of builtin PyTorch TransformerEncoder. It also prepares Matbench submissions for Roost and Wren (structure tasks only for Wren).
The Wren rewrite as a transformer encoder was necessary to run robustly across Matbench tasks as the original Wren would run into out-of-memory errors during training and inference on materials with large numbers of Wyckoff positions (< 16 was a safe cutoff). This happened even on A100 GPUs with 80 GB RAM.
Initial performance testing (training for 100 epochs and with only 3 transformer layers) suggests Wrenformer slightly beats Wren across all Matbench tasks I recorded for both.

More sets of hyperparameters

Speed difference between Wren and Wrenformer
According to @CompRhys, Wren could run 500 epochs in 5.5 h on a P100 training on 120k samples of MP data (similar to the matbench_mp_e_form dataset with 132k samples). Wrenformer only managed 207 epochs in 4h on the more powerful A100 training on matbench_mp_e_form. However, to avoid out-of-memory issues, Rhys constrained Wren to only run on systems with <= 16 Wyckoff positions. The code below shows that this lightens the workload by a factor of about 7.5, likely explaining the apparent slowdown in Wrenformer.
import pandas as pd
from aviary.wren.utils import count_wyks
from examples.mat_bench import DATA_PATHS

df = pd.read_json(DATA_PATHS["matbench_mp_e_form"])

df["n_wyckoff"] = df.wyckoff.map(count_wyks)

sum_wyckoffs_sqr = (df.n_wyckoff**2).sum()
sum_wyckoffs_lte_16_sqr = (df.query("n_wyckoff <= 16").n_wyckoff ** 2).sum()
print(f"{sum_wyckoffs_sqr=}")
print(f"{sum_wyckoffs_lte_16_sqr=}")
print(f"{sum_wyckoffs_sqr/sum_wyckoffs_lte_16_sqr=:.3}")
# prints 7.45, so Wrenformer has to do 7.45x more work, explaining the about 2x slow down
# on a more powerful GPU (Nvidia A100 vs Wren on a P100)

All looks good - I am not sure the matbench stuff should live here forever but for the time being it's the best place. Please change nn to something else I think we said networks


updated paw (3.0.6)

==> Verifying checksum for Cask paw
audit for paw: warning
 - appcast checkpoint mismatch
Expected: f622ac241ca4816e25a624da398e8b2c711b803c228cad89dfeb75eb998c0185
Actual: 28b6c54d24ff6c4249d2a94e891e4099b7aa7d1b547cec3d2314fba145490d9b
Error: audit failed for 1 cask: paw

There was a pull request template when you submitted. It must be filled.
If you were not presented a template, you may have submitted via a CLI tool (likely hub). It that was the case, you may wish to see if the tool has a way to let you use the template (hub is working on it).
In either case, please refer to the template and update your comment.

Thank you for the contribution. It needed some fixes, so they were made in 88e732ef0fdc579fe82811f2c68c4059c50bdfc2. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


chore: Ruby generate list class

Fixes
Template to generate List Class for ruby has been added
Checklist

[x] I acknowledge that all my contributions will be made under the project's license
[ ] Run make test-docker
[ ] Verify affected language:

[ ] Generate twilio-go from our OpenAPI specification using the build_twilio_go.py using python examples/build_twilio_go.py path/to/twilio-oai/spec/yaml path/to/twilio-go and inspect the diff
[ ] Run make test in twilio-go
[ ] Create a pull request in twilio-go
[ ] Provide a link below to the pull request

[ ] I have made a material change to the repo (functionality, testing, spelling, grammar)
[ ] I have read the Contribution Guidelines and my PR follows them
[ ] I have titled the PR appropriately
[ ] I have updated my branch with the main branch
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added the necessary documentation about the functionality in the appropriate .md file
[ ] I have added inline documentation to the code I modified

If you have questions, please create a GitHub Issue in this repository.

@rohith-prakash This is breaking the node examples tests.


how to connect the cabot to graphite running in docker

hi, I'm trying to connect the cabot to fetch the metrics from graphite which is running in docker localhost. but i'm not sure on which port the cabot should connect to the graphite. i try 80, 2003, 8125, 8126 all not working. how do i connect it?
In the check it shows error like this:
Error getting data from Graphite: HTTPConnectionPool(host='localhost', port=8126): Max retries exceeded with url: /render?from=-2minute&target=api-hit&format=json (Caused by NewConnectionError(': Failed to establish a new connection: [Errno 111] Connection refused',))
how i install my graphite:
docker run -d
--name graphite
--restart=always
-p 80:80
-p 2003-2004:2003-2004
-p 2023-2024:2023-2024
-p 8125:8125/udp
-p 8126:8126
graphiteapp/graphite-statsd

I don't think this sounds like a Cabot issue so will close.
First you need to work out if you can access the Graphite instance just from the command line of your host (e.g. curl http://localhost:8126/...). I'm not a Docker expert but on most environments, localhost won't work. (Docker for Mac does though).
Second, you need to connect Cabot (you don't say whether you are running Cabot via Docker or outside of a container, but I'm guessing probably the former?) to the container. This can be pretty simple or pretty complex: https://docs.docker.com/network/bridge/

@dbuxton I can connect my grafana into my graphite. i try to to access the graphite api from my browser/postman and curl as well. i'm using docker on mac to run the cabot. i don't have any authorization username/password to access the graphite tho, so i'm not sure what to fill on production.env for GRAPHITE_USER and GRAPHITE_PASS, i just leave it empty. http check from my cabot to google.com is passing, but when i try to connect to localhost it always fail

oh i can access it by setting the url as docker.for.mac.localhost
thanks!


max recipients is now optionally set-able by caller; 7 by default

not sure why 255 is the magic number for maximum possible recipients. the number is quite reminiscent of 256 / 8*32 ....

can you make maxrecipients camel case? or, you can make that a one liner with max = max || 7
The maximum value for max is 255 because it's stored in a single byte. A way you could extend this, is you could estimate the maximum recipient size from the size of the message, since messages over 8k could contain more than 255 recipients.
However... that was never really the intended use of private-box, and the code is very very simple, so I'd wouldn't recommend using private box for messages with a large number of recipients.

we can throw an error when someone tries to specify more than 255 recipients for now

got it - fixed

thanks, merged into 0.2.0!


x/mobile: Executing an example fails on Windows

[~/go/src/golang.org/x/mobile/example/basic]
$ go run main.go
# golang.org/x/mobile/app
..\..\app\shiny.go:42: w.Events undefined (type screen.Window has no field or method Events)

Duplicate of https://github.com/golang/go/issues/14401.


ci: Use reusable workflows for code quality tests

Factors out basic CI infra into reusable workflows and composite actions contained in https://github.com/lurk-lab/ci-workflows.
Also switches the link checker to fail fast and fixes the associated errors which were previously silent.
Future work:

GPU benchmarks and merge tests
Replicate in arecibo

we've switched to dtolnay/rust-toolchain across the board,
we've pushed in https://github.com/lurk-lab/arecibo/pull/238 a workflow that proves it can generate issues on regression ( https://github.com/lurk-lab/arecibo/issues/248 )

We could take this occasion to 100% solve the UX of https://github.com/lurk-lab/lurk-rs/issues/1028 here


[FR] Chinese adaptation

English is normal, but Chinese is not blurred

Same for Korean.
English is normal, but Korean is not blurred.

This effect is achieved by using fonts "Flow Circular". This font supports for 23 languages insdead of Chinese and Korean.


Explorer: Load pods not in non-active namespaces

As @itowlson had noted, selecting a Pod in a non-default namespace from the Node tree in the explorer won't load the resource in VSCode. This is due to the way the Load command works (by using the current active namespace). This should be fixed such that the Load command is aware of the namespace resources are in, so it can load them.

Sorry about my ignorance :(  @bnookala   can you detail more steps to reproduce this ? am not seeing this behaviour. One of the thing I observe in my setup which runs with OpenShift, I need to login to get access to needed namespace resources, otherwise the the extension shows error message that the user is not authorised.

The issue refers to PR #252.  This PR added a view of the pods running on a node; however those pods might be in namespaces other than the active namespace, and when you selected a pod that was in such a namespace, loading its YAML failed or misbehaved.  So you won't see it in the normal pod list (because that is scoped to the namespace), only if you drill down from a node.
The OpenShift login issue sounds worth exploring - do you want to raise a separate issue for that?  I'm not sure it would belong in the base k8s extension but it would be good to understand in case a hypothetical future OpenShift extension needed an API to hook for it.  Thanks!


Remove env cache in daily CI

Fixes # .
Checklist
Any checks which are not relevant to the PR can be pre-checked by the PR creator.
All others should be checked by the reviewer(s).
You can add extra checklist items here if required by the PR.

[ ] CHANGELOG updated
[ ] Memory profiling tests pass (see here for more details)
[ ] Tests added to cover contribution
[ ] Documentation updated

/memory_profiler

Codecov Report

Merging #4 (d083933) into main (e95df56) will increase coverage by 0.01%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   86.83%   86.85%   +0.01%
==========================================
  Files          49       49
  Lines        5468     5468
  Branches     1370     1370
==========================================
+ Hits         4748     4749       +1
  Misses        460      460
+ Partials      260      259       -1

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[VisualShader] Remove invalid graph connections when ports are removed

Supercedes #80312. (partial salvage, the undo/redo stuff had to be redone)
Fixes #79417.

@Geometror There are a couple code style/quality comments to address, then it should be good to merge.

Thanks!


App crash unexpectedly while it's opening

Description
The app closes unexpectedly while it's opening
Steps To Reproduce
open the app
Requirements
no requirements
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3085
Log File
debug.log

Thanks for the report, can you see if this is an issue in the preview version. https://files.community/download


Finish Huzaifa's PPP

Codecov Report
Base: 40.01% // Head: 40.01% // No change to project coverage :thumbsup:

Coverage data is based on head (d87d9df) compared to base (d759384).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #176   +/-   ##
=========================================
  Coverage     40.01%   40.01%
  Complexity      471      471
=========================================
  Files           122      122
  Lines          3149     3149
  Branches        313      313
=========================================
  Hits           1260     1260
  Misses         1781     1781
  Partials        108      108

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Modify card struct to reflect the game rules

Description
Currently the card struct has some fields but they no longer reflect the game rules. Each card should have dribble/defense stats as well as a role. It should also have a field to differentiate the captain from the others.  The captain should have +1/+1.
Acceptance criteria

The cards have all the attributes

I can take this ser

I can handle this too if needed

this one is already taken, can you pick another one plz @WhoIsNac


added test for accelerator Profile api

Closes: https://issues.redhat.com/browse/RHOAIENG-2004
Description
added unit test case for frontend/src/api/k8s/acceleratorProfiles.ts

How Has This Been Tested?
npm run test

Test Impact
test coverage:

Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/retest

/lgtm
/approve


py2.7 -> py3.7

Hi
I have verified that the py2.7 fileEvo_Thermal_sample.py runs fine with my hardweare but I am running a python 3.7 stack on an RPi. I kindly request that you publish a py3.7 branch.
Thanks

Hello @robmarkcole thanks for raising the issue,
Migrating to python3 is in our backlog but is not a priority for the moment so no promises ;) However, we will try to review and merge related pull requests whenever possible.
Cheers

Thanks for the info, I will close this 'issue' :)


Owns resources for pipelines

…tracepipeline

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending?


[BUG]:  AzureResourceManagerTemplateDeployment@3 Login failure: AzureUsGovernment Cloud Login

Task name
AzureResourceManagerTemplateDeployment@3
Task version
3.225.3
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[X] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Ubuntu 20.04
Task log
2023-08-09T16:28:01.7086768Z ##[section]Starting: Bicep Template deployment:
2023-08-09T16:28:01.7093154Z ==============================================================================
2023-08-09T16:28:01.7093295Z Task         : ARM template deployment
2023-08-09T16:28:01.7093375Z Description  : Deploy an Azure Resource Manager (ARM) template to all the deployment scopes
2023-08-09T16:28:01.7093499Z Version      : 3.225.3
2023-08-09T16:28:01.7093570Z Author       : Microsoft Corporation
2023-08-09T16:28:01.7093648Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/deploy/azure-resource-group-deployment
2023-08-09T16:28:01.7093782Z ==============================================================================
2023-08-09T16:28:06.3497591Z ARM Service Connection deployment scope - Subscription
2023-08-09T16:28:07.3889849Z Checking if the following resource group exists: redacted.
2023-08-09T16:28:07.3891634Z Resource group exists: true.
2023-08-09T16:28:07.3894814Z Creating deployment parameters.
2023-08-09T16:28:07.3908974Z [command]/usr/bin/az login --service-principal -u *** --password=*** --tenant redacted --allow-no-subscriptions
2023-08-09T16:28:07.3909260Z [
2023-08-09T16:28:07.3909379Z   {
2023-08-09T16:28:07.3909503Z     "cloudName": "AzureCloud",
2023-08-09T16:28:07.3909757Z     "id": "redacted",
2023-08-09T16:28:07.3909909Z     "isDefault": true,
2023-08-09T16:28:07.3910064Z     "name": "N/A(tenant level account)",
2023-08-09T16:28:07.3910213Z     "state": "Enabled",
2023-08-09T16:28:07.3910673Z     "tenantId": "redacted",
2023-08-09T16:28:07.3910831Z     "user": {
2023-08-09T16:28:07.3911088Z       "name": "***",
2023-08-09T16:28:07.3911225Z       "type": "servicePrincipal"
2023-08-09T16:28:07.3911357Z     }
2023-08-09T16:28:07.3911464Z   }
2023-08-09T16:28:07.3911579Z ]
2023-08-09T16:28:07.3915393Z [command]/usr/bin/az account set --subscription redacted
2023-08-09T16:28:07.5533544Z ERROR: The subscription of 'redacted' doesn't exist in cloud 'AzureCloud'.
2023-08-09T16:28:07.5626066Z ##[error]Error Code: [1]
2023-08-09T16:28:07.5645425Z ##[error]Error: Error in setting up subscription: %s
2023-08-09T16:28:07.5646359Z ##[error]Check out the troubleshooting guide to see if your issue is addressed: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/deploy/azure-resource-group-deployment?view=azure-devops#troubleshooting
2023-08-09T16:28:07.5647135Z ##[error][object Object]
2023-08-09T16:28:07.5708143Z ##[section]Finishing: Bicep Template deployment:

Relevant log output
2023-08-09T16:28:07.3915393Z [command]/usr/bin/az account set --subscription redacted
2023-08-09T16:28:07.5533544Z ERROR: The subscription of 'redacted' doesn't exist in cloud 'AzureCloud'.
2023-08-09T16:28:07.5626066Z ##[error]Error Code: [1]
2023-08-09T16:28:07.5645425Z ##[error]Error: Error in setting up subscription: %s
2023-08-09T16:28:07.5646359Z ##[error]Check out the troubleshooting guide to see if your issue is addressed: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/deploy/azure-resource-group-deployment?view=azure-devops#troubleshooting
2023-08-09T16:28:07.5647135Z ##[error][object Object]

Aditional info
Fails as the Cloud needs to be set to AzureUsGovernment.
No call to set cloud with setAzureCloudBasedOnServiceEndpoint in task operation AzureResourceManagerTemplateDeploymentV3/operations/Utils.ts

Preceding steps using AzureCLI@2 v2.225.0 have the below logs setting active cloud based on sp
2023-08-09T16:27:39.5027980Z Setting active cloud to: AzureUSGovernment
2023-08-09T16:27:39.5125100Z [command]/usr/bin/az cloud set -n AzureUSGovernment
2023-08-09T16:27:41.8124670Z WARNING: Switched active cloud to 'AzureUSGovernment'.
2023-08-09T16:27:41.8134101Z WARNING: Use 'az login' to log in to this cloud.
2023-08-09T16:27:41.8134839Z WARNING: Use 'az account set' to set the active subscription.

I am seeing the same error  with Microsoft Hosted

Not sure if it is the same issue.
After the Task was executed with '3.220.5' (last run with version '3.224.1' was succesful) it failed with
ERROR: AADSTS500011: The resource principal named https://management.core.windows.net/ was not found in the tenant named  ...china.... This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant. You might have sent your authentication request to the wrong tenant. Trace ID: 2368dfb3-1cdf-491b-87ac-6bac3003ba00 Correlation ID: c10b351a-823f-4baf-ab94-da12accd71c1 Timestamp: 2023-08-15 08:18:48Z
The Deployment was done against azure China region cne2.

We can confirm this issue, we have several reports that this task is not working anymore. I'm wondering why this is not fixed yet.
As customer I expect that these tasks are well tested before they are published and used by Azure DevOps.
I have the feeling that Microsoft lost interest in Azure DevOps as more and more issues are popping up and it seems nobody cares.
For us Azure DevOps is a critical service and we expect a GA ready system.

We have exactly the same issue as @vondralbra when trying to deploy resources to a China tenant.
Is it possible to use a specific version of the AzureResourceManagerTemplateDeployment@3, because it used to work until recently?

Hi everyone, task version 3.227.1 which has a potential fix must be deployed by now, can anyone run the pipeline and confirm the results here please?
Thanks

Hi everyone, task version 3.227.1 which has a potential fix must be deployed by now, can anyone run the pipeline and confirm the results here please?
Thanks

Works. Deployment to china (cne2) works again.

Hi everyone, task version 3.227.1 which has a potential fix must be deployed by now, can anyone run the pipeline and confirm the results here please?
Thanks

Works. Deployment to china (cne2) works again.

Thanks for your response, can you please confirm the task version of this success deployment?
Thanks

3.227.1

Yes, it was 3.227.1

closing as the last 3.227.1 version fixed this issue.


find command not available

It looks like it is set here?
https://github.com/home-assistant/hassos/blob/3c21a8c486870c8da14e09fa331fdae7679df4eb/buildroot/package/busybox/busybox.config#L432
But it is not available when connected to the host.

There are two issues here:

When you login via hassos-cli, it does not run ash as login shell, so /etc/profile will not be read and /usr/sbin is added to the $PATH.
The actual busybox config is set from buildroot-external/busybox.config and does not enable the find command.

@jvanderneutstulen we should fix the path on the hassos-cli script before we map the host shell


⚠ ClamAV has degraded performance

In 9d9d1b6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 298 ms

Resolved: ClamAV performance has improved in d5cea35.


Avoid return outside of function

In https://github.com/babel/ruby-babel-transpiler/pull/141, looks like theres a new failure trying to compile return [0, 2, 4].map(v => v + 1). New Babel doesn't like unknown: 'return' outside of function. Seems legit. We can just change our test to avoid this, it wasn't a detail of what we were trying to test away. This should still work on old Babels.
/cc @tricknotes

:heart:


Embedded scripts url?

Hi,
I'm using this configuration:
gulp.task('webserver', function() {
    gulp.src('app')
        .pipe(webserver({
            livereload: true,
            port: 12345,
            directoryListing: false,
            open: true,
            fallback: 'index.html'
        }));
});

The werbserver is working, but I need to add some js files to my index.html, and the scripts live outside the app folder (e.g. bower_components, dist).
How can I add these files? E.g:
<script src="/bower_components/angular/angular.min.js"></script>
Thanks

Does anybody know about this configuration?

You can't. You have to start the webserver from the root of your project, and go from there: Navigate your browser to localhost:8000/app and src="../bower_components/...".

Thank you, this helped a lot, perfect!


fix: adding support to point to api gateway distribution

Issue #, if available:
Found an issue where we don't support using the distribution supported by apigateway for custom domains.
Description of changes:
Description of how you validated changes:
validated end to end
Checklist:

[x] Write/update tests
[x] make pr passes
[x] Update documentation
[x] Verify transformed template deploys and application functions as expected
[ ] Add/update example to examples/2016-10-31

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1230 into develop will increase coverage by 0.05%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1230      +/-   ##
===========================================
+ Coverage    94.43%   94.48%   +0.05%
===========================================
  Files           72       72
  Lines         3736     3735       -1
  Branches       739      739
===========================================
+ Hits          3528     3529       +1
+ Misses         105      104       -1
+ Partials       103      102       -1

Impacted Files
Coverage Δ

samtranslator/model/apigateway.py
97.89% <ø> (-0.02%)
:arrow_down:

samtranslator/model/api/api_generator.py
95.22% <100%> (+0.56%)
:arrow_up:

Continue to review full report at Codecov.

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


Check if .md files are passed in patterns

Changes
Which issue is resolved by this Pull Request:
Resolves #903
Description of your changes:
Add a check before git clone.

@aartij22 Btw this was closed due to a GitHub bug having to do with the open-sourcing, no action was taken on my part, feel free to reopen


Components marked with ***

Hi Willem,
Congratz with the development of this great library.
I'm just having a question...
I suppose the components marked with  *** on https://wffranco.github.io/vue-strap/ are the ones you're still working on ?
Thanks in advance,
Kind regards,
Johnny

I mark that components that way to know that components are not full ready... they need some fixes (affix is not working with bottom), or not working (like typeahead), or have bugs.
I was working on it constantly some months ago, but now I have a lot of work so I fix some of them when I have time. I appreciate any help if somebody want to collaborate.


[META] Switch to 4.14 kernel

This script tracks my idea to switch to a 4.14 based kernel and related issues.
Switching to a 4.14 based kerne likely fixes #1

Depends on:

#80
#81
#82

@Thaodan : so, do you propose to switch with AOSP9 base to 4.14? I will have to ask from Sony's devs how future-proof it is and whether new blobs may support it.

If I understand it correctly it was tested by Sony devs?

I don't know. Will have to ask

May require also switch to clang: https://github.com/sonyxperiadev/kernel-sony-msm-4.14-common/commit/39779cef3700941436d7d8c448186236dff11f71

Internal some sodp team member use the GCC, but not the one bundled/removed from aosp.
They use the best one as far as I know.

The kernel builds just fine with GCC.

@Thaodan and @MartinX3 : sounds like I will have to try it soonish, as there is an interest :) . As for GCC/Clang - it may also affect interaction with the blobs.

At least when I run my device (kumano) with AOSP I had no side interaction with the kernel being build by gcc regarding blobs.

As described in https://github.com/sailfishos-sony-tama/main/issues/81#issuecomment-660619965, it is early and it is not expected to work on AOSP9.
I suggest to close all 4.14 related issues and deal with it when AOSP10 based ports will be available. Then we should port Tama to AOSP10 and switch over to that port.

@Thaodan: as I have separate test device, I will test if it works (4.14+AOSP9). If it doesn't, we will have to see how to get AOSP10 base working for SFOS. Timing is not clear for the latter, though

I have compiled 4.14 (with gcc for now) and I can boot into recovery with 4.14 blobs. However, rndis network seems not to be working then as I don't get any connection established. @Thaodan, as far as I could see, your changes in config were carried over and all is fine there.
Haven't tried to fire full SFOS yet, would like to get recovery working first. Maybe some other day.

Is a part of AOSP10 based porting effort. Is switched over there. Closing


Bump cockpit test lib to fix Browser.logout() for cockpit 258

The shell DOM changed in Cockpit 258. Pick up the Browser.logout()
change which can deal with both old and new shell versions.

This blocks the recent image refreshes like https://github.com/cockpit-project/bots/pull/2669

Triggering an extra test against the new rhel-9-0 image in https://github.com/cockpit-project/bots/pull/2669


sp-textfield Docs has multiple examples with un-closed tags

Issue in /src/pages/uxp/reference-spectrum/User Interface/sp-textfield.md
All the examples in that page starting from this point have a missing forward slash in the closing tag:
<!-- is -->
<sp-textfield disabled placeholder="Phone Number"><sp-textfield>
<!-- should be -->
<sp-textfield disabled placeholder="Phone Number"></sp-textfield>

Careless copy and paste is thus severely punished 😅

found in external dependency (see package.json "uxp-documentation": "github:adobedocs/uxp") and opened issue for this at https://github.com/AdobeDocs/uxp/issues/60


Potential slowdown with multiple snapshots

CrateDB version: 3.1.1
Environment description:
Java 1.8
Centos 7.6
3 master nodes, 2 query nodes, 2 ingest nodes, and a decent number of data nodes
Problem description:
We're backing up data once a day using the snapshot feature and ever since enabling snapshots, we've noticed our query times taking longer than usual. Also noticing the following query take considerably longer each day we run it: SELECT state from sys.snapshots where name = ? and repository = ? limit 1
We're submitting this ticket to see if there's anything obvious that points out to you for whether or not this makes sense.
Steps to reproduce:
Have a script that produces a daily snapshot
TIMESTAMP=$(date +%s+%3N)

CREATE SNAPSHOT mydb.backup_$TIMESTAMP ALL

Does the query performance degrate during the time the snapshot is being created, or also keep being degraded afterwards?

Where do you store the snapshots, S3?

Do you notice a speed difference between select count(*) from sys.snapshots and SELECT state from sys.snapshots where name = ? and repository = ? limit 1 ? We made a change in 3.2.0 that might improve the performance of your query, but the worst case would still equal the select count(*)..  variant.

@mfussenegger thanks for getting back

The performance is degraded during and after the snapshot. Queries are noticeably slower since we began enabling snapshots.

S3

No, there isn't a speed difference (took 45s to run the count(*) statement)

Thank you for the answers.
One more question: How many snapshots do you currently have?

Didn't have the time to look into this yet too closely. But one aspect that I could imagine affecting the performance for a bit longer is that the snapshot process requires disk accessed which compete with the file system cache for the regular queries. It might take a bit until the query related parts are back in the fs cache.
Can you observe that the performance eventually improves again after taking a snapshot?


Crop Beta City yeg photo

When you are viewing in mobile the styling looks off so just crop it so it alligns

Maybe a better solution here is to make the Logo image optional? Seems like in this case we don't need both the Logo and the header.

I think Beta city is the only one without a cool photo though, maybe the hardware meetup too. I think it might look a little weird if they were the odd ones out?

I think Beta city is the only one without a cool photo though, maybe the hardware meetup too. I think it might look a little weird if they were the odd ones out?

Yeah you are right, I tried without and looked odd. Created #225 to trim the whitespace off the logo sides.


using $noSelect throws an error

From what I can tell, the $noSelect needs to be a query param. However when including it as a queryParam, feathers throws the following error:
error: BadRequest: Invalid query parameter $noSelect
    at new BadRequest (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/node_modules/@feathersjs/errors/lib/index.js:86:17)
    at commons_1._.each (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/lib/filter-query.js:41:27)
    at Object.keys.forEach.key (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/node_modules/@feathersjs/commons/lib/utils.js:15:45)
    at Array.forEach (<anonymous>)
    at Object.each (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/node_modules/@feathersjs/commons/lib/utils.js:15:30)
    at cleanQuery (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/lib/filter-query.js:35:21)
    at Object.filterQuery [as default] (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/lib/filter-query.js:87:20)
    at Object.filterQuery (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/lib/service.js:45:46)
    at Object._patch (/Users/rjewing/code/personal/evolution/packages/api/node_modules/feathers-objection/lib/index.js:513:14)
    at callMethod (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/lib/service.js:12:22)
    at Object.patch (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/adapter-commons/lib/service.js:83:16)
    at processHooks.call.then.hookObject (/Users/rjewing/code/personal/evolution/packages/api/node_modules/@feathersjs/feathers/lib/hooks/index.js:56:27)
    at process.internalTickCallback (internal/process/next_tick.js:77:7)

Any non-standard query operator needs to be explicitly whitelisted in the service file:
https://blog.feathersjs.com/feathers-2019-new-years-news-f478d5f2c8cd#60e1


Feature/broken tests

broken tests example

Coverage decreased (-2.9%) to 97.133% when pulling 89c91988ad62032246d12e16899e081fec214e66 on feature/broken_tests into ed0efbd2a3718f977c01cc15b33aeb1aa4fb299c on master.


Hi

Note: for support questions, please join our Discord server

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)

Expected result (what you hoped would happen)

Actual result (unexpected outcome)

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

Hey!
Closing this but feel free to keep commenting :)


fix attempt #2 for AudioBufferSourceNode stop time miscalculation, with updated tests

2nd try at a fix for #912, with tests updated to use the corrected time reference. All tests now pass on Mac (though not on Windows but that's a separate can of worms that has nothing to do with the changes made in this pr, see comments at #942).
I still plan to come up with a test to repro the conditions that lead to #912, will commit that when I get a chance.

Hey @tambien here's a better fix for #912. Turns out the original theory of it being a currentTime vs now() issue was a red herring, though that did help me solve some other timing issues by accident.
Anyway for #912 I realized it was only happening on repeated calls to player.restart() and when there are multiple player._activeSources. player.restart() was leading to a call to player._stop which indiscriminately sets all _activeSources to the same stop time. When there were multiple _activeSources with different delayed stop times due to lookAhead - such as when restart() is called rapidly at increments < lookAhead - this caused a pileup of increasingly erroneously delayed stops. My fix is to only stop the most recently created source on player.restart(), rather than stop all _activeSources (let me know if you think there might be a more elegant way to go about this).

I've added a test for this fix (fails on dev, passes on this branch)

Thank you! Really appreciate the contributions


SW-3647 Accession species only editable if no deliveries

In preparation for an upcoming change that causes changes to an accession's
species to be propagated to seedling batches that come from that accession,
add logic to prevent editing an accession's species if the accession's seeds
have already been sent to a nursery, and seedlings have already been withdrawn
from the nursery and delivered to a planting site.
This prevents accession species changes from causing inconsistencies between
nursery withdrawals and the species populations at planting sites. In the
future we may remove this restriction and make the species changes propagate to
planting site data too, but doing so would introduce a number of edge cases
that would require more detailed requirements definition.
If the species dropdown is disabled, there is a tooltip explaining why.

https://github.com/terraware/terraware-server/pull/1269 is the server side of this, but it should be safe to merge this PR first (the new API field is marked as optional, and if it's missing, the species will be editable as normal).

Oops, forgot to include a screenshot.


Node: mediapipe models

Partially addresses https://github.com/aisingapore/PeekingDuck-Private/issues/31

Supports "Face Detection" and "Pose" MediaPipe Solution
Face Detection is implemented as an object_detection task with a face subtask.

Omits face keypoints results (for output type compatibility)
Implemented as a subtask as MediaPipe has a general object detection solution which may become available in Python

Pose estimation is implemented as a pose_estimation task with a body subtask

Converts the model's 33 keypoint output into COCO's 17 keypoint format to be compatible with other pose models.
Uses body subtask instead of person following convention in COCO dataset

Added a pose utility module for handling some common processing functions for COCO's 17-keypoint body keypoints (not yet applied to existing pose estimation models'

I encountered the following error when performing a make clean html in PeekingDuck/docs/:
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 0 source files that are out of date
updating environment: 0 added, 1 changed, 0 removed
reading sources... [100%] nodes/model.mediapipev1
WARNING: autodoc: failed to import class 'Node' from module 'model.mediapipev1'; the following exception was raised:
Traceback (most recent call last):
  File "/opt/homebrew/Caskroom/miniforge/base/envs/temp4/lib/python3.8/site-packages/sphinx/util/inspect.py", line 376, in safe_getattr
    return getattr(obj, name, *defargs)
AttributeError: module 'model.mediapipev1' has no attribute 'Node'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/opt/homebrew/Caskroom/miniforge/base/envs/temp4/lib/python3.8/site-packages/sphinx/ext/autodoc/importer.py", line 98, in import_object
    obj = attrgetter(obj, mangled_name)
  File "/opt/homebrew/Caskroom/miniforge/base/envs/temp4/lib/python3.8/site-packages/sphinx/ext/autodoc/__init__.py", line 306, in get_attr
    return autodoc_attrgetter(self.env.app, obj, name, *defargs)
  File "/opt/homebrew/Caskroom/miniforge/base/envs/temp4/lib/python3.8/site-packages/sphinx/ext/autodoc/__init__.py", line 2804, in autodoc_attrgetter
    return safe_getattr(obj, name, *defargs)
  File "/opt/homebrew/Caskroom/miniforge/base/envs/temp4/lib/python3.8/site-packages/sphinx/util/inspect.py", line 392, in safe_getattr
    raise AttributeError(name) from exc
AttributeError: Node

looking for now-outdated files... none found

Are you able to build the docs for model.mediapipev1? Wonder if this is a M1 Mac issue?
(c.f. I was able to build the docs for PeekingDuck's latest dev branch)

How to get a list of MediaPipe model types? Calling peekingduck model-hub --help only shows huggingface:
Invoking 'peekingduck model-hub --help'...
Usage: cli model-hub [OPTIONS] COMMAND [ARGS]...

  Utility commands for model hub models.

Options:
  --help  Show this message and exit.

Commands:
  huggingface  Utility commands for Hugging Face Hub models.

 ~/s/ais/y/PeekingDuck │ node-mediapipe !2 ?2

Can provide a sample object detection yaml and a pose estimation yaml for testing?

Running pytest -s -vv tests/pipeline/nodes/model/mediapipev1 returns the following error on M1 macOS, because mediapipe package is named mediapipe-silicon on M1 macOS:
❯ pytest -s -vv tests/pipeline/nodes/model/mediapipev1
======================================= test session starts =======================================
platform darwin -- Python 3.8.13, pytest-7.1.3, pluggy-1.0.0 -- /opt/homebrew/Caskroom/miniforge/base/envs/temp4/bin/python
cachedir: .pytest_cache
rootdir: /Users/dotw/src/aisg/yier/PeekingDuck, configfile: pyproject.toml
plugins: typeguard-2.13.3
collecting ... 2022-10-09 17:16:57 peekingduck.utils.requirement_checker  INFO:  mediapipe>=0.8.11 not found and is required, attempting auto-update...
ERROR: Could not find a version that satisfies the requirement mediapipe>=0.8.11 (from versions: none)
ERROR: No matching distribution found for mediapipe>=0.8.11
2022-10-09 17:16:57 peekingduck.utils.requirement_checker  ERROR:  Command '['pip', 'install', 'mediapipe>=0.8.11']' returned non-zero exit status 1.
collected 0 items / 1 error

I encountered the following error when performing a make clean html in PeekingDuck/docs/:
Fixed by adding mediapipev1 to omit list when building docs.

How to get a list of MediaPipe model types? Calling peekingduck model-hub --help only shows huggingface:

Apologies, I missed out this part when finishing up. Intially I thought mediapipe will have an API similar to huggingface so we could build some CLI commands with minimal maintanence on our side. But turns out there isn't any.
We will have to copy out the model type documentation from the official site ourselves. Then would it be better to: 1) just leave it in the Node docstrings (like normal model nodes) or 2) create some CLI commands which just prints out some predefined string. Option 2 will have a more consistent interface like huggingface but may possible create 2 locations to maintain.

Can provide a sample object detection yaml and a pose estimation yaml for testing MediaPipe's models?

nodes:
  - input.visual:
      source: https://storage.googleapis.com/peekingduck/videos/wave.mp4
  - model.mediapipe:
      #   task: object_detection
      #   subtask: face
      task: pose_estimation
      subtask: body
      model_type: 0
  - draw.bbox
  - draw.poses
  - output.screen

Running pytest -s -vv tests/pipeline/nodes/model/mediapipev1 returns the following error on M1 macOS, because mediapipe package is named mediapipe-silicon on M1 macOS:

May I know if mediapipe-silicon is installed through pip or conda? If it's through pip, I can try to modify RequirementChecker to accept an "either ... or" statement. But if it's through conda would it then be ok to make mediapipe an compulsory requirement so it skips the check?

Thanks for the updates.

The docs now build correctly.
Re CLI model-hub, it'll be good to have the same interface as HF to provide a consistent look-and-feel. Is there any chance this of this reading the relevant data from model.mediapipe node? So that we only have a single point of update.
The mediapipe-silicon is installed through pip. So can modify RequirementChecker to handle this as you suggested.

Updates needed:

model.mediapipe node needs description of both object detection and pose estimation.
Running your mediapipe yaml above produces the below output, but I think this would be fixed once 3 above is implemented.

❯ python . --config_path mp.yml
2022-10-10 18:25:05 __main__  INFO:  Run path: mp.yml
2022-10-10 18:25:05 peekingduck.declarative_loader  INFO:  Successfully loaded pipeline file.
2022-10-10 18:25:05 peekingduck.declarative_loader  INFO:  Initializing input.visual node...
2022-10-10 18:25:05 peekingduck.declarative_loader  INFO:  Config for node input.visual is updated to: 'source': https://storage.googleapis.com/peekingduck/videos/wave.mp4
2022-10-10 18:25:05 peekingduck.pipeline.nodes.input.visual  INFO:  Input size: 710 by 540
2022-10-10 18:25:05 peekingduck.declarative_loader  INFO:  Initializing model.mediapipe node...
2022-10-10 18:25:05 peekingduck.utils.requirement_checker  INFO:  mediapipe>=0.8.11 not found and is required, attempting auto-update...
ERROR: Could not find a version that satisfies the requirement mediapipe>=0.8.11 (from versions: none)
ERROR: No matching distribution found for mediapipe>=0.8.11
2022-10-10 18:25:06 peekingduck.utils.requirement_checker  ERROR:  Command '['pip', 'install', 'mediapipe>=0.8.11']' returned non-zero exit status 1.

Can resolve peekingduck/pipeline/utils/bbox/transforms.py conflict? Think this was caused by Hongnan's PR.

@ongtw The merge conflict has been fixed.


Add support and tests for using Headers object

Add support and tests for using Headers object. Currently we are only supporting a headers json.

Implemented.


Errors

Hi,
I got this errors using swift3 on ubuntu 14.04:
user@pc444:~/filterit$ swift newStreamReader.swift
newStreamReader.swift:36:30: error: cannot convert value of type 'Data' to type 'NSData!' in coercion
self.delimData = delimData as NSData!
^~~~~~~~~
newStreamReader.swift:61:47: error: cannot convert value of type 'NSMutableData!' to type 'Data' in coercion
let line = NSString(data: buffer as Data, encoding: encoding.rawValue);
^~~~~~
newStreamReader.swift:78:16: error: cannot convert value of type 'NSString?' to type 'String?' in coercion
return line as String?
^~~~
newStreamReader.swift:113:24: warning: result of call to 'seekToEndOfFile()' is unused
fileHandle.seekToEndOfFile()
Could you please help to fi this?

This class isn't written for Ubuntu because it uses foundation classes (prefix NS). I'm going to add Ubuntu support soon (today or tomorrow morning). Thanks for feedback.

I've added ubuntu support.

Thank you so much.

Also this code has memory leak. On macOS it is fixed by autoreleasepool. But Ubuntu do not has this feature. Do you has any ideas how to fix this on Ubuntu?

Hmm ... you can use this class from this page to fix this issue on ubuntu.
internal class NSAutoreleasePool {
    fileprivate static var _current = NSThreadSpecific<NSAutoreleasePool>()
    internal static var current: NSAutoreleasePool {
        return _current.get() {
            return NSAutoreleasePool()
        }
    }

    var depth: Int = 0
    var objects = [[AnyObject]]()

    fileprivate override init() { }

    fileprivate func push() {
        objects.append([AnyObject]())
        depth += 1
    }

    fileprivate func pop() {
        objects.removeLast()
        depth -= 1
    }

    func add(_ object: AnyObject) {
        objects[depth - 1].append(object)
    }
}

public func autoreleasepool(_ code: () -> ()) {
    NSAutoreleasePool.current.push()
    code()
    NSAutoreleasePool.current.pop()
}

But i'm not sure, because I'm not using swift on  ubuntu.
Let me know when you fix this.

Thank you. I will try this tomorrow and write feedback.

I try "internal class NSAutoreleasePool {..." and got:
error: use of unresolved identifier 'NSThreadSpecific'
fileprivate static var _current = NSThreadSpecific()

I don't found anything about this problem on ubuntu. Are you sure that memory leak takes place in this code?

Maybe swift for ubuntu has something like garbage collector or something similar. Sorry can't help you with this kind of problem ... :(

In any case, thank you.

I found the problem, memory leaked using this line:
let tmpData = fileHandle.readData(ofLength: chunkSize)

What tool are you using to find those leaks?

Please file new issue about memory leaks on ubuntu and provide more info (screens)

I create large(10GB) file.txt with a lot of lines (~20.000.000). After that I use this code (attached)
readIt.txt
When I execute this code, I takes about 5-10 sec that my 8GB RAM becomes empty.


What's the deal with 4.13.0-0.okd-2023-09-03-112738

It's not listed in the releases.

That's the question.  Two of my clusters listed this release as soon as the other one from 2023-09-03 had been installed.

Now is Okay
oc adm upgrade
Cluster version is 4.13.0-0.okd-2023-09-03-082426
Upstream: https://amd64.origin.releases.ci.openshift.org/graph
Channel: stable-4
No updates available. You may force an upgrade to a specific release image, but doing so may not be supported and may result in downtime or data loss.

That's unfortunate for my test cluster that I already updated to 4.13.0-0.okd-2023-09-03-112738.  One node cannot pull some images now.
Sep 05 08:42:47 ... bash[6471]: Trying to pull registry.ci.openshift.org/origin/4.13-2023-09-03-112738@sha256:25919ca30f4a1d410b2301d191b8f268f131372da04c4646a45c1127c9492e22...
Sep 05 08:42:48 ... bash[6471]: Error: initializing source docker://registry.ci.openshift.org/origin/4.13-2023-09-03-112738@sha256:25919ca30f4a1d410b2301d191b8f268f131372da04c4646a45c1127c9492e22: reading manifest sha256:25919ca30f4a1d410b2301d191b8f268f131372da04c4646a45c1127c9492e22 in registry.ci.openshift.org/origin/4.13-2023-09-03-112738: received unexpected HTTP status: 500

There are no breaking changes between these, so you can recover by "downgrading" back to stable

That's harder if the kubelet on the node does not start.  I think I'll just reprovision the node.

That's harder if the kubelet on the node does not start.

Last time this problem occurred (#1433), I managed to get around some similar problems and recover such a broken cluster - the related discussion https://github.com/okd-project/okd/discussions/1435 would contain some hints that may be useful when you need to recover kubelet in this situation (i.e. in case reprovisioning is not an option).

I'll close this.  There is nothing to do here afaict.


Button adjustments

POI buttons
Around here:
https://github.com/fpdcc/trailsy/blob/development/js/panelFunctions.js#L446

[x] Add a container (.fpccButtonContainer) around the directions button (and new button)
[x] Add a conditional to build a button for the location's web page if it exists (poi_desc --> web_link column) and have it display: "Location Webpage".

This button should have the additional secondary class and a new ID (noted below) if needed.
Do we need a separate analytics type for this?

Resulting in this:

 name]">Directions

 web_link]" id="entranceWebsite" class="fpccButton secondary" data-analyticstype="?" data-analyticsdescription="[points_of_interest --> name]">Location Webpage

CSS
CSS handled in updated newguts.css https://github.com/fpdcc/trailsy/commit/0b8cf1c4339d2dfabc41db55864b6460bd2d3519

Trail Buttons
Around here:
https://github.com/fpdcc/trailsy/blob/development/js/panelFunctions.js#L893

Remove label: PDF Trail Map:
Adjust first button text to "PDF Map"
Replace second button with a conditional to build a button for the trail's web page if it exists (trails_desc --> web_link column) and label "Trail Web Page".

This button should have the additional secondary class and a new ID (noted below) if needed.

Resulting in this:

PDF Map

Trail Web Page

@gisdevcc Can we flip the Trail Webpage and PDF Map buttons, to prevent this (all trails have a webpage, but not all trails have a PDF map):

So the result we are looking for is this. Trail Webpage first, PDF Map second (with "secondary" class added):

Trail Webpage
PDF Map

Resulting in:

Or if no PDF:

This is updated on staging


release/v1.0.0

Description
This is a major update containing a number of features. It includes:

Curation to work with Space Grant deposits
Providing TinyURL link - Long Qualtrics links a problem for Outlook
Adding METADATA folder
Dumping JSON responses (curation metadata) to METADATA folder
CI testing with Python 3.9
Bug fix for when Qualtrics information is not provided for name matches
Handling of missing README form
Add ability to retrieve Deposit Agreement PDF via URL approach

Closes #137
Closes #144
Closes #151
Closes #152
Closes #155
Closes #168
Closes #172
Closes #187
Update Changelog

[ ] README.md, changelog

Deployment stuff

[ ] NOTE: The change to v1.0.0 is because of a backwards incompatibility issue with the config file that is to address #137. This will now handle multiple deposit agreements by providing it as a list of surveyId instead of just a single entry.  For deployment, we will have both configuration files and use symbolic links to point to the correct one.

[x] Copy <1.0.0 config files as "***_vlt1.0.ini"
[x] Create v1.0.0 config files as "***_v1.0.ini"
[x] Check that METADATA folder is defined in all v1.0 config files
[ ] Symbolic link to appropriate file

[ ] Update dependencies to use pandas==1.2.3 (redata-commons uses this version) and numpy==1.20.0

Other changes:

[x] ~Add full_url for Deposit Agreement in logs (for completeness) before TinyURL conversion~ (This was already in place)
[ ] Add email handling exception for WCCFL (i.e., each survey id)? - This might be out of scope for v1.0.0
[x] Bump version on pandas and numpy (see above)
[ ] BUG: TinyURL link for Deposit Agreement does not work as intended as the Depositor Info is not pre-populated

Here's a list of manual end-to-end testing checks:

[x] CI builds with Python 3.9 is working
[ ] General checks:

[x] Ensure that dict_load function handles lists and strings correctly from config file
[ ] Ensure that Qualtrics's merged_survey method worked for all deposit agreements
[ ] Check that DepositorName object has version_no instance variable
[ ] Check that TinyURL works

[ ] Standard stage deposit with Deposit Agreement and README form article_id=6968970

[ ] Ensure it uses the regular surveyId

[x] With get_qualtrics (stage)
[ ] With get_data (stage) [multiple deposit agreement so this was not tested for 6968970]

[x] Check that TinyURL's work (get_qualtrics_links script)
[x] Ensure that data retrieval works as intended
[x] Check that retrieval of Deposit Agreement PDF URL is possible

[x] Check for incorrect URL
[x] Check for correct URL

[x] Check that README.txt form works as intended
[x] Check METADATA contents

[x] Check that curation_original_XXXXXXXX.json is stored
[x] Check that file_list_original_XXXXXXXX.json is stored
[x] Check that file_list_original_XXXXXXXX.csv is stored

[ ] Standard stage deposit with Deposit Agreement (no README form) article_id=6936134

[ ] Ensure it uses the regular surveyId

[x] With get_qualtrics (stage)
[ ] With get_data (stage)

[ ] Check that TinyURL's work (get_qualtrics_links script)

[ ] Check the deposit agreement link is correct
[ ]

[ ] Ensure that data retrieval works as intended
[ ] Check that retrieval of Deposit Agreement PDF URL is possible

[ ] Check for incorrect URL
[ ] Check for correct URL

[ ] Check that code runs fine without README form response
[ ] Check that README.txt form is the most basic (given no README form response)
[ ] Check METADATA contents

[ ] Check that curation_original_XXXXXXXX.json is stored
[ ] Check that file_list_original_XXXXXXXX.json is stored
[ ] Check that file_list_original_XXXXXXXX.csv is stored

[ ] Test Space Grant deposit with Deposit Agreement (no README form) - perform on stage

[ ] Ensure it uses the Space Grant surveyId
[ ] Check that TinyURL's work (get_qualtrics_links script)
[ ] Ensure that data retrieval works as intended
[ ] Check that retrieval of Deposit Agreement PDF URL is possible

[ ] Check for incorrect URL
[ ] Check for correct URL

[ ] Check that code runs fine without README form response
[ ] Check that README.txt form is the most basic (given no README form response)
[ ] Check METADATA contents

[ ] Check that curation_original_XXXXXXXX.json is stored
[ ] Check that file_list_original_XXXXXXXX.json is stored
[ ] Check that file_list_original_XXXXXXXX.csv is stored

[ ] Test WCCFL deposit with Deposit Agreement

[ ] Ensure it uses the WCCFL surveyId
[ ] Check that TinyURL's work (get_qualtrics_links script)
[ ] Ensure that data retrieval works as intended
[ ] Check that retrieval of Deposit Agreement PDF URL is possible

[ ] Check for incorrect URL
[ ] Check for correct URL

[ ] Check that code runs fine without README form response
[ ] Check that README.txt form is the most basic (given no README form response)
[ ] Check METADATA contents

[ ] Check that curation_original_XXXXXXXX.json is stored
[ ] Check that file_list_original_XXXXXXXX.json is stored
[ ] Check that file_list_original_XXXXXXXX.csv is stored

Bump version
v0.17.7 -> v1.0.0

[ ] README.md, installation instructions
[ ] setup.py
[ ] ldcoolp/__init__.py

When retrieving qualtrics links for 6936134, I tried the Deposit Agreement one and it returned a form that was not pre-populated. This is weird as I thought previous check seems to work:
The links are:
https://uarizona.co1.qualtrics.com/jfe/form/SV_39rs7lHGLUYFK7P?article_id=6936134&curation_id=451284&Q_PopulateResponse={"QID4": {"1": "Damian Yukio Romero Diaz", "2"<EMAIL_ADDRESS>"QID7": "Deposit %23 2 for Demo of Data Curation Scripts"}
TinyURL: https://tinyurl.com/Romero-6936134-1-DA
This somehow has alias passed in as a parameter. This did not happen with the README form.


Remove spot market code duplicated from CGCloud

Depends on https://github.com/BD2KGenomics/cgcloud/issues/248

Closing in favor of #1813.


Reduce development environment RAM requirements by 750MB.

Since we merged cd2348e9aec36d7f26475fa7d966d06588dbeb75 more than a
month ago and haven't seen any noticable regresions as a result, it's
reasonable at this point to do a corresponding decrease in our
documented RAM requirements for the Zulip development environment.

Automated message from Dropbox CLA bot
@timabbott, it looks like you've already signed the Dropbox CLA. Thanks!

@christi3k FYI.  I think before we merge this, it'd be good if you could check in one of your non-LXC VMs what the free/used RAM situation is inside the guest with the development server running, just to double-check that this isn't going to cause problems.

I've checked on Windows 10 with Vagrant. Memory usage averages ~1170MB with ~98MB free. Seems like it's okay to proceed with lowering to 1280.

Can you post free -m output?  I'm not sure whether you're counting cache/buffers, which is effectively free memory....

(zulip-venv)vagrant@vagrant-ubuntu-trusty-64:~$ free -m
             total       used       free     shared    buffers     cached
Mem:          1245        725        520         14         29        150
-/+ buffers/cache:        544        701
Swap:            0          0          0

OK, great, thanks @christi3k!  Given that data, 1.25GB is definitely plenty; we could probably even get away with 1GB, but that would likely carry some risk of problems when people are doing multiple things at the same time (especially running tools/test-js-with-casper while run-dev.py is running, since that effectively results in 2 Zulip servers running at the same time).  I guess if it's easy for you test what free -m looks like while that (and the normal run-dev.py) are running, that would answer the question of whether we can safely decrease this more.

Here's what free -m reports while running run-dev.py and tools/test-js-with-casper:
(zulip-venv)vagrant@vagrant-ubuntu-trusty-64:~$ free -m
             total       used       free     shared    buffers     cached
Mem:          1245       1173         72         26         36        389
-/+ buffers/cache:        748        497
Swap:            0          0          0

Seems to be cutting it pretty close.
Incidentally, my Windows 10 machine does not like the symlink tools/test-js-with-casper. I had to run frontend_tests/run-caser instead.

Actually, that's not cutting it particularly close -- in some sense, the line to look at is the -/+ buffers/cache line.  Basically, only 750MB are being used by running processes, about 400MB are being used to cache filesystem data in memory, and 72MB are totally unused.  So it would definitely work with 1GB given that report.  I guess I should still decide how much headroom we want to leave... anyway, thanks for the research!
The Windows issue is interesting; maybe it comes from having cloned the Zulip repo outside the virtualbox guest (and thus the symlink having been created on Windows)?

I went ahead and merged the 1.25GB version; we can always lower it to 1GB later and I think I'd prefer to leave some headroom for future services that consume extra memory.

@christi3k maybe the Windows 10 symlink thing should be its own issue?

@timabbott I don't think it needs to be its own issue. I've figured out that you just need to tell Cygwin how to create symlinks so that they'll work inside the VM. I'm updating PR #762 right now with the details.

Sounds good!


[Needs update for otp to work] Not logging in. Auth token stays in url

Version
@nuxtjs/supabase: 0.3.8
nuxt: 3.6.5
Reproduction Link
See here https://github.com/supabase/gotrue-js/issues/455#issuecomment-1652553296
Steps to reproduce
New project, install Nuxt and supabase and tryu to use magic link login
What is Expected?
Logged in.
What is actually happening?
Not logged in, auth token in URL
Can we update https://github.com/nuxt-modules/supabase/blob/main/package.json#L29 to 2.31.0?

https://github.com/nuxt-modules/supabase/issues/215

@stlbucket it is indeed.
Workaround for now is to do overrides.
https://github.com/supabase/gotrue-js/issues/455#issuecomment-1652250165
https://github.com/supabase/gotrue-js/issues/455#issuecomment-1649505404

@silverbackdan unfortunately that workaround didn't help

i just cloned this repo, updated the supabase js client, built it and linked with pnpm:
    "@nuxtjs/supabase": "file:~/work/nuxt-modules/supabase",

i got the same result, however
npm list in @nuxt/supabase:

so would this mean it is an issue with gotrue or supabase-js?

I added this button to the pages/index.vue file of this repo:
  <button @click="client.auth.signInWithOtp({ email<EMAIL_ADDRESS>options: { emailRedirectTo: 'http://localhost:3000/confirm'} })">
    LogIn OTP
  </button>

this results in the same error with supabase-js 2.31.0

https://github.com/supabase/supabase/issues/16100

the bug happens here - maybe someone can verify they get the same behavior with this repo?
https://github.com/stlbucket/supabase-cli-bug
this is a sanity check to take nuxt out of the loop - i think this verifies it's a problem with the nuxt plugin:
https://github.com/stlbucket/vue-supabase-initialize

not sure who to ask about this... @larbish ?

i have done a couple of other tests, and i think this is actually due to the state of my local supabase project.
isolating vue/supabase and nuxt/supabase in the above projects, and building fresh supabase environments for each seems to make the issues go away.
i am still trying to isolate the problem, though.

here is a clue

i think this issue can be closed

@danielward So if the access token is in the path then it usually seems to indicate that the session will not be set/saved. v 1.0.2 seems to work for me - although I need to make sure that the URL clicked in the email will open in the SAME browser as the link was sent from. Otherwise it won't work, but I'm not sure where that issue is - I haven't investigated. That's as much info as I have for now, but hopefully using the latest version and updating all deps will help you resolve this issue now.

@silverbackdan Hmm, I have 1.0.2 and am making sure the email link opens in the same browser (even the same tab) with the same problem. I've followed the documentation to the letter and then explored other package versions to help with no success; the session isn't updated with a user so I'm stuck at the first hurdle. That said, I'm considering going down the email/password sign in route with Supabase initially, esp. as they seem to limit magic links to 100 per hour or something (likely before billing for it), so I imagine this problem will go away for now. Doesn't help though if I would like to support this still, which ideally I would.

I wonder if this helps in any way? It certainly reads like it could be the cause. Interestingly that doesn't appear in Safari on Mac, but it does appear for Chrome on Mac.

Maybe not, I think that's a smoke screen coming from a password manager to be fair.

Thus far, the only conclusion I've drawn is that it feels like there's no middleware active to pull access tokens from the router's path. If I detect and parse the access token data from the path and provide that to set the session with Supabase auth, that works just fine.

I think I'm having the same issue. I just followed to the letter this tutorial:
https://supabase.com/docs/guides/getting-started/tutorials/with-nuxt-3
I'm using magic links to authenticate users but every time I click the links sent by Supabase I got redirected to the login component. @danielward your conclusion seems right to me, it is as if there's no middleware that fetches the tokens in the URL from the magic links.  From the tutorial I get that  useSupabaseUser() should retrieve this information and login the user... but no luck.

@pguerrero-at-wiris - https://supabase.nuxtjs.org/authentication - you may want to look at this instead
I think the recent changes to remove implicit flow has effected you as it did me.  I originally implemented using the link you show.  It got so broken that I re-implemented my auth, but using the link I have provided.  It seems to be working much better.

@stlbucket No luck so far on my end, same issue again where the token information is left in the URL and not handled, so the confirm page just sits on 'Waiting for login...'.

@danielward - this sounds like you are trying to use implicit flow.  if you are using pkce, from the link above:
All you need to do is to create a login.vue and confirm.vue page in the pages folder.
and they give you examples of what to do.


iOS - continueInApp and openApp not working

Hi,
I have successfully installed a custom iOS Share View but when I try to use continueInApp or openApp, nothing happens.
My app and share view targets both have the same App group and in the info.plist of the share view the bundle ID of my app and the url schema are also setup.
Any suggestion?

Thank you for the answer, I already had a url scheme setup so I used that one as the HostAppURLScheme.
Do I have to add a new url scheme different from the one I already have @Gustash?

I found my issue the HostAppURLScheme has to have :// following the scheme!
@Gustash maybe this should be specified in the documentation.


Slither: Consider using --disable-solc-warnings

Hi,
This a great work!
For information, by default slither will print the compiler warnings, which are already shown in remix.
You can prevent the duplicate information by running slither with the flag --disable-solc-warnings

@montyly  sure will do


Include checkbox in the app to show/hide the "observed" data bar chart

[x] Include checkbox in the app to show/hide the "observed" data bar chart.

Solved in #60


How to disable message "Energy requested is outside of tabulated data, using linear extrapolation"

I am often receiving the message "Energy requested is outside of tabulated data, using linear extrapolation". Could this be disabled?

Hi Daniel,
Sorry about all the warnings. I've changed the default behavior to not warn if the attenuation coefficient at the requested energy is outside of the tabulated data. I think it could be best to just change the default behavior of the warnings package, but this should be fine for now. To enable the warnings, add the kwarg warn = True when requesting attenuation coefficients.
Just to be clear, I don't know how valid the extrapolated data is (the extrapolated data is formed by a linear extension of the two edge values of tabulated data in the log-log space; the pair production values are extrapolated in a different space, but are zero for small energies).


Support new field of Context model

Mist v1.0.0-RC16 has max-conn-failure setting in Context definition which is missing in cli

Could we change this behavior of building request body into a more flexible way where every field from data section is putting into body automatically?
I'm going to add more settings for contexts in the next versions and can't use them from mist-cli during development.

Yes, for sure. But it's only required for context. Need to test it with mist how it will handle the unknown fields and some missing fields.


Failures on [co19] Roll co19 to abb96bbca3fc36e6a594967509687431d138e998...Make record positional field getters start at $1.

There are new test failures on [co19] Roll co19 to abb96bbca3fc36e6a594967509687431d138e998...Make record positional field getters start at $1..
The tests
vm/dart/records_allocation_sinking_il_test Crash (expected Pass)
vm/dart/records_return_value_unboxing_il_test Crash (expected Pass)

are failing on configurations
dartkp-strong-linux-release-x64
dartkp-strong-mac-release-arm64
dartkp-weak-asserts-linux-release-x64
dartkp-weak-asserts-mac-release-arm64

@lrhn It looks like your change bee7ab2b32adca681060d151337d3589406fb410 is the culprit, could you please take a look?

Should this be area-front-end instead of area-vm ?

Used area-vm label because vm tests are broken (or probably just not updated properly). It doesn't seem to be related to the front-end.

These failures happen because compare_il tool uses checked-in Dart SDK to run IL verification (included into tests), and older SDK doesn't understand the recent change of positional record fields starting with $1.
In order to fix these failures, we need to update checked-in SDK to 3.0.0-180.0.dev or later. I tried bumping sdk_tag in DEPS, but got the following error from gclient sync:
Errors:
  failed to resolve dart/dart-sdk/linux-amd64@version:3.0.0-180.0.dev (line 65): no such tag

@athomas Do you have any suggestions about updating checked-in Dart SDK?

@alexmarkov this has to be done by engprod. We don't have a -180.dev+ release published, yet. I'll pick this up on Thu/Fri after the next dev release goes out.

The roll fixed the tests:
CompileTimeError -> Pass (expected Pass)
language/patterns/switch_case_scope_test
Crash -> Pass (expected Pass)
vm/dart/records_allocation_sinking_il_test
vm/dart/records_return_value_unboxing_il_test


Update to include marathon-lb healthcheck config

Add configuration for the marathon-lb healthcheck. This resolved an issue I had where the healthcheck was using HTTP/1.0 and not sending a hostname

@rgardler  - as promised :-)

Hi @stuartleeks, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (stuartle). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@stuartleeks : Thanks for your contribution to the Azure documentation! The author, @rgardler, has been notified to review your proposed change.

#sign-off

@stuartleeks - Thanks for your contribution to the Azure docs. On November 16th, we announced the availability of our Azure technical documentation in a new location on docs.microsoft.com. You can read about it on our team blog. This change means the article source lives in a new GitHub repository. Because there are some changes in the source markdown as a result of the move, we can’t pull your contribution over to the new repo without creating a ton of merge conflicts. We invite you to resubmit the change (if you are willing) to the new version of the article at https://docs.microsoft.com/en-us/azure/container-service/container-service-load-balancing. Thanks for your understanding. (FYI @rgardler)


Make it possible to read file from a online source

Would be nice to be able to use a css file that resides online (Like a shared Google Spreadsheet).
For example
 csv do
  file_url 'some_url'
  separator '-'
  columns [:id, :name]
end

seems like a neat idea, just a matter of implementing it. In all honesty I probably won't have the time/inclination to do this, but PRs welcome!


Added function HasNext

The function is required for https://github.com/mandelsoft/spiff to work.

@StoykoDimitrov Thank you for the pull request! However, repositories within the cloudfoundry-attic organization are read-only copies of software that is no longer supported by the Cloud Foundry community. As such, no PRs are accepted and no further commits are made to any repo in this org. If you have an interest in contributing to Cloud Foundry projects, please feel free to do so in any of our repositories within the https://github.com/cloudfoundry or https://github.com/cloudfoundry-incubator organizations.


Studios and Reflections should be "assessments"

The Avengers of Studios and Instructors of Reflections need to take attendance after the Studios and Reflections. A uniform way of doing so could be to assign a grade of 1 to each Studio and Reflection session, and then allow the Avengers/Instructors to use the "grade moderation" feature in order to manually record that a student was absent.

Sounds good, we can easily make them Paths

We have used a google sheet for this and it gets the job done.


Use NeuroNER for SRL

First of all, thanks for making this code available Franck
my question: Are there any specific design constraints that make this code suitable only for NER (hence the name NeuroNER)? or could I use it for other tasks like Semantic Role Labeling? I am guessing the model will mimic the training data. In other words, if the training tags are interpreted as SRL then NeuroNER might very well be considered an SRL model too.
Are you able to comment on should I expect poorer performance using this code for SRL?

why don't you try?

That's an non-answer answer. I do plan to try it. But, before I do so I was checking with the author of the software on what his opinion on the matter was. But looks like the author is not active on this board anymore. Thanks anyway


Specifically handle apidoc generated files

files generated with sphinx-autodoc will inherently contain toctree directives,
and so sphinx_external_toc will warn e.g.: apidoc/my_package.rst:7: WARNING: toctree directive not expected with external-toc [etoc.toctree]
You can currently work around this by including:
suppress_warnings = ["etoc.toctree"]

but that will then ignore any legitimate issues
so it would nice to be able to "mark" a file as "it is ok for this file and its children to contain toctree"

FYI: this issue "toctree directive not expected with external-toc" also pops up using sphinx-tags


Right channel is always silent after boot up

I am using the 2ch output. After boot up, the right channel is always silent. In Audio MIDI Setup, under the output tap, the volume of the channel 2 is max (same as the left channel). It is not muted. I need to slide the volume bar to left then right to reset the volume. The bug does not happen after sleeping and waking up. I should be using 1.6.6b but not 100% sure. The volume bar in Hosting AU also showed the right channel being silent.
Soundflower-1.6.6b
Yosemite
10.10.5 (14F27)

Still exists on OS X 10.10


new subnet design

The following design changes are implemented based on a decision that was taken during a meeting with SSC network engineering.

removed protected a subnets.
changed subnets names to nonp and pbmm
adjusted hierarchichal firewall policies

closes #393

@lucstjean-ssc has an important fix to the dns feature that will be implemented prior to this PR. I will merge main afterwards


nixos/device-tree: fix package name in examples

Motivation for this change
deviceTree_rpi got renamed to device-tree_rpi a while back, so this updates the examples to reflect that.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Thanks!


Enforce SSL on app-by-app basis

Currently, a cluster admin (if you subscribe to the Google operational model) can enable / enforce redirects from HTTP to HTTPS router-wide, but it seems that it would be tremendously useful to app admins (again, if you subscribe to the Google operational model) to opt into that on an app-by-app basis if their cluster admin isn't already enforcing it router-wide.

Relates to deis/controller#579

So currently the annotation for the router router.deis.io/nginx.ssl.enforce will force all apps through HTTPS.  Would the thought be if you wanted it to be on a case by case basis to not annotate the router with router.deis.io/nginx.ssl.enforce, but to annoate the individual apps with router.deis.io/nginx.ssl.enforce

@sstarcher that's the idea, although for people using full Workflow (not just the controller) the addition of that annotation to their app's service is ideally facilitated through the deis CLI and the controller.


Add licence

Licence shall be MIT or a licence that would keep the code open-source and forbid any private patenting on the feature.

Provided with #37


NETOBSERV-1914 improved status message

Description
Improve status message and display it only if one storage is empty.

Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[X] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[X] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/label qe-approved
(based on testing on BP PR)


amplify-xcode does not escape path names

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
12.2.3
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
no manual changes were made
Describe the bug
My XCode project path contains a space
amplify init mostly worked, but failed on the last step: amplify-xcode import-config
solution was to escape the path with single quotes
Expected behavior
the path in the arguments would be esclosed with single quotes to avoid spaces in the path causing issues
Reproduction steps
create an xcode project with a space in the file path
run amplify init on the new project
Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @apanju :wave: thanks for raising this! This is a duplicate of https://github.com/aws-amplify/amplify-cli/issues/12593, and the fix will be available in the next CLI release 🎉
I'll mark this as a duplicate bug report and close in favor of tracking the existing item.


Retrieve cloudflare API key

I've stupidly lost my cloudflare API key..
where is it stored in my NPM config so i can retrieve it?
I could roll the key, but if i do that, how do i change it so that the current ssl certificate renews OK?

nevermind.. found it in letsencrypt/credentials


[WIP] RCLCPP, TF, QoS, Primitive

I'll take care of the build breaks and make sure that we didn't break anything. I wanted to start the PR for feedback and prevent duplication of efforts.

I'm migrating this to foxy. Will issue a PR when that is completed


Preparing for Release 1.0.0

Creates and moves ScreenModel API to it own module.
Remove deprecated functions
Add binary compatibility validator
Remove Androidx module and AndroidScreen
Promote Experimental APIs to Stable (removing the annotation)
Generate the API Dump

@DevSrSouza can we take a look in TabOptions api design? the current design has "expected" standards for Android like title and icon but Voyager doesn't have this anywhere else in the API anymore.
And it's also a little complicated to create these options inside the screen, this should be done by the developer from the outside. Should we see this before the release or can it be done later in a backwards compatible way?
See https://github.com/adrielcafe/voyager/issues/141

@DevSrSouza can we take a look in TabOptions api design? the current design has "expected standards" for Android like title and icon but Voyager doesn't have this anywhere else in the API anymore.
And it's also a little complicated to create these options like indeX that we dont have inside the screen, this should be done by the developer from the outside. Should we see this before the release or can it be done later in a backwards compatible way?
See #141

Yes, we can look into this in next versions and also be backwards compatible.


System.IndexOutOfRangeException when trying to open DB with password

Hi, I'm following the v5 guide but stuck with the System.IndexOutOfRangeException exception when I try to access the database with the password. Here's the code, .NET Core 3.1.
                var str = $"Filename={path};Password='{verylongpwd}'";                 using var db = new LiteDatabase(str);
The string sample is:
Filename=C:\Users\yup\AppData\Roaming\corex\storecore.file;Password='1495c305c5312dd1a9a18d9502daa0369216763ca7a6f537ddbe290241cf8aad1ca326313adec74bb98d1955747347cf0e3f087899d8bb2e0aa002ff825e1c0f25eaa79e5dfbf1c0e2daf6746a3a3f140244b764204c20c0ccede3521eaf8537ae32d4b13a04f1c387f56a8d6fa095bc53451c1892a46b8182afd94559cd7377aebc8d4a2b4883c637a359e6e67e1d8c2d789721351ebb000409329b2e875d21278b7c76724c68729e53dac50168564b8c3432018212a111c952e593829b42c296458cc0020174aaef9ca6b5661ca965004404c2bbb256bc41a8aa5c5349c615e40328a3263c45e5f96e61048149e98aa8b6f2afb59d73379e1dce5429752d8d'
If I add ; to the end of the string, I get
System.ArgumentOutOfRangeException: 'Ticks must be between DateTime.MinValue.Ticks and DateTime.MaxValue.Ticks. (Parameter 'ticks')'
It is very confusing for the moment.

Hi @panthernet, can you paste your stacktrace?

Here you go
   at LiteDB.DictionaryExtensions.<ParseKeyValue>g__ReadValue|2_1(<>c__DisplayClass2_0& )
   at LiteDB.DictionaryExtensions.ParseKeyValue(IDictionary`2 dict, String connectionString)
   at LiteDB.ConnectionString..ctor(String connectionString)
   at LiteDB.LiteDatabase..ctor(String connectionString, BsonMapper mapper)
   at X.DB.DBManager.<Session>d__1.MoveNext() in D:\Projects\x\x\Classes\DB\DBManager.cs:line 86

And this is for the ticks exception
   at System.DateTime..ctor(Int64 ticks, DateTimeKind kind)
   at LiteDB.BufferSliceExtensions.ReadDateTime(BufferSlice buffer, Int32 offset)
   at LiteDB.Engine.HeaderPage..ctor(PageBuffer buffer)
   at LiteDB.Engine.LiteEngine..ctor(EngineSettings settings)
   at LiteDB.ConnectionString.CreateEngine()
   at LiteDB.LiteDatabase..ctor(ConnectionString connectionString, BsonMapper mapper)
   at LiteDB.LiteDatabase..ctor(String connectionString, BsonMapper mapper)
   at X.DB.DBManager.<Session>d__1.MoveNext() in D:\Projects\x\x\Classes\DB\DBManager.cs:line 86

same error here

I get this error when using a connection string. I did what the document says, and i get this error: System.IndexOutOfRangeException: 'Index was outside the bounds of the array.'
This exception was originally thrown at this call stack:
	LiteDB.DictionaryExtensions.ParseKeyValue.__ReadValue|2_1(ref LiteDB.DictionaryExtensions.<>c__DisplayClass2_0)
	LiteDB.DictionaryExtensions.ParseKeyValue(System.Collections.Generic.IDictionary<string, string>, string)
	LiteDB.ConnectionString.ConnectionString(string)
	LiteDB.LiteRepository.LiteRepository(string, LiteDB.BsonMapper)
    SampleForm.Form1.Form1_Load(object, System.EventArgs) in Form1.cs
	System.Windows.Forms.Form.OnLoad(System.EventArgs)

App.config
<configuration>
    <startup>
        <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.8" />
    </startup>

    <connectionStrings>
        <add name="LiteDB" connectionString="Filename=C:\Users\<user>\Desktop\Database.db;Password='1234'" />
    </connectionStrings>
</configuration>

Source
    using (var db = new LiteDatabase(System.Configuration.ConfigurationManager.ConnectionStrings["LiteDB"].ConnectionString)) { ... }

Ok, I found a bug in ConnectionString parser, when finalize with '. I'm fixing.... for now, just remove open/close '

Got the same error:
Unhandled exception. System.ArgumentOutOfRangeException: Ticks must be between DateTime.MinValue.Ticks and DateTime.MaxValue.Ticks. (Parameter 'ticks')
   at System.DateTime..ctor(Int64 ticks, DateTimeKind kind)
   at LiteDB.BufferSliceExtensions.ReadDateTime(BufferSlice buffer, Int32 offset)
   at LiteDB.Engine.HeaderPage..ctor(PageBuffer buffer)
   at LiteDB.Engine.LiteEngine..ctor(EngineSettings settings)
   at LiteDB.ConnectionString.CreateEngine()
   at LiteDB.LiteDatabase..ctor(ConnectionString connectionString, BsonMapper mapper)
   at LiteDB.LiteDatabase..ctor(String connectionString, BsonMapper mapper)
   at ConsoleApp1.Program.Main(String[] args) in

My DB does not have any passwords set.

@alex-fomin Could you provide your datafile and the connection string you're using?

The datafile is pretty big (73Mb): https://www.dropbox.com/s/5agkno8ikr2mb2t/ProcessingQueue.db?dl=0
Connection string is just "ProcessingQueue.db"

@alex-fomin The file you sent appears to be a SQLite datafile.

Ok, I found a bug in ConnectionString parser, when finalize with '. I'm fixing.... for now, just remove open/close '

I believe the same issue is with $ character. While interpolating some strings I mistyped the name of the DB and got the same error.
Oh, and in my case, I have no password on the DB.
System.ArgumentOutOfRangeException: Ticks must be between DateTime.MinValue.Ticks and DateTime.MaxValue.Ticks.
Parameter name: ticks
   at System.DateTime..ctor(Int64 ticks, DateTimeKind kind)
   at LiteDB.BufferSliceExtensions.ReadDateTime(BufferSlice buffer, Int32 offset)
   at LiteDB.Engine.LiteEngine..ctor(EngineSettings settings)
   at LiteDB.ConnectionString.CreateEngine()
   at LiteDB.LiteDatabase..ctor(ConnectionString connectionString, BsonMapper mapper)

I have a similar scenario.
System.ArgumentOutOfRangeException: Ticks must be between DateTime.MinValue.Ticks and DateTime.MaxValue.Ticks.
Parameter name: ticks
at System.DateTime..ctor(Int64 ticks, DateTimeKind kind)
at LiteDB.BufferSliceExtensions.ReadDateTime(BufferSlice buffer, Int32 offset)
at LiteDB.Engine.LiteEngine..ctor(EngineSettings settings)
at LiteDB.SharedEngine.OpenDatabase()
at LiteDB.SharedEngine.Pragma(String name, BsonValue value)
DB is located in C:\Users<username>\AppData\Roaming<publisher><feature>\Autosave.db
DB is password protected - 16 characters, Alpha-Numerics only - upper and lower case.
DB connection mode is shared.
The full connection string is: (Identifiers removed - but can guarantee there are no unusual characters or symbols at all)
Filename=C:\users<username>\appdata\roaming<publisher><feature>\autosave.db;Password=;Connection=shared
No surrounding quotation marks, no internal quotation marks, symbols, irregular characters etc.
The Connection string works fine on over 100 user machines since the version was released containing LiteDb a week ago.
The user in question installed the application, and on the first-run - the DB was generated, not long after opening the app, the app performs a GetAll on a collection in the DB - the GetAll call triggered the exception.
I have a copy of the DB and I get the same issue if I try to use my app to access the data.
I also get the issue if I try to connect to the DB using LiteDb Studio.
For the user - deleting the DB that was originally created and letting the app create a new one resolved the issue.
@lbnascimento - would the db be helpful to you at all in debugging the issue?

I have the same problem described by @alex-nicholas
Is there a solution?


Handle routes that may have trailing slash

Currently we are seeing a situation where hitting a route directly like example.com/about will work as expected, but example.com/about/ is hitting the _error.js file since the paths do not match.
This changes strips the trailing slash before doing path comparisons so that page will be found, then using canonical urls we can get search engines to understand as well.
I'm sure there are better solutions to this problem as well, but this has worked for us in the meantime.

@fuller Thank you for the PR!
I'd like to avoid regexes whenever possible in the Lambda Handlers, they could lead to DoS attacks, perf issues etc.
Would something like this work instead:
if (path[path.length - 1] === '/') {
  // strip last / char
  path = path.slice(0, -1)
}

Also a simple test case would be great!

I will hold this off until https://github.com/zeit/next.js/pull/6421#issuecomment-472414075 is resolved! At the moment the component behaves in the same way that the next server does so will keep it like that for now.

Closing as there is no activity in https://github.com/zeit/next.js/pull/6421#issuecomment-472414075 i


apk-info 1.0.3

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Hopefully, it will be OK now :)


Golang support

I'm getting this error after I tried out Intellij Ultimate then switched to CE:
Error:Go is no longer supported by the Bazel plugin with IntelliJ Community Edition.
Please install Ultimate Edition and upgrade to the JetBrains Go plugin

Am I missing something? Should the Bazel plugin work with CE and Golang?

Go support is built on top of the JetBrains Go plugin (
https://plugins.jetbrains.com/plugin/9568-go), which is only supported in
Goland and IntelliJ UE.
On Thu, Jun 18, 2020 at 1:27 PM Yasser Elsayed<EMAIL_ADDRESS>wrote:

I'm getting this error after I tried out Intellij Ultimate then switched
to CE:
"Go is no longer supported by the Bazel plugin with IntelliJ Community Edition"
Am I missing something? Should the Bazel plugin work with CE and Golang?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/intellij/issues/1920, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3KDEOQEAUZXAVPHNLHWADRXJE77ANCNFSM4OB7V36Q
.

I see, thanks. The "no longer" part is misleading then?
Closing.

We used to support the open source Go plugin, which worked across all IDEs
(and which the new Go plugin was built from), hence the 'no longer'.
The new plugin is much better though, and gets regular updates and
improvements, so we now use that instead.
On Thu, Jun 18, 2020 at 3:21 PM Yasser Elsayed<EMAIL_ADDRESS>wrote:

Closed #1920 https://github.com/bazelbuild/intellij/issues/1920.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/intellij/issues/1920#event-3459718611, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3KDEKOTF26FNW4RI4365DRXJSMXANCNFSM4OB7V36Q
.

Thanks for the update. IMO it goes against Bazel's open source philosophy. AFAICS Intellij UE is the only editor right now with Bazel + Golang support. :(

Hi @yebrahim it sounds like a limitation imposed by JetBrains/IntelliJ not Bazel. They could allow Go targets to be executed but the overall utility would be limited to what the IDE provides more broadly.


Increase the appearance of the form

Related Issue

Issue No. #210

Changes you made :

The UI of the form is changed to improve the appearance

Describe the changes :
The form's yellow portion was not in match with the remaining part, its size and position is corrected
The changes are needed to increase the appearance
Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Before Changes Screenshots :

Output Screenshots :

@HarshDDalwadi Plz mention the issue number correctly, you have mentioned PR number over here.

@HarshDDalwadi Due to inactivity this issue will be closed.