andysprout3

made deletions of fortawesome devs to clear up package-lock in client; ran npm audit fix --force to clean up package-lock before posting PR..thx

Header/Footer initial addition styling: done.

sorry for all the commits..it was an all nighter

A commit for a change that you can test and validate, or a big fix is the right thing to do.  Less commits and you will have more to catch up if you have to restart.

@AndyAn7 overall looks great! I left few comments to clarify things. As this is how PRs are normally reviewed. You know your changes best so there may be a reason you did things the way you did. Do not expect me to be right necessarily.

K thank you!
Andrew M. An
(253) 414-5536

On May 18, 2022, at 10:12 AM, Samuli Jarvinen @.***> wrote:

@AndyAn7 overall looks great! I left few comments to clarify things. As this is how PRs are normally reviewed. You know your changes best so there may be a reason you did things the way you did. Do not expect me to be right necessarily.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Andrew M. An
(253) 414-5536
Begin forwarded message:

From: Framer Team @.>
Date: May 31, 2022 at 8:07:29 AM PDT
To: Andrew An @.>
Subject: Multiple Pages + Site Spotlight
Reply-To: @.***

Hey Andrew, Framer Sites launched last week! Everyone can now enjoy breakpoints, effects, links, styles, and the cms to design and publish incredible websites. We even made it to the top spot on Product Hunt. Thank you for the support!
If you haven’t had a chance to play around, we highly recommend the new tutorial. You’ll have your first site up and running in about 10 minutes.
Page Types
You might have seen that every new Framer project now shows you three new page types in the sidebar.
Web pages have a unique URL and user interface for setting up breakpoints and adding effects. Web pages can be linked and published.
CMS Pages are templates that pull data from the CMS and generate web pages for you.
Canvas Pages are for ideating and prototyping. You can insert prototypes on web pages and present them next to feedback forms.
Here’s a quick guide if you want to learn more.
Site Spotlight
Aaron Abentheuer and the team at Playground launched a new site for WorkLens, an intelligent productivity app for people who take photos to get things done. It’s full of cool hover and scroll effects and works flawlessly across all screen sizes.
“The site was built with the new Framer, and I’ll say it again: for me it’s hands down the future of building websites. At playgr_und we don’t even go to Figma anymore to design sites, we just start building straight away in @framer.” — Aaron on Twitter
Looking for inspiration? We’re adding new sites to the gallery every week.
That’s it for this week! Thanks for reading, and keep up the good work.
Cheers,
✌
Unsubscribe


HARP-7836: Add support for tiled GeoJson.

This change introduces an adapter that converts tiled GeoJson
to the internal respresentation used by omv-datasource.
Signed-off-by: Roberto Raggi<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request head (HARP-7836@7e23f7d). Click here to learn what that means.
The diff coverage is n/a.

Decoders tests please


Upgrade suggested Vagrant version?

The docs currently suggest developers use Vagrant 1.8.1 (December 2015). The current version is 1.8.4. According to the Vagrant changelog, there are breaking changes between 1.8.1 and 1.8.3.

Vagrant 1.8.1 is the version I tested with across all platforms. Before we update the recommended version, we should probably make sure 1.8.4 works as expected.

Vagrant 1.8.4 works as expected on OS X, Windows 10, and Ubuntu 16.04. I'd like to update the first-time contributor instructions for Ubuntu 14.04 and 16.04 to explicitly use this new vagrant version, particularly because it fixes the rubygems issue that previously required a manual patch.

Sounds good to me!


Terminal says "disabled by host in config" even when buying slots is not disabled

For some reason my terminal always says "disabled by host in config" even though I've enabled purchasing. I tried deleting all the reserved slot configs as well as starting a new save, but the message won't go away.
Honestly, I think those who disabled the purchasing of the slots shouldn't even see a "disabled" message on the terminal. It takes up a lot of space and the capital letters draw a lot of attention to it every time you open the terminal.

Seems to be fixed after I updated! I still think the disabled message isn't needed but I can confirm that the message for purchasing slots is displaying correctly now with purchasing set to true.


Deprecate repo:shortid syntax

fixes #22112
Per discussion in #22112 deprecate this little known feature. The minimum length requirement from v1.9 has also been restored.
@thaJeztah @duglin
Signed-off-by: Tonis Tiigi<EMAIL_ADDRESS>

LGTM

SGTM
/cc @stevvooe @aaronlehmann

Overall LGTM.
Not sure we should make a behavioral change now since we are deprecating it anyway. But I don't have a strong opinion.

@aaronlehmann It's just restoring the old behavior so that it is safer during the deprecation period and doesn't cause accidental collisions(for example if someone uses tags like foo:a1 etc.

@tonistiigi does the minimum length mean that I can no longer tag (or pull) an image with a tag v1, or only when using partial ID's?

@thaJeztah The length limit is only for the partial IDs, not tags.

@ncdc FYI

LGTM


[Prototype] Illustrate what generated polymorphic models should look like

An initial design proposal for polymorphic models was made in https://github.com/Azure/autorest.csharp/issues/2361, and discussed in the 08/11/2022 C# DPG design meeting.  This PR illustrates the result after making updates suggested in that meeting.  The specific commit containing the updates is here: https://github.com/Azure/azure-sdk-for-net/pull/30718/commits/36dcf11b1c067fcd4cc75fc3011ceaeb28a51e5e

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Template


"Open Files UWP here" crashes app everytime

Describe the bug
When I click on the "Open Files UWP here" in the context menu, the app crashes, every time...
To Reproduce
Steps to reproduce the behavior:

On my external hard-drive, open a folder location...
Right click to open the context menu...
Click on the "Open Files UWP here"...

Expected behavior
Files UWP should open at that location
Screenshots
If applicable, add screenshots to help explain your problem.
debug.txt
debug_fulltrust.txt

Desktop (please complete the following information):
OS-Ver: [Windows 10 Pro Insider Preview Build 20180.rs_prerelease.200725-1714]
AppVer: [v0.12.4 Preview] FROM THE STORE
Email:  [jonny<EMAIL_ADDRESS>Additional context
Add any other context about the problem here. For instance, are you building from source or using a precompiled build/snapshot?

@jonnypjohnston This method is no longer supported by Files.

@jonnypjohnston This method is no longer supported by Files.


PHP native functions

Hello, I managed to install this plugin on my Ubuntu (I had to install vim-gnome-py2 and remove vim-gnome because of python).
I am now able to complete standard objects (such as $this->generateRoute(/*...*/)) but native PHP functions are not autocompleted, it keeps saying Pattern not found:

How can I fix this?
Thanks and keep up the good job on this!

@lukepass this feature is only available in padawan.php#master, not in 0.2.1 version. So update your padawan.php

Hello and thanks for your quick answer. I tried:
composer global update mkusher/padawan:dev-master
But in my composer.json there is still version 0.21. How can I update it?

try
composer global require mkusher/padawan:dev-master

you'll need to update your minimum stability for global packages

I lowered the minimum stability but now I get this error:
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Can only install one of: react/http[dev-master, v0.4.1].
    - Can only install one of: react/http[v0.4.1, dev-master].
    - Can only install one of: react/http[dev-master, v0.4.1].
    - don't install react/http dev-master|don't install react/react v0.4.1
    - mkusher/padawan dev-master requires react/http dev-master -> satisfiable by react/http[dev-master].
    - Installation request for mkusher/padawan dev-master -> satisfiable by mkusher/padawan[dev-master].
    - Installation request for react/http (locked at v0.4.1) -> satisfiable by react/http[v0.4.1], react/react[v0.4.1].

Installation failed, reverting ./composer.json to its original content.

remove previous version of padawan and then install new one

Thanks, now I can complete native functions:

But unfortunately it doesn't work anymore on other variables :(

I already tried to restart server, restart vim and regenerate index.
Thanks.

oh, you have to re-run index generation

Hello,
I already tried to restart server, restart vim and regenerate index.
When executing PadawanUpdateIndex there's an error: Error occurred timed out

try PadawanGenerateIndex


MEGA on Cloud Storages (and an option to manually configure unrecognized drives)

What's the Problem?
Does not recognize Mega Sync folder.

Solution/Idea
If possible try to add this Storage automatically
Alternatives
It would be interesting to put an option in the settings to add unrecognized drives, where we could say the path, write a name and an icon.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
Versão: <IP_ADDRESS>
Windows Version
Windows 10 21H1 19043.1288
Additional comment
No response

Mega should be already detected.

Does mega appear in the left pane of windows explorer?
Could you upload the app's log files?

@yurnasg can you open a new feature request to allow users to manually configure unrecognized drives?

Mega should be already detected.

Does mega appear in the left pane of windows explorer?
Could you upload the app's log files?

Mega is not showing up in windows explorer either

debug.log

@yurnasg can you open a new feature request to allow users to manually configure unrecognized drives?

Yes! I can. This would also be a manual solution for other Issues I created (https://github.com/files-community/Files/issues/6795) and it opens up more possibilities for customization

Mega is not showing up in windows explorer either

Well, that explains why it does not appear in Files. I'm closing this in favor of the feature request for manual drives customization.


Def, providecommand, etc not working

I'm using the cdn to load the latest (it currently loads 5.3) MathJax, and when trying to use def or providecommand in inline math mode it just turns red. For instance:
$\def{\NN}{\mathbb N}$ and then $\NN$ results in both math items being rendered as red error text.
This can also be seen on StackEdit.io.

Your \def command uses incorrect syntax: there should not be braces around the control sequence being defined.  If you use $\def\NN{\mathbb N}$ then it should work.  (You can use braces with \newcommand, but not \def.)  So I'm marking this as "expected behavior".

Ok, thanks for the response.
Just for clarity, is providecommand not supported? (I was getting errors
with this using either syntax.)
Michael Boratko
On Dec 30, 2015 1:26 PM, "Davide P. Cervone"<EMAIL_ADDRESS>wrote:

Your \def command uses incorrect syntax: there should not be braces
around the control sequence being defined. If you use $\def\NN{\mathbb N}$
then it should work. (You can use braces with \newcommand, but not \def.)
So I'm marking this as "expected behavior".
—
Reply to this email directly or view it on GitHub
https://github.com/mathjax/MathJax/issues/1344#issuecomment-168049361.

is providecommand not supported?

Correct.  Also, the LaTeX commands always act like \def, so they don't check whether the macro is defined or not, and never produce errors.


Simplelife.build() error with latest Modelx version

Hi,
I updated modelx to the latest version and tried to incorporate the new "new_excel_range" method and was getting errors. I resorted to rerunning the basic Simplelife project I had (which previously worked) that now gives an error
"File "C:\Demosthenis_PythonRelated_Anaconda\lib\site-packages\modelx\core\base.py", line 419, in new
raise ValueError("Invalid direct constructor call.")"
lifelib version =  '0.0.14'
modelx version = '0.9.0'
Here are the order of the commands (with output):
import simplelife
model = simplelife.build()

Traceback (most recent call last):

  File "<ipython-input-102-f7600826e8fb>", line 1, in <module>
    model = simplelife.build()

  File "C:\Demosthenis\_PythonRelated\ModelX\LifeLib\mylife\simplelife.py", line 37, in build
    model = mx.new_model(name='simplelife')

  File "C:\Demosthenis\_PythonRelated\_Anaconda\lib\site-packages\modelx\core\api.py", line 101, in new_model
    """

  File "C:\Demosthenis\_PythonRelated\_Anaconda\lib\site-packages\modelx\core\system.py", line 489, in new_model
    self.currentmodel = ModelImpl(system=self, name=name)

  File "C:\Demosthenis\_PythonRelated\_Anaconda\lib\site-packages\modelx\core\model.py", line 362, in __init__
    Impl.__init__(self, system=system, parent=None, name=name)

  File "C:\Demosthenis\_PythonRelated\_Anaconda\lib\site-packages\modelx\core\base.py", line 202, in __init__
    self.interface = self.interface_cls(self)

  File "C:\Demosthenis\_PythonRelated\_Anaconda\lib\site-packages\modelx\core\base.py", line 419, in __new__
    raise ValueError("Invalid direct constructor call.")

ValueError: Invalid direct constructor call.

import lifelib

lifelib.__version__
Out[104]: '0.0.14'

mx.__version__
Out[105]: '0.9.0'

I couldn't replicate the error. Can you zip C:\Demosthenis\_PythonRelated\ModelX\LifeLib\mylife and attach it if no private information is held in it.


Fix command_exists()

type pbcopy returns nothing, when called from clipboard_copy_command().
which pbcopy and command -v pbcopy work as expected, returning
path to the executable.

type is a bash builtin command http://ss64.com/bash/type.html
Are you sourcing it with sh ?

@netei:
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin14)
Copyright (C) 2007 Free Software Foundation, Inc.

GNU talks about type in the bash release 2.0.2, http://tiswww.case.edu/php/chet/bash/CHANGES
So I don't know what is wrong with your type but it seems weird.

Hey guys,
type is in the posix standard, link. It should be safe to use. Also, no one had issues with this so far.
From what I know using which is discouraged, this stackoverflow answer talks more about it.
@vvv, can you please investigate why type is not working for you and report here? It might be useful for other users with the same problem.

@vvv What exactly are your symptoms? Do they still exist?

I don't remember, what was this issue about. Closing.


Calculated Columns do not extract properly

Which PnP repository to report the issue?

PnP PowerShell - https://github.com/OfficeDev/PnP-PowerShell

You can delete this section after you have confirmed target repository for your submission.
Category
[ x] Bug
[ ] Enhancement
Environment
[ x] Office 365 / SharePoint Online
If SharePoint on-premises, what's exact CU version:
Expected or Desired Behavior
Calculated Columns with Publishing Start/End date to not extract properly / do not provision
Observed Behavior
export behavior - the calculated column extracts as follows:

=DATE(YEAR([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}])+1,MONTH([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}])+6,DAY([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}]))

it should extract like:

=DATE(YEAR([Scheduling Start Date])+1,MONTH([Scheduling Start Date])+6,DAY([Scheduling Start Date]))

if i extract/add w/o manually fixing the XML, the field is corrupted.
Steps to Reproduce
Create a field with a calculated column
Get-SPOProvisioningTemplate -out $tpFields -Handlers Fields -Force
Apply-SPOProvisioningTemplate -out $tpFields -Handlers Fields -Force

Hi! Thank you for reporting this. Very good catch. I just merged a fix to this in the dev branch and I will close this issue.

i have downloaded the latest code - this is the information i see when exporting a Field.
=DATE(YEAR([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}])+1,MONTH([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}])+6,DAY([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}]))

Thanks ! i just tested this and it appears it works correctly for SiteCollection Columns

=DATE(YEAR([Scheduling Start Date])+1,MONTH([Scheduling Start Date])+6,DAY([Scheduling Start Date]))

but... for content type hub columns, it appears to have the old formula

=DATE(YEAR([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}])+1,MONTH([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}])+6,DAY([{fieldtitle:[{fieldtitle:[{fieldtitle:PublishingStartDate}]}]}]))


tune: support for conditional / nested / hierarchical parameter search spaces

This is not documented within the tutorial, so I assumed that, most likely, it's not directly supported. However, I see that there is some reference to hierarchical spaces in the api docs and a discussion about it in #717. I don't understand exactly the code samples there, as I focused only on the black box optimization part, and ignored the AutoML part, but I assume that in AutoML, if you need to choose an alg and then tune it, and every algorithm has specific parameters, than you kinda need nested search spaces.
Also, I think the constraints api can be used to mark incompatible configurations as invalid. I'm not sure, though, how that affects tuning performance. I haven't taken the time to properly understand the BlendSearch algorithm and how it trades off local with global search, but I imagine that the bayesian model underlying at least the global search (that must be a Gaussian Process, or something, right?), that serves as a predictor for the performance which would be attained by a certain config before running it, would have some sort of smoothness prior. Now, if using constraints assigns very low scores to invalid configurations, due to this smoothness prior, wouldn't it also assign low scores probs to the configs at the edge of feasibility? I imagine that there might be very good combinations right at the edge of the feasible region, but they will be explored later rather than sooner, and, although kinks can be modeled, they are still considered less likely than smooth regions.
So, in short, is it safe to use the relational constraints to implement conditional search spaces? Is there another, better way to let the optimizer know that a particular combo is not feasible?
Also, I know that some other black box optimization packages, e.g., HyperMapper train a separate classifier model for guessing whether a certain config is feasible or not. Do you guys use such a model to drive sampling, too?
I mean, it has the advantage that you don't have to specify the invalid space up front, which is nice if you don't know the invalid combinations a priori, and you want the optimizer to learn this by itself, but this is not very helpful if you know the invalid configs and they can be expressed through conditional rules, and want to inject this info into the model so it doesn't waste time trying out bad combinations.
So, can I define conditional search spaces and, if yes, how?
Also, It wouldn't hurt to make a reference in the tutorials about this, too, even if it's something like "we don't support this right now".
L.E.: now I can see that there's this is_valid method of Domain objects. Can (and should) the user override this function somehow to specify arbitrary valid combinations?

This is not documented, so I assume that, most likely, it's not directly supported. However, I see that there is a discussion about it in #717.
Also, I think the constraints api can be used to mark incompatible configurations as invalid. I'm not sure, though, how that affects tuning performance. I haven't taken the time to properly understand the BlendSearch algorithm and how it trades off local with global search, but I imagine that the bayesian model underlying at least the global search (that must be a Gaussian Process, or something, right?), that serves as a predictor for the performance which would be attained by a certain config before running it, would have some sort of smoothness prior. Now, if using constraints assigns very low scores to invalid configurations, due to this smoothness prior, wouldn't it also assign low scores probs to the configs at the edge of feasibility? I imagine that there might be very good combinations right at the edge of the feasibility region, but they will be explored later rather than sooner, and, although kinks can be modeled, they are still considered less likely than smooth regions.
So, in short, is it safe to use the relational constraints to implement conditional search spaces? Is there another, better way to let the optimizer know that a particular combo is not feasible?
Also, I know that some other black box optimization packages, e.g., HyperMapper train a separate classifier model for guessing whether a certain config is feasible or not. Do you guys use such a model to drive sampling, too?
I mean, it has the advantage that you don't have to specify the invalid space up front, which is nice if you don't know the invalid combinations a priori, and you want the optimizer to learn this by itself, but this is not very helpful if you do have the invalid combos.

Is there perhaps an example of how to use hierarchical / nested search spaces that I can use until you guys are done with the docs?

Is there perhaps an example of how to use hierarchical / nested search spaces that I can use until you guys are done with the docs?

hierarchical: https://github.com/microsoft/FLAML/blob/5eb99276423f8ac88b7fe38c37d9b7f37fafe4fb/test/tune/test_space.py#L47
nested:
https://github.com/microsoft/FLAML/blob/5eb99276423f8ac88b7fe38c37d9b7f37fafe4fb/test/tune/test_tune.py#L221

how about more advanced conditional search spaces than a plain choice (e.g., similar to #988)?


Value of the attribute iss of the Proof of key ownership in the credential request must be the value of client_id

if secret_post or secret_basic is chosen as client authentication method, the client_id must be the value entered by user (or value of the parameter of the configuration file of enterprise)
see OIDC4VCI Test 8 GAIN POC and try using secret_post or secret_basic

@ThierryThevenet This rule is global or just for draft 13?

@bibash28 That's for draft 11 and draft 13


Comparison onboarding/minimal-landing-zone procedures/artifacts/security-control-coverage for cloud.google.com/onboarding

Purpose: Run a clean org install and minimal lz running the enterprise checklist from below and do a subset coverage analysis with this current full LZ V2

see https://cloud.google.com/onboarding
launch https://console.cloud.google.com/cloud-setup

see previous run 10 month ago
https://github.com/CloudLandingZone/google-cloud-enterprise-setup-checklist/blob/main/onboarding.md
Questions

TF delete
TF adjust
upstream source for bug/fix tracking pulls and/or PRs
full LZ delete including non-TF console orchestrated artifacts (click-ops in console - as well as solicited user (admin) actions (adding users to groups for example)
fully automated (no click ops) parameterized json/sh input of per/org params for multi-org setups
option for full gcloud install - for everything including groups - where the client does not need IaaC, drift or change tracking

Get a new domain and org first

optionally use an existing empty org

## start checklist

Start the checklist by navigating to IAM | Identity & Organization

https://console.cloud.google.com/iam-admin/iam?organizationId=471924274947
https://console.cloud.google.com/iam-admin/org-onboarding?organizationId=471924274947
goto checklist
https://console.cloud.google.com/cloud-setup?organizationId=471924274947

step 2

Create all groups

Create admin users in the admin console
For the purpose of this onboarding checklist, we recommend that you initially add users who will participate in the checklist tasks, such as administrators (organization, network, billing, etc) and decision makers involved with your cloud setup practices. You can continue the rest of the checklist without adding all of your cloud users. At a later time, once you've set up your foundation using this checklist, we recommend adding all users. Follow these instructions  to learn about federating and provisioning users into Google Cloud.
Google Admin console is a central place for admins to manage user identities and accounts across Google services, such as Workspace, Google Cloud, Maps, etc.
Instructions:
You will need to sign into the Google Admin console with the Super Administrator account that you created in Task 1: Organization.
Once you have added your admin users, return to this tab and click Continue to complete the users and groups setup.

Manual procedure

not required in my case - as single admin already in

2: Add your administrative users to groups

no action required

There appears to be only one member in some of these groups. Do you wish to proceed anyway?

In this step, you add members to the admin groups you created earlier.The group creator will always be added as a group member at creation time. We recommend that you have more than one member per group.Instructions:Add members to your groups by clicking Add members in the table below.Repeat this for each group listed.You can also manage group members from the Groups page in the Cloud Console.
Group | Members |   |
-- | -- | -- | --
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
<EMAIL_ADDRESS>| 1 | ADD MEMBERS |
harset='utf-8'>In this step, you add members to the admin groups you created earlier.The group creator will always be added as a group member at creation time. We recommend that you have more than one member per group.Instructions:Add members to your groups by clicking Add members in the table below.Repeat this for each group listed.You can also manage group members from the Groups page in the Cloud<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>MEMBERSIn this step, you add members to the admin groups you created earlier.
The group creator will always be added as a group member at creation time. We recommend that you have more than one member per group.
Instructions:
Add members to your groups by clicking Add members in the table below.
Repeat this for each group listed.
You can also manage group members from the Groups page in the Cloud Console.
Group
Members
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1
<EMAIL_ADDRESS>1

4 Billing
however billing is already enabled with extra quota granted previously - in anticipation of more than 5 projects

gui issue

5 - Hierarchy & access

What to expect in these next few steps
Use our best-practice defaults or choose to customize
The next three steps take you through Resource Hierarchy, IAM, Networking, and Centralized Logging setup. You can accept our defaults or choose to customize further to meet your needs.
Processing requirements
Google Cloud will create a project and enable the Cloud Storage API and Cloud Infrastructure Manager API .
Google Cloud will also assign the following roles to the Organization Admin group. You can always remove them after deployment.
Cloud Storage Admin on the project
Cloud Infrastructure Manager Admin on the project
Project Billing Manager at the project level
Service Account Admin at the project level
Service Usage Admin on the project
Groups Admin of the Google Workspace
Learn more  about the roles assigned to the Organization Admin group.


WIP: replace libc with std::os::raw

This doesn’t work yet. I’m getting errors like:
 expected `*mut libc::types::common::c95::c_void`,
    found `*mut std::os::raw::c_void`

I think one of the dependencies is still using libc, but at this point it’s one yak too many to shave for me right now :) Feel free to take this up or disregard it.

This is very strange that you get this one if you use wayland via glutin. This is code from a version that is more recent than the one used by glutin (0.3 vs 0.2).
I believe the culprit is actually https://github.com/vberger/wayland-window/blob/master/Cargo.toml#L16
I'm going to correct it.

I published wayland-window 1.1 fixing this. Is it better ?

It’s possible that this crate is not actually built in servo. But because of how Cargo works, even optional dependencies end up in Cargo.lock which is where I looked for any occurrence of libc 0.1.

I'll take some time to figure out if I can get rid of libc or not at some point, but I doubt it's actually possible.
Closing this for now.


Disable error-rescue feature

Closes #138

@jansorg I'm getting the same undefined reference error on master
I just pushed up a separate pr that fixes that: #141


Update safe_app to avoid webview crash if the experimental APIs are disabled

If the experimental APIs is disabled in the browser, trying to access the safeApp.web experimental APIs from the devTools console, the devtools get disconnected form the page and a blank page is rendered in the current tab. I tried to reproduce it from the web API playground tool but I wasn't able to, it seems to be only when I instantiate a safeApp from the devTools console with:
window.safe.initialiseApp({
    id: 'net.maidsafe.test.webapp.id',
    name: 'WebApp Test',
    vendor: 'MaidSafe Ltd.'
  }, (newState) => {
    console.log("Network state changed to: ", newState);
  }).then(app => safeapp = app)

and then
safeapp.web.

This will be solved via updating safe_app version with fixes from: https://github.com/maidsafe/safe_app_nodejs/pull/320

QA verified


PHP warning in Tracy after filtering without pagination

I have datagrid without pagination - setPagination(FALSE). After filtering, e.g. in text column or date range column - Nette\Forms\Container::getValues() invoked but the form is not valid - is shown in Tracy.
datagrid 6.8.0, nette/forms 3.1.3

Thx, https://github.com/contributte/datagrid/releases/tag/v6.8.1


贵团队是否会升级长上下文的版本？

目前的上下文长度是4096，接下来是否会出长上下文的版本？

Currently, both the API and the official website support a version with a 32K context limit. If you wish to experience a longer context, you can do so at these two locations.


Bump rake to non-vulnerable version

This is probably pretty low priority, but I'd like to silence the
github security advisory emails so they're out of the ordinary and not
just filtered out subconciously.

Thanks!


QBit failure with multiple http verb support method={RequestMethod.POST,RequestMethod.PUT}

https://gist.github.com/revelfire/24e381c5bb12056d42f3

Resolved by pull request #259. (Please review my todo comment and let me know if an additional check needs to be implemented.)

Will do.
On Sunday, June 7, 2015, revelfire<EMAIL_ADDRESS>wrote:

Resolved by pull request #259
https://github.com/advantageous/qbit/pull/259. (Please review my todo
comment and let me know if an additional check needs to be implemented.)
—
Reply to this email directly or view it on GitHub
https://github.com/advantageous/qbit/issues/249#issuecomment-109802241.

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower


Allow encoding A with Show as plain text

Fixes #560.

Current coverage is 79.20%

Merging #561 into master will decrease coverage by -0.13% as of 18287fb

Powered by Codecov. Updated on successful CI builds.


adds wget command

Didn't notice this before, but you need to update docs/content/install.md as  MANUAL.md  is auto generated from it.
Thanks
Nick


incompatible version error after moving directories

I had to move the valheim server to a different disk as it was getting full. I did that through portainer by just changing the /config location but now when i try to connect to the server i am getting an incompatible version error.
Jul 15 16:31:43 /supervisord: valheim-server 07/15/2021 16:31:43: Peer<PHONE_NUMBER>1xxxxxx  has incompatible version, mine:0.156.2 remote<EMAIL_ADDRESS>

So, mine:0.156.2 remote<EMAIL_ADDRESS>means that the server doesn't have ValheimPlus loaded but the client does. The @<IP_ADDRESS> is the V+ version.

Closing for inactivity

i had same problem with it , but I've check V+ is loaded . It's still give  " incompatible  version with  mine:0.156.2 remote<EMAIL_ADDRESS>" , Is any other way to solve it?
Aug 10 19:08:28 /supervisord: valheim-server [Info   :   BepInEx] Detected Unity version: v2019.4.24f1
Aug 10 19:08:28 /supervisord: valheim-server [Message:   BepInEx] Chainloader ready
Aug 10 19:08:28 /supervisord: valheim-server [Message:   BepInEx] Chainloader started
Aug 10 19:08:28 /supervisord: valheim-server [Info   :   BepInEx] 1 plugins to load
Aug 10 19:08:28 /supervisord: valheim-server [Info   :   BepInEx] Loading [Valheim Plus <IP_ADDRESS>]
Aug 10 19:08:28 /supervisord: valheim-server [Info   :Valheim Plus] Trying to load the configuration file

I had to reinstall the server unfortunately.
On Tue, Aug 10, 2021 at 12:14 PM Niguai @.***> wrote:

i had same problem with it , but I've check V+ is loaded . It's still give
" incompatible version with mine:0.156.2 remote @.*** " , Is
any other way to solve it?
Aug 10 19:08:28 /supervisord: valheim-server [Info   :   BepInEx] Detected Unity version: v2019.4.24f1
Aug 10 19:08:28 /supervisord: valheim-server [Message:   BepInEx] Chainloader ready
Aug 10 19:08:28 /supervisord: valheim-server [Message:   BepInEx] Chainloader started
Aug 10 19:08:28 /supervisord: valheim-server [Info   :   BepInEx] 1 plugins to load
Aug 10 19:08:28 /supervisord: valheim-server [Info   :   BepInEx] Loading [Valheim Plus <IP_ADDRESS>]
Aug 10 19:08:28 /supervisord: valheim-server [Info   :Valheim Plus] Trying to load the configuration file
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lloesche/valheim-server-docker/issues/381#issuecomment-896248443,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYFQNNQV2B6L4D2P2XD7HDT4F3DDANCNFSM5AOQDCJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
Anish Mehta

@dtcuniguai do you by any chance have enforceMod=false configured? This would make V+ not report its version. If enforceMod is turned off on the server then the client MUST NOT have V+ running. No idea why it's built that way. Maybe a question for the V+ devs.

@lloesche yes，here is my configured use
[https://imgur.com/QG8H4n0](my configure)

@lloesche configured "enforceMod=true" It really works, Thanks!!


Add urlescape to default filters

It'd be great if urlescape was included in the list of default filters, since it's so commonly needed. Here's a patch that passes all the test cases.

If you like this, I can also add:
def datetimeformat(value, format='%d %B %Y %H:%M'):
    return value.strftime(format)

and
def skip(s, skip):
    return (k for k in s if k != skip)

Let me know if it's worth writing pull requests for them.

For a long time I was against url encode filters since they are encoding specific but now that everybody agrees on utf-8 as URLs it's probably fine. However I would love to see the filter accept dictionaries as well and automatically encoding them correctly.  Also one name for the filter is enough :-)
Lastly: needs tests.

It has tests! What's wrong with them?

What's wrong with having multiple names, especially when the function-name is so hard to remember like this one? It just makes it easier to use.
Nonetheless, I have removed them and implemented your other difficult requests. It feels a bit like a trial by fire. How's it look now?

It has tests! What's wrong with them?

Not sure how I missed them. Never mind then, but improved tests are always awesome :-)

What's wrong with having multiple names, especially when the function-name is so hard to remember like this one? It just makes it easier to use.

Because filters are intended to be sensible defaults to hook others in and especially with urlencode it would be awful if someone who wants a different encoding behavior would have to override all the aliases for it as well. Especially if new aliases are introduced later.  We only use aliases for core functionality that you don't want to change or is changeable from a separate API hook (for instance the autoescaping mirrors to the |safe, |e and |escape filters).
It is encouraged to alias filters as necessary in your own application.

It feels a bit like a trial by fire.

I'd rather have my contributors go the extra mile for a better final result than to pull things I regret later. Learned that the hard way.

How's it look now?

I pulled it but had to rework the implementation for Python 3 compatibility.

def datetimeformat(value, format='%d %B %Y %H:%M'):

In theory I'm all for that but the problem is that strftime on Python 2.x is not unicode safe. Also it does not work for dates before 1900. That's the reason why this was never added and we documented the use of babel instead.

def skip(s, skip):

What's the use case for this? Not that I'm against such a filter but it seems fairly specific.

Sorry about the Python 3 issues -- didn't think to test. And will try
to be better about style next time. Do you just run through pep8?
Your babel point makes sense and I'll submit skip if I find myself
using it a few more times.


Update executable tests

This PR updates the executable tests with feedback from @vickumar1981 in #5.

👍  LGTM


syntax error in PyPI version

when installing bigstream via pip install bigstream, a syntax error is thrown every time one tries to reach the bigstream package.
SyntaxError: expected ':' (piecewise_align.py, line 480)

when investigating the line in question (piecewise_align.py:480) it seems PyPI has a different version of bigstream than currently on github. Manually adding the : to line 480 resolves the issue.
PyPI piecewise_align.py:470-489

def old()                  # <--- this is line 480

    # set working copies of moving data
    if static_transform_list is not None:
        current_moving = apply_transform(
            fix, mov, fix_spacing, mov_spacing,
            transform_list=static_transform_list,
        )
        current_moving_mask = None

Yes thanks for documenting this!
I have not set up any continuous integration so PyPI often lags behind the github repository. I'm not an expert in CI/CD so it will be some time before I've been able to learn how to set it up. In the meantime I suggest to anyone having trouble with the PyPI package to install from source:
make sure the conda environment you want is active
clone the repository
from inside the repository directory run:
pip install ./
Or, if you want to hack the code more easily:
pip install -e ./

If its alright with you I'm going to close this issue since I think you've gotten bigstream working for you now?


AndroidTarget: Add boolean 'state' to airplane mode intent broadcast

Recently I've been seeing some errors with enabling airplane mode on
hikey960 - firstly I get a 'broken pipe' error followed by a 'cmd:
Can't find service: settings'. This missing boolean was the first
thing I found that had changed wrt. the code I used to use, and
adding it back appears to fix the issue.

LGTM


session not stored?

I've tried use session to store some data. But it seems data are not stored among different requests. Is it implementation issue or usage issue?
A simple use case is:
package main
import (
    "github.com/gofiber/fiber"
    "github.com/gofiber/session"
    "github.com/gofiber/session/provider/sqlite3"
)

func main(){
    app := fiber.New()

    provider := sqlite3.New(sqlite3.Config{
        DBPath: "session.db",
        TableName: "session",
    })
    sessions := session.New(session.Config{
        Provider: provider,
    })

    app.Get("/get", func(c *fiber.Ctx) {
        store := sessions.Get(c)
        defer store.Save()
        id,ok := store.Get("id").(int)
        if ok {
            c.Send("id=",id)
            store.Set("id",id+1)
        }else{
            c.Send("id empty, in session:",store.ID())
            store.Set("id",1)
        }
    })

    app.Listen(8060)
}

And when use browser to reload "http://url/get", I expect the id to be increment while session ID keeps the same.
Actual result is id always empty, and session ID keeps the same.
I also tried redis provider, the result is the same.
Am I use the API correctly? Or is there anything wrong with the implementation?

If you try id++ before calling Set, does that work?

It seems the code never successfully read "id" from the session, although it always try to set "id=1" to the session.
I've changed the code after store.Get to this:
        if ok {
            c.Send("id=",id)
            id++
            store.Set("id",id)
        }else{
            c.Send("id empty, in session:",store.ID())
            store.Set("id",2)
        }

The result is still the same, like this:

id empty, in session:97cd65b7-dd94-42e1-955c-c8dfaf4d8549

What's wrong with it?

Do you have the same behaviour using the default provider? It could be a MySQL problem

The provider is sqlite3.
It always keep the same behavior. For instance, when access http://url/get, the response is always like this(session id may be different):
id empty, in session:9c03b735-88e8-4bba-b50e-884b14109a0c
And in the session.db, there is a record with the same session id:
sqlite> select * from session; 9c03b735-88e8-4bba-b50e-884b14109a0c|gaFEkYKjS2V5xAJpZKVWYWx1ZQE=|1593387204498416100|43200000000000

I can't seem to reproduce the same behaviour 😞 could you try to update to the latest Fiber & Session packages?

@tgarm May I ask the operating system with which you are using the sqlite3 provider?

Thank you for your reply.
It runs on Ubuntu Server 18.04.4 LTS /amd64, and the version of sqlite3 is: 3.22.0-1ubuntu0.4
Version of Go is: go1.14.4 linux/amd64
I just upgraded go packages with go get -u, it shows these versions:
go: github.com/gofiber/fiber upgrade => v1.12.3
go: github.com/mattn/go-sqlite3 upgrade => v1.14.0
go: github.com/gofiber/session upgrade => v1.2.3
The problem keeps the same.

The provider is sqlite3.
It always keep the same behavior. For instance, when access http://url/get, the response is always like this(session id may be different):
id empty, in session:9c03b735-88e8-4bba-b50e-884b14109a0c
And in the session.db, there is a record with the same session id:
sqlite> select * from session; 9c03b735-88e8-4bba-b50e-884b14109a0c|gaFEkYKjS2V5xAJpZKVWYWx1ZQE=|1593387204498416100|43200000000000

you need to check if ok is return true, and value in session.db is changed (not session id)

Hi @fuadarradhi, thanks for your comment. 😊
Can you confirm this, @tgarm? Thanks!

Thank you for your concern!
ok is returned false.
As for the field values in session.db:

Before first access, no row exists
After first access, a row with the session id is created
After each following access, field last_active increases while field data and expiration always keep the same value.

@tgarm, could you try if this is fixed in https://github.com/gofiber/session/releases/tag/v1.2.4?

I've upgraded session to 1.2.4 and the result is still the same:
nothing changed in sqlite database file session.db
One thing different: Sometimes, when quit and re-rerun the go code, the following error message shows:
2020/07/23 11:44:25 session error: session: sqlite3 index last_active already exists; provider: sqlite3;
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x40 pc=0x88b135]

goroutine 1 [running]:
github.com/fasthttp/session/v2/providers/sqlite3.(*Provider).NeedGC(0x0, 0xc0000be160)
	<autogenerated>:1 +0x5
github.com/fasthttp/session/v2.(*Session).SetProvider(0xc00015e000, 0xae9020, 0x0, 0x1, 0x1)
	/home/simon/go/pkg/mod/github.com/fasthttp/session/v2@v2.2.0/session.go:64 +0x4a
github.com/gofiber/session.New(0xc0004d1ea8, 0x1, 0x1, 0x0)
	/home/simon/go/pkg/mod/github.com/gofiber/session@v1.2.4/session.go:148 +0x4e2
main.main()
	/home/simon/modtest/go/min.go:17 +0x18d
exit status 2

I just released https://github.com/gofiber/session/releases/tag/v1.2.5, this contains a fasthttp session update. Sorry to ask again, but could you try this version?

Finally, after trace into the library,  I found it's because session store does NOT support type int.
It works after changed variable type to string, like:
id,ok := store.Get("id").(string)
if ok {
    iid,err := strconv.Atoi(id)
    if err!=nil {
        fmt.Println(err)
    }
    store.Set("id",strconv.Itoa(iid+1))

@Fenny  Thank you for your concern and help.
Would you please update the README document to reflect this?

Hi @tgarm,
That's odd, because I have been playing with the session middleware as well and integers work just fine for me.
However, I've only been using the redis provider. I will test this behavior with SQLite3 some time soon.
I'll re-open the issue to confirm this behavior, thanks for your response! 🚀

github.com/gofiber/fiber v1.13.3
github.com/gofiber/session v1.2.5
still have the problem
add some testes on  source code, may be better.

@sxci, we opened an issue about the int not being support on fasthttp's end. ( int64 seems to work fine )

add some testes on source code, may be better.
I personally maintain around 32 repo's in my spare time, giving back to the community by submitting PR's is highly appreciated 👍

Also, I would like to mention the issue at FastHTTP's repo; https://github.com/fasthttp/session/issues/29

Just checked, int64 works.

I'll leave this issue open for now, once we get an update on fasthttp/session#29

I'm closing this issue, for clarification please see https://github.com/fasthttp/session/issues/29#issuecomment-672286113


unable to start Springy Store

Thanks for this wonderful project, can you please resolve this issue as no data is been fetched from the store api

@nvkforl Sorry for the late response, but it works fine from the pictures, and all test cases pass successfully.
I intentionally testing the APIs when there is no data to be fetched from API, so check about error messages.

@nvkforl Sorry for the late response, but it works fine from the pictures, and all test cases pass successfully.
I intentionally testing the APIs when there is no data to be fetched from API, so check about error messages.


Add JSON or JS Object support for defining menus

Many JS apps that would need a context menu have fully dynamic DOM elements which constantly change.  It would be nice to be able to simply define the menu options as a JS object right in the JS code!

Hi, thanks for your comment.
In my opinion data should always remain in HTML, unless it needs processing or should otherwise be dynamic. I can however create a branch and would be happy to accept a pull request if you implement the changes you need!
Cheers.


Fix soft delete check, null check

Change Summary

Fixes @manavo 's comments in https://github.com/typesense/laravel-scout-typesense-driver/pull/15#pullrequestreview-822556807

Looks good to me

@hi019 can we tag a new release with these changes? Thanks!

Yep, will do it later today

Thanks!

Released

Thanks!

By the way, would you be interested in becoming a maintainer of this repository @manavo ?

Thanks for the offer @hi019, really appreciate it! Unfortunately it's not something I can commit to at the moment, but if I have more time in the future would love to discuss again!

Sounds good!


Error" Could NOT find Protobuf (missing: Protobuf_LIBRARIES)"

Hi
I encounter some issues when compiling veloview source code according to the developer guide.
step 1,2,3,4,5 is completely the same as the guid.
step 6 command is cmake "D:\VeloView-source\Superbuild" -GNinja -DCMAKE_BUILD_TYPE=Release -DUSE_SYSTEM_qt5=True -DQt5_DIR="D:/Qt/Qt5.14.1/5.14.1/msvc2017_64/lib/cmake/Qt5".
my question is:
1.my Qt version is 5.14.1 and not 5.10,and msvc is 2017. I wonder if mismatched version will result in error,or must use the same version as the developer guid
2.when i execute ninja,error message show "CMake Error at D:/Program Files/CMake/share/cmake-3.17/Modules/FindPackageHandleStandardArgs.cmake:164 (message):
Could NOT find Protobuf (missing: Protobuf_LIBRARIES) (found version
"3.12.3")
Call Stack (most recent call first):
D:/Program Files/CMake/share/cmake-3.17/Modules/FindPackageHandleStandardArgs.cmake:445 (_FPHSA_FAILURE_MESSAGE)
D:/Program Files/CMake/share/cmake-3.17/Modules/FindProtobuf.cmake:626 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
VTK/CMake/vtkModuleMacros.cmake:888 (find_package)
ThirdParty/protobuf/CMakeLists.txt:40 (vtk_module_third_party)
-- Configuring incomplete, errors occurred!
See also "D:/VeloView-build/common-superbuild/paraview/build/CMakeFiles/CMakeOutput.log".
See also "D:/VeloView-build/common-superbuild/paraview/build/CMakeFiles/CMakeError.log".
CMake Error at D:/VeloView-build/common-superbuild/sb-paraview-configure.cmake:47 (message):
Failed with exit code 1"
can anybody who have successed in compiling give me some suggestion.thanks.

I have this same problem with 5.12.10. The required 5.10 is no longer around for some reason.
It would be nice if the powers that be would give us a clue at to which if any other version of QT will work.
This has been out for 3.5 years but still no finished version with SLAM. Why?
None of the issues on here have been addressed since July of 2019. Why?
Is this not a dead vaprware version of Paraview?
Questions about Veloview go unanswered on the Paraview forum also.
Would someone from Kitware weigh in here and help out?
@bastienjacquet @PierreGuilbertF @mwoehlke-kitware

I have this same problem with 5.12.10. The required 5.10 is no longer around for some reason.
It would be nice if the powers that be would give us a clue at to which if any other version of QT will work.
This has been out for 3.5 years but still no finished version with SLAM. Why?
None of the issues on here have been addressed since July of 2019. Why?
Is this not a dead vaprware version of Paraview?
Questions about Veloview go unanswered on the Paraview forum also.
Would someone from Kitware weigh in here and help out?
@bastienjacquet @PierreGuilbertF @mwoehlke-kitware


Properly handle the case when a group includes itself.

Description:
Properly handle the case when a group includes itself.
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Should we instead try to prevent a group from being added to itself?

As a general case a group might include itself to show group status.
I use it in customUI because I expand the inner group into badges it contains.

Haha, as usual your use case is some crazy hack 😝


Add Swift Package Manager Support

Since swinject has SPM support, this should have it also

Thanks @marlontojal for posting a request👍
Do you have good idea to set up SPM for an Xcode project? SwinjectStoryboard is used to develop iOS/macOS/tvOS UI. I'm not familiar with SPM for an Xcode project.
We tried in PR #39 but couldn't complete. Would you help us?

Thanks @marlontojal for the request. I'll add SPM support after it supports iOS/macOS/tvOS targets☀

SPM should support iOS, macOS and tvOS targets 🙂

With XCode 11 maybe it is time to revisit this. Swinject/SwinjectStoryboard and Swinject/SwinjectAutoregistration already support SPM

Indeed, it would be usefull with Xcode 11

With xcode 11 we really should think about this again.

+1 on this.

I added support swift package manager https://github.com/Swinject/SwinjectStoryboard/pull/136.
☀☀☀

This has finally been merged! Closing this ticket now.


[Windows Powershell/Command Prompt/Azure CloudShell]: The Name property of a focusable element present on the homepage,must not be null.

Windows Terminal version
1.17.3052.0
Windows build number
Windows 11 Version 22H2 (OS Build 25206.1000)
Other Software
No response
Steps to reproduce

Open Windows Terminal.
Run Accessibility insights for Window on the homepage.
Observe the issue.

User Experience: -
Screen Reader user are impacted here as name is announced for control as result, they will not be able to know the purpose of control.
Guideline Reference:
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value
Attachments:
The Name property of a focusable element must not be null.zip
Expected Behavior
The Name property of a focusable element must not be null.
Actual Behavior
The Name property of the control defined in the side of the homepage should not be null and the control should be made hidden.

@Shubham786786 In #14541, I've added a name to this element and I've also made the AccessibilityView Raw. However, now FastPass is complaining that the AccessibilityView should be set to Control, but we don't want this element to be interactable or focusable in any way. It's purely an implementation detail. Is there a way to exempt this control from FastPass? Or when you do your next pass, can you manually exempt it and not file a bug on us for it?


Migrate Offline Badge from Dynamics to here

In Dynamics, when network is down or there is 429 problem, an Offline Badge will show up at the right top of the App. We need to migrate the implementation to widget.

Implemented in #110


customizable buttonOuterSize

now it is possible to fix:
https://github.com/moschan/react-native-simple-radio-button/issues/32
buttonSize={18}
buttonOuterSize={30}
works best for real device (for me)

Hi @rmariuzzo !
Thank you for your pull request. looks good.
I will merge it and bump soon.

Credits goes for @merapi


Fix mistakes

I just fixed a few linguistic mistakes.

https://github.com/microsoft/calculator/pull/1518#issuecomment-810793068

Sorry, we are not able to make localization changes directly into the codebase. We'll file an internal bug to help address this issue. Thanks!


overflow

there's might** be overflow issues
e.g.
abs function returns x == T(0) ?  T(0) :  x < T(0) ? - x : x
since abs(min limit) == max_limit + 1
the result for abs(INT64_MIN) is -9223372036854775808 (negative)
INT64_MIN == 0 ? 0 : INT64_MIN < 0 ? -INT64_MIN : INT64_MIN
** = i didn't executed the code since i'm using it as reference

The result of gcem::abs(INT64_MIN) matches the result ofstd::abs(INT64_MIN).


Set activity instance metadata through a scheduling profile

abercrom
https://jira.jpl.nasa.gov/browse/AERIEQS-373
Moderate
It would be useful to be able to set activity instance metadata through a scheduling profile, especially the activity instance name. This would allow a scheduling rule to assign a meaningful name to activity instances created by the scheduler.

Duplicate of #756


re-add inverse breadcrumbs sass

Somehow during the backporting of inverse modifiers for buttons, breadcrumbs and back links in https://github.com/alphagov/govuk-frontend/issues/3827, the actual sass code for the inverse breadcrumbs was missed.

Looks like those changes got made against the package folder rather than src:
https://github.com/alphagov/govuk-frontend/pull/3850/files#diff-e56ff8e57885f1320a693de4328ff561a2f11055144805e68b205f0398c83518

That explains why it got removed from the dist in https://github.com/alphagov/govuk-frontend/pull/3912


Include the new Razor directive icon glyph in the RazorExtension.

https://devdiv-design.visualstudio.com/DevDiv Design/_workitems/edit/2540

@NTaylorMullen is this the same issue as https://github.com/dotnet/aspnetcore/issues/5063?

Yup!

Cool, thanks for verifying! Closing out as a dupe of https://github.com/dotnet/aspnetcore-tooling/issues/4541


Domain name validation: w3.domain.com

I tried to add a domain to my app: deis domains:add w3.domain.com
and got the following error :
{u'domain': [u'Hostname does not look valid.']}
w3.domain.com is a valid domain.

That's interesting. RFC 1123 allows each hostname label to start and end with a digit, which is what the validation should allow.
This should be as easy as adding tests for this hostname to validate this, then amending the checks at https://github.com/deis/deis/blob/master/controller/api/serializers.py#L254 to be permissive for these domains.
To confirm, are you on v1.4.1?

Yes @bacongobbler, I am indeed on v1.4.1

In this case, it should be as simple as adding the tests and removing the or label[-1].isdigit() clause.

In this case, it should be as simple as adding the tests and removing the or label[-1].isdigit() clause.

:+1:


share button

i would like to add a share button so people can easily share to tweet, linkedin etc.

pls assign me this task under hacktoberfest.

@Ashish93-mrx You can proceed with your contribution.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Include the issue number in your PR description for better tracking.

I'm excited to see your contribution as part of Hacktoberfest 2024! 🎉
Don't forget to follow me on GitHub for more updates.
Happy open-source contributing! 🚀

@neerajrathore i have made the pull request please review #54 and for this issue 'hacktoberfest' label is not added, i wanted this as hacktoberfest contribution :(.


Error if process.env.WATCH is not defined

Error
\node_modules\fork-ts-checker-webpack-plugin\lib\service.js:49
    : new IncrementalChecker_1.IncrementalChecker(typescript, exports.createNormalizedMessageFromDiagnostic, exports.createNormalizedMessageFromRuleFailure, process.env.TSCONFIG, JSON.parse(process.env.COMPILER_OPTIONS), process.env.CONTEXT, process.env.TSLINT === 'true' ? true : process.env.TSLINT || false, process.env.TSLINTAUTOFIX === 'true', process.env.WATCH === '' ? [] : process.env.WATCH.split('|'), parseInt(process.env.WORK_NUMBER, 10) || 0, parseInt(process.env.WORK_DIVISION, 10) || 1, process.env.CHECK_SYNTACTIC_ERRORS === 'true', process.env.VUE === 'true', resolveModuleName, resolveTypeReferenceDirective);

TypeError: Cannot read property 'split' of undefined
    at Object.<anonymous> (\node_modules\fork-ts-checker-webpack-plugin\lib\service.js:49:399)
    at Module._compile (module.js:635:30)

Expected behavior
It could be good to check if process.env.WATCH is defined
Steps to reproduce the issue
I used an hello world example of Nuxt.js with nuxt-i18n
Environment

fork-ts-checker-webpack-plugin: 1.3.7
typescript: 3.5.3
tslint: 5.18.0
webpack: 4.34.0
os: Windows 10

Would you like to submit a PR?

We are setting WATCH in https://github.com/TypeStrong/fork-ts-checker-webpack-plugin/blob/0c934557928d58e2c10f866dfe0ad9352ee12f13/src/index.ts#L599 before calling the service.ts.
The only way this behaviour makes sense to me would be if Windows 10 would delete environment variables with an empty string instead of keeping them. And that would have many more implications.
@johnnyreilly, if I remember correctly you're using Windows for development, right? Could you test that?

I am a Windows user yes - will try and check that when I get some time. (Lots going on right now 😁)

@pamls
Please try fork-ts-checker-webpack-plugin@alpha - I've published a new version which should resolve this issue 🚀
I will close this issue to clean-up the backlog. If this release didn't solve the issue, we can re-open this :)


Issue with body parameters

Hello,
Describe the bug
It looks like body parameters are not taken into account.
I have tried with my own code and the example given in schemathesis.
When I type the following command in example folder of schemathesis (SCHEMA_URL and PYTHONPATH are set):
schemathesis run $SCHEMA_URL
I have these errors:
===================================================================== Schemathesis test session starts =====================================================================
platform Linux -- Python 3.9.2, schemathesis-3.6.4, hypothesis-6.10.1, hypothesis_jsonschema-0.19.1, jsonschema-3.2.0
rootdir: /home/clachat/wd/codes/python/schemathesis/example
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/clachat/wd/codes/python/schemathesis/example/.hypothesis/examples')
Schema location: http://<IP_ADDRESS>:5000/api/openapi.json
Base URL: http://<IP_ADDRESS>:5000/api
Specification version: Open API 3.0.0
Workers: 1
Collected API operations: 3

GET /api/bookings/{booking_id} F                                                                                [ 33%]
POST /api/bookings/ F                                                                                [ 66%]
GET /api/bookings/ .                                                                                [100%]

================================================================================= FAILURES =================================================================================
____________________________________________________________________ GET /api/bookings/{booking_id} [P] ____________________________________________________________________
1. Received a response with 5xx status code: 500

Path parameters : {'booking_id':<PHONE_NUMBER>}
Headers         : {'User-Agent': 'schemathesis/3.6.4', 'Accept-Encoding': 'gzip, deflate', 'Accept': '*/*', 'Connection': 'keep-alive'}

----------

Response payload: `{"type": "about:blank", "title": "Internal Server Error", "detail": "Server got itself in trouble", "status": 500}`

Run this Python code to reproduce this failure:

    requests.get('http://<IP_ADDRESS>:5000/api/bookings/2147483648', headers={'User-Agent': 'schemathesis/3.6.4', 'Accept-Encoding': 'gzip, deflate', 'Accept': '*/*', 'Connection': 'keep-alive'})

Or add this option to your command line parameters: --hypothesis-seed=270120329738940535782402410985261502850
_________________________________________________________________________ POST /api/bookings/ [P] __________________________________________________________________________
1. Received a response with 5xx status code: 500

Headers         : {'User-Agent': 'schemathesis/3.6.4', 'Accept-Encoding': 'gzip, deflate', 'Accept': '*/*', 'Connection': 'keep-alive', 'Content-Type': 'application/json', 'Content-Length': '2'}
Body            : {}

----------

Response payload: `{"type": "about:blank", "title": "Internal Server Error", "detail": "Server got itself in trouble", "status": 500}`

Run this Python code to reproduce this failure:

    requests.post('http://<IP_ADDRESS>:5000/api/bookings/', headers={'User-Agent': 'schemathesis/3.6.4', 'Content-Type': 'application/json', 'Accept-Encoding': 'gzip, deflate', 'Accept': '*/*', 'Connection': 'keep-alive', 'Content-Length': '2'})

Or add this option to your command line parameters: --hypothesis-seed=295339955890644101447220614181709515798
================================================================================= SUMMARY ==================================================================================

Performed checks:
    not_a_server_error                    120 / 135 passed          FAILED

======================================================================= 1 passed, 2 failed in 1.24s ========================================================================

From docker-compose, one of the errors is:
web_1  | ERROR:connexion.apis.aiohttp_api:Error handling request
web_1  | Traceback (most recent call last):
web_1  |   File "/usr/local/lib/python3.8/site-packages/connexion/apis/aiohttp_api.py", line 56, in problems_middleware
web_1  |     response = yield from handler(request)
web_1  |   File "/usr/local/lib/python3.8/site-packages/aiohttp/web_middlewares.py", line 110, in impl
web_1  |     return await handler(request)
web_1  |   File "/usr/local/lib/python3.8/site-packages/connexion/decorators/coroutine_wrappers.py", line 23, in wrapper
web_1  |     connexion_response = yield from connexion_response
web_1  |   File "/example/app/views.py", line 23, in create_booking
web_1  |     booking_id=body["id"],
web_1  | KeyError: 'id'
web_1  | INFO:connexion.aiohttp_app:<IP_ADDRESS> [05/May/2021:12:43:51 +0000] "POST /api/bookings/ HTTP/1.1" 500 284 "-" "schemathesis/3.6.4"

id is not found in the body, so it looks like id is not given.
To reproduce the issue
Try the beginning of the examples in the schemathesis/example folder:
docker-compose up
export SCHEMA_URL="http://<IP_ADDRESS>:5000/api/openapi.json"
export PYTHONPATH=$(pwd)
schemathesis run $SCHEMA_URL

Environment

Python 3.9.2
schemathesis 3.6.3 and 3.6.4

Regards,
Cédric

Hi Cédric,
This behavior is expected. It illustrates a bug - the Booking schema does not define the id field as required

Hi Dmitry,
Thank you for your answer.
Sorry, I didn't understand that.
So, I will explain the case with my own code:
The API schema (as Open API JSON format) produced by flasgger is (some parts have been cleaned):
{
  "basePath": "/adduser_python_admin",
  "definitions": {
    "UserSchema": {
      "properties": {
        "email": {
          "format": "email",
          "type": "string"
        },
        "firstName": {
          "type": "string"
        },
        "lastName": {
          "type": "string"
        },
        "shortName": {
          "type": "string"
        }
      },
      "required": [
        "shortName"
      ],
      "type": "object"
    },
    "user.Credentials": {
      "properties": {
        "otpSecret": {
          "description": "OTP",
          "type": "string"
        },
        "password": {
          "description": "password",
          "type": "string"
        }
      }
    },
    "user.Id": {
      "properties": {
        "userId": {
          "description": "Id of created user",
          "type": "integer"
        }
      }
    },
    "user.State": {
      "properties": {
        "state": {
          "description": "wanted state",
          "type": "string"
        }
      }
    }
  },
  "info": {
  },
  "paths": {
   "/v1/pcc/{pcc_uid}/user": {
      "post": {
        "consumes": [
          "application/json"
        ],
        "deprecated": false,
        "description": "Create an user for a PCC",
        "operationId": "admin.createUser",
        "parameters": [
          {
            "in": "body",
            "name": "body",
            "required": false,
            "schema": {
              "$ref": "#/definitions/UserSchema"
            }
          },
          {
            "in": "path",
            "name": "pcc_uid",
            "required": true,
            "type": "string"
          }
        ],
        "produces": [
          "application/json"
        ],
        "responses": {
          "201": {
            "description": "Success request",
            "schema": {
              "$ref": "#/definitions/user.Id"
            }
          },
          "400": {
            "description": "Validation error"
          }
        },
        "schemes": [],
        "security": [],
        "summary": "Create the user ",
        "tags": [
          "admin"
        ]
      }
    }
  },
  "swagger": "2.0"
}

When I run a test on the route "/v1/pcc/{pcc_uid}/user" I have these errors:
E           schemathesis.exceptions.CheckFailed:
E
E           1. Received a response with 'text/html; charset=utf-8' Content-Type, but it is not declared in the schema.
E
E           Defined content types: application/json
E
E           ----------
E
E           Response payload: `No data to validate`
E
E           Run this Python code to reproduce this response:
E
E               requests.post('http://<IP_ADDRESS>:20810/adduser_python_admin/v1/pcc/lim1c-1/user', headers={'User-Agent': 'schemathesis/3.6.4', 'Accept-Encoding': 'gzip, deflate', 'Accept': '*/*', 'Connection': 'keep-alive', 'Content-Length': '0'})

In my opinion, the body parameters (as email, firstName, lastName, shortName) are not given.
Maybe, my schema is wrong.

Thank you for supplying a schema :)
The problem comes from required: false in the body parameter definition:
{
    "in": "body",
    "name": "body",
    "required": false,
    "schema": {
        "$ref": "#/definitions/UserSchema"
    }
}

As it is not required, then one of the options Schemathesis tries is not sending any payload at all. I hope that it clarifies the situation :) Let me know if you have any further questions

Hello Dmitry,
Thank you very much!
I think that my issue comes from Flasgger.
Have a good day,
Cédric.

You are very welcome, Cédric!
Feel free to reach out to me in case you have any further questions :)
Cheers,
Dmitry


I want to ask how to convert the test output to the camera coordinate system

I want to ask how to convert the test output to the camera coordinate system

Which dataset you are looking forward to do so?


Add holy trait to inline damage roll of Blessed Blood (Nephilim)

Adding the holy trait inline doesn't seem to actually affect IWR.


Fllowstatus bits appear to be cleared when running additional steps

Strange bug I ran into. Code to reproduce (runs when saved as file in root directory, requires working off  branch noah/allowfail):
import siliconcompiler

netlist = f"tests/data/oh_fifo_sync_freepdk45.vg"
def_file = f"tests/data/oh_fifo_sync.def"

jobname = 'issue_repro'
design = "oh_fifo_sync"

chip = siliconcompiler.Chip()
chip.set('design', design)
chip.set('netlist', netlist)
chip.set('asic', 'def', def_file)
chip.set('mode', 'asic')
chip.set('quiet', True)
chip.set('jobname', jobname)

chip.target('freepdk45')

# no-op import since we're not preprocessing source files
chip.set('flowgraph', 'import', '0', 'function', 'step_join')

chip.set('flowgraph', 'place', '0', 'tool', 'openroad')
chip.set('flowgraph', 'place', '0', 'input', 'import0')

chip.set('flowgraph', 'place', '1', 'tool', 'openroad')
chip.set('flowgraph', 'place', '1', 'input', 'import0')

# Illegal value, so this branch will fail!
chip.set('eda', 'openroad', 'place', '0', 'option', 'place_density', 'asdf')
# Legal value, so this branch should succeed
chip.set('eda', 'openroad', 'place', '1', 'option', 'place_density', '0.5')

# Perform minimum
chip.set('flowgraph', 'placemin', '0', 'function', 'step_minimum')
chip.set('flowgraph', 'placemin', '0', 'input', ['place0', 'place1'])

# ** TOGGLE THIS FLAG TO CONTROL BEHAVIOR **
RUN_CTS = False
if RUN_CTS:
    chip.set('flowgraph', 'cts', '0', 'tool', 'openroad')
    chip.set('flowgraph', 'cts', '0', 'input', ['placemin0'])

chip.run()

print(chip.get('flowstatus', 'place', '0', 'error')) # expect 1
print(chip.get('flowstatus', 'place', '1', 'error')) # expect 0, get 1 when RUN_CTS == True ??

When RUN_CTS is set to False, those print statements show what I'd expect:
1
0

since the first place branch fails, while the second one succeeds. However, just by setting RUN_CTS to True, they both print 1:
1
1

Fixed by #973!


Bug in payment received

[x]  only need to show in invoice# section in listing view
[x] No need filters/ give only one option like all payments
[ ] in Payment record against invoices,amount section type keep as float
[ ]  under Payment record against invoices, negative number block
[ ]  under payment received /made column,negative number block
[ ] under refund edit section,Wrong balance amount after edit of partial refund
[ ] under refund delete section,Wrong balance amount after delete of partial refund
[ ] under update section,can't update without any change- wrong popup(created with full amount)
[ ] under update section,received full amount, apply amount missing
[ ] under opening balance section,no need to show opening balance in listing; if opening balance===0
[ ] under update section(enhancement),Vendor/customer name can't change at the time of update
[ ] under creation screen,pass message like "There are no unpaid invoices associated with this customer."
[ ] under accounts entry section,not updated by edit
[x] under history section,show edit and refund details in history section
[x] view journal in detailed screen
[x] show due date in edit screen
[ ] under bank charges,credit ledger change to "selected ledger" from "accounts receivable"
[x] under filter option,add retainer payments and invoice payments
[x] under create and update of bank charges,The Bank Charges can't be more than the amount received.
[x] under create and update of bank charges,Bank charges are applied. Please select a bank account.

[ ] under history section,show edit details . but here the content is invalid


Fix always deploy to default branch problem

this PR is an update of https://github.com/Unitech/pm2-deploy/pull/160
this PR fixed the broken $ pm2 deploy production
the fix will do a simple fast-farward if ref exists in remote server

i get the error
  ○ running setup
  ○ cloning git@github.com:keymetrics/wakanda-hydroplant.git
/home/unitech/keymetrics/pm2-deploy/deploy: line 218: test: !=: unary operator expected
  ○ fast fetch
Cloning into '/var/wakanda-hub/source'...
Host key verification failed.

When running pm2 deploy production setup any clue on that?

@Unitech i have tested it with ref in ecosystem.json and ref command line options. I haven't tested  with branch field in ecosystem.json. is it documented?

@Unitech please try pm2 deploy production setup again

So, pm2-deploy now 'pulls' code instead of 'reset --hard'.
But why not give us 2 solutions ?
Current solution is good when branch can be fast forwarded and easily pulled.
But what to do when branch can't be fast forwarded and need to be reseted ?

may i know what is your use case? local changes at remote server that required reset —hard?

may i know what is your use case? local changes at remote server that required reset —hard?

Yes, it can be local changes on remote server. Person from other team can do some changes to test.
But we have another use case. Our git flow is built around rebase logic. We're preparing production branch and then rebase each /production upon production. In additional each /production contains commits. So trying to pull branch in our flow which was rebased will lead to mess of conflicts.

@KoteKotlyarov i feel your pain. have you considered you tag for deployment?

for more details https://github.com/Unitech/pm2-deploy/blob/master/deploy#L262-L276

Hi @ldarren ,
I knew that. But we don't want to update deploy file on each deployment. We're thinking about changing our Git Flow.
To deploy specific branch we can do:
pm2 deploy %file% %env% ref %branch%
I.e.: pm2 deploy deployment.json test ref origin/mybrand/hotfix

@KoteKotlyarov can try this?

@ldarren Yes, I have successfully deployed specified branch.


Dynamic Display of Local Events on "View Local Events" Button Click

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

Currently, there isn't a feature to display local events when users click on the "View Local Events" button. This limits users' ability to discover and engage with local events, potentially leading to a less engaging user experience.
Describe the solution you'd like

I would like to propose a feature where clicking on the "View Local Events" button triggers a display of local events relevant to the user's location or specified area. These events could include concerts, festivals, workshops, and other activities happening nearby.
Additionally, the code is already prepared and ready for implementation, I kindly request to be assign me to the issue
Describe alternatives you've considered

One alternative solution could be to manually list local events on a separate page or section of the website. However, this approach may require frequent updates and may not provide the most relevant or up-to-date information to users.
Demo Video/Screenshot (Mandatory )
View Local Event Button

After Clicking on the button

Are you contributing under any open-source program ?

I am a part of GSSoC 2024

@coderiksenthil u are assigned. really good feature

Thank you, @apu52, for assigning me this task. I'm currently working on it, and I expect to have it completed within the next two days.

@apu52 Will soon raise a PR

@apu52 Will soon raise a PR

yess. please try to submit your PR ASAP . As u are assigned over 1 month on this issue

Sure @apu52


How to use two profiles in a single command (advice needed)

Hi Guys
I have a requirement wherein I have to copy files from one S3 bucket to another S3 bucket. These S3 buckets have been created on two different accounts. I want to write a script to do the copy process. For this, I created two users on each account and set up profile for each in the credentials file under .aws directory. Now, when I want to copy the files from source to target, I want to do something like the following.
aws --profile source s3 cp s3:// --profile target s3://
i.e use a different profile for source bucket and another profile for target bucket. I understand that I can first execute the aws command only for source bucket and get the files onto my local system and then write another aws command to upload the files to the target bucket using the target profile. However, is there any way by which the above can be accomplished in a single aws command?
Thanks
KD

There's no way to use two profiles in a single command.
However, what you can do is use a bucket policy to give the appropriate permissions to the destination profile so that they can copy from the bucket in the source profile.

That is helpful.Thanks!


Add date prefix when defining object names

Resolves #64
I did some manual testing for the front end and back end changes here, but it could be worth linking this branch to dev if we wanted to test them simultaneously

@Eo300 @mmacduff does the job_control.py file need to be updated as well or does it work because the jobid includes the date and jobid?

@Eo300 @mmacduff does the job_control.py file need to be updated as well or does it work because the jobid includes the date and jobid?

it appears that we need to update job_control.py to me


Add dynamic backend plugins support

Description
Add support for dynamic backend plugins.
Which issue(s) does this PR fix

Fixes #526

PR acceptance criteria
Please make sure that the following steps are complete:

GitHub Actions are completed and successful
Unit Tests are updated and passing
E2E Tests are updated and passing
Documentation is updated if necessary
Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Test CI / Docker Build (pull_request) will never pass if your PR is from your personal fork. It needs to be from the janus-idp origin for that to pass.

Note: working on downstream in https://issues.redhat.com/browse/RHIDP-425


Keep resident runner connected on iOS when screen locks in debug mode

#9351
For debug mode only. We can also do something more conservative and figure out a way to stop the task when USB disconnects or something.

@chinmaygarde for thoughts

Nice!

It'd be a lot cleaner if we asked the engine what product mode it is configured for instead of having to add to the Info.plist. That way, we wouldn't have to modify templates and such.

Oh cool, we can? Can you point me to where I can read that value?

There is a FLUTTER_PRODUCT_MODE define in the engine. If we are only going
to use it in the FlutterAppDelegate, then we don't even need to expose it
as a public API.
On Thu, Apr 13, 2017 at 10:37 AM xster<EMAIL_ADDRESS>wrote:

Oh cool, we can? Can you point me to where I can read that value?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/9359#issuecomment-293970196, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACsNdM7UBiX-TmGPFUyvV5CBaNWB6HWks5rvl1UgaJpZM4M8H1t
.

Moved to engine as suggested
https://github.com/flutter/engine/pull/3610


Upgrade to rTorrent v5.5

Full Changelog: https://github.com/stickz/rtorrent/compare/v5.3-0.9.8-0.13.8...v5.5-0.9.8-0.13.8
What's Changed

https://github.com/stickz/rtorrent/pull/28
https://github.com/stickz/rtorrent/pull/29
https://github.com/stickz/rtorrent/pull/32
https://github.com/stickz/rtorrent/pull/33
https://github.com/stickz/rtorrent/pull/35
https://github.com/stickz/rtorrent/pull/36

@crazy-max UDNS is currently disabled by default. It was tested with hundreds of instances. The crash volume was too high compared to synchronous DNS resolution. Once the unreported crash is resolved, we plan to re-enable the future. We have a stack trace for the crash. Working on getting a more detailed one, as it can't be restimulated in every environment.
Two hash threads instead of one is currently working stable. When multiple downloads are active, the 30% load is successfully split between two threads of 15% each. Half the downloads are assigned to each thread. This is more scalable for multiple instances. Event signals are sent to each hash thread to active them. They basically sleep when not being used to avoid incurring overhead.
rTorrent v5.5 is the final and most stable release for version 5. I'm working on getting all platforms upgraded before the version 6 stability release. We need to ensure all bug reports are on the latest version, to avoid stale information from circulating.

Looking at UDNS handling in libtorrent I was wondering if we could have an interface to use a third-party resolver. We could then defer to a sidecar container exposing a dns forwarder such as https://github.com/crazy-max/docker-unbound

Looking at UDNS handling in libtorrent I was wondering if we could have an interface to use a third-party resolver. We could then defer to a sidecar container exposing a dns forwarder such as https://github.com/crazy-max/docker-unbound

Yes, we can add a configure option like --enable-resolver with an AC_DEFINE and LIBS addition to only include third-party resolver in the build when enabled. https://github.com/stickz/rtorrent/blob/master/libtorrent/scripts/udns.m4

Closing this off in favour of upgrading to version 6! The stability issues with UDNS are now fixed and performance is improved!


Examples not using fetch'

The documentation of Network.Discord.Rest makes it seem like there is a much better way to make requests than just using fetch and fetch'. However, all examples currently use fetch'.
Could you make at least one example showing how to use Network.Discord.Rest.Channel and similar?
The documentation is also outdated by a couple versions.

Actually, I have no clue how to use the newest version at all. It seems to be completely different from the examples...

I was working on a rework that I (somewhat) accidentally pushed to hackage, but I just started a new job and my life will be a bit busy for at least the next month. I will likely revert my changes to what you see in the documentation and add a patch to the Fetchable class to allow you to unwrap returned values (currently not possible). I apologize for the way this library has progressed recently, and provide my reasons to give context, not make excuses. I wanted to take another crack at constructing a better abstraction but ran out of time.

I see. Well, I can understand that.
If you do decide to work on it again (even if just to revert your changes) - it would be very nice if you could make it work with a more current resolver. I had a lot of trouble getting it to work because many older packages seem to refuse to compile with newer cabal.
(Most notably pandoc, which is one of your dependencies. Lens has a whole 3 dependencies I had to update manually and it has this problem itself)


UI: fix netmask is not passed to api when create share network

Description
This PR fixes the issue that ip4 netmask is not passed to cloudstack api when create a shared network.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[x] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6300 (SL-JID-1436)


Include path pollution

I’m building my project with cryptopp using FetchContent:
FetchContent_Declare(
  cryptopp_cmake
  GIT_REPOSITORY https://github.com/abdes/cryptopp-cmake.git
)
set(CRYPTOPP_BUILD_TESTING OFF)
FetchContent_MakeAvailable(cryptopp_cmake)

target_link_libraries(some_thing PRIVATE ... cryptopp::cryptopp ...)

This seems to add _deps/cryptopp_cmake-build/cryptopp to the include paths, which means if I #include <zlib.h> it includes the zlib.h from cryptopp, instead of the header from the actual zlib. (Actually I don’t use zlib.h but another dependency does, and it’s including the wrong header.) Is this the intended behavior?

Not surprising as the includes are not reachable otherwise. I can't check here atm. would it work if you remove this line?

Addendum: Or the one after that. I know I've added them, but I'm pretty sure source_dir plus the Prefix in your code should be enough.

Not surprising as the includes are not reachable otherwise. I can't check here atm. would it work if you remove this line?

Yes! Removing this line worked for me.


Master role inherits from regular api key roles

When a regular api key user creates an analysis, the master role must be able to access the created tables. The proposed solution is that the master role inherits from the regular api key role:
GRANT regular_role_XXX to master_role

Uhm, I'm worried about this approach, let's discuss.
First, as it is now, regular api key roles cannot create tables. If we want them to be able to create analysis tables, they will also be able to create regular tables.
The mixed ownership (some objects owned by the key role) is also problematic. What would happen is we delete the api key? We would have to reassign ownership of all objects, or the role deletion would failed (a worse alternative is to delete the objects 🔥).
I'm also worried about details of things like cartodbfication, which I think would create the associated triggers of a table with the role of the master, so the table would have different ownership than the triggers. This feels like a bad idea, although I cannot point out at specific problems at the moment.
Maybe we should always create the analysis tables using the master role instead? That would fix all my issues, but I'm willing to bet it would make life harder for Windshaft. 🤔

Maybe we should always create the analysis tables using the master role instead? That would fix all my issues, but I'm willing to bet it would make life harder for Windshaft.

Before moving forward to find a solution, let me shed some light on what's going on when a user creates a map with analyses in Maps API. Right now, only the user role (master) is able to create node analysis. Within the workflow of creating a map with one analysis we have:

If one node analysis requires being cached a new table is created and populated.
Performs some meta-operations, to update cdb_analysis_catalog and/or read from cdb_tablemetadata.
When creating a map that reads from that cached analysis table, Maps API performs some operations to calculate surrogate keys and affected tables. These operations are performed with user role (master)

If we want a regular api-key role will be able to create a map with analyses we need to grant permissions to read/write on cdb_analysis_catalog and read on cdb_tablemetadata. Once a cache table is created with a regular api-key role the ownership belongs to that role. So, when we create a map that uses the previous analysis the operations described in point 3 will fail because master role doesn't have perms on that table. That's why @enekid opened this ticket. ;)
If we decide this issue should be fixed in Maps API side we need to tell to Analysis API to create the analysis cache table with user role (master) and then grant perms to the actual role that creates the analysis (regular api-key role). That means, we are coupling Analysis API to Auth API and it sounds really weird to me.
Thoughts?

🤔 I think that we have a similar situation right now, don't know if the situation is the same. But if a map has an analysis that depends on a widget, if any public viewer (using no api key) clicks a widget, an instantiation request is made using no api key, but the user db role is used to create the analysis cache tables.
So there must be some way inside Windshaft already to use a different role than what is specified by the request. I'm not saying this is the way to go, but this is how we solved this in the past.
I think we  need to keep something like this, because we still need analysis tables to be able to be rendered without api key, but without making the analysis tables accessible to the publicuser (public maps with private data). In that sense, it makes some sense to me that the regular keys behave similar to the default one (use a different connection for analysis).
In your proposal: if you want to make the map created with the regular api key public (and render it without providing an apikey). If the user forces the analysis to rerun, how would the tiler know which role to use? It cannot use the public user (which is the one that matches the provided api key).
Note that this is not a problem exclusive with analysis. If we ever want to give api keys rights to CREATE TABLE we would have the same issues I highlighted in my previous comment, derived from tables owned by different roles. So if this is something we see us doing, we should think about how to solve/workaround those.
The idea to grant the keys role to master is a good solution, but it doesn't cover all cases by itself (e.g: api key deletion and reassigning ownership).

Also, another concern, the analysis API needs CREATE TABLE permissions in order to work. So if we go for the approach of giving the needed rights to the API Key role, we would end up with all regular api keys being able to create tables in the user schema (which I think is something we don't want).
Some ideas that might work:

Create different roles for SQL API and Maps API. They would have the same rights, except the Maps API one will have access to the analysis cache/catalog, etc.
We could look into using SET ROLE for the Api Key user to escalate privileges. But we would have to restrict its usage somehow in SQL Api.
We could create some SECURITY DEFINER functions to manage analysis tables (probably inside the extension). This would allow to safely manage analysis tables without granting rights on it to any user.

I think that we have a similar situation right now, don't know if the situation is the same.

Yep, you are right. We have a similar situation with public viewers which is solved through named maps™. When a public viewer clicks a widget, a named map instantiation request is made w/o api_key. Behind the scenes, Maps API uses the user role (master) to perform the analysis work. Having in mind that one of the side effects of releasing Auth API is to deprecate named maps (we are not totally sure about that, but it's there), so I'm afraid that we don't want to switch to master when an analysis is involved in map instantiation with regular api-keys.
On the other hand, we might try to solve a problem from the technical point of view or we might try to solve it by designing a totally different approach. It would be:

In the future, Analysis API will have its own entity (by creating a totally new service ¿¿ ... ??). It might a user can perform analysis w/o visualizing them with maps.
A user will want to create a regular api-key to allow to create analysis. I'm pretty sure about that ;)
Currently, we grant access to different APIs (Maps or SQL) by reading the property grants_maps or grants_sql.

I propose to add a new field in  api-key's structure to indicate that a regular api-key can create analysis (cached tables) and also it will have access to the needed metadata tables (grants_analysis). In Maps API we can check if a regular api-key can create an analysis or not whether a map instantiation involves analyses.
By default grants_analysis will be false so we can add it later as we don't want regular api-key to create analysis yet in this first stage. For master it will be true by default.
Thoughts??

After having an off-line conversation:

We aren't going to implement anything regarding regular api-key and analysis until we have a clearer version of Analysis API.
If for some reason we have to implement a workaround/solution/hack, our best option is:  create SECURITY DEFINER functions to manage analysis tables (probably inside the extension).
We need to keep user metadata map_key property when migrating to api-keys. We need them to provide a fallback mechanism and support named maps.

Anything else?

I think this covers everything. Since we are not going to implement anything (just keep using master for named maps), closing this!


[Swift 2.2] StdLib documentation improvements

Explanation: This request makes a number of improvements to the standard library documentation, including adding abstracts for several types and protocols that are missing them. These changes have all been merged into swift/master, and have been cherry-picked for this PR.
Scope: These changes are strictly limited to header documentation for the standard library. There are some minor changes to GYB logic, but only in service of improving the documentation.
SR Issue: These changes are not covered by an SR.
Risk: There should be no risk, as these changes only affect comments.
Testing: This change passes all validation tests when run via build-script -RT.

@swift-ci Please test

@natecook1000 Are you going to nominate this PR on swift-dev?

@dabrahams This is for the 2.2 branch, it needs approval from the release manager.

Sorry, I prematurely merged it.  The commit looks great to me, but I didn’t realize this was the 2.2 branch.

I approve taking this.

@dabrahams Did you end up reverting these changes from the branch?  If so, can you re-apply them?


cranelift-fuzzgen: Instruction format doesn't have a designated operand, bad opcode

Found on oss-fuzz with this input: input.gz
$ cargo +nightly fuzz run --strip-dead-code --no-default-features -s none cranelift-fuzzgen ./input
    Finished release [optimized] target(s) in 0.12s
    Finished release [optimized] target(s) in 0.12s
     Running `target/aarch64-unknown-linux-gnu/release/cranelift-fuzzgen -artifact_prefix=/home/acrichto/code/wasmtime/fuzz/artifacts/cranelift-fuzzgen/ ./clusterfuzz-testcase-minimized-cranelift-fuzzgen-4619879352762368`
WARNING: Failed to find function "__sanitizer_acquire_crash_state".
WARNING: Failed to find function "__sanitizer_print_stack_trace".
WARNING: Failed to find function "__sanitizer_set_death_callback".
INFO: Running with entropic power schedule (0xFF, 100).
INFO: Seed:<PHONE_NUMBER>
INFO: Loaded 1 modules   (123805 inline 8-bit counters): 123805 [0xaaaae7df0fd8, 0xaaaae7e0f375),
INFO: Loaded 1 PC tables (123805 PCs): 123805 [0xaaaae7e0f378,0xaaaae7ff2d48),
target/aarch64-unknown-linux-gnu/release/cranelift-fuzzgen: Running 1 inputs 1 time(s) each.
Running: ./clusterfuzz-testcase-minimized-cranelift-fuzzgen-4619879352762368
thread '<unnamed>' panicked at 'Instruction format doesn't have a designated operand, bad opcode.', cranelift/codegen/src/ir/dfg.rs:865:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
==2894== ERROR: libFuzzer: deadly signal
NOTE: libFuzzer has rudimentary signal handlers.
      Combine libFuzzer with AddressSanitizer or similar for better crash reports.
SUMMARY: libFuzzer: deadly signal
────────────────────────────────────────────────────────────────────────────────

Error: Fuzz target exited with exit status: 77

cc @afonso360

This panic is occurring when trying to generate an error context for an "instruction opcode doesn't match instruction format" error in instruction_integrity in cranelift's verifier, after doing the canonicalize_nans pass.
I patched this panic message to give more detail:

Instruction format for MultiAry { opcode: Iconcat, args: EntityList { index: 25, unused: PhantomData } } doesn't have a designated operand

So at least that tells us what's failing.

Looks like my suggestion a while back to use MultiAry for all opcodes is biting us here. The instruction format for Iconcat is Binary, which returns its first operand for typevar_operand. But MultiAry returns None from that function.
Apparently, this hasn't been an issue before because many of the instructions that use the Binary format have requires_typevar_operand=false. But Isplit and Iconcat are exceptions, and there are quite a few other more obscure instructions where we'll have the same problem.


Design inconsistency in the search bar

I think the border line shoud be higher in order to surround the black bar above "Undo":

Also, I think the greyed out text color (in dark mode) is misleading when you are able to use the button and vice versa:

Yes, it looks like those menus need to get fixed

Fixed!


Mark crates with custom build, unsafe (and other stuff worth reviewing)

https://docs.rs/cargo/0.32.0/cargo/core/package/struct.Package.html#method.has_custom_build
For unsafe we could use a quick recursive grep, maybe?
would be nice to integrate some existing safety scanning tool for Rust?

Detecting unsafe will be difficult. We'd need a full fledged parser to correctly ignore commented out code for example.

Nothing ATM. I'm postponing to 0.4

unsafe via cargo-geiger is done. custom build is a flag. All done.


[Question]: duckduckgo-search = "5.3.1b1"

Describe your problem
with current version. really doesn't work

But current version is like this:

with current version it seems it doesn't work

with  duckduckgo-search = "6.4.1"

It' working


Feature: backfill top level unit tests

Suggested solution
We should have a some sanity tests that execute the command run functions to ensure the help is being outputted correctly and there aren't any major bugs preventing the CLI from running properly (like incorrectly named / put together flags)
Related: #126

Have to sort something else out first. Will get back on this unless someone else picks it up first.