Add ability to specify features in CLI

Resolves #321.
Resolves #181.

:fire:

Another two things to consider:

What happens if a new feature is added in "current" but the same feature doesn't exist in "baseline"? Attempting to generate baseline rustdoc with that feature enabled would fail, since the feature isn't defined there.
Is there a possible and easy tweak of the CLI that would let us support the use case in #412?

For both of these, please don't implement anything yet and let's just discuss possible options and designs first. It'll be harder to think of good solutions if you've already gone ahead and implemented a particular approach.
With that in mind, what do you think?

If a feature doesn't exist in "baseline", it could just be ignored. This seems quite easy to implement.
For the mentioned use case, maybe we could add another flag, that adds features to the "current" version, something like --feature-current? I think it makes more sense this way, as the ususal use case is to test the crates on the same feature set. Also, you never want "baseline" feature set to be a superset of "current" feature set, right?

If a feature doesn't exist in "baseline", it could just be ignored. This seems quite easy to implement.

This seems reasonable to me, though we should probably print a warning message about it to make sure the user is aware.

For the mentioned use case, maybe we could add another flag, that adds features to the "current" version, something like --feature-current? I think it makes more sense this way, as the ususal use case is to test the crates on the same feature set. Also, you never want "baseline" feature set to be a superset of "current" feature set, right?

"Never" is a very strong word, I wouldn't say never. For example, maybe a library had an optional feature in v1 that is included by default in v1.1, and wants to check that the API is unchanged between v1+feature and v1.1 without the feature.
If we're adding a flag to affect the features in only one of current/baseline, we should add the symmetrical flag for the other one too.
A few other, smaller things:

cargo uses the plural "features" for its feature-related flags (e.g. in cargo add), so for consistency let's use features as well and make it work the same way as in cargo add. Since we're planning on merging into cargo, it's important that we minimize friction like this.
Our other flags that are split between current/baseline use the word "current" or "baseline" to start, so let's call the flags --baseline-features and --current-features instead of using "current" or "baseline" as a suffix.
For the feature-related flags that affect both current and baseline, let's make sure to explicitly document this fact in their help text.

@obi1kenobi how do I deal with clippy::too_many_argument lint on generate_versioned_crate function? I know that having that many arguments is not ideal, but I don't think that creating a struct to pack all the parameters there makes much more sense.

@obi1kenobi how do I deal with clippy::too_many_argument lint on generate_versioned_crate function? I know that having that many arguments is not ideal, but I don't think that creating a struct to pack all the parameters there makes much more sense.

Making a single struct to pack everything doesn't make sense, but perhaps we can group the arguments somehow? For example, consider whether any arguments are always used together with each other: if the function takes an a and b, and then whenever it passes a somewhere it also passes b, then maybe a and b should be combined.
Then see if it makes sense to move one inside the other and if so, in which direction. Or if not, see if it makes sense to make a struct that holds both of them.
This is a skill that takes practice to develop, so I don't want to do you the disservice of just giving you the answer and robbing you of an excellent opportunity to practice the skill in a safe space.

Ignore the nightly failure for now, there's a new rustdoc format version (v25) and I'm waiting on rustdoc-types to release a corresponding new version before I can release the matching new adapter version for it.

I really like the changes I made thanks to the review!

I merged main to make nightly start passing again.

Awesome job 🎉
In the next PR, could you add a few examples of the new functionality to the README? That way they'll show up on crates.io in the next version and I'll also use them for the release notes and for the release thread on Twitter / Mastodon.


TOC printed in appendix B without specifying this

I don't understand why in the PDF version a table of contents (TOC) is inserted in Appendix B, immediately after the title of Appendix B.
Additional information: this does not happen in Appendix A; appendix B has to subsections: B1 and B2.

Could you share a sample adoc and pdf file to see and test it.

I've seen a similar error with 1.4.2 when using the following document fragment:
= AsciidocFX 1.4.2
:toc:

[cols="1,5a"]
|===

| A | B

| A
|
* abc
* def

|===

It is displayed as:

Hi Rahmanusta,
Please find attached an asciidoc test document, which is anonymized version of the document I am working on. The package also contains a source image and the PDF version. The file is a zip file, but I have changed the extension to .txt because it seems that github does not allow attaching zip files.
You will see on page 17 that a TOC is inserted right after the title of Appendix B.
Also, please note the empty pages inserted before and after the good TOC inserted on page iii.
Moreover, for some unknown reason another empty page is inserted right after page 3.
Furthermore, the image on page 5 is for some reason shrank inconveniently, despite specifying for it a width of 400 pixels in the asciidoc source file. Also, another empty page is inserted right after page 5.
I also would have liked to have the long list on page 11 placed on two or three columns, without the use of a table, but I could not see this being a feature of asciidoc.
Best wishes,
Daniel
Test_document.zip.txt

Please follow the link to see workaround asciidoctor/asciidoctor#1582


Various improvements

Hi! I opened this PR by mistake. Most of what I did was add support for other content (videos and audio) and customize the UI to fit my preferences (mainly dark mode, but also small issues like spacing).
If you are interested in these changes, I can take a look at your fork and make smaller, more manageable PRs.

Yes please ! I'd be very glad if you could indeed bring the improvements such as the spacing. I also see you've made the names more generic so that's fine by me too. I'm also not forcing you to do anything.
Thanks for answering.


Fix #662: Handle undefined files array and handle empty git directories in Quic…

…kOpen.ts
Based on Issue 662:

The issue was also found in Linux and seems to popup when either a directory with inaccessible sub-dirs is <C-p>'d on or an empty git directory is <C-p>'d on

"Infinite loading" was/is a result of no data coming in from the spawn process. This is fixed by checking the exit code of filer and checking the array length of this._loadedItems

If both are 0 push the default "Bookmark" and "Open a folder" options to the menu

Moved some logic in show() into its own private function because that code is now being used in two other places

Check the array length for the non-git option (filer.stderr) for directory parsing. It's a simple undefined check then pushing the default menu options to the this._loadedItems array if the files array isn't present.

I hope this helps some. I noticed the team has commented on removing some of the areas of code that I touched in favor of async functions. I'd like to work on this too, at some point, but I think this PR gives a decent/quick solution to #662.
Hope this helps!

Awesome, thanks for the PR, @someguynamedmatt ! Handling the exit code makes sense to me - seems like we can detect several of the cases where we'd stick on "Loading..." with that.

Moved some logic in show() into its own private function because that code is now being used in two other places

Excellent!

@extr0py Changes made!

Excelllent! Just tried it out. Looks great, thanks for making the changes & the contribution, @someguynamedmatt - merging in now!


dart2js does not throw exception when truncate-dividing by 0

test language/tests_language_div_with_power_of_two_test.dart

The test is no longer failing in 79a8654e5fb30dfce902636961d6a5ffb69e6f85


Bug 2092815: fix(pruning): allows continue-on-error to be used with pruning operat…

…ions
Currently, if pruning errors are transport errors, only a warning is logged.
For users who want the pruning functionality, this can cause errors on runs
if the registry or authorization is not configured to handle deletion. Instead
of logging an error, an error is returned when pruning operations fail, but
if the error is a transport error, it can be ignored with continue-on-error.
Signed-off-by: Jennifer Power<EMAIL_ADDRESS>Description
This PR changes the way errors are handled during pruning operations. Transport errors are currently logged instead of returned. This change returns the errors by default but allows them to be skipped with continue-on-error.
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Docs Updated

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Unit tests added to test continue-on-error

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

/retest

/lgtm


Windows XAMP

Only First  Page Open

does this project work or no???

unable to get past the first page

please upload a detailed instruction how to install this tastyinger


feat: attach circuit preview image to project during create and update actions

Fixes #
Describe the changes you have made in this PR -

create Method: Added parsing and attachment of circuit preview image.
update_circuit Method: Integrated image parsing and attachment for updates.
New Method: attach_circuit_preview to handle image file attachment with unique filenames.

This would attach the image preview for circuits made on vue simulator.
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Summary by CodeRabbit

New Features

Improved image upload functionality for project creation and updates, allowing users to attach circuit preview images seamlessly.

Bug Fixes

Enhanced robustness in image handling, ensuring successful attachment of image files during project updates.

cc @tachyons

@coderabbitai Could you generate unit tests for this PR ?


button component isDisabled getter

sorry that previous fix is incomplete, missed out during test

What is the issue?

in my blade file, i'm using this component as below
@php
            $isDisabled = $isDisabled();
        @endphp
        <x-forms::button
            type="button"
            x-on:click="{{ 'state += 1' }}"
            disabled="{{ $isDisabled }}"
            color="{{ $getColor() }}"
            :attributes="$getExtraAttributeBag()"
        >
            {{ $getLabel() }}
        </x-forms::button>

so the disabled property can be changed based on condition, but the opacity is not removed even with getter
i tried to fix with
x-bind:class="{ 'opacity-70': isUploadingFile || isDisabled(), ... }"

this works but there are a bunch of console error

Can you test my changes?

with my latest changes then it works well with the disabled state being toggled multiple times
i also tested with FileUpload field. There is a opacity problem with the following steps:

show button on visible but disabled
change button to enabled
*up to now things works fine
upload file
*here the button doesn't change opacity but is not clickable

not sure how to proceed with a fix

I don't understand why the disabled state needs to be in Alpine, it's purely affected by Blade and having it stored in two places is obviously causing issues, thats why I removed it

I'm so sorry Dan, while testing on first time something is not right, now when i test again it works perfectly fine. 🤦🏻♂

Its ok! Glad it works :)


Implement Abstract Class Example

Is your feature request related to a problem? Please describe.
No, but I think this is a good starter project and having an example of abstract class would encourage developers to follow best practice.
Describe the solution you'd like
Just for abstracting Id, dates and active properties,  I recommend to have "common entity" folder, there create one abstract class called "base" and other "activeBase".
import {CreateDateColumn, UpdateDateColumn, BaseEntity, PrimaryGeneratedColumn} from 'typeorm';

export abstract class Base extends BaseEntity {

  @PrimaryGeneratedColumn('uuid') public id: string;
  @CreateDateColumn() public createdAt?: Date;
  @UpdateDateColumn() public updatedAt?: Date;
}

and
import {Column} from 'typeorm';
import {Base} from "./base.entity";

export abstract class ActiveBase extends Base {

    @Column({ default: true })
    isActive: boolean;
}

Finally, User or other entities can extend ActiveBase or Base.
Describe alternatives you've considered
Additional context
What do you think?

@imomin
Yes, I think this is a great idea. If you want you can assign yourself this ticket and open a PR.

I've made the changes but i'm unable to push to changes.

Closed with PR #37


Migrate to the new figure-linux-standard action runners

[!WARNING]
📣 Action Required
The old runners will stop working on 2024-02-22, please approve and merge this PR!
If this PR does not properly build, please let us know in slack - #devops.

This PR moves all of the workflows from our old self-hosted runners and ubuntu-latest to our new figure-linux-standard action runners. The new runners do cool things like autoscaling, so eventually our goal is to have more runners available for devs while also spending less money.
We highly recommend wiping the Actions cache for this repo before merging this PR. We've seen some issues restoring a cache made by an old runner on a new runner, which can result in dramatically slower builds. The first build on a default branch on the new runner after wiping the cache will be slow, but subsequent builds will be faster since they'll have a proper cache restore. This can be done using the gh cli (brew install gh):
gh cache delete --all -R FigureTechnologies/restricted-marker-transfer-smart-contract

Check out our Notion Blog post for more info and our FAQ.

🤖🪄 This PR was automatically created by a magic script. Please reach out to #devops with questions.
[sc-281866]

I think we need to kill this PR. 🔪 See https://figure-group.slack.com/archives/CB721JFFC/p1706812906287469

Can't use the new runners in public repos


rshell takes too much time to communicate

My rsync, for example, takes 40 seconds or so with a very few folders. Partially because every single remote command resets the module and re-compiles (small) chunks of code.

Yep - I agree. When I originally wrote rshell, it was only running with the pyboard, which has a direct USB connection. MCUs like the EPS32 or ESP8266 which don't have direct USB (and use an external USB-to-serial chip) suffer from this problem more than the other devices.
I've always wanted to implement a comms layer which gets uploaded to the remote device once and have rshell communicate with the remote device using that, but that would require some fairly extensive changes,

I fixed https://github.com/dhylands/rshell/issues/27 and some other people's issues (or so I believe), with a -s, '--suppress-reset` command-line option. Not only was it slowing things down, it was also messing with people whose board is in a state where something doesn't cleanly unload and the gc turns the soft-reboot into a hard one, which messes with the send/expect sequences in pyboard.
Pull Request is in, see the issue for all the details.


Update .travis.yml

Do not use a master version of FEMQuad.jl but newest release instead.

0.7 fails, Julia 1 and nightly pass. I think no-one is using 0.7 anymore, should we just drop 0.7 support? A quick look of the error log is not telling me anything.
CC: @TeroFrondelius, @jvaara

Yes, let's drop 0.7.


Task 26201472: Add negative test scenarios for dynamic type loading feature

Overview

Adds test cases to increase code coverage
Refactor diagnostic propagation for loading of providers with corrupted data
Simplify test code and reduce code dupliction

Microsoft Reviewers: Open in CodeFlow

Jumped the gun here, I will create another PR with the fixes


Definition of 'Protocol' must be imported from module 'ObjectiveC.Protocol' before it is required

I get the following error when compiling a project that has this library as a dependency:
MAObjCRuntime/RTProtocol.m:79:13: Definition of 'Protocol' must be imported from module 'ObjectiveC.Protocol' before it is required

It seems to only happen with XCode 7.3, running 7.2 side by side compiles and runs just fine. Wondering if something changed in 7.3 or the compiler that causes these issues?

Hit this issue when trying to compile after updating XCode. Any known workaround?

I imported objc/Protocol.h at the top of RTProtocol.m and it now compiles.
#import <objc/Protocol.h>


implement parallax occlusion mapping for textures.

[what is parallax occlusion mapping?]
Parallax occlusion mapping is used to create 3D definition in textured surfaces, using a displacement map (similar to a topography map) instead of through the generation of new geometry.
(source wiki : https://en.wikipedia.org/wiki/Parallax_occlusion_mapping)

As you can see it adds a visual improvement to the texture and gives extra detail without too much of a performance impact, also I think there is multiple ways of getting the desired effect, with each option being more complex then the other, this youtube video on unreal seems to lightly show the difference. (https://www.youtube.com/watch?v=h0xytNFiqbQ)
also I came from the youtube video, who suggested to add this in the comments I felt it needed its own dedicated post here.

Hey @Dragon20C , that's super interesting!
I did make some tests :
Without

With

My only concern right now is that is seems pretty consuming in term of perfs.
Maybe with only one texture it would be easier, but it needs to be applied to all the textures on the terrain, which seems heavy 🫤
Maybe you have better ways to do it than the way I did?
I'm open to suggestions 🙂

Hello @spimort the results looks really amazing, I am no expert in this area, but I can give my opinion on maybe what could be done, so I dont know how is everything applied for the parallax, are you using a distance function to slowly apply the parallax effect only to close objects, this could improve the fps a little, also I am not sure how the texture painting works, but would it be possible to separate the textures using a shader so each material is using a custom shader that does all the texture applying, though I dont know if this is a good idea, I will try to do some research on the topic and see if I can find some interesting results, anyways excellent results!

Thanks @Dragon20C !
I think that one solution would be to allow the user to enable it just for some textures instead of all of them.
And yeah, maybe with some distance that would be better, I agree!
The only thing I tried so far was to implement the code that Godot uses to handle the height thing in the standard material.

But maybe there are other algorithms, I did not do a crazy search about all of this.
I did a pending branch that contains what I've done: https://github.com/spimort/TerraBrush/tree/feature/add-texture-parallax
It's a rough one, stuff are hard coded in the shader right now, just to do the testing.

hello @spimort I am not familiar on how to enable parallax occlusion mapping in the terrabrush node, I made a mesh instance node to make sure the textures work, how do I enable the height in the terrabrush node.

Mesh instance

Terrabrush terrain

Oh my bad haha, the parallax thing was commented out!
I pushed the shader back.
I haven't do anything else on it as of right now.
Also, keep in mind that this branch is only an experiment. Everything is enabled by default, you should see it if you pull the branch.


Get-CertificationAuthority produces System.IO.IOException: The network path was not found.

I found issue #25 and tried several of the steps outlined there for troubleshooting. I have two CA enviroments in two seperate forests (test and prod). Both had issue. Here are some masked results. Both CAs are running on Server 2016.
[PKI.CertificateServices.CertificateAuthority]::ping("domain1.test.ad")
True

PS C:\WINDOWS\system32> $CertAdmin = new-object -com certificateauthority.admin
PS C:\WINDOWS\system32> $CertAdmin.GetCAProperty("domain1.test.ad\TestCA", 0xb, 0, 1, 0)
4

Full Exception when running Get-CertificationAuthority -Computer Domain1.test.ad
System.Management.Automation.MethodInvocationException: Exception calling "EnumEnterpriseCAs" with "2" ar
" ---> System.IO.IOException: The network path was not found.

   at Microsoft.Win32.RegistryKey.Win32ErrorStatic(Int32 errorCode, String str)
   at Microsoft.Win32.RegistryKey.OpenRemoteBaseKey(RegistryHive hKey, String machineName, RegistryView v
   at PKI.Utils.CryptoRegistry.GetRReg(String entry, String caName, String computerName, String node)
   at PKI.CertificateServices.CertificateAuthority.getType()
   at PKI.CertificateServices.CertificateAuthority.initialize()
   at PKI.CertificateServices.CertificateAuthority.EnumEnterpriseCAs(String findType, String findValue)
   at CallSite.Target(Closure , CallSite , Type , String , String )
   --- End of inner exception stack trace ---
   at System.Management.Automation.ExceptionHandlingOps.ConvertToMethodInvocationException(Exception exce
Int32 numArgs, MemberInfo memberInfo)
   at CallSite.Target(Closure , CallSite , Type , String , String )
   at System.Management.Automation.Interpreter.DynamicInstruction`4.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame

So this is pretty crazy, but after sending that stack trace I ended up doing some more testing with ProcessMonitor to see what was going on with the registry key being read from, and it was correctly reading the value, but the value was incorrect. Apparently the CA (both prod and test) had been migrated from an old server name to the current one(s) and everything had been updated except for the registry key:
HKLM\System\CurrentControlSet\Services\Certsvc\Configuration\TestCA\CAServerName
After changing this value with the following the cmdlet started working.
certutil.exe -setreg TestCA\CAServerName domain1.test.ad
Any chance the exception thrown by the PKI.Utils.CryptoRegistry.GetRReg could include any vales retrieved? or maybe a more helpful error message on Get-CertifcationAuthority?

I understand this issue. I think, I don't need to rely on CAServerName reg value when accessing Enterprise CAs and use value retrieved via ICertConfig interface. Can you open a separate issue that will include what I suggested in this post? This case is not an issue, it is a misconfiguration at client end. In order to avoid such incidents further, we need to describe specific issue which is CAServerName value retrievel from Enterprise CAs. This value makes sense only for Standalone CAs.

Okay, I will close this and create a new issue with the information above.


"Unable to decode an event from the watch stream: object to decode was longer than maximum allowed size" causing memory leak

Rancher versions:
rancher/server or rancher/rancher: 2.0.6 (verified on 2.0.8 as well)
Infrastructure Stack versions:
kubernetes (if applicable): 1.10.1
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
This has been reproduced on both single-node and HA rancher 2.0
Description:
When accessing a specific project page in a downstream cluster from the Rancher UI, the Rancher container logs start spitting out:
E0919 17:06:17.822139       1 streamwatcher.go:109] Unable to decode an event from the watch stream: object to decode was longer than maximum allowed size
2018/09/19 17:06:17 [DEBUG] closing watcher for configMap

and in the web UI when viewing the Console, can see logs that look like
subscribe.js:287 Project Subscribe disconnected (sockId=null)
socket.js:57 Socket connecting (id=16, url=wss://<rancher-server>/v3/projects/<project-id>/subscribe...)
socket.js:241 Socket opened (state=reconnecting, id=16)
subscribe.js:277 Project Subscribe connected (sockId=16) (after 1 try, 0.145 sec)
socket.js:186 Socket null closed

Currently the suspicion is a config map on the downstream cluster is causing this issue.
After this occurs for a long enough time, the rancher container memory usage will cause the container to crash/system to OOM-kill rancher

gh-15719.yaml.zip
I have found a set of configmaps that can reproduce this issue. When these configmaps are created on a cluster (i.e. kubectl create -f gh-15719.yaml), they will be added to the default namespace of the cluster. When you access the "Default" project page and monitor the browser's console, you should see the thrashing of the Subscribe/Reconnecting/Connected messages, and checking the logs you should see the spam of the Unable to decode an event. After enough time, the system will hang.
/cc @kinarashah @cjellick @cloudnautique @aemneina

Still reproducible on master

@Oats87 - fix is available in rancher/rancher:master (well, will be in roughly 10 minutes after this message). Can you give it a go and confirm the memory leak is fixed for you?

Tested with latest build from master:
Had the cofigmaps (provided in the issue) created in Default project of a Digital Ocean cluster.
Access the "Default" project using UI.

When the same scenario was tested with 2.1.0-rc4 server , the memory consumption of rancher server steadily increases (when  still in the project page)  seen it climb from ~20 % - 65% within few minutes.

Tested with Master
Utilized reproducer configmap and created the configmap on the cluster; upon utilizing Rancher to view the default project, there was no memory leak that occurred.
E0925 22:16:39.423876       5 streamwatcher.go:109] Unable to decode an event from the watch stream: tunnel disconnect
E0925 22:16:39.423883       5 streamwatcher.go:109] Unable to decode an event from the watch stream: tunnel disconnect
E0925 22:16:39.423889       5 streamwatcher.go:109] Unable to decode an event from the watch stream: tunnel disconnect
E0925 22:16:39.423895       5 streamwatcher.go:109] Unable to decode an event from the watch stream: tunnel disconnect
E0925 22:16:39.423900       5 streamwatcher.go:109] Unable to decode an event from the watch stream: tunnel disconnect
E0925 22:16:39.674013       5 streamwatcher.go:109] Unable to decode an event from the watch stream: object to decode was longer than maximum allowed size
E0925 22:16:39.949626       5 streamwatcher.go:109] Unable to decode an event from the watch stream: object to decode was longer than maximum allowed size
2018/09/25 22:16:43 [INFO] Handling backend connection request [c-q8tcd]
E0925 22:16:44.111844       5 streamwatcher.go:109] Unable to decode an event from the watch stream: object to decode was longer than maximum allowed size
E0925 22:16:44.653693       5 streamwatcher.go:109] Unable to decode an event from the watch stream: object to decode was longer than maximum allowed size

Tue Sep 25 15:21:18 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         943        5893           8         985        6571
Swap:          2043           0        2043
Tue Sep 25 15:21:33 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         943        5894           8         985        6572
Swap:          2043           0        2043
Tue Sep 25 15:21:48 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         944        5893           8         985        6571
Swap:          2043           0        2043
Tue Sep 25 15:22:03 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         943        5893           8         985        6571
Swap:          2043           0        2043
Tue Sep 25 15:22:18 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         942        5894           8         985        6572
Swap:          2043           0        2043
Tue Sep 25 15:22:33 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         941        5895           8         985        6573
Swap:          2043           0        2043
Tue Sep 25 15:22:48 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         942        5894           8         985        6572
Swap:          2043           0        2043
Tue Sep 25 15:23:03 PDT 2018
              total        used        free      shared  buff/cache   available
Mem:           7822         941        5895           8         985        6573
Swap:          2043           0        2043

@Oats87 Could you confirm that this issue is fixed when testing with v2.1.0-rc5?


Alternative for react-title-component in React version 16

Summary:
I used to use react-title-component, but since migrating to React 16 I can't find an alternative.
Expected behavior:
Having a dynamic page title depending on which page you route to.

Hi @ahmedHanie93, can you make an reproducible example on codepen or codesandbox?


installing pbiviz

the code to enter into powershell to install pbiviz using npm i -g ~    does not work and i get the error below .  can you update this page or advise wha the error is thanks
PS C:\WINDOWS\system32> npm i -g powerbi-visuals-tools
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm ERR! code ERR_SSL_DECRYPTION_FAILED_OR_BAD_RECORD_MAC
npm ERR! errno ERR_SSL_DECRYPTION_FAILED_OR_BAD_RECORD_MAC
npm ERR! request to https://registry.npmjs.org/@babel%2Fplugin-transform-for-of failed, reason: 16536:error:1408F119:SSL routines:ssl3_get_record:decryption failed or bad record mac:c:\ws\deps\openssl\openssl\ssl\record\ssl3_record.c:677:
npm ERR!
npm ERR!      C:\Users\Jools\AppData\Roaming\npm-cache_logs\2020-05-19T14_04_13_580Z-debug.log

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

ID: 8d42a638-f0b9-abd1-43da-0c1618394948
Version Independent ID: 1350b334-88ef-1aaf-61b6-6b78ab1baf82
Content: Developing a Power BI visual - Power BI
Content Source: powerbi-docs/developer/visuals/custom-visual-develop-tutorial.md
Service: powerbi
Sub-service: powerbi-custom-visuals
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

Thanks for your comment, @powerbijools . This issue has been assigned to the doc author for review.

Try using this: npm config set strict-ssl false

Hi @uve,
Can you please point @powerbijools in the right direction?
Thanks,
Kesem

Having the same problem, following the issue.

This is the command I executed: npm config set strict-ssl false ... then I returned to true

Try to upgrade your NodeJS to v12.x and install the package again.


GSOC -SpInt

This PR is to promote review and discussion of what I have produced as a result of my Google Summer of Code project, which was to develop code for spatial interaction modeling and the general analysis of spatial interaction data.
A list of the current features, and future work is here and an in-depth log of the project development can be found on the project blog. A jupyter notebook that showcases the use of the main features is here.
Some potential points of discussion:

The CountModel class: originally meant to act as a dispatcher to different types of count models and estimation techniques, it currently only dispatched to Poisson models estimated using IWLS, which makes it sort of superficial. If support for more models/estimation techniques are added then this might become more useful. If non-count models (i.e., Gaussian) are added, perhaps a better name reflective of this flexibility should be chosen.

VecMoran class: right now this class relies on list comprehensions to store the output of permutations. For large datasets, the memory overheard becomes huge, so perhaps there is a better way to store only the necessary output rather than say, the entire W object for each permutation. This problem does not arise in a regular Moran's I because the permutation does not effect the spatial organization of the spatial units as it does in the vector-based Moran's I.

OD based weights: the ODW type spatial weight, which is based on the contiguity of both origins and destinations, currently takes in an n x n origin W and and n x n destination W (often these are the same) and returns the n**2 x n**2 origin-destination W. It might be useful, depending on how development of a SAR Lag spatial interaction model goes, to have it also return n**2 x n**2 W's for the origins and destinations too. These are just the result of np.kron(np.identity(n), originW) and np.kron(destinationW, np.identity(n)) and are needed to have W objects that match the dimensions of the n x 1 vector of spatial interaction observations. This point overlaps with #847

Default intercept behavior: currently the spatial interaction models are calibrated without intercepts. This can be changed using the boolean parameter constant=True. I found that the intercept needs to be omitted for the results to match those calibrated from maximum entropy and maximum likelihood estimation, though this default behavior might not always be optimal.

Coverage decreased (-0.2%) to 82.317% when pulling 6ea99d11d6ea4955a183ed6850a8630e851cc8e2 on TaylorOshan:spint into b18c2cff22abb00a4f69fedc66ed10b013b63bee on pysal:dev.

Coverage decreased (-0.2%) to 82.317% when pulling 6ea99d11d6ea4955a183ed6850a8630e851cc8e2 on TaylorOshan:spint into b18c2cff22abb00a4f69fedc66ed10b013b63bee on pysal:dev.

This is an impressive amount of contributions :+1:
I think the pieces all make sense, even the ones such as the CountModel class where you identify future extensions that maybe will require some thought. That is fine.
One thing that would help is if the https://github.com/TaylorOshan/pysal/tree/spint/pysal/contrib/glm had a README that would let everyone know what this is intended to support. Is it primarily consumed by spint, or could it also be used separately from spint. If the former, maybe it should move into spint?

We may need to update the requirements information before merging this.

Yeah, there are no dependencies now, so should I just remove that bit?

Coverage decreased (-0.2%) to 82.308% when pulling a148a31e3e0849bc92a62cd1077491cafb9bb00e on TaylorOshan:spint into b18c2cff22abb00a4f69fedc66ed10b013b63bee on pysal:dev.

Coverage decreased (-0.2%) to 82.308% when pulling a148a31e3e0849bc92a62cd1077491cafb9bb00e on TaylorOshan:spint into b18c2cff22abb00a4f69fedc66ed10b013b63bee on pysal:dev.

Coverage decreased (-0.2%) to 82.308% when pulling a148a31e3e0849bc92a62cd1077491cafb9bb00e on TaylorOshan:spint into b18c2cff22abb00a4f69fedc66ed10b013b63bee on pysal:dev.

Coverage decreased (-0.2%) to 82.308% when pulling a148a31e3e0849bc92a62cd1077491cafb9bb00e on TaylorOshan:spint into b18c2cff22abb00a4f69fedc66ed10b013b63bee on pysal:dev.

I believe that some of these conflicts were resolved in https://github.com/TaylorOshan/pysal/pull/6#issuecomment-244579172.

Coverage increased (+0.0006%) to 82.558% when pulling 0f8821fe2a3ecb24742f82e01256ce75956cde1f on TaylorOshan:spint into 8c47e4710f73e82b6eaf05f0aa11fdda7eb873dd on pysal:dev.

Coverage increased (+0.0006%) to 82.558% when pulling 0f8821fe2a3ecb24742f82e01256ce75956cde1f on TaylorOshan:spint into 8c47e4710f73e82b6eaf05f0aa11fdda7eb873dd on pysal:dev.

Coverage increased (+0.0006%) to 82.558% when pulling 0f8821fe2a3ecb24742f82e01256ce75956cde1f on TaylorOshan:spint into 8c47e4710f73e82b6eaf05f0aa11fdda7eb873dd on pysal:dev.

Coverage increased (+0.0006%) to 82.558% when pulling 0f8821fe2a3ecb24742f82e01256ce75956cde1f on TaylorOshan:spint into 8c47e4710f73e82b6eaf05f0aa11fdda7eb873dd on pysal:dev.

Coverage increased (+0.0006%) to 82.558% when pulling 0f8821fe2a3ecb24742f82e01256ce75956cde1f on TaylorOshan:spint into 8c47e4710f73e82b6eaf05f0aa11fdda7eb873dd on pysal:dev.

Coverage increased (+0.0006%) to 82.558% when pulling 0f8821fe2a3ecb24742f82e01256ce75956cde1f on TaylorOshan:spint into 8c47e4710f73e82b6eaf05f0aa11fdda7eb873dd on pysal:dev.

Coverage increased (+0.0006%) to 82.558% when pulling 0f8821fe2a3ecb24742f82e01256ce75956cde1f on TaylorOshan:spint into 8c47e4710f73e82b6eaf05f0aa11fdda7eb873dd on pysal:dev.

Alright, after merging #6 and then making a final edit to the contrib index, I think everything is correct and up-to-date.


TextInput.on_change does not seem to work

Hello! I just started learning toga and played with tutorial "A slightly less toy example" and came across a problem. Please tell me, it's a bug or I'm doing something wrong.
Expected Behavior
I added on_change handler to f_input: f_input.on_change = calculate. I expect that when you change the value in the field, the function calculate will be called.
Current Behavior
The function calculate is not called.
Your Environment

Python Version (list the specific version number)
3.5

Operating System and Version (select from the following and list the specific version number; if your OS is not listed, list that as well)

[ ] macOS - version:
[x] Linux - distro: Mint - version: 18.3
[ ] Windows - version:
[ ] Other - name:  - version:

Toga Target (the type of app you are trying to generate)

[ ] android
[ ] cocoa
[ ] django
[x] gtk
[ ] iOS
[ ] tvOS
[ ] watchOS
[ ] winforms
[ ] win32
[ ] Other (please specify)

The TextInput.on_change event is not being called when specified at instantiation time either.
Python 3.6, Fedora 28 (gtk)
toga.TextInput(on_change=self.calcuate)
# box.add, main_window.add(box)

I believe this may be a misreport. on_change does work - but requires a focus change before it is triggered. If you only have one widget in an app, there's no way to make that focus change happen.
This is definitely something we need to address, if only because it's inconsistent with macOS.

It looks like @maranas resolved this issue back in Mar 2019 here by subscribing to the changed event of the native Gtk widget. From GTK Reference - the "changed" signal
I installed toga-0.3.0.dev20 this morning and am not experiencing this issue.
My on_change handler is being fired on every keystroke or after a cut or a paste.
Using this app with only a text input: (can be pasted directly into console)
import toga
def build(app):
    return toga.TextInput(on_change=print)

toga.App('app', 'app', startup=build).main_loop()

I think the credit for the full fix goes to @cachitas in acc206d; but I agree this is now fixed.


merge from arlac77/npm-package-template-esm-only

package.json

chore(scripts): remove markdown-doctest (scripts.posttest)

:tada: This PR is included in version 2.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


waive submodule code missing

Release archives for the latest release (v27) do not contain the libwaive submodule code.

@mobinmob Thanks for reporting this! I have just made a fix to how packages are created, and I will release v28 shortly.

Signify v28 released :tada:


Added Edit_Distance.js file

Assigned Issue Number ?
#118
Language Opted ?
JavaScript
Number of files added or updated ?
1
Name of these files ?
Algorithms=>Dynamic_Programming=>Edit_Distance=>Edit_Distance.js
Checklist

[x] I've read the contribution guidelines.
[x] I've referred the correct issue number.
[x] I've fill up this entire template correctly.

Please Check my pull request.


Allow Zero for passwordUsePreventionType minimum value

Zero is a valid value for PasswordReusePrevention. It disables reuse checking and is the default.  From line 6011 in the same file:

Specifies the number of previous passwords that IAM users are prevented from reusing. The default value of 0 means IAM users are not prevented from reusing previous passwords. Default value: 0.

Codecov Report

Merging #1270 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1270   +/-   ##
========================================
  Coverage    98.04%   98.04%
========================================
  Files           45       45
  Lines         7382     7382
========================================
  Hits          7238     7238
  Misses         144      144

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 0898ced...5b0316e. Read the comment docs.

We don't accept pull requests that modify these files as we get them from upstream. I think they intended for omitting the field to mean 0 and if you include a value it should be greater than 1. Either way I'll forward your feedback to the service team.


Update main.tf

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

We would need platops approval here aswell.

@priteshkr please find the written approval from platops:
https://hmcts-reform.slack.com/archives/D04AC8WSEHZ/p1725634465522449?thread_ts=1725634019.970739&cid=D04AC8WSEHZ


Add toggle button to show/hide the "Note Tab" bar

Using Note list and sidebar toggle buttons v1.0.3
Just like this, can you get a toggle button?
Use case:

When using split screen, referring websites, pdf and making notes in Joplin, If the Pinned Tabs Bar is shown, the area to take note is very low, so it would be better if we have a toggle button.

Example for showing split screen.

Example pic showing toggled state , but Pinned tab is making toggled tabs useless, cos no space is saved.(top right there is toggle to hide those 2 tabs)

)

There is already the command tabsToggleVisibility to which a keyboard shortcut can be assigned. I wouldn't add this additional button yet, because I don't think it would help many users, but rather disturb them. So I would wait if this feature will be requested more often before I add it to the plugin.

oh, thats nice & helpfull. Didn't knew the command. So I'm closing the issue. Good work with the pluggin. Thanks


Compiled APK?

I've tried everything i could think of, but i'm getting a lot of errors when trying to compile this (asm/types.h) and something about malloc not declared/defined and I gave up.
Can you guys help me a little?
I know this isnt a real "issue" but i didnt know how to ask for the compiled apk (and i didnt want to send you guys an email just for this)

Did you install NDK?
It is based on Xposed, and never be tested except Nexus 5


Add Finch.Error exception

Fixes https://github.com/keathley/finch/issues/142

@wojtekmach great, thanks. I figured there would be decisions I made that needed adjusting. I'll get the changes done.

@wojtekmach all set for round two. GitHub won't let me "re-request review" 🤷

LGTM!

Thanks @wojtekmach - I appreciate the very fast PR reviews.


Fix build errors by updating packages

This should fix the build errors seen because of outdated security in older packages.
I updated all dependencies to their latest versions, including major.
The big ones are docusaurus from 2.0 alpha -> 2.3 and React from 17 -> 18.
It works on my machine, so surely there won't be any problems :)
I mean I just incremented a couple of numbers. What harm could that cause?

YOLO


[FR] Plugin development template with Hotpot

It would be great where people could clone a Hotpot template repo so they can get their development up and running at ease. Aniseed already got this with embeded scripts and library.

It's a good idea, do you have ideas of what should be in it? I think it would really just be a build.fnl that called out to hotpot.api.make, and two empty fnl and lua dirs. Not sure if that's worth an extra repo but a "writing-plugins-with-fennel.md" should exist.
From what I remember, aniseed had a testing library too as well as frankly just needing more "setup" than a theoretical plugin built with hotpot would need.
There is some sticky decisions that depend on the developer, I personally gitignore lua/* so it doesn't show up in my file picker but that means I must remember to force add changed files, not sure I would want to force this on people.
Also since hotpot will always prefer things found in lua/ (to be a good neighbour), you also need to dump that dir during dev (or rebuild all the time).
Indeed, even building out a lua folder isn't a technical requirement unless you don't want to force hotpot as a requirement (though in most cases you probably don't).
I will improve the documentation around this, not sure an actual repo would be a good fit or not, probably depends how complicated the docs get!

Yes, Aniseed has a testing library, here is my suggestion
https://github.com/andreyorst/fennel-test

Not sure if i'm correct. But isn't this library test only fennel files? How much is it usable if you need to test some lua code inside neovim runtime?
Personally i would just go with plenary to test my lua code, not fennel code.

I've added a plugin guide draft.
Hotpot wont provide a template as I don't believe setup is complex enough to warrant the extra work.


Improve Test Coverage and Handle Edge Cases in value_from_test.

What type of PR is this?
/kind feature

What this PR does / why we need it:
This PR improves test coverage and handles edge cases in the value_from_test.go file.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
This PR focuses on enhancing test coverage and addressing edge cases in the value_from_test.go file.
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Could you add some cases where both value and src are present?

yeah sure, will look into it.


Multiple *.tmp.ico files accumulating in \AppData\Local\Temp\

I confirm (by marking "x" in the [ ] below):

[x ] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[x ] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x ] I have read contributing guidelines.

Summary

Steps to reproduce
In Desktop version 3.7.1, each time you load the Mattermost program, 3 tmp.ico files are created in the temp directory.
Expected behavior
These files should be created once and referenced back to unless they no longer exist.
Observed behavior

Possible fixes

As far as reading Electron source code, it read .ico files from .asar archive by generating temporary .ico files.
https://github.com/electron/electron/blob/v1.8.1/atom/common/api/atom_api_native_image.cc#L168-L184
And I think it seems difficult to remove them because base::win::ScopedHICON can't know whether the .ico file is temporary and not referenced. electron/electron#10796

Ticket for tracking: https://mattermost.atlassian.net/browse/MM-14136

Closing as the Jira ticket is closed - please re-open if the issue persists on the latest desktop app version v4.4.0,


gem install Unsuccessful

Gem::Specification.new do |s|
s.name          = 'webhoseio'
s.version       = '1.0.0'
s.licenses      = ['MIT']
s.summary       = 'webhose.io client'
s.description   = 'webhose.io client for Ruby'
s.authors       = ['Jakub Roman']
add line
s.files = git ls-files.split($/)
s.executables = s.files.grep(%r{^bin/}) { |f| File.basename(f) }
add line end
s.require_paths = ["lib"]
s.homepage      = 'https://webhose.io'
end

This suggestion looks like it was already added, and this issue can likely be closed.


V3: object orientation WIP

@alainjobart
Concerns so far:

planBuilder interface has become big, but I think this should be it. Hopefully, nothing more will be added.
route.go is sucking a lot of functions. Still growing. We'll see how big it gets.

This change is

LGTM that seems cleaner.


Add JDK/JAVA_HOME to bazel_configure.py

I found that jar wasn't available when I did bazel test heron/..., so I thought it would be good to check for the presence of a JDK in configure.
I don't know what version requirements there are, so I didn't add a check for that, but the version can be found from javac -version.

@Code0x58 - there is no need to run bazel configure anymore. It can be used to check whether all the requirements are present.


Allow control of circle line width in Line Chart.

I didn't see any way to modify the width of the circles in the line chart. This is my initial attempt at it. The current method performs a calculation halving the circle diameter. Right now I hard coded it to be the same value for the default radius, but it will look different for current implementations if they use this.
Needs invalid value checks and maybe a calculation to match previous look of halving diameter.
I have a commit for this, but I've never really shared to a github project before so learning how to do that now.

You can fork this repo, and create a new branch and you commit on this branch; when finished, push this branch to your forked repo, and create a Pull Request(PR) to this repo's master branch.
You can learn it from github, there are guidance like: https://help.github.com/articles/using-pull-requests/
Practice makes perfect :)


Use boto3 unsigned request for fastpath

This came up in https://github.com/ooni/pipeline/pull/281#discussion_r367008590
Here is the snippet on how to do it:
>>> import boto3
>>> from botocore import UNSIGNED
>>> from botocore.config import Config
>>> s3 = boto3.client('s3', config=Config(signature_version=UNSIGNED))

This would enable anybody to just run the fastpath with zeroconfig.

This has been implemented.


[utils][SR-2695] Tag checkout support for all repositories.

Add support for a specific tag checkout for all repositories.

Resolves SR-2695.

@erg What do you think of this?

Since @erg didn't respond, I'll review this real quick. This function is getting quite big. Can you split it into two functions? One for the tag and for the non-tag case?

Done!

@swift-ci Please Python lint

@swift-ci please smoke test

Oops, should be as you've suggested now.

@aleksgapp No worries. Thanks for fixing this.
@aleksgapp, @modocache. An additional question: I have not thought about this (and I am fine doing this in a follow on commit), but given that tag/scheme are exclusive options I wonder if there is a way to express them on the command line in an exclusive manner. Just doing something like --type=scheme --type=tag won't work since they require options. Maybe the thing to do is just make sure that only one is set and error otherwise? [Trying to steal a minute of your brain power = )]

I guess we could use subparsers? Maybe have something like:
./update-checkout scheme
./update-checkout tag
./update-checkout <[need a name for when we just update what is locally there]>
Not sure.

@gottesmm I had a quick look at this direction, but didn't include this to my initial implementation.
There is an parser.add_mutually_exclusive_group() option that makes this possible:
update-checkout: error: argument --scheme/--branch: not allowed with argument --tag

One more thing that isn't fitting well here is optional arguments, e.g, --github-comment relates only to scheme/branch argument and would be just skipped if given with tag argument, --skip-history is taken in account only if clone/clone-with-ssh is given.
I'm not sure if we need take care of those here? May be just adding an exclusiveness for tag/scheme options would be enough in scope of this pull request?

@aleksgapp Yes, any such changes are 100% out of scope for this PR. Lets get this in.

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge


Entity Mapping Preprocessing

Hi, first of all, thank you for the nice work.
Let's take the below input example.
"Everaldo has played for Guarani and Santa Cruz in the Campeonato Brasileiro, before moving to Mexico where he played for Chiapas and Necaxa." , entity: Guarani .
When training the model through the input, [MASK] token is added for masking Guarani entity.
Then, the model is trained by predicting [MASK] as Guarani through Cross Entropy Loss.
However, when we analyze entity_vocab.json, there isn't "Guarani".
The entity_vocab.json only have "Guarani language", "Guarani FC", "Tupi\u2013Guarani languages", "Guarani mythology".
In that example, I believe that Guarani means Guarani FC.
Therefore, is the model trained to predict [MASK] as Guarani FC?
If yes, we need to let the model know Guarani means Guarani FC.
And, I guess that we need to match Guarani with Guarani FC.
The preprocessing in https://github.com/studio-ousia/luke/blob/master/pretraining.md, deals with such issues ?
Thank you.

Hi @kimwongyuda,
The pretraining data of LUKE is constructed with text from Wikipedia, which is already annotated with ground-truth entities.
So, the ambiguity of entity mentions will not be an issue in pretraining.
In your example, if "Guarani" is not in the entity vocabulary, the answer entity of [MASK] will be [UNK], or such entities are ignored depending on the setting.


Fixing issue with pst_path in pst_from_io_files()

The new support for specifying pst_path breaks pest/pyemu where apply_array_pars() (and similar) are expecting mult files to be in subdirectories (which is the result of the _setup_..._pars() functions), as it strips the file basename from the path. As a result the input files that PEST generates are created in the wrong directory and are then not available for the apply_array_pars() functions.
I guess this slipped through the tests because there are still (dummy) mult files in the mult directories - but these wont get updated by PEST!
This PR seems to fix this issue but I am not totally across the aim/logic for the addition of the pst_path option so there is a chance that I might have broken how this works. - not sure what the "catch all" option is for this.

Coverage decreased (-5.7%) to 51.042% when pulling d8c668510763be276583882f2981643fe26c0a61 on briochh:hotfix_pst_path_arr_mults into 499ccee7a1cc2bc5e360fd59eaadb7c8a4b84954 on jtwhite79:develop.

wow - thats a big bust.  Nice catch @briochh


bundling failed: Error: require() must have a single string literal argument

Environment
Knex version:0.14.1
When using knex inot any import statment as "import knex from 'knex';" in any react -native view.
The packager is throwing below mentioned exception.
error: bundling failed: Error: require() must have a single string literal argument
at pushDependency (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/react-native/node_modules/metro-bundler/src/JSTransformer/worker/extract-dependencies.js:39:13)
at CallExpression (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/react-native/node_modules/metro-bundler/src/JSTransformer/worker/extract-dependencies.js:50:9)
at NodePath._call (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/path/context.js:76:18)
at NodePath.call (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/path/context.js:48:17)
at NodePath.visit (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/path/context.js:105:12)
at TraversalContext.visitQueue (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/context.js:150:16)
at TraversalContext.visitSingle (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/context.js:108:19)
at TraversalContext.visit (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/context.js:192:19)
at Function.traverse.node (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/index.js:114:17)
error: bundling failed: Error: require() must have a single string literal argument
at pushDependency (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/react-native/node_modules/metro-bundler/src/JSTransformer/worker/extract-dependencies.js:39:13)
at CallExpression (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/react-native/node_modules/metro-bundler/src/JSTransformer/worker/extract-dependencies.js:50:9)
at NodePath._call (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/path/context.js:76:18)
at NodePath.call (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/path/context.js:48:17)
at NodePath.visit (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/path/context.js:105:12)
at TraversalContext.visitQueue (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/context.js:150:16)
at TraversalContext.visitSingle (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/context.js:108:19)
at TraversalContext.visit (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/context.js:192:19)
at Function.traverse.node (/Users/santoshkumar/Documents/GitHub/dreams-react-native/node_modules/babel-traverse/lib/index.js:114:17)

Should it be?
import * as knex from 'knex';

Anyways this doesn't look like knex bug. Closing.


[MSG] Further sanitize some inline/HTML bulk newsletter-style messages

Our bulk filter does a good job, this was the newsletter explicitly sent.

This needs re-checking now that we’re no longer using raw_body.

@mzikherman did we re-check?

Punting on this as a super edge-case for now. For instance, forwarding a HTML newsletter would be correctly trimmed. Someone pasting HTML in their 3rd party mail client and emailing the conversation....that's going to be tricky, basically it really shouldn't be happening.


SolutionGeoPointerController: Change radiusInKilometers to radiusInMeters

This Pull Requests fixes a mistake in the Swift Code example for Query Geohashes
This same fix was already made for the Web and Java documentation.
The PR is referencing an issue from Geofire-objc.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

@googlebot I signed it!


Add check isFile to file opener

After adding auto-completion current dir as first choice it's very easy to open some directory instead of file. This check will help user didn't get confused when he will try to save such file.
Type

[ ] Bug Fix
[ ] New Feature
[x] Quality of Life Improvement

Tests

I have tested locally against:

[x] Windows
[ ] linux
[ ] OS X
[ ] BSD

I have included automated tests:

[ ] Unit tests
[ ] Integration tests
[ ] End to end tests

@Kvaz1r thanks for the fix!  I've pushed an update for two things:

The old code (which I wrote) was capitalizing error messages, which is considered bad practice.  I basically just changed the error message to lower case.
The FileOpener had access to a status.General (which can be used to show errors in the UI), but it wasn't making use of it.  It now adds the error message to f.Err before returning the error, so that the user sees the message in the UI.

Let me know if it works for you and I'll merge :)

Yes, it works for me.


WIP - Execute commands from LLM

Jira Issues:

Checklist

[ ] Jira issue has been made public if possible
[ ] [RSPEED-] is part of the PR title
[ ] PR title explains the change from the user's point of view
[ ] Code and tests are documented properly
[ ] The commits are squashed to as few commits as possible (without losing data)

TODO: Create a jira ticket for this.


Ensure Cargo.nix file is written atomically

Fixed

Fix bugs associated with non-atomic writing of the Cargo.nix file.

With this commit, we now write the Cargo.nix file to a temporary file first and atomically mv it to the correct location once the templating successful. This should ensure that if the cargo2nix command panics or otherwise exits prematurely with an error, the existing Cargo.nix file for the project is not accidentally truncated.
This introduces a new dependency on tempfile, but this could be removed again with a bit of strategic refactoring later.
Closes #76.

@snawaz The reason I went with tempfile::Builder is because it returns a tempfile::NamedTempFile, which can be persisted to a permanent path, while tempfile::tempfile() returns a regular std::fs::File which cannot be persisted to a path.


Allows sliderStyle, tooltipStyle to be a Function

I'm not sure why this PR generated such a large diff (possibly line ending issues?) but I just modified tooltipStyles() and sliderStyles() to allow a user defined function to be used to calculate the style object.
The change:
if (Array.isArray(this.tooltipStyle)) {
          return this.isRange ? this.tooltipStyle : this.tooltipStyle[1]
        } +else if (typeof this.tooltipStyle === 'function') {
           + return this.tooltipStyle(this.val)
         + } else {
          return this.isRange ? [this.tooltipStyle, this.tooltipStyle] : this.tooltipStyle
        }

Example of how I'm using sliderStyle:
const sliderBackgroundColors = [
  'black',
  'red',
  'brown',
  'orange',
  'yellow',
  'green'
]
const backgroundColorIndex = value === 0 ? 0 : (value - 1)
const newBackgroundColor = sliderBackgroundColors[backgroundColorIndex]
return {
  backgroundColor: newBackgroundColor
}

Thanks for the quick turnaround on this one @NightCatSama - when will it be released to NPM?

Wait a moment, I need to test and modify the document.

And I found that you comment a line of code.
time || this.$refs.process.offsetWidth

I know this line of code is weird, but it's necessary.
Because the browser merges style changes, just like:
elem.style.transitionDuration = '0.5s' // this line will not work
elem.style.transform = 'translateX(100px)'
elem.style.transitionDuration = '0s'

elem.style.transitionDuration = '0.5s' // this line will work
elem.style.transform = 'translateX(100px)'
elem.offsetWidth;
elem.style.transitionDuration = '0s'

I know this is not elegant, if you have a good solution, hope to submit or tell me.
Finally, thank you for your congratulation on this project.

please update to @2.4.7

Gotcha - explanation makes sense. I'll update to 2.4.7 now, thanks!


[Story] Remove references to the challenge difficulty from the challenge search page

What projects is this story for?
OpenChallenges
As a user, I want
TBD
Description
Known references are:

difficulty shown on the challenge card
difficulty filter

At the level of the backend, we can make then difficulty optional so that we can keep most of the API implementation while returning challenge objects that do not include a difficulty.
Acceptance criteria
No response
Tasks

[ ] #1449
[x] #1450

Anything else?
No response
Have you linked this story to a GitHub Project?

[X] I have linked this story to a GitHub Project and set its metadata.

The challenge difficulty filter has been removed by @vpchung from the challenge search page and from the challenge cards. For now we can keep the challenge difficulty in the data models (.sql) and API description.


Autism Identification System #525

I have completed the project titled 'Autism Identification system'  is mentioned in issue #525. Please review it and close this issue under tag GSSOC'24 @Niketkumardheeryan.
Thanks for allowing me to contribute your project.

please write a proper readme file .

Now check it, sir. If any improvement, let me know sir @Niketkumardheeryan

LGTM


Horreum: Upload all available run into Horreum and Implement daily Jenkins job

The next step for the Horreum reports would be:

upload the existing runs (available in Github) into Horreum;
create Jenkins job to run daily and upload the generated benchmark JSON to Horreum;

Please find below the URL to the Test Run config in Horreum: https://horreum.corp.redhat.com/test/308
The URL to change detections page for scalability benchmark is: https://horreum.corp.redhat.com/changes?test=keycloak-benchmark

So, all existing runs were uploaded to Horreum. The full data (including gatling statistics) contains the reports starting from Jan 26, 2024. The data before contains only stats calculated from Prometheus. So I have updated the existing reports before Jan 26 also to include the environment information (i.e. context{} JSON element). Also I have found out when we have started to run MultiAZ deployemtn with ISPN CrossDC and reports generated before marked as ran with ISPN single deployment.

Job is created in Keycloak Jenkins which is scheduled to work daily ~13:00CET, download the latest report JSON file and upload it to Horreum. Also the job is implemented to be parameterized, i.e. it takes date (in case some day was missed) and uploads data starting from that date. The PR for the job is sent here: https://gitlab.cee.redhat.com/keycloak/keycloak-qe-jenkins-csb/-/merge_requests/161
The job itself is located here: https://master-jenkins-csb-keycloak-qe.apps.ocp-c1.prod.psi.redhat.com/view/Performance/job/kb-horreum-upload/

Also please find attached the PR for the updated "result-data" branch: https://github.com/keycloak/keycloak-benchmark/pull/715


Aggregation - validate input as per PostgreSQL format

Validate the condition specified is in the proper PostgreSQL format

This issue will be addressed in cQube v5.0.

This is related to cQube v4.1 beta


Fixes: #24185 Cilium-operator crash on empty azure node pools

Empty VirtualMachineScaleSets returns 404 on ListVirtualMachineScaleSetNetworkInterfaces if there are no nodes in it. Continue with next VirtualMachineScaleSet in that case.
Please ensure your pull request adheres to the following guidelines:

[X] For first time contributors, read Submitting a pull request
[] All code is covered by unit and/or runtime tests where feasible.
[X] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[X] All commits are signed off. See the section Developer’s Certificate of Origin
[X] Provide a title or release-note blurb suitable for the release notes.
[X] Thanks for contributing!

Azure allows for empty user node pools. In that case ListVirtualMachineScaleSetNetworkInterfaces will return 404 when trying to get interfaces associated with given node pool. With this change it will continue with next VirtualMachineScaleSet if this is NotFound error.
Fixes: 24185
Fix for empty node pools on Azure

I've tested it manually on azure cluster with k8s 1.24.9

@tommyp1ckles Any ETA for the review of this?

@forgems sorry for the delay - I'll take a look.

/test


Unable to select nested text element with text selector

Since text selector returns the top most text node for text nested inside multiple text nodes, it is not possible to click the specific sub-element with the text using a text matcher.
Based on the conversation in PR 65, this might be "won't fix" but I wanted to document it as an issue.

Can we close it now?


WIP - Glue integration

[ELE-47] Add integration with S3 as a data lake

Hi @Arun-kc ,
Thanks for this work and sorry it took us so long to address this PR. We are working on improving our process with PRs and Github issues and admittedly these have been neglected in the past few months due to other priorities.
Since it's been a while and there are quite a few conflicts, I'll close this PR for now.
If you still wish to contribute this and can update the code, I'll be happy to review. Ensuring that integration tests pass on the Glue adapter (here) will also help to speed the review process.


Windows issue related to pathname

Getting an issue while cloning the git repo to my windows computer.
Windows doesn’t accept "?" as name of the path or filename.
The error it shows is:
error: invalid path 'examples/What do API Keys REALLY do?.txt'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

Issue fixed in this commit -> df2c7c


[Feature] Use a line connect each part of the stacked histogram; 使用连线连接堆叠柱状图的每一个部分

What problem does this feature solve?
请问现在是否有api可以解决如下需求，即有一个连线连接堆叠柱状图的每一个部分，即类似红线这种功能

Is there any API that can solve the following requirements, that is, there is a connection to connect every part of the stacked bar, which is similar to the function of red line
What does the proposed API look like?

Try line in graphc. Here's an example.

Code Sample
option = {
  xAxis: {
    type: 'category',
    data: ['Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun']
  },
  yAxis: {
    type: 'value'
  },
  graphic: [
    {
      type:'line',
      shape:{
        x1:170,
        y1:320,
        x2:300,
        y2:60
      },
      z:3
    }],
  series: [
    {
      data: [120, 200, 150, 80, 70, 110, 130],
      type: 'bar'
    }
  ]
};

Try line in graphc. Here's an example.
Code Sample

thank you very much，i use custom series  solve

Glad you find your way to solve it! Mind sharing your solution?

Glad you find your way to solve it! Mind sharing your solution?

参考了如下实现，代码部分只是稍作修改https://echarts.apache.org/examples/zh/editor.html?c=custom-bar-trend
大致思路就是用polyline在两个柱子之间连线，custom的demo数据如下

落实到demo上就是这个意思，所以只要根据你的堆叠柱状图图的数据，构建坐标就可以了

Thanks for sharing!


How to handle drop down elements

Can you please document a consistent way to handle drop down menus? It is not possible to do a Find-SEElement on a drop down and then use Send-SEKeys, as the field will come back as "not interactable"
In other languages' implementation of Selenium, I can do something like this:
select = Select(driver.find_element_by_id('UserDropDown'))  select.select_by_visible_text('Joe_Smith')
But I have been unable to get the syntax right in PowerShell. I see a LOT of threads on SO and other sites asking for this kind of info, so a solid example of implementing this would be awesome.

Hi @JLogan3o13  this is a work in progress function i've been working on give it a spin and let me know if it works for you.
    Param(
        [OpenQA.Selenium.IWebElement]$Element,
        [string]$Text,
        [string]$Value
    )

    if($Value){
        $SelectElement = New-Object -TypeName "OpenQA.Selenium.Support.UI.SelectElement" -ArgumentList $Element
        $SelectElement.SelectByValue($Value)
    }
    elseif($Text){
        $SelectElement = New-Object -TypeName "OpenQA.Selenium.Support.UI.SelectElement" -ArgumentList $Element
        $SelectElement.SelectByText($Text)

    }
}```

-DM

Thanks for the response. I have done something similar with Selenium in other languages, and had tried it in PS as well. It does work if the element is actually a select. I know in the particular case I am looking at right now, for example, the element has a tag other than select. So you get the dreaded: "New-Object : Exception calling ".ctor" with "1" argument(s): "Element should have been select but was td"" error. I also get it with dropdowns that are div tags. It may honestly be just malformed html on this page I'm working on. Here is the CSS I am looking at:

Once you click on the dropdown, this element is interactable:

but not the main element. I did find that if I click on it (manually) I can do something like this:
  Start-Sleep 10 #give time to manually click dropdown   $taskDropdown = Find-SeElement -Driver $chrome -Timeout 60 -Id "mainForm-_SupportGroupDisplayPaging"   Start-Sleep 5   $hosting = Find-SeElement -Driver $chrome -Timeout 30 -XPath "//div[contains(text(),'Joe_Smith')]"   $hosting.Click()
However, if the dropdown is one that dynamically expands content as you scroll down, or worse, has a "next" arrow, even that methodology becomes near useless.
As I said, could very well just be poorly coded html.

I believe the current version handles that through Get-SeSelectionOption (which act as both a Get and a Set)
V4 (to be released October 4th) will handles Select through : Set-SeSelectValue
Regarding dynamically loading select and other special select, this should be possible to handle it by scrolling into the select box, which is something I am currently working on #122  with the addtion of Actions (Keydown / Key up / mouse move, mouse drag, etc...)
You can track the progress for that second part on #122
If you think I missed something, feel free to reopen this ticket.
Thanks.


cloudflare_workers_kv_namespace with an API token results in "Authentication error"

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.4.6
on linux_amd64

provider registry.terraform.io/cloudflare/cloudflare v4.9.0

Affected resource(s)
cloudflare_workers_kv_namespace
Terraform configuration files
# I generate my Terraform configuration with Terranix, so I provide parts of the generated JSON here. It works if named `foo.tf.json` in the Terraform project directory.
{
  "provider": {
    "cloudflare": {
      "api_token": "${var.cloudflare_api_token}"
    },
  },
  "variable": {
    "cloudflare_api_token": {
      "sensitive": true,
      "type": "string"
    },
  },
  "data": {
    "cloudflare_accounts": {
      "account": {
        "name"<EMAIL_ADDRESS>      }
    },
  },
  "resource": {
    "cloudflare_workers_kv_namespace": {
      "foo": {
        "account_id": "${data.cloudflare_accounts.account.id}",
        "title": "foo"
      }
    }
  }
}

Link to debug output
https://gist.github.com/abbradar/def872bd827b97c94d9e04fc92a61fc9
Panic output
No response
Expected output
The resource is created.
Actual output
╷
│ Error: error creating workers kv namespace: Authentication error (10000)
│
│   with cloudflare_workers_kv_namespace.foo,
│   on generated.tf.json line 1642, in resource.cloudflare_workers_kv_namespace.foo:
│ 1642:       }
│
Steps to reproduce

Create a Terraform project which creates a Cloudflare KV namespace resource;
Try to apply.

Additional factoids
The same token works when deploying through Wrangler with wrangler deploy. Other Cloudflare resources that I manage work normally.
References
No response

this isn't something we can address in the provider as it is related to auth and the Terraform Provider is just another HTTP client. you'll need to debug why this credential isn't working for the operation you are attempting. my suggestion is to start outside of Terraform by making the API calls and confirming all the IDs and permissions directly.

@jacobbednarz The important thing is that Wrangler works with the same token; that's why I expected this to be a bug as I guessed they use the same API endpoints. Should I try and compare what endpoints they use?

yep - compare the URLs and payloads used.

mitmproxy helped greatly. Turns out it was me using cloudflare_accounts.foo.id instead of cloudflare_accounts.foo.accounts[0].id. Sorry for the noise :)

there are some notes at https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/contributing/provider-design.md#plural-data-sources on how we handle single vs plural data sources in this provider in case it helps in the future.


[DOCS] Add new Blog on Lakehouse

Change Logs
adds a new Data Lakehouse blog
Impact
None. Site updates
Risk level (write none, low medium or high below)
None
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[ ] CI passed

Tested Locally! The images get rendered after merging.


Build fault tolerance into jobs that use pub

pub times out just enough to produce flakiness in our various automated tests:

[ ] Travis
[ ] Chromium bots
[ ] Device lab

We should build in basic fault tolerance, where we retry N times with an exponential backoff before giving up.

See also https://github.com/flutter/flutter/issues/9898 which is about the flutter tool itself.

I'm going to close this in favour of https://github.com/dart-lang/pub/issues/2242.


Body Parser Error After Login Page

I get body parser error in after login page
WrongArgumentError:
You probably used old body-parser middleware, which is not compatible
with @admin-bro/express. In order to make it work you will have to

move body-parser invocation after the admin bro setup like this:

const adminBro = new AdminBro()
const router = new buildRouter(adminBro)
app.use(adminBro.options.rootPath, router)
// body parser goes after the AdminBro router
app.use(bodyParser())

Upgrade body-parser to the latest version and use it like this:
app.use(bodyParser.json())

at C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\@admin-bro\express\lib\buildAuthenticatedRouter.js:52:18
at Layer.handle [as handle_request] (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\layer.js:95:5)
at trim_prefix (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:317:13)
at C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:284:7
at Function.process_params (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:335:12)
at next (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:275:10)
at Function.handle (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:174:3)
at router (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:47:12)
at Layer.handle [as handle_request] (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\layer.js:95:5)
at trim_prefix (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:317:13)
at C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:284:7
at Function.process_params (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:335:12)
at next (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:275:10)
at C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:635:15
at next (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:260:14)
at Function.handle (C:\Users\KIIT\Desktop\virtuera\hey\backend\server\node_modules\express\lib\router\index.js:174:3)

const AdminBroOptions = require('./routes/adminRoute/adminRoute')
const adminBro = new AdminBro(AdminBroOptions)
app.use(adminBro.options.rootPath, adminRoute)
It worked for me!

Well I got mine solved by moving
app.use(express.json());
app.use(express.urlencoded({ extended: false }));

below
app.use(adminBro.options.rootPath, adminrouter)

error still present, not accurate and not up to date since the body-parser module is included in express
moreover I succeed to setup admin js multiple time but I'm still loosing multiple days to fix this issue
To conclud on this comment i have the error only when using AdminJSExpress.buildAuthenticatedRouter and not when using buildRouter


Allow passing arbitrary MemFlags to emit_small_mem{cpy,move}

Fixes #1170

This is ready for review. Can someone take a look?


fix symlinks

https://github.com/openshift/openshift-ansible/pull/6469 relocated rpm_q.py to a
different directory.
What does this PR do?
Fixes symlinks
How should this be manually tested?
Observe that if using openshift-ansible-contrib and OCP 3.9, the destinations for the symlinks doesn't work.
Is there a relevant Issue open for this?
No.
Who would you like to review this?
cc: @tomassedovic PTAL

/retest


[CI] Mandrel integration tests fail with Java 23 mandrel build of mandrel/24.1 on Linux

This is an issue to automatically track failures in Mandrel's CI runs.

Caused by https://github.com/Karm/mandrel-integration-tests/issues/245

Caused by: https://github.com/Karm/mandrel-integration-tests/issues/252

2024-04-12T02:33:33.4180723Z Error: Class initialization of io.quarkus.runner.ApplicationImpl failed. This error is reported at image build time because class io.quarkus.runner.ApplicationImpl is registered for linking at image build time by command line and command line. Use the option

Interesting. This is https://github.com/quarkusio/quarkus/issues/39526 (should be solved once 23+18 is being used)

The warning shown is:
Warning: Option 'DynamicProxyConfigurationResources' is deprecated and might be removed in a future release. Please refer to the GraalVM release notes. ==> expected: <true> but was: <false>

See comment https://github.com/Karm/mandrel-integration-tests/issues/244#issuecomment-2126652972 for an analysis.

See comment https://github.com/Karm/mandrel-integration-tests/issues/244#issuecomment-2126652972 for an analysis.

See this https://github.com/Karm/mandrel-integration-tests/issues/244#issuecomment-2126652972 for an analysis.

See this https://github.com/Karm/mandrel-integration-tests/issues/244#issuecomment-2126652972 for an analysis.

See this https://github.com/Karm/mandrel-integration-tests/issues/244#issuecomment-2126652972 for an analysis.

See this https://github.com/Karm/mandrel-integration-tests/issues/244#issuecomment-2126652972 for an analysis.

All of the above are these warning messages:
Warning: Option 'DynamicProxyConfigurationResources' is deprecated and might be removed in a future release. Please refer to the GraalVM release notes.

... and those are caused by:
https://github.com/oracle/graal/pull/8822

Test failures are:
 Error:  Failures:
Error:    AppReproducersTest.imageioAWTTest:501->imageioAWT:607 There were errors checking the generated image files, see:
mytest.jpg was not generated.
mytest_toL.png was not generated.
mytest.bmp was not generated.
mytest_toG.png was not generated.
mytest.gif was not generated.
mytest.svg was not generated.
mytest.png was not generated.
mytest.tiff was not generated.
mytest_Resized_Grace_M._Hopper.png was not generated.
mytest_toC.png was not generated.
mytest_toP.png was not generated.
mytest.wbmp was not generated.
mytest_toS.png was not generated. ==> expected: <true> but was: <false>
Error:    JFRTest.jfrOptionsSmokeTest:784->jfrOptionsSmoke:922 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.JFRTest/jfrOptionsSmokeTest/build-and-run.log and check these offending lines:
[warn][jfr,system] Could not initialize JDK events. jfrTracing ==> expected: <true> but was: <false>
Error:    JFRTest.jfrPerfTest:185->jfrPerfTestRun:234 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.JFRTest/jfrPerfTest/build-and-run.log and check these offending lines:
[warn][jfr,system] Could not initialize JDK events. jfrTracing
Warning: Option 'DynamicProxyConfigurationResources' is deprecated and might be removed in a future release. Please refer to the GraalVM release notes. ==> expected: <true> but was: <false>
Error:    JFRTest.jfrSmokeTest:151->jfrSmoke:722 build-and-run.log log should not contain error or warning lines that are not whitelisted. See /home/runner/work/mandrel/mandrel/mandrel-integration-tests/testsuite/target/archived-logs/org.graalvm.tests.integration.JFRTest/jfrSmokeTest/build-and-run.log and check these offending lines:
[warn][jfr,system] Could not initialize JDK events. jfrTracing ==> expected: <true> but was: <false>

So this is #268, #261 and #262. I'll handle the additional allow-list for #261.

Failure is https://github.com/Karm/mandrel-integration-tests/issues/262

Failure is https://github.com/Karm/mandrel-integration-tests/issues/262

Still https://github.com/Karm/mandrel-integration-tests/issues/262

Still https://github.com/Karm/mandrel-integration-tests/issues/262

Still https://github.com/Karm/mandrel-integration-tests/issues/262

See https://github.com/Karm/mandrel-integration-tests/issues/266#issuecomment-2410390365

https://github.com/Karm/mandrel-integration-tests/issues/266#issuecomment-2410390365


Handling "syncThreshold" key on configure for android.

Oops. I just realized there's already an open PR (https://github.com/mauron85/react-native-background-geolocation/pull/37) for that.


daemon: Do not require NodePort for WG

See commit msgs. cc @tommyp1ckles

/test

@brb The titles of PRs like this will end up in release notes, so as such it is preferable to fully expand any acronyms so that users can more easily understand the context of things like 'WG' here (or search through release notes if they're really interested in that particular feature). I'll update this one so it's clearer for the upcoming v1.16 release.


System.IO.Ports

I have a program is running on the .NET Compact Framework. There is no VPU driver on the WinCE System, so I need to change the operating System to an embedded Linux System (arm architecture), but there is no System.IO.Ports API in NETCore. Will you add the API in the future?

@wfurt is working on a PR to get this started (see #29033).
While that is not ready yet some people had success with https://github.com/jcurl/SerialPortStream (see here for example), but that may not be an option to you since it seems you already have code using the Fx API.

@pjanotti @wfurt Thanks.
On the embedded Linux system, I did a simple test and ran the program successfully.
I may need to transplant my program that is running on the .NET Compact Framework to make sure it works perfectly.


nicer implementation

tidies up #663
also use briefer description in the code, and move the longer explanation to
the test

I decided this and #663 to be in 0.10 instead of 0.11.


feat(mango): add keys_examined for execution_stats

Add another field to the shard-level Mango execution statistics to keep track of the count of keys that were examined for the query.  Note that this requires to change the way how stats are stored -- an approach similar to that of the view callback arguments was chosen, which features a map (#4394).
The change in the format also implies a potential incompatibility between different version of nodes, which may happen for mixed-version clusters.  This is handled by introducing an additional option for the view callback where the coordinator can request for the results in the new format and the old format is still recognized.  It helps to handle version differences automatically therefore no specific configuration option is required to control the migration.  Thanks @rnewson for the idea.
This is related to the covering indexes work and takes inspiration from earlier works of @mikerhodes , see #4410 and #4413 for the details.
Note that slight increase in unit test coverage for mango_execution_stats due to the implicit testing via mango_cursor_view.  The only missing part is incr_docs_examined/1 but that is not used from there, only from mango_cursor_text and mango_cursor_nouveau .
$ make eunit apps=mango

before:
[..]
Code Coverage:
[..]
mango_cursor_view     : 100%
[..]
mango_execution_stats :  63%
[..]
Total                 : 46%

after:
[..]
Code Coverage:
[..]
mango_cursor_view     : 100%
[..]
mango_execution_stats :  94%
[..]
Total                 : 47%

Checklist

[x] Code is written and works correctly
[x] Changes are covered by tests

Please do not merge this now, I would like to do some refactoring (move the process dictionary function under mango_execution_stats) per @chewbranca's recommendations.

OK -- the planned changes are there.


Redirect to a complete registration page is throwing Error...

I have this file in my dev directory  ..google_service_oauth2_userinfoplus.php but
The file google_service_oauth2_userinfoplus.php could not be found!  error popping up whenever
after i save the data to my database and redirect to a complete registration page

It looks like that file Userinfoplus.php is installed by composer at vendor/google/apiclient-services/src/Google/Service/Oauth2/Userinfoplus.php.
Did you flatten those services directories by replacing /'s with _'s?
Are you letting composer's autoloader handle the includes/requires or are you doing it manually?

We did it manually. But we haven't changed the dir structure in such way.
On Monday, February 19, 2018, Matt Whisenhunt<EMAIL_ADDRESS>wrote:

It looks like that file Userinfoplus.php is installed by composer at
vendor/google/apiclient-services/src/Google/Service/
Oauth2/Userinfoplus.php.
Did you flatten those services directories by replacing /'s with _'s?
Are you letting composer's autoloader handle the includes/requires or are
you doing it manually?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-php-client/issues/1386#issuecomment-366760184,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACURvB4tUq1ZiuFxNE0Bz_EmiXUEWwhaks5tWbC3gaJpZM4SKmZr
.

--
Abel Seyoum Ar'aya
+251-912-37-77-21
+251-91-163-94-70

Ocellinet Software Development P.L.C
Your Partner in Automating Your Domain Expertise
www.Ocellinet.com http://www.ocellinet.com/

Dynamic S.tm
Dynamic Interactive Solutions
www.Dynamicstm.com
<EMAIL_ADDRESS>
The information in this email and in any attachments is CONFIDENTIAL and
may be
PRIVILEGED. If you have received it by mistake, please destroy this message,
delete any copies held on your systems and notify the sender immediately;
you
should not copy it or disclose its contents to anyone. All messages sent to
and
from Ocellinet Software Development P.L.C, Dzineware and Dynamic Solutions
may be monitored to ensure compliance with internal policies and to protect
our business.
Emails are not secure and cannot be guaranteed to be error free because
they can be intercepted, amended, lost, destroyed or contain viruses.
Anyone who communicates with us by email is taken
to accept these risks. The content of any email addressed to our clients is
subject to our usual terms of business.

make sure your file isnt named php.php or something like that.
Beyond that Please post questions on stack overflow using the tag google-api-php-clinet I would be happy to have a look

Thank you for filing this issue. We asked some clarifying questions or suggested a course of action a week or more ago and never heard back from you. We are unable to proceed with this issue until then, so we are closing it. Please feel free to comment with more information and we will re-open this issue.


[Fix #142] Minitest/GlobalExpectations: autocorrect when receiver is lambda

This fixes an issue when autocorrecting offenses where the receiver is
a lambda function: -> { n }.
The expected correct result is _ { n } instead of _ { -> { n } }.
Fixes #142

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


Reimplementation of PR #512 with latest main branch

See PR #512 for background. This PR only purpose is to update PR #512 with the latest changes to main.

Checklist

[ ] Added a CHANGELOG.rst entry

Approving this because it's only a re-write of #512


Wrong return type for Eloquent::when() & Eloquent::where()

I've just noticed that the generated doc marks the above methods (not sure if there are more) as @return $this. However, the generate code is correct, and actually returns Builder instance.
/**
 * Apply the callback's query changes if the given "value" is true.
 *
 * @param bool $value
 * @param \Closure $callback
 * @param \Closure $default
 * @return $this                        // <<<<< this is wrong
 * @static
 */
public static function when($value, $callback, $default = null){
    return \Illuminate\Database\Eloquent\Builder::when($value, $callback, $default);  // <<<< this is correct
}

/**
 * Add a basic where clause to the query.
 *
 * @param string|\Closure $column
 * @param string $operator
 * @param mixed $value
 * @param string $boolean
 * @return $this                         // <<<<< this is wrong
 * @static
 */
public static function where($column, $operator = null, $value = null, $boolean = 'and'){
    return \Illuminate\Database\Eloquent\Builder::where($column, $operator, $value, $boolean);  // <<<< this is correct
}

This works nowadays, probably because IDEs caught up (tested with PhpStorm)
@barryvdh I think we can close this issue :}


v0.9.4

Added

support for blacklist request: set limit (depending on plan), optional plain text
support for check-block request

Changes

commands name changes (l to L, g to G). l is now the limit in blacklist request break change
color is not green anymore for 1 as score (for 0 only)

Added

support for blacklist request: set limit (depending on plan), optional plain text
support for check-block request

Changes

break change commands name changes (l to L, g to G). l is now the limit in blacklist request
color is not green anymore for 1 as score (for 0 only)

Added

support for blacklist request: set limit (depending on plan), optional plain text
support for check-block request

Changes

break change commands name changes (l to L, g to G). l is now the limit in blacklist request
color is not green anymore for 1 as score (for 0 only)


Sanity healthcheck connections

This PR is in response to some operational issues we ran into with healthchecks:

If we dont receive a health check response within the healthcheck timeout, assume the connection is bad and cancel it.
Validate that the tablet you think you are talking to is really the one you're talking to, by comparing the alias we know about with the alias returned from the healthcheck.

I added tests for both cases.
This has been running in our environment in some form for a few weeks, and in its latest form for a week.

The remaining tests are failing because in the integration tests we do not create a periodic healthcheck. So it's actually working as designed. Not sure of the best way to add the periodic healthcheck, but am looking into it.

So for each health check connection, this PR adds another go routine. We have thousands of tablets, this might grow too much.
Also, we already have code that uses healthCheckTimeout. That code would be obsolete and would need to be removed if we kept the direction of this CL.
But... I have another option:

add a cancelFunc in the healthCheckConn, right after:
conn                  queryservice.QueryService
and name it streamCancelFunc.

populate that new streamCancelFunc from the cancel function you create in go/vt/discovery/healthcheck.go line 385 in the HCC (need to take the mutex to do that).

in checkHealthCheckTimeout function, call the streamCancelFunc instead of marking the node as unhealthy.

That way we keep the nice property that we only have one watchdog go routine for all HCCs, and it can cancel the streaming healthcheck query. So error handling is only done in one place both for streaming connection being closed for other reasons, and our timeout.
Does that make sense?

@alainjobart I've incorporated your feedback, thank you -- i had somehow not thought to just put the cancellation in the existing timeout check.
I decided to keep the unhealthy handling in the checkHealthCheckTimeout function. Moving it actually ended up breaking some tests and the fixes seemed more complicated that it was worth.
I'm running this now internally and will report back tomorrow on how it handled some rolling restarts.

@michael-berlin took care of all your comments. Thanks!


Thanks for this wonderful plugin

Just would like to say thanks, it's a pretty wonderful and well made neovim plugin!

you're most welcome 😇