#2178 DI service resolution from scoped HttpContext request services for the IConsulServiceBuilder service

Fixes #2178

#2178

This fixes the issue with DI service resolution for Consul provider mentioned in #2178
Proposed Changes

ConsulProviderFactory - resolve the IConsulServiceBuilder from the HttpContext.RequestServices
Provide a unit test that detects a scope validation error for ConsulProviderFactory

Some notes:

I'm not sure if the test that detects a scope validation error is strictly required. By enabling scope validation in service provider all code that tries to resolve scoped services from singletons should make the tests red.
This PR's scope is Consul provider only. In acceptance tests I made the scope validation option opt-in, to not break other tests. It may be good to force it when the other providers are also fixed
ConsulProviderFactory.Get creates both Consul and PollConsul providers, so the fix should cover both of them

If there is anything that can be changed to improve quality - do not hesitate to propose changes

Trigger pipeline  🤣

Please ask me to initiate new builds. Are you not able to click the buttons in the Checks? 🤔

Please ask me to initiate new builds.

Okay, will do. There was a test that was flaky, so I just triggered another run by the only way I know ;)
Unfortunately, there were no other buttons that allow me to retry checks.

IMO new acceptance tests are not required here:

The existing ones and the future ones should detect invalid service resolution if the scope validation is turned on in helpers. I didn't do it for every acceptance test since the scope of this PR is limited.
More technical reason: I don't know of a way to have a non-scoped (aka root) service provider in HttpContext while using TestHost 😅

This PR's scope is Consul provider only. In acceptance tests I made the scope validation option opt-in, to not break other tests. It may be good to force it when the other providers are also fixed

Fair enough, but how many tests would break if we bypass the parameter and enforce scope validation for all tests? It would be logically preferable to resolve all issues with scoped services in one PR, but that seems to be beyond the current PR's scope.
Tomorrow, I plan to conduct research to determine how to comprehensively address all issues with scoped services...

I've identified numerous instances where scope validation is disabled in unit tests: specifically, a search for ".BuildServiceProvider(" reveals about 25 files with manual construction of service collections without validation. Unfortunately, we cannot address these in the current PR. 😢 But the final number of items ~50+ 🤯

Regrettably, the search for "WebHostBuilder" revealed a significant issue with disabled scopes validation in tests and the core library 🤯 This wasn't an issue when services were primarily designed as singletons by Tom and contributors. However, as we began to introduce more scoped services, it became apparent that our testing projects were not equipped to handle them. The subsequent task will be to enable scope validation across all areas: unit tests, acceptance tests, and the core library. At this time, it is not feasible to tackle this extensive problem within this PR.
During refactoring, it is sensible to:

Relocate the creation of the IWebHostBuilder objects in tests to the Ocelot.Testing project to enable the reuse of the helper across all testing projects. It is crucial to focus on C# statements containing new WebHostBuilder() expressions, necessitating the implementation of a wrapper.

@ggnaegi Do you approve?

@Niksson This PR has been merged.
Thank you for your contribution! 👍


Extends "spo site recyclebinitem restore" command with support to restore all items from site recycle bin. Closes #4726

Extends spo site recyclebinitem restore command with support to restore all items from site recycle bin. Closes #4726
😍 Happy Hacktoberfest !

Awesome job, we'll try te review it ASAP.

Merged manually, thank you for this enhancement!


Implement replacer for CHANGELOG file

Checks

I have...

[x] read and understood the Contributing Guidelines
[x] Updated any nessecary files such as the README.md and/or CHANGELOG.md.

Type of Pull request

[ ] Minor Change
This Pull request doesn't break existing configuration.
[ ] Major Change
This Pull request will break existing configuration.
[ ] Bug fix
This Pull request will fix a (critical) bug.
[ ] Documentation
This Pull request only changes documentation (README.md, CHANGELOG.md, etc.)
[x] Other: Changelog handling

Description
Implements the Readme-Replacer action to automatically replace the {tba} placeholder in the CHANGELOG.md when a release is published.
This saves us some work.

Closes #

The Pull request of @Andre601 has been marked as Ready!
It can now be reviewed and/or merged by Maintainers.

This is an automated response created by a GitHub Action
Mentioning the user won't have any effect!

The issue I observed on readme-replacers repo is that the action fails on release. However, it is successful if run manually. I don't understand why.

The issue I observed on readme-replacers repo is that the action fails on release. However, it is successful if run manually. I don't understand why.

It doesn't fail, but GitHub actions never runs it on time no(on release)
Very less chances to run automatically. Problem with gh actions.

It does trigger on time. But it fails. So I had to remove the trigger

The Pull request of @Andre601 is no longer marked as Ready!
Maintainers should wait with merging the Pull request until it is marked as ready again.

This is an automated response created by a GitHub Action
Mentioning the user won't have any effect!

I turn this into draft again because there seems to be a random bug(?) probably related to drafting releases where the action is triggered constantly through commits...
Need to ask GitHub community about this.

I think I found the issue, which was a syntax error on my end...

The Pull request of @Andre601 has been marked as Ready!
It can now be reviewed and/or merged by Maintainers.

This is an automated response created by a GitHub Action
Mentioning the user won't have any effect!


Changing Polymorphic class list order and hiding members

Hello
The asset is great, im just starting to get some value out of it!
But one thing thats bothering me right now is Polymorphic class lists, they cant be reordered..

Also I want to hide the one starting with "_" from the list
(I have the value above from before I knew Odin could be setting them from a list like that, its a enum that just Validates)

I'm afraid Odin Serializer has nothing to do with drawing of data in the inspector. This looks like an Odin Inspector issue, which should be reported here. I can say that this looks like it will be resolved by the new type/value selection feature which we plan to implement in Odin Inspector 3.1.
Regardsless, I'm closing this issue as it is not related to Odin Serializer.


R256pr31608

Adds missing changes to apache#31608, may need additional tests for coverage.
Tested in Dataflow against Pub/Sub for e2e validation.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @iht

Thanks!! @sjvanrossum


Deprecation Warning in scss files

When building css files with Dart Sass it results with many warning :
Deprecation Warning: Using / for division is deprecated and will be removed in Dart Sass 2.0.0.

Recommendation: math.div($gutter-width, 3)

More info and automated migrator: https://sass-lang.com/d/slash-div

Possible Solution
Use Recommendation: math.div()
see slash-div for references

Quick Fix
npm i -g sass-migrator
sass-migrator division node_modules/___YOUR_FRAMEWORK_PATH/scss/*.scss
Repeat this step for each of the files showing error using asterisks. This is temporary fix, changes will get removed if you update the framework else it's fine to use as of now.


Code style issues

Two minor nits (they don't impact behavior)...
In Edit.cshtml.cs:

In the HandleDeletedDepartment method, the deletedDepartment variable is not needed and can be removed.
setDbErrorMessage is a non-standard name for a C# method. Recommend capitalizing the first character.

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

ID: 16a5d321-117b-c37c-6ac9-b56009e7ead7
Version Independent ID: e337d2e1-7447-634a-d11a-511247e0f897
Content: Razor Pages with EF Core in ASP.NET Core - Concurrency - 8 of 8
Content Source: aspnetcore/data/ef-rp/concurrency.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @tdykstra
Microsoft Alias: riande

Associated WorkItem - 61967

@HarleyPebley correct on all accounts. We'll update this in the next doc sweep.

Fixed by #28560


Ability to change the position of the controls

The controls appear on the bottom by default. This seems weird to me. It would be nice to have the option of putting those controls above the text area.

I think the best solution is to just use CSS. Make the container a flexbox and then set the CSS order property for the controls:
.wk-container {
  display: flex;
  flex-direction: column;
}

.wk-commands {
  order: -1;
}


Correct links to correct repository

I also update :

adding a link that refer the CloudStack website
prerequesites list on the "Ansible host"
prerequesites on the CloudStack server
delete the needs of the cs submodule as now its fully in Ansible CloudStack module

After the issue I mentioned yesterday https://github.com/kubernetes/kubernetes.github.io/issues/3407. Here is my documentation modification request accordingly to the work on the new repo I created.

This change is

Hello !
As on the k8s website, actually the article regarding CloudStack is not working, do you want some help from my side for the tech-review ?
Don't hesitate if you have any questions or if you want me to show you that with my suggestion everything is working well.

@ltupin We need someone else to give independent verification that the technical aspects of the change are correct. It's just the process we use to ensure the quality of the doc changes. If you could suggest someone else that knows CloudStack and can do a tech review, that would work. Thx

@runseb ?

Runseb is now @sebgoa
I will check this thursday, i am traveling right now

Just had a quick look.
It is awkward because the repo that is referenced is a fork of a skippbox repo.
So either you need to PR the original repo. Or you need yo put this code upstream in cloudstack community or in the exoscale repo (if exoscale specifique).
I do realize that i am the original author, but we need to aim for a long term maintainable plan.

I propose, to ensure a long term maintainable plan, to put this code upstream in CloudStack community (have to define where...) ?
There is nothing really related to Exoscale except the usage of cs (we can plan to modify it to use another tool to interact with CloudStack, ie CloudMonkey ?).

let's bring in @pdion891 . Hey PL, where can we put some Ansible scripts used to create a k8s cluster in cloudstack. Somewhere upstream for CS community ? I thought of cloudstack-extras but it does not seem active right now.

Hi @sebgoa, I think the simplest answer would be to create a new project under https://github.com/apachecloudstack. seams quiet for now but that is where we should be placing side projects related to Apache Cloudstack.  Thanks ! :-)

@sebgoa @pdion891 Now that the location is https://github.com/apachecloudstack/k8s, is this PR ready to be merged?

Nice, Thanks a lot !


Option to disable Motion Detection on battery

Is your feature request related to a problem? Please describe.
Battery drains fast when motion detection is enabled.
Describe the solution you'd like
A new option in the "Motion Detection" menu: "Disable Motion Detection when running on battery"
When the power is disconnected, turn off motion detection (and maybe also other power hungry features) and let the user have to press the power button to awake the device.
When powered back, restore the "Motion Detection" state.
Thanks

You can't run the camera without using battery power, the reason is that we generate images from the camera stream for MJPEG streaming as well as motion detection.   The application does reduce the number of images produced based on the FPS you set in the camera settings.  I recommend you reduce your FPS to 5 or not use the application when running on battery, its not really meant to run on battery since its always in the foreground running.  BTW, you need to use the latest version of WallPanel to have the camera battery saving feature.

Yes I have the latest version. My idea is to completely shutdown the camera when on battery

If you choose to run the application on a device that runs on battery, my recommendation is not to use the camera feature.  Your other option is to monitory the sensor data and disable camera using MQTT commands when charge level drops below a certain amount.  However, I think there is a bug reported on disabling the camera remotely and restarting it.

You mean by remote command ? That would be feasible but I would like something really smooth and I think this could be a nice feature for wallpanel:

when mounted on the wall charger, full experience is enabled, with motion detection to turn on the screen on movement
When disconnected, temporarily disable motion detection and all camera related feature since we are on battery.
turn everything on when reconnected to power.
I’ll try to trigger it from home assistant, but that could be a nice feature. Under an option of course.

I have updated the method we use to turn off/on the camera.  Rather than a built-in feature, you will need to use the devices publish status to determine when to send a command to turn off/on the camera.  This will disable all camera operations to conserve power.  You can enable the camera again using the MQTT command as well.    It should be noted that not all device report if they are on battery or their battery level.   I will be putting out a new release for side loading and testing shortly.

Test version url: https://github.com/thanksmister/wallpanel-android/releases/tag/v<IP_ADDRESS>

Thanks a lot! I've configured mqtt devices in home assistant, and my Fire HD does well report the acPlugged property and the battery level.
I'll test your version asap

Hello.
I have tested with the latest build and it does not seem to work:
$ mosquitto_pub -t 'wallpanel/mywallpanel/command' -u xxx -P xxx -h <IP_ADDRESS> -d  -m '{"relaunch": true}'
# ==> restarts happens
$ mosquitto_pub -t 'wallpanel/mywallpanel/command' -u xxx -P xxx -h <IP_ADDRESS> -d  -m '{"camera": false}'
# ==> nothing happens
# ==> movement is then detected.

Don't know if this is related but in the menu, I cannot disable the camera as well (if i turn off the switch, i leave and go back to the camera menu the swich is enabled).
Would it is possible to send the state of the camera regularly :
wallpanel/mywallpanel/state
{"currentUrl":"http:\/\/<IP_ADDRESS>:8123\/lovelace\/default_view","screenOn":true,"brightness":29, "camera", true}

thanks

You can retest with https://github.com/thanksmister/wallpanel-android/releases/tag/v<IP_ADDRESS>

The camera still seems enabled:
Client (null) received PUBLISH (d0, q0, r0, m0, 'wallpanel/mywallpanel/command', ... (17 bytes))
{"camera": false}
Client (null) received PUBLISH (d0, q0, r0, m0, 'wallpanel/mywallpanel/connection', ... (7 bytes))
offline
Client (null) received PUBLISH (d0, q0, r0, m0, 'wallpanel/mywallpanel/connection', ... (6 bytes))
online
Client (null) received PUBLISH (d0, q0, r0, m0, 'wallpanel/mywallpanel/sensor/motion', ... (14 bytes))
{"value":true}

But now, when I go in the setting and disable the camera, it stays disabled

wallpanel/mywallpanel/state
{"currentUrl":"http://<IP_ADDRESS>:8123/lovelace/default_view","screenOn":true,"brightness":29, "camera", true}

I added this to the application data and it will be in the next release.

Published https://github.com/thanksmister/wallpanel-android/releases/tag/v<IP_ADDRESS>

There may still be issues, for example if the user has never turned on the camera and tries to start it using the MQTT command.  I don't think it will work due to camera permissions and also the user has not setup the camera (front/rear).


train.py ERROR

I can not execute train.py. Please help me!! m(_ _)m
Windows7, python3.5.4, tensorflow1.2.1, opencv3.3.1
AttributeError: Can't pickle local object 'mpgen..main'   ...why???

Traceback (most recent call last):
File "train.py", line 269, in
initial_weights=initial_weights)
File "train.py", line 241, in train
for batch_idx, (batch_xs, batch_ys) in batch_iter:
File "train.py", line 102, in wrapped
proc.start()
File "C:\ProgramData\Anaconda3\envs\svhn\lib\multiprocessing\process.py", line 105, in start
self._popen = self._Popen(self)
File "C:\ProgramData\Anaconda3\envs\svhn\lib\multiprocessing\context.py", line 212, in _Popen
return _default_context.get_context().Process._Popen(process_obj)
File "C:\ProgramData\Anaconda3\envs\svhn\lib\multiprocessing\context.py", line 313, in _Popen
return Popen(process_obj)
File "C:\ProgramData\Anaconda3\envs\svhn\lib\multiprocessing\popen_spawn_win32.py", line 66, in init
reduction.dump(process_obj, to_child)
File "C:\ProgramData\Anaconda3\envs\svhn\lib\multiprocessing\reduction.py", line 59, in dump
ForkingPickler(file, protocol).dump(obj)
AttributeError: Can't pickle local object 'mpgen..main'
(svhn) E:\yos\Tensorflow\svhn\deep-anpr-master>Traceback (most recent call last):
File "", line 1, in
File "C:\ProgramData\Anaconda3\envs\svhn\lib\multiprocessing\spawn.py", line 106, in spawn_main
exitcode = _main(fd)
File "C:\ProgramData\Anaconda3\envs\svhn\lib\multiprocessing\spawn.py", line 116, in _main
self = pickle.load(from_parent)
EOFError: Ran out of input

Does your problem have been solved? I also encountered the same problem and asked for answers. Thank you very much!

Does your problem have been solved? I also encountered the same problem and asked for answers. Thank you very much!

For me, win10 with python3.5
@mpgen => #@mpgen
add # is work for my enviroment


Convert original C# 101 to Jupyter Notebooks

Convert original C# 101 files to .ipynb files

.dib files are converted to .ipynb during image build in Dockerfile using this dotnet script.


Having trouble writing a plugin

I have a plugin:
var Foo = function() {};
Foo.prototype.apply = function() {
compiler.plugin('compilation', function(compilation, params) {
compilation.dependencyFactories.set(ConstDependency, new NullFactory());
compilation.dependencyTemplates.set(ConstDependency, new ConstDependency.Template());
});
compiler.parser.plugin('call abc.bar', function(expr) {
var param = this.evaluateExpression(expr.arguments[0]);
var dep = new ConstDependency(JSON.stringify("something else"), expr.range);
dep.loc = expr.loc;
this.state.current.addDependency(dep);
return true;
});
}
--> When I run it against this file:
var someString = abc.bar('something');
--> It gets compiled and replaced with:
var someString = 'something else';
--> But when I do this:
var abc = require('abc');
var someString = abc.bar('something');
--> It gets compiled to:
var abc = webpack_require(123);
var someString = abc.bar('something');
What's happening and why does the compiler behave differently when you have a local variable?
And how do I make it so the replacement still happens?

Moved to stackoverflow: http://stackoverflow.com/questions/37620763/how-do-i-write-a-plugin-that-replaces-any-function-execution-with-a-string-resul

Did you manage to figure this out?

Nope.  This is still an issue for me

bump

Closing due to inactivity.

Please read official docs about writing plugins :+1:

Official docs: https://webpack.js.org/contribute/writing-a-plugin/


withStyles break Jest tests

Seems like when using the withStyles HOC it'll actually break my Jest test. It creates children elements where it shouldn't.
Here is the code to reproduce it:
import React from 'react';
import ReactDOM from 'react-dom';
import { shallow } from 'enzyme';

import withStyles from 'material-ui/styles/withStyles';
import LoadingWrapper from './LoadingWrapper';

const ReproBug = ({ loading, children }) => {
  if (loading) {
    return Loading...;
  }
  return children;
}
const ReproBugWithStyles = withStyles({})(ReproBug);

const children = Dummy Children!;

it('should not render children if loading is true', () => {
  const wrapper = shallow(
    <ReproBug loading={true}>{children}</ReproBug>
  );
  expect(wrapper.contains(children)).toEqual(false);
});

it('should not render children if loading is true', () => {
  const wrapper = shallow(
    // using ReproBugWithStyles instead of ReproBug
    <ReproBugWithStyles loading={true}>{children}</ReproBugWithStyles>
  );
  expect(wrapper.contains(children)).toEqual(false); //<--- this check fails, it is actually true
});

I use the create-react-test and simply run yarn test. (they use Jest and enzyme IIRC)

I also notice this in "real world" usage beside the jest test. For example I use this kind of logic on my <LoadingWrapper> for my Apollo requests and Apollo set the loading and data (and some more) variable. So I could make a simple Wrapper which avoid loading the children elements as long as data isn't recieved.
<LoadingWrapper loading={loading}>
    <UseDataElement data={data}
</LoadingWrapper>

With this bug the <UseDataElement> element is actually "rendered" or at least triggered so I receive propTypes warning for missing data.

Mh I'll get similar errors when trying some of your intern testing code.
For example when trying this:
https://github.com/mui-org/material-ui/blob/v1-beta/src/Divider/Divider.spec.js#L17-L20
I'll get this error:
Expected value to equal:
      "hr"
Received:
      "Divider"

Maybe I should use your (testing scripts)[https://material-ui-next.com/guides/testing/#testing] instead of the create-react-app ones.

@Skaronator I was having a ton of issues testing my application. Once I found Mui's docs on testing and used their Shallow and Mount I had no issues.

It creates children elements where it shouldn't

@Skaronator The issue isn't about the children elements but with the intermediary element, it's creating. The shallow() API of enzyme only render one level depth, hence any higher-order component is going to obfuscate the children. You have different workarounds. I would encourage you to refer to the enzyme documentation, still, here they are:

using mount()
using .dive()
using our internal until() helper
using our createShallow() public API

I had some trouble getting my head around why shallow wasn't working for some tests when I started using withStyles, so I broke it down into a few self-contained examples at https://gist.github.com/a-h/21df0f432ae02a6dfed941debb0e5950
Once I did that, I realised that I needed to switch from shallow to mount since withStyles is a higher order component and 'wraps' the other component.

I found that material-ui was not breaking my tests even when using withStyles and that this worked just fine using plain old enzyme:
import { configure, shallow } from 'enzyme';
import Adapter from 'enzyme-adapter-react-16';
configure({ adapter: new Adapter() });

// CODE OMITTED HERE

it('renders title matches it with containsMatchingElement', () => {
     const wrapper = shallow(<ElfHeader />);
     const target = <Typography>Address Maven</Typography>;
     expect(wrapper.dive().containsMatchingElement(target)).toBe(true);
});

Here was the JSX I wanted to match:
<Typography variant="title" color="inherit" className={classes.flex}>
    Address Maven
</Typography>

And here was the messy code material-ui produced:
<WithStyles(Typography) variant="title" color="inherit" className="ElfHeader-flex-100">
       Address Maven
</WithStyles(Typography)>

Nevertheless, my test passed.
I say this only because this thread is a bit misleading for those who get errors and don't know their source.

The shallow = createShallow({ dive: true }); totally made my day... 🎆

Why this issue is closed?
None of those is working for me,
Still get displayed with
<WithStyles(Component)
Cannot go past through it...

I'm having the same issue as Matteo. When testing the output in Jest i get <WithStyles(MaterialComponent) when I want to test for <MaterialComponent
For example if I want to see if a Drawer component is present I have to test for <WithStyles(Drawer.
Is there a way to get the test render to be the unwrapped component? or is the only way to test for the <WithStyles(... version?

It shouldn't be too long before we start/try using hooks over higher-order component internally for the styles. I'm not sure what is the implication for shallow rendering. It will most likely solve this class of issues.

I solved my issue, was an obvious fix. I was using the component name in quotes eg .find('Avatar') instead of importing it into the test and calling it directly .find(Avatar) - no quotes. That worked.

@jondbm Solved the same way

I am able to proceed with createShallow and it works fine for Components using WithStyles but I couldn't make it work with Components which needs redux.
Works:
import PropTypes from "prop-types";
import React, { Component } from "react";
import { connect } from "react-redux";
import { makeStyles, withStyles } from '@material-ui/core/styles';
import CircularProgress from '@material-ui/core/LinearProgress';
const useStyles = (theme) => ({
root: {
flexGrow: 1,
},
extendedIcon: {
marginRight: theme.spacing(1),
}
});
class LogoutSuccess extends Component {
render() {
    const { classes } = this.props;
    return (

            <CircularProgress size={24} className={classes.extendedIcon} />

    );
};

}
LogoutSuccess.propTypes = {
classes: PropTypes.object.isRequired,
};
export default (withStyles(useStyles)(LogoutSuccess));
import { Provider } from "react-redux";
import React from "react";
import Enzyme, {  mount } from "enzyme";
import Adapter from "enzyme-adapter-react-16";
import LogoutSuccess from "../../../src/client/components/logoutsuccess";
import sinon from "sinon";
import configureMockStore from "redux-mock-store";
import axios from "axios";
import thunk from "redux-thunk";
import * as mockData from '../mock';
import { createShallow } from '@material-ui/core/test-utils';
Enzyme.configure({ adapter: new Adapter() });
const getStore = props => {
const intialState = props;
const middlewares = [thunk.withExtraArgument(axios)];
const mockstore = configureMockStore(middlewares);
const store = mockstore(intialState);
return store;
};
describe("Home Page ", () => {
let sandbox;
beforeEach(() =>
sandbox = sinon.createSandbox()
);
afterEach(() => sandbox.restore());
it('renders title matches it with containsMatchingElement', () => {
const shallow = createShallow();
const wrapper = shallow(<LogoutSuccess />);
 expect(wrapper.dive().find(CircularProgress)).toHaveLength(1);

});
});

Not working:
import PropTypes from "prop-types";
import React, { Component } from "react";
import { connect } from "react-redux";
import { makeStyles, withStyles } from '@material-ui/core/styles';
import CircularProgress from '@material-ui/core/LinearProgress';
const useStyles = (theme) => ({
root: {
flexGrow: 1,
},
extendedIcon: {
marginRight: theme.spacing(1),
}
});
class LogoutSuccess extends Component {
render() {
    const { classes } = this.props;
    return (

            <CircularProgress size={24} className={classes.extendedIcon} />

    );
};

}
const mapStateToProps = state => ({
countryCodes: state.formatbanner.countryCodes
});
LogoutSuccess.propTypes = {
classes: PropTypes.object.isRequired,
};
export default connect(mapStateToProps, null, null, { withRef: true })(withStyles(useStyles)(LogoutSuccess));
import { Provider } from "react-redux";
import React from "react";
import Enzyme, {  mount } from "enzyme";
import Adapter from "enzyme-adapter-react-16";
import LogoutSuccess from "../../../src/client/components/logoutsuccess";
import sinon from "sinon";
import configureMockStore from "redux-mock-store";
import axios from "axios";
import thunk from "redux-thunk";
import * as mockData from '../mock';
import { createShallow } from '@material-ui/core/test-utils';
Enzyme.configure({ adapter: new Adapter() });
const getStore = props => {
const intialState = props;
const middlewares = [thunk.withExtraArgument(axios)];
const mockstore = configureMockStore(middlewares);
const store = mockstore(intialState);
return store;
};
describe("Home Page ", () => {
let sandbox;
beforeEach(() =>
sandbox = sinon.createSandbox()
);
afterEach(() => sandbox.restore());
it('renders title matches it with containsMatchingElement', () => {
const shallow = createShallow();
const mockstore = configureMockStore();
const store = mockstore(mockData.data);
const wrapper = shallow(<Provider store={store}><LogoutSuccess /></Provider>);
 expect(wrapper.dive().find(CircularProgress)).toHaveLength(1);

});
});

@a-h @Skaronator @MatteoGioioso @jondbm Can someone help me if you have written test cases for components having dependent on redux?


Update Chinese translation

Added Chinese translation to keep up with the English version.

@ly5156 Would it be possible for someone on your team to give this a quick review?

Already updated according to the comments, thanks! @niusmallnan


feat: Add backoff to discovery connection

What this PR does 📖

Backs off retrying to establish a connection

Which issue(s) this PR fixes 🔨

Special notes for reviewers 🗒
Additional comments 🎤

Closing since discovery will be refactored soon to simplify connection and peer handling


how to set a shorchut key for AI interactor?

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
The AI interactor is very useful for image segmentation, but I did not find any shortcut key for this function. I have to re-select it after annotating each instance. How can I set up a hotkey for this?
Describe alternatives you've considered
No response
Additional context
No response

Default shortcut to use the latest drawing tool is N.


Could you comment on what the "researcher" in Table 1 refers to?

Hi,
Thanks for the excellent job! After reading the paper, I am not sure what the "researcher" in Table 1 refers to. Is that the label annotated by professional subjects?
Best!

The "researcher" denotes authors who design the annotation and inspectors who double-check each annotation.

Got it! Thanks very much!


Host registration url is not generated when "Add Host" for custom is launched for the first time.

Rancher Version: v0.61.0-rc3
Docker Version:10.1.
OS: Ros - v0.4.4-dev-2
Steps to Reproduce: On fresh install of rancher-server , launch "Add Host" for custom.
Notice that UI is stuck when generating the registration url. When this page is relaunched , registration url shows up as expected.

This was broken by changes to only connect to the subscribe socket when in a /env/1aX/... URL, which was a bad idea.  So yeah, now it stays connected more.

Version - master 3/2
Verified fixed


added state labels to the measurement cal matrix plot

Summary #70
Details and comments

Any idea why the Travis check failed? I followed the suggested code in: https://matplotlib.org/gallery/images_contours_and_fields/image_annotated_heatmap.html#sphx-glr-gallery-images-contours-and-fields-image-annotated-heatmap-py

I restarted the failed job, it looks like a non-deterministic failure on the measurement calibration tests caused by randomness inherent in the test. It's definitely not related to this patch which is just adjusting ticks and labels.


Raspberry PI

I am trying to run the scripts but I am facing dotenv module not found.
Anybody knows how to fix this?
I am testing using a raspberry pi.

"python -m pip install python-dotenv" you can try this one.

Yeah i tried everything with raspberry pi. It doesn't seem to work on Raspian and Ubuntu is too heavy on it. I will close this issue.

Sorry for that @loayei , I am not sure how to help you as I have never used a Raspberry.
If it can help, I run the bot on a Debian machine on Vultr, it only costs $6 a month and it never game me problems.

That's exactly what am looking for. I will try it out now. Does it have a GUI interface or is it like Ubuntu Server?
@coccoinomane

No GUI, you need to SSH into it.
Just make sure to install python > apt-get install python3 pip git
Then just follow the README :-)

(I prefer to use the Terminal, but Vultr also allows you to SSH directly via their website)

@coccoinomane Thanks alot I will try it now and see how it goes.

@coccoinomane I was able to install a GUI successfully on Vultr If you want me to add instructions to the ReadME file or something I can as soon as I successfully run the bot on it.

Its working just let me know if you need me to add it.

That's great @loayei , I am happy you managed 💪
A short tutorial would be great.
Do you feel like creating a Wiki page?
You should be able to create one here > https://github.com/coccoinomane/crabada.py/wiki

That's great @loayei , I am happy you managed 💪
A short tutorial would be great.
Do you feel like creating a Wiki page?
You should be able to create one here > https://github.com/coccoinomane/crabada.py/wiki

Yeah I can do that. I'll try to post it by tomorrow.


Add better support for imports suggestion

When imports that are declared via addExtraLib are explicitly wrapped inside declare module then IntelliSense is actually very nice.
Following setup:
monacoEditor.languages.typescript.typescriptDefaults.addExtraLib("declare module 'test/file1' { export interface Test {} }"<EMAIL_ADDRESS>monacoEditor.languages.typescript.typescriptDefaults.addExtraLib("declare module 'test/file2' { export interface Test {} }"<EMAIL_ADDRESS>
produces very nice auto-completion:

But if type definitions are missing explicit module wrapping as following:
monacoEditor.languages.typescript.typescriptDefaults.addExtraLib('export interface Test {}'<EMAIL_ADDRESS>monacoEditor.languages.typescript.typescriptDefaults.addExtraLib('export interface Test {}'<EMAIL_ADDRESS>
then IntelliSense is also completely missing:

It would be nice if the latter use case would also produce the same outcome as the first one without needing to explicitly wrap type definitions.

Subscribing because this technique is how automatic type acquisition works in the typescript playground

Is there any workaround for these issue ?

I spent a bit of time digging through old Stack Overflow answers (example), and adding the declare module 'package-name' seemed to be the missing piece here. So, thanks for this issue, and it would be great to (a) feature an example like this in the monaco editor playground, and (b) perhaps display clearer error messages when extra libs are registered in un-usable ways. I've written up the solution I finally ended up with, here: https://stackoverflow.com/a/66948535

I found a solution. Just add a simple package.json and specify dependencies.
.addExtraLib(,"file:///node_modules/foo/index.d.ts")
.addExtraLib(,"file:///node_modules/foo/bar/index.d.ts")

const packageJson = {dependencies:{
  "foo": "*",
  "foo/bar": "*",
}}

.addExtraLib(JSON.stringify(packageJson), "package.json")

No need to bundle all type definitions or wrap definitions with declare module.

When imports that are declared via addExtraLib are explicitly wrapped inside declare module then IntelliSense is actually very nice. Following setup:
monacoEditor.languages.typescript.typescriptDefaults.addExtraLib("declare module 'test/file1' { export interface Test {} }"<EMAIL_ADDRESS>monacoEditor.languages.typescript.typescriptDefaults.addExtraLib("declare module 'test/file2' { export interface Test {} }"<EMAIL_ADDRESS>
produces very nice auto-completion:
But if type definitions are missing explicit module wrapping as following:
monacoEditor.languages.typescript.typescriptDefaults.addExtraLib('export interface Test {}'<EMAIL_ADDRESS>monacoEditor.languages.typescript.typescriptDefaults.addExtraLib('export interface Test {}'<EMAIL_ADDRESS>
then IntelliSense is also completely missing:
It would be nice if the latter use case would also produce the same outcome as the first one without needing to explicitly wrap type definitions.

Why isn't the autocomplete working even when I use the first method?
See playgroud link

I debugged it, it should be caused by https://github.com/microsoft/monaco-editor/issues/2582


(firestore-multimodal-genai): handling large images with gemini

For large enough images gemini will fail, it has a limit
https://cloud.google.com/vertex-ai/docs/generative-ai/multimodal/send-multimodal-prompts#image-requirements
This needs to be investigated, not sure whether this would be a bug or feature request yet.
One potential workaround is to do compression of images over a certain size.

@cabljac was this solved in the image compression PR?


Scrolling on element happens only the first time

I have a slider of elements and a large view below the slider that is mounted when an element is clicked.
I have used scrollIntoViewIfNeeded when the large view component gets mounted.
It scrolls nicely the first time, but when i close the view and then open it again. It does not scroll.

I am guessing here but when you close the view and open it again, does scrollIntoViewIfNeeded get fired?
Everytime it is called it will measure the DOM node you're passing in, so wether you're calling it for the first time, or the 100th time shouldn't make a difference.
However the timing of when you call it does make a difference. There could be a race condition happening between the closing and opening of your view that is causing it to try and measure if your node is visible or not at the wrong time (before it is mounted again).
Can you share some code, or better yet use something like codesandbox to recreate the problem? If you do that I'll be able to give much better help 😄

This should no longer happen in v2


https://overwatch-api.net endpoint test returning "Route not found"

So the form at overwatch-api.net isn't returning any data for me. Calling the api using the browser returns the data, but not the form itself.

False alarm. My Jscript wasn't enabled. Sorry


"a trap may occur between the PTE update and the memory access"?

The ISA spec says

updates to the D bit must be exact (i.e., not speculative)

and the proposed extension spec says

a trap may occur between the PTE update and the memory access that caused the PTE update.

This is a new relaxation from the base Privileged ISA spec.

What scenario prompted us to add it?

I presume something to do with the D bit, since the A bit can be updated speculatively.
Was it something to do with page-crossing misaligned or multi-element vector accesses?

Can we please be more specific: which traps are allowed here?

Looking at Table 3.7 "Synchronous exception priority in decreasing priority order", there is a spectrum of possibilities.
Other ISAs spend pages painfully elaborating the exact fault checking sequence, and listing out allowable behaviors.

In my vocabulary, an instruction that is non-speculative is right up at the line between being able to trap and not, and this sentence gives.

So this is not new text. The privileged specification presently states "The PTE update is not required to be atomic with respect to the explicit memory access that caused the update, and the sequence is interruptible." The "sequence is interruptible" was clarified to indicate "a trap may occur between the PTE update and the memory access that caused the PTE update". It is not related specifically to D bits. Some examples - 1) When two-stage address translation is, A bits may get set in the g-stage PTEs for each vs-stage PTE  access but there may be a trap - an access fault or a page  fault or a guest page fault or any of the other traps defined to occur as part of the virtual address translation process - that may occur. 2) When the leaf VS-stage PTE is determined, the A and/or D bits may be set in the VS-stage PTE but a trap occur due to the G-stage walk to translate the GPA to an SPA. 3) After the virtual address was translated successfully a trap may occur due to the memory access itself for example - an abort signaled by the  memory, access to corrupted data (e.g., poisoned data), timeout waiting for the target to provide data, violation of a PMA or PMP rule, etc.

Yes, I agree with all those scenarios.  Other ISAs spend pages painstakingly elaborating them, but let's not do that here.  For now, a smidgeon of clarification could help the reader along: bolded text
"If a trap occurs later in the translation process then the A and/or D bit may be updated but the memory access that caused the PTE update may not occur.

If a change is to be made to that sentence, can we avoid the term "may not" since that could be construed in different ways?  I suggest either "might not" or "must not" as appropriate.

So the trap may occur later but  not necessarily in the translation process. For example due to PMA checks:

PR #7 to address this issue.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: srwindows sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


indicate HostConfig to be used for "create container" (add again)

As proposed in #13466 (comment) this re-adds a commit from master originally applied with PR #13615 to be applied on the docs branch.
Is this the way to go for docs changes?
/cc @moxiegirl
Signed-off-by: Tobias Gesellchen<EMAIL_ADDRESS>

LGTM

@gesellix yes. you are at the right place. LGTM @thaJeztah for the merge.

LGTM, thanks @gesellix !

thanks you all, too! I hope this will now stay in the right branch ;-)


Syntax hightlighting in editor

First of all, this is a great tool !
I am currently experimenting javascript templates which I include in my PHP script with the <script type="text/template" id="whatever>....
I am using Zend Studio as an editor, and I could not find a way to have syntax highlighting for the template.
Any clue on how to achieve this ?
Regards,
Ronan

Closing this as I am not sure what this is requesting.


Bridgy publish should support POSSEing tag reply to Flickr

Similar to https://github.com/snarfed/bridgy/issues/786 (but broader for any tag reply, and for Flickr instead of Facebook):
This is a feature request for POSSEing tag reply posts to Flickr.
I.e. if I post a tag reply on my own site, like documented here:
https://indieweb.org/tag-reply#How_to_simple_tag-reply
where one (or more) of the reply-to URLs is to a Flickr permalink post (likely others's photos, but could be my own that I'm adding tags to after I published it), Bridgy Publish (of my tag reply) should propagate that tag reply to that Flickr permalink.
This is the publish equivalent of the following backfeed features, and may be easier to start with (consider implementing publish before implementing backfeed for equivalent interaction)

776
488

Labels: publish
(Originally published at: http://tantek.com/2019/007/b1/bridgy-support-tag-reply-flickr)

done! @tantek let me know if you have any q's.


Bug?

Version: 2.9.1
Build: 2016-12-12 17:48
OS: Windows 7 6.1
JDK: 1.8.0_60 x86 Oracle Corporation
Dont' scrape country movie in polish language.
tmm_logs.zip

what exactly did not get scraped? did not find any obvious errors in the logs..

My OS Windows 7 and TMM in Polish language.
Earlier, the country movie scrape in Polish - now English lanuage
example - Hiszpania ---  Spain, Francja --- France, Niemcy --- Germany et

c.

Ok it works :)
thank you very much


Fix hip-clang build error.

Add missing headers.

resolves #SWDEV-199418
Summary of proposed changes:

please merge this into master branch


10 - HTML:  (Venue) Create form

Create a form in HTML to add new venues to existing list on venue page

I believe this was already completed by Zach.


Create golangci-lint.yml

Issues associated with this PR
Your PR should present related issues you want to solve.
Solutions
You should show your solutions about the issues in your PR, including the overall solutions, details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

暂时先不要合并
和 reviewdog 对比之后再定用哪个


pytorch_fairseq.py - changed needed to run on GPU

Something like the following seems to be needed to run on GPU.
def predict_next(self):
"""Call the fairseq model."""
if self.use_cuda:
lprobs, _ = self.model.forward_decoder(
torch.cuda.LongTensor([self.consumed]), self.encoder_outs
)
lprobs[0, self.pad_id] = utils.NEG_INF
return np.array(lprobs[0].cpu())
else:
lprobs, _ = self.model.forward_decoder(
torch.LongTensor([self.consumed]), self.encoder_outs
)
lprobs[0, self.pad_id] = utils.NEG_INF
return np.array(lprobs[0])

is this code tested? If yes I'm happy to update the code, or you can also make a PR / push it directly if you like.

Hi Felix!
At the moment,  i’ve only tested it as for the example in
https://ucam-smt.github.io/sgnmt/html/tutorial_pytorch.html https://ucam-smt.github.io/sgnmt/html/tutorial_pytorch.html
Is that enough testing ?   I could decode a few test sets with a few additional models, if you like.
btw,  just to note that fairseq is now at version 1.10.x , and no longer compatible with sgnmt
Bill

On 7 Dec 2020, at 14:19, fstahlberg<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
is this code tested? If yes I'm happy to update the code, or you can also make a PR / push it directly if you like.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ucam-smt/sgnmt/issues/9#issuecomment-739946858, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACIKSOY6D7NZQQNXNX44YD3STTP5ZANCNFSM4UPKVMFA.
http://schema.org/ https://github.com/ucam-smt/sgnmt/issues/9#issuecomment-739946858 https://github.com/ucam-smt/sgnmt/issues/9#issuecomment-739946858 https://github.com/

that's enough testing for now - I'll have a look at updating it to 1.10.x in a few weeks


Fix typo on contrastive loss

There is a typo on the contrastive loss wrapper. Not that it will change a lot, but it is nice to keep things straight.
https://github.com/tensorflow/addons/blob/v0.10.0/tensorflow_addons/losses/contrastive.py#L100
Best regards,
Gustavo.

Hi @gugarosa nice catch! Would you like to submit a PR to fix it?

Sure, sounds great! Let me clone the repo and submit it.


Added content to website

I have added content to the website. Resolves #54.

Thankyou @MahajanPreksha for doing this. I was thinking that someone should do this correction otherwise I have to do it by my own. Thanks again for the help.

Happy to contribute :)


Disable Ockam Heartbeats for Tcp on server side

While we don't support TCP-level keep-alive #3450, we occassionally send empty Ockam packets in tcp workers to:

Keep connection alive
Find out early if connection was closed
We call that Heartbeat. Thos packets are sent here and they're just ignored while receiving here.

The idea is that only client side (the side that initiates connection) should do that, while currently rust doesn't check that and always sends them. Possible solution is to store information about connection role (client or server) in TcpSendWorker and enable/disable heartbeats based on that

Hi @SanjoDeundiak, can i work on this issue?

@michealkeines of course, all yours.

I checked the TcpSendWorker struct
I have tried your solution, we can add is_initiator and call schedule_heartbeat() based on this value
pub(crate) struct TcpSendWorker {
    ...
    is_initiator: bool,
}

if self.is_initiator {
    self.schedule_heartbeat().await?;
}

TCP send worker pair could be created using TcpSendWorker::new_pair, we can update these two calls
//listener.rs:60 - we dont want listeners to start heartbeats as they act as server
TcpSendWorker::new_pair(ctx, handle_clone, Some(stream), peer, Vec::new(), false).await?;

//sender.rs:138 - we want any send call to start heartbeats
Self::new_pair(ctx, router_handle, stream, peer, hostnames, true).await?;

Please correct me, if i have missed something. @mrinalwadhwa @SanjoDeundiak

@michealkeines sounds great!

@michealkeines hold on, we may have race condition with #3492

@michealkeines I think this issue is superseded by #3492

@michealkeines Thank you for spending time on this!


Removing react-dom dependency

MentionsInput.jsx requires react-dom but react-dom is only listed as a devDependency
which causes projects that use react-mentions to have react-dom in their dependencies list.
React 0.14 provides React.findDOMNode() which is all react-dom was used for in this project.

Thanks for the PR. Published 0.2.2 right away ;)

Thanks @frontendphil !


Panicking when trying to complete after a >

Describe the bug
thread 'main' panicked at lib/src/cli.rs:155:9:
unexpected word: >
stack backtrace:
   0:        0x1050adc38 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h01b2beffade888b2
   1:        0x104fcdfe8 - core::fmt::write::hbadb443a71b75f23
   2:        0x105089598 - std::io::Write::write_fmt::hc09d7755e3ead5f0
   3:        0x1050b1bc0 - std::sys_common::backtrace::print::h28349e5c25acbac7
   4:        0x1050b2e24 - std::panicking::rust_panic_with_hook::h84760468187ddc85
   5:        0x1050b1d38 - std::panicking::begin_panic_handler::{{closure}}::he666a5eb600a7203
   6:        0x1050b1c9c - std::sys_common::backtrace::__rust_end_short_backtrace::h592f44d2bf9f843f
   7:        0x1050b1c90 - _rust_begin_unwind
   8:        0x1051a55b8 - core::panicking::panic_fmt::h98bbf7bdf4994454
   9:        0x10516faa4 - usage::cli::parse::h500a5ef7b3b1e2c4
  10:        0x105179cb4 - usage_cli::cli::Cli::run::hfb7d83fe75575267
  11:        0x104f6a138 - usage::main::h7a7bb9428d38e962
  12:        0x104f6853c - std::sys_common::backtrace::__rust_begin_short_backtrace::h06cf368a3fa79ced
  13:        0x104f6b5c0 - _main

To Reproduce

type mise generate pre-commit >
press Tab
observe the error above

Expected behavior
Shell completion of local files
mise doctor output
version: 2024.6.6 macos-arm64 (2024-06-20)
activated: yes
shims_on_path: no

build_info:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Thu, 20 Jun 2024 17:07:53 +0000
  Rust Version: rustc 1.79.0 (129f3b996 2024-06-10) (Homebrew)
  Profile: release

shell:
  /opt/homebrew/bin/bash
  GNU bash, version 5.2.26(1)-release (aarch64-apple-darwin23.2.0)
  Copyright (C) 2022 Free Software Foundation, Inc.
  License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

  This is free software; you are free to change and redistribute it.
  There is NO WARRANTY, to the extent permitted by law.

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/Library/Caches/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/company/repository/.mise.toml

backends:
  cargo
  core
  go
  npm
  pipx
  spm
  ubi

plugins:
  buf     https://github.com/truepay/asdf-buf.git#6c27312
  bun     (core)
  deno    (core)
  erlang  (core)
  go      (core)
  java    (core)
  node    (core)
  python  (core)
  ruby    (core)
  usage   https://github.com/jdx/mise-usage.git#fe3888a
  zig     (core)

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS>
env_vars:
  MISE_SHELL=bash

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cargo_binstall = true
  color = true
  disable_default_shorthands = false
  disable_tools = []
  experimental = true
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  http_timeout = 30
  jobs = 1
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  node_compile = false
  not_found_auto_install = true
  paranoid = false
  plugin_autoupdate_last_check_duration = "7d"
  python_default_packages_file = "/Users/ramnes/.default-python-packages"
  python_pyenv_repo = "https://github.com/pyenv/pyenv.git"
  raw = false
  trusted_config_paths = []
  quiet = false
  verbose = false
  yes = false
  ci = false
  debug = false
  trace = false
  log_level = "info"
  python_venv_auto_create = false

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false

No warnings found
No problems found

Additional context
None

I'm not able to reproduce, may have been fixed


Correct keras.ops.take behaviour with axis=None

At present the behaviour of  API keras.ops.take with Torch backend is different and incorrect compared to TF and Jax.
This is specifically when input x.ndim=2 and axis = None where x is not flattened by default with axis=None.
Hence changed the implementation and added test case to verify.
Shall fix #18984

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

Comparison is base (dfadf6a) 79.88% compared to head (d99cb03) 57.09%.

Files
Patch %
Lines

keras/backend/torch/numpy.py
0.00%
4 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19088       +/-   ##
===========================================
- Coverage   79.88%   57.09%   -22.80%
===========================================
  Files         339      339
  Lines       35680    35683        +3
  Branches     7011     7012        +1
===========================================
- Hits        28504    20372     -8132
- Misses       5588    13813     +8225
+ Partials     1588     1498       -90

Flag
Coverage Δ

keras
57.09% <0.00%> (-22.65%)
:arrow_down:

keras-jax
?

keras-numpy
57.09% <0.00%> (-0.03%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

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

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


Fix scrollpane area when using fading scroll bar on top of content

When using scrollpane with fading scroll bar with display on top, the widget area should not be reduced (since the scroll bars are expected to be on top). This is not the case for the moment, leading in this situation to have a black border the size of the scroll bar where the scroll bar appears and fade out. This micro PR fixes this.

When fading is used, the scrollbars are always on top. I believe it works correctly without this PR. With the PR, scrollbars overlap. Before PR:
~
After PR:
~

I made this to quick. My PR is wrong in the case you show. I will create a basic test case to show the issue and work out a better PR that would not break other use case.
Sorry for this.

ScrollPaneTest should show the problem, if any.
On Jan 5, 2015 1:32 PM, "Vincent Bousquet"<EMAIL_ADDRESS>wrote:

Closed #2709 https://github.com/libgdx/libgdx/pull/2709.
—
Reply to this email directly or view it on GitHub
https://github.com/libgdx/libgdx/pull/2709#event-214042999.


call python nexus directly rather than shelling out?

Lots of commands use run_nexus (https://github.com/cldf/phlorest/blob/main/src/phlorest/init.py#L262) to shell out to python-nexus. Should we just load the trees and handle them directly in python?

I thought about that. But then I thought the similarity to the "Makefile" might be useful/transparent as well. What convinced me to keep calling python-nexus' cli was that the nexus API isn't particularly convenient for this.

hmm. Looking at the api here, or rather the lack of one, yes, I can see that this would be annoying. Suggests that I should refactor this at some point :/

awesome, thanks -- that looks much clearer!


Support for Accumulo 1.8.1

See #891 for details, 1.8.1 is out now and is the recommended version to support.

Any update on when 1.8 is going to be supported?

This is targeted for an 0.9.5 release by the end of the week.  PR #1148 addresses the update.


Docs update for milestone 0.1.0

[ ] update the sphinx docs
[ ] update the repository readme
[ ] update the examples for easy parsing

Pull Request Test Coverage Report for Build 440

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.3%) to 83.903%

Files with Coverage Reduction
New Missed Lines
%

datmo/cli/driver/parser.py
1
82.61%

datmo/core/controller/code/driver/git.py
8
61.88%

Totals

Change from base Build 419:
-0.3%

Covered Lines:
2554

Relevant Lines:
3044

💛  - Coveralls


--pager="less -R" = command not found

If you add --pager="less -R" to your .ackrc file and the run it  you get something like this:
(venv) mbp0 AWS-43 $ ack TODO
sh: less -R: command not found

Hi @jeffmacdonald, thanks for the report!  This is duplicate of #447; you can solve the issue by omitting the quotes around less -R.

I filed an issue #601 to document the options.

Thanks for the info!


Code mirror position is off by one row

Whenever you start a new challenge, the first time you click on the CodeMirror-code section the line of code selected is off by one row from the cursor position. Once you start typing it seems to fix itself.
This can be extremely frustrating when you're going through multiple challenges relatively quickly.

Thank you for reporting this issue.
This is a standard message notifying you that this issue appears to be very similar to issue #7847 , so I am closing it as a duplicate.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding.


Prevent modifying INT_ENA (and other shared registers) in a way allowing race-conditions

Some drivers allow "splitting" the resource into parts (e.g. SYSTIMER/TIMGs - probably more).
Those parts usually allow modifying shared registers in some way but we need to make sure to atomically modify them (e.g. modifying it in a critical-section)

Ideally the atomicity should be achieved with a more granular construct than critical-section (on multi core chips).
critical_section::with(|| { .... }) disables interrupts on the current core and sets a global "locked" flag, and if the other core also does critical_section::with, it'll disable interrupts there and spin on this global "locked" flag until it is unlocked.
This is effectively a global mutex which is not ideal as it can unnecessarily hold up the other core with the interrupts disabled/delayed.
What should be used instead is some kind of mutex construct that works the same way as critical section but instead of using a global flag, is uses a non-global flag instead. A flag per mutex. Or a flag per register to protect. This way both both cores can lock two separate/unrelated mutexes without having to unnecessarily wait for one another.
Going even a step further it'd be even more ideal if the locking mechanism were left to the user to deal with by providing a &mut only API. I don't want to pay for a lock I don't need one.

Sure - critical_section.with is to bring the big guns in - ideally there would be CAS on registers or something like bit-banding

The current precedent is to use critical_section for this kind of synchronization, we can re open this if this changes.


Added ARM64 Build

Description of the change:
added arm64 on build
Motivation for the change:
#2686 and kiali/issues#1091
we need arm64 because the other repositories have added and are missing here.
from the tests I did it is already working.

This needs to be manually tested with an operator image.

It would be nice to check that the changes work in the Travis CI process as well.
We haven't documented the steps anywhere (but we should). I think they are:

Create image repos in quay (or another registry that supports multi-arch images) for ansible, helm, and scorecard proxy. For each image type, you need repos for the manifest list and one for each architecture (e.g. ansible-operator, ansible-operator-amd64, ansible-operator-s390x, etc.)
Set the following environment variables in the Travis settings for your fork:

ANSIBLE_IMAGE docker image name (e.g. quay.io/joelanford/ansible-operator)
HELM_IMAGE same as above, but for helm
SCORECARD_PROXY_IMAGE same as above, but for scorecard proxy
DOCKER_USERNAME credentials for your repo
DOCKER_PASSWORD credentials for your repo
DOCKER_CLI_EXPERIMENTAL  set to enabled (not sure if this is still required, depends on which version of Docker is running in Travis now)

Make a commit with [travis deploy] in the commit message on the branch with the aarch64 changes.
Check the travis build for your branch in your fork (not the PR build in the operator-sdk repo, since we don't allow PRs to build images in the operator-framework quay repo.)
Post a link in the PR to the Travis build page showing successful Deploy and Deploy multi-arch manifest lists stages with the new architecture.

@BlackMix if you're up for doing this, go for it. Otherwise, we have no problem doing this for you since we need to exercise these steps and add a doc for our own future reference.

@joelanford fine, thanks
I just did this, hoping to finish the deploy :)

well, I merged from master again and updated arm64 build for
removed that fail because markdown tests

all builds success, but have some error in unit tests and not at build.
@joelanford @camilamacedo86
link from deploy

Hi @BlackMix,
The error faced in https://travis-ci.org/github/BlackMix/operator-sdk/jobs/669087015 is because of the coveralls. You have not its TOKEN in your Travis env var configs. You can just comment the line and re-run again.

@camilamacedo86 I added token later, but if I tell you that I was using the token from another repository and was giving an error because of that? my bad haha lol
I woke up now and I started putting it right
deploy now success link

Hi @joelanford,
See: https://travis-ci.org/github/BlackMix/operator-sdk/builds/669385074
WDYT?

will this be merged @joelanford ? so I and others community could use from the official


Add unit tests

This project needs unit tests, the spec/ folder is virtually empty.
So far I have been directly testing my cops in real world codebases, but that won't cut it if I am to allow outside contributions with some peace of mind.

Closing for now, as this may stay open for a while. I will add tests when the time comes.


Run the cocotb example in docs, and cleanup dependencies

PR Summary

Add running the cocotb example as part of the documentation.
Also minimize the dependency to cocotb.
PR Checklist

[N/A] "closes #0000" is in the body of the PR description to link the related issue
[ ] new and changed code is tested
[N/A] relevant changes added to CHANGELOG.md
[N/A] new functionality is documented

The documentation job now runs the testbench but Sphinx-Gallery doesn't capture the output from the testcase, so the output below is missing. I confirmed this when I saw that prints in test_fp8_mul are captured but not in fp8_mul_test_all. It looks like the testcase is run asynchronously, so it may be because of that.
** Note: loading VHPI plugin /usr/local/lib/python3.11/site-packages/cocotb/libs/libcocotbvhpi_nvc.so
     -.--ns INFO     gpi                                ..mbed/gpi_embed.cpp:79   in set_program_name_in_venv        Did not detect Python virtual environment. Using system-wide Python interpreter
     -.--ns INFO     gpi                                ../gpi/GpiCommon.cpp:101  in gpi_print_registered_impl       VHPI registered
     0.00ns INFO     cocotb                             Running on nvc version 1.14.0
     0.00ns INFO     cocotb                             Running tests with cocotb v1.9.1 from /usr/local/lib/python3.11/site-packages/cocotb
     0.00ns INFO     cocotb                             Seeding Python random module with<PHONE_NUMBER>
/usr/local/lib/python3.11/site-packages/_pytest/assertion/rewrite.py:184: UserWarning: Python runners and associated APIs are an experimental feature and subject to change.
  exec(co, module.__dict__)
     0.00ns INFO     cocotb.regression                  Found test test_fp8_mul.fp8_mul_test_all
     0.00ns INFO     cocotb.regression                  running fp8_mul_test_all (1/1)
                                                          Exhaustive test of all inputs.
655370.00ns INFO     cocotb.regression                  fp8_mul_test_all passed
655370.00ns INFO     cocotb.regression                  ***************************************************************************************
                                                        ** TEST                           STATUS  SIM TIME (ns)  REAL TIME (s)  RATIO (ns/s) **
                                                        ***************************************************************************************
                                                        ** test_fp8_mul.fp8_mul_test_all   PASS      655370.00           3.10     211435.85  **
                                                        ***************************************************************************************
                                                        ** TESTS=1 PASS=1 FAIL=0 SKIP=0              655370.00           3.14     208877.91  **
                                                        ***************************************************************************************

** Note:<PHONE_NUMBER>01fs+0: VHPI plugin requested end of simulation

The documentation job now runs the testbench but Sphinx-Gallery doesn't capture the output from the testcase, so the output below is missing. I confirmed this when I saw that prints in test_fp8_mul are captured but not in fp8_mul_test_all. It looks like the testcase is run asynchronously, so it may be because of that.

See sphinx-gallery issue Support for async code #1250.


TypeError: msgpack5 is not a function

Hi, I'm not sure if this is a bug or me not following the examples/guuides properly, but my Javascript client keeps on giving me the following error:

This is the contents of my wwwroot/lib/signalr folder for my aspnetcore webapp:

My bad, in my html page I had the reference to msgpack5.js AFTER signalr-protocol-msgpack.js. Putting it before solves the problem. :-)


Can it install multiple versions of  both 32bit and 64bit Lua?

Can it install multiple versions of  both 32bit and 64bit Lua?

It can't right now, since it simply invokes make to build Lua. Building 32bit executable on 64bit platform should be as easy as passing MYCFLAGS=-m32 to make, so in the next version I'll add flags to set MYCFLAGS for Lua and XCFLAGS for LuaJIT for ease of customization.

I just released hererocks 0.3.0, if you upgrade you can use '--cflags' option:
hererocks lua32 -l 5.3 --cflags=-m32
hererocks lua64 -l 5.3 --cflags=-m64

Note that there are extra steps for running 32bit executables on 64bit platform, for example see this StackOverflow question.

Thanks. I will test it when I get the opportunity


Replication stores too much data in memory

I've been running into an issue where rubyrep (running in jruby) runs out of memory and throws a java exception, no matter how much memory I give it (up to 2GB).
For me, it turned out there was 10 million rows in rr_pending_changes in the slave side of my replication that I wasn't expecting to be there. And while nuking those will probably fix my issue, in general it seems like rubyrep shouldn't have such limitations.
I think it comes down to logged_change.rb, the method update, where it does a "while cursor.next?" loop. It looks like it ends up loading the entire table into memory at once.
Is there any reason there couldn't be a user definable limit here? Say load 1000, process those, and then load the next 1000?
It seems like it already does that on the query itself and the way it loops over the results a row at a time, but it defeats the purpose of doing that if you store an array with an element from every row, you still end up running out of memory.
I don't think I understand what's going on here well enough to submit a patch. Maybe you could just make the while loop break after 1000 iterations and things would just work, but I'm afraid it would break something I don't even know about.

This issue seems like a duplicate of https://github.com/rubyrep/rubyrep/issues/21


Is that any way to set the "try it out" post json body

my action like this:
[FromBodyJson,HttpPost,AllowAnonymous,ValidateRule,Produces("application/json"),Consumes("application/json")]
Task<ResultReturn> Login([Required]string userName,
[Required]string password,
[FromServices]LoginUserService loginService)
then I want to make the userName and password paramters post by json ,but on the swagger UI ,still post  form data with key=value,,
I user nswag version is NSwag.AspNetCore v13.9.4

Create a class that contains Username and Password, and have only one parameter of that type in your method.

Create a class that contains Username and Password, and have only one parameter of that type in your method.

I think I found some way to figure out :
this is my code:
https://github.com/kugarliyifan/Kugar.Core.Web.JsonTemplate/blob/master/Kugar.Core.Web.JsonTemplate/Processors/ActionParamtersToJsonBodyProcessor.cs#L19


it takes long time to synth text for picture

it take a long time to synthesize text in picuture of the provided example, and the most out of the sample is None or the len(text)==0, So how to reduce the synthesize time. Can you give same tips ,thanks.

@lkj1114889770 : It is difficult to debug with limited information.
If you could profile the script to find out which step(s) take a long time, that would help narrow down the problem. Sometimes visualization takes a long time -- so try turning that off, if on.

I've profiled the execution of gen.py. It seems that most of the time is passed in the dot operation in ransac:fit_plane_ransac. It gets called 100+ per image and on very large arrays (pts.shape if often > 200000).

Thanks @fmder : I think the points can safely be sub-sampled heavily without causing any degradation in synthesis quality.


[INLONG-6584][Sort] Add read phase metric and table level metric for MYSQL

Prepare a Pull Request
(Change the title refer to the following example)

[INLONG-6584][Sort] Add read phase metric and table level metric for MYSQL

Fixes #6584

Motivation
The background is that when the entire database is migrated, the business needs to count the indicators of table granularity. Dynamically generate table granularity MetricGroup according to the schema of data records; internally maintain MetricGroup container to realize table granularity reporting indicators.
Modifications
Inherit the original node-level indicators. Connectors with different table-level indicators use different table-level indicator entities, which are determined by the connector itself

For more review info, please see #6585


How to change images size that are rendered using '-grid-layout'?

Hi,
as the title, using normal layout I've done it, but when I switch to the grid layout the size are always the same, every value I use/change inside the api.go. That's weird because the other changes (i.e. int(p.GridPos.W * 40) are affecting the pdf layout obviously, but not the value inside the image size parameters, they work only in not grid layout.
Thanks

Sorry but getting back into the code to find an answer to this is not something I have time for now. No community feedback, so closing the issue.


[mgmt] containerservicefleet release

https://github.com/Azure/sdk-release-request/issues/5050

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-containerservicefleet


Updating documents

Below is the list of updates:

Added contact details in About us.
Updated roles and responsibilities
Updated Use case scenarios (to include when fields exceed set limit)
Updated glossary
Update PPP (need to change the file name later)

Non-documentation updates: (I accidentally committed on this branch :( )

Improve code quality (SLAP)
Added logging
todo add in more test cases

Codecov Report

Merging #222 (fc8231d) into master (5850707) will decrease coverage by 0.00%.
The diff coverage is 62.50%.

@@             Coverage Diff              @@
##             master     #222      +/-   ##
============================================
- Coverage     73.87%   73.87%   -0.01%
- Complexity      772      774       +2
============================================
  Files           109      109
  Lines          2519     2526       +7
  Branches        274      274
============================================
+ Hits           1861     1866       +5
- Misses          584      586       +2
  Partials         74       74

Impacted Files
Coverage Δ

...main/java/seedu/fast/model/person/Appointment.java
91.30% <62.50%> (-2.25%)
:arrow_down:

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 5850707...fc8231d. Read the comment docs.


Quickfix: train file name

Train file name does not match name used in the scripts in simulations (intercity vs Intercity).

@dkouzoup: could you also please rename the file accordingly in the TTOBench repository, within the branch that you are working on for EB?

@zanellia, just had a chat with Dimitris. He advises to keep the name with capital I. Unless, you want to do it I can take care of changing it. I will also address the issue with the unit test that you mentioned...

@juxhino92 sure, will do this, thanks.

Nice, thank you!

Thanks for taking care of it guys. @juxhino92 can you please also update the id name inside the json file? I think it's still with lower case i

Oh, I did not notice it. Sure, I can do it as anyhow I have to see what is wrong with the unittest.


error: no devices/emulators found

USB is overed.
C:\Users\Administrator\Desktop\scrcpy-win64-v1.18>adb usb

daemon not running; starting now at tcp:5037
daemon started successfully
error: no devices/emulators found

C:\Users\Administrator\Desktop\scrcpy-win64-v1.18>scrcpy
INFO: scrcpy 1.18 https://github.com/Genymobile/scrcpy
adb: error: failed to get feature set: no devices/emulators found
ERROR: "adb push" returned with value 1

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#adb-issues

Just updated ethernally script which facilitates wireless mirroring
https://github.com/fusionneur/ethernally
Let me know if you encounter any issue. Thank you


Pusher error: cURL error 35: SSL Issue

Hello Everyone, I've been having issues regarding Laravel Websockets.
Let me give you some context on my code. I am using Laravel Valet for local setup, but when I am trying to post anything, it gives me an error Pusher error: cURL error 35:
This is how my app.js file looks like
window.Echo = new Echo({
  broadcaster: 'pusher',
  key: process.env.MIX_PUSHER_APP_KEY,
  wsHost: process.env.MIX_PUSHER_APP_HOST,
  encrypted: true,
  wsPort: 6001,
  wssPort: 6001,
  disableStats: true,
  enabledTransports: ['ws', 'wss']
})

This is how my broadcasting.php file looks like.
'connections' => [

        'pusher' => [
            'driver' => 'pusher',
            'key' => env('PUSHER_APP_KEY'),
            'secret' => env('PUSHER_APP_SECRET'),
            'app_id' => env('PUSHER_APP_ID'),
            'options' => [
                'cluster' => env('PUSHER_APP_CLUSTER'),
                'encrypted' => true,
                'host' => '<IP_ADDRESS>',
                'port' => 6001,
                'scheme' => 'https',
                'curl_options' => [
                    CURLOPT_SSL_VERIFYHOST => 0,
                    CURLOPT_SSL_VERIFYPEER => 0,
                ],
            ],
        ],

Pusher error: cURL error 35: error:0407008A:rsa routines:RSA_padding_check_PKCS1_type_1:invalid padding (see
https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://<IP_ADDRESS>:6001/apps/local/events?
auth_key=local&auth_timestamp=1651193153&auth_version=1.0&body_md5=a91cf944aa7faa20dde52c231dd35c2e&auth_signat
ure=f69864e5444c1d1e94a22d5b9e07b060b675c23a0234f968f9e90912811d6310.

production.ERROR: cURL error 60: SSL: no alternative certificate subject name matches target host name '<IP_ADDRESS>'

@manshu did you find solution for this problem? I'm also facing the same issue. Please help me to solve this If you find any issue ? Thanks in advance

any solution?

Yes, used soketi instead of this and it worked. i’ll try to make a video in my channel. https://youtube.com/@gurulabsdev

I'm having the exact same issue with socketi though.

I'm having the exact same issue with socketi though.

Have u tried switching to socketi?

I'm using soketi, but i've just realised my PUSHER_SCHEME=http .env variable was still set to https.

I'm having the exact same issue with socketi though.

Have u tried switching to socketi?

I'm using soketi, but i've just realised my PUSHER_SCHEME=http .env variable was still set to https.

Great. I’ve it working now. It was issue with the package. Socketi works like a boss!


When runescape updates everything needs "unlocked" again

Whenever runescape updates and i need to rebuild the project in intellij everything goes back to being locked and upon opening my bank it unlocks everything there. That's not ideal but it's fine, the problem is that things I've gotten before and no longer have in my bank or my inventory are locked until I get them again.
I don't code and I figured out how to compile the project for this plugin specifically so if it's something on my end let me know.

This would most likely be happening because you lose/lack your Runelite profile somehow. On Windows 10 your unlocks are by default stored in C:\Users\[WindowsUsername]\.runelite\profiles\[RuneScapeLogin]\bronzeman-unlocks.txt.
If you continue to have this problem, I suggest backing this file up before updating, and making sure that RuneLite is running with enough permissions to write to this directory. The file is a simple text file of integer values, so you can easily edit it to combine unlocks from different computers or profiles.


Detect kerberoastable accounts using Test-PasswordQuality

I love that enhancement ! Thanks.

I love that enhancement ! Thanks. Question: What is the exact condition to trigger this ? User (or computer accounts) with an supported encryption type to 0 (=RC4) or some value that includes RC4 AND at least one SPN defined ?

Good question! And you are basically right, these conditions must be met:
SamAccountType -eq User -and
ServicePrincipalName.Count -ge 1 -and
SupportsKerberosAESEncryption -eq $false

I am actually not checking if the value includes RC4, the presence of AES128 or AES256 is enough. Computer accounts and GMSAs are intentionally ignored, as they typically have random passwords. Does it make sense, or should the test be improved?
The feature has just been released in version 4.7, you can test it yourself.

I am actually not checking if the value includes RC4, the presence of AES128 or AES256 is enough.

Isn't there a downgrade attack possible?

On our way to get rid of RC4 we found that almost all service accounts (user accounts to run services like SQL or IIS for example) do not have the msDs-supportedEncryptionTypes set at all. This tells the Kerberos service to create RC4 service tickets. If the server where this service is running on, only has the AES encryptions types enabled (this is our security baseline for member servers), every Kerberos authentication from a client will just fail.
So at least the AES bits must be enabled. I also think if besides the AES bits (decimal 24) the RC4 bit is set (decimal 28) a downgrade attack would be possible. So bad for me would be a not set attribute or all other values that enable RC4.
Our red team demonstrated kerberoasting on an service accounts with a weak password and the msDs-supportedEncryptionTypes not set.

On our way to get rid of RC4 we found that almost all service accounts (user accounts to run services like SQL or IIS for example) do not have the msDs-supportedEncryptionTypes set at all.

@CamFlyerCH  This corresponds with my experience. That's why I decided to include this simple test in DSInternals, even though it can be performed very easily using the built-in ActiveDirectory module.

I am actually not checking if the value includes RC4, the presence of AES128 or AES256 is enough.

Isn't there a downgrade attack possible?

@aseigler  Such combination cannot be achieved by just checking the checkboxes in the UI. Nevertheless, I have tried to perform encryption downgrade using a RC4 TGT and I don't think that it is possible to get a ST with session key encrypted using service's RC4 key (NT hash) if AES is also enabled on that service account.
There actually is another known encryption downgrade attack (see Rubeus.exe kerberoast /usetgtdeleg), but this one actually works even for AES-only accounts.
This is the reason why I do not bother testing the RC4 bit if AES bits are present in msDs-supportedEncryptionTypes. But I might be wrong, so the topic is still open to discussion.

I tested the "easy" way and downgraded my client to RC4 only and this resulted, that the session key was RC4, but the ticket itself rested at AES. Rubeus uses this LDAP query for detecting the RC4 enabled accounts:
'(&(samAccountType=805306368)(servicePrincipalName=*)(!samAccountName=krbtgt)(!(UserAccountControl:1.2.840.1135<IP_ADDRESS>3:=2))(!msds-supportedencryptiontypes:1.2.840.1135<IP_ADDRESS>4:=24))'
I agree with that. But the Rubeus way to still achieve receiving an RC4 ticket is concerning. Even for an AES only configured account the tgtdeleg trick works: https://github.com/GhostPack/Rubeus#tgtdeleg
.

Thanks for confirmation, @CamFlyerCH . So the detection algorithm in DSInternals is the same as in Rubeus with /rc4opsec option. Only the description in the report should be changed so that it does not give admins a false hope, because of the tgtdeleg trick.


Exiting Article from search chrashes app

On trying to get back after reading a searched article, the app crashes

Which Android version do you have?

Android 9 on an Android One Phone
Am 15.05.2019 23:52 schrieb Daniel Lang<EMAIL_ADDRESS>Which Android version do you have?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/postillonmedia/app/issues/20?email_source=notifications&email_token=AMAW62KFOWSSIP654E5Y2JTPVSAYRA5CNFSM4HNHIGD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVQBUGA#issuecomment-492837400, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMAW62NW3FVCXMKXQBXSYJ3PVSAYRANCNFSM4HNHIGDQ.

Thank you for clarification.

This issue will hopefully fixed with the next update.


Can't open your app via Windows and Ubuntu

/usr/bin/python3: can't find 'main' module in 'pianoplayer'

I cannot reproduce the problem.
with google I found this which seems relevant:
https://stackoverflow.com/questions/42681801/python-cant-find-main-module/42681826
try to comment out the last if block in pianoplayer/vtk_keyboard.py:
git clone
cd pianoplayer/pianoplayer
edit vtk_keyboard.py
...
cd ..
pip install .
pianoplayer


Fix device of the tensors in set_nms

Motivation
Fixed RuntimeError: indices should be either on cpu or on the same device as the indexed tensor (cpu) when running CrowdDet model on cuda device (the model uses MultiInstanceBBoxHead roi_head)
Modification
Just added tensor.to(device) to fix the issue in set_nms method.
        keep = keep.to(bboxes.device)
        ruler = ruler.to(bboxes.device)

versions:

torch==2.0.1
mmdet==3.0.0
mmengine==0.7.4

@max-unfinity Hi. please fix the lint

@hhaAndroid HI!
I've fixed the lint. But now it is failed on unittest:
ImportError: class CocoPanopticDataset in mmdet/datasets/coco_panoptic.py: Please install petrel_client to enable PetrelBackend
I don't know how to fix this

Hi @max-unfinity,
We'd like to express our appreciation for your valuable contributions to the mmdetection. Your efforts have significantly aided in enhancing the project's quality.
It is our pleasure to invite you to join our community thorugh Discord_Special Interest Group (SIG) channel. This is a great place to share your experiences, discuss ideas, and connect with other like-minded people. To become a part of the SIG channel, send a message to the moderator, OpenMMLab, briefly introduce yourself and mention your open-source contributions in the #introductions channel. Our team will gladly facilitate your entry. We eagerly await your presence. Please follow this link to join us: https://discord.gg/UjgXkPWNqA.
If you're on WeChat, we'd also love for you to join our community there. Just add our assistant using the WeChat ID: openmmlabwx. When sending the friend request, remember to include the remark "mmsig + Github ID".
Thanks again for your awesome contribution, and we're excited to have you as part of our community!

@hhaAndroid HI! I've fixed the lint. But now it is failed on unittest: ImportError: class "CocoPanopticDataset" in mmdet/datasets/coco_panoptic.py: Please install petrel_client to enable PetrelBackend I don't know how to fix this

hi,have solved the problem? could you tell me how to do? thank you


MRG: Add Epochs metadata to Report

This PR adds support for Epochs metadata to mne.Report.

Table can be sorted in ascending/descending order based on any column
Table can be filtered with free-form text input
Alternative "card" view provides a "long" format and is great on devices with a smaller screen
Invidiual columns can be hidden or shown
Individual rows can be copied to the clipboard
The first column (epoch number) is frozen while scrolling right

https://user-images.githubusercontent.com/2046265/147859070-2c952e07-5eef-42f3-bf48-c5f007a869e8.mov
I'm using bootstrap-table which is very much alive and actively maintained.
HTML Tables are generated via Pandas and post-hoc modified to make them work with bootstrap-tables.
Icons are provided via bootstrap-icons. I created a small script that takes as input an original bootstrap-icons CSS and font and embeds the base64-encoded font in the CSS. This makes embedding into the report easier and saves space (we're only shipping a single font instead of multiple fonts and SVGs, as are included in the full bootstrap-icons distribution).
When activating the search box, global keyboard shortcuts (T for showing / hiding all content) are disabled; when leaving the box, keyboard shortcuts are enabled again.
cc @cbrnr @drammock @wmvanvliet @larsoner

Rendered tutorial: https://40096-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/intro/70_report.html#adding-epochs

Rendered tutorial: https://40096-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/intro/70_report.html#adding-epochs

CI error seems to be unrelated.

CI error seems to be unrelated.

@GuillaumeFavelier for the CI issue it seems related to pyvista : https://dev.azure.com/mne-tools/mne-python/_build/results?buildId=17372&view=logs&j=305851a9-a7bb-55db-0042-7e2b6f48aa1c&t=77eb5bf7-3205-5b54-06c6-183c8195d846
can you have a look? 🙏

a not so critical UI glitch when the browser window is large enough:

a not so critical UI glitch when the browser window is large enough:

Ouch, this is ugly, I will have to look into this

@agramfort I cannot seem to reproduce this, which browser is this and what screen resolution / browser window size? Do you need to perform some interactions first for this problem to occur? Thanks

it's with safari on my MBP 16"

Message ID: @.***>

it's with safari on my MBP 16"

Thanks! And it happens right after viewing the report? Not interactions (scrolling etc)?

yes no interaction needed
Message ID: @.***>

pb disappears when I reduce the width of the window

@agramfort I've pushed a (temporary?) change to disable the fixed Epochs # column – I think the problem is related to that. Can you please give it another shot and see if this fixes the issue for you?
For reference, this seems to be an upstream bug, which I can reproduce using the MWE provided here:
https://live.bootstrap-table.com/example/extensions/fixed-columns.html

indeed problem solved !
Message ID: @.***>

@agramfort Thanks!
I've removed everything related to freezing the columns now. This feature can added again later if the problem gets resolved upstream.
Otherwise, this is all good to merge from my end.

I've created an upstream bug report: https://github.com/wenzhixin/bootstrap-table/issues/6014

thx @hoechenberger


Add client variant to 32-bit hotspot builds

This PR makes 32-bit hotspot builds (for windows) to include client and server vm's.
Fixes #871.

I have a question..?   Why was this only just added for 32-bit...?   Was it for a specific app that is only 32-bit?   It just seems like 64-bit would be first, nowadays?   Are you still using 32-bit windows..?

There was a specific user requirement for it, I've merged the 64 bit one as well, some folks will find it useful.

Actually jdk8 won’t build a client vm for 64-bit builds.

Actually jdk8 won’t build a client vm for 64-bit builds.

Ah oK, I'll revert that then.

https://github.com/AdoptOpenJDK/openjdk-build/pull/1114


Add fine-grained port mirroring and associated tests

These are the changes associated with the work done at the Hackfest in Sydney (2016). Have tested against existing tests and added two additional tests for new ACL format and ACL mirror.

Git reports merge conflicts.
Please resolve these conflicts and we can review again.

Conflicts removed, passing all tests except for MAX HOSTS.

Great, thanks, looks better.
There are other changes to the base test class. Why are they necessary?
There is a magic constant: self.mirror_from_port[4294967040 - acl_num] =
port_no. Where does this number come from and why is it necessary?
This change attempts to maintain ACL rules between calls. I'm not opposed
to this in principle but let's not try to do this in this particular change

we should focus solely on the new functionality added. Please remove that
code from this change.

On Sun, May 1, 2016 at 5:09 PM, Ian Welch<EMAIL_ADDRESS>wrote:

Conflicts removed, passing all tests except for MAX HOSTS.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-216016884

Thanks also for reporting the flakeyness in the MaxHosts test (it
occasionally fails in my setup also - OVS 2.4.1). I just committed a change
to the test to make it retry, if you'd like to merge that.
On Sun, May 1, 2016 at 9:56 PM, Bailey, Josh<EMAIL_ADDRESS>wrote:

Great, thanks, looks better.
There are other changes to the base test class. Why are they necessary?
There is a magic constant: self.mirror_from_port[4294967040 - acl_num] =
port_no. Where does this number come from and why is it necessary?
This change attempts to maintain ACL rules between calls. I'm not opposed
to this in principle but let's not try to do this in this particular change

we should focus solely on the new functionality added. Please remove that
code from this change.

On Sun, May 1, 2016 at 5:09 PM, Ian Welch<EMAIL_ADDRESS>wrote:

Conflicts removed, passing all tests except for MAX HOSTS.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-216016884

Ping - is it possible to get these changes actioned please? With the requested changes the diff should be very small and small changes will be easy to verify and merge.

Sorry for the late response, was not subscribed to this thread at first.
All tests seem to be passing on my end - @trungdtbk, could you please verify that everything is fine?

Hi Callum,
I run the tests. Maxhosts and Mirrortest fail if I run all the tests together. But they are successful if run individually.

Hi all;
Let's reset.
Trung - please can you see the comments earlier in this thread, about the
changes to the tests. We want to trim your change down to the minimal one
(I see for example, that you hard code OF1.3 in the test base class - why
is this is necessary)?
If there are changes needed to the base class, please add notes explaining
why to the pull request.
Thanks,
On Mon, May 16, 2016 at 1:26 PM, trungdtbk<EMAIL_ADDRESS>wrote:

Hi Callum,
I run the tests. Maxhosts and Mirrortest fail if I run all the tests
together. But they are successful if run individually.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-219329311

Hi Josh,
I used mininet 2.2.0 and it fails to create OF1.3 switches without specifying 'OpenFlow13'.
Regarding "4294967040" (=MAX of OF ports), I'm not so sure why we use that. But may be for avoiding collision because we use the same dict for both port mirroring and acl mirroring.
I've removed the codes at 707-708 regarding ACL rules.
Cheers,

OK. We need to figure out why you need OF1.3 to be specified. It isn't
required in any other test scenario I know of. Which version of OVS are you
testing with?
We can't accept a change if we're not sure what the code does. :) What do
you intend to happen?
On Mon, May 16, 2016 at 2:09 PM, trungdtbk<EMAIL_ADDRESS>wrote:

Hi Josh,
I used mininet 2.2.0 and it fails to create OF1.3 switches without
specifying 'OpenFlow13'.
Regarding "4294967040" (=MAX of OF ports), I'm not so sure why we use
that. But may be for avoiding collision because we use the same dict for
both port mirroring and acl mirroring.
I've removed the codes at 707-708 regarding ACL rules.
Cheers,
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-219334473

I use ovs 2.0.2
Suppose if we remove the magic number, then the following config will create anomaly.
Port 1 has its traffic mirrored to port 2 by acl rule
Port 3 also has its traffic mirrored to port 2 (unwanted result)
interfaces:
1:
acl_in: 3
2:
3:
acls:
3:
-rule:
actions:
allow: 1
mirror: 2

Per the comments in the test file, OVS 2.0.2 is unsupported.
Please use the recommend version and repeat the test.
On Mon, May 16, 2016 at 2:56 PM, trungdtbk<EMAIL_ADDRESS>wrote:

I use ovs 2.0.2
Suppose if we remove the magic number, then the following config will
create anomaly.
Port 1 has its traffic mirrored to port 2 by acl rule
Port 3 also has its traffic mirrored to port 2 (unwanted result)
interfaces:
1:
acl_in: 3
2:
3:
acls:
3:
-rule:
actions:
allow: 1
mirror: 2
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-219338751

@trungdtbk  and I had a chat. We think that if we use the right version of OVS that we can remove the Openflow13 flag and return the print statements in faucet test back to the previous version (they were changed to work with the older version of OVS).
Regarding the magic number. It is a bit of hack to allow reuse of the mirror_from_port data structure belonging to dp so it can record both the case where all traffic is mirrored and the selective case. It works, might need refactoring in the longer term.

Great (re OVS).
I'm sorry, we can't have the magic number hack. I'm happy to hear another
proposal (or discuss in person on Wednesday). Now that we are at V1.0 and
more and more people are relying on this code, we can't commit experimental
code to the codebase.
On Mon, May 16, 2016 at 3:30 PM, Ian Welch<EMAIL_ADDRESS>wrote:

@trungdtbk https://github.com/trungdtbk and I had a chat. We think that
if we use the right version of OVS that we can remove the Openflow13 flag
and return the print statements in faucet test back to the previous version
(they were changed to work with the older version of OVS).
Regarding the magic number. It is a bit of hack to allow reuse of the
mirror_from_port data structure belonging to dp so it can record both the
case where all traffic is mirrored and the selective case. It works, might
need refactoring in the longer term.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-219341540

I propose that we add an extra data structure to dp.py to do the housekeeping for fine grained mirroring. I think the only other change would be to modify the function add_ports_and_vlans to add any ports that are the target of those acls to the list of ports to be configured.
@trungdtbk is going to work on that tomorrow.

That sounds a lot cleaner. Great!
On Mon, May 16, 2016 at 4:45 PM, Ian Welch<EMAIL_ADDRESS>wrote:

I propose that we add an extra data structure to dp.py to do the
housekeeping for fine grained mirroring. I think the only other change
would be to modify the function add_ports_and_vlans to add any ports
that are the target of those acls to the list of ports to be configured.
@trungdtbk https://github.com/trungdtbk is going to work on that
tomorrow.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-219348007

@trungdtbk has simplified the code (removed extra data structure and optimisations re multiple drop flows).

@trungdtbk just confirmed that it passes all the tests.

I have merged it. This is a useful feature! I look forward to seeing it
used.
On Thu, May 19, 2016 at 9:55 PM, Ian Welch<EMAIL_ADDRESS>wrote:

@trungdtbk https://github.com/trungdtbk just confirmed that it passes
all the tests.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/REANNZ/faucet/pull/72#issuecomment-220279449