Whether the JNI interface supports BinaryBitmap.

Hello:
It is more convenient for the camera to scan the code use the method of
BarcodeReader.read(BinaryBitmap bitmap)  instead of
BarcodeReader.read(Bitmap bitmap)
 Are there additional interfaces?
 Best Regard!

Hmm...  yeah,there is not. What I mean is that when I use the camera to scan, what the camera returns to me is YUV data byte. It seems that I need to convert it into bitmap to use Android API.
Maybe adding a Android APIs of BarcodeReader(byte[] data,....) or BarcodeReader(BinaryBitmap bitmap,....) looks good?BinaryBitmap is java class of original Zxing library.

Hmm...  yeah,there is not. What I mean is that when I use the camera to scan, what the camera returns to me is YUV data byte. It seems that I need to convert it into bitmap to use Android API.
Maybe adding a Android APIs of BarcodeReader(byte[] data,....) or BarcodeReader(BinaryBitmap bitmap,....) looks good?BinaryBitmap is java class of original Zxing library.

I have been working on a new BarcodeReader for Android implementation a while back (mainly to be able to deprecate some internal c++ interfaces). I'll try to bring it to a usable state and commit the changes 'soonish'. The interface will allow to call the read function directly on an ImageProxy object from the Camera2/CameraX API containing YUV_420_888 data.
I'm curious: what is your reason to use this JNI wrapped c++ project instead of the upstream Java project?

I think .so library is hard to decompile, Java code is easy, and Android ProGuard is not secure when you want to add extra functionality or custom code and need to keep it secret.

So your motivation is to be able keep proprietory changes secret? If you are changing the code anyway, you should be able to modify the API to your liking without help from the open source community... ;).

See my recent Android related commit.

Thank you for your commit.
Best regard!


Error 404 on French site, link "Découvrir les fonctionnalités de QGIS"

Description
Error 404 on the French version web site when clicking the link Découvrir les fonctionnalités de QGIS
Page URL: https://www.qgis.org/fr/site/about/index.html

Actually it's the same for anything non English
@rduivenvoorde For some reason, the docs has reverted again to 6th march, before we release translation.

@DelazJ I fixed the build, some dirs were owned by root. I also added a docker-world script, seems there is only a cronjob.sh script for 3.16 and 3.22???
Tested by a 'fr' run... later this evening the normal cron run will do all other languages...

Ok, cronjob was still running cronjob.sh... which only builds english..
Fixed cronjob, should be running now

Fine now that the 3.22 docs build is stable.


chore(release): 1.6.0-rc.10

Version bump in package.json and package-lock.json for release 1.6.0-rc.10

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add null-safety support

Thank you for this great package,
Can you please add null-safety for it? I can create a PR if you are busy.

Error: Cannot run with sound null safety, because the following dependencies
don't support null safety:

 - package:sim_info

For solutions, see https://dart.dev/go/unsound-null-safety

This package seems to be still not Dart 3 compatible, Is there any hope to amend for that?


Subprocess and Pexpect tests are failing inside Docker container

Appears to be because they cannot access coloredlogs:
command: /tmp/normalize-filename/normalize-filename
args: ['/tmp/normalize-filename/normalize-filename', '/tmp/tmp9gk8kn1y/blah.txt', '--interactive', '--no-undo-log-file']
searcher: <pexpect.searcher_re object at 0x2b31f5824b38>
buffer (last 100 chars): b''
before (last 100 chars): b"gs.ColoredStreamHandler(\r\nAttributeError: 'module' object has no attribute 'ColoredStreamHandler'\r\ne"
after: <class 'pexpect.EOF'>

This can also be recreated inside a virtualenv.

This was because an old version of coloredlogs (3.1.4) was being used elsewhere. Have upgraded to coloredlogs 5.0-style syntax (which is simpler) and made that the minimum requirement.


The new device contribution guide needs to be updated

The guide https://github.com/resin-os/meta-resin/blob/master/contributing-device-support.md must be updated to reflect latest changes in meta-resin 2.0.0

Any progress on this?

@floion there was nothing done in this regards hence assigning back to you.

I was under the impression that the OS team should handle this since it is closely related to meta-resin.

@floion arguably, this is about device support, so it's somewhere in the
middle..
--
Alexandros Marinos
Founder & CEO, Resin.io
+1<PHONE_NUMBER>
@alexandrosm
On Tue, Jun 6, 2017 at 8:17 AM, Florin Sarbu<EMAIL_ADDRESS>wrote:

I was under the impression that the OS team should handle this since it is
closely related to meta-resin.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/resin-os/resinos/issues/262#issuecomment-306519411,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLUCLWPmaQv47r-hHmAvwsBQX5iGuhPks5sBW2PgaJpZM4MxuZT
.

yes, it may be in the middle, but it sits in the meta-resin repository and it is actually describing features from meta-resin


jedis.keys() return type is not compatible jedis.mget() param type

Set keys = jedis.keys(RedisStrings.MakePostKey(userNo));
String[] arrayKey = keys.toArray(new String[keys.size()]); <---------- i dont want this code line
List valueList = jedis.mget(arrayKey);

@myc0058 what do you propose?. All jedis commands that return lists / sets, etc return either List or Set, so I don't see why we need to make these to interfaces necessarily compatible.

Maybe this issue is duplicated.
We already discussed about supporting vargs vs Iterable (or maybe List) to parameters.
Two things are having pros and cons, and I don't remember precisely but we decided to stick current way.
Btw, we can discuss to break current rule because we're preparing major release.

my opinion is
jedis.keys() and jedis.mget() function is Jedis's functions.
Between the jedis's functions return type and param type should be match.
i want to make a overloading function
List jedis.mget(List keys) {
String[] arrayKey = keys.toArray(new String[keys.size()]);
return jedis.mget(arrayKey);
}
this is just about jedis's kindness.

@myc0058 you're not proposing something generic that will solve then problem when dealing with arrays vs lists vs varargs. I don't see how we can we benefit of applying this solutions to all the other commands. Overloading the function is also something that you can do in your own project; you don't neet Jedis to provide that.

Since it doesn't introduce method chaining, it is something trivial issue for us.
We're struggling to maintain SO MANY DUPLICATED methods which are just overloading methods due to support of various params.
Personally I really wish to get rid of these, and concentrate on thinking core things.
Marcos said this earlier, which I strongly agreed.
Take into account that Jedis is maintained by a very few people and it get's really difficult for us to move forward unless we take some actions to boost contributions and focus on core functionality and code simplicity.

So sorry about it, we can reject anything if it's good to maintain Jedis easier or better whether it is convenient by user's side.

thank you for answer


3 OpenReport API calls for the same report

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.41-6
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @iwiznia
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1727446418470879
Action Performed:

User A and user B are on a chat
User A adds a comment
User A creates a thread under their comment
User B clicks the thread

Expected Result:
Thread opens and no duplicate OpenReport API calls for same report
Actual Result:
3 OpenReport API calls for the same report
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
There are 2 OpenReport calls when open a thread that we never loaded yet.
What is the root cause of that problem?
The 1st OpenReport is called in this effect. The purpose of this effect is to call the OpenReport on mount, but we only want to call it when the report onyx is already loaded. We did this after migrating from withOnyx to useOnyx here.
https://github.com/Expensify/App/blob/69483ff0ef36e39fbf40c15f2caa3471dccbef17/src/pages/home/ReportScreen.tsx#L489-L496
The 2nd and 3rd call is triggered from this effect.
https://github.com/Expensify/App/blob/69483ff0ef36e39fbf40c15f2caa3471dccbef17/src/pages/home/ReportScreen.tsx#L577-L588
When we open the thread that we haven't loaded yet, the report will initially be undefined. The 1st OpenReport will optimistically merge the reportName as Chat Report.
https://github.com/Expensify/App/blob/69483ff0ef36e39fbf40c15f2caa3471dccbef17/src/libs/actions/Report.ts#L779-L783
This will update the report to {reportName: 'Chat Report'}, and so, the report onyx now contains an empty reportID.
https://github.com/Expensify/App/blob/69483ff0ef36e39fbf40c15f2caa3471dccbef17/src/pages/home/ReportScreen.tsx#L169-L173
Because onyxReportID (empty string) is different from prevReport?.reportID (undefined), the 2nd OpenReport is called. After the 1st OpenReport is finished, the onyxReportID is updated to the correct reportID (X). Again, because the onyxReportID (X) is different from prevReport?.reportID (empty string), the 3rd OpenReport is called.
What changes do you think we should make in order to solve the problem?

There are 2 solutions. First, if we see the comment here,
https://github.com/Expensify/App/blob/69483ff0ef36e39fbf40c15f2caa3471dccbef17/src/pages/home/ReportScreen.tsx#L577-L585
it's mentioned that when deeplinking from a report to another report, the report screen is not unmounted and the reportID param is not updated, so the old conditions doesn't work when comparing prev reportID params to the current reportID params.

The report screen is not unmounted part is still true today because if we deeplink to the same screen with a different params, react-navigation will just update the screen params, so this part isn't true anymore.

the reportID in the route also doesn't change.

So, the first solution is to use the old conditions by comparing the prev reportID params and the current reportID params.
if (prevReportIDFromRoute === reportIDFromRoute) {
    return;
}

The second solution is to push a new screen if the reportID is different. To do that, we can override the deeplink here.
https://github.com/Expensify/App/blob/69483ff0ef36e39fbf40c15f2caa3471dccbef17/src/libs/Navigation/linkingConfig/subscribe/index.native.ts#L21-L29
if (focusedRoute?.name === SCREENS.REPORT && hasAuthToken()) {
    Navigation.navigate(path as Route);
    return;
}

Then, we can remove the OpenReport call here.
https://github.com/Expensify/App/blob/69483ff0ef36e39fbf40c15f2caa3471dccbef17/src/pages/home/ReportScreen.tsx#L577-L585

I think #vip-vsb is the best place for this since it's related to threads

@shubham1206agra can you review @bernhardoj's proposal or should we reassign this?

That specifically happens after https://github.com/Expensify/App/pull/49056 where they add a check for the report here.
https://github.com/Expensify/App/blob/f0be8a731dc0832827dbd14d23fd49b332ff275e/src/pages/home/ReportScreen.tsx#L169-L173

So, previously, even when the report is undefined, the reportID will still be an empty string, so the 2nd OpenReport isn't called.

Again, because the onyxReportID (X) is different from prevReport?.reportID (empty string), the 3rd OpenReport is called.

But I tried reverting it locally and the 3rd OpenReport is still called when the reportID changes from empty string to the correct reportID. That's why I didn't talk about that since the multiple OpenReport will still happen.

Why is it a workaround? The current code is the workaround for an issue that isn't happening anymore.
https://github.com/Expensify/App/blob/96ffbfe27c5c1b85b062e6ab50c0adc46b0e94c3/src/pages/home/ReportScreen.tsx#L580-L586
I'm suggesting to revert back to the old condition which compares the prev and current reportID params. If my solution is a workaround, then the old code is also a workaround, which I don't think is true.
There is even a 2nd solution that will remove that condition and push a new screen instead of updating the current screen params.
I think removing a workaround is preferable compared to patching existing workaround.

I think removing a workaround is preferable compared to patching existing workaround.

I agree that this is preferable.

Why is it a workaround? The current code is the workaround for an issue that isn't happening anymore.

Is it not happening anymore because of the workaround @bernhardoj ? i.e. if we removed the workaround would the issue resurface?

Is it not happening anymore because of the workaround @bernhardoj ? i.e. if we removed the workaround would the issue resurface?

Yes, it's not happening anymore from my testing but not because of the workaround. I think it's a bug in the navigator that we used in the old code before.

OK if reverting the workaround from your code here will resolve this issue, and removing it will not cause further issues, I think that's the approach we should take

Is it? IIRC someone mentioned that this was still happening... See this conversation https://expensify.slack.com/archives/C05LX9D6E07/p1730737982412859?thread_ts=1727884875.529299&cid=C05LX9D6E07

hey @shubham1206agra - can you please check the conversation Ioni linked here - https://github.com/Expensify/App/issues/49948#issuecomment-2460028756
It looks like this issue is still happening

Issue not reproducible during KI retests. (First week)

Fixed over here: https://github.com/Expensify/App/issues/51680


Scaling the number of instructions

Hi~
Why limit the number of instructions to 52K? Will the model be better if we have more instructions?

Hi @zwhe99 it's more because we observed the generation of novel tasks slows down after 52K. We also observed the fine-tuning performance plateaus at this size based on our limited experiments on SuperNI and the new user-oriented evaluation set. I think future work can benefit from a larger seed set with more diversified formats.

I see. Thanks for the response!


Augment bounding boxes is broken when only _augment_keypoints is defined

I have an augmenter that implements _augment_keypoints and _augment_images, which should be sufficient for augmenting both vector and raster representations of data. However, I noticed when I updated imgaug, my boxes were no longer being augmented.
It looks like 0.4.0 broke the behavior where augment_bounding_boxes used to work as long as _augment_keypoints was defined.
The currently implementation in Augmenter is:
    def _augment_bounding_boxes(self, bounding_boxes_on_images, random_state,
                                parents, hooks):
        return bounding_boxes_on_images

    def _augment_polygons(self, polygons_on_images, random_state, parents,
                          hooks):
        return polygons_on_images

However, if only the two aforementioned functions are implemented then the augmenter will change the image, but the boxes will be unchanged. At the very least this should cause an Exception, but I think a better fix would simply be to use the _augment_keypoints function if it exists:
    def _augment_bounding_boxes(self, bounding_boxes_on_images, random_state,
                                parents, hooks):
        return self._augment_bounding_boxes_as_keypoints(
            bounding_boxes_on_images, random_state, parents, hooks)

    def _augment_polygons(self, polygons_on_images, random_state, parents,
                          hooks):

        return self._augment_polygons_boxes_as_keypoints(
            polygons_on_images, random_state, parents, hooks)

This issue is now fixed in master. Thanks for reporting it.


Generated code fails the clippy::str_to_string lint

Describe the bug
The generated code fails the clippy::str_to_string lint.
To Reproduce
src/main.rs
#![deny(clippy::str_to_string)]

struct A;

#[juniper::graphql_object]
impl A {
    fn f() -> &'static str {
        "a"
    }
}

fn main() {
}

Cargo.toml
# Cargo.toml
[package]
name = "x"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
juniper = "0.15.10"

cargo clippy output
Checking x v0.1.0 (C:\Users\Dinu\proj\x)
error: `to_string()` called on a `&str`
 --> src\main.rs:5:1
  |
5 | #[juniper::graphql_object]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^
  |
  = help: consider using `.to_owned()`
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#str_to_string
note: the lint level is defined here
 --> src\main.rs:1:9
  |
1 | #![deny(clippy::str_to_string)]
  |         ^^^^^^^^^^^^^^^^^^^^^
  = note: this error originates in the attribute macro `juniper::graphql_object` (in Nightly builds, run with -Z macro-backtrace for more info)

error: could not compile `x` due to previous error

Expected behavior
The generated code to pass the lint, or at least ignore it.
Additional context
Add any other context about the problem here.

juniper and juniper_codegen in Cargo.lock

# Cargo.lock
[[package]]
name = "juniper"
version = "0.15.11"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "52adf17d43d0b526eed31fac15d9312941c5c2558ffbfb105811690b96d6e2f1"
dependencies = [
 "async-trait",
 "bson",
 "chrono",
 "fnv",
 "futures",
 "futures-enum",
 "graphql-parser",
 "indexmap",
 "juniper_codegen",
 "serde",
 "smartstring",
 "static_assertions",
 "url",
 "uuid",
]

[[package]]
name = "juniper_codegen"
version = "0.15.9"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "aee97671061ad50301ba077d054d295e01d31a1868fbd07902db651f987e71db"
dependencies = [
 "proc-macro-error",
 "proc-macro2",
 "quote",
 "syn",
]

Can I try this issue? I am new to Rust though.

Of course!

This is already fixed on master and will be released in 0.16.


[controller] Parent controller should not update the current version

Parent controller should not update the current version

Currently, there are some codepaths that lead to parent controller updating the current version. Another unexpected code path that gets triggered in parent controller is RealTimeTopicSwitcher#transmitVersionSwapMessage. Since this piece wasn't written with the intention of getting executed in parent controller, it causes failures in executing tasks from the admin channel. This causes the Admin channel to get stuck for this store.
This commit adds checks that prevent the current version in parent controller to be updated.
How was this PR tested?

CI
Does this PR introduce any user-facing changes?

[X] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Can you explain why non-zero parent current-version can affect other admin operations? Maybe we should enhance that piece of logic as well?


invalid version when installing in dev mode

Trying to install Graphium in dev mode by running pip install --no-deps -e ., I faced with the following error:
File ".../lib/python3.11/site-packages/setuptools/_vendor/packaging/version.py", line 198, in __init__
          raise InvalidVersion(f"Invalid version: '{version}'")
      setuptools.extern.packaging.version.InvalidVersion: Invalid version: 'dev'

which results in:
Getting requirements to build editable did not run successfully.
  exit code: 1

The issue can be resolved by replacing the "dev" to 2.4.dev1 in the pyproject.toml and the _version.py (not sure if that second one was needed).

@jstlaurent Can you look into this? What's the best way of replacing "dev"?

@fpour: Can you give more info about your environment? Are you using pyenv, venv, Conda, or something else?
I tried the command in a clean new venv and It worked for me, so I'm trying to see what I'm missing to reproduce the issue.

I used mamba and just followed the readme procedures. Basically, just ran the following commands:
mamba env create -f env.yml -n graphium
mamba activate graphium
pip install --no-deps -e .

That's weird. I tried exactly the same series of instructions and it works fine. Maybe if you reset your conda environment?
mamba deactivate
mamba env remove -n graphium

I started a fresh env and don't see the issue anymore.

Resolved


Create shrikrishnacollegeofeducation.txt

Please add this college for jetbrains.
Shri Krishna College of Education (SKCE), Chhatarpur

@impscetadmin Pull request merged. Thank you!


(Issue #309) Introduced functional-parameters rules that allows for fp-ts Lazy functions

https://github.com/eslint-functional/eslint-plugin-functional/issues/575


It still doesn't work

Hi! I've red all issue topics, made everything you described, but still the app is not working. Aws is showing that the key was
used, but app just hanging and not responding.

Hi! Sorry you’re having trouble. Please share screenshots of your settings and any relevant configurations so I can better understand the issue. This will help me provide more targeted assistance.

Hi, I have setup the plugin and am also not able to get it working.

Example

If you can help, I can test it.
Thank you


Animation Duration must be smaller than Delay to show

When using keyframe animation like this:
<Style Selector="Border">
			<Style.Animations>
				<Animation FillMode="Forward" Delay="0:0:2" Duration="0:0:2">
					<KeyFrame Cue="0%">
						<Setter Property="Opacity" Value="1.0"/>
					</KeyFrame>
					<KeyFrame Cue="100%">
						<Setter Property="Opacity" Value="0.0"/>
					</KeyFrame>
				</Animation>
			</Style.Animations>
</Style>

if the delay is bigger than duration, the animation doesn't play at all. Effectively if I want to use delay I need to set the Duration to at least the same time for it to work, but it just plays a 2 second animation after a 2 second delay. I can't have a 1 second animation after 2 seconds delay.
I'm on Preview 4.

@lukaszossowski if you open bug reports for 11.0 previews, I would really appreciate if you can test them against latest nightly builds. Atm there are so many fixes going on, that it might have already changed.

Remark: I tried to reproduce with latest nightly and I can confirm the issue is still present. Interesting behavior.

Is it fixed?

@leonidcan don't know but don't think so. Is the issue gone for you?

@timunie I am still on 10, was wondering if it was fixed in 11 :(

Yes, it was fixed https://github.com/AvaloniaUI/Avalonia/issues/11819


Provide backup and restore of error reports in case of ELK failure

An uploaded error report should be persisted - encrypted - in the event of an ELK failure so that it can be restored at a later date.

To verify backup do the following:

Start Elasticsearch locally as a foreground task
Run up the Error Reporting Service in an IDE in debug mode
After successful start put a breakpoint against the pushToElk() method in PublicErrorReportingResource line 153 (ish)
POST the error-report.json.asc example to http://localhost:9191/error-reporting
Observe breakpoint hit then shut down Elasticsearch (CTRL+C) and wait for it to stop
Continue execution and observe creation of /var/error-reporting/error-reports containing file with UUID
Observe 201 CREATED response from service
Leave service running and restart Elasticsearch in foreground, wait for complete start
POST the error-report.json.asc example to http://localhost:9191/error-reporting
Observe insertion into Elasticsearch

To verify the ingestion (restore) process do the following (can be done immediately after the above):

Navigate to var/error-reporting/error-reports and observe presence of file with UUID (you can add a bunch of copies using error-report.json.asc as the basis
Open a shell and execute curl -X POST http://localhost:9192/tasks/ingest
Observe insertion into Elasticsearch
Observe deletion of file(s) as they are ingested
Observe "Processing" output from curl (after completion)

Ready for review and close.

Worked through the above scenario to check persistence and ingestion of error reports is working fine.
Closing.


[Improvement-4375][api] cannot delete yarn queue

[Improvement-4375][api] cannot delete yarn queue
Improvements for unable to delete queues
have delete button to delete the queue
close #4375

good job, can you add Unit Test for this PR?

good job, can you add Unit Test for this PR?

OK, I'll try

good job, can you add Unit Test for this PR?

I added it as well, can you look at it

I need some review

@amaoisnb Hi, please check the failed CI.

@amaoisnb Hi, please check the failed CI.

OK， I'll try

Please check failed UT. @amaoisnb

Codecov Report

Merging #13046 (423b495) into dev (ca760aa) will increase coverage by 0.00%.
The diff coverage is 25.92%.

:exclamation: Current head 423b495 differs from pull request most recent head b7e026f. Consider uploading reports for the commit b7e026f to get more accurate results

@@            Coverage Diff            @@
##                dev   #13046   +/-   ##
=========================================
  Coverage     38.58%   38.58%
  Complexity     4575     4575
=========================================
  Files          1260     1260
  Lines         43748    43729   -19
  Branches       4826     4801   -25
=========================================
- Hits          16878    16871    -7
+ Misses        24998    24986   -12
  Partials       1872     1872

Impacted Files
Coverage Δ

...lphinscheduler/api/controller/QueueController.java
69.23% <0.00%> (-12.59%)
:arrow_down:

...inscheduler/api/service/impl/QueueServiceImpl.java
65.81% <18.18%> (-11.04%)
:arrow_down:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <100.00%> (ø)

... and 33 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hi @SbloodyS @amaoisnb , FYI, we plan to remove yarn queue management https://github.com/apache/dolphinscheduler/pull/14310#issuecomment-1624817040, do you have any different idea about it?


i keep getting instance of Mockery_0__*, why ?

i want to mock a class called Holla does not exist ..
holla = $holla;
    }

    public function greet()
    {
        return $this->holla->say();
    }
}

and this is the test for Temp class

shouldReceive('say')->once()->andReturn('hi');

        $temp = new Temp($holla);
        $this->assertEquals('hi', $temp->greet());
    }
}

when i run the test i get:

PHPUnit 4.6.4 by Sebastian Bergmann and contributors.

E

Time: 123 ms, Memory: 3.25Mb

There was 1 error:

1) HollaTest::testICanGreet
Argument 1 passed to App\Temp::__construct() must be an instance of App\Holla, instance of Mockery_0__Holla given, called in /home/oussama/Desktop/iLearn/iLearnToMock/tests/HollaTest.php on line 13 and defined

/home/oussama/Desktop/iLearn/iLearnToMock/app/Temp.php:6
/home/oussama/Desktop/iLearn/iLearnToMock/tests/HollaTest.php:13
phar:///usr/local/bin/phpunit/phpunit/TextUI/Command.php:186
phar:///usr/local/bin/phpunit/phpunit/TextUI/Command.php:138

FAILURES!
Tests: 1, Assertions: 0, Errors: 1.

shell returned 2

Press ENTER or type command to continue

i tryed a lot of combinations and examples, but i get the same error, also i tried to dive into the source code, but once i get to Container::mock() it was too long (not profiled), and i don't really have time to track the issue.

any idea, why am getting this, or at least a heads up on where or where should i go to resolve this issue.

$holla = Mockery::mock('App\Holla');

?

@davedevelopment actually it pass (test is green), can you please explain to me why?
i spend more than 3hours with this problem, i think i deserve some explanation, and why Mockry didn't just throw a helpful error ?

Argument 1 passed to App\Temp::__construct() must be an instance of App\Holla

Message seems fairly helpful, it needs an instance of that type so that's the type to ensure you mock. It looks like you passed in a class name without the namespace.

Pretty much what @padraic says, I'd only add that there's not a whole lot Mockery can do at that point, other than registering our own error handler and that would be very messy.

@padraic yep, that's my mistake, when i saw instance of App\Holla i was thinking about the real object, which actually doesn't exist at all. then i notice after @davedevelopment that i should consider the one inside the class i want to test.
i've write about it here http://devgol.com/blog/improve-your-tests-by-mocking-objects/
thank you.

@OussamaDEV On your're blog you've put phpunit and mockery in require when they should be in require-dev.

Also, you should never use a wildcard version constraint.

@GrahamCampbell thank you i will fix that, for the wild card i just wanted to pull the latest, and actually i've found a little bug in phpunit am gonna report it later.

for the wild card i just wanted to pull the latest

That still makes no sense. PHPUnit 5 could totally break your code if if it's namespaced.

@GrahamCampbell fixed - thank you

Closing issue as it appears to be resolved.


How to keep policies up-to-date?

How can I ensure that my deployed ESLZ policies are always on the latest version? Do I have to manually check each month or can it somehow be automated?
For example, the update from February 2022: https://github.com/Azure/Enterprise-Scale/wiki/Whats-new#policy-1

Hey @vegazbabz,
Thanks for raising this.
The answer today depends on your implementation method.
If you are using the ALZ Terraform Module, https://aka.ms/alz/tf this is handled for you as you upgrade to the latest module versions.
For the portal, ARM or Bicep today you will need to keeping checking the whats new page, https://aka.ms/alz/whatsnew, and see whats changed. If new policies are added you can deploy the policy definitions again and the new ones will be added. However, if an already assigned policy is updated, if its parameters are changed you may not be able to update the definition in place due to policy limitations today.
If this happens you would have to remove the assignment then update the definition and then re-assign the policy.
We are aware this is a bit of a pain point today and are just starting to invest in a workstream called "evergreen" where we are investigating bits like this. Looping in @paulgrimley & @jfaurskov as they are scoping this currently.
Thanks
Jack

Linking to #662

Trigger ADO Sync 1

Trigger ADO Sync 2

@paulgrimley can this now be closed?

@jtracey93 yes we can close this as we now have the guidance https://aka.ms/alz/update @vegazbabz let us know if you have any feedback from consuming these and raise a new issue. Thanks


Chore: remove analyzer errors. Upgrade version surf_lint_rules.

Codecov Report

Merging #19 (c793ce6) into main (29fb411) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #19   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          178       178
=========================================
  Hits           178       178

Continue to review full report at Codecov.

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

Allready merged


How long do you train your model to converge

Hi, Thanks for your work!
I am curious about the time you train the default model from start to converge and the device you use.

which model? the one with GloVe only requires several hours on 3x2080Ti. BERT and Elmo should take longer but less than 1 day.


Add require syntax support

For webpack and similar bundlers, it is necessary to support require(module) syntax. This PR adds that and avoids ERR_PACKAGE_PATH_NOT_EXPORTED.

Thanks!


Add an indication to tell the user to save font pair.

Will add later a sketch or a screenshot of a design idea.

See 9f4bb85b0942a634a505714648fb741553189af8 and 123c24080e64042c07d5f1c092894d0305baf610.


feat: add lexical

Closes 380
Lexical support is already in lspconfig and the mason registry

@williambotman /fixup


Readme.md needs better platform support clarification

Whilst it is clear that to develop Windows Terminal, it is not clear where it is supported to run.
Therefore, somewhere near the top of the readme it should be clarified where this can be installed & whether or not it can/cannot be installed on older Windows OS'es (or could ever)

Apparently, I can't read properly (damn skim reading) - as it's there clear as day


Operation parameter extensions not parsed

I have the following in the swagger endpoint:

"/v1/Invoices/GetInventoryItemPrice": {
      "get": {
        "tags": [
          "Invoices"
        ],
        "summary": "Returns the price for the given inventory number between the two parties",
        "operationId": "V1InvoicesGetInventoryItemPriceGet",
        "consumes": [],
        "produces": [
          "application/json",
          "text/json",
          "application/xml",
          "text/xml",
          "application/bson"
        ],
        "parameters": [
          {
            "name": "fromContactId",
            "in": "query",
            "description": "",
            "required": false,
            "type": "integer",
            "format": "int32"
          },
          {
            "name": "toContactId",
            "in": "query",
            "description": "",
            "required": false,
            "type": "integer",
            "format": "int32"
          },
          {
            "name": "toContactType",
            "in": "query",
            "description": "",
            "required": false,
            "type": "string",
            "enum": [
              "User",
              "Organization"
            ],
            "x-ms-enum": {
              "name": "ContactTypes",
              "modelAsString": true,
              "values": [
                {
                  "value": 1,
                  "name": "User",
                  "description": "User"
                },
                {
                  "value": 2,
                  "name": "Organization",
                  "description": "Organization"
                }
              ]
            }
          },
          {
            "name": "inventoryId",
            "in": "query",
            "description": "",
            "required": false,
            "type": "integer",
            "format": "int32"
          },
          {
            "name": "units",
            "in": "query",
            "description": "",
            "required": false,
            "type": "number",
            "format": "double"
          }
        ],
        "responses": {
          "200": {
            "description": "Success",
            "schema": {
              "format": "double",
              "type": "number"
            }
          },
          "401": {
            "description": "Unauthorized"
          },
          "403": {
            "description": "Forbidden"
          }
        },
        "security": [
          {
            "oauth2": [
              "api1"
            ]
          }
        ]
      }
    },

Note toContactType:

{
            "name": "toContactType",
            "in": "query",
            "description": "",
            "required": false,
            "type": "string",
            "enum": [
              "User",
              "Organization"
            ],
            "x-ms-enum": {
              "name": "ContactTypes",
              "modelAsString": true,
              "values": [
                {
                  "value": 1,
                  "name": "User",
                  "description": "User"
                },
                {
                  "value": 2,
                  "name": "Organization",
                  "description": "Organization"
                }
              ]
            }
          },

which clearly has x-ms-enum as an extension.  If this was on a schema type, it works fine and the extension is pulled in. However when it's on a parameter of an operation it fails and the extension collection is 0 length.

Hey @JohnGalt1717 , can you confirm which version of the library you are using.  The latest code seems to be parsing this properly.

Microsoft.OpenApi.Readers V 1.0.1
Under .NET Core Console app if that matters. Param.Extensions.Count = 0

@JohnGalt1717 Thanks for confirming.  I hope to have a release of the 1.1 library tomorrow.  That should resolve the issue.

I tried the beta and it wasn't fixed there.

@JohnGalt1717 Any chance you have setup an ExtensionParser for x-ms-enum?

Nope. I just access it directly.

bump

@JohnGalt1717
With the latest code, I got the same result as Darrel above, with the x-ms-enum being deserialized and serialized correctly. I also see the Extension populated in the DOM as well.

Any chance you could share with us a bit more about your set-up? I'm not sure what else can affect this apart from the ExtensionParsers that Darrel already mentioned.

Also, if you could try running the Microsoft.OpenApi.Workbench project and putting your document there, do you see the x-ms-enum showing up in the result?


Being able to view the Quote Reposts under the original post

Sites like Twitter have a section where you can view the quote tweet without going to the person who quote tweeted the account.
I feel like BlueSky needs a feature like this to make what folks are saying in response to a quote post more user friendly and easy to understand. This screenshot shows where a "View Quote Posts" tab can go.

This is under discussion, see #1050.


Change base route for django app to /v2/

Currently, the Django-app base route is / and the osf route is /api/v2/. Because all routes should be versioned, the base route for the django app should be /v2/.

Closing this since the PR to change the base route has been merged.


when Download button is press it only shows the file

When you try to click the download button it shows the mp3 file to download on a separate sceen than open the file location on where to save it, you have to download the file again or save source as..

This can be fixed by adding the HTML5 attribute download to the download link. Example:
Get Audio.
Sadly I don't have time to fix it myself.

Thanks

Cheers @Code1110 👍 We did have the download attribute at some point but it must have been a regression as part of a refactor I guess.


[Amplify-AppSync-Simulator] defaultIfNullOrBlank does not return default correctly for empty string.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v18.13.0
Amplify CLI Version
2.9.4
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Describe the bug
Using $util.defaultIfNullOrEmpty to test render templates, always returns the first argument if it is an empty string defined in the template. Reference code:
https://github.com/aws-amplify/amplify-cli/blob/dev/packages/amplify-appsync-simulator/src/velocity/util/general-utils.ts#L107
  defaultIfNullOrBlank(value, defaultValue) {
    if (value) return value;
    return defaultValue;
  }

This looks like it occurs b/c an empty string coming into this function is a wrapped JavaString object so the truthy check fails.
It looks like if you were to follow suit and utilize the same approach as $util.isNullOrEmpty it should correct it.
Expected behavior
$util.defaultIfNullOrEmpty should work correctly for empty strings
Reproduction steps
Within a VTL Template:
#set ($emptyVal = "")
#set ($defaultVal = "default")
$util.defaultIfNullOrEmpty($emptyVal, $defaultVal)

Results in a rendered template of:

Where it should be:
default

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hi @KenFigueiredo, thank you for reporting this issue. I've checked the expected behavior from the AppSync console just to verify that this bug only impacts the mock plugin and confirmed that it is. My test function definition looks like:
#set ($emptyVal = "")
#set ($defaultVal = "default")
#set ($Val = $util.defaultIfNullOrEmpty($emptyVal, $defaultVal))
{
    "payload": {
        "hello": $Val,
    }
}

and it gives {hello: default} back. Marking this as a bug in the Appsync simulator.


External IP's getting successful static GET request with Auth Enabled

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
Synology NAS sitting behind NPM. Basic Auth is enabled and works for main root protection, but logs show external IP's issuing a successful static GET request and accessing images using this call:
[server address]/webapi/entry.cgi?api=SYNO.Core.Synohdpack&version=1&method=getHDIcon&res=24&retina=false&path=webman/3rdparty/DownloadStation/images/download_station_{0}.png
Also:
webman/3rdparty/FileBrowser/images/icon/FileStation_{0}.png
webman/3rdparty/Virtualization/images/VirtualManagement_{0}.png
webman/3rdparty/SynologyPhotos/images/icon/photos_{0}.png
This skips straight past the auth and shows the file, allowing the person sending this to know a Synology NAS is present.
Issuing just [server address]/webapi correctly asks for auth.
Nginx Proxy Manager Version
2.9.18
To Reproduce
Can provide the URL to my server for @jc21 or similar to assess how to resolve.
Expected behavior
Expect auth to be required for any access to the server including this. How to block "SYNO.Core.Synohdpack" request?
Screenshots
n/a
Operating System
n/a - But tested on Windows client, Mac client and iOS client and all show the same.
Additional context
n/a

For the moment I've added this to my Advanced "Custom NGINX Config":
location = /webapi {
allow <IP_ADDRESS>/24;
allow <IP_ADDRESS>;
deny  all;
}


Not all links are found when custom target_content is set.

I wanted to try out this package for an internal RAG presentation based on some of our company website data. Our website has the typical structure: header, main content and footer. Most of the links are in the header and footer part, but if I set target_content to only the main container, no links are getting found.
Is there a way to have all URLs collected outside of the container i ultimately want to parse into markdown? When i don't specify the target i am getting quite a mess of navigation and footer link data that repeats on every page (which would have to be cleaned up for every single page)..
Thanks in advance!

Edit: I think it was actually a combination of the domain, base path flags and the target content that returns zero results. once I removed all the extra flags, the target content worked as advertised.

hey @spaceemotion 👋 ... stoked you're giving md_crawler a try!
what would help tremendously is if you provided a sample url to html (or DM/email me a zip) where this is failing and args you tested. it'll make debugging easier. thx!

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
soone, @AndreyBronin, @itofarina, @MuhtasimTanmoy, @fromddy, @gmh5225, @ittranducloc, @rpedroni, @nasjp, @0xbeny, @0xfinder, @clemlak, @scolear, @piotr-roslaniec, @calnix, @ik5, @xyzpark92, @UncleJ4ck, @P-HOW, @c22, @bramazan, @TheStarBoys, @ciphersaw, @aminah-io, @lookfirst, @Marvinmw, @zklim, @zacyapjq, @fynch3r, @Bongulielmi, @deenrookie, @nguyenhoaibao, @floriansr, @tejastn10, @israelboudoux, @5z1punch, @aghareza, @vacekj, @april1989, @erikreppel, @L-Kov, @nexzaros, @icepick41, @michiboo, @brianonn, @ezeql, @mark2021blue, @0xknstntn, @Dha0, @AskBlockchain


not working with prettier-plugin-sort-json

Figured it out. Fix will be included in next release.

fix included in release 2.0.0


Teams: do not post sigs that will result in multiple UVs

When doing AddMember and there already is a member with same uid but different EldestSeqno, remove the old member (in the same sig).
When doing RemoveMember on a reset member, use proper UV instead of posting remove with user's current EldestSeqno (might also be 0 when they haven't reprovisioned yet?).

That exponential backoff would take 102300ms in total? The simple version that doesn't take so many seconds is better.

Hah, you're right, sorry for that. However, sometimes it manages to fail even with the 10 second limit...

reverted back to the simple version


Feature mempool refactor

Fixes:

https://github.com/dusk-network/dusk-blockchain/issues/165
https://github.com/dusk-network/dusk-blockchain/issues/167
https://github.com/dusk-network/dusk-blockchain/issues/168
https://github.com/dusk-network/dusk-blockchain/issues/169

Codecov Report

Merging #170 into master will increase coverage by 0.02%.
The diff coverage is 59.37%.

@@            Coverage Diff             @@
##           master     #170      +/-   ##
==========================================
+ Coverage   59.56%   59.59%   +0.02%
==========================================
  Files         108      108
  Lines        5728     5761      +33
==========================================
+ Hits         3412     3433      +21
- Misses       1804     1811       +7
- Partials      512      517       +5

Impacted Files
Coverage Δ

pkg/util/nativeutils/rpcbus/methods.go
0% <ø> (ø)
:arrow_up:

pkg/core/mempool/mempool.go
62.37% <53.75%> (-1.01%)
:arrow_down:

pkg/core/mempool/hashmap.go
70.68% <87.5%> (+2.6%)
:arrow_up:

Continue to review full report at Codecov.

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


Unzipped file.zip in the root directory and deleted the zip file

@hasan-mavlonov I am sorry, the domain case.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

@hasan-mavlonov While the domain has been abused, I wish to reconsider the issue. Could you please tell us whether case.edu had any improvements in the email usage policy? Earlier many subdomains were accessible for random people not related to university anymore (or at all). Do people still exploit domains such as survey.case.edu, etc.?

HEy,

@hasan-mavlonov While the domain has been abused, I wish to reconsider the issue. Could you please tell us whether case.edu had any improvements in the email usage policy? Earlier many subdomains were accessible for random people not related to university anymore (or at all). Do people still exploit domains such as survey.case.edu, etc.?

Hey, Philip Torchinsky
I'm not really informed about the history of its email issues. I'm just a student who got access after enrolling at Case Western Reserve University. I guess only teachers and students have an access to that domain, because its library is accessible only to people with with email ending with case.edu

@hasan-mavlonov I am sorry, the domain case.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

@hasan-mavlonov OK, Let's review the case and merge the request. We'll monitor case.edu usage closely.
Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


VSCode Insiders, Jupyter notebook not loading in remote, extension does not seem to be loading

Whenever I try to open a jupyter notebook in remote using VSCode insiders I get this message

"Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed or enabled."

and an option to "Reopen file with VS Code standard text editor"
Jupyter, Python, and Pylance are installed.
I do Not get this error in regular VSCode, this only happens in Insiders
Environment data

VS Code version: Version: 1.57.0-insider

Jupyter Extension version (available under the Extensions sidebar): v2021.6.885080120

Python Extension version (available under the Extensions sidebar): v2021.5.842923320

OS (Windows | Mac | Linux distro) and version: local, mac os 11.2 (20D64), remote, linux ubuntu 20.04

Python and/or Anaconda version: Python 3.6.9 64-bit.

Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A

Jupyter server running: N/A

Expected behaviour
Should open
Actual behaviour
Does not open
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]
Connect to a remote server.
right click on jupyter notebook file -> open with -> Jupyter Notebook.
From looking at this issue
https://github.com/microsoft/vscode-jupyter/issues/586
It seems that the extension is not loading.

@Santosh-Gupta please could you provide the contents of the Jupyter output panel in the terminal panel?

I figured out the issue. I noticed that there was a box on the bottom left that said 'Restricted mode'. I clicked on it and selected 'trust'. Now it's working again.
Sorry for the inconvenience.


9/20/2022 Publish

Docs Build status updates of commit ed7a19e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:warning:Warning
View
Details

2017-03-09-profile/docs-ref-autogen/aks.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/network/firewall.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/api-portal.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/api-portal/custom-domain.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/binding.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/binding/cosmos.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/binding/mysql.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/binding/redis.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/custom-domain.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/deployment.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/identity.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/app/log.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/application-configuration-service.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/application-configuration-service/git/repo.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/build-service/builder.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/build-service/builder/buildpack-binding.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/certificate.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/gateway.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/gateway/custom-domain.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/gateway/route-config.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/service-registry.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2017-03-09-profile/docs-ref-autogen/spring/storage.yml
:bulb:Suggestion
View (azure-cli-2017-03-09-profile)
Details

2018-03-01-hybrid/docs-ref-autogen/aks.yml
:bulb:Suggestion
View (azure-cli-2018-03-01-hybrid)
Details

This comment lists only the first 25 files in the pull request.

docs-ref-conceptual/azure-cli-extensions-list.md

Line 40, Column 2: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/azure-arc/data/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 40, Column 113: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/azure-arc/data/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 40, Column 2: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/azure-arc/data/' will be broken in isolated environments. Replace with a relative link.
Line 40, Column 113: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/azure-arc/data/' will be broken in isolated environments. Replace with a relative link.
Line 45, Column 2: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/python/api/overview/azure/ml/?view=azure-ml-py' will be broken in isolated environments. Replace with a relative link.
Line 45, Column 2: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: https://docs.microsoft.com/python/api/overview/azure/ml/?view=azure-ml-py
Line 45, Column 167: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/python/api/overview/azure/ml/?view=azure-ml-py/HISTORY.rst' will be broken in isolated environments. Replace with a relative link.
Line 45, Column 167: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: https://docs.microsoft.com/python/api/overview/azure/ml/?view=azure-ml-py/HISTORY.rst

2017-03-09-profile/docs-ref-autogen/aks.yml

Line 1164, Column 39: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'target' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 1185, Column 290: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'target' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/network/firewall.yml

Line 242, Column 93: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'indexToRemove' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 245, Column 116: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'value' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/spring/api-portal.yml

Line 25, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 41, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 67, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/spring/api-portal/custom-domain.yml

Line 34, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 57, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 78, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 99, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 124, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

2017-03-09-profile/docs-ref-autogen/spring/app.yml

Line 44, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 82, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 151, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 269, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 333, Column 117: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

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


Use IA config to change IA

Badly explained, but like, it would be good if the way you config the IA was also the way you could move/delete/rename sections of the site or something.

Fixed in #278


Create release process

We added a version of nebraska as a string to the code, so now when doing a release, we will need to remember to bump it. From my experience this thing is almost always forgotten about. To avoid that, we need a document about the release process and maybe some scripts to automate some parts of the process.

ref #192

@joaquimrocha this can be closed now

:+1:


Get Server Error, Code: 10001, System errror after click ERC20TRANSFER button

I/okhttp.OkHttpClient: --> POST https://api.particle.network/evm-chain/rpc
I/okhttp.OkHttpClient: Content-Type: application/json; charset=UTF-8
I/okhttp.OkHttpClient: Content-Length: 256
I/okhttp.OkHttpClient: {"chainId":1,"id":"fa9f8322-54e1-4de1-9480-2b5aa03512d7","jsonrpc":"2.0","method":"particle_abi_encodeFunctionCall","params":["0x84b9B910527Ad5C03A9Ca831909E21e236EA7b06","erc20_transfer",["0x4F96Fe3b7A6Cf9725f59d353F723c1bDb64CA6Aa","10000000000000"],""]}
I/okhttp.OkHttpClient: --> END POST (256-byte body)
I/okhttp.OkHttpClient: <-- HTTP FAILED: com.particle.api.infrastructure.net.data.ServerException: Server Error, code: 10001, message: System error

If you don't have the fourth parameter, don't pass,the next version will fix this problem.
{
"chainId": 1,
"id": "fa9f8322-54e1-4de1-9480-2b5aa03512d7",
"jsonrpc": "2.0",
"method": "particle_abi_encodeFunctionCall",
"params": ["0x84b9B910527Ad5C03A9Ca831909E21e236EA7b06", "erc20_transfer",
["0x4F96Fe3b7A6Cf9725f59d353F723c1bDb64CA6Aa", "10000000000000"]]
}


Fix guacamole image name bug

What is being addressed
My previous PR introduced a bug due to a spelling issues. This fixes it but also directly using the custom porter property for the image name.

/test-extended

/test


Class name autocomplete does not suggest local aliases

Created new file in an exiting composer project
There's a Zend\Diactoros\ServerRequest class available somewhere in vendor folder
Typed:<?php
namespace zzz;
use Zend\Diactoros\ServerRequest as SomeRequest;

$q = new SomeReq

Pressed Ctrl+Space
Observed:

'No suggestions'
In developer console there is suggestion returned, but language server suggests original short class name (ServerRequest in this case):[Extension Host] Content-Type: application/vscode-jsonrpc; charset=utf8
Content-Length: 878

{"result":{"isIncomplete":true,"items":[{"label":"ServerRequest","kind":7,"detail":"Zend\\Diactoros","documentation":"Server-side HTTP request\n\nExtends the Request definition to add methods for accessing incoming data,\nspecifically server parameters, cookies, matched path parameters, query\nstring arguments, body parameters, and upload file information.\n\n\"Attributes\" are discovered via decomposing the request (and usually\nspecifically the URI path), and typically will be injected by the application.\n\nRequests are considered immutable; all methods that might change state are\nimplemented such that they retain the internal state of the current\nmessage and return a new instance that contains the changed state.","sortText":null,"filterText":null,"insertText":null,"textEdit":null,"additionalTextEdits":null,"command":null,"data":null}]},"id":94,"jsonrpc":"2.0"}

I suspect either vscode-side extension or vscode itself filters this suggestion out because it doesn't match what has been typed.
Couldn't be local alias returned instead?

Is this still happening?

It does. Menu shows original class name, selecting suggestion inserts alias. However, if original class name doesn't fuzzy-match the typed text, suggestions are filtered, and there's nothing to select from:


Not correct names/icons/items

In the vote crate, several items aren't labeled right and thus give different rewards. Such as the items in the picture [Which were much better in my opinion actually lol]. On a side note, I also lost a vote key when the server went down just as I was opening the crate.

Just to confirm, you are running Pixelmon 9.0.2 yes? There's currently a mismatch of block/item IDs if you login with either Serene Seasons, or Pixelmon 9.0.1.

Closing, OP inactive.


Trasport proto impl

This implements transport(botd2) protocol.
This closes #190
@tiero @altafan please review.

@sekulicd we must update also the macaroon permissions.
We need to define a new entity "transport" and add the following permission in both AllPermissionsByMethod() and Whitelist():

"/Transport/SupportedContentTypes": {{
	Entity: EntityTransport,
	Action: "read",
}},

Done


Feature Request - OS X Support

This would be an great feature. Maybe this is possible already?

Good idea! I just added shared iOS and OSX schemes in the swift1p2 branch. Let me know if you need help setting it up. Note that you'll need Xcode 6.3 Beta 3 to build that branch.

Great, thanks!


Addon Name

Addon Manifest URL
https://tv.egydead.live/a3/manifest.json
Addon Description
No response
Language of Content
No response
Choose Labels

[X] anime
[X] asian drama
[X] bollywood
[ ] cartoons
[ ] debrid support
[X] http streams
[X] live tv
[ ] metadata
[X] misc
[X] movies
[X] music
[ ] nsfw
[ ] podcasts
[X] radios
[X] subtitles
[X] torrents
[X] tv shows
[ ] video games

i don't know what this is, but it's not a new addon, closing it


There are no commands defined in the "eav:media" namespace

Hello, I'm running Magento 2.3.6 with PHP 7.3. I have tried making sure the module is enabled. Other commands show up and work just fine. Any advice for me?

How exactly did you install the module? When you run bin/magento setup:upgrade, does it show up in your app/etc/config.php?

I have installed it via composer using the command in the readme. It does show in the config.php 'Hackathon_EAVCleaner' => 1,
I tried completely uninstalling it, removing it from the setup_module table and running cache:clean, cash:flush, setup:upgrade, then reinstalling it via composer and running setup:upgrade. It then shows in config.php but not in setup_module table.

Then I do not have any idea to be honest. It works flawlessly for me under 2.4.3. I do not see any reason why it should not work on 2.3.6 (although you should really update ^^).

Ok thank you anyway. I am in the process of updating now and issue occurs with both 2.3.6 and 2.4.3. I will report back if I find a solution.

I have installed it via composer using the command in the readme. It does show in the config.php 'Hackathon_EAVCleaner' => 1, I tried completely uninstalling it, removing it from the setup_module table and running cache:clean, cash:flush, setup:upgrade, then reinstalling it via composer and running setup:upgrade. It then shows in config.php but not in setup_module table.

It's correct that this module won't show in the setup_module table since it has no version in module.xml, so no problems there.
You could try to run php bin/magento setup:di:compile to see if you have any errors in your DI config, that could cause this behaviour.

Great suggestion about setup:di:compile. I did have an error in one of my custom extensions causing the issue. Thanks to you both for the help!

Glat that you could solve it, @cypher909!
Nice catch and debugging tip, @Tjitse-E!


Getting Started section should state that a div with id="commento" is needed

No comment textareas or buttons would show up following the Getting Started section in README.md, which is simply loading commento.min.js and init(). A div with id="commento" is needed in addition to let the comment boxes show up. (Correct me if I'm wrong.)
The Getting Started section should state that a div with id="commento" is needed, or it would be very confusing.
I will make a pull request.

For pull request see https://github.com/adtac/commento/pull/150 .


ENCODER & DECODER: set params on all encoder/decoder instances, unconditionally

OSSL_DECODER_CTX_set_params() and OSSL_ENCODER_CTX_set_params() would
stop as soon as a decoder / encoder instance failed, which leaves the
rest of them with a possibly previous and different value.
Instead, these functions will now call them all, but will return 0 if
any of the instance calls failed.

I'm wondering if it is correct to fail if any of the encoders/decoders fail. If some pass and others fail that might be ok. But I can't think what better behaviour might look like.

We'd have to have the concept of half failures...  whatever that would mean.  (30% failed, 50% failed, 74.3% failed...  maybe we should return a float :laughing:)

You could have 0 == all failed, 1 == some success, 2 == all success. That way anything greater than 0 is just "success", and if you want to know that they all succeeded then you could get that information too.

Mind you, what does "fail" actually mean for an encoder/decoder? Does it mean I don't understand this, or does it mean I do understand this, tried to process it, but couldn't. If the latter then perhaps it is ok to just say if any fail then the overall result is 0.

You could have 0 == all failed, 1 == some success, 2 == all success.

I'm aware openssl has an idiosyncratic relationship with error codes, but seeing this, I have to ask: why not
0 == all passed
1,2,3,... == number of failures

?

I'm aware openssl has an idiosyncratic relationship with error codes, but seeing this

Because virtually all functions return 0 for fail and 1 for success (including all the other *_set_params functions). This would reverse the logic, which would be hugely confusing.

Should we really re-invent yet another return code structure?  We already have a couple of fairly common ones:

the binary (0 or 1)
positive vs negative (1 = success; 0, -1, -2, ... = diverse "levels" of failure)

These both allow using fn(...) > 0 to find out if a call was successful.  Now you're inventing new return code structures with wildly different patterns!
If there is to be some kind of count of failures, I'd suggest going for the negative, that would at least be some sort of compatible with already existing patterns.  1 = success, 0 = all failed, -n = negated count of failures.
Seriously, though, I'm not sure why the count of failures is relevant.  What is the caller going to do with that?  That any of the en-/decoder instances has failed should tell the caller that they shouldn't expect any well defined result if they continue to use the OSSL_ENCODER_CTX where setting the params failed.

All good, was just asking because "1 == some success, 2 == all success" IMO sounds like a footgun waiting to happen.

positive vs negative (1 = success; 0, -1, -2, ... = diverse "levels" of failure)
1 = success, 0 = all failed, -n = negated count of failures.

Both reasonable.

Either way, this PR is approved...  I guess we can come back to failure rates at another time, if that turns out to be relevant

Either way, this PR is approved... I guess we can come back to failure rates at another time, if that turns out to be relevant

Yes, its fine. Just me thinking out aloud.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged
9096809b209a03eb3948242e702b19526e675d57 ENCODER & DECODER: set params on all encoder/decoder instances, unconditionally


DOMException: Failed to execute 'addColorStop' on 'CanvasGradient'

Hi! hey so am using this library and it has worked really well until for some reason on an specific html it throws this error DOMException: Failed to execute 'addColorStop' on 'CanvasGradient': The provided value (-0.914214) is outside the range (0.0, 1.0).(…)
this is the full error!
DOMException: Failed to execute 'addColorStop' on 'CanvasGradient': The provided value (-0.914214) is outside the range (0.0, 1.0).
at svg.Element.linearGradient.createGradient (bundle.js:7256:8)
at svg.Property.getFillStyleDefinition (bundle.js:5938:18)
at svg.Element.rect.setContext (bundle.js:6480:34)
at svg.Element.rect.render (bundle.js:6362:11)
at svg.Element.g.renderChildren (bundle.js:6390:23)
at svg.Element.g.render (bundle.js:6372:17)
at svg.Element.g.renderChildren (bundle.js:6390:23)
at svg.Element.g.render (bundle.js:6372:17)
at svg.Element.g.renderChildren (bundle.js:6390:23)
at svg.Element.g.render (bundle.js:6372:17)
Anyone has any recomendations?

Should be fixed in 1.0.0


Create bs.json

Currently, am working as a Software Engineer. In my free time, I like write blogs on medium (http://alenvlahovljak.medium.com/), etc. I'd like to to contribute to my mother tongue so I can help my community to learn to react easy. I have previous experience regarding translation. I work on a translation on a project GoCrypto: https://gocrypto.com/en/

Hi folks, sorry for no response.
We've been working on new React docs, and so translations of the old site were deprioritized.
In the meantime, it looks like our GitHub permissions sync may have broke.
Now that we've rolled out the new site and reset the repositories for new translations, I'm going to close all old pull requests including this one so that we can start from a clean slate. If you'd like to still make the same change, please open a new PR.
Thanks, and sorry for the churn.


reusePort not supported on android

Steps to reproduce
using essentially the example code this works on ios but on android the following error occurs
error third_party/dart/runtime/bin/socket_android.cc:147: reusePort not supported for Android.
 void _startDiscovery() async {
    // Start mDNS client
    await mdns.start();

    // Get the PTR record for the service
    await for (final PtrResourceRecord ptr in mdns
        .lookup<PtrResourceRecord>(ResourceRecordQuery.serverPointer(_serviceType))) {
      await for (final SrvResourceRecord srv in mdns.lookup<SrvResourceRecord>(
          ResourceRecordQuery.service(ptr.domainName))) {
        final String bundleId = ptr.domainName;
        ReadDevice(widget.isar, srv.target);
      }
    }

    // Stop mDNS client when done
    mdns.stop();
  }

Expected results
should discover mdns services
Actual results
fails
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

there was an old thread that claimed to fix this by providing rawDatagramSocketFactory but this results in the same error
class _ControllerPanelState extends State<ControllerPanel> {
  final String _serviceType = "_wled._tcp.local";
  final mdns = MDnsClient(
    // Create a custom RawDatagramSocketFactory to avoid the reusePort issue on Android
    rawDatagramSocketFactory: (dynamic host, int port,
        {bool reuseAddress = true,
          bool reusePort = false,
          int ttl = 255}) {
      return RawDatagramSocket.bind(host, port,
          reuseAddress: reuseAddress,
          reusePort: reusePort,
          ttl: ttl);
    },
  );
  List<Device> _devices = [];

  void _startDiscovery() async {
    // Start mDNS client
    await mdns.start();

    // Get the PTR record for the service
    await for (final PtrResourceRecord ptr in mdns
        .lookup<PtrResourceRecord>(ResourceRecordQuery.serverPointer(_serviceType))) {
      await for (final SrvResourceRecord srv in mdns.lookup<SrvResourceRecord>(
          ResourceRecordQuery.service(ptr.domainName))) {
        final String bundleId = ptr.domainName;
        ReadDevice(widget.isar, srv.target);
      }
    }

    // Stop mDNS client when done
    mdns.stop();
  }

Dart Socket ERROR: ../../third_party/dart/runtime/bin/socket_android.cc:147: reusePort not supported for Android.

Hi @TheMattSchiller
I checked this issue with package example on Realme 6, Android 11 device but it worked without issue. Could you also try this example and confirm?
Which Android device are you using? (OS, model)
Also, please provide the package version you see this issue and the output of flutter doctor -v. Thanks!

so i tried the example but get the same output
my code
   Future<void> _discoverDevices() async {
    // Parse the command line arguments.

    const String name = '_wled._tcp.local';
    final MDnsClient client = MDnsClient();
    // Start the client with default options.
    await client.start();

    // Get the PTR record for the service.
    await for (final PtrResourceRecord ptr in client
        .lookup<PtrResourceRecord>(ResourceRecordQuery.serverPointer(name))) {
      // Use the domainName from the PTR record to get the SRV record,
      // which will have the port and local hostname.
      // Note that duplicate messages may come through, especially if any
      // other mDNS queries are running elsewhere on the machine.
      await for (final SrvResourceRecord srv in client.lookup<SrvResourceRecord>(
          ResourceRecordQuery.service(ptr.domainName))) {
        // Domain name will be something like<EMAIL_ADDRESS>        final String bundleId =
            ptr.domainName; //.substring(0, ptr.domainName.indexOf('@'));
        print('WLED instance found at'
            '${srv.target}:${srv.port} for "$bundleId".');
        ReadDevice(widget.isar, srv.target);
      }
    }
    client.stop();

    print('Done scanning');
  }

output in debug:
D/EGL_emulation( 9401): app_time_stats: avg=57.17ms min=3.11ms max=2296.22ms count=49
E/Dart    ( 9401): Dart Socket ERROR: ../../third_party/dart/runtime/bin/socket_android.cc:147: `reusePort` not supported for Android.
E/Dart    ( 9401): Dart Socket ERROR: ../../third_party/dart/runtime/bin/socket_android.cc:147: `reusePort` not supported for Android.
E/Dart    ( 9401): Dart Socket ERROR: ../../third_party/dart/runtime/bin/socket_android.cc:147: `reusePort` not supported for Android.
E/Dart    ( 9401): Dart Socket ERROR: ../../third_party/dart/runtime/bin/socket_android.cc:147: `reusePort` not supported for Android.
D/EGL_emulation( 9401): app_time_stats: avg=20.51ms min=3.38ms max=507.80ms count=32
I/flutter ( 9401): Done scanning

my flutter doctor output:
[✓] Flutter (Channel stable, 3.16.7, on macOS 13.5.2 22G91 darwin-arm64, locale en-US)
    • Flutter version 3.16.7 on channel stable at /Users/mattschiller/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ef1af02aea (5 days ago), 2024-01-11 15:19:26 -0600
    • Engine revision 4a585b7929
    • Dart version 3.2.4
    • DevTools version 2.28.5

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/mattschiller/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/mattschiller/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.83.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 (mobile)          • 4E9CA3E1-F3A0-4E49-93AA-648464856264 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 13.5.2 22G91 darwin-arm64
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 120.0.6099.216

[✓] Network resources
    • All expected network resources are available.

• No issues found!

im running on an emulated pixel 7 pro API 34 device and a physical samsung galaxy s21 with android version 13

thank you so much for taking a look at this its a total stumper for me. This works beautifully on iOS

some potentially interesting discoveries. with this code i can avoid the error but iOS complains. Android never discovers a device
  final _mdns = MDnsClient(
    // Create a custom RawDatagramSocketFactory to avoid the reusePort issue on Android
    rawDatagramSocketFactory: (dynamic host, int port,
        {bool reuseAddress = true,
          bool reusePort = false,
          int ttl = 255}) {
      return RawDatagramSocket.bind(host, port,
          reuseAddress: reuseAddress,
          reusePort: false,
          ttl: ttl);
    },
  );

  Future<void> _discoverDevices() async {
    // Parse the command line arguments.

    const String name = '_wled._tcp.local';
    // Start the client with default options.
    await _mdns.start();

    // Get the PTR record for the service.
    await for (final PtrResourceRecord ptr in _mdns
        .lookup<PtrResourceRecord>(ResourceRecordQuery.serverPointer(name))) {
      // Use the domainName from the PTR record to get the SRV record,
      // which will have the port and local hostname.
      // Note that duplicate messages may come through, especially if any
      // other mDNS queries are running elsewhere on the machine.
      await for (final SrvResourceRecord srv in _mdns.lookup<SrvResourceRecord>(
          ResourceRecordQuery.service(ptr.domainName))) {
        // Domain name will be something like<EMAIL_ADDRESS>        final String bundleId =
            ptr.domainName; //.substring(0, ptr.domainName.indexOf('@'));
        print('WLED instance found at'
            '${srv.target}:${srv.port} for "$bundleId".');
        ReadDevice(widget.isar, srv.target);
      }
    }
    _mdns.stop();

    print('Done scanning');
  }

android output:
D/EGL_emulation( 9926): app_time_stats: avg=490.70ms min=8.27ms max=973.12ms count=2
I/flutter ( 9926): Done scanning

iOS output:
#0      _NativeSocket.bindDatagram (dart:io-patch/socket_patch.dart:1051:7)
<asynchronous suspension>
#1      _RawDatagramSocket.bind.<anonymous closure> (dart:io-patch/socket_patch.dart:2558:15)
<asynchronous suspension>
#2      MDnsClient.start (package:multicast_dns/multicast_dns.dart:110:40)
<asynchronous suspension>
#3      _ControllerPanelState._discoverDevices (package:alienlights/device/device_widget.dart:44:5)
<asynchronous suspension>

working iOS output with previous code
flutter: WLED instance found atorbital1.local:80 for "orbital1._wled._tcp.local".
flutter: WLED instance found atorbital1.local:80 for "orbital1._wled._tcp.local".
flutter: Done scanning

I wonder if this issue only reproduces on emulator. Could you help to check this on a physical device (if you have one)? (my Android emulator can not boot up currently)

@TheMattSchiller I am facing this same issue

some potentially interesting discoveries. with this code i can avoid the error but iOS complains. Android never discovers a device
  final _mdns = MDnsClient(
    // Create a custom RawDatagramSocketFactory to avoid the reusePort issue on Android
    rawDatagramSocketFactory: (dynamic host, int port,
        {bool reuseAddress = true,
          bool reusePort = false,
          int ttl = 255}) {
      return RawDatagramSocket.bind(host, port,
          reuseAddress: reuseAddress,
          reusePort: false,
          ttl: ttl);
    },
  );

  Future<void> _discoverDevices() async {
    // Parse the command line arguments.

    const String name = '_wled._tcp.local';
    // Start the client with default options.
    await _mdns.start();

    // Get the PTR record for the service.
    await for (final PtrResourceRecord ptr in _mdns
        .lookup<PtrResourceRecord>(ResourceRecordQuery.serverPointer(name))) {
      // Use the domainName from the PTR record to get the SRV record,
      // which will have the port and local hostname.
      // Note that duplicate messages may come through, especially if any
      // other mDNS queries are running elsewhere on the machine.
      await for (final SrvResourceRecord srv in _mdns.lookup<SrvResourceRecord>(
          ResourceRecordQuery.service(ptr.domainName))) {
        // Domain name will be something like<EMAIL_ADDRESS>        final String bundleId =
            ptr.domainName; //.substring(0, ptr.domainName.indexOf('@'));
        print('WLED instance found at'
            '${srv.target}:${srv.port} for "$bundleId".');
        ReadDevice(widget.isar, srv.target);
      }
    }
    _mdns.stop();

    print('Done scanning');
  }

android output:
D/EGL_emulation( 9926): app_time_stats: avg=490.70ms min=8.27ms max=973.12ms count=2
I/flutter ( 9926): Done scanning

iOS output:
#0      _NativeSocket.bindDatagram (dart:io-patch/socket_patch.dart:1051:7)
<asynchronous suspension>
#1      _RawDatagramSocket.bind.<anonymous closure> (dart:io-patch/socket_patch.dart:2558:15)
<asynchronous suspension>
#2      MDnsClient.start (package:multicast_dns/multicast_dns.dart:110:40)
<asynchronous suspension>
#3      _ControllerPanelState._discoverDevices (package:alienlights/device/device_widget.dart:44:5)
<asynchronous suspension>

working iOS output with previous code
flutter: WLED instance found atorbital1.local:80 for "orbital1._wled._tcp.local".
flutter: WLED instance found atorbital1.local:80 for "orbital1._wled._tcp.local".
flutter: Done scanning

leaving me with this option however the device discovery still does not work on android
@override
class _ControllerPanelState extends State<ControllerPanel> {
 List<Device> _devices = [];
 final _mdns = MDnsClient(
   // Create a custom RawDatagramSocketFactory to handle reusePort based on the platform
   rawDatagramSocketFactory: (dynamic host, int port,
       {bool reuseAddress = true,
         bool reusePort = false,
         int ttl = 255}) {
     if (Platform.isIOS) {
       reusePort = true; // On iOS, reusePort is set to true
     } else {
       reusePort = false; // On Android, reusePort is set to false
     }
     return RawDatagramSocket.bind(host, port,
         reuseAddress: reuseAddress,
         reusePort: reusePort,
         ttl: ttl);
   },
 );

 Future<void> _discoverDevices() async {
   // Parse the command line arguments.

   const String name = '_wled._tcp.local';
   // Start the client with default options.
   await _mdns.start();

   // Get the PTR record for the service.
   await for (final PtrResourceRecord ptr in _mdns
       .lookup<PtrResourceRecord>(ResourceRecordQuery.serverPointer(name))) {
     // Use the domainName from the PTR record to get the SRV record,
     // which will have the port and local hostname.
     // Note that duplicate messages may come through, especially if any
     // other mDNS queries are running elsewhere on the machine.
     await for (final SrvResourceRecord srv in _mdns.lookup<SrvResourceRecord>(
         ResourceRecordQuery.service(ptr.domainName))) {
       // Domain name will be something like<EMAIL_ADDRESS>       final String bundleId =
           ptr.domainName; //.substring(0, ptr.domainName.indexOf('@'));
       print('WLED instance found at'
           '${srv.target}:${srv.port} for "$bundleId".');
       ReadDevice(widget.isar, srv.target);
     }
   }
   _mdns.stop();

   print('Done scanning');
 }

ios ouput:
flutter: WLED instance found atorbital1.local:80 for "orbital1._wled._tcp.local".
flutter: WLED instance found atorbital1.local:80 for "orbital1._wled._tcp.local".
flutter: Done scanning

android output:
D/EGL_emulation( 9926): app_time_stats: avg=567.86ms min=16.60ms max=1314.59ms count=3
I/flutter ( 9926): Done scanning

Did you get any work around with this??

@TheMattSchiller I tried creating a similar Android emulator (Android 14) but the issue still doesn't appear. Checked on both multicast_dns example and your sample code below (I commented out ReadDevice as it's undefined):

Sample code
import 'package:multicast_dns/multicast_dns.dart';

Future<void> main() async {
  await _discoverDevices();
}

Future<void> _discoverDevices() async {
  // Parse the command line arguments.

  const String name = '_wled._tcp.local';
  final MDnsClient client = MDnsClient();
  // Start the client with default options.
  await client.start();

  // Get the PTR record for the service.
  await for (final PtrResourceRecord ptr in client
      .lookup<PtrResourceRecord>(ResourceRecordQuery.serverPointer(name))) {
    // Use the domainName from the PTR record to get the SRV record,
    // which will have the port and local hostname.
    // Note that duplicate messages may come through, especially if any
    // other mDNS queries are running elsewhere on the machine.
    await for (final SrvResourceRecord srv in client.lookup<SrvResourceRecord>(
        ResourceRecordQuery.service(ptr.domainName))) {
      // Domain name will be something like<EMAIL_ADDRESS>      final String bundleId =
          ptr.domainName; //.substring(0, ptr.domainName.indexOf('@'));
      print('WLED instance found at'
          '${srv.target}:${srv.port} for "$bundleId".');
      // ReadDevice(widget.isar, srv.target);
    }
  }
  client.stop();

  print('Done scanning');
}


Don't paint twice

In refactoring the way we generate the SkPicture, I accidentially made us
generate the SkPicture twice. This CL causes us to generate it only once.

lgtm


build: link in dependencies on Windows

This sets up the dependencies we need on Windows.  It helps reduce the
spew from the linker when building for windows.

CC: @millenomi @phausler @parkera @jakepetroules

@swift-ci please test


Add draft of DG enhancements for Appointments, Interface

LGTM. Do you want this to be merged or are you going to add more commits before merging?

This is what I am adding for now - I want to continue with this draft after I have modified the respective fields to use the implementation we have agreed upon.

Understood. Merging.


fix | list_collection_names requires unnecessary privileges (#680)

This fixes the issue I encountered in #680.
The idea is to only list collections to which the user has access so that it will be possible to use a View with partially restricted users.

@marinashe Could you pls check GH checks?

Ah, no, don't worry. It looks like it's an old issue that the linter caught after the update. I'll merge this at the beginning of next week.

Merged. Thank you very much for your PR!


cef_troubleshoot.py failing in RHEL due to TypeError in check_file_read_permissions

Describe the bug
When testing/troubleshooting the Sentinel Log Analytics Forwarder on a clean RHEL VM in Azure,
To Reproduce
Steps to reproduce the behavior:

Create a RHEL Marketplace VM in Azure using the following settings:
Publisher: RedHat
Offer: RHEL
SKU: 8.2
Follow the installation guide for deploying the Log Forwarder
Validate the installation and configuration as per this guide
The cef_troubleshoot.py will fail with the following error:

Traceback (most recent call last):
File "cef_troubleshoot.py", line 747, in <module> main()
File "cef_troubleshoot.py", line 721, in main omsagent_security_event_conf_validation(workspace_id=workspace_id)
File "cef_troubleshoot.py", line 547, in omsagent_security_event_conf_validation check_file_read_permissions(path, workspace_id)
File "cef_troubleshoot.py", line 466, in  octal_permissions = o.strip("\'\n")
TypeError: a bytes-like object is required, not 'str'

Expected behavior
Executing cef_troubleshoot.py won't fail
Desktop (please complete the following information):

OS: RedHat Enterprise Linux
Version: Release 8.2 (Ootpa)

Additional context
Working around the problem we have simply commented-out line 547 which reads
check_file_read_permissions(path, workspace_id)

Hi @AndyHerb,
Thanks a lot for bringing this to our attention.
Could you please open a support ticket on this issue so we could assist as soon as possible?
Thanks,
Noam.

@NoamLandress - case<PHONE_NUMBER>04772 raised as requested

@NoamLandress - Can we close this issue? Please let us know. Thanks.

Yes, @v-jayakal. Closing.


Ignore unreachable targets

I have a Sentinel in my spawn that is going nuts because there is a zombie below him that they can't actually get to. Is there anyway to have the Sentinel ignore unreachable targets?

It shouldn't be doing that at all 0.o -- there's a CanSee check, which minecraft may be filling incorrectly? I'll look into this.

Fixed! Thanks for the report :)

Great! Thanks for speedy fix :)


Fixed gz_ros2_control keys: Iron and Rolling

Due the renaming from ign to gz this package should use a different key, package names have changed too.

Iron is held for sync https://discourse.ros.org/t/preparing-for-iron-sync-2023-06-30/32099

@clalancette and @sloretz I will try a different approach, I will remove the key gz_ros2_control and I will release again gz_ros2_control

https://github.com/ros/rosdistro/pull/37952
https://github.com/ros/rosdistro/pull/37951

I'm going to close this one and assume that you'll redo the iron release in the same way as rolling above.


Don't show Zulip.com terms on other sites

While we already don't link to /terms anywhere on the site, they can still be
accessed if you navigate to /terms directly. Now, those routes will only be
exported on the Zulip.com service.
We should ideally provide a mechanism for deployments to specify their own
terms without modifying source code; in the interim, sites that have already
customised the provided Zulip.com terms can simply carry a patch reverting this
commit.
Testing:

[x] Shows up in development
[x] 404s if you force the conditional zproject/urls.py@L262 to fail

lgtm assuming you've tested!


Sort the choices in Facets

I still hope we'd have a way to sort options by count, but I don't think that's possible in the near future, unless we have a separate endpoint for qs information. I'm good with merging this PR.


CAF-3671: Updated to use Policy document from TaskData

It would be good to have the test check that the old field names are no longer present on metadata references. Aside from that it looks good.


Invalid typeof comparison value in core/xml.js

https://github.com/scratchfoundation/scratch-blocks/blob/caa389d4a44a1f643cfb3a23c9c5a7e8277697c5/core/xml.js#L645
The result of typeof should be a string, which means it won't equals to undefined. (Even typeof undefined is 'undefined')
ESLint will report
error  Invalid typeof comparison value                                            valid-typeof

MDN shows that if the attribute doesn't exist, getAttribute will return null.
Should it be modified to if (typeof type === 'undefined' || type === null) or just if (type === null)?

@alexcui03, this statement should be if (typeof type == 'undefined' || type == null) for clarification.


Silence indentation warning in script analysis

This commit silences the warning in Roblox Studio's "Script Analysis" Window
Error silenced:
W006: (285,3) Statement spans multiple lines; use indentation to silence
Script Analysis Warning:

Script Analysis Warning Fixed:

Checklist before submitting:

[x] Added entry to CHANGELOG.md
Added/updated relevant tests: Tests would not be possible as I do not believe you can interrogate/query the Script Analysis window. However I have attached an image of the script analysis window after the fix.
Added/updated documentation: Update would not have affected anything in the documentation. Minimal change.

Coverage remained the same at 94.779% when pulling afaa4a4519b42e09f20f20ae7776b039ff5cd454 on chasedig:master into b53b1db906296eb39c0b3b999b9d1175bb6c7cf9 on Roblox:master.

I have read the CLA Document and I hereby sign the CLA


Add tenant management function

When Docker initializes, as a platform administrator, adding users in the user management menu is different from tenants, so we need to distinguish them

Platform administrators only add tenants in their business, rather than directly adding users
Keep the original user management open to tenants, and the menu corresponding to the platform administrator can be hidden or retained, but it has no actual business function

CC #118 #124