deploy: add example daemon set deployment

Users have requested for DaemonSet based deployments of Kong Ingress Controller.

Even in our case we would like to have a daemonset based deployment.
Usecase: We have k8s cluster with 3 nodes deployed over openstack. Currently our environment does not support Loadbalancer type services. We can go for nodeport/hostnetwork, but that means we need to ensure that the services are deployed on all the nodes. We have a LB at the front that does a TCP forwarding to these hosts. Hence if the pod is present on each nodes, then we can make this work.
We did a similar approach for nginx ingress and it worked fine. Hoping to have the same for kong soon.

Even in our case we would like to have a daemonset based deployment.
Usecase: We have k8s cluster with 3 nodes deployed over openstack. Currently our environment does not support Loadbalancer type services. We can go for nodeport/hostnetwork, but that means we need to ensure that the services are deployed on all the nodes. We have a LB at the front that does a TCP forwarding to these hosts. Hence if the pod is present on each nodes, then we can make this work.
We did a similar approach for nginx ingress and it worked fine. Hoping to have the same for kong soon.

@Arjunkrisha Would you be open to contributing one?
It is not terribly complicated to take the existing Deployment manfiest and make a DaemonSet out of it.

@Arjunkrisha Would you be open to contributing one?
It is not terribly complicated to take the existing Deployment manfiest and make a DaemonSet out of it.

@Arjunkrisha I would recommend subscribing to https://github.com/Kong/kubernetes-ingress-controller/issues/702 as this work will structurally change the way the controller and proxies are deployed in such a way that I expect will make implementing the proxy daemonset option you're requesting here simpler.
In the meantime while waiting for the daemonset option I wanted to suggest a 3rd party open source tool which may be able to help you get some of the behavior you want, called MetalLB. I noticed you mentioned your current environment "does not support LoadBalancer type services", and MetalLB is a tool which can enable LoadBalancer type Services for you in on-prem and air-gapped environments, or otherwise in environments where cloud or other application load balancers implementations are not available to your cluster. If this sounds like something that might help you MetalLB has documentation site and a community mailing list where you can get help setting it up in your environment.

@Arjunkrisha I would recommend subscribing to https://github.com/Kong/kubernetes-ingress-controller/issues/702 as this work will structurally change the way the controller and proxies are deployed in such a way that I expect will make implementing the proxy daemonset option you're requesting here simpler.
In the meantime while waiting for the daemonset option I wanted to suggest a 3rd party open source tool which may be able to help you get some of the behavior you want, called MetalLB. I noticed you mentioned your current environment "does not support LoadBalancer type services", and MetalLB is a tool which can enable LoadBalancer type Services for you in on-prem and air-gapped environments, or otherwise in environments where cloud or other application load balancers implementations are not available to your cluster. If this sounds like something that might help you MetalLB has documentation site and a community mailing list where you can get help setting it up in your environment.

@Arjunkrisha Would you be open to contributing one?
It is not terribly complicated to take the existing Deployment manfiest and make a DaemonSet out of it.

Sure will try that :)

@Arjunkrisha Would you be open to contributing one?
It is not terribly complicated to take the existing Deployment manfiest and make a DaemonSet out of it.

Sure will try that :)

Thanks @shaneutt for the suggestion.
our k8s team is having that as an option as well. But for now to get this API gateway going we need to do it without the metallb or loadbalancer option.
Planning to modify the helm chart locally and test for daemonsets. Let me know if i need to keep anything in mind while doing this, so that i can get it working as a daemonset

Thanks @shaneutt for the suggestion.
our k8s team is having that as an option as well. But for now to get this API gateway going we need to do it without the metallb or loadbalancer option.
Planning to modify the helm chart locally and test for daemonsets. Let me know if i need to keep anything in mind while doing this, so that i can get it working as a daemonset

Hey Guys. I made the changes locally. But confused on how to create a pull request after seeing the https://github.com/Kong/charts/blob/main/CONTRIBUTING.md.
I made changes in my local branch latest. I did a git rebase next. How to create a pull request now ?

Hey Guys. I made the changes locally. But confused on how to create a pull request after seeing the https://github.com/Kong/charts/blob/main/CONTRIBUTING.md.
I made changes in my local branch latest. I did a git rebase next. How to create a pull request now ?

For some reason i could not push my branch. Any thoughts ?
(base) ARJSRIRA-M-L0XP:charts arjsrira$ git push --set-upstream origin latest
Username for 'https://github.com': arjunkrishnasb
Password for<EMAIL_ADDRESS>remote: Permission to Kong/charts.git denied to arjunkrishnasb.
fatal: unable to access 'https://github.com/Kong/charts.git/': The requested URL returned error: 403
(base) ARJSRIRA-M-L0XP:charts arjsrira$

Hey guys, let me know if you guys know what the issue is. I would love to contribute more. Though this is a minor feature addition in deployment i think this would help many :)

Hey @arjunkrishnasb have you tried setting your fork of the repo as the upstream (instead of this repo)? You can then start a PR (or draft PR if you prefer) from your fork and we can help review and provide help.

I guess you mean fork the repo and create a PR in it and then you can update in the main repo ?
Will do that today and if anything else let me know

Kindly have a look and approve if this looks good https://github.com/Kong/charts/pull/269

I tried to agree the contributer license , but it gave an internal server error

It looks like the CLA issue may be coming from the submission author info:
arjsrira seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Perhaps there's some issue with the email address used for the commits vs. what's being used to sign the CLA?

Thanks @shaneutt
All tests passed. Just need to review

when this can be merged?

Just wonder when will we have this feature ?

There is now daemonset support in the chart as of https://github.com/Kong/charts/pull/347 which was the follow up to https://github.com/Kong/charts/pull/269. Let us know if there's anything further you wish to see in this regard.


Update Driver Feedback based on Note Manager States

In Ready Mode State solid green (Launcher Mode) or white (Placer Mode)
In feeder intake start blinking the color of the mode above, once one Intake sensor has tripped, start flashing yellow (Placer mode) or purple (Launcher Mode)
Hold Placer (Solid Yellow) Hold Feeder (Solid Purple)
If in Ready to Auto Launch or Auto Launch state  launch AND angle/launcher is AtTarget() (Update AtTarget method to check if the wheels above speed and angle is within 0.25 degrees) then set the LEDs to rainbow
If in prepare to Place Amp or Trap and within target switch to rainbow


Use ANSIBLE_CONFIG, use ansible.cfg file for all static variables

Fixes #93
Currently ansible.cfg only gets used in our integration tests and does not get packaged up in a distribution.
Moved ansible.cfg to a common place and use an ENV var to point to the file whenever ansible is executed both during integration tests and using the binary.

@dkoshkin Should we merge this?


Badges (#53)

Right, I've added badges support by doing the following:

The JavaScript API now returns an extra property in the object, badge, which has four of its own properties:

type, which is either 'ok' or 'fail' at the moment
svg, just a link to either the pass or fail SVG
markdown, a link to the Forkability page and the image in markdown
html, a link to the Forkability page and the image in html

The type of badge is determined by the pass:failure ratio. If it's 90% or more (e.g. 9 passes, 1 failure or 10 passes, 1 failure, or 11 passes, 1 failure or 23 passes, 2 failures) it counts as a pass or 'ok'. Anything below 90% is a fail, and they won't even be given a badge.
The pages version now will include some text encouraging the user to "wear their badge with pride", and shows the badge in a little area of its own to highlight it, as well as giving them both the markdown and HTML (in that order) in textareas whose contents can be selected using little "(select all)" links (I understand many people dislike select-on-click textareas)

So what I'd like to know from anybody reading this (mainly @M-Zuber I'm guessing) is some opinions about:
a) The new addition to the callback response in the API (e.g., what do we think of "ok" instead of "pass" because "pass" sounds a little too opinionated esp. considering 90% is OK)
b) The copy & layout on the page
c) Whether the SVG, markdown and HTML should be included in the command line API.
d) Should we try to encourage the user to pass by showing them what their badge could look like, or perhaps showing them a failure badge?
I shall eagerly await some input...!

The code looks nice and simple, and easily refactorable to allow for more then two result types.
I'll let you know what I think of the ui once I get a chance to see it :wink:

Cool. I might just go ahead and do it later today (it's morning now where I am in South Korea) if you're busy at the mo. Your opinion would be helpful but we can always change the UI a little bit later!

I realized that I didn't actually respond to most of your points so here goes:
a) ok does sound more neutral, but I think that pass works well enough. Another option is to use forkable :wink:
b) lets wait and see :grin:
c) it would be nice to at least have an option to get the badge from the CL

this could allow easy integration of forkablity into CI builds, that probably do not want to fail if it is not forkable, but on success might want to ensure that the badge is present.
(more flags anyone? [I have to find it, but I know a lib exists to make working with CL flags easier])
d) I would either show nothing (as you do now) or a failure badge. We don't want someone misunderstanding and putting in a pass badge inappropriately

a) Ah so we could just write "forkable" in the code?! Hmm. Yeah I guess so. Awfully domain-specific though. If we decide to change the name to something else (not sure how that would ever be possible since it's the best name ever :stuck_out_tongue_winking_eye: ) it'd cause problems. Gonna stick with 'ok'
b) Cool
c) Very good point re: CI builds. I'll add it to the CL response. Just after I've finished watching Mr Robot. The flag library in Forkability is fine too, have a look at this line, and this line for examples. Easy to add new flags/options.
Excellent answers, thank you.

AH I ACCIDENTALLY CLOSED.

A) forkable is def the best name evah, but let's leave it it as you said
C) I believe what I saw was commander :wink:. Also Mr. Robot is def more important

Any reason not to merge this?

Yes! Just wanted to put a flag in to show the badge in the command line,
but then I'll merge. I'm gonna get onto that tonight :) (it's only 7:45
where I am). I've finished all of Mr Robot I could get my hands on so
there's nothing else to do, really...
--Dan
danhough.com http://www.danhough.com - @basicallydan
http://twitter.com/basicallydan
On 26 August 2015 at 19:43, Mordechai Zuber<EMAIL_ADDRESS>wrote:

Any reason not to merge this?
—
Reply to this email directly or view it on GitHub
https://github.com/basicallydan/forkability/pull/56#issuecomment-134941971
.

No pressure, I am just trying to clean up the issues list :)

I like it! Won't be long.
--Dan
danhough.com http://www.danhough.com - @basicallydan
http://twitter.com/basicallydan
On 26 August 2015 at 19:53, Mordechai Zuber<EMAIL_ADDRESS>wrote:

No pressure, I am just trying to clean up the issues list :)
—
Reply to this email directly or view it on GitHub
https://github.com/basicallydan/forkability/pull/56#issuecomment-134946617
.

So I've made it so that the badge output is very, very clearly a "failure" if it is one by making the text red and putting the word "failure". Let's put in a badge if anybody asks for it.
As well as that I've made the output a little more readable and useful. Have a look!

Sample output (from forkability)
# Forkability found 8 recommended features, and has 0 suggestions

# Features (8)
✓ Contributing document
✓ Readme document
✓ License document
✓ Changelog document
✓ .gitignore file
✓ Test suite
✓ All open issues have been acknowledged
✓ Tags are being used

---

# Suggestions (none)

---

# Forkability Badge (success)
## Just the SVG:
https://img.shields.io/badge/forkable-yes-brightgreen.svg
## Markdown:
[![This is a forkable respository](https://img.shields.io/badge/forkable-yes-brightgreen.svg)](https://basicallydan.github.io/forkability/?u=basicallydan&r=forkability)
## HTML:

Pretty sure you'll be cool with this ^^

:ship: :+1:
Great!!
Thank you.
There is still a question open for you at https://github.com/badges/shields/issues/502

https://basicallydan.github.io/forkability/?u=basicallydan&r=forkability


Remove dependency on System.ValueTuple NuGet package when targeting .Net framework 4.7

This is to avoid having to add unnecessary dependencies in consuming projects.

@onovotny, is this something that still needs attention?
This may bring a little pickle when we add overloads for e.g. Zip that use ValueTuple<...> to be symmetric with newer extension methods added to IEnumerable<T>. In particular:
public static IObservable<(TFirst First, TSecond Second)> Zip<TFirst, TSecond>(this IObservable<TFirst> first, IObservable<TSecond> second);

It's probably worth adding another build target for the first version of .NET that has ValueTuple in it.

I third this.

Are there any plans to move forward on this? Is there a good reason to keep targeting net46 when System.Reactive requires System.ValueTuple anyway? In that case might as well just target net472.

I guess another way to look at it is to consider the netstandard2.0 target as the compatibility layer for net47 applications (since netstandard2.0 does include System.ValueTuple). In that case, maybe a possible "resolution" for the issue is to simply remove the net46 target?

@clairernovotny, let's address this before we hit 5.0 RTM. I'm happy to drop 4.6 and either add 4.7.2, or just converge on .netstandard2.0 for downlevel support.

I think we can set net472 as the new min esp since it's a major version bump. Having a net472 tfm is good for two reasons:

Reduces binding redirect issues with netstandard libraries
Has our WinForms and WPF support

This has been addressed now and will be in Rx 5.0.


How to set vina score to dock complex with metals?

I heard that there is some sf than can be used in metal ions(Mg,Se,Zn...) docking in AutoDock Vina, But I couldn't find it in manual.
Is there anything to do with  v = Vina(sf_name='vina')?

The vina scoring function, which is the default (e.g., v = Vina()), handles metals. There's zinc specific potentials but that is for the autodock4 scoring function.

Though the default scoring function can deal with metals, the protein.pdb cannot be converted into pdbqt file if they have Zn,Mg...
for example, 1ew9_pocket.pdb consists of Zn and Mg, and the Error is like:
/ADFRsuite_x86_64Linux_1.0/bin/prepare_receptor -r 1ew9_pocket.pdb -A hydrogens -o 1ew9_pocket.pdbqt
'  ' apparently composed of not std residues. Deleting
Traceback (most recent call last):
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/Utilities24/prepare_receptor4.py", line 216, in <module>
    dict=dictionary)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 558, in __init__
    version=version, delete_single_nonstd_residues=delete_single_nonstd_residues)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 124, in __init__
    self.repairMol(mol, self.repair_type_list)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 174, in repairMol
    self.newHs = self.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 187, in addHydrogens
    HB.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/hydrogenBuilder.py", line 61, in addHydrogens
    babel.assignHybridization(mol.allAtoms)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 136, in assignHybridization
    self.valence_three()
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 236, in valence_three
    elif self.count_free_ox(a) >= 2: a.babel_type="Cac"
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 167, in count_free_ox
    self.count_heavy_atoms(bonded_atom) == 1:
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 157, in count_heavy_atoms
    if bonded_atom.babel_type[0] == 'H': count = count + 1
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/molecule.py", line 409, in __getattr__
    raise AttributeError('member %s not found'%member)
AttributeError: member babel_type not found

Though the default scoring function can deal with metals, the protein.pdb cannot be converted into pdbqt file if they have Zn,Mg...
for example, 1ew9_pocket.pdb consists of Zn and Mg, and the Error is like:
/ADFRsuite_x86_64Linux_1.0/bin/prepare_receptor -r 1ew9_pocket.pdb -A hydrogens -o 1ew9_pocket.pdbqt
'  ' apparently composed of not std residues. Deleting
Traceback (most recent call last):
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/Utilities24/prepare_receptor4.py", line 216, in <module>
    dict=dictionary)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 558, in __init__
    version=version, delete_single_nonstd_residues=delete_single_nonstd_residues)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 124, in __init__
    self.repairMol(mol, self.repair_type_list)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 174, in repairMol
    self.newHs = self.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 187, in addHydrogens
    HB.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/hydrogenBuilder.py", line 61, in addHydrogens
    babel.assignHybridization(mol.allAtoms)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 136, in assignHybridization
    self.valence_three()
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 236, in valence_three
    elif self.count_free_ox(a) >= 2: a.babel_type="Cac"
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 167, in count_free_ox
    self.count_heavy_atoms(bonded_atom) == 1:
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 157, in count_heavy_atoms
    if bonded_atom.babel_type[0] == 'H': count = count + 1
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/molecule.py", line 409, in __getattr__
    raise AttributeError('member %s not found'%member)
AttributeError: member babel_type not found

If so, how can we use vina to dock protein with metals? We even cannot get their pdbqt files!

@diogomart  Dear developer, I met the same questions. When there is metal ions such as Mg, the ADFR can't convert it into pdbqt file.
So it is impossible to use vina to dock complex with metal.

Hi @Kerro-junior and @Dadiao-shuai,
You should be able to get passed the issue if you are not adding hydrogens in prepare_receptor. Please see an example here: https://github.com/ccsb-scripps/AutoDock-Vina/issues/241#issuecomment-1734266302
If you think your problem is structure-specific, it would be really helpful if you could your files so others can reproduce the error.


FYI guys I added some code pointers to the underlying https://github.com/actions/toolkit/issues/713#issuecomment-1030692990 - for me it seems passing an extra option to avoid matching directiories in the globber would solve this.

    FYI guys I added some code pointers to the underlying https://github.com/actions/toolkit/issues/713#issuecomment-1030692990 - for me it seems passing an extra option to avoid matching directiories in the globber would solve this.

Originally posted by @robinp in https://github.com/actions/cache/issues/494#issuecomment-1030693277

    FYI guys I added some code pointers to the underlying https://github.com/actions/toolkit/issues/713#issuecomment-1030692990 - for me it seems passing an extra option to avoid matching directiories in the globber would solve this.

Zitat von@robinpin#494 (Kommentar)


"Error: Param not a number: a,l" on SVG path

The path for which the error occurs is quite long. Here's a fiddle to replicate:
https://jsfiddle.net/87fjznso/1/
Error occurs in both 3.1.0 and current 4.2.3. I haven't tested other versions.

I don't know how you generated the path but it's kinda funky looking. The reason why it's failing is because the path has sections like this:
a931 931 0 00-1 138

as opposed to
a931 931 0 0 0 -1 138

Note that when roughjs parses it, it thinks the number is 00 which is just 0. And it thinks that there's a missing value.
Browsers can be more forgiving.
I can change rough to interpret 00 to 0 0  but there are too many cases to consider. I suggest if you change path  string so your zeros have a space after them.

Thanks, the path is initially from wikipedia, but minified with svgo's convertPathData plugin.

yes it must be because  of the minification.
I'm not sure I'm inclined to fix this unless you cannot update your path

I definetly can change the path, it's not an issue at all - I just though you might be interested in this kind of (pretty specific) behaviour.
Maybe the path parser used by Firefox or Chrome could be found in their source code.

This happend to me too while svg-icons conversion after upgraded from svgo 1.2.2 to latest 1.3, tough in browser icons work ok, but rough can not parse some paths.
I tried to compare paths and changes but it is really a mess. Not even sure how to describe this issue in svgo repo, besides. There are not issues with optimized icons in browser, only rough can't deal with them.

There is another more complex example of converted with svgo paths
1.2.2 (this works with rough)
M63.316 8.401l1.091-1.634 1.072-1.612 2.412-3.617A.968.968 0 1066.28.467l-2.584 3.878-1.087 1.63-1.104 1.652c-3.495-1.354-7.392-2.115-11.508-2.115-4.112 0-8.009.76-11.504 2.115l-1.098-1.652-1.087-1.63-2.588-3.88a.97.97 0 00-1.343-.27.968.968 0 00-.267 1.341l2.41 3.617 1.076 1.612L36.688 8.4c-8.21 3.823-13.763 11.066-13.763 19.356H77.08c0-8.288-5.553-15.53-13.763-19.354zM38.398 20.02a2.9 2.9 0 110-5.8 2.9 2.9 0 010 5.8zm23.206 0a2.9 2.9 0 010-5.8 2.9 2.9 0 110 5.8zM23.342 31.625h-.417v42.551a6.13 6.13 0 006.124 6.126h4.42a5.794 5.794 0 00-.23 1.61v12.253a5.803 5.803 0 0011.606 0V81.912c0-.556-.086-1.1-.23-1.61h10.774a5.794 5.794 0 00-.23 1.61v12.253c0 3.202 2.6 5.802 5.803 5.802 3.202 0 5.802-2.6 5.802-5.802V81.912c0-.556-.082-1.1-.23-1.61h4.42a6.132 6.132 0 006.125-6.125V31.625h-.417zM13.253 31.625A5.801 5.801 0 007.45 37.43v24.819a5.803 5.803 0 005.802 5.802c3.202 0 5.802-2.6 5.802-5.802v-24.82a5.8 5.8 0 00-5.802-5.803zM86.751 31.625a5.8 5.8 0 00-5.802 5.804v24.819a5.802 5.802 0 005.802 5.802c3.203 0 5.798-2.6 5.798-5.802v-24.82a5.799 5.799 0 00-5.798-5.803z
v 1.3.2 (this error with rough)
M63.316 8.401l1.091-1.634 1.072-1.612 2.412-3.617A.968.968 0 0 0 66.28.467l-2.584 3.878-1.087 1.63-1.104 1.652c-3.495-1.354-7.392-2.115-11.508-2.115-4.112 0-8.009.76-11.504 2.115l-1.098-1.652-1.087-1.63-2.588-3.88a.97.97 0 0 0-1.343-.27.968.968 0 0 0-.267 1.341l2.41 3.617 1.076 1.612L36.688 8.4c-8.21 3.823-13.763 11.066-13.763 19.356H77.08c0-8.288-5.553-15.53-13.763-19.354zM38.398 20.02a2.9 2.9 0 1 1 0-5.8 2.9 2.9 0 1 1 0 5.8zm23.206 0a2.9 2.9 0 0 1 0-5.8 2.9 2.9 0 0 1 0 5.8zM23.342 31.625h-.417v42.551a6.13 6.13 0 0 0 6.124 6.126h4.42a5.794 5.794 0 0 0-.23 1.61v12.253a5.803 5.803 0 0 0 11.606 0V81.912c0-.556-.086-1.1-.23-1.61h10.774a5.794 5.794 0 0 0-.23 1.61v12.253c0 3.202 2.6 5.802 5.803 5.802 3.202 0 5.802-2.6 5.802-5.802V81.912c0-.556-.082-1.1-.23-1.61h4.42a6.132 6.132 0 0 0 6.125-6.125V31.625h-.417zM13.253 31.625A5.801 5.801 0 0 0 7.45 37.43v24.819a5.803 5.803 0 0 0 5.802 5.802c3.202 0 5.802-2.6 5.802-5.802v-24.82a5.8 5.8 0 0 0-5.802-5.803zM86.751 31.625a5.8 5.8 0 0 0-5.802 5.804v24.819a5.802 5.802 0 0 0 5.802 5.802c3.203 0 5.798-2.6 5.798-5.802v-24.82a5.799 5.799 0 0 0-5.798-5.803z


Feature: add ability to configure y-Axis title

This PR adds the ability to configure the y-Axis title of a chart:

Merged. Thanks!


Animating scrollTop on scrollable elements

I mentioned earlier that using $(element).velocity('scroll', {container: viewElement}) is fine
Now use Velocity(Element, {})??
What can I do to achieve scrollTop

https://github.com/julianshapiro/velocity/wiki/Property---ScrollTop

No answer to his question there.

No answer to his question there.

If you don't think that pointing at the exact documentation that shows how to use scrollTop is not an answer, then you might be reading the wrong question...

Yeah obviously it's me...


Docker image pull retry

This change sets a default timeout of 300s for the image pull. If the
image pull times out (300s), we will retry 3 times by default.
fixes 1954

@jtaleric please, rebase/squash your commits, you shouldn't have 4 commits in this PR.

jtaleric  ~  Code  Work  ceph-ansible  ./generate_group_vars_sample.sh
 jtaleric  ~  Code  Work  ceph-ansible  echo $?
0
 jtaleric  ~  Code  Work  ceph-ansible  git diff
 jtaleric  ~  Code  Work  ceph-ansible 
^ I did run this... It doesn't seem to do anything.

Well, it certainly did something since you pushed changes with the right content :).

This failed build doesn't seem related to my change?

@jtaleric Could you squash your commis please ?

@leseb @guits any further comments? Sorry for all the mix ups.

@guits Hey - thanks for the review again. I moved the when to the end... Also, to point out there is another task after mine with the when not at the end... ¯_(ツ)_/¯


SiteExtension gallery support for SemVerv2

When the nuget client doesn’t correctly understand the new package versioning it shows older packages using the older versioning before the new ones. When the gallery only shows one version, it shows an old version.
These docs say you need NuGet 4.3.0+
https://docs.microsoft.com/en-us/nuget/reference/package-versioning#semantic-versioning-200

Example: https://www.nuget.org/packages/AspNetCoreRuntime.3.0.x64/3.0.0-preview6.19307.2

Add @sanchitmehta.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


PasteEmbed Validation Error

When using the Pasteembed plugin, the html returned for iframes is invalid due to a space in the width attribute.

The browser still renders it ok, but the space causes validation errors. Is this due to the api? Can be it fixed at all?
Thanks.

OK, for now I adjusted the code to use:
t.html = t.html.replace('width=" ','width="');
Seems to work.

Yeah, the inserted HTML comes from noembed.
You can submit your fix on the server-side here: https://github.com/leedo/noembed
But it seems that the project is discontinued :/
I will not base the v3 paste embed feature on it, and write it full client-side I think.

No problem Alex, thank you.


use custom sms provider #385

What kind of change does this PR introduce?
feature
What is the current behavior?
Using custom (Unlimited) SMS Provider for sending OTP
What is the new behavior?
There is a new optional boolean param named "send_otp" that controls whether the API sends the OTP code to the SMS Provider or not, since the default value is true it will not break the current behavior.
Current Behavior
curl -X POST 'https://tqfkjdmpohkqylveukuhsy.supabase.net/auth/v1/otp' \
-H "apikey: <SERVICE_KEY>" \
-H "Content-Type: application/json" \
-d '{
  "phone": "6212312312312"
}'
{}

With new Param
curl -X POST 'https://tqfkjdmpohkqylveukuhsy.supabase.net/auth/v1/otp' \
-H "apikey: <SERVICE_KEY>" \
-H "Content-Type: application/json" \
-d '{
  "phone": "6212312312312",
  "send_otp": false
}'
{"otp":"414995"}

(edited)
Recently I realized that the current behavior on "/otp" route is didn't require admin credentials, so adding send_otp parameter and returning the OTP code in that URL is not safe.
I decided to add a new route inside of "/otp" that is "/otp/phone" and use requireAdminCredentials on it, so the behavior of "/otp" is still the same like current behavior, and when we request to "/otp/phone" it need a SERVICE_ROLE_KEY
Current Behavior
curl -X POST 'https://tqfkjdmpohkqylveukuhsy.supabase.net/auth/v1/otp' \
-H "Content-Type: application/json" \
-d '{
  "phone": "6212312312312"
}'
{}

New Route
curl -X POST 'https://tqfkjdmpohkqylveukuhsy.supabase.net/auth/v1/otp/phone' \
-H "apikey: <SERVICE_KEY>" \
-H "Content-Type: application/json" \
-d '{
  "phone": "6212312312312", }'
{"otp":"414995"}

Honestly, I'm not sure about the route name, and request method.
Please let me know if you have feedback.

Hi @novanda1, on second thought, it will be much cleaner for us to modify the /admin/generate_link endpoint to one that takes in a phone field and type field. This would allow users to generate a link or otp for email or phone confirmations. The endpoint name will probably have to be renamed and we'll be discussing this internally to figure out what makes sense.


Adding Mi 8 Pro support

What Changed

Nothing. Just added support for Mi 8 Pro
Reason

To support more devices =D
Checklist

[x] Is what you changed Tested?
[x] Is the Source Code Cleaned?

Fixed


Update ChunkedStream.makeSubStream to actually check if (some) data exists when the length parameter is undefined

Note how XRef.fetchUncompressed, which is used a lot for most PDF documents, is calling the makeSubStream method without providing a length argument.
In practice this results in the makeSubStream method, on the ChunkedStream instance, calling the ensureRange method with NaN as the end position, thus resulting in no data being requested despite it possibly being necessary.
This may be quite bad, since in this particular case it will lead to a new ChunkedStream being created and also a new Parser/Lexer instance. Given that it's quite possible that even the very first Parser.getObj call could throw MissingDataException, this could thus lead to wasted time/resources (since re-parsing is necessary once the data finally arrives).
You obviously need to be very careful to not have ChunkedStream.makeSubStream accidentally requesting the entire file, hence its this.end property is of no use here, but it should be possible to at least check that the start of the data is present before any potentially expensive parsing occurs.

/botio test

/botio test

ในวันที่ ส. 30 มี.ค. 2019 02:56 pdf.js test bot<EMAIL_ADDRESS>เขียนว่า:

From: Bot.io (Windows)
Received
Command cmd_test from @Snuffleupagus https://github.com/Snuffleupagus
received. Current queue size: 0
Live output at: http://<IP_ADDRESS>:8877/1bb16f98e064373/output.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/pdf.js/pull/10696#issuecomment-478129788, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AuzaSI12BjrI1D67FtG78QJFulo0RZN6ks5vbm_XgaJpZM4cSwgz
.

Looking at the browsertest results it seems that the second patch may have reduced the runtime by approximately 1%, which is great considering just how simple it really is :-)

Thank you!


Error: Denial of Service

Error: Denial of Service
The version of libxml2 packaged with Nokogiri contains several vulnerabilities. Nokogiri has mitigated these issues by upgrading to libxml 2.9.5.
It was discovered that a type confusion error existed in libxml2. An attacker could use this to specially construct XML data that could cause a denial of service or possibly execute arbitrary code. (CVE-2017-0663)
It was discovered that libxml2 did not properly validate parsed entity references. An attacker could use this to specially construct XML data that could expose sensitive information. (CVE-2017-7375)
It was discovered that a buffer overflow existed in libxml2 when handling HTTP redirects. An attacker could use this to specially construct XML data that could cause a denial of service or possibly execute arbitrary code. (CVE-2017-7376)
Marcel Böhme and Van-Thuan Pham discovered a buffer overflow in libxml2 when handling elements. An attacker could use this to specially construct XML data that could cause a denial of service or possibly execute arbitrary code. (CVE-2017-9047)
Marcel Böhme and Van-Thuan Pham discovered a buffer overread in libxml2 when handling elements. An attacker could use this to specially construct XML data that could cause a denial of service. (CVE-2017-9048)
Marcel Böhme and Van-Thuan Pham discovered multiple buffer overreads in libxml2 when handling parameter-entity references. An attacker could use these to specially construct XML data that could cause a denial of service. (CVE-2017-9049, CVE-2017-9050)
Category description: Denial of Service is any attack which causes a service to become unavailable for legitimate clients.
Solution: upgrade nokogiri to >= 1.8.1

Hi @ecopes thanks for contributing these issues! The issues may have been fixed in the master branch- can you please take a look at https://github.com/helpyio/helpy and see if you see the same issue there and if so, re-post there.


Crash in otel_metrics.go

Using a current refinery, a user has reported a crash with fatal error: concurrent map read and map write. The stack trace points to github.com/honeycombio/libhoney-go@v1.20.0/transmission/transmission.go:228 which is a call to the metrics object. It appears that the conflicting call is related to an error returned from a transmission. This is the stack trace there:
net/http.(*persistConn).roundTrip(0xc4fcc64fc0, 0xc28887ea40)
	net/http/transport.go:2620 +0x974
net/http.(*Transport).roundTrip(0xc00053e140, 0xc2d8a92000)
	net/http/transport.go:595 +0x7ba
net/http.(*Transport).RoundTrip(0xc2d8a92000?, 0xecd920?)
	net/http/roundtrip.go:17 +0x19
net/http.send(0xc18ffbff00, {0xecd920, 0xc00053e140}, {0xd5d720?, 0x48b501?, 0x1439100?})
	net/http/client.go:251 +0x5f7
net/http.(*Client).send(0xc1482413e0, 0xc18ffbff00, {0x0?, 0xc09dcf5910?, 0x1439100?})
	net/http/client.go:175 +0x9b
net/http.(*Client).do(0xc1482413e0, 0xc18ffbff00)
	net/http/client.go:715 +0x8fc
net/http.(*Client).Do(...)
	net/http/client.go:581
github.com/honeycombio/libhoney-go/transmission.(*batchAgg).fireBatch(0xc288418800, {0xc521716000?, 0xed, 0x100})
	github.com/honeycombio/libhoney-go@v1.20.0/transmission/transmission.go:452 +0x916
github.com/honeycombio/libhoney-go/transmission.(*batchAgg).Fire(0xc288418800, {0x7fd13639e1b0?, 0xc000010120?})
	github.com/honeycombio/libhoney-go@v1.20.0/transmission/transmission.go:325 +0x109
created by github.com/facebookgo/muster.(*Client).worker.func1
	github.com/facebookgo/muster@v0.0.0-20150708232844-fd3d7953fd52/muster.go:133 +0xfb

I think this is probably relating to the map of Gauge metrics being both written by an error reporting function and read by the periodic metrics posting logic. But to date I have not been able to reproduce the error.
This is to document the error in case we see it again.

We haven't seen this lately, so we think it's been fixed. Please reopen if it recurs.


Wallet Settings

Proposed Changes
this feature is disconnecting and re-connecting a wallet.
wallet settings:

disconnect:

re-connect:

Implementation

add WalletSettings component
add disconnect method into wallet fixture

Perfect!!

Perfect!!


403 error when downloading BiliBili videos

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Version
> youtube-dl -U
Latest version: 2024.07.25, Current version: 2024.07.25
youtube-dl is up to date (2024.07.25)

Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.bilibili.com/video/BV11y411e7Lj/?p=4', '--cookie', '.\\cookie.txt', '--referer', 'https://www.bilibili.com/video/BV11y411e7Lj', '--verbose']
[debug] Encodings: locale cp936, fs mbcs, out cp936, pref cp936
[debug] youtube-dl version 2024.07.25 [e1b3fa242] (single file build)
[debug] ** This version was built from the latest master code at https://github.com/ytdl-org/youtube-dl.
[debug] ** For support, visit the main site.
[debug] Python 3.4.4 (CPython AMD64 32bit) - Windows-10-10.0.22621 - OpenSSL 1.0.2d 9 Jul 2015
[debug] exe versions: ffmpeg N-112922-gebfd3912e9-20231206, ffprobe N-112922-gebfd3912e9-20231206
[debug] Proxy map: {}
[BiliBili] 11y411e7Lj: Downloading webpage
[BiliBili] 11y411e7Lj: Downloading video info page
WARNING: Unable to download JSON metadata: HTTP Error 403: Forbidden
[BiliBili] 11y411e7Lj: Downloading video info page
ERROR: Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://github.com/ytdl-org/youtube-dl/issues , using the appropriate issue template. Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose option and include the complete output.
  File "D:\a\ytdl-nightly\ytdl-nightly\youtube_dl\extractor\common.py", line 679, in _request_webpage
  File "D:\a\ytdl-nightly\ytdl-nightly\youtube_dl\YoutubeDL.py", line 2500, in urlopen
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 470, in open
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 580, in http_response
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 502, in error
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 442, in _call_chain
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 685, in http_error_302
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 470, in open
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 580, in http_response
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 508, in error
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 442, in _call_chain
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 588, in http_error_default

Description
When downloading BiliBili videos, 403 errors keep appearing. Passing in cookies and setting referer still doesn't solve the problem.

Yes, this is the current manifestation of an old bug:

#32782
https://github.com/ytdl-org/youtube-dl/issues/32722#issuecomment-1950045012

The WIP extractor should be re-tested now.


Extreme slowness after recent update

Can you reproduce this issue on default Wordpress theme (eg Storefront)?
No (tested on new site with fresh woocommerce)
Can you reproduce this issue when all other plugins are disabled except WooCommerce, Polylang and Hyyan WooCommerce Polylang Integration?
Probably yes
What product versions and settings are you using when this issue occurs?

PHP: 7.1.7
WordPress: 4.9.6
WooCommerce: 3.4.3
Polylang: 2.3.7
Hyyan WooCommerce Polylang Integration: 1.2.0
Browser: All

Steps to Reproduce
have to many coupons in the database.
What I Expected
No problems
What Happened Instead
Site is extreme slow (>= 10 sec) in network admin and main site containing the WooCommerce after around 2-4 weeks around. (Frontend works fine -- 2 sec)
The XDebug profilling showed that around 93 % of time was spent in Hyyan\WPI\Coupon->adminRegisterCouponStrings down to registerCouponForTranslation and probably to reading to database.
Comment
Instantiating 2900 coupons after the same amount of queries to the database one-by-one is just ineffective and also we don't need translated coupons strings.
Is it a bug or feature?

You are welcome to turn off the Coupons module by unchecking Coupons sync under Settings, WooPoly, Features.
If you need to keep the coupon translated product rules and just turn off the coupon translation then you can comment out these lines in Coupon.php:
            add_filter('woocommerce_cart_totals_coupon_label',
                array($this, 'translateLabel'), 20, 2);
            add_filter('woocommerce_coupon_get_description',
                array($this, 'translateDescription'), 10, 2);

Do Note that this was improved since the first version of the coupons module, now the query is only done once and then cached in a transient coupons-lang  which is fairly efficient, depending also on how you do your transient caching.  (I recommend redis cache to take the load off the database, as woocommerce does everything with transients these days).
From your report - slow specifically in wp-admin - it sounds like the transient cache is not working in admin mode does that seem to be the case and what are you using for transient cache??

But the load should only happen once or once per language max then it’ll be in the transient cache - without redis etc this is just in wp options table same as all other transients
Or are you saying this only occurs on network admin, you are doing your testing there?
Actually I have seen other issues with network admin, that should probably be a separate issue, to ensure the plugin does not activate in network admin

Hmm, what is the thing that should be in the transient cache? In order to solve what this function does I would expect there to be a PHP serialization of all coupons, but I was unable to find any of it, what I see is only:
_transient_coupons-cs_CZ a:1231:{i:0;i:7846;i:1;i:7897;i:2;i:7903;i:3;i:782...
(However I can't imagine all objects serialized that would exceed the limitations of column text... and if they would be stored per one in transient I can't really imagine how would that help performance, because the queries would still needed to be done.)

Lets ignore the network admin for now, I am not sure about it and will investigate later.

just the coupon ids are stored in the transient because coupon details are stored in woocommerce transients already so overkill to store them again.. but yes if you have 2900 coupons and you think this is causing you a problem then just it off as I suggested.
Also if the business is big enough to have 2900 coupons and spending effort on debugging performance, then I would suggest the business should also be able to afford proper hosting with the capability to optimize the site further... I can't quite imagine why it would need 2900 coupons as yet, what's the use case here?

Oh, thanks for clarifying I see that it is as expected.
I would slightly disagree about your statement of the big business, we are quite small, however use coupons excesively, for example to create a campaign on discount portals, so we easily create hundreds of tokens with lifetime of about 6 months (and moreover Woocommerce has no process for coupon deletion, manually I was able to reduce it to 1400).
If it is the way I see in Xdebug, then using Redis would not help, but maybe you don't see the problem due to using OPCache (we are not :-(), but that can be tested locally...
For now I will stick to the hotfix and will investigate if there is way to mitigate the issue or any aproach to solve it properly but it will take some time.

Hostings are tough matter, we tried different one but we had VERY BAD experience, so we backed off.

For now I will stick to the hotfix
Firstly, you don't need to stick to anything, you can turn off the coupon features as previously advised.
Also it's not a hotfix, this is in the released versions.

1400 is still a lot, are you having a coupon per product with different discount levels per product?
Anyhow, apart from turning it off the Coupon functionality of this plugin as suggested, there are two more things we could do:

in Settings, WooPoly, Features, add an option to turn off only the coupon translation and leave the coupon product/category elegibility switched on
filter the translation in wp_admin:  in the front end, coupon translation is loaded only when needed, but in the back end it is applied generally via:
add_action('wp_loaded', array($this, 'adminRegisterCouponStrings'));
Perhaps this is only needed on the string translation page itself, if we can work out how to detect that.

Good ideas! Will look into it probably in two or three weeks (due to holiday).

Tested and it works nicely! Thanks for your time :-)


Install ebtel++ on CI

This PR does two things:

Pulls the path to ebtel++ from an environment variable EBTELPLUSPLUS_DIR
Adds a few steps in the GH Actions script to install ebtel++ on the CI

This may require a bit of debugging.

Thanks, Will!  Looks good to me, so I hope this works.  Once it's merged on the main ebtel++ repository, we'll have to remember to update the repository it pulls here.

Thanks for fixing my bad cd. It looks like the CI is now failing for a different reason that I don't fully understand. It appears to be somewhere in showyourwork.

Well, it's building now -- so long as the OrcIDs are commented out.  (I suppose I could fork showyourwork and fix the problematic line to get it to build with OrcIDs . . . maybe later.)
Thank you for the help with this!


Search function on https://websites.eclipseprojects.io/velocitas/ not working

Search function on https://websites.eclipseprojects.io/velocitas/ (upper right corner: "Search this site...") is not working.
Expected Behavior
Search should offer results or message about no results found
Current Behavior
After entering some text and hitting enter key nothing happens
Possible Solution
-/-
Steps to Reproduce

Open https://websites.eclipseprojects.io/velocitas/
Klick on the search field
Enter some text
Hit enter key
--> Nothing happens

Context (Environment)
Browser: Firefox 102.7.0esr (64-Bit)
Same for: Google Chrom Version 109.0.5414.120 (Official Build) (64-bit)
Same for: Microsoft Edge Version 109.0.1518.70 (Official build) (64-bit)
OS: Windows 10
Detailed Description

Possible Implementation
Remove search field if no fast fix is possible

Fixed with https://github.com/eclipse-velocitas/velocitas-docs/pull/78


Compatibility with godot 4.x?

Thanks!

Yes you can find the download link here https://github.com/cdpude/GodotBeautifier/releases, version 2.0 is for Godot 4.x.


tensorflow to caffe?

Hi, is there a plan to convert tensorflow models to caffe?

Currently, I have no plans to convert TensorFlow models to Caffe.
The process, however, should be very similar (parse the network, reshape weights into Caffe's format...)


App Settings: ability to define default settings

Similarly to #2095, apps are going to need to be able to define their own settings.
I imagine this would be something like our own default-settings.json, where they can define a setting, a default value and any standard validations. It is also intended that app settings would have a type which relates to the type of data it is going to store (see #2102).
To go along with the data and type, there will need to be a way for settings to define what sort of input to display to the user.
Short term, that might be one of a text field, textarea, checkbox, or dropdown. Long term, it will be necessary for apps to define their own custom inputs.
As example I consider an app which provides some basic settings for customising a theme:
{
    "contact_info": {
        "type": "string",
        "defaultValue": null,
        "validations": {
            "max": 150
        },
        "input": {
            "label": "Contact Info",
            "help": "Enter your address or other contact info",
            "type": "text"
        }
    },
    "colour_scheme": {
        "type": "string",
        "defaultValue": "red",
        "validations": {
            "isIn": true
        },
        "input": {
            "label": "Choose a colour",
            "help": "Your blog colour scheme",
            "type": "select",
            "options": [
                "red",
                "green",
                "blue"
            ]
        }
    }

Some of this might be default / inherent, for example the default input for a string could be a text input. The default max length might be 150 as it is in Ghost pretty much, and select inputs may validate that the option is in the ones provided by default.
Once we have a way to define these settings, we then need to do something with it.
We probably need to pre-populate the app_settings table with the options and their default values?
Is there anything else we need to do with this?
Creating the UI to change the settings comes in a separate issue.

Temporarily closing all of the app specific issues (tagged with later). Will review & reopen those that are still relevant when we kick off that project, which should be shortly after Zelda lands.


What is the size field you aggregate here?

https://github.com/aeternity/aepp-middleware/blob/da73e1dde72647ee1d9eb1f8cae444b964653ee0/src/models.rs#L558
I don't see it in the original node, how do you calculate this field?

We just add together the size of all transactions in the database


Make it easier to write modules with commands

In the present time, if you wish to write a module, you have to declare the commands using gotham_modules_command_add so we can list them in the help module, for example, and you have to create a function named event_citizen_command() which will get called by lib gotham for every received message (you will then have to parse to know if this command is for you).
It would be easier if, when using gotham_modules_command_add(), we could pass a function pointer that would be called if the message match the added command.
This would remove a few lines from modules, making it easier for new comers to understand how to write modules.
Example event_citizen_command() function :
Eina_Bool
event_citizen_command(void *data,
                      int type EINA_UNUSED,
                      void *ev)
{
   Module_Notification *notification = data;
   Gotham_Citizen_Command *command = ev;
   Gotham_Citizen *citizen = command->citizen;

   if (strcmp(command->name, ".notification")) return EINA_TRUE;

   DBG("notification[%p] command[%p][%s] citizen[%p][%s]",
       notification, command, command->name, citizen, citizen->jid);

   command->handled = EINA_TRUE;

   if (!command->command[1]) alfred_group_print(notification, command);
   else if ((command->command[1]) && (!strcmp(command->command[1], "groupadd")))
     {
        AUTH(notification, gotham_modules_command_get(".notification groupadd"), citizen);
        alfred_group_add(notification, command);
     }
   else if ((command->command[1]) && (!strcmp(command->command[1], "groupdel")))
     {
        AUTH(notification, gotham_modules_command_get(".notification groupdel"), citizen);
        alfred_group_del(notification, command);
     }
   else if ((command->command[1]) && (!strcmp(command->command[1], "useradd")))
     {
        AUTH(notification, gotham_modules_command_get(".notification useradd"), citizen);
        alfred_user_add(notification, command);
     }
   else if ((command->command[1]) && (!strcmp(command->command[1], "userdel")))
     {
        AUTH(notification, gotham_modules_command_get(".notification userdel"), citizen);
        alfred_user_del(notification, command);
     }
   else if ((command->command[1]) && (!strcmp(command->command[1], "send")))
     {
        AUTH(notification, gotham_modules_command_get(".notification send"), citizen);
        alfred_send(notification, command);
     }
   else
     DBG("WHAT SHOULD I DO ?");

   return EINA_TRUE;
}

I think anyone would agree if we could just remove all those strcmp.

I am also work on moving the authentication checks into libgotham so modules doesnt deal with this themselves.
This will make a single point in the code that deals with this, so module writers can just forget about it.


Negative number

I want to start my slider from -100 to 100.
I can set the -100 on slider but when I try to slide bubble not show negative numbers.
Is it possible to work with negative numbers?

the fraction goes from 0.0 to 1.0, so you can just say your resulting value = fraction * 200 - 100 and have a needed display range


Showing HDR Environment is broken

com.jme3.asset.AssetNotFoundException: Textures/Sky/Path.hdr (Flipped)
    at com.jme3.asset.DesktopAssetManager.loadAsset(DesktopAssetManager.java:382)
    at com.jme3.asset.DesktopAssetManager.loadTexture(DesktopAssetManager.java:404)
    at com.jme3.util.SkyFactory.createSky(SkyFactory.java:291)
    at com.jme3.gde.core.scene.SceneApplication$11.run(SceneApplication.java:640)

I wonder how this could happen, given that I just added this feature very recently.
The file should stem from testdata.

This isn't the case, but:
            Calling pack200(/usr/lib/jvm/java-11-openjdk-amd64/bin/pack200) on /home/marc/Dokumente/02_GIT/sdk/build/installer/nbi_all/ext/infra/build/products/helloworld/build/temp/jmonkeyplatform/libs/jme3-testdata.jar to /home/marc/Dokumente/02_GIT/sdk/build/installer/nbi_all/ext/infra/build/products/helloworld/build/temp/jmonkeyplatform/libs/jme3-testdata.jar.pack.gz
  [package]
  [package] Warning: The pack200 tool is deprecated, and is planned for removal in a future JDK release.
  [package]
  [package] OpenJDK 64-Bit Server VM warning: Ignoring option PermSize; support was removed in 8.0
  [package] OpenJDK 64-Bit Server VM warning: Ignoring option MaxPermSize; support was removed in 8.0
  [package] Exception in thread "main" java.lang.OutOfMemoryError: Java heap space
  [package]     at java.base/java.util.Arrays.copyOf(Arrays.java:3745)
  [package]     at java.base/java.io.ByteArrayOutputStream.grow(ByteArrayOutputStream.java:120)
  [package]     at java.base/java.io.ByteArrayOutputStream.ensureCapacity(ByteArrayOutputStream.java:95)
  [package]     at java.base/java.io.ByteArrayOutputStream.write(ByteArrayOutputStream.java:156)
  [package]     at java.base/java.io.ByteArrayOutputStream.writeTo(ByteArrayOutputStream.java:187)
  [package]     at java.base/com.sun.java.util.jar.pack.Package$File.writeTo(Package.java:845)
  [package]     at java.base/com.sun.java.util.jar.pack.PackageWriter.writeFiles(PackageWriter.java:820)
  [package]     at java.base/com.sun.java.util.jar.pack.PackageWriter.write(PackageWriter.java:76)
  [package]     at java.base/com.sun.java.util.jar.pack.PackerImpl$DoPack.flushPackage(PackerImpl.java:583)
  [package]     at java.base/com.sun.java.util.jar.pack.PackerImpl$DoPack.flushAll(PackerImpl.java:537)
  [package]     at java.base/com.sun.java.util.jar.pack.PackerImpl$DoPack.run(PackerImpl.java:473)
  [package]     at java.base/com.sun.java.util.jar.pack
  [package] .PackerImpl.pack(PackerImpl.java:92)
  [package]     at java.base/com.sun.java.util.jar.pack.Driver.main(Driver.java:322)
  [package]
  [package] 1

happens during Installer Build.

Most HDRIs are usually pretty big in size, so maybe it runs out of memory when copying it or something?

Should be fixed in efd8093, but I guess this shouldn't be the problem because I tried to build it locally.
I should be part of TestData since a few versions anyway, though...

Still happens in main branch. Slightly shifted stacktrace:
com.jme3.asset.AssetNotFoundException: Textures/Sky/Path.hdr (Flipped)
	at com.jme3.asset.DesktopAssetManager.loadAsset(DesktopAssetManager.java:385)
	at com.jme3.asset.DesktopAssetManager.loadTexture(DesktopAssetManager.java:407)
	at com.jme3.util.SkyFactory.createSky(SkyFactory.java:305)

To reproduce, click the toggle skybox button in Scene viewer


BUG: No "content-type: application/json" in json on "token" endpoint resposes makes some clients fail, like "discourse"

What steps will reproduce the problem?
Make a request to yii oauth2 endpoint
What is the expected result?
The response should has a content-type set to "application/json"
What do you get instead?
For now I've added a hack inside discourse source files
I'm thinking to add that "content-type" to Controllers & Action source files.. any hint to start?
Many Thanks!

@raimon-segura Hi, the default Yii Response overwrote the Content-Type header. This should now be fixed in \rhertogh\Yii2Oauth2Server\helpers\Psr7Helper::psr7ToYiiResponse() by setting the Response::$format to 'raw'.
Could you please confirm the issue is fixed in the latest dev-master version?

Thanks!
El lun, 19 jun 2023 a las 11:23, rhertogh @.***>)
escribió:

@raimon-segura https://github.com/raimon-segura FYI: Released as
1.0.0-alpha13
—
Reply to this email directly, view it on GitHub
https://github.com/rhertogh/yii2-oauth2-server/issues/13#issuecomment-1596828355,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMNQIRKIBEJW4JMAWCUVL3XMALABANCNFSM6AAAAAAZJF53QU
.
You are receiving this because you were mentioned.Message ID:
@.***>


awsebcli: fixup botocore, pathspec, termcolor dependencies

Description of changes

updated botocore requirement to match upstream
updated termcolor, had a look though change log nothing stood out
update pathspec, its a also minor bump,
removed jmespath replace, not sure why it was there

tested cli, everything seems to be working
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

cc @mweinelt


Uploading files pollutes the logs

Currently, when running wunderctl up --debug we're logging all requests including req+res.
That's problematic in case of file uploads, because we're printing the file to the std out.
Ideally, we could detect if the request is a file upload and prevent logging the body to keep the logs clean.

fixed


Question: Treema doesn't haven children Treemas

I got a patch ready that fetches LatestReferenceVersion models on demand as part of the Treema extension. It effectively fixed 'Unknown' references in the level editor, except for the treema-shortened preview when a node is collapsed. Though I'm sure I'll get that one to work too.
The patches however still don't show properly. The Treema doesn't have children Treemas, as can be seen in this image:

I looked around how others did it but couldn't find the difference. I'm sure Scott could help me out really quickly?
Here's a working node for reference, with the shortened version still as 'Unknown'.

Moved to https://github.com/codecombat/treema/issues/48


CalcCellValue does not always calculate percentages correctly

Description
Percentages in spreadsheets causes two kind of errors for CalcCellValue:

In calc_test.go it ignores the percentage char.
With a spreadsheet made with WPS Office it causes a parsing error. It works fine with a spreadsheet made with LibreOffice.

Percentage char is ignored in calc_test.go
Steps to reproduce the issue:

Add percentage calculation to calc_test.go, see :

	mathCalc := map[string]string{
                ...
		"=15%":       "0.0015",
		"=1+20%":     "1.0020",

I've done this in my percentage branch: https://github.com/stanim/excelize/blob/percentage/calc_test.go#L51

Run go test.

Describe the results you received:
> go test
--- FAIL: TestCalcCellValue (8.46s)
    calc_test.go:1218:
        	Error Trace:	calc_test.go:1218
        	Error:      	Not equal:
        	            	expected: "0.0015"
        	            	actual  : "15"

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-0.0015
        	            	+15
        	Test:       	TestCalcCellValue
        	Messages:   	=15%
    calc_test.go:1218:
        	Error Trace:	calc_test.go:1218
        	Error:      	Not equal:
        	            	expected: "1.0020"
        	            	actual  : "21"

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-1.0020
        	            	+21
        	Test:       	TestCalcCellValue
        	Messages:   	=1+20%

Describe the results you expected:
I expected the tests to pass.
Percentage causes parsing error with WPS Office
Steps to reproduce the issue:

Download percentage.zip and unzip it

Run the go file: go run ./percentage.go

Describe the results you received:
> go run ./percentage.go
percentage_libre.xlsx
15

percentage_wps.xlsx
2021/08/18 18:59:17 strconv.ParseFloat: parsing "15%": invalid syntax
exit status 1

Describe the results you expected:
I expected the same correct result as percentage_libre.xlsx for percentage_wps.xlsx without parsing error.
Excelize version or commit ID:
> git log
commit 07e84b89cc080921f19c10f93f70a6b33ed6b7db (HEAD -> percentage, origin/percentage)
Author: Stani Michiels<EMAIL_ADDRESS>Date:   Wed Aug 18 13:19:07 2021 +0200

    Add percentage test to calc test

commit a55f354eb3d0c6c1b9a543ff8ff98227aa6063a6 (origin/master, origin/HEAD, master)
Author: xuri<EMAIL_ADDRESS>Date:   Tue Aug 17 00:01:44 2021 +0800

    This closes #989, closes #990

    New API: `SetRowStyle` support for set style for the rows
    Update documentation for the `GetRows`, `SetCellStyle` and `SetColStyle`

Configuration
Output of go version:
go version go1.16.7 linux/amd64

Environment details (OS, Microsoft Excel™ version, physical, etc.):

OS: Archlinux 5.13.10-arch1-1
WPS Spreadsheets (version wps-office <IP_ADDRESS>02-1)
LibreCalc (version libreoffice-fresh 7.1.5-2)

Where should this be fixed? Would you accept a pull request? Maybe I can help if you point me to right source code. Should pull requests be made against the master or v2 branch?
percentage.zip

Thanks for your feedback. I have fixed this issue, please try to upgrade the master branch code


Model Result

i give fake video but it tell the real

waht are the reason behind this
i use your trained model


Cask request: Mailspring

Cask details
Name: Mailspring
Homepage: https://getmailspring.com
Download URL: https://mailspring-builds.s3.amazonaws.com/client/20839f9b/osx/Mailspring.zip
Description: A fast and maintained fork of Nylas Mail - the best free email app for Mac, Linux, and Windows.

https://github.com/caskroom/homebrew-cask/pull/39285


Migration: implement GUI

implement GUI for handling all your Dartabase

project settings
migration
scaffolding (will come in v >= 0.7.0)

general gui implemented. 1.0.0-rc1


Can inactive windows be captured?

I seek to capture an image within a browser tab; the tab, or the entire browser, may be out of focus or minimized. Can PyAutoGUI do this?

PyAutoGUI only takes a screenshot of what is visible, so you must have the application in view on the monitor for it to work.


What would it take to abstract the storage?

My understanding from the readme is that what rule/data is implemented/backed via a clojure atom. Roughly, what do you think it would take to abstarct the storage? in my specific use case, i'm considering what it would take to back odoyle with datomic cloud. Do you see any obvious blockers or refactors?
Thanks for any input!

The library itself doesn't actually care how you store it. All the examples use atoms, but the library has no built-in assumptions about that. The functions like insert and retract just expect the session to be provided, and return a new one, so you can store it in any container you want.
As for datomic, it should be possible. I assume you want to store the matches for a given rule in your datomic db. You can get all the matches for a given rule like this:
(def rules
  (o/ruleset
    {::player-position
     [:what
      [::player ::x x]
      [::player ::y y]
      :then
      (println (o/query-all o/*session* ::player-position)) ; [{:x 10 :y 10} {:x 1 :y 5} {:x 15 :y 0}]
      ]}))

So in theory you could replace that println call with a function that stores that vector in datomic. Every time any player's position is updated, the :then block will be called again, and you'll be able to update the vector in datomic.
You will probably still need to keep the session in some kind of container though. Locally you'll need to keep the session in an atom, volatile, or something like that. But every time you update it, the :then blocks will fire and cause the data to go into your db.

As of version 0.5.0 you can now query all the individual facts from a session, so that might be a better way to store them in datomic. All you do is run (query-all session) and it'll return the facts as [id attribute value] tuples, which could then be directly stored in datomic. I added a section to the readme called "serializing a session" which talks about this.


Improvements

I found a sample which could be easily fixed by correctin UPX! signature, but I don't know how to figure out where exactly it should be changed.
How can I contact you to share some links and samples? Samples are malware

You can email larry0 at me.com.

closing.


bug: ion-toast doesn't have a buttons property in react

Bug Report
Ionic version:
[x] 4.x
Current behavior:
Typescript error: Property 'buttons' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes<{ element: HTMLIonToastElement; ...
Expected behavior:
Should compile without error.
Steps to reproduce:
Use the component below and try to compile it. I'm working off of the react documentation here
Related code:
<IonToast
    isOpen={this.state.cartPreview}
    onDidDismiss={() => this.setState({ cartPreview: false })}
    buttons={[{
        side: 'start',
        icon: 'star',
        text: 'Favorite',
        handler: () => {
            console.log('Favorite clicked');
        }
    }]}
/>

Ionic info:
Ionic:

   ionic (Ionic CLI) : 4.12.0 (/usr/local/lib/node_modules/ionic)

System:

   NodeJS : v10.14.2 (/usr/local/bin/node)
   npm    : 6.9.0
   OS     : macOS Mojave

Hi @hntrl, could you verify which version of @ionic/core you are on? Version > 4.3 should have the updated interfaces for the toast buttons.
Thanks

Going to close, if you still find issues with this let us know.  Thanks!


Add more task debug info in jobInfo api response [rest-server]

Coverage decreased (-0.06%) to 34.622% when pulling 71c04ebe966d75ca0bd9724d57f10aa9e15f630e on mintao/rest-server-debuginfo into 0de80f4fbc476ddecf7ffb04aeaa10292081aab2 on master.

  // for backward compatibility

Add doc for the new fields?
I have gave their explanation in onenote, you can refine it in PAI doc?

Refers to: src/rest-server/src/models/v2/job/k8s.js:204 in 71c04eb. [](commit_id = 71c04ebe966d75ca0bd9724d57f10aa9e15f630e, deletion_comment = False)

  virtualCluster,

Seems appExitType is useless and duplicated with appExitSpec, should we remove it?
BTW, the completionStatus.type.name only have 2 values;
type CompletionType struct {
 Name       CompletionTypeName        `json:"name" yaml:"name"`
 Attributes []CompletionTypeAttribute `json:"attributes" yaml:"attributes"`
}

type CompletionTypeName string

const (
 CompletionTypeNameSucceeded CompletionTypeName = "Succeeded"
 CompletionTypeNameFailed    CompletionTypeName = "Failed"
)

Refers to: src/rest-server/src/models/v2/job/k8s.js:303 in 71c04eb. [](commit_id = 71c04ebe966d75ca0bd9724d57f10aa9e15f630e, deletion_comment = False)

  virtualCluster,

Seems appExitType is useless and duplicated with appExitSpec, should we remove it?
BTW, the completionStatus.type.name only have 2 values;
type CompletionType struct {
 Name       CompletionTypeName        `json:"name" yaml:"name"`
 Attributes []CompletionTypeAttribute `json:"attributes" yaml:"attributes"`
}

type CompletionTypeName string

const (
 CompletionTypeNameSucceeded CompletionTypeName = "Succeeded"
 CompletionTypeNameFailed    CompletionTypeName = "Failed"
)

Refers to: src/rest-server/src/models/v2/job/k8s.js:303 in 71c04eb. [](commit_id = 71c04eb, deletion_comment = False)

  virtualCluster,

Seems appExitType is useless and duplicated with appExitSpec, should we remove it?
BTW, the completionStatus.type.name only have 2 values;
type CompletionType struct {
 Name       CompletionTypeName        `json:"name" yaml:"name"`
 Attributes []CompletionTypeAttribute `json:"attributes" yaml:"attributes"`
}

type CompletionTypeName string

const (
 CompletionTypeNameSucceeded CompletionTypeName = "Succeeded"
 CompletionTypeNameFailed    CompletionTypeName = "Failed"
)

Refers to: src/rest-server/src/models/v2/job/k8s.js:303 in 71c04eb. [](commit_id = 71c04eb, deletion_comment = False)

If we delete a previous property, I am not sure any side effects on frontend. I have checked that webportal does not use this field, but still need to make sure sdk/vscode parts.


Queekish language (default settings for wfrp)

Is your feature request related to a problem? Please describe.
I think that Air Ciela for Queekish (default settings for wfrp) is oversight

Describe the solution you'd like
Change Air Ciela to Skaven for Queekish default settings for wfrp
Additional context
Queekish is a language for wfrp ratmen race
Image from core rulebook for wfrp 4

It is, it used to be Skaven some time. There was a lot of copy-pasteing during a refactoring of the code and I missed this language.
Thanks for reporting it.


Prepare does not install relevant packages - store plugin broken

PHP Version
No response
Shopware-Cli Version
0.1.28
Actual behaviour

Expected behaviour

Steps to Reproduce?
Plugin https://github.com/FriendsOfShopware/FroshPlatformBunnycdnMediaStorage
Run composer install
Compare with shopware-cli extension prepare
when using shopware-cli prepare:

It looks like "symfony/http-client-contracts": "v2.4.0"  is falsly posted in generated composer.json for "replaces"?

Got it! This was part until 6.4.8 :-( So this error occur for newer shopware-version.
Any idea how this can be caught? Raising minVersion in Plugin is really uncool 🤔


Use updated GAPIC configuration options when using an emulator with gRPC as transport

We should make sure we are using the updated GAPIC configuration options when necessary:
https://github.com/googleapis/google-cloud-php/blob/1877a3c8b24fd52df410412f194a360a1a7a880c/Datastore/src/Connection/Grpc.php#L92-L96
https://github.com/googleapis/google-cloud-php/blob/83ae284c025f6e93b9ce835b987932c425b5a9de/PubSub/src/Connection/Grpc.php#L88-L92

port has since been deprecated and now included in serviceAddress
sslCreds is now provided through transportConfig or a custom Google\ApiCore\Transport\GrpcTransport instance via transport.

Closed by: https://github.com/googleapis/google-cloud-php/pull/1667


Exception in thread "Thread-4" java.lang.NoClassDefFoundError: ml/combust/bundle/HasBundleRegistry

I'm not sure if this issue has a resolution. Why would this error happen and how can you fix it?
I'm on Java 8, mleap 0.20.0, Scala 2.12 and pyspark 3.1.3.
https://github.com/combust/mleap/issues/442

Might be related to the issue in #842

Ha right it is related to #842 I guess but more related to #442. #442 was closed, but I am unsure why because the answer is not there. So I figure I can directly address that here ...

This was resolved after we installed Java 11 and used the malest mleap. We used mleap-databricks-runtime-fat-assembly-0.22.0. You also need to change JAVA_HOME inside of spark. The command below might work.
`
We need this new Java since the new Mleap uses Java 11.
sudo apt-get -y install openjdk-11-jdk
export JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64
If you don't do this, Spark will complain about Python 3.7 and 3.8.
export PYSPARK_PYTHON=/opt/conda/miniconda3/bin/python
export PYSPARK_DRIVER_PYTHON=/opt/conda/miniconda3/bin/python
pip install mleap
Copy the jar to the right location
cp mleap-databricks-runtime-fat-assembly-0.22.0.jar $SPARL_HOME/jars
echo "export JAVA_HOME="/usr/lib/jvm/java-11-openjdk-amd64"" >> $SPARK_HOME/conf/spark-env.sh
`


[Suggestion/Improvement] Stats of a specific location

Checklist

[x] I have read the FAQ
[x] I have checked if this issue already exists (both open and closed issues)
[x] I am using the most recent version of pogom from the master branch
[ ] If this is an installation issue, I am posting my OS, OS version, python version, pip version
[ ] I'm posting the complete log
[x] I'm using a descriptive title for this issue

My Issue
The idea here is to show a button right next to "Delete Location" button which displays a list of the pokemon found in that location.
Why?
Let's say that my friend found an Aerodactyl in an area, then the next day he find it again nearby the same place he found the first one. It would be really good to set a tracking there and let it scan for a few hours (or a whole day) and check if that area is has a spawn of Aerodactyls.
...

isnt there heatmaps for??
with the heatmaps you can see specific spawnpoints where for example aerodactyl spawns..
try it out and see if that fits your needs.

Heatmaps implements this functionality.

That's kinda what heatmaps do.


Add deleteUser method to cognito-user util

This PR simply adds the missing deleteUser method to the cognito-user util.

Thanks for the contribution! Do you mind adding a test? I know it's simple, but like to be thorough. You should be able to base it on one of the simple tests (like the 'resendConfirmationCode' test) in this file: https://github.com/paulcwatts/ember-cognito/blob/master/tests/unit/utils/cognito-user-test.js

No worries! Added a test as requested 👍

Perfect, thank you for the contribution! 👍

Released as 0.9.0. Sorry for the delay!

No worries! Thanks for releasing 🎉


Support more Twirl templates

This PR adds support for built-in Twirl templates for HTML, JS, XML and txt.  Previously, we hardcoded everything to the HTML format.  This changed the default include pattern for Twirl source sets to **/*.scala.*.  No additional configuration is needed for builds to use this.
This PR also adds support for specifying custom Twirl template format types.  Template formats need an extension, a format class name and (optionally) additional imports.  To use this, you need to configure the TwirlCompile task:
model {
    components {
        play {
            sources {
                withType(TwirlSourceSet) {
                    addUserTemplateFormat("csv", "views.formats.csv.CsvFormat", "views.formats.csv._")
                }
            }
        }
    }
}

Fixes #2566
Fixes #2670

@bigdaz I've rebased and addressed your comments


Disabling "sign-off" for members doesn't work, instead requires GPG signatures

Installed DCO bot for the cyrus projects (https://github.com/cyrusimap/cyrus-sasl and https://github.com/cyrusimap/cyrus-imapd).
Set up the bit so that members should be ignored:
https://github.com/cyrusimap/cyrus-sasl/commit/647f5a46c259afee5bc68c2687bd7715bba50e91
for example.  Instead, members must now gpg sign commits.  That seems to be the opposite result of what was intended?

I can kind of understand "organisation members get a special privilege (not having to sign-off), but to get it, they must prove they're organisation members (with GPG)", but:
If a non-organisation member submits a PR containing commits that falsely claim to be authored by an organisation member:

the PR submitter/commit author mismatch will already stand out at code review, regardless of the DCO check
reviewers can trivially verify the authenticity by just asking the org-member alleged author about it over another channel
the submitter could have trivially succeeded at the DCO check anyway, by making the same fraudulent claim in a Signed-off-by: line? (I'm not certain, I haven't poked this hard.)

On the other hand, if a non-organisation member submits a PR and isn't trying to frame an organisation member specifically, then we have no way to confirm whether or not they're actually trying to frame someone else instead.  The whole thing is already premised on a trust that the author is who the submitter says they are, which is fine actually; but it seems like if a GPG signature should be required anywhere, it's the "we have no idea who you are" case, not the "is an organisation member" case.
I guess if the whole project is already requiring GPG-signed commits, then the issue becomes academic.  But if the project isn't already requiring GPG-signed commits, it feels backwards to suddenly require it here.

+1

I think this should be configurable. I'd love to use this check, but organization membership (because it's governed by external contracts) will be enough to satisfy any legal issues. Ideally then, organization members shouldn't need to jump through extra hoops.
Having an enumeration instead of a boolean would allow people to configure it easily:
members_must_sign_off: "gpg" | "trailer" | "no"


add check for /etc/resolvconf.conf

add option to diable /etc/resolvconf.conf update
ansible facts /etc/os_release /etc/debian_version don't differentiate debian vrs raspberrypi os, so test if /etc/resolvconf.conf file exists and update if it exists
Fixes #520
tested deploying on debian 11.7 micropc and raspberrypi os 20230214 rpi 4b 4g

Could we instead just have a variable like resolvconf_file_path and let the user override it? So instead of hardcoding /etc/resolvconf.conf, we could have resolvconf_file_path: /etc/resolvconf.conf, and Debian users could set that to resolvconf_file_path: /etc/resolve.conf.

I started with the config to disable it, but then figured out checking if the file exists.  And I left that option in because it' was useful to run multiple ad block DNS containers for testing.
I looked into something like that, unfortunately they aren't the same file/format, and are usually externally managed.
on Raspberry pi OS resolvconf regenerates /etc/resolve.conf based on /etc/resolveconf.conf as well as integrate with dnsmasq/pdnsd/unbound and other name services.
On Debian NetworkManager configures /etc/resolv.conf
On Ubuntu systemd-resolved manages it.
Looking at the ansible NetworkManager module: https://docs.ansible.com/ansible/latest/collections/community/general/nmcli_module.html
I don't see a way to just add a DNS server.


enable Travis

This isn't ready yet. Currently fails because it doesn't get the right version of CMine. It gets a newer one than this code expects.


fix empty conversation_id

Description
conversation id when given empty string , it goes to the preexisting conversation id flow.
Fixed by checking for empty string .
Type of change

[ ] Refactor
[ ] New feature
[x] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/retest

/lgtm

/retest

/retest

/retest


code question

May I ask, what is the basis for clustering the data into two groups?
According to the code, I found that the final output is a model that works best.Can we train a model for each location and export the model?

Hi Yukun,
The reasoning for the clustering is that ML-based modeling uses featurized training data.  The training data could be:
i).  1 time series at a time (Prophet or ARIMA for example).  This means you might have to build 1 million specialized models, which can be time-consuming.
ii).  All the training data at once (possible with deep learning or transformer algorithms).  This means building 1 single generalized model to fit all the data at once.  If the data has distinctly different segments with different behaviors, then the 1 model might be missing accuracy.  Conceptually, it's like fitting a mean curve through a bunch of data at once.
iii).  A good compromise might be to choose reasonable clusters, then train a different model per cluster.  So, instead of 1 milliion models or only 1 model, with clustering, you might get a handful of models.
For retail data, often the natural clustering is geo- , since different products are sold by geo, and this type of clustering is more likely to group together data with similar underlying patterns.
The taxi data seemed to have real data vs what looked like too-smooth fake data (1 ride at regular time intervals).  The taxi data had only 2 out of 4 possible clusters.  For the demo, I kept only the “erratic” cluster, from the clustering technique explained here.  (Scroll down to Step 21 or cell 117):  https://github.com/aws-samples/amazon-forecast-samples/blob/ab6cf3c48fa1a22c892d997c3b7a9235a0f019c0/workshops/pre_POC_workshop/1.Getting_Data_Ready_nytaxi_weekly.ipynb
Hope that helps with the motivation for clustering!
Christy

Thank you so much！


Update statemanagement-quickstart.md, remove npm install command from JavaScript Step 1

Removed "Install the dependencies for the order-processor app" from JavaScript Step 1 as we are doing this in Step 2.
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description

Issue reference

@holopin-bot @svdspradeep Enjoy your Dapr docs badge!


fix typo in getting started

validation -> validate

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


https://issues.apache.org/jira/browse/SPARK-8754

YarnClientSchedulerBackend doesn't stop gracefully in failure conditions
In YarnClientSchedulerBackend.stop(), added a check for monitorThread.

Can one of the admins verify this patch?

LGTM

Test build #991 has started for   PR 7153 at commit 66be9ad.

Test build #991 has finished for   PR 7153 at commit 66be9ad.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class Heartbeat(workerId: String, worker: RpcEndpointRef) extends DeployMessage
  case class RegisteredWorker(master: RpcEndpointRef, masterWebUiUrl: String) extends DeployMessage
  case class RegisterApplication(appDescription: ApplicationDescription, driver: RpcEndpointRef)
  case class RegisteredApplication(appId: String, master: RpcEndpointRef) extends DeployMessage
  case class SubmitDriverResponse(
  case class KillDriverResponse(
  case class MasterChanged(master: RpcEndpointRef, masterWebUiUrl: String)
class DCT(override val uid: String)
class MinMaxScaler(override val uid: String)
class PCA (override val uid: String) extends Estimator[PCAModel] with PCAParams
class StreamingLinearAlgorithm(object):
class StreamingLinearRegressionWithSGD(StreamingLinearAlgorithm):
class AnalysisException(Exception):
case class Cast(child: Expression, dataType: DataType) extends UnaryExpression with Logging
      class SpecificOrdering extends $
    class SpecificProjection extends $
    final class SpecificRow extends $
case class Crc32(child: Expression)
    // compiled class file for the closure here will conflict with the one in callUDF (upper case).

jenkins, retest this please

retest this please

Merged build triggered.

Merged build started.

Test build #36317 has started for   PR 7153 at commit 66be9ad.

Test build #36317 has finished for   PR 7153 at commit 66be9ad.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merging into master 1.4. Thanks @devaraj-kavali


Image steganography

Pull Request Template
script name -
Brief about script
Issue no.(must) -
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt with my script

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

Few suggestions

[ ] Elaborate README more
[ ] Add requirements.txt file
[ ] Add docstrings to function
[ ] Linting

Hey Please check the requirements.txt of others or on web

Sorry about that. I have updated it

Clean the requirement.txt file, only add you're using in script not in whole environment

@all-contributors add @Sandhya1708 for code


Implement nearest neighbour iterators on Frame struct

The functions for the RTree should be in iterators on the Frame struct. This means that the tree can be created once for the frame. Using a Option means this can be done lazily, i.e. the first time it is required.
This should be two separate iterators, one for the n-nearest neighbours, and another for the neighbours within a specified distance.

fixed b811ab2a8ba31a5694efdebbf01d67f1b6485cae


⚠ Jenkins Plugins Artifactory has degraded performance

In 964cc66, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1152 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 9dbd75a after 1 hour, 49 minutes.


NavigationView shows unwanted empty block

Describe the bug
If PaneFooter is empty and IsSettingsVisible="False" NavigationView show unwanted empty bottom block if height of window is small
Steps to reproduce the bug

Steps to reproduce the behavior:

Create NavigationView and set IsSettingsVisible to "False"
Decrease height of window until you see an empty block

Expected behavior
NavigationView must hide this block, If PaneFooter empty and Settings button not visible
Screenshots

Version Info
NuGet package version:
[Microsoft.UI.Xaml 2.5.0]
[Microsoft.UI.Xaml 2.4.0]

Windows app type:

UWP
Win32

Yes
Unchecked

Windows 10 version
Saw the problem?

Insider Build (xxxxx)
Yes

October 2020 Update (19042)
Unchecked

May 2020 Update (19041)
Unchecked

November 2019 Update (18363)
Unchecked

May 2019 Update (18362)
Unchecked

October 2018 Update (17763)
Unchecked

April 2018 Update (17134)
Unchecked

Fall Creators Update (16299)
Unchecked

Creators Update (15063)
Unchecked

Device form factor
Saw the problem?

Desktop
Yes

Xbox
Unchecked

Surface Hub
Unchecked

IoT
Unchecked

Additional context
No context.

Thanks for filing this issue! This was already reported here: #3900

Thanks for filing this issue! This was already reported here: #3900

Oh, sorry for duplicate)


FYI: Okta breaking change u2f to webauthn

Figured I would file this as just an impending FYI.
Okta just transitioned our org from U2F factors to FIDO2 (WebAuthN).
This means that all our code that was looking for u2f factors suddenly broke.
I'm not sure if that change was specific to our org, or Okta wide.
For reference this is our internal change that does the bare minimum to mitigate the issue. It probably does not properly implement true webauthn, but you can use it as a guide for a U2F to WebAuthN migration path.
https://github.com/wearefair/okta-auth/pull/8/files

We're seeing this issue as well

Oof, that's rough.
We don't use u2f/webauthn Okta internally at Segment (trust me, I'd like to), so we're relying on the community for a fix here.

Yes, I noticed that when I asked Okta to enable the WebAuthn early access feature on my org's account, the U2F factor in MFA enrollment was replaced by a WebAuthn factor; i.e., it appears that you can enable either U2F or WebAuthn, but not both.

I've been working on supporting webauthn in https://github.com/Versent/saml2aws and I have an untested branch that could work for aws-okta:
https://github.com/alsmola/aws-okta/pull/1/files
I made some modifications to go-u2fhost that I'm asking to be merged, but in the meantime this change uses my fork.
If someone can test and confirm this branch with aws-okta, I'm happy to hand it over.