TP = (labels[mask] == 1).sum().float() RuntimeError: CUDA error: device-side assert triggered

Hi, thanks for your attention and problem.
I think this is because of the mismatch between labels and mask. Maybe you can check the shape of them.
Best,
Sunan.


ci: Initial pass at linting YAML files

Full yamllint rules reference: https://yamllint.readthedocs.io/en/stable/rules.html#
Closes https://github.com/meltano/hub/issues/950

@pnadolny13 this is a first pass at getting linting setup for YAML files.
It adds nice annotations: https://github.com/meltano/hub/pull/979/files#annotation_5518730485.
I'm not sure if we want to apply fixes for all files in one big PR, but I can look into that. Otherwise this approach is incremental and should only check files touched by the PR.
Wdyt?

@pnadolny13 this is a first pass at getting linting setup for YAML files.
It adds nice annotations: https://github.com/meltano/hub/pull/979/files#annotation_5518730485.
I'm not sure if we want to apply fixes for all files in one big PR, but I can look into that. Otherwise this approach is incremental and should only check files touched by the PR.
Wdyt?

@edgarrmondragon this is great! Maybe having a blurb in the CONTRIBUTING.md around how to test locally before pushing changes would be helpful also.
Yeah I like the idea of one giant PR to update all of them. I didnt see a fix type command like sqlfluff has. Is there a way to have it update the file where it can? I guess we'd have to write some scripts for that if not.

@edgarrmondragon this is great! Maybe having a blurb in the CONTRIBUTING.md around how to test locally before pushing changes would be helpful also.

Sounds good. I've added pre-commit, which can be run locally.

Yeah I like the idea of one giant PR to update all of them. I didnt see a fix type command like sqlfluff has. Is there a way to have it update the file where it can? I guess we'd have to write some scripts for that if not.

I gave opt-nc/yamlfixer a try, but it doesn't fix key ordering and isn't great at detecting multi-line strings that overflow the line length limit 😕.

@cjohnhanson thanks for the linter update!


fix(DIST-1127): Fix text colors on bright backgrounds

Ticket DIST-1127
Calculate brightness of the button color and render the proper text / icon color

:tada: This PR is included in version @typeform/embed-v1.17.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DTR:TRG 1.04 editable static files

PR is created:
https://github.com/eclipse-tractusx/sldt-digital-twin-registry/pull/347


changed lineWidth of indicator series

:+1: Really changing the game with this one

I think this change represents a real negative change in the codebase. I'm actually insulted you would dare make this change without consulting me first.

jk

Here's to the crazy ones, the misfits, the rebels, the troublemakers, the lineWidth:1.25 series in a lineWidth:2.0 world... the ones who see things differently -- they're not fond of rules... You can quote them, disagree with them, glorify or vilify them, but the only thing you can't do is ignore them because they change things... they push the human race forward, and while some may see them as the crazy ones, we see genius, because the ones who are crazy enough to think that they can change the world, are the ones who do.


Fix width of mosaic multiforms

There is some white space problem introduced when the width of columns gets reduced.

Low prio, as mosaic is not the default vis for categorical attributes anymore.

Mosaic implementation not used anymore.


Load new tilesource and completely remove the old one

var viewer = OpenSeadragon({
id: 'viewerDiv1',
tileSources: tileSources1
});
While opening another tilesource in the same div, openseadragon throws error.
viewer = OpenSeadragon({
id: 'viewerDiv1',
tileSources: tileSources2
});
Hash viewerDiv1 has already been used.The old one and  new are messed up.
Anyway to completely clear the old tilesource and to load the new tilesource.

Try using the open function instead of creating a new viewer instance with the same id:
viewer.open(tileSource)
This should load the new source and discard the old one.

Thanks...jorntx ... It's working fine

Can you this with jpeg? because nothing is happening on my end. i loaded the image using the code
tileSources: {
type: 'image',
url: '\images\a.jpg'
}
but when i tried to change the image
viewer.close()
viewer.open('\images\b.jpg')
nothing happens and the current image is just static, you cant move it or anuthing

You're correctly specifying the full tile source (which includes both the type and url) at first, but you need to do the same with open, like so:
viewer.open({
  type: 'image',
  url: '\images\b.jpg'
});

@iangilman  when I use viewer. open annotations are no longer working can u help me, please?

@ayamany224 it doesn't sound like your question has anything to do with this issue... can you please file a new one? Also, can you please provide more detail? What is "open annotations"? And were you using it with OpenSeadragon? What does it mean when you say they are no longer working?

Calling viewer.open() always zooms to the extents of the viewport; Is there a way to update the tile source in-place, without affecting the current pan/zoom? This is especially janky if we're viewing pages in collectionMode--I may be zoomed to a particular page and not want to lose that location when a tileSource is updated.

@yowzadave If you use addTiledImage instead, it doesn't trigger the pan/zoom.


Clarification on Message Submission Format

Hello Microsoft,
Good day to all of you.
This inquiry is about this document : https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/user-submission?view=o365-worldwide
I would like to clarify some details regarding the submission format you described in the document. You have the following writeup :
- The message is being reported as phishing.
- The Network Message ID is 49871234-6dc6-43e8-abcd-08d797f20abe.
- The Sender IP is <IP_ADDRESS>.
- The From address is<EMAIL_ADDRESS>- The message's subject line is "test phishing submission"
3|49871234-6dc6-43e8-abcd-08d797f20abe|<IP_ADDRESS>|test@contoso.com|(test phishing submission)

Network Message ID - there are two message-ID-related email-headers and these are : (1) X-MS-Exchange-Organization-Network-Message-Id and (2) X-MS-Exchange-CrossTenant-Network-Message-Id. Both have same values, which is best to use?
Sender IP - Is the value of 'X-Originating-IP' email-header best to use for this? Or is the IP in the topmost 'Received' header best to use?
Email Address - Do you also support "FirstName LastName<EMAIL_ADDRESS>format?
Subject - How do we escape '|' (vertical bar/pipe) and other special characters in the email subject?
Body of the Email - What happens the if email body is non-empty?
Attachment - As for attachments, do you support MSG file format only? Or are EML-files supported too? Can we use customized/use-preferred names for the attached report emails too?

Thank you very much for taking time to read my inquiries. And thank you in advance for your answers.
More power to all of you.

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

ID: e53205b5-8850-42b6-0b0c-8a61b7ef7776
Version Independent ID: 0a9e7903-ce3c-e877-e6b3-b45d09642308
Content: User submissions policy - Office 365
Content Source: microsoft-365/security/office-365-security/user-submission.md
Product: m365-security
Technology: mdo
GitHub Login: @siosulli
Microsoft Alias: siosulli

@glr0221 Thank you for the feedback.
Network Message ID - Use the  X-MS-Exchange-Organization-Network-Message-Id
Sender IP - It is 'X-Originating-IP' email-header
Email Address - Supports only  the from address (You cannot provide First name and last name) instead it can be<EMAIL_ADDRESS>if that the email that the message is sent from.
Subject - The email subject is always under ( ) So the pipes or characters do not matter.
If your question is about the subject of the message submission format - It still needs to be SafetyAPIAction|NetworkMessageId|SenderIp|FromAddress|(Message Subject)
Body of the Email - Ideally it doesn't matter (unless you are reporting false positive to<EMAIL_ADDRESS>Attachment - Use either .msg (default Outlook format) or .eml (default Outlook on the Web format) formats for the attached messages.
Hope this helps!
Thanks
Sri

Thank you very much @msbemba sri. You answers are well-appreciated.
To clarify:

What if the subject is something like this :  "We have ) and ( signs in the subject."
There is no need to escape the sample subject string stated above right? Even if I have parenthesis in the subject.

The EML part is awesome to know. Now I have some peace of mind as to the integrity and quality of data I am sending.

As a follow up question, as you know, the MSG file formats has many properties/meta-data/attributes in its structure.
Would you know or any of the developers know what attributes/properties are mandatory for the MSG file format?
Currently we are using our own MSG writer, and when submitting a report we are sending out an MSG with "significant" property values only. These property-values include PR_SENDER_EMAIL_ADDRESS_W, PR_SENDER_NAME_W, PR_MESSAGE_CLASS_W and others.
Is there a list of MSG-properties that need to be supported to ensure that the reported msg file has a complete set of data required for reports to be processed/generated correctly?

Sender IP - It is 'X-Originating-IP' email-header
There are some cases where this particular header is absent. (i.e. gmail-sent emails) So i am currently looking into "Authentication-Results: spf=none (sender IP is <IP_ADDRESS>)" as an augmentation/secondary solution to X-Originating-IP.

I hope the three items I raised are clear and made sense.
Thank you always for your support, you guys rock!

@msbemba Thank you very much for the contribution and sharing this explanation. @glr0221 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Undefined offset: 1 (getSecondaryChallengeCodeFromPage)

Hello,
When I'm trying to load a website with the code from the repo, the error "Undefined offset: 1" is returned on:
public static function getSecondaryChallengeCodeFromPage(string $page, string $code): string
    {
        preg_match('/k = \'([^\']+)\';/', $code, $kMatches);
        preg_match( '/id="' . $kMatches[1] . '">([^<]+)</', $page, $kElemMatches); <- happen here

        return $kElemMatches[1];
    }

Website: [LINK REMOVED]
PHP version: 7.2
Curl version: 7.52.1
OpenSSL version: OpenSSL 1.1.1b 26 Feb 2019

Regards,

Seems it not support https yet

Hi. Could you give me your verbose log output for the site you are trying to access?
This is the first release in a while so there will be issues. For me, I tested this library with HTTPS and it seems to work with HTTP2 for me. But everyone's scenario is different, so your logs will give me a better idea of what's going on.
It seems this undefined offset is related to TLS not being available from where ever you're running the script. Be aware Cloudflare supports P-256 and P-384 only

UPDATE: My bad. Looks like this website is using the old version of Cloudflare which does not reference calculation code from a div element. I'll factor this into the challenge solving code.

Alright, thanks for working on it!

Updated master with a fix. Pull down the latest commit and let me know if it fixes your issues guys :)

Hello,
I cloned the master branch, now this error is missing but another one come :
"Class 'SimpleJavaScriptCompilation\Model\Context' not found"
in:
```

public static function getJschlAnswerFromPage(UAMPageAttributes $pageAttributes): float
{
$codeSnippets = UAMPageChallengeCode::getSnippetsFromPage($pageAttributes->getPage());
    $ctx = new Context();

    $ctx->setCtxFunc('e', 'SimpleJavaScriptCompilation\Model\FunctionMap\GlobalFunctionMap::atob');
    $ctx->setCtxFunc('g', 'SimpleJavaScriptCompilation\Model\FunctionMap\GlobalFunctionMap::stringFromCharCode');

I'm probably missing something, so I though that installing it with composer would install the dependancy, but there is another error:

[InvalidArgumentException]
Could not find package kyranrana/cloudflare-bypass at any version matching your PHP version <IP_ADDRESS>

(doesn't work even with --ignore-platform-reqs)

Changed minimum supported PHP version to 7.2
Give it a go with the latest commit from master?

Same issue, and if I use the following command:
composer require --ignore-platform-reqs kyranrana/cloudflare-bypass:master
I get the following error:
Do not run Composer as root/super user! See https://getcomposer.org/root for details
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package kyranrana/cloudflare-bypass master is satisfiable by kyranrana/cloudflare-bypass[dev-master] but these conflict with your requirements or minimum-stability.

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

What does your composer.json look like?

Thanks! Now everything seems to be ok! :)

Awesome!

Hi, it's still working with you guys ? because I tried to install the code but always have a problems


Port written in golang

First off I just want to say thanks as I have been using seashells almost every day over the past month both at work and at home and it is absolutely awesome. Also I and ended up writing a golang port (hans-strudle/seashells if anyone is interested. Thanks!

Whoa, glad to hear you're finding it useful :)
Wanna send me a PR to add your port to the list?

Just sent the PR


added builds for additional runtimes, linux is now an alias for ubuntu

cc @DustinCampbell this may require a change to how vscode looks up what OS it's running on, you'll have to infer the distro based on similar logic that cli / coreclr use.

What will be the logic for populating the the runtime list in the script with the other linux distros? Everything done on Travis/Ubuntu and otherwise stick to default runtime?

Thanks @david-driscoll -- it will change VS Code slightly, but it's a small change and makes things better. At the moment, VS Code will grab the "linux" distro, which fails to launch on RHEL/CentOS.

@hal-ler my hope (that needs to be tested) is that even though it was built on trusty (travis) that they will run correctly on RHEL / CentOS because of the different runtime provided.  I'm assuming the binary format of OmniSharp doesn't change.
If that isn't the case... then I guess I have to setup some sort of Azure VMs to shuffle the build bits too.

I think that it should work. Just make sure to also populate the list with the new runtimes (PopulateRuntimes in build.cake).


docs: add homebrew install method for kubectl plugin

It's now possible to install the kubectl plugin with homebrew: https://formulae.brew.sh/formula/kubectl-cnpg

This should not be backported because only versions 1.23.1 and later are available on Homebrew.

Thank you for the PR. Having the plugin in Homebrew is a good thing. My only concern is that reading the docs, you could think that the CNPG developers are responsible for pushing new versions on Homebrew. Can you make it clear that publishing the plugin on Homebrew is not part of the CNPG release procedure?

Certainly, I added a note block above it.

/ok-to-merge docs only change


fix meta.crossplane.io/source annotation

following page have wrong link to https//github.com/vfarcic/devops-toolkit-crossplane/tree/master/packages/k8s (missing : after https scheme)
https://marketplace.upbound.io/configurations/devops-toolkit/dot-kubernetes/v0.4.24
This may be caused meta.crossplane.io/source annotation.
upbound official crossplane.yaml (Especially, I refer to this) not including schemes

Well spotted. Thanks a ton.


vax bugs to fix

[ ] vaccinated "individuals" reported... vs vaccinated in unknowns map ALERT box (one of these is the KNOWN metric sub object)
[ ] should say "county" (not sure where this was on page)
[ ] NAN% on table

@benhammondmusic the solution for the NaN% will depend on what we decide to do about showing or not showing the ACS data, so I think we should leave that for now.


HelpNeeded: Exception - MS.IdentityModel.Token

Hi,
I try to create a demo project with jwtbinding.
I got this code and add a new functions app (.net core 3.1).
Every time I got the exception (see attachment).

Can you help me please?
Best regards
D
avid

Have you tried adding this package to your main assembly?
I also recall this is an error in the functions runtime, you need to add a flag to your project file. I'll get back to you.

Yup, you have to set _FunctionSkipXleanOutput to true. This work-around will fix your issue for now. This is a runtime error of the Functions runtime.
https://github.com/Azure/azure-functions-vs-build-sdk/issues/397

Thank you for your great support. I managed a working solution.
I'll post some more information. May other developer have similar problems.
I started a new blank solution and add functions project (HTTP-Trigger). After that I add the Nuget package for JwtBinding and update the functions packages to the latest version.
After that I had to manipulate the project file.csproj with an editor.
fp.csproj
<PropertyGroup>
  <TargetFramework>netcoreapi3.1</TargetFramework>
  <AzureFunctionsVersion>v3</AzureFunctionsVersion>
  <_FunctionsSkipCleanOutput>true</_FunctionsSkipCleanOutput>
</PropertyGroup>

local.settings.json
    "JwtBinding:Issuer": "https://demo.identityserver.io",
    "JwtBinding:IssuerPattern": "https://demo.identityserver.io",
    "JwtBinding:Audience": "Azure-API",
    "JwtBinding:Roles": "Role1,Role2",
    "JwtBinding:Signature": "asdv234234^&%&^%&^hjsdfb2%%%"

function1.cs
    public static class Function1
    {
        [FunctionName("Function1")]
        public static async Task<IActionResult> Run(
            [HttpTrigger(AuthorizationLevel.Function, "get", "post", Route = null)] HttpRequest req,
            [JwtBinding(Roles ="Role1")]AuthorizedModel authorizedModel,
            ILogger log)

Postman - header
I add this header to my postman request
Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzY3AiOlsiZGF0YTpyZWFkIiwiZGF0YTp3cml0ZSJdLCJyb2xlcyI6WyJSb2xlMSIsIlJvbGUyIl0sInN1YiI6IklkZW50aXR5MSIsIm5iZiI6MTYwNTY5OTA3MCwiZXhwIjoxNjA2MzAzODcwLCJpYXQiOjE2MDU2OTkwNzAsImlzcyI6Imh0dHBzOi8vZGVtby5pZGVudGl0eXNlcnZlci5pbyIsImF1ZCI6IkF6dXJlLUFQSSJ9.qJfHFd72UjTKWw9e2m_fR70SMvC4fyoH5wwcfa-bVh4

Thanks for the feedback, absolutely valuable...
One remark, if the issuer is fixed, like in your example, there is no need to also provide the IssuerPattern. This only applies when (for example) using Azure AD where the issuer is https://login.microsoft.com/{something}.


All logging should be done via a single goroutine

Right now any worker that produces logs calls straight into the logging routines.  This opens up the possibility for race conditions, especially during log rotation.  We should create a logging goroutine and pass all log contents over a channel to eliminate any chance of races.

I was going to implement this during the Worker pool #11 issue.

I think lumberjack does a good job right now of handling the syslog, so Im working on getting the connections logged in a single go routine.

This is solved in issue #8

Please squash all your commits into one and add what you've done in the PR.
You can use this link as a reference https://chris.beams.io/posts/git-commit/

The code was checked in and a race condition was fixed.


Update wiki

Update the wikis:

Take new screenshots to reflect the new interface
Integrate all wiki files in a single place
Fix the layouts and other minor changes


Error handling

If there are errors when using decode command, it doesn't get caught and there is no reply

Learned regex and added the necessary error handling needed to ensure the string is valid to be decoded lol


launch screens should be optional for iOS export.

Godot version:

3.0
OS/device including version:

Issue description:

Set each launch screens one by one is time consuming for fast test project. And for the exported project users may prefer to use launch screen file instead of launch image.
Steps to reproduce:
Minimal reproduction project:

For the required Icons, some engines just need to set an image, and all the icons for each size will be generated automatically.


Bug Report: CVE-2024-0985 PostgreSQL security alert

Describe the bug
Restricted users can exploit materialized views to increase their permissions.
There is a detailed bug description at https://www.postgresql.org/support/security/CVE-2024-0985/
To Reproduce
See above.
Expected behavior
Materialized views get executed in the scope of the authenticated user.
Workarounds
n/a
Screenshots and Build Links
n/a
Additional context
The Postgres image shall be updated to a fixed version:

15.6
14.11
13.14
12.18

This issue has been fixed by https://github.com/CircleCI-Public/cimg-postgres/pull/117
Hence closing this.


Revert "Replace OutputTimeFn UDF with TimestampCombiner enum"

This reverts commit f38e4271334fced94e8dc1dc97f47b60fa810586.
It will require a synchronous Dataflow worker container bump.
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

This requires a new Dataflow worker container. A number of things appear to be possibly broken, so we can merge this in any time in order to see what the other things are.

https://builds.apache.org/view/Beam/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/2942/ should eventually be a build against this. It is unlikely to be green, as it was already broken to my knowledge.

Since it is entirely certain (from logs) that the worker will get ClassNotFound errors, and Jenkins is busted again, I will merge this.

R: @tgroh (original reviewer) or anyone after the fact

LGTM


[RFC] Add links to prev and next blog Post

The proposal is to add one of the most classic features: prev and next links in the page of a single Post

Useful for:

showing how queries can be implemented for this feature
add a couple of tests

I've been thinking about this ... and I prefer to close this as "won't fix". I don't see much gain from implementing this feature. This Demo app is about showing Symfony features, not implementing a real blog engine ... so let's not implement this. In any case, thanks a lot for your proposal!


Separate the control logic from the PlayerScript.

This is a silly suggestion that increases verbosity for the sake of abstraction.


Fix RedisStoreProxy write when passing expires_in option

Version: HEAD
Redis verison: 4.1.3
Rails version: 5.1.7
This happened while throttling requests. While I was using rack-attack 6.0 I was running into exceptions when redis was offline and noticed #445, so I updated rack-attack.  After that, I got a different exception:
> wrong number of arguments (given 4, expected 3)

redis (4.1.3) lib/redis.rb, line 819
------------------------------------

  814     #
  815     # @param [String] key
  816     # @param [Fixnum] ttl
  817     # @param [String] value
  818     # @return [String] `"OK"`
> 819     def setex(key, ttl, value)
  820       synchronize do |client|
  821         client.call([:setex, key, ttl, value.to_s])
  822       end
  823     end
  824

Full backtrace
--------------
 - redis (4.1.3) lib/redis.rb:819:in `setex'
 - /usr/local/lib/ruby/2.5.0/delegate.rb:83:in `method_missing'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/store_proxy/redis_store_proxy.rb:19:in `block in write'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/store_proxy/redis_proxy.rb:62:in `rescuing'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/store_proxy/redis_store_proxy.rb:19:in `write'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/cache.rb:79:in `do_count'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/cache.rb:26:in `count'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/throttle.rb:30:in `matched_by?'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/configuration.rb:77:in `block in throttled?'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/configuration.rb:76:in `any?'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack/configuration.rb:76:in `throttled?'
 -  () usr/local/bundle/bundler/gems/rack-attack-a7ce9a89fe68/lib/rack/attack.rb:108:in `call'

Looking at the docs, it doesn't seem possible to pass an options arguments there.
I did not add any tests since this there weren't any for lib/rack/attack/store_proxy/redis_store_proxy.rb and this seemed pretty specific, so it shouldn't go to the offline examples.
After doing so, rack attack continued working correctly.

Hi @dgmora,
Thank you for taking the time to debug and contribute.
Would like to to understand better why is that your app is using the RedisStoreProxy but it ends up in lib/redis.rb setex with 4 arguments.
Can you please share your Rack::Attack store backend configuration for your app?
Are you setting Rack::Attack.cache.store = ...?

Hi, sorry for the delay.
Rack::Attack::StoreProxy::RedisStoreProxy inherits from Rack::Attack::StoreProxy::RedisProxy and BaseProxy. I think that since the method has a different signature (3 vs 4 arguments) there's a method_missing and then it gets delegated to redis because setex is not implemented in RedisProxy or BaseProxy itself.
This is our setup in config/initializers/rack_attack.rb:
Rack::Attack.throttle(...)

ActiveSupport::Notifications.subscribe(/rack_attack/) do
  . . .
end

Rack::Attack.cache.store = Rack::Attack::StoreProxy::RedisStoreProxy.new(Redis.new(url: <url>))

I have checked the docs, and also set does not seem to accept the option raw: true, so I am not sure if it's actually achieving anything in the else in RedisStoreProxy. I don't have the whole picture here, but maybe it'd be possible to remove this whole method from RedisStoreProxy and just inherit it from RedisProxy?

Hi @dgmora ,
RedisStoreProxy is meant to proxy Redis::Store and RedisProxy is meant to proxy Redis, as can be seen in their respective .handle? methods.
So in your case you need to change the setup to:
Rack::Attack.cache.store = Rack::Attack::StoreProxy::RedisProxy.new(Redis.new(url: <url>))

Hi, I'll give this a try. Anyway, isn't the implementation still wrong?
On Fri, Nov 1, 2019, 18:28 Gonzalo Rodriguez<EMAIL_ADDRESS>wrote:

Closed #455 https://github.com/kickstarter/rack-attack/pull/455.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kickstarter/rack-attack/pull/455?email_source=notifications&email_token=AA3RIEAAAO5LPIALGXO53BDQRRRLLA5CNFSM4JFZI4Z2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUS7B74I#event-2763923441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3RIEGMDII7FFATHHMVW73QRRRLLANCNFSM4JFZI4ZQ
.


Record in comments when an issue is moved into Kanban board

Feature Description
After some using Kanban for about 1 month, it is helpful to know when and by which user an issue was moved into a kanban board, this information is useful for metrics. I think the concept is called event sourcing.
Screenshots
No response

Duplicated with #27278, and it is WIP: #29311

my bad thanks


pdf

Signed-off-by: Asra Ali<EMAIL_ADDRESS>

Is this something you want to actually merge or is it a test?


Can no longer usefully archive or move replays

In beta 2-ish I found it very convenient to be able to move my replay files from my laptops (I regularly use 3) to my desktop where I could record them with streaming software and keep them in one place. In the latest version this isn't possible since they're stored in a database file that doesn't allow accessing or manipulating individual runs.
I'm basically sticking to beta 2 because of this. :/

In that case I can look into export/import functions in the near future. I'll aim for it to be in the next pre-release.


[WIP] Adapting Pulse Instructions to the new Pulse Model

Summary
This PR presents an option for the needed adaptation of the instruction interface to the new Pulse Model (#11215, #10694) as part of #10759.
Details and comments
Demonstrated on the Play instruction, the approach taken here is the following:

New named arguments are added for mixed_frame or frame and pulse_target combination.
Type checking is used to identify if the old interface is used, values are moved to the correct arguments, and deprecation warnings are raised.
PendingDeprecationWarning is raised for use of Channel.
during __init__ the input is verified to guarantee that a valid combination was provided.
The Channel or MixedFrame are stored at operands[1], and are accessible via the property mixed_frame or channel (for backwards compatibility and only when a Channel is provided).

Pros:

No breaking changes.
Easy transition from legacy code (within the channel model)

Cons:

The property mixed_frame returns either a Channel or a MixedFrame.
Type checking is used for old interface detection.
_validate() comes out strange.

This PR depends on #11215. Relevant changes are to Play.py and associated tests. Tests are failing due to the deprecation warnings.

The development of new pulse features was moved to a dedicated feature branch, and this PR is no longer relevant.


Not sure why ethers-multicall not work in Fantom

We are trying to use ethers-multicall (https://www.npmjs.com/package/ethers-multicall) to group multiple calls to Fantom RPC, but it seems not working. Would you mind to take a look at and share any thoughts/guidance if possible?

sample reproduction code:
let mushContract = new Contract("0x883fb00742161dce2235bbf4a67df84cd0e50c08", "contractABI");

let marketPriceCall = mushContract.getMarkPrice("0x321162Cd933E2Be498Cd2267a90534A804051b11");
let exposureBalanceCall = mushContract.exposureBalances("0x43229759E12eFbe3e2A0fB0510B15e516d046442", "0x321162Cd933E2Be498Cd2267a90534A804051b11");
let tokenBalanceCall = mushContract.tokenBalances("0x43229759E12eFbe3e2A0fB0510B15e516d046442", "0x321162Cd933E2Be498Cd2267a90534A804051b11");
let exposureLeverageBpsCall = mushContract.exposureLeverageBps("0x43229759E12eFbe3e2A0fB0510B15e516d046442","0x321162Cd933E2Be498Cd2267a90534A804051b11");

let [
    marketPrice,
    exposureBalance,
    tokenBalance,
    exposureLeverageBps,
] = await callProvider.all([
    marketPriceCall,
    exposureBalanceCall,
    tokenBalanceCall,
    exposureLeverageBpsCall,
]);

I have this error on avalanche...

@mushroomsforest do I understand correctly that you need to deploy a contract on Fantom to get rid of this error? Could you point to which contract needs to be deployed?


is OpenEdge 10.2 A version supported

in appsettings.json file we have connection string as:
"ConnectionStrings": {
"OpenEdgeDbContextConnStr": "DRIVER={Progress OpenEdge 10.2A driver};host=localhost;port=12345;db=modhotel;uid=myUser;PWD=myPass;DIL=READ UNCOMMITTED;",
}
When we run the below code
public async Task<IEnumerable> GetTest()
{
using (var ctx = dbContext)
{
var test = ctx.o_kuradi.ToList();
...
}
}
we get the following error:
ERROR [IM003] Specified driver could not be loaded due to system error  193:  (Progress OpenEdge 10.2A driver, C:\Progress\OpenEdge\bin\pgoe1023.dll).
Our OpenEdge version is 10.2A and our version is 32bit.

hi @umityaz
I personally haven't tested against 10.2A but the SQL interface hasn't changed a lot so the core functionality should be the same.
That error you see generally indicates a mismatch in bitness between the application and the ODBC driver.
Most likely you only have the 32-bit version of the ODBC driver installed on your system, but your .NET application is running as 64-bit.
You have two options:

Change your .NET application to run as 32-bit
Install the 64-bit version of the ODBC Client Driver

First thing I'd try is running your .NET application as 32-bit to see if that resolves the issue. Let me know how you get on.

hi @umityaz
I personally haven't tested against 10.2A but the SQL interface hasn't changed a lot so the core functionality should be the same.
That error you see generally indicates a mismatch in bitness between the application and the ODBC driver.
Most likely you only have the 32-bit version of the ODBC driver installed on your system, but your .NET application is running as 64-bit.
You have two options:

Change your .NET application to run as 32-bit
Install the 64-bit version of the ODBC Client Driver

First thing I'd try is running your .NET application as 32-bit to see if that resolves the issue. Let me know how you get on.


Update Readme.md to explain the host network

Thank you for this amazing project, I found it very useful. I am new to Docker so it took me a while to figure out how to access the host network. I updated readme to add a quick command helping others. If you are open to PR of course.

I am using this project to reliably run tcpreplay on Windows10.

Awesome thanks :)


Reworked CI

Made caching better, it now cleans up uneeded garbage and recovers on cache misses
Split testing, clippy and rustfmt into separate jobs so they can run independently and not interrupt each other
Added cargo-bloat-action which will comment on PRs which affect dependencies and display the current number of dependencies, what dependencies have been added/removed and the final binary size each dependency inhabits
Split the docker job into a separate file since it has nothing to do with the rest of the CI (it's self-contained)
Added healthchecks to the compose file which should make running docker-compose more seamless

r? @Nemo157

It looks like cargo-bloat lacks the permissions to comment @pietroalbini I don't know how to fix that, could you take a look?

@Kixiron I'm not seeing any error related to cargo-bloat?

Github linked improperly but this should work

@Kixiron that's not possible: the GITHUB_TOKEN available in PRs is read-only, and no other secret can be used in PRs.

@Nemo157 you were the main reviewer for this, are you happy with it? If so I think it can be merged after a rebase.


Mutation to create product image

Fixes #1997
This PR adds the ability to upload product image and answers the question of how do we support file upload in GraphQL by using multipart POST requests.
File upload itself is not an issue, but I failed to use our ModelFormMutation mechanisms and found it much easier to do it with a plain Graphene mutation. It worries me that our form-based mutations are not so easy to understand and customize when looking at it after some break.
Anyway, for now, I guess we the solution for the upload. I'll try to examine the complexity of ModelFormMutation separately and think of better options.
Pull Request Checklist
(Please keep this section. It will make maintainer's life easier.)

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[ ] JavaScript code quality checks pass: eslint.

I tested the upload with curl (apparently none of the GUI clients that I used supports upload in GraphQL):
curl localhost:8000/graphql/ -F query='mutation { productImageCreate(productId: "<product_id>") { productImage { id image url order} }}' -F<EMAIL_ADDRESS>

Since we need to write dedicated forms anyway, what are the pros of sticking to form classes?

How easy/how hard is handling this kind of upload in Apollo?

@patrys I'm not sure about the complexity in Apollo, I assumed that since I'm able to manually construct the request using the standard tool and successfully upload the file, we'll somehow be able to integrate it on frontend side. I'll take a look how Apollo handles upload and see if my solution needs adjustments.
Apollo examples:
https://github.com/jaydenseric/apollo-upload-client
https://gist.github.com/tvorogme/1c0794e1455bf3fc61162e2915ec2462

I adjusted the upload mutation to the spec which is also respected by Apollo client. I had to use this package, as by default Graphene doesn't support this spec.
I also had to make a change in tests for GraphQL views. The client we were using to send requests was using multipart content type, which worked, but conflicted with graphql-multipart-request-spec. Right now all standard queries are sent as application/json.


Symfony 4 Compatibility

Unfortunately this package is currently not compatible with Symfony 4.
The first issue that seems to appear is in the service definitions when running behat:

In YamlFileLoader.php line 798:
[Symfony\Component\DependencyInjection\Exception\InvalidArgumentException]
The configuration key "scope" is unsupported for definition "friendly.alice.fixtures.loader" in "/app/vendor/knplabs/friendly-contexts/src/Knp/FriendlyContexts/services/core.yml". Allowed configuration keys are "alias", "parent", "class", "shared", "synthetic", "lazy", "public", "abstract", "deprecated", "factory", "file", "arguments", "properties", "configurator", "calls", "tags", "decorates", "decoration_inner_name", "decoration_priority", "autowire", "autoconfigure", "bind".
The configuration key "scope" is unsupported for definition "friendly.alice.fixtures.loader" in "/app/vendor/knplabs/friendly-contexts/src/Knp/FriendlyContexts/services/core.yml". Allowed configuration keys are "alias", "parent", "class", "shared", "synthetic", "lazy", "public", "abstract", "deprecated", "factory", "file", "arguments", "properties", "configurator", "calls", "tags", "decorates", "decoration_inner_name", "decoration_priority", "autowire", "autoconfigure", "bind".

There is many issues with Symfony 4 compatibility. This package is also not maintained ATM. For now you can refer to this: https://github.com/KnpLabs/FriendlyContexts/pull/247#issuecomment-381359797

Ah... ok.
When you say:

This package is also not maintained ATM.

Do you mean that the owners are not actively developing? Or does it also mean that submitted pull requests will not be reviewed and integrated (or will take a long time)?
If the repo is dead, perhaps it can be put up for adoption or retired officially? Otherwise it will block lots of people who use it from moving forward/updating while leading them into the idea that it might one day be updated.

Do you mean that the owners are not actively developing?

Owners are not. I am the official maintainer of this library. (Many things impact the decision of not maintain this lib, including lack of time, not using this library on any of my projects and not good reaction of the team while forking another project on my company organization (for maintenance reasons))

Or does it also mean that submitted pull requests will not be reviewed and integrated (or will take a long time)?

Yes.

If the repo is dead, perhaps it can be put up for adoption or retired officially? Otherwise, it will block lots of people who use it from moving forward/updating while leading them into the idea that it might one day be updated.

I understand very well that. Also, I have some very interesting work staged for commit on my computer. I'm currently working on looking for somebody to help me with the maintenance of this project. But following problems are to consider:

There's no stable version for this project. Which means we will not really able to release new things for people already using this project;
New features and issues imply huge backward compatibility breaks

Things may move next week because I think this project is a nice one and I want to make it great again, and it looks like I'm not alone. If you're interested to see this project maintained again, please put a 👍 to motive us! ❤
See you next week 😄.

Sorry for the delay...

Hey @Nek-, did anything come out of the experiments discussed?

@yakobe
A 'feature/1.xbranch has been created in order to prepare the next big release, with a lot of BC breaks, so themaster` branch remains safe.
I think it's important because most of users use this branch currently (it is recommended in the doc unfortunately).
About compatibility with Symfony4:
I forgot about it when I wrote the roadmap, but it's the next big thing planned.
However, our dependency of nelmio/alice is not compatible with Symfony4, so we first need to update it and rewrite some internal things (fixtures creation and persistance).
It has been started with https://github.com/KnpLabs/FriendlyContexts/pull/249.

Hi @Nek- and @pimolo. Is there likely to be any progress on this? Is there something that people can help with or should we consider alternative solutions?

@Nek-
I am currently upgrading my project to SF4 and all my behat steps created the last 3-4 years are strongly based of freindly-contexts. Is there a substitute or a way for me to not re-write hundreds of behat tests, still keep all the tests workingand most importantly upgrade to sf4?
I understand this is not maintained anymore, but I can't be the only one with this dilema of droping all these behat tests that were created throw out the years or re-write all of them and I hope there is something to do with it. You said you have no use for FriendlyContext in your own projects. Are you using another context package?
Thank you.

News of the migration of the project to Sf4+ ? Ty

I have the suspicion this project is abandoned. Perhaps one of the maintainers should mark it so that people are aware?

@Nek- can you give us some details about this topic ?

Sorry, I think this library can be considered as totally unmaintained. I'd like to focus the (little bit of) time I have on improving Behat itself the following weeks/months. :/

Maybe it’s time to mark it so on packagist and on the readme?
Also, perhaps someone can then create an maintained fork without fear of code divergence.

Maybe @FriendsOfBehat would be interested in adopting FriendlyContexts?

If they are interested about (I don't think so) we can have a chat about it on symfony-devs slack.


[SPARK-8080][STREAMING] Receiver.store with Iterator does not give correct count at Spark UI

In custom receiver if I call store with Iterator type (store(dataIterator: Iterator[T]): Unit ) , Spark UI does not show the correct count of records in block which leads to wrong value for Input Rate, Scheduling Delay and Input SIze. This pull request fix that issue.

Can one of the admins verify this patch?

I am sorry. My bad . Do let me know if latest change looks fine

ok to test

Merged build triggered.

Merged build started.

Test build #34131 has started for   PR 6614 at commit 20b9374.

@tdas @zsxwing

@dibbhatt could you add [STREAMING] to the title of the PR?

Test build #34131 has finished for   PR 6614 at commit 20b9374.

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

Merged build finished. Test PASSed.

added [STREAMING] to the title of the PR

I think we need a different design. There is a way to count the elements in the iterator without putting it into some intermediate buffer. Rather, the iterator is going to be consumed any way (assuming StorageLevel is serialized) by the block manager. The counting can be done while that is happening. To do this you have to construct a special CountingIterator that wraps the original iterator.
class CountingIterator[T: Manifest](iterator: Iterator[T]) extends Iterator[T] {
   var count = 0
   def hasNext(): Boolean = iterator.hasNext
   def next() = {
    count+=1
    iterator.next()
   }
}

after iterating the original iterator via this counting iterator, you can get the count of the records it had iterated through. This number can then returned through BlockStoreResult object.
How does that sound?

It sounds good Tathagata. I will update the PR with the changes .

Hi @tdas . Let me know how this looks...

Merged build triggered.

Merged build started.

Test build #34192 has started for   PR 6614 at commit f9aaa9f.

Test build #34192 has finished for   PR 6614 at commit f9aaa9f.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class CountingIterator[T: Manifest](iterator: Iterator[T]) extends Iterator[T]

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #34197 has started for   PR 6614 at commit c631a26.

Test build #34197 has finished for   PR 6614 at commit c631a26.

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

Merged build finished. Test PASSed.

This is getting a little bit complicated to think how this will work for all combinations of  ReceivedBlock types, and StorageLevels and ReceivedBlockHandler types. This must be done with unit tests. Mind adding unit tests for this A x B x C combinations in the ReceivedBlockHandlerSuite, so that we can clearly verify for what combinations we cant do this, and what combinations we do it correctly?

Sure Tathagata , will add a unit tests..

I am going to merge #6659 to prevent negative numbers in the UI. Could you please merge with the master after that PR is merged to verify that there are no negative numbers? Also, if possible, you unit test should verify this as well.

Sure

Just to understand when the message count may go wrong ..here is my understanding. Please let me know if I am wrong .
As I see the logic , if WAL is enabled , we are fine as WriteAheadLogBasedBlockHandler serialize the block . If WAL is not set  , BlockManagerBasedBlockHandler use BlockManager's putIterator call for both Iterator and ArrayBuffer.  If the storagelevel is ExternalStore or DiskStore ,  both will serialize the iterator and count comes fine... For MemoryStore also it also does a unrollSafely of the iterator if enough memory is there. The issue may come if memstore fails to do safe-unroll ( or does partial unroll ) dues to lack of free memory space. Then the reported count will show less than actual number of records in block.  This issue will happen irrespective of replication factor.
Will this be a very common behavior ? In my environment for every storage level I get correct count of number of kafka messages I pumped. Probably I need to hit the memory limit of MemoryStore unrollSafely to run out of memory....which seems to be rare scenario .
Let me know what you think.

Merged build triggered.

Merged build started.

Test build #34293 has started for   PR 6614 at commit 51de69e.

Test build #34293 timed out     for PR 6614 at commit 51de69e     after a configured wait of 175m.

Merged build finished. Test FAILed.

Good point. I think the solution is to make the CountingIterator verify whether the end of the internal iterator was reached or not (that is hasNext had returned false or not). Accordingly we will know whether the count is correct or not. In cases where the count is in complete (e.g. MEMORY_ONLY, with not enough memory), the count can be ignored because in those cases the block was not successfully put into the memory. And so the count can be considered to be zero. In fact ideally we should be throwing exceptions saying that block could not be inserted in the block manager.
Does that make sense?

Build triggered.

Build started.

Test build #34391 has started for   PR 6614 at commit c250fb5.

What I observed is , when Block is not able to unrollSafely to memory if enough space is not there, BlockManager won't try to put the block and in ReceivedBlockHandler it already throw the SparkException as it could not find the block it in PutResult. Thus block count won't go wrong if block is not able to unroll safely. So I was wrong earlier ...
For MEMORY_DISK settings , if block not able to unroll to memory, it still get deseralized to Disk. Same for WAL based store. So for those cases ( storage level = memory + disk ) , also count will come fine if Block not able to unroll.
thus I added the  isFullyConsumed in the CountingIterator but have not used it as such case will never arise that block not fully consumed and ReceivedBlockHandler still get the block ID.
I have added few test cases to cover those block unrolling scenarios also.

Test build #34391 timed out     for PR 6614 at commit c250fb5     after a configured wait of 150m.

Build finished. Test FAILed.

hi @tdas . I can see the build passed all test cases but it timed out.

@dibbhatt could you resolve the conflicts again? Thank you.

Hi @zsxwing , I resolved the conflicts . This PR modified the ReceivedBlockHandler which now takes the number Of Records, and that need to modify ReceivedBlockTrackerSuite from your PR to take the numRecords parameter... Is there any issue you see in merge ?

Build triggered.

Build started.

Test build #34403 has started for   PR 6614 at commit 28225d5.

@dibbhatt did you modify any commit by accident, or is it a GitHub bug? There are 219 files changed and it still contains conflicts.

Test build #34403 has finished for   PR 6614 at commit 28225d5.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

hi @zsxwing ...No , I have made only commit for my changes . Not sure why it says 219 files changed. If you see the commits details , you can see only files related to this PR has changed. Just now I modified comments on a file to trigger the build once again.
This 219 files changed came after I merged my repo from upstream/master to take your PR changes. And I merged only your changed with mine and committed those .
Below are the my changes since the merge...and I have not committed all these 219 files :(
https://github.com/dibbhatt/spark/commit/0892156a1f08e3092d924e57a77a871a4843b016
https://github.com/dibbhatt/spark/commit/c250fb5fb6a08d89b0aed05b5d3eec44b9991c01
https://github.com/apache/spark/commit/28225d58846e6c214fbc6f7e1ef09cda6b7cd94d

@tdas @zsxwing , if this PR is screwed up , we can close this one and create a fresh one ? How to proceed further ?

I've no idea why it's screwed up. Feel free to open a new one.

Closing this PR. Will open a new one. 6614 got issue with merging from upstream/master and lot of unwanted files has come to this PR . Will open a new one


Extract NCIT module based on data/cineca.tsv

The data/cineca.tsv file has NCIT term IDs for most rows. We want to extract a subset of NCIT for those terms, then compare that hierarchy to the asserted hierarchy in data/cineca.tsv.
MIREOT extraction should be fine for now. Since the NCIT OWL file is biggish, it would be nice to use the new MIREOT-RDFXML code from https://github.com/ontodev/robot/pull/605. (It might have fancy nested OWL annotations that would exercise the edge cases we discussed, but we don't need to get those 100% correct for this purpose.)

As mentioned by email, the NCIT mappings are medium-trustable. There may be better terms available in other resources. I'm hoping we'll be able to run mapping tools eg OxO to generate the list of mappings as cross-refs for each term.

That's fine. I updated the demo site with new versions of the trees we've got so far.


Fix variable use when querying a node

Apologies, there was a typo in my previous PR - this fixes it, and adds a test to be doubly sure.

Thanks again! The fix is published in v0.5.6


Custom tooltip totally obstructing the main report

It's quite disappointing to see that the native tool tips background are relatively transparent and the custom ones are not. This unavailable feature in the custom version is taking away from the beauty and useability  of this very useful element of Dashboarding.
Can we have the ability to set the background of a custom tooltip to be transparent?

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

ID: 97cbbb3e-3916-4340-4ddc-67bf22159bf3
Version Independent ID: 5b620628-1c10-b0bb-bd4d-1fd229260ea1
Content: Create report tooltip pages in Power BI - Power BI
Content Source: powerbi-docs/create-reports/desktop-tooltips.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @Bifarin. This issue has been assigned to the doc author for review.

Hi, @Bifarin -- I'm checking with the product team to see if it's possible to format it as you're suggesting. I'll let you know what I find out.

Hi, @Bifarin -- I checked. Currrently not possible, but on the product roadmap. So it's coming!


SQL logic test suite

To guarantee that SQL queries actually do as they're told, a very thorough test suite needs to be added.
Sqllogictest may be worth looking into: https://www.sqlite.org/sqllogictest/doc/trunk/about.wiki

I'll be committing a bit of time to get some sort of test suite running. I'd like to be able to hack on this have the confidence that my changes aren't creating any major regressions.


Duplicate active half when duplicating split pane

Description of the new feature/enhancement

When a pane is split then duplicated, the right half is always the one duplicated. It should duplicate whichever half has focus when right-clicking the tab.
Proposed technical implementation details (optional)

Hmm. This is with the right-click context menu on the tab itself, yea?

Seems like whenever the inactive tab is split, it's like we make a new default profile pane, rather than duplicating the active pane. Curious. Thanks for the bug report! It should probably do the same thing when the tab is inactive, as it does when the tab is active.

Yes, but I'm actually referring to the "Duplicate Tab" option. Open a tab, split it, change the directory on one half, then duplicate it, and the new tab created by the duplication has the path of one or the other of the split tab's panes, obviously. The problem is that it's very insistent on using whichever pane it feels like for the path in the new, duplicated tab. When I first tested it, it was always the right pane. Testing again just now, it was the left pane, then somehow I got it to switch to using the path from the right pane and I can't get it to switch back. It should use the path from whichever pane is active when the tab is duplicated.
The same problem applies to splitting an already split tab. The new pane has the same behavior as the new tab created when duplicating, and should have the path of the active pane when the tab is further split.

To add to this, I've found more unexpected behavior I think may be related. I have the starting directory for a new tab set to use the parent process directory, which I apparently wrongfully assumed meant the parent tab, but AFAICT it uses the first tab's CWD. This appears to be what issue #3158 deals with, and if that's correct this isn't a bug, but definitely, as mentioned, unexpected. However, what definitely does appear to be a bug is that even duplicating a tab results in the new tab having the CWD of the first tab, and I suspect that's the source of the previously mentioned problems.

Hello. I was about to create my own issue with a somewhate-related subject: shouldn't the "Duplicate tab" duplicate also the split pane layout?
From my understanding, when I right click on a tab that is splitted in 4 ways, I'm expecting the new created tab to be the same: being split in 4.
Am I the only one? Am I missing something?

@4wk- That we're more specifically tracking in a combo of #4674 and #9957.

To add, I think this was fixed at some point in the past. I at least can't repro it on the 1.18 selfhost build build I'm on currently. I'd suspect this was at least fixed in 1.17, probably sooner. I can't really recall which PR fixed this.
Of course, duplicating the CWD of a pane that assumes that you've got shell integration enabled, but this all looks working to me.
I tried both

Open a non-default profile in a second tab
Open a new pane in that tab with another non-default profile
focus the first tab
Right-click inactive tab...split tab -> Always splits the focused pane, and creates a copy of the focused pane

and also

Open a tab,
split it,
change the directory on one half,
then duplicate it -> new tab is created with the profile (and CWD if shell integration is enabled) of the active pane.

@zadjii-msft Thanks for your reply. If I understand correctly, I need shell integration enabled for keeping my split panes when using the "Duplicate tab" button?
I though shell integration was for CWD (current working directory), and I do not wish to have my duplicate tab on my current working directory.
I am on Version: 1.16.10261.0 so maybe I should just wait :)

@zadjii-msft After more reading, I can confirm that what I'm asking for is #4674: when using the "Duplicate tab" feature, panes should be kept the same (so if a tab is split in 3 panes, "Duplicate tab" should create a new tab with 3 panes in it).
And if I understand correctly, #9957 is needed before #4674 can be worked on. Thanks!


fix(scroll): horizontal scroll is disabled with empty rows

Fixes the issue where it is not possible to scroll horizontally when there is no data causing some columns headers to not be visible.
What kind of change does this PR introduce? (check one with "x")

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

@timowolf Could you please review this PR?


Add getBoundingBoxes getter to SelectableFragment class

This fixes the error introduced with Flutter 3.19.0

cc: @ronjb

Thanks @JeisonSanches for creating this pull request! I really appreciate the help with this library.
I published float_column 2.1.2 to pub.dev which fixes this issue and includes other bug fixes and updates.


Fix thread dumping in Lua 5.4.3+

luaL_buffinit apparently can push things onto the stack (although this is undocumented in the Lua 5.4 manual), so it needs to be called after lua_dump to make sure the function to-be-dumped is still at the top of the stack in lua_dump
Closes #553

+1. Would be nice to get https://github.com/luvit/luv/pull/551 in to test this PR fully.

Do we a CI job to test 5.4.1 and 5.4.3?

Accidentally, yes, since Windows CI is still testing against 5.4.1. But, besides that:

Luarocks 2.x uses Lua 5.3 (specifically 5.3.2)
All the LuaJIT tests test against LuaJIT-flavored Lua 5.1

So we can be reasonably sure that this is backwards-compatible.


Log stack trace to console for exceptions.

I also fixed some inconsistent indentation in these sections.

The issue is that you're catching the exceptions and logging the exception message without re-throwing the exception, effectively suppressing the stack trace which makes debugging difficult.
Yes, this approach does introduce duplication of the exception message which is also included in the stack trace. There are a couple alternatives:

Pass the whole stack trace into chalk, i.e. "Reason: ${err.stack}", which would make the entire traceback red.
Only log the stack trace if a -d/--debug flag is passed. This is the approach HackMyResume takes (which does duplicate the exception message) but I don't recommend it. Anyone using a cli tool can handle a stack trace.

Oh I see, didn't thought about the stack trace. I wouldn't say  we should always log the stacktrace  because "it's CLI  tool". The first reason is readability - even a CLI tool shouldn't always overwhelm the user with tons of output. Second, I hope I would be able to start to work on a GUI using electron and some library generating forms from JSON-Schema. For that, extending the logging wrappers would be necessary and my first reason would apply even more.
Nevertheless, the -d/--debug  is a great Idea and very meaningful extension. I try to find some time soon to implement it.
Thus, I would prefer to omit the doubled console.error calls in the code.

I guess I wouldn't mind the -d/--debug flag so much if a message was added to logError like "please run with -d/--debug for full stack trace" so that one doesn't have to puzzle over a brief and unhelpful error message.

I found some time to look at the debug flag. I actually don't want to set another parameter to each command and I think it might be useful to respect the global debug level. I didn't find a way to do it directly and it seems to me that there might be a bug in caporal, see my bugreport

I updated KissMyResume to use the Caporal V2 and added the support for the global verbose mode, see f9f3452


Player can order units to attack

Depends on #38 and #25
Acceptance criteria:

[ ] select unit
[ ] when mouse cursor is hovering over an enemy unit (or structure) then it becomes a 'attack here' icon
[ ] when left mouse button pressed when 'attack here' icon then the units will move closer to target (if needed) and destroys structure (doing damage is other story)

#25 is solved. But still depends on #38

#38 is solved. So this issue is now ready to build.

Done! yay!


logstash-forwarder support

It would be nice if you could add logstash-forwarder support.
I believe this requires SSL. But not sure how to set this up??

In fact, the logstash part of this solution is dynamic. You can update/replace the existing logstash configuration, restart the stack and it will be loaded.
So you just have to generate a SSL certificate, setup logstash-forwarder on a node and configure logstash with the lumberjack input.
See this article for more information: https://deviantony.wordpress.com/2014/05/19/centralized-logging-with-an-elk-stack-elasticsearch-logback-kibana/
Do not hesitate to ask any questions, I'll be glad to help.

@dkirrane I'm closing the issue, do no hesitate to re-open it if you got any questions.


Fix #2538 and #2801

#2538: Missed check for ambient module declaration in resolving default class name
#2801: Clean up the module preamble writing code to ensure modules with no aliases still result in valid JS

:+1: after comments are addressed.


How to force a parameter to have an example

I would like to confirm that any parameter has an example defined. So, in the example below, it would throw an error in fromDateTime but toDateTime would be OK.
      parameters:
        - name: fromDateTime
          in: query
          required: true
          schema:
            type: string
            format: yyyy-MM-dd HH:mm
        - name: toDateTime
          in: query
          required: true
          schema:
            type: string
            format: yyyy-MM-dd HH:mm
            example: 2024-01-28 11:13

Thank you!

Hi @andrefpoliveira, you can define a configurable rule like this in your redocly.yaml file to check each parameter has an example:
rules:
  rule/require-example-in-parameters:
    subject:
      type: Parameter
      property: example
    assertions:
      defined: true

Or, for your exact case, you can check that every schema inside parameters has an example:
rules:
  rule/require-example-in-parameter-schemas:
    where:
      - subject:
          type: Parameter
        assertions:
          defined: true
    subject:
      type: Schema
      property: example
    assertions:
      defined: true

Please let me know if that helps.


sateliteAOD

WHAT DOES THIS PR DO?
Derived pm2.5 from AOD

[ ] Item 1
[ ] Item 2

WHAT ISSUES ARE RELATED TO THIS PR?

Jira cards

[]

GitHub issues

Closes #

HOW DO I TEST OUT THIS PR?

[ ] Analytics: Link to Analytics README
[ ] Auth Service: Link to Auth Service README
[ ] Calibrate: Link to Calibrate README
[ ] Data Proxy: Link to Data Proxy README
[ ] Device Monitoring: Link to Device Monitoring README
[ ] Device Registry: Link to Device Registry README
[ ] Device Status: Link to Device Status README
[ ] Device Uptime: Link to Device Uptime README
[ ] Exceedances: Link to Exceedances README
[ ] Firebase: Link to Firebase README
[ ] GP Model: Link to GP Model README
[ ] Incentives: Link to Incentives README
[ ] Kafka Connectors: Link to Kafka Connectors
[ ] Locate: Link to Locate README
[ ] Metadata: Link to Metadata README
[ ] Predict: Link to Predict README
[x] Spatial: Link to Spatial
[ ] View: Link to View README
[ ] Workflows: Link to Workflows README

WHICH ENDPOINTS SHOULD BE READY FOR TESTING?:

[ ] Endpoint 1
[ ] Endpoint 2

ARE THERE ANY RELATED PRs?

[ ] Related PR 1
[ ] Related PR 2

thanks @wabinyai , this is great progress. Please check out why the deployment preview failed and attempt to fix asap: https://github.com/airqo-platform/AirQo-api/actions/runs/9229964842/job/25397242128?pr=3190

All should be fine now


Modified file format of mm10-tRNAs.ss.sort

Hi, thanks for implementing the code on finding tRFs!
I recently want to create a new fasta file for using mm39 as the reference genome. However, I cannot create a tRNAs.fa using your perl scripts of makeBedMM10.pl and makeMatureGenome.pl due to change of file formats downloaded from GtRNAdb. From the new downloaded files, only mm39-tRNAs-confidence-set.ss and mm39-tRNAs-detailed.ss can be found, while no files with extension .ss.sort. I want to ask how to modify the codes to parse these new file extensions to find the tRFs?
Many thanks.
Samuel

As the header format of the mm39-tRNAs.fa have changed, the header should be modified for the script to run. Also, mm39-tRNAs-confidence-set.ss should be used instead of mm39-tRNAs-detailed.ss.

Hi Samuel,
I appreciate your interest in this method! Unfortunately, I haven't
interacted with it in years and don't know how the new tRNA database files
are formatted.
Best,
Sara
On Thu, Feb 17, 2022 at 10:18 PM samuellawty @.***>
wrote:

As the header format of the mm39-tRNAs.fa have changed, the header should
be modified for the script to run. Also, mm39-tRNAs-confidence-set.ss
should be used instead of mm39-tRNAs-detailed.ss.
—
Reply to this email directly, view it on GitHub
https://github.com/sararselitsky/tDRmapper/issues/8#issuecomment-1043813728,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQD6T6TFBNEUXL6P3HU323U3W275ANCNFSM5OWNWKAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[Question] List of OS images to deploy in harvester?

Hello, is there a "curated" list of OS images that work wonderfully in harvester?
I was trying to create a cluster in harvester through rancher and I was wondering what good OS images that work very well with harvester.
I tried a ubuntu minimal cloud image and the vnc screen was throwing a boot error

+1, while youtube videos and some how-tos articles usually uses ubuntu minimal 20.04, it would be helpful to have more documentation and a list of officially supported images along with some basics cloud-init config examples.  Also, as Elemental development seems to be coming to an official release soon, I would suspect that using Elemental as a base image should be pretty straightforward?


Rinrab.AOVpnManager version 0.1.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

An anti-virus product reports an infection in the installer

That's because I don't have certificate to sign my build.
Is it important to add my application to winget?

Unsure how installer signing works, but i keep seeing other installer which is just blocked by smart screen and they go trough the validation (mby they got certificate but not validated with MS?), this installer is just straight up blocked, i won't approve the package but it did go trough validation so ill let someone else deal with it =)


Chain .where() to .find() cannot further filter my results

Node version: v14.16.1
Sails version 1.4.2:
ORM hook version 3.0.2:
Sockets hook version 2.0.0:
Organics hook version N/A:
Grunt hook version 4.0.0:
Uploads hook version N/A:
DB adapter & version<EMAIL_ADDRESS>Skipper adapter & version N/A:

Hi I am developing a Sails app and I want to chain .where() to .find() to further filter my results.
However, when I chain .where() to .find() as below:
let showcases = await SupplierShowcase.find({
      supplierId: "f91750ab-11b5-425d-89fa-5a4ad4a23eb9",
}).where({ createdAt: { ">":<PHONE_NUMBER>485 } });

It doesn't find all the showcases with supplierId, f91750ab-11b5-425d-89fa-5a4ad4a23eb9, which are created after<PHONE_NUMBER>485, but just returns all showcases created after<PHONE_NUMBER>485.
.where() chained to .find() cannot further filter my results.
[
    {
        "supplierId": "7367238c-c146-4c00-a5d6-13f89791527b",
        "createdAt":<PHONE_NUMBER>788
    },
    {
        "supplierId": "f91750ab-11b5-425d-89fa-5a4ad4a23eb9",
        "createdAt":<PHONE_NUMBER>176
    },
    {
        "supplierId": "7367238c-c146-4c00-a5d6-13f89791527b",
        "createdAt":<PHONE_NUMBER>657
    }
]

Anyone know how to fix this problem?
Thanks

Hi @AlanNggg, when you provide criteria in a where() it takes precedence over the criteria in the find(). You can provide a where clause in the find() by passing it in a dictionary e.g. SupplierShowcase.find({ supplierId: "f91750ab-11b5-425d-89fa-5a4ad4a23eb9", where: {createdAt: { ">":<PHONE_NUMBER>485 } }}) or you can provide all the criteria in the where() clause.
We'll update the docs to reflect this usage.

thx


Correct way to make new api calls?

Hey Guys,
A bit of a node N00B and trying to set up a simple carrier service  that will provide different shipping rates to our wholesale customers.
To accomplish this, I need to loop through the items in the cart and check to see if see if their tags array contains a specific tag. I've been really struggling with making API calls within a new post route I've set up, mainly because I'm not sure where to grab the access token and it's not super clear in the docs (or maybe just my n00bness).
What's the correct way to set up a backend API properly call using the shop credentials? Currently doing something like this, but both shopDomain and accessToken are undefined. Have also tried different flavors of grabbing it from the session without luck. Any help would be appreciated!
request.session.accessToken
req.session.accessToken
app.post('/rates', jsonParser, function (request, res) {

  const shopify = new ShopifyAPIClient({ shopName: shopDomain, accessToken: accessToken });
  shopify.product.get(1234)

  // Do more stufff

});

Ended up going the private route, but still curious on the correct way to do this.

I've been looking for a similar answer for a while. Let me know if you find out something! I can get it hitting my custom URL for rates, but it never renders the response of my api as valid rates to choose from...

@Avidexp Did you register the carrier service first?
https://help.shopify.com/en/api/reference/shipping_and_fulfillment/carrierservice#create

Yes sir I did,
I ended up getting it to work. I had my rates in just an array, not an
array wrapped in an object.
Ex.
[{
'service_name': 'Globaltranz Fast',
'description': 'fastest rate',
'service_code': 'GTZ',
'total_price': '100000.80',
'currency': 'USD',
'min_delivery_date': '2018-07-28 14:48:45 -0400',
'max_delivery_date': '2018-07-30 14:48:45 -0400'
}
rather than
{
"rates":[{
'service_name': 'Globaltranz Fast',
'description': 'fastest rate',
'service_code': 'GTZ',
'total_price': '100000.80',
'currency': 'USD',
'min_delivery_date': '2018-07-28 14:48:45 -0400',
'max_delivery_date': '2018-07-30 14:48:45 -0400'
}
}
On Thu, Jul 26, 2018 at 12:45 PM, Philip Johnson<EMAIL_ADDRESS>wrote:

@Avidexp https://github.com/Avidexp Did you register the carrier
service first?
https://help.shopify.com/en/api/reference/shipping_and_
fulfillment/carrierservice#create
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Shopify/shopify-node-app/issues/123#issuecomment-408213443,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARVdLUQWZxMCclE25KPn9llCBLsPJE3Jks5uKhxDgaJpZM4U1Q5L
.

Ah the little things : ) Did you by chance use this library? Curious to see what your code looks like. I ended up writing mine with the node express library instead as a private app vs a public one.

So I actually made a public one doing the same thing with express lol. I found it was easier to eliminate some of the noise, and just try to build it out one step at a time, rather than using the template.
I can create/edit/delete carrier services no problem now, and I can pass back static rate information that gets procesed by shopify so the user can pick the rate. The only problem I'm having now, is I cant find the shopping cart information in the requests I get from shopify lol.
I'm using simple express API like:
app.get('/shopify/callback', (req, res) => {}
Yet req.body always comes out empty. Have you experienced this as well?

I can't quite remember now, but I can show you what I'm doing to loop through the cart items. I'd recommend using bodyParser.
...
// Set vars and imports etc...
...

const bodyParser = require('body-parser');

// Create application/json parser
var jsonParser = bodyParser.json()

// Shipping Carrier Service
app.post('/rates', jsonParser, function (req, res) {

...
/// rates logic here
...

  // Save rates request
  var ratesReq = req.body;
  var products = ratesReq['rate']['items']

});

bodyParser!!
Can't believe I forgot that! You're amazing pjatx, thanks man!!

No worries man! Glad I could be helpful. I beat my head on this project for a long time : )

Closing because this project is now deprecated.


V1.4patch 3

fixing nodeJS download path

forcing to install RTL and BTC-RPC-Explrorer with admin https://github.com/rootzoll/raspiblitz/issues/929

skipping Loop during provision https://github.com/rootzoll/raspiblitz/issues/454

[ ] test if provision runs without errors with all services on

Electrs is waiting for the bitcoind.service to run so the provision gets stopped

Provision now restores everything without stopping, but takes a long time ~1 h to compile everything (expected with RTL, BTC-RPC-Explorer and ElectRS).
Running another round of test starting from the SDcard build.

OK, last test is successful.
RTL, BTC-RPC-Explorer, ElectRS and Loop restores correctly with a new SD.
Should be good for 1.4 RC1.


Support relative resolved paths and uri-style assets

The first commit updates the regular expression usdmanager uses to identify references. Previously it did not recognize uri style paths, which are supported by under ArResolver2.0.
The second commit updates the ArResolver2.0 code path to use the source as an anchor. This allows relative references to work
with non-filename asset references. Usd Manager has some internal path manipulation to allow relative paths to work with plain file paths, but it needs to use the resolver plugin for non file paths. This logic already exists when dealing with the ArResolver1.0 API, but when the 2.0 support was added this was overlooked.

Could you sign the contributor agreement and email that to<EMAIL_ADDRESS>Your contribution is very much appreciated!


Temp variables

https://github.com/SkriptLang/Skript/issues/1963

I encounter the same problem, it is a major bug, I hope it will be solved soon


Python 3.7 support

When installing in a fresh conda created python 3.7 environment, I get the following error:
(base) C:>conda create -n py37 python=3.7 numpy cython
...
(base) C:>conda activate py37
(py37) C:>pip install pyansys
Collecting pyansys
... (removed the other dependency lines that were successful)
Collecting vtk (from vtkInterface>=0.11.0->pyansys)
Could not find a version that satisfies the requirement vtk (from vtkInterface>=0.11.0->pyansys) (from versions: )
No matching distribution found for vtk (from vtkInterface>=0.11.0->pyansys)
Note: the same procedure for python 3.6 works fine.

Try installing vtk from anaconda.  There’s no PyPi vtk wheel for Python 3.7 yet, so I’m unable to build it for 3.7.  This will change as soon as they release a wheel for vtk.

I'm still waiting for vtk to release a wheel for Python 3.7 before I can support this.  I've contacted them to see when they plan in releasing it.  For now, I won't be able to support Python 3.7 since I have no way of testing it.

Looks like it will be at least December 2018 or March 2019.  See:
https://gitlab.kitware.com/vtk/vtk/issues/17407#note_466287

VTK‑8.1.1‑cp37‑cp37m‑win_amd64.whl
here , is a wheel file that you are looking to install VTK . i faced the same problem and i solved with the same file . installl the vtk .whl as i given , your problem will be solved .
cheers . Good luck.

VTK 8.1.2 is out on pypi with python3.7 support.  I'm going to update vtki and compile pyansys for python 3.7 this week.

pyansys now supports python3.7 for Windows and Linux as of 0.33.1.  If anyone out there would like to contribute a solution for building Mac OS on Travis-CI, please submit a push request.

Hello,
Why simply conda install pyansys does not work?

pyansys isn't a conda package (yet).  pip install pyansys should work for now.


plot_bug_fix

exclude void path

anyway maybe we have to rethink to plot the void traces correctly in the future


requesting to add HTL Perg

Dear @philipto I m sorry last time I accidentally forked the repo from leereilly/swot.
I hope this time everything there shouldnt be any problems.
If there is anything wrong dont hesitate and contact me.
Thanks in advance!
Official website of the school: https://htl-perg.ac.at/
Proof that the domain htl-perg.ac.at is used for mail addresses https://htl-perg.ac.at/organisation/lehrerteam

@ttimehhc Yes, you are right. everything is fine now. Pull request merged. Thank you!


k8s 1.30 Support for rancher-logging

Add k8s 1.309 Support for rancher-logging

Are we blocked on testing the chart?

I think this still has a slight blocker in that Rancher only has experimental provisioning of 1.29. So we cannot create 1.30 downstreams like I am for testing on 1.29; so I may just need to test the chart installing onto a "local" 1.30 rancher cluster.
I will start that progress once I've sorted out the current issues found on 1.29. As that issue seems it would apply to both 1.29 and 1.30 anyway.

Pass Verified in rancher v2.9.0-alpha5. Below are the test cases.
Local Cluster

Provisioned Rancher with local k8s 1.29.4+rke2r1cluster.
Installed rancher-logging:104.0.0-rc1+up4.4.0
Setup splunk server to receive logs.
Configured Rancher to send logs on splunk server and Verify logs are being received in splunk.
Upgraded Local cluster to k8s v1.30.1+rke2r1
Verified configured logging still sending logs to splunk server.
Logging passed p0 test cases.
Logging components like services, deployment and pods are up and running.
Chart annotations are correct.
Uninstall Logging app successfully on local cluster and deleted respective namespaces.
Fresh install Logging app on K8s v1.30.1+rke2r1
Configured Rancher to send logs on splunk server and Verify logs are being received in splunk.

Downstream Cluster

Provisioned Rancher with downstream k8s v1.28 cluster.
Installed rancher-logging:104.0.0-rc1+up4.4.0
Setup splunk server to receive logs.
Configured Rancher to send logs on splunk server and Verify logs are being received in splunk.
Upgraded downstream cluster to k8s V1.30 EKS
Verified configured monitoring still sending alerts.
Logging passed p0 test cases.
Logging components like services, deployment and pods are up and running.
Chart annotations are correct.
Uninstall Logging app successfully on local cluster and deleted respective namespaces.
Fresh install Logging app on K8s V1.30 EKS
Configured Rancher to send logs on splunk server and Verify logs are being received in splunk.


XSD Circular/Cyclic Reference

Hi,
I'm dealing with WSDL with circular/cyclic reference, e.g. like below
file.wsdl
    xs:import namespace="A" schemaLocation: A.xsd
A.xsd
    xs:import namespace="B" schemaLocation: B.xsd
B.xsd
    xs:import namespace="A" schemaLocation: A.xsd

I noticed the lib/wsdl.js will perform infinite loop when hitting circular reference.
Any suggestion or thought?

I too have noticed this behavior, and would lean toward classifying it as a valid bug.  The specs do not define circular references as being an error see w3.org/xsd.

I am having the same issue.

I came across this today, tried various ways to get around it, the only way to do this was to check to see if the "parent" was loading, aka, has it loaded previously, returning it here worked fine
I suggest we should have an private method that can be called internal to the WSDL, which can then do this logic?
var loading = { };

function open_wsdl(uri, options, callback) {
  var parent;

  if (typeof options === 'function') {
    callback = options;
    options = {};
  }

  if ( parent = loading[ uri ] ) {
    return callback.call( parent, null, parent )
  }

  // .....
}

Thoughs?


Sample Types: Add RootMaterialRowId

Rationale
This introduces a new RootMaterialRowId column into the exp.material table which emulates the RootMaterialLSID column. This column identifies the root material for a sample which is useful for derivates and aliquots. Additionally, in materialized Sample Type tables the RowId for the sample is now replicated into the materialized table. When joining to materialized Sample Type tables RootMaterialRowId is now used instead of RootMaterialLSID which has been shown to result in moderate to significant gain in query execution times for Sample Types with a large number of samples.
Related Pull Requests

https://github.com/LabKey/platform/pull/4844
https://github.com/LabKey/labkey-ui-components/pull/1324
https://github.com/LabKey/labkey-ui-premium/pull/230
https://github.com/LabKey/biologics/pull/2464
https://github.com/LabKey/sampleManagement/pull/2182
https://github.com/LabKey/inventory/pull/1070

Changes

Upgrade SQL scripts to add RootMaterialRowId column to exp.material table.
Upgrade code to add RowId to materialized Sample Type tables.
Refactor SampleTypeService.recomputeSampleTypeAvailableAliquotRollup into ExperimentUpgradeCode as it is only utilized during upgrade. Remove from service interface.
Update XAR export/import to translate rowId to LSID and back again for RootMaterialRowId.
Update aliquot recalculation to utilize RowId instead of LSID.
Remove RootMaterialLSID.
Bump @labkey packages

Are you working on removing rootmateriallsid from XarReader / XarExporter? Not sure what the status of manual testing is at, but should cover folder import that contains aliquots and subaliquots, including from older and new folder archives

Yes, I'm addressing that TODO. It will not be fully removed because we need to maintain the rootMaterialLSID in the XAR and translate from rowId and back.

Are you working on removing rootmateriallsid from XarReader / XarExporter? Not sure what the status of manual testing is at, but should cover folder import that contains aliquots and subaliquots, including from older and new folder archives

Yes, I'm addressing that TODO. It will not be fully removed because we need to maintain the rootMaterialLSID in the XAR and translate from rowId and back.

@XingY Alright, I've taken a look at the XAR export/import and it appears to be doing the round-trip successfully.


SSH with putty only works sometimes

Hi
This may be caused by my impenitence when it comes to using smartcards in windows. But it the majority of times i use WinCryptSSHAgent with putty it doesnt work. I get this prompt:

The error message bellow shows up when i click ok. But when i click "Cancel" it works like 10% of the time. The other times i get the error "Pageant failed to provide a signature". Is my yubikey conflicting with some other device? I can't seem to figure it out. Any tips?

Hello, it seems there is a software conflict. It turns out some software conflicts with WinCryptSSHAgent. For example: Pageant, OpenPGP, PuTTY-CAC's agent..... Do you have any other software using smartcards?

This is what i can see in my device manager. I shouldn't have any other smartcard-devices or software. Not that im aware at least.

Is it supposed to be two devices?

It's only one device. The smartcard reader is a virtual device associated with your smartcard.

Could it be caused by that i've installed gnupg with putty support? I guess that should create a new smartcard, but could it conflict in some way?

Bingo! Gnupg conflicts with windows smartcard driver, because it use a different smartcard protocol.

I uninstalled gnupg and added that seems to have solved the issue. Thank you for the clarification!


make tiflash scale-in faster

What problem does this PR solve?
Issue Number: close #8433
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests


Feature/keyboard change pan handling on hide

Summary
Describe the goal of this PR. Mention any related Issue numbers.
Requirements (place an x in each [ ])

[ ] I've read and understood the Contributing Guidelines and have done my best effort to follow them.

[ ] I've read and agree to the Code of Conduct.

[ ] I've written tests to cover the new code and functionality included in this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: haiderashfaq:x: TheiOSDudeYou have signed the CLA already but the status is still pending? Let us recheck it.


Slider

### Tasks
- [ ] #2450
- [ ] #2452
- [ ] #2451

Zazu Goal: merge to RC, a known state, stakeholder review required

design review today

Pushed fixes following design feedback on Monday, waiting on sign-off from @bhoppers2008
Outstanding bug with event-listener not being removed on un-mount. Paused for now to focus on #2886
New (?) bug for Salt site examples: examples overflow box on medium/small screen widths - will raise separate ticket

Design sign-off from Nav ✅
Still need to address event-listener bug, but otherwise ready for dev review (it's a big PR)

Goal: push to core by EOS

Addressing memory-leak bug with event-handling refactor.

Refactoring event handling to cut out some nasty edge-cases and weird behavioural quirks. Expecting dev work to finish early next week

Goal: by end of 22/07/24 PR up and seeking reviewing  during Cappuccino,
Chance for release to Core if we can get review

26/07 - design review carried out on storybook PR + site docs. A few interaction issues noticed with the range with input example. Review summarised in Design QA page in figma.

Focus and tab issues flagged by @navkaur76 . Fixed and waiting for design review.

Design issues addressed.
Working through dev feedback.

Cortado Goal:
PR Review and merged to Labs (wk1)
Identify some third-party feedback before merge to Core

Changes made. Awaiting dev re-review

Aug 12 - PR ready, 1 more review, aim for stakeholder review this week

Still waiting on second review.
No luck finding stakeholders to review so far.

Aug 16 - Could be good to merge to lab, leave there until a stakeholder can review

Closing this issue in favour of  https://github.com/jpmorganchase/salt-ds/issues/4018


Add files via upload

@Larmbs Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Added Animated File Upload Form

Fixes Issue🛠
Closes #391
Description👨💻
Added a File Upload Form that allows darg-and-drop and also upload via button features, allows multiple fileuploads with animations
Type of change📄

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
Tested on my local machine, works and looks as intended.
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Video Demo 📷
https://github.com/Rakesh9100/Beautiify/assets/86418216/e02c6910-ad61-4a8b-8823-fb8583b4ef11

Merged ✨ @SoumyadevSaha


why there is no ubuntu 22.04 benchmark results

Hi,
I was wondering why there is no ubuntu 22.04 benchmark results?
[Enter feedback here]

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

ID: 35a85cae-4495-c2be-d2d1-63c845f332ae
Version Independent ID: 4a36bbb6-e431-f210-25ec-27cffbc658f0
Content: Azure Kubernetes Service (AKS) Ubuntu image alignment with Center for Internet Security (CIS) benchmark - Azure Kubernetes Service
Content Source: articles/aks/cis-ubuntu.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

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

Hi @nfsouzaj
The reason why there are no Ubuntu 22.04 benchmark results is that the CIS Ubuntu 18.04 LTS Benchmark v2.1.0 is the latest version of the CIS benchmark for Ubuntu. The CIS benchmarks are updated periodically, and the latest version of the benchmark may not always be available for the latest version of the operating system. However, the security configurations applied to the Ubuntu 18.04 LTS image used by Azure Kubernetes Service (AKS) are designed to reduce the surface area of attack and optimize for the deployment of containers in a secure manner.

@nfsouzaj
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation

Hi, I would like information on when there will be a CIS Benchmark for the newer 22.04 version. Thank you


adapted webspace xml dist file

Q
A

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
no

Fixed tickets
none

Related issues/PRs
https://github.com/sulu/sulu/pull/2665

License
MIT

Documentation PR
none

What's in this PR?
This PR corresponds to the change in https://github.com/sulu/sulu/pull/2665.
Why?
Due to the latest changes the standard configuration here does not work anymore.

ACK


Weights not getting detected

After installing everything needed I come across and error where I get told:
(venv) D:\ai-stuff\rvc-tts-webui>app.py
Traceback (most recent call last):
File "D:\ai-stuff\rvc-tts-webui\app.py", line 35, in
raise ValueError("No model found in weights folder")
ValueError: No model found in weights folder
(venv) D:\ai-stuff\rvc-tts-webui>
Inside my weights folder there is only 2 .pth files but after manually changing the location to my folder with 20 of them they still don't get detected

I have reread the github repo and found out what my issue is


[Windows] m-switch-to-ee with 2gb might be failed during magento installation step

Steps to reproduce:

Set memory to 2048.
Run m-switch-to-ee, wait while files are synchronized, press any key to resume magento installation process.

Expected: Magento EE installed successfully.
Actual:  m-switch-to-ee fails with killed process. (not always reproducible)
Workaround: run "vagrant reload"

@arkadiych Will you be able to fix this? I cannot reproduce it on OSX


fix(github): fetchFiles should return files

fixed lost return in fetchFiles function

Resolves: #223

Nice! Thanks! 👍


[Visual Studio 2017] How to fix MaterialSkin [C# FORM]

This is a way to fix your MaterialSkin in Visual Studio 2017

If you chose C# (Form1.cs) then double click on the form background.
Go and add using MaterialSkin; to the first line & using System; to the second line if you haven't already.
Go and change public partial class Form1 : Form to public partial class Form1 : MaterialSkin.Controls.MaterialForm
Go into public form1, and add these:           var materialSkinManager = MaterialSkin.MaterialSkinManager.Instance;          materialSkinManager.AddFormToManage(this);          materialSkinManager.Theme = MaterialSkin.MaterialSkinManager.Themes.DARK;          materialSkinManager.ColorScheme = new ColorScheme(Primary.BlueGrey800, Primary.BlueGrey900, Primary.BlueGrey500, Accent.LightBlue200, TextShade.WHITE); You may edit this later.

That's it! Your done! If you are still having any issues, please tell me and I will try to find a solution to your problem.

@IncVERM
https://github.com/IgnaceMaes/MaterialSkin#implementing-materialskin-in-your-application

@IncVERM facepalm
--


Fix: SwipeAction Closure Type from RowType to Item

Fixed the closure types of the following methods:

leadingSwipeAction
trailingSwipeAction

Old:
(RowType, IndexPath) -> UISwipeActionsConfiguration?
New:
(Item, IndexPath) -> UISwipeActionsConfiguration?

Thank you!


Add option to get the spam score and spam report from sendgrid on parse

I've added the possibility to get the spam score for sendgrid. To filter out spam or not in the easy way. You can use the filter when parsing the message like this:
Mailbox::catchAll(static function(InboundEmail $email) {
      echo $email->spamScore();
      echo $email->spamReport();
});

I agree that it should support spam indication but it should be made more generic to be supported on other drivers.
I'm not convinced by the implementation (adding parameters to fromMessage). Unfortunately, I'm not too familiar with the implementation of this package so I can't provide much guidance yet, sorry.

Yeah, I agree with @julienbourdeau
Maybe it would be better to be able to pass in the complete raw request into the inbound email class.

@mpociot I've been staring at the code for a couple of hours by now, but I can't figure out how I should be able to make a decent integration with the (specific) sendgrid spam rates...


Return value of AG\ElasticApmLaravel\ServiceProvider::getAppConfig() must be of the type array, null returned

Hello,  i am facing this issue when installing the plugin.
the problem seems to be here
at C:\Users\Barone\git\safariremit\vendor\arkaitzgarro\elastic-apm-laravel\src\ServiceProvider.php:231
227▕         if ($this->app->bound(VersionResolver::class)) {
228▕             $config['appVersion'] = $this->app->make(VersionResolver::class)->getVersion();
229▕         }
230▕
➜ 231▕         return $config;
232▕     }
can you help me with this ?

Please provide:

php version
laravel version
full error message (wrapped in code blocks)

hello,
i have performed the command you proposed, and it seems to be working well
7.4.3 is the version of php in use (result of the command php -v )
thanks for the support
regards
i
On Thu, 7 Oct 2021 at 09:08, Adrian @.***> wrote:

Can you try composer require arkaitzgarro/elastic-apm-laravel --no-scripts
? Also when you write PHP Version => 7.4.3 does that mean you use PHP
7.4.3 or e.g. 8.0.7?
Probably not related, but you should also look at that warning in the
output above: Warning: Ambiguous class resolution, "App\Console\Kernel"
was found in both "C:/Users/Barone/git/safariremit/app\Console\Kernel.php"
and "C:/Users/Barone/git/safariremit/app\Kernel.php", the first will be
used.. Again, probably not related but weird.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arkaitzgarro/elastic-apm-laravel/issues/150#issuecomment-937515455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBPUF6LPLY2UON7YUXHHFTUFVBPPANCNFSM5FHKIYTQ
.

--
Ivo Barone


Extending Program for Video Frame Mode

Dear tzutalin,
I use your program for neural network training. At first, thank you very much.
However I missed a feature I desired. As I am analyzing Video Streams, I have to label countless video frames, that just look alike. So basically I extended your Program by a Video Frame Feature.
Now you can set a current base, which simply takes the currently shown labels in a memory state. Afterwards you can apply them to another frame. Since most (or at least some) neural networks do something like a regression to the picture or labeledges, I added a randomization feature, too. This allows to paste the labels from the base inlcuding the expected error humans would do.
I hope you like the extension and include it to your project.
Best, jochengietzen

Great !!!

This should be considered to be merged, i need this too @tzutalin please check PRs some of them are just perfect

This should be considered to be merged, i need this too @tzutalin please check PRs some of them are just perfect