[ZEPPELIN-2300] SHOULD NOT update paragraph text when local change exists

What is this PR for?
SHOULD NOT update paragraph text when local change exists
What type of PR is it?
[Bug Fix]
Todos
NONE
What is the Jira issue?
ZEPPELIN-2300
Related with

https://github.com/apache/zeppelin/pull/2120 (different approach)
https://issues.apache.org/jira/browse/ZEPPELIN-1492 (people are complaining on this issue. but different topic.)

How should this be tested?

Open the same note in 2 browsers (use different sessions e.g normal chrome + secret chrome or firefox + safari)
Prepare a paragraph synchronized among sessions.
Modify text in one browser.
Run the paragraph in different session.

Screenshots (if appropriate)
After

Questions:

Does the licenses files need update? - NO
Is there breaking changes for older versions? - NO
Does this needs documentation? - NO

Hi @1ambda.
What are the implications of this change?
In the scenario you describe, it makes sense.  So if person A is editing a cell and person B is running it at the same time, it will not overwrite the added content from person A, correct?
But will person A notice that person B is working in the same paragraph at the same time? Will he notice that someone ran the paragraph? I think this is the basic requirement because otherwise you will start typing at the same time and then eventually someone gets overwritten.
IMHO the only right way to make this whole websocket collaboration work is to have it as real-time as possible, transmitting each character stroke and showing others peoples cursors (like you have in Google Docs, e.g.).
BTW the gifs are very very tiny in my browser and I can't really tell what's going on there 😄  Are they big for you?

@FRosner I updated GIF.

The basic motivation is, prevent overwriting my local changes when other browsers execute paragraphs.

From the user perspective, this is a critical problem (uncontrolled loss of changes to the code) (ZEPPELIN-1492)

IMHO the only right way to make this whole websocket collaboration work..

I agree, but it requires so much modification. And this PR doesn't prevent going that way.

Understood @1ambda . Thanks for updating the gifs, now it is clear to me except one thing. When the second notebook had user1 written and the first notebook executed the paragraph with admin written, user1 was not replaced by admin at first. But then it was. How do I get the remote changes / discard my local changes because otherwise as soon as I have local changes I cannot get the remote changes, can I?
I also agree with your google docs reference. However a good first step would be an indicator that someone has the same notebook open. Next level is an indicator on a paragraph level. This solves most of the conflicts as I can usually just ask my colleague what he was planning or we are on a webex. After that it would of course be great to see where the cursor is but that's just the frosting of the cake :)
Thanks for taking care!

@FRosner
How do I get the remote changes / discard my local changes

Zeppelin will synchronize text automatically. That's the reason why it doesn't have button for that.
But if Zeppelin keep local changes, we need some buttons or method for synchronizing text. (e.g update text for every 10 seconds if we don't have local change).

It will be another feature we need to develop soon. IMO, it beyond the scope of this PR :)

@AhyoungRyu Could you help review?

Sorry I was traveling @1ambda :(
I understand what you are saying but I still don't understand exactly how it will work after this PR. Consider the following situation:

User A and B edit a paragraph at the same time
User A executes the paragraph
User B will not get his changes overwritten (due to this PR, correct?)
...

Then how does User B ever get back in sync with the server state? How will the conflict be resolved? Will User A then lose his modifications?

@FRosner If auto-save works properly,

User B's local change will update server state
And User A's paragraph text will be updated because he/she doesn't have local change
If they have local change, their paragraph will ignore newly propagated text and will overwrite User b's text later.

I am not sure if this change helps. We still have a race condition and conflict resolution is still "forceful overwrite and don't notify". If I see it correctly:
Before: Who persists the local changes earlier, wins the conflict. (User A wins in the example above)
Now: Who persists the local changes later, wins the conflict.
If this is the expected behaviour and what you wanted to achieve, then LGTM :)
PS: If live editing like in Google docs can't be easily done, we could at least lock the paragraph for all but one user. A lock is granted when a paragraphs is focused. The lock gets released either when the focus is lost or someone else wants to acquire the lock. Changes get persisted before the lock gets released (i.e. either when losing focus or someone else also wants to edit). This way you cannot overwrite any remote changes as the paragraph will always get updated when you click into it and we don't have a race condition because of the lock.

@FRosner Thanks for detailed reply.
We can't get the ultimate feature at once.

locking
realtime notification
...

IMO, This is just the first step. Then other people think about synchronization, and so on as we did. They might think auto save is not ultimate solution. If we want to resolve the whole things at once, we can't merge any of #2120 #2176 #2177..

I get what you are saying and I agree. My only point was that you need to be aware that this change only does the following thing:
Before: Who persists the local changes earlier, wins the conflict. (User A wins in the example above)
Now: Who persists the local changes later, wins the conflict. (User B wins in the example above)
And I was wondering if this is a useful change or not. If this change is considered useful, the PR looks good to me :) If not, we should not merge it.
Again, thanks for your effort and time and I didn't mean to say that we cannot go step by step. I just wanted to challenge this PR.

Let me implement locking and realtime notification soon. I will close this PR.


something compilable

Trying to add wp8 support.
Took sources from https://github.com/phonegap-build/PushPlugin and trying adopt it for current implementation.

@theromis I can't commit to putting this into the master branch as it won't work as there is no connection to the JS side. Once you've got that working we can look at merging.

Ok, will create new pull request once it will work for me.

@theromis okay, sounds good.


Error while starting

Hello, when i try to run ff start (choosen stack)
It return as follows : Error: unable to unlock account 0x85ce51204ead23f3939d1f1c172351e11b138644
Step to reproduce : ff start dev
Ubuntu 20.04 on GCP

Issue solved, by executing ff reset dev,
Closing this issue


Open Telemetry exporters not setting service name

Open Telemetry exporters are setting an empty resource but setting a service name on every span, which used to display fine in Jaeager, unfortunately this is no longer the case. It is now required that service.name is set on the resource in a span batch.

Fixed in #121


Building with a flake on NixOS

I've used Cloudef/mach-flake to set up an env to build pixi, and I get this error when trying to build:
run
└─ run pixi
   └─ zig build-exe pixi Debug native failure
error: error: unable to find Dynamic system library 'atk-1.0' using strategy 'paths_first'. searched paths:
  /home/user/build/pixi/zig-cache/mach/gpu-dawn/release-cdd4a1a/x86_64-linux-gnu/release-fast/libatk-1.0.so

Anyone have any experience here? Thanks so much

This appears to be a dependency of nfd-zig, which is used for opening native file dialogs. https://github.com/foxnne/pixi/blob/ce3f3c676c1ad2ca678f4d6163d05c2e6f73d55b/src/deps/nfd-zig/build.zig#L45-L45
Probably the 'correct' solution to this would be to make a better native file dialog library in Zig which uses dlopen or similar to get access to these functions at runtime, rather than linking dynamically to that library. At some point I want Mach itself to provide something like this.
As for fixing the nixos build, maybe @Cloudef would have an idea for how to do that - but I'm unsure myself how you would go about getting those libraries into the nix build.

Hello, I answered similar question here:
https://ziggit.dev/t/zig2nix-nix-flake-for-packaging-building-and-running-zig-projects/2830/6
Basically:
zig-env = zig2nix.outputs.zig-env.${system} {
    zig = zig2nix.outputs.packages.${system}.zig.master.bin;
    customRuntimeDeps = with pkgs; [ pkg-config ];
    customRuntimeLibs = [ atk ];
};

@slimsag You might be interested in https://flatpak.github.io/xdg-desktop-portal/
Sadly it uses dbus, but this is currently the way in linux for opening file browser and it will use the system's toolkit to do it (separate process). This originates from flatpak, but is not only limited to flatpak. Other than that, mach fully rendering the file dialog would be sure way and work / look the same everywhere.


CSV export ignores column template

The CsvDataModel::getRow ignores column templates.

Yes that is true.
Column templates are used only for rendering into html. If you would like to modify data output, you can use custom renderer (Column::setRenderer()).
Or you can export data with totally different columns that are not used for html output.
I could probably change this bahaviour, but:
1, Latte is templating engine fo HTML
2, Idea of using this engine for exporting data into CSV format sounds wrong bad to me.

Well, coul you add a notice into the documentation?

Right, I will fix that.


Feature request: determine request method

In a servlet REQUEST/requestDS, there should be a way to determine the request type:
dcl-proc myservlet;

    dcl-pi *n;
        request  likeds(REQUESTDS);
        response likeds(RESPONSEDS);
    end-pi;

    printf(request.endpoint);
    printf(request.method); //GET, POST, etc

end-proc;

Of course, if it is PUT, there needs to be a way to get content of the body. (il_getBody(request) ???)

Just to keep note, request.method was added in 46e327eb5684b67ae610ff3041a6aa4c5e9bc192.

https://github.com/NielsLiisberg/ILEastic/commit/46e327eb5684b67ae610ff3041a6aa4c5e9bc192#diff-31f80c15f7e9f88d7f242068acadb407R141

IMHO this should be moved to the registering of the endpoint and the routing else you would do a "second" routing inside your endpoint code for every endpoint  manually.
Having the info in the request ds is ok but the routing should be done automatically.

There are functions for getting the request method (il_getRequestMethod) and for the request body (il_getContent) so this issue can be closed.


Audit dynamic settings in  ReplicationSettings  module

Many dynamic settings we have are not truly dynamic . They will reflect only when a customer pauses-resumes the replication task
For ex : plugins.replication.ops_batch_size, plugins.replication.parallel_reads_per_shard .
We either need to call this out in our documentation or fix implementation to make this really dynamic

Pushed as part of 1.1 release.


/zh-CN/docs/Learn/Getting_started_with_the_web/JavaScript_basics

Request type

[ ] Correction or update

Details

https://developer.mozilla.org/zh-CN/docs/Learn/Getting_started_with_the_web/JavaScript_basics
In the "添加个性化欢迎信息" part
"1.打开 index.html， 在

Hi @Cotory !
I appreciate you trying to report an error, but I can't seem to find this bit of text on the page. Can you log into MDN and fix this yourself? I would be more than happy for you to do that, and it would be really helpful ;-)


Composite Key for idAccessor

Describe the bug
My table needs unique keys and I've found out about idAccessor. However, my unique key is currently a composite key and the accessor can only accept a string of the field name(s). Not sure if it's possible currently but if it is, how do I set it such that it's something like {"id1-id2"}? Thanks a lot for all the hard work!
Expected behavior
idAccessor can be separated by a delimiter/separator so that multiple accessors/fields can be concatenated to form a unique key. Example: idAccessor={"id1|id2"}.

That's an interesting idea. I haven't thought about composite IDs so far.
Maybe we could implement this with a function instead of using a string with a separator?
I'm thinking we could change the idAccessor prop to be of type string | (record: T) => unknown.
This way, you could do something like this:
<DataTable
  // ...
  idAccessor={({ attr1, attr2 }) => `${attr1}:${attr2}`}
/>

What do you think?

Great idea! I think allowing users to optionally set it using a function (on top of the current string implementation) is a good midpoint since composite keys are likely not a common problem for most cases, so no need to break the current approach. But don't worry about this too much if it's non-trivial, I can find workarounds in the meantime.
Thanks for the quick response as well! Really love the work you and your community have put into this project :)

This doesn't break the current approach.
Just published v2.5.6, in which idAccessor can be a string or a function:
idAccessor?: string | ((record: T) => Key);

The Key type comes from React and means string | number if I remember correctly.
Thank you for the kind words and for using Mantine DataTable!


Update: Throw errors on property read before write (fixes #2)

This adds the behavior to throw errors when a property is accessed without first setting its value. So you can do this:
var person = { name: 'Nicholas' };
var fakePerson = leche.fake(person);

var name = fakePerson.name;   // throws an error "Unexpected access of property "name"

// but if you assign it a value first, it's fine
fakePerson.name = 'Nicholas';
var name = fakePerson.name;

CR @j3tan @FredKSchott @mattwiller

Verified that @nzakas has signed the CLA. Thanks for the pull request!

Looks good to me, will definitely help catch those missing stubs. +2


Test issue to test GH Action to add to NWB Project Dashboard

This is a test.

This appears to have worked.


Implement Laravel's Automatic Package Discovery

Updates compose.json to use the Laravel Package Discovery feature eliminating the need to manually add the BugSnagServiceProvider and/or Facade to their respective arrays in config/app.php. You should also change the documentation regarding the onboard/installation process for Laravel apps.
Goal
Reduces the onboarding process to a near zero config setup
Design
Laravel has allowed package discovery since v5.5 released in August 2017.
Changeset
composer.json extras/laravel added
Testing
No tests necessary to implement this change.

The package discovery was intentionally never implemented, because the bugsnag service provider needs to be registered in a specific order.

Thus, package discovery is not suitable, and this PR should not be merged.


Adding and execute runtime benchmarking and weights for BC tewai chain

Done in this #145 #146 #147


Introducing pattern constraint causes "cue export" to spin indefinitely

What version of CUE are you using (cue version)?

cue version v0.4.4-0.20220915174651-ad253ed099e9

       -compiler gc
     CGO_ENABLED 1
          GOARCH amd64
            GOOS darwin
         GOAMD64 v1

Does this issue reproduce with the latest release?
Yes, both in version 0.4.3 and the tip as of today (ad253ed099e94ddcffd07e6f73a76735a96cfaf8).
What did you do?
I wrote a few definitions as described in preceding discussion in the "language" channel of the "CUE" Slack workspace, involving a recursive definition as part of describing Terraform's JSON output format. Building up from there, I wrote definitions for types, values of any type, values of a  particular type, and documents with fields bearing values of those types.
Most of this works fine, up until when I try to use a pattern constraint to define that general document schema, where each of the field must bear one of these values. When I try, the cue export command spins using 2.5 CPUs for as long as twenty minutes. That's the longest I've let it run so far before killing it.
I'll share the definitions that I've written so far, together with a small JSON file that shows input data against which I'd unify these definitions.

CUE definitions
#ValueType:
	// See https://github.com/zclconf/go-cty/blob/3792a7b5328c364b10c6d642e3ea0d9761338740/cty/json.go#L16-L89.
	"bool" | // Primitive types
	"number" |
	"string" |
	["list", #ValueType] |
	["map", #ValueType] | // Complex types
	["object", {[string]: #ValueType}] |
	["object", {[string]: #ValueType}, [...string]] |
	["set", #ValueType] |
	["tuple", [...#ValueType]] |
	"dynamic" // Pseudo-type

#OutputValue: {
	type:         #ValueType
	sensitive:    bool
	description?: string
	value:        _
}

// Make defining schemas more convenient.
#BoolTypedOutputValue: #OutputValue & {
	type:  "bool"
	value: bool
}
#NumberTypedOutputValue: #OutputValue & {
	type:  "number"
	value: number
}
#StringTypedOutputValue: #OutputValue & {
	type:  "string"
	value: string
}
let compositeTypedOutputValue = #OutputValue & {
	#collectionType: "list" | "map" | "object" | "set" | "tuple"
	{
		type: [#collectionType, ...]
	}
}
#ListTypedOutputValue: compositeTypedOutputValue & {
	#collectionType: "list"
	value: [...]
}
#MapTypedOutputValue: compositeTypedOutputValue & {
	#collectionType: "map"
	value: [string]: _
}
#ObjectTypedOutputValue: compositeTypedOutputValue & {
	#collectionType: "object"
	value: [string]: _
}
#SetTypedOutputValue: compositeTypedOutputValue & {
	#collectionType: "set"
	value: [...]
}
#TupleTypedOutputValue: compositeTypedOutputValue & {
	#collectionType: "tuple"
	value: [...]
}

// Output defines the schema for data emitted by the "terraform output -json" command.
// THIS IS THE PROBLEM.
//#Output: [string]: #OutputValue
// Loosen the schema just to compare the behavior.
#Output: [string]: _

_#AWSTFOutput: #Output & {
	let hostedZones = #TupleTypedOutputValue & {
		value: [...{
			id:   string // More elaborate in actual code.
			name: string // More elaborate in actual code.
		}]
	}

	acme_dns_challenge_prover_iam_role:   #ObjectTypedOutputValue
	assume_dns_record_publisher_iam_role: #ObjectTypedOutputValue
	external_dns_route_53_hosted_zones:   hostedZones
	iam_users:                            #ObjectTypedOutputValue
	iam_users: value: [string]: {
		arn:       string // More elaborate in actual code.
		unique_id: string // More elaborate in actual code.
	}
	route_53_hosted_zones: hostedZones
}

// Example use of these definitions unified with real input data
example: _#AWSTFOutput & {
	// NB: This is a JSON object pasted in, redacted in places.
	"acme_dns_challenge_prover_iam_role": {
		"sensitive": false
		"type": [
			"object",
			{
				"arn":       "string"
				"name":      "string"
				"unique_id": "string"
			},
		]
		"value": {
			"arn":       "arn:aws:iam::123456789012:role/some/path/acme-dns-challenge-prover"
			"name":      "acme-dns-challenge-prover"
			"unique_id": "AROAZWV55UJHHXTZOPWHI"
		}
	}
	"assume_dns_record_publisher_iam_role": {
		"sensitive": false
		"type": [
			"object",
			{
				"arn":       "string"
				"name":      "string"
				"unique_id": "string"
			},
		]
		"value": {
			"arn":       "arn:aws:iam::123456789012:role/some/path/dns-record-publisher"
			"name":      "dns-record-publisher"
			"unique_id": "AROAZWV55UJHO5BPDUMOB"
		}
	}
	"external_dns_route_53_hosted_zones": {
		"sensitive": false
		"type": [
			"tuple",
			[
				[
					"object",
					{
						"id":   "string"
						"name": "string"
					},
				],
				[
					"object",
					{
						"id":   "string"
						"name": "string"
					},
				],
			],
		]
		"value": [
			{
				"id":   "ZBIUQ9J8XHK3M"
				"name": "example.com"
			},
			{
				"id":   "Z08962273OP4H9PT589NQ"
				"name": "example-1.com"
			},
		]
	}
	"iam_users": {
		"sensitive": false
		"type": [
			"object",
			{
				"ExternalDNS": [
					"object",
					{
						"arn":       "string"
						"unique_id": "string"
					},
				]
				"cert-manager": [
					"object",
					{
						"arn":       "string"
						"unique_id": "string"
					},
				]
			},
		]
		"value": {
			"daExternalDNS": {
				"arn":       "arn:aws:iam::123456789012:user/some/path/ExternalDNS"
				"unique_id": "AIDAZWV55UJ0123456789"
			}
			"dacert-manager": {
				"arn":       "arn:aws:iam::123456789012:user/some/path/cert-manager"
				"unique_id": "AIDAZWV55UJ0123456789"
			}
		}
	}
	"route_53_hosted_zones": {
		"sensitive": false
		"type": [
			"tuple",
			[
				[
					"object",
					{
						"id":   "string"
						"name": "string"
					},
				],
			],
		]
		"value": [
			{
				"id":   "ZBI0123456789"
				"name": "example.com"
			},
		]
	}
}

Note the comment in the source that reads "THIS IS THE PROBLEM." If I uncomment the line after it to read as follows, cue export gets stuck:
#Output: [string]: #OutputValue

The same problem arises if I write the following:
_#AWSTFOutput: [string]: #OutputValue

That is, I'd like to express that _#AWSTFOutput is a JSON document produced by terraform output -json with a particular set of output values, such that each field in the JSON object must bear a value that conforms to #OutputValue.
Relaxing that pattern constraint to the following allows cue export to proceed:
#Output: [string]: _

However, that fails to satisfy my goal of describing a valid JSON object, as these values could be of any type.
What did you expect to see?
cue export should emit the JSON document as it was, very quickly, without complaining about any schema violations.
What did you see instead?
cue export consumes several CPUs for as long as I'll allow it to run, while emitting no output.

Note that replacing unification with embedding also induces the same problem. That is, if I write the following—with the aforementioned pattern constraint in place—this problem still occurs:
_#AWSTFOutput: {
  #Output

  // ...
}

Here's a shorter version of the above example that seems to exhibit the same problem:
#ValueType:
	["list", #ValueType] |
	["map", #ValueType] |
	["set", #ValueType] |
	["tuple", [...#ValueType]]

#OutputValue: type: #ValueType

#Output: [string]: #OutputValue

#AWSTFOutput: #Output & {
	x: #OutputValue
}

Analysis: it seems that when there is a "cross product" of a disjunction with itself, the cycle detection allows progressing one level deeper, which has the effect of causing quite significant expansion with each "cross product". The issue seems to be that the cycle context is not chained when computing disjunction cross products.
It also seems related to another problem where tracked references (for cycle detection) were inadvertently deleted. This would compound the issue. A fix for that has been sent out.

This will fix the performance issue by detecting cycles on the earliest possible point, preventing the combinatorial computation in the first place:
https://review.gerrithub.io/c/cue-lang/cue/+/543675/1


Remove credentials from secure storage on failed register attempts

Closes #98
If a network connectivity caused registering to fail, removal of credentials from secure storage prevents connecting with those credentials on next app startup.

@erdemyerebasmaz  This may cause issues where the sdk has connected with a specific seed (restored, initialized data etc...) and then on next startup connect with another seed with wrong data already populated.

@erdemyerebasmaz  What is the use case here? Why has the connect failed?

@erdemyerebasmaz What is the use case here? Why has the connect failed?

If there's no network when registering, connect fails, but the mnemonics are already stored before connect is called.
If the user has

exited the app to check on their Network and came back,
did a fresh install on Android

this got addressed with: https://github.com/breez/l-breez/issues/101

they'll connect with the stored mnemonics from the failed register attempt. It's not harmful but it's bad UX.

@erdemyerebasmaz What is the use case here? Why has the connect failed?

If there's no network when registering, connect fails, but the mnemonics are already stored before connect is called.
If the user has

exited the app to check on their Network and came back,

did a fresh install on Android

this got addressed with: Disable application backup on Android #101

they'll connect with the stored mnemonics from the failed register attempt. It's not harmful but it's bad UX.
We can close this PR, think more on it and address it as part of:

Encapsulate each connect flow; register, restore, reconnect/resume #110

I see in that case I think indeed it is better to save the mnemonics after the restore succeed rather than delete them after failure.

Closing.
This issue will be addressed as part of:

https://github.com/breez/l-breez/issues/110


when fling fast, scrolling stops at item's bottom(not last item) irregularly.

it seems URV mis-know the item is last.
my setup is below.
ultimateRecyclerView.setHasFixedSize(false);
ultimateRecyclerView.setLayoutManager(new LinearLayoutManager(this));
ultimateRecyclerView.disableLoadmore();
ultimateRecyclerView.setTranslationY(getResources().getDimensionPixelSize(R.dimen.height_top_layer));//for not using marginTop
could you please help me out?

Do you mean the recyclerview does not stop at the bottom of a item?

thank you for reply.
it means when i fling the recyclerview, sometimes it doesn’t reach at the stop position.
(just middle of items.)

How do you set the stop position?

i did nothing to set stop.
my setup is in fist comment. that's all.
and UltimateViewAdapter is using for card view v7.

I'm a little confuse about it doesn’t reach at the stop position. What is  stop position?

The scroll is supposed to move up and down with fling, but it all stopped in the middle and never goes down anymore.

Maybe it's the same problem as mine #139 ?? There's also a Video available, that shows this issue.


BUG: Defender staying running

Defender staying running.
After use "DisableDefender" then running Defender in Task Manager (and using CPU).
SmartScreen runing too.
Tested on Windows 10 1909

DisableDefender, DisableSmartScreen and bunch of other tweaks need reboot to be fully taken into effect. Generally, it's a good idea to reboot after any reconfiguration in services or HKLM hive. Would it be helpful to add this into FAQ?

Yes, I did. I tried O and O ShutUp software and it Defender and Smartscreen completely disabled after restart.

I've tried it yesterday based on your report and had no problems disabling both. Paste or attach your preset, might be related to #272


docker-compose up error

when im using docker-compose up,console send this message
node_1  | t=2024-03-27T07:26:50+0000 lvl=eror msg="Cannot configure network and rollup-config at the same time.\nStartup will proceed to use the network-parameter and ignore the rollup config.\nConflicting configuration is deprecated, and will stop the op-node from starting in the future.\n" network=base-mainnet rollup_config=mainnet/rollup.json
node_1  | t=2024-03-27T07:26:50+0000 lvl=crit msg="Application failed" message="failed to setup: unable to create the rollup node config: misconfigured L1 Beacon API endpoint: expected L1 Beacon API endpoint, but got none"

@Kamjj Hello, this means you haven't added the required L1 Beacon endpoint correctly to your .env.mainnet or .env.sepolia. If you need help, please come to Base Discord, open a #developer-support ticket and we can show you where it goes. 👍


Getting "Uncaught (in promise)" when validating file format with Yup

Getting "Uncaught (in promise)" when validating file format with Yup
When intentionally browsing a file of format other than the specified one, I get following error in the browser's console:
Uncaught (in promise) Event {isTrusted: true, type: "error", target: null, currentTarget: null, eventPhase: 0, …}

I cannot figure out where this error is originating from.
Here a piece of my code:
Component:
const MAX_RES = 3840 * 2160;
const MAX_FILE_SIZE = 5242880;
const SUPPORTED_IMAGE_TYPES = ['image/png', 'image/jpeg'];
const initialValues = { name: '', image: '' };
const validationSchema = Yup.object({
		name: Yup.string().trim().required('Banner Title is required'),
		image: Yup.mixed()
			.test('imageType', 'Image type is not supported', value =>
				checkFileType(value, SUPPORTED_IMAGE_TYPES)
			)
			.test('imageSize', `Image cannot be more than ${convertBytes(MAX_FILE_SIZE)}`, value =>
				checkFileSize(value, MAX_FILE_SIZE)
			)
			.test('imageResolution', 'Image resolution cannot be more than 4K', value =>
				checkImageResolution(value, MAX_RES)
			)
			.required('Banner Image is required'),
	});

Formik Input Type File Control:
const File = props => {
	const [field, meta] = useField(props);
	const {
		label,
		name,
		required,
		className,
		containerClassName,
		disabled,
		labelClassName,
		multiple,
		accept,
		...rest
	} = props;

	return (

			<Field id={name} name={name} {...rest}>
				{({ form }) => {
					const { setFieldValue, setFieldTouched } = form;
					return (
						<input
							{...rest}
							type='file'
							className={meta.touched && meta.error ? `${className} input-error` : `${className}`}
							onChange={event => setFieldValue(name, event.target.files[0], true)}
							onBlur={() => setFieldTouched(name)}
							multiple={multiple}
							accept={accept}
							disabled={disabled}
						/>
					);
				}}
			</Field>
			{!meta.error && field.value && typeof field.value === 'object' && (

					Selected File:
					{field.value.name.slice(0, 20) +
						'.. ' +
						field.value.name.slice(field.value.name.lastIndexOf('.'))}

			)}
			{label && (
				<label
					htmlFor={name}
					className={`${labelClassName} ${meta.touched && meta.error ? 'label-error' : ''}`}
				>
					{label}
					{required && *}

			)}
			<ErrorMessage name={name} component={TextError} />

	);
};

File.defaultProps = {
	containerClassName: 'form-group custom-file',
	className: 'custom-file-input',
	labelClassName: 'custom-file-label',
	multiple: false,
	accept: 'image/png, image/jpeg',
};

Utility Functions:
export const checkFileSize = (fileObject, maxSize) => {
	if (fileObject && fileObject instanceof File) {
		return fileObject.size <= maxSize;
	}
	return true;
};

export const checkImageResolution = async (fileObject, maxRes) => {
	if (fileObject && fileObject instanceof File) {
		const { width, height } = await new Promise((resolve, reject) => {
			const img = new Image();
			img.onload = () => resolve({ width: img.width, height: img.height });
			img.onerror = reject;
			img.src = URL.createObjectURL(fileObject);
		});
		URL.revokeObjectURL(fileObject);
		return width * height <= maxRes;
	}
	return true;
};

export const checkFileType = (fileObject, supportedTypes) => {
	if (fileObject && fileObject instanceof File) {
		return supportedTypes.includes(fileObject.type);
	}
	return true;
};

When file of different format is browsed, I should get "Image type is not supported" error instead of "Uncaught (in promise)"
CRA App running on Linux Mint 20.1

Software
Version(s)

Formik
2.2.6

React
17.0.1

TypeScript
-

Browser
Chrome

npm/Yarn
yarn

Operating System
Linux Mint 20.1

Did you try your logic outside of Yup? What happens when you apply the following with a non-image URL?
			const img = new Image();
			img.onload = () => resolve({ width: img.width, height: img.height });
			img.onerror = reject;
			img.src = URL.createObjectURL(fileObject);

Without a codesandbox it's basically impossible to guess what your issue is. But if I had to, I'd guess the browser is throwing an error on the Image object, which you'd have to prevent with a try {} catch { reject() } block

Could be even simpler than that, when you reject the Promise, you're creating an error. Without doing Promise.catch() that error is not suppressed and the exception bubbles to as an uncaught exception up through your code.

If you do catch, width and height will be undefined and undefined * undefined will probably also throw an error.

@johnrom I thought the test() functions in yup would run in a serial order so if the file check failed it would just break from there. But eveidently it is not the case. Anyway, I took your advice and wrapped that piece of erroneous code with try/catch and now it works as expected.
Thanks a lot for your time.
PS: I need help with preventing formik onBlur validation when the file browser opens. Currently if I open the file browser, I immediately get the "required" error. I do want to show the error but only if the user does not choose any file (cancels the file browser).
Can you please tell me any workaround for this situation?


feat: add chunking on update, retrieve, remove and insert

Yeah, I guess we just need to sit together for this one.


python3 compatibility

I installed cm-api for python3 using pip without issues
pip install cm-api

However, following the example on the website , I immediately get an error:
>>> from cm_api.api_client import ApiResource
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Program Files\Python\lib\site-packages\cm_api\api_client.py", line 23, in <module>
    from cm_api.http_client import HttpClient, RestException
  File "C:\Program Files\Python\lib\site-packages\cm_api\http_client.py", line 182
    except urllib2.HTTPError, ex:
                            ^
SyntaxError: invalid syntax

The error is obvious, python3 does not allow ','(comma) as syntax to declare an alias of an exception.
It seems cm-api is NOT compatible with python3 since this commit: dcc11dfcddc0d565084f19e90d4fe62a39b9fe73
I could not find a reason for reverting the compatibility.
So either this lib should be removed from the pip3 repositories or this commit should be 'unreverted'?

Still a problem more than year after initial report.
Did anyone find a workaround other that using Ansible controller executable running on Python2.7?


Last daily hour accumulates daily production

The plugin is working well, except for one recurring issue I encounter daily: the 'DTU AC daily energy' value accumulates the entire day's production during the last hour, as shown in the screenshots below. It seems the plugin is calculating the daily production twice during the final connected hour.
The sensor value seems to be okay (last screenshot), so it seems to be an issue with the energy dashboard?
DTU Pro-S
Hardware H09.06.11
Software V00.04.03

I have seen a similar graph already. The issue seems to be that once the inverter goes offline the DTU reports back a different value. Once the "drop" happens, Home Assistant thinks that the sensor has been reset since and accumulates the result. This is why you see the increase.
The question is: why does the drop happen in the first place.
Can you maybe post the reponse from the get-real-data-new request from the hoymiles-wifi lib https://github.com/suaveolent/hoymiles-wifi before and after the drop?

**Please not that I dont use al DC ports on one inverter:
This is between sunset and midnight:**
TU Response: device_serial_number: "10FD********"
timestamp:<PHONE_NUMBER>
ap: 1
firmware_version: 1
tgs_data {
serial_number: 2145**********
warning_number: 9
modulation_index_signal: 24772608
}
tgs_data {
serial_number: 2145**********
warning_number: 3
modulation_index_signal: -63
}
pv_data {
serial_number: 2145**********
port_number: 1
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 2
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 3
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 4
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 5
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 6
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 1
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 2
error_code: 589824
}
pv_data {
serial_number: 21451**********
port_number: 3
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 4
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 5
error_code: 589824
}
pv_data {
serial_number: 2145**********
port_number: 6
error_code: 589824
And this is when the inverters are active:
DTU Response: device_serial_number: "10FD**********
timestamp:<PHONE_NUMBER>
ap: 1
firmware_version: 1
tgs_data {
serial_number: 2145**********
firmware_version: 1
voltage_phase_A: 2273
voltage_phase_B: 2326
voltage_phase_C: 2280
voltage_line_AB: 3969
voltage_line_BC: 3988
voltage_line_CA: 3957
frequency: 4997
active_power: 107
current_phase_A: 1
current_phase_B: 1
current_phase_C: 1
power_factor: 999
temperature: 102
warning_number: 5
link_status: 1
modulation_index_signal: -56
}
tgs_data {
serial_number: 2145**********
firmware_version: 1
voltage_phase_A: 2273
voltage_phase_B: 2279
voltage_phase_C: 2324
voltage_line_AB: 3958
voltage_line_BC: 3985
voltage_line_CA: 3969
frequency: 4997
active_power: 221
current_phase_A: 3
current_phase_B: 3
current_phase_C: 3
power_factor: 1000
temperature: 111
warning_number: 2
link_status: 1
modulation_index_signal: -55
}
pv_data {
serial_number: 2145**********
port_number: 1
voltage: 307
power: 1
energy_total: 1149
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 2
voltage: 307
current: 11
power: 35
energy_total: 583594
energy_daily: 64
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 3
voltage: 315
power: 1
energy_total: 925
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 4
voltage: 315
current: 11
power: 36
energy_total: 633209
energy_daily: 65
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 5
voltage: 310
energy_total: 412
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 6
voltage: 310
current: 12
power: 37
energy_total: 663337
energy_daily: 67
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 1
voltage: 311
current: 13
power: 40
energy_total: 662697
energy_daily: 71
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 2
voltage: 311
current: 12
power: 39
energy_total: 661166
energy_daily: 70
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 3
voltage: 308
current: 12
power: 40
energy_total: 651761
energy_daily: 70
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 4
voltage: 308
current: 12
power: 38
energy_total: 645157
energy_daily: 68
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 5
voltage: 314
current: 12
power: 37
energy_total: 608681
energy_daily: 68
error_code: 50986752
}
pv_data {
serial_number: 2145**********
port_number: 6
voltage: 314
current: 12
power: 37
energy_total: 621869
energy_daily: 68
error_code: 50986752
}
dtu_power: 328
dtu_daily_energy: 611


Repair-NavApp - Apps no more vissible or available in Client

AL Extension Version: 157.014

Server Version: 18.0.29486.0 [OnPrem] ... and earlier

no code required!
if we use the repair-navapp command, the apps will not be available in the client (via search or something else)
we then have to uninstall and unpublish these apps and republish/reinstall these (without repair-navapp)
this seems to happen, if the published app was created on a lower serverversion than the used one
example:
Publish-NAVApp -ServerInstance BC -Path <<<FullAppPath>>>
Repair-NAVApp -ServerInstance BC -Name <<<YourAppName>>
sync-NAVApp -ServerInstance BC -Name <<<YourAppName>>
install-NAVApp -ServerInstance BC -Name <<<YourAppName>>

and the best of this story is: sometimes we get a sql error (on installing) which then causes dataloss because then we have to sync in forceMode <- this case is really hard to reproduce and i'm trying to find a solid way for you guys.

If you can reproduce the problem on the latest Business Central Developer preview version we will be happy to look further into it. Otherwise please contact the regular support channel.


Add support to specify the profile to use

My pom.xml has multiple profiles defined and I want a way to specify which one to use for my buildpack execution (and lifecycle).

Do you have reason to believe that  BP_MAVEN_BUILD_ARGUMENTS=-Pnative-image -Dmaven.test.skip=true package does not work?

No. Just lack of documentation/examples :-(
Testing to see if this works.


generic cell cluster notebook for nimbus

Currently in the generic cell clustering notebook, the heatmap widget shows valued that is scaled/Z scored. This is not informative when working with nimbus output that generates a probability score from 0-1 for each marker.
Describe the solution you'd like
Remove any data transformation from the heatmap widget used for nimbus.
@alex-l-kong

I thought there was already the option to do this? Maybe it's not turned on by default in the notebook? Pretty sure @JLrumberger added this in.

I am not sure. The only way I got this to work was by altering the ZscoreNormalize and update_gui functions to do nothing
`
class ZScoreNormalize(Normalize):
def init(self, vmin=None, vcenter=None, vmax=None):
super().init(vmin, vmax)  # Retain superclass initialization
def inverse(self, value):
    return value  # Return the value as-is

def calibrate(self, values):
    pass  # Do nothing

def __call__(self, value: np.ndarray, clip=None):
    return value  # Return the value as-is

`
and
`    def update_gui(self):
"""Update and redraw any updated GUI elements"""
self.im_cs.set_data(self.selection_mask)
self.im_cs.set_extent((0, self.mcd.cluster_count, 0, 1))
    if not self._heatmaps_stale:
        print("skipping other repaints")
        self.fig.canvas.draw()
        return

    # def _preplot(df):
    #     return df.apply(zscore).clip(upper=self.zscore_clamp_slider.value).T

    def _preplot(df):
        return df.T  # Avoiding z-score normalization and clipping`


recoil-persist not working as expected with useSetRecoilState in the useEffect hook

I want to update an atom with the query string in the URL. With useSetRecoilState I can set the atom but recoil-persist doesn't save in localStorage. I can see that the onSet callback in the effects of the atom is not triggered.
See this example:
import { atom } from 'recoil';
import { recoilPersist } from 'recoil-persist';

const { persistAtom } = recoilPersist();

export const atomState = atom({
  key: 'example',
  default: null,
  effects: [persistAtom],
});

import React from 'react';
import { Navigate } from 'react-router-dom';
import { useRecoilValue } from 'recoil';
import { atomState } from 'state';
import { useSaveQueryString } from 'hooks/useSaveQueryString';

function MobileRoute() {
  useSaveQueryString ();

  const state = useRecoilValue(atomState );

  if (state ) {
    return <Navigate to="/something" />;
  }

  return
Loading...
;
}

export default MobileRoute;

function useSaveQueryString () {
  const [search] = useSearchParams();
  const setAtom = useSetRecoilState(atomState);

useEffect(() => {
const searchAsObject = Object.fromEntries(new URLSearchParams(search));

  if (searchAsObject.id) {
    setAtom (searchAsObject.id);
  }
}, [search, setAuth])
}

How can I solve it? Some example with recoil-sync??

I had the same problem, and it turned out that it was the combination of React 18 and strict mode.
Inside an internal Recoil component called RecoilRoot_INTERNAL there is an useEffect() that initializes and cleans up the internal state store  When Recoil runs in strict mode that effect is called multiple times and during all this reinitialization all effect subscriptions generated when calling onSet are removed.
So the workaround was to bail out of strict mode.


Constant propagation for ONNXGatherOp

This patch implements constant propagation for ONNXGatherOp

Jenkins Linux s390x Build #6667 [push] Constant propagation for... started at 02:52

Jenkins Linux amd64 Build #6651 [push] Constant propagation for... started at 01:52

Jenkins Linux ppc64le Build #5763 [push] Constant propagation for... started at 02:54

Jenkins Linux amd64 Build #6651 [push] Constant propagation for... passed after 1 hr 19 min

Jenkins Linux s390x Build #6667 [push] Constant propagation for... passed after 1 hr 30 min

Jenkins Linux ppc64le Build #5763 [push] Constant propagation for... passed after 1 hr 43 min


question about xception-deeplab

Hi,
Thanks for sharing this !!!
Would you tell me from where can we find the xception model architecture details, and the training hyper-parameters ?

You can view the network structure for the details you mentioned, and refer to this link for training hyperparameters: https://github.com/PaddlePaddle/PaddleClas/blob/master/configs/Xception/Xception65_deeplab.yaml.

Thanks for telling this !!
I found this line:
https://github.com/PaddlePaddle/PaddleClas/blob/d454a22e2c93182ed3b875ef6ced90b948eff91a/ppcls/modeling/architectures/xception_deeplab.py#L240
It means that no matter whether the output stride is 32 or 16, the dilation rate is always 2. However, if the backbone is resnet, the dilation rate only changes when the output stride is changed. For example, when the output stride is 16, the last stage of resnet will enlarge its dilateion rate from 1 to 2. Is xception different from resnet in this ?

Thanks for telling this !!
I found this line:
https://github.com/PaddlePaddle/PaddleClas/blob/d454a22e2c93182ed3b875ef6ced90b948eff91a/ppcls/modeling/architectures/xception_deeplab.py#L240
It means that no matter whether the output stride is 32 or 16, the dilation rate is always 2. However, if the backbone is resnet, the dilation rate only changes when the output stride is changed. For example, when the output stride is 16, the last stage of resnet will enlarge its dilateion rate from 1 to 2. Is xception different from resnet in this ?

This backbone is just a pretrained model trained for classification. Other tasks need to determine the dilation rate according to the specific output stride. You can temporarily change some codes according to your specific needs.

Thanks for replying !! I am trying.


MA - Invalid LOINC

Describe the bug
MA received a new LOINC code that they were not expecting from Sumas Drug who was the ordering provider and also the lab facility.  They wanted to see if we could clarify with them or please send us their contact information so that we can see if they meant to send this test, which “detects but does not differentiate between SARS-CoV and SARS-CoV-2” or if they accidentally selected this option.
The MSH.10 IDs in question are:
d523819b-67dd-46e0-b74e-57c4d28dcdb0
d23b1c7c-cfb3-49ce-bba2-40185b2657a2
The test in quest was:
95209-3 - SARS-CoV+SARS-CoV-2 (COVID-19) Ag [Presence] in Respiratory specimen by Rapid immunoassay

@jorg3lopez got data files from Azure inbound for me.  I will start working on this in parallel with ticket 4990.

@Adrian-Brewster @sarahnw , @jorg3lopez and I verified the LOINC code is in the LIVID table in the database.  Please provide the contact information to MA to get with Sumas Drug provider.  And, let MA provide the update of the outcome.  Therefore, we can take action on what they what us to do with the message.

Sender is part of Prescryptive.  CC'd John at Prescryptive to get this sorted.

Yup! We are all set here @anshulkumar-usds I just forgot to close the ticket.


Remove unused folder(s)/file(s); HypervDummyResourceBase.java

Dead code, never called and consist only of auto-generated methods.
As a developer I want

a clean project to work on
a project with maintained code
a project without dead code or files

Builds ok:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 6:44.207s
[INFO] Finished at: Sun Dec 13 19:11:08 CET 2015
[INFO] Final Memory: 101M/800M
[INFO] ------------------------------------------------------------------------

Ping @bhaisaab @wido @remibergsma @DaanHoogland @miguelaferreira @wilderrodrigues

Integration tests ran fine!
Environment

2 KVM host on CentOS 7.1
1 Management Server on CentOS 7.1
Agent + Common RPMs built from source

Integration test suite 1
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \
    component/test_password_server.py \
    smoke/test_vpc_redundant.py \
    smoke/test_routers_iptables_default_policy.py \
    smoke/test_routers_network_ops.py \
    smoke/test_vpc_router_nics.py \
    smoke/test_router_dhcphosts.py \
    smoke/test_loadbalance.py \
    smoke/test_internal_lb.py \
    smoke/test_ssvm.py \
    smoke/test_vpc_vpn.py \
    smoke/test_privategw_acl.py \
    smoke/test_network.py

Result test suite 1
Check the password file in the Router VM ... === TestName: test_isolate_network_password_server | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_isolate_network_FW_PF_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_isolate_network_FW_PF_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_03_RVR_Network_check_router_state | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === TestName: test_router_dhcphosts | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok
Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
test_01_vpc_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_01_vpc_privategw_acl | Status : SUCCESS ===
ok
test_02_vpc_privategw_static_routes (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_02_vpc_privategw_static_routes | Status : SUCCESS ===
ok
test_03_rvpc_privategw_static_routes (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_03_rvpc_privategw_static_routes | Status : SUCCESS ===
ok
Test for port forwarding on source NAT ... === TestName: test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: test_02_port_fwd_on_non_src_nat | Status : SUCCESS ===
ok
Test for reboot router ... === TestName: test_reboot_router | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_1_static_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_2_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 38 tests in 21695.211s

OK

Integration test suite 2
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=false \
    smoke/test_routers.py \
    smoke/test_network_acl.py \
    smoke/test_reset_vm_on_reboot.py \
    smoke/test_vm_life_cycle.py \
    smoke/test_service_offerings.py \
    smoke/test_network.py \
    component/test_vpc_offerings.py \
    component/test_vpc_routers.py

Result test suite 2
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok
Test for delete account ... === TestName: test_delete_account | Status : SUCCESS ===
ok
Test for Associate/Disassociate public IP address for admin account ... === TestName: test_public_ip_admin_account | Status : SUCCESS ===
ok
Test for Associate/Disassociate public IP address for user account ... === TestName: test_public_ip_user_account | Status : SUCCESS ===
ok
Test for release public IP address ... === TestName: test_releaseIP | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
ok
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 41 tests in 8953.259s

OK

@borisroman you know by now I agree on those principles but your integration test results are quite useless as we are talking about hyperv code. Let's do a short period of inquisition on the usage amongst teh hyperv users and experts.

@DaanHoogland The integration tests show nothing broke on the KVM part, the code of course does need to be reviewed. But that doesn't mean the tests are useless.

@borisroman I can not decide based on these tests if the removal is ok, so to me they are useless

@borisroman @DaanHoogland Let's ask someone to test whether HyperV is still happy and then we'll know.

@remibergsma @DaanHoogland Back to the point, who could we ping for a HyperV review?

No reviews, so closing.


HUM-202 : fix smoke test for this w/out breaking functests

to test:
swiftly put hat
swiftly put hat/cat -i Makefile -s s100
swiftly -v get hat/cat -hrange:bytes=99-0


Add spinner and load error indicator to DrawerSequenceView

Description
Add an error message and a loading spinner to the drawer sequence view.
Deployment URL(s)
http://gb-sequence-view-errors.review.ensembl.org

Do we not need to show the spinner for at least a second?

The failed state doesn't get cleaned up when we change the sequence type:

I don't see the loading failure message locally. Can only see it in the review branch site. The spinner is visible in both places though.


consider using __cpp_exceptions and/or __EXCEPTIONS to disable/enable exception support

It is much more user-friendly if exception support is automatically detected.
resources:
https://gcc.gnu.org/onlinedocs/libstdc++/manual/using_exceptions.html
http://en.cppreference.com/w/cpp/experimental/feature_test

Thanks, I did not find these macros. Do you know which compilers are covered with them?

gcc and clang definitely, others I don't know. You can test for a specific compiler than go on from there. But it is a proposed standard extension too.

I guess you could also do #if defined(__cpp_excepetions) && (1 == __cpp_exceptions)

I'm checking for __EXCEPTIONS right now. No idea where I got this from...

If you do it doesn't work too well :) I tried to compile without exception support and it didn't work, I check and saw I'd have to define some macro manually to get your library to compile.

@user1095108 I added a fix (see 65dfc97d401e17befa4d9a2320660b381529304e). Does this work for you?

Sigh... MSVC seems not to use __cpp_exceptions (Appveyor fails: https://ci.appveyor.com/project/nlohmann/json/build/1770). I read about  _HAS_EXCEPTIONS so far, but did not find a reliable source for that.

I'd still check if __cpp_exceptions is defined. The macro is not part of c++98, just the exceptions themselves. Otherwise, great.

I'd still check if __cpp_exceptions is defined.

What do you mean?

From https://msdn.microsoft.com/en-us/library/b0084kay.aspx:

_CPPUNWIND Defined as 1 if one or more of the /GX (Enable Exception Handling), /clr (Common Language Runtime Compilation), or /EH (Exception Handling Model) compiler options are set. Otherwise, undefined.

It is a proposed extension of c++, gcc and clang already implement it, but MSVC has its own thing as you noticed.

I see. If the fix works for you, I can close the ticket.

sure, but you won't put this into the main branch? I think you want a
generic solution.
2017-03-11 18:21 GMT+01:00 Niels Lohmann<EMAIL_ADDRESS>
I see. If the fix works for you, I can close the ticket.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nlohmann/json/issues/498#issuecomment-285882157, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH6jVGjolDowt0m-MvgkoO2kWEv2h4ITks5rktgUgaJpZM4MaOyK
.

I will, but first I wanted your OK, and I want to wait for Travis to finish.

ok, as I wrote earlier, I'd check #if defined(__cpp_exceptions) && (1 ==
__cpp_exceptions) ...
You may have your preferences though, maybe an existent macro definition
could override everything.
2017-03-11 18:36 GMT+01:00 Niels Lohmann<EMAIL_ADDRESS>
I will, but first I wanted your OK, and I want to wait for Travis to
finish.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nlohmann/json/issues/498#issuecomment-285883095, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH6jVOPlw9gr1YvjtAsyXfYDM-SQ-xynks5rktuQgaJpZM4MaOyK
.

You're right, it is better to used defined(). But I would not care about the value of the macro - if someone defines it, it better has the standard semantics.

I'm ok with this.
g++ -x c++ -dM -E - < /dev/null | grep _cpp_exceptions
2017-03-11 18:44 GMT+01:00 Niels Lohmann<EMAIL_ADDRESS>
You're right, it is better to used defined(). But I would not care about
the value of the macro - if someone defines it, it better has the standard
semantics.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nlohmann/json/issues/498#issuecomment-285884642, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH6jVExKCs_UCxw8ct0NT0k7BLxVQptfks5rkt2SgaJpZM4MaOyK
.

$ g++ -x c++ -dM -E - < /dev/null | grep _cpp_exceptions

#define __cpp_exceptions 199711

Yes, you might want to use that or just check if it is defined :)
2017-03-11 18:53 GMT+01:00 Niels Lohmann<EMAIL_ADDRESS>
$ g++ -x c++ -dM -E - < /dev/null | grep _cpp_exceptions
#define __cpp_exceptions 199711
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nlohmann/json/issues/498#issuecomment-285885912, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH6jVF8yJZ-EN3yL0VIot22yO8x-IsXpks5rkt-agaJpZM4MaOyK
.

GCC 4.9 does not define __cpp_exceptions. I might need __EXCEPTIONS there.

It's your library, I personally don't care about 4.9 anymore.
2017-03-11 20:09 GMT+01:00 Niels Lohmann<EMAIL_ADDRESS>
GCC 4.9 does not define __cpp_exceptions. I might need __EXCEPTIONS there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nlohmann/json/issues/498#issuecomment-285892482, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH6jVGHAETwHtKvUrOvksE6PsHh0fk6aks5rkvFpgaJpZM4MaOyK
.


Gui

some improvements (at least I hope so) of the GUI
fixes #308

All single changes work fine for me :D I just have a question regarding the multiple selection: Is it still possible to select boxes using a rect?

I only tested briefly.  Some things I observed.

It looks like the MCS dialog does remember reactions from a previously loaded model (used for completion) that don't exist in a newly loaded model.
When moving multiple boxes one box is moved further than the rest.
While Shift is pressed I can select further boxes by clicking them, it would be nice if clicking a selected box again would unselect it.
I think this PR maybe does change some unrelated things at once which could be separated?

All single changes work fine for me :D I just have a question regarding the multiple selection: Is it still possible to select boxes using a rect?

Yes, this is intiated hy holding Shift and clicking on the map. Further (de-)selection of boxes is currently realized by holding the Ctrl and clicking on the boxes.

I only tested briefly. Some things I observed.
* It looks like the MCS dialog does remember reactions from a previously loaded model (used for completion) that don't exist in a newly loaded model.

Hmm, I don't get this effect.

* When moving multiple boxes one box is moved further than the rest.

I don't notice that, but what I see is that all boxes often jump at the begin of the drag event.

* While Shift is pressed I can select further boxes by clicking them, it would be nice if clicking a selected box again would unselect it.

You can unselect by holding Ctrl, but now I realize that this is somewhat inconsistent, should perhaps better check for Shift there too.

  I think this PR maybe does change some unrelated things at once which could be separated?

Yes, but I like to work on multiple things at once and find branch switching annoying (liked SVN better there) ;-) Oh, and there are more commits coming soon xD

Some other thoughts. 🤔
Maybe add a tool tip (or something similar) to the MCS dialog that explain that you can click on the map to add reactions.
I actually don't like these floating windows ... Maybe one could integrate the MCS Dialog somehow into the main window.
Similar to the navigation panel that is only visible when needed.

Some other thoughts. 🤔
Maybe add a tool tip (or something similar) to the MCS dialog that explain that you can click on the map to add reactions.
I actually don't like these floating windows ... Maybe one could integrate the MCS Dialog somehow into the main window.
Similar to the navigation panel that is only visible when needed.

I think it would be even better to have the possibility for docking/undocking, especially for the maps. Don't know whether this is easy to implement though...

Some other thoughts. 🤔
Maybe add a tool tip (or something similar) to the MCS dialog that explain that you can click on the map to add reactions.
I actually don't like these floating windows ... Maybe one could integrate the MCS Dialog somehow into the main window.
Similar to the navigation panel that is only visible when needed.

I think it would be even better to have the possibility for docking/undocking, especially for the maps. Don't know whether this is easy to implement though...

Making the MCS dialog dockable turns out to be quite simple, but I think it does not look very well, so for now I leave it as a floating window.

This was the last commit for this branch.

Right now I selecting more boxes by clinging while

All single changes work fine for me :D I just have a question regarding the multiple selection: Is it still possible to select boxes using a rect?

Yes, this is intiated hy holding Shift and clicking on the map. Further (de-)selection of boxes is currently realized by holding the Ctrl key and clicking on the boxes.

I think it would be more consistent if this was the same Meta key either Ctrl or Shift.
Also it would be nice that pressing the Meta key turned the hand cursor into a cross.

All single changes work fine for me :D I just have a question regarding the multiple selection: Is it still possible to select boxes using a rect?

Yes, this is intiated hy holding Shift and clicking on the map. Further (de-)selection of boxes is currently realized by holding the Ctrl key and clicking on the boxes.

I think it would be more consistent if this was the same Meta key either Ctrl or Shift.

Yes, that is true, from my experience Ctrl is commonly used for multiple selection, so I will make it all Ctrl.

Also it would be nice that pressing the Meta key turned the hand cursor into a cross.

Yes, will see if I can find out how to do that.

Looks functional, I only noticed one thing. While for selecting it is sufficient to click anywhere in the reaction box, deselecting does only work if you click exactly on the border of the reaction box .. which can be kind of tricky. Maybe this can be addressed later.

I think this should be doable, but since the reviewer comments just came in I will merge at this point and return to it later.


Import "RCTEventDispatcher.h"

Hey, in first place let me thank you for this package.
I've been able to run it on android however installing fabric successfully and after adding TwitterSignIn.m and TwitterSignIn.h to Libraries
#import "RCTEventDispatcher.h"
displays error, file not found.
Any idea whats missing? Isn't RCTEventDispatcher from react?

import with 0.40 version <React/RCTEventDispatcher.h>

That was the solution I found at the time. Sorry, forgot to close it.

@RUIFERNANDE5 If i execute TwitterSignin.logIn nothing happening. Throw xcode debug log this log.
2017-04-27 00:55:32.076 Frisbee[11898:1024478] [Fabric] [Fabric +with] called multiple times. Only the first call is honored, please pass all kits you wish to initialize
2017-04-27 00:55:32.077 Frisbee[11898:1024478] [TwitterKit] No matching scheme found.
2017-04-27 00:55:32.720 Frisbee[11898:1024478] [TwitterKit] did encounter error with message "Error obtaining user auth token.": Error Domain=TWTRLogInErrorDomain Code=-1 "{"errors":[{"code":32,"message":"Could not authenticate you."}]}" UserInfo={NSLocalizedDescription={"errors":[{"code":32,"message":"Could not authenticate you."}]}}


Re-surfacing path variable issues with relativeFile

Related https://github.com/microsoft/vscode/issues/70050#issuecomment-472052839 https://github.com/microsoft/vscode/issues/40256
Hi all - I read through all the conversation on the above topics, and I just wanted to resurface this information.
For some context, using vscode debugger to run jest tests is very difficult due to the above issue (as jest can only understand forward slashes, while VSCode uses OS-specific slashes). The workarounds above suggest using ${fileBaseName}, but this causes issues in monorepo-style projects. For example, if I want to run the tests in my open file a/index.ts, using fileBaseName will be a regex match for every other index.ts  in the repo (thousands in our case). There is no other way to dynamically grab the whole relative file name.
My suggestion would be to allow choosing slash used via some known syntax. Example would be ${relativeFile|slash=unix} or something similar. This would continue to be totally backward compatible, but allow for a little more configuration to solve the cases mentioned above.

For anyone else looking for a workaround for this issue, I found the best way to be by creating a small shim for jest specifically to be called by VSCode
my-jest.js
// https://github.com/microsoft/vscode/issues/70050

const regexPositionFromVSCode = process.argv.length - 1;
const fileNameRelative = process.argv[regexPositionFromVSCode];
process.argv[regexPositionFromVSCode] = fileNameRelative.replace(/\\/g, "/");

require("jest/bin/jest");

`launch.json
    {
      "type": "node",
      "request": "launch",
      "name": "Debug Unit Tests In Current File",
      "sourceMaps": true,
      "program": "${workspaceFolder}/my-jest.js",
      ...
    },

@alexr00 I think you own general variable resolution?


"Access" facet

@caseyedavis12 : What values show under the facet? just "Online" + "All Records"?
besides that:

Will these be clear enough, or do we need explanatory blurbs like on AAPB? Hopefully not...
We will use the same kind of logic as AAPB, ie selecting one unselects the other?
On other facets, if you click on a currently selected facet value, it unselects it. Does that same model work here, or should only the unselected one be clickable?
This is no explicit statement of access on the record itself, right?
"Online" or whatever we call it needs to be applied by default on searches.

@mccalluc: What values show under the facet? just "Online" + "All Records"? YES

We will use the same kind of logic as AAPB, ie selecting one unselects the other? YES
On other facets, if you click on a currently selected facet value, it unselects it. Does that same model work here, or should only the unselected one be clickable? YES
This is no explicit statement of access on the record itself, right? If it is not available online, please use this message under the video thumbnail like on AAPB: "Please note: This content has not been digitized. For inquiries regarding access or if you are interested in sponsoring the digitization of this program, please contact us at<EMAIL_ADDRESS>"Online" or whatever we call it needs to be applied by default on searches. YES

Instead of just "Online" could it say "Available Online"

(We also need the same kind of logic from AAPB that if the facet value is unspecified on a request, we add the default and redirect.)


Strict undefined check in is_UEnode not handling null.

Came across this. Sometimes node_or_nodeType.type is null (not undefined) which causes an issue in this code in use_everywhere_utilities.js.
// in use_everywhere_utilities.js
function is_UEnode(node_or_nodeType) {
    var title = node_or_nodeType.type;
    if (title===undefined) title = node_or_nodeType.comfyClass;
    if (title===undefined) return false;
    return (title.startsWith("Anything Everywhere") || title==="Seed Everywhere" || title==="Prompts Everywhere")
}

This method seems to run on construction of a ComfyNode where node_or_nodeType.type is always undefined always. However, a bunch of rgthree-nodes this is actually null and this breaks becuase of the title.startsWith. This isn't currently invoked but I was playing with the idea of better alignment with the default ComfyNode, which would allow this to run on these rgthree nodes.
I'm still trying to figure out why type is always undefined in ComfyNode and null in these rgthree nodes, but I would still consider this a bug as null could make it through to that startsWith call.
By looking at this, my recommendation would be to change the two title === undefined to tile == null (with two equals) which will match both undefined and null.

FWIW, I took a look at why there's the difference between null and undefined with ComfyUI and some rgthree-nodes.
Turns out, LiteGraph sets type to null explicitly when it's instantiating a new LGraphNode, and rgthree-nodes extend LGraph correctly.
Turns out, ComfyNodes don't actually extend LiteGraph's nodes, so never call into the LGraphNode constructor and, thus, type is left undefined. So, if anything, the high level feels like this is buggy behavior in ComfyUI's ComfyNode for not actually being an LGraphNode.
I don't feel like hacking or working aroundf LiteGraph's LGraphNode constructor, since that feels like the right thing. What this means is if I do submit the change to allow other custom nodes to run nodeCreated for rgthree-nodes, this is_UEnode will throw an error for title being null. Nothing negative should happen, just errors in the console.

huh. Nice find. Cleaned it up like this to avoid the whole null/undefined thing completely:
function is_UEnode(node_or_nodeType) {
    const title = node_or_nodeType.type ?? node_or_nodeType.comfyClass;
    return ((title) && (title.startsWith("Anything Everywhere") || title==="Seed Everywhere" || title==="Prompts Everywhere"))
}

I never really got my head around javascript truthiness :)


[zh] Update psp-to-pod-security-standards.md

Signed-off-by: Guangwen Feng<EMAIL_ADDRESS>en PR https://github.com/kubernetes/website/pull/34172/files
Also fix the incorrect link in zh page.
/assign @tengqm
PTAL, thanks.

/lgtm
BTW, is there any reason to /assign @tengqm when opening PR?
I think this PR is not that emergency to ask someone for immediate review, please bear in mind that we are all volunteering to contribute to this project.

/approve

BTW, is there any reason to /assign @tengqm when opening PR? I think this PR is not that emergency to ask someone for immediate review, please bear in mind that we are all volunteering to contribute to this project.

Sorry, since I saw other guys always add /assign @tengqm in the initial comment (e.g. Arhell), so I just did the same...
Thanks for reminding me.


CUJ Testing for Beta v2 - Creator can Drag and Drop an Image/Video into an Available Drop Target

Feature Description
In preparation for the Beta 2 release we would like to have validation and confirmation of the CUJ functionality detailed in
https://github.com/google/web-stories-wp/issues/1588
Basic shapes are drop targets
Media elements are drop targets

Verified in PR#4012


Allow Slack token to be stored per user

Create controller/service to handle user sending their Slack token and storing this somewhere for use later.  Right now can just be flat file since this might be the only data we are storing.  Then, use the token of the current user instead of having tokens hardcoded somewhere.

Currently stores to file-per-user for a simple implementation


[r2dbc] Apply R2dbcCoverter to ConstantImpl value in where clause

I guess many people define some properties of classes mapped with tables as enum.
When selecting columns defined as enum, there is no problem because EntityRowMapper using R2dbcConverter executes simple conversions and user defined conversions (registered via r2dbcCustomConversions() of AbstractR2dbcConfiguration).
However, when using the columns with the where clause like where(user.status.eq(UserStatus.ONLINE), problems occur. The generated query not only apply Converter, but also doesn't enclose the enum value with quotation marks (ex> ... where ... user.status = ONLINE), thus it usually cause db exceptions like Unknown column 'ONLINE' in 'where clause'.
If this library supports those conversions, it will be very useful.
I don't think my implementation is not the best one, so please give me advice if you have better idea.
And... I want to do similar things for having clause, but sadly QueryMetadata of QueryDSL doesn't have clearHaving() method. So I handled only where clause, for now.

Hi, @lpandzic. Could you check this PR and give some comments? I fixed tests and add supports for subqueries. Although predicates in having condition and join condition are not supported, I think this PR make building queries with enum fields easy.

Hi, can you add some tests that clearly show the intent?

I'm closing this one as stale. Feel free to reopen when you add tests.


chore(db) remove pre-workspace blue-green deployment workaround

Summary
When Kong 2.1.0 was release we needed to add special workaround to enable blue-green deployment with the introduction of workspaces.
We needed to add special select_by_cache_key to Plugins DAO. The check that we added is hurting the Kong performance, as seen reported in #8788. We got further PR from the author with #8789. While the fix is good, I do think we can remove this by now altogether. The only downside of this, is that for people who migrate directly from Kong < 2.1.x to 3.0.0, we don't support the blue-green deployment.
Issues Resolved
Fixes #8788
Closes #8789

Alternatively, we need to merge https://github.com/Kong/kong/pull/8789.


https only to prevent browser warnings

This was recently done by Lemon for FreeReg, we should ask him to do also it for FreeCen2.
Make sure no FC1 links/bookmarks get broken in the process (if https-only also gets applied to FC1, we need to verify that http URLs get automatically forwarded or rewritten to their https equivalents).

I believe Lemon has done this.  Looks like it is working fine.


Fix Package manager support

Thanks for the PR! It looks like the SwiftPM linux build isn't picking up the LinuxMain.swift. Want to try to fix that up?

Done!

Awesome, thanks!


[Event Request] SetOperationType on Service Header (Italian localization)

It would be helpful if you could add events in SetOperationType procedure on Service Header table in this same way of Sales Header and Purchase Header.
In particular, event OnBeforeSetOperationType with IsHandled parameter is missing.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Use new connection for every test case

It used to be true that we reset connection for every test case for test stability. This behavior change when we introduced the v2api because [GRPCCall resetHostSettings] now only removes the GRPCHost settings in v1 api but no longer disconnects the channel. This change recovers the original behavior.
This PR tentatively helps to reduce the ObjC test flakiness and hopefully makes the ObjC tests useful to developers again. There could be concern about this change covering up some test failures that only occur with long-lasting channels (particularly the CFStream hang on MacOS). From my local experiments it seems that we are still able to capture this type of issue in some of the large tests. Besides that, we still have bug ticket tracking this issue.

Test failures are infrastructure issue


How to manage two yml files

Hello vanister,
Thank you so much for sharing this code. I got stuck in one minor issue. I am new to aws functions. Can you please help me to get rid of this problem ?
When i import two functions yml files i got only one url
`service: contacts-api
provider:
name: aws
runtime: nodejs10.x
stage: dev
region: us-west-2
functions:

'${file(src/handlers/contacts/contacts.serverless.yml)}'
'${file(src/handlers/users/users.serverless.yml)}'

plugins:

serverless-offline

custom:
serverless-offline:
host: <IP_ADDRESS> # for docker container hosting
I got url's Serverless: Starting Offline: dev/us-west-2.
Serverless: Routes for list:
Serverless: GET /user/getall
Serverless: POST /{apiVersion}/functions/contacts-api-dev-list/invocations
Serverless: Routes for get:
Serverless: GET /user/{id}
Serverless: POST /{apiVersion}/functions/contacts-api-dev-get/invocations
Serverless: Routes for add:
Serverless: POST /user
Serverless: POST /{apiVersion}/functions/contacts-api-dev-add/invocations
Serverless: Routes for update:
Serverless: PUT /user/{id}
Serverless: POST /{apiVersion}/functions/contacts-api-dev-update/invocations
Serverless: Routes for delete:
Serverless: DELETE /user/{id}
Serverless: POST /{apiVersion}/functions/contacts-api-dev-delete/invocations
Serverless: Offline [HTTP] listening on http://<IP_ADDRESS>:3000
Serverless: Enter "rp" to replay the last request
`
Thanks in advance

I think what is happening is that you are overriding the contacts.serverless.yml file's entries.  Your users.serverless.yml file likely contain the same function names.
Try prefixing the names with list-users, get-user, etc. for users.serverless.yml.  Likewise, prefix the contacts.serverless.yml file with something similar to user, list-contacts, get-contact, etc.


How can I make BASALT env in VPT

In the repository of VPT, I found that the way to create an environment is as follows:
env = HumanSurvival(**ENV_KWARGS).make()

How do I create an environment like 'MineRLBasaltBuildVillageHouse-v0' in VPT?

[Update 1]
I have taken the following two approaches:
1. I added a name parameter control in HumanSurvival
ENV_KWARGS["name"] = "MineRLBasaltBuildVillageHouse-v0"
env = HumanSurvival(**ENV_KWARGS).make()

or
env = HumanSurvival(name="MineRLBasaltBuildVillageHouse-v0", **ENV_KWARGS).make()

but it didn't have any effect; the environment remained the same as at the beginning.
2. I replaced HumanSurvival with the creation method provided by gym
env = gym.make("MineRLBasaltMakeWaterfall-v0")

but when executing agent actions, the following error occurred:
File "/opt/conda/envs/minerl/lib/python3.10/site-packages/gym/wrappers/time_limit.py", line 18, in step
    observation, reward, done, info = self.env.step(action)
File "/opt/conda/envs/minerl/lib/python3.10/site-packages/minerl/herobraine/env_specs/basalt_specs.py", line 108, in step
    done = done or bool(action["ESC"])
KeyError: 'ESC'

[Update 2]
I just used the second method mentioned above and manually added the value ["ESC"], and it can run normally now.
env = gym.make("MineRLBasaltBuildVillageHouse-v0")

However, when I use gym to make env, the obs from env.step do not have many keys like 'location_stats' and 'inventory'. How can I get these information from env if I use gym but not HumanSurvival ?

Heya, good thing you figured it out :).
To answer your final question: you'd have to update the environment code to provide additional information like the ones you mentioned from BASALT environments. Those were removed on purpose for the BASALT competition as the idea was to make an agent that only uses the provided info.

Heya, good thing you figured it out :).
To answer your final question: you'd have to update the environment code to provide additional information like the ones you mentioned from BASALT environments. Those were removed on purpose for the BASALT competition as the idea was to make an agent that only uses the provided info.

Heya, good thing you figured it out :).
To answer your final question: you'd have to update the environment code to provide additional information like the ones you mentioned from BASALT environments. Those were removed on purpose for the BASALT competition as the idea was to make an agent that only uses the provided info.

Thank you very much \(●°u°●)/ !
I have modified the observation space which is equal to HumanSurvival, and now I can get these information mentioned above.


Run after click once install throws type initializer exception for SQLiteApiWin32Internal

I am using SQLite PCL in a portable class lib as part of a WPF desktop application.
When I run the program in debug from visual studio everything works perfectly.
If I just copy the contents of the bin directory to the target platform and run by just double clicking on the .exe, everything works perfectly.
BUT.  If I publish as a click once application, install with click once on the target platform, and run, SQLite PCL throws the following exception.  If I build my own setup and deployment project, deploy, and run, I also get this exception.
System.TypeInitializationException: The type initializer for 'SQLite.Net.Platform.Win32.SQLiteApiWin32Internal' threw an exception. ---> System.Exception: Failed to load native sqlite library
at SQLite.Net.Platform.Win32.SQLiteApiWin32Internal..cctor()
--- End of inner exception stack trace ---
at SQLite.Net.Platform.Win32.SQLiteApiWin32Internal.sqlite3_open_v2(Byte[] filename, IntPtr& db, Int32 flags, IntPtr zvfs)
at SQLite.Net.Platform.Win32.SQLiteApiWin32.Open(Byte[] filename, IDbHandle& db, Int32 flags, IntPtr zvfs)
at SQLite.Net.SQLiteConnection..ctor(ISQLitePlatform sqlitePlatform, String databasePath, SQLiteOpenFlags openFlags, Boolean storeDateTimeAsTicks, IBlobSerializer serializer, IDictionary2 extraTypeMappings)    at SQLite.Net.SQLiteConnection..ctor(ISQLitePlatform sqlitePlatform, String databasePath, Boolean storeDateTimeAsTicks, IBlobSerializer serializer, IDictionary2 extraTypeMappings)
at Wni.Core.ViewModels.EmployeeViewModel.OpenDBConnection()
at Wni.Core.ViewModels.EmployeeViewModel.OnLoadedCommand()

Seems like the sqlite binaries are not loaded properly.
I've never used clickonce so I don't know how it works..
The Win32 platform includes sqlite binaries as content files (so check that they are deployed correctly).
These binaries are preloaded for the correct cpu configuration (x86/x64) using some path logic + LoadLibrary.
I suggest you look at SQLiteApiWin32Internal() and try to see how that code could go wrong, given how the clickonce installer deploys the files.

Thanks for the hint.  It led me to the problem and solution.  I didn't need to look at the code.
Others may find a better solution but here is what I did.
I ran winmerge against the visual studio bin folder versus the click-once deployment folder.  That yielded nothing until I included subfolders.  Then I discovered the obvious.  The SQLite interop files for x86 and x64 were missing from the deployment.  Specifically: x64\SQLite.Interop.dll  and x86\SQLite.Interop.dll.
I went back to visual studio.  I created new solution folders for x86 and x64.  I copied the interop files from the bin folder into the new solution folders.  I INCLUDED THEM IN THE PROJECT AS CONTENT AND SET THE "Copy to Output Directory" PROPERTY TO "Copy If Newer" for each.  That caused the click-once publisher to include them in the deployment AND to put them in the correct relative path.
I deployed and executed.  Everything worked fine.
Thanks!!

... Oops.
This sentence:
"I created new solution folders for x86 and x64. I copied the interop files from the bin folder into the new solution folders."
is better stated:
"I created new PROJECT folders for x86 and x64.  I created the project folders in the project that click-once deploys.  That is, the project that produces the executable. I copied the interop files from the bin folder into the new solution folders."

@brianhanline: thank you! Encountered the exact same problem, solved it with your suggestion. Still bug me which dependency the Interop dll belong to though.


One gallery shown when multiple instances initiated

Hi, I have pages that need multiple galleries to be shown.
I initiate it this way:
function galleries() {
    $(".content-gallery").each(function (a, gallery) {
        var gallOpts = DEFAULT_GALL_OPTS;
        var $gall = $(gallery);
  // setting options
        $gall.unitegallery(gallOpts);
    });

html markup:

Alas, it shows only the first gallery. Any ideas?

HI. each gallery should have "ID". The gallery init is ID specific.
On Thu, Sep 10, 2015 at 9:26 PM, Alexander Ayoupov<EMAIL_ADDRESS>
wrote:

Hi, I have pages that need multiple galleries to be shown.
I initiate it this way:
function galleries() {
$(".content-gallery").each(function (a, gallery) {
var gallOpts = DEFAULT_GALL_OPTS;
var $gall = $(gallery);
// setting options
$gall.unitegallery(gallOpts);
});
html markup:

Alas, it shows only the first gallery. Any ideas?
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/29.

Am I missing something, as markup I provided has two divs with ids: gallery_2 and gallery_1.

Can you please give me link to this file online?
On Friday, September 11, 2015, Alexander Ayoupov<EMAIL_ADDRESS>wrote:

Am I missing something, as markup I provided has two divs with ids:
gallery_2 and gallery_1.
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/29#issuecomment-139485422.

Yes, sure:
http://panaggelica.org/asdgallery/asdgallery.html
My project uses jquery 1.9, but changing to 1.11 seems to do no change.

ok. cool. I'll check it tomorrow morning,
On Fri, Sep 11, 2015 at 5:21 PM, Alexander Ayoupov<EMAIL_ADDRESS>
wrote:

Yes, sure:
http://panaggelica.org/asdgallery/asdgallery.html
My project uses jquery 1.9, but changing to 1.11 seems to do no change.
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/29#issuecomment-139559379.

Please check the height on second case. You must set gallery height when
width is 100%
On Friday, September 11, 2015, maxim vendrov<EMAIL_ADDRESS>wrote:

ok. cool. I'll check it tomorrow morning,
On Fri, Sep 11, 2015 at 5:21 PM, Alexander Ayoupov <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Yes, sure:
http://panaggelica.org/asdgallery/asdgallery.html
My project uses jquery 1.9, but changing to 1.11 seems to do no change.
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/29#issuecomment-139559379
.

Thanks. It's a bit weird, but that helped

Still only seeing one gallery. Is there no way to have two appear on one page? Like, a slider and a tile?

HI, please give different ID to each gallery.
On Sun, Aug 28, 2016 at 7:41 PM, Michael Muller<EMAIL_ADDRESS>wrote:

Still only seeing one gallery. Is there no way to have two appear on one
page? Like, a slider and a tile?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vvvmax/unitegallery/issues/29#issuecomment-242984716,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC_K_h8pyynQeH19SkbQOdtkDN1I5_t7ks5qkbpQgaJpZM4F7OIV
.

While this ticket is closed I'd like to share my solution to this issue;
if ( typeof UniteGalleryMain == 'function' ) {
    $('div[id^="tiled-gallery"]').each(function(){
      $(this).unitegallery({
        tiles_type: "justified",
        lightbox_type: "compact",
        theme_enable_preloader: "false"
      });
    });
  }

So this goes loops through all occurrences of tiled-gallery's. I have set the id's to be like tiled-gallery-1, tiled-gallery-2 etc..