Better dependency installing?

A big issue I have with the installation is that the dependency installation takes too much time. How it works right now is that it installs every dependency one-by-one, which takes up a lot of time. In my attempts of installing it on Debian, this can be fixed by installing all of the dependencies in one command. The same thing could be said about Fedora, since it seems that also uses a similar method of installing dependencies one-by-one.

Reading the script, most likely if this were to be implemented then the "|| exit 1" would break and not work.
Another problem with implementing this is that if one package were to become missing or removed, then the entire command would break. But, as of right now that doesn't seem likely to happen.


Pretrained embedding algorithm

From @wetdog, I think?

That's right @Molkree , some of the models are also in tensorflow hub as I see the humpback model listed in other issue.


Wrong method signature of Definition.implements with 4 parameters

The method needs _ on the last two parameters to be consistent with the definition of the same one with 1, 2 and 3 parameters.

Fixed with 35b6da8556ecc725ce770a09587d10cba6676246


Update .gitignore and fix $path for Rojo

Before fix:

After fix:

Forgot to disable Prettier before opening a PR whoops

Thank you!


Splunk 6.5 and accessing git using https

I have this installed in a test 6.5.2 server and made a simple minded chnage to point this to our local git repo with a https URL - before I start debugging this do you have any advice or suggestions about how to make this work?
splunk cmd ./fetch_git_repo_data.sh
repository does not exist!
fetching git repo data for repository<EMAIL_ADDRESS>Cloning into bare repository '/opt/splunk/etc/apps/splunk-app-splunkgit/git-repositories/foo'...
fatal: unable to access<EMAIL_ADDRESS>Failed connect to server.foo.comt:443; Connection timed out
Unable to clone repository<EMAIL_ADDRESS>repository does not exist!

Fixed  - sort of - use a proxy to access our local git repo, and cache the login credentials so that git fetech does not prompt for them.


DokanNet NuGet dokan.dll error

Dokan version = v0.7.3-beta
DokanNet version = v<IP_ADDRESS> (via NuGet)
When you call mount on the IDokanOperations instance the following error occurs:
An unhandled exception of type 'System.EntryPointNotFoundException' occurred in DokanNet.dll

Additional information: Unable to find an entry point named 'DokanMain' in DLL 'dokan.dll'.

I then downloaded the dokan-dotnet master branch and compiled the dokannet locally and then ran the sample successfully; however, the reference via the NuGet package still does not work.

Actually, this has nothing to do with the NuGet package. It is occurs when using the 4.5.1 .NET framework.

I believe it's because your using a 64-bit operating system, build your application in AnyCPU but with "Prefer 32-bit" option checked? But in that case it should had use the 32-bit dokan.dll automatically.

I am using a 64-bit OS. I modified the build to use the "prefer 32-bit" option; however, the same error occurred.
Are the 32-bit assemblies always installed? I used the release installer and I seem to remember the options were all disabled, with being 32-bit unchecked.


add http body error and code in report or error log

Implemented on 1.0.2 version

Implemented on 1.0.2 version


Show chess leaderboard

Hi!
This is the current result

I also created two options:

category: string, the category to be shown. The options are: blitz, bullet, rapid. If not specified, then all of them are shown
top-n: int, the number of top players shown. Min is 1, max is 10. If not specified, then default is 5.

Done 👍

Great work @jeremiaaxel!
Keep contributing ❤


Warning on Standard Contract deploy on mainnet

Issue To Be Solved
Many new comers accidentally deploy FungibleToken, NonFungibleToken, NFTStoreFront contracts to their accounts, either because of a mistake or totally because they are uninformed.
Suggest A Solution
We can show a warning and prompt for confirmation in a case like this

What would the warning be? Something like "it's advised you use already deployed contract ... read more about it here"

It seems like you are trying to deploy %s to mainnet, it is a standard contract already deployed at address 0x%x. You can read more about it here: %s. Do you still want to deploy the contract (y/n)
I link we have already links for standard contracts at docs.

That would be nice, I agree 100%.

I'm not a fan of lists of contracts, I feel that should be ideally decentralised and saved in such registries, so we don't special treat any contracts and control that ourselves, but that road is longer and I think it might be ok to do that meanwhile.

@sideninja I did the most of the work here, but stuck on something.
Considering project deploy flow:

Should we replace the state reference with alias ?
As we are not exposing direct access to state, should I add some helpers to Project, such as "ContractsToDeploy()andReplaceContractWithAlias` ? or is there a better place for this?

Should we replace the contract reference in state with alias ?

This is a good idea yeah.

As we are not exposing direct access to state, should I add some helpers to Project, such as "ContractsToDeploy() and ReplaceContractWithAlias` ? or is there a better place for this?

Where are you trying to access state? In the project service you should be able to access the state directly.

Because I want to prompt the user ( if they want to deploy or replace with alias ), so best option I guess:

to do the check and prompt user ( on CLI side )
depending on user answer, do the fix / or ignore
deploy via flowkit project

So this flow will need access to state ( or some helpers accessing state as I mentioned )

Because I want to prompt the user ( if they want to deploy or replace with alias ), so best option I guess:

to do the check and prompt user ( on CLI side )
depending on user answer, do the fix / or ignore
deploy via flowkit project

So this flow will need access to state ( or some helpers accessing state as I mentioned )

You could do the prompt in the project service and then based on the answer you can proceed. In the future, we will refactor prompts out of the flowkit and just accept a prompt resolver as an argument, that way you could define the prompt in the command layer.

@sideninja managed with prompt ui on project, but just one more question remains about where to save the config ( due to cascading -f edge cases ) or I can skip saving totally ?

@bluesign we currently don't yet support saving to multiple files as per https://github.com/onflow/flow-cli/blob/7401e49afd2d7b89e15dc8f42c75fcb279f4e60f/pkg/flowkit/state.go#L79
There's a plan to make this work in the future once we find some time.


confirm link points to "#null"

Is there any reason for this? I see that in previous version you get just "#" which is fine :)

If the confirm link pointed to #, the user would be sent to the top of the page when they clicked it. #null allows us to avoid this as (most) pages don't have an #null anchor on them.

#8 should have stopped #null from appearing in the url bar

great, tnx :)

Can we make a release with this fix please? #null on url looks ugly indeed.

https://github.com/silktide/cookieconsent2/tree/1.0.3

Wow - that was quick. Thank you :)


Fallback no longer working if using vim.keymap.set

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Issues

[X] I have checked existing issues and there are no open or closed issues with the same problem.

Neovim Version
NVIM v0.7.0-dev+897-g0c0469c4f
Minimal reproducible config
if has('vim_starting')
  set encoding=utf-8
endif
scriptencoding utf-8

if &compatible
  set nocompatible
endif

let s:plug_dir = expand('/tmp/plugged/vim-plug')
if !filereadable(s:plug_dir .. '/plug.vim')
  execute printf('!curl -fLo %s/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim', s:plug_dir)
end

execute 'set runtimepath+=' . s:plug_dir
call plug#begin(s:plug_dir)
Plug 'hrsh7th/nvim-cmp'
Plug 'hrsh7th/cmp-buffer'
Plug 'hrsh7th/cmp-nvim-lsp'
Plug 'hrsh7th/vim-vsnip'
Plug 'neovim/nvim-lspconfig'
call plug#end()
PlugInstall | quit

" Setup global configuration. More on configuration below.
lua << EOF
local cmp = require "cmp"
cmp.setup {
  snippet = {
    expand = function(args)
      vim.fn["vsnip#anonymous"](args.body)
    end,
  },

  mapping = {
    ['<CR>'] = cmp.mapping.confirm({ select = true })
  },

  sources = {
    { name = "nvim_lsp" },
    { name = "buffer" },
  },
}

vim.keymap.set("i", "<CR>", function() print("hello") end, {})
EOF

lua << EOF
local capabilities = require('cmp_nvim_lsp').update_capabilities(vim.lsp.protocol.make_client_capabilities())

require'lspconfig'.cssls.setup {
  capabilities = capabilities,
}
EOF

Description
Maybe related to this issue? https://github.com/folke/which-key.nvim/pull/223
I'm using https://github.com/RishabhRD/lspactions#input and <CR> and <C-c> does not work because I'm also using <CR> for cmp.mapping.confirm and <C-c> for cmp.mapping.close
Steps to reproduce
Type  in insert mode
Expected behavior
Should print hello
Actual behavior
Does not print hello
Additional context
https://github.com/hrsh7th/nvim-cmp/commit/703e915b7f0b673a37c8b92e019bd8a5d369d4e9 This commit seems to be the culprit. Fallback works with previous commit https://github.com/hrsh7th/nvim-cmp/commit/243d5f4a120580ebb171484b5a3fe15d06cf8fae

I think this is <silent> related issue. I'll check it..

@hrsh7th The <Tab>-completion failed again after c23d810


[Storage Blob Service] Set Blob Service Properties Support in storage SDK

https://docs.microsoft.com/en-us/rest/api/storageservices/set-blob-service-properties

done


[Feature] Use pure github action commans for website sync

Currently the docs website automatically sync by a outer action: https://github.com/oam-dev/kubevela/blob/master/.github/workflows/release-docs.yaml#L15
In fact, this job could be done by the script just as the docs check action https://github.com/oam-dev/kubevela/blob/master/.github/workflows/check-docs.yml#L19

Note, we should also keep some infrequently changed config files like  index.js, docusaurus.config.js keep in repo kubevela.io. only overwrite frequently changed files.


Remove foceUpdate(callback()) in stateAccessor setState implementation

Maybe have a second callback like onUpdate() which is only called when the React update is guaranteed to have completed?


Booked Successfully.

Used Docker method to book my appointment.
Bro i have some suggestion..

Make otp verification before finding slot. And put a timer for 15 minutes to regenerate OTP.  Because after finding the available slot it take few seconds to submit otp and in that few seconds the slot gets over. So make OTP generation during submit information.

After submiting OTP.. fetch beneficiary REF ID and make a drop down box or check box to select beneficiary. It will make more easier for user to use.

Rest all is fine. Thanks for your efforts and helping people to get their slot booked.

Hi
I am glad that this helped you to book an appointment.
About your suggestions, there are other scripts which offer full automation but those require a lot of initial setups and it comes with it own pitfalls. I wanted to keep it simple. Though I am thinking of automating OTP as well with bare minimum setup. There are existing SMS forwarding apps that can be used to forward SMS to email and then having email monitoring at the server-side would help . To poll the availability we really don't need to log in. The more we try to automate the more complications we add. This is why I am keeping it simple.

@sunilgoudar
I did few improvements recently. There are two modes available now i.e Auto and Manual. In Auto mode everything is automated. Unlike other booking scripts/utilities, the user does not need to feed otp every now and then. You can go through the documentation about how it works.


Needs clarification on how folderpath contains the container name, and how to mitigate.

You can reference MS ticket<PHONE_NUMBER>010185 for more.
The documentation here talks about folderpath and triggers. However, there's a fatal flaw if you're trying to piece things together.
The trigger folderpath variable includes the container name. The dataset to import into SQL Server also includes the container name. so when you put them together, if you're dumping into a container called testa and a folder called testb, your trigger calls fail because it's trying to load from storageaccountname\testa\testa\testb.  The fix is, and I wish I were kidding, is to use
@substring(triggerBody().folderPath,add(indexof(triggerBody().folderPath,'/'),1),sub(length(triggerBody().folderPath),add(indexof(triggerBody().folderPath,'/'),1)))
instead of
@triggerbody().folderPath

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

ID: b7ee5166-ff40-f4ed-ce07-5312f935e388
Version Independent ID: 68f01b43-e3b2-003b-24cb-446a08aa0c1b
Content: Create event-based triggers - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/how-to-create-event-trigger.md
Service: data-factory
Sub-service: orchestration
GitHub Login: @chez-charlie
Microsoft Alias: chez

@mbourgon
Thanks for your feedback! We will investigate and update as appropriate.

@mbourgon
Thank you for bringing this to our attention.
I've delegated this to content author @chez-charlie, who will review it and offer their insightful opinions.

@chez-charlie
Could you please review add comments on this, update as appropriate.

Thanks @mbourgon for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Every ADR should have a corresponding article / paragraph / something in our Docs

https://github.com/microsoft/kiota/issues?q=label%3AADR+is%3Aclosed

Opened: #70, #71, #72
Everything before the date of this comment has been documented besides:

[ ] https://github.com/microsoft/kiota/issues/3566
[ ] https://github.com/microsoft/kiota/issues/2367
[ ] https://github.com/microsoft/kiota/issues/2085


Installation broken

Describe the bug
Installing onu-ui via the command provided in the getting started guide fails due to unpublished package versions:
pnpm i -D @onu-ui/preset
 ERR_PNPM_NO_MATCHING_VERSION  No matching version found for<EMAIL_ADDRESS>
This error happened while installing the dependencies of<EMAIL_ADDRESS>
The latest release of @onu-ui/utils is "1.0.14-beta.2".

If you need the full list of all 31 published versions run "$ pnpm view @onu-ui/utils versions".
Progress: resolved 641, reused 549, downloaded 0, added 0

Reproduction
pnpm i -D @onu-ui/preset
System Info
No response
Used Package Manager
pnpm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a OnuUI issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] The provided reproduction is a minimal reproducible example of the bug.

You can try to install：
pnpm add https://pkg.pr.new/onu-ui/onu-ui/@onu-ui/preset@216

@onu-ui/preset currently does not have many components, it can only provide you with a simple preview.

You can try to install：
pnpm add https://pkg.pr.new/onu-ui/onu-ui/@onu-ui/preset@216

@onu-ui/preset currently does not have many components, it can only provide you with a simple preview.

But it
> pnpm add https://pkg.pr.new/onu-ui/onu-ui/@onu-ui/preset@216
 ERR_PNPM_FETCH_404  GET https://pkg.pr.new/onu-ui/onu-ui/@onu-ui/preset@216: Not Found - 404

This error happened while installing a direct dependency of /home/elen/Some

No authorization header was set for the request.
Progress: resolved 43, reused 43, downloaded 0, added 0

Sorry for that, i create quickly release. Welcome to preview!


Better way to revoke the JWT internally

Actually when the discord-bot is making API requests it uses a JWT, which lasts for 1 hour. The bot is meant to be runned continually without interruption, this is where we need to revoke a JWT otherwise the bot won't be able to update data (i.e. a guild prefix).
There is an interval already setup which runs every 30 minutes to generate a new JWT, but instead we should use Axios's interceptors on a 401 failed response since it's the proper way to do it.

Already done by the architecture branch.


Magic Imports  - able to call tasks from files not included in task file

Environment
-tasks
-- magic-import.yaml
-- publish.yaml
tasks.yaml
#tasks/magic-import.yaml
tasks:
  - name: run-magic-root-task
    description: Build and publish the packages
    actions:
      - task: magical-working-task
  - name: magic-task
    description: run
    actions:
      - cmd: echo "I shouldnt work"

#tasks/publish.yaml
variables:
  - name: TARGET_REPO
    default: "oci://ghcr.io/defenseunicorns/packages/uds"

tasks:
  - name: publish
    description: Build and publish the packages
    actions:
      - description: Create the packages
        cmd: echo ${TARGET_REPO}
  - name: package
    description: Build and publish the packages
    actions:
      - description: Create the packages
        cmd: |
          set -e
          ZARF_ARCHITECTURE=amd64 uds run create-package --set FLAVOR=${FLAVOR}
          if [ ${FLAVOR} != "registry1" ]; then
            ZARF_ARCHITECTURE=arm64 uds run create-package --set FLAVOR=${FLAVOR}
          fi
  - name: package2
    description: Build and publish the packages
    actions:
      - description: Create the packages
        cmd: |
          set -e
          ZARF_ARCHITECTURE=amd64 uds run create-package2 --set FLAVOR=${FLAVOR}
  - name: package3
    description: Build and publish the packages
    actions:
      - description: Create the packages
        cmd: |
          set -e
          ZARF_ARCHITECTURE=amd64 uds run create-package3 --set FLAVOR=${FLAVOR}
  - name: package4
    description: Build and publish the packages
    actions:
      - description: Create the packages
        cmd: |
          set -e
          ZARF_ARCHITECTURE=amd64 uds run create-package4 --set FLAVOR=${FLAVOR}

#tasks.yaml
includes:
  - publish: ./tasks/publish.yaml
  - magic-import: ./tasks/magic-import.yaml
  - package: https://raw.githubusercontent.com/defenseunicorns/uds-common-tasks/v0.2.0/tasks/create.yaml

variables:
  - name: TARGET_REPO
    default: oci://ghcr.io/zachariahmiller/packages/uds

tasks:
  - name: var-no-worky
    actions:
      - task: publish:publish

  - name: magical-working-task
    actions:
      - cmd: echo "I'm working but i really shouldnt be"

  - name: create-package
    description: Create the Gitlab runner package
    actions:
      - cmd: echo "Parent task"
  - name: create-package2
    description: Create the Gitlab runner package
    actions:
      - task: magic-import:magic-task

  - name: create-package3
    description: Create the Gitlab runner package
    actions:
      - task: publish:package3

  - name: create-package4
    description: Create the Gitlab runner package
    actions:
      - task: publish:package2

Steps to reproduce
To test run uds run -f tasks/magic-import.yaml run-magic-root-task
tasks/magic-import.yaml is locally included in the root task
This says ERROR:  Failed to run action: task name magical-working-task not found
That is expected behavior. Now run uds run -f tasks/publish.yaml package --set FLAVOR=upstream
Expected Result:
An error about there being no task called create-package as it is not explicitly imported
Actual Result:
package gets created
Then also try uds run -f tasks/publish.yaml package2 --set FLAVOR=upstream
This also shouldnt work but returns:
     I shouldnt work
  ✔  Completed "echo "I shouldnt work""

Then try uds run -f tasks/publish.yaml package3 --set FLAVOR=upstream
That goes in some circular death loop, which makes sense.
Then try uds run -f tasks/publish.yaml package4 --set FLAVOR=upstream
This works and also shouldnt:
  ⠋  Waiting for "echo "I shouldnt work"" (no timeout)
     I shouldnt work
  ✔  Completed "echo "I shouldnt work""

  ✔  Completed "Create the packages"
  ✔  Completed "Create the packages"

will have to do some follow up testing but im not sure this is still an issue. I had to add create imports into multiple publish tasks that failed recently. 😅


Investigate using user/group sync connector for Okta

Refs #2016

This was finished in the following PRs:
https://github.com/firezone/firezone/pull/3614
https://github.com/firezone/firezone/pull/3647


CDB: fix debug output

It doesn't really help to clear a variable before printing it to the debug log.

LOL Good find! 👍
utACK

My concern here is that CountByIP is iterating over a vector of MN objects, to which objects can be added or deleted, without holding a lock on CMasternodeMan::cs.  This can cause a crash due to accessing invalid memory.
I realize this commit didn't create the iterator behavior but since the function is being significantly modified here I think the locking problem should be fixed.
Also since it looks like the GUI is typically calling this function multiple times in sequence rather than locking for each access it might make sense to have a function that returns or fills a structure containing all the node type information at once and under a single lock call.

@tgflynn 4 first commits are unrelated mess, only the last one matters ;)


Vercel is not deploying to discoverbos.org

After migrating this repository from one Vercel account to the nearbuilders, this error came up.

Though I've detached the domain and deleted the previous Vercel project, the new Vercel project still reports that the domain is already in use by another Vercel account.

Opened a case with Vercel Support. Case #00163510

Finished. Needed old team to transfer to nearbuilders team.
https://www.discoverbos.org/


Web component: Card - With inverse enabled, on hover background changes to g10/white.

Detailed description
Issue: On Card - With "inverse" option enabled, hover on the card,  background changes to g10/white.
Additional information

Expected:

@RobertaJHahn this has been fixed


version 4.0: High severity security issue due to outdated dependency

Regular Expression Denial of Service (ReDoS)
Vulnerable module: fresh
Introduced through<EMAIL_ADDRESS>Detailed paths
Introduced through<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>Remediation: Upgrade to<EMAIL_ADDRESS>https://snyk.io/test/npm/pouchdb-server?severity=high&severity=medium&severity=low
Seems easy to fix :-)

Great to see its fixed, but will there be a version 4.0.1 or something similar on NPM ?


Yiiboot\ActiveRecord\ActiveRecord::populateRelation() not support the scalar type

What steps will reproduce the problem?
When I implemented an aggregation query, the return value of the aggregation query was a scalar type. When I used populateRelation to populate the value, I found that it was not supported.
example:

class User {
    public function getTagCount(): ActiveQuery {
        // ....
    }
}

$model = $arFactory->create(User::class);
$model->populateRelation('tagCount', 10);

What is the expected result?
echo $model->tagCount; // output 10

What do you get instead?
 Yiisoft\ActiveRecord\ActiveRecord::populateRelation(): Argument #2 ($records) must be of type Yiisoft\ActiveRecord\ActiveR
  ecordInterface|array|null, int given, called in ....

It is hoped that the type of the parameter $records of the method populateRelation of Yiisoft\ActiveRecord\ActiveRecordInterface can support the scalar type.
// before
public function populateRelation(string $name, array|self|null $records): void;
// after or other better implementations
public function populateRelation(string $name, array|self|string|int|float|bool|null $records): void;

Additional info

Q
A

Version
1.0.?

PHP version
8.2

Operating system
Windows11

Relations in DB means relations between table records (one to one, one to many, many to one, many to many).
Perhaps the issue is about a result of aggregate function COUNT().
In this case you can do something like this
class User {
    private int|null $tagCount = null;

    public function getTagCountQuery(): ActiveQuery
    {
        // ....
    }

    public function getTagCount(): int
    {
        if ($this->tagCount === null) {
            $this->tagCount = $this->getTagCountQuery()->scalar();
        }

        return $this->tagCount;
    }

    public function setTagCount(int $tagCount): void
    {
        $this->tagCount = $tagCount;
    }
}

$model = $arFactory->create(User::class);
$model->setTagCount(10);

I simplified the aggregate query in yii2, but encountered this problem when migrating the code of yii2 to yii3. In yii2, populateRelation has no type, so it supports scalars.
I defined a property Aggregate to collect the relationships of those aggregate queries, and then overridden populateRelation to be able to convert the results of the aggregate query to scalar or extract only part of the data when populating data. This is indeed no longer consistent with the relationship in the DB. It is good to keep it as it is now. I will try other solutions.
#[\Attribute(\Attribute::TARGET_METHOD)]
class Aggregate
{
    public const Scalar = 0;
    public const One= 1;
    public const All= 2;

    public array $filter = [];

    public function __construct(array|string $filter = [], public int $type = Aggregate::Scalar)
    {
        $this->filter = (array)$filter;
    }

   // ....
}

class User {

   use AggregateActiveRecordTrait;

   public function getTags(): ActiveQuery {
       return $this->hasMany(Tag::class, ['user_id' => 'id']);
   }

   #[Aggregate('count')]
    public function getTagCount: ActiveQuery
    {
       return $this->getTags()->select(['user_id', 'count' => 'count(*)'])->groupBy(['user_id']);
    }
}

$query= $arFactory->createQueryTo(User::class);
$users = $query->with('tagCount')->all();

foreach($users as $user) {
    echo $user->tagCount; // echo every tagCount only by once tag count query
}


Bug: Disabling this resource fails (local.create_role)

Describe the Bug
If you set this module to enabled = false terraform will fail to plan. This is due to this line:
https://github.com/cloudposse/terraform-aws-eks-node-group/blob/master/main.tf#L59
The problem is that when enabled = false this tries to lookup a resource in a list that doesn't exist.
A workaround exists where you can set node_role_arn = ["any random string"] to make terraform plan.
Expected Behavior
The resource is cleanly removed as expected without a workaround.
Steps to Reproduce
Steps to reproduce the behavior:

Create this module somewhere.
Try to remove it with enabled = false
Run a terraform plan.

Screenshots
If applicable, add screenshots or logs to help explain your problem.
╷
│ Error: Invalid index
│
│   on .terraform/modules/eks_ephemeral_20210901.eks_node_group/main.tf line 59, in locals:
│   59:     node_role_arn = local.create_role ? join("", aws_iam_role.default.*.arn) : var.node_role_arn[0]
│     ├────────────────
│     │ var.node_role_arn is empty list of string
│
│ The given key does not identify an element in this collection value: the collection has no elements.
╵

Environment:

OS: Ubuntu
Version: 20.04

This probably has to look like
node_role_arn = local.create_role ? join("", aws_iam_role.default.*.arn) : (
  var.enabled ? var.node_role_arn[0] : ""
)

encountered the same issue, the fix worked for me it would be nice to see it merged


Log to Sentry when autoUpdater thrown an error

Fixes ooni/probe#1068
When electron-updater throws an error, we log it to Sentry in the error event handler.

Documented a follow up issue to this here: https://github.com/ooni/probe/issues/1089


SIGINT-1739: Workflow simplification Issues

log message when we are ignoring PR comment, Fix PR, SARIF etc in irrelevant context. Eg. PR comment is ignored in non PR comment context.


Android flavors_test is failing, "app-paid-release.apk" does not exist

Started on https://github.com/flutter/flutter/pull/114048
[flavors_test] [STDOUT] stderr: "build\app\outputs\flutter-apk\app-paid-release.apk" does not exist

https://ci.chromium.org/p/flutter/builders/staging/Windows_android flavors_test_win/1677
https://ci.chromium.org/p/flutter/builders/staging/Linux_android flavors_test/1
cc @a-wallen

Looks like this should have been marked unflaky in August, that would have caught it: https://github.com/flutter/flutter/issues/107681#issuecomment-1208556298

So if I'm reading this right, we have valid tests that are marked flaky and thus not blocking the tree that were never updated to accomodate test file changes? Should we just fix forward then, rather than revert (since it seems like this isn't a problem with the prod code, just the test code)?

Linux_android flavors_test was marked flaky in https://github.com/flutter/flutter/pull/114048 since that PR changed the test.  Needed to actually follow up that it's passing in prod though...
Looks like the test should be installing the debug version, since that's what was just built, not the release version.


Straight quotes are automatically replaced by curly quotes

Example:
pay online
Gets translated into:
pay online
Which browsers don't recognize as valid quotation marks and wrap them around straight quotes making the links and css style not work in this case.
I am unable to find an option to disable this behaviour (using Poedit for Mac)

Oh my bad, I honestly thought I looked everywhere. Thank you for your reply and helping me out.


API versioning

We should think about API versioning so that clients can still interact with old federation versions.

Note: there's going to be more variety and constrains (like binary size) on the clients, so we should probably push as much burden (including supporting multiple versions) on the server side as possible to make lifecycle of clients easier.

Duplicate of #1548


example inputs.ipynb doesn't run with v0.13.0

examples/src/inputs.ipynb
doesn't run with v0.13.0, since input function extForce_t(t) doesn't match the proposed input function structure (at least that's what the error says)...
ERROR: AssertionError: The given input function does not fit the needed input function pattern for FMUs, which are:
- `inputFunction!(t::fmi2Real, u::AbstractArray{fmi2Real})`
- `inputFunction!(comp::FMU2Component, t::fmi2Real, u::AbstractArray{fmi2Real})`
- `inputFunction!(comp::FMU2Component, x::Union{AbstractArray{fmi2Real,1}, Nothing}, u::AbstractArray{fmi2Real})`
- `inputFunction!(x::Union{AbstractArray{fmi2Real,1}, Nothing}, t::fmi2Real, u::AbstractArray{fmi2Real})`
- `inputFunction!(comp::FMU2Component, x::Union{AbstractArray{fmi2Real,1}, Nothing}, t::fmi2Real, u::AbstractArray{fmi2Real})`

Correct, we are currently on restructering the documentation, some of the other examples might fail as well.


Save the captured image for upload to server

Is there a way to save the capture media as file?

What do you mean by "lower than original"?
I didn't experience this problem, but I believe that the quality of the final image may vary depending on the resolution size.
As for the image size, you can try lowering the quality in (bitmap.compress(Bitmap.CompressFormat.PNG, 90, outputStream). You can also try changing the format to JPEG, but it will probably result in a image with a slightly reduced quality.

The library is only intended for capturing composable content into bitmap. With file IO operations, you can save bitmap.
See example: https://github.com/PatilShreyas/NotyKT/blob/master/noty-android/app/src/main/java/dev/shreyaspatil/noty/utils/FileUtils.kt
Closing this.


Fix failed GitHub actions pipeline

Upgrade github actions dependencies to the newest
Use ubuntu-latest for building sphinx documentation.

Codecov Report

Merging #75 (602f035) into dev (b58b8a6) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##               dev       #75   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          181       181
=========================================
  Hits           181       181

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


Makefile: update VERSION to 1.3.4

Signed-off-by: Jens Freimann<EMAIL_ADDRESS>

/override ci/prow/sandboxed-containers-operator-e2e


Anonymous mode glitch

When i was in anonymous mode i got "ping" but the @ wasn't mine

Go in anonymous mode
Wait for a random ping

https://i.imgur.com/WnmviGy.jpg
Chatterino Nightly 2.2.3-beta (commit 0eed45ae6, )
Windows 10

Duplicate of #2156


Provide meaningful exception when PageObject is lacking parameterless constructor

Current: when PO is missing parameterless constructor, MissingMethodException is thrown.
Expected: when PO is missing parameterless constructor, User Friendly exception should be returned. Exception should contain possible solution.

https://github.com/CosminSontu/SpecDrill/commit/ceeea0ce70f1802c406f5b50af77b299acf708c4


[RFC] Deprecate the Slice method of DMatrix.

This is a proposal that we delegate data slicing to external libraries.
** Motivation
DMatrix is an internal data interface, used for all XGBoost algorithm.  The DMatrix object has a few different representation, based on which some more variants can be generated at runtime.  Currently we have 3 different base representation for DMatrix, namely a CSR like SparsePage, a Ellpack sparse matrix for storing GPU histogram index, and lastly external memory batched support for above two in-memory formats.
The Slice method for DMatrix is mostly used for gridsearching where we need to split data for folds.  But the implementation is rather insufficient for many use cases as it supports only the SparsePage and without shuffling support, not to mention distributed environment support.  Also when used with ranking, it doesn't maintain the group_ptr_, which is delegated to a higher level Python wrapper script, resulting in an inconsistent support status for different language bindings.
** Moving forward
On the Python side, we have very good libraries of grid searching in scikit-learn and I believe it's not necessary for XGBoost to maintain a specialized implementation.  Also, we just added an inplace prediction function which can be extended with caching capability.  So performance is not of major concern.  We can easily delegate data slicing to external data inputs like numpy or cudf where slicing is implemented efficiently.
** Possible difficulty
Language bindings other than Python might have issue on supporting grid searching.  Currently mkfold is implemented on both R and Java.  While the slice method is implemented in R/JAVA/Scala .  Also I would very much like to keep compatibility with Julia/Go/Ruby bindings that are out of tree.  So the problem is we need to figure out what's canonical way of doing grid searching in all these bindings and try to integrate with them.

+1 delegating slicing to other libraries will make our lives much easier if it can be achieved with reasonable performance.

How does Dask ML deal with grid search and cross validation?

@hcho3 It's a problem that I want to resolve in next release, along with early stopping.  We need specialized implementation for dask grid searching, I'm hoping that we can implement something on dask-ml instead of XGBoost.

@trivialfis That's good to hear. We'll need a performant method for handling cross validation. For example, I hope that we would not duplicate elements when we create a 5-fold split of training data.

Not sure if it's useful https://www.r-bloggers.com/r-setup-a-grid-search-for-xgboost/ .

@trivialfis : I stumbled over this issue while writing a split method for xgb.DMatrix objects in R.
I am curious about the status on slice and xgb.cv? Early stopping is a core element of model tuning. I don't think that standard ML packages like Scikit-Learn support a clean, round-by-round evaluation of early stopping during cross-validation With such packages, each cross-validation fold would get its "own" optimal number of boosting rounds instead of a "common" tree count (because their folds do not communicate with each other). Thus said, I would really miss slice and xgb.cv without good alternatives.
What could be interesting: We could add an optional argument valid_ind to xgb.train(). It would specify a vector of row indices of data used for validation. All other rows in data would be used for training. The necessary slicing would be applied internally, hopefully solving memory duplications. And getting rid of slice.

@mayer79 Thanks for the detailed explanation.  The issue is updated.  I recently toke a closer look into the cv implementation in xgboost and discussed related issues with @hcho3 .  The cv is likely to stay along with all the features, now I want to move the slicing into 3 party libraries from DMatrix.  Using slice method from general data libraries can help reducing both memory usage and complexity in DMatrix.  Sorry for the confusion, I should have taken more time before opening an issue.

@trivialfis : Thanks a lot for clarification. Looking forward to the slice-free CV! Let me know if help is needed with the R Code.


references: Cache the reverse import graph

We compute this every time we answer a references query and for large workspaces
it dominates the time. However, it is static W.R.T. the workspace. In the future
when we start doing cache invalidation, this will need to be updated.
On my machine it takes subsequent references queries from 20s to 3s.
Part of #11

I assume this change is already covered by existing tests.

yup


feat: added all links related to DSA course

Issue #53 has been fixed
Added all links related to the bootcamp :

YouTube link of the Full course playlist
Link to the repository for hands on practice of git
Link to the repository of the whole syllabus and lectures cheatsheet

Command to check the links out : cm!dsa

It is showing that my code is indented properly

@Ciggzy1312 I went to your repo. It still shows it's not indented. Look here

Now check

It is showing indented in my repository :)


[dvsim] Additional simulation env Altair - Changes to nc to improve stability improvements and cleanup

multiple updates to improve stabilty and cleanup
remove using SNAPSHOTS to improve stability and performance
remove  post_cmd feature of nc_run which is not being used for any real purpose
move to using shared file descriptors across launcher and subprocesses that spawn off nc calls to avoid log file clobbering / collision and improve stability

Hi @venkatk-ot . I've assigned this to @rswarbrick but to assist with reviews - could you please add the overall context in the title - e.g. I think this is a continuation of adding addition simulation environments, so would suggest adding to the title - 'Additional simulation env Altair - Changes to nc to improve stability improvements and cleanup'.

Hi @venkatk-ot . I've assigned this to @rswarbrick but to assist with reviews - could you please add the overall context in the title - e.g. I think this is a continuation of adding addition simulation environments, so would suggest adding to the title - 'Additional simulation env Altair - Changes to nc to improve stability improvements and cleanup'.

Done

@rswarbrick , sorry for the delay in my response.
I wanted to do a little more testing before I uploaded the latest patch
As mentioned in one of the reply above, I have deleted 2 commits and added 2 new commits to this PR.
I think I have addressed all your comments above
Please take a look at the new changes


UnboundLocalError when using dynamic analysis

Some EVM bytecode inputs result in an UnboundLocalError exception. For example:
$ python octopus_eth_evm.py -g -r "6080604052348015600f57600080fd5b506004361060465760003560e01c80632f576f2014604b57806381b36467146067578063a0ef91df14606f578063bcb4ab0e146077575b600080fd5b6051607f565b6040518082815260200191505060405180910390f35b606d6088565b005b60756094565b005b607d60f3565b005b60008054905090565b6001600014609257fe5b565b3373ffffffffffffffffffffffffffffffffffffffff166108fc3073ffffffffffffffffffffffffffffffffffffffff16319081150290604051600060405180830381858888f1935050505015801560f0573d6000803e3d6000fd5b50565b600160008190555056fea165627a7a7230582097b073d8789a43fedeb6be6929b92747b0ddbcfd568d5c737d90ea40ce8efe9f0029"
Traceback (most recent call last):
  File "octopus_eth_evm.py", line 186, in <module>
    main()
  File "octopus_eth_evm.py", line 164, in main
    octo_cfg = EthereumCFG(octo_bytecode)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/cfg.py", line 9, in __new__
    return EvmCFG(bytecode, analysis=evm_analysis)
  File "/Users/bernhardmueller/Projects/octopus/octopus/arch/evm/cfg.py", line 160, in __init__
    self.run_dynamic_analysis()
  File "/Users/bernhardmueller/Projects/octopus/octopus/arch/evm/cfg.py", line 173, in run_dynamic_analysis
    emul.emulate()
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 122, in emulate
    halt = self.emulate_one_instruction(instr, state, depth)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 168, in emulate_one_instruction
    halt = self.ssa_stack_memory_storage_flow_instruction(instr, state, depth)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 433, in ssa_stack_memory_storage_flow_instruction
    self.emulate(new_state, depth=depth + 1)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 122, in emulate
    halt = self.emulate_one_instruction(instr, state, depth)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 168, in emulate_one_instruction
    halt = self.ssa_stack_memory_storage_flow_instruction(instr, state, depth)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 400, in ssa_stack_memory_storage_flow_instruction
    self.emulate(new_state, depth=depth + 1)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 122, in emulate
    halt = self.emulate_one_instruction(instr, state, depth)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 148, in emulate_one_instruction
    self.emul_comparaison_logic_instruction(instr, state)
  File "/Users/bernhardmueller/Projects/octopus/octopus/platforms/ETH/emulator.py", line 261, in emul_comparaison_logic_instruction
    instr.name, args=args)
UnboundLocalError: local variable 'args' referenced before assignment

The bytecode was generated with solc 0.5.7 from the following source:
pragma solidity ^0.5.0;

contract Test {

	uint256 myinteger;

    function doNothing() public view returns (uint256) {
    	return myinteger;
    }

    function write() public {
    	myinteger = 1;
    }

    function alwaysfail() public {
    	assert(0 == 1);
    }

    function withdrawEth() public {
    	msg.sender.transfer(address(this).balance);
    }
}

Interestingly this happens only with bytecode compiled with newer solc version. Octopus processes the same contract when compiled with 0.4.24 but fails with 0.5.7.
pragma solidity ^0.5.0;

contract Test {
	uint256[] public myinteger;
}

0.4.24:
608060405260043610603f576000357c0100000000000000000000000000000000000000000000000000000000900463ffffffff1680637c4e2261146044575b600080fd5b348015604f57600080fd5b50606c600480360381019080803590602001909291905050506082565b6040518082815260200191505060405180910390f35b600081815481101515609057fe5b9060005260206000200160009150905054815600a165627a7a72305820a1f21cb8e5428cf6d0898a479fa2441da1ee73de1679e0b316a1dea027a843d70029

0.5.7:
6080604052348015600f57600080fd5b506004361060285760003560e01c80637c4e226114602d575b600080fd5b605660048036036020811015604157600080fd5b8101908080359060200190929190505050606c565b6040518082815260200191505060405180910390f35b60008181548110607857fe5b90600052602060002001600091509050548156fea165627a7a7230582076100799af5e6466edc3d544d4782bc2c30d0684ce8cb4c0f2bb710f1ef1e6630029

Thanks Bernhard ;) i will try to take a look asap

I am experiencing the same issue when trying to construct a dynamic control flow graph.

@pventuzelo I am having this same issue 3 years later, trying to construct a CFG. I see that this repo has not been updated in the past 2 years... is there any chance that this will be fixed?

Sorry I'm not planning to fix this at the moment. You might be interested to used https://github.com/crytic/evm_cfg_builder


Proposal: testing chart - show log of test pod

Problem
Currently when we test our helm charts we get a Success or Fail.
Proposed solution
I propose to provide a flag that will allow the users to see the log of the test pod to get information about why it failed or passed
Behavior
In the helm test <command>  adding a flag--log or --output flag. In which, at the end of the test we output the test-pod log if flag is present. If the test pod is still present in the cluster(user did not use the clean up flag), the user should be able to get the output of the test-pod.

This makes sense to me

I've been fighting with a way to do this in my CI server for the last 2 days.  Having this as an option would be amazing.  Also a way to do the --cleanup without actually running the tests would be nice as well.


[Feature] Can I custom control value

Hi I saw this widget only have the onChange callback to send selected value but in my case I need to implement date range by auto select end date so I think I have issue with this case.

Hi @startmt , I think you case can be achieved with CalendarDatePicker2 widget.

I need to implement date range by auto select end date

All you need is to set the end date manually when you receive the start date, check out this code below:
CalendarDatePicker2(
          config: CalendarDatePicker2Config(calendarType: CalendarDatePicker2Type.range),
          initialValue: _rangeValueVariable,
          onValueChanged: (dates) =>
              setState(() =>
              _rangeValueVariable = [
                dates[0],
                dates[0]?.add(Duration(days: 7))
              ]),
        ),


I tried the tool to download music from youtube.... maybe a feature to consider is providing the option to not download a video at all as I am only interested in the audio files.

  I tried the tool to download music from youtube.... maybe a feature to consider is providing the option to not download a video at all as I am only interested in the audio files.

I could not test what the tool downloaded as it did not work for me but even in case I get a separate video and audio file it would be a waste to download the video just to delete it!
Originally posted by @msteer22 in https://github.com/kannagi0303/yt-dlp-gui/discussions/13#discussioncomment-4624217

Hello
Only audio can be downloaded.
I can only speculate from the information you provided, you may not have set up yt-dlp and ffmpeg correctly
As I replied to you in another issus, you can try to use the command line mode to execute yt-dlp to see if it can run normally.

Hey there,
that was quick… thank you!
I tried yt-dlp using the command prompt and it worked including the split by chapters and the download of the audio files only. At the end trying to get the command line tool to work was not as much work as I thought.
What worked for me was yt-dlp -x --audio-format mp3 --split-chapters -o chapter:%(section_title)s.%(ext)s https://www.youtube.com/watch?v=xxxx
Btw the splitchapters help of yt-dlp mentioned that ffprobe needs to be available, when I tried your tool I only copied ffmpeg as only this file specifically is mentioned in the wiki. Not sure whether this makes any difference.
The download speed with the yt-dlp was 20 mb per second the whole video downloaded in 6 seconds. Not sure why using the GUI is so slow in comparison.
Anyway I think the GUI you build makes it easier for users like me that have an aversion to using the text commands. I like my mouse GUI 😃
Regards,
Martin
Martin Steer
EnaMar GmbH
Hauptstr. 28
D-15806 Zossen
Mobile  +49 151 65 131413
<EMAIL_ADDRESS>Sitz: Zossen. Registergericht: Amtsgericht Potsdam, HRB 28131
Geschäftsführer: Verena Kuhn, Martin Steer
From: カンナギ @.>
Sent: Sunday, January 8, 2023 10:49 AM
To: kannagi0303/yt-dlp-gui @.>
Cc: Martin Steer @.>; Mention @.>
Subject: Re: [kannagi0303/yt-dlp-gui] I tried the tool to download music from youtube.... maybe a feature to consider is providing the option to not download a video at all as I am only interested in the audio files. (Issue #51)
Hello
Only audio can be downloaded.
I can only speculate from the information you provided, you may not have set up yt-dlp and ffmpeg correctly
As I replied to you in another issus, you can try to use the command line mode to execute yt-dlp to see if it can run normally.
—
Reply to this email directly, view it on GitHubhttps://github.com/kannagi0303/yt-dlp-gui/issues/51#issuecomment-1374780699, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5EKQX3USXTTC3UERG4AX3DWRKEQDANCNFSM6AAAAAATUPF2KI.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Hello
Thank you for providing the parameters and related information.
However, I don't know why the speed is slower, although sometimes it may be limited by youtube.
The GUI I made is only to provide the UI, instead of the user to execute the yt-dlp command line. In theory there should be no difference.
At present, the GUI only provides the chapter function of Video+Audio, download the entire Video or Audio separately,
Referring to your parameters, you should want to separate the Audio with only a single chapter.
I will try to add this part of the function in subsequent versions, and if ffprobe is needed for help, I will also add this description.
P.S. Sorry I edited your message, because it seems to contain your private information, I removed that part, hope you don't mind.

Hello, sorry for the wait.
Now you can directly change the file extension to the format you want when saving audio.
If you are interested, please try the new version. Thank you.


Minicon 1-4.1 is not working properly

Minicon is extraordinary tool to reduce docker images size
When i used minicon 1-4.1 to reduce docker images it reduced centos images size to 0 B.But when i tried with 1-2.1 it is working perfectly.Kindly resolve the issues in 1-4.1.

Thank you for the report. Could you please include an example of Dockerfile and minicon commandline to reproduce the error?

Hi,
there was a problem in the packages. Now it has been solved.
@srisakthigajapathy, can you please lease try it?


Poor to viewing SVG with transparent background

=> preview

@SSkyang
The popup layer is transparent, too. I'll work on it and provide background color setting of the popup layer.

@SSkyang Hi, it has been solved in v1.3.1. There will be a default white background color for the image with transparent background, and the background color can be set on the plugin settings page. You can upgrade to v1.3.1 and see how it works. out

Thank you so much!


Template framework needs better templating

new-framework.sh and new-standalone-service.sh both did a replace of the word "template" in frameworks/template, but there are legitimate uses of that word in frameworks/template/vendor. The proposed solution is to instead use a more template friendly term like {{template}} for replacements.
Also updated the template's cli and vendor dirs with a more recent version of the dcos-commons cli package

Altenative solution to this problem?

Remove frameworks/template/vendor and instead run govendor fetch github.com/mesosphere/dcos-commons/cli once the new framework has been generated

@drewkerrigan We have an open ticket to address these templating issues. I'm going to close this (since it's drifted) in favor of that future work.


Taskqueue load test

Makes progress towards AppScale/appscale-testing#312.
This PR adds load test for TaskQueue. Currently it just pushes load to the service and makes sure that TaskQueue behaves consistently. It worth to add performance comparison to master in the future.
JFYI this branch has 4 common commits with https://github.com/AppScale/appscale/pull/2850 (test helpers).
README.md for more details.

https://ocd.appscale.com:8080/view/All/job/TaskQueue-Load-Test/40/console

https://ocd.appscale.com:8080/view/All/job/TaskQueue-Load-Test/47/

Codacy complains about:

dirs variable is not used:
for root, dirs, files in os.walk(logs_dir):
I do think that it's more readable to use unused var in this case. Alternatively it would have to be named _ or root and files would have to be extracted by item[0] and item[2].
Method could be a function (or static method). def check_performance_and_exit(self):
   # TODO, compare it to some baseline
   sys.exit(0)

I wouldn't like to address this issue as it's just TODO stub which is going to be replaced with a regular method.
Unused variable bytes_noise:
bytes_work_times, bytes_noise = task.body.split(b':', 1)
This var just clarifies what is the rest of task.body.

https://ocd.appscale.com:8080/view/All/job/TaskQueue-Load-Test/59/console
2018-09-26 04:34:46: INFO === Test Cassandra implementation of Pull Queues ===
INFO TaskQueue activity stats:
 - 30220 tasks added
 - 19650 times tasks were leased
 - 8 tasks exhausted retries
 - 13160 tasks were completed successfully
 - Test execution time is 4793s

ERROR TaskQueue activity log is not consistent:
 - 408 tasks were leased before previous lease was expired
 - 19699 tasks left stalled
 - 2647 tasks were deleted more than once during the test
 - Number of added tasks (30220) is not equal to sum of deleted (13160) and exhausted (8)

Total summary:
    37121   total requests
    29638   failed requests 79.84%
       19   min response time
    63618   avg response time
   247740   max response time
       19.9 requests/s

2018-09-26 05:55:04: INFO === Test Postgres implementation of Pull Queues ===
INFO TaskQueue activity stats:
 - 63240 tasks added
 - 100992 times tasks were leased
 - 0 tasks exhausted retries
 - 63240 tasks were completed successfully
 - Test execution time is 547s
INFO TaskQueue activity log looks consistent

Total summary:
    65338   total requests
     2616   failed requests 4.00%
        7   min response time
     6230   avg response time
    45974   max response time
      114.0 requests/s


Core: Fix the nullish coalescing operator in _updateScalesRange

I got this interesting warning when playing around with the project build.
src/containers/xy-container/index.ts: The "??" operator here will always return the left operand

And the compiler was right. In case config.padding.bottom will be undefined, subtraction will get NaN. The operator will be applied to the result of the subtraction but not the config.padding.bottom.  I wonder how did we miss it. Thanks

@50rayn Cool, it's something I wanted to do for a long time


Add partitionAndSort

Looks like tests are failing because of findbugs errors.

Could you open an issue for this?

@tomwhite is this PR still alive?

@mkwhitacre I haven't had a chance to update it with Ryan's feedback. I probably won't for a bit, so feel free to take it over if you need it.


Add Nix shell without rust-analyzer

This PR:
This is probably just a personal pain point, but I find myself often waiting a very long time on rust-analyzer (which I don't use) to compile after a flake.lock update (which happens weekly).
This PR adds a shell without rust-analyzer, and uncommits .envrc.
For .envrc: given that we a) use Nix and b) don't do anything interesting in .envrc, I don't think committing the latter makes sense -- it effectively just prevents you from creating your own .envrc to choose a non-default shell.
Note also that our .envrc is very straightforward to recreate -- e.g. 1 line: use flake .#simpleShell -- and should only have to be done once per repository clone (and only if you use direnv).
This PR does not:
Key places to review:
Would this affect your own workflow?

Not sure if it's relevant to anyone's workflow, but I think it's worth noticing that since this PR removes RA from buildDeps, it makes all shells except the default one RA-less.

Not sure if it's relevant to anyone's workflow, but I think it's worth noticing that since this PR removes RA from buildDeps, it makes all shells except the default one RA-less.
Also, for all direnv+nix users I can't recommend nix-direnv enough. It caches nix shell, making subsequent runs of direnv with unchanged flake way faster.

Updated!


Update Investigations on homepage

💡 Description
Right now we have 3 landers/rovers on the homepage, we need to diversify:

Mars 2020
LRO
New Horizons / KEM2 (extended mission)
Cassini
MRO

⚔ Parent Epic / Related Tickets
No response

Status: Work starting. Should not be too difficult dependent upon images.

Code is ready for a Pull Request.


Timeline: revisit merging sources from all providers

I am a timeline consumer for local history entries. I respect the decision to show all timeline sources merged together by default. I do not agree with that decision though and think a dropdown to switch between individual providers makes more sense.
My changes are on main and it works like this:

anytime you save a working copy, a new entry is added
the entries appear as timeline entries in the "Timeline" view as "Local History"
clicking an entry diffs to the current file

Demo:

Imagine existing users that are used to seeing the git history. They will soon see their timeline spammed with entries that have nothing to do with git (that is my main worry actually).
Alternate Design
Imagine a dropdown that allows to switch between the sources:

//cc @kieferrm @egamma

Closing in favour of a model where multiple entries of a source can collapse into one. I can see whether I get that working before we ship stable.

clicking an entry diffs to the current file

This looks unexpected. Moreover it differs from what user gets for git commits (diff for commit) and I think it differs from WebStorm (which is mentioned in initial request and more in comments).
Moreover latest File Saved entry shows empty diff and first one (that might change 1 line) shows all that changes. And to guess what actually change was saved first I need to "calculate" diff between fist and second saves.
At least my expectation was to see exact diff caused by save. Are there plans to make it actual "diff by that save"?

@IllusionMH I looked at IntelliJ and Eclipse which both diff to the latest file. Nevertheless, there is options now to diff the exact change:

The major advantage of diffing to the current file is that the right hand side is editable, so you can selectively take over changes. And you will also see what you get when you restore the entry.
Imho diffing to the previous local history is quite meaningless, especially since entries appear when auto save is enabled. This is different from git commits that always group a logical change unit under 1 label.

Looks like it was long time ago when I used WS (and local history), because I think I restored some parts (not recent) but most likely I just copied.
Thanks for context menu entry, already had a chance to used it to get adequate diff (especially with removals). My case was several experiments to get thing working and "find" what I did in the middle of this process.


How to position Switch label to the left of the switch?

[x] This is a v1.x issue (v0.x is no longer maintained).
[x] I have searched the issues of this repository and believe that this is not a duplicate.

The examples here demonstrate how to add a label to various forms of input, including the Switch component.
Unfortunately, every single example on that page that includes a label has it positioned to the right of the component. I've looked through the props of FormControlLabel, and there doesn't seem to be any option that allows me to change this. In fact, looking at the code of FormControlLabel, it seems to be hard-coded his way.
I was able to work around this by setting the Label's direction to rtl (and setting the Switch's direction back to ltr):
import React from "react";
import styled from "react-emotion";
import FormControlLabel from "@material-ui/core/FormControlLabel";
import Switch from "@material-ui/core/Switch";

const StyledLabel = styled(FormControlLabel)`
  direction: ${({ rtl }) => rtl && "rtl"};
  margin: 0;
`;

const StyledSwitch = styled(Switch)`
  direction: ltr;
`;

export default ({ label, rtl, ...props }) => (
  <StyledLabel rtl={rtl} control={<StyledSwitch {...props} />} label={label} />
);

But this feels very hacky and potentially unstable to me.
Is there any way I can have a Switch with its label on the left without resorting to rtl hacks? If so, can it please be documented? If not, please consider this a feature request.

FormControlLabel wasn't designed with this use case in mind. You would need to write custom CSS to get this behavior.

I have added the waiting for users upvotes tag. I'm closing the issue as I'm not sure people are looking for such abstraction. So please upvote this issue if you are. We will prioritize our effort based on the number of upvotes.

Hi,
I hope I did the correct thing to upvote by using the thumbs up emoji.
Actually, everywhere in the Material design guidelines the switches are on the right of the label, so it would be very helpful to have an easy way to do this...

I too would like this.  imagine that you have a pane to view some page of results, and you want a select all switch at the top right.  You want the label on the left to nestle nicely into the top right corner.  this is totally in line with mobile first design and hey, bonus! it works on a desktop layout too!
I've +1'ed the ticket!  Material-UI is a component lib, and regardless if we "think" we would do something, the option to show a form label on one side or the other is ultimately a design decision which a flexible and powerful framework should be able to offer. wink

I too would like this. imagine that you have a pane to view some page of results, and you want a select all switch at the top right. You want the label on the left to nestle nicely into the top right corner. this is totally in line with mobile first design and hey, bonus! it works on a desktop layout too!
I've +1'ed the ticket! Material-UI is a component lib, and regardless if we "think" we would do something, the option to show a form label on one side or the other is ultimately a design decision which a flexible and powerful framework should be able to offer. wink

Alright, we can add a property to change the order of the control and the label:
https://github.com/mui-org/material-ui/blob/b3564a767135b933146f7fe73e3a34b14950c782/packages/material-ui/src/FormControlLabel/FormControlLabel.js#L75-L90
It should be easy to implement. In the mean time, you don't have to use the FormControlLabel component. You should be able to inverse the order in your code quite easily.

@oliviertassinari Any thoughts on prop name?

@mbrookes Not really, maybe we could follow the Tooltip wording. It's important to have a wording that works with right to left and left to right.

What about:
controlPlacement?: 'start' : 'end';

or
labelPlacement?: 'start' : 'end';

Thanks for repoening :-)
Why not make it a boolean? e.g. "controlBeforeLabel"

Hmm... good point on LTR, although while Tooltip respects the prop wording (placement="left" is always on the left, even for RTL), it would seem more practical if the positioning was reversed for RTL - but that's a whole separate issue.
start / end work as expected.

Why not make it a boolean? e.g. "controlBeforeLabel"

@caroe233 So we can accept more values, like top or bottom.

I haven't tried it with FormControlLabel, but Slider can potentially take two labels, so it might be interesting to try to accomadate that:

it is compatible with the spirit of a switch:
OFF --o ON
OFF o-- ON

it is compatible with the spirit of a switch:
OFF --o ON
OFF o-- ON

Which release will this be in?

@kaitlynbrown The next one. 😜

What if we want to put labels on both sides?

@iamkennytso I guess the best answer is: DIY. We try to focus on the most common use cases.

I've used labelPlacement="start" to put the label on the left. However, it is right-aligned. Can anyone provide a pointer for left-aligning the label?

@benmccann This would be better asked on StackOverflow. Stuff to try: text align right, flex justify end.

Thanks for the suggestions. I got some good answers on StackOverflow.

Hello,
You can add class to your v-switch (ex: .reverse-switch) and in your css you add this rule:
.reverse-switch .v-input__slot {
flex-direction: row-reverse;
}

I'd also like to put the label on the left. Because it isn't that important that the text be clickable, I'll just use ordinary text for the label for now, and just use the Switch, rather than wrap it in the FormControlLabel.

What about:
controlPlacement?: 'start' : 'end';

or
labelPlacement?: 'start' : 'end';

Thanks


Updated pagination.twig and page.php

Added some simple names for code blocks, fixed spacing in pagination.twig and changed path in page.php description.

These are great, thanks for the edits @tomlecki !


Replace version string with git commit information for non-release builds

Switching back to the classic version can be done by changing the ZMAP_VERSION variable in /CMakeLists.txt to a version number and calling 'make'.

This resolves #364.

Sample output:
$zmap -V
zmap Development Build. Commit efc1d77 - Tue Dec 27 16:33:03 2016 -0800

@paul-pearce to clarify, this handles getting the git commit when make is run correct, not just cmake?

Yes.
On Dec 28, 2016 10:06 AM, "Zakir Durumeric"<EMAIL_ADDRESS>wrote:

@paul-pearce https://github.com/paul-pearce to clarify, this handles
getting the git commit when make is run correct, not just cmake?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zmap/zmap/pull/403#issuecomment-269515617, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACIjR4NQCycx4qGMIXn4jNV_zVvRt_M-ks5rMqU9gaJpZM4LWl4M
.


[add-resource] New Plugin

Please add the following plugin: "Perlkonig/grav-plugin-pubmed"
https://github.com/Perlkonig/grav-plugin-pubmed
I created this because I wanted to learn how to insert my own coding and someone in the forums asked for this plugin specifically.

Very nice. I have a few comments.

Looks like you forgot a couple dump (https://github.com/Perlkonig/grav-plugin-pubmed/blob/master/pubmed.php#L56) also line 60
In your class you are recreating the fetch method that we already provide with grav through Response::get (here is the class) you should probably use that so it's more consistent. Here an example on how to use it, it's very simple
It appears that your cache mechanism for retrieving the publications is only local. I would suggest taking advantage of our Cache api to make it persistent so that on refresh the data is still preserved and the site loads more efficiently.

Out of curiosity, I always wondered if pubmed provided an api, if so would be really cool if one could pass an user token to get full access to the publications and not just the abstracts or the open ones.
Grav allows to protect pages so they could be secured and could become a very powerful plugin for internal states, where publications could get organized and listed in a more friendly UI. Just a thought :)

Doh! I'll get those dumps fixed as soon as our dinner guests leave. I'll take a look at the built-in fetch and cache too. I'll post here when a new release is ready. Thanks for the tips!

I'll have to look closer at the pubmed API. I did this one as a learning experience because someone mentioned it in the forums.

Changes made and new release published. Thanks again.

Looking good! Added.
Thanks.


[node_exporter] update to most recent version

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.
Is your feature request related to a problem? Please describe.
As of writing node_exporter is on 1.3.1
Describe the solution you'd like
Update node_exporter to the latest version
Describe alternatives you've considered
n/a
Additional context
n/a

Looks like 1.3.1 is the latest there too,
https://www.freshports.org/sysutils/node_exporter/
Does anything need to be done on the opnsense side to bring it over?

Is the latest here too ;)
https://github.com/opnsense/ports/commit/8d7024656a535e7b04029ab3335d26281f9c2c45#diff-fa5888535be143fda9d2c3e49662a2d15f23a1696a6f9c8c09dd5da12d886517

My mistake, looks like it is! I'll make sure to check more places next time, thanks!


build(multiview-container + dependencies): use node16 module resolution

Semi-automatic PR to switch to node16 module resolution.
Includes removal of unimplemented test stubs and unused test infrastructure.

multiview-container is the only one that really wants to retain its test infra, seems correct to remove the rest (it uses all the rest, so it can serve as a sufficient smoke test for everything at once).
multiview-container is intentionally the only one with an export and able to be run directly (description of the demo structure here: https://github.com/microsoft/FluidFramework/tree/main/examples/data-objects/multiview#readme).  This demo dates back to when views shipped as integrated render() methods of data objects, so I went a little overboard with separation to overemphasize the point that the view can/should be separate.  Now that we've laid that issue to rest, we could probably consider whether there's a more concise way to demonstrate the multiview patterns, maybe a little less package-heavy.

Sounds like the starts should be removed from most of the packages too, then. Totally fine with that happening secondarily.


Avoid usage of OpenCV in order to Pillow

Clean up codes from OpenCV calls. Use Pillow instead.

Done in 2a92d6a2cad1f5b26761285971e7e7c04fb5731a, 4b413b07d6b76216a5b6fe948267b37e45a068f7


Seeds can germinate on invalid terrain

Introduced in #699: I couldn't figure out why this was happening.
To reproduce, create a large number of acacia plants, then terraform a large number of tiles near your mushrooms to stone.
The can_build check in both spawn_structure and the sprout_seeds system should be blocking this but it isn't for some reason. I even added a test to ensure it works!

No longer relevant!


i have this error: ImportError: cannot import name 'is_safetensors_available' from 'diffusers.utils'

Traceback (most recent call last):
File "C:\Users\USUARIO\Desktop\wonder3d\test_mvdiffusion_seq.py", line 40, in
from mvdiffusion.models.unet_mv2d_condition import UNetMV2DConditionModel
File "C:\Users\USUARIO\Desktop\wonder3d\mvdiffusion\models\unet_mv2d_condition.py", line 47, in
from diffusers.utils import (
ImportError: cannot import name 'is_safetensors_available' from 'diffusers.utils' (C:\Users\USUARIO\AppData\Local\Programs\Python\Python311\Lib\site-packages\diffusers\utils_init_.py)

Your issue here is the version of diffusion[torch[.
run pip install diffusers[torch]==0.19.3 and try from there

solved!

solved!


[BUG] Empty tuples aren't optimized in conditional statements

Describe the bug
Normally statements such as x in (1, 2, 3) are optimized to not use python functions, however x in () will result in the generation of calls to python functions.
Code to reproduce the behaviour:
ctypedef enum node_type:
    NODE_NULL,
    NODE_CONSTANT,
    NODE_TRANSMIT,
    NODE_SUM,
    NODE_ACTIVATE,
    NODE_BLEND,
    NODE_MULTIPLY,
    NODE_MODULO,
    NODE_BOOL,
    NODE_INVERSE,
    NODE_SIN,
    NODE_SQRT,
    NODE_LOW_POW,
    NODE_SQUARE,
    NODE_INPUT,
    NODE_OUTPUT

ctypedef enum slot_type:
    SLOT_NULL,
    SLOT_FLOAT,
    SLOT_BOOL

cdef inline slot_type get_node_input_type(node_type node_type, slot_type type_) noexcept:
    if ((node_type) in (NODE_CONSTANT, NODE_TRANSMIT, NODE_BLEND, NODE_INPUT)):
        return type_
    elif ((node_type) in (NODE_SUM, NODE_ACTIVATE, NODE_MULTIPLY, NODE_MODULO, NODE_INVERSE, NODE_SIN, NODE_SQRT, NODE_LOW_POW, NODE_SQUARE, NODE_BOOL)):
        return SLOT_FLOAT
    elif ((node_type) in ()):
        return SLOT_BOOL

    return SLOT_NULL

Expected behaviour
They should be optimized into C switch statements.
OS
No response
Python version
3.8.3
Cython version
3.0.10
Additional context
No response

Is this x in () something that actually appears in your (generated?) code?

Yes, I use macros.

I use macros.

Could you give a (realistic) example of how you use this? I'd like to understand if this is something worth working on.

I have Nodes that input and output Slots. To reduce code duplication and increase extensibility, I have macros defined that can check commonly used properties about nodes inferred from theirs types. Somes of these macros use tuples of Node types. To increase extensibility, these tuples are defined as separate macros than the checks which they are included in so they can be used in other context (e.g. iterating over a list of types that meet the requirement).
Some of these requirements don't have any nodes that match them or exclude them but may in the future, so they are defined as empty tuples.

Still not sure what you mean by "macros".

I use the C processor on my code.

However, I doubt that variable in () takes any serious amount of time. variable and () are constants that do not need any time to look up, then whatever in () is really just jumping through a few pointers and returning 0 without doing anything worth noting. Arguably, the C compiler cannot optimise that away, so it is doing something at runtime, but that "something" is really, really cheap, especially when executed several times. If you can motivate that this consumes a non-negligible amount of time in your code, we may consider adding special cases for this, but I doubt that you'll find this being a bottleneck.

I don't know how much a performance hit it is. It probably isn't much, but the code is very optimized and the function is called a lot. Plus, the perfectionist in me gets annoyed seeing a yellow line in a sea of white.
Forgive me since I don't know much about Cython's internal architecture, but if the problem is that the lhs could have side-effects, then why don't the non-empty arrays have the same issue? None of my lhs's have side effects, but only the one with the empty tuple causes the issue.


failed configuring site but debug meter shows output

Describe the bug
Moving to Rpi native install from docker trying to install using same info in exiting working evcc.yaml and getting error:
failed configuring site
missing either grid or pv meter
Steps to reproduce
1.using same evcc.yaml in docker works fine
2.evcc meter shows output for both grid1 and pv1
3.
...

Linux
Version
evcc version 0.126.6

Please add error log

Jun 15 07:16:53 raspberrypi systemd[1]: Started evcc.service - evcc.
Jun 15 07:16:57 raspberrypi evcc[6608]: [main  ] INFO 2024/06/15 07:16:57 evcc 0.126.6
Jun 15 07:16:57 raspberrypi evcc[6608]: [main  ] INFO 2024/06/15 07:16:57 using config file: /etc/evcc.yaml
Jun 15 07:17:10 raspberrypi evcc[6608]: [main  ] ERROR 2024/06/15 07:17:10 creating vehicle Infinus failed: cannot create vehicle type 'custom': soc: missing mqtt broker configuration
Jun 15 07:17:10 raspberrypi evcc[6608]: [main  ] FATAL 2024/06/15 07:17:10 failed configuring site: missing either grid or pv meter
Jun 15 07:17:10 raspberrypi evcc[6608]: [main  ] FATAL 2024/06/15 07:17:10 will attempt restart in: 5m0s

I'm still working on the MQTT part for the custom vehicle. but the grid should be pretty straightforward.

Forgot site config which is why it would not start.....


Resources. ListByTag expects values to be in '*'

This will add tag to resource:
                   resourceGroup.Update()                               .WithTag("dealerID", "20")                               .Apply();
To be able to get resources by TagName need to add '' for the values:
azure.ResourceGroups.ListByTag("'dealerID'", "'20'").ToList()
I would expect it to be without ''

interesting catch. I'm surprised as well.

This issue was moved to Azure/azure-libraries-for-net#55


use the correct method to get lr to avoid warning.

Signed-off-by: ChengZi<EMAIL_ADDRESS>

/lgtm
/approve

Codecov Report

Merging #824 (e595e5c) into main (aab5267) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #824      +/-   ##
==========================================
- Coverage   71.10%   71.08%   -0.02%
==========================================
  Files         175      175
  Lines        7938     7938
  Branches     1303     1303
==========================================
- Hits         5644     5643       -1
- Misses       1906     1908       +2
+ Partials      388      387       -1

Impacted Files
Coverage Δ

towhee/trainer/trainer.py
68.80% <100.00%> (ø)

towhee/hub/repo_manager.py
80.18% <0.00%> (-0.91%)
:arrow_down:

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 aab5267...e595e5c. Read the comment docs.


[Bug]: IGattCharacteristic.Notify Finally exception handling

Component/Nuget
BluetoothLE Client (Shiny.BluetoothLE)
What operating system(s) are effected?

[ ] iOS 12
[ ] iOS 13
[ ] iOS 14
[ ] iOS 15
[ ] Mac Catalyst
[ ] Android 8
[ ] Android 9
[ ] Android 10
[ ] Android 11
[X] Android 12

What platform(s) are effected?

[ ] MAUI
[ ] Xamarin Native
[X] Xamarin Forms

Steps To Reproduce
I don't have a reproducible sample to give you but I ran into a situation where I have a bluetooth device where the recommended way to disconnect is to send a command to the device and it essentially resets itself. With the way I was using Shiny this caused my application to crash and the culprit was using the IGattCharacteristic.Notify extension method. The "finally" operator contains an action that can throw an exception. In my case the exception was "Failed to write to descriptor". Unfortunately (and news to me) any unhandled exception in a "finally" operator will throw an exception on the ThreadPool and crash the process (see stackoverflow link below). I've worked around this by not using Shiny's "Notify" extension method but figured you might want to take a look as it was a difficult issue to diagnose.
https://stackoverflow.com/questions/69815945/how-to-implement-a-better-finally-rx-operator
IGattCharacteristic.Notify "finally" location: https://github.com/shinyorg/shiny/blob/master/src/Shiny.BluetoothLE/Extensions_Characteristics.cs#L70
This issue will likely exist for all platforms and operating systems but I have only been testing with Xamarin Forms on Android 12.
Expected Behavior
Handle exception within a "finally" operator so application does not crash.
Actual Behavior
Action within "finally" operator throws an exception and crashes the application.
Exception or Log output
No response
Code Sample
No response
Code of Conduct

[X] My reproducible sample is NOT FROM THE SHINY SAMPLES REPOSITORY!  If you file an issue that says 'use shiny sample', YOU WILL BE BANNED FROM THE SHINY REPOS.  Read the code of conduct for why we care about this!
[X] I am using the latest stable version from nuget (sponsors can discuss previews in discord)
[X] I agree to follow this project's Code of Conduct

Thanks for the report

Thanks for the quick fix!


Fix CI

Description
See https://github.com/conda-incubator/setup-miniconda/issues/274
Fixes issue where Conda cannot create environment.
In a future PR, we will switch to micromamba.
Motivation and context

Resolves #1143
How has this been tested?

Change log

@ijpulidos can you double check the failures we are getting are what we expect and if so merge this in?


Extensions whitelist are case sensitive only

It should be fixed to ignore case. Example: "JPG" is a valid extension.
This plugin should call the method check_whitelist from carrierwave instead of creating its own, because in carrierwave fixes it 2 years ago.
https://github.com/jnicklas/carrierwave/pull/174

Wow, still an open issue. Just started using carrierwave_direct, and low and behold, customers are now having problems with uploading images due to this.


Fix automatic selection of correct alt forme

I don't know why we bothered with all that complex logic when we literally have the correct forme name available from the user's selection.

Awesome thank you, the reason for the complex logic was because it was just copied over when I was creating the alt forme selection

This should theoretically close #291


can't find library export

I use

Because you compile library, mount, bootstrap and unmount are just exports from library, it is not exposed to window, and please use dicussion or stackoverflow for questions

Because you compile library, mount, bootstrap and unmount are just exports from library, it is not exposed to window, and please use dicussion or stackoverflow for questions

Use this window.test.mount(), if you need access to your library

Use this window.test.mount(), if you need access to your library

thanks bro! It help me a lot.have a nice day.

thanks bro! It help me a lot.have a nice day.


docs: Update Keychain SDK

Summary by CodeRabbit

New Features

Introduced comprehensive documentation for the Keychain SDK, detailing its modules and functionalities for managing cryptographic operations.
Added a tutorial for building a Keychain Service using the Warden Protocol, including setup instructions and practical code examples.

Documentation

Enhanced documentation structure with a new configuration file for the Keychain SDK to improve navigation and user experience.
Updated documentation paths for the Keychain SDK to reflect a restructured organization.

@Pitasi - Rita has addressed most of your concerns. I would like to Merge this PR & then shuffle the content placement categories.


Incident comments - fix swagger quality s360

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

Changelog
Please ensure to add changelog with 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 you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[X] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[X] CLI
[ ] Terraform
[ ] No, no need to refresh 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

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please 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 there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

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.

This PR is meant to fix quality issues from S360:
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=C50B2436-B0B1-48C7-9914-3154B9C62336
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=2C97BA40-2477-48F5-B3D4-442DCBFF662D
I think there was one more, if I will find it I will add it to the PR.
@jianyexi

This PR is meant to fix quality issues from S360:
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=C50B2436-B0B1-48C7-9914-3154B9C62336
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=2C97BA40-2477-48F5-B3D4-442DCBFF662D
I think there was one more, if I will find it I will add it to the PR.
@jianyexi

@jianyexi
I am trying to fix the PR according to your comment.
Can you please add the label "DoNotMerge"?
I don't want it to automatically merge after approval do to fix in the backend as well.
Thank you :)

@jianyexi
I am trying to fix the PR according to your comment.
Can you please add the label "DoNotMerge"?
I don't want it to automatically merge after approval do to fix in the backend as well.
Thank you :)

Hi @jianyexi
This PR is meant to fix quality issues from S360:
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=C50B2436-B0B1-48C7-9914-3154B9C62336
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=2C97BA40-2477-48F5-B3D4-442DCBFF662D
Can you please add the label "DoNotMerge"?

Hi @jianyexi
This PR is meant to fix quality issues from S360:
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=C50B2436-B0B1-48C7-9914-3154B9C62336
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=2C97BA40-2477-48F5-B3D4-442DCBFF662D
Can you please add the label "DoNotMerge"?


fix: don't delete / in remove_all

This PR changes the behavior of remove_all("/") to not delete "/".  As discussed on Discord, this can put the operator into a bad state i.e. for S3 this would have previously deleted the bucket itself.

@sameer Hi, I have fixed CI failing issue by  https://github.com/apache/opendal/commit/f661165280b2d3b942e07415c550132a17172f9f， Would you mind rebasing your PR to the new master commit?

It's expected to remove the / in remove_all. I believe we should refrain from sending a delete request with / in s3 service instead.

It's expected to remove the / in remove_all.

What is the expected behavior for removing /? i.e. for the FS service, should that delete the working dir?

What is the expected behavior in general for removing /? i.e. for the FS service, should that delete the working dir?

Yep. Just like rm -rf /

Are there any other services that need to be checked? S3 was the only one I tried this with

Other services may have similar issues, but I don't think we need to address all of them in this PR. Let's first fix it for S3.

Sounds good, I'll update this PR soon


generate mex files on build

The platform (Windows, Unix, MacOS) dependent mex files need to be compiled manually every time their source code is changed. Is there a way to automatize it?

On every push to master a GitHub action can run that checks if the source code has changed. If yes, mex files are compiled on all three platforms and pushed to the repository for a new commit.

Keep mex files outside of source control and only create and package them up with builds. This has the disadvantage developers need to manually generate them after cloning the repo.

this can be revisited once we have the new windows and macOS machines

The generation of this mex file may be related to its generation environment and platform, such as MATLAB's parallel mode: Threads or processes, C/C++'s corresponding mex compiler, Intel oneapi or VS studio? After my personal test, the most basic MATLAB parallel mode is different, the mex cannot be called. Therefore, this part needs to be standardized and rigorously tested.


Possibility to use single account for all MDomain?

Hi,
is there any option, or is it possible to add one, to use same account for all MDomain declarations? Or declare which account should be used for each MDomain declaration (like declaring account in separate tag MDAccount with custom label which can than be referred from MDomain).
If I understand https://letsencrypt.org/docs/rate-limits/ correctly, with single account it should be possible to register 300 new domains per 3 hours, however since unique account is generated per MDomain, we are limited to registering 10 per 3 hours. That is huge difference. Also it is not possible to conform to recommendation to use only single account for hosting providers (https://letsencrypt.org/docs/integration-guide/).

What you are seeing is a limitation in v1.x which is rather unfortunate:

if you start without an account and 10 MDomains, each domain will not find an existing one and create a new one.
if you start with 1 MDomain, get the certificate, reload the server, the new account gets accepted and all future MDomains will reuse it.

In Version v2.x this was fixed and one can start with as many domains as you want and only a single account is created.
This was never noticed for a long time, since most people try 1 MDomain first and switch the others after that was successful. What you can do now (after making a backup of our md store files):

either throw away the complete md store directory and start fresh with only one MDomain. Enable all others after that was successful.
Edit all the md.json files in your store and replace the "account" entry with the one you want to keep. This will then be used on the next renewal. Remove all other account directories in store.

{
  "name": "mydomain.com",
  ...
  "ca": {
    "account": "ACME-localhost-0000",
  }, ...
}

Does this help?

Great, thanks for quick response.
I'm testing some concept where new domains will be dynamically added by separate vhost conf files, so I've started right away with multiple domains. Additionally I'm testing behavior on localhost, so I'm not actually able to finish verification and obtain certificate.
So in case it works as you described it, it will be sufficient for my case.


Add dynamic OGP tags

This PR adds support for dynamic OGP tags and implements them for meetings. Other pages could do something similar to get custom static OGP or dynamic OGP tags. For testing locally, I would recommend using ngrok so Discord can read the page and display the embed.

Thank you for the tip with ngrok btw :)


I needed a method to plot all fitted and observed data on one plot.

I needed a method to plot all fitted and observed data on one plot. I added this to my local copy
 def plotFitAll (self, figsize=None):
      import matplotlib.pyplot as plt
      plt.figure(figsize=figsize)
      for i in range (len (self.fitted_ts.values[0])-1):
          plt.plot (self.fitted_ts.values[:,0], self.fitted_ts.values[:,i+1])
      for i in range (len (self.observed_ts.values[0])-1):
          plt.scatter (self.observed_ts.values[:,0], self.observed_ts.values[:,i+1], marker='o')

I think it should work. You might want to do it differently to match your own way doing things however. I uploaded it to pypi

I have implemented this in the new version (a bit different code so that all plot options are available)


Language code for Denmark is 'da'

Describe the bug
Country code for Denmark is 'dk'. Language code for Denmark is 'da'.
From here: https://docs.oracle.com/cd/F19385_01/books/OnDemMigrTool/language-codes.html#c_Language_Codes_and_Their_Corresponding_Languageand_Country__apq1024117

DAN | da_DK | da | Danish | Denmark

We have inconsistent faker yaml configs for Denmark
 % head -3 src/main/resources/da-DK.yml
da-DK:
  faker:
    address:

% head -3 src/main/resources/dk.yml
dk:
  faker:
    passport:

To Reproduce
    public static void main(String[] args) {
        Faker faker = new Faker(new Locale("da", "DK"));

        for (int i = 0; i < 10; i++) {
            System.out.println(faker.passport().valid());
        }
    }

Produces following values:
675721431
A73519666
Z21455537
Y08878014
Y01691978
C48328077
X80809531
A72483039
X37609985
288690866

Only the first and  the last from above are valid for DK.
Expected behavior
Per "[0-9]{9}" we should only get 9 digit passport values for Denmark.
Versions:

OS: OSX
JDK 8/11
Faker Version 1.7

dk.yml should be removed and following:
dk:
  faker:
    passport:
      valid: "[0-9]{9}"

should be added to da-DK.yml
Additional context
Add any other context about the problem here.

@bodiam - I can see you added dk.yml back in September. Was this on purpose or an oversight ? Thanks, Luka

@bodiam - I can see you added dk.yml back in September. Was this on purpose or an oversight ? Thanks, Luka

I have no idea :) Was there any hint in the commit message?

Nothing special, something like "Making passport numbers pattern country specific". Could be it was an oversight. It's in da-DK now, no dk here:
https://www.science.co.il/language/Codes.php


Add official language support

I think that we need to add an "official language" support.
I am pretty fine handling the following:

typescript (I may remember to do javascript too)
golang
cpp

(cpp isn't supported currently, but I'll want to support this long run)
Does anyone want to volunteer as the "go to" on the language?

Even though it's difficult (like typescript), I think we want lua to have first class support as it will then make even writing the plugin easier.

I'll go throught the plugin and fiddle with it for a while and try to make it work with PHP. If that'll work I'm happy to maintain it for that language and perhaps help when available with JS/TS.

Closing this issue for now due to inactivity (plus, a lot of languages have since been added, doesn't make much sense anymore).