Enhancement: Option to allow queries against accelerated tables that return 0 results to fallback to federated source

Goal-State/What/Result
When a query that targets a dataset configured with acceleration returns 0 results, a new dataset acceleration option (on_zero_results: use_source) can be configured to specify that the query should fallback to querying against the federated source.
Why/Purpose
This allows users to get the benefits of fast accelerated queries when there is data already accelerated, but still get the results from the source if needed.
By When
Wed, Apr 24
Issue/Spec written and reviewed: Wed, Apr 24
Done-Done: Wed, Apr 24
Done-Done

[x] First Principles Driven
[x] The Algorithm
[x] PM/Design Review
[x] UX Review
[x] Threat Model / Security Review
[x] Tests
[ ] Documentation
[ ] Samples/Quickstarts/Tutorials

The Algorithm

[x] Every requirement questioned?
[x] Delete (Scope) any part you can.
[x] Simplify.
[x] Break down into smaller iterations/milestones.
[x] Opportunities for automation.

Spec
dataset.yaml
from: databricks:my_dataset
name: my_dataset
params:
  endpoint: my-workspace.databricks.com
  databricks-cluster-id: 1111-222222-abcdefgh
acceleration:
  enabled: true
  refresh_sql: SELECT * FROM my_dataset WHERE city = 'Seattle'
  on_zero_results: use_source

A query against my_dataset within Spice that looks like SELECT * FROM my_dataset WHERE city = 'Portland' would initially query against the acceleration, see that it returns 0 results and then falls back to querying against databricks directly.
How/MAP

[x] Add on_zero_results: use_source option to Acceleration parameter
[x] Intercept query result from Accelerated table and redirect to federated table if 0 rows are returned from the table scan
[x] Metrics to track how often the fallback occurs
[ ] Docs
[ ] Quickstart

Implementation (issues)

[ ] #1196

QA
Manual testing against our Azure Databricks instance

Re-opening to track docs/quickstart

This is a relatively advanced feature that we don't necessarily want to highlight, so I won't create a quickstart for it.


Yahoo scraped data with Options data

From @Flippo24
When I pull data from Yahoo, it only shows the price of the underlying and that the position is ITM. But all other values are missing. No values for the legs and the total PnL.

Originally posted by @Flippo24 in https://github.com/PaulSquires/TradeTracker/issues/16#issuecomment-1881831017

Not planned.


Updates min size on CenterContainer::set_use_top_left

The use_top_left property affects CenterContainer's minimum size. When true, the minimum size is zero. When false, the minimum size is the largest min size of all children.
Before this fix, the minimum size is not updated after toggling,  I have to trigger it manually. (e.g. In the editor, switch to another scene and switch back, similar to #34979)

Putting minimum_size_changed before queue_sort is how it's used here:
https://github.com/godotengine/godot/blob/34ad33d9e0b211b2de5855373f8626af4fc0ce11/scene/gui/container.cpp#L59-L68

Thanks!


curl: (60) server certificate verification failed. CAfile: /etc/ssl/certs/ca-certificates.crt CRLfile: none

I just updatet my docker containers after I pulled from git, and restarted my containers but I just started to getting error from browser  PR_END_OF_FILE_ERROR
So I've login into redmine container and tryed curl on localhost and this is result.

root@0ae0ad897441:/home/redmine/redmine# curl -Iv https://<IP_ADDRESS>
* Rebuilt URL to: https://<IP_ADDRESS>/
*   Trying <IP_ADDRESS>...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 443 (#0)
* found 127 certificates in /etc/ssl/certs/ca-certificates.crt
* found 508 certificates in /etc/ssl/certs
* ALPN, offering http/1.1
* SSL connection using TLS1.2 / ECDHE_RSA_AES_128_GCM_SHA256
* server certificate verification failed. CAfile: /etc/ssl/certs/ca-certificates.crt CRLfile: none
* Closing connection 0
curl: (60) server certificate verification failed. CAfile: /etc/ssl/certs/ca-certificates.crt CRLfile: none
More details here: http://curl.haxx.se/docs/sslcerts.html

curl performs SSL certificate verification by default, using a "bundle"
 of Certificate Authority (CA) public keys (CA certs). If the default
 bundle file isn't adequate, you can specify an alternate file
 using the --cacert option.
If this HTTPS server uses a certificate signed by a CA represented in
 the bundle, the certificate verification probably failed due to a
 problem with the certificate (it might be expired, or the name might
 not match the domain name in the URL).
If you'd like to turn off curl's verification of the certificate, use
 the -k (or --insecure) option.

I tryed update, and upgrade packages in container, rebuild containers, also reinstall ca-cert package, but nothing worked.

What version of the docker image are you using?  What git did you pull from?  Are the permissions on your certificates set correctly?
Some possibilities
https://stackoverflow.com/questions/21181231/server-certificate-verification-failed-cafile-etc-ssl-certs-ca-certificates-c

What version of the docker image are you using? What git did you pull from? Are the permissions on your certificates set correctly?
Some possibilities
https://stackoverflow.com/questions/21181231/server-certificate-verification-failed-cafile-etc-ssl-certs-ca-certificates-c

I freshly downloaded and reinstalled the project.

Ahh okay, so you built the docker image yourself.  How are you launching it? Can you post the output from when you launch the image.

Yeah buildet it because I changed port to 443.
Output is actually cool, with no errors
docker-compose up
Creating network "docker-redmine_default" with the default driver
Creating docker-redmine_mysql_1 ... done
Creating docker-redmine_redmine_1 ... done
Attaching to docker-redmine_mysql_1, docker-redmine_redmine_1
redmine_1  | Initializing logdir...
redmine_1  | Initializing datadir...
redmine_1  | Symlinking dotfiles...
redmine_1  | Installing configuration templates...
redmine_1  | Configuring redmine...
mysql_1    | Creating database "redmine"...
mysql_1    | Granting access to database "redmine" for user "redmine"...
mysql_1    | 2020-09-03T10:39:01.762420Z mysqld_safe Logging to syslog.
mysql_1    | 2020-09-03T10:39:01.774130Z mysqld_safe Starting mysqld daemon with databases from /var/lib/mysql
redmine_1  | Configuring redmine::database...
redmine_1  | Configuring redmine::unicorn...
redmine_1  | Configuring redmine::secret_token...
redmine_1  | Configuring redmine::max_concurrent_ajax_uploads...
redmine_1  | Configuring redmine::sudo_mode...
redmine_1  | Configuring redmine::autologin_cookie...
redmine_1  | Configuring redmine::email_delivery...
redmine_1  | Configuring redmine::backups...
redmine_1  | Configuring nginx...
redmine_1  | Configuring nginx::redmine...
redmine_1  | Configuring nginx::redmine::ssl...
redmine_1  | Configuring nginx::redmine::hsts...
redmine_1  | Installing plugins...
redmine_1  | Installing themes...
redmine_1  | 2020-09-03 12:39:03,273 CRIT Supervisor running as root (no user in config file)
redmine_1  | 2020-09-03 12:39:03,273 WARN Included extra file "/etc/supervisor/conf.d/cron.conf" during parsing
redmine_1  | 2020-09-03 12:39:03,273 WARN Included extra file "/etc/supervisor/conf.d/nginx.conf" during parsing
redmine_1  | 2020-09-03 12:39:03,273 WARN Included extra file "/etc/supervisor/conf.d/unicorn.conf" during parsing
redmine_1  | 2020-09-03 12:39:03,279 INFO RPC interface 'supervisor' initialized
redmine_1  | 2020-09-03 12:39:03,279 INFO supervisord started with pid 1
redmine_1  | 2020-09-03 12:39:04,281 INFO spawned: 'unicorn' with pid 290
redmine_1  | 2020-09-03 12:39:04,282 INFO spawned: 'cron' with pid 291
redmine_1  | 2020-09-03 12:39:04,283 INFO spawned: 'nginx' with pid 292
redmine_1  | 2020-09-03 12:39:05,599 INFO success: unicorn entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
redmine_1  | 2020-09-03 12:39:05,599 INFO success: cron entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
redmine_1  | 2020-09-03 12:39:05,599 INFO success: nginx entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)

^CGracefully stopping... (press Ctrl+C again to force)
Stopping docker-redmine_redmine_1 ... done
Stopping docker-redmine_mysql_1   ... done

Yeah buildet it because I changed port to 443.
Could you clarify this statement?  You shouldn't have to build it to get ssl working.

Yeah buildet it because I changed port to 443.

Could you clarify this statement? You shouldn't have to build it to get ssl working.

I changed nginx settings to work on 443 outside container.

With openssl I have output with "No client certificate CA names sent".

You can configure for HTTPS without modifying nginx.
https://github.com/sameersbn/docker-redmine#ssl

Well, not sure how to do it with docker-composer..but that does not solve my problem..

I added a ssl example for docker-compose
https://github.com/sameersbn/docker-redmine/blob/master/docker-compose-ssl.yml


/turnstile command  cuts off last part of the command

My bad, it was a logical error in a loop :package:
Fixed in the next update, thanks for reporting it.


Added Heston and CEV models

Added a Euler scheme. I actually don't exactly remember what's going on with self.max_depth in TradingEnvironment. I remember when I ran check_all_imports.py the TradingEnvironment.py file was throwing up errors because the default value for max_depth is None and that particular section of code was interpreting it as a float. I think what I did was just a temporary workaround to get past some meaningless type errors. Discard those changes if they're not necessary.

I don't have write access permission to merge.


Resource objects: If file doesn't exist, use default instead of crashing.

The resource system now uses Result<T>, so this combined with an or_default() style construct allows this to be implemented (or for the caller to handle the error gracefully).


Write a web app with UI that uses graph traversal

Write something that is a web app that actually relies on the graph to function.

There are two nodes that will give us trouble currently: run-javascript (because it assumes NodeVM being available) and text-asset (because it assumes Node file access package being available)
I think for now, it's fine to hack around this, but we'll need a more permanent solution soon.
@proales

Split off #43 and #42.

Added #50 .

Also - fetch node will hit CORS issues in the browser.

@PaulKinlan  did it!


WebApp - Do not show token group information in case interrupt connection

Describe the task
Do not show token group information in case interrupted connection

Version test: https://50f89b8e.subwallet-webapp.pages.dev/
Validation passed for this case


updated hudi content

Change Logs
updated videos and blog content; blog image is null- but file is added

cc @bhasudha to review

cc @vinothchandar cc @bhasudha : we created a list for now because it was faster to get content out. Sudha and I planned on optimizing it when we spend cycles on improving the docs page layout. This was only meant to be temporary.


5.1 Feedback: mounts described as "empty" when they aren't or don't need to be

Overview
The app is too liberal in using the term "Empty" for pipette mounts. We should adopt clearer language on the Devices and Protocols page.
Current Behavior
Robots page
On the robots page, both mounts for unavailable robots are listed as "Empty". Since the robots aren't in communication with the app, that information may be incorrect (and likely is incorrect in settings where users don't remove their pipettes).

Protocols page
If a protocol only uses a single pipette, it lists the other mount as "empty". This makes it seem like the protocol requires that the other pipette be removed in order to run it, when this is not the case.

Expected Behavior
Robots page
The mount sections should either show:

The pipettes that were in place when the robot was last seen.
Some text expressing that the information is not available: e.g. "Unknown" or an em dash

Protocol page
The unused mount should say "Unused" or "Not used".
Steps To Reproduce
View any unavailable robot on the Robots page or any loaded protocol on the Protocols page.
Operating system
Mac
Robot setup or anything else?
No response

@emilywools @mmencarelli
What do you think about this?
Another question empty 's first letter should be a capital letter?

Addressing Ed's expected behavior comment #1 - https://github.com/Opentrons/opentrons/issues/10032

@mikecardarelli - to ticket
protocol page - unused
wording for empty not needed because section will be taken out

New ticket for "Not used" pipettes: https://app.zenhub.com/workspaces/app--ui-5f6248dba845b000139aa767/issues/opentrons/opentrons/10180

PR Merged - https://app.zenhub.com/workspaces/app--ui-5f6248dba845b000139aa767/issues/opentrons/opentrons/10188

I think this is good


Implement SunsetHUD by Hypnotize

https://github.com/Hypnootize/Sunset-Hud

Reopening as a potential feature.

Added Hypnotize-Hud (9f304c0c164460ea8005d34c63e6989a3882f931) and HExHUD (e3b7c04ae2b063c884c1094cf73b2c159be88404)


As a student, I want to receive notifications when my friends change their telegram information

...so that I can stay up to date with their latest details.

unlikely to do, closing


Fixing sorted queries to utilize composite indexes.

Fixing sorted queries to utilize composite indexes. Unit tests in place already cover this functionality as it is simply a performance improvement.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/check-enforcer override


An ability to use Deckard on Debian Jessie

Hello!
I'm trying to migrate our DNS test suite to a new version of Deckard. But unfortunately, it does not work as expected.
I use Debian Jessie and it has a pretty outdated version of Augeas:
dpkg -l|grep augeas
ii  augeas-lenses                          1.2.0-0.2+deb8u1                   all          Set of lenses needed by libaugeas0 to parse config files
ii  libaugeas-dev                          1.2.0-0.2+deb8u1                   amd64        Development files for writing applications based on libaugeas0
ii  libaugeas0                             1.2.0-0.2+deb8u1                   amd64        Augeas configuration editing library and API

And according to https://github.com/CZ-NIC/deckard/commit/03213062917f85a19eba1ea0dffc3c937a32dc11 I need at least Augeas 1.5.
Any chance to use Deckard with old versions of Augeas?
Also, external binary dependency makes our deployment process extremely complicated because we have number of different versions of Debian :(

Comment only mentions slowness, nothing else.

we want to install newer version of Augeas because those from Ubuntu repositories are incredibly slow on some operations

Thank you for the fast response! I appreciate it! :)
But after upgrade Deckard fails for some reasons in Augeas code (I found "value MUST be a string or None" in pyaugeas):
Run test  system/tests/test_dns_71.rpl
Scenario failed:  value MUST be a string or None!

I use following scenario:
cat tests/system/tests/test_dns_71.rpl
SCENARIO_BEGIN Test case for subd.mytestzone.io A (NOERROR)

STEP QUERY external
ENTRY_BEGIN
SECTION QUESTION
subd.mytestzone.io IN A
ENTRY_END

STEP CHECK_ANSWER
ENTRY_BEGIN
MATCH all
REPLY QR AA  NOERROR
SECTION QUESTION
subd.mytestzone.io IN A
SECTION ANSWER
subd.mytestzone.io. 300 IN A <IP_ADDRESS>
ENTRY_END
``

I migrated our test suite to Python 3 but I'm still hitting same error.
Scenario failed:  value MUST be a string or None!
Traceback (most recent call last):
  File "./runtests.py", line 385, in main
    stage, _ = scenario.parse_file(fileinput.input(name))
  File "/Users/pavel/.CFGOPATHRRDNS/src/code.cfops.it/sys/rrdns/linux_venv/lib/python3.4/site-packages/pydnstest/scenario.py", line 983, in parse_file
    confpath=path, lens='Deckard', loadpath=os.path.dirname(__file__))
  File "/Users/pavel/.CFGOPATHRRDNS/src/code.cfops.it/sys/rrdns/linux_venv/lib/python3.4/site-packages/pydnstest/augwrap.py", line 70, in __init__
    self._aug.set(join(aug_load_path, 'incl[0]'), confpath)
  File "/Users/pavel/.CFGOPATHRRDNS/src/code.cfops.it/sys/rrdns/linux_venv/lib/python3.4/site-packages/augeas/__init__.py", line 163, in set
    raise TypeError("value MUST be a string or None!")
TypeError: value MUST be a string or None!

Could you help with it?
Thank you!

Hello Pavel.
sorry for taking so long to reply - I didn't have watch set on this repo.
Our CI is running on Debian Jessie so you can look into our Dockerfile to see what we did:
https://gitlab.labs.nic.cz/knot/knot-resolver/blob/master/ci/Dockerfile
Right now our Docker image contains libaugeas0 version 1.8.0-1+deb9u1 and python-augeas binding for Python 3 installed via pip, version 1.0.3.
Speaking of binary depedencies, well, one alternative I'm aware of is to use network namespaces from Linux kernel. It would mean that Deckard is Linux-only and it would also require root privileges.
BTW "official" repo is in CZ.NIC's Gitlab, I just created Github issue #7 to explain this to newcommers. It might be better to continue discussion in real time, please join us @ https://gitter.im/CZ-NIC-deckard.

Thanks! Will do :) Just created account for your GitLab bug tracker.


added a post config

@cojmckee rather then using shell command can one of these modules do what is needed https://github.com/theforeman/foreman-ansible-modules/tree/master/plugins/modules
That way its idempotent and so forth.

@cojmckee i think this module would do it: https://github.com/theforeman/foreman-ansible-modules/blob/master/plugins/modules/foreman_setting.py

since the katello/formen modules arn't on ansible galaxy still the only "good" option is to copy paste the required module into a lib directory in this project so you can use it here. Give it a shot.

roger
On Tue, Sep 3, 2019 at 9:36 AM Ian Tewksbury<EMAIL_ADDRESS>wrote:

since the katello/formen modules arn't on ansible galaxy still the only
"good" option is to copy paste the required module into a lib directory in
this project so you can use it here. Give it a shot.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RedHatOfficial/ansible-role-redhat_satellite6_installation/pull/18?email_source=notifications&email_token=AHYDS77YRNSX6QTETSZU5JTQHZR7NA5CNFSM4ITGJFWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5YG2OI#issuecomment-527461689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHYDS7ZCWYL7UXVZMVBDIELQHZR7NANCNFSM4ITGJFWA
.

--
Cory McKee
Senior Consultant
Red Hat Public Sector https://www.redhat.com
8260 Greensboro Drive
McLean, VA 22102
<EMAIL_ADDRESS>  M:<PHONE_NUMBER>
https://red.ht/sig
TRIED. TESTED. TRUSTED. https://redhat.com/trusted
@redhatjobs https://twitter.com/redhatjobs   redhatjobs
https://www.facebook.com/redhatjobs/   @redhatjobs
https://instagram.com/redhatjobs

Added a play with ansible foreman modules

@cojmckee wouldn't you need to include this theformen role in here somewhere or is that a module in Ansible now?

@itewk not yet
Will upload add the module to the role and upload it

@cojmckee i dont see the module in here, and from a quick search its not part of default ansible. can you add it to the library here?


Clarify step 12 of server upgrade

Summary

We clarify why step 12 is actually needed.
Ticket Link

https://mattermost.atlassian.net/browse/MM-27359

Gentle ping @wiersgallak.


gopls: program used 23GB of RAM on my machine

I recompiled x/tools at tip (539cdc44f86437a5d06fd0cae3173ac81bd7c8aa) today. I am also running Go tip (b649bdc7f3c99c5288c91a1ce148efadd86e19a4).
I use gopls as part of vim-go, to edit Go files. I noticed my fans started spinning, opened Activity Monitor and noticed gopls using 23GB of memory.
I likely won't have cycles to figure out what is causing the memory leak today but hopefully this information is enough to figure out what/where the problem is.

Seeing the same thing -- I updated gopls about an hour ago, and it only stops consuming memory when I run out of it (~30GB, in my case, at which point the whole system locks up).

version v0.1.6, built in $GOPATH mode

2019/10/14 15:39:34 Info:2019/10/14 15:39:34 Build info
----------
version v0.1.6, built in $GOPATH mode

Go info
-------
go version devel +4c9e757daf Mon Oct 14 16:13:27 2019 +0000 darwin/amd64

GO111MODULE="off"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/kevin/Library/Caches/go-build"
GOENV="/Users/kevin/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY="github.com/meterup"
GONOSUMDB="github.com/meterup"
GOOS="darwin"
GOPATH="/Users/kevin"
GOPRIVATE="github.com/meterup"
GOPROXY="direct"
GOROOT="/Users/kevin/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/kevin/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/s1/909yt58s4wj8h_7v7frr8rkm0000gn/T/go-build822850684=/tmp/go-build -gno-record-gcc-switches -fno-common"
2019/10/14 15:39:34 Info:2019/10/14 15:39:34 12.948224ms for GOROOT=/Users/kevin/go GOPATH=/Users/kevin GO111MODULE=off PWD=/Users/kevin go "env" "GOMOD", stderr: <<>>
2019/10/14 15:39:34 Info:2019/10/14 15:39:34 13.608131ms for GOROOT=/Users/kevin/go GOPATH=/Users/kevin GO111MODULE=off PWD=/Users/kevin go "list" "-m" "-json" "all", stderr: <<go list -m: not using modules
>>
2019/10/14 15:39:34 Info:2019/10/14 15:39:34 15.436484ms for GOROOT=/Users/kevin/go GOPATH=/Users/kevin GO111MODULE=off PWD=/Users/kevin go "list" "-e" "-json" "-compiled=true" "-test=true" "-export=false" "-deps=true" "-find=false" "--" "builtin", stderr: <<>>
gopls: getFile: file:///path/to/file.go: open /path/to/file.go: no such file or directory

I don't have time today to figure out how to enable logs. I just reverted x/tools by several commits and reinstalled.

I rolled back to 638914d249c52acb9ae338284978e8b51608b94b and have not had any problems for the rest of the afternoon. Not clear whether one caused the other, though, it's possible that I haven't re-triggered the same behavior.

I think it was a problem introduced by revision b53505e, because I downgraded to revision 5fa5b17 and the problem disappeared.

The fix will be coming shortly (CL 201098). Thank you for reporting and investigating this issue.

Now that the CL has been merged, I will close this issue.

Thanks for the quick fix Rebecca - can confirm I haven't seen this since that CL was merged and I recompiled.

in which release is this fix present?

This fix is available in the current gopls release (v0.2.2).


YMD dateformat is not functioning

What I have tried?
I have tried to add language support for Hungarian - you can look at the result in:
https://github.com/solalatus/dateparser/commit/800f6ce76337bc42ae9126b9c89c4bffe644ac32
and
https://github.com/solalatus/dateparser/commit/2bc56830efd843788208b42259048f0aeff119e2
After adding the files, generally things functioned well, but failed in some cases.
Functioning case:
"2016 július 3."
Non-functioning case:
"2016 július 15."
Details
Dots are ommitted well, the only factor that breaks the code is when the day gets to double digits.
(Tested also above and below 12 to rule out confusion with months, but that is not the case. 9 works, 10 no longer...)
Hope
If you have time to check it, maybe Hungarian language support can be added.
(Disclaimer: as an agglutinative language Hungarian is tricky, there are cases where parsing will fail, since enumerating all the suffixes is nearly impossible - they can be combined quite freely. Despite this, decent coverage is possible.)
System:
Ubuntu 16.04, Anaconda 4.1.8, Python 3.5

@solalatus this looks related to noramlization, it will be very convenient for me to debug the issue if you could create a PR. Can you do that?

@waqasshabbir pull request done. Please inform, if I can contribute in any other way!
Thx!

Hi @solalatus, I looked into it today and found out the issue is related to new parser. Your changes should work fine now. Regarding the PR, can you please also add tests? Except for tests, PR looks great. Thanks.

I have added some test, but on my side it is still failing. I did not pull any changes from the master, so it might be all ok by merge. Could you please take a look? Thx

Hi @solalatus, thank you for the contribution! I fixed the tests and merged the PR.

Thanks, it was a pleasure! :-)


remove possible stale input gradients for RepeaterCriterion

Hi Nick,
I believe I have found a bug regarding how RepeaterCriterion computes its input gradients. Namely, it doesn't remove possible stale gradients from previous backward calls by iterating through the entirety of its gradInput table. This means that if you call backward with a longer sequence, any subsequent backward calls with shorter sequences will crash because the input gradient table will contain stale vectors from the first backward call. I've made a simple fix for this for your review.
Thanks,
Victor

@vzhong Thanks!


chore(simulation): typo fix

minor typo fix, nothing critical
2 are plain text in .md file, rest are comments

thank you for PR! however, we currently have an open PR that fixes all grammatical errors in the repository: https://github.com/osmosis-labs/osmosis/pull/7059. To avoid duplicate review, I think it is best for reviewers to focus on that pull request


Update index.html

@AyushSri19-777  thnx for contributing , try this year aswell with updated repo and make sure to follow me and star this repo so that you don't miss updates regarding the new changes and issues during Hacktoberfest 2020.


fix(error): added margin at the buttom of the error component

SVCC-3252
added padding the bottom of the error component
before:

after:

blue line added to show margin

It would be better to add that margin at a "higher" (more specific) level in the food chain than that low level component (that is used in different context). Because otherwise you make it so that component has some magic margin that might not make sense in all context.
Alternative are:

In the .html page, in the layout section itself that holds the atomic-insight-query-error
In the atomic-insight-query-error.tsx file itself.

I'm the one that suggested the spacing could be added to the base component. Is there a scenario where we would not want proper spacing with whatever is around this error component?


I can't understand how to get to work my code into kivy

I just found out recently about kivy. Python i'm also studing recently. I would like application, made into kivy, which has 2 boxes for input text and returning a label with result after clicking a button. This code must be in him:

 def word_detect(a,b):
       y = []
       e = []
       s = []
       for counter in range(len(b)):
            z = []
            for word in a:
                  ml = 0
                  index = -1
                  for letter in word:
                        index += 1
                        if letter == (b[counter])[index]:
                             ml += 1
                  if ml == int((b[counter])[-1]):
                        z.append(word)
            y.append(z)
            e.extend(z)
       for i in e:
            if (y[0]).count(i)==(y[1]).count(i)==(y[2]).count(i)==1 and s.count(i)==0:
                  s.append(i)
       return s

 print word_detect(raw_input('list:   '). upper().  split(','),raw_input('words:  '). upper(). split(','))

'''
e.g:
list : total,relax,mixer,remix,scope,candy,water
words: helix 3, botex 1, below 2
result: ['relax']
RELAX
hELiX - 3 matched letters
boteX - 1 matched letter
bELow - 2 matched letters
'''

Github issues are for bug reports, for support questions please use the dedicated channels: https://github.com/kivy/kivy#support.


Paging test moved from wormtable to simulated backend.

New TestAbstractBackend class introduced in unit/test_backends.py,
gathers together backend implementation independent logic,
TestSimulatedBackend and TestWormtableBackend classes inherit from it.
TestPaging is a new subclass of TestSimulatedBackend. This setup
should provide a clean migration path out of Wormtable for testing backend.
For now, I could only pull TestVariantSets.testPagination() out of
TestWormtableBackend, as SimulatedBackend only provides mock objects
for VariantSets. As SimulatedBackend grows, the other tests in test_backends.py
can be pulled out as well.
I could technically pull out TestVariantSets.testIds() now as well, but this doesn't fit
the new TestPaging class's focus on pagination.
Addresses issue #241

This looks good, thanks @macieksmuga. I wonder if there's much point in trying to make a general superclass of our new tests and the Wormtable backend test --- we're aiming to delete TestWormtableBackend, so there's not much point in trying to share code nicely with it.
The approach I would suggest here is to make a new class, TestBackend(unittest.TestCase) that is completely separate from the current inheritence tree. This should create a simulated backend, and test the code in backend.py. This means tapping in to the top-level calls like searchVariantSets and verifying that they return the same values as we would get if we accessed the VariantSets directly. That is, we run self._backend.searchVariantSets(), and compare this with self._backend.getVariantSets().
Paging is actually the main thing we want to test within backend.py, so I think we should be a bit more comprehensive: the number of elements returned should always be <= the requested page size, we should never return 0 length pages, as well as the consistency checks we're doing here.
I guess this has taken us a long way from your actual PR - sorry! What you have done here is good and we can merge this, and move on with adding more tests if you prefer. I just thought it would be good to explain what I think we're trying to test in this module now (in the past, it was basically everything).

@jeromekelleher There were a couple of possible directions I could go in to resolve the issue, and this PR is so far a quick sketch to get your take on the matter. Thus I really appreciate your comments above.f
As you mentioned, up to this point this module has tested... kind of everything (including data-driven-like tests). Thus, I am sympathetic to your suggestion to convert this to a collection of paging-oriented tests against SimulatedBackend, and unless you comment further on the below issue/suggestion, I will suspend this PR for now and implement what you suggested above.
But I have one conceptual problem with this approach, and it's the reason why I structured the test class hierarchy the way I did: such a testing regimen would then exclude unit-test coverage of the FileSystemBackend class. Admittedly this is a pretty lightweight class that mostly hooks up lower-level access to variants, reads and references (with the bulk of the code now in AbstractBackend), but I still felt uncomfortable cutting it out of the unit tests.
With the current structure in test_backends.py, I could write unit tests specifically against methods in every class in the backend hierarchy separately. The next step of this approach would be to transition TestWormtableBackend to TestFileSystemBackend, and switch its setUp method to generate/point at the .tbi indices, an approach I believe @dcolligan started to investigate.

Ah, now I get it! Yes, everything you're saying here makes sense @macieksmuga, and I agree it would be nice to get some sort of coverage on the FileSystemBackend class. We can make a dataset out of the test data in tests/data which has exactly the right structure and all the files are already indexed. I take back the points about the class hierarchy above, you're spot on.

Thanks @jeromekelleher! I'll go ahead with the current structure, beef up SimulatedBackend to enable more tests against it (that effectively end up testing AbstractBackend, I belatedly realized), and fill out the gaps you've pointed out elsewhere.

Thinking about this a little bit more, I'll just make TestAbstractBackend use the SimulatedBackend and remove the now redundant TestSimulatedBackend class altogether (after all, the purpose of SimulatedBacked was precisely to enable running unit tests against the methods in AbstractBackend)

Sounds great @macieksmuga, I'm looking forward to seeing the results!

@jeromekelleher I spent some time trying to implement what I outlined above, and fairly quickly discovered that many of the existing tests in test_backends.py are indeed more data-driven than unit: For example, switching over to a pysam.VariantFile backed TestFileSystemBackend ran right into the current issue with HtslibVariantSet.getVariants() not implementing restriction by callSetIds.
With this experience behind me, I'm now inclined to follow your initial proposal, namely unit-test all the methods implemented in AbstractBackend with an emphasis on paging, and leave coverage in the unit-tests of the FileSystemBackend to a simple check that it loads OK, and that the various _*SetIds are correct.

@jeromekelleher I've implemented all the unit tests I can for backends.py, but am running across this:
AttributeError: 'HtslibVariantSet' object has no attribute 'getCallSets'

This is a method that's instantiated both in the Simulated- and WormtableVariantSets, so should have at least a trivial implementation in HtslibVariantSet (that possibly throws a NotImplementedException ? - I added it in for now).
Anyway, I'd strongly prefer to leave this test in and have somebody (perhaps myself) write the actual missing method rather than try to excise the test out, even though it will break the CI build. Let me know what you think about this.

This looks great, thanks @macieksmuga. For getCallSets I would add a trivial implementation in the superclass so your tests pass. This is being tackled in #286.

thanks @jeromekelleher! pushed new version with your corrections.

Great! +1, assuming CI passes.

yeah, well, 'CORS it won't ;)


[bugfix] Singularize ReasonCode representation name

Description
Adjusts representation name to be singular which was raising an error when resource.rb was trying to call plural_name on it.

:shipit:


FIx: version resolution from string

Fix
Fix version detection in string. This fixed error could break running create_zip command in case the version of OP was "pure" - 3.15.1 (instead of 3.15.1-nightly.1).

This should be probably based on develop branch


XSS prevention hardening (Return YouTube Dislike)

As in title. The code already seems to have enough amount of XSS prevention, but this PR prevents any non-number data to be displayed on the webpage. It's not a big deal on first glance, but RYD is 3rd party so it should be primary to treat it as "unsafe" (I mean yes, it's popular, but that's a good security measure to treat it as "unsafe" anyways), to even prevent unwanted text on the webpage in case of RYD getting hacked. Any non-number data will be displayed as NaN, as expected (since this should never happen if RYD works properly, otherwise there's something wrong that should be fixed).
As of now, anything that isn't a number can be displayed anyways, it's not a big deal since textContent seems to prevent XSS, but it would be scary for users to see a message like "You're hacked by XYZ".

Never thought of this, thanks. I just added it to my in-development version and it seems to work fine, so it'll be in the next beta (which is probably releasing ahead of schedule to include compatibility with YouTube's new watch grid layout).
Gonna be honest, this would be the first pull request I haven't rejected, so I'm actually kinda not sure what to do here. Not sure if I should merge this pull request, since there will not be any new updates yet, and the emerald.js in my development version has already changed a bit since Beta 1's release (not the part of the code that this alters, but still).

From what I think, if you merge this request and this code will also already be in your local Git then I think it should merge just fine. If this code wasn't in your private Git project, it could make a merge conflict, but I really doubt in this situation.

Then it should be fine merging it.
If you'd like to improve that workflow (with Visual Studio Code - yes, I prefer Notepad++ too though), let me know.

Very well. I will try it.


Problems with nested routes

I have a simple router:
const router = new Router({
	mode: 'hash',
	routes: [
		{
			path: '/',
			name: 'layout',
			component: MainLayout,
			children: [
				{
					path: '',
					name: 'Dashboard',
					component: Dashboard,
					meta: {
						title: 'Dashboard',
					},
				},
			],
		},
		{
			path: '/login',
			name: 'login',
			component: Login,
			meta: {
				title: 'Login',
				allowAnonymous: true,
			},
		},
	],
})

router.beforeEach = async (to, from, next) => {
	if (!to.meta.allowAnonymous && get(userStore) == null) {
		next('/login')
	} else {
		document.title = to.meta.title
			? `${to.meta.title} | ${window.config.appName}`
			: window.config.appName
		next()
	}
}

export default router

when I try to navigate to /login, this works nicely, but as soon as I try to navigate to the dashboard, I'm getting this error in the console:

svelte-easyroute.umd.js:1 Uncaught (in promise) TypeError: Cannot read property 'forEach' of undefined
at e (svelte-easyroute.umd.js:1)
at e.parsePaths (svelte-easyroute.umd.js:1)
at e.getPathInformation (svelte-easyroute.umd.js:1)
at new e (svelte-easyroute.umd.js:1)
at create_fragment$2 (App.svelte:3)
at init (index.mjs:1451)
at new MainLayout (App.svelte:3)
at Object.update [as p] (RouterOutlet.svelte:72)
at update (index.mjs:733)
at flush (index.mjs:702)

In the previous version (2.1.0) I used this pointed to PathService.parsePaths if that helps you.
So I tried to clone your repo and try the demo app, but that doesn't seem to work for me either. There I'm receiving

Cannot GET /

I didn't change anything in your example project, just installed necessary npm modules.
Can you help me with this please?

Unbelievable mistake. Was importing RouterOutlet instead of { RouterOutlet } and no IDE or whoever complained.


Add feature to delay listen until services have been fully configured

When a bedrock application is started using multiple workers and the application needs to hit its own services during start up, the current implementation allows for the master thread to delegate a pending request to a worker that isn't ready to handle it, resulting in failure. This is because the current behavior for bedrock-server is to start listening early (when the bedrock.admin.init event is emitted), thereby making any worker available to receive requests.
This early listening was done to allow for binding to privileged ports (e.g., ports 80/443) during startup when the application was running as root prior to switching users. This sort of setup is generally no longer used in modern systems -- where applications instead always run as some user, bind to some non-privileged port, and run behind proxies like nginx. This means we don't really need this capability.
That being said, instead of creating a breaking change to the implementation, we could instead create a configuration option to address the above issue. In order to address the issue, we need workers to delay binding to ports until after all handlers (e.g. bedrock-express routes) have been attached. We need to explore if additional changes would need to be made to accommodate this that might result in a breaking change. For example, other libs like bedrock-express might need to attach earlier than the bedrock.ready event to allow for bedrock-server to start listening on bedrock.ready... or perhaps the new config feature can cause bedrock-server to simply move its listen call to bedrock.started without issue.
Note: It's also possible that a simpler (?) implementation change where the master process continues to bind early and only the workers delay binding will work -- without need for a configuration feature (this may not be a breaking change because it doesn't change basic expectations in behavior). If so, we could potentially listen on the master thread in the same location as the current implementation and move listen to bedrock.started for the workers.

Fix landed in 2.3.3.


Animations: Add CSS Checkbox Styles

Complete the following REQUIRED checkboxes:

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this issue follows the location for request: brief description of request format, e.g. Foundations exercises: Add exercise for XYZ

The following checkbox is OPTIONAL:

[x] I would like to be assigned this issue to work on it

1. Description of the Feature Request:

2. Acceptance Criteria:

3. Additional Information:

plz assign it to me

Please provide more detail if this is a feature request, I'm a little unsure what you're asking.


code wont work for latest version of bazel

not compatible with latest bazel version

Hi Xavier,
this commit https://github.com/eth-sri/ModelsPHOG/commit/ab5a90a13cf4993c15970709586efac140d6755d updates the build to work with bazel v0.22.
cheers,
Pavol


Windows Port

This is the beginning of the port.
The HAVE_SYSLOG is kinda ugly imo, but the code is littered with these anyway... thoughts anyone else?
My next step will be implementing winsock (started doing that).
@amcclure @krixano @hb9kns would you three mind looking at this? (can't seem to request reviews from non-members of the gophernicus org).
Closes #24.

It now compiles and runs on Windows but I think I broke something on Linux, oops.


Enhanced Cairo graphics renderer

Pull request for my work on the Cairo renderer.
Already very consistent with flash: http://i.imgur.com/TDRPlvG.png
Runs BunnyMark at low FPS: http://i.imgur.com/XZBGx6g.jpg

Wrote a quick text renderer using FreeType's decompose feature. http://i.imgur.com/76WBV4l.png
Still needs some work; lime's binding doesn't support cubic curves properly. If I add cubic support it will look better but I'm guessing at this point it might break something else. Is the way to go or should I be doing it like the OpenGL renderer and using bitmaps?

It looks cubic curves are being approximated using a quadratic:
https://github.com/openfl/lime/blob/f77e9caef401650618845198cb3b13d847b10645/project/src/text/Font.cpp#L242
In the legacy renderer, I think this was the right behavior (cubic curves used to be ignored entirely, which was fun for certain fonts :wink:)
Perhaps now we should preserve cubic curves, since the renderers do have cubicCurveTo support (or at least I think they do?). If they don't, the approximation to a quadratic curve should occur there, not during a decompose

We could keep the bitmap images and use the TextFieldGraphics class like GL does, but I suppose that using vectors to render does have certain value to it, particularly if text sizes get enormous

Sorry, I should have said not to merge yet. I just wanted a place to post current work and discuss the changes.
I was wrong about the cubic curve thing. I just had a mistake in my code. It looks great now http://i.imgur.com/5TWleCD.png
In any case, you are right that lime should preserve the information and let the approximation or proper renderering take place later. I will post a pull request in lime for that.

Things are look much better. I've noticed three things in testing, some rounded rectangles (not sure it's actually a roundRect call, or just a rounded rectangle by geometry) end up looking like a triangle instead (one short side, one long side), I have some masked graphics that are not appearing (might be something with a matrix) or some word wrapping issues with text, but that's to be expected -- text will just continue to evolve (and hopefully we find ways to unify the TextField implementation further for this)

Thanks, I'll keep working on it and hopefully fix those up soon.
The TextField vector implementation is a drop in replacement for the bitmap one. It just sends draw commands to the graphics object. Problem is the OpenGL renderer isn't great at drawing shapes so text is illegible.

Huh, the GL text is supposed to go through Freetype (which renders each requested glyph to a bitmap), before doing a drawTiles render, based on the tilesheet created

However, the best implementation (other than legacy) right now is the HTML5 one, which still uses direct canvas calls, and needs to be made a bit more universal (so wordWrap and other features can be shared)

Yes the GL renderer does that. What I meant was, my vector renderer can be dropped into the GL renderer and it works because it just makes calls to the graphics object. But the GL renderer isn't great at shapes so it isn't readable.

Oh, oh, I see, so we have the ability to render using shapes or using bitmaps. Although I'm tempted to stick with one implementation (to keep maintenance simpler), I wonder if we really should have both (like, say, if the glyph is larger than X points, then just render the shape, etc)

Cairo actually has built in support for FreeType. I was able to get it to render text very nicely yesterday with pixel hinting and it looks beautiful. Almost identical to flash. I will post examples later today. Once we have Cairo being used with the GL renderer I think we will only need this one text rendering path as we can cache to bitmap (or Cairo surface) anyway and it's the same result only nicer.

Here is a comparison of Flash text rendering and the Cairo text renderer. http://i.imgur.com/MsRm1A1.png


[graphql] Allow for shorthand representation of the coin type

Description
Allow for shorthand representation of the coin type.
Test Plan

On Error:

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Thanks!
Here's the output with error propagation.

a caller providing 0x2::sui shouldn't be considered an internal error, the language misrepresents who is at fault for the error response

a caller providing 0x2::sui shouldn't be considered an internal error, the language misrepresents who is at fault for the error response

Right, so then I'd add a new error to our error type.

I agree with both of you. We'll have to revisit this later when we move this conversion to the reader side.

Note that in some cases, this is an internal error, and in some cases this is a user error -- it depends on the caller, this is why the DB layer needs its own error type.

I think it makes sense for db_data_provider to control whether this is surfaced as an internal or client error, mainly because it sits between graphql and the actual db reader (IndexerReader). When the InvalidCoinType occurs on an arg provided to a db_data_provider method, it'll always return the error as a client error. If the parsing error occurs on an object read from the table, then it will be treated as an internal error. Then,

If this is used by another db_data_provider method, that method also has enough context to determine whether it is a user error or not. Say method_b fetched an object and passed its coin_type to db_data_provider.method_a, method_a returns client error but method_b knows its an internal error.
The layer above the db_data_provider would be the graphql resolver fields, which may not have all the context needed to determine error kind. If it sees an InvalidCoinType, it may not know if its due to the args given to itself (the resolver) or if it came from somewhere else down the line


Fixes/typos

First bunch of typos in input_plugins configuration

This project is no longer maintained. I recommend you check out golja/influxdb for an updated Puppet module for InfluxDB.


Docker Backward Compatibility with Linux Kernel

I was running Docker 1.7.1 in a Linux machine and it worked fine.  I had some issues with my environment and installed docker using below command which gives latest version 1.12.2
curl -sSL https://get.docker.com/ | sh
When I tried to start docker daemon, I hit the error
FATA[0001] Your Linux kernel version 3.8.13-68.3.4.el6uek.x86_64 is not supported for running docker. Please upgrade your kernel to 3.10.0 or newer.
Any help on the below questions would be helpful.

Is there any way to download docker 1.7.1 instead of 1.12.2  ?
Is there any docker version vs Linux version compatibility matrix ?
Is there any way to override the above error message. Make 1.12.2 run on old linux kernels ?

Install script only installs latest version. Packages for old versions are still available but must be installed manually then.
It should be 3.10 for 1.7+. I think the difference is that in older versions it was only a warning.
DOCKER_NOWARN_KERNEL_VERSION=1

@sivaswami To install specific docker version try this :
wget -qO- https://get.docker.com/ | sed 's/lxc-docker/lxc-docker-1.6.2/' | sh
@tonistiigi So if we use DOCKER_NOWARN_KERNEL_VERSION=1 then it is possible to run docker on any older kernels (older than 3.10)?

@milindchawre It will then skip this validation on startup. The behavior is undefined and unsupported.

Thanks @tonistiigi @milindchawre .
I think such backward compatibility might be  required  as people  might upgrade docker more than their kernels. It would also be helpful if we have a FAQ session for installing / compatibilities

@sivaswami it is unsupported as containerisation features and networking may not work on older kernels, so we cannot support them. We go back as far as we can. There are bugs in earlier versions that cannot be worked around. They may not affect your use cases.

Closing as answered - if you have more questions please continue to ask...


CD-360809: Resque UI is broken for ruby3 because yield keyword is throwing syntaxerror.

JIRA

CD-360809 Main JIRA ticket

Code reviewers

[ ] @revathi-murali

Summary of issue

Resque UI is broken because yield in class syntax is deprecated in ruby3. Ref here.
As coupa_development is using ruby3.x this fix will help getting rid using deprecated things.
Attached deprecation warning raised when using ruby 2.7.1.

Broken UI error with Ruby3:

Summary of change

Pass html content as locals instead of passing it as block to layout.

Testing approach

Tested manually.

Feature Test Options

[x] All tests

cody rebuild

Is it possible to add some unit tests for the rendering of the resque UI?

Closing this pr as issue is specific to coupa_clm.


Is there a way to rename interfaces in docs for Symbols References?

Hey there,
I've tried to link in the Readme.md a symbol and I saw that I can set a link to a symbol:
bla bla bla [[Props]] bla bla bla
or
bla bla bla [[Props | ProjectProps]] bla bla bla
Sadly I have to leave "Props" named as is, but this name is confusing and I'm exposing this interface to the outside of the project as ProjectProps.
export { Props as ProjectProps } from "..";

Also, what happens if I have two interfaces, from different parts of the project, that are named both "Props"?

I was thinking: is there a way, like comments, to rename an interface for docs only? If not, can this be included? (EDIT: found this https://github.com/TypeStrong/typedoc/issues/1201)
E.g. JSDoc through tag @alias
/**
 * @alias ProjectProps
 */
export interface Props {
  ...
}

Result in docs would be something like this:
bla bla bla [[ProjectProps]] bla bla bla

will be like:
bla bla bla [[Props]] bla bla bla

Thank you very much.
P.s. I did not use Feature Request template because I'm not sure if this thing already exists or not.
If this becomes a Feature Request, If needed, I'll edit this post to the form.

It sounds like you are building with --mode file.
File mode is intended for projects which do not use modules. (tsc compilation with --module none). It pretends that every file is global, so any declarations with the same name will be merged.
Unfortunately, many projects (TypeDoc included) use file mode because modules mode results in incredibly ugly output that reveals internal file paths. This is being remedied with library mode (#1184) but that isn't ready for a stable release yet (though you can try it out by installing typedoc@next)
With library mode, Props will be renamed to ProjectProps in the documentation if that is what it is exported as.

Thank you for your reply.
Yeah, I'm actually using file mode. I've tried module mode but it did not seem to be suitable for the project I'm on at work (which is composed of a lot of internal imports). I should retry because I don't remember that much which was the reason, but for sure not the one you said because I currently see the full path of each declaration, etc.
How does typedoc gets told to rename Props in ProjectProps in library mode? Starting from the main file?
Anyway, the support to @alias from JSDoc should be added IMHO.

How does typedoc gets told to rename Props in ProjectProps in library mode? Starting from the main file?

Library mode will look at what is actually exported from the given entry point(s) using TypeScript's APIs to get the exported name. If you use typedoc --out docs src/index.ts, then the output will resemble file mode with all entries not exported marked with @hidden, but all exported symbols will be given the name they are exported under. If more than one entry point (or a directory) is specified, the output will be similar to modules mode where each entry point gets a top level entry and then includes all exported content inside it.
This (mostly) works already, but causes some problems if symbols are exported under multiple names. I think I'm getting close to ironing out the kinks there, but it is taking a while.

Anyway, the support to @alias from JSDoc should be added IMHO.

I'm loath to add support for @alias. TypeScript should know about all of the exports, and TypeDoc should be able to pick that information up from TypeScript. The @alias tag is a way to lie about how symbols are exported.

That's great to hear. So, I won't need anymore to use a lot of globs expressions to exclude files and folders that are getting included right now, isn't it?
About the @alias, yeah, sort of lie. Maybe it won't be needed anymore.

Correct, I hope that library mode should make it possible to stop using the exclude/external options.

I found that modules mode right now is better for the project I'm on. I don't know why I didn't like it before.
Anyway, it is a bit mode "pathy" and it has not all the voices on the side (also with toc options)

In 0.20 (currently typedoc@beta, TypeDoc asks TypeScript for the exported names, and uses those when generating documentation.


[Fix-8722][UI Next][V1.0.0-Alpha] Fix home page error bug

close #8722

Codecov Report

Merging #8731 (2321525) into dev (9c162c8) will increase coverage by 0.01%.
The diff coverage is 44.61%.

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

@@             Coverage Diff              @@
##                dev    #8731      +/-   ##
============================================
+ Coverage     45.29%   45.30%   +0.01%
- Complexity     4073     4077       +4
============================================
  Files           700      702       +2
  Lines         27050    27112      +62
  Branches       2894     2897       +3
============================================
+ Hits          12251    12284      +33
- Misses        13645    13676      +31
+ Partials       1154     1152       -2

Impacted Files
Coverage Δ

.../org/apache/dolphinscheduler/common/Constants.java
78.94% <ø> (ø)

...hinscheduler/common/enums/BlockingOpportunity.java
0.00% <0.00%> (ø)

.../dolphinscheduler/common/enums/StateEventType.java
0.00% <0.00%> (ø)

...apache/dolphinscheduler/common/model/TaskNode.java
25.20% <0.00%> (-0.21%)
:arrow_down:

...duler/common/task/blocking/BlockingParameters.java
0.00% <0.00%> (ø)

...hinscheduler/common/utils/TaskParametersUtils.java
50.00% <0.00%> (-2.18%)
:arrow_down:

...e/dolphinscheduler/dao/entity/ProcessInstance.java
66.00% <0.00%> (-1.35%)
:arrow_down:

...ache/dolphinscheduler/dao/entity/TaskInstance.java
64.45% <0.00%> (-0.40%)
:arrow_down:

...g/apache/dolphinscheduler/dao/utils/DagHelper.java
45.55% <0.00%> (-1.83%)
:arrow_down:

...dolphinscheduler/common/enums/ExecutionStatus.java
63.41% <60.00%> (+0.25%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Minor fixes in train.py (acc_wvi and torch.cuda.is_available())

This pull request fixes 2 issues:

there's a bug in the calculation of acc_wvi for the training loop
some of the torch.cuda.is_available() calls are broken

Thanks!


Select Content sizing / scrollbar

I didn't see an option to select "feature request" vs. "bug", but I guess this is mostly the former.
It would be nice to be able to configure a height for the Select.Content component, and have a scrollbar appear if it exceeds that height. It would also be nice if this somehow had sensible defaults, because right now, if the Select is inside of a Dialog and extends below the bottom of the screen, the scroll behavior for the dialog is ... weird (it's maybe not actually weird, but it feels weird that the whole dialog is scrolling instead of the contents of the select).

Hey @idapartners if you download and manually apply the recipe you can set overflow: 'auto' and maxHeight: 'md'. If you do not want to add the recipe to the project you can probably override these styles directly on a custom component.
Will make it look like this:


The format YYYY-MM isn't working with the money command

This only refers to when you are using month numbers as arguments, not when you are using options.


CB-17627 Periscope should rely on CM and YARN RM to determine cluster…

… state
See detailed description in the commit message.

retest this please

Please add detailed description to the commit message.

Please add detailed description to the commit message.

Done

@sidseth @bbihari requesting your updated review

@aakashshankar from cb-raven point of view it's ok, but I'd like someone from the periscope team to approve it first

Rebasing on top of master due to failing swagger checks


Match code generated by pythran 0.9.8

Take a reference on expressions, Pythran is able to distinguish between ref and
value, and better optimizes the former.

This should fix https://github.com/serge-sans-paille/pythran/issues/1706

This should fix https://github.com/serge-sans-paille/pythran/issues/1706

It looks like this fixed the numpy_pythran_unit test but broke the numpy_pythran test

It looks like this fixed the numpy_pythran_unit test but broke the numpy_pythran test

@da-woods am i right in thinking that the problem on master (and 3.0rc.x and 0.29.x) is in something pythran changed in a recent release?  If so, would pinning to the most recent non-world-breaking pythran be viable?

@jbrockmendel I believe you're right (although I haven't investigated fully). Pinning the version is probably a good suggestion for now. Although obviously that only really fixes the test-suite since we don't really control what version users will have.

Yeah, I'm very sorry about current situation. I've been optimizing pythran -generated code quite a lot over the past few months, and changed the C++ API in a way that's currently incompatible with cython integration. we have a bug opened for this...

since we don't really control what version users will have.

could check and raise on version mismatch.  is the dependency at compile-time or run-time?

I've created a PR to pin the test version as suggested (https://github.com/cython/cython/pull/4031). Obviously we should undo this as soon it can be made to work again.

could check and raise on version mismatch. is the dependency at compile-time or run-time?

I haven't done this for now (mainly out of laziness - it would involve working out what the actual last working version was). It's a compile-time dependency, which makes raising an error a bit easier I think.


Transaction complete: On mobile, after clicking "Share" and opening the mobile share  - auto direct to dashboard.

for mobile only:

use regular button without V -> Done
After clicking "Share" and opening the mobile share -  auto direct to dashboard.

this was the old behaviour on mobile

Verified as done on the Dev env v0.20.1
After user tap on the "Share" button they navigated to the dashboard and native share opened.


Invalid character.

Hi there,
I seem to be having an issue with selected articles which causes an error like ''', hexadecimal value 0x08, is an invalid character.' to occur.
An example article is http://www.lifehacker.com.au/2015/01/read-more-every-day-by-creating-reading-triggers/
This is running ASP.NET 4.0 and the NReadability package was downloaded via NUGet.
Current running code is:
NReadability.NReadabilityWebTranscoder tc = new NReadability.NReadabilityWebTranscoder();
NReadability.WebTranscodingInput ti = new NReadability.WebTranscodingInput(url);
NReadability.WebTranscodingResult tcr = tc.Transcode(ti); //Exception thrown on this line.
Response.Write(tcr.ExtractedContent);
(however I've tried variations of different code, including that which is included in the readme)
I reliase this is due to incorrect tags being read by the Xml reader within NReadability however I do not seem to be able to work around this.
Suggestions?

Hi,
maybe you could try to do some preprocessing on the article - for example
remove invalid characters that are non-printable anyway.
On 21 January 2015 at 08:16, mscottco<EMAIL_ADDRESS>wrote:

Hi there,
I seem to be having an issue with selected articles which causes an error
like ''', hexadecimal value 0x08, is an invalid character.' to occur.
An example article is
http://www.lifehacker.com.au/2015/01/read-more-every-day-by-creating-reading-triggers/
This is running ASP.NET 4.0 and the NReadability package was downloaded
via NUGet.
Current running code is:
NReadability.NReadabilityWebTranscoder tc = new
NReadability.NReadabilityWebTranscoder();
NReadability.WebTranscodingInput ti = new
NReadability.WebTranscodingInput(url);
NReadability.WebTranscodingResult tcr = tc.Transcode(ti); //Exception
thrown on this line.
Response.Write(tcr.ExtractedContent);
(however I've tried variations of different code, including that which is
included in the readme)
I reliase this is due to incorrect tags being read by the Xml reader
within NReadability however I do not seem to be able to work around this.
Suggestions?
—
Reply to this email directly or view it on GitHub
https://github.com/marek-stoj/NReadability/issues/19.

preprocessing does not help
var preprocessed = new Regex(@"[\x00-\x1F\x7f]", RegexOptions.Compiled).Replace(htmlArcticle, string.Empty);
var transcoder = new NReadabilityTranscoder();
var input = new TranscodingInput(preprocessed);
var transcodedContent = transcoder.Transcode(input); //throw

it removed hexadecimal value 0x08, but throw exception "Non white space characters cannot be added to content."
in System.Xml.Linq.XDocument.ValidateString(String s)
in System.Xml.Linq.XContainer.AddStringSkipNotify(String s)
in System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
in System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r, LoadOptions o)
in System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
in System.Xml.Linq.XDocument.Load(XmlReader reader)
in NReadability.SgmlDomBuilder.LoadDocument(String htmlContent)
in NReadability.SgmlDomBuilder.BuildDocument(String htmlContent)
in NReadability.NReadabilityTranscoder.TranscodeToXml(String htmlContent, String url, Boolean& mainContentExtracted, String& extractedTitle, String& nextPageUrl)
in NReadability.NReadabilityTranscoder.Transcode(TranscodingInput transcodingInput)
value of preprocessed in file https://www.dropbox.com/s/mcyooywzw03szv0/testData.html?dl=0
Can you check?


TransactionalEmailsApi.getEmailEventReport returns missing_parameter for EndDate even though no StartDate was given

I am calling this function: https://github.com/getbrevo/brevo-node/blob/07cc1ff8d503a93f17c62535df61da4d1d42796a/api/transactionalEmailsApi.ts#L726C18-L726C37
Like this:
transactionalEmailsApi.getEmailEventReport(1, 0, null, null, 1, EMAIL, 'delivered', null, null, 21);
As such, I am leaving out some params that I don't need for the query. However, I receive {"code":"missing_parameter","message":"Start date is missing"}
Expected behavior: Since neither start date nor end date is given, I should not receive an error.
Node v20.11.1

Oh, when I provide start and end date, it next tells me that the message id is missing!

It looks like I can get what I want with getTransacEmailsList(email, templateId)


Black screen shows up when a project is opened in godot.

Godot version
4.3.stable
System information
Windows 10, Intel Core i3-8130U, GL4.5, Intel UHD Graphics 620
Issue description
When ever I am opening a project I only see a black screen, but the editor runs in the background. Can anyone help me with this problem.
Steps to reproduce
Open the editor without having the black screen.
Minimal reproduction project
N/A

Can anyone help.

Please give things some time, this isn't a help desk but a big report tool where people will look when able

Please give things some time, this isn't a help desk but a big report tool where people will look when able, you gave it all of 5 minutes
If you want rapid responses turn to the other community channels

Thank You

looks the same as #71104

Duplicate of https://github.com/godotengine/godot/issues/71104 (same cause).


Review requirements

Please review #1 the collected requirements so we can forward them to the clients.

Maybe you reviewed them but then please mark the pull request as approved.

duplicate of #1


Update TextualEncoding.md

@MarioDfinity i think it would be helpful to point interested parties to an implementation

@MarioDfinity can we merge this?

@letmejustputthishere feel free to merge when CI is done.


Error When Installing Module If AWS.Tools Modules are Installed

Issue Details
When installing the AWSSSOHelper module, if the new AWS.Tools PowerShell modules are installed, the installation will fail.
PowerShell to reproduce the issue
Install-Module AWS.Tools.Common -Force
Install-Module AWSSSOHelper -Verbose -Force

Verbose Output
VERBOSE: Using the provider 'PowerShellGet' for searching packages.
VERBOSE: The -Repository parameter was not specified.  PowerShellGet will use all of the registered repositories.
VERBOSE: Getting the provider object for the PackageManagement Provider 'NuGet'.
VERBOSE: The specified Location is 'https://www.powershellgallery.com/api/v2' and PackageManagementProvider is 'NuGet'.
VERBOSE: Searching repository 'https://www.powershellgallery.com/api/v2/FindPackagesById()?id='AWSSSOHelper'' for ''.
VERBOSE: Total package yield:'1' for the specified package 'AWSSSOHelper'.
VERBOSE: Performing the operation "Install-Module" on target "Version '0.0.21' of module 'AWSSSOHelper'".
VERBOSE: The installation scope is specified to be 'AllUsers'.
VERBOSE: The specified module will be installed in 'C:\Program Files\WindowsPowerShell\Modules'.
VERBOSE: The specified Location is 'NuGet' and PackageManagementProvider is 'NuGet'.
VERBOSE: Downloading module 'AWSSSOHelper' with version '0.0.21' from the repository
'https://www.powershellgallery.com/api/v2'.
VERBOSE: Searching repository 'https://www.powershellgallery.com/api/v2/FindPackagesById()?id='AWSSSOHelper'' for ''.
VERBOSE: Searching repository 'https://www.powershellgallery.com/api/v2/FindPackagesById()?id='AWSPowerShell.NetCore''
for ''.
VERBOSE: InstallPackage' - name='AWSPowerShell.NetCore',
version='<IP_ADDRESS>',destination='C:\Users\su_heathes\AppData\Local\Temp\1273408216'
VERBOSE: DownloadPackage' - name='AWSPowerShell.NetCore',
version='<IP_ADDRESS>',destination='C:\Users\su_heathes\AppData\Local\Temp\1273408216\AWSPowerShell.NetCore.<IP_ADDRESS>\AWSPower
Shell.NetCore.<IP_ADDRESS>.nupkg', uri='https://www.powershellgallery.com/api/v2/package/AWSPowerShell.NetCore/4.0.5'
VERBOSE: Downloading 'https://www.powershellgallery.com/api/v2/package/AWSPowerShell.NetCore/4.0.5'.
VERBOSE: Completed downloading 'https://www.powershellgallery.com/api/v2/package/AWSPowerShell.NetCore/4.0.5'.
VERBOSE: Completed downloading 'AWSPowerShell.NetCore'.
VERBOSE: Hash for package 'AWSPowerShell.NetCore' does not match hash provided from the server.
VERBOSE: InstallPackageLocal' - name='AWSPowerShell.NetCore',
version='<IP_ADDRESS>',destination='C:\Users\su_heathes\AppData\Local\Temp\1273408216'
VERBOSE: InstallPackage' - name='AWSSSOHelper',
version='0.0.21',destination='C:\Users\su_heathes\AppData\Local\Temp\1273408216'
VERBOSE: DownloadPackage' - name='AWSSSOHelper',
version='0.0.21',destination='C:\Users\su_heathes\AppData\Local\Temp\1273408216\AWSSSOHelper.0.0.21\AWSSSOHelper.0.0.21
.nupkg', uri='https://www.powershellgallery.com/api/v2/package/AWSSSOHelper/0.0.21'
VERBOSE: Downloading 'https://www.powershellgallery.com/api/v2/package/AWSSSOHelper/0.0.21'.
VERBOSE: Completed downloading 'https://www.powershellgallery.com/api/v2/package/AWSSSOHelper/0.0.21'.
VERBOSE: Completed downloading 'AWSSSOHelper'.
VERBOSE: Hash for package 'AWSSSOHelper' does not match hash provided from the server.
VERBOSE: InstallPackageLocal' - name='AWSSSOHelper',
version='0.0.21',destination='C:\Users\su_heathes\AppData\Local\Temp\1273408216'
VERBOSE: Validating the 'AWSPowerShell.NetCore' module contents under
'C:\Users\su_heathes\AppData\Local\Temp\1273408216\AWSPowerShell.NetCore.<IP_ADDRESS>' path.
VERBOSE: Test-ModuleManifest successfully validated the module manifest file
'C:\Users\su_heathes\AppData\Local\Temp\1273408216\AWSPowerShell.NetCore.<IP_ADDRESS>'.
VERBOSE: Validating the authenticode signature and publisher of the catalog file or module manifest file of the module
'AWSPowerShell.NetCore'.
VERBOSE: Catalog file 'AWSPowerShell.NetCore.cat' is not found in the contents of the module 'AWSPowerShell.NetCore'
being installed.
VERBOSE: Valid authenticode signature found in the file 'AWSPowerShell.NetCore.psd1' for the module
'AWSPowerShell.NetCore'.
VERBOSE: Checking for possible command collisions for the module 'AWSPowerShell.NetCore' commands.
PackageManagement\Install-Package : The following commands are already available on this system:'Clear-AWSCredentials,C
lear-AWSDefaults,Get-AWSCredentials,Initialize-AWSDefaults,New-AWSCredentials,Set-AWSCredentials,Add-AWSLoggingListener
,Clear-AWSCredential,Clear-AWSDefaultConfiguration,Clear-AWSHistory,Clear-AWSProxy,Clear-DefaultAWSRegion,Disable-AWSMe
tricsLogging,Enable-AWSMetricsLogging,Get-AWSCmdletName,Get-AWSCredential,Get-AWSPowerShellVersion,Get-AWSPublicIpAddre
ssRange,Get-AWSRegion,Get-AWSService,Get-DefaultAWSRegion,Initialize-AWSDefaultConfiguration,New-AWSCredential,Remove-A
WSCredentialProfile,Remove-AWSLoggingListener,Set-AWSCredential,Set-AWSHistoryConfiguration,Set-AWSProxy,Set-AWSRespons
eLogging,Set-AWSSamlEndpoint,Set-AWSSamlRoleProfile,Set-DefaultAWSRegion'. This module 'AWSPowerShell.NetCore' may
override the existing commands. If you still want to install this module 'AWSPowerShell.NetCore', use -AllowClobber
parameter.
At C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\2.2.3\PSModule.psm1:9685 char:34
+ ... talledPackages = PackageManagement\Install-Package @PSBoundParameters
+                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Microsoft.Power....InstallPackage:InstallPackage) [Install-Package],
   Exception
    + FullyQualifiedErrorId : CommandAlreadyAvailable,Validate-ModuleCommandAlreadyAvailable,Microsoft.PowerShell.Pack
   ageManagement.Cmdlets.InstallPackage

Suggested Resolution
To support both the legacy and new AWS PowerShell modules, remove AWSPowerShell.NetCore as a required module, and document in the README options for prerequisite modules; AWSPowerShell.NetCore or AWS.Tools.Common, AWS.Tools.SSO and AWS.Tools.SSOOIDC.

I have a branch that fixes this if you want me to submit a PR.

Hi, thanks for sharing detailed bug report and solution.
I seem to recall that I had to specify using #Requires otherwise I had issues with modules not being imported implicitly (I didn't troubleshoot).
With your updated branch, could you confirm if after loading a clean PS session you are able to run 'Get-AWSSSORoleCredential' without first importing any AWS modules?

Yes you are right, the AWSPowerShell.NetCore module doesn't explicitly export it's functions so they won't be auto-discovered. I've added some code to my branch to check if this module exists and then explicitly import it to work around this.
This code isn't needed for the AWS.Tools modules as they explicitly export all their functions and so the module auto-loads correctly.
I've tested this new code from a clean PS session, once with AWSPowerShell.NetCore installed and once with the AWS.Tools modules installed, and both work successfully.

It is also a simple change to make this module also support PowerShell 5.1. I have made the additional changes in this branch, and tested with both the AWSPowerShell and AWS.Tools modules.

Awesome!  If you wanted to submit and PR I'll push it to PSGallery.
Thanks for the support and hope you find the module useful.


fix Dockerfile - change the bazel version to 0.28.0

In the latest master the Docker build crashes. The issue is related to Bazel configuration / version. After running
docker build --tag=open_dataset -f tutorial/cpu-jupyter.Dockerfile  as described here, the following error message is thrown in the log output:
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=0 --terminal_columns=80
INFO: Reading rc options for 'query' from /waymo-od/.bazelrc:
  'query' options: --incompatible_bzl_disallow_load_after_statement=false
ERROR: Unrecognized option: --incompatible_bzl_disallow_load_after_statement=false
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=0 --terminal_columns=80
INFO: Reading rc options for 'build' from /waymo-od/.bazelrc:
  'build' options: -c opt --cxxopt=-std=c++11 --auto_output_filter=subpackages --copt=-Wall --copt=-Wno-sign-compare --linkopt=-lrt -lm --incompatible_bzl_disallow_load_after_statement=false --action_env TF_HEADER_DIR=/usr/local/lib/python3.6/dist-packages/tensorflow_core/include --action_env TF_SHARED_LIBRARY_DIR=/usr/local/lib/python3.6/dist-packages/tensorflow_core --action_env TF_SHARED_LIBRARY_NAME=libtensorflow_framework.so.2 --action_env TF_NEED_CUDA=0 --cxxopt=-D_GLIBCXX_USE_CXX11_ABI=0
ERROR: Unrecognized option: --incompatible_bzl_disallow_load_after_statement=false

One quick fix is to just use the Bazel version 0.28.0 that is explicitly used in this colab tutorial version.

Thanks for the change. Can you sign CLA?

I've signed it.
On Mon, Apr 27, 2020 at 4:16 PM Pei Sun<EMAIL_ADDRESS>wrote:

Thanks for the change. Can you sign CLA?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waymo-research/waymo-open-dataset/pull/145#issuecomment-620284468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQ4WANLRSCNW55PLEAFQI3ROYG4FANCNFSM4MRT6RNQ
.

The check is still failing.  do you mind trying to sign it again?

@googlebot I signed it!

The check is still failing. do you mind trying to sign it again?

Should be ok now.


Added solution to the for loop problem on hackerrank in cpp

Related Issue
Solved issue number: 630
Describe the changes you've made
Added solution to the for loop problem on hackerrank in cpp
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
C++
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have documented the code.
[x] New and existing unit tests pass locally with my changes.

Hey @msbuddhu Mention your issue number that you have solved.

My issue number is #630

No, this is your pull request number. Mention the issue number in which you are assigned to do this work.

Please help me. I can't find the issue number.

Have you assigned to any issue ?

No.

That's why. See this

Get yourself assigned first to the issue on which you want to work.


Refactor getting remote metadata

Very small refactor to getting the remote metadata last modified time.

Sync on Remote will no longer spam the remote with requests to get metadata and will wait until the previous is finished.

Also I found that when getting all the files, "remotely-sync" files were being skipped instead of "remotely-secure".

Awesome, thanks for another great PR!


Schemas v3.3.0

What is the context of this PR?
Updates schemas to the latest version.
How to review
Ensure the latest schema is loaded successfully, and you are able to launch RSI.
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

Superseded by https://github.com/ONSdigital/eq-questionnaire-runner/pull/864


Strip nulls

An Advanced query if successfully executed by BCP may contain nulls in the output file.
Strip them away with sed -i 's/\x0//g' outputfile.csv

fixed by 3.0.4


Add configurable branch to autocommit.sh

Make the default branch as a configurable option, because the default branch of some newly
created repositories will be main instead of master

OK. Sorry I didn't know it was intentional.
I kept the default as master so functionally it's still the same.
Thanks for the merge!


fix #18: nested props are ignored if the first one is undefined

I did a little research on #18 and it turns out that the bug is only present if the first prop is undefined. This pr fixes the issue and adds a test-case.
closes #18

sorry reviewing these great PRs took a few days, been afk

@argyleink No prob. Living outside of github is a thing I've heard of ;)
Would you be able to push a release with these fix? Atm I need to use a fork of this package for one of my projects because of this and I would love to sync to upstream again.


Highlighting doesn't work in Chrome

I'm using PrismJS on my Ghost blog: https://berndsgn.ch/
The syntax highlighting works well in any browser (even Chrome on mobile) except for Chrome on desktop where it doesn't show at all. I'm using the latest Chrome (Version 50.0.2661.75 (64-bit)).

FWIW, I looked at your site on both Chrome & FF, and neither of them highlight. I also see an error in the console in Chrome from an unrelated script.

thanks, @mAAdhaTTah. i didn't know that i had the same problem with FF... i have no idea where it comes from though...

Interestingly, it works if I call Prism.highlightAll() from the console, so it looks like some sort of race condition. Perhaps you're loading the posts asynchronously?

This issue did not appear to be due to Prism itself, so I'm closing it.


:inbox_tray: Add Bestagon SiDB gate library

Description
Since a significant amount of the experiments use the Bestagon SiDB gate library, I propose to add it. This PR includes it with an additional tag for the canvas SiDBs (cell_type::LOGIC).
Checklist:

[x] The pull request only contains commits that are related to it.
[ ] I have added appropriate tests and documentation.
[ ] I have added a changelog entry.
[ ] I have created/adjusted the Python bindings for any new or updated functionality.
[x] I have made sure that all CI jobs on GitHub pass.
[ ] The pull request introduces no new warnings and follows the project's style guidelines.

Good point! I will!


Facing issue inside modals

I am facing issue while trying to open captcha inside modals. Captcha is not opening.
@douglasjunior Can you help to solve this issue?

This library uses the native Modal from React Native.
If you are using a native modal to render Recaptcha you may have a problem because opening a Modal over another Modal can be a trick.


MD-1120: added summary for iterations

@Agirish Can you review this PR? I've tested locally as well. Thanks.

Hey Chun, thanks for making this change. Do we only print the count of tests that passed, failed (& such) or also the list of failures, along with the exception messages for Execution Failures & Baseline mismatch summary per test for Verification Failures?
Although the total count will be useful, more useful for analysis would be the ones above.

@Agirish All the other messages are logged during each iteration, so they are available if needed to be looked at. If you feel strongly we need to log them again in the final summary, please file a separate JIRA for further discussion. Thanks for the review.

Agreed. +1


Docs: Remove code tags inside inline code blocks

Description

Was not standard. Later nameof can be utilized
How Has This Been Tested?

Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation (fix or improvement to the website or code docs)

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.

@henon Is removing the <> for components/blocks the right thing? Seems to be a standard if we look at other components like tree, select, table. Tho there are definitely some mistakes, for example the AllowUnsorted definitely shouldn't be <AllowUnsorted> as it's a parameter and InitialDirection too.

@henon Is removing the <> for components/blocks the right thing?

It would undo what I explicitly added to distinguish components from parameters in the help text. Of course the bugs where someone used the  <>  around parameter names should of course be fixed. I guess these are copy/paste errors

It would undo what I explicitly added to distinguish components from parameters in the docs. So I'd like to keep it that way, except if there are good arguments against it.
Of course the bugs where someone used the <> around parameter names should of course be fixed. I guess these are copy/paste errors.

I guess we could just add a string extension method to docs that wraps into <>. It will be faster to Ctrl+V it, since not sure if copilot can distinguish blazor component and blazor parameter.

No need, codeInline can allready do it via <CodeInline Tag> ... </CodeInline>

It would undo what I explicitly added

I was wrong here, I certainly didn't add the < and > chars, I just used the Tags parameter in other places, so this is what we should do here too.


Ability to seed conversations with arbitrary training text

Within any given project, I'd like to be able to configure the project-specific extension settings to seed each new conversion with arbitrary training text. This could be a long form set of guidelines and rules for the active project - or it could be used to fill gaps in ChatGPT's knowledge gaps.
Ideally, I'd like to be able to set the seed text in json - or (easier to maintain) as a link to a local or absolute text file path or perhaps as a remote http ref.
Example text to seed a conversation.

If you aren't sure of the answer, please answer "Unknown".

Assume we always want examples that are compatible with Python 3.7.

I'm working in a React project using Next.js. please provide answers with those contexts when possible.

Downside of this would be a slightly longer start time, but the benefit would be an iteratively improving answer experience as more guidance is built into the training text over time.

@aaronsteers I like this idea. What you are talking about is the predefined context for GPT models, which the extension already defines for each question. I could make it an option in the settings so that you can update it. And if you prefer that to be more dynamic, you could just set that option  to an empty string "" and you would provide the context in every question instead.
I will update here once it's rolled out, when I get some free time to work on this.
Thank you!

I came up with the same idea when I came across https://prompts.chat/. I started initializing my conversations with such prompts. Would be awesome to have a setting for this!

We actually addressed this issue (Labeled with future-release) in our internal version and were ready to release it next week, however we have to suddenly take down the extension due to reasons outlined in the issue #239 and our readme. I'll make sure to archive these feature requests and provide to whomever continues the journey as a reference. Our sincere apologies 🙏


HTTP ERROR 404

Hi,
I tried to debug/run this project using VS 2022 and VS Code and I got the following error in Chrome:
This localhost page can’t be found No webpage was found for the web address: https://localhost:5001/
HTTP ERROR 404

Any help will be much appreciated.
TIA

Same issue here.  Too bad this project still shows up on https://codebase.show/projects/realworld


Fails trying to add Code Generation support to Framework 4.6.2 project

That's Framework 4.6.2 / .NET Standard that should be inter-operable, correct? Doesn't work for me:
Severity	Code	Description	Project	File	Line	Suppression State
Error		Some NuGet packages were installed using a target framework different from the current target framework and may need to be reinstalled. Visit http://docs.nuget.org/docs/workflows/reinstalling-packages for more information.  Packages affected: System.AppContext, System.Linq, System.Linq.Expressions, System.Runtime.InteropServices, System.Security.Cryptography.Algorithms	Kingdom.Collections.Enumerations.Analyzers		0

I want to add code generation. I think that a Compile time only Attribute is probably best to signal that this should occur. TBD where that attribute should live, but I wonder about the line:
[CodeGenerationAttribute(typeof(DuplicateWithSuffixGenerator))]

Doesn't that sort of imply that the Attribute has a priori knowledge of the generator? In my case, I cannot guarantee users are on .NET Standard / Core yet, so what's the alternative? Seems like a potentially circular Assembly dependency in the making.

Should also clarify, as I've done in other places, I am spooling up the project from a VS2015 project, Framework 4.6.2 (or greater) target, to begin with.

Trying to walk the version back to a dependency that is at least compatible with Framework 4.6.2, and the first attempt 0.4.37 fails because:
Severity	Code	Description	Project	File	Line	Suppression State
Error		Unable to find a version of 'CodeGeneration.Roslyn.Attributes' that is compatible with 'CodeGeneration.Roslyn 0.4.37 constraint: CodeGeneration.Roslyn.Attributes (>= 1.0.0)'.			0

Your dependencies are broken in the NuGet package.

The most recent one that seems to work, or at least NuGet is satisfied resolving the dependencies, is 0.4.11.
That being said, what is the migration path moving forward? i.e. supporting Framewotk 4.6.2 or better, etc?

First of all, the NuGet versioning error was solved with v0.4.42.
Second of all, I'm not 100% sure, but I believe that this project won't work with VS version < 2017, like 2015 in your case.
And also, I think but again am not sure, it requires that you use a new SDK-style csproj format. The newest Visual Studio 2017 version 15.7 has a built in migration tool.

Thanks for the prompt feedback. I'd say that's pretty much verified with the initial exceptions I received. I was wondering about that, actually. I haven't pursued 2017 migration yet, but it is something for me to chew on in the near term. Thanks again.

It's sadly true that the current codegeneration-roslyn project only allows code generation for SDK style projects. The framework they target is irrelevant and can be anything.

Good to know. Thanks for the awareness on that front.


Camel Paho 5 integrates EMQX JWKS

Bug description
My routed defined below
@Override
    public void configure() throws UnknownHostException {

        final PahoMqtt5EndpointBuilderFactory.PahoMqtt5EndpointBuilder endpoint = pahoMqtt5("test/1/2")
                .brokerUrl(brokerUrl)
                .sslClientProps(properties)
                .clientId(InetAddress.getLocalHost().getHostName())
                .userName("123456");

        log.infof("endpoint: %s", endpoint.getRawUri());

        from(direct("sendMqtt"))
                .marshal().json(JsonLibrary.Jsonb)
                .log("${body}")
                .to(endpoint);
    }

I defined userName 123456, but the EMQX server message shows username as undefined.
EMQX Server message:
...
2023-07-20T10:59:40.420785+08:00 [debug] msg: mqtt_packet_received, mfa: emqx_channel:handle_in/2, line: 338, peername: <IP_ADDRESS>:40048, clientid: camel-paho-a2f84a1f-14a5-47f2-b4bd-c32a8354f012, packet: CONNECT(Q0, R0, D0, ClientId=camel-paho-a2f84a1f-14a5-47f2-b4bd-c32a8354f012, ProtoName=MQTT, ProtoVsn=5, CleanStart=true, KeepAlive=60, Username=undefined, Password=), tag: MQTT
...

The EMQX JEKS authentication field is defined as username.
It caused paho5 to have an authentication failure message.
Caused by: org.apache.camel.RuntimeCamelException: Not authorized. (135)

Are you running in native mode or in the standard JVM mode?

Looking at the component code, seems you must also provide a password for the authentication bits to get set correctly:
https://github.com/apache/camel/blob/main/components/camel-paho-mqtt5/src/main/java/org/apache/camel/component/paho/mqtt5/PahoMqtt5Endpoint.java#L88

I have tried to set password, but result is same.
 final PahoMqtt5EndpointBuilderFactory.PahoMqtt5EndpointBuilder endpoint = pahoMqtt5("test/1/2")
                .brokerUrl(brokerUrl)
                .sslClientProps(properties)
                .clientId(InetAddress.getLocalHost().getHostName())
                .userName("123456").password("");

I have tried MqttClient functions and it was successful.
public MqttClient mqttClient() throws UnknownHostException {
        MqttClient mqttClient;
        try {
            mqttClient = new MqttClient(this.brokerUrl, InetAddress.getLocalHost().getHostName());
            final var mqttConnectionOptions = new MqttConnectionOptions();
            mqttConnectionOptions.setAutomaticReconnect(true);
            if (properties.size() > 0) {
                mqttConnectionOptions.setSSLProperties(this.properties);
            }
            if (username.isPresent()) {
                mqttConnectionOptions.setUserName(this.username.get());
            }
            mqttClient.connect(mqttConnectionOptions);
        } catch (MqttException e) {
            throw new RuntimeException(e);
        }

        return mqttClient;
    }

Camel Route
from(direct("sendMqtt"))
                .marshal().json(JsonLibrary.Jsonb)
                .log(LoggingLevel.INFO, "Camel Paho Publisher Topic: ${headers.CamelPahoMqtt5OverrideTopic}")
                .to(pahoMqtt5("test/1")
                        .advanced()
                        .client(this.mqttClient()));

I have tried to set password

The problem is that it's empty.

I have tried to set password

The problem is that it's empty. You need to pass a valid value.
The EMQX JWKS token verification source is to choose one from username and password.

# EMQX jwks setting
authn-jwt:jwks.from

Type: enum

Default: password

Optional: username | password

Field to take JWT from.

One way to work around it could be to pass your own client instance to the component:
https://camel.apache.org/components/3.20.x/paho-mqtt5-component.html#_component_option_client
That way you're in charge of the authentication setup.

One way to work around it could be to pass your own client instance to the component:
https://camel.apache.org/components/3.20.x/paho-mqtt5-component.html#_component_option_client
That way you're in charge of the authentication setup.

The component_option_client can work for me.
Can be enhance username description or support single option for authentication in the future ?
Thanks.

Can be enhance username description or support single option for authentication in the future ?

https://issues.apache.org/jira/browse/CAMEL-19627

Thanks for your help.  👍

Fixed in https://github.com/apache/camel-quarkus/pull/5119.


README.Debian says to source /usr/share/doc/fzf/examples/completion.bash, but file no longer exists

[x ] I have read through the manual page (man fzf)
[ x] I have the latest version of fzf
[ x] I have searched through the existing issues

Info

OS

[x ] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
README.Debian says to source /usr/share/doc/fzf/examples/completion.bash, but file no longer exists.
$ cat /usr/share/doc/fzf/README.Debian
Bash
====

Append this line to ~/.bashrc to enable fzf keybindings for Bash:

   source /usr/share/doc/fzf/examples/key-bindings.bash

Note, since fzf 0.29.0-1, the bash completion is installed for bash
by default. Feel free to ignore the following instruction for fzf >= 0.29.0-1.
Append this line to ~/.bashrc to enable fuzzy auto-completion for Bash:

   source /usr/share/doc/fzf/examples/completion.bash

...
9$ ls /usr/share/doc/fzf/examples/completion.bash
ls: cannot access '/usr/share/doc/fzf/examples/completion.bash': No such file or directory

I have the same bug. I paste more info:
bash: /usr/share/doc/fzf/examples/completion.bash: No such file or directory
miguel@speccy-laptop:~$

$ dpkg -l | grep fzf
ii  fzf                                    0.30.0-1+b1                        amd64        general-purpose command-line fuzzy finder

$ lsb_release --all
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux bookworm/sid
Release:	testing
Codename:	bookworm

$ uname -a
Linux speccy-laptop 5.17.0-1-amd64 #1 SMP PREEMPT Debian 5.17.3-1 (2022-04-18) x86_64 GNU/Linux

$ dpkg -L fzf
/.
/usr
/usr/bin
/usr/bin/fzf
/usr/bin/fzf-tmux
/usr/share
/usr/share/bash-completion
/usr/share/bash-completion/completions
/usr/share/bash-completion/completions/fzf
/usr/share/doc
/usr/share/doc/fzf
/usr/share/doc/fzf/README.Debian
/usr/share/doc/fzf/changelog.Debian.amd64.gz
/usr/share/doc/fzf/changelog.Debian.gz
/usr/share/doc/fzf/changelog.gz
/usr/share/doc/fzf/copyright
/usr/share/doc/fzf/examples
/usr/share/doc/fzf/examples/completion.zsh
/usr/share/doc/fzf/examples/fzf.vim
/usr/share/doc/fzf/examples/key-bindings.bash
/usr/share/doc/fzf/examples/key-bindings.fish
/usr/share/doc/fzf/examples/key-bindings.zsh
/usr/share/doc/fzf/examples/plugin
/usr/share/fish
/usr/share/fish/vendor_functions.d
/usr/share/fish/vendor_functions.d/fzf_key_bindings.fish
/usr/share/man
/usr/share/man/man1
/usr/share/man/man1/fzf-tmux.1.gz
/usr/share/man/man1/fzf.1.gz
/usr/share/doc/fzf/examples/plugin/fzf.vim

Yesterday its okay, but today i see the same problem. This because files in example directory its does not exist, and only have one directory plugin.
# tree /usr/share/doc/fzf/examples
/usr/share/doc/fzf/examples
└── plugin
    └── fzf.vim -> ../fzf.vim

# lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 20.04 LTS
Release:        20.04
Codename:       focal

And now i temporarily disable fzf plugins in my .zshrc because the message is very disturbing.

I'm not the maintainer of the Debian package. You should report the problem to the Debian package maintainers.

I think this was related 🤔
#2480 https://bugs.launchpad.net/ubuntu/+source/fzf/+bug/1928670

Isn't the solution right in the quoted text in the README.Debian specifically this part:
Note, since fzf 0.29.0-1, the bash completion is installed for bash
by default. Feel free to ignore the following instruction for fzf >= 0.29.0-1.

Running $ dpkg -L fzf shows that the completions are installed directly at: /usr/share/bash-completion/completions/fzf.
My fzf version:
ii  fzf            0.29.0-1     amd64        general-purpose command-line fuzzy finder

The Debian documentation in /usr/share/doc/fzf/README.Debian for version 0.30.0 is invalid : file /usr/share/doc/fzf/examples/completion.bash no longer exists in the package. In place, FZF completions is installed by default with /usr/share/bash-completion/completions/fzf => no need to source it in $HOME/.bashrc.
The issue is not a upstream one but should be reported to the Debian package maintainers https://bugs.debian.org/cgi-bin/pkgreport.cgi?repeatmerged=no&src=fzf

I just installed it in pop OS! and the completion file in in the completions dir, but it doesn't work.

Am I missing something?

I use Debian, this is currently what is in my .bashrc to work on Debian, but also other distros:
  if [ -f /usr/share/bash-completion/completions/fzf ]; then
    source /usr/share/bash-completion/completions/fzf
  fi

  if [ -f /usr/share/doc/fzf/examples/key-bindings.bash ]; then
    source /usr/share/doc/fzf/examples/key-bindings.bash
  fi

@Diablo-D3 This works for me at Ubuntu 22.04 too. Thanks!

I think this was related 🤔
#2480 https://bugs.launchpad.net/ubuntu/+source/fzf/+bug/1928670

I cross-referenced this issue into launchpad #2480 above

You can also use a shortand condition like:

[ -f /usr/share/doc/fzf/examples/key-bindings.bash ] && source /usr/share/doc/fzf/examples/key-bindings.bash
[ -f /usr/share/doc/fzf/examples/completion.bash ] && source /usr/share/doc/fzf/examples/completion.bash


[feature] MDLParser

This is the primary upgrade with v1.0.0 and will provide significantly shorter accel build times, along with adding support for face flexes and skins

Initial version of MDLParser is written, it just needs to be integrated into vistrace