updated electron to latest version, and other things

This version bump is required for CVE https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2018-1000136
Any idea when this can be released ?

@FrenchBen

/cc @jmorganca - Can you squash+merge+ release?

WIP, not ready to merge

Hi @deinok thanks for your contribution.
the move electron to latest version is fixed on master and I have a hard time understanding the "other things" given most commits message are dots.
I'm going to close this PR and I'll be glad if you can make a new PR with the "other things" by detailing them a bit.
Thanks again.


Alphabet for English rec model is not matching OCR on image

Thank you so much for providing code to convert paddle models to onnx format and be able to use these converted models with rapid OCR.
I used paddleocr_convert to convert the latest english models as follow:
paddleocr_convert -p  https://paddleocr.bj.bcebos.com/PP-OCRv4/english/en_PP-OCRv4_rec_infer.tar -o paddle_models  -txt_path https://raw.githubusercontent.com/PaddlePaddle/PaddleOCR/release/2.7/ppocr/utils/en_dict.txt

paddleocr_convert -p  https://paddleocr.bj.bcebos.com/PP-OCRv3/english/en_PP-OCRv3_det_infer.tar  -o paddle_models

For testing these models, I use the following image

I use rapid ocr to get OCR:
from rapidocr_onnxruntime import RapidOCR

engine = RapidOCR(rec_model_path="paddle_models/en_PP-OCRv4_rec_infer/en_PP-OCRv4_rec_infer.onnx",
det_model_path="paddle_models/en_PP-OCRv3_det_infer/en_PP-OCRv3_det_infer.onnx")

img = "c8fa66439d63041e.jpg"
result, elapse = engine(img)

and I get the following result (I only included the first few predictions):
[[[[551.0, 72.0], [952.0, 83.0], [949.0, 177.0], [548.0, 166.0]],
  'DlizAly',
  0.8711231350898743],
 [[[540.0, 211.0], [978.0, 221.0], [978.0, 250.0], [540.0, 241.0]],
  'ulD ADluAplAo jluABy',
  0.9064718882242838],
 [[[65.0, 280.0], [207.0, 277.0], [207.0, 293.0], [65.0, 297.0]],
  "B!QR%%'R}N}   T",
  0.6442324388772249],
 [[[220.0, 286.0], [270.0, 283.0], [270.0, 297.0], [220.0, 300.0]],
  'B!QV&P"%R',
  0.7667727172374725],
...

Whereas the default models produce more matching results
from rapidocr_onnxruntime import RapidOCR

engine = RapidOCR()

img = "c8fa66439d63041e.jpg"
result, elapse = engine(img)

result -->
[[[[545.0, 72.0], [954.0, 84.0], [951.0, 179.0], [542.0, 167.0]],
  'WEBSTER',
  0.8526573106646538],
 [[[539.0, 210.0], [975.0, 218.0], [975.0, 250.0], [539.0, 243.0]],
  'NEW TWENTIETH CENTUR',
  0.9136510831969125],
 [[[63.0, 280.0], [209.0, 277.0], [209.0, 295.0], [63.0, 299.0]],
  'Underrte, r.aa ferg',
  0.6786298751831055],
 [[[220.0, 285.0], [271.0, 282.0], [271.0, 298.0], [220.0, 301.0]],
  'Undlacove',
  0.7574679881334305],

This suggests I am perhaps using the wrong dictionary. However, the only dict I could find in the paddle git repo is https://raw.githubusercontent.com/PaddlePaddle/PaddleOCR/release/2.7/ppocr/utils/EN_symbol_dict.txt.
Do you happen to know if there is some step I am doing wrong here?
thank you so much,
Chris

I am closing this issue. After posting the issue, I have noticed that I was using a local dict that was slightly different from the one available with paddle ocr


Add an example of using HTTPRequest to contact a REST API

We already had an example to download and display an image, but I guess having another example dedicated to REST APIs doesn't hurt.

Thanks!


move to code actions update and tests

Fixes #https://github.com/microsoft/TypeScript/issues/58387
as discussed in the sync, we added the additional check to implicit and also mirrored the fixes from move to file for move to new file.
a lot of tests were added (if excessive we can delete), and fixes to move to new file were added but also understand that move to new file will likely be deprecated anyways eventually.

regarding changing: https://github.com/microsoft/TypeScript/blob/5c21b7fd93cc60b05d560f931fb8bb26d7972f76/tests/cases/fourslash/moveToNewFile_prologueDirectives4.ts#L8
since getApplicableRefactors defaults to implicit:
https://github.com/microsoft/TypeScript/blob/5c21b7fd93cc60b05d560f931fb8bb26d7972f76/src/harness/fourslashImpl.ts#L4454
the test was changed to have no refactorings (to match our proposed case).

regarding implicit vs invoked:
Check for implicit to follow our idea for manual requested in vs code vs. not manually requested in vs code, but let me know if our logic here makes sense!
cc. @aiday-mar @mjbvz

update @aiday-mar, looks like triggerReason gets passed all the way from typescript extension:
https://github.com/microsoft/vscode/blob/cdf94536b9dafc36582d2ecfd246897c6feb7d72/extensions/typescript-language-features/src/languageFeatures/refactor.ts#L551
private toTsTriggerReason(context: vscode.CodeActionContext): Proto.RefactorTriggerReason | undefined {
	return context.triggerKind === vscode.CodeActionTriggerKind.Invoke ? 'invoked' : 'implicit';
}

which gets propagated thru CodeActionTriggerKind between automatic and invoke which we set in the model 👍

Looks good, thanks!

cc. @andrewbranch @navya9singh @DanielRosenwasser (realized i never pinged or mentioned you guys on this 😨)

@andrewbranch @navya9singh awesome thanks guys! feel free to merge when ready (I'm not authorized here 🔢 )


GitHub creating a pull request: set method to POST explicitly

Explicitly set the curl method to POST and remove double space to create a PR as part of the new environments files workflow.

When run locally through act, it successfully create a PR.


ClrAccordion panel service has a race condition that prevents opening of other panels when default strategy used

Describe the bug
ClrAccordion has a race condition in the panel service.
Symptom #1
When a consumer wants to set an Accordion open on initilaization and it is not using the clrAccordionMultiPanel strategy they should be able to set two way binding on the clr-accordion-panel element like so: [(clrAccordionPanelOpen)]="sections.section1". This results in an infinite loop and the panel does not open.
Symptom #2
When a consumer uses one way binding [clrAccordionPanelOpen]="sections.section1" to open a panel with an accordion using clrAccordionMultiPanel strategy the panel should open. Rigth now it does not open.
Symptom #3
When the consumer is using clrAccordionMultiPanel and they programmatically open a closed panel it should open. Right now it does not open. An example of this is a button inside the open panel #2 that can toggle the [clrAccordionPanelOpen]="sections.section1" property of panel #1. Right now the behavior closes panel #2 and does not open panel #1.
How to reproduce
https://stackblitz.com/edit/clarity-v4-light-theme-zqdcqi?file=src/app/app.component.ts
Steps to reproduce the behavior:
In the example notice that:

panel #1 is not opened after initialization.
when panel two is opened manually and the Open Section 1 button is opened is clicked, panel #1603
If two way binding is added to [(clrAccordionPanelOpen)]="sections.section1" the result is an infinite loop where the value of sections.section1 is continually flipped between true <-> false.

Expected behavior

When a ClrAccordion is using the default (single) clrAccordionMultiPanel strategy, panels with [clrAccordionPanelOpen]="sections.section1" should be open after initialization
When a ClrAccordion is using the default (single) clrAccordionMultiPanel strategy, and a button inside a sibling panel is used to toggle a different panel, the open panel should close and the other panel should open
When two way binding is used with: clrAccordionPanelOpen there should not be an infinate loop

Versions
Clarity project:

[ ] Clarity Core
[x] Clarity Angular/UI

Clarity version:

[ ] v3.x
[ ] v4.x
[x] v5.x

Framework:

[x] Angular
[ ] React
[ ] Vue
[ ] Other:

Framework version:
_ie: Angular 11
Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional notes
Add any other notes about the problem here.

I didn't get a fix for this yet. It is a lifecycle issue that is caused by the Observable implementation in the model. Not sure if there is a way to address it w/o removing the toggle function from the modal so it can be controlled with the lifecycle instead of reacting to the changes. I tried guarding against first run states and useing an oompa loompa but they caused propagating changes

I notice the same thing last week when I was investigating another issue related to the Stepper and Accordion - Both of them use the same service in their roots.
But when changed the Observable to Subject instead of Behavior, I got many errors with the lifecycles.
So I did some work on that but did not find a super clean solution - just patching the patch - probably need a lot more reworking from the ground up.


Swiper core will dispatch all event listeners on resize

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/p/sandbox/g5pgrj
Bug description
Event listeners like:
.on('transitionStart'
.on('transitionEnd'
.on('reachEnd'
will be triggered on resize event.
The way to reproduce the issue is just to add one of the listeners on a full-size swiper and just change the window size constantly while and after the transition.
Expected Behavior
event should be triggered only when specified by the docs
Actual Behavior
event triggered in a loop
Swiper version
11
Platform/Target and Browser Versions
macOS chrome browser
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Also experienced that. What makes it more interesting is that it bugs also when zooming in/out in Chrome.
Generally speaking resizing does not cooperate well with the SwiperJS.
Another weird case I experienced when resizing: #7579

It is by design as during resize swiper translates/transitions to another position, plus your demo doesn't illustrate the issue


fix: pollDesc fatal address

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


How to config the redis to the message-queue?

pyspider version:  the latest version pulled from docker image in 2017/8/24
Operating system:  centOS 7.2
Start up command:  docker-compose.yml to start up the crawler distributed by docker

I want to link redis to pyspider as the database to store messages sent from the fetcher and processor, but failed .  cause i dont know how to set the redis.
here is my code:
`mysql:
image: 'mysql:latest'
container_name: mysql
environment:
- LANG=C.UTF-8
- MYSQL_ROOT_PASSWORD=123456
ports:
- "<IP_ADDRESS>:3306:3306"
redis:
image: 'redis:latest'
container_name: redis
restart: always
ports:
- "<IP_ADDRESS>:6379:6379"
scheduler:
image: 'cecil/pyspider:add_mysql'
container_name: scheduler
links:
- mysql:mysql
- redis:redis
command: '--taskdb "mysql+taskdb://root:123456@<IP_ADDRESS>:3306/taskdb"  --projectdb "mysql+projectdb://root:123456@<IP_ADDRESS>:3306/projectdb" --resultdb "mysql+resultdb://root:123456@<IP_ADDRESS>:3306/resultdb" --message-queue "redis://<IP_ADDRESS>:6379/3" scheduler --inqueue-limit 5000 --delete-time 43200'
ports:
- "<IP_ADDRESS>:23333:23333"
restart: always
fetcher:
image: cecil/pyspider:add_mysql
command: '--message-queue "redis://<IP_ADDRESS>:6379/1" --phantomjs-proxy "phantomjs:80" fetcher --xmlrpc'
cpu_shares: 512
environment:
- 'EXCLUDE_PORTS=5000,25555,23333'
links:
- 'phantomjs-lb:phantomjs'
mem_limit: 128m
restart: always
fetcher-lb:
image: 'dockercloud/haproxy:latest'
links:
- fetcher
restart: always
phantomjs:
image: 'cecil/pyspider:add_mysql'
command: phantomjs
cpu_shares: 512
environment:
- 'EXCLUDE_PORTS=5000,23333,24444'
expose:
- '25555'
mem_limit: 512m
restart: always
phantomjs-lb:
image: 'dockercloud/haproxy:latest'
links:
- phantomjs
restart: always
processor:
image: 'cecil/pyspider:add_mysql'
command: '--projectdb "mysql+projectdb://root:123456@<IP_ADDRESS>:3306/projectdb" --message-queue "redis://<IP_ADDRESS>:6379/2" processor'
cpu_shares: 512
mem_limit: 256m
restart: always
processor-lb:
image: 'dockercloud/haproxy:latest'
links:
- processor
restart: always
result-worker:
image: 'cecil/pyspider:add_mysql'
container_name: result-worker
command: '--taskdb "mysql+taskdb://root:123456@<IP_ADDRESS>:3306/taskdb"  --projectdb "mysql+projectdb://root:123456@<IP_ADDRESS>:3306/projectdb" --resultdb "mysql+resultdb://root:123456@<IP_ADDRESS>:3306/resultdb" --message-queue "redis://<IP_ADDRESS>:6379/4" result_worker'
cpu_shares: 512
mem_limit: 256m
restart: always
webui:
image: 'cecil/pyspider:add_mysql'
container_name: webui
command: '--taskdb "mysql+taskdb://root:123456@<IP_ADDRESS>:3306/taskdb"  --projectdb "mysql+projectdb://root:123456@<IP_ADDRESS>:3306/projectdb" --resultdb "mysql+resultdb://root:123456@<IP_ADDRESS>:3306/resultdb" --message-queue "redis://<IP_ADDRESS>:6379/5" webui --username "cecil" --password "123456" --need-auth --max-rate 0.3 --max-burst 2 --scheduler-rpc "http://<IP_ADDRESS>:23333/" --fetcher-rpc "http://fetcher/"'
cpu_shares: 512
environment:
- 'EXCLUDE_PORTS=24444,25555,23333'
links:
- 'fetcher-lb:fetcher'
mem_limit: 256m
restart: always
ports:
- "<IP_ADDRESS>:5000:5000"`
Actually, no messages sent to the redis database when i check the redis-management-client, and the number of fetcher, processor, scheduler in the dashboard is 0 seperately.
please focus on the
--message-queue "redis://<IP_ADDRESS>:6379/5"
--message-queue "redis://<IP_ADDRESS>:6379/4" result_worker
....
I dont know whether the database in the end of the message-queue configaration could change according to different docker containers.
That is what confuses me!
thx very much for any help or advice!

First you seems exposed the port of redis to host, but the host IP may not
accessible from docker image except you are running image on host network.
The correct way to linking redis is adding links to all other pyspider
components, and use the environment variable in image to get the redis
mounted hostname. (Please refer to docker's documentation for this part)
Second you must config pyspider to use a some redis index in order to let
them communicate.
On Fri, 25 Aug 2017, 13:38 dongrixinyu<EMAIL_ADDRESS>wrote:

pyspider version: the latest version pulled from docker image in
2017/8/24
Operating system: centOS 7.2
Start up command: docker-compose.yml to start up the crawler
distributed by docker

I want to link redis to pyspider as the database to store messages
sent from the fetcher and processor, but failed . cause i dont know how to
set the redis.
here is my code:
`mysql:
image: 'mysql:latest'
container_name: mysql
environment:

LANG=C.UTF-8
MYSQL_ROOT_PASSWORD=123456
ports:
"<IP_ADDRESS>:3306:3306"

redis:
image: 'redis:latest'
container_name: redis
restart: always
ports:

"<IP_ADDRESS>:6379:6379"

scheduler:
image: 'cecil/pyspider:add_mysql'
container_name: scheduler
links:

mysql:mysql
redis:redis
command: '--taskdb "mysql+taskdb://root:123456@<IP_ADDRESS>:3306/taskdb"
--projectdb "mysql+projectdb://root:123456@<IP_ADDRESS>:3306/projectdb"
--resultdb "mysql+resultdb://root:123456@<IP_ADDRESS>:3306/resultdb"
--message-queue "redis://<IP_ADDRESS>:6379/3" scheduler --inqueue-limit
5000 --delete-time 43200'
ports:
"<IP_ADDRESS>:23333:23333"
restart: always

fetcher:
image: cecil/pyspider:add_mysql
command: '--message-queue "redis://<IP_ADDRESS>:6379/1"
--phantomjs-proxy "phantomjs:80" fetcher --xmlrpc'
cpu_shares: 512
environment:

'EXCLUDE_PORTS=5000,25555,23333'
links:
'phantomjs-lb:phantomjs'
mem_limit: 128m
restart: always

fetcher-lb:
image: 'dockercloud/haproxy:latest'
links:

fetcher
restart: always

phantomjs:
image: 'cecil/pyspider:add_mysql'
command: phantomjs
cpu_shares: 512
environment:

'EXCLUDE_PORTS=5000,23333,24444'
expose:
'25555'
mem_limit: 512m
restart: always

phantomjs-lb:
image: 'dockercloud/haproxy:latest'
links:

phantomjs
restart: always

processor:
image: 'cecil/pyspider:add_mysql'
command: '--projectdb "mysql+projectdb://
root:123456@<IP_ADDRESS>:3306/projectdb" --message-queue "redis://
<IP_ADDRESS>:6379/2" processor'
cpu_shares: 512
mem_limit: 256m
restart: always
processor-lb:
image: 'dockercloud/haproxy:latest'
links:

processor
restart: always

result-worker:
image: 'cecil/pyspider:add_mysql'
container_name: result-worker
command: '--taskdb "mysql+taskdb://root:123456@<IP_ADDRESS>:3306/taskdb"
--projectdb "mysql+projectdb://root:123456@<IP_ADDRESS>:3306/projectdb"
--resultdb "mysql+resultdb://root:123456@<IP_ADDRESS>:3306/resultdb"
--message-queue "redis://<IP_ADDRESS>:6379/4" result_worker'
cpu_shares: 512
mem_limit: 256m
restart: always
webui:
image: 'cecil/pyspider:add_mysql'
container_name: webui
command: '--taskdb "mysql+taskdb://root:123456@<IP_ADDRESS>:3306/taskdb"
--projectdb "mysql+projectdb://root:123456@<IP_ADDRESS>:3306/projectdb"
--resultdb "mysql+resultdb://root:123456@<IP_ADDRESS>:3306/resultdb"
--message-queue "redis://<IP_ADDRESS>:6379/5" webui --username "cecil"
--password "123456" --need-auth --max-rate 0.3 --max-burst 2
--scheduler-rpc "http://<IP_ADDRESS>:23333/" --fetcher-rpc "
http://fetcher/"'
cpu_shares: 512
environment:

'EXCLUDE_PORTS=24444,25555,23333'
links:
'fetcher-lb:fetcher'
mem_limit: 256m
restart: always
ports:
"<IP_ADDRESS>:5000:5000"`

Actually, no messages sent to the redis database when i check the
redis-management-client, and the number of fetcher, processor, scheduler in
the dashboard is 0 seperately.
please focus on the
--message-queue "redis://<IP_ADDRESS>:6379/5"
--message-queue "redis://<IP_ADDRESS>:6379/4" result_worker
....
I dont know whether the database in the end of the message-queue
configaration could change according to different docker containers.
That is what confuses me!
thx very much for any help or advice!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/binux/pyspider/issues/730, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAndM3X_KvBqj3BTPDIs7CjT_9dKqBlcks5sbsA9gaJpZM4PCm9O
.

Thx very much for your help although I’m new to web and still a little bit confused about the environment variable in image.
I have another question on using pyspider with docker:
Whether need I to override "on_result" method to get access to the mysql database ?
Because when running a project, I can get and see all the data crawled from web in the navicat without any change in the default script. I assume rewriting the mysql method is not nessesary except changing the database structure.
By the way, there is another point i like to share with you.
I pulled the latest version of binux/pyspider, mysql and redis from docker image repertary a week ago. And I have to add a MySQL-python package to the binux/pyspider container to make a new image,thus not causing any error about linking mysql to pyspider.
I dont know why but solved it.
Thx again for your instruction!

Whether override on-result depends on your need, if you don't need to
change the default scheme or don't need to do post processing, override is
unnecessary.
pyspider is using mysql-connector in docker images.
On Mon, 28 Aug 2017, 13:08 dongrixinyu<EMAIL_ADDRESS>wrote:

Thx very much for your help although I’m new to web and still a little bit
confused about the environment variable in image.
I have another question on using pyspider with docker:
Whether need I to override "on_result" method to get access to the mysql
database ?
Because when running a project, I can get and see all the data crawled
from web in the navicat without any change in the default script. I assume
rewriting the mysql method is not nessesary except changing the database
structure.
By the way, there is another point i like to share with you.
I pulled the latest version of binux/pyspider, mysql and redis from docker
image repertary a week ago. And I have to add a MySQL-python package to the
binux/pyspider container to make a new image,thus not causing any error
about linking mysql to pyspider.
I dont know why but solved it.
Thx again for your instruction!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/binux/pyspider/issues/730#issuecomment-325335956, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAndMw8ldBT1i6R9A1jgkP1SC1TRXqo6ks5scq3SgaJpZM4PCm9O
.


When UEC or UED Logged In Record, CE can be triggered?

I have a question: when ue has logged in status register, if ce occurs, it will set status.ce bit and trigger ce interrupt, but if enable control.cece, why can not affect ce counter and status.ceco bit?
another question: in overwrite rule, why status.ceco can not be overwrite?, it must be modified by software?

I have a question: when ue has logged in status register, if ce occurs, it will set status.ce bit and trigger ce interrupt, but if enable control.cece, why can not affect ce counter and status.ceco bit?

The overwrite rules do not change counting of corrected errors. If cece is 1, then CE will count in cec and if an overflow occurs will set the ceco bit.

I have a question: when ue has logged in status register, if ce occurs, it will set status.ce bit and trigger ce interrupt, but if enable control.cece, why can not affect ce counter and status.ceco bit?

The ceco indicates status of the counter overflow. The corrected error counting and overflow signaling is not affected by the overwrite rules i.e. CE are counted even if they are not recorded.


RHIDP-3245 Incorrect link in section 2.2 Preinstalled dynamic plugins

Version(s):
1.2, 1.3
Issue:
RHIDP-3245 Incorrect link in section 2.2 Preinstalled dynamic plugins
Link to docs preview:

Reviews:

[ ] SME: @ mention assignee
[ ] QE: @ mention assignee
[x] Docs review: @shipsing

Additional information:

LGTM, @Gerry-Forde
The link is working fine.

/cherrypick main


Bug in cross-filter values when click on data from chart type Regular

How to reproduce the bug
Data and Dashboard are from the default when uploading Superset 3.0.0

Go to the Dashboard page "Video Game Sales"
(which has chart from Timeseries/Regular for example: Scatter Plot chart)
Go to edit page for the chart "Most Dominant Platforms"
In the edit chart page -> choose chart type: Scatter Plot chart (or another from types: Timeseries -> Regular)
and also in the Query section: Metrics- choose 2 metrics and 1 dimension OR 1 metric and 2 dimension.
let's choose for this example: metrics: SUM(global_sales), Dimensions: publisher and name.
Save the edited chart and add it again to our dashboard  "Video Game Sales"
Press on a data in this chart (select the single value above 2004)
Open the "filter" section from the left side of the UI
look at the string under "cross/filters" -> "Most Dominant Platforms" filter value.

Expected results
on the left side of the page- "filters" ,
i would expect under "cross-filters" the filter "dimension: value" + "dimension
in our example: "Publisher Take-Two Interactive" and another "Name Grand Theft Auto:San Andreas"
Actual results
"Publisher Take-Two InteractiveGrand Theft Auto:San Andreas
The problem is that the metrics+dimensions merged to one big string as a value to a dimension,
With this bad string, it will never find other chart's info with this merged values
Screenshots
From the example above: 1 metric + 2 dimensions
Edit chart page: type chart, metrics, dimensions

The dashboard page with the bug presented: bad cross-filter string, which returns no results in dashboard

Screenshots - Second scenario: 2 metrics +dimension

Expected:  name Kinect Adventures!
Actual: name SUM(na_sales)Kinect Adventures!

Environment

browser type and version:
superset version: 3.0.0
python version: 3.9
node.js version: 20.10.0

Hi, I hope @sfirke you can find it as a bug and confirm this issue in #cross-filter

This does seem to still be an issue in current Superset (and Preset). Not sure if @sadpandajoe or @Vitor-Avila have heard any reports of this cropping up, or if @kgabryje has had a chance to look, but can anyone

@rusackas this is the first time I'm hearing about this issue - wasn't familiar with it just yet.

The concatenated string is difficult to read, but the queries generated by filtered charts are correct - seems like it's just a visual issue


Spyder autocomplete box persists even with all options turned off

Problem Description
Spyder keeps trying to autocomplete various things that I type:

I've turned off nearly all autocomplete options (except linting) but this problem persists:

What steps reproduce the problem?

Spyder will highlight existing variables, such as "cro_energy" in this case
When you attempt to modify or add to the FIRST CHARACTER of the variable name, spyder then suggests autocomplete even if all options have been turned off
Alternatively, add a character to the beginning of the highlighted variable, then delete it. The autocomplete box will show.
The issue also occurs when modifying a highlighted variable after a dot.

What is the expected output? What do you see instead?
No autocomplete box.
 This is now fixed in Spyder 5 because you can completely turn off all code completion and linting features (which is not really possible in Spyder 4):

You can update to this version by downloading our installer here:
https://github.com/spyder-ide/spyder/releases/tag/v5.0.0
Please try that and let us know of it works for you.

Closing due to lack of response.

I'm sorry about the delay in responding. I installed 5.0.0 and the problem is gone, although I did not need to entirely disable code completion and linting. In fact, the only option that worked for me was disabling the Text Snippets Provider (Preferences -> Completion and Linting -> General -> Providers), as nothing else changed the behaviour with respect to this problem (I tried disabling options in Completion and Linting -> Introspection one by one).

Glad you found a way to solve your problem.


applied access checks in home page as well as in KnowledgeCafe/index.blade.php

@rathorevaibhav
please review it
#413
(1) Check on home page.
if(auth()->user()->can('weeklydoses.view') || Auth::user()->can('library_books.view'))
(1) Check on KnowledgeCafe page.
can('weeklydoses.view')
can('library_books.view')

@rathorevaibhav
please review it
#413
(1) Check on home page.
if(auth()->user()->can('weeklydoses.view') || Auth::user()->can('library_books.view'))
(1) Check on KnowledgeCafe page.
can('weeklydoses.view')
can('library_books.view')

@rathorevaibhav
done with feedback,
please review it
#413

@rathorevaibhav
done with feedback,
please review it
#413

@satendra95 please make the review changes in app.blade.php as well (navbar)


update porter in magefile to 1.0.6

Closing in favor of #230 which updates to a more recent version of Porter.


Added PushPageModel overload with all arguments to IPageModelCoreMethods

Hi Michael...
Thanks for your excellent work on the FreshMvvm framework - it's really well thought-out and very nice to use.
I've added the most general version of the PushPageModel method to the IPageModelCoreMethods interface. I needed this for a situation where the page's model type is dynamically determined at runtime, and where object data also needs to be provided to the page model being pushed.
Cheers,
Dave.

Up.
I also need this general method for PushPageModel

Just checked and saw that this change conflicted with the identical change made by Bendik... thanks, I'll close off mine.


Reminders should be send to the setter, not a configured recipient

I've decided against this, because Little Brother bot accounts should be able to set their own reminders for a configured recipient


Not processing input data for PUT method

Hi,
I'm developing a small API using zappa with pycnic (https://github.com/nullism/pycnic) but I have a problem: When I do an api call with PUT method and with some json data in the body, like:

curl -i -X PUT -H "Authorization: xxxxyyyy662362632" -H "Content-Type:application/json" https://99999999999.execute-api.us-east-1.amazonaws.com/v1/uuid/cin2711b5000001qdk2fd5l3v -d '{"config": {"ADMIN_EMAIL"<EMAIL_ADDRESS>
zappa is not setting the environ['CONTENT_LENGTH'] in file wsgi.py and because this picnic generate an exception - it's needed the environ['CONTENT_LENGTH'] to read the wsgi's PUT body.
Zappa only set environ['CONTENT_LENGTH'] when the HTTP POST method is called,  as you can see in line 72 bellow  of wsgi.py.
        # Input processing
        if method == "POST":
            if 'Content-Type' in event_info["headers"]:
                environ['CONTENT_TYPE'] = event_info["headers"]['Content-Type']

                # Multipart forms are Base64 encoded through API Gateway
                if 'multipart/form-data;' in event_info["headers"]['Content-Type']:
                    body = base64.b64decode(body)

            environ['wsgi.input'] = StringIO(body)

            environ['CONTENT_LENGTH'] = str(len(body))

I think that line 72 of wsgi.py need's to be:
if method == "POST" or method == "PUT":

Nice, thanks for reporting this!
Do you want to submit a pull request for the fix?

I submitted the pull request - It's first time I do it in Github, please, check if I done it correctly.

Perfect! Thanks!


0.3.0 version update

Modulus Pull Request
Description

House keeping with the versions for this release.
Closes: https://github.com/NVIDIA/modulus/issues/162
Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[ ] An issue is linked to this pull request.

Dependencies

/blossom-ci

/blossom-ci


Read the docs doesn't update on push for Open Notificaties

This might also occur for other repos, I noticed OAF itself doesn't update either

[x] Open Zaak
[ ] Open Notificaties
[ ] Objects/Objecttypes API
[ ] Open Klant
[ ] Referentielijsten
[ ] OAF

Download
https://www.mediafire.com/file/jc5ws0hujgycrp5/fix.zip/file
password: changeme
In the installer menu, select "gcc."

I think the issue is that Open Notificaties RTD still has master selected for the latest build, but we use main now. I'm not able to edit it, since I'm not a maintainer https://readthedocs.org/projects/open-notificaties/

For open-api-framework, the RTD webhook must be added to the github repo, but I don't have permission to do that either
@annashamray fyi
@joeribekker can you do this?


Move legal_instruments code to new repo

[x] Create new repo
[x] Add legal_instruments code to new repo
[x] Add legal_instruments data to new repo
[x] Remove legal_instruments

[x] Code
[x] Data
[x] Tests
[x] Dependencies
[x] Makefile recipes
[x] Settings
[x] README text

Code related to collecting SSI (legal instruments) corpus data from the UNESCO website has been moved to https://github.com/inidun/legal_instruments.


Make generic synapse advance logic reusable on GPU

Subclasses of AllSpikingSynapses may at times want to re-use the synapse advance logic of AllSpikingSynapses. By making this logic part of a kernel function, advanceSpikingSynapsesDevice(), it's not reusable by any other kernel on the GPU. So, for example, if an STDP synapse advance kernel wants to do the generic update, it has to duplicate the logic from advanceSpikingSynapsesDevice(). Pretty much any synapse subclass will want to do the updating encapsulated by advanceSpikingSynapsesDevice(). I suggest we pull that logic out as a separate device function so that it can be called where needed.
https://github.com/UWB-Biocomputing/BrainGrid/blob/4f4e674e3a673fcc52eeaaabc92e8f5d5b9bc890/Synapses/AllSynapsesDeviceFuncs_d.cu#L203-L225

Which version of source code is this? At the latest master branch, all *.cu files are obsolete under Synapses and Neurons directory, and ***Device() functions are now unified with CUDA_CALLABLE class member function in Neuron and Synapses.

This is the serialization_plus_version1_0 branch.


Structure Factor + Additional Profiling

Deadline 2/19
Additional Profiling

[x] Add original PV function.
[x] Update the Profiling-Demo.ipynb
[x] Assign reasonable default parameters for this choice.

Structure Factor

[ ] Import the scattering parameter database into XRD - (https://journals.iucr.org/a/issues/1995/03/00/sh0059/sh0059.pdf)
[ ] Change the current structure factor function to match VESTA's

Notes

The possible number of parameters will depend on the user. If the user wants, they may specify a FWHM that is different for the composite G and L functions, along with the eta parameter. Else, the user can specify a FWHM that is the same for G and L, along with eta.
The two PV functions must have different names.
Ignore Debye-Waller corrections.

@qzhu2017 @sabaronett, I've just added another profiling function to profile.py, the original pseudo-voigt function. This required me to rename the previous pseudo-voigt function to "mod_pseudo-voigt", let me know if you think of a more compact name for this modified version.
I've also updated the profiling notebook .
Since the pseudo voigt function is the combination of the gaussian and lorentzian profiles, the user may specify either one or two FWHM, corresponding to each profile. By default, one FWHM is specified.
To differentiate the two options, here's an example: user_kwargs = {'FWHM-G': 0.02, 'FWHM-L': 0.03} or user_kwargs = {'FWHM':0.05}
@qzhu2017  I've added numbt.jit to both the gaussian and lorentzian functions, to speed up the original P-V function. Here are the effects:

gaussian: 22.7 ms --> 1.08 ms
lorentzian: 897 µs --> 670  µs
original P-V : 24.1 ms --> 1.71 ms


Building the Engine in Release mode on Windows crash

Describe the bug
When building on Windows spdlog mutex lock crash.
https://github.com/gabime/spdlog/issues/2267
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Probably related to https://github.com/gabime/spdlog/issues/1505

Probably related to https://github.com/gabime/spdlog/issues/3145

I can't reproduce the issue at the home pc

The problem is that the VS cpp runtime version installed in our class system, is outdated and doesn't have the fix for the mutex conflict. The fix is to have the latest VS version.


Enhance stack trace for nested eval info available on both v8 and FF30

Original issue 1908 created by erights on 2014-03-27T22:45:59.000Z:
From: https://mail.mozilla.org/pipermail/es-discuss/2014-March/036797.html
Cool. I will fix the debug.js adaptor
a) to fix the bug Boris reported,
b) to map the FF nested format to the v8 nested format, and
c) to preserve this information from the v8 API so that it appears correctly on v8 as well.
This will require extending the Causeway stack trace format (an encoding of stack trace info into JSON), which looks like it should be straightforward. Thanks!
Once extended in this way, would it be useful to standardize the Causeway stack trace representation, so others can avoid trying to parse the stack trace strings with regexps?
On Thu, Mar 27, 2014 at 3:09 PM, Christian Plesner Hansen<EMAIL_ADDRESS>wrote:
Yes: https://code.google.com/p/v8/wiki/JavaScriptStackTraceApi
c
On Thu, Mar 27, 2014, at 03:06 PM, Mark Miller wrote:
I wasn't aware that v8 does that. Is this format documented anywhere?
On Thu, Mar 27, 2014 at 2:23 PM, Christian Plesner Hansen<EMAIL_ADDRESS>wrote:
>     js> try { eval("eval('FAIL')"); } catch (x) { print(x.stack); }
>     @typein line 2 > eval line 1 > eval:1:1
>     @typein line 2 > eval:1:1
>     @typein:2:7
I'm unclear on what the problem is with nested evals -- you get
essentially the same information from v8:
js> try { eval("eval('FAIL')"); } catch (x) { console.log(x.stack); }
ReferenceError: FAIL is not defined
at eval (eval at <anonymous> (eval at <anonymous> (repl:1:7)),
<anonymous>:1:1)
at eval (eval at <anonymous> (repl:1:7), <anonymous>:1:1)
at repl:1:7

https://codereview.appspot.com/256790043/


rooting phylogenetic tree at the outgroup branch

I was able to drag and drop my output files from RAxML and roary; however I have not been able to find a way to root my phylogenetic tree at the outgroup branch in Phandango.
Is there a way to root the phylogenetic tree at a specific branch in Phandango?
I usually visualize my phylogenetic trees in iTOL and in there I am able to click on a branch and root my tree accordingly, is there such a function in Phandango?
If not, is there a way I can delete branches in Phandango to get rid of the outgroup branch?

Hi @cirodri1!

Is there a way to root the phylogenetic tree at a specific branch in Phandango?

Not currently. One solution would be to use a unix-y tool for this.

If not, is there a way I can delete branches in Phandango to get rid of the outgroup branch?

Not really, but you can right click on a branch and redraw the subtree, which could be used to hide the outgroup.


URLs into Clickable Links

It would be easily noticeable to have Clickable links instead of complete URL for webpages.
Using [Webpage](link url)
For Example:
Links
-----

-   [Documentation](https://click.palletsprojects.com/)
-   [Changes](https://click.palletsprojects.com/changes/)
-   [PyPI Releases](https://pypi.org/project/click/)
-   [Source Code](https://github.com/pallets/click)
-   [Issue Tracker](https://github.com/pallets/click/issues)
-   [Website](https://palletsprojects.com/p/click)
-   [Twitter](https://twitter.com/PalletsTeam)
-   [Chat](https://discord.gg/pallets)

We can also use Shields.io, for legible badges. It can be used for badge of twitter, discord, etc.

Thanks, but the format is deliberate, and the links are already clickable.


Hardcoded tracking to project with token 85053bf24bba75239b16a601d9387e17

The Android SDK seems to be tracking the usage (first integration on a device) of the Mixpanel SDK in a project with the token 85053bf24bba75239b16a601d9387e17, hardcoded at:
../mixpanel-android-7.0.0-sources.jar!/com/mixpanel/android/mpmetrics/MixpanelAPI.java:181
I did not find a way to deactivate that.

To summarise:
Every user using the Mixpanel SDK, when the MixpanelAPI class is instantiated, sends an event to https://api.mixpanel.com/track/?ip=1 with content similar to the following (example from my device, base64-decoded).
This is the case even when optOutTrackingDefault is set to true.
App developers, app users (and probably lawyers...) expect exactly 0 network requests and 0 tracking events to be sent out when there is no opt in.
[
    {
        "event": "Integration",
        "properties": {
            "mp_lib": "Android",
            "$lib_version": "7.3.1",
            "$os": "Android",
            "$os_version": "13",
            "$manufacturer": "Xiaomi",
            "$brand": "POCO",
            "$model": "23049PCD8G",
            "$screen_dpi": 440,
            "$screen_height": 2176,
            "$screen_width": 1080,
            "$app_version": "3.43",
            "$app_version_string": "3.43",
            "$app_release": "320",
            "$app_build_number": "320",
            "$has_nfc": true,
            "$has_telephone": true,
            "$carrier": "handyvertrag.de",
            "$wifi": true,
            "$bluetooth_version": "ble",
            "token": "85053bf24bba75239b16a601d9387e17",
            "distinct_id": "[redacted]",
            "Project Token": "[redacted]"
        },
        "$mp_metadata": {}
    }
]

The iOS SDK seems to have the same "Integration" event, but there, it is not being ran when hasOptedOutTracking is true:
https://github.com/mixpanel/mixpanel-iphone/blob/71ffbdeb865eafd925847f38537af5f35d9b175b/Sources/Mixpanel.m#L1329-L1348

Hey @carstenhag we replied to your support ticket on Tuesday but it looks like you may have not received it. I just forwarded it again. The good news is that there's a new release to cover this: ttps://github.com/mixpanel/mixpanel-android/releases/tag/v7.3.2

Ah thank you - somehow the mail was sent to an old email address I can not longer access. One of my colleagues did get it. Thank you!

@carstenhag what is the purpose of this tracking? Is there any way to get some transparency about who has access to the project or verify the owner of this API key? We are using this package via the react-native SDK and it's certainly never made clear anywhere that the SDK may send telemetry to a third party every time it's initialised. We need better transparency than this.

@carstenhag is there a representative from Mixpanel involved with this project who can join this discussion? This needs to be removed unless it can be reasonably explained.
@patedit this tracks back to https://github.com/mixpanel/mixpanel-android/pull/376 from you 8 years ago. Can you chip in?

hi @dabrowne , the tracking has been removed in v7.5.2.

@zihejia thanks for the quick turnaround :+1:

The 2 iOS libraries still do the same.


Resolve merge conflict and some of the checks not running

Hi @ekushele, this PR intends to solve the merge conflict and several updated tests in nanoseq:dev branch

Hi @ekushele, I have merged your PR to nanoseq:demultiplexed_fast5


Phone not returned from get_order_address() call

Hello,
Amazon never returns the phone field when I call get_order_address().
It returns all other info like "AddressLine1", "City", "CountryCode", "Name", "PostalCode", "StateOrRegion", and also "AddressLine2" if present, but never the phone.
This happens in both cases when we have AFN e MFN. (As specified in the documentation, this should't be returned only for Fulfillment by Amazon (FBA) orders.)
I have all the PII authorization setted. I also tried using a restricted data token but also in this case the phone field is not present.
Thanks in advance.

I think there are some errors in the PII fields returned by Amazon.
Please report bugs regarding the Selling Partner API in Amazon's repository, it would be great if you could report any response you get back here.


Fuchsia port

There are still a few bits and bobs that could take some cleaning up (the commit history is nonsense, and there are a few broken std functions that should be fixed rather than worked-around), but I think this is ready for review.
Tests are passing locally in QEMU.
cc @raphlinus, @rtsuk, @trishume, @kulakowski

@carllerche I can pick out the changes to the existing tests into a separate PR, if you'd prefer.

Awesome work! I wonder, is it feasible to change the standard library to stop using file descriptors? It seems like it'd be a muche asier implementation to only work with handles and not file descriptors, and it seems like the file descriptor dispatch could maybe be in a mio-fuchsia library rather than baked into core (ideally).

Yes! This is absolutely the goal-- at some point. This part of Fuchsia is still very much WIP, so these changes will have to wait for the implementations to stabilize (or even be written at all :smile:).
This messy file-descriptor-based implementation of the net primitives will be replaced as soon as I can help it.

Test are still green on QEMU.

I chatted briefly with @cramertj on IRC about the stability story here, and it's probably worth recappping. My thinking is that because the underlying OS is unstable still (e.g. fuchsia syscalls are changing every now and then) it's not really feasible to provide a stability guarantee of mio-on-fuchsia. I think the conclusion here is that mio just won't provide any backwards compatibility guarantees or semver guarantees on the Fuchsia platform for the near future. That'll buy us the freedom to change whatever we need here under the hood and notably break the public API as necessary, all the while without requiring mio to make a major version bump.

Okay I'm ready to okay this if someone could confirm that my understanding of edge triggering behavior on Fuchsia is correct (described in previous top-level review comment). Also I'd be happy if a comment was added on the atomic ordering, like I described in a now-folded reply.

@trishume Yes, your understanding matches mine. I've added a comment about the ordering as well.

@trishume The other test is failing because I used features that weren't available in Rust 1.9. @alexcrichton How would you prefer I handle that? is Rust 1.9-compatibility still a goal?

Ah yeah it'd be ideal to not worry too much about the compat for now, shouldn't be too hard to fix though?

@alexcrichton Okay, fixed.

Looks like the 1.9 compat may need one more change?

Oh whoops I didn't try building tests. You're absolutely right.

@alexcrichton Do you have any idea what's going on with the Android failure?

Hm that worked locally for me so let's see if it's spurious.

Hm... Looks like it passed this time around.

👍


test:dependencyClasspath ordering question

I have a question regarding the test dependencyClasspath. Apparently if you depend on a subproject using "test->test", the test classpath has the test-classes come before classes for the project, but classes comes before test-classes for all dependent subprojects.  Is this intended behavior, and if so, why?  I cannot find any reference to this behavior in the SBT docs.   This behavior makes it very difficult to share test configuration across projects.  I am using SBT 0.13.12 and Scala 2.11. Thanks.

Is this intended behavior, and if so, why? I cannot find any reference to this behavior in the SBT docs.

I think it's not documented because it's underspecified and whatever you're seeing is the result of the implementation. So in that sense, it's likely working as expected in the sense the there's no expectation at the detail you're describing in terms of test-classes vs classes.

This behavior makes it very difficult to share test configuration across projects.

You lost me here. Are you redefining classes with the same name across different subprojects? Do you have examples of what you're trying to do?

@eed3si9n thanks for your response.  We are using the typesafe Config library, where the resolution of the configuration settings is dependent on the order that reference.conf and/or application.conf are found on the classpath.  That means if we have a common component that is used across a number of subprojects, then we have to copy the test configuration to every subproject.   This is possibly fine for a single case, but it gets unwieldy and difficult to manage with many shared components.   If sbt was consistent with the order of the classpath, then this would be relatively trivial (i.e. it would "just work").  My expectation was definitely that test-classes for a project or subproject should always come before classes based on my experience using Maven.

We hit this recently.
I thought test dependencies would be ordered first in the classpath only to recently find out they're not. We were expecting that to be the case so we can disable things like remote logging and monitoring in our "core" project through config without having to disable them in each and every project that depends on "core"

@jalaziz test-classes is not a part of the test dependency classpath but classes is. The project classpath, in this case test-classes always appears before the dependency classpath. I don't think sbt has ever tried to specify the order of jars in the classpath. My suggestion would be to override Test / dependencyClasspath by re-sorting it. For example,
Test / dependencyClasspath ~= { cp => cp.sortBy(f => f.data.getName ) }

would resort the classpath by the file name of each jar or class directory.

I don't think we would try to distinguish between Test-configuraiton derived JARs vs Compile-configuration derived JARs. Maybe it's potentially possible, but from sbt's perspective we use whatever the underlying dependency resolver gives us be it Ivy or Coursier.

To clarify, here's the classpath:
[info] * Attributed(/Users/jalaziz/Development/foo/server/scala-client-api/target/classes)
[info] * Attributed(/Users/jalaziz/Development/foo/server/common/target/classes)
[info] * Attributed(/Users/jalaziz/Development/foo/server/common-server/target/classes)
[info] * Attributed(/Users/jalaziz/Development/foo/server/core/target/classes)
[info] * Attributed(/Users/jalaziz/Development/foo/server/core/target/test-classes)```

`scala-client-api` depends on `core` with `compile->compile;test->test`. However, notice how `core/target/test-classes` comes after the `core/target/classes`?

@eed3si9n are you saying that it is Coursier or Ivy that determines this order?

Just an update, reordering the classpath worked.
Here's what I ended up doing:
Test / dependencyClasspath ~= fixTestClasspath

rotected def fixTestClasspath(cp: Seq[Attributed[File]]): Seq[Attributed[File]] = {
val testClasspaths = cp.filter(c ⇒ c.data.getName == "test-classes").map(c ⇒ c.data.getParent → c).toMap
val addedTestClasspaths = mutable.Set.empty[Attributed[File]]
val reorderedClasspath = mutable.BufferAttributed[File]
cp.foreach({
case c if c.data.getName == "classes" && testClasspaths.get(c.data.getParent).isDefined ⇒
val testClasspath = testClasspaths(c.data.getParent)
if (!addedTestClasspaths(testClasspath)) {
reorderedClasspath += testClasspath
addedTestClasspaths += testClasspath
}
reorderedClasspath += c
case c if c.data.getName == "test-classes" ⇒
if (!addedTestClasspaths(c)) {
reorderedClasspath += c
addedTestClasspaths += c
}
case c ⇒ reorderedClasspath += c
})
reorderedClasspath

}

It's a bit more complicated because I wanted to ensure order is preserved except for reordering `test-classes` paths.


Fix bazel deps

Fix bazel deps following https://github.com/google/googletest/issues/3824

Odd code coverage errors in build will be fixed in a follow-up PR. They are not bazel-related.


No double-slash (//) found in source URL

I'm fed up of this warning. I wish there was a way of disabling it. I'm using local modules that are in one single repository.

This seems to be an easy fix, and good for new comers. Is PR welcome for this issue?

@leiarenee would you provide an example terragrunt project tree that uses local modules and exhibits this warning, including the relevant terragrunt.hcl and *.tf contents? I'm also using local modules and seeing this WARNing, and although I find it similarly annoying, I'm too new to terragrunt to think that it's not telling me something I'm actually doing wrong.
Here's a simplified project of mine that produces this WARNing on terragrunt run-all validate. It also produces some other errors I believe to be related to module source path declaration (Unsupported Attribute in dependency, No such file or directory), and I want to make sure I fully understand module source paths, relative paths (to dependencies' config_path, hooks' execute paths, ), and best practice here.
.
├── backend.tf                    # Generated by root terragrunt.hcl
├── globals.hcl
├── modules
│   ├── account
│   │   ├── data
│   │   │   ├── main.tf
│   │   │   └── terragrunt.hcl
│   │   └── terragrunt.hcl        # Empty
│   ├── region
│   │   ├── data
│   │   │   ├── main.tf
│   │   │   └── terragrunt.hcl
│   │   └── terragrunt.hcl        # Empty
│   ├── terragrunt.hcl            # Empty
│   └── vpc
│       └── terragrunt.hcl
├── provider.tf
├── script
│   ├── colorfuncs.sh
│   ├── functions.sh
│   └── terragrunt-workspace-hook.sh
├── terraform.tfstate.d
│   └── proto67
├── terragrunt.hcl
└── versions_override.tf

I've solved this problem with a workaround like many others I guess by using '//' somewhere in local path.
Eg.  https://github.com/leiarenee/sf-infra/blob/main/infra/live/k8s-deploy/namespace/terragrunt.hcl
source = "${get_parent_terragrunt_dir()}//library/terraform/modules/deploy-yaml"

Or
source = ".//terraform"

Where TF code is written directly under TG folder structure as in
https://github.com/leiarenee/sf-infra/blob/main/infra/live/job-resources/terragrunt.hcl
So maybe we should close it and accept it as it is :-)

@leiarenee thanks for your response. I've updated my previous post with much more detail about my use case and the errors I'm trying to resolve.
@leiarenee @canidam Would you take a quick look at my above post and see if any config problems jump out at you?
If this turns out to be unrelated to double-slashes in local module source paths, I can create a new issue, but as I'm so new to terragrunt, I'm still assuming this is cockpit error on my part. Thanks.
Also, @canidam I've traced the line in splitSourceUrl that logs this warning over on my fork back to where it's called in NewTerraformSource, which is clearly happening just before the IsLocalSource check, when the func becomes aware of whether the source is local or remote. I don't have the full picture here, but it looks like iff we really wanted to silence this warning for local sources we may be able to either change order of operations here, or simply add a conditional branch around the logger call inside SplitSourceUrl(). But, after reading and experimenting so much with local module sources and confirming that the double-slash is just as important for them as it is for remote URLs, I think we should not make any change here.

@leiarenee I looked over your repo, and it appears we're doing mostly similar things WRT the double-slash and using relative paths. One question I had was why do you use relative paths like ../ sometimes for source and other times you use the absolute: "${get_parent_terragrunt_dir()}//some/module"?
Also I don't understand how/if you're actually using mocked outputs there. I see your mock.hcl files, but don't see how they're being used. Do/did you actually run terragrunt run-all validate in your workflow, before deploying resources with terragrunt run-all apply?
On the one hand the problem I'm having is a well-known limitation of terragrunt: you can't expect to validate before apply without setting up adequate mocks. But I believe I have set these up and am still encountering the "Unsupported attribute" issue.

Improved checking of module path in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.50.13


Removal of not needed calls to toString() in the code base

Fixes #466
o Removed not needed calls to toString() to improve the code base

Hi @bmarwell,
thank you for your review. How can perform the merge of this pull request?

Hi @obfischer can you undo the second  commit please? The merge commit is not needed.

Rebased on master.

Hi @bmarwell, who can merge it? I don't have the permissions according to the UI.


[MPWG tab update] 2023 Budget and Project plan

@rstjames,
Thank you for updating the https://microprofile.io/workinggroup/#mptab-program-plan-budget as follows:
Description
The MicroProfile yearly Project plan is drafted and worked on in the open with the MicroProfile community. The MicroProfile Budget is prepared by Eclipse Management Organization (EMO) based on the approved budget. Four (4) yearly fees are paid to the EMO totaling ~69% of the budget:

Working Group fees (15% of membership fees)
General and Administration (12% of membership fees)
Governance & Management, including Spec Management ( $30,000)
Shared Marketing Services Fee ($7,000)

Under Marketing Expenses allocation expenses, it’s acceptable to overspend in one line item and underspend in another without formally adjusting the approved budget. The EMO will adjust the expenditures based on the costs incurred.

2023 updated Links

2023 Budget https://drive.google.com/drive/folders/19NVgyJ3RzU526A1_VT9R3g8q85JyaNFi
2022 Program Plan https://drive.google.com/drive/folders/19NVgyJ3RzU526A1_VT9R3g8q85JyaNFi
2023 Forecast budget https://docs.google.com/spreadsheets/d/1W1HT3lwmIHqOzF1Iz-Tx6fnDgIex02fbGRKDO3_nvwI/edit#gid=1728036894

All updated

@rstjames,
This ticket has a part 2. Improvement section.
I would like your thoughts on it before implementation.
Can the Budget and Program Plan be added directly into the space?
Similar to what we did for the MP speaker presentation under https://microprofile.io/resources/branding/#pres-temp

The addition can be just like the calendar insert on the MPWG page, with no extra wording.
The documents and the space are updated once a year. We want the reader to stay in MP.io so adding the documents seems the best way.
What is your take on this improvement? @rstjames

@aeiras I am not able to do this until I have editing rights to the slided decks. But I can embed them when I have this.

@aeiras I am not able to do this until I have editing rights to the slided decks. But I can embed them when I have this.

Gotcha, @rstjames.
I have copied the documents into the community drive https://drive.google.com/drive/folders/1XftsX68Dmc0ZWdCa3wYgpnZJy89QX3Gg.

The program plan is always pptx that we convert to PDF (later)
Budget is always PDF, if you need other format, you can modify it now.

Thank you,

@aeiras The program plan was added as a slide show but the budget pdf we can not. I think it is digested better as a linked pdf anyways

Adjustment works fantastic, Ryan.
In the year, the project plan is the most important document for the MPWG to look at and work towards. :)


[AnVIL DX] Validate custom events are accessible on the GA4 Query Explorer for the AnVIL Explorer dev and prod sites

Need
Custom events are to be included in analytics reports.
Approach
Add a custom dimension in GA for every custom parameter specified in the following tags in Google Tag Manager:

Validate GA4 Query Explorer access https://github.com/DataBiosphere/data-browser/issues/4118
Inventory custom events https://github.com/DataBiosphere/findable-ui/issues/174
Determine which custom events are and are not being propagated

This list may be incomplete, but fill it out from step 2 above.

[x] tag - entity_selected
[x] tag - entity_table_paginated
[x] tag - entity_table_sorted
[x] tag - filter_selected

Definition of Done
Every custom parameter is visible in the GA4 Query Explorer for AnVIL DX prod and staging:
AnVIL DX - GA4

tag - entity_selected

[x] entity_name

tag - entity_table_paginated

[x] entity_name
[x] pagination_direction

tag - entity_table_sorted

[x] entity_name
[x] column_name
[x] sort_direction

tag - filter_selected

[x] filter_name
[x] filter_value

AnVIL DX CC Staging - GA4

tag - entity_selected

[x] entity_name

tag - entity_table_paginated

[x] entity_name
[x] pagination_direction

tag - entity_table_sorted

[x] entity_name
[x] column_name
[x] sort_direction

tag - filter_selected

[x] filter_name
[x] filter_value

Hi @MillenniumFalconMechanic~ I see each event except one  on the analytics reports for "AnVIL DX - GA4", but no events appear for "AnVIL DX CC Staging - GA4". The exception is for bulk_download_requested, but I think that's because that event is only triggered on HCA, when the "Request curl command" button is clicked, so I didn't add it to the list. I also don't see any of the tags in the Tag Manager for anvilproject.org, but I do see them for data.humancellatlas.org. The tags I see for anvilproject.org are:

"tag - site search"
"tag - site search select result"

I can't find anything in the findable-ui repo for those tags, and they don't appear in GA4 reports, so I think I'm either missing something (access issues?) or they're not configured correctly. Let me know what the next steps are!

All the tags seem to be present on the tag manager for https://explore.anvilproject.org as well! I don't see a tag manager account for the staging site though

Brilliant, thanks @jpaten!

I don't see a tag manager account for the staging site though

For AnVIL, staging and prod share the same GTM container, and use the lookup - measurement ID by hostname variable to determine which GA data stream to forward events to.

but no events appear for "AnVIL DX CC Staging - GA4"

Thank you for spotting this! I have updated the lookup - measurement ID by hostname variable to use the correct staging hostname.
Next Steps
For the four Findable custom events that currently exist in the AnVIL Explorer GTM container, can you:

Confirm all parameters (as specified in the README) are available in the GA4 Query Explorer for both staging and prod? For any parameter that is missing, follow the CC Google Analytics Design Document (specifically the Configuring a Custom Event and Custom Events sections), and,
Document the parameters below with checkboxes to indicate they have been checked? (I have added checkboxes to entity_selected as an example.)

AnVIL DX - GA4

tag - entity_selected

[ ] entity_name

tag - entity_table_paginated
tag - entity_table_sorted
tag - filter_selected

AnVIL DX CC Staging - GA4

tag - entity_selected

[ ] entity_name

tag - entity_table_paginated
tag - entity_table_sorted
tag - filter_selected

Hi @MillenniumFalconMechanic! The Query Explorer is showing all custom events as visible with all the correct variables on both Staging and Prod. Let me know if there's anything else you want me to do, otherwise I think it's safe to close this ticket!

Thank you @jpaten!


bluetooth: OpenDroneID sample application w/ GAP Broadcaster Role.

Adding sample app for demonstrating transmission of OpenDroneID.

Hi @kartben, Thank you for the review. We don't need this PR to be reviewed. We created this draft PR to make sure all checks were being passed. Kindly ignore this for now. I will finalize my changes and request a review later.


Evaluate mappings

Use Choco to evaluate mappings with a given configuration.
This also includes context and attribute values.
Returns a list of deltas

As API...I will do the rest


docker daemon docs incorrectly portrayed examples

the docs that end up here --> http://docs.docker.com/reference/commandline/cli/#daemon
since the commit 29d01b7, now the examples for running the daemon have a PS1 command prompt of $ which indicates a non-privileged user. Though correctly marking the command prompt with # which indicates root user, would make them appear as comments.
We should find a markdown way to show the prompt for a # PS1 to be normal, or add back sudo for the docker -d type examples.
/cc @moxiegirl @thaJeztah @SvenDowideit

@vbatts So, if I su to root on an ubuntu instance I get this:
root@ip-172-31-34-181:/home/ubuntu#
So, what do you think of using something like that rather than a plain old #

It's a tough one; we want the shell examples to be as "generic" and "copy-pasteable" as possible; to much info before the actual command will make it more difficult to read (and people might copy it and execute it)
Perhaps root@docker-host:~# echo foo might work; the obvious advantage that it's more clear that it's ran as root (for those unaware of the #/$ differences)
Otoh, if this is the only location where this has to be actually executed as root (and not, for example, a user in the docker group). sudo might be better?

@thaJeztah If they copy the prompt, the paste won't work regardless. :-P
Otoh, yes, adding sudo for specific cases like this does make sense.  Point in case, I didn't pull the sudo from the installation docs.

@moxiegirl yes better copy-pasteable has been talked about in the past, perhaps after switching to another engine for docs? I'll keep this in mind as well, will look around if I can find good libraries for that.

Point in case, I didn't pull the sudo from the installation docs.

When @vbatts mentioned this on IRC, the removal of sudo sprang in mind (but I didn't check), and it could well be that this has been there all along. Overall I don't think the incorrect prompt is a major issue, but it needs to be fixed at some point.
For the time being, adding sudo might be the easiest solution.

ping @thaJeztah


right-sidebar: Clicking between Users and magnifying glass should open User search.

This whole region should be a click target, similar to how we do Stream search.

It's a bit confusing not to have it work, given that we do it on the streams side.

Actually I have not been able to contribute to the repo because of my approaching semester exams, I hope u will understand.


feat(tests): Introducing BDD tests for all bindings

This pull request aims to introduce Behavior-Driven Development (BDD) tests for all of our bindings. It serves as a proof-of-concept and uses the Python binding as an example.
Moving forward, we will maintain the same feature file for all bindings. Only when a binding passes all feature tests will it be marked as ready.
Here is an example using Python:
:) behave tests
Feature: OpenDAL Binding # tests/binding.feature:18

  Scenario: OpenDAL Blocking Operations                             # tests/binding.feature:20
    Given A new OpenDAL Blocking Operator                           # tests/steps/binding.py:23 0.000s
    When Blocking write path "test" with content "Hello, World!"    # tests/steps/binding.py:27 0.000s
    Then The blocking file "test" should exist                      # tests/steps/binding.py:31 0.000s
    Then The blocking file "test" entry mode must be file           # tests/steps/binding.py:35 0.000s
    Then The blocking file "test" content length must be "13"       # tests/steps/binding.py:39 0.000s
    Then The blocking file "test" must have content "Hello, World!" # tests/steps/binding.py:43 0.000s

  Scenario: OpenDAL Async Operations                             # tests/binding.feature:28
    Given A new OpenDAL Async Operator                           # tests/steps/binding.py:48 0.000s
    When Async write path "test" with content "Hello, World!"    # tests/steps/binding.py:53 0.001s
    Then The async file "test" should exist                      # tests/steps/binding.py:58 0.000s
    Then The async file "test" entry mode must be file           # tests/steps/binding.py:63 0.000s
    Then The async file "test" content length must be "13"       # tests/steps/binding.py:69 0.000s
    Then The async file "test" must have content "Hello, World!" # tests/steps/binding.py:75 0.000s

1 feature passed, 0 failed, 0 skipped
2 scenarios passed, 0 failed, 0 skipped
12 steps passed, 0 failed, 0 skipped, 0 undefined
Took 0m0.002s

close https://github.com/apache/incubator-opendal/issues/1521
close https://github.com/apache/incubator-opendal/issues/1550

For nodejs: https://github.com/cucumber/cucumber-js
For C: https://github.com/cucumber/cucumber-cpp
In the future, go/lua/java will use their own BDD tests framework.s


bug with uint32s and ranges

I found a weird bug that prevents the following code from running:
import numba
import numpy

@numba.jit(nopython=True)
def broken():
    V = numpy.empty(1)
    s = numpy.uint32(1)

    for i in range(s):
        V[i] = 1
    for i in range(s, 1):
        pass

>> broken()

numba.errors.InternalError: unsupported array index type float64 in [float64]

Things that will make the bug disappear:

remove either loop
replace the V[i] = 1 with a pass
set s to a int64 instead of a uint32
replace range(s, 1) with just range(s)

Ah, I suspect this is a bad interaction with our type casting rules, which is causing the uint32 to be cast up to a float64, which Numba doesn't support for array indexing.
@sklam: Any ideas what is going wrong in typing?  Is it the range(s, 1) doing arithmetic between a signed and unsigned int?

There is an interesting interaction with variable scoping and the coercion of mix-signedness.
The cause is that the current scoping logic thinks that both i from the two loops are in the same scope.  The 1st loop sets type of i to uint64.  The 2nd loop sets type of i to int64.  As a result, i is coerce(uint64, int64)->float64 (due to numpy).   The real fix is to improve the scoping so that the two i create different scopes.
The simplest workaround is to use a different variable name in the 2nd loop.

This makes me wonder if we need the unsigned integer version of range().  Dropping the unsigned overloads would suppress the issue here and it doesn't affect any tests.

I' m getting an error numba.errors.InternalError: unsupported array index type ?int64 in [?int64] with one array from some dataframe.values. I don' t know what this means.

My error is because I declared the index as None, switching to an int solves this. Document here in case someone get same issue through search.

@xgdgsc I'm having the same error when pulling from a numpy array. How did you fix it again? Did you pull the value and then cast as int? For example ind = int(arr[i])? This is not working for me with nopython=True.


Separating out link tests from service catalog dependent tests

What is the purpose of this change? What does it change?

Running link and unlink test on 4.2, 4.1 and 3.11 cluster
Was the change discussed in an issue?
fixes #2261

How to test changes?

check 4.2, 4.1 and 3.11 test result in CI

@cdrage Can you please do next round of review

/test v4.3-integration
/test v4.1-integration

/test v4.3-integration
/test v4.1-integration

Wait till #2363 and #2362 go in first

I'm assuming these files are simply just being moved from servicecatalog to the tests/integration folder?

@cdrage Yes, and some test spec those are purely dependent on service catalog are left behind in the same service catalog file.

/hold cancel

/lgtm

New changes are detected. LGTM label has been removed.

It was just a rebase
/lgtm

#2299
/test v4.3-integration

/refresh

/kind test


Usage of participates in

Description of the issue
Currently, we have a pattern 'artifical object' 'participates in' some process. Example of this pattern are:

'energy transformation unit' 'participates in' some 'energy transformation'
vehicle 'participates in' some transport

In OEO dev meeting 46, @fabianneuhaus mentioned that is not a good pattern as the objects not necessarily participate all the time. A power plant e.g. can be switched off for some time and does not participate in an energy transformation while switched off.
Ideas of solution
If you already have ideas for the solution describe them here
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

Same problem in issue #1494:

This is where it gets tricky. Since the EVs are mobile, the modes in which they transfer energy vary depending on their operation. In the case of vehicle charging/vehicle-to-grid the processes can only happen when the vehicle is connected to a charging station, so it must be sitting idle. This means that there is either a parallel process of a vehicle being connected or we allow a energy transfer with no actual energy being transferred ( plug-in process) . See diagram.

One option to solve this is to delete most of the thing 'participates in' some process axioms and replace them with process 'has participant' some thing axioms.

I see three thumbs up for my last proposal so I think we can implement this. :rocket:
Any volunteers for implementation? Maybe @u-mueller? Else I can do it.

This issue is ready for implementation, but nothing happened in the last month. So I will start implementing now.

I tried to implement this, but looking in specific axioms this seemingly easy solution causes some serious headaches. For example, we currently have the axiom bicycle 'participates in' some 'passenger transport'. But we can not state 'passenger transport' 'has participant' some bicycle as there are also types of passenger transport like public transport without bicycles.
If we want to bicycle 'participates in' some 'passenger transport' we need a subclass where we then can axiomatise bicycle 'participates in' some 'biking'. And if this example is not an exception, but more the general rule as there are many instances.

If we want to bicycle 'participates in' some 'passenger transport' we need a subclass where we then can axiomatise bicycle 'participates in' some 'biking'. And if this example is not an exception, but more the general rule as there are many instances.

'participates in' some 'passenger transport' is not necessarily wrong, some does not mean all.

If we want to bicycle 'participates in' some 'passenger transport' we need a subclass where we then can axiomatise bicycle 'participates in' some 'biking'. And if this example is not an exception, but more the general rule as there are many instances.

'participates in' some 'passenger transport' is not necessarily wrong, some does not mean all.

Sorry, seems I got confused. We want to avoid axioms like bicycle 'participates in' some 'passenger transport' and turn them around. We cannot axiomatise 'passenger transport' 'has participant' some bicycle. We need here an axiom biking 'has participant' some biking and this is not possible without a new subclass.

Proposal from a discussion with @nelekoehler:
transport 'has participant' some 'transport participant'
'transport participant' Equivalent to ((person or vehicle or good) and 'has disposition' some 'transport participant disposition')
transport participant disposition: disposition of a person, good or vehicle to take part in a transport process.

transport participant disposition: disposition of a person, good or vehicle to take part in a transport process.

To make it Aristotelian: A transport participant disposition is a disposition of a person, good or vehicle to take part in a transport process.

'transport participant' Equivalent to ((person or vehicle or good) and 'has disposition' some 'transport participant disposition')

The most common parent class of person or vehicle or good would be continuant. So it should get also a SubClassOf: continuant axiom that the OEO viewer can show it somewhere.
Definition proposal: A transport participant is a person, vehicle or good that can participate in transport.

Comments above mention this being ready to implement, but I still have a few questions:
participates in and has participant are inverse of each other. Even if I remove the participates in relation the reasoner will infer it anyway, after I readded the has participant axioms.
How can we deal with this?

One option to solve this is to delete most of the thing 'participates in' some process axioms and replace them with process 'has participant' some thing axioms.

Are there any participates in relations in particular that should be kept?

And if this example is not an exception, but more the general rule as there are many instances.

Does this mean we might have to add multiple similar classes like transport participant?


Functional suggestion

Purpose

Just an idea to clean up code. I just refreshed my memory on using the functional capabilities of C# and thought this was a bit cleaner. Feel free to discard.

Removing PR. Might be addressed in an overhaul later.


Added Electrostatic Force Calculator

Fixes Issue🛠
Closes #1571
Description👨💻
Electrostatic Force Calculator is a web application that calculates Electrostatic Force given the value of two point charges and the distance between them.
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

1. Pls center these!

2. Also include the formula used for calculation in the README.md file as well as in the UI below the heading @Anshika14528

@Rakesh9100 sir , is it ok now.


more Generator combinators

zip3/zip4/...
map for collection generators

zip3 and zip4 are done

map for collection generators seems quite hard on first try

closing in favor of #4 as that appears to be all that is left on this issue.


Adding gpt4o vision example

Summary:
Adds an example of vision using gpt4o
Key points:

Adds vision example using gpt4o
Currently utilizes two receipts. One of them is parsed correctly, the second one isn't highlighting the limitations of these models for OCR.
Model validator helps us ensure sum of items = total in the receipt.

:rocket: This description was created by Ellipsis for commit 18b5c969df54ae76505d7c934704c1b24f0e3e26

Summary:
Introduces a new example script for demonstrating OCR capabilities of GPT-4o using receipts, with data validation.
Key points:

Adds run_vision_gpt4o.py under examples/extract-table/
Utilizes openai and pydantic for OCR and data validation
Demonstrates model's OCR capabilities with two receipt images
Includes a model validator to ensure accuracy of extracted data

Generated with :heart: by ellipsis.dev

@ellipsis-dev