Python 3 support

Running into problems with Python3 (AttributeError: 'dict' object has no attribute 'iteritems')
p3512@Kaboom:~/Desktop$ which python3
/usr/bin/python3
p3512@Kaboom:~/Desktop$ mkvirtualenv mt-940 --python=/usr/bin/python3
Running virtualenv with interpreter /usr/bin/python3
Using base prefix '/usr'
New python executable in mt-940/bin/python3
Also creating executable in mt-940/bin/python
Installing setuptools, pip...done.
(mt-940)p3512@Kaboom:~/Desktop$ pip install mt-940
Downloading/unpacking mt-940
  Downloading mt-940-2.2.tar.gz
  Running setup.py (path:/home/p3512/.virtualenvs/mt-940/build/mt-940/setup.py) egg_info for package mt-940

Downloading/unpacking enum34 (from mt-940)
  Downloading enum34-1.0.4.tar.gz
  Running setup.py (path:/home/p3512/.virtualenvs/mt-940/build/enum34/setup.py) egg_info for package enum34

Installing collected packages: mt-940, enum34
  Running setup.py install for mt-940

  Running setup.py install for enum34

Successfully installed mt-940 enum34
Cleaning up...
(mt-940)p3512@Kaboom:~/Desktop$ python
Python 3.4.0 (default, Jun 19 2015, 14:20:21)
[GCC 4.8.2] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import mt940
>>> import pprint
>>> transactions = mt940.parse('abnamro.sta')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/p3512/.virtualenvs/mt-940/lib/python3.4/site-packages/mt940/parser.py", line 45, in parse
    transactions.parse(data)
  File "/home/p3512/.virtualenvs/mt-940/lib/python3.4/site-packages/mt940/models.py", line 209, in parse
    for k, v in result.iteritems():
AttributeError: 'dict' object has no attribute 'iteritems'
>>>

Haven't needed a Python 3 version myself but it should be easy to fix. I
can work on it next week.
On 3 Jul 2015 10:32, "Martijn de Geus"<EMAIL_ADDRESS>wrote:

Running into problems with Python3
p3512@Kaboom:~/Desktop$ which python3
/usr/bin/python3
p3512@Kaboom:~/Desktop$ mkvirtualenv mt-940 --python=/usr/bin/python3
Running virtualenv with interpreter /usr/bin/python3
Using base prefix '/usr'
New python executable in mt-940/bin/python3
Also creating executable in mt-940/bin/python
Installing setuptools, pip...done.
(mt-940)p3512@Kaboom:~/Desktop$ pip install mt-940
Downloading/unpacking mt-940
Downloading mt-940-2.2.tar.gz
Running setup.py (path:/home/p3512/.virtualenvs/mt-940/build/mt-940/setup.py) egg_info for package mt-940
Downloading/unpacking enum34 (from mt-940)
Downloading enum34-1.0.4.tar.gz
Running setup.py (path:/home/p3512/.virtualenvs/mt-940/build/enum34/setup.py) egg_info for package enum34
Installing collected packages: mt-940, enum34
Running setup.py install for mt-940
Running setup.py install for enum34
Successfully installed mt-940 enum34
Cleaning up...
(mt-940)p3512@Kaboom:~/Desktop$ python
Python 3.4.0 (default, Jun 19 2015, 14:20:21)
[GCC 4.8.2] on linux
Type "help", "copyright", "credits" or "license" for more information.

import mt940
import pprint
transactions = mt940.parse('abnamro.sta')
Traceback (most recent call last):
File "", line 1, in
File "/home/p3512/.virtualenvs/mt-940/lib/python3.4/site-packages/mt940/parser.py", line 45, in parse
transactions.parse(data)
File "/home/p3512/.virtualenvs/mt-940/lib/python3.4/site-packages/mt940/models.py", line 209, in parse
for k, v in result.iteritems():
AttributeError: 'dict' object has no attribute 'iteritems'

—
Reply to this email directly or view it on GitHub
https://github.com/WoLpH/mt940/issues/10.

Version 3 has just been released with Python 3 support :)


throw java.lang.NoClassDefFoundError: org/apache/shardingsphere/core/execute/ShardingExecuteDataMap

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
how to fix this exception or how to ignore this exception in log file

Bug

Which version of SkyWalking, OS and JRE?
SkyWalking 7.0,  jre

What happen?
throw exception in log,  I have checked plugin, we have apm-sharding-sphere-4.x-plugin-7.0.0.jar in agent folder

Which shardingsphere version are you using? I think you should ping ShardingSphere community, they contribute this plugin with tests in the following versions.

Also, they open this #4229 but haven't processed this still.

FYI @tristaZero @tuohai666 @terrymanu I think the ShardingSphere package rename continues happening. SkyWalking received 3+ issues reported because of the compatible issue. Please be advised.

Which shardingsphere version are you using? I think you should ping ShardingSphere community, they contribute this plugin with tests in the following versions.

Also, they open this #4229 but haven't processed this still.

thanks for your quick response ,  we use  org.apache.shardingsphere  v <IP_ADDRESS>  not RC version

more info, the first time we found this error in skywalking.log, it shows "at org.apache.skywalking.apm.plugin.shardingsphere.v4rc3.ExecuteInterceptor.beforeMethod(ExecuteInterceptor.java:42)"
so we remove apm-sharding-sphere-4.x-rc3-plugin-7.0.0.jar from plugins folder, but it still don't work

Don't have plugin for that. That is why we asked you to submit request to ShardingSphere

Don't have plugin for that. That is why we asked you to submit request to ShardingSphere

ok, thanks

@wu-sheng @wz2cool, this problem is due to package name changed. @menghaoranss from ShardingSphere community will add new plugin for that.

@wu-sheng @wz2cool, this problem is due to package name changed. @menghaoranss from ShardingSphere community will add new plugin for that.

ok thanks, currently we will remove this plugin to avoid error log

this proplem has not be fixed in version of 8.0.1


How to access reference file from Cloud Storage

Should I download reference files locally or is it possible to access them remotely? In that case, how can I do that?

If you plan to run the workflow locally then you'll need to download the files. If you plan the run the workflow on the google cloud platform then you wont need to download the files. It might be possible to use gatk4 NIO but that would require some edits to the workflow, if your intersted in NIO please ask your question in the wdl/cromwell forum. And in general please direct questions to the Gatk forums (GATK, Firecloud, WDL+Cromwell)
Link to (How to) Execute Workflows from the gatk-workflows Git Organization tutorial
Hope this helps

Thank you, I'll post further questions there.


[5.8] Allocate memory for error handling to allow handling memory exhaustion limits

Allots 10kb of memory for "memory exhaustion" errors.
It lets the application give a more descriptive error than the default Fatal Error that is thrown for memory exhaustion.
The concept was taken from Symfony's error handler: https://github.com/symfony/symfony/blob/4.4/src/Symfony/Component/ErrorHandler/ErrorHandler.php.

Provide before and after screenshots of the difference in error messages please.

This solution won't work, however, in the case that this method is called more than once?

Is setting to null actually guaranteed to release the memory. How does PHP calculate this? Do we have to wait for the garbage collector?

Also - whats the performance hit (if any) on running the 'x' repeat command 10240 times? Both in terms of time and memory?

@taylorotwell
Current Error

Better Error

@GrahamCampbell
The bootstrap method will only run each bootstrapper once on a request:
https://github.com/laravel/framework/blob/5.8/src/Illuminate/Foundation/Http/Kernel.php#L161
From what I've been able to see, setting a variable to null frees the memory immediately.
@laurencei
The function is only running once with 1 byte. The PHP source for str_repeat shows: "Heavy optimization for situations where input string is 1 byte long". Do you know of a more efficient way to allocate 10kb? Again, this idea was taken from Symfony's error handler. Throughout the history of that project, I couldn't find any discussion on this.
Thanks for the feedback and comments!

I like this approach, very useful.
Though I would find it really nice if there was a bit of documentation on the proper on what it is for.
@skylerfenn do you think you could add that?

Memory is very expensive. Is that really reasonable?


TrimSpace when reading InternalToken from a file

InternalTokens are fixed as alphanum strings therefore TrimSpace from these.
Also use isatty to not add a terminal newline when redirecting generate.
Fix #11498

I've labelled these as bug because it's a bit annoying so they should be backported - but would understand if people disagreed and marked it enhancement.

Please send backport


How to integrate OHIF viewer with existing ZFP Viewer

:hand: How to integrate OHIF viewer with existing ZFP Viewer

Description
We have ZFP Viewer where the image has been displayed, assume Study Instace UID = "1234". Now on click some buttons in ZFP viewer, OHIF viewer will be started with the same study [Study Instace UID = "1234"]. How can I implement this? Please help.
Regards,

I'm not familiar with the ZFP Viewer, but if you have Study Instance UID, you should be able to open it in OHIF viewer simply going to localhost:8080/viewer/<study_instance_uid>. The question is how do you implement a button in the ZFP viewer so that it can open a link to OHIF, but that I believe is something to discuss with ZFP Viewer developers.

Regrettably, the mentioned feature is not supported at present and is not included in our current plans. However, we would be delighted to review your pull request if you would like to contribute.


Give a method a different name from it's signature

I have been looking around the documentation but I have not found a way to specify a different name for a class method than the rust function signature.
According to the docs you can do this for a standard function like so:
#[pymodule]
fn rust2py(py: Python, m: &PyModule) -> PyResult<()> {

    #[pyfn(m, "sum_as_string")]
    fn sum_as_string_py(_py: Python, a:i64, b:i64) -> PyResult<String> {
        Ok(format!("{}", a + b))
    }

    Ok(())
}

(rust function is called 'sum_as_string_py', but in python you call 'sum_as_string'
I would love to be able to do something similar for a classes' methods:
#[pymethods]
impl Dataset {
    #[pyname = "add_feature"]
    fn add_feature_py(&mut self, matrix: &PyArray1<bool>) -> PyResult<()> {
        Ok(())
    }
}

This way you can give functions in the rust interface the same name as their python counterparts:
impl Dataset {
    fn add_feature(&mut self, matrix: Vec<bool>) -> { }
}

You can do
#[pymethods]
impl Dataset {
    #[name = "add_feature"]
    fn add_feature_py(&mut self, matrix: &PyArray1<bool>) -> PyResult<()> {
        Ok(())
    }
}

Pardon the bump but just found this when I had the same question. Maybe it would be helpful if the available annotations were listed in the guide?

@cmyr no problem at all, I agree there's a documentation issue here. I've been trying to start solving this problem by making the attribute syntax more uniform: #1567
The idea being that the attribute will become #[pyo3(name = "...")], and we'll document all the options that the #[pyo3(...)] attribute will support.
If you have a moment to read through that PR, I'd welcome your opinion on the design proposal.


Ejhumphrey/ack

updates readme for #27 and adds an acknowledgements file (to be included in the TGZ archive)

Pull Request Test Coverage Report for Build 44

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

Totals

Change from base Build 41:
0.0%

Covered Lines:
222

Relevant Lines:
236

💛  - Coveralls

Tried to follow the instructions. And then when I tried to run
./scripts/featurefy.py --file /some/audio/file.wav ./output_dir
I got this error message:
Did you forget to run ./scripts/download-deps.sh?
I am then wondering if the instructions are enough.

To give some details on the issue. I created a new miniconda py3.6 env to test it, and the issue is probably a path one related to this.
When I run ./scripts/download-deps.sh, the VGGish model is downloaded in the ./openmic/vggish/__model__ folder. However, when I run afterwards pip install ., the openmic package is getting put in ~/miniconda2/envs/py36/lib/python3.6/site-packages/openmic/vggish/, without the __model__ folder. Then the featurefy.py script gets confused.
If I manually copy the __model__ folder in ~/miniconda2/envs/py36/lib/python3.6/site-packages/openmic/vggish/, the featurefy.py script works well.
Is there a way not to have to copy the VGGish model on your conda env site-package that could be written in the instruction README? Or should we just acknowledge to be careful when using a virtual environment?

Interesting.  I'd say this is probably not great style -- __FOO__ is generally reserved in python for built-ins like repr or init.  Is it too late in the game to switch that up to __model (or _model would be better, since it communicates privacy without potentially going through namespace mangling)?

update from a sidebar: the paths in the setup file's package data were found to be erroneous (thanks @bmcfee), fixing on this issue since it came up as part of issues with installation instructions.

build is fine for 2.7/3.5/3.6, have disabled 3.4 and documented (somewhat) in #30


kong admin api is ok,but can not add connection

sorry, there are somewrong with network, fixed it.

@kcloze how did you fix it?


The command line version of this doesn't replace plurals with other.

Can you please provide a reproducible example or a failing test here: https://github.com/i18next/i18next-v4-format-converter/blob/main/test/converter.spec.js

I tested, and its only failing with your bin version published on npm.
npm install -g i18next-v4-format-converter
And run it on
{
  "user_one": "{{count}} user",
  "user_plural": "{{count}} users"
}

example:

{
  "user": "{{count}} user",
  "user_plural": "{{count}} users",
  "number_of_posts": "sing post",
  "number_of_posts_plural": "plur post"
}

➜  tmp i18next-v4-format-converter
transformed 1 files: /Users/usr/test-project/tmp

{
  "user_one": "{{count}} user",
  "user_other": "{{count}} users",
  "number_of_posts_one": "sing post",
  "number_of_posts_other": "plur post"
}

Okay, I set up a reproducible example using docker:
git clone https://github.com/dessalines/docker_i18next_converter
docker build  . --progress=plain

You can clearly see that your published version doesn't produce the correct output:
#8 [5/7] RUN cat test.json
#8 sha256:0f9b08911ff5613f9884c439978f8529340b7eb546fdd743d390145f80053378
#8 0.679 { "user": "{{count}} user", "user_plural": "{{count}} users" }
#8 DONE 0.7s

#9 [6/7] RUN i18next-v4-format-converter
#9 sha256:3e2a7599e027fc73c644c93308c9473a1fe0bea87294a4242dad17e1cfcb64a2
#9 0.939 transformed 1 files: /test
#9 DONE 1.0s

#10 [7/7] RUN cat test.json
#10 sha256:ff3ba22abd3894fb56ec780b4a96aec5482196305d0d927efc125fe9899fad31
#10 0.723 {
#10 0.723   "user_one": "{{count}} user",
#10 0.723   "user_plural": "{{count}} users"
#10 0.723 }#10 DONE 0.8s

[CLI] with 378 files, in languages ch or jp files _plural was not changed to _other. But on the second attempt, they were changed

i18next-v4-format-converter -i locales/ transformed 306 files
[CLI] with 306 files, in languages ch or jp files _plural was not changed to _other. But on the second attempt, they were changed

For me _plural is not changed to _other for files in languages id, ja and zh

These languages has no _plural suffix at all: https://jsfiddle.net/6bpxsgd4

Oh yeah understood, I guess we anyway added the _plural for other languages with same values.


[Tables] Make delete_entity also convert Partition/RowKey to string when TableEntity is passed

Description
This pull request makes delete_entity do the same conversion of all properties of the TableEntity into string as the other methods (via _add_entity_properties e.g. create_entity, upsert_entity) to allow passing a TableEntity with non-string Partition/RowKey
See https://github.com/Azure/azure-sdk-for-python/issues/26360
All SDK Contribution checklist:

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

General Guidelines and Best Practices

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

Testing Guidelines

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

I did add tests, did not get them to run though; hoping for pipelines here :)

Hi @mtin thanks for your contribution!
The PR is currently in draft, I'm not sure if you are preparing it. Could you please fix the failing CI? Or I can help the rest with your permission granted in your GitHub branch.
Thanks

Thank you for getting back to me @YalinLi0312! I was unsure if the proposed changes are desired at all; There were no further answers at https://github.com/Azure/azure-sdk-for-python/issues/26360. I also could not really make sense of the failing pipelines, I would be grateful if you could assist. I have granted you permission to my fork!

Thank you for getting back to me @YalinLi0312! I was unsure if the proposed changes are desired at all; There were no further answers at #26360. I also could not really make sense of the failing pipelines, I would be grateful if you could assist. I have granted you permission to my fork!

I have accepted your invitation. No worries, I'll help on the rest. Thanks

/azp run python - tables - test

/azp run python - tables - tests

@microsoft-github-policy-service agree

Close it as we will fix the UUID type issue in our new serialization design, thanks for your contribution as well @mtin


Replace the spdx-license-ids and spdx-exceptions packages with upstream data

The spdx-license-ids and spdx-exceptions packages just contain a single JSON file each. To reduce maintenance effort, I believe it would make sense to replace these with directly using the data from https://github.com/spdx/license-list-data/tree/main/json in this project.

I maintain all the relevant packages and prefer to track them separately.

I agree; consolidation increases maintenance effort, it doesn’t decrease it.

To clarify, the background of my request was that upstream updates seem to go unnoticed... https://github.com/jslicense/spdx-exceptions.json hasn't been updated in 4 years (despite a PR being ready to merge), missing exception like Autoconf-exception-generic.


Add LOCAL.md as guide for running locally

Had to write this before I forget the steps and the issues I encountered. View rendered here.

Rebased and merged in 04145dbc0e0569754f86ff84725a4c21dede97d4


No E-Mail Notifications will be send on up or down of check but testing the email integration works

Hi,
I am running healthchecks as a daemon using a docker-compose.yml file which looks as follows:
version: '3'
services:
  healthchecks:
    image: galexrt/healthchecks:latest
    container_name: healthchecks
    restart: always
    ports:
      - "8000:8000"
    volumes:
      - SQLite:/data
    environment:
      # DB_NAME must be set like this for the /data volume to be used
      # otherwise the SQLite db is lost on container deletion.
      DB_NAME: "/data/hc.sqlite"
      SECRET_KEY: 'secret'
      ALLOWED_HOSTS: 'localhost,hc.example.com'
      DEBUG: "False"
      DEFAULT_FROM_EMAIL<EMAIL_ADDRESS>      USE_PAYMENTS: "False"
      REGISTRATION_OPEN: "False"
      EMAIL_HOST: "mailout.example.com"
      EMAIL_PORT: "587"
      EMAIL_HOST_USER: "user"
      EMAIL_HOST_PASSWORD: "password"
      EMAIL_USE_TLS: "True"
      SITE_ROOT: "https://hc.example.com"
      SITE_NAME: "HCM"
      MASTER_BADGE_LABEL: "HCM"
      PING_ENDPOINT: "https://hc.example.com/ping/"
      TWILIO_ACCOUNT: "None"
      TWILIO_AUTH: "None"
      TWILIO_FROM: "None"
      PD_VENDOR_KEY: "None"
      TRELLO_APP_KEY: "None"
      CONTAINER_PRUNE_INTERVAL: 600
volumes:
  SQLite:

E-Mail notifications worked properly but after a few days suddenly no more notification emails (up or down of a check) were sent by healthchecks any more. Docker logs show a traceback which seams to show up on every notification attempt:
2020-10-05 00:47:18,811 INFO success: sendalerts entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/supervisor/loggers.py", line 84, in emit
    self.flush()
  File "/usr/lib/python2.7/dist-packages/supervisor/loggers.py", line 64, in flush
    self.stream.flush()
IOError: [Errno 28] No space left on device

I checked the running processes and /usr/bin/python3 manage.py sendreports --loop  was still running.
Interesting is, that if I test the email integration with the test button manually, the mail will be delivered successfully.
After restarting the docker container everything seems to be working again properly.
Does anyone know what causes the IOError?

Does your machine still enough space? No space left on device
Post the df -h of your host.

[root@x2845532 ~]# df -h
Dateisystem    Größe Benutzt Verf. Verw% Eingehängt auf
devtmpfs         16G       0   16G    0% /dev
tmpfs            16G       0   16G    0% /dev/shm
tmpfs            16G    1,6G   15G   10% /run
tmpfs            16G       0   16G    0% /sys/fs/cgroup
/dev/md1        436G    393G   22G   95% /
/dev/md0        969M    225M  678M   25% /boot
tmpfs           3,2G       0  3,2G    0% /run/user/0
overlay         436G    393G   22G   95% /var/lib/docker/overlay/0baf47a78cc8fb65c06c5dfcc7d08eed1243c2d074ffd41e04ded89d01e93406/merged

@CodeSmolder Exec into the container and run df there, the supervisor program runs healthchecks and such so it probably is related to the low percent of storage available, e.g., different volume / mounts are used for / inside the container.
Make sure quotas and such don't affect the volume(s) and container(s) you have running.

@CodeSmolder Also have you already tried to recreate the container itself? Maybe the container's root fs is full. How long is / was the container running?

@CodeSmolder Exec into the container and run df there, the supervisor program runs healthchecks and such so it probably is related to the low percent of storage available, e.g., different volume / mounts are used for / inside the container.
Make sure quotas and such don't affect the volume(s) and container(s) you have running.

root@d5a46a8ff620:/# df -h
Filesystem      Size  Used Avail Use% Mounted on
overlay         436G  393G   21G  96% /
tmpfs            64M     0   64M   0% /dev
tmpfs            16G     0   16G   0% /sys/fs/cgroup
/dev/md1        436G  393G   21G  96% /data
shm              64M     0   64M   0% /dev/shm
tmpfs            16G     0   16G   0% /proc/acpi
tmpfs            16G     0   16G   0% /proc/scsi
tmpfs            16G     0   16G   0% /sys/firmware

I have not quotas set, so there should be not affect on volume(s) and container(s)

@CodeSmolder Also have you already tried to recreate the container itself? Maybe the container's root fs is full. How long is / was the container running?

I recreated the container and then it worked again with no problem.
The container was running about a week before when the error occurred.

@CodeSmolder I have pushed a commit to lower the log sizes that supervisord will keep, that may solve the issue for good.
The image tag v1.17.0-20201019-143145-678 should become available in the next 15-30 minutes.

@galexrt: thank you :)


MapView backgroundColor is white. Transparent tiles (labels) do not appear transparent

Bug
Transparent tiles do not appear transparent. The backgroundColor of MapView is white, it's blocking when needed to display transparent tiles over elements.
Environment info

System:
OS: Windows 10 10.0.18362
CPU: (8) x64 Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz
Memory: 3.50 GB / 7.95 GB
Binaries:
Node: 12.16.1 - H:\Programs\NodeJS\node.EXE
npm: 6.13.4 - H:\Programs\NodeJS\npm.CMD
npmPackages:
react: ~16.9.0 => 16.9.0
react-native: https://github.com/expo/react-native/archive/sdk-36.0.0.tar.gz => 0.61.4
react-native-maps: 0.26.1

Steps To Reproduce
Use a MapView component with a custom provider :
Example : http://c.tile.stamen.com/terrain-labels/{z}/{x}/{y}.jpg
Describe what you expected to happen:
The tile and the MapView should appear transparent (expect the labels in the provider example).
Reproducible sample code

import MapView, { PROVIDER_DEFAULT, MAP_TYPES } from "react-native-maps";
...
<MapView
provider={PROVIDER_DEFAULT}
mapType={Platform.OS == "android" ? MAP_TYPES.NONE : "standard"}
style={{ height: "100%", width: "100%" }}
>
<MapView.UrlTile
urlTemplate={"http://c.tile.stamen.com/terrain-labels/{z}/{x}/{y}.jpg"}
shouldReplaceMapContent={true}
/>

The following tile is transparent (except the labels) but the MapView component has a white backgroundColor that covers the underlying elements :

By forcing the opacity of the MapView component (opacity={0.3}), we can see the underlying elements : (but the labels are 0.3 transparents too of course)

By putting an opacity to 0.8, the underlying map is almost covered by the MapView white background :

No need to provide an image for the opacity 1, you get the idea.
Is it possible to fix that or letting the possibility to change the backgroundColor just like 'loadingBackgroundColor'

The same issue

+1

+1


How to set a cover to a  word document?

I can add a picture or add a paragraph to a word document, but I cant figure out how height  or width i can get to make it as  a cover?
is there some css-style to control it？
thanks for your help

I don't understand your question. In general, support questions are best asked on StackOverflow and tagged with the python-docx tag.


feat: Hapi.js starter template

What kind of change does this PR introduce?
Feature: Support Hapi.js Nodejs framework (Except for Graphql template).
Did you add tests for your changes?
Yes.
If relevant, did you update the documentation?
No
Summary
Support Hapi.js Nodejs framework (except for Graphql template). By default, Express is the server template it will support. But if the user selects More option. Then the user can select all the available server template (Express.js & Hapi.js).
For Graphql template, More option is currently not available.
Fix: #66
Does this PR introduce a breaking change?
No
Other information

@jamesgeorge007 You can review this PR.

@jamesgeorge007 Please review again.


Creation and removal of containers via ctr CLI does not release inodes in /run/containerd/fifo

Description
Containerd does not release inodes in /run/containerd/fifo, if containers are created and removed in a loop via the ctr cli. This leads to 100% inode usage on the /run file system over time which prevents containerd to create new containers.
Steps to reproduce the issue:

The described behavior was noticed during a stress test which executed the following commands in a loop and stored the results as csv. The csv output was monitored for failures. In case of an exit code not equal to 0, up to 10 retries are executed before giving up that iteration.

ctr run -d docker.io/library/alpine:latest test.${iteration} sleep 60
sleep 0.1
ctr tasks pause test.${iteration}
sleep 0.1
ctr tasks resume test.${iteration}
sleep 0.1
ctr tasks kill -s SIGKILL test.${iteration}
sleep 0.1
ctr container delete test.${iteration}

During the test execution system metics (based on /proc file system and df command) were collected every 10 minutes and stored as csv.

After multiple failed iterations appeared in the log, the csv files were analyzed.

Describe the results you received:
After running the loop for 2 days and 17 min (126.064 container iterations) containerd could no longer start new containers.
The error reported by ctr was:
126065,1529869634,ctr:,OCI,runtime,create,failed:,no,space,left,on,device:,unknown,real,0.134,user,0.004,sys,0.024,1,ctr run -d docker.io/library/alpine:latest test.126065 sleep 60

During the first 126.064 cycles only in iteration 87631 and 116278 the following error occurred and could be resolved by the first retry (probably a timing problem).
87631,1529817767,real,0.292,user,0.016,sys,0.012,0,ctr run -d docker.io/library/alpine:latest test.87631 sleep 60
87631,1529817768,real,0.073,user,0.016,sys,0.000,0,ctr tasks pause test.87631
87631,1529817768,real,0.248,user,0.008,sys,0.008,0,ctr tasks resume test.87631
87631,1529817768,real,0.367,user,0.012,sys,0.004,0,ctr tasks kill -s SIGKILL test.87631
87631,1529817769,time="2018-06-24T05:22:49Z",level=error,msg="failed,to,delete,container,"test.87631"",error="cannot,delete,a,non,stopped,container:,{running,0,0001-01-01,00:00:00,+0000,UTC}",ctr:,cannot,delete,a,non,stopped,container:,{running,0,0001-01-01,00:00:00,+0000,UTC},real,0.299,user,0.004,sys,0.012,1,ctr container delete test.87631
87631,1529817770,real,0.774,user,0.012,sys,0.004,0,ctr container delete test.87631

The analysis of /run/containerd showed the following:
# created 126074 folders
root@kube-tor01-crcb1c41560b7245269aa7262e7882c27f-w2:/run/containerd/fifo# ls -l | wc -l
126074
# each folder seem to contain pipes to already deleted containers
root@kube-tor01-crcb1c41560b7245269aa7262e7882c27f-w2:/run/containerd/fifo# ls -lah 000001714
total 0
drwx------      2 root root  100 Jun 23 08:07 .
drwx------ 126075 root root 2.5M Jun 24 20:09 ..
prwx------      1 root root    0 Jun 23 08:07 test.32583-stderr
prwx------      1 root root    0 Jun 23 08:07 test.32583-stdin
prwx------      1 root root    0 Jun 23 08:07 test.32583-stdout

It looks like /run/containerd/fifo contains pipes to containers and they were not removed completely.
By checking the filesystem usage on /run I could see that there is space left, but no inodes were available any longer.
root@kube-tor01-crcb1c41560b7245269aa7262e7882c27f-w2:/run/containerd/fifo# df -i /run
Filesystem     Inodes  IUsed IFree IUse% Mounted on
tmpfs<PHONE_NUMBER>66    12  100% /run
root@kube-tor01-crcb1c41560b7245269aa7262e7882c27f-w2:/run/containerd/fifo# df -h /run
Filesystem      Size  Used Avail Use% Mounted on
tmpfs           395M   41M  354M  11% /run

Plotting the file system usage of /run shows, that inodes were not released.

To verify the problem I rebooted the system which cleaned the /run tmpfs and explicitly deleted the task before deleting the container. An additional folder remains containing manual-stderr, manual-stdin and manual-stdout.
$ ls -lah /run/containerd/fifo/ | wc -l
28
$ ctr run -d docker.io/library/alpine:latest manual sleep 60
$ ctr tasks pause manual
$ ctr tasks resume manual
$ ctr tasks kill -s SIGKILL manual
$ ctr tasks delete manual
$ ctr container delete manual
$ ls -lah /run/containerd/fifo/ | wc -l
29

Those old folders can be removed by root afterwards manually, which release 4 inodes per folder:
$ df -i . && ls | wc && rm -rf 497171102 && df -i . && ls | wc
Filesystem     Inodes IUsed  IFree IUse% Mounted on
tmpfs          505078   864 504214    1% /run
     24      24     240
Filesystem     Inodes IUsed  IFree IUse% Mounted on
tmpfs          505078   860 504218    1% /run
     23      23     230

Describe the results you expected:
Containerd should not show any degradation over time and should not fail to create new containers.
Output of containerd --version:
$ ctr version
Client:
  Version:  v1.1.0
  Revision: 209a7fc3e4a32ef71a8c7b50c68fc8398415badf

Server:
  Version:  v1.1.0
  Revision: 209a7fc3e4a32ef71a8c7b50c68fc8398415badf

$ containerd --version
containerd github.com/containerd/containerd v1.1.0 209a7fc3e4a32ef71a8c7b50c68fc8398415badf

BUG REPORT INFORMATION
Ubuntu and Linux Kernel version
$ uname -a
Linux kube-tor01-xxx-w2.cloud.ibm 4.4.0-116-generic #140-Ubuntu SMP Mon Feb 12 21:23:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

$ cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.4 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.4 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

The system holds 4 GB of RAM, the swap space is disabled.

Thanks for the fast fix :-)

@konrad-ohms no problem

Hit the same issue recently. We (Garden team at Cloud Foundry) are using containerd programmatically and were passing nil to task.LoadProcess resulting in leaked fifos. We did a similar fix in our code, but I was just curious why would cio.Load not be the default. Do you have any use cases for that?

@georgethebeatle probably just an oversight if its causing fifos to be leaked.  Feel free to submit a fix


Different approach grid with zustand

Two goals were achieved with this huge modification:

I was able to try Zustand
I got rid of unnecessary render of cells

Now there are some issues left, which came up with this setup, while were working well earlier.
And my overall impression with Zustand, well... not so amazing, anyway.

Atm I think, this pr won't be merged.

After the fix however it can be merged ;)


Swipe direction

Please add a direction option to swipe left or right.
Add a Directionality widget can do that.

Hi @devbrisk thanks for opening an issue, could you elaborate? Do you want the capability to go 4 3 2 1 rather than 1 2 3 4 when swiping left or right?

Yes exactly

Got it, will have a look at this, thanks!

Hi @devbrisk, sorry for getting back to you on this so late, I just made a new release v0.8.9. There is now support for a reverseDirection boolean property. You can simply pass it when initializing the package.
 await SwipeImageGallery(
   context: context,
   children: images,
   reverseDirection: true,
 ).show();


Fix HWS model TabulatedRatingCurve with new Ribasim checks

TabulatedRatingCurve has some new checks built in in the core (see here), which make the HWS model fail:
┌ Error: The `flow_rate` cannot decrease with increasing `level`.
│   node_id = TabulatedRatingCurve #37
└ @ Ribasim c:\Users\konin_bt\Ribasim_development\Ribasim\core\src\validation.jl:431
┌ Error: The `flow_rate` cannot decrease with increasing `level`.
│   node_id = TabulatedRatingCurve #49
└ @ Ribasim c:\Users\konin_bt\Ribasim_development\Ribasim\core\src\validation.jl:431
┌ Error: The `flow_rate` must start at 0.
│   node_id = TabulatedRatingCurve #88
│   flow_rate = 685.6
└ @ Ribasim c:\Users\konin_bt\Ribasim_development\Ribasim\core\src\validation.jl:421
┌ Error: The `flow_rate` must start at 0.
│   node_id = TabulatedRatingCurve #148
│   flow_rate = 125.4
└ @ Ribasim c:\Users\konin_bt\Ribasim_development\Ribasim\core\src\validation.jl:421

Pretty sure this is fixed now.


helm drop offsets are wrong and re-wearing is broken

-Dropping off helmet has wrong offsets and it appears at far away position
-The activator "E" is still at the position where helm was dropped while helm actually is at a far off position
-Re-wearing it by pressing "E" at where it was dropped, It has weird behavior, doesn't gets worn and  a weird black things appears over in the scene.
Related:
https://github.com/webaverse/app/issues/2605
https://github.com/webaverse/app/issues/2373
https://user-images.githubusercontent.com/35926530/181766619-e34ea6c6-b5b8-4711-9dfc-5b223bf91059.mp4

Fixed in #3437 by @tcm390

fixed


Combat Extended ComPatch

By using the Gas Traps and Shells mod and the associated compatability patch built-in with CE, I messed around with the file and ran into a few issues.
Mostly it's down to not being a good coder, but if possible give both files a look and see what's wrong. Here's the exception it's throwing when I try to load CE with the patch. Mods are just the Framework, the three gasses, and vents, and loading CE last on the list.
Exception reading Patch_81mm.xml as XML: System.Xml.XmlException: The 'li' start tag on line 15 position 6 does not match the end tag of 'value'. Line 103, position 8.
  at System.Xml.XmlTextReaderImpl.Throw (System.Exception e) [0x00027] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlTextReaderImpl.Throw (System.String res, System.String[] args) [0x00029] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlTextReaderImpl.ThrowTagMismatch (System.Xml.XmlTextReaderImpl+NodeData startTag) [0x00086] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlTextReaderImpl.ParseEndElement () [0x00085] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlTextReaderImpl.ParseElementContent () [0x00149] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlTextReaderImpl.Read () [0x00085] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlLoader.LoadNode (System.Boolean skipOverWhitespace) [0x0025b] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlLoader.LoadDocSequence (System.Xml.XmlDocument parentDoc) [0x0001b] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlLoader.Load (System.Xml.XmlDocument doc, System.Xml.XmlReader reader, System.Boolean preserveWhitespace) [0x000b4] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at System.Xml.XmlDocument.Load (System.Xml.XmlReader reader) [0x0002e] in <0f9699188f0c414ea6fb5557f5c16d15>:0
  at Verse.LoadableXmlAsset..ctor (System.String name, System.String fullFolderPath, System.String contents) [0x00049] in <95de19971c5d40878d8742747904cdcd>:0
UnityEngine.StackTraceUtility:ExtractStackTrace ()
Verse.Log:Warning (string)
Verse.LoadableXmlAsset:.ctor (string,string,string)
Verse.DirectXmlLoader/<>c__DisplayClass1_0:<XmlAssetsInModFolder>b__0 (int)
Verse.GenThreading/<>c__DisplayClass8_1:<ParallelFor>b__0 (object)
System.Threading.QueueUserWorkItemCallback:WaitCallback_Context (object)
System.Threading.ExecutionContext:RunInternal (System.Threading.ExecutionContext,System.Threading.ContextCallback,object,bool)
System.Threading.ExecutionContext:Run (System.Threading.ExecutionContext,System.Threading.ContextCallback,object,bool)
System.Threading.QueueUserWorkItemCallback:System.Threading.IThreadPoolWorkItem.ExecuteWorkItem ()
System.Threading.ThreadPoolWorkQueue:Dispatch ()
System.Threading._ThreadPoolWaitCallback:PerformWaitCallback ()

Patch_81mm.zip

Submitted patch request to CE team, waiting for more info.


Out of Date?

This page appears out of date w.r.t. the packages it is referencing. Those packages appear to work with the storage APIs (table, blob, queue) but not with service bus. The namespaces are wrong (latest version uses MicrosoftAzure over WindowAzure). So I'm a bit confused here. We're looking to go a bit more cloud native and use ServiceBus where we were previously using RabbitMQ. I'm still struggling to find how to connect to a service bus instance from PHP using the mentioned package. I've got "microsoft/azure-storage": "^0.19.1" but ServicesBuilder::getInstance()->createServiceBusService is not a function in that package. super thanks. bye for now.

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

ID: 0ed2dc56-1991-9577-fd91-11d0307b9647
Version Independent ID: 488c94c8-ccfc-81b7-d291-95e965b6691f
Content: How to use Service Bus queues with PHP
Content Source: articles/service-bus-messaging/service-bus-php-how-to-use-queues.md
Service: service-bus-messaging
GitHub Login: @axisc
Microsoft Alias: aschhab

@philrowan-mtb Thank you for your feedback! We will review and update as appropriate.

@philrowan-mtb Just checking if you have gone through this GitHub samples and see if that helps.

@DixitArora-MSFT Thank for the link. I did just run through those samples and got connected. Here is some info. Using composer package  "microsoft/windowsazure": "^0.5.7"
simple php test file looks like:
<?php
echo 'trying to connect to service bus';

require_once '../vendor/autoload.php';

use WindowsAzure\Common\ServicesBuilder;
use WindowsAzure\ServiceBus\Models\QueueInfo;
use WindowsAzure\ServiceBus\Models\BrokeredMessage;

// Change sb:// to https:// protocol as cURL does not know what to do with sb protocol
$connectionString = "Endpoint=https://[Your Endpoint Url]/;SharedAccessKeyName=RootManageSharedAccessKey;SharedAccessKey=[Your Key]";
$serviceBusRestProxy = ServicesBuilder::getInstance()->createServiceBusService($connectionString);

try {
    $queueInfo = new QueueInfo("test-q");

    // Create queue.
    $serviceBusRestProxy->createQueue($queueInfo);
    echo 'created queue "test-q"';
}
catch(ServiceException $e){
    // Handle exception based on error codes and messages.
    // Error codes and messages are here:
    // https://docs.microsoft.com/rest/api/storageservices/Common-REST-API-Error-Codes
    $code = $e->getCode();
    $error_message = $e->getMessage();
    echo $code.": ".$error_message."
";
}

try    {
    // Create message.
    $message = new BrokeredMessage();
    $message->setBody("my message");

    // Send message.
    $serviceBusRestProxy->sendQueueMessage("test-q", $message);
    echo 'dropped a test message in queue "test-q"';
}
catch(ServiceException $e){
    // Handle exception based on error codes and messages.
    // Error codes and messages are here:
    // https://docs.microsoft.com/rest/api/storageservices/Common-REST-API-Error-Codes
    $code = $e->getCode();
    $error_message = $e->getMessage();
    echo $code.": ".$error_message."
";
}

I can connect and drop messages in the queues now. Gonna play around with receiving now and topics/subscriptions. Thanks.

@philrowan-mtb Thanks for the update. Could you point out the section of the document specifically in which composer package reference is incorrect.

looks like line 40 in https://github.com/MicrosoftDocs/azure-docs/edit/master/articles/service-bus-messaging/service-bus-php-how-to-use-queues.md
that include is referencing some other markdown for azure storage package which doesn't work with service bus by the looks of it. That included file has "microsoft/azure-storage": "*" but the service bus client is in "microsoft/windowsazure": "*".

@DixitArora-MSFT it looks like this dead-ended quite a while ago but I think these 2 are related: https://github.com/MicrosoftDocs/azure-docs/issues/7505. I am guessing that Azure Storage (blob, table, queues) was included with the general SDK at some point (which also included the service bus proxy and possibly other Azure stuff). even "microsoft/azure-storage" is reported as deprecated (abandoned even Package microsoft/azure-storage is abandoned, you should avoid using it. Use microsoft/azure-storage-blob;microsoft/azure-storage-queue;microsoft/azure-storage-table;microsoft/azure-storage-file instead) by composer and recommends that the individual microsoft/azure-storage-[queue|blob|table|file]" be installed instead. I'm guessing at some point these were split and the docs just didn't get updated or something. IMO, and base don the comments from @boredom2 maybe a works with section would be useful. Like "microsoft/azure-storage works with service bus" and "microsoft/azure-storage-queue works with storeage queues".

@philrowan-mtb I have assigned this to content author to review and update as appropriate.
@spelluru Just checking if you can provide some input here.

Thank you @DixitArora-MSFT
I will work with the product team to fix the article as soon as I can.
Thank you @philrowan-mtb for reporting the issue. Appreciate it.

@axisc - we need to update the PHP quick start and perhaps publish a sample on GitHub

@spelluru this came up again today via twitter:
https://twitter.com/akrabat/status/1143881720879562754
The official PHP azure-storage library is marked as abandoned and I can't find an official tutorial for whatever replaces it…
https://twitter.com/akrabat/status/1143899698215948288
"Yes. I want to use a Service Bus queue. The "Note" in the README says: "If you are looking for the Service Bus, Service Runtime, Service Management or Media Services libraries, please visit (link: https://github.com/Azure/azure-sdk-for-php) github.com/Azure/azure-sd….", so I'm lost…"

#assign:axisc

axisc is already assigned.

Fixed
#please-close

Fixed
#please-close


Unable to Resolve node modules ?

I have Downloaded Sample file and I have installed Node modules but it's not working ..showing this image

Hi @lavarajallu
Please check you import path and try to refresh packager cache.
I don't think it's the library that causing you troubles.

Hello Thanks for your response @myaskevich i gave a exact path and install package also...


Allow logstream to stream logs from user namespace

When streaming logs from user namespace, disable filtering lines so all lines are printed.

More information on when this comes handy is available in Serving PR: https://github.com/knative/serving/pull/13293, together with usage example.
It will be very handy also in knative/eventing upgrade tests where multiple pods are started and the Sender pod is removed before the test ends.

Changes

/kind

Fixes #

Release Note

Docs

/retest

The test failures look unrelated.

/retest

Rebased. Just to be sure this works after some time.
@dprotaso Could you review this please? There's a commit which haven't been reviewed. See comment

/retest
/lgtm
/approve

@dprotaso Thank you!


Form errors are not displayed when initially set with errors prop (regression)

Reproduction
https://formkit.link/8d129abdcf48d433a06af954eac5623f
Describe the bug
Errors should be showing initial when the errors prop contains errors, but they are not. This change was introduced in 1.5.2
Environment
1.5.2

This should be fixed in 1.5.3


Query Operators bug: does not support contains

Bug Report

Steps to Reproduce

Query Operators bug: does not support contains

Other Details

Hey @zhounewz using graphql with relationships is a known limitation and is on our roadmap to tackle. You can read more about it in the discussion here! 👍
I will be closing this issue, since we will be addressing it at a later date.


Misleading error when trying to list commits for nonexistent repository

When running lakectl log on a nonexistent repository, a misleading message appears.
Example:
lakectl log lakefs://notarepo@master
Output:
Error executing command: branch 'master' not found
Expected:
Error executing command: repo 'notarepo' not found

Hey @guy-har , I would like to work on this :)

@kzuri All yours :)

Is anyone working on this?

@kzuri  do you want to proceed with this, or would you prefer that I re-assign to @sarathsp06

@guy-har Sorry for inconvenience. My laptop is at repair shop so couldn't work on this issue. Ya sure go ahead and reassign @sarathsp06 on this issue

If nobody is working on it, then you can assign it to me

@sarathsp06 do you want me to assign it to you? or should I assign it to @Sufiyan1997 ?

@guy-har I can work on this

@sarathsp06 all yours, let me know if you need any help.

@guy-har  had a brief look at the code, there are multiple options

log "executing command: 'repo/branch' not found" ( least code change )
log "executing command: repo 'repo-name' not found" ( code change and mostly one more db/cache  read  request just for log info)

-- what would you suggest

@sarathsp06 made a PR with the second option above please do have a look. I am not familiar with the design philosophies of the repo thus offered two approaches

@sarathsp06 sounds great, I would have suggested the second option.
I will check it out soon.
Thanks for contributing 💪 .


With multiple players, force only one player to be playing

After going through many plugins, this is the best media player for React I've found so thank you!
I'm using this just for audio with the File player type. I have a playlist which is just a list of multiple <ReactPlayer /> elements and I'd like to force it to only play one player at a time. Thus, if a player is playing and you press play on another player, then the first player stops. Currently, it will play any and all players simultaneously.
Looking through all of the issues I couldn't find a reference to this so my apologies if this is a duplicate.
How can I force only one player to play on the page at any given time?

In the container component holding all your <ReactPlayer>s components, you would have to keep track of which player is playing. You could possibly use array indexes as ids for your child <ReactPlayer>s, if you're not doing anything to the array that would change the order of the child <ReactPlayer>.
The <PlayerContainer> component should have a method for changing its state that keeps track of the id of the currently playing <ReactPlayer>,  pass playing={false} (or just not pass a playing prop) to every other child after state changes.

@superbuggy That's easy enough - thanks.

can you give an example of it

@superbuggy @joshuaiz  hi, i had similar problem. how did you do it? i made a working demo but can't get it to work https://codesandbox.io/s/thirsty-stonebraker-9q3zi?file=/src/App.js

@dabonnestor @anki7d Not to take away from this project but I ended up making my own React audio component that has support for only one player playing and is fully customizable.
You can find it here: https://github.com/joshuaiz/reaudio
If nothing else, you can look through the code and see how the state is handled for only one player playing at a time.

Hi @superbuggy,
Is it possible to have  component in specific React.Context?

In the container component holding all your <ReactPlayer>s components, you would have to keep track of which player is playing. You could possibly use array indexes as ids for your child <ReactPlayer>s, if you're not doing anything to the array that would change the order of the child <ReactPlayer>.
The <PlayerContainer> component should have a method for changing its state that keeps track of the id of the currently playing <ReactPlayer>, pass playing={false} (or just not pass a playing prop) to every other child after state changes.
You'd pass this method as the onPlay to each <ReactPlayer> also.
Does that make sense?

I tried this solution but ran into a problem: ReactPlayer doesn't seem to register a state change through the YouTube iframe. Has anyone else solved this issue? It seems to not be a problem on mobile for some reason.


missing the required redirect URI

i am getting the following error when i try to run quick.php (which is named as quickstart in google drive api docs) i have copy paste the exact code but getting the error when i run it from my command line
Error:
PHP Fatal error:  Uncaught InvalidArgumentException: missing the required redirect URI in C:\xampp\htdocs\dnew\vendor\google\auth\src\OAuth2.php:650
Stack trace:
#0 C:\xampp\htdocs\dnew\vendor\google\apiclient\src\Google\Client.php(339): Google\Auth\OAuth2->buildFullAuthorizationUri(Array)
#1 C:\xampp\htdocs\dnew\quick.php(26): Google_Client->createAuthUrl()
#2 C:\xampp\htdocs\dnew\quick.php(58): getClient()
#3 {main}
  thrown in C:\xampp\htdocs\dnew\vendor\google\auth\src\OAuth2.php on line 650

Fatal error: Uncaught InvalidArgumentException: missing the required redirect URI in C:\xampp\htdocs\dnew\vendor\google\auth\src\OAuth2.php:650
Stack trace:
#0 C:\xampp\htdocs\dnew\vendor\google\apiclient\src\Google\Client.php(339): Google\Auth\OAuth2->buildFullAuthorizationUri(Array)
#1 C:\xampp\htdocs\dnew\quick.php(26): Google_Client->createAuthUrl()
#2 C:\xampp\htdocs\dnew\quick.php(58): getClient()
#3 {main}
  thrown in C:\xampp\htdocs\dnew\vendor\google\auth\src\OAuth2.php on line 650

Specifications

PHP version (php -v-7.2.1):
OS (Windows):

I am getting the following error
any help appreciated.
Fatal error: Uncaught exception 'GuzzleHttp\Exception\RequestException' with message 'cURL error 60: SSL certificate problem: unable to get local issuer certificate (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)' in C:\wamp64\www\gdsample\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php:187
Stack trace:
#0 C:\wamp64\www\gdsample\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(150): GuzzleHttp\Handler\CurlFactory::createRejection(Object(GuzzleHttp\Handler\EasyHandle), Array)
#1 C:\wamp64\www\gdsample\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(103): GuzzleHttp\Handler\CurlFactory::finishError(Object(GuzzleHttp\Handler\CurlHandler), Object(GuzzleHttp\Handler\EasyHandle), Object(GuzzleHttp\Handler\CurlFactory))
#2 C:\wamp64\www\gdsample\vendor\guzzlehttp\guzzle\src\Handler\CurlHandler.php(43): GuzzleHttp\Handler\CurlFactory::finish(Object(GuzzleHttp\Handler\CurlHandler), Object(GuzzleHttp\Handler\EasyHandle), Object(GuzzleHttp\Handler\CurlFactory))
#3 C:\wamp64\www\gdsample\vendor\guzzlehttp\guzzle in C:\wamp64\www\gdsample\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php on line 187

Closing due to inactivity.

I have this same issue.
I have a credentials.json, but it does not has an 'installed' key in it. Also, I'm unsure if my json file is made correctly.

Did you use the button on the quickstart page? It should ensure the correct settings are used.

I used type 'Web browser' and got an error
PHP Fatal error:  Uncaught InvalidArgumentException: missing the required redirect URI in /root/vendor/google/auth/src/OAuth2.php:675
https://developers.google.com/gmail/api/quickstart/php#troubleshooting says "This code requires an OAuth client ID of type Other" but 'Other' isn't offered as an option in the dropdown menu.

I used type 'Web browser' and got an error
PHP Fatal error: Uncaught InvalidArgumentException: missing the required redirect URI in /root/vendor/google/auth/src/OAuth2.php:675
https://developers.google.com/gmail/api/quickstart/php#troubleshooting says "This code requires an OAuth client ID of type Other" but 'Other' isn't offered as an option in the dropdown menu.

Make sure you have added the Redirect URI in your Client-ID:

Also don't forget to download and replace the credentials.json file after you have made any changes:

I used type 'Web browser' and got an error
PHP Fatal error: Uncaught InvalidArgumentException: missing the required redirect URI in /root/vendor/google/auth/src/OAuth2.php:675
https://developers.google.com/gmail/api/quickstart/php#troubleshooting says "This code requires an OAuth client ID of type Other" but 'Other' isn't offered as an option in the dropdown menu.

Make sure you have added the Redirect URI in your Client-ID:
Also don't forget to download and replace the credentials.json file after you have made any changes:

This was very helpful.
Thanks a lot for this


Your site copy

Hi,
Your product is very very usefull. But it'll be more usefull if you give us download  facility of your full site. Would you, please, add your whole site into the download package?
Best regards.

Hello,
Why would you want to do this ? I just want to know what you want to do and it's not clear.

Hi,
At your site's samples are more attractive. And I want to use with them just copy & paste.
At downloaded copy, I didn't find all of them.
That's all my wish.
Best regards.

Sure, the commit is there.


scheduler: host filter by cloudprovider domain id

这个 PR 实现什么功能/修复什么问题:
调度器: 拥有 cloudprovider 的 host 会根据用户输入的 domain_id 进行过滤
是否需要 backport 到之前的 release 分支:

release/2.9.0
/cc @swordqiu
/area scheduler

Codecov Report

Merging #1114 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1114      +/-   ##
=========================================
+ Coverage    7.85%   7.87%   +0.01%
=========================================
  Files         356     356
  Lines       56592   56592
=========================================
+ Hits         4445    4455      +10
+ Misses      51749   51740       -9
+ Partials      398     397       -1

Flag
Coverage Δ

#aFlag
7.87% <ø> (+0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/util/rbacutils/rbac.go
75.23% <0%> (+2.53%)
:arrow_up:

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
: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 6d6bfdb...bef1848. Read the comment docs.

/lgtm
/approve


Updated asset link to latest release

Would be nice to have the asset link get updated automatically but for now we can keep it updated manually.

Set this URL to master.zip, it should always point to the master branch. Ideally, that branch will always build/work.

Sounds good. Tks


New package: NMR v0.8.0

Registering package: NMR
Repository: https://github.com/marcel-utz/NMR.jl
Created by: @marcel-utz
Version: v0.8.0
Commit: d6277d7f0a83f8d5dce784186517f79a547f24d8
Git reference: master
Description: tools for NMR spectroscopy, data processing, and simulation in Julia

In general, we try to avoid domain-specific acronyms for package names. Personally, I think "NMR" might be commonly understood enough to be at least part of a name. So if you choose NMR<Something>, that would probably be fine. It would be good to check what other Julia packages might exist in the NMR space, compare yourself with their feature set, and figure out how you want to situate this package in that space.
It might even be possible to use just "NMR" as a package name, but I think there's a pretty high bar for that. The package would have to be a very complete NMR framework, basically the "go to" package for the entire field of NMR. That also implies exceptionally good documentation (or at least a trajectory towards that). The package should also be expected to receive long-term maintenance. It appears that you might have a permanent academic position, and that would go a very long way towards guaranteeing maintenance.
In its current form, the package definitely lacks documentation. In general, the minimum requirement for a package to be registered is a README that explains what problem the package is trying to solve and gives some indication of the features and how the package can be used, e.g., by giving a usage example. For a package name like" NMR" the bar would probably be much higher. It should have a full online Documenter-based documentation.
If you think this package will live up to the expectations implied by the name "NMR", I would say maybe use a LocalRegistry for the moment, and re-register in General once the package is a bit more mature (has full documentation). If the package should be smaller in scope, I would recommend that you change the name to something longer and more descriptive.

Dear Michael,
Many thanks for your feedback. You are certainly right that the package documentation is not yet ready for prime time. We will continue to develop it and improve its documentation, and get in touch again at a later date. For the moment, we will use a local registry. The name can be made more specific as well. The package contains functionality to both analyse experimental NMR data as well as simulate NMR spin dynamics. Would a name like NmrKIT or similar be acceptable?
Best wishes,
-Marcel
Prof. Dr. Marcel Utz
@.***

On 11. Jul 2024, at 14:59, Michael Goerz @.***> wrote:
In general, we try to avoid domain-specific acronyms for package names. Personally, I think "NMR" might be commonly understood enough to be at least part of a name. So if you choose NMR, that would probably be fine. It would be good to check what other Julia packages might exist in the NMR space, compare yourself with their feature set, and figure out how you want to situate this package in that space.
It might even be possible to use just "NMR" as a package name, but I think there's a pretty high bar for that. The package would have to be a very complete NMR framework, basically the "go to" package for the entire field of NMR. That also implies exceptionally good documentation (or at least a trajectory towards that). The package should also be expected to receive long-term maintenance. It appears that you might have a permanent academic position, and that would go a very long way towards guaranteeing maintenance.
In its current form, the package definitely lacks documentation. In general, the minimum requirement for a package to be registered is a README that explains what problem the package is trying to solve and gives some indication of the features and how the package can be used, e.g., by giving a usage example. For a package name like" NMR" the bar would probably be much higher. It should have a full online Documenter-based documentation.
If you think this package will live up to the expectations implied by the name "NMR", I would say maybe use a LocalRegistry https://github.com/GunnarFarneback/LocalRegistry.jl for the moment, and re-register in General once the package is a bit more mature (has full documentation). If the package should be smaller in scope, I would recommend that you change the name to something longer and more descriptive.
—
Reply to this email directly, view it on GitHub https://github.com/JuliaRegistries/General/pull/110853#issuecomment-2222877341, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACFXILQVJJCGFRGPEKC5ZZTZLZ6TPAVCNFSM6AAAAABKWM24M2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRSHA3TOMZUGE.
You are receiving this because you were mentioned.

You are certainly right that the package documentation is not yet ready for prime time. We will continue to develop it and improve its documentation, and get in touch again at a later date. For the moment, we will use a local registry.

Sounds good!

The name can be made more specific as well. The package contains functionality to both analyse experimental NMR data as well as simulate NMR spin dynamics. Would a name like NmrKIT or similar be acceptable?

Maybe NMRKit. "NMR" still remains an acronym and should always stay in all-caps. It's still pretty generic, though. We already have an NMRTools.jl – perhaps you can still come up with something more descriptive or more creative. For these types of framework packages, especially if you want to develop any kind of branding around them, it is often better to use the "less systematic name" approach (see point 5 in the guidelines). Something more along the lines of your colleague Ilya Kuprov's "Spinach" framework for NRM in Matlab.


"Unsafe JavaScript attempt to access frame with URL" warning after upgrade to version 1.0.0

After having upgraded to 0.8.0 I've now started to sometimes get the following warning:
[10:24:39] Starting 'test:frontend:lib'...
  json2
    ✓ stringify and parse
  moment
    ✓ format strings
  underscore
    ✓ each
    ✓ filter
    ✓ clone
    ✓ contains
    ✓ extend
    ✓ findWhere
    ✓ indexOf
    ✓ isEmpty
    ✓ memoize
    ✓ template
    ✓ without
  13 passing (43ms)

Unsafe JavaScript attempt to access frame with URL about:blank from frame with URL file:///D:/MyDev/ljs_app/trunk/periscope/build/node_modules/gulp-mocha-phant
mjs/node_modules/mocha-phantomjs-core/mocha-phantomjs-core.js. Domains, protocols and ports must match.

Unsafe JavaScript attempt to access frame with URL about:blank from frame with URL file:///D:/MyDev/ljs_app/trunk/periscope/build/node_modules/gulp-mocha-phant
mjs/node_modules/mocha-phantomjs-core/mocha-phantomjs-core.js. Domains, protocols and ports must match.

Unsafe JavaScript attempt to access frame with URL about:blank from frame with URL file:///D:/MyDev/ljs_app/trunk/periscope/build/node_modules/gulp-mocha-phant
mjs/node_modules/mocha-phantomjs-core/mocha-phantomjs-core.js. Domains, protocols and ports must match.

Unsafe JavaScript attempt to access frame with URL about:blank from frame with URL file:///D:/MyDev/ljs_app/trunk/periscope/build/node_modules/gulp-mocha-phant
mjs/node_modules/mocha-phantomjs-core/mocha-phantomjs-core.js. Domains, protocols and ports must match.

[10:24:40] Finished 'test:frontend:lib' after 1.66 s

Gulpfile.js
gulp.task('test:frontend:lib', function (/*callback*/) {
	return gulp.src('test/frontend/lib/runner.html').pipe(mochaPhantomJS({'reporter': 'spec'}));
});

runner.html
<!doctype html>

		<meta charset="utf-8">
		<meta http-equiv="X-UA-Compatible" content="IE=edge">
		<meta name="viewport" content="width=device-width, initial-scale=1.0">
		<title>Mocha Test Runner</title>
		<link rel="stylesheet" href="../../../libs/mocha/mocha.css">
		<link rel="stylesheet" href="../../../dist/css/core.css">

You copied and pasted from gulp-mocha-phantomjs#38 that you opened, so we'll continue the discussion there, as mocha-phantomjs-core is not in charge of the phantomjs version. (the bug your seeing is in phantomjs - ariya/phantomjs#12697)


Can we delete the wip branch?

I think the wip branch is stale for a very long time now. Can we delete the branch?

Definitely


pass through function for color fill instead of just array

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Resources for later:
https://stackoverflow.com/questions/41848677/how-to-make-a-color-scale-in-d3-js-to-use-in-fill-attribute
https://github.com/d3/d3-scale-chromatic
The fill variable is currently
fill: Array(1) [   0: Object {curve_name: "RHOB", fill: "yes", fill_direction: "left", cutoffs: Array(3), fill_colors: Array(3), curve2: ""} ]
and takes string....what will need to be changed for it to also take a function in place of the string like:
var colors = d3.scaleQuantize()     .domain([0,50])     .range(["#5E4FA2", "#3288BD", "#66C2A5", "#ABDDA4", "#E6F598",      "#FFFFBF", "#FEE08B", "#FDAE61", "#F46D43", "#D53E4F", "#9E0142"]);


HTML hidden Attribute does not work with "form-input-hint"

HTML hidden Attribute - http://www.w3schools.com/TAgs/att_global_hidden.asp
does not work with "form-input-hint", because it conflicts with "display: inline-block;" property of "form-input-hint" class, many library depends on this Attribute for error message handling - like AngularJS
<span hidden class="form-input-hint">The name is invaild.</span>

I have updated Spectre to version 0.1.29 and the issue should have been fixed. Thanks.

@picturepan2 Awesome, thanks again.


Add support for ValueConverters in OwnedTypes

Currently if there is an OwnedType with property, that has ValueConverter set, the ValueConverter is not being used.
It is related to this PR where ValueConverters support was introduced: GH-47

Thx for Contrib.


JSON handling for non application/json content types

Some useful REST endpoints don't return with a content-type of application/json while still returning valid JSON, preventing the use of the library's built-in JSON parsing/processing.
I ran into this while working with the freely available US National Weather Service forecast API, which can only respond with GeoJSON (application/geo+json) or JSON LD (application/ld+json).
As a workaround in my project I've just called directly into the MagTag object's network.fetch_data(url) to get a response string to parse and wrangle but it'd be nice to do it in a more elegant way with these types of endpoints.

Transferred issue here because that's where the applicable code now resides.

I did a big refactor with #8 and there is now an add_json_content_type() function.

I did a big refactor with #8 and there is now an add_json_content_type() function.


gfx_hal::memory::cast_slice can cause UB

I was pointed to use cast_slice to easily send push constants to a pipeline, but upon inspection it became clear that the method doesn't respect alignment and can easily trigger UB.
Corrected code is as follows:
pub fn cast_slice<T: Pod, U: Pod>(ts: &[T]) -> Option<&[U]> {
  use core::mem::{align_of, size_of};
  // Handle ZST (this all const folds)
  if size_of::<T>() == 0 || size_of::<U>() == 0 {
    if size_of::<T>() == size_of::<U>() {
      unsafe {
        return Some(core::slice::from_raw_parts(
          ts.as_ptr() as *const U,
          ts.len(),
        ));
      }
    } else {
      return None;
    }
  }
  // Handle alignments (this const folds)
  if align_of::<U>() > align_of::<T>() {
    // possible mis-alignment at the new type (this is a real runtime check)
    if (ts.as_ptr() as usize) % align_of::<U>() != 0 {
      return None;
    }
  }
  if size_of::<T>() == size_of::<U>() {
    // same size, so we direct cast, keeping the old length
    unsafe {
      Some(core::slice::from_raw_parts(
        ts.as_ptr() as *const U,
        ts.len(),
      ))
    }
  } else {
    // we might have slop, which would cause us to fail
    let byte_size = size_of::<T>() * ts.len();
    let (new_count, new_overflow) = (byte_size / size_of::<U>(), byte_size % size_of::<U>());
    if new_overflow > 0 {
      return None;
    } else {
      unsafe {
        Some(core::slice::from_raw_parts(
          ts.as_ptr() as *const U,
          new_count,
        ))
      }
    }
  }
}

Please note that this version returns an Option<_> instead of immediately doing a panic, and the end user can handle the Option layer in whatever way they feel is appropriate.

I'll chime in here, instead of creating a new issue.
Is there a reason for CommandBuffer::push_*_constants to accept &[u32] instead of &[u8] in the first place? Vulkan doesn't have any requirements on size or alignment (and indeed, cast_slice is used to transmute passed in &[u32] back to &[u8] in vulkan backend), neither does Metal (assuming I'm reading it right) - it also just transmutes the data to the Metal-native buffer (type -> docs) and has no alignment requirement either.
Having non-byte-sized types leads to confusion as demonstrated in #2552, and might easily happen externally as well.
Having the functions accept &[u8] still requires user to transform the data from the preferred format, but much more types can be converted easily to &[u8] than to &[u32] (by virtue of having Write traits for example). Users would still often need to transmute data (either via std::mem::transmute or with cast_slice helper method, if it's still around by that time), but with &[u8] it doesn't have these subtle alignment pitfalls that are very easy to overlook.
Changing function signature is, of course, a breaking change, but given that 0.2.0 would have breaking API changes anyway, that's not a big deal?
Does the above make sense? Should I refactor this into an issue of its own?


Jsdom 6.5.1

This is an upgrade of jsdom from 5.3.0 to 6.5.1, related to the issue #939.
It brings 1 breaking change:

Element.setAttribute now expects two arguments, even for setting properties (such as "checked", "disabled"...): https://developer.mozilla.org/fr/docs/Web/API/Element/setAttribute

ah. this is red because of a linting issue in a test file.
FYI this is not reported by npm lint
@assaf do you mind fixing the typo or do you want me to prepare another pull request ?

I get the same error
[10:43:29] 'lint' errored after 3.05 s
[10:43:29] ESLintError in plugin 'gulp-eslint'
Message:
    Unnecessary { after 'for-of'.
Details:
    fileName: test/forms_test.js
    lineNumber: 1431
[10:43:30]
test/jquery_compat_test.js
  17:3  error  Unnecessary { after 'for-of'  curly

So... Can we get this pull-request merged anyway? I think those linting issues should be fixed on the master branch independently.

why jsdom 6.5.1, when recently version is 7.2.0 ?

@sintanial if you look at commits it's up to 7.0.2

@z-vr oO sorry, is was blind :)

@z-vr already is not the last version :)

It shouldn't be 7.0.2... it's not working yet ! I thought I based my PR on my 6.5.1.
I'll look at that when I'll be back to my office,  tomorrow.
So, why 6.5.1 only? Because the 7 branch rehauled the XmlHttpRequest implementation which is patched by zombie. And I need @assaf (or any one) help to continue upgrading jsdom. However the 6.5.1 already brings interresting changes compared to the current version.

If you feel to fix things broken jsdom 7.0.2, don't hesitate to fork my repo ;-) jsdom 7.x is really interesting. I would definitely appreciate to get this version!

@vcarel unfortunately i don't know jsdom, and never hear of it until today :)

I didn't know zombie before. It's never too late and I'll be grateful
forever ;-)
Le 10 déc. 2015 19:45, "Roman"<EMAIL_ADDRESS>a écrit :

@vcarel https://github.com/vcarel unfortunately i don't know jsdom, and
never hear of it until today :)
—
Reply to this email directly or view it on GitHub
https://github.com/assaf/zombie/pull/982#issuecomment-163714229.

@vcarel research in progress ;)

All right, I issued a new PR so to not include the commits related to 7.0.2 ->  #990


When extracting orders i am getting Savon::HTTPError.

Problem:
When extracting orders i am getting this error.
'raise_soap_and_http_errors!': HTTP error (500) (Savon::HTTPError)
Steps to Reproduce:
1.Checkout the master branch
2.Run ./shopify_transporter export --config=../config.yml --object=order > orders.json
3 See below error

@daveshawsa thanks for the quick response.
I already don't include 'www'.

The above command is working fine for less number of records, for instance for 15000 orders it is working fine but above that, it is producing the above error https://github.com/Shopify/shopify_transporter/issues/35#issue-360810082.
I think there is a memory issue here as well, as in case of #15 but not sure why it is not throwing a memory related exception.

@gopalvirat for both of these issues I would suggest trying some of the code in the https://github.com/Shopify/shopify_transporter/tree/extraction-scripts-by-daterange branch. There is a readme for this at https://github.com/Shopify/shopify_transporter/blob/extraction-scripts-by-daterange/sample_scripts/third_party_platform_exporting/magento/v1.9/soap_api/README.md
This will be merged with the primary tool soon, but it needs some more work before it can be fully integrated.
I'm suspecting that because there are a lot of customers, the server is not able to handle the SOAP request.
Feel free to comment here if you are running into issues with the other branch.

@surajreddy when trying to export magento customers using the customers.rb script, I get the following error:
response.rb:90:in raise_invalid_response_error!': Unable to parse response body: (Savon::InvalidResponseError)`
Exporting orders worked fine. Thanks for these features! I'm using the extraction-scripts-by-daterange branch.

@chrismarkpd I'm glad the orders export worked!
Do you mind opening a new issue for the problem with the customers.rb script? Are there more lines printed in the error? If so, can you share those in that issue as well?

@surajreddy opened up a new issue and shared more lines. Thanks!

@gopalvirat the newest version of the gem now supports extracting data: https://github.com/Shopify/shopify_transporter/releases
We've added support for retrieving data in batches in the newest version of the gem, and that should get past this error. Please re-open this issue if you are running into the error again.


time format fixes

Fixing lurking “milliseconds from epoch” times (has been done before, but somehow crept in again due to branching madness)

Covered here.

@david4096 @kozbo
Yes; & thanks for the reminder of an older PR which was based on my own comments https://github.com/ga4gh/schemas/issues/637.
But then, it is kind of confusing which things are in the pipeline & which are just forgotten / victims of branching madness ...
Proposal (involves work!): Push notes into master describing the accepted changes, which are just held back due to custodial concerns.
Example:
 // The time at which this read group was created in milliseconds from the epoch.
   int64 created = 9;
// TODO (accepted): The :ref:`ISO 8601<metadata_date_time>` time at which this record was created.
// TODO (accepted): string created = 15;

Closing in favour of https://github.com/ga4gh/schemas/pull/690.


Error in running output_collect

Project Robyn
Error running this line:
OutputCollect <- robyn_outputs(

InputCollect, OutputModels
, pareto_fronts = 1 # decrease pareto_fronts to get less output models
, calibration_constraint = 0.1 # range c(0.01, 0.1) & default at 0.1. Details see ?robyn_outputs
, csv_out = "pareto" # "pareto" or "all"
, clusters = TRUE # Set to TRUE to help reduce and select best models based on robyn_clusters()
, plot_pareto = TRUE # Set to FALSE to deactivate plotting and saving model one-pagers
, plot_folder = robyn_object # plots will be saved in the same folder as robyn_object
)

The following is the error I get:
Using robyn object location: C:/Users/ttodor/OneDrive - Vericast/Documents/Desktop
Provided 'plot_folder' doesn't exist. Using default 'plot_folder = getwd()': C:/Users/ttodor/OneDrive - Vericast/Documents/R_Studio_Projects/JNJ_01_2022

Running Pareto calculations for 10000 models on 1 front...
Error in :=(trial, x$trial) :
Check that is.data.table(DT) == TRUE. Otherwise, := and :=(...) are defined for use in j, once only and in particular ways. See help(":=").

Environment & Robyn version
R version 4.1.2 - Windows 10

FYI. I have upgraded to the latest version of Robyn


[docs] Update Wertarbyte components link

Now that v1.0.0 is not really "next" anymore, we have updated our domains.
PR for the 0.x link is incoming, too…

I don't see how this change is related to the swipeable drawer. :thinking:

@leMaik Yeah, this failing test is weird.

@oliviertassinari I'm working on the swipeable drawer for #11362 anyway.


ROS2 Porting: multi_object_tracker

multi_object_tracker
Summary
Convert the multi_object_tracker package to ROS2. A large but straightforward change with a few caveats which is why this PR is currently a draft.
Important (API) Changes:

Convert DataAssociation::assign method return type to void due to -Wreturn-type error
Pass rclcpp::Time by reference to TrackerBase::getElapsedTimeFromLastUpdate to perform duration calculation

Cosmetic Changes:

Adding #ifndef guards on all header files and removing #pragma once
Standardise use of <> vs "" in #include
Convert .h extension to .hpp
Typos

Testing

Checkout this branch and enter docker/ade environment

Build and source the repository

colcon build
. install/setup.bash

Launch file should launch without error

ros2 launch multi_object_tracker multi_object_tracker.launch.xml

Discussion Points
lookupTransform
The method lookupTransform in tf2::Buffercore no longer allows specification of lookup duration. At the moment I have left this implement to perform the lookup transform without a duration specification, see issue here for further discussion.
Update: This has been implemented using the suggestion provided in the issue and taking inspiration from https://github.com/ros2/geometry2/blob/foxy/tf2_ros/test/test_buffer.cpp#L140-L158.
uuid_msgs and unique_id Packages
This packages contains these dependencies however they have been ported (though the state of port is unclear) but not released on the ROS2 package index. I have commented on an issue tracking its progress regarding any further porting efforts.
Update: This has been implemented using the unique_identifier_msgs package which has a release available on foxy. I removed any references to Boost, using https://github.com/ros2/unique_identifier_msgs/blob/foxy/msg/UUID.msg instead. A different method of generate UUIDs using standard libraries was implemented instead of using the generate method in the unique_id package. Some references:

https://github.com/ros2/rclcpp/blob/31c202e325476f6e912d6d23c17d13393156dbf4/rclcpp_action/src/client.cpp#L377-L379
https://lowrey.me/guid-generation-in-c-11/

Apologies @jilaada, I missed when this PR switched from draft to ready for review. I will review it today.

Apologies @jilaada, I missed when this PR switched from draft to ready for review. I will review it today.

No problem, I'm still looking into the transform issue anyway.

Apologies @jilaada, I missed when this PR switched from draft to ready for review. I will review it today.

No problem, I'm still looking into the transform issue anyway.

@jilaada Are you looking for something like this https://github.com/fred-apex-ai/autoware.iv.universe/commit/32f75580b6c6c445d2424261a7fc7726f3db3e75#diff-30dcb55e06b0d11249fb5d74414c4c96ceb4b891a18142afed00452ed9ef2ec3R145?

No problem, I'm still looking into the transform issue anyway.

@jilaada Are you looking for something like this fred-apex-ai@32f7558#diff-30dcb55e06b0d11249fb5d74414c4c96ceb4b891a18142afed00452ed9ef2ec3R145?

Actually quite similar. In your case you don't need to propagate the transform and use it, you're just waiting to receive the transform to continue the execution process. In my case, I am trying to find an equivalent to a lookupTransform where the duration is set and the execution cycle is blocked until the transform is received. Here is what I have currently: https://github.com/tier4/Pilot.Auto/pull/24/files#diff-1769717c3f36f32c9c336458b8968ddb2285a9a7794ce8004f308a0892783633R77-R88

@nnmm I have addressed your PR comments. Please take a look at my changes again when you get a chance. Regarding the Eigen issue; I have the minimum set of CMake commands to find the Eigen library. Removing any of these will cause the error output I commented earlier. Regarding the for loops, see my comment here: https://github.com/tier4/Pilot.Auto/pull/24#discussion_r512392210


URL shortener Extension

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I'm always frustrated when I need to share long URLs, as they can be unwieldy and prone to errors when copying and pasting. Long URLs also take up a lot of space in messages and social media posts, making them look cluttered and less readable.
Describe the solution you'd like
A clear and concise description of what you want to happen.
A URL shortener extension that allows me to quickly and easily convert long URLs into shorter, more manageable links. The extension should provide a simple interface where I can paste a long URL and instantly get a shortened version that I can copy and share. Additionally, it would be useful if the extension offers tracking capabilities to monitor link clicks
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Using online URL shorteners, but they require navigating to a separate website and can be cluttered with ads.
Additional context
Add any other context or screenshots about the feature request here.
What problem is this feature trying to solve?
This feature aims to simplify the process of sharing long URLs by converting them into shorter, more manageable links. It addresses the need for a quick, reliable, and user-friendly method to shorten URLs, improving the aesthetics and functionality of shared links.
How do we know when the feature is complete?

Hello, I would like to work on this. could you please assign it to me under GSSOC'24

Issue will be assigned. First basis
Also this Issue added to my repository #216
Kindly raise new Issues


Replace usages of deprecated pointer functions

How to categorize this PR?

/kind cleanup
What this PR does / why we need it:
Similar to https://github.com/gardener/gardener/pull/9107
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note:

NONE

/approve


I need to be able to change a character speed when he enters forced walk

Added speedMultiplier in moveDefinitions for the cutsceneAction


Add flash_get_unique_id to hardware_flash

See https://github.com/raspberrypi/pico-examples/issues/6
This adds a function flash_get_unique_id which retrieves the unique identifier from external NOR flash. This is suitable for use as a unique board ID.
Also adds some lower-level flash functions for sending arbitrary commands to the SSI, but these are kept static.

Makes sense. I guess there is no overlap here with actual flash functionality. Will make that change.

Makes sense. I guess there is no overlap here with actual flash functionality. Will make that change.

The generic command API would be nice too. We'd be able to use it to get the size of the flash.

The generic command API would be nice too. We'd be able to use it to get the size of the flash.

Thanks @tannewt for squinting at the flash code, I want to get the pico_ library in before merging so that we can update the example code to use that, but it's good that the flash part looks solid. The IRQ problem that @kilograham pointed out is a serious gotcha. Just trying to keep a lot of plates spinning right now :)

Thanks @tannewt for squinting at the flash code, I want to get the pico_ library in before merging so that we can update the example code to use that, but it's good that the flash part looks solid. The IRQ problem that @kilograham pointed out is a serious gotcha. Just trying to keep a lot of plates spinning right now :)

Think this is ready but @kilograham  could you please look at the CMake changes in particular

Think this is ready but @kilograham  could you please look at the CMake changes in particular

Thanks, both commits look good

Thanks, both commits look good


feat: make insert_input_note() update the note if it already exists

I wonder if we should make insert_input_note() behave more like "upsert" - i.e., it will insert a new note record, or if the note recored already exists, it will update it with new information. I think it will make the logic a bit more clear and it may end up being more efficient as we'd avoid multiple round trips to the store.

Originally posted by @bobbinth in https://github.com/0xPolygonMiden/miden-client/pull/449#discussion_r1720923401

Addressed in #520 with the addition of upsert_input_note


Requires in Packagist

I am creating a new repo that relies heavily on your loader.
Everything seemed to work fine in the root, when I was developing my project, but the plugin does not seem to be able to recognize in my packagist package.
https://packagist.org/packages/mongular/mongular

The plugin must be installed in global mode (not in the require section in your composer.json project).
$ composer global require fxp/compaser-asset-plugin:@dev

In your composer.json project, you must added the @dev flag for each dependencies that use the dev-master version, or put the minimum-stability option to dev (it's the standard behavior of Composer).

Thank you!  As a heads up, from your input I created these install instructions which seem to work fine for  me.
https://github.com/jasonrichardsmith/mongular/wiki/Installation


Verify potential concurrency issue of repository calls within application services

There might be bugs when multiple repo calls happen within single application service calls (which are expected to be single transactions).
Should we include some notion of repository transaction management within application service hosts?

Should all RPC and event handlers be idempotent?

Potential issues:

ParticipationServiceHost.getActiveParticipationInvitations calls both getParticipationInvitations and getParticipantGroupList on ParticipationRepository.
ProtocolServiceHost.updateParticipantDataConfiguration calls both getByOrThrow and getVersionHistoryFor on StudyProtocolRepository.

For both of these, the repo can't become inconsistent, but the returned value may be, or the request could fail possibly. This is less critical than if there would have been two update calls, which could result in inconsistent data being stored in the repo.
Therefore, I will keep this issue open so we are aware about these two specific instances, but I will remove it from the 1.0 milestone. These can be patched and wouldn't result in API changes.
P.s. there are also calls to dependent services, but this is a different issue than this one: https://github.com/cph-cachet/carp.core-kotlin/issues/295

At the time of writing, there are no application service calls which contain two update calls. The potential reported issues above are the only remaining potential issues. To handle this, a unit of work pattern will need to be implemented for repos: #417
With the fix of #398, out-of-date repo updates can now also be handled by implementing infrastructures.


Test hangs with certCheck: true

When you checkCert: true the following test hangs.
/*jshint expr: true*/
"use strict";
var chai = require("chai");
var chaiAsPromised = require("chai-as-promised");
chai.use(chaiAsPromised);
var sinonChai = require("sinon-chai");
chai.use(sinonChai);
var chaiString = require('chai-string');
chai.use(chaiString);
var expect = chai.expect;
chai.config.includeStack = true;
var mockHelper = require("./helpers/mock_helper");
var sinon = require("sinon");
var express = require('express');
var request = require("supertest-as-promised");
var bodyParser = require('body-parser');
var path = require('path');

describe("Alexa", function() {
  var Alexa = require("../index");
  var mockRequest = mockHelper.load("intent_request_airport_info.json");

  describe("app", function() {
    var app;
    var testServer;
    var testApp;

    beforeEach(function() {
      app = express();
      app.use(bodyParser.json());
      app.set('views', path.join(__dirname, 'views'));
      app.set('view engine', 'ejs');
      testApp = new Alexa.app("testApp");
      testServer = app.listen(3000);
    });

    afterEach(function() {
      testServer.close();
    });

    context("#express with checkCert=true", function() {
      beforeEach(function() {
        testApp.express({
          expressApp: app,
          router: express.Router(),
          checkCert: true,
          debug: false
        });
      });

      it("checks cert header", function() {
        return request(testServer)
          .post('/testApp')
          .set('signaturecertchainurl', 'dummy')
          .set('signature', 'dummy')
          .send(mockRequest)
          .expect(401).then(function(res) {
            expect(res.body.status).to.equal("failure");
            expect(res.body.reason).to.equal("signature is not base64 encoded");
          });
      });
    });
  });
});

A repro without alexa-app-server in https://github.com/alexa-js/alexa-app/issues/152.

I believe that the alexa-verifier-middleware needs to be loaded before any body parsers.

@tejashah88 Maybe, but I'd like to understand why.

Basically, the middleware needs to read and process the entire request body in order for the validation to work, and express doesn't expose that on the request object directly. If you look at alexa-js/alexa-verifier-iddleware/index.js#L23, the middleware accumulates the whole data of the request before any processing happens.

@tejashah88 Are you saying we should be doing this ourselves in express? Like https://github.com/alexa-js/alexa-app/pull/155?

Also I still don't quite understand why it hangs and doesn't just fail. Why aren't any of the data and end callbacks invoked?

I think @mreinstein explained it really well here.


[PE-D][Tester E] Incorrect addition of Birthday attribute with invalid Date

Describe the bug
Application succesfully adds an entry with invalid Date (year value) value
To reproduce
addAttribute /[uuid here] /birthday 0000-10-10
Expected Behaviour
Given the invalid year value, there should be an error message indicating said invalid input
Screenshots

Labels: severity.High type.FunctionalityBug
original: macareonie/ped#6

not going to fix as technically still valid and gene-nie ideally to be used as a family storage device as well

add this to UG as well

added


Counter start position

Rather than the current hard-coded value of zero, would it be possible to add another variable to the sdfTagTool code to specify the starting number from which to begin iterating over?
For instance:
sdfTagTool.csh -in .sdf -counterTag counter -addCounter -start 50 -out .sdf
@jw-feng

If we which to do so I would suggest that we call it something like -counterStart to make it more clear what this is referring to. An alternative would be to use a sdfGroovy to increment the counter by eg. 50.
Would that work for you? I would hesitate to make the options in the tag tool more complicated unless you feel this is a frequent use case.
Alberto

@bgobbi
I just implemented -counterStart in SdfTagTool.java in the autocorrelator package.   I also added a markAsRepeatTag option to mark repeated records by tag.  I created a branch named sdfTagTool, committed my changes, and created a pull request.  Can you review and approve the pull request?


Use fs.state instead of fs.lstat when enableSymlinks is false

Summary
When enableSymlinks  property is false, we must use fs.stat for getting file information in node crawler.
Test plan

Updated  expect(fs.lstat) to expect(fs.stat)

Codecov Report

Merging #12081 (ca70153) into main (7bb400c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main   #12081   +/-   ##
=======================================
  Coverage   68.88%   68.88%
=======================================
  Files         324      324
  Lines       16669    16669
  Branches     4814     4814
=======================================
  Hits        11483    11483
  Misses       5153     5153
  Partials       33       33

Impacted Files
Coverage Δ

packages/jest-haste-map/src/crawlers/node.ts
94.17% <100.00%> (ø)

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 7bb400c...ca70153. Read the comment docs.

Thanks for the PR!
This doesn't seem right to me - we want to resolve the symlinks when they're not enabled, and keep them if they are enabled (see https://github.com/facebook/jest/pull/9351/files#diff-f8f430c6a0210c0afd3d956b5430ee2fc192fb77a0191228f91d8b9c0f66a7a0R102-R104 where the condition was introduced).
/cc @mrmeku

I agree with Simon, the original behavior before I introduced the flag was to call fs.stat. The flag is an opt in behavior so we needed to keep the original behavior when the flag is false

With that in mind, I'll close this. Thanks for taking the time to send a PR @mohammadsiyou!


screensaver with guislice?

Hello,
I have an application in which I rarely have to control or operate anything on the TFT.
would it be possible to create a kind of screensaver with guislice that reacts to touching the tft?
best regards
stefan

hi,
I have tried in the meantime to implement the screensaver.
I have made an extra page with a big button in black. After a few seconds I jump to this page.
By pressing it, I jump back to my main page.
My problem and question is now:
After the jump back a part of the background is still black and not updated.
Can I somehow say that the screen area is completely updated?

hi,
I have tried in the meantime to implement the screensaver.
I have made an extra page with a big button in black. After a few seconds I jump to this page.
By pressing it, I jump back to my main page.
My problem and question is now:
After the jump back a part of the background is still black and not updated.
Can I somehow say that the screen area is completely updated?

hi,
I made a mistake and the return page was programmed as a popup page.
with a normal page call (gslc_SetPageCur(&m_gui, main page);)it works fine.

hi,
I made a mistake and the return page was programmed as a popup page.
with a normal page call (gslc_SetPageCur(&m_gui, main page);)it works fine.


Curl scalars 4.2 [help needed - tests]

#4911

@MGatner in this case I would say we need to add label "help needed". :(.
I would love to write tests as I did  year or two ago, but at the moment hard to find time for that kinda things :(.

I would love to write tests as I did year or two ago, but at the moment hard to find time for that kinda things :(.

@nowackipawel  We all completely understand time constraints. We are all volunteers stealing  time from the rest of our life to work on this. These look like good changes. Is there no way you can take a little time to learn testing? There's a better chance of getting changes in (of which you've had quite a few good suggestions over the years) if you can do the testing. Otherwise you're asking us to take time from what we see as needs to test your code. And we all have limited time to work on this.

It seems the base branch of this PR must be changed to develop.

@nowackipawel
There are conflicts.
Could you rebase your branch?
https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#pushing-your-branch

Hi, and thanks for your contribution! As this PR has been standing for a couple of months now and no effort has been seen to add the necessary unit testing to confirm the patch, we can conclude that there's no more interest in moving this forward.
We love to see and appreciate all contributions coming in from the community. However, please make it an effort to make it easy for maintainers to verify your code (that you want to be part of the framework core) through unit testing. You cannot (and we cannot, either) expect to write only source code and delegate the testing part to others. Unfortunately, that's not how OSS works.
Closing this for now. We'd still love to see you around contributing to the framework. Feel free to open a new PR once all necessary testing is in place. Thanks!


Feature Request: Add a "Create New Issue" Button in Project/Board Pages

Hi, I find the Kanban board very useful to organize issues on Gitea, however the workflow is currently not ideal - you can only create an issue from the Issues tab, and very often I find myself wanting to creating issues when looking at the Project boards as well - this causes several back and forth between Issues and Project tabs which could be avoided by having a create issue button within the project page.
Such a button would by default be assigned the "Project" variable from where you create the issue, to save a few clicks as well.

Please try on try.gitea.io, it is there (for the latest release):

I think this is a duplicate of https://github.com/go-gitea/gitea/issues/14824


Use Bitcoin Cash magic

When on the Bitcoin Cash network, we should use Bitcoin Cash magic when connecting to peers.
Relevant commits from other projects:
https://github.com/Bitcoin-ABC/bitcoin-abc/commit/ac77d27dd02bd77f566361bd8747e87316b6992a
https://github.com/Bitcoin-ABC/bitcoin-abc/commit/61ac0e7c0d6f454c2eac8c4ce0dca7876db2fc02
https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/794

Solved by https://github.com/bitcoinxt/bitcoinxt/pull/278


added timestamp to vehicle_fingerprint

so that we can keep several different recordings of different states
of the vehicle

@m1n3rva
Good idea. I just added a widget to my phone where I can trigger a snapshot of the data via SSH. This way I can record a fingerprint wherever I am, without needing a computer.
I'm quite busy right now, so I also gave you push access to the code. But with great power comes great responsibility...