Small safety check added

Nothing major but I lost some time looking for this when I tried to make the head collision example work in my own project. :-)
On a side note, thanks for all the great examples! This toolkit is a great initiative :)

Sure, happy to adapt :)

You can just force push your changes to an existing pull request, no need to close it to make amends :)
Also, when you raise the next pull request, ensure your commit message meets the contribution guidelines otherwise gitcop will complain (it didnt before because the service was down, but its back up now).
so this would be something like:
fix(Headset): show warning if fade script is not present

I'm learning all kinds of fun stuff :)

the joys of git! it's such a minefield :/
if you wanna learn more git, then try this git based game called githug
https://github.com/Gazler/githug


Iterate upon assessment phase of service blueprint

User Story
As a product owner/designer/developer,
I want to know how the screening & assessments will take place between Network BC & ISED
so that I can create a system that meets business requirements
Description
The assessment phase of the service blueprint needs to be iterated upon based upon discussions between NWBC & ISED.
Notes

Elliott's poorly written notes are found in this table.

Also add information from data team meeting.


The feature 'EnableOIDCIssuerPreview' could not be found.

Describe the bug
Trying to register the EnableOIDCIssuerPreview on a GovCloud Cluster fails.
as far as I understand Workload Identity is in GA already which raises two concerns :

why do we still need to register OIDCIssuerPreview plugin
whether this is supported on AuzreUsGovernment
Steps To Reproduce

follow this tutorial : Azure Docs
login to AzureUsGovernment based subscription
ensure your CLI = 2.41.0 (latest)
run :

 az feature register --namespace "Microsoft.ContainerService" --name "EnableOIDCIssuerPreview"

you will get : Message: The feature 'EnableOIDCIssuerPreview' could not be found.
Expected behavior

enabling EnableWorkloadIdentityPreview should not be in Preview as WIF is in GA
enabling OIDCIssuer should not be considered as Preview Feature once WIF is in GA
Registration should succeed on all clouds.
Logs

Environment

USGovCloud Subscription
Az CLI : 2.41.0
kubectl version 1.24
AKS Cluster version 1.24.6
AKS Network : Kubenet

Additional context

@tshaiman it might be better to open this issue on https://github.com/azure/aks

moving to AKS Team


Add new toolbar in the vtk extension

Hello Guys,
I have implemented some additional commands that perform 3D viewing functionalities using VTK, this implementation is inside VTK extension now so all the functions are shown when we click MPR button.
Now we want to make a separate toolbar/extension for these 3D functionalities(I am not sure which is the best approach). I tried the way of copying vtk extension but it's not working until now and many things need to be connected so:
Which is the best approach to achieve this separation? and what are the steps?

Hi! Have you eventually succeeded in doing such modifications? If so, which steps did you follow?

Hello @MAdelElbayumi ! Did you manage to achieve this separation?
Kind regards

Thank you for your past contributions and for raising this issue related to legacy versions of our OHIF Viewer. Your time and effort have been invaluable in helping us improve the product.
We wanted to update you that our development focus has shifted to OHIF Viewer v3, which is now available for testing on viewer-dev.ohif.org. This site is deployed from our master branch and incorporates new features, optimizations, and bug fixes. If you're still using viewer.ohif.org, it's worth noting that this is deployed from our release branch and may not have the latest updates. You can read more about our branching strategies here.
Given the move to Version 3, we are closing older issues that pertain to Versions 1 and 2, as we are no longer providing support for those versions. If you find that the issue you've raised is still relevant in Version 3, we encourage you to reopen this issue or create a new one for proper triage.
Thank you for your understanding and continued support.


email as username is not valid username

Describe the bug
Instagram user with email as username, throws error:
Errors or warnings occurred:
Warning: Parameter<EMAIL_ADDRESS>for --login is not a valid username.
To Reproduce
Steps to reproduce the behavior:

use instaloader command line with<EMAIL_ADDRESS>Warning: Parameter<EMAIL_ADDRESS>for --login is not a valid username.

Expected behavior

Should pick up cookie and proceed with download.

Error messages and tracebacks
If applicable, add error messages and tracebacks to help explain your problem.
Instaloader version
4.7.1

As already described in #624, the login with your e-mail-address is not supported. Please use your nickname.

worked fine.
May be it's better to add this in error description.


[GENERAL] Website can have little bit of animations and book now section should be much better aligned and in dark mode some thing can be better

Topic

dark mode section and little bit of animations should be added

Details

When dark mode is turned on some sections are not properly optimised as well as logo are not syncing well with dark mode and animations will make the the website more smoother and book now section can be better

Assign it tom me i can work on all of it


enchantmentdescriptions support

pls add support for enchantmentdescriptions 1.19

Should Now Support Enchantment Descriptions (When v1.19.5 is released)! You're welcome to build the file for yourself in the time being, but I am still looking for possible bugs in the mod, so be cautious!


CSS nesting doesn't work on Safari

MacOS: 14.1.1

My solution:

Install  postcss-nested plugin
Add postcss-nested to postcss config

/** @type {import('postcss-load-config').Config} */
const config = {
  plugins: {
    tailwindcss: {},
    'postcss-nested': {}
  },
}

export default config

Thank you for this find!


Fakespot not working

I added Fakespot to Firefox but the icon did not appear in the browser bar.

Did you accept the privacy notice? And did you go to a product page of one of the supported sites?

It's also not working for me in Firefox.
I accepted the privacy notice and I am browsing product pages.

Can you give me an example url?

I can't work on this until I have more information.

https://www.amazon.com/gp/product/B003C1P85G

Do you see this icon in the navigation box?

Hmm... Yes I do! I guess I assumed it would work the way it does in Chrome. Is there an option to inject the ratings into the product / search pages like how it behaves in Chrome? I've found that to be really useful particularly on the search pages when comparing products.

Currently no.  I started out making this for myself and for privacy reasons I didn't want it querying every applicable page that I visited automatically.  Also, since I'm not affiliated with fakespot I'm not sure how they would take me scraping their results instead of doing a redirect, which is what I would need to do since they don't have a public api.

Ah, I thought you were affiliated. That makes sense and thank you for making this!

I suggest you make it clear in the information about the firefox add-in that it needs to be accessed via the icon inside the address box.  I had it installed for a month but never used it because I couldnt find how to make it work!  I assumed it would add an icon into the toolbar and only understood how to make it work when I came here.  It would be great if it could display the fake-spot rating by as a tip by just hovering the mouse over the icon, rather than needing to open an additional page...
thank you for making this in any case.


Nexus: Add the ability to create a list of extra ConfigMaps

For folks who are using the nexus-kubernetes-openshift plugin, it would be useful to be able to specify these configmaps as part of this chart.

I've got this working already, just interested in whether this would be something the team would be willing to add/support.

I'd definitely be interested. Feel free to open a PR with the changes and I'll take a look.

Why including support of ConfigMap directly in the Chart?
Why not simply relying on Kutomize to deploy these ConfigMaps?


Remove emptydir /tmp volume because it is unused

In https://github.com/cert-manager/approver-policy/pull/298 @inteon stopped using /tmp for the dynamically generated webhook key and certificate and deprecated the command line flag which had previously allowed the user to override the directory.
That was released in v0.10.0 and now in v0.13.0 we can remove the flag altogether and remove the emptydir volume
(arguably the emptyDir should have been removed in #298)
Removing this emptyDir volume will make life easier for platform engineers who install approver-policy because they will no longer have to consider it when choosing ephemeral storage requests and limits.
And they will no longer have to consider the approver-policy empty dir when draining a node using kubectl drain.

/retest

/retest

/approve
/lgtm


Any way to get this working with typescript@next?

Is there any way to get this to target typescript@next rather than the version bundled with the task itself?
This may be simple, but I'm pretty new to grunt and node, so you please excuse my ignorance.
Thanks for any help,
Tim

Is there any way to get this to target typescript@next rather than the version bundled with the task itself?

Lots of ways. Checkout https://github.com/TypeStrong/grunt-ts#compiler feel free to ask further if documentation is unclear :rose:


SciView on forum.image.sc toolbar

Before the final release, just for @tpietzsch <3

Is there a logo? Once there is a logo, we can add it any time. 😄

There is (any good project timeline goes: name, logo, then write code, right?)
https://github.com/scenerygraphics/artwork

Oh, right! Do you want me to add it to the partners now?

Yes please!!

I added both Scenery and SciView, with the same icon, and adapted blurbs from their online presence. Let me know if it needs correction.


value imagePullSecrets is not being passed to Installation custom resource

Expected Behavior
I have below in my values.yaml file.
installation:
  imagePullSecrets:
    - name: secretname
  registry: private-registry-url/

imagePullSecrets for Installation custom resource should be populated with the passed value
e.g.

imagePullSecrets:
  - name: secretname

Current Behavior

imagePullSecrets for Installation custom resource is not updated with the passed value and remains empty list
e.g.

imagePullSecrets: []

Steps to Reproduce

pass helm values for installation as below:

installation:
  imagePullSecrets:
    - name: secretname
  registry: private-registry-url/

Context
Trying to use private registry with already existing secret. It would be super helpful if the documentation have examples of using these helm values. values.yaml on the repo does not have registry spec under installation. but passing it with values.yaml allows installation to use that registry.
Environment

tigera-operator helm chart version 3.26.0 being deployed as terraform resource "helm_release"
Orchestrator version kubernetes EKS : v1.23

@djosborne just made some changes here that went into v3.26.0 I believe.

@caseydavenport we saw this PR from @ozdanborne following which we then tried to use the existing Kubernetes secret (with v3.26.0) for imagePullSecrets under Installation spec. not sure where the issue is, may be because the root level imagePullSecrets and Installation one are being combined here

thanks @ozdanborne for checking. Having capability of using existing secret was much needed and we would like to use it sooner(with the patch release may be ?) to avoid any manual steps post helm. appreciate all your help 👍

you can try a hash release of calico 3.27 to check whether the fix would work for you

Hi all, @tomastigera, @caseydavenport, @ozdanborne
Issue has been re-raised here: https://github.com/projectcalico/calico/issues/7738
Just inputting here because my organisation is also looking for this along with the other requesters as we're doing a big auth-everywhere thing.
Do you know a very rough timeline for v3.27.0?
And otherwise could this possibly be considered for v3.26.2?
Thank you.

https://github.com/projectcalico/calico/pull/7657


Merge the providers into a single provider

Part of #4863
Go favors composition, therefore there's no reason the providers need to be separated. That is, any implementations can simply combine multiple implementations through composition (although we deem it unlikely that a single provider would be sharded in the first place).
This is part of the layering of clients in the issue linked above, and gives us a canonical place to put methods that accept OSM object input, but need to query k8s.
Lastly, we embed the provider into the MeshCatalog, and remove its duplicative methods. If we ever need a mutli-provider, we can create one that fits the same interface.
Bonus points that this should make navigating code simpler since we're removing a layer of indirection

Codecov Report

Merging #4964 (ea47c9d) into main (548b198) will decrease coverage by 0.26%.
The diff coverage is 21.15%.

@@            Coverage Diff             @@
##             main    #4964      +/-   ##
==========================================
- Coverage   68.76%   68.50%   -0.27%
==========================================
  Files         220      219       -1
  Lines       16048    16032      -16
==========================================
- Hits        11036    10982      -54
- Misses       4960     4998      +38
  Partials       52       52

Flag
Coverage Δ

unittests
68.50% <21.15%> (-0.27%)
:arrow_down:

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

Impacted Files
Coverage Δ

cmd/osm-controller/osm-controller.go
16.80% <0.00%> (+0.27%)
:arrow_up:

pkg/catalog/mock_catalog_generated.go
0.00% <0.00%> (ø)

pkg/endpoint/types.go
100.00% <ø> (ø)

pkg/providers/kube/client.go
74.28% <ø> (ø)

pkg/service/types.go
58.82% <ø> (ø)

pkg/catalog/catalog.go
90.90% <100.00%> (-0.40%)
:arrow_down:

pkg/catalog/endpoint.go
91.30% <100.00%> (-0.37%)
:arrow_down:

pkg/catalog/ingress.go
96.26% <100.00%> (ø)

pkg/catalog/outbound_traffic_policies.go
92.74% <100.00%> (ø)

pkg/messaging/workqueue.go
89.28% <0.00%> (-10.72%)
:arrow_down:

... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Unrlated lint errors, going to merge


Updated README

I have updated https://github.com/jsuarezruiz/ArtNews (use v2), small change in the README.

Thank you :)


v4.0.0 - Portal EmailSettings - Hostname not resolved

Community Note

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

Module Version

4.0.0

Affected Resource(s)

ArcGISPortalSettings

Configuration Files

Just add an emailsettings property:
        "EmailSettings": {
            "SMTPServerAddress": "xxxxxx",
            "From"<EMAIL_ADDRESS>            "Label": "Portal TEST",
            "AuthenticationRequired": false,
            "EncryptionMethod": "SSL"
        }

Expected Behavior
The EmailSettings are set in portal like it worked with the DSC Module 3.3.2.
Actual Behavior
DSC Error in ArcGISPortalSettings --> the hostname ist missing!
09.09.2022 09:36:46: Cannot bind parameter 'Uri'. Cannot convert value "https://:7443/arcgis/portaladmin/system/emailSettings/update" to type "System.Uri". Error: "Invalid URI: The hostname could not be parsed."
09.09.2022 09:36:46: PowerShell DSC resource ArcGIS_PortalSettings  failed to execute Set-TargetResource functionality with error message: Request to https://:7443/arcgis/portaladmin/system/emailSettings/update failed. Response returned NULL
09.09.2022 09:36:46: The SendConfigurationApply function did not succeed.

Steps to Reproduce
Just add some email settings for portal an run them with DSC 4.0.0
Important Factoids

Running in Azure (westeurope)
References

@scma-esrich fyi

Without EMailSettings it works in DSC 4.0.0

Hi @lorenzmeyer77,
Thanks for the submission. This will be addressed in the next release, in the meantime, the following workaround can be used to allow the email settings to be applied successfully.
Update:
https://github.com/Esri/arcgis-powershell-dsc/blob/4a33aabf965e2098699bddc499ce1d05003e992d/Modules/ArcGIS/DSCResources/ArcGIS_PortalSettings/ArcGIS_PortalSettings.psm1#L340
To:
Update-PortalEmailSettings -PortalHostName $PortalFQDN -SMTPServerAddress $EmailSettingsSMTPServerAddress -From $EmailSettingsFrom -Label $EmailSettingsLabel -AuthenticationRequired $EmailSettingsAuthenticationRequired -Credential $EmailSettingsCredential -SMTPPort $EmailSettingsSMTPPort -EncryptionMethod $EmailSettingsEncryptionMethod -Token $token.token -Referer $Referer -Verbose

Hello @lorenzmeyer77 and @scma-esrich,
This issue has been addressed in v4.0.1.
Thanks,
Cameron K.


Error process.env['SSH_AUTH_SOCK'] undefined

would it should check process.env['SSH_AUTH_SOCK'] before assigning to sshAgentSock?
diff --git a/src/authResolver.ts b/src/authResolver.ts
index 0ac2388..d330f22 100644
--- a/src/authResolver.ts
+++ b/src/authResolver.ts
@@ -96,7 +96,7 @@ export class RemoteSSHResolver implements vscode.RemoteAuthorityResolver, vscode
                 const sshUser = sshHostConfig['User'] || sshDest.user || '';
                 const sshPort = sshHostConfig['Port'] ? parseInt(sshHostConfig['Port'], 10) : 22;

-                this.sshAgentSock = isWindows ? '\\\\.\\pipe\\openssh-ssh-agent' : (sshHostConfig['IdentityAgent'] || process.env['SSH_AUTH_SOCK']);
+                this.sshAgentSock = isWindows ? '\\\\.\\pipe\\openssh-ssh-agent' : (sshHostConfig['IdentityAgent'] || process.env['SSH_AUTH_SOCK']?(sshHostConfig['IdentityAgent'] || process.env['SSH_AUTH_SOCK']):undefined);
                 this.sshAgentSock = this.sshAgentSock ? untildify(this.sshAgentSock) : undefined;
                 const agentForward = enableAgentForwarding && (sshHostConfig['ForwardAgent'] || 'no').toLowerCase() === 'yes';
                 const agent = agentForward && this.sshAgentSock ? new ssh2.OpenSSHAgent(this.sshAgentSock) : undefined;

@oisanjaya  isn't it the same in the end? if process.env['SSH_AUTH_SOCK'] will return undefined if not present

Looking at this again, in the next line it is already doing the check you propose this.sshAgentSock = this.sshAgentSock ? untildify(this.sshAgentSock) : undefined; :thinking:

i got exception before reaching that line, it throws exception when querying process.env['SSH_AUTH_SOCK']
at this.sshAgentSock = isWindows ? '\\\\.\\pipe\\openssh-ssh-agent' : (sshHostConfig['IdentityAgent'] || process.env['SSH_AUTH_SOCK']);


[Barcode_Scanner] ITF bad data

ITF of 26 numbers yields wrong data.
For example this:

Yields random data of random length like this:
3330343537383233303435373832333034353738
Or this:
1216171160010131
Or this:
1213141516171812131415161718121314151617
Sometimes it can reproduce the same wrong data, so it's not like it's completely random, but incorrect either way.

@noga-dev : send a PR with your fix

@noga-dev : send a PR with your fix

There's no simple fix. Issue is 2-fold:
First is ml model requiring more training.
Second is instability of ITF format.
The first can be addressed, but not the second.

Hey man !! Can you assign me this task :)


weird scrolling upon clicking on item with textAutoLink enabled

I tried to enable textAutoLink with phone and then with all, after clicking on any phone it scrolls to the top of the list and I don't know why
I tried to make the parent RelativeLayout and tried to disable selection mode with no hope.
And also it's worth mentioning that it's hard to click on the phone itself if the message is consists of multiple lines

Strange scrolling MessageList (RecyclerView) after clicking on link in message has gone after I've changed layout_width property from wrap_content to match_parent in every custom message type holder.
Maybe it will help you too.


Fixed https://github.com/javascript-obfuscator/javascript-obfuscator/…

…issues/293

Coverage decreased (-0.05%) to 97.969% when pulling e7587b1632059cc8ff95a26c523c2c730d9b6de3 on rename-globals-in-class into 140c4e8bbb9fc342ae1f4709f39b27fd1254b21d on master.


Feature/about page

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 57.764%

Files with Coverage Reduction
New Missed Lines
%

pathways/models/major.py
5
82.76%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
279

Relevant Lines:
483

💛  - Coveralls


Namelock: Check if the target is namelocked

t-that wasn't the Merge button...

That moment after merge it says Trvis didn't pass, yet it looks like the commit did on the commit history page, and the output Travis spit out isn't making any sense to me...


SubjectConfirmationData Not Found

I got an exception after login and when i were back to client website. They are requires SubjectConfirmationData in response. Do you know how can i include this in response? am I missing some setting?
This issue have been fixed by upgraded package Rsk.IdentityServer4.Saml

Handled externally. Solution was to update nuget package version


Fix dangerous compile warning: Compare signed and unsigned int

    $ gcc --version
    gcc (Debian 12.2.0-14) 12.2.0

    $ cmake -DCMAKE_INSTALL_PREFIX=/usr/ -DCMAKE_BUILD_TYPE=Debug \
	-DBUILD_TESTING=ON -DVENDOR_GTEST=OFF ..
    $ make
    ...
    [ 75%] Building CXX object src/CMakeFiles/runtime.dir/output.cpp.o
    In file included from /usr/include/c++/12/cassert:44,
                     from /home/rongtao/Git/bpftrace/src/log.h:3,
                     from /home/rongtao/Git/bpftrace/src/output.cpp:3:
    /home/rongtao/Git/bpftrace/src/output.cpp: In static member function
    'static std::string bpftrace::TextOutput::hist_index_label(int, int)':
    /home/rongtao/Git/bpftrace/src/output.cpp:53:16: warning: comparison of
    integer expressions of different signedness: 'int' and 'const uint32_t'
    {aka 'const unsigned int'} [-Wsign-compare]
       53 |   assert(index >= n); // Smaller indexes are converted directly.
          |          ~~~~~~^~~~

Closing in favour of #2890. Appreciate the fix though!


echo, printf: UTF-8 sensitivity in arguments

❯ coreutils echo "$(coreutils printf 'A \xFF Z')"
error: invalid UTF-8 was detected in one or more arguments

Usage: coreutils echo [OPTIONS]... [STRING]...

For more information, try '--help'.

❯ /usr/bin/echo "$(coreutils printf 'A \xFF Z')"
A � Z

❯ coreutils printf "$(coreutils printf 'A \xFF Z')\n"
error: invalid UTF-8 was detected in one or more arguments

Usage: coreutils printf FORMATSTRING [ARGUMENT]...
       coreutils printf FORMAT [ARGUMENT]...
       coreutils printf OPTION

For more information, try '--help'.

❯ /usr/bin/printf "$(coreutils printf 'A \xFF Z')\n"
A � Z

I'm not sure if this is practically relevant. You can't print arbitrary binary data with echo/printf anyway, because the arguments are C strings (null terminated).

Fixed in echo by https://github.com/uutils/coreutils/pull/6803
Not yet fixed in printf


Zenoh program option, remove short flag

Description
To avoid conflicts, we remove all short flag from default zenoh program line argument

lgtm


Use Mono script metadata also for class->script mapping

This fixes #27095 by enabling the appropriate CSharpScript's to be reused when the mono module attempts to get a CSharpScript for a given Mono class.
This primarily occurs when one does:
in "MyNodeSubclass.cs":
class MyNodeSubclass : Node { ... }

When new MyNodeSubclass() is used, Godot will now correctly assign the script "res://MyNodeSubclass.cs" to the newly created Node, instead of assigning a script with an empty path (which will not be saved to the Scene).
The script metadata is now kept in-memory in two Map's to enable bi-directional lookup.
The only caveat is that during _update_exports, we need to pre-allocate the native Node before we call the user's constructor. Otherwise the default constructor would call back into C++ to create the native Node, which would trigger a call to ResourceLoader::load for the script that is in the process of being created. This would simply fail. If the managed object is already associated with a native object before the constructor is called, this doesn't happen.

CC @neikeq

What's the status on this PR?

Ping.


Byebug loses Keyboard input on windows

byebug version 8.2.4, using it with Rails 5.0.0beta3
Windows 10, ruby 2.2.3p173 (2015-08-18 revision 51636) [x64-mingw32]

Something quite weird happens randomly when I am using byebug. Sometimes I partially lose keyboard input. Partially because

up/down arrows for reverse history still work and pop up the text in my powershell windows
Enter key still "submits" the input,
any other key, including paste shortcut, does not do anything (apart from submitting the prevous command when I press enter)

I cannot even kill the process with Ctrl+c and I have to manually close my PS window and restart my ruby/rails process when that happens
I have no idea how I can debug that and give you feedback. Please tell me if there is something I can do, or I will keep monitoring and reply asap when I can find easily reproduceable steps

This is a dup of #59 or #63, we should probably merge those into a single bug! You could try reapplying https://github.com/deivid-rodriguez/byebug/commit/bb98cd60cfeceee8ce626a222346ef432e7d2a0e and see if it fixes the issue.


PL Functions - ALTER FUNCTION

First - thanks for the tool.
I've started to try it out this morning and so far it's quite nice - we added our own Sequence definition delta tool since your tools does not seem to support it (only values).
We have a bug though when comparing functions (PL in our case) - the function definition is alright but the ALTER FUNCTION command has a glitch, it adds the param definition 2 times:
ex:
ALTER FUNCTION public.se2_isnumeric(text)(text) OWNER TO XXX;
instead of having
ALTER FUNCTION public.se2_isnumeric(text) OWNER TO XXX;
Let me know if you think you can fix that in the coming days - if not we will build our own.
Regards
Frank

Hi Frank,
thanks for your feedback.
Sorry but we are not able to reproduce your bug; maybe it is generated from different version or settings of Postgresql?
That command is generated from this part of code:
// / add function owner   _sqls.push('ALTER FUNCTION ' + _f1.name + '(' + _f1.args + ') OWNER TO ' + options.owner)
so maybe in your case the name of the function already contains the arguments? That values are taken by the "getFunctions" function in the file Schema.js.
If you could check it and then fix it, we have the pleasure of accepting your fix.
Regards
Andrea

We're on the v10 of PG... Maybe that's the reason. I saw that we did had
access to source code and yeah I already did the fix by removing the f1
args... Args definition is indeed in the name.
Thanks for your reply
On Feb 23, 2018 05:05, "andrea-latini"<EMAIL_ADDRESS>wrote:
Hi Frank,
thanks for your feedback.
Sorry but we are not able to reproduce your bug; maybe it is generated from
different version or settings of Postgresql?
That command is generated from this part of code:
// / add function owner _sqls.push('ALTER FUNCTION ' + _f1.name + '(' +
_f1.args + ') OWNER TO ' + options.owner)
so maybe in your case the name of the function already contains the
arguments? That values are taken by the "getFunctions" function in the file
Schema.js.
If you could check it and then fix it, we have the pleasure of accepting
your fix.
Regards
Andrea
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ambrogio-dev/pg-compare/issues/10#issuecomment-367965475,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASg4EDMqvgtIkzqtMFnCv6bqGkmQodSeks5tXo1ZgaJpZM4SP1MD
.


Simple canvas optimizations

I'm trying to use Leaflet with a large amount of polylines, and I've found a few low-hanging fruits which make things snappier when using the canvas renderer.

throttle mouseover using L.Util.throttle
don't check for hover when the map is being panned (since the mouse isn't moving in relation to the map)
only check visible layers when checking for hover (by storing unculled layers in an array and only checking these layers)

Thank you for reviewing. I'm new to Leaflet, so I hope this doesn't break anything that I don't know about!

I fat-fingered the enter key wayyy before this was ready, sorry about that!

Wow, nice optimizations! I'll merge when it's ready :)

Thanks a lot for this, I've cherry-picked these changes onto my fork.
One small problem I found is that if you change the polygon styles but don't move the map (thus triggering _updatePoly but not _update), _drawnLayers just grows perpetually.
A quick change I would suggest is to make _drawnLayers an object and use the layer's _leaflet_id as an index, avoiding duplicate entries for the same layer. I've made the changes on my fork here: https://github.com/wpf500/Leaflet/commit/00c5e46a2daff196d1fe1e73f092fa12c085b2f9

@philippelatulippe @wpf500 hey, would love to see this ready for merging! Any updates?

@philippelatulippe @wpf500 bump :)

The best thing would be that layers don't get added to _drawnLayers when their style changes. Let's see if I can do that elegantly.

Alright, I cherry-picked @wpf500's commit.

:+1: thanks a lot!


Linenoise broken on Windows as of 2016.11

I had to revert the version of Linenoise shipped with star to the one before Native::Resources removal in order to get it to work on Windows 10/mingw (from Strawberry Perl)

@stmuk What kinds of problems did you see with the latest linenoise?

@stmuk What kinds of problems did you see with the latest linenoise?

The problem is the make file has a hard coded liblinenoise%SO% but also relies on the $*VM.platform-library-name elsewhere which doesn't prefix the library name with 'lib'. So it fails trying to find liblinenoise.dll when it should be linenoise.dll
I've fixed that issue in #19, but starting linenoise via perl6 after install prints out a message about a missing symbol. I suspect this is an issue with the dll being renamed to $some-sha1 ~ '.dll' when installed, and that it needs to keep its original file name (see: https://github.com/sergot/openssl/commit/91efa8c4f53e86501f23b3a6aeb4184cfedbfa9f)

@ugexe Thanks for the feedback - I've merged your PR.  Is there other work that needs to be done for this to work on Windows?

Yes, you need to implement something like https://github.com/sergot/openssl/blob/91efa8c4f53e86501f23b3a6aeb4184cfedbfa9f/lib/OpenSSL/NativeLib.pm6 and replace your use of %?RESOURCES<libraries/linenoise> with something like the &ssl-lib from OpenSSL/NativeLib.pm6. The purpose of NativeLib.pm6 is that on Windows it copies the installed perl6/install/site/resource/lakjf09jf2093jf.dll to a temporary directory, renames it to its original linenoise.dll, and returns an IO::Path pointing at that.
This is really a bug in CompUnit::Repository in that there needs to be a way to declare a file name cannot be mangled, so don't view this suggestion as anything other than a workaround.

@ugexe I'm kind of loathe to copy/paste some DLL-fixing code from the OpenSSL module, partially since it doesn't have a license in place.  I don't like Linenoise being broken on Windows, but perhaps it would be better to wait for the CompUnit system to support the non-workaround behavior you described above.

I'm not a windows user and I'm not aware of anyone newly complaining about this but I encountered this bug while following up on another issue, and then dug a bit further to understand the status of this issue, and think it would be useful to update this issue with what I see, especially point 4:

Aiui, Linenoise works on windows for rob but is broken for at least ugexe and stmuk and maybe others, maybe all others. (I've no idea but I find it hard to believe it's broken for all windows users!)

Per ugexe's comment just above, there's a "bug in CompUnit::Repository::Installation in that there needs to be a way to declare a file name cannot be mangled".

Per that same comment, there's a workaround that's been used for OpenSSL.

Per your comment just above one of the reasons you didn't copy/paste that workaround for this Linenoise module was a lack of license at that time. Since then one has been added: https://github.com/sergot/openssl/blob/master/LICENSE

Per ugexe's 2018 comment comment https://github.com/rakudo/rakudo/pull/2212#issuecomment-413698734 I'm guessing they have not fixed the above mentioned bug

@raiph Thanks for filling in this information! BTW, do you know of anyone would like to take over maintainership of Linenoise?  I haven't worked on Perl 6 for a while and free time is at a premium for me, so I can't really give it the love and attention it deserves!

OK, I guess that'll have to be us. This one is a blocker for #29, so we'll try this one to go first.

For the time being, the Makefile (and some paths) are Linux-specific. I'm trying to find a way to compile it, in its current version, in Windows, and eventually I think we'll have to create two separate Makefiles.

OK, this is working now in Windows so we have at least a procedure to carry it out. I'll try and find a way to put this in the Makefile so that it's OS-independent.


Website whitelist automoderator V2 codeblocks false positives

I tried using automoderator V2 website whitelist. The problem is that it is picking up code like this:
list.sort()

As an attempt to send a website link and triggers an unwanted action.
Is this a bug?

On further review of the responsible code i noticed that the regex being used to identify links is
(http(s)?:\/\/)?(www\.)?[-a-zA-Z0-9@:%_\+~#=]{1,256}\.[a-z]{2,6}\b([-a-zA-Z0-9@:%_\+.~#?&//=]*)

Which is too sensitive and will pick up this kind of false positives. Perhaps a solution would be to add an option to select the method for detecting links. (https://www.google.com vs google.com, for example).

A fix was implemented with #1184 and the next release will mitigate this regression.


cannot connect urm.nvidia.com while run ./build.sh

workspace/mjcf-importer-extension | main  ./build.sh                                     ok | 14:17:28
Package 'repo_build' at version '0.62.7' is missing from local storage.
packman(WARNING): HTTPSConnectionPool(host='urm.nvidia.com', port=443): Max retries exceeded with url: /artifactory/ct-omniverse-generic/pkgs/repo_build/repo_build%4<IP_ADDRESS>z (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)')))

is  urm.nvidia.com available？

Same, hitting it this morning

Same problem here.  Judging by the lack of response in 2 months, is this package even actively supported?


Get time-lapse view to select current line. Update version to 2.6.3.

Fix https://github.com/SimpsonGSD/P4EditVS/issues/20

Merged, thank you.


CAMEL-19955 - Camel-Infinispan: Review the usage of Protostream sampl…

…e domain definition and implementation in tests
Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test camel-infinispan/camel-infinispan-embedded camel-infinispan/camel-infinispan-common camel-infinispan/camel-infinispan


Update show.go to show (latitude, longitude), not (long, lat).

The general convention per ISO_6709, is that a coordinate is (lat, long), not (long, lat). It took me a minute of staring at pmtiles show output to realise the tool flipped them.
https://en.wikipedia.org/wiki/ISO_6709

oh gah, even more confusing than I realised.
Updated the PR with your suggestion.

Thanks!

https://github.com/protomaps/go-pmtiles/releases/tag/v1.14.1


Re-export 3rd party types exposed in the API

While using starknet-in-rust some types defined in other crates are exposed as part of the API. For example: cairo_lang_starknet::casm_contract_class::CasmContractClass and cairo_lang_starknet::contract_class::ContractClass, cairo_vm::felt::Felt252 or starknet_contract_class::EntryPointType.
I think it would be better if starknet-in-rust actually re-exported these types so that I don't have to add the transitive dependencies directly to my crate so that I can use the starknet-in-rust API. This is especially painful since I have to make sure to use the same versions of these 3rd party crates.
For more discussion please see https://effective-rust.com/re-export.html

Thanks, we were considering that. We'll fix it in #653.


Bump containerd and opencontainers/image-spec

Updates:

github.com/containerd/containerd from 1.6.6 to v1.6.12
github.com/opencontainers/image-spec from v1.0.3-0.20211202183452-c5a74bcca799 to v1.1.0-rc2

This also fixes CVE-2022-23471


How can one setup Rust lang completions using znap

So I'm slowly trying to move as much of zsh control over to znap and when I try to generate completions for rust with the code below I get the error _arguments:comparguments:325: can only be called from completion function
# rust and cargo completions
znap eval rust-completion 'rustup completions zsh'
znap eval cargo-completion 'rustup completions zsh cargo'

I wondering how you'll go about it.

znap eval 'foo' is a direct replacement for eval "$(foo)" only. What you are trying to replace is
rustup completions zsh > ~/.zfunc/_rustup
fpath+=~/.zfunc

You need one extra step to get what you want:
znap function _rustup 'znap eval rustup-completion "rustup completions zsh"'
compdef _rustup rustup


[API Enhancement] : Global Holiday Api

Hello @dishamodi0910,
please meged this...
🎯 Title and Issue Number
Before

After

Closes #286
🗒 Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x]  I have gone through rules of when PR will get merged.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have followed proper file structure.
[x] I have gone through CONTRIBUTING GUIDELINES.md.

Thank you for contributing!❤
Don't forget to show some love by ⭐ the repository!

Hey @sivaprasath2004, Thank you for creating PR! We will review it super soon. 😇❣
As we continue to evolve and innovate, we encourage you to share your ideas, feedback, and code to help us push the boundaries of what's possible in the world of APIs🚀.
We hope you have followed CONTRIBUTING GUIDELINES.
Thank you for being an essential part of our community. Together, we'll keep building amazing things!
Don’t forget to ⭐ our repository!
Happy Coding!✨
Will See you soon❣

@sivaprasath2004 I guess these changes aren't needed much as of now, as it's not adding any functional value to API. So closing this PR.

@sivaprasath2004 ,I wanted to reach out and let you know that we have decided to close your pull request.🙇
In case of any issues, you can contact us on Discord.
Thank you! ❣
Happy Coding! ✨
Will See you soon❣


How do you use this without flakes?

I get this error when I try using the latest version without flakes:
Run nixbuild/nix-quick-install-action@v25
Run /home/runner/work/_actions/nixbuild/nix-quick-install-action/v25/nix-quick-install.sh
/*stdin*\           : 8[7](https://github.com/anka-213/gf-lsp/actions/runs/5858046239/job/15881637273#step:3:8)746560 bytes
installing 'nix-2.16.1'
building '/nix/store/yx8dlip6m5lb4s9pl0693abpsalr78ga-user-environment.drv'...
Run /home/runner/work/_actions/nixbuild/nix-quick-install-action/v25/read-nix-config-from-flake.sh
+ jq -r '.content|gsub("[\n\t]"; "")|@base64d'
+ gh api -H 'Accept: application/vnd.github+json' -H 'X-GitHub-Api-Version: [20](https://github.com/anka-213/gf-lsp/actions/runs/5858046239/job/15881637273#step:3:22)22-11-28' '/repos/anka-213/gf-lsp/contents/flake.nix?ref=f60fce4a36646e06b70[31](https://github.com/anka-213/gf-lsp/actions/runs/5858046239/job/15881637273#step:3:34)fa48fb2ab16cebf2074'
gh: Not Found (HTTP 404)
jq: error (at <stdin>:0): null (null) cannot be matched, as it is not a string

Here's the full log: https://github.com/anka-213/gf-lsp/actions/runs/5858267269/job/15881863895

Hmm, this looks bad, thanks for reporting it. Can you work around it by setting load_nixConfig to false?

cc @blaggacao

I see. At the point of hitting this error, we probably have at least a config drift, but we could fail much more gracefully.
Wdyt?

Hmm, this looks bad, thanks for reporting it. Can you work around it by setting load_nixConfig to false?

Tried this too, doesn't help
Any suggestions?

Hello, I'm hitting the same issue here.
I'd love to be able to use nix-quick-install-action in order to leverage cache-nix-action, which seem to require this specific package to work.

@jgoux Could you try using the latest revision (nix-quick-install-action@master)? I just committed a couple of fixes. It would be ideal if you could try it out both with load_nixConfig set to false and true.

@rickynils I just tried against nix-quick-install-action@master, it seems to work with the option load_nixConfig set to false. I tried true and got the same error as before. 👍
Thanks for the reactivity!

@rickynils The output is here: https://github.com/jgoux/acme/actions/runs/6623591526/job/17990926008#step:3:52

@jgoux Would you mind re-running that workflow with load_nixConfig = true again?

@rickynils It's now working with load_nixConfig = true! Great job! https://github.com/jgoux/acme/actions/runs/6624474804/job/17993531555?pr=25#step:3:34

Fixed in v26


PDF integration

Is there any way to include a pdf in between images? What would you recommend how to do that?

if you want to open a pdf file you can use thumbnailsAsLinks and set url in https://github.com/lukasz-galka/ngx-gallery#ngxgalleryimage

thanks a lot for your quick answer! this helps ...

Is something similar like previewsAsLinks available for preview images?

@ManPla no


Library not working in frozen application (e.g. with PyInstaller)

I'm trying to build a CLI/Terminal app (frozen with PyInstaller), and it seems that the macos-notifications library starts a new process that calls my CLI again, but with unexpected parameters. Is there any way to work around this problem?

Hi @MShekow,
Sorry to hear that. Could you provide a minimal example so I can try to replicate it? I would expect it to work if you wrap all your code around in an if __name__ == "__main__" check

Sure.
main.py:
import argparse

from mac_notifications import client

if __name__ == '__main__':

    parser = argparse.ArgumentParser()
    parser.add_argument("foo")
    parser.add_argument("bar")
    parser.add_argument("baz")
    args = parser.parse_args()
    client.create_notification(title="foo", text="bar")

pyinstaller.spec:
# -*- mode: python ; coding: utf-8 -*-
import sys

block_cipher = None

a = Analysis(
    ['main.py'],
    pathex=[],
    binaries=[],
    datas=[],
    hiddenimports=[],
    hookspath=[],
    hooksconfig={},
    runtime_hooks=[],
    excludes=[],
    win_no_prefer_redirects=False,
    win_private_assemblies=False,
    cipher=block_cipher,
    noarchive=False,
)
pyz = PYZ(a.pure, a.zipped_data, cipher=block_cipher)

exe = EXE(
    pyz,
    a.scripts,
    [],
    exclude_binaries=True,
    name='testapp',
    debug=False,
    bootloader_ignore_signals=False,
    strip=False,
    upx=True,
    console=True,
    disable_windowed_traceback=False,
    argv_emulation=False,
    target_arch=None,
    codesign_identity=None,
    entitlements_file=None,
)
coll = COLLECT(
    exe,
    a.binaries,
    a.zipfiles,
    a.datas,
    strip=False,
    upx=True,
    upx_exclude=[],
    name='testapp',
)

Run pip install pyinstaller macos-notifications, then just freeze the app with pyinstaller pyinstaller.spec, then run the app with ./dist/testapp/testapp 1 2 3.
Console printout:
usage: testapp [-h] foo bar baz
testapp: error: the following arguments are required: bar, baz
usage: testapp [-h] foo bar baz
testapp: error: the following arguments are required: baz
multiprocessing/resource_tracker.py:104: UserWarning: resource_tracker: process died unexpectedly, relaunching.  Some resources might leak.

Also, very funny, I dare you to instead use the following main.py (if you want to see a logic bomb in action):
from mac_notifications import client

if __name__ == '__main__':
    client.create_notification(title="foo", text="bar")

It gives you a good impression of how it feels to hard-reboot your mac :S

Any updates on this? Running into the same issue, I'm expecting a notification but the app just starts a new instance

Thanks again for raising this issue.
Unfortunately, I don't have any updates on this. I am not really aware of the PyInstaller setup and therefore wouldn't know what is breaking it. I am not convinced this is necessarily because of the macOS-notification library, but because of a slightly different Python setup. Therefore, I will keep this issue open to allow anyone to investigate (and find) the issue. However, I hope you can respect my decision to not investigate this myself.

It seems to happen on any frozen python application


Ignore indexes managed by spanner

Ignore indexes managed by spanner.
Spanner creates secondary backing indexes for foreign keys.
Such indexes have random names and are not suitable for use via yo generated code.

Nice. I know that but forgot to fix it.


setting columnOrder does not redraw the grid

Depends on #159 being fixed first in order to be able to set the columnOrder successfully.
Expected behavior and actual behavior.
I would probably expect that setting the columnOrder property would cause the grid to immediately redraw with the new column order, but it does not unless I call grid.draw() after setting the column order.
Is this by design?  Maybe there's a reason for the current behavior, such as allowing the developer to make multiple changes to multiple properties and then redraw the grid all at once (kind of like a transactional capability)?
Steps to reproduce the problem.
Having a reference to a grid (e.g., window.grid) execute the following to reverse the column order:
window.grid.columnOrder = window.grid.schema.map((_, i) => i).sort((a, b) => b - a);

and see that even though the columns have been reversed the grid does not update/redraw.
Then issue
window.grid.draw()

and see that the grid now updates.
Specifications like the version of the project, operating system, or hardware.
Latest version, Windows 10

Yeah, it seems like it should redraw for you.  Drawing is super cheap.

For sure let's just add the draw to the setter, if you're able to do it please do open a PR.  Although headless automated testing does not work right now manual testing is a must.  Run /test/tests.html on your local machine before you make a PR to make sure all the tests pass.
The reason automated testing is broken is because headless chrome does not draw on a canvas and phantomjs does not support web components, so I'm totally screwed with testing until I have a proper CI/CD environment setup... should make a ticket for that.  I'm using coveralls right now and I'd love to get proper web hook integration and automated tests working again.  It just seems right now that there is no love for complex canvas projects that use free software (not VMs) for testing.
I want to do it, but I am but a single person.  Sometimes the more tickets I add to the project the more daunting it seems work on it.  Thanks for volunteering to help!  Not a lot of that around here.  But I must endure... VQP


dp.Group no longer supports rows

System Information

OS: OSX 12.1
Python version: 3.8.5
Python environment: poetry
Using jupyter: false
Datapane version: 0.13.2

Bug / Issue
dp.Group no longer supports the rows attribute.  While I agree with this in theory, I see no way under the 0.13.2 api to group multiple objects in a dp.Select.  Previously I could:

dp.Select(blocks=[
    dp.Group(
        dp.Text('# Group a stuff here'),
        dp.Chart(charta),
        dp.Table(dfa),
        rows=3,
        label='a',
    ),
    dp.Group(
        dp.Text('# Group b stuff here'),
        dp.Chart(chartb),
        dp.Table(dfb),
        rows=3,
        label='b',
    ),
])

While I do not mind the change to groups being row centric, I now have reports that can not be built (or at least I see no solution to build an arbitrary container around elements that I can give a label for insertion into a dp.Select.  In the use-case above if I were to put the three elements in the group side-by-side, my users would potentially be scrolling, or even experience data clipping.
Is there a way to make a select that has multiple entries under it that are not side-by-side in the current version?

It appears that the row parameter is just now assumed:
dp.Select(blocks=[
    dp.Group(
        dp.Text('# Group a stuff here'),
        dp.Chart(charta),
        dp.Table(dfa),
        label='a',
    ),
    dp.Group(
        dp.Text('# Group b stuff here'),
        dp.Chart(chartb),
        dp.Table(dfb),
        label='b',
    ),
])

appears to be the proper way to handle this now, though at first glance this did not appear obvious to me.  Perhaps a documentation change is needed?

@mands feel free to close this, my comment describes my fix.  I am leaving this open in case you want to change any of the docs to highlight this change.

Thank you for raising this @gelliot-os!
I've made a change to our documentation to remove the mention of a row parameter for the Group component.
Old:

Datapane provides a Group component which takes a list of blocks and a number of columns and/or rows and lays them out in a grid

New:

Datapane provides a Group component which takes a list of blocks and a number of columns and lays them out in a grid.


Bits missing for a successful installation: update versions, missing repository, missing dependencies

Hi, great plugin! Thanks a lot!
I found the following steps necessary to successfully cli build with your plugin.
Maybe these steps could be automatised during plugin installation? Unfortunately I know too little about Cordova plugins yet. In any case, it would be helpful to mention them in your README file.

Save downloaded google.services.json to platforms/android (i.e. not app or mobile)

Check https://developers.google.com/cloud-messaging/android/client for current Versions
com.google.gms:google-services:1.5.0-beta2
com.google.android.gms:play-services:8.3.0

Add classpath 'com.google.gms:google-services:1.5.0-beta2' to platforms/android/build.gradle

    if (gradle.gradleVersion >= "2.2") {
        dependencies {
            classpath 'com.android.tools.build:gradle:1.0.0+'
			classpath 'com.google.gms:google-services:1.5.0-beta2'
        }
    } else if (gradle.gradleVersion >= "2.1") {
        dependencies {
            classpath 'com.android.tools.build:gradle:0.14.0+'
			classpath 'com.google.gms:google-services:1.5.0-beta2'
        }
    } else {
        dependencies {
            classpath 'com.android.tools.build:gradle:0.12.0+'
			classpath 'com.google.gms:google-services:1.5.0-beta2'
}

Add jcenter to platforms/android/build.gradle

buildscript {
    repositories {
		jcenter()
		mavenCentral()
    }
...

Install plugin https://www.npmjs.com/package/cordova-plugin-gcmpushplugin

cordova plugin add cordova-plugin-gcmpushplugin

This will create a.o.

platforms/android/cordova-plugin-gcmpushplugin/hello-build-extras.gradle
plugins/cordova-plugin-gcmpushplugin/build-extras.gradle

Edit plugins/cordova-plugin-gcmpushplugin/build-extras.gradle

to reference the correct versions, e.g.

apply plugin: 'com.google.gms.google-services'

buildscript {
    repositories {
        jcenter()
    }

    dependencies {
classpath 'com.google.gms:google-services:1.5.0-beta2'
    }
}

dependencies {
 compile "com.google.android.gms:play-services:8.3.0"
}

Overwrite other extra file
cp plugins/cordova-plugin-gcmpushplugin/build-extras.gradle platforms/android/cordova-plugin-gcmpushplugin/hello-build-extras.gradle

This is amazing! Thanks a lot! This should be pushed to the main README.md page.
Along with this, I found few more issues
STEP 9:
cp plugins/cordova-plugin-gcmpushplugin/build-extras.gradle platforms/android/cordova-plugin-gcmpushplugin/android-build-extras.gradle

STEP 10: If you get an error like
UNEXPECTED TOP-LEVEL EXCEPTION:
com.android.dex.DexException: Multiple dex files define Landroid/support/annotation/AnimRes;

Then you have to create a separate gradle file by name build-extras.gradle and place it inside platforms/android directory. Inside this file you have to add:
configurations {
    all*.exclude group: 'com.android.support', module: 'support-v4'
}

That's it!. It should build after this.

@srameshr , Hey were you able to solve the problem? I am facing the same issue.

Just use phonegap-push plugin. That is way better and maintained.

ok thanks


additional string manipulation functions for template

Currently confd provides only small subset of functions listed here: https://github.com/kelseyhightower/confd/blob/master/resource/template/template_funcs.go#L11
What I would like to have is more functions related to string manipulation. Currently I'm missing replace, toUpper.
consul-template provides such functions:
https://github.com/hashicorp/consul-template#regexreplaceall
So I think it would be great idea to include those into confd.

@mariusgrigaitis Can this be closed now that #253 has been merged?

I think partially closed. Replace would also be good to have I think. Maybe we should close (rename) this and create new one?

yes, let's close and add bugs for specific string manipulation functions so we can track their progress individually (some may make sense and some may not).


Only partial syntax highlighting in NeoVim.

Syntax highlighting for Svelte in Neovim only works well for the html block. The script and style blocks aren't highlighted correctly. The script block is only partially highlighted and the css block is not highlighted at all (see the screenshot below).
My tree sitter parsers are up to date, I have html, css, typescript and svelte parsers installed. I use the current LTS version of Node (20.10.0). I have no idea how to debug this further. Any suggestions?

I found out what the issue was myself. I use Lazy.nvim as my package manager in Neovim. There I configured treesitter to be installed with config value version = "*". Apparently, this installs quite an old release of treesitter (as I found out by looking at a comment in the LazyVim distro). If you configure treesitter with version = false, a newer version will be installed where the highlighting works as expected.


Flush() doesn't upload all cached files data

There are some old and history event data in the cache folder
C:\Users\myuser\AppData\Local\segment.data.
It will only upload the latest generated cached file when Flush() is called.
The old file data would not be uploaded.
For instance, the two files are generated before. but it's not uploaded when Flush() is called.

Also, one question from this:
How does Flush() work?
It doesn't scan the cached folder according to the source code?
https://github.com/segmentio/Analytics-CSharp/blob/main/Analytics-CSharp/Segment/Analytics/Analytics.cs
public virtual void Flush() => Apply(plugin =>
{
    if (plugin is EventPlugin eventPlugin)
    {
        eventPlugin.Flush();
    }
});

@arlyxiao, the flush does scan the cached folder, but only look for the files with an extension (see here). the ones that do not have extension are temporary files. the SDK first writes events to a temporary file that has no extension, and rolls over to another temporary file when finish. the previous file is then closed with an extension name (see here).
the files in the screenshots likely are generated by previous tests. install/uninstall apps may cause this issue, since the SDK lost the track of the files. you can customize the behavior to clean up all files by providing your own storage implementation (see here)


[Bug] handle spaces in file names

Search before asking

[X] I searched the issues and found no similar issues.

Component
Other
What happened + What you expected to happen
If I have a PDF named My Important Document.pdf  the transformation produces final parquet as My%20Important%20Document.pdf
We can either

preserve space in name (preferred)
or replace spaces with _

Also we need to keep the original filename in filename attribute
Reproduction script
Feed a PDF with spaces in file name to the pipeline
Anything else
No response
OS
Ubuntu
Python
3.11.x
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Interesting finding. I think the name change might originate from the DataAccess and the transform_binary()
@blublinsky @daw3rd what do you think?

closing.  Looks like the input files had wrong names to begin with


Jetty library issues

There are two variants of jetty in the runtime. This seems to be unique to dStore and does not exist in terrapin.
(Though maybe that's changed lately?) The result is after clients call we get the following message:
09:08:57.743 [dw-29] [] WARN  o.eclipse.jetty.server.HttpChannel - /v1/tenant/123
java.lang.NoClassDefFoundError: org/eclipse/jetty/http/HttpFields$Mutable
        at ch.qos.logback.access.jetty.RequestLogImpl.makeJettyServerAdapter(RequestLogImpl.java:264)
        at ch.qos.logback.access.jetty.RequestLogImpl.log(RequestLogImpl.java:255)
        at org.eclipse.jetty.server.HttpChannel.onCompleted(HttpChannel.java:911)
        at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:467)
        at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:277)
        at org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:311)
        at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:105)
        at org.eclipse.jetty.io.ChannelEndPoint$1.run(ChannelEndPoint.java:104)
        at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:883)
        at org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:1034)
        at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: java.lang.ClassNotFoundException: org.eclipse.jetty.http.HttpFields$Mutable
        at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
        at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
        ... 11 common frames omitted

Need to figure out how to stop this.
The culprit originally appears to be the dropwizard metrics library, but evidence for that has failed.  What I've tried to date:

Force all dependencies to be of the jetty version pulled in by dropwizard. (Gradle does not make this easy.)
Removed metrics completely, including dependencies.
Tried to get everything updated to the very latest jetty. (That did not go well)

Well poop... the issue is discussed here: https://github.com/dropwizard/dropwizard/discussions/5821#discussioncomment-5037636
As it turns out, Dropwizard 3.0.x has a fix/workaround in the branch. Neither 3.x nor 4.x are released yet. I'm gonna poke at their branch and see what I can do to also provide a workaround.
The issue as it turns out is in the request logging... I kept thinking I was pulling in the wrong jetty version. Rather logback-access is at fault. In the dropwizard code, the work around is in dropwizard-request-logging

Going to wait on this... not actively working on it. It's more noise right now. Dropwizard will end up releasing a new version before we are done with svarm.

Updated to dropwizard 4... done.


spi_write_read_blocking reads/writes only 1 byte per CS cycle

I have two Raspberry Pi Picos connected via SPI, the master is running CircuitPython while the slave is using the C/C++ SDK. The Picos are not communicating as expected and after some debugging I found that the problem is the spi_write_read_blocking function of the C/C++ SDK. It seems that, while CS is pulled down, the function only reads/writes a single byte and will not read/write the following bytes until CS is pulled up and pulled down again.
This behaviour is incompatible with CircuitPython and many other SPI devices. This is a deal breaker for me because the master Pico in my project is just emulating some device I want the slave Pico to interface with, and the real device keeps CS low for the entirety of the transmission and doesn't cycle CS with each byte.
Master Pico
import board
import busio
import digitalio
from adafruit_bus_device.spi_device import SPIDevice

with busio.SPI(board.GP18, board.GP19, board.GP16) as spi_bus:
    device = SPIDevice(spi_bus, digitalio.DigitalInOut(board.GP17))

    while True:
        with device as spi:
            rx = bytearray(4)
            tx = bytearray([0x41, 0x42, 0x43, 0x44])
            spi.write_readinto(tx, rx)

Slave Pico
#include <hardware/spi.h>
#include <pico/printf.h>
#include <pico/stdlib.h>

int main() {
    stdio_init_all();

    spi_init(spi_default, 0);
    spi_set_slave(spi_default, true);
    gpio_set_function(PICO_DEFAULT_SPI_RX_PIN, GPIO_FUNC_SPI);
    gpio_set_function(PICO_DEFAULT_SPI_CSN_PIN, GPIO_FUNC_SPI);
    gpio_set_function(PICO_DEFAULT_SPI_SCK_PIN, GPIO_FUNC_SPI);
    gpio_set_function(PICO_DEFAULT_SPI_TX_PIN, GPIO_FUNC_SPI);

    uint8_t rx[4];
    uint8_t tx[4] = {'D', 'C', 'B', 'A'};
    while (true)
        spi_write_read_blocking(spi_default, tx, rx, 4);

    return 0;
}

Logic Analyzer

Here we can see that, while the master writes "ABCD" in a single CS cycle, the slave writes "DCBA" only one char per cycle.

Related to #88 ?

Yeah, I went through the PL022 spec and it seems to be a hardware limitation…
When CPHA = 0

However, in the case of continuous back-to-back transmissions, the SSPFSSOUT signal must be pulsed HIGH between each data word transfer. This is because the slave select pin freezes the data in its serial peripheral register and does not permit it to be altered if the SPH bit is logic zero. Therefore, the master device must raise the SSPFSSIN pin of the slave device between each data transfer to enable the serial peripheral data write. On completion of the continuous transfer, the SSPFSSOUT pin is returned to its idle state one SSPCLKOUT period after the last bit has been captured.

When CPHA = 1

For continuous back-to-back transmissions, the SSPFSSOUT pin remains in its active-LOW state, until the final bit of the last word has been captured, and then returns to its idle state as the previous section describes.

In my scenario I need CPHA = 0 and CS pulled down for the entire transmission, so I think PIO is the only alternative to implement this on the Pico.

sounds like this is solved

Nice


Example/bm25 hybrid search

need to update blog link only

@akashAD98 let's add the blog link once published then I'll merge this


How to get proper type in for loop collection?

How can I annotate this code to type properly?
rawReplacements: Record<string, string> := a: 'z', b: 'y', yz: 'a'
replacements: [RegExp, string][] :=
   ([RegExp(pat, 'g'), rep] for pat, rep in rawReplacements)

As far as I can see, the loop collection indeed produces a value of type [RegExp, string][] (and I need it typed that way for later code). But I get the error:
error TS2322: Type '(string | RegExp)[][]' is not assignable to type '[RegExp, string][]'.
  Type '(string | RegExp)[]' is not assignable to type '[RegExp, string]'.
    Target requires 2 element(s) but source may have fewer.

Thanks for any help!

This types correctly, though I don't quite understand why readonly is necessary when using as const:
rawReplacements: Record<string, string> := a: 'z', b: 'y', yz: 'a'
replacements: (readonly [RegExp, string])[] :=
   ([RegExp(pat, 'g'), rep] as const for pat, rep in rawReplacements)

as const indicates that the value will never change, so it needs to be readonly.
The reason this is happening is that TS is inferring that array literal (without as const) as (RegExp | string)[] rather than [RegExp, string]. Adding as [RegExp, string] (where @edemaine put as const) should work.

Ah, I was just typing that:
I can work around this by writing
(([RegExp(pat, 'g'), rep] as [RegExp,string]) for pat, rep in rawReplacements)

But it would be nice to not need the "as" clause, since the expression [RegExp(pat, 'g'), rep] seems pretty patently to have type [RegExp, string].  But now I see, TypeScript doesn't infer tuple types for array expressions because who knows, you might append another RegExp or string onto that element. There's not a notation for specifically a tuple in TypeScript/Civet, is there, other than the explicit type annotation?

I think we should default to as const when aggregating tuples but still allow people to override a different type using as ....

I'm pretty sure satisfies [unknown, ...unknown[]] forces it to be a tuple, but that's longer than the as clause in this case. See https://github.com/DanielXMoore/Civet/issues/849#issuecomment-1851254735

Aha, I just discovered that the following almost trivial operation is super useful in this context:
function tupleize<Tup extends unknown[]>(...arg: Tup): Tup arg

With that, the following works just fine:
rawReplacements  := a: 'z', b: 'y', yz: 'a'
replacements: [RegExp, string][] :=
   (tupleize RegExp(pat, 'g'), rep  for pat, rep in rawReplacements)

I might even suggest that if aggregating tuples as const is too much by default, that you might want to wrap them in tupleize instead, to give more presumptuous type inference that seems typically useful.

Solved by #1066


why? Error creating connection dial tcp <IP_ADDRESS>:28015: getsockopt: connection refused

shipyard-swarm-manager_1 | time="2015-12-03T08:39:55Z" level=info msg="Listening for HTTP" addr="<IP_ADDRESS>:3375" proto=tcp
shipyard-swarm-manager_1 | time="2015-12-03T08:39:55Z" level=error msg="Discovery error: client: etcd cluster is unavailable or misconfigured"
shipyard-swarm-manager_1 | time="2015-12-03T08:39:55Z" level=error msg="Discovery error: client: etcd cluster is unavailable or misconfigured"
shipyard-swarm-manager_1 | time="2015-12-03T08:39:55Z" level=error msg="Discovery error: Unexpected watch error"
shipyard-controller_1    | time="2015-12-03T08:39:55Z" level=info msg="shipyard version 3.0.1"
shipyard-controller_1    | time="2015-12-03T08:39:55Z" level=warning msg="Error creating connection dial tcp <IP_ADDRESS>:28015: getsockopt: connection refused"
shipyard-controller_1    | time="2015-12-03T08:39:55Z" level=fatal msg="no connections were made when creating the session"
webserver_1              | 2015-12-03 08:39:56,579 INFO spawned: 'php-fpm' with pid 11
webserver_1              | 2015-12-03 08:39:56,580 INFO spawned: 'nginx' with pid 12
webserver_1              | 2015-12-03 08:39:56,581 INFO spawned: 'memcache' with pid 13

What version of Docker?

I'm having the same problem. Docker version 1.9.1, build a34a1d5. Also appears to resemble issue #668 where the controller goes into "restarting" status.
@ehazlett any update on this?

Docker version 1.8.1, build d12ea79

What is your host OS?  It looks like perhaps the datastore container didn't start or is having trouble access it.

@liusongsen
“etcd cluster is unavailable or misconfigured"  It looks like your etcd docker (microbox/etcd ?) error. I have the same problem,but I config etcd cluster outside is well, now.
I have a problem: (when I add shipyard-swram-agent)
"getsockopt: no route to host. Are you trying to connect to a TLS-enabled daemon without TLS? "

not etcd cluster is unavailable or misconfigured
<IP_ADDRESS>:28015 is database

I suspect it is the starting order (docker-compose.yml),
I made a docker-compose.yml, this docker-compose can guarantee the order of start, but can not guarantee the right to start the next one

@liusongsen could you try the deploy script http://shipyard-project.com/docs/deploy/ and see if that works?

@ehazlett
I just wrote it in this document, but I just added those commands to the docker-compose.yml, where there are more containers, I don't have a start, and all I do is a project
Https://github.com/liusongsen/php-docker-machine

@ehazlett
I would like to join this project in shipyard, thank you for your patience.

Have you checked the routes and firewall on the instance?

The same happens to me as well by running curl -sSL https://shipyard-project.com/deploy | bash -s with Docker version 17.03.0-ce, build 60ccb22
shipyard-controller seems to be trying to connect to my host address instead of the rethinkdb link

Same. Mine is Docker version 17.05.0-ce, build 89658be + Ubuntu 16.04.2 LTS


Assertion error when running alexnet-dorefa

I simply copied your "alexnet-dorefa.py" file and tried to run the pretrained model(downloaded from your google drive):
./alexnet-dorefa.py --load alexnet-126.npy --run a.jpg --dorefa 1,2,6
where I substitute "a.jpg" to my testing picture(224*224) and the running result looks like this:
[0426 18:10:27 @sessinit.py:187] WRN Variable lastbn/variance/EMA:0 in the dict not found in the graph!
[0426 18:10:27 @sessinit.py:190] Restoring from dict ...
Traceback (most recent call last):
File "first_dorefa_alex.py", line 311, in
run_image(Model(), DictRestore(np.load(args.load, encoding='latin1').item()), args.run)
File "first_dorefa_alex.py", line 261, in run_image
predictor = OfflinePredictor(pred_config)
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\predict\base.py", line 147, in init
config.session_init.init(sess)
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\tfutils\sessinit.py", line 30, in init
self._init(sess)
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\tfutils\sessinit.py", line 34, in _init
self._run_init(sess)
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\tfutils\sessinit.py", line 191, in _run_init
upd.update({name: value for name, value in six.iteritems(self.prms) if name in intersect})
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\tfutils\varmanip.py", line 116, in update
SessionUpdate.load_value_to_var(v, value)
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\tfutils\varmanip.py", line 77, in load_value_to_var
"{}: {}!={}".format(name, varshape, val.shape)
AssertionError: fct/W: (4096, 1000)!=(512, 1000)

You're not using the correct "alexnet-126.npy" file.

You got the point, I mistook the resnet file as alexnet file... but it came another bug below, it happend as well when I tried to train the model:
[0426 23:18:57 @sessinit.py:190] Restoring from dict ...
[0426 23:18:58 @fs.py:99] WRN $TENSORPACK_DATASET not set, using C:\Users\Pengyu Li/tensorpack_data for dataset.
caffe.proto: 65.5KB [00:01, 43.4KB/s]
Succesfully downloaded caffe.proto 58196 bytes.
caffe.proto: No such file or directory
Traceback (most recent call last):
File "alexnet-dorefa.py", line 311, in
run_image(Model(), DictRestore(np.load(args.load, encoding='latin1').item()), args.run)
File "alexnet-dorefa.py", line 262, in run_image
meta = dataset.ILSVRCMeta()
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\dataflow\dataset\ilsvrc.py", line 33, in init
self.caffepb = get_caffe_pb()
File "D:\Software\Anaconda\envs\py35\lib\site-packages\tensorpack\utils\loadcaffe.py", line 128, in get_caffe_pb
"Command protoc caffe.proto --python_out . failed!"
AssertionError: Command protoc caffe.proto --python_out . failed!

You have to have protoc to run anything related to imagenet.
I don't know how to install it on windows.

Cheers! I figured it out, after manually moving some files from one to another. Appreciation. 👍
BTW, if I want to use my own dataset, the path should look like:
PATH/
train/
n02134418/
n02134418_198.JPEG
...
...
val/
ILSVRC2012_val_00000001.JPEG
...
where is the label input to your model? If I have all pictures information contained in a file with format like:
pic1.path label1
pic2.path label2
....
any suggestions for adapting my dataset to your model?

You need to write a DataFlow to handle your own data format. See documentation

Feel free to reopen if you still have questions.


Refactor time range generation code

Codecov Report
Attention: Patch coverage is 82.06897% with 26 lines in your changes missing coverage. Please review.

Project coverage is 92.12%. Comparing base (fa87ad6) to head (9450410).

Files with missing lines
Patch %
Lines

src/chronify/time_range_generator_factory.py
50.00%
9 Missing :warning:

src/chronify/annual_time_range_generator.py
60.00%
6 Missing :warning:

src/chronify/index_time_range_generator.py
64.28%
5 Missing :warning:

src/chronify/time.py
70.00%
3 Missing :warning:

...rc/chronify/representative_time_range_generator.py
85.71%
1 Missing :warning:

src/chronify/time_configs.py
75.00%
1 Missing :warning:

tests/test_store.py
87.50%
1 Missing :warning:

Additional details and impacted files
@@               Coverage Diff               @@
##           ll/rep-time      #20      +/-   ##
===============================================
- Coverage        92.30%   92.12%   -0.18%
===============================================
  Files               23       28       +5
  Lines             1078     1143      +65
===============================================
+ Hits               995     1053      +58
- Misses              83       90       +7

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

All errors reported by mypy are in the time-mapping code, and so I'm assuming that @lixiliu will address them after we merge this to her branch.


[5.7] [Test Coverage] Encryption Provider

EncryptionServiceProvider class is completely uncovered by any tests in the Framework. This PR covers both flows of the provider with a simple and small test.

I think this PR will make PRs like https://github.com/laravel/framework/pull/25321 to start break. Haven't tested it yet as I'm on my phone.


Parameter tying

I am having difficulties with parameter tying. I have two separate kernels and I want them to share their lengthscales.
I tried
kern1.lengthscale = kern2.lengthscale however this is not allowed because the parameter tree already contains kern2.lengthscale https://github.com/GPflow/GPflow/blob/master/gpflow/param.py#L805
Help is much appreciated

realized that this is a duplicate: https://github.com/GPflow/GPflow/issues/322

The easiest way to do this is to write your own kernel which shares a Param object for the shared lengthscales, have the object compute the two kernels in separate functions. May need some tweaking of the models.

Thank you. I implemented as you said and it worked, although its not very elegant.
Maybe we should consider adding a class TiedParam(Param) that does not show up in the free variables and just refers to the tf array of the other Param that it is tied with. Additionally, we need to ensure that the other Param is in the same tree... Just speculating.

In the next version of GPflow (released next Friday), you can do a very elegant solution where you define a shared external tensor for two parameters. One may be able to even put this logic in a transform.


Does this plugin support fritzdect-200 power consumtion readout?

Hello,
i want to use the Fritz Dect 200 to messure my power consumption for diffrent setups.
at the moment i plan to use 3 devices.
https://en.avm.de/products/fritzdect/fritzdect-200/
does this plugin allow to grab the data of the devices?
with kind regards
PS: sorry for my bad english

Currently fritzcollectd has only rudimentary support of dect devices. Mainly because I don't have any DECT devices for testing here.
The following values are reported to collectd: multimeter power, multimeter energy, switch state and temperature. Currently they are reported as dect-0 ... dect-X, and it's not optimal to find out which is what.
Also there's currently no support for dect devices in my dashboard on grafana.com. But you can add that easily.

Hm .. that seems to be ok 👍
i only want to show how much energy consumtion each of the dect devices report maybe as graph so you can see where the peek times of powerconsumtion are.
if i donate you a dect 200 would that increase your support for that device?

I just noticed, the ordering of my devices changed after restart of collectd (dect4 is suddenly dect3). Maybe I will find some time to debug this further, or even implement some kind of name resolution.

I have a setup with multiple (>10) DECT200 boxes and they're mixed up on every reboot or restart which is quite annoying because you have to do the identification job again (which id is which box) and (even more important) because history data gets wrong when you correct the id-to-box assignment because in influxdb data is organised by the "1-dectX" id and this id is/might be re-assigned to a different device on restart/reboot. Would be great if that's possible to fix as it renders history data useless.

I've found a simple solution to the DECT200 devices getting mixed up on restart/reboot: I've changed the query that way that fritzcollect adds the AIN to the name instead of the id. This way the id can change (and will) but the values are always logged to the same series because the AIN is constant. I replaced line 80/81:
       (ServiceAction('X_AVM-DE_Homeauto:1', 'GetGenericDeviceInfos',                        'NewIndex', 'NewIndex', 'dect'),
with:
`        (ServiceAction('X_AVM-DE_Homeauto:1', 'GetGenericDeviceInfos',
'NewIndex', 'NewAIN', 'dect'),
and the instance names change to something like this "1-dectxxxxx yyyyyy" where xxxxx yyyyyy is the AIN which will never change for a certain DECT200 box.
`

I used 'NewDeviceName' instead of 'NewAIN', this way I can see the device names I have used for the DECT 200 devices.

I've found a simple solution to the DECT200 devices getting mixed up on restart/reboot: I've changed the query that way that fritzcollect adds the AIN to the name instead of the id. This way the id can change (and will) but the values are always logged to the same series because the AIN is constant. I replaced line 80/81:
 (ServiceAction('X_AVM-DE_Homeauto:1', 'GetGenericDeviceInfos', 'NewIndex', 'NewIndex', 'dect'),
with:
 (ServiceAction('X_AVM-DE_Homeauto:1', 'GetGenericDeviceInfos', 'NewIndex', 'NewAIN', 'dect'),
and the instance names change to something like this "1-dectxxxxx yyyyyy" where xxxxx yyyyyy is the AIN which will never change for a certain DECT200 box.

In wich file I have to edit the code or do I need to reinstall collectd and edit the "init.py"?

Just check where fritzcollectd ended up being installed using "pip show fritzcollectd". I used a patch like this for my modification:
diff -u __init__.py.orig __init__.py
--- __init__.py.orig    2020-04-09 17:49:08.000000000 +0200
+++ __init__.py 2020-04-09 18:01:45.000000000 +0200
@@ -78,7 +78,7 @@
           Value('linkdownstreammax', 'bitrate'),
           'NewLayer1UpstreamMaxBitRate': Value('linkupstreammax', 'bitrate')}),
         (ServiceAction('X_AVM-DE_Homeauto:1', 'GetGenericDeviceInfos',
-                       'NewIndex', 'NewIndex', 'dect'),
+                       'NewIndex', 'NewDeviceName', ''),
          {'NewMultimeterPower': Value('power', 'power'),
           'NewMultimeterEnergy': Value('energy', 'power'),
           'NewTemperatureCelsius': Value('temperature', 'temperature'),

Just check where fritzcollectd ended up being installed using "pip show fritzcollectd". I used a patch like this for my modification:
diff -u __init__.py.orig __init__.py
--- __init__.py.orig    2020-04-09 17:49:08.000000000 +0200
+++ __init__.py 2020-04-09 18:01:45.000000000 +0200
@@ -78,7 +78,7 @@
           Value('linkdownstreammax', 'bitrate'),
           'NewLayer1UpstreamMaxBitRate': Value('linkupstreammax', 'bitrate')}),
         (ServiceAction('X_AVM-DE_Homeauto:1', 'GetGenericDeviceInfos',
-                       'NewIndex', 'NewIndex', 'dect'),
+                       'NewIndex', 'NewDeviceName', ''),
          {'NewMultimeterPower': Value('power', 'power'),
           'NewMultimeterEnergy': Value('energy', 'power'),
           'NewTemperatureCelsius': Value('temperature', 'temperature'),

Thanks for your fast response Jum. With "pip show fritzcollectd" I figured out where fritzcollectd is saved. I have no clue how to write a patch. So I changed the "init.py" directly with nano. But now I don't get any Data anymore.
Cancel the change don’t bring it back to work, "sudo apt-get update && sudo apt-get -y upgrade" just like that.
So unfortunately, I think I have to reinstall my system.

I would say you changed the white space while editing the python code, and python is very sensitive to white space changes (it is part of the language's syntax). I would try to move the backup copy of that file (before you started editing) in place and try again, really obeying python syntax rules. If you do not have any backup of the file from before editing, you will have to use:
pip install --force-reinstall fritzcollectd
To get back a clean install dir.

Now I learned how to make updates with a patch. A Really useful thing, thank you.
But with your diff it won't work nevertheless.
I think it should be
+                       'NewIndex', 'NewDeviceName', 'dect'),
instead of
+                       'NewIndex', 'NewDeviceName', ''), ?
I changed it that way and now it continues collecting data.
In influx/grafana the instance = 1-NameOfMyDect200 apears.
But the values are still saved to instance = 1-dect0 like before !?

Now I learned how to make updates with a patch. A Really useful thing, thank you.
But with your diff it won't work nevertheless.
I think it should be
+ 'NewIndex', 'NewDeviceName', 'dect'),
instead of
+ 'NewIndex', 'NewDeviceName', ''), ?
I changed it that way and now it continues collecting data.
In influx/grafana the instance = 1-NameOfMyDect200 apears.
But the values are still saved to instance = 1-dect0 like before !?

This is exactly the reason why I made that 'dect' string the empty string, I just wanted to use my own chosen name. I have not researched more into the 'NewIndex' thing, as this causes the name to be '1-myName', but this was not really a concern to me as I could easily filter that using a regular expression in Grafana/Influx, where I view my collectd data.

And by the way, could someone tell me in wich units the values are ?
Energy = W and Power = Wh ? or is it kW ?

I used in my graphs for power the label W, and it corresponded pretty much to what was going through when I turned on the electric kettle. I did not graph energy, I believed it to be not important for my usage case.

Now I learned how to make updates with a patch. A Really useful thing, thank you.
But with your diff it won't work nevertheless.
I think it should be
+ 'NewIndex', 'NewDeviceName', 'dect'),
instead of
+ 'NewIndex', 'NewDeviceName', ''), ?
I changed it that way and now it continues collecting data.
In influx/grafana the instance = 1-NameOfMyDect200 apears.
But the values are still saved to instance = 1-dect0 like before !?

This is exactly the reason why I made that 'dect' string the empty string, I just wanted to use my own chosen name. I have not researched more into the 'NewIndex' thing, as this causes the name to be '1-myName', but this was not really a concern to me as I could easily filter that using a regular expression in Grafana/Influx, where I view my collectd data.

You are right, I forgot to reboot. Now everything works the way I want.
Thank you for your help.

Thanks for the patch. I had also the problem, that the dect numbers changes during night when my FritzBox gets a new DSL connection. I'm very happy to have my own assigned names now in Grafana.
Can this change be made officially in github, @fetzerch ?
I monitor temperature, power consumption and switch state in an own Grafana dashboard. I have also plugged in 2 solar modules into a FritzDect200, so I can see how much power they produce.

Hi @UweBeckert, I also want to do this. Can you please provide your Grafana dashboard :-) Thank you!


Feature/#45 add rating modal to confirm page

Description
Added rating modal to confirm page
fixes # (issue)
#45

There is a detail with the texts in mobile mode. I could not modify the Text component.


Why don't you compile this amazing piece of software for Windows?

I mean, look at the percentage of people that use Windows. If you're gonna use that obscure C:\Windows\System32\wsl.exe, then people also need the VM option on their BIOSes enabled and most people don't know that that even exists. At least compile it for Windows running on AMD 64 bits. That's the platform I use. You'dn't've done that (you'dn't've is an actual word meaning "you wouldn't have"), would you? And I know it may be difficult to do it, because MinGW is outdated, but you could use Visual Studio to compile it since you got C++20 and C17, which suits your needs.

Duplicate of https://github.com/emojicode/emojicode/issues/10


Support async assertions

This is supported by supertest
request.get('/foo').assert(res => myPromiseAssertion(res))

Should be a simple change to something like:
'use strict';

const assert = require('assert');

module.exports = {
  assert: function (fn) {
    return this.then(function (res) {
      return Promise.resolve(fn(res))
        .then(value => assert(value))
        .then(() => res);
    });
  }
};

Currently you can do:
request.get('/foo')
  .then(res => { assert(myPromiseAssertion(res); return res })

Is that sufficient? I planned assertions to be more of a quick check. I could add a .run that doesn't touch the return value if you'd like.

Actually a .run would suit my case much better. I'm normally running some assertions in a function that may throw errors (sometimes async assertions), but don't have a truthy return value so I have to do things like this:
    await request
      .get('/analytics/socials')
      .assert(res => {
        Joi.assert(res.body.data, Joi.array().items({
          posts: Joi.number().integer(),
        }));
        return true;
      });

    await request
      .get('/analytics/socials')
      .run(res => {
        Joi.assert(res.body.data, Joi.array().items({
          posts: Joi.number().integer(),
        }));
      });

I appreciate I can just use the return value of the request, but it is nice and clean. When running a series of requests its nice and easy to see the assertions attached to the request.

I figured as much. I'll make that mixin later today when I get a chance, it shouldn't be too hard.


Enhancement - JSON.ArraySlice

Added function written by Steve Allen at RCC (with his permission) to return a slice of a JSON array including tests.
JSONAdditions.fmp12.zip

I didn't realize there was a GitHub repo for this or I would have submitted it myself.  Thanks, @mw777rcc ! :)

Ok, I've forked the project and updated all the docs and the file itself, in case a Pull Request would be welcome.  (The contributing guide says, "No pull requests", so I'll not create one unless you want me to, @toddgeist .)

Hi Steve and Mike.
Thanks for the contribution
The reason for no pull requests is that we can't actually merge the FileMaker file.  So it doesn't work very well.  So the method we use is it to generate the docs from the comments in the test file.
@mw777rcc file has everything we need to  merge this.


Order reference not in sync with order ID

Expected behavior
Order reference should have the same number as Order ID

Actual behavior
Order reference has an different number after updating than order ID
Steps to reproduce the behavior
Upgrade mollie to the latest version.
Environment
Prestashop <IP_ADDRESS>

Mollie module version: 4.3.2
PHP version: 7.2.34

Hello @Maikel726 ,
The Order reference and Order ID should not be the same according to the module, they are generated randomly, unless there is some custom logic in your shop to make them the same? If yes, we would need your credentials sent to -<EMAIL_ADDRESS>for further investigation.
Looking forward to your reply!
--
Best regards,
Invertus Support team

Hello,
We used custom code in order.php but that doesn't work anymore.

Hey @Maikel726 ,
In that case we would need your BO + FTP credentials to investigate further, as from the given information no relevant changes or suggestions could be made. If possible, please send them to -<EMAIL_ADDRESS>for further investigation!
Looking forward to your reply!
--
Best regards,
Invertus Support team

Hi @Maikel726 ,
As we've discussed this topic via email, should we close this issue for now?
Looking forward to your reply!
--
Best regards,
Invertus Support team

Hi,
You can close this topic for now.


Upgrade to RC2?

Is there any plans to upgrade this seed project to RC2 in the near future? Simply changing angular dependency versions from 2.0.0-rc.1 to 2.0.0-rc.2 gives me an UNMET PEER DEPENDENCY error form core and http dependencies.

thanks, updated to rc.2

@gdi2290 did you push the changes? Master still seem to point to rc.1: https://github.com/angular/angular2-seed/blob/master/package.json#L36-L43

It does not appear so. I think it is wise to upgrade to RC3 directly now.

I'll update this


wtx bench does not validate utf8 text

Hi, Author of fastwebsockets here. I was reading your blog post on wtx, nice work!
I noticed that the wtx bench does not validate if the text is valid UTF8 while fastwebsockets and tungstenite do that - which is a pretty big overhead for large text payloads.
Ref https://github.com/c410-f3r/regular-crates/blob/main/wtx/examples/common/mod.rs#L66
https://github.com/c410-f3r/regular-crates/blob/main/wtx/src/web_socket.rs#L136

Hello @littledivy, nice to meet you!
Text payloads have their UTF-8 contents verified when read from a stream. Let's enumerate all possible situations:

Unique FIN frame (No decompression): Whole payload is verified at https://github.com/c410-f3r/wtx/blob/c46ef8a0c01e52706ebdd8e729b8dab98564b7c7/wtx/src/web_socket.rs#L948-L950
Continuation frames (No decompression): UTF-8 continuation is verified with a in-house utf8-crate-like algorithm (https://github.com/c410-f3r/wtx/blob/main/wtx/src/misc/incomplete_utf8_char.rs). First frame is verified at https://github.com/c410-f3r/wtx/blob/c46ef8a0c01e52706ebdd8e729b8dab98564b7c7/wtx/src/web_socket.rs#L208 and the remaining frames are verified at https://github.com/c410-f3r/wtx/blob/c46ef8a0c01e52706ebdd8e729b8dab98564b7c7/wtx/src/web_socket.rs#L770-L799
Unique FIN frame (With decompression): Whole payload is verified at https://github.com/c410-f3r/wtx/blob/c46ef8a0c01e52706ebdd8e729b8dab98564b7c7/wtx/src/web_socket.rs#L948-L950
Continuation frames (With decompression): For some reason, it is not possible to decompress and verify fragmented texts so each fragment is concatenated and the whole payload is verified at https://github.com/c410-f3r/wtx/blob/c46ef8a0c01e52706ebdd8e729b8dab98564b7c7/wtx/src/web_socket.rs#L195
Control frames: They aren't supposed to be compressed and the raw bytes are verified at https://github.com/c410-f3r/wtx/blob/c46ef8a0c01e52706ebdd8e729b8dab98564b7c7/wtx/src/web_socket.rs#L716

Autobahn tests are available at https://c410-f3r.github.io/wtx/ and benchmarks at https://github.com/c410-f3r/wtx/blob/main/wtx-bench/src/main.rs. Let me know if there are more questions and I will happily answer them.

Ah, I was looking at the wrong repository (regular-crates/wtx instead of wtx), my bad. Thank you!


file_ops: Use tmp file for original linting

Short description of the problem this fixes or functionality that this introduces. This may be used for the CHANGELOG
Fix a potential issue that might lead to file corruption when edit linting is enabled

Give a summary of what the PR does, explaining any non-trivial design decisions
#3124 introduces a feature for editing: running linter twice before and after the change and only extract new errors introduced by the agent. This has some potential issues and I am working on #3649 to address them, but I feel like I am not gonna finish it in the next few days, and that PR has become harder and harder to review, thus this PR, which only focuses on a small improvement.
So what's the issue? When we run linters on the original file before our edits, we need to copy the original file and use a temporary file to lint, because linting may have side-effect (e.g. modifying the file in-place). I used the word "may" because:

Flake8 has no side-effect, so not a problem as of now.
We don't enforce this or document this "no side-effect" as a requirement for linter implementation, so side-effect is allowed.

Regardless, the "after-edit-linting" uses the same approach: backup the file before linting to avoid data corruption. We should keep our "before-edit-linting" consistent.
Why no new unittest that reproduces the issue? Well, as I have mentioned earlier, flake8 has no side-effect, so technically it's not a bug but a flaw. Therefore, there's no way to write a test that reproduces the issue.

Are there any linters that rely on relative paths between files that may be disrupted by this change?

No, not that I am aware of, and it wouldn't make sense for single-file linters (our linter interface doesn't allow multi-file linting - even if it does, I am not aware of any linter that leverage relationships between files' relative paths).
Borderline is this PR only changes the "pre-edit-linting" behavior, meaning that it's okay to miss potential linting errors (in other words, false negatives are acceptable).


Adds instructions for running the tests for the first time

Just a few comments. Thanks for contributing! :smile:

Closing this due to inactivity.


Add numerical and logarithmic x-axis for stacked-area charts

Adds the functionality to use numerical values as x-values to plot, instead of only dates - this can be achieved by setting .xAxisValueType('number') - the default behaviour will be the same (so we introduce no breaking API changes!).
The user can use the same method for the tooltip to change the formatting of the x-values in the tooltip-title.
Additionally you can modify the area-chart with .xAxisScale('logarithmic') to show a logarithmic scaled axis.
How Has This Been Tested?

Test when xAxisValueType is set to numeric that the x-Axis ticks are not dates anymore but numbers
Test when xAxisValueType is set to numeric and xAxisScale to logarithmic, that the x-Axis ticks are in the format 10^x and not dates anymore
Test the getter / setter for xAxisValueType and xAxisScale for the stacked-area chart
Test the getter / setter for xAxisValueType for the tooltip
Test when xAxisValueType is set to numeric for the tooltip, no dates are shown in the title, but the numerical value

Screenshots (if appropriate):

Types of changes

[ ] Refactor (changes the way we code something without changing its functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

I did not want to introduce any breaking change in the API, so I did not refactor the variable-namings, etc. too much. This means that it might be strange now if you have variables with date in it but the value of the variable does not need to be a date anymore.
For me at least it's now consistent to the other chart-types and the refactoring of namings, etc. may be done in a bigger MR when rolling out these features for all charts?
Checklist:

[x] Latest master code has been merged into this branch
[x] No commented out code (if required, place // TODO above with explanation)
[ ] No linting issues
[x] Build is successful
[x] Code follows the API Guidelines
[ ] Updated the documentation
[x] Added tests to cover changes
[x] All new and existing tests passed

Thanks for the review @DalerAsrorov !
I've fixed the issues mentioned by you and added some examples for the docs as well.

@DalerAsrorov
Added the same functionality for line-charts as well.
So you might only merge this one:
https://github.com/eventbrite/britecharts/pull/806

@Golodhros I went for the other PR, lets close this one.