SSH Connection to Slave Error

07/06/16 17:52:51] [SSH] Opening SSH connection to <IP_ADDRESS>:32792.
ERROR: Server rejected the 1 private key(s) for jenkins (credentialId:f89770c5-b0d6-411b-b348-3c0c63b89c4d/method:publickey)
ERROR: Failed to authenticate as jenkins with credential=f89770c5-b0d6-411b-b348-3c0c63b89c4d
java.io.IOException: Publickey authentication failed.
    at com.trilead.ssh2.auth.AuthenticationManager.authenticatePublicKey(AuthenticationManager.java:315)
    at com.trilead.ssh2.Connection.authenticateWithPublicKey(Connection.java:467)
    at com.cloudbees.jenkins.plugins.sshcredentials.impl.TrileadSSHPublicKeyAuthenticator.doAuthenticate(TrileadSSHPublicKeyAuthenticator.java:109)
    at com.cloudbees.jenkins.plugins.sshcredentials.SSHAuthenticator.authenticate(SSHAuthenticator.java:408)
    at com.cloudbees.jenkins.plugins.sshcredentials.SSHAuthenticator.authenticate(SSHAuthenticator.java:428)
    at hudson.plugins.sshslaves.SSHLauncher.openConnection(SSHLauncher.java:1173)
    at hudson.plugins.sshslaves.SSHLauncher$2.call(SSHLauncher.java:701)
    at hudson.plugins.sshslaves.SSHLauncher$2.call(SSHLauncher.java:696)
    at java.util.concurrent.FutureTask.run(FutureTask.java:266)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at java.lang.Thread.run(Thread.java:745)
Caused by: java.io.IOException: The connection is closed.
    at com.trilead.ssh2.auth.AuthenticationManager.deQueue(AuthenticationManager.java:63)
    at com.trilead.ssh2.auth.AuthenticationManager.getNextMessage(AuthenticationManager.java:86)
    at com.trilead.ssh2.auth.AuthenticationManager.authenticatePublicKey(AuthenticationManager.java:290)
    ... 11 more
Caused by: java.io.IOException: Cannot read full block, EOF reached.
    at com.trilead.ssh2.crypto.cipher.CipherInputStream.getBlock(CipherInputStream.java:81)
    at com.trilead.ssh2.crypto.cipher.CipherInputStream.read(CipherInputStream.java:108)
    at com.trilead.ssh2.transport.TransportConnection.receiveMessage(TransportConnection.java:232)
    at com.trilead.ssh2.transport.TransportManager.receiveLoop(TransportManager.java:693)
    at com.trilead.ssh2.transport.TransportManager$1.run(TransportManager.java:489)
    ... 1 more
[07/06/16 17:52:51] [SSH] Authentication failed.
hudson.AbortException: Authentication failed.
    at hudson.plugins.sshslaves.SSHLauncher.openConnection(SSHLauncher.java:1178)
    at hudson.plugins.sshslaves.SSHLauncher$2.call(SSHLauncher.java:701)
    at hudson.plugins.sshslaves.SSHLauncher$2.call(SSHLauncher.java:696)
    at java.util.concurrent.FutureTask.run(FutureTask.java:266)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at java.lang.Thread.run(Thread.java:745)
[07/06/16 17:52:51] Launch failed - cleaning up connection
[07/06/16 17:52:51] [SSH] Connection closed.

When the jenkins master attempts to connect to the slave I get the following error in the log. I've double checked that the correct public key is in the authorized_keys file. In addition if I try to connect directly I get:
System is booting up. See pam_nologin(8)

The keypair I am using was generated using the ssh-keygen -t rsa command on my host machine.

I was able to work around this issue by adding
RUN rm -f /run/nologin

at the end of the Dockerfile for the jenkins slave.

Hey @AdnaneKhan , thanks for the suggestion. Had the same error here and adding the removal of /run/nologin did the job 👍
Also, on the slave's sshd side, after running rsyslogd inside the slave you can check that (before removing the nologin) we get:
Jul  7 19:49:19 aba7116ed670 sshd[20]: fatal: Access denied for user jenkins by PAM account configuration [preauth]

Neat find. Is this happening for you right out of the box with what I have or by any chance are you changing versions on anything?
Last time I tested things seemed to work so I'll go figure out what's changed. This looks like a good update so worst case I'll apply this to the build slave image permanently.  Going to leave this issue open until I'm confident things just work out of the box one way or another.  Thanks for coming back with what you found fixed it!

Pretty much out of the box. I had updated the Jenkins version however this issue still manifested itself with the following set of steps to take Jenkins out of the mix.
While in jenkins-slave directory:
docker build .

docker run {IMAGE_ID}

ssh -i files/dummy_private_rsa_key<EMAIL_ADDRESS>
The result is then:
The authenticity of host '<IP_ADDRESS> (<IP_ADDRESS>)' can't be established.
ECDSA key fingerprint is 6b:c0:6e:c6:e8:0e:75:68:3c:05:32:0e:ba:df:40:de.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
System is booting up. See pam_nologin(8)
Connection closed by <IP_ADDRESS>

I tried this using both Docker installed directly on my CentOS 7 host, as well in the docker machine running Boot2Docker.
The Docker version I am using is 1.11.1

I'm having the same issue, but adding @AdnaneKhan's fix to the end of the slave file didn't alter anything.  Could it be linked to using Docker version 1.12.2?

This solution worked for me as well, thanks so much!  For reference, I'm running docker version 1.13.1.

Still plan on including this in my update, though On Jenkins2 I've switched to the JNLP launcher (from Yet another docker plugin) and SSH is no longer needed but I'll get the Jenkins 1.6'ish version fixed.

Will you make a new tutorial for it?

Working on a new tutorial but yes, it's all fixed in the Jenkins2 folder (I'm moving on from Jenkins 1 finally!)


Documentation for initializing

Hi,
Do you have any documentation to initialise the planner with existing data?

Hey there, sadly the project is not documented but there's an example on here using mocked data.

Thank you, I'll manage with this example. It's a good project, simple and effective. Nice work!


Documentation lacks of "operations" manual and limitations

Hi there,
I was reading the documentation, seeking for clear instructions on how to transfer properly instances between remotes with few differences (storage names, network configuration, etc...).
As I can see, there no such documentation : documentation is mostly on configuration, specials functionnality like cloud-init.
But it seems that it lacks "operational manual", like day to day actions, built actions, snapshot, backups, and so...
This is not an aggressive remark, it's just ascertainment and I think that it could be a good thing to create a chapter for those.
I based my reads on :

https://linuxcontainers.org/lxd/docs/master/
https://linuxcontainers.org/lxd/advanced-guide/

I don't take account of @stgraber blogs, as I think it can't be consider as an up-to-date, trusty and as official as the inline documentation 😋 . Hence it's a damn good reference 😄
Hashtag No offence 😸

I add the need of "good practices" for every "basic needs" : backup, remote copy, creation, use of profile etc...

I believe @ru-fu is working through our docs and refreshing them with proper usage examples section by section.

If you have a specific question that is missing from the docs today please could you post on the forum https://discuss.linuxcontainers.org thanks

Yes, I agree 100%, and as Tom says, the plan is to make the documentation more instructional bit by bit - we've started with the networking documentation and are currently working on storage.
There's also a lot of helpful information in the Tutorials on the forum and in @stgraber 's YouTube videos. For now, most of that content is just linked, but we want to incorporate the most relevant parts into the docs as well.

I may participate with pleasure, but I need "someone over my shoulder" who may correct if it's good practise or not.
Is there any way to do so?

If you have any "how to"s that you'd like to contribute, either based on your own experience or on the YouTube videos, that would be highly appreciated. :+1:
You could either make pull requests with new sections or pages to extend the current documentation (and I can review to make sure they fit in nicely), or just add them as tutorials on the forum and we can integrate them in the docs later.

The documentation has now been much improved. With usage examples for common actions.
And we will continue to improve it.
I'll close this for now, but please do open an issue or pull request if you feel specific items are still missing.


Camera calibrarion

Hi I'm wondering if anybody can help. I am trying to build an application that can map the 3D location of a human being By first extracting keypoints using PoseNet. I built a custom neural network that ingests features Provided by PoseNet. The problem that I have is that the accuracy of my application is very low and I suspect that it is due to the fact that I haven't Calibrated my camera. As I'm reading more about this and it seems that this is performed using Open CV. I was wondering if anyone had ideas on how to incorporate A calibration matrix into my solution.
PS: my knowledge of CUDA is limited, and I can not access SDK/intellisense information in my IDE, so I am not very sure what I can / can not send to net.Process() (even after reading the SDK page which states that a CUDA memory capsule is expected) - I can't seem to find information on how to construct that.
Should I calibrate my camera first in OpenCV, and then transform what videoSource is capturing to a numpy array - apply the un-distortion effect - and bring it back to a CUDA memory capsule for processing by PoseNet? if that's the high-level methodology, any ideas on the approach/gotchas/things to keep in mind to accelerate inference?

@eyast there are CUDA functions in jetson-utils for applying image de-warping given the intrinsic camera calibration parameters:
https://github.com/dusty-nv/jetson-utils/blob/8aa096774b76d899719244aa96b751337669f9ab/cuda/cudaWarp.h#L101
however, these don't have a Python API yet (C++ only)
so yes, you could have OpenCV perform the de-warping first:

https://github.com/dusty-nv/jetson-utils/blob/8aa096774b76d899719244aa96b751337669f9ab/python/examples/cuda-to-cv.py
https://github.com/dusty-nv/jetson-utils/blob/8aa096774b76d899719244aa96b751337669f9ab/python/examples/cuda-from-cv.py
https://github.com/dusty-nv/jetson-inference/blob/master/docs/aux-image.md#array-interfaces


KAFKA-5416: Re-prepare transition to CompleteCommit/Abort upon retrying append to log

In TransationStateManager, we reset the pending state if an error occurred while appending to log; this is correct except that for the TransactionMarkerChannelManager, as it will retry appending to log and if eventually it succeeded, the transaction metadata's completing transition will throw an IllegalStateException since pending state is None, this will be thrown all the way to the KafkaApis and be swallowed.

When re-enqueueing to the retry append queue, re-prepare transition to set its pending state.
A bunch of log4j improvements based the debugging experience. The main principle is to make sure all error codes that is about to sent to the client will be logged, and unnecessary log4j entries to be removed.
Also moved some log entries in ReplicationUtils.scala to trace: this is rather orthogonal to this PR but I found it rather annoying while debugging the logs.
A couple of unrelated bug fixes as pointed by @hachikuji and @apurvam .

We could simply pass a flag to the appendTransactionToLog method.

I have thought about that too. The reason I did not follow that is because we would want to add the check on the cached metadata anyways when the append fails, and if the metadata has changed we would just want to abort transiting.

Addressed comments.

retest this please

The updates LGTM. Will await the result of the builds.

retest this please

Merging to trunk and 0.11.0.


Cannot get sample app to pull secrets from key vault

[Enter feedback here]

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

ID: 2cc0b77a-df21-7007-5cec-4399d98c6dc6
Version Independent ID: efe5ae5e-ff64-f1c7-6920-4512fc8f36d1
Content: Azure Key Vault Configuration Provider in ASP.NET Core
Content Source: aspnetcore/security/key-vault-configuration.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello,
I am not a developer. I am an Azure AD admin and I am currently learning .Net core so I can assist developers looking to protect APIs with Azure AD. I have populated the appsettings.json with the required parameters. However, when I run the app, it doesn't show values from the key vault. I have attached output.

App registration created and cert uploaded
Key Vault created and access policy added giving Get & List permission to service principal
Not sure what else I am missing.
Please help.

Can you get https://docs.microsoft.com/en-us/azure/key-vault/secrets/quick-create-cli working?

I still have the sample app locally ... ran it ... and ...

... it still works ok. I did need to change one minor thing. I stored my cert for the LocalMachine, so I needed to get it from there. That would be obvious tho to a dev running the sample, as it would throw when it didn't find the cert on the machine.
It's likely going to be a configuration problem on the Azure side or between Azure and the app. If you institute logging in the app ...

Logging
Troubleshoot ASP.NET Core on Azure App Service and IIS

... you should be able to get an error message from Azure Key Vault.
Otherwise, you would just need to re-walk the steps here carefully and make sure everything is wired up correctly. Don't forget that this app has to be run in the Production environment or else it doesn't actually try to get the secret values from Azure Key Vault. If you want to do a quick flip on that then just change ...
if (context.HostingEnvironment.IsProduction())

... to ...
if (context.HostingEnvironment.IsDevelopment())

... in Program.cs.
btw @Rick-Anderson --- They expanded the portal for this scenario. These commands that the topic uses to set the secrets ... those steps can be done right in the portal now. When we wrote this topic in the ancient days of ASP.NET Core 👴 (a year or two ago :smile:), that wasn't a portal feature. I think we're probably too busy to fix it up to use that process (I know I am. 🏃😅), and the current commands should still work 🤞.

@SB-o-matic, I think this is going to turn out to be a configuration issue given that the sample app runs. If you don't get a helpful error message from Azure via logging, you'll need to walk the topic steps again (and perhaps access Azure docs on Key Vault) and see if you can determine where the problem lies.
The only change that I'm aware of is that you're not strictly required to use the Azure CLI any longer to set the secrets in Azure. You can do that in the portal now with new UI that Azure didn't have when the topic was written. However, I expect those Azure CLI commands still work if you want to use them.
We don't have the staffing to provide product support, so we can't take further action unless/until we know exactly what the problem is in a topic like this. I'm going to close this and cross-link the support channels we generally recommend if you have questions ...

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

We can re-open this issue when you find out what the problem turns out to be if it's actionable for the doc. I've added a few gotchas from prior devs to the topic's troubleshooting guidance to help steer devs in the right direction when they hit problems. If you come up with a new gotcha :boom: that we can call out there, I'll re-open this and add it to the list.


Pre Trained Model?

Really Nice Work.
I want to test it on my images, but i cannot find pre-trained model links for R3det++ or  R3det?
Can you please provide the google drive links for the models
Thanks

you mean trained model?
Due to code updates, I cannot guarantee that these old links will work. If you cannot use it, you may need to roll back to an earlier version (may be this version).

@yangxue0827 the links in earlier version are also not working (Page Not Found)

There are some models that I have deleted, and the rest you need to see the current version of MODEL_ZOO @PhenomenalOnee

You may need to retrain yourself. Because the code has been modified and improved a lot, the previous weighted probabilities cannot be used. @sergiev


Very new tokens may not get picked up by the bot / Redesign Event Mapping

Bug Description

Summary of the bug
For the bot to be aware (i.e. capable of arbitraging) any pools in the ecosystem, a pool of interest must be present in one of the event_mappings files. The event mapping files allow events with the same name to be correctly mapped from their pool address to their exchange.
This issue is most pronounced when a new token/pool(s) is added to the ecosystem and the bot is not aware.
Typically this data is maintained by running the terraformer module and updating the static files - including the event_mappings. However the state of the market demands a more dynamic solution to support speculation on new tokens.

Severity (High/Medium/Low)

Rate the severity. High: "I can't/won't use the bot due to this", Low: "I can work around it", Medium: (anything in between).
Medium severity as it only affects new tokens - but can leave Carbon orders unfilled.

Steps to Reproduce

Step-by-step reproduction method, include environment information if relevant - such as Python version and OS.
Occurs in all modes

Expected Behavior

What should happen
The minimal solution should ensure that all tokens present on Carbon (within reason) have external pools mapped such that an arb route could be created. Best: direct pair match; 2nd: support by triangle.
A better solution would permit any tokens to be arbed (independent of Carbon) but this may cause slowdown of the bot

Actual Behavior

What currently happens
New pools are added but not recognized by the bot. Bot cannot arb these. If this token appears or Carbon the order may go unfilled.

Proposed Solution
Much of the infrastructure required to solve this problem already exists in the terraformer some repurposing should occur but the minimal solution should look like this:

On startup, fetch all Carbon pairs (from all deployments on that chain)
Search all exchange Factory events for exact pair matches
Add the new pools to the appropriate event_mapping file

Optimizations
There are a number of optimizations that could be considered:

Fetching all Carbon pairs on startup PLUS every time a new strategy is created
Fetch all historical new pairs/pools can be time consuming - we could store in a static file and periodically update this - elaboration below
Add a minimum WEI filter for the pools that are added. Having low-liquidity pools is the same as not supporting the pair AND it can lead to failed txs and error messages PLUS bloats runtime
Exact match for PAIR plus find matches for TRIANGLES (could start with gas token / stables only)
The pairs list for Carbon.pairs() will contain pairs with no CURRENT liquidity on Carbon, so the pairs list could be created or cross-referenced with the StrategyUpdated events to ensure we only add relevant external exchanges

Storing historical pools
If fetching all the new pair events from the entire ecosystem is time-consuming or has RPC implications, we could store this history in the bot. The reason it would be separate from the event_mappings is that the event_mappings are iterated over during run time.
The reason it might be prohibitive to fetch every time is the shear number of blocks passed since the exchanges started e.g. multiple years worth of uniswap_v2 events.
Flow:

Seed all historic new pairs in a file (think of this as comprehensive event_mapping)
On run, update the historic file from its last block to the current block
Get the Carbon pairs, find exact matches in the historic file, and add these to the event_mappings
Default mode could ONLY store event_mappings for relevant Carbon strategies so that runtime is minimized

Impact Analysis (to be filled by contributors)

Automated Tests: Are there existing tests covering this? Do we need new tests?
Risks: Potential areas affected by this bug and its probable fix
Performance: Is this bug affecting performance?
Monitoring: Does this affect any current monitoring? Any new alerts needed?
Logging: Any changes required in logging?
Documentation: Updates needed in documentation?

I believe event-fetching on bot start will be prohibitively time-consuming.
I want to suggest an alternative:
Manual Pair finding via Factory Contracts:
We could gather the Carbon Pairs and ensure we have viable pool combinations for them by querying Uni V2/3 & Solidly Factory contracts for pools - for example the Uniswap V2 getPair, Uniswap V3 getPool, and Solidly getPool.
The process could be:

Gather all Carbon pairs.
Find pairs without viable routes (this is more tricky if considering available liquidity).
Multicall each Factory searching for pools with token combinations that can create viable pairs/triangles.
Add each pool found to static data. If the pools are added only to the mapping files, the entire process would be fairly fast.


List tables by taskType

Adding query param to list tables API to get tables by taskType.
Required for UI being built in https://github.com/apache/pinot/issues/8970

Codecov Report

Merging #9049 (c645614) into master (0c55422) will decrease coverage by 37.04%.
The diff coverage is 19.65%.

:exclamation: Current head c645614 differs from pull request most recent head e25485f. Consider uploading reports for the commit e25485f to get more accurate results

@@              Coverage Diff              @@
##             master    #9049       +/-   ##
=============================================
- Coverage     63.49%   26.45%   -37.05%
+ Complexity     4714       42     -4672
=============================================
  Files          1783     1819       +36
  Lines         94134    95931     +1797
  Branches      14140    14357      +217
=============================================
- Hits          59774    25378    -34396
- Misses        30102    68115    +38013
+ Partials       4258     2438     -1820

Flag
Coverage Δ

integration1
26.45% <19.65%> (?)

unittests1
?

unittests2
?

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

Impacted Files
Coverage Δ

...oller/api/resources/PinotTableRestletResource.java
23.64% <0.00%> (-23.19%)
:arrow_down:

...re/common/evaluators/DefaultJsonPathEvaluator.java
15.10% <0.00%> (-14.43%)
:arrow_down:

...e/operator/dociditerators/MVScanDocIdIterator.java
50.61% <0.00%> (-19.76%)
:arrow_down:

...e/operator/dociditerators/SVScanDocIdIterator.java
76.53% <0.00%> (+1.02%)
:arrow_up:

...ot/query/runtime/executor/WorkerQueryExecutor.java
0.00% <0.00%> (-90.39%)
:arrow_down:

...inot/query/runtime/operator/AggregateOperator.java
0.00% <0.00%> (-85.00%)
:arrow_down:

...local/indexsegment/mutable/MutableSegmentImpl.java
0.00% <0.00%> (-54.35%)
:arrow_down:

...e/impl/forward/FixedByteMVMutableForwardIndex.java
0.00% <0.00%> (-70.79%)
:arrow_down:

...e/impl/forward/FixedByteSVMutableForwardIndex.java
0.00% <0.00%> (-93.88%)
:arrow_down:

...ime/impl/forward/VarByteSVMutableForwardIndex.java
0.00% <0.00%> (-66.67%)
:arrow_down:

... and 1506 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 0c55422...e25485f. Read the comment docs.


Undo sometimes leaves ghost blocks in place

I've seen this only intermittently, but undo sometimes leaves ghost blocks behind after removing a block.  It might have something do with "let" or polymorphism.  I will keep looking for a way to reproduce.

I haven't seen this in a long time, and there have been fundamental enough changes that it's likely obsolete.


Add tests for lazy tile loading.

Refactored lazy tile loading so that it is lazy even when the target format is not PIL or the source image type.

@cdeepakroy In the previous PR, if you had asked for a tile iterator returning a format other than PIL or the source image format, it still was loading the tile image as part of the iterator.  This now defers the conversion to a numpy array or other format, too. (and there is a test for the deferred load).

Great! LGTM!


Godeps: semantic versioning cheggaaa/pb

Fix https://github.com/coreos/etcd/issues/4832.

/cc @xiang90 @heyitsanthony
Thanks.

lgtm

lgtm


Enhance $router with $router->controllerPath()

Hello:
Please implement a way to know the complete path of a given controller.
For example:
`
$router = \Config\Services::router();
var_dump([
$router->controllerName(),
$router->controllerPath() // To implement
]);
/*
Would give:
[
"\App\Modules\My_Module\Controllers\My_Controller",
"/my/path/to/codeigniter/app/Modules/My_Module/Controllers/My_Controller.php"
]
*/
`

Wrong place, this is the CI3 repo.


rfc: Track blazor wrappers for fast components

💬 RFC
We should consider creating a tracker sheet to track progress or to highlight for which fast components do we have blazor wrappers. If for some fast components we don't have wrappers as of now we can mention there, so that community can see the progress of the fast-blazor library. In this way, developers and the community can have a clear picture of what's there and what's not.
💻 Examples
We can track progress and availability with tables of the following columns.
Columns -

fast component name
blazor fast component name
Complete or not (with checkbox)
Remarks - This column if some property on the fast component is not yet available in razor component.

I had this motivation from looking at how the dotnet maui team is tracking progress.
https://github.com/dotnet/maui/wiki/Status

Good idea, but we are following the Fluent Web Controls here which are not always 100% in line with the Fast components

yeah. So we can track blazor components progress against Fast web components!

No, we can track against the Fluent web components (https://github.com/microsoft/fluentui/tree/master/packages/web-components)

Ohhh. Names got me confused. So this is the track I guess.
Fluent web components --> Fast web components --> blazor fast web components (.razor)
Is my understanding and sequence right?

Almost 😁
First there is Fast framework with it's own Fast WC implementation. Then Fluent WC (following the Fluent Design) are another implementation of Fast. These Fluent web components are the components we are wrapping with Blazor here.

Ahh naming is hard.
So we can track progress against fluent web components for our blazor web components right?

Ok, so to clarify, we'll add a markdown document with a table that has the following columns:

FluentUI Web Component name
Blazor component name
Complete or not (with checkbox)
Remarks - This column if some property on the fast component is not yet available in razor component.

Should we also add some links for convenience? For example: link to the Blazor component example, link to the docs.microsoft.com page for the component, etc.

Should we also add some links for convenience? For example link to the Blazor component example, link to the docs.microsoft.com page for the component, etc.

Hmm, that would be a good idea as well. But on the docs website, there are no samples regarding blazor components, I have seen samples for typescript only. https://docs.microsoft.com/en-us/fluent-ui/web-components/components/badge


feat: Array.qsort_sorted

This was a surprisingly challenging proof. It really highlights some of the pain involved in verifying programs which perform many array modifications without something isomorphic to separation logic, because one has to hold on to information about the rest of the array across many function calls that could potentially modify any part of the array.
I tried using invariants based on indexes, describing sorting in the form i < j -> as[i] <= as[j] with various restrictions on i,j in the algorithm, but this turned out to be quite painful (some remnant of this still remains in the definition of PermStabilizing). What worked better is lemmas like swap_of_append which just relate the array to a list expression like A ++ a :: B ++ b :: C where A.length = i and (A ++ a :: B).length = j, because this way the lists don't interfere with each other (there is only one A preserved before and after the operation), by comparison to array ops like set and swap which turn into if statements and arithmetic goals after every operation.

Have you considered writing List.qsort, prove this correct, and then show how they are related? I found that that is often easier, at least when I verified a Trie data structure.

Yes, I did. My original goal was to verify the sorting functions in Std.Data.Array.Merge, and all of the functions have natural list counterparts except Array.qsort, which uses Array.swap, index-based manipulation, and fixed size arrays heavily.

I see, in that case a list-based reference implementation probably won't help that much.

I'll leave this here for anyone interested; I have independently completed a specification and verification of Quicksort .. I have been working on refactoring to improve the style and elaboration time.


Fix caching for c

Uncommenting code doesn't work when running codecrafters test. Looks like a caching problem.

@andy1li it's okay to test this on production, but let's investigate this properly please - it's super weird that CI didn't catch this (it should've?)


Passthrough execution mode

Description
This PR adds support for passthrough execution mode in Ratify. This mode exists to ignore policy within the Ratify executor and provide all verifier results back to the caller for evaluation. In order for this to work well there are a handful of notable and/or breaking changes that are introduced:

Add executionMode field to executor config
Change Ratify http handler to return the entire verification result rather than just an error string on failure
Change the Results string array in VerifierResult to a Message string
Add Extensions interface in VerifierResult to allow verifiers to include metadata in the response
Add SN extension field to NotaryV2 verifier for certificate subject name
Add initial library of ConstraintTemplates supporting both default and passthrough execution modes
Remove Constraint and ConstraintTemplate from Ratify chart in favor of the default policy from the library
Remove ratify-gatekeeper from charts as it is no longer needed
Add documentation for Gatekeeper policy authoring
Change README quick start steps to utilize new default library policy
Change e2e test to use new default library policy

Fixes #171, #176
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?

[ ] CLI signature validation in default execution mode
[ ] In-cluster signature validation in default execution mode
[ ] In-cluster signature validation in passthrough execution mode

The in-cluster test cases are run against each of the ConstraintTemplates added to the library.
Checklist:

[ ] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[x] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have appropriate license header?

I have a bit more cleanup to do (tests, comments, etc.), but for the most part the intended changes are complete.


FIx default value for processing version

Fixes #10844
Status
ready
Description
Change default values of processingVer in various places of WMCore codebase
Is it backward compatible (if not, which system it affects?)
MAYBE
Related PRs
#11043
External dependencies / deployment changes

Alan, turns out WMCore has plenty of processingVer defaults pointed to None. I scan WMCore code for them, based on jenkins test failures and I think I identified the final set, unless the default values of processingVer at a different lines. Please include this PR in jenkins tests and we will see how it is going.


GEODE-2830 Required permission for executing a function should be

DATA:WRITE
Simple typo correction

LGTM +1

Merged


@FeatureGate annotation does not work together with @PostMapping

spring-cloud-azure-feature-management
Expected Behavior

If the feature is deactivated, it returns a 404 status code to the client with no response body

Current Behavior

When @FeatureGate annotation is used together with @PostMapping the default fallback is routed to a GET and does not return a 404 by default. As a workaround I had to add a fallback and an addional @GetMapping method to return a 404 status code. Is there a way to use @FeatureGate together with @PostMapping without adding addional GET methods?

Snapshot Code for Reproduce
@RequestMapping("/api/")
public class SomeController {
   ...
    @FeatureGate(feature = "SOME_FLAG", fallback = "somePath")
    @PostMapping(path = "somePath", produces = MediaType.APPLICATION_JSON_VALUE)
    public ResponseEntity someMethod(...) {
	    ...
        return ResponseEntity.created(location).build();
    }

    @GetMapping(path = "somePath", produces = MediaType.APPLICATION_JSON_VALUE)
    public ResponseEntity featureDeactivated() {
        return ResponseEntity.notFound().build();
    }

Your Environment

Version used: 1.2.1
Spring boot: 2.2.4.RELEASE
SDK version: 11

Hi @michaelegner I am looking into it, but it seems like a bug.

@michaelegner A new release of feature management just went out that should fix this issue.

Many thanks for the quick fix! We will check it out.


How to bundle / specify wasm URL?

Using webpack, it tries to load the WASM from a file:/// url and the wasm file is not bundled as an asset. Is it possible to specify the wasm  URL or make it work with webpack in some other way?

it would work correctly if you get the URL by doing
const url = new URL('./foobar.wasm', import.meta.url).toString()

I've never used Webpack and have no idea how it works. You can specify the WASM URL by passing an argument to the module factory, e.g.:
import SQLiteESMFactory from '../dist/wa-sqlite.mjs';
...
const module = await SQLiteESMFactory({
  locateFile(file) {
    return `https://rhashimoto.github.io/wa-sqlite/dist/${file}`;
  }
});

...but I don't know whether Webpack would pick that up.
Because I don't know Webpack, I don't understand your second comment - is that something you're suggesting as a user workaround, or for me to put in the package somewhere?
Is the Emscripten ES6 modularization part of the problem? Building a CommonJS module is just a matter of changing .mjs to .js in the Makefile. I can build both if that's easier to use with Webpack.
Sorry, but Webpack is not even a black box to me; I don't even know what the box looks like! 😀

Is the Emscripten ES6 modularization part of the problem?

No, webpack works fine with ES6.

I don't understand your second comment - is that something you're suggesting as a user workaround, or for me to put in the package somewhere?

If you use that segment within the library to get the wasm file path, then webpack will realize the wasm file is an asset that needs to be written to the output folder. But looking at the code, I see that the wa-sqlite.mjs code is completely generated by emscripten so I guess you don't actually have control over that, it needs to be "fixed" on their side.
Your locateFile solution works though, thanks. You should maybe add the info to the readme since I'd guess most people use a bundler / webpack.


action fails

Hey, i don't get it to work.
I know, GitHub isn't for support and stuff, but you seem to understand this much more than i do.
The action is failing over and over.
https://github.com/Hope-IT-Works/Hope-IT-Works/runs/1773246181

I don't understand why.
I created a new personal access token at https://github.com/settings/tokens

Then i added the given token to the action secrets in the repository settings

You can check my action (did not change anything) and my template
Do you know what's wrong?
Thanks in advance,
Tobi

This is probably because the default GitHub branch name has changed from master to main.
In readme-scribe.yml change branch: master to branch: main accordingly. I'll update the docs to reflect the new default.

This is probably because the default GitHub branch name has changed from master to main.
In readme-scribe.yml change branch: master to branch: main accordingly. I'll update the docs to reflect the new default.


Consistent swift coding style

Taken from this reddit comment:: https://www.reddit.com/r/swift/comments/3ttu9u/serverside_swift_is_here_its_perfect/cx9hrpc
Nice work!
I have one suggestion though - be more consistent wrt to code conventions. For example, top
level constants (from one module) are defined in
- lowercase snake case
- uppercase snake case
- camelCase
...

by 20DCCD77-E82E-4045-A
I would suggest using https://github.com/raywenderlich/swift-style-guide as your style guide. At least as a mold for a style guide. It seems to be very good.

+1

+1 on the Ray Wenderlich style guide for inspiration.

Realm even has a linter as well :D
https://github.com/realm/SwiftLint

+1 for Ray wanderlich swift guide

:+1:

I'm a big believer in the Ray Wenderlich style guide too... However I'm closing this issue as it's been hanging around for a while and a lot of code's been updated to conform to conventions :)
Thanks!


Feature/130 cloud formation as job

WIP
Doing the CloudFormation creation of S3 access points

The Further Restrictions box has been implemented
Backend db rows
API
Frontend tied

Also - any functionality that generates 'files' from the release (like Presigned Urls) will now respect this field when chosing which data types to release.

Functionality enabled is that this button can be pressed to install a CloudFormation access point in the background

It then changes to

whilst the install is happening

So this is vaguely finished - still some polishing (and for instance the "uninstall" isn't implemented). But I have the demo to AG on the 15th and I'd like this included in the demo - so would be grateful if it could be reviewed and we could look to get it merged even if not perfect.
Then I can nibble at fixes for the little issues that will come as we get to use it

The two main "algorithms" - creating the "list of allowed files for release".. and "create the access point cloudformation". I have moved these to separate helper functions and added some basic unit tests for them.


[DCOS-41674] Add test to check envvars under restart (#2677)

Add tests for Cassandra and HelloWorld to check envvars under restart

Update docker-specific files to skip rebuild of docker image in CI.

Also, if this is a backport, please mention the original PR.

Thanks for the review @porridge. You're right that this is a bit hacky. I'm going to close this and address your concerns in master instead.


Upstream PLIST handling

Motivation for this change
Need to make sure this is okay with @lnl7 first, but this is the plist generator from nix-darwin. Hopefully there is no problem with upstreaming it to Nixpkgs.
@3noch has some improvements for it that are needed by reflex-platform. I want to work with him to get this included in Nixpkgs as soon as possible (https://github.com/LnL7/nix-darwin/pull/78 still needs
some changes).

@GrahamcOfBorg build darwin.adv_cmds

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: darwin.adv_cmds
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: darwin.adv_cmds
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-darwin (full log)
Attempted: darwin.adv_cmds
Partial log (click to expand)
    /nix/store/1nzpp0fbbgbrm36v2sjxkjcpy9zjbw82-nixpkgs.xctoolchain/usr/bin/dsymutil /private/tmp/nix-build-adv_cmds-osx-10.11.6.drv-0/adv_cmds-163/Products/Release/whois -o /private/tmp/nix-build-adv_cmds-osx-10.11.6.drv-0/adv_cmds-163/Products/Release/whois.dSYM

** BUILD SUCCEEDED **
installing
post-installation fixup
gzipping man pages under /nix/store/n7bgk5p42545y5l9vkbgbp8r0p4gkcsw-adv_cmds-osx-10.11.6/share/man/
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/n7bgk5p42545y5l9vkbgbp8r0p4gkcsw-adv_cmds-osx-10.11.6/bin
patching script interpreter paths in /nix/store/n7bgk5p42545y5l9vkbgbp8r0p4gkcsw-adv_cmds-osx-10.11.6
/nix/store/n7bgk5p42545y5l9vkbgbp8r0p4gkcsw-adv_cmds-osx-10.11.6

Anyone know of a package that uses xcbuild & works in darwin?

Success on x86_64-darwin (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/mk32zacn7i3lz9anb2xji35r0xkgv43i-xcbuild-wrapper-0.1.2-pre

Success on aarch64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
building '/nix/store/aagc2wppbsxg7bsjxxqbdfgzv97hhnw1-MacOSX.platform.drv'...
building '/nix/store/j4fbrvihslh5v9jwdl0f60jrs1j3cc6n-xcbuild-wrapper-0.1.2-pre.drv'...
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre/bin
patching script interpreter paths in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre
checking for references to /build in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre...
/nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre

Success on x86_64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
building '/nix/store/h9kz2xgz2y3bb1bz355m7wgfj3pna2p6-MacOSX.platform.drv'...
building '/nix/store/vp9kiynxbhlrfr94xlr9zydrfkycdgbf-xcbuild-wrapper-0.1.2-pre.drv'...
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre/bin
patching script interpreter paths in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre
checking for references to /build in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre...
/nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre

Success on x86_64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre

Success on x86_64-darwin (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/mk32zacn7i3lz9anb2xji35r0xkgv43i-xcbuild-wrapper-0.1.2-pre

Success on aarch64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre

Ok is all caps PLIST acceptable? We can do toPlist as well if that's preferred.

Success on x86_64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre

Success on x86_64-darwin (full log)
Attempted: xcbuild
Partial log (click to expand)
copying path '/nix/store/va9b2y3kvyjdhx48jwz624kxbs0hs68g-gperf-3.1' from 'https://cache.nixos.org'...
copying path '/nix/store/j73q6p96fps8b0xwv7kjic9jglrafb6d-unifdef-2.6' from 'https://cache.nixos.org'...
building '/nix/store/4wnyzai7l4cmjhpi1xgd833wh45m5dxi-nixpkgs.xctoolchain.drv'...
building '/nix/store/ihiffqf9xpsq0dbkpaz8xpak91sd8xfn-xcbuild-wrapper-0.1.2-pre.drv'...
installing
post-installation fixup
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/mk32zacn7i3lz9anb2xji35r0xkgv43i-xcbuild-wrapper-0.1.2-pre/bin
patching script interpreter paths in /nix/store/mk32zacn7i3lz9anb2xji35r0xkgv43i-xcbuild-wrapper-0.1.2-pre
/nix/store/mk32zacn7i3lz9anb2xji35r0xkgv43i-xcbuild-wrapper-0.1.2-pre

Success on aarch64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
building '/nix/store/aagc2wppbsxg7bsjxxqbdfgzv97hhnw1-MacOSX.platform.drv'...
building '/nix/store/j4fbrvihslh5v9jwdl0f60jrs1j3cc6n-xcbuild-wrapper-0.1.2-pre.drv'...
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre/bin
patching script interpreter paths in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre
checking for references to /build in /nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre...
/nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre

Success on x86_64-darwin (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/mk32zacn7i3lz9anb2xji35r0xkgv43i-xcbuild-wrapper-0.1.2-pre

Success on aarch64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre

Success on x86_64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
building '/nix/store/20qmi4wxf2vy95mwacl0qfmjvwdj77kw-nixpkgs.xctoolchain.drv'...
building '/nix/store/79900axaq56xlzh5cyhlnvn9gw2zzy3v-xcbuild-wrapper-0.1.2-pre.drv'...
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre/bin
patching script interpreter paths in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre
checking for references to /build in /nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre...
/nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre

Success on x86_64-darwin (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/mk32zacn7i3lz9anb2xji35r0xkgv43i-xcbuild-wrapper-0.1.2-pre

Success on aarch64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/y22l4jf6bj44ly61kzr3dqwh02y92v2g-xcbuild-wrapper-0.1.2-pre

Success on x86_64-linux (full log)
Attempted: xcbuild
Partial log (click to expand)
/nix/store/5ib31pwkvwykhybg2rc235ha6jix4irw-xcbuild-wrapper-0.1.2-pre


Add virtual destructor for clang compiler

I was unable to compile the c++ code on my mac under clang. This PR adds an emtpy virtual destructor to the GenericBlock base class. I think this should enable derived objects to fully destruct, and passes clang:

Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin17.7.0
Thread model: posix

@drdavella  @perrygreenfield not sure who should review this?

awesome, thanks dan! I'm gunna go ahead and merge this...


add inline button with link to url

add inline button with link to url

@Sofbog , any updates on this?

@Sofbog , any updates on this?

fixed the commented line

Looks good 👍


Bump upjet to the commit 3afbb77

Description of your changes
This PR bump upjet to the commit 3afbb77, which fixes the issue of hiding error messages.
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

/test-examples="examples/conditionalaccess/v1beta2/location.yaml"

/test-examples="examples/groups/v1beta1/group.yaml"

/test-examples="examples/applications/v1beta2/application.yaml"


Comment node in visual scripting should move all the nodes inside

When placing comment node on the canvas, right now the node itself just serves as an outline block to visually group nodes. What I would like to see is when moving comment node, all the nodes inside the comment box should move with the comment. It would make comment box much more usable and practical.

I think the best way for organization would be a 'code collapse box' that could be used in different ways:

A commented area that can contain nodes, similar to what UE4's blueprints use. You could move or collapse this, and the node would be moved or hidden with it.

You could edit this node box in its own view. Isolated from anything else, so you don't have to worry about it.

https://github.com/godotengine/godot/issues/18267#issuecomment-382490921

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!

Fixed by https://github.com/godotengine/godot/pull/54970


feature request

Hi, is it possible to add the ability to filter by largets image?
my iphone uloaded compressed images which duplicated those uploaded by my laptop. Trouble is Its impossible to tell which duplicate is the larger image...any ideas or thoughts?

thanks for your feedback!
this feature request is already tracked here: https://ngflickrdupfinder.uservoice.com/forums/264918-general/suggestions/7640424-sort-duplicates-by-file-size-to-allow-easy-removal
unfortunately, I have not really had time to work on ng-flickrdupfinder so I don't know if it is going to happen any time soon.


Fetch release GPG key from GPG keyservers

Address https://github.com/ruimarinho/docker-bitcoin-core/issues/75. By receiving the release key from a GPG key server it becomes more difficult to perform a MITM on the binary.

@victorperin wdyt?

Seems good and simple. I just don't have permission to be a reviewer on this repo. :P
But, I can't find any way to improve this.


Blur event does not fire correctly on previously focused element when focusing the editor

Plunker demonstrating the issue.
Reproduction steps:
!) Open in ios device.
2) click the top input
3) click the editor
4) click the bottom input
5) click the editor
Notice that the focus is still in the previous input and the caret is not showing in the editor and none of the events are firing. I'm not using this wrapper in the plunker so this isn't actually an issue with the wrapper itself but I have some idea about how a fix could be added to the wrapper until they fix the issue in tinymce.
It is similar to https://github.com/tinymce/tinymce/issues/2471 and they don't seem very motivated to fix that which is why I think it could be fixed in this wrapper.

Closed as not an issue with the wrapper. I can provide the solution if you would like to fix it here.


Bug fix: We removed the CodeLens AND dot decorators for passing tests

This fixes #187 and reverts commit f614b89ea9b8be08150bbbaf04e050d5274459af.


@Samuel-Pedraza Thanks for fixing that!


Fix mail and projects methods

Fix delete_account in mail
Fix if response == String
Fix type request update_task_status in projects

:+1:


Add Extractor base class

Add a base class for extractors to remove a lot of boilerplate code
necesarry for startup/shutdown, initialization etc.
This will also tie in nicely with automatic reporting of successful or
failed runs and heartbeats to the extraction pipeline API once that is
released.

Simple usage example, creating a sine wave extractor:
from dataclasses import dataclass
from math import pi, sin
from threading import Event

import arrow

from cognite.client import CogniteClient
from cognite.extractorutils import Extractor
from cognite.extractorutils.configtools import BaseConfig, StateStoreConfig
from cognite.extractorutils.statestore import AbstractStateStore
from cognite.extractorutils.uploader import TimeSeriesUploadQueue

@dataclass
class SourceConfig:
    frequency: float

@dataclass
class ExtractorConfig:
    state_store: StateStoreConfig = StateStoreConfig()

@dataclass
class Config(BaseConfig):
    source: SourceConfig
    extractor: ExtractorConfig = ExtractorConfig()

def run(cognite: CogniteClient, states: AbstractStateStore, config: Config, stop_event: Event) -> None:
    with TimeSeriesUploadQueue(
        cdf_client=cognite,
        post_upload_function=states.post_upload_handler(),
        create_missing=True,
        max_upload_interval=30,
    ) as queue:
        while not stop_event.is_set():
            timestamp = arrow.get()
            sineval = sin(timestamp.float_timestamp * pi * config.source.frequency)
            queue.add_to_upload_queue(external_id="my-sine", datapoints=[(timestamp.int_timestamp * 1000, sineval)])
            stop_event.wait(1)

def main():
    with Extractor(
        name="My extractor", description="A test extractor", config_class=Config, run_handle=run, version="1.0.0",
    ) as extractor:
        extractor.run()

if __name__ == "__main__":
    main()

Neat! Could probably use a test.

Added a few simple tests, but parts of this is hard to do simple unit tests of. I think maybe having a small example extractor in the tests as a full system test is worth considering, which would extensively tests this module.


Medium.com incorrrect parsing

Platform: Mac
Mercury Parser Version: 2.1.0
Node Version (if a Node bug): 10
Browser Version (if a browser bug):

Expected Behavior
The parsed HTML content shouldn't include the author image
Current Behavior
Includes author image in the parsed HTML content
Steps to Reproduce
https://medium.com/flutterpub/firebase-ui-with-flutter-ccd6d4d6807

Parse any medium article
It will contain an image of the author within the parsed HTML content

Detailed Description
Ideally, the content should not include author image like any other extractor.
Possible Solution
Remove the author image from the parsed HTML content

Also, the parsed result strip subtitles (<h4>)

Is there any way, we can fix this for any other medium partner website too using the same medium.com custom extractor?


refactor: transfer state

PR Checklist
Please check if your PR fulfills the following requirements:

[X] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[X] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[X] No

Other information

 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.

HAHAHAHA!!! Nice spelling. So funny. It was consistent, though.

No kidding consistency is always the key... super easy.


Autocomplete wildcards

Would be great if I could type "__" and then invoke completion based on folder and filenames of my wildcard collection!

Run this in command prompt when you are inside wildcards folder. Then use Cusomt Scripts to add autocompletion.
for %f  in ("*.txt") do @echo %~nf
https://github.com/pythongosssss/ComfyUI-Custom-Scripts

Run this in command prompt when you are inside wildcards folder. Then use Cusomt Scripts to add autocompletion. for %f in ("*.txt") do @echo %~nf
https://github.com/pythongosssss/ComfyUI-Custom-Scripts

Oooh that's creative, great idea! But come on man, PowerShell has been around for 13 years!! (I am a bit biased, google "halr9000 powershell" to understand why.) We can do better! Also, that won't help with the underscores or recursion (all of mine are organized into folders). Hmm...
Ok this is going to be a PITA due to the relative sub-folders :D I'll hack on it for a bit. Almost got it.

Ok try this:
ls -rec -inc *.txt | Resolve-Path -Relative | % { "" + $.substring(2,$.length-6) + "" }
YES that works awesome. Thanks for the tip!

I'll leave this open in case @adieyal wants to document this tip.

Does the ComfyUI version still load from subfolders?  I can't get this node to run at all anymore because it seems to want a text input

On linux, so the powershell version isn't an option. Instead I ran the following command in my wildcards directory and pasted the output into the wildcards.txt file from the wildcards plugin I am using, at ComfyUI/custom_nodes/ComfyUI-Custom-Scripts/user/wildcards.txt
find . | sed 's/.///' | sed 's/.txt//' > fulllist.txt

On linux, so the powershell version isn't an option

Imma well acktually you here real quick, but in a good way, in case you're interested. No worries if not. :)
Pwsh has been available as "just another shell" for Linux since 2016, believe it or not! Here's an old episode of my podcast where I interviewed Jeffrey Snover on the day they launched it. I have no illusions as to its popularity on non-windows platforms, but I can tell you that the shell and language is perfectly consistent across all the supported platforms (macos also), and in fact this project supplanted the original Windows-only codebase a couple of years later.
The same team also added full support for OpenSSH into Windows. Microsoft is a much different company than they used to be. As a contrived example, from your familiar Linux box and bash shell, you could take a .ps1 script which uses a shebang to reference pwsh, a single private key or ssh agent you've configured as needed, and scp & use ssh (or ansible etc) to run that script across linux, windows, macos, and expect it to do what's in the script consistently across each platform.
References:

History of the release
Github project:: https://github.com/PowerShell/PowerShell

I was unaware of that, but also have no need of it working on multiple platforms. I could create a more involved script using zsh that would keep my wildcards autocomplete list updated in pretty much the same way if I wanted to.
But I have learned something today, so thank you

Oooh that's creative, great idea! But come on man, PowerShell has been around for 13 years!! (I am a bit biased, google "halr9000 powershell" to understand why.) We can do better! Also, that won't help with the underscores or recursion (all of mine are organized into folders). Hmm...

I feel like I've seen you before, probably when I was looking up how to do something in powershell.  Anyway I only use powershell for the sysadmin stuff it was clearly made for (like applying the 9 million 1weirdTrick registry "hacks" people list with no explanation, verifying that when MS said Pro Workstation had RDMA support for SMB3 they were very loosely defining "SMB3 and Support", and turning off memory compression like microsoft tells you to do and they should have done automatically on the machine with 512GB of ram because it isn't necessary and wasn't designed to cope with that much data.
It's very useful for things like that.  I hate the .NET-esque syntax and I'm still stuck using cmd to pipe output from ffmpeg into various programs, especially x265 since the built in library is 20-30% slower than a pipe to the external executable since that one was built with MSVC instead of the abortion that passes for a compiler that is GCC targeting Windows.
Anway I don't use this extension but here's the cmd version that preserves relative paths, run from your wildcards directory etc in case anyone wants another option.   It's similar to the original one, the only oddball portion is the weird find and replace substring-in-variable syntax in the second set command in the loop, which looks even weirder here since it has to be wrapped in a call (which forces evaluation of the %CD% hidden variable and goes farther in a batch file where you might want to put it if you're not like me and insane enough to remember how to do things like this where the %F needs to be %%F
'''cmd
for /r %F IN (*.txt) DO @(set "_o=%~dpfF" & call set "_result=%_o:%CD%=%" & @echo !_result!)
'''
EnableDelayedExpansion has to be on, so either cmd /v:on, the registry setting, or setlocal in a batch needs to be done first.  I don't think it can be done without that since the search string is a (hidden) environment variable rather than a string.  I'm sure I could write a shorter version in perl but those tend to summon elder gods so I avoid it if it's not necessary.
I'm surprised you didn't make the powershell version call the GET and POST interfaces to update the user defined wildcards text file via the web APIs:
GET /pysssss/autocomplete
returns the whole list
POST /pysssss/autocomplete
with open(file, "w", encoding="utf-8") as f:
f.write(await request.text())
I'm assuming it'll want the UTF-8 text file wrapped in JSON or some silly crap too, that's just a snippet from the code over there.  :-)  Invoke-Webrequest with a POST structure followed by streaming a file over is the example 2 in Get-Help Invoke-Webrequest --Examples so it should manage it.

Ok try this:
Get-ChildItem -Recurse -Include *.txt | Resolve-Path -Relative | ForEach-Object { "__" + $_.substring(2,$_.length-6) + "__" }  YES that works awesome. Thanks for the tip!

I am adding this for people coming to this thread but not understanding the steps:

windows explorer to your wildcards folder, then type "powershell" in the address bar and hit enter
run the code in powershell Get-ChildItem -Recurse -Include *.txt | Resolve-Path -Relative | ForEach-Object { "__" + $_.substring(2,$_.length-6) + "__" }
highlight and copy the output (Ctrl C)
go to ComfyUI in your browser and hit the gear, then hit the button do define autocomplete terms (needs ComfyUI-Custom-Scripts installed)
paste in your list of terms

Ok try this:
Get-ChildItem -Recurse -Include *.txt | Resolve-Path -Relative | ForEach-Object { "__" + $_.substring(2,$_.length-6) + "__" }  YES that works awesome. Thanks for the tip!

I am adding this for people coming to this thread but not understanding the steps:

windows explorer to your wildcards folder, then type "powershell" in the address bar and hit enter
run the code in powershell Get-ChildItem -Recurse -Include *.txt | Resolve-Path -Relative | ForEach-Object { "__" + $_.substring(2,$_.length-6) + "__" }
highlight and copy the output (Ctrl C)
go to ComfyUI in your browser and hit the gear, then hit the button do define autocomplete terms (needs ComfyUI-Custom-Scripts installed)
paste in your list of terms

Hi I follow your guide and success run autocomplete with wild txt file but what about yaml file.
Can you help me
Thank you


Fixed the MM-34128 issue

Added the support to show clear button even if the input has no value in Quickinput component

Summary

Ticket Link

Related Pull Requests

Has server changes (please link here)
Has redux changes (please link here)
Has mobile changes (please link here)

Screenshots

PR created on upstream


About Us

1. Please briefly explain what it is you coded
Implemented About Us page based on Figma design.
2. Please briefly explain your approach (new components added? modified? removed?)
Use a combination of Material UI components to code the About Us page.
3. Any packages added/updated/removed
N/A
(Optional) 4. Any feature recommendations/insights/questions/comments/etc.?

Hmm, I think you need to merge c2c-project:dev into your about-us branch.  The diff is now 4k+ files since your branch is out of date.

Ah, ok. I'll try and get that fixed later.

Hmm, I think you need to merge c2c-project:dev into your about-us branch. The diff is now 4k+ files since your branch is out of date.

Should be fixed now.

Hmm, it's still 4.5k file diff.  I think instead of a merge there needed to be a rebase.  Are you able to try that out?  This may be a good resource to read up on it https://www.atlassian.com/git/tutorials/merging-vs-rebasing
It was probably confusing that I said to merge the changes in, apologies.

Hmm, it's still 4.5k file diff. I think instead of a merge there needed to be a rebase. Are you able to try that out? This may be a good resource to read up on it https://www.atlassian.com/git/tutorials/merging-vs-rebasing
It was probably confusing that I said to merge the changes in, apologies.

Ah, ok. I'll try rebasing it.


Bluesky URL prefix should not be appended

This PR modifies

_config.yml to rename bluesky_url into bluesky_id
_includes/social.liquid to handle the renaming in the display of social media icons

Actually this is intended. Bluesky uses the AT Protocol, which is a decentralized social media network. This means that it is possible to have different services implementing this, like it is with Mastodon, and consequently urls other than https://bsky.app/profile/ for your profile.
The correct change it would be to remove this part from _includes/social.liquid and put the whole url in _config.yml. Can you do that?

Sure, that works too! I just wanted to fix the current state, which was slightly confusing

I made the appropriate change and renamed the PR


Support variadic parameters for .NET

This might be more involved that just adding the "params" keyword. Consider the following signature:
public void DoSomething(params object[] myParam)
{
}

// ...

object[] foo = // ...
// Because an object[] is also an object, the compiler treats foo as the first element
// of the params array, rather than the params array itself.
DoSomething(foo);

But in this case, there's no ambiguity:
public void DoSomething(params string[] myParam)
{
}

// ...

string[] foo = // ...

// The compiler treats foo as the params array, rather than the first element of the params array.
DoSomething(foo);

This might not matter, since generated code does not include any invocations of the generated methods. Just something to keep in mind.

I guess i'm running into a related issue to this now in 0.29 of the C# CDK
AddActions of the PolicyStatement has the following signature
[JsiiMethod("addActions"<EMAIL_ADDRESS>"[{\"name\":\"actions\",\"variadic\":true,\"type\":{\"primitive\":\"string\"}}]", false, false)]
But the generated C# method has the following signature, with a string
public virtual PolicyStatement AddActions(string actions)
Is there anyway i can create a correct string of a string params, or is this just something that need to be solved in this issue?

This has been an issue for a while.  The way I deal with this situation is to create an extension method to get the behavior I want.  For my purposes I wanted to add multiple actions via a comma delimited set of actions as a single string, but you could easily go with an IEnumerable instead.  The extension method I use is included below.  It would be better to teach JSII how to deal with variable number of arguments in C#.  Unfortunately I’m slammed on other projects at the moment, or I’d look at contributing something.
--Bonnie
    /// <summary>
    /// Adds actions to a policy statement from a comma separated list of actions names as part of initializing IAM objects.
    /// </summary>
    /// <param name="statement">What policy statement</param>
    /// <param name="actions">The names of the actions to add</param>
    /// <returns></returns>
    public static PolicyStatement AddActionsFromString(this PolicyStatement statement, string actions)
    {
        var policyStatement = statement;
        foreach (string action in actions.Split(", "))
        {
            policyStatement = policyStatement.AddAction(action);
        }
        return policyStatement;
    }

From: Erik Karlsson<EMAIL_ADDRESS>Sent: Thursday, April 25, 2019 6:15 AM
To: awslabs/jsii<EMAIL_ADDRESS>Cc: Feinberg, Bonnie<EMAIL_ADDRESS>Manual<EMAIL_ADDRESS>Subject: Re: [awslabs/jsii] Support variadic parameters for .NET (#153)
I guess i'm running into a related issue to this now in 0.29 of the C# CDK
AddActions of the PolicyStatement has the following signature
[JsiiMethod("addActions"<EMAIL_ADDRESS>"[{"name":"actions","variadic":true,"type":{"primitive":"string"}}]", false, false)]
But the generated C# method has the following signature, with a string
public virtual PolicyStatement AddActions(string actions)
Is there anyway i can create a correct string of a string params, or is this just something that need to be solved in this issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/awslabs/jsii/issues/153#issuecomment-486667767, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKCL62L6TAIWZTQYUNEVUXTPSGVGDANCNFSM4FOENHGQ.

This has been an issue for a while. The way I deal with this situation is to create an extension method to get the behavior I want. For my purposes I wanted to add multiple actions via a comma delimited set of actions as a single string, but you could easily go with an IEnumerable instead. The extension method I use is included below. It would be better to teach JSII how to deal with variable number of arguments in C#. Unfortunately I’m slammed on other projects at the moment, or I’d look at contributing something.
…
--Bonnie ///  /// Adds actions to a policy statement from a comma separated list of actions names as part of initializing IAM objects. ///  /// What policy statement /// The names of the actions to add ///  public static PolicyStatement AddActionsFromString(this PolicyStatement statement, string actions) { var policyStatement = statement; foreach (string action in actions.Split(", ")) { policyStatement = policyStatement.AddAction(action); } return policyStatement; } From: Erik Karlsson<EMAIL_ADDRESS>Sent: Thursday, April 25, 2019 6:15 AM To: awslabs/jsii<EMAIL_ADDRESS>Cc: Feinberg, Bonnie<EMAIL_ADDRESS>Manual<EMAIL_ADDRESS>Subject: Re: [awslabs/jsii] Support variadic parameters for .NET (#153) I guess i'm running into a related issue to this now in 0.29 of the C# CDK AddActions of the PolicyStatement has the following signature [JsiiMethod("addActions"<EMAIL_ADDRESS>"[{"name":"actions","variadic":true,"type":{"primitive":"string"}}]", false, false)] But the generated C# method has the following signature, with a string public virtual PolicyStatement AddActions(string actions) Is there anyway i can create a correct string of a string params, or is this just something that need to be solved in this issue? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#153 (comment)>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKCL62L6TAIWZTQYUNEVUXTPSGVGDANCNFSM4FOENHGQ.

Thank you @bfeinb
I tried something similar with a static method and looping over several actions, but then i get the following error in the app.Run() step:
Amazon.JSII.Runtime.JsiiException: 'While synthesizing mimer-ses-15/db-key/Resource: System.ArgumentException: Could not convert value 'System.Collections.Generic.Dictionary`2[System.String,System.Object]' with unrecognized type
Parameter name: value

Only reason i need it is beacuase of a related issue in the new 0.29, https://github.com/awslabs/aws-cdk/issues/2375
I guess i will have to wait for that, or just create a new statement for every action instead


Cannot Send EOS to Binance from Scatter/EOS Authority

Hi guys -
Firstly THANK YOU in advance for your help. I am moderately computer literate however the troubleshooting I have done on my own is WAY above my paygrade.
Here is what is going on.

On June 14th I was able to send 10 EOS without issue from my Authority Account connected via Scatter.

The same day I undelegated and unstaked as much of my balance as I could (leaving .5 and .1 respectively)

Currently I have 83 EOS that I am trying to send to Binance and I keep running into this error:

Transaction failed "{"code":500,"message":"Internal Service Error","error":{"code":3090003,"name":"unsatisfied_authorization","what":"Provided keys, permissions, and delays do not satisfy declared authorizations","details":[{"message":"transaction declares authority '{\"actor\":\"g42tgmzqgege\",\"permission\":\"active\"}', but does not have signatures for it under a provided delay of 0 ms, provided permissions [], provided keys [\"EOS6ZkN4MtezD6QayV7YLuYVRrx5kwoVZ7AfsbTc87HoQP8w1zsWx\"], and a delay max limit of<PHONE_NUMBER> ms","file":"authorization_manager.cpp","line_number":520,"method":"check_authorization"}]}}"

I have endeavored to do my own research but all this change Ms Cleo to this and then this command line using xyz is massively confusing.

I am running Scatter 11.0

Anyone know how to make this work?

Have you posted an issue with Scatter on their GitHub?

@Batman0323 you are using the wrong keys, see https://bloks.io/account/g42tgmzqgege#keys for the keys that are assigned to the g42tgmzqgege account.
If you are using Scatter please refer your questions to Scatter on Telegram (https://t.me/Scatter) or GitHub.


[BUG] Using Posgres Enum crash kaffy

Versions Used
Kaffy: 0.9.0
Phoenix: 1.5.0
Elixir: 1.10
What's actually happening?
When we use Postgres Enum in our schema, it crashes when we try to see the list of the elements on Kaffy interface.
Here is a short snippet of our schema
defmodule MyApp.School do
  use Ecto.Schema
  import Ecto.Changeset

  schema "schools" do
    (...)
    field :status, :string
    (...)
  end

  def changeset(school, attrs) do
    school
    (...)
    |> validate_inclusion(:status, ["active", "inactive"])
end

and our migration
def up do
    execute "create type school_status as enum ('active', 'inactive')"

    create table(:schools, primary_key: false) do
      (...)
      add :status, :school_status, null: false
    end
end

We found that if you try to use status::text like '%%' on your query instead status like '%%' it should work.
We tried to use custom query to solve this problem but there is a bypass of the original query and we can't use our query.
Here is a snipet of the code we suspsect are causing this issue (module Kaffy.ResourceQuery, lines 17 to 33)
{all, paged} =
      build_query(
        schema,
        search_fields,
        filtered_fields,
        search,
        per_page,
        ordering,
        current_offset
      )

    custom_query = Kaffy.ResourceAdmin.custom_index_query(conn, resource, paged)
    current_page = Kaffy.Utils.repo().all(custom_query)

    do_cache = if search == "" and Enum.empty?(filtered_fields), do: true, else: false
    all_count = cached_total_count(schema, do_cache, all)
    {all_count, current_page}

What should happen instead?
It should list all the elements that has enum on posgresql
Screenshots

A possible solution to this problem may be to cast all fields as string in order to perform ilike operation, like so

Thank you @Rachc for the detailed report. I really appreciate it.
@brosquinha this could work! Thanks.
I expect this to be fixed with v0.9.1 which should be released in a few days.

Thank you @Rachc for the detailed report. I really appreciate it.
@brosquinha this could work! Thanks.
I expect this to be fixed with v0.9.1 which should be released in a few days.

Should I open a PR with these changes?

Feel free to tackle this. But one thing to keep in mind is that v0.9.1 should also fix #134 which allows searching numeric fields as well.

@brosquinha on second thought, its' fine to submit a PR specifically for this issue since there's already partial work done for #134. We'll try to combine the two solutions later on.

#139 closes this one


More cleanup and realignment

Looks good. I think for the future we can remove the_findmin and other _ functions that were there for compatibility with 1.6's lack of "function first argument" support.

Looks good. I think for the future we can remove the_findmin and other _ functions that were there for compatibility with 1.6's lack of "function first argument" support.

Thanks. I implemented the _ functions because I stumbled about random allocations when using Base with newer compilers.


cmd/compile: spurious line info added to interface satisfaction error

% cat /tmp/x.go
package p

import (
	"fmt"
	"strings"
)

func main() {
	var b strings.Builder
	fmt.Fprintf(b, "hello")
}
% go tool compile -G=0 /tmp/x.go
/tmp/x.go:10:13: cannot use b (type strings.Builder) as type io.Writer in argument to fmt.Fprintf:
	strings.Builder does not implement io.Writer (Write method has pointer receiver)
% go tool compile /tmp/x.go
/tmp/x.go:10:14: cannot use b (variable of type strings.Builder) as type io.Writer in argument to fmt.Fprintf:
	strings.Builder does not implement io.Writer (Write method has pointer receiver) at $GOROOT/src/strings/builder.go:88:19
%

The $GOROOT makes the added line info not useful.
I am not sure where that's coming from.
But it's probably also just not useful in general:
much of the time, the type will be in a dependency
somewhere, and the user is not really helped
by printing the path of some file in the module cache.
The "at ..." should be deleted.
Not a beta blocker.

@rsc do you mind if I work on this incase it's not too complicated for a beginner contributor?

@AGMETEOR Feel free to take a look, but it's very likely that this will be fixed by one of the compiler developers in a week or two.

CC @griesemer @findleyr

This was introduced by CL 373634 which fixed two issues that wanted the line info...
Will remove.


MiniMisc.zoom() when setup with mini.animate causes a bug where command line renders in between buffer.

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.misc, mini.animate
Description
Basically, during scrolling animation in :lua MiniMisc.zoom() mode, the commandline renders at the center of the screen till the scroll animation is done.
Neovim version
NVIM v0.11.0-dev-1341+g5eda7aafe9
Steps to reproduce
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
	spec = {
		"echasnovski/mini.animate",
		"echasnovski/mini.misc",
	},
})

vim.o.number = true

require("mini.animate").setup()
require("mini.misc").setup()

Open a file which is big enough to be scrollable for atleast 2 seconds with the following command nvim -u repro.lua bigfile.lua.
Run the command :lua MiniMisc.zoom().
type ggG in normal mode to scroll from top of the file to bottom of the file. You can see that command line is rendered at the center of the screen during scrolling.

Expected behavior
Commandline should not be rendered at the center of the screen during scrolling animation.
Actual behavior
Commandline is rendered at the center of the screen during scrolling animation.

Thanks for the issue!
I can not reproduce. In particular, because "Open a file which is big enough to be scrollable for atleast 2 seconds ..." is not really possible with the provided 'init.lua' as it defines 250 milliseconds total scroll time.
Even with 'mini.animate' configured to have scroll of 2000 milliseconds, it is still not reproducible.
What makes you think that this is command line that is rendered on top? Could you make a screencast, please?

I apologize for not actually uploading a screencast earlier since without it this bug wouldn't make sense and is ridiculous.
https://drive.google.com/file/d/1ljBqCHl2V3PJ1Z_TujeXWgurYYMCew4r/view?usp=sharing
I am posting the google drive link since the video is over 10mb and github is not allowing me to upload.
But in this video you can see that :lua MiniMisc.zoom() appears in the middle of the screen while scrolling. By cmdline appearing in middle, that is what I meant.
As for the file used in scrolling
https://github.com/231tr0n/config/blob/main/nvim/init.lua
repro.lua is the same as above. With this setup I am able to reproduce it.

Thanks for the issue!
I can not reproduce. In particular, because "Open a file which is big enough to be scrollable for atleast 2 seconds ..." is not really possible with the provided 'init.lua' as it defines 250 milliseconds total scroll time.
Even with 'mini.animate' configured to have scroll of 2000 milliseconds, it is still not reproducible.
What makes you think that this is command line that is rendered on top? Could you make a screencast, please?

Also about the scrolling, yeah my bad I meant file enough to cause a scrolling effect. I termed it wrongly i guess.

Yes, I was able to reproduce. It appeared as a 'mini.misc' issue: zoom() window was opened with very big dimensions to not have to compute them. Big height in particular lead to this issue of redrawing command line, presumably because window was drawn beneath it. Properly computing and updating dimensions seems to fix this on latest main.
Thanks again for finding this!

Yes, I was able to reproduce. It appeared as a 'mini.misc' issue: zoom() window was opened with very big dimensions to not have to compute them. Big height in particular lead to this issue of redrawing command line, presumably because window was drawn beneath it. Properly computing and updating dimensions seems to fix this on latest main.
Thanks again for finding this!

As always, awesome work and fastest fixes.  Hope I become a software engineer like you one day!!


Use stable and semantic version constrains

should go to 1.0 branch

Sorry but should go to the "welcome" branch. Master is the current default branch where the browser lands and the install instructions MUST be correct in that webpage.

but it should still go to 1.0 branch.
then, master is synchronized with 1.0

replaced by #211, thank you @Maks3w


Tests background gradient

Is this the right way to test gradient generation? Did I miss something?

It doesn't test whether the steps are correct or whether the correct colors are drawn though?

Of curse, this will need to rewrite Generator.Canvas code. Would be the next step i will do. Than it´s possible to write new tests. But this one will check if it renders "something" or fails (blank image). Is this ok by now?
Should this code be moved to a new test,js file? Cause it´s not CSS "module" but "Generator" instead?

regexp could be:
if (/^(-webkit|-o|-moz|-ms|linear)-/.test(background_image)) {
´´´

I updated the tests/preload accordingly. It skips the test now for browsers that don't support gradients

move Generate tests into own module and fixed some small things:
https://github.com/SunboX/html2canvas/commits/tests-background-gradient
Will look into color-stops this week (+ add some tests)


Not able to access UI when starting the app from linux vm

Hi,
I have a app.py script, same script when i run with command chainlit run app.py -w after setting (API Key/Auth Secret etc) is working from my local but when i start the same app from a linux vm i am not able to access the app UI. I am using my_ip:8000 to access but still not able to access. How can i fix this. I am trying to host from a remote machine because i want my entire team to access the app.
Appreciate any help, Thanks.

Can you try adding -h to your start command and try again?

Thanks @willydouhard . Worked. Closing this.


Self-references nested inside data type do not appear to update on update_forward_refs()

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Ref: https://pydantic-docs.helpmanual.io/usage/postponed_annotations/
Sample code:
from pydantic import BaseModel

class Order(BaseModel):
    legs: Optional[list["Order"]]

Order.update_forward_refs()

Traceback:
    class Order(BaseModel):
pydantic/main.py:293: in pydantic.main.ModelMetaclass.__new__
pydantic/fields.py:410: in pydantic.fields.ModelField.infer
pydantic/fields.py:342: in pydantic.fields.ModelField.__init__
pydantic/fields.py:450: in pydantic.fields.ModelField.prepare
pydantic/fields.py:545: in pydantic.fields.ModelField._type_analysis
pydantic/fields.py:629: in pydantic.fields.ModelField._type_analysis
pydantic/fields.py:648: in pydantic.fields.ModelField._create_sub_type
pydantic/fields.py:342: in pydantic.fields.ModelField.__init__
pydantic/fields.py:456: in pydantic.fields.ModelField.prepare
pydantic/fields.py:670: in pydantic.fields.ModelField.populate_validators
pydantic/validators.py:710: in find_validators
E   RuntimeError: error checking inheritance of 'Order' (type: str)

Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.8.1
            pydantic compiled: True
                 install path: /home/adys/.cache/pypoetry/virtualenvs/alpaca-api-client-JFdRWl93-py3.9/lib/python3.9/site-packages/pydantic
               python version: 3.9.5 (default, May 12 2021, 17:14:51)  [GCC 10.2.0]
                     platform: Linux-5.12.3-arch1-1-x86_64-with-glibc2.33
     optional deps. installed: ['typing-extensions']

The correct way to define self-referencing model will be:
from typing import Optional
from pydantic import BaseModel

class Order(BaseModel):
    legs: "Optional[list[Order]]"

Order.update_forward_refs()

Or
from __future__ import annotations
from typing import Optional
from pydantic import BaseModel

class Order(BaseModel):
    legs: Optional[list[Order]]

Order.update_forward_refs()

Or
from typing import Optional
from pydantic import BaseModel
from pydantic.typing import ForwardRef

class Order(BaseModel):
    legs: Optional[list[ForwardRef("Order")]]

Order.update_forward_refs()

@uriyyo That second one would not work due to python syntax (at least in 3.9).

@jleclanche It will work, I have checked it on a python 3.9 version.
There is a from __future__ import annotations on top of the example, so all annotations will be converted into str and it is a totally valid syntax.

My bad, you're right :)

But there is a bug that I did not notice when I was writing my first comment.
When we are using typing.List it will automatically wrap all str into ForwardRef:
>>> List["int"]
typing.List[ForwardRef('int')]

But when we are using regular list it will not be wrapped:
>>> list["int"]
list['int']

I have opened PR which should handle such behavior difference.


allow customized installation path

You should give the user the opportunity to customize the installation path.

Feature added in 5039c24 and will be available in the next update

The correct commit id is 456f77389e05fe36808f3173a6ad28cfc3c15bec, sorry for confusion.


Backend Homework 2

Backend Komework 2
Description:
constructor
packages
strings
control flow statements
Drift link : https://video.drift.com/v/abPCauEANLzgF94aI7jn0KnrvcVajoLv8ZyVBkoB6Jw/
Snapshots:
question-1:

question 2 :

question 3:

Metric
Value

alert_status
:white_check_mark: OK

bugs
9

code_smells
40

reliability_rating
C (at least 1 Major Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Create and associate a 'Monitored Object'

The paragraph "Create and associate a 'Monitored Object'" fails to explain how to create a "Monitored Object".

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

ID: 744bf603-c4e7-5ffe-90ad-11735813335b
Version Independent ID: 518ed1e0-13b8-0f85-03e8-d729ec99d290
Content: Set up the Azure Monitor agent on Windows client devices (Preview) - Azure Monitor
Content Source: articles/azure-monitor/agents/azure-monitor-agent-windows-client.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @shseth
Microsoft Alias: shseth

Hi @TBSJordanNash, thanks for bringing this to our attention. I have assigned the issue to the content author to evaluate and update this doc as appropriate

cc: @shayoniseth

@TBSJordanNash Thanks for providing feedback that helps improve our documentation! This issue has been resolved and we are closing the issue. #please-close

Would you mind also giving some insight how to remove or change the region of the monitored object? In the script that was provided on the docs page some of our technical engineers ran it with the default location value.

@nbozov Thanks for the feedback! Of course. I'm opening an internal ticket for this. I'll notify you here when this update is live.

I too have had this issue - During deployment, I accidentally didn't set the location and now I can't seam to find a way to update it. I've tried using AzureCLI az monitor data-collection rule update but the --location tag is not found. Running the script a second time just returns an error of "The monitored object resource already exists in a different location 'eastus' and cannot be moved."
Any help on what the method is to update it? I've deleted the collection rule and tried to associate it to a new rule and the error still persists so this "Monitored Object" is out there, somewhere......

Would you mind also giving some insight how to remove or change the region of the monitored object? In the script that was provided on the docs page some of our technical engineers ran it with the default location value.

Did you ever figure this out?  I've even tried deleting the Collection Rule but it keeps saying that the object already exists, however I can't find the object to remove it and no commands available to update it.  Thanks.

Leave this for anyone who needs it, you can use the API Playground to easily delete the key -
Open API Playground in Azure Portal - https://portal.azure.com/#view/Microsoft_Azure_Resources/ArmPlayground
Perfrom a GET operation on providers/Microsoft.Insights/monitoredObjects/$TenantID?api-version=2021-09-01-preview & verify that the location is incorrect use a DEL operation on providers/Microsoft.Insights/monitoredObjects/$TenantID?api-version=2021-09-01-preview
Make sure to replace the tenant ID with your actual tenant ID
This will delete the entire key so use with caution, but for my use case where I could not update it, once I deleted it I was able to re-run the script with the correct location set and continue on. Hope it helps someone save at least a day trying to figure this out ;P


Fixed positive difference section in Volunteers

and also cleaned up warnings in Sponsors

Looks good man! Can you just redirect the contact button to the proper page for this section? Currently , the button is not doing anything.

Bless, i forgot that, good catch


Issue with docs.zowe.org/stable/user-guide/update-zos/

outdated update.sh script documented

The document and the script is not in use from zowe v1.2
Description

The Outdated document needs to be removed, last used in ZOWE 1.2
Pages to Update

https://docs.zowe.org/stable/user-guide/update-zos/#update-zowe-using-the-update-script
https://github.com/zowe/docs-site/blob/c1f5e20bc21d98bfa5a608a3f077198a1b62742d/docs/user-guide/update-zos.md
Screenshots

Expected behavior

The document should not exist
Additional context

Removed. Thank you!


aarch64-darwin boost159 - fail to build

Trying to install mysql5.7 on a MacBook M1, but it fails complaining about the C compiler (clang) being too recent
Steps to reproduce
On a machine with Apple M1 chipset, simply run:
nix-env -iA nixpkgs.mysql57

and the output of this command will be:
→ nix-env -iA nixpkgs.mysql57
installing 'mysql-5.7.27'
error: assertion (with lib; (((toolset == "clang") && (! ((versionOlder  (stdenv).cc.version)  "8.0.0"))) -> (! ((versionOlder  version)  "1.69")))) failed at /nix/store/sf3lb1jxaj8gid7lyavcdn1jyp6r0bph-nixpkgs-21.11pre312229.08ef0f28e3a/nixpkgs/pkgs/development/libraries/boost/generic.nix:34:1
(use '--show-trace' to show detailed location information)

Technical details
 - system: `"aarch64-darwin"`
 - host os: `Darwin 20.6.0, macOS 11.5.2`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.15`
 - channels(damien): `"darwin, home-manager, nixpkgs-21.11pre312229.08ef0f28e3a"`
 - nixpkgs: `/Users/damien/.nix-defexpr/channels/nixpkgs`

Changed the description - boost builds fine (boost169), it's the older version that is affected (boost159).

@damien-biasotto I don't see a quick way out of this, atm. Can you use mysql80 or mariadb instead in the meanwhile?

@prusnak I actually went the MariaDB way, but faced other pitfalls, that's what you get for using outdated software :P
Thanks for updating the title.
Feel free to close the issue.

@prusnak would it be possible to pin an older version of nixpkgs? I have two devices here: one with an old, patched version of nixpkgs where everything builds fine and an up to date version where it fails to build with the provided error.

I have two devices here

Are you sure these two devices are aarch64-darwin (macOS running on Apple Silicon)?

Is there any way to check this?


Environment variables not set persistently on Mac OSx in the recent versions.

Type of issue

[ ] Bug
[ ] New feature
[ ] Enhancement

Description
For local deployment, cli sets environment variables on the machine. The way to set these variables persistently on Mac OSx seems to have changed in the current version of the OS.
Hence, the cli needs to be modified accordingly.
...
Steps to reproduce
Build cli tool on Mac OSx

pcs -s local
open a new terminal and check if the env variables are set (look out for keyvault name)

Expected behavior
Variables set
Current behavior <!-- The actual behavior observed --
variables not set
Known workarounds
Manually set the variables from .env file in .pcs cache.
Possible solution
...
Context and Environment

Operating System:  Mac OSx Mojave
GitHub branch: ...
Node.js Runtime: ...

PRs for doc update
https://github.com/Azure/remote-monitoring-services-dotnet/pull/241
https://github.com/Azure/remote-monitoring-services-java/pull/136/files

Github Docs update complete.


chrony: 4.3 -> 4.4, optional mlock, add test

Description of changes

Bump chrony from 4.3 to 4.4.
Add option to disable memory locking, fixes #222629.
Add test

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test chrony chrony-ptp


just-in-time (JIT) debugging instead of this dialog box.

Everytime, when I am starting the software under Windows , even on different machines I got:
Unhandled exception has occured....
just-in-time (JIT) debugging instead of this dialog box.

@coinsinternet please post the full output of the error, not an abbreviated version.

Closing due to no response. If this problem persists, please reopen the issue.


Self hosted on old computer, elastic search using lots of cpu

Hi,
Thanks for newsblur, I have only been testing it for a few days, but seems really awesome!
I am testing it on an old laptop that I have for "self hostings", and elastic search seems to pop up every 30 seconds and cause massive cpu usage!
Is there a way to, without disabling elastic search, but to relax the schedule? I haven't even used the search functionality, so I don't care if "search is very up to date".
Thanks again !

It's kind of all or nothing. You can certainly turn off elasticsearch and other than the feed fetchers complaining in the logs, it'll work fine. There's some fine tuning we could be doing, but I don't see ES popping up often enough to have it high on the priority list of latency issues.

thanks @samuelclay for the reply, and sorry for my own late reply.
I managed to disable ES and not have any error logs by removing it from several functions ( python is not my primary language, but I have some experience in python ).
Maybe there is something misconfigured on my end, but ES was popping up all the time and making the cpu almost always at 100% (which , granted, its an old computer). But without ES, it runs fine without ever reaching 100% cpu (or even 50% usage).
If it is a "all or nothing", and search being of course important, I understand and keep this issue closed.
Thank you for your time, and thank you for newsblur, its just awesome !

It's kind of all or nothing. You can certainly turn off elasticsearch and other than the feed fetchers complaining in the logs, it'll work fine. There's some fine tuning we could be doing, but I don't see ES popping up often enough to have it high on the priority list of latency issues.

Hello!
I also faced this problem.
I stopped the container, everything seems to be working.
Please tell me what this service is responsible for and what are the consequences of disabling it?


Add Manage functionality and unzip feature.

View inflations are also changed into view bindings.

Great, I'll issue the drak mode issue latter.


FIX [mqtt/websocket] a potential blocking error https://github.com/…

related to https://github.com/emqx/nanomq/issues/1210