How to merge VOTT jsons?

Hi -
If there is more than one JPG folder/JSON in development, is there a way to combine/merge them such that the tagging work doesn't have to be redone on either?  Assuming the image file names are unique, collecting them all under the same folder wouldn't be a problem.  How about merging the JSON files?
Thanks,
Rob

This is out of scope for the tool. Probably the best way to do this is to read in the config files as two JSON objects using require and then merge their frames property using a map function.
The code would look something similar to the following prototype (not tested):
var config = require('config1.json');
var config2 = require('config2.json');

Object.keys(config.frames).map( (key, index) => {
   config.frames[key].concat( config2.frames[key]);
});

Best of luck,
-Ari


chalice-local doesn't work

Since it's the first ticket, I think "doesn't work" sums it up? :)
$ chalice-local deploy
Creating deployment package.
Creating IAM role: Maggie-dev
Traceback (most recent call last):
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/deploy/deployer.py", line 376, in deploy
    return self._deploy(config, chalice_stage_name)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/deploy/deployer.py", line 392, in _deploy
    self._executor.execute(plan)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/deploy/executor.py", line 42, in execute
    getattr(self, '_do_%s' % instruction.__class__.__name__.lower(),
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/deploy/executor.py", line 55, in _do_apicall
    result = method(**final_kwargs)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/awsclient.py", line 1022, in create_role
    raise e
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/awsclient.py", line 1018, in create_role
    self.put_role_policy(role_name=name, policy_name=name,
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/awsclient.py", line 1004, in put_role_policy
    self._client('iam').put_role_policy(
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/botocore-1.20.7-py3.8.egg/botocore/client.py", line 357, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/botocore-1.20.7-py3.8.egg/botocore/client.py", line 676, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.MalformedPolicyDocumentException: An error occurred (MalformedPolicyDocument) when calling the PutRolePolicy operation: Partition "*" is not valid for resource "arn:*:logs:*:*:*".

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/cli/__init__.py", line 636, in main
    return cli(obj={})
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/cli/__init__.py", line 189, in deploy
    deployed_values = d.deploy(config, chalice_stage_name=stage)
  File "/home/mingus/anaconda3/lib/python3.8/site-packages/chalice-1.22.1-py3.8.egg/chalice/deploy/deployer.py", line 378, in deploy
    raise ChaliceDeploymentError(e)
chalice.deploy.deployer.ChaliceDeploymentError: ERROR - While deploying your chalice application, received the following error:

 An error occurred (MalformedPolicyDocument) when calling the PutRolePolicy
 operation: Partition "*" is not valid for resource "arn:*:logs:*:*:*".

Thanks for reporting @brianmingus2 , and apologies for the long delay. We've been able to reproduce this issue, and it should be fixed in the meantime. Can you please pull the latest Docker image and give it another try? Please report here if the problem persists. Thanks!


feat: support readTime in Datastore query splitter.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕
If you write sample code, please follow the samples format.

also, we can fix clirr by adding the method to a clirr-ignored-differences.xml like https://github.com/googleapis/java-datastore/blob/main/google-cloud-datastore/clirr-ignored-differences.xml (since we added a default method, this is okay)

Done


[devbox testdata] improve rust testdata's init script to work over ssh

Summary
Updating the init-scripts here so that they can work, when invoked

locally from a sub-directory using the --config flag
over an ssh connection

How was it tested?
For scenario 1.
> cd testdata/rust
> devbox shell --config rust-stable

# saw that the init script ran fine.

For scenario 2.
the set up is more complicated.

set up containers as per https://github.com/jetpack-io/axiom/pull/2877 (sorry, this is closed source, but happy to explain)
did ssh to the vm container and had the devbox shell start

Current dependencies on/for this PR:

main

PR #326   👈

This comment was auto-generated by Graphite.


[SPARK-27849][SQL] Redact treeString of FileTable and DataSourceV2ScanExecBase

What changes were proposed in this pull request?
To follow https://github.com/apache/spark/pull/17397, the output of FileTable and DataSourceV2ScanExecBase can contain sensitive information (like Amazon keys). Such information should not end up in logs, or be exposed to non-privileged users.
This PR is to add a redaction facility for these outputs to resolve the issue. A user can enable this by setting a regex in the same spark.redaction.string.regex configuration as V1.
How was this patch tested?
Unit test

@cloud-fan

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11077/
Test PASSed.

Test build #105821 has started for PR 24719 at commit 37701d8.

Test build #105821 has started for PR 24719 at commit 37701d8.

Test build #105821 has finished for PR 24719 at commit 37701d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105821/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11091/
Test PASSed.

Test build #105835 has started for PR 24719 at commit b040d77.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11092/
Test PASSed.

Test build #105836 has started for PR 24719 at commit e017f71.

Test build #105836 has finished for PR 24719 at commit e017f71.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105836/
Test FAILed.

retest this please.

Test build #105839 has started for PR 24719 at commit e017f71.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11096/
Test PASSed.

Test build #105835 has finished for PR 24719 at commit b040d77.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105835/
Test PASSed.

Test build #105839 has finished for PR 24719 at commit e017f71.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #105839 has finished for PR 24719 at commit e017f71.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105839/
Test FAILed.

retest this please.

Test build #105846 has started for PR 24719 at commit e017f71.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11102/
Test PASSed.

Test build #105846 has finished for PR 24719 at commit e017f71.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105846/
Test PASSed.

Test build #105861 has started for PR 24719 at commit 619a124.

Test build #105861 has finished for PR 24719 at commit 619a124.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105861/
Test PASSed.

@dongjoon-hyun I think the test suites are much better now. Please review it again, thanks!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11148/
Test PASSed.

Test build #105892 has started for PR 24719 at commit 509761b.

Thank you for redesigning, @gengliangwang . I'll take a look.

I left a few comments. The others look good. Thank you for updating.

Test build #105892 has finished for PR 24719 at commit 509761b.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class DataSourceScanRedactionTest extends QueryTest with SharedSQLContext
class DataSourceScanExecRedactionSuite extends DataSourceScanRedactionTest
class DataSourceV2ScanExecRedactionSuite extends DataSourceScanRedactionTest

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105892/
Test FAILed.

Test build #105902 has started for PR 24719 at commit 2161dee.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11157/
Test PASSed.

Test build #105902 has finished for PR 24719 at commit 2161dee.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105902/
Test FAILed.

Test build #105908 has started for PR 24719 at commit 1c895f5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11161/
Test PASSed.

Test build #105908 has finished for PR 24719 at commit 1c895f5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105908/
Test PASSed.

Test build #105922 has started for PR 24719 at commit cbd3236.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11173/
Test PASSed.

Test build #105922 has finished for PR 24719 at commit cbd3236.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105922/
Test PASSed.

Thanks for the review, @dongjoon-hyun @cloud-fan

@gengliangwang Hm, weirdly I'm seeing failures in the master build like...
https://amplab.cs.berkeley.edu/jenkins/view/Spark QA Test (Dashboard)/job/spark-master-test-maven-hadoop-2.7/6415
DataSourceV2ScanExecRedactionSuite:
- treeString is redacted *** FAILED ***
  "== Parsed Logical Plan ==
  RelationV2[a#863554L, foo#863555] orc *********(redacted).7/sql/core/target/tmp/spark-9a065829-75ff-450e-90eb-ec8e344851da

The PR builder passed so I'm not sure what happened. Does that suggest anything to you?

@srowen I think the dot . is not matched.


Update actions to run on main

Since GitHub is/will change the main branch from master to main and I did the switch so all my new projects start with a main branch

Fixed here: 4eefc024e098f948f09880e8bb04012f0edb3fd3


Allows username validation against IAM groups

This change gives the option to validate the remote username against
the IAM groups containing the user invoking the lambda function. This
is an optional feature which is used in conjunction with kmsauth.
For example, if there were two groups of users, you could put your
admins in the ssh-admin IAM group to allow them to generate certificates
with a remote_username of 'admin'. Users with fewer permissions could be
in the ssh-user group to allow them to generate certificates for the 'user'
account.
The group name is configurable, however they must all be in a consistent
format, and must all contain the relevant remote_username once.

Coverage decreased (-1.3%) to 94.494% when pulling d9311db389f2f235fa1483fa07bbb9396f09b55c on hughtopping:master into d2bee4589bfd421f933448586903b8330dbbe639 on Netflix:master.

Coverage increased (+0.1%) to 95.915% when pulling fd1d8020b08acca4305051cef333204bd083ba2c on hughtopping:master into d2bee4589bfd421f933448586903b8330dbbe639 on Netflix:master.

Any idea what's keeping this from being merged? I could benefit from it also

@myoung34 - not sure I'm afraid.
@russell-lewis - please let me know if there's anything I can do to help!


Changed QONNX_TO_FINN_PYTHON verification step name to match build co…

…nfig name
QONNX_TO_FINN_PYTHON verification step is named "finn_onnx_python" in build_dataflow_config.py, but in build_data_steps.py it is named "qonnx_to_finn_python". Changed to "finn_onnx_python" to match the build config, so the verification file is created with the expected name

Thanks @hannahxy13 for adding the additional verification step for the builder flow!


Way to decode the message_blob in transactions that supports new and legacy message

This Gist, decode_radix_message_with_legacy_support.py, is intended to be pseudocode. Is there a way you could formalize this and include message decoding in this wonderful library?
https://gist.github.com/dhedey/02d23e1ff480e355d68713e201c35f50
Thank you!

Hey @chiggimps, thank you for recommending this! This is a feature that I'm currently working on in the develop branch. You can expect to see it in v1.0.1 which I will be releasing soon.

Hey @chiggimps, this has been added in v1.0.1 in the form of the radix.utils.decode_message function and it's function overrides.


[Bug] [UI Next] miss fixvalue input

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
miss fixvalue input

What you expected to happen
fix miss fixvalue input
How to reproduce
open data quality , comparison type  select fixValue
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

@zengqiyan Will you still submit pr

已收到，谢谢


Added external link for formatted geoshape results

@chaudum This links to http://geojson.io/ - but the geo_point data type links to a google maps link. Do you think this is fine, or do you think both should use geojson, or at least openmapbox?

Current coverage is 20.83% (diff: 100%)

Merging #364 into master will increase coverage by 0.09%

@@             master       #364   diff @@
==========================================
  Files            20         20
  Lines          1635       1637     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            339        341     +2
  Misses         1296       1296
  Partials          0          0

Powered by Codecov. Last update 0b481c5...68c1da2

@Autophagy I like geojson.io because you can also see the geojson input next to the map. I would point both types to geojson.io

@chaudum I can point a geopoints coordinates at geojson.io but it doesnt place a marker down at the point, like gmaps does - it just centers the map on that location. A solution could be to encode the lat & long of a geopoint into a geoarea point, like:
{
        "type": "Point",
        "coordinates": [
          15.380859374999998,
          47.81315451752768
        ]
}```

Which would place a marker, but I worry it would be confusing. OSM does support markers in the URL based on lat/long, like this http://www.openstreetmap.org/?mlat=51.510772705078125&mlon=0.054931640625#map=9/51.5108/0.0549

@Autophagy so your suggestion would be to use OSM for both geo_point and geo_shape?

@chaudum My suggestion would be to use geojson.io for geo_shape and OSM for geo_point. Geojson uses OSM, so both maps would be at least visually consistent


299 datetime.datetime timestamps tz aware

Fixes #299

Thanks for your PR. I am working on it now. I changed it to be Python 2/3.0 compatible. Also I decided to go for an additional key called "datetime" to store a datetime.datetime (including TZ=UTC) and "datetime_locale" with TZ from HFTZN to not break compatibiliy. I will make merge soon with 1-2 days.

Your PR has been adopted and merged. Please let me know. Next step is skip_duplicate

Thanks for your PR. I am working on it now. I changed it to be Python 2/3.0 compatible. Also I decided to go for an additional key called "datetime" to store a datetime.datetime (including TZ=UTC) and "datetime_local" with TZ from HFTZN to not break compatibiliy. I will make merge soon with 1-2 days.

Looks good! Agree that your implementation is preferred.
For next time, feel free to edit my branches. For me it feels nice to have my contributions showing up in the github repo :)

Yes, I will. Looking forward for new contributions from you! :-)


Multiple environments in the same scene slow down the FPS

Hi, I am using this repo for my RL training using mlagent. As the doc of mlagent recommends for parallelization, you can simply copy your environment in one scene multiple times and get a training speedup.
When I used grid-sensor and did this simple copy, I got a bad performance, e.g., the agent just did not move and the FPS is very low. I am wondering if this is a problem because you use tag to identify the Detectable GameObject and if you have multiple environments containing the same tag, then there's too much to process.
Thx.

Hi, the tagging shouldn't be an issue with multiple environments. If you look at the example projects, https://github.com/mbaske/grid-sensor/tree/master/Assets/Examples - in Food Collector for example, you can easily stack a couple of platforms on top of each other without affecting performance too much. Each sensor should only process the Detectable GameObjects inside its respective detection area.


Update R & SAS to be on same dataset & write comparison

Update R & SAS to be on same dataset & write comparison

@YeoYeoYeoYeo Thanks for your Mann Whitney/Wilcoxon rank sum content.   Just checking if you were working on any comparison between SAS & R, because @AgnieszkaTomczyk  have volunteered to take this on.


Hide true host name when using domain fronting

Kudos to @malcomvetter for suggesting the solution implemented here.
Currently, when adding a host header via the .Net framework, as is done in the PowerShell multi/launcher,  the host name gets added to two places:

The HTTP header
The Server Name Indication (SNI) extension of the TLS client hello

The SNI is always sent in cleartext, giving away the true hostname that you are using behind your domain front. This code change simply adds a request to the root of the domain front before adding the host header, allowing the TLS connection to complete. Subsequent requests reuse this pre-established TLS connection. Therefore, adding the host name header at his point results in the value only being added to the HTTP header and not showing up in the client hello.

Neat little trick!

Good find!

I'm using the latest version of PowerShell Empire (and I even checked if the code you wrote was in http.py), but the true destination is still revealed in the SNI, these are the steps I took:
listeners
uselistener http
set Name DomainFrontingHTTPS
set CertPath /home/empire/Empire/data/
set Host https://a0.awsstatic.com:443
set DefaultProfile /admin/get.php,/news.php,/login/process.php|Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) like Gecko|Host: ************.cloudfront.net
set Port 443
execute

Does perhaps somebody have an idea on how to fix this? Maybe I'm doing something wrong here.

@sjosz where you using a python launcher in this case or a powershell launcher?

PowerShell, I havent tested the Python launcher though.


Add testlib method to use stuffers as io instead of sockets

I wanted to use this setup in another PR, but don't want to rewrite it :)
Issue # (if available):
None
Description of changes:
Mutliple test classes repeat the same test setup code. Move
it to testlib so we only have to write it once.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (sm_1@1375431). Click here to learn what that means.
The diff coverage is n/a.

lgtm, one nit. nice simplification!


Create multiple gallerys with exclusive mode no thumbnails.

Hello guys.
First of all, congrats for this great plugin :)
I have been messing around with this and trying to adapt the plugin for my project, but unfortunately i cant seem to find the solution...if there is one :(
I am using JQuery Mobile and i need to have 2 slideshows in my project with no thumbnails. I can insert one slideshow with no problems, but how can i have another slideshow (with different images) in the same project without getting always this error?
Code.PhotoSwipe.activateInstance: Unable to active instance as another instance is already active
Here is the code that i use to add one slideshow:
$('div.gallery-page-produtos')
.live('pagebeforeshow', function(e){
			photoSwipeInstance = PhotoSwipe.getInstance('products');

			if (typeof photoSwipeInstance != "undefined" && photoSwipeInstance != null) {
				$('.ps-document-overlay').fadeIn();
				$('.ps-uilayer').fadeIn();
				$('.ps-carousel').fadeIn();
			}
			else
			{
				options = {
						imageScaleMethod:'zoom',
						allowUserZoom:false,
						autoStartSlideshow:true,
						captionAndToolbarHide:true,
						enableKeyboard:false,
						enableMouseWheel:false,
						preventHide: true,
						margin:0,
						maxUserZoom:0,
						minUserZoom:0,
						getImageSource: function(obj){
							return obj.url;
						},
						getImageCaption: function(obj){
							return false
						}
					},
					instance = PhotoSwipe.attach(
						[
							{ url: 'images/full/001.jpg'},
							{ url: 'images/full/002.jpg'},
							{ url: 'images/full/003.jpg'},
							{ url: 'images/full/004.jpg'},
							{ url: 'images/full/005.jpg'},
							{ url: 'images/full/006.jpg'}
						],
						options,
						('products')
					);

				instance.show(0);
			}
		})

		.live('pagebeforehide', function(e){

			$('.ps-document-overlay').fadeOut();
			$('.ps-uilayer').fadeOut();
			$('.ps-carousel').fadeOut();

		});

On page hide i'm not "killing" the gallery, i'm just hiding it. Is this the problem? Do i have to clear all instances before creating a new one? Is there another way to do this?
If someone could help me out i would be grateful.
Thanks in advance

im looking for something like this. a link will open the gallery without thumbnails and the close ('X') in the toolbar will close the gallery and go back to main page.   how do you close the gallery ??

The problem remains :( I just cant destroy the photoswipe instance and create a new one.
And i can't find the right documentation that explains the methods that exist...cause i'm sure there has to be a method to call thats destroys the instance (and detatch doenst do that :( )
help someone?

Labelled this as an implementation issue

It seems this issue is fairly old, I have no idea what the exclusive mode is, but I'd like to know if there's a workaround to trigger PhotoSwipe from a button or a link.


Handling KeyDown events does not prevent TextInput events

Describe the bug
In WPF when one handles KeyDown then TextInput event won't be received.
This is useful when handling single character shortcuts and avoiding weird side effects.
For example in my app user can rebind their shortcuts, one of such shortcuts focuses an input text box. When such letter shortcut is handled we will mark KeyEventArgs as handled and focus the text box. After text box gets focused it receives text input for the handled letter and immediately types something in it.
Expected behavior
I would expect WPF like behavior where handled KeyDown does not produce TextInput.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows
Version: master

The concept of handling e.Handled is complitly opposite to the "standard handling"
In all other keyboard event handlings i knows setting e.Handled = true means stop
handling this key here.
in avalonia e. Handled is used to tell you, that the key is already handled, so it does not
matters whatever you set in in you event handler. is is ignored. check TextBox.cs.
So what can you do.
Create an new "control" and set the style to "TextBox"
public class MyTextBox : TextBox, IStyleable
{
    System.Type IStyleable.StyleKey { get { return typeof(TextBox); } }

	//this is an excerpt of my own textbox control
    protected override void OnKeyDown(KeyEventArgs e)
    {
		if (any condition of your choice)
        {
            base.OnKeyDown(e); //the keydown event is passed th the avalonia engine
        }
		else
		{
			// handling should stop here
		}
}

In all other keyboard event handlings i knows setting e.Handled = true means stop
handling this key here.
in avalonia e. Handled is used to tell you, that the key is already handled, so it does not
matters whatever you set in in you event handler. is is ignored. check TextBox.cs.

It's a bit more complicated. Setting "e.Handled" will prevent this specific event from being raised on next control in its route (see EventRoute.cs#L163). But it won't prevent other handlers in the same route entry from being invoked (see EventRoute.cs#L171, it's just invokes whole delegate without checks before each handled). IIRC it has the same behavior as in WPF/UWP.

In this specific issue, KeyboardDevice should check if KeyDown was previously handled for the same raw event:
https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Input/KeyboardDevice.cs#L196-L247

It's more complicated than that.

KeyboardDevice doesn't necessary have the required information to decide if the event should be ignored
Depending on the system text input might be decoupled from actual event presses. Sometimes the app won't see the key press at all

So we should return the "handled" state to the system when possible, but that kinda conflicts with dispatcher priorities.

FYI: On Mac, setting e.Handled = true suppresses the text event as expected.

Looking into this a little more, as @Jeremy-Price says this is handled correctly on macOS - it seems that win32 simply always sends WM_CHAR even if WM_KEYDOWN is marked as handled. WPF has explicit "mitigation" for this in their HwndSource:
https://github.com/dotnet/wpf/blob/3ccd6f5545fbebf775262af446f2aff819a2226c/src/Microsoft.DotNet.Wpf/src/PresentationCore/System/Windows/InterOp/HwndSource.cs#L1816-L1857

Ok, so it seems that the way to do this is to not call TranslateMessage if the WM_KEYDOWN is handled, however we have a problem: TranslateMessage is called before the DispatchMessage which handles the WM_KEYDOWN so we don't know whether to call TranslateMessage or not.
Looks like WPF does this by first calling the OnPreprocessMessage method outside of the WndProc, which handles key down messages and returns the handled status.


Ability to filter AdapterCommands

I am in a scenario where I want to filter delete commands. If I do so, I enter an inconsistent state since oldList now thinks all deletes were executed.
Since oldList is private, I cannot get away with composition. As an alternative to the PR I made providing an overload method to provide "oldList". You could also opt to make oldList protected so I could use composition to add this behaviour myself which perhaps would be less confusing for users of your library.
Hope to hear back from you.
Thanks

I am in a scenario where I want to filter delete commands.

Couldn't you just filter out the delete commands after having calculated the difference: List<AdapterCommand> commands = commandsCalculator.diff(newItems);
or
Write your own  AdapterCommandProcessor (extend from this class) and do in this something  different with delete commands (or ignore them)

Yes, I could but if I filter out the DeleteCommands, DiffCalculator is left in an ambiguous state.

Could you explain why do you think that DiffCalculator could reach an ambiguous state?
The result produced by calculator.diff() is not depending nor has any reference to newList nor oldList. So modifying the result of calculator.diff() is safe and has no side effects to the internal behavior of DiffCalculator. Am I overlooking something?

If I look at the source code, the oldList is set by the first time you calculate a diff. So lets say this is [1,2,3], this results in insertRange. Now if I calculate a diff with a new list, lets say [1,2,4] that would result in a delete and a insert if I filter out the delete the list in my RV would be [1,2,3,4] but DiffCalculator thinks it is [1,2,4] because calling diff with oldList != null sets the oldList to [1,2,4].
There was an issue in my logic. Because in following situation it wouldn't work anymore.
[1,2,3] -> [1,3,4] => delete 2 at position 1, move 3 to position 1, insert 4 at position 2.
If I filter out the delete, I get a conflict with move 3 to position 1 because 2 is still at that position.
I'm currently working around this by manipulating my data in the data layer so that a delete will not occur instead of filtering out the delete commands. And I guess that's also a better solution instead of doing logic in my view. With my requirements it would require a lot more logic in the view to get it working, and nobody wants that :D
I'm not sure if my requirements could be solved tho, but you definitely will end up in an ambiguous state if you filter out commands! Users may be unaware of that.

Closed due to inactivity


Share y-axis when adding multiple line resources

Checked for duplicates
Yes - I've already checked
Alternatives considered
No - I haven't considered
Related problems
This may be part of https://github.com/NASA-AMMOS/aerie-ui/issues/781, but I'd like to call this out specifically.
I was using the "Add filter to row" function to build a row of related resources like this:

That mostly works great - I love the way it assigns colors and sets everything up really quickly. However, it puts each resource on its own axis, when I'd expect them to share a y-axis. This leads to a misleading display at first glance, because I didn't realize everything was on separate axes:

Describe the feature request
When adding multiple resources to a timeline row, especially if that's a new row, I'd like all the line resources to share one y-axis.

In the meantime I believe you can edit the row in the timeline editor and manually set each axis range; but that's fair bit of work for something that's likely a common use case :+1: Thanks for opening this issue


Auto Generate ApolloConverterFactory with response mappers

The setup process seems a little unwieldy to have to register a new ResponseMapper for each new query.  It would be nice to leverage code gen to make the process easier.  Initial proposal is to generate MyApolloConverterFactory which will already have the response mappers registered. This way adding a new query does not mean that I need to make changes to my restadapter.

That would be very handy indeed, I always forget to register these responseFieldMappers...

How about we will generate for POJO:
public abstract class ResponseFieldMappers {
  Map<Type, ResponseFieldMapper> MAPPERS = Collections.unmodifiableMap(
      new HashMap<Type, ResponseFieldMapper>() {
        {
          put(AllPlanets.Data.class, new AllPlanets.Data.Mapper(AllPlanets.Data.FACTORY));
          put(ProductsWithDate.Data.class, new ProductsWithDate.Data.Mapper(ProductsWithDate.Data.FACTORY));
          put(ProductsWithUnsupportedCustomScalarTypes.Data.class,
                new ProductsWithUnsupportedCustomScalarTypes.Data.Mapper(ProductsWithUnsupportedCustomScalarTypes.Data.FACTORY));
        }
      }
  );

  private ResponseFieldMappers() {
  }
}

And now instead of:
new ApolloConverterFactory.Builder()
            .withResponseFieldMapper(AllPlanets.Data.class, new AllPlanets.Data.Mapper(AllPlanets.Data.FACTORY))
            .withResponseFieldMapper(ProductsWithDate.Data.class, new ProductsWithDate.Data.Mapper(ProductsWithDate
                .Data.FACTORY))
            .withResponseFieldMapper(ProductsWithUnsupportedCustomScalarTypes.Data.class,
                new ProductsWithUnsupportedCustomScalarTypes.Data.Mapper(ProductsWithUnsupportedCustomScalarTypes.Data.FACTORY))
            .withCustomTypeAdapter(CustomType.DATETIME, dateCustomTypeAdapter)
            .build()

we will have this:
new ApolloConverterFactory.Builder()
            .withResponseFieldMappers(ResponseFieldMappers.MAPPERS)
            .withCustomTypeAdapter(CustomType.DATETIME, dateCustomTypeAdapter)
            .build()

So you will do it only once and no need to register new mappers, they will be added by generated to ResponseFieldMappers.MAPPERS
@martinbonnin @digitalbuddha is it works for you?

:+1:  Works for me

Me too! Great idea

The generated ResponseFieldMappers class seems not to be put in the correct package, unlike the other generated files, therefore I am not able to use it

Ahah, I just made the same comment in the PR thread: https://github.com/apollographql/apollo-android/pull/202

And while I'm at it, the fragments are generated in the fragment package, which is a bit strange, would be nice to be able to put them in com.example.apollo or whatever same package as the rest of the generated classes.

I had this patch as a solution, but its not very elegant since it just uses the file path of the first found operation as the package name:
diff --git a/apollo-compiler/src/main/kotlin/com/apollographql/android/compiler/GraphQLCompiler.kt b/apollo-compiler/src/main/kotlin/com/apollographql/android/compiler/GraphQLCompiler.kt
index e5fd66c..fd49f9c 100644
--- a/apollo-compiler/src/main/kotlin/com/apollographql/android/compiler/GraphQLCompiler.kt
+++ b/apollo-compiler/src/main/kotlin/com/apollographql/android/compiler/GraphQLCompiler.kt
@@ -61,8 +61,9 @@ open class GraphQLCompiler {
         }

     if (!context.abstractType) {
+      val packageName = operations.first().filePath.formatPackageName();
       val typeSpec = ResponseFieldMappersTypeBuilder(operationJavaClasses).build()
-      JavaFile.builder(irPackageName, typeSpec).build().writeTo(outputDir)
+      JavaFile.builder(packageName, typeSpec).build().writeTo(outputDir)
     }
   }

This is all because the IR file is directly under the graphql directory.
formatPackageName transforms 'src/main/graphql/com/example/fileName' to 'com.example', which is empty if there's no 'com/example'... So everything would be fine if the IR file were in a subdirectory. Not really sure what we want for an arborescence there...
https://github.com/sav007/apollo-android/blob/66e4e1e4c042bda2a24cdc1a23cb2de2721004d3/apollo-compiler/src/main/kotlin/com/apollographql/android/compiler/GraphQLCompiler.kt#L69

Also, I would be happy to enforce all operations to be in the same directory, I don't see a use case to have subdirectories but maybe I'm missing something. In my case, I even have all the operations in the same file :-D

For the record, and to reflect the slack discussion, everything works fine if you put your schema.json file next to your .graphql files

Thanks, putting the file there works for me! Maybe it should be changed there in the example app as well.

@chrisvis @martinbonnin Here is the logic:

Package for operation (query) defined by the relative path of graphql file from ../graphql/.. folder. So if *.graphql file path is  /Projects/apollostack/apollo-android/apollo-sample/src/main/graphql/com/example/GithuntFeedQuery.graphql the package will be com.example for GithuntFeedQuery.java

Pckage for type that can be used across different queries (fragments, enum types etc.) defined by the the relative path of schema file from ../graphql/.. folder + suffix fragment or type.
So for fragments if schema file is under /Projects/apollostack/apollo-android/apollo-sample/src/main/graphql/schema.json the package will be fragment

But I see your point, ResponseFieldMappers is generated under relative path of schema file only (without any suffix). I will create issue to put ResponseFieldMappers to package define by file path to schema + mapping suffix.


WIP tabix

This PR adds Tabix supports and indexed VCF reading.
All Tabix code is currently detail and will probably stay there for now.
Some preliminary "benchmarks":

bio w/o index
bio w/ index
bcftools w/ index

filesystem inputs
628,748
18,557
13,051

time
53.1s
1.7s
3.2s

Due to architectural problems, I don't think we can ever get the IOPS to be as low as with htslib. Please see my comments in the PR. In practice, the results seem to still be OK, but this is just one example where I tried a region very far to the end of a 300MB compressed VCF. We definitely need to do more testing.
TODO

[x] stand-alone tests for tabix input
[ ] do we want to add files to our unit tests that result in multiple BGZF blocks? Probably good idea, but also big.
[ ] discuss indexing strategy with @smehringer and @joshuak94

Do you know about the BAMIntervalTree created by @joshuak94 for indexing BAM files in another way?

Do you know about the BAMIntervalTree created by @joshuak94 for indexing BAM files in another way?

Thanks for the pointer! I talked to him about. But even if we want to support that, we also need to be able to handle the regular indexes.

It doesn't yet contain all the tests I would have liked, but this is as much as I can currently do for this feature.


card-deck, Mobile screen, Unexpected margins behavior and separations in rows

On narrow screens:

It looks like some unnecessary margins(left) appear. (Narrow-medium screen)
There is no margins(top, bottom). (Narrow screen)
With four cards: three cards on one row, one card on the next row (Narrow-medium screen)
Codepen(You will see the bug when squeeze screen horizontally):
http://codepen.io/Gavelock/pen/vgMNOx
Tested on Mozilla Firefox 53.0a2 (2017-02-05) (64-бит), Linux Ubuntu 16.04 LTS

Somehow related to #21976


Better plotting with dates

Right now it's not possible to plot dates with 2d data.
Also should make more use of the builtin matplotlib capabilities for handling dates such as:
http://matplotlib.org/api/figure_api.html#matplotlib.figure.Figure.autofmt_xdate

I think this can be closed.

Yes, this works currently.


Fix bad property key genie.retry.archived-job-get-metadata.initialDelay

A property has a bad key in code.

Coverage decreased (-0.007%) to 92.855% when pulling 1fac2fde2a578ea5b5b464a5d6c1c8b40c365b63 on mprimi:GENIE-544-retry-property-bug into b9581ee7064c5372f33b9345feef7b6c9c12712b on Netflix:master.


merge

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #8 (6132563) into master (cab110e) will increase coverage by 0.41%.
The diff coverage is 89.83%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   80.38%   80.79%   +0.41%
==========================================
  Files         318      319       +1
  Lines       25598    25633      +35
==========================================
+ Hits        20576    20711     +135
+ Misses       5022     4922     -100

Flag
Coverage Δ

hive
?

mysql
80.68% <89.83%> (?)

postgres
80.71% <89.83%> (?)

python
80.79% <89.83%> (+0.41%)
:arrow_up:

sqlite
80.32% <89.83%> (?)

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

Impacted Files
Coverage Δ

superset/db_engine_specs/mssql.py
95.45% <ø> (ø)

superset/db_engine_specs/mysql.py
94.02% <ø> (+1.49%)
:arrow_up:

superset/db_engine_specs/postgres.py
96.84% <ø> (+1.05%)
:arrow_up:

superset/viz.py
55.59% <0.00%> (-0.07%)
:arrow_down:

superset/utils/encrypt.py
88.00% <88.00%> (ø)

superset/app.py
81.37% <100.00%> (+0.19%)
:arrow_up:

superset/config.py
91.03% <100.00%> (+0.06%)
:arrow_up:

superset/connectors/druid/models.py
81.99% <100.00%> (ø)

superset/databases/commands/exceptions.py
93.18% <100.00%> (+0.15%)
:arrow_up:

superset/databases/commands/test_connection.py
98.14% <100.00%> (+0.03%)
:arrow_up:

... and 30 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cab110e...6132563. Read the comment docs.


[DNM] Pr2207 copy 2

This was used for testing mynewt-travis-ci and the functionality was merged already.


Add QualCoder

Project description
QualCoder is a qualitative data analysis application written in python3 and Qt6.
Text files can be typed in manually or loaded from txt, odt, docx, html, htm, md, epub and pdf files. Images, video and audio can also be imported for coding. Codes can be assigned to text, images and a/v selections and grouped into categories in hierarchical fashion. Various types of reports can be produced including visual coding graphs, coder comparisons and coding frequencies.
This project has been tested under Ubuntu 22.04 and Windows 10/11. It has been used on MacOS and various Linux distros. Instructions and other information are available here: https://qualcoder.wordpress.com/ and on the Github Wiki.
Metadata

homepage URL: https://qualcoder.wordpress.com/
source URL: https://github.com/ccbogel/QualCoder
license: mit
platforms: linux, darwin, windows

I started working on a flake here: https://github.com/AlexChalk/qualcoder-nix. I haven't tested anything, and it currently includes an impure shellscript, but the package is building successfully.

I think this is more idiomatic now (and can be built and run with just nix run): https://github.com/AlexChalk/qualcoder.
But I'm not sure how to add a flake to nixpkgs, and I wouldn't consider doing that before I've spent some time using the built package.


Update material root import schematic to function with v9

To give some context. We currently rely on the type checker to figure out where imports to @angular/material resolve to. This won't work in v9 anymore because we removed the primary entry-point.
This means that we either leave the migration broken and say that people should sequentially update to v8 and then to v9, or we fix it by introducing some logic to map imported symbols to their entry-point.
We decided that we want to create a map that can be used to lookup the entry-point for a given symbol by name. This map can be most likely generated through the TS compiler API.


PHP 8.1 Support

Fixes: https://github.com/TopOnePercent/baum/issues/25

@bkintanar under /.github/workflows/run_tests.yml can you add php 8.1 to the github actions test? so we can ensure everything checks out.
Change this:
strategy:       matrix:         php: [8.0, 7.4, 7.3]
To this:
strategy:       matrix:         php: [8.1, 8.0, 7.4, 7.3]

@foresight-developers done.

@foresight-developers i see warnings. but the pipeline is failing. I'm not sure why.

@bkintanar we have it set where even warnings cause it to fail. We need to look into why we are getting those warnings. Depending on that answer we may add a comment to have the test ignore them. I think you are still fine to do the migration of the xml file to the new format, as that isnt the issue I believe.

@foresight-developers ok. I've restored the updated phpunit.xml

@bkintanar we are looking into this issue over the next few days, unfortunately we cant dedicate someone to this full time currently, but we will let you know if we figure anything out. Again many thanks for supporting this project!

@bkintanar we will be looking at this in the coming weeks but if you want to try the following:
"You can rename the trait from NodeModelExtensionsTest to NodeModelExtensionsTestTrait.
Starting with PHPUnit 10, files in your configured test directory that end with a configured suffix (see testsuites>testsuite>directory) in phpunit.xml must be classes extending PHPUnit\Framework\TestCase.
I couldn't find an explanation for why it was changed unfortunately. A related issue on GitHub for some context: https://github.com/sebastianbergmann/phpunit/issues/4979."

@bkintanar looks like that has done the trick, the warning is not an issue so we will have to put something in place to ignore it, most likely in the github actions

@foresight-developers updated. sorry to take so long. can this be merged now?

@bkintanar we are reviewing the code right now, if everything checks out we can merge.

Thanks! Can we tag a release version too please?

@bkintanar I need to add to the readme the newly supported php version etc, this also supports laravel 10 right? so https://github.com/TopOnePercent/baum/pull/31 can be closed?

@bkintanar Thanks for all the help here it is: https://github.com/TopOnePercent/baum/releases/tag/3.1.0 We're based in the U.K, but I will be back online 6 hours if you need anything else.


killed rkt service does not deregister with machined

To reproduce:
1, create a systemd service that runs rkt
$ cat /run/systemd/system/test.service
[Service]
RemainAfterExit=true
ExecStart=/opt/rkt/rkt run --insecure-options=image,ondisk --net=kubenet docker://busybox --exec /bin/sleep -- 3000
KillMode=mixed

2, start the service and check machinectl
$ sudo systemctl start test.service

$ machinectl
MACHINE                CLASS     SERVICE
rkt-224ad648-f51e-4360-9ceb-7de24d400432 container nspawn

1 machines listed.

3, kill the service and check machinectl
$ sudo systemctl kill test.service

$ machinectl
MACHINE                CLASS     SERVICE
rkt-224ad648-f51e-4360-9ceb-7de24d400432 container nspawn

1 machines listed.

It shows the machine does not deregister.
I am running this with system 225 on coreos-alpha-962-0-0-v20160218 on gce.
$ systemctl --version
systemd 225
-PAM +AUDIT +SELINUX +IMA -APPARMOR +SMACK -SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT -GNUTLS -ACL +XZ -LZ4 +SECCOMP +BLKID -ELFUTILS +KMOD -IDN

Also notice that if I run systemctl stop to stop the service, the machine gets deregistered.

Why did you have remainafterexit=true in the first place?

Not sure if it is linked or not but when running my scheduled worker in containers i frequently have some
Failed to register machine: File exists
The error message might be coming from the same flags passed to systemd-nspawn (ie: keep-unit).
systemd-nspawn-register

Reported upstream: https://github.com/systemd/systemd/issues/2770

Why did you have remainafterexit=true in the first place?

This is for the old time when we didn't have rkt api service, so we used systemctl list-units to get the rkt pods we are launching.
There is still one place use that (in Garbage collection), but we definitely can update the rktnetes code to remove this.

There is still one place use that (in Garbage collection), but we definitely can update the rktnetes code to remove this. @jonboulle

@yifan-gu is this still relevant or can we close it?

Closing.


Frequently asked question

We get this a lot:
"These two warning messages when setting up phone sign-in says I am giving my company access to my phone. I don't want anyone spying on me this is my personal device!"

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

ID: 375ed522-cdda-d5e2-2791-21ab320e9e76
Version Independent ID: 2a733534-ac0c-0645-fa69-9596a7e609ec
Content: Sign in to your accounts using the Microsoft Authenticator app - Azure Active Directory
Content Source: articles/active-directory/user-help/user-help-auth-app-sign-in.md
Service: active-directory
Sub-service: user-help
GitHub Login: @eross-msft
Microsoft Alias: lizross

@jonwbstr Thanks for your feedback! We will investigate and update as appropriate.

Hey @jonwbstr can you please provide a screenshot of this issue you're saying you receive alot of?

A screenshot of an email or support ticket, or phone transcript where a
user gave us this response?
On Thu, Aug 29, 2019, 1:16 PM Frank Hu<EMAIL_ADDRESS>wrote:

Hey @jonwbstr https://github.com/jonwbstr can you please provide a
screenshot of this issue you're saying you receive alot of?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/38005?email_source=notifications&email_token=ADXTPAC7ZTMI2JJUMHQMC2LQG7775A5CNFSM4ISELHT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5PGJUI#issuecomment-526279889,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADXTPACU2T6IH2VZLS7HCRLQG7775ANCNFSM4ISELHTQ
.

Hey @jonwbstr i'm following up on this issue, can you please respond in regards to the last post sent out?

Hi Frank, the last reply is me confirming what yourooking for, please respond to my previous comment, thanks!

Hey @jonwbstr oh sorry! The comment didn't show up when I opened up the git issue.
I'm not sure I understand what the issue is here, can you please provide some more context and an explanation of the issue that you're seeing?
All screenshots that you can provide are helpful.

We help clients enable this feature and field questions from their desktop support team. A frequent question that comes in to support from users who enable this feature on their phones is:
"Hi, I decided not to enable this feature because it says I have to register the phone with the company, and I don't want the company having access to my phone."
This is one of the screen's, working on the other one. But they are both about granting concent to register the phone in organizations directory. "Registration" is being interpreted by users, that the company has a level of access to information about the phone and what is on it, that is broader than what it actually means.

User1 wrote:
"so the authenticator app is saying that this requires me to register my device, so it can be trusted to access org resources. this isnt going to take over my phone with company resources is it? just want to be sure there is no intune/company policies and/or complete access to my phone by the company."
User2 wrote:
"I just registered my device, does that mean you can see all the websites I visit now?"
User2 wrote:
"If this is required the company needs to provide me a company owned device"

The intune app protection polices requires installing the intune management portal app, but doesnt require registering the device with intune... Different issue, I got them mixed up. This is the only screenshot relevant to the phone sign in experience.

Hello @jonwbsr I see. I'm not sure what the request is. So are you saying provide a question in the FAQ with an answer for the question :
These two warning messages when setting up phone sign-in says I am giving my company access to my phone. I don't want anyone spying on me this is my personal device

All the customer comments I have shared are generated by them not
understanding what device registration does or means.
That is the question, what is device registration?
There are lots of ways to address this topic in the faq section, here are a
few examples
Question: What is device registration?
Answer: address question of company access to device
Question: Does device registration give the company access to the
registered device?
Answer: No, and here is why...
Question: What company polices are allowed by registering the device?
On Fri, Aug 30, 2019, 7:44 PM Frank Hu<EMAIL_ADDRESS>wrote:

Hello @jonwbsr I see. I'm not sure what the request is. So are you saying
provide a question in the FAQ with an answer for the question :
These two warning messages when setting up phone sign-in says I am giving
my company access to my phone. I don't want anyone spying on me this is my
personal device
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/38005?email_source=notifications&email_token=ADXTPAD5JZ4LNWVJTWQHIRTQHGWHFA5CNFSM4ISELHT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5TAHTI#issuecomment-526779341,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADXTPAGOW2PWHEPVGWZESK3QHGWHFANCNFSM4ISELHTQ
.

Hey @jonwbstr I see. Alright, i'm currently engaging some more people to determine how we should handle adding this into our FAQ. thank you for bringing this up. We appreciate it!

@jonwbstr  Thank you for this insight into our customers thoughts, it's really appreciated since we usually don't get this level of detail. I definitely think your FAQ items are valid and should be included in our content.  I will add your recommendations to my backlog (work item list) for updating the article as I can. In the meantime, I'm going to close this issue, but I will keep your info and let you know when the updates are in place and published.
#please-close

Hi @jonwbstr
Hi! I'm the writer for this content now. I added this line to the SMS sign-in article (https://docs.microsoft.com/azure/active-directory/user-help/sms-sign-in-explainer) and will add it to the Authenticator FAQ also.
"Registering a device gives your phone access to your organization's services and doesn't give your organization access to your phone."
My plan is to strategically propagate that and similar sentences throughout relevant user help.

Awesome, that is perfect!
On Thu, Apr 30, 2020, 5:33 PM Curtis Love<EMAIL_ADDRESS>wrote:

Hi @jonwbstr https://github.com/jonwbstr
Hi! I'm the writer for this content now. I added this line to the SMS
sign-in article (
https://docs.microsoft.com/azure/active-directory/user-help/sms-sign-in-explainer)
and will add it to the Authenticator FAQ also.
"Registering a device gives your phone access to your organization's
services and doesn't give your organization access to your phone."
My plan is to strategically propagate that and similar sentences
throughout relevant user help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/38005#issuecomment-622126400,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXTPAHLG246GIL43O3FIO3RPHVENANCNFSM4ISELHTQ
.


Browser Refresh not working in warning state

This could be an issue on my end but I can't seem to get handleWarnings to fire which is breaking browser refresh in a warning state. That said I can just run my client like normal with react-scripts but I was hoping to continue running with wpstart.
Here's the block of code that catches. (Inside wpThemeClient.js)

I'm having a similar issue. Any luck in a solution?

Not really, however the build command works fine with warnings. So I just use the normal react scripts start command for development and the wpscripts build command when I'm ready to publish.


 Thanks.

@antoninbas do you intent to signvthe CLA? Thanks.

done

See https://github.com/p4lang/p4runtime/pull/437#issuecomment-1622580483 for more deails.


Does this addon support Angular >=14

I saw in documentation..  it supports only react and ts. can you please confirm me on angular.
If this addon also supports angular it will super helpful to us.

Sorry, it only works with React right now. I could make the code transformations configurable so any framework or library will work, but that will take some effort.

@JeremyRH What is the capacity needed for doing the work to support Angular?  I'm considering contributing this but not sure if I understand the full scope

@kenisteward My initial instincts tell me, you would need to start from the place code is transformed after it's typed into the editor and evald here:
https://github.com/JeremyRH/storybook-addon-code-editor/blob/959bdd0bb2bc9d8791cff12424ad232e4b29bfae/src/evalModule.ts
This should be made library and framework agnostic. Anything React related would be passed in as an argument.
The Preview component calls the function above and uses the default export as a React component. That would have to change.
Storybook plugins use React components to display things. This is fine because a React component can wrap an Angular component. This does mean the Preview component needs some generic way of wrapping whatever the default export of the evaled code produces. Maybe the Preview component can accept a function prop for this.
These are my thoughts but here's probably something I'm not taking into account.

@JeremyRH thanks so much for responding!
Hopefully I can absorb the mental model of the changes quickly enough to have a rough sketch locally. I maintain angular libraries that build forms using json schema and am hoping to use this to allow for intellisense based json input authoring for component inputs.

That sounds interesting! So the user enters JSON in the editor and a form is displayed based on this JSON? And you would like intellisense when the user is typing this JSON?
Side note: I thought about this issue more. Allowing a fully custom Preview component might solve this in a cleaner way than trying to make the Preview component generic.


Filter interfaces by their associated IPs

We have a use case where it's easy for us to pass the IP(s) of the interfaces we'd like to monitor as an environment variable but quite a bit harder to know the names of those interfaces in advance.
Here's a first pass at adding another filter type that allows/disallows interfaces based on whether it's associated with the passed in IPs. There's still some logic around the poller/watcher for interfaces that'd need some additions to also watch for when IPs are attached/unattached and I'm sure other things that would need updates too.
Is there any interest in adding a feature like this? Happy to change approach from what I have here or otherwise do what still needs done to get it merged if so!

👋 @masonj188 and I are from ngrok and we are starting to use netobserv to get better visibility into our edge traffic. Let us know if this is the sort of patch you are interested in accepting and we'll push it forward. In the future we'll continue to upstream changes we need to netobserv as well.

/assign @msherif1234

wave @masonj188 and I are from ngrok and we are starting to use netobserv to get better visibility into our edge traffic. Let us know if this is the sort of patch you are interested in accepting and we'll push it forward. In the future we'll continue to upstream changes we need to netobserv as well.

Thanks @joewilliams and @masonj188 for your contribution I will definitely go over your changes, for now I just wanted to make sure u are considering single and dual stack clusters support and being able to handle both address-families IPs

Hi @msherif1234, really appreciate the quick reply!

just wanted to make sure u are considering single and dual stack clusters support and being able to handle both address-families IPs

Yep! I pushed up some test cases that cover IPv6 addresses as well, should have included those earlier. The list of IPs given in the environment variable are parsed into netip.Prefix which should cover both IPv4 and IPv6 addresses.

@joewilliams and @masonj188, I took a quick look at the code, and this will indeed ease configuration, I really like that. Are you talking about the need to configure specific IPs or about configuration by subnets/netmasks? We are talking about interfaces for specific subnets, right?
Also, can you add needed changes to https://github.com/netobserv/netobserv-ebpf-agent/blob/main/docs/config.md to describe the updated capabilities
THX for the contribution :-)

/ok-to-test

/test all

@masonj5n I would like to get this PR merged can u pls rebase it Thanks!!

@masonj5n I would like to get this PR merged can u pls rebase it Thanks!!

Hi @msherif1234, thanks for the poke! I went ahead and made some changes so it matches my last comment - where you supply an IP CIDR/range and then any interface with an IP in those ranges will make that interface included. I updated some docs and tests to reflect that.


raftstore: getter/setter for region

Add getter and setter for region, so it is easy to add a hook when we updates Peer's region.
Cc #2639

PTAL

LGTM.

LGTM


REQ: Priority DL, push to top of queue.

The ability to push a queued item to the top of the list so that DL starts immediately. Also, the ability to set a priority DL order similar to that in tor clients (high, medium, low) so high priority DL's are completed first and so on.
PS. is there a way we can flag a post as a request so that you don't have to do it? I prepend the post with REQ to help mark it at least.
PPS. Sorry if this is a dupe request. I searched and couldn't find anything similar.

This is on the radar as LFTP does support this.


[bug/feature] 1.2.13-1.11.2 Making a stone button requires killing the Ender Dragon.

When HarderSteelRecipe=true
to make a stone button.
you need
a stone corner
you need a steel anvil
you need soul  forge steel
you need soul forge ingots
you need soul flux
you need ender slag
you need cooked endstone
and to get back you have to kill the ender dragon to make the portal open up.

why does it do that??? That is so broken xD
Just rescan the building btw ;)

http://www.sargunster.com/btw/index.php?title=Hardcore_Redstone_Recipes

That doesn't solve anything tho. With Harder Steel Recipes enabled, you need steel to make the stone cutter that makes the corner stone. The steel with this setting can only be made with ender slag which you can only get from The End. Without a transdimensional/portaling mod installed, you will now have to kill The Enderdragon to activate the portal to The Overworld.

@kreezxil What he means is that it's intended

Are you kidding me? you made an entire mod to fix a recipe I would've kindly added a config for?

You can still make a fix for it.
On May 31, 2017 10:28 AM, "primetoxinz"<EMAIL_ADDRESS>wrote:

Are you kidding me? you made an entire mod to fix a recipe I would've
kindly added a config for?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/BeetoGuy/BetterWithMods/issues/386#issuecomment-305223712,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADs0iCU0ox9woSm0w3XRXpxK8XpVplXuks5r_YcLgaJpZM4NrbKA
.

I did!

Excellent.
On May 31, 2017 11:58 AM, "primetoxinz"<EMAIL_ADDRESS>wrote:

I did!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/BeetoGuy/BetterWithMods/issues/386#issuecomment-305250643,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADs0iNuRP-E473U_rTMpTcQBRlYcbvJiks5r_ZwbgaJpZM4NrbKA
.

Next time, wait before getting upset about something and making an addon or I won't be so kind


Missing requirement (Challenge [Waypoint: Use Comments to Clarify Code])

Challenge Waypoint: Use Comments to Clarify Code has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.86 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

jQuery Playground

#left-well

        <button class="btn btn-default target" id="target1">#target1</button>
        <button class="btn btn-default target" id="target2">#target2</button>
        <button class="btn btn-default target" id="target3">#target3</button>

#right-well

        <button class="btn btn-default target" id="target4">#target4</button>
        <button class="btn btn-default target" id="target5">#target5</button>
        <button class="btn btn-default target" id="target6">#target6</button>

Unlike the instructions, the requirements do not state that you have to put the comment at the top of the code.
https://gyazo.com/2384e8e2e392eede40b0d73cbe79bfe2

:+1:


Suggestion: Make get_stack() available in exported exe (release mode) too

Godot version:
Godot_v3.1.1-stable_win64
Issue description:
Currently get_stack() seems to work only in the editor, as far as I can see. I believe it may be even more useful in the exported exe (in release mode), so I suggest making it work there too.
Here is a function I was planning to use in my game as a replacement for assert():
# replacement for assert, aborting in debug mode, logging in debug and release modes
func ass (cond :  bool, # if conditional is not true, pretty fatal
          msg  := ""    # message to print to log
         ) -> void:
   var fh  := File.new() # file handle
   var err := OK         # error code

   # if failed
   if not cond:

      # open new file
      err = fh.open(D.LOG_FILE_PATH_FULL, File.READ_WRITE)
      assert(err == OK)

      # write to log
      fh.store_line("%d: %s" % [OS.get_ticks_msec(), msg])
      fh.store_line(str(get_stack()))

      # close file
      fh.close()

      # abort in debug mode
      assert(cond)

It works just fine, except that get_stack() returns an empty array in the exported exe (regardless of whether "Export With Debug" is checked or not), where it is most needed, since in the editor the information is already available in the debugger.
P.S. I expect the same could be said for print_stack() too.

Wouldn't that mean the release build should keep the stack trace? I think it's not supported there at all for performance reasons.

Yes. It would mean that.
I forgot to mention that I am suggesting to add this as an optional project setting (off by default). If one wished to pay the performance penalty for the added debug capability (I expect I would), then one would enable that setting.

I second wanting get_stack() to work in release executables. I am currently implementing a logger gdscript so users can submit logs to me when my game breaks and being able to get a stracktrace would be very useful.

Moved here:
https://github.com/godotengine/godot-proposals/issues/105


Java: Unexpected Function Invocation Behavior For Java SDK

I test the function invocation on the version of 1.0.0-rc2 by a simple plugin.
TimePlugin: just one method which supply a string of current time
when I call the getTime() function, it returns 2020-10-10 19:53:05, or maybe some other time in the past years, which is not my expected result.
I need some help to fingure this out, thanks.
the code look like the followings:
public String getTime() {
    KernelPlugin getCurrentTime = KernelPluginFactory.createFromObject(new TimePlugin(), "time");
    // add plugin into the kernel
    Kernel textCompletionKernel = textCompletionKernelBuilder.withPlugin(getCurrentTime).build();
    KernelFunction<String> kf = KernelFunctionFromPrompt.<String>builder()
         .withTemplate("tell me the current time in the format: yyyy-MM-dd HH:mm:ss")
         .withName("getCurrentTime")
         .withOutputVariable(new OutputVariable("the current time string", "java.lang.String"))
         .build();
    return textCompletionKernel.invokeAsync(kf)
         // tell to use the getCurrentTime function
        .withToolCallBehavior(ToolCallBehavior.requireKernelFunction(getCurrentTime.get("getCurrentTime")))
        .withResultType(ContextVariableTypes.getGlobalVariableTypeForClass(String.class))
        .mapNotNull(FunctionResult::getResult)
        .block();
}

public static class TimePlugin implements Serializable {
    @DefineKernelFunction(name = "getCurrentTime",
            description = "获取当前时间",
            returnType = "java.lang.String",
            returnDescription = "返回当前时间，格式为yyyy-MM-dd HH:mm:ss",
            samples = {@SKSample(inputs = "", output = "2024-04-01 01:01:01")})
    public String getCurrentTime() {
        LocalDateTime now = LocalDateTime.now();
        return DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss").format(now);
    }
}

This is a bug with required tool call invocation, effectively it gets requested multiple times, a fix is in #5962 and your sample works for me with that patch

@johnoliver Thank you for the latest update. May I ask when the next 1.x Maven jar will be released（Will it still be RC version）?

@zzzhy we are already at 1.0.1, so we are out of RC, I would guess a release next week as we have fixed a few things since 1.0.1

I believe the issue lies with the textCompletionKernelBuilder. When I switched to using chatCompletionKernel, everything worked fine. Upon reviewing the OpenAIChatCompletion.java file, I found the following code, which is responsible for handling tool calls. (The internalChatMessageContentsAsync function)

However, in OpenAITextGenerationService.java, there is no code related to tool processing.


Add all local street names to suggestions

I think it would be useful to make the streetname searchable, so that it suggests streets that are already in the database. Even nicer would be of course to have a list of all streets of Karlsruhe, but this would make it harder to use the app in other cities

That's a great idea. I already thought about using some API like Google Maps for addresses. Using streets from our database might be a good alternative.

That's a great idea. I already thought about using some API like Google Maps for addresses. Using streets from our database might be a good alternative.

implemented via #126

implemented via #126

implemented via #126

You can link PRs and issues by using this option on the right side of an issue or PR:

implemented via #126

You can link PRs and issues by using this option on the right side of an issue or PR:


Short solution needed: "How to name image for docker build" (docker)

Please help us write most modern and shortest code solution for this issue:
How to name image for docker build (technology: docker)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to name image for docker build


Add one test case for check diff in container and committed image

The test steps are:

Start a container and touch a file under /tmp
Use diff to check the container
commit the change into a new image
Use diff to check the image

LGTM


workspace or tag number value

hi
i cannot determine the tag or workspace number i am in to put it in the lemonbar ?
is there any way ?

There is not. You'll have to write a patch. :)


inkscape: add wrapGAppsHook for icons (fixed for strictDeps)

Resolves #68185.
The icons in Inkscape depend on gdk-pixbuf loaders, but because
strictDeps is set to true to fix some macOS issues it doesn't work
(see #56943). Adding librsvg to buildInputs explicitly fixes the issue.
Motivation for this change
I want to be able to use Inkscape and icons are kind of important :P
Things done

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

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

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

cc @jtojnar

Thanks.

Thank you!


Is the large dataset available?

Thanks for sharing the great work.
Is the large dataset available?

Unfortunately we are not able to share the dataset due to privacy limitations.  The best we can do in the meanwhile is to provide our training algorithms and weights so that you can vet our approach and take the weights and reinforce with your own local data.

Got it. Thanks for your great efforts very much.


How to Enable only one particular date of every month with Bootstrap DatetimePicker

Hi All,
I am using Bootstrap DatetimePicker. I want to enable 20th of every month and all other dates must be disabled for user to select. Also, user should be able to change the month and year. I tried to do something like :
var DATE = new Date();
var dte = DATE.getDate();
var Fdt = new Date();
Fdt.setDate(20);
var current = moment(Fdt);
this.$('#startdate').siblings(".datetimepicker").data("DateTimePicker").minDate(current).maxDate(current).enabledDates([current]);
With the above code the 20th get enabled and rest of the dates in current month gets disabled, but i am not able to select next month.

Hi, what about using enabledDates it'll accept and array so just capture the year when this change on the picker and the months are basically always the same or even you can do that with the months using an event change you capture the month and the year

Thanks William for the prompt response. if you have seen the code in the original thread of the issue i have already used enabledDates which works fine and sets the default date for current month as 20 in enabled mode, however i am not able to select next months date. Thats my issue, it would be great if you could help me on this.

Hi, you should for that avoid using minDate and maxDate
If you set thoose then you will unable to select anything more than the current date, just remove it and do an array for all the 12 months and always load the year depending on what year is selected for datepicker

Hi William in my case how can i use dp.update there is no example mentioned in the link, can you please help to give an example for that.

Hi, you can use this event and as on the doc say, if  YYYY it's bacause changed the year so then you do the enabledDates with that then you use the moment get, for by example years, you something like:
e.viewDate.year(); //this will get the year on which you will be
Note: e.viewDate this is a moment object so the all the methods of moment.js work on it

Hi, I'll be closing this as there's no feedback from original poster, feel free to re open it if need it.


GW cache settings

 ========================================================================================================================
Failed generating 'target/recordannotations' after 33.3s.
Error: Image build request failed with exit status 1

Looks good. (expected)


libvirt: expose a way to enable the  debug logs

What this PR does / why we need it:
Add a way to enable the libvirt debug logs, to make it easier to troubleshoot VM launch failures.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes N/A
Special notes for your reviewer:
Replaces/Obsoletes https://github.com/kubevirt/libvirt/pull/29
Release note:

Add support for optional annotation `debugLogs`. If present, enable the libvirt debug logs in the virt-launcher pod.

/hold

I'd like to test it and/or add a functest (not sure how feasible), hence the hold

/cc @cynepco3hahue @rmohr

/retest

/hold cancel

manual testing passing. A word of warning: the debug logging is VERY verbose. That's a random VM started from a couple minutes:
1109 10:48:01 fromani@yoshi ~/go/src/github.com/kubevirt.io $ oc logs virt-launcher-win2k12r2-desktop-large-btf9h | wc -l
41486

manual testing passing. A word of warning: the debug logging is VERY verbose. That's a random VM started from a couple minutes:

Haha, looks like a DOS attack :dagger:

/retest

/retest

Code looks good to me, but would it be possible to add a functest, which tests if we actually get debug logs?

Sure thing. On it.

/retest

/lgtm
/approve

all lanes were green... let's give tide a chance and see if it happens again :slightly_smiling_face:

/retest

/retest

/retest

all lanes were green... let's give tide a chance and see if it happens again

it doesn't, at least not within a reasonable time, will merge


OpenSSL v1.1.0 build issues

Getting two errors (well, one error and one warning):
  CXX      experimental/sslheaders/experimental_sslheaders_libsslhdr_la-expand.lo
experimental/sslheaders/expand.cc: In function 'void x509_expand_signature(X509*, BIO*)':
experimental/sslheaders/expand.cc:73:30: error: invalid use of incomplete type 'X509 {aka struct x509_st}'
   ASN1_BIT_STRING *sig = x509->signature;
                              ^~
In file included from experimental/sslheaders/expand.cc:19:0:
experimental/sslheaders/sslheaders.h:26:16: note: forward declaration of 'X509 {aka struct x509_st}'
 typedef struct x509_st X509;
                ^~~~~~~
Makefile:4625: recipe for target 'experimental/sslheaders/experimental_sslheaders_libsslhdr_la-expand.lo' failed
gmake[2]: [experimental/sslheaders/experimental_sslheaders_libsslhdr_la-expand.lo] Error 1 (ignored)

experimental/ssl_cert_loader/ssl-cert-loader.cc: In function 'SSL_CTX* {anonymous}::Load_Certificate(const {anonymous}::SslEntry*, std::deque<std::basic_string<char> >&)':
experimental/ssl_cert_loader/ssl-cert-loader.cc:201:36: warning: 'unsigned char* ASN1_STRING_data(ASN1_STRING*)' is deprecated [-Wdeprecated-declarations]
           char *name_ptr = (char *)ASN1_STRING_data(alt_name->d.dNSName);
                                    ^~~~~~~~~~~~~~~~
In file included from /opt/openssl/include/openssl/bn.h:31:0,
                 from /opt/openssl/include/openssl/asn1.h:24,
                 from /opt/openssl/include/openssl/objects.h:916,
                 from /opt/openssl/include/openssl/evp.h:27,
                 from /opt/openssl/include/openssl/x509.h:23,
                 from /opt/openssl/include/openssl/ssl.h:50,
                 from experimental/ssl_cert_loader/ssl-cert-loader.cc:30:
/opt/openssl/include/openssl/asn1.h:553:35: note: declared here
 DEPRECATEDIN_1_1_0(unsigned char *ASN1_STRING_data(ASN1_STRING *x))
                                   ^
/opt/openssl/include/openssl/opensslconf.h:102:35: note: in definition of macro 'DECLARE_DEPRECATED'
 # define DECLARE_DEPRECATED(f)    f __attribute__ ((deprecated));
                                   ^
/opt/openssl/include/openssl/asn1.h:553:1: note: in expansion of macro 'DEPRECATEDIN_1_1_0'
 DEPRECATEDIN_1_1_0(unsigned char *ASN1_STRING_data(ASN1_STRING *x))
 ^~~~~~~~~~~~~~~~~~
experimental/ssl_cert_loader/ssl-cert-loader.cc:201:72: warning: 'unsigned char* ASN1_STRING_data(ASN1_STRING*)' is deprecated [-Wdeprecated-declarations]
           char *name_ptr = (char *)ASN1_STRING_data(alt_name->d.dNSName);
                                                                        ^
In file included from /opt/openssl/include/openssl/bn.h:31:0,
                 from /opt/openssl/include/openssl/asn1.h:24,
                 from /opt/openssl/include/openssl/objects.h:916,
                 from /opt/openssl/include/openssl/evp.h:27,
                 from /opt/openssl/include/openssl/x509.h:23,
                 from /opt/openssl/include/openssl/ssl.h:50,
                 from experimental/ssl_cert_loader/ssl-cert-loader.cc:30:
/opt/openssl/include/openssl/asn1.h:553:35: note: declared here
 DEPRECATEDIN_1_1_0(unsigned char *ASN1_STRING_data(ASN1_STRING *x))
                                   ^
/opt/openssl/include/openssl/opensslconf.h:102:35: note: in definition of macro 'DECLARE_DEPRECATED'
 # define DECLARE_DEPRECATED(f)    f __attribute__ ((deprecated));
                                   ^
/opt/openssl/include/openssl/asn1.h:553:1: note: in expansion of macro 'DEPRECATEDIN_1_1_0'
 DEPRECATEDIN_1_1_0(unsigned char *ASN1_STRING_data(ASN1_STRING *x))
 ^~~~~~~~~~~~~~~~~~

Found this from the curl repo, which seems to address similar issues (around X509):
https://github.com/curl/curl/commit/7f683b0ea87ed158eb4bd22cdd1f26eb901a97d0

Oh, I think this is fixed in 22994b0dc3ad5827a26a6fed2ee525cc16b476d8, maybe we need to cherry pick this.

This is fixed.


Disappearing tarot cards with Relic

Mod version:
tarotcards-1.19.2-1.2-release.jar
relics-1.19.2-<IP_ADDRESS>
Describe the issue:
When using the tarot vault, the cards may disappear inside. This happens in different situations, but 100% it happens when you use any skill from the Relics mod.
How to Reproduce:
Video - https://cdn.discordapp.com/attachments/833848375383556106/1122576576547143830/Base_Profile_2023.06.25_-_<IP_ADDRESS>.mp4
Additional info:
Modpack

this looks like it could be an issue with relics modifying inventory or copying the players itemstacks wrong. im still trying to dig through their code to figure out where. you can also submit an issue with them to hopefully fix this faster

Hi, I am the developer of the Relics mod.
I can't remember any line of code in the active ability interface that affects player inventory. It uses entirely client-side code that scans CuriosAPI inventory for items, that have an ability instance. If any of relic ability was cast, it sends a packet (https://github.com/SSKirillSS/relics/blob/1.19.2/src/main/java/it/hurts/sskirillss/relics/client/hud/abilities/AbilitiesRenderHandler.java#L504) to the server with the used ability and CuriosAPI slot ID. If CuriosAPI inventory is validated successfully and ability data matches (https://github.com/SSKirillSS/relics/blob/1.19.2/src/main/java/it/hurts/sskirillss/relics/network/packets/abilities/SpellCastPacket.java#L42), ability will be triggered on the server side.
Afaik all the active abilities code works only with RelicItem instances, so it shouldn't affect any other mod.

No news?

hmm no clue what it could be then. the tarot deck doesn't use any netcode, and it just implements forge's itemhandler capability for storing cards. maybe try testing another mod that has an item with inventory, such as thermals satchel, to see if it happens to it as well?


Add another profile for x86_64

On x86_64 MacOS
./mvnw -version shows arch: "x86_64", existing "amd64" doesn't fit.
Follow up to https://github.com/crate/crate/commit/8121f192202b3aa8ccdff63232cb968bf7d6a0c3
Without this change  ./mvnw package fails with
  Could not resolve dependencies for project io.crate:crate-copy-azure:jar:5.10.0
[ERROR] dependency: org.apache.opendal:opendal-java:jar:osx-${os.arch.classifier}:0.46.3 (compile)

Unrelated LR failure
Retest this please

@mergifyio refresh

Let's backport this as well please to all works on 5.9 as well.

Let's backport this as well please to get all working on 5.9 as well.

Thanks, I forgot about the stable branch


[libcln] add libcln

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

Thanks for your PR, please fix the build error on CI pipeline. If this port doesn't support these triplets, please add supports in vcpkg.json.
libcln:x64-uwp
libcln:arm-uwp
libcln:arm-neon-android

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

Thanks for your PR, please fix the build error on CI pipeline. If this port doesn't support these triplets, please add supports in vcpkg.json.
libcln:x64-uwp
libcln:arm-uwp
libcln:arm-neon-android

What is platform expression of arm-neon-android ?

Closing this PR since it seems that no progress is being made. Please ping us to reopen if work is still being done.


Incorrect autodoc links for fkm_nonlinear

The functions referenced in docs/strength/fkm_nonlinear are not found.
Python 3.11.4 (main, Jul  5 2023, 13:45:01) [GCC 11.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pylife.strength.fkm_nonlinear
>>> pylife.strength.fkm_nonlinear.perform_fkm_nonlinear_assessment
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'pylife.strength.fkm_nonlinear' has no attribute 'perform_fkm_nonlinear_assessment'
>>> pylife.strength.fkm_nonlinear.for_material_group
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'pylife.strength.fkm_nonlinear' has no attribute 'for_material_group'
>>>

Fixed in #59


Feature/372 support pylibdmtx

Closes #372

Coverage remained the same at 88.312% when pulling e3aeb00ab77b515bb89cc39bd1430a90d7138558 on feature/372-support-pylibdmtx into cc8431855c905857950075a1885695706db73eaf on master.

Coverage remained the same at 88.312% when pulling ce5dab2b9e2f208d2a24c8e89ea97f48ffb6aac9 on feature/372-support-pylibdmtx into cc8431855c905857950075a1885695706db73eaf on master.

Coverage increased (+1.0%) to 89.262% when pulling 8fb6745f326720a66d63763e3d6a55e43ef3654f on feature/372-support-pylibdmtx into cc8431855c905857950075a1885695706db73eaf on master.


Maximum call stack size exceeded

For issues give the following informations related to your problem:

[ ] System Designer version number: (ex: v2.2.0)
https://designfirst.io
[ ] Platform: (ex: macOS version)
macOs
[ ] Your OS version: (ex: macOS 10.12.6)
macOS 10.12.6
[ ] If use on a phone, please give the model: (ex: iPhone 6S)
[ ] Steps to reproduce the problem:
open url https://designfirst.io/designer/app/index.html#x1e4211a94b10c58

Uncaught RangeError: Maximum call stack size exceeded
    at new Function (<anonymous>)
    at y (system-runtime.min.js:20)
    at Object.t.state (system-runtime.min.js:20)
    at _Logger.s (system-runtime.min.js:20)
    at _Logger.error (eval at <anonymous> (system-runtime.min.js:20), <anonymous>:3:49)
    at y (system-runtime.min.js:20)
    at Object.t.state (system-runtime.min.js:20)
    at _Logger.s (system-runtime.min.js:20)
    at _Logger.error (eval at <anonymous> (system-runtime.min.js:20), <anonymous>:3:49)
    at y (system-runtime.min.js:20)

I have made many improvements regarding error management / metamodel validation in System Designer v2.5.0. So your issue is probably fixed.
I will close this issue. If you have still the issue, please reopen it.


Enhances custom_processing.py

Accepts a directory of images
Checks the output against the label list

Can one of the admins verify this patch?

Since this PR was raised, the Resnet50 example for Vitis AI has been updated to print out more user-friendly categories as this PR proposed rather than just silently validating the response from the server. If there's another suggestion to improve this example, please raise a new PR


Error: The Amazon Appstore is not currently available in your country.

[Enter feedback here]
There is no any mention about Amazon Appstore is still locked for US region users only (as it was on previous version on this page). Really disappointed with this.

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

ID: 31f9a801-c5b5-94e7-41b9-1b2b4a1d27de
Version Independent ID: 31f9a801-c5b5-94e7-41b9-1b2b4a1d27de
Content: Windows Subsystem for Android™
Content Source: hub/android/wsa/index.md
Product: dev-environment
Technology: windows-android
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

Thanks for pointing this out @iwalker2000. I added a note about it back into this doc: https://github.com/MicrosoftDocs/windows-uwp/commit/ccee6e62c1ff0058dab09cf6ad7b37c8b72dd488
Hopefully the supported regions for running WSA will expand soon as the feature continues to be developed.


python client..

pip3 install bridges
seems to be broken. Needs to looked into rightaway, as we have uses waiting to use it.
Assigning to Alec as well, if he can find some time!

Is it? Do you have logs that show that it doesn't work? (It works here.)

works now