Wrong type hint for azure.storage.queue.QueueClient.from_connection_string() -> None

Package Name: azure.storage.queue
Package Version: 12.1.1
Operating System: MacOS 10.15.4 (19E287)
Python Version: 3.8.2

Describe the bug
This code implies that azure.storage.queue.QueueClient.from_connection_string() returns None:
https://github.com/Azure/azure-sdk-for-python/blob/ab4f3090be02dcedddf9969f565bf43670a10f7a/sdk/storage/azure-storage-queue/azure/storage/queue/_queue_client.py#L155-L161
In contrast, azure.storage.blob.BlobClient.from_connection_string() returns a BlobClient.
https://github.com/Azure/azure-sdk-for-python/blob/ab4f3090be02dcedddf9969f565bf43670a10f7a/sdk/storage/azure-storage-blob/azure/storage/blob/_blob_client.py#L235-L242
To Reproduce
Steps to reproduce the behavior:

In editor (eg PyCharm):

a = azure.storage.queue.QueueClient.from_connection_string()
a.<TAB>

Observe unhelpful autocomplete.
Add a comment # type: QueueClient
Get useful autocomplete
Now get Expected type 'QueueClient', got 'None' instead  complaint.

Expected behavior
Correct type declaration of QueueClient. Or, at the very least, not an incorrect & affirmative declared type of None.

Hi @alanjcastonguay
Thanks for catching this, we will log this as a bug and fix it later!

Shiny; thanks. 👍


Add reports edit/deletion message

Description

This PR is the implementation of  #575. It adds edit/deletion features for the report type.
Closes #575
Checklist

[x] Targeted PR against correct branch.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Wrote unit tests.
[x] Wrote integration tests (simulation & CLI).
[x] Updated the documentation.
[ ] Added an entry to the CHANGELOG.md file.
[ ] Re-reviewed Files changed in the Github PR explorer.

Closing this as per #690


Add bifrost vETH stableswap and XEX/USDC 80/20 to whitelist

Description
Doc with info provided by Bifrost: https://docs.google.com/document/d/1_ud01m6MMTsVE-__BwJQRt4Ot5Dq2YL7f6QdxwaKR6s/edit.
https://defillama.com/protocol/bifrost-liquid-staking
Team is looking to Launch next Monday.  Would be great to get this in the UI well before hand.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ x ] Other

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.
Simple whitelist add
Checklist:

[x ] I have performed a self-review of my own code
[ x] I have requested at least 1 review (If the PR is significant enough, use best judgement here)
[ x] I have commented my code where relevant, particularly in hard-to-understand areas
[ ] If package-lock.json has changes, it was intentional.  Unsure if the new version should have been pushed but i followed the instructions
[ ] The base of this PR is master if hotfix, develop if not

Merged here: #3145


Cache AppSettings in static class

Since the AppDomain will automatically recycle when changes are made to the AppSettings (web.config/app.config) it is safe to cache the setting values. This ensures we only do the conversion once and allocate the byte arrays once. This will greatly reduce the number of allocations in the critical path of Filters and Attributes.

Not sure of your preferred style for static stuff... could just as easily be a non-static class with a .Instance if you prefer.

That’s a very neat optimisation. Thank you!


Condition builder

2023 design
Background github story #349
Implementation tracking
Dependency on Figma mapping for condition builder

Adding the data structure options available for the out-of-the-box Properties.
Condition Builder - Data structure.md


features/steps/notification_service.py:260 in public function check_print_old_reports_for_cleanup

Pydocstyle issue: D400: First line should end with a period (not 'd')

This issue has been fixed already.


DEPRECATION: Importing password-strength deprecated. Use the passwordStrength service instead

I'm using Ember:
$ ember -v
ember-cli: 3.4.3
node: 8.9.4
os: darwin x64

having this issue:
DEPRECATION: Importing password-strength deprecated. Use the passwordStrength service instead

@devotox any update on this? thank you...

The problem using the non deprecated version was due to its asynchronous behaviour it caused flashes of the password strength on each keystroke rather than a smooth animation
On 12 Dec 2018, 02:23 +0000, RiTa<EMAIL_ADDRESS>wrote:

I'm using Ember:
$ ember -v
ember-cli: 3.4.3
node: 8.9.4
os: darwin x64
having this issue:
DEPRECATION: Importing password-strength deprecated. Use the passwordStrength service instead
@devotox any update on this? thank you...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Head & Tail

Hey, Richard! I just realized that, since Select List is a non-empty list as well:
head: SelectList a -> a

tail: SelectList a -> a

are possible and could be quite useful. Would you like me to open a PR?

In my case I was looking for previous and next:

prev : SelectList a -> Maybe a
prev (SelectList before _ _) =
    before
        |> List.reverse
        |> List.head

next : SelectList a -> Maybe a
next (SelectList _ _ after) =
    after
        |> List.head


Doesn't build on armhf

Getting following when trying to build on armhf with go 1.5.1:
go test  -ldflags="-X main.Build=1a4250fb5147a9a06f608a5e89dccc4f62fd20c5" github.com/derekparker/delve/cmd/dlv github.com/derekparker/delve/config github.com/derekparker/delve/dwarf/frame github.com/derekparker/delve/dwarf/line github.com/derekparker/delve/dwarf/op github.com/derekparker/delve/dwarf/reader github.com/derekparker/delve/dwarf/util github.com/derekparker/delve/proc github.com/derekparker/delve/proc/test github.com/derekparker/delve/service github.com/derekparker/delve/service/api github.com/derekparker/delve/service/debugger github.com/derekparker/delve/service/rpc github.com/derekparker/delve/service/test github.com/derekparker/delve/terminal
github.com/derekparker/delve/proc
proc/registers.go:20: undefined: registers
proc/threads_linux.go:64: undefined: Regs
github.com/derekparker/delve/proc
proc/registers.go:20: undefined: registers
proc/threads_linux.go:64: undefined: Regs
make: *** [test] Error 2

Today the error is a bit better, but still an error.
Will post their output later.

As of 2023 it looks like delve is not supported on armhf by returning at build time
service/debugger/debugger.go:32:2: found packages native (dump_linux.go) and your_linux_architecture_is_not_supported_by_delve (support_sentinel_linux.go) in /app/pkg/proc/native

and is now documented here: https://github.com/go-delve/delve/blob/master/Documentation/faq.md#-im-getting-an-error-while-compiling-delve--unsupported-architectures-and-oss


Height of the terminal is not taken into account

Hi,
I tried to use YouTube-viewer under the terminal and display the comments for the selected video. My terminal displays env variable LINES=33, but the YouTube-viewer shows more comments than it can be displayed in one screen, so I have to scroll up. That is not very convenient. Could youtube-viewer make use of terminal LINES to display content in pages which would fit the screen?
Regards,
Piotr

You can try the --results=n option, which will limit the number of results per page.
youtube-viewer --results=5 --comments 'https://www.youtube.com/watch?v=AuA2EAgAegE'

However, this will not guarantee that the comments will fit in one screen. Another approach would be to use the less command, similar to:
youtube-viewer --comments 'https://www.youtube.com/watch?v=AuA2EAgAegE' --no-interactive | less

...but built into youtube-viewer. I think this would work, but I will have to think a little bit about it. Probably I will add it as an option disabled by default.
Thanks for the report.

Thanks for the hints. It is for sure some workaround. However this paging is affecting all the places (like :h command in interactive session). Nice to have as the option or automatically used reading LINES terminal variable (if present).

Making youtube-viewer's various output functions terminal size aware would be a definite enhancement.


Remove caching of localMounts

localMounts are currently being cached at module load time, and are
never evaluated afterwards.
This is causing us a bunch of issues where mount state is not as
expected.
Remove the caching, so we evaulate local mounts each time we call the
function.
Signed-off-by: Joe Grund<EMAIL_ADDRESS>

Oops, was a little overzealous removing that part.


new(ci): add a circleCI job to test drivers on arm64

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area CI
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
Now that we have a working framework test for all our 3 drivers it would be amazing to run it on all the supported architectures. Right now Github Actions don't support arm64 machine, for this reason, I would to introduce a very simple CircleCI job to run tests on arm64. If in the next future, we will find other solutions (like self-hosted runners) we can remove this job but right now it seems a good way to go, WDYT?
Unfortunately s390x is not supported by either GHA or circleCI, so we don't have any solution :(
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I fully agree, this is a small job but is really needed for us nowadays.

Unfortunately s390x is not supported by either GHA or circleCI, so we don't have any solution :(

Fully agree! @hbrueckner gentle ping :P ahah

What about dropping  CI Build / build-modern-bpf-arm64 and build-libs-arm64 now?

Smart idea, I can create a separate job to build also scap and sinsp tests in this way we can remove the QEMU jobs :fire:

/hold

Even though I think the general direction in our org is to stop using CircleCI, I think that for the time being adding this small job brings good value to the table. Moving it to other CI platforms such as GHA in the future should be pretty straight forward, so I support this!

/milestone 0.11.0

Hi @FedeDP

Unfortunately s390x is not supported by either GHA or circleCI, so we don't have any solution :(

So sad! @hbrueckner gentle ping :P ahah

GHA self-hosted runniner prototype is around: https://github.com/actions/runner/issues/2263 .. working on getting some s390x which a recent Linux distro that support .NET in progress. I will you know when I have news!

Before removing build-libs-arm64 and build-modern-bpf-arm64 we need to remove them from test-infra required jobs :face_exhaling: let's do that in another PR :)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 665ff28ecfd5d6b9a468b6ce0d535a86316d2afe

/unhold


Introduce trace helper function

This adds the trace helper function that reduces a lot of boilerplate of adding spans for user code, allowing users to quickly wrap existing calls with a span and it looks something like this:
use Sentry\State\Scope;
use Sentry\Tracing\SpanContext;
use function Sentry\trace;

$args = [...];

$result = trace(
    function (Scope $scope) use ($args) {
        $scope->setTag('some-tag', 'some-value');

        return $this->doSomethingThatNeedsToBeTraced($args);
    },
    tap(new SpanContext, function (SpanContext $context) use ($args) {
        $context->setOp('something.measurable');
        $context->setData(['context' => $args]);
        $context->setDescription('Something measurable');
    }),
);

The SpanContext is a unfortunate object in our API at the moment and doesn't have a fluent interface so I've used the tap() helper from Laravel to keep it a little more readable in the example.
This was originally coined in Laravel (getsentry/sentry-laravel#598) but is general enough for all of PHP to benefit from.

$client = new Client();

$spanContext = new SpanContext();
$spanContext->setOp('http.client');
$spanContext->setDescription('GET http://google.com');

trace(
    function (Scope $scope) use ($client) {
        return $client->get('http://google.com');
    },
    $spanContext,
);

$spanContext = new SpanContext();
$spanContext->setOp('http.client');
$spanContext->setDescription('GET http://google.com');

$transaction = SentrySdk::getCurrentHub()->getTransaction();
$span = $transaction->startChild($spanContext);

$client = new Client();
$client->get('http://google.com');

$span->finish();

A small improvement, but it an improvement!

Yeah, the context thing is unfortunate. If it had a fluent syntax we could so some really cool things like:
trace(...)
    ->op('some.op')
    ->description('Something to trace')
    ->run()

Which honestly might be attainable actually... without breaking changes. Worth it?

Not sure tbh. We now have a way better span creator, called profiling 😛
I still think it's worth going this route in v4, but would be ok with the current approach for now.


feat(content-block/item): introducing react wrappers

Related Ticket(s)
#8626
#8631
Description
This PR adds the React wrappers for Content Block and Content Item, as well as its stories and code sandbox examples.
Changelog
New

added React wrappers for Content Block and Content Item

@IgnacioBecerra For the Content Block story, when I turn on the aside knob I get an error

Other than that, it looks good

@IgnacioBecerra Looks like it's failing prettier checks.


name threads created by jawampa

Don't think the thread ID counter needs to be atomic because the factory method will only be called from the thread that creates the NioEventLoopGroup, but it won't hurt. Otherwise it looks good.
Mergin it.
Thanks!


Reproduce an Entity Services environment for support

As a developer using Entity Services, when I run into a problem I want to be able to easily stand up another environment with a similar config in order isolate and reproduce a problem, for example, to share with MarkLogic support.
How does/should this play with the existing “support dump”?

Ping @joseespinoza


Adding form errors and post data to the AddToBasketForm

Since new product options have been added there are a lot more possibilities to create invalid forms in the AddToBasketForm.
When getting an invalid form, errors are not shown.
Also POST data is lost because the form is generated inside a template tag.
I think we can add the form errors and post data to the request.session
Then inside the templatetag we pop out the errors and the post data and add them to the newly created form.

https://github.com/django-oscar/django-oscar/pull/3911


IAM-314-Add piping, shell, and tee to rock

Added components to rock, that are necessary to logging with our current methods.

@bencekov had a chat with the COS guys, so was wondering if we could simply pipe our log outputs to stdout and "a file" and then point promtail to it instead of getting sh and tee?
this is good to merge anyway i guess, we can always pull it out if not needed

This was implemented in the application code after all (https://github.com/canonical/identity-platform-login-ui/commit/dc763681a85c492121e163e1526b901a75ca3849)


[ Bug ] Syntax Error doesn't start the toolkit

Describe the bug
The Toolkit does not start since the newest update (see screenshot). is that on my end or is it just because of the new league patch and it still is missing some updates? I even tried to re-install, but the same syntax error occured.
I even tried to re-install, but the same syntax error occured.
screenshot

To Reproduce
Steps to reproduce the behavior:

start/install toolkit
wait till updates completed
See error

Expected behavior
start of the toolkit
Screenshots
screenshot

System:

OS: win 10
Node: [e.g. 16.4]
Prod Tool Version: newest

This is usually down to DataDragon not being updated/available, although it should be as far as I know.
If it doesn't work by tonight we'll take a look at what might be going wrong.
As a workaround, you might be able to set last-downloaded-version: to 13.15 manually and the download is skipped (although you might lack some assets then).

Hey @sdomi4, which file do you update the last-downloaded-version? I'll try this now as I'm having the same issue (both existing download and fresh install)

Sorry, should have mentioned it - its in ...\prod-toolkit\modules\plugin-config\config.json

No worries! Thank you :)

Hi @sdomi4, I've added it in and saved the file but it doesn't seem to be skipping over the DataDragon download and is still crashing. Any thoughts on this?

I've taken a quick look; It seems that one of the paths changed in CommunityDragon. We'll need to update the static module to fix this.

This line will need to be updated to the new filename in CDragon.
The new URL for the file is: https://raw.communitydragon.org/latest/game/items.cdtb.bin.json
The change comes from Riot changing a filename, might not be permanent.

Should be fixed with an update of module-league-static to 3.10.0. If you manually set a last-downloaded-version in the config, make sure to clear the value to force a full download of assets.

@sdomi4 Where can i found the value to force a full download?

As a workaround, you might be able to set last-downloaded-version: to 13.15 manually and the download is skipped (although you might lack some assets then).

Here, located in the config file at ...\prod-toolkit\modules\plugin-config\config.json

I don't have it. I cannot see it here

Should be fixed with an update of module-league-static to 3.10.0. If you manually set a last-downloaded-version in the config, make sure to clear the value to force a full download of assets.

Quick question @sdomi4, how do you update modules? Apologies if this is super obvious

In the toolkit webpage, go to the modules page and you should have an update button for all modules where updates are available. Make sure to restart the toolkit after updating so everything works properly.

I cannot reach the webpage without updating tool.

I don't have it

I cannot reach the webpage without updating tool.

Good point - You can update the module by hand by downloading the 3.10.0 release module-league-static.zip and copying everything into ...\prod-toolkit\modules\module-league-static\ and overwriting existing files.

I don't have it

Yes you do, it's on line 10

That sounds like working as intended? Or is it still crashing?

It is working indeed :) Thank you very much

Just wanted to say it's working too @sdomi4, thank you for your help today on this. I'll let you know if I come across anything when I cast later today.


Firefox update broke the sidebar.

There is a huge blank space next to the tabs.

Fixed by https://github.com/hakan-demirli/Firefox_Custom_CSS/commit/0421df5bf7022701f9afb108c8aa28658d8ed430.
Thanks to this guy.


Update create-and-import-query-suggestions.md

https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/1882
Duplicated link

Removing all the links - they duplicate what is available on the right pane.


v1.35 causing testing errors with Dropdown and React Testing Library

Package name(s)
Core (@salt-ds/core), Icons (@salt-ds/icons), Theme (@salt-ds/theme)
Package version(s)
@salt-ds/core: 1.35.0
@salt-ds/icons: 1.12.1
@salt-ds/theme: 1.21.0
Description
I upgraded the above versions in our package.json. I then ran npm i.
I have this component:
// DemoFormField.tsx
import {
Dropdown,
FormField,
FormFieldLabel,
} from '@salt-ds/core';
export function DemoFormField(){
  return
    <FormField>
        <FormFieldLabel>Demo</FormFieldLabel>
        <Dropdown
          multiselect
          selected={...}
          variant="secondary"
          onSelectionChange={(_, value) => console.log(value)}
        >
         {/* options */}
        </Dropdown>
      </FormField>
}

Running nx test <project-name> causes an error in our form fields:
TypeError: activeElement.scrollIntoView is not a function
  144 |     );
  145 |     const myDropdown = screen.getByRole('combobox', { name: 'Demo' });
> 146 |     await user.click(myDropdown );`

Steps to reproduce

Upgrade these three packages in your project:

@salt-ds/core: 1.35.0
@salt-ds/icons: 1.12.1
@salt-ds/theme: 1.21.0

Run a test where you click on a <Dropdown multiselect />
Observe an error in your test: TypeError: activeElement.scrollIntoView is not a function

Expected behavior
I expected the tests to pass as they did in<EMAIL_ADDRESS>Operating system

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS
[ ] Android

Browser

[X] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

Are you a JPMorgan Chase & Co. employee?

[X] I am an employee of JPMorgan Chase & Co.

r u using jest and Create React App for testing? If so, maybe you can try add:
Element.prototype.scrollIntoView = jest.fn();
in you setupTests.js file

As @wwwzwbz, this is probably due to JSDOM being used, which doesn't support scrollIntoView https://github.com/jsdom/jsdom/issues/1695#issuecomment-449931788
You can use the solution above or look in the issue on the JSDOM repo for other solutions.

Thank you, it worked perfectly.


After upgrade to 0.6.6 Unit tests fail with : OneTimeSetUp: System.IO.FileNotFoundException : Could not load file or assembly 'Microsoft.Cds.Sdk, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The system cannot find the file specified.

We want to upgrade to version 0.6.6
However, some of our unit tests are failing now
Message:
OneTimeSetUp: System.IO.FileNotFoundException : Could not load file or assembly 'Microsoft.Cds.Sdk, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The system cannot find the file specified.
Is there something else we need to do to upgrade besides upgrading the nuget packages?

Not sure what you're using for unit testing but I had a similar (possibly same issue) because the extensions package had also to be updated:
https://github.com/DynamicsValue/fake-xrm-easy/issues/20

Could you try removing  ?
That was replaced by

Looks like your message is incomplete. Could you elaborate?
Thank you!

Removed Microsoft.Dynamics.Sdk.Messages
Reinstalled Microsoft.PowerPlatform.Dataverse.Client and Microsoft.PowerPlatform.Dataverse.Client.Dynamics  both 0.6.6
but issue is still there. :(

If you try downgrading now to netcoreapp3.1 just to rule out net6 with these package references?

downgraded to netcoreapp3.1 but issue is still there.

Removed Microsoft.Dynamics.Sdk.Messages was removed.. Please remove that from all of your tests.

Hi Matt,
I removed Microsoft.Dynamics.Sdk.Messages
but issue still happens
We are targeting .net 6.0 ( however I also tried downgrading to netcoreapp3.1 as suggested by jordi above)... still the issue persists.
Here's what we have in our project:

ahh... sorry... Original version is 0.5.5 for the 3 nugets: Client, Client.Dynamics and Dynamics.Sdk.Messages.

Removed bin and obj folders from both feature and test projects.
Closed VS 2022
Rebuilt solution
Tests are still failing with this error:
Message:
OneTimeSetUp: System.IO.FileNotFoundException : Could not load file or assembly 'Microsoft.Cds.Sdk, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The system cannot find the file specified.
I also cleared nuget caches from Toos/Options/Nuget Package Manager/General/Clear All Nuget cache(s)

dotnet test reports same failures.
No dlls in GAC
No binding redirects
packages from feature project:
Microsoft.Extensions.Logging Version=6.0.0
Microsoft.PowerPlatform.Dataverse.Client Version=0.6.6
Microsoft.PowerPlatform.Dataverse.Client.Dynamics Version=0.6.6
NetCore.AutoRegisterDi Version=2.1.0
System.Linq.Async Version=6.0.1
System.Linq.Async.Queryable Version=6.0.1
System.ServiceModel.Primitives Version=4.9.0
packages from test project:
Microsoft.NET.Test.Sdk Version= 17.2.0
Moq Version= 4.17.2
NUnit Version= 3.13.3
NUnit3TestAdapter Version= 4.2.1
coverlet.collector Version= 3.1.2

our internal feature project

Few things, you downgraded the project but I still see logging package using net6 right?
Also, Try removing the NetCore.AutoRegisterDi package, it targets .netstandard 2.0 only and the dataverse client package doesn't support that.
@MattB-msft  might be able to assist better here....

@HectorMeneses333 ,
There are few things left to check before I would have to say "I need to see the solution".

Check all projects (csproj files and such) for Nuget package references to Microsoft.cds.sdk.  If you find any, remove them.
Check all includes ( usings ) for Microsoft.cds.sdk,  if you find any, remove them.

In short, there should be no mention of Microsoft.cds.sdk anywhere in your solution.

Ran
grep -r "Microsoft.Cds.Sdk.dll" in our project directory
got this match:
./.vs/{OUR PROJECT FOLDER}/DesignTimeBuild/.dtbcache.v2
I removed it... Ran the test again... still had the issue...
doing a grep for "Microsoft.Cds.Sdk"
got this match:
Binary file ./{OUR TEST PROJECT FOLDER}/bin/Debug/net6.0/Microsoft.Xrm.Sdk.dll matches
So it looks like Microsoft.Xrm.Sdk.dll still holds a reference to Microsoft.Cds.Sdk?

So we are at the point where I now need to see what your doing :)
There is no intra dependency between any shipping lib and Microsoft.Cds.Sdk.
Is this something you can share with me on Git?  If not could you goto the Nuget Package for the Dataverse ServiceClient and click the contact the owners link.   That will send an email to us and I can reach out to you directly to setup a call to review waht you have going on.

@HectorMeneses333  I might be a bit biased, but given the time spent by you, Matt and myself just trying to build the solution and we haven't even mocked anything yet, maybe you could give FakeXrmEasy a go... supports both DataverseClient and CrmSdk.CoreAssemblies, it comes with an In-Memory database + its own query engine, and even pipeline simulation features, plus you don't have to mock anything....

Matt, I will reach out to you shortly.
Jordi, I'm not familiar with FakeXrmEasy... but will review that in the future.

Thank you for your help identifying the circular reference we had in our projects. A referenced project had a stale dll with the reference to Microsoft.Cds.Sdk.


.editorconfig

This adds .editorconfig file to avoid VisualStudio tabs vs spaces issue.
Dunno why git reports my old commits. should be merged only .editorconfig file add commit.

You need to fork a new branch from master (and not from the previous commits you did) and add the commit there, otherwise we get the other commits

Closing. will submit new one.


Fix PED UG bugs for Export and Import

Description
Fixes made in response to PED errors regarding UG documentations for Import and Export.

To be added

Type of change

[X] Documentations

How to test
Preview Link: To be added
Checklist

[X] I have updated the User Guide
[ ] I have included a preview deployment link to the updated project documentation

Codecov Report

Merging #198 (8daec33) into master (2c816e2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #198   +/-   ##
=========================================
  Coverage     72.24%   72.24%
  Complexity      542      542
=========================================
  Files            93       93
  Lines          1762     1762
  Branches        195      195
=========================================
  Hits           1273     1273
  Misses          437      437
  Partials         52       52

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 2c816e2...8daec33. Read the comment docs.


Little confused on how to get this to work locally

I think I may be misunderstanding what address to use for the last line in the config.php file. Want to run this locally.. suggestions? Localhost does not work for me

Looks like I have everything set up but the vendor autoload piece... i get this error:
Warning: require_once(vendor/autoload.php): failed to open stream: No such file or directory in C:\xampp\htdocs\models\SpotifyStats.php on line 3
Fatal error: require_once(): Failed opening required 'vendor/autoload.php' (include_path='C:\xampp\php\PEAR') in C:\xampp\htdocs\models\SpotifyStats.php on line 3

Added an autoload script, but now getting this:
Uncaught Error: Class 'SpotifyWebAPI\Session' not found in C:\xampp\htdocs\SpotifyStats.php:5 Stack trace: #0 {main} thrown in C:\xampp\htdocs\SpotifyStats.php on line 5

Sorry for the late response, on vacation at the moment.
The requirement of the redirect value in configuration is where Spotify will redirect once the user logs in and the auth token is sent back to the website. So if I click on the login button, I'll be redirected to Spotify's login page. This will then redirect me back to the auth page, which by default is http://localhost/auth.php. You only need to provide the base root of the website, the auth.php trailing is added by the script.
Seems like your Composer didn't install the MySQL library properly? Is the vendor folder located inside of the website direction with the Spotify dependencies? This can be installed by using Composer and running the following command in the root of the website. (composer require jwilsson/spotify-web-api-php)

This still doesn't help solve the original issue? I am having the same problem trying locally host. I get the following error

Warning: require_once(vendor/autoload.php): failed to open stream: No such file or directory in C:\xampp\htdocs\mySpotify-master\mySpotify-master\models\SpotifyStats.php on line 5

Fatal error: require_once(): Failed opening required 'vendor/autoload.php' (include_path='C:\xampp\php\PEAR') in C:\xampp\htdocs\mySpotify-master\mySpotify-master\models\SpotifyStats.php on line 5

@Steven-FJX Hi, did you run the composer install command? You can install composer at https://getcomposer.org/, then open up CMD in the root of the folder where mySpotify is and run the command composer require jwilsson/spotify-web-api-php. This should make the required files to load.
Let me know if you have any other questions!


Configure Quantity of BTC to Send along with Asset

This will make it faster to set up dispensers on empty addresses, if nothing else.

perhaps a more generate update to the api to have any tx generation accept an array of additional outputs to be appended prior to the change in the form of  [{address|script, value}]?
This could allow for your specific case, but also in the future with #2197 a user could open a dispenser much like they currently do (assuming a pre-signed tx can fit in an op return)
step 1) generate a pre-signed tx to open a dispenser on the desired address for asset A and encode it in an op_return script
step 2) generate a tx to send asset A to the above address using multisig and pass {op_return_script,0} as an additional output to be added to the tx.
Then when cp decodes the tx, asset A will go to the desired address and then a dispenser will be opened

another reason it would be advantageous to have additional_outputs:[{address|script, value}] would be so that stamps can be made with the API without requiring external bitcoin tx construction libraries.  We currently use bitcoinjs-lib and deconstruct the results from the API into a PSBT and then add additional p2wsh outputs to encode the stamp data (olga).
By allowing for arbitrary outputs, users will just have to do the p2wsh encoding, and then pass an array of addresses and dust values that represent the data. It lowers the barrier to entry because new devs wouldn't have to learn about the nuances of bitcoin psbt construction.

done herehttps://github.com/CounterpartyXCP/counterparty-core/pull/2873
you can now use the parameter --more-outputs:
"more_outputs": (
        str,
        None,
        "Additional outputs to include in the transaction in the format `<value>:<address>` or `<value>:<script>`",
    ),


[Feature] animate css variables itself

Is it possible to animate css variables itself?
In this nice Pokemon cards example:
https://codepen.io/simeydotme/pen/abYWJdX
made with svelte motion, they are animating the css variables.
Is something like this possible?
And does it even make sense performance-wise?
animate(
  ".wrapper",
  {
    "--t1": "400px"
  },
  {
    duration: 2,
    repeat: Infinity
  }
)

I know that svelte motion is based on framer motion/popmotion and think i understand the difference to the WAAPI-Approach from Motion One, but i wondered why it shouldn't be possible to change a css variables itself, isn't it a property like any other?
const wrapper = document.getElementById("wrapper");
animate(
  progress => {
    wrapper.style.setProperty("--t1", `${progress * 400}px`)
  },
  {
    duration: 2,
    repeat: Infinity
  }
)

I think it is possible but only browswrs where CSS.registerProperty() exists - otherwise WAAPI doesn’t know how to animate these values

I tried it here: https://codepen.io/crioco/pen/abYEKRQ
But it doesn't seem to be working.
I use Chrome (Version 103.0.5060.134) on Mac OS and CSS.registerProperty() is available.

I should have been clearer you  have to actually call it https://developer.mozilla.org/en-US/docs/Web/API/CSS/RegisterProperty

Apologies I was on a phone and missed your initial example too.
For your second example you need to call CSS.registerProperty
But doing it like your first example works as expected: https://codepen.io/SirHound/pen/ZExvVGz

From your initial post, the second example in there is how Framer Motion is actually working under the hood.
We can't do that with Motion One as part of the benefit of using WAAPI is getting all the expensive interpolator code for free. Framer/Svelte Motion etc bundle all this in, and it isn't generally as good as what's included in the browser.
Which is why this currently only works with browsers that support CSS.registerProperty.
It's not too difficult to go from what we have now to allowing the animation of number CSS variables in all browsers as that's basically how the independent transform code is working. We could additionally support unit values for pretty cheap. But anything more complicated than that is difficult.

Thanks for the Explanation and your patience. Honestly, i'm not sure if I fully get it.

This one works even if CSS.registerProperty is not available.

const wrapper = document.getElementById("wrapper");
animate(
  progress => {
    wrapper.style.setProperty("--t1", `${progress * 400}px`)
  },
  {
    duration: 2,
    repeat: Infinity
  }
)

and if i understand it right there is no WAAPi working in the background.
So far I still understand it, I think

This one does not work in any browser, even if i call CSS.registerProperty

window.CSS.registerProperty({
  name: '--t1',
  inherits: true,
  initialValue: '0',
});

animate(
  ".wrapper",
  {
    "--t1": "400px"
  },
  {
    duration: 2,
    repeat: Infinity
  }
)

I do understand that Motion One let the browser (WAAPI) do the complex unit calculation, but i dont understand why the browser can't calculate the change in my variable.
What is the differnece between animating to: "--t1": "400px"and transform: "translateX(400px). Isn't that why I call "CSS.registerProperty" so that the browser understands the variable like any other CSS property?
Sorry for so many questions.

Your explanation for point 1 is good. Point 2 there must be something amiss with the code because this technique is how we animate independent transforms in Chrome. I think syntax is marked as optional in MDN but try adding syntax: "<length>" to the registerProperty call as that's the fundamental difference here (to answer your question), without this nothing has actually changed in terms of the browser's understanding of the value except you've added some rules around defaults and inheritence.

Wow, thanks.
I was about to give up, but it works.


Marker position freezes while zooming animation.

Discussed in https://github.com/Leaflet/Leaflet/discussions/7910

Originally posted by MatixYo January 14, 2022
I'm building an app where markers' position is updated with approximated position of cars.
During zooming animation marker stays in position where it was before zooming and only moves to the correct position after the map was zoomed in. Here's an example: https://codesandbox.io/s/leaflet-awesome-markers-forked-wrjnt?file=/src/index.js
Follow the marker and try to zoom it. My question is, is there any way to prevent that marker glitching when zooming?
It happens on both Windows 11, Chrome 97, and on Android. Chrome 97. On a Mobile device, you can see it very well when zooming with + and - buttons. Using Leaflet 1.7.1

Is this duplicate of https://github.com/Leaflet/Leaflet/issues/7401?

Is this duplicate of/related to #7401?

May be related, however, this one applies to moving markers. It happens when marker changes position during zoom.

Duplicate of #7017


Implement godef through Go 1.5's vendor experiment.

Resolves properly in vendor/ folders down the tree from the file we're querying.
Things in init() weren't used anyway, as types.GoPath was never read/used.

With not much work, we could get godef to crawl through gb vendor paths too..

No? not interested?! :) I've been using this and recommending to others.. anyone who uses vendor/ paths.

Big +1 for this change

You can always install it with go get github.com/abourget/godef/... that's what I do..

Thanks, that's what i'll do. I am lost w/out godef support in emacs :)
On Wed, Dec 30, 2015 at 11:00 AM, Alexandre Bourget <
<EMAIL_ADDRESS>wrote:

You can always install it with go get github.com/abourget/godef/...
that's what I do..
—
Reply to this email directly or view it on GitHub
https://github.com/rogpeppe/godef/pull/14#issuecomment-167932812.

@abourget Thanks for your fork, it works perfectly with go-plus for Atom!

It works perfect for me!

@rogpeppe any thoughts about getting this in soon? Would be very appreciated over here.

Superceded by #33


Scheduler should warn the user (or throw error) that mesos ignores changing of framework-role and user after the framework is reqistered.

After the framework is registered it seems to be not possible to change several framework params like:

framework-role;
user;
may be some other (need to clarify a complete list);

Mesos accepts changed params, but just ignores them. That could be extremaly confusing to the user,
as the case with Vidhya shown. To improve the usability Scheduler can raise an error (or warning) if that situation happens. To my mind raising an error is the correct approach, cause it will bring the user's attention to the problem.

Suppose this is not actual with new mesos anymore.


Share a transport layer between multiple SDKs

@jlacivita  can you review?

There was some major scoping issues with the transport layer that needed to be addressed to support multiple SDKs.
The transport layer was scoped to a file/module, which means if there was 2 SDKs there were 2 transport layers. This also meant that there was two id counters for the messages.
So if core-SDK sent:
{ id: 1, method: "lifecycle.onForeground"}

Then manage SDK sent:
{ id: 1, method: "appcatalog.catalog" }

And FireboltOS responded:
{ id: 1, result: { "apps": [] } }

How does the SDK know this response is meant for the manageSDK and not the coreSDK?
We could maybe treat the two SDKs as separate clients just as if they were different apps. This would work for the websocket transport because there would be two separate websocket connections and FireboltOS can just respond through the correct one. Unfortunately that does not work with the Legacy transport. We can only target messages to a browser, not a specific client within a browser.
The solution is to only have a single transport per app, not per SDK, and thus only have a single id counter.
In order to accomplish this, we had to make the transport a global variable so it can be used by both SDKs. This would be very hard with how the transport file was written, as every variable was scoped to the module/file. So I changed the transport file to be an ES5 class instead (this is better practice anyway). And then instantiated that ES5 class as a singleton global variable to be shared across SDKs. One caveat was in the off-device static mock case, we should NOT use the same transport. As the mock transport contains defaults specific to the SDK it is in.
The end changes here look like a lot, but most of it is just moving methods into an ES5 class and change variable scoping to be this.. Viewing this review without whitespace changes helps a little bit: https://github.com/rdkcentral/firebolt-openrpc/pull/8/files?w=1


Form overview page: Up, Down and Edit buttons don't have useful accessible names

The 'Up', 'Down' and 'Edit' buttons on the form overview page don't have any additional visually hidden text, so for a screen reader user they will appear as identical buttons and it won't be clear what they do or which items they affect.

Is there a way we could have sr-only text say something like:
Move above [question above name]
Move below [question below name]


[BEAM-4414] Create more specific namespace for each IOIT in FileBasedIOIT

Replaced a namespace with a job name parsed to have only lower case alpha-numeric characters with -.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ x ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

IMO it looks ok.

@jasonkuster @lgajowy Would you take a look at this? Thanks!

LGTM


[GDAX] Canceled, not filled order

Hello, first of all, thank you for such great tool like Gekko.
I am running gekko live with GDAX what working pretty well, although I have some questions.
Using advice('short') or advice('long'), it's correctly added to order on GDAX. But sometimes, the request is not filled and order cancelled - this cause me a problem as in my strategy I am couting automatically that it's filled and do the rest of my code.
For example, I am recognizing if sell (advice(short) ) or purchase ( advice(long) ) was done as previous action. If advice is not really filled on GDAX, my code continues the wrong way as in the code is not recognized if request was filled or not.
Is there any way to do not cancel my order and wait until it's filled, or check if for example advice('long') was really filled and purchase really processed on GDAX ?
Thank you so much for any advice.

When your advice signals "short" Gekko will try to sell, it will do this by creating orders on your side of the book and if they don't fill cancel and recreate them. Documentation is poor but you can read a bit more about that here: https://gekko.wizb.it/docs/introduction/scope.html#Execution-strategy

Is there any way to do not cancel my order and wait until it's filled, or check if for example advice('long') was really filled and purchase really processed on GDAX ?

You are in luck! There now is a new handle called onTrade for your strategy that is called whenever the trader has successfully completed the trade. See here for details #1850 and you need to download the latest develop version to access it.

I hope that answers your question, if it doesn't please comment again :)

Hey Mike, that's perfect, thanks a lot.
inTrade is exactly what I was looking for, I am continue to use develop branch from now...
Just for your information, recreating the order after cancel worked and still working fine, I just looked for the way to disable cancelling what I finally found. But now with inTrade handle, no more needed to disable cancel of order.
Thanks again for your willingness and answer on my questions.
Have a great day.

no problem, and if you have more questions feel free to ask :)

Hi again, using the develop branch I am experiencing errors more often while gekko is used live on the market.
Find please errors described below...I am not sure if it has something to do, but for me it looks like the problem comes after more attemts to foll my order. Order was few times cancelled and created again, but after some time of this state, errors appear...
If it would have to do something with onTrade function which I started to use, I have there nothing more than:
method.onTrade = function() {
    this.waitingInOrder = false;
    this.diffCandles = 0;
    console.log("\torder completed, can continue...");
}

Would you be so kind to check it please?
Thank you.

(have not copied error), but console said that in file ./exchanges/gdax.js
in function: Trader.prototype.checkOrder = function(order, callback)
was not defined "data"
It was general error that cannot call status of undefind, at line: var status = data.status;

I have copied error message:
E:\gekko\gekko-develop\plugins\performanceAnalyzer\performanceAnalyzer.js:112
duration: this.roundTrip.exit.date.diff(this.roundTrip.entry.date)
^

TypeError: this.roundTrip.exit.date.diff is not a function
at PerformanceAnalyzer.handleRoundtrip (E:\gekko\gekko-develop\plugins\performanceAnalyzer\performanceAnalyzer.js:112:40)
at PerformanceAnalyzer.bound [as handleRoundtrip] (E:\gekko\gekko-develop\node_modules\lodash\dist\lodash.js:729:21)
at PerformanceAnalyzer.logRoundtripPart (E:\gekko\gekko-develop\plugins\performanceAnalyzer\performanceAnalyzer.js:94:10)
at PerformanceAnalyzer.bound [as logRoundtripPart] (E:\gekko\gekko-develop\node_modules\lodash\dist\lodash.js:729:21)
at PerformanceAnalyzer.processTrade (E:\gekko\gekko-develop\plugins\performanceAnalyzer\performanceAnalyzer.js:72:8)
at Trader.bound (E:\gekko\gekko-develop\node_modules\lodash\dist\lodash.js:729:21)
at emitOne (events.js:121:20)
at Trader.emit (events.js:211:7)
at Trader.bound [as emit] (E:\gekko\gekko-develop\node_modules\lodash\dist\lodash.js:729:21)
at Manager.Trader.manager.on.trade (E:\gekko\gekko-develop\plugins\trader\trader.js:24:10)
RECEIVED ERROR IN GEKKO<PHONE_NUMBER>34122
Child process has died.


Added Sagui implementation

Sagui architecture implementation with "custom extension" of break.

Wow, this time CodeQL was actually usefull.


Port: Added Just In Time installation flow in messaging extension action sample

Is your feature request related to a problem? Please describe.
Port these additions to javascript, python and java samples:
https://github.com/microsoft/BotBuilder-Samples/pull/3012

Teams samples moved out of BF Samples


k8s - k8s dashboard "service unavailable" if not logged into Rancher

Rancher Versions:
Server: v1.4.1
kubernetes (if applicable):
rancher/k8s:v1.5.2-rancher1-2
Docker Version:
1.12.6
OS and where are the hosts located? (cloud, bare metal, etc):
RHEL7.3, vmware
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node Rancher, external mysql db
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
k8s
Steps to Reproduce:
log into Rancher, change to a k8s env, launch k8s dashboard ui, log out of Rancher, refresh k8s dashboard ui
Results:
"Service Unavailable" on k8s dashboard ui if logged out of Rancher.

same problem

Experiencing this on RancherOS guest vm on Libvirt (Centos7) as well
Docker Server 1.4.1
ancher/k8s:v1.5.2-rancher1-2
Docker version 1.12.6, build 78d1802
RancherOS on Centos7/KVM
One guest vm for RancherOS Server
Another guest vm for the agent
Steps to reproduce:
Log into rancher, change to a k8s env, launch k8s dash ui (never logs in)
What logs or other tools can be used to troubleshoot this? Thanks in advance for any help.

I confirm having the same issue on
rancher version:1.5.1

While not the most helpful message, this is a feature.  Rancher access control controls access to the kubernetes running inside it, if you're not logged in then you can't user it.

@vincent99 That seems to contradict the reason that Rancher stopped using their own k8s pages, and decided to use the native k8s dashboard introduced in 1.4.0, what is the point of giving users the native k8s ui if i have to go to the Rancher ui anyway? #ux101

@vincent99 i got it that this is a feature, but im logged in in my case (which is btw only working with http, not https (via proxy reverse)).
Should i create a new ticket or can someone provide a link with further information and troubleshooting?
(Also - can i run default environment and k8 environment on the same machine? it disconnected my other env and im afraid i cant manage the other containers as before...)

I dont know if its related or not but seam to be having the same issue .
Rancher Versions:
Server: v1.5.10
Docker Version:
1.12.6 (for nodes)
17.03-ce (for server)
OS and where are the hosts located? (cloud, bare metal, etc):
Centos7 Xenserver
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node Rancher, external mysql db
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
k8s
Steps to Reproduce:
Deploy service and try to access dashboard
Note :
Results:
"Service Unavailable" on k8s dashboard

@dexterddit I have the same environment and facing the same problem.
But i think the reason is that my k8s-dashboard is not ready.
# kubectl get nodes
NAME        STATUS     AGE
localhost   NotReady   9m

Infrastructure Stacks is healthy but kube pod stuck in Pending.
# kubectl get pods --all-namespaces
NAMESPACE     NAME                                    READY     STATUS    RESTARTS   AGE
default       mysql-vhfpk                             0/1       Pending   0          51m
kube-system   heapster-818085469-chgfq                0/1       Pending   0          1h
kube-system   kube-dns-1208858260-nt95c               0/4       Pending   0          1h
kube-system   kubernetes-dashboard-2492700511-p8cjh   0/1       Pending   0          1h
kube-system   monitoring-grafana-832403127-crn8v      0/1       Pending   0          1h
kube-system   monitoring-influxdb-2441835288-m247m    0/1       Pending   0          1h
kube-system   tiller-deploy-3991468440-zln7b          0/1       Pending   0          1h

Where are you from? in china , u can not download from gcr.io because gfw firewall. u need change kube-system image source

@anjia0532  哈哈，被发现了。卡了这么久大家都知道是什么原因，开了代理就ok了。

@jackiehope 你是咋开的代理？(how do u use proxy？) 系统全局代理？(os global proxy？) docker代理？(docker proxy？)，我用了docker代理，不起作用(the docker proxy setting is not work)
https://docs.docker.com/engine/admin/systemd/#httphttps-proxy
我做了gcr.io的镜像(i make a gcr.io's mirror on docker hub)，超过1280个容器(more than 1280 containers) https://hub.docker.com/r/anjia0532/

@anjia0532  I forgot that, not using rancher for a few months。

This is becoming a gathering of different issues, please open a new issue when you experience problems. Regarding the original issue, I think it was already answered that the auth of Rancher is needed to access the dashboard.


Add stages models to the thorlabs kinesis motors service

Add the Z806 and Z812 to the list of accepted stages for the Thorlabs Kinesis motors.

The 2 stages we need are added to the list, and I erased the unecessary comments you told me !
This is ready for review.

@erinpougheon have you checked that the new stages use the same gear step per revolution, gear box ratio and pitch like the ones that are already in the list? I believe @a-sevin had checked their manuals to confirm the numbers.

@erinpougheon have you checked that the new stages use the same gear step per revolution, gear box ratio and pitch like the ones that are already in the list? I believe @a-sevin had checked their manuals to confirm the numbers.

I checked right now in all the manuals to be sure, and yes they all have the same gear step per revolution, gear box ratio and pitch!


Unable to issue a certificate

Reference to: #1347
Unable to issue a cert: https://www.useloom.com/share/34ae34f613184e748019e772612752bc
URL: https://blog.lovincyrus.com
Cloudflare:
CNAME      blog      alias.zeit.co
TXT		_now	QmckKAT2dxUWLcewv8jYhvxAhLDsTmwnpGJARX7wKwQ4to
Crypto
-> SSL: Full (strict)
Page rules
-> blog.lovincyrus.com/.well-known/acme-challenge/*
-> SSL: Off https://github.com/zeit/now-cli/issues/1569#issuecomment-473443677
Any help would be appreciated 🍻

I can't see the CNAME in place: https://www.whatsmydns.net/#CNAME/blog.lovincyrus.com

Interesting, I'll remove the CNAME record and insert it again.

Fixed!
Changes:

Change CNAME blog alias.zeit.co (DNS only)
Update page rule to dynamic pattern .lovincyrus.com/.well-known/acme-challenge/
Add TXT _acme-challenge.blog [value]

Notes:

https://zeit.co/blog/domain-verification

I'm stuck here too. Got the email that said my certificate expired. Nothing has changed with my Cloudflare configuration. Still I cannot verify and after running now verify, now inspect, and then now domains rm and now domains add it seems I've hit a rate-limit on issuing certs?
> Error! Too many certificates already issued for exact set of domains: www.morganatwork.com

In the blog post mentioned above ^^, there is a screenshot of cloudflare in which DNS and HTTP proxy is turned off:

Is that necessary?

@morgs32

Your CNAME for your domain (www.morganatwork.com) seems to be working fine.

Yep, that is necessary. I'd make sure that it's only DNS only like how it's being described on the blog post.
Then, I'd https://<IP_ADDRESS>/purge-cache/ your CNAME domain
It takes some time for DNS propagation. After that, you could try to repeat the same steps:
now domains rm, now domains add <domain>, now verify, now inspect, now certs create <domain>, take care of your aliases after that

Reference:
https://zeit.co/blog/now-certs


portals: Show timepicker for time fields.

Show a time picker, preferably https://flatpickr.js.org/ , and attach it to a sample field. The output should be a timestamp that can then be saved in the database.
Necessary steps: Take a look at the edit event form, and its public js scripts in the portal. Attach that timepicker to the startime and endtime fields.
File to make changes to: https://github.com/crux-bphc/fest-manager/blob/pearl/views/portals/event.jade

I will take up this issue.


when using custom schema relation "gue_jobs" does not exist

Worker failed to lock a job level=error worker-pool-id=959ce5 worker-id=959ce5/worker-0 error=could not lock a job (rollback result: ): ERROR: relation "gue_jobs" does not exist (SQLSTATE 42P01)

Can you reproduce this using test? E.g. add one more DB to services specifically for the test purpose and cover the scenario that should fail.

You have to set the schema on Postgres when opening a new connection. The schema is also called search_path. For example when using the pgx driver you can do this
config, _ := pgx.ParseConfig("host=local port=5432 user=blah password=blah dbname=db sslmode=disable")
config.RuntimeParams = map[string]string{
"application_name": "my app",
"search_path":  "custom_schema_name",
}


WIP: getting the clusters the user is authenticated for

depended on https://github.com/fabric8-launcher/ngx-launcher/pull/165 and a new release

[test]


Add type safety

Since it's possible to generate typescript definitions from an OpenAPI specification (ie with https://www.npmjs.com/package/dtsgenerator), I wonder if it would be possible to implement type safety between the validator middleware and the following middlewares.
For example, dtsgenerator would transform https://github.com/PayU/openapi-validator-middleware/blob/master/test/pet-store-swagger.yaml to the following types declarations file:
// extract of the file...
declare namespace Paths {
    namespace CreatePets {
        export interface BodyParameters {
            body: Parameters.Body;
        }
        namespace Parameters {
            export interface Body {
                name: string;
                age?: number;
                tag?: string;
                test: {
                    field1: "enum1" | "enum2";
                };
            }
        }
        namespace Responses {
            export type Default = Definitions.Error;
        }
    }
// ...

Ideally, the validator would return a request with the expected type (something like Request<{}, {}, Paths.CreatePets.Parameters.Body>, see https://stackoverflow.com/a/55413670/7351594), so that the following middleware / route handler would know the type of its req argument:
app.post(
  '/pets',
  swaggerValidation.validate,
  (req: Request<{}, {}, Paths.CreatePets.Parameters.Body>, res, next) => {
    return res.json({ result: 'OK' });
  }
);

That way, the following route handler would not have to assume validation has already been done, thus reducing code coupling.

See also https://federicoferoldi.com/2017/12/28/using-the-typescript-type-system-to-validate-express-handlers.html that applies that idea.

Thanks @severo, this is an interesting idea!
This is something that should be exposed from api-shema-builder, as this is the package that is responsible for parsing the definitions file.
We'll try to see what we can do when we'll have the time, but of course PRs are always welcome.


Improvements to target API + implement target interaction cards

In order to support cards that interact with targeting, the following updates to the target API will be needed.

[ ] Directly support 'each player chooses' cards (e.g. Wildfire Assault).
[ ] Directly support 'choose X card controlled by each player' cards (e.g. Even Handed Justice)
[ ] For convenience, either a flag needs to be added to indicate prompts that choose a card but the ability does not contain the word 'choose' (e.g. Tears of Lys), or it needs to be separated out to another property that acts similar to target.
[x] Convert cards to consistently use the target API (examine every card that still uses promptForSelect - some will probably still be necessary because the choose occurs in a 'then' clause ).
[ ] Make it easier to detect that the ability only targets a single card (hard to do without the above, but could do earlier if an indeterminate number of bugs are tolerated in the meantime)
[ ] Allow targets to be updated on the context object (High Sparrow specific)

Adding to this: a card like Arianne Martell is currently implemented using the target API. This works out well, because even though Arianne doesn't target, the card she puts into play has to be revealed during Step 1 of Initiating Abilities, which is now taken care of by the target API. However, such cards might need a separate flag or a similar but different property as well, so it won't conflict with cards that interact with cards that truly target.
Some related fresh rulings: http://www.cardgamedb.com/forums/index.php?/topic/37127-players-choices-vs-cancels/


PWN-6677 Single line recycler for ActionButton

Jira Ticket
https://p2pvalidator.atlassian.net/browse/PWN-6677
Description of Work
Single line recycler for ActionButton

hi, Nikolai. our commit writing formatting rule is:
PWN-6677 - single line recycler for ActionButton
instead of
PWN-6677 Single line recycler for ActionButton (we put dash and start message from lowercase)

i checked it myself, looks good


⚠ Bofrost.gr has degraded performance

In f4a4193, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 1594 ms

Resolved: Bofrost.gr performance has improved in d8a8486.


Add log file config variables

Hello @weibullguy! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/ramstk/utilities.py:

Line 24:9: E126 continuation line over-indented for hanging indent


[Snyk] Security upgrade typeorm from 0.2.24 to 0.2.25

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json
package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Prototype Pollution SNYK-JS-TYPEORM-590152
  528

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


Adjust Monitoring And Health Widgets to follow the guidelines

Description

Expected result

The Cards should be fixed width, not responsive.
Empty space will be allowed to right.
For small cards we should look into using analytical card as a base
the 3 dots menu is not needed for now
Implement the deep link. When a card has deep link Learn more is present. Adjust the cards that can have deep links.

Actual result

Steps to reproduce

Troubleshooting

https://sap.github.io/ui5-webcomponents-react/?path=/docs/data-display-analyticalcard--docs


REST backend omits paths given in the URL

Discussed in https://github.com/rustic-rs/rustic/discussions/27

Originally posted by dimejo June 14, 2022
I'm unable to connect to a rest-server with rustic. The command I have used:
rustic -r rest:http://<IP_ADDRESS>:8000/reponame -p repo.pass snapshots
Error: No config file found. Is there a repo?

The same command does work with restic. Any ideas?

As pointed out in #27 this is no issue if the path ends with a trailing '/'.


[CLI] [Bug] KeyError when adding a new dictionary as statistic to existing run

Description
I'm adding a new summary to the existing run following the docs https://docs.wandb.ai/library/public-api-guide.
If a new summary is a scalar value, everything works fine, but if it's a dictionary, I get the following:
$ python tmp_wandb.py
Traceback (most recent call last):
  File "tmp_wandb.py", line 5, in <module>
    run.summary.update({'new_metric_d': {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'}})
  File "C:\Users\Maria\.conda\envs\Garments\lib\site-packages\wandb\old\summary.py", line 179, in update
    write_items = self._update(key_vals, overwrite)
  File "C:\Users\Maria\.conda\envs\Garments\lib\site-packages\wandb\old\summary.py", line 198, in _update
    self._dict[key] = SummarySubDict(
  File "C:\Users\Maria\.conda\envs\Garments\lib\site-packages\wandb\old\summary.py", line 48, in __init__
    json_dict = json_dict[k]
KeyError: 'new_metric_d'

Wandb features
run.summary.update() according to the docs https://docs.wandb.ai/library/public-api-guide.
How to reproduce
This is the minimum script:
import wandb

run = wandb.Api().run(<run_path>)
run.summary.update({'new_metric_d': {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'}})
run.summary.update()

The run at <run_path> already exists and has a "failed" state (in case it's important)
Here is the run I work with: https://wandb.ai/maria_korosteleva/Garments-Reconstruction/runs/5cnaly5h/
Environment

OS: Win 10
Environment: Anaconda + Visual Code
Python Version: 3.8.5
wandb Version: 0.10.21

Hey @maria-korosteleva we'll look into the root cause.  Until then you should be able to do the following:
run.summary["new_metric_d"] = {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'}
run.summary.update()

Hey @vanpelt
Just letting you know that the issue is still there in 0.10.25. And both of the methods don't work (the one in the issue description and the one that you proposed).

run = wandb.Api().run(<run_path>)
run.summary.update({'new_metric_d': {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'}})
run.summary.update()

run = wandb.Api().run(<run_path>)
run.summary["new_metric_d"] = {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'}
run.summary.update()

Both fail with the same KeyError: 'new_metric_d'  😥

I just reproduced this issue on 0.10.27. From what I can tell, a fix might be swapping lines 144 and 145 here, so that the key is added to root._json_dict before SummarySubDict tries to access it in __init__: https://github.com/wandb/client/blob/321654aea55440622371594da9279f4e97f25607/wandb/old/summary.py#L137-L149
Something similar would also need to be done for the update method.
In the meantime, this workaround did the trick for me:
def update_run(run, k, v):
    if (isinstance(run.summary, wandb.old.summary.Summary) and k not in run.summary):
        run.summary._root_set(run.summary._path, [(k, {})])
    run.summary[k] = v

Same issue in 0.12.7. @corneliusboehm's works for me, thank you!

Yeah, same for me. @corneliusboehm 's workaround did the trick. Thanks a lot!

The issue still persists.

Issue still persists

WandB Internal User commented:
maria-korosteleva commented:
Hey @vanpelt
Just letting you know that the issue is still there in 0.10.25. And both of the methods don't work (the one in the issue description and the one that you proposed).

run = wandb.Api().run(<run_path>)
run.summary.update({'new_metric_d': {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'}})
run.summary.update()

run = wandb.Api().run(<run_path>)
run.summary["new_metric_d"] = {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'}
run.summary.update()

Both fail with the same KeyError: 'new_metric_d'  😥

For now, the workaround that worked for me is recording each dictionary entry one by one as individual scalar values with extended names, e.g.
import wandb

def add_statistic(run, tag, info):
    """Add info the run summary (e.g. stats on test set)"""
    if isinstance(info, dict):
        for key in info:
            add_statistic(run, tag + '.' + key, info[key])
    else:
        run.summary[tag] = info
        run.summary.update()

run = wandb.Api().run('maria_korosteleva/Test-Garments-Reconstruction/2ckc3ecs')
add_statistic(run, 'new_metric_d2',  {'full_loss': 2.5, 'Extra_loss': 1.5, 'type': 'dict_metric'})

WandB Internal User commented:
github-actions[bot] commented:
This issue is stale because it has been open 60 days with no activity.

WandB Internal User commented:
corneliusboehm commented:
I just reproduced this issue on 0.10.27. From what I can tell, a fix might be swapping lines 144 and 145 here, so that the key is added to root._json_dict before SummarySubDict tries to access it in __init__: https://github.com/wandb/client/blob/321654aea55440622371594da9279f4e97f25607/wandb/old/summary.py#L137-L149
Something similar would also need to be done for the update method.
In the meantime, this workaround did the trick for me:
def update_run(run, k, v):
    if (isinstance(run.summary, wandb.old.summary.Summary) and k not in run.summary):
        run.summary._root_set(run.summary._path, [(k, {})])
    run.summary[k] = v

WandB Internal User commented:
github-actions[bot] commented:
This issue is stale because it has been open 60 days with no activity.

WandB Internal User commented:
huy29433 commented:
Same issue in 0.12.7. @corneliusboehm's workaround works for me, thank you!

WandB Internal User commented:
universome commented:
The issue still persists.

WandB Internal User commented:
noamsgl commented:
Issue still persists

Do not close this issue please because it still persists and causes inconvenience.

WandB Internal User commented:
universome commented:
Do not close this issue please because it still persists and causes inconvenience.

Hi All, please note this issue with be fixed with the next SDK release. See PR above for detail.


Refactor diagnosticplots(::PseudoMaximaEVA)

Codecov Report

Merging #11 (ad839e7) into dev (c00afe4) will decrease coverage by 12.16%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##              dev      #11       +/-   ##
===========================================
- Coverage   75.83%   63.67%   -12.17%
===========================================
  Files           4        4
  Lines         331      267       -64
===========================================
- Hits          251      170       -81
- Misses         80       97       +17

Impacted Files
Coverage Δ

src/structures/pseudomaximaeva.jl
30.61% <0.00%> (-37.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 c00afe4...ad839e7. Read the comment docs.


chore: remove usages of it.each in unit tests

:pencil: Description
it.each() is currently broken in the latest version of jest, so removing usages in this repo. I found this by changing a value in the "each" array, and the test falsely passes.

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


'sn' is not recognized as an internal or external command

**********************************************************************
** Visual Studio 2019 Developer Command Prompt v16.3.4
** Copyright (c) 2019 Microsoft Corporation
**********************************************************************

C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional>sn
'sn' is not recognized as an internal or external command,
operable program or batch file.

C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional>sn.exe
'sn.exe' is not recognized as an internal or external command,
operable program or batch file.

The SN tool documentation page says

To start the tool, use the Developer Command Prompt

What am I missing?

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

ID: c2d65a40-0ea7-09ce-33fc-39525310b05e
Version Independent ID: f6cbd1c2-7708-03aa-a935-0dbfa07e6177
Content: Sn.exe (Strong Name Tool)
Content Source: docs/framework/tools/sn-exe-strong-name-tool.md
Product: dotnet-framework
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

The sn.exe tool is located in *C:\Program Files (x86)\Microsoft SDKs\Windows\v10.0A\bin\NETFX 4.8 Tools\x64*.

@Youssef1313, Thanks. Of course, I can find it and run it. The problem is the SN tool documentation page says

To start the tool, use the Developer Command Prompt

And I used to be able to do so (although it was called "Visual Studio Command Prompt" back then).
Not sure if it is a problem with the documentation of my set up.

@bzaar, On my machine, I can't run the sn command unless I'm in that directory. so I suspect it's a doc issue. but you may need to wait until the issue get reviewed and triaged.

Not sure. I can run just fine just by opening the Command prompt:

@vivmishra any clues?

VS developer command prompt should be able to locate .NET Framework SDK tools, so this is not a documentation problem.
I was able to make this work on my box with VS 16.4.1.
So, the next step would be to report this via VS Feedback tool (using Send Feedback -> Report a Problem from Help menu), if you have a consistent repro and repairing VS is unable to fix it. I would even recommend upgradng to 16.4 (as 16.3 is out of support) and see if it resolves this issue for you.

Problem still exists in 2023.


Bottom toolbar expands past screen display.  (ArcMap)

Populating the list of coordinates will eventually expand the display so the buttons on the bottom are no longer visible.

FYI @dfoll

saw the same thing in Pro #346

As of the 1509 build this does not occur in either the Pro or ArcMap version of the add-in

A limit has been put on the list but the window will still scroll past the bottom of the screen if the outputs are expanded:


Creating a Player object with accents raises json.JSONDecodeError

Example:
import rs3clans
rs3clans.Player('náme')

Player names can't have accents anyways so this should not be allowed

Now raises ConnectionError with a 403, still should be changed to not even allow a request to be made to RS3's API
>>> import rs3clans
>>> rs3clans.Player('náme')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<string>", line 16, in __init__
  File "/home/john/.local/lib/python3.7/site-packages/rs3clans/players.py", line 72, in __post_init__
    self.info = self._dict_info()
  File "/home/john/.local/lib/python3.7/site-packages/rs3clans/players.py", line 161, in _dict_info
    str_info = self._raw_info()
  File "/home/john/.local/lib/python3.7/site-packages/rs3clans/players.py", line 134, in _raw_info
    raise ConnectionError(f'Not able to connect to RS3 playerDetails API. Status code: {response.status_code}')
ConnectionError: Not able to connect to RS3 playerDetails API. Status code: 403

>>> import rs3clans
>>> rs3clans.Player('náme')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<string>", line 16, in __init__
  File "/home/john/projects/atlantisbot/.venv/lib/python3.7/site-packages/rs3clans/players.py", line 72, in __post_init__
    self.info = self._dict_info()
  File "/home/john/projects/atlantisbot/.venv/lib/python3.7/site-packages/rs3clans/players.py", line 170, in _dict_info
    info_dict = json.loads(info_list)
  File "/usr/lib/python3.7/json/__init__.py", line 348, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.7/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.7/json/decoder.py", line 353, in raw_decode
    obj, end = self.scan_once(s, idx)
json.decoder.JSONDecodeError: Invalid \escape: line 1 column 28 (char 27)


getBaseBranch js config

CI has been enabled on this repo. Please update your PR against master / push a commit to trigger CI. Thanks!


Fix up home assistant stuff

Maybe move some stuff out to nur

It's OK now- ideally document, clean up and have a nice dashboard


Is there any example? I want to implement this package in my own application and can't figure out where to start?  ❓

I am using XML-RPC ( implementation group: 'org.apache.xmlrpc', name: 'xmlrpc-client', version: '3.1.3' ) and want to try out other packages to find best one, one of the reasons for switch is: sending attachments using that library takes too much time. I am fairy new to Ktor.

Hi,
assuming you're able to get your hands on a ktor client you should be able to just create an instance of a MethodCall with the parameters you need and then use the xmlRpc extension function like you would do a get or post request with ktor, only that you pass the MethodCall as the second argument
val client: HttpClient = createHttpClient()
val call = MethodCall("some-method", IntegerValue(42), StringValue("some-string"))
val response = client.xmlRpc("https://some-server/rpc", call) as MethodResponseSuccess // or check for MethodResponseFault if you set the parameter `throwOnFault = false`
val firstParam = response.params.firstOrNull() as? StringValue ?: error("expected first parameter of response to be a StringValue")

Okay, I will try it out slowly with first step being: trying to replace my authentication system, will see if it works, it starts with
      Global.Common = object : XmlRpcClient() {init {
            setConfig(object : XmlRpcClientConfigImpl() {init {
                serverURL = URL(java.lang.String.format("%s/xmlrpc/2/common", Global.url))
                isEnabledForExceptions = true
                connectionTimeout = 5000
                replyTimeout = 5000

            }
            })
        }
        }

I read through these files and will see if it works, also is there any reason why I should use StringValue... (so Value) instead of directly something like: listOf(...) ?

I am getting this error:

java.lang.NoSuchMethodError: No virtual method writeBytes([B)V in class Ljava/io/ByteArrayOutputStream; or its super classes (declaration of 'java.io.ByteArrayOutputStream' appears in /apex/com.android.art/javalib/core-oj.jar)

I guess writeBytes is not part of the android api for java.io.ByteArrayOutputStream

I just published version 1.0.1 which is using a different method on ByteArrayOutputStream to the same result.

I read through these files and will see if it works, also is there any reason why I should use StringValue... (so Value) instead of directly something like: listOf(...) ?

I've opted for compile-time safety, as far as I know if you pass a unhandled type to Apache's xmlrpc you will get an exception at Runtime while here you will get a error at compile-time, but you have to pay with wrapping all values in the respective library type.

It worked, thanks! tomorrow will check it with attachments and if works, I will be writing some helper functions for myself cuz current format is hard to figure out, I use more of either Hashmap or Arrayof(hashmap)

I just published version 1.0.1 which is using a different method on ByteArrayOutputStream to the same result.

I read through these files and will see if it works, also is there any reason why I should use StringValue... (so Value) instead of directly something like: listOf(...) ?

I've opted for compile-time safety, as far as I know if you pass a unhandled type to Apache's xmlrpc you will get an exception at Runtime while here you will get a error at compile-time, but you have to pay with wrapping all values in the respective library type.

I am not gonna lie, this won't be a problem if you do 2 things: 1. check how it handles null from server *in xml-rpc it throws unless your server has specific flag: allow nulls
2. if you make some easy way to wrap it, for example, helper function that turns everything into desired format.

Update: I can't figure out how to have:
ArrayValue(StructValue)
So in kotlin it would be
listOf(hasMapOf(VALUE to KEY))
I've got struct working, but I am getting this error:

Here are fields on server and my result is either binary or base64, based of which object I am attaching, if image: base64 else binary


Expand Callable Protocols in type tooltips of variables explicitly typed using a Callable Protocol class's name

See: https://mypy.readthedocs.io/en/latest/protocols.html#callback-protocols
Example:
from typing import Callable, TypeVar
from typing_extensions import Protocol

T = TypeVar('T')

class Copy(Protocol):
    def __call__(self, __origin: T) -> T: ...

copy_a: Callable[[T], T]
copy_b: Copy

copy_a = copy_b  # OK
copy_b = copy_a  # Also OK

Callable based typings get expanded to function types in tooltips, while Protocol#__call__ based typings continue using the name of the class in tooltips. It would be useful to have a flag for the behavior with maybe a hybrid option that shows both the type "alias" and the underlying function type.

This enhancement request has been open for a while. Closing old issue because of lack of upvotes.


Export YellowBox API

Motivation
Allow end users to access the YellowBox API so that warnings can be properly ignored via the API first introduced in a974c140db605ecbdf8d3faa7a079b7e2dcebb09. Based on that API, you should be able to do the following:
import { YellowBox } from 'react-native';
YellowBox.ignoreWarnings(['Warning: ...']);

However, if you actually try this today, it results in a broken import error.
Test Plan
Verified using an expo instance. First tried without the YellowBox import, observed a failure to import. Then I added the import statement, and observed no warning on my device.
import React from 'react';
import { StyleSheet, Text, View, YellowBox } from 'react-native';

YellowBox.ignoreWarnings(['hey']);

export default class App extends React.Component {
  render() {
    console.warn('hey!');
    return (
      <View style={styles.container}>
        <Text>Open up App.js to start working on your app!</Text>
        <Text>Changes you make will automatically reload.</Text>
        <Text>Shake your phone to open the developer menu.</Text>
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: '#fff',
    alignItems: 'center',
    justifyContent: 'center',
  },
});

Release Notes
[GENERAL] [ENHANCEMENT] [YellowBox] - Export YellowBox API so warnings can be ignored using a non-deprecated API.

@facebook-github-bot shipit


fix: cleanup JIRA session management

to avoid a potential import-time network call
to remove complexity of JIRA client injection

I broke the tests... the fix is in #151.

@nedbat I have incorporated your changes from #151 here (via git cherry-pick);
we can close that in favor of this.


panic: in crypto/sha256.Sum224: cannot convert slice t0[:] ([]byte) to *[28]byte

I am getting the same error

me too

It looks like it's the reasons for not adapting to go 1.18, when i set it back to go 1.17.8 it worked perfectly

Thank for the feedback. I have tried with both, first actually with 1.17 and then 1.18 and it gives the same panic.

Same error here, with go 1.18.1 linux/arm64.

I found this, probably this error is unrelated to go-callvis.
https://github.com/golang/go/issues/51822

same issue, tired it with minikube
$ go-callvis ./cmd/minikube
panic: in crypto/sha512.Sum384: cannot convert slice t0[:] ([]byte) to *[48]byte

goroutine 15576 [running]:
golang.org/x/tools/go/ssa.emitConv(0x14038d55680, {0x102848cb8, 0x1406f5d8780}, {0x102844788?, 0x1400f66e330})
    /Users/medya/go/pkg/mod/golang.org/x/tools@v0.0.0-20200305224536-de023d59a5d1/go/ssa/emit.go:242 +0x6b8
golang.org/x/tools/go/ssa.(*builder).expr0(0x1027b5ae0?, 0x14038d55680, {0x102845680?, 0x1400f2cd0c0}, {0x7, {0x102844788, 0x1400f66e330}, {0x0, 0x0}})
    /Users/medya/go/pkg/mod/golang.org/x/tools@v0.0.0-20200305224536-de023d59a5d1/go/ssa/builder.go:573 +0x824
golang.org/x/tools/go/ssa.(*builder).expr(0x10280e8c0?, 0x14038d55680, {0x102845680?, 0x1400f2cd0c0?})
    /Users/medya/go/pkg/mod/golang.org/x/tools@v0.0.0-20200305224536-de023d59a5d1/go/ssa/builder.go:530 +0x13c
golang.org/x/tools/go/ssa.(*builder).assign(0x14038d55680?, 0x14038d55680?, {0x102846968?, 0x1406f5d2d80}, {0x102845680?, 0x1400f2cd0c0?}, 0x0?, 0x1404de875a0)

Possibly fixed by #121 ?

+1

mee  to .  how to solve ?

follow up, any solution?


Update Libs

Thank you so much for your contribution. This doesn't fix any issue so I will close it for now.


Get it working on general Mono and ARM devices

Hi,
Would be good if we have SkiaSharp working on general Mono instead of just on Xamarin iOS/Android. I know that nothing on current binding stop us to have it working on Mono however, build Skia native libs is a process not very well documented and with lots of tricks. If we could have guide on how to build it (at least the SkiaSharp fork) to ARMv6/v7 and other platforms, would be perfect.
As SkiaSharp home page said, we have lots of problems with Cairo that just can't be fixed today so would be perfect to remove it with Skia and SkiaSharp.
Good work!
Thanks!

@galvesribeiro I am working on that... You can try the WindowsDesktop sample on Mac. It should run and everything. Linux is next after Windows RT.

@galvesribeiro I have updated the build script in SkiaShap and you should be able to build the externals-ios or externals-android targets. This will produce a native iOS or Android library that can be included in any native app. The SkiaSharp repo does all the wo as I don't actually build on skia side.
https://github.com/mono/SkiaSharp/blob/master/build.cake#L295-L338
// helper function just to run xcode builds
var buildArch = new Action<string, string> ((sdk, arch) => {
    XCodeBuild (new XCodeBuildSettings {
        Project = "native-builds/libskia_ios/libskia_ios.xcodeproj",
        Target = "libskia_ios",
        Sdk = sdk,
        Arch = arch,
        Configuration = "Release",
    });
    if (!DirectoryExists ("native-builds/lib/ios/" + arch)) {
        CreateDirectory ("native-builds/lib/ios/" + arch);
    }
    CopyDirectory ("native-builds/libskia_ios/build/Release-" + sdk, "native-builds/lib/ios/" + arch);
});

// helper function to "build" skia whatnots
var RunGyp = new Action (() =>
{
    Information ("Running 'sync-and-gyp'...");
    Information ("\tGYP_GENERATORS = " + EnvironmentVariable ("GYP_GENERATORS"));
    Information ("\tGYP_DEFINES = " + EnvironmentVariable ("GYP_DEFINES"));

    StartProcess ("python", new ProcessSettings {
        Arguments = SKIA_PATH.CombineWithFilePath("bin/sync-and-gyp").FullPath,
        WorkingDirectory = SKIA_PATH.FullPath,
    });
});

// set up the gyp environment variables
AppendEnvironmentVariable ("PATH", DEPOT_PATH.FullPath);
SetEnvironmentVariable ("GYP_DEFINES", "skia_os='ios' skia_arch_type='arm' armv7=1 arm_neon=0");
SetEnvironmentVariable ("GYP_GENERATORS", "ninja,xcode");

// build skia whatnots
RunGyp ();

// actually un the build for each iOS arch
buildArch ("iphonesimulator", "i386");
buildArch ("iphonesimulator", "x86_64");
buildArch ("iphoneos", "armv7");
buildArch ("iphoneos", "armv7s");
buildArch ("iphoneos", "arm64");

// create the fat framework
CopyDirectory ("native-builds/lib/ios/armv7/libskia_ios.framework/", "native-builds/lib/ios/libskia_ios.framework/");
DeleteFile ("native-builds/lib/ios/libskia_ios.framework/libskia_ios");
RunLipo ("native-builds/lib/ios/", "libskia_ios.framework/libskia_ios", new [] {
    (FilePath) "i386/libskia_ios.framework/libskia_ios",
    (FilePath) "x86_64/libskia_ios.framework/libskia_ios",
    (FilePath) "armv7/libskia_ios.framework/libskia_ios",
    (FilePath) "armv7s/libskia_ios.framework/libskia_ios",
    (FilePath) "arm64/libskia_ios.framework/libskia_ios"
});

// BOOM: done! you should have a fat dynamic framework in "native-builds/lib/ios/"

To build for "general" mono, it is the same process. If you build externals-osx this produces a dylib that is usable everywhere on a mac, whether in vanilla Mono or in Xamarin.Mac.
The next bit for Linux will be the same, except that there is only a vanilla mono... Unless there is a Xamarin.Linux out there :)

I don't understood @mattleibow... How am I suppose to use that to build for armv6? Our device is an embedded ARMv6 device running Linux...

@mattleibow sorry for the dumb questions but I'm not familiar with those build scripts... I wonder how can we use our ARMv6 toolchain to build that for it...

@galvesribeiro I understand. As you are running Linux, the current build script is not going to help, but you can check this out until I manage to get it all going: https://skia.org/user/quick/linux That is what I will look at first, but you probably did already.
I haven't looked at this, but you could set the skia_arch_type='arm'.

@mattleibow any update on that?

Can we support ARMv7 212 Snapdragon?  What need to be done?

I may be wrong, but the UWP already supports ARM? Have you been able to run it at all?

@mattleibow I will give it a try. thx

Hello folks.
Are we able now to build on linux using our own toolchains and those build scripts? I tried to follow https://github.com/mono/SkiaSharp/wiki/Building-on-Linux without success. It looks like all the variables passed are being ignored (even thought I see the build script printing the variables there). I've set the following:
GYP_DEFINES='skia_gpu=0 skia_pdf= skia_os=linux skia_arch_type=arm' SK_SUPPORT_PDF=0 ./bootstrapper.sh -t everything --host=$CROSS

As I'm following the build output, it is building with GPU support :(
It looks like it is ignoring the GYP_DEFINES and also, doesn't take in consideration the --host which should set the prefix for the compiler (in this case my cross-compiler).
I think I'll have to do all that hackerish we did before, again, @mattleibow :(

the same comment from the other issue:
@galvesribeiro The new version no longer uses gyp, but gn instead. You can check out what I am doing here: https://github.com/mono/SkiaSharp/blob/master/cake/BuildExternals.cake#L715
What I really now do is just build Linux as Google specifies as a static archive, and then I create a dynamic library that links out everything except what SkiaSharp uses - ie: just the public C API is preserved.

As Google has dropped support for gyp, and switched over to gn, there is nothing I can really do to get the old build process working. In fact, the later version are removing all traces of the old gyp system.

I know, check my PM on gitter when you find some time.

Just asking this question in multiple places:
Right now I have a x64 build for Linux that was compiled on Ubuntu 14.04. I am thinking of adding a few more to help the average dev use SkiaSharp without having to build their own. What other distros or CPUs would be helpful?

To make life easier for everyone, I moved code around and added a new target to the GN files. Now, the native libSkiaSharp build is done with GN and ninja directly: https://github.com/mono/SkiaSharp/wiki/Building-on-Linux

Closing this issue as the Linux topic is getting a bit duplicated.

for building your own native file, see https://github.com/mono/SkiaSharp/wiki/Building-on-Linux
for the discussion on NuGet distribution, see https://github.com/mono/SkiaSharp/issues/312
for requesting pre-built Linux libraries, see https://github.com/mono/SkiaSharp/issues/453


[Feature]: new logs in workflows run history should force auto scroll to bottom

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
new logs in workflows run history should force auto scroll to bottom
Why should this be worked on?
To improve debugging experience

Hey team! Please add your planning poker estimate with Zenhub @nsarupr @srix


Serverinfo

i stuck at this error

Check your bot on dev portal and allow the intents. This error occurs because you didn't allowed the required  intents to the bot.

already allowed it

Check permissions aswell to the bot. This can happen when bot doesn't have required perms or if you're running the commands on unauthorized perms channel.

you guys cant fix Serverinfo and servericon command cuz its still not working

We implemented all fixes to the current version of IVON and this version seems to be doing fine with the intendent version of the discord.js version

alright, thanks


WipError: WipError 4: {code: -32000, message: Cannot find context with specified id}

From run -d chrome from crash logging. No back trace =(
@jonahwilliams

If we're running the main Dwds.start call through an async guard, where else could this crop up? Theoretically even if we're not catching, the stack trace should go through there somehow?

You mean here?https://github.com/flutter/flutter/blob/9b7b9d795edfdd12c6ad8dd6495bed789c1bac05/packages/flutter_tools/lib/src/build_runner/resident_web_runner.dart#L403
If a Future under that asyncGuard is completed with an error without passing a stack trace, or a an error is added to a Stream whose listeners haven't registered an onError callback that accepts and propagates a stack trace, then neither asyncGuard nor the containing Zone will see the stack trace.
AFAIU :-)

This is certainly coming from package:dwds. It's difficult to track down given the lack of a stack trace. For more context (pun not intended):
When package:dwds issues commands to Chrome, it provides an execution context. This is required as Chrome may have several execution contexts on a given page. For example, if a user includes multiple iframes. The context ensures that we are issuing commands that impact the running Dart application. It's possible that a request is trigger in the middle of a reload and therefore Chrome no longer has the provided context.

I had a similar issue, and it happened right after running the Web version.
Something like this:
flutter run # worked
flutter run --web-renderer=html # worked
flutter run # failed

Running flutter clean fixed the issue, but I wonder if maybe some old remnants of the previous process weren't still around. I wish I simply ran another flutter run after a minute to see if that would have worked

Do you have any suggestions for this?

The reason this error happened to me is: I've called html.window.location.reload(); before calling runApp.


fix can't use sudo in deployer

Q
A

Bug fix?
Yes

New feature?
No

BC breaks?
No

Deprecations?
No

Fixed tickets
#1235

was this not the solution or is there something else

The default is false


Documentation for using the cli

I can't seem to find any documentation regarding how to use the cli.
Maybe I can't find the link because it's too hidden?
I will try it tomorrow anyway but would be nice to have usage examples or some kind of documentation 😋

Feel free to check out documentation I wrote for my docker-image of the CLI: https://github.com/Silanael/ardrive-cli-v1/blob/master/README.md
It should give you a basic understanding on how the CLI works. Just ignore the docker/container -related parts.

@elpiel  we updated the CLI with a brand new and comprehensive readme.
Closing this one.


Compile errors

Helo,
I am running into the errors below:
react-native-boilerplate-master/node_modules/react-native-tab-view/lib/module/TabBarIndicator.js
Attempted import error: 'EasingNode' is not exported from 'react-native-reanimated'.

react-native-boilerplate-master/node_modules/react-native-tab-view/lib/module/Pager.js
Attempted import error: 'EasingNode' is not exported from 'react-native-reanimated'.

react-native-boilerplate-master/src/utils/ignore.js
Attempted import error: 'LogBox' is not exported from 'react-native-web/dist/index'.

react-native-boilerplate-master/node_modules/react-navigation-stack/lib/module/views/Header/Header.js
Attempted import error: 'MaskedViewIOS' is not exported from 'react-native-web/dist/index'.

I ran npm install beforehand. Any help would be appreciated, thanks!

Hi @nadia12344 , thanks for reporting the compile error.
I rannpm install-> expo start. But it seems works my end.
Here is my environment:

MacOS: 10.15.6
expo: 3.28.1
npm: 6.14.4

Do you have the latest expo CLI?
Also please try with yarn to install packages.
Thanks


Small enhancements for Version 1.4

The main purpose of 1.4 is to do the style cleanup and to get the 1.X series ready for maintenance mode. There are a few things I've noticed in the library that could either be cleaned up now, or they could wait for the re-write in version 2.
Consolidate forward/backward events
There are 8 methods to handle moving the calendar forward and backward. Four are simply wrappers/aliases, but the other four are used to go forward/back. These are forwardAction, forwardActionWithContext, backAction, backActionWithContext. I think we can combine these by having an optional context argument, defaulting to this when called from the userland.
A more thorough eventing update could happen in 2.0 but this could at least remove the duplicated code for now.
(Maybe) Remove the default 'month' rendering and enforce use of lengthOfTime
Right now are three rendering modes: days, months, and the default "current month". This current month is the legacy method. I think we can remove it by defaulting to a lenghtOfTime of 1 month with a period of 1. The API would need to return the same data in the 1.X branch (which is unfortunate) but it would remove this unnecessary third condition from the library.
This may be better served to wait for the 2.0 release, which I have a rough outline for that I'll present in another ticket.

I think I have a way to reduce a lot of code in the event functions by consolidating the parts that trigger events.
Also, it appears that events like onMonthChange don't trigger when an interval was specified on the config, even if the interval was a 1-month. I think this is probably correct because a) it's simpler and b) it's tough to know if you want this to trigger on say a 3-month calendar.

I'm way into these changes— consolidating feels good and forcing everything to use intervals is definitely a step in the right direction. The onMonthChange and onYearChange events firing sounds good to me, I would consider the current behavior buggy based on what you described.
I'm a little concerned about the backwards-compatibility break with template data, just because I want it to be clear to users that the data being returned is dependent on their version. It's not necessarily realistic to expect that everyone will go out and upgrade— some folks will need to maintain sites with older versions.
So: I'm curious to hear your thoughts on v2, if you think it's worth rewriting the plugin. Want to post your thoughts on that?

The onMonthChange and onYearChange events firing sounds good to me, I would consider the current behavior buggy based on what you described.

Yea I agree. This would make the events the most consistent imo. It can be added in 1.4.1.

I'm a little concerned about the backwards-compatibility break with template data, just because I want it to be clear to users that the data being returned is dependent on their version.

I think any bc breaking would happen in 2.0 but you're right. This only really has to do with month, eventsNextMonth, eventsThisMonth, eventsLastMonth. There's just some confusion during render about what is passed to the view based on the config. We can't kill these vars in any 1.X update but I think any changes here would be good to make in the v2.

So: I'm curious to hear your thoughts on v2, if you think it's worth rewriting the plugin. Want to post your thoughts on that?

I have an small outline that I can post to get the convo started. I was thinking about it mostly from:

Is there a way to break this out into components so that updating is easier?
Can the code be reduced/simplified any further?

If (2) is no then it's probably not worth it. But I think there are some gains to be had around converting everything to render via lengthOfTime, with the default being a 1-month calendar. This would allow us to cleanup the data passed to the template once and for all.

I think any bc breaking would happen in 2.0

:+1:

I have an small outline that I can post to get the convo started. I was thinking about it mostly from:

Is there a way to break this out into components so that updating is easier?
Can the code be reduced/simplified any further?

Just so we're not retreading here, have you read the original proposal we wrote for v2 back in February in #155?

Yea I think #155 is right on the money. I was thinking of breaking the calendar, config parsing, and rendering portions out but what you've described is better.
Have you started planning anything w/r/t gregorian.js yet? I could get to helping with this pretty soon.