fixing test script to allow clarinet 0.31.1

Made some changes in the test file; all tests do not pass yet ; some of them return different values than expected and some others have a runtime error; i plan to look into these tests next; 7 tests pass although for one of them i had to change the return values expected

Thank you so much for this fix @fanjum11
The problem is, it's no longer possible to access previous chain sessions with clarinet 0.3x
Out tests designed to run all cases in a sequence using same chain session.  This approach allows us to extend tests using previous chain updates/changes and simulate more real use case.
I'll be moving this fix into another branch and add some new utility functions to make our tests running with clarinet 0.3x

Sounds good; let me know if I can be of any other help here; i might fix the failing tests on my branch since I want them all to run.


Support Swarm imports in solidity

Outline
Embark supports special solidity imports as outlined in the documentation here.
The goal of this task is to support solidity file imports with Swarm URIs just like it's currently support for HTTP and git, for e.g
import "bzz:/someensdomain.eth/ERC725.sol";

Acceptance Criteria

It should support any valid Swarm URI
it should not use an external webservice, it should be able to access the file using swarm directly (e.g through the configured swarm node and/or a lib like web3.js or swarm-js)
If the imported sol file, has other imports, those should automatically be taken care of (existing behaviour for current http imports)

Hi @iurimatias, interested in this. Also new to your codebase. What's a good place to start digging around?

Hi @vyomshm we're excited to see your progress here! Please let us know if you have any questions as you progress.

@vs77bb Thanks. I'll submit a WIP PR as soon as I make some progress.

@vyomshm Can you let us know when to expect a WIP PR? The issue expires in 23 days, so we're looking to ensure this gets a WIP PR by this workweek so we can begin the review process 👍
Hope you're doing well!

👍 I'm on track to submit a WIP PR by this sunday.

Great to know, thanks @vyomshm 👍

yes.

yes.

updated PR #847

Thanks for the update @vyomshm! Looks like things are progressing in #986 will get payment issued as soon as merging is complete!


Fast loop option within Kivy

I'd like to be able to run a fast loop somehow (e.g. 500Hz), in parallel to Kivy's Clock set at a standard framerate (e.g. 30Hz).
In my application, (in addition to the UI) I need a very simple while loop that has the ability to read a serial port, and I'd like that while loop to run as fast as the processor will allow. But because I'm only running a RasPi, I don't want to set the whole Kivy Clock to be zero interval to achieve this because it puts the CPU at 100% all the time.
Many thanks for everything Kivy :-)

(appologies if I'm missing an exisiting workaround, I'm new)

You might want to use non-default Clock, such as Free Only Clock and Free Clock.
Or just create a new thread instead.


This PR introduces a new command line argument, --ssh-kill-command (still defaulting to sudo docker kill -s {signal} {container_id}") via which the user can customize the docker (or other compatible binary) to be executed.

LGTM


Error with latest image: "An error occurred while installing scanner script to remote server."

Getting error today after updating to the latest image:
https://gist.github.com/flatlinebb/97a93ac788858533b608a06541ed48e3
The most relevant section appears to be this:
File "/app/python/ssh/sshcp.py", line 97, in __run_command     raise SshcpError("Bad hostname: {}".format(self.__host)) ssh.sshcp.SshcpError: Bad hostname: <IP_ADDRESS>
I do not recognize this IP, it is not my docker network, or LAN.

What does your config look like?

I apologize, I made a small mistake. I do know that IP address - that's the server I'm connecting to. (I'm more used to using the domain name). So yes, <IP_ADDRESS> is the correct host IP, it is live and pingable from my docker VM.
Are you asking for the docker command I use to launch the container, or the actual settings.cfg file for seedsync?
Here is the docker command I use:
docker run -d --name=seedsync --hostname $HOSTNAME -p 8800:8800 -v /media/usb2/sabnzb/downloads/complete:/downloads -v /home/bart/docker_configs/seedsync:/config -v /home/bart/.ssh:/root/.ssh ipsingh06/seedsync
And the settings.cfg, with some stuff redacted:
`[General]
debug = False
verbose = False
[Lftp]
remote_address = <IP_ADDRESS>
remote_username = bart
remote_password = xxxxxxxxxxxx
remote_port = 22
remote_path = /home/bart/rtdata
local_path = /downloads
remote_path_to_scan_script = /home/bart/scanfs
use_ssh_key = True
num_max_parallel_downloads = 10
num_max_parallel_files_per_download = 5
num_max_connections_per_root_file = 10
num_max_connections_per_dir_file = 10
num_max_total_connections = 100
use_temp_file = True
[Controller]
interval_ms_remote_scan = 30000
interval_ms_local_scan = 10000
interval_ms_downloading_scan = 1000
extract_path = /tmp
use_local_path_as_extract_path = True
[Web]
port = 8800
[AutoQueue]
enabled = False
patterns_only = False
auto_extract = True
`

The new image no longer runs the app as the root user. Try changing the /root/.ssh path to /home/user/.ssh

Do I literally use the path "/home/user/.ssh", or do I change it to my username "bart"?
I'm getting this now:
` docker run -it --name=seedsync5 --hostname $HOSTNAME -p 8800:8800 --user 1000:1000 -v /media/usb2/sabnzb/downloads/complete:/downloads -v /home/bart/docker_configs/seedsync:/config -v /home/bart/.ssh:/home/bart/.ssh ipsingh06/seedsync
Traceback (most recent call last):
File "/app/python/seedsync.py", line 376, in
seedsync = Seedsync()
File "/app/python/seedsync.py", line 59, in init
config.to_file(self.config_path)
File "/app/python/common/persist.py", line 53, in to_file
with open(file_path, "w") as f:
PermissionError: [Errno 13] Permission denied: '/config/settings.cfg'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/app/python/seedsync.py", line 384, in
Seedsync.logger.exception("Caught exception")
AttributeError: 'NoneType' object has no attribute 'exception'
`
I think that the /config/settings.cfg file may still have root permissions inside the container?

Try this command
docker run -it --name=seedsync5 --hostname $HOSTNAME -p 8800:8800 -v /media/usb2/sabnzb/downloads/complete:/downloads -v /home/bart/docker_configs/seedsync:/config -v /home/bart/.ssh:/home/user/.ssh ipsingh06/seedsync

OK, the path for the config folder was wrong - it should be "/home/bart/docker/seedsync:/config", so that got us past the permission error on the settings.cfg file.
However, it's still crashing with this error:
2019-02-01 21:41:18,698 - INFO - RemoteScanner.RemoteScanner (RemoteScanner/RemoteScanner) - Installing local:/app/scanfs to remote:/home/bart/scanfs Process RemoteScanner: Traceback (most recent call last):   File "/app/python/controller/scan/remote_scanner.py", line 115, in _install_scanfs     remote_path=self.__remote_path_to_scan_script)   File "/app/python/ssh/sshcp.py", line 169, in copy     args=" ".join(args)   File "/app/python/ssh/sshcp.py", line 97, in __run_command     raise SshcpError("Bad hostname: {}".format(self.__host)) ssh.sshcp.SshcpError: Bad hostname: <IP_ADDRESS>

Sorry, accidentally closed.

OK, making progress. The username that the container runs under is seedsync, so this needs to be this:
-v /home/bart/.ssh:/home/seedsync/.ssh
Then it works. Figured it out by doing a console session inside the container, and looked at who owned the /config folder. Had to remove the the CMD in Portainer first, so it wouldn't try to execute on start.
The issue that makes it so hard to diagnose these issues is the error message coming from the container - it's useless. "Bad hostname" can mean a lot of things. In this case, it meant "I can't find the ssh key so I can't connect". I realize that I created this problem by mapping my .ssh folder inside the container, instead of creating new keys.
Any way to make the error messages a little more useful?
And here is my full docker command for those that want to map the ssh folder correctly, and maybe stumble on this through a Google search in a few years:
docker run -d --name=seedsync \
  --hostname $HOSTNAME \
  -p 8800:8800 \
  --user 1000:1000 \
  -v <downloads folder on host>:/downloads \
  -v <config folder on host>:/config \
  -v <.ssh folder on host>:/home/seedsync/.ssh \
  ipsingh06/seedsync

The user ID is the same as the user on my host, which I  got by running the id command:
id
uid=1000(bart) gid=1000(bart)

By using the same id, the files should have correct permissions.

Glad to hear it's working now. I'll look at improving the error message in the future.

Now I'm actually trying to download a file and I'm getting permission denied errors :(
 2019/02/01 17:00:04 - INFO - RemoteScanner.RemoteScanner - Skipping remote scanfs installation: already installed

2019/02/01 17:00:33 - INFO - seedsync.Controller - Received command Action.QUEUE for file xxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv

2019/02/01 17:00:34 - WARNING - ActiveScanner.ActiveScanner - Path does not exist: /downloads/xxxxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv

2019/02/01 17:00:35 - WARNING - ActiveScanner.ActiveScanner - Path does not exist: /downloads/xxxxxxxxxxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv

2019/02/01 17:00:36 - WARNING - ActiveScanner.ActiveScanner - Path does not exist: /downloads/xxxxxxxxxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv

2019/02/01 17:00:36 - ERROR - seedsync.Controller.Lftp.LftpJobStatusParser - LftpJobStateParser error: Unrecognized line 'pget: /home/bart/rtdata/xxxxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv: /downloads/xxxxxxxxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv: Permission denied'

2019/02/01 17:00:36 - ERROR - seedsync.Controller.Lftp.LftpJobStatusParser - Status: pget: /home/bart/rtdata/xxxxxxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv: /downloads/xxxxxxxxxxxxxxxx.s02e06.1080p.web.x264-tbs.mkv: Permission denied

I'm running with the --user switch. And I also tried adding these two switches:
-e PGID=1000
-e PUID=1000

Same permission error.

Changing the ownership of the /downloads folder from root to seedsync worked. The file is downloading now. Maybe modify the image and the owner of the /downloads folder to whatever user is specified when launching the container, if --user is used? Or just hard-code it to be owned by the seedsync user since that user is automatically created in the image?
This is the command I used as root in the container console, for other people's reference:
chown -Rv seedsync:seedsync /downloads

-R makes it recursive inside the folder
-v makes it verbose and tell you what it did
user:group defines the new owner
/folder-name is the folder you want to change the rights of

Odd, the /downloads folder is already owned by seedsync when the image is compiled. If it wasn't for you then something must have gone wrong while running the image.
Also if you mount a host directory to /downloads, the permissions will carry over from the host directory.
I hope everything is working as is now.


Release for macOS

Hello,
Is there any plan to release a .palette file for Telegram macOS?
Thank you

Hello,
I don't use (or even have) mac, so there's no easy way to test for me. I might look at it someday, but there are no plans at the moments.
If anyone would like to port the current tdesktop theme, contributions are welcome!

Thank you for getting back to me. I do not use Telegram enough to justify the effort required to port the theme (although it shouldn't be hard). I will keep an eye on the repo in case some more involved user would like to contribute.


Bench Bot Test PR

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet amm

/bench runtime pallet exchange

/bench runtime pallet exchange

This PR was only for bench-bot test.

This PR was only for bench-bot test.


Update Readme

Wow! There's so much content here! Amazing work! I'm reading through now and cleaning up some of the verbiage (I'll push with changes) but it doesn't looke like images/compressio.png is included in source control right now. Commenting here so I don't forget about it.


Poor performance after short use time

Hello
I've mentioned this on telegram, but to make official:
I have been experiencing massive slowdowns to the point the app becomes unusable after under an hour's usage. This problem started with version 1.8.6 and I suspect it was around commit #994b3ed
I rebuilt the app version 1.8.5 which does not have the issue. I also built from main and the issue is still present there.
Sorry I don't have time to properly investigate this but I hope this helps.

Olá
Já mencionei isso no telegrama, mas para oficializar:
Tenho experimentado lentidão massiva a ponto de o aplicativo ficar inutilizável após menos de uma hora de uso. Este problema começou com a versão 1.8.6 e suspeito que tenha ocorrido por volta do commit #994b3ed
Reconstruí a versão 1.8.5 do aplicativo que não apresenta o problema. Eu também construí a partir do main e o problema ainda está presente lá.
Desculpe, não tenho tempo para investigar isso adequadamente, mas espero que isso ajude.

Hello, check if it is not a plugin causing malfunction or your device

its a samsung a34.
No plugins and fresh install.
I'm not convinced it's what I thought it was. I'm using sftp connections and I think there may be some problem there, but I'm not sure.

Thanks for reporting, will check it and fix it.


GNOME: clean up old packages

Package removals LGTM, not sure if we could mention the attr path in the throw message though :thinking: (I am addressing this here, it looks not that easy to understand if we don't --show-trace)


Update Composer dependencies (2022-11-05-00-07)

Visual regression test passed!
View the visual regression test report


Unfortunately, it still doesn't work.

Unfortunately, it still doesn't work.
I set the passive perception DC with my hands in the token settings (value in the "Passive Perception DC" field in the settings has no effect).
After the stealth check occurs, the value is correct.
But after the token is revealed from the invisibility state, it is reset to -1 again.
Maybe I'm doing something wrong?
Screens below:

Settings

Token settings before stealth roll

Token settings after stealth roll

Token settings after reveal by skillcheck perception off another token (hero)

And this is a video capture on the screen with this issue
https://drive.google.com/file/d/15rNta93VDAoN5J3OwIWjRKXhMrScuD6G/view?usp=sharing
Originally posted by @VirusNik21 in https://github.com/Saibot393/perceptive/issues/95#issuecomment-2159788687

Author will change this in the next update.


Fix typo of causal spelling

There was a typo in spelling for causal, fixed that.

Can we get this merged?


Generate deck apkg

Use https://www.npmjs.com/package/@seangenabe/apkg

The main goal of this tool is "sync"


When the number of tables is too large, querying group information is easy to get stuck

Enhancement
there are 1W tables in a group

Thanks for you issue.


igluctl: add --version flag

We lost it after CLI refactoring.

Migrated to https://github.com/snowplow-incubator/igluctl/issues/12


Enchantment: install and setup three.js directly from npm

As I’m already writing the denoify.ts script, I’m thinking of automatizing the whole process. With a combination of tasks and the script…
So I can avoid having to bundle three.js itself with the project. Instead, I could just include the relevant package.json file🤔
Maybe even making the script work for other npm packages I may want to use 😁

I'll close this for now, as it is not a priority. A set version of three.js will do for now. And I'll do the same for any other library.
In the future, I want to implement a "bring your own library" approach.


Missing opengl dependency

just installed shady in a clean environment, got this error with nimble build
shady-0.1.3/shady/demo.nim(3, 8) Error: cannot open file: opengl

please mind that I've not been running nimble build in shady install directory, but in a new clean project with just src/my_test_shady_proj.nim

Yes, shady is just nim code to GLSL macro, in theory it needs nothing but Nim and vmath.
But the examples would be pretty boring if they just printed text output.


Job offer should have an expired status

At some point in time, it would be necessary an "expired" status. Once the job offer gets expired it should not be listed anymore.

What do you think to create a "expiration data" field ?
Because if you need to know when this offer was expired you've had this information.

@eerison expired at is good to go.
null - default; job offer is still valid.
<= today - job offer is still valid.
> today - job offer is expired.


Chisel Auto Chisel nearby causes Quickstack to not function

I hope you don't mind the lazy issue relay 😄
https://github.com/EnigmaticaModpacks/Enigmatica6/issues/4929

Found the issue, the problem is a bug in Chisel that causes sideless itemhandler accessing to error out. Relevant lines
I've added a workaround on my own end, but other mods that attempt sideless interaction will also break until this is fixed.


tableauserverclient 0.30 :snowflake:

tableauserverclient 0.30 :snowflake:
Destination channel: Snowflake
Links

PKG-3725
dev_url: https://github.com/tableau/server-client-python/tree/v0.30
conda-forge: https://github.com/conda-forge/tableauserverclient-feedstock/blob/main/recipe/meta.yaml
pypi: https://pypi.org/project/tableauserverclient
Relevant dependency PRs:

AnacondaRecipes/urllib3-feedstock#18
AnacondaRecipes/requests-feedstock#12
tests AnacondaRecipes/requests-mock-feedstock#2

Explanation of changes:

update version, fix recipe

It is required requests>=2.31.0. The latest requests==2.31.0, which is also the latest version, in our index requires urllib3>=1.21.1,<2' (instead of the urllib3 >=1.21.1,<3` specified in the upstream, because we made a conservative choice here @skupr-anaconda ).
At the same time, in this recipe, the exact urllib3==2.0.7 is needed, which clashes with the previous requirement.
@JeanChristopheMorinPerso what is your opinion? Shall we relax our requests==2.31.0 from urllib3>=1.21.1,<2' to urllib3 >=1.21.1,<3`?

@lorepirri requests should be fixed. https://inspector.pypi.io/project/requests/2.31.0/packages/70/8e/0e2d847013cb52cd35b38c009bb167a1a26b2ce6cd6965bf26b47bc0bf44/requests-2.31.0-py3-none-any.whl/requests-2.31.0.dist-info/METADATA#line.35.

@lorepirri requests should be fixed. https://inspector.pypi.io/project/requests/2.31.0/packages/70/8e/0e2d847013cb52cd35b38c009bb167a1a26b2ce6cd6965bf26b47bc0bf44/requests-2.31.0-py3-none-any.whl/requests-2.31.0.dist-info/METADATA#line.35.

Thanks! In progress at AnacondaRecipes/requests-feedstock#12

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_ceg4q_jrs0/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_28x6q90jqd/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_2bblh2zycy/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

@skupr-anaconda versioneer will not be in the final artifact. Versioneer generates a _version.py file, see https://inspector.pypi.io/project/tableauserverclient/0.30/packages/0b/a6/23eed1b99ddd7d1cddde0f77fbf01aed8f0892a1e8f3dd47807358cb7965/tableauserverclient-0.30-py3-none-any.whl/tableauserverclient/_version.py.

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_e2aczvkoh4/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_edcim67_mn/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_30ahpiu5k_/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_e6z3_c4lzk/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_70ffh5unyb/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found


rust-analyzer can't work in files with name including '_'

can't work with name "problem_131.rs"

fine with name "problem131.rs"
rust-analyzer version v0.2.513

Did you add a mod problem_131; declaration in the parent module?

Did you add a mod problem_131; declaration in the parent module?

yes, but the "problem_131.rs" works now, don't know why
:)
I created a new file with the same problem

And it never happens for modules without _? It's a really strange issue.

Sorry, I made a mistake. It's not about the name, but the path.
Those files are created by an extension. For some reason, the extension will open the file once created, but the path shown in vscode is not workspace_name>relative>path>file_name but workspace_name>definite>path>file_name

If the file first opened with a definite path, like the extension did, the analyzer won't work, later opening it manfully with a relative path either. Changing its name will work and the path is relative after renaming, but changing it back will make the analyzer fail again, maybe the old name path is cached.
I don't know how to open a file which is in the workspace with a definite path manually, so I can't verify that. Same file synchronized to another PC works. Using the extension in another workspace to create a file in problems folder then open the file in this workspace manually also works. Both ways open the file with a relative path at the first time in this workspace.


[MIG-249] DVM controller must label PVCs appropriately for rollback

Is your feature request related to a problem? Please describe.
I don't think rollback will be able to remove PVCs created by DVM controller due to lack of labels on the PVCs.
See https://github.com/konveyor/openshift-velero-plugin/pull/58 for info on labels needed

MigPlan correlation label
MigMigration correlation label

Describe the solution you'd like
Appropriate labels should be added to the PVCs created by DVM controller. Once this is in place, Rollback should be able to delete PVCs from the destination cluster using the labels above.

With the fix for this, please also update the labels set on DVM resources to use a common func to set the labels to ensure we aren't missing labels on any individual component (see https://github.com/konveyor/mig-controller/blob/master/pkg/controller/directvolumemigration/rsync.go#L233).


feat(agent): add close method

This PR adds a close method to the Agent class that aborts all pending requests and resolves #312. To do so, a small bug is fixed in the abort method where the _msgInFlight hasn't been decremented when a request is cancelled.
An additional test is added to assert that the new method works as expected.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 87.463%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
1134

Relevant Lines:
1247

💛  - Coveralls

Ping @relu91 :)

I'm not coap expert but looking at the logic seems good. This should allow freeing all resources without waiting for responses :)

Thank you for the feedback! :) I added a last couple of changes to the method so that you can both provide an optional callback and listen for close events. This way the API is more similar to the one the CoAP server provides. I think if @Apollon77 could also have a look on it then the PR should be able to be merged :)

BTW: LGTM

Add that close before 1.0 or after? ;-)

Hmm, good question ;) I think we could already integrate it in 1.0 if this would be fine for you, but we could also put it into a separate minor (or patch?) release if you prefer.

:-)


Schema Compare fails for Azure database using AAD auth

Insiders build
Version: 1.6.0
Date: 2017-12-15T12:00:00.000Z
VS Code 1.31.1
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17763
Steps to Reproduce:

Connect to an Azure Server using AAD
Start schema compare for a database inside this server same connection
Fill in the details and click compare

Expected:
Should show diff
Actual:
SQL tools service throws null ref exceptions
Notes:
Two things to fix here,

Should check for null schema models
Connection info created through AAD auth should work in schema compare APIs

Moving to planning as Kim is working on it


Update request: openai-whisper-cpp 1.7.0 → 1.7.1

Package name: openai-whisper-cpp
Latest released version: 1.7.1

Current version on the unstable channel: 1.7.0
Current version on the stable/release channel: 1.5.4

[ ] Checked the nixpkgs pull requests

Notify maintainers

@r-ryantm
@Artturin

Note for maintainers: Please tag this issue in your PR.
Hello folks, great work here,
I would also like to add the server command line. There are a few new commands available, and it's not included in the package.
https://github.com/NixOS/nixpkgs/blob/7881fbfd2e3ed1dfa315fca889b2cfd94be39337/pkgs/tools/audio/openai-whisper-cpp/default.nix#L87-L89

Many thanks for considering my request.

Add a :+1: reaction to issues you find important.

https://github.com/NixOS/nixpkgs/pull/349647

I'm not the maintainer for this package https://search.nixos.org/packages?channel=unstable&show=openai-whisper-cpp&from=0&size=50&sort=relevance&type=packages&query=openai-whisper-cpp
@dit7ya @hughobrien You should look in to using cmake for the package, it installs the libs too.


Updated migration docs with Rasa Core 0.13.0

To fix #1677

also i believe this should be based off of 0.13.x

also i believe this should be based off of 0.13.x

Is there any downside to merging this, then starting a new minor from master?  If I PR to 0.13.x, I'm going to be merging twice.

we don't release minors from master, as it may contain breaking changes. You just merge the 0.13.x branch into master afterwards (and pray for no merge conflicts, otherwise create a new branch off of 0.13.x, merge master into it and then merge it into master)

so it's approved now, but please merge it into 0.13.x rather than into master.

(Merged for version rasa_core==0.13.2)


Delay in displaying user name in LHN when replying on a thread message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a User2 chat
Send a message to User2
Click on "Reply in thread" option
Open User2's account and open the chat with User1
Click on the thread message sent by User1
Check the LHN for the thread message with User1's account

Expected Result:
The user name should be instantly displayed in the LHN along with the sent message
Actual Result:
There is a delay in displaying the user name when User2 replies to the thread message in LHN
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.29-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/4d15d955-6ba8-4825-b61c-fe65870b4409
https://github.com/Expensify/App/assets/43996225/18cd952e-faa0-4632-b420-1a994f6e1382
Expensify/Expensify Issue URL:
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686309168445669
View all open jobs on GitHub

I'm not sure what the problem is here, I am not seeing a delay. Please reopen if there is a clearer way to reproduce this

@muttmuure You can see in this video when userB reply on thread userA's account LHN (UserB : hello from user B) "UserB :" Showing after refreshing page
https://github.com/Expensify/App/assets/56740856/52bf9865-681d-4979-9a6d-d92aed91944f

@ayazhussain79 I'm not sure what you're showing me here

Could you write out a series of steps if there is another way to reproduce the bug

Please try these steps:

Open a UserB chat from UserA account
Send a message to UserB
Click on "Reply in thread" option and send a message
Open UserB's account and open UserA chat
Click "Reply in thread" on message sent by UserA
Send a message on thread from UserB
Now you have to check UserA LHN

if userB sent a message in thread like"Hello"  you will see from userA account in LHN last message "Hello"
when you refresh the page last message will be "UserB:Hello"

@muttmuure have you tried these steps: https://github.com/Expensify/App/issues/21056#issuecomment-1607402531

@ayazhussain79 I'll give it another try when I'm back online tomorrow. Thanks for the extra steps

Github is having some real problems uploading the gif/video I have of this.
It looks like this is fixed, I'm not able to reproduce it and there is no delay.

If you are able to find a PR that fixed this issue reported somewhere else, and you were the first to report it, please add it here and I'll see if you're eligible for a bug bounty.


Subscriber Trigger Multiple time on OnEvent Decorator

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
The Following Listener Triggered 2 Time on a single Emit
  @OnEvent('order.failed')
  handleOrderFailed(payload: any) {
    console.log(payload);
  }

Where as the below works currectly,
@Injectable()
export class GlobalSubscriber {
  constructor(private eventEmitter: EventEmitter2) {
    this.eventEmitter.on('**', (payload) => {
      console.log(payload);
    });
  }

}

Seems BUG, please Verify
i chnaged OS, also run same in server ubuntu, but still it trigger 2 time, i getting console.log 2 time in console.
Minimum reproduction code
https://github.com/nestjs/nest/tree/master/sample/30-event-emitter
Steps to reproduce
npm i
Expected behavior
The Event Subscriber should trigger only 1 time, and not Multiple time
Package version
1.1.1
NestJS version
8.4.4
Node.js version
14+
In which operating systems have you tested?

[ ] macOS
[X] Windows
[ ] Linux

Other
No response

Please, use our Discord channel (support) for such questions. We are using GitHub to track bugs, feature requests, and potential improvements.


Fix a few typos in documentation

Noticed two typos as I was going through the "first app" tutorial. Sorry for the noise today!

Thanks again!


Salesperson API

Summary
A new API page for Salespeople/Purchasers
I created the tests in a similar manner that I found suitable for Locations, but I did not manage to run the test. Thus, looking forward to the feedback if tests do pass in Microsoft Pipelines or how to test those calls in a container. Currently I do not have a clue how to do it.
Work Item(s)
Fixes #26806
Fixes AB#541735

For tests, you need to use Isolation Disabled test runner codeunit

This one is merged internally. Not sure why the tests are failing here. I'm inclined just to close this PR. The changes will get pushed to this repo when we push out the next code update.

Tests probably failing due to setup issue. We will have to investigate.

@aholstrup1 will force merge and update the repo to get this in.


Update README.md plugin version to latest release.

Thanks for the PR! This is now merged into master :)


docs(examples): cover different levels of immersion

Currently just a draft, elephant shrew is a stand-in picture for now
Signed-off-by: Youngone Lee<EMAIL_ADDRESS>

Currently just a draft, elephant shrew is a stand-in picture for now
Signed-off-by: Youngone Lee<EMAIL_ADDRESS>
@Leeyoungone Very cute elephant shrew!


bug: data lost when imm merge is skiped due to insufficient memory

Describe the bug
imms are drained here but they won't be restored when imm merged is skipped due to insufficient memory.
To Reproduce
No response
Expected behavior
No response
Additional context
No response

#9358


CSS CkEditor Content

Nice Bundle ;)
is there any way to configure the content css of the ckeditor?
I mean use an external css file?

Hello,
maybe this useful information should be in the doc?
If I understood well what I read here, we can access all ckeditor conf with the config array? Ii would be great to tell this in the doc, maybe in a tooltip or other artifact for important information? (I may have missed this information in the doc)

@bruno-ds Yes if you use any CKEditor options documented [here)(http://docs.ckeditor.com/#!/api/CKEDITOR.config).
This is documented in the bundle on the config section: http://symfony.com/doc/current/bundles/IvoryCKEditorBundle/usage/config.html (very first tip block).
Not sure how I can make it more visible in the doc... If you think you can improve it a recommendation/PR is very welcome :)

@egeloen : I'm sorry, it was effectively in the doc.
Maybe you could add a little longer explanation in order to make it more visible, like you could say :

You can use any native CKEditor options. The bundle uses a variable node in its config, so, you can put basically anything in this node (but no validation are done on it, the bundle trust you).

but it is at the edge, your work is perfect, sorry.

Don't be sorry, it's just a question :) I will make the block more precise tonight. Can you open an issue in order to not forget it?...


Few Fixes and Features

Added the option to set enforce_detection variable via POST method. Fixed issue when returning an error state, the system was messed up not knowing if the result was a tuple or dictionary.

currently, deepface api is serving via gunicorn. you break this because you copied app.py functions to api.py directly. app should be run from api.py.
deepface_app = app.create_app()

Thanks for the answer, I'll check what I have done wrong.
This version base is from around 2 months ago, It happens that I had just
created the pull request now.
Hope success,
Em qui., 9 de mar. de 2023 12:35, Sefik Ilkin Serengil <
@.***> escreveu:

Closed #694 https://github.com/serengil/deepface/pull/694.
—
Reply to this email directly, view it on GitHub
https://github.com/serengil/deepface/pull/694#event-8709513853, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEB4VLMP5GLXSAHEYLQRQSLW3H2FVANCNFSM6AAAAAAVVIS7IU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Portfolio: Mobile first

[ ] Add HTML Header and Headline
[ ] Add CSS Styling
[ ] Fix Linter Errors

Dear Reviewer,
This logo given in figma is a very old logo of https://medium.com/ I cannot find the old logo anywhere so I had to import the new Medium logo here; Hope you will accept it.


Cloud Code suggestion/question

Hello.
I would like to know if there is any possibility of making a Cloud Function to run each day at 1:00 PM.
I've searched through the documentations but i couldn't find anything about it.
I know it's not an issue about parse-server, but if we can't make a function to run each day at 1:00PM maybe it could be a good idea to make it happen.
Thanks a lot for your time!

Simply set up a cron-job to curl/wget that function and you're good to go.

If you want to do this in code you can also try https://www.npmjs.com/package/node-schedule

We don't provide (yet) a job API / scheduler as it's very platform specific. As @steven-supersolid or @bohemima recumbent, there are multiple available options here to run regular jobs.


Rebuild and cleanup

master -> main
0.0.1-beta1 -> 0.2.4
abstract urls by using secrets
allow ACTIONS_ALLOW_UNSECURE_COMMANDS to avoid build failures

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

Is it possible to merge this PR soon?

Hi there, thank for submitting this PR – apologies we never merged it. Wanted to let you know we're going to archive this repository in favor of a native integration. See https://github.com/hashicorp/action-waypoint/issues/45#issuecomment-840213375 for more details. Thanks again.


Gradient overflow when running the example of Adaptive Input Representations

❓ Questions and Help
What is your question?
Got inf loss and gradient overflow when running the code example of adaptive input representation with --fp16. I am trying to reproduce the results of Baevski and Auli, 2018, and the code example provided by fairseq is pretty fine with fp32. However, the model doesn't work well when I use fp16 to reduce the training time, following Baevski and Auli, 2018 . Are there any tips for preventing the model from inf loss?
Code
Almost the same as the code in this link except for the fp16 argument:
CUDA_VISIBLE_DEVICES=0,1,2,3,4,5,6,7 python train.py \
    --task language_modeling \
    $DEST/data-bin/wikitext-103 \
    --save-dir $DEST/results/debug \
    --arch transformer_lm_wiki103 \
    --max-update 286000 --lr 1.0 --t-mult 2 --lr-period-updates 270000 --lr-scheduler cosine --lr-shrink 0.75 \
    --warmup-updates 16000 --warmup-init-lr 1e-07 --stop-min-lr 1e-09 --optimizer nag --min-lr 0.0001 --clip-norm 0.1 \
    --criterion adaptive_loss --max-tokens 3072 --update-freq 3 --tokens-per-sample 3072 --seed 1 \
    --sample-break-mode none --skip-invalid-size-inputs-valid-test --ddp-backend=legacy_ddp \
    --fp16

Results:
...
2022-03-21 14:38:24 | INFO | fairseq.tasks.text_to_speech | Please install tensorboardX: pip install tensorboardX
2022-03-21 14:38:36 | INFO | fairseq.trainer | NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 64.0
epoch 001:   0%|                                                                                | 1/1401 [00:14<5:43:19, 14.71s/it]
2022-03-21 14:38:42 | INFO | fairseq.trainer | NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 32.0
epoch 001:   0%|▏                                                                                | 2/1401 [00:21<3:48:24,  9.80s/it]
2022-03-21 14:38:49 | INFO | fairseq.trainer | NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 16.0
epoch 001:   4%|███▎                                                                                | 53/1401 [01:06<14:50,  1.51it/s]
2022-03-21 14:39:29 | INFO | fairseq.trainer | NOTE: gradient overflow detected, ignoring gradient, setting loss scale to: 8.0
epoch 001:  13%|▏| 185/1401 [02:34<13:29,  1.50it/s, loss=inf, ppl=inf, wps=108973, ups=1.48, wpb=73714.1, bsz=24, num_updates=100, lr=0.00

What's your environment?

fairseq Version (e.g., 1.0 or main): main (1.0.0a0+b554f5e)
PyTorch Version (e.g., 1.0):  1.8.0+cu101
OS (e.g., Linux): Linux
How you installed fairseq (pip, source): source
Build command you used (if compiling from source): pip install --editable ./
Python version: 3.7
CUDA/cuDNN version: cu101
GPU models and configuration:
Any other relevant information:

Hi @ghrua,
Have you found the solution? I am facing the same problem...

Hi @jamfly
Yes, actually there are two solutions I think:

Detect the overflow operation step-by-step and address it.
Pretrain the ADP model for 3 epochs using FP32, and then reload the parameters when training with FP16. Please reset the optimizer when you load the FP32 model under the FP16 setting.

The second one is somehow tricky... But it works for me.

Hi @ghrua,
Thank you for your suggestions, I will at least try the second one.
I have questions regarding the training:

Have you tried using fp16 from scratch, will it turn lnf loss to the normal scale?
I noticed that you set update-freq to 3, but based on the description from the paper, they are using tokens-per-sample 4096 with 8 GPUs. I know they said that they changed to 3072 because of a better performance. But, is update-freq always set to 3?

Thank you in advance.

Have you tried using fp16 from scratch, will it turn lnf loss to the normal scale?

Yes, I have tried FP16 from scratch with many hyper-parameters, e.g., different values of warmup updates and clip norm, but they didn't work for me.

I noticed that you set update-freq to 3, but based on the description from the paper, they are using tokens-per-sample 4096 with 8 GPUs. I know they said that they changed to 3072 because of a better performance. But, is update-freq always set to 3?

In the section of batching, the author said "This gives an effective batch size of 65K tokens for WIKITEXT-103.", where 65,000 / 8 / 3072 is around 2.6. I think that's why they set update-freq to 3.

Hi @ghrua, I got it, they are using 4096 (tokens)* 8 (GPUs) * 2 (update-freq). Anyways, thank you for your kindly help and suggestions, I really appreciate it. Thank you.

can you replicate the results in paper? I ran the same recipe as yours, got a test ppl of 29.14, but the results in paper should be 18.7.

Hi @Psycoy, sorry for the late reply. Mine was 19.7, and despite being close to 18.7, it still has a gap. Can you reproduce their results?

Hi @Psycoy, sorry for the late reply. Mine was 19.7, and despite being close to 18.7, it still has a gap. Can you reproduce their results?

Yes, I can, as far as setting the right update frequency according to the gpus and batch size.

Hi @Psycoy, sorry for the late reply. Mine was 19.7, and despite being close to 18.7, it still has a gap. Can you reproduce their results?

Yes, I can, as far as setting the right update frequency according to the gpus and batch size.

Did you use which script to evaluate your model?


chore: gitignore improvement

Hi guys,
I've updated the .gitignore to include a more comprehensive set of patterns to match files and directories at any level in this repo. This will help prevent unwanted files from sneaking into commits and ensure that the repo stays clean and organized.
Summary of change
Here's a quick summary of the changes:

Added patterns for commonly ignored Rust files and directories
Included IDE/editor specific patterns for better compatibility
Made sure the patterns work for complex project structures with multiple subdirectories

Acceptance Checklist
Feature (non-breaking change which adds functionality)

[x] Have you seen it working in a development environment?

Hi @AwesomeIbex,
Feel free to let me know if you have any questions .

Awesome sir!
Please can you update your commit message to follow https://www.conventionalcommits.org/en/v1.0.0/ - I'll be happy to merge then.

Awesome sir!
Please can you update your commit message to follow https://www.conventionalcommits.org/en/v1.0.0/ - I'll be happy to merge then.

Hi @AwesomeIbex,
Thanks for the feedback, I appreciate it.
The commit message has now been updated as per your request.

@AwesomeIbex,
Further edit made to the commit message to pass the automated checks.

The header is now within the 100-character limit.
The body's lines are not longer than 100 characters.
The footer has a leading blank line.

@0xf333  haha! You're almost there, there is one line that is 132 chars, this normally gets people with commitlint. I'd edit them myself but I don't wanna steal your commit.

Hi @0xf333!
If you see the checks, there's one saying "Lint Commit Messages / commitlint (pull_request) Failing after 11s".
If you click on Details, it takes you to the output of the tool, where you can see this:

I've highlighted the line where it tells you what's failing. The header of the commit is 132 chars long.
You have correctly updated the commit message and it's much better now, however, the previous commit with the wrongly formated message is still in the PR (see here, there are two commits).
You can either squash them and keep the good message (since both are basically the same feature), or interactively rebase the commits so you are able to reword the message. Check this page to learn how to do it.

@0xf333, looks good to me! Good job!

@0xf333, looks good to me! Good job!

Thanks


Enable crash kernel for target OS

Calculate crash kernel memory parameters at post-installation time.
Generate a kdump initrd file before kdump service starts.
mkdumprd and dracut kdump module don't work well with current file system layout.
Customize the fstab file in the result initrd by ourself.

Required OS change: https://github.com/harvester/os2/pull/17
Related issue: https://github.com/harvester/harvester/issues/1335
** How to test **
Build a new ISO with the change and boot into the OS

[ ] cat /proc/cmdline: crashkernel=244M,high crashkernel=72M,low parameters should be included (values varies from machine to machine)
[ ] echo c > /proc/sysrq-trigger to simulate a crash. After rebooting, a crash dump should be store in/var/crash directory.

@johnliu55tw @weihanglo
I create a grub entry to enable the crash kernel.  Users can select the (debug) one to enable it.


Add Hyperopt to User & Developer Guide

For #677
Add Hyperopt to User Guide with documentation that explains on how to use it and to Developers guide, that explains the design and the Strategy and executor abstract classes and explains how to implement a new strategy or a new executor.

I added some comments on the user guide, cleaned up the developer guide part that needed a bit more work, and also added some todos in the developer guide

Got it. Will do the required changes. Thanks

Oh sorry, the comments are not in the my, but if you scroll up here you'll find a bunch.
Regarding the examples, usage examples are always welcome, you can show how the function is called by providing a plausible input and showing what would be a likely output (if the function returns something).

@w4nderlust : Hi, Should I update the root key measure to metric in the functionalities? There are instances where the metric keyword is used as the input parameter for the classes.

I'm doing a pass on the docs, but I noticed one thing that could be an error: in the parallel executor, when gpus is None we should check if gpus are available (there's a TF function for that) and use that instead to determine the available gpus first, if there are none, then our logic continues correctly. So we should add this additional logic before line 501.

Also noticed that we were not updating the strategy at end of the batch! I updated the code f all executors to do it, bu please doublecheck  if it is fine for you too.

@kaushikb11 Also noticed that we were not updating the strategy at end of the batch! I updated the code f all executors to do it, bu please doublecheck if it is fine for you too.

@w4nderlust I went through the code and ran it, the functionality makes sense. It's great how the code gets refined after writing tests and docs. But I didn't understand the idea of using class variables where we already have default values for the parameters.

I'm using this pattern also in the TF2 porting. The point is that if someone wants to contribute a new executor or strategy, they don't have to do and put the defaults parameters in another different function, but that function that populates the defaults can extract the parameters from the newly added class. It's much cleaner, and if, for instance, I want to change the default parameter for num workers, I have to change it only in the ParallelExecutor class and not also in the function that populates the defaults.

Another thing: I noticed only now that psutils is an external library that needs to be added to the requirements. There's no way to obtain the same result without using it, just with functions from the standard library?

Another thing: I noticed only now that psutils is an external library that needs to be added to the requirements. There's no way to obtain the same result without using it, just with functions from the standard library?

I found something interesting while searching for it. In scikit-learn,n_jobs input param for the Parameter Search Classes has the following behavior:

n_jobs is an integer, specifying the maximum number of concurrently running workers. If 1 is given, no joblib parallelism is used at all, which is useful for debugging. If set to -1, all CPUs are used. For n_jobs below -1, (n_cpus + 1 + n_jobs) are used. For example with n_jobs=-2, all CPUs but one are used.

It uses the joblib library to run the tasks parallelly. Link on how they figure out the number of CPUs the current process can use. The only downside of using os.sched_getaffinity(0) is that it appears to be UNIX only. But they have added Exception Handling and get aggregated_cpu_count by taking the minimum of the implemented methods.

Does this account for the logical cpus vs physical cpus?

Does this account for the logical cpus vs physical cpus?

They are considering logical cores as well. I am running an experiment on multiprocessing for data-intensive tasks. I am training ten instances of models using 4 cores (mp.cpu_count) and 2 cores (psutil.cpu_count(logical=False))

It kinda makes sense right? The logical cores spend more time swapping stuff in and out so they are less utilized, while the physical cores don't have to do it so are more fully utilized. Because of this anyway, I have the feeling we can safely remove the check for physical vs logical, and consider only logical. My original fear is that if we consider logical, the performance would have been degraded, but your test shows that, although it doesn't increase  much, at least it does not degrade. So I would say let's remove psutils to get the number of cores and let's use something in the standard library, even if it returns the number of logical cores instead of physical (and let's make sure it works on all platforms). Does that sound good?

Sounds good to me!


Tweetest Prep for payara-mp-tck tests

👾 Description of the issue
We run external third-party application tests at this project.  They are contained within the external directory.  We would like to improve how we run these tests.  The first step to improving how we run them is to add some documentation and to pull out the individual test properties that we use so that they are stored in the same directory as the test.
This issue is to add a README.md and a test.properties file to the /external/payara-mp-tck test directory.  As part of this prep work, since we already generate the Dockerfile, please also remove the static https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/payara-mp-tck/dockerfile/Dockerfile.
The contents of the README file can be simply some instructions for how to run the test locally.  Use with the instructions from https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/README.md#running-external-tests-locally and modify Steps 7 and 9 to reference payara-mp-tck specifically.
The contents of the test.properties file will be a key=value format, copying the information from https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/common_functions.sh#L316-L328 to populate it.
Adding these files will not change the current way we run these tests.  A separate change will be made to reference the test.properties file, at which point the contents must match those in the common_functions.sh script.
Please take the time to try and run the test locally (using the instructions you are adding in the README file).  Some of the external tests are failing.  When you run it locally, you may find that to be the case.  The reviewer can help to advise you if there are any further steps you can take to try and debug the pre-existing failure, if it exists.
📋 Step by Step
To solve this issue and contribute a fix you should check the following step-by-step list. A more detailed documentation of the workflow can be found here.  Note: that you do not need to add yourself to a Contributors.md file as outlined in the detailed instructions.

[ ] Claim this issue: Comment below.
[ ]  Fork the repository in github by simply clicking the 'fork' button.
[ ]  Check out the forked repository
[ ]  Create a feature branch for the issue. We do not have any naming definition for branches.
[ ]  Commit your changes.
[ ]  Start a Pull Request.
[ ]  Done 👍 Ask in comments for a review :)
[ ]  If the reviewer find some missing pieces or a problem they will start a discussion with you and describe the next steps how the problem can be solved.
[ ]  You did it 🎉 We will merge the fix in the master branch.
[ ]  Thanks, thanks, thanks for being part of this project as an open source contributor ❤

🤔❓ Questions
If you have any questions just ask us directly in this issue by adding a comment. You can join our community chat at Slack. Next to this you can find a general manual about open source contributions here.

Good day @smlambert
I am an outreachy applicant. please can i work on this project.

Most certainly, welcome @benny-coy, if you have questions please feel free to ask in this issue!

Thank you @smlambert .

@smlambert
Please, i need your help i am new here.
Please i have forked the repository to my GitHub account and clone it to my local machine.
I have opened the folder to this project locally. i also opened the README.md file. i saw some documentation.
My question is am i to delete it or modify it? as stated here.
The contents of the README file can be simply some instructions for how to run the test locally. Use with the instructions from "https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/README.md#running-external-tests-locally and modify Steps 7 and 9 to reference payara-mp-tck specifically." thank you.
I hope to get your reply soon.

Please have a look at this PR as an example: https://github.com/AdoptOpenJDK/openjdk-tests/pull/2451/files

@smlambert  thank you.

@smlambert
I noticed that there might be a missing word here:
Running External Camel tests locally
To run any AQA tests locally, you follow the same pattern:
0. Ensure your test machine is set up with test prereqs. For external tests, you do need Docker installed.

Download/unpack the SDK you want to (is there a missing word here?) your test machine
If yes, please let me know the missing word?
Thank you.

@smlambert
Please, checkout this line for me.
modify Steps 7 and 9 to reference payara-mp-tck specifically.
7. Export required environment variables, BUILD_LIST and EXTRA_DOCKER_ARGS (example: export BUILD_LIST=external/payara-mp-tck and export EXTRA_DOCKER_ARGS="-v $TEST_JDK_HOME:/opt/java/openjdk"

make _payara-mp-tck_test   (When you defined BUILD_LIST to point to a directory in openjdk-tests/external, then this is a testCaseName from the playlist.xml file within the directory you chose).

Please is everything OK?

please, is this Introduction ok?
External Payara-mp-tck Tests
Payara-mp-tck tests are part of the third-party application tests that helps verify that the Adoptium binaries are good by running a variety of Java applications inside of Docker containers. AdoptOpenJDK/openjdk-tests/Issue #172 lists the applications that we have initially targeted to best exercise the Adoptium binaries. For each application, we choose to run a selection of their functional tests.payara-mp-tck tests are functional tests pulled from the payara-quarkus repository.
thank you.

@smlambert
The contents of the test.properties file will be a key=value format, copying the information from https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/external/common_functions.sh#L316-L328 to populate it.
Please does this means that i  should copy the test from above url and paste in the test.properties i have created

Download/unpack the SDK* that you want to test to your test machine

*you can download binaries from adoptopenjdk.net

@smlambert
Please, is there anything I still have to do concerning this issue?

Closed via https://github.com/AdoptOpenJDK/openjdk-tests/pull/2480
For future reference, if you would have put Fixes #2480 in the comment of the PR, this issue would have been automatically closed when the PR was merged


Penv 691 get records grpc handler

- What I did
penv-691 getrecords grpc handler
integration test with grpc and pg
- How I did it
- How to verify it
- Description for the changelog

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/1.2@97b5b88). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/1.2     #210   +/-   ##
==============================================
  Coverage               ?   59.96%
==============================================
  Files                  ?       61
  Lines                  ?     4986
  Branches               ?        0
==============================================
  Hits                   ?     2990
  Misses                 ?     1721
  Partials               ?      275

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 97b5b88...4f77121. Read the comment docs.


remove controllers/cache/suite_test.go

Ⅰ. Describe what this PR does
fix the master branch test failure. This test file is not much useful, so remove it
II. Does this pull request fix one issue?

III. Special notes for reviewers if any.

Codecov Report

Merging #213 (c5b5830) into master (fd31820) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #213   +/-   ##
=======================================
  Coverage   23.74%   23.74%
=======================================
  Files          78       78
  Lines        4764     4764
=======================================
  Hits         1131     1131
  Misses       3480     3480
  Partials      153      153

Flag
Coverage Δ

unittests
23.74% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


when I navigate from one page to another, the page doesn't scroll to top

You can reproduce this issue in demo itself.

open a page, then scroll till end.
now navigate to another page. the new page doesn't scroll to top. it is also in end of the page.

  "@coreui/coreui": "3.0.0",
   "@coreui/vue": "3.0.4",

working fine in 3.0.0 beta version.

This bug is fixed in @coreui/coreui 3.2.0 version


syntax error

code not working

hello, Please assign me this issue. I am participating in hacktoberfest 2022.
thank you :)


python3Packages.shapely: broken on darwin?

Describe the bug
python38Packages.shapely does not build for me on darwin. It looks like the previous release, 1.7.0, did build on hydra (https://hydra.nixos.org/build/133213521). I'm not sure if this is a difference depending on the version of shapely or on the version of macosx.
To Reproduce
$ nix-shell -p python38Package.shapely
Failure
$ nix-shell -p python38Package.shapely
[...]
running install tests
ImportError while loading conftest '/private/tmp/nix-build-python3.8-Shapely-1.7.1.drv-0/Shapely-1.7.1/tests/conftest.py'.
tests/__init__.py:4: in <module>
    from shapely.geos import geos_version_string, lgeos, WKTWriter
/nix/store/1l2vf26wvfsaddgm9w8ml8891bv0mv79-python3.8-Shapely-1.7.1/lib/python3.8/site-packages/shapely/geos.py:65: in <module>
    free = CDLL('/nix/store/alccnh6365ppv4690rinp7mswalm7gna-Libsystem-osx-10.12.6/lib/libc.dylib').free
/nix/store/c56kq0k3f3rdgkj3s1z5hw0j4ack9gdg-python3-3.8.6/lib/python3.8/ctypes/__init__.py:373: in __init__
    self._handle = _dlopen(self._name, mode)
E   OSError: dlopen(/nix/store/alccnh6365ppv4690rinp7mswalm7gna-Libsystem-osx-10.12.6/lib/libc.dylib, 6): image not found
builder for '/nix/store/vjidc9nms507pyvb7lnbm7gfhkv41m98-python3.8-Shapely-1.7.1.drv' failed with exit code 4
error: build of '/nix/store/vjidc9nms507pyvb7lnbm7gfhkv41m98-python3.8-Shapely-1.7.1.drv' failed

It looks like this store path is specifically being added by a nixpkgs patch, but that the patch refers to a file that doesn't exist.
Additional context
The're no libc.dylib here:
$ ls /nix/store/alccnh6365ppv4690rinp7mswalm7gna-Libsystem-osx-10.12.6/lib/
bundle1.o		crt1.o			lazydylib1.o		libdbm.tbd		libinfo.tbd		libproc.tbd		librpcsvc.tbd
crt0.o			dylib1.10.5.o		libSystem.B.tbd		libdl.tbd		libm.tbd		libpthread.tbd		libutil.tbd
crt1.10.5.o		dylib1.o		libSystem.tbd		libgcc_s.10.4.tbd	libmx.tbd		libresolv.9.dylib	system
crt1.10.6.o		gcrt1.o			libc.tbd		libgcc_s.10.5.tbd	libpoll.tbd		libresolv.dylib

Notify maintainers
@knedlsepp @dotlambda
Metadata
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-darwin"`
 - host os: `Darwin 16.7.0, macOS 10.12.6`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.10`
 - channels(mcgibbon): `"nixpkgs-21.03pre259239.39fe2fc4dc4"`
 - nixpkgs: `/Users/mcgibbon/.nix-defexpr/channels/nixpkgs`

Ah, it's broken on hydra too, so it's not just me: https://hydra.nixos.org/build/133636119

I think it's possible that the regression is related to how our shapely patches interact with https://github.com/NixOS/nixpkgs/pull/98541.


Improve description of curvedir values

Looking into the bend element, I noticed a problem with the description (or current usage) of the values for "above" and "below".
The value "above" in @curvedir is described as an "Upward curve", which is matching the following:

But this means for the following slurs, that the first one is "above" and the second one "below":

I think the current usage is that the first one is actually "below" (the notes). The "mixed" value only mentions explicitly "the items it pertains to".

Are you referring to the @curvedir attribute?

Yes, I was.

Sorry, just noticed that you referenced curvedir in the title.
The first slur is @curvedir="above" since it has an upward bow, so that makes sense to me. If you're looking for the placement in relation to the staff, @place is supposed to be for this, but I notice that @place is not available on <slur> -- it probably should be.

So this relates to #948?

Not entirely sure about the English usage, but "above" and "below" seem quite bad indicators of a DIRECTION? Shouldn't it be "up"/"down" or "upwards"/"downwards" instead, which could also make more sense in the context of an upwards bend or downwards bend rather than above bend or below bend?

[rettinghaus:] I think the current usage is that the first one is actually "below" (the notes).
[ahankinson:] The first slur is @curvedir="above" since it has an upward bow

The example in <IP_ADDRESS>Elements Without Semantic Implications seems to give a good indication of the  "wrong" usage so far (curvedir=above for a "downward" directed slur placed above the notes), as indicated by @rettinghaus .

I'm with @musicEnfanthen concerning the allowed values. Nevertheless, I think that the current ( above  |  below ) values are absolutely sufficient for slurs, and @ahankinson is right that @place might do a better job here, as the placement of the slurs would also implicitly indicate their curvature.

The current definition of the values, e.g., above being an upward curve, is not appropriate for a slur, as it is both up-down (above) or down-up (below).

@pe-ro for a (visual) variety of possible bends see https://musescore.org/en/handbook/4/guitar-bends

There seems to be some inconsistency in both definition of the values and usage.
Both slurs are "UPWARD" slurs:

Both slurs are "DOWNWARD" slurs:

Verovio actually treats the value of curvedir as "positional", not "directional" indication.

MEI Developer Workshop 2024: Maybe it is helpful to split up direction and position (as mentioned before).
E.g., curvedir="down" place="above" for a downwards slur positioned above (as optional attributes, depending on the use case).
Would be a breaking change (-> MEI 6).


Authorization with Google is not working

It doesn't work on the demo as well.

Try to Signin with google
Logout
Sign in again with google
http://nodge.ru/yii-eauth/demo2/login

I solve this poblem:
In /vendor/nodge/yii2-eauth/src/oauth2
in function getCallbackUrl need replace
foreach (['code', 'state', 'redirect_uri'] as $param) {
to
foreach (['code', 'state', 'redirect_uri', 'scope'] as $param) {

@web-rider works like a charm. Thank you

@web-rider works,thanks

Need to add a fix to the main code, to be able to update through the composer. @Nodge

There's already a pull-request for this. I don't think this repo is maintained anymore. Also, you may want to follow this pull request and move that specifically to the Google OAuth Class so it has no detrimental impacts to any other services
https://github.com/Nodge/yii2-eauth/pull/120

There's already a pull-request for this. I don't think this repo is maintained anymore. Also, you may want to follow this pull request and move that specifically to the Google OAuth Class so it has no detrimental impacts to any other services
Pull Request https://github.com/Nodge/yii2-eauth/pull/120

There is an other one to unset 'hd'
			foreach (['code', 'state', 'redirect_uri', 'scope', 'authuser', 'prompt', 'session_state', 'hd'] as $param) {
				if (isset($route[$param])) {
					unset($route[$param]);
				}
			}


issue-457: Update distributionWallet fields.

…ibution wallet consistent with the assets.json

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Add distributionWallet to reference server and make the default value of the distributionWallet consistent with the assets-test.json
Why
N/A
Known limitations
N/A

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr458.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr458.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr458.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr458.previews.kube001.services.stellar-ops.com/


Restarting the python debugger does not save changed files during debug session

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  1.70.1
OS Version: Linux x64 5.15.0-43-generic

Steps to Reproduce:

Put a breakpoint in code
Start python debugging session
When stopped at breakpoint edit an change code
Restart debugging session (ctrl-shift-F5)
The changes are not saved, the debugger happily runs the old version

This behaviour is the same reported by the issue below, but within a Python debugging session:
JS Debug not updating #27880

Looks like a duplicate of #157655 which is scheduled to be fixed in upcoming 1.70.2 later this week.

/duplicate #157655

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Dealing with GO duplicate annotation in the log files

https://curation.pombase.org/dumps/latest_build/logs/log.2021-09-27-21-52-56.chado_checks
All duplicate GO annotation.  Columns are: count, systematic ID, gene name, term name, evidence, PubMed ID, 'with', session ID - CHECK FAILURE:  expected 0 but got 76
Now we have established that the UniProt data does get in, I suspect this is what is causing the majority of the duplicates.
For instance  this one:
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPAC17G6.17
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPAC29E6.01
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPAC57A10.05c
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPAC6F6.02c
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPBC1271.01c
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPBC1703.06
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPBC25B2.11
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPBC29A3.08
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPBC336.01
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPBC3H7.06c
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPBC56F2.01
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPCC18.04
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15147268	PomBase:SPCC338.16
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15555586	PomBase:SPBC776.18c
2	SPBC409.05.1	skp1	protein binding	Inferred from Physical Interaction	PMID:15555586	PomBase:SPBP16F5.02
2	SPBC646.05c.1	erg9	protein binding	Inferred from Physical Interaction	PMID:17016471	PomBase:SPAC13D6.01
all of the existing annotation seems to be in Artemis (Paper not yet curated in Canto)
AND  I remember that Skp1 has a lot of duplicate annotation from UniProt
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPAC17G6.17; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPAC29E6.01; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPAC57A10.05c; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPAC6F6.02c; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPBC1271.01c; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPBC1703.06; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPBC25B2.11; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPBC29A3.08; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPBC336.01; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPBC3H7.06c; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPBC56F2.01; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPCC18.04; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15147268; with=PomBase:SPCC338.16; date=20080122"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15555586; with=PomBase:SPBC776.18c; date=20050119"
/GO="aspect=F; term=protein binding; GOid=GO:0005515; evidence=IPI; db_xref=PMID:15555586; with=PomBase:SPBP16F5.02; date=20050119"
Clearly we don't want exact duplicate annotation from the same paper, and this should not happen any more because UniProt don't really curate fission yeast entries. But, in these cases can we filter the redundant entry if the annotation provider is UniProt  (we don't need to report these)?
This way we would know if the exact duplicates are due to something internal (like a term merge)

I reviewed these a while back, with occasional updates (most recently over a year ago, but the UniProt duplicates hardly ever change) ... see attached.
In the past I've fixed a few genuine duplicates on our end. I don't think I ever seen any that could be attributed to term merges; the most common thing seemed to be not deleting an annotation from contig files after redoing it in Canto.
duplicate_GO_to_filter_2020-06-26.txt

[ ] Action, don't report duplicate Uniprot annotations int he log file

Shouldn't these duplicate be removed by the GO filtering step?  Should I be investigating a bug in that code?

The code is working  as currently expected.
We  keep all UniPRot experimental annotation, AND we filter exact duplicates (other than date), this  annotation will stay because the "assigned by" field is different.
This is really a GO issue. Different resources shouldn't need to make a GO annotation that already exists, duplicate annotations of the same experiment, same paper/experiment redundancy is not useful.
GO should already have mechanisms in the pipeline to deal with this type of duplication since it will be an inherent feature of Noctua annotation.
It might be useful to put the UniProt and Intact EXP duplicates in a separate log. `There is nothing we can do to fix those, but it is good to have a record so that we can keep an eye on the extent of the issue. For instance, if Intact starts to do lots of pombe annotation this would generate lots of redundant IPI data.

We keep all UniPRot experimental annotation, AND we filter exact duplicates (other than date), this annotation will stay because the "assigned by" field is different.

Should the GO filtering handle that by having a priority system for assigned by too?  So that if two annotations are the same except for assigned by, we just keep one?  That seems better than having two identical annotations in Chado.

Yes that is probably what should happen. We don't need to ever load the duplicates.
However, it would be good to see a list in a separate log just to keep handle on it. We are the database authority, and we should know how much of other resources annotations we are removing (and discourage them from making such duplicate annotations). Would it be possible to put the filtered annotation of this type into a log file?

Would it be possible to put the filtered annotation of this type into a log file?

Yep.  The duplicate removal process can do that.

I've written the code to report and delete the annotations from UniProt that are a duplicates of PomBase annotations.  Here's my first attempt at a log file:
https://curation.pombase.org/kmr44/tmp/uniprot_annotations_to_delete.tsv
I realise there are a few too many columns, but do the rows like sensible?

yes, that's all I need actually...

I've added the duplicate deletion process for tonight's load.

Can we close this?

I think so, we have the file of UniProt duplicates that are filtered, and the ones which need fixing remain in the logs. Much better.


ReDoc UI - maximum number of operations/Chrome?

Hi all,
I've noticed in the last few days that our API, which uses Swashbuckle 5.0.0 (full release) has started appearing buggy within Chrome. The API makes use of the x-tagGroups feature to group endpoints into categories, and the API has grown quite a bit in the last few weeks.
The issue can be seen at https://api.billingbooth.app/index.html - when clicking any of the headers, such as Customers, Charges, then using the mouse scroll wheel, the menus on the left hand side disappear and randomly appear/disappear.

This is only happening in Chrome, and I'm not understanding whether this is a Chrome-update that has somehow screwed things up, or whether the problem is occurring due to the number of endpoints in the API.
Has anyone else experienced this?

It seems that Chrome released a new version (80.0.3987.100) two days ago, and I started noticing this problem yesterday, so it's possible the latest Chrome breaks something in the ReDoc UI?
https://chromereleases.googleblog.com/2020/02/stable-channel-update-for-desktop_11.html

Happening to almost all chrome users from what I've seen

The issue is referenced in the ReDoc repo at https://github.com/Redocly/redoc/issues/1170

So have you tried the latest version?

I'm using Swashbuckle.AspNetCore.ReDoc 5.0.0 (full release), which I'm guessing doesn't have the latest RC of ReDoc in it. It's still having this issue.


As a student financial advisor, I can edit lead details to correct mistakes.

Closed by extension of #108


Add Huggingface Integration

Fixes #727
Proposed Changes

Added Huggingface Upload & Download functionality in a subcommand.
Added library name, version & git hash in huggingface tags for huggingface uploads.
Added functionality to save a copy of config.yaml during training.

Checklist

[ ] CONTRIBUTING guide has been followed.
[x] PR is based on the current GaNDLF master .
[ ] Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change.
[ ] Function/class source code documentation added/updated (ensure typing is used to provide type hints, including and not limited to using Optional if a variable has a pre-defined value).
[x] Code has been blacked for style consistency and linting.
[ ] If applicable, version information has been updated in GANDLF/version.py.
[ ] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file.
[ ] Usage documentation has been updated, if appropriate.
[ ] Tests added or modified to cover the changes; if coverage is reduced, please give explanation.
[x] If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].

Hi @Wauplin and @NielsRogge - this PR looks good from my end. Do you have any feedback?

@pranayasinghcsmpl some lint fixes (unused variables and whatnot) will be needed for this PR. Thanks for taking care of it!
Thanks for your comments and suggestions, @Wauplin!

Support ticket generated with Codacy to explore the coverage issue.

Codacy folks suggested not to use coverage reporter for anything coming in from other forks 🙄
Anyway, we should be good to go from my end. @Wauplin is this PR good to merge for you?

@Wauplin I think this PR is good to merge from my side. Could you please confirm and resolve your comments? I think the remaining parts can be taken up in separate issues/PRs. Thoughts?


Update images in order to support Kubernetes 1.23

What this PR does / why we need it:
Update images in order to support Kubernetes 1.23. vSphere images are not updated at this point because we can't test the vSphere changes.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
xref #1685
Does this PR introduce a user-facing change?:

/assign @kron4eg


AIOHTTP

root@localhost:~/Tools/OpenRedireX# python3.7 openredirex.py -l url.txt -p payloads.txt --keyword FUZZ
Traceback (most recent call last):
File "openredirex.py", line 2, in
import aiohttp
ModuleNotFoundError: No module named 'aiohttp'

[+] Total URLs loaded : 1701
[+] Total Payloads loaded : 44
[+] Scanning Started [Only Suspected URLs with redirect history will be displayed , all others will be ignored]

I've Installed it using pip3 install aiohttp

Follow readme , and try to use this inside a virtual environment !

python3.7 -m pip install aiohttp


Allow initial mode to be set

I'm submitting a
Feature request
Current behavior
It looks like viewMode is always the default option and it is not possible to start with edit mode initially.
Expected behavior
Allow initial mode to be programmatically set (either being view or edit mode).
Minimal reproduction of the problem with instructions
No response
What is the motivation / use case for changing the behavior?
No response
Environment
No response

I've figured out a workaround using ViewChild to get the instance of the EditableComponent and call editMode.set(true) .
import { xxx, ViewChild, AfterViewInit } from '@angular/core';

export class XXX... {
  @ViewChild(EditableComponent) editableComponent!: EditableComponent;

  ngAfterViewInit() {
    this.editableComponent.editMode.set(true);
  }
}

The solution above works. There is also a method called displayEditMode which does the same thing.
this.editableComponent().displayEditMode();
This ticket can be closed now.


Compose Unity into SwiftUI whenever internal superview is changed

Closes #13
These changes are for the old/unity2020 branch. The code in the main branch does not actually compose Unity into a SwiftUI element and so is not applicable to this problem/solution.
Creating a SwiftUI element that displays Unity was already accomplished by way of the container view technique; a custom UIViewControllerRepresentable sets Unity's internal root view as its subview. However, this is not guaranteed to always work because we don't know when and if Unity will make its root view a subview of something else internally. This is indeed the case shortly after running Unity, and it resulted in our superview getting replaced and Unity not showing.
From the README

Between the moment you create the UnityFramework instance, and the moment you can display it, it seems like there is an issue. Some people try to add a delay, but this is definitely a hack.

In effect we have a race condition between who can apply their superview last, our code or Unity's code. We solve this problem by recognizing the fact that Unity's root view belongs to its root view controller which in turn belongs to its UIWindow. The window takes the role of a superview. Therefore, the superview-subview hierarchy is only established when the rootViewController property of the window is set (or reset). By registering an observer on this property, we can apply our superview immediately after the hierarchy is complete. This ensures our desired hierarchy always takes precedence and is maintained for the duration of Unity's lifecycle.
It is also worth mentioning that as a true SwiftUI element, we can resize and reposition Unity as we see fit. Unity's own documentation says

Unity as a Library only supports full-screen rendering, and doesn’t support rendering on part of the screen.

which is true by default, but we have bypassed this with the container view technique. We demonstrate this with the example element OtherUnityView.

@DavidPeicho I think I'm done making commits; it runs without error. Please review at your leisure.

I was just reading through issue #3 and I do see on higher versions of Unity that same problem. The code here works fine on version 2020.2.1f1 for which this branch was made, but I will see if I can address that problem on 2021.3.18f1 which is the version I am using in my own project.

It now works correctly on both 2020.2.1f1 and 021.3.18f1 - we can interact with our own content as well as the content within Unity itself, so the problems described in #3 are gone. I added your AutoRotate.cs script to demonstrate touch interaction with the Unity view; up to you if you want to add the color picker thing as well and/or upgrade the Unity version. At this point I'm not sure what advantage if any the main branch has over this code since the solution here is more succinct and more powerful in terms of UI. Let me know what you think.

@bdeweygit Thank you so much for forking the project! It works perfectly and even solves issues #10 #9! Really appreciate it!


The PoC and Pipeline is not about AzureML specifically.

Description

The PoC and Pipeline is not about AzureML specifically.
Fixes https://github.com/opendatahub-io/ai-edge/issues/94.
How Has This Been Tested?

Manually.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

My tests passed, ready for review.


OpenFisca-Israel rules coding

Coding of israel rules into the OpenFisca-Israel instance
needs #1
needs #4

Acceptance Criteria

Implement Straw Man (it might not be correct, it's something to point at)

@dlouise64 Can you drop the rules on here for Brenda - thanks :)

Rules here https://github.com/ServiceInnovationLab/Piccolo/issues/4#issuecomment-429688879


Powder keg fix

rebased and pointed to develop-rika

Build Artifacts

2ship-linux.zip
2ship-mac.zip
2ship-windows.zip

closing, duplicating work.


Consider MaxUserNameListLength in UniqueIDJDBCUserStoreManager

Fixes https://github.com/wso2/product-is/issues/11734
Fix

Add max user length from userstore properties
Its not there add via constant (100)

This fix also add search time property also (10000ms)
Reason for dependency addition
Recently One update in orbit moves xerces jar from svn to github and upgrade the version to 2.12.1
which caused the build failure.
Hence we need to update the dependency
We need xml-apis dependency because of build failure
Ref:
[ERROR] Tests run: 16, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.927 s <<< FAILURE! - in TestSuite
[ERROR] testGetDocumentElement(org.wso2.carbon.base.ServerConfigurationTest)  Time elapsed: 0.16 s  <<< FAILURE!
java.lang.NoClassDefFoundError: org/w3c/dom/ElementTraversal
        at org.wso2.carbon.base.ServerConfigurationTest.testGetDocumentElement(ServerConfigurationTest.java:139)
Caused by: java.lang.ClassNotFoundException: org.w3c.dom.ElementTraversal
        at org.wso2.carbon.base.ServerConfigurationTest.testGetDocumentElement(ServerConfigurationTest.java:139)

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/896039175

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/896039175
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/896333193

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/896333193
Status: failure

@Thumimku It seems the PR builder is failing

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/898231679

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/898231679
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/898400237

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/898400237
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/898626338

Merging as Dev PR builder  approval


Disable Automatic Redirection

I cannot seem to find an option to disable automatic redirection. I need to test the redirect whether it's a 301 or 302 header response.

Capybara is designed to mimic a user - they don't see the redirect response, they see the page redirected to - hencen most Capybara drivers don't provide it to you.   You can technically do it with the rack_test driver by registering one created with the follow_redirects: false option but it usually easier to just test for redirect responses in your controller specs/tests.


Add dashboard for argo workflows

What does this PR do?

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Left some comments!
Would it make sense to have template variables?

Good question! I checked and don't get a feeling there's clear variables to filter by. We'd basically be monitoring just one pod/service/endpoint. This then has some metrics that are tagged by various things, but nothing where we can say we can meaningfully narrow all metrics down.


Posts page not working

Hello,
not quite sure if I need some extra configuration for the "Posts" page - if I don't have a content/posts.md I get a 404 if I access the page, if I have a page and have layout:"posts"  on it I just get an empty post.

Ah, apprently it has to be content/posts instead of content/post . Thank you for the nice theme!


[SPARK-16862] Configurable buffer size in UnsafeSorterSpillReader

What changes were proposed in this pull request?
Jira: https://issues.apache.org/jira/browse/SPARK-16862
BufferedInputStream used in UnsafeSorterSpillReader uses the default 8k buffer to read data off disk. This PR makes it configurable to improve on disk reads. I have made the default value to be 1 MB as with that value I observed improved performance.
How was this patch tested?
I am relying on the existing unit tests.
Performance
After deploying this change to prod and setting the config to 1 mb, there was a 12% reduction in the CPU time and 19.5% reduction in CPU reservation time.

Test build #64123 has started for PR 14726 at commit c4f37b6.

Test build #64123 has finished for PR 14726 at commit c4f37b6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64123/
Test FAILed.

Test build #64135 has started for PR 14726 at commit 6d58b9d.

Test build #64135 has finished for PR 14726 at commit 6d58b9d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64135/
Test PASSed.

Thanks - merging in master.


React Dev Tools

Hello, I am in on the React Dev Tools videos of this course and I have coded up to this point and I am getting an error.  The error I am getting is the "Uncaught ReferenceError: process is not defined" and google is telling me to fix it exactly how Brian is saying not to.  I have not come across a way to fix this and still having issues, has anyone else had this error and how did you fix it?  I do not have webpack installed or any of the ones Brian said in the video ...
Uncaught ReferenceError: process is not defined     at<EMAIL_ADDRESS>(core.ts:40:17)     at __require (chunk-RSJERJUL.js?v=29b6a201:3:50)     at<EMAIL_ADDRESS>(index.ts:2:1)     at __require (chunk-RSJERJUL.js?v=29b6a201:3:50)     at<EMAIL_ADDRESS>(validate.ts:1:1)     at __require (chunk-RSJERJUL.js?v=29b6a201:3:50)     at<EMAIL_ADDRESS>(validateNode.ts:1:1)     at __require (chunk-RSJERJUL.js?v=29b6a201:3:50)     at<EMAIL_ADDRESS>(index.ts:5:1)     at __require (chunk-RSJERJUL.js?v=29b6a201:3:50)

Were you still having issues? I tested the project (using Node version 16) and it worked. Brian doesn't use Webpack in the course so you shouldn't need that. The two things I would try:

Make sure you are using Node version 16 or higher
Try pulling down the code from the next checkpoint (after the React Router lesson) and start fresh from there: https://github.com/btholt/citr-v8-project/tree/main/08-react-router


Uservoiceshow not working

I check my macbook and it worked but on my Windows pc the plugin does not work! Idk if a better discord think or a computer thing

Adding for showing error

start working to solve the issue

Thank you <3

Hi, I just wanted to ask if there was any update that we can have.
Thank you <3

Hi, I just wanted to ask if there was any update that we can have. Thank you <3

I found fix for UserPopoutBody, but im trying to found component that represent UserProfileModalHeader
I'll try to finish as soon as possible.

Hi, I just wanted to ask if there was any update that we can have. Thank you <3

I found fix for UserPopoutBody, but im trying to found component that represent UserProfileModalHeader
I'll try to finish as soon as possible.

Ok, Thank you <3

Hi, Its been a few weeks and i was wondering if there was an update (i am more asking for BD template they have for plugins)

Does not work for me, with v1.1.6.

Does not work for me, with v1.1.6.

Can you open the new issues and give more info, so i can start work around


Pluginlib implementation

[ ] Implement pluginlib for services
[ ] Implement pluginlib for actions
[ ] Fix Launch files

We can now load dashboard API plugins but the service server never appears on screens...

We can now load dashboard API plugins but the service server never appears on screens...

This has been solved


Setting extraGlobals: ['Math'] breaks in v26

💥 Regression Report
TypeError: Math.random is not a function

      at randomIntInRange (node_modules/source-map/lib/quick-sort.js:43:33)

Last working version
Worked up to version: 25.5.4
Stopped working in version: 26.0.0

I think it might be related to some transient dependency as I have other repos with<EMAIL_ADDRESS>that work just fine, but installing from scratch breaks. Yay lockfiles.

To Reproduce
Steps to reproduce the behavior:
// jest.config.js
module.exports = {
  extraGlobals: ['Math']
}

Expected behavior
Tests pass and accessing Math is fast. See https://github.com/facebook/jest/issues/5163 on why this is needed.
Link to repl or repo (highly encouraged)
https://repl.it/repls/RoundFortunateUnits#jest.config.js
Run npx envinfo --preset jest
Paste the results here:
  Binaries:
    Node: 12.16.0
    Yarn: 1.22.0
    npm: 6.13.4
  npmPackages:
    jest: ~26.0.0 => 26.0.1

Sorry about that! Inadvertently broken in a refactor. I've landed a fix along with a regression test - will release tomorrow or Monday 🙂


Plot infinite function

hallo, what is the new "plot infinite" function in the gui ? i cannot find any info abou this .

Pretty sure it means "keep plotting until you run out of space".  As opposed to "plot 1" or "plot 50" which both stop once they hit their limit.

i think this function was already there with "n-1" in the plotter, so i was hoping for an new infinitely  fast plot option !  :-)
thanks for the reply

Jago-Telesin explained it well. Many people wanted an option for it to just keep plotting. This could be done before by adding a very high number to the total plots but this just makes it a clearer option. As long as you don't have "Ignore Free Space" Checked the plotter should "stop" when there is no more free space, the queues will just be blocked from starting new plotting processes.


Fix some dark mode issues / add back the guide icons

fixes https://github.com/quarkusio/quarkusio.github.io/issues/2007
+fixes https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/Quarkus.2Eio.20search/near/444013207
The icons disappeared as the bundler property key got updated 😕

@marko-bekhta was it actually pushed to production? I don't see any changes on the website despite the workflow run in the Actions tab.

I think we just need to wait for the quarkus.io to get re-built, so that the new script is fetched... from what it seems the script here https://search.quarkus.io/static/bundle/main.js is updated ...

looks better now 🙈

Cool, thanks!


update trigger logic for #383

 Correct a bug on the way the message "PCIEvolBiol #XXX: Editorial decision possible #ReminderRecommender2ReviewsReceivedCouldMakeDecision" is set


Custom namespaces

Hi
I need to know if there is a solution to change default namespaces ns1,ns2 etc... with some custom names ?
In my case it's a real problem cause I can't call provider service which is waiting right namespaces instead of ns1...
If I send a structure like
[
data: [
name: xx
age: yy
]
]
Request is changed into
ns2:datans1:namexx</ns1:name> etc...
And provider doesn't understand it ...
Thanks for your help
Best Regards
Pascall

Hi,
I fact it was not a problem with Zend Soap, but with provider specification which says it was SOAP 1.2 but it seems to be SOAP 1.1 because it works with this soap version.
Thanks
Best Regards