component does not re-render

I have a component that renders a list of grid items from an observable array and these items render just fine on componentDidMount()
I have another component in parallel that parses, uploads a file, and triggers a store action which sets the same observable array above.
However, after the trigger the grid items doesn't seem to re-render.
Grid.js
@inject('GridStore')
@observer
class Grid extends Component {

	constructor(props) {
		super(props);
	}

	async componentDidMount() {
		const { GridStore } = this.props;
		await GridStore.loadData();

		console.log('MOUNTED', GridStore.data[0]._source.artboard_name);
	}

	componentDidUpdate() {
		const { GridStore } = this.props;

		console.log('UPDATED', GridStore.data[0]._source.artboard_name);
	}

	render() {
		const { GridStore } = this.props;
		let data = GridStore.data;
		return(
			<div>
				<GridList artboards={data}/>
			</div>
		)
	}
}

class GridList extends Component {
	render() {
		const { artboards } = this.props;
		return(
			<GridWrapper>
				{artboards.map((d,i) =>
					<GridItem key={d._id}>
						<GridItemInner>
							<LazyImg src={d._source.artboard_image_url}/>
						</GridItemInner>
					</GridItem>
				)}
			</GridWrapper>
		);
	}
}

Upload.js
@inject('UploadStore', 'AppStore', 'GridStore')
@observer
class Upload extends Component {

	constructor(props) {
		super(props);

		this.handleFile = this.handleFile.bind(this);
	}

	async handleFile(acceptedFiles) {
		const { UploadStore, AppStore, GridStore } = this.props;
		const file = acceptedFiles[0];

		let formData = new FormData();
		formData.append("sketchFile", file, file.name);

		AppStore.setIsLoading(true);
		let response = await UploadStore.uploadFile(formData);
		await GridStore.loadData()
		AppStore.setIsLoading(false);

		if(response.status == 400) {
			console.error( await response.text() );
		}

	}

	render() {
		return(
			<UploadWrapper>
				<Dropzone
					accept=".sketch"
					onDrop={this.handleFile}
					name="sketchFile"
					className="dropzone">
				<UploadIcon/>
				</Dropzone>
			</UploadWrapper>
		)
	}
}

On mobx's site in the performance section it suggests to break the lists down and that's what I did on my Grid component but I still have to manually refresh the browser to see the change in the grid items.

Note that @autobind might be in the way
Op za 15 dec. 2018 11:28 schreef Taher<EMAIL_ADDRESS>
I have a component that renders a list of grid items from an observable
array and these items render just fine on componentDidMount()
I have another component in parallel that parses, uploads a file, and
triggers a store action which sets the same observable array above.
However, after the trigger the grid items doesn't seem to re-render.
Grid.js
@inject('GridStore')
@observer
@autobind
class Grid extends Component {
constructor(props) {
super(props);
}
async componentDidMount() {
let { GridStore } = this.props;
await GridStore.loadData();
}
render() {
const { GridStore } = this.props;
let data = GridStore.data;
return(

)
}
}
@observer
class GridList extends Component {
render() {
const { artboards } = this.props;
return(

{artboards.map((d,i) =>

)}

);
}
}
Upload.js
@inject('UploadStore', 'AppStore', 'GridStore')
@observer
@autobind
class Upload extends Component {
constructor(props) {
super(props);
}
async handleFile(acceptedFiles) {
let { UploadStore, AppStore, GridStore } = this.props;
let file = acceptedFiles[0];
  let formData = new FormData();
  formData.append("sketchFile", file, file.name);

  AppStore.setIsLoading(true);
  let response = await UploadStore.uploadFile(formData);
  await GridStore.loadData();
  AppStore.setIsLoading(false);

  if(response.status == 400) {
  	console.error( await response.text() );
  }

}
render() {
return(

)
}
}
On mobx's site in the performance section it suggests to break the lists
down and that's what I did on my Grid component but I still have to
manually refresh the browser to see the change in the grid items.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/626, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhKzfJHbFjTt_IFhnG0npdW4A8g8Uks5u5M65gaJpZM4ZUrH4
.

@mweststrate autobind inside Upload.js ?  I tried binding it manually in the constructor which also doesn't trigger the component to re-render.

I added the following log inside componentDidUpdate to check whether the values are updating and it's not until I manually refresh the page.
I have also removed autobind
		const { GridStore } = this.props;
		await GridStore.loadData();

		console.log('MOUNTED', GridStore.data[0]._source.artboard_name);
	}

	componentDidUpdate() {
		const { GridStore } = this.props;

		console.log('UPDATED', GridStore.data[0]._source.artboard_name);
	}

I have also happen to find if I delay reloading the observable data, my grid component DOES re-render with new values!
I'm not sure what's going on here though?
if(response.status == 200) {
	await Promise.delay(4000);
	await GridStore.loadData();
}

can you please attach a code link to code-sandbox?

@ItamarShDev since my usecase is related to async calls it might be very hard to port it to a sandbox. Do you suggest an alternative?

@rehat101 porting isolated case shouldn't be hard. You can stub calls by creating a promise and resolving with some hardcoded json, that simulates async well enough

Hey guys, just wanted to chime in. I was literally smashing my head against the wall with an issue that may be related to yours. I feel others may stumble here looking for a solution as well.
For some reason, my component (which was injected, observed and autobinded) if rendered the first time initially worked fine. However, if a user navigated away and the component was destroyed, the next time it is rendered again, all subsequent updates to the component either via props or state stopped working.
Here is the real gotcha though, if you are using HMR, the issue will be masked in development and everything will work fine. However, once you run the production build, it no longer works correctly since HMR is no longer enabled. I have 100% confirmed this.
Eventually pulling autobind from the component make it function properly after.

Closing for inactivity


AMTagView bug: button frame is equal to CGRectZero

I don't know how I do this, but this is real mistake :)

My bad, checkout latest commit for a fix.

Thank you for operative support!


Focus follows mouse doesn't work for emacs windows

I've just started using kwm and am finding it very useful, but I'm having a problem with emacs. Focus follows mouse works for all my other applications, but if I move the mouse over emacs nothing changes.
I'm using the mac port version of emacs from https://github.com/railwaycat/homebrew-emacsmacport. Both kwm and emacs are installed through homebrew.

Have you seen #40 ?

Yes, I have. I think I have a different problem though, as I can still use keybindings to focus the emacs window and it also gets placed in the correct position. The only thing that doesn't work is the focus follow mouse feature.

Right, can you issue the following command from a terminal and see if it fixes it: kwmc config standby-on-float off

I was having the same issue previously, it seems to be an issue especific to the version of emacs port you are using on osx.
I solved it by installing a different emacs version, the one recommended by spacemacs at https://github.com/syl20bnr/spacemacs#os-x
$ brew tap d12frosted/emacs-plus
$ brew install emacs-plus --with-cocoa --with-gnutls --with-librsvg --with-imagemagick --with-spacemacs-icon
$ brew linkapps

@koekeishiya That didn't fix it :(

@OrBaruk Thanks, I'll try that now.

@OrBaruk That didn't work either. Emacs wasn't managed at all by kwm with that version, even after trying the fix that @koekeishiya mentioned from #40.

This is the rule that I'm currently using in kwmrc
kwmc config add-role AXStandardWindow Emacs

The rule provided by @OrBaruk is the one you need. The rule specified in the linked issue, is for the master branch only.


Isomorphic performance

It looks like performance functions were added in Node 8.5+.
https://nodejs.org/api/perf_hooks.html#perf_hooks_perf_hooks_performance
Right now we are using performance-now for isomorphic performance.now() functionality.
We also have a reference to performance.getEntriesByType().
Replace window.performance with perf_hooks.performance in case of Node.

Cool! So we can get rid of the performance-now polyfill as well

@arinwt @heliocliu Would this be a good first issue for an external contributor?

@tylerbutler probably, but I believe Arin already began working on this and someone already checked in an isomorphic break that depends on this that needs to be resolved soon or backed out

@heliocliu Thanks; makes sense.


System.NullReferenceException in Avalonia.Controls.Utils.KeyboardHelper

Describe the bug
var keymap = TopLevel.GetTopLevel(target)!.PlatformSettings!.HotkeyConfiguration;
TopLevel.GetTopLevel(target) returns null, causing program crash.
System.NullReferenceException
HResult=0x80004003
Message=Object reference not set to an instance of an object.
Source=Avalonia.Controls.DataGrid
StackTrace:
at Avalonia.Controls.Utils.KeyboardHelper.GetPlatformCtrlOrCmdKeyModifier(Control target) in Avalonia.Controls.Utils\KeyboardHelper.cs:line 22
......
at Avalonia.Win32.WindowImpl in src\Windows\Avalonia.Win32\WindowImpl.AppWndProc.cs 769
To Reproduce
Put DataGrid in a popup. Pressing mouse left button on datagrid let popup close immediately.
Expected behavior
Program won't crash in this situation.
Avalonia version
11.0.11
OS
Windows
Additional context
No response

Same issue... Maybe exist some workaround?

Best "workaround" would be to clone the source, try to solve the issue and file a PR.

Yeah, it should be easy to add a null check.
Not necessary the best solution there, as PlatformSettings should not be null for popups.

We observe same issue with version 11.1.3
Stack Trace
KeyboardHelper.cs in KeyModifiers KeyboardHelper.GetPlatformCtrlOrCmdKeyModifier(Control target) at line 28
KeyboardHelper.cs in void KeyboardHelper.GetMetaKeyState(Control target, KeyModifiers modifiers, out bool ctrlOrCmd, out bool shift) at line 15
DataGridColumnHeader.cs in void DataGridColumnHeader.ProcessSort(KeyModifiers keyModifiers, ListSortDirection? forcedDirection) at line 256
DataGridColumnHeader.cs in void DataGridColumnHeader.InvokeProcessSort(KeyModifiers keyModifiers, ListSortDirection? forcedDirection)+() => { } [1] at line 226
DispatcherOperation.cs in void DispatcherOperation.InvokeCore() at line 275
DispatcherOperation.cs in void DispatcherOperation.Execute() at line 263
Dispatcher.Queue.cs in void Dispatcher.ExecuteJob(DispatcherOperation job) at line 137
Dispatcher.Queue.cs in void Dispatcher.ExecuteJobsCore(bool fromExplicitBackgroundProcessingCallback) at line 168
Dispatcher.Timers.cs in void Dispatcher.OnOSTimer() at line 130
Win32Platform.cs in IntPtr Win32Platform.WndProc(IntPtr hWnd, uint msg, IntPtr wParam, IntPtr lParam) at line 138
IntPtr UnmanagedMethods.DispatchMessage(ref MSG lpmsg)
Win32DispatcherImpl.cs in void Win32DispatcherImpl.RunLoop(CancellationToken cancellationToken) at line 111
DispatcherFrame.cs in void DispatcherFrame.Run(IControlledDispatcherImpl impl) at line 121
Dispatcher.MainLoop.cs in void Dispatcher.PushFrame(DispatcherFrame frame) at line 53
Dispatcher.MainLoop.cs in void Dispatcher.MainLoop(CancellationToken cancellationToken) at line 80
ClassicDesktopStyleApplicationLifetime.cs in int ClassicDesktopStyleApplicationLifetime.Start(string[] args) at line 130
ClassicDesktopStyleApplicationLifetime.cs in int ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime(AppBuilder builder, string[] args, ShutdownMode shutdownMode) at line 278
void Program.StartApp(string[] args)
void Program.LunisolarCalendarRemoveEmptyEntries(Program, string[])
void Program.Main(string[] args)


feat: pallet proxy

:warning: This needs #68 & #69 to be merged first!

Adds pallet-proxy to Pop Network runtimes:

[x] devnet
[x] testnet

Rebased after other two listed PRs merged.

This PR introduces ~150 lines of duplicate code, signalling future technical debt. Appreciate not all of it can be refactored away due to different runtime types, but surely at least the parameter_types and ProxyType can be moved to common?
Its easy to just duplicate things, but the burden of later maintenance should be considered.

I am happy to address this.
@evilrobot-01 based on previous comments my understanding was that most likely we would move away from maintaining different runtimes. Though this should not be a excuse for not having a proper code base today.


python3-usr: PYTHONPATH of current build dir is missing

https://github.com/BobBuildTool/basement/blob/9bd2604ecf81f0680597545967618f759828a158/classes/python3-usr.yaml#L7
the current build directory has just something like that:
[libs::something-dev]=/invalid/exec/path/of/libs::something-dev
so it is not possible to use pip installed stuff in the current buildScript.
UPDATE:
https://github.com/BobBuildTool/basement/blob/9bd2604ecf81f0680597545967618f759828a158/classes/python3-usr.yaml#L10
now i realized, that just existing paths are added. so we have to add them when pip install was called already!
in my blueprint/prototype i added all paths, if they are existing or not.

I'm not sure I understand the problem correctly. Why do you need to add the paths in the current workspace? Do you want to build some python package and execute it immediately? Wouldn't this then be more appropriate for the python3-pkg class to add? Especially because it locally built python packages will be under install/...

i want to do something like that:
buildScript: |
    python3InstallPip something
    [python3 will need something]

in pipython3 layer i did this to solve the problem:
    PYTHONROOTDIR=${PWD}/install

    PYTHONPATH="${PYTHONPATH:+$PYTHONPATH}"
    for j in "" "lib-dynload" "site-packages" "dist-packages" ; do
        PYTHONPATH="${PYTHONROOTDIR}$PYTHONDESTLIB${j:+/$j}${PYTHONPATH:+:$PYTHONPATH}"
    done
    export PYTHONPATH
    export PYTHONROOTDIR

    pipInstall()
    {
        # force lib instead of Lib for MSYS
        mkdir -p ${PYTHONROOTDIR}/usr/lib
        python3 -m pip install -q $* \
            --root ${PYTHONROOTDIR} \
            --prefix usr \
            --disable-pip-version-check \
            --no-warn-script-location \
            --ignore-installed
    }

i just added all possible paths, unless if it will exists or not. currently in the python3-pkg class just paths of dependencies are handled/added.


Use filesystem library to deal with paths in Cerata and Fletchgen

Currently the type of paths in Cerata and Fletchgen is std::string. This makes handling them not very ergonomic (see #139), which is why I'd like to suggest we use the filesystem library for this. Since we're already using C++17 features, this should not be an issue. It may however be required for some people to update their compilers.

In my fix for #176 this would also be very useful, since now the paths are just joined using +. That is not very robust.

This is now an issue in Cerata: https://github.com/abs-tudelft/cerata/issues/6

@johanpel You should be able to move the actual issue between organization projects. (The "Transfer issue" link on the right)


Support PKI authentication for Cerebro

Cerebro should implement the same logic as Kibana here: https://github.com/elastic/kibana/issues/7341
This requires ElasticSearch to implement the necessary preconditions: https://github.com/elastic/elasticsearch/issues/34396

closed as we do not use Cerebro any longer


CHESS ACTIVITY : GSoC 2020 CHALLENGE

Solves #592

Full Screen functionality added.

Tutorial Added.

Game Logs visible.

Multiplayer added.

Nice. But it's just a beginning.
My remarks:

The icon for activity should be changed: why not a king pawn
Lot of features are not "Sugarized" (your footer text) and should be integrated in the toolbar
When the window is resize the tutorial don't select correctly pawns
There is permanent scrollbar over the window
You should switch position of fullscreen icon and help icon

@llaske I have made the requested changes.

added game restart functionality from toolbar
background color changes according to buddy color
removed footer and added to toolbar
resizing fixed
permanent scrollbar removed
position of fullscreen and help icon fixed

@ashish0910 @llaske could you please review the changes?

Few remarks :

On starting the game the screen overflows
Possible moves should be highlighted on selecting the character ( Also I would prefer drag and drop rather than current implementation ) .
White colour SVG icon for restart and swap should be used

@ashish0910 @llaske please review
-fixed sizing
-fixed icons
-working on move highlight

Few Remarks :

Possible moves of selected character should be highlighted
Drag and drop feature will be preferred rather than current implementation

Thanks for your proposal but it's too basic.


Update lookup libraries to return CodeableConcept object

Update lookup libraries to return CodeableConcept objects.

Updated lists with codeSystem and code properties so that they are uniform across the different lookup libraries.

Added tests for the new getCodeableConcept functions.

Added utility library to get CodeableConcept tuple from list.

Renamed lookup libaries to match object model name.

Let me know if I should change the names.  I changed toxicity_lookup -> toxicreactiontotreatment_lookup.  It looks ugly but we wanted it all lowercase and to match the object model classname.  Do we want this camelCased?
stage_lookup -> tnmstage_lookup

These libraries will hopefully facilitate hiding the coding within the objects when we modify the getters/setters in the object model(We plan to automate this process...).  This can be merged into master as the new functions are not currently being used.
Pull requests into Flux require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable (N/A) and :white_check_mark:.
Submitter:
Running the Application

[x] Manually tested in Chrome
[x] Manually tested in IE

Documentation

[x] This pull request describes why these changes were made
[x] Recognizes any potential shortcomings/bugs in the description
[x] N/A Cheat sheet is updated
[x] N/A Demo script is updated
[x] N/A Documentation on Wiki has been updated
[x] N/A Additional technical components and libraries are documented and added to wiki as needed

NoteParser

[x] N/A Note parser has been updated if structured phrases change

Code Quality

[x] 4-space indents - convert any tabs to spaces
[x] Use public class field syntax for binding functions - ex. handleChange = () => { ... };
[x] Code is commented

Tests

[x] Existing tests passed
[x] N/A Added UI tests for slim mode
[x] N/A Added UI tests for full mode
[x] Added backend tests for fluxNotes code
[x] N/A Added note parser examples to test new functionality

Reviewer 1: Name

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2: Name

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

This looks good. I think it's going to make things a lot easier to deal with!
[GQ1] Can you make getStatusCodeableConcept and getReasonCodeableConcept in progression_lookup use the status.codeSystem instead of hard-coding the code system? I know they are all the same, but just in case that changes in the future.

[GQ2] I'm torn on codeable_concept.jsx. I'm thinking the code in there should live in the CodeableConcept class within the model as a static function? At the very least, the utility function should live in a class like CodeableConceptUtils as a static function and be in the model directory. This is object model stuff so I feel like it should live somewhere in there. Thoughts?

[GQ3] In toxicity_lookup, can you update getAttributionCodeableConcept to use code for the value (instead of # plus name) and codesystem for the codesystem instead of hard coding it.

[GQ4] getGradeCodeableConcept in toxicity_lookup has the same issue. It should use grade.codeSystem for the codesystem instead of hard coding it.

[GQ5] getAdverseEventCodeableConcept also hard codes codesystem.

Hi Greg, are you looking at the hide-coding-in-objects branch?  There used to be an update-lookup-libraries branch that has been deleted(you may have a local copy.).  I believe GQ1, GQ3, and GQ4 are addressed there.
For GQ2, I can rename and move it to the model directory.  Should I create a lib or utils directory inside of model.
For GQ5, I began adding code and codeSystem into the adverseEventsOptions list but realized it would take a good amount of time to accomplish(This list spans more than 8000 lines).  Do you want me to add those 2 properties to that list?

I was looking at the code using github PR web UI. strange. Fine on GQ5. don't worry about it

For GQ2, not sure what's best yet. I guess just put it directly in model folder

Hmm interesting, are the files updated now or do you still see the previous problems?

not in the pr display of files changed for me

wait maybe. I'm confused now. It does look different

ok. don't worry about those 3 then

I addressed GQ2 and rebased off of updates to master.

This looks good Dan! Your back end tests are thorough. Nice job!


Update stable version in repo from 0.0.0 to 1.1.1

Think about adding version 1.1.1 to stable repository.
Version: stable=0.0.0 (0 days old) => latest=1.1.1 (30 days old)
Installs: stable=0 (0%), latest=54 (22.59%), total=239
Click to use developer portal
Click to edit
Note: This is an automatically generated message and not personally authored by bluefox!

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.1.1 but in the meantime an update to version 1.2.0 is suggested.
So this issue will be closed and replaced by an updated one.
@mcm1957 for evidence


Add support to publish all bijection modules with Scala 2.12

As the finagle 2.12 jars are out, picking them up and publishing all bijection modules in 2.12. Finagle has stopped supporting 2.10 (http://finagle.github.io/blog/2016/04/20/scala-210-and-java7/) and they've since moved a bunch of packages around. I've turned off building / publishing bijectionFinagleMySql for 2.10 to get around this. We could also go the finagle route and stop building / publishing for 2.10.
Also made a couple of changes to some test code to get around some code in ScalaRuntime being dropped.

java.lang.UnsupportedClassVersionError: com/twitter/finagle/mysql/TimestampValue : Unsupported major.minor version 52.0
you need to tell travis to use oraclejdk8 for 2.11 and 2.12 now, it looks like.

@johnynek yeah I just updated that. I've already switched on my machine so didn't run into it :-)

+1 from me.

Thanks @johnynek. I'll cut a new release and publish jars soon.

👍


Print the Fibonacci series for n=20

Add in maths folder

I would like to work on this issue.
Please assign it to me.

I would like to work on this issue as a part of Hacktoberfest.

Hello,
I would like to contribute to solving this issue.  Please assign it to me.

Hello, maybe the Fibonacci series for any number be more interesting. What are you thinking about it?
I would like to solve this issue for the number that the user inserts into the console.

@akshitagupta15june  i can do it please assign it to me

Sir, Can you assign me this for hacktoberfest2020, I would like to work on it?

I would like to work on this issue please allot it to me.

Can You assign this work to me??

I would like to implement this on java, Plz assign me.

Hey, could you assign me this? I would like to work on this

Hello! I would Like to work on this Issue

I want to work on this issue please assign me


Upgrade to IDEA Version 2020.1

There's a few more things that need to be changed for 2020.1 compatibility. I'll address them myself, but thanks for the PR!


Add support for AndroidKeyStore key storage

In your README, you've suggested using cipher.so, but we could also use the AndroidKeyStore to:

Store the symmetric key for API >= 23
Store an asymmetric keypair to wrap a symmetric key for  18 < API < 23

I'd be happy to submit a pull request for it. This would allow users of this library to get a much enhanced layer of security for free!

Well, I'd like to be humble, I've heard about it but I've never scratched the surface of the iceberg, what would the new implementation look like?
It is interesting, if we could work together it would be great

Feel free to open a pull request for this. I've just pushed an update (<IP_ADDRESS>) to address every problem we have encountered so far, thus the library is ready to be modified.
I'll create a separate branch for this

Hey! Any updates on this?

Hi! I was busy with personal stuff last week. Will get started on this module this week!

Hi Andrea,
Things have settled down and I have had a chance to look at this repo a little more closely.
API Support
First thing to address is that the Android Keystore can only be used in API versions 18+
I'm not sure if there are options for different constructors/library modules for different API versions.
We could also discuss having a Builder for Cryptoprefs to support multiple configurations
Key storage for APIs 18-22
Next up, the way Symmetric keystore encryption works in in APIs 18-22 is:

Generate an asymmetric keypair from the Keystore (RSA)
Generate your own symmetric key (AES)
Encrypt the AES key with the RSA Public key, and store it on the device
When encrypting or decrypting with the AES key: load the encrypted AES key from storage => decrypt with the RSA private key => use decrypted AES key
We should decide if we want to create a new preferences file for storing this key or store it in the default shared preferences.
This also brings with it issues with storage space, so we may have to include the READ_EXTERNAL_STORAGE and WRITE_EXTERNAL_STORAGE manifest options

I think that that's a very good base but maybe we could do that on another project, because dropping compatibility for old android versions, requiring additional permissions and migration techniques wouldn't be the best thing for this library.
Once some time has passed we can overcome those issues but for now I'd close this issue, hoping one day to open it again :)

Sadly there was no progress on the topic. Although it is a well written and documented project, but from a security perspective the library can't be considered as a security mitigation without the keystore support, especially with the latest progress of keystore hardware support. BTW, there is no need to use asymmetric encryption for API 23+ (Android 6+). The builder should generate a keystore based AES key (hardware protected) and then use it in order to encrypt pref file elements instead of current approach when it is being passed by the host app. RSA alg, as presented by Arsala, can be used for backward compatibility for Android 4.3-5.* versions.

Sadly there was no progress on the topic. Although it is a well written and documented project, but from a security perspective the library can't be considered as a security mitigation without the keystore support, especially with the latest progress of keystore hardware support. BTW, there is no need to use asymmetric encryption for API 23+ (Android 6+). The builder should generate a keystore based AES key (hardware protected) and then use it in order to encrypt pref file elements instead of current approach when it is being passed by the host app. RSA alg, as presented by Arsala, can be used for backward compatibility for Android 4.3-5.* versions.

Hey!
I've actually been really busy in those last months.
I wanted to re-write this library from scratch in a 2.0 version, and take advantage of the latest AndroidX security components, while achieving better Kotlin compatibility, along with some other sweet things.
Think is, and I'm gonna say that with all my humbleness, I don't know a thing about encryption.
I want to get some background, to study this field and to build a library everybody can use and trust.
Hoping to work with you in the future.
Check this out in a few months, you may like what you'll see :)
And many thanks for supporting my work.

Talking to both of you, @ArsalaBangash and @6d69636861656c 😌

Great to here about the plans! Actually there is a heavy demand among  corporate customers for such a library which supports hw security module out of the box. I'll try  finding time to write a code which will support the the most secure standards for each device/Android version mix and will have granual config support. Hopefully when it will be done you can either include it as an optional wrapper class which protects the encryption key (still not the most secure thing, since the key would be available to the app in run-time) or incorporate it into the latest 2.0 release, so the library will will use a securevault based crypto api in order to encrypt/decrypt the the config areas.

Sadly, security is a really delicate and hard topic for developers facing it for the first time.
I believe it should be much funnier and nicer to pick up.
I originally just picked up code here & there and merged it together in a library.
I'm learning lots of new things and I see this library not having tests, logging operations, asynchronous support, CI, and a full bunch of other stuff.
The release I'm aiming at is going to be a complete rewrite, which will take advantage of a bunch of Hardware/Software features to bring security at a better level.
Specifically, I'm planning to bundle in:

AndroidX Security+Encryption
Kotlin Coroutines
Android Keystore System
Obfuscated Encryption/Decryption Key
Asynchronous & Concurrent Operations

A constant factor in encryption libraries is that if your app can access and edit your preferences, so can a root user with enough knowledge.
This is why I can not bundle a complete security system out of the box, but I have to leave it open so that the user can integrate this library tightly with the target software.
As I'll have finished my current pending project, this will be the next thing I'll get down to :)

Clearly this won't be backward-compatible with any previous version. But I think It'll be worth the effort.

Sounds very promising!

Hey Andrea,
I resonate with the feeling that security and cryptography is not the gentlest topic to pick up, but it's great to see that a re-write is taking place here.
It's lovely to see that there's now a Jetpack library for cryptography. When this didn't exist, a combination of your library's skeleton and Yakiv Mospan's Medium Articles helped me when developing a secure storage solution for my company's Android app.
I agree Michael that "there is a heavy demand among corporate customers for such a library which supports hw security module out of the box."
I wish you all the best with this library and I would encourage you to look for financial backing for the great work you do through options like Open Collective

Rewrite's taking place :)

I've built a really nice base library. KsPref 2.0.0 is under heavy testing.

#13

Right now the implementation looks like this: for K-M devices we use android keystore to store a keypair, wlist on M+ devices we use the keystore to perform the operations.

I'll close the issue as the base idea was completed and implemented. Though, there is more to come :)


move kofistreambot to goodbots

kofistreambot used to integrate Ko-fi into twitch chat
https://help.ko-fi.com/hc/en-us/articles/4411251874193-Connect-Ko-fi-to-Twitch-for-Payment-Alerts

thanks.
https://github.com/arrowgent/Twitchtv-Bots-List/commit/c557c9708edc02ff98e38601498eb4b7a0b85f6d


导入引用后无法使用（Glide找不到包）

Future Task
What is the motivation?
What kind of solution can be considered?
What do you want to discuss?
Please add relevant labels

Bug Reporting
Steps to Reproduce
Actual Results (include screenshots)
Expected Results (include screenshots)
URL
OS details

Device:
OS:

Please add relevant labels

加上 glide 的依赖就可以的。


Qt build improvements and documentation

Various improvements to the Qt build:

always build core libraries as object libraries when building with Qt
build on Windows warningless
embed bitcode on iOS
fix windows symbol exports
add helper scripts (eventually to be used in the CI)

/cc @kiibimees @rinigus

This is really great work. I struggle to build and run from and m1 mac, so any help on build scripts are greatly appreciated. What I see when i run any of the four mac script os this, which doesn't have to be related to this pr even:

This is really great work. I struggle to build and run from and m1 mac, so any help on build scripts are greatly appreciated. What I see when i run any of the four mac script os this, which doesn't have to be related to this pr even:

You need to add Qt to your path, e.g. /Volumes/Extreme/Workspace/SDK/Qt/6.3.0/macos/bin for Qt 6 or /Volumes/Extreme/Workspace/SDK/Qt/5.15.2/clang_64/bin for Qt 5 (replace the path with your Qt installation).

Great! I think the script runs through now for qt5 - how can I run the build?

Great! I think the script runs through now for qt5 - how can I run the build? I'm on a ferry, without connection, but I'll install and try qt6 today as well.

All the scripts use ninja as generator so you can just run ninja and it will build it.

In the readme we could add an example of where to put the terminal and run what. So what I do for instance is opening a terminal at the root of maplibre-gl-native, and then run './platform/qt/scripts/***.sh' followed by 'ninja'.
I'm still not completely sure how I can execute or test the build.

I agree, will extend the README now.

I am planning to test it a touch later. Would be interesting to see whether it still fits Sailfish OS and manual builds on Linux

I am planning to test it a touch later. Would be interesting to see whether it still fits Sailfish OS and manual builds on Linux

It may require a more recent version of CMake (although I did not explicitly increase the requirements). Would that be a problem?

It may require a more recent version of CMake (although I did not explicitly increase the requirements). Would that be a problem?

We will see. I hope that CMake version will not limit it much - would be pity to reduce compatibility because of cmake :)

It may require a more recent version of CMake (although I did not explicitly increase the requirements). Would that be a problem?

We will see. I hope that CMake version will not limit it much - would be pity to reduce compatibility because of cmake :)

I'm happy to also setup a CI pipeline for Sailfish OS if it helps.

I try to run the script, but I get this.. any ideas?

I try to run the script, but I get this.. any ideas?

You chould actually set the installation prefix. Here the files will be "installed".

That is a very good point. I tried to just write something in the installation prefix, and now it compiles

That is a very good point. I tried to just write something in the installation prefix, and now it compiles

OK, will add additional wording to the README 😊

Got the mbgl-qt running now, and the mbgl-test-runner fails at this:
[ RUN      ] Annotations.SymbolAnnotation
[ INFO ]  "{unknown}[General]: GPU Identifier: Apple M1"
Also, I'm not completely sure when I used CMake if I'm currently using qt 5 or 6

Adding Sailfish CI isn't trivial. We are currently using OBS for the builds and I am building for large number of SFOS versions/arch combinations over there. Some builds are at https://build.sailfishos.org/package/show/sailfishos:chum:testing/qmaplibregl
On my own repo, I just started tests with your PR. Will take a bit of time to complete...

So, I am getting errors in the early build stages:
[   88s] + /usr/bin/cmake -DCMAKE_VERBOSE_MAKEFILE=ON -DCMAKE_INSTALL_PREFIX:PATH=/usr -DCMAKE_INSTALL_LIBDIR:PATH=/usr/lib -DINCLUDE_INSTALL_DIR:PATH=/usr/include -DLIB_INSTALL_DIR:PATH=/usr/lib -DSYSCONF_INSTALL_DIR:PATH=/etc -DSHARE_INSTALL_PREFIX:PATH=/usr/share -DCMAKE_SKIP_RPATH:BOOL=ON -DBUILD_SHARED_LIBS:BOOL=ON -DMBGL_WITH_QT=ON -DMBGL_WITH_WERROR=OFF -DCMAKE_INSTALL_PREFIX:PATH=/usr -DMBGL_WITH_QT_HEADLESS=OFF -DMBGL_QT_LIBRARY_ONLY=ON -DMBGL_QT_STATIC=OFF -DMBGL_QT_WITH_INTERNAL_ICU=ON .
[   88s] -- The CXX compiler identification is GNU 8.3.0
[   88s] -- The C compiler identification is GNU 8.3.0
[   88s] -- Check for working CXX compiler: /usr/bin/c++
[   88s] -- Check for working CXX compiler: /usr/bin/c++ -- works
[   88s] -- Detecting CXX compiler ABI info
[   89s] -- Detecting CXX compiler ABI info - done
[   89s] -- Detecting CXX compile features
[   89s] -- Detecting CXX compile features - done
[   89s] -- Check for working C compiler: /usr/bin/cc
[   90s] -- Check for working C compiler: /usr/bin/cc -- works
[   90s] -- Detecting C compiler ABI info
[   90s] -- Detecting C compiler ABI info - done
[   90s] -- Detecting C compile features
[   90s] -- Detecting C compile features - done
[   90s] -- Configuring GL-Native with OpenGL renderer backend
[   90s] CMake Error at vendor/csscolorparser.cmake:16 (target_link_libraries):
[   90s]   Object library target "mbgl-vendor-csscolorparser" may not link to
[   90s]   anything.
[   90s] Call Stack (most recent call first):
[   90s]   CMakeLists.txt:987 (include)
[   90s]
[   90s]
[   90s] -- Configuring incomplete, errors occurred!

This is for CMake cmake-3.11.4+git4. Earlier, it used to work.

According to this ticket, CMake 3.12 may be required: https://gitlab.kitware.com/cmake/cmake/-/issues/18010
The current minimum version set is 3.10.
I think I remember now that we had the same issue in the past. One option is to add another patch file to the build to allow the upstream configuration to be nicer, what do you think?

Got the mbgl-qt running now, which is awesome, and the mbgl-test-runner fails at this:
[ RUN ] Annotations.SymbolAnnotation [ INFO ] "{unknown}[General]: GPU Identifier: Apple M1" zsh: segmentation fault ./mbgl-test-runner
Also, I'm not sure how to select Qt5 or Qt6 now, because I didn't specify the build script like before.
I can see from CMakeCache.txt that Qt5 was used in this case by default

If you build on macOS then Qt version should be the one you added to the path. While tests build, they may not run yet.

I think I remember now that we had the same issue in the past. One option is to add another patch file to the build to allow the upstream configuration to be nicer, what do you think?

I am not very familiar with CMake and cannot judge on what's the best way. Can you tell me what I am expected to patch out? There is one more issue with the build, I'll try to look into it tomorrow and then give proper feedback

One more issue: looks like /usr/include/mbgl is not installed as a part of make install. Which would make it impossible to use qmapboxgl as a library - something we can do now and what was a purpose of earlier CMake scripts development.
Next, after patching your CMake changes with https://github.com/rinigus/maplibre-gl-native/commit/534243a80a78dda02c91d839bf3f742161420dab, I am still getting errors, but now something else than before:
[  270s] CMake Error at platform/qt/qt.cmake:147 (target_link_libraries):
[  270s]   Error evaluating generator expression:
[  270s]
[  270s]     $<PLATFORM_ID:Windows,Emscripten>
[  270s]
[  270s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  270s] Call Stack (most recent call first):
[  270s]   CMakeLists.txt:1089 (include)
[  270s]
[  270s]
[  270s] CMake Error:
[  270s]   Error evaluating generator expression:
[  270s]
[  270s]     $<PLATFORM_ID:Windows,Emscripten>
[  270s]
[  270s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  270s]
[  270s]
[  270s] CMake Error at platform/qt/qt.cmake:147 (target_link_libraries):
[  270s]   Error evaluating generator expression:
[  270s]
[  270s]     $<PLATFORM_ID:Windows,Emscripten>
[  270s]
[  270s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  270s] Call Stack (most recent call first):
[  270s]   CMakeLists.txt:1089 (include)
[  270s]
[  270s]
[  270s] CMake Error:
[  270s]   Error evaluating generator expression:
[  270s]
[  270s]     $<PLATFORM_ID:Windows,Emscripten>
[  270s]
[  270s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  270s]
[  270s]
[  270s] CMake Error at test/CMakeLists.txt:214 (target_link_libraries):
[  270s]   Error evaluating generator expression:
[  270s]
[  270s]     $<PLATFORM_ID:Windows,Emscripten>
[  270s]
[  270s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  270s]
[  270s]
[  270s] CMake Error at benchmark/CMakeLists.txt:57 (target_link_libraries):
[  270s]   Error evaluating generator expression:
[  270s]
[  270s]     $<PLATFORM_ID:Windows,Emscripten>
[  270s]
[  270s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  270s]
[  270s]
[  270s] CMake Error:
[  270s]   Error evaluating generator expression:
[  270s]
[  270s]     $<PLATFORM_ID:Windows,Emscripten>
[  270s]
[  270s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  270s]
[  270s]
[  271s] CMake Error:
[  271s]   Error evaluating generator expression:
[  271s]
[  271s]     $<PLATFORM_ID:Windows,Emscripten>
[  271s]
[  271s]   $<PLATFORM_ID> expression requires one or zero parameters.
[  271s]
[  271s]
[  271s] -- Generating done
[  271s] CMake Warning:
[  271s]   Manually-specified variables were not used by the project:
[  271s]
[  271s]     INCLUDE_INSTALL_DIR
[  271s]     LIB_INSTALL_DIR
[  271s]     LIB_SUFFIX
[  271s]     SHARE_INSTALL_PREFIX
[  271s]     SYSCONF_INSTALL_DIR
[  271s]
[  271s]
[  271s] -- Build files have been written to: /home/abuild/rpmbuild/BUILD/qmaplibregl-0.0.220220.3+ntadej.qt.improvements.20220430135841.2.gea101fb/maplibre-gl-native
[  271s] error: Bad exit status from /var/tmp/rpm-tmp.jChfv4 (%build)

So, to sum up:

Qt compatibility was reduced by timer.cpp changes
CMake is broken now for CMake 3.10 and I couldn't find solution for resolving it

Hi @rinigus, thanks for testing! Unfortunately Qt 5.6 is no longer available for install from Qt directly I may make a docker container with it. In the meantime I added a CI pipeline with CMake 3.10 to make sure this still works.
Hopefully all issues should be fixed now besides the include folder. Which headers from mbgl are actually needed as I do not see any explicit usage. I suspect mbgl::TileServerOptions?
P.S. What is your opinion on renaming the Qt bindings to QMapLibre for consistency?

Re Qt 5.6: no wonder. If you could make a docker with it, that would be helpful
Re include files: without them it is harder to develop against qmaplibregl. TileServerOptions is obvious, but there are also constants, such as mbgl::util::MIN_ZOOM. So far it was that, I haven't used anything else in mapbox-gl-qml
Re QMapLibre: Yes, why not. it would require some packaging adjustments, but it makes sense. Although, it should be done with changing headers as well (qmapbox.hpp and friends). Hmm, maybe too much work for little gain?

PS: But why not to install mbgl headers for now?

Yes, I will restore the headers for now. I disabled them as they seemed to be included by accident.

Please let me know when I should test it against SFOS versions. Thank you for working on it!

Please let me know when I should test it against SFOS versions. Thank you for working on it!

Once the CI is running I need to test the headers and then it will be ready, probably tomorrow

Anything holding this back?

Anything holding this back?

Yes, I need to finalize Linux CI and revert some changes for backwards compatibility.

@ntadej: let me know when to test SFOS builds as well

Sorry for the delay. I very much support building the libs always as OBJECT when building for Qt. Makes the whole lib file management (install/deploy/link) simpler. Perhaps we could go even further and detect STATIC automatically like it's done in Qt:
if(${QtCoreLibraryType} MATCHES STATIC_LIBRARY)
   ...
endid()```

Because the static is useful only in two cases, yes?:
* When trying to produce single target lib file in older CMake.
* When building with static version of Qt (for example iOS).

But, just a proposal. No need to take it too seriously.

Hi @rinigus, @birkskyum, this should be ready to be merged now.
@kiibimees, thank you for your proposal, I'll investigate it (but for a follow-up MR).

Thanks! I will test the builds on SFOS. Will probably be able to report back tomorrow

@ntadej: looks to be compiling nicely on SFOS versions that were working earlier. Thank you!


logit_scale

Thanks for the great work! I wonder when train or evaluate the model, why only multiple the logit_scale on video-sentence score and sentence-frame score but ignore the pixel_word_score? Does this mean that model cares more about the first two and ignore the latter?https://github.com/Ziyang412/UCoFiA/blob/517f838483af544304482bc70ee7ff4886d3dfc6/eval_v2t/modules/modeling_ucofia.py#L409

Hi, we do not consider to configure the logit_scale specifically for any scores (initialized by the following code), thanks!  https://github.com/Ziyang412/UCoFiA/blob/517f838483af544304482bc70ee7ff4886d3dfc6/train/modules/module_clip.py#L380

Sorry, my question is not expressed clearly. What I want to ask is why logit_scale is not multiplied to pixel_word_score, or do you mean that the logit_scale has not been updated?https://github.com/Ziyang412/UCoFiA/blob/517f838483af544304482bc70ee7ff4886d3dfc6/train/modules/modeling_ucofia.py#L368

I think the "logit_scale" is adapted from another codebase and causing the confusion, I think according to the below code, it is updated to the initial value each round.
https://github.com/Ziyang412/UCoFiA/blob/517f838483af544304482bc70ee7ff4886d3dfc6/train/modules/modeling_ucofia.py#L339

haha, COYG! So can I just understand that logit scale does not work in training and testing because it is always 1, and I can remove it.

Yes, I think so. COYG


load-test action

Adds GH action iter8-tools/iter8/load-test.  See https://github.com/iter8-tools/iter8/issues/1146.
Signed-off-by: Michael Kalantar<EMAIL_ADDRESS>

Can be tried using action iter8-tools/iter8/load-test@pull/1150/head (ie, reference the PR).

It is not clear we should use a sub-directory as we do here.  https://docs.github.com/en/actions/creating-actions/publishing-actions-in-github-marketplace suggests 1 action per repository and in the root of the project. On the other hand, it does work to use a sub-directory and there can be multiple actions per repo (see, for example, https://github.com/github/codeql-action).
If we have multiple actions (for example, for different experiment types) we might want an "actions" sub-directory as a way to organize them. However, this would have the negative effect of making references longer.
I suspect that the README should be combined with that at the root level.

This is starting to look very neat.

However, I think we have Iter8 wide semantic versioning issues which we will want to sort out before merging this action. I am not unsure what impact on this action will be in our quest to fix server issues (hopefully none), but it is better to at least under the server implications before the merge.

Meanwhile, I did have a couple of questions...

How are you planning to test functionality? What is the recommended approach for testing GitHub actions? How do others (good packages) test their GitHub actions? I didn't see any tests included, but assume they are WIP.
What is the issue with "." vs "_"? Helm --set also has issues with dotted values. But the workaround there seems to be '90\.5', i.e., use a quoted string with an escaped dot. Is there anything cheap  with GH actions as well instead of the length for loop?

How are you planning to test this action? What is the recommended approach for testing GitHub actions? How do others (good packages) test their GitHub actions? I didn't see any tests included, but assume they are WIP.

I have not attempted this yet beyond testing it manually with another project. I will investigate best practice and agree we need. For now, this can help stimulate further design considerations.

What is the issue with "." vs "_"? Helm --set also has issues with dotted values. But the workaround there seems to be '90.5', i.e., use a quoted string with an escaped dot. Is there anything cheap with GH actions as well instead of the long if/else switches?

The inputs get translated to environment variable names Environment variable names cannnot have . in them. Hence the use of _.  The values can also be provided to the command as arguments. I do not know if this approach would make the name irrelevant. Will explore this.

Updated to latest version of chart and added working test case.

Updated for v0.9

@sriumcp this is ready for review

After discussion, decision to create https://github.com/iter8-tools/iter8-action


Integration problem

I got the error below and could not really find any info on what I did wrong. My skills are not very advanced so there is a slight risk that I am polluting the forum.. anyway…
Logger: homeassistant.setup
Source: setup.py:291
First occurred: 13:53:16 (1 occurrences)
Last logged: 13:53:16
Unable to prepare setup for platform eufy_security.camera: Platform not found (cannot import name 'CameraEntityFeature' from 'homeassistant.components.camera' (/usr/src/homeassistant/homeassistant/components/camera/init.py))

This one as well:
Logger: homeassistant
Source: custom_components/eufy_security/alarm_control_panel.py:240
Integration: Eufy Security (documentation, issues)
First occurred: 16:14:56 (47 occurrences)
Last logged: 16:15:39
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/config/custom_components/eufy_security/init.py", line 60, in update
coordinator.async_set_updated_data(coordinator.data)
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 287, in async_set_updated_data
update_callback()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 325, in _handle_coordinator_update
self.async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 505, in async_write_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 538, in _async_write_ha_state
state = self._stringify_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 511, in _stringify_state
if (state := self.state) is None:
File "/config/custom_components/eufy_security/alarm_control_panel.py", line 240, in state
return CODES_TO_STATES[current_mode]
KeyError: -1

i need to debug this after you share this device with me following readme.
If you prefer simultaneous sharing and debugging together, i will try to set a fixed time every weekends, we can work together

Hi,
Thanks for your reply. However, I now found that the ”Unable to prepare setup…” disappeared after system update. The other one still appears but the integration still works, at least for my purposes. So I think debugging is not needed at this point.


Custom Authenticate Callback Not Called on Error

I posted this on Stackoverflow, but thought I might have more luck here:
I was hoping to utilize the custom callback to handle both successes and failures for logins in Passport's authenticate local strategy, but it looks like it's only called on success.
Here is a snippet of what I'm talking about:
passport.use(new LocalStrategy(
    {usernameField: 'email', passwordField: 'password'},
    function(email, password, done) {
        if(canLogin) done(null, user);
        else done({message: "This is an error message" }, false, { message: "Some Info" });
    }
));

app.post('/login', function(req, res, next) {
      passport.authenticate('local', function(err, user, info) {
         // Only called if err is not set
         res.send("Hello");
    });

From the client, using a bad input:
> curl -d "username=bad&password=input" "http://myurl"
Returns
> [Object object]
A 'good' input returns "Hello."
The docs imply that passport.authenticate allows the error object to trickle down and be handled by the callback, but the above shows otherwise. Am I interpreting this incorrectly?

app.post('/login', passport.authenticate('login'), function(req, res, next) {
res.send("Hello");
});
use this code


[Focused Style] Parameters and response not reset when changing operation

Hello,
When I change operation after click on the TRY button in focused style, PATH PARAMETERS, QUERY-STRING PARAMETERS and the whole RESPONSE area are not reset to empty.
Is there a way to do that ? or am I missing something ?
thanks

this has been fixed now, you may test it using the latest build from dist\rapidoc-min.js

this has been fixed now, you may test it using the latest build from dist\rapidoc-min.js


How to use Xception as backbone to train the net

Hey, I am confused about training the net with Xception as backbone. So I ask you for help about this. Waiting for responce, pls

so am I

Hey, I am confused about training the net with Xception as backbone. So I ask you for help about this. Waiting for responce, pls


p8s random no call out during cthonic vent

No logs for this but it’s a relatively common occurrence. Maybe 10-25% of the time it will not call out the first cthonic vent locations. Not a huge deal by any means.

I believe adding a 0.3s delay to the trigger is a temporary fix for the initial vent while the trigger is still worked on for the later vents.

I believe adding a 0.3s delay to the trigger is a temporary fix for the initial vent while the trigger is still worked on for the later vents.

how ? :(

is that it ?

This issue was resolved in PR https://github.com/quisquous/cactbot/pull/4830


Warning level codacy issues: Position literals first in String comparisons

Decomposition of #12
List of issues: https://app.codacy.com/manual/jdi-testing/jdi-dark/issues?&filters=W3siaWQiOiJMYW5ndWFnZSIsInZhbHVlcyI6W119LHsiaWQiOiJDYXRlZ29yeSIsInZhbHVlcyI6W119LHsiaWQiOiJMZXZlbCIsInZhbHVlcyI6WyJXYXJuaW5nIl19LHsiaWQiOiJQYXR0ZXJuIiwidmFsdWVzIjpbIjU0OTMiXX1d

Learn the problem
Fix them if confirmed or suppress otherwise (ask @hellokitties - she know-how)

COMpleted as a part of task #246


Update/Add GraalVM and graalpython

Tool information

Tool name: GraalVM
Add or update? Add
Desired version: Latest
Approximate size: 37 MiB for GraalPython and 77.4 MB  for GraalVM debs (xz compression)
If this is an add request:

Brief description of tool: A virtual machine replacing OpenJDK with a framework for jit-compilation of other languages.
URL for tool's homepage: https://github.com/oracle/graal and https://github.com/graalvm/graalpython

Virtual environments affected

[ ] macOS 10.15
[x] Ubuntu 16.04 LTS
[x] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Can this tool be installed during the build?
There are prebuilt releases. Building of GraalVM is a bit complicated.
Are you willing to submit a PR?
Maybe. I have a deb repo on GitLab platform building deb packages frem prebuilt releases. It may make sense to port the code to GH Actions and then use it to build the packages. Then the packages can be installed using apt.

@KOLANICH since the packages are fairly small and binaries are provided we shouldn't bake this into the environment. Instead I suggest creating an Action to facilitate the installation.

@KOLANICH since the packages are fairly small and binaries are provided we shouldn't bake this into the environment. Instead I suggest creating an Action to facilitate the installation.

I may be mistaken, but to install GraalVM on Linux a 420 MB heavy file has to be downloaded: https://github.com/graalvm/graalvm-ce-builds/releases/download/vm-20.3.0/graalvm-ce-java11-linux-aarch64-20.3.0.tar.gz
This is the installation instruction: https://www.graalvm.org/docs/getting-started-with-graalvm/linux/
To create Java native images, the GraalVM Native Image tool has to be installed as well: https://www.graalvm.org/reference-manual/native-image/#install-native-image
So it would be really helpful if the latest GraalVM would already be included in the Ubuntu virtual environment.

They are small ... for Microsoft. ;) It's their resources, so their problems that their software wastes them.

They are small ... for Microsoft. ;) It's their resources, so their problems that their software wastes them.

In the opening post you have written about the size:

Approximate size: 37 MiB for GraalPython and 77.4 MB for GraalVM debs (xz compression)

I think @alepauly was more interested in the file size to download. And as I have checked yesterday according to the instructions, the file size is rather 425 MByte.

They are small ... for Microsoft. ;) It's their resources, so their problems that their software wastes them.

In the opening post you have written about the size:

Approximate size: 37 MiB for GraalPython and 77.4 MB for GraalVM debs (xz compression)

I think @alepauly was more interested in the file size to download. And as I have checked yesterday according to the instructions, the file size is rather 425 MByte.


change place where directory with templates should be placed

As I see, to use templates, I should place it in root directory of project (or inside directory with plugin). But what can I do, if I want to place it inside another dir ?
root
--.vscode
--src
  -.ftttemplates

Is it able to do ?
Also, thank you for plugin, its good solution to solve routine :)

@Cabagemage have you tried using the templateFolderPath setting?
I think that this is what you're looking for.

Please see nickiepucels answer above. That should do what you need. If not feel free to come back :)


prepush runs all on setting upstream

I will not say that the problem is with Lefthook.
But the examples you can find in the documentation are just that.
I would like to ask for advice or a solution to this situation.

Suppose we have a mono repository.
We have set up something like this
pre-push:
  commands:
    a:
      root: "libs/a/"
      files: git diff --name-only HEAD @{push}
      run: npm run test
    b:
      root: "libs/b/"
      files: git diff --name-only HEAD @{push}
      run: npm run test

The problem is that when I create a new branch and make changes only for a and push it, all the tests run.
In subsequent pushes, this behavior is not observed.
Perhaps there is a more robust way to get the names of the changed files then git diff --name-only HEAD @{push}, taking into account root.

Hey! Try passing the files to the npm if the test script supports arguments
pre-push:
  commands:
    a:
      root: "libs/a/"
      files: git diff --name-only HEAD @{push}
      run: npm run test {files}
    b:
      root: "libs/b/"
      files: git diff --name-only HEAD @{push}
      run: npm run test {files}

hey!
that is an extremely specific option. what if I only changed package.json?

my current sketch is
if [[ $(git name-rev @{u}) ]]; then return  return $(git diff --name-only HEAD @{push}); else echo "to do"; fi

Ok, so I came up with such inline bash
a:
  root: "lib/any"
  files: if [[ $(git name-rev @{u}) ]]; then return $(git diff --name-only HEAD @{push}); else return $(git diff --name-only origin/HEAD HEAD); fi
  run: npm test

but it isn't working and seems not running
Is this allowed to put bash here?

Hey!

No, you can use sh syntax, but not bash here.
This sounds cool but I find it a bit complicated :thinking:

What if you try this git diff filter? Would it help?
pre-push:
  commands:
    a:
      root: "lib/a/"
      files: git diff-tree -r --name-only --diff-filter=CDMR HEAD origin/master # the changed files
      run: npm test {files} # this is important, otherwise it will just run tests without arguments


Wallet: Clone repo via SSH only

Explain your changes here:
Make it clear within the readme that you must clone the Coda repo via SSH to successfully run git submodule...
Explain how you tested your changes here:
Initially cloned the repo via HTTPS, and failed to build the wallet. Then tried SSH and was successful.
Checklist:

[ ] Tests were added for the new behavior
[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

Closes #0000
Closes #0000

Cheers!

@jkrauska One mistake the markdown URL, can you remove the bang ! so the URLs work properly. I got it mixed up with image urls.


Implement A/B test

@ctrlplusb @birkir @strues This is not an issue, I just want to ask if you ever have implemented A/B test with isomorphic react app before? Any suggestion which library and how to persist the result in server and client?

1

Hi, we're not a lot into testing, but it's pretty related to showing random things.
If you try to show a random thing on the server and the client, you'll get two different results, so here are some options to choose from:
a) pick the random item on the server and pass it to the client.
b) pick a random number on the server and select item based on that number on both the client and server.
c) have an api endpoint give the same random item for both client and server for a limited amount of time or by a token that can be reused (User agent, ip, etc.)
This usually only applies to data, so we keep the same view components and only alter the data. If we want to alter the views based on the data, I think we have to have the client include all possible view scenarios, or use a redirect based approach.

@birkir thanks for the insight bro. Meanwhile I found this react-ab-test it's really useful for implementing A/B test. Yet its support for SSR is still buggy. Maybe I'll make my own component for doing this based on your explanation. Anyway thanks alot.

@diondirza Curious where you ended up here? Did you end up creating your own component?


Update all plugins to BAP 2.0

Not sure what's required for this one, hopefully no too much except the BitVector module.
It also sounds like they're moving away from having, for every module M, a sub-module M.Set and M.Map, which I use extensively in WP, so maybe some re-factoring there as well.

Nothing is required, the Bap.Std interface is backward compatible. It would be good to eventually update cbat_tools to use the more general Bap 2.0 interfaces (i.e., Bap_core_theory and Bap_knowledge) and make VSA available in the knowledge base. But this is not immediately required. Again, old analysis will continue to work for the observed future, it just won't be able to benefit from all the new features.

It also sounds like they're moving away from having, for every module M, a sub-module M.Set and M.Map, which I use extensively in WP, so maybe some re-factoring there as well.

Well, our new modules are not exposing those interfaces (old are still having the same interface), but Base/Core_kernel is making the obsolete, so expect them to disappear from Base/Core_kernel sooner or later. The change is easy, instead of M.Map one should use just Map :)

I've recalled one semantic change that might affect you. When we reify program semantics into BIR we no longer create the ret terms. Now BAP 2.x you won't have any return dst terms, instead it will be reified to a more general call dst with noreturn, where dst is some expression denoting the set of possible return destinations.
Do you think it will affect you?

It might affect us, but I think we can handle it! Thanks again.
On Tue, Aug 20, 2019 at 10:17 AM Ivan Gotovchits<EMAIL_ADDRESS>wrote:

I've recalled one semantic change that might affect you. When we reify
program semantics into BIR we no longer create the ret terms. Now BAP 2.x
you won't have any return dst terms, instead it will be reified to a more
general call dst with noreturn, where dst is some expression denoting the
set of possible return destinations.
Do you think it will affect you?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/draperlaboratory/cbat_tools/issues/58?email_source=notifications&email_token=AA772UCWHM652APYZRCU2U3QFP4I7A5CNFSM4ING4J4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4WOL5Q#issuecomment-523036150,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA772UGESBT4WXPER3DWCKLQFP4I7ANCNFSM4ING4J4A
.

Handled in #65


The TextBox input using the PasswordChar property only accepts one character on Android

Describe the bug
Using a TextBox with the PasswordChar property only allows entering of one character.
This seems to happen only on some (Samsung afaik) devices with enabled "Predictive Text". Disabling this in the OS settings helped the customers who encountered the error.
To Reproduce
Add a simple TextBox with PasswordChar="*"

Expected behavior
Multiple characters can be entered. There should be no difference between TextBoxes with or without PasswordChar.
Avalonia version
11.1.3
OS
Android
Additional context
#15658 seems to be a very similar bug.

Please try 11.2 beta or nightly builds

Thanks. I sent some users an app version with the nightly Avalonia build. Some said it worked now, for one person the Samsung virtual keyboard crashed. I'd close this for now until I have more information about the crash situation.

I had some customers circumventing this issue by deactivating the Samsung Keyboard. The problem does not seem to happen with GBoard for example.
Since I had a lot of trouble with the TextBoxes under Android I also use the native EditText now but this opens another can of worms where there is strange behavior on Samsung devices like backspaces not registering or return not working as well. Did you have any of those as well?

@emmauss @Gillibald comment from the telegram chat


avoid overwriting screenshots with "S" key

Hi Marco!
If I try to take several screenshots from one script using "S" key, I'd only get the last one saved. This PR tries to assign a unique file name by appending a natural number to the base name.
File names would look like this:
📸 Saved rendering window to screenshot.png
📸 Saved rendering window to screenshot1.png
📸 Saved rendering window to screenshot2.png
📸 Saved rendering window to screenshot3.png
📸 Saved rendering window to screenshot4.png
...

or
$ touch screenshot3.png
$ python a_script.py
📸 Saved rendering window to screenshot.png
📸 Saved rendering window to screenshot1.png
📸 Saved rendering window to screenshot2.png
📸 Saved rendering window to screenshot4.png
📸 Saved rendering window to screenshot5.png
...

Summary by CodeRabbit

New Features

Enhanced screenshot functionality to automatically generate unique filenames, preventing overwriting of existing files.

Thanks!!


MAINTAINERS: Tweak nRF BSIM exclude filter

The exclusion filter for bsim tests is currently too broad.
There is "infrastructure" files inside these folders (not just in the top level folder) and users may add more.
So let's change the filter to exclude tests source, cmake and kconfig files, but catch the rest.

@jori-nordic @MaureenHelm sorry for the noise, I just realized one doc and the workflows were not covered.


Upgrade to mesos 1.0 libs, use labels on launched executors/tasks

Based on the conversation in #216 mesos 1.0 introduced the concept of labels that can be applied to executors and tasks.  We should use this to label launched executors with the broker ID.

In addition, we should investigate how to best name executors/tasks to fit the convention things like mesos-dns expect.

@steveniemitz fwiw I like the convention aurora uses when filling out DiscoveryInfo.
discovery: {
visibility: "CLUSTER",
name: "packager-brew.staging.nobody",
environment: "staging",
location: "ZeroZero",
ports: {
ports: [
{
number: 31720,
name: "http",
protocol: "TCP"
}
]
}
}


segmentation fault

thank you for this tool!
a error found when i run it:
[1]    2152960 segmentation fault (core dumped)  ./wl-find-cursor

my desktop is kde Plasma 6.2.4
I am sorry for my debugging skills, this error seems to appear on line 345, hope it helps you
https://github.com/cjacker/wl-find-cursor/blob/f9d99e1bc99a7b85de0c1a3ba092cb0691190ffe/main.c#L345

Thanks for quickly issue report.
GNOME/KDE didn't implement the wayland virtual-pointer protocol, wl-find-cursor depends on layer-shell and virtual-pointer protocols to get mouse position. these compositors can not be supported until they implement virtual-pointer protocol.
But there is still something need to improve in wl-find-cursor, it SHOULD be able to detect the protocol missing and now it get fixed.
By the way, KDE and GNOME its own way to find the cursor or obtain mouse position. for kwin-wayland, you can run a kwin script to obtain the mouse position:
print("Mouse position x=" + workspace.cursorPos.x + " y=" + workspace.cursorPos.y)

And, to debug a c program, you can use something like:
gdb ./wl-find-cusror
# after load, run it.
>r
# after seg fault, get backtrace
>bt

to get more backtrace information in details.


⚠ GitLab Server has degraded performance

In 113ca43, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7130 ms

Resolved: GitLab Server performance has improved in 9fcc7e8.


Array member access with negative key value does not give any compilation error

Description:
Consider the below sample
public function main() {
    byte[2] d = [1, 33];
    d[-1] = 3; // @error list index out of range
}

Note the -1 is used as the key. This program compiles successfully but gives errors at runtime.
Affected Versions:
Current master

Fixed in https://github.com/ballerina-platform/ballerina-lang/pull/33560


Add an N*Diff PPLNS reward variant

The current PPLNS offerings available are "blockavg", "dynamic" or "fixed".  I'd like to add another one to the mix: an N times current block difficulty.  This would require two changes.  The first is an addition to the global.inc.php file as follows:
$config['pplns']['ndiff']['blockcount'] = X; Where X is a number of blocks to consider
The second is a change to the pplns_payout cron adding the following:
else if ($config['pplns']['shares']['type'] == 'ndiff' && $block->getBlockCount() > 0) {     $pplns_target = round($aBlock['difficulty'] * $config['pplns']['diff']['blockcount']); }
For example, to configure a 5N difficulty, a user would choose $config['pplns']['shares']['type'] = 'ndiff'; and $config['pplns']['ndiff']['blockcount'] = 5;

Hey @jonnybravo0311,
It's been a while that I have done any mining but wouldn't your system mean that you potentially overpay users since you are looking at expected block difficulties all the time? If you had 5 lucky rounds and you still pay the block average difficulty of those blocks, you'd have no buffers left for unlucky rounds.
Maybe this is just me not getting it ;-)
Another thing, the cron code port doesn't seem to say what the rounding is you are doing. You multiply the current found block difficulty with a config setting. That doesn't seem to reflect the block average proposed?
Overall I am not certain this would work well. It may be working if all rounds are unlucky but then it may just be an issue with estimating round shares. In which case PPLNS would probably better using found block shares instead of estimated difficulties.
Looking forward to hearing your comments.

Okay it was indeed me being dumber than I should. PPLNS != PPS, sorry for that.
So if I understand correctly you look at ylur current last found block and simply multiply that blocks difficulty by your config setting. I'd rename 'blockcount' to multiplier so people don't get the impression it's related to averages.
The only issue I have here is the lack of change. The other systems were created to ensure a wide variety of shares across blocks to ensure people can't easily guess the pplns target and start hopping pools in order to fill X percent of that before jumps to another pool. You ndiff idea would allow hoppers to jump coins with a low change rate of block difficulties or coins that if changed they only change slightly. Even multiplying that by a random number will keep it guessable if you know the block difficulty and what the PPLNS target was.
Those are my two cents but I am not sold on the idea. Maybe others want to chime in. The change is fairly easy to do but I don't see the benefit yet.

You've got the gist of it, yes.  Most of the BTC PPLNS mining pools use some flavor of N * diff.  For example, kano.is uses 5N.  P2Pool uses what amounts to 3N (they actually cap the share chain at a max 3 day age as well).  The value of the Ndiff system is that you know your "ramp up" and "ramp down" times.  It's also effective at stopping a pool hopper simply by its nature.  Sure, you can jump in at the latter end of a block, but it doesn't really matter too much since there are N number of blocks worth of shares prior to you showing up.  Even if you continually do that, you're not really going to benefit since it's always going to be those total Ndiff shares in the payout calculations.
With the currently available choices, you will gravitate towards a 1N payout (convergence around 100% luck) at least from a BTC mining perspective.  Wouldn't that be far more susceptible to pool hoppers?  With some of the other coins out there that have ever-changing difficulties, then the already available choices for PPLNS payouts should continue to suffice.
Anyway... those are my 2 bits :)

I am closing all old feature requests and ideas since it doesn't seem that anyone is doing a PR against those anyway. Since I won't have the time, no need to keep these ideas open if they are not eventually being implemented.


feat: create tauri app wiring

This PR is wiring up all the existing code / PRs into the same bundle.
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

AWESOMEE... !!!!

 Going to save the real reviews for off of the feature branch, and merge it in here.


Can drawer direction be changed on the fly?

I want to change the drawer direction on the fly when the user changes language in the app and all the layout direction is changed as well?
Can the drawer direction be changed on the fly?

@emildesign have you tried this yourself? Would be interested to know the result.


[FLINK-11908][table] Port window classes into flink-api-java

What is the purpose of the change
This pull request ports window classes into flink-api-java module.
Brief change log

Convert Tumble/Slide/Session/Over related classes to interface and rename the original class from XXX to XXXImpl. For example, add a TumbleWithSize interface and rename the original TumbleWithSize to TumbleWithSizeImpl. We can't port TumbleWithSize directly into the api-java module because ExpressionParser is used in it.
Deprecate def window(window: Window): WindowedTable and add def window(window: GroupWindow): GroupWindowedTable
Deprecate class WindowedTable and add GroupWindowedTable
Add test case, such as WindowValidationTest, to test failures for the creation of window.

Verifying this change
This change is already covered by existing Window IT/UT test cases.
Furthermore, WindowValidationTest has been add to test failures for the creation of window.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot approve description
@flinkbot approve consensus

@sunjincheng121 Thanks a lot for your review. I have addressed all your comments.
@twalthr @sunjincheng121 I have also rebased to the deprecating window pr and delete the deprecated methods and classes in this one. Would be great if you can take a look.
Best, Hequn

Update, rebase to the latest deprecating window PR to unblock review. The updates contain two commits. One is the PR of deprecating window, the other one is the PR of the current issue.

Rebase to the latest flink/master

@twalthr Sorry. I missed the important messages from you. Convert the ExpressionParser into an interface is a good solution. So we can port TumbleWithSize, TumbleWithSizeOnTime, etc. directly into api-java. This also performs a reflection step at just one location.
Good idea!
I will try to update the PR asap.
Best, Hequn

@twalthr @sunjincheng121 I have updated the PR. Would be great if you can take another look.
Some changes are explained by the followings:

To make ExpressionParser.parseExpression() available, the ExpressionParser contains two static methods: parseExpression() and parseExpressionList().
As static methods cannot be overridden, I added a PlannerExpressionParser interface. The PlannerExpressionParser contains two non-static expression parser methods and are overridden by PlannerExpressionParserImpl. So once we get a PlannerExpressionParserImpl instance through refection, we can call parseExpression() in PlannerExpressionParserImpl.
Furthermore, I add a companion class PlannerExpressionParserImpl. This is because we can't new object directly.
Both ExpressionParser and PlannerExpressionParser in api-java are marked as Internal class.

Best,
Hequn

@flinkbot approve all

@twalthr Thanks for your valuable suggestions. I will move on the next PR(convert Table to the interface).


kserve 0.11.0 tests in kubeflow/manifests fails

/kind bug
What steps did you take and what happened:
Updating the kserve manifests in kubeflow/manifests from v0.10 -> v0.11 (PR kubeflow/manifests#2504) causes the CI to fail when executing the kserve tests job.
The complete execution can be found  here.
This error is blocking the Kubeflow 1.8 release as we cannot sync the latest manifests.
What did you expect to happen:
The CI of that repository executes the tests correctly and all tests succeed.
What's the InferenceService yaml:
N/A
Anything else you would like to add:
The root cause could be the recent changes done in kserve_kubeflow.yaml, where a lot of annotations and field values are have replaced kubeflow with kserve. For example:
Here
apiVersion: admissionregistration.k8s.io/v1
kind: ValidatingWebhookConfiguration
metadata:
  annotations:
    cert-manager.io/inject-ca-from: kserve/serving-cert
  creationTimestamp: null

while in the previous version of kserve, the manifests in kubeflow/manifests had a different value:
apiVersion: admissionregistration.k8s.io/v1
kind: ValidatingWebhookConfiguration
metadata:
  annotations:
    cert-manager.io/inject-ca-from: kubeflow/serving-cert

These are just examples, but there are several different fields that changed from kubeflow to kserve.
Potential fix
@annajung found out that reverting back the kubeflow values actually helps solving the CI error.
Environment:

Istio Version: 1.17.x
Knative Version: 1.10.x
KServe Version: 0.11.x
Kubeflow version: 1.8 (testing, not yet released)
Cloud Environment:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube/Kind version:
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

Thanks @DnPlas for reporting the issue!
It turns out kserve namespace is not replaced as kubeflow. We are using replacements instead of vars in kustomization file because the vars is deprecated in kustomize 5.0. We are replacing the namespace in the base kustomization file. Even though we are overriding the namespace in kubeflow overlay, unlike vars ,  in replacements the overrided namespace is not propagated. So the namespace remained as kserve. So, We also need to replace the namespace in the overlay kustomization file.


Failing to download Pre-trained Imagenet

Hello,
Unfortunately, the access to the pretrained imagenet model is no longer open. I cant download the pretrained model described in "./data/scripts/fetch_imagenet_models.sh". Is it the same one as found on ZOO (VGG16)?

you can use the same script in rbgirshick's py-faster-rcnn repository


Compilation error in XmlParser.cs

The new implementation of the Clone() method has a typo of a duplicate equals sign that prevents Avalon Studio from compiling.
var clonedStack = = new Stack<int>(new Stack<int>(_containingTagStart));

It seems that the typo has been fixed in this repo, but the AvalonStudio repo is pointing to a pre-fix version.


Add option to resolve sqs queue by ARN in SqsListener

Fixes #561
:loudspeaker: Type of change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring

:scroll: Description

This PR adds the same functionality that has been added to version 2.4.x in PR #619
This PR introduces the option to use the queue ARN to resolve an SQS queue when specifying an SQSListener. This enables the user to listen to queues from other accounts than the account used by the application/sqs client.
Limitation: As the used AmazonSQS client is region-specific, queues from other regions cannot be resolved.
:bulb: Motivation and Context

See issue #561
Since it is common to access AWS resources via the ARN, this should also be possible for the SqsListener.
:green_heart: How did you test it?
I built the project locally and ran the tests. In addition, I used a small sample application to check locally that the ARN is resolved correctly when specified as value in the SqsListener annotation.
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[x] I updated reference documentation to reflect the change
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

Looks good to me! @tomazfernandes could you please take a look?

Sure, I'll take a look! Thanks @techadri!

@tomazfernandes @MatejNedic Many thanks for the feedback. I have incorporated the suggestions and am looking forward to another review.

Looks great to me, thanks @techadri! If no one else has any further suggestions, I'll squash and merge it.

Thanks @techadri for the contribution! Hopefully more to come!

Thanks @tomazfernandes, I hope that I will be able to contribute from time to time!


Deactivate browser autocompletion for input elements

Taken from here:
For those who stumbled here like I did many times, this works (at least for now):
<input role="presentation" autocomplete="off" />
You don't need to delete ID or have dynamic names.


Fix scroll in some Android devices

Scrollify was not working correctly in some Android Devices. So, I bring you the solution.

Ah I didn't spot this, so I did the same thing although mine detects passive support, keeps existing options and makes sure the removeEventListener still works.
https://github.com/lukehaas/Scrollify/pull/245

But that PR didn't build minified version. You should accept someone else, and close other.

Building a minified version in pull requests isn't very useful since every pull request would be a merge conflict.

then Will you close this PR?

It's not my project, I'm just some dude that sent a different PR.

ok

Closing as it looks like these changes have made their way in via a different PR.


Superclass annotation for method prevails in swagger

I have the below code. The method getDetails is over-ridden in Class B and thus is not available as a web-service. But Swagger still documents it as a valid service. Any idea why does this happen and how to avoid this behavior in Class B?
Note: I cannot hide the operation in class A as this would be needed by other sub-classes.
Class A{
@Path("/details")
String getDetails(){....}
}
@Path("/path1")
Class B extends A{
@Override
String getDetails(){....}
}

Well using the @ApiOperation(hidden=true) with the Override method hides it in the documentation. But @Override disables the API-end-point. so as a suggestion, swagger should not skip such end-points.

Please retest using the latest master.  I believe this has been fixed.


Nautilus / Gparted should access to other Linuxes partitions

Gparted access only to active ext4 partition
Nautilus access to windows partitions mounted under /mnt.
The idea here is to access to Multi boot partitions such as Linux ones

Gparted works on block devices, which Windows mounts are not described as iirc, would probably not be possible as it would require raw disk access in the emulator

@hideyukn88 . My question was for access to other Linux partitions of my physical disk (ext4 formated)

@enigma131, thanks for clarification, so does this help?
https://docs.microsoft.com/en-us/windows/wsl/wsl2-mount-disk
Thanks!

My disk is partitionned: 3 partitions ext4 and 1 NTFS
The second method woks perfectly, thanks. The idea here is to make this possible via GUI... (gnome-disks ?) .. But gnome-disks seems not to work actually (packet gnome-dis-utility is installed)
(gnome-disks:99): GNOME-Disks-ERROR **: 13:49:57.603: Error getting udisks client: Could not connect: .....


Base table or view not found in laravel resource route functions

Bug description
Resource route without index and create there is an issue that Base table or view not found: 1146 ...
But it's working when used Discount::find(id); Same for all tenant model.
Steps to reproduce
I'm using multi database tenency and created api routes with laravel resource route it's an example of my controller function
/**      * Display the specified resource.      *      * @param  \App\Models\Discount $discount      * @return \Illuminate\Http\Response      */     public function show(Discount $discount)     {         return response()->json(new DiscountResource($discount->load([             'products',             'collections',             'buys_products',             'buys_collections',             'gets_products',             'gets_collections',             'customers',             'group',             'countries',             'logs',             'logs.created_user',             'logs.media',         ])), 200);     }
Expected behavior
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'eravel.discounts' doesn't exist (SQL: select *, (CASE
WHEN
(enabled = 1 AND starts_at <= CURDATE() AND ends_at IS NULL) OR
(enabled = 1 AND starts_at <= CURDATE() AND ends_at IS NOT NULL AND ends_at > CURDATE()) THEN "Active"
WHEN enabled = 1 AND ends_at IS NOT NULL AND ends_at <= CURDATE() THEN "Expired"
WHEN enabled = 1 AND starts_at > CURDATE() THEN "Scheduled"
ELSE "Disabled" END
) as status, (CASE
WHEN
(enabled = 1 AND starts_at <= CURDATE() AND ends_at IS NULL) OR
(enabled = 1 AND starts_at <= CURDATE() AND ends_at IS NOT NULL AND ends_at > CURDATE()) THEN 1
ELSE 0 END
) as is_active from discounts where id = 1 and discounts.deleted_at is null limit 1)
Laravel version
8.12
stancl/tenancy version
3.5

Hey!
Thanks for opening the issue. This doesn't seem to be a bug in package, seems to be related to your setup.
If you need quick support, you can visit our Discord community here: https://discord.gg/vHjEyrw
Thanks


Blurred texts on external screen in macOS 10.14

VSCode Version: 1.27
OS Version: macOS 10.14 Mojave

Does this issue occur when all extensions are disabled?: Yes
After I updated my macOS to 10.14, texts are blurred on external display. It's not that obvious from the screenshot but it was much sharper in 10.13.

duplicate of https://github.com/Microsoft/vscode/issues/51132
try to execute this command in terminal and restart:
defaults write -g CGFontRenderingFontSmoothingDisabled -bool NO


readme.md not matching files

Dear nateGeorge,
I want to checkout your data_collection for ssvep via bluetooth with the openbci headset. However, the Readme.md mentions files that are not present. Could you help me? also the yml file and other files are not in the project

Hmm I guess we forgot to push those changes/files. @skuhn002 do you happen to have them on your computer? I'll take a look on mine later.

Apologies for the delay, I was not aware of these comments until recently. I have added the yaml file to the project in a new branch ("add-yaml"), and I will review the readme for other missing files. Can you specify any other files you believe are missing @SjoerdTimovanVliet ?


CY-3381 Implement the external db case

This PR implements the external-db option. The implementation handles both the 6 VMs case (3 Manager + 3 RabbitMQ's) and the 3 nodes case.


When the 🧹 s go marching in

Note that some of these api reports became significantly larger. That's just reflecting reality (forgotten, actually exposed things, being added). Didn't make it part of this effort to do thorough cleanups :)

Actual live footage of @freben


AndResGuard

I took the devs advice and created my own solution to andresguard that doesn't use apktool at all.  Thanks for advice.  Without that push to do it myself, I would have not found my own solution.  I wish the dev of the apktool well.  Stay safe and healthy.

Thanks. Stay safe and feel free to upstream any fixes if you stumble across any.


Fixing one of my frequent typos

In this PR, I have:

[X] Added tests for any new features or bug fixes (if relevant)
[X] Added my changes to the CHANGELOG.md at the top, under the "Unreleased" section
[X] Ensured issues that this PR closes will be automatically closed

Reason for this PR
Docassemble's Continue button has a capital C. When I write the tests, I consistently write it out to match that format.
This clones the I tap to continue step so that it works with lower or upper case.
It also appears that my markdown linter is a little aggressive in reformatting the Changelog.
Links to any solved or related issues
None.
Any manual testing I have done to ensure my PR is working
None (sorry). One of the tests in Reports.feature is updated to use a capital I tap to Continue so if that still passes, all should be ok.

This is a great improvement and something that totally hadn't occurred to us! Thanks for this PR! If it's ok, I might tweak the implementation of the new step to avoid the duplicate code.


As a content admin, I want to export information about people in the database so I can work with it in other systems such as OpenRefine.

testing notes
note: because this export functionality was also retroactively applied to the books export tool, we need to test both tools. repeat these steps on both the items list page and people list page.

From the list page, check that a "Download all as CSV" button appears in the upper right (near "Add Item"). Click it.
You should get a prompt to download a CSV file. Accept and wait while the file is streamed - this could take a while! If it takes more than 1-2 minutes, something is wrong.
Check that the filename of the CSV matches what you were downloading and contains a timestamp, e.g. mep-people-20180920... etc.
Open Excel (but don't open the file itself yet!). Create a new workbook, then choose File -> Import and pick CSV file. Choose the file you downloaded.
Leave the data on the "Delimited" setting. Under "File origin", choose "Unicode (UTF-8)", towards the bottom of the list.
On the next screen, ensure only "comma" is checked as a delimiter. The preview should look like a normal spreadsheet. You can click through and finish the wizard without changing anything else - import the data into your existing sheet in the new workbook.
Skim through the imported data and check that item titles, person names, etc. appear correct, including special characters (for example, Tolstoy's name includes Cyrillic text). Ensure that all the relevant fields are present.
If everything looks right, go back to the admin interface for the model you're testing (books or people).
Select a subset of your results. You can do this using a combination of the checkboxes in the list and the filters at the top right (for example, try filtering people to just Creators, then clicking the top-left checkbox to select the whole page, and finally clicking "Select all 2443 people" at the bottom left to select all Creators).
At the lower left, find the dropdown that says ------------- and from it choose "Export selected to CSV". Click "Go".
From this point, you can repeat steps 2-7. Everything about the process should be identical, except the data included in the download, which should match what you selected. Smaller subsets (a few to a few hundred items) should download much more quickly.

notes for development

Create a new view to generate a CSV export of people. Include the following fields:

name
sort name
mep id
birth year
death year
sex
title
profession
is organization
updated at
notes
is creator (author or other creator of items in the library)
has account (is a library member)
in logbooks (has subscription or reimbursement events)
has card (account has an associated library lending card)

Add a button to people admin view to download the report

Should we include the new "is verified" field in this export?  If it would be easy to add, maybe we might as well--just in case we find a use for it in a later workflow.

@elspethgreen I wondered about that when I added the new verified field. It is indeed easy to include - just updated the test site with a change to include that field in the CSV export.

I think this looks good!


renderDynamic and Html::tag

because of escaping value in html::tag renderDynamic dont work.
What steps will reproduce the problem?
 Html::hiddenInput(null, $this->view->renderDynamic('return 1;'));

What is the expected result?
<input type="hidden" value="<![CDATA[YII-DYNAMIC-0]]>">

What do you get instead?
<input type="hidden" value="<![CDATA[YII-DYNAMIC-0]]>">

Additional info
How fix it right? I see two possible way.
First: detect dynamic attribute in BaseHtml::renderTagAttributes and not escape
Second: add the escaped key to DynamicContentAwareTrait::updateDynamicContent
What do you think?

Q
A

Yii version     <IP_ADDRESS>

PHP version      7.0

Operating system Debian Jessie

Is it still actual?
When I add
<?= Html::hiddenInput(null, $this->renderDynamic('return 1;')); ?>

in view template, I get
<input type="hidden" value="1">

on the web page.

Seems it isn't. Thanks for checking.

I'm able to reproduce it (you need to have full page cache enabled).
Although I'm not sure if we should introduce some magic check for this case. Usually you should use renderDynamic() to whole whole input, not only one attribute.


release: v1.1.1

See CHANGELOG.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: eed53edea45cad22e4a67115052e7b49b021919c
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 9d861d0f4d8f7d269e949fc47a45aa36aad7279e
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: b7e3e7a28fca75eab40dbed85d886b1e61776baa
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 0c7504d183e4a00710382cdf9bf082ce20d852e2
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 1acc5cc6552edfb67494ccecfbe8843fed730714
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 9ce4abfb49c6d0f857ebce5c66ca0d1b950031c2
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


The rancher API does not keep an updated version of the docker-compose (and probably the rancher-compose) after an upgrade

Rancher Version: 0.63.1
Docker Version: 1.10.1
OS: Centos 7
Steps to Reproduce: See this: https://forums.rancher.com/t/api-inconsistent-with-ui/2154
In a nutshell:
Basically create a new stack with a single service.
Use the api to upgrade the stack.
Results:
The same configuration without an updated
Expected:
The documentation indicates the api call should return An updated copy of the environment resource
Notes:
Also attempted to use POST ${environment.links.exportconfig} and it also returns the old config

Can you verify if I've understood what you were doing?

Create a stack with service. Is this using a docker-compose.yml/rancher-compose.yml through add stack or adding a service through the UI?
Upgrade the service through the API
The exported yml files of the stack (what you see when you click on "View Config" in the dropdown of the stack") are not showing the updated service.

Couple of other questions:
Were you on the "View Config" page while you were upgrading through the API?

@feniix Not sure if you are still having issues, if you are, please re-open and provide me some more details on the questions I had asked.
Thanks!

For some reason I did not get notified for the previous message.
I was doing a PoC while I found this issue, I will try to make some time to reproduce again with newer versions


Fix typos in docs

A few typo fixes in docs/advanced-features.md

@dreglad please fix DCO issue

@dreglad please fix DCO issue

@yxxhero Sure.


Update wrangling-overview.md-added a missing word

@MaxAJames - thanks for submitting this correction!