SLACK_BOT_SCOPES and SLACK_USER_SCOPES should not be Env variables

Maybe I'm missing the intention of making scopes environment variables but I can see some clear disadvantages:

Using Slack APIs sometimes requires an update to the scopes for an application. They should be written into the code as constants so that they can be tracked across version control. Rarely are scopes environment specific between development, QA, and prod.
Requiring a global set of scopes prevents the developer from using a limited subset of scopes for some users and a more expansive set of scopes for others. Scopes as an environment variable prevent variations at runtime.

@zhawtof The intention of having the values in env is mainly to manage app configuration in one place.

Requiring a global set of scopes prevents the developer from using a limited subset of scopes for some users and a more expansive set of scopes for others. Scopes as an environment variable prevent variations at runtime.

When having multiple patterns of installations, you can override the scope variables inside your fetch function. Also, the scope variables are optional, so you don't need to set them if you don't want to have them in env. Here is a simple example code to override them:
export default {
  async fetch(
    request: Request,
    env: Env,
    ctx: ExecutionContext,
  ): Promise<Response> {
    env.SLACK_BOT_SCOPES = "commands,chat:write";
    const app = new SlackOAuthApp({
      env,
      installationStore: new KVInstallationStore(env, env.SLACK_INSTALLATIONS),
      stateStore: new KVStateStore(env.SLACK_OAUTH_STATES),
      oidc: {
        callback: async (token, req) => {
          const handler = defaultOpenIDConnectCallback(env);
          return await handler(token, req);
        },
      },
    })

I hope this clarifies.

Thanks Kaz. I didn't think about being able to override the env variable. That makes a lot more sense


Trafic source stat

close #48

Codecov Report

Merging #51 into master will decrease coverage by 2.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #51      +/-   ##
==========================================
- Coverage   19.55%   17.52%   -2.03%
==========================================
  Files          10       10
  Lines         537      599      +62
==========================================
  Hits          105      105
- Misses        418      480      +62
  Partials       14       14

Impacted Files
Coverage Δ

store/postgres/user.go
22.66% <0.00%> (-17.81%)
:arrow_down:

store/postgres/postgres_errors.go
0.00% <0.00%> (ø)

store/postgres/chat.go
40.00% <0.00%> (+1.19%)
: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 f9a0ef9...3f3783e. Read the comment docs.


Update label-commenter.yml

Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

LGTM

approve

LGTM


add select to copy option

first of all: thank you for the awesome terminal, I am loving it so far.
One thing I keep stumbling upon however, is the lacking support for select to copy (at least I couldnt find any option to turn it on).
I use this feature a lot in other terminals and additional button presses are always a bit annoying.
If I already move my mouse to mark something, I usually always want it to end up in my clipboard.

I realized that this is just a mirror. Ill raise an issue upstream.


rename htlc transfer command

Codecov Report

Merging #50 into dev will increase coverage by 0.01%.
The diff coverage is 0.00%.

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


Dev package overwrites version in package

Describe your environment

MacOS
Python version: Python 3.6.3
Pipenv version: pipenv, version 9.0.0

Expected result
dev-packages do not override packages. In the example below, Django 1.11.6 should be the only version included in the lockfile.
Actual result
max:tmp pete$ pipenv install django==1.11.6
Creating a virtualenv for this project…
⠋Using base prefix '/usr/local/Cellar/python3/3.6.3/Frameworks/Python.framework/Versions/3.6'
New python executable in /Users/pete/.virtualenvs/tmp-n8UXe2BB/bin/python3.6
Also creating executable in /Users/pete/.virtualenvs/tmp-n8UXe2BB/bin/python
Installing setuptools, pip, wheel...done.

Virtualenv location: /Users/pete/.virtualenvs/tmp-n8UXe2BB
Creating a Pipfile for this project…
Installing django==1.11.6…
Collecting django==1.11.6
  Using cached Django-1.11.6-py2.py3-none-any.whl
Collecting pytz (from django==1.11.6)
  Using cached pytz-2017.3-py2.py3-none-any.whl
Installing collected packages: pytz, django
Successfully installed django-1.11.6 pytz-2017.3

Adding django==1.11.6 to Pipfile's [packages]…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Updated Pipfile.lock (548cfe)!
max:tmp pete$ pipenv install django-debug-toolbar --dev
Installing django-debug-toolbar…
Collecting django-debug-toolbar
  Using cached django_debug_toolbar-1.9.1-py2.py3-none-any.whl
Collecting sqlparse>=0.2.0 (from django-debug-toolbar)
  Using cached sqlparse-0.2.4-py2.py3-none-any.whl
Requirement already satisfied: Django>=1.8 in /Users/pete/.virtualenvs/tmp-n8UXe2BB/lib/python3.6/site-packages (from django-debug-toolbar)
Requirement already satisfied: pytz in /Users/pete/.virtualenvs/tmp-n8UXe2BB/lib/python3.6/site-packages (from Django>=1.8->django-debug-toolbar)
Installing collected packages: sqlparse, django-debug-toolbar
Successfully installed django-debug-toolbar-1.9.1 sqlparse-0.2.4

Adding django-debug-toolbar to Pipfile's [dev-packages]…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Updated Pipfile.lock (0c8a93)!
max:tmp pete$ grep \"django\" -A5 Pipfile.lock
        "django": {
            "hashes": [
                "sha256:7ab6a9c798a5f9f359ee6da3677211f883fb02ef32cebe9b29751eb7a871febf",
                "sha256:c3b42ca1efa1c0a129a9e863134cc3fe705c651dea3a04a7998019e522af0c60"
            ],
            "version": "==1.11.6"
--
        "django": {
            "hashes": [
                "sha256:af18618ce3291be5092893d8522fe3919661bf3a1fb60e3858ae74865a4f07c2",
                "sha256:9614851d4a7ff8cbd32b73c6076441f377c45a5bbff7e771798fb02c43c31f47"
            ],
            "version": "==2.0"

Django 1.11.6 (the expected package) is there, but Django 2.0 is incorrectly included in the dev-packages.
Steps to replicate
From an empty directory:
pipenv install django==1.11.6
pipenv install django-debug-toolbar --dev

Hi @ipmb, thanks for reporting.  We are aware of this bug which was first reported in #1172, and it is fixed in master by #1173 and scheduled for a hotfix release later today.  Sorry for the inconvenience (on a personal note, I introduced the bug, I really AM sorry!) and thanks for your patience.  In the meantime I'm going to close this issue-- expect a fix to hit PyPI shortly

This issue still exists in 9.0.1. To reproduce:
pipenv install Django==1.11.6
pipenv install django-debug-toolbar --dev
pipenv install --dev
pipenv run pip freeze | grep Django

Expected result: Django==1.11.6
Actual result: Django==2.0

Open issue at #1220


add ics-provider config

Adding ics-provider config to enable usage of ics images on heighliner.

Meant to do this PR  as a draft , please ignore for now .


Refactor reading of source information

The goal is to avoid that the source definitions are loaded with every call of Scientist(). In Scientist(), it is read in order to add the source names if someone provides source IDs on their own. In Original(), it is used to define the search group.
Possible solutions:

Drop adding the source names in Scientist(); load the source information files during .search_group_from_sources()
Load the source information files when calling add_source_names(); load the source information independently during .search_group_from_sources()

Perhaps it is possible to read the source information files once and store them in Scientist() (self) to avoid multiple readings. If this is the case, option 2 is to be preferred.
In the end, we should make the reading of files such that get_field_source_information() is called automatically when the files aren't there. This avoids the mandatory call of get_field_source_information().

The field & source information is used in more placed. Notably, to determine someone's field. That happens in Scientist().__init__. So we cannot drop this.


Minimize the kyber lib dependency

This PR minimizes the dependency on the kyber lib by moving its essential parts to the go/internal/group directory. Even though a lot of the code remains unchanged some modifications have been made:

dropped the support for little endian encoding
replace the XOF interface by cipher.Stream for randomness source
add support for NIST P384 and P521 curves (with extending tests and benchmark coverage for them)
added safety checks for UnmarshalBinary to avoid panics
change the way random scalars are picked: https://github.com/smartcontractkit/tdh2/compare/main...ec#diff-3d0a7d4cdafd5e1c2d5e95dcb8e838d3a873e8ce0dfe97554b355005b2884cabR142
change the way random points are picked: https://github.com/smartcontractkit/tdh2/compare/main...ec#diff-553a172090c8ffffad366cd56400e20334d1250cbde57439df4d90a45b541361R68

(The last two could be reviewed more carefully.)

Thanks @coventry for reviewing!


Win azure

Pull request is only for comparison to master branch, somehow the pre-commits fail on this branch, don't know why.

@hmgaudecker so windows still fails after removing the latex stuff.. do not really understand wafs output though...

All tests seem to fail because dot is not found:
ngraphviz.backend.ExecutableNotFound: failed to execute [\'dot\', \'-Tpng\', \'-O\', \'C:\\\\Users\\\\VssAdministrator\\\\AppData\\\\Local\\\\Temp\\\\pytest-of-VssAdministrator\\\\pytest-2\\\\test_waf_build_r0\\\\cookies\\\\bake00\\\\research_project\\\\bld\\\\out\\\\figures\\\\dag\'], make sure the Graphviz executables are on your systems\' PATH\r\n'

Does that setup work fine on your machine? I does look like graphviz installed okay, weird...

It indeeds also fails on my machine. This is rather new though... A month or so ago, it was still working.
Maybe related to https://github.com/conda-forge/graphviz-feedstock/issues/34.

Indeed an issue of the graphviz version, fixing it at 0.8 resolves this issue.

But we seem to have a new problem. The latex installation on mac seems to take too long, so azure cancels it. Should I remove the latex stuff for mac too.

Fair enough. Should be trivial to do now, right?


Fix bug when filling other slots from initial intend or when requested slot is of a different type.

Proposed changes:

When explicitly filling slots other than the requested one, in extract_other_slots the entity type of the requested slot does not matter. Instead the entity of the slot to be filled should be checked. This PR fixes that.

Consider the following example:
Domain.yaml
slots:
  animal_liked:
   type: text
forms:
  test_form:
    animal_liked:
    - type: from_entity
    role: liked
    entity: animal

We expect that if the user message i like [dogs]{"entity":"animal", "role":"liked"} opens form test_form, animal_liked is filled.
The observed behavior leaves the slot blank, since request_slot is none.
Status (please check what you already did):

[ ] added some tests for the functionality
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)
documentation update is not needed

 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.Skubch Hendrik 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.

Thanks for opening a draft pull request 🚀If you have any questions, you can direct them to @amn41 ✨


Note proper 0.22.0 release version in podspec

With the release of v0.22.0, the podspec was still pointed to a release candidate version. This should be updated to reflect the proper version number so people's Podfile.lock file pins to the stable release.

Thanks for looking into this, I'll patch it today and release 0.22.1

@bestander Sorry, couldn't get this to 0.22, can we get this into 0.22.1 ? - https://github.com/facebook/react-native/pull/6546

@satya164 #6546 does not sound that urgent.
Is it just a wording change?
When it is in master let's discuss it.

@andrewsardone thanks for pointing this out.
The bug was in a wrong value being in podspec file in this branch.
But the fix is actually reverting version to 0.0.1-master because that is what the script is expecting.
After this builds https://circleci.com/gh/facebook/react-native/4253 we have a release 0.22.1

@bestander We have releasing HMR for the first time in Android, I just think that it's confusing to show an error message for iOS on Android.

You have a good point, @satya164


Replace error message string by key/value translations

There is an error message:
"There is already a citizen using " in the sign up page if the already registered user is trying to register again. This error message is not included in the list of translation variables in en.json and therefore does not change for other languages.
There is another variable present:
"User already exists with name %s"
which might have been intended for this situation but it's not used here.
Regards,
Taras.

We will take a look.
https://github.com/DemocracyOS/app/blob/development/lib/models/citizen.js#L79

I once tried fixing this but there was some issue with the translations engine being used in a mongoose plugin and was too lazy to fix that. But yea, we should do something about that.

Closed by c2e5fcb2eb58b365d5c08c809ce6937156060c45


[Question] Can't manage to use protect_content on sending messages.

I'm trying to send messages and protect them from forward and save.
My code:
bot.sendMessage(chatId, messageText, {     parse_mode: "Markdown",     protect_content: true,     reply_markup: JSON.stringify({       inline_keyboard: myInlineButtons     }),   });
Lib version:     "node-telegram-bot-api": "^0.56.0",

Hi @tambu22 The method works like you did, and also the content have this flag (you can see it on reply to the message in your bot. The problem of the "forward" and "save" is more Telegram Clients (how it works). And yes, a protected content is possible to save in your device.
For example in Telegram MacOs when you forward a protected message you have something like this:

But if you click on retry the client send the message as copy... So yes, the protected content implementation from Telegram is something weired.


C_CPP: Text Files and Binary Files

Description 📜
Explained about types of data with program as an example. Also, explained about Text file and binary file with examples.
Fixes #3578

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] C/CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@deeqakkk Hi. I have planned to add the content in the next file which will be better to understand. I have already created an issue for it.

@deeqakkk @vasu-1 Can you check please?

Looking forward to get more PRs from you on this topic @124015001


Chore(RBAC): minimize and explain the admin rbac and operator permissions

Signed-off-by: udit<EMAIL_ADDRESS>Proposed changes

Add and explain minimum permission for operator and litmus-admin
Remove Litmus Portal Beta4 as it is not required now.

Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the commit for DCO to be passed.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Dependency

Please add the links to the dependent PR need to be merged before this (if any).

Special notes for your reviewer:

PR has been tested.


Support customizing thumb, track and progress colors for slider on Android

Motivation
Ability to customize slider colors is desirable to match the brand. Currently iOS allows using images for slider parts, but android doesn't have any customization options. This PR adds support for customizing thumbTintColor, trackTintColor and progressTintColor.
Test plan (required)
Run UIExplorer example with the changes and verify everything works fine.

cc @brentvatne

@janicduplessis kept the same names as iOS for now. should rename them in new PR

@facebook-github-bot shipit

@facebook-github-bot shipit

@satya164 Can you try updating to master again, looks like tests are failing because of an unrelated flow error that is fixed now.
@mkonicek Could you check why this wont import?

It's failing to pass our internal UIExplorer Snapshot Tests for the Slider component. I'm looking into what I can do to help unblock this - I'll probably need to update the snapshot image for the Slider as part of this diff manually.

@satya164 this is failing a Slider snapshot test in the UIExplorer. It's likely that this screenshot no longer matches now that Slider has been updated. Can you take a look and see if the screenshot needs to be updated, or if the change was unintentional?

@hramos yes, the screenshot needs to be updated. the only changes from the screenshot are removal of the text (ios only)

Waiting for @satya164 to update the screenshot so tests pass.

@mkonicek updated

@facebook-github-bot shipit


Build errors with Xcode 13.2

Describe the bug
Just a heads up, as Xcode 13.2 is still in beta, but we were trying to build an iOS project with mqtt-nio as SPM dependency, which fails when using Xcode 13.2. It throws the following build errors:

TSTLSConfiguration.swift:11:11: 'tls_protocol_version_t' is only available in iOS 13.0 or newer
TSTLSConfiguration.swift:45:35: 'tls_protocol_version_t' is only available in iOS 13.0 or newer
TSTLSConfiguration.swift:48:35: 'tls_protocol_version_t' is only available in iOS 13.0 or newer
TSTLSConfiguration.swift:65:28: 'tls_protocol_version_t' is only available in iOS 13.0 or newer
TSTLSConfiguration.swift:66:28: 'tls_protocol_version_t' is only available in iOS 13.0 or newer
TSTLSConfiguration.swift:65:54: 'TLSv10' is only available in iOS 13.0 or newer

Apple's docs on tls_protocol_version_t also list that it is available starting from iOS 13, so my guess is that this happens due to the Package.swift defining iOS 12 as platform: https://github.com/adam-fowler/mqtt-nio/blob/main/Package.swift#L7
To Reproduce
Steps to reproduce the behavior:

Integrated mqtt-nio into an iOS project via SPM
Build the project using Xcode 13.2

Expected behavior
The build should succeed.
Context (please complete the following information):

Platform: iOS
Swift/Xcode Version: Xcode 13.2.0 (13C5066c)
Version: 2.1.1

That's interesting with Xcode 13.1 there are no issues. I can kinda see why, given the related functions are gated on iOS 13.

Yeah definitely. I was also surprised when I looked into it and saw that the respective lines were not changes in 12 months, and Xcode 13.0 and 13.1 worked fine

PR #94 resolves this but I have also contacted Apple is the hope they can revert this change. Breaking changes between point releases of Xcode shouldn't be acceptable. Hopefully I'll get a response.
If not I'll merge the PR. It generates a slight breaking change  which is frustrating but then maybe I shouldn't have included this symbol in my public interface.

I just tested the fix in our project and can verify that the build error is gone. Thanks for the quick help 🙏🏻

This is released in v2.5.0


Defines SIMSIMD_CAN_COMPILE_F16 and SIMSIMD_CAN_COMPILE_BF16 unused?

I find this in CMakeLists.txt
# Check for `__fp16` support
check_c_source_compiles(
    [=[
int
main(int argc, char *argv)
{
  __fp16 a = 1.0;
  __fp16 b = a + a;
  return 0;
}
]=]
    SIMSIMD_CAN_COMPILE_F16
)

# Check for `__bf16` support
check_c_source_compiles(
    [=[
int
main(int argc, char *argv)
{
  __bf16 a = 1.0;
  __bf16 b = a + a;
  return 0;
}
]=]
    SIMSIMD_CAN_COMPILE_BF16
)

if (SIMSIMD_CAN_COMPILE_BF16)
    add_compile_definitions(SIMSIMD_CAN_COMPILE_F16)
    add_compile_definitions(SIMSIMD_CAN_COMPILE_BF16)
endif ()

but defines SIMSIMD_CAN_COMPILE_F16 and SIMSIMD_CAN_COMPILE_BF16 are nowhere else used in the repository. Is this a leftover?

Let's call it a work in progress, @rschu1ze 😄
In Rust I try compiling with different configs until one of the passes. In C/C++ we should probably use those value to infer if the native f16/bf16 implementation should be used, or if they should be emulated. But the CMake has to be extended to also include _Float16, that some compilers support.

If you have a patch in mind, please feel free to submit a PR 🤗

Well, it I had to decide, I'd keep the detection logic for compiler-specific types __fp16 and _bf16 as is, and instead wait for the new float types introduced with C++23. They made them (unfortunately) optional, and (also unfortunately) Clang doesn't provide them yet (here, P1467R9). At least they will be standardized across compilers and they will come with feature detection macros that can be used in the source code.
Anyways, perhaps C++23 is too aggressive for simsimd. I'd be good with closing this issue.


⚠ ClamAV has degraded performance

In 01c630c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 182 ms

Resolved: ClamAV performance has improved in 77775b9.


Adjust gas consumption for external handlers

Tested with hardfork

Using gas GasMetering.LoadStorageGasCost for any db read
Using gas GasMetering.SaveStorageGasCost for any db write
Added GasMetering.InvokeContractGasCost = 7_000_000 for DoInternalCall and InvokeContract methods
Added GasMetering.SetReturnGasCost = 100 to set return
Added GasMetering.GetGasLeftGasCost = 100 to get gas left
Increased CopyFromMemoryGasPerByte, CopyToMemoryGasPerByte, RecoverGasCost, VerifyGasCost

Need to calculated hardfork height before release


Transparent  status bar

Hi,
I can not make transparent the status bar
This is the code:
            <com.daimajia.slider.library.SliderLayout                 android:id="@+id/slider"                 android:layout_width="match_parent"                 android:layout_height="match_parent"                 android:fitsSystemWindows="true"                 app:layout_collapseMode="parallax"                 android:scaleType="centerCrop"                 />
But with a simple ImageView works perfectly.
Suggestions? Thank you

its not library issue. you need to setup a scrolling activity. if you're using android studio create a new activity from activity gallery presets

put this slider layout inside of that header and keep the header non scroll-able aka fixed and done you're good to go

Hi @UsmanGhauri
Thank you for the reply.
Already use a com-scrolling layout and say so.
But still I can not make the status bar transparent.
This is the layout of the code:  http://pastebin.com/zmqVqfG2

Did you set your status bar to be transparent? if so then it shows properly. see the attached image i did as sample

Hi @UsmanGhauri ,
Sorry but not work for me.

If I replace your component with a ImageView, it works.

@ant-ia is it still scroll able? try scrolling up and see if its showing transparent status bar. in my case, if i stretch the header appbar layout to max height it shows status bar with colorprimary but when i scroll up, it shows transparent status bar untill it has no more space and inflates into toolbar with colorprimary

hi @UsmanGhauri,
I want to have pretty much the opposite. ie:
t0: transparent status bar (with image slider below)
t1: when scrolling up, the status bar and toolbar take on the coloration colorPrimary
This is an example of how I would like to behave. (Sorry if I repeat it, but with the layout I posted above and will work with ImageView)

sample-demo

@ant-ia : yes it acts the same way as you mentioned in the that sample-demo link. Only that 'status bar' not the toolbar, is colored when fully stretched as defined in the xml file. and when you scroll up both status bar and toolbar are colored to colorprimarydark and colorprimary

@UsmanGhauri
true, as you say above, it works correctly during scrolling.
But I can not get the status bar transparent in the initial state, when it's all extended, or I can not have this:
how I want
but I can get only so:
as is

This problem may be coming from your overall app theme. check your app theme should be extending an appcompat theme that has no actionbar so the library can set toolbar theme correctly

hi @UsmanGhauri  , @smilecs
Trying to use a theme no action bar, as you say, still does not work
theme:
    <style name="Theme.AppCompat.Light.NoActionBar"<EMAIL_ADDRESS>       <item name="colorPrimary">@color/colorPrimary</item>         <item name="colorPrimaryDark">@color/colorPrimaryDark</item>         <item name="colorAccent">@color/colorAccent</item>         <item name="windowNoTitle">true</item>         <item name="windowActionBar">false</item>         <item name="android:windowFullscreen">false</item>         <item name="android:windowTranslucentStatus">true</item>         <item name="android:windowContentOverlay">@null</item>     </style>


v7.0.0

This PR finalizes the rewrite from #1693 on this repo's v7-dev branch. This includes proper update of the mentioned issues from #1693, a solid testing and a great and updated documentation.
What's the goal with v7?
This version improves performance a lot, while at the same time reducing bundle size. No underscore anymore and the (outdated BS3) templates are now decoupled from the package, so you are highly flexible in choosing a theme template for your forms.
Also the list of extensions is curated to highlight deprecated/abandoned packages.
Why not finalize in #1693 ?
The #1693 was done by @pouya-eghbali (great job!) and come from his fork. In order to allow multiple contributions on v7 without the need of managing a second repo, we just merged it into v7-dev and continue to work on this until we have a final version.
TODO List from #1693 review

[x] replace all var with ES6 let or const and test of the scoping does not break anything
[x] rename unclear variables, like c to have meaningful names
[x] replace indexOf and -1 comparison on Arrays with includes
[x] isObject -> export const isObject = obj => Object.prototype.toString.call(obj) === '[object Object]'
[x] check for double/multiple declared functions and merge them into a single export
[x] Testing: update testing suite to meteortesting:mocha v2
[x] Testing: introduce GitHub actions to make CI maintenance easier
[x]  allow dynamic imports via process.env flag without break actual functionality
[x] the API docs need to be rebuild, since there are updated / changed jsDoc comments in the code

Additional bugs to fix, introduced with this PR:

[x] #1696

Post-release things to do

Discussion: make moment a weak dependency, replace it with luxon / day.s or even with native Date?
The documentation needs to be updated (review outdated extensions, remove links to the old demo)
The demo has been shut down, so we need a good usable demo (using npm meteor-build-client) to publish in GitHub pages

How can I help?
We need reviewers and testers as much as possible to make this a stable version. Breaking changes and breaking compatibility with extensions need to be documented, as well as newly added functions. Tests should be added where possible.

Big Improvements here
Okay this is getting interesting. By the time I was trying to move ES5 to ES6 code I was so unsatisfied with the fact, that I don't know if this breaks anything. I started writing tests until the point where we now have a comprehensive test suite for the API and the internals it uses (ArrayTracker, FormData, Hooks, Inputs, Validation etc.).
The tests are stubbing out external behavior (SimpleSchema, MongoObject), because that would be impossible for one person to implement with limited time and resources. However the current tests are quite solid (imo), so let me summarize what we have up to this point:

The package is easily testable now (see the updated sections in the new README)
229 Unit tests, covering API and internals
GitHub CI running on push and PR, for now only on v7-dev until the branch is merged into develop
API and the internals it uses are rewritten from ES5 to ES6
To make the internals more testable there were some exports added
I tried to not change any logic so there should nothing be broken (also tested in my current project that uses this form)
Found 1-2 bugs that have been revealed during the test writing that have been fixed + commented

It would be more than awesome if someone would take on a first pre-review and check, if everything works fine. There are also still some points left (see initial post of this PR) and there is still some work for prettifier to be done on the files and coverage would also be nice2have. However, I first wanted to have some stable tests in order to see if improvements won't break anything.
I will continue to work on the points left. In the meantime it would be awesome to have people to help with testing.

@jankapunkt Might be a good idea to do RC releases so it is easier more convenient for people to test and to get it on their radar (also announce on the forums).

Thank you @StorytellerCZ I would go forward with this as a 7.0.0-rc and will then make an announcement. However I am not allowed to publish here, so I will still need someone with publish permissions to be ready at that point.

@jankapunkt I could add you or I can do it.

This package is now ready to be published, I am just waiting to get the themes to be published first and then we can finally make it official.


KORE

please review

work on consolidating builtins
fixing issues with meta up/down, and reactivating part of the tests
moving syntactic sugar from tiny to kore -- so that it is available when using other rewrite engines
improvements to tiny -- unique labels and parallelization. it is now 3x slower then the Java rewrite engine

@dwightguth, please review

When you say 3x slower than the Java rewrite engine, are you comparing it against the --pattern-matching flag run with Nailgun? because If you can get it faster than that for the C semantics, I will cheerfully start using your backend to execute programs.

I think my backend is very far from executing any C program. At the very least, we'll need a KORE definition of C. The 3x is a very rough comparison: Brandon said he executed Chucky's "sum to 1000" example in about 12s on his notebook. I've executed the same IMP program in about 40s on mine. It's a ballpark comparison, it may be 2x or 10x slower in a comprehensive comparison.
My engine would actually fare worse with nailgun because I've already parallelized the matching. So there would be fewer processors available for the nailgun's courser-grained parallelization.
I haven't spent much time optimizing so I think there are still several low-hanging fruits for making it faster. So far I've only done some indexing, caching, and parallelization. There are likely fine-grained optimizations, like eliminating some cases for matchers, which can have large effects.
Finally, my purpose is not to beat the Java engine in terms of speed, but have an implementation that has everything I need for program transformation (i.e., assoc (and some AC) matching for any term, matches in side conditions, strategies).

I continue to believe that it is a good idea to experiment with a new engine for the purpose of nailing down the API of the rewrite engine.  And it may also evolve into a fast one, maybe some pushout of it with the existing one.
In the meanwhile, we still have Maude as a state-of-the-art competitor.  My daughter told me today that half of her class spent 30 min trying to solve the following problem: you are given numbers 5,5,7,4 and you can apply any arithmetic operation on them; can you obtain 24?  Of course, I was not able to solve while driving home, but as soon as I got home I wrote the following Maude program:
mod T is
  including RAT .
  sort MSet .
  subsort Rat < MSet .
  op _,_ : MSet MSet -> MSet [assoc comm] .
  vars A B : Rat .
  rl A,B => A * B .
  rl A,B => A - B .
  rl A,B => A + B .
  rl A,B => A / B .
endm

search 5,5,7,4 =>* 24 .
show path 634 .

The output is:
Solution 1 (state 634)
states: 635  rewrites: 4805 in 4ms cpu (3ms real) (1201250
    rewrites/second)
empty substitution

No more solutions.
states: 676  rewrites: 5306 in 4ms cpu (4ms real) (1326500
    rewrites/second)
state 0, MSet: 4,5,5,7
===[ rl A,B => A / B . ]===>
state 19, MSet: 1,4,7
===[ rl A,B => A - B . ]===>
state 197, MSet: 4,6
===[ rl A,B => A * B . ]===>
state 634, NzNat: 24

So 4 miliseconds!  We have a long way to go.

I wish some day to write K programs like this:
  import BAG{RAT}
  rule A,B => A*B \/ A-B \/ A+B \/ A/B

The RHS of the rule above is a correct ML pattern.  Also, if there is only one module, no need to have module and endmodule, so the above is the complete program.  Then "prove" CTL formulae like
prove 5,5,7,4 -> E F 24

(that is, 5,5,7,4 implies there exists (E) a path on which in the future (F) we see 24).

Regarding the Maude example, it is a bit tricky... unless paying the huge heating/cooling price, it requires anywhere AC matching. My engine doesn't support that yet as it is tricky to implement. I will think about it.
The \/ rhs looks cool.

 It is crazy slow, about 10s, but I haven't implemented any optimizations.
You can have a look at the code here:
https://github.com/kframework/k/blob/kore/tiny/src/test/scala/org/kframework/tiny/GrigoreChallange.scala
There are two versions of the module there: one with normal rules (and extra variables for matching) and one with anywhere rules.
Also, there are two tests for each of them, one doing rewriting and showing all possible final results, and another searching for final pattern (in this case 24).
The commit basically adds full AC (modulo any bugs I might have planted) and anywhere (so anywhere-AC as well) matching.
Regarding speed, I think I can easily cut about 10x of the time, and another 10x with reasonable effort, but that would bring us to 100ms, still very far from Maude's 4ms.

This is very cool!  So the Rewriter class is your tiny rewriter?  How does it know that ~ is infix?  Are you assuming that for all AC operations?  Note that this is not required in Maude, for example, you can write things like f(a,f(b,c)) where f is an AC operation.  Thanks @cos .

This is very cool! So the Rewriter class is your tiny rewriter?

Yes.

How does it know that ~ is infix?

That's just Scala syntactic sugar. It's the same as writing KApply(KLabel("~"), a, b). Basically, I made a few common operators applicable to K as infix (https://github.com/kframework/k/blob/kore/kore/src/main/scala/org/kframework/kore/ScalaSugar.scala#L25-L36). There's also more sugar transforming Java/Scala primitives to KTokens, and allowing easy instantiation of KApply like this 'foo('bar('foo()) + 5, "I'm a string").

Are you assuming that for all AC operations? Note that this is not required in Maude, for example, you can write things like f(a,f(b,c)) where f is an AC operation.

Only the sugar is infix -- the rewriter itself doesn't care about syntactical issues like ~ being infix. I used ~ because I wanted to use infix syntax for clarity, and the , cannot be used as an operator in Scala.


Can't install ZXingObjC either Cocoapods or download zip

Hi all,
I encountered some problems on installation of ZXingObjC.
Firstly, I tried to put pod 'ZXingObjC', '~> 3.0' into my podfile and run pod install. The following error message is shown:
error: RPC failed; result=18, HTTP code = 200

fatal: The remote end hung up unexpectedly

fatal: early EOF

fatal: index-pack failed

Then I tried to Download ZIP directly. However, I got error when trying to unzip it. The following is the error message.
Archive:  ZXingObjC-master.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of ZXingObjC-master.zip or
        ZXingObjC-master.zip.zip, and cannot find ZXingObjC-master.zip.ZIP, period.

Anyone also encountered these problems and got the solution? Please help. Thank you so much.

I also encountered this problem


Please add re-send only to usf_share_run.py

Currently re-sending a run requires a complete re-encryption of the data.
Please add this so it will only re-send the email if the data is still there

Can be option, even better if it's auto-magic 👍


[Task][scripts]: use run-java.sh to start backend and deprecate start-backend.sh

Is there an existing issue for this?

[X] I have searched the existing issues

With the introduction of 19218 there is a duplicate script  (start_backend.sh) which does the same job as run-java.sh to avoid redundance of scripts we introduce the change
SubTasks
No response

19420 PR which fixes the issue


auto iterator returned by find to handle value depending if is string or numeric.

I want to make a function which finds a key by name in a json object it works fine when the value is double, but if the value comes as "130.34" it fails, what i need is a way to determine its type if is coming as string then to convert it to double.
the values may come as directly double or as double but in a string ex. "130.34"
double jfind(json jdata, char* keyName){
     auto keytofind = jdata.find(keyName);

// Check if  *keytofind is string or numeric ????? how to do it???

     if(keytofind != jdata.end()){
        return *keytofind;
     }else{
        return 0;
     }
}

What exactly did you find confusing? There is a lot of documentation around, see https://nlohmann.github.io/json/
double jfind(const json& jdata, const char* keyName)
{
    const auto it = jdata.find(keyName);
    if (it != jdata.end() && it->is_number())
    {
        return *it;
    }
    else
    {
        return 0.0;
    }
}


change font color

How can i cnange font color

have you tried changing the background color of the theme? (right click->options -> Looks -> theme)?

I forgot why that time I can't change font color, but I fixed it now, I think maybe is WSL's problem or just I am too fool that time to do wrong setting, all in all, I fixed it, thx !


Clean up datastore options API

hadoop input and output formats are an important external interface, and setting "datastore name" in addition to a map<String,String> is clumsy. Instead you should be able to just set the datastore plugin options. Also, datastore "name" and "type" were being used interchangably which adds to confusion. Settled on using "type" everywhere.

Included in 0.9.3


Create local cache of messages

Is your feature request related to a problem? Please describe.
When reading messages Mailchain checks the remote sent store for every API call. This is inefficient, prevents the recipient easily preserving the messages, and requires an internet connection.
Describe the solution you'd like

[ ] Implement a cache interface of Sent interface in package stores.
[ ] It should be a Read through cache
[ ] Use github.com/spf13/afero to interact with filesytem
[ ] Store messages in original format

Hi @robdefeo , may I work on this?

Hi @olimpias yes absolutely I took a recent look at this and have updated the issue to with a bit more detail. Any questions just reach out. I'll assign this to you now

Hi @robdefeo , should I define a setting for cache time? LIke MailboxState struct in settings package. I observe that we are reading configurations from a yaml file. I guess its worth to add caching time there as well. What do you think? Or I can make it hardcoded for now and I can create a separate MR for this purpose.

closed by #613


JAVA-1605: Refactor request execution model

Motivation:
Request types encapsulate their sync and async result types. This is too
restrictive: not all requests will have exactly those two types of
results. For example, if we add a CQL reactive processor, it would be
nice to reuse the Statement classes, not create new ones.
Modifications:
Make Request non generic. Change the generic Session execution method
to: execute(RequestT request, GenericType resultType)
Result:
Each request processor now handles exactly one (RequestT, ResponseT)
pair, so it is possible to add new response types for existing requests.
As a consequence, the number of built-in processor has doubled
(sync/async variants for each).

What do you think about the idea of being able to register RequestProcessors into RequestProcessorRegistry on the fly?   Say for example, I want to take an existing Cluster  and decorate in functionality via processors.  Or would you prefer that is all spec'd out beforehand?
The nice benefit of being able to do it on the fly:

Don't need to implement custom InternalDriverContext requestProcessorRegistry(), you can just wire in functionality post-hoc.
Might make it easier to add separate things that provide functionality (Reactive support, DSE support, etc. can all be wired in separately).

Although I can't see a reason why that all could be done beforehand, so maybe this isn't the best idea, just brainstorming :)

What do you think about the idea of being able to register RequestProcessors into RequestProcessorRegistry on the fly?

The biggest drawback is that this would require some form of synchronization, and processor selection is done on the hot path. So I'd rather keep an immutable structure.

need to do a little work on the maven side to facilitate the use of non-shaded guava in the tests.


:bug: check mgh deleting in crd controller

Summary
Related issue(s)
https://issues.redhat.com/browse/ACM-13478
Fixes #

traced in https://github.com/stolostron/multicluster-global-hub/pull/1064


Generator use...

Consideration on generator use: (or equivalent async generator)
Mostly here for me to not forget about this idea later, might decide it's not being added and close this.

class GenCYA(ChooseYourOwnAdventure, Generator):

    def __iter__(self):
        return self

    def send(self, v):
        if v is None:
            try:
                return self.get_prompt()
            except AlreadyDone:
                raise StopIteration
        else:
            self.turn_page(v)
            if self.check_if_done():
                return self.get_result()
            else:
                return self.get_prompt()

    def throw(self, type=None, value=None, traceback=None):
        raise StopIteration

Won't be doing this. It's neat conceptually but it's not any better than handling it the current way and adds a higher level of understanding needed for potential contributors.
Example above also has a bit of a sharp edge in having the generator yield the result which is allowed to be a different type and may need other handling, if it were ever added, the actual result should be fetched from the consumed generator instead.


[Bug] Shadow flickering on dynamic rigidbodies

Sample project:
https://playcanvas.com/project/393993/overview/shadow-flickering-bug
Just hit play and see the shadow flickering.
Adjusting shadow distance and bias wasn't able to eliminate the issue.

Turns out that happens if directional light is looking strictly down, i.e, with (0,0,0) rotation. If you rotate is slightly, for example to (-1, 1, 0) - the issues gets solved.

Definitely one for @guycalledfrank

Should be fixed in https://github.com/playcanvas/engine/issues/575


Give visual indication of which indicators have been added

If there are already indicators added to the chart it would be nice to see in the indicators drop down which have already been selected. Possibly some highlighting or a checked icon of some sort. Same goes for the selected type (candlestick, OHLC etc.)

:+1: Definitely agree with this - this has be bugging quite a few people for a while.
I'll organise some UX involvement, then look at getting this implemented.

Likewise, I totally agree with this one :) I mentioned it to james and we made a note of it so we could discuss it with the UX lot

Desired behaviour:

Duplicated by #718


Lowering aten.index.Tensor

Ticket
#535
Problem description
for example, if input.shape = (3, 4, 5), indices = [tensor([[0, 1, 1]]), tensor([[2, 1, 2]])]
then its output is [[input[0][2], input[1][1], input[1][2]]]
So lowering aten.index to several getitem/reshape/cat
# before digest
input = torch.rand([3, 4, 5])
indices = [tensor([[0, 1, 1]]), tensor([[2, 1, 2]])]
output = aten.index.Tensor(input, indices)
return output_reshape

# after digest
input = torch.rand([3, 4, 5])
indices = [tensor([[0, 1, 1]]), tensor([[2, 1, 2]])]
indices_flatten = [aten.reshape(indices[0], (2,)), aten.reshape(indices[1], (2,))]
indexing0 = getitem(indices_flatten[0], 0), getitem(indices_flatten[1], 0))
output0 = getitem(input, indexing0) # input[0][2]
indexing1 = getitem(indices_flatten[0], 1), getitem(indices_flatten[1], 1))
output1 = getitem(input, indexing0) # input[1][1]
indexing2 = getitem(indices_flatten[0], 2), getitem(indices_flatten[1], 2))
output2 = getitem(input, indexing0) # input[1][2]
output = aten.cat(output0, output1, output2)
output_reshape = aten.reshape(output, [1, 3, 5])
return output_reshape

What's changed

Add DigestAtenOps and it digest aten.index to serveral torch ops

I'll merge #532 first then this PR

It is not the same as index_fill, right?
https://github.com/tenstorrent/tt-metal/blob/main/ttnn/cpp/ttnn/operations/index_fill/index_fill_pybind.cpp

It is not the same as index_fill, right?
https://github.com/tenstorrent/tt-metal/blob/main/ttnn/cpp/ttnn/operations/index_fill/index_fill_pybind.cpp

I think ttnn.index_fill is not same with aten.index.Tensor, former seems fill some specific value to the input tensor (not change the shape), latter is get part of input accroding to indices (change the shape)

@swimdi can you please share how is Tensor returned by index used later?
I see that in pytorch they don't seem to actually calculate much but rather build an iterator.
Maybe we can see the pattern of how index is being used and avoid it.

@swimdi can you please share how is Tensor returned by index used later?
I see that in pytorch they don't seem to actually calculate much but rather build an iterator.
Maybe we can see the pattern of how index is being used and avoid it.

Take beit for example, part of its code is
view_13 = torch.ops.aten.view.default(arg223_1, [-1])
index = torch.ops.aten.index.Tensor(arg1_1, [view_13]) # <= indexing here
view_14 = torch.ops.aten.view.default(index, [197, 197, -1])
permute_3 = torch.ops.aten.permute.default(view_14, [2, 0, 1])
clone_2 = torch.ops.aten.clone.default(permute_3, memory_format = torch.contiguous_format)
unsqueeze = torch.ops.aten.unsqueeze.default(clone_2, 0)
add = torch.ops.aten.add.Tensor(div, unsqueeze)
_softmax = torch.ops.aten._softmax.default(add, -1, False)
clone_3 = torch.ops.aten.clone.default(_softmax)
expand_3 = torch.ops.aten.expand.default(clone_3, [1, 12, 197, 197])
view_15 = torch.ops.aten.view.default(expand_3, [12, 197, 197])
bmm_1 = torch.ops.aten.bmm.default(view_15, view_16)

the output of aten.index.Tensor is then go through permute=>clone=>unsqueeze=>add=>softmax=>clone=>expand=>view and be the argument of bmm
I'm not sure how to avoid it, for this beit case, the input variation is
["Tensor<[732, 12]> self = ?", "List[Optional[Tensor]] indices = [<[38809]>]"]

This means the input shape is [732, 12], and based on the values in indices, it retrieves elements from the 732 elements (each of size 12) a total of 38,809 times. Finally, the output shape will be [38809, 12].

@swimdi can you find out what Pytorch function is lowered to this list of ATen ops? I have an impression that we can fuse this back to something that might be already present in TT-NN.

class BeitRelativePositionBias(nn.Module):
    def __init__(self, config: BeitConfig, window_size: tuple) -> None:
        super().__init__()
        self.window_size = window_size
        self.num_relative_distance = (2 * window_size[0] - 1) * (2 * window_size[1] - 1) + 3
        self.relative_position_bias_table = nn.Parameter(
            torch.zeros(self.num_relative_distance, config.num_attention_heads)
        )  # 2*Wh-1 * 2*Ww-1, nH
        # cls to token & token 2 cls & cls to cls

        # get pair-wise relative position index for each token inside the window
        coords_h = torch.arange(window_size[0])
        coords_w = torch.arange(window_size[1])
        coords = torch.stack(meshgrid([coords_h, coords_w], indexing="ij"))  # 2, Wh, Ww
        coords_flatten = torch.flatten(coords, 1)  # 2, Wh*Ww
        relative_coords = coords_flatten[:, :, None] - coords_flatten[:, None, :]  # 2, Wh*Ww, Wh*Ww
        relative_coords = relative_coords.permute(1, 2, 0).contiguous()  # Wh*Ww, Wh*Ww, 2
        relative_coords[:, :, 0] += window_size[0] - 1  # shift to start from 0
        relative_coords[:, :, 1] += window_size[1] - 1
        relative_coords[:, :, 0] *= 2 * window_size[1] - 1
        relative_position_index = torch.zeros(
            size=(window_size[0] * window_size[1] + 1,) * 2, dtype=relative_coords.dtype
        )
        relative_position_index[1:, 1:] = relative_coords.sum(-1)  # Wh*Ww, Wh*Ww
        relative_position_index[0, 0:] = self.num_relative_distance - 3
        relative_position_index[0:, 0] = self.num_relative_distance - 2
        relative_position_index[0, 0] = self.num_relative_distance - 1

        self.register_buffer("relative_position_index", relative_position_index, persistent=False)

    def forward(self) -> torch.Tensor:
        relative_position_bias = self.relative_position_bias_table[self.relative_position_index.view(-1)].view(
            self.window_size[0] * self.window_size[1] + 1, self.window_size[0] * self.window_size[1] + 1, -1
        )  # Wh*Ww,Wh*Ww,nH

        return relative_position_bias.permute(2, 0, 1).contiguous()  # nH, Wh*Ww, Wh*Ww

Yes, I think BeitRelativePositionBias will produce aten.index.Tensor, and actually is this line
self.relative_position_bias_table[self.relative_position_index.view(-1)]

and it can be represented as these code to reproduce
import torch
import ttnn
import torch_ttnn
device = ttnn.open_device(device_id=0)

class PatternModule(torch.nn.Module):
    def __init__(self):
        super().__init__()

    # represented by self.relative_position_bias_table[self.relative_position_index.view(-1)]
    def forward(self, input, indices) -> torch.Tensor:
        return input[indices]

m = PatternModule()
input = torch.rand([732, 12])
indices = torch.randint(0, 100, [197,197])
option = torch_ttnn.TorchTtnnOption(device=device)
m = torch.compile(m, backend=torch_ttnn.backend, options=option)
result = m.forward(input, indices)
nodes = list(option._out_fx_graphs[0].nodes)
print(nodes)
# [arg0_1, arg1_1, index_tensor, output]
print(nodes[2].target)
# aten.index.Tensor


[AIRFLOW-3177] Change scheduler_heartbeat from gauge to counter

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Currently, the scheduler_heartbeat metric exposed with the statsd integration is a gauge. I'm proposing to change the gauge to a counter for a better integration with Prometheus via the statsd_exporter
Rather than pointing Airflow at an actual statsd server, you can point it at this exporter, which will accumulate the metrics and expose them to be scraped by Prometheus at /metrics. The problem is that once this value is set when the scheduler runs its first loop, it will always be exposed to Prometheus as 1. The scheduler can crash, or be turned off and the statsd exporter will report a 1 until it is restarted and rebuilds its internal state.
By turning this metric into a counter, we can detect an issue with the scheduler by graphing and alerting using a rate. If the rate of change of the counter drops below what it should be at (determined by the scheduler_heartbeat_secs setting), we can fire an alert.
This should be helpful for adoption in Kubernetes environments where Prometheus is pretty much the standard.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason: No existing tests around statsd

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

The logic here sounds sensible to me. I think @feng-tao makes heavy use of the metrics from Airflow for monitoring.
At the very least we will need a note in UPDATING.md about this change.

Absolutely, just updated UPDATING.md with the change notice.

Codecov Report

Merging #4027 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4027      +/-   ##
==========================================
- Coverage   75.79%   75.77%   -0.03%
==========================================
  Files         199      199
  Lines       15946    15946
==========================================
- Hits        12087    12083       -4
- Misses       3859     3863       +4

Impacted Files
Coverage Δ

airflow/jobs.py
82.13% <100%> (-0.36%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM

Thanks @schnie Merging!


Duplicated UI with ray-x/lsp_signature

Hi, Is it possible to reuse the UI of ray-x/lsp_signature. Currently both of them is popping up when suggesting function signature.

neodev doesn't provide a ui. It just configure sumneko


Prevent stale MSYS gpg-agent.exe process blocking command control

This commit fixes:
https://github.com/microsoft/vcpkg/issues/5476
The issue is that CI environments such as Appveyor's VS2017 image will wait for all processes to complete. If a stale process resides as a result, builds will hang.
There does not appear to be any good reason for gpg-agent.exe to be running once the build of icu has completed.
Without this patch builds of icu4c using CI systems will very likely hang and not in an obvious way.
Is this the right solution to this problem? Probably not but it is one solution. And it degrades gracefully in that the build will not fail if gpg-agent.exe is not running. The gpg-agent.exe will not run again once MSYS has been configured, so to test this patch, a fresh install of vcpkg is required. Open the task manager and before the icu build completes, look for gpg-agent.exe just sitting there for no reason.
Might I suggest that the issue is fixed in vcpkg MSYS instead or as well?
Please don't request further from this commit.

No way I'm logging into another account just to look at an Azure pipeline log.
Especially as I am already logged into the Microsoft owned Github.
Please feel free to reject, refactor and or resubmit this commit. I'll leave the fork up for about a month.
The offer of this merge is a one time deal. I'm trying to migrate away from vcpkg if I can in the future and so, have little interest further patches. Thanks.

Hi @heydojo, thanks for your new PR. I see some regressions from the current CI system:

x64-osx
master
test
notes

icu
Pass
Fail
Regression

x64-linux
master
test
notes

icu
Pass
Fail
Regression

Please let us know if you need any help.
Also, could you please bump the version in CONTROL file?

@NancyLi1013 unfortunately @heydojo said "Please don't request further from this commit."... whoever has writing permissions to this PR should continue the work and finish it. It should be enough to wrap the added lines
vcpkg_execute_required_process(`
	COMMAND TASKKILL /F /IM gpg-agent.exe /fi "memusage gt 2"
	WORKING_DIRECTORY ${SOURCE_PATH}
)

inside a if(NOT VCPKG_CMAKE_SYSTEM_NAME OR VCPKG_CMAKE_SYSTEM_NAME STREQUAL "WindowsStore"), since osx and linux do not require 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.:x: heydojo sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution @heydojo !
I'm sorry that the project didn't meet your expectations. If you're willing to provide feedback regarding your decision to move away from vcpkg we would like to hear it, send us an email to<EMAIL_ADDRESS>or leave a comment here.


Adding http://courses.wishtack.com/angular-2 free online training to resources

Hi,
Is there a way to add my free online training http://courses.wishtack.com/angular-2 to angular.io/resources ?
For the moment, it's in french but will soon be translated to english :)
Thank you in advance.
Younes

@StephenFluin

Thx Jules

This is now live and I believe it can be closed.

@juleskremer @StephenFluin Thank you very much!!!


(Feature Request/Proposal) Natively Add Camera Preview Window When Clicking On A Camera

Tested versions

Applies to all 4.x branches

System information
All Avaliable Operating Systems
Issue description
Since the Redot Proposal page isn't available yet I would like to use this a way to send my suggestion.
Since there is already a plug-in called Little Camera Preview in the add-ons library, it would be nice if this version of the engine natively had a camera preview when you select any camera in the scene. That way, it's easier for developers to iterate much faster without having to download a plugin, use the clunky Camera Preview button, or build the game.
Steps to reproduce
Look at Little Camera preview's source code and re-implement it via GDExtention.
 It's a good feature, and I'm glad there's an addon for it at the least.

Yeah, I'm glad that we have the option in general as a plugin. However, it think it being a core feature within the editor or a bundled GDExtension when you open up Redot would be a great quality of life improvement. If anyone wants to transfer their project back to vanilla Godot, they can do so without anything breaking their project since that feature is more of a component than baked into the source.


Add support for YAML

What would "running" YAML do? Validate the structure?
Curiosity - is this for an Ansible playbook?

Not really. I was trying to test a crawler code for youtube and the file is .yaml
I’m using Atom to run the file.
Thank you for taking the time to read this,
Shaikhah B. Alkhadhr
On Jan 25, 2017, at 10:33 PM, Kyle Kelley <notifications<EMAIL_ADDRESS>wrote:
What would "running" YAML do? Validate the structure?
Curiosity - is this for an Ansible playbook?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rgbkrk/atom-script/issues/1227#issuecomment-275209231, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYIKNtovFOnTlxGHbk_8zh-X5aCOXq5gks5rV6OZgaJpZM4Lt5Ja.

Script operates by invoking some interpreter (runner) or compiler for standard languages. I'm curious if there's a way for you to use the script "run with options" command.

I did actually. Cmd+shift+I then I specified yaml as the command, but got the same result.
Thanks for taking the time to read this,
Shaikhah B. Alkhadhr
On Jan 26, 2017, at 1:36 AM, Kyle Kelley <notifications<EMAIL_ADDRESS>wrote:
Script operates by invoking some interpreter (runner) or compiler for standard languages. I'm curious if there's a way for you to use the script "run with options" command.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rgbkrk/atom-script/issues/1227#issuecomment-275255011, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYIKNvfvjYOCGJ7c9o_ZkRlHoA2GhWCFks5rV85sgaJpZM4Lt5Ja.


Make it easier to highlight description/label for filter on type in tree

Some tree rows that show files have a 'label' part for the filename and a 'description' part for the rest of the path. We should be able to search on both of these and highlight them separately but currently it's not easy.
Currently each row item has one string from getKeyboardNavigationLabel and the tree does the fuzzy matching and gives the renderer one filterData. The renderer wants to pass separate matches and descriptionMatches t the ResourceLabel but it's not easy to translate the one filterData into two parts.
It would be easier if

The row could return multiple keyboard navigation labels, and the renderer would then get multiple FuzzyScores
Or if the keyboardNavigationLabelProvider did the fuzzy matching itself and returns some result object that the renderer gets

Thoughts?

The specific inspiration for this was the search tree. But it's not a high priority thing.


Certificates get removed before server starts (TmpDir drop)

Fixes https://github.com/umccr/htsget-rs/issues/90
Looks like we have all green and grins ;)
Thanks @mmalenic for the race condition hint, this PR is a bit of a hack but will do for now.

Well, https://github.com/umccr/htsget-rs/runs/6803593182?check_suite_focus=true#step:6:90 was totally unexpected, working on it :-S


Calling layout on mCircleView twice

https://github.com/ashakirov/Horizontal-SwipeRefreshLayout/blob/d06dba9407b6e75b21de9367e83e8eafaa296369/Example/HSwipeRefreshLayout/src/main/java/custom/widget/SwipeRefreshLayout.java#L578
Are you sure you need to call layout method on mCircleView twice? I was comparing your code with the Official SwipeRefreshLayout and saw that they had called the layout method on mCircleView just once. It also does not make any sense to call it twice on the same view, thus overriding the effect of the first method call.
Did you intent to use mProgress here instead of mCircleView?

Yes, it looks like I wrote horizontal layouting and just forgot to remove original one. Thanks!


Composer update.  Updated .gitignore.  Updated php.xml .  Added @covers annotations to tests.

Composer update.
Updated .gitignore.
Updated php.xml to abide by phpunit's new configuration scheme.
Added @covers annotations to tests in the following files:
tests/interfaces/utilities/ObjectReflectionTestTrait.php
tests/interfaces/utilities/ReflectionTestTrait.php

Do not merge, composer.json is not valid.


Virtual Environment

somehow it is not installing dependencies.

If I add this
   virtualenv venv
   source venv/bin/activate

then it is installing dependencies

To run the project from the root directory
   uvicorn app.main:app --reload

Yes you need a virtual env to run it. Python >= 3.9. After that you do poetry install

I've added some notes to the README about this.


Auro disabled button

Alaska Airlines Pull Request
Summary:
This or will add support for the disabled button style. This pr also removes support for the beta style button test.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Orion Design System Team

:tada: This PR is included in version 4.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


希望可以增加 click http://xxxxx.com/image/button.png

Ruth，你好，我把 .tag 放到服务器后，发现点击图片命令还是读的本地图片，
希望可以增加 click http://xxxxx.com/image/button.png
支持点击服务器图片，这样整个流程所需文件都放服务器，运行起来也方便。

Hi @kangyiwen for visual automation, it is best if you can take a screenshot of the element in the same machine you are running the automation on. This is because different computers have different screen resolutions, so most of the time your screenshot based on another computer would not work. As such, it is best that the images are saved locally and are specific to that machine's screen resolution.

哦 谢谢，理解了。我的使用场景是 N台一模一样的WINDOWS电脑使用，所以会想到这个需求。

This is an interesting question, and asked for the first time by user. Technically this feature can be done, but it feels heavy because the TagUI .tag file will have many long lines of click https://.....
Another idea could be TagUI auto-downloading all image files if the .tag file that is executed is hosted online. However, this has problem with the usage scenario where users want to use that own local images with an online .tag file. It will overwrite their local image files, or fail when reading missing online image files.
I don't have a clear idea which way would be good to implement a feature like this. Any thoughts, Ruth and Yi Wen? Also, it might be not a major use case to add, compared to other new TagUI features on your roadmap, Ruth.
In the meantime, something like the following can be done. At the start of your TagUI .tag file, you can configure it to download the images required, using the curl tool that is included with TagUI.
curl -k -s -L -o image1.png http://xxxxx.com/image/image1.png
curl -k -s -L -o image2.png http://xxxxx.com/image/image2.png
curl -k -s -L -o image3.png http://xxxxx.com/image/image3.png
curl -k -s -L -o image3.png http://xxxxx.com/image/image4.png

-k to allow insecure connection if the server is not SSL secured
-s is to perform the download silently
-L is to automatically handle redirection if the link redirects to another URL.
-o is to point what file name to save as. Note that by default, without including the path, I think it saves to tagui/src folder.

我想到的方案：在 .tag 里，增加一个参数，可以有下面2个值：
img_source = 'local'
img_source = 'http://xxxxxx.com/img/'
img_source 如果是带http的，就默认代码内的图片前面带这个URL，例如：
click  button.png  实际运行 click http://xxxxxx.com/img/button.png
这样代码看着就不乱了，同时本地调试和服务器使用切换很方便。
.tag 发现需要点击服务器图片时，可以先建立个零时目录  例如 temp，然后把所有图片先下载到这个目录，然后继续执行，默认图片路径都在 temp 内。
或者，执行 click 时，自动先下载图片，保存为一个随机文件名.png，然后执行 click 随机文件名.png
接下去还有 click 时，还是这样操作，保存并覆盖 随机文件名.png，然后执行 click 随机文件名.png

@kensoh to confirm you're thinking of having the feature only be for hosted online tag files? Why only for online tag files though...

When I created the online .tag feature, I think the only other support is datatable csv file, if it is available in the same URL and same name as the .tag file. Being able to pull image files from online to run locally would be great to have.
But will have to weigh that amongst other items on the roadmap, in terms of the potential userbase benefitting from this feature vs other features in mind. We can explore more on tomorrow's Zoom call.

For visual automation most likely it won't be able to completely match an online image or an image screenshot taken on another computer due to resolution issues. I don't see a big use case for this, unless the hosted image can be used for another purpose other than visual automation.

In that case, maybe use the above curl method. Yi Wen can also consider zipping up all the image files so that only 1 line of run curl is needed inside .tag file.
And can combine with some unzip command to do the unzipping.
https://github.com/kelaberetiv/TagUI/issues/1213#issuecomment-1108233807

嗯，图片先ZIP，下载解压到本地，是个方法。


Do you have saved models available?

Could you provide us with saved models?
Thanks!

Our pretrained ATMC 8bit models are available at https://drive.google.com/drive/folders/1HDGkz8BKQGn6SFRPzaTt13hFOC8JbtUD?usp=sharing
You can evaluated them using {dataset}/inference_model.py . For example, use the following script to evaluate robustness against PGD attacks with eps=4 and iter=7 on SVHN dataset:
`
#!/bin/bash
attack_algo=( pgd )
dataset=svhn
atmc quant model:
bit=8
crs=( 0.1	0.05	0.01	0.005	0.003	0.001 )
crs=( 0.1 0.01 )
for aa in "${attack_algo[@]}"
do
for cr in "${crs[@]}"
do
python ${dataset}/inference_model.py
--gpu 0
--loaddir /hdd2/haotao/atmc/model/${dataset}/atmc_${bit}
--model_name ${dataset}atmc${cr}.pth
--prefix_name quant_closest_
--data_root /hdd2/haotao/atmc/data/${dataset}
--attack_algo ${aa}
--attack_eps 4
--attack_iter 7
--prune_algo l0proj
--prune_ratio ${cr}
--quantize_algo kmeans_nnz_fixed_0_center
--quantize_bits ${bit}
--exp_logger ${dataset}atmc${bit}.txt
--abc_special
--abc_initialize
done
done
`

Thanks a lot! Much appreciated.

Sorry my question wasn't specific enough, do you have CIFAR-10 models? Thank you!


[HOTFIX] fix java style errors

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@jackylk @ravipesala  please review and merge  this pr to fix build errors

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6317/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5155/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5273/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6318/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5156/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5275/

I just prepared to modify and found that you have modified ^-^.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6319/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5157/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5276/

LGTM


CheckoutConfigService not exported

CheckoutConfigService isn't available in public API. It should be exported.

Steps for QA:

Run clean angular app
Import Spartacus core libraries
Import CheckoutConfigService
Run & build application

@psmul Can I have a bit more info please?
I've imported Spartacus core libs in package.json
I've imported CheckoutConfigService in app.module.ts
But.. using a clean angular app, it should be out of the box, and available in spartacus core? At least that's what I understood...
So, I have this:

import { ConfigModule, CheckoutConfigService } from '@spartacus/core';

But looks like it's not there. Because I'm getting this:

error TS2305: Module '"../../node_modules/@spartacus/core/spartacus-core"' has no exported member 'CheckoutConfigService'.

but looking for the file, it looks like it would be more in the storefront lib. So I also tried this:

import { B2cStorefrontModule, defaultCmsContentConfig, CheckoutConfigService } from '@spartacus/storefront';

And it still does not work for the same reason.
So my question is, where should I be importing it? And how?
And if I import it, I should be using it somewhere?

@KateChuen Okay, after consultation with a team, testing this one will be more tricky that I assumed. We will take care of testing this one, as it's impossible to test it now via simple import.

Checked and works. We can now use paths like the following with CheckoutConfigService included:
import { CheckoutConfig, CheckoutStep, CheckoutConfigService } from '../'


rm kind frontmatter - cws

What does this PR do?
removes kind frontmatter from pages ingested by docs site (agent expressions and backend)

Motivation
the kind param is no longer a user-defined frontmatter in newer versions of Hugo. it is built-in to be one of home, page, section etc.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

perfect, thanks !

/merge


Communication Rules for initial installation/setup

Hi There,
Context: I'm working with our infrastructure team to prove out an installation of the on-premise data gateway on a VM inside azure to serve as a hub for reports to access all private databases we host in our Azure tenant (Power BI service cannot access them as they have no public endpoints and the VNET Gateways are in Preview mode so we are looking at the on-premise data gateway as an interim solution).  No network communication is permitted outside of the specifications in this article and we are not able to configure the gateway.
When I run the software and try to sign-in to register the gateway I get all sorts of network/firewall/tls/healthcheck failures ... I think the document needs to be revised to scope the description of the configuration to "post-installation" or updated to include additional commentary and rules specifications regarding installation and the specific rules therein.
I'm still working with our infrastructure team to try to figure out how to get this to work but creating a landing zone and configuring a vnet with the rules you specified is insufficient (thus far).
Happy to show you screenshots and logs files of our issues if that helps you get a better idea about what the issue is and whether it's related to this document or not.
Cheers,
Fahim Kanji
<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f4daef9a-341c-696a-1d19-de05ffd67588
Version Independent ID: 7cd18c28-7227-488a-aa28-6b869058e715
Content: Adjust communication settings for the on-premises data gateway
Content Source: data-integration/gateway/service-gateway-communication.md
Product: on-premises-data-gateway
Technology: ****
GitHub Login: @arthiriyer
Microsoft Alias: arthii

EnterpriseGatewayConfigurator.exe Information: 0 : GetDataDirectory: 'C:\Users\PBIEgwService\AppData\Local\Microsoft\On-premises data gateway'
EnterpriseGatewayConfigurator.exe Information: 0 : Sign In
EnterpriseGatewayConfigurator.exe Error: 0 : Exception: Microsoft.PowerBI.DataMovement.GatewayCommon.EmailDiscoveryServiceException: Network request returned unexpected error. ---> System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.WebException: The underlying connection was closed: An unexpected error occurred on a send. ---> System.IO.IOException: Authentication failed because the remote party has closed the transport stream.
at System.Net.TlsStream.EndWrite(IAsyncResult asyncResult)
at System.Net.ConnectStream.WriteHeadersCallback(IAsyncResult ar)
--- End of inner exception stack trace ---
at System.Net.HttpWebRequest.EndGetRequestStream(IAsyncResult asyncResult, TransportContext& context)
at System.Net.Http.HttpClientHandler.GetRequestStreamCallback(IAsyncResult ar)
--- End of inner exception stack trace ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.PowerBI.DataMovement.GatewayCommon.ConfiguratorClientExtensions.d__8.MoveNext()
--- End of inner exception stack trace ---
at Microsoft.PowerBI.DataMovement.GatewayCommon.ServiceEnvironmentDetailsFactory.GetEnvironment(String emailAddress, Uri endpoint, String preferredCloud)
at Microsoft.PowerBI.DataMovement.GatewayUiCommon.WizardSignInViewModel.StartSignIn(IServiceEnvironmentDetailsFactory serviceEnvironmentDetailsFactory, ConfiguratorAzureGeo selectedGeo)
at Microsoft.PowerBI.DataMovement.GatewayUiCommon.WizardSignInViewModel.DoNext(Object param)
at Microsoft.PowerBI.DataMovement.GatewayUiCommon.WizardViewModelBase.DoNextWorker(Object param)
at Microsoft.PowerBI.DataMovement.GatewayUiCommon.RelayCommand.<>c__DisplayClass19_0.b__0()
EnterpriseGatewayConfigurator.exe Information: 0 : Update checks
EnterpriseGatewayConfigurator.exe Information: 0 : Create web request for Uri: http://www.msftncsi.com/ncsi.txt
EnterpriseGatewayConfigurator.exe Information: 0 : Create httpClient request for Uri: http://www.msftncsi.com/ncsi.txt
EnterpriseGatewayConfigurator.exe Information: 0 : Response status: 470
EnterpriseGatewayConfigurator.exe Information: 0 : Checking for PbiEgwService
EnterpriseGatewayConfigurator.exe Information: 0 : Fetch state for PbiEgwService
EnterpriseGatewayConfigurator.exe Information: 0 : Service state: Running
EnterpriseGatewayConfigurator.exe Error: 0 : Gateway service health check Network failed with error : System.Net.WebException: The remote server returned an error: (470) status code 470.
at System.Net.HttpWebRequest.GetResponse()
at Microsoft.PowerBI.DataMovement.Pipeline.GatewayCore.HealthCheck.NetworkCheck.WebRequestCheck()
at Microsoft.PowerBI.DataMovement.Pipeline.GatewayCore.HealthCheck.NetworkCheck.<>c.b__5_0()
at Microsoft.PowerBI.DataMovement.Pipeline.GatewayCore.HealthCheck.HealthCheckAction.Run(Func`1 checkMethod)
EnterpriseGatewayConfigurator.exe Information: 0 : Create httpclient request for Uri: http://www.msftncsi.com/ncsi.txt
EnterpriseGatewayConfigurator.exe Information: 0 : Response status: 470

@Teckfahimkanji thanks for the added context and information. For privacy reasons, it would be best if you work with support to report the details of the issues you faced while setting up that environment. If it's already too far gone, I understand its not worth your time.
I recommend that you edit your post and remove any personal information (email) to avoid over-sharing on the net.
Thanks, Jason


when using the option --default-credentials-userkey additional automation should occur

at the moment if the option --default-credentials-userkey is used with import openapi, the service will be created with anonymous access policy with the provided key value.
This is all good and fine until you try to test the service configuration which doesn't have any application plan created by default and most importantly no default application is created with the same key as the one provided in the option.
I think the command should cover also these 2 activities by default.

Definitely it makes sense what you ask. I will create Jira Issue.


Cannot read property $loading of undefined when using jest with nuxt component

When I try to test one of my components using nuxt and jest, I get the following error:
Cannot read property '$loading' of undefined

This is being caused by the following line of code in my component
this.$nuxt.$loading.start()

How do I prevent this error from occuring when running the test on my component?
The test file looks like this:
import { mount } from '@vue/test-utils'
import Converter from<EMAIL_ADDRESS>
describe('Converter', () => {
  test('is a Vue instance', () => {
    const wrapper = mount(Converter)
    expect(wrapper.isVueInstance()).toBeTruthy()
  })
})

This question is available on Nuxt community (#c8773)

I found a solution. The solution is to mock nuxt like this:
const wrapper = mount(Converter, {
  mocks: {
    $nuxt: {
      $loading: {
        start: () => {}
      }
    }
  }
})


why a custom Dendron Markdown Preview Enhanced

why do you install a custom version of shd101wyy.markdown-preview-enhanced ?
many of us have already the original version installed.. did you modify it?

The reason we needed to fork shd101wyy.markdown-preview-enhanced is to support Dendron specific features like note references. We'll be updating the configuration in a future release to live under a different namespace so it won't conflict if you have an existing version installed


Small error in {{elseif ...}} example

In the elseif example, we have the following:
{{elseif 5 > x}}
  <p>{{x}} is less than 5</p>

It should be 5 < x; I tried fixing this myself but couldn't figure out how to tell codemirror not to treat this as the start of a closing tag...

@larister
if x is 4
5 > 4
4 is less than 5

@proyb6 is correct 😀 The condition is 5 > x which is equivalent to x < 5 (but yep, the tag thing), which matches the sentence {{x}} is less than 5

I feel silly; I always write x > {condition} and my brain did the flip without me noticing. Thanks both.


does not work with stable dioxus

when I use use_ws_context_provider_text it gives the error expected struct "dioxus_core::scopes::ScopeState", found "&Scoped<'_>"
I downloaded it and changed the version of dioxus in cargo.toml and now it gives this error
error[E0521]: borrowed data escapes outside of function
  --> src/main.rs:56:17
   |
54 | fn Txt(cx: Scope) -> Element {
   |        --
   |        |
   |        `cx` is a reference that is only valid in the function body
   |        has type `&'1 Scoped<'1>`
55 |     use_init_atom_root(&cx);
56 |     let mut h = use_state(&cx,|| String::new());
   |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |                 |
   |                 `cx` escapes the function body here
   |                 argument requires that `'1` must outlive `'static`

@FruitieX  I have the same problem. Please, update your code.

I have given up using Dioxus for now. Too much work for my hobby projects when I need to build every UI component from scratch. Archiving this repo, feel free to take over maintenance work by forking.


woodpecker-cli misnamed in RPM and DEB releases

Component
cli
Describe the bug
woodpecker-cli's RPM and DEB packages install a binary called 'woodpecker' as opposed to 'woodpecker-cli'. The binary appears to work, but this is confusing as the examples (and tarball releases) use the name 'woodpecker-cli'.
System Info
Fedora 38 (RPM), Debian 11 (DEB)

Additional context
Fedora:
$sudo rpm -ivh https://github.com/woodpecker-ci/woodpecker/releases/download/v1.0.0/woodpecker-cli-1.0.0.x86_64.rpm
$rpm -ql woodpecker-cli
/usr/local/bin/woodpecker

Debian:
 $sudo dpkg -i woodpecker-cli_1.0.0_amd64.deb
Selecting previously unselected package woodpecker-cli.
$ dpkg -L woodpecker-cli
/usr
/usr/local
/usr/local/bin
/usr/local/bin/woodpecker

Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Checked that the bug isn't fixed in the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]
[X] Check that this is a concrete bug. For Q&A join our Discord Chat Server or the Matrix room.

In that case, should I change the report to 'standardize woodpecker-cli binary name' or something like that?

binary currently should be:
woodpecker         # cli
woodpecker-server  # server
woodpecker-agent   # agent

as it's handy for autocompletion in shell, I would argue ... but if the package name also should just be called that way I dont know ...


Add plugin to part of opensearch build

Follow this guide to include this as part of 2.2-snapshot
https://github.com/opensearch-project/opensearch-build/blob/main/ONBOARDING.md#opensearch-build

[x] Add it to opensearch-build/meta
[x] Add it to opensearch-plugins
[x] Add it to 2.2 manifest

Also, tracked here: https://github.com/opensearch-project/opensearch-build/issues/2346

dashboard-maps merged it to manifest. https://github.com/opensearch-project/opensearch-build/pull/2411, going to close it.


Add-Migration: System.BadImageFormatException -> System.Collections.Concurrent <IP_ADDRESS>

Good Afternoon for all.
Introduction
I have installed EF Core and EF SqlServer, version 1.0.0 from nuget package to work on Full .NET 4.6 application. But when I see the problem on Tracking entities failing on 'LoadAsync' I decided to move for nightly builds (alpha1-21973), but now I can't run the 'Add-migration' commands.
It's trying to load 'System.Collections.Concurrent' on <IP_ADDRESS> version, but I see on Dependencies that's requires the 4.0.12 version (already installed).
I don't know what I'm doing wrong.
The issue
Could not load file or assembly 'System.Collections.Concurrent, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies.
Exception message:
Stack trace:
System.BadImageFormatException: Could not load file or assembly 'System.Collections.Concurrent, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. Reference assemblies should not be loaded for execution.  They can only be loaded in the Reflection-only loader context. (Exception from HRESULT: 0x80131058)
File name: 'System.Collections.Concurrent, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' ---> System.BadImageFormatException: Cannot load a reference assembly for execution.
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceTable..ctor(IEnumerable`1 descriptors)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider..ctor(IEnumerable`1 serviceDescriptors, Boolean validateScopes)
   at Microsoft.EntityFrameworkCore.Design.Internal.StartupInvoker.ConfigureServices()
   at Microsoft.EntityFrameworkCore.Design.DbContextOperations..ctor(ILoggerProvider loggerProvider, Assembly assembly, Assembly startupAssembly, String environment, String contentRootPath)
   at Microsoft.EntityFrameworkCore.Design.MigrationsOperations..ctor(ILoggerProvider loggerProvider, Assembly assembly, AssemblyLoader startupAssemblyLoader, Assembly startupAssembly, String environment, String projectDir, String contentRootPath, String rootNamespace)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.<>c__DisplayClass5_0.<.ctor>b__5()
   at Microsoft.EntityFrameworkCore.Internal.LazyRef`1.get_Value()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.AddMigrationImpl(String name, String outputDir, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.AddMigration.<>c__DisplayClass0_1.<.ctor>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.<>c__DisplayClass3_0`1.<Execute>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)

Could not load file or assembly 'System.Collections.Concurrent, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. Reference assemblies should not be loaded for execution.  They can only be loaded in the Reflection-only loader context. (Exception from HRESULT: 0x80131058)
PM>

Further technical details
EF Core version: 1.1.0-alpha1-21973
Operating system: MS Windows 10 Pro 1607
Visual Studio version: VS 2015.3
Other details about my project setup:
WPF Class Library and .NET Framework 4.6.
DbContext configured with Fluent API and working fine on existing database.
Thanks.

Could you share your packages.config and App.config files as well?

Of course.
I'm trying to attach on zip format, but I receive this error: "Unfortunately, we don’t support that file type. Try again with a PNG, GIF, JPG, DOCX, PPTX, XLSX, TXT, PDF, or ZIP. " So I'm sharing a link from one drive:
https://1drv.ms/u/s!AAsFZ2XXayWeziY
Thanks.

System.Collections.Concurrent shouldn't be listed in App.config (possibly a NuGet bug) try deleting that
<dependentAssembly> element to see if it fixes the problem.
If that still doesn't work, delete App.config, run Add-BindingRedirect, and try again.

Thanks for the quick answer.
I have tried the following procedures:

Changed:

EF Triage: Closing since we believe we have answered the question. Please create a new issue if you run into new issue or reactivate/respond to this thread if something about the answer isn't clear.


Fix onRequestClose Modal warning

Fix onRequestClose Modal warning.
https://github.com/oblador/react-native-lightbox/issues/46

Hi guys,
It would be nice to close this one.

Please close this, it is pretty annoying to do android development and have to close this error whenever I open a component that uses Lightbox.

Thanks @oblador for merge!
Could you please release a new version with this fix? maybe 0.6.1


Add langchain component for Vertex Standalone Ranking API

Vertex DIY RAG APIs helps to build complex RAG systems and provide more granular control, and are suited for custom use cases.
The ranking API takes in a list of documents and reranks those documents based on how relevant the documents are to a given query. Compared to embeddings that look purely at the semantic similarity of a document and a query, the ranking API can give you a more precise score for how well a document answers a given query.
Reference : https://cloud.google.com/generative-ai-app-builder/docs/ranking
We implement this similar to other ranking APIs that langchain already integrates with - like Cohere Ranker

/gcbrun


Prolly fixed the VALIDHACKS

Might fix #1801

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/codegen/linearizer.py             2              13
------------------------------------------------------------
lines added in the tinygrad folder: -11

Tests are failing. You also need to add tests to highlight how this fixes the issue and catch regressions.

Ok, there seems to be some cases to check. I will reopen if I fix them soon enough


Dependabot Update Error

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
nuget
Package manager version
No response
Language version
net8.0
Manifest location and content before the Dependabot update
No response
dependabot.yml content

package-ecosystem: nuget
directory: "LOCATION_FOR_CSPROJ"
schedule:
interval: monthly
open-pull-requests-limit: 10

Updated dependency
No response
What you expected to see, versus what you actually saw
+--------------------------------------------------------------------------------------------------+
|                               Changes to Dependabot Pull Requests                                |
+---------+----------------------------------------------------------------------------------------+
| created | Microsoft.Azure.Functions.Worker.Extensions.Timer ( from 4.1.0 to 4.3.1 )              |
| created | Microsoft.Azure.Functions.Worker.Sdk ( from 1.17.0 to 1.18.0 )                         |
| created | Microsoft.Azure.Functions.Worker.Extensions.Http ( from 3.1.0 to 3.2.0 )               |
| created | Azure.Storage.Blobs ( from 12.21.2 to 12.22.1 )                                        |
| created | Microsoft.Azure.Functions.Worker.ApplicationInsights ( from 1.2.0 to 1.4.0 )           |
| created | Microsoft.Azure.Functions.Worker ( from 1.21.0 to 1.23.0 )                             |
| created | Microsoft.Azure.Functions.Worker.Extensions.ServiceBus ( from 5.16.0 to 5.22.0 )       |
| created | Microsoft.Azure.Functions.Worker.Extensions.Storage.Blobs ( from 6.3.0 to 6.6.0 )      |
| created | ClosedXML ( from 0.102.3 to 0.104.1 ), DocumentFormat.OpenXml ( from 2.16.0 to 3.0.1 ) |
+---------+----------------------------------------------------------------------------------------+
|        Dependencies failed to update         |
+------------------------+---------------------+
| DocumentFormat.OpenXml | update_not_possible |
+------------------------+---------------------+
Failure running container [container-id]
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

Could you provide some more information about what you're expecting to see?  Along with that, do you have a full log you could share?

I was not expecting to see something like this:

I think there must be some dependencies among the modules.
Logs:
0_Dependabot.txt

I was not expecting to see something like this:

I think there must be some dependencies among the modules.
Logs:
0_Dependabot.txt

@baynezy Thanks for your response!!
<RestorePackagesWithLockFile>true</RestorePackagesWithLockFile> is no where specified in my .csproj.

Thank you for attaching the log.  Line 2904 reports the following:
Package [DocumentFormat.OpenXml] could not be updated in [/home/dependabot/dependabot-updater/repo/<Project-Name>/<Project-Name>/Utility/Utility.csproj] because it would cause a dependency conflict.

The issue is commonly with transitive dependencies and/or shared dependencies with other packages that would need to be updated in parallel.  We've been experimenting with how to properly solve these, but there's still more progress to be made.

@brettfo Thanks for your response!!


shiftwidth=0 does not work

Hello,
setting shiftwidth=0 (it's supposed to use the tabstop value then) breaks the plugin. The foreground of the very first column is then in a different color which looks kind of strange and is unreadable (for me) because it is almost the same color as the background.
I've replaced set sw=0 with let &shiftwidth=&tabstop in my .vimrc and now it works again. However, I'm wondering why it won't work the sw=0 option as the help page (for sw) says.

I was able to replicate locally - the associated PR has been (manually) merged.


[SRJIRA-6939] Proposal of new getValue method return value signature.

This is not based on a known bug, but I found odd this was the only method not accepting Promises.

This is not based on a known bug, but I found odd this was the only method not accepting Promises.

Thanks @MaxTheBean , I've incorporated your proposed changes into the main branch, since now we're supporting Promises for getValue() as well


Incorrect highlighting when line begins with ~ (backtick, tilde, backtick)

Beginning a line with the sequence `~` causes the entirety of the rest of the file to be highlighted as if it were inline code. Take a look (image is cropped at text wrap margin).

Thanks for reporting, I'll have a look at it. I think I know what the problem is.

I can't replicate this issue, or well, I found another issue when trying to replicate it, but I can't replicate your issue. See this screenshot:

Could you first try reloading Atom (packages sometimes behave unexpected, for unknown reasons), and if that doesn't help, could you paste the entire document in which you had the error in a Gist or a Pastebin? Maybe there's something else in there that's causing the issue.
And thirdly, could you try removing the sh on line 83, just to see if that makes a difference. There's a longstanding issue that some embedded grammars do not exit properly, and I just want to take that out of the equation by treating the code-block as generic source code.

Reloading packages did not work (ctrl+alt+r on linux).
Gist here.
removal of sh did not work.

Let me know if you need anything else.

Thanks. Can you confirm that you are using the latest version of Atom (1.10.2) and of language-markdown (0.16.3), and that you are using this package and not accidentally language-gfm (it should say Markdown in the bottom right-hand corner of the window, not Github Markdown)?
I'm also not able to reproduce the issue with the data from your gist. Are there by any chance other Markdown-related packages you have installed that might interfere? And what syntax-theme are you using? (I don't think that is an issue, but it might help with replication)

Aha!  I was using language-gfm and not your language-markdown. Yours works as expected :)

Yay! I like me some problems that solve themselves! : )


Import "X" could not be resolved - Correct interpreter, lib installed, searchPath is there

Environment data

Language Server version: v2022.3.3
OS and version: Win 10 Pro 21H1 19043.1586
Python version (& distribution if applicable, e.g. Anaconda): Anaconda3 Python 3.9.7

Expected behaviour
virtual conda environment is used, correct interpreter is selected. "requests" is installed in this environment.
"pip show requests"  gives "Location: C:\Users\Sebastian.Schleicher\Anaconda3\envs\zoho\lib\site-packages"
Output of python -c 'import sys, pprint; pprint.pprint(sys.path)' is given below.
pylance should be able to resolve the import.
Actual behaviour
pylance cannot resolve imports. Running the code works fine however.
Important info: Adding the following line to settings.json solves the issue:
"python.analysis.extraPaths": [ "C:\Users\Sebastian.Schleicher\Anaconda3\envs\zoho\Lib\site-packages" ],
output of "python -c 'import sys, pprint; pprint.pprint(sys.path)'" does not change however.
Logs

XXX

Code Snippet / Additional information

python -c 'import sys, pprint; pprint.pprint(sys.path)'
['',
 'C:\\Users\\Sebastian.Schleicher\\Anaconda3\\envs\\zoho\\python310.zip',
 'C:\\Users\\Sebastian.Schleicher\\Anaconda3\\envs\\zoho\\DLLs',
 'C:\\Users\\Sebastian.Schleicher\\Anaconda3\\envs\\zoho\\lib',
 'C:\\Users\\Sebastian.Schleicher\\Anaconda3\\envs\\zoho',
 'C:\\Users\\Sebastian.Schleicher\\Anaconda3\\envs\\zoho\\lib\\site-packages']

Thanks for the report, unfortunately, this is not enough data for us to debug.
If this continues to be a problem, would you please reopen the issue and add the information in the report as described in the troubleshooting guide?