[GPU] Add option for no gpu.block_dim in GPUDistributeScfFor

This PR adds an option to generate arith.constant ops with workgroup sizes in place of gpu.block_dim ops in GPUDistributeScfFor. The gpu.block_dim ops are not currently handled properly on ROCM backends, so this option is needed to support GPUDistributeScfFor on the LLVMGPU path.

I will create an issue later about the gpu.block_dim on ROCM. It will be easier to create a repro once the LLVMGPU winograd pipeline is landed.


oncoPrintPlotter get wrong column split

Hi, PyComplexHeatmap provides Python users with convenient heatmap painting tools. I want to display variants in a population using oncoPrintPlotter. I want to show SampleID in rows and VariantID in columns. Here is the code:
cols = ['0/0','0/1','1/1','./.']
colors = ['#552365','#2e9a95','#f7eb3c','#050707']

plt.figure(figsize=(18,20))
op = oncoPrintPlotter(data = stack_df,x = 'VariantID',y = 'SampleID',
                    values = cols,colors = colors,label = 'Genotype',
                    legend_hpad = 0,show_rownames = True,show_colnames = True,
                    width = 1.8,row_gap = 0)
plt.savefig(query_region_variant_plot,bbox_inches='tight')
plt.show()

The heatmap gets the wrong variant split shown in the red circles:

I set the VariantID as the x-axis. The ticklabels in the figure cannot match the heatmap columns. And the column number in the heatmap is obviously great than the VariantID count.
The input stack_df contains the genotype information for each variant :

Here I show the input file and the output figure.
input data
variant_stack_record.txt
output figure
variant_plot.pdf

I have found the problem, the figure width I set was 18, smaller than the variant count * width (13*1.8)

Thank you for your feedback.


build: bump version

/status can-merge

/status can-merge

Codecov Report

Merging #158 (47d1ec5) into master (53b58c9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #158   +/-   ##
=======================================
  Coverage   85.72%   85.72%
=======================================
  Files          11       11
  Lines         946      946
=======================================
  Hits          811      811
  Misses         76       76
  Partials       59       59

Flag
Coverage Δ

unittests
85.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 53b58c9...47d1ec5. Read the comment docs.


how to read all elements information from .dat file by PyMapdl_reader

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
My geometry cotains two solids,so elements information in .dat file is divided into two parts(both are same material).I awnt to use PyMapdl reader to read all elements information from .dat file but it just read the first part elements information.How should I do?

📝 Steps to reproduce
like my discription of the bug
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
19.2
🐍 Which Python version are you using?
3.11
📦 Installed packages
ansys-api-fluent==0.3.12
ansys-api-mapdl==0.5.1
ansys-api-mechanical==0.1.0
ansys-api-meshing-prime==0.1.1
ansys-api-platform-instancemanagement==1.0.0
ansys-api-pyensight==0.3.0
ansys-api-sherlock==0.1.14
ansys-api-systemcoupling==0.1.0
ansys-dpf-composites==0.2.0
ansys-dpf-core==0.8.1
ansys-dpf-gate==0.3.1
ansys-dpf-gatebin==0.4.1
ansys-dpf-post==0.4.0
ansys-dynamicreporting-core==0.4.0
ansys-fluent-core==0.14.0
ansys-grantami-bomanalytics==1.1.3
ansys-grantami-bomanalytics-openapi==1.0.0
ansys-grantami-recordlists==1.0.0
ansys-grantami-serverapi-openapi==1.0.0
ansys-grpc-dpf==0.8.1
ansys-mapdl-core==0.65.1
ansys-mapdl-reader==0.52.20
ansys-math-core==0.1.1
ansys-mechanical-core==0.8.0
ansys-meshing-prime==0.4.1
ansys-motorcad-core==0.1.3
ansys-openapi-common==1.2.1
ansys-optislang-core==0.2.1
ansys-platform-instancemanagement==1.1.1
ansys-pyensight-core==0.5.3
ansys-pythonnet==3.1.0rc1
ansys-seascape==0.2.0
ansys-sherlock-core==0.2.0
ansys-systemcoupling-core==0.1.3
ansys-tools-path==0.3.2
appdirs==1.4.4
asgiref==3.7.2
build==1.0.3
cachetools==5.3.1
certifi==2023.7.22
cffi==1.16.0
charset-normalizer==3.3.0
click==8.1.7
clr-loader==0.2.6
colorama==0.4.6
contourpy==1.1.1
cryptography==41.0.4
cycler==0.12.1
Django==4.2.6
docutils==0.20.1
filelock==3.12.4
fonttools==4.43.1
geomdl==5.3.1
google-api-core==2.12.0
google-api-python-client==2.104.0
google-auth==2.23.3
google-auth-httplib2==0.1.1
googleapis-common-protos==1.61.0
grpcio==1.59.0
grpcio-status==1.48.2
h5py==3.10.0
httplib2==0.22.0
idna==3.4
importlib-metadata==6.8.0
jaraco.classes==3.3.0
keyring==24.2.0
kiwisolver==1.4.5
lxml==4.9.3
markdown-it-py==3.0.0
matplotlib==3.8.0
mdurl==0.1.2
more-itertools==10.1.0
nh3==0.2.14
numpy==1.24.3
packaging==23.2
pandas==2.1.1
Pillow==10.1.0
pkginfo==1.9.6
platformdirs==3.11.0
plumbum==1.8.2
pooch==1.7.0
protobuf==3.20.3
psutil==5.9.6
pyaedt==0.6.76
pyansys==2023.2.11
pyansys-docker==5.0.4
pyansys-tools-versioning==0.5.0
pyasn1==0.5.0
pyasn1-modules==0.3.0
pycparser==2.21
Pygments==2.16.1
pyiges==0.3.1
pyparsing==3.1.1
pypiwin32==223
pypng==0.20220715.0
pyproject_hooks==1.0.0
pyspnego==0.10.2
python-dateutil==2.8.2
pythonnet==3.0.1
pytwin==0.3.0
pytz==2023.3.post1
pyvista==0.40.4
pywin32==306
pywin32-ctypes==0.2.2
PyYAML==6.0.1
readme-renderer==42.0
requests==2.31.0
requests-negotiate-sspi==0.5.2
requests-ntlm==1.2.0
requests-toolbelt==1.0.0
rfc3986==2.0.0
rich==13.6.0
rpyc==5.3.1
rsa==4.9
scipy==1.11.3
scooby==0.9.2
six==1.16.0
sqlparse==0.4.4
sspilib==0.1.0
tqdm==4.66.1
twine==4.0.2
typing==<IP_ADDRESS>
typing_extensions==4.8.0
tzdata==2023.3
uritemplate==4.1.1
urllib3==1.26.18
vtk==9.2.6
websocket-client==1.6.4
zipp==3.17.0

@germa89 @mikerife could you help this user out?

Hi @planstress
Well, this may depend on what you want to do with the input file.  Some choices are:
1 Define a Named Selection in WB Mechanical of Part1.  This will be a element 'component' in the input file (MAPDL component).  The PyMAPDL Reader then often has an option to specify a component when performing some function like creating a result plot.
2 Use PyMAPDL and you can use its select logic as needed to work with a sub-set of the model.
3 Use PyDPF which has Scoping based on MAPDL element attributes like material ID.  A unique material ID is used by each Part in WB Mechanical even if the actual material is the same between parts.
Mike

Hi @mikerife
Thank you for replying.Honestly,I don't quite understand the methods you offered because I am rookie at this.
In this example,I want to read all elements information from dat file and then extract element type as well as nodes of every elements,so I issue thepymapdl_reader.Archive('ds.dst')to read elements information.In my opinion,it should read all elements information but it only read the first part elements actually.
I wonder why is that and how can I do to my purpose?

@planstress
Ok I read your description backwards!  I see now that you are reporting that PyMAPDL Reader has elements missing compared to the ds.dat file.  Can you share the ds.dat file?  There is not much to go on just from a screen shot.

@mikerife
of course,here it is.Thank you.
Uploading ds.zip…

Doesn't seem like the file attachment worked.

@mikerife
Sorry about that,I upload it again.
Uploading ds.zip…

Uploading ds.zip…

Uploading ds.zip…

ds.zip

@mikerife
sorry about that,I upload it again.
ds.zip

@planestress
PyMAPDL-Reader development is quite stale at the moment. I'm not quite sure why not all the elements are read in to be honest.
If I were you, I would probably switch to PyMAPDL where you can do:
from ansys.mapdl.core import launch_mapdl
mapdl = launch_mapdl()

mapdl.prep7()
mapdl.input("ds.dat")

mapdl.cmsel("s", "CRACK_TIP")
# Then you have selected the named selection called "CRACK_TIP".
...
# perform operations

This can be done, because in the ds.dat file you have attached the named selections (which are just components) are defined as:
/com,*********** Send Named Selection as Node Component ***********
CMBLOCK,CRACK_TIP,NODE,       41
(8i10)
       138       182       183       184       185       186       187       188
       189       190       191       192       193       194       195       196
       197       198       199       200       201     20501     20792     20800
     20808     20817     20826     20836     20846     20854     20862     20871
     20880     20891     20902     20913     20922     20930     20940     20949
     20958

PyMAPDL should create the proper model from the dat file, and then it should be able to read the components.

@germa89
Thank you for your reply.In fact,I want to caculate stress of some point in every element base on element shape function,so I have to get all elements information of this model include element type,nodes belong to the elements and so on.
Besides,I tried your code
from ansys.mapdl.core import launch_mapdl
mapdl = launch_mapdl()

and then occured this error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\31142\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\ansys\mapdl\core\launcher.py", line 1634, in launch_mapdl
    raise exception
  File "C:\Users\31142\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\ansys\mapdl\core\launcher.py", line 1590, in launch_mapdl
    from ansys.mapdl.core.mapdl_corba import MapdlCorba
  File "C:\Users\31142\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\ansys\mapdl\core\mapdl_corba.py", line 16, in <module>
    raise ImportError(
ImportError: The CORBA interface is only supported for Python 3.8 and earlier

My python version is 3.11 and my ansys version is 19.2.

@planestress well that error is pretty evident.  Try installing Python 3.8 and trying this again.  Or update to a newer release of MAPDL (why are you using v19.2???).

@mikerife
I pay little attention to the ansys versions,I think they are all pretty much the same.Can I achieve my purpose if I follow German's strategy?

@planestress
PyMAPDL-Reader development is quite stale at the moment. I'm not quite sure why not all the elements are read in to be honest.
If I were you, I would probably switch to PyMAPDL where you can do:
from ansys.mapdl.core import launch_mapdl
mapdl = launch_mapdl()

mapdl.prep7()
mapdl.input("ds.dat")

mapdl.cmsel("s", "CRACK_TIP")
# Then you have selected the named selection called "CRACK_TIP".
...
# perform operations

This can be done, because in the ds.dat file you have attached the named selections (which are just components) are defined as:
/com,*********** Send Named Selection as Node Component ***********
CMBLOCK,CRACK_TIP,NODE,       41
(8i10)
       138       182       183       184       185       186       187       188
       189       190       191       192       193       194       195       196
       197       198       199       200       201     20501     20792     20800
     20808     20817     20826     20836     20846     20854     20862     20871
     20880     20891     20902     20913     20922     20930     20940     20949
     20958

PyMAPDL should create the proper model from the dat file, and then it should be able to read the components.

@planestress
"I pay little attention to the ansys versions, I think they are all pretty much the same."
Depending on what feature/function you are using this could be a very dangerous/wrong statement, or not.  Let's assum now not and see what happens.
"Can I achieve my purpose if I follow German's strategy?"
Install Python 3.8 and test and see what happens.

@mikerife
I install Python3.8.10 and reinstall PyMAPDL version0.67,0.68 doesn't support Python3.8.Oh my god,why there are so many versions???Then I do as German's method but there is a new error.(I am devastated T_T)
CRITICAL - pymapdl_global -  logging - handle_exception - Uncaught exception
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "D:\Program Files (x86)\python3.8.10\lib\site-packages\ansys\mapdl\core\launcher.py", line 1634, in launch_mapdl
    raise exception
  File "D:\Program Files (x86)\python3.8.10\lib\site-packages\ansys\mapdl\core\launcher.py", line 1597, in launch_mapdl
    mapdl = MapdlCorba(
  File "D:\Program Files (x86)\python3.8.10\lib\site-packages\ansys\mapdl\core\mapdl_corba.py", line 191, in __init__
    super().__init__(
  File "D:\Program Files (x86)\python3.8.10\lib\site-packages\ansys\mapdl\core\mapdl.py", line 265, in __init__
    _sanitize_start_parm(start_parm)
  File "D:\Program Files (x86)\python3.8.10\lib\site-packages\ansys\mapdl\core\mapdl.py", line 203, in _sanitize_start_parm
    raise ValueError(f"The argument '{each_key}' is not recognaised.")
ValueError: The argument 'log_broadcast' is not recognaised.

@planestress the errors you are getting is because PyMAPDL is trying to connect using the CORBA interface which is the one recommended for v19.2. See Compatibility between MAPDL and interfaces
While it is true that most of the MAPDL commands remain fairly stable between versions, PyMAPDL interface does not. In 2021R1, the gRPC interface was introduced and since then it has been the recommended PyMAPDL interface. CORBA has been deprecated recently, and it was never fully covered by testing.
My advice is to install the latest MAPDL version (at least 2022R2), and update PyMAPDL, the experience should be way better.


panic: runtime error: invalid memory address or nil pointer dereference

I am running 0.8.0 app version with aws role that has admin access
When I run this app locally against Account A (with only mysql db instance) it worked
However when I run this app locally against Account B (this one has multiple instances with mysql postgreSQL) I get the below error
{"time":"2024-03-18T03:03:37.915452047Z","level":"INFO","msg":"starting the HTTP server component"}
{"time":"2024-03-18T03:03:52.446299831Z","level":"INFO","msg":"get RDS metrics"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x1c11be2]
goroutine 100 [running]:
github.com/qonto/prometheus-rds-exporter/internal/app/ec2.(*EC2Fetcher).GetDBInstanceTypeInformation(0xc00046df40, {0xc000434400, 0xc, 0x10})
/home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/ec2/ec2.go:94 +0x842
github.com/qonto/prometheus-rds-exporter/internal/app/exporter.(*rdsCollector).getEC2Metrics(0xc00015c508, {0x7f700a6438c0, 0xc000292680}, {0xc000434400, 0xc, 0x10})
/home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/exporter/exporter.go:436 +0x14f
created by github.com/qonto/prometheus-rds-exporter/internal/app/exporter.(*rdsCollector).fetchMetrics in goroutine 97
/home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/exporter/exporter.go:376 +0x537

ADD： I only turn on PROMETHEUS_RDS_EXPORTER_COLLECT_INSTANCE_TYPES: true  turn off other
If  delete ec2 privilege in aws iam role     I can get rds_instance_info metrics

I have the same problem, do you have a solution?
Thanks

Thanks for reporting this issue, sorry about that issue.
It's definitely a bug that we must fix!
I'll propose fix by the middle of next week.

We just released the 0.8.1 that contains the fix.
Can you update to this version and confirm that it works as expected?

Bonjour, merci pour le retour.
Je viens de tester votre nouvelle version 0.8.1 et j'ai comme erreur :
│ {"time":"2024-03-25T15:24:36.471717476Z","level":"INFO","msg":"starting the HTTP server component"}                                                                                │
│ {"time":"2024-03-25T15:24:47.29626138Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ panic: runtime error: invalid memory address or nil pointer dereference                                                                                │
│ [signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x1c206ff]                                                                                │
│                                                                                │
│ goroutine 111 [running]:                                                                                │
│ github.com/qonto/prometheus-rds-exporter/internal/app/ec2.(*EC2Fetcher).GetDBInstanceTypeInformation(0xc0006b9f40, {0xc000540a00, 0x9, 0x10})                                                                                │
│     /home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/ec2/ec2.go:95 +0x8bf                                                                                │
│ github.com/qonto/prometheus-rds-exporter/internal/app/exporter.(*rdsCollector).getEC2Metrics(0xc0000f8a08, {0x7fe2caf7d440, 0xc0002fa4e0}, {0xc000540a00, 0x9, 0x10})                                                                                │
│     /home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/exporter/exporter.go:436 +0x14f                                                                                │
│ created by github.com/qonto/prometheus-rds-exporter/internal/app/exporter.(*rdsCollector).fetchMetrics in goroutine 62                                                                                │
│     /home/runner/work/prometheus-rds-exporter/prometheus-rds-exporter/internal/app/exporter/exporter.go:376 +0x537

Voici ma configuration values.yaml :
#
# Exporter settings
#

# Enable debug mode
debug: true

# Log format (text or json)
# log-format: json

# Path under which to expose metrics
# metrics-path: /metrics

# Address to listen on for web interface
# listen-address: ":9043"

# Path to TLS certificate
# tls-cert-path: ""

# Path to private key for TLS
# tls-key-path: ""

# Enable OpenTelemetry traces
# See https://opentelemetry.io/docs/languages/sdk-configuration/otlp-exporter for configuration parameters
# enable-otel-traces: true

#
# AWS credentials
#

# AWS IAM ARN role to assume to fetch metrics
# aws-assume-role-arn: arn:aws:iam::000000000000:role/prometheus-rds-exporter

# AWS assume role session name
# aws-assume-role-session: prometheus-rds-exporter

#
# Metrics
#

# Collect AWS instances metrics (AWS Cloudwatch API)
collect-instance-metrics: true

# Collect AWS instance tags (AWS RDS API)
collect-instance-tags: true

# Collect AWS instance types information (AWS EC2 API)
collect-instance-types: true

# Collect AWS instances logs size (AWS RDS API)
collect-logs-size: true

# Collect AWS instances maintenances (AWS RDS API)
collect-maintenances: true

# Collect AWS RDS quotas (AWS quotas API)
collect-quotas: true

# Collect AWS RDS usages (AWS Cloudwatch API)
collect-usages: true

I have the same problem, do you have a solution? I only use collect-instance-types: true

Sorry about that! I'm reopening this bug and will propose a more complete fix.

The AWS SDK could return a null pointer for RDS instances that do not support optimized EBS, we were not testing this case and https://github.com/qonto/prometheus-rds-exporter/pull/152 was not enough to handle it.
We adopted a more robust approach with https://github.com/qonto/prometheus-rds-exporter/pull/154.
We have just released the v0.8.2, which should finally solve this problem.
Can you update to this version and confirm that it works as expected?

@vmercierfr
C'est résolu pour moi, merci pour la réactivité !
│ {"time":"2024-03-26T12:03:35.208829158Z","level":"INFO","msg":"starting the HTTP server component"}                                                                                │
│ {"time":"2024-03-26T12:03:47.295649385Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:04:17.294120478Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:04:47.29391695Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:05:17.293736005Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:05:47.294273412Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:06:17.294381841Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:06:47.294403924Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:07:17.294397064Z","level":"INFO","msg":"get RDS metrics"}                                                                                │
│ {"time":"2024-03-26T12:07:47.294003217Z","level":"INFO","msg":"get RDS metrics"}

Great news!
We will adopt this approach for other parts of the exporter to make it more resistant.


What does rotation means in action_trace file ?

When I save action_trace file, there is column of rotation, what does it means? the rotation is in degrees or radians ? Looking forward for the reply !

I am also facing the same issues. "reply if you have solved this issue".

The rotation is the orientation of the agent encoded as a single angle in radius around the up axis (Y).

radius is unit of angle ? or it is something else ?

sorry, radians

Thanks @angelxuanchang for fast reply !

One last question @angelxuanchang, that is this angle is in range of “-pi to pi” or “-2pi to 2pi” ?

Thanks a lot for the response

Hi @angelxuanchang  I have seen value of rotation in action_trace  files and it variate for different files. How can one generalize it ?
What is its range as in some files value of rotation is -10.8471 and in some files it is 0.4913.
Also how to convert it in degree , if I use the well known formula to convert from radian to degree (1rad * 180/pi) then it gives value more than 360.
Kindly reply me as soon as possible, looking forward for your reply. Thanks in advance for this act of kindness.


Auto Download

Please add a auto Download on update

Please checkout #317 why it's chosen to not do this.
However, there is now a direct download link available, to allow people to make auto-update scripts.

These should work fine on the most recent versions of wget:
If you want it downloaded as ViaVersion.jar:
wget -O ViaVersion.jar http://myles.us/ViaVersion/latest.jar
If you want it downloaded as ViaVersion-X.Y.Z.jar:
wget http://myles.us/ViaVersion/latest.jar --content-disposition
You could use these in a start-up script if you wanted.


Wording improvements for the matplotlib course

Some improvements to the words for the matplotlib course. It is hoped that these changes will make the course more useful as a read resource as well as a taught resource.
The biggest changes are the changes I've made to the wording of the last section on savefig and backends.

I'm happy with merging this, though I've asked a few questions. If the discussion goes anywhere, then please consider submitting a subsequent PR. :+1:
Thanks @dkillick


Very different entries with same Simulant ID

What happens?
I generated two decennial census datasets -- one without any error configuration, and one with some error configured. Checked record pairs with the same simulant ID and found some pairs of records are very different but have the same simulant ID.
Ex.
278792205 278317917 robert f dhandapani 46 01221974 216 avenidadelmundo  detroit mi 48093  referenceperson male asian 2020
278792205 278737047 francisco e gvild 11 05262008 1791 walleghenyave  marysville mi 49504 h biologicalchild male multiracialorother 2020
To Reproduce
import pseudopeople as psp
config_typo = {
'decennial_census': {
'column_noise': {
'first_name': {
'make_typos': {
'cell_probability': 0.1,
},
},
'last_name': {
'make_typos': {
'cell_probability': 0.1,
},
},
},
},
}
census = psp.generate_decennial_census(source= path/to/psp/directory, seed=0, year=2020,state='MI')
df_typo = psp.generate_decennial_census( source=path/to/psp/directory, seed=0, year=2020,state='MI', config=config_typo)
OS:
Red Hat Enterprise Linux 8.7 (Ootpa)
Pseudopeople version:
0.8.3

@joyantabasak13 278792205 doesn't look like a simulant ID to me -- I believe simulant IDs always contain underscores. Can you include the code you used to compare census and df_typo?

I'm going to close this for now, but if you are still having trouble then please re-open and send the full code to you used to compare census and df_typo.


[Feature-7777][UI] process definition page add create username

Purpose of the pull request
This pr will close #7777

Codecov Report

Merging #7778 (25bdafe) into dev (fa906e7) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #7778      +/-   ##
============================================
- Coverage     40.95%   40.84%   -0.12%
+ Complexity     3671     3667       -4
============================================
  Files           642      642
  Lines         26707    26740      +33
  Branches       2997     3006       +9
============================================
- Hits          10939    10922      -17
- Misses        14716    14766      +50
  Partials       1052     1052

Impacted Files
Coverage Δ

...he/dolphinscheduler/common/enums/SqoopJobType.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.21% <0.00%> (-8.04%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (-2.00%)
:arrow_down:

...api/service/impl/ProcessDefinitionServiceImpl.java
25.70% <0.00%> (-0.53%)
:arrow_down:

...lphinscheduler/service/process/ProcessService.java
35.31% <0.00%> (-0.28%)
:arrow_down:

...r/api/service/impl/ProcessInstanceServiceImpl.java
60.56% <0.00%> (-0.18%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
53.52% <0.00%> (+0.70%)
:arrow_up:

Continue to review full report at Codecov.

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


WebView2 does not allow to enter a 't'-character in a text input

Describe the bug
WebView2 does not show a 't' character entered in a text input in a web page.
Steps to reproduce the bug

Navigate to a web site with a text input (e.g. google.com)
Try to enter text containing the 't'-character.
The 't' will not appear

The capital 'T' is allowed.
Expected behavior
I want to be able to enter a 't'
Version Info
NuGet package version:
Microsoft.ProjectReunion 0.5.0

Windows 10 version
Saw the problem?

May 2020 Update (19041)
Yes

Additional context
I am using the WebView2 control from a C# WinUI/Reunion Desktop project.

I can't reproduce this issue in a simpler app. I need to investigate more to see what the exact context is.

The issue is that WebView2 will emulate the F5 key as soon as the 't' character is pressed.  I will close this issue and create a new one.


Custom Provider Request: IBM's watsonx.ai

Hi,
It would be great if ChainForge could support IBM's new watsonx.ai platform. It's their big bet on generative AI for trustworthy enterprise and allows usage of both IBM's LLMs and curated open-source models.
https://www.ibm.com/products/watsonx-ai
I've attached a link to the free version of the software https://dataplatform.cloud.ibm.com/registration/stepone%3Fcontext%3Dwx.
As well as their python library https://ibm.github.io/watsonx-ai-python-sdk/foundation_models.html and documentation https://dataplatform.cloud.ibm.com/docs/content/wsj/getting-started/welcome-main.html?context=wx&audience=wdp to hopefully make it a much easier task.
Best regards and thank you for making such an amazing tool!

Hi there, it's a good request, but not something I can personally spend time on. I suggest IBM (or anyone interested in this support) implement a Pull Request. We have recent PRs from the community to support Amazon Bedrock and together.ai, which I am very thankful for.


⚠ Bofrost.at has degraded performance

In 2c28af2, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1053 ms

Resolved: Bofrost.at performance has improved in b7d7f04.


can't submit from certain browsers

I'm going to preface this by saying it may not be the browser, but my workplace's firewall, but I have no way of testing that. However, I don't know why the firewall would let me access the site and run tests, but not submit final code.
This issue appears when using IE 11. Before anyone shoots me, let me say I would certainly prefer to use Chrome or Firefox, but this is a Microsoft shop through and through, and black-suited men would come to cart me away if I attempted to install a more modern browser.
The issue: I click "Submit" and it fails immediately with a message flashing in red that it "failed to send to server." It flashes once, very quickly, then is gone. If this is a problem on my end, I apologize for submitting a bogus issue.

I am using Firefox and have same problem: Submission timed out. Please try again.
The problem exists with Run Tests too.

@JelenaSparic That's a completely other failure, see #18 and the wiki page on submission timeouts

Has this ever worked for you? It is odd that it works for tests and not submissions. This is likely an issue with IE since we don't really test on it often (less than 1% of our users are using it)

It's never worked for me while using my work computer. That said, I'll give it a test on my home PC sometime. I can submit while at work on my personal laptop, using both Firefox and Chrome. I wouldn't be shocked if it were an IE issue. Unfortunately, someone important has to retire before they'll let us use better browsers.

IIRC I could both run and submit code with IE (in metro mode), so it might
be a workspace specific issue.

When I type www.codewars.com as url I get a DNS resolution error. I can only access the site if I google it and click the link from the Google results. And then it won't let me submit the first problem so I can sign up. Tried it on Chrome and Mozilla, and on two separate computers, and it does the same thing. Turned anti-virus (Avast) off, but still not "connecting to server".

@jhoffner well maybe it is the other way around - users just can't use code wars from IE, that is the reason for so small usage number. I just tried to sign up after submitting correct solution to the sign up test - it showed "Saving progress and loading next step" and just returned to the same sign up test. Tried this both with Edge and IE. Does this really mean that users are forced to install Chrome or Firefox in order to use Code wars? I hope this will be fixed...

@prokurors the landing page issue wasn't an IE specific issue. That should be resolved. IE is still not the most ideal browser to use on CW but it should work. According to our logs we seem to have users doing kata with IE.
The 1% (actually 1.5%) usage number came from our landing page, which means only 1.5% of new users trying to signup are on IE.


paru: init at 2.0.3

Description of changes
Paru is your standard pacman wrapping AUR helper with lots of features and minimal interaction.
https://github.com/Morganamilo/paru
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

1 package built:
paru
Result of nixpkgs-review pr 328180 run on x86_64-linux 1

1 package built:

paru


Fix state of wrapped layer metrics with unbatch and decode with errors function

With this PR, the state of the metric WrappedLayerMetricsWithUnbatchAndDecodeWithErrorsFunction, especially the error counts, are now correctly handled.
This also adds the abstract metric class MetricWithArbitraryCounts that provides the methods inc_counts() and get_counts() to easily handle counts lazily for arbitrary keys. This was outsourced from PrecisionRecallAndF1ForLabeledAnnotations.

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (6cfc5bc) 95.82% compared to head (cf77d33) 95.80%.

Files
Patch %
Lines

src/pie_modules/taskmodules/metrics/common.py
96.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   95.82%   95.80%   -0.02%
==========================================
  Files          46       47       +1
  Lines        3500     3508       +8
==========================================
+ Hits         3354     3361       +7
- Misses        146      147       +1

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


Master

Hi!
This pull request changes xrdp.ini file interpretation a little bit, so log can be redirected to stdout:
LogFile=-

This change is usefil for xrdp debugging and doesn't affect its normal operations

Ooops. Something went wrong, sorry!


rename isnull and isnan methods

as discussed on the last call

I don't remember what the rationale for this was, can you add one @MarcoGorelli? These methods are using the no-underscore flavor pretty much universally I think, and also in the array API for isnan.

just snake-case - you'd read it as "is null", and replacing the space with an underscore, would get "is_null"

Hmm - I'd think that that's probably not a good reason for is and has, they're kinda special in Python (isinstance, issubclass, hasattr, etc.) - that plus isnan/isnull being the current names everywhere means we should probably leave this as is.

everywhere

Just to be pedantic, but not "everywhere": for example pyarrow and polars both use "is_null" etc (but yeah, those are certainly the outliers and probably less relevant for the exercise here)


Karma cannot start Chrome version 108.0.5359.125

After Chrome auto updated to version 108.0.5359.125, karma can no longer start chrome. It gives the below output and then tries to open chrome 2 more times. Each time it will open karma in a new chrome tab (not a window) but will never connect to it and won't actually run the tests. Both ChromeHeadless and Firefox seem to run fine which makes me think it has something to do with the latest Chrome version.
19 12 2022 11:20:58.275:INFO [karma-server]: Karma v6.3.20 server started at http://localhost:9876/
19 12 2022 11:20:58.275:INFO [launcher]: Launching browsers Chrome with concurrency unlimited
19 12 2022 11:20:58.285:INFO [launcher]: Starting browser Chrome
19 12 2022 11:20:58.287:DEBUG [launcher]: null -> BEING_CAPTURED
19 12 2022 11:20:58.289:DEBUG [temp-dir]: Creating temp dir at C:\Users\chrism4\AppData\Local\Temp\karma-77373876
19 12 2022 11:20:58.291:DEBUG [launcher]: C:\Program Files\Google\Chrome\Application\chrome.exe --user-data-dir=C:\Users\chrism4\AppData\Local\Temp\karma-77373876 --enable-automation --no-default-browser-check --no-first-run --disable-default-apps --disable-popup-blocking --disable-translate --disable-background-timer-throttling --disable-renderer-backgrounding --disable-device-discovery-notifications http://localhost:9876/?id=77373876
- Generating browser application bundles (phase: building)...19 12 2022 11:21:00.449:DEBUG [launcher]: Process Chrome exited with code 0 and signal null
19 12 2022 11:21:00.451:ERROR [launcher]: Cannot start Chrome

19 12 2022 11:21:00.455:ERROR [launcher]: Chrome stdout: Opening in existing browser session.

19 12 2022 11:21:00.456:ERROR [launcher]: Chrome stderr:
19 12 2022 11:21:00.456:DEBUG [temp-dir]: Cleaning temp dir C:\Users\chrism4\AppData\Local\Temp\karma-77373876
| Generating browser application bundles (phase: building)...19 12 2022 11:21:00.916:INFO [launcher]: Trying to start Chrome again (1/2).
19 12 2022 11:21:00.918:DEBUG [launcher]: BEING_CAPTURED -> RESTARTING
19 12 2022 11:21:00.921:DEBUG [launcher]: RESTARTING -> FINISHED
19 12 2022 11:21:00.922:DEBUG [launcher]: Restarting Chrome
19 12 2022 11:21:00.923:DEBUG [launcher]: FINISHED -> BEING_CAPTURED
19 12 2022 11:21:00.923:DEBUG [temp-dir]: Creating temp dir at C:\Users\chrism4\AppData\Local\Temp\karma-77373876
19 12 2022 11:21:00.925:DEBUG [launcher]: C:\Program Files\Google\Chrome\Application\chrome.exe --user-data-dir=C:\Users\chrism4\AppData\Local\Temp\karma-77373876 --enable-automation --no-default-browser-check --no-first-run --disable-default-apps --disable-popup-blocking --disable-translate --disable-background-timer-throttling --disable-renderer-backgrounding --disable-device-discovery-notifications http://localhost:9876/?id=77373876
| Generating browser application bundles (phase: building)...19 12 2022 11:21:02.515:DEBUG [launcher]: Process Chrome exited with code 0 and signal null
19 12 2022 11:21:02.523:ERROR [launcher]: Cannot start Chrome

19 12 2022 11:21:02.530:ERROR [launcher]: Chrome stdout: Opening in existing browser session.

19 12 2022 11:21:02.531:ERROR [launcher]: Chrome stderr:
19 12 2022 11:21:02.534:DEBUG [temp-dir]: Cleaning temp dir C:\Users\chrism4\AppData\Local\Temp\karma-77373876
- Generating browser application bundles (phase: building)...19 12 2022 11:21:03.422:INFO [launcher]: Trying to start Chrome again (2/2).
19 12 2022 11:21:03.422:DEBUG [launcher]: BEING_CAPTURED -> RESTARTING
19 12 2022 11:21:03.423:DEBUG [launcher]: RESTARTING -> FINISHED
19 12 2022 11:21:03.423:DEBUG [launcher]: Restarting Chrome
19 12 2022 11:21:03.423:DEBUG [launcher]: FINISHED -> BEING_CAPTURED
19 12 2022 11:21:03.424:DEBUG [temp-dir]: Creating temp dir at C:\Users\chrism4\AppData\Local\Temp\karma-77373876
19 12 2022 11:21:03.425:DEBUG [launcher]: C:\Program Files\Google\Chrome\Application\chrome.exe --user-data-dir=C:\Users\chrism4\AppData\Local\Temp\karma-77373876 --enable-automation --no-default-browser-check --no-first-run --disable-default-apps --disable-popup-blocking --disable-translate --disable-background-timer-throttling --disable-renderer-backgrounding --disable-device-discovery-notifications http://localhost:9876/?id=77373876
/ Generating browser application bundles (phase: building)...19 12 2022 11:21:05.869:DEBUG [launcher]: Process Chrome exited with code 0 and signal null
19 12 2022 11:21:05.874:ERROR [launcher]: Cannot start Chrome

19 12 2022 11:21:05.877:ERROR [launcher]: Chrome stdout: Opening in existing browser session.

19 12 2022 11:21:05.878:ERROR [launcher]: Chrome stderr:
19 12 2022 11:21:05.879:DEBUG [temp-dir]: Cleaning temp dir C:\Users\chrism4\AppData\Local\Temp\karma-77373876
- Generating browser application bundles (phase: building)...19 12 2022 11:21:08.628:ERROR [launcher]: Chrome failed 2 times (cannot start). Giving up.
19 12 2022 11:21:08.628:DEBUG [launcher]: BEING_CAPTURED -> FINISHED

Environment Details

karma versions:
"karma": "~6.3.12",
"karma-chrome-launcher": "~3.1.1",
"karma-coverage": "~2.2.0",
"karma-firefox-launcher": "^2.1.2",
"karma-jasmine": "~4.0.0",
"karma-jasmine-html-reporter": "^1.5.0",
"karma-parallel": "^0.3.1",
"karma-sabarivka-reporter": "^3.2.0",
"karma-teamcity-reporter": "^1.1.0",
"karma-verbose-reporter": "0.0.6"

karma.conf.js:

// Karma configuration file, see link for more information
// https://karma-runner.github.io/1.0/config/configuration-file.html

module.exports = function (config) {
  config.set({
    basePath: '',
    frameworks: ['jasmine', '@angular-devkit/build-angular'],
    plugins: [
      require('karma-jasmine'),
      require('karma-chrome-launcher'),
      require('karma-firefox-launcher'),
      require('karma-jasmine-html-reporter'),
      require('karma-coverage'),
      require('karma-teamcity-reporter'),
      require('@angular-devkit/build-angular/plugins/karma')
    ],
    client: {
      clearContext: false, // leave Jasmine Spec Runner output visible in browser
      args: {
        random: false
      }
    },
    customLaunchers: {
      ChromeHeadlessNoSandbox: {
        base: 'ChromeHeadless',
        flags: [
          '--no-sandbox',
          '--disable-gpu',
          '--disable-web-security'
        ]
      }
    },
    jasmineHtmlReporter: {
      suppressAll: true // removes the duplicated traces
    },
    coverageReporter: {
      dir: require('path').join(__dirname, './coverage/ngfnex-ui'),
      subdir: '.',
      fixWebpackSourcePaths: true,
      reporters: [
        { type: 'html' },
        { type: 'text-summary' }
      ],
      check: {
        emitWarning: true,
        global: {
          statements: 80,
          branches: 80,
          functions: 80,
          lines: 80
        }
      }
    },
    reporters: ['progress', 'kjhtml', 'teamcity'],
    port: 9876,
    colors: true,
    logLevel: config.LOG_DEBUG,
    autoWatch: true,
    browsers: ['Chrome'],
    singleRun: false,
    restartOnFileChange: true
  });
};

Steps to reproduce the behaviour

Update to chrome to latest version (108.0.5359.125)
Run ng test in angular project using karma

I also have this issue

@PigeonPanet have you been able to find a fix for this? I'm still seeing this issue in Chrome version 111.0.5563.111.

Same issue here on Chrome 111.0.5563.65

I finally figured this issue out! Somehow chrome got this policy https://chromeenterprise.google/policies/?policy=UserDataDir added that will ignore the --user-data-dir flag and use whatever is set in there. Not sure how it got there but might of been added through my organization. This caused the user data dir that karma was generating to be completely ignored and therefore not work.
To fix, I deleted the registry entry from here:

I did uninstall Chrome first and then reinstalled it after I deleted it, but I'm not sure if that's required.


scatter_max vjp + bindings + tests

Proposed changes
Implementation of scatter_max vjp after this discussion, along with tests and binding for mlx-graphs.
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

Just a check:
Do you confirm that when src and update are equal at index, the gradient should be propagated both in src and update?

Thanks for the VJP! That's awesome.
Regarding the binding, same comment re #391 we're using the indexing style syntax for scatter_* ops (#394). So you can do:
a.at[idx].max(val)
Would you mind removing the scatter_max binding from this?

Didn't see this PR on .at[] thanks @awni.
I really prefer this way of using scatter, more pythonic!

I chose to use the first way as it is imo more intuitive at first glance.
(Sorry for the wrong commit msg there was a problem with my branch)


Added missing properties attribute to Feature model

updated CONTRIBUTING.md
improved tests for Feature.properties
updated documentation and tests
added note about testing to readme
updated documentation and tests
added note about testing to readme
Pull Request Check List
Resolves: #32

[*] Added tests for changed code.
[*] Updated documentation for changed code.

Good point, thanks

Does anyone know what's the problem here with poetry installation for Python 3.7?


Single value of type multi-specimen/-object in export

What is the issue ?
The multi-specimen/object record usually come in pairs or larger groups. A single record with the MOS type name is suspicious.
Detailed description of the issue.
During post processing I discovered this 'anomaly' and alerted the project lead.
Export: NHMD Herba 20231018 16 43 MG.csv
Container name value: MSO2387626551144275769
Why is it needed/relevant ?
This issues reduces the data quality.
Explain the need, relevance.
It begs the question: Which other records belong to this group?
Estimate level of effort required.
easy
What could be the challenges ?
Going back to the collection and finding the specimen
Remarks
export:
NHMD_Herba_20231018_16_43_MG.csv

Should be linked to specimen 00936835 in same export: NHMD Herba 20231018 16 43 MG.csv
Image path: N:\SCI-SNM-DigitalCollections\DaSSCo\MASTER_IMAGE_STORE\JPEGs\WORKHERB0001\2023-10-31\7e7-a-12-10-28-04-0-001-00-000-093dbc-00000.jpeg

Seems like a workflow issue, where it wasn't noticed as an MSO until imaging and first barcode (00936835) was already assigned as a single specimen, when second (00937321) was added.

fixed in Specify by adding container ID MSO2387626551144275769 to specimen 00936835.


v5.4 removes Theater mode button

Script version
5.4
Describe the bug
When using the newest version, the button for Theater mode disappears. Using the hotkey (t) also doesn't work.
Desktop (please complete the following information):
Firefox on Windows 11

I believe an embed in the latest 5.4 version overlaps the actual video, causing these issues. For me, the 5.0 version is working fine. You can use the following link to downgrade the script:
https://github.com/TheRealJoelmatic/RemoveAdblockThing/releases/download/v5.0/Youtube-Ad-blocker-Reminder-Remover.user.js

@Krishn-G thx man


"This application was configured to use a bundled Java Runtime Environment but the runtime is missing or corrupted."

Hi. I tried running FoDUpload.win.exe from https://github.com/fod-dev/fod-uploader-java/releases/tag/v3.0.8  but got an error

This application was configured to use a bundled Java Runtime Environment but the runtime is missing or corrupted.

What have I missed?

You've missed nothing, that's a bad bundling.  I need to get the build working that was generating those again and upload the latest version exe.

Is there any latest version available with above issue fix?
Still getting the error as below,

This application was configured to use a bundled Java Runtime Environment but the runtime is missing or corrupted.

I have the same issues wihen trying to use My Expat Network application. Need Help.


Test flake: SparkSubmitSuite and HiveSparkSubmitSuite

https://circleci.com/gh/palantir/spark/573#artifacts/containers/0

SparkSubmitSuite:

[info] - include an external JAR in SparkR *** FAILED *** (5 seconds, 368 milliseconds)

RecurringTimerSuite:

[info] - SPARK-10224: call 'callback' after stopping *** FAILED *** (10 milliseconds)

HiveSparkSubmitSuite:

[info] - SPARK-9757 Persist Parquet relation with decimal column *** FAILED *** (5 minutes, 0 seconds)

https://circleci.com/gh/palantir/spark/576

org.apache.spark.deploy.SparkSubmitSuite

includes jars passed in through --packages

@robert3005 how do you feel about ignoring all of SparkSubmitSuite and HiveSparkSubmitSuite since they're flakey?  I think the main risk is that a regression in normal spark submission could be introduced by the kubernetes work since that's in the realm of the submission process.
@mccheah for thoughts on whether we should root cause (Hive|)SparkSubmitSuite flakes or stop running the tests for now.

Strongly against disabling these. We mess with the submission changes too much for this to not backfire. #115 was one of such changes.

So just live with the flakes until we can't stand them anymore?

First person to get annoyed gets to fix it. If you disable those tests we will likely merge something we shouldn't have.

More concretely #130 causes a break in one of those tests and probably affects sparkr submit process. Given that it's possible to break these with packaging changes I would like to keep them on. I haven't seen them flake since we refactored the test setup.


[poll] control or baseline

I think it's time to figure out what description we are going to use in all the openproblems repos:
:+1: for control method
:-1: for baseline method
@rcannood @scottgigante-immunai @mumichae @mxposed @dburkhardt

Haha. Wouldn't call this an urgent issue, but it definitely needs to be resolved :)
I like being able to talk about positive controls and negative controls, so a :+1: from me.
Reminds me about a conversation with @slobentanzer about defining an ontology for all of the terms and concepts we use to talk about OpenProblems. I think that'd be a great idea.

I know its not urgent, just a bit of fun on a saturday night :rofl:

Still think that would be a cool thing to have. If you don't want to go full-blown semantic web immediately, we could also make drafts using the BioCypher framework (just defining some concepts in the configuration). Happy to brainstorm on that.
For instance, we could start by manually extending some subclass of EFO Information Entity.

I like being able to talk about positive controls and negative controls

Only concern about the term "control" is that in some cases (e.g. batch integration), a single baseline method can be a positive control for one metric, a negative control for another, and be totally irrelevant for a third.

I'm not familiar with EFO but control would probably be a relationship between a baseline method and a metric -- so a baseline method is a positive control for a given metric.


MM-46055: Fix toast position after copying link in ChannelInfo screen

Summary

This was a request by @matthewbirtch in https://github.com/mattermost/mattermost-mobile/pull/6502#issuecomment-1197355111
@enahum we discussed ways to do this, and you mentioned that we could customize the snackbar to override positioning (https://github.com/mattermost/mattermost-mobile/pull/6502#issuecomment-1197722545). I wasn't sure what you meant exactly, so I took a stab at this method. We're already using the sourceScreen prop to customize positioning and styles for tablet, so I thought this would be the simplest way to customize positioning when the snackBar is in the ChannelInfo screen. But let me know if you had another idea in mind -- I'm 0/5 on the eventual solution, so open to whatever you think is best.

Ticket Link

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

Checklist

[ ] Added or updated unit tests (required for all new features)
[x] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information

Android: 13, Pixel 6
iOS: 15.7, iPhone 7 plus

Screenshots
Originally:

This PR:

Release Note
NONE

Thanks @matthewbirtch and @enahum, here it is:
iPhone 13 with 34 from the bottom

iPhone SE1 with 8 from the bottom (that is indeed 8 from the bottom, even though it doesn't look like it -- the screen is very small):

Thanks @matthewbirtch and @enahum, here it is:
iPhone 13 with 34 from the bottom
iPhone SE1 with 8 from the bottom (that is indeed 8 from the bottom, even though it doesn't look like it -- the screen is very small):

Hmm, still doesn't look right to me. There must be some extra space being accounted for because 8 is the margin on the left and right of the banner, but it's a lot more space there on the bottom. In both the iPhone 13 and iPhone SE it looks like there is extra space somehow getting factored in.
It should looks like this:

iPhone 13
iPhone SE

Adding @avinashlng1080 as a reviewer as he knows the impact of this a bit more than me

Looks like the toast positioning needs to work on iPad:
Landscape

Portrait

@enahum  and @cpoile  - I had a look at this PR and refactored the snack_bar styles a bit.  I ran it through tablets, iPhone SE and iPhone 14 and Android Pixel 3a and they all displayed well.


The interface becomes unresponsive after mousewheel up to the first (or last) line

To reproduce:

open or create an empty document with vim in gnome-terminal (I can not try with other terminals)
while the cursor is already on the first line, try to scroll up with the mouse wheel a few times

Now you can continue to drink your coffee, start to learn how to use EMACS (or whatever) before the interface becomes responsive again.
Looks like mouse wheel events are converted to cursor key events and buffered.
It would be a great user experience if those mouse wheel events were ignored when the cursor is already at the top/bottom of the document ;-)
I could not find a way to implement it right away, hope somebody does.

Rurik Bugdanov wrote:

To reproduce:

open or create an empty document with vim in gnome-terminal (I can not try with other terminals)
while the cursor is already on the first line, try to scroll up with the mouse wheel a few times

Now you can continue to drink your coffee, start to learn how to use EMACS (or whatever) before the interface becomes responsive again.
Looks like mouse wheel events are converted to cursor key events and buffered.
It would be a great user experience if those mouse wheel events were ignored when the cursor is already at the top/bottom of the document ;-)
I could not find a way to implement it right away, hope somebody does.

I cannot reproduce the problem, works fine for me.  Tried xterm and
gnome-terminal 3.20.2
--
Portable Computer:  A device invented to force businessmen
to work at home, on vacation, and on business trips.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Oh, it's only when vim is run within a screen session (eg with "screen vim ...").
So it looks more a "screen" related issue.
I never had the idea to test without "screen" before your answer (I use a script named "vi" to run screen transparently, and to re-attach automatically to an existing vim session)
Thanks.


Update F-Droid

The latest version available on F-Droid is 3.6 or so. Please update.

Hi @stevenroose, the F-Droid repo is not maintained by me. You can request to include/update qBittorrent-Controller here https://f-droid.org/forums/forum/submission-queue/


[MixItUp MultiFilter] Not Working on iOS

Hello Everyone,
Just implemented MixItUp Multifilter on my project and it works like a charm on desktop browsers :
https://www.masterthehandpan.com/mix-it-up-handpan-makers-directory
BUT it does NOT work on iOS Browsers.
Attached 2 screen shots where you'll see the errors I get on Safari or Chrome

Safari : This web page was reloaded because an error occured
Chrome : Something went wrong...

Any idea of what's going on ?
Any help would be really appreciated
Thank you

Hi, thanks for the info.
Does this happen when you first load the page, or after some specific interaction?

No, right after loading the page.
Note : this bug doesn’t appear on android phones

Any ideas ? Help :)

I'm afraid I've been unable to recreate the issue on any device (including iOS safari and chrome).
Are you certain that the issue is related to MixItUp?

Have you tested the multifilter demos on your ios device? If so, do you see the same error when loading the page?
Have you tried removing MixItUp from your page, and observing whether the error still occurs?

Thanks Patrick for the follow up.
So I transferred the test page to a blank one in order to make sure there would be no interaction with any other element.
Here is the URL : https://www.masterthehandpan.com/mixitup
To answer your questions :

removing MixitUp makes the page functional with no errors
yes, testing it right into my iPhone directly ( on a desktop browser with mobile view, it's working great)

But from what I can see, the error is not always displaying.
Sometimes filters are doing their job as expected, sometimes it makes the page crash. Tested on different select fields to see if it was related to a certain select tags but it seems not. Sometimes a select field will work, sometimes not.
This could lead to think it's a hosting problem... but why would it be problematic only on iOS, and NEVER occur on desktop anyway...

This might be unrelated but I was looking at your JS:

And I can see you're using some functionality from the select filtering demo for mixitup core. You actually don't need this if you're using the multifilter extension, as all of this logic and binding is done internally. You would still need the "sorting" part of it if you had a sort select, but duplicating filtering part could cause conflicts.
Also - even in your stripped down page, looking at the JS network tab - you still have a lot of other scripts loaded and other functionality present:

It might be a good idea to create a simple static html page with just your mixitup functionality on it and test against that - or maybe use one of the demos as I suggested before. Checkout /demos/selects/ in the multifilter package. If that works fine, then start adding in other functionality until we can be sure exactly what is causing the page to crash.

Ah great - that's really helpful. I can't promise anything - but I'm keen to identify whether or not this is an issue with MixItUp or your specific integration, I just need to find a device where I can reproduce it first. This is a great starting point though.
Could you let me know the model of your iPhone and the version of iOS you're running?

This would be easier to debug if there was an actual error thrown that we could see in the console, but this kind of "crashing" behaviour where the browser actulaly gives up and has to terminate a tab is usually an indication of a long running script, accidental infinite loop, or memory leak. If you are able to run your "nojs" page without crashing, then that would indicate that mixitup is not the issue, but rather something in your custom javasript (your integration), or something that webflow is adding. It is more likely to be the former.
In terms of why its only crashing on iOS, I don't really have an idea other than that iOS might be more aggresive about terminating a tab due to the types of issues I described above.
Also, could it be possible that now that you've fixed it on the "nojs" page, that the issue might just be that webflow still has the old version cached somehow? Are you able to determine that webflow has picked up your latest changes?

I thought I'd look further in to the memory leak issue by running the memory profiler in Chrome dev tools. I didn't find anything critical - but just to show you, the comparison:
In you're standard webflow setup, you have a lot of stuff going on on the page, and a few big initial spikes that could cause a browser to crash. In total the page is using about 7MB of memory:

I've drawn a box around where MixItUp sits relative to everything else on your page (it's using about 180KB in total).
I then ran another profile on your "nojs" page, where only mixitup is present:
The page is using only 1.6MB of memory now, and there are no spikes or long call stacks that could cause a crash. In case you're wondering the other stuff is just the normal background memory usage of V8.

Based on this, my best bet is that your issue is a memory leak caused by one of these big spikes shown on the first profile. The 3 worst offenders for memory usage are:

client.js (and something called Crisp internally?) Spiking at around 600KB
webflow.dccbef211.js, spking at around 500KB
addthis_widget, spiking at around 700KB

I'm not the best at interpreting these profiles, but I think its fair to say there are more serious issues on the page.
As I don't think this is related to MixItUp, I'm going to close this ticket now. Apologies I couldn't be of more help, but hopefully this can point you in the right direction of being able to optimise your page.


Error in the Stub static references "better" example?

Hi,
shouldn't the optimized example of the GetDiscountedPrice() Method use dateTimeProvider.DayOfWeek() instead of DateTime.Now?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 51947dc8-9ae8-21fa-13db-a7ffb3f9bc0e
Version Independent ID: be661bf5-6f6e-cd18-344b-253edf5751ae
Content: Best practices for writing unit tests
Content Source: docs/core/testing/unit-testing-best-practices.md
Product: dotnet-core
GitHub Login: @jpreese
Microsoft Alias: wiwagn

Thanks for reporting this @Kornelis  We appreciate it.  You are correct, the final version of the code under test should make use of the IDateTimeProvider object.
@jpreese Do you want to submit the fix, or should I?

Good catch @Kornelis! I've added a PR for this, @BillWagner.


UnknownErrorException: The reference does not point to a /Page dictionary

Describe the bug
Some PDF files generate an exception in PdfData.extract(dataBuffer); even when info and metadata are visible inside Acrobat Reader. The exception is:
Error: UnknownErrorException: The reference does not point to a /Page dictionary.
    at D:\Projects\AMDIdeaIgniterEnvironment\exp\node_modules\pdfjs-dist\legacy\build\pdf.js:11309:31
    at async getPageNumber (D:\Projects\AMDIdeaIgniterEnvironment\exp\node_modules\pdfdataextract\dist\pdfdataextractor.js:10:18)
    at async parseOutline (D:\Projects\AMDIdeaIgniterEnvironment\exp\node_modules\pdfdataextract\dist\pdfdataextractor.js:31:23)
    at async Function.extract (D:\Projects\AMDIdeaIgniterEnvironment\exp\node_modules\pdfdataextract\dist\pdfdata.js:52:378)

The code is the one from the README file.
Expected behavior
If info and metadata exists and are visible in Adobe Reader, should be accessible from the library.
Environment (please complete the following information):

OS: Windows 10 64bits
Node.js Version 16.3.0

Additional context
GitHub refuses to past here the pdf file and the screenshots, so here are under my Dropbox:
The PDF file
The info in acrobat
The metadata

BTW, calling directly PDF.js with the example here gives no error.

@crystalfp thank you for your bug report. i rewrite the outline parsing and it should now work.
If the problem is not solved or another problem arises, please contact me again.
The new version should be public on npm soon.


Adding Gemini and VertexAI tests

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

Project coverage is 41.87%. Comparing base (cf15ce4) to head (c941ec9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #93      +/-   ##
==========================================
+ Coverage   35.67%   41.87%   +6.19%
==========================================
  Files          38       38
  Lines        1962     1963       +1
==========================================
+ Hits          700      822     +122
+ Misses       1262     1141     -121

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


⚠ Jenkins Plugins Artifactory has degraded performance

In 7dd2635, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 2848 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 6c95ab3 after 24 minutes.


Module 3 exercises [Mananu Salam]

WS2_M3E1_Template.docx

Well done, Maman. You clearly grasped the material outlined in Module 3 and successfully applied it to these exercises. Thank you for your participation! Please be sure to fill out the post-Module survey before you begin the next Module.


CORS - cross origin request not working.

My .htaccess file
RewriteEngine On

Header set Access-Control-Allow-Origin *
Header set Access-Control-Allow-Credentials true

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ index.php [QSA,L]
When i try to access my api from other host or port, CORS issue is coming. Please any one help on this.

I'd like to implement something like this in the framework. It's meant to be an API framework and it's pretty common to do CORS stuff, esp with Javascript-y things.

Another link to get some ideas from to set up CORS with security. https://stackoverflow.com/questions/8719276/cross-origin-request-headerscors-with-php-headers

@fadrian06 @krmu One of you could easily do this one. I was thinking of maybe adding a class called Cors or something in the util/ folder and then you could add some of these headers via the Flight::app()->response->header('cors header', 'value'); or something. That would be nice cause then the headers wouldn't be set before the page is rendered or body is output. That'd be the way to go.

Added this example to the new security page in the docs. https://docs.flightphp.com/learn/security

Added this example to the new security page in the docs. https://docs.flightphp.com/learn/security

Hello,
I believe this example is not very good because it simply de-activates the CORS functionality.
I used to do that way before but some of my customers did penetration tests and found this as a vulnerability.
It is much cleaner to allow a whitelist and check if the HTTP_ORIGIN is one of the allowed values.
I gave an example of my new approach -which passes my customers' penetration tests- here:
https://github.com/flightphp/core/issues/486

Noted and fixed :) I included this one as an example.....but showing them proper examples is better. I was moving too fast and didn't think through it. My bad!


Refactoring validate() to handle resource data collections

Now validate() yields either a validated resource or a resource generator callable,
depending on the value of new kwarg collection.

Example use for relationship data collections:
        with self.validate(self.resource_class, collection=True) as resource_generator:
            tags = [resource.obj.name for resource in resource_generator()]
            self.object.tags.add(*tags)

@brosner sure would be nice to get this merged!

Changes Unknown when pulling 29830b276864b73a51ee306c2ee1b6b42e4dad36 on grahamu:all-purpose-validate into ** on pinax:master**.

Changes Unknown when pulling 4ba2a87d740daa0818efd1cbb7f6a10156818da3 on grahamu:all-purpose-validate into ** on pinax:master**.

Needs four types of tests.
Testing with other systems indicate this solution works for valid and invalid single resources, and valid resource collections. Invalid resource collections have not been tested. All four test types need to be included.

Changes Unknown when pulling d1339c1c8fef6547143d6e80fa377bb8da2517f8 on grahamu:all-purpose-validate into ** on pinax:master**.


ssh pubkey login not working

i have set my public key in configuration.nix like this:
users.extraUsers.root.openssh.authorizedKeys.keys = [ "ssh-rsa AAAA... davidak" ];
the options page says that the option is users.users.<name?>.openssh.authorizedKeys.keys but it also don't work.
if i try to login i get a password prompt:
$ ssh -v server
[...]
debug1: Authentications that can continue: publickey,password,keyboard-interactive
debug1: Next authentication method: publickey
debug1: Offering RSA public key: /Users/davidak/.ssh/id_rsa
debug1: Authentications that can continue: publickey,password,keyboard-interactive
debug1: Offering RSA public key: /Users/davidak/.ssh/id_rsa_git
debug1: Authentications that can continue: publickey,password,keyboard-interactive
debug1: Trying private key: /Users/davidak/.ssh/id_dsa
debug1: Next authentication method: keyboard-interactive
Password:

the key is in /etc/ssh/authorized_keys.d/root but in 3 lines, is that correct?
ssh-rsa
AAAA...
davidak

NixOS 15.09 but i had it also with 14.12.

the key is in /etc/ssh/authorized_keys.d/root but in 3 lines, is that correct?

That's likely the cause. At least ~/.ssh/known_hosts uses one like per key.

Are you sure that the value
"ssh-rsa AAAA... davidak"

is given in one line in your configuration.nix?

Are you sure that the value is given in one line in your configuration.nix?

no :D
i pasted the config again from my editor over ssh and now it works.
maybe it was due to some paste error before the nixos-install.
i installed another vm yesterday where it works directly.

:-)


feat: Implement wait for ctl commands

Fixes #91.
In this PR I am implementing a way for all ctl commands to wait for the outcome of the command, instead of just returning after acknowledgement. This will modify all of these commands to wait by default, with a --skip-wait flag that can be passed to mimic the current behavior.
So far I've done this for ctl start actor. I'd like to hear feedback on this implementation before I go on to the rest of the commands.
Signed-off-by: Matt Wilkinson<EMAIL_ADDRESS>

@brooksmtownsend I've made all of the changes from your feedback. Could you take a look when you have a bit? Thanks!

I've finished adding the wait parameters to the commands that can support them right now (that have CloudEvents being sent over the wasmbus.evt.* channel), and rebased to pick up the spinner changes. However the CI is failing on a weird fmt error:
Warning: can't set `imports_granularity = Crate`, unstable features are only available in nightly channel.

Is this new? Can we just turn that off in the rustfmt.toml?

@brooksmtownsend this is ready for review

Last thing - is there a reason we disable the help flag on these ctl commands? Seems rather user unfriendly. I see that there's a few flags with a -h short option, could we remove those and enable all of the helps?
Can do this in another PR though

Last thing - is there a reason we disable the help flag on these ctl commands? Seems rather user unfriendly. I see that there's a few flags with a -h short option, could we remove those and enable all of the helps?

I wasn't aware we did this... we can keep that separate from this PR but every command should support -h and --help

All feedback is resolved @brooksmtownsend @stevelr. Should be ready to merge

Also - the tests didn't run on my last commit? Is there something weird going on there? It only checked the signoff

LGTM


Issue with CRUD Create/Add button

There's an issue with the clickable area of create/add button generated by Grocery CRUD as it has been hard to click on it. It does not work sometimes and I need to click several times to get it working. It's strange as others buttons (export, print, filter, etc.) are working fine.

When you click one time, it doesn't do anything?
When mouse-over does it detect that it's a button?

Hi,
It does click but after many attempts and when you kind of click on the
"right spot", which is very strange.
Regards,
2017-05-02 15:26 GMT-03:00 Jay Paul Torres Bandas<EMAIL_ADDRESS>:

When you click one time, it doesn't do anything?
When mouse-over does it detect that it's a button?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waifung0207/ci_bootstrap_3/issues/108#issuecomment-298720135,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKsXkeSZQvplfEWYs7hc_Ggu2GqQtESGks5r13VygaJpZM4MzVUC
.

Can you show me a screenshot?
Did you try inspecting the code on the button to see if there's a broken/missing line?

A screenshot won't show you what you're looking for, it's a behavior
issue...
After inspecting the code, everything seems to be in place...
2017-05-02 16:25 GMT-03:00 Jay Paul Torres Bandas<EMAIL_ADDRESS>:

Can you show me a screenshot?
Did you try inspecting the code on the button to see if there's a
broken/missing line?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/waifung0207/ci_bootstrap_3/issues/108#issuecomment-298735680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKsXkeMChssG-eNmO6ppRil352eZdoh0ks5r14NGgaJpZM4MzVUC
.


Push image tag

After #29 tags started being used but are never pushed.
#43 fixes if tags are done before push

thank you, I've merged your PR


[Bug][Conversion Algorithm] conversion failing for a specific template with error in the API being "Incorrect Dynamic Bindings"

Import Applicant Tracker-test template

On further checking, seems like this is happening because of NULL bindings in widget, remove all NULL bindings in the app, and the conversion worked

NOT FIXED @rahulramesha


Added Mousetrap.bindTo helper for event targeting

Added Mousetrap.bindTo helper which allows us to specify an element to listen for events on; the events then bubble to the document element where Mousetrap is ready and waiting for the event. We do a quick check on e.target and e.relatedTarget before invoking the callback.
However, although this helper works, if you specify multiple event listeners with different elements — yet using the same keys — then the callbacks are of course overwritten, which is the default behaviour of Mousetrap and it doesn't appear as though you're able to modify this behaviour.
Therefore I wouldn't mind some help in figuring out how to allow multiple events on the same key when you're using the event with an element context :+1: Thanks...

@Wildhoney thanks for this.  I think after I merge #253 it will no longer be necessary though.  It is actually something I have wanted to have for a while I just have not had the time.

However, although this helper works, if you specify multiple event listeners with different elements — yet using the same keys — then the callbacks are of course overwritten, which is the default behaviour of Mousetrap and it doesn't appear as though you're able to modify this behaviour.

With the changes I have made this will no longer be true since you can attach the same keys to different elements and handle them differently.

Thanks! It's good to see this feature pending. It will be terribly useful.

No longer pending!  Released it yesterday.

Even better!


Adminer Warning: Trying to access array offset on null in /usr/share/adminer/latest.php on line 1210 && ... on line 1406

Describe the bug
Accessing Adminer and trying to search something via Search data in tables show pages worth of warning. I installed showpare/dev-tools from composer and the whole image is in dev mode
To Reproduce
Steps to reproduce the behavior:
Access adminer
search for a table "cms_" in my case
Expected behavior
Should not show warning
Screenshots

Desktop (please complete the following information):

OS: Win 11
Browser: Chrome
Version  125.0.6422.144

Additional context
It might be just a php setting to show warning but I just wanted to point it out
Thanks for the time

Is there any update yet? Don't want to install phpMyAdmin every time I create a dockware instance...

There is adminerevo, a well maintained fork of the original adminer, which has not been updated for a long time.
I can create a PR to replace the dependency if desired. Let me know.

I second this. adminerevo would be great.

We have a local make file (in combination with an sh file) that replaces the adminer with the adminerevo for our local development systems.
It works like a charme.
we even use some nice plugins, like faster table filters...

We have a local make file (in combination with an sh file) that replaces the adminer with the adminerevo for our local development systems.
It works like a charme.
we even use some nice plugins, like faster table filters...

Yeah, I overwrite the adminer folder with adminerevo on initial setup.


Giving image as input in Gridworld

Hi,
Is it possible to give image as input in Gridworld environment. Can you suggest ways in which this can be done? Are there ways of converting tkinter Canvas into a numpy array which can then be fed into a ConvNet?
Thanks,
Akilesh

Hi!!
You can use ImageGrab function in PIL. I know only this way to capture image from tkinter canvas.
These links are helpful.
Get image from canvas -> https://stackoverflow.com/questions/41940945/saving-canvas-from-tkinter-to-file
Capture size doesn't fit -> https://stackoverflow.com/questions/24370228/python-imaging-library-fails-to-grab-whole-screen


Consider allowing the user to choose dot_similarity dtype

I've been using this library for a while to deploy HDC models with torchscript and one of my requirements is to carefully choose the underlying torch operations used. I noticed that dot_similarity is always executed using torch.get_default_dtype(), which is always float.  I managed to solve my issue in a fork by allowing the user to choose the dtype used in dot_similarity of MAP vectors. This change impacts the processor instructions executed when running the models and also their accuracy.
I've been thinking about merging such changes back to the library but they require API changes. For instance, the cosine_similarity would also be changed since it relies on dot_similarity.

Thank you for submitting this feature request. This seems like a useful feature so I am working on adding it (hopefully later today). The reason it currently uses the default dtype is to prevent counter overflows when the dtype of the VSATensor is torch.int8 for example.


upgrade h2database version 1.4.200 to 2.1.214

What is the purpose of the change
Fix security issues related to robot automatic submission regarding potential vulnerabilities in h2database
Related issues/PRs
Related issues: https://github.com/apache/linkis/issues/4696
Related pr: https://github.com/apache/linkis/pull/4697

Please handle the compilation failure.

https://www.baeldung.com/spring-boot-h2-jdbcsqlsyntaxerrorexception-expected-identifier
need to fix JdbcSQLSyntaxErrorException expected “identifier”

https://www.baeldung.com/spring-boot-h2-jdbcsqlsyntaxerrorexception-expected-identifier need to fix JdbcSQLSyntaxErrorException expected “identifier”

done


cannot use pyfakefs and tempfile at the same time

I am not sure if this is actually an issue or it is something that I do not understand. So I have a module that makes use of tempfile to create temporary directory and manipulate files in the directory. When I test with out pyfakefs, things are OK and tests are OK as well. But I when I load pyfakefs in another test file that will be run before this test file, things will break -- temporary dir will not be created, and writing to files in temprorary dir will fail of course.
I am using Python 3.6.1. Ubuntu 16.04.

        If ignore_errors is set, errors are ignored; otherwise, if onerror
        is set, it is called to handle the error with arguments (func,
        path, exc_info) where func is platform and implementation dependent;
        path is the argument to that function that caused it to fail; and
        exc_info is a tuple returned by sys.exc_info().  If ignore_errors
        is false and onerror is None, an exception is raised.

        """
        if ignore_errors:
            def onerror(*args):
                pass
        elif onerror is None:
            def onerror(*args):
                raise
        if _use_fd_functions:
            # While the unsafe rmtree works fine on bytes, the fd based does not.
            if isinstance(path, bytes):
                path = os.fsdecode(path)
            # Note: To guard against symlink races, we use the standard
            # lstat()/open()/fstat() trick.
            try:
>               orig_st = os.lstat(path)
E               FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmpoepn6_lx'

../.venv36/lib/python3.6/shutil.py:469: FileNotFoundError
==================================================================================== 1 failed, 1 passed in 0.19 seconds ====================================================================================

Could you please test with pyfakefs 3.4.1? There has been a similar issue  that has been fixed in this release.

I just upgraded. tests are OK now. I think can close this issue. Thanks a lot for the help


Redesign server installation screen

Fixes: #146
Old screen

New screen

lgtm 👍

Pretty! 😍

Oh and it looks really awesome!

I might make sense to fill the progress bar until the beginning of the active step btw. Now the "in progress step" already looks completed.

@joshuajansen Yeah I'm a bit in doubt about the progress bar xD Let me take a look at that :D

@joshuajansen I changed it to only fill the bar when a step is actually finished. Feels way better indeed :-)

👏

@jvanbaarsen wow that screen looks awesome


chore(src): adjust examples to work with Camunda Cloud

Changes related to https://github.com/camunda-cloud/camunda-cloud-documentation/issues/89
Change set:

Adjusts Readme to explain variant with Camunda Cloud and with local deployment
Adjusts the examples to work with either a Camunda Cloud cluster or with a local deployment

Merge should wait until new documentation is released. Otherwise, some links are not yet available.

@MiguelPires Since indentation seems to be important I added an https://editorconfig.org/ file. Not 100% sure on the settings though. Please have a look whether the settings are compliant with Go style.

@MiguelPires Since indentation seems to be important I added an https://editorconfig.org/ file. Not 100% sure on the settings though. Please have a look whether the settings are compliant with Go style.

You can just use gofmt to format the code according to the language's standards. I should've done that instead of eyeballing it in the Github suggestions, my bad. Also, did you run the examples?

You can just use gofmt to format the code according to the language's standards. I should've done that instead of eyeballing it in the Github suggestions, my bad. Also, did you run the examples?

I did run the examples with a local server

I did run the examples with a local server

Should we remove .editorconfig then if there is a dedicated tool in the Go ecosystem?

Should we remove .editorconfig then if there is a dedicated tool in the Go ecosystem?

Should we remove .editorconfig then if there is a dedicated tool in the Go ecosystem?

Sure, you can do that and also squash so it's ready to merge

Should we remove .editorconfig then if there is a dedicated tool in the Go ecosystem?

Sure, you can do that and also squash so it's ready to merge


[EventHub] update timeout error handling in test

related to #22961

API change check for azure-eventhub
API changes are not detected in this pull request for azure-eventhub


Support Extensions

Making this as a parent tracking issue for support for extensions, which are a way to define and share code that's re-usable across any Dagger SDK.
This goal was a fundamental part of the design of Dagger v0.3. We decided to start out simpler and just release the core API to start, but the original full vision of extensions are that they:

Let you write code in the language of your choice which is then automatically converted into a graphql API

This code will itself have access to the Dagger API in addition to being able to run anything else you'd normally import in your language.

Clients that want to use a set of extensions can generate an SDK which includes both the core API plus the APIs provided by the extension they depend on.

The end result is that you can write code once and then use it from any SDK in any language as easy as you would call the core API today. Extensions would also benefit from the same caching you get in the core API too.
We still have (hidden, undocumented) support for converting code->graphql schema in Go and Python. I believe our newer client codegen needs some updates to integrate with that but you can call extensions written in those languages with raw graphql today.

The details of how this will look and how we roll it out are still very much up in the air. A few selected open questions:

Should we support truly extending any part of the API (i.e. adding methods to core types like Container) or should we force each extension to be more like a "module" where it can define its own types, use types from core and other extensions, but not add methods to those "external" types.

Being more like a module may let us treat extensions more naturally as libraries, especially in languages that don't have the equivalent of "type-classes" (e.g. Traits in Rust)
See also https://github.com/dagger/dagger/issues/3386

Should we start simpler by just enabling users to provide implementations of pre-defined interfaces like "secret provider" or the With(...) mapper idea?

This may be easier to get going with as it temporarily avoids some of the hard questions around client codegen.

The idea here also extends into support for calling extensions from things we don't really consider an SDK today, such as calling your extensions from the dagger CLI using flags derived from the extension schema (which in turn suggests a relationship with support for standard project entrypoints)

just stopping by to ask about how extensions will be versions, referenced, and dependency managed? Especially considering I can import the same extension@version in multiple languages.
Would this code gen process create SDK code in all the target languages that we then have to tag, publish, and import via the usual means for the target languages?

Ancient issue, modules replaced this


fix: use correct path as input to owlbot-copy

Currently, the library generation integration test for google/iam/v1 succeeds although no java files were transferred to the final folder. A check was added in the IT to prevent this situation from passing.
This fix is intended to adapt to .OwlBot.yaml files that specify the full path in googleapis-gen to the desired source library. The tradeoff is an extra argument to generate_library.sh
This example does not use wildcards:
deep-copy-regex:
  - source: "/google/iam/v2beta/google-cloud-iam-v2beta-java/gapic-google-cloud-iam-v2beta-java/src"
    dest: "/owl-bot-staging/java-iam/v2beta/google-iam-policy/src"
  - source: "/google/iam/v2/google-cloud-iam-v2-java/gapic-google-cloud-iam-v2-java/src"
    dest: "/owl-bot-staging/java-iam/v2/google-iam-policy/src"

Our current logic uses a generated-java folder that will not produce a match for owlbot copy using the yaml above.
This logic assumed all .OwlBot.yaml files looked like this one
deep-copy-regex:
- source: "/google/cloud/alloydb/(v.*)/.*-java/proto-google-.*/src"
  dest: "/owl-bot-staging/java-alloydb/$1/proto-google-cloud-alloydb-$1/src"

This fix is intended to adapt to .OwlBot.yaml files that specify the full path in googleapis-gen to the desired source library. The tradeoff is an extra argument to generate_library.sh

I would rather us fixing the non-standard owlbot configuration rather than accommodating it in our scripts, especially at the cost of one more parameter. Is there any harm in fixing the owlbot config for iam?

This fix is intended to adapt to .OwlBot.yaml files that specify the full path in googleapis-gen to the desired source library. The tradeoff is an extra argument to generate_library.sh

I would rather us fixing the non-standard owlbot configuration rather than accommodating it in our scripts, especially at the cost of one more parameter. Is there any harm in fixing the owlbot config for iam?

@blakeli0 Sounds good. I created https://github.com/googleapis/google-cloud-java/pull/10050 to address this.

This approach is obsolete. https://github.com/googleapis/google-cloud-java/pull/10050 fixed this


HTTPS request works using the https module but not in Axios

This request works with the https library:
const options = {
    hostname: 'baseurl.com',
    path: '/accounts',
    method: 'GET',
    ca: fs.readFileSync(`${path}CA.pem`),
    cert: fs.readFileSync(`${path}CERT.pem`),
    key: fs.readFileSync(`${path}KEY.pem`),
    auth: 'user:password',
    rejectUnauthorized: false
};
const req = https.request(options, (res) => {
    res.on('data', (data) => {
        process.stdout.write(data);
    });
});
req.end();
req.on('error', (e) => {
    console.error(e);
});

but this seemingly equivalent request does not work using Axios:
const instance = axios.create({
    baseURL: 'https://baseurl.com',
    httpsAgent: new https.Agent({
        ca: fs.readFileSync(`${path}CA.pem`),
        cert: fs.readFileSync(`${path}CERT.pem`),
        key: fs.readFileSync(`${path}KEY.pem`),
        auth: 'user:password',
        rejectUnauthorized: false
    })
});

instance.get('/accounts')
  .then(_ => console.log(`response: ${_}`))
  .catch(err => console.log(`error: ${err.stack}`));

Instead it throws this error:
error: Error: write EPROTO
at Object.exports._errnoException (util.js:870:11)
at exports._exceptionWithHostPort (util.js:893:20)
at WriteWrap.afterWrite (net.js:763:14)
I've tried these variations of base url but no luck:

'https://baseurl.com:443'
'baseurl.com'
'baseurl.com:443'

any help much appreciated.

Hi,
I have practically the same issue  :(
Did i miss something ?
nodejs script :
var axios = require('axios')
var params = {
  url: 'https://raw.githubusercontent.com/mzabriskie/axios/master/README.md',
  method: 'get'
}
axios(params)
  .then(res => res.data)
  .catch(error => {
    console.error('$http:ajax:catch', error)
  })

error :
$http:ajax:catch { Error: write EPROTO<PHONE_NUMBER>55104:error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol:../deps/openssl/openssl/ssl/s23_clnt.c:794:
    at exports._errnoException (util.js:1022:11)
    at WriteWrap.afterWrite (net.js:801:14)
  code: 'EPROTO',
  errno: 'EPROTO',
  syscall: 'write',
  config:
   { adapter: [Function: httpAdapter],
     transformRequest: { '0': [Function: transformRequest] },
     transformResponse: { '0': [Function: transformResponse] },
     timeout: 0,
     xsrfCookieName: 'XSRF-TOKEN',
     xsrfHeaderName: 'X-XSRF-TOKEN',
     maxContentLength: -1,
     validateStatus: [Function: validateStatus],
     headers:
      { Accept: 'application/json, text/plain, */*',
        'User-Agent': 'axios/0.15.3',
        host: 'raw.githubusercontent.com' },
     method: 'get',
     url: 'https://raw.githubusercontent.com/mzabriskie/axios/master/README.md',
     data: undefined },
  response: undefined }

I just tested under windows on another pc (and another netword) and there is no problem ...
i'm under osx (10.11.6).
i tested with node 6.9.1 and 6.9.4
maybe a proxy issue or another bullshit 😭
best regards :)

I still have this issue not being able to hit an https url with an http proxy. I can do it fine in request and also in curl from shell. Something not working with axios. I get an EPROTO error.

Thanks @sylvainpina, looks like your issue was slightly different as I
could get the request working using the https library but not with axios.
@Anthony, does your Axios request look the same as mine? It would be
interesting to see your code.
On Fri, 3 Feb 2017 at 11:23 Anthony Ettinger<EMAIL_ADDRESS>wrote:

I still have this issue not being able to hit an https url with an http
proxy. I can do it fine in request and also in curl from shell. Something
not working with axios. I get an EPROTO error.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mzabriskie/axios/issues/662#issuecomment-277225083,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEVwZT1U9_XXxFPttw4PpFFc_Hb69sAOks5rYw47gaJpZM4Lq6oN
.

@jamesw201
This works fine in request() and with curl -x  https://ipinfo.io
const http = rp.defaults({
  proxy: 'asdf:asdf@x.x.x.x:8080',
  timeout: 4000,
  resolveWithFullResponse: true
});

Axios returns an EPROTO error if I send an https url:
const http = axios.create({
  proxy: {
    host: 'x.x.x.x',
    port: 8080,
    auth: : {
      username: 'asfd',
      password: 'asdf'
    }
  }
});

I'm having the same problem as @chovy when sending https through http proxy

I found the problem in the code. Easy fix but I'm not sure how transport should be defined.
options.path = protocol + '//' + parsed.hostname + (parsed.port ? ':' + parsed.port : '') + options.path;
This should be something like:
options.protocol = proxy.protocol || protocol;
options.path = options.protocol + '//' + parsed.hostname + (parsed.port ? ':' + parsed.port : '') + options.path;

This allow us to use an http proxy server with an https url:
{
 	proxy: {
 		...
    	        protocol: 'http'
	}
 }

I tried adding this change and get a protocol mismatch assertion error when running tests. Not sure what that's about. I'm guessing the method of sending https url to httpFollow is not working.

we have exactly same issue after upgrading to latest version of axios. Is anyone is looking at creating pull request?

This is a show stopper for us too. We have to switch to request.js because of it now.

My original post was wrong, I didn't read the documentation closely enough. The request should have looked like this:
var instance = axios.create({
      baseURL: 'https://baseurl.com',
      httpsAgent: new https.Agent({
        ca: fs.readFileSync(`${path}CA.pem`),
        cert: fs.readFileSync(`${path}CERT.pem`),
        key: fs.readFileSync(`${path}KEY.pem`),
        rejectUnauthorized: false
      }),
      auth: {
        username: 'username',
        password: 'password'
      }
});

instance.get('/accounts')
    .then(_ => console.log(`response: ${_}`))
    .catch(err => console.log(`error: ${err.stack}`));

I'm having the same problem. Has anyone solved the problem of https request over proxy? I do not want to move to request.js. (request.js in this situation works well).

i have the same issue here, very simple usage to used in nodejs , just do the https request,

This ticket was closed, but the problem described by @chovy is still there.

I still have this issue not being able to hit an https url with an http proxy. I can do it fine in request and also in curl from shell. Something not working with axios. I get an EPROTO error.

if  not  in  node  ,how  to get https module
??? @FastNinja

There is a bug in lib/defaults.js::getDefaultAdapter ()
In Jest unit tests, it returns ./lib/adapters/xhr.js even when httpsAgent: is present. XHR adapter has no support for httpsAgent:
The workaround solution is:
let axiosOverride = axios.create({
  baseUrl: 'https://localhost:5000',
  adapter: require('axios/lib/adapters/http'),
  httpsAgent: new spdy.createAgent (sslOptions),
  withCredentials: true
});

export default axiosOverride

@JamesMcGuigan thanks for your post.
I've been scratching my head for a while about why I'm getting an error similar to OP's (though it seems unrelated now).
More googling lead me to https://github.com/axios/axios/issues/1180 and setting jest's testEnvironment to node also fixes that problem.
hope this helps anyone else having this issue!

@sylvainpina  thanks for your comments, but why the other http modules do not meet the questions? such as node-fetch?

@hanzichi, no idea.
i don't have the problem with reqwest
maybe Axios is more advanced and do more check.
best regards


cilium: encryption, additional mtu fix for non-default 1500B MTU

We tried to fix the encryption route table in the tunnel mode case here, https://github.com/cilium/cilium/pull/10218 but we used the default MTU variable, EthernetMTU to calculate the newest route MTU. This only works when network facing MTU is the same as EthernetMTU, 1500B in current code base. I must have believed EthernetMTU would be changed to match current MTU, its not.
To fix use configured MTU instead of hard coded default MTU. After this we will use the correct MTU even if network facing MTU is 9k (apparently fairly common) or some other value.

This change is

test-me-please

test-me-please

Coverage decreased (-0.002%) to 45.648% when pulling fa3da1eb6b951fe0086a5425aa2e70da349c4e10 on mtu-fixes-fix into 711b37ed100cf0440bf2b803a0728b10d15fb97e on master.

tested fix on gke which uses non-standard mtu 1460 and verified route MTUs are set as expected. Before patch route MTUs are broken as well.

test-me-please

Unrelated etcd flake in unit tests


Bump bindgen to 0.58 with new terminology

Supersedes #722 since the automatic PR failed CI
bors r+

bors r+


SSL: self signed certificate

While upgrading from an older version of nodered to the latest, I tested this node.
When trying to connect to a Digital Ocean managed database with SSL=true I get the following error:
"Error: self signed certificate in certificate chain"
With SSL=false i get this error:
error: no pg_hba.conf entry for host "xxx", user "xxx", database "xxx", SSL off
I think the reason for these errors are that pg since v. 8.0 sets
rejectUnathurized = true
as  default as described here: https://node-postgres.com/announcements
At the same time DO probably self-signs their certificates as speculated here: https://github.com/Budibase/budibase/issues/1967#issuecomment-879981467
I am able to connect to my database by passing this config node as described in the documentation:
msg.pgConfig = {
  user: 'xxx',
  password:'xxx',
  host: 'xx.xx.xx',
  database: 'xxx',
  port: xxxx,
  ssl: {
  rejectUnauthorized: false
  }
};

But it would be neat to be able to set rejectUnathorized in the properties setting in the node:

I found out that you can pass json to the SSL tab and write in
{"rejectUnauthorized":false}

This way I can use the standard configuration


Added handling of nullable types suggesting using the third argument …

…while creating MockDataColumns to define nullable columns.

Thanks for your contribution @ShadowDIO I just merged your changes