Unable to navigate to https://community.chocolatey.org/installChocolatey.cmd

DISPLAY RESULTS
ERROR 404: PAGE NOT FOUND.

We moved the page that you are looking for, sorry!
You know how to hack URLs and you're trying to find something.

Try Searching Again

┆Issue is synchronized with this Gitlab issue by Unito

Can I ask what makes you think that there should be a page at that address?
https://community.chocolatey.org/installChocolatey.cmd
Were you linked to there from another location?

Hello Gary,
This link was shown or connected from some other page please check.
Please check this page
Chocolatey Software Docs | Setup / Install
https://docs.chocolatey.org/en-us/choco/setup#install-using-powershell-from-cmd.exe
Check this line
"You can also get to this file by going to
https://chocolatey.org/installChocolatey.cmd."
Best Regards
Alex philip
On Sat, Sep 10, 2022 at 2:09 PM Gary Ewan Park @.***>
wrote:

Can I ask what makes you think that there should be a page at that address?
https://community.chocolatey.org/installChocolatey.cmd
Were you linked to there from another location?
—
Reply to this email directly, view it on GitHub
https://github.com/chocolatey/home/issues/186#issuecomment-1242670193,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH7BTHDDUGU4FLUTW55GX73V5RCM7ANCNFSM6AAAAAAQJHVBWY
.
You are receiving this because you authored the thread.Message ID:
@.***>

@Allu-Philip Thank you for taking the time to report this issue. This should now be corrected.


Prevent Datastores Id's From Starting with Dot

In the root tree of the snapshot we intermix data stores, and runtime blobs. This change prevents data stores from starting with ., so runtime blobs, which all start with . can be disambiguated easily.
related #4310

The work @arinwt is doing should make this change obsolete, right? Should we just wait?

closing, as i don't think it's necessary with #4485


Failed to Load the native TensorFlow Runtime

I installed tensorflow through Anaconda with Python 3.6.2 . It got installed successfully. But I am not able to import tensorflow. and I am using a 64 bit machine with Windows 10 OS. I got following error:

import tensorflow as tf
Traceback (most recent call last):
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 18, in swig_import_helper
return importlib.import_module(mname)
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 978, in _gcd_import
File "", line 961, in _find_and_load
File "", line 950, in _find_and_load_unlocked
File "", line 648, in _load_unlocked
File "", line 560, in module_from_spec
File "", line 922, in create_module
File "", line 205, in _call_with_frames_removed
ImportError: DLL load failed: The specified module could not be found.

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 41, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 21, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 20, in swig_import_helper
return importlib.import_module('pywrap_tensorflow_internal')
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow_init_.py", line 24, in
from tensorflow.python import *
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python_init_.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 52, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 18, in swig_import_helper
return importlib.import_module(mname)
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 978, in _gcd_import
File "", line 961, in _find_and_load
File "", line 950, in _find_and_load_unlocked
File "", line 648, in _load_unlocked
File "", line 560, in module_from_spec
File "", line 922, in create_module
File "", line 205, in _call_with_frames_removed
ImportError: DLL load failed: The specified module could not be found.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 41, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 21, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 20, in swig_import_helper
return importlib.import_module('pywrap_tensorflow_internal')
File "C:\Users\HP\AppData\Local\conda\conda\envs\tensorflow\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

/CC @mrry any ideas?

Try following the instructions here, and if those fail to help please answer all of the questions in the issue template so we can triage the problem.

Pay attention to CUDA operation, since exactly this error is reported when CUDA is not working at all. Try from cmd e.g. c:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v8.0\extras\demo_suite>oceanFFT.exe
If it fails, try to update driver (Device manager / Display adapters / NVIDIA...) of your graphics card.


Check bar visibility

We've got hide() and show() also toggleShyVisibility() and isShy() both protected tho.
How are we suppose to check the visibility state of BottomBar ?

I once experimented with if (((CoordinatorLayout.LayoutParams) mBottomBar.getLayoutParams()).getBehavior() != null) {, it should be true if the bottombar is not visible, but not sure whether it still works or whether it's reliable at all.

if (this._bottombar.getBar().isShown) this._bottombar.hide();
else this._bottombar.show();

Although there's no show / hide methods in the new version (yet), you can just change the visibility like any other View now.


Add Yann to Contributors list

I'm excited about working on open source projects!

Hi  @YannMjl, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Download videos with FFmpeg

This is a request from my friend (he asked me to make this app in the first place).
Now, with the power of native android. I don't think it is impossible to do it. I will do some research and implement this feature in the future.

[ ] Setup FFmpeg on Android Studio
[ ] Try downloading mp3 and m3u8 format videos
[ ] Set download path inside settings

下載這方面可以參考TubeMate App是可以直接下載的

好的 多謝


How to indicate xenografted or transgenic animals

MiAIRR currently assumes that all entries in a record pertain to the same species, which is defined in data set 1. As species will be required (#96) and restricted by a species taxonomy (most likely NCBITAXON, #97) we need a mechanism to describe subjects that cannot be clearly assigned to a single species.  Note that it is not possible to unambiguously infer the species from target_locus as the gene symbol does often not contain this information. Therefore the sequence information could be wrongly interpreted as host sequences.

Chimeric animals that have been grafted with human bone marrow (and potentially other tissues). Here also additional fields like cell_subset would be affected.

One solution would be to introduce a "species prefix" for the fields involved. However, as this changes MiAIRR, it a slightly bigger decision.

Re-opening as PR #260 was not merged.

I believe this is done

Nope, #614 was never merged...


Feature Request - Queuing Request Logs

Hey, absolutely love this package - great work on it!
We're currently running into an issue where because every request is written to the request_logs table. This results in a single point of failure (if our writer instance fails). It also means that for even read-based operations our writer is only being used. It would be really cool if this library allowed us to queue the storing of request logs. This would allow our GET requests to only touch our load-balanced reader instances and never touch the writer. It would also speed up our requests as we currently have the request logs table on a separate database connection.

Hi @Sammyjo20 👋

absolutely love this package - great work on it!

Thanks so much 🙏

We're currently running into an issue where because every request is written to the request_logs table. This results in a single point of failure (if our writer instance fails). It also means that for even read-based operations our writer is only being used. It would be really cool if this library allowed us to queue the storing of request logs. This would allow our GET requests to only touch our load-balanced reader instances and never touch the writer. It would also speed up our requests as we currently have the request logs table on a separate database connection.

I see your use case 🤔  I originally avoided queued jobs and looked at Terminable Middleware but finally decided to use a synchronous listener (aka non-queued) for the event Illuminate\Foundation\Http\Events\RequestHandled.
Solution right now
Could we not just queue the listener we already have for the Illuminate\Foundation\Http\Events\RequestHandled? No - because the Request object is not serializable.
That means that we need to call the logger (implementing RequestLoggerInterface) synchronously, but there is no problem in using a logger that does the actual writing using a queued job.
This can actually be done already, in two ways (option 2 is easier):

Add your own driver which uses a queued job. Add this driver to the config and set that as the default
Make a new model which extends \Bilfeldt\RequestLogger\Models\RequestLog and simply override the log function to queue the actual saving. Then configure that the package should use your new child class instead.

Should we implement it into this package as a core feature?
Albeit it is already possible, then there is no build in config for it, which would be the ideal for any developer wanting to achieve this.
I originally thought there were no use cases for this, since the performance concern makes no sense simply because writing is done after the response has been returned to the user. But I do understand your concerns and could imagine there could be other edgecases.
Hence I am okay adding this as a build in feature of the package. I will keep this issue open for now, and if I find the time to add this myself I will probably do so. But I am open for any PRs for this feature already.

Hey @bilfeldt I'm really sorry for not replying sooner - I feel awful!
Thank you for such a detailed reply. It would be great if this was a feature in the library but I understand you may have other commitments at this point in time. I wanted to share this with you - we wanted to see what was slowing some of our requests down and it was what I expected:

No worries at all @Sammyjo20 :)
Just to be clear, that query is being executed after the response has been returned to the user right?

No, the middleware isn't terminatable - so it'll be part of the request - this is why it's being picked up in the request lifecycle

No, the middleware isn't terminatable - so it'll be part of the request - this is why it's being picked up in the request lifecycle

The middleware is only used to "tag" that the request should be logged. Logging happens using the event listener here https://github.com/bilfeldt/laravel-request-logger/blob/2fe2f6ebc48b607f83c8c05de9926b8ed68369d1/src/EventServiceProvider.php#L11-L15
And the event RequestHandled is fired after the response has been returned - right?

Hmm, the listener isn't queued so that leaves me to think it might be running synchronously? I might be wrong - where abouts is the RequestHandled event being fired?

Sorry - I didn't realise it was a Laravel event:
https://github.com/laravel/framework/blob/988206bcacf79829bda66af4a016778e4b4d762a/src/Illuminate/Foundation/Http/Kernel.php#L152
But it doesn't look like this is after the request lifecycle. I'll keep digging haha

Yes the listener is not queued (and cannot be, because of the serialization issue I mentioned).
I might have misunderstood the RequestHandled event. I was almost certain it fired after the response was returned to the user, but your reference would indicate I am wrong there 😒
If that is indeed the case then we/I should definitly make it possible to log sync either using a queued job or using terminatable middleware.

Sorry @Sammyjo20 had not seen your answer when I posted above.
So we agree that my original postulate was wrong and that the event RequestHandled is fired before the response is returned to the user correct?
I wonder if there is a RequestTerminated event we could listen for 🤔

I'm currently working off a fork and I will see if it works - just installing it locally - the PR will be a little crude without testing but I am happy to put one up if you like? Thank you so much for being so cooperative on this one!

I'm currently working off a fork and I will see if it works - just installing it locally - the PR will be a little crude without testing but I am happy to put one up if you like? Thank you so much for being so cooperative on this one!

Feel free to post a PR for this - absolutely 👍
A few ideas I have here:

Simply add a new queued config and then simply move this part into a queued job: https://github.com/bilfeldt/laravel-request-logger/blob/2fe2f6ebc48b607f83c8c05de9926b8ed68369d1/src/Models/RequestLog.php#L97-L124
Instead of listening for the RequestHandled event, then we register a terminatedCallback: https://github.com/laravel/framework/blob/7c680bb58dfb15923209045f99ed5cf46c31f45d/src/Illuminate/Foundation/Application.php#L1354-L1364
Instead of using the RequestHandled event then we use a terminatable middleware - that would be a breaking change, because it would require everybody to use that middleware, and there are currently other ways to enable logging.

I personally vote for combining option 1 and 2 btw 👍

I think either option 1 or 2 could be good. For people who haven't enabled queueing it will still run in sync, but for others it'll queue it. Option 2 is probably easier to implement because instead of an event being fired, you could run the terminatable middleware which checks for a request.
I'd be happy to try and change my PR to option 2 if you would like.

With option 1 you could even Job::dispatchAfterResponse() which would possibly reduce latency even further.

Good idea. Yes a PR would be much appriciated.


How to install?

Can I have instructions on how to install this and what are the prerequisites?

Hi,
Sorry, but we currently do not provide documents for potential self-hoster. It's still in early development stage and is not ready for massive production use. Please seek other project instead.

PS, your English intonation seems unnatural. Please use proper English for all kinds of communication. If you're willing to use Mandarin Chinese, just go ahead.


bastion support for availability_zones and public IP and subnets

what

Add support for availability_zones
Fix issue with public IP and subnets
tflint requirements -- removed all unused locals, variables, formatting

why

All instance types are not available in all AZs in a region
Bug fix

references

Internal Slack reference

In the future it would be best to use a instance_type_offerings query: https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/ec2_instance_type_offerings
This works just fine, though.


ts-dev-tools is not available when @ts-dev-tools/react is installed in a monorepo package

Describe the bug
ts-dev-tools does not exist in monorepo package node_module/bin directory
It works when installing @ts-dev-tools/core
To Reproduce
Steps to reproduce the behavior:

In a fresh monorepo package
Run yarn install @ts-dev-tools/react
Run yarn ts-dev-tools
See error error Command "ts-dev-tools" not found.

Expected behavior
Should execute ts-dev-tools bin:
$ /home/test/monorepo-project/packages/test/node_modules/.bin/ts-dev-tools
Usage
    ts-dev-tools install [path from project root]

Fixed in 1.2.0


happens when signing into a microsoft account

Type: Bug
I sign out of own personal microsoft account
I sign into my work microsoft account
failure
VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:35:30.018Z)
OS version: Linux x64 5.19.0-32-generic
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11700 @ 2.50GHz (16 x 4615)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 0, 0

Memory (System)
62.48GB (51.08GB free)

Process Argv
--unity-launch --crash-reporter-id 1bdc6b9b-86ab-4003-84e1-8c3f6a9358e1

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
wayland

Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vscaac:30438847
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848:30577860
nodejswelcome1cf:30587006
2e4cg342:30602488
89544117:30613380
pythonsymbol12:30652348
vscsb:30628654

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!


Show Only Active Predictors Button doesn't work in HealthCheck

pdstools version checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pdstools.

Issue description
Show Only Active Predictors Button doesn't work in HealthCheck
Reproducible example
.

Expected behavior
Show only_active button HC used to hide inactive ones but now it doesn't
Installed versions
---Version info---
pdstools: 3.4.4
Platform: macOS-14.5-arm64-arm-64bit
Python: 3.11.4 (main, Jul  5 2023, 08:40:20) [Clang 14.0.6 ]
---Dependencies---
plotly: 5.18.0
requests: 2.32.3
pydot: 1.4.2
polars: 0.20.31
pyarrow: 11.0.0
tqdm: 4.66.1
pyyaml:
aioboto3: 11.0.1
---Streamlit app dependencies---
streamlit: 1.35.0
quarto: 0.1.0
papermill: 2.4.0
itables: 1.5.1
pandas: 2.1.4
jinja2: 3.1.4
xlsxwriter: 3.1.9

Is this a health check core functionality or the streamlit app? Let's do some basic RCA on this

This appears to be fixed as of last version.


MemoryWatermark tune operation interferes all apps.

What happened:
Sandbox not full isolated.
What you expected to happen:
If app A cost massive memory which result in availble memory less than Critical MemoryWatermark, then the runtime will tune smr param: "MAX_WORKERS_PER_APP", that's now the logic. But the tune operation will scale in the running app B , the scale in operation is triggered by app A.
That sound like not  a good isolation behavior, apps interfere each other.

Would you be interested to work on this? :)

yes, sure, I woul like to that.

Happy to review your PR!

There are several approaches for implementing per-process RSS limit:

cgroup: Put each worker into its own memory cgroup.

Pro: The limit is accurate.
Con: May not play well with a sandboxed environment (seccomp/dropped privileges)

rlimit: use RLIMIT_AS to limit the address space (VSZ) of each worker process.

Pro: Simple and plays well with OS-level sandboxing.
Con: Prevents us from enabling V8 virtual memory tricks for optimizing WebAssembly memory accesses.
On the API side, memory limit should be passed to the runtime as a field in Metadata.


Kubernetes example wanted?

Hi there,
as my homelab nowadays has Kubernetes running, I migrated my magic mirror setup into it.
First of all: @bastilimbach thanks for packaging the docker image! Works nicely!
Would there be interest in the configuration I used to get my two mirrors hosted in server-only mode in Kubernetes?
A rough overview:

a configmap for per-mirror configs (containing config.js and both custom.css and main.css (the latter only because I have to)
one NFS volume to store the modules/
one deployment / service pair to run the container for each mirror instance
one shared nginx to host the calendar ICS (fed from the outside with vdirsyncer)

It's all running nicely now, but the things that bug me are:

the default MM modules will not get updated with the image (and option to always overwrite them would be nice)
~I have to carry main.css in the configmaps, as mounting the configmap to css/ would prevent access to the main.css from the image.~ (fixed see below)

If there is interest in my setup I can post them here, but also feel free to tag me ask questions in this issue.

I'm interested. Did a first hack a year ago but without a good solution for the modules ...
As this repo is restricted to serveronly I did a docker setup too which supports also the second use case (running the mm on the screen of the pi). I don't want to entice users away from here, this is the official mm-docker-repo, but I solved you're first point (override default modules) there already.

Sure! Documentation/Samples are always welcome. However I wouldn’t add such information into the readme as this is really detailed and setup specific. Maybe we create a GitHub Wiki page for this. We could also add more docker compose samples there etc.

My first instinct would be to put it into a docs/k8s/ folder inside the repo itself. That is much easier to open PRs if anything needs changing and can have its own README.md describing the relations and considerations.

so, at least for ̀main.cssI found the right way to do it:subPath` mounting my config volume.

@klaernie I just checked how GitHub Wikis are working under the hood. I thought this would be just some files in a folder like GitHub Pages. But no. It's persistent in a separate git repo which makes it super hard to contribute -.-
Therefore I would agree with you. Let's create a docs folder with a summary in the top-level README.md. Then in every folder, we can have specific readme files 👍

meanwhile I investigated several hours in this and wrote a helm chart for MagicMirror. You can find the first version here.
This setup uses normal config.js and custom.css files in subfolders. Modules have to be specified as list in the values.yaml, they are cloned (incl. npm install) at start.

@klaernie any updates? Are you still interested in creating a PR for the documentation? Otherwise I’ll close the issue.
No problem if you currently do not find the time. Just wanted to check if there is still the desire to create a PR in the future😊

I would like to see your setup before I go and make mine :)
I have a small Raspberry Pi 4 cluster in my basement, and would like to give it a job.

Yes, still planning on writing it up. Just work stuff got in the way. Sorry to leave you hanging.

Absolutely no problem 😊 I’ll leave this issue open as long as you want 👍

I too am curious as to how you are treating different modules.
Say you have some heavy weight module like facial recognition, did you find a way to have that run as a stand alone pod (with opencv and other dependencies) or does everything still have to be run on a single pod that has all these dependencies packaged into the image.

@DrSoops actually everything in my setup runs in a single pod (per instance). The PR is now up, so you can take a look how I did it. Also in my use case the device running the display for the mirror and the kubernetes cluster are not the same - my entire approach is based on taking the cpu load from my display devices (a Beaglebone Black and a RaspberryPi) and moving it onto a beefy x86 server.
So I don't know, how facial recognition is implemented (I don't have this need), but I assume you need to feed the camera somehow. Guessing that you use paviro/MMM-Facial-Recognition, you'd need to first get the webcam into the pod, or add an network camera feed input module for it to work. But then you can easily move the entire server-side of magic mirror, which includes the face detection, onto a beefy machine.

Single pod with one beefy server as a backbone. That makes sense and works, I was just hoping to be able to somehow self contain the cluster with spreading the load on extra Pis/Jetsons.


Add case management

User can

create / load case
add addresses to case
start investigation from selection of case addresses
find overlapping addresses -> inform user if addr X is already part of another case

will be moved to another compoent


Add instal task/command

Allow for the main package to be installed.
Look at installing dependencies.

Do note that when containers, bob can only install within the repository (mounted volume)


Physics tick counter is incorrect within frame

Tested versions
3.6 beta 5
(probably all earlier)
Likely also in 4.x
System information
All
Issue description
While investigating an MRP, I realised that we are incrementing the physics tick counter in the incorrect place.
Currently the physics tick counter is incremented at the end of the physics tick, whereas the physics tick does not change logically until the next physics tick begins.
This means that, especially at low tick rates, the reported tick within _process() will be one higher than the current tick.
Steps to reproduce
func _physics_process(delta):
	print("physics process - starting tick " + str(Engine.get_physics_frames()))

func _process(delta):
	print("process - physics tick reported as " + str(Engine.get_physics_frames()))

Minimal reproduction project (MRP)
physics_tick_counter.zip
Output:
process - physics tick reported as 12
physics process - starting tick 12
physics process - starting tick 13
physics process - starting tick 14
physics process - starting tick 15
physics process - starting tick 16
physics process - starting tick 17
physics process - starting tick 18
process - physics tick reported as 19
physics process - starting tick 19
physics process - starting tick 20
physics process - starting tick 21
physics process - starting tick 22
physics process - starting tick 23
physics process - starting tick 24
physics process - starting tick 25
physics process - starting tick 26
process - physics tick reported as 27

Discussion
This may appear as a kind of "glass half empty / glass half full" kind of issue, but the problem is that input etc that occurs during the frame will report the wrong physics tick before the next one has started, which can be a problem if you e.g. position objects based on the physics tick.
It's fairly easy to solve by moving the count increment to the start of the physics tick, however, this will take some care to solve properly as it likely to cause regressions. Notably already I have confirmed that Input.is_action_just_pressed() would need adjustment.
This problem is especially notable at low tick rates (e.g. using physics interpolation). At 10 tps, and 60 fps, the reported tick is considerably out of sync.
e.g.
tick 0 begins
frame 0 reports tick 1
frame 1 reports tick 1
frame 2 reports tick 1
frame 3 reports tick 1
frame 4 reports tick 1
frame 5 reports tick 1
tick 1 begins
frame 6 reports tick 2
frame 7 reports tick 2
frame 8 reports tick 2
frame 9 reports tick 2
etc

This is what I see as of 4.3.beta 5833f5978 (Linux):
process - physics tick reported as 0
physics process - starting tick 0
process - physics tick reported as 1
physics process - starting tick 1
physics process - starting tick 2
physics process - starting tick 3
physics process - starting tick 4
process - physics tick reported as 5
physics process - starting tick 5
physics process - starting tick 6
physics process - starting tick 7
physics process - starting tick 8
physics process - starting tick 9
physics process - starting tick 10
physics process - starting tick 11
physics process - starting tick 12
process - physics tick reported as 13
physics process - starting tick 13
physics process - starting tick 14
physics process - starting tick 15
physics process - starting tick 16
physics process - starting tick 17
physics process - starting tick 18
physics process - starting tick 19
physics process - starting tick 20
process - physics tick reported as 21
physics process - starting tick 21
physics process - starting tick 22
physics process - starting tick 23
physics process - starting tick 24
physics process - starting tick 25
physics process - starting tick 26
physics process - starting tick 27
physics process - starting tick 28
process - physics tick reported as 29
physics process - starting tick 29
physics process - starting tick 30
physics process - starting tick 31
physics process - starting tick 32
physics process - starting tick 33
physics process - starting tick 34
physics process - starting tick 35
physics process - starting tick 36
process - physics tick reported as 37
physics process - starting tick 37
physics process - starting tick 38
physics process - starting tick 39
physics process - starting tick 40
physics process - starting tick 41
physics process - starting tick 42
physics process - starting tick 43
physics process - starting tick 44
process - physics tick reported as 45
physics process - starting tick 45
physics process - starting tick 46
physics process - starting tick 47
physics process - starting tick 48
physics process - starting tick 49
physics process - starting tick 50
physics process - starting tick 51
physics process - starting tick 52
process - physics tick reported as 53

Does this mean it has the same issue as 3.x?

Yes, also present in 4.x. Will add to issue.

Fixed by #92941 in 3.x, but still present in 4.x, will change milestone and look at fixing there.


Add ignored test case for dev-deps propagation

Stack:

#1007
#1031
#1030 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

Replaces https://github.com/software-mansion/scarb/pull/1008


Investigate bonsaidb connections dropping after ~10 seconds

Original issue is in comment 2.
@daxpedda introduced max_idle_timeout in fabruic, which offers a simple solution to the problem, but after thinking about it, we need more logic in BonsaiDb itself to handle this in ways that I would want as a database administrator.
The Fabruic setting can only be done at initialization time, and it doesn't impact WebSocket clients. To me, BonsaiDb's idle strategy needs to work similarly for all types of connections. Additionally, if we can't turn on or off the idle timeout, then features like PubSub start requiring reconnection to function. Finally, authentication requires a back-and-forth negotiation, and we shouldn't force clients to reconnect and reauthorize unless absolutely necessary.
Thus, here's what I think we should do:

[ ] Add new Server configuration options:

idle timeout
authenticated idle timeout
server keepalive interval (must be less than idle timeout)

[ ] Set Fabruic's the max_idle_timeout to some reasonable value within BonsaiDb. quinn's documentation warns against using None due to malfunctions or bad actors being able to cause futures that never time out.
[ ] Introduce KeepAlive on both Request and Response, perhaps with unique names to indicate the direction of flow.
[ ] Introduce a server-generated KeepAlive:

If the connection should be kept alive (authenticated, or there's an active PubSub subscriber), the server should send a KeepAlive response based on the keepalive interval configuration
The client should respond with a KeepAlive request whenever it receives the server message.

[ ] Update the server to keep track of the last time a Request was seen.
[ ] Update the loops that fetch new requests from a client to have timeouts based on the last time a request was seen and the correct idle timeout based on the connection's authenticated state.

This might be a QUIC timeout thing, tell me if it's that, then I can expose a setting that disables it or let's you set whatever you like. If you like to solve it from your side you can send a ping or something to prevent it from timeout.

I appreciate your quick reply! I was recording the creation of khonsulabs/minority-game and realized that the QUIC connections were disconnecting after no traffic was sent. I'll take a look at what's happening and let you know what I think we should do.
Most of the new issues from today/yesterday that have very little info were just created on-the-fly as I was recording so that I didn't forget.

This also affects websocket connections. Given the idea of #275, I wonder if this request and it shouldn't be merged into one specific "system".


Generate binomial CDF with mu instead of prob

Is your feature request related to a problem? Please describe.
The feature is related to a problem where I am unclear as to how to translate my data into parameters: n and p for nbinom.cdf. I am sort of lost translating R to Python here given different params:
test_arr = np.array([(1, 1, 3, 1, 1, 0, 0, 2, 2, 0, 1, 3, 0, 0, 0, 0, 3, 2, 1, 5, 2, 2,
           4, 0, 7, 3, 0, 0, 2, 1, 2, 0, 1, 4, 1, 0, 0, 0, 0, 0, 0, 0, 1, 1,
           0, 0, 0, 0, 0, 2, 0, 3, 3, 1, 3, 0, 0, 2, 0, 3, 1, 0, 0, 0, 0, 1,
           0, 4, 0, 0, 0, 0, 0, 0, 1, 0, 0, 1, 2, 1, 2, 0, 0, 3, 2, 2, 0, 3,
           0, 1, 0, 1, 0, 0, 0, 2, 1, 0, 1, 0)])
nbinom.cdf(test_arr, n=?, p=?)  # how determine n and p from the data?

Describe the solution you'd like
When using R, I can obtain nbinom.cdf seemingly from the data itself
library(pscl)
arr1 <- c(1, 1, 3, 1, 1, 0, 0, 2, 2, 0, 1, 3, 0, 0, 0, 0, 3, 2, 1, 5, 2, 2,
           4, 0, 7, 3, 0, 0, 2, 1, 2, 0, 1, 4, 1, 0, 0, 0, 0, 0, 0, 0, 1, 1,
           0, 0, 0, 0, 0, 2, 0, 3, 3, 1, 3, 0, 0, 2, 0, 3, 1, 0, 0, 0, 0, 1,
           0, 4, 0, 0, 0, 0, 0, 0, 1, 0, 0, 1, 2, 1, 2, 0, 0, 3, 2, 2, 0, 3,
           0, 1, 0, 1, 0, 0, 0, 2, 1, 0, 1, 0)
m1 <- zeroinfl(arr1 ~ 1, dist='negbin')
m1$theta  # dispersion param

library(VGAM)
# size is dispersion parameter
pnbinom(arr1, size=m1$theta, mu=mean(arr1))

It would be nice to derive the CDF of a value/vector by specifying the dispersion parameter and the mean.
Describe alternatives you've considered
N/A
Additional context (e.g. screenshots)
N/A

statsmodels is supposed to have some functions to convert from GLM/Regression parameterization of negbin to the standard negbin.
scipy avoids duplicating distributions with just changes in parameterization

Wow thanks again for all your time and help. Are you able to help me better understand how I can determine n and p, if say, my data was the numpy array: test_arr ?

you can look at the code of NegativeBinomialP at predict and convert_params methods, for appropriate version of "P"
I don't find convert_params for NegativeBinomial right now, but old issue and recipe is in
https://github.com/statsmodels/statsmodels/issues/106#issuecomment-43961704

@afogarty85 When you have figured this out, would you let us know what we should add to the documentation to help others? Just recently I noticed a similar question in gh-11465.

Thanks all, this has been super helpful, particularly this: https://github.com/statsmodels/statsmodels/issues/106#issuecomment-43961704.
In terms of documentation, perhaps it might be helpful to add @josef-pkt's work showing how we can estimate n and p if we already have collected some data and then we want to proceed to other methods like the CDF.
from scipy.stats import nbinom
import statsmodels.api as sm
import numpy as np

nbinom_rv = nbinom.rvs(n=1.089, p=0.514, size=10000)
intercept = np.ones(len(nbinom_rv))
res = sm.NegativeBinomial(nbinom_rv, intercept, loglike_method='nb2').fit();
mu = np.exp(res.params[0])
alpha = res.params[1]
res.params

# find size and prob
Q = 0  # nb2, cited most common
size = 1. / alpha * mu ** Q
prob = size / (size + mu)
print('estimated params:', 'size:', round(size, 4), 'prob:', round(prob, 4))
cdf_out = nbinom.cdf(nbinom_rv, n=size, p=prob)
np.allclose(nbinom_rv, nbinom.ppf(cdf_out, size, prob))

@afogarty Looking more closely, part of your question is about fitting the distribution to data. That is a feature SciPy doesn't have for discrete distributions, but it has already been requested in gh-11948. In the meantime, you have statsmodels and, if you're interested, a recipe in gh-11948.
The other part is about parameterization in terms of mu. Would gh-13292 address that and close this PR?

As an aside to statsmodels:
The models have now get_distribution methods that create frozen scipy distributions from the parameterization in the regression models using the appropriate parameter conversion functions.
Another example is the Beta distribution where the regression model also has a mean-dispersion parameterization that needs to be converted to the standard parameterization in scipy.


Feature/141145113/add searchall functionality

What does this PR do?
This PR contains functionality that enables the user search in all uploaded documents
Description of Task to be completed?
User can search for a particular phrase in the entire document uploaded
How should this be manually tested?
Type the following :
git clone https://github.com/andela-iamao/inverted-index.git
cd inverted-index
npm install
npm start

Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?

Add search all functionality

Changes Unknown when pulling c2361a249da3ca42e177eb322f9a8a809a177193 on feature/141145113/add-searchall-functionality into ** on development**.


Remove / clean up the --no-result output flag

Currently the --no-result flag is not intuitive. It prints out the result if set to false if there is no output directory specified. This should be made more obvious. Either the flag should be renamed or this functionality should be removed since the outputDir is specified by default. I'm not even sure there is a way to have the outputDirectory be undefined.

Resolved in #827


make default flags include timeout | all sims share flags

Have the simulation rules share the SIM_FLAGS variable so that people can override them if they want.

We don't always want to run +verbose, it significantly slows simulation

I personally, like to have the +verbose flag on by default. But then again... personal preference. What are other peoples thoughts?

If you looked at how boom-template worked, +verbose was only used for generating .vpd and .out.
https://github.com/riscv-boom/boom-template/blob/4c0a00bc042f0dae27ee8cefffd5d44973a98a89/Makefrag#L55-L66

+1 for defaulting for a faster option if it's materially faster (say >10%).  During debug, the user can add extra flags if they want more information.

So Ill guess, Ill default to just having timeout as the main simulator flags.
It'll be up to the dev to add +verbose

Yes not verbose by default its way more than 10%.

Ok. Just changed the SIM_FLAGS to not include +verbose. @colinschmidt will make a PR later to deal with the extra rules for .out/.run files.

As I mentioned @colinschmidt wants to deal with these flags/rules in a separate PR so Ill keep this for now.


Is it possible that you add this to Cocopods?

Sure, just added one. Let me know if it doesn't work.


Compress option not doing anything

    bless: {
      css: {
        options: {
          compress: true,
          force: true,
          cleanup: true
        },
        files: {
          'css/stylesheets/dev/app.css': 'css/stylesheets/app.css'
        }
      }

The result is no different that when I don't compress it.

I misunderstood this also

If I minify the css first and then bless it the output will always be unminified.

+1, running grunt-bless on minified css always output unminified css...

+1 I minify first and the output is then unminified.

Looks like blessc since 4.0.0 is not expanding minified CSS, this is very annoying as it looks like after you run cssmin on the minified version it wants to process the imports (if you remove the process import statement it does weirdness)


Release docs

Resolves #271
@krzysztofzablocki what would be the best way to update html docs when doing release? I'm still thinking it might be better to have docs up to date with master, not waiting for the release.

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: danger

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: danger

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: danger

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: danger

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: danger

I had few people come to me with questions why X doesn't work and pointing to docs, I feel like outside of us almost no one uses Master version of Sourcery so its better to have published docs reflect last released version.
What we can do is that we just add new task to Rake release phase, similar how we check whether you updated changelog, you could check whether doc command is not generating a new diff meaning its up to date, if it does we fail the release and print out the command to update them

you could check whether doc command is not generating a new diff meaning its up to date, if it does we fail the release and print out the command to update them

@krzysztofzablocki would it be a better automation if we just update the docs during the release?

yeah good idea, we'll need to make automatic add, commit and push then so need to make sure we don't mess it up :)

@krzysztofzablocki how should we do that? I guess it should create a pr for release with updated docs and version bump. But that will break release process into few steps - create pr, manually merge it (unless there is automated way to do that), then actually create release.

@ilyapuchka do you think it can potentially break something? I mean, I would treat this part as a part of bumping podspec or updating changelog.

Not break, but we can simply end up with lots of undocumented things. I've tried danger plugin for jazzy that should warn if there is something undocumented in the pr, but it does not seem to work.

we can simply end up with lots of undocumented things

I guess without danger plugin it's anyhow manual process to check if everything is documented, so maybe it can be checked as a part of the PR review process, so the undocumented code will be catched during the feature PR, not during the actual release process. But I'm still new to the codebase, so I'm not sure if I get the process and your idea right 😄

Sure, but there is Danger for that =) Doing it manually is very easy to miss such things. I would expect it to at least check if undocumented rate changed, at most outline undocumented symbols.
I've tried danger-jazzy, maybe I missed something, cause I'm not familiar with Danger plugins. I guess second pair of hands to try that will not hurt.
If we do that then we simply generate and commit updated docs after version bump and we can be sure it's complete (again if not for some reason - danger will warn us in release pr).
Sounds reasonable?

@ilyapuchka totally reasonable 👍
I've checked briefly the plugin and I see that it checks the undocumented.json file (you can check it here) generated by jazzy. So I guess to be able to check the documentation with the plugin on each PR we should generate documentation on each PR.

So, what we have now:

danger checks if there are any undocumented changes in PRs
docs are updated and checked for modifications before release
docset is copied to release zip
docsets folder is ignored in the repo to avoid merge conflicts

What is not automated:

pushing updated docs to master during release

Do we need anything else? @krzysztofzablocki @Antondomashnev


Grasshopper examples integration tests

Catch workflow bugs or not working components.

[ ] Research if possible to implement

Duplicate of #593


tinycbor: init at 0.5.3

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Is there an update on this?

LGTM.
Result of nixpkgs-review pr 79874 1

1 package built:
- tinycbor

Opps, sorry I missed getting back here. Still looking good!
Result of nixpkgs-review pr 79874 1

1 package built:
- tinycbor


Vision/Multimodal

With all the growing activity and focus on multimodal models is this library restricted to tune text only LLM?
Do we plan to have Vision or more in general multimodal models tuning support?

Hi @bhack thanks for the question! We haven't added any multimodal models yet as we are working to get good coverage of text-only methods first, but it's definitely something we are considering for the future. Out of curiosity, are there any multimodal models or techniques you'd be interested in seeing specifically?

There was a recent and interesting survey at:
https://github.com/UbiquitousLearning/Efficient_Foundation_Model_Survey

Moondream2 would be a good one to begin with!

Thanks @bhack and @matbee-eth for the suggestions! At this exact moment we do not have the bandwidth to take these on but we will keep them both in mind for the near future (re Moondream2 we are currently working on adding Phi-3). In the meantime, please let us know if either of you would be willing to contribute on this front.

Thanks @bhack and @matbee-eth for the suggestions! At this exact moment we do not have the bandwidth to take these on but we will keep them both in mind for the near future (re Moondream2 we are currently working on adding Phi-3). In the meantime, please let us know if either of you would be willing to contribute on this front.

Do you know of any PR's that cover end-to-end implementation details for doing such a thing? Just to assess whether it takes novel-work or if its just conforming to some sort of protocol/design.

Also fine-tuning other Meta foundational models would be nice like recent facebookresearch/segment-anything-2

Thanks for the input! We're still a small team so working hard to provide great memory savings and performance for LLMs first, but this is 100% on our radar.
Just out of curiosity - what kind of finetuning would you want to do with SAM2? Do you have any hard data or HW constraints?

Yes generally  it could be hard mining cases, highres, HW constrains etc.. So I think that also in Vision we really have the same type of fine-tuning needs.
I really hope that we could share some common infra/component between LLM, Vision and Multimodal without building 3 different frameworks.
But this will really depend on how well torctune could abstract some concepts.
Also if I understand you are prioritizing LLM I think that an early canary Multimodal/Vision model could be useful as a Canary test to lowering the risk of a more heavy refractory on a later stage.
I think that you can ask the collaboration of some Vision/multimodal teams internally to create more critical mass around the project.

E.g. see how many comments we had on the original SAM just related to fine-tuning:
https://github.com/facebookresearch/segment-anything/issues/5

Also just to make another example. Your WIP RLHF with PPO like https://github.com/pytorch/torchtune/pull/1005 or other approaches could be still useful in Vision/Multimodal:
https://encord.com/blog/guide-to-rlhf/
So I think this is why it is important to have some canary tests  on other domains to better validate the design.

@RdoubleA Where we could track the Vision part?

We currently support multimodal models (vision + text), you can take a look at Llama3.2 Vision: https://github.com/pytorch/torchtune/blob/main/torchtune/models/llama3_2_vision/_model_builders.py which uses CLIP image encoders.
The components used here can be used to add future multimodal models, especially for DeepFusion (https://github.com/pytorch/torchtune/blob/main/torchtune/modules/model_fusion/_fusion.py). I am currently working on enabling EarlyFusion here: #1904. This would support models like Llava.
Is there any model or feature you are particularly interested in?

SAM2 now has official training code from META. Do you think we could have a pure vision model like this?

Hey @bhack , thanks for asking! Currently we don't have it in your roadmap :/. Our main focus is LLM, including multimodality. It is possible that, as we build for multimodality some vision models will be natively supported. But SAM2 is not in our plans just yet.

hi @RdoubleA,
thanks for including the clip model and LLama3.2-Vision models. I wonder if there is any plan to include SigLip model in near future, as I intend to work on it.
thanks

Thanks fro your interest in our vision capabilities! SigLIP is an important model for us to onboard as it's the primary alternative to CLIP for popular VLMs currently. We will likely add SigLIP during the upcoming months but it's not top priority as there are a number of VLMs we can onboard with CLIP first.
If SigLIP is very important for you we can take that into consideration and we would also welcome you to contribute SigLIP to the codebase if you're able and willing too. It would look fairly similar to the CLIP image model code.

Thanks fro your interest in our vision capabilities! SigLIP is an important model for us to onboard as it's the primary alternative to CLIP for popular VLMs currently. We will likely add SigLIP during the upcoming months but it's not top priority as there are a number of VLMs we can onboard with CLIP first.
If SigLIP is very important for you we can take that into consideration and we would also welcome you to contribute SigLIP to the codebase if you're able and willing too. It would look fairly similar to the CLIP image model code.

thank you @pbontrager for confirming. I am happy to work on SigLIP model and will let you know once it is completed.

Thank you that sounds great! Once you're ready to start on it could you share which siglip version you're building? If you're doing just the vision portion of the model, you should be able to implement model/siglip with just  _transform.py, _component_builders.py, _convert_weights.py, and _model_builders.py files using the clip folder a a blueprint.

@anilbatra2185 , to avoid rewriting code, you may want to write a small RFC (request for comments) PR, with a basic skeleton of the classes. For example, you may need to change the current transformer API and discussing it before writing your whole code may save you some time.
Example:
path/to/folder_a:
  |- my_function.py
  	class_a
  	class_b
 |- my_other_function.py
     util_1
     util_2

Class A:
	def method1(arg1, new_arg_2):
		...
	def method 2
		...


Enhancing events page

Making the events page more smoother and dynamic which gives perfect flow with the topic assigned to it.
link:
https://youtu.be/ebanWoSGiKY

@Virtual4087 Thank you for the advice and surely will be following it in the upcoming tasks.


POL-620 Updating deps

Codecov Report

Merging #284 (a499bc7) into master (c8a5ba9) will not change coverage.
The diff coverage is n/a.


branch 2

This change is part of the following stack:

#17

#18 ◀

#19

Change managed by git-spice.


Ability to mark Models as "unrelated"

Assume we have two tables:
Users {
id,
name
}
Statuses {
id,
user_id,
status
}
Current ActiveJDBC implementation assigns Statuses.user_id as foreign key to Users.id, but in some cases we don't want it (for example, when cache purges in parent table).

@doctorrokter I think best to implement it using annotation like:
@UnrelatedTo(User.class)
public Status extends Model {}

thoughts?

This is a best implementation, I think. In ActiveJDBC style.
Even more!
@UnrelatedTo({User.class, Another.class, OneMore.class})
public Status extends Model {}

m?

@doctorrokter, this is done: http://javalite.io/caching#unrelating-models


errors in older .sql files

I have a lot of .sql files from previous work with dBeaver.  I am finding that the older files will make funny errors when run, today, for instance
select * from pbm1712a, will say: ERROR: relation "pbm1712a" does not exist, in the older .sql file; but if I open up a new instance of the sql editor, the same command will "find" the file. I can't figure out why the below will not find pbm1712a, in the old sql file but will in a new instance:
set session my.vars.id = '033209365'; --
drop table if exists zzSet;
create table zzSet (mapn text);
insert into zzSet select trim(current_setting('my.vars.id')::text);
select * from pbm1712a where apn ilike (select mapn from zzSet)
pbm1712a is listed in dBeaver's explorer.

Thanks, Steve

Could you please check whether a correct database and schema are selected in upper toolbar of SQL Editor for older scripts?

there are no updates on the issue for a long time.
If it is still actual for you - feel free to reopen the ticket with an additional info


cron service: fix reliance on etc.timezone

Motivation for this change
#28116
This does not fully address the issue, as cron will not restart when
the timezone is changed imperatively as it presumably needs to — see discussion in issue.
Things done
NOT TESTED

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Tested and works as intended. Thanks!


Do not emit metrics produced by a failed decision or in defers called during cleanup.

Currently any side effects produced in non replay mode are always emitted even in case of decision failure or calls during goroutine cleanups (defer).
Possible solution is to buffer metrics produced during decision and emit them upon decision completion and drop them on decision failure.
Also metrics can be discarded if they are emitted during the cleanup phase.

Ideally the same should apply to logging. It would require that logger is an interface.


Azure Data Factory event-based triggers break Azure Data Lake Store Gen2

Hi,
Azure Data Factory allows creating event-based triggers on Azure Data Lake Store Gen2. However, after publishing the pipeline with a BlobEventTrigger, I can no longer access the data lake store files and I get the below errors:

Microsoft Storage Explorer

This endpoint does not support BlobStorageEvents, SoftDelete or AutomaticSnapshot. Please disable these account features if you would like to use this endpoint. RequestId:98451409-a01f-008a-24f9-df1464000000 Time:2019-03-21T15:17

Azure Portal (Data Lake Storage -> File systems)

FileSystemsBlade  MICROSOFT_AZURE_DATALAKESTORAGEGEN2

I managed to get access back to Azure Data Lake Store Gen2 only after deactivating the trigger in Azure Data Factory.
Used documentation:
https://docs.microsoft.com/en-us/azure/data-factory/how-to-create-event-trigger#

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

ID: b7ee5166-ff40-f4ed-ce07-5312f935e388
Version Independent ID: 68f01b43-e3b2-003b-24cb-446a08aa0c1b
Content: Create event-based triggers in Azure Data Factory
Content Source: articles/data-factory/how-to-create-event-trigger.md
Service: data-factory
GitHub Login: @sharonlo101
Microsoft Alias: shlo

@valdasm Thanks for the feedback. We are investigating into the issue and will update you shortly.

We have assigned the issue to the content author @sharonlo101 . They will evaluate & update the document as appropriate

Hi,
Today, I created event-based triggers on Azure Data Lake Store Gen2.
After publishing the pipeline with the event-based trigger, I could no longer access the files in the data lake store and I received the same error as above in Microsoft Storage Explorer:
This endpoint does not support BlobStorageEvents, SoftDelete or AutomaticSnapshot. Please disable these account features if you would like to use this endpoint.

Hi,
I am having the exact same issue as skiran11, is it a bug or is it a current limitation? The requirement from the client is to have event-based trigger when a file arrives in a CDM folder in ADLS Gen 2 and copy the file to another location inside ADLS Gen 2.

We are experiencing the same issue after enabled events for use in another container in the same storage account.

I am having the same exact issue. My pipeline works fine until I add an event based trigger. I receive the same error message.
This endpoint does not support BlobStorageEvents; SoftDelete or AutomaticSnapshot. Please disable these account features if you would like to use this endpoint.
Can anyone give me some insight as to how I can either fix this or if it's a limitation being worked on currently...

Getting the same error This endpoint does not support BlobStorageEvents; SoftDelete or AutomaticSnapshot. Please disable these account features if you would like to use this endpoint.

Same here. :(

I had same issue and then luckily i disabled the event based trigger and data lake containers were available again.
Is there an estimation when this feature would be available?

The solution for me, and I expect for all of us with this issue, is to delete any Event Grid Subscriptions associated with the storage account. The Event Subscriptions blade in the azure portal was not intuitive for me since it filters strictly based upon topic type, and doesn't have an "All Subscriptions" view.

Any ETA on a fix or workaround for this issue?

@djpmsft Can you please have a look at this one ?

Hi all, sorry everyone is experiencing this issue. Engineering will investigate today and hopefully we can resolve this or get a workaround shortly

Hi all,
After further investigation, this was an issue on the storage team that has since been fixed. When ADLS Gen 2 features were enabled by the customer, the account was not compatible with eventing which would cause the portal to break. As I mentioned, this has been fixed so users should no longer be running into this.
If anyone runs into this again, please contact the storage team
-Daniel

@djpmsft Thanks for leaning in to clarify here.
@valdasm I will now proceed to close this issue. Feel free to still comment and tag me or @djpmsft  if you need any further clarification.

@djpmsft  I got this error when running ADF pipeline (mapping data flow to read from Blob and write to SQL). the storage account is not data lake enabled, it is simple gen v2. I have referred blob storage linked connection as source in data flow. can I have to disable soft delete feature to make the pipeline working! could you pls help. thanks.
Error: This endpoint does not support BlobStorageEvents or SoftDelete. Please disable these account features if you would like to use this endpoint.

@bharangit if you are still facing the problem, please open a new thread in Microsoft Q&A, and we will assist you there.


Rename bichard7 repo

@AntonyBishop It looks like the apply failed for this and some of our users have now been locked out of the repos: https://github.com/ministryofjustice/github-collaborators/runs/2858730868
Please could you re-run this?

Hi @bjpirt re-running. I suspect it may fail again as the error is due to MoJ have insufficient "seats" in our Github account. This is being resolved but all changes look to be failing for this reason at the moment.

@AntonyBishop Thanks for the update - do you know when this might be resolved?

Hi @bjpirt trying to track down an ETA at the moment.

Thanks Antony


feat: add plugin security for node

Details about the configuration:
https://github.com/namics/eslint-config-namics/blob/feature/security/documentation/security.md

Closing due to inactivity.
Too bad about all the work. Sorry


Button to clear completed downloads

Is your feature request related to a problem? Please describe.
Currently downloads have to be removed one by one
Describe the solution you'd like
A button to clear all completed downloads would be very helpful
Describe alternatives you've considered
No response
Additional context
The button should go around here

Currently downloads have to be removed one by one

Theoretically it's possible to use filter button, select "completed", press "ctrl+a" and then trash button to remove, but it asks for confirmation separately for each entry. I'm not sure if button to clear all is really needed if filtering is possible, but can add button to select all entries and make one confirmation dialog for all.

Yeah I saw that you can select all, then press delete and confirm one by one but in my opinion that's not very intuitive and I think the solution you are proposing is not ideal for the same reason. I don't know if you've used download managers but they always have a button to clear completed for ease of use and simplicity, some having a confirmation dialog when clicking and also game clients like Steam and others have buttons to clear finished downloads.
Steam example:

Ok, I did what both of us talked, only I wasn't sure what icon should be for button to remove completed downloads, for now added 🧹 icon.

Hi, I apologize if I'm blind but where are the buttons located? I'm not seeing them 😅

Well, to see button you need to load extension from source, cuz it's not yet released.

Oh my bad, I thought it was already available in the current release because I updated to a new version after this issue was closed


Utilizing caddy proxy for ssh forwarding (forwarding port 22)

docker run -v /var/run/docker.sock:/tmp/docker.sock:ro -v /data/.caddy:/root/.caddy --name caddy-proxy -p 80:80 -p 443:443 -p 22:22 -e CADDY_OPTIONS="--email<EMAIL_ADDRESS>-d blackglory/caddy-proxy:0.2.0
Running the above code generates the following error
docker: Error response from daemon: driver failed programming external connectivity on endpoint caddy-proxy (fd1e2ffc107fe17cde2f236ec9afe6464142bd8ab6f56e34fd398c2b808d914f): Error starting userland proxy: listen tcp <IP_ADDRESS>:22: bind: address already in use.
Is this configurable for ssh forwarding? I need to be able to ssh into a gitlab docker container

My experience is that you do not need to use port forwarding, you only need to publish your gitlab container 22 port directly by docker.


Flow - Double posting.

@ken-de-moor how can this be reproduced?

@EdwardKerckhof didn't really do anything special, I just noticed it happening.
I was testing many things on flow at the time, commenting on other posts, likes, sharing posts... I pretty much tried all the features. Then after scrolling through my feed I noticed all my recent posts were doubled.


Not showing all the outputs of a workflow?

https://view.commonwl.org/workflows/5857df62857aba000107db47
From https://github.com/mr-c/hive-cwl-examples

Ah, this is just picking a random .cwl file, which happens to be a CommandLineTool instead of the Workflow in the directory. Probably related to #16

That is correct, picking up one of the tools due to a mistake. Fixed when 8801806 is merged, after I have fully implemented step parsing


New client save Client name as Client ID and vice versa

New client save Client name as Client ID and vice versa
in the edit page you can change the name and id but in the new client page its not work correctly

I just tested this with the current release, and the name is saved as the name and the Id is saved as the ID. Do you still have the issue with the latest repo?


Emit event when connection has been established

Hi there,
First off, thank you so much for this library! I'm starting to play with it to integrate it in our code and it looks very much like what I've been looking for.
I think it'd be good if this lib emitted an event when the connection has been established, to complement the error event. I'd like to wrap my Keyv initialisation around something like this:
async function initKeyv(url: string): Promise<Keyv> {
  const keyv = new Keyv(url);
  return new Promise((resolve, reject) => {
    keyv.once('connect', () => resolve(keyv));
    keyv.once('error', reject);
  });
}

... so that I have have local try/catch blocks to handle connection errors, which is impossible to do if the only event emitted is error.
As a partial workaround, I'm having to do something like this:
import VError from 'verror';

function initKeyv(url: string): Keyv {
  const keyv = new Keyv(url);
  keyv.once('error', (err) => {throw new VError(err, `Keyv failed to connect to ${url}`)});
  return keyv;
}

... so that at least I have more context when the (unhandled) error is thrown. This is very important when troubleshooting, since some errors are very unhelpful and provide no context as to what exactly went wrong.
I know I could listen for the connect event in the underlying backend, but every backend is different and I'm planning to use Keyv precisely to write backend-agnostic code.

Alternatively, this library could provide an async function that returned a Keyv object upon successful connection, so that I could use it as follows:
let keyv : Keyv;
try {
  keyv = await Keyv.connect(url);
} catch (err) {
  throw new VError(err, `Keyv failed to connect to ${url}`)
}

@gnarea - thanks for sending this over. At this time we are not considering to do this as much of the current implementations would not be able to support this well unless we only did this at the main Keyv system as a single function that did the handling of the connection error.
I am happy to keep this open to see if more people want this, but might not do this as there is currently a way of handling this.

@gnarea - going to close this out right now as there have not been asks for this.


chore(deps): bump https://github.com/jenkins-x/jx-cli to 3.0.242

Update jenkins-x/jx-cli to 3.0.242
Command run was ./promote/main

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign ccojocar
You can assign the PR to them by writing /assign @ccojocar in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gke
65babe9c282a2c3222a073822cdf204dccae97bd
link
/test gke

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/test gke

/test kube

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gke
65babe9c282a2c3222a073822cdf204dccae97bd
link
/test gke

kube
65babe9c282a2c3222a073822cdf204dccae97bd
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


php::apache::config doesn't work

If I try to set a value for a commented setting in php.ini file using
    php::apache::config { 'max_input_vars = 4000':
        require => Class['apache']
    }

The output looks like
==> default: Error: /Stage[main]/Main/Node[default]/Php::Apache::Config[max_input_vars = 4000]/Php::Config[max_input_vars = 4000]/Php::Config::Dwim[max_input_vars = 4000]/Ini_setting[max_input_vars = 4000 in ]: Could not evaluate: can't convert nil into String

In php.ini file there is:
; How many GET/POST/COOKIE input variables may be accepted
; max_input_vars = 1000

Maybe there is a simple workaround for that.

Finally it is a duplicate of https://github.com/jippi/puppet-php/issues/78. Should be nice to create a new tag.


Fixed contact page hover issue in quick links and contact info section.

Fixed contact page hover issue in quick links and contact info section.

Issue #915 fixed. Created PR for the same.

there are conflicts. resolve. and navbar is changed. can you review your code again??

@mansiruhil13 now have a look.


Fix two situations when no feedback was provided

Description
These changes fix two issues where the grader failed to provide feedback for students.
How?
Output is now suppressed when importing and running the module containing unittests, since grading json is printed to stdout as well. This caused issues when the test author or students forgot print-commands meant for debugging in the code.

The students are provided feedback with an error message telling them not to use those.

You think it should be return testmethod(case, *args, **kwargs) instead of just testmethod(case, *args, **kwargs) even though the test methods never really return anything?


Update version of cloud-hypervisor

Thanks for the review @MostAwesomeDude . We should catch up - I am now back at work.


Issue #7424:  test/README update with jest snapshot flow

Update according to #7424

What kind of change does this PR introduce?

Updating test/Readme with jest snapshot approach
Removed unused test module

Did you add tests for your changes?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

spotted few misleading typos there. PR updated

Thanks


Remove node-gyp dependency?

At the moment the package size of prebuild is quite large.
This is mainly caused by including node-gyp and node-ninja as a direct dependency.
Generally it is possible to rebuild native modules by executing npm rebuild package-name with the correct arguments. Such a approach would use the bundled node-gyp of npm which would significantly reduce the install time of prebuild and modules using it.
Am I missing a major downside of purely relying on npm to rebuild native modules?

I guess the main reason behind it was that we relied on a certain version of node-gyp (iirc for installing headers) which would make it difficult for users of older versions of node. That said, it probably isn't a big issue any longer.

Thanks for the explanation 👍
Would you like to receive a PR for this or is it to dangerous to rely on the node-gyp version provided by npm?

It would be really nice if we could get rid of it reliably.
@mafintosh @juliangruber any thoughts on this?

It was this commit from Juli 2015 that added node-gyp as a dependency, which was day one of this repository. I also can't find an issue talking about this, so this is probably good timing to revisit this decision. @mafintosh what do you think?

Great! I'll do some quick testing to see if my approach works.

@lgeiger Maybe we could still have it as a devDependency?

Maybe we could still have it as a devDependency?

What would be the improvement in this case?

Ignore me :)


[Release-1.31] - Bump containerd to v1.7.23

Backport fix for Bump containerd to v1.7.23

#11239
https://github.com/k3s-io/k3s/issues/11191

Validated on release-1.31 with commit https://github.com/k3s-io/k3s/commit/c946875f318ac49a407c71a7bad961705538e4e9

Single node

[2024-11-07T20:29:18.666Z] k3s version v1.31.2+k3s-c946875f (c946875f)
[2024-11-07T20:29:18.666Z] go version go1.22.8

[2024-11-07T20:29:22.878Z] Command:
[2024-11-07T20:29:22.878Z] kubectl get node -o yaml  --kubeconfig=/tmp/mdk3s131vb_kubeconfig  | grep containerd -A1
[2024-11-07T20:29:22.878Z] ---------------------
[2024-11-07T20:29:22.878Z] Assertion:
[2024-11-07T20:29:22.878Z] 1.7.23
[2024-11-07T20:29:22.878Z] ----------------------
[2024-11-07T20:29:22.878Z] Matched with result:
[2024-11-07T20:29:22.878Z] containerRuntimeVersion: containerd://1.7.23-k3s1
[2024-11-07T20:29:22.878Z]       kernelVersion: 6.8.0-1012-aws


Implements cancelWith

This PR can be cleaned up a bit (i.e. dropping the license header that is no longer required) but is otherwise ready to merge. The question here is whether this code should be added. I don't see any harm in doing so.

Cleaning up my open PR. I don't think this function ended up being needed as it's been sitting here for a while.


TabControl not Working

Hi,
My tab control doesn't work and keeps remain windows default! I used the installation instructions on the readme file. The Tab controls becomes default winform control and doesn't show up!

Hey xJoeyv,
I assume you ment it shows up as the WinForms TabControl in the designer and invisible during runtime?
You are probably missing the MaterialTabSelector with its BaseTabControl property set to the MaterialTabControl.
The MaterialSkinExample project has an example of it.

Aswell in the designer as when I debug the application it shows as a default winforms thing. I will check the example! Thanks.
Else any more help is highly appreciated


[v1.13] Add workaround for possible SetClusterProvider deadlock

fixes #29952
@aaronlehmann @aboch
Signed-off-by: Tonis Tiigi<EMAIL_ADDRESS>

Looks good to me

question: this fix reduces the probability of deadlock, but the deadlock can still happen, right?
If so, I think it is worth commenting.
Change itself is LGTM

@AkihiroSuda The deadlock should not happen through SetClusterProvider. If you look at #29952 then there is one another trace that doesn't use SetClusterProvider but we think this it is impossible to hit it with real-world usage. Still, this is a workaround, the issue is fixed with #28907

SGTM


Improve recipe import error handling.

When the recipe importer breaks, its been breaking hard. We nee to add more null/None checks to it and make sure it always returns 200 (never 500).

Fixed with #219


Add back EventSource

As noted in #4873, the changes in #4837 and #4724 caused a regression bug (a hang when opening certain projects) and were thus removed. The goal here is to put them back into the code base without causing that bug.

@davkean asked whether we should recycle an existing VS event category to avoid having to change all the infrastructure that currently captures events (RPS, VSFeedback, various scripts).

What would our alternative be?

In talking with @davkean, it sounds like we won't have to change other parts of the infrastructure as long as we change the "name" of MSBuildEventSource to Microsoft-VisualStudio-Common. Although we should make sure it still works with RPS after that, that should be a straightforward change.


[Issue] Account switching causes inconsistent screenshots

Just for reference, I use MultiMC as my launcher.
If you join a server, leave, then join on a different account, leave, then join back on the first account, then the screenshot it'll use for loading is the second account's screenshot.
Is it possible to have screenshots tied to an account or UUID? And when you join the server, it checks which account your on?

Due to how the current setup works, such is a byproduct of using the same folder. It could be reworked in the future to take in player-specific UUID if such is not difficult.


todayHighlight not showing

I've got the following initialising my datepicker:
$('.datepicker input').datepicker({
      todayHighlight: true,
      format: 'dd-mm-yyyy'
});

When I click in the input filed however todays date isn't highlighted. The 'today' class isn't being applied to the date in the picker.
There are no errors showing in the console

I can't reproduce this issue with the latest release (1.3.1), see example. Closing this for now, please comment/reopen with a jsfiddle example if this still is an issue for you with the latest release!


UserManagement

Please implement the whole logic for CRUD of a user, as well es the login,signup and signout logic. So the only thing not included is the view, the rest should be part of this ticket.
@mehrb98 mentioned it was made by the Identity package, figure out how or if you have todo it.

@mehrb98 please estimate you time. Thx

@mehrb98 please plan the tasks which are related with that, write them down and put a estimated enddate in the ticket.

@Maxiboy441  I didn't feel well, so I couldn't do it. I'm currently working on it

@mehrb98 ok, no problem. Wellbeeing goes first. If you feel you need support or that you cant handle that task, feels free to contact @feliperamoss or me.
Thx,
Maximilian Huber

@mehrb98 please estimate you time. Thx

!

@Maxiboy441 as discussed, I still have to deal with database and then I can be sure and commit.

@mehrb98 Thx for the estimation 👍
Im exited for the review :)

@mehrb98 perfect. Ill start the review asap

Hey @mehrb98 ,
I did a first review and added the comments in the PR. Please adjust accordingly:
Thx :)

Check https://github.com/Maxiboy441/PixelPrime/pull/39 and open a new PR when you ready. Dont forget to merge develop in to your feature, @feliperamoss allready did some thing that fix stuff of you, and resolve conflicts. Thx

Sadly, we have to many issues with the package and the docker setup, so we have to figure out a diffrent way of doing that


Remove GPG Prompt From Code

The code here needs to be removed.  A package should not prompt the cli from input especially if it could be used within the daemon code. It's best to just remove the prompt support and force users to specificity the keys in the request.
https://github.com/containerd/containerd/blob/master/pkg/encryption/keywrap/pgp/keywrapper_gpg.go#L107

cc @lumjjb

This also doesn't seem to make use of an agent.

SG, let me look into this.

This also doesn't seem to make use of an agent.

@dmcgowan Sorry, not sure what agent is in this context.. Is it relevant if we remove the prompt entirely?

FYI: This prompt is not a prompt that appears on the command line but a prompt that automatically answers the query for a password via a function.

There's no agent involved in this case but it's the core GPG code from golang asking for a password to a key.

Is there a better way to do it?  Just set the keys on the entity list?

This is the code that's calling this callback: https://github.com/golang/crypto/blob/master/openpgp/read.go#L188

@stefanberger ok, i understand it now


hover effect on icons

adding hover effect on github icon in footer

i would like to solve this issue . can you assign this issue to me ?


mariadb: fix clang and darwin build

The contents of clang-isfinite.patch are no longer necessary after
upstream revision 031fa6d4258574379ca3c1cbe912d235e9581982, which
includes a similar change.
Building in c++11 mode removes support for the GCC typeof extension
in favor of the standardized __typeof__. This causes errors when
using the strmake_buf macro, which relies on typeof.

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I don't know if the .so / .dylib renaming here is correct. @Izorkin would you be able to check this part?

@thefloweringash seems to be right


Add test

It will be nice to have some tests. It's a bit hard to check everything after adding changes... Think mocha can be used.

@paulirish @samccone would you like to have this. I can play with it.

Yeah i'd be happy to have some proper unit tests.
also i'm comfortable with some refactors to make that happen easier.


allow no-limit at nested (paged) QTF and NestedLoop

we need to fetch as much as the parent relation needs if we have e.g.:

order by symbol with references to more than one table e.g. t1.count + t2.count
where clause with "join condition"

so we need to allow selecting with no-limit at all internally.

ack


README contains ===

Which part of the documentation needs improvement?
README.md
Describe the problem found in the documentation
There is a === in the readme

Suggested improvement
No response
Additional context
No response

There is a PR ongoing to improve the README, please add a comment instead.

@sylvaincom note that #483 might be included here as well.

Additional thing to do: the probabl logo at the bottom is hard to read when github is in dark mode; can we add a light background to the image?

I just noticed something else: the probabl logo at the bottom is hard to read when github is in dark mode; can we add a light background to the image?

Good catch!
Maybe we can go fancy ;)

Dark mode is too experimental for this to work reliably I think

replaced by the orange logo so looks good on both light and dark themes


Vickel: Watch ufabetdirects.com

Vickel requests the watch of the watch_keyword ufabetdirects\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ufabetdirects\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by VLAZ in Charcoal HQ


Feature implement dart environment constants

This uses multi option which seems to have been implemented but unused from way back in the Aqueduct days. The idea is to implement this https://dart.dev/guides/environment-declarations for executables. conduit serve doesn't need this right now as uncompiled runs have access to shell environments.
Will test this on my machine manually and then merge.

Tests have been added /packages/cli/tests/build_test.dart


update dependency

resolve https://github.com/open-feature/docs.openfeature.dev/issues/221

cc @beeme1mr

Hi @Arhell, thanks for the PR. Just a heads up, this repo uses Renovate to update dependencies automatically.


Create grant revamped flow complete

@jacobcreech This PR is currently in draft mode, waiting for @yashikajotwani12 to complete the UI change in the form suggested by you.


Refactor ColorManager

Improvements:

Don't handle 0 color rules as a special case (related to colors_loaded)
Make it explicit that there are always the same amount of foreground/background colors and vectors of attributes
Removed code duplication between ColorManager::set_pb_colors() and View::apply_colors()

Coverage increased (+0.1%) to 57.209% when pulling 066f47b6d206be1148d8ab738e7e474adb8ff2ca on dennisschagt:refactor/color-manager into e0e76c34bf3713a9e3c94d97c388120c31ad76b3 on newsboat:master.


Missing assets React Native Windows (UWP)

Version

react-native-router-flux v3.37.0
react-native v0.38.1

Expected behaviour
Since react-native-router-flux does not seem to be platform dependent, I was excited to include this in my UWP (Universal Windows Platform) project.
Actual behaviour
However, when running with react-native run-windows the app gives an error of missing assets.
When adding these assets manually (e.g. back.windows.png) it seems to work fine.
Are you planning to support react-native-router-flux for react-native-windows?
Or more specifically, will these assets be included in a future release?

HI @twvdb! I doubt anyone here has tested with UWP. Are you willing to submit a PR with the correct assets?

Tested this approach, it works. Aswell as adding Windows specific back button images, maybe there should also be some fallback ones for "other" platforms (MacOS, Linux etc).


🌱 Avoid gocritic warning in the tests

Extract the loop body in a new function so that defer statements run at
a more predictable time.
Signed-off-by: Dmitry Tantsur<EMAIL_ADDRESS>

/approve

/approve
Trivial refactoring, let's proceed


[Input] SDL LockPosition() prevents click events initially

Release Type: latest github as of now
Version: https://github.com/stride3d/stride/commit/03e2a4aa489434404f182db40bb7c7ed25d80f81
Platform(s): Windows SDL
Describe the bug
LockPosition() before receiving any mouse down event prevents them from firing unless you unlock, press a mouse button and lock again.
To Reproduce

Setup SDL as the windowing system yourGame.Run( GameContextFactory.NewGameContextSDL() )
Lock mouse on init Game.Input.Mouse.LockPosition( true );
Press buttons and see that they aren't caught in-game.

Expected behavior
Additional context
To work around this issue you can listen to the first mouse up event before locking the window:
if( _initMouseLock == false && Input.Mouse.ReleasedButtons.Count > 0 )
{
    Input.Mouse.LockPosition( true );
    _initMouseLock = true;
}

3rd party issue.


CSRF cookie?

Is there a way through configuration to have the POST made by the provider (Google for instance) containing a given cookie? I am trying to write an application doing client-side authentication with this library but it starts failing when I activate CSRF on the server side.

I am realizing that my description was a bit messed up. Basically, I would like to be able to intercept the calls made by this library to add a cookie in it, but I am not sure how to do that. It was possible with Satellizer as you can see here:
https://github.com/mohiva/play-silhouette-angular-seed/blob/f9bbed9d03ee5da8fafd4a692a67840945d9ac32/ui/app/scripts/app.js

Basically, I wrote this dirty code to have this working locally, in the class JwtHttp setHeaders method:
    // Add CSRF token for the Play CSRF filter
    const token = this._cookieService.get('PLAY_CSRF_TOKEN');
    if (token) {
        console.log('token ' + token);
        // Play looks for a token with the name Csrf-Token
        // https://www.playframework.com/documentation/2.6.x/ScalaCsrf
        obj.headers.set('Csrf-Token', token);
    }

If this could be configurable, it would be awesome!

JwtHttp was deprecated in the latest version. You could probably achieve what you want easily through an interceptor in HttpClient model (new in angular 4.3+) if you want to use it with this library though you will need to upgrade to angular 5 and use the version 8.0.0 (currently on beta) npm i<EMAIL_ADDRESS>I'm closing this for now, tell me if this is still an issue and I'll reopen it.

Sounds perfect :). Any target date for the release?

I'll wait a week. If no critical issues emerge by then I will publish it.

I am trying to migrate, is there any example up to date with version 8.0.0? I cannot easily migrate with the changelog :(. And any reason it's not officially released?

it is officially released. what exactly is the issue? how do you initialize the module?


Update Bank.scala

Projet to make Loan calculation more realist, adding a leg to future loan capacity calculation.

Updated to New version
--
I've Seen what what tempted using 20 instead or calculating but doing such can lead to humain error, should stay with calculation using term.


Shield Den Search Not Working Properly

Hi Lenny I've got issue for shield den search.
It's seem like it not working properly I've try to search on both sword and shield for sword it's work perfect but when I switch to shield I can't search event for toxtricity It's show only milcery.

How many badges do you have on your shield savegame? Because you can only pick mons that are available with your badge count


Fix N/A age on installed package card

Changes Proposed

Check List

[ ] The title of my pull request is a short description of the changes
[ ] This PR relates to some issue:
[ ] I have documented the changes made (if applicable)
[ ] I have covered the changes with unit tests

Codecov Report
Patch and project coverage have no change.

Comparison is base (7eaded1) 30.04% compared to head (5f70c64) 30.04%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@                Coverage Diff                 @@
##           helm-dashboard-v2     #426   +/-   ##
==================================================
  Coverage              30.04%   30.04%
==================================================
  Files                     10       10
  Lines                   1348     1348
==================================================
  Hits                     405      405
  Misses                   902      902
  Partials                  41       41

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Router now can emit unhandledPaths

A router will emit unhandled paths a couple different ways.  They are found here: https://github.com/Netflix/falcor/issues/615 and found #147

Hold off on merging until call is figured out.  There is this oddity where call also takes in suffixes and paths.  These could produce unhandledPaths.  Should routers emit unhandledPaths from that?  The resulting change to the dataSource chainer will be discussed on the master ticket: https://github.com/Netflix/falcor/issues/615

@sdesai


Add setting to hide NSFW posts

Please add setting to hide posts with NSFW tag

Is there another app that does this? AFAIK there's no way to do this on the main reddit site either, except what's in https://www.reddit.com/prefs/ which just turns the nsfw labels on/off.

I never use reddit viewers on PC before, but my android client do can do this...

A fix has been merged to hide NSFW posts. You will need to make sure that this option is unchecked in your reddit settings:

I am over eighteen years old and willing to view adult content (required to view some subreddits)


Enable module builds of llvm/clang by default

What's in this pull request?
Add an option to build-script-impl to enable building llvm/clang with modules. This supersedes PR 3096.
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please smoke test

This failed because utils/update-checkout on master-next was not new enough. I cherry-picked 0fa1226828a910d384cf70e2e9871383e8034d87 and 68661f9d1fe2c9d3cf8160c2906a50f9cedf539c. @swift-ci Please smoke test

Cmake goop unrelated to this PR is blowing up, but only on Linux -- macOS passes all tests :|. Guess I'll try it again with a different bot config: @swift-ci Please test Linux platform

OK. The full linux test suite passed too. It's still a problem that the smoke tests didn't pass. I'll try it again: @swift-ci Please smoke test Linux platform

@swift-ci Please smoke test linux platform

@shahmishal Do the smoke tests clear the build directory or are they incremental?
@vedantk  If they are incremental, I would not be surprised if this fails with random errors. Changing the CMake configuration without erasing the CMake cache is not supported.

@adrian-prantl Please read the documents in docs/ContinuousIntegration.md

It looks like the smoke tests are in fact incremental. Where does that leave us? Since the validation tests passed, is it OK to merge this in?

Master branch is locked. So we wait until Mishal gives the go ahead.

@gottesmm: Thanks for pointing me to the right document. According to the document test and smoke test are incremental, so we were getting lucky.
This pull request is for master-next, not for master.

@adrian-prantl llvm and clang are incremental, however swift is clean build.
Also, CI is ready to accept new PR testing and repository is unlocked.


Optimise DB queries after introduction of close_cfds table

Relevant after #1779 is merged.
🙃 I think it could be more efficient to use joins to load all the relevant data in one query, not sure it's worth it though. We could run some performance tests to see how much of a difference it makes.
Originally posted by @da-kami in https://github.com/itchysats/itchysats/pull/1779#discussion_r847982902

This project is unmaintained


docs(packager): clarify why arch/dir/platform can't be set in Packager config

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] The changes are appropriately documented (if applicable).
[x] The changes have sufficient test coverage (if applicable).
[x] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
I didn't understand why setting the arch property in electronPackagerConfig was not supported, so I chatted briefly with @malept and came to understand that doing so creates complications regarding making that information available to each generator whereas specifying it on the command line makes it globally available. I added a short note to the README.md file to summarize this since I think it might be helpful to other developers who might want to understand why this is the case.

Oops, sorry, I often forget that. Would you like me to redo this one?

Nope, I already made a bunch of changes :smile:

To be specific:

Reworded the commit
Adjusted wording of the reason why the option can't be used
Added similar wording to dir and platform