Markdown headings are not styled

I think it works now for everyone. I suggest to close this one.

Up to you. I can no longer verify as moved away from atom.

What are you using then? :) I suggest that you try to find a maintainer then.


Remove Gulp-Util

As per https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5
Please remove gulp-util using the following API replacements:
gutil.File => https://www.npmjs.com/package/vinyl
gutil.replaceExtension => The .extname property on Vinyl objects or https://www.npmjs.com/package/replace-ext
gutil.colors => https://www.npmjs.com/package/ansi-colors
gutil.date => https://www.npmjs.com/package/date-format
gutil.log => https://www.npmjs.com/package/fancy-log
gutil.template => https://www.npmjs.com/package/lodash.template
gutil.env => https://www.npmjs.com/package/minimist
gutil.beep => https://www.npmjs.com/package/beeper
gutil.noop => https://www.npmjs.com/package/through2
gutil.isStream => Use the .isStream() method on Vinyl objects
gutil.isBuffer => Use the .isBuffer() method on Vinyl objects
gutil.isNull => Use the .isNull() method on Vinyl objects
gutil.linefeed => Use the string '\n' in your code
gutil.combine => https://www.npmjs.com/package/multipipe
gutil.buffer => https://www.npmjs.com/package/list-stream
gutil.PluginError => https://www.npmjs.com/package/plugin-error

What were you expecting to happen?
What actually happened?
Please post a sample of your gulpfile (preferably reduced to just the bit that's not working)
gulp.task(function () {});

What version of gulp are you using?
What versions of npm and node are you using?

This is already fixed on Gulp 4 and won't be fixed on Gulp 3.

This is already fixed on Gulp 4 and won't be fixed on Gulp 3.

If fixed, so fix the documentation  as well, because every person reading that documentation will report here again and again...

@serhioV Please be more understanding of open source. We are not here to serve you, we are donating our own time to this project.
@phated has been spending weeks (hundreds of hours) constantly working on the gulp 4.0 release docs.
If you just checked any of the issues/PRs you would see that. https://github.com/gulpjs/gulp/pull/2226
https://github.com/gulpjs/gulp/issues/2231
https://github.com/gulpjs/gulp/issues/2217#issuecomment-417856560
https://github.com/gulpjs/gulp/issues/1486

@serhioV Please be more understanding of open source. We are not here to serve you, we are donating our own time to this project.
@phated has been spending weeks (hundreds of hours) constantly working on the gulp 4.0 release docs.
If you just checked any of the issues/PRs you would see that. #2226
#2231
#2217 (comment)
#1486

I don't ask to be served. It's just a remark to be implemented when the community have time.

How I reemplace "gulp-util.env.envName === 'production'" to use it with minimist?
My Gulpfile.js
`
var gulp = require('gulp');
var gulpif = require('gulp-if');
var concat = require('gulp-concat');
var sourcemaps = require('gulp-sourcemaps');
var uglify = require('gulp-uglify');
var util = require('gulp-util');
gulp.task('clean', function () {
return gulp.src([
'./src/main/resources/static/js/**/*.js'
], {read: false})
.pipe(clean());
});
gulp.task('default', function () {
var uglifyFlag = util.env.envName === 'production';
return gulp.src([
    './src/main/web/app-definition.js',
    './src/main/web/**/!(app-definition)*.js'
])
    .pipe(sourcemaps.init())
    .pipe(concat('app.js'))
    .pipe(gulpif(uglifyFlag, uglify({mangle: true})))
    .pipe(gulpif(uglifyFlag, sourcemaps.write()))
    .pipe(gulp.dest('./src/main/resources/static/js/'));

});
`


Fix for #714

Add handler interceptors to builders within EventProcessingConfiguration. Reason for this is if we reinitialize the configuration, interceptor gets registered with the EventProcessor.
This PR resolve #714.

@m1l4n54v1c feel free to merge it as soon as Travis is happy.

I'm trying to figure out why is Kafka test occasionally failing. Will fix it on this branch.

Unfortunately, I didn't manage to understand why failing Kafka test fails in some cases (why it takes a significant amount of time to execute). Since it is not related to this issue, I'll merge this one.


Turn bool into a numeric type?

Should we turn bool into a numeric type to match Python's behaviour? This means all of the following equalities would hold:
True == 1
False == 0
True + True == 2
True - False == 1
2 * False == 0
...

See #263 for a PR that implements this.
Personally, I'd be in favour to increase consistency with Python. It also allows some nice patterns like
results = [measure(q) for q in qs]
parity = sum(results) % 2

What do you think? @croyzor @qartik @ss2165

If we are considering guppy as a strongly typed language, I think bool should not be implicitly treated as a numeric type. We could certainly provide casting/conversion functions for explicit coercion, if they don't exist already.

Originally posted by @qartik in https://github.com/CQCL/guppylang/issues/225#issuecomment-2139705909
Type-theoretically speaking, I wouldn't want to allow the equalities in the OP.
The pattern shown is very easy and way more explicit with a coercion function:

- results = [measure(q) for q in qs]
+ results = [int(measure(q)) for q in qs]
  parity = sum(results) % 2


ssh: add support for extension negotiation (rfc 8308)

This PR is cherry-pick from https://github.com/golang/crypto/pull/211

I will close it for now. We decided to wait for official fix in upstream.

@akosyakov  Why the need to wait? I would say this is a blocker to make gitpod-desktop extension GA, we don't know how many users have newer ssh client, without this all users using ubuntu 22.04 which is LTS version and with rsa keys wouldn't be able to connect and the error shown by the ssh client is not really helpful

I think the reasoning was because https://github.com/golang/crypto/pull/211 in progress, not something what was approved by authors. I'm not sure about the quality and completeness of the solution. I added to our sync point next monday.

I think we probably can't wait for the official PR https://github.com/golang/crypto/pull/211 merge, on the one hand, this PR hasn't had a new commit for 2 months and the last comment was a month ago, no one knows when it will be merged
Also the latest beta for macOS has upgraded the ssh-client to openssh 9.0 which means the latest macOS 13.0 will have a ton of people having this problem by the time it's released in the fall, so we may have to merge it ourselves and watch for official movement


Bug in ZigBeeEndpointImpl

Hi there.
A couple of days ago, I started to play with this great project and a netvox zigbee device.
I found what seems to be a little bug in the bindTo method of the ZigBeeEndpointImpl class.
When you are building the ZDO_BIND_REQ packet, the last parameter is declared as BindDstEP , but you are sending is endpoint.getDeviceTypeId().
I think this is a  bug, as it should be sending endpoint.getEndPointAddress().
I will try to send a patch for your consideration.
Thanks you very much for your time and work.
Regards!!!!

@alcastelo this looks like a valid fix. Would you like to make a pull request?

Thanks a lot.
I´m making a pull request right now!!!

Thank you :+1:

Thanks to you for your work!! :+1:


Endpoint: /v2/wvw/matches/stats/:id/teams/:team/top/kdr

The endpoint is broken.

See https://github.com/gw2-api/issues/issues/9 for reference.


No range allowed when using Preview command with visual selection

I am using the latest version of this plugin.
nvim version: 0.10.1
system: macOS

I do not have special settings, this is my config using lazy nvim:
  {
    "smjonas/live-command.nvim",
    -- live-command supports semantic versioning via Git tags
    -- tag = "2.*",
    config = function()
      require("live-command").setup()
    end,
  },

If you select a range, then use Preview command, I am seeing the following error:

E481: no range allowed.

This is intentional. The Preview command is meant to preview the argument passed to it exactly as provided.
So you could do pass the range like :Preview 5,10norm daw instead of :5,10Preview norm daw. On the Reddit post, someone posted about this as well. Maybe I should Change this behavior as I agree it can be unintuitive, especially in visual mode.

Thanks for the reply, my confusion comes from the usage section: https://github.com/smjonas/live-command.nvim?tab=readme-ov-file#basic-usage
There it specifically says you can use Preview command with ranges. Maybe it should be updated as well 😄

Oh thanks, I'll update the documentation


How can I change First day of the week?

How can I change First day of the week?

Use FlexibleCalendarView#setStartDayOfTheWeek.


Simplify LB policy refcounting.

This PR eliminates the two types of refs for LB policies (strong vs. weak) and replaces them with a single type of reference.
To make this work, we ensure that there is only one external owner of the LB policy, which is responsible for explicitly calling glb_lb_policy_shutdown_locked().  Prior to this PR, there were two external owners of an LB policy, both in the client_channel filter: the filter's channel_data struct was the primary owner, but we were also holding a reference in the filter's call_data so that when we switch to a new LB policy, we don't shut down the old policy until it has finished the picks that were already in flight with that old LB policy.  This PR eliminates the reference held by the call_data struct and provides a different solution for switching LB policies: now, when the old LB policy is shut down, if there is a new LB policy created, it hands off any pending picks to the new LB policy.
In order to make it easier for one LB policy to hand off picks to another, I moved all parameters used in a pick to a single struct, called grpc_lb_policy_pick_state.  This wound up having a very nice side effect, which is that this struct can double as the pending_pick struct inside the PF and RR policies, thus eliminating some unnecessary allocations.  The logic in the grpclb policy was more complex, so a wrapper struct was still needed there, but even in that case, the logic is much simpler now (e.g., I was able to combine the pending_pick and wrapped_rr_closure structs), and I've eliminated some unnecessary allocations there too.
While I was working on the grpclb code, I also eliminated the need to hold a ref the RR policy for each pick.  Since we no longer have any cases where the RR policy will be shut down out from under us, these references are no longer necessary.  I was also able to greatly simplify the pending_pings struct.


model longformer multi-card training evaluation process did not carry out multi-card average

In the multi-card training, it is necessary to carry out intermittent model accuracy evaluation, but the longformer model did not average the evaluation results during the multi-card evaluation process. Although each card does full data inference, it is not guaranteed that the inference results of all cards are consistent (due to the existence of random factors such as drop out). As a result, the evaluation results of different cards are different, and probability affects the state of state, leading to the stop of training after some cards reach the accuracy, while some cards do not reach the accuracy and continue to train, resulting in the illusion that the machine is hung. It is necessary to all_gather the information of all cards in the evaluation process and then average.

Thanks for the report @happyxuwork. please provide a screenshot of the running process or a code snippet. Besides, please @reiase confirm if the status is True. If True, here's some solution:

please @happyxuwork raise a PR to fix this, i'll reivew and merge it.
please @reiase raise a PR to fix this.
please @happyxuwork decribed the bug in detail, i'll working on a fix later

Some log evidence:
rank0 log:

rank4 log:

as show above, different cart may have little diff, if target acc is 0.64, the rank4 will stop and other will contune, resulting in the illusion that the machine is hang, so do average between all cart  will keep the acc all time, and state of training same too.

information is acknowledged @happyxuwork. Are you willing to submit a PR for this? if not, i'll working on it later

@shh2000 Follow flagperf's other models , you unified treatment is more appropriate

@shh2000 Follow flagperf's other models , you unified treatment is more appropriate

ok, i'll working on it later

@shh2000 you can just replace https://github.com/FlagOpen/FlagPerf/blob/03d762cd472591783520c73514cff2551c92a830/training/benchmarks/longformer/pytorch/train/evaluator.py#L42C9-L42C42 as fellow, can fix this
acc = total_output / num_examples
acc = reduce_tensor(acc)
def reduce_tensor(tensor):
rt = tensor.clone()
if dist.is_available() and dist.is_initialized():
dist.all_reduce(rt, op=dist.ReduceOp.SUM)
rt /= dist.get_world_size()
else:
return tensor
return rt

@shh2000 you can just replace https://github.com/FlagOpen/FlagPerf/blob/03d762cd472591783520c73514cff2551c92a830/training/benchmarks/longformer/pytorch/train/evaluator.py#L42C9-L42C42 as fellow, can fix this
acc = total_output / num_examples
acc = reduce_tensor(acc)
def reduce_tensor(tensor):
rt = tensor.clone()
if dist.is_available() and dist.is_initialized():
dist.all_reduce(rt, op=dist.ReduceOp.SUM)
rt /= dist.get_world_size()
else:
return tensor
return rt

add new pr to fix this https://github.com/FlagOpen/FlagPerf/pull/499

Thanks to your PR #499
#500 is the same as #499 . I've merged it into an temporary branch, and metax #502 has also been merged.
#499 would still remain opened for the developer of longformer. He'll check the status. If True, the temporary branch would be merged into main.

Longformer-nvidia's developer has already confirmed the status, this issue would be transfered to https://github.com/FlagOpen/FlagPerf/pull/532, this PR will be merged soon.


[flash_ctrl] Slight restructuring of the flash read pipeline

A fix for #13808 would be a slight restructuring of the flash read pipeline.

Reduce the dependency of the buffers to only "WIP" situations. Ie, the buffer is being filled. These should be blocking, ie, if all buffers are filling, we cannot accept a new transaction, since all resources are in use.

If a buffer is "valid" however, it does not need to block.  The transaction can directly takes it data and enter the pipeline. (Similar to a normal read path where the "read" is skipped).  This de-couples the transaction from needing the buffer to stay allocated and allows it to be freed up for a new transaction.

A workaround has been implemented in https://github.com/lowRISC/opentitan/pull/13815 to fix this, although the solution proposed in this issue here would be better. Leaving for a future release.


Create rohit_pr.md

Hi GSSoc Team
This is the PR I have raised
Can you review it and merge it
POSTMAN CHALLENGE
@sanjay-kv @MastanSayyad
https://github.com/GSSoC24/Postman-Challenge/pull/4119

@student-nitjalandhar merged


Box VTag to reduce size disparity between VNode variants

fixes https://github.com/yewstack/yew/issues/571
(breaking change)

Thanks @hgzimmerman! Looks like the clippy warning was complaining about VTag not VComp though 😉

Wow, missed that one, will update with the appropriate change.

Looks good! Can you please rebase though and remove the clippy allow I added recently? I also enabled clippy in CI so we can verify that this fix passes CI 😄


Compatible Windows version unclear

Hi!
I was reading this article at docs. microsoft.com and it statest that the policy "Specify deadlines for automatic updates an retarts" is recommended for Windows 10 versions 1709 and above.
If I want to deploy this setting via GPO, it states this GPO is supported on "At least Windows Server 2016, Windows 10 Version 1903". Github file confirms this. Can you update the official page?

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

ID: 44aabe97-360e-0655-09ec-5d1e352bdb1a
Version Independent ID: 1c729c42-741d-ecb7-6490-a4bfdd029263
Content: Erzwingen von Compliance-Terminen mit Richtlinien in Windows Update for Business (Windows 10) - Windows Deployment
Content Source: windows/deployment/update/wufb-compliancedeadlines.md
Product: w10
Technology: windows
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

@arcarley can you confirm or advise?

@lechku the Group Policy template unfortunately is just showing what release this policy was added in (1903), but we in fact backported this policy to 1709 and below to ensure all customers have access to such. You can set this policy for any device on 1709 and above. I will work to get the template versions allowed adjusted accordingly.

@arcarley Thank you for this clarification!
So changing my 1809 GPO should work?  I  do not need to modify the .admx first, right?

Correct, any 1809 device should be able to have the "Specify deadlines for automatic updates and restarts" policy set on it. (It should even be available in local GP or you can use the newer ADMX templates)

Closing via the answers above. Thank you.


Get Started on Web Fundamentals home page is broken

When you press Get Started on the Web Fundamentals home page, it goes to this page, which is empty now: https://material-dot-web-central.appspot.com/web/fundamentals/getting-started.html
<EMAIL_ADDRESS>for finding this.

Fixed in #2059 and #2061


Popupmenu is behind taskbar

Hi.
I tried to use your FMXTrayIcon component. It works well except for one minor flaw: the associated popup menu always appears behind the traybar icon window.
(See image)
Thank you
Image

Resolved by adding...
    ::SetWindowPos(
        Fmx::Platform::Win::ApplicationHWND(),
        HWND_TOPMOST, 0, 0, 0, 0,
        SWP_NOSIZE | SWP_NOMOVE | SWP_NOACTIVATE
    );

after
    trayIcon_->Show();

Example:
void TfrmMain::MinimizeToTrayBar()
{
    Hide();
    trayIcon_->Show();

    ::SetWindowPos(
        Fmx::Platform::Win::ApplicationHWND(),
        HWND_TOPMOST, 0, 0, 0, 0,
        SWP_NOSIZE | SWP_NOMOVE | SWP_NOACTIVATE
    );

    HideAppOnTaskbar();
}

Regards

Oh, thank you very much. I have not made edits for a long time, and I myself have used it for a long time. But I think it's time to fix it. Thanks again. Maybe today I will fix it using your suggestion

Hi.
I noticed that after minimizing the main window in the tray icon as per the code in the previous posts, when I would restore the window it would always remain topmost, always overstanding all other applications. Since this is an annoying problem, I solved it by declaring HWND_NOTOPMOST in the call to SetWindowPos when restoring from the tray bar (by double-clicking on the tray icon).
void TfrmMain::RestoreFromTrayBar()
{
    ::SetWindowPos(
        Fmx::Platform::Win::WindowHandleToPlatform( Handle )->Wnd,
        HWND_NOTOPMOST,
        0, 0, 0, 0,
        SWP_NOSIZE | SWP_NOMOVE | SWP_NOACTIVATE | SWP_SHOWWINDOW
    );
    trayIcon_->Hide();
    ShowAppOnTaskbar();
}

Note, however, that this time I did not use the application handle, but the main window handle.
To recap: when minimizing you should use the application handle and the HWND_TOPMOST flag in the SetWindowPos, and when restoring you should use the main window handle and HWND_NOTOPMOST when calling the SetWindowsPos.

Fixed popup menu display. Do additional actions before or after - not necessary


Add scheduling support to interactions

See #1205.
This PR also improves test coverage of interactions.

Coverage increased (+0.3%) to 65.863% when pulling 53b470b9faa008a9338a46ffd682df78b4fc8b9c on kentcb:Issue1205 into e41af92acf5cf3051a7413538c5f7f35e728844b on reactiveui:develop.

Howdy @aalmada can you review this and continue the conversation over at https://github.com/reactiveui/ReactiveUI/issues/1205#issuecomment-275986362 then reflect on the implementation in this pull-request?

Coverage increased (+0.3%) to 65.874% when pulling 97b3d4cd65f9de1110b7d1dea130eb3348df49a6 on kentcb:Issue1205 into a7c7f9cec78d6d23092279598a89c74aa5dd862a on reactiveui:develop.


Switching a RootViewController while PKHUD is showing causes switching back.

func switchRootViewController(rootViewController: UIViewController, animated: Bool, completion: (() -> Void)?) {
    if let window = UIApplication.sharedApplication().keyWindow {
      if animated {
        UIView.transitionWithView(window, duration: 0.5, options: .TransitionCrossDissolve, animations: {
          let oldState: Bool = UIView.areAnimationsEnabled()
          UIView.setAnimationsEnabled(false)
          window.rootViewController = rootViewController
          UIView.setAnimationsEnabled(oldState)
          }, completion: { (finished: Bool) -> () in
            if (completion != nil) {
              HUD.hide(false)
              completion!()
            }
        })
      } else {
        window.rootViewController = rootViewController
      }
    }
  }

Above is a function that I called to switch a rootViewController.
When my app switches its rootViewController while PKHUD is showing, it switches back to a previous rootViewController as soon as the PKHUD is hidden. Any suggestions to avoid this issue?

+1


Add Korean translation

could you pull from staging instead of new-design. currently it will merge new-design into staging


m2mnsdlinterface.cpp: added mutex calls to protect sn_nsdl_process_coap and sn_nsdl_exec

Added __mutex_claim before and __mutex_release after sn_nsdl_process_coap and sn_nsdl_exec as they access same resources.

rebuild


[EXAMPLES] fix typo.

Can one of the admins verify this patch?

Good catch. Trivial fix and this would otherwise prevent this example from working. I'm merging into 1.3 and master.

Thanks.

@srowen This affects the Spark 1.2.1 release as well.  (wasn't present in Spark 1.2.0 release.)  Please merge into 1.2 branch.

Created JIRA here: https://issues.apache.org/jira/browse/SPARK-6167


Country search prioritizes country code

 [TARIFF17] Searching for country should look up country code before doing a full text search


Update login test in sync with other test

What is the purpose of this change? What does it change?

Was the change discussed in an issue?
fixes - part of #1473

How to test changes?

make test-odo-login-e2e

/retest

/refresh

/test e2e-scenarios

/test integration

Agreed with @cdrage. This a not a good approach.

@kadel @cdrage The approach I was following like I followed in minishift. There the credential are passed through env variable and managed by Make file and then processed by the concern go file. Here in my changes I could not implemented env variable processing stuff through go file due to open issue https://github.com/onsi/ginkgo/issues/440 as i mentioned in one of my comment. My intention was to implement such a process which was very successful in minishift repo.

How about separating the login and logout tests completely and leaving rest on the user.
What I mean by that is that only login and logout integration test will perform login and logout commands. Normal tests will expect that they are already logged in.

Yup, Works for me

If the developer is running the test from terminal he/she will have to make sure to log in.
Scripts executing tests on CI will do the same.

+1, so i am closing the issue #1396 an it make no sense now onwards

I think that makefile wouldn't ever contain odo login, or matter of fact, any other odo commands.

Absolutely, due to certain ginkgo constraint https://github.com/onsi/ginkgo/issues/440#issuecomment-511294531 i added the login stuff in Makefile thinking as a temporary change.

Absolutely, due to certain ginkgo constraint onsi/ginkgo#440 (comment) i added the login stuff in Makefile thinking as a temporary change.

login stuff shouldn't be in anywhere in the test suites (expect login integration test).
So the ginkgo constrains shouldn't affect anything.

If the developer is running the test from terminal he/she will have to make sure to log in.
Scripts executing tests on CI will do the same.

+1, so i am closing the issue #1396 an it make no sense now onwards

If everything else works, and I can just do
oc login
make test-integration

sure lets close it

Absolutely, due to certain ginkgo constraint onsi/ginkgo#440 (comment) i added the login stuff in Makefile thinking as a temporary change.

login stuff shouldn't be in anywhere in the test suites (expect login integration test).
So the ginkgo constrains shouldn't affect anything.

If the developer is running the test from terminal he/she will have to make sure to log in.
Scripts executing tests on CI will do the same.

+1, so i am closing the issue #1396 an it make no sense now onwards

If everything else works, and I can just do
oc login
make test-integration

sure lets close it

Awesome!!!
Question: Why oc login why not odo login ?. I believe odo login works the same way as oc login does for normal user.

Question: Why oc login why not odo login ?. I believe odo login works the same way as oc login does for normal user.

Yes, it should be the same. I just use oc because sometimes I don't have odo compiled yet :-)
Also, I usually use copy login commands from openshift web console. So it is just copy/paste for me,  and right now web console shows just the oc commands.
So no real reasons, just muscle memory and laziness :D

Absolutely, due to certain ginkgo constraint onsi/ginkgo#440 (comment) i added the login stuff in Makefile thinking as a temporary change.

login stuff shouldn't be in anywhere in the test suites (expect login integration test).
So the ginkgo constrains shouldn't affect anything.

If the developer is running the test from terminal he/she will have to make sure to log in.
Scripts executing tests on CI will do the same.

+1, so i am closing the issue #1396 an it make no sense now onwards

If everything else works, and I can just do
oc login
make test-integration

sure lets close it

Awesome!!!
Question: Why oc login why not odo login ?. I believe odo login works the same way as oc login does for normal user.

I don't see anything with regards to using oc login and oc logout in Minishift: https://github.com/minishift/minishift/blob/master/Makefile
The problem with the current approach in this PR is that it assumes that the user is running minishift which may not be the case..
I do agree with @kadel that putting the login and logout commands in a separate test that can be ran by the user :+1:

I don't see anything with regards to using oc login and oc logout in Minishift: https://github.com/minishift/minishift/blob/master/Makefile

This is not the specific use of oc login or oc logout this is more about how environment variable is being used smartly in the Makefile https://github.com/minishift/minishift/blob/master/Makefile#L264-L284.  As i mentioned above the change i have done i mean login and logout in Makefile was temporary due to issue https://github.com/onsi/ginkgo/issues/440#issuecomment-511294531 (https://github.com/openshift/odo/pull/1875#discussion_r303310349). Anyway we are using a separate testfile for login , however few usecase are missing (For example: odo logout) in the test file. For now i will update the test file with some more cases, later will fix with all possible usecase if any as part of #1845

@kadel @cdrage Done with the requested changes. PTAL

Ping @mohammedzee1000 - https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/openshift_odo/1875/pull-ci-openshift-odo-master-unit/1441
This is new

/test unit

/test integration

Tests are failing.
Let's see if it's a false positive
/retest

/test integration

/test integration

/test integration

ping @cdrage @kadel

I am just worried about the 3 runs that took us to get the CI to pass @amitkrout. might wanna reduce the concurrency?
cc @kadel
removing the hold in the meantime
/hold cancel

@girishramnani I agree with Girish with regards to the retesting. Is there a reason for the flakes?

@girishramnani I agree with Girish with regards to the retesting. Is there a reason for the flakes?

@cdrage Yes, we have flake. These flake are from from OpenShift CI
level=fatal msg="failed to fetch Terraform Variables: failed to fetch dependency of \"Terraform Variables\": failed to fetch dependency of \"Bootstrap Ignition Config\": failed to fetch dependency of \"Common Manifests\": failed to generate asset \"DNS Config\": getting public zone for \"origin-ci-int-aws.dev.rhcloud.com\": No public route53 zone found matching name \"origin-ci-int-aws.dev.rhcloud.com\""

and similar such failure which we don't have any control over it except retesting. Other flake is coming from an open issue #1832. However a recent flake (occurrence is very less) is introduced.
[odo]  ✗  Waiting for build to finish [30s]
[odo]  ✗  Failed to create component with name ref-test. Please use `odo config view` to view settings used to create component. Error: unable to wait for build ref-test-app-1 to run: timed out waiting for the condition

Not sure is it a real threat or something need to be handled in odo code.These flakes are no where related to this pr change. I would say please review.
Anyway we are talking these flake in separate issues cc_ @kadel

I am just worried about the 3 runs that took us to get the CI to pass @amitkrout. might wanna reduce the concurrency?

@girishramnani please refer https://github.com/openshift/odo/pull/1909#issuecomment-511667494 and https://github.com/openshift/odo/pull/1909#issuecomment-511729801 and feel fee to add your opinion.

/test benchmark
ping @kadel @cdrage

/test integration

/test integration

/approve

I am just worried about the 3 runs that took us to get the CI to pass @amitkrout. might wanna reduce the concurrency?

@girishramnani please refer #1909 (comment) and #1909 (comment) and feel free to add your opinion.

Decrease the test node to 2 as per our cabal discussion


Backport documentation generation to v0.10.x

See #25 for details.

ci/labeler: Add missing noxfile.py
Use MkDocs to build the documentation
Move copyright and license as code comments
docs: Add missing file format to examples
docs: Add some useful cross-references.
docs: Indent examples
Use forward type annotations
docs: Indent "Returns:" properly
docs: Improve formatting of some symbols
docs: Make some minor improvements
docs: Add site version via mike
ci: Remove unnecessary checkout of submodules
ci: Build and publish documentation
Run normal linting on noxfile.py
Run mypy on noxfile.py and be more restrictive
ci: Add contents permission to publish-docs

Does this push to the gh-pages branch as well?  Would it write to a different directory, so that it doesn't overwrite the docs for the other versions?

Does this push to the gh-pages branch as well? Would it write to a different directory, so that it doesn't overwrite the docs for the other versions?

Yes, this is what mike is for. It basically checks-out the current gh-pages branch, generates the docs in the directory specified by <version> in mike deploy , overwriting that version but without touching other directories, so documentation for other versions will be left untouched, then commits. If --push` is used, it will also push, but we could also push manually using git.
Then it uses aliases for versions. I'm using latest to point to the last stable release, which is also the default (what you get if you go to the / without pointing to a specific version). I'm also using v0 to point to the latest v0.x and generating only vMAJOR.MINOR versions (so no specific docs for point releases).
mike set-default <version> just creates a index.html with a redirection. You can see this in the gh-pages branches history; here is the commit setting the default: https://github.com/frequenz-floss/frequenz-channels-python/commit/27dab7bcffc91ddb0b25b9290c65b577530069ec.

FYI one check is missing because we didn't support Python 3.11-dev in v0.10.x, I don't think it is worth changing. Maybe we can add a dummy ci job that depends on all the tests we want the PRs to pass and only have that job as a requirement for PRs, so this doesn't happen while we add/remove python versions...

because we didn't support Python 3.11-dev

we also need to switch to the stable version of 3.11, now that it has been released.

we also need to switch to the stable version of 3.11, now that it has been released.

Yeah, I was planning to do so after all this.


remove double wording

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@bartvermeersch  - please merge the commit on the copy/edit review done by JohanFreelancer9 or make the necessary changes on the PR following the suggestions made. Thank you.

@chrisda  - PR has been copy edited and is ready for final review.
Thank you.
cc: @kenwith


fix: update to eye v4.4.7

fix: update to eye v4.4.7

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

npm package (@latest dist-tag)
GitHub release
v5.4.7

Your semantic-release bot :package::rocket:


Handle exec clingy children

This fixes a few things:

prevent containerd from ignoring requests in the case where an exec process spawned children that are keeping the IO open
correctly reap children of exec processes before exiting the shim to avoid zombies
ensure that we correctly wait for our children when we received a SIGCHLD. Before with WNOHANG it was possible for us to return without wait having actually done its job (it would return 0 as pid)

--
Addresses docker/docker#26403 and docker/docker#26506

LGTM

https://github.com/docker/docker/issues/26561 may be also related

For reference, when DockerSuite.TestHealth was getting deadlocked, 2 sleep processes could be seen:

in the defunct state which had been reparented to containerd
one in a disk sleep (but dead) that was still parented to his initial shim.

The shim of the 2nd sleep wasn't receiving a SIGCHLD due to the fact that the 1st one had joined its namespace which were preventing the kernel to completely kill it.

Hum, containerd will also have to try and reap left over children from an exec in case the shim get sigkilled. I'll have to add another commit to this.

LGTM

ping @crosbymichael

LGTM


Consider Packaging this for the AUR?

This is the only working package for msi steelseries laptop keyboards that i've found that works for linux
(it's a great package)

Want to send a PR to add the required file/setup script? I can then publish with CI/actions.
https://wiki.archlinux.org/title/creating_packages

Though, would you want this package or the GUI? https://github.com/stephenlacy/msi-keyboard-gui
This is just the library.


.empty rule is active on second monitor even when there are windows

If I understand it, .empty should be for empty workspaces without any windows. I have 2 monitors and when I focus on a window on the second monitor, the .empty style is applied to the unfocused monitor.

sway 1.5
waybar 0.9.3

Any news on this ? Does anyone have a workaround ?


net/http/fcgi: eliminate race, keep request id until end of stdin

There was a race condition that could lead to child.serveRequest
removing the request ID before child.handleRequest had read the empty
FCGI_STDIN message that indicates end-of-stream which in turn could
lead to child.serveRequest blocking while trying to consume the
request body.
Now, we remove the request ID from within child.handleRequest after
the end of stdin has been detected, eliminating the race condition.
Since there are no more concurrent modifications/accesses
to child.requests, we remove the accompanying sync.Mutex.

A more detailed description (including reproducer) can be found in https://github.com/hillu/go-fcgi-breakage.


[SPARK-28849][CORE] Add a number to control transferTo calls to avoid infinite loop in some occasional cases

This patch propose a way to detect the infinite loop issue in transferTo and make it fail fast. The detailed issue is shown in https://issues.apache.org/jira/browse/SPARK-28849.
Here propose a new undocumented configuration and a way to track the number of transferTo calls, if the number is larger than the specified value, it will jump out of the loop and throw an exception.
Typically this will not be happened, and user don't need to set this configuration explicitly, in our environment it happens occasionally and cause the task hung infinitely.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14626/
Test PASSed.

Test build #109569 has started for PR 25552 at commit 50bb20d.

Test build #109569 has finished for PR 25552 at commit 50bb20d.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14930/
Test PASSed.

Test build #109902 has started for PR 25552 at commit 4cf98b8.

Test build #109902 has finished for PR 25552 at commit 4cf98b8.

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

Merged build finished. Test PASSed.

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

Build finished. Test PASSed.

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


Define our user groups, pilot partner, research partner, vendors, etc.

Lead: Drew
In-progress Mural board
https://app.mural.co/invitation/mural/primedatainput6410/1601833898368?sender=ud6a3555dd1821b626a2b3726&key=24438ceb-5133-42e7-8937-a40e547b5050
Partners
Pilot partners are jurisdictional public health departments we are actively working with to develop software product(s). Observations, input, and feedback from research and testing will contribute directly to what we build.
Research partners are jurisdictional public health departments we are engaged with in a learning capacity. Observations, input, and feedback from research and testing will inform and influence what we are building with our Pilot partners.
Users

Public health departments (state, county) [Data Input, Data Hub]

Public health leadership, public health employees, vendors and contractors

Testing locations: people actively working in day-to-day operations, people having tests administered [Data Input]

Mobile
Non-traditional
etc

"HQ": people ingesting or receiving files, quality checking or cleaning files, transforming data and taking action, or conducting analysis and reporting [Data Hub]

Epi
Data Analyst
IT

Hospitals [Data Automation]

Hospital leadership, hospital employees, vendors and contractors

Lab staff
IT
???

To do

[ ] Add to our published docs
[ ] Follow up with Tom S, who originally surfaced this

Gathering my thoughts on data automation stakeholders prior to our 10am meeting:
Data Automation End Users (users of the automation solution)

Hospital employees collecting and reporting data for the COVID-19 response
Lab employees collecting and reporting data for the COVID-19 response
State public health employees using this data and reporting it to the federal government

Data Automation Vendors (providers/enablers of the automation solution)

EMR / ERP Vendors (Epic, Cerner, AllScripts, Meditech)
Third Party Integrators (Juvare/EMResource, Cloverleaf, Appriss, National Response Portal, UI Path, Vizient, Smartsheet)

Caveat for Data Automation "pilot partners": The term "pilot" is used more broadly for Data Automation vs. the other PRIME projects. At the state level, a pilot partner is a state we are partnering with to pilot automation solutions within their state. At the hospital level, a pilot partner is a hospital or health system we are partnering with to pilot an automation solution (not necessarily in a pilot state).

Data Hub End users

Data analysts receiving, formatting, manipulating, editing lab report files and uploading into systems
Epidemiologists who perform actions based on data, such as contact tracing (everyday health actions) and analysis and reporting

Enablers/Stakeholders

Testing labs administrators who collect information at point of care, create flat files
Testing lab IT team who build/sign off on integration systems and workflows
Public Health IT team who build/sign off on systems and workflows
Public Health leadership

@drew-usds Is this pretty much done at this level? Each team has kind of started working on this independently, so I think we can close it?

Considering this closed!


zabbix: Support for 5.4

Motivation for this change
Zabbix' 5.4 branch wasn't previously supported
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

This adds zabbix54 as extra available packages, zabbix50 stays the default package. I've tested the packages with them being pulled in by NixOS' configuration.nix and so far couldn't find any errors.
I don't really know much about git or nix so I hope everything's alright :sweat_smile:

I'm sorry but we can't support this software! EOL is Jan 30, 2022, which is during the release cycle of 21.11. See https://www.zabbix.com/life_cycle_and_release_policy for details.
We should add this to NUR instead. If you don't want to do that I could.

Ah, so nixpkgs-unstable still follows the stable bi-yearly releases instead of being rolling release on bleeding edge? When 6.0 comes out, it would replace this version as the latest one (like linuxPackages_latest). Would renaming the package to zabbix_latest work?

We don't allow breaking changes during a 6 month release cycle of NixOS - so we can't update zabbix_latest to a different major version for the next 6 months. This means we end up with an unsupported version of zabbix shortly into our release cycle.
What do you think about adding this to NUR?

@mc-fugger this is great work you are doing, but we still need to drop the 5.4 release because it is going EOL in a few months. Can you please respond to my NUR comment?

Yes, I was hoping for a different answer but to the NUR it is then :grin:
Would you like to add it there or should I read up on it? Also, what will happen to the package in the NUR when 6.0 releases, making 5.4 in the NUR obsolete?

@mc-fugger sorry I can't give better news, it is just that we can't remove the 5.4 release a month or two into the NixOS 21.11 release. However we can add the 6.0 release to NixOS unstable (and stable) once it is out in a few months.
I don't have any repositories on NUR. I maintain my own 5.4 expression and deploy it myself, so you can either do that or submit it to NUR if you think it is useful to others. Either way it won't be necessary much longer... 6.0 is due out soon.
Your PR makes all of this much easier, so personally I really appreciate your work here! Thank you! :sparkle:

6.0 is out - https://www.zabbix.com/whats_new_6_0

5.4 has been EOL for a whole year: https://endoflife.date/zabbix


Movie Recommendation system

Hi, I am  15IITian , as a ML enthusiast , want to create a ml model which will recommend movies based on user interest which is a crucial feature of many famous brands like Netflix.
Dataset link-https://www.kaggle.com/datasets/grouplens/movielens-20m-dataset
Please assign this interesting project to me:)

Please approve my contribution. It works perfectly.


Make person headshots and bios editable in the admin interface

We currently add these by hand, but we can set them up to be manageable by Metro admin. Thoughts, @shrayshray?

@hancush that would be awesome.


Change to dbconn.php constructor class on dev branch

I couldn't figure out how to do a pull request.
I received the following error -
PHP Fatal error:  Uncaught Error: Call to protected DbConn::__construct() from invalid context in /var/www/html/login/ajax/createuser.php:31\nStack $
I fixed it by changing line 98 protected function __construct() to  public function __construct()
It appears as though this was committed @Brad Magyar and changed recently. It used to be simply function__construct() - I am not sure the repercussions of making this public. Someone please review

Any input?

Your change is fine, it's good habit to define visibility which is why someone will have opted to update to protected, however public will also work. Here are the difference between visibility types:

public:  your variable/function will be available from anywhere, other classes and instances of the object.

private:  your variable/function will be visible in its own class only.

protected:  your variable/function will be visible in all classes that extend current class including the parent class.

For more information you can read up on the difference here:
http://php.net/manual/en/language.oop5.visibility.php

@SammF10930 - I guess my follow-up question would be why would I receive that error if it was possible to be protected? The maintainer of this project needs to move it somewhere else for it to be available elsewhere in the code, no?

Okay - getting more into object oriented php as opposed to my experience in linear.
So what's happening here has to do with class inheritance I think.
class NewUser extends dbconn, but dbconn's function __constuct() is protected.
@SammF10930 you're saying that protected functions will be visible in all classes that extend the current class (dbconn is current, NewUser extends dbconn)  therefore why is that error happening?
Per @fethica code, he/she is taking an extension of class dbconn as NewUser and trying to use it's var's properly. I'm not quite sure what php is complaining about then.

I haven't tried it - but is there a way to pull the createUser function out of NewUser by using a getter? $a->createUser(); instead of instantiating NewUser? Maybe I'm onto the wrong idea here.

@zackoch I don't know how I missed this from so long ago, sorry about that. I'm the primary developer on this project and it's hard to keep up with all of the issues in my spare time. I ended up changing visibility of the constructor to public because I was running into inheritance issues also. My general rule of thumb is to lock things down to private or protected unless I absolutely need them to be public.
Since you asked this, I've made a lot of changes to the code base in general and consolidated the class files to be a bit more sane and logical, so the NewUser class has been merged into UserHandler and the createUser() method is static, so you don't actually need to instantiate anything to use it. You can just call it like so: UserHandler::createUser(array $userarray);
You can investigate the code of createUser to see what data actually needs to be inside of the user array parameter. Hope that helps!


[OSD-8686] Convert v4-registry-s3-size script to managed-script

https://issues.redhat.com/browse/OSD-8686
Also fixed the original script from ops-sop here: https://github.com/openshift/ops-sop/pull/1563

the script requires aws cli so installing that in the Dockerfile

/lgtm
/approve


perf: disable kvcache for semantic by default

An analysis of my previous patches suggests that use_kv_caching=True is not a win at all for the text model.
The current KV caching code has quite a bit of overhead. In particular, it (re)allocates large tensors with adding to the cache via a torch.cat call. With the coarse model it seems to result in some performance gain, but it is definitely not a win for the smaller text model. It seems that the KV cache was what was causing the bimodality in #366, I'm guessing sometimes we would get unlucky and the model would have to reallocate.
With this change I can consistently get about 280 it/second performance for the text model on an H100 after warmup.

Hm, nevermind here, I think I drew an invalid conclusion due to a mixup. It seems that caching still results in a small performance gain on the text model.


lattice draw displays in web UI

Added

UI: Lattice draw() method displays in web UI
UI: New navigation panel

Changed

UI: Animated graph changes, panel opacity

Fixed

UI: Fixed "Not Found" pages

[x] I have added the tests to cover my changes.

[x] I have updated the documentation, VERSION, and CHANGELOG accordingly.

[x] I have read the CONTRIBUTING document.

@valkostadinov quick comment, Instead of removing the previous draw function, can you add it as draw_inline() ? It is still useful to have the local draw version in hand as this one requires a start of server.

It's there already, I put it in there per your request earlier. Let me know if there's anything else I need to fix.
https://github.com/AgnostiqHQ/covalent/blob/111a14e575a668b3ec6f2260dfb0589a2271d0ac/covalent/_workflow/lattice.py#L152


[BUG] Incorrect checksum for freed object

What happened?
run pnpm build got below error
(node:8289) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)
node(8289,0x700005708000) malloc: Incorrect checksum for freed object 0x7f91791ed800: probably modified after being freed.
Corrupt value: 0x0
node(8289,0x700005708000) malloc: *** set a breakpoint in malloc_error_break to debug
 ELIFECYCLE  Command failed.
This happens after I delete build folder manually and add a new file at the same time.
Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
Chrome
Relevant log output
(node:8289) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
node(8289,0x700005708000) malloc: Incorrect checksum for freed object 0x7f91791ed800: probably modified after being freed.
Corrupt value: 0x0
node(8289,0x700005708000) malloc: *** set a breakpoint in malloc_error_break to debug
 ELIFECYCLE  Command failed.

(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

same as https://github.com/PlasmoHQ/plasmo/issues/1060


Stack overflow exception in Websocket.Client library

I get a stack overflow crash in our program by doing this:

create a ZWaveJS.NET driver and connect it to an external ZWave JS Server
stop the ZWave JS server and let the ZWaveJS.NET driver goes into reconnection mode.
wait about 10 minutes to trigger hundreds of reconnection attempts
call driver.Destroy()

=> a StackOverflowException occurs in ClientWebSocket.Dispose()
It is 100% reproducible in our program, but I wasn't able to reproduce it with a simple console program, so I must be missing something there.
Anyway, I have traced down the problem to be related to this issue in the Websocket.Client library: https://github.com/Marfusios/websocket-client/issues/124
While debugging in VS, if you "break all" while waiting in step 3 from above, you can see the hundred of recursive calls in the task used for reconnection.
A fix for this problem has been implemented in recent version of the Websocket.Client library, but unfortunately this version has dropped support for netstandard2.0 , so we cannot directly update the library from nuget unless netstandard2.0 is dropped in ZWaveJS.NET as well, which is out of question for us because our program still use .NET Framework.
So I'm not sure how to best fix this issue?
For now, I pulled the source code for Websocket.Client version 4.6.1 (the version currently used by ZwaveJS.NET) and I manually merged the fix for the recursive call problem: https://github.com/Marfusios/websocket-client/commit/df2e007c103f248e2be3f4f35460a9033ad9469d
I successfully tested that it fixes my stack overflow problem.

Thanks @spudwebb
Looks like I'll need to include the source in to the binary to address it.
and see where I can keep support for netstandard2.0 - I'll take the opportunity to bump the schema and (hopefully) provide support for Long Range

@spudwebb
Seems trying to bend the latest socket client to net standard 2.0 is more challenging than I thought.
can you point to the source for 4.6.1?

@spudwebb
I have managed to compile the latest socket client source code with netstandard2.0 support:

Embeds the latest socket client source code
Polyfilled where not supported in netstandard2.0
A few changes in source code of the socket library

If I attach the nupkg are you able to try your luck with it?
I am not sure  if surprises are awaiting, but I don't have the means to test this currently

yes, I think I can test the nupkg

OK, see this branch: https://github.com/zwave-js/ZWaveJS.NET/tree/bump-schema-%2B-Embed-Socket-Client-%2B-LR
And the nupkg that's there also : https://github.com/zwave-js/ZWaveJS.NET/blob/bump-schema-%2B-Embed-Socket-Client-%2B-LR/Visual Studio Projects/ZWaveJS.NET/ZWaveJS.NET.4.0.0.nupkg

@marcus-j-davies, it doesn't work as is, the received messages from the websocket is always a bunch of 0.
I changed the code in WebsocketClient.Listen() to use an array as the buffer instead of Memory:
        private async Task Listen(WebSocket client, CancellationToken token)
        {
            Exception? causedException = null;
            try
            {
                // define buffer here and reuse, to avoid more allocation
                const int chunkSize = 1024 * 4;
                var buffer = new byte[chunkSize];

                do
                {
                    //  ValueWebSocketReceiveResult result;
                    WebSocketReceiveResult result;
                    var ms = (RecyclableMemoryStream)_memoryStreamManager.GetStream();

                    ArraySegment<byte> bufferSegment = new ArraySegment<byte>(buffer);

                    while (true)
                    {
                        result = await client.ReceiveAsync(bufferSegment, token);
                        ms.Write(buffer, 0, result.Count);

                        if (result.EndOfMessage)
                            break;
                    }

with this change it seems to work, but honestly I'm not super confortable using this version in production as it was not designed to work on netstandard2.0 / net48 in the first place and you had to change a few things to adapt it. I would rather embed version 4.6.1 of the Websocket.Client library in ZWaveJS.NET and only add the one fix that caused the stack overflow.
I can try to do that in a new branch if you want.

Hi @spudwebb
I think I agree - feels a bit desperate, leave it with me, as I will work on adding LR support.


set image size on import

todo

when importing image, have dialog to set the physical size of the image
remove option to resize compartment sizes automatically

resolved by #100


Add missing argument to vim highlighting function

Duplicate of https://github.com/chriskempson/base16-builder/pull/308.


scroll to 0 before rendering the new page when using next/link

Closes #6462 I created a simple Jsfiddle to demonstrate the problem.

There is some problem with hash scrolling I think, I will check it tomorrow

I will also clause this one because it end up causing more serious problem than what it solve


Trigger ‘screen_view’ event failed through GoogleTagManager

Description
Our iOS project attempted to trigger an event named 'screen_view' through GoogleTagManager. By the log, the event had been triggered, but we couldn't find the corresponding event in debugView. Once we changed the event name, such as 'test_screen_view', it would trigger successfully
Reproducing the issue
No response
Firebase SDK Version
10.16.0
Xcode Version
14.3
Installation Method
CocoaPods
Firebase Product(s)
AB Testing, Analytics, DynamicLinks, Messaging, Performance, Remote Config
Targeted Platforms
iOS
Relevant Log Output
10.16.0 - [FirebaseAnalytics][I-ACS023072] Event logged. Event name, event params: screen_view, {
    device_id = ***;
    ga_debug (_dbg) = 1;
    ga_event_origin (_o) = app+gtm;
    ga_realtime (_r) = 1;
    region_name = ***;
    screen_class = ***;
    screen_name = ***;
    user_role = ***;
}
...
...
10.16.0 - [FirebaseAnalytics][I-ACS023072] Event logged. Event name, event params: test_screen_view, {
    device_id = ***;
    ga_debug (_dbg) = 1;
    ga_event_origin (_o) = app+gtm;
    ga_realtime (_r) = 1;
    region_name = ***;
    screen_class = ***;
    screen_name = ***;
    user_role = ***;
}

If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

    pod 'Firebase/Core'
    pod 'Firebase/RemoteConfig'
    pod 'FirebaseABTesting'
    pod 'Firebase/Messaging'
    pod 'Firebase/Performance'
    pod 'Firebase/DynamicLinks'
    pod 'Firebase/Analytics'
    pod 'GoogleIDFASupport', '~> 3.14.0'
    pod 'GoogleTagManager'

Hi @percival888, based on your logs, it appears that the event is being logged, but is not showing in your DebugView in the Analytics dashboard. Upon testing, it seems that the screen_view event is successfully showing in the DebugView when using Analytics only. With this, this may be reported to Google Tag Manager so they could investigate.

Okay. I will definitely send this go Google now. Thank you


feature suggestion: ee from private registry

Hi, it would be very useful to pull the execution environments images from the AWS Elastic Container Registry, is it possible to implement such feature?
Thank you for everything, this project is awesome! :)

You can add your own EE through AWX GUI.
https://docs.ansible.com/automation-controller/latest/html/userguide/execution_environments.html#use-an-ee-in-jobs

Thank you for the quick reply.

You can add your own EE through AWX GUI. https://docs.ansible.com/automation-controller/latest/html/userguide/execution_environments.html#use-an-ee-in-jobs

Yes, maybe I forgot to mention that I'm considering ECR private registries.
I already did it what you suggested, and I've configured the proper Container Registry credential using AWS as username and the output generated by the command aws ecr get-login-password as token/password.
The problem here is due to the 12 hours authentication token validity of the ECR service, that forces us to refresh it manually, which is not a sustainable workaround.

Hmm, I've never know that restriction. Thanks for updating.
I'm not familier with ECR but there is no native function on AWX that updating credentials periodicaly.
So, this is just an idea, not so smart way but should work.

Create playbook to update credentials using awx.awx.credential module; https://docs.ansible.com/ansible/latest/collections/awx/awx/credential_module.html
Run this playbook as Job Template which target AWX itself, with periodic schedule.

Alternatively I recoomend you to ask the mailing list your question.

Ah, so you have enough expertise about k8s-thing 👍 Thanks for sharing your solution,
Your solution that using cronJob and imagePullSecrets with customized pod spec is the almost best way I think.
I'm also using cronJob to invoke some small script periodicaly. My favorite way is to store the script as ConfigMap and attach it to the Job's Pod to simplify the specification for CronJob, without making the script hard-coded in specification of CronJob.
One thing, I think kubectl create ... --dry-run -o yaml | kubectl apply -f - is more stable than simple kubectl delete; kubectl create, since the secrets never deleted or dissapeared.
But anyway your solution is good way with less overhead than using AWX's schedule.


Remove DEVELOPMENT constant

Are any reasons of using webpack __DEVELOPMENT__ constant while we have process.env.NODE_ENV? Doesn't uglify remove dead code in the latter case?

Thanks! but I think the process.env.NODE_ENV is unsuitable, because chrome extension isn’t node. :|
I would use __DEVTOOLS__ and __DEVTOOLS_EXT__ to use redux-devtools and redux-devtools-extension in the future.

Yes, it is fair, but about using together __DEVELOPMENT__ and process.env.NODE_ENV meaning the same could be confusing. The pro for the latter is that our submodules (including react and redux) already use it.
Like we already may set process.env.HOT in the react-transform-hmr, we can use process.env.DEVTOOLS. But of course it's a matter of taste :)


move metrics to katex-fonts submodule

It makes more sense to have metrics generation in the same repo as font generation as both things must be done at the same time.  See https://github.com/KaTeX/katex-fonts/pull/27.

I think we can remove make metrics(and eventually remove Makefile, #1121) or call buildMetrics.sh in:
https://github.com/Khan/KaTeX/blob/7b635f79cfc28c46dcbf814d47f6977e6712797b/Makefile#L84-L85

#1135 will take care of the metrics make target.

This actually needs updating b/c I closed https://github.com/KaTeX/katex-fonts/pull/27 in preference of https://github.com/KaTeX/katex-fonts/pull/30.


import Spec objects for each test case

Issue

spec_configured_state_test is not thread-safe:

The given spec: Spec object was initialized with global spec_targets in context.py.
spec_configured_state_test overrides spec.config, execute the test case, and then change it back to the unchanged config.
In multi-processing or multi-thread context, if a config field is overridden when running one test case, the spec.config object in other concurrent tasks might be changed too.

pytest-xdist implemented parallelization with subprocess.
We want to make the test generator parallelable!

Due to 1+2, #2752 failed because some of the ex-ante tests override spec.config.PROPOSER_SCORE_BOOST temporarily, and then concurrently, test_proposer_boost_correct_head failed because it wrongly used the overridden value.
How did I fix it
To better isolate the test cases, it would be better to reduce the risk of these side effects. This PR tries to import Spec objects for each test case. The solution was found from here and here.

replaced by #2765


VB.net: inizialisize with iApp as null

CefSharp.Cef.Initialize(settings, True, Null) 'Error here
What can i do?

Pass in null for browserProcessHandler
https://github.com/cefsharp/CefSharp.MinimalExample/blob/master/CefSharp.MinimalExample.WinForms/Program.cs#L59

I've added an additional Cef.Initialze(settings, performDependencyCheck) to simplify this use case relevant commit https://github.com/cefsharp/CefSharp/commit/b0ee2f2bdaaebca472928cc10fccac5c014435d6
Changes will be included in the upcoming 85 release.


Card Order

Hello,
thank you very much for this usefull addon!
I have a question regarding the order of cards.
After fetching a whole deck for a light novel (with the api key feature), which order are these cards in?
I know that you can reorder them with using the frequency list, but I would like to have them in order of appearance (like on jpdb).
Do you know if it is possible?
Maybe it is not possible to get the information from the API, but the "default" order does not seem to be the same as on jpdb.
Any thoughts?

Hello, thank you for testing the addon !
Sadly I don't think I can access the information about how the cards are sorted using the API. At least it seems to not be random, but I cannot make sense of the order even by looking at the fields individually when generating the cards.
Here is the API https://jpdb.stoplight.io/docs/jpdb/mgsimhgxpjpqe-jpdb-io-public-api if you want to give it a look yourself I might have missed something :hugs: In practice I first call the "List deck vocabulary" api to obtain a list of Ids that I then use to call the "Lookup vocabulary". The order of the notes is the one from the "List deck vocabulary" call.

Indeed it seems that the API does not offer this functionality. I wrote an email to jpdb to ask. I will post the response when I get one.


Error rstudio::server::app_armor::enforceRestricted()

When running the following bits under MacOS 10.13.3 we (me+@holgerbrandl) get the following error:
docker run -ti  -p 8787:8787 -e USER=rstudio -e PASSWORD=rstudio -v $HOME:/home/rstudio/data etournay/tissue_miner bash
root@3cce66058047:/movies/demo# /usr/lib/rstudio-server/bin/rserver
root@3cce66058047:/movies/demo# rserver[17]: ERROR system error 22 (Invalid argument); OCCURRED AT: rstudio::core::Error rstudio::server::app_armor::enforceRestricted() /home/ubuntu/rstudio/src/cpp/server/ServerAppArmor.cpp:90; LOGGED FROM: int main(int, char* const*) /home/ubuntu/rstudio/src/cpp/server/ServerMain.cpp:506

Ideas? workarounds?

Yes, it is normal. The R server is not meant to be started this way. It should be started with /init.
In addition, it depends on the docker image. If you type in docker images in a terminal you'll list all downloaded docker images with the associated tag. Let me know which tag you have.
Best,
R

Thanks @etournay for coming back to us so quickly.
Indeed running it via kinematic works for me under macos. The docs under 1.3.1 Start and access RStudio within TissueMiner may want to mention kinematic.
Alternative, could you @piscitel try
docker run --name rocker -d -p 8787:8787 -e USER=rstudio -e PASSWORD=rstudio -v $HOME:/home/rstudio/data etournay/tissue_miner

as indicated in the manual again on your mac? This works for me as well. Make sure to remove any previous image called rocker if present beforehand.
I think it may have worked in the first place already, but echo "http://$(docker-machine ip default):8787"  is broken and fails with
Docker machine "default" does not exist. Use "docker-machine ls" to list machines. Use "docker-machine create" to add a new one.

Luckily, once it is running (check with docker ps -a), you simply access the rstudio webui via http://localhost:8787.


Eagle环境独立初始化

Init Eagle Env independently for UC homepage

com.taobao.weex has been renamed to org.apache.weex

Duplicated with https://github.com/apache/incubator-weex/pull/3022


BUG: "punctuation" split should also split at newlines

Bug Report
Confirmation:

[X] I have read and followed all the instructions provided in the README.md.
[X] I am on the latest version of both Open WebUI and Ollama.

Expected Behavior:
The speech model should consider two lines with no punctuation as two sentences, instead of a single sentence containing a newline between them.
Actual Behavior:
2 lines with no punctuation are considered a single sentence. So the speech model receives unsplited text when I use markdown bullet point lists.
Description
If I have the choice between splitting on paragraphs or on punctuation, it is implied that the latter will always result in smaller chunks. But as we can see at this line:
https://github.com/open-webui/open-webui/blob/1d225dd804575af9ae5981528dfdce695f7f7040/src/lib/utils/index.ts#L543
The chunking is done only after punctuation, ignoring newlines. I noticed that by looking at the logs from my openedai speech instance. The issue is that markdown bullet points frequently end abruptly with no punctuation.
A newline, albeit not strictly a "punctuation" should be a splitting delimiter when we set "punctuation" as the splitter. Also, each splitted sentence should have "str.strip()" applied of course.
Here's a particularly broken text:

Additional Information
Pinging @kiosion as they introduced the punctuation split in #4886

Hm, I'll look into this tomorrow. It'd be nice to do a second pass tidying that logic anyhow


Update virtual-machines-common-premium-storage.md

Updated series of VMs that support Premium Storage.

@iainfoulds - Please review the proposed changes. Thanks

@jonbeck7 and @tamram - can you review these proposed changes on VM SKUs and Premium Storage, please.

@maucaro - We appreciate your contribution. Our apologies that we have not addressed it before now.
@iainfoulds - This contribution correctly updates the list of VM types that support premium storage and should be incorporated.


Update step 2 solution with non-deprecated action from ~ to stop statement

When setting up the log filter, the solution suggests a deprecated action, namely with &~.
This results in the following warning message (also referenced in rsyslog compatibility docs):
rsyslogd[20549]: warning: ~ action is deprecated, consider using the 'stop' statement instead

Considering that the version of rsyslog running in the Killercoda environment is 8.2001.0 -- and for the sake of future proofing -- applied the proposed changing swapping ~ to stop. Testing in the lab environment shows no unintended consequences and the warning goes away.

Looks good.


[Bug]: Regression in BigtableIO in the new 2.47.0 versions

What happened?
We migrated BigtableIO in https://github.com/apache/beam/pull/24015 and seems to caused regression. Dataflow jobs started failing. We're still trying to root cause it.
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

We have a suspicion that after the migration BigtableIO is creating too many Bigtable clients and not closing the clients correctly which caused the pipeline to OOM. We're testing the fix and should have a PR early next week.


Support Polygonscan

Description
Fixes Etherscan links to work on Polygon, and allows easier integration with other networks in the future.

Closing this in favor of putting it in network config instead.
https://github.com/decent-dao/fractal-interface/commit/4189fc8ece6b00647a31b59eed84901c7d9b9e9e


bold font is ignored in banking & classical style.

I want to use textbf to highligt different parts in the CV however this is ignored in the most recent version of Tex Live
This seems to be the case only for tex live distribution 2021 and not 2020.

It seems when I change the default font. e.g. usepackage{palatino} the bold faces are also ignored.

Which version of moderncv do you use?
Versions prior to 2.2.0 had a buggy font loading procedure (probably why TeXLive 2020 works for you)
Version 2.2.0 corrected this code and set all fonts to Latin Modern (probably why TeXLive 2021 does not work for you)
Since Version 2.3.0 we removed the font loading entirely, so it's up to the user to do it.
Although you didn't provide a minimum working example, I suspect you set the fonts incorrectly. If you use LuaTeX or XeTeX (highly recommended), do it like in the template:
https://github.com/moderncv/moderncv/blob/a89a2a74552d9c39faf4c438da5ff6691c453494/template.tex#L31-L37
This should work with all versions.

Thank you!

It works thank you :)

For me the only required line seems to be loading the fontspec package. I guess it is because I only use the serif font (i.e. the “main” font).


Add interface to allow user provided extensions

Problem Statement:
Present Implementation of Base CoRIM and CoMID has PSA specific types embedded into the base types.
One example is the Environment Type which has a Class Identifier object inside it.
One type of Class Identifier is the PSA Specific Implementation ID, which is  PSA specific.
The issue it raises it:

Now any external library user uses the Class ID object will get the PSA specific Implementation ID as part of
the library implementation, which is not required.

Any new user of CoRIM, CoMID library, if they have their own ID types, will eventually pollute the library with their own types.

Solution

Create Generic Objects (like Class types) which has ONLY the base types (like OID and UUID).
Provide mechanism for adding dynamic types from the caller with API's for:

a. Registration
b. Validator
c. Getter and Setter.
d. JSON and CBOR encoder and decoder

Additional context
None

Closing this as, now addressed using:
https://github.com/veraison/corim/pull/96


2 broken links (404) in the README.md

Description
There are 2 broken links in the OpenNebula/one repository's readme. Both return 404 Not Found Error.
Link 1: documentation section
Link 2: reference about the main OpenNebula daemon configuration file
Both links work when stable is replaced by 5.12 in the URL, which suggests that the documentation structure was changed in OpenNebula >= 6.0.
To Reproduce
Click on the links from the Description section.
Expected behavior
Links pointing to the relevant sections in the documentation.
Details
N/A
Additional context
N/A

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

THANKS, updated the links to point to the sections under the new structure


Add Drip Too Hard by Lil Baby Feat. Gunna

Song Name
Drip Too Hard
Artist Name
Lil Baby Feat. Gunna
Source
Youtube
Youtube Link
THcVOf1kNh0
Direct File Link
No response
Song ID
577731
Start Offset [ms]
0.5
Extra
Created from Auto Nong.
Created for level: 107297804

reject
popular songs like F777 songs are too inconvenient to be replaced by smth unrelated and are therefore usually rejected


Add overview page for docker-compose to k8s migration

This PR adds an overview page for a set of tutorials about how to migrate a compose-based workflow to Kubernetes.
Relevant issue: https://github.com/kubernetes/website/issues/27271

I think this is work in progress, so:
/retitle [WIP] Add overview page for docker-compose to k8s migration

@sftim What would you like to see for it to be not a WIP? Would you like to have the complete set of articles available to have it merged? Or you would like to have smaller merges?

If this is ready for review @remyleone, add a PR description and retitle it - we'll review!
(you should also fix the links)

@celestehorgan Thanks for the review :) As for the third party, it helps users to bring something from a 3rd party towards a vanilla Kubernetes. So I don't think it is something that provides unfair visibility for a 3rd party. This kind of content could be really helpful for people in charge of migrating towards kubernetes. I don't think Docker will maintain content that help people going out of their platform/tools.
Is it relevant for Kubernetes or you are not that interested in this kind of content?

@remyleone thank you for your effort on this PR!
I don’t know that I agree with it being a tutorial (or a new topic at all). From Choosing a page type, a tutorial needs to tie together several Kubernetes features to accomplish a goal. I don't think we have other migration guides on /website.
Plus, we have Translate a Docker Compose File to Kubernetes Resources. If anything, perhaps you could add this content where it makes sense on that page, to provide more context and keep the compose migration content consolidated.
cc @natalisucks

@remyleone are you planning to revisit this PR anytime soon?
There are a few unaddressed review comments, but overall it's in shape. I'm gonna drop a lgtm.
/lgtm

@celestehorgan fixed :)

/lgtm
I wonder how this content compares / relates to the task: https://kubernetes.io/docs/tasks/configure-pod-container/translate-compose-kubernetes/ - potentially there's opportunity to clean up / cross reference?
The above is non-blocking feedback if anyone feels like approving this as-is.

@remyleone would you be willing to rebase this against main?

I like this PR. FWIW I don't think this should be a tutorial, as there aren't any steps to follow. So far, the content asks the reader to figure out which components are stateless vs stateful, and then directs the reader to Kompose. It doesn't meet the criteria for a tutorial topic, but could be a task. As a user, I'd expect a tutorial to show me exactly how to implement a solution using various tooling. Thoughts?

I suppose what I'd wonder is:

should we flesh this out into a tutorial? There are several concerns here and we can't provide a complete guide that covers plausible causes
should we turn this into several tasks, each with a more specific focus?

Could it be several tasks under a subsection? It'd be nice to see more planning of what the final docs would look like here.
For example, #27271 talks about having a set of tutorials (or tasks) for migrating from Docker Compose workflows. What would be in those specific tutorials? Take "Secrets" for instance. What would the goal be for a Secrets tutorial?
Would it perhaps make more sense to have a concept page that talks about Docker Compose and Kubernetes, and compares Secrets, Services, Configs, etc. to Kubernetes equivalents (if they exist), and then directs readers to the tasks for migrating each component to Kubernetes? And have one tutorial that showcases an entire migration of a sample workflow from end-to-end?

Would it perhaps make more sense to have a concept page that talks about Docker Compose and Kubernetes, and compares Secrets, Services, Configs, etc. to Kubernetes equivalents (if they exist), and then directs readers to the tasks for migrating each component to Kubernetes? And have one tutorial that showcases an entire migration of a sample workflow from end-to-end?

That sounds ideal, and also like a lot of work. PRs that move us closer to whatever desired state we settle on would be most welcome.

Good point. For this PR, do you think it's feasible to see this information in a single how-to page (since it's mostly about identifying which of your components are stateful vs stateless)? That how-to could be housed under a section called Tasks > Migrating from Docker Compose workflows. Future PRs can add to the content in the section, since this is the overview PR.

@remyleone 👋🏻 Please feel free to /reopen when you're ready to continue.
/close


Lost type hl.types.ArrayBase

Thanks for closing issue #176. Now some samples for example sample/skin demands hl.types.ArrayBase type when using hxsdl library:
C:\HaxeToolkit\haxe\lib\heaps/git/h3d/impl/GlDriver.hx:226: characters 45-102 : Type not found : hl.types.ArrayBase

Yes. I got the blues too. Samples are not working. Only errors all around.

792a82da59507bbdd236d4554343ea5c00c1bcae should have fixed it.

Thanks.


Remove @PostConstruct and @PreDestroy #987

Remove PostConstruct and PreDestroy
Remove PostConstruct and PreDestroy to migrade to OpenJDK10
Fixes#987

Also have a look at what codacy complains about https://app.codacy.com/app/FAForever/downlords-faf-client/pullRequest?prid=2699767

Please format your commits in the following way:
[Title]
[newline]
[Further description]
Fixes #[issue number]

Also only make one commit per issue. Also it seems some tests fail, please run the tests locally and see what you broke.

Next time without the [] in the commit XD @DarkHorseV1  they were meant to tell u you should not write that but insert what fits your case

@micheljung could you have a quick look?

okay, i accept it

Thanks for the effort, much appreciated!

@micheljung can u check again?


Scale Bar / ShowsScale feature request

Summary
Is it possible to get a Scale Bar for google maps and some options for the iOS map?
At the moment the prop showsScale only shows a scale bar on iOS and apple maps and is not supported on google maps. It also only shows while zooming and you cant style or place it anywhere.
For android or google maps this is not supported at all.
Is there any alternativ to show a Scale Bar?
Reproducible sample code
<MapView showsScale />

Steps to reproduce

Just look at:
Google Maps on Android
Apple Maps on iOS
Expected result
I want a Scale Bar to always be showing on iOS / Apple Maps and Android / Google Maps
Actual result
Only shows a Scale bar while zooming on Apple Maps.
React Native Maps Version
1.8.3
What platforms are you seeing the problem on?
Android, iOS (Apple Maps), iOS (Google Maps)
React Native Version
0.72.4
What version of Expo are you using?
Not using Expo
Device(s)
Iphone 12, Google pixel 6
Additional information
No response

@BennyCarlsson I just googled and found this:
https://github.com/grethi/CHGMapView
it seems like you can layout a view on top of your map that updates graphic to display scale / zoom level ... I don't see the need to do this inside the library itself since the zoom level is always available and can be used to create such a component.

Did you find the answer to this question?


Update infrastructure directory

Moved Cloud Formation template from week 1 to week 3.
Fixed formatting issues in Cloud Formation JSON.

Changed ruby version back to original value of 2.3.0. Old RailsGoat docs indicated version 1.9.3, but new OWASP rails goat use 2.3.0.


Pick up .jars and .aars from App_Resources/Android/libs

Augment the project build.gradle script in order to build with all Java and Android libraries provided by the user. This will enable them to use local libraries that cannot otherwise be found in a remote plugin repository, or should be provided offline.

This is implemented and will be available in the next official release of tns-android.

Is it possible this change is causing this issue for me?
/Users/rozar/KipoTrac/platforms/android/app/build/intermediates/manifests/full/debug/AndroidManifest.xml:52: AAPT: error: resource string/facebook_app_id (aka com.kipotrac.mobile:string/facebook_app_id) not found.

/Users/rozar/KipoTrac/platforms/android/app/build/intermediates/manifests/full/debug/AndroidManifest.xml:52: error: resource string/facebook_app_id (aka com.kipotrac.mobile:string/facebook_app_id) not found.
error: failed processing manifest.

Failed to execute aapt
com.android.ide.common.process.ProcessException: Failed to execute aapt

Was working fine until I updated.

Found a fix: https://github.com/NativeScript/nativescript-angular/issues/742


On GHC 9.2: Blocked on dependencies

Seems like the dependency chain github -> tls -> memory -> basement is failing at its end:
$ cabal install github-0.27
Resolving dependencies...
Error: cabal: Could not resolve dependencies:
[__0] trying: github-0.27 (user goal)
[__1] trying: github:-openssl
[__2] trying: tls-1.5.5 (dependency of github -openssl)
[__3] trying: memory-0.16.0 (dependency of tls)
[__4] trying: memory:+support_foundation
[__5] trying: basement-0.0.12 (dependency of memory +support_foundation)
[__6] next goal: base (dependency of github)
[__6] rejecting: base-<IP_ADDRESS>/installed-<IP_ADDRESS> (conflict: basement =>
base>=4.9 && <4.16)

Blocked by:

[ ] basement (https://github.com/haskell-foundation/foundation/issues/554, https://github.com/haskell-foundation/foundation/pull/555)

Don't use -openssl. The +openssl configuration works well. (Problems with basement aren't my problems).

Thanks for the quick advice!
With flag openssl (not the default) it seems to fail at HsOpenSSL (>=<IP_ADDRESS> && <0.12), via github -> HsOpenSSL/text/parsec (incompatible requirements on bytestring).
$ cabal install github-0.27 -f+openssl
Resolving dependencies...
Error: cabal: Could not resolve dependencies:
[__0] trying: github-0.27 (user goal)
[__1] rejecting: github:-openssl (constraint from config file, command line
flag, or user target requires opposite flag selection)
[__1] trying: github:+openssl
[__2] trying: HsOpenSSL-<IP_ADDRESS> (dependency of github +openssl)
[__3] next goal: text (dependency of github)
[__3] rejecting: text-<IP_ADDRESS>/installed-<IP_ADDRESS> (conflict: HsOpenSSL =>
bytestring>=0.9 && <0.11, text => bytestring==<IP_ADDRESS>/installed-<IP_ADDRESS>)
[__3] trying: text-<IP_ADDRESS>
[__4] trying: network-uri-<IP_ADDRESS> (dependency of github)
[__5] next goal: parsec (dependency of network-uri)
[__5] rejecting: parsec-<IP_ADDRESS>/installed-<IP_ADDRESS> (conflict: HsOpenSSL =>
bytestring>=0.9 && <0.11, parsec => bytestring==<IP_ADDRESS>/installed-<IP_ADDRESS>)
[__5] trying: parsec-<IP_ADDRESS>
[__6] next goal: base (dependency of github)
[__6] rejecting: base-<IP_ADDRESS>/installed-<IP_ADDRESS> (conflict: parsec =>
base>=4.5.0 && <4.16)

The version of HsOpenSSL that is picked by cabal-install with GHC 9.2 on my system is <IP_ADDRESS> (excluded by github-0.27):
$ cabal install HsOpenSSL
Resolving dependencies...
Build profile: -w ghc-9.2.1 -O1
In order, the following will be built (use -v for more details):
 - network-<IP_ADDRESS> (lib:network) (requires download & build)
 - HsOpenSSL-<IP_ADDRESS> (lib:HsOpenSSL) (requires download & build)
((SUCCEEDS))

https://github.com/phadej/github/blob/dd195d9b5eaba68c2408ab488af8c1dcabd2e033/cabal.project#L13-L15

Thanks!
I filed

[ ] https://github.com/vshabanov/HsOpenSSL/issues/64


Status

Yo, First thank's for creating this lovely Bot.
How about add Set Status CMD like setusername or setavatar the CMD for Status will be something like setstatus.
https://discord.js.org/#/docs/main/master/class/ClientUser?scrollTo=setStatus

Just to make sure, are you aware of config config.bot.status? https://github.com/synzen/Discord.RSS/wiki/Configuration#bot

I know there's an option from config file but what i was thinking how about to do it with a command?

There you go: https://github.com/synzen/Discord.RSS/commit/614bc94e5cf373cb64d0d55a88e0f974a47b0be1
Note that it's on the dev-web branch and not on master


Add ban/kick/unban concommand completion

just adds completion to ban, kick and unban commands.

how test this ??????
try banning, kicking and unbanning someone from the in game console, it should display the closest uid/name to what you typed in :)

huh build failed to configure cmake?

minizip apparently has a remote dependency on https://sourceware.org/ which was done at the time xD


samples: basic: blinky: implement pattern of blinks

Many boards ship with a basic "blinky" pattern running already.
In order to be able to distinguish the Zephyr blinky, this
implements a more unique pattern.  It uses no extra system
calls or functions, just enlarges the forever loop to
include longer and shorter delays during multiple LED
on and off times.
Signed-off-by: Dean Weiten<EMAIL_ADDRESS>

IMO this sample is intentionally trivial; I don't see a good reason to complicate it.

OK that's fine, I withdraw my suggested change.


Generate API client credentials

Is your feature request related to a problem? Please describe.
The authorization modal on the API give a choice of either password-based login or client-ide/secret login. Currently both are unable to be used as there's no obvious way to generate a client secret/id (even the user login requires this).
Describe the solution you'd like
A panel in the user settings to generate client credentials would be sufficient.
Describe alternatives you've considered
I've attempted to figure out how the login page gets a token. It seems to just pass a literal token to the /token endpoint along with the username and password. Can't figure out how to pass that to the swagger API page, but could be used(if the token is static) in an application.
Additional context
I'm specifically working on Twitch integration where a chat command can receive the currently playing song of a given room. This is next to impossible as API calls are locked down.

Hi @jammehcow, I apologize for the delay in responding.
Integration with other sites is not yet fully ready.
If you want, you can message me on discord to talk about it and then maybe I can release a client_id and secret for you to try our API and report bugs.


add require email confirmation option

First of all sorry for all the trouble, I'm such a noob on git lol. As I've promised I create this PR again.
Regards, Martinus

Seems this PR will require #632 to be applied first so the checks will succeed.

Thanks for this PR 👍🏼

Thanks for this PR 👍🏼

You're welcome. Thank you for this project, which is great 😃 👍

I merged your changes manually, thanks 👍


Add flag to ignore Parse method

I recently came into an issue where the existence of a Parse(string) method caused the extraction plan to fail unexpectedly.
I wished to Extract to a struct with 3 int fields using 3 capture groups. However, since the struct contains a static Parse(string) method, the extraction plan believed that this struct should only consume 1 capture group instead of the three, and threw an exception. Commenting out the parse method temporarily solved the issue, but I would like for this to be a flag I can specify.

That's a really good idea. Thinking about level of granularity here, it seems bad to ignore all Parse methods on everything. Do you have any clever ideas for how we could allow you to pass an ignore-list?

Hmm, I didn't consider the case where you'd want to ignore one parse method and use another one.
The first thing that comes to my mind if you want to have full control is some kind of format string.
I'm thinking of the edge case where you have a type like List<(Position, Position)> and you'd like to skip the parse method for one of the positions but use it for the other.

That's an additional level of granularity that I hadn't considered. :)
I was thinking more along the lines of "I really want ints to Parse even if I want to disable Parseing in your example."
Maybe we could start with just an optional argument that allows you to exclude types by name (or Type). This could probably be attached to the Extract() overloads or possibly only available if you request plan creation explicitly.
If you have thoughts on where or how the format string might go for the more fine-grained version, I'm all ears.

Revisiting a handful of suggestion issues like this one.
I can totally reproduce this:
ExtractionPlan<test>.CreatePlan(new Regex(@"(\d+) (\d+) (\d+)")).ToString("x").Dump();

struct test {
    public test(int a, int b, int c)
    {}

    public static test Parse(string str)
    {
        return new test();
    }
}

Resulting extraction Plan:
StaticParseMethod<test>[0] ()

↓----↓ ↓----↓ ↓----↓₀
(\d+)₁ (\d+)₂ (\d+)₃


Add Dependabot Automation

This PR adds a dependabot configuration file that enables weekly update checks for all github actions used within workflows in the project. Additionally, it includes a workflow that automatically merges dependabot PRs for minor and patch version updates (major version bumps must be manually merged to avoid breaking changes).
Using these features will help keep all workflows up to date with a minimal amount of manual intervention necessary.
Note: this functionality has already been added to the Spatie laravel package skeleton repository.

Thanks!


The theme setting not working when using tabs

My Vim  color scheme is molokai. And I set my Airline theme serene like this:  let g:airline_theme='serene'
It's working so good, but when I opened a tab, it's not working. But when I closed it, then the theme back to serene. And I don't know why, only I have this issue?

Not sure I understand. Can you explain in detail, what the problem is? And is this still relelvant? How is this reproducable?

no reply. closing.


Webapp capability: integrate with unread indicators

Add capabilities to the webapp for plugins to integrate with unread indicators. This might be the UI path for features like summarizing unreads.
TODO: Add screenshot and figma link when figma isn't down.

Fixed by: https://github.com/mattermost/mattermost/pull/24265


Test/Fix Serial Support

Serial support SHOULD be available & functioning already, but I'll admit - I don't actually have any serial controllers at the moment. The first thing to address for v2 is this shortcoming.

[x] Test serial support
[x] Fix serial support (if needed)

I built a sample Arduino controller, hooked it up via serial, and tested the main features. Thus far, everything seems to be working perfectly. Going to leave this open for now, but hey - serial support might actually be in/working!

Serial support has now been tested with both ESP8266-based controllers as well as Arduino-based controllers. Closing this issue.


Alter EmailMessage database structure

Change sent box - it currently displays the wrong name!

Email message should store :

To
CC

Also check the zimbra api to see if there is anything else that needs to be saved.

Hi @CreaRo! my colleague and I are working on this!

@CreaRoWould you mind to tag the issue with the hacktoberfest tag, please?

@Iakobs any updates here?

Hey, @CreaRo! so sorry! we couldn't manage to install the app and we abandoned the pull Request. So sorry I forgot to tell you.


Added option of supplying directory to rst-lint

This commit adds the feature of supplying a directory to rst-lint. The program recurses (so all files within subdirectories will be linted too) to find all the .rst files in the directory, and lints the files.
Also added tests for the same.
Closes https://github.com/twolfson/restructuredtext-lint/issues/38

Looks good but it seems like tests could use some work and we missed an edge case for traversal

@twolfson Made the changes.

This looks good at a glance. I'm unsure I'll be able to fully land it tonight but I'll do it by the end of the week

This has been released in 1.2.0. Thanks for your help on making this feature happen =D


Request: Expose ffmpeg compression backend

This might be better than a a few-size-fits-all approach. You're already using ffmpeg to convert, so it would be cool if the program just generated a text string that got passed to ffmpeg. It would expose commands like this and allow for user editing.
Some video compression tools already use this approach. It is great for power users, because they can customize the command as much as they want. It also gives the tool more forward-compatibility.
It would be even better if the tool loaded options from a config file for the ffmpeg path. Something like:
[option-flac]
command: ffmpeg -i %1 %1.flac
[option-ogg]
command: ffmpeg -i %1 %1.ogg

Then I could add Opus recipes at different bitrates, for example. :)

Yes, I'll look into a way to achieve something like this, it seems useful 👍
Perhaps as an advanced setting/option, since I don't want to take away from the usability of the program!


execution summary: Don't crash on uncomparables

This has become increasingly important since spotify/luigi#1479 where
EnumParameters have been introduced (which are normally not sortable)

LGTM

we need to fix flaky visualizer test

thanks!

@jimjh you're welcome :)


ios image picker

Thanks for iOS supported add.
But we can't select from iPhone Photos.

The process of choosing Apple Photos involves utilizing the PHImageManager API, which differs from selecting files through the file system picker.

So are we planning to implement this feature to pick images from iPhone Photos?

Take a look at https://github.com/TEAM-PREAT/peekaboo

Take a look at https://github.com/TEAM-PREAT/peekaboo

it has problems with a aspect ratio

Take a look at https://github.com/TEAM-PREAT/peekaboo

it has problems with a aspect ratio

I don’t see a bug report related to that. Did you report to the project?
However, there is a dedicated image picker library present. File picking is something completely different from a technical point of view.
I think this is out of scope for this library, but that’s of course ultimately a decision the team has not make.


CA2000 Erroneous warnings for IAsyncDisposable

Consider the following examples:
await using var asyncDisposable = new AsyncDisposable();

Foo();

This should never throw CA2000, but I see it.
Consider, also:
var asyncDisposable = new AsyncDisposable();

await using (asyncDisposable.ConfigureAwait(false))
{
    Foo();
}

I would not expect this to result in a warning as long as asyncDisposable is not accessed again outside of the await using block.
Note: In the event of ConfigureAwait() being applied, the disposable becomes type ConfiguredAsyncDisposable

CAxxxx diagnostics are triggered by a package built from this repo:
https://github.com/dotnet/roslyn-analyzers

Ah, thanks!


feat: Folder provisioning implementation

What this PR changes/adds
This PR adds support for multiple blobs transfer
Why it does that
Currently, the extension only supports transfer of single blob. It does not support like for example AWS extension does - registering source data address as 'folder' by specifying blob path prefix. It would be most likely of interest to all parties integrating the EDC on Azure to use such feature.
Further notes
Additionally this PR  adjust existing integration test BlobTransferIntegrationTest to test single blob transfer as well as multiple blobs transfer.
It would be preferred to have this change already in 0.3.2-SNAPSHOT if possible  (together with our previous merged contribution #77)
Linked Issue(s)
Closes #91

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (fcc31ec) 74.45% compared to head (fa39182) 74.05%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   74.45%   74.05%   -0.40%
==========================================
  Files          27       27
  Lines         685      713      +28
  Branches       23       27       +4
==========================================
+ Hits          510      528      +18
- Misses        171      180       +9
- Partials        4        5       +1

Files
Coverage Δ

...dc/azure/blob/validator/AzureStorageValidator.java
97.50% <100.00%> (+0.83%)
:arrow_up:

.../storage/pipeline/AzureStorageDataSinkFactory.java
100.00% <100.00%> (ø)

...torage/pipeline/AzureStorageDataSourceFactory.java
93.93% <100.00%> (+0.60%)
:arrow_up:

...azure/storage/pipeline/AzureStorageDataSource.java
81.63% <33.33%> (-11.06%)
:arrow_down:

...g/eclipse/edc/azure/blob/api/BlobStoreApiImpl.java
28.07% <0.00%> (-2.12%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix Options JSON schema description

Summary
It's seems that schemars uses the comment of a flattened struct for the enclosing struct.
In our case, the comment on CommonLintOptions was used for Options.
I moved the lint section comment to LintOptions which avoids this undesired inheritance.
Test Plan
Reviewed the json schema changes

Current dependencies on/for this PR:

main

PR #8081   👈

This comment was auto-generated by Graphite.