encrypt decrypt without password

Is  there a way to encrypt a zip file witout a password, only using private/public keypairs?
I want to make a zip file which can be opened only by a private key.

Yes there is. This feature of zip is called Strong Encryption. But this feature is PKWARE Proprietary Technology and is a patent held by them and requires license from them and that is probably the reason we don't see this feature in a lot of zip utilities. Here is a quote from the zip specification:

7.0.1 Portions of the Strong Encryption technology defined in this
specification are covered under patents and pending patent applications.
Refer to the section in this document entitled "Incorporating
PKWARE Proprietary Technology into Your Product" for more information.


templates: Move case-studies and /for pages to corporate folder.

discussion: https://chat.zulip.org/#narrow/stream/6-frontend/topic/landing.20page.20.3D.3E.20corporate
Related issue: #15396

Merged the important bug fix as 112dfe60bf4c88e0340b1f5b502460955641c3eb.
Reading this, I think we should probably just move all the landing pages in this PR -- I don't think we want to change /hello, /plans, etc. to use absolute links, since we'll want to change them back to relative links ultimately (very important for local development), and all of those are landing pages that are intended to be moved.
(But I think it does make sense to have one commit for moving the case studies, and then individual commits for individual pages like /features or /plans that are pretty big and might be linked a bunch).
Ideally, we only do relative=>absolute link migrations in places like the /help/ that we intend to not move.

Makes sense!

Thanks @andersk for the review. Sorry, I should have mentioned this was not ready for review. I was taking this task slow migrating few files at a time.

Just a bump on this -- it'd be nice to get this technical migration completed while Alya's still travelling.

This is ready for another review.

This is ready for another review.

This is great, merged, thanks @amanagr!
I think landing_nav.html can likely be moved to corporate/ as well, but it'll require a bit of work, since it's included in integrations.html; what we probably want is to only inline landing_nav for CORPORATE_ENABLED, and thus have those links move back to relative (which will improve the development environment experience when working with that part of the site, since you won't risky randomly moving to the real zulip.com). I'll post about this in the existing chat.zulip.org conversation.


Allow setting custom pragmas

This allows for someone to set any PRAGMA keys and values, next to the most common ones. See the documentation for a full list: https://www2.sqlite.org/draft/pragma.html
One use-case is to set the cipher_compatibility PRAGMA which is supported by the SQLCipher extension:
https://www.zetetic.net/sqlcipher/sqlcipher-api/#cipher_compatibility
Fixes #227

I swear I'll get to this soon. I've got another bit of work for ecto_sqlite3 I need to knock out and then will refocus on this today.


weird out of memory error on GPU/slowdown on CPU

I got back to benchmarking various DL tools on the airline dataset https://github.com/szilard/benchm-ml
For this use case I use fully connected networks, though I cannot reach the accuracy of GBMs, even after extensive tuning https://github.com/szilard/benchm-ml#deep-neural-networks
I've experimented with this before and that's when I suggested mxnet to implement sparse features https://github.com/dmlc/mxnet/issues/773
I know this is still in the works https://github.com/dmlc/mxnet/issues/1524
However, now with the larger EC2 GPU instances I can try to run the larger dataset (10M rows) with the dense representation (now it will fit in RAM with 1-hot encoding, which was not the case before).
For comparison, the 1M dataset is fast: 35 sec on p2.xlarge (1 GPU) and 70 sec on r3.8xlarge (CPU 32 cores).
On the 10M dataset, however:
on p2.8xlarge (using 1 GPU only):
[06:17:48] /home/ubuntu/mxnet/dmlc-core/include/dmlc/./logging.h:235: [06:17:48] src/storage/./pooled_storage_manager.h:79: cudaMalloc failed: out of memory
[06:17:48] /home/ubuntu/mxnet/dmlc-core/include/dmlc/logging.h:235: [06:17:48] src/engine/./threaded_engine.h:306: [06:17:48] src/storage/./pooled_storage_manager.h:79: cudaMalloc failed: out of memory
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
terminate called after throwing an instance of 'dmlc::Error'
  what():  [06:17:48] src/engine/./threaded_engine.h:306: [06:17:48] src/storage/./pooled_storage_manager.h:79: cudaMalloc failed: out of memory
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
Aborted (core dumped)

while on r3.8xlarge (CPU 32 cores) it runs for about 1 hour, i.e. 100x more time then on 1M dataset.
How can this be?
For GPU I would expect same memory usage since I'm doing mini-batches (of 128).
For CPU I would expect 10x runtime, not 100x.
Any ideas?
Code here: https://github.com/szilard/benchm-ml/blob/master/4-DL/2-mxnet.R

try USE_MKL2017 and USE_MKL_EXPERIMENTAL in config.mk for cpu performance

@piiswrong That would require MKL. But I doubt the problem is with MKL. There are 2 problems above:

GPU out-of-memory (while mini-batches are of same size, so why?)

CPU 100x slower on 10x data. Since last post I tried out keras with Tensorflow backend it's 10x train time on 10x data (not 100x).

Any ideas why 1 and 2 above?

USE_MKL2017 = 1
USE_MKL2017_EXPERIMENTAL = 1

try this

OK, would that help w the GPUs or just CPU?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


go-difflib (external module) "NO LONGER MAINTAINED"

https://github.com/pmezard/go-difflib (a partial port of Python difflib) has been updated to contain "IS NO LONGER MAINTAINED." It's docs are at: https://pkg.go.dev/github.com/pmezard/go-difflib/difflib?tab=doc

Hi @wilson-mar thanks for reporting. This is an indirect dependency via https://github.com/stretchr/testify. If this is still a concern to you, I'd raise an issue on that repository. Thanks for using DAYTONA!


VisualStudio Designer

It seems, that the "DataType switch" crashed VisualStudioDesigner. Does somebody know why? I tried to fix the MainWindowView for a long time, but i found no solution for this issue.

see #70


Allow for complete style customization

Move as much styling out of the Tailwind plugin back into the individual component file and replace twmerge with clsx to clean up the styling logic.

Convert all extended themes in tw plugin to CSS variables so that developers can quickly override styles globally.

Move as much styling out of the Tailwind plugin back into the individual component file and replace twmerge with clsx to clean up the styling logic.

This is not a viable option for this package. As originally assumed clsx is just jamming all classes into the HTML element leaving the browser to choose which class to apply at runtime which does not allow for a consistent and dependable experience for the developer. This package will continue to utilize twmerge, however, I will be doing some cleanup for readability using some of the techniques that clsx uses.
Once twmerge usage has been cleaned up and is more readable I will move into cleaning up theme names and making them globally customizable instead of just at the component level.

clsx has been removed from the entire package, committed, and pending merge into the main branch.
Next step is to stop having different styling for dark/light (ie. light-border vs dark-border). Instead utilize CSS variables that shift depending on the theme selected. (ie. --light-border='#####' and --dark-border='#####') then have a single tailwind class for border that switches the CSS variable based on the theme selected. This way users can just change the --light-border and it will update for the entire application.

Global style changes will be a larger issue that needs to be resolved and will not be addressed in this issue.


Merge develop+upstream to develop

Merging develop+upstream to develop in both omaha-build and omaha to get latest changes from upstream Omaha master; this is the omaha part.
Probably not much to review here; it's all from upstream. Just a sanity check

I've run unit tests in both dcp/develop+upstream and dcp/develop branches.
dcp/develop+upstream:
  [  FAILED  ] 13 tests, listed below:
  [  FAILED  ] WorkerWithTwoAppsTest.CheckForUpdateAsync_Large
  [  FAILED  ] WorkerWithTwoAppsTest.DownloadAsyncThenDownloadAndInstallAsync_Large
  [  FAILED  ] NetworkRequestTest.MultipleRequests
  [  FAILED  ] GoogleUpdateRecoveryTest.FixGoogleUpdate_UseRealHttpClient
  [  FAILED  ] GoogleUpdateRecoveryTest.ProductionServerResponseTest
  [  FAILED  ] SetupFilesMachineTest.Install_NotOverInstall
  [  FAILED  ] SetupFilesUserTest.Install_NotOverInstall
  [  FAILED  ] UnitTestHelpersTest.ClearGroupPolicies
  [  FAILED  ] IsForeground/WebServicesClientTest.SendUsingCup/0, where GetParam() = false
  [  FAILED  ] IsForeground/WebServicesClientTest.SendUsingCup/1, where GetParam() = true
  [  FAILED  ] IsMachine/InstallManagerTest.InstallDir_ReadOnlyFiles/1, where GetParam() = true
  [  FAILED  ] IsDomain/GoogleUpdateRecoveryRegistryProtectedTest.FixGoogleUpdate_EmptyAppInfo/0, where GetParam() = false
  [  FAILED  ] IsDomain/GoogleUpdateRecoveryRegistryProtectedTest.FixGoogleUpdate_EmptyAppInfo/1, where GetParam() = true

dcp/develop:
[  FAILED  ] 13 tests, listed below:
[  FAILED  ] SchedulerTest.UsesDebugTimer
[  FAILED  ] WorkerWithTwoAppsTest.CheckForUpdateAsync_Large
[  FAILED  ] WorkerWithTwoAppsTest.DownloadAsyncThenDownloadAndInstallAsync_Large
[  FAILED  ] NetworkRequestTest.MultipleRequests
[  FAILED  ] GoogleUpdateRecoveryTest.FixGoogleUpdate_UseRealHttpClient
[  FAILED  ] GoogleUpdateRecoveryTest.VerifyFileSignature_SignedValid
[  FAILED  ] GoogleUpdateRecoveryTest.ProductionServerResponseTest
[  FAILED  ] SetupFilesMachineTest.Install_NotOverInstall
[  FAILED  ] SetupFilesUserTest.Install_NotOverInstall
[  FAILED  ] IsForeground/WebServicesClientTest.SendUsingCup/0, where GetParam() = false
[  FAILED  ] IsForeground/WebServicesClientTest.SendUsingCup/1, where GetParam() = true
[  FAILED  ] IsDomain/GoogleUpdateRecoveryRegistryProtectedTest.FixGoogleUpdate_EmptyAppInfo/0, where GetParam() = false
[  FAILED  ] IsDomain/GoogleUpdateRecoveryRegistryProtectedTest.FixGoogleUpdate_EmptyAppInfo/1, where GetParam() = true

Holding off on this for now.


clean up tdt warnings and badge cleanup

This change should reduce the warning output of our current testsuite drastically.

Ready from my side.


Update ruletypes.rst for googlechat_proxy option

update googlechat_proxy option
Description

Checklist

[x ] I have reviewed the contributing guidelines.
[ ] I have included unit tests for my changes or additions.
[x ] I have successfully run make test-docker with my changes.
[x ] I have manually tested all relevant modes of the change in this PR.
[ ] I have updated the documentation.
[ ] I have updated the changelog.

Questions or Comments

Closing - See #913


Issue 14788 - Incorrect rejection of inout function call

https://issues.dlang.org/show_bug.cgi?id=14788

Rebased on master, and ready for ddmd.

Looks good.

Auto-merge toggled on


Create usable inferred browser widget from legacy bbop-js

This ticket is to encapsulate the revival of the legacy bbop-js browse widget into the new bbop-widget-set npm framework, and have an example of that at the traditional /amigo/browse.
This will be of some interest for evidenceontology/evidenceontology#95 .

I'm most of the way through this, but I think I'll need to thread through some of the old bracket graph code into bbop-graph, or somewhere else.

The widget has been returned to functionality, as well as slightly better integration in the amigo setup. That said, it's still the same ugly thing as before, so comments and code are welcome.

When I initially attempted the browse widget, I was using the bbop.js CDN. Now that the browse widget has been restored, is it included in the CDN?

The CDN has been deprecated. We are now trying to distribute all of the widgets, etc., via NPM.
The packages will be amigo2-instance-data and bbop-widget-set, I've just made another release that contains the updates. The API is pretty much the same, but the location different: bbop-widget-set.browse.
Please make note of #321 .
Compiled example: http://tomodachi.berkeleybop.org/amigo/browse
https://github.com/geneontology/amigo/blob/master/javascript/web/Browse.js

I do not have the documents automatically generated yet, I will take a look at that.

I didn't realize that about the CDN. Thanks! I'll checkout those links.

Sorry, it's been a crazy month; there will be a lag as process and docs catch up to the code.
You might also want to note that the two packages I mentioned are now based out of this repo, in javascript/npm/, even though they are independent packages in NPM. Once #321 is cleared to satisfaction, they may (we'll see) be moved out.

That's good to know. Thank you.

Would it be possible to get a specific data structure (JSON) input example for this widget? Thank you!

Currently, we have tree browsers based off of berkeleybop/bbop-graph at:
http://tomodachi.berkeleybop.org/amigo/dd_browse
http://tomodachi.berkeleybop.org/amigo/browse
The first of these is not a "widget" yet, although it could be made so, but is rather "apps" within AmiGO that uses the available graph structures to create itself using the jstree package.
The latter now uses the bbop-widget-set package, available on NPM separately, housed as a sub-package in the amigo repo: https://github.com/geneontology/amigo/tree/master/javascript/npm/bbop-widget-set , https://github.com/geneontology/amigo/blob/master/javascript/npm/bbop-widget-set/lib/browse.js.
The topology and transitivity graphs need to have their documentation updated, but are currently explained here: https://github.com/berkeleybop/bbop-js/wiki/Graph
The real-world use is in https://github.com/geneontology/amigo/blob/master/javascript/web/Browse.js#L101, but if you are trying to have your own data backend (instead of using GO data) you'd probably have better luck with looking at the browse.js code: https://github.com/geneontology/amigo/blob/master/javascript/npm/bbop-widget-set/lib/browse.js#L129


Compilatation error on vs2015

When compiling the development branch using the project in build\vc2015 and VS2015 Enterprise, the compiler gives the following error:
c:\users\waty\source\repos\nana\source\gui\element.cpp(267): error C2662: 'nana::rectangle &nana::rectangle::pare_off(int)': cannot convert 'this' pointer from 'const nana::rectangle' to 'nana::rectangle &'

The error disapears when you change
graph.rectangle(::nana::rectangle{ r }.pare_off(1), false, bgcolor);

to
graph.rectangle(::nana::rectangle(r).pare_off(1), false, bgcolor);

Thank you for reporting this issue. I tried it using VS2015 Community RC(14.0.22823), but the issue wasn't reproduced.

Hi,
I'm testing now with VS2015 Community RTM (Version 14.0.23107) and I confirm the problem. I "fixed" it taking a copy of the rectangle and not a reference, becouse we make a copy anyway.
	class border_depressed
		: public border_interface
	{
		bool draw(graph_reference graph, const ::nana::color& bgcolor, const ::nana::color& fgcolor, ::nana::rectangle r, element_state estate, unsigned weight)
		{
			graph.rectangle(r, false, static_cast<color_rgb>((element_state::focus_hovered == estate || element_state::focus_normal == estate) ? 0x0595E2 : 0x999A9E));
			graph.rectangle(r.pare_off(1), false, bgcolor);
			return true;
		}
	};

A fix has been checked into the develop branch.


Create library of JSON stringified 3D car models.

BMW 328 is finished

Added Helmet, Fedora, iPhone, Lampost, Teapot and Dino.

Added 10 JSON models from Princeton.


fix: latex and questions

Summary by Sourcery
Fix LaTeX formatting in the JSON data file by adjusting the formula representation to remove incorrect chemical environment commands.
Bug Fixes:

Correct the LaTeX formatting in the JSON data file by removing unnecessary '\ce{}' around variables in the formula.

@kjappelbaum should I keep this PR open and fix the other issues? or maybe just do them all here?

@sreekanth221998 @MrtinoRG did you actually check or just approved? it is important this render right

I check it but only the text. I mean I do not know if I can render it locally and how to do it :)


Mount all of the repo into the dev-frontend service

In order to get the git information from the frontend code (e.g. run
git commands in the _frontend.codex directory) the parent project
must be accessible. This is only a problem because the _frontend.*
repos are submodules; and thus not actually git repos
themselves (i.e. it's complicated). This solves fatal: No names found, cannot describe anything. when running git commands within the
submodule.
This change is being made, because additions are being made to the
frontend that require git info. The frontend build code discovers its
own version information for display in the server-status page. This
fix is required to enable that feature.
No developer changes required.

/rebase


The checked item always is 'Disable Soft Delete' on 'Configure Soft Delete Policy' dialog

Storage Explorer Version: 1.5.0
Platform/OS Version: Windows 10/ MacOS High Sierra/ Linux Ubuntu 16.04
Architecture: ia32
Build Number: 20181109.1
Commit: e721f1fb
Regression From: Previous release 1.5.0
Steps to Reproduce:

Expand one storage account -> Open the 'Configure Soft Delete Policy' dialog(The default checked item is 'Disable Soft Delete').
Try to change the default soft delete policy to 'Enable Soft Delete' -> Reopen the 'Configure Soft Delete Policy' dialog then check the result.

Expected Experience:
The checked item is 'Enabled Soft Delete' on 'Configure Soft Delete Policy' dialog.
Actual Experience:
The checked item always is 'Disable Soft Delete' on 'Configure Soft Delete Policy' dialog.

also, after the policy is enabled and you go back to the dialog, the dialog still says it's disabled.

Fix merged into hotfix branch


Dedup chunk lookup table on shard write

Fix https://github.com/xetdata/xethub/issues/3487

Hmm.  This would fix the acute problem, but the issue is that true random collisions are not deduplicated against.  I think ideally we should limit this number to the max consecutive chunks here on write.  Or simply make that not an error if more than 8 are present on lookup -- i.e. only a subset are returned.


Update call-park-and-retrieve.md

@CarolynRowe
Feel free to refine my wording. Goal here is to clarify how users are grouped to the same call park group as it's not clear here that it's dependent on the call park policy assignment.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Enhancement: Waila Support

It would be really cool if, when looking at a filter the HWYLA HUD showed what was in the filter and what advanced filtering flags are set to non-default positions.

Not ignoring this! Will look into this probably this weekend.


Move cache initialisation under supervision trees

This PR moves the caching logic into logically independent modules, and define supervision tree to start the caches and do initialisation at the same time.
The main benefit is defining a cache and warming it up is now in one place instead of spread across the application.ex and startup.ex files.
One thing to note: some caches are used together. For example config_site_type_store and :config_site_cache are both used by add_site_config_type. In this case, the restart strategy should be :one_for_all so that if a cache dies, the other is also terminated, and then both are restarted before initialisation.
This should be reviewed commit by commit where I tried to only move small bits at a time.
Ultimately I want to straighten up the startup logic which has a bunch of init code outside supervision trees, but that'll come later.
How to test that?
There shouldn't be any functional changes, so:

starting up
login as admin and doing a quick permission check (can you access protected parts?)
start up a lobby and issue a command like $help or $whois

This is a really sensible idea; I'll be implementing it in my other projects. Thank you :)

Works on integration server.


[BUG]hotkey and Back to the original look

You should set a hotkey to disable this feature.
And I think it's silly to force chest armor on for players who aren't wearing it, it causes unnecessary trouble, for example a player wants to show his original skin, but the mod will keep trying to put on the clothes in his backpack for him.
As well, if I use aerial switch sheath wings without armor, when I land I should go back to not wearing armor instead of picking a piece of clothing I have stored in my backpack and putting it on.

ok so i added keybind. But i don't wand add "back to original look" feature beause there will be many different cases like what to do when player joined game with elytra, what mod should place into chestplate slot. Or what i should mod do if item disappeared. Should it wear best possible armor or not wear it at all. How we can determine best possible armor? There is too many cases when this feature may be broken while i'm trying to keep this mod as simple as possible

You make a good point.


Fix failing integ tests due to requests/urllib3 upgrade

We need to ignore warnings about missing SAN's in SSL certs
in py26.  This is due to a python bug and we have to validate
using the CN.
These actually were caught by AWS CLI integ tests.  Given they're
warnings, they don't actually cause anything in botocore to hard
fail, whereas the CLI is very sensitive to what shows up in stdout
and stderr which explains why the CLI tests are more likely to
catch this.
I've also updated our integ smoke tests in botocore.  We now will
also verify that the smoke tests generate no warnings.  This will
catch any regressions against this issue.
cc @kyleknap @danielgtaylor

Coverage increased (+0.01%) to 94.23% when pulling 5d24c688cdfb927bee858f5a8b9820bf65b10c02 on jamesls:filter-ssl-san-py26 into 8fe6cb5a04b0bbad22aa0a140a7ece93cce8dd9e on boto:develop.

Coverage increased (+0.01%) to 94.23% when pulling 5d24c688cdfb927bee858f5a8b9820bf65b10c02 on jamesls:filter-ssl-san-py26 into 8fe6cb5a04b0bbad22aa0a140a7ece93cce8dd9e on boto:develop.

Looks good. :ship:


Make value in ValueAccessor public

This is usecase for us where we need to read the arguments as value without concerning about it's type. Eg
Current Implementation
fn get_arg<'a>(ctx: ResolverContext<'a>, name: String) -> &'a Value {
  let arg = ctx.args.get(name);

  // use a if-else ladder to check and convert from ValueAccessor to Value
  if let Ok(str) = arg..string() {
    Value::from(str).unwrap();
  } else if {
    // .. add more cases
  }
}

Proposed Implementation
fn get_arg<'a>(ctx: ResolverContext<'a>, name: String) -> &'a Value {
  ctx.args.get(name).0
}

This would be significantly faster for our usecase.

Add a method called into_inner，what do you think?😄

Sure! Happy to make that change.
What about as_value or something else with an as prefix?
I feel any to or into convey some sort of transformation, which generally is compute intensive. However, as is a conversion that in a way is coercing as type into something, without any overhead. Let me know what you think, can go with either.

Done!

When will it be released?

Waiting #1354


Relocate orquesta logic from notifier to the workflow engine

Implement queue consumer that can handle variable message types. Relocate handling of action execution completion to workflow engine. Add policy application to the workflow engine for workflow tasks.

Added a comment on aliased module import and variable naming consistency, besides that, LGTM :+1:


[CELEBORN-891] Remove pipeline feature for sort based writer

What changes were proposed in this pull request?
Remove pipeline feature for sort based writer
Why are the changes needed?
The pipeline feature is added as part of CELEBORN-295, for performance. Eventually, an unresolvable issue that would crash the JVM was identified in https://github.com/apache/incubator-celeborn/pull/1807, and after discussion, we decided to delete this feature.
Does this PR introduce any user-facing change?
No, the pipeline feature is disabled by default, there are no changes to users who use the default settings.
How was this patch tested?
Pass GA.

As this feature is broken, I cherry-picked it 0.4 to avoid confusing users.


[fast-client] Long tail retry budget for fast client

[fast-client] Long tail retry budget for fast client

Introduced RetryManager to measure and calculate retry budget based on a percentage of requests.

The RetryManager is only applied to long tail retries and currently hard coded to be 10% of the total original request rate. This means if there were any exception that's not a 429 the retry will be triggered without going through the RetryManager. If the retry budget is exhausted then the retry task will do nothing and the request will either complete eventually (original future) or time out.

How was this PR tested?
Existing and new tests
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Will add the following metrics to track retry budget behavior:

The long tail retry budget value that's currently being enforced
Token bucket stats for the long tail retry budget to monitor usage

Will also add more unit tests in the RetriableAvroGenericStoreClient layer and have at least one integration test with a much lower enforcement window for a smoke test (since long tail retry trigger is not deterministic in integration test).

Will add the following metrics to track retry budget behavior:

The long tail retry budget value that's currently being enforced
Token bucket stats for the long tail retry budget to monitor usage

Done


Attempt to modularise components and minimise dependencies.

Start by adding the cake pattern to the startup process.
This facilitates mix, match and replace  of components at startup.

wrong target branch?


Grid Ajax call not compatible with jQuery 3.1.1

As of jQuery 3.0 the .success and .error functions are deprecated and this code in mvc-grid.js at line 170 uses them:
            if (grid.sourceUrl) {                 $.ajax({                     cache: false,                     url: grid.sourceUrl + '?' + grid.query                 }).success(function (result) {                     grid.element.hide();                     grid.element.after(result);
Any plans to upgrade support to jQuery 3.x?  I made modifications to fix in my project to get by this issue.
Ed

Yes, they will be replace. It's strange that I didn't caught them on my initial check on compatibility with jQuery 3.X.


Deselection fails on vectorTileLayer when two features share the same id

When I use an ol.interaction.Select on a  ol.layer.VectorTile, and have a feature that is split up between two tiles, and then select both of these feature segments, and clear (parts of) my selection containing this feature, the selection tool fails with the following error:
Uncaught AssertionError: Assertion failed: featureKey exists in featureChangeKeys

The stack trace is:
goog.debug.Error (.point…bundle.js:2881)
goog.asserts.AssertionError (.point…bundle.js:4580)
goog.asserts.doAssertFailure_ (.point…bundle.js:4637)
goog.asserts.assert (.point…bundle.js:4667)
ol.source.Vector.removeFeatureInternal (.point…undle.js:72610)
ol.source.Vector.removeFeature (.point…undle.js:72598)
(anonymous function) (.point…undle.js:72161)
goog.events.EventTarget.fireListeners (.point…undle.js:12722)
goog.events.EventTarget.dispatchEventInternal_ (.point…undle.js:12819)
goog.events.EventTarget.dispatchEvent (.point…undle.js:12634)
ol.Collection.removeAt (.point…undle.js:24129)
ol.Collection.pop (…undle.js:24083)
ol.Collection.clear (.point…undle.js:23992)
ol.interaction.Select.handleEvent (.point…ndle.js:113046)
ol.Map.handleMapBrowserEvent (.point…undle.js:83997)
goog.events.EventTarget.fireListeners (.point…undle.js:12722)
goog.events.EventTarget.dispatchEventInternal_ (.point…undle.js:12819)
goog.events.EventTarget.dispatchEvent (.point…undle.js:12634)
(anonymous function) (.point…undle.js:49610)

Is splitting up features across tile layers, and leaving them with the same id, wrong?
Is ol.interaction.Select supported for ol.layer.VectorTile?
Or: is this an edge case problem for ol.interaction.Select?

Is splitting up features across tile layers, and leaving them with the same id, wrong?

It is not wrong, but it prevents you from using ol.interaction.Select.

Is ol.interaction.Select supported for ol.layer.VectorTile?

Yes, if you use ol.Feature (and not ol.RenderFeature) features, and if you do not have the same feature ids in different tiles.

Or: is this an edge case problem for ol.interaction.Select?

Yes. The primary purpose of vector tiles is rendering. You can use map.forEachFeatureAtPixel to get interactivity.

Would it be possible to modify ol.interaction.Select in such a way that this would become possible? And would a PR for that have any chance of being merged?

Can't you use ol.Map#forEachFeatureAtPixel instead? Remember:

The primary purpose of vector tiles is rendering.

What you could do is something like
var selectedId;
vectorTileLayer.setStyle(function(feature, resolution) {
  return feature.getId() == selectedId ? selectedStyle : defaultStyle;
});
map.on('singleclick', function(e) {
  var feature = map.forEachFeatureAtPixel(e.pixel, function(feature) {
    return feature;
  });
  selectedId = feature ? feature.getId() : undefined;
});

That seems indeed a better option than what I was thinking of. I will try that approach, thanks!

Can we close this @JWGmeligMeyling?

I was trying to resolve this issue. I do however still have no clue why the feature is removed twice internally. Seems to me that the id's are unique and that the code should just work, even with the same feature on two tiles. I ended up with disabling the assertion and now it works just fine... But there should be a cleaner work around. I was however unable to make sense of the stack and see where the removals came from, the structure with the collection in a source triggering an event that triggers a remove in another thing was not completely transparent to me. If you could give me a pointer I am defenitely willing to look further but yes we can close this issue 😄

See http://jsfiddle.net/8a4wqtjc/2/ @JWGmeligMeyling, which implements my suggestion from above. You can click on a building there to select it. If you click one that spans multiple tiles, all parts will be selected because they share the same id.

@ahocevar: I think I am running to the same issue here, but could be wrong.  I am loading a point layer using tile loading strategy.
Here's my code snippet:
selectInteraction = new ol.interaction.Select({
     condition: ol.events.condition.click
});
selectInteraction.on('select', function(e) {
     console.log( e.target.getLayer());
});
map.addInteraction(vm.selectInteraction);

This Throws an error 42:
Expected an ol.Feature, but got an ol.RenderFeature.
I understand your explanation above of why this happens, but it still feels like a bug in the API because users would naturally expect this to work.  It does work fine if I use click event on map, and search it by the pixel, but then I loose the convenience of an select interactions.
Any suggestions?

This is actually a different issue @shaunakv1. e.target.getLayer() in your application is incorrect. The getLayer() method needs to be called with a feature as argument. The assertion is misleading, and that is a bug, which I just ticketed as #5958.


Make ember-i18n translations work

Make translation keys like 'employees.name.last' work when side loaded from a flat structured JSON as you have them probably from ember-i18n

I'm not entirely following the use-case. Also, this would require a test case.

+1 for the usecase!

Re: #80

A lot of i18n apis use keys with keypaths instead of nested objects (including ember-i18n). It'd be nice if they were supported here too.

All of my translations are structure this way right now. I know I'll be override getLocalizedPath once it's released, but it'd nice to have it supported by default.

I'm honestly confused as to what isn't working. As you can see there is a test case for path lookups: https://github.com/dockyard/ember-cli-i18n/blob/master/tests/unit/utils/t-test.js#L119-L123 am I missing something here?

locales/en.js
export default {
  'foo.bar.baz': 'Foo Bar Baz'
};

templates/foo.hbs
{{t 'foo.bar.baz'}} <!-- not found -->

@thejameskyle I see, that style isn't supported because we are using Ember.get to retrieve the path from the object. get will split paths on '.' and that would expect an object of:
{
  foo: {
    bar: {
      baz: 'Foo Bar Baz'
    }
  }
}

two possible solutions:

you can override the custom handler as described here: https://github.com/dockyard/ember-cli-i18n#overriding-the-locale-lookup-handler and provide this functionality
you can expand the flat structure from into the nested format

I'm fine with that, but you still might want to include it as the flat structure is far more common in my experience and many people already have translations for their apps.

@thejameskyle what is outputting the flat structure?

Transifex http://docs.transifex.com/developer/formats/json

@bcardarella Bigger companies usually have tools/build tasks and processes to get content to manual translators who work based of a property file where the content is usually in a flat structure such as ,
foo.bar.baz=Foo Bar Baz
foo.bar.goo=Foo Bar Goo

Now one of the reasons I opened #39 was so that its easier to stick to the existing processes that organizations use and have the backend app serve the locale content through an endpoint in json format. May be whatever solution you come up with for #39 might end up solving this as well.

@sivakumar-kailasam You could use this if you wanted ember-cli-i18n-csv. It flattens the JSON files into a single CSV to send out, then re-nests them into JSON when you're done.

I think this sort of use case, while completely valid, is best handled outside of this library. I don't want to go down the path of chasing too many conditional paths as that will just end up being messy.
It would best best if someone created a higher-level addon, something like ember-cli-i18n-flat (crappy name, I know) that depended upon this library and overrode the handler.

@kellyselden awesome job on that one, my present plan is to convert between property to CSV to js n the reverse. Just wanted to cut down on the conversions by lazy loading as json.


Why is only the 👪  emoji not displayed?

Strangely, it's not a color icon even though it's included in JSON data.
😭 Please add 👪 emoji !!

This was an intentional change in the Unicode Emoji recommendations.
More explanation at https://jenniferdaniel.substack.com/p/breaking-the-cycle#§what-is-a-family
This has been adopted by other vendors. See https://emojipedia.org/family-adult-adult-child#designs for instance.

@jbicha Thank you reply, I understand the reason 😢

If this issue is resolved for you, you can close it.
(I am just a community member and don't have access to close other people's issues.)


Does not work button missing iOS 15.7.7 + uYou + palera1n

Hi has this tweak been abandoned as far as updates for iOS 15 and 16?

Nah, everything works fine.


Need visible, clickable Close All or Close Group action

Not sure if we already had an issue for this, but I'm going crazy without this action. Right click on the title simply doesn't work for me.

@joaomoreno what about we leverage the new support to change an action when you press the Alt key and if you hover over the X for closing you get the close all?

@joaomoreno do you have tabs on or disabled?  Also, have you tried using Ctrl + K + W or using the overflow menu to click "Close All?"
I'm worried that if we add a visible Close All command that it could be accidentally clicked and wipe out all the open files users have open.  That's why it's tucked in the overflow menu currently.

@bgashler1 Yes to all. I currently rebind Cmd W to close all editors in a group (model concept) because that seems to be the only way I can close a pane (view concept). In my mind there are editor panes and I can open files in them, the collection of which is kept nice and tidy by Quick Open. In our current model there are editor groups, each editor is a file. A Close X-button action makes sense in that model, where you see tabs and clicking it will close each tab. In my workflow the X is completely useless, there's not a single time I click it since I'm not managing open editors, I'm managing editor panes. So... I would like the equivalent of a Close Pane action.
@bpasero Alt is a good idea; but if I'm reaching for the keyboard, I'll just do Cmd W as I'm doing today.

I think this will be handled as soon as it is possible to configure the toolbar buttons of the editor (something Joh is working on).
I feel not good to just replace the X with the close all because that will piss off people that expect this to close one editor.
I feel also bad to just add another action because that space is limited.
After all, I feel like adding a setting for this would make most sense.
Thoughts?

👍 on a setting

Just to confirm, when tabs are disabled, the setting would determine whether or not clicking X would close the top most editor in the group or the complete group? If the default is that it continues to close the top most editor, this sounds like a reasonable approach.

I worry that introducing this setting is confusing for when tabs are enabled because then imho it does not make any much sense...

After talking to @joaomoreno he thinks that https://github.com/Microsoft/vscode/issues/9904 would help him a lot. So I suggest we close this as duplicate of https://github.com/Microsoft/vscode/issues/7046 and wait for more user feedback.

OK. I was going to suggest something like:
//This setting controls how the close button works when tabs are disabled
"workbench.editor.groupCloseButton" : "closeAllWhenTabsDisabled"

@stevencl that is great, I will add this to the other issue as suggestion!


Fix bug where rasterize() hangs on Apple silicon

Hi,
Thanks for writing fauxgl, I've found it very useful. I ran into a problem when running my project on an Apple silicon laptop. Please consider this patch which fixes the problem.
If any edges of the triangle are horizontal or vertical, it's possible
for the raXX variables, and thus d, to get a division by zero leading
to +Inf. On Intel, when +Inf is converted to int and back with
d = float64(int(d))
the result is a very large negative number which the existing check
converts to zero. On ARM (or at least on Apple Silicon which is where I
ran into the problem) the result is a very large positive number,
causing the function to hang.
You can verify this with the following small fragment of code:
var one float64 = 1
var zero float64 = 0
var divZero float64 = one / zero
var castAndCastBack = float64(int(divZero))
fmt.Printf("%f\n", castAndCastBack)
On an Intel CPU that will print a very large negative number, on Apple silicon a very large positive number.

@djn24 Thanks for this one. Switched from Windows 10 to an Apple Silicon MacOS and my code ran for some models infinite. Would be great to see this merged. @fogleman

I think I'm facing the same issue in raspberry pies...
Some models render ok, others just don't finish...


Password special characters

This app will not work if Garmin password contains special characters.
I changed my Garmin password from a word to a more secure one containing %/? (special characters) & withings-sync failed to login to garmin. Eventually I changed y garmin password to one containing only letters & numbers & all works OK again.

This is weird. My password contains a # and it works well.

OK I will experiment more tomorrow evening.

My password contains a # and is working fine.
Just a guess: For ? % * etc maybe it's help to to escape them to ? % *

I also can not reproduce this issue. Its likely caused by some escaping issues in your env vars.
See also: #17


[Feature][Sqlserver-CDC] Add multi-table e2e testcase

Search before asking

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

Description
[Sqlserver-CDC] Add multi-table read e2e testcase
add testcae into
https://github.com/apache/seatunnel/tree/dev/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-cdc-sqlserver-e2e
reference
https://github.com/apache/seatunnel/blob/dev/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-cdc-mysql-e2e/src/test/java/org/apache/seatunnel/connectors/seatunnel/cdc/mysql/MysqlCDCIT.java#L177
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@hailin0  hi, i want to do this


Backend refactoring for faster event loading and planned features

WIP
I'm planning to refactor the backend again, mostly to make retrieving of events faster, especially when querying for all events in a longer time span.
Planned step

Remove CalendarCollection and only query the db once

I had some use cases like enabling the use of different database files for different calendars in mind, but I guess that is pretty useless.
This should reduce database queries by a factor of N (configured calendars).
Optionaly
Problems
With the current database layouts we have two problems for planned features:
a) when getting all future events (see #231) we need to query the floating and localized events table.
Since we probably don't want all events at once, we'll only get say 100 events from each table, merge those lists, sort them and present to the user. We could than do the same for the second 100 events in each table and so on.
But the Nth 100 events in one table could all be before the Mth 100 events in the other table.
The naive implementation could lead to events not being properly sorted.
This can of course be circumvented by checking the start dates of all events etc.
b) Currently coloring dates is very slow
Step one above will speed this up somewhat, but probably not enough.
It would therefore be nice to have a table with what dates to color how, not getting the events from the database should speed things up.
Problem as above.
Proposed solution
Saving floating events in localized time (as configured by the user as local_timezone)
This means the floating time table needs to be rebuild when changing local_timezone (or we need to keep one table for each local timezone ever used).
The benefit will be that we can JOIN floating and localized tables when querying for events which will lead to properly sorted events.
Create a table with date-colors.
When not using all configured calendars, a table with the date colours would not be accurate though.
Check how other programs are doing this.

First part is more or less done (33c5373244138e94cb7a6d9b12c5df5fd4523838), while Calender and CalendarCollection still exist, the speedup is already there.

Original text:
WIP
I'm planning to refactor the backend again, mostly to make retrieving of events faster, especially when querying for all events in a longer time span.
Planned step

Remove CalendarCollection and only query the db once

I had some use cases like enabling the use of different database files for different calendars in mind, but I guess that is pretty useless.
This should reduce database queries by a factor of N (configured calendars).
Optionaly
Problems
With the current database layouts we have two problems for planned features:
a) when getting all future events (see #231) we need to query the floating and localized events table.
Since we probably don't want all events at once, we'll only get say 100 events from each table, merge those lists, sort them and present to the user. We could than do the same for the second 100 events in each table and so on.
But the Nth 100 events in one table could all be before the Mth 100 events in the other table.
The naive implementation could lead to events not being properly sorted.
This can of course be circumvented by checking the start dates of all events etc.
b) Currently coloring dates is very slow
Step one above will speed this up somewhat, but probably not enough.
It would therefore be nice to have a table with what dates to color how, not getting the events from the database should speed things up.
Problem as above.
Proposed solution
Saving floating events in localized time (as configured by the user as local_timezone)
This means the floating time table needs to be rebuild when changing local_timezone (or we need to keep one table for each local timezone ever used).
The benefit will be that we can JOIN floating and localized tables when querying for events which will lead to properly sorted events.
Create a table with date-colors.
When not using all configured calendars, a table with the date colours would not be accurate though.
Check how other programs are doing this.

multiprocessing is always a fun option for this...

I think in another issue somebody warned against using fork with sqlite, multiprocessing uses fork. Also vdirsyncer used multiprocessing for a very basic purpose, but dropped it because there are so many quirks.

if I understood @rogerbinns correctly, you must not pass around the connection to the database to other threads/processes, but we could always open another connection there.

Okay.   Also make sure the connection is proactively closed - do not rely on garbage collection.  The problem with garbage collection is destructors can run on any thread, and gc collection of the connection can/will result in SQLite doing work.
@untitaker whatever your opinion of multiprocessing, all it takes is any code in the app, library, library used by another library etc to use it.  And if they do that, extremely hard to debug/reproduce database corruption happens.
I also recommend reading about how to corrupt a SQLite database - https://www.sqlite.org/howtocorrupt.html


What mode does PRELU support?

There are three modes for the PRELU activation, i.e., channel-wise mode (an alpha value per channel), element-wise (an alpha value per element) mode and share-channel mode (all elements share the same alpha value). I am not sure what mode does PRELU support. If it supports more than one mode, how to configure?

@mxnet-label-bot add [Question]


antctl get networkpolicy should have consistent namespace and name fields

Describe the bug
In controller mode, antctl get networkpolicy -o table will return name but not namespace
# antctl get networkpolicy -o table
NAME             APPLIED-TO                           RULES
default-deny-all ca783db2-d6bd-521c-aafe-967d999193e7 2
default-deny-all b5a79463-975e-5784-8e6f-b4807f42f9e9 2
default-deny-all 9157abce-3c51-5285-803b-52543b194703 2

which is confusing.
In agent mode, it will return uuid as name, which makes it not straightforward to associate the result with the policy got from antrea-controller and the one in K8s API.
# antctl get networkpolicy -o table |head
NAME                                 APPLIED-TO                           RULES
0103cb1c-3891-4829-922b-86fae8759084 4fd6be1a-ec03-5b8f-9e44-7f1a4af4c117 2
03545126-bc78-42f9-bc18-2db958f47727 0473be26-4942-5647-827f-c891578402f2 2
0490f3b7-02d7-4d84-b65a-bf37777df616 90fe4568-12ce-5907-8ae0-97d7a2871274 2

To Reproduce
Via the above commands.
Expected
Controller and agent mode should get same style result and have namespace and name.
# antctl get networkpolicy -o table
NAMESPACE NAME             APPLIED-TO                           RULES
ns-1      default-deny-all ca783db2-d6bd-521c-aafe-967d999193e7 2
ns-2      default-deny-all b5a79463-975e-5784-8e6f-b4807f42f9e9 2
ns-3      default-deny-all 9157abce-3c51-5285-803b-52543b194703 2

Actual behavior
They are inconsistent and missing namespace fields.
Versions:
Please provide the following information:

Antrea version (Docker image tag). 0.5.0
Kubernetes version (use kubectl version). If your Kubernetes components have different versions, please provide the version for all of them.
Container runtime: which runtime are you using (e.g. containerd, cri-o, docker) and which version are you using?
Linux kernel version on the Kubernetes Nodes (uname -r).
If you chose to compile the Open vSwitch kernel module manually instead of using the kernel module built into the Linux kernel, which version of the OVS kernel module are you using? Include the output of modinfo openvswitch for the Kubernetes Nodes.

Additional context
Add any other context about the problem here, such as Antrea logs, kubelet logs, etc.
(Please consider pasting long output into a GitHub gist or any other pastebin.)

@ZhangYW18 could you help fix this?

Hi, I can help and take this issue.

Should I sort the rows according to "namespace" column or "name" column?

Should I sort the rows according to "namespace" column or "name" column?

It would be perfect to sort Namespace first, then name.


Fail to use google home mini as bluetooth speaker

Problem
Hi, I'm trying to use my google home mini as a bluetooth speaker to play audio from a raspberry pi 4. The raspberry uses dietpi, a distro that doesn't have pulseaudio nor pipewire, just plain ALSA. After pairing and connecting to the speaker using bluetoothctl, running aplay -D bluealsa beep.wav works just the first time, then it fails to play further sounds until a reboot is performed, followed by a reconnect. Using the home mini as a bluetooth speaker from my android phone works fine with no issues.
Here are the debug logs of the bluealsa service, which contain the successful play attempt and a couple failed attempts:
logs.txt
And here are the logs of aplay:
logs_aplay.txt
Setup
OS: Linux 5.15.84-v8+, DietPi v8.14.2, RPi 4 Model B (aarch64)
Bluealsa: I'm using the master branch of bluez-alsa (541523ae1a3c4c2fb7264543777a48a93d66cd47), built using the following flags: ../configure --enable-mp3lame  --enable-manpages --enable-systemd --enable-debug --disable-payloadcheck
BlueZ: 5.55
ALSA: aplay: version 1.2.4

It's quite possible that you've hit the bug introduced some time ago. Unfortunately, I've got no time right now to fix it properly. Try to replace these lines:
https://github.com/arkq/bluez-alsa/blob/541523ae1a3c4c2fb7264543777a48a93d66cd47/src/ba-transport.c#L1625-L1626
with:
	if ((fd = t->acquire(t)) != -1) {
		ba_transport_thread_state_set_idle(&t->thread_enc);
		ba_transport_thread_state_set_idle(&t->thread_dec);
		acquired = true;
	}

Thank you for the quick reply. I tried your fix and the result was virtually the same as before: aplay worked two times and then it got stuck. Here are the logs:  logs_aplay2.txt
Disconnecting and reconnecting to the speaker does not change anything, I always have to reboot. Not even restarting bluealsa.service fixes it :(

OK, I'll look deeper into that issue...
Restarting bluealsa.service should have fixed the "stuck" issue. However, it's quite possible that bluealsa service is not restarted.  It still got stuck after a couple aplay runs, but at least it didn't require a reboot. It was kind of strange as the bluetoothctl connect command took almost 30 seconds to run, and it even logged an error before connecting. I'll add the logs again, as I don't know if they are useful or not... logs_pkill_connect.txt

Please try the current master branch. If the issue is not fixed, please reopen it.


Social authentication with djoser for react applications

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

This closes #6116

Hey @JohnKiguru ,
Thanks for submitting this PR.
Your PR has entered our reviewing queue. Please wait until a peer-reviewer pings you with the changes to be made.
Meanwhile, we suggest you to go through other approved PRs/published articles to see if you have missed out anything, which might avoid a potential PR closure.
Thanks.

Done...

Plagiarism check complete

Done...

Thanks
On Thu, 10 Feb 2022, 09:54 Jerim Kaura, @.***> wrote:

@.**** approved this pull request.
Great article @JohnKiguru https://github.com/JohnKiguru
MOving this to the final review.
—
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/6286#pullrequestreview-878399430,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASTGUE4UZ4B7YLVI4EHMBZ3U2NOLNANCNFSM5M5O2PJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
<section-engineering-education/engineering-education/pull/6286/review/878399430
@github.com>

jumping on this one


devise method to detect when an asset's status has changed from active to inactive

Currently all assets are marked as inactive and then the assets in the application are changed to active. As a result the program doesn't know which assets are changing in status from active -> inactive

closed by #12


Added npm-force-resolutions step for immer version issue

Description of changes:

added explicit ignore relating to useEffects lint warning (with
explanation of why)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

👍 👍

👍 👍


BEGINNERS: Docker example redirects to non-existing URL after setup unless properly configured

(moved from https://github.com/go-gitea/docs/issues/173)
Background
The instructions at https://docs.gitea.io/en-us/install-with-docker/ that launched Gitea using
docker run -d --name=gitea -p 10022:22 -p 10080:3000 -v /var/lib/gitea:/data gitea/gitea:latest

informs the user that the web UI is available at http://hostname:10080/.  When one goes there, one is redirected to http://hostname:10080/install for "Initial configuration".  So far so good.  Next, when one clicks 'Install Gitea' the browser is redirected to  http://localhost:3000/user/login.
Issue
If localhost != hostname (e.g. running on a remote machine), this causes the browser to throw an error that the URL is non-existing.  This is confusing to someone who is trying out Gitea for the first time (or coming back to a new setup).  The workaround that a user might try is to go back to http://hostname:10080/, which then takes you to the Gitea front page.  At this stage it is not clear to the user whether this was a hickup in Gitea or they hacked Gitea into a semi-working state ("will this come and haunt me later or am I good?").
I assume the proper solution is for the user to configure Gitea using:

Domain: hostname
[...]
Application URL: http://hostname:10080/

At least when doing so, the browser is correctly redirected to http://hostname:10080/user/login.
What is not clear to me is if SSH Port and/or HTTP Port should be changed from 22 and 3000 to 10022 and 10080 as well.  Is there a need for 'HTTP URL' and 'SSH URL' settings too to distinguish between internal Docker ports and publicly facing ports?
Proposal
Clarify on https://docs.gitea.io/en-us/install-with-docker/ how to configure Gitea in the initial setup.
(Should this issue be addressed to https://github.com/go-gitea/gitea instead?)

Pasting in @techknowlogick's reply from gitea/docs:
Thanks for opening up this ticket @HenrikBengtsson :smile:
These are really good questions that you are asking, inside the docker container the ports should remain 3000 for HTTP and 22 for SSH (if you are running docker via the docker run command listed in the documentation). This is due to docker remapping those ports to 10080 and 10022 respectively (the -p 10022:22 -p 10080:3000 part of the docker run command).
This repo has been deprecated (as of yesterday), and the docs are now being managed in the main gitea repo.

Hi @HenrikBengtsson,
The docs site has been updated to the most recent version of the docs. Hopefully they are now more clear.

see #2270, change app.ini
LOCAL_ROOT_URL  = http://localhost:3000/

Closing issue, as docs have been updated. Please open a post in the forum if this is still an issue.

Hmm actually I wonder if we can do better: the docker instance should suggest the up address of the docker instead of localhost by default.
It's very annoying to have to remember to change this every time you download an instance to check it.
Would making this change affect tests?


⚠ ClamAV has degraded performance

In e6175c2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 439 ms

Resolved: ClamAV performance has improved in c6e98ee.


Add capabilities support for providers (e.g., hierarchy support)

A provider should announce the capabilities it supports. Starting with space  hierarchy as the first example. If space includes path annotation && provider doesn't support hierarchy --> return an error.

move to backlog per @ezrasilvera

not  relevant for ks-0.20


fix(maker-dmg): remove unnecessary appPath config value

Fixes https://github.com/electron/forge/issues/3712

Actually, I see that this is a duplicate of https://github.com/electron/forge/pull/3761, will fix that PR up and submit it instead so the original author can get credit.


[request]: sdk model structs should not have Optional type if the field is marked as required in smithy.

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Tell us about your request
Currently, all the fields in generated types are marked as Optional. But, the sdk should generate types where fields should not be marked as Optional if it is marked as @required in Smithy.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
The generated sdk model types should not have Optional type if the field is marked as required in smithy.
Are you currently working around this issue?
Because, it is marked as Optional,  there is no compile-time validation to say if the entire type is fully constructed or not.
As a work around, we create another wrapper type which does not use Optional for required fields.
Additional context
Anything else we should know?
Attachments
If you think you might have additional information that you'd like to include via an attachment, please do - we'll take a look. (Remember to remove any personally-identifiable information.)

see also #163

I think if the service team removes required trait, that is backward incompatible change from my perspective. In which case, a new version of SDK needs to be generated that customer can use.
In that way, customer will come to know explicitly that an optional field now becomes mandatory or vice-versa.

Tracking this in #536.


About using the model in other datasets

Do I understand correctly that I can parse the panoramas to some other city and use your pre-trained models? That is, for each image, extract the embedding and save them. Then submit a new image (query) to the model, get an embedding for it and find the nearest to it in the previously saved ones (using faiss for example). Or should I train a new model on my data?

Yes, that is correct, there is no need to train a new model on your data.
In fact, the paper shows that our models reaches state-of-the-art results even when it is used on images from other cities.
Only one consideration: if you are using panoramas, I suggest you extract embeddings from crops, not from the whole panorama. For example you can crop the panorama in 12 images, and pass those to the model, and obtain 12 separate embeddings

Panoramas are stored as screenshots. I.e. for one point a screenshot from the panorama at 0 degrees, 90, 180, 270.
So I need to extract several embeddings per image when saving to the database, or to extract several embeddings when searching from a query image?

I don't understand your questions much, could you rephrase?
Anyway, the idea is to pass to the model images that are squared, or almost squared. So if you have an image that is 500 x 1000 pixels, it is good to cut it in two and obtain two embeddings from it

My images are in 1920*722 resolution. Accordingly, I need to divide them into left and right parts, obtain and save the embeddings separately? And I don't need to do that for the query image.

And one more question, does it make sense to bring the images to a certain resolution and normalize or can I not do that?

The best way to crop the images is to make them as similar as possible as the queries.
If your images are 1920x722 one way to proceed is resizing them to 1360x512 and then obtain 3 partially overlapping crops of 512x512.
Normalization of the image before passing it to the model is necessary, as it is done in the code


Rip out old pipes from coda_lib

Rips out old pipes from coda_lib, and makes transition frontier controller return a pipe

I'll change the base to tfc-staging after my other PR lands


panics when running Hello World under WSL Debian with VcXsrv

I have WSL Debian running with an x server. Most x applications work fine. When I try to run the hello world, with backtrace on, I get an error that seems to be related to webgpu. The webgpu should not be used in this context. Is there a way to tell iced not use the wepgpu? I looked around but could not find it. Thank you.
cargo run
warning: Error finalizing incremental compilation session directory `/home/amber/projects/openscan/target/debug/incremental/openscan-2zoz56y28rfvu/s-fswkhvt0e8-g6htin-working`: Permission denied (os error 13)

warning: 1 warning emitted

    Finished dev [unoptimized + debuginfo] target(s) in 0.64s
     Running `target/debug/openscan`
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /home/amber/.cargo/registry/src/github.com-1ecc6299db9ec823/wgpu-native-0.4.3/src/instance.rs:474:72
stack backtrace:
   0: rust_begin_unwind
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:475
   1: core::panicking::panic_fmt
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/panicking.rs:85
   2: core::panicking::panic
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/panicking.rs:50
   3: core::option::Option<T>::unwrap
             at /home/amber/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/option.rs:370
   4: wgpu_request_adapter
             at /home/amber/.cargo/registry/src/github.com-1ecc6299db9ec823/wgpu-native-0.4.3/src/instance.rs:474
   5: wgpu::Adapter::request
             at /home/amber/.cargo/registry/src/github.com-1ecc6299db9ec823/wgpu-0.4.0/src/lib.rs:545
   6: <iced_wgpu::window::backend::Backend as iced_native::window::backend::Backend>::new
             at /home/amber/.cargo/registry/src/github.com-1ecc6299db9ec823/iced_wgpu-0.2.3/src/window/backend.rs:21
   7: iced_winit::application::Application::run
             at /home/amber/.cargo/registry/src/github.com-1ecc6299db9ec823/iced_winit-0.1.0/src/application.rs:180
   8: iced::application::Application::run
             at /home/amber/.cargo/registry/src/github.com-1ecc6299db9ec823/iced-0.1.1/src/application.rs:201
   9: openscan::main
             at ./src/main.rs:4
  10: core::ops::function::FnOnce::call_once
             at /home/amber/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ops/function.rs:227
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Afaik vulkan doesn't work on wsl and therefore wgpu doesn't work either, because wgpu is still missing opengl support. I switched to the glow (opengl) backend/feature to work around that and so far it's running nicely in wsl.

That just gives me this error, but I could be doing this wrong.
the package `openscan` depends on `iced`, with features: `glow` but `iced` does  not have these features.

Oh, I think the current crates.io version is too old for that, not sure tho because I'm using the git version. Is using the git version an option for the moment?

No, let me try that.

That fixed it, thank you!
TLDR: glow (opengl backend) is required for WSL, because vulkan is unsupported.


Add function field returning an array view of a field

Also adds a performance improvement for CPU if B=1.

Codecov Report

Merging #12 (74ccd1f) into main (8cdfe8a) will decrease coverage by 9.62%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   72.32%   62.69%   -9.63%
==========================================
  Files           2        2
  Lines         112      126      +14
==========================================
- Hits           81       79       -2
- Misses         31       47      +16

Impacted Files
Coverage Δ

src/CellArray.jl
63.41% <44.44%> (-9.98%)
:arrow_down:

Continue to review full report at Codecov.

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


Some metrics not uploaded to S3 bucket.

Thanos, Prometheus and Golang version used
thanos, version 0.1.0 (branch: HEAD, revision: ebb58c23a786b2ba714ff07b1795efea7ef37132)
build user:       root@393679b8c49c
build date:       20180918-17:02:30
go version:       go1.10.3
Prometheus: 2.4.2
Backend: S3 (RadosGW + ceph)

What happened
Not sure if this is a bug or a misconfiguration. If I remember correctly when you run the sidecar for the first time, it uploads all the metrics currently stored in local prometheus data folder. Although, I've just added a new thanos installation on our current prometheus server and only some of them were uploaded.
Here is the content local prometheus folder:
drwxr-xr-x. 3 prometheus prometheus  68 Sep 12 10:03 01CQ6CX1XCMJBCX320E1JXV4F2
drwxr-xr-x. 3 prometheus prometheus  68 Sep 25 23:04 01CR98K5Z6PXNZWAQBA6Y3E3VE
drwxr-xr-x. 3 prometheus prometheus  68 Oct  2 17:01 01CRTMQ58BRE5D6Y87RJYDA3E2
drwxr-xr-x. 3 prometheus prometheus  68 Oct  9 11:01 01CSC0WZVGGJZ8AQR23GNMB52Y
drwxr-xr-x. 3 prometheus prometheus  68 Oct 10 05:00 01CSDYNSJ0AGFNCD444G5HXBVX
drwxr-xr-x. 3 prometheus prometheus  68 Oct 10 11:00 01CSEK8V22W51W1RR8RMQCTK1Y
drwxr-xr-x. 3 prometheus prometheus  68 Oct 10 17:00 01CSF7VS0MKYSCTWW5XZ5ZH64X
drwxr-xr-x. 3 prometheus prometheus  68 Oct 10 17:00 01CSF7W076R8NBZPB2V3JD7E93
But only the following dirs were uploaded in the S3 bucket:
01CSEK8KXBYGV0A01WKZQ9XMS9
01CSET4B5BJA512MNM801PNAJ4
01CSF102DBXDQDAS77YJ4N02EG
01CSF7VS0MKYSCTWW5XZ5ZH64X
Interestingly, the content of thanos.shipper.json is like this:
{
"version": 1,
"uploaded": [
"01CQ6CX1XCMJBCX320E1JXV4F2",
"01CR98K5Z6PXNZWAQBA6Y3E3VE",
"01CRTMQ58BRE5D6Y87RJYDA3E2",
"01CSC0WZVGGJZ8AQR23GNMB52Y",
"01CSDYNSJ0AGFNCD444G5HXBVX",
"01CSEK8V22W51W1RR8RMQCTK1Y",
"01CSF7VS0MKYSCTWW5XZ5ZH64X",
"01CSF7W076R8NBZPB2V3JD7E93"
]
}
The local retention is set to 365d, I've though that maybe this was the problem and tried with 30d, 25d but the older metrics were not uploaded. I see no errors on thanos-sidecar and the new metrics  are uploaded correctly.
How I can upload the old metrics?
Thank you a lot!

Not sure if this is a bug or a misconfiguration. If I remember correctly when you run the sidecar for the first time, it uploads all the metrics currently stored in local prometheus data folder. Although, I've just added a new thanos installation on our current prometheus server and only some of them were uploaded.

This is not the case, see this: https://github.com/improbable-eng/thanos/issues/348
Only new metrics will be uploaded. There is no safe way to upload old already compacted ones.

Thanks Bartek and sorry for the noise :)


Twitch API needs updating

The Twitch API got updated and now the Twitch integration in the script doesn't work.

👍
Following on Twitch & checking for subscription status still work but the most important Unfollowing on Twitch doesn't work.
@Citrinate please don't forsake us. 😄

I just tested the "Unfollow", "Delete Retweet", and "Delete Tweet" functionality, but was unable to reproduce these types of errors.
I did find a new type of Twitter action called "twitter_hashtags", which has you sending out a Tweet through a popup box.  A new version has been released, v1.2.1, which adds support for this action type.

Twitter was never a problem, Twitch was. Unfollowing on twitch doesn't work. "Gleam.helper Error: Failed to unfollow Twitch user: xxx".

@Citrinate As @TizianoVecellio has pointed out what we already said: Twitter is not the problem, it's Twitch. twitch.tv
Thanks for all your time and help. <3

My bad, thanks for correcting me.
v1.2.2 has been released to address the changes to the Twitch API.

Thanks so much Citrinate! ❤

Thanks man, really appreciate it.


Added "run" into the readme CMD line

Just a tiny tweak so folks don't end up wondering what went wrong first time through.

Thanks!


Does it work with multiple schema files?

How does it work when schemas are not all in one schemas.ts file but broken down like this? :
📦
└ 📂 src
└ 📂 db
└ 📂 schema
├ 📜 users.ts
├ 📜 countries.ts
├ 📜 cities.ts
├ 📜 products.ts
├ 📜 clients.ts
├ 📜 enums.ts
└ 📜 etc.ts

You can make an index.ts file inside schema and re-export everything there. So, following your example:
// schema/index.ts
export * from './users';
export * from './countries';
export * from './cities';
export * from './products';
export * from './clients';
export * from './enums';
export * from './etc';

And then in the script you use to generate it you can import it as normal:
import * as schema from './schema';


Npm publish

Remove node version test matrix. Only test on node 6 instea.d
Add /dist/ folder, *.tgz and .npmrc to .gitignore
Add test for codecov report (there's a bug, but i think it's on their side)
Add semantic-release for full auto publishing
Add ghooks to pre-validate commits and pushes
Change package name to "@crazy-factory/cf-service-client"
Update readme.md to reflect deployment changes

Fixes #21

This obviously shows, that i still need to learn a lot about the correct usage of commit types. There will be a briefing related to that soon.


fix: example values for substra-tools

Description

Reference the correct image of substra-tools.
Motivation and Context

We forgot to change this reference during the migration.
How Has This Been Tested?

Starting the backend locally the image loaded correctly.

Companion PR:

https://github.com/Substra/substra-tests/pull/204


Support update of silences

Allow update of silences if they are updated in the silence repo
Checklist

[x] Update changelog in CHANGELOG.md.

towards https://github.com/giantswarm/giantswarm/issues/22219


Add file integrity checks to CI workflow

Description
Fixes https://github.com/banzaicloud/koperator/issues/1048
Pinned GH actions to specific commit SHA for security reasons.
Type of Change

[x] New Feature

Checklist

[x] I have read the contributing guidelines
[x] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests
[x] All new and existing tests pass

Depends on #1032, merging is blocked for now.

https://github.com/banzaicloud/koperator/pull/1032 is now merged.


let action creators be passed via the provider

First off, thanks so much for Redux and the amazing tooling and introductory content around it. It's been awesome to learn how all of the parts fit together.
I've been running into an issue while building out my first foray into a React/dux app dealing with action creators. I'm having to use mapDispatch a ton for just passing the same action creators into the props of each of my top-level components (without React Router I end up having around 5 top-level components so far, which I've called "containers" following the examples I've seen).
And the mapDispatch that I'm writing is often just a super plain object like { loadUser, loadTeam, loadCollection } that maps keys to keys. It's getting really boilerplate-y to have to keep doing this all the time, when really I'd just like to have all my action creators already bound to my single store.
It would be amazing to be able to pass them into the <Provider>:
<Provider store={store} actions={actions}>
  <App />
</Provider>

And be able to use that inside components as:
componentWillMount() {
  const { actions } = this.props
  actions.loadUser(...)
  actions.loadTeams(...)
  ...
}

From looking through the issues, I found that I'm not the first to suggest this, which means others are grappling with this problem too. Here are a few that mention discuss similar things:

@mmerickel suggested it in https://github.com/rackt/react-redux/issues/1, except with the extra addition of using mapDispatch as well, which I think we can eliminate.
@maxguzenski suggested it in https://github.com/rackt/react-redux/issues/23.
@gyzerok brought up the same pain point in https://github.com/rackt/redux/issues/231.
And a few of the example snippets in the Readme actually mention a similar use case (of binding all action creators easily), but they do it at connect-time for each component, instead of at the Provider level, which would mean not having to worry about it any more.

One time when it was brought up, it was discarded because of not wanting to pollute props with all of the actions, since the "clobber likelihood" would be high. I totally get the concern about not wanting to pollute props with tons of actions, but we can avoid that by nesting them in props.actions instead.
Another time, @gaearon mentioned the (important) needs of code splitting. I think that's definitely a use case to support, but surely we could have a load-later approach that works very similarly to the current mapDispatch that would facilitate loading extra modules, while also making it easy for the main use case?
Curious to hear everyone's thoughts.
From my initial use of both React and Redux, this would save me boilerplate code, and make it possible for people to write nicer boilerplate-reducing functionality on top of the base libraries.

I don't understand what is boilerplatey about specifying a single-line function several times. Can you give specific examples?

Absolutely! In my project so far, I have 5 container components, all of which need to "connected". And I haven't fleshed out the entire thing yet, so that number will grow for sure. For each one I have something along the lines of:
import { loadCollection, loadItem, loadTeam } from '../../actions'

// ...

function loadData(props) {
  const { loadCollection, loadItem, loadTeam } = props
}

// ...

const mapDispatchToProps = {
  loadCollection,
  loadItem,
  loadTeam
}

// ...

const Connected = connect(mapStateToProps, mapDispatchToProps)(Component)

But just 3 actions for a container isn't a lot; I expect some of them to need 10 or more. And each time I add or change an action, I have to make sure to update it in all of these places, or things break in ways that are sometimes hard to discern.
For example, I forgot to add loadTeam to mapDispatchToProps since it was at the bottom of the file, and then proceeded to be pretty confused since the reference still existed, it just wasn't to a "bound" action creator, so the error was non-obvious. Of course I could rewrite my import to make that not possible:
import * from '../../actions'

// ... but then I have to make my dispatch mapping more verbose ...

const mapDispatchToProps = {
  loadCollection: actions.loadCollection,
  loadItem: actions.loadItem,
  loadTeam: actions.loadTeam
}

If I actually gained from the configuration I might understand why it needed to be verbose, but for my own developer experience, the ideal for me would be to not have to worry about these kinds of things, and just have any function I exported from * as actions be bound to the store automatically. That way, even though I understand how it's all wired up, I'd never have to think about the mappings again.
 Ideally as easily as just two things to think about:
export function loadCollection(id) {}

// ...

const { actions } = this.props
actions.loadCollection(id)

If I was able to write this in myself from the top-level, I'd be more than glad. But it seems like the way the system is setup, there's no way to pass other information into the <Provider> instance, so there's no way I can do it without creating a singleton, which I want to avoid. (I may be missing a good trick here though—I guess I could attach bound actions to the store.dispatch function if I really wanted to, but that seems real hacky.) I've even looked into writing a different <Provider>, but there's so much logic in it that would need to be duplicated and kept up to date that it would be a huge pain to do.
Let me know if that all makes sense, or if I'm doing something incorrectly. Happy to discuss more. Thanks!

You can provide you own connect, and use reselect to memoize it.
import { createSelector } from 'reselect'
import { bindActionCreators } from 'redux'
import { connect as reduxConnect } from 'react-redux'
import { * as myActions } from './actions'

const mapDispatchToProps = createSelector(
  dispatch => dispatch,
  dispatch => ({actions: bindActionCreators(myActions)})
)

const connect = (mapStateToProps, mergeProps) => {
  return reduxConnect(mapStateToProps, mapDispatchToProps, mergeProps)
}

// use this connect on yours Components
export default connect

Please take a close look at all examples in the docs. What you want is either
import * as actions from '../actions';

export default connect(
  mapStateToProps,
  actions
)(MyComponent);

// this.props.loadData()

or
import * as actions from '../actions';
import { bindActionCreators } from 'redux';

const mapDispatchToProps = (dispatch) => {
  return {
    actions: bindActionCreators(actions, dispatch)
};

export default connect(
  mapStateToProps,
  mapDispatchToProps
)(MyComponent);

// this.props.actions.loadData()

There's no need to explicitly enumerate them.
By the way you can totally write your own Provider. Not sure what logic duplicate you are concerned about because there is almost no logic there—it just puts store in context and that's it.

Thanks for the response again. I actually did look at all of the examples before asking.
Both solutions solve the problem partially, but still require that the boilerplate in every file, even though I know I want every "connected" component to have access to all of the bound action creators, just like they have access to the store. Which means not only is it impossible for me remove the boilerplate completely (like with store passing), but it means that it's impossible to pave over react-redux simply for this case from another library. It has to be done in the end files.
I could create a separate <Provider>, which is pretty lightweight, but it seems like I'd also need implement a separate connect() as well, which has tons of extra logic that I'd need to carefully pass through, or reimplement, so maintaining it would be very non-trivial.
It seems like adding all of the bound action creators onto store.dispatch as properties is the only way to do it from the top-level without having to repeat in each file, so I'll go with that, even if it feels a bit hackish. Thanks


Class property Missing annotation

Flow doesn't accept this:
class {
  handleSomething = (event: E): void => {};
}

But it's ok w/ this:
class {
  handleSomething: (event: E) => void = (event) => {};
}

Is this intentional?

What do you mean? I got a syntax error and then "identifier E. Could not resolve name" for both

This is weird, I just tested it on simple case and it works fine. But in my app it gives me this:

Something else is causing this. Sry for the noise, closing it. Thanks for help 👍


Range check error in OP_LOOP, OP_LOOPNG:

regex A(?>(?:b|c(?:d|e)*?)*?_)a
text Acece_x_Acece_a_
produce a range check error
LoopStackIdx = 0  for TRegExprLoopStack = array [1 .. LoopStackMax] of integer;
revision 39aaae8d37d3399a04f84c14cd8b6c0d38abfdf6

w/out thinking I changed lower index of this type to 0
type
  TRegExprLoopStack = array [0 .. LoopStackMax] of integer;

now demo test_dlg handles it

Looking at
    LoopStackIdx: integer; // 0 - out of all loops

I don't think that is correct.
It gets increased in
      OP_LOOPENTRY:
        begin // ###0.925
          no := LoopStackIdx;
          Inc(LoopStackIdx);

OP_LOOP, OP_LOOPNG: increases and decreases it.
But that is as far as I looked.

But just to prove the point: I changed it to start at 0, and then run the below.
RegEx A(?>(?:b|c(?:d|e(?:f|g)*?)*?)*?_)a
Text: Acece_x_Acecegg_a_
The index goes to -1

Interesting is, that it does not crash, if the text has a single "g", only with the 2nd "g".
Maybe that holds some clue. But currently got other stuff to do.

I set breakpoint on  Dec(LoopStackIdx) and see that this Dec called to much, in the handled of
OP_LOOP, OP_LOOPNG:

Simple fix: after
Dec(LoopStackIdx);
add
            if LoopStackIdx < 1 then
                LoopStackIdx := 1;

Then it does not match
https://regex101.com/r/7FAwus/1
A(?>(?:b|c(?:d|e(?:f|g){0,2}?){1,1}?){2,2}?_)a
Acece_x_Acecegg_a_
Not sure if that is the same bug, but I thing so because
A(?>(?:b|c(?:d|e(?:f|g){0,2}?){0,1}?){2,2}?_)a
does match.
I only changed {1,1} to {0,1}
Despite the sequence starting with e exists exactly 2 times.
But it was logical that it would happen, if the index gets decreased wrongly, then it does not only go out of range, at some time it also points to the wrong entry.

maybe: you added additional Exit's for the OP_STAR, but did not add them for OP_LOOPNG - ie for {0,1}? .

I might have an idea what the issue is. Need to do some more checking though...

So this were actually two issues.

LoopStackIdx is incremented once in OP_LOOPENTRY
When iterating a loop, each OP_LOOP makes a recursive call. Each recursive call decrements the LoopStackIdx. That is far more often than it got incremented.
To correct for that, nested calls are wrapped. They store the LoopStackIdx and restore it (undoing the nested decrement, allowing themself to decrement again).
Some calls were not guarded. Fixed that.
A note on the side.

OP_LOOP (greedy) "must" decrement in each call, because it can do so, before making the call to test the rest of the pattern => thereby freeing one index for that rest of the call (less likely to run into an error by maxing out the available stack)
"must" well it is only an optimization for allowing more loops in later parts of the pattern.
OP_LOOPNG probably (to the best of my ability to judge it) could have the dec(LoopStackIdx removed. It returns a wrong value anyway, and the caller does fix it anyway.

When a loop tests the if at its current iteration, the pattern after the loop will be a match (and the loop therefore would have found a success), then the call to test "the pattern after" will run through the OP_LOOP of outer loops.
This is actually correct. It does actually test if the current loop, at the current iteration is fine for any iteration of the outer (that is: "not just the outers current").
But, for that to work, the LoopStackIdx must be adjusted to point to the outer loops data.

Ok, this is way more complex than it initially looked like....
But solvable.
I explain one of the issues now, because I have tracked it now, just because I then don't have to do later.... (to explain the upcoming changeset).
A simplified example:
RegEx   'Aa(x|y(x|y){3,4}?){1,3}?B'
Text   'AayyyyyyyyyyB' // 10 y

The outer loop has to much 3 times as the inner can not cover this in one go.
The inner loop matches 3 y in the first and 2nd go, and then 4 in the 3rd.

1: BRANCH (131)
6: EXACTLY (15) Aa
15: LOOPENTRY (110)  OUTER LOOP
20:  OPEN[1] (25)
25:   BRANCH (38)
30:   EXACTLY (105) x
38:   BRANCH (105)
43:   EXACTLY (51) y
51:   LOOPENTRY (92)  INNER LOOP
56:    OPEN[2] (61)
61:     BRANCH (74)
66:     EXACTLY (87) x
74:     BRANCH (87)
79:     EXACTLY (87) y
87:    CLOSE[2] (92)
92:   LOOPNG (105)  -> (56) {3,4}  INNER LOOP
105:  CLOSE[1] (110)
110: LOOPNG (123)  -> (20) {1,3}  OUTER LOOP
123: EXACTLY (131) B
131: END (0)
In the below each line is a nested call of the previous. (no backtracing in this simplified example).
So this is all going deeper and deeper on the stack (because it may need the option to backtrace if the text was different)

15: LOOPENTRY: Outer loop enters: LoopStackIdx = 1
110: LOOPNG:    Outer loop start first loop: LoopStackIdx = 1
20: matching
51: LOOPENTRY: Inner loop enters: LoopStackIdx = 2
92: LOOPNG:    Inner Loop takes 3 y
56: matching 3 times
92: LOOPNG:    Inner Loop will check MatchPrim(next) // that is: "can the text be matched if the (first instance of the) inner loop has matched 3 y"
110: LOOPNG:    Outer Loop will loop for 2nd match LoopStackIdx = 1
===> Attention LoopStackIdx = 1
20: matching
51: LOOPENTRY: Inner loop enters: LoopStackIdx = 3
===> Yes inner loop must go to 3. Because in case of backtracking the inner loop with LoopStackIdx=1 may need to continue
92: LOOPNG:    Inner Loop takes 3 y
56: matching
92: LOOPNG:    Inner Loop will check MatchPrim(next) again
110: LOOPNG:    Outer Loop will loop for 3rd match LoopStackIdx = 1
===> Attention LoopStackIdx = 1
20: matching
92: LOOPNG:    Inner loop enters: LoopStackIdx = 4
....

Each time on instruction 92, when the inner loop needs to check, if the remaining pattern (including outstanding loops of the outer loop) could match,
the inner loop calls MatchPrim(105) => which goes to the outer loop on 110.
There is only one instance of the outer loop, it always needs LoopStackIdx = 1.
There are 3 instances of inner loops, with different LoopStackIdx. So the outer LoopStackIdx can not be calculated from that.
Trying to achieve this with a "LoopStackIdx" may be possible, but will be hard...
I have a different idea...

Also the "not that complex" pattern
'Aa(x|y(x|y(x|y){3,4}){3,4}){3,4}'
needs more than the LoopStackMax=10
Depending on what is matched, there can be 4 instances of the middle loop, which can each have 4 of the inner => that would be 17.
Add a copy of the pattern at the end, and it will be 34.

I have a good idea how to solve that...

replaced by #312

@User4martin Let's close?

Wait for the PR?
But if you prefer, happy if you close it before


Added siteCSS & siteJavascript for plugins variables.

To use with Theme::plugins('siteJavascript') to place all javascript files from plugins in the same place. Good to use when you want to place all .js files in end of body without use of siteBodyEnd.
I myself always want the .js files in the end of the body so they don't need to load before the site is rendered, so it won't be a white page.
Good to group all .css and .js files together for easier source code.

Hi,
I'm not sure about this new hook because you already have siteHead, siteBodyBegin, siteBodyEnd.
And you are going to add more of the same, and the developers of the themes need to add this 3 (siteHead, siteBodyBegin, siteBodyEnd) and the 2 you propose.

Understand that. I just thought about it so you have a easy way to render all JavaScript files at the same place cause I think they should be placed at the end of the body. But as you say, it will be to many hooks to keep track of. A little messy maybe.

But you can use this webkhook, siteBodyEnd

Yes you can. My idea was so if plugin HTML wont blend with script tags. But you're absolutely right, less hassle for the user.

Another thing I made some changes on Bludit v2 respect to the Helper Theme, so if you just start working with Bludit, I recommend using the latest Beta of Bludit v2.


[language] remove diem-types dependencies from the ir compiler

This removes the diem-types dependencies from the IR compiler binary and a few library components.
Notable change: the IR compiler no longer has a default address and the client will need to specify one.

/land


Thoughts on Timers vs Managed Cycles?

Issue summary
What are your thoughts on watchers being simple timers vs using a shared request pool to throttle amount of requests to MTurk server?
Does MTurk Engine recognize 429 HTTP codes and do any sort of backoff?
Expected behavior
Perhaps throttle outgoing watchers to a shared limit, so the current timer setting is a maximum speed rather than actual speed (or a sort of priority)? It could also spread out watchers evenly so there isn't bunching of requests all at once. Bonus points if it recognized that 429s were showing up and slowed down/sped up dynamically.
It appears the recommended limit is ~1 request per second:
https://turkerhub.com/wiki/Page-Request-Error-PRE
It sounds like this is what Panda Crazy does (put all the watchers in a shared queue and only send out requests at a certain maximum rate, so if you have a bunch of watchers it just takes longer to get through all of them instead of causing PREs)?
Actual behavior
The current watcher system appears to be just simple timer based. Each watcher is independently just counting down a timer until it fires off a page request.
Steps to Reproduce the Problem
Add more than a couple watchers (ex: >5 with 5 sec timer) and notice that it gets a lot of Page Request Errors (PRE) (HTTP 429 too many requests). Or if you have multiple timers with the same duration, start them all at the same time so they all send requests bunched together instead of evenly spaced.
Specifications

Mturk Engine version: 1.6.2 (installed from GitHub)
Browser: Firefox 59.0.2 (64-bit) (MacOS 10.11.6)

Thanks for posting this issue @neocryptek. It's possible but I'm not sure if an automated system that launches multiple watchers one after the other would significantly reduce PREs. The current system allows for a good amount of clarity and simplicity with how often you can expect a watcher to be sending requests but I agree that letting the user know when a watcher causes a 429 is necessary information so they can make the decision to stop or slow it down.


Cannot authenticate

Ubuntu 18.04 MATE
Kernel Linux 4.16.11-041611-generic x86_64
google-drive-ocamlfuse, version 0.6.24
Google Chrome 67.0.3396.56 beta
When I try to authenticate using:
$ google-drive-ocamlfuse
I get the following in my browser:
500. That’s an error. There was an error. Please try again later. That’s all we know.
the URL it is at when this displays is:
https://accounts.google.com/signin/oauth?client_id=........
Have you experienced this before? I will keep trying, as this may be an issue on Google's end, but I wanted to start this process in case it is not. I have tried deleting the .gdfuse folder to ensure it is a fresh start as well, results do not change. This is a fresh install of google-drive-ocamlfuse. It has never been installed on this computer before. I have however used this on prior installs without problems. I have not before run this on 18.04 though. I believe the last time I used it was on a 17.04 install.

I have authenticated using my own OAuth2 client ID, using the method above is still not working with same results.

When I try to authenticate using:
$ google-drive-ocamlfuse
I get nothing. The browser does not launch, however the .gfuse directory is created.

This was closed as it was fixed? There is no comment as to what has happened.

No, the 500 error is returned by Google, so it's not an issue of the application. If you want, you can try to logout from all your accounts on the browser that is opening and try again (or clean all the cookies). But there is no way I can fix this.

i am using ubuntu 22.04, when i enter google-drive-ocamlfuse, the browser does not open for authentication. the .gdfuse/default/gdfuse.log is empty


ERROR Unknown Exception + Download partially failed

When i try to download with any course it's show this error :
ERROR    Unknown Exception
ERROR    'NoneType' object is not iterable
Traceback (most recent call last):
  File "c:\python27\lib\site-packages\linuxacademy_dl\cli.py", line 150, in main

    la.download()
  File "c:\python27\lib\site-packages\linuxacademy_dl\linux_academy.py", line 15
5, in download
    self._course.download()
  File "c:\python27\lib\site-packages\linuxacademy_dl\course.py", line 99, in do
wnload
    asset.download()
  File "c:\python27\lib\site-packages\linuxacademy_dl\assets.py", line 131, in d
ownload
    for resource in self._resources:
TypeError: 'NoneType' object is not iterable

And When i am download this course "https://linuxacademy.com/cp/modules/view/id/13" all its okay.
but when i am download this course "https://linuxacademy.com/cp/modules/view/id/38" only some of which can be downloaded, maybe 3 or 4 folder can downloaded.

Thank you for your report. It looks like a duplicate of #1. I fixed it already!


[dev to test] Add outlets to page layout component

In order to override page templates we can leverage outlets, so that customers can override or append content to existing page templates.

QA Steps

create an ng-template with a cxOutletRef, and assign it to a page template uid (i.e. LandingPage2Template) (can be done in storefront.component.html)
confirm that the custom content shows up in Spartacus
use before or after as alternatives

Works as intended. One qualm I noticed while QAing it that certain outlets have the same name. For example, Section2A is present in both the home page and the help/faq page. Therefore, injecting content into this outlet will render it in both pages.

Works as intended. One qualm I noticed while QAing it that certain outlets have the same name. For example, Section2A is present in both the home page and the help/faq page. Therefore, injecting content into this outlet will render it in both pages. Maybe the slots should change name or there should be a follow up item for this.

Very good point! I'm about to suggest a re-config of our backend page templates (as there are too many of them for no good reason (anymore)), this is good input in that refactor.


feat(compass): allow optional ID in CreateAssetProbeRequest

In the CreateAssetProbeRequest, allow specifying the ID for the probe. If the ID is specified, it should be used as the probe ID. If it is not specified, the ID should be created automatically. The ID if specified, should be a valid UUID.

@haveiss I have updated the PR description, does that suffice? Did you mean to request adding the context in the code?

sorry for not being clear, i mean on the PR it self


fix when attention_mask=None

Fix IPEX patched model forward when attention_mask is None.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Hi @IlyasMoutawwakil , it should be ready to merge.


Invoke-DbaSqlQuery -AppendServerInstance

Before submitting a bug report:

[x] Ensure you are able to reproduce it on the latest released version (0.9.343)
[x] Verified this bug is not already reported in an issue
[x] Verified errors are not related to permissions
[x] Can reproduce in a clean PowerShell session (clean = powershell -NoProfile)

Steps to Reproduce

PS> 'SVR1','SVR2' | Invoke-DbaSqlQuery -Query 'select 1' -AppendServerInstance

Column1 ServerInstance
------- --------------
      1 Sqlcollaborative.Dbatools.Parameter.DbaInstanceParameter[]
      1 Sqlcollaborative.Dbatools.Parameter.DbaInstanceParameter[]

Expected Behavior
It should provide the ServerInstance
Actual Behavior
It does not provide the ServerInstance anymore
Environmental data
Name                           Value
----                           -----
PSVersion                      5.1.14409.1012
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1012
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

@pollusb : thanks for the catch