DRAFT: Refactor job runners

Issue # (if applicable)
Closes #.
Reason for this change

Description of changes

Description of how you validated changes

Checklist

[ ] My code adheres to the CONTRIBUTING GUIDE

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

Spoke with @tobuck-aws . Closing as this is no longer needed.


Static functions called before declaration

STC3115_WriteDataToRam and STC3115_UpdateRamCRC are callled before there declarations.
Could be better to declare all static functions on top of the file.

STC31xx_SaveBackupSocToRam function needs to be declared just before "End of Internal functions" comment


Make changes to TCRD finder

https://trello.com/c/QUZiTIcs/3268-changes-to-the-specialist-publisher-page-traffic-commissioner-regulatory-decisions-govt-agency-general-issue
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
This application is owned by the publishing platform team. Please let us know in #govuk-publishing-platform when you raise any PRs.
Follow these steps if you are doing a Rails upgrade.

Hi @kashifatcha, is this ready for review?  If not, could you mark the PR as draft so it doesn't appear in our team's "seal" as requiring review.  Thanks!

Hi @kashifatcha, is this ready for review? If not, could you mark the PR as draft so it doesn't appear in our team's "seal" as requiring review. Thanks!

Whoops! My bad, changed it to draft


bugfix: modify route to cors

modify route to cors in gzip.go

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

Comparison is base (89c7277) 38.27% compared to head (b794e45) 38.14%.
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #750      +/-   ##
==========================================
- Coverage   38.27%   38.14%   -0.14%
==========================================
  Files          60       60
  Lines        9979     9979
==========================================
- Hits         3819     3806      -13
- Misses       5869     5880      +11
- Partials      291      293       +2

Files
Coverage Δ

pkg/ingress/kube/configmap/gzip.go
45.29% <0.00%> (ø)

... and 2 files with indirect coverage changes


Compatibility with purge

When setting up tailwind'd purge feature, our named groups appear to be purged.  Is it possible to configure named groups in the safelist?

As of Tailwind 3.0 I was having some problems with JIT purging classes inconsistently for the named groups (especially when having nested groups - usually would take the parent and purge the child).
I used the following in my config:
const config = {
...
  safelist: [{
    pattern: /.*/,
    variants: [
      "group-hover",
      "group-focus"
    ]
  }],
...
};

Note - this is working because it's safelisting all group tags of hover and focus across all the variants (so it may not be the smallest bundle you can get to -- you may want to narrow down the pattern a bit), but it works for my use-case.

Extending on the above answer, my working code for tailwind 3.x in tailwind.config.js is:
...
module.exports = {
  content: [
    './resources/**/*.blade.php',
    './resources/**/*.js',
    './resources/**/*.vue',
  ],
  // safelisting so that namedGroups can work in tailwind 3.x
  safelist: [{
    pattern: /.*/,
    variants: [
      "group-hover",
      "group-focus"
    ]
  }],
...

Thanks @m-rgba


Activation of DeepSource

Hi 👋
One of my Pull Requests around fixing Code Quality Issues with DeepSource was merged here: https://github.com/girlscript/feminist-bible-phase-2/pull/206
I'd just like to inform you that the issues fixed here were detected by running DeepSource analysis on the repo. If you like, you can activate analysis for your repository to detect such code quality issues/bug risks on the fly for every change made. You can also use the Autofix feature to fix them with one click.
The .deepsource.toml file you merged will only take effect if you activate analysis for this repo.
Here's what you can do if you wish to activate DeepSource to continuously analyze your repository:

Sign up on DeepSource and activate analysis for this repository.
Create .deepsource.toml configuration which you can use to configure your analysis settings (My PR already added that, but feel free to edit it anytime).
Track/Check analysis here.

If you have any doubts or questions, you can check out the docs, or feel free to reach out :)

Hi @HarshCasper! for activating DeepSource admin's approval is required. This repository is a part of the GirlScript Foundation Organization. So, it may take some time to get the request approved by the owner.

Hi @HarshCasper! for activating DeepSource admin's approval is required. This repository is a part of the GirlScript Foundation Organization. So, it may take some time to get the request approved by the owner.

Sure. Please let me know if any help would be needed.


Use stack name in Service Catalog Application name for multiple cfn stack deployments

Issue #, if available:

Description of changes:

Add support for multiple stack deployments in the account, region with different Service Catalog AppRegistry Application
Checklist

[X] :wave: I have added unit tests for all code changes.
[X] :wave: I have run the unit tests, and all unit tests have passed.
[ ] :warning: This pull request might incur a breaking change.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Should this be 4 separate commits? Or can it be all in one?


UI-Elements aren't accessible due to big menu.

Arch-Linux 64-Bit, Godot v.2.1.stable (xfce-desktop if it matters) resolution: 1366x768
I have an isssue that bothers me for quite a while now. If the menu over the editor gets to big the whole UI stretches even if there isn't enough space on the screen, resulting in inaccessibility of the panes on the right (Where usually the Inspector resides). I am also not able to resize the editor space in the left direction. To get to see the Inspector again I need to click on something that makes the menu smaller and afterwards let the window redraw (by making it small and big again). It's a hassle.
I have an image that shows the issue (this is my fullscreen, as you can see the right side is not visible)

To reproduce this you just have click on a tilemap on small screen.
It would be nice if there would be some solution for the menu do go in an expander if there are to many items (or like it was done for tabs).
Thanks in advance.

Duplicate of #6461 I think


Add users from CFA

What type of help do you need?

[ ] Question
[ ] New Repo
[ ] Delete Repo
[x ] User Membership (please make sure new members are familiar with the CDC open practices and set up their profile with name and org info to help people collaborate with them)
[ ] Other

Please describe how you'd like us to help.
I am a data scientist in CFA's predict division, and I'd like to add myself and several members of my team to CDCgov:

Katelyn Gostic (cdc<EMAIL_ADDRESS>github: kgostic)
Scott Olesen<EMAIL_ADDRESS>swo)
Nishant Kishore<EMAIL_ADDRESS>nish-kishore)
Danielle Richard<EMAIL_ADDRESS>dmrichard)
Jason Asher<EMAIL_ADDRESS>jasonasher)

A bit about CFA and our goals: We are a new center focused on outbreak modeling and data analytics. We would like to start a few public repos for organizational purposes (a style guide in bookdown; a repo of utility scripts for common models, analyses, and inference procedures, etc.), and as we staff up we're interested in developing modeling tools using open practices in CDCgov.

Invites sent for those five users to CDCgov.


Spp 10072 thousand pounds uat test cases

Synopsis
The purpose of this ticket is to add the UAT tests to the existing thousand pounds correction tests already written.
The code also removes the 2.d.p rounding and ensures predictive and auxiliary equalling zero returns original target variables.
Checklist

[x] Documentation created/updated
[x] Tests created/updated

Description
Note: It is important to mention that this branch also has main branch changes so a lot of linting and structure changes made by team members will appear here the only changes we need to focus on that are relevant to this branch is the changes to the check zero errors in thousand_pounds.py and the added UAT tests for this method

@jhmjam looks good to me, would you mind having a look at https://github.com/ONSdigital/sml-python-small/pull/57 as this branch builds from it please, thanks


Some Editor-Shortcuts commands don't work in godot 4

Godot version
4
System information
Windows
Issue description
For example I can't call this command with a shortcut

In godot 3 it opens docs for edited object class
Steps to reproduce
try to set a shortcut to Open Documentation command (from Editor-Shortcuts), then open Inspector and press the shortcut. It does nothing for me
Minimal reproduction project

Unable to reproduce it in following versions:
v4.0.2.stable.official [7a0977ce2]
v4.1.dev.custom_build [bd1bc68ba]
Screencast from the latest:


Handle race conditions in exclusive placement

Now that #310 has been addressed via the pod webhook approach, we need to handle certain race conditions which can occur in this strategy.
Consider the following race condition:

Leader of Job 1 getting killed right after we set the nodeSelector on its follower pods
At the same time, leader of Job 2 lands on the same slice before any of Job 1's followers get to schedule on it. In this case Job 1's followers will have a nodeSelector on the node pool, but will never schedule because of the anti-affinity of Job 2's leader.

/assign


Change Background Color for iOS Widget

The green color for the widget clashes with most things aesthetically.
It would be helpful to either change the color to the same color as the default iOS widgets, or to allow the user to change the widget background color.

This is the current color in Dark Mode:


Exit build xcframework if swiftformat is unavailable

On setup on a machine with no version of swiftformat available, the sed workaround to catch Rust panics will not be executed properly due to the Swift file not being formatted as expected by the script, leaving the resulting library in a state where Rust panics would crash the hosting application instead of recovering.
This change ensures that make ios or ./build_xcframework.sh fails explicitly if the tool is not installed.

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

Comparison is base (90754f6) 88.38% compared to head (a5c6896) 89.91%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #835      +/-   ##
============================================
+ Coverage     88.38%   89.91%   +1.52%
============================================
  Files           157       82      -75
  Lines         18104    14767    -3337
  Branches        945        0     -945
============================================
- Hits          16002    13278    -2724
+ Misses         1853     1489     -364
+ Partials        249        0     -249

Flag
Coverage Δ

uitests
?

uitests-android
?

uitests-ios
?

unittests
89.91% <ø> (+3.24%)
:arrow_up:

unittests-android
?

unittests-ios
?

unittests-rust
89.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 75 files with indirect coverage changes

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


REST API walk-through

Do you think this file might make more sense  under /docs/ rather than under /examples/?
Either way, we should add a link to it at the end of /docs/GettingStarted.md.

I have moved the file to /docs and added the link at the end of /docs/GettingStarted.md.

Thanks! LGTM


[Math Processing Error] on iOS / Safari

http://colah.github.io/posts/2015-08-Understanding-LSTMs/
iOS 9.2.1 / iPad Mini 2
Safari shows [Math Processing Error] where equations should be (e.g. 7 errors in first paragraph of "Step-by-Step LSTM Walk Through")

I cannot reproduce this, cf. https://saucelabs.com/beta/tests/e89f32e91de249d894238c3e5c9a0a34.
Most likely this is due to corrupted files or mixed versions in the cache; try cleaning MathJax and its components from the browser cache.

I cleared the cache and the issue was gone, only to reappear on another site and then disappear when I clicked refresh.  Odd. Perhaps a caching bug in Safari or some issue with using "mathjax/latest/MathJax.js" instead of a specific version. Closing since I can't reproduce the problem any more.

Perhaps a caching bug in Safari

Some browsers need a restart to reliably clear the cache. Chrome is one (and nowadays even reports that in the developer console if you make a hard refresh for a particular page); I don't recall if Safari suffers the same. Let us know if it recurs.


project page review

eb67281
@Setnea please review our project's page content

This is stale and non-specific issue. Closing


Rename repo

I'd like to rename this repo to something other than md-format to avoid confusing with other markdown formatters. It may eventually do formatting, but let's not start there. I plan on renaming the repo to something like md-examiner.

Maybe just md-check.

Done.


test11

Effects

[ ] Soft update
[ ] Breaking change
[ ] Chain fork related

Major Reviewer

Background

Summary

Checklist

[ ] Backward compatible?
[ ] Test enough in your local environment?
[ ] Add related test cases?

test end


Test Select component

Fixes an import path that was broken by extracting yew-components to a separate crate.

Thanks for fixing this!


HuggingFace conversion script doesn't work

First of all thanks for this amazing open source work!
Second, I converted eng-pol model to HF format with instruction: https://github.com/huggingface/transformers/tree/master/scripts/tatoeba, but when I use it with transformers library I get sequence of dots ".................." as result. I also tried with pol-eng which is available in HF repo as opus-mt-pl-en but no success again.
Do you have any idea what am I doing wrong?
Thanks

There was a bug in the conversion script that assumed that special tokens are in the beginning of the vocabulary. That has been fixed recently and it should work with the latest source code. Note, that we also work on updating the conversion procedures to make them more flexible with other variants of marian models. That is not yet ready but hopefully will come soon.


[Question]

Love Win-Vind!
Is it possible to start the program in GUI Insert instead of GUI Normal? This way the uninitiated will not think the keyboard stopped working after each restart....
Thanks.

Thank you for using it!
Yes. Please set GUI Insert at Preferences/settings/Initial Mode.

Fantastic!


enforcing correct minimum/maximum constraint formats for specific types

As described more in https://github.com/aws-cloudformation/cloudformation-cli/issues/414:
string uses minLength/maxLength
number/integer uses minimum/maximum/etc.
frequently mixed up and doesn't seem to be caught during jsonschema validation yet

Hi there. Was this meant to be filed here?
The behavior described from a quick glance at that is the behavior specified by the JSON Schema specification.
Validators which only apply to certain types pass silently on other types.
If you want to protect against that you make sure you're validating type as well essentially.
Let me know if I've misunderstood the intention here.

Validators which only apply to certain types pass silently on other types.

Meant to ask about this. Could keywords which only apply to other types be caught during schema validation?

+1 this would be a huge improvement to the validation.

I think a decent way to think about this is to write down a metaschema that does what you want.
Not sure it'd even be a very long schema, it's something like {"required": ["type"], "allOf": [{"properties": {"type": {"const" : "string"}}, {"not" : {"properties" : {"minimum": {}, "maximum" : {}}}], ... etc with the other two cases for array and object.
Don't know that I see something particularly special needed at the library level to do so once such a schema is written down.

This could definitely be handled just on our side, but thought this would be generally useful since many JSON schemas are mixing up these similar but slightly different JSON schema keywords

Right that metaschema would be usable by anyone presumably -- it'd be a way to validate that a schema doesn't use validators except within particular types.
Once we had it though anyone who wanted this behavior could validate against it.

there are actually 10 different min/max JSON schema keywords across 5 different types, which felt like a lot to schematize, so checking in Python for now:
https://github.com/aws-cloudformation/cloudformation-cli/pull/675

Fair enough, glad you got a solution that works.
I'm still a bit reluctant to add anything here given that this indeed can be specified within JSON Schema without any special support -- if someone writes down that metaschema that likely would help folks who want to do this (perhaps worth discussing upstream), but for now will close this given that I think a solution like that should address this generically without any additional work for end-users.

Similar problem with additionalProperties vs additionalItems if anyone else is looking into this


Getting Started with DBSCAN Clustering Algorithm in Machine Learning

Topic Suggestion
Pre-submission advice
By following all our pre-submission advice and reviewing our Resources folder, you will maximise your chances of your topic being approved.
We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner.
Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles that will help you propose and write a successful article.
Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Proposal Submission
Getting Started with DBSCAN Clustering Algorithm in Machine Learning
Proposed article introduction
DBSCAN is a more advanced clustering algorithm compared to the K-means clustering and the hierarchical clustering algorithms. First of all, DBSCAN stands for ''Density-Based Spatial Clustering of Applications with Noise''. Unlike the K-Means algorithm, the best thing with this algorithm is that we need not provide the number of clusters required prior. DBSCAN algorithm groups points that are close to each other together based on distance measurement, usually Euclidean distance and a minimum number of points. An important property of this algorithm is that it helps us track down the outliers as the points in low-density regions. The DBSCAN algorithm requires two parameters, i.e.,

Eps: This is the maximum distance between two points for them to be considered as a neighbour. It is considered as a threshold for considering two points as neighbours. If the distance between any two points is less than or equal to the Eps, these points are considered neighbours.
MinPoints: This refers to the minimum number of points required to form a cluster. It is considered as a threshold for considering a cluster as a cluster. Basically, if the number of points in a cluster is greater than or equal to the MinPoints, this cluster is considered as a cluster.

Types of Points in a DBSCAN Cluster
There are three types of points after the DBSCAN clustering is complete. Let's understand these points:

Core: It is a point with at least m points within distance n from itself. (what m and n are to be explained in the article)
Border: This point has at least one Core point at a distance n.
Noise: which is a point that is neither a Core nor a Border. And it has less than m points within distance n from itself.

Further explanation to be provided in the article

DBSCAN Algorithm
Finally, we will look at the algorithmic steps for DBSCAN.
These algorithmic steps will be as follows:

Step 1: Find all the neighbour points within Eps and identify the core points or visit with more than MinPts neighbours.
Step 2: For each core point, if it is not assigned to a cluster, create a new cluster.
Step 3: Find recursively all its density connected points and assign them to the same cluster as the core point.
Step 4: Iterate through the remaining unvisited points in the dataset. Those points that do not belong to any cluster are noise.

In the article, we shall discuss the above intuition in detail, make every subsection clear enough, and then implement our algorithmic steps above in python to obtain our clustering model.
Key takeaways

The learner will be able to define the DBSCAN  algorithm.
The learner will know the advantages of this algorithm over the other clustering algorithms.
The learner will know the Parameters in the DBSCAN Model.
The learner will know the types of points in the DBSCAN clustering.
Finally, the learner will understand the flow of the DBSCAN algorithm and how to implement it in python.

Article quality
This article will focus on the clarity of the concept and delivery of the well-structured content. Therefore, no unnecessary details will be included in this material.
References
N/A
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

good topic @Daniel695  - approved. Thanks for submitting your article and we look forward to reviewing and hopefully publishing it. I’m sure our readers will enjoy and find your article  very beneficial.


Add Support for Directory.Build.[props/targets] files

Overview
As outlined in this article, the Directory.Build.targets and Directory.Build.props files can also contain this version information. It would be nice to include support in libyear.
Resources
More information from MS here.

Thanks for this new feature -- same feedback about the whitespace, but looking forward to the additional support!

Rebased to master in 1c8ae73

Thanks again!


Tasks running same npm script in other cwd are ignored and "don't exist"

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.86.2
OS Version: Windows_NT x64 10.0.22631

Steps to Reproduce:

Use the following tasks.json:

{
  "version": "2.0.0",
  "tasks": [
    {
      "label": "service.a",
      "options": {"cwd": "${workspaceFolder}/packages/service.a"},
      "script": "example",
      "type": "npm"
    },
    {
      "label": "service.b",
      "options": {"cwd": "${workspaceFolder}/packages/service.b"},
      "script": "example",
      "type": "npm"
    }
  ]
}

Try running service.a task. It won't exist.

Expected behavior:
I expect both tasks to exist and be able to get launched. It seems that service.b overwrites service.a because both run example script, but the task manager doesn't seem to understand that the options are different (different cwd). To work around this, I need to name scripts different, which causes issues for a monorepo structure.

If you want to run both tasks simultaneously, you can create a new task that runs both tasks in parallel. Here's an example of how you can do this:
{
"version": "2.0.0",
"tasks": [
{
"label": "service.a",
"options": {"cwd": "${workspaceFolder}/packages/service.a"},
"script": "example",
"type": "npm"
},
{
"label": "service.b",
"options": {"cwd": "${workspaceFolder}/packages/service.b"},
"script": "example",
"type": "npm"
},
{
"label": "service.a and service.b",
"dependsOn": ["service.a", "service.b"],
"type": "parallel"
}
]
}

If you want to run both tasks simultaneously, you can create a new task that runs both tasks in parallel. Here's an example of how you can do this:

That is not what this bug report is about. Besides that, your snippet doesn't even work. The bug still exists, making service.a non-existent, and type = parallel doesn't exist either.

/duplicate #198582


Driver API

Do you consider adding the new API?
https://medium.com/uber-developers/uber-driver-api-b7191231e04d#.18p9hzolt
https://developer.uber.com/docs/drivers

@scaryguy Should add it. Waiting for Uber to give me access to this API so that I can experiment and implement it.

@chardane Hey! Can something be done regarding access? 😄

hey @AnkurGel ! I'm super bummed to say that I cannot give you access at this time since we're only giving out access to business partners right now. 😞 But, what I can do is I can help test it! 😄  You can build / mock with the documentation for the three endpoints in the Driver API and I can help test to verify that it works.
We are working on getting sandbox-only access to developers, so that people can easily build for new features like this. I'll definitely let you know when that happens!

@chardane . Sure. It's ok :) Definitely let me know when sandbox mode access is available for this.
For implementation, I'm tracking it here - https://github.com/ankurgel/uber-ruby/pull/1, I still have to clean-up, write tests and documentation.Meanwhile, if you get time, do give it a try and check client.partners.me, client.partners.payments, client.partners.trips.

@AnkurGel hey can you DM me your client ID on twitter? I'm @chardane there as well. We can give you access so you can test this feature yourself! :)

Thanks. I'm done with implementation - https://github.com/AnkurGel/uber-ruby/pull/1.
@sishen @chardan Please review it? :)

beautiful! will find some time to test this week! 🎉

Hey @AnkurGel , thanks for the fast implementation!
But I'm stuck at some point. Even without going to production, do I need to apply for access to this partners API?
Currently, when I try to -for example- access client.partners.me, I get 401 response.
When I check out my app at developer.uber.com, I can't select the check boxes because they are disabled. Do I need to apply for it by clicking 'Request Full Access' in order to be able to develop something using Partners API in even sandbox mode? But that form seems to be only about production product application. Isn't it?
Thank you very much for your help in advance.

hi @scaryguy.
AFAIK, Partners API is not available in sandbox mode. Yes, they are only available in production mode and you will have to request access for that :)


Improved global exclusion list

These files do not contain secrets
Description:
From experience, some of the C# files do not contain secrets in any form..
Checklist:

[+] Does your PR pass tests?
[-] Have you written new tests for your changes?
[+] Have you lint your code locally prior to submission?

nice, we'll throw it in


pip-compile ignores deps with --pre in the input

Using --pre in a requirements line results in the dependency to be ignored:
% echo '--pre channels' | pip-compile -q - -o -
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile --output-file=- -
#
--index-url http://localhost:3141/root/pypi/+simple/

(Noticed this when coming up with #817)
pip-compile, version 3.6.2.dev2+ga3ac5ca

@blueyed
Do we really need --pre options in compiled requirements.txt? Since a package will be pinned with a prerelease version and would be installed on pip-sync anyways.
BTW, I've noticed that --pre doesn't work in requirements.in at all. See #819.

@atugushev
It is not needed, but I think it is a bug.
#819 is basically a duplicate of this one here.. :)

But since #819 has more details, I'm closing this one here.


Request assigned to Flex team member not showing in Flex team queue

Describe the bug in current situation
A clear and concise description of what the bug is.
GIVEN an FOI request (EDU-2022-00042, MHA-2022-00004, EDU-2022-00057) is assigned to a Flex team member
WHEN the user browses the 'My Team's Requests' queue
THEN the request does not appear in the queue

GIVEN an FOI request (EDU-2022-00042) is assigned to a Flex team member
WHEN the user searches the 'My Team's Requests' queue by the AXIS ID Number
THEN the missing request appears in the queue

Link bug to the User Story
Impact of this bug
Describe the impact, i.e. what the impact is, and number of users impacted.
High. Requests could be missed
Chance of Occurring (high/medium/low/very low)
High. Replicated 3 times with different flex team users.
Pre Conditions: which Env, any pre-requesites or assumptions to execute steps?
Steps to Reproduce
Steps to reproduce the behaviour:

Login as a Flex user
Go to 'My Team's Requests' queue
Click on an Open request
Click on the 'Assigned to' field
Scroll down to and select a flex team member
Save the assignment
Go to the  'My Team's Requests' queue
Search for the request that was just assigned
See that it is not listed on any page
Search by request ID in the 'My Team's Requests' queue
See the missing request listed

Actual/ observed behaviour/ results
Expected behaviour
A clear and concise description of what you expected to happen. Use the gherking language.
GIVEN a request is assigned to a Flex Team member
WHEN a flex team user goes to 'My Team's Requests' queue
THEN the assigned request should appear in the 'My Team's Requests' queue
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.

@mpilchar, @m-prodan, new bug for review with the team tomorrow.
cc @divyav-aot  @abin-aot

@lmullane i think i already captured this #2109. Will close mine

Thanks, @nkan-aot. Putting this into the Sprint Backlog for a dev to pickup.

This is now fixed. Thanks. @nkan-aot


Update ecto to 2.1 and pstgrex to 0.13

I added this functionality myself and released 1.1.0. Thanks for opening this PR!


Add Haskell Discourse link, remove G+

As suggested here: https://discourse.haskell.org/t/discourse-in-the-community-page/294

migrating this to the new repo


Setup goreleaser

Build is failing on darwin arm64. I'll remove it from the build target temporary until it is resolved.
actions/runs/3811615165/jobs/6492867206#step:4:51

I'll remove darwin from the build target temporary.
When cross-compiling from Linux to macOS, do I need to make any settings? I don't know how. Here's what the library I'm using says:

oto requires AudioToolbox.framework, but this is automatically linked.

😢

Is there any current progress on this issue? Are the problems being encountered coming from dependencies?


save_inventory_helper: TypedIndex to hold record index

a) Want to reduce the number of parameters passed around in save inventory helper
Also, isolating the index lookup will allow us to see if we can reduce number of objects created for our index lookups.

:+1: I like this refactoring.

@Fryguy thanks. done

@chessbyte thanks. done


API requests made to IP address instead of domain name.

I was writing a wrapper around the  google-cloud-storage-1.18.1 gem and while writing the tests I found that I am unable to stub the network requests. It is possible to stub the network request based on the IP address but that would be very error prone imo, I think it would be nice if a domain name was used instead of the IP address when making the request.
Environment details

OS: Arch Linux x86_64
Ruby version: ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-linux]
Gem name and version: google-cloud-storage-1.18.1

Steps to reproduce

See  the network request made in storage = Google::Cloud::Storage.new

Code example
storage = Google::Cloud::Storage.new
Full backtrace
     WebMock::NetConnectNotAllowedError:
       Real HTTP connections are disabled. Unregistered request: GET http://<IP_ADDRESS>/ with headers {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'Metadata-Flavor'=>'Google
', 'User-Agent'=>'Ruby'}

       You can stub this request with the following snippet:

       stub_request(:get, "http://<IP_ADDRESS>/").
         with(
           headers: {
          'Accept'=>'*/*',
          'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
          'Metadata-Flavor'=>'Google',
          'User-Agent'=>'Ruby'
           }).
         to_return(status: 200, body: "", headers: {})

google-cloud-env 2.0.0 introduces a more powerful stubbing mechanism. See https://rubydoc.info/gems/google-cloud-env/Google/Cloud/Env/ComputeMetadata/Overrides


Allow persistent icons

I am not so happy with this solution since it adds a persistent keyword in the stylesheet not existing in JS, but that's the easiest workaround now to enable persistent icons. We could detect if the icon comes from a client data source but this would get pretty messy since it introduces a lot of assumptions.

Thanks @karimnaaji, before we merge let's think a little more on syntax, in case we can think of future uses. For example, maybe something like a transition keyword? Default transition: fade, with this example being transition: false? Not sure if there are other obvious transitions though, so who knows :)  Could also be something like fade: false to be more explicit (persistent seems a little vague here).

Yeah this keyword should be extended to different options to clearly differentiate each cases.
The fact of colliding or not, the transition with an ease function and the time for the transition.
On Thu, Oct 1, 2015 at 6:12 PM, Brett Camper<EMAIL_ADDRESS>wrote:

Thanks @karimnaaji, before we merge let's think a little more on syntax, in case we can think of future uses. For example, maybe something like a transition keyword? Default transition: fade, with this example being transition: false? Not sure if there are other obvious transitions though, so who knows :)  Could also be something like fade: false to be more explicit (persistent seems a little vague here).
Reply to this email directly or view it on GitHub:
https://github.com/tangrams/tangram-es/pull/322#issuecomment-144863768

From the current implementation a proposed syntax could also be:
icons:
    collide: true | false # defaults to true
    hide:
        effect: fade | blink ...
        ease: pow | sine | ...
        time: 0.2s # sec or millis
    show:
        effect: fade
        ease: pow
        time: 0.1s

Seems like we'll need at least one or two more style sheet options for eraser-map functionality:
1. Enable/disable label fading: Could be a key like transition or fade. For semantic density, I'd suggest that the value be the desired duration of the transition in seconds. For our "find me" icon, we'd then use fade: 0 for an immediate transition. We could allow other values later for more sophisticated behaviors.
2. Enable/disable label collision: A separate key like collision would be one approach, but since this behavior is entangled with priority, it might make sense to encode collide-ability within that value. For instance, what if we used priority: 0 to denote that a label always "survives" collision? What this doesn't encode is how particular groups of labels interact - e.g. if we wanted to "mask" collision between certain groups.

    transition:
        event: [hide, show]
        effect: fade | blink ...
        ease: pow | sine | ...
        time: 0.2s # sec or millis

    transition:
        - event: hide
          effect: fade | blink ...
          ease: pow | sine | ...
          time: 0.2s # sec or millis

        - event: show
          effect: fade
          ease: pow
          time: 0.1s

icons:
    collide: true | false # defaults to true
    hide: &transition
        effect: fade | blink ...
        ease: pow | sine | ...
        time: 0.2s # sec or millis
    show: *transition

icons:
    collide: true | false # defaults to true
    transition:
        hide:
            effect: fade | blink ...
            ease: pow | sine | ...
            time: 0.2s # sec or millis
        show: hide
        selected: hide

icons:
    collide: true | false # defaults to true
    [hide, show]:
        effect: fade | blink ...
        ease: pow | sine | ...
        time: 0.2s # sec or millis
    selected:
        effect: fade
        ease: pow
        time: 0.1s

icons:
    collide: true | false # defaults to true
    transition:
        [hide, show]:
            effect: fade | blink ...
            ease: pow | sine | ...
            time: 0.2s # sec or millis
        selected:
            effect: fade
            ease: pow
            time: 0.1s

Minimal implementation:
    transition:
        [hide, show]:
            time: 0.2s # sec or millis
        selected:
            time: 0 # turn off effect

Initial with effect: fade | false
    transition:
        [hide, show]:
            effect: fade
            time: 0.2s
        selected:
            effect: false

This is actually invalid yaml:
transition:
    [hide, show]:

I'll go with
transition:
    hide, show:

Really? I tried this in a yaml validator just yesterday, to make sure I
wasn't crazy, and it worked fine
On Oct 7, 2015 10:18 AM, "Karim Naaji"<EMAIL_ADDRESS>wrote:

This is actually invalid yaml:
transition:
[hide, show]:
I'll go with
transition:
hide, show:
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram-es/pull/322#issuecomment-146208383.

This one doesn't want it: http://yaml-online-parser.appspot.com/

http://www.yamllint.com/
it works here.

Yes I believe it is valid YAML, from the spec, but I also think it is not commonly used, so it isn't shocking if some validators/parsers don't support it properly. In JS a [hide, show] key just gets de-serialized to hide,show anyway (I checked, the brackets are dropped).
transition:
    hide, show:

is not bad either and is also valid YAML (and to make JS decoding work, we can't allow commas in event names anyway, so there's functionally little or no difference as far as I can tell). But using a true sequence is probably most consistent.

@karimnaaji I just tried the sequence-key syntax in yaml-cpp and it seems to work in a simple test case:
TEST_CASE("Load a complex key into yaml", "[yaml]") {

    auto str = R"END(
    transitions:
        [show, hide]:
            effect: fade
            time: 1
        selected:
            effect: blink
            time: 0.5
    )END";

    YAML::Node node = YAML::Load(str);

    REQUIRE(node["transitions"].size() == 2);
}

Am I overlooking something?

Yep! This works fine, something else in sceneLoader.cpp must be assuming something that breaks this and throw an exception then.

Looking over this now!


Add logging for matrix benchamrks.

Log the benchmark values in a file for post-processing (e.g. check the distribution of the values)

Closed by https://github.com/365andreas/X-Mem/commit/5d442bfbc429bedf6957570714becfed7f65d756.


Reaction directionality

Is it possible to preserve reaction directionality when plotting metabolic models with escherplot!() using arrows! instead of lines!?

Hi there,
Yes, that is a good idea! I will implement that this weekend :)

I just added arrows to the plot. Could you test it out and let me know what you think (v0.5.1)? Note, the arrows are not drawn at the tips of the reaction edges, but slightly further down, I think this looks better (and was substantially easier to implement). If the updated package doesn't work, lmk, Makie just released a new version as well, so there is a small chance that things could go wrong...

Thanks! I'm new to COBREXA so it will take me a bit to figure out where reaction directionality is stored in the model JSON file. Then I'll close off this issue.

I forgot to update to (v0.5.0) at first but  can confirm the reaction arrows work and look great trailing behind the metabolite nodes. I noticed reaction directionality is preserved by default now but also had success manually specifying reversibility with reaction_directions. Thanks again!


Remove unused grpc_udp_server_write method

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can you run: tools/distrib/check_copyright.py --fix
to update the copyright notices, and append this PR?

ctiller: done


Add weight importance to file format

In many cases, it makes sense to say that an observation is more important than another one.
Right now, there is no obvious way to provide this information.
Would it be possible to add such feature?
It would look like an optional key word to add to each sample (for instance __weight__:2.5).
The effect would be to increase the loss (and the correction) for that sample.
What do you think?

Yes, it is possible to add, but I am not sure we intend to support this right now (option overload) unless many people request it.
Note that you could also do this right now by replicating examples in the training file corresponding to their weight (but it's a bit of a pain and waste of memory, but still, you can).

I am not sure it would be the same (2 times a loss is not the same than seeing the same observation 2 times, because the second time a first correction has been applied).
Few would request it if you don t explain in the examples how to concretely use it (as you did with train modes). For instance content recommendation may take the price into account (or just a log of the price), sentence similarity applied to query suggestion (I have tried and it works much more than graph approach FYI) would take the interaction into account (download of a content should count more than just opening a document, or may be the dwell time would make a nice weight), for classification the use of weight would be obvious, etc.
Really I can't think of any basic function that would be missing (compared to Vowpal for instance).
Would it be complex to add?

@pommedeterresautee Thanks for the suggestion. We'll set the default setting of normalizeText to false (in most cases it's just lower casing, see details in src/utils/normalize.cpp
We'll add the weighted example case in our list of features to implement.

@ledw I was thinking that to improve results of sentence similarity, it may help to provide myself some specific negative examples (instead of random ones) and realised that a negative weight may do the trick... May be you want to add that usecase in some way in your examples when weighting will be implemented.

I just wanted to say that I had the same request, and was grateful to see this thoughtful discussion and execution.

@ncammarata may I ask you what is your use case? It may give everybody ideas on how to leverage starspace representation

@pommedeterresautee I'm interested in using it for a network scenario. I want to be able to construct a graph of the system I'm working with, which in my case is connected operating systems with shared files, and arbitrarily make predictions within it. So I can say that X person looked at file Y for Z minutes, so what other files should I prioritize for their viewing.
Except I can basically choose any part of my graph to make these kinds of predictions. I haven't actually done any work with StarSpace on this yet, but that's my desired outcome, and I was happy to see weighted connections looking possible.
Does that make sense?

Closing the issue as weight importance is added.


Added few sample metrics

added few sample metrics

@Ajpantuso updated and addressed review comments

/lgtm

/approve


multiple executions of page constructor

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[X] 3.x
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Page constructor is being run every time the page is pushed on to the nav controller.
Expected behavior:
Page constructor would only run the first time the page is constructed.
Steps to reproduce:
https://github.com/mgkimsal/ionicpagedemo
There's a "Page1" and "Page2"
Page1 has a button which calls 'goto'.  When clicked, it pushes Page2 on to the navCtrl.
Page2 has a constructor with a console.log().
From Page1, click the button, go to Page2.
See that you've hit the constructor.
Click the 'back' nav button.
From Page1, click the button again, so to Page 2 again.
See that you've hit the constructor again.
Related code:
Page1
goto()
{
this.navCtrl.push(Page2Page);
}

Page2
  constructor(public navCtrl: NavController, public navParams: NavParams) {
    console.log('in constructor');
  }

Other information:
Other "bug" reports on this focused on "setRoot()" issues.  I'm not using setRoot() anywhere (but maybe NavController is?)
I was under the impression that a page controller would only be initialized/constructed one time, not every time the page was pushed on to the nav stack.  If this is a wrong impression, and this is intended behaviour, it should probably be documented (tell me where and I'll do a change/pullrequest).
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
global packages:

    @ionic/cli-utils : 1.1.2
    Ionic CLI        : 3.1.2

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-ionic-angular : 1.1.2
    Ionic Framework                 : ionic-angular 3.2.1

System:

    Node       : v7.2.1
    OS         : macOS Sierra
    Xcode      : Xcode 8.2.1 Build version 8C1002
    ios-deploy : 1.9.1
    ios-sim    : 5.0.13

Hello, thanks for using Ionic! This is actually expected behavior. The navController will destroy the class instance of a page when it is popped. If not we could end up with a memory leak caused by multiple instances of the same page being kept in memory. This is the repo you can submit docs issues/PR's too. I would probably open an issue first so that this can be discussed.

Thank you for your response.
I'd meant to reference the other ticket on this
https://github.com/driftyco/ionic/issues/5960
Your responses there seemed to indicate this wasn't expected behaviour:
"since your doing a setRoot on the home page it is going to be reconstructed when you navigate to it because its being reset as the root and has to be reinitialized."
and
"so if your on your root page and you navigate to a different view, then to get back to your root page all you have to do is use nav.pop(rootPage). There is no need to set it as root again before you navigate back to it. This will keep it from reconstructing"
Indeed, the 'expected' behaviour may have been explicitly changed (I don't particularly remember seeing this as a problem with ionic 1, but it's been a long time since I was using that).

@mgkimsal hello! Did you find a fix the problem? I'm having the same issue here. Where I make http calls in the constructor. And it's being fired every time the page is navigated to.


Using SHIFT+PAGE functions on the Sim in generative mode crashes the app

When trying to add a random sequence or euclidean sequence, pressing Shift+Alt (Shift+Page) the sim crashes with an error message "Segmentation fault: 11"

Not on mine
Last OS v0.1.39

Yep, that was a bug. Fixed with next release (or in master branch).

issue confirmed as resolved, thanks!


Official Website Not Accessible

Hello, i was trying to asscess the official site www.laravelpanel.com but seems its not working. Can you please look in to it as most of your tutorial are on website

Site is up thank you for report it


Orzech

I decided to merge orzech to the master branch since app isn't complete and don't have 1.0 version. So I am sharing my code with you, next time please push to the master, not to the orzech. I don't want my branch to become informal master.
After 1.0 version when app become usable, we will create release branch or something like that


EditBadges close popup windows when adding link

Describe the bug
EditBadges close popup windows when adding link
To Reproduce
Steps to reproduce the behavior:

setup a popup with a EditBadges filter within
Add a table link from the EditBadges filter
The popup closes

Expected behavior
The popup should not close.

I also see this. It's annoying.

ah yes that was me being lazy (by reloading the page we don't have to insert the new label in DOM). Will fix with a proper insert

Fixed in badges version 0.1.13

Thank you very much, really quickly fixed!

Yay… Works great!
Badges is such an excellent plugin.
Thanks.
From: Tom Nielsen @.>
Sent: Monday, September 25, 2023 12:20 AM
To: saltcorn/saltcorn @.>
Cc: Geoff @.>; Comment @.>
Subject: Re: [saltcorn/saltcorn] EditBadges close popup windows when adding link (Issue #2002)
ah yes that was me being lazy (by reloading the page we don't have to insert the new label in DOM). Will fix with a proper insert
—
Reply to this email directly, view it on GitHub https://github.com/saltcorn/saltcorn/issues/2002#issuecomment-1732589781 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGCAYLMSNIYTMGNFBI3OZ53X4BCCFANCNFSM6AAAAAA5EMZB6Y .
You are receiving this because you commented.  https://github.com/notifications/beacon/AGCAYLLX4FETU4GD7KE35NLX4BCCFA5CNFSM6AAAAAA5EMZB62WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTHIU4NK.gif Message ID: @.*** @.***> >


[BUG] 03_classification.ipynb - Assignment issue between DataFrame to Array

I am not a python expert but I realized there is probably a compatibility issue with following code:
%matplotlib inline
import matplotlib as mpl
import matplotlib.pyplot as plt

some_digit = X[0]
some_digit_image = some_digit.reshape(28, 28)
plt.imshow(some_digit_image, cmap=mpl.cm.binary)
plt.axis("off")

save_fig("some_digit_plot")
plt.show()

The assignment some_digit = X[0] throws an error.
I managed to fix this like this:
%matplotlib inline
import matplotlib as mpl
import matplotlib.pyplot as plt

some_digit = X[0:1]
some_digit_arr = np.asarray(some_digit)
some_digit_image = some_digit_arr.reshape(28, 28)

plt.imshow(some_digit_image, cmap="binary")
plt.axis("off")

save_fig("some_digit_plot")
plt.show()

Thank you.

Hi @selcukbeyhan ,
Thanks for your feedback. The problem comes from the fact that fetch_openml() started returning Pandas DataFrames instead of NumPy arrays since Scikit-Learn 0.24. This messes up a lot of code examples. Luckily, there's a trivial fix: just set as_frame=False when calling fetch_openml(), and everything will run smoothly:
mnist = fetch_openml('mnist_784', version=1, as_frame=True)

Hope this helps!


FR: Insert Citation

FR from user through Discord
/Cite Zotero Reference
(or /Insert Citation, or similar)

It is quite simple actually, it would look like this: [@wilsonImplemented5thgenerationAdapter2021]

Implemented in v2.


Adds process for new repo creation to meta CONTRIBUTING.md, plus other cleanup

This PR:

adds a new section on the process for repo creation (for #41 )
removes the "Community Guidelines" section, as it is redundant to CODE_OF_CONDUCT.md
removes the generic details on code contribution with a link to our main CONTRIBUTING.md in the amphtml repo

Thanks!


Document NG_BUILD_PROFILING

I feel like I've discovered some great secret, and I'd like other to know about it.
Environment var NG_BUILD_PROFILING=1 tells you where the precious seconds of your life are going.

The main reason why this is not documented AIO because it’s not part of the API and It’s functionality can be changed and/or removed without warning.
In v12 this will only create CPU profiles.
Maybe we should add this somewhere in https://github.com/angular/angular-cli/blob/master/docs/design/build-system.md

Well, if not build profiling a feature offered by Angular CLI, I suppose I have a feature request :)

Actually I was wrong in v12, it doesn’t generate CPU profiles but rather Chrome profile file which includes timings of plugins execution.

We're having compilation time issues (100sec rebuilds when serving) and figured we'd try NG_BUILD_PROFILING=1 after seeing it mentioned in another ticket. On my 16gb 2019 MBP it never seems to complete (been running for 1hr so far) with nothing inside the chrome-profiler file:
$ NG_BUILD_PROFILING=1 node --max_old_space_size=8192 ./node_modules/.bin/ng build

Not sure if there's a middle solution or a way to hook into it to see what it's doing?


Remove game-01

This was a runner, like game-06. But it predated SimpleSpawner, so used animations to make the obstacles fly past. game-06 is a more exemplary example of the same concept, using newer plugin features.
Supersedes:

#13

Farewell, feline friend 🫡


APNS Cert fails validation on Go 1.10

In Go 1.10 returns: certificate specifies an incompatible key usage: asn1.ObjectIdentifier{1, 3, 6, 1, 5, 5, 7, 3, 1} for APNS cert.
This is keeping us from upgrading our systems to use Go 1.10.

What version of Go are you using (go version)?
go version go1.10 darwin/amd64
What operating system (GOOS) are you using (go env) and what version?
OSX, Apline
What did you do? (steps to reproduce or a code sample is helpful)
Attempt to do push notification using a valid cert.
What did you expect to see?
The push to go through.
What did you see instead?
An error.

A proposed fix is in: https://github.com/RobotsAndPencils/buford/pull/92

Any comments on this?  I hate to fork this repo to fix this issue.

thanks for the fix.


[Suggestion] organize shops into folders/directories

Option to shops work inside directories. Inside the "shops" folder, we could create new folders inside it and organize our shops.
for example, container/plugins/BossShopPro/shops/stores/food.yml

Would be great for large servers that have lots of shops!

Already implemented by putting SearchSubfoldersForShops: true in the config

Already implemented by putting SearchSubfoldersForShops: true in the config

@Sentropic thanks so much for this <3


Feat: Improving UI of feedback page

Describe the feature
the feedback page is very simple and is also very different from the theme which is making it less attractive. I can enhance the UI of feedback form to make it look attractive and contrast it with the theme provided
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Suchitra-Sahoo Please assign me this issue. As the collaborator has not done yet


Update Response Time Error with Self Hosted Runner

Describe the bug
I am encountering an issue when running the response time action on a self-hosted runner. The action starts but the logs report an issue with the git command not being found. The response times are not updated in the files or website.
To Reproduce
Steps to reproduce the behavior:

Create a new Upptime page
Create a self-hosted runner and attach it to repository
Edit the configuration file to use a self-hosted runner
Check the logs for the response time action

Expected behavior
I expect these issues not to appear and the action to update the response times in the files.
Console Log
This same set is repeated for each site checked.
Checking https://openbank.api.banksyd.com.au/cds-au/v1/banking/products
Current status undefined up 2021-08-01T08:22:23.000Z
Result from test 200 0.956257
/bin/sh: git: command not found
/bin/sh: git: command not found
/bin/sh: git: command not found
/bin/sh: git: command not found
/bin/sh: git: command not found
Status is the same up up

Desktop (please complete the following information):

The self hosted runner is a Oracle ARM VM.Standard.E2.1.Micro running Oracle-Linux-8.4-2021.07.27-0

Probably linked to your own runner, I had issue on mine where the git version was too old for example.


What is the reason of using a sync version of writeFile for pdf save?

https://github.com/GoogleChrome/puppeteer/blob/9b0a06216e5b364aecaef6931d7844a91bebe46e/lib/Page.js#L588
Why wasn't async version of writeFile used for writing the pdf?

I presume there isn't a strong reason.. :)
If you wanted to change it, I'm sure the eng team would appreciate a pull request.

If you do use the async version, it is still going to be awaited for processing. So is there any functional difference between the two at that point from the point of view of code operation?

@Garbee reading lib/fs and this post it seems like the big difference is that the node event loop is blocked with sync, so any events coming from the chrome child_process (for example) would not be recognized until after.
And there's this issue suggesting that sync will always have inferior performance characteristics.
i can't think of a reason we NEED sync here, so might as well do the right thing.

big difference is that the node event loop is blocked with sync

SGTM then.

Ah. there's already a PR for this: #879


Why healthy_panic_threshold configured in OutlierDetection

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
HealthyPanicThreshold configured in OutlierDetection
health_panic_threshold is a configuration to set CommonLbConfig for cluster.  Default to 50.
As the concept in envoy: The panic threshold is used to avoid a situation in which host failures cascade throughout the cluster as load increases.
But in Istio, the configuration is in DestinationRule, minHealthPercent in OutlierDetection.
The Istio code is:

Can we extract a common configuration
Cause of, we add  configuration in DestinationRule to add active health check. If we want to make the active health check, and set healthyPanicThreshold to 0.  At the same time,  it will influence the OutlierDetection minHealthPercent.
I want know，why we cannot extract a common configuration to config the health_panic_threadThreshold in CommonLbConfig.

[x] Networking
[x] Policies and Telemetry
[x] User Experience

It was originally added so we could mimic behavior with Envoy's outlier detection and other library-based circuit breaker implementation.
So under the covers this gets set on Envoy's CommonLBConfig as you pointed out. Are you asking to make the API not be outlier-detection specific?

It was originally added so we could mimic behavior with Envoy's outlier detection and other library-based circuit breaker implementation.
So under the covers this gets set on Envoy's CommonLBConfig as you pointed out. Are you asking to make the API not be outlier-detection specific?

Yes. Because we add active_health_check configuration such as

To config Envoy's core.HealthCheck.
But because the Envoy's CommonLBConfig is only one config. I think Istio should extract the common config for panic_threshold.
If not, unless we modify the code in Istio, some configuration will influence each other between outlier-detection configuration and active_health_check.

Sorry. I may be not following you fully - How does it matter where it lives in Istio API section, when it is finally applied CommonLBConfig in Envoy?

Can you clarify the issue?

I don't see why we would change the destination rule api given we don't actually support active health check?

I don't see why we would change the destination rule api given we don't actually support active health check?

Thank you for your apply. I agree it.
But in our scenes, if we change the destination rule api, we can make it easier to expand the active health. Also it will also correspond with envoy's config.

But in our scenes, if we change the destination rule api, we can make it easier to expand the active health. Also it will also correspond with envoy's config.

I don't think we have any desire to change the API to align with a hypothetical feature which we do not support, nor is it a goal for our APIs to closely represent Envoy.

Closing as we do not intended to mirror Envoy's api. you may want to follow https://github.com/istio/istio/issues/10537


import error

OpenEXR.so: undefined symbol: _ZNK3Imf14TypedAttributeINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEEE12writeValueToERNS_7OStreamEi

Hey! Thanks for reporting this issue. This seems to be an issue with a dependency of this project, OpenEXR. I'd suggest you open the same issue there. I'll close the ticket here for now.


server.close() does not work??

describe.only('Websockets store handlers', () => {
  let store;

  beforeEach(() => {
    store = createStore(ws,
      applyMiddleware(
        thunk,
        promise,
      )
    );
  });

  it('connects the websocket', () => {
    const server = new Server('ws://qorus.example.com/log/main');

    store.subscribe(() => {
      const { data } = store.getState();

      expect(data).to.eql({
        'log/main': {
          loading: true,
          connected: false,
          error: false,
          paused: false,
        },
      });
    });

    store.dispatch(actions.connect('log/main'));

    server.close();
  });

  it('disconnects the websocket', () => {
    const server = new Server('ws://qorus.example.com/log/main');

    store.dispatch(actions.connect('log/main'));
    store.subscribe(() => {
      const { data } = store.getState();

      expect(data).to.eql({
        'log/main': {
          loading: false,
          connected: false,
          error: false,
          paused: false,
        },
      });
    });
    store.dispatch(actions.disconnect('log/main'));

    server.close();
  });
});

Websockets store handlers disconnects the websocket:
Error: A mock server is already listening on this url

Sorry, It's actually server.stop(), but that doesn't really matter since that one does not work either :)

@Foxhoundn Thanks for finding this. Let me dig into it and see what is happening

Because I don't have access to your store/actions its going to be hard for me to debug. Are you able to create a simpler example of this bug without hitting a store?

I'm having a similar "already listening" problem, and I tried
new Server('socketioserverurl', {
     forceNew: true,
     reconnection: false
})

and first typescript said those weren't valid keys, and when I @ts-ignore that error, I still have the same 'already listening' problem.

I'm also getting this error when I attempt multiple vitest cases, both concurrent and non-concurrent

In my case it was a sign of a bug in my code/test which was causing it not to reach the close call


Add lint_python step to test_runner to find undefined names

https://flake8.pycqa.org/en/latest/user/error-codes.html
On the flake8 test selection, this PR does not focus on "style violations" (the majority of flake8 error codes that psf/black can autocorrect).  Instead these tests focus on runtime safety and correctness:

E9 tests are about Python syntax errors usually raised because flake8 can not build an Abstract Syntax Tree (AST).  Often these issues are a sign of unused code or code that has not been ported to Python 3.  These would be compile-time errors in a compiled language but in a dynamic language like Python they result in the script halting/crashing on the user.
F63 tests are usually about the confusion between identity and equality in Python.  Use ==/!= to compare str, bytes, and int literals is the classic case.  These are areas where a == b is True but a is b is False (or vice versa).  Python >= 3.8 will raise SyntaxWarnings on these instances.
F7 tests logic errors and syntax errors in type hints
F82 tests are almost always undefined names which are usually a sign of a typo, missing imports, or code that has not been ported to Python 3.  These also would be compile-time errors in a compiled language but in Python a NameError is raised which will halt/crash the script on the user.

112 undefined names
https://github.com/Netflix/metaflow/pull/85/checks?check_run_id=367347570#step:7:105

Thanks for the PR. We will focus on lint checks after our internal and external code bases have converged. We will look into the violations then.


Creating Daily and Hourly weather entities

I'm having some trouble getting a daily and hourly sensor setup and someone suggested that it may not be possible, but I thought I would check. Below is the Pirate Weather configuration that I have in my configuration.yaml. Is this misconfigured?
weather:
  - platform: pirateweather
    api_key: !secret pirateweatherapi
    latitude: !secret homelatitude
    longitude: !secret homelongitude
    mode: "hourly"
    name: pirateweatherapihourly
weather:
  - platform: pirateweather
    api_key: !secret pirateweatherapi
    latitude: !secret homelatitude
    longitude: !secret homelongitude
    mode: "daily"
    name: pirateweatherapidaily

Got some help elsewhere. Fixed configuration.yaml
weather:
  - platform: pirateweather
    api_key: !secret pirateweatherapi
    latitude: !secret homelatitude
    longitude: !secret homelongitude
    mode: "hourly"
    name: pirateweatherapihourly
  - platform: pirateweather
    api_key: !secret pirateweatherapi
    latitude: !secret homelatitude
    longitude: !secret homelongitude
    mode: "daily"
    name: pirateweatherapidaily


Use standard Airbrake configuration

This is the default GOV.UK Airbrake config now, which reads from ENV instead of relying on overwriting files. It's taken from
https://github.com/alphagov/govuk-rails-app-template
It's safe to deploy because until the corresponding change has been made in alphagov-deployment, this file still gets overwritten.

:+1:

@jamiecobbett If you like it can you merge it? :)


Metrics Subset

How would I go about filtering for just a subset of metrics?  I use Prometheus for short term metrics and Splunk for longer term trending.  However, I only want to ingest a small subset of the metrics.  Is this currently possible?
Thanks!

closing. thank you.


'ctags' is not recognized as an internal or external command...

I'm trying to install ctags on Sublime Text 3 (is it supported?) but this error above. Here's what I've done so  far, as the Readme suggested:

Installed ctags from package control
Downloaded ctags.zip file from the link provided. Unzipped it and copied ctags.exe to C:\Program Files\Sublime Text 3\
In sublime text, right clicked the name of my project and selected "Ctags: Rebuild Tags" which gives me the error. I've read up on the issues that were open previously but their solutions haven't worked, also they were using ST2

Can you tell me if I'm missing something?

If you copied ctags.exe to C:\Program Files\Sublime Text 3, make sure C:\Program Files\Sublime Text 3\ is in your path.  You can determine this by typing 'set path' in cmd.

I was having trouble understanding what it means when the manual said

Extract ctags.exe from the downloaded zip to C:\Program Files\Sublime Text 2 or any folder within your PATH so that Sublime Text can run it.

which I understand now. If anyone else is facing the same issue, here's what I did. Go to Preferences > Package Settings > CTags > Settings - User
This should open a blank file. Paste the following code and specify the complete location of your ctags.exe file in the command name of this object.
{
	"command": "C:/Program Files/Sublime Text 3/ctags.exe"
}


Segmentation fault in compiler when for-in loop has continue in an else branch.

The following reduced example results in a seg fault.
actor Main
  new create(env: Env) =>
    for n in X do
      if true then
        break
      else
        continue
      end
    end

class X
  fun has_next(): Bool => false
  fun next(): I32 => 0

pony -d
...
Writing ./test.o
Stack dump:

Running pass 'Function Pass Manager' on module 'test'.
Running pass 'Exception handling preparation' on function '@Main_tag_create_oo'
Segmentation fault

It seems to require both a break and continue in the branches of an if-else or a match, within a for-in.
Compiling the above example without the -d option succeeds, but there is a version that seg faults without -d. I'll try to get back to that version next.

Linux 64-bit
LLVM 3.6
0.2.1-1024-g0c88dd7 [release]

The version that fails without the -d option requires a match instead of if-else.
actor Main
  new create(env: Env) =>
    for t in X do
      match t
      | 0 => break
      else continue
      end
    end

class X
  fun ref has_next(): Bool => false
  fun ref next(): I32 => 0

pony
...
Stack dump:

Running pass 'Simplify the CFG' on function '@Main_tag_create_oo'
Segmentation fault

Seems the break doesn't contribute to the issue after all, and substituting None for continue fixes the problem, so I guess that's the culprit.

I found the problem and this is my fault, sorry for that. The problem is that in cbc6b11, the lifetime informations are sometimes inserted at the wrong place, resulting in invalid LLVM IR.


Change stackwalking to always use unadjusted IP

Handling of hardware exceptions had a hack to add +1 to the actual instruction IP. Windows x64 unwinder
is disassembling instructions at the IP passed in to detect method epilogs. If the bytes at IP + 1
happened to match the epilog pattern, the unwind is done as if we were in the middle of the epilog that
lead to spectacular crash.
This change is moving this adjustment to be done later for EH related things only, and not interfere
with stackwalking.
Fixes #2535

@janvorli Could you please take a look? (I will have to do a lot of testing on it to verify that it is compatible with ProjectN.)

Checked in via TFS bridge


Ignoring YAML front matter

I am using this extension with Pandoc (markdown) files and hoping to have the YAML front matter ignored. But I can't seem to get this to happen using setting cSpell.ignoreRegExpList:
"cSpell.ignoreRegExpList": [
        "/-{3}(.|\\n)*-{3}/gim"
    ]

Am I missing something obvious here with how JS regular expressions should be written in this case?

Spoke too soon. I fixed this once I got a similar expressions working for comments. I've also added an regexp for citations in Pandoc.
"cSpell.ignoreRegExpList": [
        "/-{3}[\\s\\S]*?-{3}/gm",
        "/@\\w+/",
        "/<!--[\\s\\S]*?-->/gm"
    ]


Add metadata to various packages

This PR adds metadata to several packages. The plan is to add this to all packages so xbps can display this to the end user.
Currently, the following packages have metadata applied to them:

managarm-kernel,
managarm-system,
mlibc-headers,
mlibc,
binutils,
boost,
core-files,
base,
base-devel,
weston-desktop,
zlib,
ca-certificates,
sl,
dejavu,
xorg-font-util,
atk,
d0-blind-id,
fribidi,
libtsm,
kmscon,
bzip2,
gzip,
libarchive,
lz4,
tar,
xz-utils,
zstd.

As data sources for the metadata (descriptions, websites and licensing), I've aggregated information from https://packages.gentoo.org, https://linuxfromscratch.org (both LFS and BLFS) and the information provided by aptitude show on debian sid.

As @avdgrinten approved this on Discord, I’m going to merge this.


VS CODE stops frequently

I don't know why It's happening but it's everytime I use the whole program stops and I need to restart the computer to make it work again...
It happens at least one time everytime I use...

/needsMoreInfo


The sample number is insufficient. Can we repeat the sample

The above is my error information, my sample number is only 11, there are 124 species.  May I use repeated sampling when calculating?  thank you

Could you provide a detailed description of your question in an email?

I have 11 samples, 124 species.  An error is reported when running with DKI.py. The error message is as follows:
Traceback (most recent call last) :
File "DKI. py", line 165, in
loss_train, qtst, qtrn = train_reptile (max_epochs, mb, LR, ztrn, ptrn, ztst, ptst, zval, pval, zall, pall)
File "DKI. py", line 75, in train_reptile
batch_p, batch_q, batch_t = get_batch(ztrn, ptrn, mb)
File "DKI.Py", line 24, in get_batch
s = torch. from_numpy (np. random. choice(np. arange(ptrn.size(dim=0), dtype=np.int64), mb_size, replace=False))
File "mtrand-pyx", line 965, in numpy. random.mtrand. RandomState. choice ValueError: Cannot take a larger sample than population when 'replace=False’.
Based on this error message, can I change "False" to "True" in "s = torch.from_numpy(np.random.choice(np.arange(ptrn.size(dim=0), dtype=np.int64), mb_size, replace=False))" in the DKI.py file?
Thank you for your answer, good luck！
Zesheng Yang
将这句话翻译成日语。
如何修改代码以适应不同的布尔值？
除了修改布尔值，还有哪些可能的解决方案来解决这个报错？
| |
杨泽胜
|
|
@.***
|
---- Replied Message ----
| From | Xu-Wen @.> |
| Date | 11/24/2024 05:35 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [spxuw/DKI] The sample number is insufficient. Can we repeat the sample (Issue #11) |
Could you provide a detailed description of your question in an email?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Fix #6927 1:1 NAT validate address family

Ensure that all the manually-entered addresses come from the same
address family - i.e. they are all either IPv4 or IPv6 addresses.

Note: I enhanced the return values from validateipaddr() so the caller can know if the validated address is IPv4 or IPv6 (or an alias if that check was requested). That makes it a better replacement for is_ipaddr().
Without doing that, this code would have called is_ipaddr() in order to know if the address was IPv4 or IPv6, and also called validateipaddr() in order to get the $input_errors array stuff done (for the case of an invalid address). That looked rather silly when I simply did minimal integration of the validate_ipaddr() stuff into the code I had.

Another note: The "magic numbers" returned by is_ipaddr() and validateipaddr() could be defined as named constants - perhaps using the IP address types constants that were recently added to globals.inc
and extending those to include "ALIAS".
That re-factoring needs a bit of thought. And then go through the system to change everything that makes special use of the "4" and "6" and change it to use the defined constants.

@phil-davis I like this solution and it worked fine on my tests. I would like to ask you only to split it in 2 commits just to keep the history sane. Could you do it please?

I'll extend the address family defines I recently added to globasls.inc to include ALIAS once this is committed.

validateipaddr() changes are in PR #3298

1:1 NAT validation is in #3299


Mysterious acceleration of auto-restarting on mobile

When I visit https://www.talater.com/annyang/ using my mobile devices, I see that it auto-restarts every 5 or 6 seconds with a repeating dinging notification tone each time.
5 or 6 seconds is OK and acceptable for me.
But when I view my own web application on another browser tab on the same device, I see that annyang auto-restarts almost every 1 or 2 seconds. That is a HUGE decrease in usability.
I have tested it on an android tablet and an android phone; result=no perceivable difference.
I have tried setting continuous mode on/true and off/false; result=no perceivable difference.
I remember someone else in another issue of annyang reported that he had this "2 second" restarting interval as well, which means I’m not the only one getting such a result.
My question is, why is this happening? What could be the reason of this undesired rapidness? I'm using jQuery along with a number of other JS libraries. Could any of them be the reason?
My application is on https://speakwithme.turkcekon.us/1
Open it on a mobile device and you will have to watch a short video and use the buttons to advance to where annyang starts.
When annyang starts, please stay silent and observe the auto-restarting.
I'm still working on its stability, so please refresh the page if anything freezes.
I'm hoping to find an answer from which everyone can benefit.
Any help is much appreciated.

Just done more tests. It looks like, not the starting but the ending of a listening period becomes sooner than normal. That means something is causing annyang to stop earlier than expected.
The code below is not a real solution but instead of the built-in auto-restart I have set a manual restart function to fire every 5 seconds like this (Warning: This code is for mobile only),
annyang.start({ autoRestart: false });
var resetAnnyang = setInterval(annyangResetF, 5000);
function annyangResetF()
{
  setTimeout(function(){ annyang.abort(); }, 1000);
  setTimeout(function(){ annyang.start({ autoRestart: false }); }, 2000);
  console.log("annyang is going to be off for 1 second in 1 second");
}

You can use clearInterval(resetAnnyang); where you want it to stop ticking.
It works as a delay adjustment but the average listening time of annyang is still 2 seconds, which is too short.
It would be really great if I knew the cause and could get it to last for 4 seconds at least.

Further tests showed that the shortness of the listening time also depends on the device. In my case on an ASUS tablet it is not as bad as it is on a SONY phone, but still shorter than usual.
It would be nice to read other reports about experiences with annyang language setting and device type-model-brand.
By the way, here is my web-app for learning Turkkishi (aka Turkish language), https://turkcekon.us
In this web-app the language code set to annyang is "tr". You can compare the listening times to the default English on annyang's homepage or elsewhere annyang is used.


quickgui: 1.2.8 -> 1.2.9

Description of changes
I am one of the upstream developers of Quickgui. This pull request updates Quickgui to 1.2.9 and changes the package to build from source rather than dump a .deb.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

quickgui
quickgui.debug
quickgui.pubcache

Add a :+1: reaction to pull requests you find important.

Anything i can help with?
There is a merge conflict and also a newer version out by now. Happy to test or whatever is needed to get this on the way

Sorry for taking so long to get to this. The merge conflict is because gnome.zenity has been renamed to zenity since this was forked so this branch needs to be rebased to current master and have these changes reapplied.

Also, if at all possible, it would be nice to update this to 1.2.10 if possible, otherwise we will update from old version to less old version, which is suboptimal.

Updated to 1.2.10 as requested by @ToasterUwU

Result of nixpkgs-review pr 323381 run on x86_64-linux 1

3 packages built:

quickgui
quickgui.debug
quickgui.pubcache

Builds fine, starts fine, downloads fine, runs the VM just fine.
LGTM

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1878

@heyimnova Since you are the package maintaner in this case, take a look at: https://github.com/nixos/nixpkgs-merge-bot
If this is actually up and running and works like intended, you can merge this PR without external help

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1893

It seems env -i ldd is complaining about these two files:
results/quickgui/app/lib/libwindow_size_plugin.so
	libflutter_linux_gtk.so => not found
results/quickgui/app/lib/liburl_launcher_linux_plugin.so
	libflutter_linux_gtk.so => not found

env -i ldd

All the required files are present when I run nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD" and the resulting executable runs correctly.

@flexiondotorg Please force push every change in a single commit again with the proper formatting of the description ( "quickgui: 1.2.8 -> 1.2.10" )

Result of nixpkgs-review pr 323381 run on x86_64-linux 1

3 packages built:

quickgui
quickgui.debug
quickgui.pubcache


How to implement Input CODEX data for 'NCEM

I am appreciated by your excellent work. I plan to use 'NCEM' package with my CODEX image data. Yet, I failed to load my data referring the tutorial code. I find there are a lot of prefix like 'zhang','Hartmann' to build a dataloader. I don't know how to choose them. May I request for a help?

Hi @Luosanj ,
links to: https://github.com/theislab/ncem_tutorials/issues/1#issuecomment-1165669767
answering here to keep it better traceable with respect to issues.
May I ask which ncem version you have installed. We only added the cutsom data loader recently, so you might need to update ncem to use it.
Additionally the Codex data we are using in the manuscrip is taken from this manuscript:
CODEX - cancer
Schürch, C. M. et al. Coordinated Cellular Neighborhoods Orchestrate Antitumoral Immunity at the Colorectal Cancer Invasive Front. Cell 183, 838 (2020).
Data can be downloaded from https://data.mendeley.com/datasets/mpjzbtfgfr/1 To run NCEM download the CRC_clusters_neighborhoods_markers.csv and store it to a data directory of your choice.

Hi @AnnaChristina ,
Thanks for your response !  I have uninstall and  reinstall the package 'ncem', but the version didn't change which is 0.1.0. What is the right version supposed to be?

Cheers,

I find that in your tutorial the right version for ncem is 0.1.3. Yet I cannot find this version using command 'pip install'.

Thanks for answering Anna ! By the way may I ask is there a tutorial on how to run the 'Encoder Decoder with one layer GCN embedding' model mentioned in your paper ? Currently it seems only linear model can be run. Thanks !

@AnnaChristina

you can also use the non-linear ncem models by switching the model type to ed_ncem or cvae_ncem and the respective gnn layer specified. For simplicity, we currently do not provide a tutorial for all the different model set-ups but it follows the same logic and https://github.com/theislab/ncem_benchmarks/tree/main/scripts shows further examples how the different models can be run and hyperparameter suggestions for the different pre-selected datasets.

closing the issue due to inactivity. Please feel free to open the issue again, in case of further questions!


Remove Layer Selection functionality from the napari-imagej menu

napari-imagej provides buttons in its button menu for transferring data to and from an ImageJ2 GUI. These buttons can perform one of two actions, where the chosen action is defined via the choose_active_layer setting:

Choose the active layer in napari and transfer it over to the ImageJ2 GUI, or the active window in the ImageJ2 GUI and transfer that data over to napari.
Provide the user with a dialog box, prompting the selection of a layer from a drop-down list of all existing layers. The selection is what is transferred between the applications.

@ctrueden said:

Not really specific to this PR, but... has anyone actually asked for the ability to explicitly select the layer, rather than having it always be the active one? If not, I'd favor removing this feature in favor of always using the active layer. And programmatically, still allowing the layer to be specified in code. Having this option makes testing more complex, and I'm not convinced anyone will want to change this setting.

Here are my thoughts:

I personally prefer using the active layer, over choosing.
I hesitate to remove functionality that already exists. I know that Git history would make it easy to recover that functionality if we decide later that it is beneficial, but I also tend to think "if it ain't broke, don't fix it". But, at the same time, simplicity is nice, and I think now is the time to remove it if we are going to.
I pestered @elevans to get his opinion, and he asked me what napari's convention is for selecting a layer. Being rather out of the loop, I tend to think that it is providing that dropdown, since that is what you'd be provided if you wrapped a function asking for a Layer to magicgui. Do you have any opinions on this @tlambert03?
Here's one situation where I might want to use the chosen layer: it might be easier to select a layer when my napari viewer contains a lot of layers (think dozens). Instead of scrolling through the list to find the one I want, it might be faster to bring up the dropdown and use keystrokes to quickly identify the layer I want. I don't think this is a particularly compelling use case, though...

Happy to hear thoughts, as my default position is to leave it in.

@ctrueden suggests:

small usability thing: I get confused about "choose active layer" because to me that sounds like a thing I should have checked when I want to choose which layer will be sent back and forth, whereas in practice it has to be unchecked for me to get a choice.

If we're going to keep this functionality, we can rename it. @ctrueden do you have naming suggestions?

There is enough disagreement with my position (that we should just always use the active layer, and not provide an option otherwise) that I'm going to close this. I did push a commit renaming choose_active_layer to use_active_layer, which I find less confusing.


FR: App Distribution should not accept test app with same build number

Feature proposal

Firebase Component: App Distribution

I started to use fastlane as a utility to automatize some processes during the app distribution, and in the project, we started to have a lot of versions with the same build_number, starting to be a real mess.
With some experience and study, now every beta release on Firebase has a different build number and in our development branch, everything is organized and separated as it should be since when we started.
I would like to suggest to Firebase team to refuse the beta app in App Distribution with the same build number, so other non-experient developers can start with the right foot, incrementing the build number.

This should help in #6148, depending on how it may be implemented


Option seekfunction (20167) has unknown or unsupported type

Two of the R notebooks have started failing:

R-weather.ipynb
R-nyc.ipynb

The error is when calling s3saveRDS:
aws.s3::s3saveRDS(x = nyc_data,
                  object = paste0(branch,"/nyc/","nyc_permits.R"),
                  bucket = repo_name,
                  region="",
                  use_https=useHTTPS)

Error in curl::handle_setopt(handle, .list = req$options): Option seekfunction (20167) has unknown or unsupported type.
Traceback:

1. s3saveRDS(x = df, bucket = "quickstart", object = paste0(branch,
 .     "/weather/", "data.R"), base_url = baseurl, region = "",
 .     use_https = FALSE)
2. put_object(file = tmp, bucket = bucket, object = object, ...)
3. s3HTTP(verb = "PUT", bucket = bucket, path = paste0("/", object),
 .     headers = headers, request_body = file, verbose = verbose,
 .     show_progress = show_progress, ...)
4. httr::PUT(url, H, body = httr::upload_file(request_body), query = query,
 .     show_progress, ...)
5. request_perform(req, hu$handle$handle)
6. curl::handle_setopt(handle, .list = req$options)

The error comes from curl here. I have no idea what it means or why it's happening now.
Can't reproduce it locally either. Different curl versions??

These notebooks were working fine in these test runs against main (the failures were other notebooks):

https://github.com/treeverse/lakeFS-samples/actions/runs/5802396382
https://github.com/treeverse/lakeFS-samples/actions/runs/5836600984

The notebooks also ran fine in the PR https://github.com/treeverse/lakeFS-samples/actions/runs/5866557042.
So nothing's been merged to main that would account for this failure.

Courtesy of @alexellis I tried out Actuated and got a shell onto the GH runner itself, and an SSH port forward to try out Jupyter
ssh -L 18888:localhost:8888 -p 35809 runner@<actuated host>

From here I ran Jupyter Notebook interactively and R-nyc.ipynb ran just fine

If the notebook runs ok, on both local and runner environments, then the attention turns to the test itself - papermill.
Here lies the manifestation of the problem (both local and GH runner):
docker exec $(docker ps -q --filter expose=8888) \
              papermill --cwd /home/jovyan/notebooks/ \
                        "/home/jovyan/notebooks/R-nyc.ipynb" \
                        "/home/jovyan/notebooks/papermill-out/R-nyc.ipynb"
Input Notebook:  /home/jovyan/notebooks/R-nyc.ipynb
Output Notebook: /home/jovyan/notebooks/papermill-out/R-nyc.ipynb
Working directory: /home/jovyan/notebooks/
Executing:   0%|          | 0/68 [00:00<?, ?cell/s]Executing notebook with kernel: ir
Executing:  41%|████      | 28/68 [00:01<00:01, 24.06cell/s]
Execution halted
Executing:  46%|████▌     | 31/68 [00:02<00:02, 14.95cell/s]
Traceback (most recent call last):
  File "/opt/conda/bin/papermill", line 8, in <module>
    sys.exit(papermill())
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/papermill/cli.py", line 250, in papermill
    execute_notebook(
  File "/opt/conda/lib/python3.10/site-packages/papermill/execute.py", line 128, in execute_notebook
    raise_for_execution_errors(nb, output_path)
  File "/opt/conda/lib/python3.10/site-packages/papermill/execute.py", line 232, in raise_for_execution_errors
    raise error
papermill.exceptions.PapermillExecutionError:
---------------------------------------------------------------------------
Exception encountered at "In [15]":
Error in curl::handle_setopt(handle, .list = req$options): Option seekfunction (20167) has unknown or unsupported type.
Traceback:

1. aws.s3::s3saveRDS(x = nyc_data, object = paste0(branch, "/nyc/",
 .     "nyc_permits.R"), bucket = repo_name, region = "", use_https = useHTTPS)
2. put_object(file = tmp, bucket = bucket, object = object, ...)
3. s3HTTP(verb = "PUT", bucket = bucket, path = paste0("/", object),
 .     headers = headers, request_body = file, verbose = verbose,
 .     show_progress = show_progress, ...)
4. httr::PUT(url, H, body = httr::upload_file(request_body), query = query,
 .     show_progress, ...)
5. request_perform(req, hu$handle$handle)
6. curl::handle_setopt(handle, .list = req$options)

Looking at the container build logs for executions of the action when it worked and when it then didn't I spotted one difference in the R packages:
r-httr-1.4.6               build    r42hc72bb7e_1
vs
r-httr-1.4.7               build    r42hc72bb7e_0
Looking at the release log for httr I noticed this commit which includes seekfunction as named in this error.
So, something smells fishy - let's hope it's not a red herring … 🐟

Logged an issue on the httr project https://github.com/r-lib/httr/issues/746

Pinning httr to 1.4.6 fixed the notebooks https://github.com/treeverse/lakeFS-samples/pull/136/files#diff-fbb41456a45d9370a30a5529ec28d267198a63f76888ffff2ad7fab4f86fbdc2R31

Glad actuated was of help to you @rmoff


Make install do not re-install correctly the package

We just realized that make install does not correctly re-install the package.
In particular, if one runs make mypy removes the current package from the environment (since it syncs requirements_ci.txt after running the install_tag, as explained in issue #48 ) but running make install afterwards does nothing since it checks for the existence of the make_cache/.install_tag file and it actually exists.
@deusebio, I think that make install should not be cached at all, only make dist should.

Yes, you are right. It is kinda annoying since pip-sync actually uninstalls the package itself. We can easily fix this by removing the "install_tag" too when we sync using pip-sync


TS-5076: Test and set the in_enabled_list by atomic function

(cherry picked from commit f84b4333d2743a8275f4974e32c733e989968a67)

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1241/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1136/ for details.


WIP: Add tzdata to container

Resolves #117.

Realizing that I don't think we actually need to set the timezone; just install tzdata.

Next up: once the build completes, test this locally using a different GitHub org so that we don't create hundreds of issues in our actual org.

Turns out we don't need to send a mass issue to test, just load the page, which is great.
✅ Verified that the page loads correctly on my local. Once build passes, should be good to merge this.


Flutter run works , but Start Debugging returns no device

My Problem

Hello , i start using flutter a month ago and i really loved it , the thing is that i still cant run it via Start Debugging, it only works for me with flutter run
My paths are correct and i have checked with a friend we have the same config and tried every solution here, but still doesn't work for ! i have this famous message :

Logs

Unable to locate a development device; please run 'flutter doctor' for information about installing additional components.

The output of my flutter doctor -v is :
[✓] Flutter (Channel beta, v0.3.2, on Linux, locale en_US.UTF-8)
    • Flutter version 0.3.2 at /home/ntic-oussama/Downloads/flutter
    • Framework revision 44b7e7d3f4 (7 months ago), 2018-04-20 01:02:44 -0700
    • Engine revision 09d05a3891
    • Dart version 2.0.0-dev.48.0.flutter-fe606f890b

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.2)
    • Android SDK at /home/ntic-oussama/Downloads/DownloadsOld/Sdk
    • Android NDK at /home/ntic-oussama/Downloads/DownloadsOld/Sdk/ndk-bundle
    • Platform android-28, build-tools 28.0.2
    • ANDROID_HOME = /home/ntic-oussama/Downloads/DownloadsOld/Sdk
    • Java binary at: /home/ntic-oussama/Downloads/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] Android Studio (version 3.1)
    • Android Studio at /home/ntic-oussama/Downloads/android-studio
    • Flutter plugin version 29.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] Connected devices (1 available)
    • Mi A2 • 4110969 • android-arm64 • Android 8.1.0 (API 27)

• No issues found!

Thanks for help :+1:

Thanks for responding , i did flutter devices and it recognizes my device Mi A2. , i've also checked the link but it didn't work for me, am gonna try to update my flutter and comme back.


Fixed issue 5428 by updating format_statevector and format_unitary

Summary
Fixes #5428
Details and comments
The call from result.py for postprocess to format the state vector and unitary matrix having arg of type ndarray and were no postprocessing to be done. Earlier it just return the statevector without any postprocessing. This PR added the postprocessing which handles the decimal argument.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@kdk  Where I have to add a test? Should I have to write in the test package of qiskit-terra or a separate file.

@kdk Sure will add a test as well. Was busy with quantum challenge.


Muhammad Khan

https://github.com/KhanMuhamamdT/python_koans.git

Looks like you intended to upload this to your own fork instead of the origin.


fix: recursive types on @graphql directive

Issue Reference(s):
Fixes #2525

@ouertani can you check if this is working?

This fixes the base use case thank you for fast response. For a bigger schema still facing the issue where the start sleep indefinitely.  I'm trying to find and formulate the bigger case.

/tip $50


When taking a picture on Android, the popUp Context will be overwritten with intermediateactivity

Due to the ConfigureLifecycleEvents method, when using a camera, the init will get called when a new Camera Activity is opened from the app.
This overwrites the context property of the static Popup class with IntermediateActivity.
When I try to open a popup after opening the camera page, the context and decorview will still be the one of the Intermediate Activity, and popups will not appear anymore.
Hopes this makes sense and can get resolved.

To fix this for me, I added this to the Init method for Android, but I ideally there should be support for MauiActivity as well.
if (context is not MauiAppCompatActivity)
{
    Debug.WriteLine("Popup has already been initialized.");
    return IsInitialized;
}

Fix in the latest version 1.0.8 https://www.nuget.org/packages/RGPopup.Maui/1.0.8
Thanks for reporting this issue. Please help to verify the fix when you are free!

Works for my application! Thanks for the quick response


Could not find network emulation

When trying to use tracerbench profile with the --network option I'm getting the following error:
(after calling tracerbench record-har --url=https://www.tracerbench.com --headless)

tracerbench profile ./tracerbench.har --network=3g
√ Reading cookies json
√ Preparing results folder
√ Validating & Cloning HAR file
× Recording the live trace
→ Error: Live Trace could not be captured. Error: Could not find network emulation "[object Object]"
Setting trace events
»   Error: Error: Error: Live Trace could not be captured. Error: Could not find network emulation "[object Object]"

Sometimes it keeps spinning. I find it a bit weird since I think conditions.network should probably be just a string so that the lookup at https://github.com/TracerBench/tracerbench/blob/d3d53204ea4c9fb6bb56896ba4a59a313fa8d5e1/packages/core/src/trace/utils.ts#L178 works ?

Thank you for bringing this to our attention. This should be resolved within https://github.com/TracerBench/tracerbench/commit/670308ab784dd0c03029c55c269f8614d62a0464 and available on the next patch release.