Draw a white circle around the icon.

This can probably be solved just by cleaning up the alpha channel on the
existing icon.
On Mon, Feb 23, 2015 at 12:56 AM, iislucas<EMAIL_ADDRESS>wrote:

Notice the odd/bad effect around the sharing access icon:
[image: image]
https://cloud.githubusercontent.com/assets/415450/6323355/b340cf90-baf6-11e4-9066-543b839d7fdc.png
Ideas:

Do-create the icon with a black background.
Draw a white circle around the icon.

—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/970.

I see why we might want a glowing icon for the chrome extension button, and next to the contact on the roster, but having a glowing icon at the bottom seems annoying to me.  Maybe we should just revert this one to the old non-glowing icons

SGTM

done https://github.com/uProxy/uproxy/pull/1024


[Feature] Adding n sided polygon calcualtion feature in 2D shapes calculator

Description
I want to add an option to calculate the area and perimeter of any n sided polygon with the help of some basic logics using Javascript.
If this feature can be added, i will be happy if you assign me this issue.
Thank you.
Screenshots

Here only hexagon is there in the options. Other polygons like hepta or octagon's area and perimeter cannot be calculated.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[X] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Pls complete your assigned issue first @Abhishek-465

@Rakesh9100, Assign this to me under SWOC24, after merging the PR for Credit card payoff calculator, as I want to complete more and more small issues faster, because the SWOC24 date is approaching to its end.

Proceed @Yuvraj960


@conda-forge-admin,please add bot automerge

@conda-forge-admin,please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/mplhep_data-feedstock#3.


upgrade: Report the admin server upgrade error if knife has failed

 Changing platform during admin server upgrade failed

agree with @rhafer that exiting on that check may be is the best option.

This is really complicated. I think that once packages are upgraded, we need to let the admin server reboot and handle possible failures on Cloud7 side. Otherwise I'm not sure what the UI should tell the user if it detects the failure during this step: upgraded crowbar has already Cloud7 sources...

This is really complicated. I think that once packages are upgraded, we need to let the admin server reboot and handle possible failures on Cloud7 side.

In that case wouldn't it make sense to just run the knife command before upgrading the packages? (i.e. after stopping chef-client)

In that case wouldn't it make sense to just run the knife command before upgrading the packages? (i.e.
after stopping chef-client)

That might work actually... I have to test it


Fix Build Issue on Ubuntu 20.04

Description

Fixes #

https://github.com/carla-simulator/carla/issues/6901

https://github.com/carla-simulator/carla/issues/6776

Added missing c++ standard header, updated library version & compile command
Updated std C++ to 17
Updated Boost to 1.83.0 and build command accordingly - use local clang packaged under Unreal Engine
Updated Proj to 9.3.0 according to the https://github.com/carla-simulator/carla/issues/6776
Made sure to build dependencies to be built with Position Independent Code

Where has this been tested?

Platform(s): Ubuntu 20.04
Python version(s): Python 3.8
Unreal Engine version(s): UE 4.26

Possible Drawbacks
Introduction of C++ 17, boost 1.83.0, proj 9.3.0 may introduce compatibility issue

This change is

I do have the the mentioned errors, however checking out this PR still results in the same 'thread' error in my case.
carla$ make PythonAPI
Util/BuildTools/Linux.mk:163: warning: overriding recipe for target 'downloadplugins'
Util/BuildTools/Linux.mk:142: warning: ignoring old recipe for target 'downloadplugins'
Setup.sh: Retrieving boost.
Setup.sh: Extracting boost for Python 3.
Building B2 engine..

A C++11 capable compiler is required for building the B2 engine.
Toolset 'clang' does not appear to support C++11.

> clang++ -x c++ -std=c++11  check_cxx11.cpp
check_cxx11.cpp:14:10: fatal error: 'thread' file not found
#include <thread>
         ^~~~~~~~
1 error generated.

** Note, the C++11 capable compiler is _only_ required for building the B2
** engine. The B2 build system allows for using any C++ level and any other
** supported language and resource in your projects.

You can specify the toolset as the argument, i.e.:
    ./build.sh [options] gcc

Toolsets supported by this script are:
    acc, clang, como, gcc, intel-darwin, intel-linux, kcc, kylix, mipspro,
    pathscale, pgi, qcc, sun, sunpro, tru64cxx, vacpp

For any toolset you can override the path to the compiler with the '--cxx'
option. You can also use additional flags for the compiler with the
'--cxxflags' option.

A special toolset; cxx, is available which is used as a fallback when a more
specific toolset is not found and the cxx command is detected. The 'cxx'
toolset will use the '--cxx' and '--cxxflags' options, if present.

Options:
    --help                  Show this help message.
    --verbose               Show messages about what this script is doing.
    --debug                 Build b2 with debug information, and no
                            optimizations.
    --guess-toolset         Print the toolset we can detect for building.
    --cxx=CXX               The compiler exec to use instead of the detected
                            compiler exec.
    --cxxflags=CXXFLAGS     The compiler flags to use in addition to the
                            flags for the detected compiler.

Failed to build B2 build engine
make: *** [Util/BuildTools/Linux.mk:142: setup] Error 1

The fix didn't work for me, either.

@ll7 @juliancao sorry for inconvenience, this error could be related to trying to build boost with clang included in Carla's fork of  UnrealEngine, would you test build with system installed g++ by removing include="${LLVM_INCLUDE},${LLVM_LIBPATH} in https://github.com/carla-simulator/carla/pull/6974/files#diff-b617d525d7cf7a5c9bb4a0a916007bd224ad09709521611a0b23df1701084624R122-R123

I tried removing include="${LLVM_INCLUDE},${LLVM_LIBPATH} , the error still exists

@ll7 @juliancao sorry for inconvenience, this error could be related to trying to build boost with clang included in Carla's fork of UnrealEngine, would you test build with system installed g++ by removing include="${LLVM_INCLUDE},${LLVM_LIBPATH} in https://github.com/carla-simulator/carla/pull/6974/files#diff-b617d525d7cf7a5c9bb4a0a916007bd224ad09709521611a0b23df1701084624R122-R123

Sorry, I upgraded to ubuntu 22.04 and no longer have any issues. Therefore, I am unable to test your suggestion.

All PRs must be against dev. We have fixed issues with Ubuntu 22.04 So please check latest docs and see if this is still necesary
https://carla.readthedocs.io/en/latest/build_linux/


Services which contain spaces not showing in MMonit

If the service I'm trying to monitor does not have a space, it works great and shows up in mMonit. But if there's a space, then no luck. Web client shows, but not bluetooth device monitor. I've tried double sets of double quotes, just one set of double quotes, throwing single quotes in there, all no bueno.
<Services>     <Process Name=""bluetooth device monitor"">       <ActionRate Count="3" Cycle="5" ActionType="5"></ActionRate>       <Resources>         <Resource Type="1" ComparisonOperator=">" Limit="800" ActionType="2" />       </Resources>     </Process>     <Process Name="webclient">       <ActionRate Count="3" Cycle="5" ActionType="5"></ActionRate>       <Resources>         <Resource Type="1" ComparisonOperator=">" Limit="800" ActionType="2" />       </Resources>     </Process>     <Filesystem Name="C:\"></Filesystem>   </Services>

Hey,
thanks for your issue.

As you can see in the image above the 'Service name' of a Windows-Service never includes whitespaces.
Therefore can you try to use the 'Service name' of the "bluetooth device monitor"-Service you are trying to monitor.
If this still does not work please leave me another comment.
Thx

Thanks for the super quick response. And while true is most cases, there can be service names with spaces. As shown in the attached picture. (FYI great job on this by the way, It's working fantastically for what I'm playing with !!)

Hey,
sry this response took a little longer.
I did a new release , using which i was able to add Services with Whitespaces in them.

Thanks for your kind words, I hope your issue is resolved in this new version.
If not, please leave another comment.


add entries() and values() to RedisMapCache

Actually should have done this when I added keys() but I forgot. This adds RedisMapCache.values() to get all values in the cache and RedisMapCache.entries() to get all keys+values in the cache, basically the same as Map.values() and Map.entries(). At some point I think it would be cool if these were implemented in all caches but that sounds like a bit too much work and I'm lazy 😎
I ended up having to loosen up resolveRedisInstance() because it was causing tests to fail. In general I think typescript types should prevent people doing genuinely stupid stuff and it gets in the way in annoying scenarios with the strict "instanceof" check - for example, circular dependencies and certain setups can cause instanceof to fail for the exact same class. I think its purpose should be more "turn this into a redis instance" instead of "make sure this is a redis instance", as much as I want it to be strict and only allow the right things I think leaning on the side of "its probably fine" makes the library more usable in weird situations that I somehow keep getting myself into. I probably got a bit carried away but I couldn't get tests to pass without changing it, and I got a taste for blood. I also added tests for it. I'm not a huge fan of the REDIS_CLASS_NAMES thing but it works for now, I'm sure one day someone will come up with a stupidly simple way to do the exact same thing.
I also reverted this commit because that was intentional, it breaks the behaviour of keys() and is accounted for in clear() and delete().

Ended up adding getMany() and using that in values() because it makes more sense and im sure getMany() can be useful in the future

Listen buddy, you're being a little too productive, this has to stop.

I have done practically nothing for two weeks, don't worry I'm not breaking any records x


Correct GDnative's godot_string_find_last method name

That was fast to mere ! o_O

🔫🔫 I'm faster than my shadow when it comes to GDNative related PRs. 🔫🔫


Keyboard covers text input on create multisig page (Android)

Description
When viewing Multix on Android (within Nova wallet using the chrome engine) the inputs do not move up above the keyboard when tapped.
Steps To Reproduce

Open Nova Wallet on Android
Navigate to Multix
Connect wallet
Open hamburger menu
Tap "New Multisig"
Add a signatory and then try to add a second

Expected Behaviour
When the input is tapped and the keyboard presented the view should adjust so that the inputs are still visible whilst the user is typing
Console Error
N/A
Screenshots / Video
https://user-images.githubusercontent.com/3982665/234660271-5d04ec30-fa2c-4a74-8290-492fabec42b1.mp4
**Test Host Specifics: **

OS: Android
Browser: Chrome engine (Nova Wallet)
Version 6.10
Wallet: Nova

Additional context
Works as intended within Nova Wallet on iOS

Can I pick this up if it's still open?

Sure. Feel free to discuss what you think could be a good solution here if you want.

I'm reading the code to try and see what the problem is. Do you have further information about the issue, like;

What file(s) contain(s) the components rendering this form?

What you can see in the video above comes from this component https://github.com/ChainSafe/Multix/blob/main/packages/ui/src/components/select/SignatorySelection.tsx#L14

Hi @Tbaut, I took a look into this problem and while it's a UX issue, there is nothing wrong with the code and logic of the related component(s).
The problem is how input fields works on Android web apps. I found some interesting solutions, but I won't recommend because they are hacky.

https://ux.stackexchange.com/questions/122951/on-screen-keyboard-covers-input-field
https://support.backendless.com/t/android-keyboard-hides-input-field/14780
https://stackoverflow.com/questions/54575309/how-to-prevent-mobile-keyboard-from-covering-html-input

If we do follow this, we will have to selectively apply them to specific browsers.
Do you have any recommendations @Tbaut

Hey @bolajahmad thank you very much for your research. This issue has been open for so long for this reason: there's indeed no easy solution :|
From your links, I think https://stackoverflow.com/a/54575847/3086912 seems to be the best hack, and the most easy to understand. I think this is something we could implement and apply only to the smaller screens (using the @media (min-width: ${({ theme }) => theme.breakpoints.values.sm}px) for instance).
Feel free to go this route.


Is it possible to pass data attributes to the contentEditable div?

We have an issue in neetoForm due to Grammarly extension interfering with the scrolling while a neeto-editor is in focus. One way to solve it is to add few data-attributes to the content editable div. Is it possible to do from neeto-editor?
@AbhayVAshokan _a
question _t

@ajmaln I will look into the possibilities and let you know.

@gaagul _a


New scalar observation

What changes do you make in this PR?

Please describe why you create this PR

Checklist

[ ] I have merged the latest main branch into current branch.
[ ] I have run bash scripts/format.sh before merging.
Please use "squash and merge" mode.

Closed since this PR is stale.


Crash the build during CI whenever linter warnings are encountered

Fixes issue #906.
Whenever someone runs the build script with the environmental variable CI set to true then it will crash when any linter warnings are encountered.
CI=true npm run build

I have tested this locally by generating an app using create-react-app and the updated react-scripts. Is this enough or should there be an automated way of testing this functionality? If so, can anyone give some advice on how to go about testing this?
I am also struggling to find the best location to document this feature. I considered updating the   npm run build section of the README with this information however, I am afraid that explaining this feature in that location will only detract from the clarity to the reader. Perhaps I should create a new section in the user guide where this is explained, any thoughts?

Thanks! Maybe add a paragraph to https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/template/README.md#continuous-integration?
Can you post a screenshot of what the log looks like with warnings?

Here is a screenshot of the output:)

I have updated the docs, let me know what you think!

👍

This patch triggers build errors even when webpack emits a warning, e.g.:
This seems to be a pre-built javascript file. Though this is possible, it's not recommended. Try to require the original source to get better results.
Since I have no access to webpack.config.js, I can't use [module.noParse](https://webpack.github.io/docs/configuration.html#module-noparse).
Is there a proper way to fix this besides ejecting?

I think we might have to change the way we catch the ESLint errors here – failing for any Webpack warning is not good, because mostly harmless warnings are sometimes also triggered by UglifyJS etc. Is there a way we can only catch ESLint errors?

@scholtzm Can you open a new issue about this problem so we can keep track of it? A merged pull request can't be reopened.

Yep, was going to suggest it as well.

That is a side affect I did not take into account:( I hope this gets fixed soon! Thanks a lot for finding this bug @scholtzm!

FWIW I didn't see this addition to the docs, and the error was not clear that this was because of process.env.CI was set (we didn't know it was). I'd suggest considering logging an explicit error message describing something like When process.env.CI = true, warnings are treated as failures. Most CI servers set this automatically.

@jayphelps 👍  Wanna send a PR?

Great feedback! This error message was included in the original PR but I removed it because I thought that just having it in the docs was enough, but I guess I was wrong ;)

Yeah. I think @jayphelps’s wording is more clear than originally was in the PR so I’d be glad to get that in.

Yep, I agree:)

Any thought on reconsidering this? I've received flak when trying to add similar functionality to linters, with their reasoning being the rules can be configured to be an error or a warning. Turning a warning into an error doesn't make a whole lot of sense.
And that logic still applies here: eslint lets you configure things to be errors or warnings. It's great that you consider unused vars to be an error. Then make it an error in the first place.

We intentionally show lint errors very obtrusively (e.g. undefined variable doesn't let you see the app at all—it's treated as a built crash). So making making more minor issues errors would be too disruptive.

Okay, how to make the build NOT to crash during build with CI set to true?


Failure Mode and Effects Analysis (FMEA) Testing

Understand the behaviour under the following disruptions:
Environment Disruptions

Disruption
Behaviour
Recovery

Kubernetes API network issues.

Artifact storage network issues.
Workflow will error.

SQL database network issues.
(a) we fail any running workflows (b) we fail to archive the workflow (c) controller will not start

Pod deleted mid execution.
Workflow will error

Node deleted "

Unable to schedule pods.

Not enough resources to schedule pods.

Workflow controller killed.
Workflows continue on restart

Extremely high number of concurrent workflows.

Set-up Disruptions(?)

Disruption
Behaviour
Recovery

RBAC changes (e.g. not able to read workflow templates).

Change configuration

Malformed resources

Questions:

How do we do network issues? Can we use a proxy?
What happens to disrupted workflows? Do they sit for stand days?

Notes:

Steady state hypothesis. Create chaos/disruption. Validate hypothesis - do we recover?
Chaos validation.

https://kubernetes.io/docs/concepts/services-networking/network-policies/
Is this supported? By K3S? EKS?


replace variables in titles

Query Variables {{p_foo}} on the Query String are great, but I would really love if the placeholders would be also replaced in query titles, visualization titles, etc.
Tnx

@runa if there will be proper UI for selecting values for variables, will you still need this?

Well, I don't really remember why I needed it in the first place; but I think it makes a lot of sense.....

I'm currently working on a UI for query parameters, and it looks like it makes this redundant. Let's revisit and see if this is still needed, after I release the version with the UI.

Hi,
that's an old one, but I want to add on this - I need the query params in the titles. My usecase: I have params coming from a query (Query based params) usually customers ids/names. I want the visualization to contain the name in the title, mainly since the current refreshing of dashboards takes different time to different panels (based on different queries obviously) and I would like the users of the dashboard to see in the title that the vis was refreshed.
WDYT?


Optional default for SystemValueHelper.

Allow an optional default value to be supplied when using the systemValue helper.
An example where this might be useful is having a safe default to proxy to a dev environment without requiring the environment variable to be set. It is also helpful to demonstrate what an appropriate value for the environment variable should be.
{
    "request": {
        "method": "ANY",
        "urlPattern": "/"
    },
    "response": {
        "proxyBaseUrl": "{{systemValue type='ENVIRONMENT' key='WIREMOCK_PROXY_BASE_URL', default='https://dev.example.com'}}",
        "transformers": [
            "response-template"
        ]
    }
}

References

https://github.com/wiremock/wiremock.org/pull/262

Submitter checklist

[x] Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
[x] The PR request is well described and justified, including the body and the references
[x] The PR title represents the desired changelog entry
[x] The repository's code style is followed (see the contributing guide)
[x] Test coverage that demonstrates that the change works as expected
[x] For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

Thanks! It has been merged

Did you see the correspond PR for wiremock.org documentation?
https://github.com/wiremock/wiremock.org/pull/262


Composite windowing fail with exception: AttributeError: 'NoneType' object has no attribute 'time'

Tried to apply a window function to the existing pipeline that was running fine. Got the following error:

Traceback (most recent call last):
  File "E:\soft\ide\PyCharm 2018.1.2\helpers\pydev\pydevd.py",
line 1664, in <module>
    main()
  File "E:\soft\ide\PyCharm 2018.1.2\helpers\pydev\pydevd.py", line
1658, in main
    globals = debugger.run(setup['file'], None, None, is_module)
  File "E:\soft\ide\PyCharm
2018.1.2\helpers\pydev\pydevd.py", line 1068, in run
    pydev_imports.execfile(file, globals, locals)
 # execute the script
  File "E:/work/source/ai-data-pipeline-research/metric_pipeline/batch_beam/batch_pipeline_main.py",
line 97, in <module>
    result.wait_until_finish()
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\direct\direct_runner.py",
line 421, in wait_until_finish
    self._executor.await_completion()
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\direct\executor.py",
line 398, in await_completion
    self._executor.await_completion()
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\direct\executor.py",
line 444, in await_completion
    six.reraise(t, v, tb)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\direct\executor.py",
line 341, in call
    finish_state)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\direct\executor.py",
line 378, in attempt_call
    evaluator.process_element(value)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\direct\transform_evaluator.py",
line 574, in process_element
    self.runner.process(element)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\common.py",
line 577, in process
    self._reraise_augmented(exn)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\common.py",
line 618, in _reraise_augmented
    six.reraise(type(new_exn), new_exn, original_traceback)
  File
"C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\common.py",
line 575, in process
    self.do_fn_invoker.invoke_process(windowed_value)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\common.py",
line 353, in invoke_process
    windowed_value, self.process_method(windowed_value.value))
  File
"C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\runners\common.py",
line 651, in process_outputs
    for result in results:
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\transforms\trigger.py",
line 942, in process_entire_key
    state, windowed_values, output_watermark):
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\transforms\trigger.py",
line 1098, in process_elements
    self.trigger_fn.on_element(value, window, context)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\transforms\trigger.py",
line 488, in on_element
    self.underlying.on_element(element, window, context)
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\transforms\trigger.py",
line 535, in on_element
    trigger.on_element(element, window, self._sub_context(context, ix))

File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\transforms\trigger.py",
line 286, in on_element
    '', TimeDomain.REAL_TIME, context.get_current_time() + self.delay)
  File
"C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\transforms\trigger.py",
line 728, in get_current_time
    return self._outer.get_current_time()
  File "C:\Users\thang\.virtualenvs\metric_pipeline-Mpf2nmv6\lib\site-packages\apache_beam\transforms\trigger.py",
line 702, in get_current_time
    return self._clock.time()
AttributeError: 'NoneType' object has
no attribute 'time' [while running 'combine all sharpe_ratio to list/CombinePerKey/GroupByKey/GroupByWindow']

the composite window function is copied from official documents:
https://beam.apache.org/documentation/programming-guide/#composite-triggers
Please refer to pipeline relevant source code below:

windowing = beam.WindowInto(FixedWindows(1 * 60),
                            trigger=Repeatedly(AfterAny(AfterCount(100),
AfterProcessingTime(delay=1 * 60))),
                            accumulation_mode=AccumulationMode.DISCARDING)
valuesPCollection
\
| 'calculate sharpe_ratio' >> beam.FlatMap(fn_calculate_sharpe_ratio) \
| 'window sharpe_ratio'
>> windowing \
| 'combine all sharpe_ratio to list' >> beam.CombineGlobally(CombineAllToListFn()).without_defaults()
\
| 'store sharpe_ratio' >> beam.FlatMap(store_metric_with_now_ts)

Imported from Jira BEAM-5132. Original Jira may contain additional context.
Reported by: thangbui.

@tvalentyn @robertwb any insights into this case where the _clock in the triggering logic is None?

I'm not sure. Presumably the runner (the old python runner in this case) should have provided it?

seems like a bug in portable runner, still reproducible at head, not reproducible in Dataflow runner.

I'll try to get some eyes an see to investigate if this looks like an easy fix.

I'm also experiencing this issue when testing locally. I haven't tried it running with any other runner other than the DirectRunner though.
test_stream = TestStream()
test_stream.add_elements(input_data)
with TestPipeline() as testpipeline:
    results = (
                testpipeline
                | test_stream
                | "Add info"
                >> beam.ParDo(MyDoFn()
            )

    (
                results
                | "Form datapoints" >> beam.ParDo(FormDataPoints())
                | "Emit datapoints on over x num elems or after y time elapsed"
                >> beam.WindowInto(
                    window.GlobalWindows(),
                    trigger=trigger.Repeatedly(
                        trigger.AfterAny(trigger.AfterCount(1), trigger.AfterProcessingTime(2))
                    ),
                    accumulation_mode=trigger.AccumulationMode.DISCARDING,
                )
                | "Group by table" >> beam.GroupBy(lambda x: x.table)
                | "print" >> Map(print)
    )

There's some good clues from the user Subhash in this jira issue: https://issues.apache.org/jira/browse/BEAM-5132 if that helps

I've just realised that I didn't have the pipeline options set to streaming:
        pipeline_options = PipelineOptions()
        pipeline_options.view_as(StandardOptions).streaming = True
        with TestPipeline(options=pipeline_options) as testpipeline:

It's now working fine

Glad you were able to find a workaround, though it is still a bug in the
sense that it should not fail with such an error even if streaming isn't
set.
On Thu, Apr 13, 2023 at 8:40 AM RobMcKiernan @.***>
wrote:

I've just realised that I didn't have the pipeline options set to
streaming:
    pipeline_options = PipelineOptions()
    pipeline_options.view_as(StandardOptions).streaming = True
    with TestPipeline(options=pipeline_options) as testpipeline:

It's now working fine
—
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/issues/19070#issuecomment-1507189914, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADWVAK6UX5M4JR3NC3TMUDXBAM5VANCNFSM5X2JJLWA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Foreign Key

Hello, Is it a bug using forms and CreateView ?
Foreign key dropdown isn't displayed in the template

It

You should follow the documentatoon and include material_js files into your page


Gemfiles: update yard to v0.9.20

To address this CVE: https://github.com/lsegal/yard/security/advisories/GHSA-xfhh-rx56-rxcr
yard it's a development dependency, so not a big deal

bors r+


Standalone plots and tables?

Is it possible to plots front-end without jupyter?
What I want to do is to create a JavaFX webview container to use beaker plots and tables without web browser.
In case it is possible, I  am quite familiar with JavaFX, but I have no experience with front end development, so I would appreciate an html template for standalone plot container and instructions about how to being together javascript part.
Of course, I will share the results if it would work out.

Yes we have an NPM package that you can load like any JS library: https://www.npmjs.com/package/beakerx
This has embedded version of the widgets that work without jupyter.
This is used by nbviewer, if you want an example.
Here is the doc: http://ipywidgets.readthedocs.io/en/stable/embedding.html

Super! Thank you very much. I will try to implement this standalone viewer thing. Probably it will be possible to run groovy scripts saved from jupyter in it.


Integrate OpenLayers with TypeScript

For several components, such as the style preview UI (see #81) and the preview map inside the StyleEditor (see #82) we need the integration of OpenLayers. Since we are using TypeScript we need some kind of TypeScript-binding for OpenLayers. Yet there is no official support for this but here can be found some ideas and infos.

I suggest using the OL Type Definitions here:
https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/ol
What do you think @KaiVolland ?

DefinitelyTyped is definitely the best source for types. :wink:


WIP: use ilpy.expressions

companion PR to https://github.com/funkelab/ilpy/pull/9
(tests will fail until that PR is merged in ilpy)

Codecov Report

Merging #30 (3fa0f65) into main (5f42466) will decrease coverage by 0.24%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
- Coverage   92.96%   92.73%   -0.24%
==========================================
  Files          29       29
  Lines         725      702      -23
==========================================
- Hits          674      651      -23
  Misses         51       51

Impacted Files
Coverage Δ

motile/constraints/constraint.py
100.00% <100.00%> (ø)

motile/constraints/max_children.py
100.00% <100.00%> (ø)

motile/constraints/max_parents.py
100.00% <100.00%> (ø)

motile/constraints/select_edge_nodes.py
100.00% <100.00%> (ø)

motile/solver.py
97.11% <100.00%> (+0.14%)
:arrow_up:

motile/variables/node_appear.py
100.00% <100.00%> (ø)

motile/variables/variable.py
66.66% <100.00%> (+3.80%)
:arrow_up:

This is now ready for review @funkey


Update CBS/CBQ transforms to use the custom KQL/CAML query option of the highlighted content web part once it's fully rolled out

We’ve also heard that some people would like the ability to add AND/OR clauses within their queries, as well as referencing custom properties they’ve applied to their content. All of this is now possible with the new custom query option as you can add your own KQL or CAML code.

Implemented


Removed Facebook link

Removed Facebook link component from the home page.
#167

Looked good to me but make sure to get 2 approvals before merging.


Minifier correctness bugs

Reproduction:  dist.zip  (comes from https://github.com/mischnic/yarn-browser)
To test: serve the folder and open index.html in the browser.
The correct (= without minification) output in the browser console is
Restoring cache...
Starting Yarn...
[YN0000]  ┌ Resolution step
[YN0000]  └ Completed in 1s 650ms
[YN0000]  ┌ Fetch step
[YN0013]  │  core-js@npm:3.22.7::__archiveUrl=https%3A%2F%2Fregistry.npmjs.org%2Fcore-js%2F-%2Fcore-js-3.22.7.tgz can't be found in the cache and will be fetched from the remote server
[YN0013]  │  js-tokens@npm:4.0.0::__archiveUrl=https%3A%2F%2Fregistry.npmjs.org%2Fjs-tokens%2F-%2Fjs-tokens-4.0.0.tgz can't be found in the cache and will be fetched from the remote server
[YN0013]  │  loose-envify@npm:1.4.0::__archiveUrl=https%3A%2F%2Fregistry.npmjs.org%2Floose-envify%2F-%2Floose-envify-1.4.0.tgz can't be found in the cache and will be fetched from the remote server
[YN0013]  │  object-assign@npm:4.1.1::__archiveUrl=https%3A%2F%2Fregistry.npmjs.org%2Fobject-assign%2F-%2Fobject-assign-4.1.1.tgz can't be found in the cache and will be fetched from the remote server
[YN0013]  │  react@npm:17.0.2::__archiveUrl=https%3A%2F%2Fregistry.npmjs.org%2Freact%2F-%2Freact-17.0.2.tgz can't be found in the cache and will be fetched from the remote server
[YN0000]  └ Completed in 2s 517ms
[YN0000]  ┌ Link step
[YN0004]  │  core-js@npm:3.22.7::__archiveUrl=https%3A%2F%2Fregistry.npmjs.org%2Fcore-js%2F-%2Fcore-js-3.22.7.tgz lists build scripts, but all build scripts have been disabled.
[YN0000]  └ Completed in 0s 968ms
[YN0000]  Done with warnings in 5s 149ms
Finished in: 5160.948974609375 ms
node_modules: (7) ['.bin', '.yarn-state.yml', 'core-js', 'js-tokens', 'loose-envify', 'object-assign', 'react']

I tried using @swc/core 1.2.194:

1st Try
This is fixed now
Using
{
  compress: true,
  mangle: true,
  toplevel: true,
  module: true,
}

Errors at runtime:
Uncaught SyntaxError: Identifier 'WarningType' has already been declared (at yarn-browser-standalone.es.min.js:131:6456)

2nd Try
Using
{
  compress: true,
  mangle: false,
  toplevel: true,
  module: true,
}

Different (harder to track down) runtime errors

-> https://github.com/swc-project/swc/issues/4841


Rework a priori antenna flow for H4C RTP

[ ] Condense down to a single ant_metrics run (since it's based on absolute cuts now, it's less sensitive to which antennas are included).
[ ] Eliminate data_inspect_maybe_good (we never look at all three notebooks)
[ ] Condense down to a single redcal_inspect notebook, eliminating either redcal_inspect_maybe_good or redcal_inspect_known_good and pass excluded antennas from auto/ant metrics to redcal

@dannyjacobs any opinions on cutting down to fewer notebooks?

After discussion on the 2/16 commissioning telecon, I'm basically free to simplify this however I think is best.

This was done in a series of commits today.


Error testing

Tests for expected errors. Works, but can be improved

No if-else, but two groups of tests, one for normal behaviour, one for exceptions.
Maybe give expected error in config.yaml for error-tests.

TODO: add a test for the last error class to satisfy the linter. ;)

TODO: add a test for the last error class to satisfy the linter. ;)

Done :)


[5.3] Changed the way after/before queue job events work

This ensures the after event is always fired, and for consistency, since we can't catch exceptions from the after event if we want to fire them in this way, I've changed the exception catching to only catch those thrown by actually executing the job.

I can't decide if we really want to fire "after" if an exception occurred. That's kind of a change in behavior. Previously, "after" meant that the job ran successfully.

Hmmm, maybe. I just see after, and think "after trying to process a job, whatever the outcome". Maybe we need another event that does actually mean that?

aw ;(


ERROR: Invalid codec guid

windows 11, compile and run old branch, when click play button, there is an error:

please help , thx

Probably AV1? Is this on a RTX 4000 series GPU?
The newer branch can handle this but sadly I haven't had time to work on it recently.

Yes it's on RTX4090, so it's codec problem?  I'll have a look at new dev branch, see if   I can do some codec compatible work.


Add move databases

Fixes #302
This operation allows to move a database instead of copy and then delete.
I still need to test this on iOS, but on android it looks like it's working.
Please let me know what else I need to change in general, as it won't change much in terms of code I believe.
Also, I wanted to add a test to the ionic angular but I think this needs the filesystem capacitor plugin and I'm not sure this dependency is OK, so I wanted to run it by you first...

Just made sure it works in iOS as well (added a small fix).
Please let me know how you would like me to proceed with the sample app.
I'm not sure github is CORS friendly but my though was around downloading a database from this repo for the test basically... I still need your approval in order to add the filesystem plugin as currently it's not used and I'm not sure what you think about it.

@HarelM you can add the filesystem plugin if you need it.

Thanks, I'll make a different PR for the sample app.
Please let me know when a new version of this plugin is available in npm with this code so I can add the relevant tests to the sample app.

@HarelM i will try to publish it in the next hours. I rely on you for the tests a i do not have any relevant code for this. For the app it would be nice if you can make it separate of the plugin. Usually it should be a test added to angular-sqlite-app-starter but someone has made some PR's on it that i did not yet merge it. So make it separate on your github share it with me and i will add a new test later in angular-sqlite-app-starter. Is that ok with you?. This will be the last release using<EMAIL_ADDRESS>after this i will make the move to<EMAIL_ADDRESS>

I'm already using capacitor 4, so feel free to have it with the capacitor 4 release.
I'm not sure I understand what you need from me in terms of testing.
I can take the fork of the updated starter and write the test there or I can use the current repo and create a test on top of that.
I will send a PR regardless, please let me know which method you prefer.

@HarelM is it working fine with the PR for capacitor4 for the test i would prefer that you add it to the angular-sqlite-app-starter. I will merge the current PR and you can start from here

@HarelM i have now updated the angular-sqlite-app-starter you can use it to implement your test

Ok, will do. Thanks for all the effort around this plugin!


Faster feedback loop for mistyped cli flags

When deploying with a vars store, we accidentally typed with only one dash, like so -vars-store. This led to some of our vms failing to become healthy, because they didn't have the variables we needed. However, this is a very slow feedback loop to realize you made a typo.
It would be really helpful to add a feature to the CLI to reduce that feedback loop.
A few different ways I could see such a feature working:

Print what flags were unrecognized and will be ignored
Check for flags which are close to the specified flag, (for instance common mispellings or the wrong number of dashes) and ask the operator if they intended to use the correct flag instead.

This would reduce the feedback loop for typos from 20+ minutes to upwards of an hour, down to just a few seconds.

Caroline && @swetharepakula

as of bosh release 263 director checks for all ((...)) before starting to deploy.
Sent from my iPhone

On Aug 29, 2017, at 4:36 PM, Caroline Taymor<EMAIL_ADDRESS>wrote:
When deploying with a vars store, I accidentally typed with only one - like so -vars-store. This led to some of our vms failing to become healthy, because they didn't have the variables we needed. However, this is a very slow feedback loop.
It would be really helpful to add a feature to the CLI to reduce that feedback loop.
A few different ways I could see such a feature working:
Print what flags were unrecognized and will be ignored
Check for flags which are close to the specified flag, (for instance common mispellings or the wrong number of dashes) and ask the operator if they intended to use the correct flag instead.
This would reduce the feedback loop for typos from 20+ minutes to upwards of an hour, down to just a few seconds.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

:+1: We're on 262. We'll roll forward.


Update

Merge updates from rstoneback/master

Did this the wrong way around :(

Coverage remained the same at 80.309% when pulling 061df7de6d429a64761cfc640648170557785d5a on aburrell:master into ac60f1a4a7a19ddff5b9bded180f9d5c4d8d4313 on rstoneback:master.


⚠ Bofrost.de has degraded performance

In d9ff79f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1323 ms

Resolved: Bofrost.de performance has improved in 24260a9 after 6 minutes.


WW-4582 Permanent patch for security issue CVE-2014-0094 adds 'class' to exclude

adds 'class' to exclude params in ParametersInterceptor (avoid ClassLoader manipulation)

This close also the CVE-2014-0112, CVE-2014-0113 and CVE-2014-0116

This isn't really needed as access to Class is blocked for any Ognl expression, see Internal Security Mechanism

Also based on a report from a user I have added this 74e26830d2849a84729b33497f729e0f033dc147 - this is almost the same as yours but it bases on a real-life example :)

ok, PR closed


Requirements

[X] I have updated Mushroom to the latest available version
[X] I did a search to see if there is a similar issue or if a pull request is open.

Current Behavior
Part of my code:

type: custom:mushroom-entity-card
name: Movie Scene
icon: mdi:movie-outline
icon_color: purple
secondary_info: none
tap_action:
action: call-service
service: scene.turn_on

Expected Behavior
Icon should have a purple color but stay's grey
Steps To Reproduce
No response
Context
YAML state

Environment
- Browser:
- HA Version: 2023.4.4
- Mushroom: 2.7.0

Anything else?
No response


Invalid argument supplied for foreach()

PHP Warning:  Invalid argument supplied for foreach() in ./vendor/cmfcmf/openweathermap-php-api/Cmfcmf/OpenWeatherMap/WeatherForecast.php on line 84

Hey @nexus79000, can you please share which error this fixes? It looks like this makes sure the code doesn't break if $xml isn't an object.
Has this been the case for you? Could you share a code example demonstrating the wrong behavior?

Closed due to lack of response.


Add "interpolate" command to operator

Is your feature request related to a problem? Please describe.
We need to write tests for the template logic in kubecf: https://github.com/cloudfoundry-incubator/kubecf/issues/1306
For this we need to render the helm template, and then apply all generated ops files against the deployment manifest. Once we have the final manifest, we can verify that it contains the expected settings (and has removed the ones we expect to be removed).
We have some code that uses bosh interpolate in an unrelated script, that we could use for now, but that does not support the additional qcopy and qmove ops that are specific to quarks-operator.
Describe the solution you'd like
I would like the have an interpolate subcommand in the operator, and quarks releases to include operator executables compiled for both Linux and macOS that can be downloaded from the Github releases pages.
Describe alternatives you've considered
@mook-as has written his personal tool that essentially does the same: https://github.com/mook-as/go-patch-cmd
We could use that, but would prefer to use the operator binary to make sure we are using the exact same code that will be used to generate the manifest at runtime.

I've just had some ideas how we can manipulate the manifest directly from the kubecf helm chart instead of using qmove/qcopy ops. If that works out, then we can continue to use bosh interpolate and won't need this feature from the operator.

This is no longer required; kubecf will not make use of the qcopy/qmove ops but manipulate the deployment manifest directly.
This also means that the changes from #913 could be reverted, but that would probably be just busy-work without providing any value.


fix: added support for "missing" matcher

Summary
This update allows us to support the "missing" matcher key.

Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal
fixes #1796
Standard checks:

[ ] Check the Deploy Preview's Demo site for your PR's functionality
[ ] Add docs when necessary

🧪 Once merged, make sure to update the version if needed and that it was published correctly.

Nice! Can you re-enable these tests: https://github.com/netlify/next-runtime/blob/mk/ef-null-check/test/e2e/modified-tests/middleware-custom-matchers/test/index.test.ts#L24

The tests are failing because you need to run prettier: https://github.com/netlify/next-runtime/actions/runs/4001502021/jobs/6867794232

Looks good! Is it possible to add an example of this new matcher working in one of our demos?


Fix how gameobjects get destroyed in play vs edit modes

Overview
Some of the pointer components have [ExecuteAlways] in order to perform validation OnEnable in the editor. This is excessive as these validation will be thrown at runtime still.
Also, many areas destroy gameobjects both in play mode and in edit mode. However, one must use DestroyImmediate in edit mode. Added function to GameObjectExtensions to simplify calling for all parties
Changes

Fixes: #5763

Verification
All tests pass

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run


Create an Unity Package of this library

Hi guys,
I was wondering if you can include a .unitypackage file from the releases of this project with the source code and the README.
Let me know and thanks in advance,
Juan

I've created a package of this library on OpenUPM so it can be added easily via the package manager. This is preferable to .unitypackage files because the Unity Package Manager can keep track of versions, package files are stored separately to your Assets folder so don't get mixed up with your own assets, you can remove the package without having to worry about missing any files, and when upgrading the package, files that are no longer needed are deleted automatically.

Ohh very interesting, thanks!! 👍

If I want to upload the RestClient, what's the package.json to create the package?

Thanks in advance!

You'll need to create one - see the Unity docs for details.
I'd also recommend creating your package out as an embedded package and then once you have a Git repo with just the package in it trying out installing it via the Git URL to make sure that it works before adding to OpenUPM.


Add Blazor templates for ASP.NET Core 2.2

Visual Studio doesn't show "Blazor" project template:

Blazor Language Services extension is installed:

Configuration:

VS 2017 15.9.0 Preview 2
.NET Core 2.2.100-preview2-009404
SDK: 2.2.100-preview2-009404
OS: Windows 10 Home x64

Is Blazor support .Net Core 2.2?

In Blazor 0.8.0, we have added .NET Core 3.0 support. As such, we're skipping 2.2.


Cannot run live server in either browser

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[ ] Feature request
[ ] Documentation issue or request
[ ] Other: <!-- Please describe: -->

Current behavior

Installed addon, rightclick html and ran live server. Firefox/chrome gets "Page not working, invalid response. ERR_INVALID_HTTP_RESPONSE". I tried changing the settings to load in both firefox and chrome
Expected behavior

HTML page should be displayed, works fine if I click my html page manually
Environment
Browser:

- [x ] Chrome (desktop) version 69.0.3407
- [x ] Firefox version 62.0.3
- [ ] Safari (desktop) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:

- Live Server: <!-- Check which version is installed -->
- Platform:  <!-- Mac, Linux, Windows -->
- Visual Studio Code: <!-- `code --version` --> 5.1.1

Others

I'm also experiencing the same problem and decided to do some testing across a few setups to see if it was consistent across all stacks.
I've isolated the problem down to MacOS Mojave as the culprit. I upgraded to Mojave from High Sierra, so I'm not sure if the upgrade causes the liver server auto-reload problem, meaning it maybe okay on a fresh install of Mojave.
Here's what I tested…

Mojave - VS Code(1.28.1) - Chrome (69.0.3497...) - Does NOT Live Reload
Mojave - VS Code(1.28.1) - Safari (12.0.14606...) - Does NOT Live Reload
High Sierra - VS Code(1.28.1) - Safari (12.0.13606...) - Live Reload Works!
Windows 10 - VS Code(1.28.1) - Chrome - Live Reload Works!

Not sure why Mojave would be blocking live reload, but it seems to be doing something.


Exception: ' Received an unexpected EOF or 0 bytes from the transport stream.'

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ X] bug report -> please search issues before submitting

Minimal steps to reproduce

Fill in the environment variables; create a .net7 console app; Use the three lines of sample code; compile successfully; run.

Any log messages given by the failure

MQTTnet.Exceptions.MqttCommunicationException: ' Received an unexpected EOF or 0 bytes from the transport stream.'

ClientConsoleApp Warning: 0 : EnvFile Not found in path C:\git\MqttApplicationSamples\mqttclients\dotnet\ClientConsoleApp\bin\Debug\net7.0 .env
ClientConsoleApp Information: 0 : Loaded Cert: CN=client10-authn-ID XYZ issued by CN=MqttBlogAppSamplesCA Intermediate CA, O=MqttBlogAppSamplesCA, not after: 26/01/2024 11:37:12
ClientConsoleApp Information: 0 : >> [1]: Trying to connect with server 'iotgrid-test-egns.westeurope-1.eventgrid.azure.net:8883'
ClientConsoleApp Information: 0 : >> [8]: Error while connecting with server
MQTTnet.Exceptions.MqttCommunicationException:  Received an unexpected EOF or 0 bytes from the transport stream.
---> System.IO.IOException:  Received an unexpected EOF or 0 bytes from the transport stream.
at System.Net.Security.SslStream.ReceiveBlobAsync[TIOAdapter](CancellationToken cancellationToken)
at System.Net.Security.SslStream.ForceAuthenticationAsync[TIOAdapter](Boolean receiveFirst, Byte[] reAuthenticationData, CancellationToken cancellationToken)
at MQTTnet.Implementations.MqttTcpChannel.ConnectAsync(CancellationToken cancellationToken)
at MQTTnet.Implementations.MqttTcpChannel.ConnectAsync(CancellationToken cancellationToken)
at MQTTnet.Adapter.MqttChannelAdapter.ConnectAsync(CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at MQTTnet.Adapter.MqttChannelAdapter.WrapAndThrowException(Exception exception)
at MQTTnet.Adapter.MqttChannelAdapter.ConnectAsync(CancellationToken cancellationToken)
at MQTTnet.Client.MqttClient.ConnectInternal(IMqttChannelAdapter channelAdapter, CancellationToken cancellationToken)
at MQTTnet.Client.MqttClient.ConnectAsync(MqttClientOptions options, CancellationToken cancellationToken)
ClientConsoleApp Information: 0 : >> [8]: Disconnecting [Timeout=00:01:40]
ClientConsoleApp Information: 0 : >> [8]: Disconnected from adapter.
ClientConsoleApp Information: 0 : >> [8]: Disconnected.

Expected/desired behavior

connected client

Language

C#

OS and Version?

Windows

Mention any other details that might be useful

Device registration and cert works for MQTTX (tested separately, not running at the same time).

Thanks! We'll be in touch soon.

Seems like your app is not loading the .env file, hence not finding the client certs. I'm adding a note to produce a better error message for this case.
Can you try with the getting_started scenario? make sure you follow the exact instructions, eg:you are running the app from the same folder with the .env file, and the pem/key files.
If you are running in VS, you can also specify the envvars from the launchSettings.json, or alternatively you can pass the .env path to MqttConnectionSettings.CreateFromEnvVars()

I tried to connect to your endpoint, and found the endpoint has not TLS enabled.
Make sure that:

MQTT is enabled in your EventGrid Namespace
You are using the MQTT endpoint az resource show --ids $res_id --query "properties.topicSpacesConfiguration.hostname"

@Hi @sandervandevelde  did you check if the endpoint was the right one? an easy way to check it supports TLS is by using openssl s_client -connect

@sandervandevelde any updates?

Hello @rido-min ,
I tried a number of things but I did not have any success.
I landed here because of the samples link seen on the EventGrid MQTT support docs.
The MQTT clients section seems quite ok regarding the documentation of the environment variables.
I skipped the getting started because it seems to be focused on some local Mosquitto server? I have to perform some 'cat' to create a certificate string for Mosquitto (is this 'cat' working in Powershell)? But it's also mentioning the Eventgrid?
All I want is a sample in C# of an MQTT client capable of connecting to the 'IoTGrid' in 20-30 lines, while sending some messages to a topic. This should be on par with the setting I need to set in the MQTTX example.
The MQTTnet extensions seen here look pretty well designed but it is overwhelming.

@sandervandevelde
Seems to me you are trying to connect to the wrong Event Grid endpoint. Each event grid instance has two endpoints, one for HTTP as <instance.region>-1.eventgrid.azure.net and another for MQTT as <instance.region>-ts.eventgrid.azure.net.
Can you double check you are using the same endpoint as in MQTTX ?
All the samples in this repo are designed to work with different brokers, and adapting to different authentication options.
Since EventGrid requires authentication with client certs, we added instructions to create certs using step, cat is just a tool to readfile the file contents and redirect to an output. Both tools work with powershell, however, and due to the C samples requirements, we have prioritized Linux/WSL over Windows.

The MQTTnet extensions seen here look pretty well designed but it is overwhelming.

curious to understand this comment, what is overwhelming? and how can we do it better?

Hello @rido-min
Success. The client can be connected. I was able to send messages using the extension library.
I found a typo (extra character) in my MQTT URL. I overlooked it twice :-( ).
Regarding that library,  it feels quite extensive:

It seems to do everything: connecting, requests, responses, commands, and even alerts.
It's more than 700 lines long.
I prefer having separate (little) applications demonstrating one task (one problem, one solution).
But maybe I'm taking MQTT too easy seeing all the serializers, X509 tooling, producers, and consumers.
Thanks for guiding me in the right direction.

yeah.. we are trying to find a balance to keep the scenarios very focused. We could limit the extensions to "connection only kind of helpers", ands move the Producer/Consumer/Commands to each sample.
The intent here was to allow customers to create new Telemetries, Alerts, or Commands apps reusing these extensions.
We have plans to productize these ideas in a former NuGet package, this is just the starting point.
Can we close this issue now?

Thanks for the extra explanation. Closed.


How to add arguments

i want to use https://docs.docker.com/network/host/ but how do i make dockerode do that? Where can i add the argument?

https://docs.docker.com/engine/api/v1.41/#tag/Network this is what you should be reading :)


CRFrnn_layer batch size

Hi,
Your implementation of the crf layer supports only batch_size = 1.
Correct?

That is correct. This is now mentioned under "Limitations" in the README. I'll leave this issue open so that I'll remember to fix it. If you have a patch to fix this, you're more than welcome to submit a pull request.

thx for your excellent work! Now I have a tensorflow framework.So can I use the crfrnnlayer as a layer behind the layers from tensorflow such as slim?

Has the batch_size issue been fixed?

Did anyone fix this?


CI: use concurrency

What does this PR do?
partial of #678
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

seems to be working as expected


Adds support to inject telemetry instances to plugins

Description
This change will make available the telemetry instances such as tracer and metrics registry to all the plugins.
Related Issues
Resolves #13635

Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] API changes companion pull request created.
[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

❌ Gradle check result for f58cd00: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/13476
https://github.com/opensearch-project/OpenSearch/issues/10006
https://github.com/opensearch-project/OpenSearch/issues/13600


Error: bundle mysql not available locally or in /home/xxx/.porter

I have,

used porter to create the mysql bundle.json
duffle to install it

When I try to build the wordpress porter.yaml which takes a dependency on mysql, it gives the error:
Error: bundle mysql not available locally or in /home/xxx/.porter
I assume I have to somehow 'install' the mysql to /home/xxx/.porter to get the wordpress porter.yaml building. How can I do that?

@isurulucky I believe you are hitting the issue fixed (or proposed to fix) in https://github.com/deislabs/porter/pull/104
I didn't explicitly mention the issue fixed in that PR (boils down to the Name property fix in pkg/config/manifest.go), but, currently, the bundle.json generated by porter build has an empty Name field.  Thus, if you were to list local bundles after the porter build, you would see an entry with no name, like so:
 $ duffle bundle list
NAME                        	VERSION                                 	DIGEST                                  	SIGNED?
                            	0.1.0                                   	9a404ed5bc50950d15ffd99daeb4222109e75eb5	true

This would be the bundle just built -- and since it is looking for the name you gave it, but which was dropped due to the bug, duffle/porter can't find the local bundle.
That's all assuming you are hitting what is described above.  Would it be possible to confirm via a duffle bundle list?  (And then, if so, perhaps taking a moment of your time to test the same workflow off of the fix PR?)  Thanks!

Thanks @vdice. Will try this and get back.


Make Filter work on most Attributes

Filter should filter not only by tag, but by most other attributes as well. The type of attribute to filter could be specified using the prefixes.

Can close for now


Cooks distance branch

Cook's distance outlier detector
A supervised regression outlier detector
Cook's distance can be used to identify points that negatively affect a regression model. A combination of each observation’s leverage and residual values are used in the measurement. Higher leverage and residuals relate to  higher Cook’s distances. Read more in the :cite:cook1977outlier --> https://www.jstor.org/stable/1268249
The script cd.py has been added to pyod/models/ containing the Cook's distance outlier detector. The code is mostly based off what has been implemented in the Yellowbrick repo but thought it would be nice to be able to call it with all the others outlier detectors in Pyod.
Hopefully this will become an useful addition to the already great repo and python package that Pyod is.

Thanks for the PR, could you do a few things to make this a bit clear?

add an simple example file as we have under /examples/knn_example.py
if possible, add a unit test like this one very simple indeed https://github.com/yzhao062/pyod/blob/master/pyod/test/test_knn.py

I will take from there~btw, please submit the PR to the development branch.
Thanks again.
Yue

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 59 (0.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.9%) to 95.066%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/models/cd.py
0
59
0.0%

Files with Coverage Reduction
New Missed Lines
%

pyod/models/lof.py
5
87.18%

Totals

Change from base Build<PHONE_NUMBER>:
-0.9%

Covered Lines:
6609

Relevant Lines:
6952

💛  - Coveralls

I think the code looks great. One last thing is the PR should be submitted to the development branch. Please resubmit in that way. Other than that, things look cool. Thanks a lot.

Excellent! Glad that this outlier detector can soon become another addition to this great package. Hopefully it has changed the PR to the Development branch

perfect. I will do a merge and some local edits. Thanks!


Hide token on missing token address

What does this PR do and why?
Closes https://github.com/jbx-protocol/juice-interface/issues/852
Screenshots
Freshly minted project (before token issuance)

Freshly minted project (after token issuance)

Existing project with no token (Juicebox DAO on testnet rinkeby)

Acceptance checklist

[x] I have evaluated the Approval Guidelines for this PR.
[x] I have tested this PR in all supported browsers.

Nice on @wraeth-eth. Given this a good test for v1 and v2 and works perfectly. Praise for getting it done for V2 as well 🙏


kdeFrameworks: 5.55 -> 5.56

Motivation for this change
https://kde.org/announcements/kde-frameworks-5.56.0.php
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

You are too fast :D


Add search functionality based on @antora/lunr-extension

CSS and JS are copied from node_modules, I followed instructions from here https://gitlab.com/antora/antora-lunr-extension
The index itself is around 600+KB so it doesnt add too much to the site.
This is how it looks like from local:

Nice. Is that supposed to work with mill docs.antora.localPages? I see the search box, but typing anything in it does not reveal any result or result box.

@lefou yes, that screenshot is from my local setup. Not sure why it doesnt work for you, any errors in browser console? Maybe try clean build again.

Nothing in console. But I noticed there is no reference to any docsearch script in the page source. Probably an issue in the build, not the browser.

Could it be my system is picking another version of antora which doesn't not include the search-scripts.js by default? I added it manually, and then it worked for me, too.
Shall I open a PR to your branch?

Let's merge this as-is. If it not works out-of-the-box, as it does for you, fine. If not, I can push my fixes afterwards.

Thank you!

 Seems it works fine on the live site. 🥳


Update proof.computer's polkadot.json slot

Swap out futureproof for weatherproof as it is now independent.

@ironoa Verified with the participant on Element


🏗 Update ubuntu version on CI workflow

We need to use ubuntu 18.04 on the GitHub Actions CI. When we update currently the tests fail. Since 18.04 is technically deprecated, we should try upgrading as soon as possible.

This no longer applies since the real issue was node version, not ubuntu version. We are back on ubuntu-latest.


Add: most_common_word

Codecov Report

Merging #118 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #118   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          51     52    +1
  Lines         576    596   +20
=====================================
+ Hits          576    596   +20

Impacted Files
Coverage Δ

problems/most-common-word/most_common_word.go
100% <100%> (ø)

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 54579c9...028ffc1. Read the comment docs.


Auto-detect the source code root if no paths are passed

Currently, if you pass no paths, the CLI fails, that's not great. Make it detect the most optimal place to run and notify that this is the folder that is used as the root

One issue with this is that this detection is quite architecture-specific, so we can't bake the FSD root detection algorithm into the Steiger core. Our choices then are:

to simplify detection, like finding a src or app folder
to make plugins provide their own routine for optimal start location (this kind of effort is hard to justify)


[Snyk] Upgrade @hashgraph/sdk from 2.41.0 to 2.42.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @hashgraph/sdk from 2.41.0 to 2.42.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 5 versions ahead of your current version.
The recommended version was released 22 days ago, on 2024-02-29.

Release notes

Package name: @hashgraph/sdk

2.42.0 - 2024-02-29What's Changed

feature: handling and externalisation improvements for account nonce updates by @ svetoslav-nikol0v in #2176

Full Changelog: v2.41.0...v2.42.0

2.42.0-beta.4 - 2024-02-23What's Changed

chore(ci): fix permissions preventing Github release creation by @ nathanklick in #2149
fix: use one of preselected nodes to execute the transaction by @ svetoslav-nikol0v in #2165

Full Changelog: v2.42.0-beta.3...v2.42.0-beta.4

2.42.0-beta.3 - 2024-02-20What's Changed

chore: ensure CI release workflow properly handles prerelease builds by @ nathanklick in #2142
fix: update copy of transaction in transaction list by @ svetoslav-nikol0v in #2139
fix: deserialize the FileAppendTransaction by @ svetoslav-nikol0v in #2145

Full Changelog: v2.42.0-beta.2...v2.42.0-beta.3

2.42.0-beta.2 - 2024-02-14What's Changed

feature: serialize and deserialize an incompleted transaction by @ svetoslav-nikol0v in #2067

Full Changelog: v2.41.0...v2.42.0-beta.2

2.42.0-beta.1 - 2024-02-12Beta release version v2.42.0-beta.1

2.41.0 - 2024-02-05What's Changed

[#2102] Cannot read properties of undefined (reading 'accountId') by @ svetoslav-nikol0v in #2103
Export Long from SDK by @ svetoslav-nikol0v in #2107

Full Changelog: v2.40.0...v2.41.0

    from @hashgraph/sdk GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.15%.  Consider uploading reports for the commit c73ad10 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2238   +/-   ##
=======================================
  Coverage   75.15%   75.15%
=======================================
  Files          13       13
  Lines         644      644
  Branches      118      118
=======================================
  Hits          484      484
  Misses        115      115
  Partials       45       45

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


Refactor memoization out, so this block is easier to comprehend

Found this one while focused on blockers (#12501).
@miq-bot add_label core, refactoring, euwe/no
@miq-bot assign @imtayadeway

:+1: Nice horizontal :scissors:
@miq-bot assign @gtanzillo

@isimluk Are there specs around this?


I'm Looking for a Code Review of My Functional Component Implementation for Math Magicians

I would like to request a code review of my implementation for a basic calculator app using functional components. Although the implementation is already completed, I am seeking feedback and suggestions for improvement.
Here is the link https://github.com/kazaneza/calculator/tree/events
If possible, I have a few specific questions:

[ ] Is my use of functional components efficient and suitable for the task?
[ ] Have I correctly utilized state and props?

I would be grateful for any assistance you can provide. Thank you.

Required Changes :recycle:

Hey @amexabee :wave:
Highlights :1st_place_medal:
heavy_check_mark Descriptive Issue Title
heavy_check_mark Following Requirements
heavy_check_mark Following Functional Programming
Good job on implementing some of the requirements, there are some issues that you still need to work on to go to the next project but you are almost there! :blush:
Optional
All the changes with optional tags are not crucial enough to prevent you from getting the approval but we highly recommend to students that they should take these changes into consideration.
Cheers and Happy coding! :clap: :clap: :clap:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
Changes Required

[ ] It would be nice if you talk about how we can use functional components in react and what changes we need to make when we use functional components instead of class components :blush: You are supposed to describe in this issue how you use functional components Kindly address these following questions in your issue description to follow the requirements :+1:

How we can use functional components in react?
What are the changes we face when we use functional components instead of class components?

Hint: You can see this project to get an idea how to answer these questions but kindly use your own words becuase i know you are smart enough :muscle:
Note: I know you have asked questions and there is no harm in asking questions and I am really interested in giving discussing with you my dear brother :blush: We can discuss your questions in the slack if you don't mind :+1:


How about basic auth for http client

It might be useful to have a helper to set basic auth headers, would you accept PR?

Yes PRs are always welcome.


Namespaced Registries

Similar to the way a GraphiteReporter can specify a prefix, it's be nice to be able to pass around Registries that enforce a particular namespace/prefix.
I'm imagining you could say something like:
MetricRegistry shared = SharedMetricRegistries.getOCreate("foo");
trackUsageStats(shared.withPrefix("usage"));
trackReportingStats(shared.withPrefix("reporting"));

Namespaced registries would ideally be MetricRegistry objects themselves, but map to another registry under the hood, such that all interactions apply the designated prefix. So these would be equivalent:
shared.timer("usage.abc")
shared.withPrefix("usage").timer("abc")

Looks like something similar was discussed in Issue #842.

Yea this looks a lot like #842, I'll track it there. Feel free to register your interest in a comment on that ticket.


Sticker list renders empty button as first element

Nevermind, this is just because my ad blocker blocks the first image


adding header menu and style + flaticon

I would like to suggest we can give a name to this project we can simply call it: Clonello! Let me know if you like the idea :)


Fix typo in README

hmm...
I cannot see your commit in this PR

I am unable to push commits because of permission issues. I think there is a typo in the README (settigns -> settings). You can delete my pull request and make the changes yourself.


Integrate SAM 2.1

Linked Issues:

#8231

Linked Pull Requests:

#8243

Summary
To integrate SAM 2.1 into CVAT, I

forked cvat,
created branch feature/sam2,
merged forked repo of @jeanchristopheruel and merged branch develop into feature/sam2,
updated SAM 2.0 to SAM 2.1,
converted decoder part to ONNX using script export_sam21_cvat.py,
split code into backend and frontend,
added CLIENT_PLUGINS argument to pass plugin cvat-ui/plugins/sam2,
and tested the model successfully within CVAT on CPU and GPU.

Motivation and context
This pull request builds upon a #8243 that aimed to integrate SAM 2 into CVAT. The progress on that contribution has been stalled, and this request serves as a continuation of integrating SAM 2.
Main enhancements:

Update to SAM 2.1
Backend encodes image
Decoder converted to ONNX with postprocessing steps
Frontend does the decoding

This way the structure of integrating SAM is maintained in SAM 2 with minimal changes.
How has this been tested?
Using the following commands:
CLIENT_PLUGINS=plugins/sam2 CVAT_HOST=localhost CVAT_VERSION=v2.21.2 docker compose -f docker-compose.yml -f docker-compose.dev.yml -f components/serverless/docker-compose.serverless.yml -p cvat up -d --build

# on cpu
./serverless/deploy_cpu.sh serverless/pytorch/facebookresearch/sam2

# on gpu
./serverless/deploy_gpu.sh serverless/pytorch/facebookresearch/sam2

, and applying the model on several images.
Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit
Release Notes

New Features

Enhanced documentation for the Computer Vision Annotation Tool (CVAT) with updated sections on automatic labeling algorithms and user support channels.
Introduced a new plugin for the Segment Anything Model 2.1 (SAM2) to facilitate interactive segmentation tasks.
Added serverless function configurations for interactive object segmentation using SAM2.

Bug Fixes

Corrected formatting issues in the documentation.

Chores

Improved configurability of the CVAT UI service with new build arguments.

Very nice work @hashJoe !  I think the next big milestone would be to integrate encoder in frontend to unlock decentralized tracking capabilities for video annotation (this is mostly all SAM2 is about)!

Very nice work @hashJoe ! I think the next big milestone would be to integrate encoder in frontend to unlock decentralized tracking capabilities for video annotation (this is mostly all SAM2 is about)!

I need to look into the video prediction and check how to implement it in CVAT. How would integrating the encoder in frontend help? There is a tracking model example which is done in backend (both encoder and decoder), here. Any insights would be helpful!

@hashJoe I beleve Sam2 encoder is lightweight enough to be supported by the frontend (I think it is 1Gb or so). Porting it to the frontend would remove the need of an inference backend server. Also, it would reduce latency associated to the request containing the state and returning the embeddings. Video annotation faster than ever. Each user have its own internal tracking state, which is a memory embedding in Sam2.

I'm following this PR

@jeanchristopheruel I need to look into the video prediction and check how to implement it in CVAT. How would integrating the encoder in frontend help? There is a tracking model example which is done in backend (both encoder and decoder), here. Any insights would be helpful!

SAM 2 for video tracking would be really transforming our annotation workflow (fish monitoring)! I hope this is possible to implement in CVAT.


Specifying timezone using the TZ environment variable does not work

Using the suggested environment variable, TZ to specify the timezone for Supercronic to use yields no change.
Using TZ=Europe/Stockholm, for example, should yield 2020-07-17T07:00:00Z as the next event for the cron 0 7 * * *.
Example Dockerfile to replicate:
FROM alpine:3

ENV SUPERCRONIC_URL=https://github.com/aptible/supercronic/releases/download/v0.1.9/supercronic-linux-amd64 \
    SUPERCRONIC=supercronic-linux-amd64 \
    SUPERCRONIC_SHA1SUM=5ddf8ea26b56d4a7ff6faecdd8966610d5cb9d85

RUN apk add --update --no-cache curl && \
  curl -fsSLO "$SUPERCRONIC_URL" && \
  echo "$SUPERCRONIC_SHA1SUM  $SUPERCRONIC" | sha1sum -c - && \
  chmod +x "$SUPERCRONIC" && \
  mv "$SUPERCRONIC" "/usr/local/bin/$SUPERCRONIC" && \
  ln -s "/usr/local/bin/$SUPERCRONIC" /usr/local/bin/supercronic

RUN echo "0 7 * * * echo 'this won't work'" > /tmp/cron

ENV TZ=Europe/Stockholm

ENTRYPOINT ["supercronic"]
CMD ["-debug", "/tmp/cron"]

Output:
time="2020-07-16T20:35:07Z" level=info msg="read crontab: /tmp/cron"
time="2020-07-16T20:35:07Z" level=debug msg="try parse(7): 0 7 * * * echo 'this won't work'[0:20] = 0 7 * * * echo 'this"
time="2020-07-16T20:35:07Z" level=debug msg="try parse(6): 0 7 * * * echo 'this won't work'[0:14] = 0 7 * * * echo"
time="2020-07-16T20:35:07Z" level=debug msg="try parse(5): 0 7 * * * echo 'this won't work'[0:9] = 0 7 * * *"
time="2020-07-16T20:35:07Z" level=debug msg="job will run next at 2020-07-17 07:00:00 +0000 UTC" job.command="echo 'this won't work'" job.position=0 job.schedule="0 7 * * *"
^Ctime="2020-07-16T20:35:12Z" level=info msg="received interrupt, shutting down"
time="2020-07-16T20:35:12Z" level=info msg="waiting for jobs to finish"
time="2020-07-16T20:35:12Z" level=debug msg="shutting down" job.command="echo 'this won't work'" job.position=0 job.schedule="0 7 * * *"
time="2020-07-16T20:35:12Z" level=info msg=exiting

Is this incorrect behavior (a bug) or is the documentation misleading on how to specify the timezone to use?

Your container needs to include tzdata for this to work, otherwise Supercronic has no idea what Europe/Stockholm is. Alpine doesn't include this by default I think, so it's missing here. Add this to your Dockerfile:
RUN apk add --update --no-cache tzdata

This should create /usr/share/zoneinfo/, which should include /usr/share/zoneinfo/Europe/Stockholm.

Great, that works! Closing. Also, see #72.


How to use as a library?

How to use as a library to use python to remove the background from a given image path.

@BanKeTang

There is currently a circular import between bg.py and utilities.py:
(ins)>>> from backgroundremover.bg import remove
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.11/site-packages/backgroundremover/bg.py", line 15, in <module>
    from .u2net import detect, u2net
  File "/usr/lib/python3.11/site-packages/backgroundremover/u2net/detect.py", line 11, in <module>
    from .. import utilities
  File "/usr/lib/python3.11/site-packages/backgroundremover/utilities.py", line 9, in <module>
    from .bg import DEVICE, Net, iter_frames, remove_many
ImportError: cannot import name 'DEVICE' from partially initialized module 'backgroundremover.bg' (most likely due to a circular import) (/usr/lib/python3.11/site-packages/backgroundremover/bg.py)

It is possible to import from the cli module, but this should not be necessary.
(ins)>>> from backgroundremover.cmd.cli import remove
(ins)>>> remove
<function remove at 0x7f27dd192ca0>

I ran into the same problem. I created a pull request with the fix here: https://github.com/nadermx/backgroundremover/pull/138

Fixed in https://github.com/nadermx/backgroundremover/commit/123d4445808c25521d06611e6e7e3aa9bb426b2f read ReadMe to on how to.  Tx @ahmad88me


user_groups: Remove unused remove_user_from_user_group.

remove_user_from_user_group's only caller has been removed in 271333301d3ca662ee5c1497fa3fa9829aaf93d1.
Its usage has been superseded by remove_members_from_user_group.

Screenshots and screen captures:
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Nice, merged, thanks @PIG208! @sahil839 FYI.


Android 6 compability

I am using my old tablet as HA dashboard, so i'd like to have the bluetooth proxy on it. Is it possible to lower minSdk to 22-23?

Ok, tried it by myself. Was used to remove txpower info, everything else looks fine on SDK version 23.

@voed I would need this for Android 7, never really compiled an Android app. I have a Linux machine and windows of course, could you give me a hand maybe how to do this/convert to lower SDK? Or share the apk if that is reasonable ...

Sure, PM me in telegram @voed1337


@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/mne-nirs-feedstock#2.


mobile: Remove manual DeleteLocalRef calls

This PR replaces manual calls to DeleteLocalRef with RAII-style deletion.
Risk Level: low
Testing: unit tests
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/assign @abeyad


DatatokenEnterprise.create_exchange(..allowed_swapper=X..) doesn't set allowed_swapper to X

The bug / how to reproduce
# Create OCEAN
OCEAN = ocean.ocean_token

# Create a DT, with template = 2
from tests.resources.helper_functions import deploy_erc721_erc20
_, DT = deploy_erc721_erc20(config, publisher_wallet, publisher_wallet, 2)

# create exchange.
(exchange, _) = DT.create_exchange(
        rate=int(1.0*10**18),
        base_token_addr=OCEAN.address,
        tx_dict={"from": publisher_wallet},
        allowed_swapper: str = ZERO_ADDRESS,
    )

# then, test. **This will fail**
assert exchange.get_allowed_swapper() == ZERO_ADDRESS

Notes
datatoken_enterprise.py::DatatokenEnterprise doesn't implement create_exchange() itself. It inherits that from datatoken.py::Datatoken.
datatoken.py::Datatoken calls self.contract.createFixedRate(), including the allowed_swapper argument. So, it should set it. But it doesn't, as we see above.
How to fix
For this issue, let's just do a workaround in ocean.py: in datatoken.py::Datatoken::create_exchange().

Specifically, after the main tx to create the exchange, check: does allowed_swapper argument == exchange.get_allowed_swapper() ?
If no, then set it via 2exchange.set_allowed_swapper(allowed_swapper)`

Alright. I just set this up as new tests in test_datatoken_enterprise.py.
AND, it didn't fail. So I must have been doing something wrong when I first encountered this.
But that's good news! There's nothing to do for this ticket.
For reference, here's the test code that I added. (Which passed)

@pytest.mark.unit
def test_exchange_swapper_issue1205_1(config, publisher_wallet):
    _, DT = deploy_erc721_erc20(config, publisher_wallet, publisher_wallet, 2)
    USDC = Datatoken(config, get_address_of_type(config, "MockUSDC"))

    # allowed_swapper should default to ZERO_ADDRESS
    (exchange, _) = DT.create_exchange(
        rate=to_wei(1),
        base_token_addr=USDC.address,
        tx_dict={"from": publisher_wallet},
        publish_market_fee=to_wei(0.1),
        with_mint=True,
    )
    exchange.get_allowed_swapper().lower() == ZERO_ADDRESS.lower()

@pytest.mark.unit
def test_exchange_swapper_issue1205_2(config,
    publisher_wallet,
    consumer_wallet,
):
    _, DT = deploy_erc721_erc20(config, publisher_wallet, publisher_wallet, 2)
    USDC = Datatoken(config, get_address_of_type(config, "MockUSDC"))

    # if we set allowed_swapper, it should take that value
    (exchange, _) = DT.create_exchange(
        rate=to_wei(1),
        base_token_addr=USDC.address,
        tx_dict={"from": publisher_wallet},
        publish_market_fee=to_wei(0.1),
        with_mint=True,
        allowed_swapper = consumer_wallet.address,
    )
    exchange.get_allowed_swapper().lower() == consumer_wallet.address.lower()


[Samples][Java] Fix disparities of 25.message-reaction sample

Related to microsoft/botbuilder-java1165
Proposed Changes
We compared the documentation/code migration/behavior of the 25.message-reaction sample between Java and C# and we found disparities and fixes that this PR includes.

Update README
Clean-up autogenerated files/folder related to Eclipse (e.g. target/.classpath)

Testing
Message Reaction sample working as expected

Merged in MS


Update README.md

@rdubya thanks for merging this.
When you have a PR which has a user who has made several commits to fix one issue, do a squash merge to keep master history tidy. Use your best judgement, but don't be afraid to use it when it makes sense.

@ioquatix Sorry about that, I'll keep that in mind for future PRs.


Migration to 2

When following the migration guide and using the same helper function, get the error:
Obj is missing complied validation method

a nested schema was decoated 🤦♂


Docker Compose with multiple configuration files

I am willing to work on a PR to use this built in Docker support to have multiple compose files as we can read here.
One of the best goals of using Docker in Production is to have 100% parity between development, test, staging and production environments.
But during development we may need some tools in top of the Production environment to help us with debug and workflow.
So to achieve 100% parity between environments and at same time have the set of tools we want for development, we must have the docker-compose.yml as the base file to be used in Production and extend from their to add the tools for Development workflow, but without modifying the Production setup components.
I mean by not modifying setup components to not switch from nginx in production to apache in development or from mysql to mariaDb or from versions of same component.
To set ports and volume mappings, plus Ips, passwords and other values as needed in each environment, the proposal on #424 will address that for us. So we will have the liberty to change this values from .env in our Laravel project.
This will allow us to:

use docker-compose.yml as the base production configuration.
use docker-compose.override.yml to use as the production release, because when we run docker-compose up it will read the overrides automatically. This file will not be tracked, once must be configured in a per project basis, but a docker-compose.override.ym-example will be provided.
use docker-compose.dev.yml that will complement the docker-compose.override.yml with the set of tools we only need for development. To boot our dev environment we will do docker-compose -f docker-compose.override.yml -f docker-compose.dev.yml up.

If you like the idea please :+1: on it.
Please share your feedback and ideas.

@Exadra37 that sounds like an amazing solution. I'd like to see it in action. Thank you.

Sounds good. 👍

After I finish the PR for my proposal in #424 I will take on this one :)

@Exadra37 @Mahmoudz I really like this idea but I fear that by modifying the current development usage we'll hit the same problems we had with #339 where user adaptation was difficult.
I personally don't mind writing docker-compose -f docker-compose.override.yml -f docker-compose.dev.yml up to start the dev environment however, following Use Docker first and learn about it later., it might be a better idea to keep the dev commands as simple as possible especially since most users use LaraDock only for dev.
Could there be a way to remove needing to specify additional files, keeping docker-compose up as the default command? Or if it's not possible we could have a dev up script (something like ./laradock/dev up as a startup command) to simplify things?

I agree with @philtrep we should have the long commands for production and the simple ones for development (by default). If that possible! Or as suggested a small command would be nice, maybe we contact this guy he created a Laradock CLI https://github.com/logaretm/laradock-cli I never used it by I like the idea.

@Mahmoudz Cool project, this makes me think we should have a section in the docs for LaraDock related projects 😃


Allow to change the background color of menu bar (Windows & some Linux)

The menu bar is inconsistent with the dark theme in particular.

To get the idea, Visual Studio Code is at the top, full Visual Studio 2015 at the bottom.

Those would have to be rewritten in JS then and the default Windows controls hidden

Totally agreed with @be5invis

Question: Why this issue hasn't a milestone included?

I guess they haven't look into this yet

Electron runs inside standard Windows window. There is a Win32 API for changing window colors and fonts. I think the discusssion should not be "is it possible?" but rather "how to do it".

The issue is the menu bar (File, Edit, Selection, ...), which is provided by Electron, not the title bar, provided by Windows.
See this Electron issue, which shows it isn't directly possible at the moment. The suggestion is to re-implement the menu bar.

It might make sense to re-implement the menu bar for Windows. With upcoming "Sets" feature in Windows 10, a custom menu would make VSCode look the part in Windows 10.

For MacOS it's implemented for atom/atom#11790 already. Can you back-port this change to VSCode?

On Linux, this should be fixed by the upcoming Electron 1.9, which will be based on Chromium 61 + GTK3. See https://github.com/electron/electron/pull/10213 and https://github.com/electron/electron/issues/10836.

Just wanted to add that I would like to see this change made also.

+1 for making this chnge

It's not the best solution but it works fine for me:
Since all lists of the menu bar are available on the command palette ctrl+shift+p , I set "window.menuBarVisibility" to "hidden" and I changed the Windows 10 color via Settings based on my vscode theme background and this is the result:

I was referred here from Make it easier to distinguish between multiple VS Code windows #51710...
in case anyone else comes across this issue, the following (in my workspace settings file) did exactly what I wanted:
{
	"workbench.colorCustomizations": {
		"statusBar.background": "#666666",
		"sideBar.background": "#444444"
	}
}

Here are the docs for those color customizations: https://code.visualstudio.com/docs/getstarted/theme-color-reference

Using a custom colour in Windows 10 and selecting to show the accent colour on Title bars will achieve the desired result.

@thedonquixotic, my mistake.. should have read the thread more thoroughly.

Hey guys, I think I might actually have a solution though this might be a bad suggestion on my part.
I've recently been fiddling around with editing the Windows theme using registry editing. It's.... probably not super recommended, and I can't tell you how right at this moment (uh... because in the process I also totally borked my install of Windows 10), but from what I've been learning (or breaking) recently it should be possible, and relatively easy to put into a bat script. I'll let you guys know when I figure more out.

@fl3x1324 Good news! but I think it's not enough since the menu bar colors should be customizable within the themes.

Hey guys, take a look at what I just found 😝
{
  "window.titleBarStyle": "custom"
}

@Emeryao It's available just in insiders version, hope it will be available in the next stable update, although it still needs some work.

@mohouyizme You was right ^^
1.25 has it
@Emeryao <3

@PawelekS Good news! it is now in the stable version which is 1.25.0.

Really great to see that dark menu and title bar have been added, thanks! I just noticed that touch input doesn't work as expected in the menu. 😆

Also in Windows if you have the taskbar on the top you will get a white line and misalignment every second time you minime and maximize the window.

I use to hide native title bars on windows by unsetting the 0xC00000 flag. While the custom title bar is great by making both the menu bar and context menus follow the dark theme, it makes the title bar unhideable. I wish there was a mixed setting: use custom menus, but use the native title bar.


[SP-5693] - Backport of PDI-18818 - TextFileInput is overwriting fiel…

…ds passed through from a previous step. (9.0 Suite)

Backport of PDI-18818 - TextFileInput is overwriting fields passed through from a previous step. (9.0 Suite)

@ssamora @pentaho/sp-branch-write @pentaho/bb-8 please review.

:white_check_mark: Build finished in 31m 21s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3684, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


[1.86->Insiders][Regression] Browser is opened with wrong address

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Insiders (broke by 08eda834f9)
OS Version: Windows 10

Steps to Reproduce:

Connect to a remote Linux host
Run az login
Browser should open at https://login.microsoftonline.com/organizations/oauth2/... but the hostname is replaced with localhost: from the querystring.

This broke by #203908, and a potential fix (which I cannot test, see https://github.com/microsoft/vscode-remote-release/issues/9556) is pending review:

#204807

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.86.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

It happens with insiders build.

I think this should be fixed now! @orgads can you confirm whether you still see this?

Sure, it is fixed, and query tunneling works as expected.

Great, thanks for confirming!


Meeting: Option to download Agenda packet is missing on upgrade test site

Meetings are missing the link to "Download Agenda Packet". Example:

Needed to turn on the step in our new pipeline to build packets. Look ok now, @shrayshray?

@hancush yes, looks good!


MNIST based Liner model - run inference e2e

Summary
Before running training, we should complete inference e2e for the based Linear model.
Few dependencies before pushing this effort:

#77

Softmax mustn't be decomposed - utilize ttnn softmax

#145

Most of the ops are working e2e. Important ops are the ones from the fwd graph (add, matmul, softmax)

Details
By completing dependencies, most of the requirements should be in place. As a next step, feel free to check this test pybuda/test/mlir/mnist/test_inference.py. It needs to be modified to use pybuda.compile instead of torch.compile + maybe a few PyBuda-related cleanups (transposes, nops, etc). But mostly, with low effort it should work out of the box.
Passing criteria

Whole models runs e2e
Golden/Silicon PCC matches
Model outputs valid results

#150 as requirement

Separated milestone is created for  e2e inference: https://github.com/orgs/tenstorrent/projects/54/views/6?sliceBy[value]=[TT-Forge]+-+MNIST+E2E+Inference