Start reproducing flowsOrder not working

Attempts to fix #1971

Investigation results
This bug is introduced by #1732. Offending source code:
https://github.com/mobile-dev-inc/maestro/blob/a9575c5b1ce8ee13d549449e6887e5a1ec4caae4/maestro-cli/src/main/java/maestro/cli/command/TestCommand.kt#L197-L198
It occurs only when all flows are specified in executionOrder.flowsOrder. Take this config.yaml as an example:
flows:
  - "features/**"
includeTags:
  - regression
executionOrder:
    continueOnFailure: true
    flowsOrder:
        - test1
        - test2
        - test3

If in features/ there is only test1.yaml, test2.yaml, and test3.yaml, then the bug occurs. But if we add test_other.yaml, then the bug is fixed.
Fix: when running in a single shard, add ExecutionPlan.sequence at the beginning of the queue of flows to execute.
Also, I'd like to have a bit more output logging.


add context menu features on macOS touch bar

Fix #981

This gives the touch bar feature to goto Help, About. It provides the version of the application and helps the user directly quit the application.

This adds feature for About Dialog to close from touch Bar on macOS.

test this

Codecov Report

Merging #982 into master will increase coverage by 78.33%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #982       +/-   ##
===========================================
+ Coverage   16.48%   94.81%   +78.33%
===========================================
  Files          25       39       +14
  Lines        2378     3667     +1289
  Branches      222      538      +316
===========================================
+ Hits          392     3477     +3085
+ Misses       1986      180     -1806
- Partials        0       10       +10

Impacted Files
Coverage Δ

browser/model/jbossfusekaraf.js
95.5% <0%> (ø)

browser/model/cygwin.js
100% <0%> (ø)

browser/model/jbosseap-autoinstall.js
100% <0%> (ø)

browser/model/virtualbox.js
89.47% <0%> (ø)

browser/model/jbossfuse.js
100% <0%> (ø)

browser/model/helpers/version.js
100% <0%> (ø)

browser/model/jbosseap.js
100% <0%> (ø)

browser/model/jdk-install.js
96.61% <0%> (ø)

browser/pages/about/controller.js
100% <0%> (ø)

browser/model/devstudio.js
96.8% <0%> (ø)

... and 29 more

Continue to review full report at Codecov.

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


How to disable understand depending on env config?

I want to be able to pass in a variable from my .env to disable/enable understand.io on different environments. For instance, I don't want to post to understand.io when working on localhost.
I can set 'enabled' on each log type in the config, but I'd prefer a global variable for enabled as well perhaps?

@SturmDrangAltaar at the moment there is no global variable which you can set to false (only each for each type of logs as you mentioned) but I think this is something what we should implement in the near future.

This could be solved by putting API key to .env
Setting no API key in env file would disable logging in preferred deployment
This would also allow to have different channels for separate deployments.

@LukasLt Yes, to use env('UNDERSTAND_TOKEN'); for different environments is a good idea.


Don't truncate a non-existent message property on data

"Cannot read property 'length' of undefined when capturing exception" with undefined message when trying to capture an exception with a data object (second argument) that contains no data. For example:
raven.captureException(new Error("foo"), {
	level: "warning",
	extra: {
		foo: "bar"
	}
});

Causes the Raven script to throw an exception because data.message is undefined and truncate only works with strings.

This should prevent this from happening.

This change is

Damn, thanks for catching this – and right before we put out 3.2.1 too. I'll update that release w/ this patch.

Note that this bug also requires you to set maxMessageLength in Raven.config – it's 0 by default, and if it's 0 it skips accessing undefined.length.

Merging – I've also added a test that I'll land on top.

See also 31d99f95829080760b18f81a1c23f16a8929ace0


initialize component fields inline avoiding some initialization bugs

Fixes #155

It's great to see you here @ivangsa! For everyone that doesn't know Iván, he was one of AppFuse's largest contributors. ❤

It's nice to see you here as well Matt! Cheers!!

@sahbi-ktifa I pushed another commit to fix initialization of field 'reverse' order for entities, which is different for pagination and infinite-scroll, which I missed..
I'm still puzzled about those 3 tests that don't pass.. I've got them failing either way I initialize that class.. :-(

Everything seems good now right?

yes, test are passing now... it was a bit harder than anticipated..
(there is onw commit to be merged, just to let you know..)
cheers!


Significant performance deterioration of basis in 0.8

Due to using ForwardDiff to compute forces_basis. Should be relatively easy to fix by will require 1-2 days of focused work.

Fixed by #246


QOS modification in LDAP fails

When the QOS of an existing user is modified and one of the QOS values already exists in LDAP, the modification fails with the following error:
[INFO  usermgmt::ldap::ldap] User modification in LDAP failed! LDAP operation result: rc=20 (attributeOrValueExists), dn: "", text: "modify/add: slurmQos: value #0 already exists"

We should ignore the existing QOS in order to avoid the error.

This might be related to #1

Was fixed with #42


ElementTagNameMap should extend HTMLElementTagNameMap

TypeScript Version:  2.2.1
Code
Along with #14782, this would let custom element definitions be able to augment only one tag name map:
export class FooElement extends HTMLElement {}
customElements.define('foo', FooElement);

declare global {
  interface HTMLElementTagNameMap {
    "foo": FooElement,
  }
}

And with #14783 ElementTagNameMap could extend both HTMLElementTagNameMap and the proposed SVGElementTagNameMap.

PRs welcomed. You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

Looks like this was fixed in https://github.com/Microsoft/TSJS-lib-generator/commit/072c5487550dda70fe82d063680b3e7d7f478090


Set departure time

I would love to be able to set the departure time through this plug-in. The use case for me is to be able to conveniently start heating the battery just before I leave home.

+1 for this. At least reading the set times: For me the use case is reading what my wifey has set in the VW app and adjust the charging schedule to meet the departure time. (I have an electricity price that varies by the hour, so I control charging to get the best price possible)

Have you tried an app like gridio?
On Wed, 27 Dec 2023 at 17:52, BugBasher28 @.***> wrote:

+1 for this. At least reading the set times: For me the use case is
reading what my wifey has set in the VW app and adjust the charging
schedule to meet the departure time. (I have an electricity price that
varies by the hour, so I control charging to get the best price possible)
—
Reply to this email directly, view it on GitHub
https://github.com/mitch-dc/volkswagen_we_connect_id/issues/231#issuecomment-1870426857,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADI3TZTM4KMRVQEAW6F2YTDYLQ73ZAVCNFSM6AAAAABAEAVGACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQGQZDMOBVG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


exit status 5: Access is denied.

Issue:
When running nvm use 10.19.0 with nvm v1.1.8 I get a exit status 5: Access is denied. error.
When running with Administrator rights I get exit status 145: The directory is not empty.
How To Reproduce:

Install nvm v1.1.7
Run nvm install 10.19.0
Install Node v16.9.1
Run nvm install 16.9.1
Install nvm v1.1.8
Run nvm use 10.19.0

Expected Behavior:
It should switch to version 10.19.0
Screenshots:

Desktop:

OS: Windows 11

First, step #1 is "Install nvm 1.1.7" - did you mean 1.1.8?
If access is denied, your account doesn't have permission. The NVM4W code doesn't throw an exit status of 5, meaning this message is passed through from the operating system.
Was Node installed prior to the installation of NVM4W? If you try to override an existing Node installation, the directory may not be empty. The installer attempts to handle this circumstance, but the installer must use the same account used to install the original version of Node... and even then it isn't guaranteed to work on some editions of Windows. This is why the recommendation is to remove prior Node installations before installing NVM4W.
In a fresh environment, using an administrative account, I cannot recreate the issue:

However; using a non-admin account, you can see the permissions issue:

I get exit status 1: Toegang geweigerd. (It is in dutch but it says access denied) when upgrading to 1.18.
I am running on Windows 10 Pro 20H2
It only says that when I execute nvm use <version>

First, step #1 is "Install nvm 1.1.7" - did you mean 1.1.8?

No i meant 1.1.7

If access is denied, your account doesn't have permission. The NVM4W code doesn't throw an exit status of 5, meaning this message is passed through from the operating system.

I didn't switch account and was working fine before installing 1.1.8

Was Node installed prior to the installation of NVM4W? If you try to override an existing Node installation, the directory may not be empty. The installer attempts to handle this circumstance, but the installer must use the same account used to install the original version of Node...

Yes node was installed prior to the install. It was the same account.

and even then it isn't guaranteed to work on some editions of Windows. This is why the recommendation is to remove prior Node installations before installing NVM4W.

That must be the reason why it failed, i did not remove prior node install before installing the new version of NVM4M.

Another workaround is going to the nvm executable (to find it, open CMD and enter: where nvm) and giving it admin rights by going to the executable properties -> compatiblity tab -> enable "run this program as administrator".

for me worked running command line as administrator


Return Custom object

Allow custom objects to be returned of return type is object
and custom_return_object (new variable) is set to a class name.

Updated my_model.md also to have documentation in sync.

Thanks!


GPTQ guide?

Is there a guide to learning how GPTQ works? I found the paper hard to follow... thanks!

maybe my blog post can help it. It includes the math derivation and related code implementation.
https://sherlock-dev.netlify.app/posts/gptq-math-derivation/
https://sherlock-dev.netlify.app/posts/gptq-code-implementation/


Scrolling freezes when at the top of long table

I have a large table with 3000-4000 rows. Scrolling up/down anywhere except at the top of the table is normal, without lag. But when at the top of the table (either when the page first loads, or when returning to the top), the page/scrolling freezes a few seconds while presumable StickyTableHeaders makes calculations.
See this example: https://jsfiddle.net/x25Lxynq/6/ .
I have tried this in Firefox 36.0.4 & IE 11.
Is there any way/change to speed things up and avoid the lag ?

large table with multi rows and columns ,header freeze and hold for few second when scroll to no floating header

anyone fix it?

Maybe this is related to #61. It seems I have found some issue and implemented some workaround that fixes 90 % of the issue in my case.
Observations and assumptions:

Firefox seems to render the page more synchronously than Chrome. In both browser, there is some relatively heavy computation (you can see hundreds of milliseconds in profiler), but Chrome does not wait for the JS. Firefox results in significantly laggy scrolling, while Chrome results in sticky header being shown after some delay. Of course, the issue is much more visible in Firefox, but this does not mean that the issue is non-existent in Chrome.
When I scroll, the toggleHeaders function is called. Some more handlers might be called in some cases (see bind function), but not in my case.
When scrolling above the table (i.e. the table header is below the top of the visible part of document) and having fixedOffset: 0 and scrollableArea: window, there are few candidates for heavy computations in the handles (other computations are trivial or unreachable in this case):

$this.offset()
base.$scrollableArea.scrollTop()
base.$scrollableArea.scrollLeft()
$this.height()
base.$clonedHeader.height()

All of these computations are also performed when scrolling within the table. But when scrolling within the table, the scrolling is smooth. So, some of these computations seems to be more complex when scrolling above the table than when scrolling within the table. My candidate for that is base.$clonedHeader.height(), because it is hidden when scrolling above the table, so the computation is probably not so trivial. (Maybe the element is not rendered, so the .height() call causes rendering every time it is called.)
The notScrolledPastBottom is sometimes evaluated needlessly. That is, when scrolledPastTop is false, then there is no need to evaluate notScrolledPastBottom, but it is evaluated anyway.

Possible solutions:
a. Do not evaluate notScrolledPastBottom if not needed. This approach is very simple to implement, as notScrolledPastBottom value is used only once. Implementing this removed the lags when scrolling above the table. However, when visibility of sticky header is to be changed, it is still laggy. Moreover, it is likely to be laggy when scrolled below the table (i.e. nothing from the table is visible and the table is above the visible part of the document).
b. Compute the height from $originalHeader rather from $clonedHeader. This should make the computation on a visible element and solve the issue. I am, however, unsure if there is some reason why $clonedHeader is used over $originalHeader. It seems to be OK in my case, but there might be some edge case. Maybe the author of the library can comment that.
Note that the issue will probably appear again when the table is hidden or in some similar edge case. But I am not sure if it is worth bothering with it.
Results
I haven't done deep testing, but any of there two approaches seems to do a good job, at least on my page with Iceweasel (i.e. Firefox branded by Debian) version 41.0. They also seem to do a good job when combined together. There is still some lag when the visibility of sticky header is toggled, but it is much better than it used to be. You can compare my current version of the page to some previous one. The previous one uses StickyTableHeaders 0.1.17. The current one uses my patched version (with both patches).

I'd like to remove the last lag – when the cloned table header is shown or hidden.
a. Use native CSS DOM instead of jQuery – this does not seem to help much.
b. Never use display: none for the cloned column. I've tried several solutions, none of them worked:

position:fixed; top: -999px caused miscalculation of column sizes, as it is computed when detached
height: 0px; overflow: hidden seems to be simply ignored in thead (at least in Firefox)

c. Calculate the header widths in updateWidth on the original header — causes some miscalculations, I don't know why. Moreover, it does not even seem to remove the lag, so there might be also another problem.
d. Use base.$originalHeader.width() instead of base.$clonedHeader.width() in updateWidth — does not help much.

For Gecko and Trident, it seems that queuing some updates adds smoothness. For Blink, it seems that synchronous (current) approach is better, because the renderer is more async.
There is the patch: https://github.com/v6ak/zbdb-stats/commit/542ae80913336f5e2625ff271fca0715f6e0d3be


Fix resizeMode="cover" when image is resized

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Since RCTResizeModeCover is implemented manually on macOS (iOS benefits from UIViewContentModeScaleAspectFill API), we need to reprocess the image any time the bounds of the view update. The updateImage: method was already called on resize, so this instead stores the original image in an ivar so it can call RCTFillImagePreservingAspectRatio from updateImage: instead.
Changelog
[macOS] [Fixed] - Fix resizeMode="cover" when image is resized
Test Plan
Before
Look at the images labeled 'Cover' and observe how they don't properly resize
https://user-images.githubusercontent.com/484044/180457216-88a05f6f-3aef-40a6-af45-8e3e2f4feef7.mov
After
https://user-images.githubusercontent.com/484044/180457246-2b09e140-1dfe-489c-8d43-56bff5e53740.mov
Before
Another demo, done internally
https://user-images.githubusercontent.com/484044/180457315-ea46deba-a9a1-4719-bbc1-7e495a19792e.mov
After
https://user-images.githubusercontent.com/484044/180457545-78eb1ab3-2401-48e6-b038-a225fee71c33.mov

@mischreiber > Does this change look good to you?

"Needs attention" label here means that we (Microsoft, aka @mischreiber) will take a deeper look. No changes requested from the PR author yet.

Anything needed from our side to get this change landed?

Anything needed from our side to get this change landed?

@mischrieber had wanted to take a deeper look, but is also currently OOF. Is this is blocking other changes from making their way through, we can prioritize this one over others. Come to think of it, it probably takes priority by default on account of being "older" than the others..


[Numpy] Add qr backward part 2 for wide matrices with m < n

Description
This is the 2nd part of the QR backward implementation. The 1st part (merged) covered square and deep matrix shapes (nrows >= ncols) and part 2 now covers the remaining wide matrix shapes (ncols > nrows).
References:
Differential Programming Tensor Networks
The added test includes a numerical check (via central differences) of the analytical gradient  since this is a novel implementation. The tests were run offline 1K times to insure against flakiness.
Checklist

[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
[x] Code is well-documented:
[x] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] QR back part 2
[x] Tests

@haojin2 PR ready for review, tnx.

@D-Roberts Thanks for your contribution! From now and on please @yzhliu for reviews of NumPy-related contributions since I'm moving my gravity from this project.

Hi @yzhliu, can you take a look? thanks

The code follows the idea in the reference Differential Programming Tensor Networks. At high level, partition/split the input A into 2 matrices X and Y and R (from A=QR decomposition) into 2 matrices U and V. Then X = QU and get X_grad by applying the gradient derivation from the square input case (m=n) with adjusted Q_grad. Also get Y_grad separately. Then A_grad is the concatenation of X_grad and Y_grad.

@mxnet-bot run ci [centos-cpu, unix-cpu, windows-gpu]

@mxnet-bot run ci [centos-cpu, unix-cpu]

@mxnet-bot run ci [unix-cpu]

@mxnet-bot run ci [unix-cpu]

@mxnet-bot run ci [unix-cpu]

Hi @yzhliu - is there anything else you'd like me to do on this? tnx

Any updates on this?

Hello @yzhliu - are we planning to merge this soon? This particular case of differentiable QR can be useful on batch, in place of LQ, or SVD in recent computer vision research for solving least squares.

This PR broke master CPU pipelines and blocks PRs (test_np_linalg_qr fails), see e.g. http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fcentos-cpu/detail/master/2093/pipeline

@D-Roberts @hzfan could you look into the issue that @ptrendx mentioned? If it can't be fixed in a couple of hours let's revert the change first.

@hzfan Thank you for your prompt assistance, I appreciate it.
@leezu @szha @DickJC123 I will resubmit a separate PR. For my future reference - what are your recommendations to avoid the "stale PR" situation? CI passed when first submitted about 3 months ago and I rebased and CI passed about 2 months ago when the PR was reviewed. All along I followed up on the PR every 2-3 weeks or so.

@D-Roberts we will likely need to automate it so that stale CI checks are invalidated. In the meantime, if the PR sits for a long time, feel free to ping me or any other committer to get more attention on it.

@D-Roberts the recommendation is to comment with "at mxnet-bot run ci [all]" where at is @


Guidance on redirecting Console.Out to PowerShell output stream

This is not an issue or bug, but I'm looking for some official guidance -- if I'm writing a binary module, in general, what would be the proper or recommended way to redirect data sent to a TextWriter to a PowerShell output stream?
Specifically, let's say I've included a 3rd-party library in my PS module that happens to write normally to Console.Out.  If I want all that output to go to PowerShell's Output stream instead so that it can be seen by the environment and the runspace pipeline -- how should I go about this?
The naive approach would be to replace Console.Out via Console.SetOut(...) with my own TextWriter instance, say a StringWriter that buffers all the output and then I can spit it out all out at once via WriteObject.  A more elegant approach would be to implement my own TextWriter whose Write(...) methods do an immediate write out via WriteOutput.
But my intuition tells me there must be a more standard approach to this?

Someone mentioned that @dlwyatt's PowerShellLoggingModule may already be doing something like this, but taking a quick look through the code, it appears this is doing capture and redirection of the Host's output, which is on the "PowerShell side" of things, not in the Console IO of the underlying .NET runtime.

Cc @lzybkr might be the best person to give guidance here

If the 3rd-party code is wrapped in an exe - it might be easiest to run the exe directly to get the output back in a PowerShell friendly way, something like:
var ps = PowerShell.Create(RunspaceMode.CurrentRunspace);
ps.AddCommand("your_command.exe");
var output = ps.Invoke();
foreach (var o in output) {
    WriteObject(o);
}

Otherwise I think your approaches are reasonable options.
It sounds like you are wrapping some 3rd-party code - so you might consider structured output instead of the strings the 3rd-party code is writing. If parsing the text output from that 3rd-party code, then you save the users of your command the trouble of parsing the text (possibly incorrectly) themselves.

Thanks!  I'll go with that.


Fix html not rendering markdown link properly

Related to #3408

Yay!

Thanks. Personally, I think we need to look into docs generator, rather than mixing HTML with/into markdown. I'm not entirely sure how the docs are generated.
if you meant to address the issues on the Docker run docs, though, you might wanna open a PR here. There is a compose file that lets you render the pages etc.

So I had a look at this again I wonder if the problem is that the original file had a markdown link mixed into HTML and thus failed to be rendered by GH properly. I might be wrong, though so I'll let others comment here.

Codecov Report

Merging #3410 (fefc0e2) into main (a9d8a05) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3410   +/-   ##
=======================================
  Coverage   56.30%   56.30%
=======================================
  Files         102      102
  Lines        7244     7244
=======================================
  Hits         4079     4079
  Misses       2524     2524
  Partials      641      641

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 a9d8a05...fefc0e2. Read the comment docs.

So I had a look at this again I wonder if the problem is that the original file had a markdown link mixed into HTML and thus failed to be rendered by GH properly. I might be wrong, though so I'll let others comment here.

What do you mean? Can you point out where this is happening, I might be able to help :)

The weird thing is that the preview pane shows it right, for some reason.


Add configmap from pillar data to dex ldap connectors

Goes together with changes from https://github.com/kubic-project/velum/pull/585

Changed to using yaml_dquote to escape attributes in all cases. Added a comment in empty if block to make it clear why it exists and also to show the comment in the output configuration file itself for clarity to anyone reviewing that file.


Foreign key field displaying ID only and not the field name in rails_admin

I have 4 models in my rails application, HQ, Area, Employee, Dealer and all are associated to each other as follows:-
Area belongs_to HQ, HQ has_many areas Employee belongs_to Area, Area has_many Employees Dealer belongs_to Area, Area has_many Dealers
I am using rails_admin to manage all the data. When I create a new Area and select its relevant HQ, the HQ name is displayed in the dropdown list as per the belongs_to association, but when I try to create new Employee or Dealer, the dropdown list for selecting the area shows only the Area ID like Area #1, Area #2 etc., and not its name like A,B,C. Also, when I click on a particular Area name to view its details in rails_admin, the header displays the text, Displaying details for Area "Area #1".
Please guide me, how to fix this issue.

I know this is an old question, but I was running into the same problem and solved it. Figured folks may run into this again in the future.
The foreign key table must have a name column. If there is no name just the ID # will be displayed. Not sure if other fields like title would work, but I know for sure fields like first_name or last_name do NOT work.
Hope that helps someone else out in the future. ⛑


Updates specific to Indonesian Journal of Southeast Asian Studies

[ ] Create page
[ ] Orange area (or whatever alternative is decided upon)
[ ] Add papers
[ ] Share on socmed

Paper is under review


Conflict with other mods during a world unload, potential re-order issue?

Overview
Hi there! I'm using Infiniverse for the provisioning of dynamic worlds, and these worlds often contain modded data from other mods, this example shows Integrated Dynamics blocks.
We've had issues recently where the server will crash when a world is unloaded, and I've attached a pastebin of the issue: https://pastebin.com/qGcQp6u5
Now particularly, what I expect is happening is that:

I request the world to be marked for unregistration at the next tick.
At the next tick, the world is then unregistered.
During this same tick, other mods are trying to tick their blocks, but the world is no longer loaded; and as this is unhandled, it crashes the server.

I looked at the code, where the unregistration is fairly straight forward, handled at the end of a tick.
@SubscribeEvent
		public static void onServerTick(final ServerTickEvent event)
		{
			if (event.phase == TickEvent.Phase.END)
			{
				MinecraftServer server = ServerLifecycleHooks.getCurrentServer();
				if (server != null)
				{
					DimensionManager.INSTANCE.unregisterScheduledDimensions(server);
				}
			}

Would this be solvable by having the priority set on this event to run last, after all other event listeners have fired and all other tickable blocks have finished ticking?
Interested in hearing your thoughts!

Hi, I am using the Code as well.  We ran into crashes with semi-loaded Chunks that are kept even after unloading the dimension. See https://github.com/AppliedEnergistics/Applied-Energistics-2/issues/7621.
In this Issue I linked a crash report, where you can see that for example AE2 causes a crash since it invalidated the network after the Unload Event and a Forge Energy Provider(Flux Networks, Pipez, ...) then ticks in the next cycle even tough the World got unloaded.
This is caused by Mods using the getBlockState or setChanged Functions since these create 1-tick chunk tickets. So to completely fix it we need to block ChunkTickets of the dimension that gets unloaded.


AEROGEAR-2096 Rebase mobile-metrics branch on master

Before going deep with other work let's rebase and merge first!
Did nothing fancy here. Just rebased mobile-metrics on master.
Wanted to do it separately as it would show lots of unrelated stuff in my PRs.

cc @josemigallas

@david-martin  could you give me write access on this repo? I can't merge

@aliok I added this repo to the team. Should be good for next time


Test: Add a test to prevent any unexpected changes to function signatures

Summary of the changes / Why this improves CrateDB
It would be good to add such test to prevent any mistakes from https://github.com/crate/crate/pull/16133.
Checklist

[ ] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

TODO: Looks like LAZY_ATTRIBUTES can be removed. Since removing it will affect the ordinals of Scalar.Feature, I think it make sense to do it after https://github.com/crate/crate/pull/16133 which removes NULLABLE and NON_NULLABLE.

Thank you, once https://github.com/crate/crate/pull/16133 and https://github.com/crate/crate/pull/16174 are merged this new test will have less value as the PRs try to make it less error prone in tagging the functions with appropriate features. And comes with the downsides mentioned. I will only use them locally for developing the PRs. Closing.


Overview information host port mapping incorrect

Perhaps I've missed something but should this in the example:
docker run -d -p 8000:8000 --name my-running-site iis-site
Not instead be:
docker run -d -p 8000:80 --name my-running-site iis-site
Since the only exposed port in the Dockerfile is 80, mapping to the container on 8000 won't work.

Agree, I test and find the same!

0caffd519f01c876d7e4d9e937f700564b2f4115


Discussion: Add a new category called "Provider Development"

I have just seen, that you support already the category "Writing Custom Providers".
Me as author of PseudoDynamic.Terraform.Toolset would love to see a category called "Provider Development" where we, the provider library authors, can be represented.
I would like to propose the following change:
...
Providers
...
Provider Development

terraform-plugin-go - A low-level Go binding for the Terraform protocol for integrations to be built on top of.
Terraform Plugin SDK - Terraform Plugin SDK enables building plugins (providers) to manage any service providers or custom in-house solutions.
Terraform Plugin Framework - A next-generation framework for building Terraform providers.
PseudoDynamic.Terraform.Plugin.Sdk - Implement Terraform provider plugins, including resources and data sources, in best C# manner.

Testing
...
May I submit a pull request? 🙂

Yeah, I think it would be a good idea, I am supporting it. @teneko

close this for now due to inactivity, feel free to create a new one.


gccgo/gollvm: incorrect finalization of ambiguous promoted methods

If a type that is stored directly in an interface data word includes ambiguous promoted methods, finalization of its methods results in error: incompatible type for receiver (cannot use type <A> as type <B>).
What version of Go are you using (go version)?

$ go version
go version unknown linux/amd64
llvm-project/llvm/tools/gollvm$ git log --oneline | head -n 1
0e34e09 gollvm: add /usr/{lib,lib32} to toolchain paths for arch linux
llvm-project/llvm/tools/gollvm/gofrontend$ git log --oneline | head -n 1
6a33e7e3 compiler: remove Array_index_expression::is_lvalue_

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
The issue may be reproduced with the following sample:
foo/foo.go
package foo

import "bar"

type Foo struct {
        *bar.Bar
}

func (f *Foo) Personal(a int) error {
        return nil
}

bar/bar.go
package bar

import "baz"

type Bar struct {
        baz.Baz1
        baz.Baz2
}

baz/baz1.go
package baz

type Baz1 int32

func (b Baz1) Promoted() int {
        return 1
}

baz/baz2.go
package baz

type Baz2 int32

func (b Baz2) Promoted() int {
        return 2
}

What did you expect to see?
No error, successful build.
What did you see instead?
An attempt to build this sample results in:
$ GO111MODULE=off go build foo
# foo
foo/foo.go:5:6: error: incompatible type for receiver (cannot use type Foo as type baz.Baz1)

I'll submit a fix for this issue soon.
cc @ianlancetaylor


Misc updates to Go Hello World tutorials

This PR includes the following changes:

Corrected some typos/grammar errors.
Removed erroneous "Beta support" notes.
Made crdb library install consistent across examples.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bb9a90d48da0ac02b31836ec100ccdd7bab04391/
Edited pages:

v19.2/build-a-go-app-with-cockroachdb-gorm.md
v19.2/build-a-go-app-with-cockroachdb.md
v20.1/build-a-go-app-with-cockroachdb-gorm.md
v20.1/build-a-go-app-with-cockroachdb.md
v20.2/build-a-go-app-with-cockroachdb-gorm.md
v20.2/build-a-go-app-with-cockroachdb.md

TFTR!


Can use my qnap as exit node but can't acces to nas data

Hi,
The exit node is working and well configured on tailscale website. Some i'm able to do :
Android-phone > NAS > Internet via exit node.
But i can't access NAS files from android phone when i try to go to my nas webinterface with :
https://Nastailscaleip:4443
or when i try to connect with android qnap app (qfiles etc...) to my nas and when using nas tailscaleip from tailscaleapp
All is working well when i'm the same lan network with wifi without tailscale.
I launched tailscale with the following command:
./tailscale -socket var/run/tailscale/tailscaled.sock up --advertise-exit-node
Edit : it is the qnap firewall, when off it's working... But can't manage to set the firewall well , if i autorise tailscale ip of my phone same issue

Thanks, @farika.
I haven't tested with the QNAP firewall. Have you tried restricting the firewall gradually to see when the problems starts? For example:

no firewall
firewall enabled but allowing all incoming IPs to all services
firewall enabled and allowing local network and  <IP_ADDRESS>/8 to all services
firewall enabled and allowing local network and your phone tailscale ip to all services
firewall enabled and allowing local network and your phone tailscale ip to file sharing services

My personal QNAP is far away and I do not want risking testing the firewall configuration :)


net/rpc: method receiver don't need to be marshaled by encoding/gob

Hey!
In the documentation of net/rpc, it is said that the method receiver T needs to be marshalable with encoding/gob. From my understanding, this is not the case, only the two arguments need to be.

/cc @robpike


facets

Track enabled plugins with Codemirror State
Track plugin settings with CM State
Convert pillbox-menus to a CM ViewPlugin
Migrate performance-view and video-creator to cm
Migrate builtin-settings to CM
Migrate bunch of plugins to CM
Migrate expr-action-buttons (and friends) to CM
Migrate Text Mode to CM
Migrate metadata to CM
Rename cmPlugins to plugins

oops


Bluetooth: Samples: Add hci_uart_async

This sample is an alternative implementation of hci_uart. The new sample differs from the existing sample in that it uses the async UART API instead of the interrupt driven API.
Included in this commit is a new test for HCI UART flow control. It's enabled for hci_uart_async. The test can also excercise the existing hci_uart sample (with some minimal changes to allow compiling in the mock controller and test suite). The existing hci_uart sample currently fails this flow control test.

Almost forgot... could you add to serial_test.c the same as #60396?

Done. But wouldn't it be better to have that enforced centrally in uart_irq_callback_user_data_set and uart_callback_set? I.e. Both of those functions would call the other with NULL (to clear the other callback) if it itself did not get NULL (to clear the callback).

Almost forgot... could you add to serial_test.c the same as #60396?

Done. But wouldn't it be better to have that enforced centrally in uart_irq_callback_user_data_set and uart_callback_set? I.e. Both of those functions would call the other with NULL (to clear the other callback) if it itself did not get NULL (to clear the callback).

That is done to allow more flexibility on the driver side. Some drivers may decide to disable certain interrupts when there is no need to do callback, which might interfere with the other API. The driver would know what needs to be done correctly, and that the current implementation let the driver decide.

Oh... you add the callback changes to the bluetooth sample commit. Could you move those to the commit which changes serial_test.c?

Oh... you add the callback changes to the bluetooth sample commit. Could you move those to the commit which changes serial_test.c?

Done. Please re-review :)

DNM until @nordic-krch has had a chance to review this please


Sending buffer yields inconsistent results

I have a buffer for a png file.
Sending the buffer as a GET response produces an inconsistent result.
For the same buffer (which starts with <Buffer 89 50 4e)
sometimes I get a file with the first bytes as hexadecimal:
89 50 4e 47 0d which is correct for a png file.
and sometimes I get a file which begins with:
ef bf bd 50 4e which is incorrect.
All of the following cases have the same behavior (work sporadically):

res.end(fileBuffer);

 res.end(fileBuffer, 'binary');

res.end(null, 'binary');```

Some other facts:
The buffer is not the problem since I tested saving it in the file system, and it is always a successful operation.
I've tried several other alternatives for sending the image with no success; like having a base64 string for the buffer and sending it with base64 encoding. But for this issue I keep the problem in the buffer case. And the problem occurs also for other kind of files, but again I keep it in the png case.
For some png files the problem never occurs. For other png files I have the inconsistent results problem.

Hi! There is likely nothing we can do without at least you providing code we can run to reproduce the issue. Just trying to write a simple app that sends that buffer always results in consistent correct responses.
So, in order for us to proceed, please either issue us a pull request that fixes the issue or you need to provide us enough information to reproduce, at least the following:

Version of Express
Version of Node.js
A complete application file that reproduces the issue we can run.
Complete instructions for how to call the app to see the issue.

And, just to help, the methods res.end and res.write are not implemented in Express, as those are just the methods that are part of Node.js core. If you are having issues with those methods, you will likely need to file a bug against Node.js, which is where those methods come from.

Thank you for your quick answer.
The exact same behavior happens with res.send(fileBuffer);.
I'll be uploading the info you request.
Thank you again.

No problem! Yea, though res.send just calls res.end, so if you have an issue there, it of course will also be in res.send, which we still won't be able to fix. I'll await all the info from you soon :)

In isolating the issue in a simple application, I encountered the problem does not occur.
Of course, it is not the same exact scenario as the original application with the problem.
So, please don't close it until I gather some more information about the issue.
Thank you for your kind help.

I identified where the problem is.
For development purposes I use grunt-connect-proxy as a proxy server in front of my node server. There it is the mess-up.
Thank you for your time.

Ah, gotcha. Thanks for the follow up! I'm sure someone will run into this same issue at some point and find the answer helpful :)


Request for support of German language in prompt injection detection model

Hello,
We have encountered an issue with the LLM-Guard scanner where it falsely detects German prompts as injections. Our use case requires handling German language inputs accurately. Currently, the model ProtectAI/deberta-v3-base-prompt-injection-v2 does not support German, leading to false positives.
Example:
[warning] Detected prompt injection injection_score=0.99
[info] Scanned prompt elapsed_time_seconds=0.025865 scores={'PromptInjection': 1.0}
[scan_prompt] ERROR Prompt 'wo kann ich mich gegen info impfen lassen' is not valid, scores: {'PromptInjection': 1.0}

Details:

LLM-Guard scanner: Prompt Injection - LLM Guard
Current model: ProtectAI/deberta-v3-base-prompt-injection-v2
Alternative model: deepset/deberta-v3-base-injection (seems like supports German language)

Configuration:
We currently cannot configure the language of the prompt directly:
input_scanners = [
    PromptInjection(threshold=settings.inp_prompt_inj_thres, match_type=MatchType.SENTENCE, use_onnx=True),
    TokenLimit(limit=settings.prompt_token_limit),
    Toxicity(threshold=settings.inp_toxic_thres, use_onnx=True),
    InvisibleText(),
]

Request:
Is it possible to either:

Use another model that supports the German language, such as deepset/deberta-v3-base-injection, or
Provide a configuration option to specify the language for the prompt injection scanner.

Thank you for your support!

I have replaced the current model with deepset/deberta-v3-base-injection, but for now with the regular version which is not ONNX. Let's see at the results during testing.

Hey @ulan-yisaev , thanks for reaching out.
Our model doesn't support non-English prompts, unfortunately. Even "deepset/deberta-v3-base-injection", although trained on some German prompts won't do it well because it is trained on top of deberta-v3-base which is an English model.
On a side note, ONNX version is the same as the normal version. The difference is only how it executes to optimize for inference.


{AppService} Delay importing is_valid_resource_id, parse_resource_id

Description
from msrestazure.tools import is_valid_resource_id, parse_resource_id is slow and can slow down the whole Azure CLI!

Delay importing them and import then only when absolutely necessary.
Also migrate to azure.mgmt.core at the same time.

Closing as this usage of top-level import appears in many modules. This won't be a problem when https://github.com/Azure/azure-cli/pull/13294 is merged.


template is throwing exception with array field

I am trying to implement multi forms in different tabs. without template it is working fine. but when i am defining template it es throwing exception

./DynamicFormBootstrapComponent class DynamicFormBootstrapComponent - inline template:0:1634 caused by: Expression has changed after it was checked. Previous value: 'undefined'. Current value: '[object Object]'.
Plunker of my usecase.
Comment template to run this example without any error

@nomi-ramzan Ok, this is a bit tricky. You can do the following to make it work:
export class NgbdTabsetSelectbyid implements OnInit {
    tabs: any[] = [
        {title: 'tab 1'}
        , {title: 'tab 2'}
    ]

    constructor(private formService: DynamicFormService, private changeDetectorRef: ChangeDetectorRef) {}

    ngOnInit() {
        for (var tab of this.tabs) {
            tab.formModel = [
                new DynamicCheckboxModel({id: "checkbox", label: "I do agree"}),
                new DynamicSelectModel({
                    id: "select",
                    label: "Example Select",
                    value: "option-3"
                    ,
                    options: [{label: "Option 1", value: "option-1",}, {
                        label: "Option 2",
                        value: "option-2"
                    }, {label: "Option 3", value: "option-3"}]
                }),
                new DynamicFormArrayModel({
                    id: "myFormArray",
                    initialCount: 5,
                    createGroup: () => {
                        return [new DynamicInputModel({id: "formArrayInput", label: "Form Array Input"})];
                    }
                })
            ];
            tab.formGroup = this.formService.createFormGroup(tab.formModel);
        }

        this.changeDetectorRef.detectChanges();
    }
}

this.changeDetectorRef.detectChanges();

Hi @udos86 thank you for your quick response. I tried this solution, also added in plunker but i am getting this.changeDetectorRef undefined in ngOnInit() function.

@nomi-ramzan As shown above you need to import it from @angular/core and inject it into the constructor as a dependency!
import { ChangeDetectorRef } from "@angular/core"
constructor(private formService: DynamicFormService, private changeDetectorRef: ChangeDetectorRef) {}

I also tried your solution even i called

this.changeDetectorRef.detectChanges();
on tab change event. Still making problem.
You can check my plunker to see the exact problem when you will change tab specially.
I also post question  on stackoverflow.

@nomi-ramzan It perfectly solves your original exception concerning change detection.
Your functions to add and remove items do not work because your expecting the wrong parameters:
<template modelId="myFormArray" let-context="context" let-index="index">
    <button type="button" (click)="removeItem(context, index)">Remove Item</button>
    <button type="button" (click)="addItem(context, index + 1)">Add Item</button>
</template>

context is a reference to the DynamicFormArrayGroupModel not to the tab.
  addItem(tab:any, context:any){
    this.formService.insertFormArrayGroup(0, tab.fGroup.get(context.id), context);
    // this.formService.addFormArrayGroup(this.arrayControl, this.arrayModel);
  }

@nomi-ramzan Ok, sorry, you're right. The problem's still there. I'll do some further investigation.

The problem doesn't appear to have anything to do with tabsets because I get the problem with a simple FormGroup.  It seems to be any sort of template used with an array of fields...
let fields: Array<DynamicFormControlModel> = [
      new DynamicFormGroupModel({
        id: "email",
        group: [
          new DynamicInputModel({
            id: "emailAddress",
            label: "Email",
            required: true,
           errorMessages: {
              required: "{{label}} is required.",
            }
          }),
          new DynamicInputModel({
            id: "emailConfirm",
            label: "Please confirm email address",
            required: true,
            errorMessages: {
              required: "{{label}} is required."
            }
          })
        ]
      })
];

I then use a simple template straight from the documentation...
```

```
I then receive the following...
error_handler.js:50 EXCEPTION: Error in ./DynamicFormBootstrapComponent class DynamicFormBootstrapComponent - inline template:0:5447 caused by: Expression has changed after it was checked. Previous value: 'undefined'. Current value: '[object Object]'.

@thardy-pd I cannot verify this as this is exactly the same I'm doing in the bootstrap example and it's working fine there.

Internally templates need to be assigned in ngAfterViewInit to support libraries' custom template directives.
The error you describe is thrown only in DevMode when a binding has changed after initialization while Angular's change detection has not been triggered. It is a very well known issue in the meantime.
However, in all my examples and tests I could not detect any problems so far.

Hi
Is there any way to define modelId dynamically ? like in your Example you defined static modelId bootstrapFormArray

@udos86  can you provide the example of  multi forms in different tabs.

@Ahsanmughal Just check the Plunker linked above by @nomi-ramzan . It seems to be working fine now.


update can-i-use

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions.

Description of the Change

Alternate Designs

Why should this be in core?

Benefits

Possible Drawbacks

Applicable issues

coverage: 94.331%. remained the same when pulling cb345cdc1f72556ae73e8d4ea37a4d86a1e2aa3e on update-can-i-use into f8f9fd57740f964df6ee31fd6514a179fce453e4 on master.


OT bug?

With 2 pads and about 6 clients, focus at least sometimes garbles text after edits from multiple clients. As a result, better test coverage is needed.

Better test coverage is now provided via TestRandom() in server_test.go and some improvements have been made as a result.
However, TestRandom is both concurrent and randomized and so turns up a mixture of problems, some of which are simply test defects. (For example, TestRandom's test client currently has a race condition wherein if some clients OPEN after other clients have already sent WRITEs then TestRandom's strand length calculation will be off, resulting in either timeouts or hangs depending on whether or not ReadDeadlines are set on the relevant websockets.)

The simple TestRandom bugs described above have been fixed but the test still occasionally fails. However, hopefully the new sequence diagram renderer will reveal the nature of the problem.

So, as it turns out, transform does not actually commute with compose and must, therefore, be executed individually for every small step rather than once at the end. However, once we do this, TestRandom passes, which is good enough to close this bug. (However, this said, real-world testing suggests that at least one OT/CC bug may remain since I managed to hit the panic("unreachable") in the current transform code during a live demo).


add pause/resume recording to MRecordRTC

Thanks @muaz-khan. Could you also make a new release so we can properly start using it?


DOMException: Blocked a frame with origin "" from accessing a cross-origin frame

Hi everyone,
We've recently installed a new instance of Kentico SP 12.0.48 in production. We have 2 sites (one MVC Site and one Admin site on 2 different domains).
In the page builder, when we try to use the "Rich Text Editor" widget, we've faced CORS exceptions. We could partially manage it by using the "CMSUrlHost" settings by now we are stuck with the following exception:

Uncaught DOMException: Blocked a frame with origin "" from accessing a cross-origin frame.

Has someone already face that problem? Do you have any idea how to solve it?
P.S. we try the X-Frame-Options which is no more supported by modern browser

Hi Julien,
Are the other page builder features working in your project (widgets, inline editors, etc..)?
Does the MVC site & Admin site share the same domain or are you using separate domains?
Is the Kentico running behind a proxy server?
After applying the 12.0.48 Hotfix, have you upgraded your MVC Kentico NuGet packages to version 12.0.48 as well?

Hi Ladislav,
Our custom widgets work well but the two of this repo are not:

video: print us "registration of ... failed"
richt text editor: do not load at all

As said, we have two separate domains: www.domain.net and admin.domain.net.
And yes, we have upgraded both Kentico and nugets up to the version 12,0,48 (there is no warning in KIM ;) )

Okay then let's start with the video widget:
Is there any error in browser console or in the event log (Event log application)?
and regarding Rich Text Editor:
What is the URL of the page in an iframe which is browser refusing to load (due to cross-origin request)? What are the response headers of this page? (You can omit sensitive data).

Can we focus on the RTE? :) it is our main blocking point for the moment.
The main URL is:
https://admin.domain.net/Admin/cmsadministration.aspx#95a82f36-9c40-45f0-86f1-39aa44db9a77
The URL of the iframe (from the page builder) is the following:
https://www.domain.net/cmsctx/pv/administrator/culture/en-US/wg/20f0bfe0-462f-4793-940e-71989941c9a4/readonly/0/pts/637106468108611823/ea/1/h/61ee323ae5c2bb3a9d1460a6f17d1502bde5f0de297850127d826c01bb048c5c/-/en-us/test?instance=3da385e4-efec-4e8a-9475-f7c2416bb3e3&uh=5bae2c2d8f5b564970ef23377a5af0c8704cac02a37874058a2b15bb29263345&editmode=1&administrationUrl=https%3A%2F%2Fadmin.domain.net%2F
The request is the following:
https://www.domain.net/cmsctx/pv/administrator/culture/en-US/wg/20f0bfe0-462f-4793-940e-71989941c9a4/readonly/0/pts/637106471144908727/ea/1/h/57f569bd4f56f83c3e16b4f7ccfa4136171260fe1bf4a63010ecd43b78b203fe/-/kentico/bundles/pageComponents/admin/scripts?v=xGajw9hCqtSh5kJbx-7zCvpAAO9Pl8S5lzKFQx4bfpc1&uh=a8d4eb88149127c958140e253d782827fbc4d201acbcdb8d1717bf3de4c89d46&administrationurl=https%3A%2F%2Fadmin-qa.upu.int%2F
The response contains all the scripts for the widgets and in this script, the exception is thrown by the froala editor at this place:

main.js l. 27161 p. 59854
[...]return window.parent.document.querySelector[...]

Looks like this call was added in Froala 3.0.6, could you try downgrading the froala-editor npm dependency to "froala-editor": "3.0.5" (without ^) in package.json, install the packages and build the scripts with this downgraded version of the froala editor?

Also try locating the request to your test page in DevTools and make sure the Content-Security-Policy header is set to frame-ancestors with your domains listed

Ok we will try to downgrade the editor.
Otherwise, the content-security-policy looks good:

Hello,
The downgrade solves the problem with the blocked frame but now there is another one :(. It seems that the licence is not found in the bundle:

If I look into the network exchange I can see that the editor tries to check the licence without success:

Futher investigation, we just found out that the licence is not included in the response when getting the instance of the RTE:

Response:

<div class="ktc-rich-text-wrapper" data-get-link-metadata-endpoint-url="/cmsctx/pv/administrator/culture/en-US/wg/50f5c4c4-8080-485c-a84d-9defa32e4508/readonly/0/pts/637109217741452983/ea/1/h/dc5eacbe366cb17660c7158f9064cd81e4770b9fef924fc475658483853b443a/-/kentico.pagebuilder/widgets/en-us/kentico.widget.richtext/getlinkmetadata?editmode=1&instance=b8764103-81d4-4d35-98b3-4c2da382562b&uh=1ac7c03e04e6b1f2d6493e4e88374e2476e0df1886b749fc7e2dd1a9861e4d64&administrationurl=https%3A%2F%2Fadmin.domain.net%2F" data-rich-text-editor-license="">

The faulting part is the:
data-rich-text-editor-license=""
Is there special condition to retrieve the licence key?

Futher investigation, we just found out that the licence is not included in the response...

The Froala license comes with 12.0.43 Hotfix. Could you please run this query against your database?
SELECT [KeyName], [KeyValue]
  FROM [CMS_SettingsKey]
  WHERE [KeyName] = 'CMSHotfixVersion' OR [KeyName] = 'CMSHotfixDataVersion'

Could you check if both values are greater or equal to 43 in both environments?

Thank for the hint, the entry was missing so it solves the second problem :)

No problem, is there any other problem regarding the Rich Text Editor or Video widget?

No not with the downgraded version of Froala. But the "DOMException: Blocked a frame with origin" remains with current version ... wait'n'see for the next version

Hi @phischepi,
We have recently released a new version of the RichText Editor. Could you install it and check if you can still reproduce this issue?

Install-Package Kentico.EMS12.MvcComponents.Widget.RichText -Version 1.2.0

Hi @phischepi,
We haven't heard from you in a while, so we are closing this issue for now. Feel free to reopen this issue if the answer above won't help you.


WebApp deployment to Windows docker

Hi, I'm missing deployment on WebApp Windows Container. Please when will be added ?

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

ID: 593c3abc-7b38-6673-62b6-7787d0dda4dd
Version Independent ID: cbb2b875-b220-596c-7b4d-c2cf6c644335
Content: Deploy a Docker container app - Azure Pipelines
Content Source: docs/pipelines/apps/cd/deploy-docker-webapp.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

Thanks for your question. It looks like your question isn't an issue with the documentation, but an issue or question specific to your implementation or a bug in the product itself.
For help with a specific task, here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

If you've encountered an issue with the product itself, to make sure it gets in front of the right people, please submit your bug here.
If you have an idea for improving the product, please submit your idea here.


Add some explanation to the lockdown mode checkbox

Adds a wordy subtitle about lockdown mode by the checkbox.

"Read more" leads here: https://keybase.io/docs/lockdown/index
cc @keybase/y2ksquad

lgtm!


HttpResponseExceptionMapper minnsing escaping in case of json response

https://github.com/tipsy/javalin/blob/640dc2f84523009dfe195f599281c41b423d82fa/src/main/java/io/javalin/http/HttpResponseExceptionMapper.kt#L20
If exception message contains double quotes then HttpResponseExceptionMapper produce invalid json.

Aaah, that's what I get for writing my own json code. Would you like to submit a PR?

Released now in 3.0.0.RC1:
<dependency>
  <groupId>io.javalin</groupId>
  <artifactId>javalin</artifactId>
  <version>3.0.0.RC1</version>
</dependency>


Don't need json tags if keeping identical names

This is the coding style.

No it's not. It's invalid Go code that would be rejected by any official Go project.


Side effect of count() on a NORMAL VIEW of a MergeTree table engine

Hi, this is possibly a functional bug and here is a test to reproduce it
CREATE TABLE NUM
(
    id UInt8,
    n UInt8
)
ENGINE = MergeTree()
PARTITION BY id
ORDER BY (n, id)

INSERT INTO NUM VALUES (1, 54), (2, 23), (3, 54), (4, 48), (5, 19);

CREATE VIEW VW AS SELECT id, n FROM NUM ORDER BY n;

SELECT *
FROM VW

┌─id─┬──n─┐
│  5 │ 19 │
│  2 │ 23 │
│  4 │ 48 │
│  1 │ 54 │
│  3 │ 54 │
└────┴────┘

So far, so good now try
select count() from VW WHERE n>50;
┌─count()─┐
│       2 │
└─────────┘

SELECT * FROM VW

┌─id─┬──n─┐
│  3 │ 54 │
│  1 │ 54 │
└────┴────┘

The VIEW has been altered because of the count(), I guess this should not happen.
Is that right ?

Can you provide your CH version? I can't reproduce it on recent master at least.

SELECT version()

┌─version()─┐
│ 18.12.17  │
└───────────┘

seems already fixed even for enable_optimize_predicate_expression = 1


datepicker does't disappear

when date select datepicker does't disappear

@omkarpise01 Are you sure you set autoclose option to true? I've tested in demo and it seems to work fine.


Fix type of Message content property

Both the getter and setter imply that the $content property could be anything. I am not 100% sure which one is correct; albeit the PR.

I've reviewed usage in components that extend the class, and they all assume a string, so this is safe.

Thanks, @roelvanduijnhoven!


[WIP] Add Facilities

Add support for Facilities.

[x] Admin UI for CRUD actions
[x] Read-only sync API
[x] Associate with Patients
[ ] Associate with Blood Pressures

The join table for blood pressures would be the same, yes?

We should also have a relationship between facility and protocol. A facility has a protocol?


move python version detection to a cmake module

This patch will move the python version string detection to a cmake module

Reviewing the pull-requests, I just realized pull-request https://github.com/python-cmake-buildsystem/python-cmake-buildsystem/pull/289 implement a similar approach introduced a module called cmake/PythonExtractVersionInfo.cmake. This is why I am closing this pull request.
Note also that I appended a Co-authored-by:  ... message trailer to the relevant commit.

Oh, thanks! I was wondering what happened to this project.


peer dependency mismatch

This package depends on marked v2 and marked-terminal. marked-terminal has marked in its peer dependencies, but wants version ^1.0.0, so v2 is not compatible with that.
Warning with npm 7:
npm WARN ERESOLVE overriding peer dependency
npm WARN Found<EMAIL_ADDRESS>npm WARN node_modules/marked
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/@adonisjs/sink
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN     node_modules/@adonisjs/assembler
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS><2.0.0" from<EMAIL_ADDRESS>npm WARN node_modules/marked-terminal
npm WARN<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN   node_modules/@adonisjs/sink

Lemme check if we can get marked-terminal to use the latest version too

Fixed


Request: Unlimited recommendations

Currently, it seems like Twinder fetches recommendations in batches of 11. The user can go through them, but then when they reach the end, the window closes, and they have to hit "Recommendations" again, then wait for a new fetch to complete.
A nice feature would be for Twinder to fetch new recommendations in the background as they're reaching the end of the batch so that they don't have to close the window and start over or wait for a batch fetch to complete.

AFAIK once you get a batch of recommendations, you can only like/pass those that are in the batch. When you ask Tinder for a new batch, the old ones return you a "Not authorized" error.
Though I can't guarantee this is true. I haven't looked into what API calls the original client makes yet, I plan to do that in the next half of the month.


Add tier labels corresponding to PEP 11

Thanks @brettcannon! :heart:

I merged so we have a baseline for tier support tags, but I noticed there are a bunch of Linux buildbots that are tagged as NO_TIER but they technically qualify as tier 1. Was this an oversight or do you want buildbot owners to manually tag those?

Great :-)

I purposefully overlooked them as PEP 11 specifies the platforms run by CI as tier 1 and purposefully doesn't mention buildbots for that tier (hence why there isn't a single tier 1 buildbot 😁). That way the "Changes which would break the main branch are not allowed to be merged" requirement can be met since buildbots don't run on PRs.
If people think that's too strict of an interpretation and want to add tier 1 buildbots I'm fine with it.


Add account menu (after signing a transaction for the community creation) showing posting key as the same as active key

After finding out more information about what led me to open https://github.com/stoodkev/hive-keychain/issues/94, I have tested the process once more with ecency + keychain and found out that the issue is being caused somewhere before or at the new account (in this case a community account specifically) is being asked to be added to Hive Keychain.

So, I am trying to understand what part of the code deducts those private keys and shows them on screen (it might be at that stage that the posting key is being shown as the same as the active key). I am guessing this could be a simple array index missed out.
The broadcasted transaction for debug purposes is here:
https://hiveblocks.com/tx/4d860c696d4dcd8bd7aa9075636ef4f9cd3a48ab

Not sure if this is related:
https://github.com/stoodkev/hive-keychain/blob/149d8f70865140eeb94f3ec00529791ced0c9ff6/html/popup.html
https://github.com/stoodkev/hive-keychain/blob/149d8f70865140eeb94f3ec00529791ced0c9ff6/html/popup.html#L200

          <input type="checkbox" checked="checked" id="posting_key" />

          <input type="checkbox" checked="true" id="active_key" />

          <input type="checkbox" checked="true" id="memo_key" />

      <button id="save_master"></button>

The keys are directly sent by the frontend that makes the request.
In the example above I'm sending a b and c as posting active and memo keys respectively:
Reference for requestAddAccount :
https://github.com/stoodkev/hive-keychain#requestaddaccount
Perhaps the dApp you used is passing the wrong key

So is it not deducted by the master key? I understood from good-karma that the website was sending only the master password.

I have now contacted good-karma. Let's wait until I can get some confirmation from their side.

Found it!
https://github.com/ecency/ecency-vision/issues/540
Thanks for the verification @stoodkev


Request : DataTemplate example

Hello Kenny,
Can you please upload any control using DataTemplate sample
Thanks & Regards,
Vinoth.

Hi Vinoth, please leave comments at https://github.com/Microsoft/cppwinrt/issues/ - I'm going to shut down this issues page to avoid confusion.


fixed: Chief Olusegun Obasanjo (No. 6)

What does this PR do?

It fixes Chief Olusegun Obasanjo (No. 6) view more page not displaying image and bio
Removed green borderlines showing on some cards (view the image below)

Screenshots (if appropriate)

Issue #85

Working as expected, well done!!!


Error starting service locally

When trying to work with the annon.api locally i get the following error:
{"timestamp":"2018-05-30T18:38:18.078Z","sourceLocation":{"line":null,"function":null,"file":null},"severity":"INFO","resource":null,"jsonPayload":{"metadata":{"error_logger":"std_info"},"message":"Application annon_api exited: Annon.start(:normal, []) returned an error: shutdown: failed to start child: Annon.AutoClustering\n    ** (EXIT) an exception was raised:\n        ** (ArgumentError) argument error\n            (annon_api) Elixir.Annon.AutoClustering.erl:57: anonymous fn/4 in Annon.AutoClustering.do_reload_config/0\n            (annon_api) Elixir.Annon.AutoClustering.erl:34: Annon.AutoClustering.init/1"}}
** (EXIT from #PID<0.73.0>) an exception was raised:
** (Mix.Error) Could not start application annon_api: Annon.start(:normal, []) returned an error: shutdown: failed to start child: Annon.AutoClustering
** (EXIT) an exception was raised:
** (ArgumentError) argument error
(annon_api) Elixir.Annon.AutoClustering.erl:57: anonymous fn/4 in Annon.AutoClustering.do_reload_config/0
(annon_api) Elixir.Annon.AutoClustering.erl:34: Annon.AutoClustering.init/1
(mix) lib/mix.ex:291: Mix.raise/1
(elixir) lib/enum.ex:737: Enum."-each/2-lists^foreach/1-0-"/2
(elixir) lib/enum.ex:737: Enum.each/2
(mix) lib/mix/tasks/app.start.ex:114: Mix.Tasks.App.Start.start/2
(mix) lib/mix/tasks/app.start.ex:86: Mix.Tasks.App.Start.run/1
(mix) lib/mix/task.ex:314: Mix.Task.run_task/3
(mix) lib/mix/tasks/run.ex:119: Mix.Tasks.Run.run/5
(mix) lib/mix/tasks/run.ex:76: Mix.Tasks.Run.run/1
{"timestamp":"2018-05-30T18:38:18.085Z","sourceLocation":{"line":null,"function":null,"file":null},"severity":"ERROR","resource":null,"jsonPayload":{"metadata":{"error_logger":"format"},"message":"Process #PID<0.1072.0> raised an exception\n** (Mix.Error) Could not start application annon_api: Annon.start(:normal, []) returned an error: shutdown: failed to start child: Annon.AutoClustering\n    ** (EXIT) an exception was raised:\n        ** (ArgumentError) argument error\n            (annon_api) Elixir.Annon.AutoClustering.erl:57: anonymous fn/4 in Annon.AutoClustering.do_reload_config/0\n            (annon_api) Elixir.Annon.AutoClustering.erl:34: Annon.AutoClustering.init/1\n    (mix) lib/mix.ex:291: Mix.raise/1\n    (elixir) lib/enum.ex:737: Enum."-each/2-lists^foreach/1-0-"/2\n    (elixir) lib/enum.ex:737: Enum.each/2\n    (mix) lib/mix/tasks/app.start.ex:114: Mix.Tasks.App.Start.start/2\n    (mix) lib/mix/tasks/app.start.ex:86: Mix.Tasks.App.Start.run/1\n    (mix) lib/mix/task.ex:314: Mix.Task.run_task/3\n    (mix) lib/mix/tasks/run.ex:119: Mix.Tasks.Run.run/5\n    (mix) lib/mix/tasks/run.ex:76: Mix.Tasks.Run.run/1"}}
Environment info:

Erlang/OTP 20 [erts-9.3.1] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:10] [hipe] [kernel-poll:false] [dtrace]
Elixir 1.6.5 (compiled with OTP 20)
postgres  - docker 'nebo15/alpine-postgre:9.6.2'
annon-web - docker 'nebo15/annon.web'
i use the .env file that in the project (after modifying the postgres parameters)
MacOS  10.13.4
Thanks

Hello @amitcomo. Can you show the exact Bash command which you used to start the containers? Also, did you override any options in .env regarding nodes discovery?
Looks like it crashed because the distribution layer failed to start (the one which is responsible configuration invalidation/synchronization if you have more than one instance deployed).

I use the following docker-compose.yml to start the app & db containers:

version: '2' services:   app:     image: "nebo15/annon.web"     links:       - "api:api"     depends_on:       - api     environment:       - MANAGEMENT_ENDPOINT=http://api:4001       - PUBLIC_ENDPOINT=http://api:4000     ports:       - "8080:8080"   postgres:     image: "nebo15/alpine-postgre:9.6.2"     ports:       - "5432"     environment:       - POSTGRES_DB=annon       - POSTGRES_USER=postgres       - POSTGRES_PASSWORD=postgres     # volumes:     #   - '.postgres:/var/lib/postgresql/data'

I use intellij plugin to start the api service (the plugin - https://github.com/KronicDeth/intellij-elixir)
nodes discovery - no change here.  I used the .env file with DB host & port change.

Appreciate your help

@amitcomo can you try running Annon with script from annon.infra?
I believe that it crashes because by default, Annon uses Kubernetes for its auto-clustering. If you start the code with IDE plugin, it would start it in dev mode, won't override any defaults and fail. You should either pass environment variables to use different clustering strategy (SKYCLUSTER_STRATEGY=Cluster.Strategy.Epmd) or edit config.ex and replace the default value there with Cluster.Strategy.Epmd.

Thanks Andrew.
I use SKYCLUSTER_STRATEGY=Cluster.Strategy.Epmd (verified it)

I'll leave this issue open to reconsider defaults later.


[kitchen/e2e] update amazon linux 2023 x86_64

What does this PR do?

Update Amazon Linux 2023 x86_64 AMI for kitchen an E2E tests with copies of AMIs ami-0bb84b8ffd87024d8, Amazon Linux 2023 x86_64, targeting runc-1.1.11-1.amzn2023.0.1.src.rpm
Motivation

Previous hosts are affected by https://nvd.nist.gov/vuln/detail/CVE-2024-21626
Incident-27459
Incident-27490
The previous Amazon Linux 2023 x86_64 in use in our e2e tests had a security vulnerability due to the version of runc, installed at docker install within the agent security kitchen tests.
The CVE mentions that the vulnerability is fixed in runc 1.1.12+, but for Amazon Linux 2023 this is fixed in runc-1.1.11-1.amzn2023.0.1+, as mentioned in https://alas.aws.amazon.com/AL2023/ALAS-2024-501.html
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/remove

/merge


[style]: Emoji in chat has a counter

What's the style idea?
We need to remove the counter on the selected emoji since it can not be incremented. kindly assign this to me under hacktoberfest'23 label
Add screenshots
Add screenshots here

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[x] I'm a GSSoC'23 contributor
[X] I want to work on this issue

makes sense. Thank you
On Wed, Oct 18, 2023 at 5:01 PM Narayan soni @.***>
wrote:

What's the style idea?
We need to remove the counter on the selected emoji since it can not be
incremented. kindly assign this to me under hacktoberfest'23 label
Add screenshots
Add screenshots here [image: Screenshot from 2023-10-18 09-33-22]
https://user-images.githubusercontent.com/94549530/276203815-9cbbfb5f-2994-4942-9aab-7f9ecfb33ba3.png
Code of Conduct

I agree to follow this project's Code of Conduct
I have read the Contributing Guidelines
https://github.com/narayan954/dummygram/blob/master/CONTRIBUTING.md
I agree to follow this project's Code of Conduct
https://github.com/narayan954/dummygram/blob/master/CODE_OF_CONDUCT.md
I'm a GSSoC'23 contributor
I want to work on this issue

hi @0xNunana https://github.com/0xNunana actually we'd be working to
enable incrementing on it as others add their reaction, that's why it's
there.
—
Reply to this email directly, view it on GitHub
https://github.com/narayan954/dummygram/issues/1280#issuecomment-1768972876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWRLMGR26MWA3KQYSEWYOVDYAADOPAVCNFSM6AAAAAA6FHPA7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRYHE3TEOBXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Create an independent browse blade for Function Apps in Ibiza

Aside from making it easier to find Function Apps, it will resolve the filter for the +Add command (https://github.com/projectkudu/AzureFunctionsPortal/issues/869).

This will be taken care of by @Tuesdaysgreen's changes, albeit in a slightly different way.

Done.


Added media query to ScrollAreaThumb in ScrollArea component

Why I added this little piece of media query: if you go to the docs in some block of code that contains the horizontal scrollbar (e.g navigation menu), you will notice that it is difficult to click on the "Collapse code" button when the scrollbar is visible, to do that, you need to click in the extreme top of the button.
In the video i'm clicking in the button and you will notice that the code don't collapse, then i'm sliding the scroll because the scroll is above the button and then i'm clicking in the top of button to collapse.
collapse-button.webm
I noticed that the piece of code that made this happen is the height and width of the scroll thumb. I also noticed that a developer inserted a comment into this part of the code that says:
// increase target size for touch devices https://www.w3.org/WAI/WCAG21/Understanding/target-size.html
If it is to increase the size for better accessibility on touch devices, then I thought it might be better to add a media query that detects if the device is touch. I did a search on stackoverflow and found that the media query "hover: none" does this, since touch devices do not have the hover event.
I also found another problem, for the horizontal bar, the scroll track doesn't work, while for the vertical bar it works normally, but not in the area of the element that is above the scrollbar (the collapse button container). If you try to click on the track you end up clicking on that element. I thought about adding a zIndex to the track, but I didn't find the scroll track primitive to do that, it would be nice if there was a primitive for the scroll track.
This pull request:

[x] Fixes a bug
[ ] Adds additional features/functionality
[ ] Updates documentation or example code
[ ] Other

Hey @igomonteiro,
Ideally I wouldn't want to complicate the example code with things that relate to our specific demo case.
We could look at raising the button a little bit instead perhaps?

I made a small change, let me know if this addresses your concerns still. I think that's a better solution.

I made a small change, let me know if this addresses your concerns still. I think that's a better solution.

Hi @benoitgrelard, yeppp, your solution is definitely better, thanks for your attention


Update colors

Why?
Changes to the design system.
https://www.figma.com/file/WzKCwRY09zn4rzRVfY0YvdRt/sats-ds-styles?type=design&node-id=1%3A2&mode=dev
What?

Remove theming
Adds new Rewards colors
Update .satsPrimary color for light appearance

Version Change
Breaking

it seems the highlight (hover) light for .satsPrimary is not longer #21324F but #092A48 (👈 note how to render colors in GitHub)
I did a quick test using this branch with the iOS app, of course, since it's a breaking change there were errors. I am interested on the Color.backgroundTopEnd that is used a bunch of places. Before approving this PR, I would like to see a branch PR on the iOS app that uses this branch version of SATSCore

@fespinoza Good catch!
yeah, there are a few screens with the gradient background, but I will fix them.


Laravel 8.83.9 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v8.83.9 branch.
Before merging, you need to:

Checkout the shift-ci-v8.83.9 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:warning: You are running version 2.8.4 of spatie/data-transfer-object. The latest version is 3.7.3. You should review this dependency to see if you can upgrade to the latest version.

:warning: The Shift build servers run PHP 8. If you are still running PHP 7.4 or lower you should run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP 7 environment.

:information_source: Shift detected your application only has the default tests. You can quickly start adding tests by using the Test Generator to automatically create the model factories, test classes, and sample tests cases for your Laravel application.


Hardcoded icon fab option which prevents generic icon usage

In templates/base.html:189
          >

the hardcoded template configuration prevents customization of icons. For example, I want to add an RSS "branded" icon to my blog footer, but doing so would require finding an associate brand icon as per Bootstrap's Icon Search
For now I'm probably just going to fork with this change (basically requiring branded icons to prefix the "fab fa-" with the icon (which may be less usable for beginners, but is fairly intuitive for me since I can just copy & paste from the search bootstrap provides)

Yes you are absolutely right.
I apologize for my mistake.
I'm a lone developer and it's very difficult to keep up with everything.

Absolutely no worries! I'm solo for 99% of what I make. If you wanna copy anything from my fork back into this, feel free :heart:

If you want to send a pull request I will accept it immediately.

I've made quite a few of customizing changes as well :sweat_smile:
But if you've looked it over and like my changes I'll do that

My customizations are running on my blog if you want to see what it looks like live: https://queenofsquiggles.github.io/

All good,
I think it's risky to bring your modifications here.
It can break many things that are working.
Anyway, thank you very much for the feedback, when I change this repository I will copy what you did.


Documenting Iterable with Spring Boot

Version 2.10.5

What kind of issue is this?

[x] Bug report.

[x] spring xml/java config that is relevant
[x] springfox specific configuration if it makes sense
[x] include any output you've received; logs, json snippets etc.; and what the expected output should be
[ ] if you have a repo that demonstrates the issue for bonus points! See this example

I have a pretty simple Spring Boot REST API using Spring Data. This is my Repository:
public interface TodoRepository extends CrudRepository<Todo, Integer> {}

Here is my controller:
@RestController
@RequestMapping(path = "/todos")
public class TodosController {
    @Autowired
    private TodoRepository todoRepository;

    @PostMapping(path = "")
    public Todo addTodo(@RequestParam String content) {
        Todo todo = new Todo();
        todo.setContent(content);
        todo.setDone(false);
        return todoRepository.save(todo);
    }

    @GetMapping(path = "")
    public Iterable<Todo> findAll() {
        return todoRepository.findAll();
    }
}

When I use springfox to generate swagger docs, pretty much every thing looks fine except the findAll endpoint. The response model is documented as an empty object {} instead of an Array of Todo.

However when I use a List<T> instead of the default Iterable<T> provided by the repository, by adding the following to my controller:
    ...
    @GetMapping(path = "/asList")
    public List<Todo> asList() {
        var result = new ArrayList<Todo>();
        todoRepository.findAll().forEach(result::add);
        return result;
    }

Then it shows up correctly.

Is there a way I can get the same behavior for the default Iterable<T> that the repository returns so that I wouldn't have to convert to a List<T> every time?
Configuration:
@Configuration
@EnableSwagger2WebMvc
@Import({ springfox.documentation.spring.data.rest.configuration.SpringDataRestConfiguration.class, })
public class Config {

    @Bean
    public Docket petApi() {
        return new Docket(DocumentationType.SWAGGER_2).select().apis(RequestHandlerSelectors.any())
                .paths(PathSelectors.any()).build().pathMapping("/").useDefaultResponseMessages(false);
    }

    @Bean
    UiConfiguration uiConfig() {
        return UiConfigurationBuilder.builder().deepLinking(true).displayOperationId(false).defaultModelsExpandDepth(1)
                .defaultModelExpandDepth(1).defaultModelRendering(ModelRendering.EXAMPLE).displayRequestDuration(false)
                .docExpansion(DocExpansion.NONE).filter(false).maxDisplayedTags(null)
                .operationsSorter(OperationsSorter.ALPHA).showExtensions(false).showCommonExtensions(false)
                .tagsSorter(TagsSorter.ALPHA).supportedSubmitMethods(UiConfiguration.Constants.DEFAULT_SUBMIT_METHODS)
                .validatorUrl(null).build();
    }
}

The reason Iterables aren't supported out of the box is cos we dont know the concrete type It represents

What would be the correct workaround?

As demonstrated by the commit, you as a library consumer needs to tell it how you want to represent iterable. I the case of the commit it creates a rule that treats Iterable as List


Get a better name

This project could use a catchier name.
The goal is to make it possible to fuzz libraries with large API surfaces. This is currently impractical because fuzzing harnesses have to be written manually. This project aims to automate that.
The name should reflect that. Please put your name suggestions in the comments, one comment per name.

fuzzmylib
Exactly one result in google (before this comment was posted): https://www.google.com/search?q=fuzzmylib
Makes for nice (hypothetical, future) cargo command: cargo fuzzmylib init, etc - similar to the established cargo fuzz. Perhaps too similar?

If it was a cargo command, that would make sense. But for the same of a project, I rather like SilkWyrm -- a mythological creature with no results related to software. Because, you see, silkworms create a lot fuzz...
... but if you'd rather not have such a lame pun for the name of it, I understand.

Google autocorrects "silkwyrm" to "silkworm", and even that produces a lot of results, so I'm afraid that's a no-go from the discoverability standpoint.
Another unique result on google: "fuzzfestation", short of "fuzz" and "infestation". Might have a negative connotation to it though?

Usually "infestation" is negative, but software has all sorts of terms where connotations are ignored, so I am not sure if it's that big a deal.
That did make me think of the more positive "fuzzfest", but that seems to be a concert.


Fix documentation; Eclipse CDT GCC build output parser regex was incorrect

Fix as per https://github.com/espressif/esp-idf/issues/2056

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

Thanks for the patch @jeremyherbert . As per update on Github issue, this was independently fixed in 75dc2f58005808fcaa86b7f560f45fa22b7884e8 based on your report.


update deadline to prevent user from setting it before date borrow

Coverage decreased (-0.8%) to 91.657% when pulling 4bcc25d7ff5de5dc48732bb36d829052cb64ae74 on lawwman:deadline into fbc46af106a21a5c373176aac198168912a14a9f on CS2103AUG2017-T17-B1:master.

Coverage decreased (-0.8%) to 91.852% when pulling f1376248237f35bfd271db77928fe99c4af6ef34 on lawwman:deadline into 52c9f16e811ca209d896a97755ebce65e7ea7f1b on CS2103AUG2017-T17-B1:master.

Coverage increased (+0.1%) to 92.805% when pulling 195e9ee71c8a226014c9ed23af498910c9894c16 on lawwman:deadline into 52c9f16e811ca209d896a97755ebce65e7ea7f1b on CS2103AUG2017-T17-B1:master.

Coverage increased (+0.1%) to 92.765% when pulling 27cdcf7621583d3dc12b2817a616587f40e97a09 on lawwman:deadline into 52c9f16e811ca209d896a97755ebce65e7ea7f1b on CS2103AUG2017-T17-B1:master.


Videos will not play on websites the use fluid player

Some sites like heavey-r where I found them using your services will not play

@PS2ClassicsVault can you please provide extra information about the bug? thank you

Closing this due to inactivity. @PS2ClassicsVault If you are still running into issues. Please let us know!


Replication IT for #12047

Description
Detailed reproduction of issue described in #12047 implemented as an IT.
The goal is to see if we can reproduce the issue – currently the issues is not reproducible.
This is WIP to drive the discussion further, do not merge it!
Related Issues
Relevant to #12047
Check List

~[ ] New functionality includes testing.~

~[ ] All tests pass~

~[ ] New functionality has been documented.~

~[ ] New functionality has javadoc added~

~[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)~
[x] Commits are signed per the DCO using --signoff
~[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)~
~[ ] Public documentation issue/PR created~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Can someone add a skip-changelog label please?


[ART-1301] release: Use digest for signing job

This is part 2 of ART-1301. Collect the digest of the release from the oc command, in order to pass it along to the signing/sign-artifacts job.
Needs https://github.com/openshift/aos-cd-jobs/pull/2067 to be merged.

:shipit:


fix(tools): use docker for development

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.

Hey @ValeraS this LGTM. Can we get the conflicts resolved?

Known issues on Windows:

npm install does not work, may help downgrade npm to version 5.7.1(Done with `npm run docker:install2)
db container stops execution with a 100 error (works without volumes). Related https://dba.stackexchange.com/questions/186478/mongodb-refuse-to-start-operation-not-permitted

Both isssues relate to shared folders (volumes).

@Bouncey @raisedadead @ValeraS as discussed in gitter, I can write the documentation for this. @ValeraS would you be able to hop on a text chat with me sometime in the next few days so I can pick your brain about what I need to document? Thanks.

@Bouncey @raisedadead @ValeraS the latest commit I just pushed has the updated docs. It should be enough for the easy path (no errors). I think this is ready for production and should be merged. For any remaining open issues we can continue to work on those seperately. But I think the best way to get dev feedback is to start using this. I also think it will cut down a lot on local dev setup questions in Gitter since most of the hard work now happens inside docker and should be 100% consistent for every local dev env.
Finally, my changes to the docs are the minimum to get people up and running. I have some ideas about how to simplify the local setup docs quite a bit but I'll do that in a separate PR since not all of it is related to docker.

@tchaffee What if users are unable to install Docker or don't want to, but prefer to develop as they have been doing with the current instructions?  If we will still support without Docker, shouldn't we have two sets of instructions?  I could be mistaken, but unless you are running Windows 10, you can not use Docker.  I believe for Windows 8.1, the user must use Docker Toolbox, which I am not sure if the same instructions would apply.

@RandellDawson the same set of instructions should work for Docker Toolbox, although that reminds me I have to add one instruction for that which I will do now. Going forward there shouldn't be any reason to set things up locally and manually and I don't think we should support it. The entire goal of Docker is to make sure our dev env is as identical to production as possible and the reduce the constant setup questions we get about mongodb, etc.

@tchaffee I totally think using Docker is the way to go, but not all Windows users are using Windows 10, which is why I brought up the part about Docker Toolbox.

@RandellDawson it looks like they can use Docker Toolbox instead of Docker Desktop: https://docs.docker.com/toolbox/overview/

I think we could have the docker instructions (once proven stable) as the primary/recommended method of working with freeCodeCamp and move the regular instructions  to a separate location (with a clear "without warranty" notice) who are unable to use docker for whatever reason.

@tchaffee I think for now, you should move the docker instructions to a new file. Let's get this in. After we push out master (which is a reasonable time for others to play with this) we can switch the docs like you intended originally in your commit.

@raisedadead I made some more changes locally. It's getting late here so tomorrow I'll move everything to a new file and restore the old version of how-to-setup-freecodecamp-locally.md. I would highly suggest putting an expiration date on the old instructions of a couple of months max and then delete them. Let's make a strong push for docker and only give access to the old docs if absolutely necessary for some strange reason. If the old instructions are there is just creates more confusion and people will accidently follow them and will get stuck and will ask for support. If someone really really needs to not use docker for some strange reason, we can always dig up the deleted file and get it to just them.

@RandellDawson a workaround was found to use docker windows in windows. I have been using docker windows(not docker toolbox) in Windows 10 Home for a month now and it has worked perfectly without any problems.

@raisedadead @ValeraS the Docker setup docs are now in their own file and this should be ready for review / changes or merge if you think it's ready. Thanks!

I am getting errors on npm run docker:seed. I am on a clean first time setup.

    Error logs

~/DEV/testFreeCodeCamp fix/docker-dev 7s
❯ npm run docker:seed

><EMAIL_ADDRESS>docker:seed /Users/raisedadead/DEV/testFreeCodeCamp
> docker-compose run --rm freecodecamp npm run seed

Starting freecodecamp_mailhog_1 ... done
Starting freecodecamp_db_1      ... done

><EMAIL_ADDRESS>seed /app
> npm-run-all -p seed:*

><EMAIL_ADDRESS>seed:auth-user /app
> cross-env DEBUG=fcc:* node ./tools/scripts/seed/seedAuthUser

><EMAIL_ADDRESS>seed:challenges /app
> cross-env DEBUG=fcc:* node ./tools/scripts/seed/seedChallenges

Oh noes!! Error seeding local auth user.
{ MongoNetworkError: failed to connect to server [db:27017] on first connect [MongoNetworkError: getaddrinfo ENOTFOUND db db:27017]
    at Pool.<anonymous> (/app/tools/scripts/seed/node_modules/mongodb-core/lib/topologies/server.js:564:11)
    at Pool.emit (events.js:189:13)
    at Connection.<anonymous> (/app/tools/scripts/seed/node_modules/mongodb-core/lib/connection/pool.js:317:12)
    at Object.onceWrapper (events.js:277:13)
    at Connection.emit (events.js:189:13)
    at Socket.<anonymous> (/app/tools/scripts/seed/node_modules/mongodb-core/lib/connection/connection.js:246:50)
    at Object.onceWrapper (events.js:277:13)
    at Socket.emit (events.js:189:13)
    at emitErrorNT (internal/streams/destroy.js:82:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
  name: 'MongoNetworkError',
  errorLabels: [ 'TransientTransactionError' ],
  [Symbol(mongoErrorContextSymbol)]: {} }
Oh noes!!
{ MongoNetworkError: failed to connect to server [db:27017] on first connect [MongoNetworkError: getaddrinfo ENOTFOUND db db:27017]
    at Pool.<anonymous> (/app/tools/scripts/seed/node_modules/mongodb-core/lib/topologies/server.js:564:11)
    at Pool.emit (events.js:189:13)
    at Connection.<anonymous> (/app/tools/scripts/seed/node_modules/mongodb-core/lib/connection/pool.js:317:12)
    at Object.onceWrapper (events.js:277:13)
    at Connection.emit (events.js:189:13)
    at Socket.<anonymous> (/app/tools/scripts/seed/node_modules/mongodb-core/lib/connection/connection.js:246:50)
    at Object.onceWrapper (events.js:277:13)
    at Socket.emit (events.js:189:13)
    at emitErrorNT (internal/streams/destroy.js:82:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
  name: 'MongoNetworkError',
  errorLabels: [ 'TransientTransactionError' ],
  [Symbol(mongoErrorContextSymbol)]: {} }

I got the following error when running npm run docker:develop the first time, then process exited.  The second time I ran npm run docker:develop it was successful, although I still see the error below in the logs each time.

    Error logs on macOS - Mojave - 10.14.3. Docker Desktop Community Version <IP_ADDRESS>

client_1        | success onPreInit — 1.007 s
client_1        | glob error { [Error: EIO: i/o error, scandir '/app/client/public/static/d/729']
client_1        |   errno: -5,
client_1        |   code: 'EIO',
client_1        |   syscall: 'scandir',
client_1        |   path: '/app/client/public/static/d/729' }
client_1        | error EIO: i/o error, scandir '/app/client/public/static/d/729'
client_1        |
client_1        |
client_1        |   Error: EIO: i/o error, scandir '/app/client/public/static/d/729'
client_1        |
client_1        | error UNHANDLED REJECTION
client_1        |
client_1        |
client_1        |   Error: EIO: i/o error, scandir '/app/client/public/static/d/729'
client_1        |
client_1        | npm ERR! code ELIFECYCLE
client_1        | npm ERR! errno 1
client_1        | npm ERR<EMAIL_ADDRESS>develop: `gatsby develop "-H" "<IP_ADDRESS>"`
client_1        | npm ERR! Exit status 1
client_1        | npm ERR!
client_1        | npm ERR! Failed at the<EMAIL_ADDRESS>develop script.
client_1        | npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
client_1        |
client_1        | npm ERR!  Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>docker:develop script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/chris/.npm/_logs/2019-03-08T23_02_38_052Z-debug.log

```</details>


TIMEOUT ERROR occurred for TCP MODBUS (IDFGH-4728)

Environment

Development Kit:      ESP32_Ethernet_kit_A_V1_1
Kit version (for WroverKit/PicoKit/DevKitC): V1.1
Module or chip used:  ESP32-WROVER-B
IDF version (run git describe --tags to find it): V4.1.1
Build System:        idf.py
Compiler version (run xtensa-esp32-elf-gcc --version to find it):
// 1.22.0-80-g6c4433a
Operating System:     Windows
(Windows only) environment type: ESP Command Prompt
Using an IDE?: No
Power Supply:   USB

Problem Description
We are not able to get the response of MODBUS TCP request from PLC.
Basically we are using ethernet for TCP MODBUS. We are using TCP sample code which is provided into the example. Using this TCP MODBUS example we are trying to read the "Holding register (8960)" from Shindler PLC. But we are not able to get the response. If we test this code with other PC (Using Modscan) It is working fine. (Hardware wise PLC are compatible)
Once more observation which I have is, If we are going read any other register like coil register which are not configured then we will get the exception response.
Expected Behavior
We have to get the response for Holding Register.
Actual Behavior
We are getting TIMOUT ERROR
Steps to reproduce

step1 : Connect our board with Shindler PCL
step2 : Change the server IP address in to the code
step3 : Build and program our device
step4 : Our board are able connect with PLC and send the command.

// If possible, attach a picture of your setup/wiring here.

Hello @harshalAcc,
Thanks for reporting.
The information provided is not enough to reproduce and find the reason for issues. Could you provide more information:

The IDF version v4.1.1 showed above does not support the Modbus TCP communication! Please clarify how you use the Modbus TCP in your project.
Are you using the modbus TCP master example from IDF and try to read the registers of PLC configured as a Modbus TCP Slave Device? If so, how do you configure the data dictionary for CID corresponding to "Holding register (8960)" from Shindler PLC in the master example? Please provide the changed code and send your existing log file of the example when you try to read the registers.
Please set Default log verbosity to Debug  in the Component config - > Log Output kconfig menu and then recompile example then repeat the connection from PLC and save log.
I need your example code with modifications and sdkconfig file to reproduce the issue.

Thanks.

Hello @alisitsyn ,
Thank you very much for your inputs,
I had configured the logging as you mentioned. I found the issue which is related to UID set by me.
Thanks

@harshalAcc,
Feel free to reopen if you have further issues.