Problem: Unable to install psutil from a requirements.txt file when running in docker

Issue
When attempting to start a docker container with the following requirements.txt file, it fails due to missing dependencies.
I narrowed down the issue to the psutil pip package requiring gcc to be installed.
I am unsure on the best way to proceed with this or if there is another way to get the psutil package on the container.
Impact
Docker
Installation method
Docker
Database
MySQL
Browser
Firefox
Semaphore Version
develop-cde8515-1723307466
Ansible Version
No response
Logs & errors
Installing additional python dependencies
Requirement already satisfied: wheel in /opt/semaphore/apps/ansible/9.4.0/venv/lib/python3.11/site-packages (from -r /etc/semaphore/requirements.txt (line 1)) (0.44.0)
Collecting psutil (from -r /etc/semaphore/requirements.txt (line 2))
  Using cached psutil-6.0.0.tar.gz (508 kB)
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'done'
  Preparing metadata (pyproject.toml): started
  Preparing metadata (pyproject.toml): finished with status 'done'
Building wheels for collected packages: psutil
  Building wheel for psutil (pyproject.toml): started
  Building wheel for psutil (pyproject.toml): finished with status 'error'
  error: subprocess-exited-with-error

  × Building wheel for psutil (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [46 lines of output]
      running bdist_wheel
      running build
      running build_py
      creating build
      creating build/lib.linux-x86_64-cpython-311
      creating build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_psaix.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_pswindows.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/__init__.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_psposix.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_common.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_psbsd.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_pslinux.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_psosx.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_compat.py -> build/lib.linux-x86_64-cpython-311/psutil
      copying psutil/_pssunos.py -> build/lib.linux-x86_64-cpython-311/psutil
      creating build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_contracts.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_misc.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/__init__.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_unicode.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_bsd.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_sunos.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_linux.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/runner.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_windows.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_connections.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_aix.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_osx.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_process_all.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_memleaks.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_system.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_process.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_testutils.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/test_posix.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
      copying psutil/tests/__main__.py -> build/lib.linux-x86_64-cpython-311/psutil/tests
Failed to build psutil
      running build_ext
      building 'psutil._psutil_linux' extension
      creating build/temp.linux-x86_64-cpython-311
      creating build/temp.linux-x86_64-cpython-311/psutil
      creating build/temp.linux-x86_64-cpython-311/psutil/arch
      creating build/temp.linux-x86_64-cpython-311/psutil/arch/linux
      gcc -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -DPSUTIL_POSIX=1 -DPSUTIL_SIZEOF_PID_T=4 -DPSUTIL_VERSION=600 -DPy_LIMITED_API=0x03060000 -DPSUTIL_ETHTOOL_MISSING_TYPES=1 -DPSUTIL_LINUX=1 -I/opt/semaphore/apps/ansible/9.4.0/venv/include -I/usr/include/python3.11 -c psutil/_psutil_common.c -o build/temp.linux-x86_64-cpython-311/psutil/_psutil_common.o
      psutil could not be installed from sources because gcc is not installed. Try running:
        sudo apk add gcc python3-dev musl-dev linux-headers
      error: command 'gcc' failed: No such file or directory
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for psutil
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (psutil)

Manual installation - system information
No response
Configuration
No response
Additional information
No response

      psutil could not be installed from sources because gcc is not installed. Try running:
        sudo apk add gcc python3-dev musl-dev linux-headers
      error: command 'gcc' failed: No such file or directory

We are can't include this packages to the default image because it takes a lot of space.


[Feature Request] Notifications

Current state

Useful information is currently relegated to the logger which only appears in console output. Eg. archives corrupted or unsupported.
Separately, there is no facility to announce/notify users within the app Eg. new release changelog

Problem

The logger information is important but easily overlooked.
Users may not be aware of new features as they become available.

Solution
Notification system.
Requirements

Notifications page
Global notification icon in nav (with dynamic badge count)
Don't allow duplicate notifications eg. Corrupted archive on next scan should not create a second notification (unless already dismissed? see questions below)
More here...

Questions

Just for Admins? Or all users? Or ability to set which notifications will show for admin vs all?
What are examples of the types of notifications we expect? (so we can try and plan the UI / generalise the columns etc.)

Archive scan related eg. Archive corrupted, unsupported
New release announcements
What else?

What are a few examples of good notification systems we can be inspired from? Eg. Github?
Are all notifications dismissable? I guess the scan related ones you'd always want to know about, but if you dismiss, it would show again after the next scan so not really an issue?

Original comments in #49
@hkalexling some initial comments added, but no doubt will need to change / be expanded.

Apart from the types listed above, we can also notify users when a MangaDex download is completed or finished with an error. These notifications should be sent to admins only.
When we have the MangaDex updater (#24), we should also notify users when new chapters are available to read. This type of notifications should be visible to all users.
I can't think of a good example right now, but what I have in mind is indeed close to what GitHub has (icon in nav and the actual notifications are on a separate page).
Here's how I imagine the notification feature would work (open for discussions)

The dynamic badge shows the number of new notifications available
When a user clicks on the icon and opens the notification page, all new notifications are marked as read (is this what you meant by "dismiss"?), and the badge disappears for that particular user
The dismissed notifications are still there, and the user can always visit the page to read them
Notifications can be manually deleted
An error notification (e.g., corrupted archives) should replace the existing notification on next scan if it's already dismissed, and it should be moved to the top of the list (assuming the notifications are sorted from most reason to oldest)


pwned_password

Hey!
It looks like https://github.com/michaelbanfield/devise-pwned_password is not maintained anymore, so what about to copy their functionality in your gem?

While they have not released an updated version, there does appear to be some activity from a few months ago.


Implement Function: Array.pick_random()

Implements: https://github.com/godotengine/godot-proposals/issues/3454
PS: This is partially implemented, since some users suggested adding for another classes, but I think each class should have their own PR, what you guys think?
test code:
var arr:Array = []

func _ready() -> void:
	arr.append_array(["test",1,2,3])
	print(arr)
	for i in range(10):
		print("Result: ", arr.pick_random())

output:
First Run:
[test, 1, 2, 3]
Result: 2
Result: 1
Result: 3
Result: 1
Result: 3
Result: 3
Result: 2
Result: 1
Result: 1
Result: 1

Second Run:
[test, 1, 2, 3]
Result: 1
Result: test
Result: 2
Result: 2
Result: 1
Result: 1
Result: 2
Result: 3
Result: 3
Result: 2

Would be interested in adding in probabilities too?
Like this?
[1, 0, 2, 3].pick_random([0.25, 0.25, 0.25, 0.25])
It's your choice to to decide whether the sum of all probabilities must be 1. It's how it works in Python.

Would be interested in adding in probabilities too? Like this? [1, 0, 2, 3].pick_random([0.25, 0.25, 0.25, 0.25]) It's your choice to to decide whether the sum of all probabilities must be 1. It's how it works in Python.

please create a proposal for that, every implementation depends on the users needs.

You need to update the class reference with --doctool and write documentation for the new method: https://docs.godotengine.org/en/latest/community/contributing/updating_the_class_reference.html
Apparently there's also a style issue but not sure which, you should likely run clang-format.

@akien-mga just fixed everything!

Please squash the commits when it's ready, we don't want 5 extra merge commits in the Git history.

Seems this includes a lot of other commits :) Needs a proper rebase.

@nonunknown has been struggling with rebasing on this one. Maybe a contributor can help salvage this?


Unable to get to advanced menu in PXT on iPhoneSE

microbit.org-ticket: 1499
Dear Sirs,
I am using PXT with iPhoneSE.
When using in block mode, I can not see the bottom side of the menu(AdvancedMenu).
I am happy if you refurbish.
Thanks,

Bumping to P1 as there are two instance of the same bug

Back to P2?
I spotted some related things to do with screen size, is it possible to roll these into one change perhaps?
This, for example, is related I think...
https://github.com/Microsoft/pxt/issues/2000

@DavidWhaleMEF Data shows there is really tine fraction of folks going to iPhone. Didn't warrant P1.

microbit.org-ticket: 1251 also logged this issue via https://github.com/Microsoft/pxt/issues/1909

@abchatra Data from app developers seems inconsistent in scale with your reports of usage? Noted that there was a line-item in a recent meeting note circulated by Jonny Austin, that you were going to look into working up some additional stats to try to understand those discrepancies?

@DavidWhaleMEF This issue is only on IPhone & not Ipad or other tablet.

I have already shared the stats with Johnny. Do you need any other stats?

@abchatra No that's fine - thanks for sharing stats with Jonny, he'll follow up with that now. Thanks for your help.

It is a problem with iPad held landscape.

@martinwork Do you see this issue in Ipad?

In landscape, I can open the advanced menu but cannot get to all items because it doesn't scroll.

@guillaumejenkins Can you take a look, when you have time?

I'll investigate when I get the chance

On an iPad, the invisibility of  the Advanced menu in landscape mode is a fairly longstanding ‘feature’.  The solution is to turn to portrait mode.  Then you get another problem: the input box for editing the file name disappears!

microbit.org-ticket: 1928
Another customer logged this as a problem

Thanks! We are working on it

Fixed in the Beta website by making the toolbox scrollable with touch gestures.
Versions with the fix:

pxt.microbit.org version: 0.12.24
Microsoft MakeCode version: 0.12.82

This is not in the live website yet, but will be when we do our next release.
@LaurenceRogers the project name text box disappearing is not a bug, it's simply because the screen width in portrait orientation is below the threshold at which we remove certain UI elements to make the UI simpler

@guillaumejenkins Tested the scrolling on iPad - excellent!
Note, on an iPad there's more space between download and disc buttons in portrait orientation than landscape, so no reason for the text box to disappear. It does appear in roughly the same space when the simulator is switched on.


Add a SQLAlchemy dialect

This makes it possible to use cockroachdb with the SQLAlchemy package and serves as a prototype for integration with other ORMs.
The tests pass as configured when run locally with a running cockroach server; I'm leaving the CI setup for a future PR.

I've updated this to reflect our improved support for TIMESTAMP WITHOUT TIME ZONE. Any python users want to take a look? I'm looking at you @cuongdo and @andreimatei.

I've reviewed this for "Pythonic-ness" and, to a lesser extent, how it fits in with the sqlalchemy dialect framework. @andreimatei, you should still take a look, especially the SQL-specific stuff.

All comments addressed.

LGTM
I've got nothing :)
I'm unsure about how people are supposed to use this repo. Are they supposed to import sqlalchemy from here? I see there's a setup.py script but I'm not sure what that does.

Real-world examples are coming soon, but the idea is that you'll install this package, and then use sqlalchemy as usual with a database url of cockroachdb://localhost/...


Remove service

The service class has less methods that the mediainfo object it wraps,
and proxies the only method it has.

Also, things inside MediaInfo should be injected, there are several Dependency Inversion Principle there. That would make the bundle a bit less trivial, and a bit more useful. I can contribute such changes in a BC way

@greg0ire I'm totally agree with you, for the moment this bundle has too few feature, to be used instead of php-mediainfo directly.
If you want to add some new feature to MhorMediaInfoBundle, I will be happy to review and merge them, into a 2.0 version of this bundle.

Closing this for now.


Cargo subcommand instead of shell scripts

TLDR: it would be nice to have something like cargo cl-codegen build and similar.
For now i think it is quite clumsy to specify magic env-var(s) and running magic shell scripts by their full paths. Though, these shell scripts might be quite handy for development, they are not really convenient for usage (and probably for maintaining).
Perhaps it would be much easier to use cranelift codegen via single binary, that can be configured once with persistent file. I am not really sure about all of nitty-gritty details, but probably it can be handled with cargo sub-command with additional config file (e.g. for target, librustc_codegen_cranelift.so path, path for sysroot, etc). Do I miss something important and what do you think about that?

I'd love to work on this, if it is available

Go ahead. Since https://github.com/bjorn3/rustc_codegen_cranelift/pull/1089 the shell scripts have been simplified a bit, so implementing this takes less work.

As of https://github.com/bjorn3/rustc_codegen_cranelift/pull/1104 the build directory now contains everything cg_clif needs to run. There still isn't a cargo subcommand yet though, but you can copy the build dir somewhere and add cargo.sh to your path. (optionally renaming it to something else)

This should now be a matter of renaming the produced cargo executable to cargo-clif. If you then add the build dir to your path you will be able to do cargo clif build. Going to rename it tomorrow.


[5.6] Add phpdoc with $factory type hint in factory stub

It would be pretty convenient to have $factory typehinted in generated factories so that the IDE knows how to handle that $factory var.
Ps. It's obviously not breaking any existing feature. Also I guess it would be pointless to test it.

No plans on adding this.


chore(ebpf): update libbpf to v1

We are maintaining quite old commit of libbpf as KubeArmor submodule.
We need this submodule for compiling our system monitor byte code on the fly.
libbpf is now v1, and some of the APIs have changed so the update is not straight forward. We need to update code in https://github.com/kubearmor/KubeArmor/blob/main/KubeArmor/BPF/system_monitor.c to remove usage of deprecated APIs.

A total of 3 vulnerabilities exist. 2 of them have been fixed upstream, and 1 remains.
Fixed:
https://osv.dev/vulnerability/OSV-2021-1562
https://osv.dev/vulnerability/OSV-2021-1576
Unpached:
https://osv.dev/vulnerability/OSV-2023-877

@daemon1024 should I raise an issue with libbpf for unpached?

It's fine let's just use the latest released version for now.
We can update again when they fix it


Webpack loader support

Webpack loaders are a special way to "load" code into webpack.
There are two syntax to use webpack loader:

Via require function
Via webpack config file

@lijunle is the issue fixed

No, not yet. I do not have full scenario to support this. If you have a project using webpack, then you can depcheck the project and resolve the webpack issues. I guess webpack might be complex because it use its naming convention to require packages.

@lijunle to check webpack dependencies do I need to use special commands

Let me clarify, webpack has many loaders, plugins, etc. There is a simple to depend on one webpack loader/plugin (check this docs). Those usage does not follow require function call. We may need to some special way to detect them.
Currently, such special command does not exist.
If you want to work on this, read webpack's document and depcheck's pluggable design doc.

@lijunle does this npm package https://github.com/dylang/npm-check supports  webpack

I don't think they support. Npm-check use depcheck to detect unused dependencies.

@lijunle so are you not checking for packages in devDependencies which is mention in package.json

@js08  I don't get your meaning, is that related to webpack issue? If not, please open another issue.

@lijunle if npm-check packages supports webpack we can use the same code in depcheck-es6

@lijunle can you tell what exactly is the issue its confusing

@lijunle https://github.com/lijunle/depcheck-es6/issues/27 if you look at this it seems you have closed

@lijunle here is an example for todomvc app, can you fix it https://github.com/rackt/redux/blob/master/examples/todomvc/webpack.config.js#L22

Hi, @js08
I am sorry to tell you that, I am not going to work on this issue. Because of the estimated complexity of this issue is high, and the benefit is not very high. I will not work on this issue until my another working project reach my bar.
However, if you have ideas on this, please go ahead to implement it. I am welcome for a PR.

This issue was moved to depcheck/depcheck#42


Issue #158 JSR-107 doc update on cache config overriding using templates

Good to go - needs  rebase though


auto batch size

suggest/enforce batch size of:
let n = series length;
return n.sqrt %>% next_power_of_two

Batch size is taken in as a parameter and rounded up silently to a power of two. This may cause the program to panic! if the batch_size is greater than the time-series length after rounding.


[BEAM-13716] Clear before creating a new virtual environment in setupVirtualenv

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @chamikaramj

LGTM. Thanks.
Should we (separately) re-factor cross-language test suites so that they behave similar to other Jenkins test suites that execute for multiple Python versions ?

Run XVR_Dataflow PostCommit

Run XVR_Flink PostCommit

Run Java PreCommit

Run Java PreCommit

LGTM. Thanks.
Should we (separately) re-factor cross-language test suites so that they behave similar to other Jenkins test suites that execute for multiple Python versions ?

I don't think it's necessary. Property-based version specification also has its own benefits; easier to add tests for new Python versions (remove tests for old Python versions).


Markdown links break when link label includes stylization

@rsonger commented on Fri Mar 19 2021

Environment data

VS Code version: 1.54.3
Jupyter Extension version (available under the Extensions sidebar): 2021.3.619093157
Python Extension version (available under the Extensions sidebar): 2021.3.658691958
OS (Windows | Mac | Linux distro) and version: MacOS 11.2.2
Python and/or Anaconda version: 3.7.10
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: N/A

Expected behaviour
When adding stylization to a link label within a markdown cell (for example, [_label_](https://www.github.com)), rendering the cell should also render the stylization of the text while maintaining full link functionality. This type of behavior is observed in common markdown editors, markdown preview extensions for VS Code (such as Markdown Preview Enhanced), even the markdown editor here on GitHub.
Actual behaviour
When the entire link label text within the square brackets of the label is stylized (emphasis or strong emphasis in particular), then the link still renders as clickable but clicking on the rendered link does not respond in any way.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Open a Jupyter Notebook in VS Code and create a new markdown cell.
Inside the cell, put the following code:

[test1](https://www.github.com)
[__test2__](https://www.github.com)
__[test3](https://www.github.com)__
[__this__ is __test4__](https://www.github.com)

Run the cell and click on "test2". Note that it does not open the link whereas "test1, "test3" and "this is test4" all work without issue.

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

> ~/anaconda3/envs/datascienv/bin/python ~/.vscode/extensions/ms-toolsai.jupyter-2021.3.619093157/pythonFiles/pyvsc-run-isolated.py -c "import ipykernel"
Error 2021-03-19 17:57:40: Jupyter Kernel Spec not found for a local connection
> ~/anaconda3/envs/datascienv/bin/python ~/.vscode/extensions/ms-toolsai.jupyter-2021.3.619093157/pythonFiles/pyvsc-run-isolated.py pip list
> ~/anaconda3/envs/datascienv/bin/python ~/.vscode/extensions/ms-toolsai.jupyter-2021.3.619093157/pythonFiles/pyvsc-run-isolated.py -c "import ipykernel"
> ~/anaconda3/envs/datascienv/bin/python ~/.vscode/extensions/ms-toolsai.jupyter-2021.3.619093157/pythonFiles/pyvsc-run-isolated.py vscode_datascience_helpers.daemon --daemon-module=vscode_datascience_helpers.kernel_launcher_daemon -v
Started kernel Python 3.7.10 64-bit ('datascienv': conda)
> ~/anaconda3/envs/datascienv/bin/python ~/.vscode/extensions/ms-toolsai.jupyter-2021.3.619093157/pythonFiles/pyvsc-run-isolated.py -c "import ipykernel"
> ~/anaconda3/envs/datascienv/bin/python ~/.vscode/extensions/ms-toolsai.jupyter-2021.3.619093157/pythonFiles/pyvsc-run-isolated.py vscode_datascience_helpers.daemon --daemon-module=vscode_datascience_helpers.kernel_launcher_daemon -v

@claudiaregio commented on Sat Mar 20 2021
Thanks @rsonger I am able to reproduce this behavior at this time

@greazer thanks for forwarding the issue.
@mjbvz I didn't reproduce this issue with the custom markdown renderer. I wonder if we want to support this in the builtin markdown as well. It looks like a bug to me.

With VS Code insiders, I can't reproduce this with the built-in markdown renderer using GitHub issues and Jupyter. @rebornix Were you able to reproduce the issue?

I believe this is an issue with custom editor based Jupyter notebooks. It should be filed against the python/jupyter extension instead

I'm going to close this as it does not seem like a VS Code issue

Yes, this bug occurs specifically in the Jupyter notebook that comes with Microsoft's Python extension. That's why I originally filed it under the vscode-jupyter extension mentioned by @greazer above.


LevelInfo fog shenanigans

Reports that the LevelInfo fog was broken by this. Re-evaluate the purpose for the change and whether it should be reverted.

It's "working as intended" as per my response on #1043
Less a 'not working properly' and more of a 'works in a confusing way if using scattersky'. Going to close this one and we can discuss a better way to approach fog handling in the other ticket.


[Bug] Cannot remove custom server

I tried adding server from the listed booru sites i found on github. After some testing, I noticed when trying to edit the server name another entry appeared (with the edited name being used while the old one is still intact). I dont know if this is true to all booru url. The booru url is tab.booru.org.
Restarting the app and clearing the cache does not fix the behavior (still cant remove). It can be temporarily be fixed by resetting the list to default though.

Update: Seems to be an isolated case. I tried readding the server and reproduced the bug as far as i can remember but i cant reproduce it anymore. I don't know what triggered it tho.


Step 14: tesseract model GT4HistOCR_50000000.997_191951

Dear reader,
this model: GT4HistOCR_5000000 0.997_191951 for tesseract is linked on the workflow guide.
But on https://ub-backup.bib.uni-mannheim.de/~stweil/ocrd-train/data/GT4HistOCR_5000000/tessdata_best/ the best seems to be GT4HistOCR_5000000 0.575_401209.traineddata
and there is a newer directory GT4HistOCR_5000000-2 .

already mentioned in #297 BTW. But ideally, we should recommend nothing other than frak2021

Didn't know https://ocr-bw.bib.uni-mannheim.de/anwendung/druckwerke/ . Thanks!


Users required to specify WiFi credentials, even if they are not using any of the Wifi based features

At current commit of dev (80e1faaa67f3785ae2f88ffea1c531b8ac611f76), users have to specify Wifi credentials, even if they're not using the Wifi features (configurables or PID manager). The wifi credentials are not used if the wifi features are not setup, so the user can provide empty strings. However, this is a sub-optimal workflow.
Need to decide if the pid_manager should be made external to the core motorgo driver OR if it needs to be modified such that the Wifi credentials are only required IF the Wifi is being configured.

Merged into dev, marked for release 1.0


Ticked version, regenerated if needed. (Double-check reqs!)

Made using tick_my_feedstocks.py!

[x] I have vetted this recipe

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

-[ ] cryptography >=2.0 (Pull)

Superceded


启动项目的时候提示缺少文件

Module not found: Can't resolve 'history/createBrowserHistory' in '/opt/huahai/test/reactSPA-master/src/common'

make sure your react-router is 4.x+


Build a new mail server

hello
If i am creating new mail server with another domain name then do i need to add it into config.postal.yml? to use that domain

No. You add the domain on the Domains option under the server

okay, then how can i use that 2nd domain SMTP.
another question, i added another domain n send test message from postal web interface but it shows pending. what will be the reason?
Also please reply my another issue which i have created few days back


Download for feature transcriptomics table not working

The download buttons for feature transcriptomics table (whole table or selected items) do not work.

Here is an example:
https://www.beta.patricbrc.org/view/Feature/PATRIC.83332.12.NC_000962.CDS.1002917.1003792.fwd#view_tab=transcriptomics

The whole table download is working now. However, downloading selected items using action bar still doesn't work.
-Maulik

Here os an example page
http://www.patric.local:3000/view/Feature/PATRIC.83332.12.NC_000962.CDS.1002917.1003792.fwd#view_tab=transcriptomics

I believe this is failing because there is no unique key defined in the transcriptomics_gene core in solr.  The streaming download requires that queries be sorted on a unique key.  Currently the query is trying to use 'pid' as the unique key, but I dont see it defined as unique in the solr schema.

@maovt  Harry will need to add a unique key into this table before the download can work, but he indicated that 'pid' isn't correct.  That was unique per sample, but not for the gene.  I think it is probably 'id' that this needs to be sorted by for downloads.

we actually had the id field with values, so please use this. I reloaded production solr. so you should be able to see it now.

I changed the pk to use 'id' instead of 'pid'.  With Harry's change, this works for both the table download and the selection download.

fixed. verifiedon beta.


Support Silent Arming

By default when I arm stay, I wish for that to be done "silently". This is so that the alarm does not do the beeping countdown. This is an option on the alarm.com mobile app.
In the web there is a checkbox "Silent Arming" that can be set. Any idea if this can be done from the plugin?

@omarshahine I apologize it took so long for me to reply. I somehow missed this. It should be doable—most likely as an option in the config file. I haven't had much time to dedicate to this project lately. Do you want to submit a pull request?

@omarshahine Actually, I take that back. The mobile site, which we've switched to a while back, does not support the silent arming or no entry delay options. You're welcome to fork this and go back to the "full" site to do so.

Closing since we no longer use the standard version of the site.


Fix fetch-conferences workflow issue

Fetch conferences workflow fails to fetch conferences from the google sheets, thus the conferences are not updates.

@Isminoula
https://docs.google.com/spreadsheets/d/1Z-uP86zVn0huoLXRPo9SI3klV11BQgfk/edit#gid=1146930065
This sheet is private, cannot fetch it.

@juvekaradheesh adding @immuntasir  to fix file sharing


Support for React Router Routes

How would I be able to support react router routes? When I visit /route, I'm getting a 404.
For production react router routes work as expected with https://github.com/fastify/fastify-static

Hi!
Thanks to checking out this repo!
I never tried to setup with react router, usually I use this with pure js. Can you give some code snippet (or even better an example repo) how you tried to set it up?

Here's a basic example of react router. When you click a link the url will change. This works as expected with fastify-webpack-hmr. When you refresh or go to the link directly fastify returns a 404.
I believe I've used https://github.com/webpack-contrib/webpack-hot-middleware ,express, and react router in the past.

I am sorry, but I still can't figure out what you try to achieve here. First of all your example is an express app and don't see how fastify comes here. Secondly I don't see any route declaration, so it is not a surprise you get 404.
Do you want to render a page on server side and kickoff the react router (which works on client side) when a page loaded? If that is the case, take a look at next.js which does exactly the same but with his own router. And you can check out the fastify-nextjs plugin.

I ran into this issue while porting one of my applications from Express to Fastify.
@lependu To add some additional context, the crux of the issue is that Single Page Applications (SPA) use browser-side routing to dynamically display content without fetching each individual page from the server.
For Express, I have used connect-history-api-fallback.
Additionally, webpack-dev-server has built-in support for this pattern (which internally leverages connect-history-api-fallback).
Unfortunately, the connect-history-api-fallback middleware does not work with Fastify, as it relies on modifying request.url which does not exist in Fastify.
I propose adding a configuration option historyApiFallback, similar to what webpack-dev-server offers, which would trigger logic analogous to connect-history-api-fallback (but specific to Fastify).

Just to clarify: the logic that I mentioned above could (and should) be implemented in a separate plugin, and then leveraged within this package.

I googled around a bit more, and I stumbled upon this thread on webpack-dev-middleware, which led me to the following solution to this issue:
const path = require('path');
// ...
fastify.use(webpackDevMiddleware);
fastify.use(webpackHotMiddleware);
// ...
fastify.get(`/*`, function(request, response) {
    const indexPath = path.join(webpackConfig.output.path, 'index.html');
    const index = webpackDevMiddleware.fileSystem.readFileSync(indexPath);

    response.type('text/html').send(index);
});

Interestingly, documentation was added for this approach in webpack-dev-middleware !44, but was subsequently removed (I have yet to track down why).

Hi! Thank you to track this down, I think, you summarized the problem precisely.
Adding client side routing support is a very complex feature, and that is the reason why I am not intend to bring that into this plugin.
On top of that, client side routing support is implemented in packages like next.js and nuxt.js, and they provide support to use them with custom servers like Fastify. That is the other reason why I rather use those packages, than implement the feature here.
If you determined to implement the client side routing support, I think your evaulation is spot on. The first step would be to create a fastify-connect-history-api-fallback plugin which uses the hook and decorator system provided by Fastify, instead of modifying the core request object. As soon as that module exists, we can implement a configuration option here, which feature I am happy to add this plugin.

Hi,
I had the same problem with Vue SPA with history mode. Just found the solution by using "@vottuscode/fastify-vue-static" instead of "fastify-static" works like a charm :-)


windows 8.0 pro not compatible

When will you have a compatible version for this system? Thanks

What error are you receiving? It should work for Windows 7.1 or higher.
If you're receiving an error that you can't install it into a specific folder - create a new folder on your desktop, and then install the app there.


Adding RabbitMQ relay support

@blinktag I'm going to merge this but hold off on making a release.
I'm going to open source our rabbitmq helper lib and integrate it into plumber. Once done, we can cut a release.


Tor service does not start

Issue description
After a nixos-rebuild run, tor.service ceased to start.
Steps to reproduce
I have tor in environment.systemPackages as well as services.tor.enable = true;. I'm on latest nixos-unstable. Not sure if this is enough to reproduce.
And here is what journalctl -xe has to say:
Apr 28 14:47:27 nixos systemd[1]: Starting Tor Daemon...
-- Subject: Unit tor.service has begun start-up
-- Defined-By: systemd
-- Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit tor.service has begun starting up.
Apr 28 14:47:27 nixos systemd[4366]: tor.service: Failed to set up mount namespacing: No such file or directory
Apr 28 14:47:27 nixos systemd[4366]: tor.service: Failed at step NAMESPACE spawning /nix/store/s5ipgml9pvh697k18nm9941fbij3d7kg-unit-script/bin/tor-pre-start: No such file or directory
-- Subject: Process /nix/store/s5ipgml9pvh697k18nm9941fbij3d7kg-unit-script/bin/tor-pre-start could not be executed
-- Defined-By: systemd
-- Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- The process /nix/store/s5ipgml9pvh697k18nm9941fbij3d7kg-unit-script/bin/tor-pre-start could not be executed and failed.
--
-- The error number returned by this process is 2.

Technical details
system: "x86_64-linux", multi-user?: yes, version: nix-env (Nix) 2.0, channels(root): "nixos-18.09pre135256.6c064e6b1f3", channels(mark): "", nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Duplicate of https://github.com/NixOS/nixpkgs/issues/39290.
Has been fixed in #39083.
Upgrading to latest nixos-unstable should solve the problem.


Clean up intermediate data

We need a robust way to clean up (remove) the intermediate dataset.

Here is an idea, what do you think?
Use zarr's fsspec URL parsing logic to get to the [X]FileSystem, and use the remove functionality?


[stable/nfs-server-provisioner] Some fixes

Is this a new chart
no
What this PR does / why we need it:

Includes https://github.com/helm/charts/pull/17547 for updates apiVersion for stable/nfs-server-provisioner that's makes possible to deploy it on Kubernetes 1.16
Fixes imagePullSecrets which are not working for stable/nfs-server-provisioner
Changes default value for rpcbind port from 51413 to 111 for stable/nfs-server-provisioner, that's makes possible to discover shares over NFSv3.
Configurable gracePeriod option for NFS-Ganesha

Which issue this PR fixes
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @kiall @verwilst

/ok-to-test

I've been eagerly awaiting https://github.com/helm/charts/pull/17547, and looks like this contains the same fix. Any idea if either will be ready Soon? 🙂
Or is someone likely to maintain it somewhere in the post-stable world I'm hearing about?

Since this touches both nfs-client and nfs-server, and i only use/maintain nfs-client, i'm not comfortable with changing default ports for the server, since i don't know what might break.

OK, PR rebased, now it includes fixes only for nfs-server-provisioner and does not conflicts with #17547
Changes for nfs-client-provisioner was moved to new PR https://github.com/helm/charts/pull/18442

/unassign @verwilst
/assign @kiall

/remove-lifecycle stale

/remove-lifecycle stale

Hi @kvaps please fix conflicts and we'll get this in! :) Thanks.

Also @kvaps if you'd be interested in adding yourself as an owner to this chart please feel free to.

Closing due the changes are moved in https://github.com/helm/charts/pull/19260


Vla/static buffer removal

Thank you. But since it's just local, IMO it's overkill to compute the length then alloc, you could just alloc 8192 directly (and it is freed immediately).

Thank you. But since it's just local, IMO it's overkill to compute the length then alloc, you could just alloc 8192 directly (and it is freed immediately).

Thought about it too. What about the length of additional args? Should I account for them too?

Thank you very much.
I made the following changes:

remove the reverted commits
rename cmd to argv in a separate commit (for diff clarity)
remove CMD_MAX from process.h and used separate local constants for windows implementation and error message
simplify the logic in sys/win/process.c

I pushed to branch novla. Please review the result to tell me if this is ok for you.
Regards

Seems fine, not entirely sure why len here https://github.com/Genymobile/scrcpy/blob/b24be9e2c2527b4667568551d93102151b4fc16d/app/src/adb.c#L84 differs from here https://github.com/Genymobile/scrcpy/blob/b24be9e2c2527b4667568551d93102151b4fc16d/app/src/sys/win/process.c#L9
since they're both being fed the same argv?

Seems fine, not entirely sure why len here […]

Oh, my inline reviews were not committed on github (still pending). The reason is explained here: https://github.com/Genymobile/scrcpy/pull/2405#discussion_r654840475
I guess 512 would be sufficient, but it's arbitrary (In practice, it will not make any difference.)

Merged. :+1:


Route handler is called three times

I use router.navigateTo(context, _routeName, transition: TransitionType.fadeIn);
My Handler is called three times. 2 times before the screen shows and than another time when I use the back button.
Route:
 router.define(invitePeopleScreen, handler: invitePeopleScreenHandler);
Handler:
var invitePeopleScreenHandler = new Handler(     handlerFunc: (BuildContext context, Map<String, dynamic> params) {       appController.changeActiveScreenState(Routes.invitePeopleScreen);       return new InvitePeopleScreen();     });
The documentation is pretty slim !!!
'

This seems like it is as designed so I am closing


always crash when using xcode 11.3, ios 13

halo help me , when i'm using this always crash on this line
let textField = cell.viewWithTag(100) as! SVPinField

try making it SVPinView

Hi @adika-juniarsa, could you help me with recreating this issue? I was able to run in on Xcode 11.3 without any issues or crashes.

Related to #41 and #49
Fixed in v1.0.9


vbump webR dependency

Currently it is using v0.2.2 whereas the newest is 0.3.2. The difference is quite big. For instance, I want to make use of webr::library_shim() which is unavailable in the v0.2.2

I think I have found a workaround of this issue. I added the following to the app code before any of library() calls.
if (packageVersion("webr") < "0.3.0") {
  .e <- as.environment("webr_shims")
  .e[["library"]] <- function(pkg, ...) {
    package <- as.character(substitute(pkg))
    if (length(find.package(package, quiet = TRUE)) == 0) {
      webr::install(package)
    }
    base::library(package, character.only = TRUE, ...)
  }
}

Looks like the issue was in shinylive assets. I have upgraded it to v0.5.0 (shinylive::assets_download("0.5.0")) and I got sufficient webr version. No need to implement above workaround anymore.


No module named eolearn.core in WaterMonitorWorkflow.ipynb

We get this error even though we did the pip install eo-learn

You probably need to check whether your jupyter-notebook kernel links to the correct Python environment.

I completely agree with the answer and closing the issue.


containerd: 1.0.1 -> 1.0.2

Semi-automatic update. These checks were performed:

built on NixOS
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd -h got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd --help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd -v and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd --version and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd -h and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd --help and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd help and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-release -h got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-release --help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-release help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-stress -h got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-stress --help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-stress help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr -h got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr --help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr help got 0 exit code
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr -v and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr --version and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr -h and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr --help and found version 1.0.2
ran /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/ctr help and found version 1.0.2
found 1.0.2 with grep in /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2
found 1.0.2 in filename of file in /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2

@GrahamcOfBorg build containerd

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘containerd-1.0.2’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/applications/virtualization/containerd/default.nix:49 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/5qx9jisqzyha30bb49hmh5j11nwcbbqv-containerd-1.0.2/bin/containerd-shim
cannot find section .dynamic
shrinking /nix/store/5qx9jisqzyha30bb49hmh5j11nwcbbqv-containerd-1.0.2/bin/containerd-stress
shrinking /nix/store/5qx9jisqzyha30bb49hmh5j11nwcbbqv-containerd-1.0.2/bin/ctr
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/5qx9jisqzyha30bb49hmh5j11nwcbbqv-containerd-1.0.2/bin
patching script interpreter paths in /nix/store/5qx9jisqzyha30bb49hmh5j11nwcbbqv-containerd-1.0.2
checking for references to /build in /nix/store/5qx9jisqzyha30bb49hmh5j11nwcbbqv-containerd-1.0.2...
cannot find section .dynamic
/nix/store/5qx9jisqzyha30bb49hmh5j11nwcbbqv-containerd-1.0.2

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-stress
shrinking /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd
shrinking /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin/containerd-shim
cannot find section .dynamic
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2/bin
patching script interpreter paths in /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2
checking for references to /tmp/nix-build-containerd-1.0.2.drv-0 in /nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2...
cannot find section .dynamic
/nix/store/r3k13i9mxm9bsm21d3s0fd35adf53gxr-containerd-1.0.2


Add minimal docs on async support in readme

There's been at least a couple of issues opened on NSpec misbehaving in async scenarios, only to find out that people were using the non-async API. This makes them (and us) loose time for something that could be documented even in a tiny tiny way, waiting for a proper documentation available on web site after its major overhaul.
I'd suggest to take the cue from comments given in these issues: #109 and #112 (please add more if available, those are the ones I remember) and use them to put up a small dedicated section in main README.md.
@amirrajan what do you think?

Is there any way to throw an exception when async is enabled? Meaning is it safe to assume that once you go async, any  nested tests must be async? If that's the case, the nspec instance could have a stateful flag that throws if a sync method is used?

And yes I still need to update the website T_T.

Sorry, I did not get what you wrote before. Just to share common understanding: as of now exceptions Vs async work fine, that is: exception is thrown from nspec async hooks, is collected by library and correctly reported to unit test code.
So, at the moment, I only see the need to explain that there exist an async part of the library.

If, by chance, you are referring to something like "detect when an async lambda is assigned to it or before", I think the answer is no. On top of my head, I don't remember the details now, but somehow it's like if by the time the async lambda gets assigned to a non-async hook, the library looses the ability to detect whether it was async or not in the first place.

If, by chance, you are referring to something like "detect when an async lambda is assigned to it or before"

Yep! That's what I mean!

I think the answer is no.

Way to bust my bubble :-P
Could we update the property setter here that sets a flag such that any subsequent calls to non async properties throw an exception?

I think I got what you're looking for, but I don't understand how it can help, sorry. I'll try to recap here, hopefully to have a common starting point. If in the same test class/context you use both sync and async hooks (each one correctly), the test all in all is correct as well:
// ...
  it["does something sync"] = () => doesSomething();
  itAsync["does something async"] = async () => doesSomethingAsync();
// ..

I think there are some cases where test cannot mix both sync and async hooks, don't remember if it's before or after, but those cases should be detected by NSpec at runtime and flagged as errors.
The problem NSpec users seem to encounter often is to assign a classic sync hook to run async code:
// ...
  it["does something async"] = async () => doesSomethingAsync();
// ..

This is basically similar to running an async void function, there's no way to wait and resume after it completes, so people find an unexpected behaviour. And I recall of no way to detect this from within NSpec sync hook when it receives the value.
How does that flag help in this case?

Gotcha >_<

Not happy about it, but that's it :O

Could we blindly await everything  that's passed into it (inside the internals of NSpec)? Too invasive? Is that even possible?

Interesting. Don't know if that's possible: at the moment

itAsync["..."] can receive a Func<Task>,
it["..."] can receive an Action

But the two of them have different possibilities as to what can be converted/ assigned to what:

In order to await everything they should all be converted to Func<Task>, but this seems not possible. On the other hand, if everything is converted to Action, if I remember correctly you loose the chance to await them, and there's no way to recover that (so to say).

... yep, I think you're right... grr...

This seems to work
        static bool IsAsync(Action action)
        {
            return action.Method.IsDefined(typeof(AsyncStateMachineAttribute),
                                           false);
        }

So inside of NSpec/Domain/ActionRegister.cs the Action this[string key] could throw an exception?

That's a great thing to know. Earlier I was inspecting the resulting Actions in debug, looking for some hint that could help distinguish syncAsAction from asyncAsAction, and I could not find any.
OT: I don't know how's that, but in the past I came through that question: just found my vote on a comment O.o

Working at a PR for this. BTW, we already introduced a working method checking for that (!!): https://github.com/mattflo/NSpec/blob/master/NSpec/Domain/Extensions/DomainExtensions.cs#L85

@amirrajan please have a look at PR #115 , if it's ok for you as well we can merge that.

@amirrajan this other PR is for the mentioned documentation on async.


Skeleton Unity Project

Just base project folders with the proper hierarchy. Also contains a
gitignore files to make it not have lots of meta data.

Sounds good to me.


IdentityAgent are not quoted where there are spaces

#37 aimed to add quotes to most of the fields, but IdentityAgent is not one of them.
It would be better to support IdentityAgent as well.
IdentityAgent usually has spaces in the value when we use thrid-party ssh agents on MacOS, where many system directory path contains space, e.g., ~/Library/Group Containers for 1password ssh agent.

@Troublor Thanks for reporting! fixed and released with v4.1.3


Add feedback feature #161

I was not able to fully test the l10n/i18n on my development machine (for some reason LOCALE_PATHS is not working). I could use some help testing that (I did add the corresponding new keys in the translation files).
Other than that, the feature is fully functional. By default, the e-mails are being sent to<EMAIL_ADDRESS>properly configured to instruct your e-mail client to reply automatically to the logged user e-mail address. See the test cases for more info.

Can you add a print?

About the email, its okay to use that one, the only concern is because he doesnt exist, o didnt setup a smtp server yet, but in the server i can use the local_settings.py and setup my own email.
o/

Yep! Here they are:

Nice, as soon as you finish the pending changes that i ask on the Review, i can merge and deploy you contribution.
o/

Despite the changes I asked, you did a very good job here. :)


[Bug]: The route forgot-password could not be found

What happened?
Since upgrading to the version 2.0.0 and following the upgrade notes of this package, the forgotten password link no longer works on the login page (screenshot below of the link I'm referring to).

I have uninstalled the package and then the forgotten password link works perfectly again.
I have even installed the plugin again, republishing all the files but the same error occurs.
How to reproduce the bug
When clicking on the 'Forgot password?' link on the login page it simply returns a 404 error.
Repository
filament-2fa
Package Version
2.0.0
PHP Version
8.2
Laravel Version
11.32
Which operating systems does with happen with?
Windows
Notes
No response

@nathanmay I found the cause of this issue, I'm working on a fix

@Baspa @nathanmay any update?

@Baspa @nathanmay I found a solution, it worked for me, let me know if it works for you.
Update config/filament-2fa.php:

'login' => \Filament\Pages\Auth\Login:class,
'register' => \Filament\Pages\Auth\Register::class,
'password_reset' => \Filament\Pages\Auth\PasswordReset\ResetPassword::class,
'request_password_reset' => Filament\Pages\Auth\PasswordReset\RequestPasswordReset::class,

Those are default Filament classes for login, register and password reset. But if you use custom classes for login, register and password reset on you Filament panel, you should use those classes instead of the Filament default ones.


Docs: Building a Module Tutorial

What is the issue?
Per discussion with Jeremy And Solomon on Discord, dropping a request to have the recent (or anything similar) workshop documented in the docs as a tutorial.
For a bit of context, I had been reading up on the prior method of developing Dagger (dagger call) and was a bit confused on where the prior method fit in with the module method. After watching the workshop it was made very clear how the module method works, why it's the right direction, and more importantly, how quickly get started with it in my project. Having this as a quick tutorial in the docs for folks to walk through would go a long ways.
Hashicorp does a great job with their tutorials, take Terraform as an example.

@scottames would love your feedback on https://github.com/dagger/dagger/pull/7326 which should address this request.

@scottames would love your feedback on #7326 which should address this request.

@vikram-dagger - Nice job with these! Those are great tutorials, I think will help a lot. I see in one of them you link back to examples in the daggerverse - having folks go through the tutorials and look at examples in the daggerverse will go a long ways.


TS-59 BUG - Duplicate TeamSheet has incorrect redirect

src/app/shared/components/ShareBar/index.tsx:31

fixed in TS-58


edits

There was an error while validating job 80119803-1496-477b-8659-509bee5bbcd2 Message:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
Details:Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase+ContextCommandException:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
---> System.AggregateException: One or more errors occurred. ---> System.ArgumentException: Error clonning repository ---> LibGit2Sharp.NotFoundException: Branch 349 not found. Available branches: origin/master
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<>c__DisplayClass34.b__29(KeyValuePair2 branch)    at System.Collections.Generic.List1.ForEach(Action1 action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<CloneRepository>b__28()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.<ExecuteAction>b__0()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetVersionAsync(String path)
at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)    --- End of inner exception stack trace ---    --- End of inner exception stack trace ---    at Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase.ThrowIfErrorsLogged()    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)


Add Node.js 6 to CI

Don’t want to scary you, but current Babel need Node.js 6 support.
Browserslist need very old Node.js as well because we can’t do major releases often to clean old Node.js versions.

As a part of #8 I added Node.js v6 to the testing matrix. v6-10 only running CJS tests as we don't expect them to be using anything else.

<EMAIL_ADDRESS>is now published and supports Node.js v6
https://github.com/alexeyraspopov/picocolors/releases/tag/v0.1.0


Drop dependency on deprecated gulp-util

Closes dlmanning/gulp-sass#646

This PR also update the versions tested with CI to match the ones used by Sass itself.
The main difference is that 0.10.0 is no longer tested because Node stopped maintaining it, and most dependencies started to require Node 4+.

Thanks. Dropped support for old node versions will require a major version bump. I wasn't planning on bumping the major until node-sass@5 landed. Is it possible to make this work with Node 0.10?

Hi,
Thanks for the info: I did not know that Sass still wants to support these versions. I'll try to find some older versions of the dependencies that support Node 0.10.0. I opened an issue to discuss this change.
Regarding the lockfiles, you are better off by keeping them, especially if you plan to support unmaintained Node versions. The lock files are intended to be committed and have no effect on consumers: they are used for library developers and CI to make sure that you develop with the same working dependencies.

Hi, I am having trouble to make node-sass work with iojs (Node 3). I pinned node-sass to exactly 4.2.0 but its install step has a dependency on request which itself depends on hoek which requires Node 4.

Correct. The latest release 4.7.2 pinned to a version on request that still
supported old node. Unfortunately that release has issues with iojs v3 so
I'm ok with not testing iojs in ci.
On 29 Dec. 2017 12:52 am, "Charles Samborski"<EMAIL_ADDRESS>wrote:

Hi, I am having trouble to make node-sass work with iojs (Node 3). I
pinned node-sass to exactly 4.2.0 but its install step has a dependency
on request which itself depends on hoek which requires Node 4.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dlmanning/gulp-sass/pull/647#issuecomment-354291590,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWDYXqzAnfJmwiQpDM3CZb0-pneesks5tE50EgaJpZM4RN4zD
.

@xzyfer
I tried my best to support it but at the moment, the best would be to merge #649 first, it's just going to cause more issues.
You are effectively asking me to drop support for iojs, so it's better to do it properly in its own PR and then come back to this issue.
Even if this Node issue is not related to gulp-util, it should be fixed first. Please also note that dropping gulp-util is important because it is causing build issues with the latest release of Gulp 4. You are not the first person I am contacting to patch this and multiple other packages have similar issues were old versions of Node are wrecking havoc the builds because they weren't touched in a year but most dependencies have moved on. We reached a point were not keeping dependencies up-to-date is causing builds to break so I'd really like to have it sorted as fast as possible. Thanks for your support :)

I was wrong about lock files, I'll remove them.
Still, please consider migrating to newer Node versions and dropping gulp-util.

hi, any update about this PR?

No. Any updates would appear on this issue.
On 7 Feb. 2018 12:42 am, "Roberto Robson"<EMAIL_ADDRESS>wrote:

hi, any update about this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dlmanning/gulp-sass/pull/647#issuecomment-363425395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWHh3pxlNOQjYWGBNuQD91ELTAXCdks5tSFaxgaJpZM4RN4zD
.

 Just  bump to next major version, like it does ALL OTHERS  MOST POPULAR PACKAGES.
Sorry for ignore basic contribution rules but this is awesome.

@xzyfer will it be fixed after 5 years (now) or we just wait node 4 EOL?

@xzyfer @dlmanning no luck on merging this one? :(

Looks like there are a couple separate breaking changes that'll land soonish.

any news if this will get merged or if a new PR to fix the merge conflicts is necessary, @demurgos @xzyfer @dlmanning ?

There is linked pr that includes these changes and more and a checklist for
when it merged.
On Mon., 26 Mar. 2018, 7:09 pm Veeck<EMAIL_ADDRESS>wrote:

any news if this will get merged or if a new PR to fix the merge conflicts
is necessary, @demurgos https://github.com/demurgos @xzyfer
https://github.com/xzyfer @dlmanning https://github.com/dlmanning ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dlmanning/gulp-sass/pull/647#issuecomment-376081104,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWKHEuBVQptqdANYb5bt8eGYjTtkQks5tiKIggaJpZM4RN4zD
.

Thanks heaps, this has been included in #667


FAQ > How is Threefold registered as an entity? > Links redirect to old manual

URL - https://manual.grid.tf/faq/faq.html#how-is-threefold-registered-as-an-entity
All the read more hyperlinks on this FAQ redirects to old manual, we may have to create the pages for these links and then make the changes :
https://library.threefold.me/info/legal/#/orgstructure/threefold__threefold_vzw
https://library.threefold.me/info/legal/#/threefold__threefold_dubai

Will be fixed when https://github.com/threefoldtech/info_grid/pull/321 is merged

Fixed in #321


Reduce size of search text

I can work on it can you assign it to me?

/bounty $10

@seveibar is the issue for search text or do you want the search results to be truncated or just the size to be reduced?

@seveibar do you want the size of the search results to be minimized or should i truncate it?

Reduce size of search text

or is it just search text (inside the input)?


Backup command is not working

1:MS CRM Backup online instance is not working..
Giving error at "Instance Name"
{already reffered to the format https://[instancename].crm[n].dynamics.com}
used the "instance name"
"Instance not found errror"
2: Used powershell cmdlets for backups as well but requirement is until bakup not finished next task wont be executed but it is working asynchronously script get executed when backup still not created
Any suggesstions

@saksingh
so if the url of your instance is https://instance1.crm11.dynamics.com
Then the instance name should be: instance1
The task has a wait in it until the backup is initiated, normally the backup will be based on the time you start the backup, so you should be able to carry on with next tasks. If you are ensure I would suggest you try the rollback and test to ensure you are getting the expected behaviour.


Upgrade ZIO to 1.0.0-RC19

Fixes #548
Upgraded ZIO and all its dependencies.
I had to disable Dotty for ZIO because it uses 0.23 but sttp uses 0.22. It can be enabled again once sttp upgrades (needs sttp-model first).

Released in 2.1.2. Thanks!


Added Swashbuckler Class

Added Swashbuckler Class

Thanks for taking a look at the code.  I wasn't 100% satisfied with the way I did that.  How would the method you are suggesting work?  What would you pass in to get back the objhandle when it is time to do a comparison?

I was thinking .get_obj_from_args(n) to get a handle from args n and n+1.

Going to go ahead and merge this one.  It's been a while now and I haven't heard any complaints.


The .en.json translation template is wrong

It seems this field is missing in .en.json:

@fnoks Interesting, because that whole overall feedback block can usually be copied and pasted in semantics.json and then h5p cli would do the rest. Will fix it when I am back home.

@fnoks That field doesn't contain anything that H5P CLI deems worthy to be translated, that's why it's not in .en.json (as in all the other content types that "Overall feedback" section).

Oh. Strange. Will check if it works :)


Aca-py startup arg updates

Make "--endpoint" required.  Make genesis required (url, file or txns) unless "--no-ledger" is specified.  Make some "indy" wallet parameters required (name, key, etc) if indy wallet type is specified.
Added "--arg-file " to allow params to be included in a file.  (File is just a list of args, not structured with yml etc)  Had to change some related logic - couldn't flag args as "required" so moved the check to the "get_settings()" method and threw "ArgsParseError"; display usage on any ArgsParseError.  (Had to fix a couple of unit tests that expected the "old" behaviour.
Related to issue #521

Codecov Report

Merging #739 into master will decrease coverage by 0.02%.
The diff coverage is 72.09%.

@@            Coverage Diff             @@
##           master     #739      +/-   ##
==========================================
- Coverage   99.01%   98.99%   -0.03%
==========================================
  Files         256      256
  Lines       13956    13975      +19
==========================================
+ Hits        13819    13835      +16
- Misses        137      140       +3

Hmmm... -0.03% takes us below 99%.  That's sad...

Note that configargparse is pointing to my fork (in requirements.txt) pending them accepting my PR

Some discussion around environment variables - the "ConfigArgParse" library adds support for env vars, but they must be configured per parameter, like for example:
    parser.add_argument("--wallet-storage-config", env_var="ACAPY_WALLET_STORAGE_CONFIG", help='....')

If the environment variable is present the value gets used, regardless of whether there is a config file or not.
However you cannot substitute env vars in the config file, like the following:
    input-transport = [["http", "${HOST}", "${PORT}"]]

The list of available environment variables (from the parser.add_argument() functions) is included in the usage() ("aca-py -h"), like:
--wallet-name
Specifies the wallet name to be used by the agent.
This is useful if your deployment has multiple
wallets. [env var: ACAPY_WALLET_REKEY]

Some discussion around environment variables - the "ConfigArgParse" library adds support for env vars, but they must be configured per parameter, like for example:
    parser.add_argument("--wallet-storage-config", env_var="ACAPY_WALLET_STORAGE_CONFIG", help='....')

If the environment variable is present the value gets used, regardless of whether there is a config file or not.
However you cannot substitute env vars in the config file, like the following:
    input-transport = [["http", "${HOST}", "${PORT}"]]

The list of available environment variables (from the parser.add_argument() functions) is included in the usage() ("aca-py -h"), like:
--wallet-rekey
Specifies .... [env var: ACAPY_WALLET_REKEY]

IMHO - this is even better.  We get DRY documentation, we tightly tie the environment variables to the arguments.  It does change how we have to environment variables, but the change looks like it moves us in the right direction.  Ideal to use this always.
Next challenge -- being able to use argparse as defined here in bash scripts :-)  -- that would be awesome...

It's not perfect - it would be nice to have the param substitution within the config file as well.
For example multi-value parameters (like "input-transport", which is also a multi-instance parameter = multi instances of multi values) would be confusing to jam into a single environment variable.

Except there is a very clear rule on how to do it, with usage information that makes it precise. What we are doing now is mixing and matching approaches that vary from variable to variable.  Neither is perfect, but I would lean to consistency.
But my vote doesn't count much (and rightly so :-) ) in these decisions.

Env vars configured for:
--admin-api-key:  ACAPY_ADMIN_API_KEY
--webhook-url:  ACAPY_WEBHOOK_URL
--seed:  ACAPY_WALLET_SEED
--wallet-key:  ACAPY_WALLET_KEY
--wallet-rekey:  ACAPY_WALLET_REKEY
--wallet-storage-config:  ACAPY_WALLET_STORAGE_CONFIG
--wallet-storage-creds:  ACAPY_WALLET_STORAGE_CREDS

--admin-api-key: ACAPY_ADMIN_API_KEY
--webhook-url: ACAPY_WEBHOOK_URL
--seed: ACAPY_WALLET_SEED
--wallet-key: ACAPY_WALLET_KEY
--wallet-rekey: ACAPY_WALLET_REKEY
--wallet-storage-config: ACAPY_WALLET_STORAGE_CONFIG
--wallet-storage-creds: ACAPY_WALLET_STORAGE_CREDS

Additional environment variables that we will likely need:
--label
--tails-server-base-url
--wallet-type
--wallet-storage-type
--wallet-storage-creds
--wallet-storage-config
--endpoint
If it was possible to use an environment variable to drive whether the agent starts in insecure mode or uses an api-key it would be good, but maybe out of scope now (this could also be tied to whether the admin interface is activated or not - I can't think of a use for an agent that has no admin interface).
In general I would think having environment variables for all - or at least as many as possible - of the settings would be a good idea, both for consistency and especially when working with cloud deployments of any kind, where they are the easiest way to deal with configurations.

--admin-api-key: ACAPY_ADMIN_API_KEY
--webhook-url: ACAPY_WEBHOOK_URL
--seed: ACAPY_WALLET_SEED
--wallet-key: ACAPY_WALLET_KEY
--wallet-rekey: ACAPY_WALLET_REKEY
--wallet-storage-config: ACAPY_WALLET_STORAGE_CONFIG
--wallet-storage-creds: ACAPY_WALLET_STORAGE_CREDS

Additional settings that we will likely need an environment variables for:
--label
--tails-server-base-url
--wallet-type
--wallet-storage-type
--wallet-storage-creds
--wallet-storage-config
--endpoint
If it was possible to use an environment variable to drive whether the agent starts in insecure mode or uses an api-key it would be good, but maybe out of scope now (this could also be tied to whether the admin interface is activated or not - I can't think of a use for an agent that has no admin interface).
In general I would think having environment variables for all - or at least as many as possible - of the settings would be a good idea, both for consistency and especially when working with cloud deployments of any kind, where they are the easiest way to deal with configurations. At the same time, I realize that some of the environment variables (see the api-key/admin interface example above) would potentially force us to build additional logic to check whether other environment variables are set based on their value. This is a bigger job than we maybe willing to do now, but will definitely make the agents more user-friendly and resilient.

Added

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@015928f). Click here to learn what that means.
The diff coverage is 72.09%.

@@            Coverage Diff            @@
##             master     #739   +/-   ##
=========================================
  Coverage          ?   98.99%
=========================================
  Files             ?      256
  Lines             ?    13975
  Branches          ?        0
=========================================
  Hits              ?    13835
  Misses            ?      140
  Partials          ?        0


[Suggestion] Remove animated textures toggle from Game Driver settings

Since animated textures are dependent to mods file (e.g. SYW Field Textures), the toml flag should be changed by the IRO and not from here

Satsuki has confirmed he has it in mod.xml, so this should be good to go.


DSC Resource for Windows Terminal

Description of the new feature/enhancement
I would like a DSC Resource so I can specify the way I want Windows Terminal configured. WinGet Configuration or the Dev Home experience could leverage a configuration file to set Windows Terminal up the way I want it.
Proposed technical implementation details (optional)
Some notes from discussion: Paths in the json file will be Hard. If someone's got a relative path in their settings.json, well, that just won't work on another machine.
The ideal case here is probably a dotfiles-like experience where we can DSC that folder of fonts, images, and settings.json.
Crawl

We can install the Terminal.

Walk

We can make some customizations to the settings.json. Maybe use a well-known URL to apply your settings.

Run

We can also install fragment extensions the user already has.
We can install distros, PWSH's etc.

Sprint

We can export the current settings.json to a dotfiles-like folder, with images, fonts, etc, and re-write the settings.json to use that path

As discussed at Build last week:

actions, schemes, themes: These are all pretty much machine-independent. (there may be minimal actions that reference profiles, but the 99% case should work fine).
fonts out of profiles could be enumerated and included in the resource
fragment extensions could also be enumerated and included in the list of dependencies

Similarly with PWSHs, WSLs, VS, and other dynamic profiles that we can enumerate

profiles are very much machine-dependent.

As a crawl, we could just include them with no modifications
walk: include them, but with some property that indicates they've been migrated / restored somehow, and the user should manually fix them?
run: We could be smarter about "migrated" profiles, and default to some sensible CWD if we can't find the startingDirectory for that profile
sprint: When the user opens a migrated profile the first time, pop a dialog to confirm the choice of what the commandline & starting directory should be.

newTabMenu is vaguely machine dependent, in the sense that it relies on the existence of profiles. But I think we're smart enough to leave entries that don't point at an existing profile out of the new tab menu. We may be able to migrate it wholesale with minimal concern for if the profiles in it migrated successfully.


Framesize fix

In Apache Flink the results of the collect() call were returned through akka to the client. This led to an inherent limitation to the size of the output of a job, as this could not exceed the akka.framesize size. In other case, akka would drop the message.
To alleviate this, without dropping the benefits brought by akka and its out-of-the-box efficiency for small-sized results, we decided to keep forwarding the non-oversized (i.e. smaller than the akka.framesize) results through akka, and use the BlobCache module for the forwarding the oversized (large) ones.
Now the JobManager receives end merges the small accumulators (as before), and simply forwards to the Client the keys to the blobs storing the oversized ones. Now it is the responsibility of the Client to do the final merging between oversized and non-oversized accumulators.

FLINK-2319
This pull request targets this ticket.

Hello guys,
This is a new pull request, for a previous ticket.
It is aligned with recent changes in the master branch.

All in all, nice work @kl0u. Do you think we could change this pull request such that we can handle "small/large" or "message/blobcached" accumulators more transparently? It would be great if we encapsulated the two types within one message and only have one method getUserAccumulators to retrieve both types. The internal logic could be handled by either the AccumulatorRegistry or the AccumulatorSnapshot class. That would make the code easier to understand and remove some code from the JobManager and ExecutionGraph classes.

Hi @mxm ,
Thanks a lot for the comments!
I integrated most of them. Please have a look and let me know what you think.
For the merging of the the different types of snapshots and handling them uniformly I do not have any current solution. If you have any, I am open, of course, to discuss it, because I agree that this would be nice.
For the comment on the getAccumulatorResultsStringified():

this is to be presented by the web interface to the user, just for monitoring purposes
this is called at the jobManager.

The problem is that the jobManager has only the blobKeys that point to the stored accumulators. The serialized data reside in the blobCache and have to be fetched in order to be inspected.
Currently the jobManager just forwards the blobKeys to the client, which fetches the blobs and does the deserialization and the final merging. This is done for jobManager scalability reasons, as given that we are talking about accumulators of arbitrary size, loading them from disk and deserializing them would be time and resource consuming. The same holds in the case that we wanted to get the type of these large accumulators (it is needed by the method). We would have to load and deserialize them at the jobManager. The currently implemented solution is just the result of this design decision. If you have any other strategy or solution that is worth implementing, let me know.

Hi @kl0u ,
Thanks for addressing my comments. I've made some new ones :) but looks much better.
As for merging the different types of snapshots, the only solution I see now, is to let all the accumulator results go through the BlobCache. That might cause some extra latency at the end of the job. I'm not sure whether we should proceed on that path but the current solution has a lot of added overhead because we need to deal with two different kind of update mechanisms.
I would like to hear some other opinions on whether we should have both update mechanisms (BlobCache and Akka message-based) or just let everything go through the BlobCache. I'm leaning more towards the latter if it is not too slow for slow accumulators.
I think you have to rebase to the latest master again because of some minor changes.

Hi @mxm ,
I totally agree that in terms of code clarity and structure, it would be better to let all accumulators pass through the BlobCache. In fact, we also had a brief discussion with @StephanEwen on the matter.
The only problem is the latency penalty that MAY result from this design choice. In other words, if for most of the usages akka is enough, then forcing small blobs to pass through the blobCache (i.e. reading and writing accumulators to disk) may be expensive.
What do you think?

Agree. Especially with the frequently live updating accumulators, the small accumulators should stay in-frame

Just rebased with the new version of the master.
Please have a look.

I will have a look in a bit.
Am currently stuck with some streaming runtime fixes and the Kafka integration. Overflowing a bit with issues right now.
Thanks for bearing with us!

No problem!
This message was just a reminder.
Thanks a lot!

I see the point in keeping the Akka message-based transport of accumulators. @kl0u Sorry, I was also on vacations. Could you rebase to the current master again? Then, I think we can finally merge your changes.

Hello!
I just rebased. Please have a look.

Hi @kl0u. I think something went wrong during rebasing.

Hi @mxm. What do you mean?

From the GitHub UI it looks like your pull request branch and the Flink master branch have diverged, i.e. your changes are no longer based on a commit from the master branch.

When I did the latest rebase it was saying that the two branches were ready to be merged.
Is there a way to see where they have diverged?

git merge-base master framesize_fix should show you the commit the branches diverged. Depending on the outcome, you might have to go back to an earlier version of your changes and rebase those against the master. It might be easier to squash all your commits before rebasing.

Thanks @mxm.
Although I don't think I will have time to fix it right now.

@kl0u That's fine. We'll be preparing a new milestone release based on the master. I think it makes sense to merge your changes afterwards for the 0.10 release.

@mxm Sounds good.

@kl0u ,hi kl0u.can you tell me how increase akka.framesize? I don't know how config akka.framesize.Thanks!

@thorntree https://ci.apache.org/projects/flink/flink-docs-stable/ops/config.html#akka-framesize

@TisonKun I have set akka.framesize: "104857600b" in flink-conf.yaml,but it still doesn't work.

@thorntree what do you mean "it still doesn't work"? Could you provide more context such as log or output that you think so?

Besides, I think you should configure as
akka.framesize: 104857600b

without the quote.


CORE-10426 synch with release

Let's hold off merging until rudiments branch works correctly again (CORE-10424 is fixed).


get FullScreenshot

As for now, the function might work as checking the full clientHeight,
parsing it as an integer, setting the viewport (might be a good idea to
lose hardcoded "1920" in favour of clientWidth or a flag making in HD,
don't know yet) and then getting the screenshot and saving it.
Few issues with this solution:

Maybe there's a better way of evaluating the JavaScript
Maybe there's no need to evaluate it whatsoever
What happens with the viewport after the screenshot has been made?
Right now it will be morphed to a full height of the page.

Will keep working on it as I have to explore the library a bit more.
Referencing #32 .

This is the sample unit test you might want to refer to:
https://github.com/ysmood/rod/blob/f9774865a0ddedd0914e2cacd7b83678ae398e3d/page_test.go#L315

Codecov Report

Merging #33 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
- Coverage   95.80%   95.76%   -0.04%
==========================================
  Files          21       21
  Lines        5217     5217
==========================================
- Hits         4998     4996       -2
- Misses        133      134       +1
- Partials       86       87       +1

Impacted Files
Coverage Δ

lib/launcher/launcher.go
81.42% <0.00%> (-0.96%)
:arrow_down:

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 9a5610a...5d4f99b. Read the comment docs.

Codecov Report

Merging #33 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
- Coverage   95.80%   95.76%   -0.04%
==========================================
  Files          21       21
  Lines        5217     5217
==========================================
- Hits         4998     4996       -2
- Misses        133      134       +1
- Partials       86       87       +1

Impacted Files
Coverage Δ

lib/launcher/launcher.go
81.42% <0.00%> (-0.96%)
:arrow_down:

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 9a5610a...446b46b. Read the comment docs.

I can see that the test are failing - that might be because of the <!DOCTYPE html> tag in scroll.html


[CSL-288] Make sure we don't confuse k blocks and k slots

@gromak

@georgeee
Check all places in code when we go deep k +- 1 slots in chain, make sure it's not blocks

@georgeee
To be done after DB layer established

@gromak
It will be handled by CSL-534.


spellflag_targ_dead does not seem to have effect

Checked with default script pack, while spellflag_targ_dead is set on spell resurrection, the spell still is allowed to be cast on players who are alive (the spell looks like successfull, reags consumed etc).

@drk84 verified 👍


Add history events pagination API

This API (ListWorkflowExecutionHistory) will supersede the existing GetWorkflowExecutionHistory.
This API provides finer grain of control, i.e. control the number of returned history events, instead of number of returned batch of events.
This commit is part of effort to solve issue #385

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


Initial Tenable Commit

Fixes #
Proposed Changes
Tenable Playbooks, Parsers, Workbook and Detections from Tenable API

Some changes in the query. Also, for validations to pass, you will need to implement a CustomTable JSON schema file that has the parser name and the schema resulting from the parser. There are plenty of examples here. Documentation is here

Added

Hi @richlilly2004, the some of the validations are still failing, please can you recheck? Thanks

Any guidance? Sorry, not sure quite how to read the issues on those.

@preetikr - Since this requires a parser and a custom data connector, includes detections and a workbook, seems like this should be brought on as a Solution at this point, thoughts?

@richlilly2004 - If you want to you can publish these as solutions. Here's the process of packaging and publishing a solution. https://github.com/Azure/Azure-Sentinel/tree/master/Solutions#guide-to-building-azure-sentinel-solutions
Thanks!

Will do

Thanks @richlilly2004 - please incorporate @shainw feedback as well so that the content can be merged. Also you'll want to move these commits to solutions folder to package and publish as solutions per guidance.

Very old PR, closing.


feat: add popular projects to ecosystem-ci

@AriPerkkio is it possible to run them here before merging?

https://github.com/vitest-dev/vitest-ecosystem-ci/actions/runs/8829575584
https://github.com/vitest-dev/vitest-ecosystem-ci/actions/runs/8829573458

Hm, @AriPerkkio excalidraw should work fine 🤔 I don't understand why it's failing :(

Hm, @AriPerkkio excalidraw should work fine 🤔 I don't understand why it's failing :(

I've been looking into this and don't really know what's going on. Corepack seems to be throwing error Usage Error: This project is configured to use pnpm, even with following changes:
export async function test(options: RunOptions) {
	await runInRepo({
		...options,
		branch: 'master',
		repo: 'excalidraw/excalidraw',
		test: 'test:app',
+		agent: "yarn",
	})
}

I think we need to ask vite-ecosystem-ci maintainers for help. It might be good to move excalidraw to separate PR and merge this one without it.

excalidraw doesn't have packageManager defined in package.json. This command fails:
https://github.com/vitest-dev/vitest-ecosystem-ci/blob/602f008f4a5f35da1ad3c01da939b5836fee80f9/utils.ts#L427

https://github.com/vitest-dev/vitest-ecosystem-ci/actions/runs/8858388425/job/24326965801

effect is failing, but they are aware of the issue, and it should be fixed soon


python311Packages.dvc-data: 2.9.1 -> 2.11.0

Diff: https://github.com/iterative/dvc-data/compare/refs/tags/2.9.1...2.11.0
Changelog: https://github.com/iterative/dvc-data/releases/tag/2.11.0
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 247185 run on x86_64-linux 1

2 packages failed to build:

python311Packages.dvclive
python311Packages.dvclive.dist

10 packages built:

dvc
dvc-with-remotes
dvc-with-remotes.dist
dvc.dist
python310Packages.dvc-data
python310Packages.dvc-data.dist
python310Packages.dvclive
python310Packages.dvclive.dist
python311Packages.dvc-data
python311Packages.dvc-data.dist


Updated maintainers' page.

Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

Don't forget me! :smile: My e-mail is<EMAIL_ADDRESS>

lgtm


Interop: Java/Kotlin

Dart currently supports interop with C code via dart:ffi. We should investigate if we can provide similar interop with Java and Kotlin code.
This would be a supplement to Flutter's plugins, and would allow for calling into Java / Kotlin directly from Dart code, also from pure-Dart apps (such as CLI apps).

//cc @liamappelbe

Is there any progress on this matter，will support in next dart release？

@Aurora-zls I think the work is ongoing in this repository: https://github.com/dart-lang/jnigen


Remove hard "template1" in pg

When you use a pooler (like pgpool) with this cookbook you can't create database with template1 as template because your connection isn't close. It will be better if we can configure the database connection.

:+1:

Also, the default should be postgres, not template1.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

At this time I'm going to close this issue out as we've deprecated this cookbook. The original intent was to abstract database implementation details away from the end user. In hindsight this was a mistake as it is difficult / impossible to make a generic interface for databases that works across all systems. Instead we plan to move the individual providers into their respective cookbooks (mysql, postgresql, sqlite, sql_server) where they can fully utilize the feature set of the underlying DB systems.


[1.3.x] enable ServiceLoader discovery across classloader layers

Backport of #5540
Since the cherry-pick applies easily, I was wondering if we could backport this?

+1 from me. @eatkins?


Version 2024.9.3 can't be updated by the playstore under Android 9

Describe the Bug
The current Version (2024.9.3) of Integreat, Malte and Hallo A. can't be updated via playstore under Android 9. Android 14 is not affected and installing via *.apk is also fine. Sometimes the update spinner is running for ever.
Steps to Reproduce

Make shure your version is less than 2014.9.3 on Android 9
Try to update the app by clicking "update"
after spinner the app is not updated and button shows 'Update'

Expected Behavior
App is updated

Android 9 is API level 28
Our minSdkVersion is 21

I have an old Note 8 with Android 9 and an OnPlus3 with A9, both have the same effect.  I uninstall Integreat then reboot and then install V9.2 by apk and try to update to 9.3 via the Appstore
https://github.com/user-attachments/assets/13e4c137-7602-4871-9583-e0c2a6f17f2d

After further tests I now strongly suspect that the error always occurs when I try to update from an apk-installed version via appstore to current version, since I tried to update from various older versions but had no problems updating my phone in the past. So "suspend" this bug till next week checking then the update by appstore.

Version 2014.9.4 was updeted normally on my real device, so I close thisbug as "no bug"