twint.get:User:'NoneType' object is not subscriptable

I get the following error popping up in my logs as a CRITICAL whenever I run a search:
twint.get:User:'NoneType' object is not subscriptable
I do search with a Username and it seems like everything runs correctly. It is just annoying that it is popping up everytime
        self.twitterSearch = twint.Config()
        self.twitterSearch.Username = 'GainzyBot'
        self.twitterSearch.Search = $BTC
        self.twitterSearch.Store_object = True
        self.twitterSearch.Hide_output = True

DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:Search
DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:run
DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:Twint:__init__
DEBUG|02/Jun/2020 17:38:23|datelock|351|140130891859776|twint.datelock:Set
DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:Twint:__init__:pandas_clean
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:RandomUserAgent
DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:Twint:main:username
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:User
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:get_connector
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:Request:Connector
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:Response
DEBUG|02/Jun/2020 17:38:23|user|351|140130891859776|twint.user:inf
CRITICAL|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:User:'NoneType' object is not subscriptable
DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:Twint:main:not-search+since+until
DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:Twint:main:twitter-search
DEBUG|02/Jun/2020 17:38:23|run|351|140130891859776|twint.run:Twint:Feed
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:RequestUrl
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:get_connector
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:RequestUrl:TwitterSearch
DEBUG|02/Jun/2020 17:38:23|url|351|140130891859776|twint.url:Search
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:Request:Connector
DEBUG|02/Jun/2020 17:38:23|get|351|140130891859776|twint.get:Response
DEBUG|02/Jun/2020 17:38:24|feed|351|140130891859776|twint.feed:Json
DEBUG|02/Jun/2020 17:38:24|run|351|140130891859776|twint.run:Twint:tweets:notLocation
DEBUG|02/Jun/2020 17:38:24|get|351|140130891859776|twint.get:Limit
DEBUG|02/Jun/2020 17:38:24|run|351|140130891859776|twint.run:Twint:main:no-more-tweets

Hi, yes same for me. The console show me this error, for every single petition .
raise ValueError("Cannot find twitter account with name = " + self.config.Username)
ValueError: Cannot find twitter account with name =
Where  is a existing user on Twitter.

#604  #756

#775

Patch merge, please update with pip3 install --user --upgrade git+https://github.com/twintproject/twint.git@origin/master#egg=twint
This issue seems to be a duplicate, closing. If not, please feel free to re-open and describe why it's not

Patch merge, please update with pip3 install --user --upgrade git+https://github.com/twintproject/twint.git@origin/master#egg=twint
This issue seems to be a duplicate, closing. If not, please feel free to re-open and describe why it's not

Thank you!

Still get this error even after updating the installation with pip3 install --user --upgrade git+https://github.com/twintproject/twint.git@origin/master#egg=twint


States Windows 10 OSes are supported,

Should these docs be updated?  I do not know if Windows 10 OSes are supported for WAC.

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

ID: e80c473e-b916-0047-64f6-3c4c0eed7df6
Version Independent ID: b70c774f-8962-0e48-8294-645c03530f40
Content: Windows Admin Center frequently asked questions
Content Source: WindowsServerDocs/manage/windows-admin-center/understand/faq.yml
GitHub Login: @nwashburn-ms
Microsoft Alias: niwashbu

Yes @1RedOne, Windows Admin Center still works with Windows 10 OS.

Can it monitor a Windows 10 OS?

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


How can I find a user's all likes?

Hello, I couldn't find an answer. How can I find the list a user's all likes?

The functionality seems no longer available.


Cannot use docker image

Hello,
First of all amazing work,
I have trouble using the docker image, since I'm facing a circular opencv import and not being able to compile with Ninja. Could you perhaps explain explicitly how you run things when you are inside docker?
BR

Got it to work through:
conda install cuda-nvcc -c nvidia
conda install -c nvidia cuda-toolkit

Can I add to this by asking what cuda version is recommended, is it 11.6?
I followed the steps including the one above and ended up with 12.4 I think and just want to confirm before I sort the cuda errors out

We used Cuda version 11.6 to test in our environment, please consider trying this version. Thank you all for showing your interest in our work.


Cannot download a file

Wave SDK Version, OS
0.22.0, Ubuntu 21.04, Firefox brower
Actual behavior
Cannot download a file from the server. The I click the link a file just opens in a new tab.
Expected behavior
To automatically download a file when a user clicks on the link.
Steps To Reproduce
from h2o_wave import main, app, Q, ui

@app('/downloads')
async def serve(q: Q):
    download_path, = await q.site.upload(['data/img.png'])
    q.page['download'] = ui.form_card(box='1 1 2 2', items = [
        ui.link(label='Download Results', path=download_path, download=True, button=True),
    ])

    await q.page.save()

Duplicate of #999


BridgeConnector not connected (spam console)

last version

VS2019, netcore 3.1, nodejs last version

Steps to Reproduce:

start app
view log console

Does not respond to Electron functions. {...} when a BridgeConnect appears, a link appears for 1-2 seconds, then signals through Electron. {...} do not arrive..

I tried it with .net core 3.1 and the Electron.NET 7.30.2. It worked wonderfully. Can you put an example application with the same misconduct on GitHub? So I can test it better.

@GregorBiswanger  thanks for reply.
i found that my problem just  on debugging  use visual studio 2019 16.4.0 ,
when i typed electronize start  in cmd.exe without visual studio ,that works fine.

Project 2GB, I 'd rather tell you how to imitate this mistake. I will try.
public async void ElectronBootstrap()
        {
            browserWindow = await Electron.WindowManager.CreateWindowAsync(new BrowserWindowOptions
            {
                Width = 1152,
                Height = 864,
//                Show = false
            });

            //            browserWindow.OnReadyToShow += () => browserWindow.Show();
            browserWindow.SetTitle("Electron.NET API Demos");
            BrowserWindow_OnShow();
        }

        private void BrowserWindow_OnShow()
        {
            Thread t = new Thread(ThreadProc);
            t.Start();
        }

        private void ThreadProc()
        {
            while (true)
            {
                Thread.Sleep(900);
//                Electron.IpcMain.Send(browserWindow, "some_js_Method", "testMessage");
            }
        }

later, upload api-demos copy controllers and views to project. And as a result almost immediately after these actions the connect flies and stops working everything connected with C #

Hi guys.. Gregor do you have any update on this issue? I have been also facing it for quite some time.
Only in my occasion it does not happen all the time. On my desktop pc that has high specs it happens rarely, but on my laptop it happens quite often.. I am in a dead end with this issue as it makes the application completeley unresponsive

@nurbat I tried your example and the connection remains with me without problems (10 minutes long). There is a bit of memory growing with your method in the long run. However, you should not send a message every 100ms by default. Otherwise you should use alternative communication. But I think you just wanted to demonstrate it as an example of how it should crash?!
I am currently also using the latest version of Electron (version 8) - this may solve the problem automatically. Electron 8 have changed something here at the IPC. I would say that the problem is solved with the latest Electron.NET version 8.31.1.

hey! @GregorBiswanger
I'm still encountering this bug with the result that my frontend doesnt react on events anymore after a reconnect.
My hypothesis is that this happens whenever there is a large cpu or ram load going on. Due to this the app-lifetime can vary from 1 minute to multiple days depending on what I'm doing with my pc.
The result however is that the user will have to manually end the app-task through the task manager since closing the app otherwise will leave the user with around 5 processes that dont close. (the app is unstartable due to that)
Like I said, it happens pretty randomly but my guess is resource loads due to other programs. There is one way to trigger this bug consistently though:

I make sure I log all events I send from my core to the frontend. (any other stream of logs will also work)
I make the app 'hang' by selecting a log forcing the program in an 'awaiting' state
I wait around 2 minutes
The I let unselect the console-logs to make the app continue
There is a high-chance (if you wait long enough; always) that the Bridge Reconnect shows up.
After this the app is left in a state where the frontend javascript calls still work, however the app doesnt update anymore.

In this broken state I can still manually refresh the page to make the app show the latest data, however events through listeners dont work anymore.
For some extra context I attached an url to a youtube video replicating this, I can provide you with some extra details or logs if you want.
My current environment:

Windows 10
ElectronNET API & CLI <IP_ADDRESS>
.NET Core 3.1

Let me know if you have any ideas/details,
Greeting,
Thom

I have the same iisue while using ipc process to call a function on front end which is run per second. I deployed my application to raspberry pi and after 5 or 10 minutes, the issue occurs.


Updating docs for applying patches to installer-aro

Which issue this PR addresses:
N/A
What this PR does / why we need it:
Adds more clarity to docs.

/azp run ci

/azp run e2e

/azp run e2e


Missing baseUrl when entering application at child route

I have a child router that comprises the steps for a wizard. When I enter the application from the default route of the parent router the canActivate callbacks in the wizard child routes all have baseUrl set correctly to the containing parent route.
If the user reloads a page here I want to redirect them to the first step of the child router and I'd like to make this not depend on the name of the route the wizard is attached to on the parent router.
Looking at the router on the navigationInstruction supplied in the canActivate callback of the child route models I saw that baseUrl was set correctly and any redirects within the wizard steps worked by only knowing sibling route names. There was no dependency on knowing the parent route. This is great and what I expect, but now for what doesn't work.
In the page reload situation the baseUrl on the router is an empty string which causes any redirects based on child route names to try and resolve using the parent router. I investigated the arguments to configureRouter for the child router and saw that the 4th argument had a route property that I could reconstruct the baseUrl from. I'm getting around the problem with this.router = route.replace('*childRoute', ''), but I'd like to not rely on something that seems to have a better option.
So is the missing baseUrl a bug or is there another way I should be looking to use router?

Sounds like #113. The base URL isn't set until the navigation is committed, so it can't be used correctly during activation. We need to figure out a good way to set or expose the base URL for the instruction being processed.


Need to upgrade to version 4.37.0 based on Facebook Developers email received

SDK Upgrade: iOS v4.37.0
{AppName} currently uses Facebook SDK for iOS v4.27.1, which relies on the recently deprecated Version 2.7 of the Graph API.
As a result, there is a risk that your app will no longer function if you don't upgrade to Facebook SDK for iOS v4.37.0.As an admin of this app, please upgrade to the latest SDK as soon as possible to ensure your app's continued performance and to take advantage of new features.

Any plans upgrading?

Please let us know if you have an ETA for the upgrade on the package.   we received the emails too and we have multiple apps at the risk of not working.

@xplatsolutions @vdhyaram please look at the repsponse to this issue -> https://github.com/xamarin/FacebookComponents/issues/89.  We are looking on updating the components to support the latest version of FB on  iOS and Android.
Regards


[meshcop] implement R bit of Security Policy TLV

Putting this PR on hold since it may cause Thread Test 5.8.4 to fail.  In particular, the test involves performing a MGMT_ACTIVE_SET.req to update the R flag.  However, the test cannot complete since the Leader immediately becomes an end device as a result.  I believe the test (and possibly spec) need to be updated to indicate that changing the R flag should be considered a breaking change and done via the Pending Operational Dataset.  We need to raise this discussion within the Thread Group.

This issue is being discussed in Thread Group JIRA SPEC-768.

Superseded by #9187


Erorr with current diffusers (0.22.0.dev0)

When running the example code, an error occurs:
...
result = pipeline(cond, num_inference_steps=75).images[0]  # it is Zero123PlusPipeline
Error coming up:
File ".../.local/lib/python3.8/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File ".../.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py", line 364, in call
encoder_hidden_states = self._encode_prompt(
File ".../.local/lib/python3.8/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py", line 249, in _encode_prompt
prompt_embeds = torch.cat([prompt_embeds_tuple[1], prompt_embeds_tuple[0]])
TypeError: expected Tensor as element 0 in argument 0, but got NoneType
I think this is due to evolved code in diffusers. However, the supported diffusers version (0.20) is sort of outdated. Diffusers is under heavy development and will andvance further. Thus, I kindly ask you for a hint how to fix the code (I could do on my own), or supply the fix.
Thank you

I am getting same error.

@drdsgvo try this command
!pip install --no-cache-dir torch==2.1.0 torchvision --index-url https://download.pytorch.org/whl/cu118
to change torch version

Thank you for reporting the issue. I think diffusers are making changes to make the methods public (_encode_prompts -> encode_prompts), so you may try that by now. The line with error actually has a comment saying “this is added for backward compatibility” but apparently it didn’t 🤣
Hopefully I will be able to fix it in 1-2 days.

Just update
        encoder_hidden_states = self._encode_prompt(
            prompt,
            self.device,
            num_images_per_prompt,
            False
        )

in https://github.com/SUDO-AI-3D/zero123plus/blob/818f325ac6c5aad8a32ccb60158c997f4920afdf/diffusers-support/pipeline.py#L364C1-L364C1 to
        encoder_hidden_states = self.encode_prompt(
            prompt,
            self.device,
            num_images_per_prompt,
            False
        )[0]

I can submit a PR for this @eliphatfs if you think needed, but that needs the version in requirements to be bumped to 0.22

Thank you. That worked!

Should have fixed in github, will update the hf pipeline soon after more testing.

TypeError: expected Tensor as element 0 in argument 0, but got NoneType

I still have the same issue, could we update the diffusers to fix this? My error is:
TypeError                                 Traceback (most recent call last)
[c:\test.ipynb Cell 1 line 2
     [21](vscode-notebook-cell:/test.ipynb#W0sZmlsZQ%3D%3D?line=20) cond = Image.open(requests.get("https://d.skis.ltd/nrp/sample-data/0_cond.png", stream=True).raw)
     [22](vscode-notebook-cell:/test.ipynb#W0sZmlsZQ%3D%3D?line=21) depth = Image.open(requests.get("https://d.skis.ltd/nrp/sample-data/0_depth.png", stream=True).raw)
---> [23](vscode-notebook-cell:test.ipynb#W0sZmlsZQ%3D%3D?line=22) result = pipeline(cond, depth_image=depth, num_inference_steps=100).images[0]
     [24](vscode-notebook-cell:/test.ipynb#W0sZmlsZQ%3D%3D?line=23) result.show()
     [25](vscode-notebook-cell:/test.ipynb#W0sZmlsZQ%3D%3D?line=24) result.save("output.png")

File [c:\Python\Python39\lib\site-packages\torch\utils\_contextlib.py:115](file:///C:Python/Python39/lib/site-packages/torch/utils/_contextlib.py:115), in context_decorator.<locals>.decorate_context(*args, **kwargs)
    [112](file:///Python/Python39/lib/site-packages/torch/utils/_contextlib.py:112) @functools.wraps(func)
    [113](file:///Python/Python39/lib/site-packages/torch/utils/_contextlib.py:113) def decorate_context(*args, **kwargs):
    [114](file:///Python/Python39/lib/site-packages/torch/utils/_contextlib.py:114)     with ctx_factory():
--> [115](file:///Python/Python39/lib/site-packages/torch/utils/_contextlib.py:115)         return func(*args, **kwargs)

File [~\.cache\huggingface\modules\diffusers_modules\local\sudo-ai--zero123plus-pipeline\5006f82f085a8a5e1131440c327cdfde09b9c41a\pipeline.py:364](https://file+.vscode-resource.vscode-cdn.net/c%3A/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:364), in Zero123PlusPipeline.__call__(self, image, prompt, num_images_per_prompt, guidance_scale, depth_image, output_type, width, height, num_inference_steps, return_dict, *args, **kwargs)
    [361](https://file+.vscode-resource.vscode-cdn.net/c%3A/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:361) global_embeds = encoded.image_embeds
    [362](https://file+.vscode-resource.vscode-cdn.net/c%3A/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:362) global_embeds = global_embeds.unsqueeze(-2)
--> [364](https://file+.vscode-resource.vscode-cdn.net/c%3A/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:364) encoder_hidden_states = self._encode_prompt(
    [365](https://file+.vscode-resource.vscode-cdn.net/c%3A/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:365)     prompt,
    [366](https://file+.vscode-resource.vscode-cdn.net/c%3A/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:366)     self.device,
    [367](https://file+.vscode-resource.vscode-cdn.net/c%3A/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:367)     num_images_per_prompt,
    [368](https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:368)     False
    [369](https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:369) )
    [370](https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/~/.cache/huggingface/modules/diffusers_modules/local/sudo-ai--zero123plus-pipeline/5006f82f085a8a5e1131440c327cdfde09b9c41a/pipeline.py:370) ramp = global_embeds.new_tensor(self.config.ramping_coefficients).unsqueeze(-1)
...
    [250](file:///Python/Python39/lib/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py:250) # concatenate for backwards comp
--> [251](file:///Python/Python39/lib/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py:251) prompt_embeds = torch.cat([prompt_embeds_tuple[1], prompt_embeds_tuple[0]])
    [253](file:///Python/Python39/lib/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion.py:253) return prompt_embeds

TypeError: expected Tensor as element 0 in argument 0, but got NoneType

I have updated the HF pipeline. Please comment if you still have issues.


Speed up test for --project --fix

PR checklist

[x] Addresses an existing issue: #3127
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

Overview of change:
We don't need to execute and fix that many rules to ensure everything works as expected. By removing whitespace from the test's tslint.json the execution time is reduced from 4.7s to 2.7s on my computer.
That's enough to not run into a timeout even if CI is really slow.
Fixes: #3127
Is there anything you'd like reviewers to focus on?
Rewriting the test to not spawn a new process only reduces the execution time by 0.2s - 0.5s
CHANGELOG.md entry:
[no-log]

Nice, good call.


Add tests

Child groups are not supported at the moments, so the tests are failing. Still the tests are working so I will merge this.


Add new article for Excel Events

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-events.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Sync with the main repo

Sync with the main repo up to the latest https://github.com/bitcoin/bitcoin/commit/ab42b2ebdbf61225e636e4c00068fd29b2790d41, which includes the recent changes in the CI.
There is no downloadable artifacts support for now. It will be done in a separated PR(s).
Guix builds:
< TBD >

@MarcoFalke
After the last push, the CI merge_base_script works in a wrong way:
bash -c "$PACKAGE_MANAGER_INSTALL git"
Hit:1 https://mirror.hetzner.com/ubuntu/packages lunar InRelease
Hit:2 https://mirror.hetzner.com/ubuntu/packages lunar-updates InRelease
Hit:3 https://mirror.hetzner.com/ubuntu/packages lunar-backports InRelease
Hit:4 https://mirror.hetzner.com/ubuntu/security lunar-security InRelease
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
git is already the newest version (1:2.39.2-1ubuntu1.1).
0 upgraded, 0 newly installed, 0 to remove and 6 not upgraded.
if [ "$CIRRUS_PR" = "" ]; then exit 0; fi
git fetch --depth=1 $CIRRUS_REPO_CLONE_URL "pull/${CIRRUS_PR}/merge"
From https://github.com/bitcoin-core/gui-qml
* branch            refs/pull/359/merge -> FETCH_HEAD
git checkout FETCH_HEAD
Previous HEAD position was 592da16 Adjust code according to changes in the main repository
HEAD is now at 6f6e8b4 Merge b1f441ff3c280b4a59ccd2c15658628877830068 into 67895f77f2da6471c5c3b67988fde7f932f77782
M	ci/test/00_setup_env_native_asan.sh

The HEAD must be at Merge 592da1615079d98afd81c074f0447122b718b125 into 67895f77f2da6471c5c3b67988fde7f932f77782, no?

@MarcoFalke
After the last push, the CI merge_base_script works in a wrong way:
bash -c "$PACKAGE_MANAGER_INSTALL git"
Hit:1 https://mirror.hetzner.com/ubuntu/packages lunar InRelease
Hit:2 https://mirror.hetzner.com/ubuntu/packages lunar-updates InRelease
Hit:3 https://mirror.hetzner.com/ubuntu/packages lunar-backports InRelease
Hit:4 https://mirror.hetzner.com/ubuntu/security lunar-security InRelease
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
git is already the newest version (1:2.39.2-1ubuntu1.1).
0 upgraded, 0 newly installed, 0 to remove and 6 not upgraded.
if [ "$CIRRUS_PR" = "" ]; then exit 0; fi
git fetch --depth=1 $CIRRUS_REPO_CLONE_URL "pull/${CIRRUS_PR}/merge"
From https://github.com/bitcoin-core/gui-qml
* branch            refs/pull/359/merge -> FETCH_HEAD
git checkout FETCH_HEAD
Previous HEAD position was 592da16 Adjust code according to changes in the main repository
HEAD is now at 6f6e8b4 Merge b1f441ff3c280b4a59ccd2c15658628877830068 into 67895f77f2da6471c5c3b67988fde7f932f77782
M	ci/test/00_setup_env_native_asan.sh

The HEAD must be at Merge 592da1615079d98afd81c074f0447122b718b125 into 67895f77f2da6471c5c3b67988fde7f932f77782, no?

UPD. Or is it just an intermittent GitHub issue?

UPD. Or is it just an intermittent GitHub issue?

Looks like that. Re-run fixes it.

Going to merge this PR to unlock migration from non-free Cirrus CI to free GitHub Actions.
Hoping someone will make post-merge review soon.

utACK 14ddf6169358526ee5f349da2a7102dcd5dcd232.
ACK merge commit b7cdd31be
Light code review:

b3261144c ci: Disable the ARM task temporarily
2719ce1ed Fix modernize-use-default-member-init clang-tidy warnings
8b025633e Adjust code according to changes in the main repository
81643a5ff ci, qml: Add required Qt Quick dev packages
dea9e513a build, qt: Enable OpenGL for Windows static builds
3d78b7a40 qml, build: Update PE_ALLOWED_LIBRARIES for Qt 5.15.2
16f77a59d net: Notify about connection number by type


Show Canceled card on card query

Why are we listing deleted cards in the first place? Where is this happening?

Generally, everything that is soft-deleted should not be returned on the API.
I would rather stop returning them, likely in
https://github.com/opencollective/opencollective-api/blob/main/server/graphql/v2/interface/Account.ts#L516


[TC-OPSTATE-2.2] Step 16 and 17 -   The PICS should be updated in the script based on VS

From chip-test-scripts created by Rajashreekalmane: CHIP-Specifications/chip-test-scripts#693
Description :
[TC-OPSTATE-2.2] Step 16 and 17 -  PICS should be updated in script has per VS
PFA Script screenshot :

PFA VS screenshot :

Test-Plan PR that updated PICS in these steps : https://github.com/CHIP-Specifications/chip-test-plans/commit/a0d33b87c34cc7851cb7254ffef12f280fe27bc5

@Rajashreekalmane fixed in PR 29727 please verify it and close the ticket.

Issue resolved in latest Image.


Include imagemin dependencies

In Node7 gulp-imagemin throws gulp-imagemin: Couldn't load default plugin warnings when running the build script. The plugins need to be explicitly included in package.json to resolve this issue.
Additions

Adds imagemin-gifsicle, imagemin-jpegtran, imagemin-optipng, and imagemin-svgo.

Changes

Updates image gulp task syntax to ES6.

Testing

./frontend.sh should build.

Notes

It isn't necessary to require and include the plugins in image.js as they will be sourced by gulp-imagemin by default, but I included them so that it's clearer where they are actually used in the codebase when referencing them in package.json.

looks good overall, there is an linter error though. Thanks for the config change.

Please re-pull and test. Thanks!


[GH-22] Add do_token to environment for GHA

Overview
Add do_token to environment for GHA
related #22

/test


Missing double letter marked as correct when should be incorrect

If the correct answer has two letters in a row that are the same, and I only type one, I should be marked as incorrect. Without this extension I am marked as incorrect. But with the extension it marks me as correct, but only sometimes. Same with the other way around.
Here I omitted a t with the extension and it shows as correct:

Here I omitted a t without the extension and it shows as incorrect (expected behavior)

And in reverse, here this should mark me as wrong for adding an extra l:

That was on  Android. On Linux (Ubuntu) I see the same behavior.
With the extension:

Without:

Although if I am missing a letter, but everything is the right case, I do get the expected behavior.

If the letter in the wrong case is not adjacent to the double letters, I also get the expected behavior:

Desktop (please complete the following information):
Anki 2.1.61 (0c1eaf4c) Python 3.9.15 Qt 6.4.2 PyQt 6.4.0
Platform: Linux-5.19.0-41-generic-x86_64-with-glibc2.36
Flags: frz=True ao=False sv=2
Add-ons, last update check: 2023-05-12 08:33:30
Since this happens on AndiDroid too, I suspect it's not related to this specific version of Anki.
Note that for security reasons I used the manual no-internet approach. So instead of installing the add-on, I just pasted the code, un-minified, into my cards.
e.g.
{{Image}}

<hr id=answer>

{{type:French}}

<!-- https://derdemystifier.github.io/AnkiIgnoreCase/ -->

<script>
(() => {
    "use strict";
    ! function() {
        const e = "code#typeans",
            t = e + ' > span[class^="type"]',
            o = e + ' br ~ span[class^="type"]';
        for (let e of document.querySelectorAll(t)) c(e);
        let r = Array.from(document.querySelectorAll(t)),
            n = Array.from(document.querySelectorAll(o)),
            s = r.filter((e => !n.includes(e)));
        if (i(s).toLowerCase() == i(n).toLowerCase()) {
            const t = document.querySelector(e);
            n.forEach((e => e.setAttribute("class", "typeGood"))), t.innerHTML = n.map((e => e.outerHTML)).join("")
        } else {
            for (var l = 0; l < (s.length < n.length ? s : n).length; l++) s[l].innerText.toLowerCase() == n[l].innerText.toLowerCase() && (s[l].setAttribute("class", "typeGood"), n[l].setAttribute("class", "typeGood"));
            const e = 3;
            for (; n.length;) {
                let t = [...s];
                for (; t.length >= e && n.length >= e;) {
                    const o = t.slice(0, e),
                        r = n.slice(0, e);
                    i(o).toLowerCase() == i(r).toLowerCase() ? (o.forEach((e => {
                        e.setAttribute("class", "typeGood")
                    })), r.forEach((e => {
                        e.setAttribute("class", "typeGood")
                    })), t.shift(), s = [...t], n.shift()) : t.shift()
                }
                n.shift()
            }
        }

        function c(e) {
            let t = e.innerText;
            for (let o = t.length - 1; o >= 0; o--) {
                let r = e.cloneNode(!0);
                r.innerHTML = t[o], e.parentNode.insertBefore(r, e.nextSibling)
            }
            e.remove()
        }

        function i(e) {
            return [...e].map((e => e.innerHTML)).join("")
        }
    }()
})();
</script>

Thanks for the nice formatting and images. I will see to this ASAP.

I'm using Anki 2.1.48 and it's working as expected on Desktop.

Anki apparently modified the comparison algorithm in version 2.1.58
I guess I will need to update my Anki version and fix it. I will try to do that in the next few days. It's always a pain to move to a newer version as many addons stop working well.

Done! You can find the new script at ignoreCase.min.js.

Side note: July in French is Juillet not Julliet :)
The script is a bit longer after the fix and that's because I'm now using a library dedicated for diff comparisons.
You can always use the addon to alleviate the pain of copy/paste and to keep your card templates cleaner. (I still haven't updated the addon though).


[Feature] Enable notification only when AFK

Similar to the existing "only fire notification while the game is in the background" setting, it'd be super useful if you could conditionally enable a notification if you've been AFK for X seconds. The measurement of "AFK" would be: Math.min(client.getKeyboardIdleTicks(), client.getMouseIdleTicks()).
The use-case would be similar to the "only fire in background", some notifications are only needed to alert you when you aren't already paying attention. For example, if you are in a boss fight, you may not want low prayer notifications since you probably already know, but yet if you are afking and stepped away from your computer, you definitely want to be notified that your prayer is about to run out. Yet in that case, the game may not be backgrounded, you are just AFK.

If this feature is accepted, here's my implementation proposal:

Next to the current "fire in background" toggle would be a new button for this new feature
When toggling, it'd expand the notification box to add a new row which contains an input box for you to enter a number 0-1500 seconds (0 to 25 minutes). So the toggle is a boolean/number hybrid, it's either off, or a number indicating the number of seconds you are idle before triggering. For simplicity, it can just always be a number where 0 indicates off.

@adamk33n3r, do you like this idea? I'm happy to add it if you like the design.

Ya I think it's good. Have a whack at it. I can see it being a little complicated, touching a lot of different parts. So it'll be a good learning experience!

I may fail or it may not be how you would have wanted it to be done. But hopefully it'll help get us started and with a few tweaks it'll be good! 😄

Ya that's what PR reviews are for! 😂

Okay got it. PR is out. I think it looks really good. Just need some new icons. I wanted to ask you if you had a vision for that. Given the UX you've currently done I assume you have more graphic design skills than me. :)


⚙ Add Prettier and ESLint config with pre-commit

⚙ Prettier and ESLint config with pre-commit

Great, thank you! :rocket: :raccoon:


#354 ADD configurable terragrunt version

Attempts to address proposed solution by @orbitz  in #354 via tenv.

Great, than you, I'm going to run it against out test suite to make sure it all works.

Thank you for your contribution!


Create basic tests

There should be some basic tests that the wrapper is performing as expected.
Should include a check that the python wrapper version matches the underlying library version.

This is done.


Add files via upload

Made a few important Changes to the html file

@BlackTimber-Labs it would be very helpful if you would review my changes and merge the commits.

Good work. I am merging your PR.
Do give us a star by hitting the star button of this repo!


AWS Go SDK up v1.42.28

Issue https://github.com/aws-controllers-k8s/community/issues/1283, if available:
Description of changes: update AWS Go SDK version to v.1.42.28 (to use in Crossplane AWS provider project)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

hey @yandooo , this PR looks good. But would you be able to add an e2e test as well for this new field?
If not, then I can do it as well, but later.
The e2e tests reside in test/e2e/tests directory. Having an e2e test will give us confidence to make the new release.

@vijtrip2 not sure why e2e is in python, but also it looks like not mocked but rather needs real AWS env to test it out. It's great but goes beyond the timeframe I can allocate to get it done. If you could give a hand it is appreciated.

@vijtrip2 not sure why e2e is in python, but also it looks like not mocked but rather needs real AWS env to test it out. It's great but goes beyond the timeframe I can allocate to get it done. If you could give a hand it is appreciated.

@vijtrip2 looking through the code changes, looks like just a single string field Spec.KubernetesNetworkConfig.IPFamily was added. I'd be OK pushing this through without an e2e test since it's a small, no-risk change.

looking through the code changes, looks like just a single string field Spec.KubernetesNetworkConfig.IPFamily was added. I'd be OK pushing this through without an e2e test since it's a small, no-risk change.

Sounds good, Jay! I will add an e2e test for this after the PR merges(since it creates an ipv6 cluster, there may be something hidden that we may need to handle), and then cut a new release very soon for eks-controller. @yandooo , thanks for your contribution and patience : )


Fix the color on table, toast, form-label, modal and card when set the data-bs-theme="dark" on the element itself and fix the theme switcher in some examples

Fix the color on table, toast, form-label, modal and card when set the  data-bs-theme="dark" on the element itself
and fix  the theme switcher in some examples
Description

Motivation & Context

Type of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring (non-breaking change)
[ ] Breaking change (fix or feature that would change existing functionality)

Checklist

[x] I have read the contributing guidelines
[ ] My code follows the code style of the project (using npm run lint)
[ ] My change introduces changes to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Live previews

https://deploy-preview-{your_pr_number}--twbs-bootstrap.netlify.app/

Related issues

This breaks light mode card variants:

https://twbs-bootstrap.netlify.app/docs/5.3/components/card/#card-styles
https://deploy-preview-38212--twbs-bootstrap.netlify.app/docs/5.3/components/card/#card-styles

I also don't see a problem in the before state for stuff like form labels and toasts. Do you have screenshots of intended before/after to better show what needs addressing and reviewing here?

ok i fixed the card
for Card
data-bs-theme="dark" on card itself
before

after

for Toasts
data-bs-theme="dark" on toast itself
before

after

dropdown example
dark dropdown without any other change to the light dropdown only the data-bs-theme="dark" on itself
this fixes the dark preview
before

after

the Theme Switcher does not work on some sample pages because there is no bootstrap.js included

https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/blog-rtl/
https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/blog/
https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/cover/
https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/grid/
https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/jumbotron/
https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/pricing/
https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/sign-in/
https://deploy-preview-38236--twbs-bootstrap.netlify.app/docs/5.3/examples/sticky-footer/

for Popover
data-bs-theme="dark" on Popover itself
before

after

for Table
data-bs-theme="dark" on Table itself
then you can set table-dark to deprecated like carousel-dark
before

after

for Labels and Floating labels
data-bs-theme="dark" on parent (form) or element itself
before

after

on accordion, alert, modal, pagination, listgroup, progress the data-bs-theme="dark" already works

fix for example pages in a new pull request #38512


[Bug]: secret on a other plugin object

NetBox Secrets plugin version
v1.7.5
NetBox version
v3.4.7
Steps to Reproduce
Configuration.py:
PLUGINS_CONFIG = {
'netbox_secrets': {
'apps': ['dcim.device', 'ipam.prefix', 'ipam.ipaddress', 'plugins.bpg.bgpsession']
Expected Behavior
I want to use secrets on a object created by the netbox-bgp plugin
Observed Behavior
502 bad gateway

You need to use like this: package_name.model_name


resolving issue #45

Description of Change
The this->number was re-initialized in case of negative numbers.
resolving #45
Checklist

[x] Added description of change
[x] PR title follows semantic
[x] Search previous suggestions before making a new one, as yours may be a duplicate.

Thanks for your contribution!
@yashsoni501  can you please add tests for this behaviour.

@ayaankhan98 the test for this already exists under bigINT/tests/operators/ostream_test.cpp
I manually checked for both the versions over this test. The test fails for the code before commit and passes after.

@yashsoni501 i know tests are there for ostream operator, what i mean is that this wrong behaviour is triggered due to error is ctor so there must be some issue with the ctor tests that is not considered while writing the tests for ctors. therefore the tests for ctor are weak kindly update those tests to cover all possible aspects.


Encode working patterns

Context
Roles must choose ONE of the following:

not possible to do 100% remotely (but could be partial)
100% remote is possible (but could be partial)

they must also describe attendance policy:
ANY of the following three, OR

2-3 days/WEEK in the office
2-3 days/MONTH in the office
No more than 1 day/MONTH

ONE of the following two

Full-time attendance required*
Remote-working only*

These have to be mapped across to Candidate preferences. Candidates can select ONLY ONE of the following:

I have NO LIMITATIONS / PREFER NOT COMMENT on my ability to commute (DEFAULT)
I need to work with my team/manager in-person FULL-TIME
I am only able to attend the office 2-3 DAYS PER WEEK
I am only able to attend the office up to 2-3 DAYS PER MONTH
I am able to attend the office ONLY OCCASIONALLY
I require an ENTIRELY REMOTE role as I am unable to attend the office

Role attendance criteria:
0 days in the office (there is no office, remote only)
No more than 1 day a month
No more than 2-3 days per month
No more than 2-3 days per week
Full time

No limitations -> Any
I require an ENTIRELY REMOTE role as I am unable to attend the office -> 0 days in the office (there is no office, remote only)
I am able to attend the office ONLY OCCASIONALLY -> No more than 1 day a month
I am only able to attend the office up to 2-3 DAYS PER MONTH -> No more than 2-3 days per month
I am only able to attend the office 2-3 DAYS PER WEEK -> No more than 2-3 days per week
I need to work with my team/manager in-person FULL-TIME -> Full time

This is being worked on in #24


Use snakefmt in pre-commit hook for snamemake files

See https://github.com/snakemake/snakefmt?tab=readme-ov-file#version-control-integration
Currently this seems happy with all the formatting in our snakemake files 👍
Tested by deliberately using alternative quotes and indentation in one *.smk file:
$ git commit -m "Test this is rejected" pyani_plus/workflows/snakemake_fastani.smk
check for added large files..............................................Passed
check for case conflicts.................................................Passed
check that executables have shebangs.................(no files to check)Skipped
check json...........................................(no files to check)Skipped
check for merge conflicts................................................Passed
check that scripts with shebangs are executable..........................Passed
check for broken symlinks............................(no files to check)Skipped
check yaml...........................................(no files to check)Skipped
debug statements (python)............................(no files to check)Skipped
detect destroyed symlinks................................................Passed
fix end of files.........................................................Passed
mixed line ending........................................................Passed
trim trailing whitespace.................................................Passed
ruff.................................................(no files to check)Skipped
ruff-format..........................................(no files to check)Skipped
snakefmt.................................................................Failed
- hook id: snakefmt
- files were modified by this hook

[INFO] Writing formatted content to pyani_plus/workflows/snakemake_fastani.smk
[INFO] All done 🎉

codespell................................................................Passed
mypy.................................................(no files to check)Skipped

This restored the file to the previous state. i.e. The autofixer worked nicely.

I recreated my example, deliberately doing odd indentation and alternative quotes:
$ git diff
diff --git a/pyani_plus/workflows/snakemake_dnadiff.smk b/pyani_plus/workflows/snakemake_dnadiff.smk
index 00122bb..026d3d8 100644
--- a/pyani_plus/workflows/snakemake_dnadiff.smk
+++ b/pyani_plus/workflows/snakemake_dnadiff.smk
@@ -1,14 +1,15 @@
 from pyani_plus.snakemake import snakemake_scheduler

-indir_files = snakemake_scheduler.check_input_stems(config["indir"])
+indir_files = snakemake_scheduler.check_input_stems(config['indir'])

-def get_genomeA(wildcards):
-    return indir_files[wildcards.genomeA]
+def get_genomeA(wildcards): return indir_files[wildcards.genomeA]

 def get_genomeB(wildcards):
-    return indir_files[wildcards.genomeB]
+  return indir_files[
+    wildcards.genomeB
+]

 # The delta rule runs nucmer
@@ -22,8 +23,8 @@ rule delta:
     output:
         "{outdir}/{genomeA}_vs_{genomeB}.delta",
     input:
-        genomeA=get_genomeA,
-        genomeB=get_genomeB,
+      genomeA=get_genomeA,
+      genomeB=get_genomeB,
     run:
         shell(
             "nucmer -p {wildcards.outdir}/{wildcards.genomeA}_vs_{wildcards.genomeB} --maxmatch {input.genomeA} {input.genomeB}"

Try the commit,
$ git commit -m "Should not accept this" pyani_plus/workflows/snakemake_dnadiff.smk
check for added large files..............................................Passed
check for case conflicts.................................................Passed
check that executables have shebangs.................(no files to check)Skipped
check json...........................................(no files to check)Skipped
check for merge conflicts................................................Passed
check that scripts with shebangs are executable..........................Passed
check for broken symlinks............................(no files to check)Skipped
check yaml...........................................(no files to check)Skipped
debug statements (python)............................(no files to check)Skipped
detect destroyed symlinks................................................Passed
fix end of files.........................................................Passed
mixed line ending........................................................Passed
trim trailing whitespace.................................................Passed
ruff.................................................(no files to check)Skipped
ruff-format..........................................(no files to check)Skipped
snakefmt.................................................................Failed
- hook id: snakefmt
- files were modified by this hook

[INFO] Writing formatted content to pyani_plus/workflows/snakemake_dnadiff.smk
[INFO] All done 🎉

codespell................................................................Passed
mypy.................................................(no files to check)Skipped

Commit blocked, fixer applied, now no changes pending:
$ git diff


Fix some warning identified in issue 697

re: github issue https://github.com/Unidata/netcdf-c/issues/697
From that issue, fix the identified errors in the following files:
libdap4/d4file.c
libdap4/d4read.c
ncgen/genc.c
ncgen/ncgen.l
ncgen/ncgen.y

This pull request introduces 1 alert - view on lgtm.com
new alerts:

1 for Duplicate include guard

Comment posted by lgtm.com

This pull request introduces 1 alert - view on lgtm.com
new alerts:

1 for Duplicate include guard

Comment posted by lgtm.com


Passes device list to Slint UI

Wait, I mean top-level module of the gui crate, not a separated crate.


feat: template custom userdata

Fixes #N/A
Description
Allows Custom User Data for amiFamily Custom to be templated. This basically allows labels and taints to be templated into the user data which is useful for custom deployments. For example I am using RKE1 (from Rancher) with Rancher on EC2 instances and bootstrap the ec2 nodes using custom data.
Example template (pseudo code, not full)
sudo docker run -d --privileged --restart=unless-stopped --net=host -v /etc/kubernetes:/etc/kubernetes -v /var/run:/var/run rancher/rancher-agent:v2.6.9 \
      --server https://rancher2 \
      --token token \
      --ca-checksum checksum \
      {{ with .Labels }}{{ range $k, $v := . }}--label {{ $k }}={{ $v }} {{ end }} \{{ end }}
      --worker

How was this change tested?
Deployment into clusters, struggled to get the test suite working with lack of docs (or couldn't locate it)
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ellistarn let me know. it would be nice to have.

@jonathan-innis to take the lead here


tree-sitter broken with neovim 0.10

This is the output in :checkhealth:
- ERROR php(injections): /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:252: Query error at 49:2. Invalid node type "string_content":
  (string_content) @injection.content
   ^

  php(injections) is concatenated from the following files:
  |    [OK]:"/home/cata/.local/share/lazy/lazy/nvim-treesitter/queries/php/injections.scm"
  | [ERROR]:"/home/cata/.local/share/lazy/lazy/php-enhanced-treesitter.nvim/queries/php/injections.scm", failed to load: /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:252: Query error at 3:2. Invalid node type "string_content":
  (string_content) @injection.content
   ^

I'm not sure how to fix this.

I think you should update the PHP parser.
You can do this with :TSInstall php.

I already did this, countless times. Even if I remove the ~/.local/share/lazy dir and reinstall everything, the error still occurs.

The string_content node was introduced in the php tree-sitter parser 1 month ago with this PR : https://github.com/tree-sitter/tree-sitter-php/pull/237
Maybe you can share your config ?

I will try to reproduce it using a fresh clone of kickstart.nvim.

Ok, I think I found something. The error occurs if I set the highlight = { ... }  entry in the table given as argument to require'nvim-treesitter.configs'.setup() function.

Do you still have this issue ?


SCB missing FPSCR, FPCAR, FPCCR

I see the CPACR in scb, but I do not see the Floating-point Status and Control Register (FPSCR), Floating-point Context Address Register (FPCAR), or Floating-point Context Control Register (FPCCR). AFAIK this is only an optional feature with Cortex-M4 and Cortex-M7 and should be included along with the CPACR behind the has_fpu cfg, but I could be mistaken. See the User Guide for more info.

Hmm, you'll find FPSCR, FPCAR, FPCCR behind the FPU proxy. Now, why is CPACR behind the SCB proxy and not the FPU proxy ... I'm not sure. It seems more consistent to have CPACR in FPU as well but that would be a breaking change.

As a result, the FPU struct would need a lot of reserved fields right?

That's not an issue. As long as one register doesn't show up in two register blocks / proxies we are fine.
@rust-embedded/cortex-m this needs a decision.

It would indeed be more consistent to have it with the FPU, I generally value consistency high and would argue to change in any case.
Comments from the rest of @rust-embedded/cortex-m ?

The ARMv7-M architecture reference lists CPACR with the rest of the SCB registers while the FP ones are 'additional SCB registers for the FP extension'; in theory you could have a cortex-m device with other coprocessors which were enabled using CPACR that were not the ARM FPU (not that I know of any such devices).

So logically I'm not sure it makes sense for CPACR to live in the FPU struct. It enables the FPU but only in the same sense that an RCC peripheral enables clocks to other peripherals; it isn't itself part of the FPU.

I propose closing this issue and leaving the registers as-is, with FPSCR, FPCAR, and FPCCR in the FPU peripheral and CPACR in the SCB. Any thoughts?

Agreed


net: shell: cm: Fix snprintf warnings on NEWLIBC

Building with NEWLIBC triggers warnings about
snprintf since stdio is no longer automatically
included by printk.h
This PR switches to using snprintk to avoid these
warnings.
Fixes #77330

I have manually tested, and I expect tests should pass with no issue

@WilliamGFish Does this fix resolve the issue for you?

@WilliamGFish Does this fix resolve the issue for you?

All works well with this amendment. Thank you


Make bot messages actionable

Add a drop down to each Botkube notification message with a list of relevant actions that:

are supported by Botkube
are applicable to the resource type
the bot has permissions to actually run
Always return enabled verbs for a given channel regardless the resource (this is what @mszostok already did)

Simplified implementation, you can hardcode some verbs for specific resources
Populate resources dynamically and attach describe if they have get
logs should be handled only for Pods
Usekubectl api-resources -owide for verbs (use client-go) if that's good idea
Create it as a generic package - sync up with #748 owner, as it will be used there as well
Can be worked on in parallel

Documentation
Basic telemetry: add eventDropdown origin for the executed command
Integration testing out of scope (as we use legacy Slack)
Support filtering from the flow 2 - https://www.figma.com/proto/UcaGQ1UiBziOQbNPmSwE0T/grep?page-id=0%3A1&node-id=56%3A443&viewport=163%2C220%2C0.08&scaling=scale-down&starting-point-node-id=56%3A443&show-proto-sidebar=1 - reuse the same modal from #788

Example:

This will make each notification (or most notifications) actionable and therefore will make the bot easier to use.
Out of scope
Stretch
If kubectl is not enabled, whenever we have the technical ability to do so, add a button to enable kubectl under each notification:

Filtering should not be via dropdown but inlined:

Update: I will hide the "Delete" option for actionable notifications for now. This doesn't impact the "Actionable messages" feature at all.
I discussed the edge case with @lpetkov (which happens when user explicitly enables the "delete" verb for given K8s resource). Unfortunately I ran out of time - sounds like a rare edge case not worth to invest in right now, as we have different priorities related to Kubecon.
I will create a follow-up task for that and we can see if people really want to delete K8s resources from notifications 🙂


[suggestion] Integrate raids to player land claims via FTBChunks

Mod Source:
https://www.curseforge.com/minecraft/mc-mods/ftb-chunks-forge
Suggestion:
Don’t get me wrong, this mod is fun and it’s exactly what I’ve been wanting in a mod-pack..however, I’d love to feel a bit more engaged and targeted.
What if there was FTBChunks integration?
For example, there would be a chance for a raid to occur that target players near land claims that destroy blocks. (toggleable of course)
What is FTBChunks?
Simply, it’s a mod that allows land claiming in modded minecraft!

Landclaims? That sounds like a very interesting idea. We can work with this. Consider it added to the planned featurelist.

has this been implemented yet?

Actually, not yet. But with reason!
We are working on a new dominion system. This system basically tracks which faction owns specific land. The intent is that this will have an FTBChunks compatibility, where a player's claims automatically give then dominion over the chunk. That should then come with all of the things that comes with it, including the potential of raids happening on any part of your dominion.

Sound great. Hopefully it will also work with Xaero's World Map / Open Parties and Claims on 1.19.2 too!


@ChatResponseBody should allow addressing

It's common to want to be able to reply privately to the person who raised the original message.  Currently, you can't do that with @ChatResponseBody, you can only set the template and mode.
Ideally it would have a further field:
respondIn:  ORIGINAL_CHAT, PRIVATE, ROOM
and then have another field where you can say which room.

related to #282

not needed I think.  We do this using WorkResponse etc.


Ashtounge Mystic - Illidari

Not benefitting from attack and defence ups; nor does it have an inventory
https://media.discordapp.net/attachments/1228766598295191613/1228766599172067418/Warcraft_III_13_04_2024_18_30_03.png?ex=662d3d12&is=661ac812&hm=40406db2ab95aef1551f57bad31b1dfeb2ed92a725989ea6fa387295718be50f&=&format=webp&quality=lossless&width=1609&height=905

I'll pick this up


Add RAII support for ink_mutex.

Sub task of #1997. Provide a reliable RAII class for holding the lock on a raw ink_mutex.

[approve ci]


Another thing. Shouldn't the project.yaml read like this for consistency:

Another thing.


Switch ca certs test fixtures to use .NET 6

Summary

.NET 5 goes out of support on May 08 2022.
Since .NET 6 will become the default version of the runtime in the buildpack, the text fixtures have been updated to use .NET 6.
Use Cases

Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See [docs](https://docs.github.com/en/github/managing-your-work-on-github/linking-a-pul
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Labeling patch since this only changes the tests.


Dynamic operations only pass valid params

When creating a Dynamic ElementOperation with streams sometimes you want to supply additional streams, which do not supply parameters to the operation. This ensures that Dynamic only passes along parameters that are accepted by the operation.

Looks fine and the change makes sense. I still don't quite get the motivation though:

When creating a Dynamic ElementOperation with streams sometimes you want to supply additional streams, which do not supply parameters to the operation.

What do you mean? That said, I see how this change would make using streams more robust....

By default an operation can supply streams classes which supply parameters for the operation, e.g. think RangeXY which supplies x_range and y_range for the datashader operation. The user can however supply additional streams, which don't supply operation parameters, so this let's you do this:
bounds = Bounds(bounds=None)
decimate(datapoints, streams=[bounds, RangeXY()])

Without getting a bunch of warnings about bounds being passed to the decimate operation.

Ok. Makes sense and the tests are passing. Merging.


Simplified setters for optional elements in cpacs_gen

Updated to newest version of cpacs_tigl_gen, which dropped const T& setters for elements of type optional.
I had a discussion with a colleague about this, and this change incurs a potential performance loss for setters of boost::optionalstd::string as 1 more copy is performed when calling the setter. However, a lot of overloads are eliminated by this change, which makes overriding them easier and shortens the general API.

I didn't find an implementation for wrapping a boost::optional with custom types yet. Since we wrapped only optionals of fundamtental types until now, the typemaps were quite straight forward.

@bernhardmgruber Do you know a way to wrap boost::optional with custom types and SWIG?
If we cannot find a way, it is better to re-enable also the boost::optional-free API functions.

@bernhardmgruber + @RlanderRISCSW I figured out now, how to wrap the boost optionals. If you are interested, you can have a look into my swig library https://github.com/DLR-SC/tigl/blob/cpacs_3/bindings/python_internal/boost_optional.i


chore: upgrade go-getter to v2

Added the fix to File here because some of the Windows tests were failing due to a race condition when caching.


Unable to configure server path

@Leonya Trying to start server at path like localhost:8080/chat, javascript implementation contains feature like var io = require('socket.io')(8080, {path: "/swift"}) . How to do this with netty-socketio?

@robymus @ntrp

This works for me fine, I just set the path like this io({path: '/socket/socket.io/'}) and use a reverse proxy.

This only changes the way how the handshake is done and the actual connection - sending message still has to be done on root path - which is not the behavior we want


update.sh: add oracle linux 7 kernels

Addresses issue #36

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

Thank you for your contribution. I'll run update.sh for this change in my testing environment, etc, and if it looks good I'll merge and update btfhub-archive.

Certainly! I ran the update.sh in my test environment again and noticed two things.

I missed a + in shortversion=$(echo "${version}" | sed -r 's:([0-9]+(\.[0-9]+){2}-[0-9]+(\.[0-9]){0,3}).*:\1:g') which should be shortversion=$(echo "${version}" | sed -r 's:([0-9]+(\.[0-9]+){2}-[0-9]+(\.[0-9]+){0,3}).*:\1:g') which led to numerous undesired versions being included.
There is an edge case I missed. The version check is in-clusive of the min and max versions, but needs to be ex-clusive of the "max" because they are the first to provide BTFs. I'll fix that by decrementing the right-most decimal of the "max" versions, which will work for this.

I'll push those changes shortly.


HDDS-5753. Split parts of misc suite

What changes were proposed in this pull request?
Extract some tests from acceptance (misc) into separate suites: acceptance (HA) and acceptance (MR).
https://issues.apache.org/jira/browse/HDDS-5753
How was this patch tested?
https://github.com/adoroszlai/hadoop-ozone/actions/runs/1242183045

+1 on this from me. This gets all the acceptance tests through in less than about 60 minutes, and my goal / hope is to get the build time down to close to 60 minutes, so this is a good step in that direction.

Thanks @errose28 and @sodonnel for the review.


s3TransferManager is uploading files with Content-Encoding metadata empty

Describe the bug
s3TransferManager is uploading files with Content-Encoding metadata empty, we are uploading some files to s3 with the crt Async client, and the metadata of the objects in s3 contains the Content-Encoding key with an empty value

We are uploading text files and we do not need this Content-Enconding
We tried setting it to null from the code, but it was still created with an empty value.
Expected behavior
Objects in s3 without the key Content-Encoding in metadata
Current behavior
Objects in s3 with the key Content-Encoding in metadata

Steps to Reproduce
Dependencies used in the project:
    implementation(platform("software.amazon.awssdk:bom:2.20.42"))
    implementation("software.amazon.awssdk:s3:2.20.42")
    implementation("software.amazon.awssdk:s3-transfer-manager:2.20.42")
    implementation("software.amazon.awssdk.crt:aws-crt:0.29.10"

Code where the s3Client and the s3TransferManager are being instantiated
    @Bean
    fun S3AsyncClient(s3Properties: S3Properties): S3AsyncClient =
        S3AsyncClient.crtBuilder()
            .region(Region.of(s3Properties.region))
            .targetThroughputInGbps(s3Properties.targetThroughputInGbps)
            .minimumPartSizeInBytes(s3Properties.minimumPartSizeInMegaBytes * MB)
            .build()

    @Bean
    fun S3TransferManager(s3AsyncClient: S3AsyncClient): S3TransferManager =
        S3TransferManager.builder()
            .s3Client(s3AsyncClient)
            .build()

Code that we are using to upload a directory to s3
override fun uploadDirectoryAsync(destinationFolder: String, directory: Path): CompletableFuture<URI> {
        val directoryUpload = s3TransferManager.uploadDirectory(
            UploadDirectoryRequest.builder()
                .source(directory)
                .bucket(s3Properties.recordingsBucketName)
                .s3Prefix(destinationFolder)
                .build()

        )
        val directoryUriParameters = mapOf("bucket" to s3Properties.recordingsBucketName, "path" to destinationFolder)
        return directoryUpload.completionFuture().thenApply { uriTemplate.expand(directoryUriParameters) }
    }

Possible Solution
No response
Context
No response
AWS Kotlin SDK version used
implementation(platform("software.amazon.awssdk:bom:2.20.42"))     implementation("software.amazon.awssdk:s3:2.20.42")     implementation("software.amazon.awssdk:s3-transfer-manager:2.20.42")     implementation("software.amazon.awssdk.crt:aws-crt:0.29.10")
Platform (JVM/JS/Native)
JVM
Operating System and version
Amazon linux 2 and running locally in MAC OS Sonoma 14.2.1

Thanks for the issue but you appear to be using the aws-sdk-java-v2 and not the AWS SDK for Kotlin.
Unfortunately I can't transfer this automatically to that repo for you because it is in a different GitHub organization. Please open an issue there.


mapper4 There seems to be a problem

I tried to run some roms that support mapper4 mmc3, but some roms are gray screen. I found a similar project on github, but I don't know how to improve jsnes with it.
rom parameter:MMC3 rom: 16 PRG bank(s), 32 CHR bank(s)
project address:NesJS mmc3 Mapper

是有这个问题，我在弄"加纳战机"的时候，没有声音没有图像。
谷歌翻译：
There is this problem. When I was working on the "Ghana fighter", there was no sound and no picture.

@ZhaoTonggang 兄弟，我看你实现了几个mapper，我对这块不太熟悉，你能帮忙看看mapper 115 吗，我发现115和4很像，我尝试用4来代替，可以出现声音，但声音有点慢。
谷歌翻译:
Brother, I see that you have implemented several mappers, I am not familiar with this piece, can you help me to see mapper 115, I found that 115 is very similar to 4, I tried to use 4 instead, the sound can appear, but the sound is a little bit slow.
/**
* Mapper 115
*
* @description http://wiki.nesdev.com/w/index.php/INES_Mapper_115
* @example
* @constructor
*/
Mappers[115] = function(nes) {
			this.nes = nes;
	};

Mappers[115].prototype = new Mappers[4]();

A few years later, but I found a possible fix for the mapper 4. Here is the PR https://github.com/bfirsh/jsnes/pull/452

I have a similar problem I can't find anything that runs downloaded roms on Chromebook except emulator js wich can't run everything.


New deployments depend on deprecated AUTH0_TOKEN for user management actions.

Observed behavior
In order to create or delete users, an Auth0 token to perform Management API actions is required. The current datatools-server setup uses an unexpiring Management API token, so it works. However, for new Auth0 tenants the code needs to be refactored to get such a token from the Auth0 Management API.
Expected behavior
Creating, updating or deleting users using Auth0 should work in new deployments.
Steps to reproduce the problem
Try to create, update or delete a user with a newly created tenant in datatools-server.
Any special notes on configuration used
Auth0 credentials for a new tenant are required to demonstrate the problem.
Version of datatools-server and datatools-ui if applicable (exact commit hash or branch name)
dev branch.

:tada: This issue has been resolved in version 3.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add github workflows

Try adding a GitHub workflow for auto commenting, on raising a issue,
or
performing test on pull request

Hi Pranav,
I would like to work towards implementing ESlint script in the code and a workflow to parse this through esLint to check for any errors in PR.

Hi Pranav,
I would like to work towards implementing ESlint script in the code and a workflow to parse this through esLint to check for any errors in PR.

Hi rachit any update further??

Hi Pranav,
I would like to work towards implementing ESlint script in the code and a workflow to parse this through esLint to check for any errors in PR.

Hi rachit any update further??

Hey, Sorry Pranav. I just got a bit busy. I'll push the PR today.

@Pranavnk15 , Sorry for the delay. I just created a PR for this. Could you please review and let me know if any changes are required?

I think I made some issue in fixing the conflict. Could you revert the merge if possible?

I think I made some issue in fixing the conflict. Could you revert the merge if possible?

I guess I have  reverted it please check

Yes, you created a PR. Could you please merge that one.

yep 1 sec

done

I guess there's some error with it, cause, as you can see the latest PR, its not functioning properly, while installing dependecies and checking

yup
What happened is that I forked your repo to work on it and while I was
working on it. There were another commits which changes package-lock.json.
I think it would be better if we take the code back to
https://github.com/Pranavnk15/Todo-List/commit/cc4fca335e4cfc899e88a70c04f94e56d643496e
and then I can add the feature again after syncing my fork.
On Mon, Oct 9, 2023 at 1:54 PM Pranav Khandagale @.***>
wrote:

I guess there's some error with it, cause, as you can see the latest PR,
its not functioning properly, while installing dependecies and checking
—
Reply to this email directly, view it on GitHub
https://github.com/Pranavnk15/Todo-List/issues/16#issuecomment-1753422685,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN7BZU2GK5E7KDO4ZMLZFK3X6Q24RAVCNFSM6AAAAAA5W7GVXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJTGQZDENRYGU
.
You are receiving this because you were assigned.Message ID:
@.***>

After you are done with this step. Could you reopen this issue and the PR.

so should I revert that commit too?

I think you're good.  just open a #24

ya done

Thanks for your contribution


Error when saving market model with Merit price as tariff

In Edit market model (so not inside my LES) I want to add Merit price as Tariff from supplier to producer.
This gives the error that " interactions value for tariff can't be blank".
Changing back to fixed price removes this error.
Sometimes when I change back again from fixed price to Merit price I can save the market model once.
However, when saving again the same error comes up.
Link to market model

cc @dennisschoenmakers

I have already fixed this locally and should have it on beta shortly. I believe this may be the cause for #1217.


datavolume tests: use Patch instead of Update

What this PR does
Before this PR: DV is updated
After this PR: DV is patched
jira-ticket: https://issues.redhat.com/browse/CNV-44579
Why we need it and why it was done in this way
Use Patch instead of Update whenever possible for shortening the tests and reducing potential conflicts.
Release note
NONE

/sig code-quality

/approve


Bugs & comments to GPy/GPy/kern/src/stationary.py

class ExpQuad(Stationary):
The class docstring reports the wrong equation. The reported equation is that for Matern52:
k(r) = \sigma^2 (1 + \sqrt{5} r + \\frac53 r^2) \exp(- \sqrt{5} r)
The correct equation would be:
k(r) = \sigma^2 \exp(-\\frac12 r^2)

class RatQuad(Stationary):
The kernel equation here is not consistent with the book "Gaussian Processes for Machine Learning, C. E. Rasmussen, C. K. I. Williams". In the book, the rational quadratic covariance function is given as equation (4.19). What is missing here is alpha (i.e. the power term) in the denominator inside the parentheses.
current implementation (with error):
docstring:
k(r) = \sigma^2 \\bigg( 1 + \\frac{r^2}{2} \\bigg)^{- \\alpha}

def K_of_r():
        r2 = np.square(r)
#         return self.variance*np.power(1. + r2/2., -self.power)
        return self.variance*np.exp(-self.power*np.log1p(r2/2.))

Note: the commented out equation matches the docstring, both are missing the alpha term in the denominator.
This is consistent with the book:
docstring:
k(r) = \sigma^2 \\bigg( 1 + \\frac{r^2}{2 \\alpha} \\bigg)^{- \\alpha}
def K_of_r():
        r2 = np.square(r)
#         return self.variance*np.power(1. + r2/(2.*self.power)., -self.power)
        return self.variance*np.exp(-self.power*np.log1p(r2/(2.*self.power)))

class Exponential(Stationary):
The class docstring is missing.

@lawrennd your PR fixed this, right?

Oh ... I believe so yes ... although if I recall the PR was triggered by adding a feature so the fix was a side-effect ... but I think this can be closed!


SchedulerName config

TL;DR
Add SchedulerName option that will propagate to PodSpec
I've updated code to use this flag in 2 places:

go/tasks/pluginmachinery/flytek8s/pod_helper.go (regular tasks)
go/tasks/plugins/k8s/sidecar/sidecar.go (sidecar tasks)

Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/lyft/flyte/issues/
Follow-up issue
NA
OR
https://github.com/lyft/flyte/issues/

PTAL @migueltol22 @EngHabu @surindersinghp
This is my first time contributing to Flyte code. Please LMK what should I do to run certain tests/checks. Thanks!

@vsbus for running unit tests you can run make test_unit

Tests were failing in the sidecar code. reverter the sidecar file

PTAL @EngHabu

looks good overall.


In resource table, unable to define COLLATE for columns

Provider Version
chanzuckerberg/snowflake 0.25.16
Terraform Version
Terraform 1.0.2
Describe the bug
When using a COLLATE declaration when defining a column, ID VARCHAR(16777216) COLLATE 'en-ci' in resource snowflake_table, using single quotes results in a conversion error when applying the plan to Snowflake after terraform successfully parses the syntax.
Using $$ as a quote identifier instead of ' results in terraform seeing a diff when comparing table columns that exist and not changing in the plan.
Changing the quote identifier to $$en-ci$$, results in a successful terraform apply, but when applying any change after the success results in a diff when comparing resources that have been applied and not changed.
Expected behavior
When defining a column that needs COLLATION, you have to use a ' or $$.  Single quotes result in terraform not able to apply, and double dollar results in terraform seeing a diff for columns that have $$ in Terraform but ' in Snowflake. (Snowflake converts to ' when applied, apparently).
Code samples and commands

Unsuccessful apply

resource snowflake_table dummytable {
  database = snowflake_database.odindummy.name
  schema = "STG"
  name = "DUMMYTABLE"

  column {
    name = "ID"
    type = "VARCHAR(16777216) COLLATE 'en-ci'"
  }
}

Results in
Error: error creating table DUMMYTABLE: 001003 (42000): SQL compilation error:
│ syntax error line 1 at position 77 unexpected '\''.
│ parse error line 1 at position 85 near '39'.
│
│   with snowflake_table.dummytable,
│   on main.tf line 97, in resource "snowflake_table" "dummytable":
│   97: resource snowflake_table dummytable {

Successfull apply

resource snowflake_table dummytable {
  database = snowflake_database.odindummy.name
  schema = "STG"
  name = "DUMMYTABLE"

  column {
    name = "ID"
    type = "VARCHAR(16777216) COLLATE $$en-ci$$"
  }
}

On next plan not changing dummytable or anything at all

Additional context
I am not sure if Snowflake allows changing how string identifiers are represented in SHOW TABLES or where that could happen in the provider module myself.

We are closing this issue as part of a cleanup described in announcement. If you believe that the issue is still valid in v0.89.0, please open a new ticket.


Add thumbp group to ingestion inventory

This fixes playbook runs for the ingestion inventory. The thumbp group was left out of https://github.com/dpla/automation/commit/efefe4b70d19e2f0d35fb89f85f1d82efb1c30bd , where it was only added to the development inventory.
This PR is to merge this change into a release-maintenance branch for version 2. The commit will be cherry-picked into develop. A release will be cut for v2.11.1 without it interfering with the pending release of version 3.

This fixes https://github.com/dpla/automation/issues/168

👍

Thanks, @moltude. If https://github.com/dpla/automation/pull/164 is going to be merged very soon, then I can omit the cherry-pick step.

@markbreedlove I was probably going to merge it today just wanted to touch base at checkin let me know how you'd like to proceed.


unbound.conf.sh does not match hosts

Describe the bug:
unbound.conf has 50% fewer lines than the hosts file it is generated from
Steps to reproduce:
generate unbound zone using the resource script provided.
Expected behavior:
hosts and unbound zone should be 1:1
Environment:

OS: I use arch BTW
Other details that you think may affect.

Additional context:
Add any other context about the problem here.

The Unbound configuration file has fewer domains because it is being generated with the --filter-subdomains option to filter out all subdomains of domains that are already blocked. This is because Unbound should already by default block any subdomains.


when resizing the screen, make sure checkbox and label are always on the same line

likely just gonna have to manually add div tags around each input + label combo and ensure formatting that way


New ZZ error estimator: SERIAL VERSION

The purpose of this PR is to provide MFEM with a flux reconstruction error estimator, η, which is both reliable and efficient for arbitrary polynomial orders p. By reliable, we mean that there exists a mesh-independent constant C such that e ≤ C η, where e denotes the true global error. By efficient, we mean that exists another mesh-independent constant C such that η ≤ C e.
The choice has been made to design this estimator using a local averaging technique first introduced by Zienkiewicz and Zhu in [1,2]. The original work proposes flux reconstruction by solving a least-squares problem which uses global polynomials to fit discrete flux values inside vertex patches. This approach does not generalize well to meshes with an arbitrary number of hanging nodes. Based in part on this drawback, we use a face-patch averaging technique proposed by Bartels and Carstensen [3].
[1] Zienkiewicz, O.C. & Zhu, J.Z. (1992). The superconvergent patch recovery and a posteriori error estimates. Part 1: The recovery technique. Int. J. Num. Meth. Engng. 33, 1331-1364.
[2] Zienkiewicz, O.C. & Zhu, J.Z. (1992). The superconvergent patch recovery and a posteriori error estimates. Part 2: Error estimates and adaptivity. Int. J. Num. Meth. Engng. 33, 1365-1382.
[3] Bartels, S., & Carstensen, C. (2002). Each averaging technique yields reliable a posteriori error control in FEM on unstructured grids. Part II: Higher order FEM. Math. Comput., 71(239), 971-994.
NOTES:

This PR is a stripped down version of  #2538
This PR was written foremost for the DRL4AMR LDRD, which currently only requires a serial implementation. A parallel implementation will come in another PR.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2815
@brendankeith
@mlstowell
@mlstowell + @dylan-copeland
02/10/22
⌛due 02/24/22
⌛due 03/03/22

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Please mention in CHANGELOG 😄

Please mention in CHANGELOG 😄

Didn't realize we were at that stage yet! This is moving fast 😃

Thanks @brendankeith! Please note @mlstowell that some changes in ex6 should not be merged. Also, I think the name for the new ZZ estimator is up for debate.

On @psocratis's suggestion, I added a command-line option to ex6 to use the LSZZ estimator

@mlstowell is there anything left for me to do before this can be merged?


[Bug]: First test

What happened?
It went really wrong. 💥
What operating system are you seeing the problem on?
Mac OS
What browsers are you seeing the problem on?
Firefox
What wallet extension are you using?
Metamask
Rules

[X] I agree to follow this project's rules

Ups, no it didn't. closing it. 😄


Someone has to write about Double in chapters/double.md

Fixed in https://github.com/crowdsourcebooks/theswiftbook/commit/b233b24ac2256a76bcfb50eb2d98306e44eee743


boost lane marker hovering

fixes #625
added DetermineHoveredLane() to DetermineHoveredElements(). Then I use this to find the correct lane marker.
declared HoveredLaneID in TrafficMangerTool and SubTool.
Additionally I took the opportunity and removed setters from SubTool.HoveredNodeID and SubTool.HoveredSegmentID. Such setters are unused, unnecessary and using them can cause bugs.

@aubergine10 OH no! why did you tested something I already had said it won't work? I was waiting for you guys to tell me if I should close down this review:

I am doing it now!

I liked the premise of the idea; in the places where it worked, it worked really well and made it easier to interact with the lane markers; sadly it just made other places much more difficult.


Fix metaclass conflict error when installing pydantic v2 and using langchain

/kind improvement
Fix metaclass conflict error when installing pydantic v2 and using langchain
thanks @yacchi

Changed to not install pydantic in the langchain test because it is installed as a dependency of langchain.
Changed to signoff commit.


add other part types to TimberAssembly

Sheeting material
3D Print facade
electrical

This has begun, I will close and we can consider specific types