Command: toggle_tabs

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1197500-command-toggle_tabs?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github).

I'm pretty sure this is just a setting so the command wouldn't be much more than Settings().Set("show_tabs", arg) or similar.

Migrated to limetext/lime-backend#41.


Level is not being read from config for custom handler

I am using Monolog version 1 with Symfony.
I'm not sure that this should be asked here or in the Symfony repo but will ask here first.
I have created a custom handler to push logs to AWS Firehose. Here is the constructor:
class FirehoseLogHandler extends AbstractProcessingHandler {

public function __construct(
        FirehoseClient $firehoseClient,
        FormatterInterface $formatter,
        $streamName,
        $level = Logger::INFO,
        $bubble = true
    ) {

        $this->firehoseClient = $firehoseClient;
        $this->streamName = $streamName;
        $this->formatter = $formatter;

        parent::__construct($level, $bubble);
    }

And here is my monolog.yaml config:
monolog:
  handlers:
    firehose_handler:
      type: service
      id: kinesis_stream_handler

    main:
      type: stream
      handler: firehose_handler
      level: error
      channels: ["!event"]

services:
  kinesis_stream_handler:
    class: App\Logger\Handler\FirehoseLogHandler
    arguments:
      $firehoseClient<EMAIL_ADDRESS>      $formatter: '@App\Logger\Formatter\FirehoseLogFormatter'
      $streamName: 'firehose-stream-test'

The problem that I am having is that the $level is always set to Logger::INFO (200), or whatever is set in the constructor. It seems to be ignoring what is set in the yaml config

Hi!
I think it happens because you do not pass the level parameter to firehose_handler.
As Symfony docs say:

This defines a stack of handlers and each handler is called in the order that it's defined.

But I can be wrong with this. Try to add level: error:
monolog:
  handlers:
    firehose_handler:
      type: service
      id: kinesis_stream_handler
      level: error


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2023-03-16T18:25:23+00:00

/retest


Azk init does not recognize Python applications

When run azk init in python application folder Django or Pyramid applications,  azk don't generate correct Azkfile.js.

Hello Mario,
For while the rules and suggestions offered by azk init command are within azk code, thus it is necessary to make the changes that will be available in the next version. Soon, they will also be available in the form of plugins, which will make updates easier to perform (as well as making contributions).
I'm assina the link for rules and suggestions file so that whoever improves this part, has a point to start:
Python:

Rules
Suggestion to v2.7
Suggestion to v3.4

Django:

Rules
Suggestion to v2.7
Suggestion to v3.4

Improvements:

[ ] Recognize * .py files.
eg:
setup.py of Pyramid
manage.py of Django

[ ] Identify dependencies.

[ ] Improve the suggestions of Provisions
    provision: [
      "pip install --upgrade --user --allow-all-external pip",
      "pip install --user --allow-all-external -r requirements.txt",
      // "python setup.py develop --prefix=/azk/pythonuserbase", // Pyramid
    ],

I'm also adding the Azkfile.js gist link that you used (with some changes):
https://gist.github.com/gullitmiranda/186ac28718a284b46f55


Formatting large number return inaccurate number

When I test my app by entering large number with decimal, it returns the wrong number. Like below:
numeral('999,999,999,999,999.68').format('0.00')
><PHONE_NUMBER>99999.63

The number you are trying to parse is too large and it doesnt fit the last decimal place. Javascript has 64bit numbers..

The number you are trying to parse is too large and it doesnt fit the last decimal place. Javascript has 64bit numbers..

Just wanna put this here so that other people might face with the same issue.
https://stackoverflow.com/questions/54800022/why-max-digits-with-decimal-in-javascript-are-only-16#answer-54803254
Thanks @fogine , I'm gonna close this issue.

Just wanna put this here so that other people might face with the same issue.
https://stackoverflow.com/questions/54800022/why-max-digits-with-decimal-in-javascript-are-only-16#answer-54803254
Thanks @fogine , I'm gonna close this issue.


[BACKLOG-34475] Updated PDI CE and EE splash background image (9.1)

:white_check_mark: Build finished in 19m 4s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1148, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


loadPaths does not work

Hi kentaroi,
I love your plugin.
However, I encountered an issue after moving my scss folder outside of my eleventy input folder.
Here is my eleventy directories config in .eleventy.js:
...
dir: {
input: "src/pages",
data: "../_data",
includes: "../_includes",
layouts: "../_layouts",
output: "_site",
}
So, my eleventy input folder is src/pages and my scss code is in src/scss.
I used the loadPaths option in my eleventy-sass plugin settings to specify the new scss location, and tried all possible symbol arrangements, one by one, and none works:
loadPaths: ["/src/scss/"],
loadPaths: ["/src/scss"],
loadPaths: ["/src/scss"],
loadPaths: ["src/scss/"],
loadPaths: ["src/scss"],
loadPaths: ["./src/scss/"],
loadPaths: ["./src/scss"],
Please note that the option includes: "../_includes", in my eleventy-sass plugin settings works as expected when I move my scss folder back to the eleventy input folder.
I would appreciate your help here.
Cheers!

eleventy-sass makes use of  Eleventy's file watching functionality.
Therefore, your Sass/SCSS files should be inside your project's input directory.
But, first of all, is there any reason why your directories config should not be like this:
return { dir: { input: "src", layouts: "_layouts" } };

Hi kentaroi,
Thanks for the quick reply and clarification.
My new folder arrangement allows for opening files in fewer nested folders during development.
Of course, there is no problem having the scss folder inside my eleventy input dir.
Best,

In your directory config, you wrote ../_data, ../_includes and ../layouts, and they might actually work well, but they seem not to be valid options, because the official doc says

Valid Options | Any valid directory inside of dir.input

Configuration options
Please correct me if I'm wrong.
Your directory structure seems to be something like this:
├── .eleventy.js
├── _site
└── src
    ├── _data
    ├── _includes
    ├── _layouts
    ├── pages
    │   └──  your-page.md
    └── scss
        └── style.scss

In that case, your directories config can be written like this:
return { dir: { input: "src", layouts: "_layouts" } };

, because output, data and includes directories are the defaults, if you set src to input property.
I recommend you to put your contents (markdown files) in src directory and its subdirectories instead of in src/pages directory and its subdirectories.
However, if you have any reason to place them in src/pages directory, you can use permalinks.
For example, add  a permalink setting to src/pages/pages.json like this:
{
  "permalink": "/{{ page.fileSlug }}/index.html"
}

The result will be the follows:
├── _site
│    └── your-page
│        └── index.html
└── src
    ├── _data
    ├── _includes
    ├── _layouts
    ├── pages
    │   └──  your-page.md
    └── scss
        └── style.scss

By the way, loadPaths is NOT for teaching Eleventy where your Sass/SCSS files are, but for informing dart-sass library how to load other Sass/SCSS files from a Sass/SCSS file by using @use, @forward or @import rules.
Again, your Sass/SCSS files should be in your input directory or its subdirectories to make Eleventy compile them.
This is how this plugin works.
If you want to make dart-sass library work independently, I recommend you to use eleventy-plugin-dart-sass, for example.

If you want to stick with your directories config, you can use addWatchTarget() method in your .eleventy.js to watch files outside of your input directory.

Hi kentaroi,
Many thanks for further clarifications, particularly that

loadPaths is NOT for teaching Eleventy where your Sass/SCSS files are, but for informing dart-sass library how to load other Sass/SCSS files from a Sass/SCSS file by using @use, @forward or @import rules

because I initially thought loadPaths was doing just that. Perhaps you could add this explanation to the plugin's readme section about the loadPaths.
Anyway, I can easily solve my folder configuration with the permalink rule.
Best regards

I'm glad to hear it 😄

Perhaps you could add this explanation to the plugin's readme section about the loadPaths.

Thanks. I will.


How do I install boost lib compiled with /guard:cf & /Qspectre flags

I added the following settings in my triplet files and hope b2 can use them when compiling boost lib. But the boost lib didn't pass the BinSkim check. How can I fix this?
set(VCPKG_CXX_FLAGS "/Qspectre /guard:cf")
set(VCPKG_LINKER_FLAGS "/guard:cf /DYNAMICBASE")
set(VCPKG_C_FLAGS "/Qspectre /guard:cf")
some logs in install-x64-windows-rel-out.log. it seems the flags in the jam file not correct.
notice: Searching 'xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel' for user-config configuration file 'user-config.jam'.
notice: Loading user-config configuration file 'user-config.jam' from 'C:/git/CMD-NativeThinClients/tools/vcpkg/buildtrees/boost-atomic/x64-windows-rel'.
xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel\user-config.jam:6: Unescaped special character in argument /guard:cf
xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel\user-config.jam:7: Unescaped special character in argument /guard:cf
xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel\user-config.jam:8: Unescaped special character in argument /machine:x64
xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel\user-config.jam:8: Unescaped special character in argument /INCREMENTAL:NO
xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel\user-config.jam:8: Unescaped special character in argument /OPT:REF
xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel\user-config.jam:8: Unescaped special character in argument /OPT:ICF
xxxx\tools\vcpkg\buildtrees\boost-atomic\x64-windows-rel\user-config.jam:8: Unescaped special character in argument /guard:cf

@mattfjh, this issue would be fixed by PR https://github.com/microsoft/vcpkg/pull/23001, could you help test if the PR works for you? the 2 flags are expected to apply correct with the changes.

Double confirmed, PR #23001 has fixed this issue.

The generated parameters doesn't seem right. should be /guard:cf /Qspectre

@mattfjh
That is intended, the compilar command-line syntax accepts both -option and /option.
https://github.com/microsoft/vcpkg/blob/c7366f8416df3b31d518d9e42aded06673bebf95/ports/vcpkg-cmake/cmake_get_vars/CMakeLists.txt#L142-L150

thanks @vicroms . I used BinSkim to scan the dll and got this error. Any idea what else could be wrong?

Ah, I think the issue happens because the linker flag is not used -  I have the following linker flag in the triplets file
set(VCPKG_LINKER_FLAGS "/GUARD:CF /DYNAMICBASE")

The flag may need to be applied to all boost dependency components, not just the boost-atomic.

Can confirm with latest commit https://github.com/microsoft/vcpkg/pull/23001/commits/74bc293ee0bfca2fb3b80f8714ccb476f07ef2fb that binskim doesn't complain about linker flags anymore.
However, I'm still not sure how to make vcpkg use a spectre mitigated runtime.


Improve coverage of errors.go

What are you thinking for testing here?
No need to add tests randomly to just improve coverage. What is beeing tested should have some value from a verification standpoint..

No need to add tests randomly to just improve coverage.

I, at least partially, disagree because a substantial amount of missing coverage is caused by randomly looking files. But you're right. This issue is too small and not worth mentioning. I initially created it because I wanted to prevent that somebody else was working on the same file to improve coverage and we do duplicate work. In the end I modified more files.

A better idea might be to test the scenarios/paths where the errors are used instead of only the errors directly in isolation.


tapping homebrew/science stopped working as of 12/22/2017

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.
Please always follow these steps:

[x ] Confirmed this is a problem with running a brew command and not brew installing or the post-install behaviour of one or more formulae? If it's a formulae-specific problem please file this issue at the relevant tap e.g. for Homebrew/homebrew-core https://github.com/Homebrew/homebrew-core/issues/new
[ x] Ran brew update and retried your prior step?
[ x] Ran brew doctor, fixed all issues and retried your prior step?
[x ] Ran brew config and brew doctor and included their output with your issue?

HOMEBREW_VERSION: 1.4.1-11-g583b61f
ORIGIN: https://github.com/Homebrew/brew
HEAD: 583b61fd324643f15bb8017a6b652afbbbfe7cbc
Last commit: 16 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: f1907dea0890ef018d8a8c1a300ecb76a930c9be
Core tap last commit: 2 hours ago
HOMEBREW_PREFIX: /usr/local
CPU: octa-core 64-bit haswell
Homebrew Ruby: 2.3.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Clang: 9.0 build 900
Git: 2.14.3 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.54.0 => /usr/bin/curl
Perl: /usr/bin/perl
Python: /usr/local/opt/python/libexec/bin/python => /usr/local/Cellar/python/2.7.14/Frameworks/Python.framework/Versions/2.7/bin/python2.7
Ruby: /usr/bin/ruby => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Java: N/A
macOS: 10.13.2-x86_64
Xcode: 9.2
CLT: <IP_ADDRESS>.1.1510905681
X11: 2.7.11 => /opt/X11
deniz-mac:Homebrew deniz$ brew doctor
Your system is ready to brew.

To help us debug your issue please explain:

What you were trying to do (and why)
What happened (include command output)
What you expected to happen
Step-by-step reproduction instructions (by running brew commands)
As I am trying to have a fresh install of homebrew, I tried to tap homebrew/science and this happens:

deniz-mac:Homebrew deniz$ brew tap homebrew/science
==> Tapping homebrew/science
Cloning into '/usr/local/Homebrew/Library/Taps/homebrew/homebrew-science'...
remote: Counting objects: 625, done.
remote: Compressing objects: 100% (620/620), done.
remote: Total 625 (delta 0), reused 28 (delta 0), pack-reused 0
Receiving objects: 100% (625/625), 536.45 KiB | 13.41 MiB/s, done.
Error: Non-file alias<EMAIL_ADDRESS>Error: Cannot tap homebrew/science: invalid syntax in tap!

That tap is being deprecated.
On another note, when posting multiple lines of code please use fenced code blocks. Otherwise the output is hard to read and all mushed together.

updated the initial post with fenced code blocks, thank you!

Please file this issue on that tap, thanks.

fixed in https://github.com/Homebrew/homebrew-science/commit/c36b2e103c23fd9898d7f55a7c3113a2ddb6fcc1


How to migrate from angular 1 to angular 2

Hi,
Firstly thank you for this microservice template, It's a great example of microservice it helped me in understanding the microservice concepts easily. Is there any steps to upgrade to angular 2 or can I get any guidance from your end? because I am new to angular part.

@anitalpatil , based on my understanding Angular 2/4 is altogether different. If you want to migrate, I would suggest that you discard the current angular controllers etc and write your own Angular 2/4 code on it's behalf.


Cache the NCI password if needed

When the user needs to use rosie go with the au and test domains, as configured in ~/.metomi/rose.conf, it is convenient to use gpg-agent to cache the NCI password. Both mosrs-setup and mosrs-auth would then need to check and cache this password.

I have noticed that rosie go through gpg-agent puts up a PIN Entry that asks for the NCI password. I am not sure if this behaviour is what is wanted. I will keep working with the current mosrs-setup and rosie go to see whether it is really necessary to enhance mosrs-setup or whether the current behaviour of  rosie go is good enough.

Note that this behaviour of rosie go happens for my configuration, which uses the au and test  data sources via accessdev . Users that don't use accessdev should not see this behaviour. I will check this.

I have checked. If the rosie configuration in ~/.metomi/rose.conf  mentions the u source but does not mention the au or test sources, then rosie go does not complain, it just uses the configured u source. So I think that the behaviour of mosrs-setup is good enough.


Cannot write to /usr/local/share on macos-14 runner

Description
Actions are not being able to write to /usr/local/share in the new M1 runner under the macos-14 image.
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[X] macOS 14
[X] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link

Failed build attempting to write to /usr/local/share on macos-14: https://github.com/douglascamata/setup-docker-macos-action/actions/runs/7745920249/job/21122954463#step:3:1

Notice that the other runners, macos-12 and macos-13 all went fine:

Passing build on macos-13: https://github.com/douglascamata/setup-docker-macos-action/actions/runs/7745920249/job/21122954116
Passing build on macos-12: https://github.com/douglascamata/setup-docker-macos-action/actions/runs/7745920249/job/21122953846

This is the source workflow file: https://github.com/douglascamata/setup-docker-macos-action/blob/main/action.yml#L58
Is it regression?
Yes, broken on macos-14 and works on all other runners.
Expected behavior
I should be able to write to /usr/local/share on macos-14, like I can on macos-13 and macos-12.
Actual behavior
I can't.
Repro steps
In a macos-14, try to create folders and files under /usr/local/share.

@douglascamata, looks like macos-13 image has runner:admin permissions for /usr/local/share. But macos-14 has root:wheel.
So to workaround it you could try to run sudo chown runner:admin /usr/local/share before the command that fails in your script.

Hello @douglascamata!
I think this is the default behaviour and it can be reproduced locally.
I am closing the issue. Please let us know if you have any questions or concerns


Extension issuejson  { 	"messages": [], 	"activationTimes": { 		"codeLoadingTime": 59, 		"activateCallTime": 48, 		"activateResolvedTime": 319, 		"activationReason": { 			"startup": true, 			"extensionId": { 				"value": "vscode.git", 				"_lower": "vscode.git" 			}, 			"activationEvent": "*" 		} 	}, 	"runtimeErrors": [], 	"runningLocation": { 		"affinity": 0, 		"kind": 1 	} }

Issue Type: Bug
Extension Name: git
Extension Version: 1.0.0
OS Version: Windows_NT x64 10.0.19044
VS Code version: 1.71.2

:warning: We have written the needed data into your clipboard. Please paste! :warning:

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


DABS - Monthly Compliance Updates and Support (FY25 Q2)

Compliance App
DABS Commission App
Update Sheet
MAT Address Problems Sheet
Benefit
The web applications benefit both the DABS compliance staff to plan licensee visits and answer other spatial questions about licensees; and better enables the DABS commission to make decisions about license applications during their review meetings.
Acceptance Criteria

Provide monthly license updates to the interactive Compliance app.
Support for DABS' use of OpenGov software, including the mostly manual ETL of a master address table (MAT) to work with the OpenGov system

Notes

The monthly updates from DABS are often provided a month or more behind the current calendar month.

Sprint 2 Tasks

[ ] 09/2024 update complete

currently waiting on 10/2024 updates from DABS

Sprint 5 Tasks

[ ] 10/2024 update complete

Sprint 6 Tasks

[ ] 11/2024 update complete

Waiting to hear if the November updates are completed in the update sheet


Manage question tags

Among the currently available question attributes, "tags" are missing. In XML, tags have the format (as exported from Moodle):
  <question>
  (...)
      <tags>
        <tag><text>e1</text>
  </tag>
        <tag><text>e2</text>
  </tag>
      </tags>
  </question>

Tags are useful to manage questions and build quizzes with random questions. The inclusion of such a feature may be useful in many cases.

I will answer roughly as in #8 and #11 : adding functionalities takes time, so I won't add them for the sake of completeness, or if it "may be useful". I personally never used tags. Do you need it? It is not clear from your message.
Besides this, tags are a bit more complicated than the other keys. Most keys come as :
<key>
  <text> foo bar </text>
</key>

But tags is a nested key, a key with tag keys within. So it would require some decent amount of work to make it work and make sure that it doesn't break everything.
This is the main reason why I do not plan to implement it in the immediate future.

Hi. Actually, the only thing I proposed for the sake of completeness was the right alignment in #8. The rest of the things I really use them. In the case of tags, I ended up using them quite a lot, as they are a convenient way to manage question banks and reveal quite useful for preparing random questions. I actually use this feature (directly in Moodle) a lot. I thought that being able to work with this attribute externally could be useful.
Anyway, I agree that tags management is quite involved. If I end up with some decent code for this, I will share it back. Thanks again.


Bugfix 2397

为了解决issue2397
主要处理级连继承的范型问题

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wangkun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wangkun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

这个commit没被接受？还是有新的解决方案？


Rehaul how comments are attached

Opening this bug so we can dupe any existing bug about comments here. I will make a quick todo list, but these may not be individually fixed. Rewriting parts of the comment algorithm will probably fix many of them in one go.

[ ] Trailing comments on opening blocks are moved down (#271)
[ ] Comments not emitted in chaining calls (#200)
[ ] Comments stripped from empty functions/if/try/etc (basically any block) (#31, #87)
[ ] Trailing comments inside a function call should stay where they are (see various issues in #135)

There is also the checklist of comment issues on nuclide in #135. We should run through that list after my rehaul. I'll pull a few core bugs from there to here.

Dunno if the list covers them but you may want to add an entry for comments within try/catch blocks.

@n1k0 Same bug as #31 and #87. #207 is a fix (not the right one) for babylon.

Yep, that's the 3rd one where they are stripped away from empty blocks. It happens with any empty block, no matter what kind of keyword is before it.

This also happens on empty files, such as Rails' application.js:
// This is a manifest file that'll be compiled into application.js, which will include all the files
// listed below.
//
// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
// or any plugin's vendor/assets/javascripts directory can be referenced here using a relative path.
//
// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
// compiled file.
//
// Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
// about supported directives.
//

ends up as an empty file. I don't really care about that specific file, because it's empty, but this one has useful content that we would like to keep (libraries.js):

// 3rd Party libs
// This is optional (in case you have `I18n is not defined` error)
// If you want to put this line, you must put it BEFORE `i18n/translations`
//= require i18n
//
// This is a must
//= require i18n/translations
//= require pusher.min
//= require jquery.min
//= require jquery_ujs
//= require jquery-ui.min
//= require plugins.jquery
//= require momentum.jquery

This too is returned as the empty file.

@francois thanks for the report, this is actually not a known bug. Do you mind creating a separate issue so that we can track it down on its own instead of having it in a comment as part of a bigger one? Thanks!

Argh, didn't autoclose, see above PR


[FLINK-23429][state-processor-api] Use Path instead of Path.getPath()…

… to preserve FileSystem info
What is the purpose of the change
Fix an FileNotFoundException in State Processor API when working with savepoints on cloud storage
Brief change log

Use Path instead of a String Path.getPath() in StatePathExtractor to preserve FileSystem info such that FileCopyFunction can copy files even if they are on cloud storage.

Verifying this change
I've tested this fix with Azure blob storage and AWS S3.
This change added tests and can be verified as follows:

Manually create an S3 bucket or an Azure blob container
Configure a Flink cluster with s3-fs-presto or azure-fs-haoop plugin enabled
Start a Flink a job and create a savepoint on S3/azure
Use state processor API to transform the savepoint to a new one and store it in a new location on S3/azure
There should not FileNotFoundException

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers:no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Yes, we should back port to 1.13. Will create a new PR.


Add OID for RPKI id-ct-signedChecklist

Reference: https://datatracker.ietf.org/doc/html/draft-ietf-sidrops-rpki-rsc - Resource Public Key Infrastructure (RPKI) object profile for Signed Checklist (RSC)
OID listed under 'SMI Security for S/MIME CMS Content Type (1.2.840.1135<IP_ADDRESS>.1)' https://www.iana.org/assignments/smi-numbers/smi-numbers.xhtml#security-smime-1
LibreSSL also added this OID: https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib/libcrypto/objects/objects.txt.diff?r1=1.30&r2=1.31&f=h
CLA: Trivial

This has been submitted as trivial.  I'm okay with that.  Could someone else confirm too please?

Yes it is trivial.

Merged to master.  Thanks for this.


patches napari bug #1050

Images that are larger than 8000 pixels are automatically converted to a pyramid by napari and are not properly displayed:
https://github.com/napari/napari/issues/1050

Thanks for this @amedyukhina. Is this an issue you were seeing with cellfinder, or is this "just in case"?

My image was not displayed by cellfinder, turned out it was napari issue. I don't know how relevant this is in general: my images have quite high resolution in xy (0.875 um), whereas the default value is 2 um.

Cool, good to know. I haven't used cellfinder on data that high res since I added the napari viewer.
Thanks!


responsiveness

It is not reponsive if this is valid issue then please assign this issue to me

don't forget to add screen shot  on pr :)
and add issue number to pr

for sure thanks for assigning this task to me


[Bug]

Minecraft version
1.20.4
Mod version
1.2.2
Operating system
Windows
Describe the bug
Doesnt work on 1.20.4
To Reproduce
Launch minecraft fabric with the mod installed and gives a error
Expected behavior
Launch minecraft
Screenshots
No response
Additional context
No response

yeah im getting the same thing on 1.20.1

Sorry for the long wait, should be fixed in 1.2.4.
Downloads: 1.20.4, 1.20.1


Can't follow instructions to Install to Azure SQL Database

These links:

https://github.com/MicrosoftDocs/sql-docs/blob/live/docs/samples/adventureworks-install-configure.md#install-to-azure-sql-database
https://docs.microsoft.com/en-us/sql/samples/adventureworks-install-configure?view=sql-server-ver15&source=docs#install-to-azure-sql-database

Have instructions to install to Azure SQL Database that I can't follow:

If you do not yet have a SQL Server in Azure, navigate to the Azure portal and create a new SQL Database. In the process of creating a database, you will create a server. Make note of the server. See this tutorial to create a database in minutes.

Connect to your Azure portal.
Select Create a resource in the top left of the navigation pane.
Select Databases and then select SQL Database.
Fill in the requested information.
In the Select Source field, select Sample (AdventureWorksLT) to restore a backup of the latest AdventureWorksLT backup.
Select Create to create your new SQL Database, which is the restored copy of the AdventureWorksLT database.

The option to use an existing sample database doesn't seem to be available.

@paulomorgado -- Paulo, thank you for your feedback.  Are you trying to follow these instructions and use your own sample database?

@WilliamAntonRohm, no. As the instructions show, I was trying to create a sample AdventureWorksLT database and that option doesn't seem to be available anymore.
I can import any bacpac I have on any of my blobs, but that requires me to download the bacpac and upload to the blob.

@paulomorgado -- Paulo, please try the tutorial mentioned in your first comment, step 16, and let me know if that is applicable to this issue.

@WilliamAntonRohm, don't you mean https://docs.microsoft.com/en-us/azure/sql-database/sql-database-import?
Nevertheless, the feature to import a database from a set of sample databases provided by Microsoft doesn't seem to exist. Thus, the documentation should be corrected to point to a way for the user to get a sample database and import it.

@paulomorgado -- Paulo, thank you for clarifying.
@MashaMSFT -- Masha, please look into this issue.

@paulomorgado
My apologies, I'm not sure I'm following the conversation. From what I gather, you're trying to create a new Azure SQL Database using a sample database, but you're saying that the steps William pointed you to won't work for you? I'm not sure I understand, because when I look at the steps, I see that it's creating the AdventureWorksLT database, I've highlighted it here:

Would you mind clarifying what you're trying to do that is different than this step?
Thanks!
Masha from the SQL Docs team

@WilliamAntonRohm, no. As the instructions show, I was trying to create a sample AdventureWorksLT database and that option doesn't seem to be available anymore.
I can import any bacpac I have on any of my blobs, but that requires me to download the bacpac and upload to the blob.

@paulomorgado
My apologies, I'm not sure I'm following the conversation. From what I gather, you're trying to create a new Azure SQL Database using a sample database, but you're saying that the steps William pointed you to won't work for you? I'm not sure I understand, because when I look at the steps, I see that it's creating the AdventureWorksLT database, I've highlighted it here:

Would you mind clarifying what you're trying to do that is different than this step?
Thanks!
Masha from the SQL Docs team

My bad, @MashaMSFT. I missed that.
But that is not exactly what it says here (source):

Install to Azure SQL Database
If you do not yet have a SQL Server in Azure, navigate to the Azure portal and create a new SQL Database. In the process of creating a database, you will create a server. Make note of the server. See this tutorial to create a database in minutes.

Connect to your Azure portal.
Select Create a resource in the top left of the navigation pane.
Select Databases and then select SQL Database.
Fill in the requested information.
In the Select Source field, select Sample (AdventureWorksLT) to restore a backup of the latest AdventureWorksLT backup.
Select Create to create your new SQL Database, which is the restored copy of the AdventureWorksLT database.

Is it?

Hi @paulomorgado , ahh, I see, yes, this portion of the article was written before the new create database experience. Thanks for bringing this to our attention, I'll get this updated :)
Thanks!
Masha from the SQL Docs team

Hi @paulomorgado , thanks for your patience and my apologies for the delay :) this article has been significantly rewritten for clarity, and your issue should be addressed with the rewrite.
I hope that helps and please let us know if there's anything more we can assist you with? I'm going to close out this Git Issue but please comment should you need anything further.
Thanks again and hope you have a wonderful rest of your day!
Masha from the SQL Docs team


[FeatureRequest] Option to return ArrayBuffer

Since  Response.text of fetch is always decoded using UTF-8, I need to change some code for scraping non-unicode site. (Yeah, it still existing...)
I think providing option for it will be nice feature for this library. returning arraybuffer if option is provided. so I can use decode(py) or TextEncoder(js) for decoding characters properly.
CC: https://stackoverflow.com/questions/55282444/javascript-fetch-characters-with-encoding-issues

Interesting, to be honest I don't really know much about JS. Maybe you could submit a PR for this?
Thank you.


Minibatch Preprocessor DL: Add optional num_classes param.

The current minibatch_preprocessor_dl() module looks at the input
table to find the number of distinct categories (class values) for the
dependent variable, and uses that number as the size of the
one-hot-encoded array. This could lead to a failure in madlib_keras fit
function if the num_classes defined in the architecture is a number
greater/different than the size of the one hot encoded array.
This commit adds two functionalities:

A new optional parameter to minibatch_preprocessor_dl() that will
be used to determine the length of the 1-hot encoded vector for the
dependent var. If the param is set to NULL, the length will be equal to
the number of distinct class values found in the dataset, else
num_classes must be greater than equal to the number of distinct class
values.
The class_values column in the summary table contains an array of
class values associated with the 1-hot encoded vector. That will have
NULL as the value for class values that we don't find any representation
for in the dataset.
We now support NULL as a valid class value for dependent variable.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/799/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/802/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/803/


Search is broken when seeding search string from selection

Issue Type: Bug
Here is my experience with search.
I see a symbol I want to search for.
I select it.
I hit ctrl/cmd-f, and the search gets seeded with my symbol.
I hit cmf-g to go to the next occurence, but there aren't any.
There are no futher occurrences because seeding from selection, ALSO selects to only search within the selection. Which happens to still be my selected symbol.
Much gyrations then ensue to ensure that, no there is no selection, and yes, please search for the next occurrence of that thing I highlighted.
VS Code version: Code 1.41.0 (9579eda04fdb3a9bba2750f15193e5fafe16b959, 2019-12-11T17:58:38.338Z)
OS version: Darwin x64 19.2.0

I think the problem is really unexpected behavior and ingrained habits.
In this case I am used to selecting the string that I want to populate the search dialog with.
Whereas, in VS Code, you simply put your cursor in the string you want to search for.
In my case, my habit conflicts with useful behavior, because when I select the string first. The search gets restricted to the selection (as you can see that would be pretty useless)
Perhaps, the restrict to selection should only be active when the selection is more than a single word. Dunno.
I just have to break a years old habit. Heh. not so easy.
Thanks,
John

On Jan 27, 2020, at 6:57 PM, Peng Lyu<EMAIL_ADDRESS>wrote:
@jschank https://github.com/jschank sorry for reaching out late, could you provide more details about how to reproduce this issue? Like text samples and keyword you are using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/87186?email_source=notifications&email_token=AAAHM2RBVMYAWM3FJJ5BWGDQ75YGRA5CNFSM4J3534PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKBQYGA#issuecomment-579013656, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAHM2V5DIJUKTASZOYB7DLQ75YGRANCNFSM4J3534PA.

It sounds like maybe you have the editor.emptySelectionClipboard setting enabled?

Hmmm, Thanks for the suggestion, that does seem to help

On Feb 3, 2020, at 10:42 AM, Rob Lourens<EMAIL_ADDRESS>wrote:
It sounds like maybe you have the editor.emptySelectionClipboard setting enabled?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/87186?email_source=notifications&email_token=AAAHM2W4SWBYNRX6I26YZ7DRBA3PZA5CNFSM4J3534PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKUJZWI#issuecomment-581475545, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAHM2S2VSUQGCPZVGIB6E3RBA3PZANCNFSM4J3534PA.

Hmm, I actually meant to mention the editor.find.autoFindInSelection setting...


Adding telegram chat i.d

How can I add telegram chat i.d which will direct all logs to my tg instead of logging in to server?

Drop your telegram ID

https://t.me/Rolly800

thats done with web-hook i can help out

thats done with web-hook i can help out

How do I contact you?

add me on icq @714653641

Can't find this i.d on icq,don't you have telegram?

add me on telegram @piperberlesimo

For years we were enjoying this issues section of this tool as an avenue to share problems encounter and discuss solutions untill bunch of scammers and rippers joined here to practice their art.Eventually,kgretzky removed this section.Now based on this post i made 2 of the well known scammers here before already contacted me that they got solutions i just want others to watch out for them because i know both for long.If anyone knows the solution to link tg to evilginx 3.0 and ready to share this knowledge I will really appreciate but also take note that i'm not desperate for it.If you want to contact me to get some $ out of me,I say to you "don't".

I have customed one too that send to tg,that's not the reason I asked such question in the first place.I want to know how to implement it myself not to get already made.

@Jill60 watsup bro, were you able to figure out how to get it to deliver results to telegram especially for o365.
is there anyway i can contact you privately

@Jill60 watsup bro, were you able to figure out how to get it to deliver results to telegram especially for o365.
is there anyway i can contact you privately

msg me on telegram @ThyOnecy

I have customed one too that send to tg,that's not the reason I asked such question in the first place.I want to know how to implement it myself not to get already made.

i just made a video today showing someone how to do that

I knew you were the ripper that tried to ripped me sometime ago with your different ID and using different ID in giving references to yourself. Another thing, you told who to use escrow? from which platform? you are a scammer and a deceiver using different ID to justify your stupidity. Your mate are hustling while you are here using someone else free tools to take advantage of the hustler.
 Same way you came on telegram trying to rip people before i put your secret the group chat. Make youself hustle see better money to commot from the poverty level you dey.

Lol, @lhost25 . It is 1 thing to have money to purchase a certain tool, its another to not have money to purchase it and tag the sellers rippers.  You're broke and you need the free version of everything which is fine. Credits to @kgretzky for helping low budget researchers like you. I have the file with config on my github repository for free. You can thank me later, when you cannot afford a real diamond you don't assume its fake. Instead you go for what you can afford!!! Peace everyone

I knew you were the ripper that tried to ripped me sometime ago with your different ID and using different ID in giving references to yourself. Another thing, you told who to use escrow? from which platform? you are a scammer and a deceiver using different ID to justify your stupidity. Your mate are hustling while you are here using someone else free tools to take advantage of the hustler.
 Same way you came on telegram trying to rip people before i put your secret the group chat. Make youself hustle see better money to commot from the poverty level you dey.
 It is 1 thing to have money to purchase a certain tool, its another to not have money to purchase it and tag the sellers rippers.  You're broke and you need the free version of everything which is fine. Credits to @kgretzky for helping low budget researchers like you. I have the file with config on my github repository for free. You can thank me later, when you cannot afford a real diamond you don't assume its fake. Instead you go for what you can afford!!! Peace everyone

I knew you were the ripper that tried to ripped me sometime ago with your different ID and using different ID in giving references to yourself. Another thing, you told who to use escrow? from which platform? you are a scammer and a deceiver using different ID to justify your stupidity. Your mate are hustling while you are here using someone else free tools to take advantage of the hustler.
 Same way you came on telegram trying to rip people before i put your secret the group chat. Make youself hustle see better money to commot from the poverty level you dey.
 It is 1 thing to have money to purchase a certain tool, its another to not have money to purchase it and tag the sellers rippers.  You're broke and you need the free version of everything which is fine. Credits to @kgretzky for helping low budget researchers like you. I have the file with config on my github repository for free. You can thank me later, when you cannot afford a real diamond you don't assume its fake. Instead you go for what you can afford!!! Peace everyone

I knew you were the ripper that tried to ripped me sometime ago with your different ID and using different ID in giving references to yourself. Another thing, you told who to use escrow? from which platform? you are a scammer and a deceiver using different ID to justify your stupidity. Your mate are hustling while you are here using someone else free tools to take advantage of the hustler.
 Same way you came on telegram trying to rip people before i put your secret the group chat. Make youself hustle see better money to commot from the poverty level you dey.
Your days are numbered and soon you will face it.

@lhost25 drop your Telegram I.D

Aiit boss. https://t.me/six9inety (@six9inety)

I have customed one too that send to tg,that's not the reason I asked such question in the first place.I want to know how to implement it myself not to get already made.

@Jill60

@Jill60 watsup bro, were you able to figure out how to get it to deliver results to telegram especially for o365.
is there anyway i can contact you privately

The one i had was for evilginx 2.4.0 the version before 3.1.0,I just wanted to be able to apply same to this version.But i will advice you not to pay these scavengers/hyenas here try to scamm.

How can I add telegram chat i.d which will direct all logs to my tg instead of logging in to server? Don't even think about it,if you plan on extracting money from me to share this.

what a great idea, i am now intrigued, are you still active? this would be a great idea since youd need less of a vps to run the campaign, excellent, do please get back to me i am interested!

Ofcourse is possible,I have used it before with 2.4.0 version before this 3.0.1/3.1.0 versions.I have no issues copying from panel since i use tmux that's 24-hours active.I just want the knowledge of how to do it myself.But if anyone is not ready to share it here but rather give their i.d,just know they are trying to scam you i'm saying this from experience.

@Jill60 watsup bro, were you able to figure out how to get it to deliver results to telegram especially for o365.
is there anyway i can contact you privately

msg me on telegram @ThyOnecy

https://t.me/Dlenny100 here

Message me on telegram and I'll test it for you to confirm the result deliverability to telegram...@nginxtell

**Hi 👽 ! If you are interested in ⚙Advanced Phishing⚙ and the ✨Evilginx3✨ tool then I offer you Google Gmail and Custom Phishlets with a ✅Free Tutorial✅ which arrives if the 📈Telegram reaches 50 Subscribers📈
⏳ GOOGLE GMAIL PHISHLET 2024 WITH PROOF ⏳
->
⛔https://www.youtube.com/watch?v=nA47UknIE6A⛔
🔓 Ask your questions and a free tutorial from 50 subs on telegram !!!
->
📩https://t.me/Noremorse1011📩**

I have integrated that feature and will share for free. Can also show you how to do it yourself. Only for educational purposes. Scammers stay away

I have integrated that feature and will share for free. Can also show you how to do it yourself. Only for educational purposes. Scammers stay away

We will appreciate if you can share for us for free. Thanks in advance.

I have added telegram to my repo , check it out


Move to individual collections for Target Instances in Pywb

Add an alternate integration with Pywb, to create individual collections for Target Instance - Harvest Results. This would either replace the default single collection use with Pywb or be an optional feature that could be enabled/disabled.
Upon indexing each Harvest Result, WCT Store would create a new collection in Pywb, defined by the Target Instance ID and Harvest Result number, -(HR-ID>, e.g. "10078623-1".
The "Review in Access Tool" URL would then need to include the generated collection ID, e.g. "wayback.wct-host/10078623-1/2023110278364/https://a-website.com/"
This would provide two main benefits

Allow discreet QA of Harvest Results, with no risk of the viewer using content from other Target Instances or Harvest Results in WCT.
Make it easier to cleanup archived and rejected Target Instances (and just old ones too), by just deleting the Pywb collection directory for that Target Instance.

Also, if a Harvest Result is not indexed, in place of the "Review in Access Tool" link, give an option to index the harvest. Then if an older Harvest Result is cleaned up by WCT but is needed again by a user at a later date, they can re-index and view the harvest.

Requirements

Create new pywb collection

construct collection name in following format "<Target Instance ID>-<Harvest Result ID>"

Call python command to index WARC files for Harvest Result

"wb-manager add <collection name> <path to WARC files>"
Check indexing completed - How TBD
Bypass existing WARC file copy to wayback input folder

Delete collection folder on filesystem when cleaning up archived/rejected Target Instances
Modify construction of "Review in Access Tool" link to use collection name

If Harvest Result not indexed, show link to trigger indexing instead.

Add application.properties configuration to enable "pywb collection mode"


Add a logo on nav bar

As you can see in the screenshot there is no logo in the navbar

Here is one example of a logo we can put on the navbar:

Assign this issue to me so I can work on it

Hi @kaushal1717 @AnupK1234 ! I'm interested in fixing this issue.
Please assign this issue to me.

The codebase is updated. Kindly pull the latest code from main branch to see the changes.


How to test with RSpec

Is there a macro or anything I can reuse in a RSpec test?
What i want is something like:
it { should validates_email_format_of(:email) }  to "Just Works".
Any idea to where do I should start is appreciate. I pretend to code it as some sort of shoulda macro and publish it back

Not sure -- I'm a heretical Rubyist in that I've never used RSpec. I've put out a Twitter call in case anybody has an answer.


Google login button issue #41

Description
Added a button with google logo image, when a user would like to login using google account.
Fixes # (issue)
#41
How to test?
Please provide a short summary how your changes can be tested?
Checklist

[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 made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

This looks ready for approval and merging. @Enqira have the changes you suggested been incorporated? If so, this can be approved and then @TITHI1244 can merge it.

@cph-kiwi Not all of them yet, but they are in the work...


More pkg_resource deprecation warnings

Found a couple more while running tests locally
Related: https://github.com/microsoft/debugpy/issues/1230

merging, if only to clear the PR list! @shatakshiiii hope this is ok, just another annoying pkg_resources deprecation warning


docs: entrypoint can be passed as string or argv

this wasn't documented whoooops. Good catch @nicks

Yep, commands passed to local or to local_resource both get run thru ValueToHostCmd. Docs updated!


Improve documentation for areStatesEqual

I am trying to limit the amount of times react-rexdux connect() fires re-renders. I've looked at the documentation here...
https://github.com/reduxjs/react-redux/blob/master/docs/api.md
... and supposedly there is a way to prevent re-renders by writing your own config option function for areStatesEqual. The problem is, my simple console.log test shows that this option never fires! Not sure what I'm doing wrong. There is not a lot of documentation on this feature, not even one example.
export const mapStateToProps = (state, ownProps) => Object.assign({
  isMobileLayout: state.deviceWidthReducer.width < 480,
  postalCode: state.userReducer.postalCode,
  productImageGroup: state.productReducer.productImageGroup,
  productId: '05047234',
}, ownProps);

export default connect(
  mapStateToProps,
  null,
  null,
  {
    pure: true,
    areStatesEqual: (next, prev) => {
      console.log('areStatesEqual?'); // <-- THIS NEVER GETS LOGGED.
      return next.productId === prev.productId;
    },
  },
)(Product);

What I'm trying to accomplish:  I don't want connect() to re-render the target component at all if the productId has not changed.

Are you running the render more than once? It won't run on the first call, but will on subsequent calls.

Yep - it's specifically this chunk here:
https://github.com/reduxjs/react-redux/blob/3349bd09c772d0268bef5613c531c10f93f98915/src/connect/selectorFactory.js#L74-L76

As for documentation, the function is documented at https://github.com/reduxjs/react-redux/blob/master/docs/api.md#connectmapstatetoprops-mapdispatchtoprops-mergeprops-options , although not in great detail:

[areStatesEqual] (Function): When pure, compares incoming store state to its previous value. Default value: strictEqual (===)

Closing in favor of #1001


Delete function

Added Delete function

Has dialog for user to confirm if they want to go ahead and delete

DeleteRow method added to DatabaseHelper

Can you add a screenshot for delete button?

@redatawfik  I put the screen record on discord as github doesn't want to play it
Noted with regards to feedback, I will Implement the changes.

Looks good to me, Please update your local repo to solve the merge conflict


Make unsubscribe form

The path /unsubscribe exists, but it's essentially a static page. Make it a form for with a single field for the email address. The code for calling the Sparkpost API can be found in https://github.com/geowa4/elixir-sparkpost-experiements.

I AM ALL TAKING YOUR ASSIGNMENTS

I was able to set up unsubscribe links using https://www.sparkpost.com/docs/user-guide/setting-up-unsubscribe-links/ so no code changes were required.

Why doesn't GitHub refresh pages automatically...


Union/Stacked support in the reference server (or as an external project ?)

The 9spock project server implements union/stacked support. We can port this code to SpockFS uWSGI plugin.
The rule to follow is that the last item in the chain is the writable one:
---spockfs-union-mount /foo,/bar,/writeme

will build a union of /foo, /bar and /writeme with every write redirected to /writeme (in copy on write mode).
Maybe it is better to have spockfs-union as an external plugin and leave the official one as simple/tiny ?

I think it's better to don't add that extra functionality, you can be able already to use an OverlayFS to combine several directories in a single mountpoint, and later use SpockFS to serve its content.


1566 quota - release 1.11

@pkdash @mjstealey I created another PR for quota against release 1.11 branch just for convenience in case quota needs to go in release 1.11. I kept the other corresponding PR against develop open for now since it has code reviews embedded. @pkdash Feel free to code review against either PR as you like, @mjstealey I'll leave it up to you which PR you will merge and close depending on whether quota implementation can go in to release 1.11 or not.

@hyi - I think this would be fine to have in the RC as we can "mute" the quota pieces until given a final 👍 in production. Makes it easier to turn on if the pieces are already in place and not have to wait for a subsequent release.

@mjstealey OK. I agree. "mute" the quota pieces are easy - just to comment out a few lines in a couple of places to "mute" the quota messages when users log in and on their profile page. The "mute" part is not done in this PR yet. Let me know if you want me to comment out those lines to "mute" them. I guess it all comes down to whether you are going to wire your script in for release 1.11 or release 1.12. This PR needs to go together with your script so that they work as expected. I can let you know which lines to comment out in order to "mute" quota messages, then you can do whatever needed for the next couple of releases. I think this is probably the easiest. Let me know which option you prefer.

@hyi - I'd think if you create a quota-info.md file at the main level of the repository and document what needs to be done to turn quota on or off that would suffice.
It will also give me a place to document the backend script information for how everything gets wired together on the Jenkins / iRODS side of the fence.

@mjstealey Have created quota-info.md in the quota branch and bring it up to date with release-candidate-1.11.0 branch. Feel free to edit quota-info.md as needed and let me know if anything I wrote there is not clear. I'll leave this PR open as is now and leave it to you to do whatever you think works best for the next couple of deployments. Do note I have already merged this PR to develop branch, so if you see any issues with that and need me to do anything, let me know.

@pkdash - Could you do a final review?
I'd think getting this into 1.11 is a good thing with the understanding that we will not be enforcing quota at this time and would obscure the output to the user based on the instructions provided by @hyi.
We can then monitor usage and quota output behind the scenes transparently to the user and have some metrics to discuss prior to enabling quota enforcement in 1.12.

@mjstealey Will do another review of this PR today.

@hyi Are there unit tests to verify that user can't create resource with uploaded file or add a file to a resource if the user is over the quota limit?

@pkdash Yes, there are unit tests to verify over-quota cases when creating new resources or adding files to existing resources. For example: https://github.com/hydroshare/hydroshare/blob/develop/hs_core/tests/api/native/test_create_resource.py#L415

@hyi - I would like to get this into 1.11.0, and if @pkdash is satisfied with the unit test question I'd think we can bring this in. I'll plan to rebuild prior to the programmers coordination call so that we can review quickly and then schedule the production deployment to www on Tuesday 7/18.

@hyi Once you have a unit test for the view function +1 from me.

@pkdash I have updated test_change_quota_holder() api native unit tests to account for over-quota scenario, and also added unit test for the corresponding view function. See if you can +1 now.

+1


Update and fix a bunch of things

Now with captured logs and progress bars, and it seems to work better too

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@10cae34). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   90.17%
=========================================
  Files             ?        1
  Lines             ?      112
  Branches          ?        0
=========================================
  Hits              ?      101
  Misses            ?       11
  Partials          ?        0

Continue to review full report at Codecov.

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


[VsTest] .testsettings not taken into account

Hey!
The Visual Studio Test task's documentation for the Run Settings File parameter specifies:

Run Settings File: Path to a runsettings or testsettings file can be specified here.

But any .testsettings file passed will be ignored.
I guess this is just an outdated documentation, but I want to make sure that .testsettings are indeed not supported and that this is not a bug.
Thanks!

Testsettings file is supported. Why do you think it is ignored?
Thanks!

In that case I'll try to reproduce the issue and get you detailed info.
As a side note, would be nice to update the tooltip in the task to reflect this, since only runsettings is specified:

Thanks!

@wbuchwalter I may be able to help you with this - the .testsettings file that I use in my application correctly deploys additional test artifacts when used locally, but does not deploy any of the supporting files when using the "Run Functional Tests" Task through TFS 2015.
You can find more info HERE.

Team,
I want to update this issue rather than opening a new one - I'm not sure if @wbuchwalter was having similar problems with .testsettings files as I am right now.
The file I specify in the build configuration screen is being correctly pushed to the member of the Machine Group that's intended to run the test:

The directory containing these files on the member of the machine group is identical to how my solution looks on my dev box:

And yet, you can see that no files are deployed, using my kung-fu-super-fast-copy-paste skills from the byzantine App Data folder:

What gives here? One of the key components of the .testsettings file is the deployment functionality that works locally. I can't tell if it's being ignored or not.

Hey,
So I spent a little bit of time trying to reproduce the issue.
Here is a simple .testsettings file that I'm using (I removed some non-relevant stuff to keep it short):
<TestSettings name="Local">
  <Deployment>
    <DeploymentItem filename="..\..\scripts\echo.ps1" />
  </Deployment>
</TestSettings>

If I don't change any settings from the default Visual Studio Test task except the Run Settings param, everything seems to work fine (log from build agent):
Executing [...]\vstest.console.exe [paths to all my assemblies] /Settings:"C:\a\1\s\tests\PartsUnlimitedWebsiteTests\local.testsettings"

But once I change another parameter, for example by checking Run In Parallel :
Executing [...]\vstest.console.exe [test assemblies] /Settings:"C:\Users\buildguest\AppData\Local\Temp\tmp1B2E.tmp"

And if I take a look at tmp1B2E.tmp:
<?xml version="1.0" encoding="utf-8"?>
<RunSettings>
  <RunConfiguration>
    <MaxCpuCount>0</MaxCpuCount>
  </RunConfiguration>
</RunSettings>

So basically, since the vsts task itself will store it's configuration in a runsettings file, everything will work fine if we pass a runsettings file as well for the Run Settings param, since both settings will be merge in something.tmp.
But this cannot work with a .testsettings file, since the merge will fail, and the runsettings of the task itself will take prescendence. Basically the testsettings file will be ignored.
(lot of settings in this sentence :) ).
Let me know if I missed something!

Ahhh okay. So, in summary: it looks like this may be specific to the VS Test Task, as I have parameters defined in the Functional Test Task (Test Filter Criteria) and my .testsettings file at least is copied into my temp directory rather than a blank slate .runsettings file.
I'll open my own issue here - thanks for the clarification!

@atinb - from @wbuchwalter's investigation it seems like it's a problem with the VS Test Task? Can the team check it out? We are tracking this as part of a partner feedback

We have filed a bug and will be fixing it in the next sprint. Run in Parallel should not work with testsettings file. As a workaround, please uncheck the "Run in Parallel" check box. Thanks!

This is fixed now. Closing the issue.


evp_test: Add the check for potential failure of allocation

Since the memory allocation may fail, the 'mac_name' and 'controls'
could be NULL.
And the 'mac_name' will be printed in mac_test_run_mac() without check.
Also the result of 'params_n +
sk_OPENSSL_STRING_num(expected->controls)' in
mac_test_run_mac() will be 'params_n - 1' if allocation fails , which
does not make sense.
Therefore, it should be better to check them in order to guarantee the
complete success of initiation.
Signed-off-by: Jiasheng Jiang<EMAIL_ADDRESS>
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

mdat will leak if the new return 0 calls are reached..

mdat will leak if the new return 0 calls are reached..

Thanks, you are right.
I will close it and submit a corrected one.

Next time you do not need to close the PR. You can just add fixup commits created with the git commit --fixup command.

Next time you do not need to close the PR. You can just add fixup commits created with the git commit --fixup command.

Thanks, I got it.


Lab 0201 - reset cloud user passwords

Module: 02
Lab/Demo: 0201
Task: 2
Step: 8
Description of issue
User password error. It's necessary to reset all cloud user passwords (as per MD101) using M365 admin center. Not MS Service nor MOD account, however. Easiest is to reset to same password as currently used by tenant. Review steps from latest MD101 lab 0201. Once done, this works fine.
Repro steps:

@WarrenAJ, thanks for the feedback on this - we are working out the best approach to deal with this specific issue (newly deployed tenants no longer use default tenant PW for all users, they're randomized). Ultimately the lab release at the end of the month with include some sort of verbiage at the beginning of the lab series to reset user passwords. Thanks!


Skeleton for remote query execution using celery.

Current change adds ability to execute the SQL queries on the celery worker.
More info about celery can be found here:

http://docs.celeryproject.org/en/latest/getting-started/first-steps-with-celery.html

Worker can be started via command caravel worker
Celery is configured in the config.py and by default it is turned off. (CELERY_CONFIG = None)
Fixed some lint issues in the tests.
Tested:

Manually tested if the query workes fine with the CELERY_CONFIG = None and with

class CeleryConfig(object):
    BROKER_URL = 'sqla+sqlite:///celerydb.sqlite'
    CELERY_IMPORTS = ('caravel.tasks', )
    CELERY_RESULT_BACKEND = 'db+sqlite:///celery_results.sqlite'
    CELERY_ANNOTATIONS = {'tasks.add': {'rate_limit': '10/s'}}
CELERY_CONFIG = CeleryConfig

Task is passed and executed:

Query works for both cases:

Reviewers:

@georgeke
@mistercrunch
@ascott

Coverage decreased (-0.3%) to 80.857% when pulling bb6d3f060b7a490e440a9a08886c1e948ded6b3d on bkyryliuk:celery into b295436bff6ad958fb75852d577f83c36667b4a0 on airbnb:master.

Coverage decreased (-0.3%) to 80.887% when pulling bb6d3f060b7a490e440a9a08886c1e948ded6b3d on bkyryliuk:celery into b295436bff6ad958fb75852d577f83c36667b4a0 on airbnb:master.

Thanks @mistercrunch
Do you have any suggestions how to unit test the bin/caravel commands and get_sql_results function?
Will mocking the pandas and db modules be fine ?

Coverage increased (+0.05%) to 81.188% when pulling bdd8b7fbdc0be0d8b72b70adfc83b05ccedd5afa on bkyryliuk:celery into b295436bff6ad958fb75852d577f83c36667b4a0 on airbnb:master.

Coverage decreased (-0.01%) to 81.174% when pulling 91e720a91fbdf820daaf851a5ba5ba12095f9875 on bkyryliuk:celery into f034f2701e886d3b0e6b41203a65ad4c3fb1bea5 on airbnb:master.

Coverage decreased (-0.04%) to 81.146% when pulling 91e720a91fbdf820daaf851a5ba5ba12095f9875 on bkyryliuk:celery into f034f2701e886d3b0e6b41203a65ad4c3fb1bea5 on airbnb:master.

Coverage increased (+0.1%) to 81.297% when pulling 91e720a91fbdf820daaf851a5ba5ba12095f9875 on bkyryliuk:celery into f034f2701e886d3b0e6b41203a65ad4c3fb1bea5 on airbnb:master.

Coverage decreased (-0.04%) to 81.146% when pulling a326eef59e4900c64fe9cfd2293ed035195875c4 on bkyryliuk:celery into f034f2701e886d3b0e6b41203a65ad4c3fb1bea5 on airbnb:master.

Coverage decreased (-0.04%) to 81.146% when pulling a326eef59e4900c64fe9cfd2293ed035195875c4 on bkyryliuk:celery into f034f2701e886d3b0e6b41203a65ad4c3fb1bea5 on airbnb:master.

Coverage decreased (-0.04%) to 81.146% when pulling 1441bb5f662e8378a88709271625521d781e0e17 on bkyryliuk:celery into 24e85f52b4ead7f8f70b8d3d7e5724d8ca597bd9 on airbnb:master.

Coverage decreased (-0.04%) to 81.146% when pulling 72c01bc7f26d91bd6ad50bdc89b1543a02183a36 on bkyryliuk:celery into 24e85f52b4ead7f8f70b8d3d7e5724d8ca597bd9 on airbnb:master.

Coverage decreased (-0.04%) to 81.146% when pulling 72c01bc7f26d91bd6ad50bdc89b1543a02183a36 on bkyryliuk:celery into 24e85f52b4ead7f8f70b8d3d7e5724d8ca597bd9 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling e81b32b103256832175d84c3b0a8e3cec004409f on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage increased (+0.1%) to 81.316% when pulling 4524278add0adbdd2ff2e4f68224b00fc74b6112 on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling 6f546fced0e634811dd023b1780e0b00c7b08d84 on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling e35ede789d1b7312b88b76fd35a3795929c0e38b on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling 0ccd992668ef43db6a7845755d69c24ba4178886 on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling 0ee0c8f18a3378edaa3a08a46adc9cd79aadb56e on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling 715f1226bab85de906e2b789756cafa7660dabf7 on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling 715f1226bab85de906e2b789756cafa7660dabf7 on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.02%) to 81.165% when pulling a7487634e91a1c63d2e1478a055112777b6705b3 on bkyryliuk:celery into 2b237f483f305bb265980d9b2f272d99fb974c47 on airbnb:master.

Coverage decreased (-0.5%) to 80.525% when pulling 314c752471e183779de4cd952360251fb8d6a113 on bkyryliuk:celery into f43e5f18d58c73b10aea3d7ca603e916bed5ed9e on airbnb:master.

Coverage decreased (-0.5%) to 80.513% when pulling e59983ad88eefbf3f526ad4e472e167a8c5f0be0 on bkyryliuk:celery into f43e5f18d58c73b10aea3d7ca603e916bed5ed9e on airbnb:master.

Coverage decreased (-0.5%) to 80.513% when pulling e59983ad88eefbf3f526ad4e472e167a8c5f0be0 on bkyryliuk:celery into f43e5f18d58c73b10aea3d7ca603e916bed5ed9e on airbnb:master.

Coverage decreased (-0.5%) to 80.513% when pulling 35b6121b674e5a30e183540c58cd990a35813fd1 on bkyryliuk:celery into 1101de5ae4073628fd41d49e9cc854e7610b1da4 on airbnb:master.

Coverage decreased (-0.6%) to 80.416% when pulling d0dca827134597cd3fcfe447676bbf2910b0fb17 on bkyryliuk:celery into 1101de5ae4073628fd41d49e9cc854e7610b1da4 on airbnb:master.

Coverage decreased (-0.6%) to 80.416% when pulling d0dca827134597cd3fcfe447676bbf2910b0fb17 on bkyryliuk:celery into 1101de5ae4073628fd41d49e9cc854e7610b1da4 on airbnb:master.

Coverage decreased (-0.6%) to 80.38% when pulling 147faea924236b625b9bb604b6e70c7de7f4aeca on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.6%) to 80.38% when pulling eaf2c5178e7597cb3329a998c513feb2d62e4ae6 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.344% when pulling cec7d31465cce5dcc538a7b43d5ad34b267e054f on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.32% when pulling a91576805d5c1532a86355c5f66b26a610631324 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.32% when pulling edbd0a5fb0adafb2a54f58974bdfd6fc7bac176c on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.285% when pulling 19d95d2ce4239aa0795f5d61ea92a17ee13e031a on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.332% when pulling f0c46152c074b3f5eb126037a7721ecdd13a97f4 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.308% when pulling 8ffbf350752a7c19ae9f694b7308a04d2b53f188 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.8%) to 80.237% when pulling 4d111703bdd56d035244d97808706179200237b3 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.8%) to 80.237% when pulling ef8865ec348e4c2f96a62ed851d77ac1d5897e5d on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.8%) to 80.19% when pulling ef8865ec348e4c2f96a62ed851d77ac1d5897e5d on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.8%) to 80.19% when pulling 300fc12d227c0380c48dd40384e569287a46aa17 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.8%) to 80.249% when pulling ad9bd89c689788cc0e7c61d23ef5a6be09ea543d on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.8%) to 80.19% when pulling 1b531c6a743355d1ad98e88b90f2e165d93df191 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.8%) to 80.225% when pulling 0b1d6217fc5705a25dcb781f82775db4506512a9 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.368% when pulling 6937cdcd35039a9a12bcc62427c8f862784cefc0 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.368% when pulling d5bacf1ddf3b7b59e62ce03c94c16c88c66f9bad on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.368% when pulling 9d9f7de0d8f35f0208195e0ef3378941e2c47a53 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.333% when pulling b645b1233e656bf9c551f5c9cb20b30c3119dd52 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage decreased (-0.7%) to 80.333% when pulling b645b1233e656bf9c551f5c9cb20b30c3119dd52 on bkyryliuk:celery into 299e31fdff1861c434c18efaed804d63572aec16 on airbnb:master.

Coverage increased (+0.2%) to 81.362% when pulling 85b2e9e97f66bd427a3e7234fa6cdfdba1961393 on bkyryliuk:celery into ee9141a31a84c7c1114163995edb718f89372a15 on airbnb:master.

Coverage increased (+0.2%) to 81.701% when pulling 565c0ecda4d29e764d934a26325de35b726bbc1c on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.2%) to 81.701% when pulling 8b339015265fc5f9ab778bac81074c5ce5179470 on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.2%) to 81.659% when pulling bb5569b5afe7565ab61417a69332e8f0dd84eeec on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.2%) to 81.654% when pulling 7a2d3b3327bf899384f941941d9cc410273517be on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.2%) to 81.654% when pulling 071974cd33b3a668e444e4b45ba6c4642a114450 on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.02%) to 81.516% when pulling 071974cd33b3a668e444e4b45ba6c4642a114450 on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage decreased (-0.007%) to 81.487% when pulling cdb5d9c388e83e6cdc3f2afda43c4bba62ed4c7b on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.02%) to 81.516% when pulling cdb5d9c388e83e6cdc3f2afda43c4bba62ed4c7b on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage decreased (-0.007%) to 81.487% when pulling 95e0c7d3575c38218c028dc1cade3640c651ed97 on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.02%) to 81.516% when pulling ccd43355165304eb7b07737ce6dba1b29463b945 on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage increased (+0.02%) to 81.516% when pulling f300c591b84a7f77ede9432760f668c93e5b9c6e on bkyryliuk:celery into aaef338539eff4ff855e0f87bcb17ca71440e251 on airbnb:master.

Coverage decreased (-0.04%) to 81.458% when pulling d7c51bc2d3a4cdb104aca54d8e72ef4dab96f61a on bkyryliuk:celery into 82a8e6316f874b28c8b8ff718917006b524cd7d6 on airbnb:master.

Coverage increased (+0.04%) to 81.443% when pulling c436328bb9390e1599879b8ec89abc06627a30b7 on bkyryliuk:celery into 7c810dbd20ef394c073ec78c0c46380f471fff4d on airbnb:master.

Coverage increased (+0.04%) to 81.443% when pulling c436328bb9390e1599879b8ec89abc06627a30b7 on bkyryliuk:celery into 7c810dbd20ef394c073ec78c0c46380f471fff4d on airbnb:master.

Coverage increased (+0.04%) to 81.443% when pulling c436328bb9390e1599879b8ec89abc06627a30b7 on bkyryliuk:celery into 7c810dbd20ef394c073ec78c0c46380f471fff4d on airbnb:master.

Coverage decreased (-0.05%) to 81.43% when pulling 862e47aa45ba871c48d69633532611d49076d5c3 on bkyryliuk:celery into cb23362a5bff9cec8354968b5a8a612126e1d82d on airbnb:master.

Coverage decreased (-0.05%) to 81.43% when pulling 7ff31bb61212da8ead0117d3df4c0b0aef9f2233 on bkyryliuk:celery into b0a1f07818cc1454bb823933a783035f753d72e8 on airbnb:master.

Coverage decreased (-0.1%) to 81.359% when pulling 87e03471ec404ae5b34ead08500c139e67fcca70 on bkyryliuk:celery into b0a1f07818cc1454bb823933a783035f753d72e8 on airbnb:master.

Coverage decreased (-0.1%) to 81.359% when pulling 87e03471ec404ae5b34ead08500c139e67fcca70 on bkyryliuk:celery into b0a1f07818cc1454bb823933a783035f753d72e8 on airbnb:master.

Coverage decreased (-0.1%) to 81.348% when pulling 7092925d67e573851e370f6911f106fd67fa3b3e on bkyryliuk:celery into b0a1f07818cc1454bb823933a783035f753d72e8 on airbnb:master.

Coverage decreased (-0.1%) to 81.348% when pulling 7092925d67e573851e370f6911f106fd67fa3b3e on bkyryliuk:celery into b0a1f07818cc1454bb823933a783035f753d72e8 on airbnb:master.

Coverage decreased (-0.1%) to 81.348% when pulling 7092925d67e573851e370f6911f106fd67fa3b3e on bkyryliuk:celery into b0a1f07818cc1454bb823933a783035f753d72e8 on airbnb:master.

Coverage decreased (-0.1%) to 81.348% when pulling 15a23d31a9b9b03a7217f44f159d115384e448b8 on bkyryliuk:celery into 572c6ee85ea4f46cca36a9d41e95a668a7a78a23 on airbnb:master.

Coverage decreased (-0.1%) to 81.348% when pulling 15a23d31a9b9b03a7217f44f159d115384e448b8 on bkyryliuk:celery into 572c6ee85ea4f46cca36a9d41e95a668a7a78a23 on airbnb:master.

Coverage decreased (-0.1%) to 81.348% when pulling 15a23d31a9b9b03a7217f44f159d115384e448b8 on bkyryliuk:celery into 572c6ee85ea4f46cca36a9d41e95a668a7a78a23 on airbnb:master.


[FEAT] Shard @daffodil/newsletter Library

:bulb: Feature request
Feature Name

The Desired Behavior

Your Use Case

Prior Work

Environment

Daffodil version: X.Y.Z
Angular version: X.Y.Z

For Tooling issues:
- Node version: XX
- Platform:

Others:

Closed by #1385


[HOPS-1576] and [HOPS-1577] update dependencies versions

Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added and passed (for bug fixes / features)
[ ] HOPS JIRA issue has been opened for this PR
[ ] All commits have been squashed down to a single commit
[ ] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-1576
https://hopshadoop.atlassian.net/browse/HOPS-1577

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Jenkins test this please.

Jenkins test this please.

Updating xerces is not very usefull and guava is breaking some dependencies.


Porting: Add Enterprise data governance using Watson Knowledge Catalog

port from https://github.com/IBM/cloudpakfordata-telco-churn-workshop

Add and modify Add Enterprise data governance using Watson Knowledge Catalog
update images

delivered in pr #42


xWebAdministration: Resolved custom Script Analyzer rules

Pull Request (PR) description

Changes to xWebAdministration

Resolved custom Script Analyzer rules that was added to the test framework.

This Pull Request (PR) fixes the following issues

Task list

[x] Added an entry under the Unreleased section of the change log in the README.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #445 into dev will not change coverage.
The diff coverage is 46.87%.

@@           Coverage Diff           @@
##              dev     #445   +/-   ##
=======================================
  Coverage   90.85%   90.85%
=======================================
  Files          17       17
  Lines        2461     2461
=======================================
  Hits         2236     2236
  Misses        225      225

Impacted Files
Coverage Δ

DSCResources/MSFT_xIisModule/MSFT_xIisModule.psm1
2.91% <0%> (ø)
:arrow_up:

DSCResources/Helper.psm1
90% <100%> (ø)
:arrow_up:

...ces/MSFT_xWebApplication/MSFT_xWebApplication.psm1
90.75% <100%> (ø)
:arrow_up:

DSCResources/MSFT_xWebsite/MSFT_xWebsite.psm1
97.18% <100%> (ø)
:arrow_up:

...SCResources/MSFT_xIisLogging/MSFT_xIisLogging.psm1
97.74% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


tube-rack-5ml-96_tube-rack-2ml-9x9

Added tube-rack-4ml-96
Added tube-rack-2ml-9x9

Coverage remained the same at 96.081% when pulling af845f4dfb6207697c5131813d6e245243c18ee6 on tube-rack-5ml-96_tube-rack-2ml-9x9 into 9e6dda1e1b4edf603138d446c3632aa8216a7f79 on master.


Errors when simplifying the original country data

I love this script, it is very useful! However the piece of code that simplify the original country of sampling has a bug and sometimes change the country. For example, it turns USA into Chad, Venezuela becomes Senegal, South Korea becomes South Africa, etc.
Thanks for your attention, Simon.

Hi Simon,
Sorry for the late answer. Some examples of the error in parsing the country can be found below:
name	accession	length	submission_date	host	country_original	country	countrycode
MF001519	MF001519	12008	2017-05-09	Homo sapiens	USA	Chad	TCD
KX352216	KX352216	148	2017-07-31	Homo sapiens	Venezuela	Senegal	SEN
KC810970	KC810970	1317	2013-10-29	Homo sapiens	South Korea	South Africa	ZAF
I've also observed a second issue. Sometimes, the script gets the wrong sampling date. For exemple, the sequence LC259094 was collected in 2016-05-11, but the script parse 2006-03-31. This issue happened with all the sequences below, all from the same batch.
 Any further issues can be dealt with by editing countrycodes.h and rebuilding.  I could not reproduce your issue with dates - it seemed to work fine for me.

Closed with adbead3


Update endpoints for repository rename

Description
This updates all of the endpoints and icon urls to respect the new repository name of kubewarden-ui.

I've just renamed the repository. Merging


Control ISHRemote usage in your organization by extra UserRoles AutomationISHRemoteReadAccess and AutomationISHRemoteWriteAccess

Problem Statement…
One feedback I got from several excited automation customers on SDL Connect 2019 is that they don’t “trust” all of their users (tech writers, localization, pub mgrs.,…) … the customer’s IT team is scared that some user deletes a lot of stuff, or re-publishes way more publications (generating load) than that they would do using the UI.
Limitations…
Client Tools use exactly the same API as ISHRemote. We are not trying to solve that anybody with C# skills can write an API program anyway. The Docs CMS only knows about UserGroups and UserRoles, all our permissions are implicit in the code and not configurable.
Solution...
A Server-Side Solution would be the perceived best solution, something that we will keep in mind when we revisit the AuthN/AuthZ flow, so the generation after current WS-Trust/WS-Fed. This does however not solve existing CMS 11/12/13/14 setups.
So let's try to handle it in ISHRemote code base as that is where most of the fear comes from :-)
Upon New-ISHSession, ISHRemote could check if the UserRole AutomationISHRemoteReadAccess and AutomationISHRemoteWriteAccess exist.

If they do not exist, full access. Current behavior.
If they do exist, then check if the passed user credentials result in a User Profile holding these UserRoles.

On every read cmdlet, do a precheck on AutomationISHRemoteReadAccess
On every write cmdlet, do a precheck on AutomationISHRemoteWriteAccess

To be clear the check would be done per cmdlet, and not in the WCF proxies offered on the IShSession object.

[ ] Implement IShSession UserRole initialization with a Write-Verbose message. Limit testing to New-IShSession only, no extra tests on all cmdlets.
[ ] Document in release notes how to create the 2 user roles with their role description. Perhaps also add a routine how to assign the roles that currently have Administrator assigned already.
[ ] Add a BeginProcessing check on all read cmdlets like Get-* and Find-* that access the web services.
[ ] Add a BeginProcessing check on all write cmdlets like Add-*, Remove-*, Set-*, Publish-*, Stop-* and Move-* that access the web services.
[ ] Document on readme.md on best practices for development.

Any other ideas? We at least expect some up-votes on this :-)

Tridion Docs 15/15.0.0 received Access Management as a Federation Gateway. One of the features of Access Management is that it can give users or groups of users permission to certain applications. ISHRemote has been preconfigured. In turn this enhancement request is resolved outside of the ISHRemote code base. More details when ISHRemote v8.0 releases.


style and transform prop for Path2D

Introduction to style and transform prop.
A distinction is made between the two, since transform will always cascade and style won't. Maybe they will merge later on, but keeping them separate is easier to implement.
style contains all the Path2DStyle like fill, stroke, ...
transform contains position rotation and skew (should implement scale later)
style and transform (currently only style implemented) also are able to have selectors as properties, starting with &:hover (thanks @dev-rb for the idea)
<Arc
  style={{
    fill: "black",
    "&:hover": {
      fill: "white"
    }
  }}
/>

will create an ellipse that will turn from black to white when hovering:
https://user-images.githubusercontent.com/10504064/230267485-980aaf99-a4d9-47f7-aaab-e8edb3e49bd2.mov
It's internally implemented as a controller like Hover( { style: props.style["&:hover"] } ).
We could add other selectors later (like &:isActive)
We (chatgpt and I) implemented deepMergeGetters to merge objects with possible nested getters. Merging objects like this is a lot less performant then Object.assign or spread (see) but it could also prevent merges from happening. We should do more performance tests to see which strategy is more beneficial.
Previous PR removed createTransformedPath in favor of transforming the canvas-context.
This created glitches in auto-render mode when blending layers with composite
https://user-images.githubusercontent.com/10504064/230267148-428a03fb-5fad-4c47-82dd-5d7d0629a2f6.mp4
So it's reverted back to transforming the path instead of ctx

Noticed some intense performance degradation, so did some performance optimizations:

solved the ctx.setTransform-bug, createTransformedPath is out again
auto-render mode is now throttled with solid-primitives/scheduled. the necessity became extra apparent with combination of Hover/Drag, since the changing styles caused by hovering was causing re-renders while dragging, causing jank. The render-effect is throttled on 120fps.
createMouseEventHandler: before we were instantiating variables each event, causing GC, and thus jank. Variables are now instantiated inside the hook and mutated, this gave a big performance boost in firefox (almost doubled fps)
renderPath now checks if fill/stroke isn't undefined/"transparent" before it sets any of the fill/stroke related styles to the canvas-context.

This combination, mostly throttled render-effect and not instantiating constants in the mousevent-handlers, boosted performance a lot, while testing Rectangle.tsx. The test is only 1 level deep (no children to the tokens) and only has a fill and composite as styles, so it's a limited test, but it gives a good indication of simple, raw rendering.
A next optimization opportunity I envision would be not to perform any checks during renderPath, but instead compose a set of canvas.ctx-commands based on the tokens style-prop, and then simply forEach those with each render.

A next optimization opportunity I envision would be not to perform any checks during renderPath, but instead compose a set of canvas.ctx-commands based on the tokens style-prop, and then simply forEach those with each render.

decided against it

performance metric in firefox with just checks

performance metric in firefox with mapped commands

I could probably expand on the mapped commands to make it perform better (now it's mapping the commands for the whole style any time any prop of the style updates), but looking at the metrics with just checks, I don't know if we can beat it. It's definitely a premature optimization.


ciao-down: Fix paths in ciao-fedora25 workload

The workload arranges for some paths to be initialised in the .profile
file, but this file is not read when logging in via an interactive shell.
This commit updates the workload to ensure that the newly created .profile
is read from .bashrc.
Signed-off-by: Mark Ryan<EMAIL_ADDRESS>

Coverage increased (+0.004%) to 65.621% when pulling c4b34001b546978a54ae2aa5b384086ff1f045a4 on markdryan:fix-fedora into f7c22f630222b00404a6f209496624293fa107b5 on 01org:master.


Add simple Button component

@nick-thompson , here is a very simple Button component conforming to the styles we discussed a while back. Thought it would be nice to get this in. If you're happy with the general approach I will get Slider sent over next and then update the Gain example to use the new slider with a Canvas draw routine and I can add a simple Button example which just bypasses the effect or mutes etc.

@nick-thompson, OK I've bumped the package.json versions and addressed the minor review comments. Let me know if that's all good and I can rebase if you'd prefer. Let me know if anything needs to be done in regards to the npm package update. If needed I can split the event work to a separate PR first so that the package update can go in and then the GainPlugin change with the updated dependency can follow. Whatever is easiest.

Merged in 3ebdf5d. Had to ammend the latest commit to bump the package.jsons again because I messed up the npm publish, my bad!


No python-base in centos, fedora etc

C/C from https://bugzilla.novell.com/show_bug.cgi?id=856838
Require: python-base is only available in suse. Could you tweak the Require for
spec-cleaner to work over centos, fedora etc please.
For more precision, I'm think about a "if suse Requires python-base else Requires python, python-???".
I can't tell you what are the python-??? you need for spec cleaner, but here are an example of distro-conditionnal blocks in spec:
# RHEL <=5
%if 0%{?rhel} && 0%{?rhel} <= 5
BuildRequires: python26-devel
Requires: python26
%endif

# RHEL > 5
%if 0%{?rhel} && 0%{?rhel} > 5
BuildRequires: python-devel
Requires: python
%endif

# FEDORA > 17
%if 0%{?fedora} >= 18
BuildRequires: python-devel
Requires: python%endif
%endif

# SuSE/openSuSE
%if 0%{?suse_version}
BuildRequires: python-devel
BuildRequires: python-setuptools
Requires: python-paramiko
Requires: python-jinja2
Requires: python-keyczar
Requires: python-yaml
Requires: python-httplib2
%endif

NB, %else also exist iirc

By default we do not handle the suse compatiblity with other distros, but patches with testsuite update are welcome.

I'd rather see opensuse-specific macros implemented on other OBS distributions projects. It's cleaner and more useful.

OK, it seems I took it completely wrong and I'm talking about different problem :)

As this is just pretty corner-case of the spec writing we won't work toward this. But if someone provides patches they will be merged.


[test] Add test codes

Changes

Test codes for integrationConfig & integrationJob types
Test code for github webhook parsing
Github action for webhook and apiserver image build
Refactor meaningless codes.

Submitter Checklist

[ ] Docs included if needed
[x] Tests included if needed
[x] Follows the good commit messages standard

/approve


Changing the difficulty in charting mode crashes game

Describe your bug here.
Basically, if you try to change the difficulty, the game crashes.
How to reproduce the bug:

Go into charting mode
Change the time to any time
Try changing the difficulty

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Other than a Lua script that says "Hello World," none.

Sir, how the frick do you change the difficulty in the chart editor? Because it's impossible.


Asking to truncate to max_length but no maximum length

2024-02-29T06:34:41.018 app[17816011be4689] ord [info] Asking to truncate to max_length but no maximum length is provided and the model has no predefined maximum length. Default to no truncation.

Should we worry about this log? We've seen that you pushed commits that truncates to longest_first on torch, but not released yet, could be related Michael?

Not related to Infinity.
We're running https://huggingface.co/naver/efficient-splade-VI-BT-large-doc model with transformers package, and indeed there's not max_length in the config.json of the model


404 issues

I specify my posterType, but I still get 404s for all the other types.
$('.topHeader').vide({
      mp4: 'headerResources/light.mp4',
      poster: 'headerResources/light.jpg',
      posterType: 'jpg'
    });

GET http://localhost:9000/headerResources/light.gif 404 (Not Found)
GET http://localhost:9000/headerResources/light.jpeg 404 (Not Found)
GET http://localhost:9000/headerResources/light.png 404 (Not Found)

Can you make it stop looking for other ones, when the poster is defined please?

Hi.
$('.topHeader').vide({
  mp4: 'headerResources/light',
  poster: 'headerResources/light'
}, { posterType: 'jpg' });

Awesome, thanks! Is there a reason why all the options can't just be in one object?

The first param is required, the second is not.

Well couldn't it still all be in one object, just with some of the properties required and others optional? I suppose it's no big deal either way. Just a bit confusing to me.

It so happened.