Filter used in options only run once

Template:
<select  v-model="selected" options="selecteList | youCanUse"></select>

Filter:
Vue.filter('youCanUse', function (value) {
  return value.map(function (item) {
        return {
            text: item.name,
            value: item,
            disabled: item.disabled
       }
  });
});

when I change the one of the  selecteList  object item's disabled property to false.
the youCanUse filter doesn't make that option's disabled property to false.
Does the filter in options only run once or I used in a wrong way ? And How can I make it work properly？

Gave it a shot, this mostly works for me:
https://jsfiddle.net/ypt9700e/
The caveat is that it doesn't update whilst the select box is still open.

@squirmy Thank you !!
I think I found the reason: I didn't initial the selecteList  disabled property
selecteList: [{
   name: 'Learn JavaScript'
}, {
   name: 'Learn Vue.js'
}],

that won't work properly.
And I found another problem:
https://jsfiddle.net/ypt9700e/1/
when I changed the v-model selected to an object, it doesn't fired the change event.

change event only fires from user input. It doesn't fire when you set the value in JavaScript.

no problems - i'm actually surprised the disabled thing worked :)
you can set the selected object, try doing this instead:
setTimeout(function () {
    demo.selected = demo.selecteList[0];
}, 2000);

your problem was that you were setting it to an object which didn't exist in the select


Using Safari and a Hardware Authenticator is impossible

The device blinks "Register Unknown" and makes it impossible to approve, for some reason.

Similar issues. https://www.reddit.com/r/ledgerwallet/comments/ukit81/comment/i7ria5g/?utm_source=share&utm_medium=web2x&context=3


msedge-selenium-tool Python package does not exist

Readme says to do 👍
pip install msedge-selenium-tool
But pip says package does not exist.
Python version used 3.7.6

Hi @teetime514, thanks for your interest in our project! The pip package is now live at https://pypi.org/project/msedge-selenium-tools/3.141.0/
Our readme was a little ahead of our actual release :) Sorry for the confusion.


Update various examples to use the Mapbox v4-API

Title says it all.
TIA for any review.
Fixes #9953

Thanks for the review.


loading nested repeats hides questions

see https://github.com/kobotoolbox/enketo-express/issues/805
Load XML:
<repeat-missing-child-example><start>2017-08-16T11:22:47.000-06:00</start><end>2017-08-16T11:22:47.000-06:00</end><today>2017-08-16</today><deviceid>localhost:BgIE8gkFewoARkO3</deviceid><program_info/><project_scope>regional</project_scope><name_of_region><region>tigray</region><name_of_zone><zone>central</zone></name_of_zone></name_of_region><name_of_region><region>afar</region><name_of_zone><zone>zone 1</zone></name_of_zone></name_of_region><program_information/><meta><instanceID>uuid:82e5a138-1848-43cc-b0b4-82a16d306e74</instanceID></meta></repeat-missing-child-example>

in form:
repeat-missing-child-example.xml.zip

The nested repeat in the second parent repeat is not cloned properly. It has no questions in the HTML markup.

branch is not initialized

selects get a data-name attribute but shouldn't. Cause might be in enketo-xslt


Fix callback for File -> Open Folder... action

References and relevant issues
Closes #6934
Description
Incorrect callback was being used for the Open Folder action. This PR fixes it. We need a testing framework for the actions and menus @lucyleeow, which we knew, but maybe we should prioritize ahead of converting the layer actions.

Nice catch! Thanks @DragaDoncila !

I agree that there needs to be better testing for the app-model stuff, but getting this obvious fix in is just good, so lets not let perfect block it.


Improve PR Review Box UI

This PR follows:

#22950

Before
The Review Box has many problems:

It doesn't work for small screens.
It has an anonying animation which makes the UI laggy.
It uses "custom dropdown menu" which is very difficult to fine tune.
$().toggle('visible') is not a correct call

jQuery just accepts any invalid duration argument: $().toggle('anyting')

The button is not a button.

After
These problems are fixed, and eliminate many !important games.

And most dropdown icons still looks good:

LG-TM bot


Autodesk Fusion compatibility

A quick patch to make the script work on Autodesk Fusion.

Hi Romain,
I've noticed that ULP is not working on Fusion, but did not have time to fix it.
Let me try it on my end and accept the pull during next week.
Thanks for the PR.

I had to simplify the script as the GROUP and DISPLAY commands seem broken in Fusion (I could not get them to work).

@motla I've verfied it on my side. I confirm that DISPLAY = mylayer LAST are non-functional in Fusion. The GROUP command is working in the commad line, but I will check it from the ULP. I will add a check for the old ULP and the new one to instruct the user tp use the correct ULP version based on Eagle release.
// For fusion it should be +9.7.x
int version = EAGLE_VERSION; // 9
int release = EAGLE_RELEASE; // 7

if(version == 9)
{
  if(release <= 7) // The last standalone version of EAGLE is 9.6.2
  {
    dlgMessageBox("This ULP is a ported version for Autodesk Fusion, For standalone Eagle use neg_silk.ulp instead","+OK");
  }
}
else if(version < 9) // Even older versions
{
    dlgMessageBox("This ULP is a ported version for Autodesk Fusion, For standalone Eagle use neg_silk.ulp instead","+OK");
}

Thanks


On responsive view, have sort buttons

On a responsive view, it is impossible to sort the columns.  Provide a way for a user to sort data

Duplicate of #6, closing


force:mdapi:deploy does not work with windows file path

Summary
Short summary of what is going on or to provide context.

Steps To Reproduce:

Try force:mdapi:deploy on a windows machine using windows file path notation
notice: '/' works as folder delimiter and '\\' (double back-slash, 'escaped' backslash) works but not the single (windows-style) '\'

Expected result
I should be able to deploy with *nix and windows file paths
Actual result
Deploy fails with error:
The flag value ".\appero quote\main\default\aura\apo_BaseInputOutput"" is not in the correct format for "directory."
Additional information
Feel free to attach a screenshot.
SFDX CLI Version:
sfdx-cli/7.20.1-d88ae4707c win32-x64 node-v10.15.3
OS and version:
current stable Win10 built

Wow, that's really old!  we have tests that assert this behavior on windows.  If there's till an issue, please open a new issue.


Consider implementing IOptionsMonitor in OptionsWrapper

From @pakrym on Wednesday, June 14, 2017 4:47:32 PM
Possibly IOptionsFactory and IOptionsSnapshot too.
Idea is to be able to pass constant options to any options consumer without requiring to implement proxy class (https://github.com/aspnet/Logging/blob/2b6d34778b013908f6d9208b8a69a8286dceac34/test/Microsoft.Extensions.Logging.AzureAppServices.Test/OptionsWrapperMonitor.cs)
Copied from original issue: aspnet/Options#200

@HaoK Can you comment on this?

Its not a perfect fit, since the wrapper today only takes 1 specific options instance.  We could have it implement IOptionsMonitor and return that single instance for all named options as well as the default I guess.  But this looks mostly like a test improvement/usage scenario...
A 'real' wrapper for the monitor should take a Dictionary<string, TOptions> to use as the map...

@HaoK Thanks. So it sounds like to me we should just close this. Agreed?

Yeah sounds fine to just close, done


Add package fstrm

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Change Log

Change
Change
Change

Does this affect the toolchain?

YES/NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: xxxx

Github defaults to the 2.0 stable release branch. If you target the main branch this should clean up the extra commits.

Let me close this PR, and rebase from main and PR into main. Sorry, not familiar with workflow just yet


Redirect to /projects if already logged in

Works around a bug where the log-in form on the home page crashes with 'id' is read-only if you log in while already logged in.

Good point. We can check if you are on the home page before redirecting. That's easy. Just move the redirect down to the home page route.


New OpenJ9 compile requirement for x86: nasm

This is specific to feature work surrounding https://github.com/eclipse/openj9/issues/2418
@nbhuiyan, can you please advise as to which version of nasm OpenJ9 requires?

@jdekonin The version of nasm we'd like is the current latest release: v2.13.03

@nbhuiyan Does your earlier comment mean that we definitely cannot use the ones supplied with the OS (2.07 in this case) is definitely not an option?

@sxa555 We MAY be able to use nasm v2.07, but I have never tested using that version. The oldest version I have successfully tried building and testing with is v2.11.08. The reason we are setting a newer version as a requirement is that one of the motivations behind making a switch to NASM was to be able to use newer instructions and features available through a modern assembler. NASM v2.07 is almost a decade old, and a lot has been fixed/added since then.

OK ... just a shame not to use the default on the distribution, but as long as it doesn't introduce any extra runtime dependency I'm happy with using it. Thanks

Resolved via #494 I believe.


Alternative SVG export

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Hi! I usually use draw.io to prepare diagrams for presentations of the projects I carry out, however I recently noticed that the option to export "Convert labels to SVG" was removed (I've seen the issues explaining the reason related to maintaining a server for it ). I usually prepare presentations using Canva, and for some reason it only supports SVG when exported using this option. I have already tested a combination of the other options present in the export dialog, and with all others it indicate that the file is not supported.
Describe the solution you'd like

A clear and concise description of what you want to happen.

From what I understand, what the previous solution accomplished was:

Export the diagram as PDF
Send the PDF to the server
Import the PDF into Inkscape
Export to SVG in Inkscape
Send back the generated SVG

Would it be possible to replicate this functionality in the desktop version of draw.io, performing the conversion locally? Considering that the user already has Inkscape installed and added to the path. I believe it would be something like exporting to PDF in a temporary file, calling draw.io, performing the conversion and deleting the temporary file, right?
Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

I was able to manually perform the conversion, inkscape --export-type="svg" <drawio_pdf_file.pdf>, but it would be much better and faster to have an automatic option directly in draw.io.
Additional context

Add any other context or screenshots about the feature request here.

Not locally, no. It adds the risk of a security hole by allowing execution of another app. I'd suggest to automate or batch process.


SystemFormatException

Hi,
We write a C# version of MJML rendering engine for HTML email. MJML is an XML language.
I am trying out your library, but when I open this file, I get an FormatException without any further details:
https://github.com/SebastianStehle/mjml-test/blob/main/TestRunner/ManyHeroes.mjml

I found the root cause:
It is these this kind of attributes
background-url=
          "https://cloud.githubusercontent.com/assets/1830348/15354890/1442159a-1cf0-11e6-92b1-b861dadf1750.jpg"

Thanks for the report.
The current implementation was not intended to have spaces or line breaks before and after the attribute name and value.
In the following, the parser can read 'node1', but the others fail.
'node2' does not throw exceptions, but it contain whitespace in the attribute name. (Its attribute name is incorrectly "name ", NOT "name")
<root>
  <node1 name="value"/>

  <node2 name ="value"/>
  <node3 name= "value"/>
  <node4 name=
    "value"/>
</root>

Is this whitespace allowed by the xml specification? It reads correctly in some other parsers, but I don't know if this should be allowed.

I have run the example through a few online validators and none of them has marked any errors. So I guess it is correct to have whitespaces.
If I remember correctly it was formatted by VSCode or Visual Studio like that to prevent long lines.

OK, I will fix that.

Thank you


mercurial support

This actually seems to work for me in a live repo.
That said, I'm not sure I'm going to keep coveralls because the UI isn't that great. I'm going to see what the alternatives are. In the meantime, I'm open to comments here.

Hi @eire1130! Do you have an example of how this works with Coveralls UI? As far as I can see in their docs, only Git is supported.


Log type seems to be removed

The log type describe here is already removed after this pr.
Now it seems that a DigestItem should be used like this.

https://github.com/paritytech/substrate/pull/2802

@kigawas The PR you mentioned is against master, and we've been focusing our efforts here on documenting v1.0. So I don't think we should remove the information you linked from the devhub.
That being said, we will, at some point, need to document newer versions, and we probably shouldn't wait until there is a huge backlog of changes built up. We're using docusaurus to host our docs and it supports versions. We haven't explored the versions feature yet, but I would welcome a PR that add's a version like "next" that documents the latest stuff in master.
Default version should definitely stay as v1.0 for the next while.

@JoshOrndorff
Reasonable, but not exactly the case. AFAIK all the links in the doc referring to substrate repo are against master, thus discrepancy will occur especially when a curious user clicks the link.
Also, the developer hub's installation guide is also based on master.
From my point of view, the develop hub doc should have a dev/stable switch at least, otherwise it'll be a disaster to follow updates when substrate 2.0 released.

Good catch on the install instructions. Our intention has been to document v1.0, but you're right that we haven't always made that clear or totally consistent.

From my point of view, the develop hub doc should have a dev/stable switch at least, otherwise it'll be a disaster to follow updates when substrate 2.0 released.

Agreed. Versioning is one of the reasons we chose docusaurus. Here are there docs about getting [set up with versioning](https://docusaurus.io/docs/en/versioning and here), and our own issue tracking it.
My suggestions is to solve that issue by setting up versioning, then make this update about Log in the 2.0.0 branch.

It would also be cool to create a page with generally tracks breaking changes that have happened between major releases. Nothing crazy to manage, but maybe some long bullet point index with a short summary of the breaking change and the PR where it happened.
We should be able to sort of generate this by looking at the history of the spec_version number in the Substrate codebase.

I think we should at least introduce some label changelog or similar in Substrate, to highlight prs that needs to go into a changelog.


Add Support  for object[] data type

I am trying to create a class object with the schema (provided below) with one of the data-types as object[]. However, when I print the schema the detected data-type of my metadata field becomes text. And then I check the code and noticed that python package does not support object[] data-type (code). Can someone add the support for this data-type. (Ref: func)
 {'dataType': ['text'], 'description': "This property was generated by Weaviate's auto-schema feature on Tue Oct 31 16:07:28 2023", 'indexFilterable': True, 'indexSearchable': True, 'name': 'metadata', 'tokenization': 'word'}]

# schema
class_obj = {
    "classes": [
        {
            "class": self.index_name,
            "vectorizer": "none",
            "properties": [
                {
                    "name": "identifier",
                    "dataType": ["text"],
                },
                {
                    "name": "text",
                    "dataType": ["text"],
                },
                {
                    "name": "metadata",
                    "dataType": ["object[]"],
                    "nestedProperties": [
                        {
                            "name": "data_type",
                            "dataType": ["text"],
                        },
                        {
                            "name": "doc_id",
                            "dataType": ["text"],
                        },
                        {
                            "name": "url",
                            "dataType": ["text"],
                        },
                        {
                            "name": "hash",
                            "dataType": ["text"],
                        },
                        {
                            "name": "app_id",
                            "dataType": ["text"],
                        },
                    ],
                },
            ],
        }
    ]
}

Hi @deven298, for clarity on your problem, what versions of the Python client and Weaviate server are you using?

Hey @tsmith023
I am using weaviate-client v3.24.2 python package.

And what about your Weaviate server version?

My weaviate server version is 1.22.1

Okay, that makes sense then!
Please update your Python client to v3.25.2 that has support for object and object[] types 😁


Add NETStandard.Library.NETFramework for older versions of Visual Studio

@bricelam @ajcvickers

@ryanbrandenburg any harm in keeping this a while longer?

The problem is NETStandard.Library.NETFramework will stop shipping at some point, so there will be a stale version on the feed until we clean it, at which point this will start to fail.

I think it is ok. We can remove this once it start to fail.

there will be a stale version on the feed until we clean it, at which point this will start to fail.

It is available on NuGet feed so restore will never fail even if we clean our feeds. If the stale version causes errors then we can remove it.

What about the time before it fails when that package is stale?

What about the time before it fails when that package is stale?

We can always remove it if it fails. Nothing stops us from doing so in future. No need to deteriorate dev experience for that.

What issue does this solve??

@ErikEJ It makes developing EF Core slightly easier with VS 15.2


Use latest SNAPSHOTs of JUnit 5 to fix Travis build

see junit-team/junit5#284

Awesome, thank you!


Placing email on public site is not safe

URL: https://pub.dartlang.org/packages/activatory
Email is not protected on pub site (placed in plain mailto link). This is unsafe and totally bad idea since any email extractor can find it.
I have removed email from pubspec, but it's still placed on pub site. Can I remove my email from pub completely?
I suppose this is a good idea to use some profile links (github?) but not emails.
For example, there are no emails on nuget (https://www.nuget.org/packages/AutoFixture/) and this doesn't seem like anyone has some problems with it.

Can I remove my email from pub completely?

At the moment no, that is not an option.
We have some ideas for how to make user-profiles and let people publish without using their email.
But right now email is the only trust point.

Personally, I don't attempt to keep my email address private, I trust in spam filters, since it easily ends up in mailing lists and bug trackers no matter what I do. In your case, your email is already publicly available in all our git commit messages.
$ git log
commit 3b3fe802e5cd86fde343d4e0c687c8fc873b90f3
Author: syberside<EMAIL_ADDRESS>Date:   Mon Dec 17 02:56:14 2018 +0300

    Update .travis.yml

(I replaced your email with ***********, so that I didn't leak it an extra time)
If you want to avoid this github has instructions on how you can use a no-reply address:
https://help.github.com/articles/about-commit-email-addresses/

For the record, this is something we want to see happen. I think the work is tracked in #1518.
Right now the whole user management is a little crude. I would love it if it was easier to find packages from the same author, etc..

Thanks for answering.
I think placing email in GitHub repo (in meta information, not in regular files) is secure enough. At least it is harder to extract.
Yep, #1518 will be nice feature. Also it will be nice to add token issuing interface to profile. But this sounds like independent feature.


Issue708

Added a new override button in the settings screen.
When override is checked, the config from settings UI will override the config from spp-plugin.yml and this happens every time the override is checked.
Whenever the spp-plugin.yml file is saved and has no error, the configuration from file will override the Settings UI  override checkbox in setting UI  is set to false.

@javamak, there is not supposed to be an override setting in the GUI. Only the config file should have the override setting. It should be based on the setting whether or not the UI is editable.

Reverted the override button and now the override happens based on the setting in the file. Please review


Command to have clembot return join date

As a mod on a server, it can be annoying when trying to tell if an account is trolling, and something very helpful would be easy access to when a user joins a server.
I propose a command that you can supply a user's @ or simply their id and ClemBot can return when they joined the server.
So far the other option is using ClemBot's user join log and searching for when that user joined but can be time consuming or troublesome if the user has changed nicknames or usernames.

yes, this is part of a larger !info command that ive been wanting

yes, this is part of a larger !info command that ive been wanting

What would you like the info command to show?

stuff like this

Are you looking for all of that information, including guild stats like rank?

and do we want to have some information locked to designated groups with claims, or the entire command locked to specific claims permissions?

i dont think we want rank at this moment tbh, some other more general info could be cool, something like total messages


You're leaking memory...

...For every request.
// These should all be disposed.
var mediaStrm = mediaItem.GetMediaStream();
var img = Image.FromStream(mediaStrm);
var proc = new ImageProcessor.ImageFactory();

https://github.com/saadahmedkhan/sitecore-fields/blob/1ddea02540cf611f3ded66bb0af7e3030a93dd75/src/Sitecore.Fields/Processors/CropProcessor.cs#L62

Thanks @JimBobSquarePants. Will fix this soon.


Add documentation for enabling Akri to connect to k3s and MicroK8s CNI

Akri uses the CNI socket and crictl on the host to query some container information.

Add connection information for K3s' embedded crictl implementation.
Add connection information for MicroK8s (including crictl installation details)

Maybe not for this PR but we may want to add crictl to our CI workflows since they use MicroK8s


Collapse Distribution side menu to just Object and References

On the item page, the right hand menu has a Distribution heading. This has subheadings that are separated into access links, geospatial web services, image, metadata. However, they are all grouped under the References subheading.
It might be easier to navigate the page if we just use the References subheading here instead of the access links, geospatial web services, etc.

Links


proteus.py crash using default.toml on main branch

Hi, as I mentioned during the PROTEUS meeting this morning, proteus.py crashes when running the default configuration file:
$ proteus start --config input/default.toml
I 'git pulled' the main branch on my local computer to make sure I'm up to date, but it still crashes (see the proteus_00.log file attached for more details). The KeyError is "mass".
Let me know if you have any idea how to fix this :)
proteus_00.log

Ah good point thanks @nichollsh ! I just did it and it works perfectly fine. I will think about it next time so I don't open a new issue for this. :)

Glad this helped @EmmaPostolec! I had a similar problem too, so figured it might be the same thing. I wonder if it's possible to add a warning for the user if they haven't updated something.

Yes, it would definitely be useful to anyone working on the code (especially new users like Bsc or Msc students) I think !

Should I reopen this issue and suggest this idea as a code improvement ? @nichollsh

Yes, this seems like a good idea.

Another similar thing to do would be to print the module configuration to the log. This makes sure the user is aware of what they are using, and also is useful for knowing what configuration was used for a run.

Nice ! So the goal would be to include warning for the user to update his code to the latest version of Proteus. One way to do it is first by printing the module configuration to the log file while running proteus.py as @nichollsh suggested.

Has this been resolved by on of the PRs?

It has been somewhat addressed by adding logging statements regarding the module environment. Maybe we could add an automated check for when the code needs updating? Something like proteus update in the CLI, which polls PyPI for the latest versions and notifies the user if something newer is available. @stefsmeets would something like this be possible?

I'm happy to pick this up and see what we can do about it. Some codes have a doctor subcommand which does something like this.


Error serializing nullable boolean

Hey!
I have a schema that looks like this:
{
  "type": "record",
  "name": "Name",
  "fields": [
    { "name": "stacked", "type": ["null", "boolean"], "default": null },
    { more fields },
    { more fields },
  ]
}

When I try to produce a message containing a boolean value in the stacked property, I get this error:
 cannot decode textual union: expected: '{'; actual: 'f' for key: "stacked"

But it works correctly without the property.

https://github.com/mostafa/xk6-kafka/issues/266#issuecomment-1927254565


Problem in enforce_detection parameter

I try to pass Image has been detect face into Verify function with parameter  enforce_detection = False in model "VGG-Face" not have this problem. But in model  "Facenet",  "OpenFace",  "DeepFace" ,"Ensemble" got returns a wrong value
img1 and img2 is a different person

   img1 = cv2.imread(img_path1)
   img2 = cv2.imread(img_path2)
   detect_face1 = DeepFace.detectFace(img1)
   detect_face2 = DeepFace.detectFace(img2
   obj = DeepFace.verify(detect_face1, detect_face2, model_name= "VGG-Face", enforce_detection=False)
   obj {'distance': 0.5694833993911743,
     'max_threshold_to_verify': 0.4,
     'model': 'VGG-Face',
     'similarity_metric': 'cosine',
     'verified': False}

Actually, the result must return false. But in model Facenet, OpenFace got returns a wrong value
  obj = DeepFace.verify(detect_face1, detect_face2, model_name= "Facenet", enforce_detection=False)
  obj {'distance': 4.112720489501953e-06,
   'max_threshold_to_verify': 0.4,
   'model': 'Facenet',
   'similarity_metric': 'cosine',
   'verified': True}

  obj = DeepFace.verify(detect_face1, detect_face2, model_name= "OpenFace",enforce_detection=False)
  obj {'distance': 0.00014126300811767578,
   'max_threshold_to_verify': 0.1,
   'model': 'OpenFace',
   'similarity_metric': 'cosine',
   'verified': True}

But I try to pass with the original Image got a result like this.
  obj = DeepFace.verify(img1, img2, model_name= "Facenet")
  obj {'distance': 0.7559900730848312,
   'max_threshold_to_verify': 0.4,
   'model': 'Facenet',
   'similarity_metric': 'cosine',
   'verified': False}

1- setting enforce_detection to False decreases model accuracy dramatically. That's why, its default value is True. If you still want to turn it off, you can run deepface but low accuracy is not surprise.
2- Those models offer high accuracy but it does not guarantee 100% accuracy. That's why, single test results might be wrong.
3- verify function reads image with opencv and it reads BGR format but detectFace function returns RGB format. That's the reason why they act different.


Block Placer & Breaker don't work on server

Industrial Foregoing Version: 1.7.1
TeslaCoreLib Version: 1.12-1.0.12
The block breaker and the block placer don't seem to work for me on my server. I tried the exact same setup in single player and it works...
Any idea how to fix that?

Is the area protected?

Yeah that also came in my mind right now. It works now, so nevermind.
But thanks for the quick reply ;)


[Question] detect verbs even if passed lowcase?

Hi,
How can I pass a complete lowcase verb but still get it recognized? I want to enable my lazy users to type
app.exe checkout as well as
app.exe Checkout
Thx
Michael

Parse the arguments like this:
new Parser(with => with.CaseInsensitiveEnumValues = true).ParseArguments<CommandLineOptions>(args)

I believe you can do this by setting ParserSettings.CaseSensitive to false while creating your Parser, like so:
new Parser(settings => settings.CaseSensitive = false);

Do note that this wil disable case sensitivity on option names as well.

Hi,
Thank you very much. This worked.
Cheers
Michael
Gesendet von Mail für Windows 10
Von: Tim Waalewijn
Gesendet: Sonntag, 14. Juni 2020 00:34
An: commandlineparser/commandline
Cc: Sigurdur42; Author
Betreff: Re: [commandlineparser/commandline] [Question] detect verbs even ifpassed lowcase? (#645)
I believe you can do this by setting ParserSettings.CaseSensitive to false while creating your Parser, like so:
new Parser(settings => settings.CaseSensitive = false);
Do note that this wil disable case sensitivity on option names as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Need cross region peering

variables.tf
variable "peer_region" {
description = "Peer region"
default     = ""
}
main.tf
resource "aws_vpc_peering_connection" "this" {
count = "${var.create_peering ? 1 : 0}"
peer_owner_id = "${var.owner_account_id}"
peer_vpc_id   = "${var.vpc_peer_id}"
vpc_id        = "${var.this_vpc_id}"
auto_accept   = "${var.auto_accept_peering}"
peer_region   = "${var.peer_region}"
}
sometimes we peer across regions.

Sorry for this huge delay with response :(
I started working on this issue in https://github.com/grem11n/terraform-aws-vpc-peering/pull/6
Unfortunately, I haven't tested it yet. Hopefully, I'll do it in the nearest days.
I'd appreciate a code review!


Netkvm in vista/x86 from virtio-win-0.1-94.iso can't work

Hi Yan,
I tried to install the netkvm virtio driver to a 32bit windows server 2008 with virtio-win-0.1-94.iso, but it will yield code 10 error. virtio-win-0.1-81.iso works!
Best regards,
Ben

Hi Ben,
We treat all the builds until virtio-win-0.1-100 as experimental and unstable. I suggest to try the latest one.
Will check if we released it.
Best regards,
Yan.

Hi Ben,
Build 100 is released:
http://alt.fedoraproject.org/pub/alt/virtio-win/latest/images/bin/
Best regards,
Yan.

Got it!
Thanks,
Ben


db event outbox (GSI-610)

Implements a wrapper around the existing DaoFactoryProtocol that adds outbox behavior for publishing.
An additional __metadata__ field is added to documents in MongoDB that encodes whether a change has
been published and whether a document was deleted or not.
It is assumed that one topic only contains events related to one resource type stored in one collection.

Bumps version to 2.2.0.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-8 of 202 (96.04%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 90.957%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hexkit/protocols/dao_outbox.py
16
19
84.21%

src/hexkit/providers/mongokafka/provider.py
126
131
96.18%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
1569

Relevant Lines:
1725

💛  - Coveralls

Implements a wrapper around the existing DaoFactoryProtocol that adds outbox behavior for publishing.
An additional metadata field is added to documents in MongoDB that encodes whether a change has
been published and whether a document was deleted or not.
It is assumed that one topic only contains events related to one resource type stored in one collection.
Bumps version to 2.2.0.


EMON1 should silence output if /var/lib/ceph/mon/*is empty

Otherwise this is going to be the output:
EMON1: secret key "None" is different than host(s): mon0

Fixed by #68


Network Performance Monitor Configuration

How do I end the configuration if I don't have an ExpressRoute? I don't see a "continue" button here...

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

ID: ff3ca71d-bc0f-f021-7eda-ed522edfd51e
Version Independent ID: 88d3f691-7358-e1ef-9763-c191d4f42e1a
Content: Network Performance Monitor solution in Azure
Content Source: articles/log-analytics/log-analytics-network-performance-monitor.md
Service: log-analytics
GitHub Login: @abshamsft
Microsoft Alias: abshamsft


Update to Babylon.js to 5.0.0-rc.9

This PR updates the version of BJS used by BabylonNative to 5.0 RC 9

Make sure to test this in BRN before merging.

I've already tested it.  We're good to go


Can't decrypt my own messages on the same device

This just happened once and was fixed after a restart anyways.
I was sending a message in #megolm:matrix.org with Fdroid v0.6.5 and as soon as the message arrived it was replaced with
** Unable to decrypt: Unable to decrypt **

Later I check this on my laptop:
** Unable to decrypt: Error: Mismatched room_id for inbound group session (expected !LMVMezeTndRRtgLKCp:matrix.org, was !UCnwUWwIKhcpaPTHtR:sw1v.org) **

the only reason i could find out if that two sessions had the same identifiers.
On the version > 0.6.5, the olm lib did not use a secured random string generator (the previous one was based on rand() instead of SecureRandom) so it might generate several times the same string.

I believe this was fixed in 0.6.7


Python api example wrong

Hello,
I setup a wrapper for soldFire API and example given can't work and will not work as is ...
endpoint_info = {'mvip': <IP_ADDRESS>, 'login': 'admin',
'password': 'SuperDooperPassword!', 'port': 443,
'url': 'https://<IP_ADDRESS>:443'}
dictionary is not correctly setup, mainly {'mvip': <IP_ADDRESS>
as pi needs to be string, right!!
also I am not able with proper dictionary to get   sf_client.volumes as this binding does not exist in client ...
Is there another branch that should be picked up ...

HI Cathmonster!
The CLI is not officially ready for release, but it will be soon. If you want to play around with it before the official 1.0 release, you can pull it from the develop branch. I just updated the help in the develop branch today. I set up the release branch so that I could get my ducks in a row before the official release, but I see it is confusing so I'll go ahead and remove the release0.0 branch to avoid further confusion.
Let me know if you have any troubles and feel free to follow the project to be updated when the actual release comes out!
Thank you,
Ariel


Use Modernizr without gulp.src

Hi,
Why must I add src of some JS files before use modernizr ? Can I use modernizr directly ?
gulp.task('modernizr', function() {
  gulp.src('./js/*.js')
    .pipe(modernizr())
    .pipe(gulp.dest("build/"))
});

Thanks for help

I think that this plugin is meant to make a custom build of modernizr by analyzing your CSS/JS files.
So you won't need to take care of which tests you are using when you'r doing the build.
And also gulp is designed to act on files :) So if you don't need to parse those files, you shouldn't need gulp for this specific task :)

Hey, I've just hacked a basic modernizr gulp stream together that doesn't use customizr (so lets you choose your features by hand).  Code is as follows:
var gulp = require('gulp'),
    modernizr = require('modernizr'),
    File = require('vinyl'),
    stream = require('stream');

// This function produces a stream.
// - First arg is the desired file name (e.g. modernizr.js),
// - second arg is a set of options to pass to the modernizr function.
function modernizrStream(name, options) {
  var wasWritten = false;

  return new stream.Readable({
    objectMode: true,
    read: function (size) {
      var self = this;
      modernizr.build(options, function (result) {
        if (!wasWritten) {
          self.push(new File({
            path: process.cwd() + "/" + name,
            contents: new Buffer(result)
          }));

          wasWritten = true;
          self.push(null);
        }
      });
    }
  })
}

// Here's an example ...
gulp.task('modernizr', function () {
  modernizrStream('modernizr.js', {
    "feature-detects": [
      "es5/array",
      "es5/date",
      "es5/function",
      "es5/object",
      "es5/specification",
      "es5/strictmode",
      "es5/string",
      "es5/syntax",
      "es5/undefined",
      "eventlistener",
      "history",
      "network/fetch"
    ]
  }).pipe(gulp.dest('public/'));
});

@Hiswe is correct, this plugin takes src-files and scans them for Modernizr references so it can generate a custom build. If it is still an issue for you @JiDai please say so.

That gulp.src thing curious me for a long time. Thanks for clarifying! @Hiswe @rejas . Maybe those explanation on homepage would help some more people.
One more question, so is this mean I don't need to include any tests in setting, gulp-modernizr will scan my files and grap whatever tests I wrote in their?

Correct @Bigdragon13th So for example if in your src files there is a line like
       if (!Modernizr.objectfit) {             doSomethingLikeCallAPolyfill();         }
this plugin will add the objectfit tests to the generated modernizr build (which can best saved somewehre via gulp.dest

Added this explanation to the README and closing the ticket for now


Bound value for index

When the index scan, need to get the boundary value for where condition. for example :
field c1 is int :

c1 > 5 --> (the range pair : {max , c1 + 1})
c1 > 1 and c1 < 5 --> (the range pair : {c1 - 1, c1 -1} )

field c1 is double , and the precise number of decimal places is 6 :

c1 > 0.5 --> (the range pair : {max , 0.500001})
c1 > 1 and c1 < 5 --> (the range pair : {1.000001, 4.999999} )

Addressed @dutor's comments. Thanks .

Simplified code logic by @dutor's proposal.


Unit tests: Failure at createContext and run because of user session hash change

A HTTP error 403 is returned at createContext and during certain test like the settings related one.
If I comment out that code
    if (global.userHashMapIDs.has(`${tenantID}#${userID}`) &&
       global.userHashMapIDs.get(`${tenantID}#${userID}`) !== userHashID) {
       throw new AppError({
         source: Constants.CENTRAL_SERVER,
         errorCode: StatusCodes.FORBIDDEN,
         message: 'User has been updated and will be logged off',
         module: MODULE_NAME,
         method: 'isSessionHashUpdated',
         user: req.user
       });
     }

in isSessionHashUpdated method, the error disappears.

@mahdibenromdhane
Double check if this is related to the unit test that reassigns the badge from one user to another one.
This is the only scenario that could call this method.

This property in the config.json must be set to false when running the unit tests:
"monitorDBChange": false,


Fallback content for default slots not rendering

I believe components with default slots are not rendering fallback content correctly.
Here are some Glitch projects setup top reproduce the problem with the Eleventy WebC plugin and raw WebC.
From what I can see,

getSlottedContentNodes fakes an AST by returning an object with childNodes.  When the node has no content, this is set to an empty array.
Then when getContentForSlot checks for the existance of slot content it is checking if { childNodes: [] }is truthy, which is always the case. This means that the default slot will never render the fallback content, and be empty. Named slots are not effected because default is the only one that is guarenteed a value.

There is a parser test that seems to cover this case and is passing, but it appears the way the test is written component here evaluates to false, so we never hit getSlottedContentNodes.
Updating getContentForSlot to check for the existance of childNodes fixes the problem, e.g.
if(slots[slotName]?.childNodes?.length) || slotName !== "default") {

If this is indeed a problem, I'm happy to put a PR together, but I'm not sure:

if this is the best way and place to fix it
the best way to update the test

so some guidence here would be much appreciated.
Thanks!

@kleinfreund filed a PR for this! It’ll go up with the next release!

Amazing! Thanks everyone!


Allow for dynamically-generated certificates, keys, etc.

Packages like node-forge let us dynamically generate self-signed certificates, private/public key pairs, etc. Developer experience can be significantly improved if we didn't have to save them to file and just pass them in directly.

Yes a few have reported this, maybe it could be time to look at it. Very low prio.


pythonPackages.z3-solver: init at <IP_ADDRESS>

Motivation for this change
I am trying to make angr, the binary analysis framework, available on NixOS.
This is part of the modules it requires.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

It looks like this duplicates the existing z3 python package.

@Pamplemousse will you continue with this? As mentioned, it looks like z3 is already present.

Not anytime soon.
Z3 is already present, but some developpers behind angr host a specific package on PyPi.
Not willing to inverstigate further for now.
Closing, will re-open if I ever want to dig into this.


update Readme.md and cleanup .c file format indent

1, update Readme.md
2, change .c file indent to linux kernel style, use tab instead of space.

I'm sorry there are some conflicts because of moving code to other folder.Please help fix this and pull request again.
tools/pylcc/guide/independ.bpf.c
tools/pylcc/guide/netcollect.3.10.bpf.c
tools/pylcc/guide/netcollect.4.plus.bpf.c


Flesh out synchronization of session span transition

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

#937  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite


Switch to Centos 8 Stream, add curl-devel to the linux CI image

This should resolve #956.

The following test failed
The following tests FAILED:
	113 - velox_functions_test (Failed)

Particularly
111: ../../velox/functions/prestosql/tests/DateTimeFunctionsTest.cpp:319: Failure
111: Expected equality of these values:
111:   21
111:   hour(Timestamp(4000000000, 0))
111:     Which is: 16-byte object <14-00 00-00 00-00 00-00 01-C5 56-80 FE-7F 00-00>
\111: ../../velox/functions/prestosql/tests/DateTimeFunctionsTest.cpp:320: Failure
111: Expected equality of these values:
111:   21
111:   hour(Timestamp(4000000000, 123000000))
111:     Which is: 16-byte object <14-00 00-00 00-00 00-00 01-C5 56-80 FE-7F 00-00>

This test currently always fails on macOS. It now fails on the latest CentOS.
The particular test is
setQueryTimeZone("Pacific/Apia");
319:  EXPECT_EQ(21, hour(Timestamp(4000000000, 0)));
319:  EXPECT_EQ(21, hour(Timestamp(4000000000, 123000000)));

However, the correct answer seems to be 20 for both. I assume there is a timezone library dependency here.
But changing to 20 seems to be the right fix as both macOS and CentOS now behave the same.

I assume there is a timezone library dependency here.
But changing to 20 seems to be the right fix as both macOS and CentOS now behave the same.

date tells the correct hour is 21?
$ TZ=Pacific/Apia date -d '@4000000000'
Tue Oct  2 21:06:40 +14 2096

Am I doing something wrong here? I'd rather disable the test for now, if we're not sure.

I assume there is a timezone library dependency here.
But changing to 20 seems to be the right fix as both macOS and CentOS now behave the same.

date tells me the correct hour is 21?
$ TZ=Pacific/Apia date -d '@4000000000'
Tue Oct  2 21:06:40 +14 2096

Am I doing something wrong here? I'd rather disable the test for now, if we're not sure.

Ah yes, the timezone database on my Mac disagrees with the one here on Fedora 34. Great. I'm just going to comment this test out for now.

On CentOS Stream 8, it is 20. Good call on disabling for now.
sh-4.4# TZ=Pacific/Apia date -d '@4000000000'
Tue Oct  2 20:06:40 +13 2096


Notifications on daily average

Notifications can be triggered on current measurment values. However European or WHO limits are set on daily or yearly averages. Would it be possible to trigger notifications on averages?

Yes! We will include an option for enabling this feature on the settings page. It will be released in the next update. Thank you for the feature request.


✨ [Request] Mealie v1

Which addon?
Mealie

Addon or app?
Addon

Describe the solution you'd like
Mealie has 'released' their V1, it's still in beta but so was the the v0.5 releases.
The 'problem' with the upgrade is they now switched to 2 different docker containers, backend and frontend.
I could switch to Tandoor, but it doesn't have a clear API. I'd eventually like to get the API support working into Home Assistant ;)
I'm not sure if it's worth the trouble to try and merge both Dockerfiles, but an initial effort was made here: https://github.com/hendrix04/mealie-combined

Hi, hendrix04 has created a great combined image! Currently it only works on amd64 so I've already switched this architecture in my addon. Aarch64 should come at one point but a base image used is only based on amd64 and also would need to be multi-arch for this to happen.
Here is the repo if you want to star it, and the issue thread where aarch64 is discussed : https://github.com/hendrix04/mealie-combined/issues/2#issuecomment-1163043687

Any update on this?

Until the upstream joint image is created I won't be able to build the addons, I think it is more difficult than expected for the upstream dev

Hi, I've pushed a first test version of the frontend and API addons in my repo. You need to first backup all your data from within the current instance of Mealie, install the new addons, run the API one, configure the API url in the front end one, and run the fronted one. Then you can import back your data. Let me know how this goes!


controllerAs in directives doesn't play well with isolate scope

I'd like to use the 'controllerAs' option in my directives. Let me cite the reasoning from An AngularJS Style Guide for Closure Users at Google:

Why? Putting methods and properties directly onto the controller, instead of building up a scope object, fits better with the Google Closure class style. Additionally, using 'controller as' makes it obvious which controller you are accessing when multiple controllers apply to an element. Since there is always a '.' in the bindings, you don't have to worry about prototypal inheritance masking primitives.

But I can see an issue with using this approach if the directive has isolate scope bindings.
angular.module('cmw').directive('fooWidget', function() {
    return {
        controller: function() {
            this.qux = '123';
        },
        controllerAs: 'fooWidget',
        scope: {
            bar: '='
        },
        template: ' {{fooWidget.qux}}  {{bar}} '
    };
});

In this case, the bar property is attached to the scope, not to the controller, which results in a confusing inconsistent situation where different properties should be looked for in different places.
As for now I see two alternatives, both not appealing to me:

to create bindings manually by calling $observe for each attribute needed, or
to forgo using 'controllerAs' at all.

It might make sense to introduce another special character for the scope entries in directive definition objects. This character would mean that the given property is a property of the controller, not the scope. What about dot?
angular.module('cmw').directive('fooWidget', function() {
    return {
        controller: function() {
            this.qux = '123';
        },
        controllerAs: 'fooWidget',
        scope: {
            bar: '=.',
            abc: '=.xyzzy'
        },
        template: ' {{fooWidget.qux}}  {{fooWidget.bar}} '
    };
});

I haven't learned much about angularjs. I use controllerAs syntax this way in directive, so the services injected in the controller are there.  :)
angular.mudule('puppy').directive("faceDirective", [function() {
    return {
        scope: {},
        controller: "LongLongDogNameController",
        controllerAs: "mojo",
        templateUrl: "a-puppy-face-template-tpl",
        link: function(scope, elem, attrs) {

            var ctrl = scope.mojo;

            ctrl.viewService.showTeeth();
        }
    };
}]);

@shrekuu - this worked for me. Thanks !

@shrekuu  a bit late but I think link function always get the ctrl as the 4th parameter, there is no need to get it from scope.


Major Points

[ ] Server can host images #8, #20 @xinyi3 Django Images
[ ] Connect to other nodes #42, #43, #44, #45 @TianZhiWang, @nettapper

[ ] Add/delete local and remote friends #25, #27
[ ] Getting posts from other users #13, #19
[ ] Refactor API to be compliant @kylecarlstrom
[ ] Add pagination
[ ] Add tests @nettapper, @joshdeng

@nettapper, @TianZhiWang, @joshdeng, @xinyi3
Let's not leave this until the last moment like last time. Let's try to have this done by Thursday evening (March 23). Check the milestone for specifics Milestone 2

@nettapper, @xinyi3, @joshdeng, @TianZhiWang
Hope you are all having a good weekend. Just a reminder that you should at least be thinking about how we are going to connect to other nodes. This is probably the hardest part of the project and we don't want to leave it until the last minute.
Try to start thinking about some of the issues that you foresee coming up. I think it makes sense to have a "Node" or "Host" model as it ties neatly with several of the server admin stories. However, we have to think about how authentication with other nodes will work. Should a node just be a mapping to a user, etc.

[ ] Posts are not encoded with source and origin upon creation
[ ] We aren't handling the case when remote author unfriends our author. Need to check friendship.


The values of the scope parameter should be consitent accross cmdlets, where they mean the same thing

https://twitter.com/Jaykul/status/1060278217360195584 . @jaykul
Naming is hard. Not THIS hard, but hard:

Set-ExecutionPolicy -Scope LocalMachine
Install-Module -Scope AllUsers
New-PSDrive -Scope Global -Persist

Perhaps we need the concept of a "session scope" or something to unify all the things that can be system-wide, shared between users, or user-specific, and have that as an API (like a layered stack) that all these cmdlets can read from?

We also need to think about compatibility, unless we want to break everyone.
Also, one of the cmdlets is in PowerShellGet ...


Rewrite service proxy and interceptor to use new api-go code.

What was changed
SDK code refactored to use new api-go APIs.
Why?
This is done to avoid tightly coupling sdk-go package to api versions, see: #971
Checklist

Closes #971

How was this tested:
Existing test coverage adjusted to suit.

Any docs updates needed?
No.

https://github.com/temporalio/api-go/pull/100 now merged, just waiting on a new release before updating this.

@robholland - https://github.com/temporalio/api-go has now been tagged as 1.14.0 whenever you want to update this


chore: use ack provider to resolve the cluster cni

What type of PR is this?
/kind feature

What this PR does / why we need it:
chore: use ack provider to resolve the cluster cni
Which issue(s) this PR fixes:

Fixes #none
Special notes for your reviewer:
Does this PR introduce a user-facing change?

cc @helen-frank for checking


Add postgres database

MySQL is good, but PostgreSQL is also good.
How about adding a way to use PostgreSQLwith custom deployment?
This would require some changes to Gemfile - probaby add a mysql and postgres groups.
Everything else should be fine since it seems to already work on Heroku.

Huginn works quite well with Postgresql, actually.  You can make those changes, or, as a hack, set ON_HEROKU=true in the environment.

I figured that out already. However, my deployment is nothing to do with Heroku.
Also, when you do that, bundler complains about double gem inclusion, so that dows not work cleanly.
You also haveto remove gem 'pg' from on_heroku block (or just fix the block). After that use bundle install without deployment to recalculate Gemfile.lock, and then with deployment. After that it works.
Overall I feels like on_heroku is a very durty hack, not worth it.

Agreed.  Would you be interested in adding better Postgresql support?

Probably, if I'll have some free time.

We definitely appreciate any contributions you can make!

I'd just add separate bundler groups for mysql and postgres, it's pretty simple.
The hard part though is to fix all places in the docs and automation scripts to use bundler's without correctly:
bundle install --without development test mysql

bundle install --without development test postgres

I can implement actual code fixes today, but I don't know about fixing the docs.

Btw, with https://github.com/rails/rails/blob/4-2-stable/railties/lib/rails/generators/rails/app/templates/config/environments/production.rb.tt#L25
Rails and heroky play well together.

Thanks!
Yea, Huginn runs pretty well on Heroku, although not on their free plan now that they've changed it.
/cc @dsander, who I think uses Postgresql.

Yes, everything works well on postgres. Changing the documentation everywhere would be some work, maybe the optional bundler groups @knu was working on in #822 would be an option?

I'm going to close this issue.  Please reopen if you want to discuss further!

Postgres is supported since some time, after configuring the database in the .env running bundle will install the pg gem and huginn will work with postgres perfectly.


LTD-5002-denial-matches-not-showing

https://uktrade.atlassian.net/browse/LTD-5002
This fixes an issue which relates to  unable to see end_user (denial matches) in the make recommendation tab.
It was originally an intermittent issue.
Turns out this is now permanently broken as a result of the denial changes.  The Link on the make recommendation page uses reference and if this is no reference then the link is invisible to the user.
The fix here is to include the new unique reference regime_reg_ref field and use this to display a hyperlink in the UI.
I have added tests in the UI also to check this is displayed as expected.
API change is here
https://github.com/uktrade/lite-api/pull/2073
These can be merged independently with no impact but the change will only work when both PRs have been merged.

What do you think about the E2E test idea?

Yeah I agree think it would be useful just seeing if there any natural place I can slot in into our e2e tests


la-go-live

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

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/toolkit/visual-studio-code-overview.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/vs-overview.md
:white_check_mark:Succeeded
View

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


Add Font:getMeasure

Takes same arguments as getWidth().
Returns all the results from lovrFontMeasure

Thanks for sending this.  Can you describe what you need it for (i.e. do you need the glyph count?  is this just for convenience?  performance?)?  The font measurers are a bit messy atm -- we just added lastLineWidth to satisfy a feature request, may add :getWrap to return each wrapped line in a table.  It helps to know what everyone's needs are to come up with the best design.

We need the total height of the wrapped text as it will later be rendered. We calculate and cache the bounds of text elements for culling.
Don't need the glyphCount but I though why not include it as it's there. Perhaps a silly thing to return.
The returned height should probably be include the Font:getLineHeight() tho.
I didn't discover until later that the height returned from lovrRasterizerMeasure is the same as Font:getHeight() * lineCount. To deduce the later you need to know a bit about the internals tho so I think it's a bit cleaner api perhaps?

Testing this a bit closer. It behaves really weird so I'm closing this for now

Demo
https://user-images.githubusercontent.com/45087/158560685-59e4ba90-1c40-4cac-a060-cac0dd1a4bbb.mov
White square is width and height as returned from getMeasure.
Red square is the width sent as the wrap argument and the height calculated with font:getHeight() * font:getLineHeight() * numberOfLines.
getMeasure returns a better height estimate when using lineSpacing != 1, otherwise the same result.
(the weirdness experienced yesterday was because I removed the lua_pushnumber(L, glyphCount); without decrementing the number of returned values -.-)

#485 starts to discuss some improvements to word wrapping -- right now words can extend past the wrap limit because "width > wrap" is checked only on spaces.  I think this explains the difference in width.
The height difference does look surprising though.

Looks like Font:getLineHeight uses lua_pushinteger, which is a mistake.

Fixed!


Double-click on a node should expand properties view (not opening the notebook)

Is your feature request related to a problem? Please describe.
All other low-code/no-code tools open the properties view when a node is double-clicked - Elyra opens the notebook
Describe the solution you'd like
Either:
Expand the properties view on a double-click and move functionality to open the notebook to the context menu
Or:
Expand the properties view on a double-click and add functionality to open the notebook to an icon inside the node
Describe alternatives you've considered
No alternative - working with Elyra when coming from other tools gives a little odd experience
Additional context
Related to #467 and #479

@bourdakos1 also suggested this a while ago.

We have the pipeline-editor built so opening properties vs notebook file is toggle-able. We will probably introduce this as a user setting as to not disrupt the current expected user experience. More info here: https://github.com/elyra-ai/pipeline-editor/issues/92

https://github.com/elyra-ai/elyra/pull/2517 will add support to customize the double click behavior. (You'll have to change the default to switch to the desired behavior you are looking for.)

@ptitzler thanks u made my day!


docs: update acceptance test docs

Added/fixed docs and configs to run acceptance tests locally. Added instructions and commands to run tests with different storage providers.

Merging - this is only test docs, so a changelog is not needed.


Investigate and adapt CouchDB Prometheus alerts

Impact of the bug
CouchDB
Describe the bug
For the last 4 or 5 days, I've been getting around 10 daily alerts:
 alertname = CouchDB service is down

for CouchDB in vocms0744. This instance contains the T0 databases, so the load should be pretty small these days.
I think the couchdb exporter is scraping data through the _stats API, in a frequency of every 30seconds.
How to reproduce it
none
Expected behavior
Investigate if CouchDB is really becoming unavailable, even for a short period of time (few seconds). This is the expression evaluated to trigger or not an alert (provided by Valentin back in March 18):
     expr: avg_over_time(couchdb_httpd_up[5m]) < .9

as I understand it, scenarios like:

2 queries failed out of 10
1 query failed out of 9, and the 10th query took too many seconds to be served, thus landing in a different 5m time range.
that would trigger it.

If those are transient, then we need to remake this expressions to stop creating false alarms.
Additional context and error message
none

Those alerts keep coming, around 10 or more every day... time to look into it.

Couchdb exporter logs were in a strange state in vocms0744. The couchdb_exporter log was dating back from yesterday, and there was no node_exporter logs at all.
I contacted Imran and he restarted couchdb_exporter on that node. Let's see if that was the actual problem.

@vkuznet Valentin, can you please remind me where we define the interval for scraping CouchDB stats? Is it using this global interval of 15 seconds
https://github.com/dmwm/CMSKubernetes/blob/4676d57e85b4f5a1945e75cd59d1a0ef7371ce04/kubernetes/cmsweb/monitoring/prometheus/services/prometheus.yaml#L2
?
I had a look at the couchdb log in vocms0744, but I fail to find any instabilities. I also had an attempt to make couchdb_exporter more verbose through either -verbosity=2 or -stderrthreshold=0, but nothing changes in the exporter log and there is no useful information there. In short, I do not know if these prometheus monitoring calls are:

timing out
failing to be served (http code >= 400)
or if couchdb really says it's down
?

If you prefer not to share some of this data here, feel free to ping me on mattermost or via email please.

I guess I was looking at the wrong place... Is this the correct place with that configuration:
https://gitlab.cern.ch/cmsmonitoring/cmsmon-configs/-/blob/master/prometheus/prometheus.yaml#L167-172
?
And this is where the couchdb rules are defined (conditions to trigger alerts in AM):
https://gitlab.cern.ch/cmsmonitoring/cmsmon-configs/-/blob/master/prometheus/couchdb.rules
I still would like to look at the stats information pushed to AM, to verify which of the 3 conditions above we are hitting. Please provide further details on how to look at them.

@amaltaro , this is correct place. In gitlab we keep all configuration files. Each service have json file which contains targets (e.g. VMs:ports to scrape), the rules file which defines records and alert rules and test file which defines unit tests for all rules. You can inspect any rule and write associated unit test for it, e.g. for couchdb we have couchdb.rules and couchdb.test. I can test them as following:
# check rules
/cvmfs/cms.cern.ch/cmsmon/promtool check rules couchdb.rules

# run unit test
/cvmfs/cms.cern.ch/cmsmon/promtool test rules couchdb.test

Please note, that I found that on AFS promtool may fail at initialization step, therefore I usually use local file system for testing, e.g. /tmp area of the node. I suggest that you inspect test file and adjust it accordingly to initiate the failure. For instance, if you change anything labesl or annotations, e.g. change summary string of unit test then you can see how it fails.
I'm not sure what do you mean by looking stats information pushed to AM. It does not work this way. All info are in prometheus where you can see ANY metric, e.g. couchdb_httpd_up. The Prometheus itself takes the AM rules and check them against the metrics. So, if you need to see your metrics I suggest that you visit prometheus itself at
https://cms-monitoring.cern.ch/prometheus/graph
here you can type your metric (and it will help you with pop-ups to find it) and plot it. You can view it as table values or graph. Then you can visit alert section at
https://cms-monitoring.cern.ch/prometheus/alerts
and see all alerts definitions.
Bottom line, you should invest some of your time to learn new monitoring tools, they are very powerful and versatile. I doubt that there is anything wrong with rules syntax itself. What you need is to know your metric values and properly setup alerts. We monitor now hundreds of systems/services and if something will fail in prometheus we'll notice it in other places. I doubt there is anything specific about the couch. Said that ,I can look at exporter itself if you'll tell me the node and grant me access to it. Usually you can query exporter via simple HTTP call, e.g.
curl http://localhost:port/metrics

and see what it returns. The couchdb exporter uses port 15984, so go to your node and make a plain curl call to see what it returns. The metrics returned in plain ascii row format, e.g. HELP string followed by TYPE string followed by metrics value pair.

@vkuznet thanks for this information, Valentin.
Yes, I wanted to check the metrics in Prometheus, not in AM. Checking the couchdb_httpd_up metric, I see it's been always 1 here:
https://cms-monitoring.cern.ch/prometheus/graph?g0.expr=couchdb_httpd_up&g0.tab=0&g0.stacked=0&g0.range_input=15m&g0.end_input=2021-04-22 12%3A26%3A04&g0.moment_input=2021-04-22 12%3A26%3A04
and in the couchdb grafana dashboard:
https://monit-grafana.cern.ch/d/S8rTdLRZz/couchdb?from=1619092473985&orgId=11&refresh=1m&to=1619096073985&var-env=prod&var-node_name=master&var-method=All&var-db_name=All&var-instance=vocms0744:15984&var-hostname=All&var-view_name=All
even though we had an alert this afternoon at 2:26pm CERN time:
 action = Please check CouchDB service on vocms0744:XXXXX and restart if necessary

Perhaps it would be easier if we could chat over slack or mattermost; or even a zoom call such that I can share my screen and have a more productive and clear chat. Please let me know if you would be available for 5 or 10min this afternoon.

Alan,
how about 19:00 CERN time, before that I'm busy with other meetings/stuff.
Let's try zoom to share the screens.
V.
On  0, Alan Malta Rodrigues @.***> wrote:

@.*** thanks for this information, Valentin.
Yes, I wanted to check the metrics in Prometheus, not in AM. Checking
the couchdb_httpd_up metric, I see it's been always 1 here:
[2]https://cms-monitoring.cern.ch/prometheus/graph?g0.expr=couchdb_http
d_up&g0.tab=0&g0.stacked=0&g0.range_input=15m&g0.end_input=2021-04-22%2
012%3A26%3A04&g0.moment_input=2021-04-22%2012%3A26%3A04
and in the couchdb grafana dashboard:
[3]https://monit-grafana.cern.ch/d/S8rTdLRZz/couchdb?from=1619092473985
&orgId=11&refresh=1m&to=1619096073985&var-env=prod&var-node_name=master
&var-method=All&var-db_name=All&var-instance=vocms0744:15984&var-hostna
me=All&var-view_name=All
even though we had an alert this afternoon at 2:26pm CERN time:
action = Please check CouchDB service on vocms0744:XXXXX and restart if necessa
ry
Perhaps it would be easier if we could chat over slack or mattermost;
or even a zoom call such that I can share my screen and have a more
productive and clear chat. Please let me know if you would be available
for 5 or 10min this afternoon.
—
You are receiving this because you were mentioned.
Reply to this email directly, [4]view it on GitHub, or [5]unsubscribe.
References

https://github.com/vkuznet
https://cms-monitoring.cern.ch/prometheus/graph?g0.expr=couchdb_httpd_up&g0.tab=0&g0.stacked=0&g0.range_input=15m&g0.end_input=2021-04-22 12:26:04&g0.moment_input=2021-04-22 12:26:04
https://monit-grafana.cern.ch/d/S8rTdLRZz/couchdb?from=1619092473985&orgId=11&refresh=1m&to=1619096073985&var-env=prod&var-node_name=master&var-method=All&var-db_name=All&var-instance=vocms0744:15984&var-hostname=All&var-view_name=All
https://github.com/dmwm/WMCore/issues/10450#issuecomment-824818144
https://github.com/notifications/unsubscribe-auth/AAA6RUTJRRUZTUANHNXYR7TTKAMOXANCNFSM426YCMIQ

Perfect! See you in the usual WMCore zoom room.

This issue has been discussed with Valentin (thank you very much!) and it's also being tracked in a JIRA ticket: https://its.cern.ch/jira/browse/CMSMONIT-378

the issue is gone after restart of Prometheus.

Indeed, I haven't got any alerts since then. Thanks for fixing it, Valentin.
I'm closing this ticket as there is nothing else to be done, neither a change to the alert expression.


Speed up ops

Make user op pre-simulation opt in

PR-Codex overview
The focus of this PR is to remove a conditional block related to SmartAccounts in the ThirdwebTransaction.cs file.
Detailed summary

Removed conditional block for SmartAccounts
Replaced with a call to rpc.SendRequestAsync for gas estimation

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.63%. Comparing base (36b4c2c) to head (4b8489f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
- Coverage   87.07%   86.63%   -0.45%
==========================================
  Files          26       26
  Lines        3730     3726       -4
  Branches      514      513       -1
==========================================
- Hits         3248     3228      -20
- Misses        482      498      +16

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


nix-channel --update when building nix dependencies

Maybe we should run nix-channel --update before building dependencies with nix to make sure everyone has the latest  local nixpkgs:
$ nix-channel --update
maybe a bob nix update cmd which does that?
This will not really solve the reproducibility issue. Maybe find a way to lock nixpkgs version. We have a way to use a certain version for nixpkgs but maybe is not that user friendly?

@Equanox @rdnt  I think we need to prio "Maybe find a way to lock nixpkgs version. "  in the context of launching the remote store.
Builds should use the same nixpkgs when building to assure reproducibility. If nixpkgs is not pinned to bob.yaml then is possible that an artifact pushed to a store can have different store paths than a machine which will pull that artifact from store.

@zuzuleinen if nix-pkgs is not explicitly specified, the latest stable commit hash should be used, and embedded into the artifacts that are built, as if it was declared in the bobfile itself.
We should also consider what to do if a child bobfile has a different nixpkgs version across an aggregate. IMHO, if we do have to pick a version, we should use the oldest commit of nixpkgs across all the aggregated bobfiles.

We should also consider handling 'stable' or 'unstable' (latest-stable / latest-unstable maybe?) value for nixpkgs to always use the latest variant. Finding the commit hash is manual work and the commit hash itself does not say if it's a stable or unstable version.

@zuzuleinen if nix-pkgs is not explicitly specified, the latest stable commit hash should be used, and embedded into the artifacts that are built, as if it was declared in the bobfile itself.

This still won't solve the reproducibility issue. Latest stable can change depending on the time a build is run

@zuzuleinen if nixpkgs version is added as input to artifacts, then it is okay if future builds will use a more recent nixpkgs. Old artifacts will become stale and eventually be deleted.


[Enhancement]: Simon Game challenge enhancement

Is your enhancement request related to a problem? Please describe.➕
Game enhancement
The game is not in center due to the instruction tab. So I would like to create a button and make the game in the middle maintaining the responsive of the website
Describe the solution you'd like
Solution steps
making a button to know about the instructions
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @anilkoduru !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @anilkoduru ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


[EngSys] upgrade dev dependency mocha to ^11.0.2

v11 drops supports for NodeJS < 18, it doesn't impact us.
NO_CI

@jeremymeng be aware that it is down to 18.18.0 and not to 18 overall as per https://github.com/mochajs/mocha/pull/5216

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

@jeremymeng be aware that it is down to 18.18.0 and not to 18 overall as per mochajs/mocha#5216

Yes. I think for us it should be fine.

agrifood-farming, policy - mgmt, arm-commerce: all existing test failures


Added predis to composer.json

Added predis to composer.json this will let koel use redis for it's session driver, cache driver, broadcast driver, and queue driver.

Thanks!


QUERY methods are not delegated

in the delegation behaviour, the QUERY methods are not delegated.
when using

  <column name="id" required="true" primaryKey="true" autoIncrement="true" type="INTEGER" />
  <column name="login" type="VARCHAR" required="true" />
  <column name="password" type="VARCHAR" required="true" />
  <behavior name="delegate">
    <parameter name="to" value="profile" />
  </behavior>

  <column name="email" type="VARCHAR" />
  <column name="telephone" type="VARCHAR" />

you can not do a accountQuery::create()->filterByEmail('foo')->find()
(copied from https://github.com/propelorm/Propel/issues/157)

@wurststulle , you can to try patch in #919, it must solve your issue

Since the merge of this commit:
https://github.com/SCIF/Propel2/commit/06914b7a4c688d3ce86b27e651e19e8be9b5afbf
we have the following problem with "php propel build"
[Propel\Runtime\Exception\PropelException]
Column with name «created_at» (delegated from table «user_payment») already exists in table «user». Probably database design mista
ke
I wondered because in our composer.json we decide to take version "2.0.0-alpha" ...

Field is double defined i.e. for example: created_at defined in parent and delegated table. How Query class can detect which table field you want to filter?

Yes, that's what we want. A created_at & updated_at timestamp on every join table.
It was possible in past so we designed it like that.

maybe we should just skip those columns which are double defined like before @SCIF?

But it's not correct, i think. What way of «overlap» detection do you suppose? What field it must filter if you will use multiple delegations with created_at/updated_at?
  <behavior name="delegate">
    <parameter name="to" value="profile, preference" />
  </behavior>

It will causes unpredictable behavior or just skip that fields.

@marcj was some faster than me. :) Yeah, we can skip double defined. Really don't know what is better — BC or correct design forcing :)

I understand your fears @SCIF but then you need another behaviour where it's possible to use default timestampable on multiple 1:1 tables. Let me explain what we do/want:
    <behavior name="delegate">
        <parameter name="to" value="user_payment, user_address, user_text" />
    </behavior>

We want to know, when a user changed/created his user_address data. Before that, the tables data is NULL.
Further, if you see this behavior like "full join data", you should create data for all delegate-tables on parent-creating. A.t.m. it's possible to work with "left join" to reach what we desire.

Seems like it makes sense. @marcj , do you agree that i will skip it?

Yeah, to me its totally ok to skip duplicated columns. It's a expected behaviour like in the actual model class, so we should go for it.

Thx guys for understanding.
We learned from that and fixed version with to 2.0.0-alpha4.
Too bad, I have been following your development happy ;)

@solidevolution, it's always a good idea to fix requirements, especially with alpha versions. :p

I know, thought we already did that but no ;)

@solidevolution , i didn't tested it on real schema, but tests passed. Check it please in my repo — https://github.com/SCIF/Propel2. If everything is ok, i will do PR tomorrow (it's deep night here).

Thanks @SCIF for your fast fix! :) @solidevolution please verify his work so we can merge it asap.

@solidevolution , did you check?

Please give me some days, there's a lot of work a.t.m.

I created schema examples and tested. Fixed my previous commit. Properies skips and tests works ok. Did PR.

@solidevolution , @marcj , any updates guys?

@SCIF if he doesn't answer you could just add a test proving your change that double defined columns are handled now correctly and send a pull-request, I'm gonna merge it then :)

Sorry for that delay. It works now, thx.

@marcj , will you merge it without unit tests? I haven't time to create it in next month (diploma work eats all free time).

Yes, I'm going to merge it this evening :smile:


📝 Mixed Tab/Space on ternary

Environment information
CLI:
  Version:                      1.5.3
  Color support:                true

Platform:
  CPU Architecture:             aarch64
  OS:                           macos

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         "xterm-256color"
  JS_RUNTIME_VERSION:           "v21.5.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "npm/10.2.4"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    false
  VCS disabled:                 true

Workspace:
  Open Documents:               0

Configuration
{
	"$schema": "https://biomejs.dev/schemas/1.5.3/schema.json",
	"formatter": {
		"indentStyle": "tab",
		"lineWidth": 120,
		"ignore": ["**/__generated__/*.graphql.ts"]
	},
	"javascript": {
		"formatter": {
			"semicolons": "asNeeded",
			"arrowParentheses": "asNeeded",
			"quoteStyle": "single",
			"jsxQuoteStyle": "double"
		}
	}
}

Playground link
https://biomejs.dev/playground/?lineWidth=120&quoteStyle=single&semicolons=as-needed&arrowParentheses=as-needed&unsafeParameterDecoratorsEnabled=false&code=ZQB4AHAAbwByAHQAIABjAG8AbgBzAHQAIABTAHQAbwByAHkATgBhAHQAaQB2AGUAQwByAGUAYQB0AGUAIAA9ACAAewAKAAkAbgBhAG0AZQA6ACAAJwBTAHQAbwByAHkATgBhAHQAaQB2AGUAQwByAGUAYQB0AGUAJwAsAAoACQBwAGEAdABoADoAIAAnAG0AbwBiAGkAbABlAC8AZQBkAGkAdABvAHIAJwAsAAoACQBwAHIAZQBwAGEAcgBlAFYAYQByAGkAYQBiAGwAZQBzADoAIAAoAF8AOgAgAHMAdAByAGkAbgBnACwAIAB7ACAAbABvAGMAYQB0AGkAbwBuACAAfQA6ACAAewAgAGwAbwBjAGEAdABpAG8AbgA6ACAAewAgAHEAdQBlAHIAeQA%2FADoAIAB7ACAAcwBjAGgAZQBtAGEASQBkAD8AOgAgAHMAdAByAGkAbgBnACAAfQAgAH0AIAB9ACkAIAA9AD4ACgAJAAkAbABvAGMAYQB0AGkAbwBuAC4AcQB1AGUAcgB5AD8ALgBzAGMAaABlAG0AYQBJAGQACgAJAAkACQA%2FACAAewAKAAkACQAJAAkACQBzAGMAaABlAG0AYQBJAGQAOgAgAGwAbwBjAGEAdABpAG8AbgAuAHEAdQBlAHIAeQAuAHMAYwBoAGUAbQBhAEkAZAAsAAoACQAJAAkACQB9AAoACQAJAAkAOgAgAHsAfQAsAAoACQBnAGUAdABDAG8AbQBwAG8AbgBlAG4AdAA6ACAAKAApACAAPQA%2BACAAaQBtAHAAbwByAHQAKAAvACoAIAB3AGUAYgBwAGEAYwBrAEMAaAB1AG4AawBOAGEAbQBlADoAIAAiAG4AYQB0AGkAdgBlACIAIAAqAC8AIAAnAC4ALwBTAHQAbwByAHkATgBhAHQAaQB2AGUAQwByAGUAYQB0AGUAJwApAC4AdABoAGUAbgAoAG0AbwBkACAAPQA%2BACAAbQBvAGQALgBkAGUAZgBhAHUAbAB0ACkALAAKAH0A
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

hum, sorry for the noise, it' doesn't seems to showup in the playground but
export const StoryNativeCreate = {
	name: 'StoryNativeCreate',
	path: 'mobile/editor',
	prepareVariables: (_: any, { location }: { location: { query?: { schemaId?: string } } }) =>
		location.query?.schemaId
			? {
					schemaId: location.query.schemaId,
				}
			: {},
	getComponent: () => import(/* webpackChunkName: "native" */ './StoryNativeCreate').then(mod => mod.default),
}

seems to be reformated with a double space before the closing }

This isn't a bug, but it's the expected behaviour.
Ternaries are indented with spaces, regardless.
Prettier works in the same way https://prettier.io/playground/#N4Igxg9gdgLgprEAuEcAeAHCAnGACSKAZ3wGUYcBPAOQEMYBLANzgGFs5648BePYADpQBMKLQC2cJHgDk5KnUYt2neDIA0QkRnoALaTPEQARgwA2cAPRwAJgwrYNWmBg46OANVrYGtYxaJpAAoAfWlaKEp1fjwzCDB6Bmg8AF9pYFj4xOh0vABHAFc4bEoAflyiMF04cVoASRtyvBIfKABzVM6UgEpeAD5nETiExmgAOkLisrHK6tqGwZgRUv5FkXWYWZr6m2lh7KgJopKZqu2GzWEl65EUtZh0lMuRNrgYVghxLCgEB7wg3o8Pp4BhfHAwIKWABUeAA7nBjDowABrVi6ApQZF0STSAQgMRKOB4vBQyyyMaWeQlRTMNgcLgybpjGDVKBBIw2fp4DljGxwABmtAKZhg3UuKRA6hAEAwo2IyFA3mwEFhAAVvAgiMgQLQzLDaJQtVLjNhaCi3qQJHAADIMH7IQVmIhwY2m80wUhIu1tZAwbBFKXO8QMX3+l2oTDFUG-XUAFWKYh8cC1SEdzsD3osAEUChB4A7demQAArIhoUiZuA5vNwAtO8OFGuq5UYFM6ogAWh+tlskpAfto5m9H3EtW1urMfaIlYAgjA-QxjAV4Krirb7anC+HdDBxGYAOq6ezJpFwUia+zMeyUbVgIhGkBMIp1KB82CkMA+WUz1-kSgWOsi1cCBnX3U0MG1Vxk2KFg+ztZ1cGbWg2jHTd6yldwEO1Yw-DgMwO02PtXDtGB9wYGwWWQAAOAAGKUOEKBgOCQlDaEA8MYD8MiKN0ZAADYpQKZ1Yz8FM03DGpjFsPkbGtCI2gKZC4AAMRwWp529cdlwgEAUhSIA

@ematipico thanks for the quick review :)
but if i'm correct, the link you provided did'nt use tabs 🤔
here https://prettier.io/playground/#N4Igxg9gdgLgprEAuEcAeAHCAnGACSKAZ3wGUYcBPAOQEMYBLANzgGFs5648BePYADpQBMKLQC2cJHgDk5KnUYt2neDIA0QkRnoALaTPEQARgwA2cAPRwAJgwrYNWmBg46OANVrYGtYxaJpAAoAfWlaKEp1fjwzCDB6Bmg8AF9pYFj4xOh0vABHAFc4bEoAflyiMF04cVoASRtyvBIfKABzVM6UgEpeAD5nETiExmgAOkLisrHK6tqGwZgRUv5FkXWYWZr6m2lh7KgJopKZqu2GzWEl65EUtZh0lMuRNrgYVghxLCgEB7wg3o8Pp4BhfHAwIKWABUeAA7nBjDowABrVi6ApQZF0STSAQgMRKOB4vBQyyyMaWeQlRTMNgcLgybpjGDVKBBIw2fp4DljGxwABmtAKZhg3UuKRA6hAEAwo2IyFA3mwEFhAAVvAgiMgQLQzLDaJQtVLjNhaCi3qQJHAADIMH7IQVmIhwY2m80wUhIu1tZAwbBFKXO8QMX3+l2oTDFUG-XUAFWKYh8cC1SEdzsD3osAEUChB4A7demQAArIhoUiZuA5vNwAtO8OFGuq5UYFM6ogAWh+tlskpAfto5m9H3EtW1urMfaIlYAgjA-QxjAV4Krirb7anC+HdDBxGYAOq6ezJpFwUia+zMeyUbVgIhGkBMIp1KB82CkMA+WUz1-kSgWOsi1cCBnX3U0MG1Vxk2KFg+ztZ1cGbWg2jHTd6yldwEO1Yw-DgMwO02PtXDtGB9wYGwWWQAAOAAGKUOEKBgOCQlDaEA8MYD8MiKN0ZAADYpQKZ1Yz8FM-QDVBxGMWw+Rsa0IjaApkLgAAxHBannb1x2XCAQBSFIgA
you can see that ternary use tabs without spaces 🤔
or i miss something?

Ternaries are weird in prettier.
The initial indentation is done using the indentation style configured in the options. That indentation is done until the ternary operator starts. Then the formatter adds two spaces for each level of ternary.

ok, i believe you ... but  if i run prettier & biome on the same code the result is different ...

Thank you, that wasn't very clear to me.
Can you please provide two playground links, one from biome and one from prettier?
We will take a look

here is the prettier link
https://prettier.io/playground/#N4Igxg9gdgLgprEAuEcAeAHCAnGACSKAZ3wGUYcBPAOQEMYBLANzgGFs5648BePYADpQBMKLQC2cJHgDk5KnUYt2neDIA0QkRnoALaTPEQARgwA2cAPRwAJgwrYNWmBg46OANVrYGtYxaJpAAoAfWkSHygAc3V+PDMIMHoGaDwAX2lgeMTk6Ey8AEcAVzhsSgB+fKIwXThxWgBJG0q8CIZo9M60gEpeAD5nEQSkxmgAOmLSirHq2vqmwZgRcv5FkXWYWbrGm2lh3KgJkrKZmu2mzWEl65E0tZhMtMuRKLgYVghxLCgEB7wg3o8Pp4BhfHAwIKWABUeAA7nBjDowABrVi6IpQZF0STSAQgMRKOB4vBQyyyMaWeRlRTMNgcLgybpjGC1KBBIw2fp4DljGxwABmtCKZhg3UuaRA6hAEAwo2IyFA3mwEFhAAVvAgiMgQLQzLDaJQtVLjNhaCi3qQJHAADLtODIQVmIhwY2m80wUhI9pRZAwbAlKXO8QMX3+l2oTClUG-XUAFVKYh8cC1SEdzsD3osAEUihB4KGAyAAFZENCkTNwHN5+2p3XpkDFauq5UYFM6ogAWh+tlskpAfto5m9H3E9W1urMfaIFYAgjA-QxjEV4KrSrafg66+HdDBxGYAOq6ezJpFwUia+zMeyUbVgIhGkBMEoNKB82CkMA+WUz1-kSgWTcnXDVwIGdfdTQwbVXGTUoWD7dpnVwZtaCiMdayAqV3EQ7UYEoDBk0-BhZT7Vx2hgfcGBsFlkAARgAJgABilDhigYDhkNQ2hAPrGA-AoqjdGQOipSKZ1Yz8FM-ULOpjFsPkbGtWhoiKFC4AAMRwep529cdlwgEA0jSIA
but  strangely the biome playground above thinks prettier insert spaces indeed, but it does not ...
maybe that changed at some point?

It seems that prettier is actually buggy. The indentation of } isn't respected.

i think it's due to the tabwith ..
like if you display the tab chars it makes more sense (ofc it doesn't look good in the playground with incorrect tabwith

sorry, i really don't want to be pusshy, but i think i'm and prettier's playground are on the same version , the latest (3.2.4)
the issue is that the playground displays the tabs as bigger
if you (ultra crappy) change the tab size to 2  artificially, and make them visible (using '.' for example)
like  (😬 )
$$('.cm-tab').forEach(e=> {console.log(e.innerHTML ='..')})
you end up with the something similar to 👆

and wich , IMHO, seems fine like this (doesn't mix tab and space for prefix indentation)

this awesome! i'll thanks for your work :)
i'll let you know when this is testable :)


Add support for optional templates in context menu, instead of hardcoded ones

First of all, thanks for making this, it definitely saved me some time and frustration :smile_cat:.
I have a use case where I need to both internationalize my context menu and have some different options than what was provided by default in this library. So I added support for an optional mainTemplate and suggestionsTemplate parameter to accomplish that end. If those params aren't provided, the defaults originally created by you are used instead.
I also swapped out underscore for lodash so I could get just the needed functions to keep the library a little slimmer (and thus make the final package slimmer too).
So everything is backwards compatible and you get some more flexibility. Let me know what you think and if you'd like to see some changes. Thanks!

Looks good @ccnokes, clean implementation and thanks for the thorough description!
If your menu items are not particular to your application, e.g. they're just localized versions of what I have here, I'd love if you found a way to bring that functionality into the module! i.e. so that the menu could be automatically localized for whatever languages you're supporting.

Thanks for the quick response!
I fixed the mistakes in the documentation on that function.
As far as "localizing the default template" -- I agree that would be cool, and I thought about doing that but then wasn't sure how to go about it :smile:. Without breaking the API, it seems like the best option would be to pass in a map of { key: translatedLabel }, apply it somehow to the menu config, and fallback to english labels if no localization map is provided. Do you have any ideas or preference on what that implementation would look like? I might be able to throw something together later tonight on that.

it seems like the best option would be to pass in a map of { key: translatedLabel }, apply it somehow to the menu config, and fallback to english labels if no localization map is provided.

So, are you adding application-specific items (and) or localizing the defaults?
If you're just localizing the defaults, I think it would make sense to have the library internalize the localization map. Rather than pass in a mainTemplate that contains the same items as DEFAULT_MAIN_TPL, just localized, the library would itself select a version of DEFAULT_MAIN_TPL that was localized.
If your mainTemplate contains only some of the same items as DEFAULT_MAIN_TPL and/or some application-specific items, then this gets harder. Two ideas:

Once the client starts to customize the menu, then localization is on them.
Or, we change this API to let the user edit (or replace) the DEFAULT_MAIN_TPL rather than just replacing it:

var buildEditorContextMenu = function(selection, mainTemplateEditor, suggestionsTemplateEditor) {
  /* snip */
  var template = _.clone(DEFAULT_MAIN_TPL);
  if (mainTemplateEditor) template = mainTemplateEditor(template);
 /* snip */
}

var menu = buildEditorContextMenu(selection, function(mainTemplate) {
  /* add menu item at the top */
  mainTemplate.unshift(myItem);

  /* or splice them in based on role */
  var indexOfCut = _.findIndex(mainTemplate, function(item) {
    return item.role === 'cut';
  });
  if (indexOfCut > 0) mainTemplate.splice(indexOfCut + 1, 0, myItem);

  return mainTemplate;
});

Splicing items anywhere except the beginning or end is perhaps a bit fragile but then again the roles are stable. This latter option would then allow the library to internalize localization even if the user was adding items.
What do you think?

So, are you adding application-specific items (and) or localizing the defaults?

Now, I'm just localizing the defaults. (I thought I was doing both originally. I'm at a new job so still figuring everything out). Regardless, it'd be nice long term to know that this menu could be flexible with adding new items if needed.
I like that idea of having the user pass functions for the templates that receive the default template. That gives you the flexibility to replace it entirely, do one-off changes, or just localize the labels however makes sense for you.
I'll work on that today. Thanks for working with me on this.

Thanks for working with me on this.

Likewise!

Ok, changes are as discussed with the exception that if you just pass an array for the two template params, they get used as is. I thought that would make the use case of a totally custom implementation easier so that you wouldn't have to do this:
buildEditorContextMenu(selection, mainTemplate => mainTemplate, suggestionTemplate => suggestionTemplate)

Let me know if there need to be any more changes :smiley_cat:

Looks great @ccnokes! I like the idea of permitting the parameters to be functions or arrays.
The last thing would be—if you're just localizing the defaults as you suggested above—how about putting your localizations into the library? Then you wouldn't need to pass the second and third parameters at all :)

I would, but I don't have them yet :frowning:. Maybe in a month or two? When they're done, I'll definitely contribute them. I personally think the passing the last two params still has value if you ever want something more custom.

Oh ok! Yes please do contribute the localizations if/when possible, but this looks great for the meantime! I definitely agree that these new params will benefit users in more custom scenarios.
Thanks for the PR!

Published v1.1.0.

Awesome, thanks!


Thanos rule metrics missing after upgrade to 0.12.0.

Thanos, Prometheus and Golang version used:
# thanos --version
thanos, version 0.12.0 (branch: HEAD, revision: 69f9c4e604d20b2e640f790957414cf3d19ecafa)
  build user:       root@aab79ed2e8d5
  build date:       20200415-09:34:28
  go version:       go1.13.6

Object Storage Provider:
irrelevant
What happened:
Thanos rule metrics endpoint is not reachable after Thanos upgrade to 0.12.0
# ps aux | grep "thanos rule"
prometh+   1149  0.0  2.4<PHONE_NUMBER>0 ?        Ssl  Apr15   3:26 /usr/bin/thanos rule --data-dir=/var/lib/thanos/rule --http-address=<IP_ADDRESS>:19902 --grpc-address=<IP_ADDRESS>:19901 --rule-file=/etc/prometheus/thanos_alerts.yml --query=localhost:19192 --alertmanagers.url=https://alerts.example.com --alert.query-url=https://metrics.example.com/all --web.external-prefix=https://metrics.example.com/thanos/query

# netstat -tulpn | grep thanos
tcp6       0      0 :::19192                :::*                    LISTEN      1151/thanos
tcp6       0      0 :::19901                :::*                    LISTEN      1149/thanos
tcp6       0      0 :::19902                :::*                    LISTEN      1149/thanos
tcp6       0      0 :::19092                :::*                    LISTEN      1151/thanos

# curl localhost:19902/
Found.

# curl --max-time 10 localhost:19902/metrics
curl: (28) Operation timed out after 10001 milliseconds with 0 out of -1 bytes received

What you expected to happen:
Metrics to be returned
How to reproduce it (as minimally and precisely as possible):
See above
Full logs to relevant components:
Nothing in logs (with info log level), I can try debug logging if necessary.

Anything else we need to know:
Previously system was running Thanos 0.11.0 and metrics were working fine. The same system has Thanos query running and metrics for that service are returned just fine.
Environment:
# cat /etc/*release
CentOS Linux release 7.7.1908 (Core)
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

CentOS Linux release 7.7.1908 (Core)
CentOS Linux release 7.7.1908 (Core)
# uname -a
Linux hostname 3.10.0-1062.18.1.el7.x86_64 #1 SMP Tue Mar 17 23:49:17 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

@yeya24, thanks for looking into this!
Additionally to the original issue, it seems that URL is rendered incorrectly here (only 1 slash before hostname):
# curl localhost:19902/
Found.

Thanks for report and details!
Looks like a duplicate of https://github.com/thanos-io/thanos/issues/2486

Hi @karlism could you please try the latest release and see if your issue still exists.
But to me, it seems this issue is unrelated to #2484. You only configure --web.external-prefix=https://metrics.example.com/thanos/query and didn't configure --web.route-prefix.
Also, the missing endpoint is /metrics, which is unrelated to --web.route-prefix as well.

@yeya24, thanks! It is working now and /metrics endpoint returns metrics properly.
I'm not certain --web.route-prefix is necessary in my case as --web.external-prefix points to NGINX load balancer that forwards connections to multiple Thanos servers in different datacenters.

Thanks for confirming!


Add support for RUST_SRC_URL var at runtime

With this PR, now the ordering to find rustc repo path is:

RUST_SRC_URL at runtime
rustc.git in current dir at runtime
RUST_SRC_URL set at compilation time

Closes #104.

I moved the upstream related changes to other PRs.


Added new methods

Added TokenSAR, SentenceSAR, and SAR methods from https://arxiv.org/abs/2307.01379
Added RényiNeg and FisherRao methods from https://arxiv.org/pdf/2212.09171.pdf (EMNLP-2023 paper)
I think, we need to return log_softmax in greedy_probs https://huggingface.co/docs/transformers/internal/generation_utils#transformers.generation.GenerateDecoderOnlyOutput.scores

We already apply log_softmax inside model.generate, as it uses this _ScoresProcessor: https://github.com/IINemo/lm-polygraph/blob/main/src/lm_polygraph/utils/model.py#L285

@cant-access-rediska0123 I see, thank you. Then, it seems we need to remove the duplicated log_softmax(-1) here:
https://github.com/IINemo/lm-polygraph/blob/main/src/lm_polygraph/utils/model.py#L310
https://github.com/IINemo/lm-polygraph/blob/main/src/lm_polygraph/stat_calculators/sample.py#L82

@cant-access-rediska0123  Can we approve this request?


Fix dashboard UI bugs and more

This PR fixes a few UI bugs I spontaneously encountered:

Fixes emojis in repo titles getting head-cut and tail-cut in dashboard feed due to introduction of 1.25 em emojis at 1 em line-height, by simply using the original 1 3/7 em value of semantic.css
Fixes regression (too long repo names should be capped to 70%) in #13828 due to flex children not respecting properties like overflow: hidden;, and removes a block of dead style code
Follow-up to #14761, removes extraneous code for top navbar and correct right margin for Font Awesome
Fixes color emphasis inversion in arc-green theme for top right buttons (edit, delete) on commit view boxes

Just came to the idea to use the server octicon for admin settings. Looks pretty understandable to me. Will add if people like it.

Could you give some screenshots?

@noerw seems to be confused about what I meant to replace with the server icon. It's this button here. Do you think it's a good replacement to the triple gear icon?
Currently in this PR:

Proposal:

Pushing for now so I won't forget about it if PR takes too long. Will revert last commit if people object the change.


how to call fcntl?

Hello,
I'd like to call this C code on Linux:
#define _GNU_SOURCE
#include <unistd.h>
#include <fcntl.h>
#include <stdio.h>

fcntl(fd_num, F_SETPIPE_SZ, 1048576)

How to do this using ctypes and ctypes.foreign?
An example that works in utop would be nice.
Is this the OCaml code I should write?
let fcntl = Ctypes.FOREIGN.foreign "fcntl" (int @-> int @-> int @-> returning int)

There isn't a reliable way to call variadic functions using ctypes.foreign.
I recommend using stub generation (ctypes.stubs) instead and, for fcntl specifically, using the unix-fcntl package.

I really wonder if it is not simple for me to write a C wrapper by hand...


Fixup pin libx11 by bumping the build number

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

I think the defaults changed in conda. or for some reason, now the pinning is always >=1.7 instead of exactly 1.6

Ok this is because the pinnings were changed in https://github.com/conda-forge/xorg-libxcomposite-feedstock/pull/2 but the number wasn't updated.

I don't mean anything malicious by the comment. It is really hard to know when to change and when not to chane the build number.
I was just trying to understand "what changed".
Thank you for helping maintain and improve these recipes.

No worries. Thank you! Does this close #4?