API and component naming conventions

The following two items are to be covered by this issue:

API naming conventions
Component definitions:

kernel
OS
subsystem
middleware
...

This seems to be a pretty broad issue.
Is this things like:

under what conditions should a function be prefixed with k_ vs sys_ vs z_?
do prefixes form a namespace, e.g. everything related to work queues should start with k_work_?
do names follow an order based on grammatical function, e.g. verb-object so k_cancel_work rather than k_work_cancel?
how do we ensure that selected prefixes clearly identify their role and don't conflict with other systems or user code (e.g. rb_)?
how do we mediate clarity, uniqueness, and identifier length (e.g. is k_work_delayable_cancel_sync acceptable)?

Does it answer questions like:

What "the Zephyr kernel" in terms of scope?  Everything in the Zephyr tree?  Only code  in /kernel?
What is "the safety certification scope"?

Or is this a list of things for which documentation needs to be provided and the details will be handled at a lower level.
A clear written description of the problem and the criteria for an acceptable solution could make discussion more effective.

API meeting 27th October 2020:
Terms:

arch: Code living in the arch/ folder
kernel: Code living in the kernel/ folder
system: TBD
OS: TBD

Standard prefixes:

arch_: TBD
k_: Public kernel APIs
z_: Internal APIs. Not to be used by code outside the module, not exposed in include/
sys_:  TBD

To be continued next week.

the old naming conventions that might shed some light on the background:
https://github.com/zephyrproject-rtos/zephyr/wiki/Naming-Conventions

API meeting 19th January 2021:
Terms:
arch: Architecture-specific code. All arch code lives in the arch/ and include/arch folders
kernel: RTOS kernel (list functionality here) . All code that lives in kernel/, kernel/include/ and lib/kernel/ (will be populated with files from lib/os/)
Note: the following files need to be moved from lib/os to lib/kernel:
rb.c
sem.c
thread_entry.c
work_q.c
heap.c
heap-validate.c
mutex.c
p4wq.c
Core OS: Kernel and basic functionality included in Zephyr:

C library
The contents of lib/os except for the ones that belong in lib/kernel

Subsystems: Hardware independent software components

POSIX
CMSIS
All subsystems in subsys/, some of those need to be moved out to drivers (disk, Bluetooth controller)

Drivers: All driver implementations in drivers/ and driver APIs in include/drivers/
Standard prefixes:
arch: arch_
kernel: k_: Public kernel APIs,
sys_: TBD
z_: Internal APIs. Not to be used by code outside the module, not exposed in include/
To be continued next week.

API meeting 19th January 2021:
Terms:
arch: Architecture-specific code. All arch code lives in the arch/ and include/arch folders
kernel: RTOS kernel (list functionality here) . All code that lives in kernel/, kernel/include/ and lib/kernel/ (will be populated with files from lib/os/)
Note: the following files need to be moved from lib/os to lib/kernel:
rb.c
sem.c
thread_entry.c
work_q.c
heap.c
heap-validate.c
mutex.c
p4wq.c
Core OS: Kernel and basic functionality included in Zephyr:

C library
The contents of lib/os except for the ones that belong in lib/kernel

Subsystems: Hardware independent software components

POSIX
CMSIS
All subsystems in subsys/, some of those need to be moved out to drivers (disk, Bluetooth controller)

Drivers: All driver implementations in drivers/ and driver APIs in include/drivers/
Standard prefixes:
arch: arch_
kernel: k_: Public kernel APIs,
sys_: TBD
z_: Internal APIs. Not to be used by code outside the module, not exposed in include/
To be continued next week.

It would be a good idea to have a link to the naming conventions in the Zephyr documentation, and possibly somewhere before any examples are discussed. For me, after taking a look at the "synchronization" example, it took some googling to find this issue and the link to the naming conventions document.

Might be worth setting some general naming convention, or how to short names. It is quite annoying, when you switch between APIs, to remind yourself whether offset in function call is now spelled "offset", "off" or "offs".


This test method of a deep compare fail to me, am I missing the right configuration?

        [TestMethod]
        public void Compare()
        {
            //This is the comparison class
            var compareLogic = new CompareLogic();

            //Create a couple objects to compare
            var workspace = new Workspace
            {
                Pages = new[]
                {
                    new Page("page1"),
                    new Page("page2")
                }
            };

            //Create a couple objects to compare
            var workspace2 = new Workspace
            {
                Pages = new[]
                {
                    new Page("page1"),
                    new Page("page3")
                }
            };

            var result = compareLogic.Compare(workspace, workspace2);

            //These will be different, write out the differences
            Assert.IsFalse(result.AreEqual);

            Console.WriteLine(result.DifferencesString);
        }

where Page class is:
   public class Page
    {
        public Page(string name)
        {
            Name = name;
        }

        #region Overrides of Object

        public override string ToString()
        {
            var result = Name;
            if (!string.IsNullOrEmpty(Status))
                result += $" ({Status})";
            return result;
        }

        #region Overrides of Object

        public override bool Equals(object obj)
        {
            // Check for null values and compare run-time types.
            if (!(obj is Page))
                return false;

            var other = (Page)obj;
            return string.Equals(Name, other.Name, StringComparison.InvariantCultureIgnoreCase);
        }

        public override int GetHashCode()
        {
            int result = 0;
            if (Name != null)
                result = Name.ToLower().GetHashCode();
            return result;
        }

        #endregion

        #endregion

        #region Properties

        public string Name { get; set; }

        #endregion
    }

@djpable It will take me a while until I can get to this.  I am recovering from an injury.

You might want to debug with the source or post a question on stack overflow.

@GregFinzer I wish you a speedy recovery!
I will try to debug with sources :)

I have seen that the problem is in the Page "Overrides of Object", in particular if i comment GetHashCode() override the library works..
That's weired because i need that overrides to use Page objects as keys in dictionaries.

@djpable it uses the hash code to properly identify circular references.  If you override GetHashCode it just has to be unique for each object.


How to select multiple options (in multi-select field)?

@Aussiroth
Hi Alvin,
What is the syntax for selecting multiple options?
Referring to this example:
https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_select_multiple
How do I write TagUI command to select both volvo and audi?
Thanks and regards,
Yang Liu

Hi,
TagUI isn't natively capable of supporting a simple command to perform multiple selects. This is actually the first time I've come across an issue of trying to select multiple options with TagUI. I've tried updating the selection code to select multiple options but it only successfully selects the last one. I'll try to see if there's a way to support this automatically, but it will probably need quite some changes to the code.
Hopefully Ken might have experience with such selections.

That's right, a typical selection dropdown TagUI uses JavaScript code to set the matched option and then trigger an UIEvent. When you do this for the 2nd time, the behaviour happens as you expect when you manually click on the 2nd option - the 1st selected option becomes unselected.
I can imagine 2 ways to address this. First may be you can try writing JavaScript code to do the selection that you want. And use the js step (js begin and js finish) to run your custom JavaScript code.
The second way is use the computer vision method of TagUI to visually click on what you want. With the Sikuli's visual automation and TagUI vision step, you can send custom commands such as Control function key. You can give a try with more details here - https://github.com/kelaberetiv/TagUI/issues/155#issuecomment-397403024

I was able to solve this by putting similar code to what's described here https://stackoverflow.com/questions/1295950/javascript-to-select-multiple-options inside a dom step (dom begin and dom finish).

Thanks @bcpeinhardt for sharing back your solution here! :)
Also copying @ruthtxh


feat: pull secret

What this PR does / why we need it:
Allows the user to set a single global imagePullSecret, for retrieving eraser component images (eraser, manager, scanner, collector) from private registries.
To use:

Create a docker-registry secret (ex mysecret) in the same namespace in which eraser will be deployed
Deploy with helm, by using the flag --set eraserPullSecretName=mysecret

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #492
Special notes for your reviewer:
Have fun and happy reviewing

Codecov Report

Merging #493 (0a02d18) into main (75a5454) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #493   +/-   ##
=======================================
  Coverage   13.02%   13.02%
=======================================
  Files          12       12
  Lines        1328     1328
=======================================
  Hits          173      173
  Misses       1137     1137
  Partials       18       18

Flag
Coverage Δ

unittests
13.02% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

test/e2e/util/utils.go
7.25% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add watchman.json

Missing Hash, license, description, ...
This should be more complex using checkver as github facebook/watchmen and some URL sticking using $matchVars to get proper version URL.
or
make it nightly.


FEATURE: Form组件提供手动控制表单项显隐的api

Version
Vben Admin V5
Description
目前表单项显隐的方式只能通过表单项其他字段来控制，无法提供调用api的方式手动控制显隐
Proposed Solution
期望提供api来手动控制表单项显隐
Alternatives Considered
No response
Additional Context
No response
Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.

联动控制：https://doc.vben.pro/components/common-ui/vben-form#表单联动
直接控制： formApi.updateSchema([{fieldName:'xxx', formItemClass: 'hidden' }])


Update footer.html

Add "/" before the fittext.js file.

Closing. Documentation presumes that baseUrls end with a "/", making this change redundant.


Cirq Circuit requires device parameter in order to use XmonSimulator

I am creating a simple circuit as follows:

var circ = new QuantumCircuit();

circ.addGate("h", -1, 0);
circ.addGate("cx", -1, [0, 1]);

circ.run();

console.log("");
console.log(circ.exportCirq());

The exported program generated by quantum-circuit.js is this:
import cirq
import numpy as np

q = [cirq.NamedQubit('q' + str(i)) for i in range(2)]

circuit = cirq.Circuit.from_ops(
    cirq.H(q[0]),
    cirq.CNOT(q[0], q[1])
)

simulator = cirq.google.XmonSimulator()
result = simulator.run(circuit)
print(result)

However, when I run the exported program in Cirq, it gives me the following error:
  File ".\src\pragma_test.py", line 12, in <module>
    result = simulator.run(circuit)
  File "C:\Users\Kaustuvi\Anaconda3\lib\site-packages\cirq\sim\sampler.py", line 42, in run
    repetitions)[0]
  File "C:\Users\Kaustuvi\Anaconda3\lib\site-packages\cirq\sim\simulator.py", line 73, in run_sweep
    repetitions=repetitions)
  File "C:\Users\Kaustuvi\Anaconda3\lib\site-packages\cirq\google\sim\xmon_simulator.py", line 153, in _run
    _verify_xmon_circuit(circuit)
  File "C:\Users\Kaustuvi\Anaconda3\lib\site-packages\cirq\google\sim\xmon_simulator.py", line 97, in _verify_xmon_circuit
    raise ValueError("XmonSimulator only accepts circuits "
ValueError: XmonSimulator only accepts circuits using an XmonDevice. Maybe you forgot to pass a `device=` parameter into `cirq.Circuit`, or intended to set `new_device=` in your call to `cirq.google.optimized_for_xmon`? Alternatively, maybe you intended to use `cirq.Simulator()` instead?

Basically, Circuit requires the device parameter in order to use XmonSimulator. Perhaps, we could use cirq.Simulator() instead of XmonSimulator? Also, if we are using simulator.run(), the user needs to explicitly add measurement operations, otherwise the simulator can return empty measurements.
Please let me know if I am missing anything.

Hi @Kaustuvi,
Thanks for reporting. Export to cirq is not tested well, but it worked in time when it was implemented. Possibly cirq was changed in meantime (and you are first to report).
What version of cirq framework you are using?
I will test and try fix (if it is a bug) later when I reach my computer.

I am using cirq 0.5.0.

XmonSimulator is replaced with Simulator for now. (Will be added as an option somewhere in the near future).
So this issue is fixed. Closing.


Publish RTCStatsReports to Fishjam

Description
Describe your changes in detail.
Motivation and Context
Why is this change required? What problem does it solve? If it fixes an open
issue, please link to the issue here.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)

Two questions remain:

Should RTCStatsReport be sent using MediaEvents, or should it use a new notification like RTCStatsReport?
Should inactive outbound tracks be sent?


Quick new project command

Would it be possible to add a quick start option like ng new PROJECT_NAME --quick and for it to install the bare minimum files as in the first "Get Started" tutorial on angular.io
angular2-quickstart
app
|  --app.component.ts
|  --app.module.ts
|  --main.ts
node_modules ...
typings ...
index.html
package.json
styles.css
systemjs.config.js
tsconfig.json
typings.json
I like building apps from scratch and I find all the other files unnecessary.

Maybe --minimal? I like the idea of a quick setup. My only question is how do you communicate that the minimal setup doesn't follow style recommendations?

We already install the minimal number of files for a CLI generated project to work. It is true this minimum is fairly big, but that is being tracked in https://github.com/angular/angular-cli/issues/1263.
angular.io uses https://github.com/angular/quickstart as the base for all it's examples, so perhaps that is what you are actually looking for.


feat: Dispatch throttling metrics

Description
Adding two new metrics for tracking the quantity of requests that have been throttled via dispatch throttling:

throttled_requests_success_count - requests that have been throttled but succeed
throttled_requests_timed_out_count - requests that have been throttled but time-out. Only occurs for Check.

The sum of these two metrics is the overall number of requests throttled. These metrics are useful for understanding the effects of your dispatch throttling configurations.
Note: The throttled_requests_timed_out_count metric is currently only useful for Check because ListUsers and ListObjects will optimistically return a partial result if the request times-out; we cannot discern a success or failure.
This PR was tested by running docker compose up and observing prometheus locally:

References

#1440
#1571
#1658

Review Checklist

[x] I have clicked on "allow edits by maintainers".
[x] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[x] I have added tests to validate that the change in functionality is working as expected

Should we have counter for how many requests was throttled
@adriantam Total number of throttled requests is the sum of the two metrics.

how many are throttled and timed out (vs. how many are throttled successfully + how many throttled timeout)?
Possibly. As an operator I think I would be more concerned about successes/failures rather than time-outs which LU and LO are susceptible to anyway. But there could also be a world where you want to know about time-outs too.

I think once we indicate in LU and LO wether a response is complete or not (https://github.com/orgs/openfga/discussions/110 ) then we can make better determinations around wether a throttle was "successful or not". Especially if we give the operator the ability to prevent partial responses.

@adriantam After additional thought, I'm changing the approach to align with your suggestion. Instead of splitting the metrics on success/failure, we're now splitting on time-out/no-time-out.

Update: I have decided to vastly simplify this PR by only adding a single throttledRequestCounter metric at this time. The reason is that detecting timeout in Check and ListUsers is straightforward but is difficult in ListObjects because there are several concurrency mechanisms working together in ways that aren't immediately clear or straightforward to test. The primary culprit of this complexity is needing to detect timeouts from both reverse expand and the check resolver. Both of these mechanisms feed into different concurrency controls so there is no single place to implement timeout detection.
We will need this timeout detection in the future so this refactor of ListObjects is necessary but not in the scope of this PR. Further, its not clear how operators will use these metrics so IMO, better to be conservative and only have the single metric for now. We can always expand at a future point in time.


fix(ui-library): align counter to right for text area with or without error

closes #1064

I just had a quick look and found one issue. If you add an error message and error message icon, they are right aligned, but they should be left aligned.
Here is how it should look according to design:

Hi @thrbnhrtmnn
Just doubled checked, I see that it is left aligned.

I just had a quick look and found one issue. If you add an error message and error message icon, they are right aligned, but they should be left aligned.
Here is how it should look according to design:

Hi @thrbnhrtmnn Just doubled checked, I see that it is left aligned.

Hey @ashk1996 , in your example hasHint = true. In this case it looks good. But if hasHint = false and only hasError = true and errorMessage is not empty, then the error message is not left aligned but right aligned, as seen in my screenshot above.
While looking at it again, if errorMessageIcon is not undefined and errorMessage is empty, the icon should also be displayed right aligned, but currently it is not displayed at all in this case. It will only appear once errorMessage is not empty anymore.
If you want we can have a quick huddle about this :-)

I just had a quick look and found one issue. If you add an error message and error message icon, they are right aligned, but they should be left aligned.
Here is how it should look according to design:

Hi @thrbnhrtmnn Just doubled checked, I see that it is left aligned.

Hey @ashk1996 , in your example hasHint = true. In this case it looks good. But if hasHint = false and only hasError = true and errorMessage is not empty, then the error message is not left aligned but right aligned, as seen in my screenshot above.
While looking at it again, if errorMessageIcon is not undefined and errorMessage is empty, the icon should also be displayed right aligned, but currently it is not displayed at all in this case. It will only appear once errorMessage is not empty anymore.
If you want we can have a quick huddle about this :-)

Thanks for pointing out. I have fixed this in the latest commit.

Hey @ashk1996 , as discussed today the icon not being displayed if the corresponding message is empty is not an issue, because this would just be a user mistake. There should not be an icon when there is no message. Neither for errors nor for hints.
After looking again the only point I found is that in case the hasHint = true, but message and icon are empty / undefined, the counter will be left aligned. As just discussed we should fix this, as long it is not a huge effort. Again this would be an user mistake, but if we can prevent it easily we should always do so.


Appstudio update test-component-pac-aviw

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-aviw-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
24 seconds
build-container

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
10 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
inspect-image

✅ Succeeded
43 seconds
clamav-scan

✅ Succeeded
19 seconds
sbom-json-check

✅ Succeeded
18 seconds
label-check

✅ Succeeded
12 seconds
show-summary

✅ Succeeded
9 seconds
show-sbom

Pipelines as Code CI/test-component-pac-aviw-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
25 seconds
build-container

✅ Succeeded
10 seconds
inspect-image

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
33 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
17 seconds
label-check

✅ Succeeded
10 seconds
show-summary

✅ Succeeded
8 seconds
show-sbom


Safety checker triggered

Hello! I'm getting an error when trying any prompt on any model, even if I change the seed value.
Any idea on how to fix it?
Thank you!

Thank you for the details @Luke-qpla! Unfortunately, I think that GPU does not have enough memory to run Stable Diffusion. We'll provide a better error message in these situations.


0.3.10 version not published to npm

Please publish it

@revolunet ping

+1 from me

There's 0.3.12 on npmjs.org, so I think this thread can be closed.

thx


Support scientific notation floats

Support for float64 numbers encoded like 10.1e-01 (see unit tests)

:+1:


C# FileDescriptor does not contain a definition for FromGeneratedCode, and others.

So I'm trying this out in a project, my packages look like:
Edit2: Also I'm using Visual Studio 2015.
<?xml version="1.0" encoding="utf-8"?>
<packages>
  <package id="Grpc" version="0.14.0" targetFramework="net461" />
  <package id="Grpc.Core" version="0.14.0" targetFramework="net461" />
  <package id="Grpc.Tools" version="0.14.0" targetFramework="net461" />
  <package id="Ix-Async" version="1.2.5" targetFramework="net461" />
  <package id="protobuf40" version="3.0.0" targetFramework="net461" />
</packages>

edit1: Also if it helps, the references being pulled in by my project:
  <ItemGroup>
    <Reference Include="Google.Protobuf, Version=<IP_ADDRESS>, Culture=neutral, processorArchitecture=MSIL">
      <HintPath>..\packages\protobuf<IP_ADDRESS>\lib\net40\Google.Protobuf.dll</HintPath>
      <Private>True</Private>
    </Reference>
    <Reference Include="Grpc.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=d754f35622e28bad, processorArchitecture=MSIL">
      <HintPath>..\packages\Grpc.Core.0.14.0\lib\net45\Grpc.Core.dll</HintPath>
      <Private>True</Private>
    </Reference>
    <Reference Include="System" />
    <Reference Include="System.Core" />
    <Reference Include="System.Interactive.Async, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL">
      <HintPath>..\packages\Ix-Async.1.2.5\lib\net45\System.Interactive.Async.dll</HintPath>
      <Private>True</Private>
    </Reference>
    <Reference Include="System.Xml.Linq" />
    <Reference Include="System.Data.DataSetExtensions" />
    <Reference Include="Microsoft.CSharp" />
    <Reference Include="System.Data" />
    <Reference Include="System.Net.Http" />
    <Reference Include="System.Xml" />
  </ItemGroup>

And it's giving me:
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0117	'JsonFormatter' does not contain a definition for 'ToDiagnosticString'	ServerManagerProto	Main.cs	566	Active
Error	CS0117	'FileDescriptor' does not contain a definition for 'FromGeneratedCode'	ServerManagerProto    Main.cs	40	Active
Error	CS1729	'GeneratedCodeInfo' does not contain a constructor that takes 6 arguments	ServerManagerProto    Main.cs	43	Active
Error	CS1729	'GeneratedCodeInfo' does not contain a constructor that takes 6 arguments	ServerManagerProto	Main.cs	44	Active
Error	CS1729	'GeneratedCodeInfo' does not contain a constructor that takes 6 arguments	ServerManagerProto	Main.cs	45	Active
Error	CS1729	'GeneratedCodeInfo' does not contain a constructor that takes 6 arguments	ServerManagerProto	Main.cs	46	Active
Error	CS0117	'JsonFormatter' does not contain a definition for 'ToDiagnosticString'	ServerManagerProto	Main.cs	99	Active
Error	CS0117	'JsonFormatter' does not contain a definition for 'ToDiagnosticString'	ServerManagerProto	Main.cs	214	Active
Error	CS0117	'JsonFormatter' does not contain a definition for 'ToDiagnosticString'	ServerManagerProto	Main.cs	358	Active

Not really sure what I'm missing here. Also, I'm building against .net 4.6.1. I've looked at the API and it looks like this should be pulling in the right assemblies for it.
Any help/info would be great, thanks!
Edit3: Also the C# example "Helloworld" does not work either, it can't find the global namespace Google. Which I had an issue with earlier, until I installed "protobuf40" on the nuget packages (Which is all that came up for "Google.Protobuf")
Edit4: Looks like the helloworld does work, so it's something on my end. I'll have to look later.

Well I figured it out, I needed to check "Include Prereleases" on NuGet to get the right Google.Protobuf.
Sorry.


Unable to load RELATIVE $ref property

We defined a swagger file ( src/main/resources/BS/ApiSpecification.swagger ) that references a property defined in a JSON file (src/main/resources/BO/Resource/ApiSpecificationBO.json).
In ApiSpecificationBO.json file there is a relative $ref property defined in another JSON (src/main/resources/BO/Common/BasicComponents.json).
The class generating fails with the fallowing error from swagger-parser:
Unable to load RELATIVE ref: ../Common/BasicComponents.json path: C:\utils-app\src\main\resources\BS
As we can see, it tries to resolve the reference to the second JSON file considering as a parent folder the one where the initial swagger file is placed, and not the folder where the JSON file containing the $ref is placed.
We used version v1.0.39.
Resources:

src/main/resources/BS/ApiSpecification.swagger:
[...]
"responses": {
                    "200": {
                        "schema": {
                            "type": "array",
                            "items": {
                                "$ref": "#/definitions/accessProfileBO"
                            }
                        }
                    }
                }
[...]
"definitions": {
        "accessProfileBO": {
            "$ref": "../BO/Resource/ApiSpecificationBO.json"
        }
    },
[...]

src/main/resources/BO/Resource/ApiSpecificationBO.json
{
    "properties": {
        "details": {
            "type": "object",
            "properties": {
                "confirmationCodeRequired": {
                    "$ref": "../Common/BasicComponents.json#/properties/indicatorType"
                }
            }
        }
    }
}

src/main/resources/BO/Common/BasicComponents.json
{
    "properties": {
        "indicatorType": {
            "type": "object",
            "properties": {
                "indicator": {
                    "type": "boolean"
                }
            }
        }
    }
}

src.zip

Please approve the fix for this issue:
https://github.com/swagger-api/swagger-parser/pull/917

Fixed by #917  please let us know if this keeps happening. Thanks!


Refactored split learning for training ControlNet with the latest API. And added reference for PerFedRLNAS.

Refactored the code for split learning with ControlNet. Removed unnecessary classes and functions. And rewrote some functions with the latest split learning interfaces under plato/
Description

To train ControlNet with split learning, we need customized datasource and models. The customized datasource is implemented in controlnet_datasource.py. Customized model classes are implemented in ControlNetSplitLearning.

How has this been tested?

To run the example of usual split learning
python examples/split_learning/controlnet_split_learning/split_learning.py -c examples/split_learning/controlnet_split_learning/split_learning_fill50k.yml

To run the example of privacy-preserving split learning
python examples/split_learning/controlnet_split_learning/split_learning.py -c examples/split_learning/controlnet_split_learning/split_learning_fill50k_pp.yml

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue) Fixes #
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code has been formatted using Black and checked using PyLint.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

Hi Dixi,
I found many packages were missing in my current plato environment when I tried to run your example program, such as wget and many packages specified in examples/split_learning/controlnet_split_learning/ControlNet/environment.yaml. Maybe it helps to update plato's requirement.txt accordingly or include another requirement.txt under the controlnet_split_learning folder to make it more convenient to install the extra pip packages needed for this project.

Hi Fei,
Thank you very much for the suggestion. I have added a requirements.txt file.


Fix custom EVP_PKEY_METHODs

Fixes #16088

I've now completed this PR and taken it out of draft.
Please take a look

Great work!

All feedback addressed. I also dropped the constification commit and moved it into #16128 so as not to hold up this one. Please reconfirm.

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 to master. Thank you.

I was going to allow 24 hours from the last update....but ok!

I was going to allow 24 hours from the last update....but ok!

Ah, right. Given these were just minor adjustments hopefully it was ok.
I now see a buildbot failure on solaris after merging this though :disappointed:


Harmony Hub commands will repeat

Hello,
After the update from 5.2.1 to 5.2.2RC3 Harmony Hub commands are no longer executed properly. The Harmony Hub button 22 will run as 2 then 22 and then 2. The Target Item in Ha-Bridge is [{"device": "49687309", "button": "2"}, {"device": "49687309", "button": "2"}]
With 5.2.1 the Target Item works.
Regards,
Andi

So, that is probably an old syntax. I would delete that item and use the Harmony Device Tab to build a button to send 2 and then use the count parameter to send it twice.

Upgrading to version v5.2.2RC4 was the solution. Now it works again.


fix SPIFFS_read length when reading beyond file size

when reading beyond file size, and when spiffs_object_read return SPIFFS_OK,
the length add to offset should be "avail" instead of len.
please review.

Nice! Thanks for finding it.
Cheers!


Home-page specific changes

This is a meta issue for the rework of the homepage specifically
Related to https://github.com/rrousselGit/riverpod/issues/2609
Some good homepage examples to take inspiration from:

https://www.framer.com/
https://tanstack.com/query/v3/
https://trpc.io/
https://stately.ai/

Things the homepage could do:

List the list of sponsors
List companies using Riverpod
List tweets of various user feedback
Offer a few animated visual demonstrations of Riverpod (whatever that is)
Maybe add a Dartpad or Zapp embed to have a Hello World app in there
List key Riverpod features

Automatic caching
Easy pull-to-refresh
Default loading/error handling
Compile-time safety for various common mistakes
Easy fetch-as-you-scroll
Inspired by widgets, but for state
Highly composable
Testable
Offers custom lint rules/refactors
(experimental) Offers project visualizer

Design ideas:

Maybe the homepage could be a waterfall/river
Features could be various obstacles, like rocks/branches
if we have branches, maybe Dash could be standing on one of them

Have you found a designer, yet? Do you need help with that?

Some folks offered to help here and there. Although at the moment I'm not sure if they are still on it.
If you have someone that'd be cool

I spend some time writing a flutter app the helped visualize some aspects of riverpod and seeing how some providers interact. It does not leverage all features but it could be helpful for new users to grasp some concepts. Maybe embedding something like this in the docs with interactions would be helpful?

I'm open to ideas there. Something interactive (or animated) could certainly be helpful
I'm not sure such a table would help though. It looks more like a devtool than an interactive documentation example.
And whatever devtool I'll add for Riverpod are unlikely to be web-compatible.


Use gpt-4o instead of using gpt-4 turbo

The system by default is usng gpt4-turbo, is it possible to use gpt-4o instead, which is suppose to be better and less expensive?
thanks

Hi @aicoder2048 I made this update to version="1.4.6" yesterday. Try it with the new version and let me know if you have any input.


[BUG] Launching the utility without firefox installed before on the host

Describe the bug
The utility crashes when launching it in a brand-new python venv. (No Firefox installed on the host)
To Reproduce
Steps to reproduce the behavior:

Create a venv on the host : python3.8 -m venv my-venv
Activate it : source my-venv/bin/activate
Install the module : pip install awscli-saml-sso
Launch the utility : awscli_saml_sso

Expected behavior
To get to the webpage on which to set our accenture credentials.
Desktop (please complete the following information):

OS: MacOS Catalina
Python 3.8.1

Additional context
awscli_saml_sso
Please configure your identity provider url [https://<fqdn>:<port>/adfs/ls/IdpInitiatedSignOn.aspx?loginToRp=urn:amazon:webservices]:
https://federation-sts.accenture.com/adfs/ls/idpinitiatedsignon.aspx?loginToRp=urn:amazon:webservices
[WDM] - There is no [macos] geckodriver for browser  in cache
INFO:WDM:There is no [macos] geckodriver for browser  in cache
[WDM] - Getting latest mozilla release info for v0.28.0
INFO:WDM:Getting latest mozilla release info for v0.28.0
[WDM] - Trying to download new driver from https://github.com/mozilla/geckodriver/releases/download/v0.28.0/geckodriver-v0.28.0-macos.tar.gz
INFO:WDM:Trying to download new driver from https://github.com/mozilla/geckodriver/releases/download/v0.28.0/geckodriver-v0.28.0-macos.tar.gz
[WDM] - Driver has been saved in cache [/Users/baptiste.ojeanson/.wdm/drivers/geckodriver/macos/v0.28.0]
INFO:WDM:Driver has been saved in cache [/Users/baptiste.ojeanson/.wdm/drivers/geckodriver/macos/v0.28.0]
Waiting for AWS SAML homepage...Traceback (most recent call last):
  File "/Users/baptiste.ojeanson/Desktop/benjam-outils/bin/awscli_saml_sso", line 10, in <module>
    sys.exit(main())
  File "/Users/baptiste.ojeanson/Desktop/benjam-outils/lib/python3.8/site-packages/awscli_saml_sso/__main__.py", line 30, in main
    browser = webdriver.Firefox(executable_path=GeckoDriverManager().install(), service_log_path=devnull)
  File "/Users/baptiste.ojeanson/Desktop/benjam-outils/lib/python3.8/site-packages/selenium/webdriver/firefox/webdriver.py", line 170, in __init__
    RemoteWebDriver.__init__(
  File "/Users/baptiste.ojeanson/Desktop/benjam-outils/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 157, in __init__
    self.start_session(capabilities, browser_profile)
  File "/Users/baptiste.ojeanson/Desktop/benjam-outils/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 252, in start_session
    response = self.execute(Command.NEW_SESSION, parameters)
  File "/Users/baptiste.ojeanson/Desktop/benjam-outils/lib/python3.8/site-packages/selenium/webdriver/remote/webdriver.py", line 321, in execute
    self.error_handler.check_response(response)
  File "/Users/baptiste.ojeanson/Desktop/benjam-outils/lib/python3.8/site-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.SessionNotCreatedException: Message: Expected browser binary location, but unable to find binary in default location, no 'moz:firefoxOptions.binary' capability provided, and no binary flag set on the command line


User grouping

Pending specification
Organizations might need some kind of hierarchical user management.
@saleiva @javisantana @andrewxhill @xavijam

Wouldn't focus on this at this time. Too much I think.

It's marked as enhancement and not assigned to a milestone because of that.

@saleiva @javisantana this is something more and more Gov clients are asking for. In Australia having groups inside an organization was a hard requirement for at least 3 clients ($50-$200K).

jumping on thread to follow this issue

@ohasselblad: this is specified at Groups wiki page, where you can track the progress.


Whether different archs are supported for student and teacher?

When building models for teacher and student in this code, the parameter args.arch is used for both student and teacher.
As written in the paper, smaller models are distilled from the largest model (a frozen teacher). How to achieve this through the above code?

Thanks for the reply ! I am also interested in this.
So no plans to add this missing code ?

Thanks! @qasfb I'm wondering do we need the Dino loss except the masked-image-modeling loss during the distillation?

Hi @qasfb

* apply the masked-image-modeling loss to all the tokens (not only the subset that is masked)

Could you explain why the MiM loss needs to be applies to all the tokens in this case?


[1.3] The implementation of the Electrical Energy Measurement Cluster is required in chip-tool and sample app

Feature description
Chip-tool and All-clusters-app has to be enabled with below mentioned attributes :

[ ] Measured
[ ] CumulativeEnergyImportedTime
[ ] CumulativeEnergyExportedTime
[ ] PeriodicEnergyImportedStartTime
[ ] PeriodicEnergyImportedEndTime
[ ] PeriodicEnergyExportedStartTime
[ ] PeriodicEnergyExportedEndTime
[ ] EphemeralEnergyImported
[ ] EphemeralEnergyExported

Chip-tool and All-clusters-app has to be enabled with below mentioned events :

[ ] CumulativeEnergyImported
[ ] CumulativeEnergyExported
[ ] PeriodicEnergyImported
[ ] PeriodicEnergyExported
[ ] EphemeralEnergyImported
[ ] EphemeralEnergyExported

Platform
raspi
Platform Version(s)
RPI - 4, 8GB RAM
Anything else?
No response

The XML and Cluster was implemented in PR #30389 and is in all-clusters-app
There is a new PR #31622 which enables it with fake test data in the energy-management-app


Git blame for whole file

Is your feature request related to a problem? Please describe.
Existing feature to show blame for only single line is slightly confused. Sometimes it's more informative to see whole blame for file.
Describe the solution you'd like
It would be nice to have ability to open service buffer on the left side to show the blame for all lines. Or whatever any better solution if you have any ideas.
Describe alternatives you've considered
Currently I'm using fugitive blame for this proposal and it is only one feature which I use in fugitive. It is possible to blame file from command line as well. But exception of highlighting is slightly disappointed and to find necessary lines is annoying in big file.
Additional context
Add any other context or screenshots about the feature request here.

What is the problem with using vim-fugitive? vim-fugitive's blame feature is likely better than anything I can implement and I use it all the time.

@lewis6991 It's slightly strange for me to keep plugin only for blame feature...

Sorry, that's not a strong enough justification. It is already an explicit non-goal to re-implement every feature of vim-fugitive unless there is a good reason.

Your git annotations are a lot nicer then fugitive, this seems like it wouldn't be too difficult to implement. Would you accept a PR to add this feature?

There's already :Gitsigns blame

There's already :Gitsigns blame

Tried it out, exactly what I was looking for thank you


Update WSA

Can I update this version of WSA and keep my data?

I successfully updated WSA just by repeating the install steps, and my data stays there.
Pay attention: do not uninstall WSA or your data will be lost.
When reinstalling, just copy and overwrite all the files extracted from the .msix to your last installation directory.
The data doesn't store there, so just overwrite them.
After overwriting, try to follow the following steps to edit product, system, system_ext, and vendor images again.

Download "GApps" via OpenGApps Project
Clone this repository and populate the directories
Final preparations
Running the scripts
Follow the Copy the edited images step and overwrite the images.

Registering the edited Windows Subsystem for Android™ Installation Package and Gaining Root Access are not needed, and there is no need to login into google account again. Because the WSA has been installed and your data will not be deleted during the reinstall steps.

@L245T Just to be clear, you repeated all install steps in the Readme starting from https://github.com/WSA-Community/WSAGAScript#download? I'm guessing you "update" WSA just by downloading the newer version from store.rg-adguard.net?

@L245T Just to be clear, you repeated all install steps in the Readme starting from https://github.com/WSA-Community/WSAGAScript#download? I'm guessing you "update" WSA just by downloading the newer version from store.rg-adguard.net?

Yes

You still have to do the register step as that's what populates the information like version information, location, and other things into the system. It will technically work without reregistering since you are replacing the files, but it can cause issues and the system will still think its the old version


[WIP] Fix the decoder issue when joins are present in the query

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/3271/

LGTM


Declaration file for the plugin + small util improvements

Due to the absence of a DefinitelyTyped package for the plugin, added an interim declaration file for the public API.
Also made a small improvement to source code by moving the arrayfying logic from inline to a dedicated utility.
Hope you'll find the PR helpful,

Mocha bump: seems like the bump breaks Travis CI flow for older Node versions - reverted

Nice thank you for the contribution 🎉

🎉 Btw, do you mind a later PR moving the codebase to TypeScript and adding some more updates? The plugin came in very handy for a lightweight build workflow with Google Apps Script Web Apps, which horrifyingly accept only .html files for the frontend. Too bad Mocha had to be bumped back to comply with the pipeline - can also take a look at what can be done.


Allow multi_field to index/analyze the main field

Is there a way to index the main field when using type: :multi_field? Right now chewy sets the first field definition to index: no.
The following is a valid ES mapping taken from their docs, but I can't seem to recreate it with chewy.
"tweet": {
    "type":     "string",
    "analyzer": "english",
    "fields": {
        "raw": {
            "type":  "string",
            "index": "not_analyzed"
        }
    }
}

If it's not possible currently, would you accept a patch that changes to use a multi_field: true option instead, and allow type to be used again and index the main field? The user could still control whether a field is indexed or not by passing index: false.

field : tweet, analyzer: 'english', value: -> { tweet.try(:strip) } do
  field : raw, index: 'not_analyzed'
end

This should work actually

The main thing here is to specify type for the multi_field and this type should not be object

You are correct, your syntax does work. It wasn't working for me because I hadn't explicitly declared a type. I was just letting chewy implicitly determine it. Once I set type: 'string' it worked perfectly. Maybe the docs could make mention of that to help people in the future?

Sure, it should be mentioned in readme


Match calculation of U with the Python implementation

As identified by @HannoSpreeuw:

Matlab has

U=(presum + rhorat*Phi^3*(1-exp(10-10/Phi))/(1-Phi))

while Python has

F = 1 - np.exp(10 - 10 / Phi)
one_minus_Phi = 1 - Phi
U = self.presum + self.rhorat * Phi ** 3 * F /one_minus_Ph

So those two extra brackets in Matlab make a difference: presum is also divided by 1-Phi in Matlab.

Solved by #11


PageView.builder problem after upgrade to Flutter 2.5

Hi. After upgrade to Flutter 2.5, PageView do not work correctly.
Code has not changed between 2.2.3 and 2.5, and when I executed on 2.5 PageView not swipe pages.
When I downgrade to 2.2.3 swipe correctly again.
I implement PageView with this code:
Expanded(                           child: PageView.builder(                             controller: _pageController,                             onPageChanged: (index) {                               setState(() {});                             },                             itemCount: 10,                             itemBuilder: (BuildContext context, int index) {                               return Container();                             },                           ),                         )
This is my system actually with 2.2.3 worked version:
flutter doctor:
`[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-arm, locale
it-IT)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[!] Android Studio (version 2020.3)
✗ Unable to find bundled Java version.
[✓] VS Code (version 1.60.1)
[✓] Connected device (2 available)
! Error: iPhone di Francesco is not connected. Xcode will continue when
iPhone di Francesco is connected. (code -13)
! Doctor found issues in 1 category.`
Error reported is not important for actual problem.
Bye.
Regards.

@marco-1988
Please see this breaking change document related to default scroll and swipe behavior in the framework. This change may not be there in stable yet, so please try to switch to latest master and check if it works.

@marco-1988
Please see this breaking change document related to default scroll and swipe behavior in the framework. This change may not be there in stable yet, so please try to switch to latest master and check if it works.

Hello, I try it, but it cannot work for PageView


The installation script for k3s is incompatible with an IPv6-only environment.

Reason behind this issue is github have no support on ipv6 until yet.
So when ever the installation script download checksum file or binary using github.com url, the script will crash.
Ps. Tried to install k3s with HTTPS_PROXY, it works but also been logged into k3s's systemd env file.

Duplicate of https://github.com/k3s-io/k3s/issues/5921
We're not tracking this here as there is nothing we can do about it.


Wrong font on eastern Windows versions

In his pull request @yang827 posted an image of PML Unit in Hull & Outfitting 12.1. Note that almost everything except for the summary and last test run labels uses a fixed width font. Nothing in the code base except for those two labels explicitly sets a font, so all controls should inherit the app's default font.
The other docked windows (Design Explorer and Command Window) also use that fixed width font for their title instead of the normal sans-serif font like the rest of the application.
Is this an issue with Hull & Outfitting or is the Chinese Windows version using a different default font? Should PML Unit explicitly set its default font during start up? (There are some stackoverflow posts about default fonts in Windows Forms applications.)
Besides the wrong font being selected, the labels above the test list should still be placed next to each other and not overlap like that.

This screenshot is taken from my laptop. I have a laptop with a 15 inch screen and 1920x1080 Resolution. The recommended scaling is 150% by default. There are many display errors in the Marine system. It will be fine when I set it to 100%.

Hm.... I can't reproduce this at all. I tried setting the scaling to 150% on Windows 10 and Windows 7 but neither PDMS nor E3D experience these font issues that you have.
I created a font-issue branch with a "Debug Fonts..." label in the TestRunnerControl. On my machines, this outputs

TestRunnerControl.Font = [Font: Name=Microsoft Sans Serif, Size=8,25, Units=3, GdiCharSet=0, GdiVerticalFont=False]
TestRunnerControl.Parent....Font = [Font: Name=Microsoft Sans Serif, Size=8,25, Units=3, GdiCharSet=0, GdiVerticalFont=False]
Control.DefaultFont = [Font: Name=Microsoft Sans Serif, Size=8,25, Units=3, GdiCharSet=0, GdiVerticalFont=False]
SystemFonts.MessageBoxFont = [Font: Name=Segoe UI, Size=9, Units=3, GdiCharSet=1, GdiVerticalFont=False]

with 100% scaling and

TestRunnerControl.Font = [Font: Name=Microsoft Sans Serif, Size=8, Units=3, GdiCharSet=0, GdiVerticalFont=False]
TestRunnerControl.Parent....Font = [Font: Name=Microsoft Sans Serif, Size=8, Units=3, GdiCharSet=0, GdiVerticalFont=False]
Control.DefaultFont = [Font: Name=Microsoft Sans Serif, Size=8, Units=3, GdiCharSet=0, GdiVerticalFont=False]
SystemFonts.MessageBoxFont = [Font: Name=Segoe UI, Size=9, Units=3, GdiCharSet=1, GdiVerticalFont=False]

with 150% scaling.
Could you check out that branch and tell me the debug output on your machine, please?

You're right. This problem is caused by the default font of the Chinese operating system. Unlike the western operating system, the dpi change does not affect the font size.
with 150% scaling.

with 100% scaling

fixed in 747ecd04be9e553a0deaf3cd53c7c9251304f4c2


New command: spo group add

Usage
m365 spo group add
Description
Creates group in the specified site
Options

Option
Description

-u, --webUrl <webUrl>
URL of the site where the group should be added.

-t, --title <title>
Title of the group to add.

--description [description]
The description for the group.

--allowMembersEditMembership
When specified members can edit membership, otherwise only owners can do this.

--onlyAllowMembersViewMembership
When specified only group members can view the membership, otherwise everyone can.

--allowRequestToJoinLeave
Allow users to request membership in this group and allow users to request to leave the group.

--autoAcceptRequestToJoinLeave
If auto-accept is enabled, users will automatically be added or removed when they make a request.

--requestToJoinLeaveEmailSetting [requestToJoinLeaveEmailSetting]
All membership requests will be sent to the e-mail address specified.

Examples
Create group with title and description
m365 spo group add --webUrl https://contoso.sharepoint.com/sites/project-x --title "Project contributors" --description "This group contains all project contributors"

Create group with membership requests
m365 spo group add --webUrl https://contoso.sharepoint.com/sites/project-x --title "Project leaders" --allowRequestToJoinLeave true --requestToJoinLeaveEmailSetting<EMAIL_ADDRESS>
Additional Info
Request:
POST https://contoso.sharepoint.com/sites/project-x/_api/web/sitegroups
Accept: application/json
Content-Type: application/json

{
    "Title": "New Site Group",
    "Description": "New Site Group description",
    "AllowMembersEditMembership": false,
    "OnlyAllowMembersViewMembership": false,
    "AllowRequestToJoinLeave": false,
    "AutoAcceptRequestToJoinLeave": false,
    "RequestToJoinLeaveEmailSetting"<EMAIL_ADDRESS>}

I'm willing to work on this one.

That's awesome @milanholemans, oh and I'm seeing 'title' where 'name' is used in the get command.
You're choosing title because that is the Name of the prop in REST?
I'm doubting if we should keep 'name'...

That's awesome @milanholemans, oh and I'm seeing 'title' where 'name' is used in the other commands.
You chose title because that is the Name of the prop in REST?
Let's use name, as that is currently used on all group commands.

Yeah about this, forgot to mention the weird naming in SharePoint. Via the API it's indeed called title while in the UI it's called name.

I noticed all other commands use name indeed. Should I change this spec to name as well?

Yeah, let's keep using name. It's more consistent. A debate might be possible in a separate issue to change it for all commands if you feel strongly about it.


Update to dotnet 9 release

The packages are updated automatically (by an automated process which runs every 6 hours), but thanks for the initiative.


Progress bar for learn_from_user_feedback

Why are these changes needed?
TeachableAgent.learn_from_user_feedback can take awhile to complete. This PR adds a progress bar.
Returning immediately for asynchronous behavior is one option, but that would cause problems for all of the current teachability tests, which immediately start a new chat assuming that the vector DB has been fully updated. If an application needs asynchronous behavior, it can easily call learn_from_user_feedback from a separate process. But our focus for now is research, and the teachability mechanisms are still evolving.

Related issue number
Closes #329

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

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

Comparison is base (0468f53) 27.81% compared to head (a1c833f) 27.61%.

Files
Patch %
Lines

autogen/agentchat/contrib/teachable_agent.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #776      +/-   ##
==========================================
- Coverage   27.81%   27.61%   -0.21%
==========================================
  Files          27       27
  Lines        3480     3480
  Branches      789      789
==========================================
- Hits          968      961       -7
- Misses       2441     2447       +6
- Partials       71       72       +1

Flag
Coverage Δ

unittests
27.61% <0.00%> (-0.15%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@gagb @qingyun-wu Do one of you have time to review this small change?


Introduce SourceChange in place of TextChange

This is the first step on the journey to replacing RazorEditorParser. We
can't just get rid of this because VS is using it today.

TODO here:

[ ] I still need to add some unit tests for the new types
[ ] I need to figure out a few test failures

Updated


display and edit are wrong in a branch

this is probably due to https://github.com/Open-MBEE/mms/issues/220
ve uses the _refId from returned elements for caching, etc

behavior on mms has been changed


Added aesthetic image background for snapshot

Added an image instead of color gradient for the use of generation of snapshot, hope you like it :+1:  thx

It has conflicting files, can you look into it, or should i resolve it ?

Hi @8tool There are some key points that would help you to solve this issue.

we don't want an image background, for a dynamic snapshot based on the size of the Input.
If there is an image as a background, it might no be feasible for different sizes of the snapshot as its dimensions will be differed.
So a color gradient is preferred but a good looking one where the user can share the snapshot on the social media.
The size of the snapshot generated should be of dynamic size based on the input it has.
The styling of the text and all the details like date, username, website name should be in proper colors in contrast to the background.

Hope these instructions will help you to understand the basic idea of the functionality. Please do make the necessary changes and do reach out if any help is required.

@8tool Please do resolve the conflicts.


Fix: [BUG] category name in the breadcrumbs

Fixes Issue
Closes #2263

Changes proposed
Fixed the breadcrumbs category link error. Added a dash(-) in between the two words.

Screenshots

Note to reviewers

I noticed that the word in the breadcrumbs in mobile view had a dash(-), so I removed it. Please let me know if it's okay or I should revert that change

@rupali-codes
Please deploy this :)

@heysolomon can you please resolve the conflicts?

@heysolomon Resolve the conflicts :)

Okay, sorry I missed this. I thought it got resolved. Working on it now

@heysolomon can you please resolve the conflicts?

Okay

@Anmol-Baranwal, @rupali-codes  see what it shows me. I can't seem to resolve the conflicts because of access rights.

@Anmol-Baranwal, @rupali-codes see what it shows me. I can't seem to resolve the conflicts because of access rights.
You cannot use web editor since the conflicts are huge.
You can resolve it through the terminal itself.

@Anmol-Baranwal, @rupali-codes see what it shows me. I can't seem to resolve the conflicts because of access rights.

You cannot use web editor since the conflicts are huge. You can resolve it through the terminal itself.

Okay

@Anmol-Baranwal, I can't seem to see the conflicts on my own end. I have done git pull, still nothing. When I get conflicts like this, I run git pull on my terminal all the affected files and areas get flagged in my IDE, but it's not working for this for some reason; I might be doing something wrong I wouldn't know. Is there something I need to do? or should I close the PR and create another one?

@Anmol-Baranwal, I can't seem to see the conflicts on my own end. I have done git pull, still nothing. When I get conflicts like this, I run git pull on my terminal all the affected files and areas get flagged in my IDE, but it's not working for this for some reason; I might be doing something wrong I wouldn't know. Is there something I need to do? or should I close the PR and create another one?

The instructions are clearly mentioned.

Otherwise, you can manually compare your file changes with the original file in the repo to understand and ultimately resolve it accordingly.

@Anmol-Baranwal, I can't seem to see the conflicts on my own end. I have done git pull, still nothing. When I get conflicts like this, I run git pull on my terminal all the affected files and areas get flagged in my IDE, but it's not working for this for some reason; I might be doing something wrong I wouldn't know. Is there something I need to do? or should I close the PR and create another one?

The instructions are clearly mentioned.

Otherwise, you can manually compare your file changes with the original file in the repo to understand and ultimately resolve it accordingly.

okay, thank you.

@rupali-codes
Please deploy and let's merge this before any other PR to avoid creating conflicts.

@rupali-codes, I can't merge it


error while running the file

Hello
connected the aht10 to pi pico, saved 2 files and run the example, i get the next error:

%Run -c $EDITOR_CONTENT
Traceback (most recent call last):
File "", line 7, in
TypeError: 'id' argument required

i can see the address "0x38" of the sensor with i2c scanner.
i was trying to understand what is the missing ID .. but cannot, please help !
Thank you

ok, found it.
i2c = I2C(0, scl=Pin(1), sda=Pin(0))   # added 0,

Seems like the machine.I2C constructor behaves a bit different on Pi Pico compared to the ESP8266 boards (where I tested the example). Closing this issue...


IE11 support: Stop using startsWith in the JS codebase

https://www.w3schools.com/jsref/jsref_startswith.asp is only supported in IE12 or newer; since we intend to support IE11 and above, we should purge use of it and lint for it to prevent new introductions.

Yeah, that's a fine solution.

@zulipbot claim


Missing "Repair Geometry" step in OSM File Loader (Load only) tool

Hi @ThomasEmge ,
I have now attempted to load both the Netherlands and Denmark with the latest build of the OSM File Loader. However, when I rendered the data, I noticed issues with the display of some of the Polygon geometries in the Denmark dataset.
I then ran the Repair Geometry tool against the data, and this revealed close to 10000 faulty geometries on the Denmark dataset, primarily, but not exclusively, "ring order issues". The latter not surprisingly, as they directly relate to the non-compulsory ring order in OSM datasets and multipolygons or closed ways defined as "area" via the area key. I don't know yet how many geometries are bad in the Netherlands import, because the Repair Geometry step is only half way, but I do some issues listed in the progress dialog (which only shows a fraction, the rest is written to a text file at the end of the process).
Did you accidentally comment out a Repair Geometry step in the latest build, that needs to be activated again?

The step is still in there, but one of the helper classes had expired.

OK, thanks, I'll close this then.


Issue with default ee_mask_value with float32 datasets

XEE's open_dataset() method uses a default integer mask value as commented in the code below
      ee_mask_value (optional): Value to mask to EE nodata values. By default,
        this is 'np.iinfo(np.int32).max' i.e.<PHONE_NUMBER>.

For image bands that are integer or float64 this works fine. But for float32 values this results in the nodata values having a large number 2.14748365e+09 causing all sort of problems. There are 2 workaround that fixes this.
Workaround 1: Specify a custom nodata value in open_dataset() such as ee_mask_value=-9999
ds = xarray.open_dataset(
    withNdvi.select('ndvi'),
    engine='ee',
    crs='EPSG:3857',
    scale=10,
    geometry=geometry,
    ee_mask_value=-9999
)

Workaround 2: Cast the input bands to float64 using toDouble()
ndviCasted = withNdvi.select('ndvi').map(lambda img: img.toDouble())
ds = xarray.open_dataset(
    ndviCasted,
    engine='ee',
    crs='EPSG:3857',
    scale=10,
    geometry=geometry,
)

It would be good to fix this so the workaround is not required. Below is a notebook demonstrating the problem and the workaround.
https://colab.research.google.com/drive/1jDcgQ3LDsHxyvTmAmnqyXAGYLUgwRwAN?usp=sharing

Hello @spatialthoughts Can you close this issue as PR #103 is merged which is resolved this issue.


ruby sample buildpack from buildpacks docs

Here's my WIP spike of adding the ruby samples from the buildpacks.io docs

Changing cnb_runtime_detect and cnb_runtime_build to fmt::Display let me use both std::io::Error and anyhow in my example. Since we are only doing eprintln! we only depend on display() being implemented.

Sadly, I'm still trying to get it working with pack. :( Running into
buildpack plan TOML parsing issues.
On Mon, Jan 4, 2021 at 9:52 AM Manuel Fuchs<EMAIL_ADDRESS>wrote:

@Malax approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Malax/libcnb/pull/5#pullrequestreview-561136849, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEASLDFX77U5W6WRN6NUTSYHP5RANCNFSM4VO3WGNA
.

Sadly, I'm still trying to get it working with pack. :( Running into
buildpack plan TOML parsing issues.
On Mon, Jan 4, 2021 at 9:52 AM Manuel Fuchs<EMAIL_ADDRESS>wrote:

@Malax approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Malax/libcnb/pull/5#pullrequestreview-561136849, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEASLDFX77U5W6WRN6NUTSYHP5RANCNFSM4VO3WGNA
.

@Malax I've now updated this to implement all the features from https://buildpacks.io/docs/buildpack-author-guide/create-buildpack/caching/

@Malax I've now updated this to implement all the features from https://buildpacks.io/docs/buildpack-author-guide/create-buildpack/caching/


Adina/file names

changed the file names to uppsercase cause unix is case sensitive.
to test: make sure JabbR still works

Tested :shipit:


CI/CD Pipeline

Automate build and releases upon push into main with GitHub Actions.

Build script for Windows implemented in #2.


Replace warning and reasoning with note

Replace warning and reasoning with note as a part of the ticket #660 to uniform hints and layout

:+1:

👍


Release this as a gem

It'd be nice to be able to easily download and install this on any computer. Probably can wait for now though.

Done!

https://github.com/apainintheneck/atlasq/releases/tag/v0.1.0
https://rubygems.org/gems/atlasq/versions/0.1.0


handle small memoryviews in extract_buffers

Coverage decreased (-0.39%) to 64.79% when pulling df3e41e05ac67bfd3fcd7d9527849e4b160356ac on minrk:memoryviews into ebec294e4b81459f2cd9e33afcfac3ac2f735668 on ipython:master.

Coverage increased (+0.02%) to 65.2% when pulling a30428f327bd32dd169a5fd0cb2dac8830dfe00f on minrk:memoryviews into ebec294e4b81459f2cd9e33afcfac3ac2f735668 on ipython:master.

Disabled @coveralls horrible comments.


Its probably my fault.

I might just be an idiot:
I'm pulling my cookies from mozilla's sqlight db. The amazon cookies I see there are named: 'session-id', 'ubid-main', and 'at-main'. I'm assigning values to my dictionary file as such; but I keep getting the following error:
File C:\ProgramData\anaconda3\Lib\site-packages\spyder_kernels\py3compat.py:356 in compat_exec
exec(code, globals, locals)
File c:\users\absol\untitled0.py:13
ap = AmazonPhotos(
File ~\AppData\Roaming\Python\Python311\site-packages\amazon_photos_api.py:82 in init
self.root = self.get_root()
File ~\AppData\Roaming\Python\Python311\site-packages\amazon_photos_api.py:907 in get_root
root = r.json()['data'][0]
AttributeError: 'NoneType' object has no attribute 'json'
I'm assuming I am doing the whole cookie thing wrong, but might need someone to spell it out for me.
On another note: I provide a service to incarcerated individuals, where I curate photos and have them printed and mailed to them through the Amazon Photos platform, but I currently have to manually enter their orders in. I keep a database of the photos locally, and have the entire catalog uploaded to Amazon Photos. It would be AWESOME if there was a way to use this API to assemble photo orders and printing projects programmatically.
Sorry if this is all just a waste of your time, and thank you sooo much for the work you are doing!

Not your fault - I had the same issues.... had to rename the cookies in the code to fix it. I added a pull request that should work for you (keep the "main" names in your cookie IDs)
#8

Thank you sooo much!
On Thu, Jan 4, 2024, 9:04 AM marketviable @.***> wrote:

Not your fault - I had the same issues.... had to rename the cookies in
the code to fix it. I added a pull request that should work for you (keep
the "main" names in your cookie IDs)
#8 https://github.com/trevorhobenshield/amazon_photos/pull/8
—
Reply to this email directly, view it on GitHub
https://github.com/trevorhobenshield/amazon_photos/issues/7#issuecomment-1877350822,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDOWEX6DKIJMBGTRROD6HPLYM3HHJAVCNFSM6AAAAABBBXESS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZXGM2TAOBSGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for catching this @marketviable. I can't merge your PR because I'm dropping the env var option as it will potentially raise the same issue in the future. Users will now copy, at minimum, the essential session-id and ubid/at variants alongside any other special cookies that may be required in certain countries.


Fragment called twice after orientation change

The problem is described here for default NavigationDrawer:
http://blog.sqisland.com/2014/06/navigationdrawer-creates-fragment-twice.html

Checking your code you use the savedInstanceState Bundle only for checking if the fragment should be replaced or not, could not be better using a boolean?

Boolean flag seems fine

I have released the 1.3 alpha now, which include this new merge with a boolean flag.
Thank you for your help!


Improve logging (audit, activity, transaction, trace, access)

Problem description
Understand the requirements of each log type and properly categorize types. Currently, supported Audit, debug and access logs. Need to differentiate the above logs types and implement them.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


pip install -u azureml-sdk[automl,explain] fails in MAC OS 10.13 High Sierra

Package Name: pip install -u azureml-sdk[automl,explain]
Package Version: new release september
Operating System: MAC OS
Python Version: 3.7

Unable to install azureml-sdk packages. Following errors found:
ERROR: Could not find a version that satisfies the requirement onnxruntime==1.0.0 (from azureml-automl-runtime~=1.13.0->azureml-train-automl~=1.13.0; extra == "automl"->azureml-sdk[automl,explain]) (from versions: 0.1.4, 0.2.1, 0.3.0, 0.4.0, 0.5.0, 1.1.0, 1.1.1, 1.3.0, 1.4.0)
ERROR: No matching distribution found for onnxruntime==1.0.0 (from azureml-automl-runtime~=1.13.0->azureml-train-automl~=1.13.0; extra == "automl"->azureml-sdk[automl,explain])

@Mehrnoosh Sameki<EMAIL_ADDRESS>/ @Cesar De la Torre<EMAIL_ADDRESS>Get Outlook for iOShttps://aka.ms/o0ukef

From: msftbot[bot]<EMAIL_ADDRESS>Sent: Tuesday, September 8, 2020 11:32:53 AM
To: Azure/azure-sdk-for-python<EMAIL_ADDRESS>Cc: azureml-github<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-python] pip install -u azureml-sdk[automl,explain] fails in MAC OS 10.13 High Sierra (#13607)
Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @azureml-githubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fazureml-github&data=02|01|Jordane%40microsoft.com|5a617726544f40ad5fe608d85425998b|72f988bf86f141af91ab2d7cd011db47|1|0|637351867756389951&sdata=RBxbahCI5G3eDLk4a2PzsALni8xts3%2B85Or%2Fk%2FAsHfM%3D&reserved=0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-python%2Fissues%2F13607%23issuecomment-689059107&data=02|01|Jordane%40microsoft.com|5a617726544f40ad5fe608d85425998b|72f988bf86f141af91ab2d7cd011db47|1|0|637351867756399946&sdata=YHvmtOibxB%2F95iAMEBMNbyxTDbmOlSubOI6iu2MrRlw%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOVYYXULY4647AZDJK2HJ53SEZ2FLANCNFSM4Q5QNQGQ&data=02|01|Jordane%40microsoft.com|5a617726544f40ad5fe608d85425998b|72f988bf86f141af91ab2d7cd011db47|1|0|637351867756399946&sdata=7RrgnU%2B8yTblRQrOK%2FK9sNq2%2BG6AeOCJvJbiIggy5cE%3D&reserved=0.

Having exact same issue in macOS Catalina as well.
ERROR: Could not find a version that satisfies the requirement onnxruntime==1.0.0 (from azureml-train-automl-runtime~=1.13.0->azureml-train-automl~=1.13.0; extra == "automl"->azureml-sdk[automl,explain]) (from versions: 1.2.0, 1.3.0, 1.4.0)
ERROR: No matching distribution found for onnxruntime==1.0.0 (from azureml-train-automl-runtime~=1.13.0->azureml-train-automl~=1.13.0; extra == "automl"->azureml-sdk[automl,explain])

Automl supports both local training or remote training on an Azure Machine Learning Compute cluster.
For local training or inference, please follow the instructions for “Setup using a Local Conda environment” from GitHub.  This is sufficient to run all the automl sample notebooks, including remote training.  This creates a Python 3.6 environment.
For remote training and inference only, you can use a smaller environment with just the azureml-sdk package. This can be used in Python 3.6, 3.7 or 3.8 environments.  This includes the automl packages needed for the sample notebook: auto-ml-regression-model-proxy.ipynb.  This notebook also requires packages: pandas<1.0.0, numpy<1.19.0, azureml-widgets, matplotlib and sklearn.

@3Mcolab Did my reply above help?
Can we close this issue?

#please-close


MINOR: Manually ported changes in 8c3c9548b636cdf760d2537afe115942d13bc003

Porting manually to 0.9.0 (no cherry-pick due to conflicts)
This will allow concurrent system test runs on the same machine

@guozhangwang Pinging for review
Running tests here:
http://jenkins.confluent.io/job/kafka_systest_branch_builder2/14

@guozhangwang Tests pass in branch builder:
http://jenkins.confluent.io/job/kafka_systest_branch_builder2/15/console

The job launched by Geoff was successful.

LGTM. Merged to 0.9.0.


Update developer guide

Codecov Report

Merging #102 (f7b8fd6) into master (ef407ca) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #102   +/-   ##
=========================================
  Coverage     43.84%   43.84%
  Complexity      522      522
=========================================
  Files           130      130
  Lines          2689     2689
  Branches        299      299
=========================================
  Hits           1179     1179
  Misses         1437     1437
  Partials         73       73

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 ef407ca...f7b8fd6. Read the comment docs.


#630: Deleting user's profile pics on deleting of user account

Fixed, working fine!

Also, When it comes to merging this, do you mind if I 'squash and merge' since there's a lot of commits in this branch which are simply merges of master?

I never mind. Please "squash and merge". I tried, but failed.


Is this project deprecated?

Seems like all macOS-like UI kit will eventually be deprecated

I am asking myself the same question over and over again... as anybody found an alternative UI Kit to this one?


Unicode symbols - setup issue

Users with characters such as á, ě, ö, etc. in their Windows username are unable to setup a wallet. These characters are common in many world languages so it would be cool to fix it. I don't think many people will be willing to create a new user account just to install and run a wallet.

The problem is our binary that are build with Rust. Most of 3rd party library doesn't process unicode strings correctly. That is why it is not allowed.
We are not fixing them, it is a limitation of technology that we are using.
Closing as wont't fix


Update supported-event-types.md

Learn Build status updates of commit ea396e5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender/supported-event-types.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
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.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Support gevent and/or other event loops

This is accomplished by pyzmq in the pyzmq.green module. Theoretically, an integration could be accomplished using nn_poll, however those docs make a reference to integrating with other event loops in the NOTE section, so a more efficient integration should be possible.
This is something I hope to look into if I can find the time, but perhaps someone else has a better lead on the subject.

A combination of nanomsg.Socket.send_fd and nanomsg.Socket.recv_fd, with gevent.socket.wait_write and gevent.socket.wait_read (respectively), may be all that's needed?

So, I wrongly assumed that because the pyzmq support for gevent was so involved, that it would be the case for nanomsg as well. The technique form the previous comment does appear to work well, however. Ideally there would be an example somewhere to show how this can be done, but this issue no longer seems necessary.

any ideas on where I might be able to see an example of how this is done?  I agree an simple example would be great...

Actually, it ended up not working for me (yet).  I get a condition where the wait_read call blocks a greenlet but then after it releases, the SUB socket.recv() still blocks.  I always end up with an extra message in the recv queue.  Same pattern works fine with os threads.

I think your problem is just with the event loop. Try putting a gevent.sleep(1) after the send. That should ensure that the listen greenlet has time to run.

You can't just wait. The event loop is not just running in the background. You have to call gevent.sleep or gevent.wait.

ok, seems like it's working in my toy example.  back to the salt mines...  Thanks for the help!

for posterity, a working example of how one might implement this with a PUB/SUB pattern.  The use of DONTWAIT is required to keep non-subscribed sock.recv() calls from blocking.
from gevent import monkey
import gevent
monkey.patch_all()
from nanomsg import Socket, PUB, SUB, SUB_SUBSCRIBE, DONTWAIT
def listen(sock):
    while True:
        gevent.socket.wait_read(sock.recv_fd)
        try:
            print sock.recv(flags=DONTWAIT)
        except Exception, e:
            pass
        gevent.sleep(0.1)

pub = Socket(PUB)
pub.bind('ipc:///tmp/new.sock')
sub = Socket(SUB)
sub.connect('ipc:///tmp/new.sock')
sub.set_string_option(SUB, SUB_SUBSCRIBE, 'h')
sub2 = Socket(SUB)
sub2.connect('ipc:///tmp/new.sock')
sub2.set_string_option(SUB, SUB_SUBSCRIBE, '')
gevent.spawn(listen, sub)
gevent.spawn(listen, sub2)
gevent.sleep()
pub.send('hi')
gevent.sleep(0.1)
pub.send('a different greeting for sub2')
gevent.sleep(0.1)


[DPR2-189] Tweak for date-time filtering

Fixed issue with date-times being filtered excluding the last day of the filter.
Refactored data repo so that it accepts filters, with metadata about what type of filter they are.
Added methods to get simplified product/report definition if only a single report is being worked with.
Changed data service to validate and map to new filters with metadata, rather than just validate.

It looks good to me.
My main comment would be if we can have a test with a case that passes with the code change that adds one day to the end date but would fail if you simply had the less than or equal to comparison as it was before.


merge from arlac77/template-kronos-module

package.json

chore(package): add arlac77/template-kronos-module (template.inheritFrom)

Codecov Report

Merging #670 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #670   +/-   ##
=======================================
  Coverage   91.87%   91.87%
=======================================
  Files           4        4
  Lines         283      283
  Branches       13       13
=======================================
  Hits          260      260
  Misses         23       23

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 0b34c89...8b73fa7. Read the comment docs.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[FEATURE] App Tracking Transparency

Is your feature request related to a problem? If so, please describe.
Godot game was just rejected with

Guideline 5.1.2 - Legal - Privacy - Data Use and Sharing
The app privacy information provided in App Store Connect indicates the app collects data in order to track the user, including Performance Data, Coarse Location, Device ID, Advertising Data, Product Interaction, and Crash Data. However, the app does not use App Tracking Transparency to request the user's permission before tracking their activity.
Apps need to receive the user’s permission through the AppTrackingTransparency framework before collecting data used to track them. This requirement protects the privacy of users.

I think I understand https://poingstudios.github.io/godot-admob-plugin/privacy/user_messaging_tools/get_started/ section of the documentation but Apple doesn't want that and expects https://developer.apple.com/documentation/apptrackingtransparency (or at least expects first + second one).
What feature or improvement would you like to see?
Is this feature accessible from Godot but with documentation missing?
Is it yet to be implemented?
Can you please provide better documentation for the whole "User Messaging" process? It is not entirely clear what kind of message Apple expects to see in order for this plugin to be used with Godot.
If there is better place to ask this questions please point me there.
If you need help implementing this I may be able to jump in but at this stage I don't know what is supported and what is missing.
Additional context
No response

Best documentation I could find so far is at
https://developers.google.com/admob/ios/privacy/strategies#request
It should be easy to add this request to the existing plugin but I'd really love to avoid fiddling with code myself -- all the new App Store submissions with Godot using godot-admob will be rejected until this is sorted out.

Hi try to add AppTrackingTransparency framework to your Xcode project

I have added NSUserTrackingUsageDescription into plist and AppTrackingTransparency framework but I am still getting what looks like wrong/web looking dialog? Invocation code is identical to one in documentation for this plugin. If this should result in Apple's privacy dialog I can assume I did something wrong on this end. Image attached.

It works! Thank you so much, I'll send some coffee over weekend ;)
Problem was that I defined both "European regulations" and "IDFA explainer" for the same iOS game. When they are defined that way admob shows web/android like dialog. Defining only "IDFA explainer" and it shows proper iOS tracking dialog. This could be something that should reach the documentation but in general this works well and I'll close the issue.


Network analysis on 3GPP mailing lists

it seems to me the Jupyter notebook for network analysis does not read the 3GPP file (.mbox), anyone else facing this? I tried both with indicating the name of the folder in the archive name space and with indicating the name of the file (after moving it to the archive folder). I also tried changing the file extension to '.mail' and '.csv', but wouldn't work anyway. The notebook still says file not found.
Anyone has any hint on this?
Thanks!

Yes, the notebookes/code-the-notebooks-depend-on need to be imrproved.
Essentially, what that particular notebook needs is a pandas.DataFrame, which you can create for Listserv 16.5 mailing lists as ListservList.from_mbox(name=... , filepath=...).to_pandas_dataframe() (for details see here ).
However, I can already tell you that there is not much to see in the particular network analysis done in the notebook, as the senders and receivers are for some reason always the same (which you can also see from the website, and I don't now why the 3GPP mailinglist is handled that way).

Thank you @Christovis!
I'll try it.

Assume this has been resolved by now


Update azure-cosmos-db-v2.md

#sign-off

#sign-off

@revinjchalil
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


update.04.06.20

Code updated for the boxes after cleaning and fixing a bug.

Pull request accepted and merged.
Many thanks.
MA
From: Daniel Redondo<EMAIL_ADDRESS>Reply-To: migariane/ColliderApp<EMAIL_ADDRESS>Date: Thursday, June 4, 2020 at 09:51
To: migariane/ColliderApp<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [migariane/ColliderApp] update.04.06.20 (#1)
Code updated for the boxes after cleaning and fixing a bug.
You can view, comment on, or merge this pull request online at:
https://github.com/migariane/ColliderApp/pull/1
Commit Summary
Merge pull request #1 from migariane/master
update_04.06.20
File Changes
M code_boxes_article.R (395)
Patch Links:
https://github.com/migariane/ColliderApp/pull/1.patch
https://github.com/migariane/ColliderApp/pull/1.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Phone number pattern for Benin and Ivory Coast

Currently, there is no phone number pattern in iD for Benin and Ivory Coast.
The E.123 format (what OpenStreetMap wiki asks for) is:

for Ivory Coast: +225 ab uvwxyz
for Benin: +229 ab uvwxyz

“ab” is the area code.
But in real-world life the phone numbers are always structured by groups of two digits:

for Ivory Coast: +225 ab uv wx yz
for Benin: +229 ab uv wx yz

I would propose something like

for Ivory Coast: +225 20 12 34 56
for Benin: +229 20 12 34 56

(20 is in both countries a landline code.)

Thanks for the info!  This is a great task for someone getting started to add those formats to phone-formats.json.

Already solved in PR #4314 .

Already solved in PR #4314 .

Thanks @manfredbrandl!  Sorry I didn't see that, first day back from vacation....

Thanks!


Instructions on how to debug extenstions [request]

i know how to debug scripts. having trouble to debug with the roots command in the middle.
using webstorm and/or iron-node.
would appreciate assistance.

Need a lot more info here to be able to figure out what your issue is and how we can help with solving it...

platforms: windows 10
i have been using some the extensions - some of them are failing for different reasons. roots-colllection had mac only path for example.
i want to be able to run roots in debug mode - having the ability to break in some the extension and figure out what is wrong.
I can't seem to do it since i am not running "node script.js" but a "root watch" command.
when trying to setup webstorm debug node engine or iron-node to run roots - it fails on syntax. i am sure there is an easy way to setup. not sure what it is.

I'm sorry, I don't really know what webstorm or iron-node are, and unfortunately I can't develop roots around these specific tools. Roots exposes a command line interface and a javascript API, and that's all I plan on having it expose.
However, if you'd like to debug an extension, you can quite easily get inside of one of them. Just go into your node_modules folder then navigate to the folder for that extension. From there, you can check out the code and add console.logs wherever you want and figure out this issue. Another good approach is to file an issue in the extension's repo and ask the author about the issue you are having. You can find roots-collections' repo here.
Hope you can figure this out, and happy to help if you have any further questions!

i have been using console logs and i have been in contact with authors of extension. that is still being half blind.
i figured since you are the main developer you have a good setup. nevermind.

Sorry that you feel that way @orenmizr! I'm afraid it's hard for me to imagine how directly interacting with the code that's being executed can be interpreted as "half blind", but alas I may simply be ignorant to the nature of your apparently superior tooling.
Keep in mind that this is an open source project, and as is the case with all interaction with any open source projects, the most important thing is to make sure you are here with a spirit of openness, politeness, and willingness to learn and contribute.
If there's other questions I can answer or anything else I can help with I will be here, using my free time to try to make your life easier :grinning:

Hey also I had a moment to take a look at iron-node. It says you can just point it to a node.js file and it will do its thing. Roots is written in node.js entirely, so it seems feasible that you'd be able to get it to work.
When you run roots watch, this is simply a command line abbreviation for a script execution, in this case it's a node script. So in theory if you can find out where that script is, you can run iron-node on that script and it will work the way you intend.
In unix, you can use which roots to reveal the location of the bin's alias. Typically it will be in /usr/local/bin/roots. On windows, I don't know because I never use windows, but I'm sure you could figure this out. Have you given this a try?
Please note that this issue has nothing to do with roots at all, and is just trying to get around a personal issue for you with your setup, which is why I closed it originally. But in the spirit of trying to be helpful, perhaps this will get you moving in the right direction.

i absolutely did not mean to offend, honestly. you've build something that is beautifully balanced between flexibility and simplicity. the fact you say you did it without the ability to pause execution and debug is admirable. I assumed that wasn't the case since you've written most of the code, hence my question.
i personally need a debug tool. i feel it makes me personally more productive. but that is subjective.
btw.
i did try directing iron-node and intelij to what i thought were the files. but it did not work for me. i do know it is not a roots question per-say :) hashtag shotinthedark
thanks.

Where were the files you were looking for? It should be just one file. Did you cat the file and see if it actually was the correct bin file? Is there a way to pass a standard command line argument along with iron-node? You'd need to call the file and also pass in the watch argument. I feel like you could figure this one out, you must be pretty close already.
Truth is that you get the same snapshot effect of stopping execution by just logging out the relevant values and letting execution complete, and/or thoroughly reading the error trace. If I am unable to figure out the issue through the error trace and maybe a console log or two, for me this usually means that there's something about the architecture or the language that I don't understand. You'll find that most javascript developers do not use IDEs or complex debugging tools, but still manage to be productive and write good code. And there's the additional benefit of not needing as many tools or depending on as many things in order to work, which is really nice, hashtag minimalism :wink:


Very large memory-consumption in SplunkHttp

Depending on whether Splunk-logging is enabled in log4j2.xml:
    <SplunkHttp
    ...
                     batch_size_count="2099"
                     batch_interval="3"/>

our application's total heap-usage rises by about 4Gb, as can be seen on these charts from jconsole! Granted, the application is verbose with multiple events per second at times, but 4Gb still seems excessive...
The receiving HEC is the same for the entire enterprise -- I doubt, that's the bottleneck. Please, advise.
Edit: the #249, which seems to be about the same/similar problem, is closed -- perhaps, prematurely. There is no good way to handle the Splunk server being down (or slow), perhaps, there should be options to simply drop log-entries, when such a situation happens -- to preserve heap -- based either on the events' age and/or severity.
The total count of such dropped messages can be kept -- and logged on its own, when possible: "HEC latency necessitated dropping of %u events".

Seeing the same issue. In our case, HTTP requests get stuck in the HTTP client's queue (okhttp3.Dispatcher.readyAsyncCalls). Same on your side?

Experiencing the same issue. In our case this even caused our application running in AWS Fargate to crash as the G1 Garbage Collector would take up 100% of available CPU cycles, which would eventually lead to ECS shutting down the application because it wasn't responding to health checks in a timely fashion.
Our temporary fix was to decrease the amount of logs we're sending to Splunk and increase the heap size, but the long-term solution will likely be to log to Cloudwatch only and export the logs with a dedicated Lambda. I'd be interested if there's any other fixes for this issue though.

As a workaround, batching log events helped in our case. That way, the amount of requests to Splunk is reduced and they’re less likely to pile up in the HTTP client.

As a workaround, batching log events helped in our case. That way, the amount of requests to Splunk is reduced and they’re less likely to pile up in the HTTP client.

Just curious, what batch-size did you find useful? We raised the number from the original 17 to 2099 -- and that still leads to the above-charted heap-usage... Our batch_interval here is 3, though -- maybe, we ought to raise that too...

We started off with the recommended values (see here)
Looking at the settings you posted initially, it appears to me that you might be sending too fast. batch_interval is interpreted in milliseconds, so you might want to raise that to 3000.

We encountered this recently, so for anyone trying to understand the problem, it looks like the default behavior of the appender is to dispatch every logging event as a discrete http request to the HEC endpoint. Those requests are enqueued in the okhttp dispatcher, which by default (send_mode=serial) only uses one thread to send the requests to the API. Those requests pile up in the okhttp buffer until the application hits an OOME.
You fix this by configuring batch_size_count or batch_size_bytes to a nonzero value, by also configuring batch_interval to a nonzero number of ms (to guarantee that an incomplete buffer is flushed if no events arrive to fill it over some period), and by optionally configuring send_mode to "parallel" so that more threads can step in to help with bursts of logging.
As I understand it, anyway.

You fix this by configuring batch_size_count or batch_size_bytes

None of these methods will fix the problem. They'll help reduce it, yes, but not eliminate completely. A fix would involve dropping events, when JVM is getting closer to hitting OOM, but no one dares to propose such a dataloss :(


issue with telegram token envirement variable

hello
im trying to cargo run but i recived this kind of error:
Failed to retrieve 'me': Api(Unknown("Not Found"))

But my token is valid and shoud be work

Hey,
Where did you set the teloxide token environment variable in?
Currently the app expects it to be set in the main.rs file, and it overwrites your environment variable if it was set before the execution of main.rs and you left those lines empty.
https://github.com/marce1l/snipers/blob/970746861cddda341e30a8d1b78ba77841a76836/src/main.rs#L22-L30
This is a dumb design by me. I will remove this part and make it so you can for example use a .env file instead. Meanwhile you can try setting the environment variable in the main.rs file.


Fix type of beam_ratio in FresnelOpticalWavefront

The beam_ratio parameter is specified as int, but should be float, since it is given by 1/Q, where Q is the integer entrance pupil padding factor.

Coverage increased (+0.6%) to 64.165% when pulling 929f285382fa4c50a13149d84a57fa3bec8a12af on sdwill:master into bc8435848636e6836118cf9d23628cb822021637 on mperrin:master.

Thanks @sdwill, good catch.


Add Support for ItemStack#deserializeBytes & ItemStack#serializeAsBytes and complete ItemMetaSerialization

Fixes #1013
Description

Completed Serialization for all Metas that where not fully serialized.
Add Support for ItemStack#deserializeBytes via BukkitObjectInputStreamMock
Add Support for ItemStack#serializeAsBytes via BukkitObjectOutputStreamMock
There where also a change in metas that where required beaucse the some clones failed
ItemFactoryMock#ensureServerConversions needed to be implemented so the deserialization can work.
Also I would apricote if someone can improve upon the UntiTest so that all metas will be fully tested.
Checklist
The following items should be checked before the pull request can be merged.

[x] Code follows existing style.
[x] Unit tests added (if applicable).

Please add a test for serialization / deserialization for every meta type ( really add a test for every method you have implemented / modified). One example test for this, is to serialize an item meta, and then check that the output when deserializing is the same class, and has the same properties. Another note is that this class persistency behavior needs to apply for UnsafeValuesMock also (is this the case currenttly?)

ItemStack#serialize automatically puts the default meta of an item, regardless if has one ore not.
So the unsafe value test indelibly tests if the meta type machtes, this test was the main reason why extend the config serialization at all because they did not match.

You still need to do this (code guidlines): go from:
if(isStatement)
    doSomething();

to this:
if(isStatement)
{
    doSomething();
}

Sry missed those, thought auto format hat replaced them...
I checked all metas and also added the one others where missing