Parallel testing on client pipeline at job level

[Final round of fixes around coverage logic still on its way but feel free to comment on current changes]
This is an effort to reduce time consumed by our client pipeline, parallelizing our tests running them on different agents. With this change we're able to reduce the time from an average of 54min to 38min, writing also the foundations to further parallelization that could keep improving the time all the way down to 25 min on my estimations but which work requires more than this wave of changes.
On a nutshell, this change creates a different template from which the client pipeline runs, which main difference is the creation of an artifact that contains some of the build output to be able to run tests on different agents. After several optimizations, by publishing content generated inside lib/dist directories and restore this content in a different agent (once we have checked out the repo and install dependencies), we're able to run our ci tests without needing to run the build process again.

Some caveats:

Losing some code coverage: given the way coverage works, coverage tests need to be run in the same agent. By running mocha and local e2e tests in the same agent, we're able to keep coverage logic in place altough we will lose coverage done by jest tests.
jests tests need to run the build: for unknown reasons, jests tests are the only ones failing with current approach of consuming lib/dist directories. I even tried getting output.log and tsbuildinfo files but yet there is something generated by the build necessary for these tests to run, so for now we could also run c:build with these tests. It does not matter in terms of time since tests don't consume a lot of time.
repeated logic on our templates: new template shares a lot of logic with build-npm-package.yml since it started as a copy of it. Also it has a lot of logic that does not need to be there since we now know it will only run for client. I don't consider these to be blocking current PR but I would like to attend this in a future PR
More agent creation: more agents are created in our pipelines given this approach so I'll do some testing to make sure there is no unexpected behavior when multiple pipeline runs at the same time.
further parallelization: A lot of these caveats need to be solved to further parallelize these tests. For example, we could split our e2e t9s tests into 2, reducing significantly the time they consume. I believe we could further reduce pipeline duration to 25-28min with further investing.

Make sure you merge this at a time when you'll be around to monitor, and give the on-call engineers a warning.


Bubble Sort

Add a solution program for bubble sort in any programming language

I want to work on this issue

Hi..I would like to work on this issue. I can provide the code in C/C++

I want to work on this issue

I can do this in Python.

@Anoushka-Ghosh @dcod3r specify the language for your solution

I want to work on this issue in C

I would like to do this in cpp

I want to work on this issue in C

@Anoushka-Ghosh assigned the issue for C solution

@deepdhar which folder should be used to upload?

@deepdhar which folder should be used to upload?

Searching and Sorting -> Bubble Sort -> Python -> your solution

Hi! I can add the javascript solution.

Go ahead @eldrin0

Hi! I would like to contribute with a solution in Java.

@deepdhar review my PR #96


Add wavy background in marketing page

Current dependencies on/for this PR:

#28  👈
master

This stack of pull requests is managed by Graphite.

Merge activity

Feb 28, 8:32 PM EST: @gneiru merged this pull request with Graphite.


Existing package check should respect test exemptions

Packages Affected:

poshtools-visualstudio2015
visualstudio-codealignment
visualstudio-linepastefix
pdbproject-vs2015
visualstudio2013-update

This is the message I sent to these folks:

This package should not have been run through testing - apologies. We had an issue with the verifier that has just been resolved. See https://github.com/chocolatey/package-verifier/issues/41

You should not get another failure message from the system for an exempt package.

I also went in an unmarked and remark the exemption checkbox (in two subsequent updates - see the image below)


Franklin onboarding journey tutorial code changes

Code changes for Franklin onboarding journey.
Videos hosted on Dynamic Media
Blocks added:
FAQ
Video Text
Test page: https://main--helix-website--kunwarsaluja.hlx.page/drafts/cashub/tutorial-test

@rofe can you please review and approve this PR? This is for the Franklin onboarding tutorial videos created by CAS Hub.
Tagging @3vil3mpir3, @kunwarsaluja

Closing this PR in favor of https://github.com/adobe/helix-website/pull/144


Adapt inboxer to Gmail?

Hi @jmarino,
sorry I couldn't find other ways to reply your post in Denys' original inboxer repo.
Just want to ask whether you are still interested in working on this project and making some fixes?
I'm still actively using inboxer. In my perspective it could still work for some time if we could make some small fixes.
There's rumors about bundles being ported to Gmail. I guess it may still take a while before we could have maybe a PWA Gmail client with bundling.
Best Bin

Hi @onefork, sorry for the delayed response. I was on travel.
I use Inboxer on a daily basis as my email client. I am interested in keeping this project going and improve it.
Feel free to share with me any fixes/changes you make to Inboxer.

You probably have already tried https://simpl.fyi?
There's even a ticket for bundle support. But the dev progress is a little bit slower than the author's plan.
Does it make sense to port this to inboxer?


Publish updates from master

Publish the latest changes from master

Nothing to add besides the topics the team previously discussed about needing to align on release notes (dates, "names for the releases"[for e.g. here we have a extra header for the major as "version 20.10"] of headers, how issues are grouped, text used in links for issues, etc.). Note: I'm not sure the term tab here applies or is to be revisited later when Allie defines a new term.


Switch to built in http client (introduced in java 11)

It seems make sense to use built in http client cause it will reduce deps and built in machinery better known/used.
what do you think?

It sounds like you might want to check out https://github.com/hiredman/clj-http-lite (and its associated forks), that would be a better place to switch to this.

See also https://github.com/gnarroway/hato

Makes sense, thanks!


Add OTelBin to the OpenTelemetry registry

OTelBin is a SaaS, OSS web-based editor for OpenTelemetry collector configurations with support for:

Visualizing for you the configured OpenTelemetry Collector pipelines as swimlanes
Validating your configuration and highlight errors against a schema and specific distributions
Sharing your OpenTelemetry Collector configurations online (requires login with a GitHub or Google account)

Run npm run test-and-fix, this will solve most issues

Run npm run test-and-fix, this will solve most issues

If any first-time contributor to this repo lands here: run of course npm i before npm run test-and-fix ;-)


Clarify when forwarding takes place

Can someone please clarify in the documentation when external forwarding of an mailbox takes place (using Set-Mailbox -FowardingSmtpAddress)? Does it take place in step 3 or in/after step 4?

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

ID: a8878304-7f4b-15c3-d2b6-8a5583930199
Version Independent ID: 329f3b45-a087-6743-0664-e74dd9e0fc46
Content: Exchange Online Protection (EOP) overview - Office 365
Content Source: microsoft-365/security/office-365-security/exchange-online-protection-overview.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@JonesMikael, those steps are for incoming mail flow. Mailbox forwarding is by definition outgoing mail flow.
So, I don't think there's any real comparison or place to insert mailbox forwarding, since it's going in a different direction.

@chrisda thanks for quick response. I agree with you if the email would be internal. But consider the example where you have a mailbox with e-mail address<EMAIL_ADDRESS>and you enable forwarding of this mailbox to an external email address, for example to a hosted CRM system at<EMAIL_ADDRESS>What I want to understand if that step 4 (Content Filtering) takes place before the email is forwarded or not.
This would make a huge different in the amount of spam that would reach<EMAIL_ADDRESS>

@JonesMikael, your best way forward is to open a support ticket and ask your question. If they give you something that you think would help the documentation, then report back here.

@JonesMikael Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


feat:  Added metric for newHeads.

Added metric for polling.
Related issue(s): #2117

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.15%. Comparing base (aafc052) to head (ffcc19e).
 Consider uploading reports for the commit aa74f33 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2215   +/-   ##
=======================================
  Coverage   75.15%   75.15%
=======================================
  Files          13       13
  Lines         644      644
  Branches      118      118
=======================================
  Hits          484      484
  Misses        115      115
  Partials       45       45

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Query: Update the resolution of string.Replace overload

Resolves #8021
Reference to correct overload: https://github.com/dotnet/coreclr/blob/master/src/mscorlib/src/System/String.Manipulation.cs#L1146
There are 4 overloads for string.Replace now.
string Replace(string oldValue, string newValue, bool ignoreCase, CultureInfo culture)
string Replace(string oldValue, string newValue, StringComparison comparisonType)
String Replace(char oldChar, char newChar)
String Replace(String oldValue, String newValue)

tests?

@moozzyk - There is not way to test this in EF till we start building on netcoreapp2.0. We already have tests for string.Replace. They are not failing because we are still on netcoreapp1.1. With this fix, at the time of migration, tests will not break in EF.

I have also filed #8060 to make MethodInfo selection deterministic in our codebase.


Tibialyzer not working/update comming?

As the topic said tibialyzer is not working properly i assume this is becouse of the updated tibia client
Will you release a update for tibialyzer?
Also i found a site "http://www.tlbialyzer.com/" is that the official tibialyzer page?
Really appreciate your work with tibialyzer btw!
I feel handicapped playing tibia without tibialyzer nowdays

I'm playing with it atm, it calculates the EXP per hour, shows the loot etc, it just doesn't recognise the HP etc.

That website is not the official Tibialyzer page. The only official Tibialyzer page is this Github repository. An update for Tibialyzer will be coming soon.

Tibialyzer dosn't catch up prey bonus loot...

Are you using the latest Tibialyzer version (1.4.3)?

im using the old one 1.3 i have just updated the database and downloaded new memory addresses each update...
isnt that enough?

Where do i download the new verison if that isn't enough?

You can download it here.


skip to examine snapshots when GC

Signed-off-by: qupeng<EMAIL_ADDRESS>What problem does this PR solve?
If there are too many stale snapshots, GC snapshot in raftstore thread could be very slow.

Maybe https://github.com/tikv/tikv/issues/8160 can be closed by this.
What is changed and how it works?
When GC snapshots, skip to examine checksum for them.
Related changes
Check List
Tests

Integration test

No new cases are added, but all cases about snapshot GC cover this change.
Release note

speed up snapshot GC

I'm wondering if the delete io operation would cause raftstore latency spike? can we move it out of raftstore thread

The logic will access peer_storage which is not available out from raftstore threads. However it's still possible because we can access RaftDB directly instead, with some more complex code. I prefer to implement it in another PR after this is merged, if it's still necessary. After this PR is merged, no disk I/O will be involved for snapshot GC.


Traverse children of the node using different visitor

Currently returning false from Visit method of XMLVisitor subclass has the following property:
 If you return 'true' from a Visit method, recursive parsing will continue. If you return
 false, no children of this node or its siblings will be visited.

I want to parse children by another custom XMLVisitor subclass while continuing to parse siblings by the current visitor.
Currently it seems not possible. XMLVisitor is quite unusable in current form, because I have to keep a bunch of state to know in which part of document I am.
Is there a way to change behavior of XMLNode::Accept and not to follow "or its siblings" of the statement?

It was modeled after....python, I think? It's been a while. If there was a reasonable API for that, it's a PR that would make sense.


Initialization of static arrays in storage

The following contract:
// SPDX-License-Identifier: AGPL-3.0-only
pragma solidity ^0.8.14;

contract WARP {
  uint256[100000000] d;
}

fails with error: FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
This only happens when the size of the array is big enough, for smaller size values it works fine.
The issue probably has to do with line https://github.com/NethermindEth/warp/blob/develop/src/passes/storageAllocator.ts#L60 in StorageAllocator pass, since it tries to create a new typeNode for each element of the array

This issue lies on line https://github.com/NethermindEth/warp/blob/develop/src/utils/defaultValueNodes.ts#L147 in VariableDeclarationInitializer pass. When it substitutes the static array for a tuple, it creates an ExpressionList containing an expression for each element of the array, which overflows the Javascript heap.
I think the solution for this could be to increase the maximum size of Node heap. But even with this, with larger arrays it is going to overflow the heap too. So, the solution also should have some throwing error for larger arrays.
@cicr99 could you please assign this task to me?

The issue you are talking about is different from this one and it's being worked on already in #821. That one refers to the initialization of static arrays in memory and this one has to do with storage.
The thing with this contract is that d is a state variable so we handle it first on the StorageAllocator pass. I looked into the bug a bit further and the transpilation breaks at this line:
return new CairoTuple(Array(narrowedLength).fill(elementType));

This is called from the StorageAllocator pass in the line I had mentioned in the previous comment. Maybe the solution for this is to create a different type of node rather than using CairoTuple. I think @rodrigo-pino had an idea about it, so maybe he can comment some more on the matter

The problem is that we treat solidity static arrays as cairo tuples. But static arrays are a subset of tuples, because though they can have n elements, each element must be of a single type T. The solution to this bug would be taking advantage of this characteristic of static arrays, where there is no need to initialize the whole n elements with type T.
Several ways I think of approaching this issue (I haven't investigated them fully):

Optimzing the CairoTuple type to group similar types, if it has 't' different types, then  it has inside a list of size t but to the outside user it still a tuple of n elements with t <= n.
Creating a type which is related (as, an ancestor or a successor) of a CairoTuple and has only type and the size. (It's important for it to be related to avoid compatibility issues further down the line)
Same idea as the previous one, but the types has no relationship between them.

I believe (2) and (3) would result in a much harder task. I would opt for (1) for the moment being since it does not exclude any other possible solution to be applied later, fix the bug right away and I think it's a cool optimization for tuple in generals.
@LuisLara-UH  @cicr99

I would go with option 3, even though it looks like these two concepts are related, they actually wouldn't share any of the implementation of their members. Going with option 1 is something I don't like very much because you'll need to handle the fact that same types might not be grouped together, which could be solved using a range system or something like that, but it sounds to me like a messier solution.
However, I reviewed the code and apparently CairoTuple class is only being used for static arrays, it was just implemented that way for it to be as generic as possible. So another idea could be to replace the implementation of this class to handle only static arrays.

Yes, right now CairoTuple in the code is not being used since I am using CairoStaticArray for static arrays. I thought it could be useful to let the implementation of CairoTuple because we could need that generic implementation later(for tuples maybe?)

Closed by #873


Load dynamic values from system env

The current code does not load dynamic env var using :system
config :sentry,
  dsn: {:system, "SENTRY_DNS"},

This is useful when deploying in production environment using exrm.
I would appreciate this feature asap as I am not receiving errors on my Sentry account or if you have any other suggestion please let me now,
Thank guys

I don't have a huge amount of experience with exrm/distillery, so please correct me if I'm off.  My understanding is that for libraries supporting the {:system, "CONFIG_KEY"} convention, when the release is built, it will store that tuple normally without fetching from System. Upon the release being run, the library will see that pattern, and read from the config at runtime.
I believe Phoenix only supports it for a couple parts of the configuration (PORT and HOST).  Would adding it for the dsn be sufficient, or would it be necessary for all configuration keys?

@mitchellhenke Thank you for your reply! Yes that's the behavior mate. I also request it on ex_statsd I can totally submit a PR you you want, wdyt?

@rafaeljesus cool, sounds good.  I can likely get to it this week.


Flaky test TestTimersOnClosedWorkflow

Detail https://api.travis-ci.org/v3/job/390894147/log.txt
--- FAIL: TestTimerQueueProcessorSuite/TestTimersOnClosedWorkflow (2.05s)
    	assertions.go:247:

	Error Trace:	timerQueueProcessor_test.go:945

	Error:      	Not equal:

	            	expected: 0x3

	            	actual  : 0x4

	Test:       	TestTimerQueueProcessorSuite/TestTimersOnClosedWorkflow

closed since we don't see it anymore


Can't use url when indexing Runway models

Description
I'm getting App\Models\Post::url must return a relationship instance when trying to index Runway models that use url within the fields configuration:
'posts' => [
    'driver' => 'local',
    'searchables' => ['runway:post'],
    'fields' => ['title', 'url'],
],

However, url does work when including it in a Runway loop:
{{ runway:post }}
    {{ url }}
{{ /runway:post }}

Steps to reproduce

Create a Laravel model and connect it to Runway (ensuring it has routing setup, too)
Setup the example search config above
Attempt to update this index using php please search:update posts

Environment
Environment
Application Name: OBFUSCATED
Laravel Version: 11.29.0
PHP Version: 8.3.12
Composer Version: 2.7.7
Environment: local
Debug Mode: ENABLED
URL: post-statamic.test
Maintenance Mode: OFF
Timezone: America/Indiana/Indianapolis
Locale: en
Cache
Config: NOT CACHED
Events: CACHED
Routes: CACHED
Views: CACHED
Drivers
Broadcasting: log
Cache: file
Database: sqlite
Logs: stack / single
Mail: log
Queue: sync
Session: file
Statamic
Addons: 11
Sites: 1
Stache Watcher: Enabled (auto)
Static Caching: Disabled
Version: 5.33.1 PRO
Statamic Addons
rias/statamic-redirect: 3.8.1
statamic-rad-pack/runway: 7.11.0
statamic/seo-pro: 6.3.0
stillat/relationships: 2.2.1
trendyminds/guide: dev-main
trendyminds/link: dev-main
trendyminds/nerf: 1.2.0
trendyminds/orbit-statamic: 2.0.0
trendyminds/statamic-geocoder: 2.0.2
trendyminds/statamic-palette: 1.2.1
trendyminds/tip: 1.0.0

Can you add the full stack trace?

Sorry for the delay in coming back to you, I missed your reply. Would you be able to make a quick demo repo showing the issue?

Actually never mind, figured it out: https://github.com/statamic-rad-pack/runway/pull/628


cant scan alipay qrcode

Clear and concise description of the problem
Hope to support scan alipay qrcode
Suggested solution
Hope to support scan alipay qrcode
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

We temporarily close this due to the lack of enough information.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required


GEOMESA-1625 Shade Curator and Storm QS Updates

Updated Docs for Storm Quickstart
Fixed supported Accumulo versions
Added Accumulo 1.8 profile to geomesa-accumulo-datastore.pom to shade curator to prevent conflicts with Accumulo 1.8 and Storm

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>

Would it be more appropriate to do the shading in whatever the storm client jar is? (e.g. in the tutorial, presumably) Seems like we wouldn't necessarily want to shade it here.

maybe we can just add a note to the documentation about the potential conflict

Managing the curator version in the geomesa-tutorials solves this problem for the storm quickstart.

Could someone flag this 1.3.0 please.


Add Alpine Support

This PR adds support for Alpine Linux distribution that uses BusyBox's timeout command.
There is an additional -t parameter needed for all BusyBox's versions from 1.21.x (Alpine 2.6) till 1.29.x (Alpine 3.9). From 1.30.x (Alpine 3.10) the additional flag is gone...
More info: #14

@kvz should be fine now ;) I was waiting for the CI to complete, it takes quick a bit of time :|

@kvz any chance to get this merged? Or do you have any comments?

Merged, thanks @jbienkowski311!!


Umbraco Forms Support

Added property renderer for umbraco forms that mimics the output in Heartcore.
To use:
services.AddUmbraco(_env, _config)
                .AddBackOffice()
                .AddWebsite()
                .AddComposers()
                .AddHeadless()
                .AddHeadlessForms()
                .Build();

Moved to draft after receiving input that a controller that consumes a forms post should be included.


Fix python micro version to avoid unexpected update

Linked issue(s):
I am suspecting recent (and past) sudden ci failure from broken virtualenv

The virtual environment found in /home/circleci/project/.poetry/virtualenvs/kolena-3aSsmiER-py3.11 seems to be broken.

could be caused by docker image update that somehow creates incompatibility with the cached virtual env (anecdotal report)
What change does this PR introduce and why?
Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

What's going to be our process for keeping these versions up to date?

I don't expect we would have to keep an eye on micro version for our purpose here, unless there are security issues reported. That said, would have to rely on engineering team for manual tracking for now.


UnboundLocalError: local variable 'image' referenced before assignment

Hello,
congratulation for your work!!
I am trying to run the provided dehazing models from the path ConvIR/Dehazing/ITS When I am running the following command  python main.py --mode test --data_dir input/haze4k --test_model checkpoints/haze4k-small.pkl --data Haze4K the following error appears
Namespace(batch_size=4, data='Haze4K', data_dir='input/haze4k', learning_rate=0.0001, mode='test', model_name='ConvIR', model_save_dir='results/ConvIR/Training-Results/', num_epoch=300, num_worker=8, print_freq=100, result_dir='results/ConvIR/images/Haze4K', resume='', save_freq=10, save_image=False, test_model='checkpoints/nhr-small.pkl', valid_freq=10, version='small', weight_decay=0)
Traceback (most recent call last):
  File "main.py", line 94, in <module>
    main(args)
  File "main.py", line 28, in main
    _eval(model, args)
  File "/home/username/ConvIR/Dehazing/ITS/eval.py", line 28, in _eval
    for iter_idx, data in enumerate(dataloader):
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 517, in __next__
    data = self._next_data()
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 557, in _next_data
    data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/username/ConvIR/Dehazing/ITS/data/data_load.py", line 85, in __getitem__
    image = F.to_tensor(image)
UnboundLocalError: local variable 'image' referenced before assignment

the folder structure is like the one proposed with the only difference being that the folders inside haze4k/test renamed from GT and IN to gt and haze respectively due to an error appearance. The image names inside the folder are 1.png and 1_0.89_1.56.png respectivelly.
Any suggestions on that?

If I use the --data haze4k the following error appears:
main.py: error: argument --data: invalid choice: 'haze4k' (choose from 'ITS', 'Haze4K', 'NHR', 'GTA5', 'real_haze')
I also tried to rename the folder with images from haze4k to Haze4K but appears the error reported of first comment

Ok. Now I met some error with file names but after correcting them the following error appears:
Namespace(batch_size=4, data='haze4k', data_dir='input/haze4k', learning_rate=0.0001, mode='test', model_name='ConvIR', model_save_dir='results/ConvIR/Training-Results/', num_epoch=300, num_worker=8, print_freq=100, result_dir='results/ConvIR/images/haze4k', resume='', save_freq=10, save_image=False, test_model='checkpoints/haze4k-small.pkl', valid_freq=10, version='small', weight_decay=0)
Traceback (most recent call last):
  File "main.py", line 94, in <module>
    main(args)
  File "main.py", line 28, in main
    _eval(model, args)
  File "/home/username/ConvIR/Dehazing/ITS/eval.py", line 41, in _eval
    pred = model(input_img)[2]
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/username/ConvIR/Dehazing/ITS/models/ConvIR.py", line 110, in forward
    z2 = self.SCM2(x_2)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/username/ConvIR/Dehazing/ITS/models/ConvIR.py", line 45, in forward
    x = self.main(x)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/container.py", line 217, in forward
    input = module(input)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/username/ConvIR/Dehazing/ITS/models/layers.py", line 26, in forward
    return self.main(x)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/container.py", line 217, in forward
    input = module(input)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 463, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/home/username/anaconda3/envs/envir38/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 459, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Given groups=1, weight of size [16, 3, 3, 3], expected input[1, 4, 768, 1296] to have 3 channels, but got 4 channels instead

Note: the dimensions of imported image are 2565x1516

I found the problem the .png files I used had 4 channels (an additional alpha channel per pixel) when I converted to 3 channel it worked! Thank you for your support

When I changed the command to python main.py --mode test --data_dir input/NHR--test_model checkpoints/nhr-small.pkl --data NHR  the error reported on the first comment appeared. I tried to change --data nhr but it continues to appear


#20 Use Profile Manager

#20 Use Profile Manager
Switch to using the pac4j ProfileManager.
This also includes a rewrite of vertx-pac4j to make it follow the patterns established
in play-pac4j and j2e-pac4j.
Note that there is some outstanding work to incorporate the latest changes made to
play-pac4j, and also to use the full pac4j authorization capability (the current codebase
uses the pac4j-identified authorities but following the vert.x model which is more
restrictive than the pac4j model). I plan to resolve both of these issues in subsequent
pull requests and will raise issues for both.

I'm aware that there is still some tidyup (some literals which really have entries in Pac4jConstants, for example) to do, and also the issues highlighted in the description of the PR above. I plan to raise separate github issues to track them so they will not be forgotten but this is quite a large set of changes, and my previous tests are now passing, so I would like to get this merged sooner rather than later.

You'll see I've added issues 22, 23, 24 and 25 to cover the outstanding work. If there's anything you see in this PR that fits into one of those issues, please mark it up in the relevant issue (all lists of work to be done help). If you see something you think should be changed, but that shouldn't prevent this being merged because it's non-fatal, please raise a new issue if it doesn't fit into the above.

Wow! Really great! It's closer than ever to other implementations and so much easier to understand.
I wrote several comments here and there.
I also realized that I'm making your life hard with my latest changes which hasn't been merged in the J2E and Play implementations, though it won't be too complicated to report them. I'm doing as fast as I can.
A very big thank.

Thanks for the feedback, many of the comments are covered under the bigger heading of issues I raised yesterday, and I propose to add your comments to those issues so I can tick them off as I progress them.
Where you have questions rather than comments I’ll answer them in place if that’s ok.
Probably I’ll get round to looking at the detail of your comments/questions on Sunday, I like the fact that half the code disappeared but the tests still passed (until I broke them last night!) and definitely feel it’s moving in the right direction, though there’s a lot to do still.
But at least the heart of the implementation is in pretty much the right place now :) Now to fix the rest of the body :)
Cheers,
Jez

On 18 Sep 2015, at 10:58, LELEU Jérôme<EMAIL_ADDRESS>wrote:
Wow! Really great! It's closer than ever to other implementations and so much easier to understand.
I wrote several comments here and there.
I also realized that I'm making your life hard with my latest changes which hasn't been merged in the J2E and Play implementations, though it won't be too complicated to report them. I'm doing as fast as I can.
A very big thank.
—
Reply to this email directly or view it on GitHub https://github.com/pac4j/vertx-pac4j/pull/21#issuecomment-141408406.


Adding time zone to date/time attribute form

Wouldn't this always output the English name?
What about using the getTimezoneDisplayName method?

Good point. Better ?

Not in front of a PC right now, but it looks ok!


Assignment 2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Creating Detect problems and patient summary functions
What did you learn from the changes you have made?
dealing with arrays in for loops, and utilizing functions inside other functions
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
not really
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
mix ups with true and false in detect zeros and detect problems
How were these changes tested?
running detect problems for each of the paths until one with a zero flagged it true
A reference to a related issue in your repository (if applicable)
Checklist

[X] I can confirm that my changes are working as intended

In order to fix the problem of assignment_1.ipynb changes in assignment_2 branch, follow the steps listed in thread https://uoft-dsi-certificates.slack.com/archives/C07VC7M4PTL/p1733888327768759

Thank you @anjali-deshpande-hub I have implemented your changes I will also be sure to create my assignment 2 branch off of master to prevent assignment 1 changes from showing up there.


Add support for Apple M1 chip

Added and if statement in FindTBB.cmake for a different location of the version header.
Updated submodules as backward-cpp had an error pertaining to aarch64

Hanks for the help! I tested on my Linux box and didn't break building on it


JSON serialization of doubles

NLog version: 4.5.6
Platform:  .Net 4.5
I just ran into an issue/inconsistency using JSON formatted logs. It turns out that NaN, PositiveInfinity, and NegativeInfinity are not valid symbols for a double according to the JSON spec,
however, they serialize as symbols in the NLog JSON Serializer.
Example output:
	"Segment": { 		"Name": "Processing", 		"Requested": "2018-06-14T23:06:43.9029798Z", 		"Finished": "2018-06-14T23:07:20.2739469Z", 		"Valid": true, 		"EstimatedRemaining": NaN, 		"Elapsed": 36.3709671 	},
There's a lot of discussion about this in various places online. The common workarounds seem to be either converting it to a string ("NaN") or replacing it with null or 0.
The reason this is having an impact is that the deserializer used in our log aggregator doesn't recognize the log message as valid JSON and refuses to parse it.

Created PR #2861

@jkoplo NLog 4.5.9 has been released: https://www.nuget.org/packages/NLog
Once again thank you for reporting this.


pacing: get_packet_send_time() is no longer Option<>

Since recovery.last_packet_scheduled_time always have a
default value (current timestamp), Having it Option<Instant>
type doesn't make sense much. No changes on public API.

@lohith-bellad PTAL


new bots

added the delete_access_key and the revoke_access_key to the bots

3 Bots = 3 PRs

And there are conflicts


Release 2.21.0

Preparing for release  2.21.0

[ ] fmiwrappers not working in installed release windows and maybe linux


Update .travis.yml for faster runs

Use Travis container for faster tests.
Cache Bundler downloads.

Coverage remained the same at 96.41% when pulling b36e0666aafb65cd6b3d205c01c1fb9a78cec023 on jc00ke:patch-1 into 02c3c29925204d96abc517fb15857d080317f99e on lotus:master.

Coverage remained the same at 96.41% when pulling b36e0666aafb65cd6b3d205c01c1fb9a78cec023 on jc00ke:patch-1 into 02c3c29925204d96abc517fb15857d080317f99e on lotus:master.

@jc00ke Thank you, Sir. :+1:


Scenario Tests of IS-5.7.0 with Oracle are disabled

This PR  https://github.com/wso2/testgrid-job-configs/pull/417 disables the scenario tests of IS 5.7.0 with Oracle-SE2-12.1 due to some infrastructure issues. This should be reverted once those issues are resolved.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Can't create a component that acts as a checkbox.

See http://esnextb.in/?gist=f0b9b22bbe837861533683dbfdd03ec4 for a minimal reproduction.
This appears to be because of the checkbox specific modelValueUpdater behavior which is not triggered except when controlProps.type is "checkbox".
As you can see from the esnextb.in simply specifying the type="checkbox" on the component is sufficient to workaround this issue.
I'm not sure what a good solution to this is, perhaps some mechanism to extend the modelValueUpdaterMap when doing createFieldClass

This will be addressed more elegantly in v1.0; adding type="checkbox" is a sensible (and expected) workaround currently.
It has to do with how props are currently mapped for custom components - right now, there is no way of knowing the mapped props until after sequenceEvents, and in V1 the mapping will be made available before sequencing.

Available in latest @beta!
<Control.checkbox component={...} />

Thanks David, I scoured through all of the docs and couldn't find a way to hook up a custom toggle / checkbox with RRF. This should be included in the docs because there's some weird behavior.
<Control component={CustomToggle} />
Clicking the checkbox would always provide a value "on".
The correct way is as you mentioned:
<Control.**checkbox** component={...} />


chore(snapshot): add success log for nexus create snapshot

bors merge

One more, should the trace when we receive the request be info?
bors cancel

One more, should the trace when we receive the request be info? bors cancel

okay. let me make the request as well info.

bors merge


Improved UI of Profile page

Fixes:  #2177
Description
I had improved the UI of profile page so that it matched with the theme of website.
I had also made it responsive for the small screens
#2177
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
Before:

After:

Checklist:
[ X] I have made this change from my own.
[X ] My code follows the style guidelines of this project.
[ X] I have performed a self-review of my own code.
[ X] My changes generate no new warnings.
[X ] I have tested the changes thoroughly before submitting this pull request.
[ X] I have provided relevant issue numbers and screenshots after making the changes.

@huamanraj , @anuragverma108 Sir, please review this PR.
Thank You!

@huamanraj , @anuragverma108 Sir, please review this PR.
Thank You!

please resolve the conflicts and create another PR!


[script-compiler] Can morpheme-match dictionaries be bundled with worker?

If .textlintrc has a morpheme-match "dictionaryPathList" set, it will continue to show relative paths even when built with script-compiler.
{
  "rules": {
    "@textlint-ja/morpheme-match": {
      "dictionaryPathList": ["./dictionary.js"]
    }
  }
}

This means that need to have an external dictionary file even after compiling textlint as a worker.
Can dictionary files also be bundled within the worker?

No, not at the moment.
because, fs module is not work in browser.
As related, textlint-rule-prh is supported.
@textlint/config-inliner inline dictionary file to textlint worker script.

No, not at the moment.
because, fs module is not work in browser.
As related, textlint-rule-prh is supported.
@textlint/config-inliner inline dictionary file to textlint worker script.


Config supports MySQL replication credentials

Fixes https://github.com/github/orchestrator/issues/323
This PR introduces three new configuration variables:

MySQLReplicationUser
MySQLReplicationPassword
MySQLReplicationCredentialsConfigFile (alternative to the above two)

These variables will be used by orchestrator to CHANGE MASTER TO ... MASTER_USER=... MASTER_PASSWORD=..., on a server that didn't have credentials (a Master), in these operations:

Setting up co-masters
Graceful master takeover

You will not need these variables if you use master_info_repository = TABLE in your MySQL servers.

@shlomi-noach this would not allow per-cluster replication credentials, which the slave_master_info table would do, although it does allow you at least to work with orchestrator when the credentials are not available in the mysql schema :1st_place_medal:
Are you planning to add support for a helper query too, which would allow per-cluster credentials?

Whoops.

Not going to use this PR. Replaced by #603

I noticed this has been open for a while - is there still a plan to integrate this feature? I'm trying to minimize changes in a MySQL env where the master.info is still being written to a file and having the ability to tell Orchestrator what the UN/PW are would be a nice to have. Not a need to have (given #603) but among those options, I think putting the creds in the Orchestrator conf would be the easiest solve.

I noticed this has been open for a while - is there still a plan to integrate this feature? I'm trying to minimize changes in a MySQL env where the master.info is still being written to a file and having the ability to tell Orchestrator what the UN/PW are would be a nice to have. Not a need to have (given #603) but among those options, I think putting the creds in the Orchestrator conf would be the easiest solve.

See https://github.com/openark/orchestrator/pull/603

Hmm that's clever, I didn't think about a static query but yep that would work! The idea was to avoid, at least for now, having to switch to storing the master info in the TABLE. That's probably the correct long term solution but has security implications (as would the query since it would get run on the DB itself). This feature request would be a little easier to secure and audit with fewer DB changes which made it compelling. The second option is indeed either #603 or switching to TABLE for the binlog info and making secure we have a good security/audit process for access to the mysql DB itself.

The idea was to avoid, at least for now, having to switch to storing the master info in the TABLE. That's probably the correct long term solution but has security implications (as would the query since it would get run on the DB itself).

You could always store the data in a table with a varbinary field with aes_encrypt, possibly reading with a view that would allow you to run with the definer's privilege to  protect direct access to the data

hello.
i setting up orchestrator at mariadb...they haven't slave_master_info table.
i write into conf file:
"MySQLReplicationUser": "replicant",
"MySQLReplicationPassword": "superpa$$",

but orchestrator not load it:
./orchestrator -c dump-config | jq | grep MySQL
  "MySQLTopologyUser": "orchestrator",
  "MySQLTopologyPassword": "---------------",
  "MySQLTopologyCredentialsConfigFile": "",
  "MySQLTopologySSLPrivateKeyFile": "",
  "MySQLTopologySSLCertFile": "",
  "MySQLTopologySSLCAFile": "",
  "MySQLTopologySSLSkipVerify": true,
  "MySQLTopologyUseMutualTLS": false,
  "MySQLTopologyUseMixedTLS": true,
  "MySQLOrchestratorHost": "------------",
  "MySQLOrchestratorMaxPoolConnections": 128,
  "MySQLOrchestratorPort": 3306,
  "MySQLOrchestratorDatabase": "orchestrator",
  "MySQLOrchestratorUser": "orchestrator",
  "MySQLOrchestratorPassword": "----------------",
  "MySQLOrchestratorCredentialsConfigFile": "",
  "MySQLOrchestratorSSLPrivateKeyFile": "",
  "MySQLOrchestratorSSLCertFile": "",
  "MySQLOrchestratorSSLCAFile": "",
  "MySQLOrchestratorSSLSkipVerify": true,
  "MySQLOrchestratorUseMutualTLS": false,
  "MySQLOrchestratorReadTimeoutSeconds": 30,
  "MySQLOrchestratorRejectReadOnly": false,
  "MySQLConnectTimeoutSeconds": 1,
  "MySQLDiscoveryReadTimeoutSeconds": 10,
  "MySQLTopologyReadTimeoutSeconds": 600,
  "MySQLConnectionLifetimeSeconds": 0,
  "MySQLHostnameResolveMethod": "@@hostname",
  "ApplyMySQLPromotionAfterMasterFailover": true,

./orchestrator --version
3.2.6
c846d43668239cad384dc31b9255a3ade3a35001

what i do wrong ? or it not impelentet for now ?


youtube-dl: 2021.03.14 -> 2021.03.25

https://github.com/ytdl-org/youtube-dl/releases/tag/2021.03.25

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117515 at 2226668e run on aarch64-linux 1

1 package skipped due to time constraints:

plex-mpv-shim

31 packages built successfully:

botamusique
catt
celluloid
curseradio
haskellPackages.cut-the-crap
hydrus
jellyfin-mpv-shim
jftui
kodi-cli
lollypop
minitube
mkchromecast
mopidy-youtube
mpc-qt
mps-youtube
persepolis
python38Packages.mpv
python38Packages.pafy
python38Packages.tubeup
youtube-dl (python38Packages.youtube-dl)
youtube-dl-light (python38Packages.youtube-dl-light)
python39Packages.mpv
python39Packages.pafy
python39Packages.tubeup
python39Packages.youtube-dl
python39Packages.youtube-dl-light
qimgv
somafm-cli
sublime-music
tartube
ytcc

1 suggestion:

warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/tools/misc/youtube-dl/default.nix:57:0:
   |
57 |     description = "Command-line tool to download videos from YouTube.com and other sites";
   | ^

20.09: cf439ccb91574cb2d870b1036f638dd13179b7a9


Update Taxon model to handle conservation status(es) and establishment means

Per Ben, here's an example query that returns establishment means:
https://api.inaturalist.org/v1/taxa?q=colt's-foot&preferred_place_id=6853


api.url_for() fails with endpoints specified by a string

I use the a common Resource class for a number of routes and therefore specify an endpoint manually. Unfortunately it appears that the api.url_for() does not accept a string endpoint, only a Resource. It is therefore not possible to generate a url for manually specified endpoints.
Could you modify the method to accept either a Resource object or a string endpoint name?
Apologies if this is already fixed, I've not seen anything in the logs.

flask_restful.url_for() basically just calls flask.url_for() with resource.endpoint. Can you try using the url_for() from Flask instead?

Hello, thanks for the reply, I've tested the flask url_for() asking for a url for the following endpoint:
api.add_resource(DiscreteCollection, PREDEFINED_ROUTE + "/equilibrium/", resource_class_args=(["timeslice"], ), endpoint="equilibrium")

which results in an exception. The error below is from a  test request context, the result is the same for a real request.
Traceback (most recent call last):
  File "/home/ameak/code/dashboard/graphing-service/runserver.py", line 1, in <module>
    from spiro import app
  File "/home/ameak/code/dashboard/graphing-service/spiro/__init__.py", line 1, in <module>
    from .app import app
  File "/home/ameak/code/dashboard/graphing-service/spiro/app.py", line 98, in <module>
    print(url_for("equilibrium"))
  File "/usr/local/depot/python335_modules/site-packages/Flask-0.10.1-py3.3.egg/flask/helpers.py", line 312, in url_for
    return appctx.app.handle_url_build_error(error, endpoint, values)
  File "/usr/local/depot/python335_modules/site-packages/Flask-0.10.1-py3.3.egg/flask/app.py", line 1641, in handle_url_build_error
    reraise(exc_type, exc_value, tb)
  File "/usr/local/depot/python335_modules/site-packages/Flask-0.10.1-py3.3.egg/flask/_compat.py", line 33, in reraise
    raise value
  File "/usr/local/depot/python335_modules/site-packages/Flask-0.10.1-py3.3.egg/flask/helpers.py", line 305, in url_for
    force_external=external)
  File "/usr/local/depot/python335_modules/site-packages/Werkzeug-0.10.4-py3.3.egg/werkzeug/routing.py", line 1678, in build
    raise BuildError(endpoint, values, method)
werkzeug.routing.BuildError: ('equilibrium', {}, None)

The DiscreteCollection class is used in multiple places in this api so I can't simply reference the object.

Looking back through the issues list for BuildError, i think #266 might have some useful info. That person fixed the issue by making sure the endpoint string started with ".".
My other guess is that your endpoint route takes parameters, but you don't provide any when calling url_for()

Thanks joshfriend, I just tried adding a "." before the endpoint name. It now appears to works correctly. This does seem to be very unintuitive though - an  endpoint named "blah" requires ".blah" to reference it!
Do you think it would break anything if the flask-restful api.url_for() automatically added the "." to string endpoints?

It looks to me like Flask doesn't do anything with auto-adding ".", so I don't think Flask-RESTful should either.


Roots of Quadratic equation in C

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
Quadratic equations are commonly used in mathematics.
Added A program that prints roots and also tells us the types of the roots of a quadratic equation ax²+bx+c=0.
Checklist

[✓ ] I've read the contribution guidelines.
[✓ ] I've checked the issue list before deciding what to submit.
[✓ ] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Roots of Quadratic equation in C [math] #6768

@HarshCasper @ankitaggarwal23 @gat786 @anushkrishnav @Apoorve73  can anyone check and merge this PR.

Updated the File according to the requested changes, @ankitaggarwal23Sir, please review it.


OAuth with Outlook fails as soon as you give permission

Description:
I'm currently trying use the outlook mail service to obtain contacts/appointments and mails. However I've stumbled upon a problem.
I created a URL scheme like described in their README

Then I created a Constants file which looks like this
struct Consumer {
    let consumerKey: String
    let consumerSecret: String
    let authorizeURL: String
    let accessTokenURL: String
    let responseType: String?
    let requestTokenURL: String?
}

let Outlook = Consumer(
    consumerKey: "",
    consumerSecret: "",
    authorizeURL: "https://login.microsoftonline.com/common/oauth2/v2.0/authorize",
    accessTokenURL: "https://login.microsoftonline.com/common/oauth2/v2.0/token",
    responseType: "code",
    requestTokenURL: nil)

I created an application for outlook on https://apps.dev.microsoft.com
generated my key and secret and filled them in inside my application.
I added the mobile application platform to my app. Which tells me to use the redirect URI urn:ietf:wg:oauth:2.0:oob
So my code to authorise looks like this
@IBAction func btn_Outlook(_ sender: Any) {
        let oauthOU = OAuth2Swift(
            consumerKey: Outlook.consumerKey,
            consumerSecret: Outlook.consumerSecret,
            authorizeUrl: Outlook.authorizeURL,
            accessTokenUrl: Outlook.accessTokenURL,
            responseType: Outlook.responseType!)

        oauthOU.authorizeURLHandler = SafariURLHandler(viewController: self, oauthSwift: oauthOU)
        oauthOU.authorize(
            withCallbackURL: "urn:ietf:wg:oauth:2.0:oob",
            scope: "https://outlook.office.com/Mail.ReadWrite https://outlook.office.com/Mail.Send https://outlook.office.com/Calendars.ReadWrite https://outlook.office.com/Contacts.ReadWrite https://outlook.office.com/Tasks.ReadWrite",
            state: state,
            success: { credential, response, parameters in
                print("logged in with \(credential), with response \(response) and parameters \(parameters)")},
            failure: { error in
                print("error occured \(error.localizedDescription)")
            }
            )
        }

when I run the code I first get a screen to enter my mail/password. When I enter my mail it will redirect me to a different page/portal where I can enter my password. When I've entered my password it will show me the permissions screen.

as soon as I hit yes it will give me an error saying "Safari cannot open the page because the address is invalid." I'm pretty sure this has to do with the redirect URI but I'm not sure what to do to actually fix this.
Could you possibly tell me where I might be going wrong with this?
EDIT:
I added urn:ietf:wg:oauth:2.0:oob to my url scheme as well but this didn't change anything.
I've also added
func application(_ app: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey : Any] = [:]) -> Bool {
  if (url.host == "oauth-callback") {
    OAuthSwift.handle(url: url)
  }
  return true
}

but it doesn't even get here.
OAuth Provider (Outlook):
OAuth Version:

[ ] Version 1
[x] Version 2

OS (Please fill the version) :

[x] iOS :
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[x] CocoaPods
[ ] Manually

Library version:

[ ] head
[x] v1.0.0
[ ] v0.6
[ ] other: (Please fill in the version you are using.)

Xcode version:

[x] 8.0 (Swift 3.0)

[ ] 8.0 (Swift 2.3)

[ ] 7.3.1

[ ] other: (Please fill in the version you are using.)

[ ] objective c

urn:ietf:wg:oauth:2.0:oob is not a valid url scheme and cannot be open by safari
there is no ://
if you create the url scheme like readme say, use it.. oauthTestAll:// in your callbackurl
and if microsoft don't accept it, you must do something else (go to wiki, maybe https://github.com/OAuthSwift/OAuthSwift/wiki/API-with-only-HTTP-scheme-into-callback-URL could help )

microsoft only accepts urn:ietf:wg:oauth:2.0:oob that's a value they give you and you can't change.
I tried to use the http route but the thing is that outlook redirects me to the specified url instantly when it opens up the login page.
I think outlook redirects to the correct login page based on your mail address. When it tries to redirect it will automatically open the redirect url. I'm not sure how to fix this.

maybe you choose native app or mobile app instead of web app, that's why you can change the redirect uri (some other providers as some configuration like that)
please provide a link to the documentation, maybe some one can help

I add one comment on this issue
let oauthOU = OAuth2Swift -> retain issue, the object oauthOU must be set as class var


cachetools KeyError from auth/access_token_client.py

Describe the bug
We occasionally catch KeyError exceptions raised from sp_api/auth/access_token_client.py:50 when calling Reports.create_report.
0: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 97"
1: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 70"
2: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 418"
3: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 113"
4: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 366"
5: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 494"
6: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 79"
7: "/venv/lib/python3.12/site-packages/cachetools/__init__.py: 423"
8: "/venv/lib/python3.12/site-packages/sp_api/auth/access_token_client.py: 50"
9: "/venv/lib/python3.12/site-packages/sp_api/base/client.py: 70"
10: "/venv/lib/python3.12/site-packages/sp_api/base/client.py: 63"
11: "/venv/lib/python3.12/site-packages/sp_api/base/client.py: 97"
12: "/venv/lib/python3.12/site-packages/sp_api/api/reports/reports.py: 113"
13: "/venv/lib/python3.12/site-packages/sp_api/base/helpers.py: 21"

To Reproduce
I'm not sure, yet, what conditions cause the KeyError.
Expected behavior
Handle the KeyError in the library. If there actually is an exceptional condition, raise a more specific exception.
Desktop (please complete the following information):

OS: Linux (cloud)

Additional context

Using python-amazon-sp-api 1.4.0.
Using cachetools 5.3.2.
Using python 3.12.2.
We're calling Reports.create_report from another thread (via asyncio.to_thread).

Is this a long running environment that's sending the requests to the thread?

Is this a long running environment that's sending the requests to the thread?

The process might run for several hours. Never more than a day.

That's the problem. The access token is valid for an hour, and if it gets deleted just when you're requesting, the key doesn't exist. I'll have a look if there's an easy fix, I haven't used long running tasks for this in years.

Thanks. If the condition can be handled transparent, great, but even if there was just a more specific exception raised we could catch and know to just retry, that'd work.

Yeah, going to identify and clarify it at least 😄

I think I'm getting the same error. I upgraded my OS which upgraded Python then this started happening. It's 100% reproducible. It was working everyday for the previous year before I upgraded. I tried removing pycache in case that was related, but it made no difference.
`$ pip list
Package              Version

boto3                1.34.79
botocore             1.34.79
cachetools           5.3.3
certifi              2024.2.2
charset-normalizer   3.3.2
confuse              2.0.1
idna                 3.6
jmespath             1.0.1
pip                  23.2.1
python-amazon-sp-api 1.4.3
python-dateutil      2.9.0.post0
PyYAML               6.0.1
requests             2.31.0
s3transfer           0.10.1
setuptools           67.7.2
six                  1.16.0
urllib3              1.26.18
wheel                0.40.0
`
`  File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/cachetools/init.py", line 68, in getitem
return self.__data[key]
KeyError: 'access_token_57823aada8f070e62b53ef9039dbabf0'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/auth/access_token_client.py", line 46, in get_auth
access_token = cache[cache_key]
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/cachetools/init.py", line 418, in getitem
return cache_getitem(self, key)
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/cachetools/init.py", line 70, in getitem
return self.missing(key)
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/cachetools/init.py", line 97, in missing
raise KeyError(key)
KeyError: 'access_token_57823aada8f070e62b53ef9039dbabf0'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./get_inventory.py", line 25, in
report_request = reports_api.create_report(reportType="GET_FLAT_FILE_OPEN_LISTINGS_DATA", dataStartTime=str(date.today())+"T04:00:00", dataEndTime=str(date.today())+"T04:00:00")
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/base/helpers.py", line 21, in wrapper
return function(*args, **kwargs)
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/api/reports/reports.py", line 113, in create_report
return self._request(kwargs.pop('path'), data=kwargs)
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/base/client.py", line 97, in _request
headers=headers or self.headers,
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/base/client.py", line 63, in headers
'x-amz-access-token': self.restricted_data_token or self.auth.access_token,
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/base/client.py", line 70, in auth
return self._auth.get_auth()
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/auth/access_token_client.py", line 49, in get_auth
access_token = self._request(request_url, self.data, self.headers)
File "/home/gerald/amazon-sp-api/lib/python3.8/site-packages/sp_api/auth/access_token_client.py", line 35, in _request
raise AuthorizationError(error_code, error_message, response.status_code)
sp_api.auth.exceptions.AuthorizationError: ('invalid_client', 'Client authentication failed', 401)
`


[Bug]: Infinite reload, UI is inaccessible

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Describe the bug and reproduction steps
https://github.com/user-attachments/assets/e7559067-2b1c-4720-a435-0986a9b85249
https://github.com/user-attachments/assets/9e64f8bb-0513-45e7-a11e-0393cca97ba1
OpenHands Installation
Docker command in README
OpenHands Version
0.17
Operating System
WSL on Windows
Logs, Errors, Screenshots, and Additional Context
It happens whether the command is
docker run
  -p 3000:3000
  --env LOG_ALL_EVENTS=true
  --env SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:main-nikolaik
  --env WORKSPACE_MOUNT_PATH=C:\repositories\extensions\youtube-time-manager
  --name openhands-app
  --pull always
  --add-host host.docker.internal:host-gateway
  -v C:/repositories/extensions/youtube-time-manager:/opt/workspace_base
  -v /var/run/docker.sock:/var/run/docker.sock
  -v ~/.openhands-state:/.openhands-state
  --rm
  docker.all-hands.dev/all-hands-ai/openhands:0.17

or
docker run
  -p 3000:3000
  --env LOG_ALL_EVENTS=true
  --env SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:main-nikolaik
  --env WORKSPACE_MOUNT_PATH=C:\repositories\extensions\youtube-time-manager
  --name openhands-app
  --pull always
  --add-host host.docker.internal:host-gateway
  -v C:/repositories/extensions/youtube-time-manager:/opt/workspace_base
  -v /var/run/docker.sock:/var/run/docker.sock
  -v ~/.openhands-state:/.openhands-state
  --rm
  docker.all-hands.dev/all-hands-ai/openhands:main

It also happens if I change to
--env SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:0.17-nikolaik

I just tried emptying the cache (right click on the reload button) and now I can enter OpenHands

Yeah probably something to do with the new settings transition...not sure.

It's probably worth writing some code to check whether the settings schema in the local storage matches the old schema and if so, either write an algorithm to transform to the new format or empty the local storage and then call location.reload()

There's been several changes regarding the settings on main. I'll try to test the scenarios for when we release when the settings are transitioned. Hopefully others don't run into this.


lib/os/heap: make printed heap info more useful

Turn sys_heap_dump() into sys_heap_print_info() to better reflect
what it actually does, and improve the information being printed.

On Tue, 16 Mar 2021, Andy Ross wrote:

Maybe switch it to use the log subsystem while you're in here?  Seems like a good fit.

Will have a look, and make a separate PR for that.
Incidentally this PR highlighted the fact that we're wasteful in the
bucker array on 64-bit systems and the overhead can be cut in half
there. That's another PR to come.


Export results of Ecuador: El Piedrero fixed on OSM

Is already fixed on OSM the province of Guayas [ISO-2 EC-G]
adding El Piedrero that belongs to Guayas.
Can you please generate the geoJSON of Guayas to your export files repo
Thanks 🙂

Thanks for noting this. I've made the required changes to the repo which allows fixing this and I started a full reprocessing on my server. It should be ready in about an hour.

your work is amazing thanks


as aks update command does not exist

While attempting to run az aks update -g <RGName> -n <AKSName> --enable-managed-identity, the following error is returned:
az aks: 'update' is not in the 'az aks' command group. See 'az aks --help'.

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

ID: 6a9c74a8-9f1a-3d3d-3aed-f8ec45f92e05
Version Independent ID: 01e98043-dc80-490e-a761-8d558ff4e6df
Content: Use managed identities in Azure Kubernetes Service - Azure Kubernetes Service
Content Source: articles/aks/use-managed-identity.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@vanpyrzericj Thank you for your feedback . We will investigate and update the thread.

@vanpyrzericj Apologies for the delay in response and all the inconvenience caused because of the issue.
The az-aks-update command is part of the Azure CLI AKS Preview Extension.
You need to Install this extension using the below CLI command to make it work. You can refer to this for more info
az extension add --name aks-preview
Please refer to this for more details: https://github.com/Azure/azure-cli-extensions/tree/master/src/aks-preview
Hope it helps!!!

@vanpyrzericj
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


[Snyk] Upgrade usb from 2.4.3 to 2.5.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade usb from 2.4.3 to 2.5.2.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 4 versions ahead of your current version.
The recommended version was released a month ago, on 2022-09-24.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

2.5.2 is broken, wait for the next release.

We're already on 2.11.0.


Add support for CSS in JS using the emotion library

This PR adds functionality to create CSS in JS using the emotion JS library.
It also generalizes a bit the to_string function
It also provides some functionality to define and load in Pluto (or outside) JS libraries from NPM (using esm.sh)

Closing as not going to be implemented here anymore


fixed search_by_name function

Resolves issue:
https://github.com/turbonomic/vmt-connect/issues/1

Resubmit to the dev branch please


font size choices

I've got used to the font size on the usual MSDN docs website.  The docs.ms.com font size (and perhaps line spacing) is different enough to notice.  Could you consider providing a smaller font choice?

⚠ Idea migrated from UserVoice
Created By: Caleb D
Created On: 2017/05/01 15:30:13 +0000
Votes at Migration: 1
Supporters at Migration: 1

Thanks for the feedback. We're designing for a combination of different screen sizes, resolutions, and devices as well as an optimal easy-to-read line length. Right now, we're at a pretty standard font size when compared to other documentation and long-format sites. However, the designers are considering some updates to the site and will keep this feedback in mind.

@ShevaDas assigning to you for follow-up once the changes are live.

@dend We're not currently planning to make any changes but this will be one of the things we consider as we go through an overall review of site UX over the next few months. I'll keep you updated if we do wind up changing it.

Closing this item as we've made some updates to our site typography.


Is it possible to assign the "£" unit to the financial numbers? Also Current Savings discussion.

Is it possible to assign the "£" unit to the financial numbers?  I saw that the items have the £ in the name eg "£ Earned Last Month" but I was wondering if there was an issue assigning the unit of measurement in the integration.
Failing that, I think that the "Current Savings" item needs to be labelled with a £ for consistency eg "£ Current Savings"
"Current Savings" is a rate but is it savings/hour?
All other financials are described as "Earnings" while this one is "Savings".  Would "£ Current Hourly Earn Rate" be more accurate?

The newest release 2024.8.1 now has the units of savings as 'GBP'
The current savings isn't savings/hour as far as I'm aware it's just the estimated savings for the last 30 minutes.


[#567] check ruby 3.1

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a090ab4) 34.92% compared to head (61919cd) 34.92%.
 Consider uploading reports for the commit bb1bf75 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   34.92%   34.92%
=======================================
  Files          47       47
  Lines        1300     1300
=======================================
  Hits          454      454
  Misses        846      846

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Foldable: add toArray and toReadOnlyArray instances

👋
I saw an open PR, "Added Option.toArray" (https://github.com/gcanti/fp-ts/pull/1155), and noticed @gcanti recommended instead adding toArray to Foldable.
I went ahead and added it. However, I'm not 100% sure about the implementation.
In 2.x+, Array is deprecated in favor of ReadOnlyArray, but the original implementation of toArray uses a combination of reduce and arr.push.
Possible problem: .push does not exist on ReadOnlyArray since it mutates the array in place. This method is probably faster, but it might not desirable in the 2.x+ world.
Would it make sense to add another instance, toReadOnlyArray?
Update: I went ahead and also added a toReadOnlyArray instance. There appeared to be some prior art in the Map, Set, and Record modules.
I'm open to feedback and/or edits.

[x] Fork the repository and create your branch from master.
[x] Run npm install in the repository root.
[x] If you've fixed a bug or added code that should be tested, add tests!
[x] Ensure the test suite passes (npm test).

@newswim thanks for this PR, I would only keep the readonly array version (which is future proof<EMAIL_ADDRESS>will drop all mutable modules https://github.com/gcanti/fp-ts/issues/1226)
/**
 * Transforms a `Foldable` into a read-only array.
 *
 * @example
 * import { toArray } from 'fp-ts/lib/Foldable'
 * import { tree, make } from 'fp-ts/lib/Tree'
 *
 * const t = make(1, [make(2, []), make(3, []), make(4, [])])
 * assert.deepStrictEqual(toArray(tree)(t), [1, 2, 3, 4])
 *
 * @since 2.8.0
 */
export function toArray<F extends URIS4>(F: Foldable4<F>): <S, R, E, A>(fa: Kind4<F, S, R, E, A>) => ReadonlyArray
export function toArray<F extends URIS3>(F: Foldable3<F>): <R, E, A>(fa: Kind3<F, R, E, A>) => ReadonlyArray
export function toArray<F extends URIS3, E>(F: Foldable3C<F, E>): <R, A>(fa: Kind3<F, R, E, A>) => ReadonlyArray
export function toArray<F extends URIS2>(F: Foldable2<F>): <E, A>(fa: Kind2<F, E, A>) => ReadonlyArray
export function toArray<F extends URIS2, E>(F: Foldable2C<F, E>): (fa: Kind2<F, E, A>) => ReadonlyArray
export function toArray<F extends URIS>(F: Foldable1<F>): (fa: Kind<F, A>) => ReadonlyArray
export function toArray<F>(F: Foldable<F>): (fa: HKT<F, A>) => ReadonlyArray
export function toArray<F>(F: Foldable<F>): (fa: HKT<F, A>) => ReadonlyArray {
  return (fa: HKT<F, A>) =>
    // tslint:disable-next-line: readonly-array
    F.reduce(fa, [], (acc: Array, a) => {
      acc.push(a)
      return acc
    })
}

p.s.
docs are auto-generated by running npm run docs

Ah, thank you!
I believe I made the appropriate updates, let me know!


Pose Landmarks x and y: Landmark coordinates normalized to [0.0, 1.0] out of range

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
Yes
OS Platform and Distribution
Ubunut 20.04
Programming Language and version
Python 3.8.10
MediaPipe version
0.10.1
Solution
Pose landmark detection
Describe the actual behavior
I'm getting x,y values of the landmark out of the [0, 1] range described here
Describe the expected behaviour
To get x,y values of the landmark in the range [0, 1]
Standalone code/steps you may have used to try to get what you need
#!/usr/bin/env python3

import numpy as np
import cv2
import mediapipe as mp
import time

class HumanPoseDetection:
    def __init__(self):
        # TODO: change the path
        model_path = "/home/user/models/pose_landmarker_full.task"
        BaseOptions = mp.tasks.BaseOptions
        self.PoseLandmarker = mp.tasks.vision.PoseLandmarker
        PoseLandmarkerOptions = mp.tasks.vision.PoseLandmarkerOptions
        self.result = mp.tasks.vision.PoseLandmarkerResult
        VisionRunningMode = mp.tasks.vision.RunningMode

        self.options = PoseLandmarkerOptions(
            base_options=BaseOptions(model_asset_path=model_path),
            running_mode=VisionRunningMode.LIVE_STREAM,
            result_callback=self.callback
            )

    def callback(self, result, output_image, timestamp_ms):
        if(result.pose_landmarks):
            self.result = result.pose_landmarks[0]
            for idx, elem in enumerate(self.result):
                if(0 <= elem.x <= 1 and 0 <= elem.y <= 1):
                    pass
                else:
                    print("Warning out of range values: {}".format(elem))

    def detect_pose(self):
        cap = cv2.VideoCapture(0)
        with self.PoseLandmarker.create_from_options(self.options) as landmarker:
            while cap.isOpened():
                _, image = cap.read()
                image = cv2.cvtColor(image, cv2.COLOR_BGR2RGB)
                image = cv2.resize(image, (224, 224))
                mp_image = mp.Image(image_format=mp.ImageFormat.SRGB, data=image)
                frame_timestamp_ms = int(time.time() * 1000)
                landmarker.detect_async(mp_image, frame_timestamp_ms)

if __name__=="__main__":
    HPD_ = HumanPoseDetection()
    HPD_.detect_pose()

@bhomaidan1990,
Pose solution which you are using are legacy solution now, We have already upgraded the pose landmark API as guided here And ended support for the legacy pose solution.
We recommend you to try out new upgraded solution(Pose Python Example), Which will provide a superset of capabilities available in the legacy solutions.
Also, Please let us know if you encounter the similar error in upgraded Pose solution. Thank you!

@kuaashish  I'm using the new upgraded solution and not the legacy one ! But there is no documentation provided for the landmark values, so I assumed the old documentation!

@kuaashish  I have updated the Solution to Pose landmark detection instead of the legacy pose, still the Landmark coordinates are not normalized to the range [0.0, 1.0]

+1

Can you provide an example value that was returned from the graph?

@bhomaidan1990 Could you give this utility a try in your example and let me know if it fixes the issue?
https://github.com/google/mediapipe/blob/37b68714b8c1e9f2ec22ff91c0ef7bb4c10227b7/mediapipe/python/solutions/drawing_utils.py#L49
I believe this was used with the legacy Pose Landmarker solution too.

Oh and I believe there was already a TODO in the utility here (for out-of-bound image coordinates)
It's possible to apply a fix/workaround here.
https://github.com/google/mediapipe/blob/37b68714b8c1e9f2ec22ff91c0ef7bb4c10227b7/mediapipe/python/solutions/drawing_utils.py#L61

I'd suggest clamping or wrapping the values. Use whatever works best for your usecase.
Example of clamping:
def clamp(value):
    return max(0.0, min(1.0, value))

@schmidt-sebastian

Can you provide an example value that was returned from the graph?

NormalizedLandmark(x=0.3442286252975464, y=1.663205623626709, z=0.6675763130187988, visibility=0.0005908278399147093, presence=2.2982736481935717e-05)
NormalizedLandmark(x=0.525449275970459, y=1.726546287536621, z=0.16685475409030914, visibility=0.0006492086104117334, presence=1.1043830454582348e-05)
NormalizedLandmark(x=0.351003497838974, y=1.7091314792633057, z=0.6834827661514282, visibility=0.0016230528708547354, presence=1.9313172742840834e-05)
NormalizedLandmark(x=0.42790234088897705, y=1.763134241104126, z=-0.2120414823293686, visibility=0.0004914221935905516, presence=1.980218439712189e-05)
NormalizedLandmark(x=0.3111097812652588, y=1.7536125183105469, z=0.3353509306907654, visibility=0.0009863366140052676, presence=3.027161983482074e-05)
NormalizedLandmark(x=0.5848674774169922, y=1.0461585521697998, z=-0.22029514610767365, visibility=0.2053297609090805, presence=0.026988795027136803)
NormalizedLandmark(x=0.4060169458389282, y=1.0328614711761475, z=0.22312480211257935, visibility=0.2098243534564972, presence=0.026073340326547623)

@kinaryml

clamping or wrapping the values

This suggestion while it works, it yields inacurate results as all invisible landmarks will be at the image border, which might distort the pose and cause some issues for further pose processing, e.g pose forcasting.

@schmidt-sebastian

Can you provide an example value that was returned from the graph?

NormalizedLandmark(x=0.3442286252975464, y=1.663205623626709, z=0.6675763130187988, visibility=0.0005908278399147093, presence=2.2982736481935717e-05)
NormalizedLandmark(x=0.525449275970459, y=1.726546287536621, z=0.16685475409030914, visibility=0.0006492086104117334, presence=1.1043830454582348e-05)
NormalizedLandmark(x=0.351003497838974, y=1.7091314792633057, z=0.6834827661514282, visibility=0.0016230528708547354, presence=1.9313172742840834e-05)
NormalizedLandmark(x=0.42790234088897705, y=1.763134241104126, z=-0.2120414823293686, visibility=0.0004914221935905516, presence=1.980218439712189e-05)
NormalizedLandmark(x=0.3111097812652588, y=1.7536125183105469, z=0.3353509306907654, visibility=0.0009863366140052676, presence=3.027161983482074e-05)
NormalizedLandmark(x=0.5848674774169922, y=1.0461585521697998, z=-0.22029514610767365, visibility=0.2053297609090805, presence=0.026988795027136803)
NormalizedLandmark(x=0.4060169458389282, y=1.0328614711761475, z=0.22312480211257935, visibility=0.2098243534564972, presence=0.026073340326547623)

@kinaryml

clamping or wrapping the values

This suggestion while it works, it yields inacurate results as all invisible landmarks will be at the image border, which might distort the pose and cause some issues for further pose processing, e.g pose forcasting.

Yep that's true and it's only fixable in the C++ layer.

Any update ? thanks :)

I have the same problem - there are coordinates that go beyond [0:1] and because of this, some skeletons are drawn incorrectly

I have found that coordinates go out of range when the keypoint is offscreen. If you don't want that, use min_pose_detection_confidence in your PoseLandmarkerOptions.
More detail in my answer on the related StackOverflow.


Increase promote did retries

Doh! I think I must have accidentally deleted the wait in between retries somehow and not noticed.
Hopefully this should prevent the errors during promote did.

Test fail is super interesting — a CredDef that is attempting to be published fails because it already exists on the ledger.  I checked the ledger and N1LoA9rMwyskHEW9csYpQJ:3:CL:360946:Faber.agent.Schema_DriversLicense_v2 was created on Feb. 16 — http://test.bcovrin.vonx.io:3707/tx/BCOVRIN_TEST/domain/360950.
How does that happen?  That means that the DID being used in the test was the same as being used on Feb. 16.
🤯

Test fail is super interesting — a CredDef that is attempting to be published fails because it already exists on the ledger. I checked the ledger and N1LoA9rMwyskHEW9csYpQJ:3:CL:360946:Faber.agent.Schema_DriversLicense_v2 was created on Feb. 16 — http://test.bcovrin.vonx.io:3707/tx/BCOVRIN_TEST/domain/360950.
How does that happen? That means that the DID being used in the test was the same as being used on Feb. 16.
🤯

That's the other fail I've seen. Not as often.
Feb 16. That doesn't make any sense???

The only thing I can think of is that the same seed for the DID was generated on both test runs.  Is it possible we’re not being random enough when coming up with the seed?

I just had another fail like this 69HTqHL7PjCS8K4Bp44g46:3:CL:293648:Faber.agent.Schema_DriversLicense_v2 from Jan 29. Seems like the tests must be recreating the same dids :/


chore(main): release 1.12.0

:robot: I have created a release beep boop
1.12.0 (2023-02-12)
Features

Add note to export (fc6793f)
Add Notes input to RegistrationInfoView (92d03e6)

Bug Fixes

crash when accessing RegistrationInfoView for attendee without tshirt size (eed311f)

Miscellaneous Chores

check: Disable checks that don't support merge_group event (abd3c68)
deps: bump barryvdh/laravel-ide-helper from 2.12.3 to 2.13.0 (a1f4a46)
deps: bump brianium/paratest from 6.8.1 to 6.9.0 (7609865)
deps: bump laravel-vite-plugin from 0.7.3 to 0.7.4 (f39373d)
deps: bump laravel/dusk from 7.5.0 to 7.6.0 (5bab9fb)
deps: bump laravel/sail from 1.19.0 to 1.20.1 (396ff45)
deps: bump prettier from 2.8.3 to 2.8.4 (c51ab23)
deps: bump tailwindcss from 3.2.4 to 3.2.6 (05d68bf)
stabilize test (0178ca2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rotex1800/registration/releases/tag/v1.12.0 :sunflower:


Kestrel topic updates

Fixes #12101
Fixes #12113

Error confirmed: Store is required or WindowsCryptographicException: 'The parameter is incorrect' is thrown.
Similar to what we just did for config, call out the namespace for the section that pertains to Kestrel options.
... and that Logging namespace isn't needed ... probably a leftover from an earlier sample update that had some logging in there.

@Rick-Anderson This is a quick one to :eye:.


[SPARK-27698] Add new method for getting pushed down filters in Parquet file reader

What changes were proposed in this pull request?
To return accurate pushed filters in Parquet file scan(https://github.com/apache/spark/pull/24327#pullrequestreview-234775673), we can process the original data source filters in the following way:

For "And" operators, split the conjunctive predicates and try converting each of them. After that
1.1 if partially predicate pushed down is allowed, return convertible results;
1.2 otherwise, return the whole predicate if convertible, or empty result if not convertible.

For other operators, they are not able to be partially pushed down.
2.1 if the entire predicate is convertible, return itself
2.2 otherwise, return an empty result.

This PR also contains code refactoring. Currently ParquetFilters. createFilter  accepts parameter schema: MessageType and create field mapping for every input filter. We can make it a class member and avoid creating the nameToParquetField mapping for every input filter.
How was this patch tested?
Unit test

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10640/
Test PASSed.

Test build #105376 has started for PR 24597 at commit 75fe737.

@dongjoon-hyun @wangyum @rdblue @cloud-fan

mark this as WIP before https://github.com/apache/spark/pull/24598 is merged.

Test build #105376 has finished for PR 24597 at commit 75fe737.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105376/
Test PASSed.

Test build #105527 has started for PR 24597 at commit fa8f48c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10785/
Test PASSed.

Test build #105529 has started for PR 24597 at commit b22ea80.

Test build #105527 has finished for PR 24597 at commit fa8f48c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105527/
Test PASSed.

Test build #105529 has finished for PR 24597 at commit b22ea80.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105529/
Test PASSed.

This is ready for review @dongjoon-hyun @wangyum @rdblue @cloud-fan

Thank you for pinging me, @gengliangwang . Shall we wait for one day? Currently, after SPARK-27699, HiveOrcFilterSuite failure is reported in Hadoop 3.2 profile. The fix is under testing and will be merged tomorrow. @wangyum and @HyukjinKwon is working on that.

retest this please.

@dongjoon-hyun @cloud-fan Please review this, so that we can continue the migration of Parquet V2.

Test build #105613 has started for PR 24597 at commit b22ea80.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10865/
Test PASSed.

Test build #105613 has finished for PR 24597 at commit b22ea80.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105613/
Test PASSed.

thanks, merging to master!

Looks fine but can you clarify the relation between convertibleFilters and createFilters?


chore: bump models for release 0.17.0

Issue #

Description of changes

New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/test


Validation rule architecture compliance

Closes #166

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.703%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1554

Relevant Lines:
1902

💛  - Coveralls

Looks like I'm too slow

Sorry! 😬


Merge of development with TLS 1.3 MVP into the TLS 1.3 prototype

The TLS 1.3 MVP needs to be merged (in the git sense) into the TLS 1.3 prototype.
It could be done incrementally but that's not a must.
It will probably be necessary to prepare the TLS 1.3 prototype before the merge by applying the TLS 1.3 specific coding rules defined in tls13-experimental.md to the prototype (at least one commit per rule).
After the merge, all test_tls13* all.sh components must still run successfully.
Justification
We want to keep the TLS 1.3 support in Mbed TLS and the TLS 1.3 prototype to converge.

Completed, see https://github.com/hannestschofenig/mbedtls/pull/392.


Setup the comments model urls for the comments endpoint of the plugin Backend Pending Review #277

The url is 127.0.0:8000/comments
To retrieve url is 127.0.0:8000/comments/id e.g 127.0.0:8000/comments/1


AllowJs and duplicate identifier

Potential bug, potential feature, depending on how you look at it :)
In a javascript file, using allowJs, shouldn't all valid javascript be valid to the typescript compiler? I found several of the other duplicate identifier issues that have been brought up. I don't see this being exactly the same thing though.
TypeScript Version:  2.2.3
Code
This is technically valid JS code. I wouldn't ever do it myself, but it happens in large codebases.
var amount = amount();
function amount() {
  return 123;
}

It results in a duplicate identifier error on the amount symbol.
I can definitely see this being the type of thing that typescript should prevent, but being that it's a .js file I'm kind of on the fence.

It is a pattern that is not recommended. the main reason is the type changes in ways that are rather intuitive.
you can always use // @ts-ignore comments to avoid reporting the error.

@mhegazy oh I totally agree with you re not recommended. I didn't realize I could use that comment though so thanks for bringing that up.
My point was really just that on the process of incrementally introducing TS to an existing codebase, it was kind of a surprise because I was using allowJs. It is nice though that TS lets us know that it's something we really should avoid

given that the change is not simple; I would like to get more user input on this.


Fix logic check error for update GPU groupDetails

Description
This PR fix logic check error for update GPU groupDetails.
It should update GPU groupDetails when there are non-empty infos.

Types of changes

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

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

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

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

@blueorangutan package

@sureshanaparti a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3473

@blueorangutan test


Auto build based on python

The former bash script was getting to complicated and hard to read, at least for me ;)
This is a port to python which should give us a little bit more flexibility for the changes coming to bioconda and conda-forge in the next month.
It also adds a test task, and extracts tests from the meta.yaml file.
ping @thriqon

I think this is working pretty well, I have created 500 containers in the last day :)
@thriqon let me know what you think so we can merge it!

It looks nice, and if it works, thats great :) My only issue is that it uses an out-of-container conda, which shouldn't be neccessary... After all, a containerized build is reproducible ;-)

@thriqon good point. But this is only for getting conda-build as lib. I need this to parse the meta.yaml files. Are you fine with merging now and improving on it later on?

yes, go ahead :+1:

Awesome! Thanks!


[Bug]:  Contribution.md Needs to be re-done

What did you expect to happen?
We are not a part of ACM anymore and there is a lot of documentation that still mentions/links them.
What happened?

^ Example of something that needs to be changed
Version
Beta 0.1.0
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Also change all Athena with UTD Trends


Hide warnings/issues in test code unless test showing, report correct line # and file location for test issues. Fixes #2231

Encountering another example of 'phantom' issues (with line numbers past the end of the source) within the google IO dart pad contest prompted me to fix this bug once and for all.  It is too easy for this to occur and potentially very confusing to the user.  (See my comment on #2231 for more background)
This PR fixes embed.dart so that it does this:

if there are warning/info issues within the test code they are hidden by default, unless the test code is showing. (Errors in test code are always shown)
all issues that occur within the test code are reported with the correct line numbers and as 'Line X of test.dart'
clicking on issues originating from the test code active show the test and active the test tab and place the cursor at the issue.
clicking on issues from the user's dart code active that tab and place the cursor at the issue (as before, but the dart tab is automatically activated).

Note about other details about this PR:
In the Embed() constructor there is an array called tabNames in which there was an entry called 'editor' - when the actual creating of the tabs occurs there was a check for if(name=='editor')... and if so contextName was set to 'dart'.
(This contextName variable is was was used for the actual tab name.)
The 'editor' value was used for making the HTML ID for that tab and then for the ga.sendEvent() call.
The problem with this is that other places in the code 'editor' was being (incorrectly!) used in tab name checks to see if the 'dart' tab was active.  This was incorrect as there was no tab named 'editor', but you can see how this happened because a quick look at that tabNames array made it appoear like 'editor' was a tab name.
I have reversed the logic.  I now use 'dart' in the tabNames array, and then I check to see if the current tab is 'dart' and use 'editor' for the HTML ID and ga.sendEvent()
I also fixed places where 'editor' was being used to use the correct 'dart'.
_getActiveSourceCode() was one of these, and the only reason it worked for copying source was that the default was to use the dart source.

I suspect I confused this PR by landing #2274. Please resolve conflicts.


The Intercom API request failed with the code: 410, after 3 attempts.

Version info

intercom-rails version: 0.2.35
Rails version: <IP_ADDRESS>

Expected behavior
After run an import, all data is exported to Intercom.
Actual behavior
Without any code changes from our side, we started to receive this message: The Intercom API request failed with the code: 410, after 3 attempts.
This message is not describe in Intercom documentation (while a HTTP 410 does it means that the resource requested is no longer available and will not be available again, i.e, it was intentionally removed).
Steps to reproduce

We're using a Worker with this method to import data:

  def self.run_import(options)
    collection = options[:collection]
    intercom = IntercomRails::Import.new(options)
    intercom.collection = collection
    intercom.run_import(collection)
  end

Logs
The Intercom API request failed with the code: 410, after 3 attempts.
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/intercom-rails-0.2.29/lib/intercom-rails/import.rb:142:in `perform_request'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/intercom-rails-0.2.29/lib/intercom-rails/import.rb:148:in `perform_request'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/intercom-rails-0.2.29/lib/intercom-rails/import.rb:148:in `perform_request'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/intercom-rails-0.2.29/lib/intercom-rails/import.rb:148:in `perform_request'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/intercom-rails-0.2.29/lib/intercom-rails/import.rb:132:in `send_users'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/intercom-rails-0.2.29/lib/intercom-rails/import.rb:64:in `block in run'
/var/my_project/releases/2016-06-13_13_25_52/config/initializers/intercom_extension.rb:49:in `block in batches'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/batches.rb:126:in `find_in_batches'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/activerecord-deprecated_finders-1.0.4/lib/active_record/deprecated_finders/relation.rb:70:in `find_in_batches'
/var/my_project/releases/2016-06-13_13_25_52/config/initializers/intercom_extension.rb:47:in `batches'
/var/my_project/releases/2016-06-13_13_25_52/vendor/bundle/ruby/2.1.0/gems/intercom-rails-0.2.29/lib/intercom-rails/import.rb:63:in `run'
/var/my_project/releases/2016-06-13_13_25_52/config/initializers/intercom_extension.rb:22:in `run_import'
/var/my_project/releases/2016-06-13_13_25_52/config/initializers/intercom_extension.rb:17:in `run_import'
/var/my_project/releases/2016-06-13_13_25_52/app/workers/intercom_worker.rb:8:in `perform'

Duplicate of https://github.com/intercom/intercom-rails/issues/202
Hi @raelgc
This endpoint is deprecated and the importer will be removed from this gem in the next version. Please use the (much more effective) bulk endpoint: https://developers.intercom.io/reference#bulk-user-operations
This is wrapped in intercom-ruby if you need it.
Thanks,
Bob

Thank you for the quick answer, @bobjflong!