add Popper & Reference innerRef prop

Fixes #140
tested this in Blueprint's Popover locally (via yarn link) and was able to get it rendering without the infinite loop!
@FezVrasta this was my first experience with Flow and I must say it was rather subpar compared to TypeScript. Why did you choose to use Flow for this project and would you consider migrating to TypeScript? (I'd be happy to do the conversion, it'll take like an hour.)

Thanks for the PR, the decision to use Flow has been carefully pondered and I'm not willing to switch to TypeScript.
Some of the reasons are:

TS is not JS
TS is not a sound type system
Flow inference is way better than TS

🤔 I would argue that Flow is not JS either, but the .js extension confusingly suggests that it is: you can't actually execute the code until you've run it through a transpiler and removed the non-JS type statements! The .ts extension makes this extra step obvious, plus the TS compiler is basically Babel + Flow + polyfills in one tool. TS is definitely JS (with purely additive type information) -- it deliberately adheres to the latest ECMAScript specs and provides consistent support for upcoming actual JS features long before they're widely available in browsers.

You can write Flow annotations as JS comments. The point is that once the types are stipped, the resulting code is javascript.
TypeScript on the other hand is a superset of JavaScript, it introduces new language features such as the public and private class properties. Once comments are stripped, you don't get valid JavaScript

TS is also hard to swallow for the first time. With experience you see more advantages.

@FezVrasta flow discussion aside, any more work required on this PR?

@FezVrasta ping on this? would love to ship this and start using it in Blueprint.

I'm sorry, been a bit busy lately

thanks @FezVrasta!! i suppose the next step is to ship beta.7?


About PER

As the paper mentioned, "In order to reduce the inhibition, there is a prototypical entropy regularization (PER) terms to slow down the convergence speed of the dominant modality." But I didn't find it in the codes. May I ask where it is implemented?

Hello, have you found the relevant code?


Potential wrong use of assertion

We encountered an error while using the SDK where GetAll would always throw. After looking through the code we saw that where an instance of this class is created (TrafficManagerProfileResource), the ctor with the 3rd parameter is never called. I'm guessing that's from the AutoRest codegen.
Maybe adding AssertNotNull rather than AssertNull would fix the problem in the codegen?
How we made it work was to use reflection to get _profileData and set it before calling GetAll()
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

Bump

Well despite I approved this PR because the assert is really incorrect, this is not enough to get the methods added in this file working - the new GetAll and GetAllAsync methods require the profileData to be set, but the only ctor that could set it is internal and no other methods in the SDK are calling it. Therefore it will always be null...


Feature: Add close() method to PublicClient and WalletClient that invokes close() on the WebSocket transport.

Description of the Desired Feature
Assuming I use a Websocket transport, I'd like to be able to invoke a close method on the clients directly,

Motivation and Use Case
This would allow me to have cleaner code because I wouldn't need to add both the client (to call its methods however needed) and the transport (for the single purpose of being able to invoke the close() method on app shutdown), to my class. It would allow for a cleaner dev experience and keep my Class prototype more minimal.
Proposed Solution
Add a close() method to PublicClient and WalletClient. It if the close() method exists on the transport and invokes it if it does, (b/c close will not exist on the REST transport).

Context: This is a low-level SDK, it should be simple and logically straightforward.
Adding “close” to PublicClient/WalletClient violates the SRP principle and reinforces coupling for the sake of syntactic sugar.
I think this is a bad idea.
If you need “close” in PublicClient/WalletClient, I can suggest extending the source classes/types:
import { type IRESTTransport, PublicClient, WebSocketTransport } from "@nktkas/hyperliquid";

interface ExtendsIRESTTransport extends IRESTTransport {
    close?(): void | Promise<void>;
}

class ExtendsPublicClient extends PublicClient {
    constructor(public override transport: ExtendsIRESTTransport) {
        super(transport);
    }

    async close(): Promise<void> {
        return await this.transport.close?.();
    }
}

// Example usage
const transport = new WebSocketTransport();
const client = new ExtendsPublicClient(transport);
await client.close();

If you need “close” in PublicClient/WalletClient, I can suggest extending the source classes/types:

I came up with a more elegant solution. I can add Generics to PublicClient/WalletClient. This is a better solution than any of the options above. It should also solve your problem, provided you are using TypeScript.

Implemented in https://github.com/nktkas/hyperliquid/commit/4ad416d3c207e7dc98e2a288621a51ccb7fc81ae.
Now you can use features of passed transport via PublicClient/WalletClient:
import { HttpTransport, PublicClient, WebSocketTransport } from "@nktkas/hyperliquid";

// Example 1:
const transport = new WebSocketTransport();
const client = new PublicClient({ transport }); // PublicClient<WebSocketTransport>
await client.transport.close(); // No type error

// Example 2:
const transport = new HttpTransport();
const client = new PublicClient({ transport }); // PublicClient<HttpTransport>
await client.transport.close(); // TSError: Property 'close' does not exist on type 'HttpTransport'.

Neat. Thanks! I'll watch for the new release.


turbomole

Hello, the patch of turbomole 7.1 cannot find. could you update the link? Thanks

https://github.com/Z-H-Sun/CS_CCME_Posts/raw/tags/hidden/turbomole/patch.zip


Please provide delete steps

Currently, there is no documentation or steps for deleting CRs (ISTIO components) & operator itself. Usually, we can run kubectl delete -f , but the steps are not clear.
Please provide steps to delete completely.

Hi @mossuchida, we've added the necessary steps to the README.md.
Thank you for your suggestion!


Broken Doodle results page

Yeah.

The doodle code seems to have slowly been removed from the search results page. It should be run here, as it doesn't affect isch results pages, just regular ones.
Specifically, document.querySelector(".jfN4p") (searchLogo[0]) is null on a doodle day. The div returned by querySelector(".logo.Ib7Efc") contains an image two levels down which can be modified this way:
document.querySelector(".logo.Ib7Efc").children[0].children[0].src = "https://upload.wikimedia.org/wikipedia/commons/3/3e/Google_2011_logo.png";

On a Doodle day, the first two (only the first!) images with class YQ4gaf (which is also used for a lot of images in the actual results), can be removed:
document.querySelector(".YQ4gaf").remove();
document.querySelector(".YQ4gaf").remove();

This classname requires extreme care as it used to cause issues that were solved by removing the whole previous solution to this issue in commit 03388b6.

https://github.com/toydotgame/old-google/blob/656e211aad05515e57ccead1b0e3a781f1a5c7fe/replace.js#L77-L78
Line 77 can run, however line 78 returns null and fails.

Fixed it also there's a container div for the two doodle bg images so I just delete that. Also RIP isch tbm key gone but not forgotten (SVG logos are no more on google dot com)

I thought the tbm GET key was gone completely, usurped by the udm ID-based key for search tabs, but it appears tbm=vid is still used for Video search results.


column to extract VAF when standardVC

Hello,
on the SigProfilerClusters Wiki page, there seems to be an inconsistency in the documentation.

If your VAF is recorded in the 8th column of your VCF as AF=xx, set standardVC=True, TCGA=False, and sanger=False.

the above states that standardVC=True when VAF is in the 8th while the below says that standardVC=True when VAF is in the 7th column.

Parameter | Variable Type | Description
standardVC | Boolean | Parameter that informs the tool of what format the VAF scores are provided. This is required when subClassify=True and sanger=False and TCGA=False and when the data contains VAF formatted in the 7th column as AF=XX (default=False)

yet, in classifyFunctions.py, vaf = float(lines[9].split("AF=")[1].split(";")[0])  vaf seems to be extracted from the 10th column.
i'd appreaciate it if you could clarify this issue.
Thanks a lot!
elif standardVC:
		vafs = {}
		for vcfFile in vcf_files:
			sample = vcfFile.split(".")[0]
			vafs[sample] = {}
			with open(vcf_path + vcfFile) as f:
				for lines in f:
					if lines[0] == "#":
						continue
					lines = lines.strip().split()
					chrom = lines[0]
					pos = lines[1]
					ref = lines[3]
					alt = lines[4]
					try:
						vaf = float(lines[9].split("AF=")[1].split(";")[0])
					except:
						vaf = -1.5
					keyLine = ":".join([chrom, pos, ref, alt])
					vafs[sample][keyLine] = vaf

Hi,
Thank you for your feedback. Indeed, this was an inconsistency, which has now been fixed in the OSF and function doc strings. The proper format should contain VAFs in the 10th column when using standardVC=True.
This should resolve the issue; however, please feel free to reopen or start a new issue if necessary.
Best.


⚠ Bofrost.fr has degraded performance

In 108996a, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1331 ms

Resolved: Bofrost.fr performance has improved in 179555d after 8 minutes.


function uninstallWg()

Add "uninstall generated client file" to the uninstallWg() function. Something like this, but removing all custom WG config files:
function uninstallWg(){
                #REMOVE USER CONFIG
                # Home directory of the user, where the client configuration will be written
                CLIENT_NAME=$(grep -E "^### Client" "/etc/wireguard/${SERVER_WG_NIC}.conf" | cut$
                if [ -e "/home/${CLIENT_NAME}" ]; then # if $1 is a user name
                        HOME_DIR="/home/${CLIENT_NAME}"
                elif [ "${SUDO_USER}" ]; then # if not, use SUDO_USER
                        HOME_DIR="/home/${SUDO_USER}"
                else # if not SUDO_USER, use /root
                        HOME_DIR="/root"
                fi
                echo "usr config delete: ${HOME_DIR}/${SERVER_WG_NIC}-client-${CLIENT_NAME}.conf"
                # remove generated client file
                rm -f "${HOME_DIR}/${SERVER_WG_NIC}"-client-*.conf

                rm -rf /etc/wireguard
		rm -f /etc/sysctl.d/wg.conf
}

P.S. If you do not delete the custom WG configuration files and remove the protective cover, then when you reinstall it, configurations 2, 3, 4, 5 ... for one user will appear in the custom file, and the QR code will become larger and displayed with errors

Partially fixed with https://github.com/angristan/wireguard-install/commit/258eb9441b81f4bb35922fa6cf43fbbf3cf2241e. But yes, we could try to remove client configs when uninstalling.


tools/create-image.sh: support for foreign architectures

For now only s390x is supported as the foreign architecture.
Signed-off-by: Alexander Egorenkov<EMAIL_ADDRESS>
Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

This seems to be somewhat distro-specific.
uname -i says unknown on my machine. So if we move this to create-gce-image.sh and always pass arch in pkg/build, it will think we are doing cross-build even for native build...
What package qemu-$ARCH-static is supposed to come from? I don't have it on my machine, but I have emulation qemu installed, but the binary is named just qemu-s390x.
And the same package changes binfmt_mist accordingly. Messing with global machine settings in this script looks undesirable (we ask for root but that's just because linux is designed in such a way that creation of a local file requires to be a machine admin). Doesn't the package change binfmt_misc in your distro?
Perhaps at this point we should aim at running inside of the syz-env container as it has the emulation qemu installed now with a fixed name... though not the binfmt_misc.
Are there any downsides for using --first/second-stage for native build as well? I am sure tomorrow somebody will change the invocation of debootstrap for native build, but not for the cross-build. Using the same workflow for all modes will make better tested.

This seems to be somewhat distro-specific.
uname -i says unknown on my machine. So if we move this to create-gce-image.sh and always pass arch in pkg/build, it will think we are doing cross-build even for native build...

You are right, it seems to work on Fedora :) uname -m should work, i think.

What package qemu-$ARCH-static is supposed to come from? I don't have it on my machine, but I have emulation qemu installed, but the binary is named just qemu-s390x.
And the same package changes binfmt_mist accordingly. Messing with global machine settings in this script looks undesirable (we ask for root but that's just because linux is designed in such a way that creation of a local file requires to be a machine admin). Doesn't the package change binfmt_misc in your distro?

I installed the package "qemu-user-static". And the package manager also initializes the binfmt entries, on Debian and Fedora.

Perhaps at this point we should aim at running inside of the syz-env container as it has the emulation qemu installed now with a fixed name... though not the binfmt_misc.

I tested first on my Fedora and now tried inside syz-env. The package manager complains that it can't mount binfmt, permission denied :( One solution would be to make the container privileged.

Are there any downsides for using --first/second-stage for native build as well? I am sure tomorrow somebody will change the invocation of debootstrap for native build, but not for the cross-build. Using the same workflow for all modes will make better tested.

I'll check it.

Thanks for useful feedback

I installed the package "qemu-user-static". And the package manager also initializes the binfmt entries, on Debian and Fedora.

Good. I think we should not mess with binfmt_misc here then, just require a properly installed package.
I have qemu-user-static package in my debian disto as well. We should mention at the top or in the error message that we expect it to be installed.

I tested first on my Fedora and now tried inside syz-env. The package manager complains that it can't mount binfmt, permission denied :( One solution would be to make the container privileged or mount binfmt as volume ?

I don't think this is resolvable at all. binfmt misc is global per machine and is not virtualized by containers (or is it? I don't think so). Binfmt misc is modified during package installation, so when we just download the container, the host binfmt misc is not affected.
In the longer run I think we should (1) build images inside of a VM (like some pkg/build implementation do, then this does not have problems with binfmt misc and can work cross-OS too), (2) merge create-image.sh and create-gce-image.sh functionality, (3) and finally rewrite this is Go.
And in short term I don't think there are any good, reliable solutions. So let's just do some minimal amount of improvement and merge.

I don't think this is resolvable at all. binfmt misc is global per machine and is not virtualized by containers (or is it? I don't think so). Binfmt misc is modified during package installation, so when we just download the container, the host binfmt misc is not affected.

I managed to built the root image for s390x with the modified syz-env and using a privileged container to mount binfmt.
Unfortunately, a native build doesn't work in 2 stages :(

In the longer run I think we should (1) build images inside of a VM (like some pkg/build implementation do, then this does not have problems with binfmt misc and can work cross-OS too), (2) merge create-image.sh and create-gce-image.sh functionality, (3) and finally rewrite this is Go.
And in short term I don't think there are any good, reliable solutions. So let's just do some minimal amount of improvement and merge.

Actually, i do not use debian root image for my syzkaller setup. In my opinion buildroot is by far the better choice:

Boot time is way shorter :) Even w/o KVM i need a minute to boot into login prompt !!!
Images are a lot smaller
No root needed !!!

Currently, i'm in the process (finally) of upstreaming s390x architecture to buildroot and when it is ready i'll create a new PR to document the setup with buildroot.

Buildroot is in plans (#1679), but I remembered that it does not support s390 and decided to not mention it to you in the previous reply :)

Pls do not merge yet, trouble with loop device inside container :(

ok, ready for review, tested now natiave root image build for x86_64 on fedora inside container and also foreign s390x root image build.


Remove Reincarnation and Paladin Master Training from Paladins

Il hijack this. il remove the master training but not reincarnation, its just gonna be a quest reward for lordaeron


Users are not allowed to cancel pending requests in android app

This operation is not supported on the backend. That's why it's not in the Android app

I believe this is a stretch feature for backend currently


[feat] Custom URL Scheme

Describe the problem
Hello everyone,
first, thanks for the great product. Tauri might still be fresh-paint, but the product is beautiful and I can't wait to move away from Electron. Great job!
Right now, we are working on an app using Firebase and OAuth, and are obviously experiencing problems with Authentication flows. Another of our needs for the same app is to pass context between a website and the desktop app.
For the latter, the dream is to click on a button on the website which would open (or reuse existing window) the app and pass on the context in a callback dedicated to receiving such context info.
I see Custom URL Scheme registration as the unique possibility of doing this, and I dug a little in the issue history and found I wasn't alone:
https://github.com/tauri-apps/tauri/issues/323
https://github.com/tauri-apps/tauri/issues/1421
I see you guys were hoping to implement this by RC, and I just wanted to ask if you had any updates regarding this?
As for the use-case, consider that it's a big +1 on "I'd love this built into the product".
Let me know if I can provide you with any more details regarding the use-case.
Best,
Loïc
Describe the solution you'd like

A way to declare in the configuration a custom url scheme which would be registered within the OS, associating the url scheme with the app
An API to declare a callback able to handle any external request coming from a browser for instance

Alternatives considered
Copy-paste of contextual data (which is the only way of doing so right now)
Additional context
No response

please follow https://github.com/tauri-apps/tauri/issues/323 for updates, no need to have duplicates


frankenphp: set custom version for build, fixes #397

Output looks like:
FrankenPHP 1.2.5 Caddy v2.8.4 => /usr/src/caddy@(devel)

Awesome. Could you add "WolfiPHP" or something like that in the string so we can easily find that it's a custom build il needed to debug?


DUPLO-26746 [TF] ADD new sql version support for gcp cloud sql

Overview
... (don't forget to put your clickup ticket ID in the title) ...
Summary of changes
Added version support for  POSTGRES_16,POSTGRES_17,SQLSERVER_2022_WEB ,SQLSERVER_2022_EXPRESS,SQLSERVER_2022_ENTERPRISE ,SQLSERVER_2022_STANDARD
in duplocloud_gcp_sql_database_instance resource
This PR does the following:

...
...

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[ ] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-26746 [TF] ADD new sql version support for gcp cloud sql


Help sent to stderr, not stdout

Minor annoyance, but this means normal pager (more) use needs the extra redirect, which, yeah, is annoying:
hugo help server 2>&1 | more

Many would prefer
hugo help server | more

Should be a relatively easy fix that I will eventually PR if no one else beats me. (Just not able to for about 3 weeks.)

This is from the cobra library, not Hugo: https://github.com/spf13/cobra/issues/114

This issue has been automatically marked as stale because it has not been commented on for at least four months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still valuable, please open a proposal at https://discuss.gohugo.io/.
This issue will automatically be closed in four months if no further activity occurs. Thank you for all your contributions.

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.


New tutorial pushed to production for author.

I created a new Pull request https://github.com/SAPDocuments/Tutorials/pull/7154 for this merge after some updates.


snap support

A candidate/stable release snap is created for cross platform installation among various Linux distros, including CentOS, KDE Neon, openSUSE, Red Hat and many more.
snap is already live on : https://snapcraft.io/hexyl
you can check out that.

You should probably not commit the snap, but instead setup CI to build it.

Thank you very much for your contribution!
As @bjorn3 stated, we don't really want to add a binary snap file to this repository. Also, as I am not maintaining the snap file myself (and not planning to), I would prefer if distribution-specific files would be hosted somewhere else (e.g. in a separate repository).
I didn't really understand why you would like to add additional testdata files?
What we can definitely do is to add installation instructions for snap to the README.

@sharkdp , thanks for check out my pull request.
I understand your concern and i am very happy to host a separate repository for snap distribution of this repo.

@purvesh-linux11 Thank you!
I'm going to close this PR. Happy to include the README changes at some point.


Test Job 2

AWARENESS
2023-11-07
Saw job on LinkedIn
ATTRACTION

I like this jobs because it is in a very convenient location
the company is a well known company and there will be lots of opportunities to grow

INTEREST
APPLICATION

2023-11-07
Applied via Website

INTERVIEW

2023-11-14
Telephone Interview

FEEDBACK

2023-11-17
Feedback received; Need more experience, recommend I try to work on some real world applications such as contributing to Open Source projects

OFFER
UNSUCCESSFUL

2023-11-17
Unsuccessful, but feedback suggested an area I can work on
.

2023-11-17
Unsuccessful, but feedback suggested an area I can work on


Incomplete support for Corosync 2.x

This module has no proper way to enable Pacemaker without a service plugin. The documented way of enabling Pacemaker as per https://github.com/puppet-community/puppet-corosync/blob/master/README.md is to add
corosync::service { 'pacemaker':
  version => 0,
}

which is a bad idea to begin with, since version 1 had long been the preferred service plugin.
However, this configuration mode makes no sense at all on systems past Corosync 2.0 which did away with service plugins. This means that you currently have to do
service { 'pacemaker':
  ensure => running,
  require => Class['corosync'],
}

to get Pacemaker to run with Corosync 2.x, and then have all cs_* resources require Service['pacemaker'], which really ought to be an auto-dependency.

We had this exact issue with the puppet module yesterday! Thank you for the info @fghaas - hopefully this can be patched soon?

@sammcj I'm not entirely sure how much work it would be; particularly getting the autodependencies right is probably somewhat invasive. Does the suggested workaround work for you, i.e. adding service { 'pacemaker': } and then require => Service['pacemaker'] on the cs_ resource types?

@fghaas not really - it seems the module is missing a lot of logic around dealing with pcs in general. We've just installed crmsh from SUSE on CentOS 7 and forced the module to use crm instead which works 'alright'.

@sammcj Elaborate please? I'm running this module (from git master; 0.7.0 is broken as per #144) with pcs and haven't run into major snags yet.

For the time being, the most economic workaround for users who don't rely on virtual resources will be
Service['pacemaker'] -> Cs_location<| |>

etc. for each cs_* type.

Well yes that's true, but I'm still curious as to what other missing logic @sammcj is referring to.

Depending on the scope of that, I'd like to have a new 'pcs provider is not up to par with crm' issue for that, so as not to fuzz the conversation here too much.

If it indeed is not.

Is this still an issue?

Will be closed by #245.


[ Feat ] Cleanup and improves

Changes

[x] improve format config
[x] modify wrong version in Scarb_deploy.toml
[x] update script
[x] dd overlay so that we can grant writer role for each actions to necessary models much faster and easier!
[x] update world address n account address for slot env
[x] update README
[x] add vscode task so that we can easily setup katana and torii keeping data and latest version.

Thanks for the PR, LGTM
Keep in mind that the World Address changes with every version of Dojo... so needs to be updated, even in the .vscode/tasks.json


Add const values for SDD400 rippleSHIELD dimmer

Added model id for SDD400 pulled from mitmproxy logs. Also added value to is_light and is_dimmable lists

Thanks!


Add oauth scope to allow communication with gcr

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: selmanj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gyliu513
If they are not already assigned, you can assign the PR to them by writing /assign @gyliu513 in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @selmanj. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/assign @gyliu513

Pinging CR so it isn't marked as stale.


explain lingua franca

... to those of us without Latin education

_linf

I would either leave lingua franca or just replace it with common language. I am happy with some jargon busting, using latin here does not add anything

FYI, lingua franca occurs in the Longman Dictionary of Contemporary English  :-)

I don't fear it. But I think it is a term that a lot of readers would not be familiar with and would need to lookup - and that without clear need why we would make them do that.


The filter 'route.gateway-id' for DescribeRouteTables has not been implemented in Moto yet.

Would you consider adding support for route.gateway-id filter in the DescribeRouteTables command?

Marking as an enhancement! Thanks for raising this @jawillis


Integration tests need to be updated

Integration tests need to be updated to keep pace with latest cookbooks and attribute refactoring work.

TODO:

Add databag files for composer, magento, mysql w/o encrypted secret values.
Document how / when / where to generate secret values in order to run integration tests.

 - ohai (5.1.0)
       Installing Cookbook Gems:
       Compiling Cookbooks...
       id: development: no such user
       [2017-08-14T21:45:51+00:00] WARN: Skipping action: '' of non-existing user 'development'
       [2017-08-14T21:45:51+00:00] WARN: Skipping action: '' of non-existing user 'development'
       id: development: no such user
       [2017-08-14T21:45:51+00:00] WARN: Skipping action: '' of non-existing user 'development'
       [2017-08-14T21:45:51+00:00] WARN: Skipping action: '' of non-existing user 'development'

       ================================================================================
       Recipe Compile Error in /tmp/kitchen/cache/cookbooks/cop_magento/recipes/data-bag.rb
       ================================================================================

       Net::HTTPServerException
       ------------------------
       404 "Not Found"

       Cookbook Trace:
       ---------------
         /tmp/kitchen/cache/cookbooks/cop_magento/recipes/data-bag.rb:11:in `check_bag_existance'
         /tmp/kitchen/cache/cookbooks/cop_magento/recipes/data-bag.rb:37:in `block in from_file'
         /tmp/kitchen/cache/cookbooks/cop_magento/recipes/data-bag.rb:36:in `each'
         /tmp/kitchen/cache/cookbooks/cop_magento/recipes/data-bag.rb:36:in `from_file'

       Relevant File Content:
       ----------------------
       /tmp/kitchen/cache/cookbooks/cop_magento/recipes/data-bag.rb:

         4:  required_data_bags = [
         5:      'composer',
         6:      'magento',
         7:      'mysql'
         8:  ]
         9:
        10:  def check_bag_existance(bag_name)
        11>>     search(bag_name, "id:#{$bag_environment}").any? ? true : log_error("You don't have a data-bag for environment: #{bag_name} / #{$bag_environment}")
        12:  end
        13:
        14:  def log_error(error_msg)
        15:      Chef::Log.fatal(error_msg)
        16:      fail error_msg
        17:  end
        18:
        19:  def update_mysql_vars(data_bag_item)
        20:      node.normal['mysql']['root_password']                                = data_bag_item['root-password']

       System Info:
       ------------
       chef_version=13.1.31
       platform=centos
       platform_version=7.3.1611
       ruby=ruby 2.4.1p111 (2017-03-22 revision 58053) [x86_64-linux]
       program_name=chef-client worker: ppid=17298;start=21:45:48;
       executable=/opt/chef/bin/chef-client

       Running handlers:
       [2017-08-14T21:45:51+00:00] ERROR: Running exception handlers
       [2017-08-14T21:45:51+00:00] ERROR: Running exception handlers
       Running handlers complete
       [2017-08-14T21:45:51+00:00] ERROR: Exception handlers complete
       [2017-08-14T21:45:51+00:00] ERROR: Exception handlers complete
       Chef Client failed. 0 resources updated in 03 seconds
       [2017-08-14T21:45:51+00:00] FATAL: Stacktrace dumped to /tmp/kitchen/cache/chef-stacktrace.out
       [2017-08-14T21:45:51+00:00] FATAL: Stacktrace dumped to /tmp/kitchen/cache/chef-stacktrace.out
       [2017-08-14T21:45:51+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
       [2017-08-14T21:45:51+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
       [2017-08-14T21:45:51+00:00] ERROR: 404 "Not Found"
       [2017-08-14T21:45:51+00:00] ERROR: 404 "Not Found"
       [2017-08-14T21:45:51+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
       [2017-08-14T21:45:51+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 1 actions failed.
>>>>>>     Converge failed on instance <default-centos-7>.  Please see .kitchen/logs/default-centos-7.log for more details
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration


Webpage links to GH for downloads, no downloads found

On your webpage you link to https://github.com/elcojacobs/ShiftPWM/downloads for downloads, there are none on that adress.
I understand if you don't have the time to update the links but please at least upload README.md to your repo with instructions on how to compile this.

@D3add3d The ShiftPWM library is here:
https://github.com/elcojacobs/ShiftPWM/archive/master.zip
Or are you looking for something else?

@marmilicious well.. I can just clone it* using git but I'm just saying there are no files on given adress and 80% of the links on his webpage are dead.
*but I still don't know why is there a .cpp file and all the other non-sense

Do you realize your message is being read by people here on the Github, who do not necessarily know or have the link to the web page you're reporting?

@PaulStoffregen here you go: http://www.elcojacobs.com/shiftpwm/

None of us have access to edit that web page.  Sorry.


uv pip compile sometimes outputs ansi escapes in file

Experienced behaviour: Can't call "rye add" or "rye sync" when uv is enabled in rye, from a jupyterlab python kernel command line.
I think this could be a bug in uv, that's why I'm reporting it here. This is how I reproduce it:

Using jupyterlab (v3.6 or v4.1 both show this problem), create a new notebook in a new directory.
Write 'numpy>=1.2' to 'requirements.txt'
Run !uv pip compile -o "outputfile" "requirements.txt"  in a python notebook
Bug: outputfile contains ansi escapes in the lockfile output.

For some reason this happens in the notebook environment but not in a regular terminal. The original bug in rye seems to be related to this - it also gets this behaviour when called from a notebook. Rye could work around it using --no-color, but it might be something that can be fixed in uv.
When using rye in the notebook the output looks something like this:
!rye add numpy

Added numpy>=1.26.4 as regular dependency
Reusing already existing virtualenv
Generating production lockfile: /.../newjupyterlabproject/requirements.lock
error: Unexpected '', expected '-c', '-e', '-r' or the start of a requirement in `/tmp/.tmpXbPWlj/requirements.txt` at position 221
error: could not write production lockfile for project

Caused by:
    failed to generate lockfile

uv 0.1.11
Linux

It also affects VSCode notebooks 👍


Top Taskbar Issue

I have my taskbar at the top and the icons overlap the opened program.
This only happens when I'm using browsers (Firefox and Edge) and TaskbarX transparent style.
Here's how it looks like with and without TaskbarX

Sadly this is something i can't fix. Because it's not caused by TaskbarX. This indeed only happens with browsers... Chrome does it too.


[FEATURE]: "Get KOIN" on koinos.io

Is there an existing issue for this?

[X] I have searched the existing issues

New feature
Use the same design as the "wallets" section to add a "Where to get KOIN" section on koinos.io.
Anything else?
No response

Resolved by #49


Issue with CDC Sink Connector from Debezium MongoDB Connector

Using the example for Change Data Capture Mode from the README, we are seeing an issue that throws an unrecoverable exception right after the connector config is posted.
Connector def posted below:

And error thrown posted below:

The collection exists in mongo and mongo is reachable from kafka connect. The confluent schema registry is running and is reachable from kafka connect. Also, other kafka streams applications are able to deserialize these debezium-generated avro messages fine from the same topic the failing connector is reading from. Please let us know if you need any more information to help us solve. Thank you.

@kalbmj thx for reaching out. I'm happy to see that you try to make use of the mongodb cdc handler.
what you report is an unfortunate regression bug which most likely happened due to one of my recent refactorings. luckily it will be easy to get rid of the error :) I'll provide a fix as well as a test case which reproduces and addresses this issue as soon as I find some time, most probably later today.

Thank you for looking into this. Another related question, do you have (or have you seen) any use case examples for CDC from MongoDB to MySQL?

MongoDBHandler is extending DebeziumCDCHandler but they are both maintaining their own operations map.    DebeziumCDCHandler never sets it operations map with values, but that is where the getCdcOperation is at
So we added
public DebeziumCdcHandler(MongoDbSinkConnectorConfig config) {
super(config);
logger.info("DebeziumCdcHandler contructor config {}",config.toString());
operations.put(OperationType.CREATE,new MongoDbInsert());
operations.put(OperationType.READ,new MongoDbInsert());
operations.put(OperationType.UPDATE,new MongoDbUpdate());
operations.put(OperationType.DELETE,new MongoDbDelete());
    logger.info("DebeziumCdcHandler operations map {}",operations);
    logger.info("DebeziumCdcHandler operations map size {}",operations.size());
}

Now working

while this indeed works it's not good way to leave it like this :) I'm currently working on a fix together with an additional test. as soon as it it merged I'll let you know and you might want to use the new version instead.

@kalbmj fixed -> see https://github.com/hpgrahsl/kafka-connect-mongodb/pull/12
have fun!


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in 5d83796.


validation_error when creating a new release with docker

$ flynn-host version
v20170116.0p2

$ flynn --version
v20170217.0

$ flynn -c test -a app docker push docker/app
flynn: getting image config with "docker inspect -f {{ json .Config }} docker/app"
flynn: tagging Docker image with "docker tag docker/app docker.example.com/app:latest"
flynn: pushing Docker image with "docker push docker.example.com/app:latest"
The push refers to a repository [docker.example.com/app]
28709a368e58: Layer already exists
7ab4c3a7c561: Layer already exists
...
88f1e0775099: Layer already exists
fe4c16cbf7a4: Layer already exists
latest: digest: sha256:09c8d7c8ea3441d2d1bd265e5b7f7aff4138b4aab72a515e7184cc2abdafa659 size: 41843
flynn: image pushed, waiting for artifact creation
flynn: deploying release using artifact URI http://blobstore.discoverd/docker-receive/images/97445b583235e3837539be435dfd6deca43f67b2bf96fe2e6edd0b6516acfe11.json
validation_error: release is not deployable

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


Tan11389/fix Citra broken links

Updates our URLs and text to be consistent with the new developer site.

@ldanzinger please review the updated URLs when you can


Editing peers is different from Viewing a peer

The edit peer mode should be distinct from viewing a peer (two separate buttons).
The main 'peer' menu where they are all listed should be able to directly delete a peer.
Add a tool hint for the IP Address field explaining what it means, and why can't it be edited?
When deleting a peer, the confirmation should include what it is you are confirming. And apparently there's an error when I try to delete the default Laptop peer " Fix the errors before saving. Warnings can be ignored. ".
That warning shouldn't move the buttons across
Move the 'add peer' button to the top, above the table, and give it a clear name 'add peer'.

Editing and viewing can remain the same. Not going to write code for something which does not cause any issues.
I have plans for deleting objects from the list page itself. Will do it in next phase.
IP address field has field has tool-tip now.
Warning now appears below the buttons.
That add-button - I am still not sure where it should appear. I have tried options like above the table, below the table. They all look equally ugly.


Add query: WebP adoption by WordPress version.

Calculate the percent of pages that use WebP images, broken down by WordPress version and client (desktop/mobile)

version
pct_webp_mobile
pct_webp_desktop

6.0
0.092
0.173

5.9
0.067
0.154

5.8
0.054
0.14

5.7
0.032
0.12

5.6
0.028
0.12

5.5
0.026
0.117

5.4
0.026
0.118

5.3
0.024
0.113

5.2
0.02
0.111

5.1
0.014
0.102

5.0
0.014
0.107

4.9
0.012
0.098

4.8
0.011
0.097

4.7
0.009
0.091

4.6
0.008
0.08

4.5
0.008
0.079

4.4
0.011
0.092

4.3
0.008
0.073

4.2
0.008
0.081

4.1
0.006
0.08

4.0
0.008
0.072

3.9
0.004
0.069

Based on Oct 2022 data

@felixarntz this is updated now with the proper location and link.
I tested adding AND category='CMS' to the query (based on the comment on #18) and didn't see any change in the results - so leaving that off for now.


Feature auto link

Fix #185

@kepta I just updated the tld regexp, also handled a corner case (see the last commit)


TypeError: object.init() takes exactly one argument (the instance to initialize)

Hi,
I have been using the holidays package since the last six months for my enterprise's project and never have had this issue before:

I don't know if this comes from the recent release of version 0.25 or not. But could someone please check on this and help me out. Thanks a ton in advance!

Thanks for teh response @ngerald . Do you have some sort of evidence to back this claim? For instance, is this bug a result of introducing entity lazy loading as in https://github.com/dr-prodigy/python-holidays/pull/1087?

This issue on the holidays package repo should serve as evidence: https://github.com/dr-prodigy/python-holidays/issues/1200

@TaherHabib if it's related to prophet you need to update to the most recent version that has the issue fixed.
If it's something else -- your enterprise project's code needs to be changed to import entries from holidays.countries / holidays.financial instead of directly from holidays for inheritance.

Hi @arkid15r, many thanks for responding.
No, the issue is not related to prophet in this particular case. It is rather that I am creating my enterprise's own holiday class which, by design, inherits from holidays.AT or holidays.DE to populate the standard calendar holidays; and on top, I am adding a few more holidays to the dict which are unique to my enterprise.
It is when the interpreter executes this inheritance code that the error gets generated.

Also, your proposed fix doesn't solve my issue. Based on your suggestion, I basically:
changed the class definition/inheritance from this:
import holidays
class SomeHolidays(holidays.AT)

To this:
import holidays
class SomeHolidays(holidays.countries.AT)

And now, it gives me this error:  AttributeError: module 'holidays' has no attribute 'countries'
I am sure I am making some mistake here in terms of interpreting your suggested solution, could you please elaborate on the matter? Thanks!

@TaherHabib, please, try
from holidays.countries import AT

class SomeHolidays(AT):
    """
    <Body of the class>
    """

Thank you @KJhellico for the suggestion. It seems to have worked for me :)
However, now I am faced with a different issue! :/
The inheritance and class initialization works fine now, however, when I call the MyEnterpriseHolidays method get_dict_holidays(self, subdiv) which basically returns the holiday dictionary (object of SomeHolidays(<country_code: AT or DE>)) based on the country and the subdivision, I get the following error traceback when running for Austria:

Line where the error occurs:

Pytest Summary of all tests using MyEnterpriseHolidays():

Skeleton of my class:
from holidays.countries import AT

class MyEnterpriseHolidays():
    def __init__(self, ...):
        # <class attributes incl. `country_code`>

    def get_dict_holidays(self, subdiv):
        if self.country_code == 'AT':
            return SomeHolidaysAT(
                        subdiv=subdiv,
                        years=list(range(some_past_yr, some_future_yr))
                        )
        elif self.country_code == 'DE':
            return SomeHolidaysDE(
                        subdiv=subdiv,
                        years=list(range(some_past_yr, some_future_yr))
                        )

class SomeHolidaysAT(AT):
    """
    <Body of the class>
    """

class SomeHolidaysDE(DE):
    """
    <Body of the class>
    """

Any suggestions on where this might be going wrong or how to fix it? Thanks once again! Really appreciate the help

This error means that most likely holiday from year other than year is being added somewhere in SomeHolidaysAT::_populate().

@KJhellico Thanks a lot for your inputs. The package now works just fine, as expected, again! :)


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit c57f74d:
:clock10: Pending: waiting for processors (31 builds ahead of you)
:warning: Docs Build is busy, currently there are 31 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit c57f74d:
:clock10: Full build: the average full build time is 7 min(s) 50 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit c57f74d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Warning in google console Tests

Users running the app on devices running Android 12 or later will see two splash screens.
To resolve the issue, release an updated version of the application that uses the SplashScreen API.

Any updates on this?

same, any update?


Update azure-devops-auditing.md

Updating list to include new audit event types: User and Token/Authentication

@angelwong : Thanks for your contribution! The author(s) have been notified to review your proposed change. @roferg

Hi @chcomley - @roferg is listed in the metadata author attribute, but they aren't in the Microsoft Docs organization, so I can't assign the pull request to them. I'll assign it to you instead, OK? Thanks.

Hi @angelwong - If you're internal to Microsoft and a PM, per the internal CG, you should work in the private repo. Let me know if you need information about working in the private repo; I can send it to you in email. Thanks.

@chcomley Thanks for your review.
When the pull request is set to Ready for review, we'll check for updates and either merge or check in again with the content team. Thanks.


TECH: StringDeduplicator: deduplicate only emptystring by default

Everything else might be too small to outweigh the overhead, or not even that redundant to begin with. We should provide some other means to decide on a field to field basis when to deduplicate inside de-externalization.

Is this an assumption or baked by data?

It is backed by the fact that loading WatchDAO from dumps yields a DAO that looks smaller, but a StringDuplicator that is much fatter, so totals are far worse. This is at least part of the reason why heap consumption keeps slowly creeping up over time.

The optimum solution would be to define the behavior through an annotation property, so that fields which profit greatly from content deduplication can make use of it again. For the general case though, we should enforce only scenarios where the win is guaranteed, as with redundant emptystring instances. Plus, for any temporary entities not kept in memory, deduplication means just a waste of additional allocation and gc.


Enhancement - Keyboard shortcuts

I would love to be able to use the arrow keys or the ability to bind them to other keys for the next/prev action/turn. I'd love if it WASD and the arrow keys could work as those are some of the most basic shortcuts, also my left arrow is broken so I'd love an alternative besides just the mouse.
The most basic setup could be the following.

Left / A for last action
Right / D for next action
Up or W for last turn
Down or S for next turn

These should be available under Control Settings → Rebind Keys when you hover near the top.

oh wow, didn't see that at all. Maybe include the key bind info to the readme for higher visibility?


Auto correct using cookstyle

Description
Autocorrect using cookstyle

          2 Errors

:no_entry_sign:
Please add labels to this Pull Request

:no_entry_sign:
Please include a CHANGELOG entry.

  Generated by :no_entry_sign: Danger


Next.js With Express and Spdy

Bug report
Describe the bug
I've read the other issues pertaining to http2 and next, and how next shouldn't really interfere with you using spdy. However i've come across an error I can't get past, which seems to only come up with next.js in the mix. Maybe somebody will recognize the error or be able to provide me with some more info.
When I run my app with a basic express only get request, /plain in the example, the app runs, serves up with h2 and displays the message on screen. However, when i run the app with .get('*', (req, res) => handle(req, res)) i get the error listed below.
This leads me to believe it's coming from the handle function, which is necessary for next.js, am i correct in this? Maybe i'm just passing things around improperly? Can anybody see a potential fix for this? I'm happy to provide any more information or code at your request. Thanks!
To Reproduce
// Dependencies
const next        = require('next'),
      express     = require('express'),
      spdy        = require('spdy'),
      fs          = require('fs'),
      dev         = process.env.NODE_ENV !== "production",
      app         = next({ dev }),
      handle      = app.getRequestHandler()

// Spdy Setup
const spdyOptions = {
  key: fs.readFileSync( `server.key` ),
  cert: fs.readFileSync( `server.crt` )
}

// Server
app
  .prepare()
  .then( () => {

    const server = express()

    server

      // Plain Express Route
      .get('/plain', (req, res) => {
        res.status(200).json({message: 'ok'})
      })

      // Next.js Route
      .get('*', (req, res) => handle(req, res))

    spdy
      .createServer(spdyOptions, server)
      .listen(3103, err => {
        if (err) throw err
      })

  })
  .catch( ex => {
    console.error(ex.stack)
    process.exit(1)
  })

Expected behavior
The server should start and serve the next app but instead I get this error:
internal/stream_base_commons.js:62
  var err = req.handle.writev(req, chunks, allBuffers);
                       ^
TypeError: req.handle.writev is not a function
    at writevGeneric (internal/stream_base_commons.js:62:24)
    at Socket._writeGeneric (net.js:711:5)
    at Socket._writev (net.js:720:8)
    at doWrite (_stream_writable.js:408:12)
    at clearBuffer (_stream_writable.js:517:5)
    at Socket.Writable.uncork (_stream_writable.js:314:7)
    at connectionCorkNT (_http_outgoing.js:641:8)
    at process._tickCallback (internal/process/next_tick.js:63:19)

System information

OS: macOS High Sierra
Version of Next.js:  7.0.0

This actually works on node v8, i was running this on node v10. Closing this issue as it doesn't seem to have anything to do with next, rather a node issue.

Same problem (node v10) you can use something like this
https = dev ? require('https') : require('spdy')


MIS on AHS notebook

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Do we want a reference to the MIS paper?


CAMEL-14562: camel-elastichsearch - Use testcontainers for testing

@lburgazzoli can you please fix the codestyle?

done


using dirty in a module

Hi,
I am trying to dirty in a module, sample code below, that code returns 'not loaded', what is a good way to put dirty in a module? Thanks
var dirty = require('dirty');
var db = dirty('sample.db');

var loaded = false

db.on('load', function() {
    console.log('loaded')
    loaded = true
})

exports.get_all_items = function() {
    return new Promise(function (resolve, reject) {
        if (loaded)
            resovle(loaded) // just a test for now, should return all items in the db
        else
            reject('not loaded')
    })
}

found a solution:
https://gist.github.com/angelochen960/d1eeaf761e2504876919e7ade9697235


Interceptors not running

I have imported HttpModule and HttpInterceptorModule, and created an interceptor service:
import { Injectable } from '@angular/core';
import { Http } from '@angular/http';
import { HttpInterceptorService } from 'ng-http-interceptor';

@Injectable()
export class HttpErrorResponseInterceptor {

  constructor(private http: Http, private httpInterceptor: HttpInterceptorService) { }

  initialise() {

    this.httpInterceptor.request().addInterceptor((data, method) => {
      console.log(method, data);
      return data;
    });

    this.httpInterceptor.response().addInterceptor((res, method) => {
      return res.do(r => console.log(method, r));
    });

    this.http.get('/')
      .subscribe(console.log);
  }
}

initialise() is then called in the top level AppComponent.
When I debug in Chrome/IE/Firefox I can see http.get is called - but the interceptors never run, and there are no errors. I have tried injecting InterceptableHttp instead of Http, and the interceptors still don't run.
I am using Angular 4, specifically, the versions I am using are:
"@angular/http": "^4.3.1",
"ng-http-interceptor": "^3.1.2",

What is the order of imports of HttpModule and HttpInterceptorModule?
Because if you have HttpModule after HttpInterceptorModule it will override proxied Http service with original one.

The problem still occurs if I import HttpModule first. I was wondering if something like that was occurring, but then I would think InterceptableHttp should still work, which it doesn't.

Yeah you are right, InterceptableHttp should have been worked any way...
It's weird why requests are not intercepted, can you please make a plunkr with this issue reproduced?

I have the same setup as you, and I was using HttpInterceptorModule.noOverrideHttp() and I didn't get any logs, I switched to normal module HttpInterceptorModule and now I can see all requests. (Ionic 3 app).

I have identified the cause of the issue. Another plugin I am using in my project seems to provide its own HTTP implementation via a factory:
https://github.com/aitboudad/ng-loading-bar
As each plugin provides its own implementation of HTTP I think the 2 plugins are incompatible.

Yeah, only the last Http provided will be used. But you should be able to still use InterceptableHttp to make interceptable requests.

I really need to be able to intercept ALL Http requests.  Everything throughout my entire app lifecycle, including all DI.

This is a really interesting case when you really want to compose your DI providers but not completely override them.
So far I don't know any solution of how to achieve this but it definitely makes sense in such scenarios.
Maybe we should open an issue in official Angular repo and discuss this problem/feature...
What do you guys think?


docs: introduce a new page for neutron adoption

This is largely the same info as captured in other_services page. The latter now goes away because it's not the right place to keep adoption steps documented for any service. Any new service should start with its own separate page.
Note: neutron instructions cover only starting API service for now. Later, the page will be expanded to cover SR-IOV and any other scenarios that may require additional adoption steps.

merging as we have 2 approvals

recheck

recheck


naive encoding circuit to the ecc module

This seems like a very good start, and the fundamental math seem to be on the right track.
There are some software engineering things we need to clean up though.

There are some git related issues too.

you have been developing on your master branch instead of making a separate branch -- that can make keeping this work in sync with the main repo quite difficult and we should probably fix it
there are merge conflicts between your changes and how the library has evolved since you have started making the changes. This is usually fixed with a merge commit or a rebase, but both of these are difficult if you have worked on top of the master branch.


Button to turn composer pattern rec off

When a pattern follower is set, sometimes composer will open with the pattern follower engaged and it's annoying. A quick button to 'turn off and back on' once a pattern has been made would be nice.
The issue is that if you reopen the composer and it does a pattern follow, if you recenter the grid it'll be off when you do a run.

Ok - I think this is how it works now (First time using this cool framework called mermaid to make diagrams)
flowchart TD;
    nextImage-.->goodPatternFollow;
    nextImage-.->nullPatternFollow;
    goodPatternFollow-->updateOffset;
    nullPatternFollow-->stored;
    updateOffset-->stored;
    stored-->nextImage;
    stored-->composer;
    nullOffset--Initial Value-->stored;
    stored:::someclass;
    classDef someclass fill:#f96;
    nullOffset:::nullclass;
    classDef nullclass fill:#f69;

And you would like it to be like this?
flowchart TD;
    nextImage-.->goodPatternFollow;
    nextImage-.->nullPatternFollow;
    goodPatternFollow-->updateOffset;
    nullPatternFollow-->stored;
    updateOffset-->stored;
    stored-->nextImage;
    stored-.-toggle;
    nullOffset--Initial Value-->stored;
    stored:::someclass;
    classDef someclass fill:#f96;
    nullOffset:::nullclass;
    classDef nullclass fill:#f69;
    nullOffset-.-toggle;
    toggle-->composer;

@riversn does this look correct?

Yeah

I think I fixed this? I noticed that the Feature Layout in the composer always uses the follower pattern offset, but the Feature Grid never does. So, now the Feature Grid always uses the follower pattern offset. I think this removes the confusion as the composer will now always reflect the viewer and any grid location changes will be in relation to what you are looking at, not what you are looking at +/- the possible follower pattern. That being said - I might have missed the nail - we'll see.

one annoying thing I noticed is teaching a new pattern after successful pattern rec - best to do it on the very first image of the population and then you don't run into that issue

I want the composer to never shift the grid position, force pattern in the viewer, or running should be the only time the pattern rec gets applied to a grid

Yeah I did the opposite - I can flip it just as easily.
But right now, If PR happens and then you stop the run and open composer, you can edit the grid and feature and those changes will be consistent with null & future PR offsets.
If I do the opposite, you would basically have to go to back to the image you set up the PR on for the grid to line up again.
In the version you have now - it is half and half - the grid never gets the PR and the feature always gets the PR. Whichever you decide (always use or never use) they will both at least be the same.

maybe always on is fine if there's a manual on and off pattern rec button.

okay - moved to #119


bootstrap-vcpkg.sh -disableMetrics flag not passed to bootstrap-vcpkg.bat

On Windows, calling bootstrap-vcpkg.sh will redirect to bootstrap-vcpkg.bat.
However, -disableMetrics flag is not passed to bootstrap-vcpkg.bat

Oops! Lemme fix it right now.


Error when setting version attribute

When locking the version in my wrapper cookbook I get the following error I can't seem to get fixed.
If I construct the web_version mysql I get '2.4.1-1' as expected.
node.default['icinga2']['web2']['enable'] = true
node.default['icinga2']['web2']['version']['icingaweb2'] = '2.4.1'
node.default['icinga2']['web2']['version']['icingacli'] = '2.4.1'

NoMethodError
-------------
Undefined node attribute or method `+' on `node'

Cookbook Trace:
---------------
  /var/chef/cache/cookbooks/icinga2/recipes/server_web2.rb:21:in `from_file'
  /var/chef/cache/cookbooks/icinga2/recipes/server.rb:46:in `from_file'
  /var/chef/cache/cookbooks/valtechnl_icinga/recipes/server.rb:17:in `from_file'
  /var/chef/cache/cookbooks/valtechnl_icinga/recipes/default.rb:7:in `from_file'

Relevant File Content:
----------------------
/var/chef/cache/cookbooks/icinga2/recipes/server_web2.rb:

 14:  # Unless required by applicable law or agreed to in writing, software
 15:  # distributed under the License is distributed on an "AS IS" BASIS,
 16:  # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 17:  # See the License for the specific language governing permissions and
 18:  # limitations under the License.
 19:  #
 20:
 21>> web2_version = node['icinga2']['web2']['install_method'] == 'source' ? 'v' + node['icinga2']['web2']['version'] : node['icinga2']['web2']['version'] + '-' + node['icinga2']['web2']['release'] + node['icinga2']['icinga2_version_suffix'].to_s
 22:  cli_version = node['icinga2']['web2']['version'] + '-' + node['icinga2']['web2']['release'] + node['icinga2']['icinga2_version_suffix']
 23:
 24:  puts web2_version
 25:
 26:  directory node['icinga2']['web2']['conf_dir'] do
 27:    owner node[node['icinga2']['web_engine']]['user']
 28:    group node[node['icinga2']['web_engine']]['group']
 29:    mode '02770'
 30:  end

@dpnl87 Could you please perform upstream pull and retry?

@vkhatri I'm running on master for this repo. Not really sure what you mean with upstream?

Sorry for the confusion, I meant to say pull master branch.
Current master branch has simplified attributes.
https://github.com/Icinga/chef-icinga2/blob/master/attributes/server_web2.rb#L6
default['icinga2']['web2']['version']['icingaweb2'] = '2.4.1-1'
default['icinga2']['web2']['version']['icingacli'] = '2.4.1-1'

https://github.com/Icinga/chef-icinga2/blob/master/recipes/server_web2.rb#L22 :
cli_package_version = node['icinga2']['web2']['version']['icingacli'] + node['icinga2']['icinga2_version_suffix']
web2_package_version = node['icinga2']['web2']['version']['icingaweb2'] + node['icinga2']['icinga2_version_suffix']
web2_source_version = 'v' + node['icinga2']['web2']['version']['icingaweb2'].split('-')[0]

I'll do some more testing on a simulated environment. For now I was able to work around this issue by setting node.default['icinga2']['ignore_version'] = true

Closing as the issue was fixed.


Build fails on enabling "tests building" in latest docker

CMake Error at thirdparty/CMakeLists.txt:12 (add_subdirectory):
The source directory
/root/gstreamer-plugins/thirdparty/googletest

does not contain a CMakeLists.txt file.
-- Found OpenCV: /usr/local (found version "4.1.0") found components:  core imgproc
-- Checking for module 'gstreamer-check-1.0'
--   Found gstreamer-check-1.0, version 1.16.0
-- Found OpenCV: /usr/local (found version "4.1.0") found components:  core imgproc imgcodecs
-- Configuring incomplete, errors occurred!
See also "/root/gstreamer-plugins/build/CMakeFiles/CMakeOutput.log".
See also "/root/gstreamer-plugins/build/CMakeFiles/CMakeError.log".
The command '/bin/sh -c mkdir -p gstreamer-plugins/build         && cd gstreamer-plugins/build         && cmake         -DCMAKE_INSTALL_PREFIX=/usr ..         -DVERSION_PATCH=${SOURCE_REV}         -DGIT_INFO=${GIT_INFO}         -DBUILD_SHARED_LIBS=ON         -DENABLE_PAHO_INSTALLATION=${ENABLE_PAHO_INSTALLATION}         -DENABLE_RDKAFKA_INSTALLATION=${ENABLE_RDKAFKA_INSTALLATION}         -DCMAKE_INSTALL_PREFIX=/usr ..         && make -j $(nproc)         && make install         && echo "/usr/lib/va-gstreamer-plugins" >> /etc/ld.so.conf.d/opencv-dldt-gst.conf && ldconfig' returned a non-zero code: 1

Hi @vijnasu,
Can you recheck? Don't see line from the error. There could be mistake during uploading release, but seems that CMakeLists.txt should be fine right now.

@vijnasu , cannot reproduce and no answers.  Feel free to reopen.


Longitude.me example no longer works in Google Chrome

This is not strictly an issue with this code, but an issue with the website you link to as an example.
The website shows a popup that says:

You have to allow the website to know your location

The Chrome DevTools window shows the following message:

getCurrentPosition() and watchPosition() no longer work on insecure origins. To use this feature, you should consider switching your application to a secure origin, such as HTTPS. See https://goo.gl/rStTGz for more details.

It is free (and very easy) to get a certificate from LetsEncrypt.org.

Thanks for the report! I'm aware of this, and I plan to set up TLS for Longitude.me (with a cert from Let's Encrypt) when I get some time over.


V1.1 features

Refactor Person to Student
Remove Phone field for Student

Codecov Report

Merging #86 (adc8a57) into master (c863611) will decrease coverage by 1.00%.
The diff coverage is 78.26%.

@@             Coverage Diff              @@
##             master      #86      +/-   ##
============================================
- Coverage     60.00%   58.99%   -1.01%
+ Complexity      407      389      -18
============================================
  Files            87       85       -2
  Lines          1510     1456      -54
  Branches        161      152       -9
============================================
- Hits            906      859      -47
+ Misses          561      557       -4
+ Partials         43       40       -3

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...a/seedu/address/logic/commands/AddTaskCommand.java
0.00% <ø> (ø)

...edu/address/logic/commands/ListStudentCommand.java
0.00% <0.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
80.00% <ø> (-5.72%)
:arrow_down:

.../seedu/address/logic/parser/FindCommandParser.java
100.00% <ø> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
69.44% <ø> (-6.65%)
:arrow_down:

...n/java/seedu/address/model/student/Attendance.java
75.00% <ø> (ø)

...c/main/java/seedu/address/model/student/Email.java
80.00% <ø> (ø)

...rc/main/java/seedu/address/model/student/Name.java
80.00% <ø> (ø)

...ava/seedu/address/model/student/Participation.java
75.00% <ø> (ø)

... and 26 more

Continue to review full report at Codecov.

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


APPS/pkey etc.: fix case where infile and outfile are the same

When using e.g. openssl pkey -in test.priv.pem -aes128 -out test.priv.pem
the file was reset to zero length before reading its contents.
This was recently reported on the openssl-users mailing list.

A test shouldn't be a problem...

A test shouldn't be a problem...

In principle I agree.
Yet is there already any test/recipes/*.t specifically for the pkey app?
For some reason, I couldn't find it -
does this mean that so far there have been no such tests, as exist for most other apps?
So where do you suggest adding such a test?
Should we go for a new one, like 15-test_pkey.t ?

This has been a pretty long-lasting bug.
It was already incurred when  the pkey utility was added in commit 3e84b6e15fe6a580177b9e242c66e1d1ab6c2164, in March 2006.

I wouldn't be surprised if similar ones are in other apps.

The one jumps on my mind is pkcs8

I wouldn't be surprised if similar ones exist in other apps.

The one jumps on my mind is pkcs8

Inevitably, several more apps bear this bug, revealed for most part very quickly by
grep -E "infile|outfile|derfile|reqin" apps/*.c | grep -E "open|load"

Besides pkcs8, also dhparam, dsaparam, ecparam, ocsp, and storeutl are affected.
Which demonstrates again that it is bad practice to copy&paste long chunks of source code,
copying its bugs, tool, and making it more and more tedious to spot and remove them.
Only storeutl, which is much newer, has a different and more complex structure, which allows for recursion,
which made detection and fix a little harder.
I'll add fixes for all of them here.

I am afraid that fixing all the apps at once will make this pretty large and thus riskier for merging to all stable branches.
Also the openssl pkey command is invoked in many test recipes, so it cannot be said it is entirely untested. But yeah, there are no negative tests or tests focused solely on the pkey app. So adding a new recipe for that would be appropriate.

I am afraid that fixing all the apps at once will make this pretty large and thus riskier for merging to all stable branches.

Good point - for sure the fix of storeutl should be separate, already because it came much later.
I'll see which very similar apps can be reasonably combined here,
or shall we go for effort of having 7 separate PRs?

Also the openssl pkey command is invoked in many test recipes, so it cannot be said it is entirely untested.

Yes, that's why I wrote: "direct tests".

But yeah, there are no negative tests or tests focused solely on the pkey app. So adding a new recipe for that would be appropriate.

All right, so will add test/recipes/15-test_pkey.t - but I don't promise full test coverage 😉

I'll see which very similar apps can be reasonably combined here,
or shall we go for the effort of having 7 separate PRs?

Yeah, if the fixes are simple and done in very much similar way, I would combine them.

Gosh, adding the tests was much more work than doing the fixes.

Yeah, if the fixes are simple and done in very much similar way, I would combine them.

So I combined here all fixes except those of storeutl.

Weirdly, GitHub CI / no-shared-ubuntu (pull_request) failed as follows:
30-test_evp_pkey_provided.t           (Wstat: 0 Tests: 2 Failed: 1)
  Failed test:  1
  Parse errors: Tests out of sequence.  Found (1) but expected (2)
                Bad plan.  You planned 1 tests but ran 2.

Let's see if this is a transient issue, I'll rebase, squashing the recent fixups.

Should the docs be updated?
At what point exactly is the output file truncated?
If a user aborts (Ctrl-C, or password mismatch) the process, is the input file already truncated?
And perhaps even warn users about lack of atomicity, one can end up without either the unecrypted input or the desired output (or the converse if the operation is decryption, or some other "in place" transformation).

Should the docs be updated?

Well, I'd say the new behaviour is what one would expect to happen.
Still we can point out that in case the input and output files are the same, the file is updated in situ.

At what point exactly is the output file truncated? If a user aborts (Ctrl-C, or password mismatch) the process, is the input file already truncated?

As can be seen from the sources or tried out, the password is read even before reading the key,
and the output is truncated and written after that.
so if a user decides to abort the password input, nothing happens.

And perhaps even warn users about lack of atomicity, one can end up without either the unecrypted input or the desired output (or the converse if the operation is decryption, or some other "in place" transformation).

While it is generally understood that in case a process that involves writing data is interrupted, the result is undefined,
we can make clear that the user may safely interrupt password input, meaning that the output file is not touched.
This can be an interesting info also for the standard case where input and output files are not the same.

Oh, turns out that Stephen was well aware of the original limitation of the pkey app -
see his documentation in commit 49131a7d942d85bc3a8d649e23ff14f0da18ee4c.
So users should have better RTFM and strictly speaking this is a described feature (rather than a bug).
Yet still this is a needless limitation that is easily overlooked and then leads to silently destroying the key file contents,
so IMHO it is fair to call it a bug.
Same holds for the pkcs8, dhparam, dsaparam, and ecparam,  apps.
Yet for the very similar pkeyparam the limitation was not documented, so there it is clearly a bug.

Updated the docs as briefly exchanged above.
On this occasion fixed a couple of glitches in those texts.

This pull request is ready to merge

Approving for master branch only. IMO this is too invasive and in a sense a feature - this is apparent by the documentation changes needed.

Well, as mentioned,

for pkeyparam the limitation was not documented, so there it is for sure a bug.
for the apps where this needless limitation is documented,
this is easily overlooked and leads to destroying the key file contents.

for pkeyparam the limitation was not documented, so there it is for sure a bug.

Yeah, but the stable branch fix could be just adding the missing documentation.
If you want to appeal to OTC, please feel free so.

Merged to the master branch. Thank you for your contribution.
Keeping this open in case you want to appeal to OTC in regards to the backport. Otherwise please close this.

Merged to the master branch. Thank you for your contribution.
Keeping this open in case you want to appeal to OTC in regards to the backport. Otherwise please close this.

Thanks. I give up on pursuing a potential backport.

I think it is wrong, and this PR should've been backported. OTC made a bad call.

There was no OTC decision. If you want OTC to decide, please open a new issue describing what you want to be decided.


add resolve solution for Order Ascending Strings

Description:
Given a string, you need to write a method that order every letter in this string in ascending order.
Also, you should validate that the given string is not empty or null. If so, the method should return:
"Invalid String!";
Examples
new Ordering().orderWord("hello world") => " dehllloorw"
new Ordering().orderWord("bobby") => "bbboy"
new Ordering().orderWord("") => "Invalid String!"
Notes
• the given string can be lowercase and uppercase.
• the string could include spaces or other special characters like '# ! or ,'

Done with all coments resolve @jresendiz27

And verify the tests are run in travis.ci I dont show message for tests passed :/

Ok ok I check!


added default action handlers

fixes https://github.com/infinitered/reduxsauce/issues/30

Thanks! I’ll take a closer look in a moment. 👏

I've updated my commit. Fixed few small bugs.

@skellock I'm sorry, did you have a chance to look at that PR? If you need something more from me on that, please let me know.

Nope it looks good. Sorry for the delay. Time has been tight lately.

shipping now in 0.7. tyvm!


Doc support for version 11.3.1 of highlight.js

Also link to highlight.js in readme

Attempt to resolve #88


Define a common interface for IRDIS and IFS data analysis

The interface should be identical to be able to follow the exact same procedure to pre-process and reduce the IRDIS and IFS data sets.

Done with 8f0e398a8b6494d13760d774ab01f07f9609a406.
The configuration of the reduction can be configured by changing the values in the reduction_config dictionary.


Customize Standard Resource Container Properties

Allow customization of standard resource containers (CTBs, etc) using input files.

resolved in commit e13f8cdeff8d84fa219fe04817823c3a2d99ba45


fix: naming and storybook type issue workaround

#222

Implement decisison to not prefix boolean props.
Propose workaround for storybook issue

#222

is it possible the ticket is not properly linked?

this is empty? intentionally?


Seeing bugs with angular property bindings

@triniwiz
I played around a little with the new plugin. Unfortunately, I cannot get things working. I prepared a demo showing some of the issues I run into. You can find it here:
https://github.com/cjohn001/mlkit-test
The demo shows 2 different issues. I assume both has to do with the angular bindings which seem to not work as expected.

Issue on IOS: The app runs and shows video, but it does not detect any barcodes.  When debugging, I see that _onScanCallback(result) is called and holds a barcode. I see that the source of the problem is given in the following line:

https://github.com/NativeScript/mlkit/blob/e308a2d89c1dda6a3bfb304e9c3e4d6bc74ccf1e/packages/mlkit-core/index.ios.ts#L140
At that point this.detectionType and this.onDetection both have a value of undefined. I set both in the Angular bindings as you can see in the demo. I have not been able to figure out what is wrong with the attribute bindings, would be great if you could have a look into it.

Issue on Android: When I try to run the demo app on Android, it directly crashes. Seems like the BarcodeFormat has an issue on Android.

System.err: Calling js method onCreateView failed
System.err: Error: com.tns.system.classes.loading.LookedUpClassNotFound: Class "io.github.triniwiz.fancycamera.barcodescanning.BarcodeScanner.BarcodeFormat" not found.
System.err:
System.err: StackTrace:
System.err: [barcodeFormats:setNative](file: src/webpack:/mlkit-test/node_modules/@nativescript/mlkit-core/index.android.js:233:0)
System.err:     at applyAllNativeSetters(file: src/webpack:/mlkit-test/node_modules/@nativescript/core/ui/core/properties/index.js:1136:0)
System.err:     at initNativeView(file: src/webpack:/mlkit-test/node_modules/@nativescript/core/ui/core/properties/index.js:1079:0)
System.err:     at onResumeNativeUpdates(file: src/webpack:/mlkit-test/node_modules/@nativescript/core/ui/core/view-base/index.js:737:22)
System.err:     at _resumeNativeUpdates(file: src/webpack:/mlkit-test/node_modules/@nativescript/core/ui/core/view-base/index.js:280:0)
System.err:     at onLoaded(file: src/webpack:/mlkit-test/node_modules/@nativescript/core/ui/core/view-base/index.js:235:0)
System.err:     at onLoaded(file: src/webpack:/mlkit-test/node_modules/@nativescript/core/ui/core/view/index.android.js:309:0)
System.err:     at onLoaded(file: src/webpack:/mlkit-test/node_modules/@nativescript/mlkit-core/index.android.js:335:0)
System.err:     at (file: src/webpack:/mlkit-test/node_modules/@nativescript/core/ui/core/view-base/index.js:318:0)

The demo only has the base template

@triniwiz: I am sorry, I do not understand. What do you mean with base template? I were speaking about the demo I wrote under the given link
https://github.com/cjohn001/mlkit-test

There is new code to check in the repo

@triniwiz : I am sorry, I had forgotten to check in everything. Now its there.
You are meaning I should not use
"@nativescript/mlkit-barcode-scanning": "^1.0.0-alpha.0",
"@nativescript/mlkit-core": "^1.0.0-alpha.0",
as there is new code in this repository? Hence, I should build the plugins myself to get the latest?


Add "fal" to values for "pack" to support font awesome light icons

Hi first contribution to a project, very minor one at that!
Not 100% sure it is worth it, but it would have helped me from a documentation point of view.

testing pull requests.


Add tests for trace proto validation

Add some unit tests to check error responses for malformed protos

Closed in favor of https://github.com/googleinterns/cloud-operations-api-mock/pull/9


Create pseudoelement for blank pages

emd2H4_Q1.html
Add blank page as text

I've styles this should be in next build, however would you like blank pages to loo different from page breaks? This is what it looks like  now:

I like it -- I think that both blank pages and page breaks being lighter than the main text is very nice.

Looks beautiful! Would it be possible to display the value of @n instead of the phrase "Phrase break"?

What is @n?

My screen reader does read the text from the pseudoelement, so there's our answer to the accessibility question for it :)

@pszpak To clarify what's needed, the div.page[data-empty="true"] will most likely have a child div which has a data-n attribute, which is what contains the signature or folio number that needs to be displayed. Example from your screenshot above:

Change "Page break" to "Page beginning".

Changed to "Page beginning" in rev. 13612

I believe this is all done, closing issue.


All GitHub builds failing since latest release

Currently when running any action, the following error occurs:

It's due to this commit.
Thank you

I'm facing the same issue. Please check!


No tun on sync button after update

VSCode Version: 1.43.2
OS Version: WIN10 PRO x64

After installing the newest version of VSCODE there is no "Turn on Settings Sync" button in settings for the new settings sync feature.

/question
This feature is only in Insiders builds (1.44) at this moment.
You can install it from https://code.visualstudio.com/insiders/


add --xdsl-anonymize option mutually exclusive with --mlir-anonymize

Sometimes the names carry semantic meaning, so it's useful to check that they are preserved.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 97.72%. Comparing base (79f61fd) to head (dd776ca).

Files
Patch %
Lines

filecheckize/main.py
88.88%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              main       #2      +/-   ##
===========================================
- Coverage   100.00%   97.72%   -2.28%
===========================================
  Files            1        1
  Lines           38       44       +6
  Branches         7        9       +2
===========================================
+ Hits            38       43       +5
- Partials         0        1       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Overhauled and improved profiling

The aim of this PR is to improve the profiling capabilities of Devito (also fixing issue #65 )

Profiling information are now stored in a numpy array in the Propagator instead of being printed to stdout
Profiles the kernel, the time loop stencils (before and after space loops) and the space loops.
The logic for executing Propagator.cfunction has been moved down to the propagator. This should also make easier to execute it when the user uses the low level Propagator interface directly.

This needs to be tested on MIC.

Works on MIC

How did you test this?
On Mon, 8 Aug 2016 at 15:46 Vincenzo Pandolfo<EMAIL_ADDRESS>wrote:

Works on MIC
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/opesci/devito/pull/74#issuecomment-238260431, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFJRA67-ncl_SE216Bxf6tsE8cG4gxb7ks5qd0E8gaJpZM4JePN5
.

@pvelesko did


this.el.querySelector(...) is null

Hi,
I am executing
send_update(LiveSelect.Component, id: "workshop-tags", value: list_of_tags())

inside an update/2 callback of a live component, the live select updates correctly, but I got a
Uncaught TypeError: this.el.querySelector(...) is null

message, and the offending line is https://github.com/maxmarcon/live_select/blob/30c9fab961e554c416aa5bbcf7bd7554f78d5cc1/assets/js/live_select.js#L82
This breaks my javascript script.
Am I doing something wrong?
Any idea how to solve?
Thank you

Hi!
what's the reason for updating value and not options ?

I want to programmatically set the value of live_select (it's an edit form).
I followed this https://hexdocs.pm/live_select/LiveSelect.html#module-controlling-the-selection-programmatically
Please note that the value is set correctly and I see the existing tags pre-selected.

I see. Can you show me more code please? Like how you're rendering the component, and the function that sends the update. Thanks

Ah!
In the live component that used live_select I had an hook which for whatever reason used async mount instead of mount (it had some await inside).
I converted the parent live component hook mount to mount and the awaits inside to .then and it now works like a charm!
Probably the parent live component wasn't really mounted when the live_select javascript did run, so this.el was undefined.
Sorry for the noise!
Thank you very much for this library!

No worries! Glad you figured it out and thank you for using the library. Always ready to help.


Add modal dialogue when waitlist-ing a process

The ability to set the waitlist attribute was added in c1b45f7cfa7e719d813ef4bb6d18762131bba132.
However, it would be much nicer to have a modal dialogue that would allow users to add a comment on why a process is being waitlisted. It could be quite simple, just a textbox with the title "Waitlist this process", label "Comment", and cancel / confirm buttons.
On clicking confirm, there should be two API calls. One to:
/add-attribute/<process id>?attr=waitlist&value=1
And the other to:
/add-attribute/<process id>?attr=waitlist_comment&value=<comment text>

Did I get this correct, supposing that this modal should be opened by clicking Waitlist button on the process_detail page (somewhere in the markWaitlist function?

Yes, exactly. I just thought a model window would be easiest, but however you want to do it, we need a way to be able to insert an (optional) comment when clicking the waitlist button, as it is easy to forget the reason why we waitlisted something afterwards.

Some aditional fixes in PR #73.


Fixing GodSVG's messy infrastructure.

Just writing out what's on my mind right now, as I try to figure out a good infrastructure that would allow to make SVG elements and attributes configurable from multiple places.
This post might be edited a few times as I rethink things.
New approach
SVG singleton
The core idea is that the SVG singleton will hold data, which will be the defacto way to manipulate anything about the SVG. This data will be independently able to be manipulated by:

The CodeEdit, which displays the code of the SVG. In the future, I want to make it editable. Manipulations will only happen when the SVG inside it is valid.
The inspector, which shows all the editable properties. Manipulations will happen when tags are moved, deleted, added, or attributes are changed.
The draggable control handles. Manipulations will happen when they are being dragged.

Data structures
As before, there will be:

SVGData, holds width, height, and an array of SVGTags. (I know this is just a tag, but right now I think this makes more sense)
SVGTag, with a title and a Dictionary of Strings to SVGAttributes. There are various types of SVGTags, each having their own titles and attributes.
SVGAttribute, with a type (e.g. color, int, flag, enum) and a value.

This is a rough simplification. So, these RefCounteds should also emit specific signals when they are changed.

SVGAttribute would have a value_changed() attribute. SVGTag would be subscribed to this and emit attribute_changed() itself.
SVGTag will have just the attribute_changed() signal for now. SVGData will be subscribed to this signal and emit attribute_changed() itself.
SVGData will have an attribute_changed() signal as explained. It will also have resized(), tag_deleted(), tag_added(), tag_moved(), and tags_changed_unknown() signals.

TODO: What about individual path commands in a path definition?
TODO: How to signal information about which specific things updated?
Display
The "Display" TextureRect will be subscribed to all SVGData signals with its queue_svg_update() function, always updating the SVG when something changes.
CodeEdit
Talking: When I make it editable, I plan to make it so after every edit, the text is checked for being a valid svg string. If it is, then gather from it all the values needed to make up a width, a height (I'll address viewBox later), and an array of SVGTags, and assign all that to the existing SVGData. This will emit the tags_changed_unknown() and resized() signals.
Listening: It will listen to all SVGData signals and use tags_to_string() to translate the data. But if it's currently being edited, it won't change its text. Unlike other ways to edit the SVG, we don't want to change the text to a standard generated one while the user is editing it.
Draggable handles
They will be handled in a TextureRect overlaid on top of the SVG.
Talking: Each handle will hold a reference to its the relevant SVGAttributes in the svg data: one for its x-pos, and one for its y-pos. While a handle is being dragged, it will use its X and Y position to update these two attributes.
Listening: All SVGData signals except for tag_moved() will be listened to. They will trigger a full rebuild of the handles, except for attribute_changed() which will search for the appropriate handle and update its x or y position.
TODO: What about individual path commands in a path definition?
Inspector
Talking: Each tag editor will hold a reference to its relevant SVGTag in the SVGData, and likewise each of the fields will hold a reference to its relevant SVGAttribute. The signals that will be emitted on each possible action are pretty self-explanatory.
Listening: All SVGData signals except for resized() will be listened to. tags_changed_unknown() will trigger a full rebuild, while the others will only change the relevant parts.
TODO: How to find which tag editor or attribute editor corresponds to a certain tag? Iterating and checking?
TODO: How to handle selections? Eventually, I want to have it so selecting a tag or a handle or such is reflected in all three places (e.g. clicking on a tag highlights its handles, and also highlights it subtly in the CodeEdit)

TODO: How to handle selections? Eventually, I want to have it so selecting a tag or a handle or such is reflected in all three places (e.g. clicking on a tag highlights its handles, and also highlights it subtly in the CodeEdit)

This should be a selected_tag that would trigger a selected_tag_changed signal on being set and all place that needs to adapt should just listen to it.
Something like
var selected_tag : SVGTag :
     set(value) :
          selected_tag_changed.emit(value)
          selected_tag = value

TODO: How to find which tag editor or attribute editor corresponds to a certain tag? Iterating and checking?

maybe they should be listening directly to things that are emitted by the SVGTag instead of the whole SVG?

Alrighty, having such a signal in the SVG singleton really seems like the best option.


limit payload size with lua script

Hi,
I tried to write to following script in order to limit request size to 2MB, but when using body():length() it returns 413 (Payload too large) when reaching 1MB (and doesn't continue with the function).
function envoy_on_request(request_handle)
  local max_bytes = 2 * 1024 * 1024
  local body_size = request_handle:body():length()
  if body_size > max_bytes then
    request_handle:respond(
      {[":status"] = "413"})
  end
end

Does anyone knows how to solve that? Thanks.

I would just let Envoy do this limiting for you, either via the buffer filter or the built-in buffering limits. See https://www.envoyproxy.io/docs/envoy/latest/faq/configuration/flow_control for more information.


Changes on CustomLogConsumer to accept LogFilter maps

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (276d3ec) 69.89% compared to head (54993a5) 69.71%.
Report is 2 commits behind head on main.

Files
Patch %
Lines

cpp_utils/src/cpp/logging/CustomStdLogConsumer.cpp
0.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
- Coverage   69.89%   69.71%   -0.19%
==========================================
  Files          60       60
  Lines        1455     1456       +1
  Branches      388      389       +1
==========================================
- Hits         1017     1015       -2
- Misses        213      214       +1
- Partials      225      227       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Remove spurious executable permissions

Pull requests to master will not be accepted until v1.0 is merged in the coming weeks. After it supersedes the current version of Polonium, I will merge this PR.

@zeroxoneafour was this accidentally closed?

Closed as a consequence of v1.0 being merged. Its not like this PR is revolutionary or anything, if this is an actual issue I'll just chown the problematic files.


Proxy feedback

Feedback for proxy.
Expects the proxy_score.csv file to be tab delimited.

I've tested this with and without feedback  on the 20150301 data.

Great idea to separate Feedback from ProxyPreLDA, it looks cleaner. I will implement a similar approach next sprint.
The only thing about this implementation is that looks a bit different than the ones for DNS and Flow where we create an RDD[Feedback] and then we call the method .toDF() while this one uses sqlContext.createDataFrame on a RDD[Row].
I like both but I think we need to choose one to keep consistency.

about the toDF versus createDataFrame call:
toDF was getting the schema for the dataframe from the fields of the case class....  and I want the schema to come from the column identifiers in ProxySchema , not the case class fields and keeping the fields of the case class in sync with ProxySchema kind of defeats the purpose of having ProxySchema
so.... createDataFrame allows us to pass in the schema explicitly whereas toDF doesn't

I get it now, it got confused with the Feedback case class and the .select. I can see what you are doing. It's great.

actually, the way I found the whole issue was that the case class field names didn't match the schema in the table, so I was getting run time SQL errors and once I figured out where they were coming from I was like "oh man, I guess the case class isn't the way to go"
which is sad, since normally case classes are among my favorite things in scala

LGTM


[DRAFT] Validate build pipeline behavior for version incrementing

This draft uses upstream and existing unified pipelines to validate behavior. Note that it is skipping publishing steps in the ci.yml

@danieljurek, @KarishmaGhiya, Is this PR still relevant?

This pipeline is no longer relevant. Closing. Thanks!


Correction to 'gridder.cut' function

With this correction we can give any pair of opposite vertices of the area to cut, instead of giving the bottom-left vertex first and the upper-right in second place, as before.
Checklist:

[ ] Make tests for new code
[ ] Create/update docstrings
[ ] Include relevant equations and citations in docstrings
[x] Code follows PEP8 style conventions
[ ] Code and docs have been spellchecked
[ ] Include new dependencies in docs, requirements.txt, README, and .travis.yml
[ ] Documentation builds properly
[ ] All tests pass
[ ] Can be merged
[ ] Changelog entry (leave for last)
[ ] First-time contributor? Add yourself to doc/contributors.rst (leave for last)

@santis19 thank you for this fix! This code is quite old (~4 years) and I didn't do nearly as much defensive programming back then. It is really helpful to have more eyes looking at the code.
I like that you caught a potential sources of bugs. I think it would be better to raise an error and warn the user that the input was wrong. If we silently fix the input, this could hide a bug the code calling this function. It is better to fail fast and often.
A way to this is using the assert  statement. See some of the code in the gravmag.tesseroid module for an example of using assert to check function input.
It would also be nice to have a unit test that checks if the code really raises the error when it should. Have a look at the test/test_gravmag_tesseroid.py   file for an example of testing asserts . I can give you a have with that later if you'd like.

Thank you for your response Leo!
If you think that raising an error is better I'd do it, but in this case I think that is better to fix the input because the old situation is included in the new one. What I mean is that if any code intentionally calls this function putting the lower left vertix first and then the upper right, it will continue working as it is.
But, in every day work when we call this function we care about cutting an area, without concerning about the order of the vertices of the rectangle. So that's why I think that it's better to fix the input than raising an error.
Greetings!

@santis19 I thought a lot about this. I agree that being flexible with the inputs can be nice when you're using the function to run a script. However, this functions is used in the internals of some gravmag modules (all that have to do with windowing procedures, like Euler deconvolution). So allowing wrong input in gridder.cut can hide bugs in things like fatiando.gravmag.euler. And these bugs will be hard to find.
I still think that the best thing to do is to fail with an error for wrong input. If you're writing a script and get this error, than you know right away and can easily fix it.

Yes, I think you're right Leo!

I'm closing this Pull Request because it's been inactive for a long time. The branch is probably very out of sync with master.
If you want to continue working of this, please sync your fork, update your branch, and re-submit the PR.  Let me know if you need any help doing this!
Sorry for the hassle! This is not a critique of your work, just a way to keep things fresh in the project. I hope you'll consider re-submitting.
This is a message template.