ENH: add quick_tree, distance_matrix to new Alignment

Summary by Sourcery
Add new methods 'distance_matrix' and 'quick_tree' to the Alignment class for calculating pairwise distances and generating phylogenetic trees, respectively. Enhance the 'get_translation' method with a 'name_map' parameter. Introduce comprehensive tests for the new methods and sequence renaming functionality.
New Features:

Introduce the 'distance_matrix' method to calculate pairwise distances between sequences in an alignment, with options for different calculators and handling of invalid data.
Add the 'quick_tree' method to generate a phylogenetic tree from an alignment, with support for bootstrapping and handling of invalid data.

Enhancements:

Improve the 'get_translation' method by adding a 'name_map' parameter to better handle sequence naming.

Tests:

Add tests for the 'distance_matrix' method to ensure correct calculation of pairwise distances, including handling of invalid data.
Introduce tests for the 'quick_tree' method to verify the generation of phylogenetic trees and the handling of bootstrap replicates.
Enhance tests for sequence renaming in the 'iter_seqs' method and the 'get_translation' method to ensure correct functionality.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

40 of 43 (93.02%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 92.162%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/cogent3/core/new_alignment.py
35
38
92.11%

Files with Coverage Reduction
New Missed Lines
%

src/cogent3/maths/scipy_optimize.py
2
84.92%

Totals

Change from base Build<PHONE_NUMBER>6:
-0.003%

Covered Lines:
33770

Relevant Lines:
36642

💛  - Coveralls


Skype Calling Bot - assigning a phone number

First of all, thank you for the demos and for the framework itself.
I was able to run demo-skypecalling bot and call the bot using Skype. Great!
Hardest part was to find out which urls should I put into the Skype config page and bot config, then wait around 5 minutes for settings to propagate (especially when using ngrok and changing url every day).
Skype has a paid feature called Skype Number.
How do one gets a phone number assigned to a Skype calling bot?
I'm fine paying for the Skype Number for the bot 👍

Currently adding a phone number to a Skype calling bot is not supported. We will evaluate enabling this for a later release.

Thank you!
Alex

On Jan 19, 2017, at 16:33, KrishnanA<EMAIL_ADDRESS>wrote:
Currently adding a phone number to a Skype calling bot is not supported. We will evaluate enabling this for a later release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Any advance on it? Other companies like Nuance are already offering a complete package for create a bot and be able to use a phone number to answer phone calls. So, are you going to add this feature, or should we try other alternatives?


declaring operator as dependent on another where that manages multiple CRDs

In the ClusterServiceVersion we can mark one operator as depending on another with a required section. The required operator is then installed together with the main operator if it isn't already present. I've used this in case where the dependent operator has just a single CRD (it was the seldon core operator) and it worked fine. But then I tried to depend on the service mesh operator, which owns three CRDs:
    required:
      - kind: ServiceMeshControlPlane
        name: servicemeshcontrolplanes.maistra.io
        version: v1
        displayName: Istio Service Mesh Control Plane
        description: An Istio control plane installation
      - kind: ServiceMeshMemberRoll
        name: maistra.io
        version: v1
        displayName: Istio Service Mesh Member Roll
        description: A list of namespaces in Service Mesh
      - kind: ServiceMeshMember
        name: servicemeshmembers.maistra.io
        version: v1
        displayName: Istio Service Mesh Member
        description: Marks the containing namespace as a member of the referenced Service

But I'm not able been able to get that working and I don't get any error. It just doesn't install the ClusterServiceVersion. I'm thinking it doesn't work in the case where there's multiple CRDs managed by a single operator or perhaps there's something else special about this operator.

@ryandawsonuk this question would be better answered by the OLM team. Can you close this and re-open in https://github.com/operator-framework/operator-lifecycle-manager/? Thanks!


scalarmult() supports degenerate public-keys (insecure)

Currently scalarmult() accepts all-zero public keys, for which the result (DH shared secret) will always be zero regardless of the private key used.
Against this, libsodium's crypto_scalarmult_curve25519() returns a non-zero value if it encounters such degenerate keys. You should therefore check its return value when calling ffi::crypto_scalarmult_curve25519(&mut q, n, p);.
This is a similar issue as just reported to rbnacl https://github.com/cryptosphere/rbnacl/issues/152

Thanks for the find.
On Thu, 26 Jan 2017 at 13:39, JP Aumasson<EMAIL_ADDRESS>wrote:

Currently scalarmult() accepts all-zero public keys, for which the result
(DH shared secret) will always be zero regardless of the private key used.
Against this, libsodium's crypto_scalarmult_curve25519() returns a
non-zero value if it encounters such degenerate keys. You should therefore
check its return value when calling ffi::crypto_scalarmult_curve25519(&mut
q, n, p);.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dnaq/sodiumoxide/issues/154, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPVBUmKPq5XWWsY6Plw6nGGEN3FqAwkks5rWJP4gaJpZM4LumFd
.

@veorq Can you explain why you consider this an issue?
In typical protocols an attacker who can trick you into thinking a low order public key belongs to your communication partner could also trick you into using a public key for which the attacker knows the corresponding private key.

How do I validate Curve25519 public keys?
Don't. The Curve25519 function was carefully designed to allow all 32-byte strings as Diffie-Hellman public keys.
There are some unusual non-Diffie-Hellman elliptic-curve protocols that need to ensure ``contributory'' behavior. In those protocols, you should reject [...]. But these exclusions are unnecessary for Diffie-Hellman.

This check ensures joint key control, to prevent dishonest parties from choosing the shared secret. Whether this check is required depends on the protocol and on the threat model. It may be superfluous, but it can't hurt.


Why the sigmoid in count_rewards()

Hi,
In python\RL\train.py, after adding the ease of answering reward and semantic coherence, the sigmoid of the reward is scaled by 1.1
total_loss = sigmoid(total_loss) * 1.1
What was the purpose of the sigmoid and the scaling (1.1) in this line 261?
Also, I noticed you didn't weight each reward by lambda like in the "Deep Reinforcement Learning for Dialogue Generation" paper. Was this on purpose?
Thanks!

The sigmoid and the scaling is the empirical setting.
Paper didn't explain how to obtain the lambda value.
Maybe it's obtain from experiments, and is highly depend on training data.
So I ignore the lambda.


Version 4

Any idea on when version 4 will be in npm?
I am having problems using it straight from github:
In my packages.json file:
"react-sticky": "git://github.com/captivationsoftware/react-sticky.git",
when I do webpack build, it complains:
32% 20/53 build modulesModuleNotFoundError: Module not found: Error: Cannot resolve module 'react-sticky' in /app/reports/client/app/financials
at /app/reports/node_modules/webpack/lib/Compilation.js:229:38
at onDoneResolving (/app/reports/node_modules/webpack/lib/NormalModuleFactory.js:29:20)
at /app/reports/node_modules/webpack/lib/NormalModuleFactory.js:85:20
.
.
resolve module react-sticky in /app/reports/client/app/financials
looking for modules in /app/reports/node_modules
resolve 'file' react-sticky in /app/reports/node_modules
resolve file
/app/reports/node_modules/react-sticky is not a file
/app/reports/node_modules/react-sticky.js doesn't exist
/app/reports/node_modules/react-sticky.jsx doesn't exist
/app/reports/node_modules/react-stickyindex.js doesn't exist
/app/reports/node_modules/react-stickyindex.jsx doesn't exist
/app/reports/node_modules/react-sticky.styl doesn't exist
resolve 'file' or 'directory' /app/reports/node_modules/react-sticky
resolve file
/app/reports/node_modules/react-sticky is not a file
/app/reports/node_modules/react-sticky.js doesn't exist
/app/reports/node_modules/react-sticky.jsx doesn't exist
/app/reports/node_modules/react-stickyindex.js doesn't exist
/app/reports/node_modules/react-stickyindex.jsx doesn't exist
/app/reports/node_modules/react-sticky.styl doesn't exist
resolve directory
directory default file index
resolve file index in /app/reports/node_modules/react-sticky
/app/reports/node_modules/react-sticky/index doesn't exist
/app/reports/node_modules/react-sticky/index.js doesn't exist
/app/reports/node_modules/react-sticky/index.jsx doesn't exist
/app/reports/node_modules/react-sticky/indexindex.js doesn't exist
/app/reports/node_modules/react-sticky/indexindex.jsx doesn't exist
/app/reports/node_modules/react-sticky/index.styl doesn't exist
use lib/index.js from package.json
resolve 'file' or 'directory' lib/index.js in /app/reports/node_modules/react-sticky
resolve file
/app/reports/node_modules/react-sticky/lib/index.js doesn't exist
/app/reports/node_modules/react-sticky/lib/index.js.js doesn't exist
/app/reports/node_modules/react-sticky/lib/index.js.jsx doesn't exist
/app/reports/node_modules/react-sticky/lib/index.jsindex.js doesn't exist
/app/reports/node_modules/react-sticky/lib/index.jsindex.jsx doesn't exist
/app/reports/node_modules/react-sticky/lib/index.js.styl doesn't exist
resolve directory
/app/reports/node_modules/react-sticky/lib/index.js doesn't exist (directory default file)
/app/reports/node_modules/react-sticky/lib/index.js/package.json doesn't exist (directory description file)

I'm just wrapping up tests.... I have addressed your issue on my local machine that I am yet to push.
I'm aiming for this weekend

Okay sounds good thanks

@dbarbalato thanks for doing this, looking forward!

Released! Available on npm as 4.0.1 because I pushed a beta version up on accident...

Great thanks!  Is there anything major you know of that isn't working with this release?

So I'm pretty confident this is the best release yet. Its def not compatible with v3, hence the change from 3 to 4, but I think the changes are for the best. Its far easier to get going with sticky elements right away.
Some issues I noticed:
1.) In doing some preliminary testing with a project using react-router, I noticed my sticky elements weren't updating during route events like I was expecting them to due to the fact that react-router does not emit any window events that this library listens for. However, this could/should be addressed on a project-specific basis - doesn't require a modification to this library.
2.) Sticky elements work best when they have an explicit height, or at the bare minimum, don't derive their vertical space from margins of their child elements. Some experimentation may be required as you work with this new version, but as I said earlier, I'm very confident users will see better functionality right out of the gate.

:tada:

@dbarbalato. I'm coming across the problem with react-sticky not working with route correctly. You mean that it should be handled on a project specific basis. Any tips on how to do that?

Sorry, I don't personally have any. Maybe somebody else will...


upload host galaxy template from a file

Hi,
Thanks for this very nice tool.
Just a question about the host galaxy template. If I want to upload a template from a file, where should I give it to the model? For example, when I do
mm,best_fit=my_shape.sync_fit(check_host_gal_template=True,
Ep_start=None,
minimizer='lsb',
silent=True,
fit_range=[10. , 21.])
my_shape.IC_fit(fit_range=[23., 29.],minimizer='minuit',silent=True)
p=my_shape.plot_shape_fit()
p.rescale(y_min=-15)
It checks the host galaxy and returns the parameters. But I want to give the template from a file.
Can you please help with this ?

Hi,
in principle it is very easy to build a model with arbitrary templates (even if I still need to write no notebook for the documentation). But for the sedshaper, in the current version, the code will use the default one. I will add the possibility to load a custom shape in the sedshaper, as new feature in the new release. But if you want to  do it with the current release,  I can make a new branch, then you should checkout that branch, and install from the source. On the contrary, if you want to load the template when you do the final model minimisation, then it can be done easily also with the current version.

Hi Andrea,
Thanks for your answer. Can you please help me to understand how to load the template during the final minimization ?
Thanks a lot!

Hi,
this is the link to notebook wit the example: https://drive.google.com/drive/folders/1G3TAnNrEpAcTbZq20M7Cwp4jVWpH5bv5
There are some caveats in boldface in the notebook. The most importanti is that, waiting for the new release, now you should prepare your template with a scale on x and y axis as much as possible close to that of the custom one, and you have to take the starting value of the nuFnu_p from the sedshaping stage. Everything is in the notebook, I am attaching also the custom template that is loaded, that by the way is the same of the default, but it proofs that it works. Please, let me know if you need further support. I any case, I will make a prerelease soon, with the custom model enabled also in the sed shaping process.

Thank you very much Andrea!
It works, thanks for your help.

Good,
so I close the issue. I will send you a notice when I have a prototipe with the new branch to test.
Cheers
Andrea


Customize by subclassing or by init args?

Traditionally, extension objects such as the model and session were customized by subclassing SQLAlchemy and overriding methods.  Newer customizations have been accomplished by adding more args to __init__.  As it stands:

customize by subclass:

db.Model with make_declarative_base which uses _BoundDeclarativeMeta by default.
db.session with make_session and make_scoped_session, with SignallingSession by default.

customize by __init__ args:

session options, including scope function
db.Query with query_class, defaults to BaseQuery
db.Model.metadata with metadata, defaults to None
db.Model base class (the base class of the declarative meta) with model_class

Either pattern is viable for all points, I don't really mind either way.  What I don't like is the inconsistency.  Half one way, half another, and a bunch of crossover between the two.  We should decide on a customization style and be consistent about it.

Really good point. I'm inclined to standardise on customisations as __init__ args, as it encapsulates complexity within the Flask-SQLAlchemy lib, rather than pushing it down to users who would potentially have to refactor quite a bit of their application in order to start making use of a customisation.
This approach could cause __init__ methods to become complex and ungainly long term as more customisations are added, but that would seem to me to be a lesser evil.

For similar reasons as @immunda, I'm a fan of the __init__ approach.  Further, and it might not be entirely rational, needing to override methods reminds me of the pure virtual methods of abstract base classes; if there are defaults that are functional, I think specifying changes in default behavior in the initializer is cleaner than just relying on MRO.

I think there's a good argument for both styles. While some things like specifying a query or model class can only really be implemented by altering SQLAlchemy.__init__, other things could be handled by introducing or encouraging subclassing.
I'd say if it's a specialization, use a subclass. If it broadens use, change __init__. But that's me.

Further input on this from other folks?

There's one other pattern, which off the top of my head I know Flask-Login uses: class variables and decorator methods on the extension object.  If you want to override something, you either assign it to the extension, or decorate it with a method.  That could be interesting to explore as well.

@davidism Something like:
@db.something
def do_the_stuff(...):
    ...

I've used this pattern in Flask-Transfer and found it very handy for callbacks.

Customizing most of this via __init__ args feels like the simplest solution to me.

Seems like we're settling on __init__ args, @davidism given you raised this, any objection if we call that consensus?

I'm calling it! __init__ it is 😁


[Designer] Background image not occupying entire container

Problem Description
When you add a background image to a widgets board container you can see a slight white border across the widgets board container. The background image is either not occupying the entire container, or the container size is larger than it should be. It is much more noticeable on the right side of the card. The corners also show this, and I believe there might also be a tiny bit on the left side, but can't tell completely due to the dotted lines around the card.
Card JSON
{
    "type": "AdaptiveCard",
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.6",
    "backgroundImage": {
        "url": "https://coolbackgrounds.io/images/backgrounds/black/black-contour-f41038db.svg"
    }
}

Screenshot

@anna-dingler would you be able to take a look at this when you get a chance?

Hi , My issue is when we are adding the background image in the post adaptive card using the url from google the  image was not visible in the background of teams notification using the html code in send an notification(v3)  in the flow of  power automate These are the data i am runnin gin the flow
body {
margin: 0;
 padding: 16px;
background-image: url('https://img.freepik.com/premium-vector/abstract-modern-blue-white-gradient-background_181182-131.jpg');

Hi @ibrahimirfan17,
I am closing this issue because the original bug has been fixed.
Can you please file a new issue on our repo and include a sample card payload that shows the bug?
Thank you!
Anna


Expansion not preserving label neighbourhood

Doing this contraction then expansion in 3D does not preserve the label neighbourhood of the (yellow) 3D scalar:
→→
I think the bug is probably in expansion.
Data diff (left (red) is expected original neighbourhood, right (green) is observed incorrect neighbourhood):
Diff < left / right > :
 DiagramN {
     source: DiagramN {
         source: DiagramN {
             source: Diagram0(
                 0:0⁺,
             ),
             cospans: [],
         },
         cospans: [],
     },
     cospans: [
         Cospan {
             forward: RewriteN {
                 dimension: 2,
                 cones: [
                     Cone {
                         index: 0,
                         internal: ConeInternal {
                             source: [],
                             target: Cospan {
                                 forward: RewriteN {
                                     dimension: 1,
                                     cones: [
                                         Cone {
                                             index: 0,
                                             internal: ConeInternal {
                                                 source: [],
                                                 target: Cospan {
                                                     forward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                     backward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                 },
                                                 regular_slices: [
                                                     Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     1,
                                                                 ),
                                                                 [
                                                                     Regular(
                                                                         0,
                                                                     ),
                                                                 ],
                                                             ),
                                                         ),
                                                     ),
                                                 ],
                                                 singular_slices: [],
                                             },
                                         },
                                     ],
                                 },
                                 backward: RewriteN {
                                     dimension: 1,
                                     cones: [
                                         Cone {
                                             index: 0,
                                             internal: ConeInternal {
                                                 source: [],
                                                 target: Cospan {
                                                     forward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                     backward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                 },
                                                 regular_slices: [
                                                     Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     1,
                                                                 ),
                                                                 [
                                                                     Regular(
                                                                         0,
                                                                     ),
                                                                 ],
                                                             ),
                                                         ),
                                                     ),
                                                 ],
                                                 singular_slices: [],
                                             },
                                         },
                                     ],
                                 },
                             },
                             regular_slices: [
                                 RewriteN {
                                     dimension: 1,
                                     cones: [
                                         Cone {
                                             index: 0,
                                             internal: ConeInternal {
                                                 source: [],
                                                 target: Cospan {
                                                     forward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                     backward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                 },
                                                 regular_slices: [
                                                     Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
<                                                                    0,
>                                                                    1,
                                                                 ),
                                                                 [
                                                                     Regular(
                                                                         0,
                                                                     ),
<                                                                    Regular(
<                                                                        0,
<                                                                    ),
                                                                 ],
                                                             ),
                                                         ),
                                                     ),
                                                 ],
                                                 singular_slices: [],
                                             },
                                         },
                                     ],
                                 },
                             ],
                             singular_slices: [],
                         },
                     },
                 ],
             },
             backward: RewriteN {
                 dimension: 2,
                 cones: [
                     Cone {
                         index: 0,
                         internal: ConeInternal {
                             source: [],
                             target: Cospan {
                                 forward: RewriteN {
                                     dimension: 1,
                                     cones: [
                                         Cone {
                                             index: 0,
                                             internal: ConeInternal {
                                                 source: [],
                                                 target: Cospan {
                                                     forward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                     backward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                 },
                                                 regular_slices: [
                                                     Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     1,
                                                                 ),
                                                                 [
                                                                     Regular(
                                                                         0,
                                                                     ),
                                                                 ],
                                                             ),
                                                         ),
                                                     ),
                                                 ],
                                                 singular_slices: [],
                                             },
                                         },
                                     ],
                                 },
                                 backward: RewriteN {
                                     dimension: 1,
                                     cones: [
                                         Cone {
                                             index: 0,
                                             internal: ConeInternal {
                                                 source: [],
                                                 target: Cospan {
                                                     forward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                     backward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                 },
                                                 regular_slices: [
                                                     Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     1,
                                                                 ),
                                                                 [
                                                                     Regular(
                                                                         0,
                                                                     ),
                                                                 ],
                                                             ),
                                                         ),
                                                     ),
                                                 ],
                                                 singular_slices: [],
                                             },
                                         },
                                     ],
                                 },
                             },
                             regular_slices: [
                                 RewriteN {
                                     dimension: 1,
                                     cones: [
                                         Cone {
                                             index: 0,
                                             internal: ConeInternal {
                                                 source: [],
                                                 target: Cospan {
                                                     forward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Source,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                     backward: Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
                                                                     Target,
                                                                     2,
                                                                 ),
                                                                 [],
                                                             ),
                                                         ),
                                                     ),
                                                 },
                                                 regular_slices: [
                                                     Rewrite0(
                                                         0:0⁺,
                                                         2:3⁺,
                                                         Some(
                                                             (
                                                                 2,
                                                                 BoundaryPath(
<                                                                    Target,
<                                                                    0,
>                                                                    Source,
>                                                                    1,
                                                                 ),
                                                                 [
<                                                                    Regular(
<                                                                        0,
<                                                                    ),
                                                                     Regular(
                                                                         0,
                                                                     ),
                                                                 ],
                                                             ),
                                                         ),
                                                     ),
                                                 ],
                                                 singular_slices: [],
                                             },
                                         },
                                     ],
                                 },
                             ],
                             singular_slices: [],
                         },
                     },
                 ],
             },
         },
     ],
 }

This was caused by e9637e4078870f0d585b5ece60b1e31a8945e268 which we are reverting, so I am closing this.


Make multiple languages limited to the language of the entry lg file

If we have an entry file: a.xxx.lg, (xxx is the locale, en-us, en, fr-fr .etc.)all the import file while is imported by this entry file or the children file, would fallback language from xxx.
So, if a user uses a.en.lg as entry file, and import b.en-us.lg, it would make some exception because LG would find a.lg file from b.en.lg and with the fallback file b.lg, even if the current locale is "en-us".

Pull Request Test Coverage Report for Build 99189

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.9%) to 78.008%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/MultiLanguageGeneratorBase.cs
1
83.33%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/TemplateEngineLanguageGenerator.cs
6
82.86%

Totals

Change from base Build 99136:
-0.9%

Covered Lines:
20807

Relevant Lines:
26673

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.ApplicationInsights.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll compared against version 4.6.3

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.ApplicationInsights.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll compared against version 4.6.3


Improve performance in recognizer with similar regexes

There are some implementations of PatternRecognizer that export several regexes in the patterns array because the small difference between the regexes influences the score. An example of this is UsItinRecognizer.
PATTERNS = [
    Pattern(
        "Itin (very weak)",
        r"\b9\d{2}[- ](5\d|6[0-5]|7\d|8[0-8]|9([0-2]|[4-9]))\d{4}\b|\b9\d{2}(5\d|6[0-5]|7\d|8[0-8]|9([0-2]|[4-9]))[- ]\d{4}\b",  # noqa: E501
        0.05,
    ),
    Pattern(
        "Itin (weak)",
        r"\b9\d{2}(5\d|6[0-5]|7\d|8[0-8]|9([0-2]|[4-9]))\d{4}\b",  # noqa: E501
        0.3,
    ),
    Pattern(
        "Itin (medium)",
        r"\b9\d{2}[- ](5\d|6[0-5]|7\d|8[0-8]|9([0-2]|[4-9]))[- ]\d{4}\b",  # noqa: E501
        0.5,
    ),
]

The difference between those 3 Patterns is the separators - and space in an ITIN, matching the patterns 9XX-XX-XXxX, 9XX-XXXXXX , 9XXXX-XXXX or 9XXXXXXXX. The 3 regexes are necessary for the current implementation because they represent different pattern names and scores.
Having multiple regexes is not optimal, since every regex needs to match every text that needs to be analyzed. Reducing the number of regexes while keeping different pattern names and scores would improve the Presidio performance.
Proposal

Add a get_pattern_from_match(pattern: Pattern, regex_match) -> Pattern method to PatternRecognizer
This method will return by default in PatternRecognizer the same pattern passed in parameters
It can be overridden by specific recognizers, like UsItinRecognizer
Some recognizers now can reduce the number of regexes and implement get_pattern_from_match. Let's take a look at UsItinRecognizer:

The 3 regexes can be reduced to one, that matches the separators [- ] in regex named groups
r"\b9\d{2}(?P<firstSeparator>[- ])?(5\d|6[0-5]|7\d|8[0-8]|9([0-2]|[4-9]))(?P<secondSeparator>[- ])?\d{4}\b"
The implementation of get_pattern_from_match in  UsItinRecognizer will return different instances of Pattern by checking if the groups firstSeparator and secondSeparator have content in the match info.
To check the groups is way more optimal than having 3 different regexes that need to match the entire text.

Detected recognizers that could be improved

AbaRoutingRecognizer: 2 regexes that could be reduced to 1. A similar case to UsItinRecognizer but with the - symbol as the separator.
AuAbnRecognizer: 2 regexes that could be reduced to 1. A similar case to UsItinRecognizer but with space as the separator.
AuAcnRecognizer: 2 regexes that could be reduced to 1. A similar case to UsItinRecognizer but with space as the separator.
AuMedicareRecognizer: 2 regexes that could be reduced to 1. A similar case to UsItinRecognizer but with space as the separator.
AuTfnRecognizer: 2 regexes that could be reduced to 1. A similar case to UsItinRecognizer but with space as the separator.
DateRecognizer: The following date formats could be reduced to one regex: mm/dd/yyyy, mm/dd/yy, dd/mm/yyyy, dd/mm/yy, mm-dd-yyyy, dd-mm-yyyy, dd.mm.yyyy, dd.mm.yy, dd-MMM-yyyy, dd-MMM-yy.
SgFinRecognizer: 2 regexes that could be reduced to 1. The [STFG] part could be checked on get_pattern_from_match.
UrlRecognizer: 2 regexes that could be reduced to 1. The url schema could be checked on get_pattern_from_match.
UsItinRecognizer: Case initially mentioned
UsPassportRecognizer: 2 regexes that could be reduced to 1. If the 1rst character is a digit or a letter could be checked on get_pattern_from_match.

Hi @aperezfals, this is a great suggestion. We'll review it internally and get back to you.

I would be happy to implement this in a PR if the internal feedback is good :)


WARNING: IdP wants authn requests signed

Hi
we have been trying to get the SAML plugin working with our new Jenkins setup in AWS.
When I configure the plugin and enter the metadata from the IdP and try to access jenkins via Okta, we get the following errors in the logs and the login just fails.
Apr 27, 2015 5:31:37 PM org.pac4j.saml.sso.Saml2WebSSOProfileHandler sendMessage
WARNING: IdP wants authn requests signed, it will perhaps reject your authn requests unless you provide a keystore
Are you able to help us with what we are missing here?
I'm not using https for jenkins, I don't know if that's a requirement for using this SAML plugin.
thanks
Gregor

I've used this with Okta in AWS, so I think you should be able to get it to work. I don't have time to help track down the problem, but let me know if you figure anything out

Hi, thank you for filing this issue. Apologies for closing this issue, but the right place to track is on https://issues.jenkins-ci.org with component "saml-plugin". We are removing the issue tracker from this repo to avoid further confusion. Please refile on the JIRA


Can we enable lazy download when LheInputFiles is true

It looks like the lhe eos space is exported via xrootd, and therefore we should be able to run pLHE type of workflows at other place than CERN, but having the lazy download should be enabled so as to limit the read.
How does one change that ?

@vlimant - is this one still relevant at all?  If so, maybe it's something @khurtado could do.

it relevant for the pLHE workflows

Yes, I suggest we first try running a remote workflow just to check whether LHE files can really be accessed from outside CERN (I see no reason why not though).
If it works fine, we need then to instrument EventBased algo to add a baggage to the job flagging it reads LHE files.
Reopening and assigning it to @khurtado for April. Kenyi, please ping me when you start working on this and I can give you further directions.

As discussed in the CompOps meeting, this feature will be important for some requests/campaigns that will hit the production system at any time.
Given that Kenyi is busy with the condor stuff, and other things, let me take this one.


Add support to 'listen' for tags

This adds support to break the read_passive_target function up into two parts if desired. First tell the PN532 to listen for tags, and then explicitly ask to read the UID from the tag. This can be useful when trying to simultaneously scan tags and run timing sensitive code, like updating neopixels, and don't want to wait for the timeout delay of read_passive_target.
This addresses #36.

I will test out the code from this PR this week.

@FoamyGuy wow thanks!

nice!

Thanks @FoamyGuy & @ladyada!

You are welcome. And thank you as well @dunkmann00, this functionality is a great addition that allows the PN532 to be more easily used in complex / interesting projects. I appreciate you working on this and adding it to the Library.


Incorrect Window Size - BEATs

Describe the bug
Model I am using (UniLM, MiniLM, LayoutLM ...): BEATs
The problem arises when using:

[x] the official example scripts: (give details below)
[ ] my own modified scripts: (give details below)

A clear and concise description of what the bug is.
When I try to use a finetune trained model of BEATs with my own audio file, I get the following error. The code is identical to the load Fine-tuned Models section of the README in BEATs except that I load own audio file with torchaudio and then use its size to create the padding mask. I've ensured that the sampling rate of the audio file is 16khz.

To Reproduce
Steps to reproduce the behavior:

Replace audio_input_16khz by using torchaudio.load() with your own ~10s audio file.
Replace padding_mask with the size of the loaded audio file.

Expected behavior
A clear and concise description of what you expected to happen.
Labels of the audio file should be printed.

Platform: Google Colaboratory
Python version: 3.8
PyTorch version (GPU?): 1.13.1

Hi @jeremyng353 , please make sure the "audio_input_16khz" variable is a torch tensor rather than the tuple of (tensor, sample_rate), which is the returned value of torchaudio.load() function.


[FEATURE] Add type to action/function const variables

Description
The generated const variables for actions/functions should include a __type property that is either "function" or "action".
Suggested Solution
Add a __type property to const variables of operations whose value is either "function" or "action".
// operation
export declare const fSingleParamSingleReturn: { (val: Books | null): Book | null, __parameters: {val: Books | null}, __returns: Book | null, __type : "function" };
// operation
export declare const aSingleParamSingleReturn: { (val: Books | null): Book | null, __parameters: {val: Books | null}, __returns: Book | null, __type : "action" };

Alternatives
No response
Additional Context
This information can be used to dynamically decide whether to use a POST or GET request.
Our backend (CAP) transpiles the generated types by cds-typer and exposes it to the frontend (UI5). With those types we can typesafe call the services of the backend from the frontend. However, we're missing the information whether an operation is an action or function.

Hi Daniel,
the approach using kind works for me, thank you.

May I ask how your process for exposing the transpiled types looks like?

I'll set up an example repo showcasing it.

Hi Daniel,
I've set up an example repo demonstrating the usage of cds-typer on both the backend and frontend github.com/aschmidt93/cds-typer-fullstack-example.
Note that this is in an early stage and many improvements need still to be made.


Add fit-tool integration

Thanks for the great project! If possible could you add a rudimentary fit tool akin to the TBrowser-included tool (might be out of scope / technically impossible)?

And thanks for this issue too, @dnwerner!
I originally thought about adding a command to the extension that opens an integrated TBrowser and let's you do everything that you would expect, but I don't think this is possible with JavaScript ROOT -- maybe @linev can let us know if there are any plans to support the TBrowser class within JSROOT?

No - fitting will require rewriting a lot of code, which is implemented in ROOT.
Therefore in such case one have to install and use ROOT directly

Thanks for another very quick reply and for the info @linev!
@dnwerner I'm going to close this issue as it indeed seems to be out of scope


Enumerable not implemented for Ecto.Date

I decided to use proper_case in my project. The experience was wonderful. It was very easy to integrate it. I however have an issue.
I am trying to call an api endpoint that returns a json that has field with Ecto.Date type. This is throwing an error.
 ** (Protocol.UndefinedError) protocol Enumerable not implemented for #Ecto.Date<2016-12-20>
        (elixir) lib/enum.ex:1: Enumerable.impl_for!/1
        (elixir) lib/enum.ex:116: Enumerable.reduce/3
        (elixir) lib/enum.ex:1627: Enum.reduce/3
        (proper_case) lib/proper_case.ex:14: anonymous fn/2 in ProperCase.to_camel_case/1
        (stdlib) lists.erl:1262: :lists.foldl/3
        (elixir) lib/enum.ex:1184: Enum."-map/2-lists^map/1-0-"/2
        (proper_case) lib/json_encoder/camel_case.ex:2: ProperCase.JSONEncoder.CamelCase.encode_to_iodata!/1
        (phoenix) lib/phoenix/controller.ex:642: Phoenix.Controller.do_render/4
        (portal_api) web/controllers/v1/academic_session_controller.ex:1: PortalApi.V1.AcademicSessionController.ac
tion/2
        (portal_api) web/controllers/v1/academic_session_controller.ex:1: PortalApi.V1.AcademicSessionController.ph
oenix_controller_pipeline/2
        (portal_api) lib/portal_api/endpoint.ex:1: PortalApi.Endpoint.instrument/4
        (portal_api) lib/phoenix/router.ex:261: PortalApi.Router.dispatch/2
        (portal_api) web/router.ex:1: PortalApi.Router.do_call/2

Please kindly see to it soon so that I fix this on my project
I also decided to try the demo project that I use to see how easy it could be to integrate proper_case.
The view below results in an error
def render("user.json", %{user: user}) do
    %{id: user.id,
      first_name: user.first_name,
      last_name: user.last_name,
      email: user.email,
      encrypted_password: user.encrypted_password,
      inserted_at: user.inserted_at
    }
  end

because of the inserted_at field that I introduced.

@smithaitufe Sorry for taking so long to respond. @expede has submitted a PR that I think will solve your issue :)


How to disable pagination for relationships?

Given 2 resources:
jsonapi_resources :companies
jsonapi_resources :users

User has_many :companies
default_paginator = :paged
/companies request is paginated and that's what I want. But I also want to disable it for relationship request /users/4/companies. How to do this?

This hacky solution from Guy Segev seems to work: stackoverflow.com/questions/41215951/disable-pagination-for-relationships/42006163
class CustomNonePaginator < JSONAPI::Paginator
  def initialize
  end

  def apply(relation, _order_options)
    relation
  end

  def calculate_page_count(record_count)
    record_count
  end
end

class JSONAPI::RequestParser
  def parse_pagination(page)
    if disable_pagination?
      @paginator = CustomNonePaginator.new
    else
      original_parse_pagination(page)
    end
  end

  def disable_pagination?
     # your logic here
     # request params are available through @params or @context variables
     # so you get your action, path or any context data
     @params[:action] == 'get_related_resources'
  end

  def original_parse_pagination(page)
    paginator_name = @resource_klass._paginator
    @paginator = JSONAPI::Paginator.paginator_for(paginator_name).new(page) unless paginator_name == :none
  rescue JSONAPI::Exceptions::Error => e
    @errors.concat(e.errors)
  end
end


[FR] Disable mypy and pylint check

FR has been rebranded to DI, so disable mypy and pylint check in this package to avoid ci failing from new mypy/pylint versions.

API change check
API changes are not detected in this pull request.

Wait why?
Aren't you using a fixed version of mypy ? (to avoid breaking changes)

Wait why?
Aren't you using a fixed version of mypy ? (to avoid breaking changes)

No, we'll upgrade mypy version to 1.6.1 from 2024-01-15. cc: @kristapratico


SiteFactory Uses Same API As Sites List Command

Here's a start of work to resolve #457.  Largely untested outside of being able to terminus drush uli --site=<org site>.  I'm also unsure about how to handle the sites in build state.  Is there an easy way to get an organization site "stuck" in a build state to use for figuring this out?

@bthompson-mindgrub The entirety of SiteFactory is being thrown out in my next pull request, in preference to a Sites collection.

@TeslaDethray Nice!


bug(md-tooltip): maximum call stack exceeded when tooltip-position is not "below" - RC6

Angular @ 2.0.0-rc.6
Material @ 2.0.0-alpha.7-12
When tooltip-position is set to something else than default (eg. after) the site completely breaks with Maximum call stack size exceeded exception.
Since there is no base plunker with these versions I won't be posting one. The error is easily reproducible though: change https://github.com/angular/material2/blob/master/src/demo-app/tooltip/tooltip-demo.ts#L12 from below to after and run the demo app.
Edit: Haven't really looked at the code in depth, but it seems like this line causes the error. It creates multiple overlayRef for a single tooltip. Removing it makes the component work perfectly. Although it didn't throw before rc.6, it still created way too many overlayRef.

I have the same error with v2.0.0-alpha.8-1

Same here... before RC.6 everything worked fine. For me default "below" value also throws an error, but in this case the element that the tooltip is set to is shown. Stacktrace:
ORIGINAL EXCEPTION: Maximum call stack size exceeded
ErrorHandler.handleError @ core.umd.js:5997
next @ core.umd.js:9458
schedulerFn @ core.umd.js:8705
SafeSubscriber.__tryOrUnsub @ Subscriber.js:223
SafeSubscriber.next @ Subscriber.js:172
Subscriber._next @ Subscriber.js:125
Subscriber.next @ Subscriber.js:89
Subject.next @ Subject.js:55
EventEmitter.emit @ core.umd.js:8697
onError @ core.umd.js:8922
onHandleError @ core.umd.js:8793
ZoneDelegate.handleError @ zone.js:196
Zone.runTask @ zone.js:128
ZoneTask.invoke @ zone.js:293
core.umd.js:6000

All angular2-material modules I have in version 2.0.0-alpha.8-1 and import them as shown in your all.ts file (without the dialog module that is not yet published to npm).

Angular @ 2.0.0-rc.7
Material @ 2.0.0-alpha.8-1
Same problem.

Updated ng2 to 2.0 final release, material still at 2.0.0-alpha.8-1.
Same problem.

Angular @ 2.0.0
Material @ 2.0.0-alpha.7-4
Same Problem

Whatever position is set as the default works. If I manually change the predefined position inside the tooltip.js file that position will work and all of the others fail.

Angular @ 2.0.0-rc.5
Material @ 2.0.0-alpha.8-2
Same Problem

Issue still present in
Angular @ 2.0.1
Material @ 2.0.0-alpha.9-3

Got the same thing :)

@andrewseguin can you confirm that this issue is fixed on master?

Cannot reproduce the issue in master now that tooltip logic has been drastically changed. Looks like this issue was fixed.


Add version: JetBrains.Rider version 222.3962.23

Result: Installation Failed

Manifest
Add/Remove Programs

Name
JetBrains Rider
PyCharm 2022.2.1

Version
222.3962.23
222.3739.56

Publisher
JetBrains s.r.o.
JetBrains s.r.o.

ProductCode

PyCharm 2022.2.1

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 2808
Microsoft Reviewers: Open in CodeFlow

Close with reason: Already merged;


Can't load files with spaces in build

Images referenced in Vue components do not show up in release build gotten by doing "yarn tauri:build". The images do show up in the dev build gotten with "yarn tauri:serve".
Tauri info:

Dependencies:

It seems like the custom protocol doesn't support files with spaces in the names...we'll definitely need to fix this.

same problem
https://github.com/Manonicu/remix
branch dev


Change

How do I add a username

I'm not sure what widget you're talking about. Is this for Twitter Widget? If so, you can see the documentation on the GitHub page for how to set username in the widget parameter

I have looked but can’t seem to change
Sent from John Kerby

On Mar 21, 2021, at 12:42 PM, mvan231 @.***> wrote:

I'm not sure what widget you're talking about. Is this for Twitter Widget? If so, you can see the documentation on the GitHub page for how to set username in the widget parameter
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I'm confused by what you mean. The instructions just state to use the widget settings to set the widget parameter. Did you not see that option there?

I'm going to close this issue. If you still have problems, please comment again here


Schedule weekly calls with hackEDU Cohort 1 & 2

This is something that we discussed doing before but never executed on.
A weekly call time has been setup for:

[ ] Cohort 1 by Harrison
[ ] Cohort 2 by Vene

/cc @hellyeah @zachlatta

We should have a point person in each cohort who takes this on.
Someone who's organized and always available.

@zachlatta, I'm happy to take this task for Harrison and Vene, thoughts?

Sounds good.

@paked is going to work on this when he gets a chance. He's currently very busy. Because this task is not complete, I won't check off cohort 1's checkbox.
I reached out to Victor to see if I can chat with him over the phone to talk about this. Waiting for a response.

:+1:
On Thu, Sep 10, 2015 at 3:25 PM, Jonathan Leung<EMAIL_ADDRESS>wrote:

@paked https://github.com/paked is going to work on this when he gets a
chance. He's currently very busy. Because this task is not complete, I
won't check off cohort 1's checkbox.
I reached out to Victor to see if I can chat with him over the phone to
talk about this. Waiting for a response.
—
Reply to this email directly or view it on GitHub
https://github.com/hackedu/meta/issues/137#issuecomment-139399636.

--
Zach Latta
hackEDU |<EMAIL_ADDRESS>

Just had a call with Victor to confirm this.
Status now is that both @paked and Victor are aware of the plans and will most likely execute on this this coming week.

Can confirm. Am @paked.
This will be done ASAP.
On Sun, Sep 13, 2015 at 11:08 AM Jonathan Leung<EMAIL_ADDRESS>wrote:

Just had a call with Victor to confirm this.
Status now is that both @paked https://github.com/paked and Victor are
aware of the plans and will most likely execute on this this coming week.
—
Reply to this email directly or view it on GitHub
https://github.com/hackedu/meta/issues/137#issuecomment-139833174.

OK. I'm setting this up now. Just pinged the messenger chat to generate some interest.

Progress update: We are having an initial call at some point this weekend. Most people seem interested.

Can you push to get everyone on that call? I think it's important that we
set the precedent that the cohort is as only strong as its weakest link
from the very beginning.
On Tue, Sep 15, 2015 at 2:54 PM, Harrison Shoebridge <
<EMAIL_ADDRESS>wrote:

Progress update: We are having an initial call at some point this weekend.
Most people seem interested.
—
Reply to this email directly or view it on GitHub
https://github.com/hackedu/meta/issues/137#issuecomment-140557768.

--
Zach Latta
hackEDU |<EMAIL_ADDRESS>

@zachlatta I'm going to try. However it seems that me doing it doesn't have
the same prestige, as say you or @jonleung organizing it. IMO this is
something which will have to be built upon, instead of starting strong from
day 1.
On Wed, Sep 16, 2015 at 7:56 AM Zach Latta<EMAIL_ADDRESS>wrote:

Can you push to get everyone on that call? I think it's important that we
set the precedent that the cohort is as only strong as its weakest link
from the very beginning.
On Tue, Sep 15, 2015 at 2:54 PM, Harrison Shoebridge <
<EMAIL_ADDRESS>wrote:

Progress update: We are having an initial call at some point this
weekend.
Most people seem interested.
—
Reply to this email directly or view it on GitHub
https://github.com/hackedu/meta/issues/137#issuecomment-140557768.

--
Zach Latta
hackEDU |<EMAIL_ADDRESS>—
Reply to this email directly or view it on GitHub
https://github.com/hackedu/meta/issues/137#issuecomment-140557960.

I agree with @paked that if this is organic, it will be hard to get everyone to be on the call.
The call just happened earlier today and wasn't too successful. I'll leave @Bogidon to comment why.

I'm leaving this an open issue here because I'll be doing work throughout the week to make sure that another hopefully more successful call happens next week.

Can you close this and create a new issue to track the call for this week?
I think that may be a bit more clear.
On Sat, Sep 19, 2015 at 7:32 PM, Jonathan Leung<EMAIL_ADDRESS>wrote:

I'm leaving this an open issue here because I'll be doing work throughout
the week to make sure that another hopefully more successful call happens
next week.
—
Reply to this email directly or view it on GitHub
https://github.com/hackedu/meta/issues/137#issuecomment-141729978.

--
Zach Latta
hackEDU |<EMAIL_ADDRESS>

@zachlatta, yup I added your suggestion to my queue of work.

Closing because @zachlatta made a new issue for this week's sprint.


The details for Standard Banner is supposed to say "4 star" instead of "5 star" in the bottom part.

Thanks for the attention, some translations still don't include the variable to change the rarity automatically, so I've fixed it, I hope it shows up as expected by now.


Fix limit to display the best-sellers products

Questions
Answers

Branch?
dev

Description?
when we use Configuration::get('config_var',10) return false cause it doesn't consider 10 as default_value but as lang_id

Type?
bug fix

Category?
MO

BC breaks?
no

Deprecations?
no

Fixed ticket?
http://forge.prestashop.com/browse/PSCSX-9296

How to test?
BO->dashboard->block "bestsellers", the products are well displayed

@Quetzacoalt91 done ;)

The line initially concerned by the PR is okay for me. However, I will need more explanation on the other parts of the SQL request before reviewing the other commit.
Thanks!

Thank you @hibatallahAouadni


Webcam gets rotated on android devices

Hi, On editor its working fine, but on android devices, the webcam gets rotated. Please check and update.

You can mirror or rotate the RawImage in editor.

Thanks! Did that and it's working now.
Want to ask one more thing, will this plugin work for ios too?

Yes. But you need Mac and xcode to rebuild.

Hi, Thanks for the reply,
Yes, I am building it on Mac and in xcode, but it's working on the iOS devices, Are there any specific changes that need to be done for the iOS?

I don't quite understand your question. You said "you build on Mac and xcode, but it's working on the iOS device" If it's working on iOS devices, what's your question?

Sorry for the writing mistake, "not" is missing in that, actually it is
am building it on Mac and in xcode, but it's not working on the iOS devices

Did you follow the instructions here?
https://docs.unity3d.com/Manual/UnityCloudBuildiOS.html
Maybe some permission you didn't set on.

Hi,
I followed all the Unity xcode build instructions and also given camera permission. My build is working fine on ios phone. Camera permission is okay and working fine but not scanning the QR code, this is the only issue. On android everything working fine. I am stuck here. Please help!

That could be ZXing library issue. You can try to debug to see if ZXing can read the bar code or not.
Or even replace my ZXing library with any other 3rd party .net QR library.
https://www.google.com/search?q=.+net+qr+library


[Storage] Add warning for passing prefix to list_blobs API

There have been a couple customer incidents where passing prefix seems to fall off and do nothing. Internally it is prefix=, but somewhere along the way of Track1 -> Track2 migration, the parameter for the customer-facing API changed to name_starts_with.
To alleviate this, we will raise a warning regarding this behavior.

API change check
API changes are not detected in this pull request.

Bump to rehydrate

One day.. ™


feat(experimental): render katex with math-api

However, it still depends on the math shortcode and it is still difficult to get around goldmark.


Resque worker skips resizing job

I am currently trying out pageflow. Uploading images to S3 seems to work fine, but it stops at the 'Processing'-step.
The resizing.log reads the following:
I, [2018-05-01T17:48:24.316050 #12535]  INFO -- : Pageflow::ProcessFileJob - perform for Pageflow::ImageFile 11
I, [2018-05-01T17:48:24.537290 #12535]  INFO -- : Pageflow::ProcessFileJob - Pageflow::ImageFile 11 not found. Skipping job.

I've gone through the Troubleshooting section, but with no luck. ImageMagick is installed.
Any help would be much appreciated, thanks!

This log output occurs when the record for the job cannot be found in the database. This can happen, for example, if the image has already been deleted again before the processing job ran. If it happens consistently, it can be a sign that the web server and the Resque workers are looking at different databases. The Resque workers could be running in a different Rails env for example (i.e. development vs production). Could this be the case here?

This indeed seemed to be the case. Now it says processing failure ;) but I'll figure that out myself. Thanks for your help, @tf


[API] Prefix for frontend and backend

It would be cool to have a prefix (eg: /api/v1/) for the backend and another (eg: /api-front/) to the frontend!
Choice would be in the API package:
example: {
    type: 'POST', //(['GET']|'PUT'|'PATCH'|'POST'|'DELETE') [optional]
    https: true, //(true|[false]) must call this api method via HTTPS [optional]
    url: '/example?id={id}', //([/apiPrefix/apiPackageName]|/<apiPathnameAfterPrefix>) [optional]
    target: 'frontend' //(['backend']|'frontend']) [optional]
}

I haven't encountered such use case before, will look into it.

@Billybob After further thought and investigations, I think a better approach would be to create serve clients (e.g. https://github.com/floralvikings/jira-connector) if you need to access more than 1 endpoint in your app. Try to generalize the logic and conventions for different endpoints seemed not to be practical.


Consider changing the order of arguments for set_resp_cookie/4

Have Opts at the end like all other functions. We might want to allow both orders with a warning for the deprecated one for a while.

Done locally, just need to commit and push tomorrow.


Add a new optional http VM bootstrap check method

First "really naive" proposal for #230
This is more a basis for discussion than a real proposal that can be merged as it is.
It must be discussed, refined and probably implemented in a more elegant way than that.
This implementation uses go stdlib http client without any tuning, and have added some tests related to checkMode selection. Since TLS-PSK ciphers are not supported yet ( https://github.com/golang/go/issues/6379 ) , I've drop the idea to leverage this kind of authentication / crypto between the controller and the VM for the moment.
It implements a new optional virtualMachineBootstrapCheck structure for KubevirtMachine & KubevirtMachineTemplate Custom Resources.
apiVersion: infrastructure.cluster.x-k8s.io/v1alpha1
kind: KubevirtMachineTemplate
metadata:
  name: yet-another-machine-template
spec:
  template:
    spec:
      virtualMachineBootstrapCheck: # optional
        mode: http # possible values are : "ssh" or "http" (defaults to ssh, for backward compatibility)
        port: 6440 # only used for http (defaults to 6440)
      virtualMachineTemplate:
        metadata:
          [...]
        spec:
          [...]

As an alternative to SSH, a new contract is expected on the VM side: a web server must listen on the chosen port (6440 by default) and expose the bootstrap-success.complete Sentinel File to http://<VM_IP>:<PORT>/bootstrap-success.complete.
There's currently no http auth (the bootstrap status is public), but with more time a TLS-PSK or mTLS mechanism could be imagined between controller and its managed VM. At this moment, Certs would need to be managed by the controller the same way as SSH keys.
In this patch, SSH keys are still managed, injected and mandatory to the controller global behaviour. But this point could be perfectly enhanced to make SSH related stuff completely optional when Bootstrap Check is made through HTTP. But I don't consider it as a blocker : at the moment, we can perfectly keep ssh management even if bootstrap file check is not done through SSH
Let me know if this makes sense for you or not (and if we should dig deeper in this direction !).
Thanks

Following last Syncup Meeting, we dropping this idea in favor of :

short term : bypassing Sentinel file check with a "none" policy
mid term : working on a alternate qemu-guest-agent usage to check file
long term : making the whole SSH logic optional (for keygen & cloud-init management...) with something like a Sentinel File check mode selector in spec

closing


No arguments being passed into CFLint

Currently, if someone just runs:
java -jar ./CFLint-1.4.0-SNAPSHOT-all.jar
they're getting the help prepended with
Set -folder or -stdin to use CFLint
But is that the right messaging? For instance, it can most certainly being used with -file alone, can't it? What's the minimum (set of) cli argument(s) that needs to be supplied?

-folder and -file are essentially aliases
On Mon, Jan 1, 2018 at 1:57 AM, Kai Koenig<EMAIL_ADDRESS>wrote:

Currently, if someone just runs:
java -jar ./CFLint-1.4.0-SNAPSHOT-all.jar
they're getting the help prepended with
Set -folder or -stdin to use CFLint
But is that the right messaging? For instance, it can most certainly being
used with -file alone, can't it? What's the minimum (set of) cli
argument(s) that needs to be supplied?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cflint/CFLint/issues/521, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFtZu78diuUn4rqcWMNWC3vwl8Gx6nBQks5tGIHVgaJpZM4RQAUB
.

I will fix the wording of this message in my branch for #329.


Squashed/rebased #1373

As requested by @jzaefferer in #1512, we have taken care of squashing/rebasing #1373 while maintaining appropriate authorship.

@Arkni missed this PR... could you rebase it against master?

This PR is a rebase of #1373 and this was opened 4 months ago under the presumption that it would be landed more/less in a timely manner (i.e. without merge conflict). What's the guarantee that this second rebase will be landed in a timely manner?

@twolfson I willing to land it and now also have the required permissions todo it :-)

Alright, taking a look at rebasing now

The test is failing due to this check (src/core.js#L643):
if ( element.hasAttribute( "contenteditable" ) ) {
	val = $element.text();
} else {
	val = $element.val();
}

The mocked input file does not have the |hasAttribute` method. A quick fix could be:
 test/methods.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/methods.js b/test/methods.js
index 0848357..cf29f04 100644
--- a/test/methods.js
+++ b/test/methods.js
@@ -37,7 +37,8 @@ function acceptFileDummyInput( filename, mimeType ) {
                type: "file",
                files: fileList,
                nodeName: "INPUT",
-               value: "/tmp/fake_value"
+               value: "/tmp/fake_value",
+               hasAttribute: function() { return false; }
        };
 }

Sorry, got side-tracked on other work. Thanks for the pointer. I will try that out in a bit

@Arkni That patch worked great, thanks =)
@staabm This PR should be good to go

Will land it tomorrow after beeing back on the oc. Thx so far.

Sweet, thanks =)

@twolfson landed it right now after a few minor changes in the commit message.
thank you!

Cool, thanks =)


featModify the init_be and start_be scripts to meet the requirements for rapid Docker startup.

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #45267
Problem Summary:
To meet the needs of rapid Docker startup, I have made adjustments to two related scripts in the Docker startup process. First, I added a parameter --skip to the start_be.sh script, which will skip the size checks for swap, ulimit, and max_map_count. At the same time, I used --console to start the process and print logs. The reason why I did not use the --daemon daemon command to execute is that starting with a foreground log printing method in a Docker container is the correct and reliable approach.
At the same time, I added a check logic for a be.conf configuration item in the init_be.sh script: if it is the first time starting, append the configuration item skip_check_ulimit = true to the be.conf file to skip the ulimit value check in the BE process. In summary, these adjustments can meet the basic requirements for rapid Docker startup usage.
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[x] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Set the ulimit, swap, and max_map_count check values on the host machine to values that do not meet the requirements.
Run sh start_be.sh --skip to start the BE process, which starts up normally.
Using the --daemon or --console option to start, the BE process fails to start normally.

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[x] Confirm the release note
[x] Confirm test cases
[x] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

run buildall


[WIP] 3728 - Add ability to search by user email to superuser user view

Resolves #3728
Description
I have added a second search bar for filtering by email as requested in issue #3728, as well as a clear button that will clear both search inputs. I chose to add the clear button because the two search inputs update the filters when any letter is added or removed to either search bar, and there is no dedicated search button.
I have added Rspec tests for the email search and clear button functionality, in spec/system/admin/users_system_spec.rb, to keep it with the user name search tests.
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I have manually tested the changes, and added additional tests to the users_system_spec.rb file as mentioned above.
To find the new search bar:
sign in as<EMAIL_ADDRESS>select Users > All Users in the sidebar

@Kernapillar this is marked as WIP - is it ready for review?

@Kernapillar this is marked as WIP - is it ready for review?

Yeah It is, I  set it as WIP because I wasn't sure if the "clear" button fits in the project well or not, I meant to leave a note about it but didnt get a chance.

@cielf Thanks for the feedback, I just pushed a version without the clear button and its tests, might as well keep it simple!

Just checked this out on staging.   So nice!


Add support for GraphQL::Pagination::Connection

Add support for GraphQL::Pagination::Connection.

Resolved with https://github.com/CompanyCam/graphql-searchkick/pull/8


Feature: number of visible items

Hey there !
Is it possible to configure the number of visible items ? It seems like it's 5 by default (2 above, the selected one, then 2 below).
Is it possible to customise it to get it down to for example 3 ? (one above, the selected one, and the one below)
Thanks !

Hi @Alaa-Ben!
There is no easy way to do this yet, but you can use a more complex way for any animation: redefine the renderItemContainer and calculate animation rules for each index based on the useScrollContentOffset and your data.

Thanks for the reply!
I've checked the source code and I see some occurrences of hard coded fives, etc.
Do you think making it customisable is interesting enough to make it a feature in this lib ?

Do you think making it customisable is interesting enough to make it a feature in this lib ?

Yes, it seems to me that this will be a useful feature!
Speaking from the point of view of the public API, it seems that it would be nice to provide the ability to simply specify the visible faces of the drum, for example, 3, 5, 7, etc.
But these are my first thoughts so far, maybe you have better ideas!

@Alaa-Ben Released in 1.1.0!)

Love it, very helpful thanks a lot !


Export results does not work in JATOS that builted on an aliyun cloud server

When I try to download all results with Results -> Export Results -> All, it fails to download the result file.
It pops out a small window which address is _http://<IP_ADDRESS>:9000/assets/lib/jatos-gui/streamsaver/mitm.html_  . Then I pressed F 12,  the following error shows on the console: Failed to load resource: the server responded with a status of 404 (Not Found)  ping:1      . When I click the "ping:1" , an additional message shows: Requested page /assets/lib/jatos-gui/streamsaver/ping couldn't be found.  The screenshots is attached at the bottom.
I am using JATOS 3.7.2 , which was installed on a Ubuntu 20.04 64bit system, using aliyun cloud server. My instance on the aluyun cloud server has 2CPU and 2GB memory, with 1MB bandwidth.  On my Jatos, there is only a 60KB test project and I am just downloading a 10KB test result file.   Though I have reinstalled several times and read many issues like #234 on GitHub issues here and others on CogSci forum, I don't see on how to apply them to the problem above.
Why it fails to download and how to fix it?  Any help is appreciated!

An additional thing is, I use the same jzip project as mentioned in the above running on my local JATOS, or running on the mindprobe JATOS, exporting result files works very well. So I am wondering that it might not be my HTML script's problem ...

Unfortunately the library JATOS uses to download the files has to open and iframe and request some additional files. Somehow this is blocked in your browser/network. You can try with a different browser or a different network (e.g. from home).
Best
Kristian

Thank you, Kristian.
I have test whether it is the Cloud Server's firewall problem. I closed the firewall and opened all the port as a test, it doesn't solve the problem.
It always prompts the message that it cannot find the file "ping" underlying the directory "assets/lib/jatos-gui/streamsaver" . The required file is missing so that the request failed.
Why it cannot find the ping file when it requests? Does something go wrong in the JATOS package?  Are there any similar errors about ping file missing when exporting results reported before?

Many thanks for your attention!

Okay, I was able to reproduce your issue: if I use JATOS without HTTPS and not on localhost this StreamSaver.js library behaves differently and opens an popup window instead of an iFrame. This 'ping' is actually not a resource that should exists on the server (JATOS), it's a way for StreamSaver.js to keep the service worker alive (as far as I understand). It intercepts every URL and checks if it has a 'ping'.
Now the question is why the StreamSaver.js doesn't start the download in the popup window. I don't know (yet). I hopefully can continue looking into this tomorrow. But if you just want to get it to work you can use HTTPS. The whole problem doesn't happen with a 'secure context'. But to have HTTPS you need a domain name. Do you have a domain for your server?

A fix is part of the next release 3.7.4


Feature Request: Configure by Biome Name

It would be fantastic to be able to have the option to configure the crops by Biome Name, as well as by Biome Type. When using a mod such as Biomes O Plenty… there are so many biomes that have similar types to one another, but are in reality completely different in terms of being conducive to growing specific crops. Combing a mod like BOP with a mod like HarvestCraft… and used in conjunction with BiomeCrops mod, would allow the user to have fine-grain control over specific crops growing in specific biomes, as opposed the limitless possibilities encountered when configuring by Biome Type.
Is that something you could implement rather easily?

Yeah, it's easy to implement. I was thinking it should have a "whitelist" and a "blacklist" like the current biome type system, where the biomes in the whitelist would be added after resolving the list from the biome types, then biomes from the blacklist removed after that. I think this would allow for fairly complete control.

That'd be perfect!


Feature request: Lint and format Jupyter notebooks exported to markdown with Jupytext

I have a use case for Ruff and Ruff formatter that is a bit related to some of the other Markdown / Docstring feature requests but specifically I hoped to run Ruff and Ruff formatter on Jupyter notebooks that had been exported to markdown with Jupytext.
The company I'm at prefer converting notebooks to Markdown as it makes the notebook diffs much easier to read on Bitbucket / GitHub.
At first I noticed I could add markdown as a target file format for Ruff formatter and linter which got my hopes up that this would just work:
  - repo: https://github.com/charliermarsh/ruff-pre-commit
    rev: v0.1.6
    hooks:
      - id: ruff-format
        types_or: [python, pyi, jupyter, markdown]
      - id: ruff
        args: [--fix, --exit-non-zero-on-fix]
        types_or: [python, pyi, jupyter, markdown]

But when I ran Ruff I see it is failing to parse the markdown properly - I had hoped it would just run on the python comment code blocks in the same way it would parse Jupyter notebook cells and ignore all the other markdown content but its obviously trying to parse all the markdown, e.g.

Hey, we currently don't have support for linting / formatting Python code in markdown blocks (https://github.com/astral-sh/ruff/issues/8237, https://github.com/astral-sh/ruff/issues/3792). I'll close this in favor of the markdown issue for bookkeeping purposes as I think that should solve this but correct me if I'm wrong here.

Hmm, actually it would be a bit different as for markdown we wouldn't need to have the concatenated source code from all code blocks but if it's a notebook converted to markdown then I think it should have context from other code blocks? @owenlamont Do you think this is true?
Another solution currently that I can think of is to lint / format before converting it to markdown. I'm not sure how feasible this would be given my lack of knowledge about your setup.

Hi @dhruvmanila - yeah it would have to have the concatenated source code - I can see Ruff still tracks which code was in which Jupyter cell when raising warnings so if it could treat comment blocks exactly as Jupyter cells are treated that would be ideal.
As a work-around it could be exported to ipynb, linted and formatted, then re-exported to markdown - but that would be onerous. When working with Jupytext the notebook never gets persisted (in any permanent/visible way) as an ipynb - it gets loaded from Markdown and saved back to Markdown.
The ideal solution (from my perspective) would be to parse the YAML front matter of the Markdown, identify this as a Juptext generated Markdown, then recognise the code blocks need to be concatenated and treated as notebook cells. I totally understand though if this use case is too niche to justify the effort though. I can't speak much as to how many people use this format - as a repo jupytext  is relatively popular (around 6k users - I recognise some relatively prominent Jupyter developers as contributors).

There's a similar request for quarto notebooks (#6140), and generally for Python code included in Markdown code blocks (#3792).

I think I'm looking at the same issue.  We also use Myst Markdown notebooks in several projects and have been using the jupytext ability to pipe code through black to apply Python formatting:
$ jupytext --pipe black docs/source/users/pmodel/c3c4model.md
[jupytext] Reading docs/source/users/pmodel/c3c4model.md in format md
[jupytext] Executing black -
reformatted -

All done! ✨ 🍰 ✨
1 file reformatted.
[jupytext] Writing docs/source/users/pmodel/c3c4model.md in format md:myst

We have that as part of a pre-commit hook to ensure that the code in our notebooks is properly formatted.
  - repo: https://github.com/mwouts/jupytext
    rev: v1.16.2
    hooks:
    - id: jupytext
      args: [--pipe, black]
      files: docs/source
      additional_dependencies:
        - black==24.4.2 # Matches hook

I haven't been able to work out exactly what happens, but the jupytext.cli module provides a pipe_notebook function that is used to round trip something (I think it must be just the code cell contents?) through black.

OK - so jupytext converts the notebook to percent format, which is a python file with the markdown content stored as comments. black can then run on the code alone and the format can be converted back.

Probably related: Ruff does currently not see py:percent files as notebooks. For example. for py:percent files, #8590 is still unresolved. (I'm not sure if this comment belongs here. I'm happy to open a new issue for it.)

Yes, that's true. Currently, Ruff only supports the official Jupyter Notebook format and none of the other formats.

@tovrstra py percent belongs here because #11160 was redirected here.

Just to add details about our use case.
jupytext allows you to pair notebook formats so that you can have a synchronised MyST .md and .ipynb files. It seems like one solution here is that ruff formats the .ipynb version and then jupytext can resynchronise the files.
However, we're not using paired notebooks and only have the MyST .md versions in our repo. That does mean that the files don't contain the outputs of running code, so we don't get the nicely rendered versions in GitHub, but we also avoid quite a lot of commit chaff from trivial changes to .ipynb files and only commit changes to the notebook source. So I guess with the existing functionality, we could convert files to .ipynb, format using ruff, convert back to MyST .md and delete the .ipynb formats.