Support ClusterIP type in services created by listener-operator

Description
Needed for https://github.com/stackabletech/listener-operator/issues/3

Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)
[ ] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

bors r+


Update subnet activation logic

This PR updates the subnet activation logic in State so that it aligns with that in the contract.

Unfortunately we didn't have the latest Solidity code here. I just updated to the last and final version here: https://github.com/consensus-shipyard/ipc-monorepo/pull/502/commits/6653ce44780adf9405e299582386908e43dba734.
Of course, that broke the build, so I'm fixing it now.


[feature] Refactor of ExtraTrees preview for eventual merge with Random Forest

Description
This generalizes the ExtraTrees code to allow for commonality between Random Forest and Extra Trees for the eventual move out of preview.  It now makes the dependency on scikit-learn on a class above the public API.
Changes proposed in this pull request:

scikit-learn ExtraForestRegressor inheritance to ForestRegressor inheritance
scikit-learn ExtraForestClassifier inheritance to ForestClassifier inheritance

/intelci: run

@KulikovNikita, @icfaust, where is green PreCommit job for last commit?

http://intel-ci.intel.com/ee4b2c8e-e60f-f127-9bc2-a4bf010d0e2e

It is expected that #1424 will be merged before. And also I don't understand why we are merging PR with red CI?

@icfaust @KulikovNikita Please, give more time to reviewers for the review. Whole the time this PR was marked as a draft

Description
This generalizes the ExtraTrees code to allow for commonality between Random Forest and Extra Trees for the eventual move out of preview. It now makes the dependency on scikit-learn on a class above the public API.
Changes proposed in this pull request:

scikit-learn ExtraForestRegressor inheritance to ForestRegressor inheritance
scikit-learn ExtraForestClassifier inheritance to ForestClassifier inheritance

This generalizes the ExtraTrees code to allow for commonality between Random Forest and Extra Trees for the eventual move out of preview.

this is not the reason why it is on preview. The reason is perf issues and disabled tests that should be resolved.

@samir-nasibli , if there are additional rules to how we review code for this codebase, it should be written down and documented. These changes follow through on what was presented to the architecture board for what RF/ET should look like out of preview.  I assumed everything was good to go with a positive code-owner review and CI check. The private CI issues were described as related to linalg issues, otherwise green, and the public CI passes all tests. The additional public nightly CI check on this codebase being red (issues on the codebase associated with anaconda) is not due to extratrees algorithms.
If this is not our procedure for PRs than please let me and others know. For example, if one code owner isn't sufficient, lets try and make it two for all PRs and have a discussion about it. Otherwise, it makes @KulikovNikita 's review meaningless.

@icfaust Thank you very much for your work, actually it looks very good to me.
I am deeply sorry for misunderstanding. My feedbacks was based only on https://github.com/intel/scikit-learn-intelex/pull/1425#issuecomment-1705631411 check. So they could be ignored.
About the PR rules/templates, as an option we can continue discussion here https://github.com/intel/scikit-learn-intelex/pull/1388. Feel free to share your thoughts


Create an issue template to track 3rd-party dependency IP issues

We should define an organization wide issue template that can be used to track IP issues created by the dash tool.
Information that should be contained:

Links to the GitLab issues
Info on 'ability to merge'. I.e. restricted but issue number, non rejected ...

Hi @SebastianBezold,
Would this not mean you have a GitLab and a GitHub issue?
Do we need more information that is allready on our page?
https://eclipse-tractusx.github.io/docs/oss/issues#eclipse-gitlab-ip-issue-tracker

Hi @FaGru3n,
yes we would have two tickets. The IP issues in Gitlab are created by dash and in the past I also mentioned them in the PR with new dependencies, that I was reviewing.

@SebastianBezold  Thanks for the input, updated my dragt PR as ready for review:
modified the information in the template a bit to make it clear what is information sharing and what is really needed like suggested:

Links to the GitLab issues
Info on 'ability to merge'. I.e. restricted but issue number, non rejected ...

maybe you can disscuss with me the wording for the explaination for each section

Hi @FaGru3n,
I don't think we need an issue template. Like mentioned, the links are added to a PR directly instead of a separate issue.
Will close this issue here. Was just an idea back then, but working with this more regularly brought me to the conclusion, that separate tasks would just be overhead


chore: upgrade nwaku to 0.33.0 and check interop tests

Problem
Old version of nwaku is used in our CI.
Solution
Use latest
Notes

Double check that interop tests with nwaku master are fine, should be the case as nwaku got released just recently.
relevan discussion - https://discord.com/channels/1110799176264056863/1290371816384106529/1290371818863198330

@danisharora099 feel free to move to in progress when you start working on it, I probably misunderstood that you were working on it


Update versions

Update buildToolsVersion, compileSdkVersion, targetSdkVersion and Gradle to latest versions

I signed it
Date: Mon, 7 Sep 2015 02:13:03 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: Re: [android-testing-templates] Update versions (#11)
Thanks for your pull request.  It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).
Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify.  Thanks.
If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please let us know the company's name.
—
Reply to this email directly or view it on GitHub.


Increase test timeout - fixes #2875

Increases timeout for test from 1 sec to 3 sec
Fixes #2875

A similar test is failing here: https://ci.appveyor.com/project/openrefine/openrefine/builds/33897681
So I will increase the timeout there as well.


added option param to support additional filepicker.io params

Filepicker.io API supports more than the limited filepicker_client API. Added options param so that users can decide to exploit additional parameters.

This PR has been open for over a year. Is anyone still maintaining this repo?

I apologize. This repo has been off my radar for a while.
Recently Filepicker changed their name to Filestack and introduced a new API that I may work on an integration with in the future. The new API looks to be mostly compatible with the old one, just a different set of URLs to use. However there may be some things I'll need to adjust related to "/convert" functionality.

No worries, I have a number of changes I'd like to make to this client for a project we're working on. Would you like some help with maintenance?

Thank you for the offer. I'll be fine for now. Please feel free to submit any changes you'd like to incorporate.


build: fix release

Coverage remained the same at 100.0% when pulling 959a56c360f274994a3e5004b9398b7720813b41 on fix-release into 7cef73198aeeceb34a675ad9ed9789a830e4c2c9 on main.

:tada: This PR is included in version 2.0.1 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


chore: switch Lekko SDK usage from server props to API route

Fixes: ####

Upgraded Lekko node server SDK: 0.0.9 -> 0.2.0
Moved Lekko config fetches from getServerSideProps (which caused delays on each page load, see #1840) to an API route

API route is at /api/internal/lekko and currently fetches all configs
Cleaned up to remove home CTA button configs which seem to not be used anymore

Added hook for querying new Lekko API route
Updated relevant components to use new hook

Test Plan

Tested locally

Deployment instructions
[Add any special deployment instructions here]

Thanks @DavidSGK !


Migrate hub.docker.com "drud" organization to "ddev"

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
We don't need the "drud" name anywhere, so time to migrate to it.
Describe your solution

Rename "drud" to "ddev" if that's possible and redirects work
Otherwise, create new "ddev" organization. We'll need to get them to

Give us open-source rate
Allow unlimited downloads on it, as they do with drud.

Describe alternatives
No response
Additional context
No response

Sent request to docker support.

Via docker support and a friendly previous owner of the "ddev" org, we now have that org, which will be great.
I have asked them to sponsor the open-source "ddev" org and also turn off the rate limiting that is default on hub.docker.com

I did the required OSS application for the ddev org on hub.docker.com


inflating BasicFieldLabelView error

android.view.InflateException: Binary XML file line #16: Binary XML file line #16: Error inflating class com.google.blockly.android.ui.fieldview.BasicFieldLabelView

@huangxiaojingCN  Do you have any context for this error? Label views are very numerous in the Blockly standard blocks, and we have not seen any such issues.

@AnmAtAnm
There is my error log

There isn't much detail in that log, but I'm guessing your application theme is not parented to BlocklyVerticalTheme.
From our getting started guide:

In the applications AppTheme, found in res/values/styles.xml, set the parent theme to BlocklyVerticalTheme.
<style name="AppTheme" parent="BlocklyVerticalTheme">
    <!-- Customize your theme here. -->
    ...
</style>

This parent style configures several defaults for the blocks and fields. You can find more details in the style's definition.


Error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.37.32822\bin\HostX86\x64\cl.exe' failed with exit code 2

Please provide us with the following information:

Getting below error for several component in requirement.txt.
**copying aiohttp.hash\hdrs.py.hash -> build\lib.win-amd64-cpython-312\aiohttp.hash
running build_ext
building 'aiohttp._websocket' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for aiohttp
Building wheel for frozenlist (pyproject.toml) ... error
error: subprocess-exited-with-error
× Building wheel for frozenlist (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [33 lines of output]
*********************
* Accelerated build *
***********************
After installation getting below error:
error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.37.32822\bin\HostX86\x64\cl.exe' failed with exit code 2
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for aiohttp
Building wheel for tiktoken (pyproject.toml) ... error
error: subprocess-exited-with-error
× Building wheel for tiktoken (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [38 lines of output]
This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

It looks like you are using Python 3.12 which is a very recent release and isn't supported by all our dependencies yet. Please install Python 3.11 instead or use GitHub Codespaces.
Marking this as a duplicate of https://github.com/Azure-Samples/azure-search-openai-demo/issues/752

upgrading pycocotools to 2.0.8 helped resolve this issue


Upgrade dependencies and fix Travis

See #81.

Thanks, ComFreek, have done merge - I will try cloning your version, and see what happens!
Thanks,
P.

My most recent commit to this repository should finally fix Travis and the tests.


Date parsing for 'slack' format year is off by one

While working on porting tests to Busted, I've spotted holes in almost every utility being tested. I've tried to plug these holes so that as a fuller range of possibilities is covered. Mostly they've worked fine, but I just bumped into one that looks like a bug.
The pl.Date utility allows parsing some input formats using a 'slack' parsing string (i.e. 1 for month instead of 01). This works for day and month as expected, which is what was being tested before. I tried to expand this to test 2 digit year handling and discovered the results shift by a year!
local Date = require("pl.Date")
local fmt = Date.Format('d/m/yy')
local str = '1/5/98'
print(fmt:parse(str))

2097-05-01

Really? Assuming this millennium is one thing (note this handling is quite inconsistent), but offsetting by one is even worse. But wait, this works:
local str = '1/5/01'

2001-05-01

The cutoff seems to be 35. Lower than than works, higher is off by one.

Just saw #285.


In iOS the image is goes black when i change the aspect ratio.

I have used this library in my project. If i change the aspect ratio of the the image view is goes black. It's not occurring for all image. It's occurring for particular size image.

The issue in minimumZoomScale. any idea how to fix this issue ?

any update ?


Menu icons - is there an option to enable / disable them

I noticed you added with #43 support for menu icons.
so far i haven't added any icon to my menu - which results in empty space. See the attached screenshot.

Is there an option to enable or disable the menu icons?
Additional question:
Have you ever tried using fontawesome as a source for electron menu icons?

The icon view is based on the new Microsoft Edge (Chromium), so you cannot remove the extra space left on the left side of the menu item. If you use a checkbox item, the icon is check.

It is not possible to add fontawesome icons, as the variable for icon that provides electron is used. In any case, I'll check it out and be reporting the progress right here.

It is not possible to add fontawesome icons, as the variable for icon that provides electron is used.

Thanks - so i was guessing right.
Having access to fontawesome would it make much easier to fill all those icons needed for a complex menu ... but i understand if that's not the project target.

In any case, I'll check it out and be reporting the progress right here.
So should i keep the issue open as you consider supporting it?

I haven't tried it yet but you should be able to use fontawesome icons directly using their SVGs.
See: https://github.com/FortAwesome/Font-Awesome/tree/master/svgs

Hi @janvorisek

I haven't tried it yet but you should be able to use fontawesome icons directly using their SVGs.

Seems like it doesnt work.
I did try with a classic image like in the example - that works
icon: __dirname + '/app/img/icon/icon.png',
Then i tried applying that to an fontawesome svg like that
icon: __dirname +<EMAIL_ADDRESS>This throws no error, but the entire submenu entry is missing
So while my file menu containes

Settings
Separator
Exit

it then only showed

Separator
Exit

@yafp That's strange. Can you check if SVGs are loading correctly in your app (webpack/rollup config)? It's still strange that the item is missing. I'd expect an error/empty image instead.

@yafp Converting the icons you want into PNGs and including them in your app may be possible too. Brutal but could be a temporary solution 😄

Oh, i have to correct myself.
Not only the 1 item is missing - instead it seems like my app is falling back to the electron default menu entries.

I have to correct myself as well. I investigated electron menu a bit and the icon accepts NativeImage which cant be SVG according to the electron docs.
My current idea is to load the SVGs in my app main.ts using webpack and manually converting each into a NativeImage which can then be supplied directly to the MenuItem item prop. I'll share some sample code if I am successful 😄. I am also curious for any simpler solutions/examples.

I have to correct myself as well. I investigated electron menu a bit and the icon accepts NativeImage which cant be SVG according to the electron docs.

Good catch, thanks @janvorisek
for the sake of completeness
While trying to use the .svg my error reporting using Sentry game me the following:
TypeError
Error processing argument at index 1, conversion failure from /home/foo/bar/app/img/address-book.svg

Array.forEach
Called from: electron/js2c/browser_init.js in Function../lib/browser/api/menu.js.Menu.buildFromTemplate
Array.forEach
Called from: electron/js2c/browser_init.js in Function../lib/browser/api/menu.js.Menu.buildFromTemplate
app:///main.js in createMenu at line 239:21

While i noticed no error output in the console.
No idea if it's possible to handle that better to prevent falling back to the default menu.

to sum it up:

Using FontAwesome is not possible
Menu Icons (introduced with #43) can't be disabled


Add push notifications support

The mobile app integration has support for push notifications. This allows this app to register as a target. To do so, we should set push_token and push_url during the mobile app registration from #5 (docs). We can use the same one as iOS.
@robbiet480 could you briefly explain how to get push_token and push_url?

For adding push notifications, you can try most affordable tool www.cronberry.com.

push_token comes straight out of the Firebase SDK. push_url should be set to https://mobile-apps.home-assistant.io/api/sendPushNotification. That is a generic endpoint which supports both iOS and Android (and web too). We can get @CedrickFlocon added to the https://github.com/home-assistant/mobile-apps-fcm-push repo (currently private so that people can't find possible bugs to allow bypassing the rate limits) and I am happy to spend some 1 on 1 time with them tuning the server side code to best support Android push notifications.

I put the very basic building blocks in at #96. As a side note, the https://github.com/home-assistant/mobile-apps-fcm-push repo is now public.

Is there any particular reason to not implement fcm directly into HA other than imposing a message limit on users?
Ariela and others do it and I'm not aware of any issues with that approach.
https://github.com/MCrissDev/HANotify

@MCrissDev did you encounter any issues with that approach?

@quthla We aren't interested in that path because it adds a lot of extra complexity. We'd have to distribute keys in code and have issues if we needed to rotate them. I'd also worry about hitting FCM rate limits.

Also AFAIK Ariela is moving towards the mobile_app notify system as well.

Okay, I just wondered because on the website it says "home automation that puts local control and privacy first."
Notifications should at least be end to end encrypted then to ensure the privacy HA promises.

@quthla It would arguably be worse to implement FCM directly in HA due to the key distribution issue. The keys would have full access to at least all of the Firebase APIs as Firebase doesn't support keys scoped just for specific APIs such as FCM. This would allow anyone to list all of the tokens ever registered on the apps and spam as well as possibly influence end users Home Assistants.
End to end encryption has been suggested and I would like to implement it, but even then we can't fully encrypt the payloads because FCM and APNS require being able to inspect the payloads for fields like badge or sound to be able to properly notify users.

@robbiet480

End to end encryption has been suggested and I would like to implement it, but even then we can't fully encrypt the payloads because FCM and APNS require being able to inspect the payloads for fields like badge or sound to be able to properly notify users.

At least for Android that is wrong because the app builds the notification instance itself.

@quthla No, that's correct for Android. Please read the documentation for sending notifications via FCM, specifically this and this.

@robbiet480 Please see this example to understand how FCM works on Android.
I think the data you mean is only required if the app does not implement its own FirebaseMessagingService.

That is true of the latter fields but the first link I shared contains things that the OS deals with such as priority, ttl and collapse_key. Those are things that can't be encrypted.

That is correct but those fields are not necessarily needed and should not contain any sensible information anyway.

priority is required for proper delivery to Apple Watch as of iOS 13 and watchOS 6
I've had multiple users request support for collapse_key on iOS and I'm sure they'll come on Android
As I said, iOS requires access to some fields like badge and sound. Firebase allows us to present a universal solution for sending notifications to both platforms. This allows us to do things like have one set of documentation and only have to deal with one set of syntax issues. I think it would be more detrimental to have to split the infrastructure into iOS and Android specific setups.

Again, I'm not opposed to end to end encryption, it just has tradeoffs and isn't a priority for me. We absolutely have a strong mandate for privacy and I want to respect that, but at the same time thus far I'm the only one who has put time and effort into building notification solutions on either platform as well as the server side and I just want to get things working as simply as possible, for now. If someone is willing to step forward with a solution on both the client side (most likely both platforms I might add) and server side to implement end to end encryption, then we would absolutely welcome that. For now, yes, people are putting their faith in us that we (the core Home Assistant team) aren't doing anything nefarious with their notifications.

I was only referring to Android. It's been quite a while since I worked with notifications on iOS.
Even with end to end encrypted notifications on Android you can still use Firebase. It just requires some slight modifications to the mobile_app integration and the lamba(?) code to forward the encrypted payload to Firebase.
And I'm not really worried that you could do anything with the notifications but there's also Amazon and Google. And when there's any easy technical solution to ensure privacy, then that should be went for.


error installing arch/v0.6

installing  branch arch/v0.6, getting error
pip install .
Processing /scratch1/eh22/bw/radical.entk
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-o7nsfvw9-build/setup.py", line 49
        def visit((prefix, strip, found), dirname, names):
                  ^
    SyntaxError: invalid syntax

Whats the Python and pip version being used here?  Thanks.

python 3.6.3 | packaged by conda-forge
pip 9.0.1            py36h6c6f9ce_4    anaconda

I'm afraid we do not yet support python-3, you'll need something like python-2.7.

ah, ok


Correct constant checking

Prior to this change I would be getting Constant CURLOPT_CONNECTTIMEOUT_MS already defined since PHP would try to re-define a constant.

@toniperic 👍 Nice, thanks!

@OndraM you're welcome.
Out of curiosity: when do you tag new (patch) versions?

@toniperic I'm just going to release 1.4.1 with this fix in a few moments :).

@OndraM sounds great. Thanks! 👍


chore(deps): update cf

#629  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite


Negative values for best_bid_tick and best_bid

I am loading some data into the simulator and printing some debug data and found some negative values or non-initialized values for best_bid_tick and best_bid. Ask is fine but bid is wrong. Not sure yet if is some data wrong.
I am using python here.
Here is the code:
while hbt.elapse(100_000):
    print(f'ts: {hbt.current_timestamp}, best_bid_tick: {hbt.best_bid_tick}, low_bid_tick: {hbt.low_bid_tick}, best_bid: {hbt.best_bid}, bid_depth: {hbt.bid_depth}')
    print(f'ts: {hbt.current_timestamp}, best_ask_tick: {hbt.best_ask_tick}, high_ask_tick: {hbt.high_ask_tick}, best_ask: {hbt.best_ask}, ask_depth: {hbt.ask_depth}')

And here is the result:
ts:<PHONE_NUMBER>747000, best_bid_tick: -9223372036854775807, low_bid_tick: 25104, best_bid: -4.611686018427388e+19, bid_depth: {27674: 1003.0, 26691: 28.0, 25960: 2.0, 25600: 20.0, ...}

ts:<PHONE_NUMBER>747000, best_ask_tick: 25104, high_ask_tick: 25402, best_ask: 125520.0, ask_depth: {25400: 76.0, 25385: 1.0, 25384: 2.0, 25380: 52.0, 25369: 64.0, ...}

When there is no best bid or best ask, it sets INVALID_MIN or INVALID_MAX, respectively. Refer to this.
If the initial snapshot is not provided at the beginning of the backtest or if the depth is cleared and has not yet been recovered by the snapshot or natural refresh, this situation can occur. If the data is valid, Elapsing until the best bid and ask have valid values resolves the issue.


Service Request by Nick Bove

[ ] Qualtrics Service Request Form was received from User
[ ] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed
[ ] Qualtrics Post-Service Request Survey was sent to User

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

Hi Katy,
Thanks for reaching out. I’m including Donrie and Jessica to
help provide further guidance. I’ve responded with my thoughts below. Also
please feel free to use our github account to keep our team in the loop with
any issues/updates to the IPA Tracker

https://einsteinmed.co1.qualtrics.com/jfe/form/SV_4TvKdfrpwnbgBrE
:

From<EMAIL_ADDRESS><EMAIL_ADDRESS>Sent: Tuesday, September 28, 2021 11:54 AM
To: Nicholas Bove<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: IPA Tracker Questions

CAUTION: This email comes from an external source; the   attachments and/or links may compromise our secure environment. Do not open   or click on suspicious emails. Please click on the “Phish Alert” button on   the top right of the Outlook dashboard to report any suspicious emails.
Nick,

We have some questions
regarding the IPA Tracker for Putnam.

Under OEND regarding A1 and
B2, we believe this should be changed from doses to kits?  Since there are
two doses in each kit, we will have to double the numbers if we use doses.

The correct language should be “# of naloxone
units distributed in communities”. I’ll update the language in the dropdown for
primary measures.

A5 is under MOUD and it
should be under OEND.  However, it is also redundant to A1 and the numbers
will be duplicate if reported under both?

I’ve moved A5 to OEND. Although the
strategies are similar, data would have to be reported under both strategies.

F1 - number of non-fatal
opioid overdose events.   We already report this every month, should we
report it here again?

Yes, please report it here as a measure for F1 on a monthly basis.

Can you please clarify if
“3.7 Number of community-based opioid overdose prevention education and
naloxone distribution (OEND) programs” refers to an organization that is a NY
OOPP or refers to an OEND training event?

Jessica, can you please clarify how this measure is defined?

Thank you,

Katy

Katherine J. Roberts, Ed.D., M.P.H., MCHES, CPH
Data Coordinator
Putnam HEALing Communities Study
917-817-0785

@donriepurcell we need to rapidly and proactively provide feedback to each county to address questions like these. All measures need proper units. And there shouldn’t be any measures that is not MONITORABLE, so to speak.

Hi Nick,
Thanks for getting back to us. I have a couple of follow up comments/concerns please see below.
I’ve moved A5 to OEND. Although the strategies are similar, data would have to be reported under both strategies.

This is confusing to us – if we report in both places we would be double reporting the number of naloxone units distributed in Putnam. Can you explain the rational for this?

Sure, not a problem. The IPA Tracker is built to be IPA specific. The data collected are generated into reports that can only be reported at the IPA level. The tracker doesn’t read across IPA’s and loop data together if the measure is similar. This was created in this way in order for counties to be able to report on measures that might be worded the same but might differ from partner to partner. In strategies A1 vs A5, they both have very similar measures but will need to be reported separately to align with the way the tracker was built. I hope this is helpful!
Best,
Nick


Move all stubbing to CFG side

We now have a MethodStubber that works on Jimple and a CfgStubber that works on CFG level. Fooling around with library methods is messy. Better to move all functionality of the MethodStubber into CfgStubber if possible.

This is indeed annoying. The reason why I did not merge them back then was as follows:
The CfgStubber should only replace a method by something that havocs the return value and do nothing if there is no return value.
The MethodStubber should deal with the problem that there are many constructors that do not have a body. If we don't add a body that sets the dynamicType, we get assume false all over the place. I.e., constructors have to be stubbed differently from other methods.
But, indeed, they could be merged. And I think once #47 is done, this will become trivial.


appliance_console should be aware when running in a container

the appliance_console cli is not fitted to a container,
basically it does all sorts of system administration operations and part of those are not relevant in a container, there for it needs to be container aware, and not to show the none relevant options.
The overview displayed initially
(taken from my env ... removed some details from my env )
Hostname:                43b6d5b76b4f
IP Address:              <IP_ADDRESS>
Netmask:                 <IP_ADDRESS>
Gateway:                 <IP_ADDRESS>
Primary DNS:             -- removed --
Secondary DNS:        -- removed ---
Search Order:            -- removed --
MAC Address:            -- removed --
Timezone:                America/New_York
Local Database:          running
CFME Database:           postgres @ localhost
Database/Region:         vmdb_production / 0
External Auth:           not configured
CFME Version:            <IP_ADDRESS>-beta2.6
CFME Console:            https://<IP_ADDRESS>
while displaying this data is o.k although some of the information is not relevant (e.g. the ip is internal),  changing some of them may be a risk (to the host) or impossible (as the appliance_console crashes when trying to do so). keep in mind that we are still running a --privileged container which might have permissions to mess the host the container is running on
The appliance_console menu

Set DHCP Network Configuration
Set Static Network Configuration
Test Network Configuration
Set Hostname
Set Timezone
Set Date and Time
Restore Database From Backup
Configure Database
Extend Temporary Storage
Configure External Authentication (httpd)
Generate Custom Encryption Key
Harden Appliance Using SCAP Configuration
Stop EVM Server Processes
Start EVM Server Processes
Restart Appliance
Shut Down Appliance
Summary Information
Quit

1, 2, 3 should not appear in a container - as network is controlled externally to the container by the infrastructure (and the IP addresses shown are internal)
4  should be hidden   - when changing it the appliance_console crashes (but succeeds changing the hostname for the container .. on next execution)
6 should be hidden - as it changes the host time as well
9 should be hidden - as it crashes the appliance_console , and parted is not installed at all (it is very risky to install it in a privileged container)
15 should be hidden - it only causes the container to stop and can't start it again (as this is controlled externally)
I'm not sure about #12

@bazulay @simaishi Agreed on options 1,2,3,4,6,9,15 to be hidden on the container appliance, I would like to test and ensure we have proper httpd mods for option 10.
I think it makes sense to include 12 , also have to check oscap-scanner is actually being pulled.

@bazulay @fbladilo Is there an easy way to detect whether we're running inside a container?

@gtanzillo  I've never found an official answer but I've seen these approaches:
Check existance of /.dockerenv or /.dockerinit files
Cgroup check "grep docker /proc/self/cgroup"

Probably a combination of them would be safer. I hope this helps.

@gtanzillo @fbladilo
we can simply add a config value (in container build time) to any chosen config file something like "containerized=true/false" and the appliance console should pick it up on runtime.
This should be good enough + it will ease up testing for the developer , he can simply add this config manually and test the console.
about which config file to use, Greg just let us know where you want it , and we'll make it happen

Would you be able to define an env variable at build time? Something like CONTAINERIZED_MIQ? That would be the easiest for us. But, if not, the file would work too.

@gtanzillo we already set one on the current build
https://github.com/ManageIQ/manageiq/blob/master/Dockerfile#L2
Please keep in mind that we add this ourselves, my comment above referred to other options that are automatically provided by docker (not set by the image author).

@gtanzillo @fbladilo
Franco keep me honest here, but IIRC  "ENV container docker" is only available on build time ?

@gtanzillo @bazulay Barak, ENV is available during build-time and persist in run-time. Different case for ARGs which are only available at build-time.

@gtanzillo @bazulay
As I suspected, option 9 "Configure External Authentication (httpd)" is missing packages, will create a small PR to pull them in and refer to this issue. Take a look below :
--
Configure External Authentication (httpd)
ipa-client RPM is not installed
sssd-dbus RPM is not installed
Appliance Installation is not valid for enabling External Authentication
External Authentication configuration failed!
Press any key to continue.

@fbladilo FYI, there are two dozens or so packages not included in Dockerfile, but included in kickstart.  We'll need to go through others and identify what else is needed..  https://github.com/ManageIQ/manageiq/issues/8561

@simaishi good call, I'll start a review and post in #8651 for discussion.

@gtanzillo @bazulay @kbrock I believe in the container environment detection options are :

Author supplied environmental variable (already in place)
Author created config file
Docker supplied .dockerenv or .dockerinit
Cgroup check for docker

I'm inclined for 1,2 because we can ensure they are there, 3 and 4 are in Docker area and subject to change (although I hardly see 4 changing) . Thoughts?

@fbladilo @kbrock @gtanzillo
I agree that either 1 or 2 are the best for this case.
kbrock - if you prefer #2 above than please let us know so we'll submit a PR to cover that (and which file do you prefer using for this purpose). in case you prefer  #1 than you are good to go.

I lean towards 1, but we do tend to have all env variables defined in /etc/defaults/evm
Not sure if that scheme is relevant in the Docker world.
Is the env lowercase "container" a standard out there for docker?
Kinda wish it were a little more bash env friendly like "DOCKER_CONTAINER" or something.
But I guess it is what it is.

I agree option 1, "Author supplied env variable (already in place)"
I'd also like to confirm what options should be exposed for now on a container.
We can change this as functionality changes, e.g. add 10 back once supported.
Do we want to allow the user to set the timezone?
1) Set DHCP Network Configuration
2) Set Static Network Configuration
3) Test Network Configuration
4) Set Hostname
5) Set Timezone
6) Set Date and Time
7) Restore Database From Backup
8) Configure Database
9) Extend Temporary Storage
10) Configure External Authentication (httpd)
11) Generate Custom Encryption Key
12) Harden Appliance Using SCAP Configuration
13) Stop EVM Server Processes
14) Start EVM Server Processes
15) Restart Appliance
16) Shut Down Appliance
17) Summary Information
18) Quit

@fbladilo @kbrock @Fryguy
We can do whatever you think fit (both functionality & project wise),
We can do boath - add DOCKER_CONTAINER as environment variable & to /etc/defaults/evm.
But if the console loads the /etc/defaults/evm anyway than it seem that the env variable is redundant.
 Think Timezone: no but @bazulay would be better reference for that

Timezone can stay as it does not effect the host the container is running on.

Please note:

Configure Database -> 1) Create Internal Database will cause appliance_console to crash (https://github.com/ManageIQ/manageiq/issues/8587).

So for the top menus we want to keep for the container, the sub-menus will need to be looked at.

@bazulay @kbrock @jvlcek
I'm seeing a slightly different list for appliance_console in manageiq:latest DH image, was your list from the CFME build?
Advanced Setting

Set DHCP Network Configuration
Set Static Network Configuration
Test Network Configuration
Set Hostname
Set Timezone
Set Date and Time
Restore Database From Backup
Configure Database
Configure External Authentication (httpd)
Update External Authentication Options
Generate Custom Encryption Key
Stop EVM Server Processes
Start EVM Server Processes
Restart Appliance
Shut Down Appliance
Summary Information
Quit

Choose the advanced setting:
Option "extend temporary storage" is not there but there is an extra "update external auth options".

@kbrock @bazulay @jvlcek
I like the idea of having all MIQ env variables contained in /etc/default/evm, instead of using
docker ENV. I can submit a small PR that injects CONTAINER=true via Dockerfile just like we have others such as APPLIANCE=true. Thoughts?

@fbladilo That's fine with me too.

@fbladilo The menu upstream (MiQ) and downstream (CFME) are different.

@bazulay Should this be closed now that https://github.com/ManageIQ/manageiq/pull/9136 has been merged?

@jvlcek @fbladilo
As far as I'm concerned this issue is handled and can be closed,
In case an issue comes up with one of the sub menus than a separate issue should be opened.

@fbladilo @bazulay @gtanzillo Seem I don't have the privs to close this. Can one of you please close it?

Closing since this is addressed by https://github.com/ManageIQ/manageiq/pull/9136


All projects in VFT solution escaped their folders

cc @brettfo Any idea what could have caused this?

Any reproduction @vasily-kirichenko?
I updated my CodeLens to the latest master and only experienced improvements!

Yeah this is still an issue on latest nightlies

I'll be honest, I'm not even sure what we're doing in the solution file.  These files are horrendous and map guids to guids.  It doesn't appear anything which could have caused this was changed in that file.

Fixed.  @vasily-kirichenko  thanks for reporting this


Unable to turn off auto-fetching behaviour for PRs in review

It appears that the "GitHub Pull Requests and Issues" extension ignores all settings governing automatic repository fetches in VS Code. My setup relies on a physical interaction (touching my Yubikey) for SSH authentication to the remote, so any actions I haven't personally initiated are likely to fail. For this reason, I'd like to turn off automatic fetch behaviour.

Extension version: v0.54.1
VSCode Version: 1.73.1
OS: Darwin arm64 22.1.0

Steps to Reproduce:

Set git.autofetch and githubRepositories.autoFetch.enabled to false
Check out a PR for review

Expected:
No git fetch or git pull operations occurring, no notifications
Actual
Error notifications appear periodically saying An error occurred when fetching the repository, as SSH authentication fails.

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/4202


Add integration test of anonymous access.

Closes #6

Pull Request Test Coverage Report for Build 60

468 of 485 (96.49%) changed or added relevant lines in 10 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+7.09%) to 90.097%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

adam/tests/integration_tests/stm_propagation_module_test.py
32
33
96.97%

adam/tests/integration_tests/backwards_propagation_test.py
49
50
98.0%

adam/tests/integration_tests/basic_test.py
18
19
94.74%

adam/tests/integration_tests/permissions_test.py
132
133
99.25%

adam/tests/integration_tests/groups_test.py
84
85
98.82%

adam/tests/integration_tests/batch_runner_test.py
43
46
93.48%

adam/tests/integration_tests/anonymous_test.py
49
52
94.23%

adam/tests/integration_tests/projects_test.py
56
62
90.32%

Files with Coverage Reduction
New Missed Lines
%

adam/auth.py
2
92.68%

Totals

Change from base Build 36:
7.09%

Covered Lines:
2147

Relevant Lines:
2383

💛  - Coveralls

My bad: the changes to groups_test.py should have been on the broader integration tests pull request. Basically they're necessary to ensure that two simultaneous runs don't conflict, so rules like "the test account will always start with no groups" might not hold because another test might currently be running.


feat: add build-isolation options

Adds the ability to do build-isolation.
Would be good to get in: https://github.com/astral-sh/uv/pull/6517/, so it's more useful in issue like: https://github.com/prefix-dev/pixi/issues/1880 and https://github.com/prefix-dev/pixi/issues/1834
edit:
The 6517 PR mentioned above has been integrated, started UV upgrade here: https://github.com/prefix-dev/pixi/pull/1936 which we need to have the added test to succeed.
TODO:

[x] Documenation
[x] Json Schema
[x] Rust test
[ ] (nice to have) One example that uses it

Would be good if could start reviewing this @baszalmstra @ruben-arts

@ruben-arts also added an example that builds/installs detectron2 from source.

Thanksss


Add support for image pull secrets in the helm chart.

Wanted to use your chart with a private acr repository.

@nclaeys thanks for your contribution!
I will merge it.
I just want to know, if this information is not secret, what the purpose of this change is. Why do you need a private repository for this Docker Hub image?
I am using docker hub proxy for pulling this image in my clusters.

There are 2 situations that make the dockerhub image unfeasible in our setups:

we have installations that have no outbound internet access and where we thus cannot pull from dockerhub
Certain customers have policies on their k8s cluster that flag all images that do not come from a trusted registry.

For both cases we replicate the image in a private registry. Thanks for merging!

Thanks


Added docs about resource_url()

This will fix https://github.com/puli/issues/issues/182

Thanks :)


Review and refactor Permissions logic

Currently permissions editing is decided by request_methods in the query params, we should change this.
It can be based on individual params but again there should be a unified editing state (or not?)

Completed in https://github.com/getAlby/hub/pull/14 /  https://github.com/getAlby/hub/pull/273


Failing to Deploy

Hi All,
Trying to test and deploy this to Sentinel but when im attempting to deploy im consistently getting the below. ive followed in some of the other issues which ive now resolved but i cannot get past this crowdstrike playbook, is there anyway to bypass it? or to fix and deploy.

Hi @GodOfNekos, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 07-06-2024. Thanks!

Hi @v-rusraut Has there been any updates yet, as noticed today after syncing im not able to even get past the login attempts to try and deploy, now getting the below

I would like to be able to get this deployed ASAP how can i go about that?

Hi @GodOfNekos,
This error occurred due to the version of Az.Account has been increased. Downgrading to version 2.19.0 of Az.Accounts will resolve the issue. Use below command.
Install-Module -Name Az.Accounts -RequiredVersion 2.19.0 -Force
Thanks

@v-rusraut Im not a dev, where do i put this to get it working?, i just want to be able to deploy to a test environment right now for some R&D but not able to do that as i cannot deploy at all. unfortunately not my area of expertise and documentation is not clear on how to resolve these kinds of issues.

NVM fixed it, now just awaiting fix for crowdstrike error, when can this be expected as it is already beyond 07-06-2024 or was that meant to be 06-07-2024?

Hi @GodOfNekos,
There is custom deployment option available to deploy the playbook.
Please refer attachment which contain steps to deploy playbook using custom deploy option.
CustomDeploymentsteps.docx
Please refer readme file for your reference.
https://github.com/Azure/Azure-Sentinel/blob/master/Playbooks/CrowdStrike/Playbooks/CrowdStrike_ResponsefromTeams/readme.md
Thanks

@v-rusraut Unable to open document for Custom Deployment due to the Information Management Rights.
Finally got to a point with the deployment after removing content which is not needed for my environment. But failing right at the end of the deployment now. Can you let me know what the fix is for this.

Hi @GodOfNekos,
Please provide detail steps which you are following for deployment, so we can reproduce this issue.
Are you using Sentinel Repositories option to deploy this playbook ?

Thanks

Hi @v-rusraut
I have just been following the https://github.com/Azure/Azure-Sentinel/blob/master/GettingStarted.md and using the Sentinel Repositories on Azure. The only thing i have done which was different from the instructions was not creating a fork of the Repo, as this was intentionally going to be a test environment, i didnt want variables to change too much while i worked on how to deploy and finding out what was actually useful for my environments.

Hi @GodOfNekos,
Need more time to investigate this issue, we will update you.

@v-rusraut Unable to open document for Custom Deployment due to the Information Management Rights.
Finally got to a point with the deployment after removing content which is not needed for my environment. But failing right at the end of the deployment now. Can you let me know what the fix is for this.

Hi @GodOfNekos, based on your shared screenshot, you're getting error after perform operation to Main/master branch. Which is not accessible.
Please correct if I'm wrong, you're planning to deploy the contents (Playbook, workbook, Solution, etc.) into your azure sentinel portal?
Or you want to contribute this repro? but while committing the file your getting error's?
Kindly let us know your inputs on it, it would be really helpful for us to provide resolution to your issue. Thanks!

@v-sudkharat I am trying to deploy to my Sentinel workspace.

@GodOfNekos, thank you for your response.
One of the customers was facing a similar issue, like the one you are currently experiencing: https://github.com/Azure/Azure-Sentinel/issues/9876. As a resolution, we provided a repository link containing the deployable ARM template format. Below, I am sharing the Microsoft documentation and the repository link with you :
MS doc - https://learn.microsoft.com/en-us/azure/sentinel/ci-cd-custom-deploy?tabs=github
Sample repo - https://github.com/SentinelCICD/RepositoriesSampleContent
I Hope it help you. Kindly let us know if you have any queries or above links help you to resolve issue.
Thanks!

Hi @GodOfNekos, Waiting for your response on above comment, kindly let us know if anything for us. Thanks!

Hi @GodOfNekos, Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond to it in the next 2 days. If we don't receive a response by 10-07-2024 date, we will be closing this issue.
Thanks!

Hi @GodOfNekos, since we have not received a response in the last 5 days, we are closing your issue as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


Handle streaming loop if client has disconnected

🚀 Feature
If the client is disconnected while streaming the response we still complete the loop. We can stop the loop on BrokenPipeError.
x = lit_api.decode_request(x_enc)
y_gen = lit_api.predict(x)
y_enc_gen = lit_api.encode_response(y_gen)
for y_enc in y_enc_gen:
    ####### Detect disconnected client on BrokenPipeError  ############
    with contextlib.suppress(BrokenPipeError):
        pipe_s.send(y_enc)

Motivation

Pitch

Alternatives

Additional context

cc: @lantiga

@lantiga we should stop processing request if the client has disconnected and allow other requests in the queue to process. wdyt?

@aniketmaurya Non-streaming requests should also terminate the work if the client disconnects as well: https://www.loom.com/share/d6d97f0f34fd4084a72ff99471581cd7

@andyland in the case when the model worker has picked your request from queue and has started processing then we can't cancel it out unless we do some process communication which might add some bottlenecks but it's definitely worth looking into. Thanks for raising it.

the internal architecture has changed and we will solve it in a different way. This issue is now to be tracked in #165


Standardize header organization

Applied standardized header formatting.

This should fix the problems https://github.com/votca/tools/pull/351

Needs https://github.com/votca/votca/pull/632


Default effects

Adds basic default effects. They are implemented in the eval.ml file in a simple way that seems to work.
Switching between types was slightly improvised by looking at the rest of the code. All the type information is in the pervasives.eff file.
Also adds throwing an error when trying to define an effect with a name already in use.
Looking through the files the 'add_effect' is only used in the shell, so i used Error.runtime.

Thanks @zigaLuksic!


Podman creates directory when copying a file from and to container

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

when podman copies file to/from container, it creates a directory with the same name on destination and copies file to it
Steps to reproduce the issue:
echo "blabla" > /tmp/bla
podman cp --pause=false /tmp/bla instance:/root/
podman exec -it instance ls -lash /root/
4.0K drwx------.  2 root root 4.0K Jun 20 17:07 bla

podman exec -it instance ls -lash /root/bla
total 12K
4.0K drwx------. 2 root root 4.0K Jun 20 17:07 .
4.0K dr-xr-x---. 6 root root 4.0K Jun 20 17:07 ..
4.0K -rw-rw-r--. 1 root root    7 Jun 20 17:07 bla

Exactly the same thing when copying file from container to host.
Describe the results you received:
Directories with files
Describe the results you expected:
Just files
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
podman version 1.4.0

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.11.10
  podman version: 1.4.0
host:
  BuildahVersion: 1.8.3
  Conmon:
    package: podman-1.4.0-2.fc29.x86_64
    path: /usr/libexec/podman/conmon
    version: 'conmon version 0.2.0, commit: 80f19b1591b03dbbc53bf28b154230c7ea372281'
  Distribution:
    distribution: fedora
    version: "29"
  MemFree: 362270720
  MemTotal:<PHONE_NUMBER>8
  OCIRuntime:
    package: runc-1.0.0-93.dev.gitb9b6cc6.fc29.x86_64
    path: /usr/bin/runc
    version: |-
      runc version 1.0.0-rc8+dev
      commit: 82f4855a8421018c9f4d74fbcf2da7f8ad1e11fa
      spec: 1.0.1-dev
  SwapFree:<PHONE_NUMBER>0
  SwapTotal:<PHONE_NUMBER>4
  arch: amd64
  cpus: 4
  hostname: sshnaidm.remote
  kernel: 5.1.8-200.fc29.x86_64
  os: linux
  rootless: true
  uptime: 173h 16m 38.78s (Approximately 7.21 days)
registries:
  blocked: null
  insecure: null
  search:
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.access.redhat.com
  - registry.centos.org
store:
  ConfigFile: /home/sshnaidm/.config/containers/storage.conf
  ContainerStore:
    number: 1
  GraphDriverName: overlay
  GraphOptions:
  - overlay.mount_program=/usr/bin/fuse-overlayfs
  GraphRoot: /home/sshnaidm/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 26
  RunRoot: /tmp/1000
  VolumePath: /home/sshnaidm/.local/share/containers/storage/volumes

I still have this issue with podman 1.8

I still have this issue with podman 1.8

This has been dixed. Can you privide your steps to reproduce the issue?


Expand mask method in partitioning examples

In GitLab by @JoerivanEngelen on Nov 17, 2023, 14:39
Stems from https://gitlab.com/deltares/imod/imod-python/-/work_items/641
When setting up tests for partitioning, we sometimes wish to make cells inactive in a model. This means that those cells must be made inactive in the model's idomain array, and then data associated to those cells must be removed from all packages in the model.  We would like to use the "mask" method for that, but that doesn't work - after masking all the packages, we still get validation errors when writing the simulation.
Currently the packages are masked in a loop:
    # deactivate some cells on idomain
    idomain = simulation["GWF_1"].domain
    deactivated_cells = slice(93, 101)
    idomain.loc[{"mesh2d_nFaces": deactivated_cells}] = 0

    # The cells we just deactivated on idomain must be deactivated on package inputs too.
    for name, package in simulation["GWF_1"].items():
        if not isinstance(package, Well):
            for arrayname in package.dataset.keys():
                if "mesh2d_nFaces" in package[arrayname].coords:
                    if np.issubdtype(package[arrayname].dtype, float):
                        mask_value = np.nan
                    else:
                        mask_value = 0
                    package[arrayname].loc[{"mesh2d_nFaces": deactivated_cells}] = mask_value

But ideally we would like to use the (now private) _mask_all_packages  function of "model" for that
Refinement

Make private method _mask_all_packages public
Update example
Update test/fixtures where this is also done.
If masking results in empty packages, remove them
Add Validation for when all values are accidentily deactivated

merged to master


scripts: twister: set Python3_EXECUTABLE when calling cmake

west build sets WEST_PYTHON when calling cmake, but west twister
does not
This causes things to not work as expected when using a non-global python
The path code was copied from the scripts/west_commands/build.py
Signed-off-by: Mara Furland<EMAIL_ADDRESS>

It's failing tests such as test_projectbuilder_cmake_assemble_args_single in scripts/tests/twister/test_runner.py cause the cmake args are tested against a hardcoded list
There don't appear to be any similar tests for west build to copy, and I don't think hardcoding the string /opt/hostedtoolcache/Python/3.10.15/x64/bin/python is acceptable in tests, so how should these tests get fixed?

..  how should these tests get fixed?

I think the test can be adjusted to compose expected args using the current python3, or mock sys.executable

Okay, thanks for the feedback!
I switched to defining Python3_EXECUTABLE in the cmake_args section, and I added the defines to the test runner so the tests pass(at least locally)

It looks like twister blackbox test_coverage.py test_coverage_basedir is failing because the regex '{"files": \\[], "gcovr/format_version": ".*"}' does not match the json {'gcovr/format_version': '0.10', 'files': []} which I don't think is related to this change?

It looks like twister blackbox test_coverage.py test_coverage_basedir is failing because the regex '{"files": \\[], "gcovr/format_version": ".*"}' does not match the json {'gcovr/format_version': '0.10', 'files': []} which I don't think is related to this change?

probably it was lucky random mismatch to the wrong pattern - to fix it with #79578

probably it was lucky random mismatch to the wrong pattern - to fix it with #79578

That patch fixes it for me locally
That test fails the same way locally on the main branch, and the patch fixes it there too


Use krun class, avoid stderr

This is an attempt to switch the KAVM class over to using KRun class functionalities instead of manually calling krun.
Blocked on: https://github.com/runtimeverification/avm-semantics/pull/196
Blocked on: https://github.com/runtimeverification/avm-semantics/pull/189
Blocked on: https://github.com/runtimeverification/pyk/pull/105
Blocked on: https://github.com/runtimeverification/pyk/pull/104
Currently, we're seeing a slowdown using this technique:

Typically, I should be able to run each simulation in ~140-200ms, but now we're seeing ~700-1100ms. Most of the time is wasted in Kore -> Kast conversion in the pyk code. To improve this (by about 500ms I estimate), we should do:

Implement get_cell over Pattern instead of having to convert the whole term to Kast before selecting a cell.
If that is still not fast enough, implement kore_json_to_dict instead of kjson_to_dict, to avoid having to convert to Kast at all.

This design is definitely better, but we need to work on performance, as outlined. Thanks @ehildenb!

@ehildenb I've cherry-picked most of your commits into #222, closing this one.


Connect with Funder Registry

Recommendation 56. COPIM needs to keep a watching brief on Funder Registry and build in the possibility of adding three additional pieces of metadata - funder name, funder id, and grant id.

For documentation purposes: Open Funder Registry to transition into Research Organization Registry (ROR) - https://www.crossref.org/blog/open-funder-registry-to-transition-into-research-organization-registry-ror/


Implement test command

This repo is a prototype CLI for automated testing of releases. This PR adds the basic implementation of the single command standup test which (in theory) creates a release on a CP, creates a cluster using that release, runs a test against that cluster, and cleans up once it finishes.
Checklist

[ ] Update changelog in CHANGELOG.md.

Merging this to iterate from here


Rollback "Effectively" article

Docs Build status updates of commit cbc7aac:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Closed to avoid the whole branch-out-of-sync problem after conflicts were resolved.  Reopening as a new PR.


[mpeg2d] Set m_isOpaqueMemory to false directly

The problem was found by static code analyze
Signed-off-by: Denis Volkov<EMAIL_ADDRESS>

@dmitryermilov ,
Code analyzer found possible nullptr dereference for pOpqExt (please see the picture below) :

As you wrote previously, field m_isOpaqueMemory set to false by default in constructor, and it can be true (go to if branch before dereference - 3) ONLY if there is OPAQUE memory - we go to if branch (before 2) and set m_isOpaqueMemory to true directly - in this case pOpqExt always not null.
I think it's a good solution to set m_isOpaqueMemory to false directly in "else" branch (2) - we can avoid misunderstanding for developers and code analyzer too.
What do you think?

I believe there is no real problem here. Thus,  it doesn't make sense to introduce additional assignment to false if  m_isOpaqueMemory is set false by ctor.

@dmitryermilov ,
OK, I agree. Field m_isOpaqueMemory sets to false in 2 places - constructor and Close() method, and sets to true only in one place (for now) - inside AllocFrames (line 1804).
But I was confused that field  m_isOpaqueMemory   is a "public", and it's possible to change this field outside of the class. So, this was the first reason why I created this patch.
For now we don't change value of the m_isOpaqueMemory anywhere except 3 places which I wrote previously, outside of the class this field is used read-only.
But it's possible to get some problems in the future and I think it's a time to avoid them.
I agree that current fix (set m_isOpaqueMemory  to true) is not good, and I suggest to add checking pOpqExt for nullptr and reporting an error MFX_ERR_UNDEFINED_BEHAVIOR before dereference.
Also, I think that it's better to hide m_isOpaqueMemory  to protected and create special method for getting this value (for example, IsOpaqueMemory()).
What do you think about that? If we will get an agreement I can do both of these changes.

I don't see a real problem here (agree w/ @dmitryermilov) and at the same time I don't think this is a big deal to set [m_isOpaqueMemory = false] here explicitly to make analyzer happy.
@DenWolf

it's better to hide m_isOpaqueMemory to protected
Agrre, it makes sense. Maybe even private.

If so, at line 1824 I'd replace m_isOpaqueMemory  with pOpqExt


Show program admins summary view of which other programs their applicants applied for

Program admins want to understand what programs their customers are also applying for (or are eligible for but aren't applying to), so they can give their customers tips about new programs they might find helpful.
So on the program admin's page for a given program, show a table that lists all the other programs on this CiviForm instance, and next to each program, show the percent of applicants to your program that also applied to that program.

Can do a work-around with reporting outside CiviForm, but this would still be a nice-to-have.


How does one set the info in the Android notification?

I am doing this:
 await CrossMediaManager.Current.Play(new MediaFile(url, Plugin.MediaManager.Abstractions.Enums.MediaFileType.Audio, Plugin.MediaManager.Abstractions.Enums.ResourceAvailability.Remote)
                            {
                                Metadata = new MediaFileMetadata
                                {
                                    Title = Item.Name,
                                    Artist = "Volunteer Park Trust",
                                    Album = "Walking Tour"
                                }
                            });

I am not sure if the mp3 has any meta data in it, so was trying to set it manually?

The Android notification extracts whatever data may or may not be in the ID3 tags and displays that, overwriting anything you have manually entered.  I am not sure if it was initially designed that a user can enter their own meta data, however I have added a bool ExtractMetadata property to the media file, allowing a user to prevent the overwrite.
PR#285 addresses this issue on Android

I will fix up the ID3 tags on these MP3s, that makes sense :)


StringParser.php - expression is always true

The expression:
            if (ord($this->data[$this->offset]) != 0x0A || // Line feed
                ord($this->data[$this->offset]) != 0x0d    // Carriage return
               )

in function skipComment is always true
if  ord($this->data[$this->offset])  is  0x0A -> (ord($this->data[$this->offset]) != 0x0d ) is true -> whole expression is true
if  ord($this->data[$this->offset])  is  0x0d -> (ord($this->data[$this->offset]) != 0x0A ) is true -> whole expression is true
if  ord($this->data[$this->offset])  is  0x0C (some other value) -> (ord($this->data[$this->offset]) != 0x0A ) is true -> whole expression is true
This possible defect found by AppChecker

We are sorry, but ZF1 has reached its End of Life (EOL) so only security-related fixes are currently being accepted.


add ability to parse entire feeds

Hey, I wanted the ability to parse my entire Medium.com RSS feed. I figure I'm probably not the only one.
I kept the existing usage functional (i.e. printing to STDOUT) but added an --output option to either direct output to a file, or to a directory if we're parsing a feed. The resulting filenames are generated using the post's title, removing special chars.
Most of index.js has been moved to get-post.js. The only change there is at the end, where the resulting text is passed to the callback if present, and otherwise is sent to console.log().
Hope this is helpful - happy to refactor if there's anything you want to change.

Thanks @bobby-brennan! I merged this into my fork and then into this master, but somehow your contribution got lost. Do you know how I can resolve this?

Hmm not sure - you can always copy/paste the changes. No worries if I lose my "contributor" credit :)

The changes are there, just not the credit :(. Sorry I messed up there. Closing this then.


No Recipe for Some Melter Items

hello so im playing modpack FTB Revelation in a server (v1.4) and i wanted to make some tritium today and i noticed some of the melter recipes are missing . like :
Molten Boron
Molten Lithium
hope you take a look , thanks.

Hey there - this was fixed in v2.9d and version 1.5 of the pack has the update :)


Speed up CI testing by precompiling only once

It's a small improvement only, but it seems like precompilation is indeed only happening once anymore.


[SPARK-40061][SQL][DOCS] Document cast of ANSI intervals

What changes were proposed in this pull request?
In the PR, I propose to update the doc page https://spark.apache.org/docs/latest/sql-ref-ansi-compliance.html#cast about new behaviour of CAST introduced by the following PRs:

[SPARK-40008][SQL] Support casting of integrals to ANSI intervals
[SPARK-39451][SQL] Support casting intervals to integrals in ANSI mode
[SPARK-39470][SQL] Support cast of ANSI intervals to decimals
[SPARK-40014][SQL] Support cast of decimals to ANSI intervals

Why are the changes needed?
To inform Spark SQL about new behaviour of CAST.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
By building Spark SQL docs and check them manually:
$ SKIP_SCALADOC=1 SKIP_PYTHONDOC=1 SKIP_RDOC=1 bundle exec jekyll build

Merging to master. Thank you, @HyukjinKwon for review.


fix links to online docs in template config files

Almost all template configuration files in doc/config/conf.d/ have broken links, as shown below.

Merged in 36a266ec29ce943e3485b11524161f323534e50b

You are welcome, @stbuehler !
BTW, if you would like to add the "opt-out for ETag & Last-Modified headers"  feature, please do not hesitate to ask and I'll be happy to create a new PR for it. BTW, the reason why I "recycled" this PR was because I was unsure whether it would be desired or not (and I still don't know). So just drop me a line and I'll submit it. BTW, I've got another patch for mod_ssi with  SSI_ECHO_AUTHORITY, SSI_ECHO_SCHEME and, most interesting, SSI_ECHO_SCRIPT_URI, which makes SCRIPT_URI available in pages handled by the SSI module. SCRIPT_URI is really handy for adding links to the W3C Validator:
...
This is much better than having /check?uri=referer because some users configure their browser to only send referers to the same site, (or completely disable referers). Having SCRIPT_URI makes it possible to create links to the W3C Validator that work for everyone. Moreover, SCRIPT_URI already exists in Apache. Just let me know if you are interested and I'll be glad to submit it in a PR. Thanks!

@fbrosson Didn't realize you recycled the PR... please don't do it again :)
The additional SSI variables sound fine (if they are also supported by apache), it would be nice if you could submit them.
I'm not sure what the "opt-out for ETag & Last-Modified headers" feature would be doing (I can't see the diff anymore); but I guess it sounds fine to not send them in a dynamically created page by default.

Thanks for your message and sorry about the recycled PR. I promisse I won't do that anymore.
I know that Apache defines SCRIPT_URI but I need to check whether it also defines SCHEME and AUTHORITY.
Having AUTHORITY is probably not very important because with HTTP/1.1 the server also has the information in HTTP_HOST.
Having SCHEME is handy in situations where you serve both http and https and want to display one image for http and another for https. Imagine you have images of the QR codes encoding the address of your main page in each of the protocols, http and https. Putting the QR code in the page makes it very easy to use the scan feature of modern smartphones to copy/paste an address from one screen to a mobile device, all that with the very same protocol used on the source screen. You could name your images qrcode-http.png and qrcode-https.png and then include something like  and mod_ssi would do the job. That's both easier and faster than inserting tests with SSI expressions to compare the contents of the SSI variable HTTPS to see if it equals "ON", if the configuration includes such a definition.
The other small advantage of having SCHEME is that you can easily "create" links that look like  protocol-relative links without having to rely on the browser's ability to handle true protocol-relative links (i.e. ://). But I do agree that such browsers are not common and that it is not smart to use this method to create such fake protocol-relative links just for these browsers.
Well, let me know which variables you would like to have and I'll create a PR with a patch for adding them. What do you think of adding both SCRIPT_URI and SCHEME ?
Regarding the opt-out feature, here's a small description of why I think it would be nice to have it.
Most of my pages look static, but I use mod_ssi to have their timestamps automaticaly displayed (with the famous and obvious  ). So for all these pages the normal behaviour of mod_ssi is perfect. But I also have some dynamic pages (e.g. a GeoIP page that displays information related to the IP address of the client), also handled by mod_ssi, for which I don't need to send any ETag or Last-Modified headers. It does not harm to send them, but since they are not necessary for such dynamic pages, it is nice to be able to opt-out on a context-based config. The feature is really simple and I'm pretty sure it can't break anything.
If you are also interested in that feature, let me know what is your favorite syntax to invert the default behaviour. Would you prefer to have this:

ssi.omit-etag = "enable"
ssi.omit-last-modified = "enable"

(which was my first idea), or ths:

ssi.send-etag = "disable"
ssi.send-last-modified = "disable"

(which was my second idea, probably slightly better) ?
Of course, I'll take care of making the change in a way that it is neutral to configurations that do not specify any setting.
In case you are interested in both of my proposals, then I guess it is best to have 2 separate PRs.
Thanks!

Regarding the ETag and Last-Modified headers,  you wrote this in your last message:

I guess it sounds fine to not send them in a dynamically created page by default

I also think it's a good idea. (It did not catch my attention immediatly, sorry).
We can use "disable" as default values for ssi.send-etag and ssi.send-last-modified and allow the user to opt-in on any context with:

ssi.send-etag = "enable"
ssi.send-last-modified = "enable"

Do you think the suggested keywords are fine? Would you prefer to have something else?
I'll create a PR for that with appropriate comments in the template ssi.conf file to explain this tiny change and emphasize that it only affects dynamic pages handled by mod_ssi.
Regarding the SCRIPT_URI (and maybe SCHEME), I have started making a list of which important and interesting SSI vars exist (or are missing) in Apache vs Lighttpd. I'll come back once it is finished so that you can chose which ones you would like to have added. Cheers!

The ssi.send-* names sound fine to me. Regarding the variables I have no problem synchronizing our list with apache, as long as the variables can be easily and correctly implemented on our side.

OK, cool. I'll jump into the opt-in feature very soon. Regarding the variables, SCHEME does not exist in Apache but maybe if Lighttpd adds it then Apache will follow, who knows? SCHEME is really easy to add. It can be accessible to SSI pages as SSI vars, and it can even be made available very easily as an environment variable. For SCRIPT_URI it is also easy to make it available as an SSI var (and I already have the patch for that). But it is not trivial (for me) to also make it available as an environment variable. I know Apache makes SCRIPT_URI available as an SSI var, but I'm not yet 100% sure that it also makes it available as an environment variable. I need to check that. Anyway, making it available in Lighttpd as an SSI var is a good start. And I'm pretty sure that from my patch it will be easy for several other developers to suggest a few small changes to make it also available as an environment variable. It's just a matter of calling the right routines to concatenate buffers. I knew how to do this on 1.4.35 but there were some changes since and I did not refresh my patch to make it also create the environment variable for SCRIPT_URI.

If apache doesn't have a different name for SCHEME I guess it should be fine for now. (CGI environment probably only sets HTTPS=on for https and otherwise not at all.)

After diving into some Apache dumps of dynamic pages launching ENV, some searching on the net, and finally grabbing the source tarball of Apache 2.4.18 (the current latest) and examining its source code I can confirm that it does not define any var as either http or https. As you pointed out, the only related var is HTTPS (but as you also point out it is not done the way we could do it in Lightty). And since SCHEME is not already defined in Apache and seems to be a good candidate for a var that could be set to either http or https, I think it is safe to use that name. I haven't checked yet with Nginx but I"ll try to do so.
BTW, if you grep apr_table_setn recursively you will find apr_table_setn(r->subprocess_env, "HTTPS", "on"); in modules/arch/netware/mod_nw_ssl.c, apr_table_setn(e, "REQUEST_URI", original_uri(r)); in server/util_script.c and several other env vars (but nothing equivalent to the SCHEME var we could add to Lightty). I'll create a PR with for these 2 vars asap.


Update NSIS Installer script

@Wagnard
Add Italian language/Add strings for Italian
Add multilanguage shortcut name
Add estimated program size in Windows program list
Add file exe poperties
In copyright infoi get automaically current year by system date
For source path use a variable
It means more flexible use for the future (requet less changes).

Thanks, I didn't really bother with that script but I like the changes.


Something is wrong w/ breaks

The current examples for solobox show that the buttons break at kind of random intervals.
Would like the behavior to be: Buttons continue to right unless explicitly forced by div_maker.

ahhh... Ya entendi. Al parecer, el primer botton siempre causa un linebreak, pero el resto no. Porque sera?


Autocomplete undeclared references

I think it is quite common for people to write a reference to an undeclared property or variable first and then write the declaration of it. And not the other way around.
Given the below code:
class View {
  bind() {
    this.data.on('change:title', this.renderTitle);
  }

  renderTitle() {
    // ...
  }
}

If I would have written the above code from scratch, I would often write the reference this.renderTitle first before the declaration of renderTitle:
    this.data.on('change:title', this.renderTitle);

So it would be good if it could autocomplete renderTitle when I am about to write the declaration for it:
  renderTitle() {
    // ...
  }

The actual suggestion here is somewhat difficult from a technical perspective. I wonder if an "add missing member" code-fix would possibly help alleviate some of the pain of the current experience.

I wonder if an "add missing member" code-fix would possibly help alleviate some of the pain of the current experience.

Yes that would also be preferrable.

Closing due to lack of feedback


Fix integration with API Platform 2.4, the other way around

Another approach.
See https://github.com/api-platform/core/pull/2732#discussion_r275135410
Closes #399.

I would prefer this one over #399. Don't forget the changes in composer.json in this PR, though.


Adds detection of ES2015's new Object functions

Hi there,
I'm using this library for a while, and recently I've noticed this issue, and I thought it would be a good idea to add this feature.
I'm testing for the existence of: assign, setPrototypeOf getOwnPropertySymbols in Object.
I also included the tests.
LMK if you think things should be different.
Eliran

Hi, sorry for the slow response. Does this new method cover all new Object methods? After a quick glance at the new features I think we're missing Object.is. Could you add that please?

Sure, no problem. Probably just missed it.
I added it and the tests seem to pass.
Let me know if you have any more comments.

released in v1.4.0 - thanks mate. 👍

My pleasure :)


List the detected circular imports in the console

Description
I'm meeting an issue with circular imports while using "import type" declaration.
Vite does detect the circular imports and reload the whole page:
12:37:21 [vite] page reload src/components/billing/BillingList.vue (circular imports) (x2)
But where are the circular imports ? Vite is not enough verbose.
Suggested solution
Make vue logs more verbose and display the circular imports that was detected:

12:37:21 [vite] page reload src/components/billing/BillingList.vue (circular imports) (x2)
BillingList.vue > apitypes.ts > userStore.ts > apitype.ts

Alternative
Waiting on madge to support vuejs files: https://github.com/pahen/madge/issues/122
(still pending since 2017)
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

You can use vite --debug hmr.
See https://vitejs.dev/guide/troubleshooting.html#a-full-reload-happens-instead-of-hmr for more details.


generate xml and html report in the same execution

I need to generate the xml and html reports in the same execution. No, I haven't seen that there is an option that allows it

A new "all" option needs to be added to the report type dropdown:

Currently I have configured my releases with the XML option and using PowerShell I download the report in HTML.
If you need help I can help you with PowerShell.

I think I have solved it in:
https://github.com/kasunkv/owasp-zap-vsts-task/pull/60
Can someone validate it?


Improve tracing usage and examples

This is on top of #29 which should be merged first

oh oops I ended up with review comments from #29 in here didn't I :sweat_smile: it's mostly just small nits on that end though


CoreFoundation: clean up CF_CONST_STR macro definition

There were six different variants with minor tweaks to the macro definition.
Use a few more macros to abstract out the retain count initializer, the tag
name, and the const-ness of the struct.  This allows the variants to be
collapsed into a single definition.

@swift-ci please test

CC @parkera


Add support for user password rotation

Is your feature request related to a problem? Please describe.
Changes to userPasswords are ignored (only picked up on first start)
Describe the solution you'd like
changes to userPasswords are applied to the container
I believe this is an upstream limitation of the bitnami image similar to https://github.com/bitnami/containers/issues/44545
The second start shows the following traces indicating "Using persisted data" and not proceeding with existing configuration
Defaulted container "openldap-stack-ha" out of: openldap-stack-ha, init-schema (init), init-tls-secret (init)
 10:58:25.14 INFO  ==> ** Starting LDAP setup **
 10:58:25.24 INFO  ==> Validating settings in LDAP_* env vars
 10:58:25.26 INFO  ==> Initializing OpenLDAP...
 10:58:25.26 DEBUG ==> Ensuring expected directories/files exist...
 10:58:25.29 INFO  ==> Using persisted data
 10:58:25.30 INFO  ==> ** LDAP setup finished! **

https://github.com/bitnami/containers/blob/54360af174f405dd7770a867ea880e8469639f3b/bitnami/openldap/2.6/debian-12/rootfs/opt/bitnami/scripts/openldap/setup.sh#L18-L27
https://github.com/bitnami/containers/blob/54360af174f405dd7770a867ea880e8469639f3b/bitnami/openldap/2.6/debian-12/rootfs/opt/bitnami/scripts/libopenldap.sh#L611-L619
https://github.com/bitnami/containers/blob/54360af174f405dd7770a867ea880e8469639f3b/bitnami/openldap/2.6/debian-12/rootfs/opt/bitnami/scripts/libopenldap.sh#L592-L600
Describe alternatives you've considered
Rotating user passwords using ldap cli
Additional context
Thanks again for the great work on this project !

It's in my fork https://github.com/jp-gouin/containers/tree/main/bitnami/openldap
It's not the plan to maintain a fork however one mandatory PR that I submitted has been reverted. I need to find an alternative to the PR to use the upstream one again.


version

https://github.com/waa/baculabackupreport/blob/72c3209f390397f19d94b3a48b683202035f0286/baculabackupreport.py#L4
This is not, in fact, version 1.0, nor is that the release date.

Hello Daniel,
While git log shows that the information in that line is indeed true/correct, you are right, it does not belong in the script there. Every time I see it it bothers me too.
I'll modify it to more clearly show that the initial release was on that date and remove the version 1.0 too.
Thanks for the comment. :)


added Derivative.delist

related to #274 #495
Opposite method to list()

Codecov Report

Merging #496 (7c50126) into main (6d3c8a5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #496   +/-   ##
=======================================
  Coverage   99.55%   99.55%
=======================================
  Files          49       49
  Lines        1803     1805    +2
=======================================
+ Hits         1795     1797    +2
  Misses          8        8

Impacted Files
Coverage Δ

pfhedge/instruments/derivative/base.py
98.26% <100.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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

Cool, must be useful. Thank you a lot!


Improved accessibility

Description
This PR fixes #1992
Notes for Reviewers
Signed commits

[X] Yes, I signed my commits.

@Venkata-Sai-Vishwanath-robo
Thank you for your contribution!
Let's discuss this during the website call on Monday(today) at 6:30 PM IST (7:00 AM CT).
Please add it as an agenda item to the meeting minutes.


Code overflow and support inline code?

It's better to add overflow: auto; into code style to avoid code overflow. I am wondering if this blog could support the inline code ``. Also, I used <marquee> for the notion type: callout. This is my personal suggestion.

Hi, thanks for the suggestion. I updated the code component with overflow: auto. Inline-code from Notion is supported in this example and is applied in the textBlock renderer


add CircleTimer

https://github.com/jaeryo2357/circleTimer
circleTimer is based on Circle SeekBar

@jaeryo2357
I like the animation.


https://www.facebook.com/nawipauwigi.uwigi.3Update hello-world.md

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Hack fb

1

MuhammadPaskalisTagi:patch-2


Restart API samples, cpp and cs

Description
Samples for the restart feature in 1.1 Preview 1 and onwards. An extremely simple/straightforward example of how to call the restart method and handle error responses in both cpp (console) and cs (winui) solutions.
Target Release
1.1 Preview 1.

/azp run SamplesCI-AppLifecycle


enhancement: a11y rule "mouse-events-have-key-events" fails with a warning.

Describe the bug
jsx-a11y eslint plugin was recently added to the project and this rule -
jsx-a11y/mouse-events-have-key-events - fails with a warning.
This issue offers more context: https://github.com/open-sauced/app/issues/2264
Steps to reproduce
The following components and files are causing the rule to fail:

components/molecules/CardHorizontalBarChart/card-horizontal-bar-chart.tsx
components/shared/InfoTooltip.tsx

.take


Refactoring gilded rose

Regarding commit messages, there are many articles about it, attached an article with some general good guideline and a few examples to commit structures:
https://reflectoring.io/meaningful-commit-messages/#:~:text=The commit message should describe,commit message can be helpful
Here are the guidelines we agreed upon in recolabs:
https://recolabs.atlassian.net/wiki/spaces/RND/pages/718275150/Commit+history+looks+good+-+git


Remove obsolete branches

Looks good already! You have all the necessary files there and the repo is nice and clean. Looks like the module is also slowly taking shape. One thing I noticed is that you have lots of open branches, I recommend deleting the ones you don't use anymore. You can delete them on GitHub, but you also need to delete them on your local machine (remember, you have a complete copy of the repo locally). You can do so using git fetch -p (this will prune the branches), sometimes you may have to do git branch -d mybranch of even using a capital D, if there are changes you did not want to commit.
You can also delete them from VSCode on the git tab, clicking on the three dots in the top right of the git bar, and selecting "Branch", etc.

Thanks @iulusoy for the reminder 🙂
@vivienzahs @rdlaura I removed the branches on the remote.
I also deleted any obsolete branches for me locally. Once you have done the same, feel free to comment here or just close the issue.


make more distinct colors in tetris

please add pink instead of black

var y = { size: 3, blocks: [0x0C60, 0x4C80, 0xC600, 0x2640], color: 'white' };


Create 2023-12-13 Apache Struts

Added notice about Apache Struts RCE CVE-2023-50164

Thanks for your PR @idarlund !
Because this issue was fixed and reported upon on 6 days ago (https://struts.apache.org/announce-2023#a20231207-1), I wouldn't consider it appropriate for a notice. Generally, my guideline is that the high-impact vulnerability needs to have been announced or found within the last 24 hours, unless there are special circumstances (e.g. the vulnerability has been found to be more trivial to exploit than initially realized, etc).
If I'm missing something please let me know, but for now I'm going to close your PR. You have my sincere gratitude for taking the time to write it up though.

Sure no worries.
The vulnerability in Struts is 6 days old, but this seems to be an emerging vulnerability since it's in a framework. Cisco just released their bulletin, it seems Atlassian are quiet but probably impacted and Shadow Server are seeing active exploitation at the moment: https://twitter.com/Shadowserver/status/1734919288257974380

There's also published a POC here on github 9 hours ago: https://github.com/jakabakos/CVE-2023-50164-Apache-Struts-RCE

Thanks, I appreciate the information. In this particular case I'm still going to say that I'll leave the PR closed. It just looks like there is considerable amount of chatter online about this particular vulnerability and all of the usual lists have already sent their warnings (even governmental agencies which are normally quite slow on the uptake). For this reason I think bug alert has already missed its window to be effective.
Again, I appreciate your contribution!


Multiple spa with multitenant

I am looking to redirect to an admin spa when tenant is null, otherwise redirect to user spa
app.Map("/admin",
   adminApp =>
   {
     adminApp.UseSpa(spa =>
     {
       spa.Options.SourcePath = "angular/admin";
       spa.Options.DefaultPageStaticFileOptions = new StaticFileOptions
       {
           FileProvider = new PhysicalFileProvider(Path.Combine(Directory.GetCurrentDirectory(), "angular", "admin"))
       };

       if (env.IsDevelopment())
         spa.UseProxyToSpaDevelopmentServer("http://localhost:4200");
      });
    });

  app.Map("/user",
    userApp =>
    {
      userApp.UseSpa(spa =>
      {
        spa.Options.SourcePath = "angular/user";
        spa.Options.DefaultPageStaticFileOptions = new StaticFileOptions
        {
            FileProvider = new PhysicalFileProvider(Path.Combine(Directory.GetCurrentDirectory(), "angular", "user"))
        };

        if (env.IsDevelopment())
          spa.UseProxyToSpaDevelopmentServer("http://localhost:4201");
      });
  });

In the code used by routes, I try to do it through the tenantInfo, is it possible to do it or would there be a better way?
Thanks

hi, I'm not too familiar with Angular but I noticed you have spa.Options there. Have you looked into using per-tenant options? You could have  default tenant called _admin or similar and register the static tenant strategy last. This will cause the tenant to default to _admin if no other tenant was found:
services.UseMultiTenant<MyTenantInfo>()...WithStaticStrategy("_admin") // static strategy registered after any other strategy used
    .WithPerTenantOptions<SpaOptions>((options, tenantInfo) =>
    {
        if(tenantInfo.name == "_admin")
        {
           spa.Options.SourcePath = "angular/admin";
           spa.Options.DefaultPageStaticFileOptions = new StaticFileOptions
           {
           FileProvider = new PhysicalFileProvider(Path.Combine(Directory.GetCurrentDirectory(), "angular", "admin"))
           };
         }
    });

I'm not sure how the spa options are used but this way the admin one would be set if no other tenant was found, otherwise it would use your regular spa options.

hi, I'm not too familiar with Angular but I noticed you have spa.Options there. Have you looked into using per-tenant options? You could have default tenant called _admin or similar and register the static tenant strategy last. This will cause the tenant to default to _admin if no other tenant was found:
services.UseMultiTenant<MyTenantInfo>()...WithStaticStrategy("_admin") // static strategy registered after any other strategy used
    .WithPerTenantOptions<SpaOptions>((options, tenantInfo) =>
    {
        if(tenantInfo.name == "_admin")
        {
           spa.Options.SourcePath = "angular/admin";
           spa.Options.DefaultPageStaticFileOptions = new StaticFileOptions
           {
           FileProvider = new PhysicalFileProvider(Path.Combine(Directory.GetCurrentDirectory(), "angular", "admin"))
           };
         }
    });

I'm not sure how the spa options are used but this way the admin one would be set if no other tenant was found, otherwise it would use your regular spa options. I think this way you wouldn't have to use map but instead a route value to determine the tenant or some other strategy.

Hm, hard for me to be much help with my lack of knowledge on the SPA stuff -- do you have a projet on github I can fork to take a look at the issue closer?

Hm, hard for me to be much help with my lack of knowledge on the SPA stuff -- do you have a projet on github I can fork to take a look at the issue closer?

I have uploaded an example project, I hope you can review it please thanks

Thanks for the repo. It seems this approach for SPA has been deprecated but I think you can still get this to work.
I think the best option will be to se MapWhen like this:
app.MapWhen(httpContext => httpContext.GetMultiTenantContext<TenantInfo>()?.TenantInfo == null,
   adminApp =>
   {
     adminApp.UseSpa(spa =>
     {
       spa.Options.SourcePath = "angular/admin";
       spa.Options.DefaultPageStaticFileOptions = new StaticFileOptions
       {
           FileProvider = new PhysicalFileProvider(Path.Combine(Directory.GetCurrentDirectory(), "angular", "admin"))
       };

       if (env.IsDevelopment())
         spa.UseProxyToSpaDevelopmentServer("http://localhost:4200");
      });
    });

app.MapWhen(httpContext => httpContext.GetMultiTenantContext<TenantInfo>()?.TenantInfo != null,
    userApp =>
    {
      userApp.UseSpa(spa =>
      {
        spa.Options.SourcePath = "angular/user";
        spa.Options.DefaultPageStaticFileOptions = new StaticFileOptions
        {
            FileProvider = new PhysicalFileProvider(Path.Combine(Directory.GetCurrentDirectory(), "angular", "user"))
        };

        if (env.IsDevelopment())
          spa.UseProxyToSpaDevelopmentServer("http://localhost:4201");
      });
  });

Good luck and let me now how it works out.


Scrollable signposts

Hi,
First of all thanks for this great plugin!
When implementing signposts in one of our projects, I noticed that the overlay failed to handle large amount of signposts (>100 in our case). In this case, the overlay extends over the whole screen, but doesn't scroll together with the rest of the content. Because of this, only some signposts are shown and the bottom of the signpost list is never reachable.
To fix this, i suggest to add scrolling behavior to the overlay if the number of signposts exceeds the height of the view. I additionally changed the maxHeight of the overlay to 50%, but of course this change is optional and adding a scrollbar would also fix the issue if the full width is exceeded.
|--------|---------|

Before
After

Thank you very much for contributing this patch @dnlbauer ! The changes work well in my tests.

@dnlbauer FYI this is now released in v1.2 of the extension on the Chrome Webstore.


Fixes inconsistencies between listen and emit

Please specify parts of this PR update:

[x] Specification
[x] Schema
[ ] Examples
[ ] Extensions
[ ] Use Cases
[ ] Community
[x] CTK
[ ] Other

Discussion or Issue link:
Closes #917 (and other little minor typo)
What this PR does:

Fixes inconsistencies between listen and emit

Added new event section in the specification documentation
Fixed the sample of emit in the specification documentation
Added new event definition in the JSON schema and referenced it in emit.event.with and eventFilter.with

Fixes a typo in the set feature
Fixed type of an http call endpoint by adding string
Added new (missing) endpoint section in the specification documentation
Escaped * character
Fixed double https://https://
Fixed links to runtime expressions
Fixed typo in implictly

Additional information:
-

@JBBianchi can you please rebase?


Laravel 9.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 9.x.
Before merging, you need to:

Checkout the l9-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 9. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/mula.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "lukeraymonddowning/mula": "dev-l9-compatibility",
    }
}

Finally, run: composer update


ValueError: StreamingTextDataset() got an unexpected keyword argument: {'cache_limit': '100gb'}

Look here, looks like you forgot update with last StreamingDataset from streaming in your StreamingTextDataset

@vchiley


Add resources to worker and jobs

Implements #701
Adds flags for the controller to set the worker container resources.
Adds project configuration to set the init-container vcs-sidecar resources.

@vdice Thanks for the comments! I've done what you suggested

This looks great; thanks again @jchanam !


#5 fixes the issue with cent amounts

Stripe API returns amounts in cents. This PR fixes this issue.

Fantastic! Thanks for the PR @DmitryKey 🎉

yay! Thanks for accepting @jsjoeio. These was my first code in node :)


fix the unmount rootfs step is not performed on the scale down node.

unmount rootfs is executed when the sealer delete -m command deletes some nodes.
delete duplicate function.
desired Clusterfile is saved before the runtime during cluster create or scale up.

/test apply