Atom startup broken, empty treeview, and also creates weird files

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

I did not x the above since I cannot reproduce the exact problem in safe mode. In safe mode the treeview is empty, but the weird files aren't opened.
Description
I open Atom, the treeview is now empty, and two non-existent files are opened that I never created.
One file is named ATOM_DISABLE_SHELLING_OUT_FOR_ENVIRONMENT=false, and appears as in my home directory. The other file is named atom, and appears as in usr/bin. Neither file actually exists. They are just created, yet not saved.
As far as any other state, as far as I can tell any app state is just forgotten by Atom. But I can't tell why. Is it related to the treeview getting reset? Is it because of the weird files getting opened? I don't know.
Steps to Reproduce
I have no way to describe how to reproduce the behavior because I simply opened atom yesterday, and it worked. And after opening it today it didn't work. The only difference between yesterday, and today is that I restarted my computer.
Expected behavior:
Atom opens with my projects in the treeview, and doesn't open weird files.
Actual behavior:
On starting Atom the Treeview doesn't open. When treeview is opened no projects are in it. After adding a project if I close Atom, and start it again the treeview is empty again.
If I add a project on restarting Atom the ~/.atom/storage/application.json gets rewritten removing all projects.
On starting Atom weird files ~/ATOM_DISABLE_SHELLING_OUT_FOR_ENVIRONMENT=false, and usr/bin/atom are opened. These files are not yet written to my hard drive.
Reproduces how often:
Every time starting Atom.
Versions

apm  2.5.0
atom 1.48.0
ubuntu 20.04 LTS

Additional Information

I tried deleting the .atom directory. After nuking .atom the problem persists. I thought of posting this problem in the forums, but this doesn't seem like something that can be simply fixed with advice. It's a very strange behavior which I had no hand in initiating.
To me it seems like atom is opening with atom, and ~/ATOM_DISABLE_SHELLING_OUT_FOR_ENVIRONMENT=false as files as if those were added in the command line. But I'm not running Atom from the command line. And that doesn't explain why the ~/.atom/storage/application.json file keeps getting overwritten.

Okay. I have an update. I uninstalled Atom. Then I deleted the ~/.atom, and `~/.config/Atom directories.
I then reinstalled Atom. When I started the newly installed Atom the problem was still there. Weird!
It's a snap in the Ubuntu repos.
While writing this comment I stopped, and uninstalled Atom again. Then I went to the website, downloaded the .deb file, installed that, and the problem appears to be resolved for me.
I'm not closing this issue yet because there still might be something to sort out. The solution probably shouldn't be to reinstall from a different source. I deleted all config files just in case, and that shouldn't be a solution either.
Does anyone one know who authors the snaps? This newly installed version is the same version as the one I had installed before. So the problem must be in the snap.

I can confirm this issue with the snap package. I have not tried yet the .deb version.

@Danfro Oh good. I was thinking I might be the only one. I wish I knew more about Atom internals. I'm dying to know if it's a conflict with something Atom does, or just bad packaging. There's been no response so far by anyone at snapcrafters.

For me, everything is fine if I just open atom from the terminal, however the snap generated .desktop file in /var/lib/snapd/desktop/applications/atom_atom.desktop has the line
Exec=env BAMF_DESKTOP_FILE_HINT=/var/lib/snapd/desktop/applications/atom_atom.desktop /snap/bin/atom ATOM_DISABLE_SHELLING_OUT_FOR_ENVIRONMENT=false /usr/bin/atom %F

which atom is apparently interpreting "as use /snap/bin/atom to open these two files".

Thanks for reaching out!
We've determined that you are currently using an unofficial build or distribution of Atom. Often these customized versions of Atom are modified versions of the Stable branch of Atom with mismatched versions of built-in components. These updated components are taken from the Beta channel or master branch and then injected into the Stable version and a new Atom package is generated. Because of the way Atom is constructed, using these mismatched components can cause mysterious and hard-to-diagnose problems. You can find out more about why we chose to not support unofficial distributions here.
You can find instructions for installing an official version of Atom in the Flight Manual. If you are still seeing this problem on an official build please file a new issue, thanks!

@Aerijo Yes, I have installed from the official distribution (.deb file), and don't have the same problems. As I wrote above, but I'm rewriting it here so it's easy for others to find.

I have the same issue on windows with the official build


Gateway - [WARN ] Unable to connect to host -> sip provider.(Verify your network status)

Some gateway registrations fail with [WARN ] Unable to connect to host -> sip provider.(Verify your network status)
I have the option to test multiple sip providers and for some reason, I'm getting many that fail to register.  I see the Unable to connect to host error.
The sip provider Linphone always registers, but (for example) localphone.com and mynetphone both fail.
Below is an example where you can see that localphone.com is failing, but linphone works.  You can see the first message received back from Linphone in the logs (I cut out the rest).
I am able to run a sip client on the server and register towards localphone, so this can not be a firewall issue.  I've tried with both UDP and TCP
[DEBUG] registry.Registry.register [gateway {"apiVersion":"v1beta1","kind":"Gateway","metadata":{"ref":"gwLinphone","name":"Linphone","createdOn":"2020-09-11T08:09:41.617Z","modifiedOn":"2020-09-11T08:09:41.617Z"},"spec":{"host":"sip.linphone.org","transport":"tcp","credentials":{"username":"myusername","secret":"****"}}}]
[DEBUG] registry.Registry.register [gateway {"apiVersion":"v1beta1","kind":"Gateway","metadata":{"ref":"gwLocalphone","name":"Localphone","createdOn":"2020-09-11T08:09:41.617Z","modifiedOn":"2020-09-11T08:09:41.617Z"},"spec":{"host":"localphone.com","transport":"tcp","credentials":{"username":"myusername","secret":"****"}}}]
[DEBUG] core.processor.RequestProcessor.process [route type UNKNOWN]
[DEBUG] core.processor.RequestProcessor.process [entity info null]
[DEBUG] core.processor.RequestProcessor.process [running handler for method `REGISTER`]
[DEBUG] core.processor.RegistryHandler.doProcess [via addr {"host":"<IP_ADDRESS>","port":5060}]
[WARN ] Unable to connect to host -> `localphone.com`.(Verify your network status)
[DEBUG] core.processor.RequestProcessor.process [route type UNKNOWN]
[DEBUG] core.processor.RequestProcessor.process [entity info null]
[DEBUG] core.processor.RequestProcessor.process [running handler for method `REGISTER`]
[DEBUG] core.processor.RegistryHandler.doProcess [via addr {"host":"<IP_ADDRESS>","port":5060}]
[DEBUG] SIP/2.0 401 Unauthorized
Via: SIP/2.0/TCP <IP_ADDRESS>:10168;rport=49175;branch=z9hG4bK-313837-4d54c8963ad7c3f6d1df6d7cfd24bb3e;received=<IP_ADDRESS>
From: <sip:myusername@sip.linphone.org>;tag=345308ab

What other debugging can I do to see why this is failing?  Thanks

Perhaps the best way to troubleshot this is using wireshark or sngrep.


adds tip to methods documentation about underscore and dollar prefixed namespace collisions with methods

This PR is in reference to https://github.com/vuejs/vue/issues/6312.
Please note that this PR also add .idea/ to gitignore.

@JustinBeaudry Btw, I'm leaning towards closing this in favor of the runtime error we're discussing in the other issue, possibly combined a recommended rule in eslint-plugin-vue. For less commonly encountered gotchas where we can provide a just-in-time warning, I prefer to leave it out of the docs to reduce cognitive overhead.

@chrisvfritz I'm torn on this. Whilst I do agree that preventing cognitive overload with caveats and gotchas is important, this one is particularly nasty and could have many emergent properties and propagations of effect. The API docs feels like it should be the source of truth for this, even if the runtime warns in dev, and even if the eslint plugin helps prevent the issue.

@JustinBeaudry I can see where you're coming from and it's possible I'll change my mind later, but at least for now, I think the added cognitive load is probably more likely to prevent people from absorbing the more vital information. We can place enough detail in warnings to help people understand why the rule exists and what they can do as an alternative.
One other side note though, referencing a comment in the issue:

I think it's unreasonable to expect people to have to respect naming strategies for methods

I had the same initial feeling coming to Vue from React. But over time, this "limitation" became one of my favorite features, because it leads to much more readable components.
For example, I've seen React components passed this.props.todos, containing an array from an API, then that array is normalized into a new array in this.state.todos, and finally filtered in a this.todos method. There are no name conflicts due to the scoping, but I'd argue the conceptual conflicts are worse.
This kind of conceptually sloppy code isn't possible in Vue. You're forced to use a different name for each prop, datum, computed property, and method - so you have to really think about what each thing actually is. The prop might be called todosApiResponse, the datum todos, then the method (or more appropriately, computed property in Vue) filteredTodos. Now the component makes more sense. It tells a clear story.
Similarly with conventions around _ and $, when I see an _, I know it's internal to Vue. When I see a $, I know it's a Vue instance property from outside the component - either internal, or from a mixin/plugin. In the future, Symbols can prevent code conflicts by allowing naming conflicts, but I'm not convinced that's even a good idea.

I can see where you're coming from and it's possible I'll change my mind later, but at least for now, I think the added cognitive load is probably more likely to prevent people from absorbing the more vital information. We can place enough detail in warnings to help people understand why the rule exists and what they can do as an alternative.

:bow:

I had the same initial feeling coming to Vue from React. But over time, this "limitation" became one of my favorite features, because it leads to much more readable components.

I agree.

Similarly with conventions around _ and $, when I see an _, I know it's internal to Vue. When I see a $, I know it's a Vue instance property from outside the component - either internal, or from a mixin/plugin. In the future, Symbols can prevent code conflicts by allowing naming conflicts, but I'm not convinced that's even a good idea.

I think a convention is fine and it makes sense.

In the future, Symbols can prevent code conflicts by allowing naming conflicts, but I'm not convinced that's even a good idea.

With Symbols I think the goal of it should be to scope Vue's internal methods (leaving an open namespace), but not to scope out top-level properties (props, methods, etc.).


nixos/grub: Remove > from submenu title, unbreak grub-reboot

Grub default or next boot entry can be set by GRUB_DEFAULT/grub-set-default/grub-reboot, and > is the separator between submenu and menuentry title1.
Using > in submenu or menuentry title will break this functionality.
After this change, any boot entry (include specialisation) can be boot only once using command like this:
grub-reboot 'NixOS - All configurations>NixOS - Configuration 532 (2024-01-30 - 24.05.20231225.e1fa12d)>NixOS - Configuration 532 - (nvidia - 1970-01-01 - nvidia-24.05.20231225.e1fa12d)'

Description of changes

Things done

Built on platform(s)

[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3375


updated content on link expired page when signing in

Context
Simplified the content on the page that tells users their sign in link has expired. This work is part of the reviewing of all sign in screens.
Changes proposed in this pull request
Content now reads:
[H1] The link you selected has expired
The sign in link expires after one hour and only works once so you need a new link.
[Button] Email me a new link
Link to Trello card
https://trello.com/c/EnpbshwR/506-review-the-email-link-expired-screen
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

 Review the "email link expired" screen


Modus checkbox focus state corrections - focused exclusively on the checkbox and ignoring the label

Description
Modus checkbox focus state corrections - focused exclusively on the checkbox and ignoring the label
References #1393
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?

Checklist

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

as discussed @sankarNous the PR should be merged to main


Add ubuntu based e2e tests for log enricher

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Add an ubuntu based e2e test for syslog and containerd (or docker shim) testing.
Which issue(s) this PR fixes:
None
Does this PR have test?
Yes
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

Codecov Report

Merging #538 (a75a7f8) into master (2a5edd1) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a75a7f8 differs from pull request most recent head 7f28249. Consider uploading reports for the commit 7f28249 to get more accurate results

@@           Coverage Diff           @@
##           master     #538   +/-   ##
=======================================
  Coverage   44.08%   44.08%
=======================================
  Files          25       25
  Lines        1311     1311
=======================================
  Hits          578      578
  Misses        696      696
  Partials       37       37

/test pull-security-profiles-operator-test-e2e

/retest

This works and tests now via /var/log/syslog on Ubuntu and dockershim :upside_down_face:


Add TRISO 2D and 3D examples

clsoes #236
The examples are from the published paper which has been reviewed through LRS before.

Job VTB Documentation on 9155e65 wanted to post the following:
View the site here
This comment will be updated on new commits.

Thanks for the PR! This is real close to being ready.

@GiudGiud Bad rebase I need to clean my branch.

messed up with the rebase. I am going to close this one and create a new one see  #370


Integrations page improvements

Hello @Saumya-Sunder ,
Can you please take a pass on our integrations page (https://github.com/Seagate/cortx/blob/main/doc/integrations/README.rst) and make it look a bit nicer?
The only thing I can think of to make it look nicer is to add the Splunk logo but please feel free to add additional ways to improve the aesthetics if you can think of any.

Hi @johnbent done https://github.com/Seagate/cortx/pull/457


Save multiple surveys with one SAVE button

Are you requesting a feature, reporting a bug or asking a question?
Question / Request a feature
What is the current behavior?
Saving a survey individually is fine and working
What is the expected behavior?
Is there a possibility to save multiple surveys with a single save button?
How would you reproduce the current behavior (if this is a bug)?
We have a single survey each in multiple expansion panels. (Angular material is used for expansion panels).
we have to implement, Page level saving, i.e., surveys from all expansion panels needs to be saved with a single save button which is at the end of the page.

Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
your_code_here

Specify your

browser: chrome
browser version:
surveyjs platform (angular or react or jquery or knockout or vue):  angular 5
surveyjs version: 1.0.25

If you create multiple expansion panels, you can collect links to each SurveyJS editor instance.
So you can iterate through the editor instances and call editor.saveButtonClick() function.

We don't have surveyjs editor within our application. We create surveys using online survey builder and copy the json in our local db

@ashygk Do you want do save all surveys?
every survey has the data property JSON format, survey.data. You may get the data from the current page as: survey.currentPage.getValue() - it will return values in JSON format all visibles questions on the current page.
Please read this help topic for more information.
Thank you,
Andrew

Hi, I have question related to diff issue. Could you please check my question there
https://github.com/surveyjs/surveyjs/issues/384


WIP: Ability to export and import credentials into folder

Currently there is no tool to export and import credentials into folders,
This would be a nice feature for butler.

Sorry, wrong repo. First want to merge it into our master.


[KHO-237] TF Deployment Fix

Checklist

[X] I have performed a self-review of my code
[X] I have reached out to another developer to review my code
[X] New and existing unit tests pass locally with my changes
[X] All Lint and CI checks are passing

See successful Actions Workflow.


Upgrade Astro and Starlight versions

Reported by user:

Starlight's sidebar doesn't include a trailing slash ("/") at the end of the URL, leading to a 301 redirect to the page. It is a minor thing, but SEO-wise it is not great. I tried to look it up online but couldn't find any solution 😦

Astro team response:

You are on an older version
Quite an old version of the 3 libraries
starlight-blog fixed the trailing slash on 0.6.0, but your project is on 0.4.0
It requires Astro 4 and Starlight 0.19 since 0.5.0, but your project is using Astro 3 and Starlight 0.13
You might want to upgrade them

Thread: https://ptb.discord.com/channels/686873244791210014/1232786035230904341/1232786035230904341
We should upgrade our Astro version and Starlight version to get the benefits of latest fixes.

Hi @infomiho, I want to work on this issue. Can you please assign this to me?

As discussed in Discord, here is my first try at it:
Ricardos-MacBook-Pro:aiblogarticles1 rbatista19$ cd blog
Ricardos-MacBook-Pro:blog rbatista19$ npx @astrojs/upgrade

 astro   Integration upgrade in progress.

      ◼  @astrojs/tailwind is up to date on v5.1.0
      ●  @astrojs/check will be updated to v0.5.10
      ●  @astrojs/starlight will be updated to v0.21.5
      ●  @astrojs/starlight-tailwind will be updated to v2.0.2
      ▲  astro will be updated to  v4.7.0

  wait   One package has breaking changes. Continue?
         Yes

 check   Be sure to follow the CHANGELOG.
         astro Upgrade to Astro v4

 ██████  Installing dependencies with npm...

      ▲  error Dependencies failed to install, please run the following command manually:
npm install<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Ricardos-MacBook-Pro:blog rbatista19$ npm install<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@astrojs/starlight
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@astrojs/starlight-tailwind
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/starlight-blog
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   1 more (the root project)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/astro
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@astrojs/starlight
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /Users/rbatista19/.npm/_logs/2024-04-29T16_28_36_980Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /Users/rbatista19/.npm/_logs/2024-04-29T16_28_36_980Z-debug-0.log

resolved in https://github.com/wasp-lang/open-saas/pull/122


Could you help explain in extract_local_patterns Line 60

Hi @cavalli1234 @Dawars  !
Thanks so much for sharing the code. I love reading your paper, and it proves that there are so many good combinations that can create a state of the art performance. I am learning about it from your paper.
Here,  I tried to understand how the outlier rejection works from the code. However, I have stumped upon extract_local_patterns Line 60. I wonder what are you trying to do ? Could you please help explain?
Line:60 expanded_local_scores = scores[tokp1] + ransidx.type(scores.dtype)

I wonder why you have added them together?
The score and ransidx are variables that are used for a totally different purpose. The score was containing the ratio of the lowest value and the second-lowest value (I guess it is related to Lowe's thresholding). Meanwhile, the ransidx was from Line 51, which are the row of  fnn_to_seed_local_consistency_map_corr whose number of neighbors are higher than a specified value of inliers, or specifically, it is...:
**Line:51**    ransidx, tokp1 = torch.where(fnn_to_seed_local_consistency_map_corr)

Or is this the way you re-order to be the same as ransidx? Could you please kindly explain....  :)

Hi @GabbySuwichaya!
Your intuition is correct, this is indeed the way I sort samples internally to each independent ransac problem. So here is how it works:
As you already mentioned, scores holds the descriptor distance ratios, but any scoring in the range [0, 1] could be plugged in. Here the assumption is that correspondences with lower scores are more likely to be correct than others, so we want them to be more likely to be sampled in following RANSACs. The RANSAC implementation in AdaLAM (inspired by PROSAC) samples points deterministically giving priority to the first examples: https://github.com/cavalli1234/AdaLAM/blob/47de1dd620734b3c42a852d093bbba2285334de5/adalam/ransac.py#L89
So we want low scores to be first. However, just sorting the scores would mix up all local RANSAC problems which should be instead solved independently in parallel. What prevents mixing up distinct problems is the ransidx which literally identifies each problem with an integer index.
ransidx comes from:
https://github.com/cavalli1234/AdaLAM/blob/47de1dd620734b3c42a852d093bbba2285334de5/adalam/core.py#L51
where fnn_to_seed_local_consistency_map_corr comes from function extract_neighborhood_sets which assigns keypoints to seeds. fnn_to_seed_local_consistency_map_corr is a (num_seeds, num_keypoints) boolean matrix which holds the assignment. Keep in mind that each seed with its local neighborhood is associated to a specific local RANSAC problem, so the index of the seed is the index of the distinct RANSAC problem. This is how the where function extracts the ransac identifiers indices at the first output. tokp1 is instead indexed over all keypoints, so it can be used to find the corresponding score of each keypoint in its corresponding ransac problem with score[tokp1].
So as you said, scores and ransidx have very different and apparently incompatible meanings. However, we want the output to be sorted by scores with consecutively equal ransidx. Given that scores are always smaller or equal to 1 and ransidx is integer, the two signals never mix up when summed and allow to actually obtain the desired sorting and apply a deterministic variant of PROSAC sampling.
Let me know if the explanation is clear or any further is needed! And thank you for the interest!

@cavalli1234 Thank you so much for the information and the clarification.

Keep in mind that each seed with its local neighborhood is associated to a specific local RANSAC problem, so the index of the seed is the index of the distinct RANSAC problem.

Is this related to preparing the index for which each of the different local neighborhoods will have different RANSAC parameters in the paper?

Also, could you please recommend the parameter settings of AdaLAM for the case where the number of keyponts is lower than 2000?  Let's say 512, 1,024, and 2048 keypoints.
At the moment, I am using the default setting, which I guess is the setting at 8,000 keypoints ...  The best case of your method is 8,000 keypoints (on the Image Matching challenge) is that right?
However, I would like to give it the best scenarios at the lower number of keypoints as well. If you have separate sets of parameters for the Image Matching challenge and for the general case, please feel free to share both...
I  would like to use your method at its best.....

@GabbySuwichaya
yes exactly the ransidx is the same index as the i in seeds S_i and their neighborhoods N_i in the paper, so all of their respective RANSACs are going to be independent.
For the parameter settings, I generally found these defaults to be quite robust to changing setups. The main issue with few keypoints is that the local statistics are poorer and provide less discriminative information to separate outliers properly. So a significant drop in performance for adalam is expected with decreasing number of keypoints. I do not have specific parameters for working with few keypoints, but I would definitely look into tuning parameters to increase neighborhood sizes (first increase search_expansion for increasing with same seeds and more overlap or increase area_ratio for increasing with same overlap and less seeds if you want to keep it fast). Larger neighborhoods generally mean less strictly affine patterns, so it might be necessary to also reduce the min_confidence to account for that.

I see. Thank you so much.
Your explanation does help me understand a lot. I will close the issue for now.
When I have more questions I will come back again.


Inconsistency in opacity documentation

i stumbled upon my bug again https://github.com/openlayers/openlayers/issues/10236
The JS Doc doesnt match the implementation of the option opacity in some cases:
Js Files in ol/scource/ for property opacity:
TileSource (in Tile.js):
jsdoc no default for opacity
sets opaque to false if undefined.
UrlTile extends Tile
jsdoc no default  for opacity
constructor passes on opaque from options to super without modification
TileImage extends UrlTile
jsdoc: default opaque=true
constructor passes on opaque from options to super without modification
XYZ extends TileImage
jsdoc: default opaque=true
constructor passes on opaque from options to super without modification
OSM extends XYZ
jsdoc: default opaque=true
defaults to true if options.opqaue is undefined
So OSM is the only Layer scource that behaves according to the jsdoc (default is true) but is also the only one that behaves not like the others. They default to opaque=false despite the js doc says otherwise.

@Kai-W A pull request that fixes the API docs would be appreciated. Thanks for your continued effort on this.


arangod.log: communication error: 'connection was locally canceled'

My Environment

ArangoDB Version:        3.6.7-1
Storage Engine:             RocksDB
Deployment Mode:        Cluster
Deployment Strategy:   ArangoDB Starter
Configuration:              3 machines
Infrastructure:               own
Operating System:        CentOS 7.3
Total RAM in your machine:        64Gb each
Disks in use:        HDD

Component, Query & Data
Affected feature:
slow query
AQL query (if applicable):
AQL explain and/or profile (if applicable):
Problem:
If I run a long time AQL query which will take time more than 1 hour,  then there is an error message in one of coordinator's arangod.log:
[21741] ERROR [abcdf] {communication} communication error: 'Connection was locally canceled'
[21741] WARNING [8d035] {aql} During AQL query shutdown: Query ID 2543 not found server: PRMR-21448c66-85a9-471e-b706-3799a3a9afcf
[21741] WARNING [8bcee] {queries} slow query: '......', bind_vars: {....}, database: test, user: test,
 took: 3630.278753s

There is also another error  message in one of dbserver's arangod.log:
[21720] ERROR [baef6] {aql} Timeout waiting for query 14279
[21720] INFO [88857] {cluster} Server CRDN-55575f8f6-da43-40cc-28c4-dadfa134vff rebooted, aborting its old jobs now.
[21720] ERROR [abcdf] {communication} communication error: 'Connection reset by perr'
[21720] INFO [88857] {cluster} Server CRDN-55575f8f6-da43-40cc-28c4-dadfa134vff rebooted, aborting its old jobs now.
[21720] WARNINg [8d035] {aql} During AQL query shutdown: Query ID 10021478 not found on server: CRDN-55575f8f6-da43-40cc-28c4-dadfa134vff rebooted

By the way, how could I understand the log for arangodb services? For example, what does the string  containing 5 chars in [] mean? How could I recognize the id for each service?
Expected result:
A lone time AQL query could be executed successfully.

[8d035]  is used to isolate the place where the log line was emitted - https://github.com/arangodb/arangodb/blob/3.6-no-scheduler-fix/arangod/Aql/RemoteExecutor.cpp#L376 in this case.

@mathsyouth is it possible that the Coordinator runs out of memory while during query execution? Can you check whether memory consumption increases up to the installed amount of memory before it reboots?

@Simran-B How could I check whether memory consumption increases up to the installed amount of memory before it reboots?

The simplest way would probably to run top in a shell on the Coordinator and check the memory usage per process.

@mathsyouth Did you have a chance to check the RAM usage?

@Simran-B I have logged the RAM usage every 3 seconds and verified that you are right. When the error happened, the RAM usage is about 99%.

Hi,
You could revisit your queries using explain and profile to see the number of documents that are transfered up to the coordinator, in order to find measures to apply filters that can be pushed out to the DB-Servers.
Do you have any queries in special that cause the situation? or is it just that there is more application load that the system can maintain?


adding vendore chart.js angular-chart.js

not sure if this is issue or just my lack of understanding of the setup. I am trying to add chart.js support to the vendor.js build file

bower install angular-chart.js --save

in gulpfile I added the libs to the build
vendorJS: {

        './libs/angular-chart.js/dist/angular-chart.js'

vendorCSS: {

        './libs/angular-chart.js/dist/angular-chart.css', // charts JS

I am getting the following error after build

gulp build

[16:23:19] Finished 'fonts' after 2.41 s
events.js:85
throw er; // Unhandled 'error' event
^
Error: Cannot find module 'angular' from 'libs\angular-chart.js\dist'
at node_modules\browserify\node_modules\resolve\lib\async.js:46:17
at process (node_modules\browserify\node_modules\resolve\lib\async.js:173:43)
at ondir (node_modules\browserify\node_modules\resolve\lib\async.js:188:17)
at load (node_modules\browserify\node_modules\resolve\lib\async.js:69:43)
at onex (node_modules\browserify\node_modules\resolve\lib\async.js:92:31)
at node_modules\browserify\node_modules\resolve\lib\async.js:22:47
at FSReqWrap.oncomplete (fs.js:95:15)
I also need to add the chart.js to the angular injection process in common.js
// app/common/common.js
...
require('chart.js');
module.exports = angular.module('common',
[
'chart.js',
...

hey, sorry for the late response. Been a busy few days for me. Did you get this sorted out? It looks like you had it configured correctly, but I haven't had a chance to test this myself yet.

Thanks for getting back to me.
It is still an issue. I am not sure this is my configuration or Chart.js doesn't support browserify.
appreciate any progress on this, thanks.

Looks like the angular-chart documentation is incorrect on their homepage (http://jtblin.github.io/angular-chart.js/). Seems to be updated on the github page, though. That was really lame of them.
'./libs/Chart.js/Chart.js' -> this is the path you need

Their homepage gives this path:

But on the github page they give this:

Looks like you need both. The angular-chart file definitely seems to have an issue.

https://github.com/jtblin/angular-chart.js/issues/220
I've opened an issue on angular-chart. Seems to be an issue on their end.

From my understanding (I haven't tried personally) you should be able to work with the node package instead of the bower component for now. As such, I'm going to close this ticket unless there's anything else.


Updating unfurled messages

Is it possible to let bot update his own messages (somehow monitor lifecycle of giltab's entities), for example mark MR as merged, Issue as closed etc.?
Probably it will require setting up webhook from gitlab, but I think it can help a lot to developers who are posting requests for code reviews in slack channels.

Ooooh, good idea! Difficult to plan the best architecture for this though so I'd leave this issue for later 😇

and then also consider that the links could be posted to multiple channels, and otherwise multiple times :)


Display effect model of activity type in the UI

ramaswam
https://jira.jpl.nasa.gov/browse/AERIEQS-327
Low
Planners can view implementation of an activity type via single click to understand its behavior.
One solution could be to link Aerie mission model to a github repo (of a specific branch or version), so that activity implementation can be linked.

Closing as out of scope. If we ever get another request for this we can create a new well-scoped ticket.


the download page for windows is broken

Problem description

Problem location

I saw a problem on the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL:

Other:

Project version(s) affected

Suggestions for a fix

You didn't provide the link that was broken, but I suspect this was related to https://github.com/docker/docker.github.io/pull/10691
should be fixed by https://github.com/docker/docker.github.io/pull/10691 (which will be published on docs.docker.com soon)
Meanwhile, the correct URL is https://docs.docker.com/docker-for-windows/install/


Tesseract v5: hocr_font_info 1 dont return font name (x_font)

Hi,
I'm trying to run tesseract's last version with hocr and hocr_font_info activated to obtain the name and size of the font.
 I read in other issues (https://github.com/tesseract-ocr/tesseract/issues/684)
that this problem could be by the tesseract version.
Is possible to obtain the font name in the last tesseract version?
 print("Tesseract version: ", pytesseract.get_tesseract_version())
         Tesseract version:  <IP_ADDRESS>20118

Thanks in advance!!

LSTM model does not provide WordFontAttributes and information provided by legacy engine (a.k.a tesseract 3) are IMHO not reliable.

We do not support 3rd party tools. Try to test with the tesseract command line tool. Use tesseract version 5.2.0.
If the issue still exists with tesseract itself, provide the full command you used, and the input image.

@amitdo @emigomez How can I install  Tesseract <IP_ADDRESS>20118 in ubuntu

@an1018, this is an issue tracker, not a forum.
Please use our forum for asking questions.

where are you running the code?


Safer amount conversion from Stellar amounts to host chain amounts

Stellar amounts are stored in i64 with 7 decimals. Host chain amounts are stored in u128 with 12 decimals.
When converting Stellar amounts to host chain amounts, the conversion (i.e., multiplication with 100_000) is currently done in i64 instead of widening it first to u128. This can lead to overflows that are currently just saturated to i64::MAX.
Instead we should first widen the value to u128 and then multiply with 100_000.

This only becomes a problem at amounts at least 2^63 / 10^12 ≈ 9.2 million. But it's good to fix now as the fix is simple.


Debugging fails if test name contains (, ), possibly more

We have a test that is specified as follows:
it("(true)", () => {
   // ... test implementation
}

When using Debug Jest to run this, the extension falls over with the following error message:

syntax error near unexpected token `('

This could probably be fixed by always surrounding value for the -t parameter with single quotes. Instead of
... --inspect-brk=35753 node_modules/.bin/jest -i /lambda/src/parser/parser.micro.ts -t (true)

this would work:
... --inspect-brk=35753 node_modules/.bin/jest -i /lambda/src/parser/parser.micro.ts -t '(true)'

While at the moment we observe this for parenthesis only, it is possible that other special characters may cause a similar problem.
Note that we can run jest normally, so we believe this is a problem with how this extension crafts the command line for jest. Also choosing Run Jest surrounds that parameter value with single quotes, e.g.
node '/lambda/node_modules/.bin/jest' '/lambda/src/parser/parser.micro.ts' -t '(true)'

Our setup:

VS Code version 1.36.1
vscode-jest-runner version 0.3.6

@ManfredLange
Thx for your report!
i try to reproduce your issue:
  it("(true)", async () => {
    const hours = await ComplainCollection.getHoursByDate('1234', '2017-10-01');
    expect(hours).toBe('0');
  });

╰─>$ node '/home/firsttris/Projects/timetracking-app/node_modules/.bin/jest' '/home/firsttris/Projects/timetracking-app/packages/server/src/collections/ComplainCollection.test.ts' -t '(true)'
 PASS  packages/server/src/collections/ComplainCollection.test.ts
  ComplainCollection Test
    ✓ (true) (18ms)

but the test passes.
also tried with single quotes in the test name
vscode-jest-runner version 0.3.6
On which OS are you?
Which Terminal are you using? (.e.g. powershell, bash, cmd, fish)

@ManfredLange the quotes are missing, u are right.

it think we need to quote() here https://github.com/firsttris/vscode-jest-runner/blob/master/src/extension.ts#L115
like we did for the normal run here
https://github.com/firsttris/vscode-jest-runner/blob/master/src/extension.ts#L60

fixed
https://github.com/firsttris/vscode-jest-runner/commit/353b4b4feb998cc17ba550061da17cf02b1aee15
in version version 0.3.8
╰─>$ cd /home/firsttris/Projects/timetracking-app ; /usr/bin/node --inspect-brk=39416 node_modules/.bin/jest -i '/home/firsttris/Projects/timetracking-app/packages/server/src/collections/ComplainCollection.test.ts' -t '(true)'
Debugger listening on ws://<IP_ADDRESS>:39416/c1eda623-d1ab-4568-8c42-c4fa6a905ba4
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
[ts-jest][DEPRECATED] - replace any occurrences of "ts-jest/dist/preprocessor.js" or  "<rootDir>/node_modules/ts-jest/preprocessor.js" in the 'transform' section o
f your Jest config with just "ts-jest".
 PASS  packages/server/src/collections/ComplainCollection.test.ts
  ComplainCollection Test
    ✓ (true) (20ms)

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        5.101s
Ran all test suites matching /\/home\/firsttris\/Projects\/timetracking-app\/packages\/server\/src\/collections\/ComplainCollection.test.ts/i with tests matching "(true)".
Waiting for the debugger to disconnect...
fish: “/usr/bin/node --inspect-brk=394…” terminated by signal SIGKILL (Forced quit)

please re-open if the issue still exists.
regards
Tristan


Add uz transtation for form

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
fix statement

Thanks!


[FEATURE] Remove SfProductOption as it is no longer needed

Describe how this feature will help us improve
Remove the component itself and from all the places it appears. According to new designs it is no longer needed.

Product Option is still used in Product component. It also appears in Scrollable stories.


Add localization file watcher

What problem are you trying to solve?
Closes https://github.com/Shopify/checkout-web/issues/9581.
Necessary for https://github.com/Shopify/checkout-web/issues/9365.
This is a clean slate branch for this older draft pull request
This is a follow-up PR which will move where extension.Localization is being processed from happening at runtime for HTTP GET /extensions/, to a new watcher: build.WatchLocalization.
The goal of this PR is to increase performance by moving the processing of locales (which requires fs i/o) to the offline build watcher which will only re-process localization if the locale files have changed.
The localization property for an extension in the HTTP GET /extensions/ response should remain the same. When a locale file is changed, it should also notify websocket subscribers of the change.

Example response
Some properties are omitted for brevity
{
  "extensions": [
    {
      "assets": {
        "main": {
          "name": "main",
          "url": "https://7cbc-2600-1700-5658-3c50-fdd0-8885-93fc-9d51.ngrok.io/extensions/00000000-0000-0000-0000-000000000000/assets/main.js",
          "lastUpdated":<PHONE_NUMBER>
        }
      },
      "development": {
        "resource": {
          "url": "cart/1234"
        },
        "root": {
          "url": "https://7cbc-2600-1700-5658-3c50-fdd0-8885-93fc-9d51.ngrok.io/extensions/00000000-0000-0000-0000-000000000000"
        }
      },
      "extensionPoints": ["Checkout::Feature::Render"],
      "localization": {
        "defaultLocale": "en",
        "lastUpdated":<PHONE_NUMBER>,
        "translations": {
          "en": {
            "welcome": "Welcome to the {{extensionPoint}} extension!"
          },
          "fr": {
            "welcome": "Bienvenue dans l'extension {{extensionPoint}}!"
          }
        }
      },
      "metafields": [],
      "type": "checkout_ui_extension",
      "uuid": "00000000-0000-0000-0000-000000000000",
      "version": "",
      "surface": "checkout",
      "title": "Checkout UI Test Extension"
    }
  ]
}

How are you solving it?

Remove the getExtensionsWithLocalization() from happening at runtime on response to the /extensions/ endpoint
Implement fsnotify in build.WatchLocalization to update an extension's Localization only when its locale files have changed.
https://github.com/Shopify/shopify-cli-extensions/blob/be7546b8d063f855eeec4bbb8e4eb7a7a08c8d73/build/build.go#L148
To avoid interfering with assets' LastUpdated attribute, add a distinct LastUpdated timestamp attribute for Localization.
https://github.com/Shopify/shopify-cli-extensions/blob/be7546b8d063f855eeec4bbb8e4eb7a7a08c8d73/core/core.go#L86-L90

Tophatting

Complete tophatting instructions
Create a Spin workspace:
spin up checkout-web,online-store-web --name cli-locales

Local setup
While that's booting, set up your local machine.
In a local shell, get the shopify-cli-extensions patch, and expose its package for linking:
dev clone shopify-cli-extensions
git checkout watch-locales
dev up
make bootstrap
cd packages/shopify-cli-extensions
yarn link

In a local shell, setup shopify-cli and symlink shopify-cli-extensions:
dev clone shopify-cli
dev up
rake extensions:symlink

In a local shell, get a reference extension from checkout-ui-extension-dev, and link it to shopify-cli-extensions:
dev clone checkout-ui-extension-dev
git checkout new-cli
dev up
yarn link "@shopify/shopify-cli-extensions"

If you don't have ngrok configured, get a token and add it to your local machine:
ngrok authtoken [the token string]

The checkout-ui-extension-dev repo provides yarn shopify, an alias to your local shopify-cli executable.
Make sure you are logged in to a development store (in production) that's configured for extension development to work around this bug. Example:
yarn shopify logout
yarn shopify login --store=https://your-dev-store.myshopify.com/

Select an extension you use for development.
yarn shopify extension connect

Run shopify extension serve using an alias:
yarn start

The output should end with something like:
┏━━ Serving extension… ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
┃ DEBUG Sending configuration data to extension development server …
┃ DEBUG Starting message processing threads to relay output produced by the extension development server …
┃ Shopify CLI Extensions Server is now available at https://c52a-2600-1700-5658-3c50-11e0-9470-dd4c-d96e.ngrok.io
┃ <extension-name> (successfully built localization)
┃ 2022/04/04 09:59:14 Watcher added for: locales
┃ <extension-name> (Build succeeded)

Viewing on checkout
Connect to the spin instance previously created and checkout the mleandres/extensions-localization-hot-reload branch for checkout-web:
spin shell
> cd checkout-web
> git fetch origin mleandres/extensions-localization-hot-reload
> git checkout mleandres/extensions-localization-hot-reload
> update

Open your spin instance's shop website and create a checkout.
Copy the ngrok URL from the output of the previous yarn start
Append the following to the checkout URL in your browser, using your own ngrok URL:

?dev=https://76bc-96-20-124-218.ngrok.io/extensions/

Example checkout URL:

https://shop1.shopify.cli-locales.martin-gilles-gingras.us.spin.dev/checkouts/c/a7a099f6700a0c343b3ac92c34935df5/information?dev=https://76bc-96-20-124-218.ngrok.io/extensions/

In your browser, open the Network tab, use the WS filter and open the extensions/ item:

This will show all incoming updates to either the code or the locale files of your running checkout-ui-extension-dev extension:

Inspect the contents of each update to validate the proper data is being sent.
Using the mleandres/extensions-localization-hot-reload branch for checkout-web, you should see hot reloading of both extension code and localizations when receiving such updates.

Result

Result video in Checkout
https://user-images.githubusercontent.com/3000613/161583386-9cdcff46-f1a8-4810-9643-d16a97a701db.mov

Notes
This PR will set us up to implement proper hot module reloading in checkout-ui-extensions when locales change, via this other pull request.
When a locale file changes, the localization property for that extension should be updated automatically both within the HTTP GET /extensions/ endpoint and trigger an update in the websocket via api.Notify([]core.Extension{extension}). api.Notify takes care of merging the data and notifying all subscribers for you.

@ginmrt I just tophatted this again and can confirm this is working well on my end 👍
Specific tests:

Update a translation value for the current locale
Update a translation value for the default fallback locale
Remove a translation key + value
Add a translation key + value
Add a new locale file and translation keys
Delete a locale file
Rename a locale file

This looks good to me!

@ginmrt it looks like from your latest commit that you are no longer returning a null value for localizations when they do not exist and instead empty objects. This could cause a problem in checkout-web as it expects a null value for localizations in that case.
I also just tried to tophat this change it appears to have broken the hot reload for translations in checkout-web.


Add Digitalfabrik support post

Closing as this is too much on the side of promotion. It could be "transformed" to a valuable and interesting blog post though.


horizontal alignment

Using the current demo https://mathjax.github.io/mj3-demos/mj3-tex2html-global.html with Firefox/Linux the input \displaystyle\sum (1)\textstyle\sum (1)\scriptstyle\sum (1)\scriptscriptstyle\sum (1) looks like

where the bottom edges of brackets, numbers and operators are aligned, instead of

(LaTeX) or

(math.stackexchange)

I am not able to reproduce your result in Firefox on Mac OS.  On the other hand, if I disable one of the CSS rules, I do get your result, so I'm wondering if that rule isn't triggering properly for you.
Can you open the developer console and inspect one of the parentheses, and see if you see the highlighted rule below:

(the one for mix-symmetric > mjx-c)?  If I disable this rule, I see the same result that you see.

I think the rule is there:

If I disable the rule, it does not change anything.

vertical-align: top

vertical-align: 1em

vertical-align: bottom

OK, that is helpful.  Can you try this?  Select the mjx-symmetric element surrounding one of the parentheses, and add display: inline-block to the rule for that element:

Then select all the mjx-mstyle elements (one at a time) and add display: inline-block as element properties:

You'll have to do it individually, since there is no CSS rule for mjx-mstyle currently that would let you do it for all at once.
See if doing that resolves the issue (fingers crossed).

I think that is the issue. I was able to apply it to the first bracket:

Maybe I am just too stupid to apply it to the others, but it seemed like the display: inline-block got inherited from the first one, so it was already shown in the console with the green marking and I couldn't add it again.

OK, I've pushed a modified version for testing that includes the display: inline-block settings.  Put your example into that and see if it works properly.

No, that doesn't work. It ist the same as the unmodified version.

For the screenshot I uploaded earlier, I might have added some display: inline-block property falsely to some other element as well.

OK thanks.  I'll have to think of something else.  What flavor of linux are you using, and what version of Firefox?

debian stretch with firefox version 52.8.0

I can reproduce that issue on Firefox/Linux v58.2.0 and v61.0b7 (aurora channel).

I tried MacOS, Firefox v61.0beta and could not reproduce the problem.
However, on ChromeOS, Firefox v60.0.1 I can see the same issue. So it is probably a Linux problem.

Do you mean that you could not reproduce the same issue on Windows?

I am also able to reproduce on windows.  So it looks like Mac OS is the outlier.  I will have to rethink how this is being handled.   Rats!

This has been merged into the master branch.


Feature/Reprediction

Issue #, if available:
#50
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Close #50


[feature] option to return only keys from v3 "etcdctl get"

Currently when I do ETCDCTL_API=3 etcdctl get --prefix /, it returns both keys and values.
I will want to have better control over this, like returning only keys, returning keys and versions.
The other option is to use etcd client library to write a program to do this. But programming and compiling took a lot of time :)
It would be great if etcdctl can provide options to control the output.

@xiang90 As we receive the GetResponse object can we just filter the output based on what the user has asked for (like --keys or --values or --with-version). If we are to just filter the GetResponse object then this would make sense for SimplePrinter only right? For JSON and protobuf the user will have access to the GetResponse object and they can filter it themselves.

@ajityagaty I think we'd better to do this in the server-side. People would like to avoid huge traffic going through decoding + network path.

@xiang90 Yes, makes sense. Would it be ok if I look at it?

@ajityagaty Sure. Go ahead. Thanks! Probably we want to add a optional field in the range request (no_value or only_key).

@xiang90 Thanks! Will look into it.

@xiang90 I was looking at the changes that I'll have to make. The RangeRequest will have a flag to indicate "keys only". If the flag is set then in kvServer.Range method, after reading the keys from the store, would it be enough if we set the Value field of every mvccpb.KeyValue to nil?

@ajityagaty That is how I would expect it to work.


php -c {custom php.ini} doesn't work

Describe the bug

$ cat php.ini
date.timezone=Asia/Tokyo

$ php -c php.ini | grep timezone
Default timezone => UTC
date.timezone => UTC => UTC # This should be overwritten with Asia/Tokyo

Version

[x] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems

ubuntu-latest (Ubuntu 18.04 LTS)
PHP versions

8.1
To Reproduce

name: CI

on: push

jobs:
  test:
    runs-on: ubuntu-latest

    steps:
      - uses: shivammathur/setup-php@v2
        with:
          php-version: 8.1

      - uses: actions/checkout@v2

      - name: debug
        run: |
          echo " "
          echo "$ php -i | grep php.ini"
          php -i | grep php.ini
          echo " "
          echo "$ php -i | grep timezone"
          php -i | grep timezone
          echo " "
          echo "$ cat php.ini"
          cat php.ini
          echo " "
          echo "$ php -c php.ini -i | grep php.ini"
          php -c php.ini -i | grep php.ini
          echo " "
          echo "$ php -c php.ini -i | grep timezone"
          php -c php.ini -i | grep timezone

Result of the debug step of above workflow is following.

Loaded Configuration File is correctly overwritten with /home/runner/work/{repo}/{repo}/php.ini, but date.timezone is not changed🤔
Expected behavior

$ php -c php.ini -i | grep timezone
Default timezone => UTC
date.timezone => Asia/Tokyo => Asia/Tokyo

Screenshots/Logs

See above.
Additional context

Are you willing to submit a PR?

@ttskch I pushed a fix, instead of v2 can you test with the develop branch and let me know.
- uses: shivammathur/setup-php@develop
  with:
    php-version: 8.1

@shivammathur It works well!!👍

Release 2.19.0 with the fix.
https://github.com/shivammathur/setup-php/releases/tag/2.19.0


Determine conformance container image version at runtime for GKE

GKE's cluster version now defaults to v1.24.1, which no longer creates
secret for service account automatically, the verification in old test
cases would fail. This patch makes the test determine conformance
container image version based on the cluster's version to avoid such
issue.
However, the test suite Netpol introduced in new conformance image uses
a Namespace creation function which is not robust, leanding to random
test failures in GKE test. This patch skips it temporarily.
Besides, legacy NetworkPolicy tests have been skipped by mistake for a
while because of a conflicting skip regex. This patch fixes it.
For https://github.com/antrea-io/antrea/issues/3762
Signed-off-by: Quan Tian<EMAIL_ADDRESS>

LGTM Are we running legacy NetworkPolicy tests for all 3 clouds?

Yes, the latest patch updated to run legacy NetworkPolicy tests only for all 3 clouds because AKS also seems unable to run it due to some some issues: https://jenkins.antrea-ci.rocks/view/cloud/job/cloud-antrea-aks-conformance-net-policy/287/. I think it's perhaps better to wait for NetPol being more reliable before running it on any cloud.
netpol-1199-x/a Ready:False: Unschedulable: 0/2 nodes are available: 2 Too many pods.
netpol-1199-x/b Ready:False: Unschedulable: 0/2 nodes are available: 2 Too many pods.
netpol-1199-x/c Ready:False: Unschedulable: 0/2 nodes are available: 2 Too many pods.
netpol-1199-y/a Ready:False: Unschedulable: 0/2 nodes are available: 2 Too many pods.
netpol-1199-y/b Ready:False: Unschedulable: 0/2 nodes are available: 2 Too many pods.
netpol-1199-y/c Ready:False: Unschedulable: 0/2 nodes are available: 2 Too many pods.
netpol-1199-z/a Ready:False: Unschedulable: 0/2 nodes are available: 2 Too many pods.

/skip-all


Which split was used for DAVIS?

The readme states:
    ├──── davis90/
    |   ├──── bear/
    |   ├──── ...
    |   └──── walking/

but if bear and walking are the first and last entry respectively then the best match I can find is the DAVIS 2017 train split, not val?  Any insight would be appreciated, thanks!

Ok actually it looks like no splits were used and all the videos were included instead. Would you mind confirming this is correct please? Thanks again in advance!

Hi, sorry for the late reply.
The Davis90 test set used here can be downloaded from here, and this is provided by the authors of the QVI paper. I didn't check if this fully matches the DAVIS-17 splits.
Thanks.


DOTS GAME

PR Description 📜
issue no:#3074
i have added the DOTS game wherein the center dot has to shoot other upcoming dots to gain points .the other dots are of any shape and size & if they touch the center point or the player the game end .Itt also keep score for the game starting off with the initial score of 0

[✔ ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[✔ ] I have performed a self-review of my own code or work.
[✔ ] I have commented my code, particularly in hard-to-understand areas.
✔[ ] My changes generates no new warnings.
[ ✔] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[✔ ] I have added screenshot for website preview in assets/images
[✔ ] I have added entries for my game in main README.md
[ ✔] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[✔ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @vedanshipathak ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀


docs: add NavigationContainer to upgrading-from-4.x guide

The v5 migration guide was missing information about NavigationContainer.

Thanks for your contribution! Looks good to me. Can we merge it @satya164 ?


Could not resolve all dependencies for configuration ':react-native-admob:_debugCompile'

This error occurred when running cd android && ./gradlew installRelease from my app's base directory. Below is the full stack trace.
Lochlans-MacBook-Pro:appname loch$ cd android && ./gradlew installRelease
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':app'.
> A problem occurred configuring project ':react-native-admob'.
   > Could not resolve all dependencies for configuration ':react-native-admob:_debugCompile'.
      > Could not find com.google.android.gms:play-services-ads:8.3.0.
        Searched in the following locations:
            file:/Users/loch/.m2/repository/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.pom
            file:/Users/loch/.m2/repository/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.jar
            https://jcenter.bintray.com/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.pom
            https://jcenter.bintray.com/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.jar
            file:/Users/loch/Documents/appname/node_modules/node_modules/react-native/android/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.pom
            file:/Users/loch/Documents/appname/node_modules/node_modules/react-native/android/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.jar
            file:/usr/local/opt/android-sdk/extras/android/m2repository/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.pom
            file:/usr/local/opt/android-sdk/extras/android/m2repository/com/google/android/gms/play-services-ads/8.3.0/play-services-ads-8.3.0.jar
        Required by:
            TRN:react-native-admob:unspecified

Open your android folder in Android Studio, and go to Tools > Android > Sync Project with Gradle files to ensure that the play services will be referenced correctly. I was then able to build from CLI as per usual.

This can only be done with Android Studio ?


RuntimeError: batch % im2col_step_ == 0 ASSERT FAILED

Environment:

python 3.6
torch 1.0

Error occurs as follows when the batch_size is 128 or 192 or 256. But when batch_size = 64  is ok.
File "/home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/DCN-1.0-py3.6-linux-x86_64.egg/modules/deform_conv.py", line 62, in forward
    self.im2col_step)
  File "/home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/DCN-1.0-py3.6-linux-x86_64.egg/functions/deform_conv_func.py", line 34, in forward
    ctx.im2col_step)

RuntimeError: batch % im2col_step_ == 0 ASSERT FAILED at /home/gzh/ocr/Deformable-Convolution-V2-PyTorch-pytorch_1.0.0/src/cuda/deform_conv_cuda.cu:58, please report a bug to PyTorch. batch(%d) must divide im2col_step(%d)11764 (deform_conv_cuda_forward at /home/gzh/ocr/Deformable-Convolution-V2-PyTorch-pytorch_1.0.0/src/cuda/deform_conv_cuda.cu:58)

frame #0: std::function<std::string ()>::operator()() const + 0x11 (0x7f49387f4fe1 in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/torch/lib/libc10.so)
frame #1: c10::Error::Error(c10::SourceLocation, std::string const&) + 0x2a (0x7f49387f4dfa in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/torch/lib/libc10.so)
frame #2: deform_conv_cuda_forward(at::Tensor const&, at::Tensor const&, at::Tensor const&, at::Tensor const&, int, int, int, int, int, int, int, int, int, int, int) + 0x909 (0x7f493475a884 in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/DCN-1.0-py3.6-linux-x86_64.egg/DCN.cpython-36m-x86_64-linux-gnu.so)
frame #3: deform_conv_forward(at::Tensor const&, at::Tensor const&, at::Tensor const&, at::Tensor const&, int, int, int, int, int, int, int, int, int, int, int) + 0x79 (0x7f493473fb89 in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/DCN-1.0-py3.6-linux-x86_64.egg/DCN.cpython-36m-x86_64-linux-gnu.so)
frame #4: <unknown function> + 0x2dc97 (0x7f493474cc97 in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/DCN-1.0-py3.6-linux-x86_64.egg/DCN.cpython-36m-x86_64-linux-gnu.so)
frame #5: <unknown function> + 0x2dd3e (0x7f493474cd3e in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/DCN-1.0-py3.6-linux-x86_64.egg/DCN.cpython-36m-x86_64-linux-gnu.so)
frame #6: <unknown function> + 0x2a209 (0x7f4934749209 in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/DCN-1.0-py3.6-linux-x86_64.egg/DCN.cpython-36m-x86_64-linux-gnu.so)
<omitting python frames>
frame #14: THPFunction_apply(_object*, _object*) + 0x581 (0x7f4972b264d1 in /home/gzh/SoftWare/tf1.4/anaconda2/envs/python36/lib/python3.6/site-packages/torch/lib/libtorch_python.so)

I found that the im2col_step = 1 by default, it's strange why batch % im2col_step_  != 0  as the bug refers to?

Hi,
I also encountered this bug, it seems some details in "DCN" lib are not handled well enough to avoid this problem, so when it comes to the last step during training process, some tensors cannot be divided.
My solution is to set batch_size = 1, and it will work but time-consumingly.
@chengdazhi Hope u can fix this bug, it shouldn't take u much time.
Thanks!
Ian

Yes,when  I  set  batch_size to 64,it  will  not  wrong here

Hi, I came across the same problem, but it seems that it is not a bug. Because the "im2col_step" is a parameter. As is shown bellow:

We can always choose a proper "im2col_step" ensuring that "batch % im2col_step_ == 0".
To this end, I believe the problem is solved. However, I would like to know how "im2col_step" influences efficiency? Does a bigger value boost efficiency?

The 'batch' here does not represent batch size, it may be a concept in the CUDA core.

I agree with you, I also wonder how im2col_step influence the results.

I set batch_size = 64  ,and  im2col_step=64,but  still meet the error


Serde-like deserialization for Python to Rust?

Is there serde-like deserialization to convert Python objects to Rust?
Something like:
struct Foo(String, String);

// x is of type `PyAny`
let rustTuple: &Foo = x.as_rust_obj().unwrap();

See https://github.com/davidhewitt/pythonize

Closing as a duplicate of https://github.com/PyO3/pyo3/issues/566


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.23.12

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.23.11 -> v1.23.12

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.23.12
Compare Source
Bug Fixes

deps: update module github.com/ibm-cloud/power-go-client to v1.5.2 (#684) (c45f020)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


Mobile Responsiveness

Currently the site is not responsive and does not display properly on mobile.

I want to work on this, I am a contributor under JWOC.

@EmberTSeal the site is being redesigned . so let wait for that, thn u can see if making it responsive is needed or not

All right, thank you for responding.

I want to work on this issue under JWOC

As of now its not needed.


Update issue templates

added templates for feature-request and domain-access-reuest

@whatnick let's add one for bug-report too


Fix journald

Fix Vector agent DaemosSet for collect journald service logs
Add docs
Update helm to v0.0.6

Fix this issue - https://github.com/kaasops/vector-operator/issues/59


Ignore cache on .xhr fix

IgnoreCache flag should not cause the clear cache functionality to be ignored. Otherwise UI gets the old cached version on next request.

Merged, thanks


Verisure Smartplug/Smart Switches not working

Home Assistant release (hass --version):
0.44.1
Python release (python3 --version):
Python 3.4.2
Component/platform:
Verisure
Verisure SmartPlug
Description of problem:
It's no longer possible to turn Verisure smart switches on and off
More people with same problem:
https://community.home-assistant.io/t/verisure-smart-switches-not-working/15975
Expected:
It should be possible to turn it on and off
Problem-relevant configuration.yaml entries and steps to reproduce:
verisure:
  username: !secret verisure_username
  password: !secret verisure_password

Enable Verisure
Try to turn a Smartplug on or off
Nothing

Traceback (if applicable):

Additional info:

File "/usr/lib/python3.4/asyncio/tasks.py", line 233, in _step
result = coro.throw(exc)
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/core.py", line 1015, in _event_to_service_call
yield from service_handler.func(service_call)
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/switch/init.py", line 104, in async_handle_switch_service
yield from switch.async_turn_on()
File "/usr/lib/python3.4/asyncio/futures.py", line 388, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
value = future.result()
File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
raise self._exception
File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
result = self.fn(*self.args, **self.kwargs)
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/switch/verisure.py", line 53, in turn_on
hub.my_pages.smartplug.set(self._id, 'on')
File "/home/homeassistant/.homeassistant/deps/verisure/devices/smartplug.py", line 39, in set
return not self._session.post(COMMAND_URL, data)
File "/home/homeassistant/.homeassistant/deps/verisure/session.py", line 161, in post
self.validate_response(response)
File "/home/homeassistant/.homeassistant/deps/verisure/session.py", line 227, in validate_response
response.text.encode('utf-8'))))
File "/home/homeassistant/.homeassistant/deps/verisure/session.py", line 234, in raise_response_error
raise default_error
verisure.session.ResponseError: Unable to validate response form My Pages, status code: 404 - Data: b''
2017-05-11 17:56:19 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/lib/python3.4/asyncio/tasks.py", line 233, in _step
result = coro.throw(exc)
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/core.py", line 1015, in _event_to_service_call
yield from service_handler.func(service_call)
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/switch/init.py", line 104, in async_handle_switch_service
yield from switch.async_turn_on()
File "/usr/lib/python3.4/asyncio/futures.py", line 388, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
value = future.result()
File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
raise self._exception
File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
result = self.fn(*self.args, **self.kwargs)
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/switch/verisure.py", line 53, in turn_on
hub.my_pages.smartplug.set(self._id, 'on')
File "/home/homeassistant/.homeassistant/deps/verisure/devices/smartplug.py", line 39, in set
return not self._session.post(COMMAND_URL, data)
File "/home/homeassistant/.homeassistant/deps/verisure/session.py", line 161, in post
self.validate_response(response)
File "/home/homeassistant/.homeassistant/deps/verisure/session.py", line 227, in validate_response
response.text.encode('utf-8'))))
File "/home/homeassistant/.homeassistant/deps/verisure/session.py", line 234, in raise_response_error
raise default_error
verisure.session.ResponseError: Unable to validate response form My Pages, status code: 404 - Data: b''

I have found the issue.
Verisure has updated the URL for onoffplug.cmd.
In deps/verisure/devices/smartplug.py the COMMAND_URL has to be changed to /settings/smartplug/onoffplug.cmd.

Related: https://github.com/persandstrom/python-verisure/issues/74

This should be fixed in #7394

I will close this now, reopen if you still experience the issus


'Manage Todos' menu item seems broken

In prooph v7 this item does not do anything. Didn't test master though

same on master ;) No features defined for the menu item, yet

Can be closed. Solved in the light theme #101


Consolidate datafusion benchmarks into dfbench

Is your feature request related to a problem or challenge?
DataFusion currently has four different benchmark runners for comparing different versions of datafusion with itself, which have substantial overlap in their functionality and code
cargo run --release --bin nyctaxi
cargo run --release --bin h2o
cargo run --release --bin parquet
cargo run --release --bin tpch

I would like to add a 5th one (ClickBench) https://github.com/apache/arrow-datafusion/issues/6994 but would like to avoid copy/pasting yet again mostly the same thing
Describe the solution you'd like
Thus I propose a new consoidated binary dfbench run like this:
cargo run --release --bin benchmark -- nyctaxi
cargo run --release --bin benchmark -- h2o
cargo run --release --bin benchmark -- parquet
cargo run --release --bin benchmark -- tpch
cargo run --release --bin benchmark -- clickbench (I want to add this)

Describe alternatives you've considered
Follow the same pattern
Additional context
Task List

[x] Make consolidated runner dfbench: https://github.com/apache/arrow-datafusion/pull/7054
[x] consolidate tpch: https://github.com/apache/arrow-datafusion/pull/7054
[x] Update the benchmark readme
[x] consolidate nyctaxi
[ ] consolidate h2o -- tracked by https://github.com/apache/arrow-datafusion/issues/7209
[x] consolidate parquet
[x] Update bench.sh for new commands
[x] Move tpch queries into queries/tpch rather than queries

I have completed all the work I have planned for this epic -- consolidating h20.ai bench is tracked by https://github.com/apache/arrow-datafusion/issues/7209 so I am going to close this one as complete


Dump1090-mutability install script adding extra NET_BIND_ADDRESS statements to /etc/default/dump1090-mutability

Spotted this at the end of /etc/defaults/dump1090-mutability and need to investigate...
# Additional options that are passed to the Daemon.
EXTRA_ARGS=""
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=""
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=""
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=
#NET_BIND_ADDRESS=

Well that broken upgrade variable referenced here didn't fix this. Need to get onto this being it still persists.

If another bug fix release is made this should be there and not held off until the next feature release...


Friendly error messages

I just bumped into an EMR problem and it took me a while to figure out what the problem was and which steps I should skip. This PR includes a friendly error message to help with troubleshooting.

Hey @bernardosrulzon - thanks for this. We agree that it's quite confusing what resolution you should attempt if the job fails - @ihortom is currently working on a flowchart mapping the job steps to their corresponding data moves, and will start documenting different resolutions for the different cases.
We can't merge your specific suggestion here, because the solution Try running with --skip staging,enrich won't be correct in all cases. For example, at Snowplow, the DirectoryNotEmptyError is almost always because a prior run has not yet completed, rather than because the corresponding archive step has failed.
However, I think your idea of friendly error messages makes a lot of sense. My suggestion is that once @ihortom has finished his troubleshooting guide, we link to that guide from these error messages. Thoughts?

Ah, good to see you're working on this! Makes sense - I'll close this PR.
Cheers!


refactor(ES6): WebEnvironmentPlugin.test.js

What kind of change does this PR introduce?
refactor, ES5 => ES6

Did you add tests for your changes?
All existing tests pass

If relevant, link to documentation update:
N/A

Summary
Refactor WebEnvironmentPlugin.test.js to ES6

Does this PR introduce a breaking change?
No

Thanks


Update Take ownership part

In the app import section,
Please update point 4.
You can import an existing app which you created on App Studio to Developer Portal. To import an already published app to Developer Portal, the app owner needs to raise a service request through admin portal to transfer the ownership over the app ID.
Can we remove the service request through admin portal?
Please include this instead,
Global tenant admin of the Teams app will now be able to take ownership of any apps on their tenant that is missing owners.

[Enter feedback here]

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

ID: 7da63b6f-15d7-a6dd-82fc-d183afaeb816
Version Independent ID: 82d1ca83-28dd-220e-ab0c-a4833f8f8143
Content: Developer Portal for Teams - Teams
Content Source: msteams-platform/concepts/build-and-test/teams-developer-portal.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Hi @srinidyMSFT - Could you please help here to update the document accordingly?


Change Hook

Hi,
Some program use CreateFontIndirectA instead CreateFontIndirectW.
From disassembler I saw APIs point to CreateFontIndirectExW.
So, please hook CreateFontIndirectExW instead CreateFontIndirectW.
It will support more target.
Please take a look.
Thanks in advance,

Thanks, fixed.


required parameter

The language may not always be required but would probably default to required = true? (It adds a * to the control label.)

I added the required parameter (defaults to false)


Research and design for versioning, update and upgrades

One of the main feature we want to provide with this version of 0-OS is the "auto-update". The idea is that the system needs to be able to update itself and each of its component with the minimum downtime possible for the workloads. Minimum in most case would be no downtime at all.
Keeping this in mind actually drives quite a bit the design decision to make for all module of 0-OS.
One of them I would like to discuss here is how are we going to handle versioning of the system.
Since every component is by definition modular and choose be able to be change in place with another one or another version, a single version for the OS is not going to be something meaningful.  So instead of sticking with plain version I had another idea.
Having 3 "flavor" of 0-OS, main, dev and test like tfchain does.
Having these 3 "flavor" will allow to actually have different network into the grid:

Dev: ephemeral network only setup to develop and test new features. Can be created and reset at anytime
Test: Mostly stable feature that needs to be tested at scale, allow preview and test of new features. Always the latest greatest. This network can be rest sometimes, by should be relatively stable.
Main: Released of stable version. Used to run the real grid with real money. Cannot be reset ever. Only stable and battle tested feature reach this level.

This allow each component to progress at its own pace, have a separate semantic version. A flavor will be composed of different version of each modules. Once a new version of a module is ready, it will just make its way trough the 3 flavor from dev to main.
Each "flavor" will actually create a separate grid, the node will always ever connect to other node with the same "version". This simplify the code cause you know you don't have to deal with different version and can be sure of the feature of the node you're talking too.
The flow of a new feature will bubble up from dev to main. Every time a new feature is upgraded to the next level, all the node in this network will receive the update automatically.
This makes upgrading a network trivial and also ensure that upgrade procedure are at least tested at scale 2 time before reaching main net.

This approach looks good to me. We need, in addition, to define what should be versioned and how we will upgrade it.
We need well defined way to upgrade different parts, like host (cores, userland tools, hypervisor, drivers, kernel, ...), and running subsystems (containers, vm, ...)

We need well defined way to upgrade different parts, like host (cores, userland tools, hypervisor, drivers, kernel, ...), and running subsystems (containers, vm, ...)

Indeed this is a very important part I want to get right from the beginning.
First thing I think is to decide how are we going to publish what version of each component each "flavor" will have to run. Some kind of metadata service which all the node can query daily to know if there is some new goodies for them to run. Something equivalent to an apt-get update.
Then once we know there are some upgrade available, we could also have different upgrade procedure depending on the component. Some component which are 100% stateless can probably be upgrade immediately. While some other component might need to wait for a more "quite" time to upgrade since it would imply a small downtime.
Then we also need to define what is the medium we will use to get the new component on the node. Flist could be used. That would do something like:

discover upgrade are available
mount the flist component of the new version in a temporary location
copy the component in place to the filesystem
if required, restart the processes

Now doing it like this implies that at each boot, all the required component will have to be downloaded like that. Since all the new component won't be available in the kernel image.
Following this train of though that would mean we need to have some kind of installation procedure since we need to have a disk where to put all of these things.
In the first version we avoid this install step cause everything was included in the kernel image. But this force a hard reboot to update anything, so we don't want to go there again.

I'm still completely against installing something on disk to be used. This was the main feature from the beginning and I don't see any good reason to move on. This still provide the flexibility we have (no risk of root corruption except if memory is corrupted, always up-to-date image when booting, no disk needed for basic usage, ...).
But I think we can find a good mix between live environment (no setup) and version control.
The live image could contains a more restricted set of things, and get only minimal tools: hardware initialization, reaching internet, quering for available software.
It's already a requirement that if your node doesn't have internet, nothing will start anyway (it already works like this, if internet is not reachable, no service starts). I don't see any reason why downloading needed stuff at that time is a problem.
As soon as this stage is ready, the cycle you suggest can be put in place.

The problem with this way of working is that its kills the internet connection of farms of a certain size when all the machine boot at the same time for example.
 So what I'm thinking about is not really a full installation, but at least have a cache that can keep the binaries of the different module on disk. So at boot we can check if there is an update available. If not, we just re-use what's in the cache. If the cache is cold, then we re-download.

It's already a requirement that if your node doesn't have internet, nothing will start anyway (it already works like this, if internet is not reachable, no service starts).

This is indeed the case now, but I think we can improve on this a bit. Imagine a farm of nodes in a location where internet connection is not super stable. It could be that for a period of time, the internet is not reachable, but the LAN of the farm is working. I think that would be nice that all the services can already be started at this point. It could be there are some workload that doesn't requires internet that can already start running.

If we can put in place a real p2p system, bandwidth within the same LAN would not be a problem anymore.
If you want to reuse something already downloaded, we need to ensure all checksum etc. The idea downloading everything and not storing anything is to avoid any possible corruption/modification by downloading latest version each time.

I started computing everything that has been said here into specs: https://github.com/threefoldtech/zosv2/pull/32
I'll close the issue here to continue the discussion on the draft PR


Errors in Docs about "Migrate from sw-precache or sw-toolbox"

Library Affected:
None.
Browser & Platform:
It doesn't matter :)
Issue or Feature Request Description:
I am reading the docs about Migrate from sw-precache or sw-toolbox - sw-toolbox migrations, trying to migrate from sw-toolbox to workbox, but got undefined when I was trying with code snippets provided.
Then I found out that in the code snippets they are using workbox.router, not workbox.routing.

Thanks! This will be fixed in https://github.com/google/WebFundamentals/pull/6792


Add support for set-clipboard.

This change allows a program running in the terminal to set the clipboard via the OSC 52 escape code. This is useful with remote commands running under tmux.
To test, set the tmux configuration variable "set-clipboard" to "on", and select some text. The text will be copied to the primary clipboard.
The best documentation that I've found for this escape code is here:
http://invisible-island.net/xterm/ctlseqs/ctlseqs.html#h2-Operating-System-Commands

Checks fail due to bad merge on Cargo.lock. Will check tomorrow.

Updated the PR to fix Cargo.lock. I was briefly confused as to why the [root] section was gone, but that appears to be due to https://github.com/rust-lang/cargo/issues/4563.


Implement new Patch skin

font
color

no functionality changes

Sketch file Patch UI 150915

ok so can't assign to multiple people. So just gonna mention @imgrby here too & you guys can sort it out :)


Update README

We should have a high level description of the project in the README including some steps for deploying the project.

I've been trying for three days and I can't get this thing to run, I think it needs some environment variables or something. I can't get it to run locally or on Azure.
If someone could give me some guidance on where to put the settings for Cosmos DB, storage, etc. I can flesh it out into a doc. The WinGet.RestSource.Infrastructure project just looks like some kind of template with no information on what I need to spin up or where, although maybe that's just because I don't understand ARM templates :D

@jedieaston
@zacharp and I have been talking about this over the last couple of days. More guidance will be coming as soon as his current feature work is wrapped up. We've been talking about a "minimum" configuration to just stand-up a lightweight developer type environment as opposed to an enterprise grade configuration. The database choice can also be abstracted by an interface so other "database" technologies could be used.

@RDMacLachlan is about to make a PR with a PowerShell script and instructions for deploying the project. We still need to build a release pipeline so the .zip files can be published in a release. Optionally someone from the community might want to look at the PowerShell script and add some extra magic to download the latest released artifacts to make deploying even more simple.

This looks good. Any chance of getting instructions on local development? If not this will help anyway, but those Azure for Students credits only go so far :D

Hi @jedieaston,
As part of this PR, we did include documentation on how to manually create each object in Azure. While the script and manual creation will consume Azure credits, the manual guidance does outline the individual objects required to host a Windows Package Manager private repository.
@denelon - Do you think we could make use of the Azure Cosmos DB Emulator, and a webserver to host the private repository on-premises? Can we make use of a SQL Database?
Creating a Windows Package Manager private repository guidance:
https://github.com/microsoft/winget-cli-restsource/blob/da4168a61fdd0b35900831bd22338249ae886bf4/doc/new-private-repository-azure.md

@RDMacLachlan I believe there is an interface for the storage. One could build a SQL interface. I'm not sure about the Cosmos DB Emulator scenario. I am not aware of anything preventing that technology stack.


Scratch note sometimes auto-creates with bad frontmatter

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[x] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[ ] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[x] Publish.  External Site Publish
[ ] Markdown.  Markdown features, such as syntax support and features

Describe the bug
Scratch notes created with the 'Create Scratch note' command  sometimes have the title frontmatter value created without quotes. This leads to the build error value.replace() is not a function when running npx dendron publish build.
To Reproduce
Steps to reproduce the behavior:

Have scratch note config set to the following:

    scratch:
        name: ''
        dateFormat: y.MM.dd.HHmmss
        addBehavior: childOfCurrent

Use the command ctrl + shift + s to create a new scratch note
Press Enter to create the note.

Expected behavior
I expected the note to have a title frontmatter value encased in quotes.
Desktop (please complete the following information)

OS: Windows
Browser: n/a
Version: 0.80.2

Additional context
I have noticed that this error occurs most frequently when the title of the note has a leading zero. I haven't been able to reproduce it 100% of the time.
Dendron Log file
Please attach the output of > Dev:Dendron: Open Logs here
scratch-log-output.txt

Thanks for reporting the issue!
I was unable to repro this. Does this happen more frequently?

I was just able to repro. It happens most frequently between 12AM and 09:59AM

Hi
None of my frantmatters is created with quotes.
I also have this error when publishing: TypeError: value.replace is not a function.
I noticed that dendron doctor fixFrontmatter removes quotes where I have added them.
What is finally expected? Quoted titles or unquoted titles?
Here is the error I get when publishing:
 generating metadata for publishing...
⠼ error {"stack":"TypeError: value.replace is not a function\n    at encode (/home/bastien/notes/cwt/node_modules/stringify-entities/lib/core.js:7:17)\n    at encode (/home/bastien/notes/cwt/node_modules/stringify-entities/lib/encode.js:12:10)\n    at Object.serializeText [as text] (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/text.js:12:7)\n    at serialize (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/one.js:25:29)\n    at all (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/all.js:14:22)\n    at Object.serializeElement [as element] (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/element.js:33:13)\n    at serialize (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/one.js:25:29)\n    at all (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/all.js:14:22)\n    at Object.serializeElement [as element] (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/element.js:33:13)\n    at serialize (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/one.js:25:29)","message":"value.replace is not a function"}
{"level":50,"time":1644834671418,"pid":1875,"hostname":"PC-Curuba","name":"publish <cmd>","stack":"TypeError: value.replace is not a function\n    at encode (/home/bastien/notes/cwt/node_modules/stringify-entities/lib/core.js:7:17)\n    at encode (/home/bastien/notes/cwt/node_modules/stringify-entities/lib/encode.js:12:10)\n    at Object.serializeText [as text] (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/text.js:12:7)\n    at serialize (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/one.js:25:29)\n    at all (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/all.js:14:22)\n    at Object.serializeElement [as element] (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/element.js:33:13)\n    at serialize (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/one.js:25:29)\n    at all (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/all.js:14:22)\n    at Object.serializeElement [as element] (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/element.js:33:13)\n    at serialize (/home/bastien/notes/cwt/node_modules/hast-util-to-html/lib/one.js:25:29)","type":"Error","msg":"value.replace is not a function"}

I can confirm that missing title quotes is the issue.
I wrote the following script and after executing it, everything exports properly.
for f in vault/*.md
do sed -i -E "s/^title: ([^']*)$/title: '\1'/g" "$f"
done

To expand on this, this seems to be a larger and confusing bug:

It isn't just a scratch note issue but a Dendron issue with handling notes with names starting with 0 (zero).
Scratch notes that include 0 in the title will note be quoted properly
If you create any note, such as my.note.01234, it WILL include quotes
You can add quotes after the fact to a scratch note, but things can get strange if you try to run Dendron: Doctor -> fixFrontmatter, as it doesn't know how to handle notes starting with a zero. It seems to rename the title attribute of the note, and remove quotes!
If you purposely break a working note, such as my.note.01234 by renaming the title: '01234', to be title: 01234, you will run into the same weird behavior from Dendron: Doctor -> fixFrontmatter where it doesn't know what to do, and it will rename the title to a completely different value

To replicate the scratch behavior, simply make the note:
scratch.2<IP_ADDRESS>8

The title attribute will not add quotes, and Dendron: Doctor -> fixFrontmatter will get confused by it.
Can also replicate by making my.note.01234, removing quotes, and attempting to run Dendron: Doctor -> fixFrontmatter

Another potentially related bug, where Dendron doesn't know what to do with notes starting with 0 inside of title when using commands like Copy Note Link.
Example:

Note with title 09 (ex. my.journal.2022.03.09)
Try using Copy Note Link
Pasting the link will show that the Title wasn't properly copied, resulting in [[0|my.journal.2022.03.09]] instead of [[09|my.journal.2022.03.09]]

The trend seems to be that 0 at the start of a title causes weird behavior across multiple commands in Dendron

Hi, we added a fix towards this problem with Dendron 0.86. Dendron now always makes sure that the title is internally a string (not a number) to avoid this issue.
If you were experiencing this problem before, could you let us know if you keep experiencing it after updating?

The problem has been slightly fixed.
When creating a note with the name foo.01.02, where foo.01 does not currently exist, the note foo.01 has a title value without quotes, while foo.01.02 has a title value with quotes.

@raeyulca I see, does it still cause errors?

Yes, it still produces the value.replace is not a function error.


bug(pip cache): pip cache is not properly used

Description
WARNING: The directory '/home/envd/.cache/pip' or its parent directory is not owned or is not writable by the current user. The cache has been disabled. Check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.

Meet this problem again. We need to add unit test for this

Cannot reproduce it here. Can you share your process to reproduce?

Cannot reproduce now. Probably related to old development cache


A way to send Ctrl+C and other control characters

Specially for this like MicroPython and CircuitPython serial, where Ctrl+C is used to enter the REPL.
Btw, great web serial terminal, I love it!

Hi, thanks!
Recently started a circuit python project and thought the same. Will definitely add it as soon as I can take some time for it :)

Awesome, thanks!
I'm not too sure how to trigger it though. I can see the setting, so the update has been deployed and I must be loading it, but if I press the keys "ctrl"+"c" in the input field nothing happens? Even after clicking "send"?

Ahh, works in Chrome but not in Edge (macOS 12.1 x64, Edge v103.0.1264.5).


Code issues

When I was training the patch classifier, I ran into a problem. In the phc_model.py file, the local variable 'out_6' referenced before assignment

Hi, thank you for reporting the issue. I fixed the problem, it should work now :)

Hi, thank you for your reply. Can you share your pytorchtools.EarlyStopping file? I found that the one I downloaded from the internet is not the same as yours.

What do you mean share? It's already in the repository under early-stopping-pytorch/pytorchtools.py

I'm sorry, I didn't notice your update of this file. Thank you for sharing!


Parse CSV

Script to parse existing poll244.csv file into agency, location, and incident report tables.

Looks pretty good, once you or I make the changes we can have another round of reviews

Thanks for the review. I can make the changes.

Looks good, just change it to use viewport biasing

I've moved the viewport coordinates to config and now use Ben's get_agency_by_name changes to ensure case-insensitivity.

Looks good, once we know how many reports are not successfully geocoded and what to do with them we can merge

What is tests_backup?

@thomaslee1911 How do I run the script locally and see how many fail to geocode?

I moved the actual tests to test_backup, so test only has the script that parses the entire file. To run the script you should be able to just python manage.py test. It should output a count of geocode no-responses and a string with each of the failed addresses.

That seems like a hack^ Can we add a manage.py argument so we could run it like
`python manage.py parse_csv --file "<file name here">

At this point, the code looks pretty good. However, I figure that there are quite a few edge cases we may not be considering, so I will wait until the rest of the app is done before we merge (that way we can do a complete test).