SPConfigWizard may result in inccorect psconfig command is used on SP2016

ISSUE TITLE:

SPConfigWizard may result in inccorect psconfig command is used on SP2016
ISSUE DESCRIPTION (this template):

I updated an SP2016 RTM with the JULY2016 CU
When using the command SPConfigWizard on a SP2016 RTM installation run psconfig this resulted in that an correct command line switch was used.
the command line that was used was
psconfig.exe -cmd upgrade -inplace b2b -wait -force -cmd installcheck -noinstallcheck
the command line that should have been used is.
PSConfig.exe -cmd upgrade -inplace b2b -force -cmd applicationcontent -install -cmd installfeatures
this as SP2016 as a featurep ack released
Details of the scenario you tried and the problem that is occurring
Due to that incorrect command line was used.
the issue described in this link occures. and Cetnral admin is not fully functional,
https://sharepoint.stackexchange.com/questions/247258/sp2016-central-admin-error-databinding-system-data-datarowview-does-not-cont
Verbose logs showing the problem
N/A
Suggested solution to the issue
maybe a switch to allow for adding installfeatures command
The DSC configuration that is used to reproduce the issue (as detailed as possible)
SPConfigWizard ConfigWiz
{
IsSingleInstance  = 'Yes'
DependsOn = "[SPProductUpdate]July2018"
PSDSCRunAsCredential = $credsSPSetup
}

The operating system the target node is running
OsName               : Microsoft Windows Server 2012 R2 Datacenter
OsOperatingSystemSKU : DatacenterServerEdition
OsArchitecture       : 64-bit
WindowsBuildLabEx    : 9600.19153.amd64fre.winblue_ltsb.180908-0600
OsLanguage           : en-US
OsMuiLanguages       : {en-US}
Version of SharePoint that is used (e.g. SharePoint 2016)
Sp2016
Version and build of PowerShell the target node is running
Name                           Value

PSVersion                      5.1.14409.1012
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1012
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
Version of the DSC module that was used ('dev' if using current dev branch)
SharepointDSC 3.0

Example of better code:
`
if ((Get-SPDSCInstalledProductVersion).FileMajorPart -eq 15)
{
$binaryDir = Join-Path $env:CommonProgramFiles "Microsoft Shared\Web Server Extensions\15\BIN"
}
else
{
$binaryDir = Join-Path $env:CommonProgramFiles "Microsoft Shared\Web Server Extensions\16\BIN"
}
$psconfigExe = Join-Path -Path $binaryDir -ChildPath "psconfig.exe"
         $stdOutTempFile = "$env:TEMP\$((New-Guid).Guid)"
         $stdErrTempFile = "$env:TEMP\$((New-Guid).Guid)"
         $psconfig = Start-Process -FilePath $psconfigExe `
                              -ArgumentList "-cmd upgrade -inplace b2b -wait -force -cmd applicationcontent -install -cmd installfeatures -cmd secureresources -cmd services -install -cmd helpcollections -installall" `
                              -RedirectStandardError $stdErrTempFile `
                              -RedirectStandardOutput $stdOutTempFile `
                              -Wait `
                              -PassThru
        $cmdOutput = Get-Content -Path $stdOutTempFile -Raw
        $cmdError = Get-Content -Path $stdErrTempFile -Raw
        if ($cmdError) {
                Write-Verbose -Message $cmdError.Trim()
            }
        if ($cmdOutput) {
                Write-Verbose -Message $cmdOutput.Trim()
            }
         Write-Verbose -Message $psconfig.ExitCode
         return $psconfig.ExitCode

`

The updates to the PSConfig parameters have been corrected according to the article of Stefan Gossner via #1012 and PR #1016
I like the suggestion by @paololec. Will update the resource in my next Bugfix PR

Did some further thinking about adding the logging suggested by @paololec. PSConfig already creates a log file. By storing the logging and outputting that to screen, it increases the logging tremendously, but doesn't add any additional troubleshooting info. So I don't think we should add this.

@NikCharlebois What is your opinion?

I don't see any issues displaying it whenever the -Verbose switch is used. I think there might be some useful info in there. At a minimum prompt the user about the location of the log file and let them access the file on the server if need be.

The amount of output it generates is quite large. If you deploy a multiserver farm, this will be added to the screen output multiple times. And since PSConfig already generates a log file.....
But I like the idea of displaying the log file location. Will implement that for sure!

Yes, I use -verbose and I like to see it on screen , but also with link to log files location can be good.

Fixed in PR #1036


proper usage of --import-dir, or; how to use hs-to-coq on codebases with imports

Hello!
I have a minimal example here. the hs-to-coq executable was created with stack install, but there is some nix in here

ls/tree

$ ls
edits Main.hs hs-to-coq/ shell.nix

Main.hs

module Main where

import Control.Lens

data Datum = Datum { aString :: String, anInt :: Int } deriving Show

foo :: String -> Maybe String
foo "barski" = Just "barski"
foo _ = Nothing

someLens :: Datum -> Maybe Datum
someLens = lens aString (\datum string -> datum {aString = string}) foo

main :: IO ()
main = do
  let d = Datum "barski" 1
      e = Datum "bazzzz" 1
      d' = someLens d
      e' = someLens e
  print d'
  print e'

shell.nix

{ pkgs ? import <nixpkgs> {}
, ghcVersion ? "ghc884"  # matches hs-to-coq/default.nix
}:
let
  hsenv = pkgs.haskell.packages.${ghcVersion}.ghcWithPackages (self: [
    self.lens
  ]);
  scratchDev = pkgs.stdenv.mkDerivation {
    name = "scratch-lens";
    src = ./.;
    buildInputs = [ hsenv ];
    buildPhase = ''
      ghc --version
      ghc-pkg list
    '';
    installPhase = "mkdir -p $out";
  };
in
pkgs.mkShell {
  # buildInputs is for dependencies you'd need "at run time",
  # were you to to use nix-build not nix-shell and build whatever you were working on
  buildInputs = [ scratchDev hsenv ];
}

Notice that lens appears in hs-to-coq.cabal.

edits

rename value Datum.Datum = Datum.MkDatum

And when we run we fail
$ hs-to-coq --permissive -e hs-to-coq/base/edits --iface-dir hs-to-coq/base -e edits Main.hs -o .
Main.hs:3:1: error:
    Could not find module ‘Control.Lens’
    Use -v to see a list of the files searched for.
  |
3 | import Control.Lens
  | ^^^^^^^^^^^^^^^^^^^

Perhaps this is what --import-dir is for?

where are my libraries for this version of ghc?

nix-shell
which ghc
# /nix/store/b33hk9dshra06v2z140zf0nr8gw9hk3h-ghc-8.8.4-with-packages/bin/ghc
ls /nix/store/b33hk9dshra06v2z140zf0nr8gw9hk3h-ghc-8.8.4-with-packages/lib/ghc-8.8.4/x86_64-linux-ghc-8.8.4/
# .... lens-4.19.2-DKB7mu1opy3FSq86GPdtGR ...

ok, lens is in that directory. So if we run...
hs-to-coq --permissive -e hs-to-coq/base/edits --iface-dir hs-to-coq/base --import-dir //nix/store/b33hk9dshra06v2z140zf0nr8gw9hk3h-ghc-8.8.4-with-packages/lib/ghc-8.8.4/x86_64-linux-ghc-8.8.4/ -e edits Main.hs -o .

...well, we get the same error.
Let's add
axiomatize module Control.Lens

to edits.
Same error.
It seems to me like there's a version of ghc running inside the executable. Is this basically correct? If so, it would make sense that another version of ghc wouldn't be able to make this work.
It'd be great to have isolation and not have to install packages to some global ghc that the hs-to-coq executable calls upon. Moreover, it'd be great to have observability into what packages are reachable by the ghc that the hs-to-coq executable calls upon.

I've been wrapping my head around the wc example, looks like it has the components I need, thanks!

That does seem like the right solution. You need a file Data/ByteString.v somehow, and running hs-to-coq on Data/ByteString.hs (with axiomatize module) is the easiest way, or you stub it manually.

I'm confused. If I have a Data/ByteString.hs, isn't running hs-to-coq on it a replacement for axiomatize module? in other words, doesn't having a .v file by running hs-to-coq make axiomatize module sort of obsolete (in the context of that particular need)? Is it because axiomatize module is used so as to not descend into the entire library and only convert one .hs file namely Data/ByteString.hs to .v, so the edits file would have to axiomatize module the deeper parts of the library while hs-to-coqing shallower parts?


test(slider): add unit tests to Slider

add unit tests to Slider

close in favor of #423 due to broken build code here and forced rebase


Support Join

Seems like currently we can't do a join operation.
@waterlink Will you support it in the upcoming version?

Yeah currently functionality is very basic. If you want you can try to create a PR for that. I can guide you through current codebase if you want to do that. Otherwise it is on my to-do list 😀

I'd really like to try that out :+1:

@waterlink how shall i start with this :smile:

First we need to decide the interface. Which args should join receive? What will it return?

Well i really like AR and would really like the have the same interface as possible :+1:

So I imagine something along these lines:
class Model
  # args map directly to 'JOIN #{table_name} ON #{on_query}'
  def join(model_klass, on_query : Hash|Query) : JoinedModels
end

And the usage example is:
Post.join(User, {"posts.author_id" => "users.id"})
# or equivalent
Post.join(User, criteria("posts.author_id") == criteria("users.id"))
    # => JoinedModels < Post, User, ... >

Looks like JoinedModels should have the same querying methods as Model, like:

where
all

I am not sure what should they return. Array(Post) ? In that case there is no access to fields of other table..

Maybe every model could have something like this:
posts = Post.join(User, criteria("posts.author_id") == criteria("users.id")).all

post = posts.first
post.title     # => "Hello world post"

user = User.from_joined(post)
user.name     # => "Oleksii"

Other way to do it is to define all such possible relationships beforehand (aka belongs_to and has_one and has_many):
class Post < ActiveRecord::Model
  belongs_to User
end

posts = Post.join(User, criteria("posts.author_id") == criteria("users.id")).all
post.first.title           # => "Hello world post"
post.first.user.name      # => "Oleksii"

Though the problem awaits here:
class Post < ActiveRecord::Model
  # at this point you get compile error, since `User` is not defined yet
  belongs_to User
end

class User < ActiveRecord::Model
  has_many Post
end

You get compile error, because User is not defined yet.

Since it is macro, we can fix it by not using User right away, but rather define new method inside of macro:
macro belongs_to(model)
  {% model_name = model.id.stringify.gsub(/.+:/, "").downcase %}
  def {{model_name.id}}
    {{model.id}}.from_joined(self)
  end
end

Maybe belongs_to (and friends) could provide optional parameter for name of the method to help user avoid conflicts, like:
macro belongs_to(model, method_name = nil)
  {% unless method_name %}
    {% method_name = model.id.stringify.gsub(/.+:/, "").downcase %}
  {% end %}

  def {{method_name.id}}
    {{model.id}}.from_joined(self)
  end
end

WDYT @sdogruyol ?

@waterlink I like this and your way of handling compiler error :+1:
class Post < ActiveRecord::Model
  belongs_to User
end

posts = Post.join(User, criteria("posts.author_id") == criteria("users.id")).all
post.first.title           # => "Hello world post"
post.first.user.name      # => "Oleksii"

Ok then. There is a plan for public interface now :+1:
Now on the internals. Currently all the heavy lifting (actual querying) is done by Adapter protocol implementations.
You can see this protocol here: https://github.com/waterlink/active_record.cr/blob/master/src/adapter.cr#L19-L30
Currently, as you can see, adapter is supposed to have knowledge only about one table.
I was thinking about extending it like this:
abstract def self.build(table_name, join_specs : Array(JoinSpec), primary_field, fields, register = true)

where JoinSpec is:
struct JoinSpec
  property table_name :: String
  property on_query :: Query
  def initialize(@table_name, @on_query) end
end

By default, when you are not using join, join_specs is empty [] of JoinSpec.
Adapters, accordingly will add relevant query to make a join, or blow up with an error if it is not supported (some databases don't have concept of join).

@waterlink Would you mind starting this and let me join on the way :smile:

No problem :)

Hi!
Any idea when you'll be able to implement joins?
Really would love to port a rb project to cr, but without joins, it will be nearly impossible.

https://twitter.com/waterlink000/status/831403009724788736


Guest Configuration S360 Swagger fixes

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[X ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month. - NA
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month. - September 2021
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[X] No refresh required for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Can someone please help me with the failures here?
I have updated the PR to point it to the new default main branch, but still it is complaining.

Can someone please help me with the failures here?
I have updated the PR to point it to the new default main branch, but still it is complaining.

which failure do you mean? @aashish1987

Failing after 28s — Build #Azure OpenAPI failed
Failing after 21s — ShouldSendPRToMain failed
If you look into the section - Some checks were not successful you can see above two failures.
Not sure if those are warnings.
Also can someone please help me with the review.

/azp run


Add a simple FTP client

This PR adds an FTP client to the NIH servers, implementing basic retrieval methods in python.

This will likely be part of the pipeline we use to fill our data store, the ElasticSearch


Unable to run repo:create from non-git folder

$ gh help repo:create

 [ERROR] Unable to get the repository information, Git remote "origin" should be set for automatic detection.

OK, this requires a bit more work :sob: the RepositoryCreateCommand depends on the adapter but can't require that the command is executed from a Git repository.
We can split-up the GitRepoSubscriber into a listener for adapter-aware commands and a second listener for commands that need the org/repo options. Plus as a small bonus we are further reducing the complexity of GitRepoSubscriber::initialize()
Also using having the --org option in this command doesn't make sense, I actually assumed this was 'used' as target-org :smile:

hmm not sure what is the use case here, to bootstrap a project on an empty folder?

Yes :) I have initialized an empty git repository and want to create an empty repository on GitHub.

I assume you mean repo:create for creating a repository on upstream github or other? I would look more into keeping this even more simpler and have a global configuration that does not depend on local folder. However the splitting into weird class names is not fun, if we do we have to make code more plain and using better names and document the reason why we do that.

Closed in favor of #459


Revert 1102_tokenproxy_command.test

Proposed changes
This PR reverts 1102_tokenproxy_command.test and updates Keyrock and PEP proxy docker images.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[X] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you
did and what alternatives you considered, etc...

Coverage: 100.0%. Remained the same when pulling 5b5c819fa6dda19752b837857b35ecc22f230884 on fisuda:update/testcase into aa7bd3cc9257b0cdf917c234b2f9ad57c2ad9682 on lets-fiware:main.


Add unofficial Medium SDK - Swift

Project URL

Description

Checklist

[x] Only one project is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Travis CI pass
[x] Supports iOS 7 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

You are deleting a project
Please fix this

Let me know if you need help

Fixing with another pull request


Show menu slide out on the front

Hi,
I am having an issue when I am trying to open the menu on the front view controller which is not part of the view controllers array of sideMenuConfig.
is there any way I can open the menu on the viewController class that is not part of sideMenuConfig?

Yes, you can access the side menu in any viewController class that is not part of sideMenuConfig using SSSideMenuControls. openOrCloseSideMenu().


Added support for statuses/build Resource

Adds support for the build status API.

Ping @lucascherifi, might be helpful for Jenkins/Bitbucket integration ;-)

I just noticed that PRs should be sent on Bitbucket ...


RFC: treewide: python: argparse default configuration allows shortened command arguments

Introduction
An issue has been identified in the argparse python library with its default configuration in which it enables automatic argument abbreviation (not user-set abbreviation as is common place in command line utilities). This causes bad script usage because commands can be added in the future which then silently and wholly without warning change the behaviour of what the script does.
Problem description
This seems to be a whole zephyr-wide issue with any python code making use of argparse, this library decided that defaulting to allowing shortened command line arguments was a good idea, unlike traditional commands where you have a short and long argument e.g. -h and --help, this python library will, assuming you have a --help command accept this as --h, --he, --hel and --help. This is a pretty huge oversight as if a command exists, is wrongly called with a short argument and then a new command is added which uses the shortened argument, this will silently change the whole behaviour of the script, and indeed 2 whole test scripts for west flash commands start failing when this option is disabled and one starts failing because a new command has been added which the test script was wrongly using for the command previous.

Proposed change
A tree-wide change request for fixing this issue so that argparse is set to not allow it to generate its own shortened commands, when commands are added they can manually specify the shortened command to allow alongside the long argument.
Detailed RFC
This requires setting allow_abbrev=False on all ArgumentParser() instantiations, and will require updating any script usage that does not use the full long argument name (or specified short argument name, if added).
Dependencies
New python code which uses argparse should have allow_abbrev=False included and any invokations of scripts should use the proper argument names.
Concerns and Unresolved Questions
This will only impact code in zephyr, it may also need propegating to other zephyr-hosted repositories and they should also be updated (if they have python scripts using argparse) to set allow_abbrev=False to prevent future issues with those scripts too.
User scripts for custom builds might also need fixing, this is unknown.
Alternatives
Leave it as it is, which is a big risk with silent breakage, and possible security risk to CI tasks (i.e. user/customer production CI builds)

Default description: https://docs.python.org/3/library/argparse.html#allow-abbrev

How do we still allow abbreviations on the command line?

How do we still allow abbreviations on the command line?

You would add a long and short argument for a command when adding the command, e.g. for -h and --help it would look similar to: parser.add_argument("-h", "--help", help="Show help text"). Any sort of automatic abbreviation is prohibited.


Client Error:

The API was working well but suddenly I am getting the client error,
404 Client Error: Not Found for url: https://osome-botometer.p.rapidapi.com/2/check_account
How to resolve it?

We recently disabled the free API, please subscribe to the Pro one. See announcement at https://twitter.com/Botometer/status/1250557098708144131
Please refer to https://github.com/IUNetSci/botometer-python/wiki/Using-the-Pro-API for instruction

Yes, but as per the basic plan I should get bot score for 2000 account in
24 hours but it's not working.
On Sun, May 3, 2020 at 3:17 AM Kaicheng(Kevin) Yang <
<EMAIL_ADDRESS>wrote:

Closed #46 https://github.com/IUNetSci/botometer-python/issues/46.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IUNetSci/botometer-python/issues/46#event-3296957736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6IKFU43CTK3RD5VKPRPVDRPTA2VANCNFSM4MX3RMAA
.

--
Gautam Kishore Shahi,

When you say "it's not working", do you mean you can't get any or you can get some but not 2000 a day?

I am not able to get any bot score in the last 30 hours, before that it was
working fine.
My data size has only 375 accounts.
How to resolve this issue?
On Sun, May 3, 2020 at 3:30 AM Kaicheng(Kevin) Yang <
<EMAIL_ADDRESS>wrote:

When you say "it's not working", do you mean you can't get any or you can
get some but not 2000 a day?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IUNetSci/botometer-python/issues/46#issuecomment-623039270,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6IKFTWQHQREQBGJNCUHLDRPTCM3ANCNFSM4MX3RMAA
.

--
Gautam Kishore Shahi,

Please read the announcement I shared above

The description says some changes for a pro account but for Free-API link
is not working(https://market.mashape.com/OSoMe/botometer#check-account).
On Sun, May 3, 2020 at 3:36 AM Kaicheng(Kevin) Yang <
<EMAIL_ADDRESS>wrote:

Please read the announcement I shared above
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IUNetSci/botometer-python/issues/46#issuecomment-623039875,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6IKFTO5CPKMT24PKGMS2LRPTDBNANCNFSM4MX3RMAA
.

--
Gautam Kishore Shahi,

It's not working because we disabled it. Please follow the instruction in the announcement to subscribe to the PRO API.

Okay, got it, even to access the 2000 limit, I need to register to pro
account.
But the link https://market.mashape.com/OSoMe/botometer-pro is not working.
On Sun, May 3, 2020 at 3:47 AM Kaicheng(Kevin) Yang <
<EMAIL_ADDRESS>wrote:

It's not working because we disabled it. Please follow the instruction in
the announcement to subscribe to the PRO API.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IUNetSci/botometer-python/issues/46#issuecomment-623040809,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6IKFXJZKATEWQRSELGC43RPTELXANCNFSM4MX3RMAA
.

--
Gautam Kishore Shahi,

Here is the link https://rapidapi.com/OSoMe/api/botometer-pro
Would you mind telling me where did you find  https://market.mashape.com/OSoMe/botometer-pro?

Thanks.
https://github.com/IUNetSci/botometer-python/wiki/Using-the-Pro-API#endpoint-url
In the above page, some URL refers to
https://market.mashape.com/OSoMe/botometer-pro
On Sun, May 3, 2020 at 3:55 AM Kaicheng(Kevin) Yang <
<EMAIL_ADDRESS>wrote:

Here is the link https://rapidapi.com/OSoMe/api/botometer-pro
Would you mind telling me where did you find
https://market.mashape.com/OSoMe/botometer-pro?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IUNetSci/botometer-python/issues/46#issuecomment-623041517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6IKFT4NKWJ7QOSKYPDXETRPTFJLANCNFSM4MX3RMAA
.

--
Gautam Kishore Shahi,

Thank you! I'll fix them now.

Thanks, it working again.
BTW, in each request how tweets do you process?
Do I need to send a batch of tweets(like 100 or 180) for each request or
Botometer internally do it?
On Sun, May 3, 2020 at 4:01 AM Kaicheng(Kevin) Yang <
<EMAIL_ADDRESS>wrote:

Thank you! I'll fix them now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IUNetSci/botometer-python/issues/46#issuecomment-623041931,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6IKFULSCI3KO5POM7S3XDRPTF5XANCNFSM4MX3RMAA
.

--
Gautam Kishore Shahi,

Please refer to the README: https://github.com/IUNetSci/botometer-python

Hi, my account is a pro one, but I still have the problem of 404 Client Error.

@bitscott1102 we just updated the python package, could you try to upgrade it with pip and see if it works?

That's great! It's working! Thanks!!


Separate Issues & PR page

I know we discussed having PRs and issues page on the dashboard but was thinking maybe we COULD have separate page for PRs and Issues but this time having more information like a total count of issues and have filters for which repos have different PRs and issues.

I am not sure about this. Couldn't they view this on Github per repo as well? How do we add value beyond github?

Like we could add a filter to only show issues and PRs for certain repositories. We could do a filter to show the oldest issues and PRs, we could filter to show which issues we manually assigned to ourselves and which issues were assigned by another user. We could show the total number of issues and PRs assigned to a user. It's just an idea since all we are showing at the moment are ALL the PRs and issues assigned to a user.
All in all, its just an idea. I do agree we could do with just the dashboard page but am open to suggestions or we could even move the filters to the dashboard.

We could do oldest and most recent repos for sure, maybe we could add those as additional features when we are done with the overall application?

Yeah that's a good point. We only have a week left and we should focus on what we already have. Anyways I will close this issue. Thanks for the constructive discussion!


"Isolate cannot be reloaded" results in hot reload stack and tool doesn't quit

I wasn't paying attention, but had my phone plugged in and was doing a hot reload every few minutes (it does it automatically when I hit the key that brings up the analysis). Eventually, something broke and then the output from the tool looked like this:
Performing hot reload...
Hot reload failed:
code = 109
message = Isolate cannot be reloaded
dart:async/future.dart 237                                 Future.Future.error
package:flutter_tools/src/vmservice.dart 859               Isolate.reloadSources
===== asynchronous gap ===========================
dart:async/zone.dart 1085                                  _CustomZone.registerBinaryCallback
dart:async-patch/async_patch.dart 39                       _asyncErrorWrapperHelper
package:flutter_tools/src/vmservice.dart                   Isolate.reloadSources
package:flutter_tools/src/run_hot.dart 439                 HotRunner._reloadSources
===== asynchronous gap ===========================
dart:async/zone.dart 1073                                  _CustomZone.registerUnaryCallback
dart:async-patch/async_patch.dart 30                       _asyncThenWrapperHelper
package:flutter_tools/src/run_hot.dart                     HotRunner._reloadSources
package:flutter_tools/src/run_hot.dart 389                 HotRunner.restart
===== asynchronous gap ===========================
dart:async/zone.dart 1061                                  _CustomZone.registerCallback
dart:async/zone.dart 949                                   _CustomZone.bindCallback
dart:async/schedule_microtask.dart 148                     scheduleMicrotask
dart:async/future.dart 182                                 Future.Future.microtask
package:flutter_tools/src/run_hot.dart                     HotRunner.restart
package:flutter_tools/src/resident_runner.dart 187         ResidentRunner._handleSignal
===== asynchronous gap ===========================
dart:async/zone.dart 1061                                  _CustomZone.registerCallback
dart:async/zone.dart 949                                   _CustomZone.bindCallback
dart:async/schedule_microtask.dart 148                     scheduleMicrotask
...

@Hixie do you have any idea how to reproduce this?

None whatsoever, sorry.

any ideas on how to reproduce this? If reproducing this is not easy can we close this issue as stale.


how to avoid CUDA out of memory?

All of the training scripts specified in the README give errors like the following:
RuntimeError: CUDA out of memory. Tried to allocate 4.00 GiB (GPU 0; 15.78 GiB total capacity; 10.52 GiB already allocated; 3.86 GiB free; 10.77 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

CUDA Version: 11.7
using 8x Tesla V100-SXM2 (with 16GB memory)
reducing --batch_size 32 didn't help
passing --microbatch 1 didn't help

Lowest I managed to get was 32GB with batch size 1... but that was ages ago

Same, I was hoping there would be some flag configuration that would allow using a GPU (or multiple GPUs) with less memory

Same, I was hoping there would be some flag configuration that would allow using a GPU (or multiple GPUs) with less memory

Since it based on pytorch you might be able to implement some optimization like HF's accelerate or deepspeed for RAM offloading


Blur shadow is showing on cropper

When I am presenting CropViewController Image is showing in background and when I am touching to crop then Image is showing .

Can you attach a screenshot? It would be easier to know more detail about this issue.


Update sbt-sonatype to 2.6

Updates org.xerial.sbt:sbt-sonatype from 2.5 to 2.6.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

Codecov Report

Merging #313 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #313   +/-   ##
========================================
  Coverage    61.29%   61.29%
========================================
  Files           50       50
  Lines         1540     1540
  Branches        40       40
========================================
  Hits           944      944
  Misses         596      596

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09bae75...a5bb3b0. Read the comment docs.


Refs #25645 -- Removed useless fallback in SpatialiteSpatialRefSys.wkt.

https://code.djangoproject.com/ticket/25645

A more informative commit message might be something like "Removed SpatiaLite 3.x compatibility in SpatialiteSpatialRefSys.wkt." -- is that accurate?

Yes, it's more informative, thanks. Changed commit message to the proposed one.


Fix docstrings

Changes

Addresses doc string issues.

Known Issues

We can add more examples to the docstring.

Few minor comments


The ImageOptim app for Mac OS X optimizes better than the gem

Hey, Ivan! Thanks for the :gem:.
Due to your gem I found the ImageOptim app and tried it first. The app is just great :heart:
After that I've started using the image_optim gem and encountered a weaker optimization.
Example files:

http://i.imgur.com/UXkqvAO.png (118,049 bytes initially; 113,710 bytes after the image_optim gem) – http://i.imgur.com/BO3EkjX.png (112,355 bytes after the ImageOptim app)
http://i.imgur.com/8m8tSKG.png (7,189 bytes initially; the same after the image_optim gem) – http://i.imgur.com/RJhFJkF.png (7,100 bytes after the ImageOptim app)
http://i.imgur.com/H4vwBSK.jpg (158,163 bytes initially; 149,239 bytes after the image_optim gem) – http://i.imgur.com/PQNcSrW.jpg (147,383 bytes after the ImageOptim app)

I can provide more example files if necessary. Those were first 3 images I tried to compress with the image_optim gem.
What I did to use the gem:

Added gem 'image_optim', '~> 0.21.0' and gem 'image_optim_pack', '~> <IP_ADDRESS>50802' to Gemfile and bundle installed them
Installed binaries with brew install advancecomp gifsicle jhead jpegoptim jpeg optipng pngcrush pngquant (ran brew update first) and optionally installed svgo + pngout – those installations probably had no effect

Questions
Is that expected? How can I optimize with help of the image_optim gem as well as with help of the ImageOptim app?

I see right away that I need to update the README.markdown as when using image_optim_pack you don't need to install anything except svgo and pngout.
image_optim is expected to be as good as possible. Asking if it is expected for ImageOptim to optimise better than image_optim is a very strange question. You can optimise using both tools and there is a cli for ImageOptim. And everyone is very welcome to contribute.


Axis Error

Hi
Was trying to use the cleaning application but I am getting the following error:

/ ||  _ \ / |  / || |  ___   __ _  _ __
| |  _ | |) |_ \ | |    | | / _ \ / ` || ' \
| || ||  __/  ) || | | ||  __/| (| || | | |
_|||    |/  _||| ___| _,||| ||
Version:  1.0.0
Reading your trace...
Loading the model...
Predicting points...
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
Traceback (most recent call last):
File "/disk1/raksha/GPSClean/raksha-clean-env/bin/gpsclean", line 8, in
sys.exit(main())
File "/disk1/raksha/GPSClean/raksha-clean-env/lib/python3.9/site-packages/gpsclean/gpsclean.py", line 77, in main
segments, indices, predictions = ft.predict_trace(interpreter, deltas, 15, 2)
File "/disk1/raksha/GPSClean/raksha-clean-env/lib/python3.9/site-packages/gpsclean/FullTraining.py", line 114, in predict_trace
predictions=np.argmax(predicted_segments,axis=2)
File "<array_function internals>", line 180, in argmax
File "/disk1/raksha/GPSClean/raksha-clean-env/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 1216, in argmax
return _wrapfunc(a, 'argmax', axis=axis, out=out, **kwds)
File "/disk1/raksha/GPSClean/raksha-clean-env/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 54, in _wrapfunc
return _wrapit(obj, method, *args, **kwds)
File "/disk1/raksha/GPSClean/raksha-clean-env/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 43, in _wrapit
result = getattr(asarray(obj), method)(*args, **kwds)
numpy.AxisError: axis 2 is out of bounds for array of dimension 1
Attaching the sample gpx file
https://drive.google.com/file/d/1yNoTzNmDQM5jwPiSksnbVg52ssLKaw-X/view?usp=sharing

Hi! Thank you for reporting this, let me investigate it and I will inform you as soon as possible.

HI @raksha-agarwal ! I investigate a bit your issue and found it resides in the fact that the GPX trace does not contain any track but only waypoints, while the package expects at least a GPX track... And yes, for sure the exception is not good and the validation should check that and provide an useful message. I promise I will implement it in a couple of days.
In any case, it is possible to convert a trace made of only waypoints in a standard track by using, for example, https://www.gpsbabel.org/. Using the following command I was able to convert the trace and apply the cleaning procedure:
gpsbabel -i gpx -f sample.gpx -x transform,trk=wpt -o gpx -F track2.gpx
I attach the converted track and the result of the cleaning procedure in a zip folder. Please let me know if I can help you an futher or if we can consider the issue close. Thank you!
sample.zip

Hi @sbettid
Thank you for the help. It is working now.
You may consider the issue to be closed.
Regards


fix(app): styling on single image view button

:pencil: Description

Fixes styling issue when there is only one image view button.

:tada: This PR is included in version 1.8.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Review eslint-plugin-import

https://www.npmjs.com/package/eslint-plugin-import

Blocking issue: https://github.com/benmosher/eslint-plugin-import/issues/142


CVE-2015-1872 and CVE-2017-11719

CVE-2015-1872 and CVE-2017-11719 research conducted

Take two upvotes for both, Interesting vulnerabilities and great descriptions!

+2 upvotes for the first CVE, +1 upvote for the second


Feature request: Custom Size Export

Would it possible to have an option to set the export dimensions to be the exact dimensions of the image (Rather than A4, A3, etc)?
So then there would be no white space around the output image.

This is by design (it's a print export plugin, after all), but that's a good request.


Add sharable/reloadable map options

I would like to share seeds and location with my friends.
Could you please add a button that creates a link containing the important map settings/location?
Also please don't erase the information from the url so that I can easily press reload or reopen the browser without loosing my location.
Similar to this:
https://www.chunkbase.com/apps/seed-map#seed=2419912010728942704&platform=java_1_20&dimension=overworld&x=0&z=0&zoom=0.5
Or dynmaps:

Thanks for creating this amazing tool.

duplicate of #11


Amusing Typographical Error.

There is an amusing typographical error in the documentatoin.
Expected Behaviour
Documentation has an error.
Current Behaviour
Documentation has an error.
Possible Solution
Corrects typographical error.
Steps to Reproduce (for bugs)
Visit github.com/alexellis/arkade and view the README.md.  Locate the error
Context
I was reading the documentation to understand the project.
Your Environment

What Kubernetes distribution are you using?
N/A

Operating System and version (e.g. Linux, Windows, MacOS):
N/A

What arkade version is this?
HEAD

See 141


Code completion of Keras Moudle no longer work since 2.6 (removed keras)

Code completion of Keras Module no longer work in VSCode after Keras code is remvoed from TensorFlow 2.6

@edwardyehuang
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here and provide some more details on this issue. Thanks!


Add Razer Panthera EVO Arcade Stick

With help from this comment on Reddit
Tested by editing my Steam udev rules locally, I can confirm it enables the controller in Steam.

This rule was added in steam-launcher <IP_ADDRESS>, currently a public beta.


Add DMI mux with DMI tests

This PR addresses #148 and adds tests for DMI module with the mux feature.

LGTM


libtool expecting several missing binary and include paths

Outstanding Tasks:

[ ] Has this issue or https://github.com/habitat-sh/core-plans/issues/3395 been resolved?
[ ] If yes, then enable the skipped tests in controls/libtool_works.rb and verify

Context
Notice that libtool --config returns a number of (1) binary paths including GREP, SED, and NM and (2) directories:
[17][default:/src:0]# /hab/pkgs/core/libtool/2.4.6/20200612110137/bin/libtool --config
...
...

# A sed program that does not truncate output.
SED="/hab/pkgs/core/sed/4.5/20200305230928/bin/sed"
...
...
# A grep program that handles long lines.
GREP="/hab/pkgs/core/grep/3.3/20200305232635/bin/grep"

# An ERE matcher.
EGREP="/hab/pkgs/core/grep/3.3/20200305232635/bin/grep -E"

# A literal string matcher.
FGREP="/hab/pkgs/core/grep/3.3/20200305232635/bin/grep -F"

# A BSD- or MS-compatible name lister.
NM="/hab/pkgs/core/binutils/2.32/20200305174809/bin/nm -B"
...
...

# LTCC compiler flags.
LTCFLAGS="-I/hab/pkgs/core/make/4.2.1/20200306002515/include -I/hab/pkgs/core/gcc/9.1.0/20200305180723/include -I/hab/pkgs/core/glibc/2.29/20200305172459/include -I/hab/pkgs/core/binutils/2.32/20200305174809/include"

...
...
# Command to truncate a binary pipe.
lt_truncate_bin="/hab/pkgs/core/coreutils/8.30/20200305231640/bin/dd bs=4096 count=1"
...
...

# The linker used to build libraries.
LD="/hab/pkgs/core/binutils/2.32/20200305174809/bin/ld"
...
...
# The library search path used internally by the compiler when linking
# a shared library.
compiler_lib_search_path=""

[18][default:/src:0]#

However, at least 2 of the binaries that are defined in here do not actually exist in the installation:
     ×  File /hab/pkgs/core/binutils/2.32/20200305174809/bin/nm -B is expected to exist
     expected File /hab/pkgs/core/binutils/2.32/20200305174809/bin/nm -B to exist
     ×  File /hab/pkgs/core/coreutils/8.30/20200305231640/bin/dd  is expected to exist
     expected File /hab/pkgs/core/coreutils/8.30/20200305231640/bin/dd  to exist

and at least 2 of the include directories:
     ×  File /hab/pkgs/core/make/4.2.1/20200306002515/include is expected to exist
     expected File /hab/pkgs/core/make/4.2.1/20200306002515/include to exist
     ×  File /hab/pkgs/core/gcc/9.1.0/20200305180723/include is expected to exist
     expected File /hab/pkgs/core/gcc/9.1.0/20200305180723/include to exist

A identical issue has been raised against the core-plans repository as well:  https://github.com/habitat-sh/core-plans/issues/3395

Update.  The above issue seems to happen when doing a 'cold-boot' of core/libtool in a new environment.  In other words, open a new hab studio, hab pkg install the core/libtool and then check the existence of all files and directories contained in the libtool --config.  For example:
hab studio enter
hab pkg install core/libtool --binlink
libtool --config

after testing for file/directory existence something like the following was observed:
     ×  File /hab/pkgs/core/make/4.2.1/20200306002515/include is expected to exist
     expected File /hab/pkgs/core/make/4.2.1/20200306002515/include to exist
     ×  File /hab/pkgs/core/gcc/9.1.0/20200305180723/include is expected to exist
     expected File /hab/pkgs/core/gcc/9.1.0/20200305180723/include to exist

However, after re-building core/libtool in the same hab studio environment, the above failures disappear:
     ✔  File /hab/pkgs/core/make/4.2.1/20200306002515/include is expected to exist
     ✔  File /hab/pkgs/core/gcc/9.1.0/20200305180723/include is expected to exist
     ✔  File /hab/pkgs/core/glibc/2.29/20200305172459/include is expected to exist
     ✔  File /hab/pkgs/core/binutils/2.32/20200305174809/include is expected to exist

Interpretation?  It seems that core/libtool configures itself according to the dependencies that are in its build environment.  The same dependencies, however, may not appear in the downstream environment


Low Hanging Fruit: Applied cargo clippy suggested changes.

They all seems reasonable to me.
Signed-off-by: Hanif Bin Ariffin<EMAIL_ADDRESS>

How did you get clippy to warn on all this stuff?
If I run cargo clippy from the CLI, I don't see these warnings. Is there a different config file that you're using?

This looks good; merging!
Thank you for the contribution.

I use nightly haha


Allow enable proc to accept Rack::Request

Some requests can be routed differently, depending on the HTTP verb.
For instance, in FormKeep, we display the Ember app for GET /f/:token,
but handle a Form submission via a POST /f/:token.

@rondale-sc what do you think about deprecating the current enabled: -> {} configuration value.
We'd leave it unchanged, fire off a warning, then introduce a new config lambda that only accepts the request.
I'm blanking on a good name for the key... :is_ember, :is_ember_request, :enable_build, :build?
I'm not particularly fond of any of those but I hope I'm communicating my intent.

I think :is_ember_request would help someone new looking at the code to understand the intent clearly.

Closing in favor of https://github.com/thoughtbot/ember-cli-rails/pull/263


Error: Node type GeometryNodeAttributeTransfer undefined

It looks like in the latest version of Blender (I'm using 3.4.1) this Node type became deprecated.
So I've got a runtime error in the solidify_colors_node_group function on line 679.

3.4 broke a few things I haven't quite had time to fix. I've been meaning to do a rewrite of the node network anyways, since the latest release added some features that would make this a lot less hacky.
In the meantime, 3.0-3.3 should still work. You can also check out Joey Carlino's video on how to do something pretty similar to this add-on if you're just using one material in 3.4.


Recursive tail call without a stopping condition should mark the function as never

Not sure if this is By Design, Limitation, Bug or simply left out.
TypeScript Version: All Versions
Code
// this has a stopping condition, a simple return statement could mark it finite.
// return type inferred as void: OK!
function foo(num: number) {
  if (num === 0) {
    return;
  }
  return foo(num - 1);
}

// no stopping condition
// return type inferred as void: :/
function bar() {
  bar();
}

Expected behavior:
bar should be never.
Actual behavior:
bar is void.

bar doesn't have a return statement thus the inferred type void seems fair. On the other hand I'm surprised you get an inferred type for foo as typescript doesn't infer self recursive functions at all.

Well returning recursive call without explicit type annotation is an error as it should be. I don't think this is worth exploring.


Make helpers support RSpec

Fixes https://github.com/bullet-train-co/magic_test/issues/7 since the change in https://github.com/bullet-train-co/magic_test/commit/ff8cb144927611349bf7ce14a1d9f632faf536cf didn't make it to the ok method.
Also moved the other non-public method to private.
I am now able to call ok but am getting a different error:

@andrewculver since the Capybara error I'm getting is unrelated, I say we merge this.  I will re-open a new issue if I'm still having problems.

@rubendinho Agreed. I think we're good here for now! Keep us posted on the other issue!


Various changes to the improve the functionality of the listing pages

Include a filter by prison
Add tabs to filter by yours, or others comparisons

LGTM

LGTM


Meta-data is overwritten with subsequent signup requests

Bug report
Describe the bug
Possible security vulnerability
When sending a signUp api call to an already existing user  with attached meta-data, the user's existing meta-data is overwritten in an unauthenticated context.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Find a user that exists and has existing meta-data in your supabase project
Issue another signUp api request for that same user's email with new meta-data
observer the meta-data on the API response

const { user, session, error } = await supabase.auth.signUp({
        email: email,
        password: password,
    },{
        data:{
            phone: phone,
            name: name,
            company: company
        }
    })

Expected behavior

If a user is already signed up, the metadata should not be updated
Nice to have: If a user is already signed up the API should respond with something like HTTP status code 409 instead of  User object.

Update - if I use the service account to inspect the auth.users table the metadata is not updated there so it seems that its just a bug in how the API endpoint responds upon a secondary signUp call.

This is actually a security fix where duplicate signup requests will return a successful response (with a randomized UUID) to prevent leakage of whether user has signed up for a service before. See https://github.com/supabase/supabase-js/issues/296#issuecomment-962391773. I think the auth documentation will need some updating to reflect this, since there have been a much of similar issues raised about this behaviour recently.

Knowing that a user has already signed up is imho not in and of itself a security risk unless the api allows unlimited amounts of requests to try to 'brute force' out an accounts list.  I would argue that knowing that an account already exists is important for most developers when building a useable workflow especially as users over time may have forgotten if they already signed up. Having to track this through a 'profiles' table or similar makes the auth feature less elegant .


Fetch deployed contract instances pt. 1

What was wrong?
Need an API for retrieving deployed contract instances if a package has one.
How was it fixed?
Started laying the groundwork for such an api
Cute Animal Picture

I went ahead and merged.  There was some minor cleanup that maybe you can roll into the next PR?

@pipermerriam Sounds good, I'll make sure they get included.


Imageless article cards don't display correctly

While using the theme I encountered a few posts with imageless article cards like the following and noticed they didn't look quite right:

It seems to me like this is caused by the following CSS:
.layout-single-column .status-card:has(.icon-file-text) {
    display: none;
}

I'm not sure if that particular rule is used anywhere else, but when I removed it, the card looked more correct (and I presume this is how all imageless article cards are actually supposed to look like):

Thanks. I think it was added due to this issue. I'll fix this in some point. Pull requests always welcome.


DOC: improved gmres doc on preconditioning (scipy.sparse.linalg)

Reference issue

What does this implement/fix?

GMRES documentation doesn't say if left or right preconditioning is used. Clearly, the answer is left.
Additional information

Thanks for the PR. Is this in response to an issue? If so, please post it. I'm not sure if this is something that the average user would care about, since the type of preconditioning used by the function is more of a "behind the scenes" kind of detail, and it's not hard to infer it from the fact that the function only asks for the inverse of the preconditioner.

Hi,
I haven't found any issue related to this. I needed the answer for my own use and found it before opening the issue.
Actually, the type of preconditioning seems to almost never be documented, except for  QMR, when both left and right preconditioners are accepted (arguments M1 and M2). For me it is quite important to know it, especially to adapt my tolerance (e.g. is the actual residual minimized, or the preconditioned one ?), but I don't know if I'm an average user.
PETSc documents this for all solvers.


npm test fails on dependency

The error is "babel-node not found"
To repro:
git clone git@github.com:LeisureLink-FE/react-drawer-hoc.git
cd react-drawer-hoc
npm i
npm test

All dependencies should be present now.

lgtm


ARROW-1695: [Serialization] Fix reference counting of numpy arrays created in custom serializer

This uses the NumPyBuffer built into Arrow's Tensor facility to protect the numpy arrays holding the Tensors to be serialized. See also the problem description in https://issues.apache.org/jira/browse/ARROW-1695.

Yes, this is correct! It is decref'd explicitly here: https://github.com/pcmoritz/arrow/blob/7e23bb5e7cd666e311595f76d60dbd08bf71920e/cpp/src/arrow/python/python_to_arrow.cc#L674

The test failure is in the go bindings and unrelates to this PR (cc @kou are you aware of this problem?). I'll merge this since I'd like to do some follow up work that needs this fix.

The test failure is in the go bindings and unrelated to this PR

I created #1234 to fix it.

Awesome, thanks :)


kokoro: fix dubious ownership

Kokoro clones repos with a different user used to run the build steps, meaning if some git command must be run at build time, they will fail because of this dubious ownership issue.
This is a known Kororo issue.
Fixing this is required to generate the version file using git history.
Signed-off-by: Nathan Gauër<EMAIL_ADDRESS>

Kokoro is not happy because I pushed while it started. Waiting for it to settle down before restarting all jobs.


Output image info for LLDB debugging

Summary
When the GenJitDebug feature is enabled, output the relocation information of the JIT code to enable debugging with LLDB.
Test plan
TODO

Codecov Report

Merging #350 (0e38b97) into main (8dd1182) will decrease coverage by 0.03%.
Report is 2 commits behind head on main.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main     #350      +/-   ##
==========================================
- Coverage   68.01%   67.98%   -0.03%
==========================================
  Files         114      114
  Lines       11138    11168      +30
  Branches     1864     1869       +5
==========================================
+ Hits         7575     7593      +18
- Misses       2595     2602       +7
- Partials      968      973       +5

Files Changed
Coverage Δ

oi/OICompiler.cpp
79.29% <62.50%> (-2.04%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Hack: create user on addmessage without existing user

Added the same hack as follow and unfollow. If addmessage is called with a non-existing user, it will create the user.
Added prometheus counter for the usage of this hack.

Ignore this pull-request. Branch created from issue was created from main (thus getting commits on main, that does not exist on dev - don't know why we have this, but whatevs :D)


ocm: 0.1.73 -> 1.0.2

bumps version from 0.1.73 to 1.0.2
skip TestCLI on all platforms as it now requires gnupg which has issues in the sandbox, in addition to networking related issues
remove meta.platforms = platforms.all which is redundant with buildGoModule's default
add myself as a maintainer (motivation: I use this tool daily so I have an interest in using the most up-to-date version)

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@ofborg build ocm

Thanks!


Right click on any block event error

(Required) What is your Quests version / build number (do not say "latest")?
v3.5.7-b573
(Required) Please describe the problem and how to replicate it. Any console errors?
Yep, Error log http://prntscr.com/m1delx
(Optional) Can you provide an example quest from quests.yml using Pastebin.com?
https://pastebin.com/wndFKxRH
Error when right click on any block https://pastebin.com/vqKxXhVh

Duplicate of #594
Use latest dev build.


cssloader: Rename "browsers" to "overrideBrowserslist"

Running react-static currently prints this warning:
Replace Autoprefixer browsers option to Browserslist config.
Use browserslist key in package.json or .browserslistrc file.

Using browsers option cause some error. Browserslist config
can be used for Babel, Autoprefixer, postcss-normalize and other tools.

If you really need to use option, rename it to overrideBrowserslist.

Learn more at:
https://github.com/browserslist/browserslist#readme
https://twitter.com/browserslist

It goes away by renaming browsers to overrideBrowserslist as
suggested.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

When does this come to release?


Update stale bitmonero reference

Update a stale bitmonero reference so edollard can run concurrently on the same machine.

Ping?


Publish to PyPI

https://pypi.org/project/scancodeio/ is 404.
While PyPI may not be the recommended way to install this software, publishing to PyPI allows SCA tools to inspect this software (even basic tools like https://www.wheelodex.org/projects/plugincode/rdepends/ are useful), and Python distro packaging tools use PyPI as a source.

This makes perfect sense. Scheduling for v33

Latest release published on pypi at https://pypi.org/project/scancodeio/


Is there a way to delete database?

Hi there, I was wondering if there is a way to delete the database or drop it or something like that?
Or would I have to go and manually delete all the tables?
Something like
SQLite.deleteDatabase({name: 'your_db_name', location: 'one of the 3 locations'});
Used in react-native-SQLite-storage?
Thanks,
Nihar

@craftzdog any ideas?


Simple item-label option

I wonder if I can display the count of selected elements rather than the list of all element properties?
i.e. If I checked 3 elements in the list it will simply display 3 Checked.
Thanks in advance for the answer.
Mathieu

Hi @freemind-
Yes it's available. Please read the details in the manual.

@isteven I read the doc + the code. I only figure out to set the default button label but I dont see how I can display my custom button label onItemClick... Could you please help. Thanks.

@isteven nevermind, max-labels will do the job :)


cleanPhoneNumber only working for 10-digit numbers

In the newly added cleanPhoneNumber method of AttendeeImporterService.java, there's the bit starting at line 179 (of commit d26e5a1a):
        if (p.matches("\\d{10}")) {
            output = p.substring(0, 3) + "-" + p.substring(3, 6) + "-" + p.substring(6, 10);
        } else {
            output = phoneNumber.replaceAll("^\\+[^0-9 -]", "").trim();
        }

If the p.matches expression is true, everything works out as expected, but if it gets to the branch with the replaceAll, that doesn't really weed anything out. The most it will do is remove the first two characters of the phone number, if they happen to be a "+" followed by a non-digit-space-or-dash.
I thought maybe you might want something like .replaceAll("[^0-9 +-]", ""), which would strip out everything but digits, spaces, or dashes, but then that doesn't really do the right thing with a phone number like "206-233-1419 x10349", which would just become "206-233-1419 10349". Honestly I'm not sure what the right thing to do with something like that is... maybe you need another, separate replacement for dealing with extension numbers like that?
(The phone number given in the paragraph above comes from "Dana Monahan" in the import data set - if you search for that attendee you'll see it imported as-is from text.csv)

My original intention was something like:

Remove everything but digits
If 10 digits are left, format it like a phone number ("xxx-xxx-xxxx")
Otherwise, assume it's an international number. Remove letters but don't try to format it nicely.

However, you're right:

3 doesn't work with the regex there now
An "x" will break validation for phone numbers on the form #2
The "x" will get lost.

Thinking about it the other way around, these seem like "good" numbers:
123-123-1234
123-123-1234 x5678
867-5309
+64 6-759 1234
So I'll also need to adjust the form validation to include "x" and "+"

Cleaned that up some. Note that I'm not checking for anything too crazy, like multiple "x"'s or a "+" somewhere besides the first character. I want to clean things up as much as possible, but if someone really wants to go nuts with the phone number, who am I to stop them? :)


Button and Type Text

Add example with button at menu and text type,
text type for example to use custom clan tag, to say what clan you want

make the sdk rage features ready and beginner hvh cheat friendly

M


pr-check.yml:82: Повысить fail-under до 96

The puzzle 1206-941398ab from #1206 has to be resolved:
https://github.com/blablatdinov/tg-quranbot/blob/0c942f98da8005c7401cc1a099b16201546ae240/.github/workflows/pr-check.yml#L82-L82
The puzzle was created by @blablatdinov on 02-Sep-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 1206-941398ab has disappeared from the source code, that's why I closed this issue.


[Feature]: Add Support for Tezos FA1.2 and FA2 Tokens in Ledger Live

Library name
@ledgerhq/hw-app-tezos
Describe your feature request
Feature Summary:
Request to add support for Tezos tokens following the FA1.2 (fungible token) and FA2 (multi-asset) standards within Ledger Live. These standards form the foundation for a variety of assets within the Tezos ecosystem, including stablecoins, interest-bearing tokens, and NFT Art assets. Enabling support for these tokens would allow Ledger Live users to securely store, manage, and interact with Tezos-native assets in one unified platform, ensuring compatibility with Tezos’ growing DeFi and commerce sectors.
By integrating these standards, Ledger Live would extend its leadership as the go-to platform for hardware-based security while serving the expanding demand for Tezos token management.
Why It’s Important:
Tezos is experiencing significant growth as a DeFi and NFT hub, with increasing activity across its Layer 1 ecosystem. The FA1.2 and FA2 token standards are the backbone of this activity, representing everything from stablecoins to complex DeFi assets and NFT Art tokens.
Supporting these standards would unlock access for Ledger Live users to manage some of the most important assets on Tezos, including:

USD Tether (USDt) on Tezos
USD Tez (USDtz)
ETH Tez (ETHtz)
Sirius (SIRS)
tzBTC (tzBTC)
Youves USD (uUSD)
Kolibri USD (kUSD)
CTez (FA1.2 standard collateral Tezos)
Various Interest Bearing Tokens on Tezos.
and countless more

Adding support for these tokens enables Ledger Live to tap into:

Growing Adoption: Users holding Tezos-based assets increasingly require secure solutions for storage and interaction.
Diversified Token Management: Ledger Live would become a central hub for users to access not only popular stablecoins but also liquidity pool tokens and yield-bearing assets.
Expanding Tezos Ecosystem: With a robust foundation for DeFi, commerce, and NFT Art, Tezos-native assets are essential tools for traders, collectors, and investors.

Ledger Live users managing Tezos tokens will benefit from the platform’s well-recognized security and user experience, removing the need for external or fragmented solutions.
Impact and Benefits:

Streamlined management of Tezos-based tokens for Ledger Live users.
Enhanced integration with Tezos DeFi, NFT Art platforms, and Liquidity Baking tools.
Positions Ledger Live as a trusted solution for securely storing and interacting with Tezos-native assets.
Fosters broader adoption of Ledger Live by the Tezos community, as Ledger becomes an indispensable tool for Tezos users.

Technical Documentation for Standards:

FA1.2 Token Standard
FA2 Token Standard
General Tezos Token Standards
Additional Reference - FA1.2

That would be great!

Make it happen @ledgerlive


Flaky performance around kafka connections

Flaky performance around kafka connections due to constant reconnect/disconnect, frequent NoBrokersAvailable errors crash threads

Before, we were closing Kafka connection after producing a bunch of messages, e.g.:
class Communication_manager:
    ...
    def produce_message(self, message_dict: dict):
        ...
        try:
            ...
            response = kafka_producer.send(self.kafka_topic_name, message_json.encode("utf-8")
            log.debug("Message sent, waiting for kafka_producer.flush()")
            kafka_producer.flush(timeout=10.0)
        except Exception:
            ...
        finally:
            if kafka_producer:
                kafka_producer.close()
                log.debug("Connection with KafkaProducer was closed")

Now, we keep the connection open during all the full object life, e.g.:
class Communication_manager:
   ...
    def __del__(self) -> None:
        self.close_consumer()
        self.close_producer()

    def produce_message(self, message_dict: dict):
        self.connect_producer()
        encoded_message = self.serialize_and_encode(message_dict)
        try:
            log.debug(f"Sending message to topic '{self.kafka_topic_name}'")
            response = self.kafka_producer.send(self.kafka_topic_name, encoded_message)
            log.debug(f"Message sent, response was: {response}")
            self.kafka_producer.flush(timeout=10.0)
            log.debug("Message flushed")
        except Exception:
            ...
        else:
            log.info(f"Message sent to Kafka")


[QA] docker-compose logs --no-color prints colors with ocis logs

docker-compose -f $compose_yml logs --no-color -f --tail=10
Shows the docker logs without colors. Mostly, except ocis logs:

Expected behaviour: all components appear in black / white

Workaround:
$ cat ~/bin/monochrome
#! /bin/sh
# remove ansi color sequences like eg \033[0m and \033[36m and friends.
#
sed -e s/[[0-9]*m//g "$@"

Use as e.g.
docker-compose logs ... | monochrome


Upgrade listener dependency incl. linting issues fix

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


Adding how to find minimum element of an array using stack in O(1) time complexity

💥 Proposal
I am faiz,an participant of GSSOC' 2021 and a avid competitive coder.I find that there is no solution of how to find minimum element of an array using stack in your repository.Hence please assign me to do this.I will write all its operation for example push(),pop(),isempty() etc.
My idea is that in finding the minimum element of an array using stack in O(1) space complexity as well as time complexity we can define a min val that stores current minimum in stack.To handle  minimum element removal from stack  we can push 2n-min into the stack instead of pushing n so that previous mimimum element can be retrieved using current min.
Have you read the Contributing Guidelines on Pull Requests?
YES.

please assign me this as it doesn't exists presently in the stack section @siddharth25pandey @atarax665 @HarshCasper

@zhcet19 Hi Faiz, Kindly specify in which language you want to work, so that I can assign you?

I want to work in c++

@zhcet19 I assigned you, Happy Coding!😊

Hi kindly assign me I also want to contribute in this issue


Check the file existence of application.conf.j2

Since the initial openwhisk release does not package the folder tests, we
need to make sure application.conf.j2 exists in order to copy to the remote
server.

Description

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #3803 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3803   +/-   ##
=======================================
  Coverage   75.15%   75.15%
=======================================
  Files         132      132
  Lines        6142     6142
  Branches      373      373
=======================================
  Hits         4616     4616
  Misses       1526     1526

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 05fca70...00e64b5. Read the comment docs.


 _\ _\
////  //  ///__///////
Create a new ZanPhp application.
Which type application would you create? (use  to select)
❯ ● HTTP
○ TCP
Your application name: (ex: zanphp-demo)
Use default application name: zanphp-demo
Please input a output directory: zanphp-http
Your composer name: (ex: zanphp/zanhttp)
Use composer name: zanphp/zanhttp
Your application namespace: (ex: Com\Youzan\ZanHttpDemo)
Use default namespace: Com\Youzan\ZanHttpDemo
Downloading the source code archive ...
PHP Warning:  file_get_contents(): SSL operation failed with code 1. OpenSSL Error messages:
error:14090086:SSL routines:ssl3_get_server_certificate:certificate verify failed in /home/ubuntu/.config/composer/vendor/youzan/zan-installer/src/Installer.php on line 319
Warning: file_get_contents(): SSL operation failed with code 1. OpenSSL Error messages:
error:14090086:SSL routines:ssl3_get_server_certificate:certificate verify failed in /home/ubuntu/.config/composer/vendor/youzan/zan-installer/src/Installer.php on line 319
PHP Warning:  file_get_contents(): Failed to enable crypto in /home/ubuntu/.config/composer/vendor/youzan/zan-installer/src/Installer.php on line 319
Warning: file_get_contents(): Failed to enable crypto in /home/ubuntu/.config/composer/vendor/youzan/zan-installer/src/Installer.php on line 319
PHP Warning:  file_get_contents(https://codeload.github.com/youzan/zanhttp-boilerplate/zip/master): failed to open stream: operation failed in /home/ubuntu/.config/composer/vendor/youzan/zan-installer/src/Installer.php on line 319
Warning: file_get_contents(https://codeload.github.com/youzan/zanhttp-boilerplate/zip/master): failed to open stream: operation failed in /home/ubuntu/.config/composer/vendor/youzan/zan-installer/src/Installer.php on line 319
ERROR: Download code fail :(

php version 7.1.6
openssl version 1.0.2l

Maybe something go wrong with your CA certificate, Then we omit CA verify when visiting https.
git pull this repository and retry, thanks

ok

I try pull this repository, have resolve.


Improvements for LIDO

Re: https://github.com/LLNL/serac/issues/774
Here's a summary of some of the issues and which have been addressed so far in this PR (:heavy_check_mark: => addressed, :x: => still working):
:heavy_check_mark: 1. (Bug) Some missing implementation for 0th-order (constant) L2 spaces.
:heavy_check_mark: 2. (QoL) Be more explicit about not supporting L2 spaces on boundary integrals.
:heavy_check_mark: 3. (QoL) User-specified arguments per q-function.

:heavy_check_mark: 3. (QoL) User-specified arguments per q-function.

Before: q-functions had to accept all parameters present in the problem:
SolidFunctional<p, dim, Parameters<This, That, TheOtherThing> > solid_solver(...);
solid_solver.setMaterial([](auto x, auto du_dx, auto this_q, auto that_q, auto the_other_thing_q){
   ...
});

Now, you can write the following instead:
SolidFunctional<p, dim, Parameters<This, That, TheOtherThing> > solid_solver(...);
solid_solver.setMaterial(DependsOn<0, 2>{}, [](auto x, auto du_dx, auto this_q, auto the_other_thing_q){
   ...
});

:heavy_check_mark: 4. (QoL) Clarify these types for specifying loads and tractions
:x: ~5. (QoL) Member functions that require explicit template parameters~ (I'm going to skip this for now, it's a breaking interface change that wouldn't actually provide much benefit)
:heavy_check_mark: 6. (Bug) some boundary integral arrays were being sized incorrectly
:heavy_check_mark: 7. (Bug) update signatures for quantity of interest member function aliases AddVolumeIntegral() and AddAreaIntegral()
:heavy_check_mark: 8. (Bug) fix indexing bug when using vector-valued trial spaces in volume integrals for quantities of interest.
:heavy_check_mark: 9. (QoL) update readthedocs entries for Functional to include expectations of q-function interface for Quantities of Interest

:heavy_check_mark: 10. (QoL) give users an option to bypass setMaterial calls and register a custom q-function to the residual.

Before: material models make a lot of assumptions (materials don't depend on spatial coordinate or displacement vector explicitly, homogeneous mass density in reference configuration, ... )
SolidFunctional<p, dim> solid_solver(...);
solid_solver.setMaterial(NeoHookean{rho, K, G});

Now, there's another option, where you can write something like:
double lambda = ...;
double mu = ...;
solid_solver.addCustomDomainIntegral(DependsOn<>{}, [=](auto x, auto displacement, auto acceleration){
  auto du_dx = serac::get<1>(displacement);

  auto I       = Identity<dim>();
  auto epsilon = 0.5 * (transpose(du_dx) + du_dx);
  auto stress = lambda * tr(epsilon) * I + 2.0 * mu * epsilon;

  auto d2u_dt2 = serac::get<0>(acceleration);
  double rho = 1.0 + x[0]; // spatially-varying density

  return serac::tuple{rho * d2u_dt2, stress};
});

Note: this feature is experimental, and isn't properly tested yet.

For reference: much of the credit for these bugfixes goes to @kswartz92, for uncovering a lot of gaps in our test suite coverage

new contributions to the developer guide (related to "quantities of interest") can be viewed here:
https://serac.readthedocs.io/en/mish2-improvements_for_lido/sphinx/dev_guide/functional.html

some timings comparing compilation of C++ files compiled with CXX language in CMake:
sam@provolone:~/code/serac/build-GCC-Debug/tests$ time make -j
...
[100%] Built target functional_qoi
[100%] Built target functional_comparisons
[100%] Built target functional_nonlinear

real    0m20.359s
user    1m42.744s

as opposed to as CUDA language in CMake
sam@provolone:~/code/serac/build-GCC-Debug/tests$ time make -j
...
[100%] Built target functional_qoi
[100%] Built target functional_comparisons
[100%] Built target functional_nonlinear

real    0m36.634s
user    3m20.530s

Codecov Report

Merging #782 (f99deb2) into develop (07b8ef1) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #782      +/-   ##
===========================================
+ Coverage    94.85%   94.93%   +0.07%
===========================================
  Files          140      140
  Lines         8763     8881     +118
===========================================
+ Hits          8312     8431     +119
+ Misses         451      450       -1

Impacted Files
Coverage Δ

...rac/numerics/functional/tests/functional_basic.cpp
100.00% <ø> (ø)

...rics/functional/tests/functional_boundary_test.cpp
84.53% <ø> (ø)

...erics/functional/tests/functional_multiphysics.cpp
100.00% <ø> (ø)

...numerics/functional/tests/functional_nonlinear.cpp
100.00% <ø> (ø)

...ac/physics/materials/solid_functional_material.hpp
100.00% <ø> (ø)

...rc/serac/numerics/functional/boundary_integral.hpp
100.00% <100.00%> (+3.57%)
:arrow_up:

.../numerics/functional/boundary_integral_kernels.hpp
100.00% <100.00%> (ø)

src/serac/numerics/functional/domain_integral.hpp
100.00% <100.00%> (ø)

...ac/numerics/functional/domain_integral_kernels.hpp
100.00% <100.00%> (ø)

src/serac/numerics/functional/functional.hpp
100.00% <100.00%> (ø)

... and 15 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

are there plans to add a setPressure type method to solid functional? If so, would this PR be the appropriate place to do so?

what's the current status and plan moving forward for supporting parameters on L2 spaces? I'm mostly concerned with material parameters since they are commonly parameterized on L2 in topology optimization

are there plans to add a setPressure type method to solid functional? If so, would this PR be the appropriate place to do so?

Currently, (Piola) pressures can be applied as a special case of the setPiolaTraction member function. We could add a convenience method that wraps a user-specified pressure function and multiplies it by $\hat{n}$. One concern I have is that we end up with a lot of overloads, e.g.

setPiolaTraction(traction_function)
setPiolaTraction(DependsOn<...>, traction_function)
setCauchyTraction(traction_function)
setCauchyTraction(DependsOn<...>, traction_function)
setPiolaPressure(pressure_function)
setPiolaPressure(DependsOn<...>, pressure_function)
setCauchyPressure(pressure_function)
setCauchyPressure(DependsOn<...>, pressure_function)

which makes it more tedious maintain. The other point is that deeply-nesting lambdas may impact performance, but we should collect data before making any decisions on that criterion.

what's the current status and plan moving forward for supporting parameters on L2 spaces? I'm mostly concerned with material parameters since they are commonly parameterized on L2 in topology optimization

L2 spaces are supported for domain integrals. This PR adds support for 0th-order (one value per element) L2 spaces. Getting L2 to work on the boundary integrals is more challenging supported, as mfem seems to believe users shouldn't be able to do this at all. I don't agree with their philosophy, but regardless, it means that we would be on our own for coming up with a hack to get the appropriate information necessary to enable the L2 trial spaces on the boundary.
For reference, the tools we need to enable L2 spaces on the boundary are:

L->E gather operation
E->L scatter-add operation
a way to get the dof ids associated with a given boundary element (must be consistent with the L->E and E->L mappings as well).

see: https://github.com/mfem/mfem/issues/2495

Currently, (Piola) pressures can be applied as a special case of the setPiolaTraction member function.

This is semantics, but to me "pressure"  implies a follower load. That is, pressure is a traction that acts only in the current normal direction. I think this is worth supporting with a specialized function because of the rotation of the normal. I would vote that we don't need a setPiolaPressure() method, if by that you mean "constant times reference normal vector".

Personally, I'm okay with using the traction interface to define pressure loads. This also makes our lido wrappers slightly easier to maintain. I'm guessing most people writing a custom load function would understand how to use the normal. @sethwatts do you have any objections?

No objections. I agree that anyone writing a custom load should be on the hook for understanding how to use normals. Are you intending to support a pressure load as a standard one in the lido wrapper?


Initial Docs Development

Tasks:

[x] Setup gh-pages
[x] Add dev instructions to readme
[x] Backbone / Underscore skeleton
[x] Add sections + TOC
[x] Add intro
[x] Add downloads

Publishing!


Instantiating praw.models.Submission from _data fails on subreddit

Issue Description
When trying to instantiate a Submission class from previously serialized instance (with the models being recursively dumped to JSON as their __dict__, omitting the _reddit attribute), produces the following error:
Traceback (most recent call last):
  File "import.py", line 167, in <module>
    submission = praw.models.Submission(reddit, _data=submission)
  File "lib\site-packages\praw\models\reddit\submission.py", line 116, in __init__
    super(Submission, self).__init__(reddit, _data)
  File "lib\site-packages\praw\models\reddit\base.py", line 53, in __init__
    super(RedditBase, self).__init__(reddit, _data)
  File "lib\site-packages\praw\models\base.py", line 41, in __init__
    setattr(self, attribute, value)
  File "lib\site-packages\praw\models\reddit\submission.py", line 135, in __setattr__
    value = Subreddit(self._reddit, value)
  File "lib\site-packages\praw\models\reddit\subreddit.py", line 355, in __init__
    self._path = API_PATH['subreddit'].format(subreddit=self)
TypeError: __str__ returned non-string (type dict)

This happens because the second-to-last call at (submission.py#L145) assumes the wrong arguments to Subreddit.__init__(reddit, display_name=None, _data=None) defined at (subreddit.py#L385).
This leads to the implicit self.__str__ in the next call returning self.display_name, which instead of a string now contains the whole dictionary intended to be passed to _data.
I know instantiating-from-data is probably a very little used mechanism, but it almost just works.
Fix
The line (submission.py#L145):
value = Subreddit(self._reddit, value)

should be:
value = Subreddit(self._reddit, _data=value)

System Information

PRAW Version: 5.4.0
Python Version: 3.5.2
Operating System: Windows

Thanks for reporting. Could you make a PR please along with any necessary tests?

Sure thing.
After patching this, I noticed a similar issue in Redditor.from_data (redditor.py#L21) also called from the same place.
I'll create some unit tests and make that PR.

You're awesome. Thanks!

I guess this is a bit more complicated than I thought. This "fix" conflicts with how PRAW parses the reddit API.
When PRAW recieves the JSON data from reddit (e.g. a listing from reddit.com/search.json) the "subreddit" and "author" fields are just plain strings. PRAW then instantiates those as bare-bones Subreddit and Redditor models using the calls described above (with the correct argument).

Changing these calls to expect _data dictionaries instead (like I proposed) would break the parsing of reddit responses.
Dispatch based on datatype (string vs. dict) could be done, but in that case the appropriate call should probably be to _objector.objectify instead.
The reddit API currently does not respond with a submission that includes fully populated fields, but if it did, it would probably respond with a redditor {"kind": "t2", "data": {...}} and subreddit {"kind": "t5", "data": {...}} thing dict embedded inside rather than just the plain data dict.

Proposal to close this issue as user error
I was attempting to use PRAW internals in a way they weren't designed to be. I will change my method of serialization for now.
It would be nice if PRAW had a native way of serializing objects back into reddit API compatible JSON (and instantiating/parsing again at a later time). But I guess this would require some substantial re-architecting of the internals.

Closing for inactivity.


nm bridge: Support replacing unmanaged ports

When replacing NM unmanaged linux bridge port with new,
the unmanaged interface is not taken over by NM which lead to
old ports co-exist with new ports after verification stage.
To support that:

Create NM connection even the interface is DOWN as unmanaged
interface is shown as DOWN by NM.
Mark the interface as managed by NM.
Activate the profile for unmanaged interface before other actions.
So the configure could be applied.

Add test case for replacing unmanaged linux bridge ports.

Fixing downstream bug: https://bugzilla.redhat.com/show_bug.cgi?id=1855189

Need PR #1229 to pass the CI.


set nfsServerAddress is empty or delete nfsServerAddress in mysqlcluster.yaml file,auto backup is failed

Describe the problem
set nfsServerAddress is empty or delete nfsServerAddress in mysqlcluster.yaml file,auto backup is failed。the buckup error as follow:
 2022-08-22T08:49:06.140Z	INFO	sidecar	initialize a backup	{"host": "sample-mysql-0.sample-mysql.default:8082", "endpoint": "/xbackup"}

 2022-08-22T08:53:27.999Z	ERROR	sidecar	run command failed	{"error": "fail to get backup: %!s(<nil>), code: 500 Internal Server Error"}

 main.main.func3

 	/workspace/cmd/sidecar/main.go:84

 github.com/spf13/cobra.(*Command).execute

 	/go/pkg/mod/github.com/spf13/cobra@v1.1.3/command.go:856

 github.com/spf13/cobra.(*Command).ExecuteC

 	/go/pkg/mod/github.com/spf13/cobra@v1.1.3/command.go:960

 github.com/spf13/cobra.(*Command).Execute

 	/go/pkg/mod/github.com/spf13/cobra@v1.1.3/command.go:897

 main.main

 	/workspace/cmd/sidecar/main.go:96

 runtime.main

 	/usr/local/go/src/runtime/proc.go:225

To Reproduce

1.set nfsServerAddress is empty or delete nfsServerAddress in mysqlcluster.yaml file
2.kubectl apply -f mysqlcluster.yaml to create cluster
Expected behavior

1、use webhook to  intercept  nfsServerAddress   is empty
2、use webhook to  intercept  nfsServerAddress   is domain name
Environment:

RadonDB MySQL version:2.2.1

add webhook， make empty nfs server address and backup schedue to be error.


Protect get_dir from causing PHP errors

See: https://forum.pfsense.org/index.php?topic=114570.0
Thanks!

Merged to all 3 branches, thanks @NewEraCracker


proposal: x/tools/go/packages: Provide an AST loader for single binary applications

This request describes a need for a new function that provides the same results as the Load() function of the golang.org/x/tools/go/packages package but that allows embedding in a single binary.
This request is based on end-user use cases with UML diagrams, on an implementation of UML diagrams in go that uses the packages.Load() function and on the description of the current limitations of this Load() function.
Uses cases for using UML diagrams at run time
UML diagrams are useful artifacts for documenting code but they can be also useful at runtime as it is shown here with both structural and state UML diagrams.
For UML structural diagram, 2 use cases can be met at run time:

Visualizing the UML structural data model to understand how data is stored and the relationships between data items
Using the UML structural diagram to pinpoint class shapes and access the corresponding table

The following movie illustrates the use case with a toy plane simulation. At time stamp 10'', the end-user opens the UML diagrams describing the simulation model (use case #1) and then he checks the values of the Liner agent with clicking on the Liner class shape (use case #2).
https://vimeo.com/709811836
For UML state diagrams, there is one other case that can be useful when developing simulations of protocols between devices that are statefull agents. In this case, it is interesting to to check the state of each agent during the simulation. The following movie shows such a use case. It is a toy laundromat simulation. The UX provides insight into the state of the washer and the state of the machine.
https://vimeo.com/709811805
Persistence of UML diagrams in go code
In the above use cases, UML diagrams are integral parts of applications developed with the gong framework, a go+angular full stack framework for developing applications for system engineers.
With gong, UML diagrams such a the simple one below are marshaled as go code.

var Test uml.Classdiagram = uml.Classdiagram{
	Classshapes: []*uml.Classshape{
		{
			Struct: &(models.Country{}),
			Position: &uml.Position{
				X: 20.000000,
				Y: 20.000000,
			},
			Width:  240.000000,
			Heigth: 78.000000,
			Links: []*uml.Link{
				{
					Field: models.Country{}.Hello,
					Middlevertice: &uml.Vertice{
						X: 380.000000,
						Y: 129.000000,
					},
					TargetMultiplicity: "0..1",
					SourceMultiplicity: "*",
				},
			},
			Fields: []*uml.Field{
				{
					Field: models.Country{}.Name,
				},
			},
		},
[...]

Load of the UML diagram with the packages.Load() go function
In a gong diagram, every shape references a go element such a struct or a field via a a zero value of the struct of the field of the actual go code.
[...]
			Struct: &(models.Country{}),
[...]
			                Field: models.Country{}.Hello,
[...]
                                         Field: models.Country{}.Name,

This solution brings two benefits:

go compilation provides good confidence that the UML diagram is in sync with the go models
it allows the go refactoring tool (gopls) to refactor the UML diagram while refactoring the code (for renaming a struct or a field).

At run time, an Unmarshall function loads the go code by using the packages.Load() function and then parses the Abstract Syntax Tree to find which UML shapes references which go struct or field
Limitations of the current packages.Load function
In system engineering, it is interesting to ship simulation binaries to interested stakeholders. This is where go is interesting because one great advantage of go is the single binary feature. However, the above simulations cannot be considered single binary for two limitations.
packages.Load makes a system call to the go compiler
In order to be executed, uses cases described above cannot be executed on a machine where go is not installed. Indeed, the packages.Load function makes a system call to the go compiler.
package.Load cannot support embed directories
In order to be executed, uses cases described above needs the go files describing the UML diagram as well as the go model both accessible in local directories. This is because, unless mistaken, one cannot use the embed.FS type in the Dir field of the packages.Config struct that is passed as a parameter to the packages.Load function.
Expected feature
A packages.<name to be defined>() function that does not makes a system call to the go compiler and that allows embedded go files and that returns the AST.

CC @taking @alandonovan

If you mean that packages.Load as it exists today should work without requiring Go to be installed, that generally seems like a bad idea to me. You could potentially do it by embedding the entire Go toolchain and standard library into the binary (hundreds of megabytes!), to then extract it in a temporary directory and use it.
A simpler alternative, if you only need ASTs, is https://pkg.go.dev/go/parser#ParseDir. That does require you to have the Go package already on the filesystem somewhere, but it then means you don't need go/packages at all.

@mvdan , thks for the feedback. I retract the proposal.

You could potentially do it by embedding the entire Go toolchain and standard library into the binary (hundreds of megabytes!), to then extract it in a temporary directory and use it.

That would be doable indeed but complex to implement from my perspective. My idea of the go standard library is to provide functions with most complexity hidden from the programmer. BTW : indicating in the package.Load() documentation that the go tool need to be installed in the execution environnement would be a plus in my opinion.

A simpler alternative, if you only need ASTs, is https://pkg.go.dev/go/parser#ParseDir.

Thank-you for pointing this but the example above needs a bit more than the results of the of parser.ParseDir()parser.ParseDir()` function.
Field: models.Country{}.Hello,

In order to construe the link between the Country shape and the Hello shape, the UML diagram unmarshaller needs to know that the type of the Hello field is of type pointer to Hello struct. The unmarshaller get the type of association fields by  packages.Load().. Maybe I am mistaken but getting the type would not be possible with parser.ParseDir() (the type is within the package).
PS : a workaround is possible with parser.ParseDir() but this is not the proper forum for this discussion

@mvdan , thks for the feedback. I retract the proposal.

Thanks for writing up your interesting use case so clearly.
The root of the problem is that the go list embodies a lot of computation on Go workspaces that it would be infeasible to maintain as a linked library. Also, golang.org/go/packages  is intended to work with other build systems than go, such as Bazel and Blaze.

In system engineering, it is interesting to ship simulation binaries to interested stakeholders.

I would have thought that a stakeholder who cares about the code at the level of UML diagrams would be working in an environment where 'go build' works. But if not, perhaps you could package your program as Linux container image that contains a Go toolchain.

This proposal has been declined as retracted.
— rsc for the proposal review group


Clean up

Tidied code.

Tidied ViewableCart to use new ORM, and removed unused property
Tidied MatchObjectFilter ->getFilter to be a bit more readable.
FIX: Order->debug was showing the items and modifiers lists twice.

This all looks good to me, @jedateach.

@markguinn I'm guessing you're ok with me doing the merge then?
I'm kind of thinking a good practice is that the person who wrote the code should not be the one who merges it.
In this regard, I need #333 looked at for a current project.

No problem, Jeremy. I thought since you hadn't squashed the commits you might be planning to add more to it. Since this is really 3 changes I'd be ok with leaving them as three commits if you are.