Some problems about quantization accuracy

There are several problems in the process of testing quantization accuracy
A resnet50 model file is downloaded from models zoo. The link is as follows:
https://www.xilinx.com/bin/public/openDownload?filename=tf_resnetv1_50_imagenet_224_224_6.97G_2.0.zip
According to the display in readme, the accuracy test officially given by Xilinx is as follows:

Q1:
As shown in the figure above, in the downloaded files, is the file in the quantified folder the model file after running quantization (vai_q)? Is the test accuracy result officially given a model file after quantification (vai_q)? Is this precision consistent with the model precision generated into XModel after executing compilation (vai_c)?

Q2:
During the compilation (vai_c) of Vitis AI, will there be a loss of accuracy when the model is compressed?
Q3:
If the above quantitative test results cannot represent the final accuracy results, are there other test cases?

Hi guy,
I am deploying my model using vitis-ai pytorch quantizer.The accuracy of the quantized model is quite similar to the original  model(tested both of two models on GPU). But, when i deploy it on FPGA, the accuracy of compiled model is greatly reduced compared to the quantized model. I am not sure how this problem comes from the begginning and how to solve it. Can you guys help me?
Thank all!

@jzymessi
A1:yes.  the file in the "quantified" folder the model file after running quantization (vai_q)?
A2:Yes, there will  be a different of accuracy when the model is compressed, but very a bit.
@qianglin-xlnx  maybe answer your question

@lebacuong
I have been facing the same issue. Were you able to find a solution?


[CH] to_json function result is diffrent from spark when the input is struct type

Backend
CH (ClickHouse)
Bug description
to_json function result is different from spark when the input is struct type, when query such by gluten select to_json(struct(name)) from test_tbl1, the result is

when query by spark, the result is

Spark version
Spark-3.3.x
Spark configurations
No response
System information
No response
Relevant logs
No response

Maybe fixed by https://github.com/ClickHouse/ClickHouse/pull/55956


PageKit Update Theme and Extension Error

Can not update themes or extensions in PageKit.
When clicking the update button for a theme or extension within the Admin area, the install loading bar is displayed blank and the loading icon spins and seems to timeout.
I do get an XHR Error in Chrome but have tested FF as well and cannot update.
Technical Details

Pagekit version: 1.0.2
Webserver: Apache
Database: MySQL
PHP Version: 5.5.31

Troubleshooting

[x] I have enabled debug mode: https://pagekit.com/docs/troubleshooting/debug-mode
[x] I have verified the server requirements: https://pagekit.com/docs/getting-started/requirements
[x] I have tried disabling all installed extensions
[x] I have checked the browser developer console for JavaScript errors

Please provide the contents of the failed XHR.

This request is still pending. Please wait until it's finished and provide the response (not request).

Please reopen if the issue still persists.


stopScan does not immediately stop scanning

Problem:
After calling stopScan the callback in startScan can still be called with new results (due to the async nature of how scan results are reported from native code). This can cause problems for applications that call stopScan, then continue to connect. If the application assumes no more scan results (reasonable assumption), it can still happen that there is more scan results and connect can in that case be called again, with an "already connected" error as result. This is not the expected or desired behaviour.
Solution:
Implement a flag in ble.js that controls when scan results are reported. Enable the flag when calling startScan, and disable it when calling stopScan.

Fixed by this pull request: https://github.com/evothings/cordova-ble/pull/118

Fixed, part of pull request https://github.com/evothings/cordova-ble/pull/118


Build error after update to Angular 14 with 'single-spa-angular' v7.0.0

Issue:
Error: node_modules/single-spa-angular/src/public_api.d.ts:2:34 - error TS2307: Cannot find module './single-spa-angular' or its corresponding type declarations.
2 export { singleSpaAngular } from './single-spa-angular';
Steps to reproduce:
Step1: Created a brand new Angular 14 application.
Step2: Installed single-spa into application of version '5.9.3' using command 'npm i<EMAIL_ADDRESS>Step2 : Installed single-spa-angular with version 7.0.0 using command 'npm i<EMAIL_ADDRESS>Step3: Tried to compile the application using 'ng serve'
Step4: compilation failed with below error:
Issue:
Error: node_modules/single-spa-angular/src/public_api.d.ts:2:34 - error TS2307: Cannot find module './single-spa-angular' or its corresponding type declarations.
2 export { singleSpaAngular } from './single-spa-angular';
Expected Behavior
Application should be build/compiled successfully
Actual Behavior
Build error in application
Analysis:
When i did 'npm i single-spa-angular' it downloaded single-spa-angular having src file only below 4 items
node_modules\single-spa-angular\src\extra-providers.d.ts,
node_modules\single-spa-angular\src\prod-mode.d.ts,
node_modules\single-spa-angular\src\public_api.d.ts,
node_modules\single-spa-angular\src\types.d.ts
but if we see the code of public.api.d.ts file it has reference for singleSpaAngular as below
export { enableProdMode } from './prod-mode';
export { singleSpaAngular } from './single-spa-angular';
export { getSingleSpaExtraProviders } from './extra-providers';
but there is no file downloaded with name singleSpaAngular in node_modules\single-spa-angular\src location.
When i check the GIT location https://github.com/single-spa/single-spa-angular/tree/master/libs/single-spa-angular/src i found the the file single-spa-angular.ts which didn't come with npm install.
I did the work around. Just copy pasted this file manually and Issue Fixed !!! Application started.
But definitely there is some issue in npm install.

As I told you in the previous issue I can’t do more than guessing what might happened. I also asked to re-open the previous issue if you have a reproduction provided, but not opening the new one since it’s impossible to track the same issue with multiple ones.

I have published 7.0.1. This has been a regression in ng-packagr itself. I closed the issue because I wasn't able to replicate this behavior in the current workspace. Sorry for the inconvenience.


Add user selection when creating case

We have added the ability to assign a user to a case when reassigning a case, now we should add the ability to assign a user to case when creating the case.
The plan of action is to add the user dropdown field to the modal used to create cases, similarly to how we added it to the modal for reassigning a case.

@justinvdm Ready for review.
@rowanseymour pinging you here too. It should be quite similar to the previous PR.

:+1:


Stop using Dimension traits for inverse transform

What is the impact overall ?
Also we might rediscuss how we dispatch the arch-specific functions sooner or later.

It shaved a few seconds (1% off a clean build) off of compile times and doesn't seem to impact runtime.

It should reduce the binary size as well.

30.4MB to 29.4MB. not bad.

Unrelated to this PR: For release binaries we should run strip also. Since we build with debug symbols, that can reduce the binary size by 90%. (Even with default settings for Cargo, strip still gives 50% binary size reduction.)

I assumed we split the debug information in the action already. I guess might be good to document it.

I went and measured performance without the inverse assembly and there doesn't seem to be a performance impact. I changed the commit message to include this info.

Thank you :)


Prevent polluting prototypes with enumerable props

Fixes https://github.com/TranscryptOrg/Transcrypt/issues/816
Assigning functions directly to the prototypes of Number, Array, String, etc., causes them to be enumerable, meaning they can show up in "for .. in .." loops and cause unexpected behavior!
Instead we can use Object.defineProperty (for which there is already a helper called __setproperty__). With this method, i) new props are non-enumerable by default, and ii) we can avoid unnecessarily reassigning the same methods, in case there are multiple instances of Transcrypt running.

Put dict methods on the dict.prototype
I noted that list methods get assigned to Array.prototype, str methods get assigned to String.prototype, etc.
dict methods do not get assigned to Object.prototype (I am guessing this is for a good reason)
The current code has the dict methods being to each new instance in the constructor. It is tidier to declare them on the dict prototype and then create each dict instance from that.
This is also more efficient because properties placed on the dict prototype need only be declared once, as opposed to being declared for each dict.

The properties & prototype chain for an example dict foo:

Before (on master):

After changes

This is one I've had an eye on to tackle at some point sooner than later, so thanks for looking into it.  I can't dig into it right at this moment, but are there any possible unintended side effects by changing this that you can think of?

If for some reason, someone was expecting the methods of str or list being included while iterating over an Array or String, they would no longer see those.
for (let e in []) {
  console.log(e)
}
// on master, prints all the methods of py `list`
// after this PR, prints nothing

I don't know why someone would be relying on that behavior though.


Selected Date usually does not scroll to selected date

It sometimes work but mostly does not work. Is there any fix for this ?

let min = Date()
let max = Date().addingTimeInterval(60 * 60 * 24 * 4)
let picker = DateTimePicker.show(minimumDate: min, maximumDate: max)


libkrunfw: 3.8.1 -> 3.9.0, libkrun: 1.4.8 -> 1.5.0

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Works as expected modulo https://github.com/NixOS/nixpkgs/issues/212728

Result of nixpkgs-review pr 212723 run on aarch64-darwin 1

3 packages built:

krunvm
libkrun
libkrunfw


Testing, please ignore. (myhhau)

Bug bounty test - please ignore.... Please DO NOT APPROVE THIS! (myhhau)

This is phishing, trying to execute malicious code.

This account is trying to inject this malicious code in many google and mozilla repos (699 as of now), including Temporian (https://github.com/google/temporian/pull/85).


Write a service to run coav-control on startup for Intel Aero

It worth reminds us that our "product" is the library and not coav-control.  coav-control is great for development but too naive (a infinity loop as mainloop) to be a final product. We should state very clearly that this "service" is an example of how to automatically run a coav app on a drone.

PR #75


Where can I get a list of excel styling properties

Hello,
Where can I find out the different properties that can be used for styling the excel sheet?
The XLSXML Wiki page provides a few examples, but I couldn't find a page with a list of properties.
https://github.com/agershun/alasql/wiki/XLSXML
Can someone help?
Thanks

Hmm. Its a good question.

I have the same question. It will be really helpful if there is a document list all the styling properties that we can use in alasql


Adding the pipeline for the task explanation and Llm

📝 Description

I added the Task explanation and built the pipeline for the future LLM that will categorize and explain the image.
I modified the visualization such the explanation can be seen in the image outputted.

✨ Changes
Select what type of change your PR is:

[ ] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🔨 Refactor (non-breaking change which refactors the code base)
[X] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 📚 Documentation update
[ ] 🔒 Security update

✅ Checklist
Before you submit your pull request, please make sure you have completed the following steps:

[ ] 📋 I have summarized my changes in the CHANGELOG and followed the guidelines for my type of change (skip for minor changes, documentation updates, and test enhancements).
[ ] 📚 I have made the necessary updates to the documentation (if applicable).
[ ] 🧪 I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).

For more information about code review checklists, see the Code Review Checklist.

Hello, I'm looking to make this contribution as effective as possible, so please feel free to share any suggestions, concerns, or ideas you might have.
Thanks in advance for your time and input!


Content expiration time

When creating new post, its time to live should be set - 5/15/30min, 1/6/12 hours, 1/2/3 days.
This time to live will be used for filtering up to date content.

Implemented but needs to be optimized as it's currently client side due to firebase restrictions.


Cannot access this within #equals block

Dynamic template (handlebars)
{{#each sections}}

{{ insert this.type }} {{ insert this.name }}

  {{#equals this.type "industry" }}

Within equals type industry

      type {{ insert this.type "default=nothing" }}
      (should be industry)

  {{/equals}}

  {{#if this.industry}}

Within if industry

type {{this.type}} (should be industry)

  {{/if}}
{{/each}}

Data
{
  "sections": [
    {
      "industry": true,
      "type": "industry",
      "name": "Insurance"
    }
  ]
}

Expected content
industry Insurance

Within equals type industry

type industry (should be industry)

Within if industry

type industry (should be industry)

Actual content
industry Insurance

Within equals type industry

type nothing (should be industry)

Within if industry

type industry (should be industry)

Link to page:
https://sendgrid.com/docs/for-developers/sending-email/using-handlebars/#equals

The issue is that "this" is relative to the block you are using it in. So "this" has the context of the #each block, but loses scope when used inside the #equals block. You can add "../" in front of "this.type" like "../this.type" (or "../type") in the #equals block to access the parent #each block's scope. I have found this is the case for SendGrid's newer handlebars helpers like equals, notEquals, greaterThan, and lessThan, but for whatever reason the #if helper does not create a new context, so you can still use "this" without changing scope like you demonstrated


Narrow message body

I want to point out that Message Body on Contact us page is too narrow. Message Body should be bigger (as in prosilver) because it will contain more or less text, the other fields only need one line.

Bug fixed in version 1.5.4.


Multi-context safety

When enabling Electron's app.allowRendererProcessReuse option I think fsevents makes the whole thing crash once the app is reloaded, causing fsevents to be loaded again I guess, and a filesystem event is triggered.
The issue appears to be that fsevent is not multi-context safe.
More info on this: https://github.com/electron/electron/issues/18397#issuecomment-613754729 and https://github.com/electron/electron/issues/18397#issuecomment-613793288

Which version are you working with (fsevents & electron both)?

fsevents: 2.1.2
electron: 8.2.2

I did a bit of research and it seems the multi-context things isn’t quite the issue. It’s the use of event-loops and static things which are allocated by process (rather than per thread).
So it’ll take a bit to unwind that.

Stop using static variables (instead use per context ones)
Ensure that the eventloop syncing can survive that (given that it means not using the standard loop anymore(

Can you please try v2.2.0 which is currently tagged as next (but will likely move to latest shortly)

@pipobscure Sorry I was using fsevents via chokidar but I'm not using chokidar anymore.


Initial Adoption of Continous Deployment using semantic-release

To ease the modification/contribution to publish pipeline there has been some discussion of adopting a continuous deployment system for AtomLinter packages. To accomplish this we are making use of semantic-release and @semantic-release/apm-config.
This pull request has been dynamically created using a script. While the result is not perfect, it does accompish most of the grunt work of adopting continous deployment. There is some reconciliation that needs to be done before this can be merged.
Among the things needed to be reconciled, an APM Token and GitHub Token will need to be added to this repo's continous integration (most likely Travis CI) configuration for automated deployments to work.
Again, this Pull Request has been created by a script made by @keplersj. Please mention him if something has gone wrong, and he'll be happy to help.
ref: AtomLinter/Meta#18
cc: @Arcanemagus

I've already used semantic-release for some projects of mine (atom-coverage is a public one). I agree it really helps with automatic releasing/deployment/publishing.
That said, I've had some issues with it in the past, which leads me to provide some insights about this PR:

npx is a NPM package that is not listed in the project dependencies (neither development nor production) and it's not installed globally on TravisCI. Maybe I'm wrong, but I think it should be installed in a before_script key to have it available prior to executing npx semantic-release on the deploy stage.
We should add the semantic-release badge to the README.md too, so anybody that reaches the project knows immediately that s/he must stick with it.
@commitlint/config-conventional enforces conventional-commits. Since semantic-release can be configured to support a variety of conventions, I think we should remark it too with another badge.
I still have troubles with auto-updating the changelog on deployment in my projects. Just commenting it, I suppose @keplersj 's PR already manages it :wink:
I think the current TravisCI for this PR fails because of old config (60s vs the current 5min after merging #83), so it has nothing to do with this PR itself.

npx is a NPM package that is not listed in the project dependencies

This PR is just a starting block, there are many additional changes that are required before it can be merged. This is an example of a simpler complete version of this PR: https://github.com/AtomLinter/linter-phpmd/pull/118, while alternatively it can get as messy as https://github.com/AtomLinter/atom-linter-perlcritic/pull/51 + https://github.com/AtomLinter/atom-linter-perlcritic/pull/52 + https://github.com/AtomLinter/atom-linter-perlcritic/commit/7d1b63063fe85c6077f5487ec788abcd4d561388 to get fully working 😆.

We should add the semantic-release badge to the README.md too, so anybody that reaches the project knows immediately that s/he must stick with it.
@commitlint/config-conventional enforces conventional-commits. Since semantic-release can be configured to support a variety of conventions, I think we should remark it too with another badge.

Great ideas! Note that part of this is that commits can't be merged without fitting into the commit standard, and assuming the dev runs npm install a precommit hook is installed that checks the message before it even gets into the local repo.

I still have troubles with auto-updating the changelog on deployment in my projects.

Interesting, I've yet to see an issue with this on the ones done so far.

I'm happy to take care of getting this merged and working, just wanted confirmation from the people working on it that the Conventional Commits standard wouldn't be an issue 😉.

Cool, deploy step worked with no further changes needed 😀.


Update 66f1afbd9998e9c985d8e73b.md

issue [HACKTOBERFEST 2024] - Bash Scripting Quiz #56232 has been solved according to given requirements
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56232

Hi there,
Thanks for creating this pull request.
The linked issue has not been opened for contribution. We will not be able to review your pull request.

this issue is open for hacktoberfest, since i picked it from there only

@Anurag127001 as there is written inside the issue, the issue will be open during the freeCodeCamp event for Hacktoberfest in a few hours


feat: add a new statement in DefaultEffector.lua function DefaultEffector:mergeEffects

add a some(where (p_eft == allow)) || !some(where (p_eft == deny)) statement in DefaultEffector.lua and add some comment
Signed-off-by: 蒋振邦<EMAIL_ADDRESS>

@Edmond-J-A Hi, I think it is useless , refer to : #756, it is recommended to use strategy priority #550.

got it


Implement GitHub PR Issue Checker Workflow

Closes #146
Usage:

When a pull request is created or edited, the workflow automatically triggers.

The workflow checks for:

A non-empty PR description.
A valid issue reference format (FIX #<issue-number> or FEAT, or DOC #NEW).

If either check fails, the PR will be marked as failing, and the author will receive clear error messages in the PR comments.
If all checks pass, a confirmation message will indicate that the PR is valid.

Note: Kindly add all the labels added to the issue, to this PR!
@rohitinu6

Add labels!! And assign me!

@smog-root mentors will review and merge the PR

@smog-root mentors will review and merge the PR

You can add the labels right??

@smog-root Please link the issue using keywords. (fixes/closes..) You can find the detailed info about the same here:
https://docs.github.com/en/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword
If needed, please check out merged PRs for reference.
Thanks & regards.

@smog-root mentors will review and merge the PR

You can add the labels right??

I will add the labels after my review.

@Mayureshd-18 changes done! ✅

You can review now I added close
closes
closed
fix
fixes
fixed
resolve
resolves
resolved

@smog-root Don't change the workflow jobs. I meant to change the PR description to link the issue by using keywords. (fixes/closes). Please check other PRs that are merged for reference.

@Mayureshd-18 is it okay now!??

@Mayureshd-18 is it okay now!??

@smog-root  Edit your first comment.
The word 'closes' is styled as **\closes**.
Remove the asterisks.

@jvedsaqib done 👍

@Mayureshd-18 it's level2 on issue!

don't decrease the level!

Add level2 label @rohitinu6

@MastanSayyad why you removed level label?? Add level1!!


Image export script returns error

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Download latest data.json (from dev-v2.6 branch) in path $HOME/bin/data.json
Clone latest rancher/charts in rancherpath and checkout origin/dev-v2.6
Clone latest rancher/system-charts in rancher path and checkout origin/dev-v2.6: HOME=$(pwd)  REPO=rancher TAG=dev go run pkg/image/export/main.go system-charts charts/assets rancher/rancher:master-head rancher/rancher-agent:master-head

Result:
Error similar to the following is encountered: 2021/06/14 17:28:46 Image [calico/cni:v3.18.1] does not start with rancher/ exit status 1
Other details that may be helpful:
This is because rke2 1.21.1-rc2 has invalid images in its rke2-images-all.linux-amd64.txt file. Updating kdm to use rc3 should resolve this issue https://github.com/rancher/kontainer-driver-metadata/blob/dev-v2.6/channels-rke2.yaml#L11.

Related issue: https://github.com/rancher/rancher/issues/32403

This is awaiting a Rancher v2.6 RC for QA validation. Validation should pass if the images list contains all expected images in the RC.

Available to test in v2.6.0-rc1

@cmurphy , please validate this one when you have time.

Validated on v2.6.0-rc1.
Validation steps: from rancher source code, ran:
git clone -b dev-v2.6 git@github.com:rancher/charts
git clone -b dev-v2.6 git@github.com:rancher/system-charts
CATTLE_KDM_BRANCH=dev-v2.6
curl -sLf https://releases.rancher.com/kontainer-driver-metadata/${CATTLE_KDM_BRANCH}/data.json > bin/data.json
HOME=$(pwd) REPO=rancher TAG=dev go run pkg/image/export/main.go system-charts charts/assets rancher/rancher:v2.6.0-rc1 rancher/rancher-agent:v2.6.0-rc1

Result: command executes successfully, outputs rancher-images.txt with valid image list.


feat(proto): add canary protos

feat(proto): add canary protos

docs(config): update README with canary migration instructions
The existing canary section of the halconfig is difficult to represent in proto. Let's require users to migrate each integration block they care about from an item in a list to an entry in a map.

Sorry, forgot to rebase off your protoc version bump, which explains why make checkproto was passing locally but failing on the CI server. Should be good to go now!


Menu clarification on Quickstart Workload pages

URLs this PR updates:

https://rancher.com/docs/rancher/v2.x/en/quick-start-guide/workload/quickstart-deploy-workload-ingress/
https://rancher.com/docs/rancher/v2.x/en/quick-start-guide/workload/quickstart-deploy-workload-nodeport/

If you had already clicked around in the cluster before you started this guide, when you do step 3 you will default to the last page you had open. That might be inside the Apps menu item, or Workloads->LB. We should clarify that they want to be on Workloads->Workloads, even though its a bit redundant.

Looks good to me


Add a path reminder that temporarily saves the discarded commit message

Resolves #682

Keep and display original message for git hook
Just report the tmpfile path, basically ".git/COMMIT_EDITMSG"

I think this is needless because of the typos displays the given path .git/COMMIT_EDITMSG in the error...
> git commit -m ':) minimumn typo example # this commit message includes typo' --allow-empty
error: `minimumn` should be `minimum`
  --> .git/COMMIT_EDITMSG:1:4
  |
1 | :) minimumn typo example # this commit message includes typo
  |    ^^^^^^^^
  |


The 'python_requires' field is ignored in setup.cfg (not in setup.py).

I'm using setuptools version '41.0.1.post20190616' (commit 8aeff6b2c9a64e47ad2a22533d7e65c08cd4103f).
$ cd /tmp/setup/py
$ cat setup.py
from setuptools import setup
setup(python_requires='invalid')
$ python setup.py install
error in setup command: 'python_requires' must be a string containing valid version specifiers; Invalid specifier: 'invalid'
$ cd /tmp/setup/cfg
$ cat setup.cfg
[metadata]
python_requires=invalid
$ python setup.py install
running install
running bdist_egg
running egg_info
writing UNKNOWN.egg-info/PKG-INFO
writing dependency_links to UNKNOWN.egg-info/dependency_links.txt
writing top-level names to UNKNOWN.egg-info/top_level.txt
reading manifest file 'UNKNOWN.egg-info/SOURCES.txt'
writing manifest file 'UNKNOWN.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
warning: install_lib: 'build/lib' does not exist -- no Python modules to install
creating build/bdist.linux-x86_64/egg
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying UNKNOWN.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
zip_safe flag not set; analyzing archive contents...
creating 'dist/UNKNOWN-0.0.0-py3.7.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing UNKNOWN-0.0.0-py3.7.egg
Copying UNKNOWN-0.0.0-py3.7.egg to /tmp/setup/cfg/v/lib/python3.7/site-packages
Adding UNKNOWN 0.0.0 to easy-install.pth file
Installed /tmp/setup/cfg/v/lib/python3.7/site-packages/UNKNOWN-0.0.0-py3.7.egg
Processing dependencies for UNKNOWN==0.0.0
Finished processing dependencies for UNKNOWN==0.0.0
$ echo $?
0
I thing "python setup.py install" should also fail when using setup.cfg.

https://github.com/pypa/setuptools/issues/999 was about a similar thing, so maybe this is a regression.

Note that python_requires belong in the [options] section. Still the same issue though.
It's not a regression, but something that should have been taken into account when implementing #999.

Related: #1633.


confirmCardPayment method needs to be made available.

https://stripe.com/docs/payments/accept-a-payment#web-submit-payment
confirmPaymentIntent got deprecated and now confirmCardPayment has replaced it.

For anyone stuck on this, you can access it from the Stripe instance returned from the package while you're waiting for this to be made available directly.
import { instance as StripeInstance } from "vue-stripe-elements-plus";

StripeInstance.confirmCardPayment();

This is working currently
import { instance as StripeInstance } from "vue-stripe-elements-plus";
.....
.....
StripeInstance.confirmCardPayment(paymentIntent.client_secret, {
            payment_method: {
              card: this.$refs.stripeCard.$refs.element._element,
              billing_details: {
                name: paymentIntent.user_name,
                email: paymentIntent.user_email_address
              }
            }
          }).then(result => {
            console.log(result);
            if (result.error) {
              // Show error to your customer (e.g., insufficient funds)
              console.log(result.error.message);
            } else {
              // The payment has been processed!
              if (result.paymentIntent.status === "succeeded") {
                // Show a success message to your customer
                // There's a risk of the customer closing the window before callback
                // execution. Set up a webhook or plugin to listen for the
                // payment_intent.succeeded event that handles any business critical
                // post-payment actions.
              }
            }
          });
        }

For v1.0.0:
const StripeElements = this.$refs.elms // vue component instance
StripeElements.instance.confirmCardPayment()


KeepassXC - Free, open source, cross-platform password manager. [Open-Source Software]Freeware

I think there is a Typing error:
Arround the Open-Source  Symbol are [....]
KeepassXC - Free, open source, cross-platform password manager. [Open-Source Software]Freeware

@fishooo0815 thx!


[parent] Light client integration

APIs

Fetching states and schnorr signatures from DA nodes

Consensus states
Schnorr signatures
Do we have the corresponding DA interfaces?
Prover also needs to store the stake table (from L1) for checking DA nodes’ Schnorr signatures

Proof generation
Submit proofs to the contract

Deployment (ask Mat)

Tasks

[ ] #2021
[ ] #1987
[ ] #1988

High Uncertainty: fallback + robustness (malicious withholding leader etc)

Idea: the replicas monitor the lightclient contract and delete their Schnorr signatures once the contract is up to date. Honest leaders monitor the lightclient contract as well and call the replicas/ state prover service as needed until catchup is achieved.

Sorry, that was outdated description, I copied the notion section over. @philippecamacho
We don't have to deal with garbage collection in our work


Add filters to read auth fns

Right now, read permissions can only be defined on a per-document basis (not per-key). Sometimes you have to split information into multiple documents to support granular permissions. It would be pretty easy to let read auth fns return a filtered version of the given document based on who the client is.

Biff no longer has auth fns


⚠ Bofrost.lu has degraded performance

In 5a7ab06, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1039 ms

Resolved: Bofrost.lu performance has improved in 43c902a after 6 minutes.


python3Packages.bashlex: 0.14 -> 0.15

Motivation for this change
Update to latest upstream release 0.15
Things done

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

[x] 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/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129045 run on x86_64-linux 1

4 packages built:

python38Packages.bashlex
python38Packages.compiledb
python39Packages.bashlex
python39Packages.compiledb

Result of nixpkgs-review pr 129045 at 37b421ae run on aarch64-linux 1

4 packages built successfully:

python38Packages.bashlex
python38Packages.compiledb
python39Packages.bashlex
python39Packages.compiledb


Trouble installing: "No matching distribution found for Mopidy-PiDi"

Hello.
I am trying to install mopidy-pidi using the install script of mopidy. However, I am getting the following error:
Installing Pirate Audio plugins...
Collecting Mopidy-PiDi
  Could not find a version that satisfies the requirement Mopidy-PiDi (from versions: )
No matching distribution found for Mopidy-PiDi

I am using python 3.5 on a raspberrypi that has octopi installed (which I suppose uses raspbian).
Regards

By the way, I am using the audio AMP shimp, so maybe this library is not needed?

You shouldn't need this library with AMP SHIM, since it's for displaying album art on a screen.
New-ish versions of Mopidy require Python >= 3.7, so you're going to have some trouble installing on 3.5. You'll need a newer OS.

Thank you.
I just ran the installation script on another raspberry pi and had no problems.
So this was not a problem before (because it was not needed) and it is not a problem now.
Regards


ps:scale does not understand the new dyno types

There seems to be a bug in trying to scale applications using the ps:scale command with the new dyno types.
I keep hitting this error:
$ heroku ps:scale web=1:Standard-2X worker=1:Standard-1X --app appname
! Usage: heroku ps:scale DYNO1=AMOUNT1[:SIZE] [DYNO2=AMOUNT2 ...]
! Must specify DYNO and AMOUNT to scale.
$ heroku ps:scale web=1:standard-2X worker=1:standard-1X --app appname
! Usage: heroku ps:scale DYNO1=AMOUNT1[:SIZE] [DYNO2=AMOUNT2 ...]
! Must specify DYNO and AMOUNT to scale.

This syntax works with the old 1X, 2X, PX dynos, but not the new names.

In some cases I can use ps:type to change the type, eg:
$ heroku ps:type --app foo-bar-app Free
Changing dyno type... done.
dyno  type  qty  cost/mo
----  ----  ---  -------
web   Free    1        0

But in some cases I can't:
$ heroku ps:type --app other-similar-app Free
Changing dyno type... failed
 !    Can not set dyno type on this app. Please use `heroku ps:scale` to change process size and scale.

What's the right approach?

More info in my case - once I use the web interface to change the dyno type (even if I change it back to the original value) then heroku toolbelt works on it. So if I go to other-similar-app from the example above and set the dyno type to Hobby, save, then set it back to Free and save and then run the command again it works.

I deleted my earlier comments - they were about an unrelated issue (you can't use heroku toolbelt to change the dyno type until you take an action in the web ui first).
However - I believe this issue is caused by a failure in the regex here:
https://github.com/heroku/heroku/blob/master/lib/heroku/command/ps.rb#L232
My guess is that the \w+ needs to be [\w-]+.

fixed in 3.37.4


You need to update the install scripts for the new Sickrage

There is a new version of sickrage and your script ( awesome I LOVE IT ) does not take this into affect

wow you answered over here as well.  Yes I am now on dockstarter.   A bit of learning curve but I will get it


🍒 Cherry pick PR #9632 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/9632 to staging 🍒

🚀 Cherry-picked to staging by @roryabraham in version: 1.1.79-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @roryabraham in version: 1.1.79-17 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


method ToString and Equals for each FB

Hi,
Should this kind of issue be reported in the forked repo (TcOpenGroup/TcOpen) or here? It is not really clear...
Is your feature request related to a problem? Please describe.
In my code, it gives times that I need to compare 2 FBs together. Since we are in POO, it might be that an object is equal to another even if there are diffrent instances.
I also like to have, in my HMI, the to_string represantation of the reference of the FB. Ex: My mover have a ref to FB_PieceHolder, a ref to FB_MoverAxis, a ref to FB_Station and a ref to FB_Process. When I select a Mover on my HMI, I would display which reference is currently in use by this mover. By calling MyMover.MyPieceHolder.ToString(), it will return the string made from MyMover.MyPieceHolder. Really nice and really usefull.
Maybe we can include more standard method (like the standard object class method in c#)...
Describe the solution you'd like
Create or add those method in the TcoObject.TcPOU Abstract FB
For the .Equals, see "addition context"
For the ToString method, I have tried:

using var := __VARINFO(THIS^); ToString := var.System; // Generate error since __VARINFO is not possible on THIS statement
using attribute {instance-path} on a local variable _sToStringVar but it does not exactly what I will do
Passing a string in FB_init wich will be used by the ToString method. Not really convinient but the best solution I found until now...
Create a _iInstanceCounter in the VAR_STAT region of the FB that incremant on each FB_Init call, and then: ToString := CONCAT('FB_Name', TO_STRING(_iInstanceCounter)). This is okay when we don't dynamically create and delete object ...

Describe alternatives you've considered
Add it in the interface... But since those method can be overloaded by the subclass, they really should take place in the abstract class.
Additional context
see here for more detail of the implemtation
https://docs.microsoft.com/en-us/dotnet/api/system.object?view=net-5.0
Also my discussion around the Equals method (look a bit under):
https://github.com/Barteling/STweep/issues/38
Also take a look of the implementation of a Equals method here (use the search and write: METHOD PUBLIC AssertEquals)
https://github.com/tcunit/TcUnit/blob/master/TcUnit/TcUnit/POUs/FB_TestSuite.TcPOU
And I would like to join the slack chanel... but I am not a member of the group so I cannot join ...

Hi @GISED-Link it is OK reporting here... we PR all here to TcOpenGroup  later.
Equals(ITcoObject)
TcoObject contains _identity which is the address of that object assigned at construction FB_init. That should be unique for each instance.
TcoObject
https://github.com/Inxton/TcOpen/blob/492b74095c07195d2958b25f17b7e204e1b6d2ff/src/TcoCore/src/XaeTcoCore/TcoCore/POUs/Prototypes/TcoObject/TcoObject.TcPOU#L111
ITcoObject
https://github.com/Inxton/TcOpen/blob/492b74095c07195d2958b25f17b7e204e1b6d2ff/src/TcoCore/src/XaeTcoCore/TcoCore/POUs/Prototypes/TcoObject/ITcoObject.TcIO#L13
I guess we can use that to implement the equality method.
@TomKovac could you have a look at that plz? This looks like useful idea...
ToString()
it looks a bit trickier, we could have constant string with type name, that should also include namespace and maybe be even library id... (we could get into troubles with type names in dependencies  on larger projects) What I see as a bit of trouble is that we will need to maintain that type id string when refactoring... that may lead to inconsistencies.
I would like to investigate if we can workaround _VARINFO(THIS^) issue. @TomKovac will we check that?
Will contact Dustin about slack channel...

@PTKu I'm not OK with your thinking about the Equals method. Imagine I have a FB_Pos2D which contains a X and a Y position. Only comparing if 2 instances are the same is not enough because I really want to know if X and Y coordinate are equal or not. We can overload each Equals method to compare the content, but in my opinion this should be done by default in the abstract class (and in this case, ignore the _identity).
By the way, why is _identity used for? Since we can get this value by using THIS or ADR(my_instance_of_FB)? I already have implemented a method called Equals which do:
METHOD Equals : BOOL
VAR_INPUT
    object : POINTER TO FB_SameFB_Type;
END_VAR
// code section
Equals := FALSE;

IF object <> 0 THEN
    Equals := THIS = object;
END_IF

And that works perfectly... It just don't check the content but in my case this is not needed, yet.
About the ToString, it is really tricky... but it shouldn't. The best way will be that Beckhoff (or Codesys) do something with its compiler and provide a built-in method, like it is the case for Enum when we add the {Attribute to_string} decorator.

@GISED-Link sorry I missed you point...
Comparing content needs method override as you say. The input parameter will need to be ITcoObject. Then for the comparison we will need to cast that object to appropriate interface for the comparison (should work, not sure, I will check it over the weekend).
_identity is there to track objects on the higher level applications.

@GISED-Link something like this?
TcoObject.Equals override
FUNCTION_BLOCK TcoObjectEqualsTest EXTENDS TcoCore.TcoObject
VAR
	_SomeNumber : INT;
	_SomeString : STRING;
END_VAR
//---------------------------------------------------------
METHOD PUBLIC Equals : BOOL
VAR_INPUT
	Object : ITcoObject;
END_VAR

VAR
	_object : POINTER TO TcoObjectEqualsTest;
END_VAR
//----------------------------------------------------------

IF(__QUERYPOINTER(Object,  _object)) THEN
	Equals := _object^._SomeNumber = THIS^._SomeNumber AND _object^._SomeString = THIS^._SomeString;
END_IF;

default implementation of TcoObject would return reference equals like result

ToString() deserves more discussion. If we are to copycat .net, then ToString() in default implementation does return string representation of type. However many classes have their own override of ToString() returning something different than type. I would rather implement GetType() method for getting information about the type... what's your take on that...
I guess there will be workarround the __VARINFO issue, I will check that next week.

@GISED-Link with ToString()/GetType thing I am out of ideas at this point. I did try several things with no success. Let's keep this open for the moment I will ask around about other options. I would rather not take the path of passing parameters at construction or anything one must do manually to get type name.

@PTKu I'm at the same point as you... I've trying one or two more things without any success. Maybe we can try to add an {attribute}... I've seen some json library that use this kind of things. Maybe it can only be used by Tcom object written in c++...
For the Equals method, it seems ok. I still  don't really understand why the attribute _instance is prefered than THIS, but it is ok

Implementation of equality method in #38
I am creating separate issue for GetType()/ToString() in #37


Jython start-up performance optimization

Originally submitted to Google Code by @mkorpela on 25 Jul 2011

Currently Robot Framework acceptance tests with Jython take about 5 times as much time as same tests with CPython. And I would expect that the difference in execution time is similar with other test sets.
I suggest that we measure where the time is consumed when running in Jython. We could afterwards code the time critical parts in a language that produces native JVM bytecode (for example in Scala or Clojure).

anyone tried to nailgun jython startup?

@badjolo Yes. It works wonders! This is what I did:

git clone https://github.com/facebook/nailgun.git
cd nailgun

To generate nailgun-server.jar (assuming Maven is in your PATH):

mvn -Dmaven.javadoc.skip=true package
cp nailgun-server/target/nailgun-server-1.0.1-uber.jar nailgun-server.jar

Run the server with:
java -cp "robotframework.jar;nailgun-server.jar" com.facebook.nailgun.NGServer
To run Robot Framework:

cd nailgun-client/py
python ng.py org.robotframework.RobotFramework

The first time it takes some seconds. Afterwards it is lightning fast.
Note: WIth JRE 11 (and maybe any version greater than 8) you will get some warnings regarding the use of reflection by Jython.


[WIP] remove dependency on ui name

So that tests relying on the name don't break

Very minor request: rename "uiName" to "headerTitle"?

I'm looking into selecting the div directly, skipping the need for another variable


Add attribute(s) to each generated style tag

I'd like to be able to add a custom attribute to each style tag with some sort of global config. I know the option is there for jss: https://cssinjs.org/jss-api/?v=v10.0.0-alpha.22#create-style-sheet

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
Should have a global config that will add a given style element attribute key and value to one or more generated style elements.
Current Behavior 😯
Does not have a global config that will add a given style element attribute key and value to one or more generated style elements.
Examples 🌈
I'd like to add the data-senna-track="temporary" to each style tag:

Context 🔦
We are using Liferay and it needs to know which style tags to remove on page navigation.

We are using Liferay and it needs to know which style tags to remove on page navigation.

I'm curious about that. What's the need?
This sounds like more a feature request for JSS than something we should handle. The style element is created in
https://github.com/cssinjs/jss/blob/0af646d31962518f9603493825d43f3410ff6c14/packages/jss/src/DomRenderer.js#L320
Have you considered using a MutationObserver on the head element to add your attributes?

The CMS is still fairly new to pure JS frontend development (They have always used JSPs in the past) so they haven't thought about JSS yet.
And I haven't considered a mutation observer but I will give that a try.
Thanks.


add samples for aad b2c with only spring security

Purpose
relate issue:https://github.com/Azure/azure-sdk-for-java/issues/23713
we can use spring security to realize functions provide by azure-spring-boot-starter-active-directory-b2c.
So we plan to deprecate the azure-spring-boot-starter-active-directory-b2c.
we add these samples to show that b2c can work will with spring security.
Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Other Information

add samples for AAD B2C work as web application.

We support Password reset and Profile edit by provide related uri instead of create client Registration
User can add logoutSuccessUrl to clear cache when logout
AAD B2C support reset password in login page:https://docs.microsoft.com/en-us/azure/active-directory-b2c/add-password-reset-policy?pivots=b2c-user-flow

add samples for AAD B2C work as web api.

Spring security support multiTenancy, custom validate audience, and custom JwtAuthenticationConverter

@chenrujun @michaelqi793
please help to review this pr when you have time🙏.

Closing this PR, use this issue to track this task: https://github.com/Azure-Samples/spring-boot-application-with-azure-active-directory/issues/2


[YUNIKORN-248] Remove VolumesPlugin from the core

Hey @manirajv06,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 52194de0-c078-11ea-8f0c-c508e01afe59

hi @manirajv06
Thanks for submitting the PR. Could you pls rebase to the latest master?
Recent merged PR caused some conflict for this change.
Thank you!

@yangwwei Thanks for review. Rebased the patch.

Hey @manirajv06,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 50b772c0-c2cf-11ea-bbc7-2b020bbf5e3a


Project file uses SRCROOT; should be PROJECT_DIR #690

This should have no effect for regular standalone builds, and enables embedded builds as a submodule in some build environments where SRCROOT is not per-project.

Slow for the slow response and thank you @deborahgoldsmith!


Delete duplicated item

Connected Machine agent version is duplicated in the bullet list.

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

#label:"aq-pr-triaged"

Thanks for the catch @tksh164 !
#sign-off


Pause and resume around the initial getInfo.

This should hopefully enable faster startup on fast downloads.

LGTM. Please add how it was tested to the description above.


[WIP] ENH: Logical AND of shape-mismatched sparse variables

Addresses #127.
Currently modifying input models to use "and_" instead of "and".
Approach is to verify that onsets, durations and amplitudes match before taking the intersection.

So I've thought about this some more and I'm kind of apprehensive about pushing a lot of logic into _transform inside the and_ class, as that stuff (i.e., alignment, validation, etc.) is supposed to all be handled by the base Transformation class in a generic way. Given that fixing the factor transformation to zero-pad events would solve the problem in this particular case, I incline to patch that instead. Does that work for you? (I'll still cherry pick the other fixes that map transformation names, etc.)

Sure, that works. Was just pushing what I've done in case it was useful.

I fixed factor and added a bunch of tests for both factor and the logical transformations. From my perspective this is ready to merge once tests pass, but let me know if you catch anything else.

This works for me.


How to change tolerance of the rcnn model?

Dear developers,
I trained the model and it showed good performance. Now I want to decrease the threshold for whatever that lowers the recall rate, making it more tolerable to the object deformation. May you give some advice?

You can adjust the default configuration, located in tools/cfgs/default.yaml
The score threshold e.g. for car-class is 0.3 in testing
# config of testing
SCORE_THRESH: 0.3
Also define your own parameters in the eval_rcnn.py script to overide cfg.RCNN.SCORE_THRESH to customize your PointRCNN setup
parser.add_argument('--threshold', dest='threshold', default=0.5, type=float, help='')

Thank you! Your tip is indeed appreciated!
I will close the issue now.


oxc-resolver as Jest custom resolver

Jest often has bad performance at least partially due to resolver performance

https://github.com/jestjs/jest/issues/10833

Could oxc-resolver help? I've tried to configure it myself but ran into some issues. Moreso curious if this could be helpful in large monolithic apps.

I don't know. I'm not seeing a consensus that the resolver is the actual bottleneck in the linked issue.
No actionable items from our side so close as not planned.


Backup plugin settings

Backup plugin settings
https://discourse.joplinapp.org/t/export-import-settings/14668

Currently only settings of plugins are saved which have been specified for the setting using storage: SettingStorage.File. As these are stored in the json and not in the DB. For all others there is currently no API from Joplin.

The following might work:
joplin.settings.globalValue(`plugin-${pluginId}.${settingName}`)

It would require hardcoding plugin IDs and setting names though.

Ok, but unfortunately this is not a real solution. Because you need to know all the settings. Furthermore, there is no easy way to import back into Joplin.


Nice Table Formatting broken when adding colored text

I am inserting in a dict this string
CSI="\x1B["
driverName = "Sebastian Vettel"
driverName = CSI + "31;40m" + driverName + CSI + "0m"

currentStanding[ driverPosition ] = driverName

and then using your method to print this dict
with output(output_type="dict", initial_len=1, interval=200, ) as currentStanding:
when I try to output raw string it is all nice, but when I add the escape sequence to print the colored name it prints two columns..
Thanks

I think is the escape sequence that influence the column width calculation, which cause the output refresh bug.
I will give it a try, not sure right now :/

kindly thanks for your reply and feedback .
you're a friend
Il 06 Feb 2018 2:22 PM, "Yinzo"<EMAIL_ADDRESS>ha scritto:

I think is the escape sequence that influence the column width
calculation, which cause the output refresh bug.
I will give it a try, not sure right now :/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yinzo/reprint/issues/9#issuecomment-363420334, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AYhR_3RcZbctlO_x7e-EuW5aH0V4HIRtks5tSFIJgaJpZM4R58Xi
.

reprint 0.5.1 has been released to PYPI, use pip install -U reprint to update the package.
It fix the problem that colored text breaks the text formatting.
Thank you for your report, feel free to let me know if there still have problems ;D


Show link text in problem report?

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Problem
Warnings output by remark-lint-no-dead-urls show the URL, but not the link text, making the report less useful than it could be:
45:3-45:116    warning Link to https://chrome.google.com/webstore/detail/wildfire/djhgeeodemlfdpmcccdekfalbhllcoim/ is dead                                                        no-dead-urls remark-lint

Solution
Showing the link text in the warning / error message:
45:3-45:116    warning Link to [Wildfire Chrome Extension](https://chrome.google.com/webstore/detail/wildfire/djhgeeodemlfdpmcccdekfalbhllcoim/) is dead                                                        no-dead-urls remark-lint

If the URL doesn't have link text, then the output stays as it is now.
Alternatives
Not formatting it like a Markdown link:
45:3-45:116    warning Link "Wildfire Chrome Extension" to https://chrome.google.com/webstore/detail/wildfire/djhgeeodemlfdpmcccdekfalbhllcoim/ is dead                                                        no-dead-urls remark-lint

@karlhorky don‘t you think this would be not so useful / pretty annoying for other cases?
This example seems to be about a confusing url with very clear link text. I get that it helps a bit for this case.
I’d imagine that typically URLs are very clear. And link text is often vague.
I’d think that more text is often, at best, not needed.
And also often, at worst: more confusing.

I don’t agree with the thesis of: no link text is not useful report.

Not sure that this is my position, maybe I need to re-word. I think there's a ton of value already provided, but there is important context missing, for certain use cases.

Is positional info not enough for you?

So I'll comment on a use case - maybe it's a common use case too?
Use case: running this on a large corpus of material which is growing and maintained by multiple people, with links on a periodic basis. Ideally, we'd love to have the following workflow, which would speed things up for us:

Look at the broken link in the failing CI run on GitHub Actions (no cloning / repo navigation)
Highlight the error message, which includes the URL and the link text, and right click -> Search on Google
Very often, this will yield already a good replacement for us. If not, we need to look deeper.
Repeat with multiple links at once, finding alternatives and compiling them before even opening code / finding the right file, etc
Finally, open code, paste all the changed links, commit, push, etc

Without this, it requires an extra step and more friction to open the file and check what the context is.
The title is minimal context to provide along with the error message, so I would argue it's good context to provide.
With a "verbose" option, I could imagine that it shows a snippet of the Markdown / MDX source with a pointer showing where the offending link is - but that's not this proposal here.

I’d think that more text is often, at best, not needed.
And also often, at worst: more confusing.

Strongly disagree with this approach regarding error message design - providing more baseline context in error messages is almost always worth it to users versus the additional amount of space it takes.
"Baseline" being that there is a dropping-off point where it stops making sense (but in those cases, if the info is important enough, then a link to an external error message with more details is often good).

This seems like an exceptionally bad URL that proves the rule (most URLs are good).

Not sure about the "most URLs are good" assertion - I would assert the opposite actually. That the vast majority of URLs are bad, and that there are some inspirational movement in the last 10-15 years towards URLs which are better. But both are just gut-feel assertions I guess, probably not objective details to make decisions on.

I believe the URL to be the most important part.

Agreed, maybe the formatting needs work. The title could also be moved to the end:
45:3-45:116 warning Link https://some-place.com/wildfire-chrome-extension/ (link text "Wildfire Chrome Extension") is dead no-dead-urls remark-lint

Or an option, default off.

And link text is often vague.

Interesting, I would agree - in my feeling, a bit less frequently than that I feel URLs are vague.
In our case, our link text is high quality, because we control it, so it's only the URLs that are vague.

I think you are looking for a different reporter.
You want a codeframe of the surrounding code.

Perhaps you can try with https://github.com/vfile/vfile-to-eslint and https://github.com/eslint-community/eslint-formatter-codeframe. Something like that.

Ok nice, thanks!
I guess this should be closed as wontfix then?

Also, why not do step 5 before 2/3/4. You perform the step anyway. Use your code editor. Have these lint tools in your editor. You can click on this lint message and go to the markdown. There you see [Wildfire Chrome Extension](https://some-place.com/wildfire-chrome-extension/), which you can then Google?
Take also a look at https://eslint.org/play/.
The error message is 'a' is assigned a value but never used., not 'a' (which holds value `'b'`) is assigned a value but never used..
And Strings must use doublequote., not Strings (in this case `'b'` in variable `a`) must use doublequote.
Other remark lint rules are similar to ESLint I think: https://github.com/remarkjs/remark-lint#what-is-this. Uniform error reasons. With perhaps a variable of what is wrong and what is right. But not auxiliary information.
Closing as wontfix, indeed! It sounds like a different reporter is what you are looking for.

Also, why not do step 5 before 2/3/4. You perform the step anyway. Use your code editor.

yep, that's the extra step / friction I mentioned above (friction = context switching):

Without this, it requires an extra step and more friction to open the file and check what the context is.

Take also a look at eslint.org/play.
The error message is 'a' is assigned a value but never used., not 'a' (which holds value `'b'`) is assigned a value but never used..

Agreed with your general point - there is a balance.
See also counterpoint - for years, browsers said "Cannot read properties a of undefined", which was missing information. Now in newer browsers, they changed it to show the missing information, and people were very publicly happy about that.
In a vacuum, remark-lint-no-dead-urls may not care about things that are not the URL. But maybe users do.

I don't see the context switching argument because you switch contexts already. Now from a to b. Then from b to a?
Re the browsers: what is the "good" example you mean?
As for some users wanting some things: sure. The systems in place are very extendable. Errors are objects: you get the node in question too. And reporters can report the source code if users want the source code.

I don't see the context switching argument because you switch contexts already. Now from a to b. Then from b to a?

It's one extra context switch (cli/editor -> browser -> cli/editor vs browser -> cli/editor)

Re the browsers: what is the "good" example you mean?

Can't find the exact one I mean, but here are 2 others:
> z = undefined
> const {a} = z
VM3326:1 Uncaught TypeError: Cannot destructure property 'a' of 'z' as it is undefined.

> const a = {}
> a.b()
VM2947:1 Uncaught TypeError: a.b is not a function

It's one extra context switch (cli/editor -> browser -> cli/editor vs browser -> cli/editor)

I don’t understand. CI fails. You context switch to your editor. You fix those URLs which are shown in your editor, by going through the Google for each URL. You have to be in your editor for saving the changed URL every time already. Push to origin. Done.

As for the error messages you show: I think they’re good messages. I see them as an argument for having good messages, for not showing “undefined”, for not saying “URL is dead”. I do not see them as an argument for showing more info next to the URL.

CI fails. You context switch to your editor. You fix those URLs which are shown in your editor, by going through the Google for each URL.

"by going through the Google" is the 2nd context switch. If it was right after "CI fails" (without switching to your editor), you eliminate 1 context switch.

Admittedly, I'm pretty picky about context switching. Not only because less of it makes me faster, but also because it confuses people less when watching me demonstrate something.
Workflow with only URL:

CI fails (browser -> GitHub Actions logs)
Context switch 1: to your editor to check the link text
Context switch 2: to browser to Google for the URL
Context switch 3: to editor to commit + push etc.

Workflow with extra context

CI fails (browser -> GitHub Actions logs)
Context switch 1: to new tab to Google for the URL
Context switch 2: to editor to commit + push etc.

@karlhorky Since you feel very strongly about what information you want presented.
A custom reporter sounds perfect for your use case.
You could even try to integrate it into an interactive terminal like https://www.warp.dev/ to cut out the opening the browser step.

That said I agree with @wooorm and @remcohaszing's comments, I don't see a major value add in including this, and see it as diverting away from the conventions that most lint/analysis tools follow.

@karlhorky your diagram of steps isn’t like that for multiple dead URLs.

In browser: CI fails. Switch to editor.
(LOOP) In editor: copy all the context you like. Switch to Google
In browser: find some URL you want, switch to editor.
In editor: paste URL, go to next dead URL, repeat at LOOP

It sounds like you don’t use the tools there. Only tools in CI.

We usually handle each URL individually, so the workflow + number of context switches doesn't change.
But even with batching multiple URLs, doesn't change much actually, there's still 1 extra added context switch:
Workflow with only URL:

CI fails with multiple failing URLs (browser -> GitHub Actions logs)
Context switch 1: to your editor to check the link text
Context switch 2: to browser to Google for the URL + text
Context switch 3: to editor to commit + push etc.

Workflow with extra link text context:

CI fails with multiple failing URLs (browser -> GitHub Actions logs)
Context switch 1: for each URL, to new tab to Google for the URL + text
Context switch 2: after done with all, to editor to commit + push etc.

I am terribly sorry but I do not understand at all... I do not grasp your workflow.
I recommend using these tools locally too: you can have them in your editor. It sounds like you only use these tools in CI.


Remove propTypes assignment entirely by default

Currently, when a production bundle is built, prop types assignment is transpiled & minified to:
Foo.propTypes = {}

With this change, that statement is removed entirely, saving a few bytes in each component.
The downside is that it's a slightly less safe transform because accessing a specific prop type (e.g. Foo.propTypes.className) will throw in production instead of evaluating to undefined. IMO this is fine because we should never be doing this, and if we ever do need to, we can export the propTypes object manually which de-opts the transform.
Note that something like {...Foo.propTypes, bar: PropTypes.string} (which does occasionally appear in our codebase, for better or worse) will not throw regardless.
~(Please hold off on merging for now because this is a breaking change and I'd like to get a minor release out first.)~ Nvm, going to release this + #57 + #58 in a major anyway.

do you know how much this will reduce our bundle size? it seems like there's decent risk (crashing our app), and i'm not sure that saving a few btyes per component would be worth the risk.
on the other hand, if the goal is to also use this to strictly enforce that we never access a prop type directly (and shaving a little off the bundle size would just be a bonus), then i'd be happy to roll this out

it seems like there's decent risk (crashing our app)

We already have this plugin set to 'remove' when the Babel env is 'production' (true for all apps we have deployed) which is even more extreme than 'unsafe-wrap'. (CRA has the same default, FWIW.)
IMO the risk is extremely low, because there's no practical use case for accessing a specific prop type. If it helps reassure you, there are no instances of /propTypes\./ anywhere in our codebase.
This change would only affect our library builds, where I consider the risk even lower since it's even less likely to access propTypes across a library boundary. It's essentially aligning how our libraries transpile prop types assignment with how our apps already do this.

(Also we already have a lint rule to guard against foreign prop type access.)

this sounds good to me, then. @nabeel- any thoughts?

Nope, lgtm!
I believe we had a bit of a discussion when we wanted to first introduce this plugin in https://github.com/kensho/babel-preset-kensho/commit/e7de8fc782770e76b23fa46c0c5ce59a286f3a74 and decided to be more relaxed with wrap. This seems in line with what we had planned to do and I haven't see anything new crop up that would indicate we should change.


[GWS] schedule enhancement2

Coverage increased (+0.05%) to 80.013% when pulling 2ba8a48050e7cef3a46795af4276d39430cfb0b7 on sunny4381:features/gws-schedule-enhancement2 into bc01a792356e66f50d18ff939392f9beb6746845 on shirasagi:dev/gws.

Coverage increased (+0.02%) to 79.981% when pulling 2c9987d3e28fc6749f68b4752306a9b7f5814bbc on sunny4381:features/gws-schedule-enhancement2 into bc01a792356e66f50d18ff939392f9beb6746845 on shirasagi:dev/gws.


Intermediate bounds

@huanzhang12 However, such obtained bounds will not be the same as in CROWN method, won't they? I wanted to look at intermediate bounds from particular methods. Is it possible to obtain those, even though the efficiency is reduced?
Originally posted by @nurlanov-zh in https://github.com/KaidiXu/auto_LiRPA/issues/22#issuecomment-1092565059

Hi @nurlanov-zh , now you may call compute_bounds to get the bounds for a specific node by specifying final_node_name:
bounded_model.compute_bounds(method='CROWN', final_node_name=node.name)


feat: parallel ensembles

Including support for parallel ensembles using mini-batch:

OnlineBagging
OnlineAdwinBagging
LeveragingBagging
AdaptiveRandomForest

It also included a tutorial notebook demonstrating how to use the parallel versions.

Conflicts solved, updated the jar, now we just need to update the notebook


Support conditional HTTP requests based on content available in cache

See related discussion in #16.

The fetch request should now send correct conditional HTTP requests. Note the possible avoidNetworkRequests config flag to force the use of the local cache.
One further improvement would be to properly support max-age and/or Expires directives. Right now, the crawler simply considers that cache entries are valid for the duration of the crawl.


Unable to use format on ReferenceArrayField

What you were expecting:
When using the ReferenceArrayField, I want to be able to use the format attribute to convert records to the format expected by the field
What happened instead:
I receive the following error: Cannot read property 'data' of undefined
Steps to reproduce:

Create a record with an array property of nested objects, ex:

{
  name: "My record"
  tags: [{id: 1}, {id: 2}]
}

Make sure Resources exist both for your object and the nested record
In your List view, add a ReferenceArrayField for the given property
Add a format attribute to map the nested object properties to a flat array of ids expected by the field

format={tags => tags.map(tag => tag.id)}

Related code:
You can find a short instance here: https://codesandbox.io/s/yp7r693759
Other information:
It seems like the EmbeddedArrayField from MhdSyrwan/aor-embedded-array  may be what I want to use instead, but it seems like the format/parse attributes should allow me to handle nested structures.
Environment

Admin-on-rest version: 1.4.0
React version: 16.2.0
Browser: Chrome 64.0.3282.140

Thanks for your suggestion.
But adding a format prop to all components isn't the admin-on-rest way. Any transformation of data from your API format to the format expected by admin-on-rest should take place in the restClient.
We won't add the format prop to the <ReferenceArrayField>.

Understood. I was just under the impression that most of the fields sat on top of Redux Forms fields, which I know can support that property, and your docs seemed to suggest support as well, at least from the input side. That said, I'll look in to the aor-embedded-array library. Thanks!

Indeed, the format option is supported on input components, not field components.


What does each node feature represents?

Thanks for sharing your code!
The node features have 70 elements for each node, but what does each represent?
The paper describes that 20 of them are amino acid id and others are conservation score, accessible surface etc. but I can't figure out which is which.
Could you please specify the order of the node features?

Hi @shionhonda , Appendix A of my MS thesis gives more details on how the node and edge features were computed and how many there are of each. The order presented in the appendix should match the order of features in the data sets.
Here is a link to the MS thesis:
https://mountainscholar.org/handle/10217/185661

Thanks so much! It helps a lot!

@fouticus Is there any chance you might be able to provide the script for node n edge feature calculations?

I'm checking with my Coauthor on sharing that code, since they wrote most of the feature generation code. This has been frequently asked for, I just haven't had much bandwidth to address it recently!


make error

how to solve it ?thanks @AIBluefisher

The message shows libgtest is not found in /usr/local/lib. Thus, you can check the directory of libgtest_main.a at first, it could be in /usr/lib.

Also, I found an issue in the CMakeLists.txt:
`CMake
message("-- Check for gtest")
find_package(GTest REQUIRED)
if (GTEST_FOUND)
message("-- Found GTest: ${GTEST_INCLUDE_DIRS}")
set(GTEST_LIBRARIES ${GTEST_LIBRARIES}
/usr/local/lib/libgtest_main.a
/usr/local/lib/libgtest.a)
message("-- Linked Libraries: ${GTEST_LIBRARIES}")
else (GTEST_FOUND)
message(FATAL_ERROR "Can't find GTest. Please set GTEST_INCLUDE_DIRS & GTEST_LIBRARIES")
endif (GTEST_FOUND)
`
I have fixed it by replacing ${GTEST_LIBRARIES},  /usr/local/lib/libgtest_main.a and /usr/local/lib/libgtest.a by
${GTEST_MAIN_LIBRARY} and ${GTEST_LIBRARY}. Now you can use the latest version to fix this issue.

Thanks for proposing this issue.


Get return code and GLIBC dockerfile fix

Had hit issue like this one, had updated the prow job to not use the dockerfile. With this change, I'm hoping to actually use the dockerfile here instead of configuring myself in prow
Adding proper return code if a high alert is found, see passing job of new return code here: https://mastern-jenkins-csb-openshift-qe.apps.ocp-c1.prod.psi.redhat.com/job/scale-ci/job/paige-e2e-multibranch/job/ssml/254/console
https://issues.redhat.com/browse/OCPQE-23853

@asood-rh are you able to take a look at this PR

/lgtm


Update UI for components-react FilterBuilder

Changes
Visual updates for the Filter Builder in components-react, based on discussion between the Pineapple and presentation teams.
Before:

After:

API changes include removal of nesting, as we will be able to support "IN" and "BETWEEN" operators instead.

Testing

Updated tests to reflect the removal of nesting.

@morganadrales, I thought we discussed deprecating only the prop rather than the whole component. Was there a reason to change the plan?

@morganadrales, I thought we discussed deprecating only the prop rather than the whole component. Was there a reason to change the plan?

Hi Grigas. Apologies, I created this PR before having to hastily leave before my honeymoon and I should've marked it as draft to prevent premature reviewal.
Originally, since the logic for supporting nesting has been removed, I duplicated the original component to prevent a breaking change. However, I have now updated the PR so that the ruleDepthLimit prop has been deprecated instead to prevent a breaking change - although note that it's still not being used since all code related to groups/nesting has been removed.
I still have some work to do on this PR to reach 100% code coverage (currently at 99.5%) but would you mind taking another pass at this PR? I can address any additional concerns while I attempt to get the code coverage back up to 100%.

@FlyersPh9 Can you have a look at the CSS, there is a lot of changes and px values, maybe we can/should do it differently.

@FlyersPh9 Can you have a look at the CSS, there is a lot of changes and px values, maybe we can/should do it differently.

I just hopped on a call with @morganadrales and looked through the SCSS files together.  Quick summary:

Where possible, use an iTwinUI CSS variable instead of a hard coded pixel value, this would consist of:

Border radius
Sizing / spacing
Font sizes
One exception being border thickness staying at 1px.  We are considering adding a --iui-border-thickness variable in a minor release.

In other areas where an iTwinUI size CSS variable might not match exactly, converting PX to REM I think will be a worthwhile change.
I saw :focus { outline: 0 } in there.  Wasn't sure where it was being applied, but just make sure everything that is actionable can be focused via pressing the TAB key on the keyboard.
And bonus points: iTwinUI switched to logical properties. Example: margin-right to margin-inline-end.  Would be nice to match.

Duplicate of https://github.com/iTwin/appui/pull/686


Feature request: An option to hide values on top of the handles would be nice to have

This was already requested in #99 , but I don't really want to add such a feature since we can do it with CSS:
.rz-bubble {
     display: none
}
```

All labels hidden: http://jsfiddle.net/2n4ay3oh/
Only the values ones: http://jsfiddle.net/fax330z1/

@ValentinH Yep that was what I did actually, just thought it would be more convenient. Closing this then.
Btw, awesome library. After spending endless hours with nouislider and still failing to get it to work properly, this was a lifesaver! Thanks for all the :fish:

Thanks :smiley:


Increase postgres statement timeout

Pull Request Test Coverage Report for Build 1189

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.752%

Totals

Change from base Build 1185:
0.0%

Covered Lines:
2329

Relevant Lines:
2458

💛  - Coveralls


test(ESLint): lint for test/ dir.

@homu r+

:pushpin: Commit 06256d5 has been approved by saneyuki

:hourglass: Testing commit 06256d5 with merge 42684bc...

:sunny: Test successful - status


Tooling to generate SBRP should be included in the SBRP repo

The tooling is currently internal.  There is nothing proprietary about the tooling.  It should reside within the SBRP repo to make it easier for anyone to contribute to the tooling as well as to use it to create new packages.

Fixed with https://github.com/dotnet/source-build-reference-packages/pull/294, https://github.com/dotnet/source-build-reference-packages/pull/300 & https://github.com/dotnet/source-build-reference-packages/pull/302


Feat/okp4 inflation calculation

Implements and declares the custom OKP4 function for inflation calculation, according to the following formula:
inflation for the block of the year X = 0.15 * inflationRateChange^(X-1) / blocksPerYear

as stated in whitepaper/tokenomics#staking-rewards, where:

inflationRateChange is the rate of inflation for each year, equal to 0.8 in the OKP4 network tokenomics.
blocksPerYear is the (expected) total number of blocks in a year. E.g. approx. 5259600 blocks for a year at the rate of one block every 6 seconds.

Example of mint module configuration: 15% initial inflation - 0.8 inflation rate per year
    "mint": {
      "minter": {
        "inflation": "0.150000000000000000",
        "annual_provisions": "0.000000000000000000"
      },
      "params": {
        "mint_denom": "uknow",
        "inflation_rate_change": "0.800000000000000000",
        "inflation_max": "0.000000000000000000",
        "inflation_min": "0.000000000000000000",
        "goal_bonded": "0.000000000000000000",
        "blocks_per_year": "5259600"
      }
    }

:tada: This PR is included in version 2.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


New package: DynACof v0.1.0

Registering package: DynACof
Repository: https://github.com/VEZY/DynACof.jl
Created by: @VEZY
Version: v0.1.0
Commit: 3dbb67ef30422c8854306467d4748f1d70c04b9a
Git reference: master

Needs a [compat] section in Project.toml with at least an entry for julia itself, e.g.:
[compat]
julia = "1"

would mean that the package is compatible with Julia versions [1.0.0, 2.0.0). Also, it's a slightly oddly capitalized and abbreviated name. Is it based on something?

I added the [compat] to the Project.toml of the package, thank you for the suggestion. I commited the changes in commit 2e4aed54b925e416432f7d6911de37a84b5e2829.
Do I have to register the package again (I used the Web Interface) or do I have to proceed differently?
The name of the package refers to a biophysical model called DynACof, for Dynamic Agroforestry Coffee crop model. I ported it from pure R because it was too slow (see DynACof).
I can change the name to dynacof if you think it's better.

Yes, please trigger registration again on the new commit. That will update this PR.
Regarding the name, capitalized is more idiomatic for Julia, but perhaps DynamicAgroCoffee would be a bit clearer? Up to you though.

I think I'll stick with DynACof because it will be easier to relate to the journal article and to the R package. But thank you for your suggestion!


stand-alone transpose function

It would be nice to have the function lazy_transpose(Dataset, axes=None) similar to numpy's

def lazy_transpose(dset):
    return dset.lazy_transepose()

This is added in 3557198e52ffcb0b9961be4b1e0f9cd4801d96c8 .


Add CfnDsl Template Compliation Support

Context
We want to support CfnDsl. Refer: https://github.com/envato/stack_master/issues/81
CfnDsl: https://github.com/stevenjack/cfndsl
Change
Use the cfndsl gem internally to compile any ".rb" templates.
Considerations
We don't support passing in variables to CfnDsl templates, yet.

:+1:


JSX instructions missing "npm install babel-plugin-syntax-jsx"

Code to reproduce the issue:
Follow steps to use JSX at http://cycle.js.org/getting-started.html

The following only works if you are building with Babel: (1) install the npm packages babel-plugin-transform-react-jsx and snabbdom-jsx; (2) specify a pragma for JSX as shown in the following example .babelrc file:
...
(3) import Snabbdom JSX as import {html} from 'snabbdom-jsx';, and then you can utilize JSX:

Expected behavior:
Babel transpiles JSX without error.
Actual behavior:
Module build failed: ReferenceError: Unknown plugin "syntax-jsx" specified in "base" at 0
Solution:
npm install babel-plugin-syntax-jsx --save-dev
Versions of packages used:
"babel-cli": "^6.11.4",
"babel-plugin-syntax-jsx": "^6.13.0",
"babel-plugin-transform-react-jsx": "^6.8.0",
"babel-preset-es2015": "^6.13.2",
"snabbdom-jsx": "^0.3.0",
"webpack": "^1.13.2"

@abrinsmead take a look at this example and how it doesn't include babel-plugin-syntax-jsx https://github.com/cyclejs/cyclejs/blob/master/examples/jsx-seconds-elapsed/package.json

Thanks @staltz. I'll try using the pattern from the demo and see what happens.  I'll also try with just babel, without webpack.


feat: notifications

[x] Add notifications table & schemas
[x] Notify users on notifications
[x] Add follow notification
[x] Add notification toast
[ ] Add notifications page
[ ] Add indication for unread notifications
[ ] Set notification is_read to true when read

@benawad can't use useTypeSafeTranslation with dynamic strings.

you can add a new circle button on the home page (like scheduled rooms) to go to notifications page

we probably also want to show the # of unread notifications, but that's going to be a little tricky

@benawad
https://user-images.githubusercontent.com/38838675/110243527-9658b400-7f7c-11eb-99d9-82516ccacf04.mp4

@benawad should I only show unread notifications in the notifications page? that way we can delete read notifications.

no don't worry about that, show all of them (paginated of course), we can always add a cron job later to delete really old notifs.
add a follow back button (only if you don't already follow them)


Api: support route throttling

Requested by Adrián Mouly

Add support for configuring throttling at per route level.

Requested by Ashishkumar


Using websockets, allow messages to be loaded when already on the Messages/Notifications page

Either send the message directly using websockets, or trigger this to fetch new messages

Time spent: 5 hrs


Recovery phrase: Added words

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
#1166
✅ Checklist

[x] Github issue details are up to date for people to QA.
[x] I have tested all my changes.

🕵♂ Notes for Code Reviewer
fetch words from mneumonic and show to user
🙈 Screenshots
👯♀ Paired with
Solo

@n13  Nik lets use this PR more for UI.
Ill work on the actual words generation from the key separately

@nik pls take a look at my last comment


feat: allows control over relationship add new button

Description
Allows control over the relationship "add new" button as discussed here: https://github.com/payloadcms/payload/discussions/1750
Does so by exposing a new admin.allowCreate property on the relationship field config.

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Checklist:

[x] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[x] I have made corresponding changes to the documentation

Yes! This is awesome! I'm not gonna submit a review but I did have a look through the code so I potentially can submit something like this in the future. So consider this a teeny tiny lgtm 😇


Dirty attributes update

This adds a way to pass in an original tuple to update command and it will skip the update if nothing was changed:
users.try { update(:by_id, 1).change(name: 'Jane').to(name: 'Jane Doe') }

As a bonus it adds pg-specific update which uses returning :)

I'm gonna merge this in since we use it in our app. Command DSL added in this PR is a subject to change and will be discussed further in rom-rb/rom#129

nice, plus postgres specify update code ;]


Page is now responsive

tested for chrome and firefox on linux

@AM1CODES could you please label this hacktoberfest-accepted


fix sidecar proxy-init image override.

resolve: https://github.com/istio/istio/issues/29902
[ ] Configuration Infrastructure
[ ] Docs
[ X ] Installation
[ X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Can you please check this:
istio version 1.8.1
The following annotations are not applied to the injected istio-proxy sidecar.
proxy.istio.io/config: | # configure an env variables
     proxyMetadata:
       OUTPUT_CERTS: / etc / istio-certs
       JWT_POLICY: first-party-jwt

and it uses the image docker.io/istio/proxyv2:1.7.5 by default.

Can you please check this:
istio version 1.8.1
The following annotations are not applied to the injected istio-proxy sidecar.
proxy.istio.io/config: | # configure an env variables
     proxyMetadata:
       OUTPUT_CERTS: / etc / istio-certs
       JWT_POLICY: first-party-jwt

and it uses the image docker.io/istio/proxyv2:1.7.5 by default.

@bethmage Can you log a separate issue? This PR is to fix proxy image annotation: sidecar.istio.io/proxyImage

@bethmage Can you log a separate issue? This PR is to fix proxy image annotation: sidecar.istio.io/proxyImage


Backup configuration validation is wrong

I think that wrong validation logic is used for the backup config:
backup_period_in_hours allows ["24" "48" "72" "96" "120" "144" "168" "336" "504" "672" "720"]
backup_retention_period_in_hours allows ["6" "8" "12" "16" "20" "24" "36" "48"]
I guess it should be the other way around.

:facepalm:
fixed in release 1.2.1
thanks for raising the issue and sorry about that.


Browse dialog for save path does not show all content of folders.

I think the save path dialog should show the files in folders even if you only are supposed to choose a folder.
The files should not be able to be selected as savepath.

After investigation, when setting FOS_PICKFOLDERS there doesn't seem to be a way to also show files.


MB Arabic Content Hyperlinks-Part 3

Two issues with links in Housing for Seniors/اسكان كبارالسن

Both issues are not fixed.

Also, text with asterisk should be bold.

Fixed


Question about reproduce 71mIOU

Thanks for Amazing code . When reproduce your mIOU encounter some problem:
1-when Achieving 71 mIOU, do you train from imageNet pretrain model?
1- Can not  loading pretrain model from "models/model_store.py" because of broken link. so I use pytorch offical model instead. but some parameter can not be load due to "deep_stem" flag in ResNet model. So I set deep_stem=False. Will this hurt performance?
I use pytorch1.4 with cuda10. Currently achieve mIOU 66.1 with ResNet50 backbone

I ran into same issue. Did you solve it with the current ResNet?

Additionally, do you change any other hyperprams?

I can not reproduce the 71 mIoU claim but I got 69.7 mIoU on Cityscapes-val set which is pretty close using this for ResNet50 and this for testing
Log

Hi, good job, do you change any other hyperparams?

hi, guys, maybe you can try this repo: https://github.com/SegmentationBLWX/sssegmentation
ICNet implemented in this repo achieves 76+% miou and this repo also supports many other sota/real-time models

i have achieved a 0.695_best_model.pth which is based on resnet50 ,but i still can not reproduce the 71.0mIOU😂

我无法重现 71 mIoU 声明，但我在 Cityscapes-val 集上得到了 69.7 mIoU，这非常接近ResNet50和测试
日志

That 71.0MIou refers to the 71.0best_model trained by resnet50, rather than evaluate on Cityscaoes-val


Add CMake support

With this PR, the project can now be built with CMake, which makes building on Linux a lot less painful, since it will find the required libraries on the system automatically:
cmake .
make -j 4
./riko4

The Visual Studio solution should still work as before. I've replaced lfs.c with another version I found, since strerror_s is an MSVC-exclusive function. It will probably generate a warning, in which case, add _CRT_SECURE_NO_WARNINGS as a preprocessor definition in your project settings. I can't do that since I'm not on Windows right now.

Looks good, thanks!


Redis running over TCP returning HTTP response

Do you want to request a feature or report a bug?
Bug
I'm trying to use Redis with Traefik but am running into some issues.
I followed this issue: https://github.com/containous/traefik/issues/4930 in how they configured Redis with docker compose, but when doing so and trying to PING the redis db, I get the following error:
Error: Protocol error, got "H" as reply type byte

After going through some forums and StackOverFlow, it seems like this issue may come up when instead of TCP, Traefik uses HTTP (although in the dashboard it does show up as TCP).
Here is my docker compose file:
version: "3"

networks:
  proxy:
    external: true
  internal:
    external: false
services:

  hog:
    image: traefik:v2.1.2
    command:
      - --entrypoints.web.address=:80
      - --entrypoints.redis.address=:50000
      - --providers.docker
      - --api
      - --ping=true
    ports:
      - "50000:50000"
      - "80:80"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    networks:
      - proxy
    labels:
      - "traefik.http.routers.api.rule=Host(`subdomain.domain.tld`)"
      - "traefik.http.routers.api.service=api@internal"
      - "traefik.http.routers.api.middlewares=auth"
      - "traefik.http.middlewares.auth.basicauth.users=admin:password"

  redis:
    image: redis:5
    labels:
      - "traefik.tcp.routers.redis.entrypoints=redis"
      - "traefik.tcp.routers.redis.rule=HostSNI(`another-subdomain.domain.tld`)"
      - "traefik.tcp.services.redis.loadbalancer.server.port=50000"
      - "traefik.tcp.routers.redis.tls=true"
    command: ["redis-server", "/usr/local/etc/redis/redis.conf", "--appendonly", "yes", "--requirepass", "password"]
    hostname: redis
    volumes:
      - ./redis-data:/data
      - ./redis.conf:/usr/local/etc/redis/redis.conf

Even without a redis.conf and using the given conf, I get the same issue. Any help would be greatly appreciated.
To be clear, it seems like I'm able to connect to the DB since I get Redis' shell when I do redis-cli like the following:
redis-cli --no-auth-warning -h another-subdomain.domain.tld -p 50000 -a 'password'

But when I do PING within that shell I get that error aforementioned.

@dantuluri Please note that the ping utility does not use TCP, but ICMP, which is a different layer, and uses different protocols than TCP.

Tried to connect to Redis via rq-dashboard and got a response as such:
redis.exceptions.InvalidResponse: Protocol Error: H, b'TTP/1.1 400 Bad Request'

I'll try to diagnose this issue further, but am thinking about if Traefik is really needed (what are the benefits of Traefik for a simple TCP routing?).

Followed issue #4930 and did
for i in `seq 1 10`; do echo 'ping' | nc subdomain.domain.tld 50000; done | uniq -c

And got a response something like the following:
 1 HTTP/1.1 400 Bad Request
   1 Content-Type: text/plain; charset=utf-8
   1 Connection: close
   1
   1 400 Bad RequestHTTP/1.1 400 Bad Request
   1 Content-Type: text/plain; charset=utf-8
   1 Connection: close
   1
   1 400 Bad RequestHTTP/1.1 400 Bad Request
   1 Content-Type: text/plain; charset=utf-8
   1 Connection: close
   1
   1 400 Bad RequestHTTP/1.1 400 Bad Request
   1 Content-Type: text/plain; charset=utf-8
   1 Connection: close
   1
   1 400 Bad RequestHTTP/1.1 400 Bad Request
   1 Content-Type: text/plain; charset=utf-8
   1 Connection: close
   1
   1 400 Bad RequestHTTP/1.1 400 Bad Request
   1 Content-Type: text/plain; charset=utf-8
   1 Connection: close
   1
   1 400 Bad RequestHTTP/1.1 400 Bad Request
   1 Content-Type: text/plain; charset=utf-8
   1 Connection: close

Which makes me think of where in my config am I using HTTP? According to the dashboard and my config, I should be using TCP, right? Could this be a Traefik issue or something else?

Hi @dantuluri
On your router api, you didn't specify the entrypoints so it is attach to your redis entrypoint too. If you change this, it should fix your problem.

Forgot to update this issue, but yes, I did see that and fixed it accordingly.
I thought it may because I have some firewall, but starting a simple TCP worked fine.
Here's my updated docker-compose file (still running into the HTTP issue):
version: "3"

networks:
  proxy:
    external: true
  internal:
    external: false
services:

  hog:
    # The official v2.0 Traefik docker image
    image: traefik:v2.1.2
    # Enables the web UI and tells Traefik to listen to docker
    command:
      - --entrypoints.web.address=:80
      - --entrypoints.redis.address=:50000
      - --providers.docker
      - --api
      - --ping=true
    ports:
      - "50000:50000"
      - "80:80"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    networks:
      - proxy
    labels:
      - "traefik.http.routers.api.rule=Host(`subdomain.domain.tld`)"
      - "traefik.http.routers.api.service=api@internal"
      - "traefik.http.routers.api.middlewares=auth"
      - "traefik.http.middlewares.auth.basicauth.users=admin:password"

  redis:
    image: redis:5
    networks:
      - proxy
    labels:
      - "traefik.enable=true"
      - "traefik.tcp.routers.redis.entrypoints=redis"
      - "traefik.tcp.routers.redis.service=redis"
      - "traefik.tcp.routers.redis.rule=Host(`subdomain.domain.tld`)"
      - "traefik.tcp.routers.redis.rule=HostSNI(`*`)"
      - "traefik.tcp.services.redis.loadbalancer.server.port=50000"
      - "traefik.tcp.routers.redis.tls=true"
    volumes:
      - ./redis-data:/data
    command: ["redis-server", "--appendonly", "yes", "--requirepass", "password"]
    hostname: redis

Hi @juliens, I took a closer look at your comments and made some respective changes (which looking at it from face value, makes sense and I'm not getting errors anymore).
However, for some odd reason when I'm trying to connect, it lags forever, meaning the operation times out everytime.
Here's my updated docker file:
  hog:
    image: traefik:v2.1.2
    command:
      - --entrypoints.web.address=:80
      - --entrypoints.redis.address=:50000
      - --providers.docker
      - --api
      - --ping=true
    ports:
      - "80:80"
   volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    networks:
      - proxy
    labels:
      - "traefik.http.routers.api.rule=Host(`subdomain.domain.tld`)"
      - "traefik.http.routers.api.service=api@internal"
      - "traefik.http.routers.api.middlewares=auth"
      - "traefik.http.middlewares.auth.basicauth.users=admin:password"
      - "traefik.http.routers.api.entrypoints=web"

  redis:
    image: redis:5
    networks:
      - proxy
    expose:
      - 6379
    ports:
      - 50000:6379
    labels:
      - "traefik.enable=true"
      - "traefik.tcp.routers.redis.entrypoints=redis"
      - "traefik.tcp.routers.redis.service=redis"
      - "traefik.tcp.routers.redis.rule=Host(`subdomain.domain.tld`)"
      - "traefik.tcp.routers.redis.rule=HostSNI(`*`)"
      - "traefik.tcp.services.redis.loadbalancer.server.port=50000"
      - "traefik.tcp.routers.redis.tls=true"
    volumes:
      - ./redis-data:/data
      - ./redis.conf:/usr/local/etc/redis/redis.conf
    command: ["redis-server", "/usr/local/etc/redis/redis.conf", "--appendonly", "yes", "--requirepass", "password"]
    hostname: redis
    restart: always

The port :50000 must be open on Traefik
traefik.tcp.services.redis.loadbalancer.server.port=50000" must be traefik.tcp.services.redis.loadbalancer.server.port=6379"

Turned out to be a Cloudflare issue at the end (endless hanging).
Thanks everyone!
I'll close this now and hope people who want to setup Redis with Traefik 2 can hopefully debug their issues with this case.


Docker tag for latest binary release

It looks like the Docker image tags aren't up-to-date/in sync with the binary releases on Github. While I'm successfully using the latest tag on Docker for now, in general it's preferable to tag the docker image to a specific release for compatibility reasons etc. Just wondering if that's something you'd be able to do, at least for major versions such as v3.
Also, thanks for writing this tool! It's been working out great so far. Cheers!

I don't maintain the Docker images. @mrnetops can you jump in?

Maintaining the docker build and tags outside of the main release is a
little hinky.  I'm about to fly out to family for the holidays, but I'll
work on cleaning it up when I get back.
On Sat, Dec 21, 2019 at 11:16 AM Peter Bourgon<EMAIL_ADDRESS>wrote:

I don't maintain the Docker images. @mrnetops
https://github.com/mrnetops can you jump in?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/peterbourgon/fastly-exporter/issues/38?email_source=notifications&email_token=ABPUIWMCYO72EAFFC3D37O3QZZTSRA5CNFSM4J6COFM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHPB4OQ#issuecomment-568204858,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPUIWNCLMBEB2KVF2W2TALQZZTSRANCNFSM4J6COFMQ
.

--
It turns out I'm secretly evil

@mrnetops Friendly ping :)